I don't know why I pulled this request if my file was empty. I'm sorry.
I just wrote directly in the file  the SQL Statement needed without the SQL Statement results.


Oh, good. Looking at how scouts are using handleChanceIncs, it looks like they're not processing themes for the exact same reason monsters weren't, so if this works for monsters it should hopefully resolve issue 297 as well.


Why?


Cool. Thanks for doing this.


I don't think that's a problem in practice...


Debug messages like this shouldn't be in the view. There are gems that can inject this other ways, but in particular I want that center column to look the same way in dev that it will on the live site.


What fascinates me is the ability to add attributes to certain elements.  I'm not sure if href itself is one of them, but the interplay between referencing something further down a page, just by tagging that item, are starting to open up so many possibilities, and really make me want to put them to use. And I'm really starting to like HTML because of the simplification it brings to methods of navigating a website, i.e., you don't have to list the whole URL if it's within your site.


I like the idea of the functionality but I would prefer if the implementation did not rely on Objective-C runtime functions.

Instead, we could easily add a new method definition that includes a keyPath argument. It will require a bit more code, but I think it results in a better implementation.


Also, please don't commit on master. Open pull requests from feature branches.


ahh good point


TeamCity pull requests :: netty [Build 242](https://garage.netty.io/teamcity/viewLog.html?buildId=1559&buildTypeId=pulls_netty&guest=1) is now running


It does, but there's a caveat: If a scout ever is of secondary race, it uses primary race themes. Fixing that and then stopping for today.


I forgot to put here:
source data/import-table-structure.sql;


Hey Cal, I'm not sure if you noticed, but this repo contains an existing Jekyll site. What you've added works, though you have also removed the Jekyll functionality (that is, the old structure), is that intentional?


Our core bindings is the forward slash as a delimiter, the default repository should just be an instance of TranslationRepository stored as 'concrete/translation/repository' or something like that.


is it worth mentioning that the failure occurs after the MR is done? That it does not fail right away when the an existing partition key is found?


Does this require any changes to the JavaScript or does WordPress do the character replacement?


Hey Kevin,

I added some documentation for building Sigil on Linux to the repo. It damn
near wore me out to do it, but I knuckled down and got it done. ;)

Later,
Doug
On Sep 21, 2015 11:44 AM, "Doug Massay" <douglaslmassay@gmail.com> wrote:

> They're fine. I just hadn't noticed them before (or had forgotten if you
> said something). The "greater than or equal to" implication of the symbol
> makes perfect sense, now that you mention it. I just wasn't making that
> connection for some reason.
> On Sep 21, 2015 10:38 AM, "Kevin Hendricks" <notifications@github.com>
> wrote:
>
>> Hi Doug,
>>
>> If you mean adding the Greater than or Equal to char (>=) symbol in from
>> of the fields for setting the Plugin Engine, then yes, I added those when I
>> thought we were going to go with Python 3.5 and wanted to make it clear
>> that any Python 3 >= 3.4 should work here.  They appear just fine on my
>> Mac.  If they mess anything up, please feel free to remove them.
>>
>> Take care,
>>
>> Kevin
>>
>> On Sep 21, 2015, at 08:22 AM, Doug Massay <notifications@github.com>
>> wrote:
>>
>> Hey Kevin... just checking if you purposely made these changes to the
>> PPluginWidget.ui form when you updated the documentation in this commit, or
>> if something weird happened. I just noticed those symbols showing up in the
>> Manage Plugin preferences widget the other day and tracked it down to this
>> commit.
>> —
>> Reply to this email directly or view it on GitHub.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/dougmassay/Sigil/commit/2730aa52f982d7480c1f03af4b8297de8f0a34ba#commitcomment-13344193>
>> .
>>
>


LGTM


#5167-ExampleFile-VariableRefrigerantFlow_FluidTCtrl_5Zone.idf-Warnings (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1880 of 1881 tests passed)

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


I just checked over the Example here and it looks like I will need to update input.js, and should probably extract the text from admin.js (used in ACF5). Are you using ACF4 or 5?

https://codex.wordpress.org/Function_Reference/wp_localize_script#Example


super angular.js ese nombre


Need update the JS too. I'm using ACF4.


FYI, css comes from here: https://github.com/kilink/ghdiff/blob/fc471d1b0368d03ca69478979872e7f109122962/src/ghdiff.py#L15-L48
(I imagine there is a better way to get the css than copy-pasting)


I sent one commit for "js/input.js" too.


thanks! Pull both to DigitalO and my machine, n'esct-ce pas?


Bingo!


I am getting the following message: ERROR INVALID SCOPE REQUESTED. More than likely user error. Can you direct me please.


s.selector is just a string at this point, not a DOM Element.


first file for GA FEWD course


Got it. I will delete the hidden folder just before git.repo.clone for every repository.


Great explanation, I'm looking forward to learning about data locks.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (809 of 809 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-21.79%25-green.svg)


> I'd actually argue that having it here is better because it'll remove some races.

The race is still there, but you have reduced the window of opportunity. To remove the race altogether, it needs to be part of the same transaction.

> Usually the order is clear resolved -> set status. If a resolve call gets in between these two it will remain set and not cleared even though the unit is not going to be in an error state anymore.

Do we need ClearResolved at all then?


aasdasd


I just found that we still get an error when attempting to parse that tea with the TM on it.

Adding another line afterwards, like:

tea_name = tea_name.decode('utf-8')

does the trick for me.


@wonder-sk brilliant work, thanks.

The only thing I feel could be improved is to add a visual indication that a group is set to be mutually exclusive.

How hard would it be to change the layers' check boxes into radio buttons? That would make it utterly clear, and would be a nice visual touch.

Otherwise, a simpler option would be to have a different group icon.


This text might be useful somewhere:
```
# we'll eventually want to move away from
# including `orchestra/accounts` separately from the rest of the orchestra
# URLs, but we're
# limited in doing so.  The `orchestra:` namespace on the main
# orchestra include below causes all account urls to be
# namespaced into orchestra as well, which breaks all of
# the django registration views/html.  We'll either need
# an upstream fix to django-registration to support other namespaces
# or to reimplement all of the default `registration.backends.default.urls`
# views.
```

and

```
# Logout then login is not available as a standard django
    # registration route.
```


@steelbrain great!!! :D


# VICTORY!
Now you should join up!

