However, it does highlight an issue here - this code isn't internationalizable. The joining should be implemented in a mustache template instead of in code.


ack, according to https://github.com/python-babel/babel/pull/278#issuecomment-166254392


done! threw in the pull request.


This line should be tidied up. Presumably, the aim is to create a list of numbers, which represents a sine wave. However, without a comment, it is not clear what is trying to be achieved.


@tchaikov I was tempted to write something like that. But I believe a simpler notion helps with understanding.


done


Hi Helina, no this is the prediction stack.


There are no doc-strings. Use doc-strings to document your code according to the PEP guidelines.


Good point. I'll add it to the commit guidelines.


Ultimately, we should probably be factoring up these kinds of things, so that when the model changes, the tests change too. For now, go ahead and make mock objects based on the chat model.


Cool.

Thank you for this wonderful book. It is the best stap book ever.
I highly appreciate you combine the dtrace and stap. I have zero background
on dtrace but it's helpful to understand the stap part. To me, it's acting
as "DONOT",  "AVOID" or "RESTRICTION" of stap.

Thanks,
Yuchen

On Sat, Dec 19, 2015 at 9:55 AM, Sergey Klyaus <notifications@github.com>
wrote:

> Thanks for your attention. I am definitely a victim of copy-and-paste here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/silentdai/dtrace-stap-book/commit/7d4b1c5132f88b78d1d322171f9209a8b4565365#commitcomment-15081894>
> .
>

--
Yuchen Dai


Yep. We should have constructor with credential as parameter.
But in many places, we downcast the specific credential types because of https://github.com/ddf-project/DDF/blob/master/core/src/main/java/io/ddf/datasource/DataSourceDescriptor.java#L16. Do you have any good suggestion for this?


@ucay : saat ini iya om... ok buat issue aja om :-)


嗯嗯，我错了


reack


