TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 37](http://teamcity.otenv.com/viewLog.html?buildId=1315577&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 37](http://teamcity.otenv.com/viewLog.html?buildId=1315577&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1 (new); exit code 2 (new) Build time: 0:0:0


qp2 ahi


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 36](http://teamcity.otenv.com/viewLog.html?buildId=1315576&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


@Hvitnov Seems good to me, thanks. If you could just clean up the added blank lines (in .h mainly), that would be perfect :)


1: M110 missing component name


Doh! Thanks!


I needed the loss_fct (i.e. a Python function) because the optimizer provides the symbolic input variables. Maybe those symbolic input should be in the dataset? I'll add an issue to discuss this as this should be modified in its own PR.


@Gillou68310 Looks like we get a GLES 3.x context automatically, even if only a GLES 2.0 context is requested.  That's nice, because it means we don't have to modify any of the SDL2, mupen64plus-core, or ae-bridge code.  It wouldn't surprise me if some of the device drivers don't follow spec (I'm looking at you Qualcomm), so we may need to keep a lookout for that.  But for the time being it's nice we don't have to muck with upstream stuff.

I posted on the paulscode forum, but just to let you know, I heavily rebased the gliden64-integration branch and renamed the original (so we don't lose the comments or break public record).


Reworded this comment, and the one in CaptureHashCodeAndReinitialize.


It probably does need to change the database engine, since this commit didn't fix it for Travis (or new 12.04 users in general). I should've pushed this to a branch first to test. Plus Travis should just use `createuser.sh` instead (where you've already added the postgis extension command) to better match the install documentation.


An idea that I had...is to only count "Bad Moves" (Quiets <0, Bad Captures) for move count base pruning.


fix of #287


This is it! With this commit, both DMG-GBC games and GBA games can boot using the same emulator. It uses a primitive method to check the file extension, but GBE+ is now a 3-in-1 emulator (DMG, GBC, GBA). Huzzah \o/


@ericstj Do we need to coordinate timing on our changes here, or is there a compatibility window?


@redwire I'm not 100% sure here, but I think you've overwritten merge changes from yesterday. Namely:
- `detention_date` does not exist anymore (it is `event_start`)
- `release_date` does not exist anymore (it is `event_end`)
- `rights_violations` does not exist anymore (is it `event_types`)

As per the client's requests. I had changed those fields in the `parsers` and `entities` files.

/cc @billdoran


changes made


better off returning right away to

- avoid the large code churn (could difficult for reviewers)
- avoid increasing nested level of code blocks.

i.e.
```
if (--waiting_on) {
  return;
}

// do the assertions here
```

what do you think?


could use Mutex::Locker here.


:pray:


just released v1.1.1


> Travis should just use createuser.sh instead (where you've already added the postgis extension command) to better match the install documentation.

Yes, good point!


Wir haben sogar Bremsscheiben :D. Finde bis auf das offensichtliche das Video verdammt gut. Wusste nicht das man sowas in "kurzer" Zeit hinbekommt - klasse! Das ist ein BMW Auto oder? @schweinekrippe Bist du dir sicher, dass wir das dürfen?


I'll add this in the issue about the Dataset.


*Whooves


I am on the daily from 17th,  it is just fine. Need any logs?


Yeah, that will work.

You firs solution will also work, although look weird in the js code
```javascript
var AS_MODAL = ("{{as_modal}}" === 'True');
```
Or given that `as_modal` is dumped to JSON in the view:
```javascript
var AS_MODAL = JSON.parse("{{as_modal}}");
```

Any option is fine :)


A new attack has been announced, see: https://community.qualys.com/blogs/securitylabs/2015/05/21/ssl-labs-117-obsolete-crypto-rc4-and-logjam and https://weakdh.org/

The default discourse installation, without this patch, is vulnerable due to the use of DHE. You can test it here: https://weakdh.org/sysadmin.html

Applying this patch seems more urgent to me now as it solves the issue (is not vulnerable any more).
I've also tried the variation of keeping the long list and adding a ":!DHE:!EDH" at the end and solves the issue too.

So, let me know if you prefer this patch or the one with the long list and the ":!DHE:!EDH" at the end. But let's fix this issue ASAP, IMHO, and then do the dh_param thing if we want to (that is not a 1 line patch).

Thanks a lot,
Rodrigo


I'll add this suggestion to the issue about how to handle the loss function.


fix #27


Right, good catch.


I'm not sure about this. This looks inordinately brittle to me. I think it would be clearer to write a unit test that checks  `unique_email`, a unit test that checks that the `EditProfileSchema` email property has the right validators. Perhaps part of the reason this is so hard to test is that you're essentially writing integrated tests here.


I'm not a big fan of the extra whitespace at the start of param lists and the end but in the interest of time I will merge this now. Different folks have different coding standard and different pretty formatters. I usually try to blend in with the existing style.  Thanks


Good to see it go:)


This should be

    info = sc.data[sc.data[star] == star]


oops,
thanks for the fix!


s/thier/their


Because sphinx-apidoc is not recursive like I was hoping it would be, so I either need a place where all the UPM modules are, or I need to generate a stub during configure that includes all the modules and place it in the binary dir. And two, you also need the links to all the so's up to and including mraa for generating the pages.
I'll be looking into removing the `install` dependency but it won't be straightforward mainly because of 2.


On first reading it sounded to me like we are saying that the caller needs to pad the input with a zero-byte if it is odd-length. Could be worth emphasising that this is not required?

(Tony Rogvall who wrote the original SIMD checksum had an interesting idea for an API tweak for optimization btw. The idea is to actually insert zeros into the area following the payload so that no loop is needed to take care of the unaligned part at the end. For example the AVX2 version would store enough zeros that it can round the length up to a multiple of 32 bytes. Have never experimented to see if this would have a meaningful impact though.. maybe with AVX512.)


Yes you are right. I should have remove the save/load methods since they are not tested at all. I'll remove anything related to saving and loading and make another PR to add.


TankDeadband (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1648 of 1651 tests passed)

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


@sk413025

您何時改姓啦XD

```
def markchang(**my):
```

另外

```
""" Learning Support-Vector """
```

這是給你隨便取英文職稱的唷^^

可以參考大家的XD


Same here as above.


I'll create an issue to discuss it.


Hi,

I think this is the worng issue for SD Problems.

And i need a better errordiscription
- which command is send
- is the file correct written to the SD Card
- when is the error shown
- ....


@schweinekrippe @UniSabina @dauphin001
Ich habe Sebastian gefragt, ob er uns sagt, wann wir präsentieren. Die Antwort war nein. Sie wollten das so machen, dass sie am Mitwoch einfach würfeln. Die einzige Möglichkeit nicht am Mittwoch vortragen zu müssen wäre, wenn jemand von uns einen nichtverschiebaren Unitermin oder eine Vorlesung hat.

@dauphin001 : Gibt es Änderungen die mich betreffen? D.h. muss ich eine weitere Graphik erstellen?

Ich werde das neue Automodell am Wochenende fertigstellen und Plausibilitätschecks durchführen, bevor ich die Ableitungen bestimmen werde.


やっぱりPython対話シェルをいい感じで使うにはreadlineが必要でした


Good catch, +1


Is this not exposed as sodiumoxide::crypto::assymetricbox::PUBLICKEYBYTES ?


On 21/05/15 17:30, Troels Henriksen wrote:

> svip <notifications@github.com> writes:
>
>> Og det er pænere at omdefinere funktionen efter tilstand?!
>
> Har du formået at undgå at bemærke at jeg normalt koder
> funktionsorienteret?

Dette er sh!  Ikke Haskell!  Med mindre det er i orden at kode
funktionsorienteret i et skalprogram.


Hmmm. This doesn't make sense to me as a place to add this code. `AsyncFormViewMapper` is a class that can be used as a `__view_mapper__` in a controller to turn a normal form handler into one that speaks JSON. Adding this behaviour here affects every form in `accounts/views.py`, which doesn't sound right to me.


- ¿`Estamos felices de anunciar` mejor que `Estamos emocionados`?
- ¿`Por lo que` en lugar de `Por esto`?


Nice Melissa! I like your writing voice. My only suggestion would be to add a couple sentences that describes your plan for tackling JavaScript practice. Will you use outside sources like CodeSchool? Review class material? Both? From there you would have a reference point to expand on for future journal entries or a topic in the parking lot about what elements of your plan were successful or unsuccessful.


Start


@courthead: "#customButton" was provided by Stripe. I just used what they provided to get an MVP.


Please fix the commit message to use standard git style.


Oops, I meant "Updated to version '5'"


This indentation is wrong.


Richard,

No worries on the whitespace/formatting request.

Thank you for the quick response, and I will get you fixed whitespace later
today.

__ __ __ __ __

Anthony Hildoer
Principal Engineer
BlueRival Software

text | anthony@bluerival.com
voice | 720.689.2001
skype | ahildoer
icq | 423790250

On May 21, 2015 9:49 AM, "Richard Ragan" <notifications@github.com> wrote:

> I'm not a big fan of the extra whitespace at the start of param lists and
> the end but in the interest of time I will merge this now. Different folks
> have different coding standard and different pretty formatters. I usually
> try to blend in with the existing style. Thanks
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlueRival/try-require/commit/d9e54e4aaff98b6646c209b66858cdfe45ec4b7b#commitcomment-11309146>
> .
>


- ¿`moveremos` -> `se moverá`?
- ¿`continuaremos la convergencia` -> `convergiendo a`?


Because `update_rules` *change* the gradients we need to update the `updates` dictionary. Whether  `update_rules` should change or contribute to the gradients should be discuss in a issue.


Manifest.h line 143 the javadoc comment is on the wrong declaration.


This also fixes #1509


maybe you could add one line between extends and the implementation?


Perhaps the query string for access_token is still missing.
```url = 'https://graph.facebook.com/%s/permissions?fb_exchange_token=%s' % (facebook_id,access_token)```


svip <notifications@github.com> writes:

> On 21/05/15 17:30, Troels Henriksen wrote:
>
>> svip <notifications@github.com> writes:
>>
>>> Og det er pænere at omdefinere funktionen efter tilstand?!
>>
>> Har du formået at undgå at bemærke at jeg normalt koder
>> funktionsorienteret?
>
> Dette er sh!  Ikke Haskell!  Med mindre det er i orden at kode
> funktionsorienteret i et skalprogram.

Det er altid i orden, især når alternativet er at kode svip-orienteret,
dvs. noget som ikke virker.

--
\  Troels
/\ Henriksen


a comment


Not Python 3 compatible use .format() instead


I have tried running a CyclomaticComplexityCheck on haxe-checkstyle itself and I found a few stacktraces.
My investigation showed two problems (haxe 3.2.0 on Linux 64bit):
- thresholds array has no default -> filter error in line 43
- default expressions of switch blocks can contain e.expr = null, which does not seem to be handled by "default:"-case leading to "Invalid field access : index" in line 57

I tried to fix the problems, but I am not sure if my solution has any side effects:  https://github.com/AlexHaxe/haxe-checkstyle/commit/011b3a5b664daa1bcc53de76e5ba0afc6faf7657


Effectively, I think they should be in the `Optimizer`


- white line


roger that, that is my one issue with the homework is figuring out where to put things.
but yeah having the db connection separated from models will help when we add more models to our app.


Sugerencias (y así me ahorro las interrogaciones :D)
- `[repo]` -> `[repo de jasnell]`
Correcciones
- `sientete` -> `siéntete`


I agree.


Relevant to https://bbpress.trac.wordpress.org/ticket/2309#comment:18


Correcciones:
- `comunidad` -> `Comunidad` (tal y como está en el original)


I don't see the error. I also compared it the previous working code fc9fd0fe4c446070838bbe72d8e3fe6df173c990 and didn't see the difference. Both bits of code concatenate scores across subjects, then apply the cluster stats and save


On 21/05/15 17:55, Troels Henriksen wrote:
> svip <notifications@github.com> writes:
>
>  > On 21/05/15 17:30, Troels Henriksen wrote:
>  >
>  >> svip <notifications@github.com> writes:
>  >>
>  >>> Og det er pænere at omdefinere funktionen efter tilstand?!
>  >>
>  >> Har du formået at undgå at bemærke at jeg normalt koder
>  >> funktionsorienteret?
>  >
>  > Dette er sh! Ikke Haskell! Med mindre det er i orden at kode
>  > funktionsorienteret i et skalprogram.
>
> Det er altid i orden, især når alternativet er at kode svip-orienteret,
> dvs. noget som ikke virker.

Det er svært at kode skalprogrammer på Windows!


Minor nits LGTM


I think you should remove the functions in this file that you don't actually need.


Interesting :) It added for disable support `core-js`? Why not added also `Object(Symbol()) instanceof Symbol`? :)


