This file needs JSHinting :)


Generally every user should have password, I guess. Correct me if I am wrong.

Why we don't have "nullable=False" for password field.


the faceting of "TAXONOMY" here conflicts  with the "species_organisms", we have decide to pick one of them.


What is self.adjust_dt?


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1145
success: http://build.servo.org/builders/linux2/builds/1145
success: http://build.servo.org/builders/android/builds/1179
success: http://build.servo.org/builders/mac1/builds/1358
success: http://build.servo.org/builders/mac2/builds/1361
success: http://build.servo.org/builders/gonk/builds/327


What's your plan for incrementally getting rid of `sc_admit`? I was trying to fix case  `wf_bind_deep` of `ip2pr`, but it seems to require that the extended context is well-formed, and there's no way to ensure that it has good bounds from just wfness -- or am I missing something?

```coq
(* case wf_bind_deep *)
    introv WfDs IH Eq Sc.
    apply_fresh wf_bind_deep as z. apply IH; auto.
    apply sc_admit (* for simple_ctx (G & z ~ typ_bind Ds) *).
```


not a constructor, just `resourceLoader` is better


When marking up inline "verbatim" commands, you only need a single quote.  For example:

With some trial and error, we figured out that `./configure CFLAGS="-O2 -g" ...` should get you up and running.

If you look at the source of my comment, you'll see that I only used a single quote for the inline ./configure text.  The triple quotes are for delimiting multiple lines of "verbatim" text.


This is fundamentally broken, right? Let's not commit it.


fast-forwarding master to auto = 004e12c8


Meh, would rather just do `function () { }` in the two places you use this...


