The burden of... copy-paste?

While doubled getters are indeed an ugly consequence of practicing const-correctness, even in this case it seems that the *bigger* burden was being forced to touch *three* separate places to enable compilation. ;)


because then some of the elements 30..39 would be interleaved with the first thirty elements


Do we need tempConnector != null check according to your method implementation?


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.149.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=66700&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:44:48


Can we be consistent with our punctuation?  Notice some lines end in periods, others don't in 1.0.3.


удалил ключи, выполнил add_quadkeys.php и его лог:
string(151) "ALTER TABLE `geo`
            ADD COLUMN `quadkey` BIGINT(20) UNSIGNED DEFAULT NULL,
            DROP INDEX `Coords`,
            ADD INDEX (`quadkey`)"
Failed to alter table geo: (1091) Can't DROP 'Coords'; check that column/key existsstring(155) "ALTER TABLE `mem_geo`
            ADD COLUMN `quadkey` BIGINT(20) UNSIGNED DEFAULT NULL,
            DROP INDEX `Coords`,
            ADD INDEX (`quadkey`)"
Failed to alter table mem_geo: (1091) Can't DROP 'Coords'; check that column/key exists

mysql> desc mem_geo;
+-----------+---------------------+------+-----+---------+-------+
| Field     | Type                | Null | Key | Default | Extra |
+-----------+---------------------+------+-----+---------+-------+
| BSSID     | bigint(15) unsigned | NO   | PRI | NULL    |       |
| latitude  | float(12,8)         | YES  |     | NULL    |       |
| longitude | float(12,8)         | YES  |     | NULL    |       |
| quadkey   | bigint(20) unsigned | YES  | MUL | NULL    |       |
+-----------+---------------------+------+-----+---------+-------+
4 rows in set (0,01 sec)

mysql> desc geo;
+-----------+---------------------+------+-----+---------+-------+
| Field     | Type                | Null | Key | Default | Extra |
+-----------+---------------------+------+-----+---------+-------+
| BSSID     | bigint(15) unsigned | NO   | PRI | NULL    |       |
| latitude  | float(12,8)         | YES  |     | NULL    |       |
| longitude | float(12,8)         | YES  |     | NULL    |       |
| quadkey   | bigint(20) unsigned | YES  | MUL | NULL    |       |
+-----------+---------------------+------+-----+---------+-------+

в схеме уже нет `Coords`, может тогда просто вырезать строки ( DROP INDEX `Coords`, ) из скрипта?


it's calling the callback, not creating it. please change name.


Should include the registrant's toString method in the error, and the scope in which it is already registered. e.g. format("\"%s\" is already registered in scope \"%s\"", scoped, scope). You could change hasScopeInHierarchy to return the found scope.


I'd call callback fn since this is mapping a function and callback resonates as async continuation pasing style stuff, but I guess it is personal opinion :stuck_out_tongue:


Надо делать описание внутри state_machine. Это позволит навесить дополнительную валидацию на определённые state.


Hi @ianhirschfeld, your fix works for the Unity encoding header issue, but the "Wrong number of arguments" issue happens in other places with rails 4.2.

I pulled back the latest rack 1.6 version and applied your fix in `lib/rack/multipart/parser.rb`, and everything works fine now : https://github.com/glyph-fr/rack/tree/1-6-stable

Hope this can save some time to somebody


So YAPF is switching the format on this file from what @ephur submitted yesterday.  Are you running an old YAPF or is Richard?

https://github.com/metacloud/molecule/pull/23/files


Method is calling the callback, not creating it. Please change name.


:point_up:
> Merge remote-tracking branch 'origin/widgets' into widgets-dashboard-…


e.g. `executeCallback`


Should the message name be `callCallback` or `callback` or `callBack`. :) What do you think?


cc @piensaenpixel too look into if cartodb-popup could/should be removed


