I don't think these have a use right now and we should remove them


@Naios even worse if a reference to a local variable std::list<uint8> is passed, that's free'd when going out of scope


These were the only problems with this tag among all of the EADs.  When I had checked EADiva, it turns out that the tag is the pluralized "dimensions" rather than the singular.  I believe I have solved all instances of it.


done!

On 08/24/2015 12:33 PM, Xiaodong Sun wrote:
>
> Can you please make the |ListGroupItem| class implement the |com.google.gwt.user.client.ui.HasHTML| and |com.google.gwt.user.client.ui.HasText| interfaces ?
>
> |...
> import com.google.gwt.user.client.ui.HasHTML;
> import com.google.gwt.user.client.ui.HasText;
>
> public class ListGroupItem extends ComplexWidget implements HasType<ListGroupItemType>, HasText, HasHTML {
> ...
> |
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bauna/gwtbootstrap3/commit/c98ab9ab3e150d3685b22a96726865bbe8b95bc1#commitcomment-12860238>.
>


This comment doesn't seem quite right since we set it just below.


파일 맨마지막의 줄바꿈 문자를 또 잊었네요.


What bug was this fixing? You're using an id now, which is going to be a problem if you want to include more than one of these on a page.


This might be added to every doc page. Thoughts?


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 401](https://teamcity.labkey.org/viewLog.html?buildId=267998&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:14
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


I think you should break this out into a few different tests. Think about shipping forms youve filled out; they dont just say "invalid address" they say "invalid city/state/zip/etc."


Sorry I meant commit 14dc9f4da8eba04a417b7e7370f0ebf694e17033


We don't do anything with priority here, so not sure what the comment is about.


Missing comma here caused syntax error.


Anything I can help with?


These files need *some* sort of copyright, and maybe an attribution if it's inspired by code elsewhere.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1274 of 1274 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)


The TABs look OK in the actual code editor. Will fix the other comments.


So, I was able to overcome this with something like that:
``` python
step = 0
maxusers = 10000
while step < maxusers:
u = app.api.filter(
    User,
    email=email,
    kwargs={'marker':step, 'maxitems':1000})
    if u: break
    step+=1000
```

Is there any plans to release version 1.1.6 on PyPi?


It is ok that it is a prototype that is not meant to be merged. It is what the WIP tag is for - there are two WIP PRs in CoreCLR repo right.

We need to make sure that *all* people who are watching CoreCLR repo are given the heads up on what is happening. In this case, the best way to do that is by submitting WIP pull request.


@EclipseGc , is this the best way to inject in plugins from a library without defining them with annotations? I wasn't able to find anyone else doing this kind of thing.


@danielo515: http://conceptf1.blogspot.com/2014/01/javascript-best-practices-loop-optimization.html


You need an ID here for the removal code to find. You can do something like this:

```
<%= content_tag :li, id: "bookmark-#{bookmark.id}", class: "bookmark" do %>
...
```

You will remove the li and replace it with that tag, and then replace the closing li with an `<% end %>`


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


jump_statement.cpp의 변경 사항은 세번째 커밋에 속해야 할 것 같네요.


@sgade do you have any logs? 'Cause I tested this with our slack adapter and it did work.


YEAAAH, it was bugging me for a while but I couldn’t find the error. Then I realised it was not line 108 that went wrong but row 108… LOL (now fixed)


Correct error Call to undefined method Smile_ElasticSearch_Model_Resource_Catalog_Product_Collection::addFieldsToFilter() in app/code/core/Mage/CatalogSearch/Model/Advanced.php on line 212


Ah yeah!


You dont really have much testing around logging in other than merging carts. What about something like this?
```
Scenario: logging in saves current items
    Given the cart has 2 items in it
    And I login
    And I logout/clear browser cookies
    And I login
    Then the cart has 2 items in it
```


why do this change? This caused problem when i do`SomeModel.create data` use ajax, then it send a PUT request after sent a POST request, because the id in POST request's response doesn't equal to the auto created cid. i think ChangeID method should be put in Ajax.disable


Minor updates to compound names to handle embedded special characters (like '+').  The compounds.master.mods file was updated a while ago but I hadn't pushed the corresponding update to compounds.master.tsv.


Could’ve sworn I tried this & it didn't work (when trying to deal with my attempts to build 1.0.1 on Mac), but I’ll give it another shot again this week.


I spent big parts of my DjangoCon Europe 2014 adding Python 3 and RTD to this project.. and some media files backup stuff... lots of good learning :)


@palamago @Cidro movi estos defaults acá en vez de tenerlos en la inicializacion de la vista. Quizá no sea la mejor idea...


1: M111 unrecognized component name: base


Thank you.No kidding.…… :(
I hope somebody can pick up this,although it is not realistic...
Chinese people's servility let me very helpless...I'm really sad.


Your SOS has been heard but we cannot do anything about it. You can buy yourself a Brésilienne when you're done.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1277 of 1277 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)


@palamago esto causaba el bug en la seleccion del tipo de chart, no se por que mierda, pero hay que usar currentTarget siempre, el `target` es el originador del evento que el currentTarget.


Remove the line if not necessary, commenting them out just makes the code bulkier.


@Pilchie, it was done this way so that other test runners (such as TestDriven.NET) would also have the specified configuration by default.


:clap:


I assume the import changes here are leftovers from before a refactor?


skipBlocks is a crude way to test purity. It's better to use the existing test in `tpd`, like this:

    def skipBlocks(t: Tree): Tree = t match {
      case Block(_, t1) => skipBlocks(t1)
      case  => t
    }
    skipBlocks(tree.rhs) match {
      case lit: Literal if sym.is(Final) && isPureExpr(tree.rhs) => lit
      case =>
         var rhs = tree.rhs.changeOwnerAfter(sym, field, thisTransform)
         ...
    }


I think I just got caught by my own 'circular' logic here -- this tries to install `rcppredis` from the ropensci drat repo, but the ropensci drat repo is using this Docker image to build stuff in the first place!  Once it deploys from Circle I should be able to add rcppredis back in (without `installGithub`, which I'm trying to phase out)


Add 'be'


category list 기능 추가


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


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


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


This will need to go down below 'speciesprefs' int(1) NOT NULL,


Oh, I just see `api.user_search()` function that do the thing with no need of all this cycles and 1000 records limit :)


when the socket is closed or an error happened the callback return false


The socks need to be added onto the end of this list.


Same here, end of the list.


This one might worth become a private variable. What do you think @yankodimitrov.


This doesn't seem to be on any branch


This is technically fine here, because it's just sanitizing it, but it should really be in order, therefore, at the bottom.


We cannot simply serialize that data, instead of this we should raise an Exception as it's a bad practice to store arrays in database.


Needs to be at the bottom of the list.


yep


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


I'm not sure why this is all relative pathing now, the file should really be completely non-relative, but haven't gotten to that yet.


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


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


adding a nullptr check should be mandatory in these cases xd


Thanks !


That could definitely be the case.

I am not familiar with the build process of these JS files, but I can adapt this code accordingly.

Is this file the source of the minified version?


pushed


kdm should be an exception too I guess.


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


Hey, I'm currently working on a full installer for your script(s).
I just wanted to ask, which one I should use.
"loader.user.js" or "bot.user.js" + "launcher.user.js"?


Compile will be faster... but Link is needed for `ng-transclude`d items.
I have update the [layout.js](https://github.com/angular/material/blob/master/src/core/services/layout/layout.js#L126-L130) to:

*  transform lyout attributes selectors to class(es) during both compile and post link phases
*  remove the deprecated attribute selector **after** transformation.


ping @moizjv @jlipps


General README file


test


ping @jlipps @moizjv


With that change, apt::setting is called with "undef" priority param, so it seems that the default value is used ('50'). Due to that, after the upgrade of the module from 1.8.0 to 2.1.1, all my existing pref files have been kept but new files with 50xxx have been created.


:) +1


Clear bug in new formatter.


yes since it's a vector, but if the internals of this structure changes (for use another container for instance)  everything will have to change, with this macro only this piece of code will need to change.


@ccrs: as I explained to you yesterday, this is not related to null pointers. read the comments above


Kewl. I am all for doing less `install_github` as you know :)


Having checked the string startswith what you want, why not just slice off that many characters from its front?


Section II assignments for FizzBuzz and FuzzBizzNezz.


touch README.md
git init
git add README.md
git commit -m "comentario"
git remote add origin https://github.com/josekeru/Repositorio.git
git push -u origin master


Yep, that makes sense.


You're using SF2D_PLACE_RAM when creating the texture, but the addresses must be in the linear heap.


Ignores "do not wrap" setting. This was formatted in a perfectly reasonable way and should not have been reformatted given the "do not wrap" setting.


Ignores "do not wrap" setting. This was formatted in a perfectly reasonable way and should not have been reformatted given the "do not wrap" setting. The new format is worse than the original.


Hi, I think we are going to need to add a migration for this :)


That is something i want your opnions. Initially it was made refcount and the unref returned the handle (and if refcount == 0, NULL). But k-s told me to make unref a void function, so i had to keep an external reference count in the the module. So, or i remove reference count here or change the unref signature. Which is better ?


Because of our API layers, we end up "demuxing" the results of select call -- calling select for each one -- to pass them to MHD. However, `MHD_run_from_select` would be able to handle all FDs at once here if we had the information available.

Might be worth adding a note in the code about this issue. If we ever reach the "let's optimize the webserver!", we will probably have to deal with this and see if we can do better than issueing one call per connection.


Bug.


This closes #40


Bug. Ignores "preserve white space between code and line comments"


Ignores "do not wrap" setting. This was formatted in a perfectly reasonable way and should not have been reformatted given the "do not wrap" setting. Reformatted code is worse.


The "proper" way would be to set OSMIUM_INCLUDE_DIR before calling find_package(Osmium), this would ensure that the proper directory is used.


Do we need this alert?


Would make sense to have a warning or debug message here if the `MHD_run_from_select` fail?


@mccalluc I think this condition on lines 109-111 will throw an error when  `params[:f][field] == nil`. I think you may need to change the second value in the disjunction to `params[:f] and params[:f][field].any? ...`.... see what I'm getting at?


