Cool change, theming support could be important to people and I would not want to do this myself so great work.
I also like how MDIWindow does not keep styles itself but gets them from StylesCSS class, that is convenient.
However, tt looks confusing and bug prone, with the way theme is changed and applied.

1. Having to ensure that every property is adjusted by hand is scary to do and maintain.
2. Doing it at MDICanvas level is an odd place since it does not have styles information itself.

I would create a separate class for each theme that implement some interface that enforces them all to have the needed styles.  Then, current StyleCSS would be a proxy to it.

Basic idea:
```
ThemeInterface {

  public taskBarIconStyleCSS;
  public controlButtonsStyleCSS;
   ......
}

StyleCSS {
private ThemeInterface theme;

public setTheme(ThemeInterface theme) {
   this.theme = theme;
// bindings should trigger observers here and across the MDICanvas
}
}
```

MDIWindow would be permanently bound to StyleCSS by having it passed in constructor. Instead, the StyleCSS be changed at runtime. Since .bind() is used the changes will apply everywhere.

Instantiate with
```
style = new StyleCSS(new DefaultTheme()));
canvas = new MDICanvas(StyleCSS style);
```
and change at runtime with

```
style.setTheme(new DarkTheme());
```

I'll will clean this up over the weekend, unless you beat me to it.


Dallas Farruggia


Well talking is fine. But yes, that does seem to still allow them to use commands. Rip code.


you're missing:  as did many **other** things!


"isinstance(requires, IS_EMPTY_OR)" would fail if not requires, so the outer if is superfluous


FYI you could also do `f = (message) -> begin`


Where is the app tho?


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


to be consistent with other c list functions this can return -ERANGE without filling in the result when max_peers is too small


You say on documentation that only consumed data will be removed from buffer, but here you remove all data.


Alternatively could be written as `return !(contains(message.message, "Use of undeclared symbol $glob") &&(!isempty(message.scope) || message.file != ctx.file))`


:+1: thanks for such a great tool in thinking sphinx!


@TomZiesmer Just wondering for my own interest what the benefit of this is.


Why are some of these strings explicitly using `{}` and others are not? For QString the default constructor will be called regardless of whether you have `{}` or not, so remove it.


Add a testcase for this


leaving a comment woohoo


print "Hello, World!"


Use the browser’s inspector to see how the data is formatted. This will give you a sense of how to retrieve it. Also look at import.io and ‘xpath queries’.

On Nov 25, 2015, at 1:50 PM, Beth M <notifications@github.com<mailto:notifications@github.com>> wrote:

Hmmm...interesting. If we could create something in Gelphi that somehow represents this process, that would be really cool! Analyzing the ethics is a good next step as well and is essential for any app that operates by storing personal browsing data. Susan mentioned that she would try to ask the developers some questions about the code/data storage, but it's hard to say how much they'll be willing to elaborate on this.

—
Reply to this email directly or view it on GitHub<https://github.com/digh5000/student-tutorials/commit/d21fdadb066508ee686688e1626a6cd509829cbe#commitcomment-14624406>.


Кто скрывается под ником Серега?


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


If user press ESC instead of to type master password, the app is not closing :/


@Ercenk in the current code (and in rc1-final) the `ConfigurationSection` constructor allows empty paths. We had a restriction but now empty keys should be allowed. Where are you seeing the exception?


Would it not make sense to also support filtering just based on tag name (without value) ? Then we just remove all tuples where the tag doesn't match.


Hehe I knew this comment will come :). It allows to construct ```PositionalArgument``` with only 1 argument, i.e. ```PositionalArgument{"name"}``` without having to declare an explicit constructor.


LGTM


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


@nathanielc I added a new heading here and below (l293) in an attempt to break up the long stream section. I think it helped improve readability, but feel free to revert or modify the headings as needed.


And you cannot do this without writing the `{}`? I find most strange, could you provide some sort of a reference on this?


could make sense yes.


ouch, that's true. I'll fix before commit. I happened in the last change. sorry
https://github.com/ceolin/soletta/commit/b5df8fb7b49c0bc89b6e9f8f0a6b547dbfbab536#


Woot!


Could you implement this please?


I believe I've been added to this organization by mistake

----
Colin McNaughton

On Wed, Nov 25, 2015 at 1:07 PM, Syed Salahuddin <notifications@github.com>
wrote:

> Also you should check out how mozilla did their todo app (which is sortof
> like your surveys)
> https://github.com/tastejs/todomvc/tree/gh-pages/examples/vanillajs
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Devendavies/Fist-of-Five/commit/62763e99768fc59ad23d23974414c4f207af450d#commitcomment-14623525>
> .
>


Yeah, I started with 7 as the exception and changed it just in order to follow the style of the lines above.  I agree that we ought to use versioncmp throughout really, though with 6 being the exception it'll be ready for 8 as-is.

I think the module may have supported 5 at one stage, but our supported min is 6 in metadata.json.


:cry:


Actually, we don't need any() here as we don't have a vector of args (which I had assumed in the example I sent you).

I just pushed a fix -- no PR for that...


ahh right, but actually it is ibv_exp_qp_create who fails


Excellent!
On Nov 25, 2015 1:31 PM, "borisbstyle" <notifications@github.com> wrote:

> See this commit from a while back
> 8129f47
> <https://github.com/borisbstyle/betaflight/commit/8129f47c69035472d0d0340f19cb954eccadfaf9>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/borisbstyle/betaflight/commit/3d9c17712670ef3d5b44b9e07b5880702c8c1764#commitcomment-14625177>
> .
>


I like how you laid out a plan for `get_best_move` in your README. Here's what you wrote there:

> Problem2: The current difficulity level is "Medium", change it to "hard". Solution: To problem solve this, we would first have to understand how the game algorithm is computing the best move. The following steps illustrate the computation of the "best move". FIRST:
>
> - If the middle spot hasnt been marked, mark the middle spot.
> - Go through every single spot, and make a mark as the computer, then check to see if there is a winner. If there is a winner, mark the spot, Computer wins. If not, move to next step....
> - Go through every single spot, and make a mark as the Human, then check to see if there is a winner. If there is a winner, mark the spot, this way we are blocking the Human from winning.
> - If 2 and 3 aren't true, generate a random spot, and mark it. (The algorithm for generating a random spot is FAULTY, I will elaborate on this more.)
> - In order to produce an algorithm that is "hard", rather than "medium", the generation of a random spot is what need to be adjusted, The computer is defeatable when it marks a spot that is a corner spot, rather than the middle spots. I demonstrate this in the Video with Examples.
> - The random spot generator is faulty, as it doesnt account for spots that are already taken. It keeps running the same rand method until it finds a spot that hasn't been generated. This can be fixed by removing the spots in the rand method that are taken.

What if the `get_best_move` method read more like this description?

```rb
def get_best_move
  if middle_spot_unmarked?
    mark_middle_spot
  elsif computer_wins?
    mark_winning_spot
  elsif human_wins?
    mark_blocking_spot
  else
    mark_random_spot
  end
end
```

Or even:

```rb
def get_best_move
  mark_middle_spot if middle_spot_unmarked?
  mark_winning_spot if computer_wins?
  mark_blocking_spot if human_wins?
  mark_random_spot
end
```

You'll have to pass arguments here and there, but by extracting and naming the components of this method, you can make your code describe the way you're thinking about the problem, without support from a README or whiteboard explanation.


out :
>jt.Configurator.INFO:  Settings : Presets loaded : "PresetDefault.json"
jt.Configurator.INFO:  Initializing ControlSurface...
jt.Configurator.INFO:  Number of groups in controlSurface : 1
jt.Configurator.INFO:  Number of groups in Preset : 1
jt.Configurator.INFO:  Loading group : 0
jt.Configurator.INFO:  Number of tracks in group : 2
jt.Configurator.INFO:  Track gain :  0.98  for 0
jt.Configurator.INFO:  Track Pan :  1  for 0
jt.Configurator.INFO:  Boost index :  2  for 0
jt.Configurator.INFO:  Mute state :  true  for 0
jt.Configurator.INFO:  Track gain :  1.2  for 1
jt.Configurator.INFO:  Track Pan :  -1  for 1
jt.Configurator.INFO:  Boost index :  1  for 1
jt.Configurator.INFO:  Mute state :  true  for 1
jt.Configurator.INFO:  Reseting local inputs...
jt.Configurator.INFO:  	Input reset on channel  0
jt.Configurator.INFO:  	Input reset on channel  1
jt.Configurator.INFO:  Reseting local inputs done!


Why is there a calls alias? Wouldn't we need to already have the call relationship present for this to work? `calls` somehow implies that it will compute the relationship iteself.


http://en.cppreference.com/w/cpp/language/aggregate_initialization look for "If the number of initializer clauses is less than"


No worries, it drew attention to some other issues that needed to be addressed and we're better for it now!


Откуда взялось java.lang.String? Вполне будет работать просто String...


Sorry about the delay in checking this, the CentOS server where we run 7.6 has been running something else for the past few days.


well doesn't that reference say that it works just fine even without the `{}`? have you tried?


might be useful to be able to specify remote cluster uuid too, in case you want to change config from somewhere that doen't have access to the other cluster (due to a temp. failure or by policy)


Awesome. Thank you!


Ok, if we do not support 5, then 6 is the exception.


