All of these comments are suggestions of how this could be done differently, not asking them all to be modified.

This query is not correct, in that it depends on the order by which calendar_dates are added. Example: Let's imagine for a moment that we created your dates in reverse, or out of order. Because you a querying by calendar_date_ids, you might end up with any number of dates in your range

```Ruby
@holiday_dates = HolidayDate.where(country: c, calendar calendar_date_id >= ? and calendar_date_id <= ?", c, filter_cal.first.id, filter_cal.last.id).paginate(page: params[:page], per_page: 8)
```


Why? `not in` is the usual Python idiom.


buenisima la textura del barco


Ahh thanks I finally get it. So you need to do `jspm install github:PureMVC/puremvc-typescript-multicore-framework@1.1` explicitly because it won't be detected as the latest version as it isn't a valid semver. I've uploaded the fix for this in https://github.com/jspm/registry/commit/62e098bd342f63c4d91e88953fb9c0a63d677e15.


"The issue with FedoraLdp.updateSparql() is that it already has two catches (the first catch is relying on instanceof;": I guess that's why I'm wondering about a more precise exception. Wouldn't that get rid of the need for `instanceof` and by factoring out at least one exception, get rid of the need for nested exceptions? But I may very well be misunderstanding this.


@imaus10 Please update `br -help`


Meh, then it is a core bug or something else, my bad.


At the following line:
https://github.com/fcrepo4/fcrepo4/pull/811/files#diff-4a14fb4dcbde6f7d32274c43ca723c16R369

Could you just add: `catch (final IllegalArgumentException iae)`?


I made a helper method that will calculate the savings. I iterated through the array of meters that was found for the city, and depending on if the utility is electric, gas, or water, I push the number of dollars saved into an array. Then, I add up each of the numbers in the array for each electric, gas, and water savings, and I add them all together for the total savings.


why are we using `uint8_t` here, btw?


I think the next 40 lines would be clearer if consolidated in a `for` loop, since it's the same code with just the filenames and text descriptions changing (right?).


When deleting text from the C++14 standard (as opposed to text unique to the Ranges TS), use `\oldoldtxt` instead of `\oldtxt`. That way, when it's time to fix up the markup, I know to replace `\oldoldtxt` with `\removed`, and stuff marked `\oldtxt` can just be deleted.


Is this import necessary?


Ah. yes, good. I was following intellij's.  so I will change the name.


Just `print()` will do the trick.


You're right I was wrong doing this. We should have two keys. One for the restart case and one for the update case.


Actually, this table *is* used when the setting store_session_in_db is set!
Unless there is another change linked to this (like replacement by Doctrine or so), this will have to be reverted.


The issue @timmywil links to suggests the generic (no restriction approach) caused issues in Chrome packaged apps. There may be ways to detect that while leaving the generic approach open to most environments. After all `onunload` is a pretty webby thing.

Some ideas could be seeing if Chrome packaged apps have `'onunload' in window` as `true` or if it errors on just assigning the event listener (then you could just wrap it in a try-catch).


Can you handle it?


In the Nexus spec, whitespace separators are to be ignored, and the SYMBOLS command values must be quoted. I agree, though, that all whitespace rather than just spaces should be removed. I'll get to it and the other particularly helpful suggestions you made.


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

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


Of course. I was trying to avoid having another catch block for readability concerns. But yes, that would clean up FedoraResourceImpl calling code.


I decided to create pull request, since I already have required patches: #720


My personal attitude is that you can have as many `catch` blocks as you like, just comment them. That's just the way Java is and there's not really a better idiom. (Since Java lacks a Success-Fail monad or true union types or any of the things that would provide better idioms.)


Ah, helpful answer, thank you!


The Fortress will often be an outpost, won't it?  Which would make this targetConstruction contribution zero.  I suggest something like what I am about to suggest for the shield regen techs:
```
SetShield value = Value + max(min(5.0, Value), 0.25*Target.Construction)
```


We used to do just that, but then jquery/jquery/issues/1772 happened and led us to b49720582b3e39309c4d762bf2aaacdecd66e51e.


I'm not sure why you are mentioning auto-increment here.  This is used to read the slave starting from the specified register.


There is no need to check if it's empty or nor since the for statement will do it anyway.


include client/app/shared to be concat'ed

maybe rename the destination file to be app.js


length


Check how this is used _despawnedSporesCount = 0;


> Typo.

Fixed, thanks.


@jjfarrell ICD is covered in ontologies; ICD11 is semantically structured with ontologies in mind. (Actually not sure about ICD-O? Forgot...)


Have you removed ```analogReference()``` for a reason? If the user call this function in his sketch this will result in a linker error.

Also, I think Readme should be updated to point out the ESP.readVDD function will not work after this change.


> We used to do just that, but then jquery/jquery#1772 happened and led us to b497205.

Yap I get that. I think that since a Chrome packaged app seems niche compared to IE/Edge support and that since `onunload` is webby compared to something in an application scenario it makes sense to detect Chrome or its lack of support for `onunload` than it does to attempt to restrict to IE.


dfsdf


Yes, it is hardcoded, but there would be two hardcoded variables for the same thing. We use VFAT_RESULTS_BLOCKS to create the results partition via mkdosfs while VFAT_RESULTS_SIZE is used to calculate the disk image. Thus, if we ever grow the results partition, both variables need to change. We could control everything with a single VFAT_RESULTS_BLOCKS variable.


`long lived` => `long-lived`


I believe that the developer can read the code faster if you read 8M rather than 8388608.


worth noting this will read the ***same*** block multiple times.


`print()`, I think, will print `()`. However, `print` by itself should output just a newline, which should be the same behavior as `print('')`. I think I do the later for aesthetic pleasure.


change the final `or` => `and`


"At release Telegraf..." may make more sense as "As of today, Telegraf...".


Couple nits, but it's a nice read!


Nope, it will die. I copied it from a separate file that I was doing testing.


The code deployed to staging has the `permittedCrossDomainPolicies` entry removed.  But it is still supplied by the server with the default value.  (Run `curl -i https://webservices-staging.sagebridge.org/` and look at the results.)  I though it would be more explicit if we also specify it in `application.conf`.


Good point ajs6f. I'll redo this.


Thanks for the clarification! if you add your files to the boot partition, the increased size should be already accounted for. However, the changes you propose touch only the results partition.


I would add a config statement, such as "DataType" for this thing, so that it's not hardcoded in the code (I can do it later). Basically, make the c_datatype variable conifgurable.


I had considered that, but I think testing readability / comprehension should trump brevity here.


Hi Tom

First of all I would like to thank you for such a great plugin!

1. Yes, it compares the previous year at the same time (month + day).
2. No idea, this is my first fork on github :)
3. It works only for the ride activities but it can be change easily to support running activities

I would like to do some stuff:
 - optimize the comparison because for now it always download all statistics from previous years
 - charts for the comparison to previous years (there is such a graph on VeloViewer but why should I pay for it when I can make it by myself)
 - additional statistics for segments on a ride like comparison to the best result

Maybe my work will be good enough to merge it to your develop branch :)

regards


Ok, don't worry for this, it'll be done soon (ASAP).

Let's stop the hostilities.


One small edit, otherwise LGTM!


Subi dois modelos para os testes de conferência de valor.

O primeiro busca os valores na tabela do demonstrativo incluída no pacote.
O segundo insere os valores diretamente no script de teste.

Estou preferindo o segundo, por ser mais simples e fácil de ler.
O primeiro retorna erro se usarmos caracteres especiais (como acentos), o que requer o uso de expressões regulares para funcionar.


Unfortunately, a I can't get a ruby version of 2.2.1 yet (not released on rbenv yet, 2.3 is out but not 2.2.1), but this might be rewritten as:

```Ruby
HolidayDate.where(country: c).calendar_dates.where(date: [filter_cal.first, filter_cal.last])
```


*readme


nothing to say


@brantburnett  -

Sounds good. Ideally make each of these features a separate change-sets (commit or set of commits) and pull request so they are easier to review and merge. As it stands now their are two separate change-sets that I see here: Where Filtering and ANY.

Great work!

-Jeff


Haha looking forward to the rant. Didn't put 2 and 2 together with your "wip" and merged `user-story-3` into `master`, but I have `user-story-4` pull request with `master` still open. Work on that branch or `master`?

