Thanks Mark!


Thanks for the info, this all makes sense now. I agree on updating the readme for each IDE. I will get to it sometime soon.

I might also open a discussion on this issue to make it even more automated. Keep an eye out for that later.


Remember to add the new arguments to the config template


Hey, thanks!


I see. Nevertheless, the amount of data stored in memory grows with time. Im not sure how much this will be of an impact :)


This is an optimization because forEach() has more complicated logic per spec to deal with sparse arrays. Also it's better to not allocate a function when we can easily avoid that.

No, we can't just limit ourselves to the last index because subscriptions can also be removed. However we do have an optimization to avoid cloning the array on every dispatch: see the next few commits.


since this is per event basis, i am pretty sure this is negligible.


On second thought i think this should ona context basis


The changes look good. yay.


Yeah, sorry Dan. I looked at this commit before noticing the follow-up commit. :)


Ok, I'm _totally_ doing this from now on.


change back to Jeef Leek's website style.


Oops. I should add this configs to the linter.


Could do:
awk '/^SHA256/{print $2; exit}'


First comment


Ah, shoot. I think I mixed up Pawn with Vala.

On January 31, 2016, at 4:27 PM, Ben <notifications@github.com> wrote:

https://wiki.gnome.org/Projects/Vala/Manual/Preprocessor

It's not possible to define a preprocessor symbol inside the Vala code (like with C). The only way to define a symbol is to feed it through the valac option -D.

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


As for your last point, I believe modern engines optimize array.length access in the loop anyway and hoisting that call doesn't make it faster.


Right!?


The new code is simpler but there is *nothing* that ensures it is correct. If the linked list implementation changes in the future this code will silently produce the wrong measurements.