Mantén mejor la intención original:
> Microsoft está utilizando node.js en su máquina virtual `Chakra`


Right you are! I hadn't caught up with the changes made in response to that email before I committed this. Sorry.
I'm working on the replacements now.


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

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


You could be a bit more explicit in this error, that it was in the process of trying to find weave


lol


The JSON.parse looks the best, unfortunately though as_modal is a context variable in a Django response, so I don't think I can parse it as such, maybe I'm wrong:

```python
broader_context = {"graph": graph,
                               "as_modal": as_modal}
    response = render('reports_base.html',
                                   context_instance=RequestContext(request,
                                                                                broader_context))
```


o..


A really minor thing, but :

if "nojs_trigger_submitted" not in self.data:

... reads better.


this is awesome... when will we see a PR for this commit? :-)


cool, just making sure before things go further... :+1:


Missing spaces?! Shouldn't it be
```bash
if [ -n "$UDEV" ] && [ "$(expr substr $(uname -s) 1 5)" == "Linux" ]; then
```
Regarding #16


Remove the alert; replace it with a hidden div that display the messages. Alert tends to interrupt the game play.


same as above


What is this test actually testing for? None? if part is None: etc...


Made a PR but @mtgred has a different implementation for hosting in the works.


This change breaks browserify-ng-html2js for any angular version that doesn't correctly support npm. E.g all 1.2.x versions of angular will not work here.


I've wondered about that. I moved most of them in there I think :) We should file a bug to get them out I guess.


Another minor issue, but I'd trim the white space { "field ... {"field"  ...


Please fix commit message to use the git standard style.


This is missing the closing tag. %}


@brantwynn dfs_med_lift depends on panopoly_theme.


k


This change breaks browserify-ng-html2js for any angular version that doesn't correctly support npm. E.g all 1.2.x versions of angular will not work.


The problem now is that we can't use DI without a reflector.
E.g. benchpress is running in Dart VM and using `.toFactory`, but we don't have reflection capabilities enabled. For benchpress we can just enable reflection capabilities, but maybe this is a property that we want to preserve?

E.g. the reflector could expose a flag whether reflection is supported at all and we would check that flag here.


Same as above. What do you mean with "Se ha llenado el tablero 2"?


This isn't quite right. With this change, we'd set `self.sub_seq` to `seq`, then immediately compare them to each other, decide they are equal, and skip to the next notification.

This is testing for a sequence ordering bug in pycups that was fixed in 1.9.34. However, other places in the code (e.g. authconn.py) already explicitly require pycups >= 1.9.60.

Given that, I think the best fix here is to simply remove the `try:`...`except` part looking for the pycups 1.9.34 bug.


`load: bkUtils.loadAjax`

wouldn't ^ be the same ?


`save: bkUtils.saveAjax`


Remove alert, replace it with something else.


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


any reason why exec vs match?


- `mover a io.js` -> `mover io.js`

PD. ¿@ipeluffo Estábamos traduciendo TC?


Same as above with this alert "Casilla ocupada"


Same as above


Hello Dave,
I have tested your code with Romeo robot and it works ! so happy about it :)
Could we add this configuration file?


Oh, never mind, got it wrong until I saw the test.


Consider a more descriptive name like `MOBILE_FIELD_CONFIG_ERRORS`


test


It should work. Rendering of context variables happens before Javascript is put to work. Although we might need something like `JSON.parse("{{ as_modal|escapejs }}")`, not sure about how the variable is coming off the context.


-  `La reunión del Hardware WG en [YouTube]` -> `Reunión del Grupo de Trabajo de Hardware en [Youtube]`


- `junio` -> `Junio`


- `julio` -> `Julio`


Not sure if a MegaSyncPrivate instance can be retrieved on this way. Haven't tested yet.


I am working with the latest version. I tried adding this as css is not being copied to build vendor files. But getting this error. Can you tell me is it resolved in different way in the new build ?

C:\frontend\app>grunt build
Warning: Task "recess:build" not found. Use --force to continue.

Aborted due to warnings.

C:\frontend\app>

I want to add css as well as font files (ex: bootstrap glyphicons).


maybe call the variable `childInj` so it is clear which one it is.


Some benchmark results on that commit compared to the one just before :

=====================
Before
=====================
- 0 artificial lag
```
BenchmarkApplyESIMulti	  200000	      8137 ns/op
BenchmarkApplyESIWithSimpleInclude	   10000	    167104 ns/op
```
- 10 ms artificial lag
```
BenchmarkApplyESIMulti	  200000	      8163 ns/op
BenchmarkApplyESIWithSimpleInclude	     100	  20731104 ns/op
```

=====================
After
=====================
- 0 artificial lag
```
BenchmarkApplyESIMulti	  100000	     14400 ns/op
BenchmarkApplyESIWithSimpleInclude	   10000	    120876 ns/op
```
- 10 ms artificial lag
```
BenchmarkApplyESIMulti	  100000	     14170 ns/op
BenchmarkApplyESIWithSimpleInclude	     100	  10448278 ns/op
```

Conclusion : slower on operations without valid includes, but more efficient when there are some
Can probably be improved


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


threshold default = yep, it should have a default but maybe it shouldn't be an empty array. shouldn't break anything but it would make the class useless

default on null = you sure an ExprDef can be null in Expr? it is not defined as Null<ExprDef>. In any case, default should catch null as well. Can you post the stacktrace for that issue somewhere? (or else i'll try myself tomorrow)


Merge


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 28](http://teamcity.otenv.com/viewLog.html?buildId=1315630&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


If transit really is optional, then may I suggest that the dependency on it be marked as such in the project.clj. Opt-in > opt-out, etc.


dead code?


With Matt out, I took a look and I say merge it!


TankDeadband (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1146 of 1146 tests passed)

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


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 28](http://teamcity.otenv.com/viewLog.html?buildId=1315630&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 29](http://teamcity.otenv.com/viewLog.html?buildId=1315631&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


Yes


I chose widths based on when the layout started to break rather than using screen size dimensions.


I made a pull request for this.
https://github.com/davedelong/DDMathParser/pull/102

Tested on actual iPhone 4.


Why no camelCase?


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 29](http://teamcity.otenv.com/viewLog.html?buildId=1315631&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 33 Build time: 0:0:0


dsf


sdfsf


its there for review, i'll remove it once we're sure it's fine to remove. Ping @dmcgowan @stevvooe


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 38](http://teamcity.otenv.com/viewLog.html?buildId=1315635&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


@dmcgowan @stevvooe basically i replaced the dead code below by this line.


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 38](http://teamcity.otenv.com/viewLog.html?buildId=1315635&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 39](http://teamcity.otenv.com/viewLog.html?buildId=1315636&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 39](http://teamcity.otenv.com/viewLog.html?buildId=1315636&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


Hey @tarlepp, can you explain why you disabled models in the global space? I am mostly curious. I only noticed this because I like to mess around in the sails console and I had some trouble at first until I figured this out.


Thanks!


leave commit


Or, perhaps better, move all of this to a new file `mobile_field_config.py`


sdf


I agree that for now this is fine... however, I'm not sure they're really incompatible.


You are right.  I did all my testing with uClibc 0.9.33.2.  I imagine all that is required is that this patch also be applied to 0.9.32.1.

I did not test 0.9.32.1, but I do not see why it would not work, except for the this patch of course.  I can add a patch for that.  Good catch.


I think removing the grey600 icons is a bad idea. For pre-Lollipop, this is the most important "color".


It should be, I would think it best we don't use the full expansion here and refer to all uses as sign::PUBLICKEYBYTES and asymmetricbox::PUBLICKEYBYTES otherwise it's difficult to know which is being used in the code.


I tried doing it that way at first, but I couldn't get it to work. I guess I didn't try hard enough, because
it seems to be working now :)


Fixed with:

```javascript
var asModal = {{ as_modal|yesno:"true,false" }};
```


I'd probably make the equality comparer a singleton, but other than that, it looks define. (Weird that it's needed, but hey...)


The [contract specifies](https://github.com/SpongePowered/SpongeAPI/blob/42a5d59a60f61ee0d5707a7832142b3a80c537cb/src/main/java/org/spongepowered/api/Platform.java#L76-L77) that any new keys set on this map should be shared (i.e. same map instance) and existing keys should be unmodifiable. I can provide this implementation if you wish, but creating a new hash map every time won't cut it.


- automáticamente ejecutará -> ejecutará automáticamente


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2122 of 2201 tests passed)

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


