TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2009](https://teamcity.mesosphere.io/viewLog.html?buildId=63027&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:21


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2011](https://teamcity.mesosphere.io/viewLog.html?buildId=63029&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:20


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 2012](https://teamcity.mesosphere.io/viewLog.html?buildId=63030&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:03:21


When using Begin/End methods, the Connect was an async operation. This becomes a sync operation with the change. Do we want to call method like ConnectAsync?


pickle is a binary export of objects


Had not looked at the tests first. Running them now. A new test will be written to cover the noParse option. The most common case is when two modules are defined in one file, and so the second module, as a dependency, should not be required() by webpack when the first one will always bring in the second.


This is breaking some tests in almost every browser: http://swarm.jquery.org/job/1530


indeed.


This uses the same `transactionId` for all tests. Pass it in for each call or generate a new one each time.


This we may want to refractor out into its own class (ExportService or something) that accepts a List of headings and List of arrays for the data. That way we can reuse the code for cases as well


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1420 of 1420 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.37%25-green.svg)


属性之间需要空格


Well we just did a lot of work to refactor those manager classes into just those functions we have here, right? So no wonder it looks similar.


process ve thread'li çözüm


Shouldn't you cast to int64_t?


(any reason you're using underscore_case here?)


nit: maybe the closing paren on the `}` line? (don't have to change this if it's not easy or whatever)


I will fix that (CRLF) problem.
When I do, I will close this pull request and create a new one.

NB. Actually my memory fails me.
This change has noting to do with changing 'DAO's.
However on talk it was decided to remove support for 1.8.x.
(https://talk.openmrs.org/t/upgrading-openmrs-2-0-modules-to-support-hibernate-4/3209/8)


should this `}` be on its own line?


PUT используется, когда url-у соответствует весь передаваемый объект. Т.е. в вашем случае должна предполагаться передача состояния всей коллекции users, а не одного пользователя (если здесь предполагается что-то REST подобное).


Or I'll do this, because I'm going to merge it with some code from @takluyver's PR in order to add tests.


"a little more time...."


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


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


i like this one


There are ways to get the app base without DNX APIs that we can use


The purpose is to get familiar with the manipulation of a repository.


Looks like this broke CoreFX build.  We have many nested project.jsons in tests and reference assembly hierarchy.  Thanks  @iamjasonp for identifying this.  The result is that they end up silently not being restored.

How is this the correct behavior?

/cc @weshaggard @joshfree


blizz doesn't send the teleport packet for transports anymore btw


What is the reason of using `trim()` instead of `rtrim()` here?


Thank you.


use a config file to set this values with tokenHeader and tokenPrefix


《21551106陈受恩》这个文件才是最后要交的作业


We need to add the globbing feature yesterday. Diving was always wrong. See he many other issues it causes (does more harm than good)


Creating a boolean 2d array. It does the job here, and for minesweeper there may not be a better alternative other than a 2d array of objects. Just you rarely see them and it tends to indicate some improvements could be done


So, I just geeked out on some of these results, and it's a gold mine. See, in particular, the email thread "CSS and presentational markup." The others offer rich details, too, but the aforementioned, I think, will be a must. :)


it does, but with the comment I was thrown off a bit. Also it does the exception handling etc. Personal preference again is to toss it into a function that way it's more readable


only note with iMinsweeper is that some languages you an iClassName to demonstrate an interface. I can't recall the paradigm in Java so just a fyi if you were working with multiple people they may get confused with the name


Sorry for delay. Can you squash this to single commit? If not, fine. I can do it myself.

Also one final thing. Return value of last call to calloc is not checked for NULL. If we get out of memory situation, this code with crash.


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


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


I think `field` is always null here so the cogs will never show. We _could_ add support for marking generated fields, but I think just remove this span for now.


Showing a message dialog instead of writing on the ```System.err``` wouldn't be nicer?


:metal:


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


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


~`static const char*`~


我执行了meteor add audit-argument-checks后感觉不是很正确,没有出现check这个包,只能手动添加这行


I've added a case for `wire.CmdGetBlocks`.


Sorry, I forgot that in fact there's the `AngularJS` branch in the `Bridge` repository you should switch to in order to have the project referenced! I believe the readme file linked above states it correctly.


Not good, please correct as design


Sounds good, updated.


I'm looking forward to the development of this project sir. By the way, is there any chance that epanet emitter will also have an API in python?


By the way, the `Presentation` brach was used just to step thru a small presentation I did. You should be following the `master` (for Visual Studio) and/or `osx` for MonoDevelop on OS X or Linux build/running.

The `Demos` repo should be on its `master` branch. (this `AngularJSDemo` repo is pointed as a submodule by the `Demos` one -- repo is [here](https://github.com/bridgedotnet/Demos)).

The instructions on the readme file should still be actual and correct.


lol caching is so annoying


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


This is necessary to adjust the frames of subviews that might be used by the subclass. Without this code, the frame of subviews within layoutSubviews are the frames in the nib.


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


I also share the sentiment that the runtime wouldn't necessarily handle this, but my goal here is to paint an end-to-end picture from the perspective of various users. When we make decisions in the runtime, or the binary format (for example), we should consider how the full picture is impacted and what we ask of the various participants.


I suppose not. It just seems like something could go awry if we're taking the summary ID from the url and using it as the post ID in `@post = Post.find(params[:id])`. That line (stop me if i'm misunderstanding) will take the summary ID and and look for a Post that has the same-numbered post ID, almost like an alias. It's not possible that the IDs would get off (if, say we started destroying summaries but not posts)? Bear in mind that I'm not proposing an alternative :grin:


Please help. @pocke .  :octocat:


My previous comment applies here, too. If we are comfortable asking developer tool vendors to do this, great, but we should make that decision explicitly, not default into it (and then discover we force larger than we expected costs on the vendors - for instance. Not saying what you're proposing here would do that, to be clear).


Так верстают только мудаки ппц...


It is incorrect to instantiate a new context in here. Instead, change the function signature to New(ctx context.Context, params driverParameters), and use ctx.


The point of the variation is that such constrained environments exist (phones, gaming consoles) and they may pose additional challenges.


Maybe change the commit message to say something like "migrate to Java 8 lambdas"


OK, I'll tweak this :octocat:


hahaha.. thanks


if you collect the Context in New(), you can omit it from the param list here. Or if this is satisfying an interface, fine to leave as-is. Same for the rest of the methods.


为了跟正文区分下，开头的信息请用 `>` 标明，例如：

```
> 原文：<br/>
> 翻译：[@xxx](http://github.com/xxx)<br/>
> 校订：....
```


102867486-FixRectangleShapeTest-5173 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1947 of 1956 tests passed)

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


中英混排时请在英文（包括数字）周边确保有空格，阅读视野会更开阔。


I changed my mind, the correct thing to do is to continue to get a context for each of these methods (as well as New()), since this could potentially be used in many different requests, if this package backs a server. Each of those requests would have their own context, rather than the background used to call New(), so they need a way to pass their context along.


:+1: handy function!


在 zh-cn/security/ 下添加之前，请同时：

1、先在 en/security 下添加英文原文的 Markdown 格式。提交为一笔记录
2、复制一份到 zh-cn/security 下，提交为一笔记录
3、翻译成中文，提交为一笔记录

这样主要是为了方便后续跟英文的同步跟踪，也方面校订者方便直接左右对照校订。


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


use the context from PutContent()'s parameter, rather than d.context() and cloud.WithContext() by extension.


You fixed it before I found the bug, thanks.


在转为 Markdown 以后，部分格式可以适当调整，比如这里的 * 号，直接用 * 保持缩进，或者用 - 号也可以。


4665_hpwh_energybal (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1891 of 1897 tests passed)

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


as with PutContent, use the Context passed as a parameter.


This line cause install failed on FreeBSD

maybe it should be `'gnureadline' if sys.platform == 'darwin' else ''`


I feel like this is a typo, but I could be wrong. :)


and here, and you get the idea.


ok let me check this closer. Sorry for the delay. I will try to see what's wrong locally.

``` bash
git checkout master
# Already on 'master'
# Your branch is up-to-date with 'origin/master'.
git fetch upstream;git merge upstream/master
# Already up-to-date.
```

Switching to your branch

``` bash
git checkout -b 749/1 upstream/issues/749/1
# Branch 749/1 set up to track remote branch issues/749/1 from upstream.
# Switched to a new branch '749/1'
```

testing and having issues with grunt but this is a separate thing.

Testing to see if categories is available at different places.
So I was totally wrong about `__init__.py` sorry about that @magsout
A "better" place for this is the config file. The thing which helped me with that is http://flask.pocoo.org/docs/0.10/templating/#standard-context

>  Changed in version 0.10: This is now always available, even in imported templates.

The this here is the flask.config object.

So for example as a test. I erase the code in `init` and added to `config.py`

```python
# Status categories used in the project
# 'new', 'needsdiagnosis', 'needscontact', 'contactready' , 'sitewait', 'close'
# Creating the model
Category = namedtuple('Category', ['name', 'dataAttribute', 'label'])
CATEGORIES = []
cat_labels = [('new', 'new', 'New Issues'),
              ('needsDiagnosis', 'needsdiagnosis', 'Needs Diagnosis'),
              ('needsContact', 'needscontact', 'Needs Contact'),
              ('sitewait', 'sitewait', 'Site Contacted'),
              ('close', 'closed', 'Closed')]
# populating the categories for issues status
for cat_label in cat_labels:
    CATEGORIES.append(Category(name=cat_label[0],
                               dataAttribute=cat_label[1],
                               label=cat_label[2]))
```

Then to test I tried in the views.py

```
@app.route('/issues/<int:number>')
def show_issue(number):
    '''Route to display a single issue.'''
    print app.config['CATEGORIES']
    if g.user:
        get_user_info()
    return render_template('issue.html', number=number)
```

Just to see if the variables was accessible there. Then requested a ramdon issue and checked the logs

```
→ python run.py
 * Running on http://localhost:5000/ (Press CTRL+C to quit)
 * Restarting with stat
 * Detected change in '/Users/karl/code/webcompat.com/webcompat/views.py', reloading
 * Restarting with stat
[Category(name='new', dataAttribute='new', label='New Issues'), Category(name='needsDiagnosis', dataAttribute='needsdiagnosis', label='Needs Diagnosis'), Category(name='needsContact', dataAttribute='needscontact', label='Needs Contact'), Category(name='sitewait', dataAttribute='sitewait', label='Site Contacted'), Category(name='close', dataAttribute='closed', label='Closed')]
127.0.0.1 - - [22/Oct/2015 11:37:34] "GET /issues/283 HTTP/1.1" 200 -
```

At least so far so good.

but I need to find out about the grunt thing so my UI is working.


:smile:


>+[定期][お知らせ][iOS 版 feather] 書き込みが規制の件について Twitter 社からの回答が無いため一時的にアプリの公開を停止致しました。アンインストールを行うと再度インストールすることができませんのでご注意下さい。https://twitter.com/feather_ios/status/653191019702255616

Twitter社からの回答がないのでだと お知らせ済みのメールでやり取りしてますと違っているので少し変えたほうがいいと思います


hi dillbyrne, can you set random scren size for mobile, when is in random mobile profile?
because we have random mobile with scren size desktop!! no random mobile with scren size mobile!
keep going bro.


I don't think there's any reason you'd need to do this. it passes along the client but what does that matter?

Oh, I see, that client is authenticated. You know, *this* is why I hate that github doesn't let me batch my comments. At this point I'd go and delete all the pending comments about use of d.context() above, but now I can't.


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


类似这种也可以直接转为 Markdown 支持的

1. AAAA
2. BBBB
3. CCCC


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


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


SQL comments are -- at the beginning of the line, with a space between '--' and the line's contents.


换成：

* UNIX UID，GID，模式;
* Windows 用户 ID;


Pieter Wuille <notifications@github.com> writes:
> C doesn't have a bool type. I can turn the compressed back into int, but
> it's not an externally facing call, so it probably doesn't matter much.

It does officially since C99, and it's been pretty standard for even
longer.

Technically it's called _Bool; #include <stdbool.h> to get "bool".

Cheers,
Rusty.


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/0c3e3773877d762c528dc90c6d18c01677d6f258


merging basho/riak_cs/feature/releasenote-2.1.0-ja = 0c3e3773 into borshop-integration-1270-feature/releasenote-2.1.0-ja


basho/riak_cs/feature/releasenote-2.1.0-ja = 0c3e3773 merged ok, testing candidate = f6f28e68


Yes


aliasing a package to have the same name as the current package is a bit weird. what's wrong with storage?


There was a conflict between the two branches. The index.html had the standard arrows around the conflicted areas, I resolved it and went with unordered list from the master and pushed it back up. None of that work is being reflected here though?


这里可以直接用加强：

`*注*` 效果：

*注*


Extra points: file a bug to finally rename `Browser` to `Tab`?


… and rename `toTab` to `(to|as)RemoteTab`.


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

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


Remove the `tabManager` argument — it's not needed now.


@L3w1s-L1u 目前只对基本格式做了 Review，具体翻译动作需要等到先提交一份英文作为记录后才好 Review。


It was more like I forgot to put filter = player.GetAll() on the other traces.


@jmccartie is there any reason why we need `rescue nil` here? or I am missing something here? Because what I thought, `Rails.application.config.active_record.primary_key` either return assigned value or nil. `rescue nil` will catch everything even `NoMethodError` or `NameError` which usually happen because of typo.


SingleSubtabInMemory (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2340 of 2365 tests passed)

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


Nice job! @jmccartie


If you want the normal corresponding to a face index, there's an existing function for that.


複数追加する方法調べて書いてみたんだけど、古そうだね( ；∀；)
<< [:name,...]で書き直しました。


Just ditch all this rotate crap. The wiring code uses a slightly different construction that does a better job.


:+1:  That would be better for keeping the nest under you when constructing it.


ok solved my grunt issue.  I can test more properly now.


@Zidane Not all blocks have item forms (e.g. air). Could you address this?


My gut is to disagree with this as `However`.  Could you help me understand how it's contrary?


@randalburns Any reason you commented out the metadata in jsonprojinfo?


In the sense that it is not exclusively used inside a controller.


I made a mix of it...
Don't look on my commits to the manifest because is different my local one.
When i merged the commits for each repo i included commits also from the latest stable branch (don't remember now) So by short it's a mixt of the latest stable branch and cm-12.1


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


this isn't allowed? :o


@ronchifabricio LOL!


is there a connection time limit or such? or do you only disconnect from the service to avoid timeout/resources?


You also need to track the `rollback()`, then we can actually get rid of the `TODO` in RaptorPageSink


nit: I think the open `{` should be moved to its own line, and these two lines indented one further to be consistent with `null` and `true` below


102867486-FixRectangleShapeTest-5173 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1947 of 1956 tests passed)

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


Success* ;)


What is happening here? Is there a diff between the different compilers somehow?


delete readme file


@hillelcoren Just as a suggestion, instead of having 3 commits with the same name you can squash those extra commits by rebasing. Here's a very easy tutorial: https://mattstauffer.co/blog/squashing-git-commits-with-interactive-rebase

It really helps to have a "clean" commit history in my opinion.


How do I do that? Sorry to ask.


How do you make sure that this is used only on Windows? Also, I thouhgt that nowadays unix slashes work well under Windows, even when mixed with backslaches.


> Change them back if you would prefer.

Sigh, I guess it's futile to bother, as It's just a matter of time before someone else runs one of these tools on the code again.  I don't want to have to keep editing them back again and again.

>  I don't commit those changes that take away useful formatting.

We don't agree as to what's useful then.


Errrrrr what are these? I can't read it


Success, again


Connecting to a different IRC server without password after this changed worked OK for me, so I think it's happy to have nulls passed


Cassie and Russ tried to comment on this file simultaneously.


One thing to note is that the script in this example is formatted to be easily readable. The actual would look like this:

```
"script": "def transform(input, emitter, context):\n    tax = input['subtotal'] * 0.0975\n    if (tax > 1000.0):\n        context.getMetrics().count('tax.above.1000', 1)\n    if (tax < 0.0):\n        context.getLogger().info('Received record with negative subtotal')\n    emitter.emit({\n        'subtotal': input['subtotal'],\n        'tax': tax,\n        'total': input['subtotal'] + tax\n    })"
```


Why?


Where's `TAG_HELP` defined?


i'm assuming perseus still builds after this? in theory this could be stateful; maybe @spicyj knows whether this require is important or not. (I'm assuming not if perseus is still compiling)