In general, the increasing use `size_of` is unfortunate (here, and in #26) and goes against my original design for `HeapSizeOf`. See the "There are two possible ways..." comment at https://github.com/servo/heapsize/blob/master/src/lib.rs#L57-L70. The key challenge of writing code that measures memory usage of data structures is ensuring that it's correct. The design I used in Servo was based on the one in Firefox, which evolved over several years in response to experience. Every use of `size_of` and hard-coded magic number introduces potential for incorrect measurements in the future.


It's just a mandatory HTTPS change. The URL only needs to change to https://download.sysinternals.com/files/bginfo.zip


Thank you!


@nnethercote The older code was not ensuring anything was correct either but the size of the LinkedList itself, and what was definitely not ensured was safety.


500g limit is duplicated


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1272](http://teamcity.monogame.net/viewLog.html?buildId=14213&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:28:59


but i also get this with stone pressure plates


PFFFf yeah


Sorry my intention was to make another PR for this commit.

Here are some screens from the multiple:
http://imgur.com/a/WFt45


@b44rd I agree whole heartedly that we should adhere to some style guide.
However, I believe that this should be done programmatically as a pre-commit hook instead of a config. Something like https://www.npmjs.com/package/jscs is what I have in mind.

Your thoughts?


Committed as https://github.com/w3c/csswg-drafts/commit/001619619546ae602818d3fd45f8b5ac3e7ce917


IMO the way to go is to make it safe, then make it correct by making it so pervasive in the community that it gets upstreamed in stdlib. Maybe that's just wishful thinking though.


REVERT THIS.


Thanks for correcting my grammar @daniel-acuna!


Ah


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13768](https://teamcity.mesosphere.io/viewLog.html?buildId=119425&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 216 Build time: 00:12:32


@mkambol SwtLinkOrLabel is used only by Sqoop dialogs, as far as i can tell, to switch between Basic and Advanced Modes. Didn't show up in a search of commons


fixed


lol yeah I was browsing commits the other day and this totally flew over my head. Was confused af when I saw my build broke, but quickly realized why it did. xD


Yeah, I remembered here 14a90d84ec62756f764e12eeeda3f943c5a448ac


Don't get it. Highly suspicious to me.


Because the library is private and therefore needs an access token which is in the npmrc file.


ObjexxFCL41 (DeadParrot) - x86_64-MacOS-10.9-clang: OK (2062 of 2062 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.

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


I could really use this ability to list/find snapshots (especially by tags).


I need encapsulation this method in the small methods.


Should be a "*" before sclk parameter (kw only thingy).  sclk,mosi,miso defaults are: pin13, pin15, pin14 respectively.


Only bits=8 is supported.  But I think it's important to keep the parameter in case things change in the future.


"The length of the buffers should be the same.  The buffers can be the same object."


also made cursors save in file type again


It is so cool! 多少money啊？


What happens if all 4 wheels are 100% ? _fixable will be false and nothing can be done?


> However we do have an optimization to avoid cloning the array on every dispatch: see the next few commits.

@gaearon if you ping me once the optimizations are complete I can migrate them to redux-batched-subscribe.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13770](https://teamcity.mesosphere.io/viewLog.html?buildId=119441&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 220 Build time: 00:03:51


Made isRefreshing Non static


This constants and a few code lines below, tell me that maybe this `ProfileFragment` is not the right abstraction.


Was this your typo? My current version doesn not have this error?


This was in the January newsletter, and when I contacted Jason at Eastridge, he said they did not have an opening. So, unless you have new information, you might want to omit this from the February newsletter.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13769](https://teamcity.mesosphere.io/viewLog.html?buildId=119430&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 220 Build time: 00:08:36


The repository contains several php scripts used to replicate, ie synchronize data from/to a MySQL database to a MSAccess (Jet) database.  The PHP code is called from a windows standalone application through a winsock connection.  The files in this repository are for purposes of archiving source code.


`SDL_GetKeyState` doesn't seem to be defined by Emscripten's version of SDL. This seems to be reported in https://github.com/kripken/emscripten/issues/545.


the `extern "C"` is so we can easily add the function to `EMTERPRETIFY_WHITELIST` without having to worry about C++ name mangling.


Octree (DeadParrot) - x86_64-MacOS-10.9-clang: OK (2062 of 2062 tests passed, 3 test warnings)

 * 3 tests had: AUD diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 1 test had: SHD diffs.

![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 the object part of the assignment, the actual project is further up the branch


`SDL_WaitEvent` isn't supported by emscripten's version of SDL because it'd block, so we'll use `SDL_PollEvent` instead and do [`emscripten_sleep()`](http://kripken.github.io/emscripten-site/docs/api_reference/emscripten.h.html#c.emscripten_sleep) in the loop.

In retrospect, I suppose a better abstraction might have been to just define our own implementation of `SDL_WaitEvent` that calls `SDL_PollEvent` in a loop with the sleeping. Would actually be nice if this was integrated into Emscripten upstream, so that Emterpreted functions could actually call `SDL_WaitEvent` without fuss.


Keeping this `lockScreen` here resulted in an assertion failure during `SDL_FillRect()` because Emscripten's SDL implementation expects the given surface to be unlocked. So we'll move it further down...


Why?


Also thanks to https://github.com/streltsoff


Couldn't find a better place to put this.


While Emscripten does currently have [experimental support for Pthreads](https://kripken.github.io/emscripten-site/docs/porting/pthreads.html), it only works on Firefox Nightly. So I tried just removing linking with the library to see if things still worked, and they do overall...

But, uh, this might also be why music isn't working. :disappointed:


Remove this from the docs.


Currently doing course (web-frameworks). First upload. 3rd week completed with assignment. [Date: 01/02/16 (MM/DD/YY)]


Remove this change.


see #4 for bug in this


I just saw this! :P Remind me to show you how I resize them


```javascript
var multi;

if(typeof client.batch === 'function') {
    multi = client.batch();
} else {
    multi = client.multi();
}
```


Remove as well.
But please test the module changes with an older version of node-redis please!


I ran into another error and haven't looked at it since. I am interested in this approach but don't have any native dependencies in current lambda projects pushing me to look further right this moment.


@sfmskywalker

There is already the static `HttpContextAccessor.IsBackgroundHttpContext(HttpContext httpContext)` method that do the same. Do you think it's useful to use it if we make it public and / or make an extension?

Best


Instead of this if else block you should create fullLein using [node.js path join function](https://nodejs.org/api/path.html#path_path_join_path1_path2). Note that I didn't use that on line 12. I didn't realize that this was an option until reviewing your code. It *should* handle the OS path separator differences and also whether it ends with a slash or not. I'd want to test that.


Bele :-)


This should be a CAPS_CASE constant defined at the top. leinExec should be set to that value on line 15.


Thanks!


@SchuylerMaclay No need to make class variables.


You just need to add `favoriteFood` to your `Sukanya` object


Glad to see you making a big.LITTLE version. Thank you.


@l12436 You'll want to add this to your fork.


operator< for Climb::Input is still not quite right.  It's possible for (a<b) and (b<a) to both be true.


Could I just put the cookie dough inside the object?


It would be good to have a comment here explaining what this code is doing and why it's necessary. I'm not exactly sure what problem you're solving with this at the moment. Based on the code here the comment would probably be "When leinPath is not just whitespace and it does not end with lein then it should append lein to the end of the path."


Nope.


pull requests for changes would be nice @mikeal, for maximum visibility


Don't merge in conflicts


Wasnt favorite food supposed to be its own seperate object?


mailto link I reckon


What do you mean?
If its a car than _repairable will include every hitpoint type exept the wheels. If than one of the other parts are damaged _fixable will be != nil. So the wheel hitpoints shouldn't matter at all.


for me, nothing at all happened


Yep. You just need to assign it to a property. An object can have another object referenced inside of it.


It is done. And I promise to remember it :)


Got it, thanks for the response! Yeah, I love this idea as a lambda package deployment option for my CD, but looks like the package needs some maintenance, it's otherwise quite nicely built! I've otherwise been trying to keep outside dependencies out of my lambda packages, but as my lambda strategy grows, it's hard to avoid. Right now, my dependencies are simple and don't require target compiling so I can do the icky stop-gap of committing node_modules. Anyway, thanks again, I'll see if I can hear anything back from the author, maybe I'll have to fork this myself.


Can you commit your latest changes so I can use the textures?


follow code style, should be ```} else {```


 #1 **done**
now i can create release and each release contain dist/ version of jQuery.


@tappleby So far I don't think there's anything I can squeeze out of this


@tappleby You'll want to look at three recently added tests in `createStore.spec.js` as there's some behavior that is hard to get right when optimizing


tested both the git copy and 3.6.21 beta of the skin and the git and 1.0.35 version of the helper script.

trying to change the view from the side blade no longer brings up the menu to choose and select views - so not sure if this is a helper script issue or skin issue.  might want to see if you have the same issue on your install.


Ah I see, I missed a line.. :)


@timaschew @wkrueger


It is done.


@priyagupta, code updated based on your advice.


111303866_ATSingleDuctReheat (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2089 of 2089 tests passed, 11 test warnings)

 * 549 tests had: AUD diffs.
 * 11 tests had: BND diffs.
 * 2 tests had: EIO diffs.
 * 2 tests had: ESO small diffs.
 * 8 tests had: MDD diffs.
 * 8 tests had: MTD diffs.
 * 8 tests had: RDD diffs.

![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)


oh and caught this in the log - just an FYI if you hadn't already caught that.

17:14:08 T:8764   ERROR: Missing operand
17:14:08 T:8764   ERROR: Error parsing boolean expression SubString(Container.FolderPath,pvr://recordings/active/) + !StringCompare(Container.FolderPath,pvr://recordings/active/) + !IsEmpty(ListItem.Art(thumb)) +


исправлено в следующем коммите


CNAME file?


The goal of this code is to write a program that can connect to SQL Server and automate reporting tasks from sources from within and outside of a server's databases i.e. Outlook messages and harddrive CSV.


111303866_ATSingleDuctReheat (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


This document will need to be updated in light of the SDK.


удален


Esse commit resolve o #7


No worries - thanks for the heads up.


Исправлено


@daveedgarcia @kesne

https://travis-ci.org/jquatier/eureka-js-client/jobs/106119219

:+1:


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


Hey @mattfysh, good ideas! I too had a feeling that this might be a bit awkward. Thanks for taking a look!

The purpose of the change is to explore the idea of introducing an image optimization step (I know there's a JIRA issue under FE infra somewhere), My idea was to use the custom transform function, which runs an optimizer based on the file type (using something like https://www.npmjs.com/package/imagemin) and then delegate to `processInline` to finish the task.


Hi, Why should we use public static?


The zooming and the glitch with when you only can spectate after spawning in the game are the only two things that needs to be fixed.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13771](https://teamcity.mesosphere.io/viewLog.html?buildId=119483&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:02:01


Wow guys, that's awesome! I want to try it out with something!


Roger, I broke it up into 5 commits — hope that helps!


Do links still work even when you explicitly define multiple networks? I rather like having links to explicitly define relationships here (and means it's easy to convert to a Tutum Stackfile!)


go to https://github.com/AJS-development/Ogar-unlimited/pull/34


Hey Patrick, I am still getting the error. I only get this error when I try to share authenticated user:
       view()->share('user', \Auth::user());

Here is the error:
  [InvalidArgumentException]
  Driver [doctrine] not supported.

Thanks.


ok


I would have preferred best review ever ;-)


okkkk


@sfmskywalker

That's what i've first proposed in my PR. This to prevent from an exception when base url is not set. But for infos, I also needed to add the `HostingEnvironment.ApplicationVirtualPath` (if a virtual directory is used as OrchardLocal). For more details see #6320.

Best


Why is there an extra file added here?


Leave a line note


@sfmskywalker

There is already the static `HttpContextAccessor.IsBackgroundHttpContext(HttpContext httpContext)` method that do the same. Do you think it's useful to use it if we make it public and / or make an extension?

Best


Sorry, It should be private.


Did you get those entries from stock sec_config file ?
Actually not so important (my question). I am just trying to understand the logic :P


Thanks for adding my module! :smile:


:clap: :clap: :clap: :clap: :clap: :clap: :clap:


111303866_ATSingleDuctReheat (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (957 of 957 tests passed, 0 test warnings)

![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-26.87%25-green.svg)


https://github.com/strongloop/loopback-component-passport/commit/4cf8153a7b11ace76bf75977fa9ca238cf8910ad#commitcomment-15789277Leave a line note


this is gone but still used around line 65, should that now be

```
function fontspec.mathfontdimen(fnt, str)
    local mathdimens = get_math_dimension(fnt, str)
    if mathdimens then
        tex.sprint(-2,mathdimens)
        tex.sprint(-2,"sp")
    else
        tex.sprint(-2,"0pt")
    end
end
```


Thanks for creating the module. Getting robust WebRTC working in Node remains elusive, with many believing Electron being the easiest way to get things working. Until native support, modules like `electron-webrtc` are quite valuable.


It is done on all new modules.


Is there a reason you kept the `errors if [front|back] is missing` tests?


all good


Is there a reason you kept these as separate vars instead of using a `BANK_ACCOUNT` object like `ADDRESS`?


fixed


Do we need to make `default` optional? Can we not load default env and then override any env variables passed through the command line?


second ...


@ekachandrasetiawan : can untuk syntax dapenin nomor dn di coba di pisah object nya

Karena biar bisa di inheritance di depan nanti,,

pakai method get_sequence_no()

![Image of activity](http://snag.gy/xmrGo.jpg)


For some tests (like addresses) you create and then retrieve the created one. But in others (like here) you just retrieve from a hard coded id. Was this intentional? Do you have a preference between to the two to keep them consistent?


@miguelcobain Meant to push this to my fork, but I missed up and accidentally pushed to yours. Let me know if you're not ok with these changes and I can revert.


Yeah that shouldn't be there.


It is done.


so as a developer without knowing liferay conventions ,  I would expect my project name as bnd symbolic name .


I believe this is a regression with regards to #28654.  Was this purposeful or was the parent revision of this commit not part of devel?


# three
`xx`
[www.baidu.com](url)


Maybe switch this to `states` or `countries` to preemptively avoid changing this in the future?


Maybe change this to `instanceof` to stay consistent


@joehuchette I'm worried this should be ``sqrt(2)`` not ``1/sqrt(2)``, could you sanity check?


I will do that.


What problem do you fix?


Don't merge this, change done in development branch.


fixed


SystemUI force closes if you turn it off and restart SystemUI


@votca/csg : Pushes now automatically update documentation on http://votca.github.io/doxygen/!


Hi


Fixed a minor warning.


`s/(through the BGE logic on this question is a bit strange)//` (does not help :-)


This is incorrect! It needs to create a new PlayerDataObject when none exists, not the other way around.


The way is similar as we do in [deep-insights](https://github.com/CartoDB/deep-insights.js/blob/master/grunt-tasks/scss.js#L12), compiling key scss stylesheet (as @matallo suggested).

@viddo @javierarce


I don't see any problem in having this here, can you please explain the issue?


Regarding callins -> there's no reason to have a callin for things that can only be changed from lua.


Would be useful to give the equation between `logic_step_max`, `physics_step_max` and `n`. That would make the use of these parameters much more meaningful, I suppose.


This is probably the worst thing, requiring cartodb.js#v4 assets from the deep-insights node modules :S.

@viddo @javierarce


I think can be just based on indexingThread.isActive() (or something like that...)


Kijken even wanneer het nodig is om @ te gebruiken en of het handig is om attr_accessor te gebruiken. ( zie http://stackoverflow.com/questions/14319347/variables-in-ruby-on-rails)


`s/phase/step (see pseudo-code above)/`


No changes in bootstrap.php are required as it doesn't have anything to do with log factory changes.
It also sets the AUTHORIZE_NET_LOG separately on line 12, so no null check required.


Hi K.Takata!

On Mo, 01 Feb 2016, K.Takata wrote:

> I think Ruby should be also listed here.

Opps, this was forgotten. I pushed a change. Thanks!

Best,
Christian
--
Wenn die Zukunft nicht mehr das ist, was sie mal war, heißt das,
daß die Vergangenheit sich ändern wird?


it probably should be `3.0.4`?


`s/the Logic Bricks of each component will make/each component [#]_ make/`
and:
```
.. [#] components actually invoke their `default_action` method via the Blender *logic bricks*
```


We can't rely on the `node_modules` tree structure, e.g. a dedupe (and newer version of npm by default) would flatten the tree structure to avoid duplicate code. We need some other way to access these files.


Please, could you move this test to a separate file (test/unit/common/runtime-bind.js)? Then, instead of accessing the runtimeBind via `Globalize._runtimeBind` we could access it directly (via `define('src/common/runtime-bind')`). Please just let me know if this isn't clear.


TeamCity pull requests :: netty [Build 1162](https://garage.netty.io/teamcity/viewLog.html?buildId=5277&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6402, ignored: 23 Build time: 00:43:29


`s/some calls to ``default_action`` can be skipped/MORSE can skip calls to ``default_action``/`


# markdown test


@renewooller, +1


Closes #3


Apanya yang semangat kaka


cool, but why deprecate this XD


Aah, ok - I think I've found it here:
http://help.eclipse.org/juno/index.jsp?topic=%2Forg.eclipse.platform.doc.isv%2Freference%2Fextension-points%2Forg_eclipse_core_expressions_definitions.html&anchor=e.count


`3.0.4.` is what you mean?


I ended up abandoning that idea and came up with:

![this](https://imgur.com/EjbxUEq.png)


Either this function or the caller should log about errors. I figure in this case, this function would be better.


Nitpick: we prefer the for (;;) syntax, see CODING_STYLE.


Solo traveling? wah berarti travelingnya pas waktu jones dong ya cu... :dancers:


@ktoso is the `HttpConnectionContext` type here a typo? It's causing Java calls to `.bind` to blow up if a java `HttpsConnectionContext` is provided (which, of course, can not be cast to `HttpConnectionContext`).


if thing you do in the loop is this check you might as well use a `while (d)` loop instead...


`s/by setting the frequency in the Game Logic Sensor or more conveniently specifying the desired frequency in the builder script (using :py:meth:morse.builder.abstractcomponent.AbstractComponent.frequency/by setting the a component-specific frequency with :py:meth:morse.builder.abstractcomponent.AbstractComponent.frequency./`


That's arguable.
You definitely want to be able to keep property-updating modules separate from property-change-listening ones, so you'll end up writing a observer-pattern infrastructure.

It's probably better to provide such an interface engine side because 1) it's less error prone & makes the engine better rather than being YetAnotherLuaIt most people won't right 2) unifies engine & lua changes (notice how mass *can* change "from the engine" - although it happens indirectly through transports)


## annotation


hahaha..lupaa dwi..kayanya sih gk jones, wwkwkwkkw


comment


this error should be caught and handled.


habe linux und eig auch getestet ob der build noch funktioniert, wird das kompilieren manchmal übersprungen?


Not sure to understand here...
Do you mean:
`The actual frequency of the component can be read back using the service ...`? Otherwise, I'm not sure what you are `computing`...


I think each device should also be checked for a device node, and the appropriate match be created.


test push


Yes, thank you.


eigentlich nicht. seltsam...


Container interface chould extend Closable interface.


should it be? if($this->logger $this->logger->getLogFile())


As I remember, there were some reasons to name it StreamProjection?


Hey @flixr,
could you explain this one to me so I don't make this mistake again? I thought you had to be either fixedwing or rotorcraft and the shared folder was for common files. It's not so obvious to me why you need to separate these as you did here.


Spise nez posilat values v eventu, tak by bylo lepsi tam mit 2way data binding, aby pak sly snadno pridat validace. Nejlepe pouzit Template driven form. Viz tady: https://dl.dropboxusercontent.com/u/738990/Books/Become_a_Ninja_with_Angular2.html#_template_driven


s/expected_table/expected_tables/
s/availble/available/


Remove Closable interface see comment below.


yes.
Since the installed root filesystem has different policy the the running one . Then we get error code.


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


nfufufu


should be Wisembly BTW


You're welcome. I overlooked this one: A function that


it's absolutely not less error prone! code is code, and making your own lua callins (you can easily wrap SetUnitMass for instance and call whatever you need) is just as buggy as engine code and considerably more flexible (maybe we don't want to call it for the transporters?).


Would be nice to say in 5 words what happen if you disable vsync -> `s/possible to disable v-sync using/possible to disable v-sync (this lets MORSE reach higher frequencies, assuming your hardware permits it) using/`


Oh, indeed. I guess I confused it with repository names which I believe can contain spaces (in pacman). All good then.


What is the license of the software you copied `gluErrorString` from?


Mention that vsync is disabled by default


+ how to add annotation
+ please tell


When this call fails this should result in abortion.


@computermacgyver thank you so much for help!


Jak przejdzie będzie koljny commit, bo {$event.customerphone} jest NULL do usunięcia.


I was told to

>Ignore odd-numbered minor versions of atk, gdk-pixbuf, glib and gtk because these are development releases.

Is this true of pango as well then?


@autotune Hi, am working with the ec2_vpc module and I have narrowed down a [bug](http://stackoverflow.com/questions/35125292/cant-figure-out-why-subnet-is-being-updated) to this code. Any chance this could be rolled back or fixed? I will try help figure it out, but I am no guru in python. Just looking at the code it looks like some of the stuff should be extracted into methods or even it's own class.


The above mentioned error is fixed here https://github.com/practiceprovider/openemr/commit/72df047245f3c2ce9cec5e8d6b2c09254ebe55fd and some more deprecated constructors fixes as well


You forgot adding sd_journal_has_runtime_files.xml?


Pacman does integrity checks (using GPG) on packages by default, before installing them. This takes a really long time, disabling it speeds up the experiment. It's especially annoying because before building foo, we need to do integrity checks on all transitive deps of foo, even if we've already done them in a different container when building a different package---so lots of redundant work.

Shall I elaborate on this in comments?


Let's move this code into its own file (e.g., `src/util/function/name.js`), so here we have only:

```diff
 var argsStr = JSON.stringify( args ),
-		fnName = fn.name || /^function\s+([\w\$]+)\s*\(/.exec( fn.toString() )[ 1 ], /* 1 */
+		fnName = functionName( fn )
   		locale = cldr.locale;

+	// If name of the function is not available, this is most likely due uglification,
+	// which most likely means we are in production, and runtimeBind here is not necessary.
+	if ( !fnName ) {
+		return fn;
+	}
```

PS: It will be easier in the future to revert `functionName( fn )` back to `fn.name` when we don't need the special handling for IE.


Yup, odd numbers are for development


@Kireti ich dachte in der Aufgabe sollen "keine vordefinierten Tupel" verwendet werden? Ich weiss aber selbst nicht genau was jetzt damit gemeint ist.


Hi Stefan,
Something must have changed on Google side. In my opinion if it was a bug in the lib code it would manifest itself from the start.


Please add variable types. For example: `@param {String} path`


About the code in src/util/function/name.js, let's handle it for IE only if `fn.name === undefined`


No problem. Thank you for making this plugin. It looks very useful, and I'm excited to see it added to Gephi.


@aofall wtf, 我记得证书是 google.com ，不管了，吃了饭再说、、、

ps: 前天扫了 Google IP，睡一觉起来发现基本全挂了。没有动力更新了。


Fixed a minor warning.


@loverajoel I'm sorry for this commit. I just want to push last commit whitch translated some tips. I'm a new guy in git. Hope it doesn't cause troubles. :disappointed:


No line break between function prototype and opening bracket, please. See CODING_STYLE...


Shit, hatte ich nicht gesehen
On Feb 1, 2016 11:17 AM, "Karim Abou Zeid" <notifications@github.com> wrote:

> @Kireti <https://github.com/Kireti> ich dachte in der Aufgabe sollen
> "keine vordefinierten Tupel" verwendet werden? Ich weiss aber selbst nicht
> genau was jetzt damit gemeint ist.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Kireti/RWTHUebungen/commit/601a028c7bd45624aabe3098049a4c3b38e90703#commitcomment-15795155>
> .
>


I see people asking themselves the question: what if I want one sensor to run at higher frequency?
-> the way the documentation presents time management, it seems that the logical workflow is:

- set a global simulation frequency (using `simulator_frequency`) 'as high as possible'
- set (lower) frequencies for each of the sensors

while --if I'm right-- the easier/better workflow is really:
- set the frequencies of each of the component to what they need
- let MORSE manage its own 'main' frequency.

Said that way, note that it would be really useful to have a global `env.set_default_component_frequency` (as you suggested at some point) instead of randomly hardcoding it to 60FPS...


Of course! ;-)


Did'nt we agree to use 2moves book for regression tests?


here you can specify a min and a max port just to enure you get a valid port 0, 1<<16


thanks again


Not that it would matter much, but wouldn't it be more obvious if we'd build the thing strictly from front to back, i.e. move the line above before the construction loop?


efecan yazdım


This label should read "delete" if `archive_assets` isn't enabled


null is not supported - you can use `REGION_SETTING.exists` if you wanna check if it was set?


Ah yes, we still want sourcemaps here in development, and for babel as well. The css whitespace plugin however does not support sourcemaps. SugarSS will when it comes out!


Ah ok! Thanks.


If I read this correctly, this is going to break the integration for all existing installations that have `Setting.sys_api_enabled?` set to `true`, right?

That is unfortunate, but at least this will only break for users who at one point has decided they want that extra security. Breaking the integration to give them that seems like a decent tradeoff.


Does this change not require any changes in the test suite?


is this interface needed?


(I see now, that my code already did it like this... But still, let's make this more obvious while we are at it...)


1) It is more error prone because it's possible to go around the wrapped version of SetUnitMass by invoking SetUnitMass directly (and hence the notification wouldn't fire).
2) It makes it much harder to create compatible Lua code as different modules could use different wrappers.
3) It's hard (impossible?) to have gadgets change properties it & widgets be notified on the change.

I don't see how it's more flexible. Property change notification system seems very clear in its scope and is pretty basic (you just get notifications for the properties & units that you've registered interest for), and you're free to create arbitrary Lua code to handle it. This is similar in how unit position change (UnitMoved?) notifications are handled.


`check_enabled` seems too generic; how about something like `verify_sys_api_key`?


The defconfig filename starts with a capital letter? (Never seen that before.)


default should be Collections.emptyList()


Is this responsibility for the package?


you need to set r from errno on failure here...


Thanks.
It should be fixed with 68c928a


:trollface:


To be more clear: [[Page]] in supposed to lead to a page named "Page" in the current space. There is absolutely no reason the logic suddenly change and if the terminal page does not exist then you should get a nested page in the exact same place: <currentspace>.Page.WebHome.


Add a line:
`synchronize_with_external_clock # optional`


I'd rewrite this as:

    auto mode = QString::fromStdString(arguments.at(2)->GetStringValue());
    if (mode.empty()) {
        mode = QStringLiteral("W");
    }
    QFile::OpenMode modeCode = mode.compare(QLatin1String("W"), Qt::CaseInsensitive) == 0 ? QFile::WriteOnly : QFile::Append;


nowadays we generally try to initialize fields like this already when we declare the variable. i.e. above, put `.un.sun_family = AF_UNIX` between the {} initialization of the sockaddr union... Also see CODING_STYLE.


Yup.


Also there was `Linksprite_pcDuino3_Nano_defconfig` previously as well :)


Is this line redundant now?


put on stack:

    QFile f(QString::fromStdString(filename.ToString()));
    if (!f.open(...)) {
        qDebug() << ...;
        return false;
    }
    const File fs(f);
    fs.write(...);
    return true; // should auto-close fs


asuna dafuq


Ändere das noch schnell ab, sollte kein Problem sein

Am 1. Februar 2016 um 11:20 schrieb Alexander Tegtmeier <
alexandertegtmeier@gmail.com>:

> Shit, hatte ich nicht gesehen
> On Feb 1, 2016 11:17 AM, "Karim Abou Zeid" <notifications@github.com>
> wrote:
>
>> @Kireti <https://github.com/Kireti> ich dachte in der Aufgabe sollen
>> "keine vordefinierten Tupel" verwendet werden? Ich weiss aber selbst nicht
>> genau was jetzt damit gemeint ist.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Kireti/RWTHUebungen/commit/601a028c7bd45624aabe3098049a4c3b38e90703#commitcomment-15795155>
>> .
>>
>


I think that's rather:
```
self.exec_path = kwargs.get("exec_path", None)
```


put onto stack


you are leaking it here otherwise


also put onto stack


No need for exec_path in this file IMHO


stack


Will this 'mcbc/2016-01/' entity Id continue to work past Jan 2016? I'll have a poke and see if I can prove one way or another now.


@bradymiller After running code inspector I found errors around 750

Mostly errors are related to

1. Deprecated methods (split, ereg, magic_quotes etc)
2. Dynamic method called statically
3. Classes with names int, string, float, and bool are forbidden in PHP 7

Let me know if I should start fixing these issues?


IMO the generated stuff should not be in the `src` directory, but in a let's say `generated` directory. Then the autoloader should load both the `src` directory and the `generated` directory


This still works:
```
[root@dt04 vm-pre-start]# xe vm-shutdown vm=dlvm
The request was asynchronously cancelled.
task: OpaqueRef:050e3b0c-3dfa-d43a-ad67-ebb5c733ebf9
```

This is because this is handled at a higher level. A hard_reboot will cancel the Xapi task (and associated) Xenopsd task from the message forwarding layer. The clean shutdown will raise the canceled exception.


`line` is 0-based, you want `.line !== undefined` (I've had exactly the same bug).


good catch, fixed now, ty


Thanks @tuffnatty - PR is here, if you test it? https://github.com/ckan/ckan/pull/2861


Looks great otherwise!


missing license header, esp. important if you copied it from elsewhere


also misses a license header


Technically we should.  Otherwise people might think that sclk etc can be specified using positional args, which they can't.


олд


here you are opiniated on using the `guzzle` factory by default when you're not sure it exists AND not sure of the version. This kind of ruins the interest of using a client abstraction IMO


I suggest you make this a direct member

    QTextStream m_fileStream;

saves one runtime allocation whenever this object is created, and also fixes the memory leak that currently exists here.


unused?


Great :D


Ah, see that you changed it to ```.findItemByEntityId('mcbc/' + currentMonth + '/' + a.entityId);``` later. All good.


You should update your README file. See the link below: https://github.com/makersacademy/takeaway-challenge/blob/master/docs/review.md#readme-updated


Confused me a bit to have this static on the extension, but the config value is read from the owner record. Not sure how to solve this though...


I think this is complex enough now so that we should maybe add a tiny static function close-by that does the mapping in a switch() statement, and is just used here... i.e. `const char *job_type_to_selinux_permission(JobType job_type)` or so...


After this commit the build fails here because the sdcc is unable to find ../../Kernel/include/userstructs.h [1]

This happens because /opt/fcc/include/syscalls.h tries to include that new file, but the . directory is not in the header search path, and thus the compilation fails.

One way to fix the build would be to simply add . to the header search path [2].

[1] http://dpaste.com/1WXY6P9
[2] http://dpaste.com/1THQ209


Great minds think alike... ;-) I see that you next patch pretty much does that...


This is just a default implem, you could pass your fatcotry to the constructor.


Nitpick: I think this would be a very good candidate for a switch statement...


Виглядить ок, хоча для більш кращого рев'ю мені потрібно іще підучити Rails


definitely this whole method should be refactored/split to smaller parts


Yeah sure - I can update them to `#!/usr/bin/env python` if that will be better?


Aha... OK... this has been missed off elsewhere in the docs. Good spot. I'll go through them this evening and make sure they're all described properly.


This is revolutionary coding. :ok_hand: :100:


Which would then again discourage liquidity providers that use bots IMHO. Is spamming the order books really much of an issue? There is nothing to be gained by it. It's a different thing for operations which you can use to put data on the chain, but I don't see this for order_creation operations


I just took the definition of the GL_ERRORS in hte openGL doc.

2016-02-01 11:14 GMT+01:00 Laurent Rineau <notifications@github.com>:

> What is the license of the software you copied gluErrorString from?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/maxGimeno/cgal/commit/9abf38bfcede979120028c09e8b2274989231192#commitcomment-15795058>
> .
>


Looks good. Actually I have already sent my own PR #2860 (tested). But your solution seems to be a little bit cleaner. Sorry I can't test it at the moment, need to deploy a whole new testing environment.


This method is used in GCD storage implementation ([DsEntityStorage.java](https://github.com/SpineEventEngine/gae-java/blob/49337f00023eb35c9884ab5ba80ac4d77a15ba1c/gcd/src/main/java/org/spine3/server/storage/datastore/DsEntityStorage.java)).


I see your point. I am thinking that we might need to distinguish liquid markets from non-liquid markets to prevent spamming in non-used markets and enable cheap trading in liquid ones


I am pretty sure we should reset the field if isempty(optarg) holds. Also, to indicate that the memory isn't owned by us anymore (because it's semantically part of argv[]), the arg_idenfitier field should become const now...

(But note that as mentioned I already have a fix queued for this, that does this..)


See below.


oh really? ;)


Based on a comment made below, I would start with a first paragraph describing the most common use-case (and recommended workflow):

---

Recommended workflow
--------------------------------

Time management in simulation is a complex matter: you may want to simulate different sensors at different speeds, you may want to run faster-than-real-time simulations, you may want to synchronize the simulator with an external time reference, etc.

MORSE tries hard to make *easy things easy, and complex scenarios possible*.

Consider the following simple example:
```python
from morse.builder import *

# Append ATRV robot to the scene
robot = ATRV()

# Append an actuator
motion = MotionVW()
robot.append(motion)

# Append a sensor
pose = Pose()
pose.translate(z = 0.75)
pose.frequency(200)
robot.append(pose)

# Configure the robot on the 'socket' interface
robot.add_default_interface('socket')

env = Environment('indoors-1/indoor-1')
```

This is the typical scenario: we tell MORSE that the pose sensor should output values at 200Hz, and we let MORSE manage all other time-related questions. In this case, MORSE will run its main loop at 200Hz (hardware permitting! otherwise, MORSE will warn you that the desired frequency can not be reached), and will attempt to update the physics at real-time speed. Note that we did not have to specify a frequency for our motion actuator: by default, all MORSE components run at 60Hz.

Now, imagine you want to accelerate your simulation. Just add the following lines to your builder script:

```python
# [...]
env = Environment('indoors-1/indoor-1')
env.set_time_scale(1.5)
```

Now, MORSE will attempt (again, hardware permitting!) to run the simulation at x1.5 real-time. The pose sensor, for instance, will produce reading at 1.5 X 200 = 300Hz. The physics engine will accordingly run faster, etc.

---

...we may need even more of these user-oriented examples!


(off topic) have you seen your domain openspa.info doesn't work any more?


Thank you :+1:


indent


fixes #1


Can filled order really pay a 'flat fee'? Or are you mixing this up with a percentage market fee as defined by the issuer of an asset (which totally makes sense as described in the introduction).
I am not sure it makes sense to prevent spam at this point in time where we barely see 100 transactions a day.


Yeah I think it would be better.


Great, thanks!


Each for different entity type - Customer And CustomerSetting.


變數名稱不要跟method name一樣


it is part of the execute_logic (one of the first step in fact). Blender does not have any notion of clock sync, even with the last patch.


The fee pool is a feature that enables comfort to your customers. Shouldn't we ask for a fee then?


It is far more simple to rely on DMA APIs for the non-scattered allocation instead of trying to reinvent the wheel. So I unified as much as I think it is reasonable but at this point trying to bring in more code that is already well cooked elsewhere looks to me like a waste of my time.


Why doing this in two steps? Calling the service twice?


@maartenkling sorry that's jenkins automatically reporting (and by accident being me the one impersonating it).

No need to worry about it at all.


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 140](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=734&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


@gyst  sorry that's jenkins automatically reporting (and by accident being me the one impersonating it).

No need to worry about it at all. Sorry for the noise.


I wasn't suggesting to drop using DMA APIs, just to also fill in struct pages so that there aren't too many branches.


You also need to create the attribute in the constructor.


I think you need to fix:
 https://github.com/jirikuncar/invenio-oaiserver/blob/restart/tests%2Ftest_response.py#L44
 https://github.com/jirikuncar/invenio-oaiserver/blob/restart/tests%2Ftest_response.py#L92


Don't return it at all for now


RDP needs not to be capitalized in order to be correctly interpreted by Guacamole.


Why is this project deprecated?


Those are completely different flows. We cannot merge them. Really.


I suggest using ::class syntax


You should also delete the corresponding `<img>`-tag from `index.html`.


Please use the standard copyright text used in Trusted Firmware:

     Copyright (c) XXXX, ARM Ltd and Contributors. All rights reserved.

Linaro is already a recognized contributor, and you will be contributing on behalf of them.


The "warning" I mention in the commit message was not referring to our formal definition of "warning". Sorry. Info was correct. This message is shown when the import profile sees something unusual that the user probably didn't intend to do (load an empty profile), but the feature will keep on working fine for the rest.


By adding more methods on IdGeneratorFactory?


But you don't include that in the query!


is this commetn about the preceding code or following code?


ok j'ai corrigé dans le code plus haut plutôt que dans la boucle


Ok, let me find a better solution.


Well, that is not really true. It is disabled automatically if you let the simulator computes the time setting, but the true default setting is enabled (otherwise it breaks lot of test in FixedTimeStrategies mode).


Done! I created a pull-request with my commit.


It's not retrieval, but transmission of data we've got from Google API. Nevermind about that. Still, we need to validate it on back-end before usage.


It's better to add "Ms" suffix instead of comment.


This is what we actually **have to** remove :)


extra new line


Never heard of this before, but it seems like a good thing to add.


There's a thing with MetaDataStore, but perhaps it can override the ` getNumberOfReservedLowIds` method as well. Although it's different there.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13798](https://teamcity.mesosphere.io/viewLog.html?buildId=119685&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 220 Build time: 00:08:25


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13795](https://teamcity.mesosphere.io/viewLog.html?buildId=119686&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 220 Build time: 00:08:25


don't commit commented code


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13797](https://teamcity.mesosphere.io/viewLog.html?buildId=119688&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 220 Build time: 00:08:45


check [decimal decimal] + long, or [double double] + bigint


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13796](https://teamcity.mesosphere.io/viewLog.html?buildId=119687&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 220 Build time: 00:08:48


> As for your last point, I believe modern engines optimize array.length access in the loop anyway and hoisting that call doesn't make it faster.

Not sure about that.

```js
var index,
    test;

test = {};

Object.defineProperty(test, 'count', {
    get () {
        console.log('A');

        return 100;
    }
})

for (index = 0; index < test.count; index++) {
    console.log('B');
}

```

https://jsfiddle.net/qw1y8g3y/

`test.count` is access on each iteration, the same way that `test.length` would. That said, I could not confirm this using JSPerf, http://jsperf.com/quick-for-vs-foreach.


@adamcheasley according to the style guide we prefer single quotes over double quotes whenever possible, so a flake8 plugin checks for it.


Because the method is named as "hastimer" and not "hasactivetimer", but it's no problem to keep the active check here? But then we have 1 method for enabled, and one method for disabled timers.


Same as above.


> The actual frequency of the component can be read back using the service ...? Otherwise, I'm not sure what you are computing...

Not a service, a property. It is an information for sensor / actuator developers (hence the page is in dev part)


But closing a RecordCursor is about closing the page cursor.


This introduces a bug. The text of the legend is now centred by default

```r
library("ggplot2")

d <- data.frame(x=1:3, y=1:3, foo=c("a", "b", "c"))
qplot(x, y, colour=a, data=d)

d <- data.frame(x=1:3, y=1:3, foo=c("a", "bbbb", "c"))
qplot(x, y, colour=foo, data=d)
```

but not when the text is interpreted from the call directly

```r
qplot(x=1:3, y=1:3, colour=factor(1:3))
```


I remember talking about this and deciding that we didn't like this, preferring normal case for names (at least).


See other note about this, don't want to go back to that


@adamcheasley the list of flake8 plugins used is at https://github.com/plone/buildout.coredev/blob/5.0/experimental/qa.cfg


Hmmm, did I get this right? This would set the the NO_NEW_PRIVS bit in all cases where User= is used? i.e. no services with User= set could ever use suid binaries anymore? THis should check if seccomp is actually set, too, no?


wrong name. should be kEmptyLogQueueNotifierTimeout. add Ms too.


seems as an issue on the code.


line note


Fedora doesn't know a "nobody" user, but only one called "nfsnobody"...


bye


@vojtechkubin :+1:


@android10 could you explain why did you decide to change the initialization for fragment?


Is this commit merged already?


Please follow the TF coding guidelines regarding use of data types:
https://github.com/ARM-software/arm-trusted-firmware/wiki/Trusted-Firmware-Coding-Guidelines#use-of-built-in-c-and-stdlib-data-types

In this case, you should use `unsigned int` instead of `uin32_t`. I know that other TF code does not obey these guidelines but we haven't got round to applying remedial action on existing code yet!


I think this is now better, right?


I don't see anything wrong with the code at the first sight (aside from a few PEP infringements).

Writing a test suite for the pruning part would probably give you the certainty you lack. Can't help you there, though - the one who writes the code knows best what needs be asserted (and when).


bør vi ikke også kalle inPrefferedLanguage på partTitle?


Well !Active returns true if the state is "PVR_TIMER_STATE_DISABLED" as well with "PVR_TIMER_STATE_ABORTED" and "PVR_TIMER_STATE_CANCELLED". This is intended as we don not want to show aborted and cancelled timers as active in the GUI. I could rename the method "hastimerdisabled" to "hastimerinactive" or sush?


@cobatake - I'm not sure if I mentioned this to you before, but you can use markdown on the files in the website instead of HTML. Here's a good example.

https://daringfireball.net/projects/markdown/

@aobatake - Might be good to take note of this as well.


Agreed :-)


this by the way does not impact the tests results but simply the build process, then obviously tests are performed on the browsers so they are not affected by this configuration.


This is not exactly increasing interval approach, on the other side, I think it should work and it is much better approach than the original one.


Why is this?


Jo.


Think you meant `../src/cli`?


и переименовать с правильным префиксом)


@rtyler the image does not fit into my browser
<img width="1082" alt="screen shot 2016-02-01 at 11 56 33" src="https://cloud.githubusercontent.com/assets/3000480/12715479/de6f1dc8-c8da-11e5-8a02-564494a65834.png">


> I'd really consider landing the HTML generation bit in a separate pull request, this one is already large enough.

This PR doesn't contain HTML stuff, I've removed that pull request since I'm completely rewriting it. I'm splitting the HTML pull request in two: post-processing (to 'decorate' the JSON files with our analysis) and HTML generation, since these two were mixed up before.

> One other comment, after seeing the code, it might make sense to turn some of the files like toolchains/android/setup.pyor toolchains/vanilla/setup.py into shell scripts

They are ugly, but I have them as python scripts because I want to use the same log() function that all the rest of the code uses (so that we capture the output of the setup also). It's useful for debugging, especially when we add a new toolchain and need to ensure that the setup is working correctly. I'll refactor this later, it will involve turning log() into a separate executable that can be run from the shell script


WTF... Ok, monday, sorry...


1) You have to trust lua coders to not make silly bugs (while preventing them from hitting non-obvious obscure bugs), otherwise there's no point in having an engine at all.
2) The gadget proliferation era of nearly-similar *A games is no longer here. You must read and adapt code before putting it in your game.
3) When necessary it's possible, and it's absolutely unnecessary to call a widget callin for every changed attribute.

Registering for stuff isn't really working, since usually you find yourself registering for 90% of units/projectiles in the game.

UnitMoved isn't sent to lua for a reason.


docker-compose run --rm  web `command`
Is easier to use as the docker exec and no running container is needed.


See https://github.com/simonjbeaumont/xen-api/commit/1e750dc8720a76e4248d2ec96fb8617c8e080d3b


See https://github.com/xapi-project/xen-api/commit/f57b4de52f3dada4d5b51cbf0535a156da0383e0


@oleg-nenashev I wonder if this can be easily fixed with CSS; that would be better than hardcoding the width, as the new website design should have more horizontal space (I think).


no currently not. Good idea to add one


:tada:


Add lite to the title param too in this link.


Même après le reshape ce n'est pas X ?

-----Message d'origine-----
De : "chiptuned" <notifications@github.com>
Envoyé : 01/02/2016 10:50
À : "OsbourneDurden/WannaKnowMySecretIdentity" <WannaKnowMySecretIdentity@noreply.github.com>
Cc : "OsbourneDurden" <sarembaud.nicolas@gmail.com>
Objet : Re: [WannaKnowMySecretIdentity] Indentation et calcul de A modifié (db0cb4c)

X n'est pas le X de AX=0, je l'avais appelé V car c'est son nom, qui a du sens :
https://en.wikipedia.org/wiki/Singular_value_decomposition
http://fr.mathworks.com/help/matlab/ref/svd.html
—
Reply to this email directly or view it on GitHub.


Тут может выбрать другой термин (из другого словаря). Я для этого и делал проверку на принадлежность термина словарю в цикле.


@orrc agreed, CSS would be better


我不是說了先檢查`a.BelongedFaction != null`...


I'd guess I prefer it, reads better and is more consistent.


Fixes OctopusDeploy/Issues#2280


Ah yes. Fixed!


It is not possible to have a local group and a local user using the same name (https://technet.microsoft.com/en-us/library/cc783323(v=ws.10).aspx)


done


Yup, the dense threshold is the store header for that store. It's in the vein of removing specific methods and functionality from store implementations to one day only have RecordStore references.


TeamCity pull requests :: netty [Build 1163](https://garage.netty.io/teamcity/viewLog.html?buildId=5278&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4911, ignored: 23 Build time: 00:50:51


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


see #24 .
currently the binaries of Z3 (including the part with Java and JNI) are missing,
thus compilation and testing is not possible.


FIXED


>>You either need to adjust naming if your logic is correct, to not conflict with the definition of PVR_TIMER_STATE_DISABLED

> I could rename the method "hastimerdisabled" to "hastimerinactive" or sush?

"istimerinactive" and LISTITEM_ISTIMERINACTIVE should be fine. (not "has", please")


seems like something wrong with tests or code :(

Failed tests:   testMap(org.red5.io.AMFIOTest): expected:<34.0> but was:<34>
  testList(org.red5.io.AMFIOTest): expected:<1.0> but was:<1>

Tests in error:
  testInteger(org.red5.io.AMFIOTest): java.lang.Double cannot be cast to java.lang.Integer
  testNegativeInteger(org.red5.io.AMFIOTest): java.lang.Double cannot be cast to java.lang.Integer


I don't get this, `Defaults` is a static List with 5 items, so why would this ever be empty?


@jccazeaux thanks! Fixed


This says getRandomHost yet the hosts are always retrieved in order of their index, which based on the static `List`, will never be random?


I've repaired my build process and established that these css changes are in effect. I'm not able to see how min-height has any effect. I tested on chrome and firefox. I've tested by viewing:

http://ward.bay.wiki.org/about-cypher-plugin.html

Which is both short and has a green halo. I also don't see any negative effect from specifying the min-height so I'm ok with merging this commit. Sorry to not be more helpful.


Makes sense, but please doublecheck that this change of semantics does not have any bad side effects. You in fact have to check all places where LISTITEM_HASTIMER and LISTITEM_HASTIMERSCHEDULE is used!


Yeah I know. But at least the travis job runs under the LTS version of node, which will be supported longer than v0.12. Also v4.2 is quite a bit faster (without code coverage).


確認しました。特にコメントはありません。


good idea, just done


actually ..overloading


Shuldn't we continue to use fixed version?


@foi thank you! :)


sadface :cry:


I think this logic is wrong as index 0 will only ever be used once because once `indexCurrent = (Defaults.FALLBACK_HOSTS.size() - 1)`, it is then set to 0 in line 403, and then set to 1 in this return statement.

This would be a lot easier as simply:

```java
int indexCurrent = Defaults.FALLBACK_HOSTS.indexOf(activeHost);
return Defaults.FALLBACK_HOSTS.get((indexCurrent + 1) % Defaults.FALLBACK_HOSTS.size());
```


Yes, this was a work around, i removed it.


Also, we should name this getRandomFallbackHost() for clarity


i saw the post on XDA!


sure sure, the comment was just to clarify this to possible readers :)


1) These aren't silly bugs: if you commit them, your changes would still happen, but you won't get any feedback in some other place. It won't be easy to detect & debug.
2) This doesn't imply game similarity. I absolutely disagree that we should aim for no code sharing. Libraries/frameworks are the future, and the only way to make new projects doable with small teams. It's the copy-paste-hack GUI widgets & concrete game mechanic gadgets that should die out.
3) It's not reasonable to use Lua gadget(synced)->gadget(unsynced)->widget communication just to be able to handle property changes. Also, I don't see why you would want to handle changes for **every** attribute. You would usually only care for a select few. The point is to register both the unitID/featureID and the propertyName, e.g. Spring.RegisterPropertyListener(unitID, "mass"), and handle it as a callin: OnPropertyChanged(unitID, propertyName, value)


Forth doesn't need an inner interpreter to be a real Forth.  Many modern Forths compile directly to optimised machine code.


Apres le reshape c'est carrément la matrice de projection. X c'est la dernière colonne de V(avant ton commit). Je rectifierai

-----Message d'origine-----
De : "OsbourneDurden" <notifications@github.com>
Envoyé : 01/02/2016 12:00
À : "OsbourneDurden/WannaKnowMySecretIdentity" <WannaKnowMySecretIdentity@noreply.github.com>
Cc : "chiptuned" <pro@vincent.maison>
Objet : Re: [WannaKnowMySecretIdentity] Indentation et calcul de A modifié (db0cb4c)

Même après le reshape ce n'est pas X ?

-----Message d'origine-----
De : "chiptuned" <notifications@github.com>
Envoyé : 01/02/2016 10:50
À : "OsbourneDurden/WannaKnowMySecretIdentity" <WannaKnowMySecretIdentity@noreply.github.com>
Cc : "OsbourneDurden" <sarembaud.nicolas@gmail.com>
Objet : Re: [WannaKnowMySecretIdentity] Indentation et calcul de A modifié (db0cb4c)

X n'est pas le X de AX=0, je l'avais appelé V car c'est son nom, qui a du sens :
https://en.wikipedia.org/wiki/Singular_value_decomposition
http://fr.mathworks.com/help/matlab/ref/svd.html
—
Reply to this email directly or view it on GitHub.
—
Reply to this email directly or view it on GitHub.


you're right, I'll fix it


Yeah, looks better now!


Resolves #21


Done, see #33


Good point.. it does seem to be defunct now.


k.


yea

Date: Mon, 1 Feb 2016 02:54:46 -0800
From: notifications@github.com
To: node_starter@noreply.github.com
CC: panjan@panjan.cz
Subject: Re: [node_starter] feat: waiting for polite shutdown (0d5eab8)

This is not exactly increasing interval approach, on the other side, I think it should work and it is much better approach than the original one.

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


I don't actually agree with @paddybyers that this functionality is right, it's not in the spec anywhere. If the primary host fails for an HTTP request, we try the fallback hosts up until the max retry count.  However, for all subsequent HTTP requests, we don't necessarily default to the last successful fallback host for a number of reasons:

* Our latency based routing will correct itself after around 1-2 minutes, so `rest.ably.io` may go down briefly if your closest data centre is unhealthy, but this is a transient issue and will be fixed within a minute or two by DNS if the data centre remains unhealthy.  So within 2 minutes, you can expect `rest.ably.io` to route to your next nearest data centre.
* If one REST request failed because of a network issue for example, this logic would then result in every subsequent request being routed to a data centre much further away for as long as the library is instanced.

@paddybyers, do you agree that this is wrong and not according to the spec?


When will `Defaults.FALLBACK_HOSTS.isEmpty()` every be empty?


Oh, perfect!  Now that you mention it, I remember reading about 2.0 exactly what you said - they want to break out list functions and make them their own dedicated modules.

Very cool.  I'm checking out your code as we speak.


V8 (and other engines too) are smarter than we give them credit for.
http://mrale.ph/blog/2014/12/24/array-length-caching.html


hm jeg kan ikke se at denne funksjonen funker

Så da er det vel en god ide som du sa - å putte den i eget bibliotek, med unitester


much github
so style


*Should be 'gulp build task'


Why is this needed?  If you see http://docs.ably.io/client-lib-development-guide/features/#RSC15b, this fallback behaviour only applies to requests to `rest.ably.io`, whereas all tests run against `sandbox-rest.ably.io`.  So this is unnecessary throughout the test suite, and http://docs.ably.io/client-lib-development-guide/features/#RSC15b needs to be added to the client library


Is there a reason this is commented out instead of being removed?


By "installation", I mean both the initial installation *and* any dependencies listed in a project's `project.json` file for dependencies as seen in https://github.com/jumanjihouse/docker-dotnet/blob/master/sample/project.json

Perhaps this will change in the future as dotnet on linux matures and/or as "we" (the linux community) get more understanding of how mono and coreclr work.

Another option to explore is 2 images: one for dev and one for runtime-only. https://github.com/jumanjihouse/docker-dotnet/issues/9


Same for this one.


isn't it better to rely on bundle.getBundleContext().getDataFile(""). That way you make less assumptions and you are closer to the code which is used inside the fs classloader itself (https://github.com/apache/sling/blob/trunk/bundles/commons/fsclassloader/src/main/java/org/apache/sling/commons/fsclassloader/impl/FSClassLoaderProvider.java#L88)


This list should be randomised


I like this but I don't seen where the `url` field is being used a a unique key


Certainly mod operator works better in this case


Other than the couple of comments looks good to me - although I'm not a Gtk expert.


Тут остатки от SparseArray. Цикл можно переделать на for each.


awkwaaaaard....


To avoid term clash with traditional projection from RDBMS. But the prefix does more harm, and we can avoid the naming clash by redefining the 'old' projection on our side.


for each


Whitespace is wrong here.  Also, not that comfortable with the mix of cases here, i.e. upper case `HTTP_MAXRETRYCOUNT`, yet camelCase for connectTimeout.  Also, if we do go with upper case, why is this `HTTP_MAXRETRYCOUNT` and not `HTTP_MAX_RETRY_COUNT`?


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 140](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=734&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **SUCCESS**
Summary: Running Build time: 00:41:32


I ended up working around this in conda-build-all (https://github.com/SciTools/conda-build-all/pull/20).

The problem remains in conda-build, but users of conda-build-all will not suffer for it.


imo it's an insane thing to implement such a complex and heavy system on the engine when you can get better perf when polling for it every frame in lua.


Этот файл нужен в репозитории?


this test does verify that dns.resolvePtr returns ENOTFOUND for a bogus PTR lookup


Not necessary - the previous line saves it too.


The reason for this was that we disabled the dialog at the end of the story asking for feedback by commenting the lines. These two imports here were for the dialog.


And more food for thought: http://mrale.ph/blog/2012/12/15/microbenchmarks-fairy-tale.html


Instead of iterating wouldn't it be better to directly get the bundle via https://osgi.org/javadoc/r4v43/core/org/osgi/framework/FrameworkUtil.html#getBundle%28java.lang.Class%29?


Updated href attribute.


You don't need this - you are not doing anything with it. Plus, it was a one-time charge, not a recurring payment, so no need to do anything with their stripe account.


I wanted to make sure that when setting the view to the presenter the fragment view is already created.
Also, this fragment is being retained, for instance, I did not want to reload data when an orientation change happens.

By doing it the other way on ```onActivityCreated()``` method I was not taking advantage of the fragment retained instance and data loading was being triggered whenever the activity is recreated on each orientation change.

You can see this behavior on the logcat: each UseCase execution is triggered only once :). This was the easiest way I found. If you have any other alternative I will glad to hear it :)


