Nit,  Get rid of excess whitespace.  server.go line 240 for example.

Have the ResponseWriter go first, for consistency.

func (daemon *Daemon) LogAction(w http.ResponseWriter, action string, id string) error {

Break this apart into two different patch sets, one for audit and one for logging.


I'm not sure about the utility of checking for a specific role, since a role is just a group of permissions and we are doing all acl checks on a permission level?


This is a waypoint commit for #127 that will break things until CoordMapper instances elsewhere can be adapted, as in the case of #133.


@tknorris can you help me with this? do you have any idea why? or is this unneeded?


I'm sure you're working on this but none of this data is getting saved yet right?


i don't think that a try/catch is needed since lua does the check on it's own. In this case s will be "" (or should be based on http://www.lua.org/pil/24.2.2.html ) when the supplied value is no string.

when you want you can add a check wether s is empty (or use luaL_checktype(L, n, LUA_TSTRING); ) and print an error message instead.


It uses the `for` attribute when it's inserted into the value form, so I'll just remove the `aria-label ` attribute.


avoid passing non-initialized members to a syscall... you'd better do something like this since you are polling just 1 fd:
```
struct pollfd pollfd = {
    .fd = fd,
    .events = POLLOUT,
};

int ret = poll(&pollfd, 1, timeout);
```


@jenius think this is legit? we only really need to make sure it returns


why are you adding timeout in the middle of the parameters and later on changing it to the end?  Wouldn't be better to just add to the end from beginning?


Please submit a PR if you'd like.


True.  I pushed up a commit that uses pings to sync instead (is there a better way?).


LGTM.


FWIW, I think unused argument warnings are a really bad idea. It can be quite reasonable to add parameters that are currently unused when writing an API. And the other way round, when removing the need for a parameter in a widely used API, it's often not required to break existing users.


This is causing a parse error:

    /Users/username/.oh-my-zsh/plugins/jira/jira.plugin.zsh:52: parse error near `else'


Everything in this page still suggests that there is not `Event` metadata, rather only `Stream` metadata.

This is SUPER confusing, especially with respect to this: [this post](https://groups.google.com/d/msg/event-store/1b5eNaQH1Mo/MnkveUmijfgJ)

The `Event Metadata` and `Stream Metadata` should probably be described separately.  If, in fact, they are separate.  To be honest, I'm still not sure if they are different, or if when you write different metadata to an event, it will overwrite the corresponding stream metadata.


Changed "referred" to "referred to"


Sounds good!


Agree with all these. Thanks!


Probably, flavoUr/flavoUrs, if you want to be consistent


I'm wondering whether it should be just url for material, but maybe a pattern so that we could also generate links to individual exercises
e.g. `http://example.com/:week/exercise:id`


Thank you!


Added 2 functions:
clearAll: deregisters all events.
replace: overwrites the function and properties associated with an existing event, with the function and properties passed to replace function

I also added a small piece of documentation for the 'on' function


is there any case in which we don't want the O_NONBLOCK?  We could avoid the entire fcntl() calls by just adding the O_NONBLOCK to open() since it's the only thing that makes sense for us.  Also it's missing the O_CLOEXEC, otherwise we will "leak" the fd to other processes when we are exec'ing them (for example for pigpiod)


This is a default configuration not general, i'd keep with the original one for now. Thanks for your help anyway


You should be able to use the repository model's getRepo method for this:

https://github.com/mozilla/treeherder/blob/master/ui/js/models/repository.js#L23

(don't be confused by the fact that it's called getByName, it gets exported as getRepo later)

See e.g.:

https://github.com/mozilla/treeherder/blob/master/ui/js/compareperf.js#L195


`| currency` filter for monies.


@cewing to be honest with you, I don't remember a thing about this. Would have to take a look.


Thanks !


"sainitzers" spelled wrong in comment


fancy, +1

und fixes #121


Yes. Would you please deploy it?


This is quite a common error on Hydra – all log seems OK until the end but the derivation is failed. I've rebuilt a few times with this commit included without encountering a problem.


Done.


I was finding running

    rs.print(peers.getOnlineList())

was crashing for me. Must admit I didn't debug - but assumed this change would fix it (it did).


Already done.


Doesn't this break using a full path?


Hi @patois. This is for the 2.2 version. I mistook its version in branch name, forgive me.
I've tried it with libgcc, libc, libctru, and 3ds_backlight symbols. Still lack its main part, as i only tried hard to recover those already in libraries. Yes most names are already with their prototype, from compared ones.
I've built latest libntrplg with 3ds_backlight, and get its elf file. And there is clue from function orders - 44(wow) forget to change some symbols' order, those are only removed from the main repo.
Hope this helps you, as NTR 2.2 might likely to be its final version. So at least it contains more interesting stuff. Also binary of this version is easier to be found, yep.
I do know why i failed to use the idc file now - it does load, however the offset mismatch, and it broke. Even that's convenient for users than ASM itself. I don't know your IDA version, so without idb shared.
BTW, the tool itself broke some functions - i even thought those are excluded - and not all restored.


Better for avoid many problems in server folder side, don't be afraid a SELINUX is in place just in case of malicious attack about that point


small typo socektAddress -> socketAddress


Please remove trailing comma.


understood @cguardia.  If you could, please do take a look?


@aldenml, I don't have any specific agreement with @arvidn about merging these changes. He's been reviewing and merging my commits one-at-a-time. He hasn't NACKed anything yet so I expect that process will continue.


Created index.html file


Just for testing to add new C++ Programming


@iemejia, that's a tox-specific variable. If you are just copying and pasting the command for executing py.test, then just omit that.

BTW, can convert our tests to the less verbose mode of py.test? I think that the tests are clearer that way, in my very humble opinion (and in my limited experience).


Good catch - that's my fault, I've been using that pattern


Can you add "quiet" as a synonym for "progout=None", at least for the time being until we have hammered out a truely common interface for the analysis classes? We are using "quiet=True" for a number of analysis classes and in particular use the option in the unit tests. The default would be "quite=False".


Minimal class is fine. +1


Understood, thanks for clarifying that. Then we should be fine here.


use quite=True to suppress any progress output during tests


@buchgr Do you want to just create a PR for this?


yay node...


Looking good, just add the minor "quiet" keyword, squash and merge (@richardjgowers can do the merge, too)


I think we should also do `array_map('strtolower', $permissions);` so that we always compare lowercase to lowercase.


merge this


Very good idea. Thank you!


`realpath` is not available by default*. I needed to install the package `coreutils` to make this stop failing.

I was taking a look to a way of provide realpath as a function here and there are some dirty hacks: http://stackoverflow.com/questions/3572030/bash-script-absolute-path-with-osx

I don't really understand the need of the change, but I am pretty sure that there is a good reason behind it. For this piece of code perhaps something like:

```
while [[ "$PROJECT_ROOT" != "/" && ! -e "$PROJECT_ROOT/.venv" ]]; do
    PROJECT_ROOT=$PROJECT_ROOT/..
done
```

Without caring too much about the `..` on the environment variable.

* I've tried on Mac & ubuntu 14.04 server


```js
UserFactory.createUser(user)
 .then(function () {
   return AuthService.login(user)
})
```


You don't need gcc in this file, its already in Packages. ;)


Yes, please add all.

> Am 15.06.2015 um 21:13 schrieb Bert <notifications@github.com>:
>
> Sure. I guess the commented out part should also be included as ToDo.
>
> —
> Reply to this email directly or view it on GitHub.
>


What is "release"? It doesn't seem to be a valid parameter. It still works because Travis script `gimme` ends up reverting to default option "auto" when given an invalid parameter. But perhaps a valid parameter should be given directly?

E.g., see https://travis-ci.org/spf13/viper/jobs/66876555:

```
$ git clone --depth=50 git://github.com/spf13/viper.git spf13/viper
$ eval "$(gimme release)"
I don't have any idea what to do with 'release'.
  (using type 'auto')
```

Relevant source code of `gimme` is [here](https://github.com/meatballhat/gimme/blob/f2342466bf48a069fd0a9a0112c18dd863de12be/gimme#L8-L9).


breaks autocomplete


Now this is really great debug @d-jsb
Thanks a lot!


breaks autocomplete #3231


Good.  This addresses the comment I left in my implementation below.  We can remove that comment now.


Update...Wine for Linux 32 (going Windows route)....I FAILED at the Alexandria and got thrown for a "loop", but I did install IceCream image resizer (via WINE) this morning no problem....for the small segment probably not worth it on your end, but having a hell of a time with Alexandria on my end, but not with the example of IceCream (as example for feedback).


Did you mean ```wireLestener```?


Fixed in next commit


Huh, nice catch, thanks - see #39 :)


新增了一个插入行的方法


新增了一个删除行的方法


Just a suggestion, but maybe we should, I dunno, add Lex, Rob, and Nano to the contributor list?


增加了一个getData方法，不过为什么不直接返回dataList？


如果renderer是一个字符串（已经拼接好了的东西，那么就直接返回）


这里到了最后到底要不要加那个renderer


lazy commit messages, that's what


现在为了实现强大的jquery dom操作，所有的返回都是jquery对象，之前是字符串，这样效率会降低很多，想想办法啊啊啊啊啊啊


Build Setting issue : fixed by changing the numbers of the Level.


@Yuta-Okugawa
ここに限らずなんだけれど、変数ひとつひとつをダブルクォート（""）で囲っている理由を教えて下さい。
以下の理由より、変数単体をダブルクォートで囲うのは控えた方が良いです。
- ダブルクォートなくても文脈で文字列であることは伝わる
- 可読性が落ちる
- strlen("$num") だったり strlen($num) とでばらついている、統一した方が精神衛生上良い


this was silly but I wanted to confirm sourcetree was properly config'd


There is no clean way I'm aware of for such async dependency injection with mocha.

But I'm probably going to tell @ashlynbaum to refactor this using something like this.
```javascript
var requestAppWrapper = require('./helpers/request_app.js');

describe('Test', function() {
  var requestApp = requestAppWrapper();

  before(requestApp.startApp);

  describe('Endpoint', function() {
    it('sould send a request', function(done) {
      requestApp.r().get('GET /test').end(done);
    });
  });
})
```

@ahdinosaur what do you think about it ?


rax retry tests


got it. later i will check it. see #45


I didn't think there'd be so many dependencies. I don't know if I can merge this.


yeah that makes sense. :+1:


Each image team can decide on its own and simply join in.
It's as easy as adding the distro name to the list.


Can I bug you to add proper documentation to transforms as you add them?


@Geno1024  我们已经约定 不用 `您` 改用 `你`了

您 => 你  请更正


lgtm


reviewed  其余没什么问题


This `ModelName.new.some_method` style sort of hints that the method in question should probably be a class method. You stumbled on a pretty good organizational style though considering that it wasn't clear to most folks how to structure this application at all! :)


Something like:

```python
from config import API_VERSION

api.add_resource(
    UsersAPI,
    '/taskx/api/v{version}/users'.format(version=API_VERSION),
    endpoint='users')
```


this was so you could do things like this:
https://github.com/parthpatel1001/event#with-a-object

Idk if thats a good decision or not, kind of just stole this:
http://laravel.com/docs/4.2/events#using-classes-as-listeners


That's possible but
- I don't understand why ImGui DX9 works well. tried to remove point sampling in the texture with bgfx dx9 and still have a problem.
- The rounded shapes are all off center in bgfx's implementation. Don't know why yet but I'd like to fix that too so I got side tracked into tweaking the ortho matrix see it that would help.


^100%
totally did not come up with this


Nice


This is redundant and I think you presumed it not working because of #59??


ah yea -- i meant to use it here https://github.com/parthpatel1001/event/commit/844f02b30d7939bba3fe929673ad4b1a1ca5fa3c#diff-b823eaab299e005f016eb051fbff13c8R21


会造成什么地方不正常的bug？


I agree that this is a good standard for the project.
 On Jun 14, 2015 10:39 PM, "JordanCheney" <notifications@github.com> wrote:

> Can I bug you to add proper documentation to transforms as you add them?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/biometrics/openbr/commit/706efbacd4c5213cfe1ddfaabc3adc942ffe7b49#commitcomment-11672748>
> .
>


Y los cambios en el mantenedor2.asp?


I think your previous version ( Clear()) is Better., more meaningful, less error prone.

we can use extension method for StringBuilder int Net20 too :)


Fixed in https://github.com/uwescience/myria/commit/53652322252e1174872c61a808752907052403e3


Is there a condition where self.providers would be Falsy but the OverrideFieldData wrapper would have been instantiated?


"powersuspend.c:171:1: warning: ISO C90 forbids mixed declarations and code" - can you help me, please?


Yes, that is strange. Can't figure out why this does not works. Also some time ago I found other problem with touring-view: surface, smoothness, access works only in old engine, not in OpenGL engine. But I can't believe that old engine supports such a thing because you replace whole highway lines section and there are no surface, smoothness, access support (in touring view).


LGTM, lot less hacky too :)


We should probably redirect to the user's initial destination (if known) and only go to a default route if initial destination is unknown.


Why for Enter key we just submit and for Return key we check for no modifiers? Maybe a small comment is in order.


Yes and no. Yes: In the shader `RGB(128,0,0)` and `RGB(255,0,0)` are give the same color.
No: In the current implementation the "brightness" = color value information is extracted when you set the color in `C4FoWLight`.

So far every light has a brightness of 0.5 that goes to the upper half of the object light texture.
The value of `RGB(128,0,0)` is about 0.5, the value of `RGB(255,0,0)` is 1.0. This is multiplied with the brightness, resulting in 0.5*0.5 = 0.25 and 1.0*0.5 = 0.5 respectively.
So setting the light color to `RGB(128,0,0)` gives a light that is actually darker than the `RGB(255,0,0)` light.


The id should be a `$stateParams` of the `listingState.details`.


I like when 'go test' is instantaneous. For instance rangeSize 1000 and windowSize max 50 executes in 0.134s, while this commit executes in 5.74 s on my machine


Let's remember what the user was requesting so that we can redirect him/her there after they log in.


Ok, the way you've set this up should work for now. I wouldn't really want it outside the paper branch though.


As always in open source. If it bugs me, I try to debug it ;)