0.23.1.. That explains :smile:


I cannot claim that I really understand these changes. What I understand is that the CMake magic is used instead of explicit grouping, which makes perfect sense. Somehow I still feel that the config file of the build system should not need to be much more than 10 lines long, so this is a good step in the right direction.


oh SNAP. Is this manifesto coming back elsewhere?


Здесь, скорее, POST, т.к. PUT требует передавать тот объект, на который указывает url (т.е. вся коллекция compositions). Кроме того, многократное повторение PUT не должно отличаться от однократного по side-effect-ам.


I don't know, either it's Windows 10 or JDK 1.8.0_60 - my guess is on the JDK. What version do you use?


(nit: indentation; not that I mind if it stays this way either)


This is just a sanity check to catch a bug in the engine or connector. It will be caught by unit tests, so no need to fail early.


why not just `if tag:` ?


Fixed in 8e22de9


Sigh.  Our repositories are littered with hardwired file paths, and I've yet to think of a good way around it.  In any case, I think I will try to move this to the GCAM group directory.  Then we can at least be sure that everything will work for anyone using PIC.


clone is a bit awkward not guaranteed to do what you think it's going to do. I would just System.arrayCopy instead if you need a new copy


line 89


MONO_PATH ends in assemblies_path through check_path_env.

The way assemblies_path is set is through the API call mono_set_assemblies_path.

mono_set_assemblies_path usage is pretty common among embedders.


because it's a string


++ to this simplification :)


Thanks for sharing the article!

I'm struggling to keep the code clean, let alone the commits...


ok man ;)
then you can build it ;)