or maybe get the uuid from rados_conf (it could be stored in the cluster's ceph.conf too)


@thenickcox passing tests, woot!


Now that you have your testing infrastructure in place, it would be a good idea to check in the tests you wrote for this file alongside the production code. In addition to helping drive good design and verifying that your code works as expected, tests are useful as documentation for others who read or modify your code later.


VS screenshot when extracting mmaps of map 249
![49a16c9](https://cloud.githubusercontent.com/assets/1153754/11407906/f76fbf12-93b5-11e5-8b78-f90bec1fcd50.png)


Worth adding a cautionary note about the precedence of the various files where this setting may be present. For example, if present in any of dt-site.xml, application META-INF/properties.xml and in site/conf/app.xml, the third overrides the second which overrides the first. A pointer to the packages doc where the full precedence list is given is useful.


This line could be unclear to others who aren't familiar with ANSI terminal escape codes. It's also duplicated. Is there a good name for this?


aqui não funcionou, fiz todas as mudanças necessárias, e eu estou com uns problemas, me conte, o seu quest log roda alguma missão da bigfoot burden's?


"in each container" => each container hosting an operator with an output port connected to an input port outside the container.


Done.


forward the arg ? (we used to call `enqueueTask` with the callback)


> I'm on ubuntu

Should be

```bash
sudo apt-get install libsoundio-dev
```


nvm, misread that


well I get:
```Envision/InformationScripting/src/queries/Count.cpp:87:85: error: missing field 'syntax_' initializer [-Werror,-Wmissing-field-initializers]```


Ah, right on the html, I didn't look above. I'm not sure if I want to make a mocks folder or not if there is a mock template, and the name presumably is currently for a folder since the `s` is on there. Maybe just change it to `/mock/` to make it more general.


I'm not knocking you for this because it's esoteric Ruby knowledge, but just so you know, bare `puts` without an argument is the same as `puts ""` in Ruby.


The callgraph indeed has to be present. True the name is not optimal, any suggestions? but it looked somewhat nice:
```callgraph -global | calls -self```

maybe ```canCall``` or something is a better name


Funktioniert glaube ich noch nicht so wie es soll


Add note about where these values can be found.


@niltonvasques daqui a pouco testo


Sort of... I meant it to be `ln -s $DIR $XDG_CONFIG_HOME/nvim`, then it totally works.


whoops I do remember this warning now that you say it... I'll just undo the changes! Thanks though :p


Great observation Sasha!

I absolutely agree with you, this code improvement will make pretty
much easier the code maintenance and it will be much more elegant.

My first thought was to create a .css Style Class and load it when*
setTheme() *is called. I think it will be easier to maintain and to add
styles.

We can see which approach will be easier to develop, use and maintain.

--
*Lincoln Minto, CTFL*
Analista Desenvolvedor

*Cel: *55 71 9150-9824 | *Skype:* lincolnminto
*Blog:* http://lincolnminto.wordpress.com/

2015-11-25 17:33 GMT-02:00 Sasha Alex Romanenko <notifications@github.com>:

> Cool change, theming support could be important to people and I would not
> want to do this myself so great work.
> I also like how MDIWindow does not keep styles itself but gets them from
> StylesCSS class, that is convenient.
> However, tt looks confusing and bug prone, with the way theme is changed
> and applied.
>
>    1. Having to ensure that every property is adjusted by hand is scary
>    to do and maintain.
>    2. Doing it at MDICanvas level is an odd place since it does not have
>    styles information itself.
>
> I would create a separate class for each theme that implement some
> interface that enforces them all to have the needed styles. Then, current
> StyleCSS would be a proxy to it.
>
> Basic idea:
>
> ThemeInterface {
>
>   public taskBarIconStyleCSS;
>   public controlButtonsStyleCSS;
>    ......
> }
>
> StyleCSS {
> private ThemeInterface theme;
>
> public setTheme(ThemeInterface theme) {
>    this.theme = theme;
> // bindings should trigger observers here and across the MDICanvas
> }
> }
>
> MDIWindow would be permanently bound to StyleCSS by having it passed in
> constructor. Instead, the StyleCSS be changed at runtime. Since .bind() is
> used the changes will apply everywhere.
>
> Instantiate with
>
> style = new StyleCSS(new DefaultTheme()));
> canvas = new MDICanvas(StyleCSS style);
>
> and change at runtime with
>
> style.setTheme(new DarkTheme());
>
> I'll will clean this up over the weekend, unless you beat me to it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lincolnminto/javaFXMDI/commit/4e0f153fce92581a785fdd89d9831f572209e08f#commitcomment-14625222>
> .
>


@asutton @thehexia: `llvm::IRBuilder::CreateNeg()`, which calls `llvm::BinaryOperator::CreateNeg()`, actually does what you want here (and also works for all value types, _and_ folds constants).

From llvm/IR/IRBuilder.h:
```cpp
  Value *CreateNeg(Value *V, const Twine &Name = "",
                   bool HasNUW = false, bool HasNSW = false) {
    if (Constant *VC = dyn_cast<Constant>(V))
      return Insert(Folder.CreateNeg(VC, HasNUW, HasNSW), Name);
    BinaryOperator *BO = Insert(BinaryOperator::CreateNeg(V), Name);
    if (HasNUW) BO->setHasNoUnsignedWrap();
    if (HasNSW) BO->setHasNoSignedWrap();
    return BO;
  }
```

From IR/Instructions.cpp:
```cpp
BinaryOperator *BinaryOperator::CreateNeg(Value *Op, const Twine &Name,
                                          Instruction *InsertBefore) {
  Value *zero = ConstantFP::getZeroValueForNegation(Op->getType());
  return new BinaryOperator(Instruction::Sub,
                            zero, Op,
                            Op->getType(), Name, InsertBefore);
}
```


Add note that this should be based on the maximum need for all the fields of the operator -- both transient and non-transient.


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


that doesn't work.
I'm talking about it here: https://lmms.io/forum/viewtopic.php?f=4&t=4573


Nem me atentei a isso. Na verdade, fui atrás do problema relatado em https://github.com/otsolutions/Vanaheim/issues/7


Again, `llvm::IRBuilder` already has a member function (`llvm::IRBuilder::CreateNot()`) that does this.

From llvm/IR/IRBuilder.h:
```cpp
  Value *CreateNot(Value *V, const Twine &Name = "") {
    if (Constant *VC = dyn_cast<Constant>(V))
      return Insert(Folder.CreateNot(VC), Name);
    return Insert(BinaryOperator::CreateNot(V), Name);
  }
```

From IR/Instructions.cpp:
```cpp
BinaryOperator *BinaryOperator::CreateNot(Value *Op, const Twine &Name,
                                          Instruction *InsertBefore) {
  Constant *C = Constant::getAllOnesValue(Op->getType());
  return new BinaryOperator(Instruction::Xor, Op, C,
                            Op->getType(), Name, InsertBefore);
}
```


One huge benefit of objects is that they can help store and modify state in a controlled way. Global variables are dangerous because they expose this state to the world. Since `$player_one` and `$player_two` are global, they can be modified, overwritten, or deleted by any object or method that cares to modify them.

Can you think of a better way to give this class access to the players that it needs?


@mfonville here (I have inconsistent nicknaming ;-) )


which gcc versions did you test?

afaik switch/case is only optimized when it has >3 cases, so the default could be the fourth in your tests?!

also there are A LOT of more of such compile warnings about missing cases (sorry, i've missed these in the first run / commit)

imo such micro optimizations should be avoided as they clutter the code and newer gcc versions possible fail on these in a different way.


Well, although the syntax you present is shorter, I'm not convinced that it is really appropriate. Reading the whole command I understand what it does, but if I just read the `| calls -self` part, I would think it does the same thing. Even for demonstrational purposes `callgraph -global | canReach calls -self` is better because it illustrates the use of the `canReach` basic block which users can immediately imagine using for other things.


Ow, I did not know that is possible in JavaFX. Sounds even better.


Alright. Though what I read on cppreference suggests that this should just work without warnings and has well defined semantics.


how do you know what is this values?
ClientRsa = client.BaseAddress + 0x3899A0;
ClientServerStart = client.BaseAddress + 0x48F5FC;
ClientServerEnd = client.BaseAddress + 0x48F600;
ClientServerStep = 0x30;
ClientServerDistanceHostname = 0x04;
ClientServerDistanceIP = 0x1C;
ClientServerDistancePort = 0x28;
ClientServerMax = 10;
ClientSelectedCharacter = client.BaseAddress + 0x5D244C;
ClientMultiClient = client.BaseAddress + 0x158277;

ClientStatus = client.BaseAddress + 0x31CA38;//?

PlayerGoX = client.BaseAddress + 0x5D5030;
 PlayerGoY = client.BaseAddress + 0x5D5028;
PlayerGoZ = client.BaseAddress + 0x5D5004;

ClientBattleListStart = client.BaseAddress + 0x62BF90;


Add that the default is 512MB. Also, in my experiments, even a value as small as 128MB works OK, so why would larger values be needed ? I'm guessing it will be needed to buffer output when the downstream operator is not able to consume tuples fast enough but I'm not entirely sure ...


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


@polomani, хз


I'm afraid I can't fix that since I have no access to your internal repo.
I will appreciate it if you would push the changes into GitHub when you can.
Thanks a lot.


@asutton @thehexia: Again, `llvm::IRBuilder` already has a member function (`llvm::IRBuilder::CreateNot()`) that does this.

From llvm/IR/IRBuilder.h:
```cpp
  Value *CreateNot(Value *V, const Twine &Name = "") {
    if (Constant *VC = dyn_cast<Constant>(V))
      return Insert(Folder.CreateNot(VC), Name);
    return Insert(BinaryOperator::CreateNot(V), Name);
  }
```

From IR/Instructions.cpp:
```cpp
BinaryOperator *BinaryOperator::CreateNot(Value *Op, const Twine &Name,
                                          Instruction *InsertBefore) {
  Constant *C = Constant::getAllOnesValue(Op->getType());
  return new BinaryOperator(Instruction::Xor, Op, C,
                            Op->getType(), Name, InsertBefore);
}
```


Hey, cool! Thanks for including peewee.


@rtri

there are several more of such compile warnings in

rts/Rendering/LuaObjectDrawer.cpp
rts/Lua/LuaMaterial.cpp
rts/Lua/LuaMaterial.h

can will you fix thise?

also https://github.com/spring/spring/commit/a6c9a9a602c9dbd7644bef492147b1710253a3e5 (+comment)


dict is reserved because it's a builtin. Initializing the dictionary with values is less to type and looks cleaner.


You had ``what`` already declared. Remove here, or remove in header of the function. Better remove here for consistency with the rest of the library.


The time formats can be combined into one


I believe we won't need rktInfo anymore :)


You "Format" is an exported type with direct identifier access, which makes it public, you will have to add a line of comment above it or change to lower case "f"


Where it break?


Note that this initial checkin on feature/Batching-Support branch is work-in-progress and currently the files compile but the project will fail during linking.


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


Sorry, I linked to the wrong patch: https://gist.github.com/talltom/f73f09feb7f5e9d8425a


Use the command in the path. If you need the full path on mac we need to find a workaround.


Made changes according to your comments.
see here:
https://github.com/hsyuan/gpdb/commit/f50ce3a3789c394023d02308c83f9441c01ec63f


From http://linux.die.net/man/2/bind:

"On success, zero is returned. On error, -1 is returned, and errno is set appropriately."

For error, better check for no success. ``!= 0``.

Also for consistency on the checks.


Same here, please jump to a rollback routine.


Possibly, especially if Jetpack is one of them.


Please remove, put it below directly.


Use search instead of findall()[0]. group(1) gets the first result in parentheses. Multiline is not needed.


Please remove. This is only used once.


Need to be sure to check permissions here.
Also let's split this out to variables.
$is_draft_revision = ($vid === $this->draftRevision);
$is_current = (!$this->draftRevision && $vid === $node->getRevisionId());
if (ACCESS_CHECK && ($is_draft_revision || $is_current)) {}


nit: no space in between `E :`


Oops! I messed things up here.


This is the default for the tun/tap device. Move it inside the "else" with the ``alloc_viface`` call.


I pretty much just c/p what I found in `updates.js`. I haven't really tested that so far.


I'd already applied that one as part of https://github.com/smart-facility/cognicity-schema/commit/6418f4fdcb36f36aac3b02596fe026d045fdf7f2#commitcomment-14585217 - did the patch cover both? (Sorry, haven't really read it, been super busy this morning)