@gaearon [GMTA](https://en.wiktionary.org/wiki/great_minds_think_alike)


Assuming your downgrading_wikis! saves the changes to the wikis, you don't need to save again.


Этот метод может дёргаться довольно часто. Не возникнет проблем из-за этого? Может, где-то завести флажок dirty и не запускать аплоад, если не нужно?


no hard path forbidden have a look on the test to understand how get the path of the bench


( ^ω^ )


Note:
Now Android M Development for VilleC2 is started.
There are very few people is still on this phone, and almost no one can test new KitKat kernels. To debug KitKat's bugs is very hard now since my phone is broken half year ago. So let's start Android MarshMallow now.
Creating M for this device is difficult, but I'd really like to have a try.


How can polling possible be better performance? If it were, we wouldn't have any callins in the first place!


look comment


noooo!


Хотя вижу метод nativeHasSomethingToUpload().


Looks like you are mixing view permissions into here, but yet you also have pundit. Can you think of a way you could have put this in the wiki policy instead?


private?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1273](http://teamcity.monogame.net/viewLog.html?buildId=14217&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


"a direct request from the log" makes it sound like the log is generating the request.

How about...
s/a direct request from the log discloses/this will disclose/


ping 不通可能未必代表IP被屏蔽。
江苏移动，64.233.162.82，64.15.122.160 均仍可用，更新后提示证书错误。


Does this looks good?


You probably meant 0.5%


а вот это кстати бессмысленный тест. ты же не реализовывал здесь никакой функциональности «проверить количество аргументов», это дефолтное поведение руби. я же тебе не зря в прошлый раз писал «проверять поведение при некорректных аргументах (ЕСЛИ ПРИМЕНИМО)»

логика в том, что «проверка ради того чтобы что-нибудь проверить» низачем не нужна; важно проверить как ведёт себя твой код — в разных ситуациях.


Corrected typo.


Now config label return host:port/database


So we intend to add the dialog back later?


> However, for all subsequent HTTP requests, we don't necessarily default to the last successful fallback host

It was not clear on the spec. Still, this is a quick change that can be done in 5 min


Por qué le habéis puesto la GPL v2, lo normal es la GPL v3, qué tiene la v3 que no te gusta?


Yes, I am hoping that will happen when we release the last chapter maybe.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13799](https://teamcity.mesosphere.io/viewLog.html?buildId=119692&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 220 Build time: 00:03:52


it's better than having a huge system for managing callins/registrations for every change to a bazillion properties


I would do something like:
```python
data = data.decode("utf-8", "ignore")
```
so only that one line would change. The less changes the better IMO.
BTW, did you test this with unicode data? If so, could you produce debug logs please?


Hmmm, you called this on user, but have it as a helper. Helpers are generally for putting logic in that affects views. You probably wanted to put this method in the model. Also, there is an easier way than running through all of the wikis one at a time:

```
def downgrading_wikis!  #that's what you actually refer to it as when you called it in your controller
  wikis.update_all(private: false)
end
```

This assumes you move it to the user model so that you are accessing the user's list of wikis.


"img" elements should not require closing tags. For best HTML and XHTML compatibility <img [...] /> might be useful though (note the space before the slash!).


You shouldn't be using bare Exceptions.


Зачем потребовалось задавать имя приложению?


kek: https://gitlab.prok.pw/KCauldron/KCauldron/commit/12b0a52ae035ba89e922743d6f70fbdc72d230cb


Component for estimating the optimal missing values for an operator using multi-objective optimization (NSGI-II). In summary, for each missing value of an abstract operator:
1) IReS estimates the optimal value in a given range for all the available matching materialized operators
2) Selects the materialized operator that achieves the optimal performance with respect to the objectives (currently execution time)