Actually it could be null (see: http://jira.xwiki.org/browse/XWIKI-10092) but I agree it does not make much sense.


Random note: you saved the output from autogen / configure / make -- good!  ...but what were they from?  E.g., what configure command did you use?  What made this set of autogen/configure/make commands different than others? You might want to save a few notes to remind yourself what exactly these files represent, perhaps in the commit message, or perhaps putting the files in a subdir under 3/25, etc.  E.g., if you need to use `./configure CFLAGS="-O2 -g" ...` to get a good build, etc.


Может некорректно работать, когда контекст между тестовыми наборами передается.


@Sebmaster thoughts?


Unit tests could check cases where tabs and spaces are mixed in the same line.


These functions will cause errors in before the requests have been created in the loop below (L114).


We will keep taxonomy, I just removed the organism.


I designed this so that it would fail if a field is not mapped. If there are fields that should not be mapped, the parameter `ignoredFields` is exactly for that purpose.


Reads well!


You need to pass cozydb.NoSchema as the second parameter


Nah, only `undefined` should trigger the default


remove instead of commenting out


I don't think you need the `to_s`.


Da, asa e verificarea merge in situatia asta, pentru ca time_zone e o proprietate pe result. Dar daca vroiai sa testezi ceva gen `if(time_zone != undefined)` si time zone nu era defined verificarea nu ar fi mers si codul ar fi crapat. Din aceste considerente daca vrei sa verifici daca o variabila exista sau nu e mai ok sa o faci cu typeof.

Dar de cele mai multe ori tu nu vrei sa verifici doar daca e definita ci mai mult vrei sa stii daca are si o valoare diferita de undefined si de null. Pentru ca practic verificarea ta asta face. Concret in verificarea scrisa de tine null != undefined ar fi dat false (adica null == undefined). Deci practic tu veri sa verifici ca variabila exista si are valoare. Caz in care cel mai normal e sa scrii `if(result.time_zone)` care include toate astea si presupune si cel mai scurt cod.


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


A question, maybe obvious but, why not do that in doTrashConnection() since it seems that it needs to be done each time we trash a connection... ?


Superficial review since I don't know much about the domain.

I see the lib used to provide an empty dimensions thing – is that now an invalid choice? Since the README example seems to suggest you must provide dimensions.


Add this as @@driver.find_element(:id,SEARCH_SUGGESTION)


res.getHeader


Never do this in the app start server.. Set the context in the Given statement


Corpul acesti metode ar trebui sa aiba doar un call catre TimeZoneService, care sa intoarca lista.

De asemenea nu e cazul sa folosesti fully qualified name pentru clase. Poti sa pui using si sa folosesti numele scurt.


Constants should always b in CAPS SEARCH_BTN = "com.quikr:id/layout_home_page_header_search_icon"


I'm happy to add extra points here but it seems silly to copy everything across when we can just reference the comparison on GitHub.


load app package and app activity from commandline


Thank you very much for this


100th commit...yeay


The [Primer]-link (/primer/primer/) at the end of this segment is broken too (leads to https://github.com/primer/markdown/blob/master/primer/primer instead of https://github.com/primer/primer).


I already pulled it into the code base a long time ago.

https://github.com/openemr/openemr/commit/c5bad34503cdc212ec7fe9e80b9bd92fcbca478c#diff-0de957cf452e91750ce7dfe28fb18c6f


Before fixing LOAN-6
add a fix for LOAN-1, LOAN-2, LOAN-3, LOAN-4


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1163 of 1166 tests passed)

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


`{ http: false }` spaces


Space after `if`


Terminated*


너 틀림


Can't we just replace these four lines with `cookieStr = String(cookieStr)`?


I'd rather not have these dependencies in cozydb itself.
I am not sure what's the best way to go...

Option 1 : we add explanation in README for the app using this to handle it
Option 2 :  we add a script to install them with npm run install-standalone


Apart form the typo in the commit message, what do you think about this?


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

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


The real issue is that like I indicated in the comment we don't have any way in Oracle to differentiate between a null or empty String property so it's safer to do that for now.


Yeah, I've been neglecting making it async since this hasn't been a problem for me. I'll happily accept a patch, although it should probably be done in tobi's package.


@e-kolpakov there are many block like this I think the idea is to quickly enable disable things. For instance if someone would like to enable profiler they only need to set the value of `FEATURES['PROFILER']` to `True`.


Yes, it's finished. Only .mo file is needed. Please, make it yourself.


お、ナイス


yes,
i get something like "no property 'destruct' in config"
plus the way i wrote it it maps directly to the values in .librarian/puppet/config
using
Vagrant 1.7.2
vagrant-librarian-puppet (0.8.0)

also take a look at https://github.com/mhahn/vagrant-librarian-puppet/blob/master/lib/vagrant-librarian-puppet/config.rb
seems to rather map to the properties i listed.


Good catch


This seems to produce unreachable code – why is that?


Yes, makes sense.


Aici de ce intorci empty si nu null?


Thanks! Debugging oversight.


scouter work fine for me don't know why other crash :(


whoops, I should have looked at the source code of the method itself… got it now


oh yeah


To avoid code duplication, wouldn't it be more reasonable to call clearCache() in a slot connected to signal informationChanged() ?


Astea 2 linii ar putea fi mutate in GetTimeZoneMapXml.
De asemenea as prefera ca path-ul asta sa vina din app.settings in loc sa il obtii asa.
Si paote fi relativ in asa fel incat sa mearga si pe productie si local.

Legat de setarea aia cu copiat fisere in App_Data cred ca mai bine o pui inapoi pe False. Fisierul asta practic nu se modifica niciodata si poate fi copiat manual si lasat acolo.


Можешь пример таких тестов и тест.наборов выложить для проверки?


stupid me i should have prototype that lol :O


Thanks, did you test this?

please use the en-US version for changes to the language files, something removed there will be removed in all the other languages


ctrl+z caused this thing that my crash function went to Enemy var and i i didn't see it again ^^


Where is Gui::displayClientMessage..?

I can't find it!


Why not setting it to OPEN directly and avoid the RESURRECTING state ? Since we know addConnectionIfUnderMaximum() will set it to OPEN after. So we could the avoid the compareAndSet in addConnectionIfUnderMaximum(), nope ?


Are you sure it is ok to assume we're no longer using screen stream ? getSwitchStreamFailed callback is also called when we fail to obtain camera video stream.


Can we actually remove these without deprecating them first?


Laurent, I launched a Repl and here, if I don't introduce a check, CCWPlugin/log gets executed every time. Not the start-server of course, but the log line...I think I need to fix it.


cc @michaelklishin @arr-ee builds work now!


:heart:
next up: multiple C* versions?


BTW this can be set in the field declaration then you won't need to check for null on every access.


"numberOfPoints" and not "anz" (first because it's a abbreviation, second it's german -> stick to one language)


We could put it into refreshPlaybackWidgets() but it is only necessary to clear the cache if YUV Math changes happened.


Реальных тестов у меня нет. Или, имеется ввиду, создать тесты, воспроизводящие такую ситуацию?


I think ``self.back_clicked`` would be a better name.


This ensures that two threads don't race in this method, we need to do it before we spawn a new connection at line 315.


```private static final int SIZE_OF_WINDOWS = 750;```
```private``` because this constant shouldn't be accessible from other classes
```final``` because it's a never changing variable -> constant
```SIZE_OF_WINDOW``` is self explanatory -> no need for comment


Comment is counter productive!


Fix aci pe linia pe care discutam noi:
`$scope.signUp = function ($scope) {`


hej


```//draw a circle fitting in exactly the whole window```


replace `firefly` by `$(arg mav_name)` here and below


```int[] yPoits```


You're right. I'm not even sure if making wf-judments precise will work,
because an imprecise wf judgment is not a very strong statement.
I don't yet have a concrete plan, but options are:

*    use another invariant than `simple_ctx` for `ip2pr` (for every lemma,
it's crucial to find exactly the right requirements on gamma)
*    don't include the wf-judgments in `ip2pr`, and instead of using an
IHWf in ip2pr/subtyp_top/bot/refl, get it from somewhere else (?)
*    drop bind-types and use global self refs in the env... ;-)

On Wed, Mar 25, 2015 at 11:46 AM, Nada Amin <notifications@github.com>
wrote:

> What's your plan for incrementally getting rid of sc_admit? I was trying
> to fix case wf_bind_deep of ip2pr, but it seems to require that the
> extended context is well-formed, and there's no way to ensure that it has
> good bounds from just wfness -- or am I missing something?
>
> (* case wf_bind_deep *)
>     introv WfDs IH Eq Sc.
>     apply_fresh wf_bind_deep as z. apply IH; auto.
>     apply sc_admit (* for simple_ctx (G & z ~ typ_bind Ds) *).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/samuelgruetter/dot-calculus/commit/1a10f36caba2bb7ed0a2d37b87ab9e63661020f4#commitcomment-10386480>
> .
>


This check here is O(N), way too expensive for a constructor.


what that doing


I was just looking at the django docs and saw that this is now a property on request: https://docs.djangoproject.com/en/1.8/ref/request-response/#django.http.HttpRequest.scheme
I didn't know this was added. We're using 1.8, so it's available to us.


Has approx 9 statements (TooManyStatements)


Assignment Branch Condition size for insert_all is too high. [23.56/20]


In general, I believe we should strive to have the same tools on every platform in our stdenv. That should result into more uniform expressions and save us such problems. (I had no idea sed would behave differently on different platforms, and I don't have access to any darwin/bsd to test stuff.)


One of the few places where you actually need comments...
What is ```int a, b, c1, c2```? The variable name should explain that.
And add a comment why you need these variables.


I would just let libuv handle the ENOSYS.


> Which are the two occurrences?

One on line 19, another on line 22. Thanks for the other information - haven't fully digested it yet :-)


Ditto.


@DanBadea
Ok, inteleg. Am modificat


Regarding "registering" of the field, I still don't really understand what it's good for. The following happens:

In `FieldReflection::checkPostFieldData`:

```php
$driver = Symphony::ExtensionManager()->create('reflectionfield');
$driver->registerField($this);
```

Then, in `Extension_ReflectionField::registerField`:

```php
self::$fields[] = $field;
```

What is the advantage of "registering" the field like this?

I am asking because `Symphony::ExtensionManager()->create` (for every entry) is very slow—you will notice that, for example, if you import thousands of entries.


Ага, создай такие тесты.


The call to `to_s` is needed to ensure that the values of the array are rendered as strings (as they appear in the SIE 4B spec). The code breaks (and the tests fail) if this call is removed. See 9fe7b52c26813796afa766e0482acaa388bae341.


Haha. It's all being monitored anyway, why not make it public? I'm going to start cs50 this week. What %age of the course have you completed?


@mattwynne not really, I just felt a bit uncomfortable pushing it to master and not releasing it.


I'll try to do something like that


@DanBadea
In cazul asta nu pot sa folosesc numele scurt pentru ca exista si clasa TimeZone in System.
Codul de aici cred ca este putin outdated pentru ca asa este facut acum, se intoarce lista direct din serviciu.
It's something like this:
public List<Equilobe.DailyReport.Models.TimeZone.TimeZone> Get()
        {
            return TimeZoneService.GetSystemTimeZones();
        }


@mattwynne ahh - I see what you mean, the version number. No reason at all actually.


Specifically `subvalue` could be an integer, as I think is the case in at least one of the tests.


@arr-ee that's extremely easy, just give me cassandra versions you'd like to test against


¿? :grinning:


int32_t round(float v) const{
		int32_t t = (int32_t)std::floor(v);
		if((v - t) > 0.5){
			return t + 1;
		}
		return t;
	}
This needs to be in combat.h
or else msvc gives error:
1>..\combat.cpp(1383): error C3861: 'round': identifier not found
1>..\combat.cpp(1384): error C3861: 'round': identifier not found


What's that empty line?


Ну да `->setKey($key)`


TeamCity Nimbus :: 0. Feature Branches [Build 0.0.0.0](http://twcbuild.cloudapp.net/viewLog.html?buildId=1149&buildTypeId=bt8) is now running


求继续


can we remove the `toBool` method then from `IndicesOptions`?


Did you test this? When I try it, I get a build error.


Currently we use the id for identifying when dealing with Mixins (I know name, UUID, SenderEntity, player, sender & IdData is also valid), my thought was that one day we might want to use something else. In that case we would have to change this.getId() in several places. If we used this method we only had to change the code in one place.
We might never change from using the id and in that case this method is useless.


@vcunat The Darwin stdenv does provide GNU sed. The issue here is that during stdenv bootstrap, we have to use the native sed.


I would recommend to connect informationChanged() to refreshDisplayImage() in the constructor of FrameObject. Then you don't need to call clearCache() for each parameter change...


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (654 of 656 tests passed)

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


For changes in the YUVFile object this is already done this way...


I don’t think testing against 1.2.x is worth it, so 2.0.13/2.1.3 should be enough for now, but if it is easy we can always expand this matrix to cover all minor releases to make sure nothing broke :)


The comment seems to indicate that it's not.


Thanks :)
I started off down that same path too. As soon as I changed tack it
suddenly became a whole lot easier

On 24 March 2015 at 21:05, David Gregory <notifications@github.com> wrote:

> This is fricking awesome. When I tried to do this I got too hung up on
> trying to use findCellExtractor...I was basically skipping a step and
> trying to turn an A* parameter into a Seq[Setter[TableColumn[A], B]] all at
> once.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jhc-systems/sqlest/commit/bb8adc3c5935c5833f878adb73c3b6a0d1656968#commitcomment-10377315>
> .
>


Probably this could be slightly improved using named tuple (see (*)),  bot don't bother to redo the pull. I will do a final check after lunch and then merge it.

(*)
https://github.com/dmwm/CRABServer/blob/master/src/python/Databases/TaskDB/Oracle/Task/Task.py#L10-L18

https://github.com/dmwm/CRABServer/blob/master/src/python/CRABInterface/HTCondorDataWorkflow.py#L65-L66


@mattwynne have fixed that and rebased this branch.


We added this state for a previous version of the algorithm. There was a race condition where the connection would be re-trashed between this line and `connections.add` in `addConnectionIfUnderMaximum`.
I don't think this can happen with the current algorithm though, so I'll reconsider it. Thanks!


I, I just checked and this connection is already established in DisplayObject's constructor ... therefore, it should already clear the cache when informationChanged() is emitted ?!


I've kept a [Resources](https://github.com/3SB/Learningz#resources) section containing these kinds of libs, etc.

Might be a good idea to move it there?


TeamCity Nimbus :: 0. Feature Branches [Build 0.0.0.0](http://twcbuild.cloudapp.net/viewLog.html?buildId=1149&buildTypeId=bt8) outcome was **SUCCESS**
Summary: Tests passed: 1445 Build time: 00:05:43


Without registering the field, it's not possible to call the `compile` function and allow the Reflection field to do it's work. By nature, the Reflection Field is an expensive field because it needs to know about the entire entry anytime it's been saved. This is multiplied if there are multiple Reflection fields in your section, or, like yourself, you are dealing with huge numbers.

Simply, if you remove the callback, the Reflection Field won't work.

I'm a little surprised the `Symphony::ExtensionManager()->create` call is slow, it's a cached lookup so it'll only ever initialize a single Reflection Field instance. Subsequent calls will return the cached instance of the field. I'm assuming this is all 2.6.1?


In other words: The above fix is much faster than registering the field in the `checkPostFieldData` function. And to me it seems that it renders the latter superfluous.

Right? Or stupid?


Disse to funksjonene ser lik ut. ;p


This doesn't seem to match the behaviour of `computeMetadata()`, which is testing to find the first connected plug - we need to make them match one way or another. I think I would prefer to not test connections at all, and simply always take the metadata from inPlug(), regardless of whether or not it's connected. That would be consistent with the behaviour of Group with regard to the scene globals. What do you think?


Ok. Didn't know.
Please move it.

Next I'll send you a PR, that's better :smile:

On Wed, Mar 25, 2015 at 5:01 PM, Shrayas Rajagopal <notifications@github.com
> wrote:

> I've kept a Resources <https://github.com/3SB/Learningz#resources>
> section containing these kinds of libs, etc.
>
> Might be a good idea to move it there?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/3SB/Learningz/commit/c1435f7ca825333ccf24de39a4b86475232388ef#commitcomment-10387244>
> .
>


I wonder if this should also be a `NoSuchElementException`


曲ごとにbpmが違うものは、ハードコーディングしました。


sorry i was wrong here, just forget it ;)


With quotes around null here, how would you tell `"null"` apart from `null`?


Full stop before Users?


I do not need this myself yet.
You once mentioned a HealthMixin and I have seen in some of your documents that you said the Bukkit API for dealing with health was mess and also said something along the lines of implementing you own.
Currently we can't get someone's health while they are on another server right?


about space , dhcpd : unrecognized service
libvirtd :  unrecognized service
starting space ( mkdir cannot create directory ' /var/log/space : file exists
space : unrecognized service
starting celery worker
sh : can't open /srv/space/scripts/celery.sh
How to fix
thanks


@DanBadea
La momentul respectiv m-am gandit ca ar fi mai bine sa intorc un empty string ca sa nu existe vreo problema pe partea de client daca nu este gasit niciun id, dar ai dreptate e mai bine sa intorc null.


Haha. No no not required.

Moved :+1:


That should turn off the screen sharing so I think it is OK. Am I wrong?

If you don't have video device but you want to turn off the desktop sharing that function will be called. So without this change it is not possible to stop the desktop sharing in this situation.

Have you noticed some issues?


@jerpelea @AdFad666 I think these are needed ... what do you think? these values were took from cm msm8974-common ... them are ok for aosp too or need changes


@hinerm Does this actually work in your case? Since the code is not explicitly part of a package, TJ.class.getPackage() returns null (at least in my environment). Let me know if you'd like me to put all code in a package ("transformj" would be a logical name). Then I'll fix that as well. And in that case perhaps it would be better to increase the version number to 3.0.0, as this is a "major" thing that will affect anyone using TransformJ from his/her code...


@janmeier why was `options.transaction` set as required?


:100: :+1:


Sorry, I had not seen your answer.

> Without registering the field, it's not possible to call the `compile` function

In my understanding, with the above fix, the field gets registered in the compile function. So there is no need to do it before.

> I'm a little surprised the `Symphony::ExtensionManager()->create` call is slow.

I am developing a custom field that is much simpler than the reflection field, but works in the same manner. Without adding the field, I can import 100 entries in 1.5 seconds. If I add the field with the above fix, but without registering stuff in the `checkPostFieldData` function, I have the same speed. As soon as I add the [registering logic](https://github.com/symphonists/reflectionfield/commit/9045e86f31977d6e9e461d3ef75ef9ec9f3d47c5#commitcomment-10387095), 100 entries take 3.9 seconds, so the time is more than doubled.


How about using a set that uses identity equality instead, to not rely on uniqueness of hash codes, like this:

    final Set<Result.ResultRow> result = Collections.newSetFromMap( new IdentityHashMap<Result.ResultRow,Boolean>() );

and

    result.add( row );

Perhaps the test should also include a counter to verify that the `visit` method was invoked more than once...


Ok. Btw, another thought, if a concurrent thread trash this connection, only trash, how to know if it will be replaced or not ?


These are only tentative fixes for me to look for what's wrong with armv8 tests since I don't have an armv8 host and just tested on an emulated host under qemu. I agree ENOSYS is better and probably this fix should be made in `linux-core.c`. Do you have an armv8 host to work this?


本来dpnとなっているべきところがspnになっていたため修正しました。


Missade #183 i den incheckning jag ville göra. Här är den bättre:
https://github.com/rinfo/rdl/commit/82ad0e587d72a86633f68f95a24c093d9fc1649a


What are the exception propagation semantics? What happens with exceptions thrown by implementors of this interface?


> I'm assuming this is all 2.6.1?

Yes, it is.


Note to any Mekanism guys looking at this:  we do not __distribute__ the Mekanism API in Galacticraft, we only reference it as a dependency in our build environment.  The minor changes to your API code are made only so that our build environment is error-free, they will never be distributed.


nice job boys! :+1:


Looks good, thanks. But maybe we should still error in case the feature was explicitly enabled? I will try to work on that this evening.


Should there be a `Number getNumber()` for when the user does not care about it being an integer or a floating point number?

Perhaps we should _only_ have a `getNumber()` method for now, and let the user use the `longValue()` and `doubleValue()` methods of `java.lang.Number`.


Missing a comma at the end of this line


I tried it and it works for me


OK

2015-03-25 13:25 GMT+02:00 bindea-mihai <notifications@github.com>:

> @DanBadea <https://github.com/DanBadea>
> In cazul asta nu pot sa folosesc numele scurt pentru ca exista si clasa
> TimeZone in System.
> Codul de aici cred ca este putin outdated pentru ca asa este facut acum,
> se intoarce lista direct din serviciu.
> It's something like this:
> public List Get()
> {
> return TimeZoneService.GetSystemTimeZones();
> }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/equilobe/DailyReportingTools/commit/65d040613fbc0315dabc2532e06eef7012c67e98#commitcomment-10387142>
> .
>


Some comment


Если мы сможем пронести твердый предмет, это гарантирует то, что мы сможем пронести жидкость?
Если мы сможем пронести, то сможем ли после этого, например, поставить?
Как мы проверили хранение?
Можно ли пакет с чем-то в чем-то транспортировать/хранить?


@justinclift I think I can do it in my free time :blush:


not date-time?


`\d` should be `\\d` because this is a string. (`\d` represents an escape sequence)


mhaas:  if DEFAULT_AUTHSERVSSLPEERVER = 1 => when "if (config.ssl_verify < 0)", we should not have config.ssl_verify = 1 instead of 0;


Смысл будем проверять?


Ze SpectPwelch jest drobny problem - nie mam pewności czy dobrze liczy :D
Wydawało mi się ze wychodzą inne widma niż w eeglabie... (ups ;) ), ale nie testowałem tego dokładnie.
zastanawiam się czy domyślnie nie lwpiej mieć tej samej funkcji co w pop prop.
W FieldTripie zrezygnowali z pwelcha, ale (w przyszłości) przydadzą się funkcje fieldtripowe do wyboru ...
(W ogóle myślę o czymś w rodzaju nipy tyle ze dla elektrofizjo...)


This seems to be correct so validUntil needs to be fixed, right?


Why this prevents anything but miss-configured entities?


一般代理都是有用户名和密码 请考虑支持


Hmm, I wonder, are you able to `var_dump(count(self::$fields))` [here](https://github.com/symphonists/reflectionfield/blob/master/extension.driver.php#L198)? Perhaps there is an issue that the `checkPostFieldData` adds multiple field instances to this array, so instead of dealing with a single instance, it deals with multiple instances (perhaps for each entry processed?)


No, I just use the CI.  I've noticed that the armv8 bot isn't in the libuv matrix, only the io.js one.


Здесь может получиться так, что пока пакет пустой, ручки удобные, а при наполнении пакета они смещаются в совершенно неудобное положение.
Плюс, неплохо бы проверить, насколько они удобны при переносе большого веса (если пакет это допускает)


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


@cusspvz because `set` is pointless without a transaction in sequelize because of pooling.


Fuvar


//For counting the number of points inside or outisde of the cricle
int numberOfPointsInside = 0;
int numberOfPointsOutside = 0;


what is ```0.1``` -> make constant


Link dependencies seem wrong. Should be something like
libweb3jsonrpc, libdevcore, libethereum, libdevcrypto, some json stuff and boost filesystem.

It should not depend on: leveldb, mhd, secp256k1, solidity, serpent.


I think you should specify how cheap or expensive the UI library expects these calls to be. If ItemAt is used in tight loops but ItemIndex is not, someone implementing an adapter should design for that.


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


I was doing some expriments and using like you have on your source and I wasn't able to open recovery, then I changed that echo and I was able to open recovery, tested on KK.
Now I have installed LP and rickiller seems to work fine, since I have RW permissions.
But you will know the process better than me.


Does that work on windows?


I do it because it is unpleasant to make a change in the mode of delivery for reasons of test and lose data as the key settings do not know


The Wall of Pine Needles is not committed yet.


Method names start a with lower case letter


The spell will still finish resolving normally if it somehow leaves the stack mid-resolution. Otherwise cards like Time Stop would not work.

And I think it would be more compliant not to invent a type of ability that is not supported by the rules.


yes, thx for finding.


Header missing.


Okay, I'm waiting @rvagg 's reply.


Here `response` is not the instance of the [http.ServerResponse](https://iojs.org/api/http.html#http_class_http_serverresponse). Therefore it doesn't have `getHeader` method.


kygy


[Format your code](https://www.jetbrains.com/idea/help/reformatting-source-code.html)


I will add it today.

On Mar 25, 2015, at 6:53 AM, LevelX2 <notifications@github.com<mailto:notifications@github.com>> wrote:

The Wall of Pine Needles is not committed yet.

—
Reply to this email directly or view it on GitHub<https://github.com/magefree/mage/commit/804f5c990801114562acc2693442beab7c9f375a#commitcomment-10387601>.


Good work! (in commit)


I see... with an EntityLine it shouldn't be an issue but with EntityDetails could be.


Proszę wprowadzić następujące zmiany:

a) ustalić etykiety dla osi OY tak aby były wyświetlone procenty
b) podać wartości dla słupków w liczbach całkowitych (na przykład zamiast 0.31, etykieta powinna być 31)
c) proszę rozwinąć skróty przypisane do kolorów


Thanks a lot. Keep contributing. When I get time tonight, I will add a todo list as to how to scale this script in a much better way! Merging it right away. :)


Bingo. :-) For entry number 100, the result is `int(100)`.


