> This is the weird kind of polymorphism-plus-inheritance combo that we have to deal with when we can't have full-fledged generics.

Yeah - there are a lot of subclasses being introduced here - it appears we have to create "pairs" of classes for every situation: match a view controller with a specific data source. I would expect this to be architected so you can have a single view controller class that can be composed with different data sources to get the effect - or you can use many view controller classes that all configure a data source internally - obviating the need for custom data source classes.

Having many view controller classes and matching data sources is a bit of a code smell - dual inheritance chains.

I do think there is a case for multiple view controllers - but it seems it would better to scope that to only the visual differences: modal vs. non-modal. (If that difference couldn't be handled by one class with case statements)

It would seem to be cleaner if the image view controllers could accept any type of image data source. If the data source was a black box, this would seem to be possible? Is there something holding us back from doing this?

> or have them override some kind of factory method and let the base class decide when to create the data source.

This would actually dovetail ver cleanly with the strategy above: Have class methods that take an argument and then configure the image gallery with the appropriate data source.


You disgrace me.


This should return `true`.


Don't remove the test case.


do we need that function? if not remove it!


Please put `{` and `}` in right positions, such as `didSet {`.


That would be a good idea. After I test this code I'll move it.


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


why cant this be done on the server?


ACK.


The size of this map is the size of the corpus. Use this instead of the hardcoded `numTaxa`.


sick


This must be added to the end of the file - see the comment at the start.


Thanks


Too many `!` in this line. It's unsafe. Please use `if let` to unwrap variables.


Bit of a weird name, isn't it, looking at the description of this function?


"getTopologicalRelationship" and "isTopologicalRelationship" done for Point2DPoint2D


Ok I merged it and am building your new patch for docker-1.10.  Could you submit it upstream and see what dockers reaction is. for --no-image-volume


@rhatdan sure!


this is just for displaying the position, there is no server action afterwards


Actually, there's more than one place that contributors should be added to; grep for Dewey/Hutty to find them all.


Nice solve !!!


Add a comment with the graph here to make it clear where the 3/2 and 2/7 come from.  Maybe just say "look at the example in `SignalCompare`"


Blank function? Please remove unused code.


If error is printed - it should start from upper case character. If it's passed as return value - it should start with lower case character.

Look at https://golang.org/doc/effective_go.html and later at https://github.com/golang/go/wiki/CodeReviewComments


yes, it's used by another function


Je comprends ça en lisant la phrase: "Parce que Scala est interopérable avec Java, il est interopérable avec JavaScript". Je crois pas que c'est ce que tu voulais dire :) (C'est probablement le "ainsi" qui fait un mauvais lien)


ACK.