![giphy](https://cloud.githubusercontent.com/assets/6549337/8259905/dc65c2dc-1685-11e5-85e6-ebc192cc93a5.gif)


KVIRC also has an option in the connection settings to send commands once
connected, despite the recent changes it has been very easy to adapt.
Simply add one or two lines to your "on connect" dialogue box and you're
good to go.

On Fri, Jun 19, 2015 at 6:46 AM, Andrew Bashore <notifications@github.com>
wrote:

> Last time I used HexChat there was an options in the profile settings for
> the server to send a command once you are connected which is not really too
> much extra work.
>
> On Fri, Jun 19, 2015 at 6:11 AM, Matt Gajownik <notifications@github.com>
> wrote:
>
> > ..and so everyone using a regular IRC client can no longer see who is
> > online because it requires a *custom capability only Twitch IRC
> requires*?
> > This is *frustrating*. The previous method worked just fine.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/justintv/Twitch-API/commit/71c17ae05d29aa520aad73a1434bf2aa8c94e392#commitcomment-11764909
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/justintv/Twitch-API/commit/71c17ae05d29aa520aad73a1434bf2aa8c94e392#commitcomment-11765389>
> .
>


This break master


Heard about [list comprehensions](https://docs.python.org/2/tutorial/datastructures.html#list-comprehensions)?
[this is  good](http://howchoo.com/g/ngi2zddjzdf/how-to-use-list-comprehension-in-python)


Thanks :-)

ps. my new email is `jpaulson@jimmy.harvard.edu`


YAY!!! I like the use of glyphs, they are super fast vs images


Answers my holiday API question


Let's either revert this or push a hotfix ASAP.


Is this a hash of a specific search you made?


There is really no need to point fingers. If you have a problem with a particular commit or think someone made a mistake, please just bring it to the table and discuss the matter itself; there is no problem at all in making a mistake, we all learn from our past, and we can always fix things as long as we communicate properly, just like this one, but please just do not take anything personal.

I would like to remind everybody that this is an open community, and any contribution from anybody is always welcome, and that has nothing to do with whether you have write access to the repo or not - this was why we moved to github in the first place. And let's keep it a cool and healthy community. Most of us are just doing this as a hobby so there is no need to get mad at someone who did or committed something you did not agree - just bring it up and we can all discuss and find the best solution, this is the way we move forward for the project.

And last thing: since I gave @antenore the administrative rights, I will of course respect his administrative decision, but on the other hand, keep in mind that giving someone access does not make him a contributor; or revoking someone's access does certainly NOT remove him from the contributor list or even the copyright of the codes he wrote. So please do not get upset about those trivial things; if anyone is still caring about the project, please just keep discussing things and submitting codes, and your contribution should be recorded in the system and relevant files and everybody who uses the project will be able to see it - and it you think otherwise just bring it up and communicate, and you are always welcome!


IMO, this warrants an editorial comment about why this exists, although if we have an appendix or a separate paper, that's good enough. Also, this is *not* a value concept, so maybe it's in the wrong section?


I modified a lot of the code last year when I was trying to get a handle on the parser functionality for debugging purposes. This was an inadvertent change as was much of the whitespace differences, which is why I put "cosmetic changes" in my pull request description.


Here user match is dependent on sb1 and sb2,
Instead it should be dependent on vars either passed, or some internal object variables.
so that you can use the same function in different case.


tested and working


I would just delete this editorial comment.


It will return.


It is good practice to always catch the specific error here.  It is a practice I am still getting used to as well but I am trying to do it as well.  There are numerous explanations out there about that if you want to see some of the general why bother conversations out there.  Like I said, I'm still trying to do that as well.


@gibson042 Are you able to test out some of the approached [I mentioned earlier](https://github.com/jquery/sizzle/commit/28f4c67c508719cc76e2075dce0e4cbe0e325919#commitcomment-11772907)?


I would have used it for the "show" and "edit" buttons, but I get an after image when I tried implementing it.


This was a blunder on my part. I am reverting this section back to the original.


Aha! I was lost in trying to get `\removed` and `\oldtxt` to interact properly. I'll fixup this and `unique_copy`.


Or just `boy = Date.today.Date.today.beginning_of_year`


Here you have statically coded the dockers connection which is something you fixed in previous files.  That said, I have an idea I'd like to pass by you that might make this much easier without a dockers connection.  If you can hold up, we can discuss Monday when I return from PTO.


I would wrap this whole section in an `\oldtxt` or `{\color{oldclr} ... }`.


The comment above is editorial in nature, not normative. Just make it part of the preceding `ednote`.


Looking back at this again, it was probably needed for the cross compile case. I'll test with and without this when cross compiling for a raspi.


Looks like this assumes docker is already installed (which I assume is the case for GCI?). It would be nice if we could make this work even when docker isn't installed by default - otherwise I think it won't be as easy for people to use if they have to build their own image before running this.


we should not hard code this path.
In general its not a good idea to hard code anything. So this can be stored inside a variable in the class init method itself, or (https://docs.python.org/2/tutorial/classes.html#class-and-instance-variables)[class variable]


I can envision a problem with having a `('"', "'")` tuple in these cases. For example, say the enclosing quote style is a double-quote, then a single-quote would be an acceptable symbol but would cause a parsing error.


Add the project to the Ceylon Model also if the `IJavaProject` is not found.


admin


Add the project only if the Ceylon nature is enabled. Then think at adding it also when enabling the CeylonNature on an existing Java project.


Or:
```Ruby
filter_cal = CalendarDate.where(date: [boy, eoy])
```
The reason why I am nit picking is that using the sql strings are not as interchangeable between different versions of databases. For example: active record will be able run the command above `CalendarDate.where(date: [boy, eoy])` on almost any relational database, given the correct adapter. That is not necessarily true of `CalendarDate.where("date >= ? and date <= ?", boy, eoy)`.

Does that many any sense?


Your'e awesome man.. thanks for continuing this!


`\oldoldtxt` just forwards to `\oldtxt` -- there's no semantic difference. It's only to make my life easier later. I didn't think to use it before, so this last round of edits was extremely painful.


Suggested from #83


here too try not be dependent on sb1 and sb2 instead pass variables


dmillerw the original code didn't work because your if statement limited the growth to either the nether or light level above 9. But in infernal skies light level in overworld never goes above 8 (right next to a glowflower).

I know this fixes the immediate problem, but maybe a future revision would allow light sensitivity to be configurable?


What is the difference between using tabs and spaces?


This is new and I do not see it called anywhere.  Is it needed?


Not for at least a week, and even then I'm not sure. I do like the ideas, but I still hate this whole stupid block, especially when everyone has to suffer because of IE (and now Edge).


Changed Game class to match new AI changes


Semicolons are not necessary, Babel is clever enough to add them for you. :smirk_cat:


@jeffrymorris,

Would you like me to send you a pull request for the where filtering?  Or
is there some other refactoring on that part of the work that you'd like to
see before I do?

Thanks,
Brant

On Fri, Jun 19, 2015 at 2:19 PM, Jeffry Morris <notifications@github.com>
wrote:

> @brantburnett <https://github.com/brantburnett> -
>
> Sounds good. Ideally make each of these features a separate change-sets
> (commit or set of commits) and pull request so they are easier to review
> and merge. As it stands now their are two separate change-sets that I see
> here: Where Filtering and ANY.
>
> Great work!
>
> -Jeff
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brantburnett/Linq2Couchbase/commit/431c6fc0671ac3baab8d79698ff74f6064dc0fd4#commitcomment-11773165>
> .
>

--
Thanks,
Brant Burnett
CenterEdge Software
(336) 598-5934
http://centeredgesoftware.com


Yea, I'd prefer to limit this unload handler to as few browsers as possible.


Can we plz get use out of fishing poles, minecart, boat, and rails PLZ.


I thought that using ``attr_reader`` will allow us to call ``height`` and `orange_count` on an instance, it just won't allow to set up a new value


Thanks but http://blog.izs.me/post/2353458699/an-open-letter-to-javascript-leaders-regarding :)


I actually started making local changes like these, so that I could call into some of these requests directly (like using `requestNotifications()` on a button in one of my apps to turn on notifications). But the problem I ran into was when the permissions were disabled or unauthorized. So what I did locally was modify `showDeniedAlert()` and `showDisabledAlert()` to take in an optional parameter:

    presentingViewController: UIViewController? = nil

Then the last line in each method changes to:

    (presentingViewController ?? self).presentViewController(alert, animated: true, completion: nil)

Then instead of just calling `requestNotifications()` directly, I first check for `Unauthorized` and `Disabled`, and if found, call into the now-public permissions scope windows with a reference to my active view controller. e.g.:

    switch PermissionScope().statusNotifications() {
        case .Unknown:
            pscope.requestNotifications()
        case .Unauthorized:
            pscope.showDeniedAlert(PermissionType.Notifications, presentingViewController: self)
        case .Disabled:
            pscope.showDisabledAlert(PermissionType.Notifications, presentingViewController: self)
        case .Authorized:
            // already approved
    }

But this is obviously kludgy. Is there a way that requestNotifications() (and all the rest of the request functions that are now public) might be able to detect what view controller called them, and pass *that* into the showDisabledAlert/showDeniedAlert methods, rather than them using `self` automatically as they are now?


Awesome!!!


Anything involving filesystem I/O should probably test against multiple node versions


Should we be forward'ing `a` here, too?


I know the names of the schemas are a little arbitrary.  But once we settle on some patterns, let's update (or replace) the diagram too
https://docs.google.com/drawings/d/1szHIpgnvlCM0CHlzMx9Ayi8S9_E-DmeNNAT9MAH3I-Q/edit


Also umcommented something in the YAML conf file so it compiled without errors @licks92


Doesn't adding an unload handler defeat the bfcache on all browsers? If so we should definitely limit the damage to IE/Edge but that is a pretty heavy penalty to bear.


For clear indicators of potential failures, we can run the "client" and "node" tests as separate tasks with in the travis job


something probably needs to be done with the leaderboard for one player games, or anybody could just get free points.
one option would be to just disable the leaderboard if nobody joins after 2-3 rounds, so that it still counts people who join late.


I didn't print it because the version of the function that takes a hash wasn't printing it. That doesn't really make much sense, and we're protected by `BEAST_DEBUG` so I'll run a quick test to see how bad the output is, and either always remove the condition, or  remove the print entirely.


MAJOR


tested and working


90664130_AlternativeModelPAT (macumber) - Win64-Windows-7-VisualStudio-12: OK (2194 of 2213 tests passed)

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


Sorry @owndroid I remembering doing a git reset before pushing updates

😤😤😤 stupid git
Please delete the blah.patch aswell


What will happen if the user enters alias as 'Username'?


Should be "If \tcode{U} is an **lvalue** reference type ..."


I think `options` when it is passed to `_format` will be missing values for the SYMBOLS command when there is whitespace. I will have to look into this, since I can't recall what was going on. I wrote this part last year and need to reacquaint myself.


I missed it when reviewing this, but you guys assume here that all shared variables of the computation graph are parameters... which is quite a strong assumption. Accumulators of aggregators are perhaps the most frequent exception.


@bitprophet I'm laughing because that's totally something I would have done.


UIApplication.sharedApplication().delegate.window!.rootViewController ? :/
Or add a parent, optional, parameter in the *show* function


CC @CvX


I don't really see what you uncommented :/


Do you happen to know who the official maintainer is of this project?  If there is one, I can turn over the root fork to them instead of keeping it around myself.


The last sentence is editorial.


it's simpler (and more efficient) to use `ocaml-hex` to do this


It's pretty much how we're indenting long line wraps everywhere now. eg. https://github.com/ripple/rippled/blob/develop/src/ripple/overlay/impl/PeerImp.cpp#L208-L212


Heh, I ran into problems with ".." too. Not that it's a good solution and probably not particularly relevant any more but: https://github.com/kanaka/clojurescript/commit/c1b628793c2b0c8ec66fe5a0195658ec496a10ea


DesignDaysGridView (evanweaver) - 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.8%25-red.svg)


What should this concept do when passed a cv or ref qualified type? Ditto for the others.


Hmmm never mind, I think I was conflating the 2 variables.


Oh this is super cool! I've been meaning to write another one of these to get a decent fat driver in place, and you've beat me to the punch. Awesome! Ping me if/when you want me to merge it in.


`<nit>` inconsistent `{` placement.


Yes that does make sense. I think the string was easier for me to see the relationship. But, it is more practical the other way.


Strike this last sentence.


HelloWorld


You could shorten this line to update `@current_user` only if it's `nil` using the `||=` operator:

`@current_user ||= User.find(session[:user_id])`

That will set `@current_user` to `User.find....` only if it's `nil`.


FWIW here's an excerpt from the comments of some [ALSA ChangeLogs](http://www.alsa-project.org/main/index.php/Detailed_changes_v1.0.27.2_v1.0.28) regarding the (no longer) use of `PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP`:
```quote
    - conf.c: use portable way to initialize recursive mutex
    PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP is not in POSIX, as _NP
    (non-portable) suggests.
    exposing such a symbol in musl libc would lock in the ABI for all
    times and makes it impossible to do future changes to the under-
    lying struct without hideous symbol versioning hacks.
    use the portable way instead: pthread_once was designed for such
    cases.
```
This probably means a patch for opencv could or should be devised that uses `pthread_once()` instead of how the code does it now. I haven't looked yet, so I may be totally wrong, though.


Works for me.


I like that you put this in a helper!


Those lines wouldn't go here. You would add these as a PHP role in the salt-states repository, and then define a php role here.


Just delete this editorial note.


I think this would be a good place for an editorial note directing readers to the appendix or paper where we explain the redesigned core concept hierarchy.


Completely missed that change in characters....


What do `k` and `v` represent here?


The which call worked for me on Windows. I'm guessing this is tripping up Windows.


I didn't seem to need the latter? I'll open a PR shortly, should I make that second change too?


I put it in under the assumption I would need it for `tagged_tuple`. When I actually *read* the `tagged_tuple` spec and realized it was unnecessary, I left it in as a response to the inevitable reviewer critique "But if `Constructible` requires object types, how do we constrain `tuple`?"

I'll be sure to give it coverage in the rationale appendix. Since it describes the union of "Constructible value-semantic object type" and "bindable reference type" I would call it a *partial* value concept. There's certainly no more appropriate section now, although that may change in the future if we develop other partial value concepts.


## This is a markdown file


I'm not sure to understand in which case you need to do merge the packages in `file` and the ones in `state.installed`


I have question for you please respond @RaySinlao


This query on `Song` should probably belong in an action, or a helper if it's needed in more than one view. Something like `@songs = Song.order(id: :desc)`, which you can then use in the view.


Fails to build...
My guess, from the Dbus changelog: "Automake 1.13 is now required when compiling from git or modifying
  the build system", is that it really needs that exact version (my default automake is 1.14, but I do have automake-1.13 around, with that name).
Commenting out the autogen part in dbus.mk does allow the build to succeed.


changed the link value from the start of the line to anywhere in the line. Fixed link detection in sentences.


This doesn't address the possible windows Path.


-1 Utiliza shortcuts:
border: 1px solid #c9d3dd;


It *is* part of that note.


Strike this out-of-date editorial note.


@gmcculloug
It is needed for the next PR which checks if a particular object is a state machine so that it can save the state variables when it goes to the next level


Nope. That's just a fluke of the diff engine.


This URL is fine, but conventionally you'll see URLs map to resources in a REST-ful way; so for example, a RESTful way of describing this particular route would be `song/new`.

You'll see more of REST next week when you're learning Rails!


As far as I can remember I just offered a pull request. Anyway, today I created an updated pull request on the current source code.


I'm just toying around, I don't know how those lines interact with `base: *` and trying to figure it out. I'll make a PR once I've got better bearings.


I don't like leaving the ugly formatting, but it really does simplify the delta. Boo.


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

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


s/@\\added\{\\oldtxt/@\\oldtxt/


OK, I will add the applicationId column and let you know when it's ready.

I will also try to index metadata in the search engine.


Oh yes, there is a /much/ more appropriate way to do that ;)

The root cause for the problem is that you're coming from an hrm_human_resource view, but the components you want to see inline are of the pr_person table - yet pr_person doesn't have a forward link to hrm_human_resource because it's actually the master table!

Or, in other words: what you're trying to do is to inline one component (e.g. pr_contact) into a form of another component (hrm_human_resource) of the same master table (pr_person) - and that isn't actually what inline-components do - for a reason:

This kind of double-reversed link (component => master <= component) contradicts the model constraints and can therefore not be supported generically: it is easy to see that it would go terribly wrong if the master record didn't yet exist. That is even worse if there is additionally a super-entity involved!

The obvious solution is to not come from an hrm_human_resource view ;)

By the time when we introduced the HR model, we didn't have inline components - not even anything similar. That is why we implemented an hrm_human_resource view with an S3AddPersonWidget -- instead of doing it the correct way around and implement a pr_person view with the hrm_human_resource details inline, which would just work, without /any/ workarounds.

The problem is that since then, the HR controllers have grown in complexity beyond the point where we could resolve the issue with any reasonable effort - and *therefore* we do need these workarounds here and there.

Taking it by cost vs. benefit, though, these hacks /are/ the best solution. They may be ugly, and yes - confusing at times - but that is nothing compared to the costs of a "proper solution" for this case ;)

I'd rather praise your efficiency here than to call your workaround "bad" ;)


fuck


Do we want `Assignable<T&, U>()` here?


package_dir, packages, install_requires are all things I was not able to figure out.


it's not much necessary. it looks like it is just used where we try to describe the jl_value_t to DWARF.


We have constants for that


Minor nit: check for FI_SUCCESS, not 0.


Published to http://ui.edev.d2l.com/86_b77f0f72e44771d7687972d4861b7dbdd48fb307


Agree - removal might be best as I am forgetful.


Same as above -- check for FI_SUCCESS.


Space check


When you run the startup script, it takes the full path of the executable, determines if it is run from the install base (line 9 is the regex checking that it ends with /bin/openbazaar). Determines the install base (line 10 replaces trailing /bin/openbazaar with /share/openbazaar). Then it changes curdir to there and runs python. In case of no install base it assumes development mode (line 13). And in dev mode it looks for the install in the current dir.

This is the standard practice to have the shell doing this, and maybe even more. You can't easily do this from python, because this shell finds python executable to run. A bit of chicken-n-egg problem.

The reason I added this PR was that the script was bash-specific and not as nicely structured. Knowing shell, this actually looking very good to me.

So I doubt this should be changed.


Nice speedups. :)