> why? is there a design decision behind that?

Yea. I'm mostly Linux user and there's no `LINUX` macro available, `UNIX` includes OS X and FreeBSD, so the only way to determine Linux is to use `if(CMAKE_SYSTEM MATCHES Linux)`, thus for other operating systems the style should be matched :)


what?


Thanks for the catch. It's pretty rough to suddenly cope with all these name changes. Thanks for taking care of them all.


I don't think this condition is ever satisfied, is it? It looks like `lastErr` is never `nil` if we arrive to this point.


I don't have strong feelings either way, I guess. If it works the same putting it into the third_party directory, then let's just do that. :)


i believe this was in error.


whoops, borked the issue reference: onsip/SIP.js#175


Ooh, that's much nicer :).


Ah, thanks for catching that. Fixed in the latest commit.


`NewMockNode` sounds unlikely too fail, but maybe “Should have constructed a mock node.”?


https://github.com/PhamQuocTien132/nukeviet/blob/56a5ac8e10d79cf5984995154e551edb74a2c4ca/modules/news/funcs/viewcat.php

https://github.com/PhamQuocTien132/nukeviet/blob/56a5ac8e10d79cf5984995154e551edb74a2c4ca/themes/modern/modules/news/theme.php

A Tiến kiểm tra giúp em những chổ này, tên hàm vẫn chưa dc thay đổi theo nè a.


A simple solution for this is to use the fog-aws gem instead of the fog gem.  Fog-aws has its own dependency on Fog that is loose and better managed than the various conflicting knife projects.


is it possible that this method can be called from multiple threads?


Good


- cambios que se vienen en -> cambios de
- que la volverán inusable -> que la harán inservible


sdf


nit: could make these member variables const, since they never change.


Why does this need to be called? just curious.


Should probably squash this commit with thn previous two. Makes no sense. npm and bower made their own commits by default or something.


As per description: "`directory delete` fails for read-only files" and when copying VCRuntime140.dll from VS redist it is read-only.


The comment doesn't match the actual `value`, does it?


Yes. After the SDK refactor, we'll pass in a generic AdListener instead.


Something I forgot to talk about in the commit message: `clientX` and `clientY`. Why does `.fastClick()` take in `clientX`/`Y` if `.seek()` takes in `pageX`/`Y`? Well, `document.elementFromPoint()` _has_ to take in `clientX`/`Y`, whereas `.seek()` just happens to because it's a little easier to work with `jQuery::offset` using `pageX`/`Y`, and that's likely to change, because `jQuery::offset` has poor performance, so MathQuill may want to start working with `getBoundingClientRects` directly, which would make it actually easier to work with `clientX`/`Y` than `pageX`/`Y` in the first place.

For now, though, I'm not gonna port everything over the whole `.seek()` system to `pageX`/`Y`, instead `.fastClick()` just converts `clientX`/`Y` to `pageX`/`Y` by way of `window.pageXOffset` and `window.pageYOffset`, which [according to QuirksMode](http://www.quirksmode.org/mobile/tableViewport.html#t10) is the difference between them and has very good support.


I forgot about it, just copypasted header
next time :)


If we were to add this sort of thing in SciJava land, it would probably [go here](https://github.com/scijava/scijava-log-slf4j/blob/scijava-log-slf4j-1.0.2/src/main/java/org/scijava/log/slf4j/SLF4JLogService.java#L176-L178).


@mattandrews To-did a To-do


Ah, that's not so bad for string comparision :).  We probably want: `err == nil || !strings…` to avoid a `nil` dereference with `err.Error()`.  And we know the whole error string for this test, so we can use:

    if err == nil ||  string(err.Error()) != "invalid path: /ipfs/" {
        …
    }


current_update_in_epoch ?

If you don't like it I think nb_updates_in_current_epoch is fine for now.


ok


I don't see why you say they are destructive updates? Clearly, the method `apply` is not well explained (read at all). It uses the `params_updates` and returns a new dict with an updated version the `params`.


See above comment


After you made this change I'm having trouble getting my models to work...
why did you get rid of modelName and modelFolder?
I can't figure out how to get my model instantiated.

My models aren't in the same folder as my repositories so it can't seem to find them if I just leave the $model attribute blank...

if I try and set the $model variable to my model location 'App\User' etc... then it never instantiates it and it is just a string.

I can't see a correct way to do this using what you currently have available.


I agree, I'll mention that in the PR.


not me ;)


Working on dutch translation: what's a subscription in this context? I think it's a dangerous word to use in customer account area. (Wait - what did I just subscribe to?)


Good idea. Feel free to submit a PR or I can do it, either way.


"def" is an Expr object that with "expr => null" in the cases where I saw the stacktrace.
Setting `def = null;` before `evaulateExpr(def)` also "fixes" the error.
I agree that default should catch "null", but somehow it doesn't, at least on my system (it happens on haxe 3.2.0 and 3.1.3 both linux 64bit).

One of the stacktraces I get when running `neko run.n -s checkstyle -c resources/config.json` in haxe-checkstyle root folder:

    checkstyle/checks/ParameterNumberCheck.hx:1:1: error: Check CyclomaticComplexity failed: Invalid field access : index
    Stacktrace:
    Called from checks/CyclomaticComplexityCheck.hx line 57
    Called from checks/CyclomaticComplexityCheck.hx line 84
    Called from checks/CyclomaticComplexityCheck.hx line 77
    Called from /usr/lib/haxe/std/Lambda.hx line 181
    Called from checks/CyclomaticComplexityCheck.hx line 76
    Called from checks/CyclomaticComplexityCheck.hx line 79
    Called from checks/CyclomaticComplexityCheck.hx line 77
    Called from /usr/lib/haxe/std/Lambda.hx line 181
    Called from checks/CyclomaticComplexityCheck.hx line 76
    Called from checks/CyclomaticComplexityCheck.hx line 41
    Called from /usr/lib/haxe/std/Lambda.hx line 148
    Called from checks/CyclomaticComplexityCheck.hx line 30
    Called from /usr/lib/haxe/std/Lambda.hx line 148
    Called from checks/CyclomaticComplexityCheck.hx line 19
    Called from checks/Check.hx line 17
    Called from Checker.hx line 135`


TankDeadband (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1654 of 1657 tests passed)

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


Unfortunately this breaks on Windows:

![image](https://cloud.githubusercontent.com/assets/7850073/7753579/03d8fce2-ffef-11e4-8c5f-58d966bf5d19.png)

Can you use another command that can work for either platform? Because as it is now, a Windows user will not be able to run it (without making changes).


Hehe, I needed to check three times until I realized those 2 lines weren’t identical. ;) We should really only use dashes, never underscores.


Nice.


The whole point of this method is to use a single query in order to reduce the load of the database.
BTW, did you find a bug in this method?


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


Did you forget to add `/whisper` in the Info.lua file?


Reverted! https://github.com/ofnode/of/commit/e21ed0f


When we embed DartPad, it'll be way up at the top of the homepage! Soon...


don't change this yet (see comment for whole commit)


I don't think it matter, this is done a compilation time. At runtime this will become a element-wise product which is really fast on GPU and not slow on CPU.


The new added test seems exactly the same as the one above.


Flexible Box Layout Module not supported by: IE (8,9,10), Android Browser (4.2-4.3), IE Mobile (10), UC Browser for Android (9.9)


CSS3 selectors not supported by: IE (8)


CSS Generated content for pseudo-elements not supported by: IE (8)


CSS3 Border-radius (rounded corners) not supported by: IE (8), Opera Mini (5.0-8.0)


CSS outline not supported by: Opera Mini (5.0-8.0)


CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)


To what version of oimo does this correspond?  Is it backwards compatible? Or did it never work?


This issue is that it is a stats script, but it saves in a `subject` folder:
```
fname = pkl_fname(typ, subject, subscore['name'])
with open(pkl_fname, 'wb') as f:
pickle.dump([scores, p_values], f)
```


This looks like a useful and easy change to have in 3.0.1 as well. However, the 3.0.x branch still needs to support TYPO3 4.5. Could you please make a separate PR and not include the change for isoToTimestamp()?


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


The isoToTimestamp() change can of course go into the TYPO3 7 PR though


I'm not sure. I think it's a good way to avoid the need for adding single-purpose member functions, but it can be overused.


You can use GenericMath.floor here, it removes the (int) cast.


@Zakamurite напиши плиз на orionic.gm@gmail.com, есть разговор )


produces = "application/json;charset=UTF-8"  определено у класса, потому оно будет просто наследоваться.


You have to think symbolically: `lr` is a shared variable that will be update since we added in the update dictionary. At runtime, each time we do a Theano update, the following will happen: put a new value in `lr` that is the current value of `lr` time a decreasing constant `dc`.


@typesupply Did you mean to write “losslessly”?

Thanks for this btw.


typo: arbitrarily


Cool, I'll get a PR over soon.


The reason is the way leaf and intermediate nodes are mapped to indices in the "node2count" map. Allowing negative arguments to value2leaf() would cause the return value to possibly conflict with the index of an intermediate node. I wasn't able to quickly come up with an encoding that avoids this problem; if you come up with one, pull request welcome :)


Rename to be a Markdown file, so the README is rendered as HTML on the GitHub project page:

```bash
mv README.rdoc README.md
```


yep, @nsoranzo is right, duplicate test case.


This is usefull, Great!


Ground cover shouldn't use a Random because the values will change depending on the sequence in which the cover is generated. Normally you would want to use something like Noise.intValueNoise() to generate a value for the coordinates. This value is guaranteed to be deterministic.


For the above three lines, I might make the variables all caps + snake_case to make it clear that they are constants.


Ok


а точно надо Set проставлять?


CSS Generated content for pseudo-elements not supported by: IE (8)


CSS3 selectors not supported by: IE (8)


Flexible Box Layout Module not supported by: IE (8,9,10), Android Browser (4.2-4.3), IE Mobile (10), UC Browser for Android (9.9)


CSS outline not supported by: Opera Mini (5.0-8.0)


CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)


CSS3 Border-radius (rounded corners) not supported by: IE (8), Opera Mini (5.0-8.0)


nit: spaces on either side of -


`chunkZ, chunkZ`


@hakdogan Hocam elinize sağlık, elastic node açılması için kullanıcıya port ve host girmesi için ayrıca config dosyasında bir destek sunalım mı tıpkı mongod bağlantısında yaptığımız gibi?


Because these are values for display.


Again, I would make these REGULAR_GREY, TRASH_GREY


I added it on the shout command as an alias.
I followed your suggestions and added them in a single command.
But maybe I should add it as separate in info.lua...


done


Please use `true` here as an example. The `false` should be omitted for consistency with the initializations below.


I guess the code of dump should also be updated but it's not critical since dump is only for debugging


Block coordinates are absolute. The buffer min is it's lowest bound in world space. You can just use the given coordinates directly.


I'm also worried with line 226


This looks like MC code. This should use mixins. We don't actually want to copy code.


ну нам же надо всех участников вернуть. я думаю, что @ResponseBody все сделает за нас :) Ты реализуй, мы посмотрим.


Changes in Article 2 done.
-Next work on header


@Notsosof1Let's go over the row height of the table when we talk. I have one thing to show you.


"Add tag" is shorter


"Añadir etiqueta"


The `bool` is currently used in OverlayImpl.cpp line 585 in the expression `e->getScore(score(e))`, which calls a `std::function` to return the `bool`.  Currently different clients of `OverlayImpl::selectPeers` pass in different functors to this `std::function`, presumably because they have different criteria for adding the "has item" bonus.  These include `ScoreHasLedger` and `ScoreHasTxSet`.  Certainly having the caller adjust the score is also possible, but this would require us to publicly expose the score-adjustment-amount `spHaveItem` which is currently a private detail of `PeerImp::getScore`.  The current encapsulation of `spHaveItem` seems reasonable to me.


Yup. Published.


Awesome.


just wanted to mention that I recently added request signing support in the upstream repo in case you're interested. (see https://github.com/willnorris/imageproxy#signed-requests)


r+


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

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


saw approval from MonsieurNicolas
at https://github.com/jedmccaleb/stellar-core/commit/9a3ce9e999fb7fec18b73fef019c48d16dd2526c


merging jedmccaleb/stellar-core/fetch = 9a3ce9e9 into auto


jedmccaleb/stellar-core/fetch = 9a3ce9e9 merged ok, testing candidate = f50d895d


if sylius.config.classes is not defined $configClasses is still empty


No longer true?


You got it!!! You can click the little edit button like in the screenshot from the issue and go at it


> This drdump site is really nonintuitive ;/ I just realized that I had bookmarked wrong app (1.7.8), instead of 1.7.8.*. And I was really suprised that we had zero reports, hehe. I would have helped with reports.

Sorry for that, it's probably because I changed the app id just before pushing so that we start on clean basis.

> I have a question tho. If the crash report is assigned to a bug and it's is marked as won't fix does user get info about it when similar crash happens? Do we need to set "Response Url"?

The user is redirected to a page showing that the bug won't be fixed in MPC-HC but I misunderstood what the DrDump guy told me and the message is in fact mostly visible to us as it was thought as an technical explanation for other devs. So I guess we should change the way we handle those cases and use the response URL instead.

I would create some tickets with explanations for each cases (for example crash in ffdshow, crash in Haali Media Spliter, crash in Intel video driver, etc.) and close them as "won't fix" (of course we would reuse the URL each time we get a crash that fits an existing ticket). This way we can write a message that will be shown to users getting a known crash and they will be able to register if they want more assistance. What do you think? /CC @XhmikosR


Thanks, I missed that one. I'm not convinced but I don't feel strongly enough about this.


No need to call the parent method


CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)


CSS3 selectors not supported by: IE (8)


CSS outline not supported by: Opera Mini (5.0-8.0)


CSS3 Border-radius (rounded corners) not supported by: IE (8), Opera Mini (5.0-8.0)


Flexible Box Layout Module not supported by: IE (8,9,10), Android Browser (4.2-4.3), IE Mobile (10), UC Browser for Android (9.9)


CSS Generated content for pseudo-elements not supported by: IE (8)


I'll open a task to remove that dependency. Its not actually necessary.


This is a temporary fix right? Would be best to keep hash access consistent across the board. If you want to support both types of lookups, ActiveSupport has a really helpful method called `with_indifferent_access` that allows accessing a hash using strings or symbols.


TankDeadband (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1654 of 1657 tests passed)

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


I think is safer to create an action as CreateVariable, where you can control things like if the tag is repeated. Also, to mark the game.json as modified, so tags are saved.


I think all this should be something like:
```js
var version = service.nodeVersion.split('/')[0];
if (!version) {
  return '';
}