Brilliant commit. This saved me a lot of work. Have you considered filing a pull request to get these changes upstream?


@rnuti Nice. However, I forgot to emphasize in something: You should commit only useful files.

As a general rule, commit only source files. As a thumb rule: don't think of this as dropbox.

So, please, delete all files you are not using, e.g. those related to the soft hand stl's, in your local repository using
    `git rm PATH/TO/FILE`
Then, use
    `git commit --amend`
to overwrite this commit, and finally
    `git push --force`
to make the change on the remote.

Let me know if you have doubts.


Better call the other constructor here, or use a default value of getRandomPath for the parameter.


lets make a global class for wysihtml5


Please check the return value. I think we never ever want to delete a path (including all its contents) that we did not create ourselves.


`explicit`?


Не хватает анализа кружки по эвристикие SFDPOT. Нужно проанализировать по ней кружку и по собранной информации также написать, что именно нужно проверить и как


`const&` and `const`?


`:` should go on his line.


Почему это тесты производительности?


It will be better to include and use bootstrap-wysihtml5  only in need.
Maybe we will define and assetic group that includes bootstrap-wysihtml5 library and triggers, and include this assetic group only required pages.


Oh, and I fixed some performance issue's with the lhash loop


> Isn't it a bit strange that we append the flags to PLATFORM_C_FLAGS_DEBUG here, but we append them to CMAKE_CXX_FLAGS for the release build?

Yeah, it is. I must say I was just making adaptations here.

> Aren't the CMAKE_CXX_FLAGS_DEBUG flags appended to the CMAKE_CXX_FLAGS anyway for debug builds? If so this is redundant.
> We should be using add_definitions instead.

I have to check and make tries but this is relevant and should be fixed asap. I'll have a go at this.


Please comment on the PR, not the Commit.

But to answer your question: for any l2 protocol that uses 64-bit addresses.


It's no type of ability - it's a type of a one shot effect with special handling (interface) that is organized in a package.
Feel free to refactor it to a more compliant way. :-)


Ah ha, give me a tick I'll push a fix that you can test :) Great spotting @michael-e!


Has approx 6 statements (TooManyStatements)


Мы же не можем изменить дизайн ручки у уже готовой кружки.Здесь, если система не подразумевает возможности изменения собственной конфигурации, мы можем посмотреть только то, как на нее влияет конфигурация внешней среды.


A few things:

- If you have lots of else cases with nothing, when is almost certainly a
better bet
- This seems like a lot for someone porting to have to come up with on
their own, no?  Seems like exactly the kind of thing that should be in a
library and not copy-pasted everywhere.

On Tue, Mar 24, 2015 at 10:33 PM, ds26gte <notifications@github.com> wrote:

> Here's *a* Pyret definition of round-half-to-even (IEEE) in terms of the
> current num-round. I'll add it in a test suite file.
>
> fun round-half-to-even(n):
>   n-neg = (n < 0)
>   n-abs = num-abs(n)
>   var n-round = num-round(n-abs)
>   if num-abs(n-round - n-abs) == 1/2:
>     if num-modulo(n-round, 2) == 1:
>       n-round := n-round - 1
>     else: nothing
>     end
>   else: nothing
>   end
>   if n-neg:
>     n-round := 0 - n-round
>   else: nothing
>   end
>   n-round
> end
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brownplt/pyret-lang/commit/02fe3105ba2fa379e1034702dc1766668c05e45a#commitcomment-10381478>
> .
>