Аналогично, здесь тоже лучше POST, если передается не сама коллекция compositions, а один её элемент.


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **11/15** |
| _DVB | 5/5 |
| _DVD | **0/3** |
| _DVR-MS | 2/2 |
| _General | **6/24** |
| _Hauppage | **2/3** |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **5/21** |
| _WTV | 13/13 |
| _XDS | **15/34** |
| _Options | **75/87** |
| _DTVCC | 14/14 |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/127) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


Interesting, def seems like an improvemnt


@AgtLucas HUEHUEHE BRBR


We should get in the habit of writing `map` functions as list comprehensions.  (I am the perpetrator of this one---old Lisp habits die hard.)


Please don't abbreviate "deleg"


Create a dummy ShardRecorder and test the values after shard creation.
Also test for rollback


@stramel I had to add it here because I was getting an error saying it was trying to execute `.toLowerCase()` in undefined value.


Do I understand correctly that our build time is effectively doubled now ? :smile:


:+1: approve; this looks good to me!


Thank you, working great!


This should be in a global .gitignore file - never in project file!


I create the PF arrays currently by using Parser::in2post. I then store them for the run I am doing.

The changes that I made to the in2post function already allow functions to be passed through (eg. sin, cos).  I would think it would probably require a change in the if statement around line 291 (by adding $self::AFNC), to include the advanced functions there and then some more code around line 362 to actually process the function.


lgtm


Suggest to add `npm-debug.log` file too.


Wow, that must be old!
Go, remove.


Intellij config file?


Vai ficar cabuloso


Can we now enable this path on INDIVIDUAL_ALLOCATIONS?

It was originally disabled out of convenience.


Yeah, it was a relief to figure out that Johnny-Five _would_ _not_ be shipping with these weird voodoo incantations :)


The way you did the toast message makes it so it only shows up if you click refresh on a character header. The main refresh button has no indication as to whether it's worked or not. It's even further confusing because the 'button' stays visibly in a clicked state leaving one to wonder if they've even pressed it. If you move the toast outside of the element if statement, you start seeing the main refresh button actually working. But, this adds 4 separate toast messages. It looks like you're not using the standard toastr library, either, so simply adding 'preventDuplicates: true' doesn't work.


oh yeah right totally missed that...


Ok will check

Sent from my iPhone

> On Oct 22, 2015, at 4:12 PM, Erik <notifications@github.com> wrote:
>
> Even on the latest commit? Everything passes for me right now with the latest commit. Right after I pushed this one I added checks back into the pose estimation test which might have been failing until 751749a
>
> —
> Reply to this email directly or view it on GitHub.
>


Could you extend your unit test for this method, I would like to check for example User.equals(null)


I really should learn how to spell.... ;)


seems CLI arguments are directly imported into the default section. This won't apply to this commit, but maybe we should define another special section for CLI?


hmmmm, strange since it didn't exist before. No issue with that though.

However, you could use lodash instead of underscore, and then use the fancy `_.get(node, 'callee.object.name')` to check if it exists lol


Yeah I'll have to put all this stuff back up, because I deleted a TON of
shit.

On Thu, Oct 22, 2015 at 2:35 PM, Rickard Sunden <notifications@github.com>
wrote:

> create_table :gametracks
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danodemo/hangr/commit/735ec9f25090fa7f416a1f025d95df4c5e87df1f#commitcomment-13934913>
> .
>

--
*Daniel O. Nicodemo*
Consultant
*Danico Consultant Services*
Fayetteville, AR 72701


This is a result of using the GCAM-China region mappings.  The China region is superseded by the China subregions.  I suppose we should have tested this with the 32-region version of the region mapping, but since all we really want to test is that the queries run and produce output, we can leave it the way it is for now.


> Do I understand correctly that our build time is effectively doubled now ? :smile:

On your mac, you'll barely notice - and even on my old laptop it's still faster than the integration-tests... :grin:


I created a macro instead, and used it where possible, to keep the overhead minimal.


Why not "return nullptr;" ? c++11 is now 4 years old and we should use it.


jdk1.8.0_45, Mac


while you're fixing typos: s/indicated/indicate/


not really, that's just how it looked in the csgo module and i didn't even notice


can you expand on this? this comment isn't really useful in that it doesn't tell you what `value` is used *for*


В каком формате задается сообщение? В смысле, тут будет json, как и выше, или само содержимое msg_content?


nit: variant or value


ok, if it works it's good I guess


done


no point in having the 'in-game' status all the time
also the same thing will have to be done with dota 2, and steam might go kookoo with everything at once


I don't know any details about inner_indices(), values(), and storage(), but it looks reasonable.


why not using `get_tag_path`?


Remember, though: You should **NEVER** rebase anything when you already pushed it to github (rule of thumb: if anyone else has the changes, don't rebase). If you keep that in mind, things are quite easy.


could be `{rank_id, wins, rank_name: rankNames[rank_id]}` if you'd want


А что означает число 1 перед msg?


Darn, Intellij showed the file as changed after doing 'gradlew wrapper' so I didn't check the contents. I'll fix after work today.


it's just a placeholder right now


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


The `proc_irr_share` function is obsolete in versions of GCAM that explicitly output irrigated and rainfed crop totals.  In theory we should keep it around for a while for backward compatibility, but at this point I'm not even sure we _are_ backward compatible, so maybe we should just get rid of it.


i guess


can use `steamId of plugin.friends.personaStates`


I am all in for nullptr as well as the complete c++11, but I will not decide that, especially as some compilers by default still compile c++03 :/

Lets decide this as a group.

________________________________
From: Tobias Glasmachers [notifications@github.com]
Sent: Thursday, October 22, 2015 10:14 PM
To: Shark-ML/Shark
Cc: Oswin Krause
Subject: Re: [Shark] fixed some bugs (22dbdb0)

Why not "return nullptr;" ? c++11 is now 4 years old and we should use it.

—
Reply to this email directly or view it on GitHub<https://github.com/Shark-ML/Shark/commit/22dbdb03860ce897dc2da052c4c87f34814370c6#commitcomment-13937032>.


forgot


Hey Lin,

Thanks for the Code. appreciate your enthusiasm. I have few questions for you.

a) Calibration: How you do calibrate the sensor? Do you put the sensor in a clean chamber during calibration?

b) I think the mass conversion units should be in ug/m3 not in mg/m3?

c) Have you wondered about reducing the flow rate of the fan to improve the sensitivity of the sensor? I think the fan is rated to 1.2 cfm which is very high typically for a scattering device.?What are your thoughts? Please share your calibration and comparison with some standard device.


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


А как сервер узнает, какие сообщение нужно отдавать?
Наверное, где-то должен передаваться пользователь? Или комната, в которой он находится?


I'd probably recommend `continue unless enabled` here (removes one indentation level), but I'm not sure it works correctly with iced...


@ybur-yug you can call `flat_map` instead of `map` so you don't have to call `flatten` on it ;)


А каким образом могут получиться разные room_id?


Про два разных коммита понял, но было поздно (
Замечания учту.


Is this where the overflow had happened? Well, never count backwards with an unsigned type. Peano lässt grüßen :)


and yes it's dumb and i didn't notice


Il faut que "obj" soit déclaré en dehors de la fonction de callback afin d'y donner accès dans les tests qui suivent.


Hi @molinch, first of all, thanks for all the work on the FFImageLoading library, it's really nice!!

Of course I'm happy to do a PR, however it was a "quick and dirty" fix to allow the CachedImage to be used for bundled assets again and have the correct 1x, 2x or 3x image being loaded. So a bit of code cleanup or refactoring might be needed. :) If I find the time, I'll have a look, but it's very busy for the moment.

An other small change makes the fade-in only active when the image is i.e. downloaded from the internet and not loaded from cache. This allows for a faster ui "feeling" when i.e. used in a scrollview. But it may be better to make this behavior configurable somewhere as it depends on the use case.


Looks all good, but I prefer "return nullptr" over "return 0" since it expresses the intent.


> to use PHP 5.4 or greater

DevTools still use 5.3 :)
It seems need upgrade dependencies ;)


@Cranapple Just in case you're wondering what happened the water depth, I just hard-coded it to 4 for the time being. It actually makes the scene look much better -- and when I finally throw a parallaxing 3D background in it will look even better.


Forgot to add to the commit message... I also added a new Skybox


Dammit!


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


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


Yes, we should.  Ultimately, when all the downscaling code is rewritten as python modules, we won't write output files at all; we'll just return the tables and pass them to the downscaling code directly.


:+1:
Сам использу везде классы без скобок, если в конструктор ничего передавать не надо


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


Pretty sure it's not important. :+1:


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


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


>Platform.Links.DataBase.Core -> Platform.Data.Kernel
Platform.Links.DataBase.CoreTests -> Platform.Data.Kernel.Tests
>
>Merge Platform.Links.DataBase.CoreNet & Platform.Links.DataBase.CoreUnsafe into Platform.Data.Core
>
>Platform.Links.DataBase.IdServer -> Platform.Data.IdServer
Platform.Links.DataBase.MasterServer -> Platform.Data.MasterServer
Platform.Links.DataBase.SlaveServer -> Platform.Data.SlaveServer
Platform.Links.DataBase.Terminal -> Platform.Data.ConsoleTerminal
Platform.Links.DataBase.WebTerminal -> Platform.Data.WebTerminal
>
>Platform.Links.FileSystem.Core -> Platform.System.IO.Core
Platform.Links.System.Core -> Platform.System.Core
Platform.Links.System.Helpers -> Platform.Helpers
Platform.Links.System.Memory -> Platform.Memory
Platform.System.Windows -> Platform.WindowsAPI