Was not sure how to do assignment for the dynamic... I get this error
```scala


sms and email have the same message form(MessageForm)...


`expected` is a crappy mechanism. This test will pass if any line throws that exception.

```
try {
  root.register(scoped);
  fail("expected IllegalStateException");
} catch (IllegalStateException expected) {
  // Pass
}
```


Ditto.


![continuation passing](http://media1.giphy.com/media/O6kb3Pa3Kgbsc/giphy.gif)


Addresses pootlepress/18-Tags#67 and  pootlepress/18-Tags#66


Serial number of some app should not be here


This story should be about unmanaged source sets. Managed source-sets should be handled as part of "Plugin author defines @Managed subtype of LanguageSourceSet".


I think 404 is okay, though I don't really have a strong opinion on this.


Maybe worth mentioning that this is an unmanaged internal view.


ack


Для работы с формами предпочтительнее использовать гем simple_form (https://github.com/plataformatec/simple_form). У него много функционала из коробки для отображения контролов, переводов для них, которые берутся из i18n и др.


We should send a 404 with the error, something like https://github.com/joakin/loot/blob/master/lib/routes/restbase-req-handler.js#L27-L34

I also agree with logging the error, I would recommend logging `e.stack` for getting a full stacktrace (better diagnosing if looking at the log)


A SEMICOLON!

![](http://media4.giphy.com/media/Qw4X3FOWnLI44Hj29pe/giphy.gif)


Again, this should be about unmanaged types. We should cover things like:

- default implementation is not a non-abstract class
- default implementation does not implement internal view
- internal view is not an interface


I also changed the size of the window.


Can we throw some exception here? with a message like "Invalid Mediator type found" inside a else block.


this never pass or something changed so we lower the bar?


Уже при написании этого контроллера должны быть тесты.


I suppose that using other character doesn't fix the problem entirely because other rule would enforce a blank space to be after the initial hash, isn't it?


Очень интересное решение, но оно не всегда кастомизируется.


What about instead removing that commented code?


I think I was confused by looking at some of the webrtc example from the webrtc group.. here
https://webrtc.github.io
https://github.com/webrtc/samples/blob/gh-pages/src/content/getusermedia/gum/js/main.js

 So just to be clear.. you want to work from this spec
 http://www.w3.org/TR/2013/WD-mediacapture-streams-20130903/

And I would make this it's own file/package ?
Would I remove the `MediaStream` and `MediaStreamTrack` from lib and add them to this new package as well?

Thanks for your help :)


Seems this class does not provide any specific features related to a factory class, ex. returning some objects
Only having a single utility method, still do we need this class?


This use to Timeout is not necessary either. You can use `read_timeout` just like here: https://github.com/ambethia/recaptcha/blob/35b45ccc61372ebce4830a0eccd26a64c1298dfe/lib/recaptcha.rb#L59


@grosser Let me know if you'd like a PR for that.


ewe


maybe `hasPackage`  is better since it's made for `getpackage`.


Actually the previous version did not had these lines, I added this after trying to the the value of the ReactiveVar, and it did not fired the Autorun methods.. which makes sense... do you have another idea?


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


Typically in the Builder pattern you just have a `Builder` inner class and from context, `CommandDescription.Builder` is clear that it's – you guessed it – a _builder_. But it's not important ^^ (`CommandDescription.CommandBuilder`, hmm)


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/4207/console
Build was first broken at http://ci.cgeo.org/job/c-geo/4202/


should be fixed now


can you add a method to AdaptiveMetropolisSampler that returns N samples?  this would save you a few lines of code here and below (and could be generally useful)


ack


Ок, предлагаю в таблице сайта переименовать Url на MainUrl или HostUrl?


TED x Talk 를 soup 객체를 통해서 조회했을 때, 우리가 원했던 소스가 나타나지 않아서 참조할 사이트를 TED-Ed 로 바꿨다.


We (@piensaenpixel  and I) were thinking of bringing those styles back so that infowindows look right with legacy markup work, but I think we should remove `cartodb-popup` from here.


space around '-'


Разве нельзя это сделать более простым способом, вызвав у объекта метод state_events?


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


Yes, he talked to me indeed. Did you update the txt file with his credits btw?


God dammit, I left that there again u_u


Because it is what you get when you have a VFR file, different patterns with the differents frame rate encountered. With a VFR file, during  my tests at the time, i've got very easely a length of pattern (m_pattern.Size()) of 4 or 5. And so the purpose of this, to properly play a VFR file (don't have stuttering), if the VFR file condition is detected (m_VFRCounter >= VFR_DETECTION_THRESHOLD), is to lock the framerate to the highest detected, to play properly, and also prevent any future switch by the original algorithm. With the actual value, it took around 30s to trig the detection on a VFR file.


You can't change the motor thrust pin values because they are still the same on the hardware side. I reverted this, but be careful when changing things that can have side effects.


Updated for GCC 4.9


Yes, the lack of a standardized 'asprintf()' / 'vasprintf()' is a real pity, since it would be a *really* handy tool. I'm thinking about providing an implementation in libntp that can be used if the system does not provide one. As long as top performance is not an issue this is fairly simple, provided a) there's 'va_copy()' and b) a working 'vsnprintf()'. There's more than one location where it would be useful.


@mtrachtenberg  made a change for test


line level comment


other yml files do not have conf here, are you sure this works across the board?


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.600.967577e5be45acf7939490aaf9e49485065eb151](http://build.palaso.org/viewLog.html?buildId=66708&buildTypeId=bt322) outcome was **FAILURE**
Summary: Execution timeout (new); tests passed: 2148, ignored: 78 Build time: 00:42:29


@sorvell This change is causing ```<link rel="stylesheet" type="text/css" ...>``` tags, which are valid, to be ignored, but I think this change was just meant to ignore stylesheets with non-standard types, so can we add ```'link[rel=stylesheet][type="text/css"]'``` and ```'style[type="text/css"]'``` to the list of importsSelectors?


Should this be parametrized?


Does rippled even build in Qt anymore?


When I pulled this change, go can't find the UnsetLinkIP method. It doesn't seem to to be in the Tenus code on github. Is there something else needed to make this work?


Updated for GCC 5.0


"Thirst" question. I'm brilliant!


In IntelliJ, right-click top folder and select "Compile module 'authme'"


Before some of these items lived at self.config['ansible'] and others were at self.config['molecule']  were all instances consolidated to self.config['molecule']....?


accountcode is not always a domain name its that way by default but I don't see it as a great default because the domain name can be changed.


Have you seen #10828?
I don't think unionfs is using more ram than available. At least in the stack trace I showed on that issue, it was actually p9fs the one that tries to allocate an unreasonable amount of (contiguous) memory, and increasing the RAM size will probably not help all that much (see the issue for more details).


 ∞  (venv)  ephur@RIMAYNAR-M-61CL  ~/Projects/epsilon/ansible-systems/roles/test.metapod_lite  ± master   yapf --version
yapf 0.2.9

Yeah mine was older for sure ;)


:+1: LGTM


Не всегда нужно 100500 роутов для решения таких задач.
Опять же, проблема с кастомной логикой.
В качестве универсального решения, можно создать один роут и передавать в качестве параметра ему event.


Ah, good point.  `Boolean` is not `<:` `js.Any`, so yes, that will fail.  @sjrd following your comment a couple of days ago in another PR regarding using `Any` over `js.Any`, is there a rationale behind `updateDynamic` and various other `js.Dynamic` methods typing as `js.Any`?

I would say that casting should work okay (it's bit ugly, but it's controlled and safe, at least).


@wizeman ah ok, well it helped giving us a successful tested job at least for unstable release...


Hacky fix for now. Will have to explore more later. But basically I t wasn't compiling and I wanted to sleep.


Isn't that the same for STANDARD_COVERAGE?

This is a preparation for the time when NEWEST is set to a higher version. Otherwise the person increasing the NEWEST version has to adapt the PRESERVE_ORDER and GROUP_BY_INSTANCES version lists (arrays).

I would propose to modify MultiVersionSpecRunner.createExecutions() in a way that it does not add duplicates to the list of (planned) executions. This can be done by calling toUnique() on array "versionUnderTest" before adding the versions to the list "execution".


Lol, no i will help, but you will have todo allot too😉


@aalba6675 but what about -c ${ceph_conf_file}? shouldn't we still need to be able to have that passed?


comentari


Фабрика должна инициализировать значение по умолчанию, которое выставляется в модели у state_machine.
Если требуется task именно с этим state, лучше создать дополнительный trait, не затрагивая базовую фабрику.


this is gone in 1.0.2


Would it be possible to run the initialization
from different place, than to do "if (first)"?


es git kei einzahl vo Spesen. pls revert


NICE


AFAIK the oldest distros still supported are RHEL/Centos 6 and Ubuntu 12.04. The former comes with libstdc++ 4.4 and the latter with libstdc++ 4.6.

FWIW, Firfox has listed libstdc++ 4.3 as the minimume supported version since 4.0


how do you suggest to fix the false positive detection in this case? it detect this

CPullupCorrection: detected pattern of length 2: 33333.33 50083.33, frameduration: 41708.333333

then loses pattern and detect this again. after 3 times it switched to 30 hz which is wrong


@naucoin Thanks for the report. These has been fixed. Consider grabbing the latest version.


es git s wort Spesen ned ide einzahl. pls revert


Is there a gain by making this change?  Specifically it was `Void` to make it clear that there was never any `onNext()` from this `Stream`, only a `onComplete()`.


Ok found it, thanks!


I did this because I'm not sure what FORWARD/BACKWARD are used for right now so I don't want to change them yet. I think its a hardware side thing, maybe @ddworken can tell us how the motor controller works.
Also Serial.print(0) doesn't print a string "0" but rather a null terminator (I think).

You can add back STALL_THRESHOLD


Can we proceed even with an exception?


can these tests be easily expanded to check the standard deviation of the samples (i.e., are there analytic/numerical values to compare to)?  if not, ignore this comment.


If you know that browsers are now implementing the `20150414` version, then feel free to go for that one (since, ultimately we'd need to update it), although I think it's perfectly reasonable to implement `20130903`, for now.

Yes, I'd move these both into a `org.scalajs.dom.experimental.mediastream` package, with a link to the spec you are following and a note mentioning it's incomplete and implemented only to extent required to support webrtc.


wetsch ned di glich reihefolg nä wie be setAttachment obe?^^


Why protecting the monitor structure is needed?
(perhaps the log entry can mention this)


We are refactoring away from event.  Since everything is specific messagelist or group, events add very little value except in exceptional cases.


Не подошёл
+ при неверном вводе страница сбрасывает стили и не показывает сообщения об ошибках


Why are we not using ATTRIBUTE_SEPERATOR in append methods above and replacing it here?


capitalize per


Структура таблицы Sites сейчас - id, name , предлагаете добавить поле MainUrl или HostUrl ?


Oh! I totally forgot you did
![hicoqaz](https://cloud.githubusercontent.com/assets/12707525/11509452/22d17fc2-982c-11e5-9500-eb70fc59e956.gif)


@simonbrandhof No more support of read timeout?


No, sorry, it isn’t. I got this far, but don’t have the time to do the full implementation. We can’t build this on top of the current border drawing code, and just using a shape or shapelayer makes it look ugly.

At the very least, I wanted to ship the mapping code, so that somebody else can pick it up. I had the code lying around for 6 weeks now.


The `B` is redundant on this line and grey in IntelliJ meaning it can be removed.  The compiler can figure _that_ out I guess :grimacing:.


@vmcloud need to check that request_uri.find("//") >= 0 before anything else


I think this may have introduced a regression. In many transcriptions the word appears around 200ms late.


How about
$_kvm_mode = "vm86", "dpmi", "vm86_dpmi" ?
This would be consistent with $_cpuemu.


do you have a VFR file where it would detect exactly the same pattern 3 times in a row?


I am a professional developer don't you lol at me


@SabreCat I'd say that if it's used only in the tests it should belong to the tests not to the real codebase


@manstis Now that I looked at this in the WB, this actually isn't sufficient. Problem is when a file is added/removed/saved or the lock status changes, this method is invoked and causes the project explorer to collapse. This is weird as you're editing a file and all of a sudden the explorer collapses all asset types. It needs to remember which ones were open and not collapse them all or they should only be collapsed by default on startup, then remain expanded.


Here, we can convert to char array first and then loop using enhanced for loop.
public char[] toCharArray()


I actually think this is a dead file...


@Sandy4999 should pass defer_user_update as third param, not always true


A single method in a class, can't we use some util classes for this method?


I think it's legit, I'd import them from the where they're exported the first time, not through index, in future we may want to stop loading everything at once from index and just load something when we need it


Seems to be fixed, thanks!


I'm not sure they were too creative to begin with. "Creative"


OP is good m8, gib 110.


Why return empty string for the sequesnce type??


Yup.


Picku koji sou kod


When your as good as i am all you need is an extra ; and one less /


I really like having a common function that makes these time determinations.  But I see @seelabs' concern about whether we use `app_.timeKeeper().closeTIme()` or `app_.timeKeeper().now()`.  Here are the places in the prior revision that used to use `closeTime()`:
 o `ValidationsImp::addValidation()`
 o `PeerImp::onMessage()`
Here are the places in the prior version that used (and still use) `now()`:
 o `ValidationsImp::getValidationCount()`
 o `ValidationsImp::getCurrentTrustedValidations()`
 o `ValidationsImp::getCurrentValidations()`
It would be great to consistently use either `now()` or `closeTime()`.  Otherwise we'll be wondering which is the right reference to use during maintenance.  I'll trust you to make the right choice whether it is okay to consistently use one of these references.


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


Left in by mistake -- I'll remove it.


@koistya you should make an new patch 0.1.2 in order to update this on [npm@git-push](https://www.npmjs.com/package/git-push).


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/4208/console
Build was first broken at http://ci.cgeo.org/job/c-geo/4202/


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


Good names.  Nice improvement.


I prefer following order of elements in a class,
variables
constructor
public methods
private methods
getters and setters


Было бы здорово в качестве параметра его передавать


an empty line between class xx: and def __init__, check PEP8


Function name should be lower case, foo_bar style.


Even better!


I did, just now. I also put up a small fix for something he brought to my attention with the rifle's zoom.


maybe change this to a javadoc author tag throughout?


I know that ReadTheDocs supports multiple versionsing, but I think it is based on maintaining the documentation in the same repository that holds the corresponding module/version. We'd definitely need a way to prevent redundant versions of the same content (that is applicable to multiple versions). Maybe Chris and I can kick it around and come up with some possibilities.


Waste of memory. vals, as a list, is only to hold all element and print before return, why not print on the fly?

while runner is not None:
        print runner.val,
print ''


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


done


SEED a commit just in case


final?


can return a primitive


final


use is to check whether is None.
if runner is not None:

'is' is identity check, '==' is equality check.


whitespace


It seems too many code lines for a constructor, shall we have a separate method for creating sequence from json element?
what do you think?


what if val_list is empty, you will have index out of boundary error


Did you intend to remove the earplug toggle from the communications menu with the addition of the PB control? It doesn't look like this was re-added elsewhere.


Note, this is a breaking change for those that used a second param to the `filter` function (the only case I could find was at https://github.com/stefanpenner/broccoli-stew/blob/42717fcdfbcf1e211316e4b16697d14b530f38c4/tests/map-test.js#L25-L30).


no trailing empty lines.


@HiAaronCody Touchè -- thanks for that.


@anurse @piotrpMSFT  It will work with the leading `/` but for consistency's sake we should omit it.


run `hh_client restart` - facebook/hhvm#6551


default scope variable, why?


@karamcnair I think Maybe's should get pinged, anyway; since they've shown interest in the event


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


final.  also, although i like defining classes for each of the samplers and exposing the likelihood calculations, i'm not sure if we need to put the samplers in their own class.  maybe put them in AFModel and make them protected so they can be tested?


Checking if @trilinos/shylu works for comments on commits. Can someone from shylu team who unwateched Trilinos repo confirm if they see this message.


It can't be `nil` because `GetConversationWith` is actually "get or create conversation with".


Don't use var name that is less than three chars. wtf is s supposed to mean


Firste commit


This change works as expected and improves visibility. I personally endorse this being merged into the master branch.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.150.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=66716&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:37:43


whitespace


:thumbsup:


parenthesis is not necessary.


hello, can i have your email or skype usernaname? i would like to talk about this project


Yes.

On 12/01/2015 11:20 AM, Siva Rajamanickam wrote:
>
> Checking if @trilinos/shylu
> <https://github.com/orgs/trilinos/teams/shylu> works for comments on
> commits. Can someone from shylu team who unwateched Trilinos repo
> confirm if they see this message.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/trilinos/Trilinos/commit/7ab207091e506893c4368b02d7e902d7ff9adac6#commitcomment-14716057>.
>


```
ps = self.get_config("client", "peers.preferred", "").split(",")
preferred_peers = tuple([p.strip() for p in ps if p != ""])
```


Added another program to download images from the logo feed of Abduzeedo.com


String concatenation is expensive. I usually do this.
result = []
result.append(string)

return ''.join(result)


Why calling super() manually here?


Ok I'll look at tomorrow; shouldn't be particularly taxing.


@jcfr Updated to the latest version and that built for me, thanks!


Hola Sergio,

¿Puedes explicarme porque desglosas los **status_code** mayores que 99 en **pending** y **cancel**? ¿No deberia ser simplemente **done** y **error**? Lo he estado testeando con el entorno de pruebas de Redsys y dejar en estado **pending** o **cancel** una transacción, cuando en el fichero *controllers/main.py* haces un **website.sale_reset(..)** justo después de crear la transacción,

class WebsiteSale(website_sale):
    @http.route(['/shop/payment/transaction/<int:acquirer_id>'], type='json',
                auth="public", website=True)
    def payment_transaction(self, acquirer_id):
        tx_id = super(WebsiteSale, self).payment_transaction(acquirer_id)
        cr, context = request.cr, request.context
        acquirer_obj = request.registry.get('payment.acquirer')
        acquirer = acquirer_obj.browse(
            cr, SUPERUSER_ID, acquirer_id, context=context)
        if acquirer.provider == 'redsys':
            request.website.sale_reset(context=request.context)
        return tx_id

se pierde toda opción de recuperar la transacción posteriormente para completarla.

Y respecto a este **website.sale_reset(...)** no sería más adecuado ponerlo dentro de la rama **done**?

De todas formas pongamos donde pongamos el **website.sale_reset(...)** el método **def cart_update_json(...)** y **def payment_validate(...)** (aunque este último no necesariamente se ejecuta) del fichero *addons/website_sale/controllers/main.py* resetean eventualmente los datos haciendo igualmente imposible recuperar la transacción.

Te lo pregunto para asegurarme que no se me escapa nada antes de proponer un pull.

Muchas gracias de antemano
Eric


Any explanation why this is forbidden now? I'm actually using this for my TestTools: https://github.com/lastzero/test-tools/blob/master/src/TestTools/TestCase/UnitTestCase.php line 66. I'm still testing with 3.0, but up to 2.8 I didn't notice any problems with cloning the container. Your change easy to circumvent in inherited classes as well, as long as you don't make the container final (which I hope, you don't).


Idem que lo anterior.


just do `../chrome`, initial `./` is unnecessary


Do we need to allow to create this exception without any arguements?


Looks good!  People searching Google for "Moses LM OOV feature" will find
it.

--Michael

On Tue, Dec 1, 2015 at 11:20 AM, Ulrich Germann <notifications@github.com>
wrote:

> Added here:
> http://www.statmt.org/moses/?n=FactoredTraining.BuildingLanguageModel. Let
> me know if you can thing of a better place to put it. I'm still having a
> hard time finding my way around the Moses documentation.
>
> - Uli
>
> On Tue, Dec 1, 2015 at 3:52 PM, Michael Denkowski <
> notifications@github.com>
>
> wrote:
>
> > You're totally right; KenLM supports oov-feature. I didn't have the
> latest
> > commit of moses the first time I tested it. It may be helpful to add a
> > note about enabling this feature to the "Building a Language Model"
> section
> > on statmt.org so people searching for "Moses LM OOV" find it. I see a
> few
> > old posts on the mailing list about people asking whether it's supported.
> > Thanks again!
> >
> > Best,
> > Michael
> >
> > On Mon, Nov 30, 2015 at 7:08 PM, Ulrich Germann <
> notifications@github.com>
> >
> > wrote:
> >
> > > Hi Michael,
> > >
> > > but it should, since it's derived from LanguageModel (check (virtual)
> > void
> > > (?) LanguageModel::SetParameter() in moses/LM/Base.cpp) and doesn't
> > > override EvaluateInIsolation. Is the "looks like" based on code
> > inspection
> > > or looking at feature vectors in the output? If the latter, it's
> > probably a
> > > bug, not a feature. On the other hand, the commented-out special member
> > > variable is likely to be residue of just fiddling with the code. If you
> > > want to investigate, I'd suggest that you look at the output with
> feature
> > > values (e.g. from n-best lists) first. Somewhere in the logging output
> (I
> > > can't remember where, there's a record in which oder feature values are
> > > reported ("moses --show-weights -f moses.ini" should produce
> > interpretable
> > > information as well).
> > >
> > > Cheers - Uli
> > >
> > >
> > > On Mon, Nov 30, 2015 at 11:28 PM, Michael Denkowski <
> > >
> > > notifications@github.com> wrote:
> > >
> > > > Thanks for the explanation! It looks like KenLM doesn't support
> > > > oov-feature=1, though it does track the number of OOVs in part of the
> > > > code. I may look at implementing this when I have some time.
> > > >
> > > > Best,
> > > > Michael
> > > >
> > > > On Mon, Nov 30, 2015 at 6:11 PM, Ulrich Germann <
> > > notifications@github.com>
> > > > wrote:
> > > >
> > > >
> > > > > Hi Michael,
> > > > >
> > > > > David Madl implemented most of this change, so he may {want|have}
> to
> > > > chime
> > > > > in on the details, especially if I'm misrepresenting things here.
> We
> > > are
> > > > > currently working towards reducing the dependency of Moses
> components
> > > on
> > > > > static data, in hopes that Moses components can eventually be
> > > > instantiated
> > > > > even outside of the Moses Monolith.
> > > > >
> > > > > In essence, this change only moves the specification "OOV feature
> > > > enabled"
> > > > > from the global space (via moses.ini) into the local space (via the
> > > > > configuration line (oov-feature=1) within moses.ini, which can be
> > > > supplied
> > > > > to the constructor even when StaticData is not available globally).
> > So
> > > if
> > > > > you have any legacy moses.ini files that do have
> > > > >
> > > > > [lmodel-enable-oov]
> > > > > 1
> > > > >
> > > > > you'll have to rewrite them into
> > > > >
> > > > > WhateverLM ... oov-feature=1 ...
> > > > >
> > > > > The abstract LM class LanguageModel sets a boolean member
> > > > > m_enableOOVFeature during construction and sets the feature count
> to
> > 2
> > > > > (instead of 1) if m_enableOOVFeature is true. The (virtual) default
> > > > > implementation of LanguageModel::EvaluateInIsolation(...) provides
> > the
> > > > OOV
> > > > > feature and counts OOVs.
> > > > >
> > > > > LM sub-class implementations that override this virtual function
> must
> > > > take
> > > > > responsibility for this themselves. This is unchanged from the way
> > > things
> > > > > were before. According to 'find moses/LM -name '*.h' | xargs grep
> > > > > EvaluateInIsolation', none of the current implementations in the
> core
> > > > Moses
> > > > > code repository actually do override EvaluateInIsolation().
> > > > >
> > > > > LDHT (whatever that is, it seems to be related to RANDLM, which in
> my
> > > > > opinion is primarily of historical interest) will need some fixing
> up
> > > (it
> > > > > resets m_enableOOVFeature to false but doesn't update the
> > > > > m_numScoreComponents), but I'm inclined not to cross that bridge
> > > before I
> > > > > get there.
> > > > >
> > > > > Feature functions are now registered in the respective FF factory
> > after
> > > > > they have been constructed (instead of
> > > > FeatureFunction::FeatureFunction()),
> > > > > because the number of feature values a FF provides is now unknown
> > until
> > > > the
> > > > > FF has been constructed. That's a purely technical change (with
> some
> > > > > benefits for future modularization).
> > > > >
> > > > > So bottom line, except for the need to enable oov-feature=1
> > > individually
> > > > on
> > > > > each LM feature in moses.ini (instead of the [lmodel-enable-oov]
> > > > > specification), nothing should have changed from the user
> > perspective.
> > > > >
> > > > > I hope I got it right and I hope this helps
> > > > >
> > > > > - Uli
> > > > >
> > > > >
> > > > > On Mon, Nov 30, 2015 at 9:25 PM, Michael Denkowski <
> > > > > notifications@github.com
> > > > > > wrote:
> > > > >
> > > > > > Does this mean there's a generally usable LM OOV feature or do
> > > > individual
> > > > > > LMs have to implement it? Does the KenLM interface support it?
> > > > > >
> > > > > > —
> > > > > > Reply to this email directly or view it on GitHub
> > > > > > <
> > > > >
> > > >
> > >
> >
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14693766
> > > > > >
> > > > > > .
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Ulrich Germann
> > > > > Senior Researcher
> > > > > School of Informatics
> > > > > University of Edinburgh
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14695969
> > > > >
> > > > > .
> > > >
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14696321
> > > >
> > >
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > Ulrich Germann
> > > Senior Researcher
> > > School of Informatics
> > > University of Edinburgh
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14696894
> > >
> > > .
> >
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14712112
> >
>
> > .
> >
>
>
>
> --
> Ulrich Germann
> Senior Researcher
> School of Informatics
> University of Edinburgh
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/moses-smt/mosesdecoder/commit/bcb0212b5b7a9535eb859f228bcfdf833aa57462#commitcomment-14712953>
> .
>


Οκ

Στις 1 Δεκ 2015 7:38 μμ, ο χρήστης "Liakis Stefanos" <
notifications@github.com> έγραψε:
>
> sms and email have the same message form(MessageForm)...
>
> —
> Reply to this email directly or view it on GitHub.


misspelled :)


I'm trying to wrap my head around the recent changes to the DMA system, as a recent rebase with these changes broke my somewhat-brittle I2S code. I suspect that is happening because the DMA idle counter is timing out and shutting down the DMA engine.
I found the `dma_tickle` function which seems like it could be the solution to my problem. However, it doesn't get used anywhere outside of `dma.c`, and even there only in `dma_disable_clock`; and it isn't exposed in `dma.h`
Should this function be exposed? Or should I be looking elsewhere for a method to reset the DMA idle counter to prevent it from timing out?


Although `conversation.GetOurKeys()` is not `nil`, `conversation.GetOurCurrentKey()` is `nil`.

It happens because `ourCurrentKey` is only assigned when `setKeyMatchingVersion()` is called.


LGTM modulo verbal comments.


Yep. One other advantage of having it take msg as a parameter is that it becomes a purer function. It no longer depends on any global state of Ironfist. Everything needed to understand the function is available in the function body. If the Lua API changes (say if we switch to LuaJIT or something), it will still work. Indeed, you can get a sense of everything it can do just by looking at the parameter list and not reading the code. This is the big reason why I recommend having it take msg as a parameter.


@ifed01 this will have the unintended consequence of users need to specify the correct num_shards param when trying to finish the job. I don't think this is a good interface. Maybe just handle the NULL dereference case at RGWOrphanSearch::init(), and make sure the num_shards are ignored?


yeah. It's a good point!


Seems like this one could've used an isnullorempty as well


> I have it as part of my local testing setup. Did you want me to post it somewhere?

Thats OK, as long as you later dont forget to nag us to somehow integrate this into the real repo, when the paper is done.


Which only happens when a conversation commits to a version. It means we have no idea which key will be used, until the conversation engages in an AKE. So we simply can't show this message.


shouldn't this be tablet as in the other place?

Maybe this layout styles should be in app.less applied to it's children so that they all share layout spacing

```less
.App {
  >* {
    margin: 0 16px;

    .larger-than-mobile({
      margin: 0 48px;
      max-width: 1000px;
    });

  // ...
```

That way when we add other pages (About, History, Watchlist, etc) they will all share the same layout.

What do you think?


Seems like this could be shortened quite a bit.  Maybe something like

```
petsc_use_debug=`cat ${PETSC_DIR}/{.,${PETSC_ARCH}}/include/petscconf.h 2>/dev/null | grep -c PETSC_USE_DEBUG`
```

If either file exists and contains PETSC_USE_DEBUG then we get a positive integer; else we get 0.  We never need a second test and we never get an empty string, so instead of 4 ```if``` blocks we still just need one.


Doesn't just using their username ping them? I must have missed testing that - I thought that was just formatting between bold (and in green) & italic (in grey). Nevermind then - more Zulip for me to learn!


@jmunhoz this breaks compatibility with older entries.


Woo, didn't know about [`accessKey`](https://developer.mozilla.org/en-US/docs/Web/API/Element/accessKey). Nice


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/4209/console
Build was first broken at http://ci.cgeo.org/job/c-geo/4202/


Your implementation reset the zone. Which means that the previous zone callback was still on the stack. This created incorrect behavior in the WTF trace since I would be missing the close messages from one test, and then have suprios extra close messages would show up on other tests.


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


That wasn't my concern, but rather `redux-puch`.


Can 1000 become some constant?


Ah yes. Good point.


I'm not sure we should be doing this assert. Potentially a platform may have multiple TZC-400 instances, which need to be programmed separately. There doesn't seem to be a good reason to prevent this. Can we remove this line?


see https://github.com/symfony/symfony/issues/15261 and https://github.com/symfony/symfony/pull/15290


If autosuggest is disabled, shouldn't [these](https://github.com/Khan/live-editor/blob/master/tests/tooltips/tooltips_test.js#L74-L91) tests be disabled for now?


Forgot to sync before editing.


parts of this file were taken from openMPI


I did not investigate it too deeply, but with a common setup of Mac OS X (and Xcode), with QT4 installed through "homebrew', running `qmake` does use this section of `linux-g++` even though it's running on a Mac. it is possible that `clang` that is aliased as `gcc/g++` emulates compatibility with `linux-g++` and thus this is triggered.


Ouch, didn't even notice that :P well spotted!


Hmm...  Guess we need to go back and introduce a code-style to the `1.x` branch.


Also nomenclature change in the build_exchange_df_flexible method: "quantity" is now called "value".


`DomainObjectObserverRegistry`


Nope, doesn't work.

The gcc-5.2 package has been updated so it doesn't spam warnings anymore though, so that's now an option.


Этот скрипт должен 1 раз выполнятся при переходе. Удаление Coords естественно повторно не нужно выполнять. Удали строки, попробуй снова запустить


this should be on line two, followed by an empty line


Thanks.

On 12/01/2015 11:22 AM, Erik Boman wrote:
> Yes.
>
> On 12/01/2015 11:20 AM, Siva Rajamanickam wrote:
> >
> > Checking if @trilinos/shylu
> > <https://github.com/orgs/trilinos/teams/shylu> works for comments on
> > commits. Can someone from shylu team who unwateched Trilinos repo
> > confirm if they see this message.
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/trilinos/Trilinos/commit/7ab207091e506893c4368b02d7e902d7ff9adac6#commitcomment-14716057>.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/trilinos/Trilinos/commit/7ab207091e506893c4368b02d7e902d7ff9adac6#commitcomment-14716108>.
>


feel free to add your name


is this c&p?


I suggest to change "and executing a linking algorithm" to "and executing an algorithm" -> not all algorithms that can be executed are directly related to linking (e.g. text extraction, removal of bibliographies, learning patterns...)


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#fLdj9okWRxWGjE3zcNNl3A


please add an error code for the exceptions, simply use a unix timestamp


@ndokos  I will check #112, seems then this fails out of this PR.


It probably makes more sense for this class to receive `authenticated_service` as a param to `__init__` and make it caller's responsibility to call `authenticate_with_google()`


For me to remember. This should be in alphabetical order.


to go nicely with `register()`, `isRegistered()` might be better


Congratulations on keeping word definitions short, the code looks really nice!
Also, where did you get the images ? Are you the copyright owner ?


phpDoc comment


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


Build 14 test results:  26 passed, 0 failed, 0 errored, 26 total Merge pull request #31 from buzzfeed/fix/example-project

Fix building of the example project


VFR file will never create the same pattern 3 times in a row, that's the point of a VFR file, there is no pattern, sequence  or anything, it's random. For exemple,  for a VFR file with a max at 29.97, you'll have the sequence patterns wich will have a random organisation of (at least, but there can be more frame  rate) the values  from 29.97 and 23.976, with a variable size (can easely  vary from 3 to 6).
My answer to your first question would be : why a 33333.33 was present if the file is a constant frame rate not at 30Hz ? Or if it's a VFR but without 30Hz inside ? I mean, maybe the fix is not  to be done here. If (i insist on the IF) wrong information is provided to the algorithm, the first way is to fix the sending of wrong information.


Should be `self._authenticated_service`


why inject the data mapper, but not the IndexQueue and ObserverRegistry?


is that needed by an interface? There's no interface declared for this class..


Can you additional negative tests cases. Example for date add year, quarter and month, select an interval which is large enough that it generates out of range exception.


What's the middle name :question:


When will the vendor blobs be officially available for the kanuti/tulip ?


Hi, can you detail a little what exactly does not work for you in setting up the staging builds?


this feels very redundant, any reason for this wrapper?


Adding additional text


@MartinOtter FYI: This is the writeMatrix function.


No, but I've given due credit in the docs for the snake-game vocab. The source was BSD licensed, so I guess this is okay? But, I'm not an expect on licenses.


asdf


Interestingly those tests passed: https://travis-ci.org/Khan/live-editor/builds/94219850.  Not sure what's going on.


:shipit: :v: :ghost:


Yes, still supported, but it was a nonsense to have it configured in a cache class. It's in WsClientProvider, which is responsible for configuring HTTP connection.


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


`$classToTableNameMap`


That looks like a Coryizum, He did that a lot and they were really hard to catch


Yep, solved for me as well. Thanks.


Will raise IndexError in case of empty string, like - "".


We shouldn't be creating a `BigQueryJobResultCollector` because this tightly couples `BigQueryCall` and `BigQueryJobResultCollector`. Instead, it should receive an initialized `BigQueryJobResultCollector` in the `__init__` and save it like `self._result_collector`.


i know you're just following the other code, but understand why I don't like this - I don't like that the state is modified in the render.


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


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/4210/console
Build was first broken at http://ci.cgeo.org/job/c-geo/4202/


Hi Jürgen,
Unfortunately, the second line kind of broke the black bar because it's too high for the design now. Maybe we could write the versions side by side?


"This API is under heavy development and far from stable" -> I prefer the wording on the home page which states that "the API is in rapid development. ". The API is not complete yet and may have some bugs but it does not seem to be far from stable to me.


use lower case for arguments and variables.


Ok "nag Bernd" has been added to my to-do list.


Thank you very much! :)
The tickets state, that container cloning is **not working anyways**, although I'm obviously doing this with great success. Something is wrong here...


The DMA idle counter should only be timing out when there are no users of DMA.

The definition of DMA being in use is that somebody has called dma_init but not yet called dma_deinit.
If you're initializing the DMA stuff yourself and NOT calling dma_init/dma_deinit then that would definitely cause a problem. I should have exposed it in dma.h - I think I just forgot.

dma_tickle resets the idle clock, which normally shouldn't be needed, since the dma_idle checker checks the dma_enable_mask. dma_init calls dma_enable_clock which sets the bit in dma_enable_mask for the DMA stream being used. dma_deinit calls dma_disable_clock which clears the bit in dma_enable_mask, and the dma idle timeout only really disables the clock when both dma_enable_mask == 0 and the timeout has occurred.


I'd rather avoid adding more and more to the Util class and instead put this into the DatabaseUtility.


This one definitely belongs into `DatabaseUtility`


Thanks!


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


I'm using `list()` here to explicitly make a copy since `self.delete_attachment()` could modify the blobs dict, which might cause a "dictionary changed size during iteration" error. In Python2 `dict.keys()` returns a list (a copy of the keys), but in Python 3 `dict.keys()` returns a view object and modification during iteration causes an error.


Make arguments final since you are here.


i can confirm the PCH issue as well. running Debian 8 "Jessie", gcc (Debian 4.9.2-10) 4.9.2, using the master branch of mangos-wotlk.


add your favorite color


In general it's a great start! It feels like there's a lot of duplicated code. I was thinking more about something like what Hans posted here: https://gist.github.com/hhoechtl/cd57e7b68e8fe6642165

Instead of duplicating code, I'd rather call the existing RecordMonitor and GarbageCollector or make this new code part of those existing classes and then register RecordMonitor and GarbageCollector as observers for Extras persistence.

The improvements to the RecordMonitor methods sound good.


see https://github.com/fex-team/yog2/issues/68


You should probably reference a local file (testscheme://localhost/.../biomuta.csv) so that tests don't break when BioMuta changes their output.


Seems to work great :+1:
Congrats!


What is this file about?


Odd english, maybe "Plex Update has successfully downloaded version ......" ?


Why the echo?


I think reverseLegend should be true for all of our graphs with a legend. The reversal is only useful when the graph gets to be narrow enough that the legend items are displayed vertically, but that can happen for any graph after resizing the window. Perhaps make this option true by default.


I wonder if there's a custom icon we can use.


In lots of cases it doesn't matter, but it's safer to put the mixin first, and the base class second. That is, list classes in the order you want them searched.


Check return code and do a nicer status, for example, "Notifying user...OK/Fail"


In Fedora we generally require *packaged* scripts to use /usr/bin/python2 or /usr/bin/python3. Exactly to avoid stuff in /usr/local or ~/bin. But since those tests are not installed, I guess that the my point is moot. It really doesn't matter.


@davedega My email is i.s.tarasov@gmail.com. We can arrange skype call later.


because we already keep reference to BitboardImpl in the MoveGenerator, lets do not pass opponentPieces to this funciton as we can get that info here also.

Same goes for generateKnightCapturingMoves and other function. It is okay if it needs to check color each time in these functions, because later we can use non-type arguments template to specify the color from the very top and have all the functions instantiated for both colors at compile time and avoid checking color each time like we do it now. But lets inroduce non-type arguments only when it is a real bottleneck, for now we have a lot to get there :)


Wow! Outstanding work.


... though I just looked at the ImageFetchingMixin, and there's no reason for it to be a class at all.  It can just be a helper function that is used by tests.


@rncbc It's Massimo by the way :laughing:


Consider adding another variable $svg after svg is defined since $(svg) is used several times.


Sorry, I removed them because they already were in `/app/src/main/res/`, so I thought there they are unnessesary.


Consider extracting 420 to a constant.


Just a note, in this case you could just return `{ isSearching: true }` since that's all the state, but if we add more properties to the reducer we would have to change all the `case`s to extend the existing state. Tradeoffs and all that... :thought_balloon: :+1:


@stevelinton The diffs below look like a side-effect of this commit (they appeared in 6pm test, but not after your morning commits). If the code is stable, I could update manual examples, otherwise I'd rather wait till it stabilises, if more changes are coming.
```
============================================================
########> Diff in [ "./../../lib/grp.gd", 3017, 3022 ]
# Input is:
NormalClosure(g,Subgroup(g,[(1,2,3)]));
# Expected output:
Group([ (1,2,3), (1,3,4) ])
# But found:
Group([ (1,2,3), (2,3,4) ])
########
########> Diff in [ "./../../lib/grp.gd", 1412, 1416 ]
# Input is:
DerivedSubgroup(g);
# Expected output:
Group([ (1,3,2), (1,4,3) ])
# But found:
Group([ (1,3,2), (2,4,3) ])
########
########> Diff in [ "./../../lib/grp.gd", 1194, 1197 ]
# Input is:
PerfectResiduum(Group((1,2,3,4,5),(1,2)));
# Expected output:
Group([ (1,3,2), (2,4,3), (1,3)(4,5) ])
# But found:
Group([ (1,3,2), (1,4,3), (1,5,4) ])
########
########> Diff in [ "./../../lib/grp.gd", 2071, 2074 ]
# Input is:
SupersolvableResiduum(g);
# Expected output:
Group([ (1,3)(2,4), (1,4)(2,3) ])
# But found:
Group([ (1,2)(3,4), (1,4)(2,3) ])
########
########> Diff in [ "./../../lib/grp.gd", 3186, 3190 ]
# Input is:
s:=SubnormalSeries(g,Group((1,2)(3,4)));
# Expected output:
[ Group([ (1,2,3,4), (1,2) ]), Group([ (1,2)(3,4), (1,3)(2,4) ]),
  Group([ (1,2)(3,4) ]) ]
# But found:
[ Group([ (1,2,3,4), (1,2) ]), Group([ (1,2)(3,4), (1,4)(2,3) ]),
  Group([ (1,2)(3,4) ]) ]
########
########> Diff in [ "./../../lib/grp.gd", 1222, 1233 ]
# Input is:
r:=RepresentativesPerfectSubgroups(m11);
# Expected output:
[ Group([ (3,9,11)(4,8,6)(5,10,7), (1,3)(2,11)(4,6)(7,8) ]),
  Group([ (2,3,4)(5,6,8)(7,11,9), (1,3)(2,11)(4,6)(7,8) ]),
  Group([ (4,11,10,9)(5,7,8,6), (1,6,4)(2,9,7)(8,11,10) ]),
  Group([ (3,5,8)(4,11,7)(6,9,10), (1,3)(2,11)(4,6)(7,8) ]), M(11),
  Group(()) ]
# But found:
[ Group([ (3,5,8)(4,11,7)(6,9,10), (1,5)(2,3)(6,9)(10,11) ]),
  Group([ (2,4,6)(3,10,7)(5,9,11), (1,5)(2,3)(6,9)(10,11) ]),
  Group([ (3,6,4,9)(5,11,7,8), (1,9,11)(4,7,8)(5,6,10) ]),
  Group([ (3,4,10)(5,11,6)(7,9,8), (1,5)(2,3)(6,9)(10,11) ]), M(11),
  Group(()) ]
########
########> Diff in [ "./../../lib/grp.gd", 1253, 1263 ]
# Input is:
ConjugacyClassesPerfectSubgroups(m11);
# Expected output:
[ Group( [ ( 3, 9,11)( 4, 8, 6)( 5,10, 7),
      ( 1, 3)( 2,11)( 4, 6)( 7, 8) ] )^G,
  Group( [ ( 2, 3, 4)( 5, 6, 8)( 7,11, 9),
      ( 1, 3)( 2,11)( 4, 6)( 7, 8) ] )^G,
  Group( [ ( 4,11,10, 9)( 5, 7, 8, 6), ( 1, 6, 4)( 2, 9, 7)( 8,11,10)
     ] )^G,
  Group( [ ( 3, 5, 8)( 4,11, 7)( 6, 9,10),
      ( 1, 3)( 2,11)( 4, 6)( 7, 8) ] )^G, M(11)^G, Group( () )^G ]
# But found:
[ Group( [ ( 3, 5, 8)( 4,11, 7)( 6, 9,10),
      ( 1, 5)( 2, 3)( 6, 9)(10,11) ] )^G,
  Group( [ ( 2, 4, 6)( 3,10, 7)( 5, 9,11),
      ( 1, 5)( 2, 3)( 6, 9)(10,11) ] )^G,
  Group( [ ( 3, 6, 4, 9)( 5,11, 7, 8), ( 1, 9,11)( 4, 7, 8)( 5, 6,10)
     ] )^G,
  Group( [ ( 3, 4,10)( 5,11, 6)( 7, 9, 8),
      ( 1, 5)( 2, 3)( 6, 9)(10,11) ] )^G, M(11)^G, Group( () )^G ]
########
########> Diff in [ "./../../lib/ctbl.gd", 4363, 4389 ]
# Input is:
kernel:= KernelOfCharacter( irr[3] );
# Expected output:
Group([ (1,2)(3,4), (1,4)(2,3) ])
# But found:
Group([ (1,2)(3,4), (1,3)(2,4) ])
########
########> Diff in [ "./../../lib/ctbl.gd", 4363, 4389 ]
# Input is:
NormalSubgroupClassesInfo( tbl );
# Expected output:
rec( nsg := [ Group([ (1,2)(3,4), (1,4)(2,3) ]) ],
  nsgclasses := [ [ 1, 3 ] ], nsgfactors := [  ] )
# But found:
rec( nsg := [ Group([ (1,2)(3,4), (1,3)(2,4) ]) ],
  nsgclasses := [ [ 1, 3 ] ], nsgfactors := [  ] )
########
########> Diff in [ "./../../lib/ctbl.gd", 4363, 4389 ]
# Input is:
NormalSubgroupClassesInfo( tbl );
# Expected output:
rec( nsg := [ Group([ (1,2)(3,4), (1,4)(2,3) ]) ],
  nsgclasses := [ [ 1, 3 ] ], nsgfactors := [ Group([ f1, f2 ]) ] )
# But found:
rec( nsg := [ Group([ (1,2)(3,4), (1,3)(2,4) ]) ],
  nsgclasses := [ [ 1, 3 ] ], nsgfactors := [ Group([ f1, f2 ]) ] )
########
########> Diff in [ "./../../lib/ctblfuns.gd", 545, 569 ]
# Input is:
lin:= LinearCharacters( DerivedSubgroup( g ) );
# Expected output:
[ Character( CharacterTable( Alt( [ 1 .. 4 ] ) ), [ 1, 1, 1, 1 ] ),
  Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
    [ 1, 1, E(3)^2, E(3) ] ),
  Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
    [ 1, 1, E(3), E(3)^2 ] ) ]