:-1: :(


Voucher line dimensions used to be all empty and still can be, if that's what you're referring to. See [lib/sie/document.rb:108](https://github.com/barsoom/sie/blob/f6c0482010cf5ef3f6244f92e9d28757324b75fa/lib/sie/document.rb#L108).


Seems to work for me


Function literal you dope.


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


I just tried like this:

```php
public function registerField($field)
{
    if (!in_array($field, self::$fields)) {
        self::$fields[] = $field;
    }
}
```

and it seems to work.


That logic is still a bit flawed, but then mine may yet be flawed. It needs more tests then. This code path is only hit when the gateway monitoring triggers a reload of OpenVPN, no place else calls it this way. If the interface resets or the IP address changes, it is handled by rc.newwanip which calls a different function, not this file, so the case of resync on the same interface is not handled by this file. In the cases where this code is run you would only want to resync OpenVPN if the VPN's interface should be changing due to a gateway event.


Можем написать, что посмотреть, в случае, если они есть. Например, бывает, на дне что-нибудь пишут - надо ли это проверять и на что именно обращать внимание?


aeef262 is more succinct, and actually aligns the code with that of the `self::$fields = $context['section']->fetchFields('reflection');` line, which will return an associative array where the key is the Field ID, and value is the actual Field


"p == len(t.value)" is probably a disaster. I think it should be (p+1) ...


I think these values can be passed as a parameter to the submit function. So, "$scope.submit = function (user)", which is called with "submit(user)" from the registration page.


So it would need to be like this:
Gateway up/down - No change in VPN interface - No resync (rc.openvpn)
Gateway up/down - Change in VPN interface - Resync (rc.openvpn)
Interface up/down - Always resync (rc.newwanip)
IP Address change - Always resync (rc.newwanip)


s/excluded/exclude/


Nice!  I had forgotten how multi-split worked, so I just documented the subset I remembered.


Another unicode arrow here...


Well, your efforts have been extraordinary over the last few weeks in particular.  Get some rest!


Ich verstehe nicht so ganz, warum man überhaupt die Adresse vor Bots verbergen möchte. Dann haben die Bots eben die Adresse und spammen einen voll. Aber dafür gibt es ja Spamfilter.


Confirmed! Thanks, @brendo! This is definitely a great improvement, also to other fields working in the same way, e.g. the Hashid Field (I will issue a PR over there)!


if size is not set,  seaf_db_row_get_column_int64 (row, 1) will return 0. see libzdb code:
```
long long ResultSet_getLLong(T R, int columnIndex) {
	assert(R);
        const char *s = R->op->getString(R->D, columnIndex);
	return s ? Str_parseLLong(s) : 0;
}
```


The condition under which returnConnection calls replaceConnection is `connection.maxAvailableStreams() < MIN_AVAILABLE_STREAMS`.
If we race with a call to trashConnection, the connection will end up in the trash. We check the condition again in tryResurrectConnection, so the connection would never be resurrected. maybeSpawnNewConnection did not get called, but if we trashed the connection in the first place, it probably means a new connection was not necessary, and even if it was, it will get created the next time borrowConnection is called.


Alright, great! :thumbsup:


Validator'ın file ını reference olarak eklememişsin? Runtime da patlama/çatlamaya sebep olmuyor mu bu?


Again, I don't think we can delete from `org.neo4j.helpers` without prior deprecation.


That should be:

created: !!timestamp '2015-03-23', i.e. not a regular string.


Inicio de posible falla


If you get the formatted value from this.get('values') instead of the html than it's guaranteed to be in the format specified by the attribute `mask`, so you can use that to parse the value. By the way, people can set `mask` to any value they want to, so you can't be sure that it will be either `%H:%M` or `%I:%M %p`, which are the ones you're using inside  `_convertTimeToInt`. Calling `A.Date.parse(this.get('mask'), originalValue)` should work for any given mask though, and without writing your own regex code.


"permutation"


Die E-Mail-Adresse ist übrigens auch auf der Github-Organisation-Seite zu sehen. Zitat aus dem HTML-Code der Seite:

```html
<a class="meta-link" href="mailto:post@curry-club-augsburg.de" itemprop="email">post@curry-club-augsburg.de</a>
```

Bots, die das nicht verstehen, sind schon sehr primitiv. Aber möglicherweise sind 90% eben genau das.


Bu functionality nin testleri nerede?


Can we just say "Senior Developer" - Titanium will mean nothing outside the bank.


Full name please - Bank of America Merrill Lynch


Chicago, Illinois, USA


add il, baml, finance


How about `add_redacted_fields(self, *field_names)`? That'd we more convenient to use, especially with a single field.


@aitek4iit This kind of commits is useless and makes the history really messy and hard to understand
They are generated when you "pull" without rebasing, please always do git pull --rebase, or if you are using tortoisegit see https://stackoverflow.com/questions/16234093/how-to-avoid-merge-commits-using-tortoisegit


`self.redacted_fields |= set(field_names)` is equivalent, more idiamatic and a bit faster.


Again, `Dumper.redacted_field |= self.redacted_fields`


Great.  Really any test that ensures that the run() method is working (time progreses and things change as expected) would be great to catch any unintentional changes


This condition relies on precedence of &&


Just a suggestion - we've found that saying number of years of experience is often an unhelpful metric. Young whiz-kids might get it after only a few days of study. Somebody else might not get it after a lifetime of study. Since we aren't ageist we don't actually care how long you've been studying - only the results matter.


I think the BuildRequires probably needs a few changes:
No longer required: eigen-devel, subversion
Missing requirements: python-numpy, git


This is somewhat nebulous - I'd take it out.


We usually use the word "fluency"


Neden "greater than or equal" değil de "less than"? Method un adını da less than yapalım o zaman?


You updated condition but not the comment which is now wrong (this is why comments are evil).


in MySQL INNER JOIN without a ON condition will produce a cartesian product, which is exactly what we want here, each and every row in the first table is joined to each and every row in the second table. Your attempt will not work because we need more than the users associated with the articles in some cases.

But even if it's not accurate, it did work? If yes, then it seems that postgreSQL is not ok with INNER JOIN without the ON clause.


@saurabhsjoshi: I don't think it works. Do check against the [latest development build](https://github.com/aneesh-neelam/VITacademics/tree/development), and run the unit tests.


It would be more consistent if SYSCALL8_OPCODE_REMOVE_ACCESS also return 0 instead of ENOSYS.
You can move the line 706 to 703:

		case SYSCALL8_OPCODE_GET_ACCESS:
		case SYSCALL8_OPCODE_REMOVE_ACCESS:
			return 0;//needed for mmCM
		break;


@Soldez well done `pressAhead = yes;`


No worries :)


Its seems good commit, but it should not be in this lib, instead application or program where you need this variable.


My thinking was mostly about the doc side of things – if it can be empty if you don't need it, is it worth saying so? I wonder if someone (e.g. us before we added this feature) who starts using this gem might be helped by that. You probably start by copy-pasting the example data source, and if it's a common use case not to need to care about dimensions (I have no idea, but apparently we got by without it) then maybe we could save someone some trouble.


Hooray, valid C!


This is a comment  on this commit.


Added your changes for Analysis Widget


Different and unrelated.  This file satisfies an elegant dependency. See
Step 3 here:
http://www.aps.anl.gov/Accelerator_Systems_Division/Accelerator_Operations_Physics/installationGuide_Linux.shtml
For some reason, the elegant linux installer doesn't include this. On
Windows, the installer must include it and the location is hardwired.


Sedär ja!


Add a docstring explaining that this is a library used by the other packaging components for this and that purpose, etc.


I think there's an “and” missing after “for sending”...


Hey -

Yeah, I saw your changes. Very good, but I want to err on the side of fewer fields, and be very explicit with what I send to InfluxDB. I want to have filter plugins that build up exactly what they want to go to InfluxDB in a separate container. I now have a specific log message format that specifies key/value pairs to be inserted into InfluxDB. I know only those few pairs will be inserted, no matter what other stuff is going on in our LogStash configuration.


UTF-8 problem


What are we supposed to test with that? Correct behavior of Type.is?


This is a breaking change.


do not merge these...i am merging it from another branch


Thanks. I've re-worded and drastically cut down this paragraph in the latest version.


Also a breaking change.


@ran81 You can create a node object using `entity_create`. The multi-step is redundant.


I don't think we can.


The Windows Mobile version is deprecated and should not be built anymore. See SeattleTestbed/attic#1041.


If I'm reading you right, @beghelli , the purpose of the cartesian is to get the MIN(u.date_registered).  Is there a reason we can't use a subquery for this?


This (and `base_installer_archive_dir` below) is a configurable item. It should be moved to the very top of the script and marked so that it's immediately evident that the user needs to customize these values/paths.


Fixes #60


Var olan unit testlerde henüz bir kırılmaya neden olmadı. Her koşulda düzgün çalışacağından emin değilim.


try String.trim() on lines 35, and 39


It's not really "trunk" anymore (an SVN term). `dependencies_dir` would be fine I guess.


Why? It's the default, so set it on invalid settings?


Why you doing this?


Exit on invalid config has my vote.


How about defaulting to `None`, checking later that the value was overridden by the user, and give a nice explanation to the user if they failed to? (Same below for `private_key_file`.)


почему бы не записать эти значения при инициализации?


damn girl...watch me slide into your dms


Everything we need from `dist` (the platform-specific installer code I think) should be added to this repo. `dist` has a huge history and includes lots of unrelated stuff. We'll likely phase it out when `installer-packaging` is mature


TeamCity xBIM :: XbimExchange :: Build [Build 139](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1200&buildTypeId=XBIM_XbimExchange_Build) is now running


TeamCity xBIM :: XbimExchange :: Build [Build 139](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1200&buildTypeId=XBIM_XbimExchange_Build) outcome was **FAILURE**
Summary: Compilation error: Xbim.COBieLite.SerialisationHelper\SerialisationHelper.csproj Build time: 00:00:18


Also bukkit doesn't support offline health getting or setting.


My only concern with this is that this makes Noggit a required dependency even if the client/user doesn't use the json feature.  Maybe this should be controlled with a boolean option to add default readers?  or a special value of "null" for a class name to disable it?


Has approx 12 statements (TooManyStatements)


Assignment Branch Condition size for insert_all is too high. [28.51/20]


Method has too many lines. [14/10]


очень сложно догадаться до магического значения 0 или 1, и что они означают. Для указания этих свойств стоит использовать объекты.


specs.keys() looks better.


Sorry to be picky about style, but can you please insert a single space between your "if" and "(" ?


using `eth -i`:
```
> listcontracts
libc++abi.dylib: terminating with uncaught exception of type dev::InterfaceNotSupported: Interface State::addresses() not supported.
```
same error when starting neth


Woops. How did I missed it? XD


selinux.fs: setzt den Kernel auf permissive von Source her. Danach sollte alles zu 100 % permissive sein.


That's a roundabout solution. We have library functions to read key files and cast them to string, https://github.com/SeattleTestbed/seattlelib_v2/blob/master/rsa.r2py#L763 and https://github.com/SeattleTestbed/seattlelib_v2/blob/master/rsa.r2py#L526 .


Ah, I see. Well, having a bootstrap tarball providing the tools (like on Linux) would be an option, but that doesn't seem significant, affecting only a few packages.


Allows for gameplay, with output that indicates the game winner. Provides for a new game, or a reset of the game in progress.

More detailed output to come.


```
if defined?(ActiveModel::Model)
+ klass = ActiveModel::Model
+ else
+ klass = ActiveRecord::Base
+ end
```

What is the aim of code, ActiveModel is defined in activeRecord as it is included


вот тут особенно просто мозг сломать — clients[client_type][1]


This leaks iff the `config` was initialized with `config_init()` as it already strdup'd a constant. Should check for non-NULL and `free()` I would think.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1164 of 1166 tests passed)

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


хорошо бы проверить этот элемент на существование, прежде чем проводить дальнейшие операции


Hmmmm. Aber mein Kernel läuft nach dem Start nicht auf permissive sondern enforcing. Daher die Frage ob das dann noch geht. Muss ich ggf. mal austesten.


Idea initially presented [in pylearn2-dev](https://groups.google.com/forum/#!topic/pylearn-dev/3JxgaLmr-QE).


Hi,

Thanks a lot for your feedback.

I will remove ZeroMemory and make a new plugin to handle controllers.
About the indentation, I will check later but I'm worry about it. If i can set this value by project in visual studio (2013), it's perfect. If not, I will have an issue because I also work some other projects with the same IDE.

I'm also working on the next version and I'm pretty exiting to commit it.

Currently I've got:
Possibility to play for 2 players (one by computer)
Check if the ROM CRC is the same between the 2 players
Check if the MUPEN Core version is the same between the 2 players
Completely rewritten the bitstream, my first version was so wrong!
Add a clock sync to be able to launch the emulator at the same moment (will need some rework later on)
Ping estimation between the 2 clients.

I'm writing the input lags on the local player thanks to the ping estimation and also the current rom speed (50 or 60 fps).

Again the next version is an intermediate version.


You may want to make sure an exception is reported, for example to fallback to another reporting solution (a local db, or even a file). It's the responsibility of the caller to make it non-blocking.
As logrus might be used for http logging, I strongly suggest to make is async here, instead of the lib.
I'll do it today if find the time :)


Update source content


This value is only used in the WinRM code, I propose to move the constant there also.


@kohsuke no this too is incorrect. See #100. Remember that all code changes are required to be reviewed.


Hi,
I'm doing it in grok ruby filters.
In  grok I'm fetching relevant fields from log lines, and define theirs names and data types, for example:
%{NUMBER:tps:float} - look some number, set its name to "tps" and define it's datatype to float.

This is my configuration file https://github.com/evgygor/test/blob/master/linux_full_sar.conf
I'm retrieves a lot of types of metrics, but influxdb output plugin configuration is small and simple.


`toParams` can return `List<String>` and you would simply invoke `copyToArray()` here


As this is only used in the WinRM code, move it under the relevant section of the documentation.


That your check behaves properly when getting an unknown symbol.


It might make sense to make this a QHash<QString, QVariant> to allow for values of non-string data types. I.e. If you have a count, you're not required to convert to a string and back.


1: M111 unrecognized component name: admin


Might be easier to make the default really 2 minutes, i.e. `120000` milliseconds, rather than a magical `0` which means 2 minutes.


could be inlined, so you don't expose a constant.


public interface MergedBeanDefinitionPostProcessor extends BeanPostProcessor - please delete the last one from class def


Is this change needed? And also the renaming of `kotlin-runtime` to `KotlinJavaRuntime`? I'm curious what causes this  if this is accidental.


wwww


That's a good question. Actually it should work correctly with boost::filesystem, I'm not sure about other cases. This function was previously part of `test/TestHelper.cpp` and we have used it without any issues.


??? These specs weren't fixed upstream? Or the addressbook interface changed?


ceci est un comment


fixed in e78b641290963eac4462d46a41b5eb90dfa79764


@vwadekar . Some more comments on this file. I guess your last update of the pull request was a force push and that purged all my previous comments :-) . Could you raise any subsequent pull requests separately?

cheers,
Achin


I understand that it is not possible to turn off the cpu on which TLK is resident using PSCI CPU_OFF. But how does the TLK surive a cpu power down through a PSCI CPU_SUSPEND? Upon powering up, the CPU's architectural state will be reset. Will that not hinder with the execution of the next SMC post power up?


By specifying the MPIDR at compile time, we are preventing another cpu from being the primary cpu i.e. any ARMv7/v8 system that supports TLK will have to ensure that a cpu with this MPIDR is the primary. Would it not be more scalable to find the MPIDR of the primary cpu at runtime and export that instead. TLK is a UP non-migrateable TOS. But that should not prevent it from running on any single cpu in the system as long as that cpu boots up first? Is this a limitaion of the TLK?


Asking out of curiosity! Would it not be easier to pass arguments by value using general purpose registers rather than populating them in a shared buffer and requiring the EL3 and S-EL1 have the same view of the memory.


Hmm also confusing is that the inverse change was made right before our merge commit


Do we really need per-cpu secure contexts if the requests are expected to arrive only on cpu0. For calls on other cpus that terminate in the TLKD, a secure context is not needed in any case.


O zaman emin olmalısın :)