I think OnInitializing gets called on existing ContentItems as well (refer ```DefaultContentManager.Get(int id, VersionOptions options, QueryHints hints)```). Wouldn't reset site setting values every time it gets loaded?


TeamCity ReOpenLDAP :: devel-2.5 [Build 101](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=35&buildTypeId=ReOpenLDAP_Devel25&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:41:10


Yes, but that changed, for CMake 3.3.1 this says:

 ``FOUND_VAR <result-var>``
   Obselete.  Specifies either ``<PackageName>_FOUND`` or
   ``<PACKAGENAME>_FOUND`` as the result variable.  This exists only
   for compatibility with older versions of CMake and is now ignored.
   Result variables of both names are always set for compatibility.

Hm, found a typo in it ;)


@ignasigarcia yes this is the source file.

Thanks for the PR by the way


I do not know if there is software out there already using it, from our side this is not a problem. I'm ok with remove it, but it should be very clear since the code is public and someone can be already using it.


removed unnecessary cast


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


I added the function mainly as a convenience, because I found myself constructing a `FileSequence` object often enough, and added the `ls` argument because I seemed to be doing that fairly often as well. I left it defaulting off because the `ls` adds a performance factor, so it seemed best to opt in to that. I'm not too fused about switching it to default true though if you prefer.


You can implement glStencilMask now, the GPU_SetStencilFunc function signature in ctrulib has been updated to match what the variables actually do in hardware


This will need to be able to be changed for the embedded player


Actually, these should probably be relative links.


The problem:

There are many Selenium-based projects such as Appium (java_client),
Selendroid, IOSDriver and so on. They are used not only
for the web/browser testing and they have their own WebElement
implementations. Also there is probably need to re-implement all changed
interfaces for some end user purposes.

Current design doesn't allow user to use their own WebElement subclasses
without additional casting. It is annoying and makes code dirty. Any
trying to avoid this causes problems.

This change is backward compatible with existing projects and
will allow to create factories or user's customized Webdrivers and
WebElements that return desired WebElement subclasses.


I'm not sure this change belongs in this commit?


See https://github.com/vert-x3/vertx-mysql-postgresql-client/commit/7d063fa79b77cf80809aad3f7cfe2e3b07838644


I think that was just copied from `FileSystemTest` and I forgot to remove it.


Ignores "do not wrap" setting. This was formatted in a perfectly reasonable way and should not have been reformatted given the "do not wrap" setting. We use a "soft" line length of 120 and a "hard" line length of 130. So, if it is over 120 it probably should be on a new line, but over 120 it must be.


first commit


Can I cast a vote for wrapping at each end of the file?


Ignores "do not wrap" setting. This was formatted in a perfectly reasonable way and should not have been reformatted given the "do not wrap" setting.


test 1


Look good for me. Thanks for you pull request


@justinslack What do you think of this?


This is awesome, I am very much looking forward to this getting pushed over to --pre. Will be very useful.


Shouldn't this work?

    [downloaded directly](archive/master.zip)


Perhaps:
    [Releases](releases) in the project page.


Yeah... or maybe add a configuration file? I'll take a patch for kdm for now if you like.


The loader is for single time installing. If you use that, you seamlessly get every update.

If you use "bot.user.js" + "launcher.user.js" you manually need to get the updates when there is a new version.


This condition was removed here: https://github.com/official-stockfish/Stockfish/commit/27efc5ac996ffc679395317c8bbb16aca996296c

Remember that the 'bestMove' condition entails bestValue >= beta at NonPV nodes.


Additional blank lines have been added between constant and associated static initializer.


Try:

    [Google Test Developer's Guide](docs/DevGuide.md)


Ignores "do not wrap" setting. This was formatted in a perfectly reasonable way and should not have been reformatted given the "do not wrap" setting. Reformatted code is far worse.


actually, `CRAY` is quite common in HPC, too if you are adding those from the list :)


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1277 of 1277 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)


Ok...thanks for pointing that out.

I will delete  the test.