# But found:
[ Character( CharacterTable( Alt( [ 1 .. 4 ] ) ), [ 1, 1, 1, 1 ] ),
  Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
  [ 1, 1, E(3), E(3)^2 ] ),
  Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
  [ 1, 1, E(3)^2, E(3) ] ) ]
########
########> Diff in [ "./../../lib/ctblfuns.gd", 545, 569 ]
# Input is:
List( lin, chi -> chi ^ (1,2) );
# Expected output:
[ Character( CharacterTable( Alt( [ 1 .. 4 ] ) ), [ 1, 1, 1, 1 ] ),
  Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
    [ 1, 1, E(3), E(3)^2 ] ),
  Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
    [ 1, 1, E(3)^2, E(3) ] ) ]
# But found:
[ Character( CharacterTable( Alt( [ 1 .. 4 ] ) ), [ 1, 1, 1, 1 ] ),
  Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
  [ 1, 1, E(3)^2, E(3) ] ),
  Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
  [ 1, 1, E(3), E(3)^2 ] ) ]
########
########> Diff in [ "./../../lib/ctblfuns.gd", 545, 569 ]
# Input is:
Orbit( GaloisGroup( CF(3) ), lin[2] );
# Expected output:
[ Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
    [ 1, 1, E(3)^2, E(3) ] ),
  Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
    [ 1, 1, E(3), E(3)^2 ] ) ]