Started at https://github.com/Konard/LinksPlatform/commit/8cf16032c667dc9e0621fa04ea9701cc0c505645 , and finished here the refactoring plan done.

Video: https://www.youtube.com/watch?v=ZreQqgloCmk


apparently separators can have null text params.

text != null && text.equals ?


Oddly indented.


done.


Why is the $ here, when i try to Vulcanize it it sent a parser error.


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


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


The Same here, $


done


Since we only run one at a time, wouldn't we want to only pickup the first one?


>  It is highly **recommended** to use PHP 5.4 or greater

Recommended but not essential. ;)


This is just a dummy value and is not used -- it is passed to a no-op `ShardRecorder`.


i'm sure it'll become a long guy about how you kill noobs who don't enable plugin, in time


What the heck is this!?!


Good catch


Seriously, What?


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


I have not tried out how the slashes work. But this is used inside of Visual studio so this might be a different parser and the stackoverflow page i found explicitely writes this. So I assume it is needed.

This code should also work on other generators that support source_groups. If a generator does not support it, this is all silently ignored as the generator can not transform it into code. But the code above should also work for XCode, but I have not tried it


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


@Cranapple Definitely going to need your assistance on this one. So I didn't know that you can add delay to an Object before its destroyed but for some reason, it's not deleting our particle objects after they've finished. The 3f, is for a three second delay, but it's never getting called. Quite strange.

:crystal_ball: And yes I did try explicitly setting a reference to Instantiate return variable and throwing it in Destroy. Yes.


Ok.. So these were duplicates, right?


What about:
```python
self.unescaped_coafile = os.path.abspath("./.coafile")
self.coafile = re.escape(self.unescaped_coafile)
```


This bug should be fixed with pull request #855


do this kind of thing in .travis.yml, not deps/build.jl, so you don't run a risk of accidentally tagging code that does checkouts on user machines


It would be better to use "%u" and a cast to unsigned int.
The maybe even better one is:

#ifdef _WIN32
#define SIZE_T_SPECIFIER "%Iu"
#else
#define SIZE_T_SPECIFIER "%zu"
#endif
fprintf(fp, SIZE_T_SPECIFIER " " SIZE_T_SPECIFIER "\n255\n", sx, sy);

Don't take the latter too serious :)


Do we need a way to pass over any that are actually in progress as well?


I did not have thought to like her, good :)


And what about [`JPQLTemplates:54-58`](https://github.com/querydsl/querydsl/blob/1cbef8b87dd93fa57106d5e744222c67839340a4/querydsl-jpa/src/main/java/com/querydsl/jpa/JPQLTemplates.java#L54-L58)?


Ahoj,
tohle není dobrý:

class PbsConfig(object):
...
    def __init__(self, preset=None):
    ...
        self.name = preset[0]
        self.walltime = preset[2]
        self.nodes = preset[3]
        self.ppn = preset[4]
        self.mem = preset[5]
        self.scratch = preset[5]

v tom aby se prase vyznalo, například tam nemáš index 1 a máš tam 2x 5. Má to tak být? klidně dej do init defaultní hodnoty a pak přiřazuj:

self.pbs=PbsConfig()
self.pbs.name=some.name
self.pbs.walltime=some.walltime

Hlavní je aby se to dalo vystopovat. V krajním případě tam dej aspon dictionary např preset['name'] už se kontroluje lépe.


changed to flat map from map and a flatten call after


we do! this is new: https://github.com/Khan/style-guides/blob/master/style/react.md#order-your-methods-with-lifecycle-first-and-render-last


@ccx-testerbot runtests master


Why did the access modifier change here?


Thank you for the request. It has been added to the queue (id: 128). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/128) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


That makes sense. The assumption is that there are images to process and the intent is those images will be up-to-date.


Ah ha! I found it.


that's weird


I don't understand the difference. What was the bug, what is the fix?


Hmm...its showing 86% to me. Nonetheless - I did not change the coverage badge. Maybe a hick-up of the coverall service?


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


I mean, that's a weird order to me


It should probably check it on line 73, so that if a non-object is used as data for object template nothing is done - otherwise it will crash on data.hasOwnProperty() on line 77 anyway. On the other hand if template[key] is not object we are ok and can set any value we got in data to that key (no more template checks to do)


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


@update-submodule: rocksdb


I was just going to delete this branch as soon as I saw Travis greenlight it - but you think thats a bad idea?


.cpp


You know, I think the `minimum_version` test is actually left-over from an attempt at using associative arrays for YAML parsing.  I'll remove that (good catch, again!).  The second test is to ensure the script has been sourced so that environment variables can be exported, I'll add a comment.


Whoops! Thanks.

LGTM


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


Feel free to update this, I seem to have some issue with some versions where it couldn't find some classes. Just picked a version that worked and I had installed. :smile:


Together with the right error message (several names match), the less specific ambigious mapping method error was shown.


My reasoning for making this private is that it could potentially include sensitive or upsetting terms, or terms that I'd rather keep obfuscated, in case anyone wanted to circumvent them with malicious intent.

That's my general reasoning. Maybe I could split them into public & private? Since certainly not all of them will fall into the above categories.


HBIRE_vector_take2 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1958 of 1960 tests passed)

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


uncritical change, go ahead


Fun to see that you also notice the typo error
https://github.com/yixuan/spectra/commit/3f775dfbe5aec4b2b4f149e1b4b3c44ea281b503


will fix!


Pieter Wuille <notifications@github.com> writes:
> Well we try to remain C89 as much as possible (only notable exception is
> uint64_t), for compatibility with small hardware devices and analysis tools.

OK, but if you have uint64_t, you have bool.  And for interface code you
can fairly safely "#define bool int" if it were ever an issue.

Cheers,
Rusty.


They will be deleted automatically by the `ShardCleaner` when it removes created shards for failed transactions. I don't think we want to duplicate all that logic here, especially deleting from the backup store, which uses multiple threads (since the backup store can be slow). Though we could delete the local files.


what it does it that I first recursively search for all files. Then I remove the common path with the current source directory, cut off the file name and use that as a group.

