Does this result in unprivileged users being able to get to /var/lib/lxd/snapshots/staleprivilegedcontainer/rootfs/sbin/vulnerable_setuid_binary ?

Some piece along that path must be 700.


Never used this module, does it search the OS paths for the specified binary name?


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/dev/9617afb861e49330284cc71d29b2a7ff67b313a5/index.html)


@smeezekitty
Of course they shouldn't. All my pro content policy posts should be read as "have them, but only enforce them if they really are harming something". They should try to defend the user as far as they can and only enforce them if it's really bad for a major of the community or for them, not just because some people want a project to be removed.


With these indentation fixes you may want to rebase your branch from the latest commit to `master`. See the `upstream` branch within the `readme`.


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/0.1.0/index.html)


Out of curiousity/ignorance, what is the advantage of this over filepath.Dir(path) ?


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/dev/a87c31989953b1328746b3379167d8be7be35cb1/index.html)


1718_setpoint_managers (mbadams5) - x86_64-MacOS-10.9-clang: Tests Failed

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


@pbochis For your consideration :wink:


Most Sensei's should be respected


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/dev/289bdffab626625bd3f7829213cdb3d279a52418/index.html)


fix the name here


Not sure why you would declare `functions` here. They really should be documented & implemented within the `[tools](https://github.com/jas-/node-libnmap/blob/master/lib/node-libnmap.js#L350-L606)` object.


Previously we had builders for different types and it was horible complex. The API will take care of the validation, so we can keep the isRequired method.


yeah I'll take care of that..


Same as above with regards to proper function defining.


It doesn't create snapshots - snapshots is 0700. But in another usecase it creates containers where everything ?must? be 0755.


no, this does not help.
my guess would be it's about priorities


done


Why this is not needed anymore?


don't think filepath.Dir goes one path back, will test this right now.


Hm. Can confirm that this is an issue, not sure why. Will try to come up
with an answer.

On Thu, Jul 30, 2015 at 12:40 PM, bethlakshmi <notifications@github.com>
wrote:

> First test cases...
>
> Need help…
> @jonkiparsky  - I am stupidly re-coding UserFactory and ProfileFactory
> in my ticketing/tests/factories.py to make this work.
>
> What I want - is to figure out how to import and reuse your factories
> in gbe/tests/factories - but I’m stuck.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/bethlakshmi/GBE2/commit/03c73269a9fa9581c9085b3ff9397794ce238e1f


Make toasts higher than the modal and the modal shade.


What happens to extra args that are not objects? They're just discarded?


FYI @johnhuangclever, `fmt` will automatically call the `String()` method on the object if it is being marshaled into `%s`.
(there's an edge case rule that dictates that `error` types will use `Error()` instead of `String()` automatically)

So you can simply do:
```go
return fmt.Errorf("process timeout after %s. Unable to kill process, error: %s", conf.CmdTimeout, err)
```


done.


Receive the contents of the field after focus lost


need to write another sp for deleting the cartItem. have to be careful to make sure that we're not just deleting the entry, but also returning the inventory back to the run (taking it from reserved back to available)


tab -> spaces


Ah!  yeah, so it does  (create snapshots as 0700).  Very good, thanks :)


I may change this to `oninput` to get instant results. May be annoying though.


Ok, will do.


Execute the below if there's a `?` in the URL of the page


This shouldn't be here, but on odoo/odoo


Esta base de datos ha sido obtenida en julio de 2015 a través de una petición de Acceso a la Información realizada a la Dirección General de Tráfico. Ha sido utilizada para elaborar esta información:
http://www.elconfidencial.com/espana/2015-07-31/accidentes-carreteras-espana-animales-jabalies_949456


Oh, you're right! Good catch


You can use directly the view->details->model here


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

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


@stvstnfrd yes I didn't notice it was set to off in dev.py so that makes sense to be consistent. My original thought was that since Albert was changing this just to get his thing to work then it shouldn't make a global change like this.


Actually, it seem the code is correct, just the map is not


Oops?


* `decodeURI(document.URL)` - Makes objects like `%20` back into spaces
* `\?repo=` - first arg must be `repo`
* `Icons|Illustrations|Concepts` - repo must be either Icons, Illustrations, or Concepts (case insensitive because of the `/i` at the end)
* `&author=(.+)` author is the rest of the URL


done: https://github.com/balderdashy/sails/pull/3126


Делать `setDirty()` лучше после проведения всех изменений. Иначе может случайно получиться так, что сохранение сработает после установки `setDirty()`, но до изменения содержимого `object`.


@tadatuta I was thinking about it. Let's still call it `.once()`. I'll just change the semantics and make it a `=== previously-saved-bemjson`. I just realized that putting the properties on BEMJSON will have the same effect as `===` anyway.

Going to figure out how to do it in bem-xjst@1.x in a best possible way.


backported to b32 https://github.com/pagespeed/mod_pagespeed/commit/c5e438b67f545155d4bf36e375ee463509880015


This wont be necesary when natilus view becomes a overlay... not sure if worth to add it here and remove it later