# But found:
[ Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
  [ 1, 1, E(3), E(3)^2 ] ),
  Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
  [ 1, 1, E(3)^2, E(3) ] ) ]
########
########> Diff in [ "./../../lib/ctblfuns.gd", 545, 569 ]
# Input is:
(1,2,3)^lin[2];
# Expected output:
E(3)^2
# But found:
E(3)
########
########> Diff in [ "./../../lib/ctblfuns.gd", 612, 622 ]
# Input is:
ComplexConjugate( lin[2] );
# Expected output:
Character( CharacterTable( Alt( [ 1 .. 4 ] ) ), [ 1, 1, E(3), E(3)^2
 ] )
# But found:
Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
 [ 1, 1, E(3)^2, E(3) ] )
########
########> Diff in [ "./../../lib/ctblfuns.gd", 612, 622 ]
# Input is:
GaloisCyc( lin[2], 5 );
# Expected output:
Character( CharacterTable( Alt( [ 1 .. 4 ] ) ), [ 1, 1, E(3), E(3)^2
 ] )
# But found:
Character( CharacterTable( Alt( [ 1 .. 4 ] ) ),
 [ 1, 1, E(3)^2, E(3) ] )
########
########> Diff in [ "./../../lib/ctblfuns.gd", 612, 622 ]
# Input is:
Permuted( lin[2], (2,3,4) );
# Expected output:
ClassFunction( CharacterTable( Alt( [ 1 .. 4 ] ) ),
[ 1, E(3), 1, E(3)^2 ] )
# But found:
ClassFunction( CharacterTable( Alt( [ 1 .. 4 ] ) ),
 [ 1, E(3)^2, 1, E(3) ] )