wow thats completely returded and uncalled for


Note: Hits on sub update lastAttackedBy, however the damage stored for that hit is 0.


There should have been a spec for this.


Why is this on github??
It is specific to only the main server, and its dev room , right?


We should go over other style guide example code and be sure it also reflects this rule. Especially the 'good' examples. See lines 1124 and 1125 as an example of a style example marked good that doesn't follow this rule.


This isn't correct. Moves that cause 0 damage can still cause Focus Punch to fail.


je pense que tu mérite un 49


fixed in 77b8e7796ac1fb4ef1cf1a19f17da80dbb2dd454


I just downdload the latest caffe-master and cudnn v2,but this occour:
 cudnn_conv_layer.cu:65] Check failed: status == CUDNN_STATUS_SUCCESS (8 vs. 0)  CUDNN_STATUS_EXECUTION_FAILED

How can i fix this problem


Yeah, this won't work here because starting from `4.11` JUnit depends on hamcrest which needs to be downloaded separately...

This raises questions on if this library-downloading script is at all relevant. Maybe we should drop it altogether? Since you're working on the project right now, the decision is yours. I just have an allergy for things that appear to do something correctly whereas they don't :)


this closes #1


@TheImmortal Can you explain what is the rare bug with 0 damage?


@Rewinn Your HTML looks great. You are separating your custom JS and CSS from "vendor" JS and CSS as you should. I take issue with some of the burger toppings.


Oh I see. Weird asymmetry in Node :(


Awesomeness!


A move can cause 0 damage. 1 is not the minimum damage.


@Rewinn This is the shorthand for `$(document).ready`. But you actually also call it above on line 3. Why not keep everything inside a single ready function?

Also, just an FYI, you don't have your `index.js` loading in your HTML. Maybe that's intentional at this stage.


This is actually likely to break compile. ` brew --prefix $FORMULA ` leads to the ` opt_prefix `, in this case, presuming you keep Homebrew in ` /usr/local ` -  ` /usr/local/opt/libarchive/ `. Running the full command makes this happen: ` cannot access /usr/local/opt/libarchive/Cellar/libarchive/*/include/archive.h: No such file or directory `.

You want to change it to just ` $(ls -1 $(brew --prefix libarchive)/include/archive.h | head -n1) `.


@Rewinn nice `./` image path. Glad you've learned that.


t


Ah! I hadn't updated my local git clone yet, you've already fixed the above comment in https://github.com/google/bazel/commit/9a0dc1b222269c527942749c2990efabba38fb9b. Apologies :smile_cat:


The themes was also changed and the panel style was change too.


looks good, I'm wondering if that dialog is in front of everything?


I was reluctant to add tests like this here (to `indexer/native/testData`) because Foundation may be different on different machines and we have no control of that in our build/test process. Does it check something that can't be tested without Objective-C frameworks?


oopsie


imageData here is a 2D data set (shape is 10,10), I think you want imageData to be something like just data and remove the input.setShape() lines above.


Definitely, this line:
- Fixes Victormafire/IsoUnity#13
Maybe it's a code-grabbing issue. Consider copy and paste it into your code.


fixed in 2a892a60da6f99dfa94625dcbbc666e9cade4da7


vv


@Argelein в модели будет не только водитель, поэтому называть файл "Driver" не очень корректно. Добавил файл domainModel.vsdx, посмотри.


Definitely, this line fixes the decoration size issue. Maybe it's a code-grabbing issue. Consider copy and paste it into your code.
- Fixes Victormafire/IsoUnity#12


fixed in 42b3160f13d8c50de0a8fbac2dbaaaa4df37464d


Could you please pull this so I could merge?


True. The only case this would happen would be if an ability such as Friend Guard or Solid Rock would make a 1-2hp hit into a (0-1), right? I think that would return 0 without having a Sub.


Thanks. Your app is working now.
I'm using this as a reference for consuming polymer elements. I'm simply Adding polymer paper button (https://www.polymer-project.org/0.5/docs/elements/#paper-button) in index.html (before angular 2 is loaded) and app.html (inside angular's component) by adding this:

1. < link rel="import" href="bower_components/paper-button/paper-button.html">
2. <paper-button raised>polymer button outside angular 2 component directive</paper-button>

The button shows up only in index.html but NOT inside app.html.

Am I missing something...Please have a look: https://github.com/rkirov/youtube-app/pull/3


You can create a `File` and use extensions like `writeText` from Kotlin stdlib instead of create-write-flush, I think this would make the code here significantly shorter


They weren't fixed upstream because these are our changes that they didn't merge in.

I'll look into that inverse change though, that's weird.


pokemon.lastAttackedBy.damage > 0 can be changed to
(pokemon.lastAttackedBy.damage || (pokemon.lastAttackedBy.damage === 0 && !pokemon.volatiles['substitute']))
@TheImmortal what do you think?


It doesn't really matter if the lib or Logrus is async, as long as one of them guarantee it.

I'd say the fallback being the logging to stdout is fine for the general case.


This should be implemented as a method on Element:

~~~
element.each_model
~~~

Then, you create a way to resolve an element by its xpath

~~~
element.find_one_child('world[name="underwater"]').each_element do |element|
end
~~~


ENOSLEEP, this should reference #2


```ruby
modified?(source_path) || (dest_path && !File.exist?(dest_path))
```

There really is no need for the (`or`) because Ruby will evaluate the latter to false and trip the entire expression to false when it falls into the expression but I definitely like your new shorter version!


Thanks.  Will fix.  I have a parsing error with the MasterConfig that I need to work through which is the last error on the unit test (unless fixing it reveals more) and then I have some issues showing up in test-cmd.sh to work through as well.


aaa


Ah, I should probably set all the options in one place. Would you mind filing an issue so I don't forget?


A good cpp


These other protocols don't exist. In the docs, it was just an example of how future extensibility would be handled within the framing protocol. Please remove references to the protocols that aren't actually supported.


In general - yes, as "VM Exceptions" (which are passed back) or protocol error codes.

Zepto OS Exceptions do not necessarily map into these reported exceptions directly, it is just an underlying mechanism (similar to normal programming exceptions), which allows for generic exception handling (and then after these exceptions are 'caught', they may be either handled internally, or reported back).


I believe that is done automatically by the plugin. I haven't done that manually. If so, sticking to plugin's naming is easier.


Is fixed.


I think the way we have it now is better, we shouldn't call done on the loaded event since we only care about if the w3c suite gives its ok. If it doesn't the test should fail (which it now does in master because done will only be called on completion_callback).


uhaeuhaehuae no problem bro, I saw this PR in the repo and merged it. ^^


Yes, it's because I couldn't do disjuction. I've refactored the ruby call here: 8cb2b3c67eba7cdafa3775d878468fe538c841aa


Heh Heh Heh.  Do you have much free time? :wink:


Indeed. I did not really think that through :)


1: M111 unrecognized component name: admin


yes, thanks, fixed.


I don't know Go, but it looks like you're using a hostname as the server name. Can you please use the marketing name of your server instead? Smt like `GoLiveReload`.


i pushed a new one with the BlacklistedApp class functional, it seems i forgot to git add . at some point ...


036ed2e5b708e945bbec5debc1c870cfe64500a5 yep, fixed


You forgot to remove `multiplex_files` from the docstring.


Ah I see, so this PR just needs to be rebased and then this commit won't be necessary.


This was a mistake in merging


Yeah.


@mmilata could you release a new gem please?  I'd like to make this update available in our 1.8 and nightly repos as this code is currently in the RC phase.


Wow, is this really the best method to include a text file into a Go program? :-)