Framework used: http://moeaframework.org/


@ebrehault Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.api/27//violations
```
ERROR: /home/jenkins/workspace/package-plone.api/src/plone/api/content.py Imports are incorrectly sorted.
src/plone/api/content.py:0:1: I001 found unsorted imports
src/plone/api/content.py:98:13: Q000 Remove bad quotes.
src/plone/api/content.py:99:13: Q000 Remove bad quotes.
src/plone/api/content.py:100:13: Q000 Remove bad quotes.
src/plone/api/content.py:268:1: C901 'delete' is too complex (15)
src/plone/api/content.py:300:25: Q000 Remove bad quotes.
src/plone/api/content.py:322:25: Q000 Remove bad quotes.
src/plone/api/content.py:360:1: C901 '_wf_transitions_for' is too complex (12)
src/plone/api/content.py:454:17: Q000 Remove bad quotes.
src/plone/api/content.py:455:17: Q000 Remove bad quotes.
src/plone/api/content.py:482:17: Q000 Remove bad quotes.
src/plone/api/content.py:521:13: Q000 Remove bad quotes.
src/plone/api/content.py:522:13: Q000 Remove bad quotes.
ERROR: /home/jenkins/workspace/package-plone.api/src/plone/api/env.py Imports are incorrectly sorted.
src/plone/api/env.py:0:1: I001 found unsorted imports
ERROR: /home/jenkins/workspace/package-plone.api/src/plone/api/portal.py Imports are incorrectly sorted.
src/plone/api/portal.py:0:1: I001 found unsorted imports
ERROR: /home/jenkins/workspace/package-plone.api/src/plone/api/user.py Imports are incorrectly sorted.
src/plone/api/user.py:0:1: I001 found unsorted imports
src/plone/api/user.py:70:13: Q000 Remove bad quotes.
src/plone/api/user.py:71:13: Q000 Remove bad quotes.
src/plone/api/user.py:357:37: Q000 Remove bad quotes.
src/plone/api/user.py:400:37: Q000 Remove bad quotes.
src/plone/api/validation.py:20:13: Q000 Remove bad quotes.
src/plone/api/validation.py:21:13: Q000 Remove bad quotes.
src/plone/api/validation.py:22:32: Q000 Remove bad quotes.
src/plone/api/validation.py:66:21: Q000 Remove bad quotes.
src/plone/api/validation.py:67:25: Q000 Remove bad quotes.
src/plone/api/validation.py:96:21: Q000 Remove bad quotes.
src/plone/api/validation.py:97:25: Q000 Remove bad quotes.
src/plone/api/validation.py:127:21: Q000 Remove bad quotes.
src/plone/api/validation.py:128:21: Q000 Remove bad quotes.
src/plone/api/validation.py:128:45: Q000 Remove bad quotes.
ERROR: /home/jenkins/workspace/package-plone.api/src/plone/api/tests/base.py Imports are incorrectly sorted.
src/plone/api/tests/base.py:0:1: I001 found unsorted imports
src/plone/api/tests/base.py:32:47: Q000 Remove bad quotes.
src/plone/api/tests/base.py:60:28: Q000 Remove bad quotes.
src/plone/api/tests/base.py:62:28: Q000 Remove bad quotes.
ERROR: /home/jenkins/workspace/package-plone.api/src/plone/api/tests/test_content.py Imports are incorrectly sorted.
src/plone/api/tests/test_content.py:0:1: I001 found unsorted imports
src/plone/api/tests/test_content.py:137:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:271:37: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:879:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:880:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:881:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1015:17: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1024:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1025:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1026:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1027:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1028:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1029:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1030:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1031:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1032:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1033:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1034:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1035:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1036:13: Q000 Remove bad quotes.
src/plone/api/tests/test_content.py:1037:13: Q000 Remove bad quotes.
ERROR: /home/jenkins/workspace/package-plone.api/src/plone/api/tests/test_doctests.py Imports are incorrectly sorted.
src/plone/api/tests/test_doctests.py:0:1: I001 found unsorted imports
src/plone/api/tests/test_doctests.py:120:13: T001 print statement found.
ERROR: /home/jenkins/workspace/package-plone.api/src/plone/api/tests/test_env.py Imports are incorrectly sorted.
src/plone/api/tests/test_env.py:0:1: I001 found unsorted imports
src/plone/api/tests/test_env.py:30:14: D001 found declarePublic replace it with AccessControl.ClassSecurityInfo.public
src/plone/api/tests/test_env.py:31:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:32:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:33:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:34:14: D001 found declarePrivate replace it with AccessControl.ClassSecurityInfo.private
ERROR: /home/jenkins/workspace/package-plone.api/src/plone/api/tests/test_group.py Imports are incorrectly sorted.
src/plone/api/tests/test_group.py:0:1: I001 found unsorted imports
ERROR: /home/jenkins/workspace/package-plone.api/src/plone/api/tests/test_portal.py Imports are incorrectly sorted.
src/plone/api/tests/test_portal.py:0:1: I001 found unsorted imports
src/plone/api/tests/test_portal.py:61:16: P002 found "hasattr", consider replacing it
src/plone/api/tests/test_portal.py:125:41: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:125:57: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:163:13: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:164:13: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:194:22: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:204:22: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:213:23: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:214:20: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:215:21: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:229:23: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:230:21: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:251:27: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:252:24: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:253:25: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:272:23: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:273:21: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:279:9: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:289:23: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:290:21: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:299:9: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:311:23: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:312:21: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:336:27: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:337:24: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:338:25: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:345:23: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:346:20: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:347:21: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:524:44: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:535:44: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:544:44: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:582:44: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:614:44: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:615:38: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:616:38: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:617:38: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:713:44: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:747:44: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:748:38: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:749:38: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:750:38: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:785:32: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:787:38: Q000 Remove bad quotes.
src/plone/api/tests/test_portal.py:788:38: Q000 Remove bad quotes.
ERROR: /home/jenkins/workspace/package-plone.api/src/plone/api/tests/test_user.py Imports are incorrectly sorted.
src/plone/api/tests/test_user.py:0:1: I001 found unsorted imports
src/plone/api/tests/test_user.py:641:20: Q000 Remove bad quotes.
src/plone/api/tests/test_validation.py:37:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:40:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:65:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:70:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:74:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:79:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:84:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:91:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:98:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:123:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:124:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:131:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:132:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:133:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:159:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:160:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:165:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:166:14: D001 found assertEquals replace it with assertEqual
src/plone/api/tests/test_validation.py:167:14: D001 found assertEquals replace it with assertEqual
```
You can run that yourself using buildout.coredev ``experimental/qa.cfg`` config file.