var runtime = 'node'
if (semver.major(verion) !== 0) {
  runtime = 'io.js';
}
return path.join('.nvm', 'versions', runtime, 'v' + service.nodeVersion, 'bin', 'node');
```


This commit fixes issues that arise with an update to Google Chrome 43.


Oh, I didn't notice that ;) Yeah, you should probably separate them, because I don't think `/whisper` shows up in help now.


I'm a little worried about changing KeychainWrapper without upstreaming the fixes.


Kill this indentation-only change.


Same throughout. It's third-party code.


`let`?


`($max_cnt < 3) ? 6 : $max_cnt` says that if $max_cnt (defender->level) was 4 (not less than 3), it would stay 4, and thus be a randint(5,4) :)  If the defender level was 3 (which is also not less than 3), it would stay 3, and if it was 2, then it would become 6.

What is interesting here is the tradeoff between using a high level spy (to overcome defenders) and low level spy (to keep number of conspirators caught to a minimum) for insurrections.


Thanks. I might recommend rephrasing my original message to something like this - if you see fit:

> Warning: File descriptor limit %d is too low for production sites. Recommend at least ulimit -n %d


How i can add class to all uploaded image ?
I use Twitter-Bootstrap and i need add class="img-responsive" for all images what i upload with ckeditor.
How can i do that ?
Don`t want all the time press "Source" and add class to all images. Thx for it.


I think you should preserve the original method signatures alongside to avoid breaking other consumers.


Cheers!

On Thu, 21 May 2015 17:42 Adones Pitogo <notifications@github.com> wrote:

> Yup. Published.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/adonespitogo/angular-base64-upload/commit/de2b0a7fe89e13ba50ba40b9ddf7ff0cfbaf8533#commitcomment-11310553>
> .
>


@rata 421b9abef0fec6d495d009a82aa1311c43203cf3 introduced a 2048-bit dhparam. I don't think there's anything else to fix here. The example nginx configuration from https://weakdh.org/sysadmin.html is direct copypasta from Mozilla's intermediate list.


Shouldn't we be using the Apache license here?


I did upstream the fixes, but haven't heard from him yet. I don't think we should block features waiting for upstream to accept changes.


POD types are undefined if you don't initialize them, so `bool` members have to be initialized to either `false` or `true` explicitly.  Just leaving initialization to the default constructor for POD types gives you junk.  A C++11 option is to use `mExists{}`, which will zero-initialize a POD type.


This approach resolves the DNS link twice.  Is the only problem with the old approach the lack of an `/ipns/` prefix?  How about:

    host := strings.SplitN(r.Host, ":", 2)[0]
    if len(host) > 0 && isd.IsDomain(host) {  // does IsDomain not handle zero-length strings?
        name := "/ipns/" + host
        if p, err := n.Namesys.Resolve(ctx, name); err == nil {
            r.URL.Path = path.Join(p.String, r.URL.Path)
        }
    }


I'll add this one liner to the SPL today then so it doesn't result in build failures.


@jensens is this ok for a new release? We need a CHANGELOG and a README: we can add the one from pypi. Ok?


We need to keep the `order_by('event__begins_at')` in place on this QS.  The order of `events_now` will not affect the order of `registrations`, and we want to make sure we get the *soonest* event.


yes, that's an oversight on my part, will fix


Make sense, but I steel find the erliear return more readable.


Most of these changes are actually because I pulled a new version of this in (since my upstream fix is based on master). But I'm confused about the indentation in them as well. Maybe somewhere in copy-pasting XCode did this. I'll check.


Rather than create a new list, perhaps we should just empty the existing list with `del self.data[:]` ?


Yes, I should try this style! Thank you for pointing it out to me!


Sounds reasonable. I'll remove the if completely then.


exactly what I was thinking; good call.


Yes, I did it in a bigger step that could be probably splitted into smaller ones.


Let's have a look at this one tomorrow.


Sorry @colmjude - thanks for fixing it over in https://github.com/alphagov/govuk_template/pull/150.


No particular reason; I copied it over without testing it. I've removed that function call from the latest version of the PR.


It's not a big deal, but it should have a `hasOwnProperty(eventString)` or use `Object.keys`.


@mgorczyca Are you sure about this? Both APIs will call `request.end()`. See:

https://github.com/request/request/blob/master/request.js#L716-L724


Focusable does not influence banning, so you don't need this call. (The rule is: If `client_isvisible()` changed, we need to update the banning, see `banning_refresh()` (which is a no-op unless `banning_need_update()` was called)).


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


you might want to consider `rebase = preserve`. It's the same as `true` but it also preserves merge commits instead of flattening them.


Yeah.. who wrote this crap code and left it completely untested? :)

Seriously, this code should be separated out and tested independently. Right now the only way to exercise it is by starting the whole clang thing which goes through two fork/execs and lives in a huge address space with lots of ocaml and C++ stuff.


@ozlerhakan Olabilir Hakan, daha pek çok şey(örneğin endeksleme öncesi index varsa silinmesi vb) olabilir, olmalı ama öncelikle bulk operasyonunda, json dosyasının okunmasının optimize edilmesi gerekir. Çalışma zamanında meydana gelen hatalar için log yönetiminin de iyileştirilmesi gerekiyor.


Mmm, I'll probably introduce a more generic concept of Amount...or Money.