I think I'd prefer a where clause here, because it's a pretty long line


A where clause would reduce the rightward drift here


And here


Ugly. Oh well :(


11.24的效果比较好


A where clause would look nicer here I think


Please set MTU. For already existing interfaces the MTU can be read from file ``/sys/class/net/{iface}/mtu``. Please read it and set it.


Use `FIXME(bholley):` —that's the style in layout


nit: Indent to the `(` like rustfmt does


Why did you do this here? for the tun/tap it isn't required, as it is performed by the ``up()`` call. See line #L724.

Remove this line.


Suggest where clause, rightward drift


Did this test fail previously, or does the above change cause this to fail?


README.md Modified.


revert


这里写的有问题，参数不是通过globals传递的，因为这不是函数的调用，(perfexpert和analyzer)就不属于同一个工程，其通过设置环境变量来进行参数传递，后面也会更新这个错误注释


Why did you import `Win32GraphicsConfig` or `Win32GraphicsDevice`?


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


@mkonicek   'convention the package name would end with Package.java' , why not let the library provider write some config file to specify that , I think it will be reasonable.


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


nit: Issue number in this comment please.


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


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


Okay, in that case, let's kill `debug`, and I think session time should be a system property. We should put the session timeout = 180 value into all the `standalone.xml` files, since we do want that to be configurable.

- [ ] If `authenticatedSessionTimeoutMinutes` isn't used anywhere, that's a bug we need to fix.


*series
*that will work


Added a folder


> Oh, wow this must have changed relatively recently. It definitely didn't use to work that setting {fields} affected reactivity.

And that's exactly why Tracker every so often gets its foot snagged on something. Nobody uses this feature. Even the people that do--myself included--can't be bothered with optimizing `fields` until you get caught up in a Blaze storm.

My point is MDG is really outa their mind when it comes to the nonsense they're kickin about React when they haven't done the diligence and instead are sweeping the real culprit--Tracker--under the rug. With that premature PR announcement meant to assuage our concerns and brace us for an inevitable future, they're being sloppy.


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


This isn't quite going to work as you expect. obj.states is fetched dynamically (except for the core cycle cache), so if you were to press NVDA+tab or similar, you'd here "selected". Instead, I suggest you use an overlay class and override the states property to remove the state.

Doing this also avoids the need to have code for this in both chooseNVDAObjectOverlayClasses and event_NVDAObject_init. You'll still need to remove that "List" class, though, as you do now.


添田です。

クエリストリングからは、種別番号・個数・総額を受け取ってください。
種別を定義してprintしていただいていますが、
そうではなく、クエリストリングから種別番号を取得し、その番号（1なら雑貨）にあった商品種別を表示してください。

また、先日お話したとおり、PHPの途中で<?phpの記述がありますが、ミスの元ですので消しちゃってください。

あと、ポイント計算のところで総額を3で割っていますが、個数もクエリストリングで渡していますので、
そちらを利用する様に変更してください。


Worst case, a configure invalidation of touching too many times just means other trees sharing the same source checkout re-run configure which doesn't take that long for libuv. We could potentially move all 3 of these touches to immediately after the checkout/untarring of the source, though that's maybe a little messy to put in to the git-externals define.


These should be chef.io


This should be Chef Software, Inc.


This is BC break for user-land onError handlers that are type-hinted as \Exception. (Also you forgot to update the prototype in property comment).


@shuyangzhou you meant @juliocamarero I think.


@alexbegt Uh huh

> On Nov 24, 2015, at 5:59 PM, Alexander (alexbegt) <notifications@github.com> wrote:
>
> and from me of course, credit to alex anyday
>
> —
> Reply to this email directly or view it on GitHub.
>


Everything should be working now with the latest commits - even fixed a bug in EwaScaler that was introduced when I updated it to use ManagedTexture.

Now on to the next bit - make the Audio class implement IFilter. Any chance you could do that bit? :)


You realise I work at MDG?

The reason to switch to React isn't really performance. Certainly it's good to get the optimizations that React brings, but the biggest reason is that React's component model is very much in line with what most people agree a "Blaze 2" should look like.

So I think you are missing the point when you assume that we don't understand the limitations of Tracker. I think you'd be surprised about what we do and don't understand about the limitations of current Meteor (not withstanding my misconception about minimongo above).


좃병신 ㅋㅋ


やっていただきたいのは、'a'を20個【連結】する事です。
'a'を20回echoしていただいてもそう見えますが、
変数の中にaを20個収めてください。


extra brackets here, the call below doesnt match this structure...

instead:

```python
            message = {
                'equal': {'text': u"We can proceed with the update. New update has same DB version", 'type': logger.DEBUG},
                'upgrade': {'text': u"We can't proceed with the update. New update has a new DB version. Please manually update", 'type': logger.WARNING},
                'downgrade': {'text': u"We can't proceed with the update. New update has a old DB version. It's not possible to downgrade", 'type': logger.ERROR}
            }
```


良いですね！Pythonやられてたというだけはあると思います。
他の人は別の変数を作る事に躊躇されていました。
必要な変数はどんどん作りましょう！


@sospartan Having a convention is always easier, and it means that it'll also work with most of the existing modules as most follow this convention.


HTMLの話になりますが、
で始まったら
で閉じます。
HTMLのタグを閉じる場合は、「<」ではなく「</」になります。


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


sounds good


Uh huh...right. And how is it different from the math for the rainbow beacon?


1. only the data points uploaded by device (data points uploaded with deviceKey) will trigger the action.
2. url -> URL


Yeah its a negative test, because the request data is actually invalid.


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


ok


okkk


yamada


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


select


Hope I didn't break something with this, it looked like an oversight relative to the issue.  I am happy to put together some tests and revert if necessary.  I will go create one right now for Jackson.


I was able to remove LifecycleMethodsWithRequestScope without breaking the
build.

However, the build after that failed, apparently because of a change in
https://github.com/zanata/zanata-server/commit/75f8106ac56a63cc12f469a0c251a5fbb7bcb021

On 24 November 2015 at 16:28, Patrick Huang <notifications@github.com>
wrote:

> ZanataInit still need this interceptor AFAIK
>
> On Tue, Nov 24, 2015 at 3:49 PM, Sean Flanigan <notifications@github.com>
> wrote:
>
> > Actually, the idea in LifecycleMethodsWithRequestScopeInterceptor was to
> > implement an interceptor (not a bean) for PostConstruct and PreDestroy
> > methods, a bit like this article:
> >
> >
> http://javaevangelist.blogspot.com.au/2013/08/jsfcdi-tip-of-day-postconstruct.html
> >
> > To fix the interceptor, I think the method just needs to return void, as
> > the warning message says.
> >
> > However, we shouldn't need this interceptor any more, since we now
> > pre-cache everything we'll need in ZanataIdentity.fireLogoutEvent.
> > Activating contexts while destroying another context seems to be a really
> > bad idea anyway.
> >
> > I'll get rid of LifecycleMethodsWithRequestScopeInterceptor and
> > LifecycleMethodsWithRequestScope.
> >
> > On 5 November 2015 at 10:13, Carlos Munoz <notifications@github.com>
> > wrote:
> >
> > > This should not be a @PostConstruct since it's not a bean really. If it
> > > is, we should split it into an interceptor and a bean each with their
> own
> > > logic.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/zanata/zanata-server/commit/a009191d26d07bb1a8caa23e17b0a90668b4bce6#commitcomment-14199431
> > >
> > > .
> > >
> >
> >
> >
> > --
> > Sean Flanigan
> >
> > Principal Software Engineer
> > Globalisation Tools Engineering
> > Red Hat
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/zanata/zanata-server/commit/a009191d26d07bb1a8caa23e17b0a90668b4bce6#commitcomment-14579482
> >
> > .
> >
>
>
>
> --
> Patrick Huang
> Senior Software Engineer
> Engineering - Internationalisation
> Red Hat, Asia-Pacific Pty Ltd
> Level 1, 193 North Quay
> Brisbane 4000
> Office: +61 7 3514 8278
> Fax: +61 7 3514 8199
> IRC: pahuang
> github: github.com/huangp
> Website: www.redhat.com
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zanata/zanata-server/commit/a009191d26d07bb1a8caa23e17b0a90668b4bce6#commitcomment-14579969>
> .
>

--
Sean Flanigan

Principal Software Engineer
Globalisation Tools Engineering
Red Hat


i18n required.


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


i18n again.


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


> You realise I work at MDG?

But you didn't always brother. Didn't they just bring Percolate on as their main development partner within the last year?

Either way, it doesn't change anything. People at MDG think that React can't render Blaze 1, or Geoff wouldn't have said we'll have to "port" our templates, etc. I don't think people internally over there have done the research I have on what it takes to make a React adapter for Blaze. If it's possible, why are they scaring everyone. Cuz they don't know. Evan did bit of prematurely research in May--but not enough.

Blaze was never far off from the right component model--they just should have taken the leap long ago when David Greenspan built iterations on the component-based version. Regardless, interface-wise React is basically Blaze Components with the key changes i've been mentioning. Implementation-wise isn't what we or any of the complaining developers are talkin about.

So what I'm saying is the proper strategy is to build `Meteor.Components`--in fact that's what I'm doing, what today's launch was about, and already did for Blaze. The real one. Not Blaze Components, which missed the mark `state`-wise. Then you get it so you can switch out the renderer. Get everyone comfortable moving forward with the component system. After all, the blaze component system is just a wrapper of Blaze 1 like Blaze Components--i.e. not that hard to build and already basically exists. Then as the React component system catches up, developers begin to switch over the renderer to React. And all while doing that, MDG is working on the Blaze 1 renderer (i.e. Sideburns). Get that done, and then insure you can use blaze 1 stile code along side the new `Meteor.Components` which can be rendered with either Blaze or React. Then on top of that, there are tools to transpile old Blaze as a build step, and tools to permanently transpile all your files so the build step isn't needed. That way you can optimize your transpiled blaze components to look more like the new `Meteor.Components` interface.

Bottom line is: React can render Blaze 1. So why MDG would not make that a top priority in their agenda is, like I said, nonsense. But it's all good, they just haven't done the work in this area.


a


:smile:


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


FewDocFixes (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


does piping that whole string through `translate` suffice?


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


Why not just call `room.add()` instead of creating a wrapper function? Regardless, the collection should be registering for events, but not on the socket. I don't think you should be trying to attach the socket to the collection--after all, you will have many sockets when multiple users connect, hence why we need a collection in the first place!


oh wait we need to define a key for it like `help.debugMode.desc` or something and define a default right?


- 52115102


Hmm, I originally thought this is not a good idea, but it seems that \Error is not allowed either...
Also stumbled on a strange behavior - e.g. call to an undefined function or just type error results in same weird message about exceptions not being allowed in __toString.
https://bugs.php.net/bug.php?id=70967


Thanks. Fixed.


No. Create a key, eg: 'debug.mode.description', and add it to medic-api/translations/messages-en.properties. Just piping it through translate won't add the key to the db, so users won't be able to configure it.


Read the documentation for event.trigger: http://backbonejs.org/#Events-trigger Basically, whatever you pass as the second argument when triggering an event will show up as the argument on the listener. However, your event names cannot have spaces in them, so you may need to register for `"chatMessage"` instead of `"chat message"`.


FewDocFixes (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)


maybe use `debug_mode.title` and `debug_mode.desc`?


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


See above note about whitespace in event names. In jQuery, and so also in Backbone (which follows the same conventions) event names must be a single word. By registering for "chat message", it thinks you want to register for two events, one named chat and one named message.


Are there many other cases we need it now or is initing the behaviors the only current use?


If you like.


What is this variable for? It seems never used.


I don't think you should try to pass the socket to the function--you can't send a socket over itself.


Don't convert the user to JSON before adding it. That breaks the connection, because the socket gets lost when it gets serialized. Instead, just add it directly.


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


That's a smart way of doing it. Especially because it puts the bulk of the work on the database instead of trying to do it all in Java. :+1:


Need the following settings to be more responsive and enough capture buffer


I can imagine closing the database connection on a persistent application is important. Otherwise there might be some loose ends. In PHP it isn't super necessary, but that's mostly because the app shuts down after every page load.


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


Rebased onto b8


Any idea when 0.9.9 will be released on the site? Or can we already do it manually through the github?


Pretty sure this should say softlayer! :)


hehe


hehhe


I created a ticket - https://github.com/openbabel/openbabel/issues/238 with details why this fix is needed.


@satya164  I agree convention is easier , but reliability is very important to a decent library .
You can ensure your own code having a 'Package.java' for the feature you wanna provide , but the other libraries your depends on and build with .


#he


@dachary. In my environment，because as root, this patch don't make TestLFNIndex::remove_object failed. The reason is:
   if (getuid() != 0)
      EXPECT_EQ(-EACCES, remove_object(path, hoid));
    EXPECT_EQ(0, ::chmod("PATH_1", 0700));
    EXPECT_EQ(-ENOENT, remove_object(path, hoid));
If hoid don't exist and no permission. Which is the return error? EACCES or ENOENT. In fact, it return EACESS. But LFNIndex in osd code, the permission check by osd. So i think we should remove those check.


Help me see this bug
11-25 10:50:11.025 6814-6814/? E/SELinux: [DEBUG] get_category: variable seinfocat: default sensitivity: NULL, cateogry: NULL
11-25 10:50:11.175 6814-6837/com.fringefy.urbo.app E/RenderScript: Failed loading RS driver: dlopen failed: could not load library "librs_adreno.so" needed by "libRSDriver_adreno.so"; caused by library "librs_adreno.so" not found
11-25 10:50:11.175 6814-6837/com.fringefy.urbo.app E/RenderScript: Failed to load runtime libRSDriver_adreno.so, loading default
11-25 10:50:12.465 6814-6883/com.fringefy.urbo.app E/dalvikvm: VM aborting
11-25 10:50:12.465 6814-6883/com.fringefy.urbo.app A/libc: Fatal signal 6 (SIGABRT) at 0x00001a9e (code=-6), thread 6883 (CameraThread)
11-25 10:50:12.475 6814-6814/com.fringefy.urbo.app E/AndroidRuntime: FATAL EXCEPTION: main
                                                                     Process: com.fringefy.urbo.app, PID: 6814
                                                                     java.lang.IllegalStateException: Could not execute method of the activity
                                                                         at android.view.View$1.onClick(View.java:4035)
                                                                         at android.view.View.performClick(View.java:4881)
                                                                         at android.view.View$PerformClick.run(View.java:19592)
                                                                         at android.os.Handler.handleCallback(Handler.java:733)
                                                                         at android.os.Handler.dispatchMessage(Handler.java:95)
                                                                         at android.os.Looper.loop(Looper.java:146)
                                                                         at android.app.ActivityThread.main(ActivityThread.java:5756)
                                                                         at java.lang.reflect.Method.invokeNative(Native Method)
                                                                         at java.lang.reflect.Method.invoke(Method.java:515)
                                                                         at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1291)
                                                                         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1107)
                                                                         at dalvik.system.NativeStart.main(Native Method)
                                                                      Caused by: java.lang.reflect.InvocationTargetException
                                                                         at java.lang.reflect.Method.invokeNative(Native Method)
                                                                         at java.lang.reflect.Method.invoke(Method.java:515)
                                                                         at android.view.View$1.onClick(View.java:4030)
                                                                         at android.view.View.performClick(View.java:4881)
                                                                         at android.view.View$PerformClick.run(View.java:19592)
                                                                         at android.os.Handler.handleCallback(Handler.java:733)
                                                                         at android.os.Handler.dispatchMessage(Handler.java:95)
                                                                         at android.os.Looper.loop(Looper.java:146)
                                                                         at android.app.ActivityThread.main(ActivityThread.java:5756)
                                                                         at java.lang.reflect.Method.invokeNative(Native Method)
                                                                         at java.lang.reflect.Method.invoke(Method.java:515)
                                                                         at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1291)
                                                                         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1107)
                                                                         at dalvik.system.NativeStart.main(Native Method)
                                                                      Caused by: java.lang.NullPointerException
                                                                         at com.fringefy.urbo.app.MainActivity.onClick(MainActivity.java:88)
                                                                         at java.lang.reflect.Method.invokeNative(Native Method)
                                                                         at java.lang.reflect.Method.invoke(Method.java:515)
                                                                         at android.view.View$1.onClick(View.java:4030)
                                                                         at android.view.View.performClick(View.java:4881)
                                                                         at android.view.View$PerformClick.run(View.java:19592)
                                                                         at android.os.Handler.handleCallback(Handler.java:733)
                                                                         at android.os.Handler.dispatchMessage(Handler.java:95)
                                                                         at android.os.Looper.loop(Looper.java:146)
                                                                         at android.app.ActivityThread.main(ActivityThread.java:5756)
                                                                         at java.lang.reflect.Method.invokeNative(Native Method)
                                                                         at java.lang.reflect.Method.invoke(Method.java:515)
                                                                         at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1291)
                                                                         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1107)
                                                                         at dalvik.system.NativeStart.main(Native Method)