########
########> Diff in [ "./../../lib/ctblfuns.gd", 1501, 1505 ]
# Input is:
List( Irr( S4 ), KernelOfCharacter );
# Expected output:
[ Alt( [ 1 .. 4 ] ), Group(()), Group([ (1,2)(3,4), (1,3)(2,4) ]),
  Group(()), Group([ (), (1,2), (1,2)(3,4), (1,2,3), (1,2,3,4) ]) ]
# But found:
[ Alt( [ 1 .. 4 ] ), Group(()), Group([ (1,2)(3,4), (1,4)(2,3) ]),
  Group(()), Group([ (), (1,2), (1,2)(3,4), (1,2,3), (1,2,3,4) ]) ]
########
########> Diff in [ "./../../lib/ctblfuns.gd", 1345, 1350 ]
# Input is:
List( Irr( S4 ), CentreOfCharacter );
# Expected output:
[ Group([ (), (1,2), (1,2)(3,4), (1,2,3), (1,2,3,4) ]), Group(()),
  Group([ (1,2)(3,4), (1,3)(2,4) ]), Group(()),
  Group([ (), (1,2), (1,2)(3,4), (1,2,3), (1,2,3,4) ]) ]
# But found:
[ Group([ (), (1,2), (1,2)(3,4), (1,2,3), (1,2,3,4) ]), Group(()),
  Group([ (1,2)(3,4), (1,4)(2,3) ]), Group(()), Group([ (), (1,2), (1,
   2)(3,4), (1,2,3), (1,2,3,4) ]) ]