The code is what was intended: variable * order, { `getter`, `adder`, `data } x { [ `FIRST`, `SECOND` ], [ `SECOND`, `FIRST` ] }.


@ggouaillardet Out of curiosity is there a reason to use poll (NULL, 0, 1) over sleep/usleep/nanosleep? I am looking at CID 1270134 which was introduced by this commit. One fix is to move to one of the sleep functions and the other is to check the return code of poll ().


There is no such signal. You need to call `signal_add`. Trying to emit a non-existing signal should generate a warning on stderr which you surely noticed during testing. ;-)


I would prefer the static import, sorry forgot about that :).

On Thursday, May 21, 2015, PEMapModder <notifications@github.com> wrote:

> Why not make the class implement PeProtocolConst or import static
> PeProtocolConst.*? These are good ideas too.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/BlockServer/commit/1d876d44609e32d30c3bda30745555d52a45ab49#commitcomment-11307965>
> .
>

--


Is it good or bad? I'm guessing bad.

On Thursday, May 21, 2015, PEMapModder <notifications@github.com> wrote:

> code style :P
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/BlockServer/commit/aa242a5007cfbf95e4af9a72941d694cf3b3687d#commitcomment-11307978>
> .
>

--


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


ah... i see, well glad to see people contributing either way. Wish I knew clojure, would love to jump in!


I'd move `c->focusable_set = true` into `client_set_focusable`. Also, that function currently does the wrong thing the first time focusable is set, I'll write another line comment for that.


@Rbn3D So can we remove the dependency on your fork in composer.json?


I see what you're saying. This code is a mess, so it's easy to get confused on exactly what they're looking for here..


Done, added to the SPL as zfsonlinux/spl@dc5e8b70416e5d511bc361309bd426c767177723


Removed in 994c48a


This should be `c->focusable != s || !c->focusable_set` since we do want to emit the signal also the first time this property is set. Then you'd need to move `c->focusable_set = true'  inside the if.


Isn't this weird because all readers read until eof and finalizers are called in Close?


http://opensource.org/licenses/mit-license.php
http://code.google.com/p/flexigrid/

Flexigrid¶

by Paulo P. Marinas

Lightweight but rich data grid with re-sizable columns and a scrolling data to match the headers, plus an ability to connect to an XML or JSON data source using Ajax to load the content.

Similar in concept with the Ext Grid only its pure jQuery love, which makes it light weight and follows the jQuery mantra of running with the least amount of configuration.

Features
•Resizable columns
•Resizable height and width
•Sortable column headers
•Cool theme
•Can convert an ordinary table
•Ability to connect to an ajax data source (XML and JSON)
•Paging
•Show/hide columns
•Toolbar (new)
•Search (new)
•Accessible API
•Resizable Width
•JSON Support
•Toolbar
•Table Toggle Button
•Show/Hide Columns control have been move to the column headers (try it by mouseovering a header and clicking a black triangle on the right)
•Fixed paging problem on multiple instances
•Mootools and Prototype noConflict() compatibility problems fixed
•New onError event on ajax interaction, (it will pass what the server said in a variable called data), allowing you to handle server problems
•New $().flexAddData method, allows you to directly add new data to the grid using your own data source.
•New preProcess API, allows you to modify or process data sent by server before passing it to Flexigrid, allowing you to use your own JSON format for example.
•Single Rows Select just use { singleSelect: true } in the options
•Quick Search
•Many more
•I'm planning to add an Editable and Resortable rows feature, as well as other cool GUI features.

One of my main goal for the plugin is ultimately to keep it lightweight!


Added to themes in 994c48a


Sorry, didn't read it right in the middle of the night :(


Tamamdır hocam.


Exactly. So, if the MAGIC_NUMBER > 18, we use a while loop, and decrement MAGIC_NUMBER until it is NOT > 0. Can you tell from this loop what the final value of MAGIC_NUMBER would be, depending on its initial value?


`$ composer install` to keep consistency with the others bridges


How about `semantic-format-tag-concise-prototype `? `semantic-format-tag-name-c-mode` looks horrid in `c-mode` since you can't even recognize between a variable and a function tag, and we don't have one for C++ too.


lol


CSS outline not supported by: Opera Mini (5.0-8.0)


CSS Generated content for pseudo-elements not supported by: IE (8)


CSS3 selectors not supported by: IE (8)


CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)


Flexible Box Layout Module not supported by: IE (8,9,10), Android Browser (4.2-4.3), IE Mobile (10), UC Browser for Android (9.9)


CSS3 Border-radius (rounded corners) not supported by: IE (8), Opera Mini (5.0-8.0)


Oh right on the 5,4, though why are we discussing it now that it's fixed?
low level attacking spy has no effect on the number of conspirators picked up.  Number of conspirators is determined by the level of the defending spy.


@abhidixit how about instead of gen_trans_id(), have a set_req_id() method that both sets req_id, and creates trans_id? Also, can you move the implementation to rgw_common.cc?


I think `self.data[:] = []` may also be equivalent, and perhaps more clear than using `del`. All three of these methods should garbage collect the same, but the first method won't change any references to the same list if they happen to have been referenced elsewhere (I think that was the concern).

Relevant link: http://stackoverflow.com/questions/850795/clearing-python-lists


fast-forwarding master to auto = f50d895d


How about:
```js
var version = service.nodeVersion.split('/')[0];
if (!version) {
  return '';
}

var runtime = 'node'
if (semver.major(verion) !== 0) {
  runtime = 'io.js';
}
return path.join(bosco.findHomeFolder(), '.nvm', 'versions', runtime, 'v' + service.nodeVersion, 'bin', 'node');
```


That is a great idea, but hmm -- I'm afraid, reference erasure would happen at the moment I pass these variables (`key1`, `key2`, `f`) individually to `std::make_tuple()` or `std::tie()`.

Suggestion: Keep it as is, revisit as we figure out where exactly copies are unacceptable and start unit-testing this code with noncopyable (but movable) types.


@NielsZeilemaker I've been debugging this for the whole day and I couldn't find anything wrong with the old function. But it didn't work. I tried to do it in the stupid way, one by one, and the problem dispeared for at least 3 hours. This one is less efficient, but it seems to solve the problem somehow. I will test with this branch for more times to see if it solves the issue.


Fyi, you're not supposed to create a new `FlxRandom` instance each time, you can just use `FlxG.random`. ;)


'make prtp' in a directory with memcad using clangml will exercise the frontend enough to trigger that
issue. I have 90 tests passing now with ocaml 4.02.0; better than 1 previously.


nice work, David!


Can we change this to:

'Something's gone wrong - try again in a little while'


:+1: done in 5725888.


Some newcomers told me they did not really understand the MNE family and what is what, we should lower barriers and add some context wherever appropriate.


Good catch, removed that by accident.


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


problem is still exists and on clean install.


@abenzzine Hi. I'm working thru your roadmap. There are no submissions for the BlocBrowser. Perhaps it wasn't pushed to GitHub? I'm moving on to review Blocstagram now.


Why is this not what `Pinning.Pin()` does internally?  And I'm concerned about not passing down a context or getting back errors that we can check (but I haven't looked at the pinning API, so maybe there's nothing to do about that at the moment).


But what's causing the none value? It not being returned by this method?
That seems to be impossible. I would add an assert in the addorget method to check the length of the array returned by this method


RUMAH KAISAR BINTARO 7

Miliki segera Rumah Kaisar Bintaro 7 (Asep Iskandar : 081281561468)

(lokasi Nyaman, Strategis, Bebas Banjir & Nilai Investasi Tinggi)

Progress sedang tahap pembangunan, Fasilitas Keamanan 24 Jam + CCTV & Garden.

Akses Angkutan Umum, Stasiun Jurang Mangu & Tol Pondok Aren Bintaro Exchange.

Dekat dengan Pusat Perbelanjaan, Perkantoran, Pendidikan, Rumah Sakit dll.

Promo saat ini** :

5 Pembeli pertama bila KPR mendapatkan Harga Cash:

DP Suka-suka.

Doorprize Honda Mobilio.

Dan dapatkan keuntungan Menarik bila Pembelian dengan cara bayat Cash Keras :

Bonus liburan ke Bali untuk 2 Orang.

Free biaya BPHTB,

Free biaya Notaris,

Sertifikat SHM

Jaminan Keuntungan Investasi 20% Pertahun (Buy Back Guarantee 100% dlm 5 Tahun).

Jaminan Sewa Rp20juta Pertahun (bila tdk dihuni Max 5 Tahun senilai Rp100juta dibayar dimuka)

Doorprize Honda Mobilio.

MARKETING:

ASEP ISKANDAR

081281561468 (WhatsApp)
087871847897
085779954195
Listrik: 1300 watt
Sumber air: SUMUR PANTEK
AC: YA
Apakah mobil masuk? YA
Bebas banjir? YA


Agreed. Both are clear.


CSS outline not supported by: Opera Mini (5.0-8.0)


CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)


Yes, it's still outstanding. Please file a bug.


if we stop splitting the doc between MNE-C and MNE-Python it will help...


For ``prior_update`` and ``post_update`` (and epoch).


Looks like you are signing only remote url (i may be wrong here - i looked at https://github.com/willnorris/imageproxy/commit/a9efefc8e7392aa4416650fc1c8a5c983a728614 ). I wanted to allow specific image sizes only, so i'm signing both options and remote url.
(also, signing scheme is similar to one used in nginx  secure_link module)

On a side note, i added remote_url encryption, so with signing image url looks like this:
http://pr0.zoon.ru/p26zsjLS28irlOz7xch6mA==/130x170,q85/-4ZnhDtkl3jcL8LhzjXMqYQpVSh2LaHvBhJNLPRbTc0FkHkcm0r5zYPQO7cPo9q8IEAYSQ_3fnw=
https://github.com/shepik/imageproxy/commit/ab1257122f45fb203a46588ed30b2a6cca0c9331
I don't know if you consider this a useful feature or not


Yay. much nicer.


By the way I wonder if this rather large case statement is necessary. Could we not use good old OOP for this? I.e. define a method `error_identifier` on each error class which would make this method a mere:

```ruby
def error_identifier
  if Array(represented.errors).empty?
    represented.error_identifier
  else
    'urn:openproject-org:api:v3:errors:MultipleErrors'
  end
end
```


I laughed at this line. FYI.


Good Morning, Toni,
I agree with you. There's a scope for performance improvement. I could
consolidate all debug prints into one method and add a boolean check for
debug for performance.

On Wed, May 20, 2015 at 8:48 AM, Toni Moreno <notifications@github.com>
wrote:

> Hi @lbasavap <https://github.com/lbasavap> you have included after every
> out.printf() a logger.debug() but you are rebuilding the output string t
>
> out.printf("%s %f %s%n", node, value/n, SDF.parse(sample.getTimestamp()).getTime() / 1000);
> String str = new String(String.format("%s %f %s%n", node, value/n, SDF.parse(sample.getTimestamp()).getTime() / 1000));
> logger.debug("Graphite Output Average: " + str);
>
> Perhaps could be faster if we build the output string only once, somethin
> like that.
>
> String str = new String(String.format("%s %f %s%n", node, value/n, SDF.parse(sample.getTimestamp()).getTime() / 1000));
>    logger.debug("Graphite Output Average: " + str);
>  out.print(str);
>
> What do you think about that?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SYNAXON/GraphiteReceiver/commit/6020b67bfba48f74675cda40e4b9c38089c5d4b2#commitcomment-11287137>
> .
>


If people understand that there is two packages then their relationship is not clear. It occured a few times that people thought MNE-Python just called MNE in the background and was only scripting layer, etc. Therefore more explicit words here.


agreed. I truly believe proper naming conventions. I like your suggestions.

On Wed, May 20, 2015 at 8:48 AM, Toni Moreno <notifications@github.com>
wrote:

> Hi , @lbasavap <https://github.com/lbasavap> , I'm reviewing your great
> work in order to upgrade my instalation and fix #13
> <https://github.com/SYNAXON/GraphiteReceiver/issues/13> .
>
> I suggest change "disconnection_graphite_after" by a more suiteble
> property name like "graphite_force_reconnect_timeout" , since you are not
> disconnecting you are rather force a reconnect , isn't it?
>
> I have doubts on what exactly means 600000 . ¿ Seconds , miliseconds?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SYNAXON/GraphiteReceiver/commit/6020b67bfba48f74675cda40e4b9c38089c5d4b2#commitcomment-11287141>
> .
>


Did you intend for ItemGetter to be called only once ? If that's the case, it could work the way it is.
Otherwise, the point is mostly to make the objects react the same way.


```sh
git push --tags origin master
```

Apparently you've done that in the past: https://github.com/jquery/download.jqueryui.com/commit/a9af23ba63ee1031f98349f5dcfae8f510b7f01f


Ignore this, outdated.


@HowardHinnant Great analysis! This could be the subject of a comment..


I also checked the length, all fine. I will check again including the cache, I suspect it might be the cache which has None values.


Sounds like a good target for the sprint :)