FewDocFixes (Myoldmopar) - i386-Windows-7-VisualStudio-12: Tests Failed

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


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


Threw together some tests and this is definitely a problem for Jackson.  Expect a PR opened tonight with a fix.


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


need this for channel hopping other than CHAN_WIDTH_40


Hello,

If you don't mind me asking, where did you find your data set? I'm a graduate student at the University of Buenos Aires and I'm interested in doing a similar project to your.


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


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


@jdillard
s/drowdowns/dropdowns


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


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


不delete了?


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


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


hmm, Tis fix fixed Dragon 1, dragon 2


I don't understand why need max_chan_width here? The width and center in ifctrl_iwset_freq is not used.


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


done


@Uaman, створити енам з таки змінними взагалі не варіант?


More intermediate Ruby:

Initializing a class with some argument, storing it in an instance variable, and giving it a getter method with the same name as the instance variable is so common that there's a shortcut called `attr_reader`:

```rb
class Whoa

  attr_reader :cool

  def initialize(cool)
    @cool = cool
  end
end
```

Is the same as:

```rb
class Whoa

  def initialize(cool)
    @cool = cool
  end

  def cool
    @cool
  end
end
```

You can read about `attr_reader` (and others) here: http://ruby-doc.org/core-2.1.1/Module.html#method-i-attr_reader


**Не понял, как реализовать блок с загрузкой изображений.** :neutral_face:


got passed that error now at least :) (I assume the tarball you sent me included that fix, right?)


are you sure `baseStart = 10` syntax is allowed?


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


yes, use logger instead.


Add that the default is 1GB and we rarely see a need to go higher even in production; what is this space used for ? How do I estimate this need ? I found that 300MB was too small but 512MB worked OK for my word count application.


You should change all thee `I`s to `we`s


It has to be in update, it's not rendering, just checking which features are in view.


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


@mattnworb So we'll need to somehow keep this logic here since `connection.getResponseCode()` will try to authenticate the server if it's HTTPS.


This section should probably be put into separate GitHub issues?


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


Booooom

Sent from my iPhone

> On 25 Nov 2015, at 19:49, Luan Santos <notifications@github.com> wrote:
>
> Switch entirely to fish (http://fishshell.com/)
>
> @julz @totherme
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/luan/dotfiles/commit/3c49cad8bd14fe44bd7e83a6f07616c5109baafa

>


as follows => by adding this dependency stanza


I think this must be a typo. It should be 13099, for the central value of the CT14nlo set, right?


I see what happened here. The name is currently based on the rotation from vehicle frame to sensor frame, not the other way around. I'm not sure if that matches the convention. I thought it did at the time.


The name of this function clearly shows what it's does (centralize shared knowledge), good.  BUT, the comment above should CLEARLY say WHAT knowledge should be kept in sync, and WHERE.  That's at least two places, the frontent router, and the backend router.  The comment should specify the file and method.


Because it's stable... Duh


add_exceptions (mbadams5) - x86_64-MacOS-10.9-clang: OK (1966 of 1966 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)


Should this be the responsibility of the `Game` now that you have a `View` class?

One good way to think about separation of concerns as you extract classes is the Single Responsibility Principle, one component of the "SOLID" principles for OO design (https://en.wikipedia.org/wiki/SOLID_(object-oriented_design)). The Single Responsibility Principle says that a class should have only one reason to change.

If you decided to stop clearing the terminal between menu screens, how many places in your code would have to change? If there's more than one, it's an indication that concerns may not be as cleanly separated across classes as they could be.


Não é GTA é GTM xD


Yes; but I don't really know what half of them mean.


Good point; consistency.


@Landy005 please stop asking "why you do this" and "why you do that", it's getting really annoying.


YW :)


This method is over 100 lines long! Sometimes long methods are necessary, but usually very long method bodies are a code smell.

Ask yourself: Is this method as simple and expressive as it possibly can be? Has everything that can be usefully named been extracted?


| which gcc versions did you test?
all are affected

see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66264


as follows => in pom.xml as follows
[also, all the values are empty in the stanza below; is that what is intended or did the github display swallow some characters ?]


I'm a bit confused here. I think the CT14 set should run from the central value of 13100 to 13156. So I think that line 209 in runcmsgrid_powheg.sh must be a typo. I also don't understand the purpose of the 5000 weights. Isn't this just CT14 again? Though in this case there seems to also be a typo, line 299 should be 3156, right? Can someone clarify this for me?


Nitpicking here, but that line is the heart of the function.  Did we really want to put it in an assignment through a ternary operator, in a continued variable definition? ;)

As soon as we support a third case (users come to mind...), we'll have to rewrite it into a case statement.  No reason to do it now, but I was puzzled for a few seconds.


I'm going to name some responsibilities as I see them in this method, to help you sort out where the boundaries of this class should be. Here's one: delegating output to the `View`.


Another responsibility: getting user input.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1436 of 1436 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.5%25-green.svg)


Another responsibility: Directly displaying output.


Please don't worry about forking to do your own thing! It's open source, so you're always welcome to build your own version :)


Another responsibility: Controlling logic based on user input.


I wrote a OO refactoring before seeing your comment.


wrong, it _really_ renders


@romjerome Please don't commit anything. @vantu5z and I are working
on it.  Any versions here on github are too old to be committed.  Thanks.


Expose some ports? And -dbpath will take precedence over the config, which may be wrong. Is there a way we can environment variable that?