########
########> Diff in [ "./../../lib/ctblmono.gd", 265, 282 ]
# Input is:
TestHomogeneous( chi, n );
# Expected output:
rec( character := Character( CharacterTable( Group(
    [ [ [ 0*Z(3), Z(3) ], [ Z(3)^0, 0*Z(3) ] ],
      [ [ Z(3), 0*Z(3) ], [ 0*Z(3), Z(3) ] ],
      [ [ Z(3)^0, Z(3) ], [ Z(3), Z(3) ] ] ]) ), [ 1, -1, 1, -1, 1 ] )
    , comment := "restriction checked", isHomogeneous := false,
  multiplicity := 1 )
# But found:
rec( character := Character( CharacterTable( Group(
    [ [ [ 0*Z(3), Z(3) ], [ Z(3)^0, 0*Z(3) ] ],
      [ [ Z(3), 0*Z(3) ], [ 0*Z(3), Z(3) ] ],
      [ [ Z(3), Z(3)^0 ], [ Z(3)^0, Z(3)^0 ] ] ]) ),
  [ 1, -1, 1, -1, 1 ] ), comment := "restriction checked",
  isHomogeneous := false, multiplicity := 1 )
########
########> Diff in [ "./../../lib/ctblmono.gd", 345, 358 ]
# Input is:
TestQuasiPrimitive( chi );
# Expected output:
rec( character := Character( CharacterTable( Group(
    [ [ [ 0*Z(3), Z(3) ], [ Z(3)^0, 0*Z(3) ] ],
      [ [ Z(3), 0*Z(3) ], [ 0*Z(3), Z(3) ] ],
      [ [ Z(3)^0, Z(3) ], [ Z(3), Z(3) ] ] ]) ), [ 1, -1, 1, -1, 1 ] )
    , comment := "restriction checked", isQuasiPrimitive := false )
# But found:
rec( character := Character( CharacterTable( Group(
    [ [ [ 0*Z(3), Z(3) ], [ Z(3)^0, 0*Z(3) ] ],
      [ [ Z(3), 0*Z(3) ], [ 0*Z(3), Z(3) ] ],
      [ [ Z(3), Z(3)^0 ], [ Z(3)^0, Z(3)^0 ] ] ]) ),
  [ 1, -1, 1, -1, 1 ] ), comment := "restriction checked",
  isQuasiPrimitive := false )
########
=========OUTPUT END: testmanuals with autoloaded packages=========
```


other than comments, code looks good!


There's a space after Visual Basic. Is that a problem?


This looks good - when it comes to CSS time - we should assign classes to the different parts of the article, so we can style independently.


Yes, I will add a better readme/changelog


Hm, why does it make sense? As the docs say, reactive var invalidates an autorun whenever its value changes.
Did you try btw to place search property in that example before the function with Parties? It seems  `value()` [here](https://github.com/Urigo/angular-meteor/blob/6998c0d3d76c0c72db6ec67de07426b6b0cd0822/packages/angular-meteor-data/modules/angular-meteor-reactive-context.js#L75) might be executed when reactive variable is not yet defined?


You guys really do not understand what you are doing.
The rule is look at it, understand it, do not break it until you understand what it is doing, if needed make some fixes!

Anyhow i just stumbled here from the IoT and though it would help someone that has no guidance a word of sit down and ask requirements the code was written for :)


You should rename "icone" to "icon" and modify the relevant parts of your doc/code.


Awesome idea, @danielglauser - hopefully we'll never have to flail around figuring out what port's open again. LGTM.


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


Why not use UserFactory?


Is this the correct definition of the derivative of abs2?
This overwrites the definition used in Calculus.jl, and gives the wrong derivative
```julia
differentiate("abs2(x)", [:x])
1-element Array{Any,1}:
 :(if x > 0
        1
    else
        -1
    end)
```


@smgom can you remove "e" from order description? :)


When I tried a similar change to AllegroInputStream (by converting the handle type from int to jlong) I found that Allegro examples crashed on my phone (LG G3). This commit should be well tested on all the various platforms (arm 32/64, and x86 32/64) to ensure it isn't subtley broken.

I had previously sent an email to the Allegro5 list about this issue on August 18, 2015.


well the versions in res are .png, and the images in assets are svg.


can we move it to the beginning or even in separate file like version.php?


@smgom what if there is no shipping?


Should this also parse vbproj files?


I've mentioned that secure params will be accessible in a build, hopefully it makes report more clear.


There should not be platform variation in this file. If you need different mapping settings in different platforms, then this file should provide both V2M_MAP_FLASH_0_RW and V2M_MAP_FLASH0_RO mappings, which are selected by the platform.

However, in this case I don't think there needs to be variation. There is no harm mapping Flash 0 as RW in *all* ARM standard platforms. So just remove the build flag variation and use the RW definition.


please make it localized and at the key to openwapp.en-US.properties


Just as a side note.  It is a little tricky to then use the Union types.. as the following does not compile
```scala
RTCIceServer(urls = "turn:turn.conversant.im:443" )
```

And instead you have to define like so:
```scala
val iceUrls: String | js.Array[String] = "turn:turn.conversant.im:443"
RTCIceServer(urls = iceUrls )
```
Perhaps we provide nicer ways to build in the factory object?


I still see different capitalization styles throughout the TOC. See how you capitalized some of the words inside the titles too and some you didn't?
For example: Working with NuGet Packages, Publishing NuGet Packages, etc. are capitalized. Writing console applications, Overview and moving pieces, etc. have only the first word capitalized. Maybe I'm super picky and you don't care. :relaxed:


We should return this promise so that redux waits until the whole chain is completed for triggering the changes to the subscribers. See http://rackt.org/redux/docs/advanced/AsyncActions.html#async-action-creators for the examples (it always returns the fetch promise to the redux thunk)


launchSMBXIGuess()


those pattern can occur when some 23.976 content was telecined and recorded from tv. absolutely valid.

> If (i insist on the IF) wrong information is provided to the algorithm, the first way is to fix the sending of wrong information.

that is no option because you can' t do this for live streams. you remember our discussion on the pr? finally you persuaded me and now we have the second false positive case :)


Arquivos coletados no moodle(ufmg) e atualizados através do computador da empresa


All the rest LGTM! :+1:


Setting RESIN_STAGING_BUILD = "yes" either in local.conf or even in the
resin.inc file never adds "resin-staging" to the list of DISTRO_FEATURES so
it always disables the serial.getty in systemd. While the logic of using
@bb.utils.contains is sound according to the yocto documentation, it
appears to never evaluate to "true" regardless of where and how
RESIN_STAGING_BUILD = "yes" is set. I spent a lot of time testing and using
this yesterday, and it would never work. It would only work when
resin-staging was explicitly assigned to DISTRO_FEATURES directly or via
the _append method.

On Tue, Dec 1, 2015 at 10:51 AM, Jonth4 <notifications@github.com> wrote:

> Hi, can you detail a little what exactly does not work for you in setting
> up the staging builds?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/resin-io/meta-resin/commit/cb22fd3a41e44997d9773f7f71105bab780734c1#commitcomment-14716818>
> .
>

--
Donovan Sydow
elarm, Inc.
Director of Engineering
donovan@elarm.com
206.229.8740


This isn't returned implicitly because we are using `=> { }`, in this case it doesn't matter much because it is a synchronous action, but if the stopSearchRequest was a async action then we should return the promise that dispatch returns here.


Seems right :+1:


`{foo,bar}` is a bash-ism, right?  I never know if we are allowed to assume bash when writing m4 tests, I don't doubt we have done it already though...


Suggest rewriting to:

```
except (HttpError, ResponseNotReady) as as e:
  raise BigQueryCommunicationError('Query failed with %s:\n %s', e.resp, query_string)
except Exception as e:
  raise BigQueryCommunicationError('Query failed with %s:\n %s', e.message, query_string)
