I added the minimal tests for now, since there were none before, and part of verifying that the migrated code works.
I didn't want to get into the rabbit hole of testing all the old props being set or not, and what values are really expected.


Changed the background color.


:sunglasses:


@adriatic The dist folder is ignored but still committed. I'm not sure what's best git practice to solve that.


Blank


Further info on these changes:
* IE7 is down to ~0.2% of GOV.UK visits and is definitely out of our 95% cut-off point.
* Edge is firmly inside the 95% and is slowly increasing.
* Safari 7 on OS X is lower than IE7 at this point.
* Combined Android Browser totals dropped out of the 95% this week, but we’d like to wait a while to confirm that trend before removing them completely.


String blank body


Didn't know that :disappointed:

```ruby
h = Hash.new([])
h[:foo] == []

h[:foo] << 1
h[:foo] == 1

# and now
h[:bar] == 1
h[:any_foo_bar] == 1
```


Shouldn't this TODO be removed, then?


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


Raw


you could add a break here so we stop checking for directions once we've found one


The alignment is still bad...

![skjermbilde 2016-01-15 kl 12 25 17](https://cloud.githubusercontent.com/assets/227836/12352068/162671ea-bb83-11e5-9de1-beb728d1f97b.png)


whitespace issues


Check the ternary in L186. For remote paths `request` is used (not `createReadStream`) and it returns a stream.


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


>Sorry for the inconvenience but I've no idea how that happened.

Hey @dd721411, don't worry, no inconvenience here, I was just wondering.

>I was translating your app Android IMSI-Catcher Detector on Weblate last night. I have finished some strings in Vietnamese but it seems like those strings haven't been updated on GitHub.

Ah, I see! Just checked the strings on Weblate, they've been saved successfuly. Thanks for your work!

>As I'm a noob here, could you please tell me how to fix the problem?

It will solve itself since Weblate pushes changes automatically once enough strings are translated. Guess you'll have to continue your translations a bit so that Weblate feels that it's about time. ;-)


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


This introduced a bug, since the new visit_with wasn't equivalent to the old _visit_with method.


you pushed to master!!!!!


Renamed to ArgumentParser


What page do you need this on? I could move these all to the main function that's run on all pages.


Why the setTimeout() ?

This is usually a bad practice.


Thanks, @Bobbyjoness,
Sounds really interesting. Could you add to this fort some benchmark tests comparing the gain/improvements of the whole library in terms of fastness of execution (I believe there might be a small improvement in terms of speed accessing the grid map) and also in terms of memory consumption ?
Much thanks again.


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


Just noticed you may want to check your git config as your comits aren't showing as being linked to your Github account.


@nebhale : was reverting to `@After` intentional? If so, what was your reasoning?


Updated.

(branch cgroupns.v9-tests)


Hi, since this optimisation i crash in function GetHashCode when it returns "return hash1 + (hash2 * 1566083941);".
I don't know why.
If i replace comparer with "StringComparer.OrdinalIgnoreCase" all works fine.
Can you help me ?


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


This has nothing to do with this patch (which, incidentally, was merged four years ago, so this question should go to the users list). The code here is not designed to set/update attributes. It's purely to enable a policy of checking if TLS certificate values are correct and return an accept or reject based on that.

Just do your check/update in the inner-tunnel post-auth.


optional_vars was undefined here..


Time Table -> Timetable


ビックリマークが1つ多いです。直してください。


:+1: For the usage of C++11 features :)


What's a Sticky app?


FactoryGirl has a `create_list` method... you should look into that.


I returns an error at line 126

TypeError: data.entry is undefined
return data.entry.media$group.media$thumbnail[2].url;


testing


cannot merge:
 - "i386 Linux" state is "failure"
 - "AMD64 Linux" state is "success"


Gute Arbeit!


Minor, but I find it kind of odd that we have a `.strip` here for this argument but not the other, and vice-versa in the `Introvert` method. I'd prefer moving them both to `Introvert` if you don't mind.


I don't see a `POSTS_PER_PAGE` constant defined in `BlogPost`


```
Give the runtime system a hint about how much CPU time the current NIF call has consumed since last hint, or since the start of the NIF if no previous hint has been given. The time is given as a percent of the timeslice that a process is allowed to execute Erlang code until it may be suspended to give time for other runnable processes. The scheduling timeslice is not an exact entity, but can usually be approximated to about 1 millisecond.
```

I dont really understand how this `timeslice` figure represents cpu time used here.
If we decode a 10,000 string in one pass of this do while loop then
10*(10000-7500)/4/6000 ~= 1
if we decode 10
10*(10-7.5)/4/6000 ~= 0.001
So i think however much work we do we're telling the scheduler pretty much the same
https://github.com/erlang/otp/commit/ef3566f48e08#diff-0b759333d868109feae99e131985bbadR1451


Curiosity about this kind of sensor: does it output values map to integer value easily? I mean, what are the readings returned? Couldn't it return fraction values? If so, maybe it should output three 'raw' float packets, and user is up to convert to an RGB packet using `converter/float-to-rgb`.
Or values read are always integer?


The len parameter is unused. All this use of fgets and sscanf makes me uneasy.


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


Have we benchmarked how long it takes to do decode 10,000 char? As if its < 1 millisec then we may not even need to report this at all


This is great! Would you mind adding other tests for the cases I mentioned (see the `JimJohnJames`/etc. examples in this file).


I need it on controller="config", action="index". (/config/general/).


You know i have killed before?


Committed to trunk


Why would you need to load `remoteShows` for the config page? That makes no sense. You seem to be fragmenting the whole project now.


Playing it safe, eh? 128 bytes to write like 12 bytes at most? :)


great information, good to have!


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


Solving problem of string value convertion: {"a":"test"}  - >  <a>"test"</a>


close #51


Why not just `sprintf(knob_path, "%s/cgroup.procs", knob_dir)`?


Same here.


А если исключение возникнет... такое не может быть? как sqlite к этому отнесётся ?

А если в столбце был NULL то оно не попытается на него сделать юникодную операцию?


if we are after the performance (i came from #7014), in addition to the refcount bumping, the "dynamic_cast" also introduce some performance penalty because the C++ runtime need to access the RTTI for the object to see if it is a valid cast or not.

but i understand that we can not avoid this by either shape CollectionHandle a base class of the "Connection" of the different backends, because they are too different to extract a common interface.

just want to mention that "dynamic_cast" is not tax free.


on peut retirer à du coup?


Grammar: "the fact that i had been" capitalisation of I
Perhaps new sentence "For me the most significant issue was not practising the practical side of programming, at first I felt lost,[new sentence] this might be due to the fact that i had been out of education for an extended period of time.


@stefanpenner can you also fix 1.12.x? It broke the code of the main app of the company I'm working for (not production tho). Really annoying. Now I have to fix and check out everything works properly.

Maybe I'm confused, but should you not create a new tag for every patch?
http://semver.org/

Tnx


@cbenz : Quelle est la raison profonde ? J'ai besoin de mieux comprendre pour savoir éviter ce genre de problème. On va être amener à générer des variables à la volée plus souvent, notamment dans des extensions et/ou des réformes.

ps: Me dire si commenter ici te va ou si tu préfères par mail pour ce genre de point (c'est aussi pour vérifier que tu les vois).


This change will prevent the harvester to import all the records if a server returns less records than requested (see also the warn on line 301).

Guess it should be
```
   start += returnedCount;
```

Also see this related commit https://github.com/geonetwork/core-geonetwork/commit/3a974c68ed3c4fc7e083dd74d7b36358ef85c463 (sorry for reformatting the file) which removed the bad behaviour.

I guess we should add some more harvester configs to deal with misbehaving CSW servers :(


Are you sure that `in_intensity_both` unit is Lux? I thought that `in_illuminance` returned lux...


Shouldn't this be 4.5 now?


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


So, probably json should be update to say that the values are 'unit-less'.


Bien Manuel!   ahora podriamos probar subir clases Java?


Is it possible to have a release at this point with split recipes? The reason being via Berkshelf only version 1.5.2 is available in the supermarket.


backdoor interface must not be exposed to the public.


@tackerm , I replied you here: https://github.com/liip/LiipFunctionalTestBundle/pull/226#issuecomment-171938040


you use an IDE (PHPStorm) but i only use a text editor.. Lol


Excelente!!


Hi mate. You added unnecessary obj when you cherry-picked the commit.


Acho que o`before_action` no topo do arquivo, logo após a definição do controller fica mais legal não?


ここ、代入しなくていいですね


Rebar seamlessly get fetched during ./configure and this one will be ignored.


Please always use braces for if-statements.


ДУРАК


This change will make the harvester end its work earlier if the server returns less records than requested.

Also see this related commit 3a974c6 (sorry for reformatting the file) which removed the bad behaviour.


Надо добавить синтаксический контроль в Sublime русского языка, а то уже поднапрягает ;-)


`lang` -> `expectedLang`


`lng` -> `lang`


No necessarily for this patch, but on future maybe you could get an option stating how many channels are available, and add "in_voltage0", "in_voltage1", etc., instead of hard-coding only one channel.


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


I added a test for this in https://github.com/mapbox/mapbox-gl-native/commit/5fe586633cb15c99f77ca478446679acf42fddd1 and it didn't break after this test. That said, we have a very high number of different states and execution orders, and I'm sure some of them are broken. However, this is a separate issue: https://github.com/mapbox/mapbox-gl-native/issues/3569


I would use waitpid, but that's probably not so important anyway.


`foundSeriesIds` -> `expectedSeriesIds`


Do not forget that the unit is 'milivolts (mV)'.


And use an output port array, naturally.


please also add an empty string as a default value here


ok, updated.


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


https://github.com/erlang/otp/blob/ef3566f48e08481821eee1c7260cdd4ca05fdefc/erts/emulator/beam/erl_nif.c#L1454-L1457

So the "percentage" we give if transformed into a number of reductions (reductions ~= 21*percentage)


Why not? :)


You are checking if malloc succeeded, but what about failures of the `eventfd` calls?


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


ok


@BridgeAR Fair point, i'll ammend it.


```
In the Beam emulator, the reduction counter is normally incremented by one for each function and BIF call. A context switch is forced when the counter reaches the maximum number of reductions for a process (2000 reductions in OTP R12B)." (still 2k in R16B)
```
So 1 percentage in `enif_consume_timeslice` is indicating the call was 21 times more expensive than a standard erlang function


yes, changing...


Also zwei Templates schreiben, eines für Windwos und eines für Linux?


Hi, can you create PR for this commit? :)