![](https://media.giphy.com/media/I0mekTKZc0oak/giphy.gif)


before we rebuild here anything we have to backport some fixes.


Nice, maybe move it over to the right so it is inline with all the rest of
the elements.

On Mon, Sep 21, 2015, 5:51 PM Trevor Dowds <notifications@github.com> wrote:

> [image: screen shot 2015-09-21 at 5 51 16 pm]
> <https://cloud.githubusercontent.com/assets/5726560/10008884/65cb4748-6089-11e5-805e-5f9f23e13854.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bmun/huxley/commit/30de503b675e00214ccc53ff8090693a17946682#commitcomment-13356667>
> .
>


Ummm... nope.


Also, this shouldn't be kept as a local reference. It should be passed in an update function.


no entiendo tu codigo


And we don't initialize the player here. The code that loads the level takes care of that.


could we use c.PeerURLs directly?


At the moment, when there's damage and while the screen is tinted red, the screen will shake randomly within a range of +/- 1 degree. What I could do is reduce that range based on the amount of damage.


很好，但有几个问题：

* 测试用例的编写包含**用例编号**、**测试目的**、**测试步骤**、**预期结果**、**实际结果**。请参考[这里](https://github.com/heweneayun/EayunOS-testcase/blob/master/Function_Test/WebAdmin/%E4%B8%BB%E6%9C%BA/Fence.md)的格式来写。

* 你写的是测试用例，不是管理员手册，不用太多篇幅来写用法，主要写出具体的步骤即可，并分成多个具体的测试。比如你可以写这么几个测试用例：

  * 备份数据库
  * 恢复数据库

  然后在每个用例里面写上具体测试步骤

* 改动的文件不应该是这个，你把改动放在[这个](https://github.com/eayun/EayunOS-testcase/tree/master/Install_Test)目录下，

* 我们目前使用gitbook来展现文档。但是该目录很久没更新了，需要与 gitbook 兼容。你可以研究一下 gitbook，在该目录下加一个 SUMMARY.md


**manjaro-tools** is now at [bbc69f8678d](https://github.com/manjaro/manjaro-tools/commits/bbc69f8678d241725c1301daab25063c0f796eb3). Any changes made so far should go into **0.9.13** and first tested with **manjaro-tools-git** for a while.


-Shows a list of album titles returned by http://jsonplaceholder.typicode.com/albums with the name of the user the album belongs to.

-When an album is clicked, show its photos returned by http://jsonplaceholder.typicode.com/photos as thumbnails.

-When a thumbnail is clicked, the full sized photo should be displayed.


div part is not fixed yet.


> By global state you mean it should do the whole going through api, state and back to the remotestate watcher?

That is what I meant, but when you put it like that... ;)
On second review, this is okay.


那麻烦你解释一下Unity和Cocos2d-x里的代码？


TeamCity pull requests :: netty [Build 237](https://garage.netty.io/teamcity/viewLog.html?buildId=1538&buildTypeId=pulls_netty&guest=1) is now running


+1 for updating readme


Hey mate; is one of your goals to get this working on the new iTunes?


vcvc


there is no need to install Capstone from source, as it is available in PyPi with package "capstone": https://pypi.python.org/pypi/capstone
so you can put it into requirements.txt.

this is a nice work, keep it up!


I see, the s_true[:,0] is much cleaner, thanks for the tip!


Looks good. Thanks


Good. Thanks


Excellent work dave! I can't wait to read how it works.
![1f44d-2](https://cloud.githubusercontent.com/assets/10352970/10009847/9a602c5e-60b0-11e5-8165-21047a1da96c.png)


Some small indentation problem in javascript (in scope). The rest looks good.


If you could indent your code it would be most appreciated.  Also it is best practice to name these type of functions in lowerCamelCase.  It doesn't affect you code working but helps readability.

The only thing you are missing is preventing the default action i.e.

     function readMore (e) {
         e.preventDefault();
         $('#show-this-on-click').slideDown();
         $('.readmore').hide();
         $('.readless').show();
    }


Semicolons are optional but highly advisable!


Same thing with the preventDefault here


Cache clear on view? Why? What is the point of extensions.updates cache if it will be always cleaned.
Please revisit this.


I completely agree with this statement. The examples used are very straightforward and very easy to understand. Something that would have been hard to understand, he makes it look simple and very straightforward.


原先是跳过这两个字节不写吧。。感觉区别不大


TeamCity pull requests :: netty [Build 238](https://garage.netty.io/teamcity/viewLog.html?buildId=1539&buildTypeId=pulls_netty&guest=1) is now running


I still recommend to use systemd-bootx64.efi provided by systemd here.


This is the most efficient implementation I have seen for getting the measurements. I timed it using the method from class today (timeit) and it took ~3.51 µs per loop. Well done!


INTEGRANTE: JOSE ALEXANDER MUÑOZ DELGADO


They weren't removed just consolidated down to single commands since CLI scanning now supports multiple paths, comma spaced.


love that slippery slope bro! Thats why were here.
![1f44d](https://cloud.githubusercontent.com/assets/10352970/10009911/727f066e-60b1-11e5-87e6-134a8b361568.png)


No need to do an `else` branch here, since the `if` returns. avoiding extra levels of nesting is nice. (Same with line 64.)


@suninthesky , just realized that there's one bit of functionality that wasn't transferred over from the original page: the recalculation of the "count" next to each tag as tags are clicked.

Check out the master branch. In marketplace/index.html you see this at line 61:
```
<li style="cursor: pointer" class="list-group-item" ng-repeat="tag in tags | filter:{tag_category_id: category.id }  | orderBy: '-taggings_count'"  ng-click="tagFilter(tag)"  ng-model="tag.active" btn-checkbox ng-class="(((tag.taggings_count == 0)&&(!tag.active))||(isLoading)) ? 'better-button-disabled' : 'better-button-able'">{{tag.name}}<span class="badge">{{tag.taggings_count}}</span></li>
```

And in marketplace_ctrl.js.coffee at line 9, 16, and 50 you see functionality that updates tag counts on the fly, as the user applies more filters. Is it possible to pull this same functionality over to your new, sweet, horizontal filters?

Taylor


Bjarne hates this but I'm in favor of this returning `const QString&` instead, although it's more or less a micro-optimization in this case.


I believe that it takes 34 seconds in total, here was my algorithm:
((B/E)−1) + log_2(E)
And plugging in the parameters for this problem, the answer is:
(256/8−1) + log2(8) = 34 seconds

Let me know what you think.


I have just use container by main element  for each images but event hover of you using mouse window, i fixed it to move on element, it work.
Please see image below
http://prntscr.com/8j27hy


I found some of the terms a little confusing too. I thought I was going to be the only one that found the article confusing. It provides all the necessary information, but some of it was just confusing and hard to understand if you know little about HTML.


I'd think this would be qError() or something?


Trying to apply comments to every single function. Work in progress.


Bare pointer. Should probably be unique_ptr or shared_ptr or QSharedPtr or something. (Otherwise you have a memory leak because you don't have a destructor that frees it.)


Who is this?


I really don't see any advantage.
Its just the iso we boot in efi, that has nothing to do with init. And we don't use gummiboot itself, we just copy that efi file to the iso efi boot dir.
And the installed system uses grub.


2.3.3 ;)


Can you use a real license like `MIT`?


rawr


Hi @tomantonyp, as far as I know, file response is not supported yet in the default Java client codegen and was not support before either (it is supported in the [`okhttp-gson` template](https://github.com/swagger-api/swagger-codegen/blob/master/modules/swagger-codegen/src/main/resources/Java/libraries/okhttp-gson/ApiClient.mustache#L515) though).

Could you post your the relative part of the response definition in your spec? That is, what `type` and `format` your spec uses for the file download endpoint?
In addition, did you do some hack before to make file download work?


Much better now. Just remember to give ALL your members the RAII treatment. :)


hello


Following the logic from the previous explanation, I think this one should be:
(256/128−1) + log_2(128) = 8 seconds

Where the most possible employees we would want counting is 256/2=128 in order to keep them all productive (as employees can only had 2 numbers at a time). Let me know if you agree.


It also supports JSX. Not sure whether Syntastic has a notion of that though.


是否可以用它省事儿些呢？http://laravel.com/docs/5.1/eloquent-mutators#attribute-casting


@fabricioflores `ProfileEditorTree` debe pintar el nodo de otro color para indicar que fue seleccionado


It is simple. The efi-file gets updated with each **systemd** package. Soon or later you stuck with an old not working **efi-file** on your end by keeping the outdated file from **gummiboot**. That's all.


This is an amazing thing to point out. I think I  missed this part because the article was a little confusing to me. For 99% of the pages to be broken and we can still surf is hard to imagine! But it actually happens everyday! Wow


a


Once again, using my logic from above and adding in the communication overhead, I think this one should be:
In the case of having infinite employees or our maximum needed of 128 we arrive at (256/128)-1 + 2*log_2(128) = 2-1+2*7 = 15 seconds. Let me know if you agree.


PR at #13.


wow, this sure looks good in the graphs


Previously, `select2.css` was downloaded as `select2.scss`, which allowed sass to `@import 'select2'`. I believe this change was unintentional.


Perhaps I am slow here, but what does this file exactly? What's the function, what does it provide?
We use on iso prebootloader, seemingly static efi shells, and this efi file.


Oops, yes! Nice catch. :) It's fixed now.


What is the stack trace for the new error?
On Sep 21, 2015 7:31 PM, "ddossant" <notifications@github.com> wrote:

> still getting a null error. Here is what my config file for the trace file
> looks like:
>
>
> C:\Users\IBM_ADMIN\Documents\GitHub\oltpbench\config\traces\wikipedia-100k.trace
> 10
> 10.1.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oltpbenchmark/oltpbench/commit/9ec650bcf0518de44d780c575bbf7a41264b6991#commitcomment-13355840>
> .
>


@angular-ui/grid How did this break the build?
I think reverting would probably be the best course of action here.


Guys, i've been seeing this problem :

Do you guys see the same thing? Template inacessible due to protection?
<img width="937" alt="screen shot 2015-09-21 at 7 52 11 pm" src="https://cloud.githubusercontent.com/assets/6835537/10010062/6b97ebde-609a-11e5-90a5-3c9844fdc044.png">


Yeah—sorry! I've been using it without any obvious problems. I say go for it!


Caller


I'm not sure about this one, I have seen a few different explanations but I think that the most efficient bag strategy would be closer to the amount of time it takes 8 employees to add the bags, ~34 seconds (plus the communication overheard so maybe near 40 seconds).


I like how you summarized the how did we get here article. I have said a few times that it was a little bit confusing. Reading your summary helped me to understand a few concepts that i probably missed or didn't understand. You do a nice job of explaining and summarizing everything well in a simple and easy way.


I suspect that this happens very rarely, so I'm starting with this.


Well done! You have efficient, clean written code. However, you could have used some more explanation behind problem 5.


Also, could reduce the payload traffic by adding `?include_fields=has_unconfirmed` param to the request.


test


I agree the tile he used about "everything we know about "XHTML" is wrong. The title right away grabbed my attention and made me focus on that section a little more. I didn't even know anything about XHTML, but i like how he explained that part really well and cleared up some of the things that might not be true.


Matrix_matrix_multiply is definitely taking me a while. Have tried a few approaches but they get complex pretty quick. Thinking of using the matrix_vector_multiply function by passing it the first matrix and the columns from the second matrix as vectors - the output from that would be each of the columns for the final matrix.


TeamCity pull requests :: netty [Build 242](https://garage.netty.io/teamcity/viewLog.html?buildId=1559&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:55


Hi Mamta,
Don't we need the ehcache dependency in pom.xml, unless the new dependency of spring-jms takes care of the caching issue?


I think `Math.max(x, x) === x`, if `x` already is a Number. Was it `Math.max(b[0], b[1])` that you wanted to write?


I will not be in front of computer to the end of the week. If you have
better solution and time will be great.

I was thinking of more general refactoring of the cahing in imagekit but I
had other things to do and not enough time to implement my idea.
На 22.09.2015 00:23 "Matt Deacalion" <notifications@github.com> написа:

> Are you still working on this Venelin? Django 1.9 Alpha is due for release
> today and ImageKit is not compatible with it. I'm available to take this on
> if need be.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vstoykov/django-imagekit/commit/c26f8a0538778969a64ee471ce99b25a04865a8e#commitcomment-13353513>
> .
>


That beats the number of steps to "contributing to MOOSE"!


oops, yes


Good job using the heading 1!


What about adding some general text?


Not my intention. I just wanted something to work temporarily because Dwight wants to free up the server that we're not using.


I like how this is being tied in with last week's discussion. I personally might see it as a point at which that particular branch progressed forward, but that PART was definitely linear.  Because to me this all sort of a part of the same offshoot.  To me, it's also cool that technological progress is being "workshop'd" among people all over the world, in order to see what the best possible solutions are for something for which there is a need.


Currently we have two bubble-dir-ctrl:
1. bubble-hw-export-ctrl
2. bubble-hw-import-ctrl
It should be one single bubble-ctrl that is fed with different datasets.


Will it be with logger class?


This line plus the commit message are priceless


Ah, okay. Well if we are going to make the switch to this repository, you need to update the DNS record(s) and the CNAME file. With that, the content in the repo before this commit should work.


Hi. Thanks for the fix. I pushed this and a followup fix to take care of the other places where HTTPError was imported


See also b3658bc0.


Single batch is impossible


Yes


no, admin is getting that dependency from ambra. We don't have to mention it here. That spring-jms thing is different. Somehow for spring-jms we were getting old version, so I explicitly need to mention in pom to use latest version of spring.


OK, I logged into the servers and it seems that we have iPhone 8.0 SDK installed so it should work with everything from that SDK version. I think newer SDK versions require a newer xcode.


This is unfortunately not correct. It should use the Source class from verilator.py. I'm not changing this now however as most of the verilator stuff in section.py needs to be moved to verilator.py instead


Will do.


Removed the extra line to maintain the standard formatting


Also of note here is that this removes the removal of asset orders which fixes the 'only one txn was processed when two should have been' because of mucking around with the generator.


fine with me


@ctrueden should there be net as well??  I think I have the latest preference file, but I don't see net...


Very nice. I like.


Тут прям что-то очень странное все)

Вместо `function formValidation() { ... }` и `formElement.onload = formValidation()` (тут ты вызываешь функцию и то, что она вернет — `undefined`, записываешь в `onload`) надо что-то вроде такого:

```javascript
restoreFormValueFromCookies();
hideTip();
for (var i = 0; i < FIELDS_TO_VALIDATE.length; i++) {
    FIELDS_TO_VALIDATE[i].onchange = hideTip;
}
```


Hrm, this bit wasn't in the final change. Should it have been?


Thanka to have found so many bugs :)
Open a new issue with a link to the logcat, I'll fix it ASAP.


Should we just remove this?

We may also want to go through and remove everything that subclasses Event.


I like to think of these closing tags as the pressing of buttons. Too often I only think of writing I do as something which is constantly moving forward, and only exists at the point at which my cursor (or pen) currently is.  For computers, they need the constant presence and command of the writing, and so the simple action of my starting to write in bold and then hitting control-B (or pressing the BOLD button) will continue to exist for them long after I'm done actually doing it.


That removal of everything else wouldn't be blocking this though.


Notr: eclipse Mars has a bug in the formatter which causes Imports to be sorted other than expected.


TeamCity Dirigo Edge :: Deploy Pull Request [Build 81](http://tfs2013:8081/viewLog.html?buildId=815&buildTypeId=DirigoEdge_DeployPullRequest) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


@aptxkid Yeah it would -- that's the intention of making it recursive with a flag to make sure a `FysomError` doesn't escape. If it raises some other exception though I think I'm fine with that -- we'd want to know about that.


if we wanted to keep a check like this... another (kind of dirty) idea is to use the debug http interface of the scheduler to query the registry state. it's a little further removed from accessing internal state like this, but depends on the output of a debug interface (which may be OK for testing, as long as we leave xref comments or build some simple funcs for ser/deser).


I have EasternEdgeRobotics.github.io/CNAME updated for www.easternedgerobotics.com, and I edited the DNS info in doteasy as www.easternedgerobotics.com IN CNAME easternedgerobotics.github.io


tested and working


pause_before_proceeding isn't needed now that we are running headless - previously, IJ1 macros would be run asynchronously in some cases and this let the user watch the UI and decide when it was done.


Fixed.


I guess so, yes


Can't show ImageJ now that it's headless.


See above - dead code


To clarify -- the `__trigger_fail_on_error` flag is there to prevent the stack overflow/infinite recursion.


This was also a legacy of asynchronous operation.


In the case of this report, we believe that the potential for deadlocks and the report getting a timeout exception (especially when this report is requested during times of heavy check scanning/processing) is more detrimental than the chance it reads data from an uncommitted transaction. We think users would expect different (inconsistent) results during times of heaving processing, where they would not expect it to get an error.


You are totally right!


Actually, this seems very strange. Is it true that any output to `STDERR` from SQLPlus* should be considered an error, even if SQLPlus* itself returns success?


Runs on Mac headless now :+1:, right?


Should I add in comments for the other methods in the PR?


Always run the Javabridge headless. Also note that you shouldn't bother running with the "--no-headless" command-line switch.


Previous error looks better from JS perspective. I am looking into what we can do here for better error messaging


Until cars DON'T use wheels, Alex! :) What I actually wanted to ask about, though, was if anybody understood exactly what it was about XHTML which made pages shut down with any kind of error. Was it something inherent within the coding which made that happen, or was it something that the W3C itself ruled on and then all the major browsers had to comply?


Thanks for the explanation David.


tested and working


Most changes here are dedicated to running ImageJ on any thread instead of creating lambdas to be run on the UI thread.


Maybe preserve the comments that say what these are. "XGG_Computer_CV01_04751" doesn't exactly scream "Aircraft carrier full", and the logic that makes this work is a bit of an amusing hack.


How was this an problem? Should be the same, no?


Don't show or hide images. Headless.


Many WindowManager functions removed. WindowManager really only supports the temporary current image at this point.,


It would be good but I'm not too fussed about it.  I'd probably do it for the ones that aren't obvious, like "insertPill", and maybe the "lock/unlock" ones, but the rest of them are fairly intuitive.  So, it'd be good form to have javadocs on all of them but I wouldn't block the PR on it or anything.


**require-jsdoc/require-jsdoc**: Missing @jsdoc for autocompleteFor


**Eslint**: 'return' outside of function


**require-jsdoc/require-jsdoc**: Missing @jsdoc for componentWillReceiveProps


This is a rebase error. We changed the Note in w3c@ed30005.


@flyingsparx Do you have/want/need a Django example? I could probably whip that up for you. Just lmk.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1349 of 1349 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 upgrade is surgically designed to pick the minimal upgrades needed to make things work on Java 8.


You are creating variables on the view


This one didn't have the FocusArmy check that your generalised version now has for every VO. This is probably a non problem, but just make sure this sound actually works still.


Good catch!


@vanmh Added some comments explaining what I'm doing here: https://github.com/kiva/vertex/commit/fa0289069ba175d2e85e8532c411d07384f07ebc


Great!  I'll test/build against 8.0 then.

On Tue, Sep 22, 2015 at 1:07 PM, codenameone <notifications@github.com>
wrote:

> OK, I logged into the servers and it seems that we have iPhone 8.0 SDK
> installed so it should work with everything from that SDK version. I think
> newer SDK versions require a newer xcode.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/codenameone/CodenameOne/commit/7a1187ffc268d0ab77ad3e74e81bbff06ddd34fb#commitcomment-13376565>
> .
>

--
Steve Hannah
Web Lite Solutions Corp.


why is `$window.scrollTo(0,0);` necessary?


Why are we forcing dismissOnEnter to shadow the inputMode flag?


I'm not convinced that this needs to be in its own package


ah, right you are. I thought it's handled later, but was wrong - thanks!

On Tue, 22 Sep 2015 at 21:05 Ian Lake <notifications@github.com> wrote:

> It seems that this may lead to duplicate Uris in the final ArrayList (ones
> added in getData() and from clipData). Can you update uris to be a Set
> and then convert it into an ArrayList before calling startService()?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/muro/muzei/commit/3b644ffe3c4c1c8207ac38fe6312a42c6240139d#commitcomment-13375414>
> .
>


This table is a constant, move it out of the function: we don't need to reconstruct it every time we play a sound effect.


What the description says? It's there to avoid having to override the entire package set when changing supported options. Whether you'd want to use it or not is entirely up to you.


I didn't see the updated version of this haha but this is better. BUT if you have noticed, it gives us some broken images because sometimes there is no item6 or item5 etc... I would like that to be fixed before merged with develop.


Good


I don't think this new explanation of column width is an improvement, other than the link to the Microsoft documentation. My issue is that "the number of characters in the default font" adds neither accuracy nor precision over the old explanation. Being truly precise requires a ton of verbiage; maybe it's enough to aim for slightly better grammar and slightly greater precision. Something like

[begin quote]
The ``width`` parameter sets the column width in the same units used by Excel, which is the number of digits that can fit in the column using the default font. The default column width is 8.43. For more specifics, see `this article <https://support.microsoft.com/en-us/kb/214123> `_ in the Microsoft support documentation.

Unfortunately, there is no way [etc.]
[end quote]

(I tried to make the comment preview look right. There's an extraneous space before the closing backtick in the link to get it to show up as a backtick.)


s/time.Duration(1)// ?


TeamCity pull requests :: netty [Build 243](https://garage.netty.io/teamcity/viewLog.html?buildId=1560&buildTypeId=pulls_netty&guest=1) is now running


Looks like this is more leftovers from ZeP's defunct attempt to work around the targeting-stealthed-units bug. (It was a ridiculous amount of code and didn't really work).


tested and working


I think *user* consent would be better.


TeamCity pull requests :: netty [Build 243](https://garage.netty.io/teamcity/viewLog.html?buildId=1560&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:53


@ctruden this is a monkey patch to the Java 8 Nashorn / Rhino fight. I check to see if the engine is Nashorn and if it is, I run the mozilla_compat script similarly to how it's done [here](https://github.com/scijava/scripting-javascript/blob/167d606b8ef18e502b8f16a530eccd2c55c5cf12/src/main/java/org/scijava/plugins/scripting/javascript/JavaScriptScriptLanguage.java#L88). I'm a little shy about submitting a patch to Wayne - I am guessing that this works in the context of ImageJ 1.0 somehow. LMK if I should do that.


@matjames007 because there's a scrolling issue with these popups.


Fixes #32.


return instead of break poll


We can now use formated json, can't we? It would be more readable.


What's the tory on tatus?


thanks!


Hi Rikki, you also need to change the name of your GitHub repository to be "portfolio" by clicking the `Settings` button on the right hand side of the page. The `config.yaml` file sets the base URL of your site, but GitHub is ultimately responsible for setting up the publicly-accessible URL for the project. By default that URL will always be `http(s)://<username>.github.io/<projectname>` unless you specifically override the settings. Read more [here](https://help.github.com/articles/user-organization-and-project-pages/#project-pages)


97668326-WholeBuildingLoad (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1848 of 1882 tests passed)

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


Woran hast du fest gemacht, dass das cg-verfahren richtig läuft?


er, surely this should be outside the process: block, otherwise it's going to block here if it comes back after first hitting 157


Gah, this is so good! The whole site is looking amazing!


Since this a DB schema update, we also need a migration.


never mind, now I see the buffer


This is changing slowly and I've just wrote about ck value on the forum under Suggestions and ideas.


Pass a valid parent element to the `$mdDialog` so that the dialog can be properly displayed.
```
var parentEl = angular.element(document.body);
```
That's a sample... check out angular material docs.


Thank you @hackdna I'll add that as well.


Sorry, I did not mean to push this :-1:


Removed some dead imports to fix missing classes caused by upgrade of ImageJ.


I've just wrote about ck value on the forum under Suggestions and ideas.


That scroll puts it in the right place - it's a workaround tremaine showed me and it gets the job done. Any alternative ideas?


LGTM.


tested and working


The markup dit eat
 \<BISHOP\>
in your fragment but I think your code might indeed have the same effect ;-)


To PRE.


When you upgrade to the latest schema (which is kind of done for you by the OMEXMLService), it really messes up the namespaces which makes the upgraded XML unloadable. I removed the OriginalMetadata elements and normalized a lot of the namespace use to make it work with the OME .xsl upgrade files. IMHO, Bio-formats typically makes its own OME-XML, so it never runs into the problems I saw.


I must be missing something here - overloads of ```begin/end``` that Do The Right Thing for ```char```?


Actually - this would be done in the controller in the index method


A value of '0' is truthy.  No need to test for it explicitly.  It should be removed from this test and the other location in this file.  Do you know why we are also testing for 0 (the number)?  I'm not sure we do this anywhere else in the code.


@GantMan Does this line mean that if the device is currently rotated into landscape that this method will return that landscape mode is supported? The reason I ask is because our app only supports portrait, so I want to make sure this is returning the correct value. Should this somehow check the `app.interface_orientations` ?


Why do you need the page variable at this stage? Why not bind this to the whole document?


We've already tried to implement swipe gestures by binding them to the whole document (everything you can touch on your smartphone screen), Unfortunately gestures worked only every 10th swipe (Android 5.0 to 5.1), sometimes not working at all. In order to fix this, we've tried to calculate with screen x y coords --> This also wasn't working, so we've removed swipe gesture code later on by this commit:
[0763222f0c6c5b65c878b629901f220bdeaa7135](https://github.com/HTWK-App/Smartphone-App/commit/0763222f0c6c5b65c878b629901f220bdeaa7135)
See lines 160 to 166 in main.html.

Beside this, your code will break swipe gestures in menu entries canteen and timetable.


@0x00B1 this branch makes the in-situ sequencing pipeline work and more importantly makes changes that are generally needed to make IJ1 macros work. I want to test it on the Mac to make sure I didn't break things there. Will submit a pull request when that is done.


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


I guess this won't do -- they're going to want it in single quotes...


Closes #16


Implement auth


@bnorthan Yes you're right, `net` should be listed too, in between `javax` and `org` of course.

@dietzc Thanks for the reminder. I still use Luna.


Cierra Issue https://github.com/InspectorIncognito/androidApp/issues/7


So you had the right idea here, but actually went just a tad too far. We should change this logic so that before you show the title, you decide if that title should be its normal self, or the word "CENSORED". So instead of putting another loop here, just make some kind of conditional that determines whether to show title or censored based on the moded value of something like the post.id.


Tested and seems to work nicely, this one is straightforward.

(ping @ajhager)


tested and working


Complier error. Fixed.


These icons and splashs are great. We will use them. Thank you!


Use `SGEN_ASSERT`.


You could also write `@sponsored_post = @topic.sponsored_posts.build(sponsored_post_params)` and then you wouldn't need the line below.


I just realized that these are technically not needed. They came along while copy pasting from leveldown :) I can remove them if you like.


101682488_EvapCoolPrimaryAirSizing (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1351 of 1351 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-70.26%25-green.svg)


Will `react >= v0.14` be a hard dependency for react-leaflet 0.8? `React.render` is deprecated in `v0.14`, but it's still included, so this seems overkill.


You are right. My mistake. I meant that we would want to use GitHub login as externalId. Currently we are using: [1] id, which is unique number like 1, 2, ... that is assigned to the user. The `login` was absent in Google, and that why I had to use is with Google, but for GitHub login is always there. See also discussion on this issue: [2]. But, anyway, even when we would add this option it should be called something like: `use-login-as-external-id` or some such.

[1] https://github.com/mdxp/gerrit-oauth-provider/blob/master/src/main/java/com/googlesource/gerrit/plugins/oauth/GitHubOAuthService.java#L102
[2] https://github.com/davido/gerrit-oauth-provider/issues/33#issuecomment-97393541


**no-undef**: "expect" is not defined.


**no-undef**: "ReactDOM" is not defined.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "before" is not defined.


**no-undef**: "describe" is not defined.


**no-undef**: "it" is not defined.


Awesome thanks!


tabs vs spaces didn't work out too well...


Nice, so this is an alternative method to running `cpuid_dump_kernel.bin`.


Thanks for the comments!


**no-unused-vars**: clone is defined but never used


I need to review all the places I've used temporary member functions (like copy, divide, etc.) and replace them with Ops, when the required ops are ready.


OK


Is this really the same as the removed code?


Hey @stevemdixon! My intuition is telling me that we can work on how things look in vanilla html first before adding the React components. I think with certain things subject to change it might be faster to create the examples in the style guide's `public/index.html` which would help us in viewing and sharing the guide as well as tweaking any css needed before creating components.

What do you think about adding examples of the `.Header` and `.Card` in regular html for now then when there's a few of those ready we can figure out how best to make re-usable components out of them?

As a side note and this may be a personal style, but in thinking how we write CSS and the style used in `HeaderBar.scss` above, I'm thinking it could be good to try to settle on how we want to use write things. In my personal style, I find it a bit harder to parse the CSS if there is quite a bit of white space in between the property and it's value. I find my eye zig-zagging back and forth between the two a bit. I think this is great for variables and things where it's more important to view the variable name itself than actually know it's value, but in css, I'm finding it harder to read. As a secondary thought, what is your stance on ordering of properties in a class? I'm fond of [this style](http://codeguide.co/#css-declaration-order) although I know some people prefer alphabetical. What's your feeling here?

I think part of the style guide is not only a visual style guide, but a guide for writing clear css and React components. Excited to hear what you think!!


ok


I don't think that fixes it; Unless I'm missing something, there aren't overloads of ```begin/end``` which work on null-terminated strings. (Making ```substring``` a ```string_view``` does the trick, though)


A few enters and tabs will make this class a bit more readable


PULL REQUESTS INSTRUCTIONS
1-Read assignment carefully
re-write in my own verbiage if needed
look for clues for order etc.
identify main headings and subheadings
take my time

2 create a WIP issue with my name and assignment number

3-merge any open pr’s from previous assignment

4-make a small change in any new file and it will be candidate for a pull request

5-open new pull request for each repo being altered

6-copy and paste whatever section or group of check boxes I am focusing on, into the actual msg field above in the commit or

7- paste link to the pull request
note: keep pull requests on the same on going file
this doesn’t have to be the issue file, although  that would be the most efficient method


:ship:


breaks siol (just for the record)


saw approval from macintux
at https://github.com/basho/clique/commit/328ff84b169f4e373a1a47aad08cb33f5c93424e


merging basho/clique/feature/return-status-codes = 328ff84b into borshop-integration-66-feature/return-status-codes


basho/clique/feature/return-status-codes = 328ff84b merged ok, testing candidate = 59ff69dd


You've reformatted the whole file (automatic formatter?). This makes it really hard to keep track of your changes, because I have to compare the whole file line by line to be sure that you've not implemented something bad (like spy code). Our original file is not the worst one, so can you please reverse the formatting and add your changes to the original formatted file?


My understanding is that @LeonYang5114 is working on math ops.  As soon as they are ready I can integrate them into my code.   If some things (like in-place math ops) are not being worked on, then we should file an issue.


Clever


Thanks! :)


@savenkom IMO this commit is just wrong...


OK


Configurable in settings right? Should we have a way to configure this in the admin?


Will fix.


Boletad e registro


TeamCity Dirigo Edge :: Deploy Pull Request [Build 82](http://tfs2013:8081/viewLog.html?buildId=834&buildTypeId=DirigoEdge_DeployPullRequest) outcome was **SUCCESS**
Summary: Running Build time: 00:00:05


I agree, I tried to fix that mistake in a subsequent commit.


I'm gonna ask @PaulPrice for help on this one - I don't actually know how important it is to have this environment variable set, or what happens when it isn't.


I agree! Duckett's book is visually sumptuous. Your explanation on Duckett, in chapter 4 is completely on point. you describes how the a and the href function work to create the Inter-net. I tried to capture that concept as well, but you wrote it out with clarity and directness. It's also pretty neat that so many people are able to read your writing and come off with so many opinions. I'm curious as to what the consensus on Pilgrim is. As for me, Go Lineation team!


C++14 has:

    template< class T, std::size_t N > constexpr T* begin( T (&array)[N] );

http://en.cppreference.com/w/cpp/iterator/begin


Yes, as you pointed out I realized my initial mistake.


@normanhh3 this should have gone out through CI - the build seems to have passed - are you having trouble with the `.25` version of the package?

https://ci.appveyor.com/project/serilog/serilog-sinks-email/build/25

Cheers,
Nick


Ah, amusing :)


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1349 of 1349 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)


Wrong commit message. Should say,

"Added list generation file. Adapted from generate-mailing-list-hoc-latam-excluding-brazil"


Note that this does not slow down the game if damage isn't high enough (has to be over 12 points). Afaik it should slow down the game on _every_ last hit ?


Works for me and +1 here.

Just to provide full context for everyone here. I asked for this  because of a report from ragesoss on #mediawiki-visualeditor

"<ragesoss> .. several students in different courses have run into parsoid 400 errors when trying to edit pages with VE .. "

After some discussion, we figured that some of these are more recent beyond the re-rendering fixes.

"<ragesoss> subbu: from the report I have, it was happening last night, and the article is [[Pittsburgh]]."

enwiki:Pittsburgh is being edited regularly, so this couldn't be the same reason as all the other 400 errors.


You're right:

    template< class T, std::size_t N >
    constexpr T* begin( T (&array)[N] );

won't cut it.


Hi Paul,
I have made a commit addressing your review comments.
Please take a look.
thanks
Padmini

On Tue, Sep 22, 2015 at 7:34 AM, pborman <notifications@github.com> wrote:

> I have posted a few comments that should be addressed.
>
> Please send mail to borman@google.com once addressed (I have not figured
> out this whole git and github.com workflow)
>
> Thanks
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/padmini-m/goyang/commit/cdb00d223ecc83cc030f8bee60e00b013a0f1383#commitcomment-13368659>
> .
>


`rom-sql` provides migration tasks, just `require "rom/sql/rake_task" in your Rakefile


You could also remove the ".innerHTML"s


...which works for (e.g.) an *array* of ```char```, but not for a ```const char *```...


Thanks for the finding! Can you wrote a c test case? Ill fixit asap

> On 22 Sep 2015, at 22:47, condret <notifications@github.com> wrote:
>
> breaks siol (just for the record)
>
> —
> Reply to this email directly or view it on GitHub.
>


Hmm you are probably right, I haven't gotten a reply yet but I assume that I will have to change it.


I know this one might not be 100% finished, but I'd love to chat more about the use of descendant selectors as well as descendant generic tag selectors in components. [This article by Mark Dotto](http://markdotto.com/2012/03/02/stop-the-cascade/) highlights a couple issues that we could run into down the line with this practice. [This post on CSS-Tricks](https://css-tricks.com/efficiently-rendering-css/) also has some great thoughts on generic tags and rendering performance too :)

Sorry to throw so much in here, but I think these are things to think about as we're writing performant and re-usable CSS for the style guide! If I were to suggest an alternative here, the css could be:
```css
.Header { ... }
.Header--headline { ... }
/* or */
.Header--title { ... }
```
Any thoughts on this one?


The PBS include and lib files aren't always installed in `/usr/include` and `/usr/lib`, so this provides a means in the build file from HSC for finding them.  If the PBS files aren't found, the resultant mpiexec doesn't have support for launching over multiple nodes using PBS techniques, and falls back to rsh or ssh which usually fails.
Perhaps there's a more idiomatic way of doing this with eupspkg?


minor typo


Yup. Type decay will kill it (like I did my comment).

Updated example to use const std::string&.


101884090-Venting-Availability-Schedule-does-not-work-properly-in-the-AFN-model (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1280 of 1280 tests passed)

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


TeamCity pull requests :: netty [Build 244](https://garage.netty.io/teamcity/viewLog.html?buildId=1561&buildTypeId=pulls_netty&guest=1) is now running


You're quite right. No compiler to test this with sadly. A general issue with language changes maybe.


Excellent!  Was just thinking about this last night.  Thanks for adding this!


indent level over 9000!


TeamCity Dirigo Edge :: Deploy Pull Request [Build 83](http://tfs2013:8081/viewLog.html?buildId=839&buildTypeId=DirigoEdge_DeployPullRequest) outcome was **SUCCESS**
Summary: Running Build time: 00:00:08


So it sounds pretty important for PBS to be available? Should we be packaging PBS as well and just making PBS a straight dependency?


Should this be https://github.com/w3c/encrypted-media/commits/gh-pages/encrypted-media-respec.html? Although we generally do, we haven't always modified index.html for every commit. For example, when making a series of small or related changes.


**no-undef**: "it" is not defined.


**no-undef**: "expect" is not defined.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "before" is not defined.


**no-undef**: "ReactDOM" is not defined.


**no-undef**: "describe" is not defined.


**Eslint**: 'return' outside of function


**no-unused-vars**: clone is defined but never used


**no-unused-vars**: onChangeSpy is defined but never used


**no-undef**: "TestUtils" is not defined.


**no-unused-vars**: component is defined but never used


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


Is line 28 in HAL/Camera/CameraDevice.h supposed to be commented out? It sort-of crashes all my applications when it's commented and m_cam is not set...


Good point.  Do you happen to know of any packages that use that approach for me to use as an example?


Nice. I previously hacked something like this in by reopening `Hash` to toy with one of the kostya/benchmarks


Per jeromekelleher's comment this should be:

```
locationClause = "({})".format("||".join(locations))
```


Yes, I agree, it is better. I will update the PR.


yes please


This didn't actually fix the syntax highlighting in GitHub, but it looks fine in Vim either way.


test


Any reason not to enable Perl syntax highlighting?


Sort off yeah


setName pour changer le nom de joueur
cette méthode est uniquement appelé au début après la création du donjon pour que l'utilisateur choisisse le nom


True


should this be `make VERBOSE=1` per your last commit?


Sorry, David.  Not used to working with Forks...I'm basically trying to get this modification/improvement that I needed for my application (but also thought would be good for the community) into your project.  You've fixed bugs that I was unable to get the benefit of b/c of my modifications:

https://github.com/newhouse/node-google-play/commit/602740f282d3c58192d3497ce40ab18b99a5953b#diff-9ccb9b0bdd7e6cd41dfc6512731c2719

My change's tests were failing b/c the ENV exports were not happening/updated, and I was trying to figure all that out.  Any advice would be much appreciated!


Another formatting example:

```
filter =  "FILTER ({})".format(" && ".join(filters)
```


Strange thing is, in eclipse the formatting looks fine.


#4298-Recurring-error-with-Heat-Exchanger-when-using-cycling-fan-system (rraustad) - i386-Windows-7-VisualStudio-12: OK (1853 of 1881 tests passed)

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


all tests pass:
success: http://buildbot.bos1/builders/test-clique/builds/86


We don't want to be packaging PBS.  We only want to use it if it's available.  If it's not available, then there's no point in packaging it because one generally wants a cluster (and a bunch of custom configuration) to use PBS.  But the problem is it can be available but not installed in a place that mpich's configure script searches for it.


stray `#` should be removed.


Fixed in f9d1c58f2.


I agree that the relevant information added focus on "at the time of the query". I will extract only that information and put it in the first sentence.


There's probably no reason to enable SSLv3. OpenSSL which postgres uses has had support for TLS 1.0 since 1999.


And what about edits to supporting files or the .svg? I understand wanting to avoid the registry files, but we might lose other files. (The plan is to move those to their own directory, though they might still appear in the log. Should they have their own repo?)


Makes sense. Let me know what you want to use and I'm happy to make the changes for that configuration option.


And the problem is that, if you bind twice, two callbacks are called. Correct?


This is only a partial correction. You’re forgetting blind users.


Dick move.

This breaks one of our installations by not only polluting the output with pointless crap, but with a high-byte unicode character that borks puppet.


nits:
* The 'H' is capitalized.
* I don't know where this appears, but it seems something more formal might be better.


e.g. https://github.com/lsst/log4cxx/tree/master/patches or https://github.com/lsst/protobuf/tree/master/patches

It's a patch file in a patches directory. `eupspkg` applies them and reports doing so.


They should never have been putting regex's in the target.
If it worked, it would have been a bug/loophole.


`cd` or just English might be more useful than `chdir`.


You might want to warn people that this will take a very long time the first run.


Ok. Can you suggest some comments to add to this file that explains this? Otherwise confusion will continue to occur. It's really an optional dependency.


Maybe we could disable tests during the install to speed it up a bit.


don't look! its not ready yet... :stuck_out_tongue_closed_eyes:


Ah shit! I was planning an enhancement that was later deemed unnecessary.
Forgot they were there. :(
On Sep 22, 2015 3:55 PM, "John Calabrese" <notifications@github.com> wrote:

> say it aint so...
> delete me
> delete me
> :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pivotal-pez/cfmgo/commit/1efcdc5b769c658bd206b10dd1d1f2fd1da94919#commitcomment-13376331>
> .
>


Missing period.


TeamCity Dirigo Edge :: Deploy Pull Request [Build 84](http://tfs2013:8081/viewLog.html?buildId=844&buildTypeId=DirigoEdge_DeployPullRequest) outcome was **SUCCESS**
Summary: Running Build time: 00:00:11


j'ai mis le json d'une maniere brute dans un String


Per jeromekelleher: "Why the leading underscores? For things like id we follow the common Python convention of `id_` to avoid clobbering the global function."


Why `_evidence` and not `evidence`?


They may need Perl too. :-)


Should it be `del(['.tmp', 'dist']).then(function() {cb();});`?  I'm running into an issue where gulp quits after the clean task.


Should it be `CNF`?


Ah, I guess the writer doesn't support Windows so they should have some version of Perl. Maybe we should add that caveat right away.


From http://hsca.ipmu.jp/public/pipeline/installation.html :

> mpich doesn’t seem to be particularly careful in looking for the headers and libraries for PBS/torque. If these are not installed in /usr/include and /usr/lib, then you should set the following environment variables:
> * PBS_INCLUDE_DIR: to set the include directory for PBS/torque (look for the tm.h header file).
> * PBS_LIB_DIR: to set the lib directory for PBS/torque (look for the libtorque.so file)
> * PBS_DIR: to set both the include and lib directories for PBS/torque; this is equivalent to setting PBS_INCLUDE_DIR=$PBS_DIR/include and PBS_LIB_DIR=$PBS_DIR/lib


They didn't like `s` as a variable name here.


@colleenmcginnis too late!

:turtle:


I had previously asked @gsibley about this change and he said it's not necessary for it to be commented. I have fixed that in affd9207e81161d8e1ec01bbea31623dcb29ef58.


Why COMPILING_ON_WIN32 rather than BUILD_WIN32?


TeamCity pull requests :: netty [Build 244](https://garage.netty.io/teamcity/viewLog.html?buildId=1561&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5200, ignored: 21 Build time: 00:15:13


And why WIN32 instead of WIN64?


Even reading the OpenSSL documentation again, it's really not obvious which one to use where.


resolved


This protects against null `req.name`s being passed here, but the question is, should there be any coming in in the first place? Specifically, should `npm-package-arg` be forced to either throw an error or return a name?


of course ;-)


The idea is that you should override getMenuAdapter to customize it, not hack the result.


changing the content of previously uploaded files breaks past tags. please change the filename for new uploads


`$ALG = n - 2$`?


This file should be destroyed.


TeamCity Dirigo Edge :: Deploy Pull Request [Build 85](http://tfs2013:8081/viewLog.html?buildId=848&buildTypeId=DirigoEdge_DeployPullRequest) outcome was **SUCCESS**
Summary: Running Build time: 00:00:05


yes, I (am a dork who) meant dot


The last sentence doesn't read that well to me.


This is temporary -- we eventually plan to support those executables. Add a TODO?


Since you're not building them, is this needed?


surely not quite true - if we reset them, we'd break all existing cli installations!


`isinstance(annotation[source], list)`

"But really, we should always ensure that it is a list beforehand, unless it's very awkward."


This UI filtering is not good in the long term. I understand there is no easy way to filter it in the SQL, but at least should be filtered in the librarian.


Sorry; I had pushed two versions of the file within a minute of each other so I didn't bother to bump the file revision name.


so maybe something like `getMenuAdapter(View view, boolean includeOpenAction)` ?


Oh I wasn't meaning to commit AUTOTUNE_LENGTH of 8 before trying it out. We should experiment with it.


I don't think this is a good idea


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1347 of 1347 tests passed)

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


I might rewrite "You'll need to use..." as something like: "You'll need to use Perl to write your own MMDB files, but you can read the files with the officially supported [.NET, PHP, Java and Python readers](https://github.com/maxmind?utf8=%E2%9C%93&query=reader) in addition to unsupported third party MMDB readers."


why are the errors being ignored here? shouldnt they cause test failure?


But it should make more sense to auto-tune for length 8 or 9 or so, not 27...


or somehow pass that parameter inside `getMenuAdapter()` to the `MenuAdapter` constructor


The way Win32 linking works, this would need to be added to all plugins, won't it? Want to make a comment to that effect above all plugin rules?


And SHA shows no difference between EVP and non-EVP, so no speedup for MPPE key derivation. Boo...


That's the how. I'd like a why in the file as well. If I look in this file I want to be told when I should be concerned by the lack of PBS.


This was missing


TeamCity Dirigo Edge :: Deploy Pull Request [Build 86](http://tfs2013:8081/viewLog.html?buildId=852&buildTypeId=DirigoEdge_DeployPullRequest) outcome was **SUCCESS**
Summary: Running Build time: 00:00:05


tested and working


Will this file be present unconditionally, or do you need to check something like HAVE_GNULIB_CONFIG_H?


You already override the method (https://github.com/frostwire/frostwire-android/blob/5d0530e8298862362e456710fdb5d33011c4d29f/src/com/frostwire/android/gui/adapters/FileListAdapter.java#L101), let's put the logic there.


Shouldn't that spec test that `post.votes.first.user` is `post.user`? Right now, it simply tests that the `vote` subject set in the the specs bootstrapping code has been properly created…


ah and never got tagged so okay this time


I'm also having trouble with `libosxunwind`; linux systems are complaining about not being able to download it;
```
make: *** No rule to make target `libosxunwind-0.0.1/Makefile', needed by `get-osxunwind'.  Stop.
```

This happens during `make -C deps getall`.


Wait, you're not building this. Why do you need this #ifdef?


Nit: two spaces before comment.


nitpick: please fix spelling here: `requestsDefautParams` => `requestsDefaultParams`


Can we resolve this TODO?


Despite the similar names, the functionalities are quite different IMO.
cider-find-file seems to be like `find-file` and it doesn't require a connection.
cider-find-var is more like `find-tag` and requires a connection.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.974.cdea38ae01751c31983c31ed6195ce005f8d21e9](http://build.palaso.org/viewLog.html?buildId=63661&buildTypeId=bt431) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 514, ignored: 6 Build time: 00:02:11
### Failed tests
```
GlyssenTests.dll: GlyssenTests.Dialogs.VoiceActorAssignmentViewModelTests.GetMultiColumnActorDataTable_ActorAssigned_AssignedActorSortsLast: <no details avaliable>

GlyssenTests.dll: GlyssenTests.Dialogs.VoiceActorAssignmentViewModelTests.GetMultiColumnActorDataTable_NoActorsAssigned_GetsAllActorsInAlphbeticalOrder: <no details avaliable>

```


Hi @WarrenFaith and @fcurella, I did a direct port from the [PHP version](https://github.com/fzaninotto/Faker/tree/master/src/Faker/Provider/de_DE) of Faker. It might make sense to also integrate the things done in context of [this](https://github.com/fzaninotto/Faker/issues/361) issue ("Improve german first names").

Unfortunately, I do not have the time right now.


Why the extra indent?


RIP in metadata


Editing the file HTWK-App.css is not desirable, because this is an auto generated file of jQueryMobile (JQM), that can be updated by the JQM ThemeRoller to future versions of JQM. I can't assess whether your changes broke this mechanism or not, but we want to be able to update to future version of JQM.
The file defaultThemeChanges.css is the right file to apply any changes to the default theme (like overrides). Please revert your changes at the file HTWK-App.css and apply them to the file defaultThemeChanges.css.

These are two Screenshots of the App, with the design you've implemented:
![bildschirmfoto von 2015-09-22 22-59-16](https://cloud.githubusercontent.com/assets/855967/10031500/cb66ac42-617d-11e5-9feb-06c543ad8543.png)
![bildschirmfoto von 2015-09-22 22-57-51](https://cloud.githubusercontent.com/assets/855967/10031499/cb50e4e8-617d-11e5-98df-fafd6b6f861d.png)
Reachable by the link: [Preview](https://rawgit.com/vincent-t/Smartphone-App/34353df5f2e52fdd4ec947fed82ab88dde987b9c/www/main.html)

In my opinion, the text inside the header bar is too small or the header bar itself has too much height. Just look at your gmail android app to get a good example to compare to (we've used Googles Inbox app as an inspiration).
With your design, the panelmenu is styled in a color that reminds me of the blue walls of baby rooms.I don't like this color and I think it's not a good color for an app menu, nor I've seen this color inside any app I know. In Addition you can't read texts like "Hauptmenü" very good, because there's not enough contrast and the blue is clashing with the blue of the header bar.


vou corrigir esse codigo depois ;)


s/module/package


;_; rippies


The second value is a hash reference, not just a hash. Also, I don't think it needs to be capitalized or code quoted.


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

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


Works for me.


All of these would look nicer with syntax highlighting on.


@robmadden `create_vote` doesn't send an email to the owner of the post :)


Actually, the "Cam" argument is no longer an option, as the Create function signature was changed also (see DeviceRegistry line 38 new / line 42 old in 4dcb4ef8c27e136b85d253f789c3f45923ac9122)


Why the parentheses around the call? Also below.


same as above (spelling)


`map_key_type_callback` might deserve a bit more explanation as I'd assume it would be the source of a lot of confusion.


That's the style they're using.


Views which inherit "from" this one? :)


I still think something is out of sync on your box. Builds clean on my box after fetching latest from repo.. also jenkins builds are passing. Lets take this offline and figure out what's amiss.


Thanks.  Done.


Some of these commands are meant to be used on both cider-mode and cider-repl-mode, and cider-mode requires cider-repl, so I put them here.
I can add a FIXME to remind us to revisit this.


user*


Not sure I would call the Perl-based tree iterator fast. ;-)


I'll add a comment to the build script.


@reidmv merged the PR


does this help?
```
diff --git a/deps/Makefile b/deps/Makefile
index 1656375..14d968b 100644
--- a/deps/Makefile
+++ b/deps/Makefile
@@ -1770,7 +1770,7 @@ distclean-osxunwind:
                $(BUILDDIR)/libosxunwind-$(OSXUNWIND_VER)

-get-osxunwind: libosxunwind-$(OSXUNWIND_VER)/Makefile
+get-osxunwind: $(BUILDDIR)/libosxunwind-$(OSXUNWIND_VER)/Makefile
 configure-osxunwind: get-osxunwind
 compile-osxunwind: $(OSXUNWIND_OBJ_SOURCE)
 check-osxunwind: compile-osxunwind
```


There seems to be some code duplication between this and linux_main, as well as some artificial divergence. Most of the necessary parts are already #ifdef'd. Seems to me you can add #ifdef HAVE_SIGNAL around the signal handler installation part in linux_main, conditionally define it whenever WIN32 is not defined, and reuse most of the code.


Tests appear to be failing. Can you take a look?

Also, typo: should be "position".


@osvalr It is Product Lot because it is referred to lot/serial number of product


If your cluster uses PBS and there's no `pbs` in the list of available launchers (`mpiexec --help | grep "launcher to use"`) after you install mpich, then you should be concerned.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.975.551c54cfa710b5df0518604b7c0b693857ecd4eb](http://build.palaso.org/viewLog.html?buildId=63662&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 516, ignored: 6 Build time: 00:02:56


Might wanna try something like:
long startTime = System.currentTimeMillis();
long showtime = 2*1000;
while (startTime+showtime <= System.currentTimeMillis())

Might be smoother by using the time class, but you should probably ask time about that.


In fact, it would probably make more sense for cider-repl to depend on cider-mode, and not the other way around.


Let's pick a style and go with it: click_link("View...")

v.

click_link "Meals"


About time I started a repo for my scripts. ;)


This appears to be a typo, as the namespace is called `messaging`


Change to smthng more meeningfull


I up your undo with my own undo:  8777894


No. For triangles `index.itemSize === 3` makes sense, no? At least that's how I implemented it in the Blender exporter. IIRC I got it from somewhere else, but I don't quite remember where...


@abought OK! Thanks!


Changed.


@natemcmaster shouldn't this have been `apnetcidev` ?


Yes and no; if I make that change, this happens:
```
$ make -C deps get-osxunwind
make: Entering directory `/home/sabae/src/julia/deps'
/home/sabae/src/julia/deps/jldownload /home/sabae/src/julia/deps/srccache/libosxunwind-0.0.1.tar.gz https://github.com/JuliaLang/libosxunwind/archive/v0.0.1.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   331  100   331    0     0    203      0  0:00:01  0:00:01 --:--:--   203
100 84306  100 84306    0     0  38034      0  0:00:02  0:00:02 --:--:--  218k
/home/sabae/src/julia/deps/jlchecksum /home/sabae/src/julia/deps/srccache/libosxunwind-0.0.1.tar.gz
cd build && `which gtar 2>/dev/null || which tar 2>/dev/null` xfz /home/sabae/src/julia/deps/srccache/libosxunwind-0.0.1.tar.gz
/bin/sh: 1: cd: can't cd to build
make: *** [build/libosxunwind-0.0.1/Makefile] Error 2
make: Leaving directory `/home/sabae/src/julia/deps'
```


Maybe we should make it clearer that the GeoIP2 data will not necessarily be correct for the complete range if they only base it off the first in the range. One solution is to enable merging and add each IP individually, but that doesn't scale well. A better solution would be to add the data from the CSV rather than the binary.


Does this 1 work here?

Would it make more sense to call

category = Category...

category.id ??


`use-email-as-username`, adjust documentation and add check for null, and this PR should be fine.


Yes. litedev has since been deleted.


Maybe we should add `Just another WordPlate site` as default in the config?


Iterating over the search tree with Perl is extremely slow. They'd probably be better off using the CSV as input (or using the Go reader's iterator).


What do you think?


Can you update npm with this change?


provide navigation for back office UI


Uhh...


space after "for", seems that is the style of the samples


Do you mean ``cmd.run``?

I don't think we'll be changing this any time soon, as this field is modeled after the API response you get from inspecting a container.


Good catch, fixed.


Thank you @mschoebel , I can take it from here :)


What is this `target` for? Looks unused.


try running it from `deps`?


You have this line twice (once after the `cd` and here).


OK. Will fix.


This needs to be reverted


where is that stuff used in sdb?


The `-C` flag changes directory for you; and if I try it from within `deps`, I still get the same error:

```
$ make get-osxunwind
/home/sabae/src/julia/deps/jlchecksum /home/sabae/src/julia/deps/srccache/libosxunwind-0.0.1.tar.gz
cd build && `which gtar 2>/dev/null || which tar 2>/dev/null` xfz /home/sabae/src/julia/deps/srccache/libosxunwind-0.0.1.tar.gz
/bin/sh: 1: cd: can't cd to build
make: *** [build/libosxunwind-0.0.1/Makefile] Error 2
```


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (811 of 811 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-21.61%25-green.svg)


TeamCity Dirigo Edge :: Deploy Pull Request [Build 87](http://tfs2013:8081/viewLog.html?buildId=859&buildTypeId=DirigoEdge_DeployPullRequest) outcome was **SUCCESS**
Summary: Running Build time: 00:00:07


Usually when we downgrade something the number goes down ;)


do |category|


Tab this in.


Ok.


Changed all PathUtilities calls to Path.


rep 2


Test me


Looking good!


Any reason to prefer this struct with a static method instead of changing `OpenPdfInHelper` to have an optional `init?` that does the same checks?


Any problem with #including this unconditionally?


Get Hyped


Done in 18c682ff814adbd40a9a0ddfee277e7778aedcce


@ytrezq could you explain further?


`NSLocalizedString`


I'm a big fan of including only the things that you use.


LGTM


Isn't this what it's already defined to? Also below.


Why don't you just change pc.PhongMaterial to be physical by default? I realise this might change people's visuals but we want to remove non-physical at some point. So better sooner rather than later?


There's a better way to fix this. Location and user agent are device properties (they should be visible even for logged out users). Take a look at my clone of this repo for a better fix.


Can you name this top-level entry point something less awkward like `AddContextDataToReadOptimized.add()`, and supply an overload with fewer parameters that uses sensible defaults for `bigShardSize`, `outputShardSize`, `margin`, and `marginForRef`? We want this common operation of "adding overlapping data to a read" to be simple to use, if possible.


In my tests it slows down game on every last hit. Don't you have same behaviour?

The OR boolean should consider this aspect:
if(damage > 12.0f || h->health == 0) {

I have just re-tested with light attacks and it works here...


Huh?!


Oops, too tired, missed the OR. Tested it properly now, stuff works.


Wow thanks alot , going to test it now! :+1:


To PRE.


Maybe make this a public lazy getter?


As the notes in the targets file say, this targets file is to clean up files places by the SharpDX nuget in our `\nodes` folder. If we're using the NuGet packages, this kind of thing is unavoidable because NuGets will always want to drop their dependencies in the `$(OutputPath)` for the project.


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


Nice! I assume GDB can decompress this on the fly?


Not exactly a useful message anymore. Can you keep the "to" and just print the charid? Might as well keep the from since it's there now, so have it print both


Yeah, it would probably be good to enable it for stuff like Glibc and boost.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 684](https://teamcity.labkey.org/viewLog.html?buildId=273540&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:25:01
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

##### there are 23 more failed tests, see build details
```


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


If there are tasks we want one has done before another started, they are more like sequential tasks that cannot be done in parallel.


can this just be neutron.neutron_as_gateway ?  Why do we need the enabled ?
I think it's better to put default values in the defaults/main.yml rather than have hardcoded defaults in the tasks.


Might want to user a lazy getter instead of a function as mentioned above (`openInView` is a misleading function name since it simply returns the view, but doesn't actually *open* anything).


I know it is outdated, but I am working on [a new version of it](https://github.com/peerlibrary/meteor-reactive-publish). I think it is much cleaner approach to use reactivity than publish-composite or publish-with-relations, because it allows other sources of reactivity, not just queries.


@jch : I am currently gathering more info about that problem. I’ll probably tell more about it next Monday.


In the case that cashiers have to get bags from you one at a time, there is actually 2 additional seconds of lag during which no adding operations can be done, because not enough sums have been calculated yet (because handing the bags out is slow). This would give a lower bound of 260 seconds.

In the case that each cashier gives N/2 bags to the next cashier, you can just stop after each cashier has 2 bags, and start adding from there - this will save you two seconds overall.


Thanks @aldenml , [much better now](https://github.com/frostwire/frostwire-android/commit/a3af8a0c2106cc3aba354dae2ef7fe507f2acb5e)


Why you think those other packages are better?


iirc, you could just use the html.unescape that is included in python3.4
unless older python version is require.

it cover more cases,  even the `&#x` hexadecimal form.

`msg = html.unescape(msg)`

```python
>>> for escaped_html in ["<",">",""","'","&","'"]:
...     print(html.unescape(escaped_html))
...
<
>
"
'
&
'
```


To PRE.


OCD nits: `webViewControler` -> `webViewController`


Eh? You link to -lpthread. Do the MinGW pthread headers not define pthread_mutex_t?


Yes, it just loads it without complaints. I tried to be sure. (Well, my old 7.7 version didn't, but the current one did.)


Compiling with -posix should obviate the need for this change.


@Mirocow `brew audit --strict http-accounting-nginx-module`
```
==> audit problems
http-accounting-nginx-module:
 * Stable: version 0.5 is redundant with version scanned from URL
 * Stable: Use GitHub tarballs rather than zipballs (url is https://github.com/Lax/ngx_http_accounting_module/archive/v0.5.zip).
 * Description is too long. "name: desc" should be less than 80 characters.
Length is calculated as http-accounting-nginx-module + desc. (currently 168)
```


Yes, https://github.com/MinetestForFun/server-minetestforfun-hungry_games/blob/813934641ae845d8b64f756693dd285f7809a938/mods/throwing/crafts.lua#L2


Not sure if using a line graph makes sense here, since we're talking about discrete values


Example debugging output - if Pid file is not found, then the pid_for function returns false and this subsequently creates an error in the pid_alive function.

jenkins@ip-172-31-47-197:~/jobs/ci-develop-the_wall$ bluepill testworker-develop start --no-privileged
pid_file: /var/lib/jenkins/.bluepill/pids/testworker-develop.pid
pid: 7151
pid_file: /var/lib/jenkins/.bluepill/pids/the_wall-test_develop.pid
pid: 1936
pid_file: /var/lib/jenkins/.bluepill/pids/testworker-husky.pid
pid: false
/var/lib/jenkins/.rvm/gems/ruby-2.1.2/gems/bluepill-0.1.1/lib/bluepill/system.rb:23:in `kill': no implicit conversion of false into Integer (TypeError)
	from /var/lib/jenkins/.rvm/gems/ruby-2.1.2/gems/bluepill-0.1.1/lib/bluepill/system.rb:23:in `pid_alive?'
	from /var/lib/jenkins/.rvm/gems/ruby-2.1.2/gems/bluepill-0.1.1/lib/bluepill/controller.rb:90:in `block in cleanup_bluepill_directory'
	from /var/lib/jenkins/.rvm/gems/ruby-2.1.2/gems/bluepill-0.1.1/lib/bluepill/controller.rb:87:in `each'
	from /var/lib/jenkins/.rvm/gems/ruby-2.1.2/gems/bluepill-0.1.1/lib/bluepill/controller.rb:87:in `cleanup_bluepill_directory'
	from /var/lib/jenkins/.rvm/gems/ruby-2.1.2/gems/bluepill-0.1.1/lib/bluepill/controller.rb:15:in `initialize'
	from /var/lib/jenkins/.rvm/gems/ruby-2.1.2/gems/bluepill-0.1.1/bin/bluepill:77:in `new'
	from /var/lib/jenkins/.rvm/gems/ruby-2.1.2/gems/bluepill-0.1.1/bin/bluepill:77:in `<top (required)>'
	from /var/lib/jenkins/.rvm/gems/ruby-2.1.2/bin/bluepill:23:in `load'
	from /var/lib/jenkins/.rvm/gems/ruby-2.1.2/bin/bluepill:23:in `<main>'
	from /var/lib/jenkins/.rvm/gems/ruby-2.1.2/bin/ruby_executable_hooks:15:in `eval'
	from /var/lib/jenkins/.rvm/gems/ruby-2.1.2/bin/ruby_executable_hooks:15:in `<main>'


this all looks good except for the conflict in deals index controller -- can you set up the search functionality to work within the policy scope? Search should only return the same things allowed by policy scope -- published and not private.


Ah, ignore my other comments about making a lazy getter. Makes more sense in this commit :)


Wait, why does the configure check for getpwnam_r succeed on Windows? And why wouldn't we use the functionality if it exists?


OK, thanks for the info - I'll fix this tomorrow. I simply replaced `SslProtocols.Default` with what it was [document to mean](https://msdn.microsoft.com/en-us/library/system.security.authentication.sslprotocols(v=vs.110).aspx).


Since the maximum value in the picture is four maybe it is better to set x axis max to 4?


Reminder that trying to create a unit without a model will crash the game.

Using my parser the only thing you need to add are the Model and Attack particle


@Edholm uhm, leaving prints


Should this be guarded by something like HAVE_GETPWNAM instead?


You must add the mongoid gem to your Gemfile so your Rails app knows how to connect to MongoDB.


you forgot the `msg =`, someone notice it and patch it in https://github.com/JoeStaines/ch.py/commit/da3ad00315c683eb77e0f7194b9c03e833ef6d9e .
it hasn't been merge back cause nobody bother do to it yet.


Nit: should be XxxNative not XXXInterop (we decided Interop.XxxInterop was a bit redundant.


infinitely? :scream:


Not sure if deleting this is wise @Edholm maybe @lindskogen can tell?


TeamCity pull requests :: netty [Build 245](https://garage.netty.io/teamcity/viewLog.html?buildId=1562&buildTypeId=pulls_netty&guest=1) is now running


In line P6B.py:43, the / operator in Python automatically returns a float, given ints, so the *1.0 is unnecessary


Hey @benhoskings I noticed that this is breaking for Ruby 1.8.7. I am getting the following exception:

```
/usr/local/babushka/lib/babushka/core_patches/integer.rb:27:in `xsecs': uninitialized constant Float::INFINITY
```


#4298-Recurring-error-with-Heat-Exchanger-when-using-cycling-fan-system (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1855 of 1881 tests passed)

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


good to use logspace :)


@snugthejoiner I just pushed a commit, letter_opener is now working.

![popkey](https://m.popkey.co/aye3X78GZTmYDTP2OHKmFTHidGs=/original-ce497a90-4cb6-4ff5-b890-27e98747a50f.gif/6gJW.gif?c=desktop&l=curated&f=.gif)


Removed unnecessary reference.


`Node#removeSelf` is deprecated. Use `Node#remove`


Didn't tried yet, but code looks good


This kind of cut&paste seems bad for maintainability. Why not just include `ec2-data.nix`?


Is this only used in Windows-specific code?


@dpobel, @yannickroger @mhyndle could we start to track a bc doc in this repo, and start to document these changes? In our case these kind of changes affects studio, so should find a way to track it to make it more clear there are bc breaks.


you could blame me for it, all those property are crazy,
they are useless and they show up in the interactive help
making things even more confusing than it need to be.

i added the `_` to remove them from it and mostly to break bots
which use those function instead of using the property
ie: getName() instead of name

as they say, "There should be one **and preferably only one** obvious way to do it."


index is a `BufferAttribute`. `itemSize` will always be `1` as far as I'm aware.
However, yes... In this case we're avoiding the `/ 1` division that `.count` is doing.


This feels like an encapsulation violation, and it makes me wary that we're modifying browser constraints in some separate component like this. If anything else manages the web view constraints, this could quickly get hairy.

Generally, the pattern we've been using is that views are responsible for defining layout of their own subviews, and the parent view/controller would responsible for the layout of that view in its parent context. I'd recommend a similar pattern here: have `OpenPdfInHelper` create a generic view, then BVC can set layout parameters for that view in the more general browser hierarchy.


ack


Yay! Thx!


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (809 of 809 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-21.59%25-green.svg)


This is only used by getpwnam_r, since we don't have a working implementation of it I had to guard that to not get warnings about unused variable.


Then they should change the defaults ;)
Almost everybody has switched off SSLv3.

"Specifies that either Secure Sockets Layer (SSL) 3.0 or Transport Layer Security (TLS) 1.0 are acceptable for secure communications" they're not specifying TLS 1.1 and 1.2 here...


You need to add mongoid to your Gemfile so that Rails know how to connect to the MongoDB database.


Oh, looks like you addressed this here, too. Might want to just fold these first two commits together since this one seems to revert/fix a number of things from the first commit.


101682488_EvapCoolPrimaryAirSizing (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1883 of 1890 tests passed)

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


s_true[:,0], s_true[:,1], s_true[:,2] may be a easier way to locate x,y,z coords.


This file is made during win_build.sh.


Casting to unsigned long and using %lu should work better.


change lines to make it more readable.


Add a space after `##`?


Can we define a PRI-like format specifier and avoid repeating the whole line?


Empty line.


Sorry, I see that you have it below.


It builds without it on Linux.


C:\Users\IBM_ADMIN\Documents\GitHub\oltpbench>oltpbenchmark.bat -b wikipedia -c
config/sample_wiki_config.xml --create=true --load=true
Exception in thread "main" java.lang.RuntimeException: Failed to create new inst
ance of WikipediaBenchmark
        at com.oltpbenchmark.util.ClassUtil.newInstance(ClassUtil.java:182)
        at com.oltpbenchmark.util.ClassUtil.newInstance(ClassUtil.java:166)
        at com.oltpbenchmark.DBWorkload.main(DBWorkload.java:231)
Caused by: java.lang.reflect.InvocationTargetException
        at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)

        at sun.reflect.NativeConstructorAccessorImpl.newInstance(Unknown Source)

        at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(Unknown Sou
rce)
        at java.lang.reflect.Constructor.newInstance(Unknown Source)
        at com.oltpbenchmark.util.ClassUtil.newInstance(ClassUtil.java:180)
        ... 2 more
Caused by: java.lang.NullPointerException
        at java.io.File.<init>(Unknown Source)
        at com.oltpbenchmark.benchmarks.wikipedia.WikipediaBenchmark.<init>(Wiki
pediaBenchmark.java:65)
        ... 7 more


The title say "The check for password is right but the fetched user is the first with right login, not login and entity"
So what should be fixed is the function checkLoginPassEntity, not the main.inc.php ?


Seems redundant. Why doesn't HAVE_PTHREAD_H kick in?


@snopoke suggested `PillowCheckpoint` as a better name


Please change this to say

```ruby
gem 'mongoid', '~> 5.0.0'
```


Added Google Analytics


Might be cleaner to just #undef all five of ERROR, WARNING, NOTICE, INFO, and DEBUG...


Option pricing under normal inverse gaussian distributions


I think you meant indentation, not indexation.


sauced-up is a terrific verb


Additional Functions: add to your path


Consider putting usings at the beginning of the file.


TeamCity pull requests :: netty [Build 245](https://garage.netty.io/teamcity/viewLog.html?buildId=1562&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5200, ignored: 21 Build time: 00:14:28


I've checked every other call to `UIUtils.openFile(...)` and in every ocassion the `mime` parameter is correct.


O, good point!   Can you test.  I'm sort of flying blind on some of these fixes.


Can we handle this in configure.ac? See, e.g., lines 207-210 of https://searchcode.com/codesearch/view/19483329/


Theoretically the sequential time can only be four times faster than parallel time because there are four processes.


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


wow lmao


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


This is a hack. Can we mark it as such (e.g., with a TODO)?


Casting to unsigned long and using %lu should work better.


Can we define a PRI-like format specifier and avoid repeating the whole line?


Practically we use an UID for each task and then put them together by this UID after all tasks are done.


This is more or less the core of PyE, that being stated basic/general knowledge of C++ and reverse engineering is necessary to make the most out of this source. Nothing included in this source is overly complicated though, this was written when I was 17 or so.

First things first, as explained, learn some basic C++ ( syntax, semantics, operators, memory types, variables, classes, and objects ). Once you feel comfortable, compile basic programs and pick them apart with tampering tools/debuggers like Cheat Engine, Ollydbg, Immunity Debugger, etc.


You can use some functions such as logspace.


You can just make this:

```ruby
gem 'mongoid', '~> 5.0.0'
```


closes #3


I'm leery of removing this step until we have a way to ensure that appropriate release notes are updated in the appropriate PRs. Also, we need to update the version number in the release notes at some point.


Why is this not needed on non-Windows platforms? Is this a transitive include? Ditto everywhere you conditionally include sys/stat.h.


It's not fully safe to assume that STDERR implies an overall error. I reversed this commit before the final pull request. It was also interfering with existing tests.


Okay, thank you.


Yup! This a work in progress. Thank you for the reminder :)


> I'm leery of removing this step until we have a way to ensure that appropriate release notes are updated in the appropriate PRs

This change puts a point in the contributing document for a reviewer to check that there is an appropriate release note when necessary. Can you think of something else?

> Also, we need to update the version number in the release notes at some point.

This change puts "`|version|`" in the release notes document.


maybe this is why the images are blurry? Can you use them at the exact size you need?


Done, VS keeps placing them there :(


Why is this line marked as changed?


Wick thought - should we just be url decoding the text instead? (that way making all of the existing messages work)


I always wonder about these being added if it's worth... You aren't using it anywhere, so it doesn't look so.


If it's not too difficult, I'd delete all of this here rather than churning these lines since they will just get deleted anyway in later commits.


Please delete the Astyle executable from this PR


HAL_BOARD_LINUX can use "unsigned" instead of uint8_t.


Cool. It is updated and ready for you to review.


See #642


Hi,

Does that affect how we should send the HTTP authorization header using JS? We used to send "Bearer ...." before. Does it change now?


TeamCity pull requests :: netty [Build 246](https://garage.netty.io/teamcity/viewLog.html?buildId=1563&buildTypeId=pulls_netty&guest=1) is now running


I just received a text message lol


You dropped the includes


I don't understand what you mean by "lmk thrashing". This commit increases the uncompressed capacity of zRAM from 256 MB to 384 MB, The amount of physical memory typically consumed by zRAM with the default lmk minfree values remains at around 50-60 MB. This commit does not reduce the amount of free memory you have available. It just allows for more background apps to be put into compressed swap before they are killed. Reducing zRAM to 128 MB would result in a lot more background apps being killed unnecessarily.


Ugh, me again. So sorry.


Hey. Nope, it doesn't affect the "challenge" header at all. The only thing it affects is the `SignInAsync` call, where you need to use `oidc-server` instead of `Bearer` as the authentication scheme.


Curious, why are the methods of this module defined in the included hook (which is called every time the module is included)?


iOS convention uses CamelCase for constants, and we used to do that, too. Any reason we're changing? Makes me cry a little inside seeing things like `DefaultUX.cornerRadius`, followed by `NSTextAlignment.Center` like you have below.


This should be in the std lib imports section. Even though setuptools isn't in the std lib, it might as well be (regardless of what isort thinks).


Mình dùng sublime 2 portable hđh win10... Mình cũng thử trên ubuntu nữa.
Mình bấm phím tắt mà trên statusbar thấy ko hiện VI/EN. Mình thử chuyển
sang sublime 3 xem sao thì thấy vẫn thế.

Xem video bạn làm thấy bấm hình như F2 là được mà nhỉ.

Thanks bạn
On Sep 12, 2015 4:30 PM, "tuanta" <notifications@github.com> wrote:

> Chào bạn, bạn có thể mô tả rõ hơn về tình huống của bạn, phiên bản
> Sublime, HĐH,... .
>
> Hiện plugin có hỗ trợ cho Sublime Text 3 trên Linux.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yehnkay/VN_IME/commit/e5d9a78e825e95559841fb9555558a0005e59dad#commitcomment-13195773>
> .
>


I realize that "interface" may be a reserved word of some sort (probably a macro), but can we just rename the parameter to interface_ or _interface instead of removing the name? Or, if you remove the name, remove it from both parameters.


I think 29c5028 should get you what you need to make it easier to diagnose failures.


TeamCity pull requests :: netty [Build 247](https://garage.netty.io/teamcity/viewLog.html?buildId=1564&buildTypeId=pulls_netty&guest=1) is now running


you don't need to take this lock since register_timer_process() works without lock.


Shiny!


leftover?


Don't forget the "Organisation is offline" message too.

If it is URL decoded (and I'll be surprised if something is not already doling that) then the full text of the message can be put into the exception as well as the code. That will help if there are other variations on the text.

Talking of which, this presumably only works if your language on Xero is set to English, so it definitely feels like it is the wrong approach anyway.


Do we do this unconditionally? Don't you need to guard it by some Windows define?


Use a PRI macro?


How about `JSON.stringify(violations, null, 2)` to [pretty print it per MDN](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/JSON/stringify)?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 685](https://teamcity.labkey.org/viewLog.html?buildId=273561&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:25:13
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

##### there are 23 more failed tests, see build details
```


Casting to unsigned long and using %lu should work better.


:goat:


Can we define a PRI-like format specifier and avoid repeating the whole line?


Seems details.removevote was added 2011:
 https://github.com/Flyspray/flyspray/commit/0629e381702e89b84287dbc65692a02107f2dd49#diff-631a55b00ae70533162f910f132776c5

But removevote-button wasn't ported 2012(?) from now nonexistend templates/ to the CleanFS/ theme.


Yeah I was going to do that, but then according to the docs the others don't get returned in the same format... well... that's if their docs are correct... bit hard to test without finding an org that is actually unavailable... or Xero turning the API off for maintenance!

http://developer.xero.com/documentation/getting-started/http-response-codes/#title5


Looks great!


@jschall does this makes sense in terms of where that code should live?


I'll work on passing the track details to the popup from the search results


Reverted but removed the default values since the defaults were only used by Default above.


@maya it adds to it, but they were saved at 230px and a little blurry to begin with. @mollieru is saving out some higher res images. Will update soon!