The problem is that there are no strong conventions on Windows how to group header files. It is for example common to have src and headers in one directory and then having one folder for each library. During installation all headers of all projects are then copied into one single directory. Thus the default behavior of cmake/visual studio is to display all headers as a group headers/*.h which is a mess. This code enforces our structure onto the system.


Everything has to be tracked and cleaned up if the process crashes (in which case rollback is never called).


it doesn't like the conditional being `true` and I didn't want to nolint that over the whole file so :\


Shouldn't this better say "exit" instead of "exits"?


You should use the GLOBAL.initialDirectory code of the init file


unused import


We don't do anything for rollback -- see above


yesssss :D


А для чего делать "/apiv1/reminders/user/{nickname}/reminders" ? Можно сделать просто "/apiv1/user/{nickname}/reminders", считая reminders стандартной коллекцией с операциями GET для получения коллекции, POST для добавления элемента и т.д.


I actually just copy/paste from the most current repo, so the typo fix was
your patch ;)

On Thu, Oct 22, 2015 at 1:30 PM Pierre Moulon <notifications@github.com>
wrote:

> Fun to see that you also notice the typo error
> yixuan/spectra@3f775df
> <https://github.com/yixuan/spectra/commit/3f775dfbe5aec4b2b4f149e1b4b3c44ea281b503>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sweeneychris/TheiaSfM/commit/a819d67586e2532b27a4ca0f94c7117b6142bafa#commitcomment-13937363>
> .
>


My latest Eclipse version here comes with a new version of Checkstyle that interprets our checkstyle rules a bit stricter. In this case, it reported "RedundantModifier". I'll prepare a pull-request to update the checkstyle version that we execute with maven as well. Then we can discuss if that is a rule that we want to keep or maybe ignore...


Instead of the above, have you considered using something like `git describe --dirty --tags`?


Here it must be fisheye in the <> !


Not really, window "w" is not loaded yet (so javascript is not working in window w yet) and does not include any JS libraries itself - prototype/jquery running in current window can only modify DOM in current window as far as I know. What is wrong with "write" in the first place, if it is the most straightforward way to do the job?


ehhh it doesn't fit if I indent it so I guess I'll leave it >_>


Okay, let's commit. We can fix this later if problems should ever show up.


thanks @ariabuckles! :D


deleting the branch is fine, but just as a matter of habit, travis test modifications should be local to .travis.yml whenever possible


compile error. unsigned int() is not parsed as one type but two types, so we have the int() cast and some stray unsigned in the code.


Ah, I see. Yes, I recall those "header" groups in the VS navigation bar. Quite handy for small projects, but sure, a nightmare for Shark.


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


We should fix error handling (i.e. username already exists) for next release. From my POV no hotfix necessary, bug fix definitely requested.


this seems to be missing the va_end() ?


On 22/10/15 20:40, Lars Brinkhoff wrote:
> Hello,
>
> Do you use Emacs, and if so, do you edit Kvaser t-script files?

I don't use t-script files.
Currently I don't use emacs either.

>
> I'm asking, because I'm trying to figure out if there's any interest out
> there in Kvaser open-sourcing our Emacs package for t scripts.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/eliotb/canlib-samples/commit/95bdf93746aabad06d0195c26d471013261efed6#commitcomment-13919541>.
>

--
Eliot


Is the other info not relevant?


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

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


PyArray_GETPTR3 is a macro using the variable so it should be fine


If we're talking about about docker then "Cgroup.Parent" is not always "docker". There is "--cgroup-parent" flag that you can set to any value.


This may crash if filter_expression is NULL.


also, it may be better to use vdprintf(), as that will allow us to continue to use fd 1
then a dprintf() for the \n


nvm... I said I'm tired :P


ack


Hello


@Tarmil Is this something you'd prefer we fix in Suave?


I know what NearTeleport does. blizzard doesn't send it for transports teleporting in the same map. as i said.


saw approval from lordnull
at https://github.com/basho/riak_kv/commit/23535f725c9322f09ce03368de2a3f4c2a40f5ab


merging basho/riak_kv/feature/new_encoding = 23535f72 into borshop-integration-1224-feature/new_encoding


basho/riak_kv/feature/new_encoding = 23535f72 merged ok, testing candidate = a3614649


We really don't want c++ constructs in the massive arrays ...


Changed to `filter_expression ? filter_expression : "NULL"`


indentation.


@erasche @bgruening >> 404, nice test case for the Proxy?


@dgreisen-cfpb - delete all removed sections.


You're correct, it doesn't work like this now in this file. What I was talking about is the other implementation of this interface inside libcontainer/cgroups/fs. If at all possible we should keep these two implementations consistent with each other, so that user would not have to know which of them is used at a particular moment.


I'm just surprised at our linter. `while (true)` is totally valid and has good use cases imo :).


редагувати тут??


I don't quite understand. What's your problem?


це тільки коментар


Не совсем понятно, почему запрос POST /apiv1/reminders/id/{id} позволяет добавить модератора
Кажется, логичнее было бы сделать ресурс  /apiv1/reminders/{id}/moderators, описывающий коллекцию модераторов для напоминания с идентификатором {id}. И работать с коллекцией с помощью стандартных операций GET, POST, DELETE.


this has nothing to do with this line


because JavaScript getMonth() is 0-based (and human-readable months are 1-based). The fix uses a PedigreeDate wrapper around the date, which has convenient getMonthName() absent in JS built-in Date, as well as getBestPrecisionStringDDMMYYY() which does the months correctly (plus incapsulates JS Date year mayhem - all those 1900 + ....)


that's fine!


if anyone has a copy, yes. It will be a good test case.


vdprintf() and dprintf here too I think


No. The first check needs to be after acquiring the mutex because the service may be waiting on the mutex when the operator is between windows. Then teardown is called outside of a window, when teardown is called the service resumes but it should not perform it's action outside of a window, so it needs to be terminated.

The second check is done to terminate the service as soon as possible, if that check was not there way may have to wait executeIntervalMillis before we can terminate.


and here


The messages are very different in different compilers... I could change it to an expression like `(message in JDK|message in Eclipse)`... but I wasn't sure, as it's a message generated by the compiler and not one that originates from our code (and thus is out of our reach if/when that's changed). WDYT?


why did the order change.?  Isn't the java version release6 not specific anymore..? BTW, should the order of diagnostics be important at all?


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


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


Falta modificar os links para o admin ainda


+1
It would be nice to get one of these Beta8 PRs merged and onto Nuget.

One thing to note, the dependency on xunit.runner.dnx is a little too open.  That package has been somewhat out of sync with the rest of the Aspnet development releases, so it would probably be best to lock it to the version that is known to work with beta.  "xunit.runner.dnx": "2.1.0-beta6-*", would probably be safer.


squashed commits


Если предполагается что-то похожее на REST, то, обычно, для добавления элемента в коллекцию используют POST запрос на саму коллекцию, или PUT запрос на конкретный элемент (если id известен снаружи).


Здесь скорее PUT, т.к. задан конкретный USER_ID.


Pokud vymýšlíš něco napříč komunikátorama stačí udělat funkci

def set_pbs(com_pbs,pers_pbs)
    com_pbs.name=pers_pbs.name
...
set_pbs(self.pbs, pers_pbs)

S tím že perl předá první parametr referencí

nebo:

def create_pbs(pers_pbs)
    com_pbs = PbsConfig()
    com_pbs.name=pers_pbs.name
    ....
    return com_pbs

self.pbs=create_pbs(pers_pbs)

Ještě ti chybí popisek u save a load_from_json_file


Just a point of clarification. Schema says href fields can't be null. Once BrianG and Sam's PRs are merged I believe that in cases where `forked_from.links.related.href == None` that whole tree will be gone. `forked_from` just won't appear in the object if it's not been forked.


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


А как получить полный список лотов?


review complete.


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **12/15** |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | **18/24** |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **5/21** |
| _WTV | 13/13 |
| _XDS | **33/34** |
| _Options | **82/87** |
| _DTVCC | 14/14 |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/128) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


@lperiquito the original code had:
`
if (len == 1 && str[0] == 'Z')
return true;
`
but it seems that this case was removed and will now return false. Not sure why this case was there to start with, but needs to verify that wasn't a valid case.


can we add a comment here as to why we need this extra check?


I hope.


will do


@ghitchens Actually is `Nerves.CLI.Cell.JRTP`, right?


How come this path was truncated? This breaks gulp sass pre-compilation for me on dev-master. Want me to add it back or update the installer to create this directory?


please move the `else` to after the closing scope at the previous line


@bbryantdccouncil - skips (b) - needs technical amendment? do you want me to comment on these?


review complete


I've removed that release6 thingy, as we must build with JDK 8 anyway.

The order is kinda important, as javac reports only the first message that is reported for a file/line. So the order in which we report the messages has impact on what's displayed in javac. Eclipse also retains the order in which we report the errors (which in this case is the same order as in which the properties are defined in the bean class).

If all that doesn't count, it was also easier to implement the logic that picks out the message that is supposed to be reported in javac :grin:


saw approval from javajolt
at https://github.com/basho/riak_pb/commit/0a8229bf091c8ae09f6f8d13447ac7b2b9cc88b1


merging basho/riak_pb/bk-rts-414-float-types = 0a8229bf into borshop-integration-139-bk-rts-414-float-types


basho/riak_pb/bk-rts-414-float-types = 0a8229bf merged ok, testing candidate = f5af6fb6


В каком виде отправляются данные? Если отправляется только конкретная ставка, то её можно вынести в отдельный ресурс.
GET /api/v1/lots/LOT_ID мог бы возвращать только информацию о лоте. Причем, обычный пользователь не мог бы делать PUT /api/v1/lots/LOT_ID - т.е. не мог бы менять сам лот.
А GET/PUT /api/v1/lots/LOT_ID/bid можно было бы использовать для чтения/записи ставки для текущего пользователя.


Use `$q` not Promise so it keeps angular in the loop. Also then we don't need to think about browser support.


This will resolve multiple times on settings change. What happens in this case? Is the promise.then fired?


I rewrote this in https://github.com/ImaginationForPeople/assembl/commit/549f78fbb9532859f284c1785ea0aa577c9e3a52 .  There were 2 issues.

1- Don't use magic numbers in setTImeout.  If you just want to maintain browser responsiveness, use setTimeout(something, 1), it enough to put it on the call queue and allow the broser to process events.  In case your actual goal was to save CPU for other things to update faster, you have to user a more complex worker pattern.

2- Totally normal that you didn't see it, but ideas:update actually maintains the collection, and has nothing directly to do directly with the view.  The problem here is that by delaying the collection update, especially by 2 seconds, you can introduce hard to debug race conditions.  Such things should be dealt with when processing the events, not when throwing them.  In other words, deal with them in the views, not the models.


It seems related https://github.com/ParsePlatform/Parse-SDK-iOS-OSX/issues/452


Done.


Whilst you are looking at this code I think that at the moment the machine usage is being reset (as if you are replaced the machine) when you edit the room/move the machine - which it should not (unless it has already been fixed recently)  So perhaps this is something you could look into at the same time, as there ought to be a mechanism which remembers actual usage?


Why not take a safer tact and just change the `PATH` prior to running cmake?

As long as `cmake` doesn't need to call `git`, this should be OK right?


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


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


HBIRE_vector_take2 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1958 of 1960 tests passed)

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


@milmazz you are correct, i screwed up and didn't save the file after editing.  I'll fix


I had this problem with the singleton children of BESFileLockingCache. Here's an example of an end of lifecycle that I used. I believe this to not bork things and it doesn't leak memory.
```c++
BESStoredDapResultCache *
BESStoredDapResultCache::get_instance(const string &data_root_dir, const string &stored_results_subdir, const string &result_file_prefix, unsigned long long max_cache_size)
{
    if (d_instance == 0){
    	if(dir_exists(data_root_dir)){
        	try {
                d_instance = new BESStoredDapResultCache(data_root_dir, stored_results_subdir, result_file_prefix, max_cache_size);
#ifdef HAVE_ATEXIT
                atexit(delete_instance);
#endif
        	}
        	catch(BESInternalError &bie){
        	    BESDEBUG("cache", "[ERROR] BESStoredDapResultCache::get_instance(): Failed to obtain cache! msg: " << bie.get_message() << endl);
        	}
    	}
    }
    return d_instance;
}
void BESStoredDapResultCache::delete_instance() { delete d_instance; d_instance = 0; }
```

When I modified one of these cache classes by adding a virtual destructor and then calling delete_instance() from inside it - THE BAD THINGS HAPPENED>


Some questions. (See above). But looks good in general. I think this is good stuff (and needed for the eclipse problems).. Thanks!


AddInitialCodeTests (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1953 of 1954 tests passed)

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


Here's a sniff of grom'gol to UC since you don't seem to want to believe me. https://gist.githubusercontent.com/Zedron/1afd077360b875868756/raw/47654198f5c30e80c1b79c3cdb6346196e421ab3/trans_teleport.txt


this is the real one


this is fake


now that's a funny name...


Dzięki za feedback. Kodzimy sobie z kolegą od Javy i pewne naleciałości z tego "dziwnego" języka się tu pojawiają. ;)


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


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


Świetna nazwa pliku ;)


fake one


move else


Hey @DrKLO. Since you closed the issue tracker, I'm not sure how else to contact you.

As noted in https://github.com/telegramdesktop/tdesktop/issues/1137, the issue tracker for this repo has been closed, apparently without (public) reason.
Could you shed some light, please? It seems rather strange.


@maxrothman, here's what I've come up with for the data migration. Per @explorerleslie, we can arbitrarily assign "bad state" users to either of their current memberships.

I've also elected to make the backwards migration a no-op like we discussed earlier. I'm still not entirely sure if there's more we need to do re: your questions about migrating from scratch, won't `for cohort_group in orm.CourseUserGroup.objects.all():` above effectively turn into an iteration over `range(0,0)` and do nothing?

I'm also curious if you'd have any thoughts/recommendations when we actually perform these migrations - should we do this entire thing transactionally with a table lock, to prevent more bad situations being created while we're fixing the current ones? My current thought is that if we apply the 0005 migration, then push the code changes, we'll have protection against double-enrollments, and at that point we can run the 0006 migration to fix previously-created errors.


Would `console.error` be a better choice?  That should give you more log granularity on the server.

Also, "unknown" is misspelled.


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


`
<time>Z
<time>±hh:mm
<time>±hhmm
<time>±hh
`


Nit: ... issue a conditional DELETE when acknowledging by including the If-Match header field.


-- Acid Part
http://paste2.org/z590mm5Y


Sorry about tabstops.


What changed here? Extra space?


please fix your commit messages when squashing.


missing handline of timezone


indentation


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.48](https://pixelated.design/build/viewLog.html?buildId=170&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 275 Build time: 00:00:17


Thanks for the code Meg!

Note that it breaks if the user has Mixpanel event names that contain spaces since the url needs "+" instead of spaces while the sig calculation needs to use the space and not "+". Adding a gsub(" ","+",event) instead of just event in the args_url paste solves this.


console.log


You know, I found a way about how to make all the releases :)
For the fact that's almost impossible to compile three ROMs in one weekend and also publish them I was thinking to make in phases like : This weekend for example ariesve and next week  CM12.1 and RR both at once.  What do you think man?


Addresses #32.  @indira1026 @danweili @jonglee1 Please see the updated user scenario for details on the interaction between the bird view editor and the street viewer.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_pb/builds/209


It's been reliable :) And there are some places where cmake calls git, such as https://github.com/puppetlabs/leatherman/blob/master/cmake/GetGitRevisionDescription.cmake.


@naveenoid I think most of the warnings were not hiding serious issue, but this function with an empty body could explain some of the strange behavior that we were seeing.


`class` is a native attribute, and attributes are assigned with `$` : https://www.polymer-project.org/1.0/docs/devguide/data-binding.html#attribute-binding


Still hard coding but... it's works with any language


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


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


Yes, sorry for not doing it on these edits!


program/include/rcmail_output_html.php and /rcube_sieve_engine.php (on line 1609) use the Q function (not the rcube::Q).


The [win] at the end of this line causes this .yaml file to become invalid. Please remove this to avoid an error in parsing the meta.yaml file to build vtk.


looks like your "optimization" effectively removed localization!


Why use sha256 and not sha1 here? Maybe it should be conditional based on user choice.


@Alexander01998, whats is with the "FullBlink" suggestion? (Records all packets)


time_remain - время в секундах?
Со временем в API всегда не очень просто. Если указывать относительное время, то такой ответ нельзя кэшировать. И непонятно, как учитывать время, которое тратится на выполнение самого запроса.

Возможно, лучше указывать конкретный момент времени. Например, Unix timestamp. Или в формате ISO 8601: https://ru.wikipedia.org/wiki/ISO_8601


Done.


`echo -n 1234567890 | md5sum`


The docs should be updated to reflect this (@kelunik and your blog post)


@wlbragg I'm a bit confused why you're using/need these properties. The rollspeed is already transmitted via `/engines/engine[7]`. See https://github.com/Juanvvc/c172p-detailed/blob/12b80dbaae5793b4ec6663a6cbff5175b6ecec55/c172p-set.xml#L520-L524 This is the information that the model should be using to rotate the wheels.

It looks like you [disabled it](https://github.com/Juanvvc/c172p-detailed/blob/12b80dbaae5793b4ec6663a6cbff5175b6ecec55/Models/c172p.xml#L234-L236) in `Models/c172p.xml`?

I think you need to revert back to `/engines/engine[7]` because of backward compatibility. Also, these `/fdm/jsbsim/...` properties are local and are not transmitted over MP.


@tobrun You need to delete `jid`! (not `icon) and do this after `std_string_from_jstring` below V


Are you sure that resty.sha256 is available to everyone? If not, its availability should probably be tested before it is used.


the newer plane must be more trustey and reliable


Version 1.0


Also forgot to add that I added the EVE-Overhaul atmosphere effect to Kerbin to give it a glow


stadmin should return a JSON for this to work all the way through


If it ain't broke, don't fix it. (Or create an issue about it and get to it later) =P


What's going on here?


Please put the file under  "2015-01-01-preview" folder


The bug was caused by the fix introduced for another bug in this commit. So the fixed bug wasn't there before the commit.

The initial fix was to make sure players spawn correctly when starting a new game.

Confusion...


My editor removes trailing whitespace.  Sorry 'bout that.


fast-forwarding end-to-end/timeseries to borshop-integration-139-bk-rts-414-float-types = f5af6fb6


Okay gotcha, I updated the confirm method.

On Thu, Oct 22, 2015 at 2:01 PM, Christopher Beck <notifications@github.com>
wrote:

> Hi Hillary, we don't want to actually save the user at this point - we
> just want to show this page to make sure all is well before we save them.
> Otherwise this is no different than create...
>
> You can also remove lines 34 - 39
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hillfeller/bloccit/commit/dbe7a368280f36da03d3520180bdc1c7bd8a5759#commitcomment-13934179>
> .
>


Try not to leave large chunks of commented code around. Since we're using Git, you can get it back later if you need to.


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


That is crazy!


Just caught that the `$200 and under` group should have a min-amount of $0 because right now it shows all the refunds.


Please update 2015-01-01\deploymentTemplate.json to include the new resource type references.


Same thing here, this can probably be enabled now that this has been simplified


Have a small mistake here related for travis!

Another doubt, what I need do, when the idx is less of 2?


`npm run pretest` will create that file. I don't know what are ZIP sources, but if you check out the github repository and run `npm run pretest`, this should be created for you.


previous discussion https://github.com/reddit/reddit-plugin-adzerk/pull/102#discussion_r42806566


So I had kind of assumed this was moved out the way because we need to use another `sh` somewhere... but given that you're already ensuring that mingw is in the PATH earlier than other tools... I'm actually not sure why this is here?


accidentally commented out?


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


Could you also add **volume**? https://github.com/schemaorg/schemaorg/issues/544


CMake complains if you try to generate makefiles for MinGW while sh.exe is on the PATH. It's really dumb.


Right, I mixed it up with the edges of the epipolar line. Sorry.


It actually won't work if tarantool works in 32 bit chroot/jail because CMAKE_SYSTEM_PROCESSOR shows information about OS kernel instead real environment. You should test os arch like Darwin at cmake/arch.cmake


Because 8444 is the default port for the CSA Controller. This URL is used for hyperlinks to the Subscription Modification and Service Instance Pages within the MPP (which by default runs on Port 8089)

Clicking them will open this MPP instance in a new TAB... and hopefully you allready have a active MPP session with a Consumer Admin user then you can do any modification/public action on behalf of those users.

If you dont have a active session open the MPP will redirect you to the login... which unfortunely doesnt redirect to the referer.


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


We've stuck with third person throughout (as you began). (Not 'I').


I don't think it is good idea to remove modifier state automatically like that. People may want to combine their macro with modifier keys. Let's say you define a macro as 'Alt+Tab' for 'Cycle windows' it is natural to expect Shift + the macro works as 'Cycle windows reverse'.

Instead macro commands should be extended so that you can tweak modifier state in your macro definition.
Like, SM(), for store modifier state, RM() for restore modifier state and CM() for clear modifier? Current Macro implementation is clearly poor and should be redesigned from scratch probably.
https://github.com/tmk/tmk_keyboard/blob/master/doc/keymap.md#231-macro-commands

I'll file this as an issue, thanks.


comes


next to `cmd` but '(`ctrl` on Windows)' (which I guess is correct... lol


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


@lperiquito I think it should be int param, use 2 for v2, 3, for v3. We can change that later if needed


Why remove F# Core from packages?


indentation


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


If "runbooks" can be a child resource of automationAccounts, you need to create two separate types for them, and reference them in Ln 40 and Ln 80 in 2015-01-01/deploymentTemplate.json.
Please use "databases" and "databasesChild" in 2014-04-01-preview/Microsoft.Sql.json as example.


added gitter badge


not "2015-01-01-preview"?


This should be in a separate and future PR since testing isn't flushed out.


there is an invalid test case that fails and I can't fix quickly


Awesome you just made my day!!!


ditto


I'll have to more closely review this, I'm not aware of what the implications of passing the guest token is here - was copying it across from the old code.


Can we call it lint and put it in the projects hash?


115200 is adequate as default and higher baudrates gives trouble due to stepper ISR limits


@lperiquito as I said in the earlier version of the PR, it'd be better to create structures for these types that are passed and then init and encode them. You only need to to have members in these structures for the entities that you need to encode. It'll make the code _much_ more readable.


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


@creativepsyco
Thanks for the guidance :)
I will surely look up to it.


It totally makes sense!!

ooo heres a read about android: being changed

http://stackoverflow.com/questions/28140662/is-androidshowasaction-deprecated

you probally all ready know this... but the difference i see with app:showAsAction="always" vs android:showAsAction="always" is

with app you get the 3 dots
with android you get the send pic.

so i think you way is totally the correct way...

btw thank you so much

thebigoso


+1, need this :)


Seems like this return value could benefit from a `namedtuple`, that way you can do both:
```python
response, form, cases = self._doCreateCaseWithMultimedia(...)
# and
case = self._doCreateCaseWithMultimedia(...).cases[0]
```


This might be breaking change.
Would you mind reinstating it, deprecate and introduce an new one?
Yes, I know that hasn't been reachable from the XML Parser. But it is public setter anyway and it is available from Java Config, e.g. SI Java DSL
Thanks


space after if (be consistent)


send commit from desktop


@davegarrett, I added SHA-224 to the same list as MD5.  I had to remove it from one other place.


Presumably this could just be a `go run`. We might just want to wait for #4822 to land as well, but I like your get-er-done style here.


What exactly are you seeing here?
We don't have any production like `K ::= "(" K ")" [bracket]` anymore. If several supersorts of Val have parentheses as a bracket, and the set doesn't have a unqiue LUB, an ambiguity error is expected.


Invalid download link :(


SimpleArray: http://ge.tt/5mW4nVQ2/v/0?c
Invalid download link also :(


Are you sure that you need this class with constants ?
I did like that many times but now I prefer to put string value directly everywhere


merge


I might be wrong, but I think most of our examples are for single objects. In particular, they also get checked into `examples/example_test.go`, and I'm not sure that supports multiple schemas. (See my recent #16069 as an example.)


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


use try-with-resources instead. If the IOException gets thrown while readLine() happens you might leak resources here.


The class itself can be moved to _converter_ package


> ... what this code originally did ...

These are the relevant commits:

* bce06c572ca07aed20fbd2ff915139631e6eb9f2
* f7a1099bd647d2a02fe0712b0310cd4597b6546a
* d7ca99c4e9f37dff6cfc4000d590c7280b86503e
* 4222ccd207daa05410409ce9072cc2dfb4921aa5
* 98dc5614d62156c1ce9def17c62e6ce4928aad4f
* 4eaf41a6c280856de1b55849a3f880d6494cef6e (root)


102867486-FixRectangleShapeTest-5173 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1947 of 1956 tests passed)

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


extra line here? Also, line before @author


final


The class itself can be moved to controllers package as it's a part of spring mvc


What's up with this check? Isn't ```property``` guaranteed to be in ```properties```?


Added Line 03


Thanks!

Let's see if we can get the proxy online by this weekend, and then after that I'll finish up with the sha256sum for all of IUC/planemo.


why are these variables not final?


why is this not final? Why not move the instantiation of the object into the constructor?


jainijaya.com


could it be more obvious to put *appname* in `properties.xml` as its name suggest that is a file to be modified?


format javadocs properly (cmd + shift + F)


format


Can you provide a likely working PV / PVC in the PR? Then you can point to it and say "just edit the name".


Closing issue SoftwareEngineeringToolDemos/ICSE-2013-LAMBDAFICATOR #7


Someone's been writing too many JIT compilers.


When you're creating objects for examples, providing links to the objects somewhere in the example is nice so that people don't have to hunt the toplevel directory to explore the structures (if they're looking at it in the browser).


Could also the logPattern be defined on the `properties.xml` file?


For this and above, you don't want release-1.0 description. (I know this is boilerplate.. there's a different boilerplate form for 1.1 or 1.2+-only stuff, though)


Did you verify if this is the only thing that needs to be freed?


Use void - if you don't want to return user's details


why are these package-protected and do they need to be? can we alphabetize these static variables to have some sort of sane ordering?


For this README and above, use backticks for `make_secrets.go`


nope. ewf was moved into radare2-extras.


Delete *ErrorDto* from _dto_ package

General rule for dto objects - they should be generated automatically from .xsd file for example


be consistent with how you're catching exceptions (same line as closing brace [}] or different line).


I was kind of expecting to see the syslog service tied to the haproxy example somewhere. How are they linked?


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


nope. this is only optional at runtime. this dependency should go for building valabind, which is required for radare2-bindings


use a Logger instead of printing this stack trace


nope. this is necessary only for radare2-bindings


doesn't seem like you're using req here, simplify the method?


gotcha thanks--it's good now


Closing issue SoftwareEngineeringToolDemos/ICSE-2013-LAMBDAFICATOR #4


this dependency is checked in configure, but not used in any part of the code. I will remove the option in configure


For the future, do you mind setting eclipse to put the bracket and ```else if``` on the same line?


The signature is the same as the normal callback, except for the return type. And while the JVM would know the difference, the compiler doesn't like that. So no.


This is not necessary for radare2. if you want to use lua you'll have to find this in radare2-bindings.


dont worry man ;)
you can build it whenever you want ;)


no openssl used yet. will probably be implemented in future releases for the RSocket APIs


that's a good point. for now i'm gonna leave it cause it matches the way `submit_form_locally` returns data, but in a separate PR i can do that. (or you suggesting just making it a named tuple for this test)


This will throw an error on Unicode strings...  Does anyone know what Java does for Unicode?  UTF-8?


Why not cherry-pick?


why not use java.util.Properties?


you can drop --enable-cparse. mainly because its deprecated stuff that was work-in-progress for experimentation, but currently its not used anywhere in the code. I'll remove that configure flag too.


> propTypes are more like meta data about a component so its nice to define them apart from all the behavior.

I disagree – the nice thing about having the `propTypes` defined _in_ the component file is that you have a clear, consistent place to look for what props a component uses. It's part of the definition of the component.  The stuff in `propTypes.es6.js` makes sense where it is because it's actually defining custom _types_ (e.g. a `comment` is a specific type of object that a component may require as a `prop`).  Moving the `propTypes` definition out of the component's class file just adds some unneeded indirection, and would complicate imports unnecessarily.  Unless I'm not understanding you correctly, in which case, ignore me.

As for having `propTypes` defined multiple ways, that's true in a sense.  This `static` syntax is functionally identical to what was going on before, it's just a newer syntax for defining static properties and methods (i.e. properties and methods on the `constructor` function) _inside_ the class block.  So they're really the same.  And as for the stuff in `propTypes.es6.js`, those are really a different thing altogether, as mentioned above.


[Presentation 1.txt](https://github.com/jengol/FirstTestRepository/files/18926/Presentation.1.txt)


wrong oil


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1944 of 1953 tests passed)

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


can the Exceptions here be more specific? For example, FNFE, IOException etc. Also I would prefer if we check that the configFile exists before instantiating ExecutionConfiguration instead of relying on a try { } catch { }


space after for, space before :


This is also pretty unfortunate ;)


Filed that in #116.


Oh, you're right. It's my mistake. Fixed by #1805.


Thanks for the catch @artembilan - will add a deprecated setter for backwards compatibility


```internalToJSONRelationshipPropertyMapping``` sooo long... Don't you fingers get tired :)


space after if


give an example of what the path should look like? I don't really understand just by reading this


Works for me, hope this gets merged in to the CocoaPod soon.


hey @droberts84 I have a question about this


Is there any other functionality that is expected to be affected by the local properties at the time the job was kicked off? Would it be safer to propagate all local properties, or would that lead to any unexpected behavior.


it doesn't seem like userID is being used anywhere. Also, final variables, and private if possible.


final?


Closing issue SoftwareEngineeringToolDemos/ICSE-2013-LAMBDAFICATOR #9


format javadoc


System.out.println(addEmployee(i,name));


Closing issue SoftwareEngineeringToolDemos/ICSE-2013-MCT #14


>
>  WDYT?
>
Get your point. Leave it as you have it now.
>


prefer using enums to make this more extensible without having to keep track of magic strings.


//$NON_NLS-1$ ?


I'd prefer if you did a `rebase` and `push -f` instead of merging.


I'm not really worried about `let` at all – `let` is actually _easier_ to reason about _because_ it doesn't do hoisting.  Babel compiles it to `var` by altering the names slightly to avoid collision and doing compile-time error checking when you try to, for example, use something before it's instantiated.

The reason I personally prefer conditional components to be defined inline like this is that you can pretty much just look at the return statement from `render` and see all of the possible shapes the resulting HTML might take.  When components get a little complicated, you have to do this scrolling back and forth dance to see what markup some variable in the template might produce and what the conditions required to produce it are.  Personally I'd rather just see it inline.  And if the conditions are so complicated that the ternary statement becomes an ugly mess, that's probably an indicator that you need to break that out into a separate component.


Since the config isn't very complicated, maybe consider augmenting the config with get<field> methods so that you don't need to expose these magic key-strings. For example, getPythonExecutable().


leaked resource


indentation


Alternatively you could use `void` from `Control.Monad`. Just FYI, it's not a style requirement.


Yes! IMO we should really start trying to avoid `{...this.props}` when possible.  It makes it harder to tell what's really going on.  The case that it really makes sense for is if you have some component that's acting as a _wrapper_ for another component, where you just want to "pass through" all the props to the child component.  Otherwise, explicit is better!


Weird, fixed it and the other one as well.


Cool thank you for the PR.


web110 uses github? Cool!


This will work as-is, but will count against our server's IP rate limit. Can we follow convention and do a check for g.user and do a `proxy_request` if non-auth'd? GitHub has bumped @webcompat-bot's rate limit. :arrow_up:

(I have some ideas for how to clean that up in the future, it's ripe for some refactoring).


не шей мне срок!


Apparently I forgot to save this. Oops.


Just one point here: `_make_test` considers the second argument to be a test "uid". From the docstring:

```
:param uid: test uid (by default used as id and name)
```

This changes that, and also the "id" param which gets set to the same value. Have you checked that this is safe and makes sense?


Wouldn't this be better?

            size_t qbits = EVP_MD_size(evpmd);


ss


We need a comma after the `}` here, otherwise everything else in this `var` expression will leak as a global variable, including `formatFile` and `TRANSITION_DURATION` below.


Same as above re: comma.


@setriplette I think I see in here the abbreviation mark under the 'p'. That's why I chose "partes" over "pies" but I wanted to make sure that you agree with this change.


Some people were saying they could not use binding redirects with FSharp.Core, so we tried to remove it. I'm thinking this might be a mistake.


Sure I'll do this and send e new patch.


@devilcoders Are you going to propose a PR for this change?


@burner Yes, and (AIUI) Andrei is saying that if we *need* to add all these tiny functions that don't *do anything* but fix compatibility and eat space, then we're doing something wrong, and need to fix it on another level (possibly in the language itself).


mailto is definitely supported. I'll update that to `URI`


Hopefully this will help in issue #35.


Great! excellent!


myWebPage.github.io/testPage


space after if


Cmake--


you can rely on the compiler to infer the type -- new ArrayList<>


Change it to `grep -qE '(x86 boot sector)|(ISO 9660 CD-ROM)'`


@evaleev  please take a look at this to ensure I am not doing anything stupid. This should reduce the amount of dummy index generated. Hence, reduceWick should be speeded up.


@mperham does this look right to you?


When running pro_setup.sh, it might ask you to add config.secret_key = ''' to config/initializers/devise.rb.


This is specific to each computer, and shouldn't be committed like this.


Which bug is this commit fixing?

I used this method to transform the mouse position into the displayObject's space, which worked without any problems. Now this commit breaks everything for me.

-Trixt0r


no, just a list of requirements which are pretty specific


TeamCity Azmq :: Build (Windows) [Build 176](https://78.46.71.115/viewLog.html?buildId=638&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 147](https://78.46.71.115/viewLog.html?buildId=637&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 129](https://78.46.71.115/viewLog.html?buildId=639&buildTypeId=Azmq_BuildOsx&guest=1) is now running


@IIvec: Now we are following a different routes: You keep a one-parameter optimisation problem for a functional with one variable but give up a strictly logarithmic formula, whereas I need a second parameter (total number of threads) and keep a logarithmic index-depth formula.

How does an assumed system with n-fold number of cores which individually are n-times less powerful behave?

Regards, Alfred


I think you got it confused here.  `(i < 1000)` means success, not error.


Doesn't IDEA have autocomplete?


And error should most definitely NOT return 1!


AddInitialCodeTests (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1976 of 1980 tests passed)

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


TeamCity Azmq :: Build (OSX) [Build 129](https://78.46.71.115/viewLog.html?buildId=639&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 29 Build time: 00:02:42


why does this need to be synchronized? Nothing in here is referring to an instance variable as activeProcess is a local reference.


repeating


TeamCity Azmq :: Build (Windows) [Build 176](https://78.46.71.115/viewLog.html?buildId=638&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 29 Build time: 00:02:59


Yes. Do you need to update the class attribute in the call to instrument?


TeamCity Azmq :: Build (Linux) [Build 147](https://78.46.71.115/viewLog.html?buildId=637&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 29 Build time: 00:03:17


i dunno how i messed it up but i did

I'm going to bed ok


two commas? o.o


Got it, sorry


Excuse me sir, but you are not giving proper credits for my website: www.androidmodguide.blogspot.com


В качестве базы данных и хранилища результатов насканированного интернета можно использовать  Links Platform.


OK then, it makes sense to use jQuery UI's tabs() for this. I tried it and the vertical tabs themselves get made into big button things (looks like a car crash, but that can be styled later). But now the tabs don't switch when clicked. Looking at the demo you linked to, I wonder if this is because jQuery UI expects everything -- tabs and panels -- to be wrapped in a single div? In your original markup there are two divs: one (`txp-layout-4col-cell-1alt`) for the tabs, and the other (`txp-layout-4col-cell-2-3-4`) for the panels.

Do you think this might be the cause? And if so, what markup / class names would you prefer we use in order to appease jQuery UI and make the tabs work.


It seems that any `x` prefixed with `"--ghc-options"` has already been caught at line 542?

Or is it simply that the `"="`s are missing?


If ips and @proxies are arrays, can be just `ips - @proxies`


Yeah, I noticed that happened when I went right back to the door placement stage but I think it was working for just moving the machine ( I tested moving the machine to get the smoke animation to follow).

I don't suppose you know the behaviour of the original game here?
Does the player get charged if they go back to the door placement (making it fair for machine usage to reset)?

> On 22 Oct 2015, at 21:53, MarkL <notifications@github.com> wrote:
>
> Whilst you are looking at this code I think that at the moment the machine usage is being reset (as if you are replaced the machine) when you edit the room/move the machine - which it should not (unless it has already been fixed recently) So perhaps this is something you could look into at the same time, as there ought to be a mechanism which remembers actual usage?
>
> —
> Reply to this email directly or view it on GitHub.
>


oh, great, thanks. it have obviously changed recently, it was 255 before. but i wonder why the gnu social devs didn't change it to _text_ when they decided to change from 255 to 191.


@naveenoid  Merge artifact. If you log in on https://travis-ci.org , you will get automatically notified when your commit results in a failed build, as in these case: https://travis-ci.org/robotology/idyntree/builds/86845643 .


No, `var...in` lists all properties inherited from the prototype chain. `hasOwnProperty` checks that it is a property directly in the object.

Prototype.js has `.each` which only loops over direct members, but we're trying not to use too much protype.js code in the pedigree.


And if 'key' is a bytearray, data is never assigned to.


It's ok your code!!!


Ah, yes, I believe I do. Thanks. :)


@fhemberger can you give me npm publish rights? Also, I've got you in the README and LICENSE. Is that what you were looking for?


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


just pushed this change


@IsaacSanch those tokens included in the appveyor.yml file are encrypted and specific to the halcyon build on appveyor. It's fine for them to be included in the git repo. See http://www.appveyor.com/docs/appveyor-yml and examples in the wild like https://github.com/HangfireIO/Hangfire/blob/master/appveyor.yml.


Wrong behavior.
It filters out all other headers!
Only the connectionHeader with close value can be reserved.


you're swallowing the stack trace here, which might make debugging hard. Can you log it with a logger?


indentation


Could you comment on what is going on here and why?


Hibakezelés még nincs benne, de maga a kinézet már egész jó.


void moonWalk() {
  for (int i = 0; i < 5; i++) {
    reverse();
    right();
    reverse();
    left();
  }
}


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


I don't really like this patch because now I have to use the mouse for editing multiline repl commands.

Do you not have this problem too?


What happens if the thread panics?


There's also some guidance floating around saying that it's best to avoid forcing people to manage FSharp.Core from NuGet when they might not want to. It seemed like the right thing to do, but I have to admit that the whole area (and don't even get me started on CoreCLR) seems really hazy right now. Is it likely to cause problems for you?


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