@allustin пинганул бы для приличия)))
в Атоме вроде есть плагины по проверке правописания. Можно погуглить/дописать свое.


That's misleading. Maybe it should be named `actor_id`.


@saintedlama The few places I looked before commenting on the PR used `!foo` already - this is what prompted me to make the comment. I didn't see any place that used `typeof XYZ === 'undefined'`, but didn't look that hard :smiley:


Thank you sir.


Resolve #61


I've implemented the use of a new `ALEMBIC_MAYA_PLUGIN_EXPORT` symbol in d6d20a92458053e20bf3c881580a5c8cb1464bae

It's not configurable because it will always need to be turned on. My intention was that if there needs to be configurable exports for whatever reason then there would be a separate `ALEMBIC_MAYA_EXPORTS` option for doing that.


Pretty, and backwards compatible, the old parser really didn't check context: https://github.com/RigsOfRods/rigs-of-rods/blob/bd680916356ea033716c96582305a37347644b9a/source/main/physics/input_output/SerializedRig.cpp#L2750-L2812
This is another of my super-optimistic per-documentation designs :relaxed:


Сеттерів тут не треба... тобі треба конструктор написати, який прийме code (int) рангу


I am confused, why does this work? to check if it breaks in @_set_data... and later use @valid_data as a sentinel flag?


We need to run benchmarks to prove that we're not taking > 1 millisec for any possible 10,000 input if we're ignoring the return value of this call


Well, I always obey the rule "if I can easily guarantee (forever) something, I don't care about anything less than the guarantee" and therefore I've suggested the double quotes :)

Regarding performance hit, there will be some e.g. on GNU platforms. But not because of some formatting (that's totally negligible in the context of shell scripting), but because `echo` is built-in command in bigger shells (e.g. `bash`) and therefore there is a huge difference between running a routine in an already running program and forking and `exec()`ing a standalone binary. In my eyes it is though worth the penalty (in case we would have some tight loop, I would consider some other solution, but still POSIX-compliant).


Why not to use init list to init the impl_? By the way. Seems the `#include "utils/pimpl.h"` should come first


I am not sure that you should be using internal URL's in here. I would recommend that you make this configurable in the Plugin but not in the package. Then we can have custom help and install URL's...


WTF MAN?


Ok, I'll create a new PR.


You should mark these as not serializable!


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


The target_pid is also a misleading name.


I think it would be worthwhile writing some simple `proper` tests for this module, we should prove that
1) string == encode(decode(string)
2) decode(string) == base64:decode(string)
3) encode(string) == base64:encode(string)


@nixel2007 тут такое дело - пока только Sublime поддерживается в части редактирования
это я больше для себя заметку сделал


Ok, will add this to the wiki.


cannot merge:
 - "i386 Linux" state is "failure"
 - "AMD64 Linux" state is "failure"


hello，world！


No idea. Really dumb commit message. Ive learned to write better ones.

My best guess is that my previous method didnt factor in leap years/seconds and DST


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


Seems dtor is redundant. Body is empty. Propose to remote the declaration too.


Line 72


same here - this test also changes resolution inside.

I guess it's better to call it `resolutionChange` not `resolutionTranstion`


this block should be kept. the idea is that all additional fields are reset during logout or login by another user


Хорошо ли это, ставить процедуры на уровне бд? На сколько мне известно, если веб-приложение можно маштабировать, то с БД это гораздо тяжелее, поэтому в своей практике запомнилось делать валидации и т.п. на стороне приложения. Как считаешь?


Popping from the front of a list is `O(n)` rather than amortised `O(1)` when popping from the end.  Use a `collections.deque` for a fifo queue.


тут точно нет ошибки?


@allustin ок.
если что-то будет нужно/можно добавить в пакеты для редакторов - ты знаешь, куда писать :)


Ill remember to include a repro example in future commit messages


Again, if you want fast insertion at the front of a queue-like object, use a `collections.deque`


*CHEERS*


Instead of `getpid()`, you probably could use `children[id].pid` and use `target_pid` variable in `sprintf` below instead of calling getpid() again.


Check for failure?


You're right. I fixed this at 34323776693ec25ce33e8d71992a5e044ec4c828.
Thanks!


Euh non. Le séparateur est bien ", à ", qu'on remplace dans sa dernière occurrence par " et à ".


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


Propose to rearrange the code this way
```
/// utils::Pipe::Impl
where here Impl declaration and implementation.
class utils::Pipe::Impl {....};
it's methods
/// utils::Pipe
and Pipes methods
```
This way the Impl declaration and definition will be close to each other.


Note to @wence- and myself: this is also performance problem then.


In django.contrib.auth.models for validation of username field using '^[\w.@+-]+$' regexp.