According to [mdn](https://developer.mozilla.org/en-US/docs/Web/API/XMLHttpRequest/responseText) - we can't always get the responseText, in some cases (like contentType is json) we'll get an error here.
Since we're not sending it anyways - removing it


rain.


@AlexBulankou @kamilszostak - FYI


good idea!


Why does it have to run in `dist` to use Babel?


Why this dependency? The library targets do not need to be built in order to generate the documentation (ignoring generated header files here), so there is no dependency (and therefore this file does not need to be modified for other projects)


See section "on varints" https://github.com/jbenet/random-ideas/issues/1


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1038](http://teamcity.monogame.net/viewLog.html?buildId=12787&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


ill remove this


104921238_EnergyPlus_Performance_Framework (lefticus) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


http://blog.durandal.io/2015/12/17/more-aurelia-performance-bug-fixes-docs-and-typescript-awesome/ mentioned that there is now a TypeScript version of the skeleton-navigation project. I added a link to make it easier to find.


2245154-NewVRFHRModel (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1986 of 1986 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)


:beetle: since we are getting this from the `$location.path()` the path here cannot be be always denoting the logged inuser. `$svcEmployee.loggedInUser` does not currently give the correct object and hence this hack of getting the empno of the logged in user from $location


Running vulcanize in `dist` feels like a bad practice to me. It seems like if you need to output files before you can run them through vulcanize you should do so in a `tmp` dir. I'm not as familiar with gulp as I would like to be, but this feels like something that it should be able to handle. In other words, outputting to temporary directories feels like a gulp anti-pattern and likely there's a way to pull this off with streams maybe...


@ragesoss FYI. This approach will be preferable to the inline CSS and `<br />` tag.


nice, thanks!


Thanks for all your help! Btw what happened to that RF censor hack you had ?


Alright, remember how we were going to use branches?  How am I supposed to Merge the changes I've also made to this file?


Immutability FTW. Easier to detect when an array changes if it's actually a different array – otherwise you need fancy containers with mutation events. Like this you can just do: if (newList != oldList) renderAway();


+### well hello pepole


ShellSettings is not in Abstracction. It could be if the logic to load it was not inside. Just a POCO (we talked about it)


seems like some distro have backport overlay(fs) to their kernel source tree, which is not label as >= 3.18
```
$ lsb_release -d
Description:	Ubuntu 14.04.3 LTS
```

```
$ uname -r
3.16.0-57-generic
```

```
$ modinfo overlayfs
filename:       /lib/modules/3.16.0-57-generic/kernel/fs/overlayfs/overlayfs.ko
alias:          fs-overlayfs
license:        GPL
description:    Overlay filesystem
author:         Miklos Szeredi <miklos@szeredi.hu>
srcversion:     35D0993AD5FBD675F9C5DB8
depends:
intree:         Y
vermagic:       3.16.0-57-generic SMP mod_unload modversions
signer:         Magrathea: Glacier signing key
sig_key:        8B:33:3F:BD:53:E5:7F:5F:9D:72:A5:E3:98:47:A0:37:01:4B:2A:5A
sig_hashalgo:   sha512
```

what if we check only for basic support? through the command `$ modinfo overlayfs` ?


@paulproteus @zarvox FWIW I like this added Zen. It accurately represents Sandstorm as a more accessible and interesting option compared to other operating systems.


should be basic requirements check, I think... :(


IShapeTableProvider is called at startup, and these methods change the state of bindings. We have to do it this way.


104921238_EnergyPlus_Performance_Framework (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1432 of 1432 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.49%25-green.svg)


Not fixed. Too small pictures are still being uploaded. No more immediate display, but pic saved to db and on next page reload it'll be shown.


Please put `\s*` between `if` and `__name__` as well.


Which one do you prefer? I tend not to use Linq if I can.


When will this return statement be reached?


`"/api/v1/switch"` :p


@thenickcox This introduces a regression: before the course begins, it will say "This week" instead of "Week 1"


:pushpin: since the service function is to return a promise, the dummy employee would be filled up on resolve. `$svcEmployee.loggedInUser` would then get the data when the promise is resolved


Fixed in eb6c9e949a13a9edf1e07cd4b74fa0787a468a3d


Or rather, "First Week" as before.


@JacobGross: it is already there!

  https://github.com/leanprover/lean/blob/master/library/logic/identities.lean#L42-L43

(I found it by typing `not_or` and using tab-completion.)


this is now fixed , loggedInUser can be read from the static service


This breaks string attributes when used with any of the numeric comparison options (greater_than, etc).  This happens because the string is no longer cast to a numeric class before being compared - so if we have an attribute with the value "12" and we use greater_than: 1 option we get an "*** ArgumentError Exception: comparison of String with 1 failed" error


Never, in any situation I can think of.  Technically, it will happen if nnb gets a negative number in it somehow, though I'm not aware of any way that nextnonblank() will return a negative number.  I'm not a vim scripting expert, so I left a return statement in there at the end in case some very weird situation manages to occur.  If you think this is foolish or unnecessary, I have no objection to removing it.


Is this change actually in the 20151216 release? I'm still seeing spurious JSC_USELESS_CODE warnings with empty JSDoc with that release, but a build from HEAD is fine.


Also made default theme Dark.


Also fixed hotfix not working properly


Sure, will do.  I wasn't even intending to issue a pull request for this commit, but I'm happy to make the change and merge it in if you want it.  (I'm a longtime github consumer, but new to the pull request system.)


That's quite an improvement at this stage.  It's not clear what is partially done and remaining.  Will a test for Solidity also be added?


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


This isn't quite clear to me. How does one acquire xcode-cli, and what does xcode-select do?


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


Looks very cool!

Can we have more detailed helper message describing all the YAML config, such as "variables:", "choices", "min", "type"? In Python code, you can rely on "Tab" key to browse the fields so all these might be discovered by user easily. Same for existing YAML configs for chart magic.


What you're doing here was already possible with ValueKind, it just wasn't consistently done. For example, see propagateLiveness in DeadCodeElimination.cpp, which defines TERMINATOR() and VALUE() to generate an exhaustive switch, vs. markTerminatorArgsLive in the same file, which doesn't do this.


MArray_Elim (DeadParrot) - x86_64-MacOS-10.9-clang: OK (1995 of 1995 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)


Right below (`$!`). It is not evaluated yet (single quotes).


There are new strings (in a new paragraph at the end) for 1.07 version


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1038](http://teamcity.monogame.net/viewLog.html?buildId=12787&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 555, ignored: 6 Build time: 00:27:18


@ragesoss Fixed via 4fe1fcc. Good eye!


I think it's fine to leave in, I was just wondering if you had thought of a case that I had overlooked. Would you mind adding a short comment indicating that it shouldn't be reached?


Sure thing.


Thanks for sending it as a PR, I appreciate it!


Range should not be ignored, since it is a valid header to be part of signed headers.


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


rename this as

`tmpFile = `${filePath}.${result.etag}.part.minio-js``

Every where else we will use the same style.  a ".part.<lib>"


looks good, cheers


作法というほどのものはないと思いますが，

1. 実行前に `:w` で保存してしまう（無名バッファを考慮する必要あり）
2. 一時ファイルに書きだして差し替える（vim-go の `:GoFmt` とかがそうしてます）
3. 現在のバッファの文字列を直接食わせて出力結果でバッファを置き換える

とかじゃないでしょうか．
対象コマンドのインターフェース知らないのでなんともですが…


> I would think that if a user sets up a cost on a timesheet product, he
> would expect this cost to be used, which is not the case anymore with this
> commit.
>
That could be true, and that case might need to be corrected again - even
though it was not covered by any of the module's tests (was it really the
module's main goal?)
Please discuss it with @antonylesuisse, as he was very much in favor of
removing this use case.
The premise is that most of the time service companies sell services with
different products (Service Pack, Consulting Days, etc.) than they are
actually consumed (Employee hours). So the case where they might want to
see different SO product costs used on timesheet lines for the same
employee seems to be the odd one rather than the default.

In any case, for timesheet entries linked to SO lines, the UoM of the line
*must* be the company's project UoM, because there is no way to input them
in any other UoM, as you've noticed. In that context, even if there is no
cost on the employee, using the SO product makes little sense because the
UoM are likely to be different. What do you do with the cost then? Convert
it from the SO product UoM to the company's project UoM, hoping that
they're compatible and that it was the user's desired behavior? Isn't it a
bit far-fetched?

And if we don't do it for timesheets linked to SO lines, are we still going
to use the SO product at all?
Perhaps this shows we're trying to mix two different concepts (sale product
vs timesheet product) within the same db field(s) after all?

BTW for further improvement, let's consider the following:

- the `_get_timesheet_cost` method looks like it could be merged with
`_get_sale_order_line` with appropriate overrides, rather than having
**two** magical methods called for every create/write of analytic lines (to
simplify the code in these "new" modules)
- improve the tests to match the desired behavior (see WIP with
@antonylesuisse and @tivisse)
- cleanup/clarify the module description according to the above
- the "delivery quantity" field on SO lines is quite unusable for tracking
consumption as soon as you're selling in larger units than you're
consuming, e.g. 1x 25-hours support pack, or 2x day of consulting, consumed
in hours. The "delivery quantity" will jump on low-granularity values such
as "0.10" as soon as the first hour is consumed, which is a pretty bad
indicator of the actual consumption. It seems the use of different UoM
between sales and timesheet was globally overlooked.

Thanks!

PS: I believe the other parts of the patch with the `unit_amount` vs
`amount` used in the domains for `_compute_analytic` was quite needed
irregardless, because the amount is not even used in `_compute_analytic`
(so that's not consistent), and because the `amount` could still be zero in
some cases, e.g. when no employee cost is set (the default) and product
cost is zero too, or no product is set at all.


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


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


It looks like it is because the release was built just before this rollback. To get rid of the warning you can add any declaration such as `/** @type {?} */` or `/** @public */`.


Outlier1/master now restored with 2015 season code


this is a vertical margin no?


again a vertical margin?


bueno


> On Dec 21, 2015, at 6:30 PM, Mark Lacey <notifications@github.com> wrote:
>
> What you're doing here was already possible with ValueKind, it just wasn't consistently done. For example, see propagateLiveness in DeadCodeElimination.cpp, which defines TERMINATOR() and VALUE() to generate an exhaustive switch, vs. markTerminatorArgsLive in the same file, which doesn't do this.
>
>

Ok.

So are you suggesting that we use that approach instead? I think this is a bit cleaner b/c at the use-site you do not need to #include every time.

>
> —
> Reply to this email directly or view it on GitHub <https://github.com/apple/swift/commit/9c37bddc60100c28572f6703e73824a79d91ab3f#commitcomment-15108797>.
>


We need to handle to create top level directories here, since a filepath can have top level directories as well.  Do not know if createWriteStream can create top level directories.


Hmm, I can fix the case in the postgresql database fields (they are only views) so the match the hardcoded fields in the toolbox. If the quoting works with access, then leave it, as it breaks other bits working with postgresql.


Should we have a switch specific expires_in value?


use the short syntax `if (!core->lines_cache) {`


errors must be shown using `eprintf`


same as above


wrap the access of this in a function?
sub cache {
   return $CHI_CACHE;
}


use `r_num_math` instead of `atoi`


smells like a very dangerous crash


cache the value of atoi(input+2) because its calculated twice and verify the boundaries on the same variable


Parameter `echo` nastavi na bodisi `TRUE` ali `FALSE`, ali pa ga odstrani. Spremenljivke `razpredelnice` nimaš, zato se prevajanje poročila tukaj ustavi.


eprintf


handle r_cons_break()


check return value of r_io_read_at ? maybe use vread?


what about initializing this in the `sl` command instead of the config var callback?


@twalpole could you open an issue? It is easier to track the work that needs to be done. Thanks


#HelloWorld.md
##This is a markdown file


Recipes using gem dust were returned as gems, allowing to avoid implosion compressor. For example the diamond grinding head. 4 dusts and 1 diamond return 5 diamonds. Now they return 5 diamond dusts that must go trough the implosion compressor.


unnecessary if. free already checks for null


Before we rename, we need to verify the content received was exactly of same length and we read exactly so much. Throw error if we received partial data.


Done, thanks!


We started and maintain this project. I’ve already given him full permission to merge my tvOS code with iNDS. He can merge my code anytime, but I will continue to contribute to the original.


Ne uporabljaj šumnikov in presledkov za imena blokov, saj jih LaTeX najverjetneje ne bo požrl. Primeren nadomestek za presledek je tukaj `-` ali `_` (tudi pika očitno LaTeXu ni všeč).


Optional feedback / subjective opinion: Personally I find this transition hover effect to be a little slow, and I think it makes the buttons seem sluggish. It's also weird that they fade out instantly rather than over 1s when you un-hover them.


Thanks @nicklockwood! :rocket:


Why the inversion here?


# Pre-work - *TipCalculator*

**Name of your app** is a tip calculator application for iOS.

Submitted by: **Morgan Wixted**

Time spent: **3** hours spent in total

## User Stories

The following **required** functionality is complete:
* [* ] User can enter a bill amount, choose a tip percentage, and see the tip and total values.

The following **optional** features are implemented:
* [ ] Settings page to change the default tip percentage.
* [ ] UI animations
* [ ] Remembering the bill amount across app restarts (if <10mins)
* [ ] Using locale-specific currency and currency thousands separators.
* [ ] Making sure the keyboard is always visible and the bill amount is always the first responder. This way the user doesn't have to tap anywhere to use this app. Just launch the app and start typing.

The following **additional** features are implemented:

- [ ] List anything else that you can get done to improve the app functionality!

## Video Walkthrough

Here's a walkthrough of implemented user stories:

<https://drive.google.com/open?id=0B8f6ujiCr8mSTTRFWWI1ZExqVUk>

GIF created with [LiceCap](http://www.cockos.com/licecap/).

## Notes

Describe any challenges encountered while building the app.
I was using a different version of XCode, so some features were different. I had to delete my project and start from scratch because things just weren't working. I also spent a lot of time trying to get the settings screen implemented, but I never quite figured out how. I am going to email and ask for help with that. I am now trying to implement different features of the app, which will be in future submissions.

## License

    Copyright [2015] [Morgan Wixted
![tipcalculator_1](https://cloud.githubusercontent.com/assets/14119170/11945307/64c28106-a81c-11e5-99f5-ac1ec068890c.gif)
]

    Licensed under the Apache License, Version 2.0 (the "License");
    you may not use this file except in compliance with the License.
    You may obtain a copy of the License at

        http://www.apache.org/licenses/LICENSE-2.0

    Unless required by applicable law or agreed to in writing, software
    distributed under the License is distributed on an "AS IS" BASIS,
    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    See the License for the specific language governing permissions and
    limitations under the License.


The filter has been changed:
https://github.com/uhm-coe/insert-pages/commit/7e1e85d6d4611b6bdea9613ae0889bfa13e1d4da

Updated in version 2.9.1:
https://wordpress.org/plugins/insert-pages/changelog/


This is telling to generate the source maps as inline file instead of as an external file. https://github.com/sass/sass/blob/bce9509f396225d721501ea1070a6871b708abb1/lib/sass/source/map.rb#L83


I don't think so.


Theoretically it should still work -- the way censorship works on RumbleFighter, is the client loads a list of censored strings line by line from one of the NSZ archives located in their resource directory. Therefor, if you unpack the archive, wipe the list, and repack, your client won't censor IGNs or messages created on your client. Censorship is not done via the server.


Perhaps consider pulling this function out of the loop and giving it a name?


$pf::Switch::CHI_CACHE is not enough ?


Optional: This is ok but it would probably be better to do `gettext("Create a %(mozilla_backpack)s account, or log in to your existing account", { mozilla_backpack: '<a href="...">' + gettext("Mozilla Backpack") + '</a>', })`


So this means that the description, attribution, etc. fields will always return lists, right?


If you do something like
```
  private def configFromFile(devFileName: String): Config = knobs.loadImmutable(
    Optional(FileResource(new File(devFileName)) or ClassPathResource(ConfigFileName))
      :: Nil
  ).runFor(5.seconds)
```
Then you can override the default configuration with a developer configuration.


typo


I did a quick, not thorough,  scan of boost 1.60, did not find `namespace helper`, but I did find `namespace util` in boost.wave. I'll let @berkus to decide.


@melloc This looks good to me.


Hello Swift^_^


I like the TermKind enum and having getTermKind() because it does make it cleaner.

I wonder if TermKind should really have 'Invalid' though, as opposed to ValueKindAsTermKind's switch just having an llvm_unreachable() directly in it for the non-terminator cases, since that's what I believe any caller is going to end up doing with Invalid.


if not isinstance(chart_options, dict)? chart_options can be a string if content is not a valid yaml.


if choices is None raise Exception? Same for 'set'.


Just wondering, do you know when this fix will be integrated into a release?


The thing that should be cool is to have a way to cache directly in the Net::SNMP object and add a third call like: $self->{_sessionRead}->get_table( -baseoid => $OID_ifDesc )->cache
By this way we will be able to cache only snmp request that are static, like for getVersion but not for getPhonesLLDPAtIfIndex.


I need a program to unpack a nsz.


Please stop posting ignorant comments. This change predate the so call migration which was never agreed ahead of time. This is the latest version. All further development shall be based on this and ONLY this version.


Yeah, the test case should be fixed.


2245154-NewVRFHRModel (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1986 of 1986 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)


Why was this reverted?  Very useful use case, that doesn't appear to be accounted for in 2.0.


@humblehacker Hm. I think the only valid way to specify boolean options right now is to include `is_string: false`. If left out, `is_string` will default to `true`, making the inferred `type` (via `ConfigItem#data_type`) be `String`. That then triggers the "must be a String!" validation message.

`is_string: false` combined with not providing a value for `type` effectively skirts around the automatic data type validation, allowing boolean params to work.

Does that seem right, @andrewpage?


And do you know which nsz file by chance because there are a lot.


This is good and beautiful. But in applications that use the Electron execPath is not bin NodeJS (ref https://github.com/sindresorhus/get-grunt-tasks/blob/master/index.js#L14).

Project that uses electron when called get-grunt-taks it returns the entire log and not enters Then, the returns are not tasks, but the log (https://github.com/sindresorhus/get-grunt-tasks/blob/master/index.js#L18).

That's odd, perhaps ugly but it works -> https://github.com/gutierri/get-grunt-tasks/commit/cac9a8fe76ff8d7772e4ba4169414e5bf5b25ed8

What do you think?


I wonder if we can use "name" property of the control to group checkboxes. That way we can save the code of constructing and parsing control Ids. Multiple checkboxes can share the same name.


2245154-NewVRFHRModel (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)


why remove the semicolon?


I just pushed a fix. The unit test fails, anyway:
```
======================================================================
ERROR: test_SAT_getting_links (__main__.ESATsolverTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "./tests.py", line 249, in setUp
    self.po = ParseOptions(use_sat=True)
TypeError: __init__() got an unexpected keyword argument 'use_sat'

----------------------------------------------------------------------
```


@Jonahss was this fix ported to appium-ios-simulator?


Yes. Nearly everything is a list now. It turned out to be relatively common that fields which you would not expect to be lists (label, title, attribution) are given as lists. The API is pretty vague about this stuff.


The logic in this commit is broken.  https://github.com/atom/node-pathwatcher/issues/98


```
lib/exq/manager/server.ex:127: warning: clauses for the same def should be grouped together, def handle_call/3 was previously defined (lib/exq/manager/server.ex:82)

lib/exq/manager/server.ex:136: warning: clauses for the same def should be grouped together, def handle_cast/2 was previously defined (lib/exq/manager/server.ex:112)
```


"expected picker, slider, text or set" --> "expected picker, slider, text, set or boolean".


Here I notices that in the namespaces remains `foo.core$macros`


@vyskocilm Cool idea. Just wanted to give it a try. Unfortunately it can't find `fake_cpp`:

```
$ pwd
/Users/paddor/src/.../czmq/testapi
$ which cpp
/usr/bin/cpp
$ python ../../../../zproject/mkapi.py -I$HOME/src/python/pycparser/utils/fake_libc_include ../include/zsys.h
Traceback (most recent call last):
  File "../../../../zproject/mkapi.py", line 331, in <module>
    main()
  File "../../../../zproject/mkapi.py", line 321, in main
    decls = get_func_decls(args.header, args)
  File "../../../../zproject/mkapi.py", line 185, in get_func_decls
    cpp_args=cpp_args)
  File "/Library/Python/2.7/site-packages/pycparser/__init__.py", line 86, in parse_file
    text = preprocess_file(filename, cpp_path, cpp_args)
  File "/Library/Python/2.7/site-packages/pycparser/__init__.py", line 49, in preprocess_file
    ('Original error: %s' % e))
RuntimeError: Unable to invoke 'cpp'.  Make sure its path was passed correctly
Original error: [Errno 2] No such file or directory
```

Looks like in [mkapi](https://github.com/vyskocilm/mkapi) the file `fake_cpp` exists, but not in zproject. Or did I miss something?


Makes sense. Better to have them consistently be lists if that's a corner case we need to deal with.


Lol
I was from phone
But gave proper credits in commit message 😹😹😹😹


on the "type" values --- it seems picker, slider are control types, and text, boolean, set are value types. Maybe we can use control types for all of them? For example, picker, slider, textbox, checkbox (for both boolean and set)?


min and max must be supplied? If not raise exception?


```
lib/exq/redis/job_stat.ex:25: warning: variable error is unused
lib/exq/redis/job_queue.ex:81: warning: variable namespace is unused
lib/exq/redis/job_queue.ex:153: warning: variable schedule_queue is unused
lib/exq/manager/server.ex:97: warning: variable from is unused
lib/exq/manager/server.ex:102: warning: variable from is unused
lib/exq/manager/server.ex:107: warning: variable from is unused
lib/exq/manager/server.ex:128: warning: variable namespace is unused
lib/exq/manager/server.ex:193: warning: variable queue is unused
lib/exq/worker/server.ex:39: warning: variable func_or_empty is unused
lib/exq/worker/server.ex:41: warning: variable args is unused
```


still needs to be here?


:clap:


OK


Not sure the whether the final script will strip out all the leading spaces in the literal string between """? If so, the leading spaces makes the code more readable. Otherwise, we have to remove them?


You can't leak TTR, we will sue your ass


make sure env is a dict but not a string?


```
lib/exq/redis/job_stat.ex:9: warning: unused alias Job
lib/exq/redis/job_stat.ex:8: warning: unused alias Json
lib/exq/redis/job_stat.ex:10: warning: unused alias Process
lib/exq/manager/server.ex:4: warning: unused alias Stats
lib/exq/stats/server.ex:4: warning: unused alias Connection
lib/exq/stats/server.ex:7: warning: unused alias Json
```


@jhass Thank you! :bow:


Looks good


Does it throw?


The datasources from grafana (v2.5 & v2.6) is a list of dictionaries. This when statement never passes looking for a dictionary key of "name" in a list which is problematic when you are running this role more than once. The existence of the datasource is not detected and the Grafana API returns an (uninformative) 500 error.


Do we need to handle the case that model.cellElement is null?


@rafaelfranca done - Issue #22744


Sure, this one was necessary


so that's why my hydra keychord stopped working!


exacto trutho.


We should abstract this away into one or more 'active' providers.


Nice, I need to see how this all looks like in the built documentation


How about let's make this work for `RequireJS` and others so that we can merge our commits back to the main repo?

    var merge = typeof require === 'undefined' ? _.merge : require('lodash.merge');


This could be provided via the new /providers endpoint or via the /defaults endpoint. These endpoints would basically just be thin layers over the config parameters.


No, but remains in the known namespaces for no reason.


This is only used for test configuration. If they wanna change something they should just change that file. Is there a reason you wouldn't just handle it that way?


커밋 메세지가... 이게 뭐냠 ;;;


2245154-NewVRFHRModel (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1986 of 1986 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)


Octree (DeadParrot) - x86_64-MacOS-10.9-clang: OK (1981 of 1981 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)


The check is actually on [`*loaded*`](https://github.com/ScalaConsultants/replumb/blob/master/src/cljs/replumb/repl.cljs#L821) but the purge function does the cleaning of the AST as well...there is something weird then...maybe it does not clean because we don't convert to symbol somewhere?


读书报告全文+演示 PPT 提交


For test configuration, that's fine. I was cross threaded and didn't notice this was in the tests right away.


[Uploading prueba.txt…]()

prueba


saw approval from javajolt
at https://github.com/basho/riak_kv/commit/441e1b69d24a35dd5d4659bef8108da75c6e9a42


merging basho/riak_kv/feature/az/ddl-schema-export = 441e1b69 into borshop-integration-1312-feature/az/ddl-schema-export


basho/riak_kv/feature/az/ddl-schema-export = 441e1b69 merged ok, testing candidate = 0023c23b


For sure I'd like to do that. I only broke it out to get it working at 2am
and didn't spend much time making it work outside of my limited
environment.

On Monday, December 21, 2015, DaHoopster <notifications@github.com> wrote:

> How about let's make this work for RequireJS and others so that we can
> merge our commits back to the main repo?
>
> var merge = typeof require === 'undefined' ? _.merge : require('lodash.merge');
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ArthurThornton/factory-girl/commit/313c8e19b071fe1e4c8acf4176c992389d45b1f5#commitcomment-15109310>
> .
>


ちゃげ


Extract method and reuse below.


This leads to:

    Failed to load plugin from module "avocado_vt.plugins.vt_list": ImportError('No module named autotest.client.shared',)
    Failed to load plugin from module "avocado_vt.plugins.vt": ImportError('No module named autotest.client.shared',)
P

Which looks good


Yep, that's what I had before.  Then I saw the new deprecation warning and tried to update my code to remove the use of `is_string`.  Which is what brought me here. :)


Aye, this was fixed in a later commit, I'm pretty sure.


'Goal of replacing C' sounds like hubris -- 'Fills the same niche as C'


Hah, `baidu.com` is a Chinese search engine. It was bad (we all use Google actually) for searching but very good for network testing :P


I like the bridge :(


cool !


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


Yeah, I wasn't happy I removed it. I also like the whole sentence, but I don't want it on the front page.


no moar RedisToGo?


I wondered about that, but didn't check.


For whatever reason the link just doesn't work and constantly trips my CI, so I'd rather not have it.


Cool, if you could fix the case in your views that would be ace. You, Bec, Darren and Benoit are now using your postgresql ddb along with the toolbox so let me know when the case is fixed and I'll put the quotes back on.


I just wrote it freestyle, and also convinced myself that it works. I'm not sure if there is a standardized algorithm that it could be replaced with or if there's a faster way to do it.


Ok, will fix


nice!


never ignore the errors


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


Probably so!


[prueba.txt](https://github.com/reingart/pyafipws/files/69209/prueba.txt)
[test.zip](https://github.com/reingart/pyafipws/files/69210/test.zip)


AFAIK should just do the same as below: `store_ty` the result from `Invoke` to the destination.


why fail and return err?


Shadow si ves esto estás pendeja :v


Missed a "none" in the comment.


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


Couple of bugs:
Chapter title is null - Ch. #. What is null supposed to be? The title of the chapter or the manga?
Also, some issues with the seekbar and reading LTR vs RTL.


This was reverted to fix build breakage.


This was reverted to fix build breakage. Sorry for not including that in the commit msg.


It's a bit of a hack in order to grab arguments in the right order. I.e. imagine the following arguments map:

* href
* options
* callback

Where `href` and `callback` are required, but options is optional. I.e. one could call the it like `fn(href, callback)` or `fn(href, options, callback)`. The inversion allows it to alternate back and forth from taking arguments from left and right.


LGTM


2245154-NewVRFHRModel (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (912 of 912 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-25.87%25-green.svg)


A nice and compact recursive code that is fun to run with small samples. But the factorial takes too long after 9-10 elements.


Thanks to @ldionne for the cmake/ctest/travis code, borrowed from Boost.Hana.


Naming convention.


static import `Access`


The Rest URL was changed from locationdb to locations


interesting spelling!


Crashes here.


@glennc Can you review the X.509 section?


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2996


Suggest this instead:
```python
output_filename = re.sub(r'\W+', '_', f)
```
Also, suggest factoring that out into a function like `normalize_download_filenames`.


I'd have analyze this carefully in order to come up with something, but I guess I'll have to start, I'll try to build a slacko, but I have to unplug a hard drive in order to kill the older one, so basically this could take all my attention and I might up editing everything, so I need to sure that my time and effort will be worthwhile


TDB Reader does not work. Sample *.tdb: https://yadi.sk/d/a8bdjv6OmN4rG. Fix?


Good catch. Will UnknownHostException cause a crash or is it handled gracefully?


how can you solve `slow consumer` problem here?


Those headers can also moved into .m.


这个for循环无法退出


@alisan617, react-bootstrap-table does not support custom class on pagination buttons. But your feedback is a good improvements, I'll add it in the near future :)
Thanks


almost there. only some nits left.


I think this would simplify changing UIView's `originX`, `originY`, `width`, `height`.


s/"[vm_init] vm initialization failed..\n"/"vm_init(vm[%d]) is failed\n", i/.


Ditto.


This revision is clean.


Can you delete this one too?


s/"[vcpu_create] vcpu create failed...\n"/"[vcpu_create(vcpu[%d]] is failed...\n", i/.


Jovem, tudo muito bonito, compilou certo aqui, mas tu esqueceste-te de mudar lá no main ahahah (depois faz isso)
Proximas metas
- Tudo sobre o fogo (Exceto a ultima frase "No final do turno, o fogo tem 50% ...", isso eu já fiz) (pag.4)
- Tudo sobre brecha (pag.4)
- Ataque xenomorfo (pag.6)
- Campo de Pó cósmico (pag.6)
- Definir as unidades (como class) Geigermorfo, casulo, blob, Mxy(...) (pag, 7,8)
- As únicas características totalmente concluídas são, Respira, flamejante, robótico(?), reparador, combatente, tripulação, inimigo, move e armado, todas as outras estão por fazer. Algumas delas requerem atenção em vários pontos. É melhor fazer uma de cada vez do que tentar fazer todas ao mesmo tempo. (pag.8, 9)
- Definir as salas (como class) Auto-reparador, Sis. Seg. Interno, Alujamento capitao (pag.11, 12)


finally \o/


s/Modified memory mapping to run both VMs/Rollback memory description to boot Linux kernel on VM0/


I have more comments which I'll add later because I'm traveling right now and reviewing from my iPad


Looks like there is no need in this function anymore.


@armadsen thanks for the thorough explanation :+1:


got it.


There is no need to have multiples ids anymore


how about extracting that to a variable, e.g. like so

```ruby
  def repository_changes
    klass = @diff["repositories"].changed.first.first.class
    changed_elements("repositories", attributes: klass.attributes, key: klass.key)
  end
```


pull  2015/12/22


Strange why do not the try-catch block not catch the error?


Kappa123 NeXoness wasn't here


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1044](http://teamcity.monogame.net/viewLog.html?buildId=12831&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 555, ignored: 6 Build time: 00:30:54


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1045](http://teamcity.monogame.net/viewLog.html?buildId=12835&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


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


Works perfect now!


@jeremyeder fixed,was testing .txt and json at same time, abonded .txt and we will use just json for collecting oc outputs


`--prefix .cef.` is just a syntactic sugar for `--key .cef.* --rekey .cef.* --shift 5` (which formats only the keys starting with `.cef.` and truncates the `.cef.` prefix from them)

+1 for the grammar based vp. It was forgotten.


Can't you send `data['geometry'] = null` instead of this keyword?


@bertrandkeller il manque le fichier correspondant dans le commit :open_mouth:


We had a typo?


@rogatty
Can these three be somehow simplified? Something like Ember.computed with parameter from template.


`ORDER BY created DESC`


不错不错


No ideas.
It trigger when I import this script in another one.
```js
import FetchInterceptor from './fetch-intercept';
// or
var FetchInterceptor = require('./fetch-intercept');
```
It doesn't require whatwg-fetch according to the condition(try-catch doesn't work either).
Stangely.


Is it possible to omit the hard-code using?


No it used to be aliased, it's not anymore


Why are numbers being translated using qsTr? ```text: "120"``` works just as well.


Looks like the platform does not show unit code to user.


> TypeError: __init__() got an unexpected keyword argument 'use_sat'

Strange. Maybe the Python SWIG stuff somehow didn't get recompiled?
Please try to recompile from scratch...


Typo: `MS-OXCICAL`


Remove this


And this


Looks like a tricky fix - many thanks for figuring that one out!


Should Go to be used if Reload page is there?


So, I've thought that it could because of the compile.


What about 'return res.send...'?


So everything is working now ?


Got it


percona-server is HEAD. it will be 5.8 after some time, so better to add a version


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


this may caused simcard recognize error, I don't know why but on my phone it's don't recognize simcard/modem version if i delete this file


Following insane coding style here. The C-functions has an space before `(`


We can probably drop libstlport too. Looks like only libmmcamera_hdr_gb_lib.so needs it but on M soak tests there isn't libstlport (at least on osprey).


Test


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


Thank you!


The abovementioned comment applies here.


And here.


You can remove all of the default filters with `user_es = UserES().remove_default_filters()` - that should work as expected.  Are you sure you don't want the `"not_deleted"` filter in this case, though?  That asserts that the `"base_doc" == "CouchUser"`.
I think you ran into an edge case I hadn't accounted for, by removing all default filters and not passing any new ones.  This PR should address that issue: https://github.com/dimagi/commcare-hq/pull/9690
Let me know if that doesn't sound like what happened here, I'd like to figure it out.


Should there be an `else` for that if statement, to return something?


Une constante, comme son nom l'indique ... est constante. Pourtant rep prend une nouvelle valeur 3 lignes plus bas. Ne serait ce pas ce qu'on pourrait appeler ... une variable ??


Ok I haven't any experience with react-native. I added this try-catch-block so it is less error prone to hook into `fetch`. A compromise would be to add a flag to attach that can make the try-catch-block optional. Would this be a solution?


Why don't you initialise `tz` to `nil` to avoid repeating code here?


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


I now get this error when running `./node_modules/.bin/gulp`. It seems that doesn’t load DBDefs in any way; it’s only `script/compile_resources.sh` that runs `admin/ShowDBDefs` before calling gulp.

So, I guess we should change `INSTALL.md` and tell people to run that compile script, not gulp directly, right?


the same, better to add the version and do not point to master HEAD


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


Don't waste your time - they are invalid


how is the linked diff related to EAC3?


Hi @ralphdolmans , thanks for the PR. Are you sure that including that headerfile (`<openssl/e_os.h>`) is necessary ? Which Operating System are you using ?
As far as I know a `make depend` should take care of the correct includes (including that one).


Why do you use return here?


do we have compat package for debian? we're going to replace every mysql version or libs won't be replaced?


If the objects inside a div with .shrink class are wider than the browser on iOS devices, the responsive layout got broken.
With a max-with of 100% this bug is getting fixed.


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


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


@pb00067: Very nice work. Looks like distinct_iter_11 is the most promising. I think you should schedule:
* LTC = 30+0.3 (half the usual time for SMP tests) using 15 threads.
* If it passes, run a couple of extra tests to verify that there is no regression on 3 and 7 threads: SPRT(-5,0).
Makes sense ?


The assignment itself is not truthy. Value of the assignment expression is whatever was on the right side of it.

```ruby
>> (x = 5)
=> 5
>> (x = true)
=> true
>> (x = false)
=> false
```

This is in fact the thing that allows you to assign to multiple variables in one line:

```ruby
x = y = 42
# is the same as:
x = (y = 42)
```


Seems Gmail also does the same as GitHub.


Good find. Derp by me. It's fixed.


There is also `script/compile_resources.pl` which only calls the `.sh` …


Вчера обсуждали в чате, ссылка и ее название должно совпадать http://second.by


reack


@geek


> when socket is closed and trying to read from it, the returned result is not an error but an 0 sized buffer.

This is true, but I believe this is an invalid behavior on `rust-utp`s side. That is, the library should return an error and not a buffer of size 0. Otherwise it would be impossible to distinguish between "error occurred" and "remote sent us a datagram of size 0".


4.0 and 4.1 are very old MySQL versions


I would use `Ember.computed.equal` and move class names to the template.
```
isFlagged: Ember.computed.equal('model.status', 'flagged')

<button class="button-group-item {{#if isFlagged 'highlighted' 'inactive'}}"
```

What do you think?


```
          if (isRaw && info.m_wantsIECPassthrough)
          {
            switch (format.m_streamInfo.m_type)
            {
              case CAEStreamInfo::STREAM_TYPE_EAC3:
                if (format.m_streamInfo.m_sampleRate)
                  samplerate = 4 * format.m_streamInfo.m_sampleRate;
                else
                  samplerate = 192000;
                break;
```


ack, assuming you grepped :)


how to run this edmodo in virtualenv.. what is django version and python version


tested and LGTM


aospディレクトリ作成して、その中で、cm-13．0ブランチと同じ構造作ろうと思います。
　＃java sorceはsymlinkでやってみようかと
そうする事で直接googleの変更マージでき、
変更があった時だけ、ビルドした.soを突っ込むという運用でいけるのではないかな？と

名前ですが、androidだと元のツリーにもある名前で混乱しそうですし、
かといってcmとするのも変だし、googleっぽくaospという名前にしてみました。


final?


You're missing a closing quote here :)


Although I would use SLF4J in an application, I kind of feel that adding an external logging API dependency to this small library is not required. The logging is purely of a debugging nature and is only on application start-up. SLF4J brings no real benefit in this case.

-1


Do you have any argumentation? Cause I've just tested again (sending an email with this gif to erikvandeven100@gmail.com) and it just works perfectly fine. It counted down till `9 12 25 52` (last frame before the "it's almost new year"). I re-opened my mail and it started counting down from `9 12 25 45`


Mistake, should leave synchronization


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1045](http://teamcity.monogame.net/viewLog.html?buildId=12835&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 555, ignored: 6 Build time: 00:30:55


Переходить по успешной ссылке имеет смысл только после успешного входа


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 125](https://teamcity.labkey.org/viewLog.html?buildId=292798&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:02:26


あっ、resouceもsymlinkですね


I understood that from code (the intent should have been documented in the
commit msg though) but I think the name prefix doesnt really describe what
it does. What about "subtree"?
On Dec 22, 2015 11:07 AM, "Tibor Benke" <notifications@github.com> wrote:

> --prefix .cef. is just a syntactic sugar for --key .cef.* --rekey .cef.*
> --shift 5 (which formats only the keys starting with .cef. and truncates
> the .cef. prefix from them)
>
> +1 for the grammar based vp. It was forgotten.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/balabit/syslog-ng/commit/055c204a43264f19fd8b897408eb09f76d29bdd0#commitcomment-15115076>
> .
>


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


My mistake, mixed 2 issues: http://pastebin.com/k76i3NmB


@AnriShitko


In what case you get a WKT?


ack


The reformatting seems to be caused by a setting on IntelliJ which I have manually switched off, and encouraged my colleagues to do the same. This is distinct from the reformatting rules that the style formatting settings impose in the case of Javadoc, and apparently is not checked in.  Preferences, General, Strip trailing spaces on Save (I've set it to "None").


While running tests through Travis CI (where the build passing comes from) I set it up to do 6, 7 and 8. Removed 6 due to an issue, but 7 and 8 worked, so maybe make it Java 7? (by the way, not using Eclipse, using NetBeans IDE)


Isn't `null` ambigous? Shouldn't it mean remove the geometry?


how I would do
https://github.com/acdlite/flux-standard-action

```JS
const SET_REGIONS = 'SET_REGIONS';
const SET_DOMAINS = 'SET_DOMAINS';

export function initRegions() {
  return dispatch => {
    getRegions((err, regions) => {
      dispatch({
        type: types.SET_REGIONS,
        payload: {
          regions,
        },
      });
      dispatch(setRegion(regions[0]));
    });
  };
}

export function setRegion(region) {
  return dispatch => {
    dispatch({
      type: SET_SCOPE_REGION,
      payload: {
        region,
      }
    });

    // then get domains and dispatch the action asynchronously
    getDomains(region, domains => {
      dispatch({
        type: SET_SCOPE_DOMAINS,
        payload: {
          domains,
        },
      });

      dispatch(setDomain(domains[0]));
    });
  }
}

export function setDomain(domain) {
  return (dispatch, getState) => {
    dispatch({
      type: SET_SCOPE_DOMAIN,
      payload: {
        domain,
      },
    });

    getWorkflowExecutions(region, domain, workflowExecutions => {
      dispatch({
        type: SET_SCOPE_WORKFLOW_EXECUTIONS,
        payload: {
          workflowExecutions,
        },
      });
    });
  };
}

const initialState = {
  regions: null,
  scope: null,
};

export default function context(state = initialState, { type, payload, meta }) {
  switch (type) {
    case SET_REGIONS:
      return  {
        ...state,
        regions: payload.regions,
      };

    case SET_SCOPE_REGION:
      return  {
        ...state,
        scope: {
          region: payload.region,
        },
      };

    case SET_SCOPE_DOMAINS:
      return  {
        ...state,
        scope: {
          ...state.scope,
          domains: payload.domains,
        },
      };

    case SET_SCOPE_DOMAIN:
      return  {
        ...state,
        scope: {
          ...state.scope,
          domain: payload.domain,
        },
      };

    case SET_SCOPE_WORKFLOW_EXECUTIONS:
      return  {
        ...state,
        scope: {
          ...state.scope,
          workflowExecutions: payload.workflowExecutions,
        },
      };
  }
}
```

Then either in your root component or in main.js,
you call dispatch(initRegions());

In app/components/WorkflowExecutions.js
You can test if context.scope.workflowExecutions is defined (you can you lodash.get to test deepply safely)

In app/components/ContextSelector.js
```JS
render() {
  return (
    <div className="ContextSelector">
      <div className="box">

        <div className="label">Region</div>
        {this.props.regions &&
          <select value="{this.props.region}" onChange={this.handleChangeRegion.bind(this)}>
            {this.state.regions.map(region =>
              <option value={region} key={region}>{region}</option>
            )}
          </select>
        }
        {!this.props.regions && <i class="fa fa-circle-o-notch fa-spin"></i>}

        <div className="label">Domain</div>
        {this.props.domains &&
          <select value="{this.props.domain}" onChange={this.handleChangeDomain.bind(this)}>
            {this.props.domains.map(domain =>
              <option value={domain} key={domain}>{domain}</option>
            )}
          </select>
        }
        {!this.props.domains && <i class="fa fa-circle-o-notch fa-spin"></i>}
      </div>
    </div>
  )
}
```


`git finish` didn't; so I've done this last by hand.


This should fix the bug named in #307, however before I close this issue (and upload the new source code to GitHub) I first need to make sure that this issue is indeed fixed (my builder made this commit automatically).

Immediately after that I will prepare a new download and I hope that it will work then. ;)


is it a patch from Fedora or from Oracle?


Some WKT were in use in the tests.


Oh, I've found out that React-Native has a bundle method which works like webpack.
I think could it because that the bundle method recognise this `require('whatwg-fetch')` as requiring package. So, it tried to load 'whatwg-fetch' in compile.
I think it is caused by the keyword 'require'(no matter try-catch used or not).
It seemed to be a conflict between React-Native and other environments.
Unfortunately, a flag would not be able to solve this problem.


-1, unless you plan to follow this at the same time with the release of a `bitpay` package that replaces previous functionality.  Strongly recommend a [private NPM](https://www.npmjs.com/private-modules) if you are having internal name conflicts.


success = это строка, данное условие, практически невыполнимо, т.к. сайт очень редко отвечает пустой строкой.

Получается, что на этом сайте нет информации о неправильном логине, блок проверки успешного входа должен быть таким:
// попадать в блок проверки надо при любой неожиданной ситуации
if (success != 'true') {
    if(success === 'false')
        throw new AnyBalance.Error('Неверный логин или пароль', null, true);
    AnyBalance.trace(html);
    throw new AnyBalance.Error('Не удалось зайти в личный кабинет. Сайт изменен?');
}


@AnryShitko


`ClusterNotFoundException` is actually a valid error which shouldn't be ignored, the real cause is that we're not passing `Cluster` attribute to `DescribeServices` few lines above - hence it uses the default cluster (named `default` I think), which may not exist.

Besides that, there's actually more to be fixed in ECS service deletion.

I'm fixing all the things I know about already in https://github.com/TimeIncOSS/terraform/commit/86269da86a89e0131834bbda5512795b16078218#diff-541cd13c0077eb59a7e6688908a5cc8eR217


unack, no test for the warning anymore?


yes, this looks good


repo too :)


As far as I know you cannot use #if helpers inside " ".


ah below, ok


@stieglma This is a bug, not a feature.
Default implementation is (at least conceptually) a private API, and should not be used/tested directly.
If native implementations exist for all solvers than the default implementation is irrelevant (though they don't)


I changed the format to include the description of the error (```self.desc```) when a detail was present as well. The reason for the commit was that I was very baffled when my code panicked with a ```retrieving column created_by: created_by (23)``` which, without the description of the error, was pretty cryptic to me.


'two' isn't necessary to be grammatically correct :/  AND it's one more word that has to be changed when we get to 3, 4, 5 etc servers!


this would be useful for other occasions too, though you should mention that you don't only close it but also remove it


I would say `data['geometry']['geom']` means use an empty geometry (because you are also sending back the version of the current geometry). And `data['geometry'] = null` would mean skip.


@anandhak Why should we remove `counter_cache` in the gem ? I am wondering if it be possible to override this in the codebase instead, so that we can get back to using the official gem since your [PR](https://github.com/medihack/make_flaggable/pull/24/files) got merged.


Sorry, there is no `#`. See https://guides.emberjs.com/v2.2.0/templates/conditionals/


Maybe you can leave a comment saying that it is only for the tests?


> This line became useless, it should be removed.

Submit a PR!


Is this backwards? I would have thought that doing .get() blindly would cause the exception, and would only be appropriate if the plugin absolutely could not function without that data?


название папки = id = ab-game-bananawars
Формат идентификатора "ab-<group>-<name>"


An overall comment since this is a new setup: request you to add class level and method level comments in all files


Clever, works! Thanks


```bool Beam::skeleton``` => ```bool Beam::m_is_skeletonview_active``` please.


@rjernst why do we have these checks here? Can't a module contain packaging files? If it needs them, where should they be placed? These questions come from ingest, I am looking into making it a module in the feature/ingest module.


Иконка старая из копипаста, а д.б. от бананы с соблюдением требований, которые описаны в документации, https://github.com/dukei/any-balance-providers/wiki/Icon


I don't get it, both of these commits are in `1.2.1` release, why not just use that?


I think it is still good to test default implementations if we have code for them.

Otherwise one might end up adding a new solver without internal tactics, the default implementation gets used for the first time, and the test fails, and now one does not know whether the default implementation or the solver package is broken.

Whether something is public API or not is not relevant for the question whether it should be tested.


@ertrzyiks orientujesz sie czy ja musze jakos dodatkowo dodawać assety na heroku? Bo po przegraniu na produkcjie nie wystwietla background-image poprawnie, pewnie ścieżke źle ustawiłem.


@ccd0 Either configuration breaks specific environments. Make sure that your APP_URL .env item is set correctly and without a trailing forward slash. If it is set correctly, I think you should just edit the config file locally. I'm not sure what I can do to satisfy all environments.


Why is this here?  _assemblies is only referenced here.


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 76](https://teamcity.labkey.org/viewLog.html?buildId=292761&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 32 (16 new), passed: 259, muted: 2 Build time: 01:33:40
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


res.send should never be returned. But i removed it because the error is now thrown by getAuth


Why do I need this file in this repo? This repo is a module that will be used inside a framework (silverstripe).


Он-лайн игра BananaWars


It will crash if no `--mucpass` given.


Сейчас так допустимо. Но сотрудники компании указывают копирайт компании


Hey I tested this on my klte and in cpuz in the sensors tab when you place your finger on hr sensor it turns on and after a few seconds it gives the accurate reading or heart rate. So I think its working fine atleast in apps that support this sensor.


Hey, hey, hey, have you changed your password yet?


Nice catch, thanks


hmm I think you're missing an entry in xwiki.extension.features though


It also works in Phoneinfo app.


@barryvdh now when error occurred (ex. ModelNotFoundException) there is no CORS headers in response. May be we need `->addActualRequestHeaders()` before `$next($request)`?


Why do we hard code icon image in App Deck?


need a newline here -- grab the editorconfig extension to have these things sorted out automatically


When I access this via the IP address `172.17.0.3` it creates a bucket named `172`.


not yet...


@alex-ab does this comment reflect the current state?


Nit: I found this construction a bit subtle. I find side-effecting iterators can be a bit tricky. Maybe rewrite it to extract the index first (in a pure fashion) and then do the side-effects afterwards, something like:

```rust
let index = match self.scopes.iter_mut().enumerate().rev().find(|(_, s)| s.extent == extent) {
    Some((index, _)) => index,
    None => error;
};
// add the drop to the scope in question
self.scopes[index].drops.push(...);
// invalidate any cached drops from this scope and all scopes pushed after it
for scope in &mut self.scopes[index..] {
    scope.cached_block = None;
}
```


about the commit message longer explanation, please explain why instead of what. The what is obvious from the changes :) something like "move upstream default configuration file into seperate config package to allow easy overrides when using this recipe in a project"


Escape '\'?


I really hope that #307 is fixed now as this is getting silly.


I just opened a WIP PR for it, probably better to have this discussion there: #15601


probably best to explain if the .local file is enough to actually configure bind, or downstream users would need to rename that file


Pre-existing, but I think we ought to be iterating this array in *reverse* order!


说好的预览地址呢？:flushed:


Do you mean getting `A` from function/property without specified return type?
e.g.
```kotlin
package foo

fun getA() = A()
```

implicit.kt
```kotlin
package foo

fun getAImplicit() = getA()
```


Add tests to ensure your escaper works?


OK! But I would extract it to a wrap function.


ok..i alredy change my password...


When I first read this, I thought that it was generating the drops in the wrong order. In fact, I now think it is correct, but a comment might help. Something like:

```
// Given a scope array like [S0, S1, S2], generates corresponding cleanup blocks B0, B1, B2 setup like:
//     [resume] <- B0 <- B1 <- B2
// Note that we generate the outermost cleanups first and then iterate down to the inner ones.
```


Initial commit -  Diskover code


Nit: I personally would remove this extra check. The loop below that populates the cached blocks seems fine, and then we can just remove the call to `expect()` below.


ForgeMultipart is not available in 1.8...  Also 1.8 dependencies "may" no work due to the rework of WorldRenderer (Tesselator in 1.7.10). Only CodeChickenLib have been ported in 1.8.8 (not actually available in the MinecraftForge's Maven repository)


Oh sorry, didn't notice set arena(null) did that.


Yes makes sense. you definitely should


That is, just change this to `self.scopes.iter().rev().flat_map(|b| b.cached_block).next()`, maybe with a comment like:

```
// Return the innermost cached block, if any. Note that if there are no cleanups in scope we return None
```


but first test it out. are you sure this counter cache was removed for that reason. also you will need extra columns in db so a migration if you are doing this.


I intended reusing the existing icon set in AppDeck instead of duplicating them in SmartSystem.


So the (only) purpose of `commitLock` is just to prevent multiple commits being run at the same time, right?  It shouldn't cause any locking contention with `purgeCache` (that is supposed to be handled by `purgeLock`?

Aside: why is purgeCache still synchronized?  Is this just to prevent multiple purges happening at the same time?


Signed-off-by: YanKunLi <845245370@qq.com>


I'll add the new line.
Any idea of how I could change `'./1.4.1-2'` and the `lib/adapters/typescript/1.4.1-2.coffee` file name to be able to support any version instead of being so specific?
Tried a few variations of `./1.x.x` with no luck, I'm probably overlooking something simple...


outputText value name is repeated ...


Обработки ошибок практически нет.
Где обработка 50x ошибок сервера?
где обработка неверного логина/пароля?

Логика работы здесь пошаговая и легче читать сверху вниз как газету.
Разбивание на несколько функций только усложняет понимание, т.к. нужно "собирать кубики в голове".
См. очень внимательно,  как сделано /extra/development/ab-stark-typical-provider
Нужно сделать примерно также.


this is not a typo, it's better to just flush changes on Job entity here


This is clearer to read, IMO


update


I removed this assert, as it is already handled in the next line of code (it throws an InvalidOperationException).
This means that this was an improper use of Debug.Assert, as this is actually a situation that is possible to generate in Release


Correct me if I'm wrong, but I don't think there is a way that a node's key can be null.
If I am wrong, can you show me an example of how this can arise, so that I can add a test for it.


See above.


Fixes #211 as well.


This makes the exception message more specific and harmonizes it with other exception messages in the libary


Correct me if I'm wrong, but I don't think there is a way that dictionary can be null.
If I am wrong, can you show me an example of how this can arise, so that I can add a test for it.


@chelmuth: most the time - typically i update r9_debug as soon as the referenced commit becomes used by genode master


@rochoa I've got this in the refactor PR, not sure if that's what you're referring to: https://github.com/CartoDB/d3.cartodb/blob/clean-render/src/renderer.js#L152-L171


Did you mean to remove the doctype, you need for well-formed HTML AFAIK.


"backticks" - Hugo Giraudel.


こちらの対応漏れですね。すみません。再度登録します。


This is in the wrong place, I'll move it when I add all your PR comments.


Signed-off-by:YanKunLi <845245370@qq.com>


Yeah you can make it `1.x` and it should work just fine. This is the case for almost every other adapter, you can jump into the folders and see. Any range that is compatible with semver will work :grinning:


Classnames have been removed. :D


ini header sama implementasinya dimana ya??


I guess you could leave one of them completely empty, to ensure it works without `{}` ?


nevermind, just saw the dedicated test below :)


mysql-5.7-libmysqlclient-symbols.patch was from Oracle for symbols, but it is removed now since it's not used for v20 of library.
mysql-5.7-sharedlib-rename.patch is our from 5.6 centos7 ported to 5.7 for library rename (there's no upstream patches for this since they didn't rename their library).


I'll remove this double newline


Yuck. :) So, rather than all this manual type construction, you could get the `DefId` for the lang item and then call `tcx.lookup_item_type`. You can then use the `fn_args` and `fn_ret` method to extract out the argument and return types.


Ill add a newline here


You need to use the same version of Roxygen that I do, otherwise we're going to have these spurious changes inserted by roxygen all the time.

I haven't mentioned this yet, since I didn't want to complicate things thus far, but you have to do all your development for this project using R-3.1.x to conform to AzureML and the Jupyter service.

You also have to install your packages from a snapshot date.

I use the following `.Rprofile`:

```
Sys.setenv(tz="UTC")
options(repos = c(CRAN = "https://mran.revolutionanalytics.com/snapshot/2015-06-20"))
.libPaths("~/R/win-library/3.1-mran-2015-06-20")
message("Using library: ", .libPaths()[1])
```


@dstd Yes, you were right. I was a bit naive with my implementation.

BTW, with version 0.66, I take your concern very seriously and implemented a way to override the defaults by publicize 2 variables;

`Manager.instance.socketFactory` and `Manager.instance.hostnameVerifier`

https://github.com/kittinunf/Fuel/blob/master/fuel/src/main/kotlin/com/github/kittinunf/fuel/core/Manager.kt#L27

Gotta say thank you to you. I am appreciated it and it just makes this lib a little better :+1:


Great job !


This code could change the type of object.  You should detect the class of object (GenericCSV, GenericTSV, ARFF, etc.) and honour this.


while we're at it - could you remove `newly formed`? the foundation isn't THAT young anymore


Merged here.


So what trans does is to create a constant tuple. That's...perhaps a bit inconvenient here in MIR land, but it does seem like the right approach. The EASIEST thing to do would be to just take three arguments -- I suspect the reason we do not is because this fn has a lot of calls, and that'd be more IR than using a constant. But what you're doing here, where you create a temporary tuple, is also going to be wasted IR.


The function should honour existing description and family_od


Эту строку можно удалить


Maybe we should just add a special "filename-linenumber" kind of constant that trans can understand for now? Not sure what's best here.


I meant about the changes you are doing here. About the "corrected_x", you are actually wrapping it, right?


I read the program file that you guys are uploading dummy dummy dummy ??  Nice and very professional you're all done I want a new organization and I will be a part of it learning and supervising to make sure that no one  abuses my trust again .


建议把
if plan.keyIndex == -1 {
 +		return nil, errors.ErrIRNoShardingKey
 +	}
这部分逻辑放在generateKeyIndex中，更符合Go语言的写法。


This check should be carried out before communicating to the webservice.  Use `datasets(ws, filter = "my datasets")` to see if the dataset could possibly be deleted.


ありがとうございます！！この流れでいいと思います！
ゲーム画面のデザインはこれから優先してつくっていきますね。
それに加えて、ゲーム画面だけじゃなくて、
・ゲームの遊び方の説明ページ(①の解決になる、、！？)
・感想、お問い合わせページ
・Q&Aページ
を作ろうと考えています。いくつかのページをつくった方が、よりwebページっぽくなると思ったので時間があれば進めていく予定です。

現段階で気になる点は、
・ランキング表に名前が入らない
・過去の記録も含めて順位付けをしたい
なので、なほさんがもし余裕があればお願いできますか、、?


一つ質問したいことがあり、
プルリクエストを送らせていただきました。

api_server.pyの92行目のreturn json.dumps(user_data)という処理で，
MongoDBから、いま現在挿入したドキュメントを取得して、
ブラウザ側に返答する処理をしています。

しかし、TypeError: ObjectId('567930fd7168c74c23b15a0a') is not JSON serializable
というエラーが出ました。
ObjectId('')という形式によりjson.dumpsできないという状況です。
この型を整形しようとして、ObjectIdを削除してjson.dumpsするという方法が見つかりましたが、
ObjectId('')型の整形については分からない状況です。
ObjectIdは正誤判定後にドキュメントをアップデートする際に必要になってくるので、ブラウザに返したいと考えています。

この対処方法について、見ていただけると幸いです。
宜しくお願いします。


Too awful, need revert.


Who let the sponge programmer in? Optionals die.


END_PORTAL_FRAM should be END_PORTAL_FRAME.


What are you using that for, @bitboxer ?


issue #49


Yes, right. Thanks.


好的，那我重提


net


good day, you still working in this project?


This is a somewhat new way of typing the name of our capital city, typically we tend to add more l-chars... But angry bald guys is a new variant I have not seen before.


You've linked to the old docs.


The helper is removed because it was a stupid idea imo. Feel free to add it back in your own app if you really want it.


Since game validation will be done later, this Fixes #25


The following call is missing here `type(BanTypes.PROFILE)`, causing and ISException.

See the default implementation in Common: [SpongeBanBuilder](https://github.com/SpongePowered/SpongeCommon/blob/master/src/main/java/org/spongepowered/common/ban/SpongeBanBuilder.java#L60)
````java
@Override
public Ban.Builder profile(org.spongepowered.api.profile.GameProfile profile) {
    checkNotNull(profile, "Profile cannot be null!");
    checkState(this.banType == BanTypes.PROFILE, "Cannot set a GameProfile if the BanType is not BanTypes.PROFILE!");
    this.profile = profile;
    return this;
}
````


Тут пусто, а должна быть история (первая версия), см. как сделано у других провайдерах


Aw sorry, I parsed the word "wrap" differently :) Yes, I'll pass that to a wrap function in `geo`


Okay, I'm I right to assume that `r9` will be forwarded to `r9_ext` then?


> The EASIEST thing to do would be to just take three arguments -- I suspect the reason we do not is because this fn has a lot of calls, and that'd be more IR than using a constant.

It was explained to me we take the tuple, because that used to generate smaller binaries with the old trans. I suspect things might be different in the MIR trans, but its not like I have an option of changing the prototype of language item now.

> But what you're doing here, where you create a temporary tuple, is also going to be wasted IR.

Why? The MIR (as it looks in graphviz) looks pretty compact; more compact than calling equivalent function that has been built by MIR itself.

> Maybe we should just add a special "filename-linenumber" kind of constant that trans can understand for now?

Note that while panic_bound_check needs/wants a &(filename, linenumber), regular panic wants a &(message, filename, linenumber), so that would not be composable, I think?


I think we should discuss this. Why was it reverted? Was it causing bugs somewhere else? If so, I believe it should have been fixed there, because:

1. Treating `Ok(0, _)` as an error goes against the `Error` semantics.
2. Treating `Ok(0, _)` as an error is not a standard for [any read operation](https://doc.rust-lang.org/stable/std/io/trait.Read.html#method.read)
3. Treating `Ok(0, _)` as an error makes it impossible for the remote peer to send us a packet with zero size payload.


I would prefer to not log special symbols. They also appear in console and could be confusing


Nice!


En fait j'ai fait quelques erreur dans ma conception.
J'ai commencé par les méthode des classes UV et graph. Et ensuite Jeu. Mais du coup je tente d'appeler des méthode sur des objet dont j'ai pas de référence.

De plus j'ai essayé d'appeler les méthode de Jeu en tant que classe et non en tant qu'objet instancier danas le main. D'ou l'erreur de non static method.

Je modifie ça dans l'après midi et je push ce soir


Will do


Added Comments


This worked out very nicely. Will be reflected in my next pull request.


@GrahamCampbell Do you think it's worth adding into the upgrade docs?


Same here


Yes. As soon as the commit becomes genode master I rename it to r9


ok


I don't know, it's also strange. Maybe @faxm0dem can help us to choose the right name from a user's perspective :)


Me desculpe edy eu cheguei a fazer o commit mas  os arquivos não foram enviados para o git abaixo esta o link dos arquivos da prova vou tentar se vc me permitir vou fazer novamente o commit

https://ide.c9.io/maximilam/car-parking-exam

Date: Mon, 21 Dec 2015 18:45:49 -0800
From: notifications@github.com
To: car-parking-exam@noreply.github.com
CC: maximilam_rdm@hotmail.com
Subject: Re: [car-parking-exam] first commit (e36d762)

Olá @maximilam, por favor, habilite a adição de issues no seu repositório.

Não encontrei a implementação da sua avaliação!

Abraços,

Edy Segura

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


Yes. I've already raised this issue with Taylor in private. We'll get the docs updated within 48 hours.


Shouldn't 112 be enough ?


@CGenie Thanks for this I am not into docker yet I will be soon. However I take your word for it is correct.


commentsection


@GrahamCampbell Awesome, nice work. Happy to make a PR if needed.


It's rather `JobClassesRegistry`
Generally I'm not sure it was really necessary to change names from 'Handler' to 'Job'.


True.


inserire altre stanze?


That would work also.


@igalic Using `|` where?


yes but dont call the parameter mirror, it will get confusing


Do we need this intermediary method?


Just remove the puts, leave the rest. You can tell if it runs based on the output already. Look at your tests.


should we add support here for FreeBSD etc. as well?


No hay que confundir _Apache OpenOffice_ con _OpenOffice.org_: el proyecto que alberga la fundación Apache no incluye el «.org» de su proyecto antecesor en su nombre.


Optionals are bae, shoo @nickrobson


Since jobs are created within manager it's better to pass `tmpDir` directly to abstract job's constructor.


Instead of `;`?


No.
We need to verify it switches to https, otherwise you aren't testing https.


fddfg


>>・ゲームの遊び方の説明ページ(①の解決になる、、！？)
>>・感想、お問い合わせページ
>>・Q&Aページ
了解です！ゲームの遊び方のところは，「ゲームの遊び方」みたいなのをクリックしたら
飛べるようなボタンを名前とか入力するところに入れておきたいよね！
手伝うことあればいってくださーい！

>>・ランキング表に名前が入らない
⇒たぶん上のステップ通りにやっていくと，名前も表に入るはず！
>>・過去の記録も含めて順位付けをしたい
⇒点数によるソートもテーブル更新時に行うようにするね！
あと，プレイヤーの名前と，スコアをプレイヤーの絵の横に表示できるようにしておくね～！！


my comment


Вы код вообще запускали в отладчике, Хром расширение ставили? :)
AB.parseInt() нет такого метода, есть AB.parseBalance()


Dùng github có nhiều cái thú vị không anh, chủ yếu cho mục đích gì vậy, Vốn
tiếng anh của em hơi kém

2015-12-21 20:40 GMT+07:00 Kang8M <notifications@github.com>:

> minh hoc khoa 16 ban.
>
> 2015-12-16 20:43 GMT+07:00 khongphaiha <notifications@github.com>:
>
> > Cho hoi anh hoc khóa nào HUBT vậy, Lớp nào ajk, Em cũng học HUBT K17
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/JetBrains/swot/commit/68a2388164f26c282b35d7f07d1c25567acc0b82#commitcomment-15014060
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JetBrains/swot/commit/68a2388164f26c282b35d7f07d1c25567acc0b82#commitcomment-15096478>
> .
>


for (var i = 1, len = this.arguments.length; i < len; i++)


a


worth to make a reusable constant for KBps?


there is no tooltipType in the hourlyNetworkUsage hash - why?


why is cpu handling part of a network patch?


Use Doc type comments.


Да. Проверял ещё и на мобильнике который удалось достать

вт, 22 дек. 2015 г. в 14:39, Rinat Mukhatrov <notifications@github.com>:

> Вы код вообще запускали в отладчике, Хром расширение ставили? :)
> AB.parseInt() нет такого метода, есть AB.parseBalance()
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/astrostellar/ab-game-bananawars/commit/191ade2566f09c39cf207ea6baa8ffbf615111f9#commitcomment-15116738>
> .
>


D'ailleurs un truc me chiffone avec Joueur.
Les différents joueur connaisse Jeu mais ils ne se connaisse pas entre eux et personne d'autre ne les connais.

Du coup en fin de tour pas possible de changer de joueur faisant son tour.
Et pendant les tours non plus pour déplacer les autres ou bien donner des cartes etc etc


Небольшое замечание. Если в таблице будет таблица, то код может перестать работать :)
У нас есть функция getElement()


[This](https://docs.npmjs.com/getting-started/semantic-versioning) video is better than a text document but has a lot of npm specific parts in it. Can we use it?.


I'd do: `rv[[j]] <- as.numeric(rv[[j]])` instead. I'm always uneasy about direct assignment to class. Check this out for instance:

    l <- 1:10
    storage.mode(as.numeric(l))
    mode(as.numeric(l))
    class(l) <- 'numeric'
    storage.mode(l)
    mode(l)


Fails to compile with:
```
src/core/socket.c: In function 'socket_done':
src/core/socket.c:168:19: error: void value not ignored as it ought to be
         s->fdname = free(s->fdname);
```


Version  2.0.1 is not working. I am getting the following error
ReferenceError: e is not defined
lastCode = e.data.result[0].Value;


Well. Usually `|` is "or". No reason to keep the bitwise meanings


you mean "market" module, "marketing" module should still be there


This keeps changes to admins.txt from being commited.


Oh, so like `foo | bar | baz`? It's probably a better idea, but I have to change the arglist separator first, and I don't know what to use to replate `{ a b | a + b }` :(


I don't know if the slashes are needed or not, but I have seen it written both ways.


@rarkins This is now available in 0.7.0.


So much of this had to be re-written for it to work anyway, I am not sure I made it very much more efficient to do it this way.


No prefix `:` for args in Siren? Sorry, I havn't look around much. I think `;` is fine, but I'm biased...


Hmm, while this certainly prevents from path traversal, it also adds a regression that using something like `myskins/red` would not work anymore. Although this is nowhere documented, I still consider this a valid use case.

Maybe the check could instead compare the (resolved) skin path to be within `$path`?


(separated with whitespace because I thought it looked clean and more organized)


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


Если пользователя заблокируют, то сообщение об ошибке в Вашем коде будет ложным.
Нужно доставать текст ошибки.


I believe it was unintentional for this to be added, considering it is in the fucking gitingore.... but I guess it's here now *shrugs*


ありがとうございます！！よろしくお願いします！！


Had to re-write this part, as ALL other pipes just reference the parent function, BUT this pipe cant, because it is a child of a child. Recommend doing it some other way.


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


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1046](http://teamcity.monogame.net/viewLog.html?buildId=12849&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


I think we should consider deploying this now. I have been running this setup almost continuously for a further 5 months now with zero false positives.


Quadratic behavior?


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


any chance of a new release tag? ;)


Sorry, commented the wrong commit... This is for d6ed2845eddf.


Ve de deu certo

Laís Vidal de Oliveira

> Em 22 de dez de 2015, às 00:38, Edy Segura <notifications@github.com> escreveu:
>
> Laís, por favor, habilite as issues na configuração do seu repositório!
>
> Você não alterou os nomes propriamente!
>
>
> —
> Reply to this email directly or view it on GitHub.
>


No, I've always found `[ :a :b | ... ]` kinda meh. It would solve quite a few ambiguity problems though.


Hi John, I found the function add_child a little bit unconfortable about the order of the parameters. Should it be like the append function (child, key)?


На този файл не му е тука мястото


@moollaza thanks for your considerate reply

> I hope you can understand where I was coming from now

I follow the reasoning re trimming down the list and, despite my deep feelings towards each and every one of Redis' commands, I tend to agree with it. That said, I think the current sheet should be added with HLL (http://redis.io/commands#hyperloglog) and Geo (http://redis.io/commands#geo) - both basically belong to the same class of the other data structures and won't increase the sheet's length significantly.

> I'll speak with the team to see find out why it's offline...

Great - I wasn't aware of Fathead, sounds cool - waiting to see working.

> If the commands aren't likely to change for a long time...

No promises, but given history new commands are added and existing commands do get updates from time to time. However, if you could integrate with the doc's JSON file, then updates to the Goodies repo should be painless.

> If you have any other ideas or feedback I'd be happy to hear it.

None at the moment, but thanks for opening the door - filed it under "For future reference" :)


Fails to compile with:
```
src/core/socket.c: In function 'socket_done':
src/core/socket.c:168:19: error: void value not ignored as it ought to be
         s->fdname = free(s->fdname);
```
Looks ok in commit c52f06de.


Dunno, you tell me.


LGTM


interpolate_text should be used with gettext, now this message is not translated (and not collected)


ini main sama headernya mana??


Students/30421/Filip O. Sebastian Mihai/Digital school grading book INCERCARE/src/catalog/brain/Students.java

I don't really understand what you mean. I have this class Students. And all the students are entered in an arraylist which is placed here. So whenever i want I can access every student from each classroom, is this not enough ? What else should it do?


tuils -> utils


I think here should be `node_t* nxt = TX_LOAD(&cur->next);`
For example, if there is a linked list `... --> X --> Y --> ...`. and T1 tries to remove Y, while T2 removes X, T2 is unaware that somebody is modifying X's next pointer and both may commit resulting in a list `... --> X --> ...` which is not correct.


No. You also have 802.1Q (VLAN Tagging and Priority)[0][1] and I've seen that this is already treated in the canonical names[2], although they are missing some headers (there is no mention of TPID).
[0] - https://en.wikipedia.org/wiki/Ethernet_frame
[1] - https://en.wikipedia.org/wiki/IEEE_802.1Q
[2] - https://github.com/radustoenescu/Symnetic/blob/98383df074ccb908b1b9d63a9c5a493f53896423/src/main/scala/org/change/v2/util/canonicalnames/package.scala


Release on gamejolt is delayed due to THIS:
![image](https://cloud.githubusercontent.com/assets/11202073/11954881/8f64d8c4-a8ac-11e5-936c-cdbb4be1f2f5.png)

Very annoying, and it means that according to GameJolt I can upload to sourceforge while being off-line which is quite rich!


use "{}".format(target) (for consistency with other log messages)


be welcome!


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


Yes, but this way, your students only have 1 attribute,  a name. Which is not enough! They should have a date of birth, an id, a list of marks at specific subjects and so on and so forth...


And well, you don't have a `Student` class in which all these belong


The rest of the code expects a skin to be a subfolder of skins/ folder. I don't think we support your case in other places either, e.g. where we list available skins for selection.


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


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


What about auto locks? Current way is error prone. We have to remember what kind of release we should call.


You forgot uploading 'save.png'. Please also remove 'disk.png', if it is not needed somewhere else.


Or it not used for CLI?


What if I'll acquire for reading and then accidentally call release for writing? Can we prevent this somehow?


Thank you @mhugent and Timo

Can you run

    ./scripts/astyle.sh src/core/geometry/qgsgeometryutils.cpp

to fix the indentation and tests.

FYI, there is a simple automated solution to avoid formatting issues

https://github.com/m-kuhn/QGIS-Website/blob/qtcreator.rst/source/site/getinvolved/development/git.rst#setup-pre-commit-hook


Если следовать логике остальных параметров, то настройка (та что в settings проекта) должна называться CITIES_LIGHT_INCLUDE_CITY_TYPES


@thomaswue you can now do for example

```
$ jt bench debug classic-fannkuch-redux -Djvmci.option.TruffleFunctionInlining=false
```


Yeahm fixed already. Sorry. Does that fix your memory leak?


Обязательно надо будет на Омерзительную Восьмёрку сходить, эх.


Christmas gift


Which Commit comment?

From: Joshua Freedman [mailto:notifications@github.com]
Sent: Monday, December 21, 2015 21:55
To: FoxDenStudio/FoxCore <FoxCore@noreply.github.com>
Cc: Mike Liu <gravityreallyhatesme@gmail.com>
Subject: Re: [FoxCore] iCanNetwork (818b33e)

@gravityfox <https://github.com/gravityfox>  And you yelled at me for my commit comment...

—
Reply to this email directly or view it on GitHub <https://github.com/FoxDenStudio/FoxCore/commit/818b33ea1dd1382a0509a257337fb4b75de9c51e#commitcomment-15111348> .  <https://github.com/notifications/beacon/ACRcItf9qIocgqRzxBQ0Nbi2fQFB7eNJks5pSN1IgaJpZM4G5Oao.gif>


I see. I thought that i should have another panel which will appear when you click on the student name, and there are going to be all the marks and absences from each student separately.


Hey, may I ask why did you remove the Nested resources section (from the RESTful Resource Controllers) from the docs? Is it deprecated?


Please add "$(VC_IncludePath)" to the top of the include directory.

No, I can't.


Yes, I think so.


modified


You're missing error reporting.
And it's probably better to do that in a job.   It can happen that the parent directory gets moved or so.


No point, you can't run Python 2.6 on AppEngine, and WTForms doesn't support 2.5.


I didn't really understand what it meant. I'm guessing the coder meant that the equals() method would return that these two are the same, but that would be "for either object, the two objects must be the same."


Ok


same, see above


There should be a span class for this. Flavo knows them better than I do offhand

Basically replace '\red' with <span class='something'> and then end the string with </span>


Not helpful in this case, since there's never a `Material` value that is `null`.
Optionals should only be used when `null` would otherwise indicate both "missing" and "null value".


Why though?

And what is the recommended way to handle nested resources like each photo's comments?


If you just add `default='last_first'` to the field definintion, you do not need this method nor calling it in `get_default_partner_names_order`.


Keep in mind that this means that the only version of aeson published on Hackage bloodhound can use is the one with some show-stopping bugs such as bos/aeson#313


I guess these flags should be --skip-<X> instead of --<X>.


Using `$GIT_DIR/info/exclude` or your `core.excludesFile` in your git-config would be a more appropriate place for these gitignore entries.


Include them for consistency/cleanliness. There are times you can get away without them, but we prefer if you explicitly define the entire path each time.


I'd recommend not making them nested, because Laravel won't actually resolve them as nested in the database anyway. If comment 3 belongs to photo 1, you can still access it from photo 2's comment route, right?


Thus, nested resources are kind of pointless imo.


Still compiling... ;)

I will update in the evening and report back tomorrow.


Unless you want to setup a custom binding.


I agree. I think a builder is more suitable here.


I know what to cache and what no to cache, it's based on the oid you are asking.


@tsauerwein, I am getting `{"status": "error", "errors": [{"location": "body", "name": "document.geometry", "description": "\"None\" is not a mapping type: Does not implement dict-like functionality."}]` with `geometry=null`.

See https://github.com/c2corg/v6_api/commit/ee595a74a04b61d3cf7b9d7a913d514d45ec718c


I assume you meant "here" not "her"


So just create a flat photocomments resource?

It really is the same as a nested photos.comments resource (except for the url structure)?


This part is removed so I would say it's ok regarding your comment.


changed the controls to X for attack and A for special
It's cause it's easier to tap X while holding A, which is helpful for santa and tooth fairy


I know a lot of Rubyist's will disagree but I always disliked def_delegators because it's less obvious what is going on to the user at quick read and during audits, I think this should be split up to `def_delegator`


Version 5 is bugged!. text is not positioned correctly in scene2d ui.  heiro 3 was good but I wonder why it have blurry text with small fonts (like size 12) but v5 dont fix the blurry either (for all the 3 modes). FreetypeFont impress me. Text looks (if not) perfect.


@malirod  Yes, you are right. But I see a big possible regression with this locks changing. Propose to keep them as is


put dev-master on your composer ;-)


There's a function called Clamp that'll save you a few lines here. Below is an example usage.

//Clamp to ensure all vars are in the proper range
	Red = Clamp(Red, 0, 255)
	Green = Clamp(Green, 0, 255)
	Blue = Clamp(Blue, 0, 255)


`def_delegator :@obj, :write?, :output`


Hmm, ich hab hier "Christian" und "Raphael" synonym verwendet :D


We need to test that if the directories that becomes a file contained sub directories that are removed from the server, these directories get properly removed locally


To clarify:
Pg13 isn't a requirement, it just looked awkward to me, in a project like this ;)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1046](http://teamcity.monogame.net/viewLog.html?buildId=12849&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 555, ignored: 6 Build time: 00:32:06


@malirod I don't see how to do that without additional locks


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1047](http://teamcity.monogame.net/viewLog.html?buildId=12853&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Creo que no has hecho push

On Mon, Dec 21, 2015 at 8:48 PM, Juan Julián Merelo Guervós <
notifications@github.com> wrote:

> Revisión final. Envía, @fernand0
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ReVision-Docencia-20/Columnas/commit/652145042e0ad4008bfadb6584040ac67f60e799

--
Fernando Tricas
http://elmundoesimperfecto.com/


Makes sense, opened https://jira.percona.com/browse/BLD-379 to fix it.


True. I removed this copy.


same as in another comment, will be fixed in BLD-379


When one is the only one working on a project, one can get very frustrated/bored and often both. My immediate response to those feelings is often crude.


I propose to change this at least in some places. where change is trivial: wrap some small peace of code and use auto lock there. In case of some more complex logic we can leave it.


Well I tried to dig deeper and it seems to me that the url structure, which also gives you the 'parent' resource in your method, are really the main values of the nested resources.

If that's deprecated, how do I achieve it without it?


Another place for span classes.


Removed... Sorry about that.


Span classesssssssss


Thanks for the feedback. Will definitely pick up on this later. Quickly fixed this for a deadline, you probably know how it is ;-)


True. It is cleaner without it.


Hi @buma, I'm converting permissions back to bit flags and am a bit confused by what's going on here. Why is there a method to return only the flags in static field PERMISSION_FLAGS? To facilitate collaboration, we should aim to have clear documentation on every field and method.


sdfs


+        boolean isSuccess = false;
应该是
+        boolean isSuccess = true;
吧？


Ist ja fast das Gleiche... :D


Hello, I've been trying this out and found that `extends` blocks don't need an `endblock`.


Should probably throw `RuntimeError` like `{}.freeze` will, though personally I'm a fan of throwing our own base errors like `CannotModifyImmutableDrop < StandardError`, `Jekyll.const_set(:StandardError, Class.new(StandardError)` so that we can rescue all our errors and let any that aren't ours surface.


We could (Iibc also contains bindings for the various BSDs), but they do not seem to ship hand written assembler implementations, e.g. FreeBsd's [libc](https://github.com/freebsd/freebsd/tree/master/lib/libc/amd64/string) (as glibc does), so there is probably no big speed benefit.


Option's getOrElse uses by name for the default argument, we should probably do the same here.


Here, a hug for you:
*hugs*


Also, what does it mean that the oneway flag is "fake"? Why would we want to indicate that permissions are different in both directions, and why are we using a flag called "one way" to do that?