@nurdymuny We're reusing alph_paginate partial across the application. This line causing issues because @employer_profile variable will not be present when you're accessing HBX portal....right now broker applications pagination broken under HBX Admin.


Changing "in" to "to" isn't right here. The way I understand it, you have to call `app.commandLine.appendSwitch` in your app's main script; no where else will work due to timing. The switches aren't appended to the main script; they're appended to an array of switches.


Half a week for this fix
:_(


Isn't this a breaking change not a minor update?


So what name do you suggest? "StorageInterface"?


I wasn't sure on this one but you are probably right.


LGTM


You can do this. Something went error without -y sometimes.

/Tomek
On Jul 23, 2015 5:49 PM, "Jens Janssen" <notifications@github.com> wrote:

> Can you revert a commit to development?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SiLab-Bonn/basil/commit/80b4c48eaf5594f985339538650129d64539a982#commitcomment-12323893>
> .
>


Will change it in another PR (where making EtcdHelper private).


Do we like this log message format for all "tracking" data dump?
i.e.
[TRACKER] action tenantId [$tenantId]: $body


How about use one option with Enum type?
It'll be more expressive and less error-prone
May it more expensive? As I understand -- no.

//cc @udalov


Yes, that's weird. I will look at core.


For example here.. https://github.com/minio/mc/blob/master/config.go#L206 we can add entries for "localhost:9000" and also "127.0.0.1:9000"  - of course it requires regeneration of your local configs.


Para entender que tipo de sangrias y cuales no implica el border-box, ver el dibujito de sangrias en las developer tools css.


Added to Player class.


A nitpick but we generally prefer an `===` comparison to `null` over using the `is_null` function.


:heart:


ok invert it


оно в облаке гита сохраняется
ссылку кидаешь где надо.
вот я и юзаю.

2015-07-23 19:34 GMT+03:00 Мицкевич Юрий <notifications@github.com>:

> картинку можно не отправлять
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Nikitostalkerby/FPS/commit/8735c0d9542bd43b3c9e9db8f07219864dbf82ab#commitcomment-12324824>
> .
>


Ok, what is the suggested fix ?


thanks so much. I am going with the php lib.


-22877332.00 == tan(90)


Very confusing, picture please?


you should do this at open() and cache it inside sol_spi, using it for further transfers.


LGTM


hi


Actually, after we nail down some of the specifications of where and what we want, I will probably create a common static Tracker class that will contain and instantiate the TenantTracker, with a common method to obtain the TenantTrackerMbean and allow multiple services to use it (ingest, rollup, query).


> There's a trade-off here as well. There is better locality when the state is in the cursor, and it makes the job of the optimizer much easier to not have to worry about indirections. When benchmarking the comprehensions test, I found the result was very sensitive to the indirections and ended up moving state into the cursor for better perf.

I'm guessing you didn't benchmark `view::repeat(std::vector<double>(73948795)` ;) I have an action item filed in the depths of my brain to some day develop a reasonable heuristic - maybe `is_trivially_copyable && is_roughly_cache_line_sized` - to determine when the state should go in the cursors.

> Also, if mutable state is stored in the range instead of the cursor, then the range cannot have const begin/end members.

`mutable` is a reasonable solution for a single-pass range. The iterator semantics make it impossible for other threads to observe the state while some thread is mutating it, so data races aren't possible in a program without undefined behavior.


Does this refer to CPOL and CPHA combinations? If so, maybe we could use a better name. Its not clear if the fist bit is CPOL or CPHA if you just use numbers.


@nataliejeremy Not sure we should link to http://php.net/manual/en/migration56.incompatible.php because it is for migrating to PHP 5.6, which isn't an option on Pantheon currently.


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

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


Aaaahh!!! No pensé que ibas a actualizar el código para que funcione con una nueva versión de Django!

Voy a subir mañana lo que estuve haciendo yo al repo, así optimizamos esfuerzos ;)


It's in line 612 - line 617, the string "new_node" is appended to each pair, so that its length become 3.
In this version, pair with newly created node has three item including two nodes and string "new_node", pair without new node has two. I think it maybe better to just keep the string in pair to help know if there is a new node.


Shouldn't the speed be in Hz here? Also, shouldn't you be able to use other values as well?


Added it to .gitignore


Okay, logged a v0.99-400 bug with a pointer to this commit for possible backporting.


this does break backwards compatibility with anyone who sets HUDSON_HOME and uses hpi:run - but well whatever Hudson is long gone,


Hab so ein Gefühl, dass ich's auch weiß :smirk:


Think there's a missing `{` here.


Why did you remove const here?


spaces between `) {`.


Thx. I've also changed validations in other models 926fba552ba7373f79482bd091f30033ced02c18


What's going on here exactly?


HotFix-ReadVars (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1762 of 1763 tests passed)

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


You can pass link (i.e. employers_employer_profile_path(@employer_profile, status: status)) as a variable while rending the partial.


ok.


What's the purpose of the `i +` in the conditionals?


What do you mean by more expensive? These options have no representation in the binary output. I think bools are OK here, although the name `ignore` gives a bunch of false impressions. How about renaming it to something related to comparison of protos?


This stops retrying when the request fails. Instead of removing the handler, just stop it if a request is in progress like here: https://github.com/voat/voat/blob/02c84e762ccd5a74a9d984130050fa148e0749ac/Voat/Voat.UI/Scripts/voat.js#L487-496


Yeah, this issue is annoying. It might be worth removing the submodule and doing a symlink or something. There is a command do it automatically, and you could add it to .git/hooks. I think it is:
```
git submodule foreach git pull
```


You can have just one `console.log`, how can you do that?

You can also get this down to 2 conditionals, an if and an else if


можно просто загрузить, и не отправлять


I'm reluctant to use css-text.  I used it in 1.15 for self-contained, and got bitten by a nasty bug.
https://github.com/yesodweb/css-text/issues/6
Looking into the code further, I found lots of problems which still haven't been resolved.  I don't really trust the library in its current state. Do we need it for this simple purpose?


Extend this documentation saying that new transfers will fail if there was an existing one that did not finish yet (IOW: we do not queue transfers). Mention that user should flag the transfer is pending on his own and clear the flag when the `@a transfer_cb` is called back.


This should be a log not a comment.


This should only be added if needed. But as we use gulp-eslint, we won't need it right?


You might run into double execution of your regular buttons' clicks. I fixed it this way:
https://github.com/TeamBliss-LP/android_frameworks_base/commit/5d418444f7951bada2b5f5532c367f8671b542f3


Example:

<% if full_index_url.present? %>
<li class="<%= 'active' if params[:page].nil? %>"><%= link_to ' ALL '.html_safe, full_index_url, remote: true %>

<% end %>

pass full_index_url variable to the "render" partial...


Can we add `fp._` to `Predef`? For that matter, can we somehow add `scalaz._` and `Scalaz._`?


Looks like it works - SL warned me after I changed download dir to %windows%\system32


I did a while back, but it's not clear to me that current status since
quincyatt was closing apps randomly I thought?

On Thu, Jul 23, 2015 at 1:50 AM, gimmeitorilltell <notifications@github.com>
wrote:

> Dan I thought you tested this, sorry.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CandyDevices/device_samsung_msm8660-common/commit/c2d6dc1f64f83e913720d06c025e2bc91a9ae3d1#commitcomment-12313931>
> .
>


Sorry about that! :innocent:


Note, the django server doesnt call any of these yet, but they are working and I tested them manually. Next step is to integrate the calls into the django views.


Typo?


97902934_ZoneMixing (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed

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


I'd say `ignore_in_comparison` or `skip_in_comparison`


%s and use `sol_util_strerrora()` to make it clear the reason why that failed.


Probably not. Originally I didn't want to reimplement a function that already existed. But it's simple enough to copy. If you want I can redo the PR without css-text.


Hey owersun, did you implement your FBdev output device, too? I have compiled your sources and get a:

kodi.bin: undefined symbol: eglGetDisplay

error on the fbdev Mali drivers. Kodi 14 from your sources ist working well.


Какой в этом смысл? Если исходить из стандарта C++11, то nullptr является
рекомендуемым для инициализации указателя...

2015-07-23 20:35 GMT+03:00 Dmitry Mamontov <notifications@github.com>:

> Все nullptr заменить на NULL.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RandyyDSH/qChess/commit/7cf2536629514d4b51e6d27dd6cb9ac03c961fb2#commitcomment-12326164>
> .
>


HotFix-ReadVars (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1762 of 1763 tests passed)

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


Это касательно NULL.

23 июля 2015 г., 21:02 пользователь Данил Щербаков <randyydsh@gmail.com>
написал:

> Какой в этом смысл? Если исходить из стандарта C++11, то nullptr является
> рекомендуемым для инициализации указателя...
>
> 2015-07-23 20:35 GMT+03:00 Dmitry Mamontov <notifications@github.com>:
>
>> Все nullptr заменить на NULL.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/RandyyDSH/qChess/commit/7cf2536629514d4b51e6d27dd6cb9ac03c961fb2#commitcomment-12326164>
>> .
>>
>
>


same as above, `sol_util_strerrora()` to show why it failed. You can also put this whole core logic in a single function and call it from both thread and idler, as it should always be the same.


Ahahaha – yes. It’s a race condition in Emacs. I can start typing a match pattern before the grep buffer is open, and then the characters end up in the code buffer I was in. Slightly annoying.


97902934_ZoneMixing (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Oh yeah. That should be enough. I am removing this. Thanks :-)


`skip_in_comparison` sounds good.


HotFix-ReadVars (Myoldmopar) - 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)


Hoàn thành part1.


Hey just a general note for both java and python:

anytime you do something like:
        counter = counter + 1;

instead do:
        counter += 1;

It's the more idiomatic and succinct way of doing it. I'm haven't gotten a chance to take a super close look at both the assignments you posted yet, but I will soon. Nice work posting them up here. We also need to change your .gitignore file so your machine doesn't automatically commit unnecessary files like the .ctxt and .class files.


I'm not sure it needs to be more obvious than getting a TypeError when you forget to pass filters.


FYI, @jdegoes was not in favor of overloading these methods, which is why we had `beAnyLeftDisj`. But that's the pattern in specs2, so I'd be OK with it.


Can this method be implemented without the use of a `var`?