It can't be hardcoded. It's gotta pull in the data from the current year and then the footer text defined in the options page


It currently returns an ```object``` so given that I think it will always be returning a string I agree that we should probably consider changing the return type.


@brad when running ansible manually inside the vm (with sudo), bower complained because it doesn't like running as root unless `--allow-root` is present. And i changed npm just so it matched :p


do you mean why getControl() crashes Kodi on edit controls added via WindowXML? I don't know why, but it has ever since Frodo.


just to show I'm reviewing every commit I calculated the performance impact of this one: negligible

benchmark                              old ns/op     new ns/op     delta
Benchmark_10values_windowsize1         465           465           +0.00%
Benchmark_100values_windowsize10       8678          8775          +1.12%
Benchmark_10Kvalues_windowsize100      1696491       1703720       +0.43%
Benchmark_10Kvalues_windowsize1000     2378690       2384317       +0.24%


This would mean there are var declares after code in a function, but I can't see that anywhere near line 171 ?

What toolchain do you use ?

Do you nust compile the kernel or part of a ROM ?


Additionally, I noticed that this doesn't include any seeding of roles and permissions.


see PR #532


OK!


I don't think a bool should be used, as it's not intuitive to me what `true`/`false` would mean in the context of this function.


@cpjk You should use `Module.split/1` instead.

```elixir
opts[:model]
|> Module.split
|> List.last
|> Mix.Utils.underscore
|> pluralize_if_needed(conn)
|> String.to_atom
```


There is a more official release now available by the author. I gotta admit fq_pie has not been evaluated enough by any standard, on any platform, nor have I looked at the release below, so try at your own risk. It may well be quite interesting in terms of packet pacing, and other features, in addition to pie itself. My early benchmarks were quite nice (comparable to cake and fq_codel), but I was concerned about it responding overly aggressively to udp, ping, and syn floods at the time I did that. sch_fq added non-tcp protections (fq_pie is based on sch_fq), but I do not know if the author tracked those in his latest release.

fq-pie kernel module
https://github.com/hironoriokano/fq-pie.git

iproute2 for fq-pie
https://github.com/hironoriokano/iproute2_fq-pie.git


Variable `response` is a misnomer.


There's a typo in `Fucntion`.


results like the one below makes me distrust the tool. The window size 1 code path doesn't even touch the same lines as this commit and was the most affected in the benchmark below

benchmark                              old ns/op     new ns/op     delta
Benchmark_10values_windowsize1         525           465           -11.43%
Benchmark_100values_windowsize10       8873          8678          -2.20%
Benchmark_10Kvalues_windowsize100      1684218       1696491       +0.73%
Benchmark_10Kvalues_windowsize1000     2368741       2378690       +0.42%


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


wtf


Do we need also +#include <GLES/glplatform.h> in glext.h? Cause it shows me the error:  error : unknown type name 'GL_API' which 'GL_API' is defined in glplatform.h


Heh DX Only :D


I don't know how to reply to that


there's an alternative patch that also passes the tests with less code

