In every samplewidget of the WebAppbuilder 1.2 the wabVersion, in the manifest, is set to "1.3".


parallel builds fail only sometimes, that's why they're hard to reproduce and fix, no problem.


"newser" worked.


4952-FixBranchAgain (rraustad) - i386-Windows-7-VisualStudio-12: OK (1815 of 1815 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)


maybe add a comment to indicate where these defaults came from (I presume it was [here](https://docs.python.org/2/library/logging.handlers.html#sysloghandler)?). Do these need to be configurable?


http://one.ithanks.kr 생성


Same here on GS3 (T-Mobile): Chrome failed during startup with an
unexpected error

On Wed, Sep 2, 2015 at 1:19 PM, mfonville <notifications@github.com> wrote:

> I just discovered the same, and wanted to report the same error. I am
> looking into it now.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opengapps/arm/commit/1c406ec1dce8f9c75b42eadc61d02305622facf1#commitcomment-13030172>
> .
>

--
<http://www.facebook.com/djalik>  <http://twitter.com/#!/DJAlik>
<http://www.linkedin.com/in/alexrusinov>
<http://plus.google.com/114580313768137070684>
<http://www.mixcloud.com/djalik/>
<http://psnprofiles.com/DJAlik>


let's talk about the jquery thing as well. i'm fine either way but we should just evaluate exactly what we're using it for and if it's needed for that. if it's just a few things then we might be able to get by without it. maybe not if people care about ie8.


Utils sounds good but should I use flask-paginate instead? As suggested in Slack channel?
________________________________
From: Anand Chitipothu<mailto:notifications@github.com>
Sent: 02/09/2015 23:13
To: RaghavPro/cleansweep<mailto:cleansweep@noreply.github.com>
Cc: Raghav Sharma<mailto:Raghav_ftw@hotmail.com>
Subject: Re: [cleansweep] Added a macro for pagination. (9cb03d0)

Looks good to me.

I'm trying to move all the code away from top-level. It would be better to move `pagination.py` some other place. May be `core` or `utils`?

It would be useful to add unittest for this.

---
Reply to this email directly or view it on GitHub:
https://github.com/RaghavPro/cleansweep/commit/9cb03d041d18d7feba0e92232a23b5cc82dba416#commitcomment-13029454


Initial commit


I'm having trouble reproducing this.  Is there a stack trace?


might be nice to make this configurable; sometimes might want to see debug logs, for example.


Change increaseRate to 1 + increaseRate (0.05 to 1 + 0.05).


@schildbach Sounds plausible candidate for a backport


I do have a demo page in this repo you can look at under src/FormElements/demon/index.html. Maybe that will help. We plan to provide this markup sample in our guidance.


> .. do you have spaces or tabs in yours?

Nah I don't , but I don't see why the comment detection should be so strict, especially since it doesn't complicate the regex much.

Given a `~/.snaprc` containing
```
INTERACTIVE:true
 #INTERACTIVE:false
```

running `doas ./snap` results in this
```
msg: ./snap[312]: [: false: unexpected operator/operand
msg: ./snap[318]: [: false: unexpected operator/operand
msg: ./snap[324]: [: false: unexpected operator/operand
```

presumably because snap tries to parse the option line. I don't observe the issue with my patch.

The meaning and origin of the above error message above isn't exactly obvious, so a user who accidentally hits the space bar (or tab) before `#` might be very confused.


does having the basic configuration mean that log messages will get sent to stdout or stderr as well?


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


I think the test stripe keys can remain in the gruntfile.  It makes it much easier (read: possible) to run and test a local version.


IMO the i_video code shouldn't be aware that gametic exists. It's probably better to just set a flag for "should draw a disk icon in the next frame we blit to the screen".


HFE Metadata update


New FIR table and direct column added in HFE metadata


Hooray! Well done, @snugthejoiner. I am worried about having to override current_user. Thanks for keeping me posted.

![popkey](http://d3htz3r28tzhqz.cloudfront.net/wUYaerRirBIFZHkXfQiG70od03A=/original-02c53062-3086-4ef3-917c-8c60d7df221f.gif/wGlvA.gif?c=desktop&l=search&f=.gif)


It's a vfs directory so CDirectory will show busy dialog it it takes too long


Thanks .... :+1:


error parameter here should be `NSError **`, right? Otherwise you cannot get the error back out of the callback (same issues below).


That would require importing Model, which unfortunately isn't possible.


hello this is from my issue #100 ?
i did glow from pyplug is slow performance.. so i want to try your plug for glow :-)
i'm waiting


I'm ok with adding support for specifying multiple shortcuts in add-ins. We don't need to surface this in the UI. If a user sets a custom shortcut, it would override all default shortcuts.


And I agree this fix is not right.


Being preferred by upstream doesn't make this more correct. Explicit prefixes are too useful to avoid header conflicts, be it with SDL1 headers (possibly used indirectly through third party library) or with SDL2pp headers (with your change, only the fact I've used .hh extension saved you from conflicts). They also make SDL2 and SDL2pp includes clearly distinguishable when reading the code.


Oh, I did reproduce it by trying to upload a *.tif file.  What type of picture are you attempting to upload?


Testing
secret_access_key elsevier


It'd be nice to report this upstream so that it can get included in the next Mini-XML release:
https://www.msweet.org/projects.php?Z3


You don't need QString::number() in this case. arg() can take int too.


@milancurcic sure are right, the externals should be compiled into their own room. Tomorrow I can try to fix it.


+1 for replacing cp with `if err := io.Copy(...); err != nil { util.HandleError(err) }`


No test?


Good to merge


It's a question whether hurry such changes into 15.09 now, though there's a few weeks and it sounds it could be done in a backward-compatible way.


Thank you for the advice!

But I've already considered to 'play' with PYTHONPATH. The fact is this environment variable is not always defined on Linux.
Playing with the configuration files in the user directory is not good too, since we need the 'local' lib folder only for MCEdit, and the users may not want to have it permanently added to their environment.

Moreover, the needed libs are installed in the user lib folder only when a complying version is not found on the system. The point is to not overwrite stuff that other applications may need with the system instaled version.

In addition, the MCEdit Linux installer try to install as much as possible the needed stuff without root elevation, so modifying system files is not good. (By the way, this is still a work in progress...)

Quote from http://unix.stackexchange.com/questions/10113/configure-python-to-include-another-directory-when-looking-for-packages:

> 5 . Change the module search path on a per script basis. This is done by appending to sys.path, namely
>
> import sys
> sys.path.append(some_additional_path)
>
> This will only work for the script that is importing this. This is normally used, as far as I know, for casual use,  when importing modules in nonstandard locations, like from somewhere in a home directory.

This is the solution I use, with the difference that I don't add the paths to the end of the `sys.path` list, but in front. This ensure that the MCEdit specific libs ar found before the system ones, and the change is not persistent.


You can add tabs to align this part of the code :-)