Ok I ported this to straight python so hopefully it maintains all the intentions of this script.


Thanks for doing this.


I don't this we should report this error here - I think handling it up above (where this is called) is more appropriate.  Generally we don't log and return an error except in very unusual cases.


Whatever, I just don't see the reason or point.


whitespace between `there` and `::`


checkout the work in the 14.04support branch and the recent commits to the salt-states repo. That should give you some bearings. :+1:


For Windows users so that its compatible.


Just looking at this code in relation to the other PR re the UMD loader.  The connect() function on Host returns a promise, however that promise may never resolve, and is never rejected either.  Does that seem a bit odd?  I didn't the consumer using that promise, but presumable they might use it?


this is a bad joke.


Why should hwnd be optional?


Here if none of the storage account has the prefix '%storagePrefix%' then we create a new storage account. Is this the behavior we decided or should we pick one of the existing stg account?

we will ask Khalid and can make change later based on feedback.


Keep the latest comming


How are you launching meteor?


LGTM, [ will confirm with Khalid on stg account] but good to merge now.


While I understand that this isn't a bootstrap issue directly, isn't it arguably grounds for a minor version bump rather than a patch? Totally broke our builds using `~3.3.4`.


I think it's unnecessary


Ooops look like I missed something. I meant to change them all to:
```
if (ret) {
        /* stuff */
}
```


+1 to 4.1.


THIS SHOULD BE MERGED BY NOW :P


Hi pixiez, can you say something more to the project. Or you just track down all the examples provided by Pyomo?


os prints do testing.py estão desatualizados, linhas 12 e 15


looks good.


Looks like this fun expected jarName without extension


Looks like there is a typo.


I think this is the appropriate place to require copy independence a-la ericniebler/stl2#38. Do you want to take a stab at wording for that, or should I? I was planning to adopt the axioms as described [here](http://www.stepanovpapers.com/DeSt98.pdf).


You can use a wildcard, 127.0.0.1:* to connect to the next available port, then unbind it:
you can use pull.getsockopt(zmq.ZMQ_LAST_ENDPOINT) to get the address, then you can re-bind.

This will make it so the test does not fail if something else is on port 5556.


I assume you meant in the *request* function, not *show*, since if you are calling show(), the PermissionScope view controller is displayed, and then the denied/disabled methods work fine.

So perhaps an optional view controller could be passed into the requestNotifications() methods, and if there are any errors, those VCs get passed into the showDeniedAlert/showDisabledAlert methods?


I don't see any 'new' function in there


this is an awesome joke :)


The other thing I haven't figured out about PermissionScope yet, when using it like this, is how I can easily detect when the request is finished.


Normally it would in Python 2.7, but we do `from __future__ import print_function` at the top so it mimics Python 3 behavior. Just `print()` should do what you want here.


We tend to import libs on the top of the module, not in functions.


Maybe just

```Ruby
["US", "BE", "BR", "GB", "NO"].each do |c|
```


Yikes, do do need that?


Hi mouts,

```php
if (substr($file, 0, 1)  != '.' && is_dir(...) && is_file(...){
$themeArray[] = $file;
}
```
also ignores the .htaccess file there.

I wonder why I didn't stumbled over it yet...


Good point.


I just added a jQuery selector to element to prevent issues on minification


Bedoel je hier niet aperture?


The "Matrix Gaben" picture is not loading for me ingame, it is just black instead.
In browser it does work.


Hier gaat het dus mis als het wordt ingelezen met testfile_parse, want die verwacht een prefix zonder de haakjesuitleg


@EleDiaz Don't push directly, use PR to make it easier for people to follow the changes.


it can be static


Nice, we will finally see when someone will die in-game :)


I suppose I could also use an `std::vector<uint32_t>` for this.


Als je de kleur in RGBValue stopt dan hoef je niet handmatig te clampen.


Just a note to myself, we might want to move this include inside a head block - as it is right now it'll put the HTML includes inside the body (which will be frowned upon at 2nd review).

We need to find all of the parent templates that include course_navigation and put this include inside it's header block.


Alternatively You can make JS_TEST_DATA protected


I've just lost a bunch of hair trying to figure out why my project suddenly stopped working.

I agree with @bitwit, at least a minor change should be made. Totally breaks every app from now on that uses automatic dependency injection into HTML files in the build process (like mine).


Perhaps making `authChangeClosure` public, and then we can set this closure directly if we want to hear back once the permissions have changed?


In retrospect, I could have const-ified this.  I'll add a fix-up commit that does so.


Great work, I was just about to fork and make a PR!


Yeah so the `connect()` promise will resolve for the host when it receives the `ready` event from the client, and will resolve for the client when it receives the CSRF tokens from the host. There isn't really a reject case... unless we want to be a timeout on it.

The point of the connect is so that you wait until the ports are talking to each other before sending events/requests. So typical usage might look something like:

    var host = new Host(...);
    host.onRequest('this', 'that')
      .onRequest('foo', 'bar')
      .onEvent('bling', () => { alert('blang'); } )
      .connect().then(() => {
        host.sendEvent('ooga');
        host.request('booga').then(...);
      });


Just checked inside kernel/cpu.c.....
look at cpu_maps_update_begin() and cpu_maps_update_done() function.... they are called inside
cpu_up and cpu_down function. Each function lock a mutex var called "cpu_add_remove_lock",
so if it is locked, mutex wait until it returns unlocked so it prevent that a cpu become online and offline before executing get_online_cpus() fn.


hey @jinhw1989 I am working on a project that might require some bespoke change on pyomo library so I keep my own version of it.

Just out of interests, how did you find out this project? I basically uploaded it less than 24 hours ago ...


Can we assert somehow that the `sum` really inlined.


TODO for me: there's a hard-coded `kMaxLength` reference in `src/node_buffer.h` that's still `0x3fffffff`.  That should go.


This is some very repetitive code.  Can you think of a way to make it a little more generic?


This error is return by allocator up to here:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/registry/service/rest.go#L90
If we failed to update etcd entry we will return some error, but it'd be hard to distinguish it here.

As I don't see a good place to log this error I removed this log message completely.


For consistency: on Android, if a phone number is not present among the contacts a grey "#" is shown