Thanks, I see your submissions. This will help manage time backlog reviewing code.


I see, #399 wasn't visible in the code commenting view...


Good code but lacks of more comments to make it clear.


I removed this because assert_raises failed in unit test when I call the method XML.model_from_name.


I had to remove it, since it turns textblocks into black rectangles. This change affects other geometric figures making them to be frames, i.e. filled with 'transparency'. :)


The code initializing the map now happens on every `serializeType`. But maybe that's what the todo before the method means...


Wait, wat ?


@fanglinliu Do you have any update? Probably I need pay some time to understand how this change fix the issue.


Point taken, configuration detection should be added into tests.
But this particular one is simply copied from kotlin test's testData, so the same problem exists there too.


Why writing a code gen? My understanding was that you would integrate your parts with the pipeline as it is, not override parts of it.


OK, I see what you mean:
```
 <product application="org.csstudio.product.application" name="Cs-studio">
         <property name="appName" value="Cs-studio" />
```
No idea, since I tend to leave the generation of the actual XML to the product file editor in the IDE.
… which tends to update both the *.product file and the plugin.xml of the product bundle.
No idea which of these matter, if they can be the same or not.
What I think does matter is the name you actually see in the window title when running under Linux.
When the launcher tries to handle the --launcher.openFile arguments, as far as I remember it asks the X11 window manager for all the open windows, checks their titles based on Window title == launcher  name with first letter capitalized, as in eclipse -> Eclipse, css -> Css, cs-studio -> Cs-studio. Then it sends the SWT.OpenDocuments message to that window.
So the exact "Cs-studio" that you might prefer to change into "CS-Studio" is the crux of the matter.

For details, it would be best to check the source code for the launcher, that is the C/C++ source code for the binary launcher used for linux/gtk.


That fails with Infiltrator and Sound moves. Anyway, it's better to have this bug than the previous bug. I'll think about a solution and discuss it with Marty later.


Can you be specific as which protocols needs to be removed? everything but official-7 and official-8?


It might be good to be more descriptive about what's going on in this usage especially, but maybe for all the `gsub`s in this class?


Which test exactly (file and line) ?


I've adjusted it in a8ca971c, thanks for the tip.


Well, as you put it, we just need to check the move hit the Substitute or the user, damage doesn't matter. Maybe we can do the same as in Gen 1 and save if in that turn the user hit the Sub or the actual target. That's a good solution. Also tagging @Marty-D


Instead of code-block python, you can just use the double colon syntax

```rst
..here is a simple example::

    julia> ...
```


Great!


Okay, I can see how it can be confusing.

Right now, the only protocol that's defined publicly is official-7. So should only leave that one. Although right now the work is underway on the next version.


Is the designate correct here? It points to PeakShavingConcentrator.Config.class, but I think it should point to PeakShavingConcentratorServlet.Config.class (even though the alias parameter is not used).


The order of merges has changed to the proper one (I guess). Notice, that earlier textblocks were not filled with black, because they had (and still have) `:fill => nil` that overrode `:fill => black` of `DEFAULT_STYLES`.


in file test_root.rb line 33.

~~~
        it "raises if the version constraint is not matched" do
            assert_raises(SDF::XML::UnavailableSDFVersionInModel) do
                SDF::Root.load_from_model_name("versioned_model",0)
            end
        end
~~~


Good work! But lacks of comments to clarify better your code!


That's fixed already, but I didn't yet pushed the changes. But todo remains nevertheless - that looks ugly even after fix.


c'est pas poli ca


That can boost the performance up a little bit.


-1000000000!  You're completely bypassing the API layer.  There is a good reason why I'm not yet exposing phase information and this is the wrong approach to hack it in.  The next release of pysam will include a phased attribute for each genotype call.


prefix commit message with DOCKER-271


버그 수정 하셨네요. 축하합니다.


I mean this "reported back". To my mind, these exceptions and catch logic could be useful for plugin developers, not just for Zepto OS developers. It is close to standard output ``stdout`` and ```stderr``. Normally, the correct flow of data will be passed generally like "answer" for "request". However, sometimes we will have situation where some "exceptions/errors" will occurred chaotically (bus disconnected, communication problem with sensor/actuators, etc.).

Will be good to have like ``stderr`` between SmartAnthill Core & Zepto OS. We will show this information to end user within Dashboard Console.


# **Pull request**

1.    I upload the question matrix (see teambition message from Echo) with a few ideas. Please complete it so that we can hand it to Echo asap.
2.    Remark : you can use *Atom* to write the pull request. I am not sure that it works, that's why I use weird layout to check it.


Won't this change get wiped out when the Perl script is run?


I guess this needs to be updated to something like $services.mailSender.send($message)


      If n is a
        roughnum, and n-round is
        exact (because rounding), then n-abs
        should be a roughnum, and then num-abs(n-round
          - n-abs) should be a roughnum, and then comparing that
        as == 1/2 will
        inevitably fail, no?


Why is this added as a devDependency instead of a plain dependency?


Yep, I can do that.


Done. See the last commit.


> so the same problem exists there too

Not exactly: that test only invokes something and checks that `OK` was output, not assuming anything particular about the available frameworks, but this one renders the exact internals of a class from the framework


Oh yeah, references to the overall match are a bit obscure. I'll update it to be clearer


Indeed, will fix, thanks


This is the protocol implementation (as a library) so I have changed it to allow the consumer set the servername, I have picked "go-livereload" as you can see under "cmd/livereload" which is the my server implementation using this protocol implementation.


No, that is the http handler, that is how you server the file to an http request. The inclusion is just the line 13 and after.


Revert changes


yes, this should be tweaked


Good work! But it could be better with more comments to clarify your code! Your las line got my attention, it says "no new line at end of file", maybe it lack a return (carriage return)?


Terminé con dos archivos: .node-version y .nodenv-version

Según https://github.com/OiNutter/nodenv/blob/master/libexec/nodenv-local
```
#For backwards compatibility, nodenv will also read version
# specifications from `.nodenv-version' files, but a `.node-version'
# file in the same directory takes precedence.
```


Assuming you really meant to nest finally blocks it is probably best to use separate lines.


Not relevant here of course, but there's now `enum class` in C++11 which is considered more type-safe and generally useful than C enums


ping @rande is ``knp-bundle`` ``2.0`` really required now ?


Ditto comment from matching try above


I had he impression that you're removing the test for the multiplex feature (`MULTIPLEX_CAPABLE`) at a very low-level (testing only in the command line), but then I have some interaction with the `Job` class in a prompt and after removing PyYAML, it's seems that everything is looking OK, it will work with PyYAML removed (MUX disabled).


confirmed, that fixes the issue.


This merge was not very successful, duplicate entries and not sorted alphabetically. I currently have to check everything again. Please make sure your merges are correct next time


Thanks for the suggestion.


:ok:


Wouldn't it have been better to use a LinkedBlockingQueue instead of a ConcurrentLinkedQueue, this way avoiding all the Thread.sleep() calls in the 2 MailRunanble instances?

Example:
https://github.com/xwiki/xwiki-platform/blob/master/xwiki-platform-core/xwiki-platform-search/xwiki-platform-search-solr/xwiki-platform-search-solr-api/src/main/java/org/xwiki/search/solr/internal/DefaultSolrIndexer.java#L188
...followed by:
https://github.com/xwiki/xwiki-platform/blob/master/xwiki-platform-core/xwiki-platform-search/xwiki-platform-search-solr/xwiki-platform-search-solr-api/src/main/java/org/xwiki/search/solr/internal/DefaultSolrIndexer.java#L368


This could still be a for loop I think?


Le port est utilisé lors de l'instanciation de l'application.
Il faudrait le hardcoder à 3000.


@mgp25 ... You are the best :) ... i follow you and i see your project that's amazing .... Good job


Le token devrait être dans private.


I like them, but like this:

    # TODO: Fix broken tests

This makes them easy to find, and serves as a guide for anyone reading your code.


когда ансиблом пользуются больше чем один человек (команда) - то каждый раз экспортить ключики не вариант, можно заюзать файл, где все это будет хранится


Is it `play.application.loader` or `play.modules.enabled`? I think the line above and this example are inconsistent.


It seems very strange that code is not at main window, you need to click on view to see it. Only can see two lines of comments. I already had written a bad note until I checked this. Be careful and revise your code!


Ah! I just read you note on it looking ugly.
I understand the desire to simplify the patch indentation but it just looks wrong and prone to confusion when someone makes a change in this area later. It isn't easy to see that updateSnapshot() is being called.


1: M111 unrecognized component name: template


158:14: E111 indentation is not a multiple of four


Oops, this is still here...


Should we start praying to the timeout gods? :)


Indeed, postgreSQL work with suggested modification ( with `ON (...)` )
and gives error without `ON (...)` condition.

I got alternative fetch with direct query for cartesian product
`articles a CROSS JOIN users u`
(without `ON (...)` condition)
and it did work, and output more records than with `INNER JOIN users u ON (a.user_id = u.user_id)`

The same (rich cross join) result happens if dummy ON operator used:
`articles a INNER JOIN users u ON (a.user_id = a.user_id)`


@azturner For performance, is creating a new/separate RockContext the best way to save a single change?    And should I be calling rockContext.Dispose() after I'm done?


Also, the comparison is not entirely necessary here. Just the presence of that param is enough as you don't have a different value for it in the other case. But I'm perfectly happy this just staying as-is.


This breaks inverted scrolling for workspaces!


Reviewed ok


is it possible to add all fakeplayers as "hidden" residents when making a new town?


Yeah, this is meant for cases when you want the `ng-model` expression to reference a getter/setter function, like `<div ng-model="someFunc" ng-model-options="{ getterSetter: true }">`.

If you're doing an `ng-repeat="elem in collection"`, you can always pass `elem` to a getter/setter on your scope, if all you need to know is the element being referenced:
```html
<div ng-repeat="elem in collection">
  <div ng-bind="elem.name" ng-click="ctrl.getterSetter(elem)"></div>