```


Y este salto de linea?


Pisanie kodu, żeby ułatwić sobie testy nie jest za dobrym pomysłem


Napaka pri popravljanju


@thatoddmailbox It doesnt work.


Amoniak


How come it can work for you? Run `npm link` and you will get `getPackage` is not a function or something. See the method below.


Rebuilding from scrach did it. Thanks


He estado mirando tu makefile, tienes repetida esta línea: KBUILD_CFLAGS += $(call cc-option, -fno-var-tracking-assignments), correspondiente a las líneas 607,609;)


Ah, yes, sorry, I meant just the `MediaStream`.  (I'm a bit distracted at the moment so apologies for the delays here).  It's a bit annoying that we have that in `lib`, since it's not in IE or Safari, and thus really shouldn't be there.

Okay. For the purposes of proceeding, I would create an `experimental.mediastream` and put your `MediaStreamTrack` in there, and anything else you need to add from the media capture and streams api.  Leave the `MediaStream` class in place for now, filling it out according to whichever spec you pick.

It may well be the case that there are other classes that are incorrectly outside of `experimental`.  I will do a full audit of these tomorrow and I'll discuss with the other maintainers what the best course of action is- but we'll address that in a separate PR.  `0.9.0` is a major version release so we probably have a  bit of wiggle room to tidy up, and that shouldn't be your problem :).


getPackage throws an error, based on this error you decide what to do: if it's critical for your plugin you terminate a process, otherwise, you may continue


This is a bug. The abs2 rule shouldn't be there.

Thanks for pointing out.

Feng

On Tue, Dec 1, 2015 at 1:11 PM, Asbjørn Nilsen Riseth <
notifications@github.com> wrote:

> Is this the correct definition of the derivative of abs2?
> This overwrites the definition used in Calculus.jl, and gives the wrong
> derivative
>
> differentiate("abs2(x)", [:x])1-element Array{Any,1}:
>  :(if x > 0
>         1
>     else
>         -1
>     end)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fqiang/ReverseDiffTape.jl/commit/0188788a2ab311e6103ac4d9c2469f4fbf21e0bf#commitcomment-14717299>
> .
>


Does this mean that FCS will now require Mono 4.2.x and above?  Thanks


Where is this variable used?


It would help to have style changes in a separate commit.


Same question - seems unused.


Are you okay defining this for ``Dual{Complex}`` as well?


Yes, there can be more contextPair instances than threads. Basically number of contextPair instances created will be the same as maximum number of concurrent inflight requests that the service can handle. Since netty is IO event based, there can be more inflight HTTP request than number of active threads, especially for the case when client is sending large body or the service is streaming chunks of response body (yet to be implemented).

Currently the queue size is not limited. The idea behind is this:

1.If there is a constant load to the service, meaning a relatively constant number of inflight requests are being handled by the server, then it's beneficial keeping all the context pair created to reduce overhead in creating and destroying user handler class (which is contained inside the context pair).
2. If the traffic is bursty, we still need to create that amount of context pair instances to handle the burst. When the traffic gets quiet down, there is a chance that some of the threads will be idle long enough and terminated by the executor. When that happen, the context pair instances stored inside the queue that ties to the idle thread will get removed from the cache (either by GC or by the periodic cleanup logic). If we are unlucky such that no thread is idle long enough, we'll be retaining all context pair instances, but they will get used when the next burst comes in.


Not sure what `?` means: you have exactly the same behavior with `require`. Only one difference from require here is that you're adding specific path to require


Have you tried this python on prefix? I have some problems here. Prefix does have ver `2.10.0` which seems to build properly but not this version?


Check the L64 please, you are going to receive `undefined is not a function`. That's not even the error from require.


The "isPaused" bug is a missing line in a typescript development module, but since the master branch doesn't include all the build stuff, I can't update it with just commits.  Pull the "phaser.comments.d.ts" and "phaser.d.ts" files from components/phaser/typescript directory of the gh-pages branch if you want to be able to run that functionality locally.

I suppose we should keep that line commented out in the master branch since that's a bug in how grunt/bower builds the dependencies (maybe the repository they're pulling from?), but it runs fine in the web-hosted app.


And I should just shut up now that i've read a few commits ahead.


Interesting concept. However, I think it's going to send the message to the owner of the *planet*, not the ship.


as a solution can we add pattern(-1) != pattern to vfr detection?
so at least 3 losses and two different patterns?


Need to call internal abort macro.


Wouldn't it be better to get this from the JDK's System properties rather than duplicating it in the edda properties file?


Could you add a line of documentation for the NewUsernameForm class ? (like the other class in the file)


@megatronir
I think this is just the standard evade function that it's used for all creatures.
I don't think it will make a significant impact on the script (unless you have some custom definition in the addon).


Nope, everything works


We are using weakKeys() here, so the Thread can be GC'ed when it is completed, but the value is still strongly reference by the cache, hence we should be able to get it back and calling close for all entries inside the queue. There is a unit-test covering that.


I'm wondering if this is a good opportunity to remove this definition. I don't see how this could be useful and I'm not sure its meaningful to have this definition. For example, `abs2(x::Union(Complex{Real},Quaternion{Real}))=real(x*conj(x))` It's also true that abs2(x::Dual) = real(x*conj(x) ) for the definition of `real` we agreed on and the definition of `conj` here (with the exception of x=0+\eps, but that could be changed https://github.com/JuliaDiff/DualNumbers.jl/issues/14)

I found only one reference to a `conjdual` like function in a thesis (http://mediatum.ub.tum.de/doc/1273706/195643554083.pdf),but it doesn't seem to be actually used. I haven't found mentions of `dual conjugation` anywhere else.


This should be moved to an experimental package


As with this


Can this be submitted upstream?


The region should not be changed.


Thanks; that helps because I was simultaneously trying to have up to date and out of date development versions of Bokeh, and now I can just use the up to date ones.


Keep this as is for now- we'll clean up later once we've assessed the potential impact.


`rnpm link`.


The changes in this commit strip off the /dev path from the tty device name, before passing it to the BusyBox getty program.

When trying to get finit to open a login prompt on a BusyBox system, I was not able to get a login prompt to appear. Booted up with "finit_debug" in the kernel cmd line, and noticed that finit was producing the following messages:

finit:tty.c:tty_start() - Starting TTY: /sbin/getty on /dev/ttyO2
Jan  1 02:03:16 overo authpriv.err getty[699]: /dev//dev/ttyO2: No such file or directory

It looks like the BusyBox getty program won't work with tty devices prefixed with the /dev path. After the changes, my system shows a login prompt successfully.


This should also be moved to an experimental package.


you're forcing it unnecessarily to do O(n) even when you're not on a debug build.


We are currently using an indentation of two spaces for the html files throughout the project.  This seems to be using tabs. Could you change indentation for the these lines in the file ?


Should we clear `q` and `results` when starting a new search session? Otherwise old query and results will show up?

I'll get to testing soon and verify if my suspicions are true.


Might want to add to this comment that the BadgeClassFactory created a badge with the test_slug, so we get back the original factory-made class, and will be checking that.


Ah, I think I've got it. The problem was the * after "target/". I have fixed it and added some more ignores.


These two additions should not be exposed here, I think, since they are experimental apis.


This shouldn't be committed.


I have no idea how that happened.  Reverting.


Can you try a "bitbake -e > build.log" in a build directory for which you have added RESIN_STAGING_BUILD ?= "yes" and then look in the output log for the "DISTRO_FEATURES=" string to see how it gets assigned?
On my side things look as expected so I am curious about what is happening on your side


Same here, it would be nice if you could correct the indentation to make it 2 spaces.


Sure!


Nice!

http://media4.giphy.com/media/TEFplLVRDMWBi/giphy.gif


**Done**, minus logging the `e.stack` part. Obvs.


The execute does throw checked exception, but of course it can throw unchecked one (e.g. illegal argument exception), although with the current implementation, it doesn't. It does feel safer to do the release in finally. Will add that.


Same as above.


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


My rational for having the media capture api in it's own file, is so we don't tangle two separate specifications


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/4211/console
Build was first broken at http://ci.cgeo.org/job/c-geo/4202/


I think you're right.


OK!
But is it possible to use svg instead of png when possible? Then you won't be needing to generate tots of images for every pixel density. I thought about it when noticed that in some apps I had blurred action bar icons. (Because I have 1.5 ration on my phone)
Also, on web it's also an issue since browser renders page as if the screen resolution was 1.5 smaller.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7491](https://teamcity.mesosphere.io/viewLog.html?buildId=83811&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 205 Build time: 00:04:45
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Got it working. Seems to work fine for me on Android 5.0. Needs more testing first before it's ready though.


This seems to be duplicating the same caching we have in NonCloseableHiveClientWithCaching. Can we refactor these into one class and use the class instances?


Yep.  Makes sense to me :+1:


+9000 LGTM


@nywillb It is for the background of events that you check off.


any reason for the name change? just curious


This is probably going to be a PageList component that we're going to reuse in lots of places, like in MF.

No need to extract it now, just a note. Maybe add a TODO?


Note, valbooth.com will expire on Oct 29, 2016. I'll try to be better about updating people at that time.


As long as i know svg only works on android 5.0 and higher.


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


pret pour css


This `li` is going to need a `key` attribute so that react knows when it get's new results which nodes it needs to blow up or move (and be more efficient).

I think `key='${result.title}` should be enough.

Source: https://facebook.github.io/react/docs/multiple-components.html#dynamic-children


to begin a game after one has started requires you to exit the program first


Oops, should've been expat_opts


I was testing different termination strengths to see if it would clean up LCD artefacts.  I had to change the LCD timing in the kernel instead.


Updating the RAM configuration for our board.


hi. schickst mir mal bitte deine email Adresse an info@rexponsive.de. danke. chris


Also, only teamcity system properties and env. variables are vulnerable.
Config params are not passed to agent, are they?
1 дек. 2015 г. 22:13 пользователь "nd" <notifications@github.com> написал:

> I've mentioned that secure params will be accessible in a build, hopefully
> it makes report more clear.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JetBrains/commit-status-publisher/commit/826262d6abebafb9ca625527fe36182f9bd273fc#commitcomment-14717356>
> .
>


I don't exactly see why Transformers and TransfomerFactory are registered as containers. In the example at https://github.com/fusepoolP3/overall-architecture/blob/master/transformer-registry-api.md#example the registration is not a container. If they should be containers then this patch should be applied, otherwise the respective type should be removed. Was the problem justs using LDPRs that are not LDPCs?


Why?


Since ChangeUserTests and ChangePasswordTests contain user registration/account related tests, could you move these to approval_frame/tests.py (which currently contain other user login tests)? Also, remove newline characters from 441-471.


Whoops. I have the LCD initialization turned off because I was trying to get it to work in the kernel. Next commit will fix this.


Ah, indentation is a bit confusing. It is 2 spaces for com.google package and 4 spaces for everything else. I have just added an .editorconfig file reflecting that fact (thought I have no idea if this is supported by Eclipse).

Yes, feel please create a pull request for that. I will try to create tests afterwards.


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


This is very copy pasta. At the very least we should add a mixin to share with the main app layout.

I think the div `<div className='Overlay'>` will end up being an immediate descendant of `.App` and the layout css will be applied to it (if we do it as I commented waay before). That way this wouldn't be necessary here.


Please remove this line.


Я пробовала сначала через last-child(для border)  и first-child для margin-left, но что-то не заработало...
Пришлось воспользоваться "костылями". Переделать?


Please a standard licence header (copy it from another file).


Stupid to post login details. They are changed now.


shit, did I forget to build ACC with the XP toolset?


Need to update the UART pinmux so we can see the terminal output.


If installed Copr packages are still not updated, it would be better to bump up the release number with each new build and completely drop Non Numeric Release naming.


Please remove this doc.


Ostia!! Pues verdad!!! Gracias!! Me la liaria el github al hacer el merge...tengo más duplicadas. Que raro que no me diera ningún warning. Jajajaja. Gracias de nuevo. Y en serio, no te molestes con el zzmoove, no va a tirar, si quieres probar prueba, pero es más para los snapdragon más actuales ;)


Sorry, no more patches regarding confirm_action are expected to be coming from me.
Scrolls/potions can be inscribed with !r/!q and other items like crystal ball of energy, sack of spiders, decks, wands, rods etc. are relatively safe to be used IMHO.


Ah I missed the credit in the docs.
From the website "This article, along with any associated source code and files, is licensed under The BSD License" so the images are also BSD licensed which means it's all good.


I found another issue. Also IsInUse() could create problems if it is not implemented in further add-on classes.

But that should not be handled in this PR.


```STALL_THRESHOLD``` is needed. It is used because we can't run the motor controllers/motors at lower speeds. (so we avoid running them at less than 80/255)