oh ! nice :)


@jensens Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-borg.localrole/4//violations
```
borg/localrole/workspace.py:161:1: C901 'WorkspaceLocalRoleManager.checkLocalRolesAllowed' is too complex (14)
```
You can run that yourself using buildout.coredev ``experimental/qa.cfg`` config file.


This is another case where you are mixing what you learned in the foundation with what can be done in Pundit. Not wrong here, per se, but it is possible to have this kind of logic in Pundit.


Okay, makes sense.


"except Exception as e:" bare?

it catches all exceptions. its the goal there


Maybe. The complexity of implementing it could be reduced with some macros, but I'm sure it'd involve some work.
I'd like to hear what kloot meant on the "fast property update system" - although it might not be directly related to Lua.


Need to fix formatting.


@ebrehault sorry for the noise I was testing the new user that will report about code analysis.

You could rebase your branch on top of master though :-)


You actually don't need a table for this, and you don't have it tracking any data anyway ;-)


Did you create an Amount class? I didn't see it in this commit...


- --metric_resolution=60s


Instead of testing resolvePtr to return ENOTFOUND for "8.8.8.8", i think its better to use the "Known Invalid" .invalid TLD, and this can be tested for all dns.resolveXXX() functions.


Такие поля принято наызвать is_worker/is_recruiter