Hmm this is still wrong :(

Need to cater for players with names such as [ Che, Chesster, Sho, Shouter ] probably more simpler would be to use a isplayer condition.

Investigating ..


`handle_` is not pointer. Assign zero.


"foo, à bar et à  à foo"


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 67](https://teamcity.labkey.org/viewLog.html?buildId=297038&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 34 Build time: 01:05:53
### Failed tests
```
org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

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


And use zero everywhere for the `handle`


Thanks, I didn't know. However, the ordering within a bucket - I think - is irrelevant, so I'll just pop from the end.


il faudrait +4 plutot que +1


In kernel, 4.4 loads trampoline without problems.


(ou juste users.substr(0, comma) + ' et' + users.substr(comma+1))


Yes. https://github.com/caskroom/homebrew-cask/issues/15603.


the method does not the same then my code, take care of arguments[1]!
e.g.:
````
    count($arguments) < 2 && (!isset($arguments[0]) || is_scalar($arguments[0])) && (!isset($arguments[1]) || is_scalar($arguments[1]))


Oulha, oui, sorry. J'suis pas réveillé.

Je regarde pour Travis aussi, mais je ne comprends pas (à vue de nez) pourquoi il ne trouve pas le div. (L50 de tests_helpers.py)


It looks strange that in these if blocks there is no return. Is this old and tested logic? I mean, if ,e.g. we have failed to close the socket, is it right to call `unlink`?


Here I can just simply append, and then reverse the list at return.


what about changing to `if (!bytes_to_write )`?


Thanks - will comment further on that issue.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 91](https://teamcity.labkey.org/viewLog.html?buildId=297031&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 16, passed: 277, muted: 1 Build time: 01:43:08
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

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

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


add comment `// namespace`


No, it's because they're required to implement the interface.


Well i cant now cause dan just put the whole merge request on ice.


I'll make some changes when I get back from holidays (~3 weeks)


thanks


doesn't it suffice to simply declare impala::fancy as extern? Is it really necessary to have this wrapper function for msvc?


"an image" instead of "a image"


Yes, I agree. Silly to change this here.


Классы обычно именуются с большой буквы. Переменный с маленькой, чтобы не путаться.


I think the similar line needs to be added in https://github.com/windup/windup/blob/master/test-util/src/main/java/org/jboss/windup/testutil/html/TestReportIndexReportUtil.java#L46 to pass test WindupArchitectureMediumBinaryModeTest.testRunWindupMediumWithFernflower

```
Tests in error:
  WindupArchitectureMediumBinaryModeTest.testRunWindupMediumWithFernflower:64->validateReports:193->validateReportIndex:180 » WebDriverException_$$_javassist_345501ae-a4ba-4228-aa16-0f6498eeee06
```


Какой здесь модификатор доступа и почему не паблик?


Yes, the above I can absolutely do.

I have no experience with integration tests for bash scripts though so any guidance on how the test framework works would be appreciated.


parce que c'est un div. Essaye . au lieu de text()


Where's @aodzip LOL, Safari lags with 0.1 fps here!


Multiple empty lines


Why Impl have to set Pipe as friend?


A const is non-serialized per defnition, I would have thought?

However I actually would liked to have these serialized, so that the admin can change the download and help urls that gets presented to end-users in the error message.


@samdark I just search book title on google and found cover with much better quality. This only affects PacktPub books - others are fine. Those I found:
https://www.packtpub.com/sites/default/files/2277OS.jpg
http://is3.mzstatic.com/image/thumb/Publication/v4/04/31/11/043111c9-3a5a-7419-84e0-e8861ab1e4b8/source/1118x1400sr.jpg
https://www.packtpub.com/sites/default/files/8727OS%20Agile%20Web%20Application%20Development%20with%20Yii.jpg
https://www.packtpub.com/sites/default/files/7734OS_Yii%20Project%20Blueprint_Cover_0.jpg


what is this text?


i use github and notepad ++


Remove this empty line


oops, committed with your user Paully, sorry... (from PLR server)


I think issue #6 is related to this line, right? Why did you changed this param?


i will fix these things later today i am busy right now


Raczej powinno być:
```php
$access['table'][50]['allow_reg']	= '^re_';
```


Why not to create `SetName` method? Or make just this name public.


Thanks!


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


shouldn't it be `if placeholder not in request.toolbar.placeholder_list` ?


propose to use `if (!ConnectNamedPipe)`


Hi, I am trying to use run_continuously in conjunction with schedule.  So that I can run some program when ever a button is pressed but the other program will run once everyday using ""schedule.every.day.at("10:30").do(job)"".  My attached test program is attached, please can someone put me on the correct path for make this work. Many Thanks Jameo
[run_continuously.txt](https://github.com/mrhwick/schedule/files/91839/run_continuously.txt)


right click menu


Same as in the posix version. Is it ok to use two `if` block here without any reaction except the logging?


result is BOOL, why there is check against zero? propose `if (!result)`


external comments


Taches effectuées :
copie des fichiers header.php et footer.php woo-functions.php sur le theme enfant.
modifications des fichiers header,footer,woo-functions sur le theme enfant
création des pages FAQ,CGV,CGU,A propos par defaut.
ajout d'email du site: elego2016@gmail.com


internal comment


reply from MA


It's less a question for msvc rather than CMake and there are issue detecting exportable global variables.
https://cmake.org/cmake/help/v3.4/prop_tgt/WINDOWS_EXPORT_ALL_SYMBOLS.html#prop_tgt:WINDOWS_EXPORT_ALL_SYMBOLS

Sent from my Samsung device

-------- Original message --------
From: Roland Leißa <notifications@github.com>
Date: 2016/01/15  13:26  (GMT+01:00)
To: AnyDSL/impala <impala@noreply.github.com>
Cc: Stefan Lemme <stefan.lemme@dfki.de>
Subject: Re: [impala] msvc compatibility (fc0d022)

doesn't it suffice to simply declare impala::fancy as extern? Is it really necessary to have this wrapper function for msvc?

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


I just change play secret key from default to a more sensible key


Why not? Maybe it already closed but still exists


I know this commit is old, but seen that remains one last reference in the current lines 330 - 332:

```
#ifdef WITH_DTLS
extern resource_t res_dtls;
#endif
```


## Response Codes


This looks quite interesting!


I prefer to compare numerical values with 0 explicitly. If you insist I can change


Well the closing paragraph tag is technically optional, but there's no harm having it.


@rhempel , do you mean readthedocs theme, or the general reference docs?


Because I think pipe should receive name on creating. Its normal behavior for IO device like pipe. `SetName` interface looks like redundant.


first commit


Just for your info (I don't think it applies here), QgsVectorFileWriter can now also produce geometry collections


Oh yes I did!


I would not say that we can see both letters at the same time in a single eye fixation, i don't think so. However, as your eyes moves fast during saccades, you can perceive all the scene globally and this allows A and B cells perceived luminance values to be compared in a relative manner. But the perceived luminance of each cell is a shorter range process.
I would better say that when looking at one letter, our fovea also perceives the neighboring cells. As retina works in a center surround manner, 'A' cell being surrounded by lighter cells can be perceived darker. As a comparison, cell 'B' 's neighborhood is darker and is then perceived lighter.
This formulation should be fused with following phrases (that are ok).


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 203](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=672&buildTypeId=XbimEssentials_XbimEssentials) is now running


@NamelessCoder slap, @NamelessCoder slap ;D
you broke the functionality to use "existing" groupnames like "common" :)


whats the reason  for that check?


OK, that with the options clearly my fault, I missed that completely and relied uppon the existence of this two as they were already in the code, my bad sorry!

I now also think it's better to default to SSL verification, as even if self signed certificates are default on Proxmox VE you should explicitly state that it's okay to not verify it.

Do you already have a patch prepared? Else I would make the "dirty work", I mean it's partly my fault.

Thanks for noticing this!


Thanks.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 203](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=672&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:02:07


The ssl security fix caused a mass rebuild of the kde4 stack and I was looking for the commit which caused it.


Need to check the result of the strdup


Yes, it is optional in the spec. On the other hand validation is hard without it and my editor get's all fidgety. I have bindings for jumping between opening and closing tags and that goes all wacky without closing tags. Likewise auto-indentation flops without.

But most importantly, I needed something that would force a rebuild of the static site to see if Travis is minding its manners yet ;-)


/cc @michaelficarra - it looks like you might want to fix it in Shift as well (tried online).


Сделай репозиторий приватным))


You may declare the variable this way:
```
/**
 * @type {ol.geom.GeometryType}
 */
this.drawType;
```
Then you can use it as a normal variable (no brackets) and it will be typed for closure.


This is not really necessary, but IMO looks better. :)


cc @nicklockwood


I send again the code, because I do it six months ago.


I do not think this is correct since one might still run community database with HA jars in the classpath.
Why do we need this code inside `CommunityFacadeFactory` which has `community` in it's name?


@HonzaKral isn't map already lazy here? I'm just reading code and confused why add `iter(...)`. Thanks!


Do you need the cast?


comment doesn't match implementation


ok


Have you tried `tig`? It's fairly nice. :)


removing title removes popup for tag label added in WINDUP-856


Wouldn't this be more clear and have the same effect?

```javascript
filter: function(feature, layer) {
  return drawingLayers.indexOf(layer) > -1;
}
```


Why are these enums inner and not top level?


Do you need cloning the geometry?


@fneu maybe you could do the variable renaming in an own commit...?


see `test` dir.


Yes, actually I've done something like this in a subsequent commit.


Unfortunately, the number of maximal Docker container the server can run has been reached, or almost reached. Yesterday's tests finished at 20:15, after a start at 21:36 the day before.

I will try to dispatch our Docker images to several test platforms.


No sure, how hot/dot affected by source/target stats and were to get it.
Btw, S_CREATURE_CHANGE_HP is sending only for those, who had already hit the target.


I totally love `sans-serif-smallcaps`, but there has to be a proper, official documentation for these fonts. Unless I'm missing something, the only way to learn about the supported fonts and their API level by searching on the internet. :(


2 points:

tipo : "natuarally"

You may complete this paragraph with the following italic addons "What you will actually need **when processing a single frame but if you only need steady state response** (and that the example code does), is to repeatedly feed the retina with the same frame, as you would do with a still video. **You can also set retina temporal parameters to 0 to get steady state immediately (*photoreceptorsTemporalConstant* and *hcellsTemporalConstant* parameters of the xml file).**


Update from original


We are not catching this exception anywhere. What is the flow in case the system process id cannot be retrieved?
Also, does this work for processes spawned via the backend? I imagine it is, but please confirm.


I think it would be better to assume this function is called inside a digest cycle.
You could call apply in the calling function.


Yes, because ``ol.Feature.getGeometry()`` return type is ``{ol.geom.Geometry|undefined}``
http://openlayers.org/en/v3.12.1/apidoc/ol.Feature.html#getGeometry
whereas ``this.geojsonFormat_.writeGeometry()`` below expects ``ol.geom.Geometry``
http://openlayers.org/en/v3.12.1/apidoc/ol.format.GeoJSON.html#writeGeometry


Both - I looked at the output of this effort and it's very attractive. Without having too close a look just yet, I am hoping that we'll be able to use this documentation framework to generate language specific examples of each API, and also a language specific overview of how the API fits in with the best practices for that language. It's not just a collection of objects and methods :-)


Nice Organizing. Too bad you removed the summon one cog... Why?


According to the javadoc of `Enum` class I'd prefer `#toString()` to `name()`


Is this method restricted to points? If yes it could be clarified in the name.