TeamCity DNN JavaScript Libraries :: CI [Build 55](http://teamcity.engagesoftware.com/viewLog.html?buildId=323&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


`oadm` has many commands to help manage `master-config.yaml`, `node-config.yaml`, and associated certificates and support files.  Are you sure you don't need it?


TeamCity DNN JavaScript Libraries :: CI [Build 55](http://teamcity.engagesoftware.com/viewLog.html?buildId=323&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:8


TeamCity DNN JavaScript Libraries :: CI [Build 56](http://teamcity.engagesoftware.com/viewLog.html?buildId=324&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 56](http://teamcity.engagesoftware.com/viewLog.html?buildId=324&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:7


This looks odd... I guess not overriding add is fine


Fixed: https://github.com/deantomasevic/multilingual-press/commit/bc5e9417f4968b1f4517fda74d9c178b7bee789c


TeamCity DNN JavaScript Libraries :: CI [Build 57](http://teamcity.engagesoftware.com/viewLog.html?buildId=325&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


yes it is ;-)


* `params[0]` = "?repo=Icons&author=Author Name (Name)"
* `params[1]` = "Icons"
* `params[2]` = "Author Name (Name)"


TeamCity DNN JavaScript Libraries :: CI [Build 57](http://teamcity.engagesoftware.com/viewLog.html?buildId=325&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:7


@pmattern Pls update to the latest lxqt-globalkeys. I left, by mistake, half of the changes out the commit.
This kind of thing tells me that it's vacation time.


Edit value seen by the form, the PHP, and DOM


tab -> space


Change in input for the author


ditto


Open the MaterializeCSS modal


dito


All authors: comma seperated


Commit message still says the description of the class at the start.


Yes, that should work; I also don't see the issue anymore. I came across this while implementing file transfers but maybe I just messed up my local checkout...


`value` Is the input's text for author


Make an array to work with later.
* `value` = "Daniel Hickman"
* `valuear` = ["Daniel", "Hickman"]


tab -> spaces


Logic is from here btw: https://github.com/liferay/liferay-portal/blob/2f2fcb3781d146a1557abf1446b7ddc46fe96367/portal-impl/src/com/liferay/portal/jsonwebservice/action/JSONWebServiceInvokerAction.java#L106-L128


Create/clear the suggestion list


Oh, you're right! Good catch


1718_setpoint_managers (mbadams5) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Ok!


@coolamit `require_once PMC_THEME_UNIT_TEST_ROOT. '/config.php';` was removed but I don't see `require_once PMC_THEME_UNIT_TEST_ROOT. '/classes/config.php';` added, from the commit message it seems like an oversight. Can you confirm the code works as intended?


For each entry into `valuear`, go through the following.
(Will go through once as "Daniel" and another as "Hickman" for ["Daniel", "Hickman"])


close https://github.com/CartoDB/cartodb/issues/4604


make a constant


Why?


Create a new regular expression object with the variable. Case insensitive
(Will go through once as "Daniel" and another as "Hickman" for ["Daniel", "Hickman"])


This comment from when looking at pull request from foo branch.


I don't follow. I'm referring to this:

    @gen.coroutine
    @vcr.use_cassette(...)
    def foo():
        ...

You can't do this:

    @vcr.use_cassette(...)
    @gen.coroutine
    def foo():
       ...

Because the wrapped function after coroutine gets applied is no longer a generator. It's a function that returns a Future object.


Bring NV item from xiaomi back or WiFi won't work


@neappx: That requires some meaningfully objective definition of harm,
which I don't think is a realistic possibility. Better for them to just
stay out of it unless there's a court involved.

On Thu, Jul 30, 2015, 12:53 PM neappx <notifications@github.com> wrote:

> @smeezekitty <https://github.com/smeezekitty>
> Of course they shouldn't. All my pro content policy posts should be read
> as "have them, but only enforce them if they really are harming something".
> They should try to defend the user as far as they can and only enforce them
> if it's really bad for a major of the community or for them, not just
> because some people want a project to be removed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12450502>
> .
>


It is needed because we already add a GtkScrolledWindow to the overlay. Otherwise, views would just override the scrolled window and, well, things won't scroll anymore.


Not really related, but still important: keep in mind that a theme could (/ should be able to) live anywhere (i.e. Inside vendors).


Should be `('grid',)`  Not sure if that's causing bugs or not but it should...


what made you think of that?


Don't think this will be removed later. It redirects the new widget to the internal scrolled window, which is unavoidable.


Thanks for the tip, didn't think about putting a default.


I've put your suggestions into the [translation website](https://translations.launchpad.net/connectbot/trunk/+pots/fortune) and will import them soon.


There was already flash code, and now it is present twice.  I don't understand.  It seems like it's doing the opposite of what your intent was (going from showing something once to twice).


I made a update


I don't like where this conversation is heading honestly. Like I said before, I'd rather fix the root cause than patch a patch because there is (most likely) a conflict with another add in.


I was wondering about something like that. Thanks


W innym miejscu dodałem przykład (na razie z moim repozytorium) o importowaniu i o module random, więc ten przykład z tąd usunąć, albo przenieść po opisie importowania.


1718_setpoint_managers (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Thanks, I don't know why I got confused on that and ended up with something complicated...


Fixed.


I knew I forgot something...


Not an objection here but why change int to var here ? Any reasons beyond cosmetics ?


I like it this way :)


done


Does this count as an API change and does it need to be handled with deprecation?


Good catch. Somehow I erased it. Fixed.


to zadziała tylko pod linuxem. Usunąłbym obie linijki (import oraz os.system). To nic nie popsuje, a będzie mniej magii.


It causes a crash, since we're adding the scrolled window to the view. Thus, when it performs the check, it always receives a non-NULL child (the scrolled window) and fails.


I think the inclusion of this parameter is causing it to lose the coefficient parameters.  It's not really straightforward currently to subclass a model and add new parameters on top of the existing parameters (though it would be nice).  I think astropy/astropy#3679 should help with that though, since it provides a mechanism to inherit parameters from the parent class.

Other than that this (by which I mean the nested evaluates) seems like a feasible approach for the time being.


This kernel is not ready to be used
Il 30/lug/2015 20:13, rooque <notifications@github.com> ha scritto:Bring NV item from xiaomi back or WiFi won't work

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


This can be a classmethod but it doesn't matter much.


So, now there are no packages that use it.


dlaczego od 0?


Actually, I take it back, maybe not, since this subclasses Polynomial1D.


that'll basically take only the first 6 characters of the zone name. It is very common for zone names to share a prefix and just have a different suffix, so this will make it so that a lot of zones will share the same "uuid". How about taking the last 6 bytes? or the first 3 bytes and last 3 bytes? or something along these lines?


90946736_SHRgt1_4567 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1263 of 1263 tests passed)

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


Super, tego brakowało :smile:


Ran or run?


Ass covering LOL


Напляках нещо, ама направо махнах email частта, щото създаваше проблеми при създаването. Ако някой иска да си сменя мейла, да каже на админите да влязат в рейлс конзолата. Толкоз.


Imo pr could be merged.


All in a day's work


The original makes it easier to put comments, which I took advantage of after you comment :)


compile-c-c++ ..\..\bin\msvc-10.0\release\boost-source\deprecated-functions-off\
geoip-static\i2p-off\link-static\optimization-space\threading-multi\src\utf8.obj

utf8.cpp
..\..\src\utf8.cpp(61) : error C2039: back_inserter: is not a member of "std"
..\..\src\utf8.cpp(61) : error C3861: back_inserter: identifier not found
..\..\src\utf8.cpp(78) : error C2039: back_inserter: is not a member of "std"
..\..\src\utf8.cpp(78) : error C3861: back_inserter: identifier not found

    call "D:\Program Files (x86)\Microsoft Visual Studio 10.0\vc\vcvarsall.bat"
x86 >nul
cl /Zm800 -nologo @"..\..\bin\msvc-10.0\release\boost-source\deprecated-function
s-off\geoip-static\i2p-off\link-static\optimization-space\threading-multi\src\ut
f8.obj.rsp"

...failed compile-c-c++ ..\..\bin\msvc-10.0\release\boost-source\deprecated-func
tions-off\geoip-static\i2p-off\link-static\optimization-space\threading-multi\sr
c\utf8.obj...

I was able to build it only after I've reverted this patch


all integration tests except upgrade ones...


Nice. This fixed my issue #4 - closes #4 for me.


@edolyne We should use this method to return the site name on your plugin blocks.


My gut feeling says we should wait until ppx stabilizes. Complicating the
build process is exactly the kind of thing that can trip up new comers and
having just spend a non-trivial percentage of my hacking cycles this summer
doing boring chores making Frenetic simpler, I'm hesitant to add complexity
back in.

Having said that, deriving seems so awesome, especially for show, that I
can very much see the attraction.

Overall, I guess as long as we document the fragile _oasis hacks, I'm
reluctantly okay with it.

-N

On Thu, Jul 30, 2015 at 6:50 AM, Arjun Guha <notifications@github.com>
wrote:

> @craig-riecke <https://github.com/craig-riecke> @jnfoster
> <https://github.com/jnfoster>
>
> We should discuss if we want to use this or not.
>
> I've used the ppx_deriving extension (supported by OCaml 4.02, which we
> all use) to auto-generate pretty printers for the entire OpenFlow 1.3
> library by simply adding [@@deriving show at the end of every type. For
> most datatypes, this does exactly the right thing. There are a few cases
> where we may want more control (e.g., printing 32-bit integers that
> represent IP addresses as dotted-decimal). We can do this easily by
> specifying a custom printer for an individual field in a datatype.
>
> Here is the caveat: it seems as though we can't use a Camlp4 extension and
> a PPX extension within the same file (camlp4 appears to choke if you use a
> PPX extension). In principle this is not a big deal. We use these
> extensions right now in Frenetic:
>
>    - cstruct.syntax in the binary parser
>    - ULex and some stuff for mucking with OCaml ASTs in
>    Frenetic_NetKAT_Parser and _Lexer
>    - sexp.syntax in a couple of type definitions as a cheap pretty
>    printer (and we could replace these with [@@deriving show] and get
>    much nicer printing)
>
> Unfortunately, _oasis causes trouble. We configure our project to use
> syntax extensions by adding sexplib.syntax, cstruct.syntax, etc. to the
> BuildDepends section of the library. This runs Camlp4 on all files,
> including any file that uses a PPX extension, and the build breaks.
>
> The solution is to remove the Camlp4 dependencies from the _oasis file
> and add them explicitly to the _tags file after the OASIS_STOP section.
> In this commit, I've added this line to _tags:
>
> <lib/*.ml{,i}> and not <lib/Frenetic_OpenFlow0x04.ml>: syntax_camlp4o, package(sexplib.syntax), package(cstruct.syntax), package(fieldslib.syntax), package(camlp4.lib), package(camlp4.extend), package(camlp4)
>
> This line invokes Camlp4 in all files except the one where I've used the
> PPX extension. It's sort of ugly.
>
> Thoughts?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/frenetic-lang/frenetic/commit/f8b5b9feb67b382ddc7ba3fea031cda2f0563473#commitcomment-12440880>
> .
>


Updated with English tips
#845


@azturner Will do, thanks for the heads up!


Same as above, but will grab all text from after a space and then a comma - `, ` to right before the comma that ends the match - `(?!(?!,)`.
* valuear[i] can equal "Crutchley"
* This regex object will generate `/(.+), ([\s\w\(\)]*Crutchley[\s\w\(\)]*)(?!(?!,),)(.+)/i`
The regex object means `(.+)` grab all (one or more) non-end-line characters until... `, ` a comma and a space `([\s\w\(\)]*` grab all (0 or more) whitespace characters, alphanumeric characters, and parenthesis `Crutchley` grab 'Crutchley' `[\s\w\(\)]*` grab all (0 or more) whitespace characters, alphanumeric characters, and parenthesis until... `(?!(?!,),)` The character is a comma. I don't know how I came up with this, but it forces the 'greedy' grab all statements to not grab it `,)(.+)` collect the rest of the characters.

Note: I'll be changing both `(.+)` to `(.*)` so that it doesn't force off the first and last character. This would fix `createme` appearing as `createm` (since he's last on the list)


See if  "Daniel" is somewhere on the authors list if `valuear` = `["Daniel", "Hickman"]`


Add a suggestion to the array following `authorregexmatch`'s regex object


Hovhovhovhov, det er @Eckankar du skal sige halløj til!


This is an old and inaccurate version (wmd calc is wrong) that Cindee and I wrote. I thought we already pulled my new version?

> On Jul 30, 2015, at 10:33 AM, Katelyn Arnemann <notifications@github.com> wrote:
>
> Merge pull request #21 from mb3152/master
>
> Add nodal roles for weighted partitions from @mb3152
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/nipy/brainx/commit/9087055942d56243e84099e00e8203c3d0d8149e


Set matched so if it's not changed, we'll know that there was never a match in the following `for` loop


https://github.com/nipy/brainx/blob/master/brainx/nodal_roles.py is the correct version


For each suggestion we have, do the following


asdf


Should be fine, I think


This commit will cause camera FC issue, use this patch instead https://github.com/Meticulus/android_device_samsung_codinalte/commit/271f1d7803afb889793035e78467e9ff1fd26d1f .


qwerkjasdf;lkj


if the suggestion currently going through the `for` loop is as long as the list of authors, the greedy regex took over so there wasn't a match. We won't make a toast or change `matched`


Maybe subclass from UIControl rather than UIButton.  http://stackoverflow.com/a/30765284/3400034


Add what? Add a message bag? Add a message?

In my opinion the name should be changed.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.864.e2663186c4450b44a9e341506b19b26cad9a08fb](http://build.palaso.org/viewLog.html?buildId=60263&buildTypeId=bt431) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 397, ignored: 3 Build time: 00:02:06
### Failed tests
```
GlyssenTests.dll: GlyssenTests.Utilities.VerseRefExtensionsTests.SendScrReference: <no details avaliable>

```


could you make this
```
return (value || '').slice(start - 1, start - 1 + n)
```
The call chain doesn't ensure value is non-null


This revision is clean.


Have you tried after latest changes? I have just installed clang package and added compiler in Qt Creator


I don't think it would be that annoying. May even be helpful


LGTM :shipit:


This is already implemented using lifetime array which provides all choice we need. Please don't change again the API, because we need to test on stable base now.


Just a friendly comment :)

Consider -- is there a faster way of doing this? This takes o(3n) time. I say that there is an in-place solution that takes only O(2n) time.


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/dev/d0ef4996bd0e87440c8f5daf28990fc466523569/index.html)


you did good


if the suggestion currently going through the `for` loop was matched, lets set the variable and then make a toast.
* `onclick="setAuthor(\'' + sug[i] + '\');"` This will evaluate to `onclick='setAuthor("Corbin Crutchley (crutchcorn)")' which we'll make the function further down in this document
* `function(){Materialize.toast('<span>You are creating a new author</span>', 10000);}` When it's dismissed, assume the user didn't set it to the suggested author. This need more testing currently.


this is correct. however, this test could be adapted to inherit from BaseLeapTest instead (by just importing the basetest module the events wil lbe disabled), and instead of the repetition below we can just invoke the setUpEnv classmethod.

other testsuites were adapted to that new method, but it looks like some tests in soledad were not.


This is a new change


If `matched` wasn't changed to `true`, we'll make a toast to tell the user that it wasn't matched, so whatever he typed was a new author.


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/0.1.1/index.html)


Figured this was manual. No way that you made it so that it read from GitHub


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/dev/fb7a1b48d8ac8cacdbf18aad264ced6ab2bd7971/index.html)


Because user that do not have credentials can skip them without getting any errors.
But in general I agree that ProfileCredentialsProviders can simplify the things. I will try it in the next version.


Example:
```
$this->visit('edit-user-page')->seeInInput('#first_name', 'Duilio');
```


fact


Maybe extract this logic to static public method?


Keep in mind that `newConversation` is a helper created to setup a conversation in the tests.

This should not be used by any consumer of the library - particularly because setting a version beforehand is invalid.


Set the `author` input in our form to the changed author. This may also be use to autocorrect without asking first (Corbin should always be the full dir name).


I don't think we need this file.  We already have all of these routes in the product page.  There should be some middleware or session checks there to ensure a user is an admin.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.865.6985c96dcf15b73e33ee09c9aaf264fac396fc01](http://build.palaso.org/viewLog.html?buildId=60264&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 398, ignored: 3 Build time: 00:02:38


Nah, let's do `wrap`


For this entire document, I forgot the commas aren't required to make `[` `]` so it was previously adding literal commas multiple times while I was also escaping other instances.


The vals.queryCharValue(valind) here can be removed since it has been done in line 1657.


extra


Pushed out `.wrap()` for 3.x


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.866.572b1c627fdcf02fa65e097260c43f79c7bc67b2](http://build.palaso.org/viewLog.html?buildId=60265&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 396, ignored: 3 Build time: 00:02:22


I see your point, but I wonder if, given it's an `external` project of sorts, keeping the non relative imports is OK?


This conflict needs to be resolved.


Gawd damnit! Thanks @akoenig! :+1:


Should this also be LDAP_COMPARE_EXT_S as in line 2135? And similar lines below.


> In ActionCable a connection instance instantiates a WebSocket connection

I think the websocket is actually initiated from the javascript (client) when this is run: `Cable.createConsumer 'ws://localhost:28080'`. That makes the request to the puma server (ActionCable server) to upgrade the connection to a websocket, which the server then responds by either upgrading the connection or not.

> Are these files going to be relatively sparse in most cases? I say this because they just seem to define a generic connection and a generic channel.

I think those will be pretty sparse to start with, since they are just a common superclass, much like `ApplicationController` is a superclass for most controllers in an app

> Nothing else subclasses Class Connection, so can we assume that any connection is automatically handled by that code? So in terms of instantiating a connection, i.e., opening up another browser to the application, can we assume a connection is automatically established as long as the user is verified?

As to how the app knows what to do with the `ApplicationCable::Connection`, I'm not sure right now. I don't see it being hooked up anywhere or called by anything. I guess as we dig in more and get things working then it will become more clear. But I think your assumption is correct, that the `ApplicationCable::Connection` is called by something to authenticate the websocket (ie: return whether or not to upgrade the connection to a websocket).

**EDIT:** It looks like the [ActionCable code](https://github.com/rails/actioncable/blob/1310e580644f458069161b5b7df6c8c0f7812f7f/lib/action_cable/server/configuration.rb) specifically requires a class named `ApplicationCable::Connection`. See line 14 of that file.

> In the follow method in the comments_channel.rb file, I do not know what the following code is doing: stream_from "messages:#{data['message_id'].to_i}:comments". Is ActionCable on the server side receiving a stream from a URL (or path) in double quotes?

The [comments in the source file for `ActionCable::Channel::Streams`](https://github.com/rails/actioncable/blob/master/lib/action_cable/channel/streams.rb) look pretty informative.

So let's take the `#follow` method and dissect it:

    def follow(data)
      stop_all_streams
      stream_from "messages:#{data['message_id'].to_i}:comments"
    end

So the first line in the method unsubscribes the current user from any broadcasts for the `CommentsChannel` channel. The second line subscribes the current user to any broadcast with the name `"messages:[id]:comments"`. I think that name could be anything you want it to be, as long as it is scoped properly to whatever you're trying to do.

So for the game waiting area, you could have all users waiting for the same game (let's give the game an id of 246) subscribe to any broadcasts with the name of `"game:246:waiting"`. So every player who joins a given game would subscribe to that broadcast, and each time a new player joins a game, a broadcast would be sent out notifying all subscribed players (in this case, the waiting players) of the new player.

I think by Saturday we should be able to tackle getting some basic websockets code working. We'll struggle through it together :-)


@systemAce24185 have you seen my comments here?


or `false` *otherwise*


If we are going to do relative imports (I have some slight doubts about the approach), then this is very fine.


@crutchcorn sadly, yeah. I rather have to update it when we get someone that commits often. The GitHub API is a bit more confusing.


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


You can eliminate this set of braces now, right?


Here, you change ldap_simple_bind_s to ldap_bind_s. But, in your commit comment, the ldap_simple_bind_s should be changed to ldap_sasl_bind_s.


Seeking one more change, as noted inline.


It would be nice if the "value" string contained more info about what property you've failed to set, but unfortunately i think that's against ArgumentNullException's contract and what you're doing is what we should do.


@simonastoyanova nice!!! whenever you get a chance, could you write a short page in [the wiki](https://github.com/PerseusDL/tei-conversion-tools/wiki) about this?


We should also filter private networks such as the whole range of 127.0.0.0/8, 192.168.0.0/16, 172.16.0.0/12, 10.0.0.0/8 https://fr.wikipedia.org/wiki/R%C3%A9seau_priv%C3%A9


@janbar - I agree, but this extra feature is to support wmc, not mythtv. It would be a slight improvement on mythtv imo, but if you prefer no change for now, no problem.


Why you just don't delete this line, it is useless


The METOC feature classes don't have an identity field, which is causing this tool to fail. A solution would be:
```
                REQUIRED_FIELDS = ['symbolset', 'entity']
                OPTIONAL_FIELDS = ['context', 'modifier1', 'modifier2', 'echelon', \
			'mobility', 'array', 'indicator', 'operationalcondition', 'identity']
```


This change does not mention in your commit comment. I guess this is a valid change.


yes, tried latest. can you give me the installed packages?
im stuck in
include/QtCore/qcompilerdetection.h
'utility' file not found


Maybe there should be a new sub-title, because it is only targeted at content producers, not directly FreshRSS users


Does this have to be exported? I'm not seeing this method if I build off of master.


Yes. Done.


This is not needed, is it... and what if it fails... it's a bad place to have an error...


http://bugs.python.org/issue21238 is a good place to start


@jhbabon, cool!


90946736_SHRgt1_4567 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1776 of 1779 tests passed)

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


LGTM


looks good!


I had also looked at the md5 etc etc
I was gone for the afternoon (meeting a friend, dinner, a beer etc) but I will finish this evening the work on the crazy-lib handling so that it should work in tomorrow's build (CEST)


Consider telling that this parameter will be automatically created during install (and update?) and it is therefore a good idea to install FreshRSS from the real (routable) address to ensure that the detection will work


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Get rid of this logic using an iterator


Since it only impacted a test class I didn't think much of it.
But I filed an issue: https://github.com/vt-middleware/cryptacular/issues/23


oops, thanks.


Über den Test dazu (check_training)


@springmeyer
good catch!
Forgot to remove.

Shapefile/GeoJSON test fails, if git is setup to change Linux line endings to Windows line endings (default setting) on check out.

Checked out GeoJSON gets converted to Windows line endings, but test writes with Linux endings => file sizes (that's what the test checks) differ => test failed.

Luckily AppVeyor is setup to check out Linux line endings.


That issue is closed; filing a new ticket at bugs.python.org would be a lot better: keep it simple, and please include a small self contained code example so that we can see how to faithfully reproduce it. Thanks!


use Closeable instead


I was thinking about multiple points for each label, but I'm not sure how to fix the angle so that all the points stay in a horizontal line.  But I agree that we shouldn't worry about it too much unless there's a problem. I'll take out the centering for now.


Thanks


http://stackoverflow.com/questions/14083111/python3-encoding-declaration ;)


The old code was updating the buffer position to point after the varint, but the new code is not. This is causing errors in the Go compatibility tests.

This bug didn't show up in Perl testing because Perl never generates different tags.


I have only folowing packages: clang, clang-3.5, libclang-common-3.5-dev, libclang1-3.5, libllvm3.5, llvm-3.5, llvm-3.5-runtime


97647908-DocIssues4615-again (rglazer2) - x86_64-MacOS-10.9-clang: Tests Failed

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


If done manually, the Web site will also have to ping the hub


Yes, I haven't committed yet, but I made the change to all lowercases.


This version no longer exists in debian's pool: http://ftp.debian.org/debian/pool/main/d/debootstrap/

This page https://packages.qa.debian.org/d/debootstrap.html lists 1.0.67 as the stable version.

What's openwrt's policy here?


Hey I had the most amazing day of my life I had pizza I am so happy


Good article :+1: I have written a few comments. Let me know whether there is any special aspect for which you would like more feedback. The language could generally be improved, but I think it is good enough :-)


Env handling with default values can be abstracted away with https://github.com/kelseyhightower/envconfig ... I've used it for my [Go-based app on Heroku](https://github.com/srid/feelings/blob/master/feelings/config.go#L9).


Yup. There's lots of options here. Feel free to incorporate it if you're feeling adventurous.


should not be needed for a note project


Ran the following command:
`rails generate sorcery:install user_activation reset_password brute_force_protection http_basic_auth activity_logging external`


That's exactly what I proposed.


This include has been dropped in PR #307.  As stated by Sean, this was intended to support AF_IB addresses, but the verbs provider currently doesn't support AF_IB addresses.


@sameoldmadness
В данном случае ничего другого из неймспейса `BEM` в шаблонах не ожидалось, поэтому работает. На реальных проектах, где в `BEM` лежит локализация — да, стоит использовать твой вариант.


slides Alessandro added


Hello,

can you add 30702 ?

thanks


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

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

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

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

```


@BergWerkGIS aha, so the line ending issue must be the problem I encountered that caused me to do this https://github.com/mapnik/node-mapnik/commit/183d8d9a03057312805debc3a82a4d1c2d7eb879?


(pageSize && page > 1 this.cachedQueryTerm === params.term).

I propose this change.


> @neappx: Forum hosts though lock forums if they violate their ToS. I'm not for moderation in general by GitHub, my point was that having no content policy at all isn't a good move from GitHub's perspective, even though freedom of speech is good for the user (as long as people don't take it too far).

Many forum hosts follow an "anything legal is fine" policy, either formally or informally.

> Also, just because there is a risk doesn't mean you shouldn't try to reduce the possibility of the risk happening.

It does, if that attempt to "reduce the risk" results in a significantly worse tradeoff (which is almost always the case in these kind of situations, to be honest).

> With Content Policy I mean the classic ToS stuff you find on almost any site like G7 & 8 here on GitHub and not any specific filtering:

I am very strongly against such catch-all clauses (like 7). They allow exactly these kind of situations, and are not necessary when going by ethics rather than profitability.

> All my pro content policy posts should be read as "have them, but only enforce them if they really are harming someone."

Selective enforcement is the worst kind of enforcement. Because now you've given staff a wildcard to act against anybody or anything they don't like, and point at the ToS to say "well, we told you we could do that". And others will defend them.

> They should try to defend the user as far as they can and only enforce them if it's really bad for a major of the community or for them, not just because some people want a project to be removed.

They should not. That is also the responsibility of the maintainer(s).


Added comma?


This line seems weird for me. New `coordMaps` has been calculated in Line 307, which might be different from the old `coordMaps` in Line 305. `spherCoods` uses the new `coordMaps`, but `x` uses the old one. Adding those two variables together might rarely cause some numerical errors if their `coordMaps` don't match.


This is great whats the licensing ?


FWIW, I think Arun has AF_IB support ready to go.  He was delaying because we are seeing issues that traced back to ibacm, which traced further back to opensm, and which point the trail ended, but we're working on extending the trial with additional metaphors.


1718_setpoint_managers (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (2259 of 2289 tests passed)

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


fix: fixes #325


This commit LGTM.


Thanks for the suggestions.  Here you go: http://bugs.python.org/issue24758


I'll add comments :)


It's ridiculous the dance we have to do with NSNumbers...


I guess making explicily is for the floating bar is clearer. real_floating_bar_set_short_status


floating bar


No real reason beyond cosmetics, I just use var everywhere (almost)


dito


@jpkrohling yes, it is expected that the license headers in groovy files will have to be reformatted due to this change We have just a few of them.


I think we talked about this before but it's much more common (and safer) to use copy on all classes that might mutate (NSString, NSArray, NSSet, NSDictionary).


@sethtrain these aren't case sensitive, right?


now is called model (also for the function name), and you don't have multiple. So you just need the view parameter


Since 'session' is private shouldn't this be
```inline operator const CassSession * () const```
if at all?


`oninput` creates at least a toast for every character. I don't like how it would look and feel especially on mobile.


two on done_loadings?


Yes and no.

We _would_ want to submit 2 separate PRs (one to origin and another to
upstream), but we shouldn't wait to "cherry-pick later". Since this
feature breaks under certain conditions without the change to
`devstack.py`, we don't want to merge the feature without it. We also
don't want to wait around for edX to merge, as that _could_ take some
time. So you could imagine something like:

- PR to Stanford-Online
    - Commit 1: Restore LTI Upload
    - Commit 2: Disable HTTPS on devstack
- PR to edX
    - Commit 1: Disable HTTPS on devstack


this will have to adapt the changes on disconnect_directory_signals


:+1: Thanks for this! Holding my horses for a long time to not do it


RIght.  I didn't see any other example of case insensitivity language so I didn't know what to put.


So seeing now this, I think you can use disconnect_model_handlers to disconnect the signals as well, and that's it.


This was modified to ensure that it doesn't try to write the new file at all if ensure is telling it to remove it.  Basically doesn't cause the ruby type/provider to fire off if we're already trying to remove that zone.  And if we remove that zone, we should reload the firewall so that it sees that change.


1718_setpoint_managers (mbadams5) - i386-Windows-7-VisualStudio-12: OK (2268 of 2289 tests passed)

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


good point, with the boost::optional conversion


Initial commit for #5


That broke our working installation of jupyterhub running at `http://address/prefix/` Now almost every redirect has `base_url` twice. Reverting that one line fixed the issue


It was causing write and reload no matter what if rich_rules were present in firewalld_zone. Also provided better support for family, created method exec_firewall as firewall-cmd has issues with flat command line being sent to it from this ruby class with items such as --permanent. Implemented get-zone-of-interface and remove-interface from zone if it has been switched over to another interface. Updates the permanent as well as making the change on the fly. Also, important!... modified XML printout to use Pretty.compact = true as firewall-cmd fails at XML handling and if XML values, inside elements, are on separate lines rather than all one line with open and close elements, firewall-cmd that edits the interface using --permanent will delete all the text inside those problem elements. firewall-cmd's fault, I worked the support in for their poor XML implementation.


@esoliani I was wondering -- is it possible to only iterate the first 20 from the view, so we could always return the full list here?

I'm fine with it as-is too, I'll leave it to your choice.


Cool, no worries just wanted to note that in the PR for when we make some future updates to the formatting of this doc (to follow the same kind of layout as the [contacts endpoint](https://github.com/3dna/api_docs/blob/master/doc/contacts_api.md) doc)


thanks


i wouldnt put this on the nav bar, i dont really ever see that kind of thing there on other sites


Similarly for inline operator CassCluster *() const


Looks like a good entry from what I could tell. The code is actually koweblomke's at github.com/koweblomke/puppet-firewalld, forked from jpopelka's waiting on a pull acceptance, still doesn't seem to fix any attempt to create a firewalld_zone and then add rich_rules to it via the type/provider for rich_rules.  It seems to still think it is a new rule, so it re-applies it and then firewalld-cmd --reload unfortunately.  This needs to be fixed before calling this module version 1.0


I'm not sure i grok why this test is needed. What does it mean if it fails?


this.addAbility(ability) is missing.

Did you test this card?


Same reason as commit 60892d832357cdb35859007027e65beddaa29056
Looks like a good entry from what I could tell. The code is actually koweblomke's at github.com/koweblomke/puppet-firewalld, forked from jpopelka's waiting on a pull acceptance, still doesn't seem to fix any attempt to create a firewalld_zone and then add rich_rules to it via the type/provider for rich_rules. It seems to still think it is a new rule, so it re-applies it and then firewalld-cmd --reload unfortunately. This needs to be fixed before calling this module version 1.0


Steve,
Thanks!!  A few more pieces of the puzzle!!  I think I will hold back on any more comments on this commit as I think I can look at some of the code you referred to and try to absorb what DHH is trying to do.  I downloaded a simple publish/subscribe ruby program using the redis ruby gem (from Thoughtbot).  It is really simple but it is helping me to understand how the entire publish/subscribe flow is working with a bit more detail (along with reading the documentation on the publish and subscribe methods from rubydoc).  A few blogs game me more understanding of the Redis publish/subscribe implementation under Rails.  I read the Redis documentation on PUBLISH and SUBSCRIBE and I understand enough of what those commands are doing on the Redis server.  I did have one more question before closing.  Going back to the last line in the follow method -> stream_from "messages:#{data['message_id'].to_i}:comments" this is referring to the URL, correct?  Obviously "data" is a hash being passed to the method "follow", and "to_i" is being used to strip away any non-integer characters after the leading digits rendered by the value data['message_id'].


I know ... is just an comment


g


v


Header says it all.  Adding some required documentation and linting.


Steve,
By the way thanks a lot!!  You spent a lot of time answering my questions.  I appreciate it.


Also I wasn't able to get nested commands working, e.g.:
```
-> getTitle()
    -> title()
```
it throws `Error: Can't wait without a fiber`. Does this work for you?


This app is intended to work in python 2, thus the declaration.


please rename to IdeaDocumentationProvider


The accelerator assignments are producing some runtime messages on Linux and OSX:

11:22:50: Debug: Unknown accel modifier: 'left'
11:22:50: Debug: Unknown accel modifier: 'click/left'
11:22:50: Debug: Unrecognized accel key 'drag', accel string ignored.
11:22:50: Debug: Unknown accel modifier: 'left'
11:22:50: Debug: Unrecognized accel key 'click', accel string ignored.
11:22:50: Debug: Unknown accel modifier: 'right'
11:22:50: Debug: Unrecognized accel key 'click', accel string ignored.
11:22:50: Debug: Unknown accel modifier: 'left'
11:22:50: Debug: Unknown accel modifier: 'click/left'
11:22:50: Debug: Unrecognized accel key 'drag', accel string ignored.
11:22:50: Debug: Unknown accel modifier: 'left'
11:22:50: Debug: Unrecognized accel key 'click', accel string ignored.
11:22:50: Debug: Unknown accel modifier: 'right'
11:22:50: Debug: Unrecognized accel key 'click', accel string ignored.


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1774 of 1777 tests passed)

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


DATASTD-829 is the SubTask for this commit


Quotes were required as the invert rules are not actually booleans, they are strings which contain the boolean word.


@danielturek When you get a chance, maybe you can take a look at this?  I'd like to be able to pass the priors in directly using the nimbleCode format, because I think something like:

```r
rho ~ dgamma(10, 1)
sigGP ~ dunif(0, 1e5)
sigOE ~ dunif(0, 1e5)
```

is more consise than giving an argument like:

```r
list(RHO = quote("dgamma(10,1)"), SIGGP = quote("dunif(0, 1e5)"), SIGOE = quote("dunif(0, 1e5)")
```

even though that approach has advantages in ease of manipulation.

I've attempted to code the former here, but as you can see this doesn't work.


@pivanov  много е готино пич, но няма никаква промяна в хтмл-а


Nullability annotations.


How do u get the mod on tablet


maybe we can make an `'ALL'` or `'NATIONAL'` mode for the `rank` function, and put it in with [`school.ranks`](https://github.com/WorldBank-Transport/edudash/commit/46d3da715aabd3ad2cfb8cf848f706b2474b58ac#diff-510df911a16a7a4c87c23cc164f9e2e9R184) if that makes sense?

I haven't looked at how `drawNationalRanking` works yet, so I might not be seeing the whole picture. Would it be hard to move it to a directive so we can connect it from the view instead of here?


constant............ , please fix


The only important part to this commit is the resource target supporting an empty value.  This was so that it can bypass the safe_insync in property.rb and then back to nil when writing xml file, this is so we can comment out target and have it actually notice and flush and write a new zonefile.

All the debug puts lines were removed in the future commits.


Don't forget to create a file named `author.txt` in the "..\mage\Utils\" directory in which you only write your username. This will be used for @author in the generated file.


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


You don't need these non-capturing groups here.


Done –> https://github.com/heroku/spew/commits/use_envconfig


@joepie91
>Selective enforcement is the worst kind of enforcement.

Absolutely. No clear rule that you might be breaking yet suddenly you get content removed.
And similar content is allowed to stay. This is exactly the problem at Reddit.
Selective enforcement sucks


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

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


I'd rather have no curlies here - a single-line if would do just fine.


final


A blank line


> … and your releases are guaranteed to be unromantic and unsentimental

Cute :smile:


"The opponent is chosen when the "at end of turn" triggered ability resolves."
So you can't use the TargetOpponent().
You have to handle it during init of the continuous effect or add a controlling oneShotEffect.


Dobre e da sa v edin commit dvata file!


the commit will kill old savegames - you have to change the savegame version too


Use {this} instead of the card name in the rule test.


Inconsistent commit messages:

* Some messages end with period, some don't.
* Some commit messages use imperative mood ("Prevent the lib/constants.php file from being commited to git"), some don't ("Added another step to the running things section of README.md").


Harmless I suppose.

We seem not yet agreed that this development is a good idea, and I may end
up hiding it.

PRL

On Thu, Jul 30, 2015 at 3:29 PM, Leland Lucius <notifications@github.com>
wrote:

> The accelerator assignments are producing some runtime messages on Linux
> and OSX:
>
> 11:22:50: Debug: Unknown accel modifier: 'left'
> 11:22:50: Debug: Unknown accel modifier: 'click/left'
> 11:22:50: Debug: Unrecognized accel key 'drag', accel string ignored.
> 11:22:50: Debug: Unknown accel modifier: 'left'
> 11:22:50: Debug: Unrecognized accel key 'click', accel string ignored.
> 11:22:50: Debug: Unknown accel modifier: 'right'
> 11:22:50: Debug: Unrecognized accel key 'click', accel string ignored.
> 11:22:50: Debug: Unknown accel modifier: 'left'
> 11:22:50: Debug: Unknown accel modifier: 'click/left'
> 11:22:50: Debug: Unrecognized accel key 'drag', accel string ignored.
> 11:22:50: Debug: Unknown accel modifier: 'left'
> 11:22:50: Debug: Unrecognized accel key 'click', accel string ignored.
> 11:22:50: Debug: Unknown accel modifier: 'right'
> 11:22:50: Debug: Unrecognized accel key 'click', accel string ignored.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/audacity/audacity/commit/197700967ffbfceca46e43a48c673b2398736226#commitcomment-12452558>
> .
>


In release 2.0.0 I have to change this from:
```return $soldier->player;```
to:
```return $soldier;```
To get live scoreboard and chat to work, otherwise it throws the following exception:
```production.ERROR: exception 'ErrorException' with message 'Trying to get property of non-object' in /var/www/BFAdminCP/app/bfacp/Helpers/Main.php:742```

```
Stack trace:
#0 /var/www/BFAdminCP/app/bfacp/Helpers/Main.php(742): Illuminate\Exception\Handler->handleError(8, 'Trying to get p...', '/var/www/BFAdmi...', 742, Array)
#1 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Support/Facades/Facade.php(211): BFACP\Helpers\Main->getAdminPlayer(Object(BFACP\Account\User), '1')
#2 /var/www/BFAdminCP/app/bfacp/Repositories/Scoreboard/LiveServerRepository.php(1201): Illuminate\Support\Facades\Facade::__callStatic('getAdminPlayer', Array)
#3 /var/www/BFAdminCP/app/bfacp/Repositories/Scoreboard/LiveServerRepository.php(1201): BFACP\Facades\Main::getAdminPlayer(Object(BFACP\Account\User), '1')
#4 /var/www/BFAdminCP/app/bfacp/Repositories/Scoreboard/LiveServerRepository.php(159): BFACP\Repositories\Scoreboard\LiveServerRepository->setAdmin()
#5 /var/www/BFAdminCP/app/bfacp/Http/Controllers/Api/ServersController.php(97): BFACP\Repositories\Scoreboard\LiveServerRepository->__construct(Object(BFACP\Battlefield\Server))
#6 [internal function]: BFACP\Http\Controllers\Api\ServersController->scoreboard('2')
#7 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Routing/Controller.php(231): call_user_func_array(Array, Array)
#8 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Routing/ControllerDispatcher.php(93): Illuminate\Routing\Controller->callAction('scoreboard', Array)
#9 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Routing/ControllerDispatcher.php(62): Illuminate\Routing\ControllerDispatcher->call(Object(BFACP\Http\Controllers\Api\ServersController), Object(Dingo\Api\Routing\Route), 'scoreboard')
#10 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Routing/Router.php(967): Illuminate\Routing\ControllerDispatcher->dispatch(Object(Dingo\Api\Routing\Route), Object(Illuminate\Http\Request), 'BFACP\\Http\\Cont...', 'scoreboard')
#11 [internal function]: Illuminate\Routing\Router->Illuminate\Routing\{closure}('2')
#12 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Routing/Route.php(109): call_user_func_array(Object(Closure), Array)
#13 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Routing/Router.php(1033): Illuminate\Routing\Route->run(Object(Illuminate\Http\Request))
#14 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Routing/Router.php(1001): Illuminate\Routing\Router->dispatchToRoute(Object(Illuminate\Http\Request))
#15 /var/www/BFAdminCP/vendor/dingo/api/src/Routing/Router.php(216): Illuminate\Routing\Router->dispatch(Object(Illuminate\Http\Request))
#16 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(775): Dingo\Api\Routing\Router->dispatch(Object(Illuminate\Http\Request))
#17 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(745): Illuminate\Foundation\Application->dispatch(Object(Illuminate\Http\Request))
#18 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Session/Middleware.php(72): Illuminate\Foundation\Application->handle(Object(Illuminate\Http\Request), 1, true)
#19 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Cookie/Queue.php(47): Illuminate\Session\Middleware->handle(Object(Illuminate\Http\Request), 1, true)
#20 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Cookie/Guard.php(51): Illuminate\Cookie\Queue->handle(Object(Illuminate\Http\Request), 1, true)
#21 /var/www/BFAdminCP/vendor/stack/builder/src/Stack/StackedHttpKernel.php(23): Illuminate\Cookie\Guard->handle(Object(Illuminate\Http\Request), 1, true)
#22 /var/www/BFAdminCP/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(641): Stack\StackedHttpKernel->handle(Object(Illuminate\Http\Request))
#23 /var/www/BFAdminCP/public/index.php(49): Illuminate\Foundation\Application->run()
#24 {main} [] []
```

HTH


Yes, the view should show the number of records. In fact they want to have the 100 top school.


I don't think these methods improve readability - they are really simple and we can actually benefit from having their code inlined.


Unless you can understand what this comment is saying without looking at the code you might want to rephrase it a bit. I can't parse it :)


No database creation statements.


Outcome should be PutManaInPool


warum gibst du TRNode nicht einfach einen konstruktor? dann kann man sich das initialisieren hier sparen?!?


No dots at the end of effect rule texts


There's no need to create another stringbuilder - you can reuse the one you already have.


It would be better if the error stated which specific fields were missing (ex. 'entity,' and 'symbolset'


Because you are in the apply of an continuous effect this message will be posted multiple times during a turn.


Commit is not atomic (http://www.freshconsulting.com/atomic-commits/):

commit message says "Added another step to the running things section of README.md", but actually it does two things:

* it adds another mentioned step;
* it also removes some dependency system.


I'd better replace this code with something like ```Character.isWhitespace(messagePart.charAt(0))```. The reason for that is covering more whitespace characters than you have written here. E.g. you don't cover newlines on windows with this code.


closes #11


You call the LdapSimpleBind after LdapUtils::. In line 199, you call the LdapInit without LdapUtils::. Not consistent?


Scrapping all links from From particular website


All the array _matching items had issues with running insync? and having a good, matching result.  I added these so that if they're set to nil or empty(they're always empty, not nil, due to the way the array_matching all is done) they will be checked against the IS value to see if it is nil, if it is, do nothing; else, suggest to provider that the value has changed. That way we can comment out values or set them to [] and it will notice it should flush and write zonefile. Default insync method checks and if .empty? it returns true and does not submit for flush and write, not what we want.


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

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


Again, the boolean here needs to be a string.


Commit message is very generic: "create functionality" might mean a lot of things.


@stmeissner please remove this file


@crom35 Please share the full sketch as a gist.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.867.ee3c6ba7f68e6c8c5b637693fff18046dbda9b06](http://build.palaso.org/viewLog.html?buildId=60266&buildTypeId=bt431) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 397, ignored: 3 Build time: 00:01:38
### Failed tests
```
GlyssenTests.dll: GlyssenTests.Utilities.VerseRefExtensionsTests.SendScrReference: <no details avaliable>

```


I just discovered that, at least with hardware transactions, padding the random state to a full cacheline has a significant effect when it is heavily accessed and changed.  Hardware transactions may be magnifying the effect in my case, but the place to ensure that in your code is here.  There are two words in the header of the byte array, so if 48 is used instead of 8 here [1] it fills a cacheline (if aligned, it will be at least word aligned and we only need a word of state).  You can do something similar by doing `U.new (cn * f)` for some factor `f` or changing the type to be some unboxed tuple...

Anyway, you might play around with this.  If you have `perf` on your system you will know it is working by seeing a decrease in L1 cache misses when running with the padding.

[1]: https://github.com/cchalmers/pcg-random/blob/pure/src/System/Random/PCG/Fast/Pure.hs#L185


When we call `[self okButtonEnabled:YES]` we seem to always call `[self.passwordTextField becomeFirstResponder];` right after it. Is there any reason that we can't move that code inside the `okButtonEnabled` function?


@DanBrooker I mistakenly did this with npm hence the 1.6.1 tag. My bad. This is my first atom package to contribute to.


Function undocumented. Personally for me parameter `$post` is not intuitive.

Take a look at http://www.phpdoc.org/docs/latest/references/phpdoc/tags/param.html and other tags.


O que é isso ???


Seems like this function is doing too much.


@nervgh & @panzj: I really appreciate the incredible work done in this library, which has been a huge godsend in my own development work—which is why I hesitate to mention:

It's quite important that a library such as this adheres to its semantic versioning standards, which means this should not have been included in a PATCH version, but instead should have caused a bump to the next MINOR version, i.e., 1.2.0 (no need for a MAJOR version bump here, @raoulus).

Otherwise, myself and many others pulling in this lib via bower get an unexpected surprise when updating (if specifying anything resolving to 1.1.7 and newer). The following should, under normal circumstances, keep the developer safe from breaking changes:

`"angular-file-upload": "~1.1.5"`

But with this change landing in a PATCH revision (1.1.7), it requires an update to:

`"angular-file-upload": "~1.1.7"`

and a tweak to the expected lib path in order to get things up and running again—all of which was necessary in my team's CICD environment.

Anyway, my apologies if this comes across as overly critical or harsh. It is my attempt to make clear statements for anyone seeking information about this change.

Thank you again for all your fantastic work in this library and your contributions to OSS.


What do you think of adding an optional list of extra valid contents to
RequestsTransport.__init__?

On Thu, Jul 30, 2015 at 9:34 AM Alexander D. Kanevskiy <
notifications@github.com> wrote:

> Not really. I've end-up with solution like this:
>
> # Message plugin to handle xom+xml SOAP repliesfrom suds.plugin import MessagePlugin
>
> class XOMPlugin(MessagePlugin):
>     def received(self, context):
>         answer = context.reply.decode()
>         xml_message = re.search(
>             r'(<soap\:Envelope.*<\/soap\:Envelope>)\r?\n', answer, re.M)
>         if xml_message:
>             reply = xml_message.group(1) + '\n'
>             context.reply = reply.encode()
>
> client = suds.client.Client(api_url, plugins=[XOMPlugin()])
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/armooo/suds_requests/commit/6af6737aa0b49232e5faf7845aed256114587ab0#commitcomment-12448703>
> .
>


Good catch! Just changed that.


There's no good way to write a general swap function or macro in ANSI C89.  GCC has typeof(), but this is still trying to build on old old stuff.

Also, I hate TUPLE!.  What it is, is a value cell is the size of 4 platform pointers, with one of those pointers used for the value's header.  Then it uses the space of the other 3 to try and fit a limited-size vector of bytes inside of it.  It looks like a SERIES! because you can pick values out of it *(pick 10.20.30.40 3 => 30)*...but you can't get a position into it.  And unlike other series-like things, it has value semantics instead of reference semantics.

This wacky type was used for colors (R.G.B or R.G.B.A), IP addresses, and versioning numbers.  No one these days likes using decimal numbers for colors anyway, we have IPV6, and what happens if a version component gets bigger than 255?  My dislike of TUPLE! probably subliminally (or not so subliminally) contributes to me sort of wishing it away.  Perhaps I wanted it to break.  :-/

But no, we should keep everything working until such time as conscious changes are made.  Still, that doesn't mean I'm going to bend over backwards to improve the darned thing.


maybe call this `batchFuture`


@joepie91
Man, you really are making this hard for me :(

>Many forum and blog hosts follow an "anything legal is fine" policy, either formally or informally.

Really? I just checked ProBoard's, FreeForums/LeFora (CrowdGather) and Forumoation (the first four entries on Google for "free forum host" all have a clause similar to GitHub's one. And that's what I'm used to from free web hosts. If you really have a service that says "anything legal is fine", then try common things to piss them off like abusing your forum as a download portal (can even just be legal content; high traffic is important), getting involved in a shitstorm (e.g. (Anti-)GamerGate) or just getting DDoSed (maybe more than one time) and see if they really let you on their platform, especially for free. If they do, they are great.

>It does, if that attempt to "reduce the risk" results in a significantly worse tradeoff (which is almost always the case in these kind of situations, to be honest).

Depends on the case and your definition of a "worse tradeoff".

>I am very strongly against such catch-all clauses (like 7). They allow exactly these kind of situations, and are not necessary when going by ethics rather than profitability.

It's the run-off-the-mill clause many services have that allows them to remove problematic stuff if they have to, even if it's not legally required but still for their own sake. It kinda sucks, but it's better than writing "If your stuff may get us in trouble, we may delete it" and it may actually give you a way to appeal it.

@joepie91 & @smeezekitty

>Selective enforcement is the worst kind of enforcement. Because now you've given staff a wildcard to act against anybody or anything they don't like, and point at the ToS to say "well, we told you we could do that". And others will defend them.

>They should not. That is also the responsibility of the maintainer(s).

>Absolutely. No clear rule that you might be breaking yet suddenly you get content removed.
And similar content is allowed to stay. This is exactly the problem at Reddit.
Selective enforcement sucks

Well, it sucks, but should they hurt themselves and maybe even other users? Though the somewhat recent Reddit drama with fatpeoplehate was properly the opposite of protecting themselves (they harmed themselves). It really depends of the case which action is better for the provider/company and depending of what kind of service you are running, other users. The Reddit one surely was a fail.


@iapazmino ^^


This isn't a big deal, more so a comment. The function name isn't a great descriptor of what the function is actually doing. If I was reading through the code, i'd think that this function enables or disables ONLY the ok button, which is only partially true -- I think a better name might be `setUserInteractionEnabled` ... which more accurately describes what the method is doing.


Seems like well formatted html (Y).


I will pull this in. There does need to be a script_end_procedure at the end so the script logs the usage statistics.


ALL GLORY TO CHAIRMAN PAO


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

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

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

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

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

```


Inconsistent CSS style. In other lines you do insert space after ':', e.g. "width:100%;"


@caseyedavis12 @sroosa Do all records that have `<pbcoreAnnotation annotationType='Outside URL'>` also have `<pbcoreAnnotation annotationType='organization'>`?


How about adding an `isLeaf()` method to `SqlStageExecution`


frf\


s


@monicao thanks for the review, fixed the code


The rank function has a problem of how to solve order the dataset. I will leave it for the future.

The drawNationalRank is an old nasty function from the previous code. It creates the chart.


I'll need to look at this in more detail as for the final packer related builds the box should be the one that packer produces not the download from bento.


Yes, it should be exported. Thanks.


req.query will always be an object, empty if no query string. This is more windy than need be.


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

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


BTW: I would like to drop that script here in future and its onmouseover and onmouseout events and replace it by CSS rules.

Why?
  * It would work even if javascript is disabled in browser.
  * possibilitys to filter the search interactively by clicking on table cells (would be new feature), needs some storing of current search working (again) (or long a-tags with params which I would like to avoid in favor of shorter pagesizes) instead of going always to task details page.

Something like:
```css
.descbox{display:none;position:absolute;}
#task_listtable{ position:relative;} /* for setting a reference (xy)-point for descbox */
/* or */
#task_listtable tr { position:relative;} /* for setting a reference (xy)-point for descbox */
#task_listtable tr:hover .descbox {display:block;top:60px;left:20px;}
```


Why not promises? (or .create)?


Very obscure variable names `c` and `t`.


this probably shouldn't be in this commit


Why not use next as the second argument to .then?


I think the `#OS X' comment could be dropped, and all the *something patterns should be sorted alphabetically (whether or not ignoring case, I don't really care).

So, I think, the lines
```
*.exe
*.dSYM
```
added in commits 15518fbf and 25bd4c42 should be moved as well.


Any tool that removes 7,000 of maintained code is pretty nice!  I'd be willing to throw in developer cycles to converting the existing refs of sexplib -> deriving, since it appears our use of it is pretty superficial.  I wish cstruct were ppx already, but it looks like it's close: https://github.com/mirage/ocaml-cstruct/pull/59.

The bottom line for me is temporary _oasis hacks are fine if there's a clear path to a ppx-only world.  The only thing that concerns me is the NetKAT syntax extension.  One of camlp4's virtues is it can compile arbitrary grammars.  NetKAT is an arbitrary grammar - it's not valid OCaml - and so it's unclear to me how you could re-do it in ppx.

But maybe that's a moot point.  Maybe Rest+Json is the way to do NetKAT from here on out.


`validate` is generic term. Not so clear what this function validates.


@afred as far as I know, all records coming out of the AMS have to have <pbcoreAnnotation annotationType='organization'>


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.868.ee3c6ba7f68e6c8c5b637693fff18046dbda9b06](http://build.palaso.org/viewLog.html?buildId=60268&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 398, ignored: 3 Build time: 00:02:39


You need to add the 'dist_' prefix here, otherwise the man page will not be included in the tarball. Something like this:

dist_man3_MANS = doc/libpaf-ppr.3


What if something else throws an error (besides your throw on line 39)?


Look into using lodash.extend.


done


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.869.7668c6933d4539001c6fa6531af76702c52a78b1](http://build.palaso.org/viewLog.html?buildId=60269&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Exit code 1 (new)) Build time: 00:00:35


HTML file is indeted with spaces whereas JavaScript and CSS files are indented with tabs - inconsistent?


will do


BTW:
The line
```
john-local.conf
```
added in commit cad538c4 should be moved immediately after the
```
Makefile
```
line.


I tell that few lines later. I don't know in which case people install FreshRSS from a non real address. Maybe we should consider to give possibility to edit this value during installation / in the administration panel?


Actually, horizontal/vertical collide fields were supposed to identify "incoming direction" of collision, i.e. vertical_collide value of 0x01 meant "floor collision", and value of 0x02 meant "ceiling collision". Same with horizontal_collide, value of 0x01 supposed to mean "front collision", 0x02 meant "back collision", 0x04 meant "right collision" and 0x08 meant "left collision".

Currently it's unimplemented, but it may come in very handy when we'll program position corrector to "turn" Lara against the wall.


Without these 4 lines, the task-bar uze icon is gone :/


Looks good. I'm pretty sure I would have made the same mistake, just by looking at the code I get lost already :-p


There is a way to represent a collection via query string.


I will open a ticket to filter those addresses


@robbat2 this should meet your need right?


Good job


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.870.f26fd83575aa2174f4b1ad92a32792a7d790d334](http://build.palaso.org/viewLog.html?buildId=60271&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 397, ignored: 4 Build time: 00:02:35


Possible SQL injection. Seems like $_POST parameters are not escaped in `insert()` function too.


Oh, you already figured it out by yourself... So only thing to fix here is to specify horizontal collision types.


This looks good. Recommend that you push live.


It allows us to do the update via a join, rather than either doing N updates or encoding the update in N/M huge case expressions.


Too bad this went away. I was so proud of this useless but cute feature ;P


For me, sounds weird remove the header in the validation hehe


Doesn't seem like this is used in any way.


Are you sure we won't need these? I remember Gh0stBlade said that he is unable to implement camera targets because of lack of lookat functionality for matrix operations. So this (as well as inverseFrustum) could be handy.


http://blaze.pydata.org/en/latest/uri.html gives examples of table URI but http://docs.sqlalchemy.org/en/latest/core/engines.html gives examples of database URI


nevermind, this is right


hmm. that's cool. didn't know you could do that in Github. #YouOnlyLearnAlways


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/73303584


insert error handling logic


@danielgindi my peer QA engnieer will start to test it soon(late this week, actually today I guess) after I finish the migration. If I found issues I would report ASAP. It seems working right now in my bar charts library.


These are fine to leave in as examples, just wrap them in a "if __name__ == '__main__':" block.


用&x#D;&x#A;换行不行么


请不要使用 Windows 换行符以适应跨平台。


在其他地方显示会不正常么（突然发现github也是即时更新留言的）


Remove code rather than comment it out.


Testei. Funcionou normalmente, só alterei a entrada, o restante funcionou
do mesmo jeito. Não deu certo aí?

2015-07-29 22:47 GMT-03:00 Pedro Pisandelli <notifications@github.com>:

> @AndrePaiva <https://github.com/AndrePaiva> você testou esse código?
> Acredito que ele não vai retornar nada...
> Alguma sugestão a mais?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AndrePaiva/trainee2015/commit/8c1242584f5b2e69ba25124ea86b4ca83ec2d9d2#commitcomment-12434791>
> .
>


Kill this.


I like the changes. Let's go ahead and merge.


Com as minúsculas eu não testei, mas vi o método pra deixar todas
maiúsculas.

Em 29 de julho de 2015 23:06, Andre Paiva <andrep3@gmail.com> escreveu:

> Testei. Funcionou normalmente, só alterei a entrada, o restante funcionou
> do mesmo jeito. Não deu certo aí?
>
> 2015-07-29 22:47 GMT-03:00 Pedro Pisandelli <notifications@github.com>:
>
>> @AndrePaiva <https://github.com/AndrePaiva> você testou esse código?
>> Acredito que ele não vai retornar nada...
>> Alguma sugestão a mais?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/AndrePaiva/trainee2015/commit/8c1242584f5b2e69ba25124ea86b4ca83ec2d9d2#commitcomment-12434791>
>> .
>>
>
>


没尝试过，请问你用 `\r\n` 显示的是一个换行符还是两个换行符？

无论如何正如我所说使用 Windows 换行符是“bad practice”。而且使用 `\n` 可以省一个字节。


编译时，要求必须安装thrift，否则编译报错，但是实际上应该是可选项，我并不想使用thrift。可以考虑可配置吗？


You might try naming your local branch something like "master-jk-submodules" so that there is no "/" in the branch name.  The tests may have some form of strange dependency that the local branch must not include a "/".


三维向量和三维坐标的区别在哪……


Can we maybe ask the widget strategy what to include in the fragment request? That way if it's the blog or something you could do something with it. We could have a special loader that adds the model name and what its ID is so we can always have the subject content accessible and the current page.


@latobarita: retry


本质上没区别。

替换原因是一会儿使用向量一会儿使用坐标比较搞，而且正如你指出的那个地方所指定的并不是绝对坐标。因此干脆全部使用向量。


I'm also getting getting random failures in this test, but in the 192-bits check. I suspect this is because `Bytes()` wont have 40 bytes when it has leading zeroes.

Maybe we can just check if the keys are not nil (or zero values).


Extract utility methods such as these in a utility class StringChecks so that you can simply invoke them as StringChecks.isNullOrEmpty(string);


Same as above comment


@darkain84
I recognized the issue. And I planned to find the way to reject the request while reading request stream.


Why can we say
`sentence = text.toLowercase();`


이건 내잘못인가봄.....ㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎㅎ
살려주셈...


`sizeof(REBCNT) == 4` is a pervasive premise of the system:

https://github.com/rebol/rebol/blob/25033f897b2bd466068d7663563cd3ff64740b94/src/include/reb-c.h#L140

If it's broken then nothing would work, the interpreter would refuse to start.  So In terms of the "how negligible risk is it to change" department, this one is far under my radar.  There'd be a billion commits if I stressed over it... :-/

(The one question might be if it should be REBINT vs. REBCNT)


Fix issue #62 , #63 , Deliver enhancement for issue #64 , #65 , #66


I've included a bunch of this work upstream. I'm sure I missed stuff though, and other parts I didn't understand sufficiently to include


space between `for` and `(`


We already did so much work in WordFrequency to count frequencies. Reuse that? Let's extract wordFrequency logic in `Strings` and use that in `WordFrequency` and this class


Yes, I have not doubt about `sizeof(REBCNT) == 4`, just thought this change was orthogonal to this commit. But not a big deal.


I agree with your equation: `Xb,d = Xg,d . R'`, I have the same thing in my derivation as step 4.  I'm not sure I understand how you're getting the A and B matrices though.  Your second equation is the standard equation for a linear system, so I think jumping there from your first equation sort of inherently assumes linearity.

The state equation you want (if it's linear) is
`X_dot = A.X + B.u`
Where A and B are constant matrices.  You need to derive this by summing the forces on the robot and relating them to acceleration.  When doing this, the result I got was that A is a function of `d theta / dt`.  What did you do to get your values for A and B?


@hxgdzyuyi 已 merge :)


add a space between `}` and `else`


add a space between `for` and `(`


Class added.


In the javadoc, it's important to mention that Method tokenizes the sentence on *whitespace* and counts number of times each word appears.


this change is for export_simulation / projutils app


that's not correct, see above


at this moment, 'migration_dirty_pages' is the number of remote dirty pages


Okay that is as I suspected.  So it suggests it should go through with the Mark *([it didn't before](https://github.com/zsx/r3/blob/d0811786bb64016e41a192d63ea10d8754021336/src/core/m-gc.c#L521)...)* and just not free the data during the sweep.  That looks to be already covered in [Free_Series()](https://github.com/zsx/r3/blob/d0811786bb64016e41a192d63ea10d8754021336/src/core/m-pools.c#L575), so we can remove this check, correct?


I noticed this on other files, is this something that's meant to be wrapped in a ${} or is it python code that should just have a % at the start?


Actually, I'm still a little confused. Does this mean each thread will execute the ``for`` loop separately? In this case, I would have to use the thread IDs to index different FSRs aking to what one might do in a CUDA kernel. Or is OpenMP smart enough to unroll the loop across the threads?


Do we really want to have this class available in the DataModel component. I believe it's something more for DataModelServices.


Beautiful!


Could probably use itertools.cycle(api) instead of [api]*len()


TeamCity FubarDev.AfA :: Test-Build für Pull-Requests [Build 8](https://build.fubar-dev.com/viewLog.html?buildId=67&buildTypeId=FubarDevAfA_TestBuildFRPullRequests&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 10 Build time: 00:00:14


Please elaborate on *Probability of each word* - You may say
`Computes probability of occurrence of a certain word as (number of times the word occurs / total number of words)`


Pasting code that uses standard physical tab width of 8, indent of 4 in 0.3.10:
![image](https://cloud.githubusercontent.com/assets/11798751/8974497/3f33c06a-3640-11e5-89ed-560e57a78adb.png)
You can see there are a few glitches (which is why I started investigating the bugs)


this was causing some weird errors when i ran rspec - since its a rails app you don't need to worry about AS being missing :)


variable name should make sense. `var` should be renamed to `probability`


My hesitation with changing the ordering is primarily just to stay compatible with any future implementations of unums in hardware (which of course don't exist yet). However I'm already thinking about variations, so I think the ordering is still up for negotiation.

When you get about halfway through the book, he describes a 64 bit unum that includes flags for NaN, etc... I thought it was strange that there was metadata split up, and I'm open to a reorganization of those bits. Lets figure out the best layout here and convince the hardware designers to follow our lead :)

> On Jul 29, 2015, at 8:49 PM, Scott P. Jones <notifications@github.com> wrote:
>
> Would it be possible to rearrange the format, for Julia's purposes?
> I'm assuming that the fractional part is not 2s complement, but is it normalized?
> (Sorry, I haven't gotten that far in the book! Working on it!)
> If I were doing this for packing purposes, and the fraction was not normalized, I'd probably set things something like:
> | fraction | exponent | signbit | ubit | exponent size - 1 | fraction size - 1 |
> to make it easier to mask off the sign bit, and I might be able to use different type bytes (from the packing format) to represent the simplest numbers. This may be totally screwed up though!
>
> —
> Reply to this email directly or view it on GitHub.
>


Team: One thing I discovered after Title Source came in, is that our datatype for the MSRP should be a 'decimal'. Double is a floating point while, decimal is a fixed point number. Currency should use a fixed point like decimal so improper rounding wouldn't occur when doing math. It doesn't effect our project because we're not calculating with the price, but if we wanted to use the data elsewhere in a larger application with big dollars....etc.


Please do not hard code. I don't want too many icons in my status bar


Missed one: https://github.com/clark800/ripple-lib/blob/c88334cc7ae70b513b9ea51fae203f2cf169c269/test/fixtures/api/responses/get-transactions.json#L112-L116

assert-diff is broken, and the diff is misleading


Please see how to write javadoc. You should use `@param @return @throws` etc


JavaDoc doesn't make sense. Please update it.


Just a heads up that using C# 6 code will probably force us to require .Net 4.6 instead of the current 4.0, which means Windows XP users will no longer be able to use Pinta.  (4.0 is the last version available for XP.)  I know Mono has supported C# 6 for a while, but I don't know if it's been long enough that the versions of Mono available in various Linux distro repositories have been updated to it.


I *think* we do. Clang runs on Windows now and is trying to be MSVC compatible. Plus there is clang-cl.
I will revisit this again though.


I know it's just a test, but it's always a good habit to close the stream.


`"RE"` should be `technic.receiver` (or whatever the variable name was).


Why make explicit call to countWordFrequency? Client code should just be able to pass text to `TermVector` and get Vector. See spreadsheet for TermVector usage.


first github


Missed one: https://github.com/ripple/ripple-lib/blob/c88334cc7ae70b513b9ea51fae203f2cf169c269/test/fixtures/api/responses/get-transactions.json#L112-L116

assert-diff is broken, and the diff is misleading


Getting an error from this @ line 103: 'for' limit must be a number.


@pedro-guimaraes @NyKaren @jow57 caso algum dia commitem na sua master como fiz por desatenção, caso branch correta não tenha sido criada ou vazia, então criem-na e deem `git push origim branchCorreta`, e para atualizar o repo local: `git fetch origin` e `git pull origin master` `git pull origin novaBranch`, que o repo local vai ficar certo


Thanks for the reminder. I'll be picking this up again.


Thanks!  Will try this out at home.


Thanks.  `<aside>` I got an email notice of your comment.  I may have set up a github preference to receive such notices and forgotten about it.  Ah!  Look under your profile, settings, Notification center, if you would like to receive emails when someone comments on your P/R's.  If you don't want such emails, please disregard.


@jnealtowns It's strange that we got latest from ONL github yesterday and "make onl-x86", the kernel config is still not including the needed USB config (though we can see the USB config is already set via browsing the github). Do you know what may go wrong? Does the building process do not get the latest kernel config? Thanks.


Works locally, but gives me a 403 when run from Travis. Not sure yet what I will do about it.


$(CURDIR) is only available in GNU Make, From what I can see the rest of the Makefile is POSIX standard.
What is the target for this Makefile? Should we find for an alternative to $(CURDIR)?


Pasting code into 0.4, with #11719, but not this commit:
![image](https://cloud.githubusercontent.com/assets/11798751/8974738/5198c43c-3643-11e5-9423-5d48d0daa150.png)

Pasting code into 0.4 master, with this commit:
![image](https://cloud.githubusercontent.com/assets/11798751/8974675/7c97bc16-3642-11e5-82e2-37d7ae018d07.png)

Since #12369 removed the global for the default value, there is no way now to get it working correctly again.


Why was this commit made? It seems to break setting your collection as a hash. [According to this documentation, you should be allowed to do that.](http://jekyllrb.com/docs/collections/#step-1-tell-jekyll-to-read-in-your-collection)


related #11


Since these codes are for profiling, although they are relatively lightweight, they are on the critical path, is it better and more elegant to put them under some #ifdef logic, and compile and turn on them on demand


Added </body> and </html> tags


Added two end tags


l;l;l;l;l


The breakdown of the 3 fields in an ordinary (non-UNWORD) any-word are like this:

    typedef struct Reb_Word {
        REBCNT	sym;		// Index of the word's symbol
        REBINT	index;		// Index of the word in the frame
        REBSER	*frame;		// Frame in which the word is defined
    } REBWRD;

The symbol index of the word of course indicates its unique symbol ID for the UTF8 string name it has *(which takes case insensitivity into account...so case-insensitivity for lookup is done as another step above that)*.

If the word is bound, then the frame is not a NULL pointer.  *(And in debug builds only, the index is set to be INDEX_WORD_UNBOUND for safety...as you recently found out.)*

But if the "frame" pointer is not NULL, then some combination of `frame+index` is supposed to be able to get you to a REBVAL*...what one might think of as the "variable" the word is bound to.  Something I learned early on in looking at the source was that `frame` is not necessarily a pointer to a "frame" as described here:

https://github.com/rebol/rebol/blob/25033f897b2bd466068d7663563cd3ff64740b94/src/core/c-frame.c#L30

If it were, then all you'd need to do would be to say BLK_SKIP(frame, index) and you'd have your variable.  *(Modulo concerns for writers of checking to see if that word in the frame is protected... as you more recently found out.)*

But then there are function calls, and for an ordinary FUNCTION!, Rebol does not allocate a "real frame" (which would essentially be an OBJECT!) for its arguments and locals.  The idea is that it would be too expensive and then you'd have to garbage collect them.  But also, if each function call required all the words in the body to have their *frame* pointers adjusted, you'd have to be working with a copy of the function body on each call where all the words had been modified!

*(Note: That's what actually happens with CLOSURE...an object gets allocated, and the body has to be copied and fixed up before running it!  :-/)*

So the trick that Rebol uses is to not copy the body, and to put negative index numbers for all the locals and arguments.  Then the "frame" pointer is made to point to the list of unwords that was processed from the spec, and used for type checking and such when a function is called.  Any identifying series would do, but historically Rebol did not make a copy of the spec or body when you passed them into `make function!`... so you couldn't uniquely identify which function was being called from either of those.  *(This is dangerous, and I believe the copy should be made by Rebol, and once it is you could pick any one of the three.)*

But the point of being able to identify the function is that then you can look up the stack to find the most recent invocation (if any) and be able to get that recent invocation's stack frame values for those variables.  I've made it fairly readable to follow along in Get_Var_Core:

https://github.com/metaeducation/ren-c/blob/f7e0d396f8745ea4232a80dfb44efe3d25d5b9ba/src/core/c-frame.c#L1142


 Yes, It would be better to return a value to indicate the errors.  For this function , even if RAND_bytes returns false, it is still functioning.
Thanks. ,


Wat.

Which part of the expression are you casting?


rax retry tests


The declaration, `type func(void)` is also valid in C++, so why not just use it?


It's not such a big deal, I can rearrange the bits when I pack/unpack if need be.
I'm not a hardware designer, so I can't tell if one bit pattern is better than another either for a hardware implementation.  I wonder if we could contact Gustafson? ;-)


恩恩可以


Isn't dmtcp_checkpoint() a blocking call? Otherwise, how would you return only after ckpt?


`inline` is redundant here, as these functions are defined *in* `DpiHelper`.


Why does not include bumping up version to 1.0.0 in this commit?
This issue will be fixed at #16.


I think mapgen is acceptable as a single word since it's commonly used.  Filesys should be separated though.

It's really hard to come up with a good, well defined line.  We don't want everything crammed together, but we don't want to require separators in common compound words either.  It's kind of up to your best judgement.  Area store just seems like something that should be separated to me since it isn't a common compound word.


これ足しておこう
```C#
using System.IO
```


Hi Brady.
I have family in town. I'll be away from my dev box this week.  :)


ReadVarsSmallDiffTest (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1770 of 1771 tests passed)

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


I tried... Couldn't find an email address easily. Someone should send him a link to the discussion though.

> On Jul 29, 2015, at 10:49 PM, Scott P. Jones <notifications@github.com> wrote:
>
> It's not such a big deal, I can rearrange the bits when I pack/unpack if need be.
> I'm not a hardware designer, so I can't tell if one bit pattern is better than another either for a hardware implementation. I wonder if we could contact Gustafson? ;-)
>
> —
> Reply to this email directly or view it on GitHub.
>


This has caused me hours of frustration! I had legitimate assert_() methods on the classes being mocked and now, all of the sudden, I'm getting attribute errors.  Took forever to figure out what was going on.  There should be a better error message than simply throwing an AttributeError.  Where would I file an issue with this suggestion?


reformat please.


New line before else


Revert this. You can use the IDEA Eclipse Code Formatter plugin to properly format the files according to our [rule](https://github.com/excilys/androidannotations/blob/develop/AndroidAnnotations/eclipse-formatter.xml).


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


ok sounds good

On Thu, Jul 30, 2015 at 3:50 PM, Ernesto Soliani <notifications@github.com>
wrote:

> The rank function has a problem of how to solve order the dataset. I will
> leave it for the future.
>
> The drawNationalRank is an old nasty function from the previous code. It
> creates the chart.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/WorldBank-Transport/edudash/commit/46d3da715aabd3ad2cfb8cf848f706b2474b58ac#commitcomment-12452988>
> .
>


Without keychain dependency  mavensmate is not starting. It fails with the following error:
Could not add project:  System keychain is not support and there is no "password" property in project config/.settings file.


I would just set a variable for the method name, and use it in the for loop. That way you could remove this code duplication.


asType() is not needed, there is on overload for TypeElement in AnnotationHelper


Are you using all these namespaces?  I don't see that much new code that would require all of the SDK namespaces.


how about something like this?
found it in the extension section.

Ernie
![Uploading latest10.jpg…]()


You have to check for nullness for the `actionBarActivity` and `appcompatActivity` references. They are null if support v7 is not on the classpath, and these types cannot be found.


@ChrisKitching this allows ACU to be loaded in UEF T1 transport in any mission and that is really not intended at all


I forgot the feature of the new chmod feature, which comes enabled for PirateBox


Guys, what's the whole story with using gettimeofday? I thought we only need CPU clock or something like that to step the world. Can we get rid of it somehow?


extend this from `Activity`, so it will have all the necessary method, and the generated method will compile.


What was the issue here?


Please change the signature to `public virtual byte[]`
https://msdn.microsoft.com/ru-ru/library/fyza88w1%28v=vs.110%29.aspx


req.query is, minimally, already an empty object. This is more windy than it needs to be.


Two issues:

- Our NetKAT syntax extension: it has bit-rotted. I say we ditch it. I think we should hack Frenetic using REST and JSON. Even if we wanted to revive the syntax extension, it would be a separate package so it could simply not use PPX.

- Going "pure PPX": as you point out, we can't really do that unless all our dependencies ditch PPX too, which is unlikely to happen anytime soon. So, if we wanted to use PPX, we would have to use PPX and Camlp4, which will involve the kind of `_tags` hack in this commit. But, **this is not going to be a temporary hack...**


- we usually use phabricator instead of github pull requests, but for this diff its fine.
- this seems broken in multiple places for the basic case where whitelist = ['!some_folder'.] Changes is a unit-tested project anyway, so let's just add some unit tests for in_project_files_whitelist
- I'm ok with this hack, but I'm a temporary contractor. I'll ping one or two dev-tools engineers to confirm they're fine with this


Check out [this](https://github.com/excilys/androidannotations/blob/96b773dbddb5205d1f974789d36bcc0e64710f07/AndroidAnnotations/androidannotations/src/test/java/org/androidannotations/generation/ActivityIntentFragmentTest.java) test for an example to see how to add your fake class to the classpath.


I don't think it's bit-rotted. On the contrary, Craig replaced the old parser with a Camlp4 one.


verified this commit is very operational, and very good. formatResults, erccAnalysis, saveArtemisPlots O.K.


Fixed in 2c2f99b897b1d724b0b5d2353fe3d45eac917121 and will be released shortly.


That resulted in some very interesting behavior :)


For horizontal_collide, only 0x01 was used, so I didn't think there were other collision directions. Yet it should be easy to extend it.


But it can't be used in quotation mode, right?


What if you put next as the second argument to the first .then?


Yes you are right, I missed this line into your snippet, it's now fixed with a NPE wrapper because sometimes the controller could be null


Why not use promises here? or .create?


Yeah you're correct.  I reversed it in the comment when doing it by hand.  Fixed and amended and force pushed.


A comment here will be welcome


This is a dangerous assumption to make. If you don't use the `e`, something else may have thrown an error besides line 39.


This is related to the comments about miniblog_for_user from a while ago. You can't actually use "None" anymore because of changes in template behavior, and this particular line wasn't breaking anything but might cause someone reading the code to try to use it in the future.


Any thoughts on making this a dot file name?


This will need to get updated in #1529, but I think it's fine to just update it there after this gets merged.


I didn't remember, but I can add lookat back in if there's a need for it.