That's not the only incompatibility. Also I don't really like the doc strings for levels which seem rather end-user oriented with their mix of log report considerations/action policy which do not belong here in my opinion. I personally prefer [this take](https://github.com/dbuenzli/logs/blob/acb6951b568ec0aae7a6b0ac580395514063de6b/src/logs.mli#L418-L431) on the usage semantics which is more programmer oriented.


This may be personal opinion, but I think it is cleaner to make some of these variables local to the methods where they are used, rather then modifying and accessing them from various places. Pass them into the methods that need to read them.


If both input argument `ert` and `self.__ert` are None we will end up calling `self.__ert.free()` with an invalid reference.


causes compiling error


Still no window or duckie tv tray icon.

![screen shot 2015-12-03 at 12 40 32](https://cloud.githubusercontent.com/assets/13501166/11565117/4c3133f4-99bb-11e5-957c-1f3ac4e12717.png)


@snowtimeglass I updated the manual for v2.5


Removing related Objects


I don't think GitHub is the right place to discuss this but if I'm not mistaken there will be no nightlies without a maintainer. Else there would be nobody to watch / maintain and fix those nightlies.


can there be DEAD and Matching state? Isn't that redundant?


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


et qu'une erreur s'y glisse (pas de "rait")


It appears that the name of this function, its docstring and its implementation are in contradiction, aren't they?


wtf


Did you mean to leave in these debugging statements?


OK. `buildURL` test was added.


You can put this function in `extension UIViewController` to reduce the parameter.


Would have been enough to just remove it from the DEFAULT_FILTERS ?


Should we move this check little later. May be after container.removalInProgress check. Also putting this logic in an helper function will help keeping containerStart() function small.


ddd


It's not about whether the replacement is sufficient, it's more about the compatibility. R3-gui.r3 (which wasn't written by me) is using it, and people talked about it at CC, perhaps there are more people using it and we just don't know, we should try not to break the compatibility unless there is a compelling reason.


To me, it looks like the area for this example should be 0.  If the condition is never satisfied, then there is no area between the curves, no?


Is Homebrew no longer asking that multi-part PRs be squashed?  If we should still be squashing, please leave this link in there.  I always forget how to squash and need to refer back to it.


If so, I wonder why it isn't checked at line: 478?


ACK.


Ymdhis forces to midday, not middnight.

Should be YmdHis


testt


exitDuration is used only there and few lines later, and so it could be removed from defined vars if line 70 will be in `else` block.
Also in error message You should print error value - Atoi can for example fail if You provide too big number. Also again - args[2] should be natural number (less than uint limit), because 0 is also valid unsigned integer.


Was the output too big?


make sense (I wouldn't write another func just for this)


ACK (maybe also ```10.4.3``` would apply here).


It's better if you set default value for these properties, such as `""` for String and `0` for Int. By this way, you needn't to unwrap them before using it later.


Hi,
Yes of course. The interpreter will be built as an exercise for a course taken on the University of Technology in Vienna. Our supervisor is Anton Ertl (http://www.complang.tuwien.ac.at/anton/home.html), a professor with a deep understanding of the Forth language itself.
Actually he is the author of the Compiler-generator Gray among other smaller projects in Forth (http://www.complang.tuwien.ac.at/projects/forth.html)


Thanks for testing it!


fix Issue #26


good catch - i'll update it;


reverted in https://github.com/mapnik/mapnik-packaging/commit/1aa97053f4440116a11a1e6e2ce717bf0dac5962.


InvalidModelValueErrors are JSON-API errors. Please add a `/source/pointer` field to the error message like in [NodeLinksSerializer.create()](https://github.com/CenterForOpenScience/osf.io/blob/develop/api/nodes/serializers.py#L334).


I recommend treating the function codes in the link layer as the logical OR of PRM | FUNC, and just pretending like PRM doesn't exist.  Certain function codes work w/ PRM set, and others with it only unset so they might as well have just had it part of the function code, e.g. see:

https://github.com/automatak/dnp3/blob/2.0.x/cpp/libs/include/opendnp3/gen/LinkFunction.h


OK thanks.  (FWIW I also have very few extensions.  I get the delay when switching between TastOps.fs and TypeCheckere.fs (or any two other files really).  Unloading the FSharp.Test project made things much faster, and I profiled nearly all the time was in some operation that was scanning the VS solution hierarchy.)


Here


Please fix this commit message "to a v expensive".


Same as below - if You will put defer/go in `else` block - there is no need to declare listener in var block (and probably this will remove need to declare err in vars block because of use `:=` operator in this and other `if` arguments).


ACK (related with aide => IDS).


@rmcmx


I think Chris was working on a list directory method for System. We could use this in at least two places but for now it is Ok, I think. We should just remember to refactor it.


Lengths < 5 have crashed poor implementations. Definitely want a failure callback for this.


Well, we could apply the same logic as in `terminateOngoingHandshake` where we only terminate the handshake but leave an established session (and thus the connection itself) intact.

Why do you think that we terminate the connection on non-fatal `HandshakeException`s? The `handleExceptionDuringHandshake` is (should be) invoked when a `HandshakeException` is thrown during a handshake and it distinguishes between fatal (terminate) and warning level (discard). At least that was the intended behavior :-)


MF's R can be cancelled by movement if youre looking for another channeled spell


fast-forwarding end-to-end/timeseries to borshop-integration-1289-at/configure_concurrent_queries = 2964182b


C'est pas nécessaire de mettre une footnote, tu réutilise le terme SPA nul part ailleurs. Je mettrais directement `d'une "Single Page Application"`.


I think it's more appropriate to compute this in the `Comb` class, rather than here in `GCodePlanner` and passing it via the constructor to `Comb`. All this function seems to need is `storage`, which is already available to `Comb`.


This should be in the https://github.com/Homebrew/homebrew/blob/master/share/doc/homebrew/How-To-Open-a-Homebrew-Pull-Request-(and-get-it-merged).md document if anywhere.


throw in a clf command :+1:


There is some duplication - v= isn't the same, but start= and list= are.

I'll try and add Vimeo support in the next day or two get an idea how much is actually redundant.


On the rails 4 side of things, this test passes if I change the next line to

```
override_config: {"export_dir" => "/tmp/katello-repo-exports/repo_export", :start_date => "2010-01-01T00:00:00Z".to_time.iso8601.to_s})
```

I am fine with making that change on the rails4 branch since they rails 4 seems to handle dates differently


Note that this was already merged in 0.6.0.


It is weird. There is a glitch with Mac, and I can't click on the menu
items. Then I force them to redraw and it works. But this workaround
does not work with this context menu.

I'll investigate this.


ACK (```8.5.a```)


Comandi laterali suona meglio.


Tu devrais directement que le but est de générer du HTML à partir de scala. On dirait que tu tournes autour du pot ;)


ACK.


I have updated the comment.


Yes.


Inconisistent naming, what is "CanBeCanceledByUser", "CanBeCanceledByUser" or "channel"?


Shouldn't this be `Cheetah::ExecutionFailed` ?


Following this approach, google accounts migrated from Adium won't work, because we still need to skip the SRV.


There were some diffs in the answer files. But there were very few so I have updated the answer file and removed this snippet.


Some tests create an anonymous partition and these partition names are not constant.


J'éliminerais le 2e "de cette librairie": ceci n'est qu'un rapide tour d'horizon de quelques fonctionnalités de base, n'hésitez pas à explorer...


This should check the error message. The only reason why the test didn't fail before I guess.


I have updated the tests and removed this snippet from here.


Nope, my bad! Thanks


This looks quite strange. While you're at it, please document why you're moving inside while it's already inside according to `is_inside`. I get what you're doing only due to the commit message of this commit.


There were some diffs. But again there were very few of them so I updated the answer file and removed this.


And I haven't manually set them to `talk.google.com`. This is default configuration for Adium, expecting the strict behavior from libpurple.


I have updated the tests.


This guc was turned on by default when the tests were checked in. I have confirmed with QP team that this is no longer turned on by default and that answer file needs to be updated to reflect the current behavior.


if jobFlow != null, wouldn't it skip looping through the jobs?
Is that the intended effect?


Cleaned up text alignment.


:heart_eyes_cat: :thumbsup:


ACK (this rule is preventing password re-use => ```8.2.5``` is valid).


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


Beautiful


This is no longer required. I have removed this from the init_file.


只要 conversation 创建成功即可返回，不需要等到 staff 并发数保存成功。


ACK.


Generally you'd prefer a ```const int``` rather than a ```#define```, because the former typechecks during compile time and the latter does not.


dfg


In line 16, declare `tweets` like this to make sure it's never nil then and you don't need to unwrap it when using.
```Swift
var tweets = [Tweet]()
```


@sjrd could you have a look at this whether you think the overall architecture is reasonable when you have time?


@rmcmx


I don't know why I didn't fixed those. Thank you very much! :wine_glass:


ACK for ```8.7.c``` (preventing unauthorized access to /etc/group).


Which one?


This access is not coalesced. Threads 0-31 access locations 1-32. You want aligned accesses.


Prefer 1.0f rather than 1.


ACK (again, prevents unauthorized access to /etc/group) => ```8.7.c``` is valid.


@sjrd could you have a look at the architecture in the JSEnvs when you have time?


Yeah maybe Residue index 0 needs to be no Residue, but then Res tables are sized nRes+1 which is ugly. I guess that's today's headache


this method is a bit special ...
I'd simply have it return the actual location instead of a index in a list used later on ...
rename it to get_template_location_by_ip(...)


ACK (same case just /etc/passwd file now).


Could this have just been changed to `CFBundleName`?


ACK.


compilation error caused by missing "using namespace std" in get_impulse_response header file


You should remove this commented code.


OK, whatever. I clearly don't understand the distinction here, or what is happening. It sounds to make like you're talking about a different behavior than what I am talking about. Why doesn't talk.google.com work? Because it does an SRV lookup and that fails? Why does it do an SRV lookup on talk.google.com in the first place? It's supposed to do an SRV lookup on gmail.com or whatever, right?


Every thread writes back to the same memory location? Probably not what you want; again there will be cache effects, and the compiler might do some optimization there as well. Safer (in terms of this experiment) to do ```g_odata[i] = result```.


No shame here, hope everything's fine for you since Arrrrrcamp :smile:


```7.1``` makes sense (the rules is ensure unauthorized modification of boot configuration isn't possible). => ACK


Can we avoid using magic numbers? Maybe do a `CRM_Core_OptionGroup::vaues()` or some such?


Magic number.


Checking different property of the very same location as in previous case => ACK for ```7.1```


- "Android retains application screens in memory, and multitasking can result in the retention of an entire application in memory (even if the user logs out of their account). This allows an attacker that finds or steals a device to navigate directly to retained screens, which may include sensitive user data as part of the GUI. For example, if a user logs-out of a banking app but doesn't quit or close the app, a screen displaying transaction activity may be retained and viewable to an attacker."?
- Is multitasking as mentioned above just switching from one application to another or is it more than that?


Totally, I recently moved to La Réunion :sunglasses: :palm_tree:


TeamCity TripleA :: UnitTest [Build 259](http://home.k-moeller.dk/viewLog.html?buildId=333&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 259](http://home.k-moeller.dk/viewLog.html?buildId=333&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Number of tests is too small: 0 (new); unable to find runner parameter nunit_path required to run this build (new) Build time: 00:00:10


TeamCity TripleA :: UnitTest [Build 260](http://home.k-moeller.dk/viewLog.html?buildId=334&buildTypeId=TripleA_UnitTest&guest=1) is now running


Was this meant to be equal to the nozzle size? Surely there are lots of models with pieces thinner than 6mm for which this would give a problem.


still compiling error from like 3 commits ago lol


TeamCity TripleA :: UnitTest [Build 260](http://home.k-moeller.dk/viewLog.html?buildId=334&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Number of tests is too small: 0; unable to find runner parameter nunit_path required to run this build (new) Build time: 00:00:09


I think there is a problem here: Twig_Filter is an abstract class.

With Symfony-2.7 or 2.8, I get this:
Fatal Error: Cannot instantiate abstract class Twig_Filter


@tkocisky, since you modified this function, you must have understood why it is necessary in first place.
Why can't we simply:
```lua
for i = 1, T do table.insert(clones, model:clone(params, gradParams) end
```


```8.1.1``` makes sense for this rule. ACK.


- "While it's against Android design principles to quit your own app, it's far more secure because quitting the app will destroy any retained GUI screens."?


"que sea lo que dios quiera "... pero que funcione !!! :smile:
Exitos!!!


See: https://github.com/readium/readium-shared-js/commit/d41393053e9b9da541aa2610366c2bef7bba6658


Do you mean, we will not be able to run CI on windows because of bashism?
I haven't completely understand why you used word "relevance".


Please remove commented out code.


See: https://github.com/readium/readium-js-viewer/commit/f95b959922f3ab13e4997c0759b559a4b4d097cd


TeamCity TripleA :: UnitTest [Build 261](http://home.k-moeller.dk/viewLog.html?buildId=336&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 261](http://home.k-moeller.dk/viewLog.html?buildId=336&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Number of tests is too small: 0; unable to find runner parameter nunit_path required to run this build Build time: 00:00:09


varias cosas arregladas por lo visto !!! :smile:
Exitos!!!


```10.5.1, 10.5.2``` makes sense. ACK


It's (thankfully) no longer true that we can't change the name afterwards :)


ACK (ditto like in previous case).


Actually the project is starting the next weeks. Then you will see the commits coming :-)


ACK.


Hey sir, does this speed up the scanner? Does it support QR and data matrix?


std::string &name, core::log::Logger &logger


- "Any time an activity is initiated or a screen is accessed, execute a check to determine whether the user is in a logged-in state. If the user is not logged in, present the log-in screen."?


pssst


Kay!


Did you mean to resolve this conflict?


TeamCity TripleA :: UnitTest [Build 262](http://home.k-moeller.dk/viewLog.html?buildId=338&buildTypeId=TripleA_UnitTest&guest=1) is now running


@brianhatchl should be committed now.  let me know if that's not the case.


TeamCity TripleA :: UnitTest [Build 262](http://home.k-moeller.dk/viewLog.html?buildId=338&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Number of tests is too small: 0; failed to find C:\Program Files (x86)\Common Files\NUnit.org\nunit-console\nunit3-console.exe\nunit-console.exe (new) Build time: 00:00:06


hungarian


Bist du dir sicher, dass mit diesem Commit nicht auch der String in der rechten Bildhälfte geändert wird?

Links wäre **Übernehmen** korrekt, rechts hingegen **Hinzufügen**.
![unbenannt-1](https://cloud.githubusercontent.com/assets/1107939/11452148/a00b2566-95dd-11e5-963a-63b5e0a018a1.jpg)


EconomizerIntegration (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Its not that much they have 69.2HP against laser what is just a bit more than plate with 60.Plate its heavier thought but cheaper and faster to build.


https://github.com/twigphp/Twig/blob/master/lib/Twig/Filter.php#L19 Which version TWIG you use?


Все понял, хотелось бы еще спросить что именно нужно в презентации указать?
Рассказать об основных технологиях и принципах работы?

3 декабря 2015 г., 10:30 пользователь Mironenko <notifications@github.com>
написал:

> По отчету есть комментарии:
> 1. Номера страниц хорошо бы иметь.
> 2. Вместо исходного кода в математической составляющей лучше все-таки
> представить формулы и описания.
> 3. Диаграммы и схемы были бы полезнее исходного кода.
> 4. Не уверен, что имею право подписи за Быкова А. Ю.. На всякий случай
> подготовьте два титульных листа.
> Ожидается, что отчет дополняет вашу работу, а не дублирует ее.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DrDarkySoul/homework/commit/dcbd87ffc5ab0d01afc0b782f1bc9e8e118eec79#commitcomment-14754751>
> .
>


Gracias! !
El 03/12/2015 13:04, Diego van Haaster <notifications@github.com> escribió:varias cosas arregladas por lo visto !!!
Exitos!!!

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


oops, failed to actually call this method; fixed in #495


TeamCity TripleA :: UnitTest [Build 263](http://home.k-moeller.dk/viewLog.html?buildId=340&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 263](http://home.k-moeller.dk/viewLog.html?buildId=340&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Number of tests is too small: 0; failed to find C:\Program Files (x86)\NUnit.org\nunit-console\nunit3-console.exe\nunit-console.exe Build time: 00:00:06


there is no need to set nproc and my_pe to 0 here


TeamCity TripleA :: UnitTest [Build 264](http://home.k-moeller.dk/viewLog.html?buildId=342&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 264](http://home.k-moeller.dk/viewLog.html?buildId=342&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Number of tests is too small: 0; failed to find C:\Program Files (x86)\NUnit.org\nunit-console\nunit-console.exe Build time: 00:00:06


Make *sure* you understand what the third parameter in the ```<<<>>>``` kernel call is. It is shared memory, allocated per block. You are not using shared memory in your kernel, so you should not use that parameter at all (it will default to zero). What you are doing in your latest checkin is correct. If you declare shared memory above the max (48 kB on your GPU, I think), the kernel will not run at all (there will be a runtime error, but you are not checking for runtime errors, so maybe you should look into those as well). Both topics (shared memory and error checking) are in the CUDA C Programming Guide.


Created *.h and *.cpp files for each class in the project.
Verification of included headers in each header in the project.


This has severe performance implications because it is being called every time we rotate the map. This means that rotating the map is very laggy.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1100.327177da3c845a621aae35ecaf09a025187c1588](http://build.palaso.org/viewLog.html?buildId=66912&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 734, ignored: 6 Build time: 00:07:18


return ;


wew


return statement


EconomizerIntegration (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


TeamCity TripleA :: UnitTest [Build 265](http://home.k-moeller.dk/viewLog.html?buildId=344&buildTypeId=TripleA_UnitTest&guest=1) is now running


TeamCity TripleA :: UnitTest [Build 265](http://home.k-moeller.dk/viewLog.html?buildId=344&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Number of tests is too small: 0; failed to find C:\Program Files (x86)\NUnit.org\nunit-console\nunit-console.exe Build time: 00:00:07


$scope.model.value.values[language.isoCode].length is always 0/undefined for simple objects like {title:"something", description:"something"} -


TeamCity TripleA :: UnitTest [Build 266](http://home.k-moeller.dk/viewLog.html?buildId=346&buildTypeId=TripleA_UnitTest&guest=1) is now running


We definitely should support this.


i ran the code and the symbols are rendering correctly, it appears. not sure if i should give a thumb, since i cant verify the proper use of the symbols.


日本語が下手


> We need to delete antiblind, ,this does everything it does.

This will only kill blindness potions, but not beacon effects, etc.


TeamCity TripleA :: UnitTest [Build 266](http://home.k-moeller.dk/viewLog.html?buildId=346&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 142 Build time: 00:00:38


I use Twig-1.23.

All stable versions of Twig have this. Branch master seems to be for Twig-2.0 which is not released yet.


TeamCity TripleA :: UnitTest [Build 267](http://home.k-moeller.dk/viewLog.html?buildId=347&buildTypeId=TripleA_UnitTest&guest=1) is now running


refs #150


TeamCity TripleA :: UnitTest [Build 267](http://home.k-moeller.dk/viewLog.html?buildId=347&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 141 Build time: 00:00:35
### Failed tests
```
CashRegister.Test.Unit.dll: CashRegister.Test.Unit.Payment.PaymentControllerunitTest.Tally_StartChange1000_return1000: <no details avaliable>

```


TeamCity TripleA :: UnitTest [Build 268](http://home.k-moeller.dk/viewLog.html?buildId=348&buildTypeId=TripleA_UnitTest&guest=1) is now running


Looks good to me. Thanks!


TeamCity TripleA :: UnitTest [Build 268](http://home.k-moeller.dk/viewLog.html?buildId=348&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Tests passed: 203; number of inspection errors is too large: 12 (new) Build time: 00:00:43


這邊少一個空白哦！


Updated the url in the readme.


這個也是


Please remove the join as it causes issues when contacts are inserted and do not have purl records. Leave the group by.


чому не через | translate ?


@ueg1990 Instead of using `gist_example_list_data[0]`, any reason not to use a single gist for the example data?

## A single gist:
https://developer.github.com/v3/gists/#get-a-single-gist


I will get this fixed, good call.


ファイル名が微妙なのでsub.cssに変える


Uhh... let me rephrase the title... It specifically fixes the start menu to be a stationary object in relation to the oculus camera. That said, another bug was discovered during this commit that pretty much screws Oculus support over until we find the source. Adding it to issues now.


## Supported Linux Operating Systems


Yea, I've fixed this on a local build, need to committ it back.


ці "придатний" - "непридатний" не можна тримати десь в ресурс файлі?


Why is there still a concat here? I thought Pradeep had previously changed them in all queries in all sites to a more performant approach.


The idea here was that `basePath` in Transpiler effectively is the `cwd` for the purposes of the `Transpiler`. I'd rather keep the code independent of a global state side-effect like the current working directory. Can you apply base path on relative paths without depending on the `cwd`?


Why the concat?


Okay, great! Not too many people know about `git voodoo` :joy:


замінити на if (!data)


why the concat?


「呼びたい」が最初の文節と最後の文節で重複しているのでどっちかどうにかする


translate?


Ticketed in https://github.com/mapbox/mapbox-gl-native/issues/3174


I don't understand why we need `process.cwd` and `path.resolve` here at all. If we'd just take `p` as what it is and keep all paths relative, shouldn't `mkdirSync` work as expected?


hilokiさんのブログにリンク貼る


не обов*язково ці умови обгортати ще в одні дужки


Merge all


aaa


Also, I think it is a good idea to use `yaml_parse_file()` directly rather than creating a `Config` object every time.


qq


should we mention that this will be empty for now? I feel like it might be helpful. I don't insist though


dsfsd


bre


lgtm modolo comment above


What a mess. Thanks for reporting.


ccc


dcfsdef


sdf


sssss


So the problem was, when I was looping, I wanted the migration before the one that was greater than the datetime.  If I picked 10012015, and the migrations were:

20151010144703_migration_1
20151011144703_migration_1  - What I want to migrate to
20151012144703_migration_1  - Where the loop actually stops
20151013144703_migration_1

That's why I had to track the previous migration.


Are you doing a release soon @axelfontaine ?


LGTM.


最後がポエムでいいのか、もう一度考える


якщо формат дати використовується ще десь нра бекенді - винести його в Constants


The reason is that the output of a single gist is different from the output of a list of gists. If you look at https://developer.github.com/v3/gists/#list-gists, the values of the 'files' key has no 'content' key and that is what i wanted to test


Aren't we not supposed to call the API this way? Why not use the standard signature?


i.e. change this code to not use absPath, but rather just start with `var pathToCreate = '';`, and in each iteration `pathToCreate = path.join(pathToCreate, dirName);`?


typo. 不必申請帳號密碼**即**可使用


екстеншини теж винести в константи (Constants interface) : e.g. : String EXCEL_FILE_EXTENSION = "xls"


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


Might improve readability by doing:

```
$relationshipTypeId = $relationship['relationship_type_id'];
```

rather than all the nesting.


Replace NG-Grid with UI-Grid


Or not, merging is hard.


pas de ça dans les vues, s'il vous plait. Au mieux dans un modèle, au pire dans un contrôlleur. [`sizeof`](http://php.net/sizeof) est très très rarement utilisé.


Hi @f2prateek
Have you now got enough information to add the settings into the UI? Should we expect this to take long?


@gustl-arg I see you renamed fermat-book to 1-fermat-book. As the server reads the book from to generate the PDF's in the website, it needs a constant name of the main file to read, so we need the main asciidoc with a fixed name and avoid changing it. If that name is changed, the server can't find the book. what will be the final name?


> Why doesn't talk.google.com work? Because it does an SRV lookup and that fails?

Yes.

What have SRV records is `google.com` and `gmail.com`. Then it will fallback to the origin domain (the domain in my username) at port 5222 and connect directly. `gmail.com` does not listen to port `5222`, neither `thoughtworks.com`. So it will fail to do the TCP binding.

> Why does it do an SRV lookup on talk.google.com in the first place?

Per the spec, we are recommended to skip the SRV in this case because there is a configured `Server`. We only do because we decided to not be strict, and have `SkipSRVLookup`. Setting it when the user specify a custom server in the UI does not fix, because this is an imported account.

> It's supposed to do an SRV lookup on gmail.com or whatever, right?

Per spec, it is. But we do the SRV lookup using the configured server, otherwise would have no effect on the cases you mentioned before (direct manipulation of the config file, or imported from earlier).

This is how it works now:

If `SkipSRVLookup`
 - Connect directly to `Server` at default port. If there is no `Server`, we use origin domain.

No `SkipSRVLookup`
 - SRV to `Server`. If there is no `Server`, we use origin domain. (the spec says it should always be origin domain).
- If there is no SRV record,  connect directly to `Server` at default port. If there is no `Server`, we use origin domain.

I suspect we should always do the SRV lookup to the origin domain, regardless whether a custom server is configured or not, then fallback to the origin domain at default port. But in this case we would ignore the configured server unless `SkipSRVLookup` is set.


Internal
On Thu, Dec 3, 2015 at 10:56 AM Alfredo Canziani <notifications@github.com>
wrote:

> Which one?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wojciechz/learning_to_execute/commit/04a1659e471233ddb81280d9d98113286c1c459d#commitcomment-14765818>
> .
>


ça ressemble à une erreur.


All references to "virtual interface" when referencing to EPON should really be to "logical link", which is what is defined in 802.3. Hence the consistent change in this module.


dfafda


vbv


Can be done. But we'd need a cancel button alongside the save. Do you have a pattern for that?

Probably should do the same on email change page too.


Can this be overridden at the page or template level?


Does `.once()` work on `$(window)`?


Nice nice work on this Mike!


Issue https://github.com/centreon/centreon-plugins/issues/158


Forget my remark about non-fatal `HandshakeException`, I missed the `handleExceptionDuringHandshake` method.

I still don't know about connection ending ... Maybe this is safer to close both. The case were we have an established session + an ongoing handshake is no so common. I'm right, it's just in re-negotiation case ?
Should we send an INTERNAL_ERROR alert to the peer ?


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


Do you have to reboot to get this to take affect? I picked it into candy but only sticks after a reboot...


Ah damnit. I specifically noted "I need to undo this" and I didn't.  Fixed


thx opdenkamp :) and, since I'm also a comment freak. some comments as to why even do this, would be nice for those that stumble along long after the original authors are gone.


It's IE10+ that may be a problem. I say fuck it for now, but we can add the `display: inline-block` as a fallback if necessary since IE<10 will ignore `display: flex-item`.

Do you want me to add that?


Totally correct, not for 5.1 at all this is just my mistake!


Probably so since it works for normal logins.


We'll abstract an icon/button component once we use it more than once :grinning:


Section is blank? Also, change 'Understand' to 'Understanding'


Done


We should probably explain in a comment why we need to do this.


If you're going to setAppendStage() in pushNewBuffer(), the line above can be safely removed.


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


No you are not, the thing is that that fetch will only happen if the article is not in cache (https://github.com/joakin/loot-ui/blob/0b70b50b1f8723d0e17a0e892e123ecd718f2013/lib/client/actions.js#L57-L63 and https://github.com/joakin/loot-ui/blob/0b70b50b1f8723d0e17a0e892e123ecd718f2013/lib/client/actions.js#L76-L83)

We will have to implement some kind of cache invalidation soon


quick observations:

In Main.java, unclear why we need to create a new Map. Why not just add (around lie 31):

appProps.put("dispatcher", "http://mitlib-scads.appspot.com/workflows");

This would still assign the value, but would *not* wipe out any existing properties passed in.
Maybe even better:

if (! appProps.containsKey("dispatcher")) {
  appProps.put("dispatcher", "http://mitlib-scads.appspot.com/workflows");
}

which would only assign if missing, etc

Other observation in MetadataSpec.java:

Might want to leave a comment about the commented out fields - it could be somewhat mysterious to later readers....

:+1:  commit


mistake here.
"bs.command.enableMod=Better Sprinting is now allowed on the server."


solves redmine #417


TeamCity TripleA :: UnitTest [Build 269](http://home.k-moeller.dk/viewLog.html?buildId=349&buildTypeId=TripleA_UnitTest&guest=1) is now running


Oh! I see your point. I think you are not incorrect. But you are mistaken about the amount of information I've got before cloning it into FGMEMBERS.

On Nov 2nd/2015 alwall wrote
http://forum.flightgear.org/viewtopic.php?f=4&t=27894#p262799
-------------------------------------
>  I decide to collaborate with FlightGear project, making a <b>GNU/GPL Cessna 152 II</b> (IFR Panel) model

So I found the license style declared. Only I wanted to make sure to have the proper license file added as to reduce the risk of confusion.


Are you questioning whether `{{> 01-molecules/06-components/03-comment-header.mustache }}` is a valid partial?


RÈGLEMENT GÉNÉRAL DE L'AUTORITÉ DES MARCHÉS FINANCIERS/Livre IV/Titre I/Chapitre unique/Section 1/Sous-section 2/Article 411-11.md

Non on écrit "de l'OPCVM"


TeamCity TripleA :: UnitTest [Build 269](http://home.k-moeller.dk/viewLog.html?buildId=349&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Tests passed: 203; number of inspection errors is too large: 12 Build time: 00:00:37


@matzew its not working yet. And at this moment im not sure why. But i will send u PR, when it will be ready.


thanks for quick turn round. I did a quick test and test was passed. thanks.

2015-12-03 08:32:11.409806 7ffa54f654c0  0  set rocksdb option compression = kNoCompression
2015-12-03 08:32:11.409819 7ffa54f654c0  0  set rocksdb option max_write_buffer_number = 16
2015-12-03 08:32:11.409831 7ffa54f654c0  0  set rocksdb option min_write_buffer_number_to_merge = 6
2015-12-03 08:32:11.409869 7ffa54f654c0  0  set rocksdb option compression = kNoCompression
2015-12-03 08:32:11.409879 7ffa54f654c0  0  set rocksdb option max_write_buffer_number = 16
2015-12-03 08:32:11.409890 7ffa54f654c0  0  set rocksdb option min_write_buffer_number_to_merge = 6
2015-12-03 08:32:11.412155 7ffa54f654c0  1 newstore(/mnt/nvmedevice) _open_db opened rocksdb path /mnt/nvmedevice options compression=kNoCompression,max_write_buffer_number=16,min_write_buffer_number_to_merge=6
2015-12-03 08:32:11.412235 7ffa54f654c0  1 newstore(/mnt/nvmedevice) _recover_next_fid old fid_max 0/0
2015-12-03 08:32:11.412256 7ffa54f654c0  1 newstore(/mnt/nvmedevice) _recover_next_nid old nid_max 0
Write cycle 0
2015-12-03 08:32:11.414237 7ffa40fe9700  0 Writing 2 GB in blocks of 4 KB
2015-12-03 08:35:14.678450 7ffa54f654c0  0 Wrote 2 GB in 183263948us, at a rate of 11443 KB/s and 2860 iops
2015-12-03 08:35:17.074995 7ffa54f654c0  1 newstore(/mnt/nvmedevice) umount


Good point - I'll probably rewrite it to work as a test tool and display some output to the console. The original file was simply a front end to test my library, back when I had a single Makefile generating a static binary.


No! This field is for the /passport/ country code, not the ISO code!! That is e.g. "SWE" for Sweden.

Max length is 4 according to international standard.


We should also have a test for this?!


Cette modification de correspond pas à la demande, cf PJ
[AMP_France_Liquidity_contracts_merged.pdf](https://github.com/schambon/rg/files/51055/AMP_France_Liquidity_contracts_merged.pdf)


> the values of the 'files' key has no 'content' key and that is what i wanted to test

@ueg1990 the attributes in a gist should be the same for a gist in a list. From https://developer.github.com/v3/gists/#get-a-single-gist, I do see the `content` key.

Also, in the unit test, the only reference to `gist_example_list_data` is to access the a single element `gist_example_list_data[0]`.

```javascript
  "files": {
    "ring.erl": {
      "size": 932,
      "raw_url": "https://gist.githubusercontent.com/raw/365370/8c4d2d43d178df44f4c03a7f2ac0ff512853564e/ring.erl",
      "type": "text/plain",
      "language": "Erlang",
      "truncated": false,
      "content": "contents of gist"
    }
  },
```


You use this exact code twice (also on line 293+). Perhaps create a small function?


TeamCity TripleA :: UnitTest [Build 270](http://home.k-moeller.dk/viewLog.html?buildId=350&buildTypeId=TripleA_UnitTest&guest=1) is now running


agree


TeamCity TripleA :: UnitTest [Build 270](http://home.k-moeller.dk/viewLog.html?buildId=350&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Tests passed: 203; number of inspection errors is too large: 12 Build time: 00:00:37


Congratulations!


Changes look fine, thanks @Justoce


Thanks, an important detail! :-D


you can try


Amazingly this was already buggy; I fixed this in 60e294d72ef1e86758e09a9347fb063c0c31c55c.


Adding `ansible_sudo=true` to the inventory file allows it to work without the `--sudo` flag as well. Though that obviously affects how things are provisioned.


OK, corrigé


Minor, but I think we can make these non-temp files, since the `deleteOnExit` hook will apply to them as well (thus elliminating the generated part of the file name). Will check and update on merge.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7892](https://teamcity.mesosphere.io/viewLog.html?buildId=85636&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:57


What problem, exactly, does this commit solve? I see that you're dealing with the edge case where contactIds is a string instead of an array, but why do we need to look up the existing contacts before saving the passed ones?

Also, it occurs to me that this create function doesn't have a way to deal with the contacts that are excluded from the params. In other words, say I load up Project 1, and it has Smith and Jones as managers. I cross out Jones and submit. Only Smith is passed in, and there's no code to deal with the removal of Jones.

Maybe we should delete all the relationships and recreate them each time based on the params. Of course, if the API is being used for a simple update (fix a typo in the title), then that would be undesirable. Is the case of Jones addressed elsewhere?


It would have been super nice to have here some warning that this has changed from 1.0.0. As I'm targeting to production use, I downloaded 1.0.0 and not a release candidate of newer version. Github was showing me the new directory structure although version 1.0.0 had a different one which led to some errors and debugging time. Well, solved now anyway, but for others trying the library for the first time, this is sort of issue can be bad reputation for the library.


The idea is that if I pass a `CsvOptions` then the `toPb` sets both `csvOptions` and `format` in the proto, if I pass a `FormatOptions` instead the method only sets `format`.


Вмовили )


良いとする。


`[CWE 200: Information Exposure] (http://cwe.mitre.org/data/definitions/200.html)`
We want to format all references to CWEs in the same way (including the title)


I thought that the header guard would't allow that... actually we could put it inside the gp regression class (as private?)


You can drop this line.  The compiler only complains about missing declarations for non-static functions (which is why the declaration was in there before).


@mhgoldman I *think* we might be talking about different things. The line that might have problems using stale data is:

```ruby
sortable by: [:course_name] + Region.all.map(&:to_sanitized_s), default: 'course_created_at'
```

If the app server is running, and you add new Regions to the db in `rails c`, without restarting, do the new Regions sorting display/work as expected with `sortable`?


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


Thank you. Now I know why PSGI wasn't working when I renamed it's META.info. It's been way too long since I've been active in this community, I've forgotten how some things work.


EconomizerIntegration (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (887 of 887 tests passed)

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


Typo here.


It's not only countries issuing passports - this validator is wrong. Similar for driving licenses.

It's not about GIS ;)


Iptables_Semantics: theory Code_haskell
Timing Iptables_Semantics (8 threads, 127.808s elapsed time, 803.096s cpu time, 95.060s GC time, factor 6.28)
Finished Iptables_Semantics (0:03:34 elapsed time, 0:17:07 cpu time, factor 4.79)
Running TEST ...
TEST: theory IP_Address_Space_Examples_All_Small
TEST: theory IP_Address_Space_Examples_All_Large
Timing TEST (8 threads, 9939.816s elapsed time, 61896.892s cpu time, 1458.168s GC time, factor 6.23)
Finished TEST (2:45:50 elapsed time, 17:11:48 cpu time, factor 6.22)
Finished at Thu Dec  3 16:58:20 CET 2015
2:49:31 elapsed time, 17:29:27 cpu time, factor 6.19
../memusg ../Isabelle2015/bin/isabelle build -d . -v -c TEST  62809.85s user 158.88s system 619% cpu 2:49:31.36 total
memusg: peak=623316


I would also hope that it will be extremely rare that such an unexpected RuntimeException will be thrown. In any case this reflects an implementation error in our stack ...
So, I think terminating the connection (incl. any ongoing handshake) by means of a FATAL INTERNAL_ERROR Alert is a good strategy for now :-) Let's see how we fare in reality with it ... I will add this tomorrow if you don't mind. Then we can have a final look at the code and merge. We should then also talk to Matthias regarding a 1.0.1 bugfix release ...


When someone reads the "We have disabled ignoring" sentence a year from now, it will mean nothing to him, without knowing the history.


If assertions are turned off in Release mode, mt_options.threads won't get decremented


[Новый текстовый документ.txt](https://github.com/vyuzh/fisrst-repka/files/51066/default.txt)


@giangbinhtran This is mentioned in https://github.com/EUMSSI/EUMSSI-platform/wiki/API#core-services where all current documentation should be. I am currently updating the documentation. If you find anything wrong, please inform me.

In fact, I believe that your change is wrong: the source name goes in the `source` field, whereas the metadata coming from the source (in Eumssi format) goes in `meta.source` (in contrast to `meta.original` which is the raw unmapped metadata and `meta.extracted` which is metadata generated through analysis processes.)


Format nitpick - `} else {`


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7893](https://teamcity.mesosphere.io/viewLog.html?buildId=85637&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:27


Calling this `$contacts` is a little confusing. Later, when you reference $contact['id'], I expect that to be the contact ID. Maybe call it `$projectContacts` instead?


"You can install the qt4 optional dependency in order to use the graphical interface which is based on Qt. This is not required if you intend to use VirtualBox in command-line only." - Arch Wiki
Doesnt make any sense install qt4 when installing the guest packages, since you don't even gone use virtualbox inside virtualbox :P


good


Ahhhh, yes, we totally are talking about different things. I'll have to
mull that one over...

On Thu, Dec 3, 2015 at 11:45 AM, Eliot Sykes <notifications@github.com>
wrote:

> @mhgoldman <https://github.com/mhgoldman> I *think* we might be talking
> about different things. The line that might have problems using stale data
> is:
>
> sortable by: [:course_name] + Region.all.map(&:to_sanitized_s), default: 'course_created_at'
>
> If the app server is running, and you add new Regions to the db in rails c,
> without restarting, do the new Regions sorting display/work as expected
> with sortable?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mhgoldman/course_manager/commit/b142a26b19db41f546f07f4f0029d6763c1d7cff#commitcomment-14767242>
> .
>


This is a little hard to read. Some short-lived variables might help with that.


Refugees for example have "XXB", the German passport has just "D".


typo: emtpy


I add country :)


remove empty line


I think we don't use backsticks in the error messages.


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


Nice :smile:


Probably this needs removing?


It could be easier to maintain if a python string was used for this section.

```
    epilog = ('ESPA Bulk Download Client Version 1.0.0. [Tested with Python 2.7]\n'
              'Retrieves all completed scenes for the user/order\n'
              'and places them into the target directory.\n'
              'Scenes are organized by order.\n\n'
              'It is safe to cancel and restart the client, as it will\n'
              'only download scenes one time (per directory)\n'
              ' \n'
              '*** Important ***\n'
              'If you intend to automate execution of this script,\n'
              'please take care to ensure only 1 instance runs at a time.\n'
              'Also please do not schedule execution more frequently than\n'
              'once per hour.\n'
              ' \n'
              '------------\n'
              'Examples:\n'
              '------------\n'
              'Linux/Mac: ./download_espa_order.py -e your_email@server.com -o ALL -d /some/directory/with/free/space\n\n'
              'Windows:   C:\python27\python download_espa_order.py -e your_email@server.com -o ALL -d C:\some\directory\with\\free\space'
              '\n ')
```
Or possibly using the python mutli-line string format
```
epilog = '''ESPA Bulk Download Client Version 1.0.0. [Tested with Python 2.7]
and places them into the target directory.
...
'''
```


So I spent way too long in the internals of [mongoid](https://github.com/mongodb/mongoid), [origin](https://github.com/mongoid/origin) and [moped](https://github.com/mongoid/moped).

The question is, could somebody craft a string for `type`, so that something unexpected can be executed. Something like `links."drop_tables()`.

What I found is that the keys are not escaped, but that our hash is sent to Mongo [serialised as BSON](https://github.com/mongoid/moped/blob/1c019fbf9e3477c5adf41996e92b10b199cce96d/lib/moped/protocol/message.rb#L166). MongoDB deserialises the payload, and gets back a hash-like object to use.

This would mean that at no point the Mongo entire query is converted into a string, so that the "links.user_input" string can be mis-interpreted. Unlike SQL with unprepared statements, which sends a string to the server.

TLDR: keys in a hash for `where` are safe.

@bradwright hope this makes sense.


And maybe it wouldn't be problem at all if regions aren't regularly added.


TeamCity TripleA :: UnitTest [Build 271](http://home.k-moeller.dk/viewLog.html?buildId=351&buildTypeId=TripleA_UnitTest&guest=1) is now running


Apparently there was a legitimate conflict, it appears as though readme was updated in master after this branch was created and then updated in this branch, am going to manually update the readme to have the changes from this branch as well as the other changes from the meeting.


can't happen.  I'll remove


Haha, so as the world works.. I got bitten by this, BUT! Have a better solution rather. Hopefully I can have a PR ready by the end of today :)


TeamCity TripleA :: UnitTest [Build 271](http://home.k-moeller.dk/viewLog.html?buildId=351&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Tests passed: 203; number of inspection errors is too large: 12 Build time: 00:00:36


Really, can this be more cryptic as a name ?


Couple of spelling/grammar errors here:

> Docker will automatically deliver a `SIGTERM` with `docker stop`; Not when...

Should be a comma, not a semi-colon, and shouldn't have "Not" capitalized.

> Make sure this timeout is less than the docker stop timout

Should be "timeout"


Pretty sure I already griped Jacques00 out for this >.>


Lol whoops well the plus is they are the exact same file ;)


Thanks.


Yeah, it occurred to me to wonder. I'm fine either way, it's just not completely clear from the documentation.


Virtualbox needs qt , without it UI doesn't start :P
VirtualBox: supR3HardenedMainGetTrustedMain: dlopen("/usr/lib/virtualbox/VirtualBox.so",) failed: libQtCore.so.4: cannot open shared object file: No such file or directory


You can delete the dsdt_code/dsdt_size fields form AcpiMiscInfo in the same patch, too.


@NatsuArashi My fault. Thanks for reminding. :)


Hello Swift


So Swift codename was Swift!


If you need an extra field for the country of origin, then you must add one - this field is used to describe the identity document and that does not use alpha-2 codes. That is why it is "country_code" not "country".

The validator invalidates a lot of data - this field simply has a different meaning.


This is great.


Nvm
Wront virtualbox ...


awesome, thanks!

On Thu, Dec 3, 2015 at 5:36 PM, pecsi <notifications@github.com> wrote:

> @matzew <https://github.com/matzew> its not working yet. And at this
> moment im not sure why. But i will send u PR, when it will be ready.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pecsi/aerogear-unifiedpush-server/commit/5443a545f9adbed76e592313262b5f8a43f24f24#commitcomment-14767027>
> .
>

--
Matthias Wessendorf

blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf


Can this be replaced by PCMachineClass fields that tell acpi-build.c what to do?


Ah, I see now that this is handled in your next commit. Still feel like there are some outstanding questions, though:

1. Mightn't we accidentally delete relationships for simple edits? If I just change the title, do I risk destroying the relationship data? Maybe we can add a check that says, don't delete extraneous relationships unless project_contacts is set.
2. I'm not sure that this approach is more efficient than just dropping all the relationships if project_contacts is set and adding back only those specified. What do you think?


@jQueryBase64MyAss ily


Mental. The future started 5 years ago.


I think that you should add a check for sub_type->flags & DL_TYPE_FLAG_HAS_SUBDATA and decide if you should do the patch-loop or just call the old code.


People drilling down to page 916! :D


Yes, the latest commit seems to fix it. Thanks!


Why are you still using your own class ? This file should just be removed... :(


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


What's the purpose of this change?


TeamCity TripleA :: UnitTest [Build 272](http://home.k-moeller.dk/viewLog.html?buildId=352&buildTypeId=TripleA_UnitTest&guest=1) is now running


:sparkles:


@helmuthdu fixed :)


Crashes where? It works for me..


haha @nunogoncalves  !
And it's not easy to find out, you need to be playing between num. ranges


So, the way I'm currently imagining that this might work (pseudocode):

```rb
class PuppetDBHAState
   class RackHAStateManager
      ...
   end

   class PuppetServerHAStateManager
      def initialize(puppet_server_terminus_shared_context)
         @context = puppet_server_terminus_shared_context
      end
   end

   # class-level variable to use as state manager
   @ha_state_manager = RackHAStateManager.new

   # initializer made available for puppet server to call during startup
   def self.initialize_for_puppet_server(puppet_server_terminus_shared_context)
      @ha_state_manager = PuppetServerHAStateManager.new(puppet_server_terminus_shared_context)
   end
end
```

In this model, we could add something in the server code, where there was a generic ConcurrentHashMap that was created at startup, in the Clojure layer, to be used to store shared state for any terminus that wants to re-use data across multiple JRuby instances.  It would have a key for each terminus and then the values would be up to the terminus to populate (with access to any of the basic Java concurrent data structures in `java.util.concurrent`).

We could have a config setting in puppet server that allowed you to provide a list of terminus classes, and then during our JRuby startup, we could iterate over that list and call the `initialize_for_puppet_server` method for each terminus, and pass in the shared state object.

This initialization logic would obviously have to have some error handling for cases where the terminus wasn't installed, but that should be doable.

Sorry it has taken me so long to find time to respond to this stuff, I know that it would have been more useful to do this when the original conversations were going on and the context was fresher in everyone's minds.  We're pretty much out of the Ankeny stuff at this point, so if it would be helpful to get together on a whiteboard or in a video chat sometime soon I can definitely find time for it.


TeamCity TripleA :: UnitTest [Build 272](http://home.k-moeller.dk/viewLog.html?buildId=352&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Tests passed: 203; number of inspection errors is too large: 13 Build time: 00:00:36


Дякую за те, що подивилися. Буду більш уважним.


Me too. lol


Much less work if you do it on the commandline :wink:


Hello!


`Editor` is usually injected in "consumer" services. If it's final it can't be mocked...


TeamCity TripleA :: UnitTest [Build 273](http://home.k-moeller.dk/viewLog.html?buildId=353&buildTypeId=TripleA_UnitTest&guest=1) is now running


你好！Swift！


@rmcmx


I'm pretty sure this breaks multicast and/or hybrid operation (at least on Linux), I don't remember why now but there was a reason why this wasn't there for multicast. This needs carefully tested before release. Note that this also introduces an extra hidden behaviour behind ignore_daemon_lock.

In the future we may want to change this to add a listen address setting (because currently we can't pick a source address for interfaces with multiple IPs - secondary IPs can appear as interface aliases, but they don't have to).


TeamCity TripleA :: UnitTest [Build 273](http://home.k-moeller.dk/viewLog.html?buildId=353&buildTypeId=TripleA_UnitTest&guest=1) outcome was **FAILURE**
Summary: Tests passed: 203; number of inspection errors is too large: 12 Build time: 00:00:36


Yay!


National ID cards are following the same schema as passports, whereas driving licenses follow https://en.wikipedia.org/wiki/List_of_international_vehicle_registration_codes
...which can be 1-3 letters, some using ISO-alpha2, some ISO-alpha3, and some not ISO at all.


Pröva nått sånt här.

for (int i = 0; i < pVector->size; i++)
{
	if (strcmp(word, pVector->data[i]) > 1)
	{
		continue;
	}
	else if (strcmp(word, pVector->data[i]) < 1)
	{
		return i;
	}
        else
        {
                // Ordet finns redan, return error, nått som jag ska fixa ;)
         }
}


The code looks good to me but I haven't had a chance to thoroughly test it
with the customer API yet. When I merge to my internal branch where I have
that all setup I'll investigate and fix any bugs.

On Tue, Dec 1, 2015 at 2:45 PM, Elan Ruusamäe <notifications@github.com>
wrote:

> this changeset was more than trivial considering it was rather small file
>
> could you @balsdorf <https://github.com/balsdorf> have review of this
> changeset and complain if you don't like something. as i likely broke some
> existing flows which did not see to exist.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/eventum/eventum/commit/aa7ce3b6f4642a71f4cd29efb7147d7d61b1be8b#commitcomment-14719789>
> .
>

--
Bryan Alsdorf
MariaDB Corporation | http://mariadb.com


Onward to the future!


Gracias por la solucion


The purpose of the Invalid state is to detect fatal conditions in the AppendPipeline state machine. If you set ok to true here, you're defeating that purpose. We need to analyze what's wrong with the the specific case that needs having an "ok" here.


it's a buggy line from [IE11 regression in 0.7.7] (https://github.com/Leaflet/Leaflet/issues/4061)


E esse enum vazio ae?


Hi~


The docstring got duplicated somewhere along the way (probably during my rebase).  Please remove one.


It is in original code though, so it makes me wonder why it is like that?

2015-12-03 17:53 GMT+01:00 Rebecca Schlussel <notifications@github.com>:

> can't happen. I'll remove
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Teradata/re2j/commit/ee5be0d4a1f1c26d33ae2d8f0cbd9b7437dcc946#commitcomment-14767419>
> .
>


I'm here for the fun! Have fun hacking around, folks!


second part of the issue [#2717](https://github.com/shogun-toolbox/shogun/issues/2717)


dickhead is not a good variable name please stop


Since `self.messages()` gets reset to `[]`, won't this line always evaluate to true?


Forget about what I mentioned above about upper vs lower case - You should be consistent with rest of file and in other parts there is used convention `stderr("fetch: error message: %v", err)` in case of error, or `stdout("fetch: something informational")` in case of informational messages. With this construction messages can be hidden in quiet mode, or shown in verbose mode.


git log --reverse


When finished, please remove these two console.logs.


Forget about what I mentioned above about upper vs lower case - You should be consistent with rest of file and in other parts there is used convention stderr("fetch: error message: %v", err) in case of error, or stdout("fetch: something informational") in case of informational messages. With this construction messages can be hidden in quiet mode, or shown in verbose mode.


PascalCase eim...


Glad I'm not the only person who thought about coming here! :)


better to remove console.log(json) for production :)


Gotcha. I missed that that was your intention and thought of a location mix up.


hello swift


Hello Swift!


Wonder what he had in mind when he did this first commit.  "THIS IS GONNA TAKE OVER THE WORLD!!"

Yes, yes it did.


:+1: I will test to see if this commit has made a difference with
https://github.com/pimcore/pimcore/issues/294

I still could not work out what was happening in there, but this commit will certainly help clarity


Hi Swift,

I will take my time to know more about you :kissing_heart:

Thanks and cheers,
Tai


#795 Tabular File Upload Table View Issue


Project, not event.


LGTM


Thank you for Swift.


You are right that `upload` and `delete` need to be implemented for the provider to work.  Instead of throwing a Python NotImplementedError, they should throw an HTTP 501 Not Implemented error, with an appropriate error message.  Looking through `waterbutler.core.exceptions`, I don't see a generic one, so you'll need to add that.  I might suggest a `ReadOnlyProviderError` with a 501 code and an appropriate message.  That way, if someone munges the url to make an upload request, they get a sensible error rather than a 500.


Organizational nitpick, please move this up with `validate_path`.


Parece que mola...


hola se que su grupo de trabajo final tiene un space invaders  el cual funciona bien , me estaria preguntando si podrian por favor pasarme el codigo de disparo
muchas gracias


Thanks for the detailed report @terhechte :)

Hmmm in your screenshot it actually looks like you've got the wrong completion results (the correct results would just have 2 completion suggestions: `a` and `b`.) I'm guessing this is because you've added `1` to the offset which would place XCode's completion point at the next line. This would be the cause of the timeouts I imagine hmmm...

I've made some big changes to company-sourcekit (nathankot/company-sourcekit@d587b10b2427) that rely on terhechte/SourceKittenDaemon#17. This should fix the problem with the first completion not working and make things smoother overall (it was spinning up too many daemons prior as well.) so please try it out with the new sourcekittendaemon binary and let me know how it goes!

If it's still not working with your test project could you try it on the sourcekittendaemon project itself? It seems to work for me on that project, but not any other ones :S


Maybe we should have a PLATFORM(BRCM) or some similar ifdef around this change.


This "verbose" syntax is not yet on the patternlab.io documentation. It was a feature recently requested for those that desire a clearer understanding of where a partial comes from. My preference is still the 'key' approach, as in `molecules-comment-header` but I can understand how that might fatigue people over time. [This release](https://github.com/pattern-lab/patternlab-node/releases/tag/v0.12.0) introduced the functionality. I tried my first to review it but it may have quirks.

So really whenever we test for:
`{{> molecules-comment-header }}`

we also attempt to support:
`{{> 01-molecules/06-components/03-comment-header.mustache }}`

Hope this helps.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 86](https://teamcity.labkey.org/viewLog.html?buildId=289138&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 6, passed: 41 Build time: 01:12:17
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

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

org.labkey.test.tests.CDSTest.verifyCounts: <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.verifyAssaySummary: <no details avaliable>

```


swift ha


Out of curiosity what was breaking without this change?


@carlosjoserg No, include guard only prevents multiple inclusions (that would cause compilation fail).
But since those headers are included in ALL cpp files, gcc creates multiple compilation units, that compile just fine (thanks to the include guard); but when the linker arrives it sees that function is defined in multiple compilation unit and it doesnt know which one must be linked to assemble the program (even if they are all identical!), hence it fails.

Look at #5 for possible solutions i highlighted, yes one of them, is moving that function inside GPRegression class


Ah. Glad to hear.

Anders

On Thu, Dec 3, 2015 at 8:58 AM, gokulnair <notifications@github.com> wrote:

> Yes, the latest commit seems to fix it. Thanks!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Andersbakken/rtags/commit/b7738c9a7f15802b1bad608e0c833e97e1a76ae7#commitcomment-14767543>
> .
>


that is the thing...its not...look at the example data in the two links: https://developer.github.com/v3/gists/#get-a-single-gist and https://developer.github.com/v3/gists/#list-gists and look at the field files in both examples. the one in list-gists does not have the key 'content'.

The only reason i took the first element of gist_example_list_data is because i was writing unit tests for the class GistFile and each instance takes only one element of a gist. But because i wanted to test the scenario  where the key 'content' is not available i used the example of list gist data


@wluyima do you have any idea what could be wrong here? My main page displays: OpenMRS Platform ${project.version}.${revisionNumber} Running!
So it does not replace the above variables.


This should be done only in the subdirectories that needs it (it should be just Runtime eventually). It is not applicable to all Native projects (System.Private.CoreLib.Native in particular).


Hello Swift！


good old tick :ok_hand:


If we are going to have this pattern, it should probably also be used on other edit panes too. Would be better UX anyway to have that cancel functionality throughout don't you think?


so Cool!


Hello there !


Hmm. I thought @cfieber and I tested this specifically?


I agree.  Lowercase "methodgen" it is.  I will make the change right now.


Hello Swift!


In gantry frameworks its not floating all the way.http://bit.ly/1lbP9SX
Thanks
--GS


Please top spamming


Hm... Passing the original 'visitationHelper' object instead of 'visitationHelper.Clone()' into the recursive function will populate it with visited objects effectively causing three further calls of 'context.ApplyChangesOnProperties' (lines 190-192) do nothing. Not sure it this is intended.


Yes, thank you! I'll note that in the comments. I like the key syntax
better too. It's more semantic.

I was specifically wondering whether the verbose syntax should be:
{{> 01-molecules/06-components/03-comment-header.mustache }}
or
{{> 01-molecules/06-components/03-comment-header }}

Seems helpful, or least harmless, to be tolerant of both.

On Thu, Dec 3, 2015 at 11:09 AM Brian Muenzenmeyer <notifications@github.com>
wrote:

> This "verbose" syntax is not yet on the patternlab.io documentation. It
> was a feature recently requested for those that desire a clearer
> understanding of where a partial comes from. My preference is still the
> 'key' approach, as in molecules-comment-header but I can understand how
> that might fatigue people over time. This release
> <https://github.com/pattern-lab/patternlab-node/releases/tag/v0.12.0>
> introduced the functionality. I tried my first to review it but it may have
> quirks.
>
> So really whenever we test for:
> {{> molecules-comment-header }}
>
> we also attempt to support:
> {{> 01-molecules/06-components/03-comment-header.mustache }}
>
> Hope this helps.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geoffp/patternlab-node/commit/ee6fcfc6500fea4be9a07cd005e7c26d95ca1858#commitcomment-14767839>
> .
>


Nice to see you Swift!


@shigeki Could you update the comment to say `node`?


I love Swift!


デフォルトを直したらテストも直してください :bow:


Thanks,Chris Lattner


你好,Swift


lets see


这个博客暂时不用了,这是我的新博客地址:man1ac.me


Bonjour Swift.


Maybe it would look better to have these in inc\Runtime?


Notes:
-Database used is from a free mongodb hosting service (modulus.io), but I also ran it from a local database and it worked just fine.
-The console will show all the entries in the database.


Could you put a TODO here, for the distant future to indicate that this is somewhat expensive operation, especially if we want to run this query often. We'll need a more general mechanism to compute (and possibly cache) reverse dependencies in Envision.


Taylor Swift hype!


I don't really know c++, so maybe I'm misunderstanding how it works, but it looks to me like in the original code the special states aren't real state objects. In that case you need to check that it's not a special state anytime you want to actually access a field.


Also another issue fixed was the code for adding/removing class.


I have added a MRP function. Users can select two planes perpendicular to the source image plane to reconstruct corresponding images by left-clicking on the source image, or move the planes pixel by pixel by right-clicking on the source image.


@trecloux can you fix this line's formatting?


I found CHANGES.txt, which you probably meant.


- should "Broadcast Intent" be "sendBroadcast (Intent)"?
- add comma before "...the data"


Hello Swift :)


This seems a little odd to me. I'd be interested in hearing the rationale behind defaulting to a reserved word surrounded in backpacks when importing. Seems like it wouldn't be easily discoverable... :confused:

Thoughts?


Yes, you are right.


What are you testing for with `path[0] === '.'?`


It's not very obvious that there is an edx-specific addition to the license here at the end of the file. I think most people will read the title "AGPLv3" and assume the contents are the standard license.


Thanks for the tip. I will try this tonight.


Bring it light


Hello Swift!!


Haha yep, but breaks for those you don't have angler synced :P


Neither of these conferences look like software testing conferences. One appears to be about assessment testing.


Welcome :)


In what way is `std::initializer_list` not a `View` of its backing array?


Thank you for fixing the bug. I personally think it is nice to compare resulting pointer returned from new() operator against NULL as few C++ implementation (IIRC, Microsoft VC) does not throw exception when new() fails.


 hello


welcome :dancer:


It's unsafe when unwrapping using `!`. You can unwrap like this:
```Swift
if let dictionary = dictionary["user"] as? NSDictionary {
  let reweetUser = User(dictionary: dictionary)
  // other code
}
```


@bmuenzenmeyer, does this look sane, or is this a unit test cop-out? What was there before looks like a verbose partial call, but without the 00-atoms bucket in front. Do we need to support that?


- I don't think "Intent" should be capitalized here (in Android documentation it doesn't seem to be). I'm easily persuadable otherwise if someone feels strongly. I'd just want to make sure we're consistent.
- "When another application initiates activity by sending a broadcast intent, malicious apps can read the data included in the intent."?
- "For example, if an app invokes and passes a URL to the Android web browser, an attacker could sniff that URL."?


Okey then, that makes sense, but why it can't happen in our code that it is
a special state? Can it be a DEAD state? If so, then it doesn't impose any
action at the end of loop probably.

2015-12-03 18:18 GMT+01:00 Rebecca Schlussel <notifications@github.com>:

> I don't really know c++, so maybe I'm misunderstanding how it works, but
> it looks to me like in the original code the special states aren't real
> state objects. In that case you need to check that it's not a special state
> anytime you want to actually access a field.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Teradata/re2j/commit/ee5be0d4a1f1c26d33ae2d8f0cbd9b7437dcc946#commitcomment-14768071>
> .
>


My point is you're buffing Energy resists across the board (Powered Armor, anyone?) while reducing Projectile and Explosive resistances.


I think you were right, this is a problem. I think I can solve it by
letting sortable take a Proc or something... I'll work on it

On Thu, Dec 3, 2015 at 11:52 AM, Eliot Sykes <notifications@github.com>
wrote:

> And maybe it wouldn't be problem at all if regions aren't regularly added.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mhgoldman/course_manager/commit/b142a26b19db41f546f07f4f0029d6763c1d7cff#commitcomment-14767407>
> .
>


Please put `{` and `}` at right positions.


fixed in #6060


Hello! :+1:


I wasn't aware of it, but I have addressed it by removing all underscores for class and ID names.  I also added UUID generation to address the issue you brought up with the Glow Filter ID... overkill, I know, but should avoid conflicts!


@bradenmacdonald Thanks for adding a LICENSE file! (Comment for testing purposes.)


- "Do not pass sensitive data between apps using broadcast intents. instead, use explicit intents."?


Use `if let` to unwrap `response` before using.
```Swift
if let dictionary = response as? NSDictionary {
  tweet = Tweet(dictionary: dictionary)
  // other code
}
```


TeamCity TripleA :: UnitTest [Build 274](http://home.k-moeller.dk/viewLog.html?buildId=354&buildTypeId=TripleA_UnitTest&guest=1) is now running


a


Looks like unnecessary blank line. Don't put a blank lines after `{` and before `}`, the prefered format should be
```Swift
someBlock {
  someOtherBlock { // no blank line here
  }
  // blank line here is okay
  someVar = someMoreCode
  ...
  // blank line here is okay
  someFinalCleanup() // no blank line at end of block either
}
```
In short, in a block, no blank line right after the start `{` and right before the `}`. Blank lines are welcome anywhere else in the body though. Don't use double lines to separate your code.


`[CWE 285: Improper Authorization](http://cwe.mitre.org/data/definitions/285.html)`


TeamCity TripleA :: UnitTest [Build 274](http://home.k-moeller.dk/viewLog.html?buildId=354&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 203 Build time: 00:00:37


Note line 18 has a "d" at the end. Guess it is typo.


When is the plan to provide a new tag/build with this newly added license designation? [WebJars](http://www.webjars.org/) is needing that to deploy a new version.


Shouldn't this be changed to UMR too?


> I haven't profiled it but I would expect this to have a performance impact

It definitely does, on `somatic-standard` WGS run this added ~3min (2 BAMs/~100GB each).  I think my interpretation was that if someone is providing a reference they _do_ want to override the mdtags. I guess we still use the reference for the reference base (which maybe we can't figure out from the reads).  I'd fine making it an option as long there is a way to override the mdtags, but I think for consistency if there is a reference provided, using it should be the default?  Maybe re-generating the mdtags isn't the best way, but it was the simplest with how most of the other code is generated.  Alternatively, we could give each read access to a reference? Would need more thought, so I'm fine with any of the options here.


OMG `Jul 17, 2010`, the first commit.


Is it not legitimate for `driver` to be `nil` as `ipamDriver` can be?


TeamCity TripleA :: UnitTest [Build 275](http://home.k-moeller.dk/viewLog.html?buildId=355&buildTypeId=TripleA_UnitTest&guest=1) is now running


Why not. Do it not mean fetch json to variable?


Hey,

sorry the screenshot was misleading. I got the correct results with the changed offset (i.e. a and b).

I'll try the changes later on (i.e. tomorrow), have a meeting coming up


Got it.  Thanks.

On Wed, Dec 2, 2015 at 7:53 PM, Matt Brender <notifications@github.com>
wrote:

> Thanks @vCloudernBeer <https://github.com/vCloudernBeer>! Would you mind
> adding another commit that removes checking off that I'm confirmed &
> speaking at the event? The line right now would be |SCALEx14 2016|Jan
> 21-24|LA,CA|||TBD| I think. Just push this up to the current branch and
> I'll know I'm able to merge after that!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vCloudernBeer/transparent-travel/commit/f12461e3c7962c67aa09f4a37f448209fb1e0801#commitcomment-14752699>
> .
>


By which I mean, why aren't these `handleMethod` registrations inside an `if driver != nil {` block


Thx Aokromes, but... everytime i try to be up to date i lose my personal changes, is there any way to keep my changes, while i keep my TC up to date, i tried with 'git bash' but even with this i ge full bunch of errors.


TeamCity TripleA :: UnitTest [Build 275](http://home.k-moeller.dk/viewLog.html?buildId=355&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 203 Build time: 00:00:36


Yea, I guess I was just wondering if there was a way to simplify this so the user can't provide inconsistent options and it just does the right thing.  Basically it should be able to decide local or not based on the file scheme and at the point each has their 'best', so really the only user option should be whether the user wants to use `HadoopBAM` for local files right?


Yes I"ve got here. I will fix this.


That only happens in jetty because the page is within your project space so maven hasn't been run to replace the variables, but if you run a generated war file from the maven build, it should be fine, you can try it out in tomcat and you will see what i mean, just ignore it.


The method is changed .


Unfortunately I think this violates the premise of this particular unit test (which you can see in the title[I suck at writing unit test titles]), which was to ensure the list_item_hunter could find the verbose partials.

With your changes it is more akin to the test above it named `process_list_item_partials finds partials and outputs repeated renders`, right?

Important to test, yes, but it would be nice if we could cover the verbose syntax too. If this is literally the last thing left and it looks like a big refactor or something, I'd be okay with putting a [known issue](https://github.com/pattern-lab/patternlab-node/wiki/Known-Issues-&-Deviations-from-Pattern-Lab-PHP) in for this (hopefully only temporary).


Should this not have gone into cryptonite?


Brackets (parentheses): `()`
Quotes (quotation marks): `'' ""`
:)


hoge


TeamCity TripleA :: UnitTest [Build 276](http://home.k-moeller.dk/viewLog.html?buildId=356&buildTypeId=TripleA_UnitTest&guest=1) is now running


@ellisonleao
O ícone do github não está aparecendo.

http://staging-brasil-pyladies.herokuapp.com/ladies/


Oh i see! :)


Shouldn't that be moved out of the surrounding if-block?


I think this might not be quite right if the module has "from" in the name which [seems fairly common](https://www.npmjs.com/search?q=from). I think you'll need to use a slightly different strategy here with one pattern for `const`/`let`/`var` stuff and an entirely different pattern for `import` stuff.


TeamCity TripleA :: UnitTest [Build 276](http://home.k-moeller.dk/viewLog.html?buildId=356&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 203 Build time: 00:00:36


Welcome Swift! :D


You should follow Swift coding convention. Don't use upper case for constants' name.


Hello,Swift!!!!!!!


I think that it should. Once I’m happy with it and it’s all documented and whatnot I’ll offer it up.

> On Dec 3, 2015, at 9:33 AM, Thomas M. DuBuisson <notifications@github.com> wrote:
>
> Should this not have gone into cryptonite?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/glguy/ssh-hans/commit/da11ba507745c43e78608e8d297351d329ed09c9#commitcomment-14768455>.
>


Wow!  Don't tell me this does what I think it does...


In override functions, if you don't implement any special thing, you can remove them.


Ah! This actually makes a lot of sense given this proposal: https://github.com/apple/swift-evolution/blob/master/proposals/0001-keywords-as-argument-labels.md

If that proposal is adopted, then this is super reasonable. :+1:


hello，swift


Please remove unnecessary comments.


:boom: Awesome they have the entire history here...


Another thing that looks off is that against the current `test/` file system, there is no atoms directory which the first half of the [key](https://github.com/pattern-lab/patternlab-node/blob/master/builder/object_factory.js#L28) derives from.

One thing I wish I would have done better is create these objects clearer. You can really see what two years of unorganized/at-times-not-intelligent-programming does by looking at the object_factory, or for that matter the whole code base !


Modifica integrata in prossimo commit


It's better if you set default value for these properties, such as `""` for String and `0` for Int. By this way, you needn't to unwrap them before using it later.


@Mihica  Why? this is not restful


@aninhalacerda Resolvido em  https://github.com/pyladies-brazil/br-pyladies-pelican/commit/4399725bd91cbf13ae1659a4b08c8cfb9d8838b6


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


@avandecreme FYI...


G'day, Swift!


What happens when a callback isn't passed?


You can declare `tweets` like this to make sure it cannot be nil and you don't need to unwrap it when using.
```Swift
var tweets = [Tweet]()
```


Hello, World!


Dayum. We all missed that!


TeamCity TripleA :: UnitTest [Build 277](http://home.k-moeller.dk/viewLog.html?buildId=357&buildTypeId=TripleA_UnitTest&guest=1) is now running


Please keep `{` and `}` at right positions.


TeamCity TripleA :: UnitTest [Build 277](http://home.k-moeller.dk/viewLog.html?buildId=357&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 203 Build time: 00:00:36


Merging, thanks.


Of course! Sorry, multiple caches was – and is still – confusing me.


Modifica integrata nel prossimo commit


Typo *keepSlientWhenBlankSelected*


You should use `extension` to implement `UITextViewDelegate`.
```Swift
extension ComposeViewController: UITextViewDelegate {
  // your code
}
```


Hey, that IS prettier. What did you use to make them? Thanks. I like the promo and the new 128px, but I think I want to make a new icon16, that's a little graphic instead of letters. I made this extension in 2-3 hours during a meeting, including the icons, so it was a *bit* rushed.


You can remove this function.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7898](https://teamcity.mesosphere.io/viewLog.html?buildId=85652&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout Build time: 01:00:09


Thanks!


初めまして！


TeamCity TripleA :: UnitTest [Build 278](http://home.k-moeller.dk/viewLog.html?buildId=358&buildTypeId=TripleA_UnitTest&guest=1) is now running


@rickalm Docker driver may have changed? They're using Graylog2/go-gelf which supports chunked messages and gzip as well. See [daemon/logger/gelf/gelf.go:64](https://github.com/docker/docker/blob/3856c5efa685013b5e82347d5812de4c7788d4ff/daemon/logger/gelf/gelf.go#L64)


But if I migrate to a specific datetime, using -d, it should also migrate to that specific entry.

``$version <= $dateString;``


TeamCity TripleA :: UnitTest [Build 278](http://home.k-moeller.dk/viewLog.html?buildId=358&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 203 Build time: 00:00:40


Reverted at https://github.com/openmrs/openmrs-module-htmlformentry/commit/06d5dd35672cc030eab1334bc8b162a7cb7c2f3f


ok.. :)

allora il punto e': che sia ben visibile :))
On Dec 3, 2015 4:07 PM, "Brando Preda" <notifications@github.com> wrote:

> non mi piace molto
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brando91/STAN/commit/9bd858c94914dc2f9b71506ca9b508622c316303#commitcomment-14764274>
> .
>


abadidea


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


Your mistake was making the license even longer. Try this:

Copyright (c) [yyyy], [name of copyright owner]
All rights reserved.

Redistribution and use in source and binary forms, with or without
modification, are permitted provided that the following conditions are met:

* Redistributions of source code must retain the above copyright notice, this
  list of conditions and the following disclaimer.

* Redistributions in binary form must reproduce the above copyright notice,
  this list of conditions and the following disclaimer in the documentation
  and/or other materials provided with the distribution.

THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.


You can use `segue` to transfer from `TweetsViewController` to `TweetDetailViewController`.


do { print("swift test") }


Almost tempted to suggest what about merging -t / -d such that you can -t CCYY | CCYYMM | CCYYMMDD | CCYYMMDDHH | CCYYDDMMHHII | CCYYMMDDHHIISS.


1.6 should already be the default for SciJava components? Do we really need this?


Great to see the history here. For over 5 years of development, nice :+1:


Shouldn't this checking be made outside the if-block?
When having entity types in Groovy, a NullPointerException is thrown at line 190 because `definition.getPrimaryMember()` returns `null` for the `metaClass` property while `definition.couldSerialize()` is `false` but wasn't checked due to `persistentProperty` being `null`.


- "Remote-check-deposit apps allow a person to take a picture of a check with their mobile phone's camera and then send the image to their financial institution for deposit into their account."?
- "Many of these apps will retain the check image (or part of it) in the mobile device's NAND memory even after it is deleted."?


This commit has a lot of (good!) style changes mixed in. In the future, it is best to isolate them as their own commit, before the functional changes.


These "Allow authors to set background and text color of draggable items" commits look good!

One last thing - can you change the `placeholder` example text from "red / #ff0000" to "blue" and "white" so it matches the default? Sorry I didn't think of that earlier, but it would make things clearer. Then this will be +1 and you can merge.


Ya true enough.  I had assumed people would put in the general date and use -t for the specific stamp, but I can add that in too.

> On Dec 3, 2015, at 10:44 AM, Richard Quadling <notifications@github.com> wrote:
>
> But if I migrate to a specific datetime, using -d, it should also migrate to that specific entry.
>
> $version <= $dateString;
>


JSON_TOKENS короче и логичнее :) но это "на вкус и цвет" конечно


This variable rename is unrelated to the stated purpose of this commit. Best if it were separate.


Wow, wow :)


Let's not encourage private mails. Best to just put your name without an email, so that people communicate using public channels.


Heat Transfer Project


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1441 of 1441 tests passed)

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


Hello swift!


Hello Apple


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7900](https://teamcity.mesosphere.io/viewLog.html?buildId=85654&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:09


saw approval from bkerley
at https://github.com/basho/riak-ruby-client/commit/a9aa9580beb047e899cac0975970034783945a0f


merging basho/riak-ruby-client/fixes/lrb/bucket-type-spec = a9aa9580 into borshop-integration-250-fixes/lrb/bucket-type-spec


basho/riak-ruby-client/fixes/lrb/bucket-type-spec = a9aa9580 merged ok, testing candidate = 25652ec9


Lets make this game impossible


TeamCity TripleA :: UnitTest [Build 279](http://home.k-moeller.dk/viewLog.html?buildId=359&buildTypeId=TripleA_UnitTest&guest=1) is now running


The SciJava coding style wants private methods in their own section after public ones. See:

http://imagej.net/Coding_style#Ordering_of_code_blocks


TeamCity TripleA :: UnitTest [Build 279](http://home.k-moeller.dk/viewLog.html?buildId=359&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 203 Build time: 00:00:36


size in bytes  / KB ?


hello


Hello Swift!


Add my favorite color
+ - [ ] green


Nice! So the caching happens because SerTypeInfo for a type is cached using Serialize.memo, right?


Looking elsewhere in this file I can see how this is confusing, because sometimes I stubbed in patterns within the shipped file system that dont exist, and later I actually used the test directory. Given the choice, I like the idea of having unit tests physically work with files in the test directory but that's sometimes a bit of a chore. </braindumb>


These changes are unrelated to the NPE fix, and should be in their own commit.


Both of these methods should be const.


EconomizerIntegration (lgu1234) - i386-Windows-7-VisualStudio-12: Tests Failed

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


After carefully analyzing the changes here, it seems that the mutex refactoring out of PadInfo is equivalent to what was in place before, so the change is ok for me. However, we've had several memory leaks and crashes related with these operations in the past, so if they happen again, we should have a look at this patch first.


This is great!..Let's roll people. This is the new generation compiler!! Protocol-oriented Programming!!!! OOP is dead!!!


Wow!!


Thanks for catching this!  I'll put this fix in, probably push a release today or tomorrow.


на мой взгляд StringHelper нужно рефакторить. Ему совершенно не обязательно в конструктор принимать аргументы, это можно все сделать на static методах, если хочется такой архитектуры. что-то вроде:
StringHelper.removeInsignificantSymbols(test);

И название у него странное, причем тут String? Скорее это какие-то JsonUtils :)


Well.... Almost right this time


Hi @clintonskitson,

This is not necessary as any IO error will emit the name of the file that produces the error. The original line would suffice.


hmmm this was inadvertent, restored in a5b436c065829f8957fb7f190b13092e2aca9452


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

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

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

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

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

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

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

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

```


Hi @clintonskitson,

I understand the idea of adding additional context, but the getIQN error already produces meaningful messages. The stacktrace will note that the error was returned through this function, and wrapping an error to just provide an additional message is not really necessary and often just adds complexity to debugging. Especially since we control the internal error in this case.


The arguments seem to be in the wrong order. According to
https://docs.angularjs.org/api/ng/service/$http#post
it should be:
```
post(url, data, [config]);
```
!?


sd


oops accidentally pressed ctrl-b when latex was set as build system


Fixed error that caused data check to skip sub-modules if ID module was complete. The issue was with the If-Else statement structure.


What is the reason to ignoring the nan values, instead of casting them to 0? This will significantly affect the score of clustering with clusters of size 1, and make the score biased towards larger number of clusters.


```==``` compares references. Two different objects (lists, in this case) will always be two different references. ```result == expected``` can, in this case, never be true.


readability


EconomizerIntegration (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1425 of 1425 tests passed)

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


Sweet!  Thanks.


Hi @clintonskitson,

Regarding the feedback above, I'm personally not a fan of wrapping errors with additional messages. However, if you do, please use fields or `WithError` instead of formatting it as a string.


this is really buggy


No longer auto-fills coordinates and location descriptions.


These two lines can be combined to

    FEATURES_WITH_BADGES_ENABLED = dict(settings.FEATURES, ENABLE_OPENBADGES=True)

(Up to you what you prefer.)


Two notes on assertions, other than the one above:

- whenever you can, use the most specific assertion. In this case, ```assertEquals()```. See for yourself how much better the exception will be in the negative case.
- you should look into ```AssertJ```. We make prety heavy use of it in our own test suite and the assertions are just so much better that way.


Overall, adding tests first is a nice touch!


This seems to be a bug. You're looking up instances from the bean manager and never destroying them when clearing the components from the view.


test comment


Ah, ok, well the length was pathOfDirs, so a direction was always undefined. It's fixed now with my commit since we want pathOfDirs' length


Hi @clintonskitson,

Unless there is any other case where preemption comes into play, aside from mounting, why not simplify the property as `rexray.volume.preempt` or if you need `mount` then `rexray.volume.mount.preempt`. Just a suggestion; I don't feel particularly strongly about it.


Are you dumb?


Use _


What is the point of having this here. It fucking grows the clone if it succeeds without doing the find_dead_player check