If we are going to have a `BasicPinnerCB` to build this function (which you currently have in the `herlpers` package), I think we might as well move it into the main importer package and use it here.


I think tickets for every single case will be hard to maintain. And we should let user know what exactly happened. In many cases they are dealing with old bugged filters which is easily fixed by uninstalling them.

Dynamic page with general explanation that the crash is outside MPC-HC and with information about crashing module will probably be easier to do.

I mean we can create tickets for every crash, but it is dull job :<


Add to #142


In general, something like [os.IsExist](https://golang.org/pkg/os/#IsExist) ([impl](https://golang.org/src/os/error_unix.go)) would be easier to maintain, it also deals nicely with `err == nil` and the helper can also access unexported fields (if needed).


TankDeadband (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1654 of 1657 tests passed)

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


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

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


Not a very descriptive commit message.


all these two operations do is set a value in a map, theres nothing fancy going on here. Normally, pinning.Pin would do this, but the problem is that we may be adding a large directory structure, and we only want to pin the root of that. So the fast and easy way of doing this is to just pin everything indirectly, then change the root pin to recursive like i'm doing here.


Update branch to master.


Yeah, I agree with @kasper93. The whole idea is to reduce the workflow...


Looks good to me.  This would also make it easier for us to implement the progress bar without a progress-bar-specific argument.


Per my testing I understood that ESX/VM is always associated with the
cluster. Therefore, Graphite receiver must receive cluster. Since our
requirement was only for ESX/VM I included only these entities in
traverseEntity in InitializeClusterMap method. We could enhance to include
other entities. Once we include all supported entities (we could make it
configurable in graphite.xml; like vm,esx,datastore, and etc) I don't think
there's a need for enable_cluster_map. Please correct me if I am wrong.

On Wed, May 20, 2015 at 8:56 AM, Toni Moreno <notifications@github.com>
wrote:

> @lbasavap <https://github.com/lbasavap> , if you do that you are ignoring
> all metrics from any entity not belonging to any cluster, isn't it?
>
> Also you are assuming that the entityName is always a host or vm , isn't
> it?. What happens with other resources? Perhaps could be good to add a
> property like "enable_cluster_map" to activate/deactivate the cluster
> mapping
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SYNAXON/GraphiteReceiver/commit/6020b67bfba48f74675cda40e4b9c38089c5d4b2#commitcomment-11287346>
> .
>


Firstly Thank you for all the icons sets and the work you put into these.
However I have to ask, could you please add Mac icons ? I know these would work on Mac anyway, though there are no Mac applications or folders etc.

Thanks again.


Хм, а можно в нескольких словах что кардинально изменилось, интересно же :+1:


nit: if a method with a given name is not a method or not exists, you will get some ugly exception like "scala.ScalaReflectionException: <none> is not a method". Maybe it would be better to require(member.isMethod) first with appropriate message?


Colud this cause security issues?


I haven't tested too much, but it looks like it should work. I hope to eventually update some of the other functions to use this one internally.


I just wanted to say how very sad I am that I did my semi-regular pass through AFNetworking for spelling errors and found them all fixed already!

But seriously: thanks. :)


@aghayoor I agree that the best is to force a human to fix it.


What do you mean by "Dynamic page"?  Something like external_filters_crash.php?filter_name=ffdshow I guess?


Does "go" instead of open work for windows?


CSS3 Box-sizing not supported by: IE (8,9,10,11), Chrome (37,41,42), Safari (8,7.1), Opera (26,27,12.1), iOS Safari (8,7.0-7.1,8.1-8.3), Opera Mini (5.0-8.0), Android Browser (4.2-4.3,4.4,4.4.3-4.4.4), Chrome for Android (42), IE Mobile (10,11), UC Browser for Android (9.9)


Flexible Box Layout Module not supported by: IE (8,9,10), Android Browser (4.2-4.3), IE Mobile (10), UC Browser for Android (9.9)


CSS3 selectors not supported by: IE (8)


CSS outline not supported by: Opera Mini (5.0-8.0)


CSS3 Border-radius (rounded corners) not supported by: IE (8), Opera Mini (5.0-8.0)


CSS Generated content for pseudo-elements not supported by: IE (8)


Yes, exactly.


Yes. I wasn't sure it would work, sorry.

On Thu, May 21, 2015 at 10:15 AM, Hannes Schmidt <notifications@github.com>
wrote:

> Not a very descriptive commit message.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BD2KGenomics/jobTree/commit/637ea8361bb8faa780afdf3ef0b3f19fe0efd244#commitcomment-11311242>
> .
>


> I think we generally use 0.0f syntax.

Ok, I can use that.


Роли были назначены


Элемент с таким именем уже существует


Имя класса правил, связанных с этим элементом


Ah, I see the reasoning here.
I guess I'm open to all three alternatives.

But, if you keep it this way, I'd put interesting default values. (Like, maybe once every 10 updates ?)


Объект класса правила не может быть создан


Класс правил должен быть наследником класса "yii\rbac\Rule"


Only the things in MainActivity and activity_main.xml are important. The changes to activity_about.xml, were made by accident. Its only naming things.


Описание (Необязательно)


I liked cluster_map_refresh_timeout attribute name. Every one hour (3600
seconds) cluster map will be reinitialized. That means, GraphiteReceiver
connects to vCenter and gets all available ESX/VM and initializes the hash.
This helps to get newly added ESX/VMs and improves the performance. Also,
avoids reinitialization during runtime. I hope this helps.

On Wed, May 20, 2015 at 9:03 AM, Toni Moreno <notifications@github.com>
wrote:

> @lbasavap <https://github.com/lbasavap> Perhaps
> ."cache_refresh_interval".is not a good property name IMHO the property
> should be more explicit .. something like "cluster_map_refresh_interval" or
> "cluster_map_refresh_timeout"
>
> what exactly means 3600 ? "seconds" ?.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SYNAXON/GraphiteReceiver/commit/6020b67bfba48f74675cda40e4b9c38089c5d4b2#commitcomment-11287501>
> .
>


Thank you , thank you.... all works !!!!!


Test 2


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

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


Added test 2 for testing purposes


Since its in a unit test, I guess I don't care as much as if it was in production code.


Enhanced the readme file


one more question - what if there are more methods of the same name? anyway you can do it without `asInstanceOf` by:
`tpe.member(newTermName(methodName)).asMethod.paramss.head.map(_.typeSignature)`


Thanks!


@Therzok I don't see why not, but I don't use it :)


Thanks.  I knew that's where we wanted it.  I have it hacked into my _-next_ branch for the moment and will remove it when I wrap it up later today.


isIndexValid and nextIndex could probably stand to just be replaced by implementing `java.util.Iterator<SerializedJSONRows>`.  Not a big deal, we can push this into a later refactor.


Не существует ни ролей ни разрешений с именем "{0}"


I think the paragraph you added should be moved to this location, or potentially made into it's own definition entirely.


fair point


Or even better, `semantic-format-tag-uml-concise-prototype` since it also includes return type. Having extra information is important; otherwise you are reducing `helm-semantic` to be less useful than `helm-imenu`:

- You have to enable `semantic-mode`, which many people don't like.
- It takes a brief delay to parse a large buffer (i..e 1 MB C/C++ source file). With `helm-imenu`, you retrieve it much faster, not really noticeable compared with small buffers.
- Currently, `helm-semantic` is already worse out of the box compared with `helm-imenu`, since you can't even tell between a variable/function tags immediately. You have to rely on theme colors, which is not always obvious, given some themes have monochrome colors.


same as above - no need to explicitly cast


yeah, it also lets us do fun things like live streaming a little bit easier since we can get the blocks right as they are added.


nit: test_explanation_template_sanitization , not sanity_check


A ``Timer`` task could have an internal value which would be updated once at initialization (``init``, not ``__init__``) and once at ``finish``.

This opens the questions of having a logger which would be used to collect data for further analysis after the training (like keeping track of train error vs valid error for plotting). We'd need a general mecanism to collect those data.

Or, we could put all these analysis as ``finish`` task. The point is that we need to find something that'll permit us to get data out.


Um, why are we running cppunit on flame-kk-eng?


Okay, would prefer to wait for those changes. You would have tested this when coding those functions.
This code is used in a few places which I don't want to break and don't have much time myself to debug.


It looks like `rxe->arbiter.skb_count++;` can be move out of the if statement and then we only need to check `if (!rxe->arbiter.skb_count)`.


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

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


What was the concern with the original code? That another thread could also pop from the list between the `len(self.data) > 0` check and the popping? Would that happen? From what I see, the simulator thread is always appending to the list, while the component thread is always popping, so we should be okay.

My concern with the new code is that if something gets appended to the end of the list while we're looping through, will it still get sent? Also, with making a new list, what if `gather_data` gets the `self.data` attribute, then the component thread does `self.data = []`, and then `gather_data` does the append?


Perfect! You can also use floaty /= 3;