Just to make sure I understand what's going on here. Can you verify:
- The powerset of boys is created to have all possible combinations of which boys are telling the truth
- That set is then filtered to produce only those sets which contain three boys telling the truth (the teacher's statement)
- Then with honestList you filter out all the worlds in which the boys do not agree with each other (e.g. in which one of the boys says the other one is lying)
- Finally you take the honestList and determine which person(s) are accused by all honest people


Isn't this always false from the line below? Maybe the line above should set it to true?


Since you changed the function, you need to pass "limit" not "limit/1024."


It looks like the comment is not up to date.


Not done, but since you insist .... http://downloads.natron.fr/~olear/Glow.ofx.bundle-0.1-Linux-x86-release-64.tgz


No problem. I created a PR here: https://github.com/Olivine-Labs/lustache/pull/18


Получаю ошибку "Trying to get property of non-object" в
$action = $event->sender->module->requestedAction;
Если вернуть к $action = $event->sender->requestedAction;
то все работает. Приложение на основе yii2-basic.


The assignment should be deleted here.


Dat klopt helemaal Bouke!

Op 2 sep. 2015 20:46 schreef bgrevelt <notifications@github.com>:
Just to make sure I understand what's going on here. Can you verify:
- The powerset of boys is created to have all possible combinations of which boys are telling the truth
- That set is then filtered to produce only those sets which contain three boys telling the truth (the teacher's statement)
- Then with honestList you filter out all the worlds in which the boys do not agree with each other (e.g. in which one of the boys says the other one is lying)
- Finally you take the honestList and determine which person(s) are accused by all honest people

---
Reply to this email directly or view it on GitHub:
https://github.com/bgrevelt/PT_MA1_3/commit/e59e747395f88bec9b50cad1e8208a778f4af229#commitcomment-13030848


On Sep 2, 2015 8:31 PM, "Guillermo Bonvehí" <notifications@github.com>
wrote:
>
> There's no access to the current model from inside the widget.
> There are two solutions:
> 1 - Reload selects using javascript on load, I did this at home and
forgot to commit it, not the best solution but it works.
> 2 - We could set the current instance to the widget by setting the model
instance from a model's constructor. I think this is more hackish than 1
and less flexible I guess.
> Here's an example of option 2: http://stackoverflow.com/a/2135647
> self.fields['municipality'].widget.form_instance = self

I'd vote for the hackish one if it didn't require defining extra form class
explicitly, as this cancels the biggest advantage of this package (just use
it). This could be handled 'behind the scenes', but it would require using
custom ModelAdmin which uses custom ModelForms... (or patch Django directly
and/or make a feature request (buahahahha...))

So it seems we're left with #1 - maybe not beautiful, but doable with 1-2
lines of JS. If you have it done, push it somewhere and I'll happily give
it a try at some point.


borrar  setaFired();
quedo arrasrtrado del rebase


Most of our check scans are TIF's :(

David Stevens

*Web Developer*NewSpring Church
www.newspring.cc

On Wed, Sep 2, 2015 at 2:42 PM, Mike Peterson <notifications@github.com>
wrote:

> Oh, I did reproduce it by trying to upload a *.tif file. What type of
> picture are you attempting to upload?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SparkDevNetwork/Rock/commit/a7a87379ad8462026290dde3df8cf6d90963441c#commitcomment-13030763>
> .
>


I don't understand the code with the test. `a + 1` broadcast 1 but it is a `numpy_expr<sum, type(a), brodcast(1)>` acces and not a `broadcast(xxx)` acces. Am I wrong?


what is the purpose of this file?


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


New example 2


I think @edolstra already fixed it: c090efb9d84b7be09c1109de7a39e6ac711d6b16. I guess that works because the channels have been merged (no more nixpkgs vs nixos channel distinction?).


r+


Please use `flask-paginate`.


this.model.set(selectedProduct);


empd (jdwoods21) - x86_64-MacOS-10.9-clang: OK (1786 of 1791 tests passed)

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


it's FASTER than pyplug glow...!!! AMAZING...


I can make it go faster if built against OpenCL


Maybe rename it to `GetAddonsFromRepository()` or something like that?


Please fix and submit again. Humanity will be grateful.


No way! I think you're on the right track with this one lol


Add unit test to cover the bug fixed.


I'm good with leaving it as is.


saw approval from graydon
at https://github.com/MonsieurNicolas/stellar-core/commit/874a01b889f48caa11c9ec45fa5c3774f13e75b7


merging MonsieurNicolas/stellar-core/versionInfo = 874a01b8 into auto


MonsieurNicolas/stellar-core/versionInfo = 874a01b8 merged ok, testing candidate = e4950d05


It's supported by most text editors and IDEs, it forces formatting constraints.


This is exactly what `virtualenv` is for.


La cosa es que "particionar" no está en el DRAE. Aunque tampoco es que me importe mucho a estas alturas lo que esté o no ahí. Haciendo una búsqueda rápida veo los dos términos usados en este contexto.

¿Y "hacer una partición"?


Indentation looks fine in sublime. I'm not sure why it looks funny here.


please give feedback (does it work as it should?)


![olear-glowofx](https://cloud.githubusercontent.com/assets/12126929/9641103/37d6e26c-51b5-11e5-9c4a-18b67158ccac.png)
it's true faster, but glowofx good but poor features.. you can see pyplug of glow..
thank you for create glowOFX


Did you forget something in this commit? Or did you mean to have it unstylized?


One pedantic comment: s/make/mark/, IMHO. Nothing is truly made here. (I thought it as well when seeing the funcs earlier, but don't care as much about that -- it's primarily an internal API there.)


OK. I'll fix it up quick and let you know


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (759 of 759 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-9.37%25-green.svg)


the two test failures in test-malaria.R started happening with this change to animint.js.

When we do

```
selectized_array[s_name] = $temp[0].selectize;
```

I see in firebug that `$temp[0]` is undefined. Does this commit alter that somehow?

![animint-bug-temp0-undefined](https://cloud.githubusercontent.com/assets/932850/9641163/3cd7ce3e-5183-11e5-86d0-fb58467a7c6c.png)


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


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


Ho shit. With template inheritance this is not so simple.
We need to manage this globally, avoiding `if ($content_only)` in theme files would be great.
I think we can change the layout on runtime in this case and use the stream resource to generate an empty layout on the fly, see http://www.smarty.net/docs/en/resources.extends.tpl


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


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


wtf this test is crap. echo -n is not portable !! enforces buffering of the output so it will probably fail to pipe. and you can just specify an empty string with ""


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.950.9ee6294221b75ce07660e7ec16057999d29e0486](http://build.palaso.org/viewLog.html?buildId=62555&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 499, ignored: 6 Build time: 00:07:47


This should be `utils.css` if you pull the latest from commercial


The example below is the way we'll need to specify methods in a module, since we can't directly link to them. Thanks @danielkocsis; looks good!


Походу этой строки быть не должно было! Это ты не до конца конфликт разрешил


I think I need this fix. The Transfer button no longer works for me on the "Transfer from JCR to file system" choice. Nothing happens, nothing in browser console or the Tomcat log.

Any possibility we could get a new release of PRS with this fix? The latest in the Marketplace seems to be 14.12.10.


Ok, will see what I can do ;)


worked for me and for now.. nothing crash and bug for now..
i will keep to report of glow and idea for glow ok? it's possible turn on and off for openCL and openGL swap them? (this is base idea...depend from you CRAZY CODE..?)
:-)


Instead of interning one can use asdf/uiop:symbol-call. Something like
```lisp
(uiop:symbol-call :prove 'run c)
```


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


OpenCL must be built in ImageMagick, will enable OpenCL after Natron v2.


Agreed, @danielkocsis. It looks good! Thanks.


Eclipse is telling me "Resource leak: 'reporterSpecificClassloader' is never closed"


Eclipse is also telling me: "Resource leak: 'reporterHelperClassLoader' is never closed"


(really for line 14, but I'm too retarded to get github to allow me to comment on line 14 in this commit)
Isn't this incorrect?
It looks like Arnold is saying that both Peter and Matthew are lying.


ok but openCL 1.2 or 2.0? with GPU mode?


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


thanks,  q(). is working again, but exit is not clean
````
2> 22:00:18.856 [warning] lager_error_logger_h dropped 26 messages in the last second that exceeded the limit of 50 messages/sec
22:00:18.856 [error] Supervisor ranch_sup had child {ranch_listener_sup,http} started with ranch_listener_sup:start_link(http, 100, ranch_tcp, [{port,8001}], cowboy_protocol, [{env,[{dispatch,[{'_',[],[{[<<"static">>,'...'],[],cowboy_static,{priv_dir,web,"static",[{...}]}},...]}]}]},...]) at <0.195.0> exit with reason killed in context child_terminated
22:00:18.857 [error] Supervisor cowboy_simple_bridge_sup had child cowboy started with cowboy:start_http(http, 100, [{port,8001}], [{env,[{dispatch,[{'_',[],[{[<<"static">>,'...'],[],cowboy_static,{priv_dir,web,"static",[{...}]}},...]}]}]},...]) at <0.195.0> exit with reason killed in context shutdown_error
22:00:18.858 [error] CRASH REPORT Process <0.196.0> with 0 neighbours exited with reason: killed in ranch_conns_sup:terminate/3 line 217
22:00:18.859 [error] CRASH REPORT Process <0.332.0> with 0 neighbours exited with reason: no match of right hand value {error,eaddrinuse} in ranch_acceptors_sup:init/1 line 30 in gen_server:init_it/6 line 330
22:00:18.859 [error] Supervisor {<0.330.0>,ranch_listener_sup} had child ranch_acceptors_sup started with ranch_acceptors_sup:start_link(http, 100, ranch_tcp, [{port,8001}]) at undefined exit with reason no match of right hand value {error,eaddrinuse} in ranch_acceptors_sup:init/1 line 30 in context start_error
22:00:18.860 [error] Supervisor ranch_sup had child {ranch_listener_sup,http} started with ranch_listener_sup:start_link(http, 100, ranch_tcp, [{port,8001}], cowboy_protocol, [{env,[{dispatch,[{'_',[],[{[<<"static">>,'...'],[],cowboy_static,{priv_dir,web,"static",[{...}]}},...]}]}]},...]) at <0.195.0> exit with reason {shutdown,{failed_to_start_child,ranch_acceptors_sup,{{badmatch,{error,eaddrinuse}},[{ranch_acceptors_sup,init,1,[{file,"src/ranch_acceptors_sup.erl"},{line,30}]},{supervisor,init,1,[{file,"supervisor.erl"},{line,243}]},{gen_server,init_it,6,[{file,"gen_server.erl"},{line,306}]},{proc_lib,init_p_do_apply,3,[{file,"proc_lib.erl"},{line,237}]}]}}} in context start_error
22:00:18.862 [error] gen_server <0.197.0> terminated with reason: killed
22:00:18.862 [error] CRASH REPORT Process <0.197.0> with 0 neighbours exited with reason: killed in gen_server:terminate/7 line 804

````


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


Driversiti =)


Also, I've checked the logic and you don't need to call $mdUtil.nextTick(checkVisibility) at any point. The new function is always called by angular, not by dom events.


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 48](http://teamcity.otenv.com/viewLog.html?buildId=1491449&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **FAILURE**
Summary: Tests passed: 34; exit code 1 (new) Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 47](http://teamcity.otenv.com/viewLog.html?buildId=1491448&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **FAILURE**
Summary: Tests passed: 34; exit code 1 Build time: 0:0:0


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


Why not allow secondary screenshots in the UI? I don't see a reason not to, tbh, maybe it would help the user see how he can do it.


If CellCollection has a definition field, why are you copying the definition from the context here? Couldn't you simplify things by making a builder method that copies the sort and definition from an existing collection?


jap wird gemacht.
LibreCalc ist richtig, ich schiebe das auch mal mit rein.

Das mit LaTeX... überleg dir das nochmal, Sebastian...


Awesome. Thank you @davidearlotto. Apologies for the delay!


Ok, I maybe shouldn't have accepted the patch as-is, but I frankly didn't see immediately how to propagate exceptions (or at least error messages) from user-defined SQL-functions to the calling code in any other way.  This was admittedly a not quite unimportant shortcoming of the bindings.

Since apparently some people were not happy with the API-change, I've thought about it some more and found a somewhat sneaky solution using thread-specific data.  I may be wrong, but it seems there really is no way using the SQLite3 API to propagate this kind of information.

If your user-defined SQL-function raises an exception, it will now propagate all the way through the exec-call.  The change should be thread-safe and only required additional linking with the pthread-library, which will quite surely already be linked in anyway.

Could you guys please check out the branch at: https://github.com/mmottl/sqlite3-ocaml/tree/user-exception-handling

If you are happy, I'll make an immediate new release before other people start fixing their code for the now obsolete v3.


Specify that this should be a global include, for sites that do not necessarily have **scripts.php**.


@Thrandis A warning will be triggered with ```stack(tensors=tensors, axis=axis)```.

You should also verify that ```stack(tensors, axis=axis)``` does not trigger a warning message.


See how dams and river locks are built.


Well, suppose that the auxiliary formula we supply occurs on both sides, how do we know on which side to perform the substitiution?


To get TomLanguage you can use TomlLanguage.INSTANCE


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

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


It seems strange to use the builder here, inside a method of the class itself, and when you have all the information already available and don't need to incrementally call builder methods.


Specify that we're actually removing this from the existing Require config too


I can do that.


fast-forwarding master to auto = e4950d05


LGTM.


what pub mean? public? I don't like that. Do you have another standard way for internal function and rename the internal function instead? Also, why make an internal version do skip the assert?


Thank you for the quick revert @dcramer


I've not been able to test it yet unfortunately. I can't alter the source
on our cluster so I'm trying to get a local copy installed on my profile
that I can tinker with.

The bit I wasn't too sure about was writing the tests for checking for the
read depth flag if the window flag has been set.

Of course I've decided to start doing this the day I go on holiday, but
over the next month I'll get it tested and submit a pull request once I'm
happy with it.
On Wed, 2 Sep 2015 at 16:24, Colby Chiang <notifications@github.com> wrote:

> thanks for doing this. does it work? I think it might need to be "w:" with
> the colon in line 1194
> while getopts ":hB:S:D:R:o:m:r:x:T:t:Adw:gPkvK:" OPTION
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/s-boardman/speedseq/commit/afb16f3b261a45f34eec2cb06b01ce74ecf4e21c#commitcomment-13026154>
> .
>


Hah, good call, I might have to genericize these strings into Language at some point so I can give 'em "the treatment" without needing to recompile the script all the time.


http://downloads.natron.fr/~olear/Glow.ofx.bundle-0.1-Linux-x86-release-64-OpenCL.tgz

Should work, untested (don't have OpenCL on my laptop).


Thank you @szaghi


I squashed the 4 commits on master into this one, hopefully it goes smoothly...


I especially dislike the current interface because I wonder whether it's possible or intentional that the new cell collection is using a different definition than the one it's copied from.


This update is TDLS specific. The CAPI spec has 'type' as one of the mandatory parameters for dev_send_frame. So, this update is OK

...
...
else if (strcasecmp(str, "TDLS") == 0)
            {
                tdlsFrame_t *tdls = &sf->frameType.tdls;

                sf->program= PROG_TYPE_TDLS;
                for(;;)
                {
                    str = strtok_r(NULL, ",", &pcmdStr);
                    if(str == NULL || str[0] == '\0')
                        break;

                    if (strcasecmp(str, "type") == 0)
                    {
                        str = strtok_r(NULL, ",", &pcmdStr);
                        if (strcasecmp(str, "discovery") == 0)
                        {
                            tdls->eFrameName= TDLS_TYPE_DISCOVERY;
                        }
...
...


Ok.  pi-timolo has been updated.  See Readme.md for details.  Note there is
an upgrade feature as well per Readme.md

Let me know if you have any problems.

Regards
Claude ...

On Fri, Aug 28, 2015 at 5:25 PM, Emmanuel Odeke <notifications@github.com>
wrote:

> Awesome, glad to know it is all working! Thank you too and cheers. I'll
> check out and setup pi-timolo sometime soon :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pageauc/pi-timolo/commit/8c18ec7be66d578bb273ecc24a04b6d3b9ca3ea9#commitcomment-12956812>
> .
>

--
See my YouTube Channel at http://www.youtube.com/user/pageaucp


@chrisdone I think this may be causing the `32-unlisted-module` integration test to fail.


It only dawned on me when I played a map and saw an Invulnerability next to a corrupted vending machine and realized the potential for abuse. And I agree. We should use LANGUAGE as much as possible for output strings.


This reverted the temporary hack to get the daily digest to send out the last 19 days (since it stopped working).


Doh! Yet another magic "number". Could we introduce a constant and give it a helpful name? Please.


Did you just forget to remove the above wild card import?


убирай callback - function(Blueprint $table) { }
Schema::drop('users'); - и все.


Fair enough, I'll try to fix this tomorrow.


what?! that's awesome I need to go see what's new in 4.0. thanks!


nice, I haven't had a chance to look into


but good work


TeamCity Muon :: Muon Java Test [Build 50](http://teamcity.simplicityitself.com/viewLog.html?buildId=1722&buildTypeId=MuonCoreJava) is now running


TeamCity Muon :: Muon Java Test [Build 50](http://teamcity.simplicityitself.com/viewLog.html?buildId=1722&buildTypeId=MuonCoreJava) outcome was **FAILURE**
Summary: Tests failed: 6, passed: 0 Build time: 00:00:09
### Failed tests
```
com.simplicityitself.services.ServiceCompositionSpec.can compose multiple services with failover: <no details avaliable>

com.simplicityitself.services.MuonShutdownSpec.Muon can be cleanly shut down and replaced in a single process: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple concurrent muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

io.muoncore.stream.MultiDistinctSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

```


This is a ```static``` method.


@wwmayer many applications have that feature. It allows you to save a snapshot of your current work without touching the original file. Differently than "save as", if you press "save" later, you will save your original file, not the copy.

Later on we can use this in an autosave function to save a temp file at intervals, without risking to screw the original file, but it can have other uses too.


Actually this was already addressed in another commit of the same pull request:
https://github.com/ably/ably-python/pull/30/files#diff-7dfe7699975d14d16449f85005ab532fR48


We can, but you always get the stupid warning. That's just annoying. And we don't rely or assume Cygwin is Win32. Thus, just quiet it.


i cant compare glow-opencl and glow-ofx.. because maybe same name node..you can rename:
glowOFX and glowOFX-openCL for testng..?


Yes, the issue seems fixed. Thanks to the unification you can omit the `.pkgs` part. I still makes a difference if you have `-iA nixos.foo` or `-iA nixpkgs.foo` (if one isn't lucky; or whatever else you name your channels).


This doesn't seem to be using the cell bag attribute that you restored in the revert. Just going by assoc runs some risk of confusion with user-defined bags that didn't come from the configuration, and skip cells because `size == 1` fails.


@tdhock nice! Maybe we should remove the `if (Sys.getenv("TRAVIS") == "true") {` bit now?


Wow, that was quick, I didn't even pull request yet :D

I agree in general that it's good to not just use headers in the root directory. In practice, for the case of SDL, one would hope that they got installed to the specialized subdirectory, then the build system takes care of finding them and adding that version-specific include path. I totally agree it's wise to use a prefix on include dirs - notice I didn't adjust the SDL2pp internal includes (though I know some who would say those should be double-quotes with relative paths, to be sure you're including the peer header - this advice I am undecided on)

Initially I made this change because the upstream-built MSVC binaries of SDL don't have an `SDL2` directory between `/include` and the actual headers. (A practice which I agree is unwise) Thus, when I built [my SDL2 find module](https://github.com/rpavlik/cmake-modules/blob/master/FindSDL2.cmake), I sought to successfully find SDL2 from the distributed binaries.

However, I hit a lot of bumps in getting libSDL2pp to build on even MSVC 2013, more than I expected (not your fault, MSVC's - darn that incomplete C++11 support!), and I didn't offhand have a build of SDL for MSVC2015, so I switched to MinGW/MSYS2 - haven't looked to see what their include setup is.

Thanks for looking at this! I'm happy to know that this library is active, even if I can't manage to use it yet - it felt like such a waste/duplicated effort writing my own little RAII wrappers for SDL.


I agree, this is a problem similar to the one with the ```TermContext``` field. Unfortunately, I don't we have time to solve it properly. For now, I will just do what you suggested.


@btlachance has already implemented a user-exposed syntax for this actually, he just hasn't pushed the PR yet.


Do you want to keep this feature of selecting the node automatically in this PR?


@sledgehammer999 Now it's plain text in bold style. I have not forgotten anything, but if you want to do it any other way I can remove the commit.


No that's good. I just didn't see the bold statement before.


Didn't know that, thanks for the tips


Oh, you're trying fish?


Thanks for remembering to revert this!

_______________________
*Edward West*
mobile 415.225.7335
Facebook <http://www.facebook.com/edwardjwest>  Twitter
<http://www.twitter.com/edwardwest>  LinkedIn
<http://www.linkedin.com/in/edwardwest>

On Wed, Sep 2, 2015 at 12:13 PM, Ray Marceau <notifications@github.com>
wrote:

> This reverted the temporary hack to get the daily digest to send out the
> last 19 days (since it stopped working).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Hylozoic/hylo-node/commit/f98832549e1f37ce37126299d04de9f37acd389b#commitcomment-13031448>
> .
>


TeamCity LoomSDK :: Populate Loom Version And Notify GitHub [Build 4103](http://52.88.6.210:8111/viewLog.html?buildId=711&buildTypeId=LoomSDK_PopulateLoomVersionAndNotifyGitHub) is now running


Good point. Let me remove this line, and push.  This PR will be updated.


sorry if i missed it but I don't see any db installation steps (the hackademic db i mean) perhaps you can add code in bootstrap to do that with a default admin username/password used for testing only?


oh wow! thanks daniel for this work! is there no way to automatically register them? this seems to be suboptimal ;-)


This should not be a problem if ```cellSorts``` is properly computed (that is, only contain cell sorts). As far as ```size == 1```, I thought is true for all cells with ```multiplicity=*```. There is similar code in this class.


TeamCity Muon :: Muon Java Test [Build 51](http://teamcity.simplicityitself.com/viewLog.html?buildId=1723&buildTypeId=MuonCoreJava) is now running


Is there a reason why the inheritance part was removed? What about if MinAccess is a custom group?

Also, is the fallback part above now obsolete?


TeamCity Muon :: Muon Java Test [Build 51](http://teamcity.simplicityitself.com/viewLog.html?buildId=1723&buildTypeId=MuonCoreJava) outcome was **FAILURE**
Summary: Tests failed: 6, passed: 0 Build time: 00:00:09
### Failed tests
```
com.simplicityitself.services.ServiceCompositionSpec.can compose multiple services with failover: <no details avaliable>

com.simplicityitself.services.MuonShutdownSpec.Muon can be cleanly shut down and replaced in a single process: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple concurrent muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

io.muoncore.stream.MultiDistinctSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

```


TeamCity Muon :: Muon Java Test [Build 52](http://teamcity.simplicityitself.com/viewLog.html?buildId=1724&buildTypeId=MuonCoreJava) is now running


Since the `logged_in` property shouldn't change often, it would be nice to avoid this HTTP request overhead every time the route changes. We might be able to bootstrap the user model with data on page load.


TeamCity Muon :: Muon Java Test [Build 52](http://teamcity.simplicityitself.com/viewLog.html?buildId=1724&buildTypeId=MuonCoreJava) outcome was **FAILURE**
Summary: Tests failed: 6, passed: 0 Build time: 00:00:09
### Failed tests
```
com.simplicityitself.services.ServiceCompositionSpec.can compose multiple services with failover: <no details avaliable>

com.simplicityitself.services.MuonShutdownSpec.Muon can be cleanly shut down and replaced in a single process: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple concurrent muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

io.muoncore.stream.MultiDistinctSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

```


TeamCity Muon :: Muon Java Test [Build 53](http://teamcity.simplicityitself.com/viewLog.html?buildId=1725&buildTypeId=MuonCoreJava) is now running


TeamCity Muon :: Muon Java Test [Build 53](http://teamcity.simplicityitself.com/viewLog.html?buildId=1725&buildTypeId=MuonCoreJava) outcome was **FAILURE**
Summary: Tests failed: 6, passed: 0 Build time: 00:00:12
### Failed tests
```
com.simplicityitself.services.ServiceCompositionSpec.can compose multiple services with failover: <no details avaliable>

com.simplicityitself.services.MuonShutdownSpec.Muon can be cleanly shut down and replaced in a single process: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple concurrent muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

io.muoncore.stream.MultiDistinctSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

```


Can these be named arguments?


tabulate(n)(f) returns an n-dimensional array over n using f. Is it assumed that tabulate is a call to range?
@ktakagaki


roll version number


Changed to the "official" jQuery plugin like @dregad encouraged here https://github.com/fg-ok/EasyEnter/commit/2dc7f7af5c1346bea458caab9272488ff90e16de


@ibauersachs, I'm asking for a confirmation here. We used to have a libjitsi bundle which included the libjitsi library and added a manifest and a BundleActivator. Consequently, we have two libjitsi Maven artifacts: libjitsi for the libjitsi library and jitsi-libjitsi for the libjitsi bundle. Now there's the libjitsi library only which is an OSGI bundle and the old "libjitsi bundle" is no more. Then I should remove the jitsi-libjitsi artifact for good, right (and change the dependencies on jitsi-libjitsi to libjitsi, of course)?


I like this version, not too verbose like Java, and also not too obsessed with "every keystroke is precious" like the Scala examples I have seen lately (resulting in indecipherable code)


Merge #7 Fix html structure on generating breadcrumb for multi level category leaf


TeamCity Muon :: Muon Java Test [Build 54](http://teamcity.simplicityitself.com/viewLog.html?buildId=1726&buildTypeId=MuonCoreJava) is now running


TeamCity Muon :: Muon Java Test [Build 54](http://teamcity.simplicityitself.com/viewLog.html?buildId=1726&buildTypeId=MuonCoreJava) outcome was **FAILURE**
Summary: Tests failed: 6, passed: 0 Build time: 00:00:09
### Failed tests
```
com.simplicityitself.services.ServiceCompositionSpec.can compose multiple services with failover: <no details avaliable>

com.simplicityitself.services.MuonShutdownSpec.Muon can be cleanly shut down and replaced in a single process: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple concurrent muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

io.muoncore.stream.MultiDistinctSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

```


TeamCity Muon :: Muon Java Test [Build 55](http://teamcity.simplicityitself.com/viewLog.html?buildId=1727&buildTypeId=MuonCoreJava) is now running


TeamCity Muon :: Muon Java Test [Build 55](http://teamcity.simplicityitself.com/viewLog.html?buildId=1727&buildTypeId=MuonCoreJava) outcome was **FAILURE**
Summary: Tests failed: 6, passed: 0 Build time: 00:00:09
### Failed tests
```
com.simplicityitself.services.ServiceCompositionSpec.can compose multiple services with failover: <no details avaliable>

com.simplicityitself.services.MuonShutdownSpec.Muon can be cleanly shut down and replaced in a single process: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple concurrent muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

io.muoncore.stream.MultiDistinctSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

```


Create a function that does all of these steps...where you pass it the word `set` or `clear`

If that doesn't make sense I definitely can try to explain better


The compiler sets __EXCEPTIONS if it compiles with exceptions. If it does, some might still fail, because e.g. some other internal stuff fails, typically that's clang on some windows-targeted distributions. However, those also would set __EXCEPTIONS. Hence, the second check determines whether the build actually is possible with exceptions. It deactivates them, it that fails. That should ensure a user not to blame Hana but the compiler for not compiling. Handling within the hana/includes is completely independend.


This line, not sure if the original line is in error, or is intended.


nil, nada, zilch, no bueno


cellBagProductions seemsx to be computed just by checking the assoc attribute, rather than actually looking at the cell bag attribute.


1579840-GroundTemps (mmatts) - i386-Windows-7-VisualStudio-12: OK (1801 of 1824 tests passed)

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


Very artificial example, but is it common to treat all compound statements in Scala as expressions resulting in a return value, and not a statement with side-effects like the following example?

// Solution-4.scala
// Solution to Exercise 4 in "Compound Expressions"

val distance = 9
val activity = "biking"

val willDo = {
  if(activity == "running") {
    distance < 10
  } else if(activity == "biking") {
    distance < 20
  } else if(activity == "walking") {
    distance < 6
  } else {
      false
  }
}

println(activity + ": " + willDo)

/* OUTPUT_SHOULD_BE
biking: true
*/


Note how they align the '!=' here.


TeamCity Muon :: Muon Java Test [Build 56](http://teamcity.simplicityitself.com/viewLog.html?buildId=1728&buildTypeId=MuonCoreJava) is now running


I was trying to push the warnings in #9456 exactly because I know just moving definitions somewhere won't make people stop using them. They would then be painful to delete. IMO it's better to give warnings for half a year, for example, and with the new release then remove the old aliases (this approach seems common anyway).


TeamCity Muon :: Muon Java Test [Build 56](http://teamcity.simplicityitself.com/viewLog.html?buildId=1728&buildTypeId=MuonCoreJava) outcome was **FAILURE**
Summary: Tests failed: 6, passed: 0 Build time: 00:00:09
### Failed tests
```
com.simplicityitself.services.ServiceCompositionSpec.can compose multiple services with failover: <no details avaliable>

com.simplicityitself.services.MuonShutdownSpec.Muon can be cleanly shut down and replaced in a single process: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeSpec.Multiple concurrent muon based subscribers can open the same channel: <no details avaliable>

io.muoncore.stream.MultiSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

io.muoncore.stream.MultiDistinctSubscribeLoadedSpec.A loaded channel remains available.: <no details avaliable>

```


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2257 of 2332 tests passed)

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


Because IsAdmin and IsSuperAdmin are more reliable
MinAccess is not allowed to be a custom group
The fallback part is not obsolete


Uh, would've been easier to keep this branch tracking master exactly


In fact, the 2.4.0-rc1 branch does not have the new Map functionality yet. You may use one of the very recent builds: http://sourceforge.net/projects/casadi/files/CasADi/branches/develop-branch/


<img width="1280" alt="screen shot 2015-09-02 at 10 10 46 am" src="https://cloud.githubusercontent.com/assets/14082101/9641942/48e2996c-516e-11e5-89e6-2530d3277150.png">


2.4.0-rc2 should be out next week..


Do we need to set this?


Kali, your suggested updates are a good improvement. Can you please submit a pull request for the updates involving usage of strncpy() and sizeof() function? Thanks


Ah, missed the `if not priv` part.


Will probably want to remove it from the app-layer as well?

The setup scripts currently uses these placeholders to reliably create a new entry without having to be yaml aware.  Its too bad be we can't use Python - emitting the YAML would reformat the whole file.


<img width="1280" alt="screen shot 2015-09-02 at 10 59 55 am" src="https://cloud.githubusercontent.com/assets/14082101/9641985/81fcc344-516e-11e5-85bb-6210a8bf49ab.png">


seems ok ^^


<img width="1280" alt="screen shot 2015-09-02 at 11 15 59 am" src="https://cloud.githubusercontent.com/assets/14082101/9641997/915a74da-516e-11e5-94d6-1e1323079e1e.png">


EDAPT_Issues2 (macumber) - Win64-Windows-7-VisualStudio-12: OK (2308 of 2343 tests passed)

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


<img width="1280" alt="screen shot 2015-09-02 at 11 49 52 am" src="https://cloud.githubusercontent.com/assets/14082101/9642005/9dd2c302-516e-11e5-8dfb-415e4f41bb3c.png">


Maybe it would be good to note that scripting is only in pcbnew, for people that wonders if it exists in the other apps. (Could easily be done in the intro of this chapter, one sentence)


@lyubomir Yes, please remove jitsi-libjitsi. Sorry, another part that I missed. Where is jitsi-libjitsi used currently? There might be some adaptions needed.


Looks Good!


Not important yet, will add more features and compare against the pyplug (link?)


OpSchemeFixup (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

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


OpSchemeFixup (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


As you see, your approach works in practice ^_^. It's a bad thing it spams in `nix-env` though -- but your latest idea[1] seems to handle this problem.

[1]: https://github.com/NixOS/nixpkgs/commit/8e8e23de337d3140eb2e17694696b8e000d2e0d6#commitcomment-13013473


Yes!!!


I try to keep the Grantlee implementation as close to the django implementation as possible, and this diverges. The parse_until in the Parser is just a list https://github.com/django/django/blob/master/django/template/base.py#L454, and it is used as `if command in parse_until`, so it does not create regular expressions from the strings in the list. We can also see that the tag implementation just passes a list of strings: https://github.com/django/django/blob/master/django/template/defaulttags.py#L1028

Is the divergence really needed? Can we implement this the same way django did?


Thanks.


Mainly because I want to return a Hash and not a class with methods.
If you prefer I can refactor to
```ruby
ref = OpenSCAP::Xccdf::Reference.new(OpenSCAP.oscap_reference_iterator_next refs_it)
refs << log
```
Which would represent the references


Hi, how could I help you with this?


@ibauersachs, it's totally fine. It's a dependency of jicofo and jitsi-videobridge. Thank you very much for the confirmation here and the help with Maven in general!


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

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

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

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

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

```


Yip. Giving it another try since ~ half a year. So far: cool beans.

*But*: Some things you are used to in Bash or Zsh just don't work the same way. Setting environment variables, chaining commands, variable interpolation, the lot.

And I don't want to lose that muscle memory. :sob:

On 2 September 2015 21:21:33 CEST, Lucas Dohmen <notifications@github.com> wrote:
>Oh, you're trying fish?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/bascht/dotfiles-public/commit/bee3561eadcf080017b7cfcea508f8f98453ea68#commitcomment-13031571


91958608DOASEffectOnZoneSizing (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


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


/cc @adamyonk


I agree with the fact that lambdas haven't been introduced at this point. This new code won't run though, throwing a `NameError: name 'tree' is not defined`.
I was actually a bit confused about this myself. Since its a lambda wich doesn't take arguments but uses the 'tree' variable name inside of it, not sure what it is supposed to do.
It probably should either be the simple version without the lambda:

    some_dict = collections.defaultdict(list)

Or if the point of the lambda was to specify the default_factory it would be something like this:

    tree = lambda df: collections.defaultdict(df)
    some_dict = tree(list)


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


TeamCity LoomSDK :: Deploy to Site [Build 3](http://52.88.6.210:8111/viewLog.html?buildId=722&buildTypeId=LoomSDK_DeployToSite) outcome was **FAILURE**
Summary: Snapshot dependencies failed to start: 2, failed: 2 Build time: 00:00:00


I am sorry for the problem.

2015-09-02 18:40 GMT+02:00 mfonville <notifications@github.com>:

> @NicholasBuse <https://github.com/NicholasBuse> thank you :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opengapps/arm/commit/10b809a96936389c4cafea5644bd4dba7daf8186#commitcomment-13028160>
> .
>


TeamCity LoomSDK :: Populate Loom Version And Notify GitHub [Build 4104](http://52.88.6.210:8111/viewLog.html?buildId=723&buildTypeId=LoomSDK_PopulateLoomVersionAndNotifyGitHub) is now running


Can't do this because remoteStdin.Close's signature is `func() error`, not `func()`


This checkout would be better as a boolean "useForBuild" that gets passed down into this method, not making the code aware what the command is.


I don't think you should be using an h2 for this


Right, I'll need to take another look into the issue, since I didn't base the implementation on what django did I ended up with something that worked but surely was completely different, I'll try to take a look at the python code and try to make something similar...


I've removed the `cp` function and replaced it with pure `io.Copy`, and added `util.HandleError(err)` as well. Are you ok deferring the channel change until later i.e. not as part of this PR?


yes - but in different processes.  The `command_cb` subscription is in `__init__`, so happens before the fork;  the `barrier_cb` is in `initialize()`, which is the first thing called after the fork.


Nie testowałem tego, bo nie miałem czasu, ani możliwości. Wobec tego pinguję do @efiku, @da-an, @marcinlacina :D


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

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


I can cherry-pick them again and force push


This should fix the issue: https://github.com/opengapps/opengapps/commit/0142095ef8ecf53656d192deaea13df6be12868c

We should be using the newest zipalign from recent Android SDKs and not the Ubuntu's repository one.


@Foggalong Slack icon works good. Gitter icon do not display.


Some early comments:
You really don't need the Sequence class. Keep requests in a list, convert to json when necessary
Pls provide a default ctor, add, insert methods, with varargs so we can add multiple requests
Instead of getOperationPathParam, use instanceof to look at the request type


ok, fixed in https://github.com/SparkDevNetwork/Rock/commit/d0aacf6efc0a76d16f3a691feed4ad6ad112b601


As is, this comment is redundant. You may want to say something like "perform additional checking" instead.


Nice! I've really gotta get familiar with all the UIOP goodies. :)


What is the use of this for loop?


Still not one atomic call...  When I say atomic, I mean we dispatch exactly one call to etcd to write the object.  Which means you need a method on the REST that allows you to save both spec and status at the same time.  That method would only be used from this code path, but it allows you to bypass the normal protections on spec and status (when called within the same codebase).


Just ``git merge master``. There's nothing there at this point that shouldn't go into release-0.10.


Jetzt bekomme ich:
```
TASK: [fastd | Set blacklist.sh runable] **************************************
failed: [vpn06.bremen.freifunk.net] => {"failed": true, "path": "/etc/fastd/ffhb/blacklist.sh", "state": "absent"}
msg: file (/etc/fastd/ffhb/blacklist.sh) is absent, cannot continue
```


Not really sure; I haven't really used Python. :) Just happened to be reading up on it, read this tutorial, and thought that example didn't need the lambda.


It bothers me that `let` typechecking needs to be aware of signatures like that. Would it be possible to do this work as signature expressions get typechecked?


BTW: the `command` pubsub is somewhat of a catchall, really.  Which is illustrated by the fact that I can  think of no callback on the command channel which would be able to interpret all incoming commands.  Since volume on that channel is relatively low, I think that is at this point ok, but _nice_ it is not.  I though about removing those noisy 'unknown command' logs, as that really is expected behavior at this point...


Will this check work correctly on 32 bits systems? time_t is not always guaranteed to be larger than int32.


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

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


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

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

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

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

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

```


I don't understand this reasoning.


Always stderr, but yes.  I can remove that, but it is usually easy enough to redirect stderr to /dev/null if you are running from a script, and at the console you probably want to see that anyway.  It also makes sure that log messages generated outside of the labrad namespace will still get logged to the console.


Can signatures refer to aliases? Does this affect ordering?


Is this really supposed to have the same status message as the one above?


Explain the *general* use of this argument (i.e., perform additional, context-dependent checking in the context of the `letrec` body), not just the particular use that you have found so far.


As above, I don't like the coupling between `letrec` and signatures.


A neater way to write this:

```
DataMessage message;
while ((message = mQueue.poll()) != null) {
    ....
```

Or:

```
for (DataMessage message = mQueue.poll(); message != null; message = mQueue.poll()) {
    ....
```


This was a refactor of 49 (not 50)


could call this "create_runreport_response"


Reading your statement I take it you don't expect the following then:
```
> grep "unknown command" *|cut -f 9 -d:|sort|uniq -c
      3  unknown command in barrier cb
     20  unknown command in command cb
```


RoomAirflowNetwork (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (760 of 760 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-9.47%25-green.svg)


Suggest improving the rationale here. The provided implementation suffers from this exact same issue (creating lots of objects). It doesn't recycle, either, which might mean that it's churning through more memory (depending on how spiffy the Handler usage is).


Handle in what way?


RoomAirflowNetwork (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Yes, I've been using Goodreads to find and "track" books I've wanted to read for awhile now. It's just so unappealing, which is why I'm glad I found your reading list!

I'm not sure why Goodreads felt the need to rearrange the books this time. The only change that should have occurred is _Station Eleven_ being promoted to `currently-reading`. The John Scalzi books seem to be in a weird order that I can't figure out, but the rest appear to be sorted correctly. How odd.


And we could call this "create_getresults_response",  and pull it out into a helper, since it's similar in both this and the following test.


Ooo, very nice.


Still keep this in an else? We know it won't do anything if the active transport is not shut down.


Bakit mo po minerge ung branch mo sa master? :))


but how do you set caret template in side the html template


I only tested on ubuntu, I don't have a mac.  I have changed this to allow the socket to be specified on the command line, and also to fail on an unknown platform rather than try UDP on localhost because there is no way to detect an error.


Is it necessary to special-case those too? If so, at least have an `~or` pattern for all of them.


Great, thanks!


ok
ahoj prosim posli mi tvoj fastshare.host
ja som ten moj kompilovat v linuxovom busy boxe a mozno tam je chyba .
Posli mi tvoj subor: fastshare.host a ja ho vyskusam.
popripade skus ty ten moj posielam ho v prilohe.

> ** Wed, 02 Sep 2015 02:23:19 -0700 - "feixm1/fastshare.host" <reply@reply.github.com> **

>
> Tak vytvoreny host funguje, po vzoleni do DS a vyplneni prihlasovacich udajov mi v pohode stahuje...
>
> —
Reply to this email directly or [view it on GitHub](https://github.com/feixm1/fastshare.host/commit/a2f38f103bf5d0e2c84253c471c3b32bd86a5c2a#commitcomment-13018411).
>
>


In which cases is this code necessary, then?


fix_vscoils_coilsystem (mbadams5) - x86_64-MacOS-10.9-clang: Tests Failed

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


The only thing I can think of is after Goodreads sorts by author, it falls back to sorting by date added.

[Screenshot](http://seano.link/cp1l)


Actually, if one of you could make the CONTENT=0 instead of 10 that would be great thanks


Please remove.


@gpolitis java.util.Objects is Java 7. Please either raise the compilation level of all Jitsi projects to 1.7 (preferred) or drop that stuff.


Commitメッセージの書き方間違えた・・・・・・


@lyubomir They both don't use a proper OSGi implementation, right? I'm therefore not quite sure how it will deal with the FMJ fragmet. Basically a fragment (FMJ) shares the classloader of the host (libjitsi). But I'm assuming the whole custom "OSGi" implementation doesn't use classloaders anyway, so this probably isn't a problem.


Leaving this remark on this commit b/c... I don't know where else...

This is awesome! I always end up saying something like, "In X situation, you would want to use a BlockingQueue instead." Thanks for making an example to show this off.


Does `t->s` make sense here? I mean, `init-depend`s are not types, are they?


I agree it's good to have messages go to the console as well; just wanted to make sure that was the case.


John, this is very close to an existing class we have, ICommand/Command. ICommand has:

	public interface ICommand
	{
		void Execute();
		bool Enabled { get; set; }
		event EventHandler EnabledChanged;
	}

Do we need to introduce this new class, or can we instead keep with that and perhaps expand it?
E.g. we could expand ICommand to include the label (in which case we should specify whether it's in English, or a string id, or vernacular).


Would it make sense to wrap the printed representation in something that made it clear that this is a signature?


What about `+"12.34e5"`?


Thanks @bserdar - I had toyed with not using the Sequence class, so I'll just rip it out.

Is the purpose of the insert method to insert a request before another request that may have already been added to the collection?  Is there a logical way to order requests or should it be left 100% up to the user?


This is *not* sufficient testing.


But then you will get ACLs through the Security component, won't you?


@gashcrumb FYI  removed my marvelous logo change and did some subtle reorg


hex(None)とすると、
TypeError: hex() argument can't be converted to hex
となるため、def __str__(self):では、
version,msg_type,xidにNoneが入った時に
エクセプションを上げないような処理が組み込まれているようです。
https://github.com/osrg/ryu/commit/afccf3954b211089dbb8265342a0b470a1e3053f
本処理が必要か？いれるとしたらどういれる？の検討が必要そうです。
＃__str__で入れて、エラーDebugで入れない理由はなさそうなので、
＃「def __str__(self):側の処理をコピペ」で良い気もしますが。。


Update copyright dates from "2008, 2009, 2010" to "2008-2015"


to keep it consistent with the rest of the file, remove newline between """ and the text. Start with capital letter, and align description for params like we have it for other functions.


should be 2015,  not 2008, 2009, 2010


switched the logic to assume success defining the class. this takes out the race and the synchronized block


i'd group it into system and lsst, so add empty line after "unittest", remove empty line after "utilsTests


actually move that after dafPersist to keep it alphabetically


done


ok


The invite system currently deletes accepted invitations, so this actually isn't possible for us atm.  The best we could do would be to check whether the event has created orders and if so to assume that some invitations have been sent – but even test orders would trigger that condition.

I suppose we could change the invitation behavior to keep them around longer, but that would definitely be outside the scope of this ticket. :-)

Also, because accepting an invitation to an event always creates an Order tied to a user account, and because orders are unique per user/event, we don't need to worry about the invite getting lost. If they haven't accepted it, the organizer can resend; if they have, the organizer can just send them a link.


@blazek @wonder-sk I've compiled and tested this fix, which I confirm fixed both the thread warnings as well as the segfault crasher (on my machine).

Kudos for getting rid of an obscure crasher.


了解了。


hi @eapache, i'm moving in a slow pace. it would be nice if you can help me, i'm was a little stuck on the AST traversal, but i started a draft, take a look: https://github.com/guilherme/starscope/commit/4cc92efe1b078e6fc43ad3816c7b7aec429e4b26


de acuerdo


wow, good catch, scary that we almost released without this


Good catch, d213937 fixes this


What is my reasoning? I'm looking for nodes that define functions, and try to make the extract* walk through them and do throws with the definitions.


Nice add of develop word


Thank you for the attention :)


本ルートはSWから受信したメッセージですので、
「version, msg_type, xid が None になる」
可能性は低いと思いましたので、特に対応はいたしませんでした。

共有いただいたコミットの箇所は、Ryu で作成途中（シリアライズ前など）のメッセージを
文字列化する可能性があるルートですので、対処が必要だったんだと思います。


And I've cut a new gem (v1.3.4) to release the fix.


Can you post some sample code of your approach? Just by instantiating the SerialPort drains my cpu:

    new serialport.SerialPort('/dev/cu.wchusbserial1420');


The culprit for #729 ... argh!  Avoid v4.1.4... purported fixed in 4.1.5


Branch message too!


ok fixed all except for adding a test for bean-style. coming up next.


なるほど。たしかにOFメッセージインスタンスつくりたての場合にSTR化したら
エラーになってしまいますもんね。了解しました。対応は不要だと思います。


we don't need this anymore now that we've disabled it inside the .eslintrc file


86139842-AddAutoSizing (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (759 of 759 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-9.37%25-green.svg)


This is for authorization that you mentioned earlier. I decided to add this instead of using policy, going with the theme of building from scratch. The point is making sure the ```user_id``` in performing the  ```show``` is same as ```current_user```


rax retry tests


94006714-MultipleTowers (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1812 of 1812 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)


actually in debug mode, most compilers init memory with zeroes. In -O2, they don't. That would explain a lot in this case.


why not keep the same style, one use array, the other use pointer?
please change to in[xx]=out[xx];


> > 3. `bs_capt` field is not copied.
>
> It should be copied with change bcc3070737763d2034476dcc8b3286993c630973.

And it should *not* be copied by `pcap_stats()` with change e9f0f1c80644924057b2203acb8429085b23857d, as it wasn't copied in any version of WinPcap, as, I guess, older versions of WinPcap didn't have a `ps_capt` member of `struct pcap_stat`, so if it were filled in by newer versions, programs compiled against older versions would pass in a shorter structure and WinPcap would overwrite whatever came after the structure.

If somebody wants the captured packet count, they need to use `pcap_stats_ex()`.


is it possible to have word wrap?


> > `if (!PacketGetStats(p->adapter, &bstats) {`
>  ')' is missed

Fixed in e9f0f1c80644924057b2203acb8429085b23857d.


is this supposed to be empty?


there is nothing defined in here. Do we need this method or does it work without it?


how would you feel about moving the header tag into `layouts/_header.html.erb`?


how would you feel about moving the footer tag into `layouts/_footer.html.erb`?


86139842-AddAutoSizing (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1784 of 1815 tests passed)

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


The indentation in this file seems off. How would you feel about making it 2 space indented?


`r` isn't very descriptive. How would you feel about changing this to `result`?


101884090--NewVenting-Availability-Schedule (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (760 of 760 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-9.67%25-green.svg)


Won't this cause a memory leak since we never free the existing label? If we explicitly calloc and assign the label in `progressbar_new_with_format`, we could blithely free the existing label before this line and ensure that we don't leak existing labels.

Will need to add a corresponding free() to `progressbar_free`, also.

This change is a little more far-reaching than this -- can you patch it up with the memory fixes above? I like this fix, but can't merge it as-is.


have you tried to add a flag to disable tnr?
when we run basic pipeline, we need tnr disabled.


This is a very long line. How would you feel about wrapping it at 80 chars?


Good to see some code!


how would you feel about adding a newline after this?


> I do not understand why `pcap_stat` has field `bs_capt`. But not `ps_capt`.

Fixed in eec6bc9ea2d30d5b737cd4078a5879afc117c05d.

>  And why WinPcap `bpf_stat` has field `ps_ifdrop` but not `bs_ifdrop`.

Because they made a mistake naming the structure member.


Cool :D
On Sep 2, 2015 8:04 AM, "TJSteveMX" <notifications@github.com> wrote:

> [image: ic_radium_isotope]
> <https://cloud.githubusercontent.com/assets/6558902/9621821/542f6414-50e0-11e5-9366-fd127abf9fc8.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Team-Radium/packages_apps_Settings/commit/be80afd9f2670855d6f7ca80715261d751a0404f#commitcomment-13013897>
> .
>


lol


101884090--NewVenting-Availability-Schedule (lgu1234) - x86_64-MacOS-10.9-clang: OK (1808 of 1808 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)


@Dignifiedquire What's this?


I recommend placing a file header on every file. I believe the Station files already have this with our version of the Apache license. From what I recall reading last year, the license-on-each-file is not required as long as we have the LICENSE file, but it is strongly recommended for instance in case the file is taken away by itself.


This is incorrect. I want to _minimize_ the total distance between the centroid and the nodes in its cluster, not take the average of those distances.


assert is not good enough here since <property> interact directly with application. if in release version, assert will not invoked, it's better to return property errors.


I believe the controller is necessary. I received an error for missing controller before I added this.


welp.


This has been filled in. The content was pushed later.


take latest


Are these not XRP only ?


Yes. I agree.


Consider changing to `#!/usr/bin/python` as suggested [here](http://unix.stackexchange.com/questions/29608/why-is-it-better-to-use-usr-bin-env-name-instead-of-path-to-name-as-my).


Yes I agree.


I'll take a look and change this.


Yes I agree


Yes I agree. I couldn't remember the guideline length.


Oh... it was a mistake... :stuck_out_tongue_closed_eyes:


testing


[Code without event sending when block breaking] (https://github.com/CovertJaguar/Railcraft/blob/master/src/main/java/mods/railcraft/common/items/ItemCrowbar.java#L222-L262)
[Original Enchantment] (
https://github.com/CovertJaguar/Railcraft/blob/master/src/main/java/mods/railcraft/common/items/enchantment/EnchantmentDestruction.java)
It seems that another glitch exists. I'd test it on the server later.


I assume this is what comes out of the box. Do we have any plans to switch to something else, or do we want to continue with SQLite until there's a reason to change to something else?


Ah, they probably are, I just copied the test from Payment.


Não entendi...


make that /api/nfPitchers


enum definition. make words first alphabet capital. and it's better to append the attribute name at the end of each item.
e.g BasicbayerStage, TonemappingStage


If it's generated and we don't need it, why does it need to be committed?


This is @ivykoala LGTM


Is it possible to accept an OpaqueSpan here? This makes the callers simpler, and they don't have to know about the internals of OpaqueSpan and SpanData.


Are you sure this does what you think it does? What is the result here? What is assigned to `NAMESPACE['id']`?


there's a large code block of
    #if HAVE_LIBCL
    ...
    #endif
in switch cases. better to move capture to there to save a #if ... #end to make code clear.


Not sure why this `debugger;' is showing up here. I originally had forgotten to remove this, but removed it, amended the commit, and force pushed to my branch.  The file looks correct in the source on my fork:
https://github.com/Alex-Krautmann/ignitionjs/blob/master/src/ignition.js#L336

When you check out the branch locally the debugger statement isn't there either. Maybe it is something with github.


Ah ok, gotcha, makes sense now!


Why do we need to iterate against the object and return errors list? Why we don't just return the allErrorList array?


delete it


Hey Carter, thanks for adding me.


Should this section be in a dedicated INSTALL.md file?


Same as above


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1812 of 1813 tests passed)

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


@tridge For a moment I thought I was seeing a great path forward :-)... no, just the "for eclipse users"


@cylonicraider I believe this will be of interest!


Hey Organi5m, thanks for adding me.


less codes


Hey Cub, thanks for adding me.


Autodesk_Performance (DeadParrot) - x86_64-MacOS-10.9-clang: OK (1808 of 1808 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)


@timmc This fixes the firefox pane title issue you reported a few days ago.


Hey backjpack, thanks for adding me.


Haven't had a lot of time to look at this yet, but it seems questionable to introduce the apply themes at the content level, especially bundled as part of this larger checkin


Echoing my earlier concern about adding this to the abstraction, this method is doing less than nothing.


:metal:


@ibauersachs, right, jitsi-android-osgi does not implement custom/bundle classloaders so the OSGi specifics shouldn't pose any problems.


@lazedo why was this removed?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.951.af2943f650d907d395e6648586cdf57497153849](http://build.palaso.org/viewLog.html?buildId=62560&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 501, ignored: 6 Build time: 00:05:08


Back for the worst-case analysis (to my knowledge) by using Mega Lucario again. Assume the moves are randomized to be sampled in the following order: Aura Sphere, Close Combat, Flash Cannon, Swords Dance, Nasty Plot, Bullet Punch, Dark Pulse, Ice Punch, Crunch.

The first run as such is `["aurasphere", "closecombat", "flashcannon", "swordsdance"]`. As per the `setupType` rules, Aura Sphere is rejected since it's definitely a Physical setup. The moveset is now `["closecombat", "flashcannon", "swordsdance", "nastyplot"]`.

This is where things get nasty. The `setupType` is now randomized because there are an equal number of Physical and Special moves, but it does so on every run. This means that Special can be rolled on the first check, Physical on the second, and Special on the third, rejecting Close Combat, Flash Cannon, and Swords Dance, in that order.

Now the moveset is `["nastyplot", "bulletpunch", "darkpulse", "icepunch"]`. Nasty Plot will be rejected because there aren't enough Special moves, leaving this Mega Lucario with `["bulletpunch", "darkpulse", "icepunch", "crunch"]` as its set. While it still has a STAB move and no contradictory setup, it will have two Dark moves on the same set, which is definitely not a good idea.


Mostly, I think at least. I'm just waiting for some reaction from Eelco (or enough time to make it). Reversal ping-pong serves noone.


Is there a need to be able to resolve URIs at this point vs. when the instance is created?


This is not on a new line


Reference test:
(just trying to remove squares attacked by Minors in the Rook mobility)

http://tests.stockfishchess.org/tests/view/551610710ebc5902160ec4b7
http://tests.stockfishchess.org/tests/view/5344f1670ebc5942e74f57a8

The original result when the Queen mobility was changed in April 2014 looked quite strong at LTC
https://github.com/psqtab/Stockfish/pull/1


Inconsistent Indentation.Please, Fix!


> Organization


Done, although it doesn't currently do much -- we could add DEBUG messages within the node, but we have no way to get that information from the servers.  Right now, anything from stdout is treated as INFO, anything from stderr is WARN.


I think you need to change the name here.


I think this was a bad commit.  Kind of ends in the middle of line 57.


I realize there will always be edge cases, but I think that in case of a tie, one should be chosen over the other without randomizing.


It's not *necessary* but it's recommended to always have it. You're right that it only really matters if things are going to change but react will output a warning.


use coco oil


Give a space between `{{` and `url()` . It should be something like `{{ url('help') }}`


localStorage is per domain


reack


Yes. The instance might be in a shared static variable used by multiple pages. The "callback" URI is typically a relative URI to enable scenarios like testing on localhost, and where possible we want to consume the default value "/oauth2callback". So the methods called by the pages can pass in their own URI to resolve it relative to them.


This is being deleted from development.


Do we need toolkit.js at this point?. What does it do?


ack


you don't need to pass action and appcmd_argument, either will work `:set` is faster than `"set"`


@lolautruche @wizhippo This merge is breaking default siteaccess somehow.
When siteaccess is not specified in command, on a demo install, for config `var_dir` I get:
- With patch: `var/storage/`
- Withouth the patch: `var/ezdemo_site` *(the correct one)*

In both cases value is correct by adding ` --siteaccess=demo_site`.

This is one of the blocking issues for Solr work, as I can not specify siteaccess in the generic part of the installer code.

cc @emodric @bdunogier


```
v:ConCommand("play " .. GM.Config.lockdownsound .. "\n")
```


Initial commit


What does `chart.js` do in this app presently?


A Git Commit is typically feature oriented.

For example, if I were to have made this commit, I'd have called it: Adding Switch Statement
or Switch Statement on Name
or something that's even more related to why it's needed or something. (I understand that this is for just learning, so the 'Why' bit of this doesn't really come into play).


Is `=== true` required here? Doesn't `assert` check for boolean values by default?


for "compare against the pyplug (link?)" i'm not know because "red weight, blue weight, green weight"  not know what is needed..uhmm.. instead : brightness, gamma, saturation, is good but thare is plug modulate.ofx.opencl..maybe this is not needed.. but is possible good "color multiplier"..
a moment i will go to aftereffect check compare glow of AE and natron ok? a moment bit


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


Not sure the benefit of having this helper is over having `_renderLeftSide`, `_renderBoards` ... methods. It would make more sense if you were using it dynamically but all it's doing is adding a `switch` overhead. Though ignore this if you prefer this style.


See https://github.com/bdunogier/subber/blob/master/features/scripts/sickbeard_wrapper.feature, with the matching Context: https://github.com/bdunogier/subber/blob/master/features/bootstrap/ScriptContext.php.

It works just fine.


Updating gh-pages with latest draft


Indent properly.


Remove dead code.


saw approval from nickelization
at https://github.com/basho/riak_ensemble/commit/cfbb6119b82f7c9850dc3daefc3749c0e5706afc


merging basho/riak_ensemble/feature/subscribe-improvements = cfbb6119 into borshop-integration-84-feature/subscribe-improvements


basho/riak_ensemble/feature/subscribe-improvements = cfbb6119 merged ok, testing candidate = a62b0cc3


Consider creating a `getFormData` method and using the same design pattern as other forms. Maybe even move `setFields` into `ModelValidationMixin`?


Does this need to be silent?


Why the quotes? Be precise in your description.


This name is too generic.


@roger- Should this close #20 ?


Nit: whitespace around `==`


Maybe it's something implemented bit never used. I think that @maker that
probably written this a lot ago can clarify in python when this is executed
and why we never seen any exception on this
Il 02/Set/2015 21:19, "Claudio A." <notifications@github.com> ha scritto:

> check it out @evilaliv3, was a typo and has never been triggered
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/globaleaks/GlobaLeaks/commit/27e36be9c4b1b89e0918a6a73fc47c1ad7257eef
>


This is not used, only shadowed.


Could you use free-id sets instead?


These loops are the same so they can actually be combined


I need the pyplug to compare, if you got it please post a link :)


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.488.28d143f52451feae12a93e5643c87bc3cfab80f9](http://build.palaso.org/viewLog.html?buildId=62565&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 140 Build time: 00:12:47


@dotansimha, consider looking at these guys' dropdown directive:
https://github.com/singlecomm/angular-sc-select
http://singlecomm.github.io/angular-sc-select/

ES6 compliant, supports local and remote pagination, all the common features people expect a multi-select dropdown to support.


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

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


This is weird, can you just check if `post` is `null` instead of doing this?


Good lord what have I done


Could you add indentation here?


Looks like your fix solved travis build issue


having to wrap a single %-formatting arg in a tuple like this is why I prefer the `format` method. But YMMV


```Ruby
sortedBooks = shelf.books.sort_by{ |book|
  [book['book']['authors']['author']['name'], book['book']['title']]
}
```


You realize, this doesn't work right? And my commit c23c72887f3e89aa3e399b96e754237ee170e0f6 worked around this by re-running stravaup.  Ideally the final `curl` call would iterate over all arguments so it doesn't need to request a new token each time.

Your code bails if there isn't **exactly one argument**:
https://github.com/mpolla/stravaup/blob/3dc3ac91502fcfa9e32ab1dd8d467c93f817c4b0/stravaup#L23

Furthermore, when using `find | xargs` I'd recommend using `find -print0 | xargs -0` to avoid surprises with whitespace.


OK. My thoughts: make a fixture which compiles ctypes_dylib and returns a path to it. Then, pass that fixture's name as a parameter to the ctypes-tests, replacing libname (which is current gs) with this library path. Does that capture your idea?


IntelliJ IDEA its doing this automatically.


With free-id sets, this would just be checking whether the intersection in empty.


This is starting to look much cleaner, well done


So the problem that you had with the loop before (with it going four times) is that you have

loopCounter = 0

and then you were saying <= degFahren.length (which would output 3)

so you go through the loop the first time with loopCounter = 0, then =1, then =2, then =3.

Now that you've switched it to < degFahren.length, you don't go through the loop when loopCounter = 3


Put this in `componentDidUpdate` and `componentDidMount` instead.


Er, wait, not exactly. It would be checking whether the set and the list have the same length.


How to send*
Please correct :-)


@Ruined1 yes ;-;


There's a typo : merge


I think we can finally commit an .enyoconfig file now that 0.5.1 of the dev tools are released.


Do `handleSuccess` and `handleFailure` get used here? Or are these remnants of old code? I noticed these functions appear in several places but only appear to be used in `LoginScreen`.


Does this really use the clip "under the playhead" or the selected clip? I don't think I want this to use implicit selection.


Thank you. :)

On Wed, Sep 2, 2015 at 1:18 PM, Ed Allonby <notifications@github.com> wrote:

> This is starting to look much cleaner, well done
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FatherFoxxy/CSharpTradeOffers/commit/0f1eaf9f96688399eb2934e2517fd25360b52a02#commitcomment-13032650>
> .
>


So here, you are saying that we go through this loop when age > 0, but we never actually get them to enter another age (so it's either always true -- in which case it goes forever, or always false -- in which case it doesn't ever go)


That should be `DECODE` and not `READ`


IntelliJ IDEA is doing that import automatically. I think it doesn't mind


91958608DOASEffectOnZoneSizing (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (763 of 763 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-10.33%25-green.svg)


Add a comment explaining the variance of each position.


Don't need that


Missing semicolon


I'd have a new line here, @twizmwazin's choice though.


Hi David, what is the task here? I am a bit lost hehe


This feels really hacky atm, so I'm looking to improve this in the future.


Use Oracle's naming conventions - ```String name = "iDev_"; boolean autoUpdate = false; boolean dontdothis = true;```


Can you explain why this is necessary? And why it's only necessary for signatures and not anything else?


Indentation.


Nice, thanks!


Consider replacing form with normal link, unless this makes styling difficult. If you add `data-url="/"` you don't even need to bind any events.


Idem.


I'd probably use ```Map<String, TeamModule>```, @twizmwazin's choice.


Ok, just let me know whether it works. Then we can merge it into master.


Same thing as earlier.


Up and down already change playlist index with ctrl+ changing position. Maybe alt+ should affect whether it is opened instead of using n.


Lol, we fixed it at the same time :D


You don't need this right?


This should likely have an os.path.exists check so that the memoization is discarded if the binary has moved under foot.  That only really ever happened for upgrades across major versions where the active python shifted.

...and yeah, I'm not trying to run you through the ringer on this PR- I'm just remembering past issues :)


Are there any procedure bindings in `racket/unit` that would need types in the base type environment?


Also added a border around avatars and made them wear their second layer.


I can certainly get rid of the cache portion.  The intent is to avoid duplicate flags if the variable already contains them.  This is especially problematic for cache variables but not limited to them.  For example, imagine your CFLAGS environment variable is set to "-march=native -Wall".  Without the macro, the first invocation of cmake will result in "-Wall" being listed twice.  If the variables are forced cache values, then each "configure" (think of using ccmake, for example) will add another copy of it.  The macro lets you ensure that a given option is only added once, regardless of whether or not the variable is cached.


New line here and after this void.


Does not handle the case where a reader would be slow enough to try to read an index which has been overwritten.


Das ist scheinbar nicht das gesuchte Zeichen.


Think so, thanks!


@EasyAsABC123 This is a learning experience for me. I appreciate the feedback so I can implement this as cleanly as possible.

I see that I can just set a default action. However when you want to `clear`, the command is

     appcmd clear config fooo

Doesn't the config() function need to set an argument?

```ruby
action :config do
  config()
end

action :clear do
  config(action = :clear)
end
...

def config(action = :set)
  ...
  appcmd #{action} config  ...

```


all tests pass:
success: http://buildbot.bos1/builders/test-riak_ensemble/builds/230


Why has this been deleted? Is there any way to specify the output file name in the junit reporter section in karma configuration?


Hier auch noch mal. Ich hoffe du machst das nicht alles per Hand.


I generated a dozen Lucario sets before pushing this and it was fine. Randomizing may not be the best, but it's better than what we had previously. Alternatively, we could accept the setup that is first in the case of ties.


`PDK.logout` has a callback, maybe call `onAuthRequired` then?


Maybe this should be err.toString() ? :wink:


This breaks playhead from ever appearing. That's why I did not include it here. Is it causing you problems without this change? If so, maybe we leave it in with this regression until Qt is updated again.


Thank you. This is one change I wanted to see done.


test


ATM it's using entirely implicit selection. Is there a good reason for requiring the user to click before invoking the trim? When clicking, you move the playhead, so it could be frustrating if you lose your position if you forget to select in advance... Ctrl+space helps, but I'm not sure about how many will realize that.


Consider renaming to `InputField` or something? `FormGroup` makes me think of a collection of form field inputs.


```ruby
action :config do
  #calling a function without parameter will use default
  config()
end

action :clear do
  #you only need to pass the value you want action to be
  config(:clear)
end
...

#creating a function with default parameter action set to :set
def config(action = :set)
  ...
  #this is perfect you don't need the action #{action} do thing...just the stuff the :config action used to have with the one word as a parameter like you have below
  appcmd #{action} config  ...
```


Judging by pull requests I get, libSDL2pp was successfully used on windows. Around a year ago it was with MSVC - it haven't had support for some C++11 constructs like noexcept so I've got a branch with these removed and some other places hacked. Since the branch has fallen behind too much since then I've dropped it, however I've also heard that newer MSVC has improved C++11 support so I hoped this should not be a problem any more.

And judging by https://github.com/AMDmi3/libSDL2pp/commit/32f5bf0b1898fd2d3f078b65cebdeb4fcac96a9c, it's also used with mingw. I guess I should try to build it with mingw as well - I've had success with SDL-using projects before (even though it was cross-compilation, all I needed was to set up cmake toolchain file) but never tried with libSDL2pp.

> Initially I made this change because the upstream-built MSVC binaries of SDL don't have an SDL2 directory between /include and the actual headers. (A practice which I agree is unwise)

What is actually unwise on SDL2 devs side is to use different schemas between *nix and windows builds, as this makes life harder for users. However I'd just try to place SDL2 includes under SDL2/ subdirectory.

> Thus, when I built my SDL2 find module, I sought to successfully find SDL2 from the distributed binaries.

This one's really overcomplicated if you ask me. I don't think it's a good idea to implement such complex logic in cmake search module - they should find header and library, so there should be single FIND_PATH and FIND_LIBRARY directives, not much more (though I'm not against an idea to use pkg-config output as a hint to these, which is a common patter as well; I may try to implement this and see if it helps you). On systems where there are standard include and library locations this works out of box, on windows you just need to help cmake a bit by [specifying](http://www.cmake.org/Wiki/CMake_Useful_Variables#Environment_Variables) CMAKE_INCLUDE_PATH and CMAKE_LIBRARY_PATH

> though I know some who would say those should be double-quotes with relative paths, to be sure you're including the peer header - this advice I am undecided on

This one is interesting - I remember taking this decison, but don't remember any arguments for either ```""``` or ```<>```, and can't google any now. Maybe I just looked at what larger libs (boost) do. Well this may be changed if there's reason, however I'm still strongly against dropping SDL2/ from #includes.


Hmm, that appears to sort by first name. I'll fiddle with it and see if there's a way to do this easily. Like you said before, the best solution is probably just Goodreads sorting correctly by default.


It doesn't seem like this would update `m_read_only` if set_dht_settings() is called with a different setting, or am I misreading this?


You can stop it from doing that in the settings. We shouldn't be using wildcard imports.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.469.28d143f52451feae12a93e5643c87bc3cfab80f9](http://build.palaso.org/viewLog.html?buildId=62569&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3357, ignored: 134 Build time: 00:24:23


@designmatty Any opinion on this?


it turns out that this code won't be ran until the 7 day timer expires while waiting for $KEYS_UP to not be zero. I think $STACK_NAME is not part of the default user-data, and it should be $NUBIS_STACK

See https://github.com/Nubisproject/nubis-puppet-eip/pull/11


Jakbyś nie skracał zmiennych to ten komentarz wyżej byłby zbędny.


camelCase


Done with my review. Looks good!


Spelling: `won't keep this node`


Players also need to be immune to SPELL_EFFECT_ATTACK_ME. And I would suggest using a IsImmuneToSpellEffect override for Player, because shitty code can override ApplySpellImmune during a player object's lifetime.


Specify the types just to make everything easier


Thank you!

2015-09-02 20:35 GMT+03:00 Bjørn Forsman <notifications@github.com>:

> Currently you have to install nixos.gnome3.gnome-video-effects into your
> profile.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/dc4d5aa29ab97b46105fb20c21f496b5e5840d84#commitcomment-13029292>
> .
>

--
Sincerely,
Arseniy Seroka


mhmm


Oh, do we not support ints for `expose`? We probably should - I imagine a lot of people have YAML like this:

```yaml
expose:
  - 8000
```


No, I just included threw them inn everywhere initially. Then, when rebasing on your commit only this one was left. If this should be in here, then we're probably missing something like
onVisibleChanged: requestPaint()
but:
Bad news, I just got the crash again moments ago. This time I see it when I move the zoom slider in my project. Have you experienced the crash yourself? It would be nice to confirm whether I'm the only one seeing this.


Nie, cofnij to.


This seems to be breaking the checkout process.  I think you meant to put $this after $objOrder.


Do whatever you want with the mgba core, at this point I don't care.

Just don't let it be said that I tried to warn you about this beforehand.


Whats up with the </li>


Assuming you need the `resources :seasons` under orga, you could also write

```
namespace :orga do
  resources :teams
  resources :seasons do
    collection do
      patch :switch_seasons
    end
  end
  resources :seasons
end
```

I put it on collection, because I am assuming you never actually specify the season (since it always works on the `current_season`?)

Alternative you could consider a route on the member, which would allow to change the parameters of any season, for as far as that is applicable, but then in your form you would have to write `switch_seasons_orga_season_path(current_season)` (tip: check `rake routes` to get the correct path string).

As an aside I find the `switch_seasons` action-name confusing: you are actually just updating the phase of the current season, so I would prefer `switch_phase`.

Also, instead of `@fakeseason` just use `@season`. You are faking the phase, but the season is the real season (well sort of). Also a bit confusing because sometimes you use `@season` (back to reality) and sometimes you use the `@fakeseason`. Keep it simple :smile:


Yeah, not great that we have to do this but it's probably best for now.

FYI, I think `kwargs.pop(key, None)` will do the same thing as the combined `get` and `del`, but that needn't hold back the merge.


yup...


good point.