Is there a better place to put this? @yuriks


早就不扫了，费电费时
防火墙最近有新招数，伪造目标服务器返回tcp包，造成443端口开放的假象。
比如64.233.177.114，这个不是gws，而是给check.googlezip.net 验证使用的，但是因为证书是 *.google.com 而被当成gws 处理掉了。使用端口扫描会发现这ip的80、443是开放的，但无法使用
ping和tracert/traceroute的结果也是正常的，但是如果你使用tcping或者tcproute就会发现异常，具体表现在tcping的延迟远远低于ping，是省际网或者国内骨干网级的延迟，tcproute会发现包并没有走到谷歌骨干网，而是在国际出口就返回结果了
除了本身就不开443和不提供正常web服务的，其他大部分被这种手法办掉了，所以单纯的扫描已经无意义
但64.233.162.0/24 这段俄罗斯的不知道为什么一直在放水不封，不知道是想钓出更多ip还是什么


empty не поддерживает в php до 5.5


Hey, just a heads up:

I've the notifications to Materialize toasts and done page animation related changes. I'm all for removing them but there will be/are merge conflicts: https://github.com/earlymarket/CoPosition/commit/09f1bcccc2c2b18735e7ca7c37019a336b2490c6


Угу.


bla bla bla


subject и message лучше разместить в шаблонах, их уже рендерить с конкретными данными, например с помощью render_to_string().