What is this power?


Thanks Martin for your fast response, Appreciated ;) .


There are no dumb questions. Just me assuming something is in the vernacular.

They're like [characterization tests](https://en.wikipedia.org/wiki/Characterization_test), only speckier!

> In computer programming, a characterization test is a means to describe (characterize) the actual behavior of an existing piece of software, and therefore protect existing behavior of legacy code against unintended changes via automated testing. This term was coined by Michael Feathers.


shouldn't need to cb = NULL here, you also do not do it for idler.


I'd prefer we don't rely on reference semantics here. Please add ticket.SetOrder back, even if the way it is currently implemented means we're reassigning the same reference.  Removing it obfuscates the intent of the method and the way data flows to the ticket.


Adding `Modified by Yash Khandelwal, GSoC'15` to all the files I've edited will be accurate, right?


Переведите на C++03, не у всех C++11 может быть под рукой.


`Array`s are mutable, so you should be able to make these two `val`s.


Yeah … I think we can eventually get rid of our versions, but that will require a couple PRs against scalaz-specs2 and/or some changes to our code. I tried, but didn’t want to dig into it too deep among these other changes.


Şahane!!


I like this for the way it cleans up our imports a bit, and also for distancing us a bit from the large Java and Scala "predef".


test


I saw that (I actually picked from that commit, =p), I was testing out the difference between them, both seems okay, no double execution yet for me. I'll test some more, I'll probably end up using your way anyway. Thanks for the heads up.


Thanks for cleaning these up. Should the GetOrderById method use the GetOrderByIdInternal implementation as well?