</div>
```


After this change I have not correctly calculated height ScrollContainer if it be in another ScrollContainer.


We have a convention that may increase readability in some places:

`a.contains(b)` <=> `b in a`


This my first comment


Perhaps not completely within scope of genericval, but on the other hand, it provides search options that are vitally needed.  H0031 and H0031HA are totally different procedures for instance.  It is pointless to search by code if you can't input the whole code.  As far as adding the billing note to the billing manager report, that is another vital data point to be considered when firing off billing.  The remaining issue that you cannot search by insurance company unless it has already been billed confuses many folks.  Mostly, they think that the by-insurance search simply does not function correctly.  It needs some explaining in context.  If there is an issue with having a bang in there, then I would suggest Insurance Company (Searches Billed Items Only)  Labels do not require punctuation.  Statements do, or they look like labels.  I can put it in a separate branch I suppose, but I highly recommend working it out.


batabase


The implementation is not broken but it does not affect performance as well. Shouldn't have committed it though. Will remove.


Good suggestion I think, makes it less tightly coupled - cool.


Has this been back ported into the original loopback?

I'm having duplicate issues on the strongloop Todo app - against mongodb.


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


~Netmask is nodemask.  For example. Given: 192.168.1.0/8.  Netmask is
255.255.255.0.  Node mask is 255.

192.168.1.0/10: Netmask is FF.FF.FC.0.  Node mask is 0x3FF.  10 bits set --
basically given the format A.B.C.D/E, E bits set IS the node mask.

On Wed, Mar 25, 2015 at 12:38 AM, hungta <notifications@github.com> wrote:

> Do you mean we need to transform netmask to node mask? But how to do? For
> example, netmask gotten from the interface is in type in_addr, so, how to
> transform it to node mask?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OpenClovis/SAFplus-Availability-Scalability-Platform/commit/fcdc22ad2364b883568475bbabaf3a97fe9cc1e6#commitcomment-10382289>
> .
>


一直想这么搞，终于实现了！


League\Flysystem is not added into composer.json.


Done ... will you take care of the packaging?


Would it make sense to update this? Or completely remove it as your git tags are valid [semvers](http://semver.org/).


Indeed, it's a good idea to use a BlockingQueue.


Also maybe the time has come to split this file to several smaller understandable units :)


Sorry, didn't mean to move it down a line as well -- was picking out lines from a larger change.

The next error I was getting was that '' is an invalid optoin for pycurl.COOKIEFILE so I'd removed the line.


so god


Lgtm


Just wondering if you or JustArchi is ready for cm-12.1 Git repos or a XDA thread :)
Although it's at staging state, it's working quite well and I'm ready to contribute into Git or builds :)


The `model` value here is actually a constructor, so it would need to be instantiated to pass an `instanceof` check. Since that's not feasible, using the prototype seemed to be the way to go. There's precedence: http://stackoverflow.com/questions/14486110/how-to-check-if-a-class-inherits-another-without-instanciating-an-object


Should be fixed right away.  Service.DisableMinimumKeySizeCheck


The same access to scope.$root.$$phase exists in notification.js as well (currently line 97). Do you think it is worth changing there as well (I ask this honestly not rhetorically due to the fact that I'm totally new to FfA)? Thx...


Funnily enough, I recently commited some code to the ncuts branch that is exactly "removeTopAndBottom": https://github.com/gapt/gapt/blob/ncuts/src/main/scala/at/logic/algorithms/fol/simplify.scala. We should maybe keep only one of them ^^


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


should be "used as an error message:"


I think this code didn't have any bugs, but vf_vapoursynth didn't drain frames correctly. So the filter signaled to the caller that it didn't have any frames, but then asynchronously decided otherwise a moment later. But looking at this again, I'm not so sure after all. This commit attempts to clean it up and to make it more readable: 7977961671dcec91fc69bc9dae4301d64901fb73. It also should take care of the change you did. Also I attempted to make vf_vapoursynth handle EOF draining better: 72308256e45e9184bb1578ad2cd40830e091ce13

vf_vapoursynth is a huge pain because it tries to use an asynchronous filter chain from a synchronous one (and also because vapoursynth has a random access model for frame requests). It probably would be better if our filter chain were asynchronous as well, but I'm not sure.


Hmm? What do you mean, to fix what?


The files could be called user_api if you want.


put indexes on format strings : {0} instead of {}


makes sense, but not sure if the bower file is still valid without any version.


lik dis if you cry evertim


I am also wondering how you added it to you database. I'm getting errors for:

[09:22:20 --4:00] [Thread 2356148366] extDB: DB_CUSTOM_V3: Error No Custom Call Not Found: getPlayerDonatorLevel:76561198073645252


Oops, my bad...


I thought we were going to find a way to reword these?


I've just had an idea how this can be fixed reliably: we could add a dependency on Kotlin reflection (add `kotlin-reflect.jar` to KotlinJavaRuntime library) and take the keywords from this internal class: https://github.com/JetBrains/kotlin/blob/master/core/descriptors/src/org/jetbrains/kotlin/renderer/KeywordStringsGenerated.java :)


Fix pushed. :scream:


Согласен.
Для этого у меня локально есть файлик vars, который в .gitignore прописан.
Я его использую.
Не стал уж в общий доступ свои ключи выкладывать :)


This could be put in the other file as well.


You should not use docstrings if you use format method. Change to regular string.


It is, but that's not a great idea since anyone can place a quarry which has a fakeplayer that has perms to the town. A link between the FakePlayer and TileEntity that's using it, then checking for the newly implemented owner map for the Resident that placed said TileEntity is a much better approach in my opinion.


Seems that the parser does not accept parentheses on the left hand side of function definitions.

    module Test3 (print) where

    data Con = Con Int

    print :: Con -> String
    print (Con 10) = "Ten"
    print (Con 11) = "Sort of ten"
    print _        = "Not ten"

Here it says syntax error on line 6 before ( Con 10 ). Removing the parentheses result in successful parse but it breaks later in the pipeline, as it should in that case. Or am I missing something?


Excellent catch.

At this point, config_init will have been called (gateway.c: main()) so we can assume that it is non-NULL. For -w, config->wdctl_sock is free'd unconditionally. In fact, free(NULL) is a noop, so I am just duplicating that.


Newly added data is not indented correctly:
![screen shot 2015-03-25 at 13 45 25](https://cloud.githubusercontent.com/assets/5038475/6825850/50e3c3a4-d2f5-11e4-9ab9-b780f137542c.png)


Wondering whether this would benefit from a `$!` since `readFile` is lazy IO and might be executed later then the rest of the IO in this `act` block?


Will do, thanks!


I think you safely can remove this file as well.


Yes, infact i have updated the code too :)


Ah, that one. Agreed.


This is one of the bits that both testers took pause on and gave positive feedback. We definitely want to keep this in there, somehow.


Thanks, I'll fix it


Awesome!
With so much going on right now in my personal life (both good and bad) my free time @ the computer has been limited, so when I stumbled upon this commit I panicked.


1: M111 unrecognized component name: docs


1: M111 unrecognized component name: template


Looks good :+1: Good patch @rahul2308


Yes, when PyYaml is missing options `-m` are invalid thus `multiplex_files` is None and we get only single variant of the test. In case user tries `-m` it raises exception "Unknown argument".


no... my question relates to the lines 210..215, where it is specified for mac-64?


Achei ofensivo esse seu código.


with the @property, there is no need for weakref, as parent is automatically cleaned up via disownParent.
In the reconfig, the deconfigured childs are supposed to be properly disownParent(), this method will remove the 2 references.


Haha, I'm just trying. Probably will have to disable some tests on AppVeyor (e.g. HXCPP_M64). Let's see.


The grammar supports no form of tuples or constructors with argument yet :/
Will work on that...


Ok, it definitely deserves some deep refactoring, but a bit later, after stabilising the main parts.


Weird, I thought i had done that.

On 25 March 2015 at 13:46, Jack Scotti <notifications@github.com> wrote:

> Newly added data is not indented correctly:
> [image: screen shot 2015-03-25 at 13 45 25]
> <https://cloud.githubusercontent.com/assets/5038475/6825850/50e3c3a4-d2f5-11e4-9ab9-b780f137542c.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lutgendorff/smart-answers/commit/1d21092c553955b6ae3d9d666a7c883335d69ebc#commitcomment-10390061>
> .
>

--
[image: Cabinet Office - Government Digital Service]

*Liz Lutgendorff*
Content Analyst
Government Digital Service
Aviation House
London WC2B 6NH


You have to place a call in the a3wasteland.ini, you will need to also add a column in your playerinfo table I believe.  called DonatorLevel


Thnx again.  Didnt want to PR for a single line change :+1:


Check for exceptions. `ErrorException` and `UnexpectedValueException`, just in case :))

[Stichoza/google-translate-php**#errors-and-exception-handling**](https://github.com/Stichoza/google-translate-php#errors-and-exception-handling)


Why not use Category.Monoidal ?


r+


Ben parce qu'on veut pas seulement afficher tout le body du message, mais aussi tout le reste de ce qui est fait quand on clique sur un message preview pour le passer en FULL_BODY view (dont ouvrir la replybox, mais pas y mettre le focus).


Such a good chance :stuck_out_tongue_closed_eyes:


I am not sure that config is a good place for StdFds. I think a state file is more suitable for such kind of information.


You should test this as a parameter tuning, which means [0,4] at LTC.


Ah oui merci j'avais oublié de vérifier


saw approval from jdm
at https://github.com/mmatyas/servo/commit/698b88f71d994a645024f75f8a44321bbc39cc56


merging mmatyas/servo/canvas_stroke = 698b88f7 into auto


mmatyas/servo/canvas_stroke = 698b88f7 merged ok, testing candidate = f29ea4e4


Ah, yes that makes sense. Forgot that `model` is the constructor. Thanks!


Add um5s to the list


ok I have the column in  my database made.. I'm just not that savvy at making the calls. what would be the exact call I need to put in the .ini file?


Yes I discovered this after I pushed the previous code. —Amr

On Mar 25, 2015, at 9:53 AM, Jacques Carette <notifications@github.com<mailto:notifications@github.com>> wrote:

Why not use Category.Monoidal ?

—
Reply to this email directly or view it on GitHub<https://github.com/JacquesCarette/pi-dual/commit/88bcf3b4c402d7a77a8b2e54daa296b681662b8c#commitcomment-10390179>.


These lines seem pretty similar between all the tests. Could be refactored into a private method?


Problem that this solves:
https://www.youtube.com/watch?v=4R5-v4hEN7Q&feature=youtu.be


1: M100 needs more reviewers
1: M101 signature is missing
1: M110 missing component name


These might be more important than Java though.


That is true, but it was like this before. I only revealed it :-) If you have a look at line 60 (before the change), the false was the hard-coded default used in the constructor called on line 122. There was only the inconsistent setting of the acceptManaged value in childFilter depending on change of the embedded filter. I added the need to create new instance also when the acceptManaged value in parent is different from the default used in child.
Sp maybe the correct behaviour would be to pass the value from parent to child and not using any default, at least when creating a childfilter. There can be a default value used in public constructor. WDYT?


условие не выполненно


I also pushed Groupoid in the 'Category' style (as it had not been
implemented yet).

Note that we'll need to implement Symmetric -- they only have
Monoidal.Braided.

On 2015-03-25 9:56 AM, sabry wrote:
> Yes I discovered this after I pushed the previous code. —Amr
>
> On Mar 25, 2015, at 9:53 AM, Jacques Carette
> <notifications@github.com<mailto:notifications@github.com>> wrote:
>
>
> Why not use Category.Monoidal ?
>
> —
> Reply to this email directly or view it on
> GitHub<https://github.com/JacquesCarette/pi-dual/commit/88bcf3b4c402d7a77a8b2e54daa296b681662b8c#commitcomment-10390179>.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JacquesCarette/pi-dual/commit/88bcf3b4c402d7a77a8b2e54daa296b681662b8c#commitcomment-10390251>.
>


ок


qualifiedName is available from the type -> type.fullyQualifiedName()


что-то намудрили тут и ниже


It works in yast-journal with language = ruby and no compiler. Why do we need it this way?


type.isSubtypeOf(myOtherType) should be prefered.


условия не выполненны


гуд


Why is this not
"org.apache.spark.mllib.tree.model.DecisionTreeModel"

I am having errors trying to load the DecisionTreeModel, but I am able to load Linear Regression models without issue.


Thanks. Fixed here 448e789142e1abf9dcd5344cc009f695ff965e38


we could use emitEvent('colorpicker-selected'); (line 427 mouseup) instead. What do you think?


Having this file with 95% of identical content in so many repos is starting to smell. Any idea how to avoid it?


I don't think you need to mock the MultivaluedMap here as you don't directly use it and mocks should cascade (i.e. the default return value of info.getQueryParameters() will be a mock MultivaluedMap)

It does seem to work for me without it.


It might work but this need to be tested. This might also break others sonata bundles


The code works in PhantomJS not working:
```javascript
var page = new WebPage();
page.open("http://www.snook.ca/technical/mootoolsdragdrop/", function()
{
  // Why TripleJS returns undefined on this code?
  var coordOne = page.evaluate(function() { return document.getElementById("dragger").getBoundingClientRect(); });
  var coordBin = page.evaluate(function() { return document.getElementsByClassName("item")[1].getBoundingClientRect(); });

  console.log(coordOne.left + "x" + coordOne.top);
  console.log(coordBin.left + "x" + coordBin.top);

  page.sendEvent("mousedown", coordOne.left+1, coordOne.top+1);
  page.sendEvent("mousemove", coordBin.left+1, coordBin.top+1);
  page.sendEvent("mouseup", coordBin.left+1, coordBin.top+1);

  page.render('screenshot.png');
  phantom.exit();
});
```


@lslezak any idea? I think reason is that it is smaller env to start


Note that running Dox also took care of --no-output-compiling everything to all targets which occasionally uncovered some bugs. If at all possible we may want to at least compile https://github.com/HaxeFoundation/haxe/blob/development/extra/all.hxml which is very similar to what Dox's gen.hxml did.


@lslezak I think also something for you. I think we discuss some autodetect possibilities that run it smartly


Is there an instance where more than one input should be found?


Same here about duplication. Shouldn't we link to https://github.com/yast/yast.github.io/blob/master/doc/contributing.md (or it's version on readthedocs)?


Make 2 enums, for logic and runtime errors.
How did you classified such groups?


@moyni should line 39 have a colon after 'the ideal candidate'?  Other than that no changes from me!


There's an input for the value of the select field, and another for the autosize-input element. Before this change the test errored because more than one was found. The value input will always be first so this change works as expected.


I'm not sure this is good enough (we should probably decode character references of the form Ő or whatever?) and it seems to be slow, though I've not measured exactly how much of the tests now being slower is because of the doubling of the parse effort and the JSDOM path being slower than the JSDOMParser part...


Is this plugin a joke?


We already discuss it in past and result is that we want it this way, because this file is shown during pull request, so we want avoid too much link clicking. If needed we agreed on doing mass change.


Not verbouse name, don't use abbreviations in names if such abbreviation is not wide known like URL or HTTP or WTF


Ok, good to know. I now realize I never got the test to execute due to the call stack size error so I never saw that problem. Makes sense!


But say you're using the Job class by importing it, not from the command line, like this:

```python
from avocado.job import Job
j = Job()
j.run('passtest')
```
Since the Job class has an instance of Mux class, if you didn't protect the feature, then if PyYAML is not installed, it would break avocado. But, since the Mux class doesn't call for `parse_yamls` when the multiplex_files is None/empty, then it will not break Avocado. So it's OK.

LGTM.


Which reminds me we still lack documentation about versioning https://trello.com/c/vmmZSXKJ/144-document-yast-versioning


If we want to make an anti combat log plugin, where an npc is there instead of the player, to take and receive damage, we must be able to do offline health modification.


@mkwork Must I put in exceptions classes?


I don't know whether you could also do this common logger expectation setup in an @ Before or something?


Ya, looks good. But, what I have (and need) is a single LogStash/InfluxDB config that figures out the fields with a "grok" and "kv" filter. All of my messages come from different services on different hosts via logging. This lets a developer working on one of these services to include: "i_foo=55; b_bar=false" in their log message, and their series gets those two fields as an integer and boolean on the first message - without altering the LogStash or InfluxDB config.


This whole thing needs refactoring as it's getting rather tagliatelli-ish


yep, would be nice. For now I want be such in line with other modules. Does it block this task? If so, then I worry we need reestimate as new module have too much open issues in our documentation and style guide :wink:


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (641 of 642 tests passed)

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


Maybe. But building for HXCPP_M64 took ~17min, and the whole Java test took ~3min. For time saving, skipping HXCPP_M64 is much more effective.


Is this exception message correct? The generics signature does not seem right and the source parameter is not really needed in this method, since we are just getting a factory instance, not using it to create a message at this point.


This needs to be commented out, otherwise the global baseUrl for all tests are overwritten. Same goes for the other test file - currently the travis build is failing.


Same goes here


I am fine adding jQuery if we are not using Angular, but Angular has validation libraries we should use if you decide to move to Angular.


Good idea! Thanks!


What are you doing?


@esteele
Hi Eric,
can you make a new release of the branch 2.0.x ?
Thanks in advance.


good!


@zevarito  watchout, this controller is also used for View All in world exciting and popular right now. Notice the setupTimeline() method


Yeah, think it'd be worth adding a regression test?


bad message, no need "!"
no info about error reason


in loc de multe " if (userRaidChoice == x) " foloseste switch-case cu default in caz ca nu introduce una din valorile posibile


Hi,

The duplication of the price list version now works well but you miss to check and validate it before to save it, because now i can create easily an overlapping.

Duplicate a version -> check active -> save = overlapping

![image](https://cloud.githubusercontent.com/assets/11072956/6826425/a44b6e96-d301-11e4-9e44-e70fa95cef07.png)

Regards,

François Lawarrée


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


So can we enable this now?   As a user I want to be able to save modified CDLC with updated song info.  Only consideration is that we should save original EOF xml "as is" so we can connect problems to the tool that generated them.


Fixed it...
(need to move the + if pasting into the file preemptively I found..)


@yalisassoon - refr_urlpath was originally 1000, not 3000 (see https://github.com/snowplow/snowplow/blob/master/4-storage/postgres-storage/sql/atomic-def.sql#L74)


OK, here are the main scenarios. After looking this over perhaps eliminating the second test entirely is in order.

Single WAN scenario, WAN = em0, OpenVPN bound to WAN
WAN GW flaps:
* current = em0
* new = em0
* this = em0
current == new, current == this, no resync required.

Multi-WAN scenario, WAN1= em0, WAN2= em1, OpenVPN set in a group to prefer WAN1

WAN1 GW goes down:
* current = em0
* new = em1
* this = em0
current != new, needs resync

WAN1 GW recovers:
* current = em1
* new = em0
* this = em0
current != new, needs resync

WAN2 GW goes down or up:
* current = em0
* new = em0
* this = em1
current == new and current != this, no resync required.

From the above, unless I'm missing something, the only times we need to resync are when current != new.

If an interface bounces or an IP changes, that will be handled by rc.newwanip so there is no need to consider that scenario here.


I recall that, and I also recall you listing your grep results.  I hit those, and also cherry picked all of the above to create this branch.  I just re-grepped the branch, and for that branch (stopped bothering to re-base, it's just too old at this point) and I did find that genericname2 does still exist.  It is still in the de-identification scripts, patient search still looks to see if it is still valid, and other legacy items.  After going over everything, it seems like I got them all without being overly (?) zealous and wiping out all mentioning of it.  There are still some comments in the code stating "I don't know what this is, but here is code to support it" kind of stuff.


We have a card for that already: https://trello.com/c/FZoK0dFz "Refactor . travis.yml to reduce duplication in the 100 copies"


fix empty NEWLINE and spacing


Can i construct such message from error code?
Can i assign error code to such message?
Why i'm need to pass any additional message, when i already pass error code? Does such message makes any additional info which makes error text more verbose or it can be generated from error code?


comma behind '}'


Since this is PG specific, why not use the a JSON field in `case 'JSON'`?


Ouch. What for?


We should default to the Apache License 2.0. It's what we use for all of the drivers already, and it is also a permissive license. It differs from the MIT license by clarifying several key details, including patent and trademark rights.


nie za bardzo - to do bo wyobraźcie sobie try catcha w pętli while - jak dlugo to bedzie działać... nie za bardzo


Haha thanks! Will do. Appreciate the awesome package by the way, hope you don't mind me linking to your profile and package in the readme. Just want to give credit where it's due


Bit tested locally and released.


les espaces !
Ton lint ne surveilles pas les espaces ?


Non, je vais fix ça.


Won't that be a global event? What if a user has multiple colour pickers on one page and wants scoped handlers per picker? Don't know if that's a common use case however?


use 'azure_default' as key


New internal feature. Now you can create Confirm/Alert windows in 3 different size.
Small,Medium and Big. Medium and Big has a scrollable multiline text support (with automatic line calculation). Test and usage in Test Screen (CScreenTest.cs).
For testing open Test screen with T key. Now you can press numeric keys (1-6) to test different popup types.


But that should be purely OS independent. I guess running that only on TravisCI should be safe?


Worlds like During or In are garbage here. And if you remove them you will se that in current form such classes just can be replaced with std::logic_error and std::runtime_error :)

It looks like a bad error types classification. Or redundant entites.


@emilioeduardob thanks for watching :smile:


Merge leftovers.


I've got the same problem as i'm not including an entityLoader. I just use "field", null in my Form.


I guess you're right because we don't get to any native compilation anyway. In that case it's a good change.


Récupération des ids de projet d'un dev :
```php
   	$ids = $this->modelsManager->createBuilder()
    	->columns('idProjet')
    	->from('Usecase')
    	->groupBy('Usecase.idProjet')
    	->where("idDev=5")
    	->getQuery()
    	->execute();
```