That is very suspicious, if you can't figure out why this happens right now, that's ok as you may get some ideas later, I hope :)

Also please read this to see why `$scope.$digest()` may be a better idea to call than `$scope.$apply()` for performance: http://glebbahmutov.com/blog/local-angular-scopes/


Did Symfony not handle this charset thing as you expected?


Thanks boss :)


if the using existing infrastructure is hard we can extract call of `sum` to a function and check that stringified version doesn't contain calls


Hey Chris.  Do you remember why we didn't incorporate the new safe_yaml into VTS?  Hope things are going well at TPT!


The only change here is the addition of the `&&` on the parameters of the requires clause to prevent decay of array and function types and avoid weirdness like `Assignable<int*&,int[32]> == true`. It was before and is still the case that `Assignable` doesn't differentiate between `T`, `T&`, and `T&&` on the left-hand side: `a` is an lvalue expression of type `T` in either case. This is different from `is_assignable`, which preserves the value category of all the arguments - I assumed that you intended that difference when you wrote `Assignable` and left it untouched.


I've reverted the one for the accept header. Otherwise, as I said, symfony handles the accept header for us, not the content type header, we're doing that raw.


Duplication has been fixed in 7df6da7.


Anything for you Subhi. You know that.


**rejected**
I like the idea of the ordered ini, but we want to keep the module powershell v2 compatible
the equivalent will be added to the module using the suggestion in [pull #7](https://github.com/lipkau/PsIni/pull/7) which shall keep it posh v2 compatible.

thanks for the suggestion


@ALL
This solves the problem with the fuselage going transparent when viewd from certain angles.
Back to solid.
Thus now, @JWocky seems we solved step 1, and we ready to jump into cockpit implantation

@sanhozay
I want to attempt creating a ac file for the cargo and the winglets versions, thus, I want to know how the "base" system is going to implement multiple versions effectively?

Thanks everyone,
IHCOL

![fgfs-screen-024](https://cloud.githubusercontent.com/assets/10859459/8261307/3d4c413c-167e-11e5-848e-a8f4710d6c8d.png)


if the using existing infrastructure is hard we can extract call of sum to a function and check that stringified version doesn't contain calls


I WILL NOT BE SUPPORTING THIS.


Gotter otter :) Another situation where a simple test can teach us something.


I suggest to use new syntax.

In this example it will be `through: :friendships`


Gotcha. I'll convert to the non-emptystring print.


it's ok


@kbaum I don't follow what you mean by "didn't incorporate the new safe_yaml"  but I'm sure this specific change was made just to be compatible with some other gem.


What about adding `Options` in? eg:

```
  RewriteEngine On
  RewriteBase /
  Options -MultiViews -Indexes +FollowSymlinks
```


I think you should create a separate module diagrams-svg/src/Diagrams/Backend/SVG/Attriubtes.hs

that contains  the `NodeId` type, the `nodeId` function and it's instances.
The `renderNodeId` function can stay here as you suggest. You then need to figure out out how to deal with the
attribute in https://github.com/diagrams/diagrams-svg/blob/master/src/Diagrams/Backend/SVG.hs.
I like the names `SvgId` and `SvgClass`, better than `NodeId`. I don't think you have to worry about restricting the type of `nodeId` since it will only be in scope if the user imports the SVG backend.


Thanks for that one! I really hate to fly without a plane around me ;-)

What is the actual difference between cargo and pax version and what is the actual difference between the regular and the winglet version? Because if we talk here the same cell and wings, only in one case with other doors or additional winglets at the end of the wings, we may want to consider submodels.

If not, for the JSB-version, just copy the set file and the model file (you need that for the Yasims too) and change the first path-tag in the model file to the name of the new Freighter-model.ac or winglet-model.ac.


Make "CS8019" as a const..


This is shitty and stupid malware code. Yes, I went there.


(You mean him removing the code, or me adding the code)?


@amir-qayyum-khan I am unable to get this setting to appear in the studio in advanced settings for a course because there is no `"CUSTOM_COURSES_EDX"` setting in any of the cms envs.  Does this setting need to be added to a cms setting file in order to allow this switch to be toggled?


Him trying to add this code.


winglet:
http://www.aviationspectator.com/files/images/Boeing-727-025.preview.jpg

Cargo: Most windows are gone with the wing (at 350KIAS)
http://www.airlinereporter.com/wp-content/uploads/2012/08/FedEx1.jpg


@parndt thanks!  so excited about a first pull request!!!


okay


I think we should use a GString for both command and response. The API will then be usable for binary protocols too. There's also the advantage that the caller doesn't have to guess in advance how large the response might be - which makes sense given that this is a feature to be used in exploring currently unsupported commands.


wat, I added this code, in the mean time, until stan releases a new version in which this can be fixed. this guy forked the mod and then removed the code.


I tried leaving a comment asking nicely to update with update_version.sh.

I'm guessing you didn't use update_version.sh :P


This ends up calling `v8::String::NewFromUtf8()` and V8 now decodes this UTF-8 sequence to EF BF BD a.k.a. the replacement character.  I don't think there is anything we can do here unless we start decoding the UTF-8 ourselves.


Lazy? :(


I'm not sure if I should be updating this file.  @yosuke-furukawa Can you comment?


Oh totally.. but it looks like VTS is one commit behind in rails_admin.  Another words, we never upgraded to this latest version.  Trying to upgrade us to ruby 2.2.1.  Friday afternoon project :smile:


This is really really really REALLY dumb jared.
(I had my website inverted which made it look like this was adding.)


Wait.. i just answered my own question... you tried ruby 2.2 upgrade a while back.. you must have done this because it was needed for that upgrade!


My suggestion would be:
```c
SR_API int sr_dev_custom_command(const struct sr_dev_inst *sdi,
                                 const GString *command, GString *response);
```


I fixed it in pull request https://github.com/wchill/steamSummerMinigame/pull/327

Literally you need to use http://i.imgur.com/titbsfQ.png url, because original is hotlink-protected.


Makes sense. Check my 2.2 branch


Doing this from my phone so may be missing something, but the comment above looks unfinished.


I know what I am doing. This is NOT permanent code. This will be changing when minetweaker updates. and honestly, 2/3 of my issues were related to /mt reload. So this IS a viable TEMPORARY solution.


If sdi is not set then we should return SR_ERR_ARG.

If the sdi doesn't have a driver, or the driver doesn't implement custom commands, the correct error is SR_ERR_NA.


Provide an example of each command


You are acting like an entitled little brat. You were last night, and you are doing it again right now. This will not bode well with the rest of the community.


fast-forwarding develop to borshop-integration-500-bugfix/zl/dont-recreate-index-unless-ping-throws-a-404 = a4e2773b


i see you


Agreed.  After we try things out a bit, we can settle and document.

On Fri, Jun 19, 2015 at 11:38 AM, Sam Mankiewicz <notifications@github.com>
wrote:

> I know the names of the schemas are a little arbitrary. But once we settle
> on some patterns, let's update (or replace) the diagram too
>
> https://docs.google.com/drawings/d/1szHIpgnvlCM0CHlzMx9Ayi8S9_E-DmeNNAT9MAH3I-Q/edit
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kiva/vertex/commit/07195f422e88a866f97466abe592c28970e14672#commitcomment-11773468>
> .
>

--

*Van Mittal-Henkle | Sr. Software Engineer*

*Kiva.org – Loans that Change Lives*

875 Howard St, Ste #340, San Francisco, CA 94103


sdfg


All of these variable were only used by the FilterViewController(.m) implementation.  Putting them on the interface (.h) means they would be accessible to outside classes.

As one who is pretending to be a client/user of this class, it was overwhelming to see so many variables on the interface not be totally sure what to do with them.


My bad. Title might not be enough. geo, locale, title, and url are probably good pieces of data to log


It was the timeout case I was thinking of - the host might be waiting indefinitely if there is an problem.  It seems like one way or another, the promise should be fulfilled. Do you think there are some exception cases that should result in a reject (i.e. where that internal ready event is not triggered)?


Could you add a middle initial? Even if you don't have one :smile:


Same thing as the previous commit, except with functions instead of variables.  I left only the methods that are required to build what's currently in this project on the interface.


How many tokens shold `until p q` consume?


Then add varargs for the command parameters.


NP. Those fields will be included.


The winglets are really bent up, not just added and the end of the wing seems to be a little wider, so this is like a second model.
The cargo door and the removal of the windows ... well, you can make a model without windows and a big hole where the cargo door is and then give us the cargo door itseltf as second .ac-file. Makes it easier to animate it. Or at least group it under a nice name and we are good.


Have you read this?
http://www.reddit.com/r/feedthebeast/comments/3acdqc/psa_never_use_mt_reload_or_anything_of_the_like/
I am tired of people sending me issues related to mt reload. I literally can not fix those. they are to the mod telling nei to cache their recipes. Also as I spoke with zixxl in IRC, I will not blatently say "you can not reload scripts" I will be taking the TE method. having a config option to "noob" proof the pack. a person simply has to change a single config, and it will work. however I stil lwill not support issues that are caused by reloading. If you have problems with this or even with me, please can we talk in IRC to get it sorted out. as I do not like being on the bad side of mod authors.


Interesting. I like how it allows for a custom wrappers.
You might see if you could merge this upstream.


Yes interesting, but the situation is a little different here as opencv is written in C++. There should be a portable C++ solution as well, though. Thanks for this, nice find!


@rafreis The added behavior here should probably mimic DefaultQuoteStrategy - though this method is deprecated and should probably not be messed with that much. The issue here is if the platform does not support schemas, this behavior is wrong.


What the function did was good. You might want to remove the assets here, as it was before.
Otherwise, you can just call ```JSON.parse(JSON.stringify())``` directory instead of creating this function


the same - it does not work or at least, I was not able to configure it properly.


That is correct.

See the (still draft?) installation docs for the recommended best practice for doing this (devstack & full stack)

http://draft-custom-courses.readthedocs.org/en/latest/building_course/custom_courses.html#system-installation <http://draft-custom-courses.readthedocs.org/en/latest/building_course/custom_courses.html#system-installation>

> On Jun 19, 2015, at 3:32 PM, Cris Ewing <notifications@github.com> wrote:
>
> @amir-qayyum-khan <https://github.com/amir-qayyum-khan> I am unable to get this setting to appear in the studio in advanced settings for a course because there is no "CUSTOM_COURSES_EDX" setting in any of the cms envs. Does this setting need to be added to a cms setting file in order to allow this switch to be toggled?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/edx/edx-platform/commit/6a41cd7bfadb84bdc90d25316db9158d8c6f3ac6#commitcomment-11774294>.
>


You're welcome :-)


internationalization on the label


1. If student choosing ```signup_preference``` yes and instructor approve this suggestion, Expertiza will check whether student have team or not. If not, create team for this student ```(lazy initialization)``` and team user;
2. If student already in one team, Expertiza will check whether this team has topic or not. If not, assign this topic to this team and clean the waitlists.


yes. I think I was planning modifying the ac into three different ones

1. the regular 727-230.ac
2. the 727-230-winglet.ac (use this for a GA variant)
3. the 727-230-cargo.ac

This will greatly easy my job as modeller, and I think if we can get you (the system guys) to load an different ac with three different -set.xml files, then we can even set which liveries work for each one

example

the regular one seeing the commercial airlines liveries
the cargo one seeing the cargo liveries
the GA with winglet seeing the GA liveries.
The military one (with also the regular ac) seeing the military liveries

That's what I 'd love to see happenng

@IAHM-COL


From what I've seen bfcache support is a mixed bag across browsers in terms of events/features and this issue is further reduced because it only applies to iframes.

The last time I checked, it was a while ago, jQuery's iframe support story (scripting from parent to iframe) wasn't 100% supported/worked through. With that in mind I think having this handled with minimal restrictions is appropriate for the iframe scenario.


The creation of this object here is a little confusing:

As a new user to this class, it's unclear to me why I would pass self.tableView in as a parameter when I haven't declared a tableView anywhere.  If/since the superclass is aware of this, can't it just do the assignment inside the init function call?

It feels like a little bit of "inheritance leak"(i might have made this term up :) where the aspects that the super class should be able to set and manage now have to be handled by the child class.


For #58.


Good assertion because `map`ping should preserve state.


not on *nix :+1: windowze user here.


We need not to update this file. This is a just rule for eslint.
If we will remove `smalloc` module, I will send a PR to eslint repo.


OK, sounds fine to me, assuming the regression tests still pass consistently (or at least, as well as before).


Okay, I got it. The games.c command had incorrect target reference. That is, it was not run together with gamesxml2c compilation.

It is fixed in pull request now.


You should execute shutdown-agents at the end for stop all not daemon threads used by pmap
http://conj.io/store/v1/org.clojure/clojure/1.7.0-alpha4/clj/clojure.core/shutdown-agents/


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/0a34d49f3bba7e372268b5f3ebe253da92d4f15c


merging basho/yokozuna/backport/zl/yokozuna-pr-500 = 0a34d49f into borshop-integration-501-backport/zl/yokozuna-pr-500


basho/yokozuna/backport/zl/yokozuna-pr-500 = 0a34d49f merged ok, testing candidate = a82231d1


Could use `f` here.


i am confused, why do we need the releases and release_file_parser extensions back?


set up to practice with Ted


It is, but I only moved it.  I believe you are the original author.  :-)