...saves...


May be explicitly say 8 flits?


Default input reader of...


Хардкод - совсем плохо. Емейл от которого пишем лучше разместить в настройки.


s/optinal/optional/


calculate area of a circle by radius


ok


这两段服务器并没有禁ping，ping不通就可以说明异常了
证书错误可能是中间人攻击，得小心处理
当然，这次的ip是有问题的，证书错误正常


Лучшим дизайном было бы отделить функцию рассылки почты, которая принимала бы все необходимое для отправки письма, и вторую функцию - отсылки письма регистрации, которая пользовалась бы первой. Отсыка письма еще будет не раз нужна в проекте.


TeamCity Civic Assembly :: Build backend + frontend [Build 47](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=604&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Would people be confused with packet width? Would it be better to explicitly say the flit data width or at least use data width instead of packet width to avoid confusion between packet width and packet length.


If you're going to swallow all exceptions indiscriminately then there's no reason to be doing it in the library, do it during the call to the library.  The purpose of doing try..excepts is to handle things a certain way based on the presences (or absence) of a specific exception.  In addition swallowing *all* exceptions can hide other issues that *should* be raised as errors to be resolved.  For example the `dateutil` library that we used had a too-broad exception call when importing Windows related methods, resulting in the library never loading the Windows code, never giving any indication that it wasn't using the code, yet failing spectacularly on Windows.


Комментарий непонятен, user будет None в случае провала аутентификации.


Not the collection itself, but randomising the usage would be better.


Ok cool now it makes sense that the problem was the orientation change in order to not re-fetch data from the network. Because for sure on ```onActivityCreated``` you already know that the view is created as it is a state after ```onViewCreated```.

BTW check this http://hannesdorfmann.com/mosby/viewstate/


If the previous `fnctl()` call to set the `FD_CLOEXEC` flag fails, this function will not work properly. Might be a good idea to have a `sol_util_fd_set_flag()` variant that uses `F_GETFD` and `F_SETFD` instead of their `FL` variants and use it here instead.


but I want to catch all network/connection/requests errors. there's only "session.get" inside the try


oops, ja den virker nok ikke slik som det står. det bør være
`_.find(text, function (value, lang) {
                    return lang === "nb";
                  })`
etc.


Concerning the workflow, the second approach is definitively easier, but I don't know if it is "better". Without v-sync and a good computer, we can easily reach 1000Hz simulation. And  it may make sense to run at this speed for physics stability / accuracy (even if most logic call are empty).


You might also want to add:
# Copyright 2016 Reliance Jio Infocomm Ltd.


Probably a screen height thing:
![screen shot 2016-02-01 at 11 43 26](https://cloud.githubusercontent.com/assets/1552489/12716519/0c46615e-c8d9-11e5-9c74-183e858723b2.png)


Thanks!


В Хаскеле есть стандартная функция `id`, которую лучше использовать вместо такой лямбды (\более читаемо получится).


Why would it be better @gokhanbarisaker?


All requests errors are handled by catching `requests.exceptions.RequestException.`


True. I should add credits to readme. Thanks.


@heath here's my regex issue you mentioned yesterday.  The + should have been outside the ]:  `^[0-9]+`


TeamCity pull requests :: netty [Build 1164](https://garage.netty.io/teamcity/viewLog.html?buildId=5279&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6402, ignored: 23 Build time: 00:43:10


Are you sure they are the only two controller/list URLs?

What about the match ':controller(/:action(/:id(.:format)))' which looks like a real security hole. Can we express that as a more definitive list?


Is there any need for the #ifdef __PIC__ ?
David


even sockets timeout ?!


@soffokl next commit will fix it automatically :)


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.api/28//violations
```
src/plone/api/content.py:268:1: C901 'delete' is too complex (15)
src/plone/api/content.py:360:1: C901 '_wf_transitions_for' is too complex (12)
src/plone/api/tests/test_doctests.py:121:13: T001 print statement found.
src/plone/api/tests/test_env.py:31:14: D001 found declarePublic replace it with AccessControl.ClassSecurityInfo.public
src/plone/api/tests/test_env.py:32:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:33:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:34:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:35:14: D001 found declarePrivate replace it with AccessControl.ClassSecurityInfo.private
```
You can run that yourself using buildout.coredev ``experimental/qa.cfg`` config file.


test


#1 sizzle already included in jQuery
- [ ] remove this line


Не забывай про перевод строки в конце файла.


te


程序main class


Was this import tested? If this were to work, 'ThreadLocalMetrics' module should reside in the PYTHONPATH root. But, it is isn't so. The path is different in the pull request @
https://github.com/souvik-ray/monitoring/pull/1


Можно спрятать


The XSI version returns an integer error (0 on success or -1 on error with errno set), always using the passed buf, instead of an string such as the GNU version (which might not even touch the passed buffers as it might return pointers to internal tables or use it to write an error message in case the errno is invalid).

This will most likely lead to NULL-pointer dereference (as 0 is cast to (void*)0), or `~0` in case of error (where it returns `-1`).


Hi Kai,

Thanks for your response. do you mean to change just the name of this
method?

On Mon, Feb 1, 2016 at 1:15 AM, Kai Liao <notifications@github.com> wrote:

> Hi @AymanDF <https://github.com/AymanDF> Thanks for your great work.
> I like the idea shrim the size for switch case, but I don't think that the
> method name casePositionXXX provides good enough readability. I would
> suggest you change the method name to what it really do, for example
> grid(), fromxml(). What's your idea?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DevFactory/BottomSheet/commit/62895d155cc8f417753efa465eef64813797bc2e#commitcomment-15788533>
> .
>


По-английски правильней было бы сказать is_remote.


Я бы для санитарии добавил try {} catch (IOOBE). Чёт не доверяю я пацанам из myTracker :)


here as well...


> this fallback behaviour only applies to requests to rest.ably.io

We have test cases for failing requests to unreachable hosts. Also, we don't want to perform fallback with that specific test case. e.g., `some.example.host` instead of `sandbox-rest.ably.io`

> http://docs.ably.io/client-lib-development-guide/features/#RSC15b needs to be added to the client library

Within our current spec, we are not supporting any other host than `rest.ably.io` for HTTP calls. In theory, adding extra checks, or APIs (setting custom host, etc...) does not makes sense to me.


This may actually be the result of clearing the buffers and not the actual devices. Need to investigate.


#1 Sizzle already included in jQuery
- [ ] remove this line
- [ ] delete this directory `bower_components/sizzle/`


Ok, when you are done and merge into dev I'll pull it into mine and resolve the conflicts there. Thanks


What is the intention behind this decoration?


TeamCity Civic Assembly :: Build backend + frontend [Build 47](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=604&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 118 (40 new), passed: 54, ignored: 4 Build time: 00:12:31
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1454327140069: <no details avaliable>

User profile tests: Should not change user profile info: <no details avaliable>

User profile tests: Should change user profile info: <no details avaliable>

User profile tests: Should cancel upload new user avatar: <no details avaliable>

User profile tests: Should upload new user avatar: <no details avaliable>

User profile tests: Should not change user password: <no details avaliable>

User profile tests: Should change user password: <no details avaliable>

User profile tests: Should not change user organization: <no details avaliable>

Basic Event Categories CRUD Actions: Should Add an Event Category: <no details avaliable>

Basic Event Categories CRUD Actions: Should Not Edit an Event Category: <no details avaliable>

Basic Event Categories CRUD Actions: Should Edit an Event Category: <no details avaliable>

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


Окееей)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13800](https://teamcity.mesosphere.io/viewLog.html?buildId=119693&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:25:47


Эти папки тоже следует удалять из репозитория.


так и надо было написать `notReverse = id` :)


