[![Nineteen Lincolns](http://api.artsmia.org/images/111203/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=111203)


[![Nineteen Lincolns](http://api.artsmia.org/images/111204/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=111204)


[![Qur'anic board](http://api.artsmia.org/images/111379/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=111379)


[![The Kuehn Children, Tyrol](http://api.artsmia.org/images/111869/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=111869)


[![Army tunic](http://api.artsmia.org/images/112158/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=112158)


[![Portrait of Georg von Roggenbach at age 38](http://api.artsmia.org/images/112644/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=112644)


[![Sea Circus (RGB branch)](http://api.artsmia.org/images/112652/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=112652)


[![Sea Circus (shell & ball)](http://api.artsmia.org/images/112653/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=112653)


[![Muhammad Ali](http://api.artsmia.org/images/113343/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=113343)


[![Gelede mask](http://api.artsmia.org/images/114437/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=114437)


[![Man with Birds](http://api.artsmia.org/images/114904/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=114904)


[![Bones II, from the "Deep South" series](http://api.artsmia.org/images/114969/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=114969)


[![Enrique Dimitri Magnussen](http://api.artsmia.org/images/115016/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=115016)


[![Maksim Obruchev](http://api.artsmia.org/images/115021/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=115021)


[![Diana Ginzborg](http://api.artsmia.org/images/115025/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=115025)


[![Abbott & Cordova, 7 August 1971](http://api.artsmia.org/images/115352/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=115352)


[![Ecce Homo](http://api.artsmia.org/images/115357/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=115357)


[![Untitled (Atomic Bomb Test Crater, Nevada, U.S.A.)](http://api.artsmia.org/images/115770/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=115770)


[![Untitled (Atomic Bomb Test Crater, Nevada, U.S.A.)](http://api.artsmia.org/images/115771/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=115771)


[![Atomic Bomb Exploding, Bikini-Atoll ("Bravo" Blast, 1st March, 1954)](http://api.artsmia.org/images/115772/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=115772)


[![Atomic Bomb Exploding, Bikini-Atoll ("Bravo" Blast, 1st March, 1954)](http://api.artsmia.org/images/115827/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=115827)


[![Atomic Bomb Exploding, Bikini-Atoll ("Bravo" Blast, 1st March, 1954)](http://api.artsmia.org/images/115828/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=115828)


[![Portrait of Jan Nicquet](http://api.artsmia.org/images/116509/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=116509)


[![Calla Lily](http://api.artsmia.org/images/116528/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=116528)


[![Calla Lily](http://api.artsmia.org/images/116530/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=116530)


[![136 Mini Film Cameras in the Smithsonian Institution History of Photography Collection With Old Style Photoshop Filter](http://api.artsmia.org/images/118837/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=118837)


[![Three German Soldiers](http://api.artsmia.org/images/118943/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=118943)


[![Saint Jerome](http://api.artsmia.org/images/118947/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=118947)


[![Saint Jerome, unfinished](http://api.artsmia.org/images/118948/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=118948)


[![Former Tea House in a Park next to the Afghan Exhibition of Economic and Social Achievements in the Shah Shahid District of Kabul, Afghanistan:  Chronotopia](http://api.artsmia.org/images/119397/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119397)


[![Second-hand Reading](http://api.artsmia.org/images/119599/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119599)


[![The Great Standard Bearer](http://api.artsmia.org/images/119658/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119658)


[![The Watch](http://api.artsmia.org/images/119768/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=119768)


[![St. James](http://api.artsmia.org/images/120054/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120054)


[![Nocturne:  Palaces](http://api.artsmia.org/images/120316/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120316)


[![Effet de Pluie](http://api.artsmia.org/images/120643/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120643)


@jackdpeterson Can you squash all your commits into one patch please? (Or multiple commits if appropriate)


Do that in the same patch were you added -std=c++11. In fact do a c++11 only patch at the root of your patch stack.


This won't be necessary. Already fixed the bug. Also tested it in Firefox.


Did you publish the livedb change this appears to necessitate? I [don't see it yet](https://github.com/share/livedb/blob/bb0373507cb741ab5ee8a2bcb9b160ae51ca8646/lib/memory.js#L136).


This shouldn't be done/merged in this way. Have you tested it in other languages? Note that Telegram is an international messenger. At least in Korean, this prevent message being sent when users press enter key. See #52


rax retry tests


r+


Is String#valueOf(Object) really needed here? args[0] is already a String.


precardused时机加flag


Ugh


Pattern-match on the result of `makeCovProcNoise dt s_a` to extract the `L.V4 (L.V4 Double)` inside it.


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2092 of 2182 tests passed)

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


saw approval from jdm
at https://github.com/gilles-leblanc/servo/commit/70aa253a65317b5ed11ac7013c648faa71e19144


merging gilles-leblanc/servo/issue-5063 = 70aa253a into auto


gilles-leblanc/servo/issue-5063 = 70aa253a merged ok, testing candidate = 1a73766d


@asparke2 while you are cleaning stuff up, can we remove the Vagrantfile and Berksfile?  I don;t think these do anything for this project right?


应该是升中间版本


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

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


This patch has been applied in u-boot mainline master. When do code submit, just need to rebase to mainline master.


@jhendrixMSFT , @stephentoub -
This is the way I'm heading for the combinatorial tests.  Unfortunately, I'm a bit worried that any failures are going to be "mysterious", because I'm passing around unadorned `Func`s (which don't print out nicely), or `ParallelQuery<T>`s (which all look alike...).  Should I do something like wrap these in a pair/tuple with an originating string (with the assumption that would also get printed, or maybe a custom type or two), or just go ahead and leave it for checking via debugging?
Also, what do you want me to do about long-running tests?  For some stuff I've cut down on the upper-bound, but they're still running long (compared to originally).  Should I keep the sizes used for the original `OuterLoop` tests, or try to match the runtime instead?  (I wish you could add traits to individual test runs or the output of a `MemberData` - I'm annoyed at the need to put in a wrapper)


@Fsu0413 这样弄明显能给扩展带来简洁。。。追求简洁不是一件好事么？
我们的目的就是为了增强用户体验，如果为难用户那么岂不是违背了这点？


#電卓


We could have left our CovProcNoise and worked with CovProcNoise's inside of `step` instead of M44's if we wanted type safety inside of step. That'd require making types for the intermediate parts of the computation, and (trivial) functions for the transitions, which just unwrap the matrices, do the multiplication, and rewrap. Probably not necessary in this case - if we're confident about the implementation of `step`, we can just guard the input arguments with CovProcNoise wrapper and trust that `step` does the right thing.


Gitception


this close #20


SetProperty and RemoveProperty both now do a copy-on-write of the entire property map while holding the write lock. Because it makes a copy, RemoveProperty effectively does a delete by not copying the property with key `key` into the new map.


These changes improved robot titles not handling of JSON errors.


Excellent work my friend.


A estrutura do código foi re-identado.


This seems like a bug, shouldn't it be <=?


That's making sense. I will merge this


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2172 of 2196 tests passed)

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


RemoveProperty() now mutates the map after letting Properties() hand out references to it. If you add a test like https://github.com/cloudfoundry-incubator/garden-linux/blob/7491ccc69d2d8f0a1ae211c71bd4296a542418dc/old/linux_backend/linux_container_test.go#L2095-L2106 but with RemoveProperty instead of SetProperty, I expect it will fail. We should have covered that case in the original PR.


Fixing my branch


How many different places in the views do you have the issue of displaying skills? Could a 'partial' be the answer?
http://guides.rubyonrails.org/layouts_and_rendering.html#using-partials
You have repeated the code to display skills twice in this view. The only difference is which container the skills are coming from - sounds like a parameter of a partial to me.


75127368_DCFSEC (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1644 of 1651 tests passed)

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


This is the bug. Move it into the conditional below. Ugh.


75127368_DCFSEC (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (634 of 635 tests passed)

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


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Yes please all branches I did last night would be great if they got merged and pushed to the server before 1 pm tomorrow please ;)

On Mar 5, 2015, at 8:26 PM, Winona Salesky wrote:

> @davidamichelson, this did not get merged into the master branch. Would you like me to do that?
>
> —
> Reply to this email directly or view it on GitHub.
>


Ah, it's great one more pair of eyes to review the docs...Thanks!


@asparke2 this should get moved into actual test tasks under a rake test target


This commit is called "restricted links based on signin/admin" but I don't see the conditionals that achieve this. Was it a poorly name commit? Perhaps I am missing something.


Merge conflict (found by SummerWish).


It's good to see this toned down. The first writing or making is usually bad. It's good to have a different perspective look at this too.


```haskell
h :: M24
z_ :: M41
(h !*! z_) :: M21
x :: M14
x !-! (h !*! z_)  --  <- type error, subtracting a 1x4 from a 2x1 I think
```


@jprichardson it was about matching the WIF compression flag with the compression of the public key in the `redeemScript`.

As of right now, it compares the EC points directly, ignoring this, but it may change in future so better to be consistent.


I think the condition should be (p->plane_count > 2), and the (p->plane_count == 2) case here is redundant.


+1, Looks good to me.


merge this to /server...


Nope. We deal with provider migrations for accounts manually right now, and don't allow changing the provider type during the reauth flow. If you want a seamless flow, you'll want the appropriate confirmation with the user that they wish the migrate their account, if it's associated with multiple users / developers.


good omega.


The commit messages from this Git repository are explained very well.
Go check.
On Mar 6, 2015 3:34 AM, "jgcaap" <notifications@github.com> wrote:

> hey, could you tell me where is from this linaro tool chain? Thanks
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/arter97/arm-linux-androideabi-4.8/commit/0f6acd2f68a5bbba5eb7d9766853398104aa5fc5#commitcomment-10057450>
> .
>


added login form #4


this has to be fixed before we merge this.


If BOARD is already an absolute path, won't this write the directory twice?


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

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


it seems like `BOARD=$(readlink -f ${BOARD})` would suffice.


Hi Emma, this file should not be included in the `compiler` directory.


I think this bit of logic should live in an extra function somewhere, that binds a fbotex to a certain texture. This would allow us to avoid the messiness of “overwriting” tex0 or something.


Oops, it's removed in the latest commit


Closes #35


If finish_pass_fbo is reworked to not automatically set any textures unless requested, this could be done by just binding the chroma_merge_fbo to tex1 up above.


75127368_DCFSEC (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1644 of 1651 tests passed)

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


hello!I Have another quetion!
how to map like this? User.Role.Permission

class User{
public int UserID;
[ResultColumn] public Role Role;
}
class Role{
public int RoleID;
public string RoleName;
[ResultColumn] public Permission Permission ;
}
class Permission{public string PermissionName;}
when I use NPoco.Sql  "User.Role.Permission" Is Not Mapped, [System.NullReferenceException]
Select("User.*,Role.*,Person.*).From("User").LeftJoin(“Role”).On(...).LeftJoin("Permission").On("..")
when I use NPoco.Linq.IQueryProvider is can  mapped


Yeah, I think cutting "currently" is merited. I just did that in d0456d1. I read that email when Ryan sent it (and it's one of my favorite emails of his). However, as I talked about in #39, I'm trying to not comment specifically on DNSSEC's merits. In the context of this guidance, it could be a real distraction.


Don't worry, line notes send notifications too. =)


lll


A good practice in java is the use of CamelCase to name variables/methods/etc


Is it worth caching the output of mp_get_*_coeffs? (Just check to see if the parameters to that command have changed, and re-use the last calculated matrix instead)


style: ```color = vec4(..., color.a);``` could simply be ```color.rgb = ...```


all tests pass:
success: http://build.servo.org/builders/linux1/builds/944
success: http://build.servo.org/builders/linux2/builds/942
success: http://build.servo.org/builders/android/builds/980
success: http://build.servo.org/builders/mac1/builds/1159
success: http://build.servo.org/builders/mac2/builds/1160
success: http://build.servo.org/builders/gonk/builds/130


這邊應該不用裝 capistrano-rvm，遠端伺服器應該是使用 rbenv。


I'm confused by this too


We surely want to cache this somehow, as LUT computation can be quite slow.


fast-forwarding master to auto = 1a73766d


@wasabeef I have added the Apache License V2. Could you update your README.md? Thank you!


About half of this code is redundant and should be skipped when antiringing is disabled. I'll fix it later


Copy pasting means, there are a quite a bit of concepts that are foreign to our domain that we need to own and maintain in future upgrades.  Now this slows down our ability to upgrade to the newer versions.  I would expect that you would provide a helper class to support these bridge scenarios while changing the public interface :(


這邊建議拆成多行。


Great. Couldn't tell :) I haven't used GH in awhile.

On Thu, Mar 5, 2015 at 10:40 PM Eric Mill <notifications@github.com> wrote:

> Don't worry, line notes send notifications too. =)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GSA/https/commit/0ff7a3cebdfcf9546bfc205e85fd4f730978ab3f#commitcomment-10064783>
> .
>


@williamldennis `master` has a method that can capture the video frame and process as a photo. it was derived from the code above and another pull request. try using that, let me know what you think in terms of quality. i've been considering updating it to not use CoreImage too.


I see you modified the root check a little bit from what I had. Great job though


Doesn't work(works only in local network)
don't know how fix this :c


這邊也改成跟 linked_files 一樣的寫法即可。

使用 fetch(:linked_dirs, []).push 是為了追加設定進去 linked_dirs，我們應該不需要。


inconsistent indentation when continuing arguments on the next line


You're a pro.


I had to read this method to understand what it's for. Change its name to something more descriptive such as "render_request_buttons"


"Forthcoming." :(
On Thu, Mar 5, 2015 at 22:11 Matt Post <notifications@github.com> wrote:

> Packing phrase tables now works without prepending [X]
>
> The packed grammar format requires each trie node to fit within 2 GB under
> a trie indexed by the first character, so prepending the [X] to each source
> side *before* packing was problematic. Now, we pack the regular phrase
> rules, and only add the [X] when loading the grammar and using the rules.
> This lets us pack much, much larger phrase tables without the forthcoming
> pack-multiple-grammars fix from @jganitkevitch
>
> Also got rid of "phrase" format keyword (must use "moses")
>
> Some bug fixes with computing the alignments
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/joshua-decoder/joshua/commit/
> 368b7f34b737a4c97f0a1ab2e12eb44617a106de
>


develop (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1139 of 1139 tests passed)

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


其他用不到的設定也可以砍掉，這點看個人習慣。不過上 production 的話建議還是整理乾淨。


Something being unclear to *you* does not make it a bug.

The use of strings is to be avoided, because they are inefficient and interface code needs to have low overhead. This change will be rejected.


I think this API is suboptimal. Instead, consider this semantic change:

1. Rename the function to something like “pass_sample_scaled”
2. Have the output variable and source texture/coord passed as parameters

In effect, this would replace “output = texture(input, coord)” by pass_sample_scaled("output", "input", "coord");

So in the case of chroma scaling, this might directly be called with something like pass_sample_scaled("color.gb", "texture1", "texcoord1"); or whatever.


A controller must not directly access a repository, only through the corresponding service.


A solid choice, sir, to comment out the items so they can still be used for testing purposes.


Gracias Loli Delgado de gran ayuda :)


This is my first real entry into my README file so we'll see what this has to offer.  I'm a learnin' me some code so let's hope for the best.


Should parse_slice() lose some of the code that it would be duplicating with jv_array_slice()?


那也明显不是这么加的啊……至少把skillname带上啊


I don't know if we should be using time to determine when to launch servers rather than events and state.


deja vu. kinda.


Awesome! I pulled to see everything was perfect. I'll send your payment soon as possilbe. ;)


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1142 of 1142 tests passed)

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


I'd prefer to not flag nodes as seeds but instead have a seeds list in the CassandraCluster object. I'm under the impression that seeds won't change often.


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (634 of 634 tests passed)

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


What does this buy us?


Handle the entity links in the API itself by searching for links


Is this intended? What happens when obj.get_ctnrs get called on a system?


Nah, it's fine. I've made that mistake too.

@svenssonjoel you should reset your local repo to delete this patch too, so it doesn't creep back in with a merge.


PrettyGridView (kbenne) - i386-Windows-7-VisualStudio-12: OK (2182 of 2204 tests passed)

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


Already merged I believe
On Mar 6, 2015 7:07 AM, "Paul Peavyhouse" <notifications@github.com> wrote:

> @jongerrish <https://github.com/jongerrish> Thanks for this change.
> When will this be pushed to 3.0-SNAPSHOT?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robolectric/robolectric/commit/599cfa4b5abf4a3eafe8889da66f3785da810553#commitcomment-10062086>
> .
>


Cool. :) I mentioned this just in case it'd be helpful.


If the responses table in the db has a composite primary key or unique key defined then yes. Otherwise it's a bit chicken and egg: I think since the Task 6 user testing we made the button to respond go away if there is already a response by that user, denying them the ability to respond again.


I'm an optimist!


Please move `if (serverTaskStatusOutstanding) {...` from `performHealthCheck` out so that `performHealthCheck` doesn't have a dependency on the driver.


the else is superfluous


Why do you implement this Service and never use it ?


I already made honza aware of this option for corosync and he will look
at it for corosync udp. It´s tricky to backport tho because of how
corosync init the sockets, but doable.

On 3/6/2015 1:26 AM, Andrew Beekhof wrote:
> We really need to get this into rhel for osp
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fabbione/kronosnet/commit/4d622d54755ccd7eac8b45dedef1c6b96aa90f0c#commitcomment-10063059>.
>


I might just manually go in and adjust the database to avoid having to drop everything. Will be more involved than just adding a row, but it would save our data so...


All return points of `startJob` should send some sort of taskStatus. Please change this method to return `TaskStatus` which can then be sent by the driver, rather than having many points where `sendStatusUpdate` can be called.


Looks like ```logged_in_user``` is very similar in three different controllers. Not very DRY. Could it be re-written to work for all three and moved somewhere more general - say the ApplicationController?


And it isn't preceded by `var` making it an unused global.

I think `"use strict";` is in order.


We probably want to have some detail here so the scheduler can intelligently handle if a job couldn't start because a process wasn't running.


bingo


Options.Options

See how crazy that looks @HaoK


Oops I spelled the file name wrong... The command should be `./yada -i inputs/ladder.2`. But with this `sto` commit even `genome` aborts, so does `intruder`. Did we change anything in `TransItem`?


What?


@arthurvr You up for any pull requests? Your comments are spot on.


develop (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1142 of 1142 tests passed)

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


Also this entry assertion is almost exactly the same as what's done in `startJob`. Are (can) they actually be the same and be factored into a helper method to avoid duplication.


#668 and not #688 :P


Where in the app is this code triggered from? I don't remember being able to destroy/delete a user.


実装したUIが想像以上に重くて、使いものにならない


If there is no current job running we should probably return a status stating such.


@pboyer If I have a blank workspace and I add a node, the behavior should be for that workspace to execute immediately. This kills that. Can I disable slienceNodeModifications for the node addition case?


+1
as no actual code change ...


> "Seed-assignment commit. I'm not sure if I wrote the comment.create! correctly for the unique faker data"

Did the data seed correctly?


75127368_DCFSEC (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1142 of 1144 tests passed)

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


```<% if not request.responses.empty? %>``` is an alternative that does not result in iterating over every response just to determine that there are more than zero.
```[1,2,3].any? {|x| x}```     ->     true, but iterates over all elements of the array
```[1,2,3].amy?```                      SAME as above - the omitted block is added by ruby
```[].any?```                        ->    false
```[1,2,3].empty?```            ->    false
```[].empty?```                    ->    true


This feature is new in Bloc and a bit raw in my opinion.  What are you thoughts on it?


Would be good to copy this test, and do `sourcepath = compileJava.classpath` to verify that works.


You spelt test wrong!


code.LINE( @"		self.HTTP_%@( requestURI ).PARAM( [self.req objectToDictionary] )%@;", [self.method uppercaseString], fileListParam );

这个怎么理解？


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/098229d46100b6d5c96956f308e08b41d474f06f


merging basho/riak_cs/refactor/assert-util-error-empty = 098229d4 into borshop-integration-1078-refactor/assert-util-error-empty


basho/riak_cs/refactor/assert-util-error-empty = 098229d4 merged ok, testing candidate = 7283e3b9


@adammurdoch do you want `@Incubating` here?


With the above patch, both r and s could have leading zero byte(s),

The problems come from converting to ASN.1, that treats r and s an INTEGER.
The value of nLen (the base point order n) is not provided, so leading zero bytes may be lost.

PKCS#11 in the same paragraph says: "For signatures created
by a token, the resulting signature is always of length 2nLen."

And PKCS#11 goes on to address our problem:
"Note: For applications, it is recommended to encode the signature as an octet string of
length two times nLen if possible. This ensures that the application works with PKCS#11
modules which have been implemented based on an older version of this document.
Older versions required all signatures to have length two times nLen. It may be
impossible to encode the signature with the maximum length of two times nLen if the
application just gets the integer values of r and s (i.e. without leading zeros), but does not
know the base point order n, because r and s can have any value between zero and the
base point order n."


How long should genome take when run without arguments?

On Thu, Mar 5, 2015 at 11:22 PM, Yihe (William) Huang <
notifications@github.com> wrote:

> Oops I spelled the file name wrong... The command should be ./yada -i
> inputs/ladder.2. But with this sto commit even genome aborts, so does
> intruder. Did we change anything in TransItem?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nathanielherman/sto/commit/c5ef5d7253b272e533386045cb887866a5a03476#commitcomment-10065020>
> .
>


It should be, yes. Haven't thought about it actually


Agreed.


@changokun, i keep seeing this kind of thing scattered about, and i'm curious about it. can you explain your intent, here? i feel like i'm missing something.


4 seconds at most?
On Mar 5, 2015 11:37 PM, "Eddie Kohler" <notifications@github.com> wrote:

> How long should genome take when run without arguments?
>
>
> On Thu, Mar 5, 2015 at 11:22 PM, Yihe (William) Huang <
> notifications@github.com> wrote:
>
> > Oops I spelled the file name wrong... The command should be ./yada -i
> > inputs/ladder.2. But with this sto commit even genome aborts, so does
> > intruder. Did we change anything in TransItem?
> >
> > --
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/nathanielherman/sto/commit/c5ef5d7253b272e533386045cb887866a5a03476#commitcomment-10065020
> >
> > .
> >
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/nathanielherman/sto/commit/c5ef5d7253b272e533386045cb887866a5a03476#commitcomment-10065129>
> .
>


What is this "Merge remote-tracking branch". Is this a development outside of python and workshop branch?


develop (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1142 of 1142 tests passed)

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


Trying to follow http://semver.org/


unfortunately, I wasn't able to trigger the bug in the test, which is worrying to me.  In the vim demo, with these contents, head is 3 and lineLength is 0.  In the test, it's line 0 which has length of 5.  I wasn't able to determine why.


Hmm… I can take a look tomorrow

On Thu, Mar 5, 2015 at 11:40 PM, Travis Heppe <notifications@github.com>
wrote:

> unfortunately, I wasn't able to trigger the bug in the test, which is worrying to me.  In the vim demo, with these contents, head is 3 and lineLength is 0.  In the test, it's line 0 which has length of 5.  I wasn't able to determine why.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/codemirror/CodeMirror/commit/10ac9368607696e14f2789bf184f3659407d4bc3#commitcomment-10065150


This is my favorite piece of documentation ever.


These are the tracks of my study and notes to remind me of what I`ve learned.


Yikes. I guess node modification includes node collection changes? That was
my incorrect assumption here. Let's talk tomorrow.

On Thu, Mar 5, 2015, 11:27 PM Ian Keough <notifications@github.com> wrote:

> @pboyer <https://github.com/pboyer> If I have a blank workspace and I add
> a node, the behavior should be for that workspace to execute immediately.
> This kills that. Can I disable slienceNodeModifications for the node
> addition case?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DynamoDS/Dynamo/commit/17d52e2f7c5c3a6a0ec849e236b004672963182b#commitcomment-10065061>
> .
>


Nico buenos dias:
Necesito un favor.. ando medio complicado con los horarios y necesito
ponerme al dia con los topics..Por eso te pido si me podes mandar un
ejemplo de como usar underscore.. y uno de como implementar json porque no
llegue a verlos bien y me estoy atrasando demasiado me parece...hasta
recien estuve viendo dust y handlebars y masomenos pude hacer funcionar
algo (que es lo que te subi al github).
Si esto puede ser por la mañana te lo agradezco asi cuando vuelva de aleman
tipo 12:30 me pongo a verlo y tratar de terminar el topic 04.
Muchas gracias y disculpa la molestia a estas horas.
Saludos.

On Thu, Mar 5, 2015 at 9:41 AM, nicolasronsmansglobant <
notifications@github.com> wrote:

> Good work!! Except the jQuery shim, the overall implementation is good!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/joaquinferroni/Bootcamp/commit/0ef7166951a0d748100ebe11a3a01cad5dd35c1d#commitcomment-10050402>
> .
>


I'm not crazy about this method it creates odd coupling between this and ApiController.

We can revisit in latter though.


not aliases. subclasses. so, this would be useful if, say `LeaveCommand` was slightly different than `GoCommand`. like, maybe added a slightly different message or something.

in ruby, classes are objects, too, so if you just want to alias a class (in this case, for the purpose of having multiple command words which do the same thing) you can just assign them to a new variable.

```ruby
class LeaveCommand < GoCommand; end   # => subclass
LeaveCommand = GoCommand              # => alias
```


82420068_GridViewMultiEdit (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

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


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/863


这个提交错了,后面这个不需要更新,只是因为我们项目是直接取值,


retest this please


thanks for catching this!


Make this method private.


pointless return statement


Thanks @piranha, you've given me pause for thought.  I'm leaving both `dispatch` and `dispatch-sync` in place.  Up to you which you use :-)    BTW,  v0.2.0 is better at avoiding the horrors of  `async.core` effected stack traces.


気づくのが遅れました。
治ったとのことでよかったです。

試行錯誤を繰り返す時はgit でローカルコミットをしておくのをお勧めします。
差分が追えるので、何が悪かったのか一目瞭然です


What do you think about using a library like https://github.com/rs/SDWebImage and replace the above with:

```
if (avatarItem.avatarImageURL) {
  [self sd_setImageWithURL: [NSURL URLWithString:avatarItem.avatarImageURL]];
} else if (avatarItem.avatarImage) {
```


Cherry Clears here, and yeah.. it's better, but it still happens :(


Could you actually release this please? I'm hitting an issue with this old version of `bcrypt`.


In all seriousness though, i have improved the aesthetics of the page, the CSS seems to work fine except on really small monitors (will need a separate mobile version). Also i have added a checklist site for viewing which teams have yet to be pit scouted or photographed. Server bulletproofing and client error detection, as well as adopting the code to the new scouting sheet has yet to be finished.


Yuck.
```
def no_more_yuck(request, response)  # find a non yucky name
  assignment = request.assignments.find_by_user_id(response.user_id)
  comments = !assignment.comments.blank? if assignment
  return "Your Comments: #{comments}" if assignment and not comments
  return "Message: #{response.message}" if response.message
  ""
end
```


fast-forwarding release/1.5 to borshop-integration-1078-refactor/assert-util-error-empty = 7283e3b9


Yeah I guess so, if people are actually holding onto IOptions instead of the actual instance, so:

IOptions.Value and IOptions.GetNamedValue?  Ugh yet another rename :/


Counting twice? There's already a badge at the bottom of the readme.


Where is Web\Controllers\ComponentsController.cs?  It's referenced from the .csproj but not included.


Yeah this looks really weird,  does this exist just so you can hang ConfigureForm from it?


Given that both items are `Integer` it's will be confusing trying to remember which tuple value is seeds and which is nodes. Create a more expressive type to be the return type for this method.


Maybe we recommend that people unwrap? The problem is the T isn't injected directly so this only works with ctor injection.


![screenshot 2015-03-05 21 15 38](https://cloud.githubusercontent.com/assets/422381/6520510/01dc4360-c384-11e4-9c15-661809c1fe6e.png)


If we just want to hide it, maybe just have ConfigureFormTagHelper live in the Mvc.TagHelpers namespace instead of in DI namespace like all of the other top level framework Configure methods?


D'oh, I've avcidentally reverted a piece of your previous patches. Sorry.


Okay I see, so we want services.ConfigureTagHelpers().ConfigureForm(), hrm...


We want a "home" for tag helper options configuration methods, rather than having them all off of `IServiceCollection`


BTW this is just the first, i'm sweeping through all the tag helpers and creating options objects where they add value.


Why is Tag helpers special here?


I'm not sure if it is a good idea to ignore the `idInjection` settings from the base model. For example, we have a `TransientModel` with `idInjection` set to `false` and its sub-models expect to not inject an id.


See comments lower down


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

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


Unused import


Need to import `android.os.Build`


Rename to `wasOpened` to be consistent with the other variables


试试


We wouldn't normally use an `||` statement in a test like this.

Also, this test isn't triggering the validation. Instead try something like this...

```ruby
it 'only allows -1 or 1 as values' do
  @vote.value = 'a'
  expect(@vote).not_to be_valid
  expect(@vote.errors[:value]).to eq('must be 1 or -1')
end
```


PS I'm rather hoping you weren't depending on ```nil``` being returned for any logical tests. If you were, then those tests would have to be changed (given my method) to test for ```!blank?```.


Replace `11` with `Build.VERSION_CODES.HONEYCOMB`


So identity had the same issue, where we had a bunch of sub options, identity chose to just have all of the sub options hang off of IdentityOptions, so if tag helpers did the same thing, it would something like.  The config binding looks nice and just works for free this way too
```
public class TagHelperOptions {
   public FormHelperOptions Form {get; set; } = new FormHelperOptions();
}

services.ConfigureTagHelper(o => o.Form.FormThingy = true);


It seems as though you're missing the `Vote` model and the changes to the `Post` model to make these tests pass.

What message do you get when you run `rspec`?


This is superseded entirely by #123.


Use <kbd>Tab</kbd> instead of spaces.


I think this is a bit distracting since they're not terribly complicated either way.


Something is off here. Your indentation is messed up and there is an ```end``` in a weird place.


This parenthetical “(utf-8)” is misleading. Unicode is different from UTF-8. A Unicode string is a sequence of characters U+0000…U+10FFFF. UTF-8 is one encoding of Unicode as sequences of bytes 0x00…0xFF. This library (rightly) accepts Unicode characters, not UTF-8 bytes.

```python
>>> ad.detect_alphabet(u'Cyrillic and кириллический')
set(['LATIN', 'CYRILLIC'])
>>> ad.detect_alphabet(u'Cyrillic and кириллический'.encode('utf-8'))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "alphabet_detector/alphabet_detector.py", line 25, in detect_alphabet
    for char in unistr if char.isalpha())
  File "alphabet_detector/alphabet_detector.py", line 25, in <genexpr>
    for char in unistr if char.isalpha())
TypeError: must be unicode, not str
```


thank you!


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

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


Does this work? Is it some weird pattern I don't know about? ```end``` within div tags. Are you sure about this?


Would it be simpler to just put last_force_resend in op_target_t?


@l4u the arduino download link is broken


please be consistent with the https://github.com/ceph/ceph/blob/master/CodingStyle. to be specific, you might want to an space before PGP_NUM. would be better to have an individual line for each enum constant.

but i don't see lots of merit of introducing enum here other than translating "if {} else if {}" into "switch case", but i am okay with it.


Agreed - I suggested a re-write of another yucky method, and those ideas could help here.


Would require to change the min.js also. Anyway let me know when you have updated the code so we can update composer.


Could do something like this, also much nicer for tests to directly stuff in an options instance instead of the gross IOptions mocking
```
[Activate]
protected IOptions<FormTagHelperOptions> OptionsAccessor { get; set; }

private FormTagHelperOptions _options;
protected internal FormTagHelperOptions Options {
  get { if (_options == null) { _options = OptionsAccessor.Options; } return _options; }
  set { _options = value; }
}


we use postfix of `_t` for typedef'ed type names.


tidy up


Swamp Mirrorer cannot be SSed while Barrier Statue is active


has_item<true>() only gets called in read_only_item(). It seems like it'd be more clear to have has_item and has_read_only_item() then always passing a template arg?


Yeah this always felt gross to me, seems nicer to just let tests directly set an options instance instead requiring the mocking the IOptions service.  I think @divega disagreed with me on this one too though, not sure if we got consensus on this one.


<index: arrays="tab/tabs"> maybe?
or
<index: arrays="tab"> also is OK?


could add a space after `if`


Just to be sure I'm decide to leave it this way as I need it for AI too. Though in future this one likely going into callback as currently it's kind a use data it's shouldn't use.

But in this case it's not critical.


Thanks!


Yes, actually. Upstart is a typical pronoun with typical capitalization, but systemd is actually supposed to be all lower cased. There's even a spelling section on the systemd project page: http://www.freedesktop.org/wiki/Software/systemd/


Moved thread specific stuff from ...__apply_edit functions.


I'm going to poke around YT over the next day to try and find any edge cases I missed.  Expect to push a new version this weekend.


This is missing an `a`. Should be `edd-invalid-customer-user_id` ?


LGTM


I had to figure out how to fix it so that they would pass in the Voting
lesson. Got them all to pass, no worries.

On Fri, Mar 6, 2015 at 12:16 AM, Felix Clack <notifications@github.com>
wrote:

> It seems as though you're missing the Vote model and the changes to the
> Post model to make these tests pass.
>
> What message do you get when you run rspec?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kellobri/bloccit_project/commit/d7bb6630681ec0c9f8c7f7e4ad40c8eb5a9c3edc#commitcomment-10065457>
> .
>


Looks like we don't need save here and this function might be further specialized on maintaining Thread --> Tag relations.


Looks like isaacs/node-glob#139 is resolved now @ericf


i am not sure that we should assert in this case even user is supposed to talk with monitor via the ceph command which will validate the the command before sending it over to mon daemon. allowing a malfunction client to bring down the monitor is not a good idea, i think.


PrettyGridView (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2182 of 2204 tests passed)

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


Much better name. :+1:


Was broken, and I don't expect I shall play that game again. I only played base wars anyhow.


I've commented this out because calling method on Thread.save caused errors in the test cases (that was the case before this commit as well).


personally, i'd suggest not encode the type name into the variable name. could use something like `found` instead.


and this variable name is not ideal, IMHO.


```
def render_apply_button(request)
  if request.user_id != @current_user.id   # is current_user an instance variable?
    # or is it accessible the way you use it in the next condition this without @ ? Does it matter? Be consistent!
    made_response = request.responders.include?(current_user)
    btn_label = made_response ? "Applied" : "Apply"
    link_to btn_label, new_request_response_url(request), :class => "btn btn-primary btn-xs", :disabled => made_response
end
```


Great.

—
Thanks,
Felix

On Fri, Mar 6, 2015 at 3:36 PM, kellobri <notifications@github.com> wrote:

> I had to figure out how to fix it so that they would pass in the Voting
> lesson. Got them all to pass, no worries.
> On Fri, Mar 6, 2015 at 12:16 AM, Felix Clack <notifications@github.com>
> wrote:
>> It seems as though you're missing the Vote model and the changes to the
>> Post model to make these tests pass.
>>
>> What message do you get when you run rspec?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/kellobri/bloccit_project/commit/d7bb6630681ec0c9f8c7f7e4ad40c8eb5a9c3edc#commitcomment-10065457>
>> .
>>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/kellobri/bloccit_project/commit/d7bb6630681ec0c9f8c7f7e4ad40c8eb5a9c3edc#commitcomment-10065571


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Chris i think the pick a card class will eventually become a menu of the possible melds the player will have to choose from that the sequence tester will print out the cards that fall into the meld criteria.  Looks pretty solid so far though. thanks for commenting your code.


I sort of mean as a different texturing option. Like how we have in the pre-existing system for the variations.


```haskell
x :: M21
h :: M24
z_ :: M41 -- I know there is no M41, but there should be
```
therefore

```haskell
 (h !*! z_) :: M21
```
and the subtraction should be fine.

In regard to the invertablity check, the only way that s would be noninvertable is due to the granularity of the pixels happening to produce a situation where the there is a randomly occur linear dependence between rows or columns of the matrix. An easy way to prevent this that might lead to a performance benefit (or might not) is to intentionally add some noise to the matrix.


Detach still gets called. I've had to do:
```ts
    /** override to do nothing */
    detach() { }
```


I don't think this is correct. The whisper on disk format doesn't handle fractional or signed integer seconds. Timestamps that don't fit into a uint32 shouldn't be valid afaict.


would be great if you could add a test case in `qa/workunits/cephtool/test.sh`!


The reason that x is a 2x1 is because we do not measure velocity, only position, but our model state (z_) has both vel and position and is therefore 4x1


Is this file still used?


Fixes #259


Travis is making work for you ;)


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1649 of 1649 tests passed)

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


Yep, this broke time zones for me.


If there were several images then they would have the same id, but this is forbidden by HTML5 standard. I think it is better to keep the "ttmlImage_" suffix and increment id part.


why toggle unconditionally?


One last thing is that it is probably not wise to simply return the old position in the very rare case that s is not invertable since we have new samples coming in, and s likely is not going to change much step to step. I would opt for simply using the old s. This, unfortunately in functional land, will result in yet another input argument...


la modif est à annulé. je la reprends proprement.

désolé pour le dérangement, je débute sur github.


kind of a lame commit message, @jacquestardie


I think we may need to make visual style a configurable option instead of hard-coding it here.


See any repeated code? would a partial for your partial help?


0という数字に名前を付けたい。


フィールドはprivateにしてgetter/setterを作ったほうがいいかも。


No changelog entry?


I would say this is the most important contribution I've ever made to this project.


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1643 of 1643 tests passed)

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


let's remove showMobile - and leave it as undocumented for those (few/none) using it


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1142 of 1142 tests passed)

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


is there a reason you're wrapping this in a group?


Can we make it so that if `symbolType` is provided, we use a static value instead of this function?


Seems to have no effect


@YangLeoZhao-Reflektion please review :dancer:


@salimane What was the error?

Can you try http://downloads.arduino.cc/arduino-1.6.0-macosx.zip


Use [gitignore](http://git-scm.com/docs/gitignore).


Nope. Bad habit from an older piece of code. Sorry.


Do you think wrapping in a `assert_deprecated` block where these methods get tested would be a good idea as well?


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/d5f4415ac99f1b3ca197f0faa04ce5fbb56156a9


basho/riak_cs/refactor/multibag-simpler-state-transition = d5f4415a merged ok, testing candidate = 751485d7


merging basho/riak_cs/refactor/multibag-simpler-state-transition = d5f4415a into borshop-integration-1080-refactor/multibag-simpler-state-transition


:thumbsup:


> And it isn't preceded by var making it an unused global.

Not true. This is preceded by var (the line above). It's a multiple variable declaration.


@sukima Sure.


What's a baking glass? lol


Events dispatched through eventBus are 'public' and listeners are attached via MediaPlayer.addEventListener API. This means that even name should be more specific so that people who used dash.js could understand that this event is related to captioning, nut buffer updateend or something else. In this case it could be "fragmentedTextUpdated". We put all public event names as constants into MediaPlayer.js in MediaPlayer.events object.


This is just for testing


@ash2k You use the cache? I'm about to switch to bcrypt.js #13 Does that work for you? Most of the requests related to this come from Windows people not being able to build bcrypt


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (635 of 635 tests passed)

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


TBH, I'm not really sure what's going on behind the scenes so I just made the warning vague. Feel free to add more details although I don't think they're necessary.


To answer my own question, I think this client cannot initiate and that you would need to do that from another client. For example in a browser at http://demo.openwebrtc.io


What's that? )

On Thu, Mar 5, 2015 at 5:34 PM, Necsi Vivien <notifications@github.com>
wrote:

> 'data';
> var s = [];
> if (schema.type === 'object' || schema.$ref instanceof Object ||
> schema.type === 'array') {
>
>    - s.push('');
>    - s.push(''); if (schema.title)
>    - s.push('' + _.T('formEdit'+schema.title) + '');
>    - s.push('' + _.T('form'+schema.title) + ''); // object if
>    (schema.type === 'object' ||
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/Vivi270/schema2form/commit/a8416ca8e3c1af666905341dd98c0e98ed3ed3af#commitcomment-10052376>
> .
>


Make sense to just make it a constant?


Gah, fixes #211.


Hi, I see you're using `encoding/xml` package for parsing HTML.

Is there a reason you chose it over [`golang.org/x/net/html`](https://godoc.org/golang.org/x/net/html) package which specifically targets HTML?


The same as above - event name should be more specific


> I have tried my best using a brush

Imo that did not work well.
I`d completely remove the branding anyway, because serves no functional purpose.
Displaying the branding in the GUI is a bad habit in DJ software we should no longer follow.


Tests say I'm wrong, closing!


I'm not rubyspec expert, but I heard rubyspec should be readable.
Is such programmable spec allowed?


test


@ImALuckyGuy @ChangeTip 0.5 bit


Hi @imaluckyguy, @Wilee999 sent you a Bitcoin tip worth 0.5 bit, and I'm here to deliver it ➔ **[collect your tip](https://www.changetip.com/collect/577672)**.

**[Learn more about ChangeTip](https://www.changetip.com/tip-online/github)**


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/866


congratulations ur rich!!111


Fixes issues #19 and #24


In main.cpp, there are errors: Undefined reference to 'EE::interpreter', Undefined reference to 'EE::state', Undefined reference to 'Memory::bios'


抱歉，发现第11行有个错误。 的 => 地 麻烦修改下吧。


the less data is put into intents the better. yes, please


You're welcome!


You can use:
`Core::VERSION::string`

and
`Core::Pro::VERSION::string`
`

Which includes product title and version number.


This whole snippet does not match the code style in the rest of the file.


HI , I am coding for Electric compass. I use file
https://github.com/kriswiner/FRDM-FXS-MULT/blob/master/FXOS8700CQ/FXOS8700CQBasicExample.ino
.

When I apply power I have  heding angle  = A degree, then I turn of power
and apply again I have heding angle  = B degree. Could you tell me Why A
and B diffirent?
if you can, please give me a project for electric compass use sensor
FXOS8700CQ.
mail: soncadpro@gmail.com
thank you so much.


Seems i forgot to rename to something more human friendly. (This is my only comment)


As the internal SRAM is 32k bytes, when boot up from MMC/SD card, only <= 24k bytes boot file (boot.bin) can be read correctly. So, in order to keep the u-boot-spl.bin less than 24k bytes. Using the following workaround.
--->8---
diff --git a/Makefile b/Makefile
index 9747bd2..8b6adfc 100644
--- a/Makefile
+++ b/Makefile
@@ -654,7 +654,7 @@ libs-y += $(if $(BOARDDIR),board/$(BOARDDIR)/)

 libs-y := $(sort $(libs-y))

-u-boot-dirs    := $(patsubst %/,%,$(filter %/, $(libs-y))) tools examples
+u-boot-dirs    := $(patsubst %/,%,$(filter %/, $(libs-y))) tools

 u-boot-alldirs := $(sort $(u-boot-dirs) $(patsubst %/,%,$(filter %/, $(libs-))))

diff --git a/include/configs/at91sam9x5ek.h b/include/configs/at91sam9x5ek.h
index cc1294a..da49328 100644
--- a/include/configs/at91sam9x5ek.h
+++ b/include/configs/at91sam9x5ek.h
@@ -256,6 +256,12 @@
 #define ATMEL_BASE_MPDDRC              ATMEL_BASE_DDRSDRC

 #ifdef CONFIG_SYS_USE_MMC
+
+/* Temp solution */
+#define CONFIG_SYS_ICACHE_OFF
+#define CONFIG_SYS_DCACHE_OFF
+#define CONFIG_SYS_THUMB_BUILD
+
 #define CONFIG_SPL_LDSCRIPT            arch/arm/mach-at91/arm926ejs/u-boot-spl.lds
 #define CONFIG_SPL_MMC_SUPPORT
 #define CONFIG_SYS_U_BOOT_MAX_SIZE_SECTORS     0x400
---8<---


fast-forwarding release/1.5 to borshop-integration-1080-refactor/multibag-simpler-state-transition = 751485d7


As the internal SRAM is 32k bytes, when boot up from MMC/SD card, only <= 24k bytes boot file (boot.bin) can be read correctly. So, in order to keep the u-boot-spl.bin less than 24k bytes. Using the following workaround.
--->8---
diff --git a/Makefile b/Makefile
index 9747bd2..8b6adfc 100644
--- a/Makefile
+++ b/Makefile
@@ -654,7 +654,7 @@ libs-y += $(if $(BOARDDIR),board/$(BOARDDIR)/)

libs-y := $(sort $(libs-y))

-u-boot-dirs := $(patsubst %/,%,$(filter %/, $(libs-y))) tools examples
+u-boot-dirs := $(patsubst %/,%,$(filter %/, $(libs-y))) tools

u-boot-alldirs := $(sort $(u-boot-dirs) $(patsubst %/,%,$(filter %/, $(libs-))))

diff --git a/include/configs/at91sam9n12ek.h b/include/configs/at91sam9n12ek.h
index cc1294a..da49328 100644
--- a/include/configs/at91sam9n12ek.h
+++ b/include/configs/at91sam9n12ek.h
@@ -256,6 +256,12 @@
#define ATMEL_BASE_MPDDRC ATMEL_BASE_DDRSDRC

#ifdef CONFIG_SYS_USE_MMC
+
+/* Temp solution */
+#define CONFIG_SYS_ICACHE_OFF
+#define CONFIG_SYS_DCACHE_OFF
+#define CONFIG_SYS_THUMB_BUILD
+
#define CONFIG_SPL_LDSCRIPT arch/arm/mach-at91/arm926ejs/u-boot-spl.lds
#define CONFIG_SPL_MMC_SUPPORT
#define CONFIG_SYS_U_BOOT_MAX_SIZE_SECTORS 0x400
---8<---


:facepunch:


The plugin shouldn't be making this decisions, it is what we created the ContentService for.


drtaiga, hello hope are well, I would like to ask for the information on his blog regarding the sensor Pyranometer


This method is required here as a means of indirection. As per your guidance, you want to be able to handle other event types in the future -- if they are not MoEvents, we need to be able to handle that at a higher level and route them to the appropriate handler. Without this, do we just assume that every event occurring will be an MoEvent and have it handled there?


sas


Images in the map subdirectory?


It's a class method. Could as easily be moved into a standalone method, and then it can reference anything it wants.


this is ContentService's concern


Use:

https://github.com/dradis/dradis-plugins/blob/master/lib/dradis/plugins/content_service.rb#L53-L55


This will break the parts that rely on the file in the build directory. (index.html and the concat task)


Use:
https://github.com/dradis/dradis-plugins/blob/master/lib/dradis/plugins/content_service.rb#L16-L18


@darktears the blink rev 1aee103dc09918ac467f939a16cc449d7a4d865b is not available on next branch.


I guess I was lazy. Feel free to PR ;)


ContentService


75127368_DCFSEC (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


It could also be MoDeletion, or MoRemove, or MoEliminated...


#17 resolved.


Using latest master @2fc4c5bdcad05ca020b620e5cb62b92455af0782 on OSX
Having Mic1-2, and Aux 1-2 configured, nothing is displayed when toggling MIC.
See picture in 1st comment.


Yeah we can give it a try


Banners also revert to black when you break a block while holding one, so this will fix it.


This makes MassTrans and Hashtable segfault because they try to turn an integer write_value into a string write_value. This treats the current value as std::string (when it's really an int), and thus segfault.

One way we could do this is to assert(!has_write()) in add_write() and have a method change_write() which takes both a new value and the type of the old value, free_packed's the old value and then puts in the new value (in the more common case that you just want to update a write value with the same type you can use write_value()).

'Course that makes it 3 different methods you might use to update a value... Dunno if there's a great solution to this besides just storing deconstructor pointers, the old way was that add_write() just expected you to have called free_packed if you called add_write() again.


Bad merge?


You can specify `.gitignore` as well.


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2164 of 2182 tests passed)

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


“Then you can use this union type to generate tokens inside lexer.” -- Maybe better “Then you can build an FsLex lexer to generate tokens of this very type.”

s/despite fact/despite the fact/
s/an union/a union/

“must be first on the files list before lexer” -- I'd rather say “must precede the lexer-generated files on the file list.” The first usually come the AST definitions.


Perhaps needs empty lines before and after (check https://github.com/belczyk/FsLexYacc/blob/master/docs/content/jsonParserExample.md for the final formatting).


`Assoc` in backticks.


Let us have fun


`codePointAt` has some error handling, too.


...then add this XML element after it


You can even append status icon of Travis-CI to indicate the build/verification status as [libchewing](https://github.com/chewing/libchewing) did in file [README.md](https://github.com/chewing/libchewing/blob/master/README.md).


@dkhamsing Thanks for adding my apps, but in terms of Parse Challenge App it already is in App Store: https://itunes.apple.com/us/app/lets-challenge-me/id944004497


@villesau this is where we use the `.jshintrc-spec` file. It was made for use with the `jshint:serverTest` task. Feel free to rename if you'd like to use it with another build system.


Don't mix independent changes into single GIT commit.


indent


Not perhaps important in this case, but the `.target` files usually come the last in msbuild project files. This way, the targets may examine and optionally imeratively modify the items, e. g. add properties to them.


75127368_DCFSEC (mjwitte) - x86_64-MacOS-10.9-clang: OK (1639 of 1645 tests passed)

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


This fixes it?


Please explain that what you put between the `%{` and `}%` marks is copied verbatim into the generated F# file.


This is when merging a branh into another one.
> On 6 Mar 2015, at 05:38, Omar Brown <notifications@github.com> wrote:
>
> What is this "Merge remote-tracking branch". Is this a development outside of python and workshop branch?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/MrKepzie/Natron/commit/f0e45bb91f2ba85790ea3c52bb117082b55ceafd#commitcomment-10065135>.
>


I can post an MO to the REST API that does not have any naming props, and just a DN and it handles it fine. Please discuss this with Sergey to determine what is the correct behavior


Terimakasih PRnya :smile: . Sudah saya merge. Kata Source code dan Sign out tetap saya pertahankan dalam bahasa inggris karena masih lebih terasa familiar dengan English


This plugin is nice but def. needs a little of cleanup, I will submit some PRs this evening.


Including files in project may require clicking on the "Show all files" option. I think this fact is worth mentioning. People of various VS skills may try to follow this tutorial.


A comment?


mainValue is taken from the time series data source query, should always be a number. What case do you have where you need this? Have you written strings to InfluxDB as values?


Did this really need the new Exception type?


Heh :+1:


whoa - "goobering" and "unborking" in one commit message. easy does it!


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1649 of 1649 tests passed)

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


Of course.


“Rules are made of list of productions.” -- maybe better “”each rule defines a list of productions.” if I understand your intent.


Guys,  I can assure you that it is needed by many people.

Actually I was compiling 5.0.5 3 days ago and I have added the patch manually (I must have missed information that you added it already). During compilation I got messages that patch has been already added (I don't remember exact wording), so I skipped it.

I am not sure what happened exactly,  but unless the patch is not in the upstream yet,  please don't remove it from OE.


s/an ordinary/ordinary/


Standart python carbon accepts messages with float timestamp in pickle format. When migrating to go-carbon messages by some customers no longer reach. I can give an example later.

I also think that it is wrong. But the standard carbon takes them


Oops?


I already did it yesterday night. :)


Wrong fix. Don't use that!


Okay Thanks


@TODO: Add check that timestamp between 0 and max_uint32


“tokens pattern”. I would rather call it a string of terminal (token) and non-terminals (production) identifiers. Up to you, but I perceive a slight oversimplification here. But up to you really.


Why was BLCraftingManager removed?

It was meant to allow for different crafting tables that used different recipes. And there is now no way to have recipes that go in reverse.


A bit confusing commit message as the previous commit did not touch ``client/puavo-dnssd-printer-client``.


simpleSearch
基于全文检索原理使用java的简单实现


I wouldn't include all those comments and setUp and tearDown.


Please revert all such changes in this file


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1143 of 1143 tests passed)

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


Please revert this change


Like at line -> For example, at line

Assoc is JsonValue union type -> It is a constructor of this type, not the type itself.


its easier to just export it to maya and fix it that way or just make a trigger offset


1. add `using System.Timers;`
2. explicitly mark field as `private`


perfect


@amitmurthy is this ok? I got this error in the tests:

```
     * examples            ERROR (unhandled task failure): unlink: no such file or directory (ENOENT)
 in unlink at fs.jl:98
 in rm at file.jl:57
 in manage at /home/jeff/src/julia/usr/share/doc/julia/examples/clustermanager/simple/UnixDomainCM.jl:72
 in deregister_worker at multi.jl:362
 in anonymous at task.jl:923
```


From other side if we will do it in separate method business logic contaminated by checking permissions


changelog (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2195 of 2212 tests passed)

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


Thanks for the tip! No reason in particular. The xml package just suited my needs and can be configured to parse html with no issues (e.g., escaping/unescaping and taking into account autoclosed tags). I'll look at the other package in greater depth soon. Looks like it may make it easier to get the corresponding source html for a token.


why remove this?


Hi,

that would be very much appreciated since I don't know a lot on the topic.
I don't see a problem with going 2.0 but would still like to keep the
changes to the API as small as possible.

Regards
Kalle

On Thu, Mar 5, 2015 at 11:47 PM, Gábor Bakos <notifications@github.com>
wrote:

> Hi Kalle,
> I am going to check the OSGi issue this weekend. It might be the case that
> there are incompatible changes, which would "require" a 2.0 release (in
> case you prefer to follow semantic versioning).
> (I am going to submit a bug report about not escaping in the ' and
> probably other characters too when Option#AS_PATH_LIST is requested. It
> might be not too hard to fix, probably I'll give it a try too.)
> Thanks for your work
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jayway/JsonPath/commit/96ab579bfc4291d097394a99283c824247943734#commitcomment-10061788>
> .
>


75127368_DCFSEC (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (635 of 635 tests passed)

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


Yay, nice work! :)

Hey, can you open Snap! And a web inspector and see what font is being used?


I do not really know how to better put that, but I do not think this is right to say that one rule “calls” another, as the language is purely declarative. Really, when the parser completes the `rev_object_fields` production, it also creates an `object_fields` nonterminal using the assigned action. Maybe that requires a little rewording.

Also I would not call the order in which the list is constructed “wrong” -- it is constructed in reverse order by design, as are list in many algorithms.


@gestadieu issue with the file. programs method is there twice, breaks any symfony task.


dont worry - the hostnames there still have their IPv4 address ;)


the standard indeed both defines +hhmi and +hh:mi, also Z could be used by people using UTC

I suggest we use:
```
private String isoDateFormat(String s) {
        if (NumberUtils.isDigits(s)) {
            // legacy mode
            int i = s.indexOf(' ');
            long time = Long.parseLong(s.substring(0, i));
            return FastDateFormat.getInstance(ISO_8601).format(new Date(time * 1000)) + s.substring(i);
        } else {
            // already in ISO format
            return s;
        }
}
```


Thanks for your contribution, it is a nice tutorial really. I'll review from the "Lexer" chapter down in a couple of next days.


Assuming you are not changing the orientation of the sensor, the sensor is
likely not properly calibrated. How are you calibrating the magnetometer?

-----Original Message-----
From: sonoi [mailto:notifications@github.com]
Sent: March 5, 2015 10:34 PM
To: kriswiner/FRDM-FXS-MULT
Cc: Kris Winer
Subject: Re: [FRDM-FXS-MULT] Create FXOS8700CQBasicExample.ino (30ad70c)

HI , I am coding for Electric compass. I use file
https://github.com/kriswiner/FRDM-FXS-MULT/blob/master/FXOS8700CQ/FXOS8700CQ
BasicExample.ino
.

When I apply power I have heding angle = A degree, then I turn of power
and apply again I have heding angle = B degree. Could you tell me Why A
and B diffirent?
if you can, please give me a project for electric compass use sensor
FXOS8700CQ.
mail: soncadpro@gmail.com
thank you so much.

-
Reply to this email directly or view it on GitHub
<https://github.com/kriswiner/FRDM-FXS-MULT/commit/30ad70cab499a138df6e757c8
823f77e0b9c27aa#commitcomment-10065940> .
<https://github.com/notifications/beacon/AGY1qhJjDmEsyjuYMhvAQ-FOtGtd-V2Eks5
nyUHhgaJpZM4Cv3_i.gif>


I think, it will perceived much worse.


@justinstern I don't think `WC_Coupon::get_coupon_error()` and `WC_Coupon::get_coupon_message()` are interchangeable :smile_cat:


test


Could you add constructor overload which takes timer interval as third argument and move timer's initialization code in that constructor. As a result end-user should have two options of creation `ModbusTcpSlave`:
1. without timer (existing constructor should be used)
2. with timer (yours constructor overload should be used)
Also don't forget to add correspond overload of factory method `CreateTcp`


88621024-ElectricLoadCenterStorageBattery-Crash (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1649 of 1649 tests passed)

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


There is no advantage in changing these include_once into require_once , that is why we reverted the changes in the first commit


Name it `OnTimer` or more clearly


Freezing the string causes the call to `Gem::Version.new(MIN_VER)` in the verify_dependencies method defined at line 160 to throw an exception since it apparently tries to strip! the frozen string.

(Apologies if commenting here is inappropriate.  I had just pulled HEAD and built the gem since I saw there were changes that looked like they might fix my issue with {{vagrant_root}} not getting interpreted in config[:pre_create_command], which does seem to be the case, but I had to remove the .freeze from MIN_VER first.)


Installation section in https://wiki.jenkins-ci.org/display/JENKINS/Meet+Jenkins

> To run Jenkins, you need the Java Runtime Environment (JRE) 1.6

So all class files in plugin need to keep Java class file version 50.0.


I better rephrase it


These changes are required to work the view and download PDF in patient portal


You should probably use ctype_digit() instead of a regex.


mark it as `private static` and name it `IsSocketConnected`.
Also `System.Net.Sockets` move to usings.


just `return poll && available;`


fixes #3


Nuevo Pull request creado


deleted unnecessary headers


Business School Murska Sobota


почему тут через таймаут?


That's interesting and is good to know  for the future(I am guessing it's a variable scope issue).


артефакт. уже удалил


я думю если ты в сессию записваешь в SessionStorage то и очищать ее тоже надо там, тут только вызывть методы


не совсем понял


Your changes in the first commit were going the other direction(from require_once to include_once), which didn't make sense. Note that if the path is wrong(or file does not exist) for include_once, it just throws a error in the log but the script keeps going (thus if the path is wrong, could bypass authentication), whereas if the path is wrong for require_once, then the script dies. So, require_once is almost always best to use over include_once.


Interesting.

    curl: (7) Failed to connect to 23.235.44.133 <http://23.235.44.133>: Host
is down
That tells us that your computer wasn't able to connect to Github for whatever reason at that point. Odd.

    $ nvm install 0.10*
    *Version '0.10' not found - try `nvm ls-remote` to browse available versions.*
    $ nvm ls-remote
      N/A
That tells me that your NVM - for whatever reason - does not find any installable node versions online and, therefor, can't install anything.
[This StackOverflow thread](http://stackoverflow.com/questions/26476744/nvm-ls-remote-command-results-in-n-a) suggests its an SSL problem...

Try this:

```shell
nvm ls-remote
export NVM_NODEJS_ORG_MIRROR=http://nodejs.org/dist
export NVM_IOJS_ORG_MIRROR=http://iojs.org/dist
nvm ls-remote
```
and see if you then have versions available to install


@bors: retry


зачем в рутскоуп записывать стейт?


тоже артефакт от велосипеда остался


Like !


Awesome, can you please publish to bower :)


saw approval from jgraham
at https://github.com/Ms2ger/servo/commit/74306037019bb42eda541423535eb1b8b2c79f24


merging Ms2ger/servo/wpt_20150304 = 74306037 into auto


Ms2ger/servo/wpt_20150304 = 74306037 merged ok, testing candidate = 75b4b159


My First Repository on GitHub


Yes, please see explanation at https://github.com/aspnet/XRE/pull/1324


75127368_DCFSEC (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1644 of 1651 tests passed)

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


The assignment you do in the for loops of the crossover function is incorrect. Since they are both pointers, both the child and parent are pointing at the same thing. You can test this by changeing to code to:

                          offspring1TestCases[j] = parent1TestCases[j];
			  offspring2TestCases[j] = parent2TestCases[j];
			  cout << "\t Orig: " <<offspring1TestCases[j]->getEdgesCovered()[0] << "\n";
			  parent1TestCases[j]->getEdgesCovered()[0] = 1;
			  cout << "\t after: " <<offspring1TestCases[j]->getEdgesCovered()[0] << "\n";

Without looking over the code more, i think the easiest fix is to overide the assignment operator with our own version.


Can you provide an example of a Dn with naming properties where the naming properties are not required?  The for loop here is going through naming properties.  It makes no sense to me that some of those properties are optional.  By definition, naming properties are required to create the Dn.


Good job!  You make simple.


Commit messages for rendering HTML should include which platform the render was for.


I apologize for my tediousness. No need to escape «/» because it's not metacharacter. Escaping «/» required only if it used as regexp delimiter.


replace `1000` with constant


I don't know and I'm not going to spend hours on figuring out - if you think such change would be an improvement send a PR - and document it briefly.


Minor typo: beacuse -> because


@emmser: @moonso has written a package for dealing with ``.ped`` files that we should use to build pedigree files. We should take a look at this together at some point!


TeamCity Azmq :: Build (Windows) [Build 107](https://78.46.71.115/viewLog.html?buildId=439&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 65](https://78.46.71.115/viewLog.html?buildId=440&buildTypeId=Azmq_BuildOsx&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 81](https://78.46.71.115/viewLog.html?buildId=438&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 81](https://78.46.71.115/viewLog.html?buildId=438&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:09


I think this might have broken installation for non squirrel-aware apps.

We're currently seeing that shortcuts are being created pointing to the update.exe in SquirrelTemp rather than the one in <my app name>.   I'll back it out of our fork and see if things improve.


TeamCity Azmq :: Build (OSX) [Build 65](https://78.46.71.115/viewLog.html?buildId=440&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:15


This is due to : https://github.com/rails/rails/commit/1f31488499111fdfce79d8dc1cc8fb008f7cdb25

We should use the `#reflect_on_association` method which can accept symbols too, so it's compatible with both pre and post 4.2 versions of AR.

```ruby
model = parent_model.reflect_on_association(attribute_name).klass
```


TeamCity Azmq :: Build (Windows) [Build 107](https://78.46.71.115/viewLog.html?buildId=439&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:00:17


rgb to hsv de bir takım düzeltmeler.


Модификаторы --map, --info и остальные излишни, т.к. ссылки все одинаковые в меню. Модификаторы нудны там, где нужно что-то модифицировать.


Message box box wizard seems to be a config-specific feature. I am suggesting to implement scriptable wizards (also known as macros) for common config-pack-specific routines (such as setting up complex scripts). Such macroses can be sahred across forums (or in database) or included in config pack.


stripe больше похож на элемент reason, чем на самостоятельный блок


Ekstra test added


The library does not support Python Version 3.X


    foreach (var master in _masters.ToList())
    {
        if (SocketConnected(master.Value.TcpClient.Client) == false)
            master.Value.Dispose();
    }


Hvala za opomnik. Načeloma sem mislil tale vikend tole skončat.


I saw that the implementation of the methods of a `CygnusMemoryChannel` and a `CygnusFileChannel' are (basically, I include a comment about this below) the same. So, and just wondering, would it make any sense to make 'CygnusChannel' and abstract class to be able to provide implementations of these methods. Another option would be to include default implementations of these methods in the 'CygnusChannel' interface. I mention this a way to avoid duplicating code and providing slightly different implementations of these methods when they should be the same (please, see my next comment below).


This is a version number, not a date.
Version strings cannot be compared by Rime unless they are in the form of dot-separated integers.


please add spaces here


The implementation of the `take()` method is slightly different from the one in the FileCygnusChannel class. Is there any reason for this? This kind o situations could be avoided using abstract classes or default interface methods implementations.

Please, take my comments as made from someone who is refreshing Java at the moment :p


addLoadEvent();


It would be great to have some comments or info about these params ;) Thanks!


Javadoc comments highly appreciated ;) Thanks!


第一次提交 测试.doc


I tried, but didn't see anything about fonts.


I am sorry. I just saw that it extends Logger so this Javadoc could probably be ignored. An @Override annotation is probably the one appreciated ;)


@Override annotation appreciated ;)


it's basic. as said, uncommenting it worked for me, but the part that in in master and reads like
```php
/*
        if ( $this->Login != '' )
        {
            $options['login'] = $this->Login;
            $options['password'] = $this->Password;
        }
```


What is the difference with the test at line 78 ?


Javadoc highly appreciated ;) Thanks!


Weird, but ok. :) this seems nicer than the default, haha.


Should we use `System.out.println()` for logging? Just wondering :)


Verkar fortfarande vara den gamla versionen. Glömt att pusha?


Any reason to use this `5000` value? Shouldn't it be declared as a constant at the beginning of the class for easier tweaking ;)


Leftovers?


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


Comment test


Fixed in https://github.com/para-cms/para/commit/9ad6e59af1ead66b5bef37e4be46e033ba77c65c - I tested with LBAP a bit and this shouldn't cause any incompatibilities


Acked-by: Feng Yang


要学会使用style对重复的样式进行统一的定义


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2164 of 2182 tests passed)

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


Any reason for this change? Just out of my curiosity :)


I think there is already a tons of pull request, but the maintenainer seems not work on this anymore... Maybe somebody else should take care of the project. Anyway I will make a pull request soon.


Great. Glad it was helpful to mention it! :)

This package looks very interesting and I'll be sure to follow its progress (as well as the GopherJS issue you've opened).


For Pristine Skywise, I modeled it after Kami of the Painted Road. Everything is functional, but both cards incorrectly ask for the choice of color before the trigger is even shown on the stack. The choice of color should be done as the ability resolves though. I am not sure how to fix it. If someone else could help, I would appreciate it.


@kaspth this has been fixed in 4.2.1 (just tried rc3) and migrations worked.


you bet @samanpwbb. I have a jshint branch I've started too to try to catch more: https://github.com/mapbox/mapbox-studio/compare/jshint


I'll change the name of the test, but in this case, the provided settings (new settings) does not contains the Cobertura report path.


for new add-on extensions we went with kodi.foo so it would become kodi.addon.game
the metadata in the addons will be kodi.addon.metadata
In case you bump to alpha2


Should we be using `System.err.println()` for logging? There are other occurrences of this in this same class.


I'll just quote @Zidane here:

> [19:50:44] @Zidane:	Whoever reverts that commit will have hell to pay


Okay ill try your changes tomorrow. Ive been reading and trying to figure out the proper way of loading the stereo camera calibration file, and still a bit confused on how camera info manager deal with it. For some reason i am getting a parsing error, which i believe is what you said you need to manually parse the file. Going to look into this more first thing after my classes tomorrow. 2:11am on my side of the planet :)

Krystian Gebis
University of Illinois at Chicago

> On Mar 5, 2015, at 7:49 PM, gitunit <notifications@github.com> wrote:
>
> mate, read you tomorrow, its 2:50 a.m. on my side of the planet ^^
>
> —
> Reply to this email directly or view it on GitHub.
>


@l4u 3 hours ago, the link was inaccessible, so I thought the download link had changed. The link is back now .
Sorry for the false alarm.
Thanks


Actually I don't think this test is necessary. A not found menu entry would cause a 404 and `og_context_determine_context()` would never be called.


Instead of adding the choice to the ability, add it manually in the effect at the appropriate moment. Technically, I think we could remove every choice added to abilities as choices are meant to be taken once the ability resolves. (Other than modes and damage splitting).


Submitted #1 to fix most all build issues I had experienced.


UTF-8 is a character encoding. It's a method of converting from sequences of characters to sequences of bytes and vice-versa. It's the character encoding I'm expecting stuff in. There are others. Example:

In [3]: unicode('Поиск', 'utf-8')
Out[3]: u'\u041f\u043e\u0438\u0441\u043a'

In [4]: unicode('Поиск', 'utf-16')
Out[4]: u'\u9fd0\ubed0\ub8d0\u81d1\ubad0'


75127368_DCFSEC (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1645 of 1651 tests passed)

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


So i see the code was actually correct; this implementation is not the most straight forward. When i get a chance i will probally implement assignment operators for each class and rewrite some of this.


Silly zsh \$var expansion caused a wrong commit message. Correct is "*Now honors the $item paramter, hence $router_item renamed to $item.*"


75127368_DCFSEC (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1645 of 1651 tests passed)

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


Why would 8 bytes be easier than 4? They are unaligned nonetheless.


I added thread.sleep and re-execution in order to retry


There needs to be a `.data` for every field. The field itself always resolves to `True`.


container_name: id: message


not needed


s/will return/returns


This must only be in ER2, NOT in this repository!


no need to check for NULL.


if == err { err } else { info }


this toggles the setting on every submit...


That of course depends if the fields are not finalized yet (like >9 now)
isn't the functionality still needed if people can find targets that
could be proposed if the final field moves a bit in either direction.

Cheers,
Rasmus

On 05-03-2015 18:13, Tom Barclay wrote:
>
> Hi Rasmus, is there a need for this functionality?
> I removed it because we don't want the community to submit targets
> that are not on silicon.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mrtommyb/K2fov/commit/14e404e1e29c4d6ec078e6a476cc6edcdb9d4784#commitcomment-10055875>.
>


Det här börjar ju bli larvigt.


Nope.. this doesn't work. I'm not blaming you though, as this code has no coverage.
The type of dictionary will always become Dictionary<object, object> instead of Dictionary<TKey,TValue>.


In smart-components you may need to notify user that the example uses smart-button component.


it 'merges elements...


Shouldn't this compared against the $MaxAttachmentSize config instead of 5M?


please fix the indentation, and all other below.


indentation


same that above, it "bla bla" should be readable as an English sentence


@opdenkamp This should be a CPVRChannelPtr.


ok


ini seharusnya  gini <email>ferdinand.neman@gmail.com</email> ?


`this.classList ? this.contains(name)` should be `this.classList ? this.classList.contains(name)`


Hey, have you noticed the discussion here: https://github.com/felskrone/salt-eventsd/issues/32

Do you mind if i add this to the salt-eventsd repository?


Yea it's not the cleanest. Was just trying to get it working first.
On Mar 6, 2015 3:18 AM, "Akiira" <notifications@github.com> wrote:

> So i see the code was actually correct; this implementation is not the
> most straight forward. When i get a chance i will probally implement
> assignment operators for each class and rewrite some of this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Akiira/Comp-524-program/commit/b7186b61b0c5927ec6dcf7f71587f6d54a3b9adb#commitcomment-10066942>
> .
>


TeamCity cat-angular :: build [Build 47](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=16005&buildTypeId=CatAngular_Build) is now running


NP (nitpicking): Refactorize is Spanglish :-)  Refactor


I believe there's an error here, as this would evaluate to the equivalent of `if ("boolean" && score >= 0)`.

What you are trying to do should be written like `if (typeof score !== 'undefined' && score >= 0)` to accomplish a sane check whether the score is defined or not.


 <email><email>

apa g seharusnya
 "<email></email>"


Yes, it just tries to cleanup a unix domain socket entry in the filesystem if present. I'll take a look if it is required at all.


A bit better but still I would make it different:
```
- (void)signInUsingFacebookWithSuccess:(DVSVoidBlock)success failure:(DVSErrorBlock)failure {

   /*This also could be an object like:
     DVSAccountStore *store = [DVSAccountStore alloc] initWIthAppIDKey:self.httpClient.configuration.facebookAppID permissionKey:@[@"email"]];
    [store requestAccessWithCompletion:^(BOOL granted, ACAccount *account, NSError *error) {

            if (granted) {
                  [self makeRequestWithAccount:account success:success failure:failure];
        } else if (failure != NULL) {
            failure(error);
        }
    }
*/
    ACAccountStore *accountStore = [[ACAccountStore alloc] init];
    ACAccountType *facebookAccountType = [accountStore accountTypeWithAccountTypeIdentifier:ACAccountTypeIdentifierFacebook];
    NSDictionary *options = @{ACFacebookAppIdKey : self.httpClient.configuration.facebookAppID,
                              ACFacebookPermissionsKey : @[@"email"],
                              ACFacebookAudienceKey:ACFacebookAudienceOnlyMe};

    [accountStore requestAccessToAccountsWithType:facebookAccountType options:options completion:^(BOOL granted, NSError *error) {

        if (granted) {
            NSArray *accounts = [accountStore accountsWithAccountType:facebookAccountType];
            NSAssert([accounts count] > 0, NSLocalizedString(@"At least one Facebook account should exist!", nil));
            ACAccount *account = [accounts lastObject];
            [self makeRequestWithAccount:account success:success failure:failure];

        } else if (failure != NULL) {
            failure(error);
        }
    }];
}

//this method could be written similar and would return `parameters` in block
- (void)makeRequestWithAccount:(ACAccount *)account success:(DVSVoidBlock)success failure:(DVSErrorBlock)failure {
    SLRequest *request = [SLRequest requestForServiceType:SLServiceTypeFacebook requestMethod:SLRequestMethodGET URL:[NSURL URLWithString:@"https://graph.facebook.com/me"] parameters:nil];
    request.account = account;

    [request performRequestWithHandler:^(NSData *data, NSURLResponse *response, NSError *error) {
        if (!error && ((NSHTTPURLResponse *)response).statusCode == 200) {
            NSError *deserializationError;
            NSDictionary *userData = [NSJSONSerialization JSONObjectWithData:data options:0 error:&deserializationError];
            if (userData != nil && deserializationError == nil) {
                NSDictionary *parameters = [DVSOAuthJSONParameters dictionaryForParametersWithProvider:DVSOAuthProviderFacebook oAuthToken:account.credential.oauthToken userID:userData[@"id"] userEmail:userData[@"email"]];

                [self.httpClient signInUsingFacebookUser:self.user parameters:parameters success:success failure:failure];
            } else if (failure != NULL) {
               failure(deserializationError);
            }
        } else if (failure != NULL) {
            failure(error);
        };
    }];
}
```

Moreover please take a look on [this line](https://github.com/netguru/devise-ios/blob/master/Devise/DVSUserManager.m#L88) and tell me if `facebookAppID` should be taken from `[DVSConfiguration sharedConfiguration]` rather than `self.httpClient.configuration`? I think it should be the 2nd option because developer want to inject his own `configuration` via:
```
- (instancetype)initWithUser:(DVSUser *)user configuration:(DVSConfiguration *)configuration
```
and it would be different configuration than shared one. I thought...

Sorry for formatting in code above. Hard to indent code in github :smile:


@wilsonge @phproberto can we mark this deprecated for 3.5? I think we need to move this to 4.0 as we can break with 4.0 B/C?


`<= 7` for maximum code symmetry please :smile_cat:


None of these look like real instance variables. They should be constants in most cases. Having everything as an instance variable in inheritance from YCP Killer


I'll have a look at the dependencies. I also have a rough idea how we could address 2), I'll see if I can find the time.


ok, I'll file a follow up bug for this


SUSE coding style: use double quotes. https://github.com/SUSE/style-guides/blob/master/Ruby.md


TeamCity cat-angular :: build [Build 47](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=16005&buildTypeId=CatAngular_Build) outcome was **SUCCESS**
Summary: Tests passed: 29 Build time: 00:04:18


TeamCity cat-angular :: build [Build 49](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=16007&buildTypeId=CatAngular_Build) is now running


TeamCity cat-angular :: build [Build 48](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=16006&buildTypeId=CatAngular_Build) is now running


If you are not modifying ```configuration``` remove the ```deep_copy```. More YCP killer heritage.


You can modify QEMU with the [simple instruction trace patch](https://launchpadlibrarian.net/140372313/0001-Simple-instruction-trace.patch) applied as [ARM: invalid code execution after subs instruction](https://bugs.launchpad.net/qemu/+bug/1182344) mentioned.

Then, you can analyze the instruction trace.


totally wrong commit message... :sweat: Should be:

> **Click** is smart **enough**


This forces the filter to be true for all related package versions at the same time. We however want to the problem to be shown if at least one related package version in the problems fits the version filter.
For instance, if a problems has related versions `["3.10.0-1", 3.11.0-1"]` and the filter is `since_version=3.10.1` we want the problem to be shown, but it is now not.
Inspiration can be drawn from the `exclude_taintflag` filter.


@drewolson You will need this commit if you want to use forms for update too. I have updated the gist with Ecto.Changeset implementation:

https://gist.github.com/josevalim/c949a9db1339b3c7b48e


@goatslacker I didn't register it yet because bower complains about a missing `bower.json`.
```sh
bower eventemitter3#*     invalid-meta eventemitter3 is missing "main" entry in bower.json
bower eventemitter3#*     invalid-meta eventemitter3 is missing "ignore" entry in bower.json
```
I guess this is because the latest git tag did not include the `bower.json` manifest. I will register the next release.


TeamCity cat-angular :: build [Build 49](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=16007&buildTypeId=CatAngular_Build) outcome was **FAILURE**
Summary: Gradle exception (new); exit code 1 (new) Build time: 00:03:07


hmm interesting.. let me check


should go to section "users / helpers" (line 260).


Fixed


修正しました。


+1 for using something more descriptive than "Ogm...".

Apart from the name, a better comment on this class describing its purpose would be great (why we cannot use the existing `JtaTransaction` from ORM). Emmanuel once told me, but I forgot :)

I'm not sure what "JTA transactions exclusively from the TransactionManager" really means. "Exclusively" in contrast to what?


Hi @dstogov, does this means the previous condition is not your real intension but  a bug?


Just a heads up that dateutil doesn't seem to be available, at least on Openelec.    Can't get the script working since this update ;)


Thanks for looking into it. I have a few comments though:

1. 'http://jabber.org/protocol/pubsub#auto-create' is probably not advertised in Service Discovery.
2. I know OF-858 says "PEP", but actually I think normal PubSub should also support auto-create.


    scale: ['#ffffff', '#a1cd3f']

This resulted in erroneous display of data - a count of 1 was displayed in white and thus not distinguishable from a 0.


I agree. I'm not sure that I feel strongly enough to do anything right now though. My gut reaction is to tackle that when I'm near the code next.


This seems... inefficient.


To bad the examples and scenarios have been removed.
Even if you propose the configuration in the app, the examples might be useful to see what is possible in which combination. You could just remove the CLI commands.


please use literal syntax.


Yep, it is always better to leave it clear for Jackson which creator should it use. I wonder, if you don't specify it, is it nondeterministic which one will it use or will it have preferences over some?
The changes look good to me.


Perfect ;)


what if you have?


if the result is nan/inf then we should not warn.


to be honest I think you should make dictionary `parameters` here as immutable object.


I read it again. it should be.


No tests to verify that the call to xsltproc remains sane after the refactoring? In the long term, it's true it can look like a superfluous or even counterproductive test (testing the implementation not the result). But in order to verify the refactoring is safe I would appreciate to have it.


@bisAwesome
Yes, dateutil, trakt and arrow are new modules created by me.
I have already send them off for repo inclusion, but if you want to install them manual, go ahead:

https://github.com/Razzeee/script.module.trakt
https://github.com/Razzeee/script.module.dateutil
https://github.com/Razzeee/script.module.arrow


> I believe TimeHistogram is not a good name, firstly, because a binned spike train can also be clipped (i.e., only a 0/1). In that case it is not a histogram anymore.

I understand that you use different terminology in your lab, but for me a "binned spike train" is not a spike train, it is a histogram. A spike train to me is a list/array of event times, and nothing else.

I agree that a binary version of a spike time histogram is not really a histogram any more, but neither is it a spike train. Of the two, I think `TimeHistogram` is the least objectionable name.

> Second, it is not a histogram of time, but more importantly specifically of a spike train. Thus the name is not very telling about what the class does.

What is spike-train specific about it? I think there will be many places in Elephant where we may wish to use time histograms, so it makes sense to have a `TimeHistogram` class, and this generic class is entirely sufficient, in my opinion, to represent the result of binning a spike train.

> Third, TimeHistogram sounds more like an analysis, but in fact this class provides more of a data representation.

To me, a histogram is a data representation.


(Add a little more detail here, too?  It will be changed again during proper UI design, but still...)