Okay, thanks.  I'll back out this change before I land the pull request.


```php
$name = array($this->table['name']);

if (! empty($this->table['schema'])) {
    array_unshift($name, $this->table['schema']);
}

if (isset($this->table['quoted'])) {
    $name = array_map($name, array($platform, 'quoteIdentifier'));
}
return implode(".", $name);
```


I thought it was useful when I still used command line interface to install packages from jobs. As for now the `build_and_install_package` function from `opam-lib` is used, I guess this function turns up to be useless then


@gibson042
> Not for at least a week, and even then I'm not sure. I do like the ideas, but I still hate this whole stupid block, especially when everyone has to suffer because of IE (and now Edge).

That's cool. This issue will be tackled but currently not planned in the immediate Edge release.


For #58.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 75](https://teamcity.labkey.org/viewLog.html?buildId=255918&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 1 Build time: 00:15:16
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSUpgradeTest: <no details avaliable>

```


Could also do

```
val xs = List(1, 2)
val (ys, rng2) = sequence(xs.map(State.unit))
ys mustEqual xs


Hi. I think you're basing this off the master branch. Can you take a look at the develop branch instead?

See https://github.com/nikclayton/android-squeezer/blob/master/HACKING.md for details.

Thanks.


Having a dream of this thing to be accepted one day: https://github.com/rails/rails/pull/8794


One of the comments on #66 pointed out that this line will fail because `this` is undefined here. The `eachRelationship` callback function needs a `bind(this)` at the end.


Thanks for the suggestions. I'll get those implemented.


Thank you for contributing.

On Fri, Jun 19, 2015 at 3:53 PM Andrew Martin <notifications@github.com>
wrote:

> Thanks for the suggestions. I'll get those implemented.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andrewthad/diagrams-svg/commit/e57c9140a9fa71259df843fce63bc334a59239a0#commitcomment-11774633>
> .
>


Apparantly it was, good that it's back.


Oh, hah. Should have finished with "calls into JS". Eh. I'll get to it later.


Hey @zxdgoal,

Why did you add the ```allowedContent``` and ```requiredContent``` properties?

If you remove them, the plugin still works correctly.


Good! Learned a new useful library out there


I'll leave the "I2C-SMBus" form, since it helps the user grasp what heck of command is that (SMBus write byte is a widely documented operation, not to be confused with other types of writes there can be).


This broke many regression tests on evergreen: https://evergreen.mongodb.com/build/mongodb_mongo_master_ubuntu1404_rocksdb_54e8c6b505d8a53014677ae0a1579695229022aa_15_06_19_17_56_27


Will there be a combi model ?

Sent from my iPhone

> On 20 Jun, 2015, at 3:44 am, FGDATA <notifications@github.com> wrote:
>
> yes. I think I was planning modifying the ac into three different ones
>
> the regular 727-230.ac
> the 727-230-winglet.ac (use this for a GA variant)
> the 727-230-cargo.ac
> This will greatly easy my job as modeller, and I think if we can get you (the system guys) to load an different ac with three different -set.xml files, then we can even set which liveries work for each one
>
> example
>
> the regular one seeing the commercial airlines liveries
> the cargo one seeing the cargo liveries
> the GA with winglet seeing the GA liveries.
> The military one (with also the regular ac) seeing the military liveries
>
> That's what I 'd love to see happenng
>
> @IAHM-COL
>
> —
> Reply to this email directly or view it on GitHub.
>


Nice addition, looks fine.


90664130_AlternativeModelPAT (macumber) - Win64-Windows-7-VisualStudio-12: OK (2184 of 2213 tests passed)

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


Could we document which value of this option makes it the GA4GH scheme?


The same. This commit appears to be broken.


delete


why not have ```initiateScheme``` be part of the constructor for ```GA4GHSchemeWithMissingAsNoCall``` or ```GA4GHScheme```.


Missed that, sorry...

The same applies for the other pull request I made, of course ;-)


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/948ca1daf927478aaacd3d8e504862deaefbe5df


merging MonsieurNicolas/stellar-core/scpFixesCounters = 948ca1da into auto


MonsieurNicolas/stellar-core/scpFixesCounters = 948ca1da merged ok, testing candidate = 6dc2e1e8


到此一游


Early return here?


This is not a good idea, it could make %100 levels be skipped.


刚来到github请多多关照


encoding?


The implementation here is exactly the same same as that inherited from MagicPermanet's superclass, MagicObjectImpl. Is this necessary to prevent a bug/crash?


Nitpick: please move the assignment outside of the clause for clarity


The reason I used
```ruby
agent_jar = "#{new_resource.install_dir}/newrelic/newrelic.jar"
```
here is that New Relic's Java agent auto-install behavior is hard coded to point to app/path/newrelic/newrelic.jar (disgusting, I know). In other words, the install path must ALWAYS have the newrelic directory containing the newrelic.jar. This is also why I was appending /newrelic to node['newrelic']['java_agent']['app_location'] in other parts of my code.


How is this struct different than just having a `uint32_t`?