Merge prov


Some remarks concerning syntax.php:
1. 1st patch hunk: the comments are only left-others from copy-pasting
2. 3rd patch hunk: lots of duplication with PNG code, testing `($data['type'] == 'svg')` twice for the prefix and suffix parts would be better IMHO
3. second-to-last patch hunk: seems unneeded because the "-encodeurl" overrides that (no image is generated)
4. what happens with the ODT code called elsewhere in this file (look for "`$renderer->_odtAddImage(`")? does it support SVG already?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13801](https://teamcity.mesosphere.io/viewLog.html?buildId=119694&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:49


@ygorshenin @mpimenov @vng @kshalnev @gardster @mgsergio @syershov
Before we switched to gtest, I've added these useful macro.


We need to add the new files to the spec file. And remove the old ones from the repo?


This commit, brought to you by the letter "Z", and the number 1


interesting


Hum, that's a good point: we don't want MORSE to give the feeling that it sucks up all the CPU (which is going to be interpreted as 'MORSE is slow'). On the other hand, I don't see why the freq would go above what is requested by the fastest component?


#1 sizzle already included in jQuery
- [ ] remove this line


Повторение полей другой модели может свидетельствовать об ошибке дизайна. Возможно пользователь уже указал эту информацию в своем профиле?


Have you tested you can change it on-the-fly?
Samsung's SELinux(which seems to be the base of this) doesn't allow that.


Настройки Celery должны задаваться централизовано, а приложение использовать их в том виде, в котором дали.


@ezavaleta Thanks for reporting the issue. It should be fixed in master


didn't you mix up parameters order? I guess it should be `new Request($m, $url)`


Isn't this a duplicate of the line just above?


Yes its on the fly change.


Testing for Customer here seems unnecessary, since it is top level, and that is covered by no relation tests, right? What we need to test is CustomerSetting. We also need a test on this which checks relationship type, so that's a 3rd test.


#1 sizzle already included in jQuery
- [ ] remove "**sizzle**" from this line


It depends whether GCC adds CPIC to non-PIC files, as then ld would give the same warning because the files would be the other way round. I will check what GCC does later.


@WWGate It's a very rough mistake of filter's usage.
The first parameter ALWAYS must be value which is being returned by apply_filters() function.

`$gslug = apply_filters('wpp::groups::label', $gslug, $group_name );`

But NOT:

`$gslug = apply_filters('wpp::groups::label', $group_name, $gslug );`


This path should be fetched from python build script


Overengineering - почему просто не написать здесь тайтл, и не передавать в контексте?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13802](https://teamcity.mesosphere.io/viewLog.html?buildId=119695&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 220 Build time: 00:08:12


s/neither/none/
s/throws/throw a/


You could modify the RequestContext constructor to have a placeholder for the metrics rather than caching it in the request itself.
OR...Even simpler, you could exercise Python's power with dynamic attributes and create a placeholder right here.


<br> - обычно признак плохой верстки, и тут почему-то закрывающий(!) тег.


nope.


I'll try.


Socket timeouts aren't properly caught by `urllib`, thus I don't believe `requests` catches them directly, however `socket.timeout` could be added to the caught exception. e.g.:

```python
try:
    # FAIL
except (requests.exceptions.RequestException, socket.timeout) as error:
    # FALL BACK
```

If you need to catch *all* socket related errors and treat them the same, then you can use `socket.error`.  And if even that isn't enough, `socket.error` is a subclass of `IOError` which is  in turn a subclass of `EnvironmentError` which is a subclass of `StandardError` which is then a subclass of `Exception`.  You have all these levels of exceptions available before you get to a bare exception, there is no reason other than lazy programming to catch all exceptions at this level.  About the only time it is generally acceptable to catch a bare exception is at the outermost levels of a program to prevent the program from crashing and guarantee up-time and even then care should be taken to log in detail the errors raised for troubleshooting.