@@ -78,7 +78,7 @@ func (m *MovingMedian) Push(v float64) {
        if m.nitems == len(m.queue) {
                heapIndex := itemPtr.idx
                if heapIndex < minHeapLen && itemPtr == m.minHeap.itemHeap[heapIndex] {
-                       if v >= m.minHeap.itemHeap[0].f {
+                       if v >= m.maxHeap.itemHeap[0].f {
                                itemPtr.f = v
                                heap.Fix(&m.minHeap, heapIndex)
                                return
@@ -95,7 +95,7 @@ func (m *MovingMedian) Push(v float64) {
                        heap.Fix(&m.maxHeap, 0)
                        return
                } else {
-                       if v <= m.maxHeap.itemHeap[0].f {
+                       if v <= m.minHeap.itemHeap[0].f {
                                itemPtr.f = v
                                heap.Fix(&m.maxHeap, heapIndex)
                                return


`ng-src`


Other end of that reviews thing.


Can we just expose static helper classes in each provider that does this instead?  GoogleHelper.GetXyz(JObject)


Can we just expose static helper classes in each provider that does this instead? GoogleHelper.GetXyz(JObject)  I think nuking the contexts is a good thing


Error checking in general is worth writing.


I guess we could call this "connectivity" or "effective distance" or something?  When applied to an actual system, this would probably be actual distance, but we're just assuming all patches are equidistant to isolate the effects of quality.


Parameter is probably better than a query string ... but whatevz.


PacketFence, not PAcketFence!


Kernel, SaberMod 4.9.3 with Os

https://github.com/AuxXxilium/aux_kernel_note4/commit/4a2b276a1dbef0d7cffea7c478aa6c1a0f6ecd8d


ok, so we stick with `OK` `FAIL` as return error state. Thanks.


positionnés ou nommés


Yup, we can.


nice change


I think we can use TAILQ (queue(3)) here instead?  (Not a big deal -- we can clean up at a later time and focus on the functionality first).


Making a request to get a image url?


commenter toute la ligne ? flag, bold, hidden


DesignDaysGridView (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 think the two can be combined as strcpy(bename, "ROOT/").


getArt, no?


Connectivity is implicitly a function of time through the dependence on the occupancy of all the patches.  Could make this explicit by adding `(t)` to the appropriate terms.


Ah, that was me trying to fix the docs when they were broken.  :)

http://sphinx-doc.org/domains.html


Could you please wrap at 80 columns and leave an empty line, as e.g. in lines 46-50 above?


Traduire flag, bold et hidden ?


Same here.


And here.


Traduire bold et hidden ?


No validation.


Traduire flag et bold ?


Or requested from the server.


well spotted


Paramètres


Any way to leverage our `$baseline` values for this? Same for the width.


Since the `build-extra` repository hosts several mildly related sub-projects, could you prefix the commit subject with "portable:" and give a really brief summary in the commit message's body, rounding it up with a link to the original GitHub issue? That way the commit history would help readers find the context and rationale for this change more easily.


Please use `$font-semibold` for this value.


WAT IS HAPPENIN


Not sure I understand the question.  Both patch quality distributions are initialized the same way, with half the patches randomly chosen to be initially occupied by susceptible hosts.


Autre traduction pour device ? support ?


damn, this will give me trouble at the Travis CI job. I'll fix it and re-send.


Please use `$font-bold` for this value.


Please use `$font-bold`


dire('Bob', ...


Not handling error or waiting for success


Can we remove the element here, and only rely on the class? For example:
```
return self.q(css="div.problem div.capa_inputtype.textline div.correct.status").is_present()
```


Removed it again.


humeur


resultat


I guess 'invade' would be more appropriate here, since we're looking at whether the pathogen is able to spread after being randomly introduced on a single patch.


@aldenml

1- Yes, this adds the ability to send a DHT request over UDP directly to a specific peer's DHT listen port.
2- The direct request interface uses a stub traversal so it's not suitable for implementing generic traversals. Allowing the user to provide their own traversal class would require a much larger interface. Such an interface would be of limited usefulness since traversals only work reliably if most of the network implements the request type being sent. Allowing the user to initiate existing traversals via the session interface would be doable, but I suspect an application which would want to do that is specialized enough to warrant a separate client which instantiates `libtorrent::dht::node` directly.


Toute application


Win win then, lets do that


Why are we using inline-styles? Is this something we can place into the external CSS?


le même objet


Hello !

I forgot to say that I used stock CM11 : display-caf-new & media-caf-new for a full working camera & video enc/dec !

Very, very good @dianlujitao  !


Okay, bear with me please, I really hate frontends. Can you elaborate why we don't add these in the xml?


Add f postfix to avoid cast to double.
Turn into multiplication even though smart compiler will figure that out
More decimals


Why not use "*" like the other ones?


salut


Is the slash there used to escape the asterisk?


In Cargo, `gleam = "0.1"` is equivalent to `gleam = "^0.1"` which is equivalent to `gleam = ">=0.1 <0.2"`. So `cargo update` will automatically pick up 0.1.1, 0.1.2 etc when we publish them. If we follow SemVer, 0.2 means that there was a breaking change and we might not want to update automatically. I think this is what we want.

https://users.rust-lang.org/t/should-be-used-for-cargo-dependencies/313 has some more discussion.


API did change. There is no p and team parameter.
Please rebase on more recent master


Redundant comment. Could be expanded upon with a file header.


Hyago, tive que realizar a mesclagem das divisão localmente. depois te ensino como é que faz.


@nmittler psssst dude ... how did you find this? :smile:  ... don't have approval yet :(.


Because if you do, when you do getControl() on the text entry field, it crashes XBMC every time. You have to do getControl() in order to get what the user enters as their PIN to send to trakt. When I say crash I mean like HARD crash...completely locks up Kodi...Note that the crashes are more prevalent on some platforms like Linux.


I'm wondering why all the song and dance with the ternary expressions, when the fields are spat out by rippled as {true, false} to begin with, and there's better ways to coerce if it wasn't the case.

Separately, there seems to be a whole lot of machinery here for pagination etc, so was curious why we don't copy the `balance` from each line.  I'm not familiar with the ripple-rest API or what part of it this is for. Maybe we just aren't interested in it?


I'm using 'pandemics' here following the vocab used in Hess and Gog, to refer to the case where all hosts are infectious (i.e. there are no remaining susceptible hosts, but the population is not extinct).


The total test time for rangeSize 10,000 and max windowSize 100 is 3s instead of 5.7s if you remove this commit


I learned that's called [caterpillar case](http://stackoverflow.com/questions/11273282/whats-the-name-for-dash-seperated-case).

Anyways...rather than a positive vote here, I want to object to *lower* Camel Case.  Given how many fields are one word, it'd be easy for people to think they're all lower case.  Haskell gets away with it because most identifiers are multiple words.


Yes.


ajouter / ajouterPar
additionner / Additionner par


There is no variable declaration ... can you point me to your repo ?


ajouterPar
AdditionnerPar


Then I guess we've been using "*" when we shouldn't have been.


use link above please


I think lowercase-underscore works well. Will send proposal.


creeAdditionneur


I knew this would be a slow down, but I wanted to make sure that it was clear the median routine didn't have any bugs, and also has the advantage that we don't depend on any external packages anymore.  I can revert this.


Did this change also? I mean, it's related to the wrapper path, and not really to the binary.


Just getting the magnet because torrent, sometime got some error, i don't know why.
So now torrent are downloaded https://torrage.com/torrent/ with the magnet hash.


Actually I checked this and you did `find_by_customer_token`, why? I suggested `.find_by` only.


Fixed it: ddcad49ec6b8f31bc3daf3a1fbea7eac58d61ff0


I had to back-track on this. The problem was that a mixture of fatty-acyl ACPs either had the formula for the Phosphopantetheine co-factor, or didn't, depending on their sources.  I included it in the end of for all of the ones that are in the current ModelTemplates.


Yes there is, but I didn't change it.


:exclamation: :grey_exclamation: !


I don't know what the ternary expressions are for. This was just copied over from ripple-rest and will be rewritten or removed soon.


Erro!


ajouteur / additionneur


ajouterPar


creeAjouter


creeAjouteur


This description is inaccurate


rippled won't return falsy values here


Could you perhaps explain what you are doing here? :) Or extract some finished parts and merge them into devel?

Or even better, finish it :) :)


phpdoc забыл


you dont need -t here


Не забыть убрать это меню.


Voici un exemple de test d'égalité de fonctions de premier niveau, de méthodes statiques et de méthodes d'instance


It can be removed if running on bash.


One could, in theory, construct an `OverrideFieldData` with an empty provider list.


Utilizo el mvc5 para servir el HTML, pero después voy manejando todo desde angular.


By not going through `wrap`, that is.


de méthodes d'instance


Проверку на гостя в init()


It's worth noting that if you're talking about about doing this from an extension then you can use `session_impl::dht` to get a pointer to the `dht_tracker` object and generate all the announces you want with that.


The errors are there for a reason! It means your chroot isn't working properly!


`return null;`


No, probably not - we should just break out of the loop I think (stop returning items)


Since `grib_release` may raise `GribInternalError` and there are multiple messages, I'm not sure it can be ensured that all messages are released without ignoring some errors.

It is true that the docstring says messages are released in this function though.


@kren1 Where to these three variables actually get set? Presumably they come from the stuff under ``global: secure`` but I don't see anything in the scripts that assigns to these variables.


@amirch1 - this is an old branch and we are not using it anymore. We moved to work with 2.31 that has the JSON ps integration. We can close this pull & branch


@ssiloti I appreciate your comments. (if you have time to continue this thread)

I think the default behavior of libtorrent (or a reasonable mainline DHT implementation) is rely any "traversal" type command as a find_peer. Am'I wrong?

We managed to hack a custom session level ```dht_announce``` and ```dht_get_peers```, but it's a hack (https://github.com/frostwire/frostwire-jlibtorrent/blob/master/swig/session_extend.h), and I see you implemented "dht extensions plugins". I would like to accommodate to this idea if it ever get's to master.

Thanks


Como utilizo DataBase First, entonces genero el context con la herramienta de Visual Studio 2013 que me genera las clases necesarias para ser utilizada por el Entity Framework y así comunicarse con la base de datos.


The reasoning behind concluding that the bottom row demonstrates the existence of an ecological trap is based on the definition of a trap by:
* preferentially selected habitat
* that reduces fitness relative to other habitat

The bottom row assumes preference for high quality habitat, and the positive slope (i.e. the increase in disease risk with habitat quality) indicates fitness costs on high quality habitat, thereby making it a trap.
In the top row, high quality habitat still has a fitness cost from increased disease risk, but there's no habitat preference so we can't call it a trap.


can it get any better


This is an awesome addition, making it more safe and useful.


I paid in fulll


The pipe to /tmp/env from the old script wasn't an output redirect of the grep. It was there to null-out the /tmp/env file in-case the deployment node was also the first infrastructure node. That way, the environment variable set two lines down would be fresh.


fixed at 3b5c622


Needs updating then you can pull


This is something being applied and overridden by the javascript to show / hide this area. I didn't touch any of the behavioral side of this so I'll presume that can be done in a future capa cleanup story. Behaves as expected otherwise.


OK - thanks for the heads up, Jesse. I need to research the consequences of this on customers, but it looks like updating the parent sooner rather than later. Cheers!


I didn't think we actually wanted to use $baseline for anything other than margin & padding. Has that changed with the PL work?


trailing ,


app/tmp es donde CakePHP almacena datos temporales. Esta carpeta es usada para almacenar descripciones de modelos, registros (logs), la cache de las vistas y la información de sesión de los usuarios.
https://git-scm.com/book/es/v1/Fundamentos-de-Git-Guardando-cambios-en-el-repositorio#Ignorando-archivos


Crap, line 166 is wrong, if you don't set debug, there's a '}' missing, good catch ;)


Nice!


{{ (listing.description | limitTo:letterLimit) + '...' }}


s/cloude/cloud/


This happened automatically when I ran `npm install && npm run build-docs` as the docs suggest. :|


{{ (listing.description | limitTo:letterLimit) | ellipses }}


Is this still necessary? (Also, how do I make `cabal2nix` generate this line?)


SymEngine doesn't need to be built as a shared library anymore. Both `-DBUILD_SHARED_LIBS=on` and `-DBUILD_SHARED_LIBS=off` will work. So I would just remove it from here. You can mention that both shared and static builds work with the Ruby wrappers.


fixed in #200


We've had so few warnings for so long that I turned on `-Werror` recently, so this is more than avoiding an unused argument warning. It will fail the build.

At any rate, you can _still have unused arguments_, they just must be annotated as such with `__attribute__((unused))` or whatnot.

What would you suggest: should I turn off unused argument warnings altogether (and if so, should I also turn off unused declaration warnings)? They do have a habit of catching variables or arguments I've written during development but ended up not needing later.


now vs dies with
`fatal error C1083: Cannot open include file: 'boost/filesystem.hpp': No such file or directory`
did you manage to get it to run on your windows machine?


@i2y Without the `==` you will get this error message when running `python  test/check_py_source.py`:

```
##############################
Error in module fact
##############################
  File "fact.py", line 22
    if ==(n, 1):
```

This means the `==` is needed. Otherwise it would generate invalid Python code. It seems like the fact that the AST can be converted to bytecode that runs is not sufficient to get syntactically correct Python.

With the added `==` this line in `fact.py` will be translated to `if (n == 1):`, which is valid Python.

In my opinion, generating valid Python code for all Mochi code would be very valuable.


thank you very much


Well... turns out that this is just as wrong as the original version. ^^


Thanks! I didn't know that function existed.


Note this is not edited anywhere so shouldn't be reference


Perhaps better to call this `self.base_url`?

Also, I would build `self.base_url` with the version parameter once in the constructor rather than computing it in the request method.


@ronchifabricio Tá certo.


It wasn't pointless in my other mods you derp.


this line from Eric is stupid


Stop breaking things.


Alteration to allow returning mean value of probes across subjects


@ngoldman your prepended emoji are informative AND entertaining! :gem:


Stop it...


I didn't have it installed on Ubuntu either. Should we add realpath to oh-my-zsh dependencies maybe?


`add_compile_definintions` is better for this.


Yes, but reading it, does the behavior sounds obvious?


I've been trying to figure out how to put them in a little box for reuse instead of memorizing/macros :)


Nope, thank YOU for making me see this ;)


Getting back into the swing of things with my mod.
I haven't grabbed the arcade machine and started changing everything....
Also, now that I think about it, I'm pretty sure I did switch to a mod count with Thaumic Pipes.


Good catch! Removed long description.


Is there `long long double`? ;P


Because this got rolled back... Remember?


On 2015-06-15 13:45:52 -0700, Jason Petersen wrote:
> We've had so few warnings for so long that I turned on `-Werror`
> recently, so this is more than avoiding an unused argument warning. It
> will fail the build.

-Wno-unused-arguments...

> At any rate, you can _still have unused arguments_, they just must be
> annotated as such with `__attribute__((unused))` or whatnot.

Which is pretty ugly, and doesn't work across compilers. Sure postgres
could turn on the warning if the compiler supports
__attribute__((unused)), but it'd be a fair amount of work.

C++ has a comparatively nice solution to that problem. If you explicitly
don't need a variable, simply leave off its name in the function
definition (not declaration).

Especially if you have callback type interfaces - of which there exists
lots in postgres - it's hard to avoid unused arguments.

> What would you suggest: should I turn off unused argument warnings
> altogether (and if so, should I also turn off unused declaration
> warnings)? They do have a habit of catching variables or arguments
> I've written during development but ended up not needing later.

Hm. I'd personally switch them off. Using warnings that postgres doesn't
use and where hundreds of violating sites exist seems likely to end up
being painful.


Hello,

since this pull request was made,
new discussons risen on forums and
some people are accepting adding files like this.

You can take a look at the following links:
https://moodle.org/mod/forum/discuss.php?d=275466
https://tracker.moodle.org/browse/MDL-48114
https://moodle.org/mod/forum/discuss.php?d=312215

I've made some pull requests to plugins that i use
and that have their source on github and until this point
6 plugins have merged and 1 rejected.

Kind regards,
Daniel


Perhaps better to call this `self.base_url`, and computer it during the constructor rather than computing it repeatedly in request?


Yes if you can please convert them, I will read them so I can learn to do that after. By the way thanks for the git add -p tip, it really changed the way i do commits now.


Will do in the next commit


Use `nullptr`, not `0`.


Yes, you're right. No need to document that.

I'd put in the commit message: "also drop obvious and thus useless comment about the wrapper when it's expressed as a relative path."


I see what you did there.. :smile:


Yes, there is a fourth scenario of pathogen-induced extinction, but this is generally subsumed under the 'widespread disease' scenario and differences in the relative tendency of low and high quality habitat to lead to extinction factors in to the differences in median population size in figure 2.

Should 'high pathogen endemicity' replace 'pandemic' to refer to the case where all host populations become infected?


looks good


This might be a silly question from a clj/cljs noob but did you mean `lein repl` or `boot repl`?


I think we should keep the code for default and case 1 differently. Default case be changed but other definition of cases will be same. Thats the reason I kept both differently.


Please don't revert this, I like it


Is this structure how the loading and saving of modifications/input is handled, ie, precipitation is a type of modification?


I believe both NULL and INVALIDA_HANDLE_VALUE (which are not the same underlying value) signify the handle is invalid. Different API calls use different conventions.


I have set up a separate repository for the landing page. It is at https://github.com/issuewise/landing.
landing.html can be found at wsgi/myproject/landing/templates
the css files and image files can be found at wsgi/myproject/landing/static
@dvidby0 inform me when you make a change and when the change is ready to be pushed to the landing page.


`Process` doesn't own the process `HANDLE`. Why does `Thread`?


I don't think single quotes are necessary on the key, and I'd swap the double for single on the name value below.


New version...
Um... I can't even remember...


Nope.


@deividmarques acho que aqui pode apontar para o V3, não estamos mantendo o v2.


in this case it will already be false else error will be retured in the if check itself. this will be unnecessory.


Unnecessary extra line?


`status` is not written to.


Looks like the issue is here since the initial commit


Here also if the bool is true then it will throw an error even before we can assign it false. Thats why I didnt include it.


Dang - looks like the intent was to remove the odd error handling - not the fetch


Alto commit papu


review corrections here  https://github.com/yangqiao/rtems/commit/5cbb754cae13f3878d2fd00e309730848736494d


I don't understand this change, in light of the rest of the commit. Could you please explain it?


I can't even remember...


`override`?


we need that without it its giving a unregonized error after the revert everything is kool


Yeah I add whatever I can... removed irrelevant college EC's at least.

I really can't wait for the day I don't ever need it again (ahem VC)


Hmm. No, I think this is right, if I remember correctly...


We don't use `override` for now. I'll convert everything at once instead of having `override` on some and not others.


What's the point of this `memset`?


This may not apply cleanly in the future as this line has changed upstream.

There have been some major recent changes to perf_reader.cc. As mentioned elsewhere, I suggest you pull in the latest upstream code.


We've had the issue before that only "known" exceptions will trigger this, there may be 500s that don't conform to this format, consider checking if `responseJSON` exists before using.


Functionality for precipitation is correct after this commit.


Leftovers from Linux interfaces. I have to match what's in the base class. This will probably need some refactoring at some point.


we'll step through it tomorrow. I'm off to bed now, knowing that this delivers wrong results for negative coordinates. ;)


Зачем эта избыточная проверка? Тем более, что она приводит к неоднозначным результатам. Меньше ли здоровье, чем 200%? Эта функция выдаст `false`, что очевидно неверно.


No, youre just using a bad toolchian. If you revert this revert, it will cause 25x more errors


I guess I expected to see that the occupancy of the patches is explicity a fxn of time.


develop (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1163 of 1163 tests passed)

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


И зачем это обилие скобок?


It should. That's a bug in `Windows::Process`. Will fix.


Hrm, the only thing I can find in trac or the kodi forum is this http://forum.kodi.tv/showthread.php?tid=166581&highlight=getcontrol+crash already quiet old. I'll try to do a branch and split this again so that everything that is down to design is in the xml. Let's see what happens.


@buchgr lol ... it must have cc'd me because you tagged me in the description :)


OK


This is the original file that goes with hw11 instead of my results from hw10


I think we want this the other way around. Imagine you set the CONFIG_PATH in a .bashrc file somewhere and wanted to pass `--config` to the cli. It would just ignore the cli arg


This commit is because I found a good page explaining localized strings: http://phildow.net/2012/08/18/localizing-formatted-strings-on-ios/

They recommended to force the string to have the formatting itself rather than build the formatting in the code.


agreed.


space after `&&`


@Flet I'm mostly making it up as I go along.

The only thing I try to be strict about is the version commit -- always `:package: X.Y.Z` and hopefully only containing version bump in package.json and CHANGELOG.md.

Otherwise I just remember the short ones or look for something appropriate on http://emoji.muan.co/ when I'm bored

````
:memo:
:art:
:factory:
:bug:
:recycle:
:x:
:new:
:up:
```

:grin: :+1:


I tried when I added it to SALTS just a couple of months ago and it still caused crashes. I first ran into the problem a couple of years ago so I knew what it was immediately. You can, of course, test it again if you want though.


yeah sorry I'd forgot to push some changes. should be OK now but the copy of src and bin still remains


OK,  I see now that you state it as "no susceptible hosts populations persist"....I think I just read it as complete host extinction.  seems fine now.


very much so :flushed:


@litch you're right, my edits are totally out of context.  I think I just latched on to the first opportunity to add a metadata example, but that's not event metadata there, it's stream metadata, so the metadata I added makes no sense.


:+1:

Looking forward, to see this in action :)


`ExtendedRegisters + 24`


Please amend this commit to fix the typo here too


@amanpwl92 change it to com.mobicomkit.support.phone.number


@amanpwl92 support_phone_number


@amanpwl92 remove it


Is it wise to mutate the passed hash? I can see this causing bugs in the caller. I will fix this as well


yep, I've got `nix-shell` stuck on `.hdevtools.sock` a couple of times

> (Also, how do I make cabal2nix generate this line?
have node idea, sorry


https://tvheadend.org/boards/14/topics/16786?r=16800#message-16800

3.4 uses version 10, do we really need version 12 for Isengard?


Unused...?


Where did you get that from?


I wonder if we should change these names to be a bit more descriptive, something like "createFacebookMapping", or whatever it is this method does


Cool! I'm really glad to see that sharing the 2.0 disassembly turned out to be useful after all :) I probably won't continue reverse engineering it myself any further since my point initially was figuring out what it is doing under the hood and whether it'd be potentially dangerous to run it on actual hardware. It'd be cool to see some of its features re-implemented and open sourced, tho. What are your plans? BTW did you actually find any indicators that it's been built with the official Nintendo SDK? If that was the case, calling this thing an "anti piracy CFW" would be kinda paradox.


@surdupetru  update you git for view your changes :smile:


Will be.


Function names are already function pointers - `printNum` and `&printNum` are equivalent, but one is less line noise.


Nm, saw you fixed this later


@amanpwl92 change this method signature to accept only context and change the name to:
public static String getApplicationKey(Context context) {
     return Utils.getMetaDataValue(context, MobiComKitServer.APPLICATION_KEY_HEADER_VALUE_METADATA))
}

Then we wouldn't need to specify the same constant that multiple places.


This breaks every setup that uses the stable Tvheadend v3.4 setup.
While it still works fine with v3.4.


@haiderm222, you're reverting my changes and bringing back the static header...


This line appears to be causing the problems from bug #5360. Should cloak be in a div? That's where I've [seen it used](http://www.ng-newsletter.com/advent2013/#!/day/3).


:+1: for decoupling from osin


@amanpwl92 create support instance once inside the method.


using the stock toolchain mate :(


should we validate against an IP?


I can't figure out why the zip file changed on Github, but I confirmed the contents are the same and the formula works as long as the SHA1 is updated.


Is it still possible to have contours enabled (for some zoom level) e.g. in hiking mode but not in other modes (car, bike, foot)? This was possible with the "--" setting. This is a very important feature.


Leave a comment?


Yes, that's how we're storing precipitation.


Fixes issue where if `translate` module is not available, placeholder appears as incorrectly. Expects calling app to call translate filter on its own.

@olegrise @rodrigopavezi please review. Thanks.


No, I think adding it as a dependency would be preposterous: it is only used by one feature of one plugin.


LGTM


This case is no longer properly handled (connecting neurons to ensemble tries to access 'decoders' in `build_learning_rule`). I've added a fixup. Maybe we should add a test for this case, too?


Must be a bad download. No one else gets that error.
On Jun 15, 2015 5:11 PM, "Tarun93" <notifications@github.com> wrote:

> using the stock toolchain mate :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TeamHorizon/android_build/commit/4aeb64f42d73063cc1ecc0a8dac8c37cd2eac4f3#commitcomment-11689587>
> .
>


I guess I prefer your metapopulation definition of a trap where high quality habitats result in lower metapopulation size.

What isn't clear is the connection between reduced fitness (your definition) and prob. of infection (the plot). Rows 1 and 2 don't seem all that different, so it seems like the preference part is not that important here.


:heart_eyes: You removed something from my todo list ;)


saw approval from jonmeredith
at https://github.com/basho/riak_kv/commit/9972c1bfb2418dc13c7502d3a279f737d96c24c5


merging basho/riak_kv/bugfix/csm/rdb-backport-vnode-status-file-2.0 = 9972c1bf into borshop-integration-1143-bugfix/csm/rdb-backport-vnode-status-file-2.0


basho/riak_kv/bugfix/csm/rdb-backport-vnode-status-file-2.0 = 9972c1bf merged ok, testing candidate = ec9b41ec


Made changes


please line up indentation


Agreed, definitely should add a test.


@ChangeTip, send 5 cryptotrauma to @MaxFangX  (


It was unintentional, just use an older build for now.


Hi @maxfangx, I've delivered a Bitcoin tip worth 5 cryptotrauma (1,014 bits/$0.25) from @KBrizzle to your ChangeTip wallet.

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


I am using an older build, once I found the real cause.


wokay mate will resync from start n revert back thanx for the quick reply :)


I haven't used them to compile SDC (and have no idea what it will take to make it work) but as project file for Mono-D to get semantic analysis of all sources. Now it is `grep` again :(


You might want to have a check here to ensure that the currency and issuer specified is the same as the currency and issuer of the amount that we are trying to deliver.


That's a good point.  I'm pretty confident that static-time checking was a side effect of needing a static analysis, not a designed feature, but that doesn't mean that it isn't useful.

Also note that due to Meteor's current execution model (no require, just run a bunch of stuff in a row) syntax errors get caught pretty fast anyway. Well, on the server at least.

Note that the static checking involves yet another JS parser that isn't Node/V8 or Babel, so as we change the definition of Meteor code, we will have to do work to change the checker as well.  And if we move towards a world where everything gets run through Babel, that will move checking earlier again.  ie, will we still want to run Esprima-based syntax checking in a world where linker is deprecated?

I've raised this as a question on `#product`. My impulse right now is that performance is more important than this accidental feature, but happy to revert it if the consensus/DRI decision is that syntax checking JS files is a useful feature rather than a random side effect.


Nice! this is way better than my template idea. To make it even simpler could overload the "=" operator in the struct


Published to http://ui.edev.d2l.com/72_b1a5a29cd098edfc53ab479fd8295dd0b5bd3675


golint: domain/govpool/mapping.go:32:5: exported var MAPPING should have comment or be unexported


Can you check if #4025 fixes it for you?


When calculate_gradients_* normalizes a 0 gradient vector, it outputs an error but moves on and works correctly.


I think this confusion stems from the occupancy vs. population perspectives.  I think on the population level, so consider it to be a pandemic if it spreads throughout, but still has low prevalence. From an occupancy perspective that may translate into 100% disease occupancy (despite low prevalence).  Might be worth a few sentences where ever you first introduce "pandemic"


Yes, it was reported in #4027.


Thanks. Patch applied with minor tweaks.


If you want added points for these homeworks, describe what you think the homework is asking you to learn. I assume this one looks something like:

* Use function calls
* Understand the difference between a direct function call and an indirect callback.


warning


And that's all what was needed? Any explanation why DateTime.Min/MaxValue are so poisonous?


share the code with the v2 feed


@ChangeTip, send 5 cryptotrauma to @MaxFangX (


I'm confused... :(


Oh, right, the infamous `.hdevtools.sock` thing. It's really fucking annoying that I need to fiddle with biegunka.nix in Vim if I want to update it properly.
I wonder if teaching hdevtools to put its socket in a saner location would work.


Hi @maxfangx, I've delivered a Bitcoin tip worth 5 cryptotrauma (1,014 bits/$0.25) from @KBrizzle to your ChangeTip wallet.

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


Would you mind re-writing this to explicitly compare the array's members and length?


Thanks. Patch applied with some minor whitespace tweaks


find_by defaults to the record id column, correct?  find_by_custom_token is the rails way to specify the column to query and passing it the data it needs.  Isn't this what you suggested?


Is that so? which Zsh are you running on these OSes?


No need for compareArray.js here


This url should never have `index.json`, it's the resolved package address


I don't think this should be nested here.. upacting is the union of up and acting, not the intersection, right?


This implicitly tests the return value of `add`. Since that detail isn't related to the behavior under tests (and since you have explicit tests for the return value), could you change this to three separate statements?


Secondo me non dovremmo inserire anche le schede di trasparenza nei repo. Sono affari dello studente recuperarli, noi trebbiano solo il materiale didattico.


Code in this commit would allow players to tag other players (as the `EntityLiving` type check on entities in the RFID item doesn't exclude `EntityPlayerMP` when tagging entities), though this tag data would be un-readable because of the reader logic.

Perhaps it would be a feature to change the reader code to handle this case, or a bug-fix to change the RFID item to not allow tagging `EntityPlayerMP` instances?

$0.02


You can use `assert.sameValue` here for consistency with the other tests


Sorry, my comment wasn't clear enough. IMO, `https://graph.facebook.com/v2.x/` is the base_url. To wit, don't have both `self.base_url` and `self.url`; build `self.base_url` in the constructor, and concatenate the desired path and query params on to `self.base_url` in the request method.


@ChangeTip @MaxFangX 5 cryptotrauma


Just a question here. Did we have to add the headless option in multiple poms? If so, couldn't we just add it to the parent pom?


What exactly is the purpose of this driver?


21 files changed - that's much more than the commit summary says. Were all the changes intentional?


nice :+1:


The test body and file name don't agree here


I refactored HttpGetAsync(string urlSuffix) to use this method and also created an HttpGetAsync(Uri url) to support passing in a full url (for Apex Rest)


Working on it ;-)


@ChangeTip @jimlyndon 5 cryptotrauma


So we do have headless in the parent.pom, this is really just removing the -noverify option though


Lazy solution as proposed by Lord: https://github.com/Underground78/mpc-hc/commit/aedd150620dea7e9bbd75797533559c4026ef4f8.

Also:
```
<Lord> weird stuff reminds me
<Lord>         SendMessage(WM_CANCELMODE); // unfocus main menu if system menu was entered with alt+space
<Lord> whoever added this to CMainFrame::OnUnInitMenuPopup
<Lord> (it breaks the menu too, in a different way)
<Lord> I see what he was trying to do... but it's just broken
<Lord> really just remove it(tm)
<Lord> it is trying to close the menu differently than normal windows behavior, and breaks
<Lord> try cycling through menus with left/right keys
<Lord> (it randomly closes)
<Lord> and really what it was trying to "fix" was just normal windows behavior
```


I managed to hack it to compile for 3.10, will check out how it actually works on the devices. I will look at the updated repos tonight from work and hope to find something workable:) Thank you for the heads up on the new updates.


Is there an easy way to memoize by SCC?  Otherwise perhaps you should insert all of the messages in the SCC to the memotable at the end since it appears work is always performed for the full SCC.


Hi @jimlyndon, I've delivered a Bitcoin tip worth 5 cryptotrauma (1,014 bits/$0.25) from @KBrizzle to your ChangeTip wallet.

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


@delcypher they get set in the secure part.  You don't see the assigment part, evrything is done within travis CI


Ah! Make sense. Thanks for the clarification.


@ChangeTip @nicm 5 cryptotrauma


So will copying the whole descriptor actually copy over getters and setters defined on the interface?


Hi @nicm, @KBrizzle sent you a Bitcoin tip worth 5 cryptotrauma (1,014 bits/$0.25), and I'm here to deliver it ➔ **[collect your tip](https://www.changetip.com/collect/858380)**.

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


Just checked out his branch locally and it works like a charm.

Sorry I didn't take a look to the issues/PR before commenting.


Hi,
I made the changes as per your request.


This is redundant after refactoring.  Also handles errors differently than the factored out method.


Bad description, not merging pull request, turd


Please no abbreviations. :(


Excellent point!! Yes memoizing by SCC should be easy. I'll make this change.


Boa, to enviando a correção!


Surface Id up to client.


Yep checkout `Module.join` for the reverse!


I really need to work on a better way to update the live site


No problem, all of this is fresh from today! Thanks for reporting.

#4025 should be merged soon. Hang in there.


I just updated sdc32-experimental, it's now on the same state as this repo. Just with dub-files and purity-infrence support. (Disabled in release-mode)


r=jedmccaleb


Is the problem that the loopback Connection doesn't have teh correct feature bits set?  We should probably fix that too?


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/efae5a8d6edfdee7fc78ba85016a4334efdd54e3


merging graydon/stellar-core/loadgen-fixes = efae5a8d into auto


graydon/stellar-core/loadgen-fixes = efae5a8d merged ok, testing candidate = 16266592


One very probable issue here is at the hardware level. If you're trying to source ACLK from LFXT (the low frequency crystal), we've discovered that there's a silicon bug in some of the older MSP430FR5969 devices that are in use on some of the older WISP5-LRGs:

![capture](https://cloud.githubusercontent.com/assets/615107/8171097/3ea82d1e-136a-11e5-8de4-c0135e3263d6.PNG)

Unfortunately the workaround isn't perfect; you need to decrease the crystal drive strength to the lowest setting, then when initially starting up the crystal you must wait for the oscillator fault flag to go low (to ensure crystal is stable). This takes a long time and burns a lot of energy. Best fix: It appears the most modern FR5969 devices don't have this issue, so hopefully we'll be able to guarantee at some point in the future that new WISPs wont have this issue. For now, we must either avoid crystal use or deal with the extra energy cost at startup.

Things will still run... When an oscillator fault is detected with the LFXT, ACLK will default to the VLO. However, unfortunately the VLO (Very Low power/frequency Oscillator) has a super wide typical frequency range and large PVT variation compared to other clocks, so not great to use as a timing reference.

Hope this is clear,

Aaron Parks


Thanks. Patch applied (with some minor formatting changes). Please check the implementation of the ConfigProviderInfo(AuthConfigProvider, List<RegistrationListener>, boolean) constructor. The listeners are currently unused which doesn't look right given the other code in that class.


Global cpu min/max controller (screen on / off, 2-step touchboost, wakeupkick and userspace lockout).

wakeupkick = cpumin raised to xx Hz for yy time
userspace lockout = accept, ignore or refuse scaling min/max changes

Allows to fully control CPU freq in-kernel if you have on inkernel thermal engine and hotplug driver, locking our userspace (like ROMs, mpdecision etc.).

I also remove all kinds of in-kernel touchboost to prefer mine (customizable 2 step = 2 freqs, customizable delay, customizable tap count to step up).


Looks good, Leo! Besides the minor changes I suggested, you can also remove any includes for Test262Error.js because that file is included by default


Unless you are following some code pattern that common in the gjs overrides, I'd rename to unimplemented_fns or something more descriptive.


Would you mind checking the latest master? ASDF w/ SBCL should work.


This goes to stderr.  I'll just change it for a `fmt.Println` with a distinctive prefix, and thus also avoid some extra debugging info that comes with error logs.  Other that that, LGTM, so I'll merge this PR once I've made this change.


cool :)

I'm envisioning a little git `prepare-commit-msg` hook to automagically prepend the correct emoji.

http://blog.bartoszmajsak.com/blog/2012/11/07/lazy-developers-toolbox-number-1-prepend-git-commit-messages/
https://gist.github.com/EmmanuelOga/2926764

Anyways, enough daydreaming :)


Save a byte! http://email.about.com/od/emailbehindthescenes/f/address_length.htm


Yes, I think that the complete lack of anything about Event Metadata in the documentation is the source of a fair amount of confusion here...


No argument with that. My chroot wasn't working then. Switched system since then and only built Iguleder's jessie.  Not sure how to delete old pull requests!


Yep, I'll do soon ...do not worry :)

... it happened because I added a few other libraries stock huawei ...


description comment of -set-governor does not match --governor


Метод не требует авторизации + нужно обновить версию (т.к. в последней версии работает, а в 5.9 нет)


Принимается?


Prueba de comentario.


@Kontakter, посмотри код


I changed the name in fb8d4d6.


Shut up cam....


lol


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2123


@Mazda--


@icewind1991 how is this change supposed to fix anything?


The directory structure is confusing. The references inside my html file
are such that they refer to files in the same directory as the html file.
But in your directory structure you have the html file and the other files
in different folders. Further, the css files are in a different location
than the images.

1. Do I have to change my references inside the html file then?
2. This is more important: in order for the fb metadata to work correctly,
I need to make sure that I have the references right. So for example, the
thumbnail that shows up when you share it on fb currently has the url
www.issuewise.org/thumbnail.png, which assumes that the html file which is
served by www.issuewise.org is in the same directory as the thumbnail.png.
Will that fuck up? Where should I put the thumbnail now? Same location as
the html file or inside images?

On an aside, it's possible for example in Google App Engine to specify a
static directory from which all content is served instead of a dynamic
server-side script that serves content. Can such a set-up be specified?

Abhranil · abhranil.net

On Mon, Jun 15, 2015 at 3:55 PM, Dibya Chakravorty <notifications@github.com
> wrote:

> I have set up a separate repository for the landing page. It is at
> https://github.com/issuewise/landing.
> landing.html can be found at wsgi/myproject/landing/templates
>
> the css files and image files can be found at wsgi/myproject/landing/static
> @dvidby0 <https://github.com/dvidby0> inform me when you make a change
> and when the change is ready to be pushed to the landing page.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/issuewise/issuewise/commit/4774aac4df927b56de08dd351b04fafc4010dc62#commitcomment-11689245>
> .
>


Trailing commas and fat arrows don't seem to be used by other gjs code.

Fine with me adding them in though.


Isn't this the same idea as VIPSlots?


Arch 5.0.8
Centos 4.3.10
FreeBSD 4.0.2

All of them login fine. In fact my patch was to fix the issue logging in on FreeBSD which doesn't have url-quote-magic so whenever I would type nothing would happen except getting an error about missing file url-quote-magic.


Also if you chained to another filter here, you could just assign the whole operation, to the `errors` or whatever you call it variable, rather than updating it as a side effect.

I'd find that more readable.


thanks


No! Thats incorrect. `.find_by` is used to search any column or columns! `.find` is used to search by id column only. I have exactly suggested _only_ `.find_by`. I remember telling you to **_not_** use `.find_by_whatever` either in previous PR.


Ditto here, why not map instead of for each, and just assign to propertyObj?


size=46, options=array('nullable'=>false, default='::'), same for x_forwarded_for.


Eh, that was sorta intended... but I've decided to not let players be tagged... at least for now, I have a fix locally just not pushed it.


It looks like there were previous changes in production that had not been committed before.  The change in the commit message was made by me today, but I didn't really have a message to include about anything else.  The other changes represent what has already been in production for awhile.

I'll give the other admins a friendly reminder that they should make sure that changes are committed and pushed into github :-)

-Matt

>>> Christian Boltz <notifications@github.com> 6/15/2015 3:23 PM >>>

21 files changed - that's much more than the commit summary says. Were all the changes intentional?

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


please take another look at Packages: there is gcc-libs in Packages, but not gcc.
i am pretty sure that gcc from base-devel group is needed for yaourt. gcc-libs is not enough.

is this your mistake or mine?


Simona Pinkova


code tag


VIPSlots does not work for PM 1.5


https://github.com/facebook/jest/issues/116#issuecomment-63154034 hmm


Nice work! Great integration of two projects we've worked on so far :)


Preference keys shouldn't be renamed.


Ah right, my fault, I have mistaken it with the libs it depends on.


I guess you broke the tests now ;)


Does this fix the jQuery problem where .post wasn't automatically parsing the JSON on error responses? (Which would remove the need for [this nonsense](https://github.com/cloudflare/cfssl/blob/jacob/serve/cli/serve/static/bundle.html#L99))


What?? You still have things on your todo list?
But you're committing days and nights to this project, so you should be done soon ;-p


I introduced a new bug, this needs to be released if err == EXIT_FAILURE


@HaoK done, PR updated. :smile:


What about older feeds?  I'm getting an error trying to install modules from my local repository: "Unable to download from NuGet feed: The remote server returned an error: (400) Bad Request".

My local repository is running http://www.nuget.org/packages/NuGet.Server v2.8.50926.602.


Typo on line 31 causes composer to return an error on install


Hm, anyway here is what I meant exactly:

`user = User.find_by(customer_token: event.data.object.customer)`


dope


I gather you're probs following the convention here.

But no version of c++ actually require forward declaring you variable right?


fast-forwarding 2.0 to borshop-integration-1143-bugfix/csm/rdb-backport-vnode-status-file-2.0 = ec9b41ec


I got that error when trying to use the old OData format on any NuGet server - ProGet, self-hosted NuGet.Server, etc. The updated format is what I see nuget.exe pushing to NuGet.org on the wire and it enables PSGet to interact with newer format feeds. I'm not sure there's a way to easily add support for new AND old feeds unless you want to add a try/catch to attempt multiple queries.


The big change for this branch. We can not use stdout, only stderr.


Right. FWIW, I think being able to tag players would be cool, though I can see balance and implementation issues. Reading all the tags from an entity in both the inventory and on person is a pretty easy task.


Just Learning


why don't you make this pythonic? (.py) :P


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/66936938


yeah, i tried, and it compiles now, but recusive copy of directories doesn't seem to work. There's a recipe on SO (http://stackoverflow.com/questions/8593608/how-can-i-copy-a-directory-using-boost-filesystem) that might help with this, but its safe to assume that directories are not copied with their contents currently on any platform.

I had it fixed for vs2015 (and the whole project generator running again) using vs2015's impl of std::filesystem in #3929, but it's probably better to have a more general boost-based version out there. i'll update that PR once directory copying works again. Or I could look into this tomorrow morning.


Nice work! It works great. You could take it a step further and add user input flexibility by using the toLowerCase() method, in order to handle all variations of upper case and lower case. Good work!


ok @surdupetru  Now I'm trying to do recovery for Huawei p8


Ok, I guess I was just following your lead to research the solution and didn't realize you were providing the specific solution.  Thanks for the notes.


Nice! It works great. You could take it a step further and add user input flexibility by using the toLowerCase() method, in order to handle all variations of upper case and lower case. Good work!


Give it a try!


After delete the condition `openerp.SUPERUSER_ID != uid`, logged in admin user got error message when try to merge contacts, since it is admin this error should not displayed, fixed in 1ef1fe3bc3a266f014e9336b728ed99a5351b155


fixes #1


So this is how chaining to interface parent is going to look?

I imagine trying to get `this.parent()` would be hard to get working and possibly ambiguous at times?


yeahh.¡


Thanks, hopefully the author will fix it soon


I'll see if I can rebuild a workspace before long.


Great work!


you may have to do a hard reset of git to flush the bad stuff and get back to the current commit. I forget the command but I have mucked up before and recovered fine. Just search, there are plenty of answers out there.


fast-forwarding master to auto = 16266592


@blesh If `Observer` is already implementing the `Subscriber` interface, why not just extend `Subscription`, `SerialSubscription`, or `CompositeSubscription`?


Does any test code try putting a getting/setter on the interface and inheriting that from the interface? Would that work? Is it expected to work?

Seems like that would be useful.


> narritives

to narratives.


NIT: get rid of these blank lines


NIT: extra space


are we getting rid of this logic?


just pushed the gems - did prepare everything before I got the OK to go :)


No. From [the docs](http://api.jquery.com/jQuery.ajax/):

> In response to a successful request, the function's arguments are the same as those of .done(): data, textStatus, and the jqXHR object. For failed requests the arguments are the same as those of .fail(): the jqXHR object, textStatus, and errorThrown. Refer to deferred.always() for implementation details.


I'm really glad you posted on this topic, I think the conversation about colorblindness needs to be had more widely.


NIT: can you import com.rackspacecloud.blueflood.io.DiscoveryIO so that you can use only DiscoveryIO?


Damnit not again


SHAME 🔔 🔔
SHAME 🔔 🔔
SHAME 🔔 🔔
SHAME 🔔 🔔
SHAME 🔔 🔔
SHAME 🔔 🔔
SHAME 🔔 🔔
SHAME 🔔 🔔
SHAME 🔔 🔔
SHAME 🔔 🔔


Thoughts: https://github.com/PC-Logix/OpenSecurity/commit/30fd655b777bd9bc67b9f37ed08d1b917bcb9887 ?


Why is the reason behind that? You don't use it anymore, found some another solution or something else?


Sure.


I think you broke your code because kv() doesn't seem to be defined from this module.


#### Changes in representation
* The RHS of a rule(i.e expression of a Leaf object)  can be an **Application** or a **Variable** only
* The arguments to any Application can be another **Application** or a **Variable** only
* If a Symbol x appears in either of above cases, write it as - <br/>
*Application.new(x,[ ])*


That's fair.


I'm working on it. Just made the NIT changes for now.


All right


I didn't get what exactly!!


Well, one could argue here that you could put the path attribution before the `self.get`, to consolidate the code.


If this is specified and environment_id, do we support that?


I assume you mean `-Wno-unused-parameter`. I pushed a change up as #122 and will merge it shortly. I'll rebase #121 on it so I don't need the `__attribute__`s in that change set.


Looks like a good idea and it also finished yellow.  Maybe the values in the Outpost array can be increased now that it's being used more correctly?


Eh ma pensa se "domani" il prof dovesse toglierli dal sito o magari non è raggiungibile, averle nel repo non credo sia così "sconveniente".


Just the version number ;)


It seems to be scaling not propertly.

-720x480 and 1280x720 (Voodoo Chronincles)

![voodoo_720](https://cloud.githubusercontent.com/assets/3000282/8171824/c4ad94c0-13ed-11e5-933d-030b6dc7d3d2.jpg)

![voodoo_1280](https://cloud.githubusercontent.com/assets/3000282/8171860/fac0e350-13ed-11e5-8d0d-6e87b83440c1.jpg)

-720x480(Terraria) , 1280x720 crashed

![terraria_720](https://cloud.githubusercontent.com/assets/3000282/8171834/ce8d0e12-13ed-11e5-8659-4c23676a7c4f.jpg)


:trollface:


I still work with Visual Studio 2008. Someone made a pull request with Visual Studio 2013 project files however I don't really know what to do with it. I would like to continue using 2008.


Looks like this has been merged in. Closing.


The patch has been applied but I have a number of concerns:
- The patch triggered a number of basic IDE warnings that should have been already fixed (use of <> and @Override)
- The code is completely uncommented. While I can guess at what most of the code is meant to be doing there needs to be some comments to explain what is going on and why. Where appropriate, reference the JASPIC 1.1 spec.
- All user messages, exception messages etc. should use i18n (StringManager).
- There are no unit tests (note most Tomcat unit tests are more like integration tests)
- In JaspicAuthenticator.authenticate() request.getLocalName() is not the way to get a unique name for the
web application (assuming that is what is required).
- In JaspicAuthenticator.getAuthConfig() is a single, shared call back handler the correct model? Would per request/response call back handler instances remove the need for ThreadLocals?
- The use of a ThreadLocal JaspicCallbackHandler needs to be justified. Is a ThreadLocal really the only solution? If so there needs to be a very clear comment explaining why. Also, the ThreadLocal must be cleared as soon as it is no longer required.
- Will JaspicCallbackHandler.handle(Callback[]) only be called once per authentication? If not, the ThreadLocal is overwritten.
- In JaspicCallbackHandler how is the PrincipalGroupCallback associated with the authenticated Principal?
- The use of instanceof in PrincipalGroupCallback.addCallback doesn't look right. Why not use separate methods?
- PrincipalGroupCallback.getPrincipal() has an empty if block


Good initiative :)


you don't need to typecast it to an 'int'.


Yeah, this is pretty rough.  An alternate things would be to assume that the profile was in default state when the test starts, choose no, and then ... oh wait, you'll never see the prompt again.


Unfortunately ranges for local time and utc time are not overlapping and DateTime.MinValue.ToUniversalTime() is actually bigger than DateTime.MaxValue.


If it works for you it works for me.


To answer your question: simply put the image file inside wsgi/myproject/landing/static/landing/images and refer to it on the html file using {% static 'landing/images/imagename.ext' %}.
Everything should work then.


@gyaresu I had to undo this to get it to work for me. With the `path.join` in, I get the following error when I run `node program.js`:

```
/Users/chadoh/Code/learnyoureact/program.js:6
app.set('views', path.join(__dirname, '/views'));
                 ^
ReferenceError: path is not defined
    at Object.<anonymous> (/Users/chadoh/Code/learnyoureact/program.js:6:18)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)
    at node.js:814:3
```

When I change it back to how it was before this commit, everything works fine.


I don't see memoryStream being used anywhere now.


@jacobwilliams What's up with this subroutine? It is completely dead code, AFAICT, and would be nice to remove unless you have plans for it...


1667_SketchUpInspectorNewZone (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: OK (2188 of 2213 tests passed)

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


Good point. People are used to `strpos` but still. What about `find`, `findNth` or `indexOf`?


looks good on SBCL 1.1.12-1.fc20 (Linux Fedora 20) with its stock asdf. (I also ran all the tests, they all pass).


Wrong comment. Also seems pretty useless.


OK, I changed it to throw an exception now.
I hope this is the right way to handle a change via GitHub.


sgdfg


Sadly, it sounds like we'll need to leave the .bat scripts around for
support of older systems like Windows XP, until we lift that requirement.

On Mon, Jun 15, 2015 at 5:37 AM, Glenn <notifications@github.com> wrote:

> @benbc <https://github.com/benbc> I've confirmed that that syntax won't
> work. The best I can do is;
>
> Powershell -NoProfile -ExecutionPolicy Bypass ^
>  -Command "$ErrorActionPreference = 'Stop'; Import-Module '%~dp0Neo4j-Management.psd1'; Exit (Get-Neo4jServer '%~dp0..' | Start-Neo4jServer -Console -Wait)"
>
> I can't split the powershell command with a caret as cmd.exe still splits
> the string.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/glennsarti/neo4j/commit/bc9fab2d6a88ea75f0f00a22f6e95300e012283b#commitcomment-11679236>
> .
>

--


* TODO: rewrite this as it is doing way too much and is very messy now.


* lib/objenesis-1.2.jar should not be removed (the lib dir is used by project.yaml from Nate)
* Can you revert the changes in the imports? They're not needed for this change.

Apart from this I'd say this can be merged.


Dropping these obligations concerns me. I'm not sure what's the best course of action here.


1667_SketchUpInspectorNewZone (DavidGoldwasser) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2160 of 2213 tests passed)

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


Okay, got it. But, the facebook thumbnail metadata needs to be an
*absolute* image
url, not a relative url. So if I have an image in the images folder, what
is its absolute address?

Abhranil · abhranil.net

On Mon, Jun 15, 2015 at 5:12 PM, Dibya Chakravorty <notifications@github.com
> wrote:

> To answer your question: simply put the image file inside
> wsgi/myproject/landing/static/landing/images and refer to it on the html
> file using {% static 'landing/images/imagename.ext' %}.
> Everything should work then.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/issuewise/issuewise/commit/4774aac4df927b56de08dd351b04fafc4010dc62#commitcomment-11690576>
> .
>


Series should be SeriesSet. The idea being that they are both Sets. Tags are the set identifiers, and the values of the members of those sets are either a Number or a Series


Agree.


seriesSet again here. Not going to point all these out... you get the idea I think


May we should call these Set Identifiers are something? Or at least make the connection that Groups are the set identifiers.


Btw, i'm wondering about the cache expiration, i set it to one week because code was recent etc... but i think we can easily increase it to few months. I think we should have a default duration in the lib, and drop this param from scripts. The main point is to control the localStorage size somehow, mainly to preserve performance, but also space (this isn't much an issue, but the space is limited to 5-10MB). Any thought ?


Should this be find(domain, callback) ??


Not compiling...

Fix in https://github.com/KleeGroup/focus-components/pull/115


Thank you @cgestes - Fixed with 8158e253aa38f2016c15bb45d4a10510f6bd1875


Calculate the delay distribution under incident-free condition 11


@Bernardstanislas Could you add numeral in the browserify build as an external tool: `window.numeral`


This variable is no more required, is it?


Another interesting problem which I have no idea how to solve. This method returns byte-offset however `substring` accepts codepoint-offset. How do prevent people from passing `pos` value to `substring`? I actually think that we should **consider making `pos` private** because of that.


I'm thinking of removing this. I thought it would be useful but if you want to use `this` in the chained-up function then you have to bind it yourself. At that point you might as well do `.call(this, argument)`


Yeah, setting a default duration in the lib seems to be a good idea.
I've however no idea what the best duration is. For Discogs, I guess the cache size can increase quickly.


Incorrect `@return` annotation.


Can you rename the args (pr?) to be more meaningful?


This is duplicated with switch block in first intro step, please consolidate.


I would call this "customer" rather than "cust". Its cleaner and easier to read.


@nikomatsakis I agree, I have done something similar before and remember it causing a problem down the road.


Hi, thanks for this. I'll definitely add the license. I'll look this through tomorrow but if I remember right, the reason I didn't use primitives is because if the value is missing from the GPX file then it will return null rather than the primitive default which could be misleading e.g. elevation = 0

Thanks


Hello,
It breaks the `client_credentials` grant type, because the username is not needed.


Hi @chadoh
I see the problem: `path` is not required in the exercise description or the solution but has indeed made it's way into my `program.js`

Good catch.

(Your timing is great because I'm sitting here trying to figure out why `prop_and_state` doesn't work for me. I can add items but not delete them.)


Whats "pr"? We should fix the naming everywhere :)


Maybe call this getUnpaidOrders(filter), if you force the "unpaid" flag?
Or maybe it still could be useful for the accountant to see *all* orders, even paid ones?


Why does this function just return the argument it got in?


I think we need to come up with a better test for this.  Feels like a gross hack.


Ah I guess it returns that, or an empty list, but still I think that would be better handle in the calling code.

So that this function only does one thing, and can be called something more specific like _getGojectInterfaces or something.


All the browsers (tables) that can contain many objects should have pagination.
Can you add a task to the Excel file to add pagination everywhere where needed?


1667_SketchUpInspectorNewZone (DavidGoldwasser) - 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)


How about using struct.unpack or is it not supported by RPython?


So ```
let interfaces = params.Requries || [];
let gobjectInterfaces = _getGobjectInterfaces(interfaces);
```
more readable to me.


You were supposed to have that code change approved by me.
Did somebody else take a eye on it before you pushed it to devel ?


Is getFishNames() used anywhere?


I approved it.
On Jun 15, 2015 6:34 PM, "Julien Semaan" <notifications@github.com> wrote:

> You were supposed to have that code change approved by me.
> Did somebody else take a eye on it before you pushed it to devel ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/inverse-inc/packetfence/commit/fcf7d7c6c92a5a8ce3bbfb2f26352aab3b08036a#commitcomment-11690936>
> .
>


One I idea I had was using the --code arg to run after QGIS is run and do the test in that file. Have a file to test if a plugin is loaded, or the settings are going to non default locations, etc.


Maybe getFishTypes() for being consistent in naming?
getCustomers(), getOrders... go without "List" in the end.


@pya Sorry. I misunderstood about the == operator. You are right.
And so I think '=' should be removed from this line.


Perfect!, I'll give that a try.


It's the only test that does not test the libraries, but the app startup itself and a bit of the python plugin mechanism.
Better ideas are welcome! But if it's only the timeout I don't mind increasing it.


Added the changes.


I will test some ideas and see if they are better


This will need pagination and filters (same as many other data browsers where there can be many objects - orders, fish batches, etc.)


Please see my comment/question here.


sorry, we forgot this last week.


I was getting this same issue with the newer version of LLVM. Thank you for the fix! I was fixing this by using a 'memset' (i.e. memset(&(kv.value), 0, sizeof(JudyValue));) ; however, I think just setting the value to NULL is sufficient enough.


look at model:459, for a specialized example


:+1:
@acasey for VSIX authoring.


Ah. Should I revert your change here, then?


@sraue I don't know what this is but there is a spelling mistake on B.png "remote control"


Thanks a lot @severedsolo


okay on the length.

i dont think you *need* to specify if it's sorted or not... but i knew thats what you were doing.


Just tried this using the -L option and having the full path s3 location in a file and it failed to download anything from s3 because it appears to have assembled a wrong s3 address.


Should this be:

    A specific parameter name can only be in one of the three sets.


belongs_to :drink


has n, :drinks


We should also follow the one-sentence-per-line rule in the comments (similar as in the design articles).


belongs_to :company


abbreviations :(


has n, :ingredients


I have some edits I was going to submit today. Happy to put this back to `__dirname` unless you want to just do it now as a separate PR.


Delete `app.db` and restart foreman.


@Dicebot the problem with keeping half broken things around is that people get used to them being half broken, have their workaround and it never get fixed.

Here is a good example of this. Another is DMD's current GC. Things need to get fixed or eradicated. If you think dub.json are useful, please provide a working set with some viable test plan.


@kongchen I've implemented a way to ignore/skip Types during the read step.

Readme is updated also.


Also, search for `runpack` in [lambada](https://bitbucket.org/krono/lambada/src/75f8f465d384e30425700ea8aaa58382635b9fa6/src/lambada/ocaml/bytecodefile.py?at=master)


Do you some apps example? I have discovered 0 issues.. :\


@fprochazka And what name would you choose?


@dhylands FYI.  I checked this new version with rshell and it seems to work correctly.


> Please don't get me wrong, but what's the point of keeping of code, if it's no longer needed?

Just temporary, I wanted to make sure it all works first before completely stripping out the ```walletModel``` entirely :)


temasek this depends on the version you have on google play services. from what i foun versions after may creates problems and this helps solve it.

@arter97  what's the harm of having this here?


this watch can close for so many reasons, including etcd watch index out of range (https://github.com/GoogleCloudPlatform/kubernetes/issues/8700). I feel like it would be a pretty big source of test flake. If you're confident it won't please add a todo and file a cleanup bug or something.


I missed where our previous discussion went, we don't need this service right?


Deletes are a known source of slowness in the kubelet, so I don't know how to accurately measure back to back runs. The delete on the apiserver takes seconds, but the kubelet is still killing containers on the node, stalling the syncLoop. On jenking I suggest running the latency test for just one podsPerMinion setting to avoid this problem.

In theory the delets on the kubelet should be done by the time the rc creates another 1000 pods, and the scheduler assigns them, so we might just be fine. Just stating that back to back runs might lead to hard to debug situations.

Even deleting the namespace will return immediately, but in the bg the namespace is deleting events from etcd, which affect the performance of etcd for a good 15m after on a 3000 pod cluster.


please add a comment explaining runLatencyTest if there isn't one somewhere else in the vicinity


@aceman444: Yes you are right, that was one purpose why I always had it,
thanks for pointing it out. May be worth putting it back in default, as in
Touring view it currently seems to be ignored...

Leonid, can you maybe summarize your observations about the differences
between the old and the new engine to Victor, I think it is a bug and we
should have this investigated...

On Mon, Jun 15, 2015 at 11:11 PM, aceman444 <notifications@github.com>
wrote:

> Is it still possible to have contours enabled (for some zoom level) e.g.
> in hiking mode but not in other modes (car, bike, foot)? This was possible
> with the "--" setting. This is a very important feature.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/osmandapp/OsmAnd-resources/commit/9f61efadc2b1fa7a07431eea52be5ce2eefa0842#commitcomment-11689591>
> .
>


This looks promising! Thank you!


Ale spoko, analytics raczej sprawdza domenę z której ktoś wysyła requesty ;)


Sure sure, we can move it - though that raises another point: "if there actually was an update" - we should log if this update fails because there is no scenario where we should be adding a pending object and having that ```update_tally_map``` fail.


> Edit: figured it out

Yep, that's it :)


If your new references look like all the other ones, then I have nothing to add.


Cool. Let me know if you're unable to do it for any reason. I'll try it myself.


OK, I'm actually going to remove the `path` variable.


Good to hear!


@JanTvrdik :+1: for separate class


I'll let you take care of it. No rush. Thanks!


@chrisseaton Just FYI, I think there was one MRI test regression here:
```
     [exec] Finished tests in 286.887000s, 9.3800 tests/s, 635.4732 assertions/s.
     [exec]
     [exec]   1) Failure:
     [exec] TestJSONGenericObject#test_json_generic_object_load [	from /home/travis/build/jruby/jruby/test/mri/json/test_json_generic_object.rb:57]:
     [exec] Failed assertion, no message given.
```


Did you figure out how to get things working for Alpine? I'm in need of this too.


Pro Tip: Never use the words "simple" or "readable" in a commit message for a patch that uses the :? operator unless the words are negated.


hey looking at this again last ngiht...u aren't using ur remove_entry method in these tests. they are wrong and so is ur remove_etnry method. I'm sorry I didn't catch it before.

remove_entry cannot use .find - that is an active record rails query and not made for using in plan ruby class's. I didnt clarify that with u.

try again and do some research on how to simply find an object from the self.entries array and use the plan .delete on it


Generally I like in-line comments.


I left this as a pointer for now, since this widget will be used very rarely.  Why allocate memory that we don't have to?
(I have fixed the variable name)


[![The Fighter of the Spirit](http://api.artsmia.org/images/1366/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1366)


[![Tahkt-I-Sulayman Variation II](http://api.artsmia.org/images/1704/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1704)


[![Chinese Rock Garden](http://api.artsmia.org/images/31068/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=31068)


[![Black Moonlight](http://api.artsmia.org/images/65021/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=65021)


[![Iris Awakening Hypnos (Ending Night)](http://api.artsmia.org/images/120122/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120122)


[![Aurora (Morning)](http://api.artsmia.org/images/120123/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120123)


[![Apollo at the Zenith (Noon)](http://api.artsmia.org/images/120124/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120124)


[![Apollo Descending (Evening)](http://api.artsmia.org/images/120125/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120125)


[![Endymion and Diana (Early Night)](http://api.artsmia.org/images/120126/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120126)


[![Sacrifice to Diana (Midnight)](http://api.artsmia.org/images/120127/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=120127)


[![The Madonna of the Pinks](http://api.artsmia.org/images/123075/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123075)


[!["Elephant and Three Clowns on Tub" mechanical bank](http://api.artsmia.org/images/80418/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80418)


[!["Chief Big Moon" mechanical bank](http://api.artsmia.org/images/80420/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80420)


[!["Bismark [sic] Bank" mechanical bank](http://api.artsmia.org/images/80441/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80441)


[!["Hoop-La Bank" mechanical bank](http://api.artsmia.org/images/80763/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80763)


[!["Trick Dog, Six-Part Base" mechanical bank](http://api.artsmia.org/images/80765/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80765)


[!["Hen and Chick" mechanical bank](http://api.artsmia.org/images/80916/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=80916)


[!["Acrobat" mechanical bank](http://api.artsmia.org/images/81089/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81089)


[!["Uncle Sam" mechanical bank](http://api.artsmia.org/images/81095/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81095)


[!["Eagle and Eaglets" or "American Eagle" mechanical bank](http://api.artsmia.org/images/81102/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81102)


[!["Creedmoor" mechanical bank](http://api.artsmia.org/images/81104/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81104)


[!["Monkey with Tray" or "Monkey, Automatic" mechanical bank](http://api.artsmia.org/images/81107/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81107)


[!["Lion and Two Monkeys" mechanical bank](http://api.artsmia.org/images/81111/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81111)


[!["Teddy and the Bear" mechanical bank](http://api.artsmia.org/images/81115/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81115)


[!["Professor Pug Frog's Great Bicycle Feat" mechanical bank](http://api.artsmia.org/images/81134/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81134)


[!["Girl Skipping Rope" mechanical bank](http://api.artsmia.org/images/81138/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81138)


[!["Clown, Harlequin and Columbine" mechanical bank](http://api.artsmia.org/images/81144/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81144)


[!["Bull Dog Bank" mechanical bank](http://api.artsmia.org/images/81328/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81328)


[!["Flatiron Building Bank, Large" still bank](http://api.artsmia.org/images/81400/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81400)


[!["Springing Cat" mechanical bank](http://api.artsmia.org/images/81604/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81604)


[!["Commonwealth Three-Coin Registering and Savings Bank" still bank](http://api.artsmia.org/images/81626/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81626)


[!["Owl, Turns Head" mechanical bank](http://api.artsmia.org/images/81634/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81634)


[!["U.S. Tank" still bank](http://api.artsmia.org/images/81743/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81743)


[!["Fidelity Trust Vault with Little Lord Fauntleroy" still bank](http://api.artsmia.org/images/81757/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81757)


[!["Frog Bank" (or "Two Frogs") mechanical bank](http://api.artsmia.org/images/81863/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81863)


[!["Frog on Round Base"](http://api.artsmia.org/images/81866/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81866)


[!["Tammany Bank" or "Little Fat Man" mechanical bank](http://api.artsmia.org/images/81890/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81890)


[!["Humpty Dumpty" mechanical bank](http://api.artsmia.org/images/81960/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81960)


[!["Independence Hall Tower" semi-mechanical bank](http://api.artsmia.org/images/81983/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=81983)


[!["Lindy Bank" still bank](http://api.artsmia.org/images/82108/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=82108)


[!["Sputnik Bank" still bank](http://api.artsmia.org/images/82718/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=82718)


[!["Columbia" still bank](http://api.artsmia.org/images/83684/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=83684)


[!["Tower Bank" safe still bank](http://api.artsmia.org/images/83685/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=83685)


[!["Hall's Excelsior" or "Cashier Bank" mechanical bank](http://api.artsmia.org/images/83706/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=83706)


[!["Bank of America Cash Drawer (registering)" still bank](http://api.artsmia.org/images/90098/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=90098)


[!["Japanese Safe" still bank](http://api.artsmia.org/images/90129/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=90129)