strictly speaking, the result was undefined (http://stackoverflow.com/questions/1626446/what-is-the-size-of-an-empty-struct-in-c). GCC just gave a good answer.


I worry that this approach leaves us open to failures in compile; would it be better to use e.g. linecache to get the file contents, which is already unicode aware.


New Relic's Java agent also expects the jar to always be named `newrelic.jar` so good thing you kept that part. Unfortunately it still needs the newrelic parent directory mentioned above.


Btw your commits haven been including a lot of unrelated whitespace changes recently, this makes it hard to figure out where are the actual important code changes. Perhaps an overzealous editing setting? I'm fine with correcting whitespace issues, but they should be in a separate refactoring commit and not mixed in with actual functionality changes.


:thumbsup: Indeed this workaround works, it simply removes the album art (which I don't care at all).

Thanks for your help, much appreciated :smile:


I noticed that some code paths in this function return FALSE without throwing an error, but I am not sure if that's OK.


No problem! I hadn't submitted a PR only because I hadn't done much testing with it yet, but I'll try and get on that sooner than later. The whole motivation for the changes was to provide a way to avoid the "jitter" within a given hue that can happen when animating across many frames with a high number of colors. If I can find some free time this weekend, I'll try to get back to working on this.


Yeah, you do have to limit it, otherwise it tries to run *everything* at once.


We can't do this in cachecontroller, . Open an issue on ionrock/cachecontrol

Besides if you set current_age = 0 the cache will never be invalidated and never refresh with a new request.


I wish I knew how you found this :gem:


![captura de tela de 2015-06-18 23 07 42](https://cloud.githubusercontent.com/assets/6960002/8246096/d9ce4e9e-160e-11e5-9fcd-f3652e710703.png)
@iagosilvany olha como fica na minha tela, da pra chegar mais pra esquerda


This will backfire when people buy 70-80k wormholes tomorrow.


https://github.com/jam2in/arcus-java-repl/issues/13#issuecomment-113038005 수정 내용입니다.


I tested the `-s` switch quite a bit and it doesn't work as you would expect. If your install_dir and app_location differ, the agent's auto-installer will hardcode app_location/newrelic/newrelic.jar into the app's startup and of course that will break the app because the jar is actually installed at install_dir.

Best to remove it completely (as it was in my commit).


This is the file I forgot to include in the last PR


I don't think this logic does what it's supposed to do. This will cause wormhole spam when people join games later, or simply join with more than 86400 wormholes at launch.


To elaborate, the problem is that once only a few people with the most wormholes start spamming them, it dictates for the whole room that there will be no more %100 rounds.


One thing that me and @mattdangerw were commenting on is how inconvenient it is for classes implementing the interface to always have to specify the new property here (or use an override, with next commit) even if they don't need to do anything with it.
If getters and setters can be implemented on the interface especially, things should just work if someone only specifies that an interface is implemented by the class without re-stating the properties.
Similarly, things should just work if an implementation only re-implements the getter and setter without re-stating the property.


没事常来哦！


Yep, please see:
b/21934841

On Thu, Jun 18, 2015, 6:37 PM Glenn Lewis <notifications@github.com> wrote:

> @tmatsuo <https://github.com/tmatsuo> - do you know why the Travis CI
> tests are failing?
> When running the integration test locally, I get this:
>
> integration_test.go:126: PullWait error: pubsub: cannot decode the retrieved message at index: 0, PullResponse: &{AckId:dQNNHlAbEGEIBERNK0EPKVgUWQYyODM2LwgRHFEZDDsLRk1SKhsNN1METwZzeWs3KgkXEzgUACUwAk8HeE4FKxpWYhlyfWF1a15GAQtCUisLX00HaVZcZz1zMRFze2RwY14XBwpNV1bRn6M1ZjxRDk9eIz81ICg1QlVWBwY9HFELH3cOAjYYCFMGICIjIg Message:0xc2083f95f0}
>
> FAIL
> FAIL google.golang.org/cloud/pubsub 5.624s
>
> Do you see this too?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GoogleCloudPlatform/gcloud-golang/commit/ae54c7850096ec392a6694d6955f9456487dd0b5#commitcomment-11759883>
> .
>


Controlling things in-game, and writing software to display the state of microcontrollers and stuff.  Going to take this thing somewhere different.


why dizint work?


Indentation


Thanks! Can you write a quick test for this that has absurdly high energies, and verify that it fails when you comment out this line?


You left out:
```ruby
app_user node['newrelic']['java_agent']['app_user'] unless node['newrelic']['java_agent']['app_user'].nil?
app_group node['newrelic']['java_agent']['app_group'] unless node['newrelic']['java_agent']['app_group'].nil?
logfile ::File.basename(node['newrelic']['application_monitoring']['logfile']) unless node['newrelic']['application_monitoring']['logfile'].nil?
logfile_path (node['newrelic']['application_monitoring']['logfile'].nil? ? node['newrelic']['application_monitoring']['logfile_path'] : ::File.dirname(node['newrelic']['application_monitoring']['logfile'])) unless !node['newrelic']['application_monitoring']['logfile_path'].nil?
```
Without these, you can't pass any of these attributes to the LWRP...


Hi @djoos,

I left some comments on your commit. Also, I'm confused why you left out the `::File` namespace fix? That's a big one as mentioned by @qwyzyx on my PR.

Thanks


Nice find on the TinySound lib, looks like a good way to go. Is there a reason we are not using the jar as we do with the other dependencies (commons-io, miglayout, etc), but rather we are tracking the source code in the repo itself?


In https://git.gnome.org/browse/gjs/commit/?id=7a8496400a3d07963df25cdfdbc74991db75b107 I used `GLib.test_expect_message()`. Not sure if that helps.


How come you left out the accompanying templates/default/agent/nodejs/newrelic.js.erb ? capture_params and ignored_params will never make it to the agent configuration unless they have corresponding entries in the config template. Maybe you already have it or I am overlooking something. Let me know.

Cheers


I wonder if you can unref `new_pspec` after this.


这个问题我想到两个解法：
* 创建一个当前目录的临时css文件，把需要的样式写进去；
* 直接把css文件内容写到每一个页面的`<style></style>`里面;
需要pullrequest吗？


It is not Prompt Sample.
It is just JavaScript and php sample code which shows boolean and valuable briefly.


it is way to make valuable at php.


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


It is way to make bool(??)


How did you know I created it? Github is so confusing! Anyway - I'll update the copy over the weekend and we'll be up and running...


Unfortunately it does persist in our index… filter the commits or rebase them some day.


A little credit at least in the commit message would be nice. https://github.com/thegrb93/Starfall/commit/84bd172adbf448e940c9016c2c551135f2fb23df


Am I blind or is there no difference? Or is github bugged for me?


needs a test case.


fast-forwarding master to auto = 6dc2e1e8


You can't do this!

`OnTransportAttach` is called on a transport when a unit has attached to it.
`OnAttachedToTransport` is called on the attaching unit when it attaches to the transport.

These are very much non-interchangeable. You got away with it because in this case both are no-ops. In principle, other uses of this base class could catch fire as a result of this change.


Then we're both blind :-)
Maybe spaces instead of tabs?


@caoyuan  — just to let you know, I've been using the exact same params.txt format as used by ZMERT, which means that I skip the lower and upper bound values. This is easier than generating a separate params.txt for each of the tuners.


por lo que pude entender $getByHandle  permite integrar una variable en la vista y consultarla para poder obtener datos como la posicion que se encuentra desde su top. en este caso llamada contentScroll y esta se agrega en la vista como delegate-handle="contentScroll"


$getByHandle Fue la solución a mi problema, en un principio no podía obtener la posición de la vista si venia delegada desde otra vista anterior,


> There are definitely two distinct modes for using the deck editor.

You've hit the nail on the head! I'm talking about the deck editor in support of limited duels, where you can tweak the randomly generated decks. While @ShawnieBoy is talking about it in relation to Firemind, which focuses on the Constructed aspect of the game.

@ShawnieBoy do you mean you use the deck editor to create decks for submission to firemind? Like to clarify this particular use case better.

For me, the deck editor is a quick way to tweak the randomly generated decks. Eg. You are mostly happy with the deck except that you really hate to play with a particular card, so you swap it out with something else that you like better. Then you start to play.

For this use case, I noticed that the deck editor doesn't respect the cube selected for the duel. Say I'm only interesting in playing with cards in Standard for my duels. The initial random deck generation will use just the cards in the standard cube, however in the deck editor it will show all available cards in the card pool. For deck editor opened for a duel, the cube filter should be initialilzed with the cube selected for the duel.


rax retry tests


:astonished: I'm trying to test my app should allow iframing on some condition. but this prevents the default headers to be added.


No related bug, just plays a bit strange. I'm not sure if this also gives the AI the opportunity to choose no, after activating the ability.

I'm using a new IDE at the moment, I think it can be a bit overzealous with formatting, I'll see if I can tone it down.


closes #1


If I remember correctly GetGuards() only returned units assisting the unit, not units actively ordered to repair something.


I'm losing it - replied thinking it was the issue I put up.

So is .hasCounters(counterType) already supported then? In that case, it can be reverted. Didn't realise that.


Just ran the test with the new code and it looks good! HEre is the commit:

https://github.com/makwanza/ios-exercises/commit/3b28bd1e55b3f637f8dea0d26927fd6b07795112
Thanks for helping out!


Before, there were two elements, but I determined that the second was never set to anything but 1.
I also changed that earlier today, but need to push it to GitHub...


Can you clarify what you mean by plays a bit strange? How is hasCounters related to the way the AI plays?

From what I can see, this is code duplication that can be eliminated without changing functionality. I guess you are adding this as a test to see if it affects something else. As far as I can tell this change doesn't affect the behavior of the MagicPermanent class at all, since it originally inherits this implementation from its parent class, now it overrides that method with the same implementation.

Eg. in simpler form it is something like
```
class A {
  public int do() { return 5 }
}

class B extends A {
  public int do() {return 5}
}
```


Thanks for doing this!


This looks like a borkd merge, no? notice the dashes in the beginning of the removed rows.


>@ShawnieBoy do you mean you use the deck editor to create decks for submission to firemind? Like to clarify this particular use case better.

Yes, or just to play constructed vs the AI.


Looks like he manually copied code out of a diff and took the `-` with him when he did.


No ASP.NET usa sempre `@Url.Content` para o que estiver dentro da pasta `Content`:

```Razor
<img src="@Url.Content("~/Content/img/logo.png")" />
```


The lines added in https://github.com/wchill/steamSummerMinigame/commit/b140001b2cb2a192a7dc0271243304e9615c097c were prefixed with dashes, presumably due to being copied from a diff somewhere. This commit removes the dashes.


hello


Utiliza o `@Url.Content`:

`<link rel="icon" href="@Url.Content("~/content/img/favicon.ico")" type="image/x-icon">`


It's just a scala + android + sbt frame, mostly for myself.


# R Programming - Coursera
# Programming Assignment 2 - Lexical Scoping
# (How to calculate, cache, and retrieve the inverse of a square matrix)
# Submitter:  A. Nichols
# Date:  18 June 2015

# To source this code in the R console, use this (minus the "#"):
# source("20150618lexical_scoping_assignment_v2.R")

# Initialize two 3x3 matrices (and their inverses) to test this code:

# Test matrix 1 (and its inverse):
testv1 <- c(.2, .4, .7, 1, 2, 3, 9, 10, 11)
testm1 <- matrix(testv1, nrow = 3, ncol = 3)
invtestm1 <- solve(testm1)

# Test matrix 2 (and its inverse):
testv2 <- c(3, 0, 9, .1, .2, .3, 1, 7, 18)
testm2 <- matrix(testv2, nrow = 3, ncol = 3)
invtestm2 <- solve(testm2)

# Initialize an empty matrix (and its inverse) for the code below:
newm <- matrix()
invnewm <- matrix()

# The first function here accepts a matrix and can cache its inverse.
# To use makeCacheMatrix, subset the function and pass it an argument:

# Example 1:
# Subset/call with:		a <- makeCacheMatrix(testm1)
# Return current matrix:	a$get()

# Example 2:
# Subset with "a":		a <- makeCacheMatrix()
# Set current matrix:		a$set(testm2)
# Return current matrix:	a$get()


What sort of speedup are we talking here out of curiosity? Like, 1k rows in less than 250 milliseconds speedup?


I didn't notice the jar link at the time so yes would probably be best to use that instead but I will leave the decision to you.


Probably need to fix isGetUserMediaSupported too?


hasCounters(counterType) is inherited from the parent class of MagicPermanent, otherwise it can't compile since this is a method required in the MagicObject interface. Removed the duplication in https://github.com/magarena/magarena/commit/22223077f180712955f448877f57a06f352df55d


This row needs a Math.floor() around it before return. Too many decimals are shown currently. : )


makeCacheMatrix <- function(newm) {
	  invnewm <- NULL

	  # "set" enables user to input a square matrix to find its inverse
        set <- function(y) {
                newm <<- y
                invnewm <<- NULL
        }

	  # "get" enables user to see the current matrix
        get <- function() newm

	  # "setinv" returns the inverse of the current matrix, if
	  # cacheSolve has been called (otherwise, it returns
	  # NULL, ... or something else if a$setinv() has been used)
	  setinv <- function(q) invnewm <<- q

	  # "getinv" stores the inverse of the last matrix sent to
	  # cacheSolve (below)
        getinv <<- function() invnewm

	  # The list command stores four functions in makeCacheMatrix;
	  # it enables objects sent to makeCacheMatrix to have all
	  # four functions.
        list(set = set, get = get, setinv = setinv, getinv = getinv)
}


Hrm. Well if there's any hope of achieving the same end result without requiring a GCC extension, I'm all ears.


Ok, will use the jar then, for consistency with other dependencies and to make it clear this isn't code that we wrote.


1682_waterheater_stratified (mbadams5) - x86_64-MacOS-10.9-clang: Tests Failed

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


1.0 released?


Excellent feedback.  I was unaware of both of those.  Will fix.


# The function below, "cacheSolve", computes the inverse of a matrix
# stored in makeCacheMatrix.  If the inverse of the matrix has previously
# been determined, cacheSolve returns the existing inverse of the matrix
# without doing additional calculations.  cacheSolve updates "setinv"
# in makeCacheMatrix, if appropriate.


Thanks for the changes, especially with `routes`. Curious as to why change the `Welcome` controller name to `Shallows`?


cacheSolve <- function(newm) {
        invnewm <- a$getinv()  # gets current inverse from makeCacheMatrix
        if(!is.null(invnewm)) {
                message("getting cached inverse matrix")
                return(invnewm)
		    # Note: cacheSolve is only a cache for the inverse
		    # of a matrix.  You must call makeCacheMatrix
 		    # first to ensure correct output from cacheSolve.
        }
	  # If the inverse of the matrix has not already been determined,
	  # cacheSolve executes additional steps:
        mdata <- a$get()        # get current matrix from makeCacheMatrix
        invnewm <- solve(mdata) # determine inverse of current matrix
	  a$setinv(invnewm)	  # update 'setinv' in makeCacheMatrix
        invnewm                 # output inverse of current matrix
}

# Compare calculated inverse matrix for testm1 with invtestm1
# Compare calculated inverse matrix for testm2 with invtestm2

##### End of program #####


left the color: out of the style rule


Tolong ini dihilangkan..
http://awesomescreenshot.com/08c4zvs91f


From my count, the old version was calling append() {(N * (1 + C + M * (2 + C*3)) where N = # of data series, M = # of data points in series, C = # of columns in series} times. This was reduced to simple N append() calls.

Each append() call can cause a re-layout of the page, which is a fairly expensive operation. Effectively, a "SELECT * FROM Downtown" query was taking my Laptop about 10 seconds and now it takes about 1.


Nice work!  Keep it up!!  \(^_^)/


test


doubule fac(int x)
{
      return(!x?1:x*fac(x-1));
}
int gcd(int x,int y)
{
      return(y?gcd(y,x%y):x);
}
void bubble(int *a,int n)
{
      int i,j,tmp;
      for(i=0;i<n-1;i++)
      for(j=i+1;j<n;j++)
      if(a[i]>a[j])
      {
            tmp = a[i];
            a[i] = a[j];
            a[j] = tmp;
      }
}
int main(int argc char *argv[])
{

}


resources :advertisements


Fixed.


# R Programming - Coursera
# Programming Assignment 2 - Lexical Scoping
# (How to calculate, cache, and retrieve the inverse of a square matrix)
# Submitter:  A. Nichols
# Date:  18 June 2015

# To source this code in the R console, use this (minus the "#"):
# source("20150618lexical_scoping_assignment_v2.R")

# Initialize two 3x3 matrices (and their inverses) to test this code:

# Test matrix 1 (and its inverse):
testv1 <- c(.2, .4, .7, 1, 2, 3, 9, 10, 11)
testm1 <- matrix(testv1, nrow = 3, ncol = 3)
invtestm1 <- solve(testm1)

# Test matrix 2 (and its inverse):
testv2 <- c(3, 0, 9, .1, .2, .3, 1, 7, 18)
testm2 <- matrix(testv2, nrow = 3, ncol = 3)
invtestm2 <- solve(testm2)

# Initialize an empty matrix (and its inverse) for the code below:
newm <- matrix()
invnewm <- matrix()

# The first function here accepts a matrix and can cache its inverse.
# To use makeCacheMatrix, subset the function and pass it an argument:

# Example 1:
# Subset/call with:		a <- makeCacheMatrix(testm1)
# Return current matrix:	a$get()

# Example 2:
# Subset with "a":		a <- makeCacheMatrix()
# Set current matrix:		a$set(testm2)
# Return current matrix:	a$get()


From this line you can see the interpretation by JS


Look at how I do build scripts recently with `winapi-build`.


Given all the changes I made to features relative to the number of changes here it might be worth it to make another branch and make your changes one more time.  Just a thought that could simplify merging later.  Sorry for the inconvenience.


@ataravati where you able to get ether of these events to work? I am also trying to get onAfterRemoveCroppedImg to work without much luck.


That's painfully stupid :/


This line cannot be interpreted by PHP. just HTML can read this as TAG.


hello world


a


But there also exists `GetAssistingUnitsList`. How does this differ from GetGuards, then?


That already existed code. I think the previous writer open the FILE* to use FOREACH_LINE. I could not find useful macro for file descriptor (not FILE*). Anyway I try with only file descriptor.


same comment as above.


The same issue was present when a single entry was moved from the left sibling into the parent and from the parent in to the current.
I also modified the code to use NUnit so I can run the tests on mono-develop on macos.


you can have this be in the constructor of child classes


spaces here and below


Oops, missed that. Fixed in 49e83efd7bf4ab687989eba8c9d74a14a4ed034d.


Yeah I'm not sure. I'm fine with just leaving it for now and seeing what kind of error cases we run into.


I was looking for the same thing to what you did. I wanted to style active tab on navigation bar accordingly to the middle view templates as "users" or "all products"


CREDITS? ;)


@pixiez search inside GitHub, Actually, I would like to implement a stochastic model based on PySP. And the way I did before, which in AMPL, seemed a bit sophisticated to handle data.


No worries.

The big change for this stuff on develop is that the code now tries to canonicalise the URLs for images as soon as they're received from the server, so that it only has to be done once, instead of potentially every time you use the image.  That was done in https://github.com/nikclayton/android-squeezer/commit/8e56a6eefe0fa106b60422e1a6ecd277cd0fc1b8.


CSS3 Transitions not supported by: IE (6,7,8,9)


This tests needs comments explaining what is being tested


*commented


Comment what these additional test cases are testing.


Try to avoid non specific language like "stuff" in code comments.


Okay, your code is more clean, but it do the same. I guess that the behavior here is not wrong. You can see it in https://github.com/rafreis/doctrine2/commit/a4e734f9143fbb6fa20bb60f805bd47e8395c73b#diff-6909d9b89566a747ffbeb0921ad6cf13R25


should we not check that `arguments.length > 1` before passing `arguments[1]`?


Don't use <TAB>s, use 4 spaces instead


Small point but if we are considering this function to be the prediction and it happens to only need to update the covariance it may be better named `nkf_predict` or something.


You should change this when the pr lands.


Grazie Jacopo

Se vuoi dare una mano a migliorare lo stato attuale dell'archivio, welcome
on board :)
Anch'io ho brevemente lavorato con lucene ed ero curiosa di testare solr.
Se ho tempo faccio qualcosina domenica.

s1m0

2015-06-18 22:40 GMT+02:00 crazyhg <notifications@github.com>:

> Ciao,
>
> se avete bisogno di una mano io sono disponibile. Seguo spesso hookii
> anche se non commento, almeno posso aiutare un po' qui.
>
> Tempo fa ho usato brevemente Elasticsearch, anch'esso basato su Lucene..
> Solr non dovrebbe essere troppo diverso.
> In caso potete contattarmi a questo username sulla mail di google.
>
> Jacopo
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ernitron/hookii/commit/20d7101bfbd5e8eb754eab3f3d1ecef66d365bf7#commitcomment-11755919>
> .
>


This is the only comment that got updated to Doxygen format. Also would be a good time to document all the struct members if you are going to do a few.


Remember, before the Chrome sandboxed packaged apps issue it didn't have a vendor guard on it and everyone was OK with it having a reduced blast radius of being for iframes only. See https://github.com/jquery/sizzle/pull/231.


Comment needs to open with `/**` for Doxygen.


I think this change broke the caseIDToIndexMap, although I don't know enough about the model to see any visible effect of it.

Currently, if you inspect the caseIDToIndexMap of any collection, every index is "NaN."

It looks like this is because the first case that is created has no parent, so parentID is undefined, so caseCounts[parentID] is undefined.

Previously, the equality operator was used, and since null == undefined, caseCounts[parentID] was assigned 0, and caseCounts[parentID]++ was 1.

Now with the identity operator, caseCounts[parentID] remains undefined and caseCounts[parentID]++ is NaN, which is propagated for all future caseCounts.

However, as I said, I don't currently see any effect of this.

I am currently implementing the ability to insert a case into an arbitrary index of a collection, so I had assumed that it would be important to keep this map correct. But if this map being all "NaN" hasn't caused any ill-effects, then maybe the whole map isn't needed?

If the map is needed after all, this could be fixed by using `SC.none` instead of `=== null`


Du hast hier Englisch und Deutsch gewechselt. :smile:


Document the return values with `\return`


Does this change result in a diff in the generated output or not?


Nope


Thanks for confirming.


You might consider asking Elsie what she thinks of the logo


I'm sorry, I couldn't help.


Just for posterity, I believe that the assertion, which was added is correct. It is just that in the get last error code we are not following the contract of pre-declaring the intent to "wait until durable" and that's why the assertion gets hit there.

We could fix it by adding this call in CmdGetLastError::run.

Also, as of the latest WT drop, there is no need to predeclare this intent on the WT cursor, so we could potentially get rid of this call entirely.


```php
        $tableName = $this->table['name'];

        if ( ! empty($this->table['schema'])) {
            $tableName = $this->table['schema']
                . (( ! $platform->supportsSchemas() && $platform->canEmulateSchemas()) ? '__' : '.')
                . $this->table['name'];
        }

        return isset($this->table['quoted'])
            ? $platform->quoteIdentifier($tableName)
            : $tableName;
```

from DefaultQuoteStrategy...


@rvagg I don't think this indentation is to your liking, but I tried my best :) I think I have most of your code style in my head but I'm unsure of object literals in arrays.


Actually, now that I think of it I'm pretty sure it can be a relative path in the repo.


It does work under cygwin, weird file issues pending


Now that you've switched to ES6, I think in Port you could use for/in to iterate the values.  Might tidy iup (the already simple) code in the loop.  Also, I think "let" should be used instead of "var".  "let" is what "var" should have been.


@kanaka haha, yeah it just needed fixing at `confirm-var-exists`.


READ ME

This movie trailer website was made with python, and shows the trailers of the movies when clicked. It is probably the best webiste on the web.

CONTACT

If you want to contact the authors, you will not be able to.

LAST EDITED

The lastest copy was added to this file on June 19, 2019.

NOTICE

This code may be subject to copyright.


 /*B3_Bump.ino

   Drive the TWO-WHEELED Bread Board Bot (BBbot, B^3)
   forward.  When a whisker bump sensor on either side hits something,
   back up and turn slightly away from that direction and resume
   forward path.

   Arduino: Arduino Mega 256 v3 Clone
   Motor Shield: Adafruit assembled Motor Shield for Arduino v2
   ---->  http://www.adafruit.com/products/1438

   Programmer: Dave Eslinger; June 12, 2015
   Revisions:
*/
#include <Wire.h>
#include <Adafruit_MotorShield.h>
#include <math.h>
#include <breadboardbot.h>

// Create the motor shield object with the default I2C address
Adafruit_MotorShield AFMS = Adafruit_MotorShield();

// Define Constants

const byte LEFT_BUMP_PIN = 47;    // Define DIGITAL Pins for left
const byte RIGHT_BUMP_PIN = 46;   // and right bump sensors

const byte FORWARD_SPEED = 150;   // Define normal speeds
const byte BACKWARD_SPEED = 100;  // and backup/turn speed
const int  TURN_DURATION = 600;   // Turn length in milliseconds

// Define 'ports' for motors.
const byte LEFT_MOTOR_PORT = 3;
const byte RIGHT_MOTOR_PORT = 1;
// Create pointers to motor control objects
Adafruit_DCMotor *motorLeft = AFMS.getMotor(LEFT_MOTOR_PORT);
Adafruit_DCMotor *motorRight = AFMS.getMotor(RIGHT_MOTOR_PORT);

void setup(void){
  AFMS.begin();  // create with the default frequency 1.6KHz
  // Turn off all motors to start, just a good habit
  motorLeft->run(RELEASE);
  motorRight->run(RELEASE);
  Serial.begin(9600);  //Begin serial communcation

  /*Set up Bump Pins with Arduino internal pullup resistors
    This will make them always high unless a bump switch is hit,
    which will make a connection to ground and they will read low. */
  pinMode(LEFT_BUMP_PIN,INPUT_PULLUP);
  pinMode(RIGHT_BUMP_PIN,INPUT_PULLUP);

  delay(2000);  // Two second delay to get the robot ready
}

void loop(){
  /*  Assuming no switches closed initially.  Drive forward: */
  motorLeft->setSpeed(FORWARD_SPEED);
  motorRight->setSpeed(FORWARD_SPEED);
  while(digitalRead(LEFT_BUMP_PIN) && digitalRead(RIGHT_BUMP_PIN)) {
    motorLeft->run(FORWARD);
    motorRight->run(FORWARD);
  }

  /* If you got here, one of the bump switches was closed */

  /* First check the LEFT sensor: */
  if(! digitalRead(LEFT_BUMP_PIN)) { // the LEFT side switch was bumped
    motorLeft->setSpeed(BACKWARD_SPEED/2); // Slowly back up and turn to right
    motorRight->setSpeed(BACKWARD_SPEED);
    motorLeft->run(BACKWARD);
    motorRight->run(BACKWARD);
    delay(TURN_DURATION);                  // for specified duration
    motorLeft->run(RELEASE);               // Then stop power to the motors
    motorRight->run(RELEASE);              // and move to next section of code
  }

  /* Then check the right sensor: */
   if(! digitalRead(RIGHT_BUMP_PIN)) { // the RIGHT side switch was bumped
    motorLeft->setSpeed(BACKWARD_SPEED); // Slowly back up and turn to left
    motorRight->setSpeed(BACKWARD_SPEED/2);
    motorLeft->run(BACKWARD);
    motorRight->run(BACKWARD);
    delay(TURN_DURATION);                 // for specified duration
    motorLeft->run(RELEASE);               // Then stop power to the motors
    motorRight->run(RELEASE);              // and move to next section of code
  }

   /*That is all!  Now go back to the beginning of the loop and
     drive straight ahead until somehting is bumped. */
}


This is a markdown file


Well Dr. Eslinger, it's been fun having you as a teacher.


cool, vielen Dank, Martin!
Hoffe, dass ich bald mehr Zeit habe, was in PyEmma zu implementieren.

LG,
Sebastian

Am 19.06.15 um 20:39 schrieb Martin K. Scherer:
> added chi1 feature (thanks to @sstolzen007) with test. All angle features return as many
> descriptive strings as features.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/marscher/PyEMMA/commit/831cd4152eb0aa8c8ce44c37d73af31f4f022d9b


Thanks! I will change that and keep it in mind next time.


How are we getting a response here?  It seems like we're just telling the actor to go check the permission, but we are not listening to any response from the actor?  Is this somehow working and I just don't know why? @gbggrant @jacarey


I've created a pull request mostly to centralize what I'm currently doing and get feedback on it, even if it's not incorporated: https://github.com/nickoneill/PermissionScope/pull/39


what is this?


Ack, thanks a lot, changed as requested.


:shipit:


I ran some tests with some whitespace within the values for the SYMBOLS command, and indeed in these instances, the parser (or tokenizer - not sure of structure here) quits the line for symbols upon finding a whitespace character. This is a preexisting bug that was overlooked, because I presume, most users use the Nexus parser for sequence or tree data. I would have added this error message to call the attention of the user to the issue and give a resolution. Obviously, fixing the bug is preferred.


Issue #40.


Looks like I'm using environmental variables wrong on -e MONGO_URL. I'm rewriting it. You can see how it was by looking at circle.yml.


Both fixed, thanks.


@dom96: Deprecated here.


...and here.


Dodaliśmy parsowanie tekstu, jednak nie mogliśmy przesłać wylosowanej litery to Activity Gra (kod jest, ale nie wiemy czemu nie działa, jest dodany komentarz kodzie do Activity Gra)


Good job guys I totally forgotten aout this issue!

I understood what you meant by the commit message. But be careful.

The message leads the reader to understand that you fixed a konacha issue and I don't think this fixes konacha's issue. By the description the issue was with sprockets rather then konacha.

And it gets even more confusing when you use `issue #123` because github wrongly autogenerates a link to Prezento's issue rather then Konacha's.


Disabling it for all the files was easier at that time.
I'm using @arter97 Linaro (https://github.com/arter97/linaro-lto).


CSS3 Transitions not supported by: IE (6,7,8,9)


Dodaliśmy parsowanie tekstu, jednak nie mogliśmy przesłać wylosowanej litery to Activity Gra (kod jest, ale nie wiemy czemu nie działa, jest dodany komentarz kodzie do Activity Gra) Ten zip po pobraniu nie chce się kompilować (przez program Github for Windows, który ciągle mówił o błędach w synchronizacji). Działający plik jest w wiki.


I thought maybe we should check `uname` for Darwin, but then I thought maybe it's an HFS thing.  But I don't actually know. So thanks for merging it anyway. :)


@ibuildthecloud
I must be doing something stupid, because `./script/run` fails with:
```
[    0.839033] VFS: Cannot open root device "(null)" or unknown-block(0,0): error -6
[    0.839460] Please append a correct "root=" boot option; here are the available partitions:
[    0.839936] Kernel panic - not syncing: VFS: Unable to mount root fs on unknown-block(0,0)
[    0.840402] CPU: 0 PID: 1 Comm: swapper/0 Not tainted 3.19.3-rancher #1
[    0.840775] Hardware name:   BHYVE, BIOS 1.00 03/14/2014
[    0.841167]  0000000000000000 ffff880000085000 ffffffff813f98e4 ffffffff8176d3ec
[    0.841634]  ffffffff813f6fc5 ffffea0000002140 ffffffff00000010 ffff88003e20be78
[    0.842169]  ffff88003e20be18 ffff88003e20be68 ffff88003e20be90 0000000000000000
[    0.842696] Call Trace:
[    0.842889]  [<ffffffff813f98e4>] ? dump_stack+0x40/0x50
[    0.843203]  [<ffffffff813f6fc5>] ? panic+0xc8/0x1f6
[    0.843480]  [<ffffffff81aac39f>] ? mount_block_root+0x28c/0x29b
[    0.843811]  [<ffffffff81aac531>] ? prepare_namespace+0x133/0x169
[    0.844147]  [<ffffffff81aac062>] ? kernel_init_freeable+0x1f8/0x207
[    0.844498]  [<ffffffff81aab77e>] ? initcall_blacklist+0xb4/0xb4
[    0.844832]  [<ffffffff8105f63e>] ? finish_task_switch+0x8a/0xd2
[    0.845167]  [<ffffffff813eed79>] ? rest_init+0x70/0x70
[    0.845457]  [<ffffffff813eed7f>] ? kernel_init+0x6/0xd7
[    0.845753]  [<ffffffff813fdf18>] ? ret_from_fork+0x58/0x90
[    0.846064]  [<ffffffff813eed79>] ? rest_init+0x70/0x70
[    0.846405] Kernel Offset: 0x0 from 0xffffffff81000000 (relocation range: 0xffffffff80000000-0xffffffff9fffffff)
```


I'm not too sure about pulling in Profile2's SakaiProxy just for the sake of getting a config property. What if Roster gets loaded before Profile2, if that can happen? I'd just read the Profile2 prop in the roster code myself and have one less coupling.


Again, it seems like we're just performing a tell.  I know the GoogleObjectRoleRetriever will respond with a parent ! <response> of it's own.

So don't we need to add the following to the recieve listener?
case role: String => TODO1*
case GcsFailure(status: StatusCode, message: String) => TODO2**

where:
TODO1 sends the role back up to  authorizationsForEntity (with the caveat that I don't know exactly how, see above?)

TODO2 bubbles the error back up to PerRequest for requestContext completion


> FWIW this issue will be tackled on the MS side but currently not planned in the pending MS Edge release.

No surprise there; it was first [reported](https://connect.microsoft.com/IE/feedback/details/802251/script70-permission-denied-error-when-trying-to-access-old-document-from-reloaded-iframe) against the _IE11_ release preview.

> everyone was OK with it having a reduced blast radius (being for iframes only)

"OK" is [a bit strong](https://github.com/jquery/sizzle/pull/231/files#diff-88f3c23061b4e51e832a730dc0d97356R478), but point taken. I would not object to once again applying it to all browsers.


This $Id$ string is what caused the FUZZ check to fail. Please remove it.


Why change CFLAGS (a standard way to handle the flags throughout most of the settings files)?


## ReSharper support
If you love the productivity suite from JetBrains the new `.DotSettings` file will make your life much easier. Includes:

* Setting to configure ReSharper to the conventions of Signum Framework
* All the *VS snippets* has been translated to ReSharper *Code Templates*, using all the cool features like **automatic capitalization for fields**  (`name` -> `Name`... Weeeee!!), intelligent Availability, etc...

### How to Install

`Resharper -> Manage Options -> Solution (Team - Shared) -> (+) -> Open Setting`

And select the file:

`$MyApp$\Framework\Snippets\Signum.Framework.DotSettings`


ack


Is that true for KK, MK and MA?


We should endeavor not to include a `case` querystring param at all if it's not true.


Wouldn't an enum be easier to read?


@bqpd, this is on a branch -- hesitant to even open a PR for it. I just changed 'Parameters' to 'Arguments' because I think that terminology is more standard. ok to merge w/o PR?


This is not necessary on my system (Mac OS X 10.10). Without your patch, the modules are correctly installed to `/.../install/libexec/python/site-packages/smtk`. If you look in GNUInstallDirs.cmake (where `CMAKE_INSTALL_LIBEXECDIR` is defined), you'll see that it specifically does **not** contain `CMAKE_INSTALL_PREFIX`. There is another variable named `CMAKE_INSTALL_FULL_LIBEXECDIR` that does contain the absolute path. Perhaps the superbuild is incorrectly setting `CMAKE_INSTALL_PREFIX`?


url_for() has an _anchor kwarg for this: http://flask.pocoo.org/docs/0.10/api/#flask.url_for


@gustavoguichard Awesome! Thanks :)


Please protect these settings with a guard for lower version - AIX 5.3 has all these functions.


Fixed with https://github.com/mongodb-partners/mongo-rocks/commit/022edcaa45003888f082c1c9c8969d8cdfa95e0c

:)


instead of `href="geojson.js"` it should be `src="geojson.js"`


This is a good move, and one I think I attempted before but couldn't complete.

It seems there's a domain concept and object that needs to come out of the repeated 'hook data' pattern. Could we simply make an Action right away at that point? Or could we just wrap the original hook with something that returns the `source_location` of the original block from the user's code?