Not sure but ``getCoordinatesFromGeometry_`` performs a ``transform`` which is ``in place``
http://openlayers.org/en/v3.12.1/apidoc/ol.geom.Point.html#transform
I feared that it might impact the original geometry in the model...


crawler_init receives the tx_crawler_lib as an argument, so no need to create a new instance.


"olderThanX" should really be configurable. What if I have processes that have a longer run time? Scraping tasks etc. would have a longer run time. Thus, it should be configurable how long active processes are accepted to live.


Not sure to understand what you mean.


This tutorial sounds nice !
I just added few line notes within the text for enhancement purpose.
Thank you for the work. this is nice to see how people do reformulate discussions. You did a nice synthesis !


not sure if that works flawless in all cases, sorting for sourceranges is done based on the beginning, right? So `previous.end` might actually be smaller than `source_range.end`.


unack


Well, it's not a backend, and it's only the mix-in for backends which are "backed by a file" so that's why it was named that way. I'm okay with it like it is.


Makes sense. ``getCoordinatesFromPoint_``?


As discussed:
- req->useUnionStation() optimization --> yes, do this
- findMatchingGroup() is intended to be a private method, but set public for unit tests --> added comment to clarify this
- Pool::asyncGet (not Pool::getAsync) is a close-by place (next in flow) where findMatchingGroup() could be used privately, but it has no idea of requests, union station loggers, and would need the existing "get from pool" to be moved within the lock (although these locks should be very short so maybe that's OK, and in the current methode we'd need to switch to that anyway for safety)


right


In this case it is better to write an assert:
`goog.asserts.assert(geometry);`


Please cast $systemProcessId to int, as we use this to pass it into an exec command, which would be a security vulnerability otherwise.


please make a test where the first range is columns 2 to 5 and second is 3 to 4


You probably also need an `@export`.


you'll also want to test that more than two source ranges are successfully joined


Thanks Matthias! But as I understand such geometries still valid from GEOS point of view and should be correctly handled


*___*
Oh, thank you.


This can be simplified by removing the `@private` and only keeping the declaration.
See https://github.com/c2corg/v6_ui/commit/dc5e4c63d99ce1729e06280c258bd964e2c73b39#commitcomment-15473815


hmmm exposing settings kinda sucks but i get why you did it. Do we still need UseSetting??


I did that, it takes ~12 mcs on my machine to decode 10k bytes, it should consume ~67 reductions during that call.
Knowing that single 'context' have 2k reductions, in ~1ms (time it takes to process 819Kb on my machine), the NIF should consume ~4200 reductions, and yield at least twice.


Is there any possible scenario that the "ps" command is not available on the target system? What if this is no Linux?


What if "kill" is not available / no Linux?


Is this (apparently) harmless message related? #4923
```(EE) systemd-logind: failed to get session: PID 23064 does not belong to any known session```


Please use $GLOBALS['TYPO3_DB']->fullQuoteStr


increases the interval between scans for wifi networks.


Right, thanks for explanation!


Sorry have been busy with other projects recently, but you're quite right..

https://github.com/bee-keeper/django-invitations/commit/01ffd7825ae99823d9ff8c291fda1469b192c265


You are right, I don't need the `iter` call around it, I was just being careful and had my memory of working with non-lazy `map` interfere.

Thanks for the note, I will remove the extra call


actually, could you rather make a `SourceRange.join` function that does the functionality of your function so others can use that also? That's similar to the function that splits diffs.


So far so good! (or [So near so far](https://en.wikipedia.org/wiki/So_Near,_So_Far_%28Musings_for_Miles%29) for you're just getting started with testing and TDD).


Would it be easier to use GeneralUtility::trimExplode(' ', $string, TRUE); ?


No goog! ;)
You may use `source.getFeatures().forEach` instead.


TeamCity Civic Assembly :: Build backend + frontend [Build 4](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=308&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Hi guys,

Just chiming in so that I may get notified when you guys post something here. ;)

@ClxS, I'd love it if you posted your patches to UBT somewhere, as I was going to modify UBT myself, but duplicating the work is not the smartest thing to do. ;)

Regards,

Leszek


I think you a reverting a recent (in progress) commit.


TeamCity Civic Assembly :: Build backend + frontend [Build 4](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=308&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new); exit code 123 (new)) Build time: 00:00:29