```FORWARD```/```BACKWARD``` are just constants that he defined to clean up the code. ```#DEFINE``` just replaces every instance of ```FORWARD``` with ```LOW``` (in the compiler). So as long as ```FORWARD``` and ```BACKWARD``` are removed from the code everywhere else, this should be fine. (Though I'm not sure of the advantage of removing them)


I am confused with this change. We used to not dispatch the call through the payment pool at this level. The dispatching was happening in [PaymentGatewayProcessor](https://github.com/killbill/killbill/blob/master/payment/src/main/java/org/killbill/billing/payment/core/PaymentGatewayProcessor.java#L80). So now we end up with two threads, which (maybe?) explains the behavior we were discussing this morning/


NIT: no need to indent here


Please see other com.google interfaces for doc "style". This interface is not bound to REST API. I will create a change for official Gerrit API later based on this interface additions.


Nitpicking here, but could you perhaps use more generic names like 'user1ces' and 'user1ces@gmail.com'  instead of 'aaron', 'rupal' and 'matt' (So that they doesn't suggest any particular user) :smiley: ?


thanks


@Trass3r updated to VS15 update one and it just started working (syntax highlighting)...
Though FMA instructions are wrong (they are all numbered based on the args positions in formula like `vfmadd231ps`) and not all AVX2 instructions are included. (Also AVX/ sse with vex (v) prefix are ignored too) I can probably fix that though.


Gerrit uses this style of code blocks: {@code ...}.


@mogoodrich yes. When would you like to do the release?


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


Too bad, but at least they did it in 5.0. Naturally, vector graphics are the best instrument for drawing UI, especially if it's flat(/material).


I found somewhere on internet that it works on older Android versions, but
you have to place it in assets instead of in drawable...
Op 1 dec. 2015 20:31 schreef "Christian Schabesberger" <
notifications@github.com>:

> As long as i know svg only works on android 5.0 and higher.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/theScrabi/NewPipe/commit/bd76a12b90850a09bd465fbccc7e289645b23a02#commitcomment-14717814>
> .
>


You are right... :)
Fixing it right now.. our solution is to sort the helpers (first run the non-function and then the functions, so the reactive component will created before try to use it..)

Thanks a lot!


Why are there imports not only in the middle of files, but in the middle of functions?


Me fio, no te preocupes, de hecho el autor lleva un kernel del one plus 2 que tiene el snapdragon 820 y no lo incluyó, lo que me extrañó bastante.

Y nada, para eso estamos, cualquier cosilla si veo algo te digo


I think 8890 should not be exposed to prevent any access that doesn't go via the proxy. Not sure what port 1111 is for.


FewDocFixes (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1975 of 1975 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)


Interesting. Yes the latter 2 points are known.


Listing some addressbooks where the Account ID is hard coded.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7494](https://teamcity.mesosphere.io/viewLog.html?buildId=83835&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:38


If this only works on newer browsers, I guess the most cross browser way of dealing with this would be using a lifecycle method, something like this

http://jsbin.com/kuruyu/2/edit?html,output

BTW please don't use a class, use a React.createClass if we can't keep this as a stateless component.


Can confirm that this fixes #6612 as well as handling `NULL` passed to `%d` :+1:

Thanks @JoelMarcey!


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 75](https://teamcity.labkey.org/viewLog.html?buildId=288678&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:42


imho passt hier der Singular besser


This docstring could use more details about what the arguments mean.


LGTM


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7496](https://teamcity.mesosphere.io/viewLog.html?buildId=83846&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:11


true


Why?


@sumitkmr558 Send me your template.


Classic async

![](http://media2.giphy.com/media/120YEqUcr3qPde/giphy.gif)


Your "innocent" change in the release-firmware.sh script broke compilation for all Taranis other than X9E:
```
# make lua fields for X9E which has most fields
make lua_exports.inc PCB=TARANIS PCBREV=REV9E
```

Running that before the
```
tar czf ./opentx.tgz opentx/radio/src opentx/radio/util
```
caused that X9E lua_exports.inc to be included in the tar, and thus be used for all compilations (it is not regenerated if already present), with symbols unknown on non-9E.

I moved it after the tar is generated and all is now good. Saving one 'cd' sounded good - but the 2 hours of debugging it caused didn't so much.

When do we add the scripts to the repo again? ;)


Is an "await" missing here?


108954406-ImproveWarningWhenCoilHeatingWaterSizesToZero (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1440 of 1440 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-70.5%25-green.svg)


not a huge fan of this parsing approach, this is structured text we should be able to come up with a regex that validate the line is in the expected format.

We have this style in the previous mac address parsing code too and we ended up with all sorts of weird things that aren't macs, a regex to validate some fields match what is expected - like the speed field should be a number and the port has a predictable format, easy enough to validate a line is in the right format via those.

That makes it a bit more robust against small changes to the firmware output formats or prevent parsing error messages as if they are someting they are not etc.


Should merge in branch `server-api4` - created a new branch because there are changes in this branch that are not reconciled


GOOD RIDDANCE!


Hmm, interesting!  We're using Finit with BusyBox getty at work, and I've set it up for for my own embedded devboard as well, https://github.com/troglobit/troglos ... I wonder if there are some
inconsistencies to how I've documented things?

This works for me, https://github.com/troglobit/troglos/blob/master/initramfs/etc/finit.conf

    # Allow login on ttyUSB0, for systems with no dedicated console port
    tty [12345] /dev/ttyAMA0 115200 vt100
    tty [12345] /dev/ttyUSB0 115200 vt100
    console /dev/ttyAMA0

(I saw your email btw, using the GitHub tracker is better for others :)


The subject of discussion was "what happens in case of dropped frame ?", and my question was  "Is it possible that dropped frames create a lower value in the patterns ?", and the answer was "no".
Good idea. I suggest even the followig : the same way  there is a counter for the losses, add a counter for pattern(-1)!=pattern, reseted and handled the same way than the counter for losses, add  put its threshold global  variable at 2, and trig vfr detection if both counter are >= to their threshold value.


Note - this is a small commit. Probably should be merged into the rest of the server-api work


yay!


```if port.is_a?(Array)``` is a bit better


asserts are disabled in release builds afaik...


Thanks, i'll be happy to merge this but i can't seem to find out how. This isnt a pull request nor a fork, which kind of dark github functionnality are u using?


Да, переименовать.
![2015-12-01 22_44_17- docx - openoffice writer](https://cloud.githubusercontent.com/assets/4870890/11512049/5402b394-987d-11e5-9f25-2e888ea710af.jpg)


Looks you have a lot more mistakes :)
It will be possible to build & package this by using the commands used in the griffon-install file, main call there is "scons install"
When running that it shows all your deps are wrong, it needs gtk3 versions of all, webkitgtk3, gtksourceview3....
I can upload an old webkitgtk3 package for you (it as removed from the repos since the move to calamares as installer), it is a very long build.....so better try an existing first just to check griffon.


> We used to not dispatch the call through the payment pool at this level

Yes, that's issue https://github.com/killbill/killbill/issues/391: we want to make sure that the control plugin calls are wrapped (to handle timeouts).

> So now we end up with two threads, which (maybe?) explains the behavior we were discussing this morning?

Looking at it in more details, I was indeed wrong: when coming from the control state machine, [PaymentOperation](https://github.com/killbill/killbill/blob/master/payment/src/main/java/org/killbill/billing/payment/core/sm/payments/PaymentOperation.java#L70-L83) will skip the second dispatching (I got confused by the flag, it doesn't simply skip locking the account, it skips both locking the account **and** the dispatching).

So it looks like I need to do something similar here (I'll probably just introduce a similar flag in `PaymentGatewayProcessor`).


not final?


Do we have an idea of how large this course_group_settings will get?  It's a little awkward to have one setting that has to be iterated to look for a course key.


put a # in front of the btnsearch and it'll work


Are these all None because we expect the badge to have already been created?  If so, can't we omit them?


Thanks, let me add some tests to be sure, and I'll fix it.
I was reusing the other animation methods and I need to improve the comments
but I'm fixing the position as well.
Let's say, childCount=20, selectedAvatar=0, visibleChilds=11..19, getChildAt(0) == 0 or 11?


Thank you!

On 1 December 2015 at 16:39, Wu Lin <notifications@github.com> wrote:

> see
> #2932 <https://github.com/shogun-toolbox/shogun/pull/2932>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shogun-toolbox/shogun/commit/4191546749aa2ced4a4d0ae05cb0ea73ada6007c#commitcomment-14711777>
> .
>


also update the heyzap version, you've included 9.3.0


We cannot use the unsafe annotation here, since it does an explicit `type(str) is` check. We have immutable string subclasses specialized on certain strategies (to avoid the extra strategy field).


Although it appears that the $query and order by sections of these annotated queries were redundant, a max-date function relies on some of these queries returning in ascending order, as indicated by the "1" on the filters.  Have you tested to confirm that scheduled data updates in your MongoDB data stores are not incorrectly overwriting any documents, or creating duplicates?

Likewise, can you also confirm which issue this pull request solves, in particular, as referenced on Gitter?


Quotes should include the part in `[...]`.


This is a similar quoting issue.


We are trying to follow [PEP 8](https://www.python.org/dev/peps/pep-0008/)  for most of our python code. I shall soon update the README regarding it.
According to E231: All commas should be followed by a space (similar to writing English, makes the code more readable)


@legoboyvdlp
Now that's a clean job :+1:
Keep up the nice splashes coming :package:

IH-COL


@Razzee
> There are two more places in this class where this might be nice to add.
> https://github.com/AchimTuran/xbmc/blob/CDatabase-return-value-fix/xbmc/dbwrappers/Database.cpp#L204
> https://github.com/AchimTuran/xbmc/blob/CDatabase-return-value-fix/xbmc/dbwrappers/Database.cpp#L550

I fixed the other places too. But I'm not sure about the fixes from 521 - 566. Would be nice if someone could review it.


final classes? Perhaps put in a namespace to keep them together? (since they do not share common supertype)


:+1: though its not clear from the context that thats the intention and someone is likely to come along and change it not knowing the need for a copy.


After enabling the wrappers on Windows, the delegates for the actual methods will no longer be needed. I'm removing them. Also removing the dummy unmanaged "this" pointer in the actual methods.


igraph source file for unix


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7495](https://teamcity.mesosphere.io/viewLog.html?buildId=83845&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 174 Build time: 00:12:42
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


couch got angry when I tried to use a name starting with `_`


On 12/01/2015 07:06 PM, Massimo Callegari wrote:
> @rncbc <https://github.com/rncbc> It's Massimo by the way :laughing:
>

oops!

i guess i keyed in the portuguese spelling, as "Maximo" sounds
phonetically the same as italian "Massimo"

sorry. no pun intended whatsoever
--
rncbc aka. Rui Nuno Capela


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 74](https://teamcity.labkey.org/viewLog.html?buildId=288671&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 17 (3 new), passed: 30 Build time: 01:09:26
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


This test case seems to fail for dates < 10. It appears to return  'Member since: December **01**, 2015' while the response contains 'Member since: December **1**, 2015' . Using '%B %-d, %Y' seems to help.


#5 #6


What version of BusyBox are you using?  I checked the [getty sources](http://git.busybox.net/busybox/tree/loginutils/getty.c#n212) and it looks like this:

		if (G.tty_name[0] != '/')
			G.tty_name = xasprintf("/dev/%s", G.tty_name); /* will leak it */

So it should work both with *or* without `/dev` prefix to the TTY ...


This breaks stuff for me.
Must I upgrade to F# 4.4?


This no longer has the right DefaultAttributes.concatAll


This change should probably be extracted into a separate PR.


hani extrahiert und denn hätses grad so gno, chans scho abändere