But that's completely a matter of opinion. I use it because it's a couple of less characters.


this is now missing the flame-kk-eng tests


Great work!


Enhanced test 2


this is missing the test task definition now


I think so, tab completing quoted strings is important.


This same test task is also above under cppunit


Again, awesome stuff John. Keep up the great work!


missing flame-kk-eng task that was originally on b2g-i


This has been included in [Matrioska 0.3.0](https://rubygems.org/gems/matrioska)


Umm... I'm not following. The relationship object definition refers to the terms *related resource URL* and *resource linkage*, in that order, so I put the definitions in the same order. Would you like them to be switched around?


> My concern with the new code is that if something gets appended to the end of the list while we're looping through, will it still get sent?

That was my original reason for the pop approach.  The thread that is running the simulation will be adding to that list, and I didn't want to miss any items for sending.


Perfect.


88737258-EPW-Ruby-Access (jasondegraw) - 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)


@crosbymichael https://github.com/tiborvass/docker/commit/81fb20907d18e715dd574a382f41610445f67cb0#diff-7025b8c38be826dd40b8f9f14310b7c1R119


Good catch! .... and very sloppy test before final commit :-/


ycmd isn't a fan of global packages but local copies. This ensures we don't mess with the user's system and that we have a hermetic build that doesn't conflict with whatever the user is doing with their global configuration.


@garyj I welcome your feedback.


You need to make sure to generate valid ServicePermissions. This means the value can have a trailing wildcard (`.*`) or no wildcard. So `*Helper` cannot be the value of a ServicePermission since it is not valid. So you need to do some more data validation here since filter wildcards are more powerful than what [ServicePermission supports](https://osgi.org/javadoc/r5/core/org/osgi/framework/ServicePermission.html#ServicePermission%28java.lang.String,%20java.lang.String%29).


:smile:


If popping from the start is a commonly used function, then we should be using [`deque`s](https://docs.python.org/2/library/collections.html#collections.deque) instead of lists. Then there will be no performance hit from popping from the front, so this code can stay as is (well, with `popleft` instead of `pop(0)`).


No, initialization lists default to value initialization, see for example http://stackoverflow.com/questions/11164394/constructor-initialization-list-with-empty-initialization

Also, pointers and ints below are also PODs.


functionally equal to rosbagToPointCloud


Good -aj


Don't need else here since the if body returns.


@dosten We know. the initial commit was created by duplicating the basic structure of a bridge, to be able to start usign PRs. Unfortunately, it was taken from an older branch of Symfony, not from the 2.8 branch.


As a side note, that even works for array types, which is awesome.


Si j'ai bien compris, dans le truc sur les promises que je t'ai envoyé, il ai dit que deferred ne devrait jamais être utilisé.


Be careful, this can end with lots of perl processes causing CPU trashing. :warning:


@mikedurbin, did you happen to touch base with @awead on how the https://github.com/projecthydra-labs/fedora-migrate mappings compare to this table?


Why these new includes?


Looks good to me. There is no `close` method on the object returned by `net.connect`, only a `destroy` method. Also, Returning the error via callback is consistent with handling in nearby code.

Reference: https://nodejs.org/api/net.html#net_class_net_socket


+1. deque is also thread-safe (https://docs.python.org/2/library/collections.html#collections.deque)


My fear is that if you have two tasks working on the same variable, without a way to combine the updates, one would override the other. By duplicate, I meant lost updates due to override. By forgotten, I meant that if you don't code your task right, the update might be lost. Frankly, I admit I haven't put a lot of time on it, but I'm still not clear on the right update path (which update gets passed to what object).

Something that could be easier (only an idea) : ``Trainer`` could manage the master update list by registering the tasks as it is put into the task list, then the optimizer, etc. This way, every task would manage its own updates.


@dpgeorge is `esp.wifi` supported? I am kinda looking to do a similar construction to be able to support:

`from Crypto.Cipher import AES`


этот тест не работает значение не меняется после клика


I love helpful charts! And also citing sources inline. I think that's my favorite use of comments. n_n


Could the docs explaining the new feature be updated, too?


@ricardo change the ozoneconfig file so it doesn't have the di2e links, rebase, then force push this branch back up.  Otherwise the alt tags look like a good start until we get guidance on what alt tag should say.


The time per epoch is already managed in a task, isn't it?


Why not make the timeout a variable?


These includes aren't needed, are they? This header uses nothing from them.


You brobably should add a method displayStandardError on the InfoModal service to avoid code duplication.


Awesome! Not sure where is best to put comments... but thanks for the great work!


это, скорее всего, можно отрефакторить


Looks good to me.


Urgh, that seems like a cheap hack and is racy on slow X11 connections.

Can't we update the xkb state from events, without throwing away the full state on every event?


tous


Whoo hoo! Congratulations on getting this to work in the db!


tooltipe ?


I guess using a ``while`` loop with some kind of epoch stopping criterion would be more general.


Fair enough. I usually assume something is going to work, such that if it does work I don't have garbage in the commit log. You can also change the commit log message of an unpushed commit by amending it via "git commit --amend" before you push. Again, SourceTree makes all of this very easy.


Oh! Clever!


Me, too! I don't think we've covered recursion yet in class, but it's my favorite secret loop so far.


I must be missing something really obvious... what does this solve?


@Nevcairiel: Do you keep the PDB files when releasing a stable version?


Second-order derivatives, for example. But those are just another method for model to be entered. Probably for the next PR (or anytime I'll need those).


Like cutting out the first part of the block? ;)


This fix was to get OsmAnd to build.


Sure but to honest I don't like it. If it wasn't for the nightlies I would
have it turned off completely.
On May 21, 2015 8:17 PM, "Underground78" <notifications@github.com> wrote:

> @XhmikosR <https://github.com/XhmikosR>: Can we integrate a php page
> easily on our static site?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Underground78/mpc-hc/commit/39cbfa563e7aed1f9397f74e87a1d508d3064ae3#commitcomment-11311326>
> .
>


Nice job! I'm just curious to know why you used k as your looping variable.


What the purpose of it ? (Just to understand)
Favories is not a word. (By the way your code begins to be very readable but there is a lot of typos like this and you should take care about it.)


If you're having to do this the bucket's set up wrong :(


Of course, they are even linked on D9 in the original post:
http://forum.doom9.org/showthread.php?t=156191


Now there is a matching bug to "tm": http://trac.cppcheck.net/ticket/6701


sorry I hastily rebased...this is basically what I added thanks to your tests.


What about this approach?

```python
    def update_client(self, client):
        count = len(self.data)
        if count > 0:
            for d in self.data[:count]:
                client.write(d, binary=True)
            del self.data[:count]
```


Right, I didn't think about D9 and there were not on Github.


Well done. I thought about doing this yesterday when you showed it to me yesterday.


Fixed in 785242


So what do you think we should do? If we don't tell which external filter or driver it's basically useless.


Doesn't this make the _flags.recalc_leash_Z = true; redundant?  That flag is supposed to trigger these getting recalculated.  Anyway, if we're going to explicitly call calc_leash_length, I think all the logic with those flags is no longer necessary?


I should add an example of the result.
![00](https://cloud.githubusercontent.com/assets/818053/7755055/47920dbe-ffa6-11e4-92d0-63403d3053d0.png)


Nice work John!


Looks good to me.


Хороший вопрос) Видимо я в начале сохранила в jpg, а уже потом png, но jpg удалить забыла.


But it doesn't see the ``updates_from_param_modifiers`` dict.

Suppose that two ``param_modifier`` tries to make an update to the same variable. One would override the other and an update would be lost.

Or maybe I don't get the updates at all...


lol name. You have two different functions ? One to add a lot of tracks and one to add a few tracks ? Either it is an odd name since it makes think it is the case.


Excellent addition :)


compare with undefined should use !== and ===


I don't understand why you need PHP since I haven't completely understood
the suggestion. Can't you do the same on the client side with Javascript?
On May 21, 2015 8:43 PM, "Underground78" <notifications@github.com> wrote:

> So what do you think we should do? If we don't tell which external filter
> or driver it's basically useless.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Underground78/mpc-hc/commit/39cbfa563e7aed1f9397f74e87a1d508d3064ae3#commitcomment-11311829>
> .
>


Переделала на сss)


`deque`s sound fine to me.


Tu Do <notifications@github.com> writes:

> How about semantic-format-tag-concise-prototype?

Done.

--
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997


Should be fixed in 558e459da3368aacabbeb3fb35d98e7742cca11a and 709aaba876a56c3757668736813a447543a363fc.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 305](http://192.168.30.225:8111/viewLog.html?buildId=1248&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Removing this check broke the build on Windows


Seems alright, but more complicated than it needs to be... I think the `deque` is the natural data structure to use in this case, and it's been in the standard library since Python 2.5 or something so we don't have to worry about compatibility.


I originally did have the signature covering the options as well, but I grew concerned about the difficulty in correctly constructing the data to be signed (see https://github.com/willnorris/imageproxy/issues/11#issuecomment-101140435).  I really wanted to try and keep the signature in the current options path segment rather than introduce another one just for the signature.  Though I could see someone attempting a DoS attack by requesting images at tons of different sizes... that's certainly something I've thought about.

As for encrypting the remote URL, I've certainly thought about it (or other methods of masking the remote URL), but thus far have shied away from it, mainly because I couldn't come up with a compelling use case.  I'm curious, what are you using it for (if you don't mind sharing).


учту!)


Choose a convention and stick with it with for "var i=0": you use var tr = 0 just above (with spaces).


You've got the right idea here in checking to see if there's any hair to cut, but just being above 0 is not sufficient, unless you're somehow checking to see if a resulting hair cut will give a Dog negative amounts of hair.

So, try and rethink what value mHairLength needs to be greater than in order to get a haircut.


Hmm maybe if we parse the URL, not sure I would call this clean though. The idea was to write a generic text for external filter crash and automatically display the right name just by passing it in the URL. The crash reporter would then open this page if a similar crash happens again.


Wtf had the release plugin done here...


``requireNonNull``


causes an assertion failure on 32 bit? https://travis-ci.org/JuliaLang/julia/jobs/63521090
```julia-debug: dump.c:1012: jl_deserialize_value: Assertion `bp' failed.```


The if statement (if num_files > 0) is probably not needed.


Done.


It's still unnecessary use of memory. Depending on the number of parameters, this could grow a lot.
Why use a vector? Can't you use a shared scalar ?


Is ```resolve``` defined at this point?


Same as `=== false`?


Browser should default to Sauce to run in Jenkins


I am trying to use fill color to white so that before original image gets load it remains on the screen but it doesn't work.


Please don't break existing public interfaces. Other classes expected a `getRooms` method without args. There was no reason to break it, it only begs for merge conflicts. If you need more, add a new method instead :wink:


On Thu, May 21, 2015 at 10:15:13AM -0700, Jeromy Johnson wrote:
> So the fast and easy way of doing this is to just pin everything
> indirectly, then change the root pin to recursive like i'm doing
> here.

Ah, that makes sense.

I'm still concerned about the lack of contexts or returned errors.  Is
pin removal/addition something that never blocks or fails?


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


The shared ``lr`` variable is never read. The update would happen and you'd override it the next iteration on line 32.


What is scope.play ?
To keep consistence with your code conventions, you should write "length - 1" (with spaces).


we need to reference the brunch from the local path, ,i.e.:

    npm install && ./node_modules/brunch/bin/brunch build

Otherwise this will fail unless users have previously installed brunch globally.


whatever you like, just to remind you :smile:


@wilderrodrigues It's OK to use Artifacts, although people outside SBP cannot access it.


ou :hankey: fixed it.


Ja i weiss, das hani nu dinne gha fu de alt methode, wohni nu müe gha han. Aber es isch ez scho klar: id für DB und uid für java klasse ;)