FacilityTabGridView (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Argh....


Slightly more functional:

`(0 to 8) foreach { x => ...`


Ah, gotcha. It was slightly confusing at first to read the explanation of setting `flex-shrink`, and then seeing the shorthand `flex`, and it got me wondering if specifying `flex: none` was better than simply `flex-shrink: none`.

Also, browsers automatically letting content shrink may not be specific to just Chrome, Opera, and Safari, as the spec gives [the initial value as `flex-shrink: 1`](https://developer.mozilla.org/en-US/docs/Web/CSS/flex#Summary).

This was a small thing. Thank you for replying.


I'm not sure but this may cause "emacs --daemon" to crash on startup.  I upgraded projectile this morning and starting up emacs as daemon started to crash.

This change seems to be the only noticeable change from my previous version from MELPA.


`hasMessage`?


Parts of this commit were introduced into the repo already on June 16 and credited correctly to Clark Scheff for his code to get theme resources:
https://github.com/TeamBliss-LP/android_frameworks_base/commit/17f0e7b9c1c9751cc8a4ba67bfaf757cea835622


If you return true here the timeout is renewed, but it may expire before sol_spi_transfer() is finished, then the second transfer is lost (since we do not queue).

In this case we need to return 'false' and set the mdata->timer = NULL, creating a new timer from spi_transfer_cb.


Yes I did. I didn't test it though, since I don't have the device at my hand right now.
But it should be fine.
You error looks like something going from libEGL.so
Can you check first that your kodi.bin is linked to correct libEGL.so by ldd and second in the log of kodi start does it really uses FBDev EGL implementation.


If I can be of help, let me know. :)


@dmfenton definitely agree we need to work on errors, that's been bothering me a lot as I've been reading through code. The `err` parameter should be an actual `Error` object as in `callback(new Error('something bad happened'))`. Very often we're sending back a plain string instead of an error object, and often enough that string is dated and not very descriptive of the problem. I think we need a future refactor to focus on standardizing and improving error handling and reporting across the project.


No, seems coordinate order was right, it has something to do with rotation.


then `{{#if hasMessage}}`


Any error messages/stacktrace?


Hum, idlers are low in priority and may not even run in the case of a heavy loaded (fd/timer) systems. Change this to a 0 timeout timer.

Sorry about that, I know I said to use the idler before :-(


switch


 I write like
Stephen King

I Write Like. Analyze your writing!


FacilityTabGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Recommend using `rank` (1 - 8) and `file` (A - H) for the rows and columns. See http://gerenciaycambio.blogspot.com/2011/09/el-ajedrez-estrategico-de-la-rse-i-el.html. You can use `Char`s for the file and use integer arithmetic on them.


Very confused about the intent of these two ternary expressions??


Uhhhmm... Actually this commit has nothing to do with 51f6465 :sweat_smile:


Please remove this variable, then call `removeCompletion` before the `if` statement below.


this should probably clear reverts as well.


Then we can just keep the `return false` here.


And we can just keep the `return true` here.


I don't think this helped - the Thames Wharf machine now flies upwards.
This is the code that works for me:
```C++
ent->m_transform.getOrigin() += ent->m_transform.getBasis() * btVector3(dx, dy, dz)
```
I tried it simply because that's how it was computed before the C++ merge.


> Also, browsers automatically letting content shrink may not be specific to just Chrome, Opera, and Safari, as the spec gives the initial value as flex-shrink: 1.

Those are the browsers currently affected by Flexbug 1, that's what this is for.


zzz


Small issue. The spacing of this line looks a little weird to me.


since you removed the -x check from inside execute_hook you are not ignoring non executable files anymore.  Imagine that you maintain your hooks in a repository in which you have non-executable files like README.md, .gitignore, etc.... You would then trying to execute then if you pass the directory.


Confused about the style here - on one line you're relying on the implicit `this` for `getName()` and on the next line you're explicitly using `this.getName()` ... why?


s/Every/All/


as said above, I think the check should continue to be inside execute_hook


there are way too many layers to dig through until one finds out what votes.0 and votes.1 are.


por mi perfecto


Ah, I see, thanks! I should have known that, I've read his (excellent) book :stuck_out_tongue_closed_eyes:


Daylighting_Speed (DeadParrot) - x86_64-MacOS-10.9-clang: Tests Failed

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


This is strange, because it was exactly the previous fix, but you just swapped the arguments and now it works! What kind of magic is that? :sob:


I'd recommend a different class name for Action, since it is so general of a name. If it were namespaced as THREE.Animation.Action I think that'd be fine or you could call it AnimationAction.


:+1: Aha - that explains it!


added


You wouldn't need to rid yourself of the numbers to the right of the decimal. Instead, you would just set a variable that allows you to save the numbers to the right of the decimal point, and then use them later in the result.

Here's Dan's solution:
var remainder = n % d;
var evenlyDivisible = n - remainder;
var wholes = evenlyDivisible / d;       //always an integer!
var result = wholes + ' ' + remainder + '/' + d;

Let me know if this doesn't make sense, or if you have any further questions. Overall, though, really nice work on all of the above hw problems!!!!


The ternary is not irrelevant here, it avoids calling `getClass()` on a `null` instance.


You already have reference to CacheStatus object from status map under key "ALL_CACHES", there's no need to perform put operation again.


The same as above.


:+1: I am a bad speler.  :smiley_cat:


This one *is* irrelevant today.  I'm guessing at some time in the past there was a `getClass()` call here too.


RatingResult already was a tuple of six values, and now one of them is a tuple itself...? please no :) as the count is already part of the RatingResult, you could consider only storing the absolute number of votes in the distribution and do the percentage calculation in the template. or do something else, but i don't think this additional complexity is necessary.


And who said that every operation should be commutative?


This is mostly because CommonMark requires spaces for indented code blocks.


Странно - я буфер до 64к увеличивал.


Exactly, the previous one works with the columns of the matrix, this one with rows.


This commit breaks date formatting such that every week the relative time format resets. Formats that should be `1 week ago` switches to just `5 hours ago` for example. What was it attempting to fix?


this should be: "SCRIPT_DIR=$(dirname $(realpath ${BASH_SOURCE[0]}))" as in other scripts otherwise it won't work in this scenario:

```
cd ~/p
git clone ...toolbox.git
cd ~/bin/
ln -s ~/p/toolbox/arch-installer.sh
```

(~/bin/ is very common to be left in the PATH. I used something like that in the past)


Works for me, did you update packages?


Can we close it manually, or at least keep it open while mouse is hovered over?


Other than formatting issues, it looks good


Molta bene!


HotFix-ReadVars (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1763 of 1763 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)


This comment seems inaccurate.
It actually reads zero partitions, not the first partition, right?


Minor: the description is confusing. I wonder if isn't best to separate the command behavior in a flag for itself: -c "command" -x "hook.sh" -x "hookdir".


@johnousterhout I copied this from RamCloud.cc::~RamCloud(). From my understanding this reset notifies the other end points about the end of sessions. Do you agree to move the objectFinder.reset() to ~Context()?
Also, I couldn't find such reset in MasterService's code. Should it be called only on client? If you agree that this should be called both in client and master, I will just remove the TODO.


@gic4107, I think so.


@maartenbusstra To clarify, are you no longer getting DOM Exception 8?


The camera follow lines will only run for the first hero spawned, but if that works we can write those lines at another location


That's correct - as many parts of the code depend on System calls, this makes mocking easier.


Nope. Those find counts exist only for caches imported from geocaching.com, but for other sources they are zero. Therefore it is more a performance optimization to use the numbers at all.


Yeah so this apparently happened twice. It's fine, two commits are better than one.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.808.7987b10dd85fce10d8e57dcb3e7ccf56302d3797](http://build.palaso.org/viewLog.html?buildId=59976&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:03:01


saw approval from mrallen1
at https://github.com/basho/riak_core/commit/b23de82aa508b4b34e9d61fdcdeea383044a2db9


merging basho/riak_core/nem-clique-0.3.2-version-bump = b23de82a into borshop-integration-754-nem-clique-0.3.2-version-bump


basho/riak_core/nem-clique-0.3.2-version-bump = b23de82a merged ok, testing candidate = 0c89aee8


Actually it looks like just the wrong conditional was flipped...


I have to go back to school! :laughing:


It was a slip of mine... linear algebra was one of my best courses, so I guess my brain refused to think about that tiny multiplication...


Please avoid such newline modification.


Thanks for making Font Awesome. @ChangeTip, give $10 to @davegandy.


We can. Though we'll need to spend a bit more time on this, e.g. this flash message will stay on the screen no matter what user does, even if she opens next page or another user profile or single post page -> it's really confusing. Having auto-hide makes UX cleaner.


why calling bash like this?  If you do so you can't pass stdout redirections anymore.


Why we need three scripts instead of one (overlay only) or two (overlay-append, overlay-replace -- given that they don't seem to share much code)?


This only runs webpack -w and not starting the server, but this kind of configuration may be possible using nodemon.
Good place to start: https://www.npmjs.com/package/nodemon


the errors in koop pain me, wish I had known better...


The test case is trivial, and don't put that within this commit.


Hi @davegandy, @BladeBronson sent you a Bitcoin tip worth 35,336 bits ($10.00), and I'm here to deliver it ➔ **[collect your tip](https://www.changetip.com/collect/874572)**.

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


Why not to put "version" inside mkDerivation?


Either in usage or as a comment in the script, add a short sentence explaining what it does.


ack


self.templates was undefined until I fixed it


Why not to put "version" inside mkDerivation, as before?


Oh that one is actually wrong, I played around with it, but the Property mechanism should work just fine.


why `open`?


I think it doesn't work on multiple threads.


I was copying from the CoreInstaller.


The comment is incorrect.


Any Difference?


ldd /usr/lib/kodi/kodi.bin |grep EGL
        libEGL.so.1 => /usr/lib/opengl/mali/lib/libEGL.so.1 (0xb6a05000)

The path is ok (it is on Gentoo)

19:52:30 T:3037663232  NOTICE: CEGLNativeTypeFbdev::CheckCompatibility FBDev device: 21, info.xres 1920 info.yres 1080 info.upper_margin 0 info.lower_margin 0 info.pixclock 0
19:52:30 T:3037663232  NOTICE: InitWindowSystem: Using EGL Implementation: FBDev

So it does it, as far as I understand.

I had installed libsdl2/sdl2_gfx to get the jostick workiing. Reverted this and compiling again right now. I will report.


`@ThreadSafe`


We called this version "jit0" to represent its beginning as a simplified JIT compiler rather than being functioned.


I still think this pattern is kinda clunky. Just to keep in mind, these triple-layer quotes are a bear to read. Next time, maybe breaking up the final shell command into smaller variable-assigned chunks or something. It'd result in a shorter, interpreted command string at the end.


This is a markdown file


:+1: , although this branch is going to be nuked since the approach doesn't actually work like I'd originally intended and Alex is working on a better version.


Yup - so one of the problems with not updating the transform before the animation block is if the user rotates the phone while in the tab tray, the transform is no longer valid so before we animate, we update the transform to be valid for the current orientation. I'm going to break this out into smaller methods that explain it a bit more so it's clear.


Minor: "copying current system pacman ..." instead of "my".


This doesn't work as expected with the Closure Compiler, since `'__html'` will not get renamed, but access via the dot property (and setting it via an unquoted object literal) will.

`keyOf({__html: null})` in `props.dangerouslySetInnerHTML` seems more appropriate. I can send a PR if you agree.


Bei mir sieht der Tip genauso aus, wie aufm PC. Was ist bei dir kaputt?


`ProtoCompareUtil`?


Not sure I understand the problem, but maybe it's better to use tar -xf $tarball -C $ROOTFS


In my opinion, optimally the C++ side should deal with all memory allocation and de-allocation, as it is the longer "living" part. (i.e. a script always starts after Envision starts and ends before Envision ends)


What if you set `max` to the value of the first item in `arr`? That way you could skip the `max.nil?` condition when setting the new maximum in your loop below, since it could never be `nil`.

In that case, you'll have to do a different check for when `arr` is empty. A clean way to do something like that is using an early return. At the top of your method, you could:

`return nil if arr.empty?`, for example. That way, if `arr` is empty it immediately returns nil and stop executing the rest of the method.


`isEquals` or just `equals`?


oops, this is append, not replace. ok.


IncrementalCacheImpl.isClassOpenPartNotChanged --- some anonymous subclass is created.


`this.` not needed


@frankrui in general, add as few object to  `$scope` as possible. Since this is only used inside the `popModal` function, I moved it there


`brew install git` will fix that for you homeboi


`print(square.draw())` seems awkward. I'd either do

`print(square.toString)` if `Square` implements `toString`, which can be just `print(square)`

or

`square.draw()`

which can be written very concisely as

`row foreach draw`!


Good work, this code is clean and easy to follow! Good indentation and good descriptive variable names.

Also, I left a small comment up above for another option you could consider for solving this problem.


That seems reasonable, though the original intention of this property was that it gets sent down from the server in which case you would want it to be __html exactly and not get transformed, more like closure externs.

If this still seems best to you, extract it to a constant at the top of the file?


If I understood the talk correctly, no. As only Property uses this class in shared_ptr we never have to clone this. Or did I miss something?


oh right. copy & paste stupidity...


Prefer meaningful names to variables. Don't worry about long names, the JavaScript interpreter doesn't care :)


I'll change name to `equals`.


`[].slice.call(elements)` => `Array.from(elements)` [mdn](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/from) :relaxed:


Yep, I also prefer 2 separate scripts... If they share code, put that as a separate scripts that you source on both of them with ". common_bla.sh"


This is a personal preference of mine - I prefer to iterate over arrays with `.forEach`, I find it more semantic


Ok, will do for now.


Thank you.


So with `toString` you could also do `row foreach print()`


Prefer leaving the names of fields consistent - there's no reason to call this field `metricName` if it's already called `label` somewhere else


The title of this should have been:

:arrow_up: Fix Xcode 7b4 compiling

This was really changing because of an implicit dependency change. Right? Still trying to apply the emoji's listed in [CONTRIBUTING](https://github.com/TabletopAssistant/DiceKit/blob/master/CONTRIBUTING.md#git-commit-messages).


fixed comments in both test cases


If a value is only going to be used once or assigned to an array immediately after being created, there's no reason to create a variable for it :)


all tests pass:
success: http://buildbot.bos1/builders/test-riak_core/builds/1647


Looks like this commit broke build.
Please see http://ci.cgeo.org/job/c-geo/3924/console to remedy the problem.


No, the first loop updates the statusMap entry to set whether a rehash is happening on each individual cache or any cache. This loop does the same for topology changes.


@danielballan this is not a csxism. This is how it is done at CHX and XPD too.


I was not talking about the ternary operator being irrelevant, I was talking about the getClass itself. ```getValueTransformation()``` yields a ```String``` and ```getZoneReplicationFactor()``` yields a ```HashMap<Integer, Integer>```. It is irrelevant to compare classes in these cases as they will always be the same.


Maybe will be better just use `==` for primitive types and enums?
Is it harder to implement?


As mentioned on the bug it might be better to accumulate the mouse events rather than discarding - see [this comment](https://bugreports.qt.io/browse/QTBUG-22269?focusedCommentId=188825&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-188825) on an old Qt bug for example.

> We ran into the same issue. We worked around the problem by overriding QWebView::wheelEvent, and doing the following:
> When a wheelEvent comes in, we start a 25 ms single-shot timer and process the wheelEvent. For any future wheelEvent's that come in while the timer is active, we just accumulate the event->delta( )'s (and pos & globalPos values, too). When the timer finally fires, the accumulated deltas are packaged into a QWheelEvent and delivered to QWebView::wheelEvent. (One further refinement is that we only do this for wheelEvents that have NoButton and NoModifier.)
> Ultimately the problem is that the wheelEvents come in too fast for QWebView to keep up. By accumulating the wheelEvents and delivering them in one packet every 25 ms, QWebView has enough time in between to handle them.


Yes, role is missing in S110: [s110](http://infocenter.nordicsemi.com/index.jsp?topic=%2Fcom.nordic.infocenter.s110.api.v8.0.0%2Fstructble__gap__evt__connected__t.html&cp=2_7_0_0_0_2_1_3_14), [S130](http://infocenter.nordicsemi.com/topic/com.nordic.infocenter.s130.api.v1.0.0/structble__gap__evt__connected__t.html?cp=2_7_2_1_0_2_1_3_14)

I did a diff between the S110 (v8.0) and S130 headers, and this seemed to be the only significant difference.
This patch attempts to homogenize the structures handled by the upper API, and the other one allows to only provide one set of headers for both S110 and S130.


`@VisibleForTesting`


`CONFIG+=man.extra` is gone since v2.0.2 or u8sand/Baka-MPlayer@0f8717c.


> `mutable` is a reasonable solution for a single-pass range.

I understand that this respects the letter of the standard in the case of single-pass ranges, but it's still a lie. If the range object is mutated during iteration, then it shouldn't have const `begin`/`end` members. It's the principle of least surprise.


i approve


HotFix-ReadVars (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (729 of 729 tests passed)

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


These are compile-time constant integers, there is nothing to export.


I guess I should put ```this.``` in the if as well. I copied the conditions from the ```equals()``` method and was lazy xD ...


This would bypass
```
 [self ork_setAnswer:[text length] ? text : ORKNullAnswerValue()];
     [super inputValueDidChange];
```
at the end.


Why the file is called `application.yml.example`? Everyone has to rename it on their machine, before they can start the project


@adrielhampton can you also add the `people#add` endpoint (recently added) here for endpoints that can be used to create folks in the nation? It behaves the same as `people#push`, but won't overwrite existing values in the nation.


@mdarefin Hm, this change doesn't look like a rename. If you use SourceTree, you can easily pick the lines to commit. You can also do it in Eclipse but I don't find it so straight forward.


sounds good


I have found the code I used to solve the above bug in the past:

    class QWebViewScrollFix(QWebView):

        def __init__(self, **kwargs):
            super(QWebViewScrollFix, self).__init__(**kwargs)

            self.wheelBugTimer = QTimer()
            self.wheelBugTimer.start(25)
            self.wheelBugTimer.timeout.connect(self.wheelTrigger);

            self.wheelBugDirAccumulator = dict()
            self.wheelBugDirAccumulator[ Qt.Orientation.Horizontal ] = 0
            self.wheelBugDirAccumulator[ Qt.Orientation.Vertical ] = 0

            self.wheelBugLatest = dict()

        def wheelEvent(self, e):

            self.wheelBugDirAccumulator[ e.orientation() ] += e.delta()
            self.wheelBugLatest[ e.orientation() ] = {
                    'pos': e.pos(),
                    'buttons': e.buttons(),
                    'modifiers': e.modifiers(),
                }

            if e.buttons() or e.modifiers():
                self.wheelTrigger()

            if abs( self.wheelBugDirAccumulator[ e.orientation() ] ) > 1000:
                self.wheelTrigger()

            e.setAccepted(True)
            return

        def wheelTrigger(self):

            for o, e in self.wheelBugLatest.items():
                event = QWheelEvent( e['pos'], self.wheelBugDirAccumulator[ o ] , e['buttons'], e['modifiers'], o )
                QWebView.wheelEvent(self, event)
                self.wheelBugDirAccumulator[ o ] = 0

It's not a straight solution as this addresses wheelEvent rather than move.


Shouldn't this change have two files?


N.B. The comment it wrong s/b

Bug Fix: No red flash on FMU reboot during programming


Available*


Rails has URL-helpers. They are for building URLs.


Note it doesn't mean that `checkEquals` is not good :)


FWIW, these 7 lines appear to me to be exact duplicates of 7 newly added lines in `LedgerConsensusImp.cpp` (https://github.com/ximinez/rippled/commit/366a88c88ee673cb7c3e26331e386958cdfbaec3#diff-c71c719ae07d12ad17a81571fcea7f26R1159).  Sometimes copying code is appropriate.  Is this one of those cases?  Your call.


Don't leave the dead code.


What happens if this constraint is violated?


Strawberries and syrup should never mix! Please revert this change :astonished:


Very confusing indeed.

This is copied from the ```equals()``` logic, which is confusing in more ways than one.

I could have fixed ```equals()```, but I was wary of breaking something that might have inadvertently relied on the broken/weird behavior.

In any case, I think ```equals()```, ```hashcode()``` and ```diff()``` should behave a in a completely consistent manner, even if that means consistently weird.

-F


You used `factory_girl` with `faker`. Do you find it justifiable to add 2 gems to a project just to generate a sample URL? How about this code:
```
Link.create(given_url: 'http://example.com/foobar')
```
One line, no additional gems required


Instead, please use `#if defined(__x86_64__)` and `#elif defined(__arm__)` to specify the appropriate inclusion.


Look who is doing React now =D


Don't touch x86-64 implementation.


Be sure to change WORDLIST_FILENAME and file path in getStoryString()!


Hi,
By now I was looking for still some bugs . However , I found some and repaired by sending final ( hopefully ) corrected version .
LINK - http://pastebin.com/BXhku5TM


not given https://github.com/Pomax/react-onclickoutside/pull/21/files


No localization files?`%lang(zh_CN) %{_datadir}/%{name}/translations/baka-mplayer_zh_CN.qm` et cetera after build with `CONFIG+=install_translations`.

Pardon my ignorance if this is handled automatically by rpm.


I'll try to implement it.


Typo on line 90 (not your doing).  "modificaiton" -> "modification".


Maybe we don't need these?


No magic :P boost/python.hpp includes Python.h which includes a file which defines a struct called slots, thus we have a conflict with the Qt slots definition. (See also: http://stackoverflow.com/questions/27045323/embedding-python-3-4-into-c-qt-application/28916330#28916330)


Quick and dirty solution. Array better for including other types (if necessary?). But, if ('string' in [array]) prevented var t from ever being assigned.


my tendancy would be to put them somewhere in the blueflood-http submodule, since that is where they will be executing.


Don't leave the dead code.


We can safely replace all `callp` with `bl <addr>`.


What does this test case check? If the same line of code gives the same result?


@jmwohl
This is probably good enough for our purposes.


Instead of using inheritance (which gets harder to reason about because this class is meant to be thread-safe), I'd just have a PartitionedOutputBufferManager that takes a partitioning function in its constructor and pass it in directly when it's instantiated in SqlStageExecution. Besides the partitioning function, there are not real differences between HashOutputBufferManager and RoundRobinOutputBufferManager, so having extra classes for them is a bit weird and slightly overkill.


Não esqueça de fechar a tag </a>


It seems that you didn't do `git rebase` well.


Lets figure out where in the blueflood-http module to put them, and I think the answer will fall out from that decision.


@jmwohl
Is this a holdover from old code? It returned surveys instead of submissions.


Don't do that. We should always use the consistent CFLAGS.


Seems fine.  We want to log the exact request as well.


uniqueId has a very strange type. A non-const reference to a shared_ptr means: "I'm possibly going to change what this pointer points at, and I might take ownership of a reference". It doesn't seem to be the case. Passing a `UniqueId const &` seems more appropriate.


@rodfersou `self.portal.getPhysicalPath()` has always the same value


@rodfersou please documetn the constructor


Why do you need a DatabaseCleaner?


Wat r u doing?!


Hmm


Does this do anything?


Seems a bit cleaner to put it in the logger, but I don't feel strongly about it.


Why doesn't this method notify `outputBufferTarget`? More generally, what's the protocol for when outputBufferTarget is called?


+1 @ari-gold thanks!
@nataliejeremy, my vote is:
>For custom code, see the  **Backward Incompatible Changes** documentation for migrating from one PHP Version to another in the [PHP Manual](http://php.net/manual/en/appendices.php).


Are you sure it doesn't work? https://github.com/pihvi/assert-diff/commit/7adbbe489d7f34e0967d2c1e5b3f735137d6d77c


a quick fix is including this right after `<base href>`
`<script>baseElement = document.querySelector('base');baseElement.attr = baseElement.getAttribute;</script>`


choo choo


# Simple using

 <link rel="stylesheet" href="jquery.modal.css" type="text/css" media="screen" />

 <script src="jquery.modal.min.js" type="text/javascript" charset="utf-8"></script>


@indutny Hi, I am not sure why is this here https://github.com/nodejs/io.js/blob/master/lib/_debug_agent.js#L29 ?


vshpwh_strat_integration_bug (nmerket) - x86_64-MacOS-10.9-clang: OK (1756 of 1757 tests passed)

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


No stock buttons anymore?


I think the API would be nicer, if you allowed multiple tenants and stored them in a set, (and allowed turning each one off individually.)  What do you think?


first, second, third = numbers_array.slice(index, 3)


80 char limitation?


Do you think this is an issue with the hackrf recipe, though? All recipes can override the template settings, after all.


Why reassign it? Why not just name it correctly in the param above?


Maybe just include this as an `&&` with the statement above?


Ditto here. Nested ifs like this can just turn into `&&`


these `if`s (last 2 or all 3) maybe merged


Yeah, I think this is okay. The `OpenLedger` code sits in parallel to the existing code for now, so we can verify that it is producing the correct results. When we throw out the old code and make `OpenLedger` the "real" open ledger, these code blocks will diverge.


@rachelwhitton Funny--that is what I was asking Ari right now. I'll update it :)


Thanks a lot @MarcelGerber LGTM ;)


Please document the new setting in the asciidoc. There is an overview over all the properties that can be set in the settings.xml.


80 chars


This bug is related to the encoding of the strings.
When a string is not found in the translations file, it is returned with a wrong encoding, and can cause problems.

By the way, thank you for the 'new' bug. It is fixed now.


looks like, even though I don't select not to push other files, eclipse IDE push all of them to eclipse


@rodfersou you forgot to remove the registration of the `resolve_uid.ResolveUID` utility


@chandralaxmi : please make the code object oriented. for eg: create a cell class, board, player etc.


cc @veged @tadatuta This is much more optimal way of doing stuff like https://github.com/bem/bem-core/blob/4086fdfc4ff9e260ca0f2abdeaa495c9ab4edf3f/common.blocks/page/page.bemhtml#L3 . What do you think?


I think we should upload the current used fontawesome files here.
I think they are more complete for flyspray purposes.

The 'fat ugly button' css has not much to do with the font icons.
I really want this self hosted and not dependend of a cdn hosting.


this is ok.


you could match and string format

fields.length match {
  case headers.length => headers.zip(fields).toMap
  case _ => throw new RuntimeException(s"TSV parsing error in line ${tsvLine.toString}: wrong number of fields")
}


They have been deprecated by GTK+ devs in July 2013. See [this](https://mail.gnome.org/archives/gtk-devel-list/2013-July/msg00000.html) mail for more info.


Please change this comment to:
`// We don't want autocompletion when deleting characters, but we do still want the search panel to update. Currently we do neither; see bug 1166781.`


@rodfersou isn't just easier to iterate over the previous pipline section? why do you need to inherit from the constructor?


This doesn't look right to me. The test "child 2 and "grandchild 2" did not run, isn't that a bug?


ah, nice - I was thinking that the `<img` tags might cause reflows when they loaded, but you've totally got that covered.


I don't think so...
https://github.com/gnuradio/recipes/blob/master/hackrf.lwr#L29

If I commented out this line from the template, my extra cmake flags were picked up correctly. With this line present in the template it wiped out my cmake flags.


This is very similar to what ive been working on. I had the player give a
clue about reservables only if it had a clue match greater than one, if
thats true I then have it find the highest amount of matches possible
between different reservables and their colors and numbers. it bumped down
our average to 19.06. I havent tried changing its place in the take_ turn,
and I also wanna try and filter it to only give those clues when the
reservable cards age is high.

I can work on this a bit more today, and show you what ive come up with
later if you wanna take a look tonight.

On Thu, Jul 23, 2015 at 1:30 PM, Mark Spurlock <notifications@github.com>
wrote:

> I added (privately, this isn't pushed yet) an inferred_reserved thing that
> was basically analogous to inferred_playables, but it wasn't working out.
> Too easy to either (a) fail to infer the intention of the clue, or (b)
> misconstrue it as implying the card is playable.
>
> So I'm thinking of scrapping that (for now) and just relying on the
> "players try not to discard stuff they have info about" behavior. So at the
> point where we decide to give a "reserve this card" clue, all we have to do
> is make sure the clue can't be misconstrued as "play this card". So the
> clue would have to meet one of two conditions: (a) the clue is about more
> than one card, meaning the receiving player won't try to infer anything
> about it, or (b) if it is just one card, the receiving player will try to
> infer that it's playable but there won't be any playable cards that match
> it. In that case, he won't infer anything about it, but he'll be less
> likely to discard it since he has information about it.
>
> The other part of this is knowing how much to prioritize "reserve card"
> clues. I think it's generally better to tell a player "play this" as
> opposed to "reserve that", but we may need to experiment to find the sweet
> spot. And in general, the decision making process that goes on during
> take_turn() probably needs to get smarter.
>
> I was planning on doing this work tonight, but it sounds like you might be
> toying with some similar ideas, so if you'd like to take a crack at it
> instead just say the word.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Spurlock/Hanabi/commit/b1b24eea3bddb244e16e8a821e23d803ec1a400d#commitcomment-12326037>
> .
>


I'm not seeing `assert` used in this file.  Do we need this include?


@raphaelpor :+1:


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

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

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

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

```


Great work. I will pull this for now because it seems working. Would be nice if a follow-up PR can fix the comments I made. This is a big change, so obviously there are a few glitches. I will open an Issue for those.

Thanks!


Everybody is reacting :dancer:


There was a requirement that code should only include human-friendly characters - characters `1`(one), `l`(lowecase L), `I`(capital i), `0`(zero), `O`(capital o) are not allowed. I can't find where they are excluded


missed spaces around `-`


Hm. Yeah, use SourceTree.


Nit: remove extra newline here.


I just noticed that the contents of `test/test/unfinished.tap` don't event remotely match the output described in https://github.com/isaacs/node-tap/issues/129#issuecomment-102215471

@isaacs I think this commit accidentally added a regression test to ensure that this bug never gets fixed! @sam-github has reported as #152


Tests look good.


``` scala
val headers = tsvIt.hasNext match {
  true => tsvIt.next().trim.split('\t')
  false => throw new RuntimeException("TSV has no header")
}
```


Oh okay... you meant options... maybe make a markdown sub-list instead? What do you think @wdkaye


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/3925/console
Build was first broken at http://ci.cgeo.org/job/c-geo/3924/


there may be a better way to do match on just true or false in scala, and in general with my comments here on scala idiomaticy, grain of salt it


@rauljmz This should fix the install issues you were having yesterday. Will add to [pre-release feed](https://www.myget.org/F/sitecore-glimpse-prerelease/) asap.

Then use:

    install-package Sitecore.Glimpse.Mvc5 -source https://www.myget.org/F/sitecore-glimpse-prerelease/


This will be fixed in rc4


Just curious @gdi2290 ... Wasn't the Alpha 32 fix supposed just ensure <base> tag was present (https://github.com/angular/angular/issues/3096)?.... curious why this hack is needed (I do see the error that is caused without it), but I am confused as to what Alpha 32 fixed since a <base> tag is present...


@lcampos71 looks good!


This will be fixed in rc4 :-)


What about name collisions from tracks being combined across multiple actions?


Please change this to:
`// shouldChangeCharactersInRange is called before the text changes, and SELtextDidChange is called after. Since the text has changed, remove the completion here, and SELtextDidChange will fire the callback to get the new autocompletion.`


is there another case other than this?

if not, i think you can do:

``` scala
val tsvData = tsvIt.zipWithIndex.map( (line, idx) => parseLine(line, idx, headers) )
```


What happens if 2 users simultaneously send a request?


has this removed the members one?


Oh, that's cute.


Woohoo!


that probably has a warning for not being exhaustive or if not is just not necessary to have a case


I want to be the last author?


Can you check the alignment requirement for x86? We might not have to reserve 3008 bytes.


LGTM (kinda)


I'm going through this in serial fashion, so I haven't cross-checked this by how it is used. Is this cumulative time or delta time? If delta, then I'd recommend `delta` or `deltaTime`.


hey @geoffjentry do you have an opinion


No. I've changed the way its handled due to it being more complex than the others (like the standard horse)


Ok, did a new pull request. I wouldn't call it elegant but I got it to work and with only one console log. :O


:+1: :bang: :boom: :shipit:


gic4107's [jit-construct](https://github.com/gic4107/jit-construct) works as expected. You can check his implementation.


Hoping goes all internalonly


Thanks for the fix! Took some time for us to catch this one. Just upgraded to latest version of the plugin, hopefully this fixes our app crashes.


Nit: Don't change this line


or @mcovarr I GUESS


@nfeske: please add to staging


Hm, this fix is a bit weird as you said :) I'll try to figure out if there's a better way to handle this...


ok


For consistency - perhaps this should go in a View menu.


duplicates L157-L166


Nice change.  Personally, I find this new formatting easier to read.


Bit of redundant whitespace here.


and here


i guess this is with one decimal? do we really need that? if yes, please align the filter's name and the formatting syntax with percentage_value.


Undefined behaviour. Some backends might work just fine, some might return errors on every call, since the framework wasn't initialized in the new thread.


The profile changes here are not working as you would expect due to how 'activeByDefault' works. See https://issues.jboss.org/browse/HWKMETRICS-184


Sounds good!

I think the score decrease might be caused by giving "reserve" clues when they aren't necessary yet. So you might try only considering giving "reserve" clues when all these conditions are met:

1) Player has no known playable cards
2) Player has no known useless cards
3) The oldest card in his hand (hand_age[0]) is reserved.

This way we're pretty sure we don't give "reserve" clues unless they're definitely needed right then.


"captureFrame" probably.


From my knowlegde the `:disabled` selector matches, but the `disabled` property should remain.  Which means, if you want to know wether an input is true-ly enabled/disabled you should check the selector matching and not the ``disabled`` property.


I can one-line it but IntelliJ gives me a whole lotta red if I try to drop the case. `tsvIt.zipWithIndex.map { case (line, idx) => parseLine( line, idx, headers ) }` seems to be the best I can do


... i think you need to rename count to total_count now :(


@aks- there is a TODO to figure it out :)


@paf31 Is this OK?


that was due to the use of `.attr` which is jQuery rather than `getAttribute`
https://github.com/angular/angular/commit/8296dcec09d9b8be0d4cfcc492000d5cc401cfa0#diff-b6e7e7a163b4d38f3ed67950d8d93826R282


This algorithm skips some values - after *9999* goes *aaaab*, it skips the *aaaaa*


There seem to be problems compiling it. Have you encountered them? http://hydra.nixos.org/build/23951123


@hvelarde because I want to run the code into __iter__ from the constructor


Recommend using:
var keys = data.hierarchy[ h ].keys;
to avoid having to copy / paste


xD


Awesome, thanks for the commit. Adding now!


please change this to something like "ensure ordering of answers"


jajajaj excelente. fijate que acá hice andar bastante, por ahí te sirve :)


The name collisions are allow, it means that multiple tracks are driving the same property.  The result should be based on a weighted blend between the multiple tracks according to the weights of their respective Actions.


:tophat:


please figure out whether the None is used anywhere and if not, remove the total_count parameter


I can't find how you verify this controller works? A controller spec or an integration spec?


duplicates body of `Descriptors.Descriptor.typeName`


Currently it is absolute time.  It is just easier this way.  I'll let someone else track what time it is and how to increment it.  This allows for complete random access to an animation.


Okay.


I think You can use `Descriptors.GenericDescriptor` instead of `Descriptors.Descriptor` and drop second fun


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

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

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

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

```


Closes https://github.com/TrinityCore/TrinityCore/issues/15148


//new2
has to be the first line in scripts.txt to switch the battleye analysis of script usage on


Why did you change this line?


ist zu weit rechts und wird deshalb abgeschnitten


x^2 is the same as X*X


@nfeske: please add to staging or remove the original commit - as wanted ;-)


nitpick, but I find middlewares to be easier to understand when this parameter is called `handler` instead of just `f`


Daylighting_Speed (DeadParrot) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Awesome, to match the pattern of AddSecurity() can you please make SecurityType the first parameter, then an overload could accept symbol-only which checks if the symbol is in Portfolio, use the security type of that asset, if not in portfolio, default to equity.


Heh. I looked at it too and was wondering -- is it to hold the event loop open one more tick?


What about testing css files that are not in a module? sibling to the sass directory?


Added become to sudo this step. Thanks for the catch @sergii-tkachenko


Change it to https:// to prevent being blocked as active mixed-content and breaking the layout. Consider also changing the protocol to https:// in the images URLs too.


`Descriptors.GenericDescriptor` is private in `Descriptors`.


Remove `out port` and return the port


Remove it


Hi,
By now I was looking for still some bugs . However , I found some and repaired by sending final ( hopefully ) corrected version .
LINK- http://pastebin.com/Xvzi6DPw


# Options

### Body

```javascript
MustangModal.popup({ body: "Hello world." }).open();
```

### Title

```javascript
MustangModal.popup({ body: "Hello world.", title: "This is a title" }).open();
```

### Width & Height

```javascript
  MustangModal.popup({

                body: "Hello world.",
                title: "This is a title",
                width: 250,
                height: 100

            }).open();
```

### Buttons

```javascript
MustangModal.prop({

                body: "Hello world.",
                title: "This is a title",
                buttons: [
                {
                    id: "btnClose",  				//Optional field. If id not set, will be added random button id as automatically.
                    text: "Close",				    //Optional field. Default name is "Button Name".
                    type: "primary",				//Optional field. This field uses bootstrapt button types. Default type is "default".
                    callback: function() {

                        alert("clicked the close button");
                        MustangModal.close();
                    }
                }]

            }).open();
```


Port = GetPortFromHostName(insecureHostName, 80)


Port = GetPortFromHostName(secureHostName, 443)


Added some options of mustang-modal


I think it looks good, yep!


Let me also confuse you with another proposal: maybe `areEqual`? :)


Thanks for the tests!


You will get a better error message "use of deleted operator=" if you make these public. Now you will get "use of private operator=".


looks good to me.


Dude, who made lucky an executive, i never said that xD
I have no problem if he''s an executive, i trust him but don't promote people without me knowing it.


That's why I think the `onPersistSuccess` should be called first. When the implementer decides to put some logic in this hook and maybe change the state, we cannot be sure that the actor is in a consistent state when an exception is thrown. That's why I think that the handler from persist should not be called in this case.

IMHO the read model should not directly  be updated via the events that are emitted in this way. There is no guarantee that you will receive all events. My idea is to have a PersistentView listening and issuing Update commands to self to update from the journal. This will give far better guarantees.


I verified the behavior in IE11 before adding the note. The field looks disabled and you cannot keyboard focus it. you can focus it via the mouse and script, at which point you can edit the contents. While you *can* focus the element, the focus event is not dispatched to the element, but `document.body` instead (with `event.target === document.body === document.activeElement`). I *assume* `:focus` won't apply either.

The way I'd patch this, if I had to, would be to add `input[type="text"][disabled], fieldset[disabled] input[type="text"] { pointer-events: none; }` (not using `:disabled` just to make sure…).

Do you think there's further action required?

I <3 this kind of feedback, very observant, thank you!


OK, thanks -- that might still be a bug in PyBOMBS rather than the recipe and/or template. This is a bad place to discuss this, though -- can you post an issue somewhere? Thanks!


@chrisbrightwell When you pull this (if you do), make sure to do the following:

Hopefully you have NodeJS installed (check with `node --version`). If you don't have it, grab the latest from their website.

After making sure you have NodeJS, run `npm install -g mup`.

When you start up Meteor, if you want to use Mailchimp, make sure you run `meteor --settings settings.json` instead of just `meteor`.


@aredridel no, it kicks off the initialization, which doesn't happen without it for some reason.


@dpgeorge , @ampli : FYI, bogus failures for thompson bothered me for long time.


fast-forwarding 2.1 to borshop-integration-754-nem-clique-0.3.2-version-bump = 0c89aee8


inb4 I can't into English


good idea, let me tweak it to allow multiple tenants.


lol what the hell


Архив был лишний


Isn't `commits` a map from repo ID to `[Commit]`? We should probably keep it the same type.


Ah, thank you.


écart ±1 filtre, probablement la façon de calcul a une erreur


@aggelosQQ Alright! :P


Great find. Thanks for this fixed


> I understand that this respects the letter of the standard in the case of single-pass ranges, but it's still a lie. If the range object is mutated during iteration, then it shouldn't have const begin/end members.

It's only a lie if the mutation is a *modification*, i.e., visible to required equality-preserving expressions. I'm not completely clear on how the properties of single-pass iterators apply to single-pass ranges. E.g., after incrementing an input iterator all of its copies have invalid values - can I call begin on multiple copies of the same single-pass range? I would think so. Can I call begin on a single-pass range after having called begin on a copy and incrementing the resulting iterator? I would think not.


True. I'm thinking of dropping the distance_from_origin function and this paragraph anyway. It might work better in the traits chapter.


You need to change the default in config/client.py it's not the same.


Will this break all of the Kafka scripts that use zookeeper for admin related commands? ex: kafka-topics.sh


`static NSString *const ...`


Please move this into your global git ignore and not here. See https://help.github.com/articles/ignoring-files/#create-a-global-gitignore


Why are we adding this entry?


reppo


Thanks, @OlsonDev. You're right. I'll correct that.


in this commit i have 2 different programs from http://rosalind.info/…
…problems/

1) http://rosalind.info/problems/ini/   about counting nucleotides in sequence

2) http://rosalind.info/problems/dbpr/  reading proteins processes from http://www.uniprot.org/uniprot/.txt
(MUST TO FIX IT) searching processes in file with " P:" string --- it's bad i think


we can probably remove the `$results` var and just run the query w/o that line.


Daylighting_Speed (DeadParrot) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


I am happy with any of these: `checkEquals`, `equals`, `areEquals`, and I'll change the name of generated method if you think it will be more meaningful.


It will be fixed.


:sunglasses:


If you're removing this, remove the command line flag, too, and the MigrateKeys function, and I'll find the issue that that fixes. Or you can defer this to later (but then don't remove this part).


How can I make this dialog appear so I can check the link?


@aslakhellesoy  The important thing is that after the "prepare for the next development cycle", all examples that refer to snapshot versions in there pom, has been updated to the new snapshot version of cucumber-jvm. If this is done by the maven release plugin or manually during/after the release process is not important.


`the Lock`


Alex, feel free to make that change - I'm about to leave for a trip and haven't brushed up my github skills since making these doc edits last year.


@nfeske: please add to staging


`when` and `was displayed.`


Good catch, 1b59ba0f622bc7fc151509cd9cef8f76d28d8f83 repairs this..


:+1: thanks


> so the [`:disabled`](https://developer.mozilla.org/en-US/docs/Web/CSS/%3Adisabled) state does not change.
Oh, I thought this is a general warning not a warning for IE11 only.

About your other question. There is no good way to do this.

I wrote a polyfill some years ago. Here is an example: http://jsfiddle.net/trixta/K8EPm/

The key idea with this polyfill, is that getters and setters of native DOM elements aren't polyfilled, but the jQuery interfaces (i.e.: $.attr, $.prop) are. This way this thing also worked in IE6.

About your ``{ pointer-events: none; }`` approach, I can only say, very neat trick.


97903100_SolarCollectors (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)


If you change it to just plain "Storage", what does that conflict with? That'd be my first choice of name.

My second choice would involve the package name changing to storage so that this type would be "storage.Interface", but that's more work.

I guess "StorageInterface" is a compromise that I can live with :)


y


Note, I am moving this code back to its original position since I changed the #if from !PLATFORM_UNIX to FEATURE_WIN32_REGISTRY


Hi,
For the cluster region I think it would better to allow the user to enter the full endpoint of the ECS from the config, as this would make it somewhat more future proof.
thanks
Ali


I missed it


@rpg777 I'm getting this build error:

1>  RCC: Error in 'E:/openstudio3/openstudiocore/src/radiance/radiance.qrc': Cannot find file 'resources/blinds.xml'

Do you have this file?  Might need to add to git?


Add a docstring here please.


+Nodes are free implement different routing algorithms.

Does that not mean that nodes could then not conform with the mesh topology of routing and start centralized routing?

Sent from my iPod

> On Jul 23, 2015, at 1:45 AM, Felix Ableitner <notifications@github.com> wrote:
>
> What do you mean exactly?
>
> —
> Reply to this email directly or view it on GitHub.
>


You're right. How nice of you to see, and tell :smile:


HotFix-ReadVars (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1246 of 1247 tests passed)

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


Do you need to add the filter again?


In a nutshell:
- Ensure we're not interrupted whilst switching ring
- Push SS for ring3
- Push ESP for ring3 (Hardcoded for now: Badness)
- No longer stuffing the EIP address in a register before pushing.

That sound about right, @philipstears ?


For data science course in coursera


встановленняи - зайва літера 'и'


回滚


Anyway we can extract common logic


This fixes issue #32


We never add an extra comma in that case.

The point is that since `parent_xblock` has a default value, we shouldn't need to know of it's existence. So, we shouldn't need to imply that an additional parameter exists.


Looks pretty good. Next commit, make some tests!


I came across this IE problem on [caniuse disabled](http://caniuse.com/#search=disabled). My manual test confirmed that `ally/is/disabled` properly returned true for `<fieldset disabled><input type="text">`, so I guess we don't need to do anything about this right now - at least not in the context of "is the element focusable" :)


Колега, сори, май ти прецаках промените. Трябва пак да си избереш задачи от списъка. Пушнали сме по едно и също време почти.


I'll do it.


Hi sweet.. kaya yan..


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

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


Fixes #93.


not sure how it's here


This is great but I don't know how to accept the changes...


Sorry, yes.  I have some traces that I took of particular button presses to
try and determine which commands were what.  I'll try to allocate some time
to find that stuff and get it published.

On Wed, Jul 22, 2015 at 3:26 AM, prebbz <notifications@github.com> wrote:

> Hi Steve!
> Did you have any answer to my last question?
> Many thanks,
> -Marcus
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/srwalter/cbus-emulator/commit/cc46c5dac43213ac6b56ee9a8943f4d759519623#commitcomment-12292189>
> .
>

--
-Steven Walter <stevenrwalter@gmail.com>


@jdi-li bittesehr


Yup. Still need to or IF in the pushed eflags as well.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.809.2c71813ec9fe1ec7bd513c646d861931dbd069cb](http://build.palaso.org/viewLog.html?buildId=59979&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:03:18


@simonmichal
Can you please add the following two entries for the XrdFileCache Package:
* deadlock on immediate file close (e.g. xrdcp to non-writable output)
* fixed errors on some posix operations using virtual mount

The change does not have to be necessary tagged if it is included in  XRootD ReleaseNote  web page.


This fixes issue #26


This is a good change whoever suggested this must really have a good command of English and its idioms I applaud them.


@route yeah, I checked. Looks like I should change my tests now. Thanks.


Il manque l'entete xml


В контексті маєтся на увазі "Заголовок" - назва сторінки може бути відмінною від заголовку


I did address this in the new version, after making some research. It's much nicer now.


You can't do this - the CombinedOutput() makes this wai tuntil the executed command is done - which causes 'lxc start t1' to hang until t1 is shutdown.


Мабуть краще буде "Відновити налаштування"


ibuohho


I think this way sounds better:
Bash scripts are commonly used in Mac/iOS development to make repetitive operations hands-free. However sometimes things go wrong, and in a perfect world I’d like to be properly informed about the errors.


Merci @LeMagnesium ! On te vois plus sur IRC, que ce passe-t-il ? :)


здесь можно оставлять какие то комменты


@ldoktor I'm positive it'll be merged in the correct order, don't worry.


No useful messages/stacktrace.

Loading 00debian-vars...
Loading 00debian-vars...done
Loading /etc/emacs/site-start.d/50auto-complete-el.el (source)...
Loading /etc/emacs/site-start.d/50auto-complete-el.el (source)...done
Loading /etc/emacs/site-start.d/50autoconf.el (source)...
Loading /etc/emacs/site-start.d/50autoconf.el (source)...done
Loading /etc/emacs/site-start.d/50dictionaries-common.el (source)...
Loading /var/cache/dictionaries-common/emacsen-ispell-dicts.el (source)...
Error while loading 50dictionaries-common: Symbol's value as variable is void: debian-aspell-only-dictionary-alist
Loading /etc/emacs/site-start.d/50doxymacs.el (source)...
Loading /etc/emacs/site-start.d/50doxymacs.el (source)...done
Loading /etc/emacs/site-start.d/50magit.el (source)...
Loading /usr/share/emacs/site-lisp/magit/magit-install.el (source)...
Loading /usr/share/emacs/site-lisp/magit/magit-install.el (source)...done
Loading /etc/emacs/site-start.d/50magit.el (source)...done
Loading /etc/emacs/site-start.d/50pymacs.el (source)...
Loading /etc/emacs/site-start.d/50pymacs.el (source)...done

Backtrace:
emacs[0x4fe09b]
emacs[0x4e429e]
emacs[0x4fcd8e]
emacs[0x4fcf93]
emacs[0x4fd00f]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x10340)[0x7ffff1f88340]
emacs[0x447ede]
emacs[0x5260e0]
emacs[0x559618]
emacs[0x58ee05]
emacs[0x5590df]
emacs[0x559443]
emacs[0x58ee05]
emacs[0x559443]
emacs[0x58ee05]
emacs[0x559443]
emacs[0x58ea78]
emacs[0x55825e]
emacs[0x58ee50]
emacs[0x559443]
emacs[0x58ee05]
emacs[0x559443]
emacs[0x58ee05]
emacs[0x559443]
emacs[0x58ee05]
emacs[0x559443]
emacs[0x558d16]
emacs[0x55bfa1]
emacs[0x559618]
emacs[0x58ee05]
emacs[0x5590df]
emacs[0x559443]
emacs[0x58ee05]
emacs[0x5590df]
emacs[0x559443]
emacs[0x58ee05]
emacs[0x558af0]
emacs[0x57d0b4]
emacs[0x57d628]
emacs[0x55ade0]
emacs[0x558844]
...
Error: server did not start correctly


I assume this was trying to connect to the database.
It is not defined elsewhere in the file.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.810.5cfadd7a3ee1b06766f2b2345c062b6296ff630a](http://build.palaso.org/viewLog.html?buildId=59980&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:02:24


yup it was renamed, actually. will fix...


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 199](https://teamcity.labkey.org/viewLog.html?buildId=262308&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 20 Build time: 00:37:37
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBinnedPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySavedGroupPlot: <no details avaliable>

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

```


versão 0.0


These member functions should probably be const.


warnings at compile time for both Objective-C/Swift, and even while typing.

The subsequent phase sounds kind of repetitive, I think its better to say something like: Let me show how to use the `Report Navigator`.


No, you're right :-)

I created https://github.com/youtube/vitess/pull/910 to remove it.

DONE


Lets remove the `load` argument. `delegate` is called once (and passes the default value), so this is future coding.

Also we shouldn't trample over Active Support's `delegate` method. Rename to `relation_delegate` :smile:


:scissors: this line and indent under private.


I would have done
```python
assert expression.attrib['form'] in ['arrhenius', 'Arrhenius']
```
or
```python
assert expression.attrib['form'].lower() == 'arrhenius'
```
to make the code a bit easier to read and maintain, though it might turn out that yours is faster (which would only make it better if this were in a speed-critical loop)


Can you add a comment saying why Popen is important?


AddCalcCoilSurfTempSource (kbenne) - i386-Windows-7-VisualStudio-12: OK (1762 of 1763 tests passed)

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


感谢您的支持！


I would have done
```python
        if kin.attrib.get("primeID") == 'rk00000000':
            return
```
To make it clearer what is happening and why. It's best if the code follows your logic.
eg. It's not that we want to save primeIDs of non-0 entries (we want to do a whole ton of stuff for non-0 entries!) but that we want to abort immediately for 0 entries.

My version is also 2 lines shorter.


it seems some newer emoji code points are missing. I'm cross-referencing with this (up to v8.0): http://unicode.org/emoji/charts/full-emoji-list.html and this http://apps.timwhitlock.info/emoji/tables/unicode. Is there a reason to support only 6.3?


Actually, this would be better, as it would avoid doing the `kin.attrib.get("primeID")` lookup twice:

```python
        rkPrimeID = kin.attrib.get("primeID")
        if rkPrimeID == 'rk00000000':
            return
````


Unncessary


What about seeding it with more values?


说都GV


SettingActivity를 부르는 로직을 함수루틴을 활용하여 부르도록 수정함.


@sjohannes / @mathbr do you know anything about this compilation stuff?


Thanks for the quick reaction.

Do you think the issue at https://github.com/aspnet/Hosting/issues/307 might somehow be related to your recent refactoring?


no comments.... :P


@alankessler I saw your comment on one of the issues and thought it was a good idea. Thanks for all your support!


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1246 of 1247 tests passed)

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


Sorry osmcache.CheckFeature() is renamed to osmcache.initFeature() in develop.


Hellow


Why do this?


Yeah that's definitely a possibility. It's quite difficult to test, actually... I guess you could try running it a few thousand times and counting the number of duplicates, making some changes and then running the test again.

I remember I struggled with finding good sources of decent data which gave reasonable names, though I guess it's pretty clear that even what I've got now doesn't necessarily give great results either.


You could possibly seed it with census data: http://www2.census.gov/topics/genealogy/1990surnames/dist.all.last


There is a chance that Vanilla Morrowind does not use actor height.


@halfluck This commit regresses behavior of your controller or you're just reporting a bug?


git reset --hard


Add `browser.HandleDestroyed -= BrowserHandleDestroyed;`?


Modified for use in the V3.02, which uses a Atmega88:
https://www.dropbox.com/s/88zve7kq2mc0h38/ATmega88.hex?dl=0


@cgutman Ah sorry wrong section, I was just trying to understand what was the issue by reading through the code, scratching my head for ages when two identical controllers have different behaviour.


tried to add color to textbox


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


updated


@pedrohills ao invés de [Pedro Henrique](https://www.youtube.com/user/DjPedroHills1/about) coloca seu usuário no Twitter por favor. :+1:


Nesting, using blaze. Supporting bokeh/bokeh#2442


r+


What's this?


I think we should probably upgrade this to a fatal error: not having an `ApCorrMap` when you're supposed to be applying it seems like a serious problem in configuration, not something the pipeline should recover from.  And if (e.g.) one CCD's fluxes aren't aperture-corrected out of a visit while the others are, because something went wrong on that CCD, I don't think we can use that CCD for most subsequent processing.


Yes, that could help, though you might end up with names that sound the names of people, I guess combined with the other data we already have it could work.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.800.4ff52fbbf33fbefe68810170efb82bdc7a484bf8](http://build.palaso.org/viewLog.html?buildId=59939&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 390, ignored: 3 Build time: 00:05:37


@halfluck All the special case for ASUS Gamepad is: Back -> Start and Mode (Power button) -> Select.

Without the special case, Back should work as Select and Mode should bring up the Steam overlay (if you're streaming Steam). There won't be a start button mapped though. Open a bug and I can add support for this.


well you also undid refactoring the size out.


So many kinds of Awesome, thank you very much!


Worst homepage design ever.

LGTM :)


should change ceasercipher to take in a filename as input to encrypt as opposed to using a static string to encrypt


Are you using this anywhere? It seems like you should be using it instead of `test.csv` below.


You could test whether your errors message is in `upload_csv.data`.


Vg


Shouldn't `jass` be `jaas`?


In general, I think we should start to move away from having the question bank tag content be a single line of text, just to make it more readable. To this end, I was thinking it might make sense to make each dataset an inner tag inside the <Map> tag. Admittedly, I started off the convention of making the tag just one line, so this comment is a bit hypocritical :) I'm not sure the best format, but maybe something like this would be nice?

<Map>
  <Projection prop1="value1" prop2="value2" />
  <Dataset name="Countries" colors=[...] otherProp="value" />
  <Dataset name="Lakes" colors=[...] otherProp="value" />
  <Label name="Dublin" coordinates=[x, y] />
</Map>

Let me know what you think


what if they enter a negative number?


Looks like we need to create `org.zanata.seam.security.RunAsOperation` (minus the `run` method), and switch to it.


Err... this doesn't look right. It *should* be 16, not 1.


Is it possible that the name should have been bcpc-vm-bootstrap


A field that is a pointer to a parent model will have `field.rel.parent_link = True`. You can check to see if a field is a parent link using `field.rel is not None and field.rel.parent_link`. The Django docs have more information on [customising the parent link field](https://docs.djangoproject.com/en/1.8/topics/db/models/#specifying-the-parent-link-field)


Ah the same problem exists with this example. I realise this is my fault, but can you make this more of a proper mock that doesn't rely on the implementation?


Not much; I know it detects when there are too many (artist, album)
pairs in one directory (the assumption is that it's a directory where
the user dumps random music files), but I don't know what Exaile does
with that information.


I don't understand :/


Why do this?


I think it would be cleaner to put methods like these into a new superclass, eg `org.zanata.seam.security.Identity`.


Then I think we should call it `org.zanata.seam.security.jaas.SeamLoginModule`.


Can't any of these classes inject the identity object?


Could you please make pull request for the develop branch?


setellite freature

please install a spell checker


Use this type of syntax:

    ("$!xwiki.get...." == "1")

The `$!` means that if that expression fails to be processed, or outputs a null, then nothing is printed. This means that if that property doesn't exist, it will compare the empty string against `"1"`, and the result is false.


I think it would be better to keep the separation, by creating `org.zanata.seam.security.management.action.UserAction`.


This should be crediting Seam's IdentityManager, and I think it too should live in `org.zanata.seam`.


Again, we should move the Seam code into a superclass in `org.zanata.seam`, with header comments.


Should have probably noted, even though we'll probably handle settings with the HTML5 menu imo we should also have a simple settings menu built in, for people who aren't using the HTML5 menu for whatever reason.

It was simple to build in, and should be simple to extend too, so there's no reason not to have it really.


Related to #1


Just something to keep on the back burner: If our notion of a dataset becomes more complex, I think it might make sense to create a javascript 'class' representing a dataset, and then have this.state.datasets be a list of Dataset objects. The reason I think this is important is there is kind of an unspoken agreement right now between all components about what a dataset is. Right now, it seems OK since datasets are pretty simple, but if datasets start to encapsulate more information than they currently do, then this unspoken agreement could get more annoying to maintain.


Code looks great to me. Thanks again for helping


Thanks for the PR! I agree this method is a bit quick and dirty as written, starting with the name we gave it :)

In this method, it is really just checking to make sure the docker client binary can talk to the hosts Docker socket. Given that we are using the Docker command line interface, the server version will always be Null if the version is older then the packaged client. So there isn't a need to actually verify the version.

As an enhancement, we could get the version over the socket bypassing the command line tool with something like:

```
if [ -z ${server_version} ]; then
     server_version=$(echo -e "GET /version HTTP/1.1\r\n" |nc -U /var/run/docker.sock | grep '^{' | jq .Version)
     while ;do
           # verify version is <= client_version
           ...
     done
fi
```
That would make it possible for the while loop check to detect the server version is < client version, and narrow down the failure cause for the user.

Thoughts?


The null situations vary enough, that it would make sense to split the client and server check into two.  If client_version is null the failure is likely due to something around the `docker` command not being executable.


For the server_version null case, it should also include the possible reasons why server_version is null before exiting:

Ensure Docker version >= ${client_version}
Docker socket is mounted non-readonly.


This documentation shouldn't be in the private section. Perhaps the delegate *call* can be moved to the public section?


The docker command line tool can not get a version of the server if it is older. This means if the server version is not null, it is greater than or equal to the command line version. So the while loop will always succeed here.


Add a newline.


Move the comment up a line, to after `do`


Оновити  або очистити


Lets only add a single newline here.


The same as previous


Add a newline after this.


Now I'm 99% sure this is the culprit.

When I run emacs in non-daemon mode, I see "/dev/null" printed after the "Loading /etc/emacs/site-start.d/50pymacs.el (source)...done" line in the *Messages* buffer.  I believe it's coming from the "find" command in this change.

But when I run emacs in daemon mode, I don't see "/dev/null" after that line and see the stacktrace instead.

So it looks like the shell command, "find /dev/null -readable" seems to cause the crash in the daemon mode.

I'm running emacs on Ubuntu FYI.


It doesn't have to- changing it now


The module namespace doesn't match the filename. This should be `Relation`.


:+1: Thanks @tifayuki ! I just ran into this issue today, and you've already got a solution. You are awesome!


Can you make a new version of the pod with this update?


This class has unclear ownership semantics. Who owns the memory pointed to by instances of this class?


Well, it should probably be `active_record/relation/batches/batches_delegator.rb`.


Suggestion:
To keep this blog post short, I've created a simple script that checks which build configurations were used:


No, I didn't see that error on my machine, and `hydra.cryp.to` didn't produce that failure either. Not sure what is going on. Parallel building, maybe?


These should probably be const.


vshpwh_strat_integration_bug (nmerket) - i386-Windows-7-VisualStudio-12: OK (1763 of 1763 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)


On second thought, maybe not. If you want to make these things assignable, leave them as non-const.


Okay!  I'll remember that!


The issue is that if a side condition matches but does not succeed, then we may call the #fresh hook and increment the counter as a result even though the rule does not actually apply. Then when the rule does apply, the subsequent counter value skips over several values. Yes, a search will cause counter values to skip if we maintain a global counter, but it will still work if we maintain one counter per subject term, which is the plan anyway, as well as how the Java backend currently works.


OnePlus One.


Are you sure it does not also need read/write permissions?


Breaking compilation the day before the major release. Keep'em coming!


@isaku If you want to be more like PC, just add intimate invisibility 2 and hide particles. It isint too hard, and it keeps it mostly the same as pc :3


nl2br will convert tet not html content into html coded conyent. But what if content is already html coded. Don t you want to use dol_htmentitiesbr ?


You sure about this?


I meant to put infinate not intimate


Why can't we return a delegator from `in_batches` when there's no block given? Doesn't BatchesDelegator `each` work the same as `in_batches.each`?


done !  nice suggestion ! https://github.com/dchbx/enroll/commit/0f106bf


This comment has now become invalid, so please remote it.


This is breaking our Packer salt provisioner.

```
   virtualbox-iso: OK
    virtualbox-iso:  * ERROR: You need to allow pip based installations (-P) in order to install the python package 'tornado >= 4.0'
    virtualbox-iso:
    virtualbox-iso: Usage :  bootstrap-salt.sh [options] <install-type> <install-type-args>
```


@rtibbles look ok?


So the single cell is used for the sorting of cells in the regular case because we need to handle * cells as if they represent 0 or more children of the parent cell. Thus we need to treat the parent cell as if its child cell is the element sort in order for the algorithm to work.  However, in the case where we are generating a predicate from a B:Bag variable, we want it to represent the collection sort because that is the actual sort that is present at that spot in the configuration.


I'm going to add this functionality with slightly different changes, so that it's triggered by the lack of an MKL modulefile rather than a missing ROLLCOMPILER value.


True, but request will also fail if the item has been marked as read too.


Visual Studio 2015 version is 14, 2013 was 12, they skipped unlucky 13.


Unfortunately yes. We don't want to invoke reset when graph is transiting state. I must be honest, our renderer code is not handling threading well. But fixing this would require deep changes.


done


97903100_SolarCollectors (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed

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


No lo usamos al final?


Alright then.


I reverted the original commit to avoid the potential risk of exhaustion core's virtual memory.


I guess he means that there is a typo in this line. Should be `bin15` instead `bin`.


guess exist for the forward in the v2 (not the back-ward). But I don't think it is useful to add its support now. Don't work on this.


Actually in both lines he commented. God catch!
On Jul 23, 2015 11:11 PM, "Kacper Michajłow" <notifications@github.com>
wrote:

> I guess he means that there is a typo in this line. Should be bin15
> instead bin.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Underground78/mpc-hc/commit/c7e545c45587f78fb26014e96de90c6335997960#commitcomment-12329390>
> .
>


done. Thanks for taking care of the qualms of the autopilot. I'm afraid though that I pushed the patches to staging some minutes too late.


Yes, my reply was kinda random.


@edolstra  Nice! I had this on my mind recently - glad to see it implemented.

One question, though: in `corepkgs/fetchurl.nix`, don't you need to add an `inherit executable;` in the body of the `derivation { ... }`?


Shouldn't a Gen[Rational] also use BigInt as nominator and denominator occasionally? With this Gen the entire BigRational code paths will not be tested.


97903100_SolarCollectors (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Se usa en algun controller? Lo he quitado, en princpio el demonio ,
laravael solo usa ES.

Por cierto alguien sabe que sitio es mañana??

No lo he apuntado

El dijous, 23 de juliol de 2015, majandi <notifications@github.com> va
escriure:

> No lo usamos al final?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jlightyear/bootcampinc/commit/6a3e3a836fa9c8da56d33532307103991f536568#commitcomment-12329376>
> .
>


Yes we already know that.....


Тег нав только один раз на страничке


In the rc, it have the time algo for both gradients. Can you add that?


config.dnn.conv.algo_bwd? Check it for the 3 ops.


Fixed.


Fixed, changed to `linux_modules`.


The immediate solution is on the contents above.


can't understand why it doesn't use simple `applyNext`/`applyCtx` instead of custom guard `this._pageInit` and `local`

@tadatuta do you remember the reason?


*comments


@axelav whats brew? @azlyth do you know?


RadianceRefactor (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed

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


it looks like you replaced two smaller functions with a single (somewhat bigger one).
Is that right?
What was the rational/motivation for this?


Thanks @s0undt3ch, I'm using the packer salt provisioner so was a bit confused on how to pass
the -P flag. However, I believe I've tracked it down to the **bootstrap_args** in my provisioner.

```
{
"type": "salt-masterless",
 "local_state_tree": "salt/",
"local_pillar_roots": "pillar/",
"bootstrap_args": "-P"
}
```

Thanks


I just remember — it's because of repeat block itself in new `ctx`


Me da mucha alegría las cosas que se están implementando!!!  @andru255 @VictorJSV @carloshs92 @jjhoncv :100:


so `once()` API looks reasonable :+1:


I don't think this requires a code change, but if this is the new condition, then the following checkState() will never fail, right?


Daylighting_Speed (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


I had a closer look at this and I think the "duplicate" cookies you see are not duplicate, they are for different domains. There is a sessionid for `store.steampowered.com` and one for `steamcommunity.com`. I guess only the store one is there after login because the "transform" sends you there.
So you will probably have to do a HTTP GET to a `steamcommunity.com` page prior to send the POST for the correkt cookie to be in self.session.cookies (use `self._log_cookies()` to check).
As other fields (like `Summary`) are reset if they are not send with the POST data you might have to fetch all current values anyways so you would have the correct sessionid then.


@acfoltzer Yeah, I think that makes sense. It seems like it isn't an issue now, fortunately!


I don't have an issue not including this attribute, but why is `pwdChangedTime` any different than some other changeable attribute such as `domainsVisited`?


Good idea. Thanks!


This is only `require`d, not used, nor do I see a reason we'll use it any time soon. Ditch it?


This is only used once in `bin/www`, as a weird replace to `console.log`. erm, why?


@bueller yeah, its this new hipster thing where you steep coffee grounds in room temperature or cold water for an extended period of time.


Again, not used. Is weird. Me no gets it.


Where does it print the stacktrace? It's exactly the same as the non-development handler.


You forgot to add javadocs :(


RadianceRefactor (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


I haven't noticed any changes from the user/UI perspective.  Does this commit/refactor cause any change there?


life saver commit.  it was not returning a promise calling a great deal of inconsistent issue.


wtf is this, why


Hola, Estimado, una consulta, estoy probando tu sistema y esta genial, lo malo que no encuentro donde agregar los Equipos nuevas, solo hay la opción de Asignar Equipo, espero que puedas ayudarme, así también puedo ayudarte en algunas cosas que se, esta muy interesante tu sistema, se ve que tratas de mejorarlo cada día, espero tu respuesta amigo.


Daylighting_Speed (DeadParrot) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I was just about to ask you for this


a


meh, not that useful


Excellent! Was going to do this myself, but didn't think to use submodules, which is very cool. :+1:

Would you mind doing the same for RTMidi? I will be implementing Cocoa MIDI shortly so if I use RTMidi it'd be great to use the latest version. :smiley:


This is why we need to sort out what should be used for `User` uniqueness.

Should we include all attributes, including timestamps, and update our unit tests to not check equality between the `User` being saved and the `User` being loaded because of changed timestamps or other user meta-data. Or should we only base uniqueness off of only the guid? Or what's a happy medium?


I disagree that it has to be on a separate commit. In this transition, we can only say the code was inspired in file `path/to/file.py`. I'm going to put it in the same commit.


When cleaning up, now the AME Source folder will also be moved to the Trash.


Effectively, this means a new db connection is opened on every request. Why is it so php?