I'm confused why it's OK to return here (but might be just because I'm not familiar with MHD). Will all the requests end up calling multiple times the handler callback?

It seems we don't do anything for the first call, but do for all the other calls of the same request. I would expect either handling things either on the first call or the last.


@VoyagerOne The value that's compared against 512 is v, not v * 64.


yep you are right.  just removing L#77 solves all the problems i was seeing.


Hello.
Found link to this project at your [article](http://habrahabr.ru/company/redmadrobot/blog/265117/).

You do not need to add JavaVersion.VERSION_1_7 if you have both JDK on machine: JDK8 for running Android Studio and JDK7 for building by gradle.

If you allow I ask some questions not about this commit, but about article.
NDK is very common used when you already have C++ program, so it need to compile for Android too.
Did you try compile code that common for several platform?
n that case you have common storage that do not located in src sub folder of Android project. So you adding some path into gradle script, but there is some [surprise](https://android-review.googlesource.com/#/c/160999/) so it impossible at current release (1.3) of Android Studio. I try my self at snapshot of 1.4 to compile such of project but it still fail.


Thanks @jennyd, fixing that sorted the commit ordering out too :+1:


@fervisa Are line 12 and line 18 correct?
According to the documentation [here](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/Reduce), the syntax for reduce is
`arr.reduce(callback[, initialValue])`
Shouldn't it be `, 0)`?
Thanks for the tutorial btw, pretty easy to follow :+1:


good find. removed in c72f0b4


Achei bacana!


Bug. Ignores "enable block comment formatting" being false.


acho que fica bom!


Ignores "do not wrap" setting. This was formatted in a perfectly reasonable way and should not have been reformatted given the "do not wrap" setting.


I'd say “Jouez à vos jeux Dreamcast préférés sur votre PC” seems more natural to me.


Ignores "do not wrap" setting. This was formatted in a perfectly reasonable way and should not have been reformatted given the "do not wrap" setting. Reformatted code is worse.


@bpbond, any objection to removing the next couple lines of commented code?  In particular, line 303 leads a reader to wonder whether the merge really is supposed to be gone, or whether it got accidentally swept up in removing the timing output.


For request type we don't handle yet (GET, PUT, anything that's not POST/HEAD), I think it's better to: add a warning / debug message here, and make a response with HTTP code 501 or 404 (or something that make sense in this scenario).


But what if the list consists of the only element? Then I change this->next.


Does input_mask act like the mask specified in [glStencilFunc](https://www.opengl.org/sdk/docs/man/docbook4/xhtml/glStencilFunc.xml)?


We should probably remove this and the other console.log.


:+1: Nitpick: I'd format it like this instead:

```clojure
(def NAMES
  {200 "OK"
   204 "No Content"
   302 "Found"
   400 "Bad Request"
   404 "Not Found"
   405 "Method Not Allowed"
   501 "Not Implemented"})
```

That way the lines stop leaning over to the right so much, and also pulls the closing parens/braces up as is customary in Clojure.


@DU-jdto
Coffee hasn't kicked in yet... :)


This would be nice to include in the Snowplow CLI application...


Wow, this was pretty broken. I guess we didn't run with any initmods disabled in any of the tests!


Google Translate in combination with my French lessons I had in school = French lvl: It's over 9000!


Appveyor isn't automatically building. Any ideas on how to make that happen?


Hello @sukima ,
acording to the performance tests linked on the page you have posted,this method is one of the slowests ones.


Probably not, I just did that because all the other filters did. Shall I remove it?


yep


Why this header?


Unfortunately, the source you have linked to no longer exists. Can you give another source for your towgs84 parameters? Unfortunately, the GDAL that comes with OSGEO4W and Ubuntu still uses the 3-parms-version by NIMA, which is surely less accurate. But you might end up with some nasty offsets if QGIS runs GDAL commands in the background.
I checked the EPSG database, which is the ultimative source for GDAL parameters. They offer different values for the Czech Republic and Slovakia, but none of them meet yours.


this line is not necessary.


should have a case to read more than what bl is able to offer, and expect an `end_of_buffer` exception.


`EXPECT_EQ` if you please.


ditto


@flavorjones Appveyor will build based on which branches do you allow in `appveyor.yml`

Right now the failures you see are caused by missing `build: off`.

See here:
https://github.com/luislavena/gem-compiler/blob/master/appveyor.yml


Could you comment on iterator class a bit more detailed? Remind me about this tomorrow.


It has been used before but is now obsolete, should I change it and rebase or apply a patch with all changes later?


Unrelated to the patch but looking at this line I think that we should at some point rename:

- `sol_http_param` to `sol_http_params` (or depending on how it is used, simply inline it's definition inside the request / response structures)
- `sol_http_param_value` to `sol_http_param`.


does react-redux state anywhere that it only supports React 13? I thought context was available in 12 and earlier too.

I would also additional minor versions


Too bad Savvy isn't doing anything special (something that would require Savvy's state) here. That is, there is not escaping, filtering or anything else that involves Savvy. This is just an utility implementation of a missing PHP core function `implode_callback`. Not very 'S' for "SOLID".

I'll add it back to maintain backwards compatibility and use your example as a test.


I don't remember exactly, but I guess I could try that instead if I also add `setIncludeSequences()/getIncludeSequences()`. I don't *think* I'm using `setPath/getPath` anywhere other than 31b4f462b926af7cc367d924e2a38371db2191f2, and I think that was why I needed it.


Shouldn't really cancel the pending in those points ? Exaclty as it's done with timeout


@shanethehat I cannot accept this commit because package "roave/security-advisories" is not a released version. The minimum-stability requirement for Magento 2 is alpha now. I will accept your last commit. Thanks for your contribution.


good catch, i'd suggest you to extract this fix into a separated pull request.


Vs 2013 wraps, and so does the implementation I pushed last night.


I think "After React 0.14" should be "Beginning with React 0.14"

The first is technically ambiguous (even though most people would figure it out)


Currently this is correct, frontend and backend credentials are the same . In future version it will be possible to have different credentials for frontend and backend to enhance security.


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1798 of 1798 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 is completly wrong, i thought i have changed. Should be get here and somehting invalid otherwise. I'll fix it


That sounds OK. I'm not too concerned about the BC break, and this could be a supplied filter or implementation recipe. I didn't spend much time looking, but I believe there was another project in PEAR's old svn.php.net repo that used it as well... but my memory isn't so good any more.


Looks like I misspoke. The core PHP implementation is `array_reduce`.


Why not Log.e?


Optional suggestion:
var v = vertices;
this.bottomFace = new Face(v[0], v[1] .. )

or

function populateFacesFromVertices(v)
{
 // this block becomes smaller.
}


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I like this idea.


Remove comment


I believe that incorrectly evaluates to https://github.com/google/googletest/blob/master/archive/master.zip


Good. Add a assert.
If there is no assert, make a assert function.


@trusktr Accepted!

Ok, Now I see what you are doing with infamous/infamous. I like the layout. I have seen people compiling to the dist folder I mention above prior to deploying to npmjs. Maybe your solution is better, because you would have an npm package for each (browserify, webpack) and the default would be the ES6 solution (JSPM).

May be easier to remove the glslify from the equation than ask everyone to fix their package managers to accommodate a glslify transform (#5). I will get this up so everyone can see the option.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1277 of 1277 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)


It will be better to use `"".equals(token)` instead of null and empty check as `"".equals(token)` is null safe


Remove


I have tried to replicate the style used in google apps like YouTube:
![screenshot_2015-08-04-09-07-37](https://cloud.githubusercontent.com/assets/1138694/9446114/bad63c24-4a87-11e5-9bf0-8f032ffbc61d.png)

Therefore I think this needs to be an option rather than the default.


Bracket new line


Hey man, whatever you did in one of the more recent updates made me not able to run the mod AT ALL, whatever it be my dev version or my compiled version...
This is all the Xash console spits out at me:

Video: Intel(R) HD Graphics 4000
UI_SetColors: colors.lst not found
UI_LoadBmpButtons: btns_main.bmp not found
Warning: FS_LoadImage: couldn't load "gfx/shell/splash"
Warning: FS_LoadImage: couldn't load "gfx/shell/menu_newgame_header"
Warning: FS_LoadImage: couldn't load "gfx/shell/menu_newgame_hl"
Audio: DirectSound
Warning: couldn't load scripts/sentences.txt
Host_InitError: can't initialize client.dll


probably we should sync `debian/control` and `ceph.spec.in` accordingly.


Is `;` it necessary? Shouldn't it catch `InterruptedException` instead of general Exception?


Updated for the developer's guide, not sure myself if it's possible in a simple way for the archive and release links to be relative.


Same as the above.


We should call the `ObjVertex`


Share a sol_buffer between the loop iterations? We did something similar when handling port names in the flow code. I suspect (but haven't deeply checked) that HTTP doesn't impose limitation on those.

Alternatively, check that all the headers will fit in this and give a proper error message.


LGTM


@vumrao any change to the radosgw-admin usage needs to be accompanied with the same change going into `ceph/src/test/cli/radosgw-admin/help.t`


Sorry, are you saying that Savvy _is_ supposed to be doing some sort of filtering or escaping? Because that is missing from the current (pre-PR) implementation.


Yeah, I think that's the easiest solution for now, although It'd be great to have a module system for writing shaders on the application side. I suppose we can specifically add glslify to our app dependencies if we wish. It's such a great thing.


Add a warning or debug message here for the unhandled cases (that we may want to handle in the future)?

The default case probably include invalid stuff, that warrant a warning message saying we ignored it.


Optional suggestion:

Have a float compare function instead of rounding it off before comparing.
= Float.equals(float1, float2, leastCount)
{
    return Math.abs(float1-float2) < leastCount;
}


Why? Isn't this maybe an issue with your obj models?


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.463.1c6f9f1f74f9e82e0fc75f27c9120e361fe7290d](http://build.palaso.org/viewLog.html?buildId=61811&buildTypeId=bt223) outcome was **FAILURE**
Summary: Compilation error: PalasoUIWindowsForms\PalasoUIWindowsForms.csproj (new) Build time: 00:03:27


ah, forget it, I see you did it!


what I wrote is null safe too, and is the better way to write it


@vumrao the check here should be if --max-buckets was specified. Wherever we use RGW_DEFAULT_MAX_BUCKETS, replace it with the cct->_conf->rgw_user_max_buckets.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.438.1c6f9f1f74f9e82e0fc75f27c9120e361fe7290d](http://build.palaso.org/viewLog.html?buildId=61812&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests failed: 262 (260 new), passed: 2985, ignored: 235 Build time: 00:10:15
### Failed tests
```
Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_CreateItemAfterFirstCall_EntryIsReturnedAndSortedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_DeleteAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_DeleteAllItemsAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_DeleteByIdAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_ModifyAndSaveAfterFirstCall_EntryIsModifiedAndSortedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByDefinition_ModifyAndSaveMultipleAfterFirstCall_EntriesModifiedAndSortedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByHeadWord_CreateItemAfterFirstCall_EntryIsReturnedAndSortedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByHeadWord_DeleteAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByHeadWord_DeleteAllItemsAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByHeadWord_DeleteByIdAfterFirstCall_EntryIsDeletedInResultSet: <no details avaliable>

Palaso.DictionaryServices.Tests.dll: Palaso.DictionaryServices.Tests.LiftLexEntryRepositoryCachingTests.GetAllEntriesSortedByHeadWord_ModifyAndSaveAfterFirstCall_EntryIsModifiedAndSortedInResultSet: <no details avaliable>

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


no need for this block


Yeah good call. I'll change it.


This breaks our app since it only works on the client... can you please add a check for `typeof navigator !=='undefined'`? Cheers


Great, thanks for the contribution!


either compare it here to ->_conf->rgw_max_user_max_buckets, or do it unconditionally


If you open the url http://docs.sequelizejs.com/en/latest/docs/models-definition/ you can see that:

```javascript
// Comments can be specified for each field for MySQL and PG
hasComment: { type: Sequelize.INTEGER, comment: "I'm a comment!" },
```

But this feature was not implemented in postgres dialect. I added this.


@luislavena I'm past the build:off option.

As far as branches, I'm not specifying in the yml file, but my project settings are appropriate:

![screenshot from 2015-08-24 12 52 19](https://cloud.githubusercontent.com/assets/8207/9446333/13215248-4a5f-11e5-9643-04d2d9411c12.png)


Yes indeed, at the same time as it was pushed to Github.


space before bracket


Hello. No, i just needed to change color of buttons for project what i'm currently working on, and decided what adding this option is a better way than overriding color through css.
Anyway, i seen what you recently did a massive refactoring, so my branch is unusable for you now.


Should be `\fIphy\fR 0 ...`


On Mon, 24 Aug 2015, Kefu Chai wrote:
> probably we should sync debian/control and ceph.spec.in accordingly.

They already install boost-devel or libboost-dev... configure and the
automatic shlibs stuff handle the rest, I think?


We need to figure out how the artwork directory is installed, probably package data? And how it is referenced from cellprofiler/icons/__init__.py. To resolve #1508, I have to assume that the unfrozen CellProfiler finds "artwork" at "../../cellprofiler/icons" which is a bit of a hack.


@afred: Unless I'm confused, that's already covered on the line above. Originally:
```ruby
params[:f] and params[:f][field] and params[:f][field].include?(value)
```

... but looking at it now, I need to remind myself why `params[:f][field].include?(value)` is still necessary.


I don't know if escaping was part of it, necessary, or expected. It appears to have been more of a helper method to reduce the amount of rendering logic placed inside views.

Here's the original contribution, from @cellog
http://svn.php.net/viewvc?view=revision&revision=289521
http://svn.php.net/viewvc/pear2/Templates_Savant/trunk/src/Templates/Savant/Main.php?r1=289521&r2=289520&pathrev=289521


Still too high, apparently. http://hydra.nixos.org/build/24983009/nixlog/1/tail-reload


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.439.fb4038ac5fe88a34b9b382e890f5f9a7d0503301](http://build.palaso.org/viewLog.html?buildId=61813&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3336, ignored: 134 Build time: 00:09:03


what is scass


Appreciate the effort, but I'm working on the parser already. This change in the code will cause parsing of `a[a[i]]` to fail.


Not at all, my mistake for leaving that crap in.


I don't think this is right.  We really need to figure out how to make this class either work, or do nothing in tests, because this will make it not actually do what we want in product code.

@jasonmalinowski


True, that's why we had a reference to a pointer. We probably should add an MooseAsserts when using displaced problems in case the order of anything gets switched around. That's not necessary right now since it's not there now. Just something to do later.


According to the spec the max message size is an unsigned 4 byte, which is larger than an int.


merged in master


It doesn't seem to do this (or at least not via the browser UI). I guess that's because the browser doesn't re-use the Path object, but reconstructs it and all its siblings from the parent path. Is there another way to hang on to a Path object that is linked to a plug (once I remove the getPath method that you don't like)?


Actually its a tricky problem.
if `export FOO=42bar`
then `%%FOO%%` is replaced to `"42bar"`
but
if `export FOO=42`
then `%%FOO%%` is replaced by `42` (without quotation) which is not readable by
`.to_json` was needed to bring back quotation.
now we fix this problem, but open and old one.


Are there any interesting files that should be reviewed here, or is this just mechanical?


I was using this model as a reference http://paulbourke.net/dataformats/obj/minobj.html


Yes it's clear, I do understand what is happening in the current code and how the patch fixes. I'm thinking about how this code will grow over time. Suppose we have a "caio" protocol client that might need clean up and shutdown:

        r = sol_network_init()
        if (r < 0) goto error_init

        r = sol_http_client_init()
        if (r < 0) goto error_http

        r = sol_caio_client_init()
        if (r < 0) goto error_caio

         return 0

    error_caio:
        sol_http_shutdown()
    error_http:
        sol_network_shutdown()
    error_init:
        return r

Imagine we can enable/disable both caio and http at compile time. I think trying to write this shutdown code with ifdefs will become complicated. By assuming these functions always exist we simplify initialization without any runtime/size costs -- compiler will be able to inline the empty functions.


We should call this `ObjMeshPart` or `ObjModelPart` just to give a better context for `Obj`.


Okay, but this was enabled??:

CONFIG_F2FS_FS_SECURITY


I think that if you want this to work with both Django 1.6 and Django 1.7+ you need to add `south_migrations` to setup.py:

```
@@ -110,6 +110,7 @@ def run(self):
         'database_files.management',
         'database_files.management.commands',
         'database_files.migrations',
+        'database_files.south_migrations',
     ],
     #https://pypi.python.org/pypi?%3Aaction=list_classifiers
     classifiers=[
```


Bracket new line


heh, I too, am writing my own C++11 PLY reader/writer :)


scass is sass but in rails


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

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


Sure. I meant, use the vector macro to implement the params macro. If you change from vector to something else, you change your macro.


What about all of the `FEProblem::add*` methods? They all still all receive copies of the InputParameter object.


Great option


I generally recommend minimizing diff lines in a PR to make it easier to review, merge, backport, etc.


Is it public? I want it to be functionally the same as the original because
there are a lot of places that still use the original parser.
On Aug 24, 2015 10:03 AM, "Dimitri Diakopoulos" <notifications@github.com>
wrote:

> heh, I too, am writing my own C++11 PLY reader/writer :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bclucas/alloy/commit/0287fd1f93025605d392b626a10970c8a05ad1d9#commitcomment-12862072>
> .
>


works like a charm, thanks


Yes, copies are allowed up until Factory::create is called, which the add
methods do.

On Mon, Aug 24, 2015 at 11:04 AM, Cody Permann <notifications@github.com>
wrote:

> What about all of the FEProblem::add* methods? They all still all receive
> copies of the InputParameter object.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/idaholab/moose/commit/e789203f7358299163d06b80b97d38e60220682f#commitcomment-12862089>
> .
>


Terminal=false >> true )


I guess we'd be restricted to `ParameterisedHolders` if we did it dynamically because we need to register such things on a node type. That might be ok for my current use case though.


net.LookupTXT


I would be careful about a strict less-than check here. The runtime might create goroutines as part of the garbage collection, which is probably why you saw failures on 1.5 with the new gc.  I would give a small grace buffer. If update is leaking it will be obvious even if you use start < end * 1.1.

Also note running tests in parallel creates more goroutines and 1.5 uses GOMAXPROCS=NumCPU instead of 1, which will also affect your goroutine count.


This is all fine but I think we should somehow note that this is an addition to Solar's rules. Maybe just another line of comment `# Note: the following 6 lines are Jumbo-only`


Invalid "orphan commit" ! I can't delete it !


Let's add references to the tracker (Fixes: #NNNN) and the BZ in this commit message


i see your point, i've additional checks in the for, so i can't use SOL_VECTOR_FOREACH_IDX


in the next few weeks... crunch time happened but I got about 80% of the
way there. It's compatible with the the Turk exporter/importer on all the
files I've tested so far. I'm mostly focused on what a more sane API looks
like

On Mon, Aug 24, 2015 at 10:05 AM, Blake Lucas <notifications@github.com>
wrote:

> Is it public? I want it to be functionally the same as the original because
> there are a lot of places that still use the original parser.
> On Aug 24, 2015 10:03 AM, "Dimitri Diakopoulos" <notifications@github.com>
> wrote:
>
> > heh, I too, am writing my own C++11 PLY reader/writer :)
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/bclucas/alloy/commit/0287fd1f93025605d392b626a10970c8a05ad1d9#commitcomment-12862072
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bclucas/alloy/commit/0287fd1f93025605d392b626a10970c8a05ad1d9#commitcomment-12862099>
> .
>

--
@ddiakopoulos <http://twitter.com/ddiakopoulos>
http://www.dimitridiakopoulos.com/
<http://mtiid.calarts.edu>


This statement would be a lot clearer if you used `if` instead of `unless`


why not add DB here? only on google-disc?


commentaire test


Dial(..., grpc.WithTransportCredentials(credentials.NewTLS(raftTLS))), and the credentials object can be shared between NewServer and Dial


Ah, ok, so that's not even for the technic radiation calculation? Name confused me there a bit.

Why not manage that on side of the armormod? Or at least rename it. Perhaps someone in the future will come along and change that value, because it's "too high for reactor shielding" and mess with the armor mod's interpretation of the value, because the relationship isn't clear. :/


Hmm, this is awkward for supporting ed25519 keys... the library expects them as [32]byte.


@mapkyca It'd be fair to do that, although we'd need to cache the message, and know enough to reset the cache on upgrade.


Is this a leftover from debugging ?


I'm going to work on debugging it this week, but have no immediate plans to change how one interfaces with the parser. I'm mostly concerned about making sure memory is allocated and freed correctly. There are lots of features in this parser that aren't used because there's no example code. Like .. the "other properties" and "propagation" rules. If there was  better high-level interface, it could be used in a very generic way.


I mean inside the task, to check if referenced variable is still avilable


Not a bad idea. If he picks up some of these rules, then we can change the comment.

I am still working on this.  Frank is providing additional suggestions.  I have yet to post a notification of these new rules on the john-users list. I think I will post that notify to the 'has anyone looked at the Ashley Madison' thread, since these rules work pretty good there.  But before doing that, I want to make sure they are right.


Connections depending on conf is awkward. Everything would be so nice if we had dns or something that let us dial by node id. I guess the server can expose a function to access its current conf from here. There still is the issue of raft caching these connections -- we should make sure to kick nodes form that cache when they are kicked from raft (so when they are readded they get a fresh conn).


This should be fixed with vector-fftw-0.1.3.3.


Thanks for fixing this! I should've done it, but kept slipping my mind :-/


I do not like this line one bit


Oh, I see it now :)

np!


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.464.fb4038ac5fe88a34b9b382e890f5f9a7d0503301](http://build.palaso.org/viewLog.html?buildId=61814&buildTypeId=bt223) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3307, ignored: 137 Build time: 00:26:16
### Failed tests
```
Palaso.Tests.dll: Palaso.Tests.Progress.ConsoleProgressTests.LongRunningMethodUsingConsoleHandler_ProducesLog: <no details avaliable>

```


Won't this crash?


More background on this approach to testing React classes with child components: https://discuss.reactjs.org/t/whats-the-prefered-way-to-test-react-js-components/26/2


I found myself re-inventing this code (as a one liner) once a week or so and it just hit me I shouldn't have to :laughing:


There are a handful of compiler options that must be disabled in order to use the /await option to enable coroutine support. Some of the samples demonstrate using coroutines with Modern so I had to disable those options. You can happily enable these options but then you won't be able to use the support for resumable functions. A future compiler update will lift these restrictions.


lgtm


call function? what it should do?


So there was some error with the previous "animsation" scripts? As in the animated page loading thing? I liked how the pages loaded in the previous version, can was use that same effect?


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1804 of 1804 tests passed)

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


BTW I think the A-M name is misppeled in the rule comment


:heart:


Fair point. I was following this chain of thought a while ago by forcing `runtime.GC()` before the counts and setting GOMAXPROCS to 1 (as it is in 1.4). I still get the "leak" tho, but only in the first iteration:

    $ go test -v -run TestGoRoutineLeak -count 5 -cpu 1
    === RUN   TestGoRoutineLeak
    --- FAIL: TestGoRoutineLeak (0.28s)
            tdigest_test.go:179: Number of goroutines increased after a series of Update calls. Started with 5, Ended with 7
    === RUN   TestGoRoutineLeak
    --- PASS: TestGoRoutineLeak (0.30s)
    === RUN   TestGoRoutineLeak
    --- PASS: TestGoRoutineLeak (0.32s)
    === RUN   TestGoRoutineLeak
    --- PASS: TestGoRoutineLeak (0.29s)
    === RUN   TestGoRoutineLeak
    --- PASS: TestGoRoutineLeak (0.26s)
    FAIL
    exit status 1
    FAIL    github.com/caio/go-tdigest      1.460s

(note: I don't mark this test as parallel-able)

Instead of a grace buffer and then suffering with updating the threshold whenever it fails, maybe I should warm it up with a few calls to `Update` before checking the nr of goroutines? Or is that still too dependant on implementation details?


I am running these single rules over a 1000 'random' logins from the AM dump. I had already run -single (it is still running, and not getting much).

The -single had found 19, and has been running for about 30 hours, and is only at 56%

This new mode found these in 17 minutes (it is 26% through)

```
545454           (teenie545454)
jimmy            (jimmy79706)
mwash            (mwash10)
freckles         (Freckles2000)
franco           (franco7894)
bernie           (bernie58)
joejoe           (joejoe104)
Joeyjoe1         (Joeyjoe2015)
freckles         (Freckles2000)
alejandro        (alejandro12121212)
38933            (eduardo38933)
jimmy            (jimmy79706)
bernie           (bernie58)
franco           (franco7894)
```

So this rule set is pretty damn good.

> BTW I think the A-M name is misppeled in the rule comment

lol, I might just change it to A-M, to protect the guilty.


is this for debugging?


README.md is in markdown format; need to review markdown;
attempted add of files w/+ mechanism in github; not sure if this actually worked;
added file appear to be empty;
done


Why that `cast_helper` struct ._.


It's just writing out variables into a mixlib config style file for chef-marketplace.  Since strict mode isn't enabled nothing happened, but that's intentional at the moment.


@Pilchie, I think you are right. Since this will create a foregroundThread object per thread, rather than per test or per testclass. Perhaps a concurrent dictionary would be better (or making it non static)


Was this logic wrong before? Why AppendCommaSeparatedValues?


`jspm bundle` with a global package map fails with
`err  ReferenceError: normalize is not defined
         at XXX\node_modules\systemjs-builder\node_modules\systemjs\dist\system.src.js:3327:24`


There are only a few animations that Internet Explorer will display that actually look "normal." The previous version used transitions that slid in from the sides or top/bottom of the page but on IE it made the panels "flip" into perspective which looked incredibly stupid.

The new version is a stable animation which looks exactly the same on all browsers and isn't ridiculous so that will be the new default to start with. I'm going to look into how to use different animations for specific browsers which will let us bring the old style back from chrome/firefox/safari.


@qiaohaijun 加油军哥！


If you don't mind, a PR with that method would be fantastic!


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

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


Having two extra goroutines after running 1000 updates doesn't count as a leak to me :)
If you were leaking, I'd expect there to be at least an additional 100 or whatever makes it obvious.


you're right. I clean it up

On Mon, Aug 24, 2015 at 1:33 AM, Mike Krüger <notifications@github.com>
wrote:

> I do not really understand this change. You're now always calling
> FixEmptyFinallyWithCatchClause
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Kavignon/RefactoringEssentials/commit/c5af9bcb8533534d1f879661e6b2c50e08b953c1#commitcomment-12850160>
> .
>


I think we should refrain from posting cleartexts from that dump here at this time. I edited your post. Lawyers can cause more damage than hackers.


I see that you went with function-style push and pop


Did you patch single.c according to Solar's recommendation in john-users? I did.
I also set SingleWordsPairMax = 0 in my local config file.

And I made sure to set BF_X2 to 3 in x86-64.h (which Solar avoided for __AVX__, but which is 30% faster on my Haswell CPU).


Sorry, Kevin, in this comment https://github.com/saltybeagle/Savvy/commit/5e720002d0ca227451b6e6d9900cbbad6b6a2f81#commitcomment-12861592 I was thinking "Plugin" but said "Filter"


I guess the OpenCL format will perform very very bad with this many salts?


@arif-ali  We are trying to find a balance between this new process and not causing too much overhead work.  The consensus at the moment is that the pull request does give us a chance to review what is going into the product and make it better.  However we are still trying to get used to everything.

Feel free to review/comment/share opinions as much as you like! :-)


Hmm, now I started to think that it wasn't, which is strange, because I had build errors due CONFIG_F2FS_FS_SECURITY=y and CONFIG_SECURITY=n. Anyway I probably won't check it now, tired of restoring backups.


This is buggy. The extra space between `) ##` will be included in the output, which causes the `{{html}}` macro to be considered _inline_, which causes its output to be wrapped with a `<p>`, which then causes the previous page arrow not to match the `#diffDiagnosis-search-results > span` selector anymore.

In general, trailing comments are discouraged, they should be on their own right above the line they document.


@shepmaster suggested this morning that the sentence "In the same way that a person has trouble seeing typos in a paper that they have written themselves" to the issues for the book ;)


Very interesting patch: although simple it is difficult for me to figure it out, so I have rewritten in the equivalent way (modulo rounding errors):

    int w = 512 - abs(int(v));

    if (w > 0)
        table[pc][to] = (table[pc][to] + v * 64) * w / 512;
    else
        table[pc][to] = VALUE_ZERO;

I have also plot the graph with x-axis = abs(v) assuming starting value table[pc][to] = 0, it is a quadratic.

![image](https://cloud.githubusercontent.com/assets/1099265/9447158/285ae570-4a96-11e5-9f2c-501b65216be8.png)


No autocorrect in the terminal, apparently :(


bcrypt in generat is rather GPU unfriendly.


Opencl is about unusable for this format.  My 7900 is about 1/10 the speed of a single AVX2 CPU.


@flavioribeiro I already removed it. The situations where I could reproduce the error (mostly with animated resizes/translations of the player) have already been fixed. There were also some other issues where the player was displaying content behind it in pages which have a texture/image background.

If there's still an error on the g1 page, we can fix it through the globo.com player, because it does not seem to be related to clappr anyway.


I don't think we need to keep this exactly the same as OGA db.js. All the `App.whatever` functions were intended as placeholders to be replaced with whatever new hotness NGA needed. You can probably just replace this with `service.broadcast('databaseUpgrade');`


Frank, your ``` @?D Q >4 ``` rule is damn good.  Out of 15 found, 3 of them were the digits.  Very good 'catch'


@csaunders FYI, there's a problem with this change. It fixes the remote delete issue but the watch fails randomly now because of unkown event error. I'll have to take another shot at it sometime.


Bug. No way this should be indented.


Is translation really do not work?
For me, working also after these corrections.


Ahh gotcha. That is what IE stands for. I gotta catch up on my computer nerd lingo...


Append and AppendCSV used to do the same thing, except Append took `string` and AppendCSV took `params string[]`, but both resulted in `new[] { "value1, value2" }`. AppendValues (now Append) results in `new[] { "value1", "value2" }`


Photovoltaics (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


I do not know how to test it.

1. it's seems like it was a server config related (unmapped locale)
2. how would you unit test this ?

It's seem to me more of a integration problem... Or should `findProviderClassname()` be tested with a empty locale ?


This change is not helpful.


Can we add a validation to check service network and cluster network doesn't overlap?
I won't block the PR for this, can be done as a separate change.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 8](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=120&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


I will try to do some crude bench-marking on 20-cores, trying to find the "best" values for A and B on my system, where max_slaves_per_splitpoint = A + depth / (B * ONE_PLY);


did u have local changes in those files?
git checkout meta-brands/meta-ceryon/recipes-drivers/ceryon-dvb-modules-7000s.bb meta-brands/meta-ceryon/recipes-drivers/ceryon-dvb-modules-7100s.bb meta-brands/meta-ceryon/recipes-drivers/ceryon-dvb-modules-7200s.bb
git checkout 3.2
git pull


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 8](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=120&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:16


This beginning of sentence is not needed (it's a guide for the text you have to put)


this works; though it will fire a bit more frequently.


I have been doing the hand cut/paste thing for years. I can not believe I had not figured this one out a long time ago, especially when we 'normalized' the output.


This feature makes easy for the community to contribute back, so it is kind of related to #68. I really liked this feature! Have you thought about a `community_contribution` folder, repository, or something like this?


This should be the name/title of the module (not the technical name): `Export rows...`


Sort to hurt you so, but this has changed, you'll have to rebase :+1:


This should have the same length of the title. There's one PR to the guidelines to add also the same line below title to get another style more significant, so please advance it putting here.


comment


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (754 of 754 tests passed)

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


Indeed, considering that our bonus v = depth * depth, I can directly put depth on x-axsis and I get this werid result:

![image](https://cloud.githubusercontent.com/assets/1099265/9447353/337d9172-4a97-11e5-9b82-2d8df18b38c1.png)

Here we can see that after depth 22 we go at zero and remain at zero. Joona, I am a bit worried about very high depth at very long TC, becuase I cannot figureout how it works in thse cases. It seems the available depth "width" is of about 20 plyes...


@lazedo Thank you so much!


`SF2D_PLACE_RAM` is on the linear heap: https://github.com/xerpi/sf2dlib/blob/master/libsf2d/source/sf2d_texture.c#L52


We can always tweak this - good to have one in place! :+1:


ah! Sorry, I missed the parens around the disjunction.


Nice catch!  Fixed in https://github.com/chef-partners/marketplace_image/commit/ad193b1f5e82da46d9483646593d3bc7d24ae33d


Why is last better than first? Did you find a case where a disk has two storage domains?


Skip all unused comments. More clear code - more common.


```go
	buckets, err := ListBuckets(os.Getenv("TEST_PROJECT_ID"))
	if err != nil {
		log.Fatal(err.Error())
	}

	// Print out the results
	for _, bucket := range buckets {
		fmt.Println(bucket.Name)
	}
```


Ya vi el código de colores.


哦我的上帝。你看每天都要删仓库重新建一遍么。这是闹哪样。


My editor enforces newlines at the ends of files, so that's all that was; sorry.


Why would this be reformatted?


Good; and `git rebase` was smart enough to detect that, too!


Hi! I've tried this example in the Energia IDE, but I have some problems when I try to connect two or more clients to the AP. The first client seems to connect without problems, but the second one does not connect. I've tried with different deviced (PCs, mobile phone) without success.

Can you help me?
Many thanks,
Simone


It is impossible to check from outside if a lambda capture is still valid. You could use std::weak_ptr in the lambda for a similar effect, take a look at the `TaskContext` implementation which is safe to use when its TaskScheduler was destroyed.


how about removing that html?


说笑了，刚刚搭好个框架而已，代码丑陋无比。。。话说刚刚重构了一番←_←正在构思新关卡←_←


Nice clean setup of your design. Good use of jQuery, Bootstrap. Looking forward to your JavaScript file.


edddadfaaf


Extra lines. (( if you want separate some code sections, one line will be enough.


Only set this status message after checking if the old and new membership types have different financial types.


Ty missed that


Good catch!


Only set this status message if old and new membership types have different minimum fees at civicrm/admin/member/membershipType?reset=1.


The above graph is with starting value of table[pc][to] = 0, below I plot also with starting value 10000 and 20000 and I see a plateau around 30K, history does not grow anymore above that point.

![image](https://cloud.githubusercontent.com/assets/1099265/9447650/dbb3295a-4a98-11e5-85e9-f3410e480e76.png)

![image](https://cloud.githubusercontent.com/assets/1099265/9447661/ed97e48a-4a98-11e5-9824-de24cfd172d9.png)


Will this allow mono builds to use the NuGet package? If so, will you be pushing a new package soon?


std::min(abs(int(v)), 512) / 512;

This only output 1 or 0.
If it is 1 then the entry is cleared.
If 0 then nothing happens.


Resuelto
2fba66d648eb1def3d1c88ae3fac947c1c7036d8


We couldn't use bindValue as variables passed to this function are stored only as references which is highlighted in manual :)


damn :D


Yes, this pull request is for azkaban/multipleexecutor branch


That's a good point.  It won't race today but it's due to an accident of implementation vs. an explicit design goal.  I will mull over this a bit.


Isn't it better to use a list?


well, it still doesn't work :D


@Yixi You have to modify the checks in the handlers as well. See, for example: https://github.com/Yixi/ritzy/blob/c26a18c100026cb2144ffb27770a8335889a0c36/src/components/TextInput.js#L219


This line has no effect since you don't use `CMAKE_THREAD_LIBS_INIT` anywhere.


You can't us the variable here because you did not find package `Threads` here.


Will update that check.


Looks like we lost support for 400hz chips.
This should be an easy fix, in the Begin, check the flag for 400hz chips and drop the baud rate in half.


It seems like in the majority of call sites, `ls = True` is being passed. In the minority where it isn't, `fileSequence()` seems to be used as a cheap way of saying "is this a file sequence?" rather than "give me a file sequence with no useable frames". Does that sound like a reasonable assessment?


(Unless Mousetrap is converting the command+something into ctrl+something in the handler? Did you check this?)


This line will include all JS(coffee) files to your project. No good idea add empty files. Check files and remove unused.


@VoyagerOne take another cup of coffee :-)

std::min() output values goes from 0 to 512, then are multiplied by a possibly big value in table[pc][to] and only at the end are divided by 512.


Ah I see.
Will give it a shot later today :)
Thank you so much for the explanation.

2015-08-24 9:32 GMT+02:00 Garrett Brown <notifications@github.com>:

> here's a perfect example of why each add-on has its own button map.
> Compare yabause buttonmap.xml
> <https://github.com/kodi-game/game.libretro.yabause/blob/master/game.libretro.yabause/buttonmap.xml>
> with Genesis plus buttonmap.xml
> <https://github.com/kodi-game/game.libretro.genplus/blob/master/game.libretro.genplus/buttonmap.xml>.
> "x" and "y" are swapped between the two different emulators. I had to crawl
> through their libretro implementations (here
> <https://github.com/libretro/yabause/blob/master/libretro/libretro.c#L622>
> and here
> <https://github.com/libretro/Genesis-Plus-GX/blob/master/libretro/libretro.c#L1611>)
> for this info
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kodi-game/game.libretro/commit/cc23f54b7f891b5bc6277baff4027eb63221c9a9#commitcomment-12851244>
> .
>


Added [Here](https://github.com/SparkDevNetwork/Rock/commit/b44aae5cb9ef38385be06944dfc5e5dcf38a1e24) (sorry).


Yep, for sure


@hkraji Ispravljeno i ovo :)..


References are very bad, we should avoid them as debugging of the references could lead to madness.


Very nice project overall. Looking forward to the final product.


Although, that properties section could easily be re-arranged so ls=True is only written once, and then its the minority I guess


neat, thanks for updating the writeXML method. Is it more common to do it this way than to create the xml object from inside the task and output it?


overflow is fine as hashcode is suppose to give us a uniform spread of hashes to bucketize collections of an object.
p.s. this is an auto-generated method by eclipse


Yeah, remove it I reckon. It's useful for the generic UI elements because they don't know much about the underlying filters they represent. But in this case we know we're always just turning sequence view on and off.


Take your time; it seems like we have other test failures with Win32-API-0.82 on Perl 5.14 with VC6 (plus some hanging), so we'll revert back to 0.75 for now.  @andygrundman has more details, if you want/need them.


Yes, I asked forkineye about that but I can't test it as I only have the new ones.


Thanks so much for looking at and working with the code, @theRemix.
I will look these over at greater length soon.
In the mean time, take a look at issue #3. For this project, the ability to simply parse args seems ancillary to the ability to generate data. So I considered, early on trying to separate that functionality... via two files.
args.py is intended to do all my arg parsing and at the end, output a return that could be easily fed to the separate file data generator so that data generator could then generate the data.

The tie between the two is still tenuous/non-existent.


Using spaces instead of tabs.


Maybe simpler:

```ruby
def self.verb; '#{v}'; end
```


heh that's true. I've pushed 19951fc30c0a5d5eda238d3122b5b862e4568524 which actually does the warmup thing I mentioned, but - as it often happens - I might be proven wrong and I'll get back at your suggestion :)

Thanks for the help!


Good work porting. Waiting for you to initiate a 3.10 project. :)


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


don't  use Camel Cases names for css classes.


@afred: Thank you for pointing that out. Right now, I can't see a reason why the first part of the disjunction should be there... At the very least tests are still passing with this change.


I think we could have a possility to only generate SQL query data optionally, but by default everyone want's to execute the query immediately :)


Wait, shouldn't you iterate over handlePacket result and add them one by one?


Great function and integration of a Third-Party extension.


Good work! Nice code overall.


@baiarex

It seems the CC3200 can only handle one client in AP mode for the moment.

See [Number of client supported by CC3200 in AP mode](https://e2e.ti.com/support/wireless_connectivity/f/968/t/353453) and [CC3200 AP multiple clients feature](https://e2e.ti.com/support/wireless_connectivity/f/968/t/388122)


go ahead and make the update, I will see if I still have one around.


f


delete your local repro and make new git clone and all ok


Yep, I was quite shocked when I saw that TWRP for i9300 can't understand lzop -9, because apparently lzop -9 requires special config while building.


yeah that nerves xD


multiple different objects can have same hashcode while in equals we want to have a deep comparison of objects. Hashcode and equals have different responsibilities
p.s. this is an auto-generated method by eclipse


good catch. updated.


haha...my brain is broken today.  I need European espresso...


Bit of a zombie post, but if you want to submit the PR for this I'll merge it.


1. Yes, just add a space on both sides of the codefencing. The paragraph above the code fencer opener should have a blank line under it, as well. The whole codefence areas has a blank line above and below it to make it stand out.

Like (paragraph before the code)

This (the code + fencing)

Here (paragraph after it)

2. sweet!

3. All of our docs are rendered using Grav. So, in general, we follow the Grav markdown guides. Here is a link to the documentation: [right here](http://learn.getgrav.org/content/markdown)

Don't worry about the labels and stuff at the top of the page. Best practice, if you're creating a new document that never existed before, is to copy an existing one's folder and work from there.

There are a few tools we use (such as those callouts on top of images) that are a bit tricky to explain, but I can add them after the fact easily enough (if needed).

Thanks again! Please submit the pull request when you're ready to have it become part of the docs site.


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)


Ben Gamari <notifications@github.com> writes:

> @alpha0010 then ask the linux-mm folks about it. As it turns out the
> [thread](http://thread.gmane.org/gmane.linux.kernel.mm/65493) that my
> last interaction with the mm guys spurred discussed `FADV_NOREUSE`. In
> fact, an (unfortunately incorrect) implementation was even offered. It
> might be good to bring up the matter again; they would likely want to
> know if there is demand for a feature that is currently not
> implemented.

This commit was superseded some while back.  Please see the list (or the
master commit log) if you're interested in the further developments.
For now, we're just using fmincore, but if someone would like to pursue
the issue further at the kernel level, that'd be great.

Thanks
--
Rob Browning
rlb @defaultvalue.org and @debian.org
GPG as of 2011-07-10 E6A9 DA3C C9FD 1FF8 C676 D2C4 C0F0 39E9 ED1B 597A
GPG as of 2002-11-03 14DD 432F AE39 534D B592 F9A0 25C8 D377 8C7E 73A4


yay tests for catching this


What do you reckon to matching the API to those use cases? A cheap `bool isFileSequence()` method and an expensive argumentless `fileSequence()` method that always returns a useable sequence?


@dcs619 , @nairdo Updated an older migration here, can you update manually in your environments that have already run this migration.


Hay que abstraer de cmsis también.... :) creo estas tratando de que funcione en hw y no es la prioridad, mi prioridad es que funcione en C. sin hardware :)


I can't say with certainty whether it's more common, but I would argue that it's more flexible so far as not requiring the code calling the function to merge an output XML object's content.

Date: Mon, 24 Aug 2015 10:51:41 -0700
From: notifications@github.com
To: AndroidSchedulerApp@noreply.github.com
CC: peter_lu18@hotmail.com
Subject: Re: [AndroidSchedulerApp] Updated Task class' WriteXML method and added a unit test that generates and validates an XML file (50befc1)

neat, thanks for updating the writeXML method. Is it more common to do it this way than to create the xml object from inside the task and output it?

—
Reply to this email directly or view it on GitHub.


updated


Have you used anywhere all this empty controllers? (app/controllers/users/---------)


si ojo, solo si se usa todo el tiempo. no se si es una funcion que llamas una vez o muchas veces de forma ciclica.


No; this is the equivalent of `ruby -w`. After reconsideration, I don't see a need for verbose mode when running these hooks, so I will remove this line.


Ah - I see what you mean.
Heh - that's what I had before, but I kind of liked the if-scope idiom, and I'd seen it somewhere else, so I thought I'd try it out. Anyway, changed it back.


Line 2, 6 and 8 are questionable. Does jetty maintain (create) files in /usr/share/share/jetty, /etc/logrotate.d and /etc/tmpfiles.d?

From an administration POV is would make sense to give these a private type but that would be inconsistent with the remainder of the policy.


I may keep this fork private because I had to implement some custom functionality for my business (e.g. widlcards and no caching) but if you see any value I'm more than happy to contribute back with a PR.


From my small tests it still speeds up first boot sequence by using preoptimized boot classes, and preoptimized app classes, even if ART still needs to do the work with other (not-preoptimized) APKs.

I'll leave it for some more tests, revert if it won't be worth it in my opinion.


??? See question above.


Hmm yeah, maybe you're right...

Alan

On Mon, Aug 24, 2015 at 4:31 PM James Oguya <notifications@github.com>
wrote:

> I don't know if we should be focusing much on a non-LTS version. Anyway,
> for the PostgreSQL stuff, I think we should be using postgresql from the PGDG
> repo <http://apt.postgresql.org/pub/repos/apt/> instead of depending on
> the version provided by Ubuntu.
>
> Check this out oguya/ansible-ckan#4
> <https://github.com/oguya/ansible-ckan/issues/4>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ilri/rmg-ansible-public/commit/cebabfc1a33e361f04c33585e1306cf3c4740f5c#commitcomment-12857248>
> .
>


Does that files_search_rw() actually exist?


yo usaria un namespace para esta variable
speechrec_pdm2pcm_FIR1coeffs o algo asi. se entiende ahora?


@bigdaz checkstyle jar file contains one config file in root folder and a lot of files in documentation. So, do you propose to use config file in root folder of checkstyle jar file?


does that type "jetty_rw_t" actually exist?


scss .... a typo


Ok, agreed  -- deleted the 0.3.1 release notes; we'll go with the convention of only making notes for the x.0.0 major releases.


:heart:


Oh i see you created it. I think that is probably a bit too much, but else i would use "jetty_content_rw_t" instead i suspect.


omg, what a typo :stuck_out_tongue_closed_eyes:


Any chance we can get the ability to get these borders back by setting a prop flag? Was kinda surprised to see my avatar borders gone today :)


We don't want to ignore a duplicate entry but show an error to the user.


in this case, is it up to the caller to make sure it's passing an XML object to the function?

ie in the function itself, do we still have to check if the parameters are valid?


Actually the "should remove the item if an empty text string was entered" in "Editing" group is already failing because the final list is missing two elements instead of one.


What about this sequence of operations?

- Use browser to set plug to 'seqA.###.tif 1-10'
- Use string field to set plug to 'seqB.###.tif 1-10'

Will that last operation cause the frame list suffix to get pushed into the path name?

I don't think I understand the question about hanging on to a path object though - is that something you need to be able to do?


To make it unambiguous, why not make the method receiver a value object instead of a pointer (same with the return type)? That way, it's clear what's happening.

As it is, it appears to do what you expect but the reader has to parse both the receiver, the internals of the method, and the return value. I still wasn't convinced until I ran similar code in the go playground.


you can pass to redirect extra parameters, like this:
redirect_to somewhere, flash: {error: 'message'} - more elegant.


Testing commit's on this GUI


We can, and do in a follow-up commit.


Not happy about it either, but old constructors will start to get deprecated and will be removed.


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1798 of 1798 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)


You forgot to take this one out :)


This would be a great place to document what the match-groups in your previous regex are by using some nice argument names.


But it wouldn't work as a link opening directory containing profile?


FourPipeBeamAirTerminal (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (1804 of 1804 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)


Nevermind...

```
avatar: {
            borderColor: Colors.white
        },
```

In my theme :)


see https://github.com/callemall/material-ui/pull/1252


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

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


@jodoglevy , your change to the merge method reintroduces issue #21. Merging, instead of replacing, the data is what fixed this issue.


To avoid leaking `zfs_dbgmsg_t` structures your going to need to add a `kmem_free()` here for the _KERNEL case.  This is why the tracepoint patch initially left these functions enabled.


Please use curly braces when the consequent extends past the current line.


Out of curiosity, why do these functions need to be added in the parser instead of the frontend?


Do you mean the validity of the XML object being passed in? The function within the Task class will likely never be accessed outside of the Scheduler class, so one can generally assume that the input will always be valid, but It certainly wouldn't hurt to add validation logic.

Date: Mon, 24 Aug 2015 11:07:18 -0700
From: notifications@github.com
To: AndroidSchedulerApp@noreply.github.com
CC: peter_lu18@hotmail.com
Subject: Re: [AndroidSchedulerApp] Updated Task class' WriteXML method and added a unit test that generates and validates an XML file (50befc1)

in this case, is it up to the caller to make sure it's passing an XML object to the function?

ie in the function itself, do we still have to check if the parameters are valid?

—
Reply to this email directly or view it on GitHub.


Ceolin explained in person this one. I missed the fact that we are using the same callback for conns fds and the listen fd. Minor: I think is worth a note either here or in the callback code about this.


@twalpole setting that value *at all* is not supported


Quick reply :) Found it, thanks


Semicolon after


test


@jodoglevy also I can't tell from your comment what the problem was with edit asset. Please open up an issue on that and assign it to me and I'll try to find the underlying problem.


This line is throwing an exception after the student chooses the affinity cycle. Please test this in you applications before the enrolment season.


I see. But that merge broke updating assets, which I think is more
important.
On Aug 24, 2015 11:12 AM, "alstab" <notifications@github.com> wrote:

> @jodoglevy <https://github.com/jodoglevy> , your change to the merge
> method reintroduces issue #21
> <https://github.com/azureautomation/azure-automation-ise-addon/issues/21>.
> Merging, instead of replacing, the data is what fixed this issue.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/azureautomation/azure-automation-ise-addon/commit/38f05c48020b56c6c47e0cee29c8d9aae56c83fd#commitcomment-12863446>
> .
>


IIRC, `OpRef` was intended to be immutable. Better to create a new `OpRef` instead.


And if we decide it's not immutable anymore, we would want setters for all the fields, not just the args.


If we actually investigate why updating assets wasn't working, we can fix both problems, there's no reason we have to pick just one. But again, I don't know what problem you are referring to.


what that doing here?


I thought an array  would be faster.


just a reminder: another commit with cache needs to be done.


I don't believe this is required.  I think the image default is to provide transports for all protocols if this is empty.


Great! Much better! Would you mind tagging the current master? I'd rather lean on a release instead of a git repo.


just a reminder: (in another commit) env['JobID'] has to be used.


alright thanks


This is required, otherwise the readiness probe will fail (I think).


This has has changed from add to addAll() in a newer commit.


Hi! It's doing something really weird with my picture (http://www.pyladies.com/locations/paris/) :smile_cat: . Is it possible to do it differently?


you can download the hpi file under releases here at github.


Should be "In your application, please include [...]".


These are currently required, but I think there's a JIRA to make them optional.  OK to leave them as required for now.


And  also adding missing boxes


Any reason this does not use a build_style?


I wanted to at least add an option of using function style pushes and pops.
As symbolic equivalents I want to first implement X= operators ( with X operators like +,-,/,*.. etc ) and then simply use list concatenation for pushes:  list int x = [1,2,3], x += [4].
Equivalently for sets, which also gives a remove with -= {x}.


Epic work :)


I realized that a condition for section is necessary:

     source ||= File.join(PackageProvider.root, 'config', 'package_provider.yml') if section.nil?


@jesssummerfield Great copy! Especially love the part covering the blog posts for the Account Executive position. I found one typo which I've commented on inline. All good otherwise!
Thanks for your help with this. x


Value going back to zero is not intended for large depths. I was of course
aware of it, but I approximated the impact to be "small".

Instead of:
table[pc][to] -= table[pc][to] * std::min(abs(int(v)), 512) / 512;

I'd like to write:
for (int i = 0; i < v; i++)
    table[pc][to] -= table[pc][to] / 512

For small v, it doesn't make a big difference. But for large v, it's a bad
approximation...

On Mon, Aug 24, 2015 at 6:28 PM, Marco Costalba <notifications@github.com>
wrote:

> Very interesting patch: although simple it is difficult for me to figure
> it out, so I have rewritten in the equivalent way (modulo rounding errors):
>
> int w = 512 - abs(int(v));
>
> if (w > 0)
>     table[pc][to] = (table[pc][to] + v * 64) * w / 512;
> else
>     table[pc][to] = VALUE_ZERO;
>
> I have also plot the graph with x-axis = abs(v) assuming starting value
> table[pc][to] = 0, it is a quadratic.
>
> [image: image]
> <https://cloud.githubusercontent.com/assets/1099265/9447158/285ae570-4a96-11e5-9f2c-501b65216be8.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zamar/Stockfish/commit/9d148843bce4d653480786cbdebade141ab75dbf#commitcomment-12862531>
> .
>


The template meta-programming evades some annoying issues with C++'s template argument disambiguation.


What about calling ".to_s" instead of ".to_json"? Shouldn't this be enough?

On Mon, Aug 24, 2015 at 7:01 PM, Wojciech Sielski <notifications@github.com>
wrote:

> Actually its a tricky problem.
> if export FOO=42bar
> then %%FOO%% is replaced to "42bar"
> but
> if export FOO=42
> then %%FOO%% is replaced by 42 (without quotation) which is not readable
> by
> .to_json was needed to bring back quotation.
> now we fix this problem, but open and old one.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vgrigoruk/marathon_deploy/commit/4081d7a010c98e1c7e9c0a2c3df71fe006090d9e#commitcomment-12862030>
> .
>

--
Sincerely,
Vitalii Grygoruk


We need the new line between lines 15 and 16


I don't think this is required (see above comment for amq templates).  EAP is hard coded to use openwire/tcp, since that is the only protocol supported by the A-MQ connector.  We might want to improve the documentation for this parameter on the EAP+A-MQ templates to note this.


It deny access to cart if user not authorized


You need a new line after line 6. There should be 1 blank line at the end of the file.


It is more about disks than storage domains. I used last because the last disk in a list seems to be the first (aka _root_) disk of the template.


It was adding `setAutoRequestFocus(false);` in `src/com/marklalor/javasim/simulation/frames/subframes/ImageSubframe.java` that fixed the main bug.


Why aren't we simply requiring Java 1.8? 1.6 and 1.7 are EOL for quite some time now. The distros have it in their stable releases (and libjitsi isn't in any of them). The desktop Jitsi ships with 1.8 runtimes as well.


duplication code. You can use authorize_resourse. see [more](https://github.com/ryanb/cancan/wiki/authorizing-controller-actions#authorize_resource)


It is a conflict on the merge
Thanks for the answers


Ok. I don't knew this


Thank you for your reply and the link. This is going to sound so stupid.
I can't figure out where to get the button "Create pull request"
You must be laughing your pants off. I am sure. I am going to start clicking a few things see what happens.

PS: I am not that familiar with Github. So I am going to do some read up on https://help.github.com/articles/using-pull-requests/

That should get me starting. I think it is smarter for both of us if I make first more changes to this particular page and then submit it to you. So you don't have to do the same page twice.


GC roots are usually targetting out-side the nix-store, to symbolic links which are handled by the user, such that they can garbage collect it by removing the symbolic link.

gcroot -> /…/.profile -> nix/store/…


@matthewd - ok -- it is breaking tests in devise, I guess the test needs changing then


Huh, I've modeled it after other two-char tokens... but I guess I've missed something


you can hard-code it into the image itself, e.g.:

```
<img src="https://foo.bar" height=400/>
```

The `height` is just a random number, be sure to test it locally.  You can also use `width` (as well as using both).


This is only required for generating SSL configuration, but is not required by the image.  If not specified, https will not be configured for the server.


Same as above HTTPS variable.


Yes, of course.

Now that I look at it, in the streamlined case, I'd like to get rid of ```zdm``` and the size-counting altogether and simply ```kmem_asprintf()``` the message (which would still have to be freed).


This is required for the readiness probe to work.  Without it, the amq pod won't come up.


Could you explain, please, about 'another task'? I understand your first sentence
as to move this check to checkstyle task action. Do I understand you right?


You make an interesting point. This is just a quick fix for the "lazy" ones. As you can see, my code doesn't break anything as session_params is  an optional parameter that can be passed.Would you think it would be  better to add this to the documentation on slumber and still leave this code on?


Artifact of merge?


Adding License Checker test case file containing two test cases total


Fixed!  @azturner+++++++++


Static import


Not every path has return value.


This is not required.  It's only used if the project that needs to be built is not in the root of the repository.


please remove


Not sure how you generated this, but you may want to look at http://tools.medialab.sciences-po.fr/iwanthue/ to generate a "visually distinct" set of colours.


そういえば通常ツイート機能もつけた


did you meant `lib.escapeShellArg`?


Ok, this is best, lets add the link and merge this.

o/


As I understand documentation - authorize_resourse used for RESTful controller. Here we have non-RESTful controller. [More info](https://github.com/ryanb/cancan/wiki/Non-RESTful-Controllers)


ok, sure


Why do we need shims?


For all DB types, uid/pwd and database are required.  Admin pwd is also required for MongDB.  See here: https://docs.openshift.com/enterprise/3.0/using_images/db_images/overview.html


Write a unit test in CParserTests and try parsing `a[a[i]]` -- parser is expecting two ] tokens, not a single ]] token.


When you edit an asset value in our UI, then go to edit it again, the asset value is not the value you just set it to. In the .json files it is set correctly, just not in the UI


We should probably make it so that the USER_HOME or whatever it's called is also localized to the virtualenv, otherwise it defeats a lot of the purpose.


Hi @zeeamber, this `>>>>>>>` is the Git mark for not conflicts, i think you forgotten this one here.

*(I dont know if you already resolved and just skip this or not even resolved).


Just exercise text and images -- no video stuff etc in there.


Any particular reason why this was bumped to 0.4.5 and above?


nit: Capitalize NixUP, idem below.


@cagryInside Thanks for getting this ready for the PR!  I'm going through and adding some additional documentation.  Looking at this line - I believe that the EXCEPTION here will never be reached?  I believe it should be removed, because  we do actually support searching for keys outside the key range of the key-value map.  If you agree, I'll remove this case here in the switch.  I'll leave it in `EOutOfRangeStrategy` enum for now, just to limit the scope of this PR.  But, I believe it should eventually be removed there, and from `InterpolationLibrary` as well.


Should we warn that there's no change_cb ?


Ow, sorry, forget this, i saw you fixed this in the next commit


This doesn't work as intended.

This is in the HTML of an actual Arastta install:

    Arastta Marketplace © 2015. Powered by Arastta eCommerce

But this regular expression will only match "Arastta" before the `>` of the `<a` opening tag.

It will still work, however, because Wappalyzer matches regexes case insensitively; because of this, the `Arastta`part of your regex will match the `arastta` in `arastta.org`.


Ah, never mind, I mis-read the use of this.  If `EXCEPTION` is passed in as a strategy by the caller, then it will be thrown.  So, its ok as is.


Looks good.


should use single quotes for consistency


inconsistent spacing around the conditional


inconsistent spacing relating to closing parens


I have a couple of other small tweaks I'd like to get checked in and then
I'll cut a v0.11.0 release. I'll try to do that in the next day or two!

On Mon, Aug 24, 2015 at 2:21 PM, Frederik Bosch <notifications@github.com>
wrote:

> Great! Much better! Would you mind tagging the current master? I'd rather
> lean on a release instead of a git repo.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zslayton/stomp-rs/commit/18a006f3270f45159683b906460cdb6f3d675255#commitcomment-12863606>
> .
>


does this need to be checked for existence? It would probably be good to throw a friendly error if wtVersion is not specified in pkg.config, or even if pkg.config is not defined


Forgot I could just use make_install_args. I fixed it. Thanks


tested and working


@MCGallaspy @benjaoming This link should be https://learningequality.org/r/deb-bundle-installer-latest


Nice trick, but you might want to explicit the `config = {};` argument instead of taking a random one from `pkgs` with the `callPackage` function.


[See it in action](https://asciinema.org/a/1sh4v4uqhdgp9dnos7eokx1vr).


Yup, because ES6 classes don't support newless instantiation.


@FWAJL
Doh!  I goofed it because the case is not lowercase, it is camelcase, and this particular one has only one word, so lower case only.


What were the files before?  What did you change them to?  Github doesn't tell us the file permission changes... @tremainebuchanan


Separate issue for that?


We wasn't warning it before this patch, that's why I am not warning now. But, maybe it is a good idea to warn that. What do you think?


If there's nothing else in the container, you could probably do `this._annotationContainer.selectAll("*").remove()` to empty the container.


Danke! Geb ich gern weiter, nur fürchte ich sind wir eh schon fast durch :/


rename `Nixuser` to `NixUP`.


So, I think we leave the LICENSE file like this, but in the code we put the year and Google...


big header here


Thanks for making this change. Suggest to remove commented code.


unnecessary create extra method only for once using.


I think you want to compare against >DBL_EPSILON and <-DBL_EPSILON, not 0. The implied condition of the final 'return 0' is that diffpopularity==0, which simply isn't proper for floating point comparison.

Please keep to style, as well -- always use braces with if, even for single line bodies.


I was receiving interrupt messages, even if i did not connected the pins.
Setting the pull-up worked perfect!


Reading this, I think this isn't enough, because `Path` and `RelPath` extend `BasePathImpl` — otherwise you'd have had to drop the tests using this (labeled `'Comparison`).

> Remove symmetric path-comparison operator to make it more obvious that these aren't your normal total ordered comparisons

I'll still go ??? with this API — it's more obviously weird, but the intention seems still far from obvious.


@tremainebuchanan what happens if there is no report to send?


Product.find(params[:id]) - do the same, but more clear.


It would be really great if we could use JDK 1.8, but, unfortunately, I think the default JDK in Ubuntu 14.04 LTS is OpenJDK 1.7.


It should be this one: http://freegis.fsv.cvut.cz/gwiki/S-JTSK_/_%C4%8Ceskoslovensk%C3%BD_transforma%C4%8Dn%C3%AD_kl%C3%AD%C4%8D


This is inconsistent with what's provided for search output -- %.2f versus %f. I tend to think that stating popularity with 2 significant digits is probably accurate enough for anyone.


Perhaps it would be easier to rmove the __EMPTY_OPTION_VALUE__ and replace with an explicit default: interactsWith . see #44


@ibauersachs it would be really great if we could use JDK 1.8, but, unfortunately, I think the default JDK in Ubuntu 14.04 LTS is OpenJDK 1.7.


Any plans to add some tests for that? I started to move all the *repository related*-logic to a participants service, and created a new IT for it (**participantsITSpec**). But if you like, I can add the additional coverage when I migrate the participants route towards service usage.


And some functions (will be added to admin panels)


nice!


Same here. I don't think we need 6 significant digits for popularity.


@PhilipShilling download page is broke


We should activate the new profile immediatly for "test" and "switch".
`if [ -n "$canRun" ]; then`


FourPipeBeamAirTerminal (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (1804 of 1804 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 was the only file left in the `src/` directory. Since git doesn't support empty directories, this removed the directory from your repository, making the directions in https://github.com/ingydotnet/...#installation somewhat confusing.

I've fixed this in https://github.com/ingydotnet/.../pull/19


meeeeeeeerci


@rei-vilo is right, only one client is supported by the CC3200 so there is nothing we can really do about this in Energia.


Why did you add some extra indentation here?


You need to close the file


delete this repository


wellington is wt.exe on windows. https://ci.appveyor.com/project/drewwells/wellington/build/artifacts


so since u have resources here u done need get 'advertisements/index' or get 'advertisements/show'


add some comments


A one-time fix along with this update would fix that. Some sort of query to check if they're on bat hunt with the old var.  It's not easy to just check via sql though, i suppose


Okay. I think I get it now. I believe that function _should_ be doing the configured escaping as your example from AuthNFailures makes the assumption that the values passed in the `$array` and `$selected` are safe (they are not escaped anywhere else). Thus using this method currently leaves your app vulnerable to injection.


Awesome :) Done!


@item.save - better way for checking errrors


The changes in this doesn't align with the description.... @tremainebuchanan

"Removed FROM_EMAIL and renamed FROM_EMAIL to REPLY_TO."


Should we rename _water to _fertilizzer?


Good point Bob, I will update this.


@zetok HTML is a serious problem for translators. Thus, if HTML is used, better move it to source. In this example it would become:

```c++
QString html = QStringLiteral("<p><a href=\"file:///Dir_Path\"><span style=\"text-decoration: NONE; color:#000000;\">%1</span></a></p>");
dirPrLink->setTextFormat(Qt::RichText);
dirPrLink->setText(html.arg(tr("Current profile location:  Dir_Path")));
```


I think qdesktopsevices should be used for that. http://doc.qt.io/qt-5/qdesktopservices.html


This could still end up empty if the user is not an editor (i.e., doesn't have any of the requisite Management permissions). Looks like the "Administration" menu uses the format "if permA or permB or permC...". I think that is the approach that should be taken here.


лол


Is that CoffeeScript on line 97?


I think this is the most urgent problem.  As soon as we have a solution for this one, we can iterate.


Leave a comment


LGTM


Because. They had to go XD


ES6 arrow functions.


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1279 of 1280 tests passed)

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


nit: `strings.ToUpper(name)`?


if (_webDriver == null) {
             return;
         }
-
         _webDriver.quit();
         _webDriver = null;


we might be the first people to have a ruby app talk to an rest API served via IIS Express...
but we needed to change this line for it to work.
they are adding a colon after the version. like

so our regexp is now `/\AHTTP(?:\/(\d+\.\d+))?:?\s+(\d\d\d)(?:\s+(.*))?\z/`

interested in a patch ?

CC: @jdaudelin


Right. Coulda sworn I had it like that...