Strange indentation: I would have writen "function(artist) {" on the same line or go to the line before the ".then".


nice seeds, David!


- If you set FAST_FAIL=True in the command line, the test will abort if the page returns a 404. This saves us from having to run every scenario if the page is not available.
- Added handle error tags so that test will take a browser snapshot if a tagged step fails


Same strange indent


This does not work for me, the agent of the SSVM enters Alert state. When I delete the template from the db and restart agent it happily connects. Shall we have a look tomorrow? :-)


Updated Quick Calculator FAQs page and corresponding URL


length


рр


Thanks! :)

But I have just noticed a bug in the way the icons are loaded though. But i think it should only affect users that are using the special CURRENT_DOMAIN for the icon url. I will try to find a fix for it.


Are you sure about removing the `m_ImageProcessingLock` lock here?


Check out this podcast on iTunes  Fresh Air NPR                                                                                (2067) Arts, Podcasts, TV & Film, Society & Culture May 20, 2015 View Item   Fresh Air from WHYY, the Peabody Award-winning weekday magazine of contemporary arts and issues, is one of public radio's most popular programs. Hosted by Terry Gross, the show features intimate conversations with today's biggest luminaries. You have not been added to any email lists. Copyright © 2015 Apple


Oh, interesting! Most of my experimentation pre-ADA was JavaScript, so I developed the habit of always defining something before it gets called (inside another function definition or not).

I like this idea of setting it up for the human reader to understand better, not the computer interpreter. :)

But it's still only possible in Ruby because the actual method call is at the end of the file, right? Like: the interpreter is already aware of the other stuff, because it had to pass through it to get to that last line. Yes? No? ~~Jeri, just go google it? >_>~~


принять


Oops. Forgot to rename this to RespokeSocketRocket


hani gmacht, ebe da hani nu fehler dinne gha und han nöd gnau gwüsst wie dasi mues mit hql joine, han jetzt d uid gno, kei angst ;)


Good fix :-). Thanks


So you mean to say you want a new page on the website.

I still don't see why we need this. I thought drdump was supposed to take
care of this.
On May 21, 2015 8:48 PM, "Underground78" <notifications@github.com> wrote:

> Hmm maybe if we parse the URL, not sure I would call this clean though.
> The idea was to write a generic text for external filter crash and
> automatically display the right name just by passing it in the URL. The
> crash reporter would then open this page if a similar crash happens again.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Underground78/mpc-hc/commit/39cbfa563e7aed1f9397f74e87a1d508d3064ae3#commitcomment-11311953>
> .
>


I think "Error Objects" would be consistent


as far as nginx secure_link... are you actually offloading the signature checking to nginx?  If not, is there some other value to using a compatible signature method?


I had them over their blocks at first but learned the hard way that it was super annoying to have to scroll through to find the one I wanted to change as I added more synonyms. So I'm 100% with you.


@wesnick Yes


```java
if(limit < allRooms.size)
 allRooms = allRooms.subList(0, limit)
}
```
saves you 2 branches and 5 sloc...


TeamCity Ybackend :: synteny/ybackend [Build 27](http://192.168.30.225:8111/viewLog.html?buildId=1249&buildTypeId=Ybackend_SyntenyYbackend&guest=1) is now running


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 305](http://192.168.30.225:8111/viewLog.html?buildId=1248&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:45


Fixed.


Fine to merge once that one nit is handled. :+1:


What if m.value is <0?


You should put a blank line between your functions.


@Diego-F-Aguirre The issue you are having is caused by this line. It should be `NSString *accessToken = [urlString substringFromIndex:indexOfTokenStarting];` Consider the difference. The result was that the access token was not being saved - and that is required to fetch data. I found this by placing NSLog(...) statements in various spots.


Ah right, didn't realize that. I just moved this from sponge vanilla (https://github.com/SpongePowered/SpongeVanilla/commit/2d293b85893422f1e4402de2b11fcafb20a59585)


nai es ischmer ez scho alles klar, han abr scho bits müe gha... V.a. willi denkt gha han dassmer uf DB level mitem gliche söti joine wie uf java level, damits kei komplikatione git.


And 'subtarct' / some misspellings or 'submarine' / hey look I can 4 submarine 4 in this calculator!


Was the code broken or did you introduce an unused dependency to `$rootScope`?


I would have used i in all of my loops, but it gave me errors when I did.


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1650 of 1650 tests passed)

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


What if m.value is calculated to be less than 0.  Shouldn't you have an extra condition of if m.value>0?

if (m.value>VALULE_ZERO &&  (ourAttacks | theirAttacks)
          &&  (theirAttacks & to_sq(m))
          && !(ourAttacks & to_sq(m)))
        m.value = VALUE_ZERO;


If you look at the comments for the params of this constructor, you'll see that funny looking Artist... artists parameter.

This isn't just for kicks, and is actually a very powerful feature of parameter passing in Java and it is called variable length parameters. While you are correct in saying that you *could* pass in an array here, that's not exactly what we're looking for.

Check this quick post about variable length parameters and then refactor this cute to not take an array as a parameter, but rather store the variable length number of artists into the array in the constructor.

http://www.deitel.com/articles/java_tutorials/20060106/VariableLengthArgumentLists.html


Please use `$log`. It's already injected into the controller and prevents failure on other browsers


I think we need to add some default values here, double check the comments.


this in templates? To what does it resolve?


Thanks ! How did you hear about it ?

This library has the same goal as the MDTraj, and sare some functionalities with the MDAnalysis, which are also in this list.


Either the name of the function isFollowedOrganizer is wrong or nothing is done (I bet for the first choice).
Indeed, isFollowedOrganizer should return a Boolean (as all the functions beginning by is). If it only returned a Boolean (and do nothing else as it should do if it respects the single responsability for a function) the for loop would have no effects.


Currently the message is for meant for devs and isn't really visible for the user. So he would know that we won't fix the issue but the reason is not so obviously visible. The DrDump guy apparently thinks the "response URL" should be used for that but I might get it to show the message to the users.

I see some reasons to use the response URL though:
- it is easier to control the appearance of the page so we can basically write more text, with links or even images if needed.
- the message wouldn't be duplicated (so we can change it more easily if needed).
- just pasting an URL might be easier on the long run.


console logs should not be commit


That's strange... We can check that out later if you'd like.

On Thu, May 21, 2015 at 1:58 PM, John McIntosh II <notifications@github.com>
wrote:

> I would have used i in all of my loops, but it gave me errors when I did.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ver2point0/android-source/commit/eabf9c7cf993c7cb5b8b96f3e7fa0bb08f7a975b#commitcomment-11312238>
> .
>


As you can see, the distribution for the sitting trials (for when 1 person is just standing and the other is walking around), has a much fatter tail towards the either step values than that of the distribution of steps when both people are walking around.


It would have to be greater than 0.15f because that is the HAIR_CUT_LENGTH.


Ok, I did not realize that you also maintain Julia.jl =). So it should be through the julia-user mailing list.


/api/deck/new/shuffle/ Is actually the correct url.
/api/deck/shuffle/?cards=AS,2S,KS,AD,2D,KD,AC,2C,KC,AH,2H,KH returns a 404.


ok


This breaks compatibility with mongoid-paperclip


Are you sure that locking `m_ImageProcessingLock` isn't enough? To be honest I have not looked it details yet but I mean if you lock it, doesn't it ensure that the queue is not accessed? Also I would be very careful with removing the main `m_ImageProcessingLock` lock when emptying the samples. So maybe we should add the `m_SampleQueueLock` locks but still leave the existing ones.


Whoa, this is both cool and terrifying. Is there really no better way than OCR? I know that my EC2 instances have some way of pulling out boot console data, and I'm hoping that it's not OCR'ing my stuff.


I really do prefer to create a real bean class in java for testing. For example, in a real Java bean, the fields are private.


TeamCity Ybackend :: synteny/ybackend [Build 27](http://192.168.30.225:8111/viewLog.html?buildId=1249&buildTypeId=Ybackend_SyntenyYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:26


Exactly! Go ahead and make that change and commit with a message of what you've fixed.


User seen this page when crash is uploaded https://drdump.com/UploadedReport.aspx?DumpID=3588551 There is ability to see problem history and explanation, but clicking on "Problem" link. User then is redirected to this page https://drdump.com/Problem.aspx?ProblemID=132609&Login=guest The information is hidden too deep, that's where "response url" come for help.

Probably if we have grater control on `Problem state` text we could reuse this page, but we currently can't edit it.


LGTM


the same comment as above


ok
but in addition I anyway want to change all

    if (x)
        something();

to

    it (x) {
        something();
    }


Finally thanks. For the time being.


Thanks! :+1:


@copumpkin that's for GUI tests ;)


fixed with #128 half an hour ago ;-)


wow! what a precision :)
btw. can multiple includes be chained?


That was special, because this string is in general WP files, so what's the reason giving translators one phrase more?


Select is closer to:

```
result = []
array.each do |el|
  result << el if yield(el)
end
result
```

Reject is similar. Probably something like:
```
result = []
array.each do |el|
  result << el unless yield(el)
end
result
```.

Good on ya hitting hard mode. :)


Thanks for your note! Can you send a PR rolling back this change, with a regression test?


Oh, so there's a separate way to get console text out of the system in non-graphical scenarios?