TeamCity Civic Assembly :: Build backend + frontend [Build 5](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=309&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


Material flow is NOT in `mm/s`! This could create lots of confusion since the UMO and several other printers measure material flow in mm/s (i.e. filament length) but the UM2, several other printers and Cura measure material flow in mm³/s (i.e. filament volume).


Should I?

`clone(2)` returns an int.


Thanks


In your doc(http://docs.celeryproject.org/en/latest/userguide/workers.html#time-limits), says gevent pool supports revoke(with and without terminate), think you guys should add a note that it doesn't.


you can drop this commit, I introduced this getMessageId() method not long ago and it will be needed soon


$processExists is undefined if $returnArray is empty.


Absolutely, I think here you check the input geometries, this only applies where output geometries are dismissed because of a format incompatible with the produced output file.

It's not yet implemented anywhere in processing, just wanted to make you aware of it ;)


The property is only needed in the controller so I have made it private. Here is the code used in a following commit:
```
  /**
   * @type {?ol.geom.GeometryType}
   * @private
   */
  this.drawType_ = this['drawType'] ?
    /** @type {ol.geom.GeometryType} */ (this['drawType']) : null;
```

Do you think I could simply write:
```
  /**
   * @type {?ol.geom.GeometryType}
   * @private
   */
  this.drawType_ = this['drawType'];
```
?


Ok, will not be integrated.


It can't work that way : if you have a list, you probably need more than one element.


nop, you should copy the whole list


OK
Why removing ``@private``? We have this keyword for all private properties in the current controller and the others.


bravo! ai facut un comit


probably wrong, too


e ok


> Does this imply that any return will cause the binary to become immutable + therefore collectable by the GC?

I assume yes.
See usage of `enif_make_new_binary`, `enif_make_binary`, `enif_alloc_binary` in [nif_SUITE.c](https://github.com/erlang/otp/blob/1523be48ab4071b158412f4b06fe9c8d6ba3e73c/erts/emulator/test/nif_SUITE_data/nif_SUITE.c)


I knew you would say that and have tried first but it said: "impossible to delete while reading".


@nk0leg no. Local scope should be closed before `cleanup()` call.


you should index the list


Right, put this commit is just about refactoring the API.
If i have some spare time, i may implement impacts in Cassandra only.


Honestly I don't understand why goog would be a problem. We rely on it for OL and ngeo anyway...


Good catch, we need to process NULL correctly. I will fix it in the next commit.


same comment for ES only.


you need equality on every elements


ok jelena je push projekat


ok.
Note that it does not change that much because glibc implementation of getpid() caches the pid.


'system_process_id' is of type INT in the Database, so setting it to empty string is not the right type. Maybe set to 0 instead?


this is wrong. This will switch account if the "take-a-break" string is not detected. It should be an If/EndIf inside of the statement.


same elements in different order => objects not equal ? Maybe we need a Set instead of a List ?


asyncGet can return an information object that tells you whether a spawn is needed and what the queue status is at the moment of invocation. No need for it to know anything about Union Station loggers or requests. You can then process the status object later.


`dirname(@__FILE__)` is probably better here, otherwise you're not necessarily getting the version of resources from the version of the package that's trying to run


See my previous comment : it's probably a Set


mark


Next time try to avoid leaving too much space like this. :)


message is not ok. I could help to use a "list assertion" here


You can write just `npm test`. Could you also fix it? =^_^=


Haha, thx dude :)


@tobibeer I read #2180 and although I agree that the scrolling code/behavior can be confusing and possibly buggy, I think my comment *here* referrers to a different problem. I looked at how the story views are written and it seems it grabs the first tiddler from the `$:/StoryList` to manage any scrolling behavior. In the case of an `$:/tags/AboveStory` it would be scrolled off view because it isn't part of the `$:/StoryList`.

After seeing it's implementation I believe this is intentional because the intent for the `$:/tags/AboveStory` is to provide markup out side of a tiddler and inside the story river so that styling can accomplish better effects like giving a backgrounds image or other non-story content.

What threw me off was that the documentation didn't say that. When I first tried my own `$:/tags/AboveStory` I was surprised it was scrolled out of view because I was under the impression that it was designed for content not styling/markup.

If was designed for content then perhaps the `<$list>` widget could be upgraded to take that into account and perform it's scroll-to based on a story list that includes all the `$:/tags/AboveStory`.

With this understanding I can say the `50px` reference above may be a bit of a red herring. Hard coding a value that is the size of the `$:/tags/AboveStory` tiddler does indeed fix the problem (consequently suggesting that the `50px` should be configurable). But that assessment is clouding the true issue in that from the perspective of the TiddlyWiki system any `$:/tags/AboveStory` is **not** considered a part of the `$:/StoryList` which is the sole source of truth that system relies on to perform proper scrolling.

I would love to initiate a dialog on ideas for how this could be implemented in such a way to include them or code an exception for the initial boot up.


The purpose is to provide hook where you can attach when a handler is destroyed and by doing this with IDisposable you are hiding this intent. As a new user who tries to use Cronus it wont be apparent for me that I have the ability to use such hook when the handler is destroyed.


yay I can comment on you


I thought it was useless to Summon 1 cog


`.photos` in the template uses `all_images` as does this code, and that's an array, so I think it should be okay and always be in the same order, someone editing the same report in a different window notwithstanding :)


@afgane The URL hiding requires about 3 lines of code each - and I'm wondering whether it's worth it. Couldn''t find an alternative other than this: http://stackoverflow.com/questions/22616973/django-rest-framework-use-different-serializers-in-the-same-modelviewset
Just noting the solution above in case we want to perform different serializations for list and detail views.


As a best practice, always provide a unit for numerical values—say, prefer `5px` over `5`.

I believe leaving the unit out is recognized by older browser which would fallback to using pixels, but newer browser will probably silently fail and fallback to a value of 0, for `0` is the only numerical value you shouldn't provide a unit to:

``` cs
padding: 5px 0; // Means 5px for top and bottom padding, and no padding on the horizontal axis.
```


Same here regarding whether we are on UNIX or not.


ok


typo. Method name should be "remove", not "removed".


See next commit, I did change `10000` to `8 * 1024` and placed it in macro, potentially macro can be redefined in `rebar.config` so the max size of `CHUNK` can be different, here, we're trying to bump reductions without assumptions on the max size of the encoded string.


@daveh86, this is a little scary -- `strdup` depends on a nul byte already existing, so my guess is that dstrdup can't be used here.


I don't agree, the point of this task is to understand what is the impact of having several MailboxIds, here you don't do that because you don't spread the change. You have to try to propagate as much as needed and see where it will need some more work for next sprint.


TeamCity Civic Assembly :: Build backend + frontend [Build 6](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=312&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


This regex matches all videos from the full episodes section.


Cool! Nice and clean implementation of an important feature. Shows how well the Console UI code is structured :)


Right, thanks.


No need to unset at the end, the stack context is cleaned up when this method is done.


Compatible with previous version :+1:


I suggest doing the apply here: https://github.com/c2corg/v6_ui/commit/3d775793e2ef17ca2a3874f32695eabf8e7653c0#diff-76e7887928fc480e2a6dcd68ecec1ac0R123


Fixed in the next merge request. Thanks for noticing!


The issue I'm trying to avoid is that if we bundle and distribute precompiled packages, `@__FILE__` is going to point to the wrong place.


TeamCity Civic Assembly :: Build backend + frontend [Build 6](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=312&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:20


Just in case, this file should not be committed


this file should not be committed


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 90](https://teamcity.labkey.org/viewLog.html?buildId=297036&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:13:52


@dra14825 do not * import. just import what you really need.


gradle folder should not be committed


I can't remember having malloc in CF. When did it happen?


@dra14825 please capitalize only first letter of acronym: *classUuids* instead of *classUUIDs*


`Pkg.dir` clearly will, by why would `@__FILE__` ever be wrong?


Hey men plase remove this line. Ours PO don't like comments


This regex is invalid, should be `[^']+`.


@dra14825 same as before, * import


again


I know it could, but that's not going to work. The log entry needs to be opened before asyncGet starts doing things, and the extra_info must be written at the time the entry is opened.

The method is not so async as its name implies. If it finds a free group - probably the most common flow - it goes straight into the sessionCheckedOut callback, which assumes that the getFromPool log entry has already been opened, which will not be the case if the checkout is still waiting for the result. Anyway, even if this weren't obviously the case even the async flows could race.


Why pointer to gyro_lpf_hz?


TeamCity Civic Assembly :: Build backend + frontend [Build 7](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=313&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


TeamCity Civic Assembly :: Build backend + frontend [Build 5](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=309&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 19, passed: 1459, ignored: 12) Build time: 00:16:47
### Failed tests
```
Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should not create new media type if cancelled: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

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


This won't correctly handle multiline comments. Should be changed to `/(?:\s|\/\/.*|\/\*[\s\S]*?\*\/)*/g`.


This variable should be attribute of class.
In order to re-use it.


Right, I missed the call to transform.


Do You know when It'll be Availible on the Public Repo's?

Anyways Really good Fix


Here, you forgot to "close" your `<button>` tag wih `</button>`. Using breaklines and indentation can help noticing the lack for a closing tag:

``` erb
<button class="social-btn">
  <%= link_to "Follow me", "https://www.facebook.com/trybloc", class: "social-btn" %>
</button> <!-- was missing! -->
```

Once the ERB chunks are processed by the Ruby interpreter, it generates the following HTML:

``` html
<button class="social-btn">
  <a href="https://www.facebook.com/trybloc", class="social-btn">Follow-me</a>
</button>
```

Notice the generated closing tag `</a>` for the link. Using a Rails helper such as `link_to` allows you to:

* *type less code* (no need for handling tags, opening and closing)
* *handle dynamic values* (made-up example: `link_to "Follow-me", user_twitter_account(current_user)` where `user_twitter_account` would be a Ruby method returning a string with a Twitter URL, accounting for the value of the currently signed-in user reported by the argument `current_user`)


Sounds good.


@dra14825 can you please instead add *radiologyProperties* as class member and autowire it.


@refractalize is this an assertion or a finder?


`webpage` is redundant.


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


I can't find the body of this message. Are you sending for which game was the person made a volunteer?


ok


@dra14825  please change these to *@Test*


> The log entry needs to be opened before asyncGet starts doing things, and the extra_info must be written at the time the entry is opened.

Why must the log entry be opened before asyncGet starts doing things? Why can't you do it afterwards? The indexer doesn't care about the ordering.


Looks fine to me


Afaict, separating out the tests like this should avoid the focus issue described in #59


The gogs data, like git repo,...


please remove temp var, since its not needed


You're right !! :+1:


You could move these two files in `/doc` ;)


Yeah, I noticed that too. I changed it to `[^]` in  8f46eb5ffa52


See slack.


close #79


I would rather log the whole context object :see_no_evil: (also for the other logging methods)


do we need to do this?
should we just rely on jquery directly? like `require('jquery')`


Ah cool, didn't see that.


You are calling to this method several times and you are loading the properties file several times. this is not a good practice.
I think this method should be moved to a Singleton class and load the properties file one time.


Please wrap the email address in < > instead of brackets


Move this to `else` branch?


Ok, but i will do that in separate commits per implementation, no?


is this precondition checking for View Visits or View Encounters?


A project I've been working with have reported a number of accessibility issues which I need to refine and then discuss in a separate thread but one of the issues raised is the variation in the keys required to navigate around page contents. In investigating this on a button navigator page for example it seems there are a couple of issues. Firstly ideally tab should cycle through the navigator buttons as well as the rest of the page content. This seems to work on accordian nav but not on button nav. Secondly it seems we either need to remove the case 38 and case 40 detection that I added previously or add further conditions to the keydown function to only work when the button nav buttons don't have focus. e.g. in a button navigator pressing the tab key will go to the first button but then the user has to use arrow keys to open the other nested pages. Without case 38 & 40 in place the user can use left/right arrows or up/down arrows but with case 38 and 40 in place up/down navigates to the next/previous page.


Birthday maybe?


You also have to change it here...


Your restart commands using service should be invoke-rc.d (deb-systemd-invoke for OMV 3/jessie).

invoke-rc.d nfs-kernel-server restart


I still want it to hotplug or else the device gets hot, but I wanted the threshold adjustable. It's much easier to simply modify /etc/thermal-engine.conf but I don't want any modifications to the system partition. So I leave thermal engine running, but have the kernel driver handle the hotplugging.


default should be other way around (backwards compatibility!) unless otherwise stated by the core team.


This could use a comment to explain what it's doing.


Why do you think only implementations are impacted ? I'm pretty sure the whole codebase is, but you don't see it because it stop the problem propagation too early.
However, you need to track every time you spotted something to fix, because it will be hard to find otherwise.


I'm confused about this. The slice appears to be just making a copy of the result of `getOwnPropertyNames`, but why do you need to do that?


I think you can separate this class in two classes.
ResourcesSteps
ResponsesSteps


On peut commenter sur github quand il s'agit de commenter du code.

La raison profonde est que `tax_benefit_system.column_by_name` est créé par [`TaxBenefitSystem.__init__`](https://github.com/openfisca/openfisca-core/blob/bcfe01a4737b0ea804edc96bc73631b7bed627ac/openfisca_core/taxbenefitsystems.py#L46-L50).

Dans [`tests.base`](https://github.com/openfisca/openfisca-france-indirect-taxation/blob/afe9a9d181eb7dcbc1927bf34d78142568e150b6/openfisca_france_indirect_taxation/tests/base.py#L39-L40) `TaxBenefitSystem` est instancé et `prefill_cache` est appelé seulement ensuite, mais trop tard car `tax_benefit_system.column_by_name` a déjà été créé. Du coup les variables générées par les sous-fonctions de `prefill_cache` ne sont pas indexées dans `column_by_name`.

J'ai simplement ajouté un moyen de redéclencher l'indexation de `tax_benefit_system.column_by_name` via la méthode `TaxBenefitSystem.index_columns` que j'appelle [à la fin de `prefill_cache`](https://github.com/openfisca/openfisca-france-indirect-taxation/blob/afe9a9d181eb7dcbc1927bf34d78142568e150b6/openfisca_france_indirect_taxation/__init__.py#L57).


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


sorry i forget to remove when i copied this.
```python
webpage = self._download_webpage(url, video_id)
```


How is this supposed to work?

If I translate the new strings to something that exists here in Sweden, will the tutorial display these items in the map, or will it still display eg. "Three Rivers City Hall"?

Must the translated name exist in reality on the map, or is it enough that I just find something that sounds Swedish?


```
In [1]: pow(0.000001, -10000000000)
OverflowError: (34, 'Numerical result out of range')
```

Вангую sqliteу не понравится исключение в пользовательской функции


Not sure if related to this commit but GridView and DetailView stop working. Every cell of both GridView and DetailView is shown as empty (no value in red).


Tell me you actually broke the only style guide rule that matters.


We need the `.rbenv-version` file to be available before running bundle now that we use that (rather than the `$RBENV_VERSION` env var) to decide on the ruby version to use


Nope.  Message-Authenticator and CUI are implemented as special cases in encode_rfc_hdr().  *All* other attributes should have non-zero length.  The encode_value() function MUST NOT encode zero-length attributes.  The code on line 724 checks for this.


Perhaps this `availability_profile=test-ap1` should be a variable? Same question goes for `availability_profile=egi-fedcloud` too.


Autrement dit on perçoit le besoin de séparer l'instanciation du `TaxBenefitSystem` de l'indexation des variables.

Tout ce raisonnement serait simplifé quand l'issue https://github.com/openfisca/openfisca-core/issues/366 sera codée.


i will fix it now.
i made the change quickly and didn't verify the change carefully.


code review


hi chuanwen,

nginx-upsync-module 与ngx_fair 实现功能不同，对nginx 的嵌入也不同；

nginx-upsync-module 只是增删后端的peer，对其它没有任何影响，也就谈不上对keepalive 的不支持；

功能纯粹，可以说是零依赖、零干扰.


no - this is no the intention of the context.

The context is basically used for string interpolation:

```php
$logger->error("User {user} deleted file {fileName}",[
    'user' => 'peter',
    'fileName' => 'welcome.txt'
]);


won't the statically linked bins go away?


Agreed... window.$ is not always jquery anyway.


Btw I know the value of `5 5 5 5` is what the walkthrough video showcases, but I strongly disagree with that and I reported the issue to the curriculum maintainers ;)


<!-- PMABOT:TAB -->
This commit is using tab character for indentation instead of spaces, what is mandated by phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: libraries/dbi/DBIMysqli.php


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: doc/config.rst, libraries/config.default.php


Done, https://github.com/php-fusion/PHP-Fusion/commit/e567e87ad0a1f623bb01b8357e9b888d1ae75111 :)


Is there a reason to use `reduce` instead of `forEach` here? I think it would be clearer if you did something like:

```
var node = {type: ctorName};
propNames.forEach(function(prop) { ... });
return node;
```

To me, `reduce` usually implies a functional transformation, but that's not the case here. (Assuming I understand this code correctly.)


I'm not sure if there are any reason to test this extensively. It worked for me, and the changelog for lodash is pretty clear.


It works fine with this approach.


谢谢修改(•ૢ⚈͒⌄⚈͒•ૢ)


This unbreaks it?


Thanks for adding this -- it will be very handy!


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


TeamCity Civic Assembly :: Build backend + frontend [Build 7](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=313&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 19, passed: 1484, ignored: 12 Build time: 00:16:55
### Failed tests
```
Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should not create new media type if cancelled: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

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


did it after merge


Спасибо, но game/translate.rpy, зачем он тут?


We used to have `Pull` and `Push` in separate libraries but that turned out to be inconvenient since you need both vector types most of the time anyways and they were not really splittable.

I would have expected `Seq` to land in the `Vector` library as well for the same reason, is this different from the `Pull/Push` scenario?


to ma się znaleźć w configu aplikacji. Piszę to drugi raz.

Przykłady:
http://stackoverflow.com/questions/13043530/what-is-app-config-in-c-net-how-to-use-it
http://stackoverflow.com/questions/8476103/accessing-database-connection-string-using-app-config-in-c-sharp-winform


ah - got it :)


good call :)


I think you can use datatable here. because your method will not support if you need to add more fields.
Using datatable is more flexible.
```
    When I did a post  with:
      | Content    | Icon |
      | MyProject | 1     |
```


Then please also adjust the PHPDoc about this hidden feature ;)


TeamCity Civic Assembly :: Build backend + frontend [Build 8](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=320&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


@janosvitok I was not aware of it either, and I can't find a reference to this in Qt docs, but it looks like signals are protected in Qt4 and public in Qt5...


It is different in that it involves different types. If I write `optional<optional<int>> o = none` with the intention of initializing the inner `optional` then I already know that `none` is special for `optional`, and that my code won't do what I want. It's unambiguous.

On the other hand, `optional<any> o = none` is ambiguous, but as you say it is redundant too. And perhaps more importantly, `optional<T> o = none` with `T = any` is not ambiguous and it behaves as expected.


> Then please also adjust the PHPDoc about this hidden feature ;)

I meant the `message` entry in context.


In fact this work well apart if you compile with `+mpi`
I did not see you PR so I did some modifications in #346.
So either I adapt #346 once this PR is merged or you can include the modification in this one and cancel my PR


And drop this.


@clebertsuconic Why do we need to specify a cluster connection URI type vs just a URI type?


This key should have been deprecated.


I added the 0.1.7 version because, for some reason, when I published the 0.1.8 version it didn't upload the JARs, only the AARs.

https://repo1.maven.org/maven2/me/sneer/sneer-android-api/0.1.8/

I'm going to take a look at this ASAP.


IIRC it wasn't written to feed data into the HTTP API. Also right now it produces json that doesn't work as input for the HTTP API.


How's this for an announcement? :)

<https://mail.haskell.org/pipermail/arch-haskell/2016-January/004962.html>


This is great.


fix #16


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


Awesome


:heart:


I think you can use the following step definition
```
@Then("^The response should have a \"(.*?)\" attribute with value: \"(.*?)\"$")
```


ok


@sharkwouter , to merge changes, doesn't' a git pull work in your local tree?


@hpi1 thx for the feedback, I'll go through it again when I find some time.


Is `lang-def-generator` still the appropriate name, since we're not doing generation any more?


(nitpick 1: `@functools.wraps(func)` to replace setting `__name__` etc. See execute.py's `python`)


I think that here it is missing another validation
```
Then The response should contain the data with the string value "Icon" and "4"
```


@jerrypy Hey, can you make a PR that fixes the docs?


Yes, here is the body of the message
app/views/volunteermailer/became_volunteer_email.html.erb


(nitpick 2: two spaces before, one space after `#` per PEP 8)


You shouldn't need this type annotation?


If max int is the limit why can't we change the param type?


you are correct, this is a leftover from when I was trying to work some stuff out.


Alright. Got it. :+1:

Sent from Outlook Mobile<https://aka.ms/blhgte>

On Fri, Jan 15, 2016 at 5:54 AM -0800, "Vinay C K" <notifications@github.com<mailto:notifications@github.com>> wrote:

Yes, here is the body of the message
app/views/volunteermailer/became_volunteer_email.html.erb

-
Reply to this email directly or view it on GitHub<https://github.com/7andahalf/spandan/commit/1ee7e4987b9aacedec433adf3661211f139c8c51#commitcomment-15475024>.


Use previous, shorter prdownloads format: `http://dl.sf.net/libpng/libpng-$portVersion.tar.gz`


... from one backend to ...?


You forgot to update the checksum.


Huh? Lexer tokens emitted by Parser are a stable interface too, though that's not obvious. Moreover, we explicitly do not maintain any compatibility whatsoever with the MRI lexer, so it is irrelevant what it does.


How do you guarantee that exists that id?

I think you need to add a precondition first.


frameworks/base/telephony/java/com/android/ims/ImsCallProfile.java:221: error: variable EXTRA_CALL_RAT_TYPE is already defined in class ImsCallProfile
    public static final String EXTRA_CALL_RAT_TYPE = "CallRadioTech";


s/PDNS/PowerDNS/


Merci je comprends mieux !


i was getting 404 error.
i thought the problem is in the signature but i'm wrong it's in theplatform regex.
i will revert this commit and fix the extraction in another commit.


Unfortunately, this tweak to my PR might cause a `ConcurrentModificationException` - changes to `fileStates` must be synchronized on the `FWMH` instance.

I thought about this but decided to leave it since another `close()` will be a no-op anyhow.

If we leave it here, it must be synchronized.


This brakes **Program\Empty** unit test.
'seems the empty file problem was not solved after all -we just looked at the wrong places.


This is much more clear to me.


TeamCity Civic Assembly :: Build all [Build 1](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=324&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


For this PR, I don't think we need tests since we only start the test container once.
Don't worry about that, it's about TCP connections likes smtp, image and authentication.
Thank you.


Not convinced this is correct, just followed what was happening before. Is the channel number even used for FrSkyPWM? If not this should be 0.


TeamCity Civic Assembly :: Build all [Build 1](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=324&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new); exit code 123 (new)) Build time: 00:00:19


TeamCity Civic Assembly :: Build all [Build 2](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=325&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


The vgg16_weights.h5 file is available here:

https://drive.google.com/file/d/0Bz7KyqmuGsilT0J5dmRCM0ROVHc/view


Sonar points out that this should be `static`.


Neither do I. Maybe we can use something like this `create_function('$severity, $message', 'throw new ErrorException($message);')`.

A full working example is in https://github.com/notthatbad/silverstripe-website-parsing/blob/master/code/fetchers/ContentsFetcher.php


TeamCity Civic Assembly :: Build all [Build 2](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=325&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new); exit code 1 (new) Build time: 00:01:23


@athomschke  Wir koennen nicht `/lively4-core/` komplett aus dem Pfad nehmen, da github.io die Pfade `/lively4-core/draft/...` usw. erwartet. Deshalb ist es wohl am besten, wenn wir unsere lokalen Server auch so laufen lassen, dass sie mit den Pfaden zurechtkommen. Vielleicht kannst du die Export Sachen so anpassen, dass das funktioniert?


1. When an exception arises inside a user-defined function, SQLite replaces such an exception with "OperationalError: user-defined function raised exception". I think this is acceptable behavior, and doubt that some additional processing inside a function can provide better results.

2. Regarding memory issues with expressions like `pow(2, 100000000)`, I suspect this is not the problem that Pony ORM should care about :)


TeamCity Civic Assembly :: Build all [Build 3](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=326&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


기존에 노출된 정보들 히스토리에서 지우다보니까, iolo comment가 모두 지워졌네요 ㅠ_ㅠ
주의하겠습니다 !


So this should probably be overloaded if small molecule mode where unit cell volume could easily be < 50 Ang^3? Thinking of xia2 usage of this code.


Is this change strictly necessary? In my environment at least, it doesn’t seem to be. That is, if I just keep it as `<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER"/>` in this `.classpath` file and I have my **Eclipse > Preferences > Installed JREs** set to a Java8 JRE as the default, everything works as expected.

On the other hand, in my environment if I do explicitly set the value in this `.classpath` file to include the `/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-1.8"/> ` part, then I get this error:
 ```
Build path entry is missing: org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-1.8
```
As far as I can tell, that’s because in my Eclipse settings on OSX, the name for my Java8 install is not `JavaSE-1.8` but instead `Java SE 8`. If I manually change the name in Eclipse to `JavaSE-1.8`, then the `Build path entry is missing` error goes away and everything works fine.

So, it seems that setting the  value in the `.classpath` file to include the `/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-1.8"/> ` part causes Eclipse to look only for a JRE in its installed JREs list whose name is the literal string `JavaSE-1.8`. That seems very fragile and I would rather not do it unless we really need to for some reason.


I really like this decorator, I didn't know you could do that so easily! One thing though, shouldn't you add data=results here? If you do, I can convert my call here:
https://github.com/open-data/ckanext-stcndm/blob/96aac7c1a7fa66f9727e5431549c04ddc7ae8d91/ckanext/stcndm/logic/releases.py#L292-L294


TeamCity Civic Assembly :: Build all [Build 3](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=326&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:32


ans -> and


seperate -> separate


Confirmed, reverted https://github.com/yiisoft/yii2/commit/f72e9d5bcf4642a0fe54ccbc3bd58d7e05e3ff7b


Non online


혹시. 쓰로틀링온도에관련해서알아봐주실수있으신가요?
어째좀액정온도가살짝따끈해진다하면
빅코어가 634로고정되어버립니다.
이때문에 버벅거림과 시퓨스케쥴이안맞는지 배터리가썩좋지않은거같습니다. 쓰로틀링의해제보다는 기준점을좀높여야할거같습니다


@sethlyons Why have you done this? for which distro?

It breaks our Debian setup.


Hi @ivanr,

Just to be sure: I assume we are allowed to take this data?

Cheers, Dirk


I'm just waiting for one to come :grimacing:


looks good to me. :)


Yes I did a while ago, thanks. I concluded that I'm most comfortable inside Vim :)


`const` would work here, as you don't reassign var later


If a control is visible to the user, I'll need to add it to the "uses IControlThemable" list, just to make sure any/all run-time checks are correctly applied.  This ends up being a good way to discover which controls I've missed, so it's a great step forward.

And sticking translations and theming together has always been the plan, since the two are related.  (99.99% of the time, changing one requires refreshing the other.)  But until I have visual themes fully implemented, all this is hard to test.  :/  Guess I better get cracking!


Is it inside the /zh_TW/ path or it has been moved?


Nice, it's possible to specify that we want the AAR file https://github.com/felipebueno/Accounts/commit/84d126c0079aea5a3285649ba9b4845373de31db


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


This is giving me a compile error: "The expression of type int is boxed into Integer"


Why does this not work for non-english text?
````ruby
> text = "R.I.P. someone"
> PragmaticTokenizer::Tokenizer.new(text,classic_filter: true, language: :en).tokenize
=> ["rip", "someone"]
> PragmaticTokenizer::Tokenizer.new(text,classic_filter: true, language: :de).tokenize
=> ["r", "i", "p", "someone"]
````


TeamCity Civic Assembly :: Build backend + frontend [Build 8](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=320&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 19, passed: 1484, ignored: 12 Build time: 00:18:06
### Failed tests
```
Document type tests: Should not create new document type if cancelled: <no details avaliable>

Document type tests: Should add new document type: <no details avaliable>

Document type tests: Should not change document type: <no details avaliable>

Document type tests: Should change document type: <no details avaliable>

Document type tests: Should not delete document type if cancel pressed: <no details avaliable>

Document type tests: Should delete document type: <no details avaliable>

Media type tests: Should not create new media type if cancelled: <no details avaliable>

Media type tests: Should add new media type: <no details avaliable>

Media type tests: Should not change media type: <no details avaliable>

Media type tests: Should change media type: <no details avaliable>

Media type tests: Should not delete media type if cancel pressed: <no details avaliable>

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


Surprisingly, it works with both in controller specs, however favor using `put` instead of `post` when testing the `update` action so its clearer that the PUT HTTP verb is intended to be used with updates.


doesn't work. Server crashes if you don't move from container to other container (like from container to ground)


Though I still need to investigate why it didn't generate the JARs


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


looks like a component to me


these pages should not be tracked. They are generated runtime. Why they appear in the diffs?


ack


thats actually an INPUT option ;-)


Or probably actually 1 so it shows Chan 1 instead of Disabled.


why the change to com.telscale instead of org.mobicents ?


It should be Mobicents-jEIR not TelScale


there's some other place in the doc where this is still referenced (I think you moved that text around in this PR.)


Please don't commit directly into `master`, even for trivial fixes. This breaks the output of `git log --oneline --first-parent`, which is supposed to list only merged PRs and tags. When editing though web UI, GitHub suggests an option to commit into a new branch and start a PR.


why commenting those properties ?


it's kind of both :-)...


looks good. thanks


Although they are related, there's no need to exercise the `edit` action in the `update` action tests.

When we get round to writing feature specs in the projects, you'll learn how to write an **integration test** that exercises `edit` and `update` in the same test.

Integration tests check that the various parts (typically called units) of the app work together as intended.

In this case the units are the edit and update actions.


ok
let's say A is a non_owning buffer to ptr1
let's say B is a non_owning buffer to ptr2

A = B must std copy the contents of ptr2 into ptr1 not makeing A "points" to B


why not keep it a constant, this is best practice AFAIK for logger classes as they often heavily accessed


:+1: super work


This makes me happy.


Bad changes, adding a cool design to runtime.


The Ajax project had strange custom compiler settings. Should be working now.


Found this change at http://savannah.nongnu.org/bugs/?28471


This use of `build-tools` is semantically wrong IMO

I.e. `build-tools: ghc` does not influence which compiler cabal is allowed to use, but rather `build-tools` requests that `ghc` must be available, no matter which compiler is ultimately used by Cabal for compiling haskell code.


In addition to @mallowigi's comment, I think you should mention that the callback for the `forEach`, `map` and `filter` functions take the following arguments:

```javascript
Array.prototype.forEach(function(value, index, array) {
  // code goes here...
})
```
where `value` is the current element being processed in the array, `index` is the index of the current element being processed in the array and `array` is the array that the function is being applied to.


Since the change didn't have an effect, t wanted to get back to the original behavior so that we'd have a control for changes to the MicroPCF to work against.  It will probably make it back to `@Before` once the MicroPCF problems are fixed since it currently assumes a clean instance at start (which your local MicroPCF might not be).  All that is a long way of saying that things are still in flux.


Stopped test because default margin should have been 2, not 4.


Changed to ConfrenceRequest as Conference is already as response Model


Agreed.


Well... Those were suggestions, not corrections.


the URL path is wrong. Isn't it?


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


> If I translate the new strings to something that exists here in Sweden, will the tutorial display these items in the map

Yes, there is code in `intro.js` that will switch the built-in names with translated ones if they exist.

You don't need to literally translate "Three Rivers City Hall", instead you could name it something more Swedish sounding.  It doesn't need to be a feature that exists in reality.

[Here is a link to the string on Transfiex](https://www.transifex.com/ideditor/id-editor/translate/#sv/core/68297081) to get you started!


Agreed as we have conferencecode with leading zero and also without leading zero for custom access code. So it should be string as input


Eventually :)


`@__FILE__` becomes a compile-time string, rather than being resolved dynamically at run time. So if I compile a module on my laptop and run the compiled code on yours, the string will be the same, and hence point to the wrong place.


Ok. Now I got you.


There will be not any scenario where User is valid, but he cannot see other users


Moreover, the cabal solver does not support yet backtracking based on `build-tools` yet.
If you actually want to constraint which ghc versions are supported by a package, you'd rather have to say something like

```
if impl(ghc >= 7.0 && < 7.12)
   build-depends: base<0
```

However, this explicitly *requires* `ghc`. But you can easily formulate conditions in slightly different ways, e.g.

```
if impl(ghc)
  if impl(ghc >= 7.0 && < 7.12)
     build-depends: base<0
```

which says that in case `ghc` was used by cabal, then it must be within that version range.


Takk og lov for at du klarte å ta den avgjørelsen. Vi holdt alle pusten i spenning!


Тут бага
Предположим, у мастера стоит view.saveDetail =  false
Тогда сразу идем к строке 237. По-умолчанию, detailsToProcess = 0
далее вызывается дефолтовый коллбэк из строки 219:
проверка --0==0 дает false и мастер не рефрешится. хотя это явно ожидалось.


TeamCity Civic Assembly :: Build all [Build 4](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=330&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


It's a good idea to disable the save button when the user click on a workflow/action button. What is the desired behavior? Stay on edition mode, disable save button or back read-only mode?
(Here, it's just a fix to reload the view if the action fail after the write. The old version is never display but in fact the user can click on the workflow button and on the save button.)


TeamCity Civic Assembly :: Build all [Build 5](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=331&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


@mattheworiordan NACK


Has changed


TeamCity Civic Assembly :: Build all [Build 4](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=330&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 137 (new); exit code 123 (new)) Build time: 00:00:18


TeamCity Civic Assembly :: Build all [Build 5](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=331&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner; exit code 123 (new) Build time: 00:00:10


So we just need to add copy( non_owning..., non_owning... ) and call it from here.


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


you can also change the name of note to note_id since it is essentially the note's id and not a note object :+1:


roughly yes


For access code data type changes agreed. Done in all places


Can we re-use this variable?


in line 173 you have got li {} . It's a little messy. For the umpteenth time : use nested syntax


I think you didn't wrap this test because it is replaced in a later commit, but that does make the history a bit wonky. So if you don't mind add the `tape` wrapper here as well.


//Firda Aminy Ma'ruf 1301144311 IF3801
package primenumber;

import java.util.Scanner;

/**
 *
 * @author Iwan Ma'ruf
 */
public class PrimeNumber {

    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        Scanner input=new Scanner(System.in);
        System.out.print("Please enter a number: ");
	int number=input.nextInt();
        int p=0;
		if (number == 1)
		{
			System.out.println("1 is NOT a prime number. Please enter a number that is greater than 1");
		}
		else
		{
			for (int i=1;i<=number;i++) {
				if(number%i==0)
					p=p+1;
			}
			if(p == 2)
			{
				System.out.println("That number is a PRIME number");
			}
			else
			{
				System.out.println("That number is NOT a prime number");
			}
		}
   }
}


TeamCity Civic Assembly :: Build all [Build 6](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=332&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


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


No one will understand that this does unless he looks at the code of ``renamePage()``. Does it "rename the rename page"? No, it simply takes you to the rename page, so why not ``gotoPage()``? Always check the existing page objects. See the [CreatePagePage](https://github.com/xwiki/xwiki-platform/blob/master/xwiki-platform-core/xwiki-platform-test/xwiki-platform-test-ui/src/main/java/org/xwiki/test/ui/po/CreatePagePage.java#L53) for instance.


Well, this is because in latest jSS7 version(3.0.13.22) in there is no more  org.mobicents.ss7.management.console.ShellServer class Instead there is a class com.telscale.ss7.management.console.ShellServer inside  shell-server-impl-3.0.1322.jar
I didn't four the org.mobicents.ss7.management.console.ShellServer class in latest jSS7 version (it has been moved/refactored to the com.telscale package?)
Did I missed something?


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


For personalid base information on present on AWV. Need to discuss with Amit and will come to you.


Are these needed actually? They seem unused?


TeamCity Civic Assembly :: Build all [Build 6](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=332&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:15


The stack overflow only happens since https://github.com/dotty-staging/dotty/commit/9a3dbf9a6fb8e2c09ed89507e57ef48363e21dad is applied, are you sure that this commit didn't introduce a bug in `containsRefinedThis` that causes this?


I wonder if one or more of these processes missing makes the nfs server operation actually unusable. nfs is a complex thing.


TeamCity Civic Assembly :: Build all [Build 7](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=333&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


Sure, I will correct it


Might be worth wrapping this in parenthesis to make the intent explicit.  The -= operator looks very similar to the - operator, which has the same precedence as +.

this.sentBufferSize -= (command.getCommandString().length() + 1);


What's "toto" :)


The moodle response 500 error :

The website encountered an error while retrieving http://www.besttopractice.com/course/modedit.php?add=hvp&type=&course=14&section=0&return=0&sr=0. It may be down for maintenance or configured incorrectly.

What can i do ?


:+1: Missing out `"export DISPLAY=:99.0"` for xvfb trips up a lot of folks when getting this setup. Good job including it.


This is not our API, why `client.eventEmitter.on`, it's simply [`client.on`](http://docs.ably.io/client-lib-development-guide/features/#RTE3) because the EventEmitter is mixed into the `Connection` object


I think if you will validate http code errors in your tests, this condition will not be necessary


Same here, `subscribeToStateChanges` is not a documented method, it's `channel.on` for state changes


This eval rule should also take in consideration the target which IIRC can be either `body` or `rawbody`. E.g.

```
body          DC_GIF_TEXT_RATIO       eval:image_to_text_ratio('gif',0.000, 0.008)
rawbody          RAW_DC_GIF_TEXT_RATIO       eval:image_to_text_ratio('gif',0.000, 0.008)
```

We have the raw one available in the message in `msg.raw_text`


fgdg


There are issues with the suggested change. The instances of BucketIdPair are being accessed by 2 threads- one will be setting the time, and the other will be using this time to compare. So getTime/setTime are synchronized. This introduces a delay to bucketAccessed which is more frequent.  The is a timer task which will run at regular intervals so IMO it is better to introduce delays in the `run()`.

Another problem with the suggested change is that because of `remove()` will have to use a synchronized version of this queue `PriorityBlockingQueue` this again makes bucketAccessed() slow.

I find the option I suggested earlier more efficient:
I can have a priority queue which is used only in run(...). Every-time run(...) is invoked, all the items from the main map (currently called heap) are added to it and then the queue is cleared. It will be better than creating copies.


Done


That's great, thanks :P


You don't have to recreate the page object each time with ``new RenamePage()``. You create it once and you put it in a variable.

    RenamePage renamePage = new RenamePage();

But in your case you don't even have to create the page object because the call to ``renamePage()`` does it for you: it ``return new RenamePage();`` so you can simply write:

    RenamePage renamePage = RenamePage.renamePage();

Of course, it would be more clear with ``gotoPage()``..


Welcome


Hi @173210 That has been a while not seeing you on gbatemp. You must be quite busy at the time.
BTW does `tmio_wait_respend` mean.. `tmio_wait_respond`? Oh That's not exactly a problem (lol).


I rolled back my old test and commited a new one


Could we put braces around this, I realize it isn't technically required for one line ifs but I feel it is always better to be safe than sorry for this.

We don't want to have someone mistakenly write
```r
if (is.null(upstream))
  message("no upstream found!")
  return(invisible(FALSE))
```
Then wonder why this function always returns `FALSE`.


until(-l) all proc(-c)ess(+e)s


TeamCity Civic Assembly :: Build all [Build 7](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=333&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:04:04