TeamCity pull requests :: netty [Build 748](https://garage.netty.io/teamcity/viewLog.html?buildId=3606&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5694, ignored: 21 Build time: 00:41:35


TeamCity pull requests :: netty [Build 749](https://garage.netty.io/teamcity/viewLog.html?buildId=3607&buildTypeId=pulls_netty&guest=1) is now running


The build did pass after the commit / push.
see https://travis-ci.org/oshoukry/openpojo

Can you please elaborate on what errors you're seeing?

- Osman


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


What do the numbers `1` and `2` represent here? What if `order1` was `2` and `order2` was `1`? Bare numbers without descriptive names are a code smell usually called "magic numbers".

One way to prevent them is to extract constants, if the numbers have meaningful names, e.g.:

```rb
class Circle
  PI = 3.14159

  def initialize(radius)
    @radius = radius
  end

  def area
    PI * area ** 2
  end
end
```
Maybe there's a name you can extract here, like `PLAYER_ORDER` or something. Even better, there may be a way to avoid using these numbers at all.


They weren't deleted.


Where did you get this crap?


`Function.identity()` :smiley:


Maybe another responsibility: Configuring the game and players.


Should that be {'spouse': self._('Unknown')} ??


That's a pity. But I understand your wishes to know when line was last edited.


TeamCity pull requests :: netty [Build 747](https://garage.netty.io/teamcity/viewLog.html?buildId=3605&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5694, ignored: 21 Build time: 01:19:47


Maybe another responsibility: Starting the game.


TeamCity pull requests :: netty [Build 750](https://garage.netty.io/teamcity/viewLog.html?buildId=3608&buildTypeId=pulls_netty&guest=1) is now running


Instead of having every implementation construct a map, can we have a ResultBuilder that internally manages a map and which provides convenience methods that allow to associate a failure with a task?


That's a rather narrow fix. If you have already entered the recv(), you still won't be interrupted. Also, why does this only apply to the non-SSL codepath?

There were some large changes to signal handling in 9.5, to fix this and other similar cases where send/receive from client was not interruptible. But those were quite large and invasive patches. Maybe we should indeed do some quick for some of the issues, but this seems pretty ad hoc.


BUT you did do a hard reset before it started working, right?


Misspelled canola


Also, should we use IdentityHashMap for that?


Doh! Thanks


Enable the Parallel LCD driver.


echo statements were there for testing. I use old skool debugging techniques.


Remove the penguin logo from boot


That's okay. I actually hadn't even inspected the thing first.


Add the Spidev driver needed to create /dev/spidev0.0 type interface. Used in Android to talk to the i/o board.


But there is ```-w``` key for git-blame
```Ignore whitespace when comparing the parent’s version and the child’s to find where the lines came from.```


Add the touchscreen driver


Кажется, тут ошибка. Должно быть ASSERT_EQ(6, key)?


Add the option to add the touchscreen driver during a "make menuconfig".


Tell the compiler to make the touchscreen .c files.


Соня, подпишись реальной фамилией. Идентифицировал тебя через гугль - который привел меня к твоему аккаунту на CodeForces. Кстати - держись! Не дай им скинуть тебя в "серые"! :)


Please ignore! Check out the second one!


why this change?


Кажется, тут ошибка. Должно быть ASSERT_EQ(4, key)?


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


Option to build the microtips lcd init driver during "make menuconfig".


You may have heard of the DRY principle, for "don't repeat yourself." (If not, you can read about it here: https://en.wikipedia.org/wiki/Don%27t_repeat_yourself). Avoiding duplication in code makes it much easier to read, but also much easier to change. For example, what if you wanted to insert a new message after printing the board? As designed here, you'd have to insert the same line in four different places: after line 126, 131, 138, and 143!

Most of the code inside these `if` blocks is identical, with a few different magic numbers and global variables. Can you think of a way to remove this duplication?


Tell the compiler to make the .c file.


I think that 5+0.1 are not stable time controls, especially when more such games are shared on one computer (31 cores machines). I think 5+0.05 is better, if we can't afford 10+0.1.


because this commit alone did not resolve the issue until I reflashed stock.
having already flashed Vanir and booted, reverting back to toybox's mount didn't break anything.

so if this commit doesn't break it after the fact, and couldn't fix it in my case... then I think the behavioral change was a result of the flash-all.sh... even if the system was booted a few times in the middle.

AOSP is derping pretty hard with the order of operations of changing encryption passwords... my guess is some metadata got out of sync, which required my B&E script to get into the phone, but that same issue would have persisted... i bet if i was back in that "correct password is incorrect, butthole" state, formatted /data, and tried to boot, it would have stayed at the google logo.


also add to the exclusion list in `contrib/add_license_to_files.jl`


This file doesn't actually work yet. Currently the LCD is being initialized in U-Boot.


Getting a EmptyStackException when pressing back on homescreen at launch.


Any reason we are not enabling MT by default?


Me encantó estee


The 4.4.3 release needs different LCD timing than the 4.4.2 release. Not sure why.


Remember that in addition to new features, one of the requests was to make this flexible for future changes. Are these methods an improvement?


Responsibility: storing data.


Responsibility: displaying data.


Responsibility: getting user input.


Changing this to "lcd" tells the framebuffer to use the next node.


What if we wanted to display a 4x4 board? An n-by-n board? Is there a way to make this method more flexible?


I chatted with deadwolf about it this morning. I should be able to drum up the missing frames with half an hour or so in GIMP, so it's no big deal. Just hasn't been at the top of my to-do list recently; depression and a terrible job have really got me running on empty lately.


If you make an issue to make issues containing those items (and quote the
original piece) that would be appreciated...

I'm currently on mobile, sorry for the typos!

On Thu, 26 Nov 2015 7:05 am Etienne Bruines <notifications@github.com>
wrote:

> Good point; consistency.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EtienneBruines/engi/commit/3c57ece597094d816dccf15545941fc47fe2a1a2#commitcomment-14625858>
> .
>


[Default parameters](https://developer.mozilla.org/en/docs/Web/JavaScript/Reference/Functions/default_parameters) aren't supported by browsers other than firefox (and chrome) yet.


The config file is in kmip/
The certs are in kmip/demos/certs

In the current state, the tests and demos won't find the certs because they are looking in the wrong dir.


"PCI" matches to the PCI configuration in drivers/video/mxc/mxc_lcdif.c.


Do you steel think that  I should rewrite this part of code: remove unnecessary _given variables and check opt_X for NULL?


The diff on this leaves much to be desired...


Same question here. Is this method resistant to change?


Is this code-path still needed? If anyone really wants Single threaded decoding, can't they just set threads=1 ?


Ok, к следующему четвергу жду рабочую версию. Желательно, чтобы она не слишкол отклонялась от ТЗ https://drive.google.com/open?id=0B3B7rF2bqaJqUUVxMkRGQXNNdUk


"fsl,lcd" tells this node to use the kernel driver with the name "lcd". Which is drivers/video/mxc/mxc_lcdif.c.


The change on line 56 generates an error when compiling/installing pinocchio for me on several computers. It works fine with the old version of that line.


We shouldn't print anything here..


TeamCity pull requests :: netty [Build 749](https://garage.netty.io/teamcity/viewLog.html?buildId=3607&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5292, ignored: 21 Build time: 00:14:44


The convention for the name is to have the rotations done in the same order as the name. So: `ROTATION_ROLL_90_PITCH_68_YAW_293` means:
ROLL_90
PITCH_68
YAW_293

In order.

And this python snippet shows the rotation using it in this order: https://gist.github.com/lucasdemarchi/a3e4eb5a230cd741bc69


This pin mux group is defined in imx6qdl.dtsi


They should be gone now.


TeamCity pull requests :: netty [Build 750](https://garage.netty.io/teamcity/viewLog.html?buildId=3608&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 4 (4 new), passed: 3637, ignored: 12 Build time: 00:13:00
### Failed tests
```
io.netty.channel.ChannelOutboundBufferTest.testEmptyNioBuffers: <no details avaliable>

io.netty.channel.ChannelOutboundBufferTest.testNioBuffersSingleBacked: <no details avaliable>

io.netty.channel.ChannelOutboundBufferTest.testNioBuffersExpand: <no details avaliable>

io.netty.channel.ChannelOutboundBufferTest.testNioBuffersExpand2: <no details avaliable>

```


math is an empty dir. A git add of the tree did not include it yet the build refers to it. It works in my env as the dir exists. Soln: cull the math callout as per Martins change.

Same needs to be done for RM57 I believe.


Why this error? I don't understand.


thanks, great!

(for the reference: https://github.com/spring/spring/commit/8257ae0f2cd6551d4624aa994dbd94658f1288fd )


Comments can be useful, and I know that there are lots of resources that encourage beginners to write comments, but in my opinion, they are usually a code smell!

If you are using descriptive method and variable names, comments should almost always be unnecessary. So when you write a comment, look at your names and see if you can change them to make the comment unnecessary.


You're right. I am removing it.


Here's an example. What if you extracted a method called `human_vs_human` rather than using a comment?


Does this comment provide anything more than the method name?


I removed it already.


Haven't coded logic for that turning into a logout button yet. Will be fixed soon.

Sent from my iPhone

> On Nov 25, 2015, at 4:51 PM, mikenpeacock <notifications@github.com> wrote:
>
> Getting a EmptyStackException when pressing back on homescreen at launch.
>
> —
> Reply to this email directly or view it on GitHub.
>


This doesn't actually do anything. I forgot to take this out. The driver in drivers/video/mxc/microtips_lcd.c used to be a platform driver (just a general driver) but now I've changed it to a spi driver. So the driver is actually loaded a little further down in ecspi2.


You have to '39' cases


I am also not sure.


Is working, does work...как угодно, но не is works!


who pushed first? ;)
On 25 Nov 2015, at 21:28, Konstantinos Chatzilygeroudis <notifications@github.com> wrote:

> I removed it already.
>
> �
> Reply to this email directly or view it on GitHub.
>

� Jean-Baptiste Mouret
http://pages.isir.upmc.fr/~mouret/


Thanks.


Apparently "hog" is a catch-all pin mux group that is loaded by default. Here I'm only muxing the LCD reset pin. I'm not sure if this works yet.


Oh my, seems like we soon hit the record of the longest combined issue... There are two aspects in this issue: The actual MS compiler (or its emulation), and the MSVC build system. And what I get from the above, we should treat it as such.

I also find that documentation rather unhelpful:
```
-fms-compatibility-version=<value>
                          Dot-separated value representing the Microsoft compiler version
                          number to report in _MSC_VER (0 = don't define it (default))
-fmsc-version=<value>   Microsoft compiler version number to report in _MSC_VER (0 = don't
                          define it (default))
```
What I understand, the compiler would need to be `MSVC` with a high enough version `MSVC_VERSION` to both indicate being the MS compiler and in principle able to provide C++14. The problem there is the emulation-part of clang-cl. By reducing the emulated version with either of the two `-fms...` arguments, also C++14 gets deactivated since not yet being available at that time. But there is another flag to enable C++14 explicitly, some original `/...` MS flag, which does not change the MSVC_VERSION. But for that check, there is the `_cpp_version` macro.

The second is the MSbuild build system in conjunction with cmake. My original warning aimed at telling solely that if you're using cmake with visual studio, you also need to tell cmake about that other platform toolset to choose clang-cl, with the intend of users not complaining by mistake about hana actually not working with visual studio and stopping trying it. Clang-cl is somewhat new to most users (and myself), at least judging from the very few actually helpful google results.

Ultimately, there is another side issue: Kitware writes that instead of copying other cmake compilable projects to the own project, users should directly cross-reference to them. So, ideally, the whole `CheckCxxCompilerSupport.cmake` and `config.hpp` tests should not only prevent wrong usage for building Hana's own tests, but also when embedded in an actual user's code, which again would be compiled by some other users, not knowing immediately about the issue, or even about Hana.

Long story short: That's why I would advice to keep the build system warning in, just simplify everything down the development chain. But that more or less only applies to this very Visual Studio issue because it is so much wide-spread; covering all the two dozens cmake generators and build systems is, like you wrote, out of scope.

However, although I try to embed such tests and warnings in my own code, user conversations typically go like `doesn't work` - `what's written on the screen?` - `I should do XYZ` - `have you done that?` - `No, should I ?`.

Ninja: Yeah, did something like that, but here cmake unfortunately always finds the wrong compiler, linker or just isn't able to compile some cmake compiler test files. Will try that some time later.


@podviaznikov what exactly do you mean/want to add to perun?


What's RTL?


I think me, but don't worry. Everything is okay now!


Will we migrate to C++14? To avoid this kind of return type repetition. (I don't know if it is a good idea du to MSVC Support)


What are the new features?


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


We might want to make sure with another assert, that this block gets executed when calling the setter.


Is it possible to avoid using deprecated API?


Is there a reason to keep the getter and setter tests merged as a single test method?


We don't need action bar


@pieroit let's add the `@since 3.2.0` annotation here.


Fixed error message in https://github.com/QuantConnect/Lean/commit/a852d7690dc84339972609e66602bcf4c9f472f6


does not exist on msvc


I think, for first iteration it is more reasonable to use ToggleButton instead.


maybe we should provide a version for this guy too (the same we do a bit above?)


I'll expand a little on the "magic numbers" stuff. Is there a reason `Human` and `Computer` objects need to know about their order in the game? Or can the play order be handled externally to these objects?


is that a typo on line 51? should "bach" be "bash" ?


@ArnavMohan I havent tested anything. I didnt finish the rotateImage method at all and I'm not too sure if what I have at the moment is how youre supposed to do it either, so you can finish the rotateImage method.


TeamCity game of life :: dame [Build 7](http://localhost:83/viewLog.html?buildId=70&buildTypeId=Game_Dame) is now running


@evil-at-wow I'm getting the following error when trying to install the latest version of WotLK CMaNGOS:
mangos@SNOWFALL:~/Build$ make
Scanning dependencies of target genrev
[  0%] Building CXX object CMakeFiles/genrev.dir/src/tools/genrevision/genrevision.cpp.o
Linking CXX executable genrev
[  0%] Built target genrev
Scanning dependencies of target revision.h
make[2]: *** [CMakeFiles/revision.h] Segmentation fault (core dumped)
make[1]: *** [CMakeFiles/revision.h.dir/all] Error 2
make: *** [all] Error 2

Any clue how I can fix this?


I think DialogFragment is more reasonable here.


Always put strings in resources


Awesome commit! You're my hero.


"Refactoring" means changing code without changing behavior. It looks like this change might have caused the game to crash. If that's the case, what gives you confidence that your code changes aren't changing behavior?


Method is unused


Could you be more specific? What is the error message you get?


FlxAssets.hx:34: characters 0-47 : Type not found : ByteArrayData
Openfl 3.4.0
Lime 2.7.0


@pieroit this needs to be changed to `===`.

More over you can use one line?

    if ( isset( $this->raw_custom_field['export_type'] ) && 'xsd:date' === $this->raw_custom_field['export_type'] ) {
        ..

even better, to avoid reassigning the `$this->date_format` variable:

    $this->date_format = ( ( isset( $this->raw_custom_field['export_type'] ) && 'xsd:date' === $this->raw_custom_field['export_type'] )
        ? 'Y/m/d' : 'Y/m/d H:i' ) ;


Why this change is necessary?


TeamCity game of life :: dame [Build 7](http://localhost:83/viewLog.html?buildId=70&buildTypeId=Game_Dame) outcome was **SUCCESS**
Summary: Running Build time: 00:02:02


Are you sure this is reasonable behaviour?


I'm not sure if this is the most sensible approach, but I'm open to suggestions for other approaches.


No need for using regexes here. Urlparse can extract the hostname: https://docs.python.org/3.0/library/urllib.parse.html


`np.arange(days_in_window, dtype=np.float64)`


@pieroit try to avoid reassigning variables, use the ternary operator:

    $pickerDate  = ( empty( $date ) ? '' :  esc_attr( date( $this->date_format, strtotime( $date ) ) ) );


@pieroit look how this can be simplified:

    $timepicker = ( strpos( $this->date_format, 'H:i') !== false );


We're replacing d with the normalized angle bisect here. Might be confused with the argument d we pass into this function. Maybe we could use a different variable name? :P


Theme should be *.NoActionbar


@pieroit reformat according to WP PHP Coding Standards.


@namreeb I'm getting the following error when trying to install the latest version of WotLK CMaNGOS:
mangos@SNOWFALL:~/Build$ make
Scanning dependencies of target genrev
[ 0%] Building CXX object CMakeFiles/genrev.dir/src/tools/genrevision/genrevision.cpp.o
Linking CXX executable genrev
[ 0%] Built target genrev
Scanning dependencies of target revision.h
make[2]: *** [CMakeFiles/revision.h] Segmentation fault (core dumped)
make[1]: *** [CMakeFiles/revision.h.dir/all] Error 2
make: *** [all] Error 2

Any clue how I can fix this?

I've compiled and installed version 1.59 of boost. This is my CMAKE command:
cmake ../mangos -DCMAKE_INSTALL_PREFIX=../mangos/run -DPCH=1 -DDEBUG=0 -DBOOST_ROOT=/usr/local/boost_1_59_0/

NOTE: I am not sure if this has to with this commit, or https://github.com/cmangos/mangos-wotlk/commit/434d1334. I have left a note in both and will remove the wrong one at a later time.


If you input a url like this:
https://vimeo.com/107989648
into the sample app and set the quality to low a an exception is thrown:
`Application Specific Information:
*** Terminating app due to uncaught exception 'NSRangeException', reason: '*** -[__NSArrayI objectAtIndex:]: index 18446744073709551615 beyond bounds [0 .. 3]'
terminating with uncaught exception of type NSException
abort() called

Application Specific Backtrace 1:
0   CoreFoundation                      0x00007fff98bbfae2 __exceptionPreprocess + 178
1   libobjc.A.dylib                     0x00007fff90142f7e objc_exception_throw + 48
2   CoreFoundation                      0x00007fff98a8a554 -[__NSArrayI objectAtIndex:] + 164
3   MacSample                           0x000000010ac43698 __57-[YTVimeoExtractor URLSession:task:didCompleteWithError:]_block_invoke + 152
4   CoreFoundation                      0x00007fff98b15f7f __NSArrayGetIndexPassingTest + 575
5   MacSample                           0x000000010ac430ca -[YTVimeoExtractor URLSession:task:didCompleteWithError:] + 890
6   CFNetwork                           0x00007fff8a22d57e __51-[NSURLSession delegate_task:didCompleteWithError:]_block_invoke167 + 76
7   Foundation                          0x00007fff92c6b29c __NSBLOCKOPERATION_IS_CALLING_OUT_TO_A_BLOCK__ + 7
8   Foundation                          0x00007fff92c6b00c -[NSBlockOperation main] + 100
9   Foundation                          0x00007fff92c6a2fb -[__NSOperationInternal _start:] + 654
10  Foundation                          0x00007fff92c69de7 __NSOQSchedule_f + 194
11  libdispatch.dylib                   0x00007fff9765a33f _dispatch_client_callout + 8
12  libdispatch.dylib                   0x00007fff9765ef6f _dispatch_queue_drain + 754
13  libdispatch.dylib                   0x00007fff9766563b _dispatch_queue_invoke + 549
14  libdispatch.dylib                   0x00007fff9765dc87 _dispatch_root_queue_drain + 538
15  libdispatch.dylib                   0x00007fff9765da34 _dispatch_worker_thread3 + 91
16  libsystem_pthread.dylib             0x00007fff907c568f _pthread_wqthread + 1129
17  libsystem_pthread.dylib             0x00007fff907c3365 start_wqthread + 13`


ВТФ?
Вот что от тебя просят:
 "Пример 2
 Ввод:
 http://javarush.ru/alpha/index.html?obj=3.14&name=Amigo
 Вывод:
 obj name
 double 3.14"

А ты тестишь только "obj name" без "double 3.14". Халтурка-с.


Rotate image already programmed in Transform.rotate().


But in some case you have optional dependencies, and this is not handled by the current system, I am wrong? Would this be easy to implement, or do this require lot of work in the Pkg system ?


I am not sure, whether I completely like the idea of representing the currently centered object by a number. Regardless of this design decision, however, you could use the built-in modulo function here :)


@Anonymousx12 Hah look at that, the room tells who reported you, when they do. Nice. I'll add it soon, cheers.


TeamCity game of life :: dame [Build 8](http://localhost:83/viewLog.html?buildId=71&buildTypeId=Game_Dame) is now running


Yes, we should.
With the xcompiler user, doing a manual "sudo" has been beyond me to see what the installed version is to correct this Dockerfile.
Please do fix this up, if you can see the right explicit version to use.


@sunshineo Cannot reproduce your issue with the latest version. I created a folder called ```html``` with a static ```index.html``` file on the host system. Then I ran a container with ```docker run -d --name nginx -p 80:80 -v `pwd`/html:/usr/local/nginx/html:ro h3nrik/nginx-ldap```. The expected static page can be viewed in a browser. And the error log file is there and can be read with ```docker exec -it nginx cat /var/log/nginx/error.log```.


TeamCity game of life :: dame [Build 8](http://localhost:83/viewLog.html?buildId=71&buildTypeId=Game_Dame) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:07


TeamCity game of life :: dame [Build 9](http://localhost:83/viewLog.html?buildId=72&buildTypeId=Game_Dame) is now running


TeamCity game of life :: dame [Build 9](http://localhost:83/viewLog.html?buildId=72&buildTypeId=Game_Dame) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:07


/usr/bin/ld: ne peut trouver -lpinocchio
/usr/bin/ld: ne peut trouver -lpinocchio
collect2: error: ld returned 1 exit status
make[2]: *** [lib/python/pinocchio/libpinocchio_pywrap.so] Erreur 1

I think @jcarpent is more able to understand and explain the problem than me.


ah, more missed branding stuff. I will rever thtese to org.taulabs (they will be re-branded as well)


LGTM


In other places we called functions like this [`set-<name>`](https://github.com/ps-tuebingen/gravity/blob/master/mission.rkt#L80). By this convention you could call the function: `set-centered-object`


the indentation rules for this file are not defined


Yes.


Meant article in commit message, not admin


Any hope of adding this to HEAD?


TeamCity game of life :: dame [Build 10](http://localhost:83/viewLog.html?buildId=73&buildTypeId=Game_Dame) is now running


well, after the 'cd' has run, you should already be in the right directory, I can revert.


what is the benefit for this ?


TeamCity game of life :: dame [Build 10](http://localhost:83/viewLog.html?buildId=73&buildTypeId=Game_Dame) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:08


TeamCity game of life :: dame [Build 11](http://localhost:83/viewLog.html?buildId=74&buildTypeId=Game_Dame) is now running


TeamCity game of life :: dame [Build 11](http://localhost:83/viewLog.html?buildId=74&buildTypeId=Game_Dame) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:10


It's part of the branding changes for BrainFPV, still to go on another branch. Will take out, thanks


My solution was to remove the `270p` from the `availableVideoQuality` array because `filesInfo` quality key only had `360` but not `270` which is quite odd.


same as https://github.com/fujin/dronin/commit/b011f866f561f3b4089ab17a9887581b6cca32d3#commitcomment-14626609


how should this be indented?


TeamCity LabKey :: Trunk :: LabkeyTrunk :: CDS postgres 9.4 [Build 15](https://teamcity.labkey.org/viewLog.html?buildId=287607&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 16, passed: 12 Build time: 00:47:32
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Unknown, part of branding for BrainFPV package distribution. I assume so. :question: @mluessi @BrainFPV


I will extract to the branding branch anyway.


Maybe more informative comment/documentation? i.e., I know what "NML" is but probably most won't. Overall you might want to say "this module is based on XYZ of ... but adapted by ... for the case of Desmond input" or something along those lines.


@jardix22
```'element' in settings``` isn't necessary any more?


Hi there... Long time not seen.

I wish my post would be more helpful but the game crashes on loading a level:

http://www.wdlmaster.de/TRCL/log16.html

so I can't test a thing.


asfd


We discussed clarifying the comment on line 61 - analysis is in dimensionless numbers (units of kT).


These fixes are for changes to dev version of openfl released yesterday, are you using dev version of openfl and lime?


Yes, exactly, it was already fixed in [this commit](https://github.com/kraft001/railsbox/commit/1e01d43a92208c9b75c41b0f55adffa7187cde4e)


cayley.cfg? That's what the examples and everything are, anyway


Thanks for the clarification @mikemcquaid and thanks @vszakats for the PR :smile:


the OUTPUTDIR var is out of context here. It should be "${INPUT}/metadata" and then OUTPUTDIR could be used on line 21 and 27


Interesting!


As a result it use to work fine if you run twice cmake/generate, but not so much if you only run it once


* Great the using ... = ...!!


Prefer print( to print for python 3 compatibility.


Should _mkdir the OUTPUTDIR first, just in case it doesn't exist prior to this.


@rdoc101 lets move this to new store print-store and dispatch an action "PRINT:ACTION" instead


I suggest if the $INPUT is a file then the script could just skip it and give an error. The idea here is that the input is a directory.


Do you have a better solution than these ?


This commit message isn't clear enough. The changes are:
1. Check for the existence of project relationships before adding them.
2. Place api create calls in try {} catch {} statements
3. Move some uninstall MySQL procedures that were failing into SQL into PHP.


In an ideal world, the credentials file would be a path variable or something like that.

If used though, it should probably have permissions 600 instead of 644.


looks like method `present` was renamed to `finish`


@rdoc101 not sure if it would be cleaner to pass id?


@rdoc101 the printAction handler first declares store dependencies


TeamCity LabKey :: Trunk :: LabkeyTrunk :: Git postgres 9.4 [Build 811](https://teamcity.labkey.org/viewLog.html?buildId=287611&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 48 (1 new), passed: 242 Build time: 00:55:44
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.JUnitTest$RemoteTest.org.labkey.core.admin.test.SchemaXMLTestCase: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


Remove some unneeded apps.


Why are you defining an extra function which doesn't exists in Python?


We've only got one card and it's on mmcblk1.


Just trying to remove some unneeded stuff.


@oconnorr I don't think we should move to this version until some rather fundamental libraries have been ported to it, like ssreflect. Further, it doesn't build on Darwin yet.


TeamCity pull requests :: netty [Build 751](https://garage.netty.io/teamcity/viewLog.html?buildId=3611&buildTypeId=pulls_netty&guest=1) is now running


double /?


@rdoc101 get category data from category stores, eg issues then create list of titles by iterating over results similar to https://github.com/hrcnz/npa/blob/master/app/src/node_modules/components/list/CategoryList.js


What if we change the calling signature to return (string, error) and then return both out and err? That way, we can choose to ignore it in cases like the present, but other usages will be able to handle an error differently.


Raise the dim brightness to something that is still visible.


Is http:// the only valid web protocol?


never meant to publish it that way. oops

However, the delay I added on line 1215 is necessary in my AVR setup. If
this delay is not added, most of replies to the extended read are lost due
to bufferoverflows or reboots AVR.

Ideas on how to solve this in another way?

Sven

> Rolled back vscp_firmware.c/h from some happy hacking ;-) @SvenZ80
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/grodansparadis/vscp_firmware/commit/3bae4326c2d3256243966298c66e3da50e28dcf6
>


This is nice error handling :+1:


Hide the status bar. Removing SystemUI application will have the same effect.


do you really need to cancel other people's builds? CI will need to run anyway if you do open a PR


This, on the other hand, is silent failure. Why make this choice? Just curious, since you commented on it...


add_exceptions (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1972 of 1972 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)


It might be nice to give the user some instructions here, in code, as to how to run the program. If I ran it without an argument, that's a pretty cryptic error message...


This is for the signal level, right? If that is the case then this looks good. In the end (95% sure) we don't rely on the signal definitions as defined in SUSYTools for anything that we store in SusyNt


Hum, ok  it is the missing "operator.xor". I didn't know it is a c++ keyword.


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


You have to go to members from AndroidWhatever and find daafdroid / AndroidWhatever.
Here you can find the app.


Danke :) (Einrast-Indikator Benachrichtigung vielleicht noch mit 2. Bindestrich? ... schnell weg...)


I've reverted for the short term. I'll restore it once at least ssreflect and mathcomp are ready.


I'll leave some general comments here, since there's not any other good place to put them. I left you a lot of small-scale code review comments, but want to take a step back and talk about the big stuff.

I really like how methodically you approached this problem. It's clear that you thought about your solutions, wrote down a plan of action, and made efforts to explain them in your README and video. Those features seem to work.

But in focusing so closely on those problems, I think you lost sight of the bigger problem with this codebase:

> "...the code itself is a mess.  It’s untested and therefore unmaintainable.  It’s poorly‐written and inflexible.  This puts us in a difficult position because we have a number of features we would like to add and we’re hoping you can help."

I'd like you to revise this submission and approach code cleanliness and maintainability as methodically as you did the feature requests. Some things to consider:

- Does each class have a single responsibility? Does each one have a good name?
- Is the code easy or hard to change? Is there duplication?
- Are methods as short as they can possibly be? Do they have descriptive names?
- Do the objects in the code model the domain of Tic-Tac-Toe?

Take some time and give it another pass with these concerns in mind.


Cierra issue https://github.com/InspectorIncognito/server/issues/16


hj


I think, a customer shouldn't automaticalle get an order.

By the way, it should be List<Order> instead of just one order.


I wasn't sure if this is going to work at all on windows. I canceled two Win64 run only because they are clearly stuck. One of them was actually running for [50min+](https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.11230/job/ry2cj0bmpycwp0ml) the [other one](https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.11231/job/s35iy6fy7wchtkyf) was also at 38min.


add_exceptions (mbadams5) - 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)


What domain(s) did you test this with? I don't get any output on www.jana.com.

Might be nice to give the user some sort of status, e.g. "starting to scrape %domain..."


We shouldn't be constructing a `Comm` as the default argument.

I don't think the constructed `Comm` should be a global.

I don't think this should be a `classmethod` of either `PoolMaster` or `PoolSlave` --- it transcends that distinction (note that it checks the rank and instantiates the correct type).


Why isn't this `Batch`?


INDENT PROBLEM!


Wenigstens isses nicht 'Tüpografie'...  ;)


No problem.

Do this in the send routine instead.  Look at the Paris code
<https://github.com/grodansparadis/can4vscp_paris/blob/master/main.c>
around line 2634. Wait for up to a second for a frame to be sent due to
the same problem.

Keep up the good work!

Cheers
/Ake

Den 2015-11-25 kl. 21:59, skrev svenZ80:
> never meant to publish it that way. oops
>
> However, the delay I added on line 1215 is necessary in my AVR setup. If
> this delay is not added, most of replies to the extended read are lost due
> to bufferoverflows or reboots AVR.
>
> Ideas on how to solve this in another way?
>
> Sven
>
>
>
>
>
> > Rolled back vscp_firmware.c/h from some happy hacking ;-) @SvenZ80
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/grodansparadis/vscp_firmware/commit/3bae4326c2d3256243966298c66e3da50e28dcf6
> >
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/grodansparadis/vscp_firmware/commit/3bae4326c2d3256243966298c66e3da50e28dcf6#commitcomment-14626819>.
>

--
Be Hungry - Stay Foolish!
Åke Hedman, Grodans Paradis AB / Paradise of the Frog
Brattbergavägen 17, 82050 LOS, SWEDEN
Phone: +46 (0)840011835 CellPhone: +46 (0)730-533146
http://www.grodansparadis.com http://www.vscp.org


TeamCity pull requests :: netty [Build 751](https://garage.netty.io/teamcity/viewLog.html?buildId=3611&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 3637, ignored: 12 Build time: 00:08:37
### Failed tests
```
io.netty.channel.ChannelOutboundBufferTest.testEmptyNioBuffers: <no details avaliable>

io.netty.channel.ChannelOutboundBufferTest.testNioBuffersSingleBacked: <no details avaliable>

io.netty.channel.ChannelOutboundBufferTest.testNioBuffersExpand: <no details avaliable>

io.netty.channel.ChannelOutboundBufferTest.testNioBuffersExpand2: <no details avaliable>

```


What's an alternative :)


@rdoc101: maybe we should move the action-definition to a separate function so it can be used from the category-printing routines