Thanks for the notice, and thanks for starting up this project, it made me learn a bit of C++!


are you high


Uh, no, a mock wouldn't work either. You'd still need TEST_EXIT or a EXPECT_ASSERT or so.


Missing Javadoc `@since 5.3`.


Can you be more specific about the reason of the change?


Do we want Fedora specific configuration here?


Tu peux changer le nom de cette classe ?


Supprimer ce commentaire


Deleting


mydiv n'est pas un très bon sélecteur. tu peux pas trouver un meilleur nom pour cette classe CSS ?


Please @Zrp200  run `rspec` before you are ready to merge. Because now it does not pass.

https://travis-ci.org/szymon33/battleship/builds/113174402


MyVideo


commentaire à supprimer


espace avant et après le >


#FFFFFF = #FFF


Or chunk CSSLint entirely like we did in v4 (#17370).


你这真的能编译吗，不用 name={icon}？


i can't test it because the db isn't filled yet. but everything else works


That was not freeze in the first place.


Yes - I'm testing it locally - the other day I noticed some kernel panics because I was nulling the timer's function (which I then fixed), so I'll probably wait a few more days before I enable it - but it should happen Real Soon Now(tm)


Winkelangaben stimmen nicht mehr! Bitte ändern


add a description.


No, union operator adds values from the latter, ignoring keys that exist in the former.

```php
$config = [
              'class' => 'yii\i18n\PhpMessageSource',
              'basePath' => "default path",
              'forceTranslation' => true
          ];

$globalConfig = [ 'basePath' => "i want override this path" ];

$config += $globalConfig;

print_r($config);
```

output:

```
Array
(
    [class] => yii\i18n\PhpMessageSource
    [basePath] => default path
    [forceTranslation] => 1
)
```


Thanks @Godin, I will add it


I guess that could be made into something generic - but if Fedora infra folks don't mind I'd keep it this way. Or do you see other issues?


Correct... and that's what array_merge should do as well..


@kba Should'nt this direct to the github version of the script?


Reading a bit on the bug referred to in the comments, I now see what this is doing. I don't see any issues with this whatsoever, so I will merge it immediately, but with a different description, as it was confusing for me.


Sure, if you guys prefer that. I still believe it can catch easy mistakes.


@Richard-Yunchao & @qkmiao please help to review it, thank you.
https://android.googlesource.com/platform/external/deqp/+/master/modules/gles3/functional/es3fTransformFeedbackTests.cpp#1189


Nothing else from me, so :+1:


Let's stick to bbatsov ruby-style guide. Shell we?

https://github.com/bbatsov/ruby-style-guide#literal-array-hash


Same here. Rady the following link https://github.com/bbatsov/ruby-style-guide#literal-array-hash


@dbmeneses I already created PR #809


Yes


sweet :+1:


It'll be better if you can replace this with LogUtils.error(LOG_TAG, "message", e) instead. Same for other places.


I'll fix it tomorrow


This is not the case if `mc.Track` is true, but the positioning isn't relevant. Perhaps we could split the `mc.MouseX` and `mc.MouseY` assignments to a separate `if`-statement.


It is related with /deqp/functional/gles3/vertexarrays.html


Executors.newSingleThreadExecutor() creates an ExecutorService. Creating it every time is expensive. May be you can make on ExecutorService as a class object and have fixed number of threads? And use that here.


@danielpassos see my comments here: https://github.com/danielpassos/helloworld-ios/pull/1


array_merge() behaves differently:

```php
$config = [
              'class' => 'yii\i18n\PhpMessageSource',
              'basePath' => "default path",
              'forceTranslation' => true
          ];

$globalConfig = [ 'basePath' => "i want override this path" ];

$config = array_merge($config, $globalConfig);

print_r($config);
```

output:

```
Array
(
    [class] => yii\i18n\PhpMessageSource
    [basePath] => i want override this path
    [forceTranslation] => 1
)
```

We need array_merge(), because we want to overwrite $config (defalut) with $globalConfig.


Please revert / remove this commit.  This should checkout the code on your local machine and it does it using the `init.R` file.  On production there is a different method to call that file.


Get Linq data source interface


Remove "cur"


The `ResourceType` doesn't use this. It's print will always put quotes around the title
```
  "#{capitalize_segments(t.type_name)}['#{t.title}']"
```


I agree with the thought (that one should only enable it if needed), but looking at the code: there's almost no performance impact.


IMO, we can't remove the old version like this; it could leak to breaking backward-compatibility.


RQL查询语言在这里已经可以兼容Linq数据源了

This method makes the RQL data source query language compatible with Linq data source


why isn't this just `repositionTimer = setTimeout(scrollHandle, 100);` ?


Add a Linq data source interface at the storage technology descriptions


Other than minor comment above, LGTM in testing; scroll activity is nicely debounced now.


This line adds a bug "one char per column" in CSV output.


it is good !


Was just presenting more options. I don't speak for anyone besides myself.


@tvsonar how will this work when C# and VB.NET won't share the same `SonarAnalysisContext` because of different versions?


ok


LGTM, I will test it and tell you the results.


This is not the way to go, because this breaks the action for all other entities that might use `MouseComponent`s.

The idea is that every Entity (object) can add a `MouseComponent`, which will be filled with useful mouse information w.r.t. that same object. When you change something within `Mouse` (which is the information set by the system, once per frame), it has consequences for all other `MouseComponent`s that might have to be processed.


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


Still missing space before last }


这是第二次测试


Oh I was just stealing the class template from somewhere else (some of the other toXYZ functions). So we might want to fix that there as well.


So you want to keep testing 0.12?


@xyjwsj thanks for reporting, it should be fixed with 1fb0564


Just curious: what is the advantage of `setCustomBaudrate()` that you have implemented? Why the same functionality cannot be implemented using the existing API?


hi how can i use this cheat on cs go ?


czy to nie powinno pójść do jakiegoś confingu?


same here, is there going to be a single `SonarAnalysisContext.ShouldAnalysisBeDisabled` when the C# and VB.NET analyzers will be at different versions?


We should have a new CCLint version soon which has the ability to disable the check for a single line. Currently, we can't do that.

I guess we can ignore the issue for now since the error doesn't block the build anyway (which seems like a bug to me but that's another issue :) )


Hi, the order of the list bullets were changed, Xpages now is the 3rd one after Liberty and Nodejs.


Yes


@Zrp200 Ruby style. Don't use parallel assignment like this https://github.com/bbatsov/ruby-style-guide#parallel-assignment


How to clear cache? i did `npm cache clean` but getting the error . Unable to run `heroku login`


Wow @tpaviot , thanks for the instantianious resolution...


To make it more clear: OK ;)


If we want to change it after processing all `MouseComponent`s, there's the `Post()` method we can use:
```go
func (m *MouseSystem) Post() {
    Mouse.Action = 99
}
```

**However**, looking at the callbacks we receive (`engi_glfw.go`):
```go
	window.SetCursorPosCallback(func(window *glfw.Window, x, y float64) {
		Mouse.X, Mouse.Y = float32(x), float32(y)
		Mouse.Action = MOVE
	})

	window.SetMouseButtonCallback(func(window *glfw.Window, b glfw.MouseButton, a glfw.Action, m glfw.ModifierKey) {
		x, y := window.GetCursorPos()
		Mouse.X, Mouse.Y = float32(x), float32(y)

		if a == glfw.Press {
			Mouse.Action = PRESS
		} else {
			Mouse.Action = RELEASE
		}
```

This is set pretty much every frame. We could add another `Action` called `NEUTRAL` or something, within `input.go`. Then we could indeed reset the value to neutral within the `Post()` function.

Good catch! I didn't realize (when writing the callbacks) that they would need resetting.


Shouldn't it be possible to keep the old methods? And extract the default sound categories from the `sounds.json` file `https://bugs.mojang.com/secure/attachment/102693/sounds.json`.


Should I add 4 and 5 or just keep 0.12 then?


nadaram ke male khodam bashe male moshtarie ziadam faza nadare. mikharim 500 meg baseshe?


Since this is only available for sql cases you could just do `CaseAccessorsSQL.hard_delete_cases`


It seems like most of your code is written in this commit. Why is that?


>Which deletion?

Is a revert appropriate for the commit efea8c08299f5034cfc1ab1442707696ba0b78d8 (questionable removal of a call for the script "latexmk")?

>Appropriate or not I don't know …

Why did you commit such a change if your knowledge was incomplete for the software dependencies that are involved for the desired safe documentation generation from TeX source files?


case, then (missing comma)


LGTM. C++ code should be at:　https://android.googlesource.com/platform/external/deqp/+/master/modules/glshared/glsVertexArrayTests.cpp#1189


Hi @kd7tck ! Actually, this issue is not that critical, it only happens for OpenGL 1.1 path (by default raylib uses OpenGL 3.3) and mipmap generation must be called by the user with `GenTextureMipmaps()`. Actually, at some point I plan dropping OpenGL 1.1 support.

But, ok, let me verify current develop branch and I'll push changes to master.


ack


mvp


it's not solved yet I had to regenerate all SWIG files. It will be fixed in a few minutes


Add link to live site.


Finally, as we have now essential settings in prod;py, we cant run with sole mysite.settings, so yeah, this diff is ok
if you move li13 back to li10


I don't know. Why was the change made?


You're right. This is not the fastest way. I added StringIO test cases to the ruby-performance repo. Best case, StringIO is about 30% slower than just appending to a string.


**Nitpick**
You are including a lot of mixins here. While mixin might seem like a silver bullet, it is not. Check out this links to learn more about mixins and form of reuse in ruby OOP. I and the community recommend composition.
- [Mixins Pitfall](http://definingterms.com/2013/03/23/pitfalls-of-ruby-mixins/)
- [OOP and Reuse](http://naildrivin5.com/blog/2012/12/19/re-use-in-oo-inheritance.html)


This is the same as `>=0.6 <1.0.0`


ok, so we keep it as it for now


Last bits are cannot be stored because 0xffffff / 64. = 262143.984375
So you need to add +1 uint64 in your mask


Actually it is not `topologyKey`, `node.Labels[topologyKey]` is a label value.
Maybe a name `topologyVal` or `countIndex` is better.


@oridb  -- on track and plan to add them once we test them.


Its a while ago when I did this. The problem was in setting non standard baudrates. When I tried to set a custom rate, I always got an EINVAL. The newer termios2 supports setting non standard baudrates - and it worked on my end.
The blog article [1] describes the issue in more detail.

[1] http://www.downtowndougbrown.com/2013/11/linux-custom-serial-baud-rates/


Does not seem to be any kind of issue tracker so submitting it here...

The allocating of lines for the commit graph in PlotCommitList does not work correctly...

Sometimes lines are re-used when they shouldn't be.
Kris


This solution will not work for snatching torrents/magnet files, through the manual search. For this I have implemented an Auth Hook. I'm finetuning this. The idea is that all the cookie checks will be removed from login(), as login is just supposed to login.

Hi i'm using this commit to create some awareness: @pymedusa/developers, and to see if my change will also help you @medariox

Then a new dict will be added to GenericProvider, which you can use to configure your provider. For ex. if you know that the provider needs 4 cookies, and you want to check for expired cookies. You could set this:
```self.auth_hooks = {'min_amount_cookies': 4, 'check_cookie_expired': True}```

If you want to do something really fancy, you can overwrite and place the method the following method in your provider:

```
    def _custom_auth_check(self):
        """ Use this function if you'd like to have some control over the authentication checks
        before searching the provider or snatching a result. The _custom_auth_check is hooked through _get_auth_hook()
        to the helpers.getURL() function.
        """
        print('Do some fancy stuff here')
        return
```

The above function will also only be run just before it tries to perform the GET or POST.


Shouldn't the correct syntax be using the [getLogger](https://docs.python.org/2/library/logging.html#logger-objects):

```python
social_logger = logging.getLogger('social')
````

Using Django, for me it logs `No handlers could be found for logger "social"`

Whereas it's configured in my `settings.py` (among other things):

```python
LOGGING = {
    'version': 1,
    'disable_existing_loggers': False,
    'filters': {
        'require_debug_false': {
            '()': 'django.utils.log.RequireDebugFalse'
            }
        },
    'formatters': {
        'simple': {
            'format': '%(levelname)s %(message)s'
            }
        },
    'handlers': {
        'console': {
            'level': 'INFO',
            'filters': ['require_debug_false'],
            'class': 'logging.StreamHandler',
            'formatter': 'console',
            }
        },
    'loggers': {
        'social': {
            'handlers': ['simple'],
            'level': 'WARNING',
            'propagate': True
        }
    }
}
```

I have other packages configured that way, and it works.


No it's not. `^0.6` means `0.6.*`. `~0.6` would mean `>=0.6 <1.0.0`.


As discussed this, we'll cross that bridge when we come to it. But it would be best to keep that common part common between C# and VB.NET


0.x behaves differently because `^` strictly respects semver, and minor releases in 0.x can be breaking.


**Critical**
`encrypt_char` and `decrypt_char` are very similar methods and can be refactored to DRY things up. One way is to call `encrypt_char` inside `decrypt_char` like this
```ruby
def decrypt_char(char, pos)
  pos *= -1
  encrypt_char(char, pos)
end
```


コンボ数の表示 #283


I've made a pull request. Since Django supports just putting a named url there, I think it is better to support this also in django-two-factor-auth


- Where is the usage?
- This place is for sd card images


```go
	var maxCount int
	var minCount int
	counts := map[string]int{}
```
Better to move the definition down to here, since they won't be used until the conversion.


Would be amazing if you can get it to work!


`hljs.initHighlighting should not be used in conjunction with `reveal-code-focus`, which is mentioned [here in the Gotchas slide](https://demoneaux.github.io/reveal-code-focus/#/2/5).


There should be a blank before `catch`.


@EduFdez !! Cuando veas que está listo, me abres un pull request, ok?? :-)

Que en pocas semanas quisiera sacar la release 1.4.0... de hecho, a menos que veas que va para largo, he añadido [un ticket](https://github.com/MRPT/mrpt/milestones/Release%201.4.0) para esperar a que esté PbMap.


The assert() I tested was more sophisticated. With plain assert(), the mock() method won't work.


@oetiker, while it makes sense to run the recurring only when not running once, it should also be in place, however, when not daemonizing: the effect of not being able to SIGTERM the process even when not daemonizing is still there!


You might be interested in [custom fragments](https://demoneaux.github.io/reveal-code-focus/#/3/3) which only display the current fragment.


i think you should use this link instead http://coala.readthedocs.org/en/latest/Users/Tutorials/Git_Help.html#getting-to-work , as it points exactly to the adding a remote part.


At least we hopefully only need to fix it in one place if we've caught all the offenders until then...


**Critical**
The engine class is doing 2 major things. It handles encrytion and decryption which is against `Single Responsibility Principle`. One way to adhere to this principle is by using delegation. Check out this blog post
- [How to delegate like a Boss](http://goodheads.io/2016/02/24/how-to-delegate-like-a-boss/)


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


OK, this is getting better. A few comments, on MQTT test tray to close the client (`client.close()` or something similar) just before calling `done()`.

On 'Request method bolck' if you want to create the server before running the test use the statement `before()`, like:
```javascript
describe('hooks', function() {

  before(function() {
    // runs before all tests in this block
  });

  it(function() {
    // runs a test in this block
  });
});
```


@ChALkeR where do you get the styles? and why newline at eof is bad? as for me, it confusing, because in [another thread](https://github.com/qmlweb/qmlweb/pull/84/files#r54847053) you tell: "add newline".. so we have 2 contradict same-time styles turned on?


Was the format param intentional, here?
I vaguely remember a decision to leave them out... As most requests would be JSON.


i dont think the first comma is gramatically correct. imo you should use the second one only


**Critical**
`decrypt`  and `encrypt` method are very similar and should be DRYed up.


you should call getDfp outside so it doesn't get executed twice


see above..


See above..


Yes, it was changed here: https://github.com/wtsi-hgi/cookie-monster/blob/develop/setup.py#L4 after I noticed pip complaining that "the egg does not match the fragment".

It is convention for the "fragment" (package name) to be lower snake case but, by looking at what I've got installed using `pip list`, there doesn't appear to be any standard for naming the egg (the python "jar" so to speak). For simplicity, I've decided to use the same names for both.


Jasne, mozna przeniesc do konfiguracji :)


why did you remove all the OWL2 java files?


That's not a newline in eof, I'm talking about this one:
```
    }
>
  }
```

Line 7, the one that I commented (just above the comment).


*** Code Quality: Nitpick***
You should always endeavor to delete your commented out code in your production ready app


Actually the doc says it returns

```
 {"hasErrors" : false,
         "results" : [{
            "referenceId" : "ref1",
            "id" : "001D000000K0fXOIAZ"},{
            "referenceId" : "ref2",
            "id" : "001D000000K0fXPIAZ"}]
}
```

but it looks like it is returning just the content of results.


It didn't work when i was testing without it, eg:
`php tbg_cli api:list_issuefields someproject task`


***Code Quality: Nitpick***
While this is a good use of localStorage angular has a well known method of $window.localStorage which does much of the same thing that the localStorage does in the angular context.


**Nitpick**

This module can easily be converted to a class maybe called `FileDataStore` which will be using when composing other classes as against using mixins.
Once this is done, you should be able to remove `file` in the name of your methods.


I found the devil in the details...
Line 309 -> Line 316 in Mob.cs
Nice one


It was originally suggested by @olim7t : I think the idea is to provide each JVM with a random integer to use as a fake PID. E.g. you could have a shell script that spawns several JVM instances in a for loop, each one being assigned a "PID" of 1, 2, 3 etc. It doesn't really matter if it's not the real PID, as long as each JVM as a distinct number, UUIDs generated by them will not collide.


a


**Nitpick**
`puts` should only be used in your `cli`.


Yeah, sure


Espero tenerlo listo a lo largo de la proxima semana :-)

Eduardo

El 3 de marzo de 2016, 10:28, Jose Luis Blanco-Claraco <
notifications@github.com> escribió:

> @EduFdez <https://github.com/EduFdez> !! Cuando veas que está listo, me
> abres un pull request, ok?? :-)
>
> Que en pocas semanas quisiera sacar la release 1.4.0... de hecho, a menos
> que veas que va para largo, he añadido un ticket
> <https://github.com/MRPT/mrpt/milestones/Release%201.4.0> para esperar a
> que esté PbMap.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EduFdez/mrpt/commit/9a309cc0a9f55d97d0007049bdc9e0c35314b581#commitcomment-16468062>
> .
>


I don't know why this thing happened. The files appeared as deleted locally on my machine for a long time whenever I was creating a new branch, but this deletion was never pushed to github. Could you restore them?


Oh, so maybe directly linked to the cli part?
Did not test that yet..


Can you help me remember how to decode the alphabet soup in `MOVBLZX`? I assume this is what Intel calls `movzx` (move with zero-extend) and that the `L` means a 32-bit destination, but what is the `B`?


***Code Quality: Nitpick***
window.alert() affects user experience. You should try and implement another popover notification either using bootstrap popovers or material design dialog.


## This is a markdown file.


Really? The thread there is not registered with the SafepointManager and should never appear (it's not a "Ruby thread"). What would be the value of the `thread` variable?


```c++
// assertions.h (simplified)
#define ASSERT(test_) \
    ((test_) ? (void)0 : onAssert(&l_this_file[0], (int)__LINE__))

// test double for onAssert()
void onAssert(char const * const file, int line) {
    mock_c()->actualCall("onAssert")
            ->withStringParameters("file", file)
            ->withIntParameters("line", (int) line);
}

// useful.c - production code
void my_thing(void * other_thing)
{
    ASSERT(other_thing);
    // do useful stuff with non null other_thing
}

```c++
TEST(useful, assert) {
    mock().expectOneCall("onAssert")
          .withParameter("file", "useful.c")
          .withParameter("line", 4);
    my_thing(0);
    mock().checkExpectations();
    mock().clear();
}
```


I would probably change the help message. Something like "Use only cache to display timeline." Not refreshing the cache is more a side effect than the intended behaviour.


this means we start all the services a fresh many many times. shouldn't the services be started only once (and possibly restarted if they stopped for some reason)?


One to discuss face-to-face - this commit fixed the issue of not using latest data for me. If there's a different method on production is this even needed? Again, one to discuss.


It could probably be removed from routes and appended to remote url in cli commands.


Possibly somewhere inside `$this->getRemoteURL('api_viewissue', $url_options)`
You pass in "format" as part of the `$url_options`, but maybe it won't be added if its not part of the url?
If you want, I can do some testing later today...


Wow, it is alive!
@egorbunov


You are right. I should format codes when I complete codes.


projectに関連したimageを保存するためのテーブルを別に作ろう


done


Any reason to limit this to 5 subdomains? Why not using `Split`?


@wjwwood I saw on rviz that you were the one touching the .mesh files. Maybe you could give me a tip about the offset here?


Hi hobu,
You are right!
After reverted line 137. I can get correct result now. By the way, my platform is Kubuntu 14.04 64bit. Please ask me for more information or testing if this is not enough.
Sorry for late reply, For the last time, I've been overloaded and forgot this thread.


I think that's the case. It would be great if you could test it as well. :)


Hi, could you rename this file to be test.log in the test script and here too ?


@wjwwood I saw on rviz that you were the one touching the .mesh files. Maybe you could give me a tip about the offset here? :)


@artokai Just out of curiosity.

At MockConfigurationProvider.ts line 18 you're just returning the Promise directly, while on configuration.ts Line 29 you're initializing the promise before returning it. Any particular reason?


Why the `newVersion()`?


I can create global variable "test_setting_id"
url = "students/test_settings/" + test_setting_id + "/tests/finish_test"
but you said "Try to avoid global variables"


faviconとりあえず今はいらないので消しちゃいました。


@jwgrenning  The ASSERT() implementation is a generalization of Q_ASSERT() & friends, which is part of the QP framework originally (http://www.state-machine.com/). It is quite easy to implement with one header file and can report either filename / line or (with a slightly different macro) module name and id (which is better suited for testing). The above outlines the basic idea well enough, I think.

Personally, I believe that for a contract-based implementation this kind of ASSERT() macro is preferable because you are probably going to have quite a few assertions and you want to be sure not just any assertion triggered, but the right one.


We also need to relax (delete) these lines in Sanity.h also:

~~~~
      #if ENABLED(Z_MIN_PROBE_REPEATABILITY_TEST)
        #error Z_MIN_PROBE_REPEATABILITY_TEST is not supported with DELTA yet.
      #endig
~~~~


Can't you just compare `SI` and `R13` here?


Yes, that is true. Then we can `@limit = Integer(limit || 20)`


***Code Quality: Critical***
```Use a capture variable for this when using the controllerAs syntax. Choose a consistent variable name such as vm, which stands for ViewModel.

Why?: The this keyword is contextual and when used within a function inside a controller may change its context. Capturing the context of this avoids encountering this problem.
```

  ```javascript
  /* avoid */
  function CustomerController() {
      this.name = {};
      this.sendMessage = function() { };
  }
  ```

  ```javascript
  /* recommended */
  function CustomerController() {
      var vm = this;
      vm.name = {};
      vm.sendMessage = function() { };
  }
  ```

Source [Google Angular StyleGuide](https://github.com/johnpapa/angular-styleguide/blob/master/a1/README.md)


all is fine here, just use `string interpolation` for coffee scripts
like
"#{$('.test-form').attr('action')}/finish_test" instead of `one_var + "some_string"`


Re-using the flags is really cute. Nice trick.


thank you. I have several observations.

A. calling `onPause()` does not make memory consumption any better however I think it is good idea to call it anyway.

B. simple calling `onXYZ` in `AirMapView`'s `LifecycleEventListener` leads to problems (as you have already predicted :)) It seems that this listener creates leak itself since it is never removed from react context (I saw multiple "onPause" events even though my application have max 1 map instance at a time). I will try to clean the listener up.


*** Code Quality: Nitpick ***
This function should have been delegated to a service to allow for more re-usability of your code base. This is a good place of using a .factory service


Whoever xylo is, he made my day with this commit.
Getdown rulez!


This comment can be removed.


If it was to prevent `node_modules` from being linted, it could have been fixed (`ul` lacked).


"#{some_variable}/finish_test" in coffee file doesn't works. It works in
javascript:
  $(document).ready(function() {
    "#{current_user.id}"
  });


***Code Quality: Awesome***
Nice use of ng-switch directive.


@valib I think we should always let the renderer do as much as possible so if it supports the subtitles we let it


Hello there, is this working as expected? Can you write some documentation as well? Is blade (extending) limited by this change. Thanks


This will restart the service every 5 minutes. Shouldn't we check if the config changed before restarting the service?


Does `create` (and probably `update`) need to be updated as well?


***Code Quality: Awesome***
Excellent use of the filter directive and a seamless combination with your ng-repeat


you are right.


Musisz stworzyć maina, jak chcesz, żeby GUI działało. Alt+Insert->Generate main()


**Nitpick - Code Quality**
We can have this here instead
```
@limit = Integer(limit || 20)
@limit = 100 if @limit > 100
```


It seems the bug is caused by the buggy `context.getThreadManager().getCurrentThread()`.


Test  from remote 332016


arf ... strict aliasing warnings are about the worst ones to care about.
Most of them are false positives by the way.

Problem is, these warnings are set by default within FB's compile farm, so it's important to take care of them.

I'm a bit surprised it causes problems here, because you don't do anything with these pointers. No read, no write, just set them. Usually, it works fine.


Site is down


Oh duh, I forgot that `src` is `[]byte` not `[]uint32`. This is "load 8 bits into a 32-bit register, zero extending".

I think you've got a bug though: you want `MOVBQZX` to ensure that the upper 32 bits are also zero, right? I'm not positive that the code below preserves that invariant. (Maybe you have some proof?)


How come? Is someone (cordova-lib) passes `events` instance, you just ignore it?


***Code Quality: Awesome ***
Excellent use of ng-include to keep your Code DRY


Same here


The complete implementation of the assertions looks like this:
```c++
/*!*************************************************************
 * @file
 * @brief  General purpose assertions for use in production
 *         code (simplified from Q_ASSERT_ID(). These assertions
 *         can be removed entirely be defining NASSERT.
 ***************************************************************/
#ifndef __ASSERTIONS_H__
#define __ASSERTIONS_H__

#ifdef NASSERT /* NASSERT defined--assertion checking disabled */

    #define DEFINE_THIS_FILE
    #define DEFINE_THIS_MODULE(name_)
    #define Q_ASSERT(test_)             ((void)0)
    #define Q_ASSERT_ID(id_, test_)     ((void)0)

#else       /* NASSERT not defined--assertion checking enabled */
/**
 * @brief Macro to specify the current file name (including
 *        absolute path) to the assert() function.
 */
#define DEFINE_THIS_FILE \
    static char const l_this_file[] = __FILE__;

/**
 * @brief Macro to specify the current module name
 *        to the assert() function.
 */
#define DEFINE_THIS_MODULE(name_) \
    static char const l_this_file[] = name_;

#define ASSERT(test_) \
    ((test_) ? (void)0 : onAssert(&l_this_file[0], (int)__LINE__))

/**
 * @brief Macro to call the assert function with an ID rather
 *        than a line number (which may change over time).
 */
#define ASSERT_ID(id_, test_) \
    ((test_) ? (void)0 : onAssert(&l_this_file[0], (int)(id_)))

/**
 * @brief Assertion handler to be defined by the application.
 */

 void onAssert(
    char const* const file,      /**< file-/modulename as text */
        int line       /**< line numer or ID in numeric format */
    );

#endif                                              /* NASSERT */

#endif                                     /* __ASSERTIONS_H__ */
```
I'd say it's pretty powerful, and of course, `onAssert()` can easily be mocked and tested. I prefer working with ASSERT_ID() for testing, because line numbers change all the time.


A few notes:

1. Please rename `subscribeToSelfOrForwardToExternalEvents` to something more readable (maybe `setupEvents`?)
2. `subscribeToSelfOrForwardToExternalEvents` is called only from `Api.js`, IMO it'd be better to keep it there as module-level function.
3. Could we get rid of `this.events` at all and just require `cordova-common`'s events when necessary?
4. is `adjustLoggerLevel` is something that we need in `CordovaLogger`, what do you think?


***Code Quality: Nitpick***
You should avoid the use of confirm prompt and alert as it has adverse effect on user experience


Typo for `its`


Oops!


Compare `SI` to `R13`?


at last something I understand!


No, we actually do need the original code. :-)  With the latest TP branch of Kafka this new example code won't compile any longer.  I suppose you tested locally against an older version of the Kafka branch.

```
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.3:compile (default-compile) on project streams-examples: Compilation failure
[ERROR] /Users/mnoll/git/miguno/examples/kafka-streams/src/main/java/io/confluent/examples/streams/UserRegionLambdaExample.java:[70,20] incompatible types: org.apache.kafka.streams.kstream.KTable<org.apache.kafka.streams.KeyValue<java.lang.String,org.apache.avro.generic.GenericRecord>,java.lang.Long> cannot be converted to org.apache.kafka.streams.kstream.KTable<java.lang.String,java.lang.Long>
```


Throw a PHP Fatal error...
```
PHP Fatal error:  Call to a member function getSteps() on null in /Users/<name>/src/github.com/Behatch/contexts/src/Context/DebugContext.php on line 100

Fatal error: Call to a member function getSteps() on null in /Users/<name>/src/github.com/Behatch/contexts/src/Context/DebugContext.php on line 100
```


Qué quiere decir GCM? poner tal vez un nombre de función más genérico. O poner más documentación.


This is where the metaphor breaks, because a real beer has multiple ingredients, and real ingredients are each used in different  beers. I don't think that's true of these components.


Looks good to me


helloworld


Ok. thanks for noticing.


Isn’t a beer factory what we call a ‘brewery’? So what’s the difference between a brewery and a supplier?


overwrite


***Code Quality: Critical***
Your ng-click should be pointed to a function experession not an evaluation. as this can lead buggy code. and might not be easily debuggable.


Must be some issue while merging. I'll check this tonight and fix.


correct


No use of this


Typo for `its`


Not objecting, but what's the rationale? Does this often happen?


setting case properties directly on the case should only ever be done in tests. In production it should be done via a form. Refactored to make this clearer here: https://github.com/dimagi/commcare-hq/commit/29798d0f3bb9425f00f5484b83ac7c30af5f5c17


As far as I can tell this can never be true, even in the Go code, unless `int` is 16 bits, right?


I think `resolve` and `resolveOptional` would be better names; `get` in names is usually a code smell.


I still believe we need to check the full stack frame, so if you agree I'd rather leave here the call to `svc_cx_validate_sf` and re-implement the function `svc_cx_validate_sf` to call the intrinsics. Also, please note that the function [already performs the unprivileged access](https://github.com/ARMmbed/uvisor/blob/66103b91ea7a21e8bd7cf272a3a9aafb3d7e39a2/core/system/src/svc_cx.c#L36) but with `asm volatile` instead of intrinsics.

I believe we should not use `vmpu_unpriv_uint32_write/read` for the stack tampering. All writes to the destination stack use a pointer (`dst_sp`) that is trusted by uVisor (we maintain that stack pointer).

OTOH the source stack pointer is taken from the user space (`__get_PSP()`) and should be checked, but we do that with `svc_cx_validate_sf(__get_PSP())`. Since the verification has been done, I also claim we do not need the unprivileged read for `src_sp[7]` in the line:

```C
dst_sp[7] = src_sp[7] | (dst_sp_align << 9);
```

as the pointer has already been validated.

If for any reason the unprivileged read was needed, we should re-shuffle our MPU switch:
```C
/* This can be read now because we are technically in the context of src_box. */
src_sp_7 = vmpu_unpriv_uint32_read((uint32_t) (src_sp + 7));

/* Switch MPU configurations. After this we are technically in the context of dst_box. */
vmpu_switch(src_box, dst_box);

/* Tamper the destination stack frame. We can because we switched contexts (unprivileged access). */
vmpu_unpriv_uint32_write((uint32_t) (dst_sp + 0), a0);
vmpu_unpriv_uint32_write((uint32_t) (dst_sp + 1), a1);
... /* Etc. */
/* For this line we need to use the previously stored src_sp_7, which otherwise would not be
 * readable here (we are in the context of dst_box; src_sp[7] belongs to src_box. */
vmpu_unpriv_uint32_write((uint32_t) (dst_sp + 7), src_sp_7 | (dst_sp_align << 9));
```


Seems I also fixed some whitespace \o/


initKeyboard


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


oops!


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


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


You could  also simplify this with what I posted above:
```
return iot.createKeysAndCertificateAsync(params).then(function(data){
  var lineBreak = /\n/g;
  var pattern = '#####';
  return {
    cert: data.certificatePem.replace(lineBreak, pattern),
    privateKey: data.keyPair.PrivateKey.replace(lineBreak, pattern),
    certificateArn: data.certificateArn
  });
});


hmm... Which exception is being catched here?
I actually don't understand this. verify_signature_not_encrypted_msg returns a deferred (and it has an errback that will raise an invalidsignatureError).

another thing I don't quite get is, how's possible that this function will return either a result (tuple of data, signkey) OR a deferred?


It's understandable to be burned out from too much coding in a day. You should definitely take another crack at test coverage when you're able, though.


Same here


And here


...and here :)


sure @Ceilican, you can include me.


Fix also rooms **mng** path.


@jmunhoz I think we should only do that if the error is -ERR_NO_SUCH_BUCKET


Won't this be problematic if a UpnpHandle was created via new, but UpnpManager::start wasn't called?

I guess what I'm trying to say, is that shouldn't this be checking that self.client is non-zero and self.cookie is not ptr::null_mut() ?


You could  just `return` this and no need for the `new Promise` above either.


isn't it clearer if the call to verify_signature_not_encrypted is done as a callback at this point?
https://github.com/pixelated/leap_mail/blob/f3b054c6a9f4abd63d66766c60f264dc3a2a0722/src/leap/mail/incoming/service.py#L469 (explaining the reasons stated above)


you can use switch statements here


well, that's why you'd have a placeholder. it gives you a reference point, so you could for example add F# and C# after that, and add yours before it.

see also https://github.com/mono/mono-addins/commit/490ca9b6e92185e2fcbeca8b7056a7edfd8e0bb2


This can probably replace basicVarDeclStmt


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


`token_set = tokens[i:i + 3]` - We're observing space between arithmetic operators for this class (pep226). This one is nit-picky, but your linter should be flagging this. This avoids code like `tokens[[i:i+3]j+2*5]`.


django-registration-redux: https://django-registration-redux.readthedocs.org/en/latest/quickstart.html#quickstart

@davidhadrianus89 @novikesumaningtyas please run:
* pip install -r requirements.txt
* python manage.py migrate


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


When calculating window base, make sure you don't have any off-by-one errors. Cases to consider: what happens when you receive an seq num in an ack that is less than/equal to/bigger than the window base, etc...


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1577 of 1577 tests passed, 0 test warnings)

![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.84%25-green.svg)


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


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


dddddd


We have an entire last week to consolidate, clean up and UX enhancement. We can do this at the end


A suggestion to keep subject/object number consistent, and to make it simpler/easier to read. If it's important to mention HTML document, feel free to put it back in. But it seems like that's clear from the context.

Custom attributes may contain HTML tags. It is the client's responsibility to escape or encode this data before displaying it. Use cross scripting [best-practices] (https://www.owasp.org/index.php/XSS_(Cross_Site_Scripting)_Prevention_Cheat_Sheet).


@rlorenzo Can you please review my code now


@rlorenzo Can you review my code


yes thats a good thing to keep the module updated :+1:


ddasd


Yeah, we could also add a `type` as part of the event. I think we can use trick `e.constructor.name` to get the type name


Line might be too long?


I'm reading the ```poll``` man page and I see this text:

> The field revents is an output parameter, filled by the kernel with the events that actually occurred. The bits returned in revents can include any of those specified in events, or one of the values POLLERR, POLLHUP, or POLLNVAL. (These three bits are meaningless in the events field, and will be set in the revents field whenever the corresponding condition is true.)

So, I don't think you care about POLLIN, and setting POLLHUP is meaningless. I think it's possible that pdf.events could be set to 0.


Use single quotes ' then you don't have to escape the double quote enclosed


@krutigarga can't switch on a class


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


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


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


I didn't touch the Manifest, Android Studio did it!


I did do a force reset on device tree and vendor because they got a little messy, I just pushed some new stuff now, was it stuck on the oneplus screen or bootanimation? If your using my tree's without modifications try to clone them again and if it doesn't work get me a log, I made a clean build this morning and all is working well.


coder pad vs google docs spacings


Seem like here are some codes for development purpose.


End this line with a trailing comma to preserve blame

`categoryErrorText: Constants.ERRORS.REQUIRED,`


Better this way @chochos @thradec and @ckulenkampff ?


Yes, thats the plan. The Cortex-A53 in the bcm2810 SOC will run in 32bit mode. I would like to test my changes but I am still waiting on delivery of my RPi 3!


@gpaulsen It is better to reply on the PR view, but I do get the mails for replying directly on commits, too.


Remove logging in release candidate


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Break this line into multiple lines


"this" seems confusing to me. Wouldn't something like listeners be clearer?


Awesome...got it Thanks!


Super!


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


Redirecting to page _URL_

";
will become
Redirecting to page _URL_


why using paths.dofile here?
this script is supposed to run with Lua, not with torch.


Documentation
-------------------------
These programs are written in Python 2.7. They do not require any additional installation of
programs or modules. All of these programs are created for sole purpose of re-familiarizing myself with the syntax and intricacies of Python. Some of them are incomplete and by no means perfect.

Author : Nhieu Duong


:+1:

It would be awesome if you can get that super-protocolhack (heh) to work :)


what was the error?


This would be one of the ways to display the option tags in html through javascript  $('#city-options').append('<option val="' +cities[i]+ '"> '+cities[i]+' </option>');


+1 awesome!


Do you need some sort of markup to indicate that code starts here? I think it might be three of the little 'ticks' (`) that are the lower case ~ key.


Watch out for empty lines like this. Every file needs an EOF (end of file) line, which is just one blank line after your last line of code


Thanks for your comment Alex,

So conf_select was introduced in HDP v 2.2.0.0 and what HDP version introduced distro_select support?
I think there is a confusion between ambari and HDP versions

Is actually  distro-select (hdp-select) version directly needed as value is any comparison?


I think we should consider making all of the case changes at once.  This has a larger impact at this point because there are potentially docs etc that all need to change.


This would allow you to grab the value of the selected option tags:
 var city = $("#city-options").find("option:selected").val();


remove if unused.


Location class for robot etc, probably incomplete but first class work


This seems like a strange thing to check into the repo because it has hard coded hostname, username, password.


Another way to achieve the task of changing the background would be: $('body').addClass('nyc');


I think you mean 'whether', not 'weather' :-P


It wouldn't be the first time that amd64 is weird. :-) FWIW, what the Plan 9 assembler calls MOVBLZX and MOVBQZX are actually the same instruction. Gas calls it movzbl:

```
$ cat main.go
package main

func foo()

func main() {
	foo()
}
$ cat foo_amd64.s
TEXT ·foo(SB),0,$0
	MOVQ $0x1234, AX
	MOVBLZX (AX), BX
	MOVBQZX (AX), BX
	MOVQ $0x4000, AX
	MOVBLSX (AX), BX
	MOVBQSX (AX), BX
	MOVQ $0x5000, AX
	RET
$ go build -o x . && objdump -d x |sed -n '/main.foo.:/,/retq/p'
0000000000401060 <main.foo>:
  401060:	48 c7 c0 34 12 00 00 	mov    $0x1234,%rax
  401067:	0f b6 18             	movzbl (%rax),%ebx
  40106a:	0f b6 18             	movzbl (%rax),%ebx
  40106d:	48 c7 c0 00 40 00 00 	mov    $0x4000,%rax
  401074:	0f be 18             	movsbl (%rax),%ebx
  401077:	48 0f be 18          	movsbq (%rax),%rbx
  40107b:	48 c7 c0 00 50 00 00 	mov    $0x5000,%rax
  401082:	c3                   	retq
```

I chose the L version to match the x's type (uint32) in the decode_other.go code.


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


I'm not quite sure what you mean by "binary" as a language. Also normally things ending in `.h` would be header files: so `curses.h` would be a header and `clrscr()` would be a function. I'm not across curses so I don't know whether conio is a function or a header in this case - sorry!


Fixed it!


this finally closes #582; we just ignore tests failures under Plone 4.2.


nit: can you do

```
if Verbose {
  body, ... ReadAll ..
} else {
  io.Copy(ioutil.Discard, res.Body)
}
```


because you may index a nil value at line 90 for which the chunk element is just 1-0(ie period),  if you donot specify this field.


well, using paths.dofile will make the code running smoothly,  but using dofile, i get an error, so i change it.


I noticed one calcite issue when doing DRILL-4372. Calcite treats decimal literal numbers as DECIMAL type. Thus, as query like, select 1.2 from (values(1)), does not trigger shortcut.

From regression point of view, it is fine to leave it for now and improve later.


WHYYYY? Did somebody force pushed this branch at some point???


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1016 of 1020 tests passed, 0 test warnings)

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


It's in the phar.


Hello Sir - Need urgent help Sir.
Using EAP 6.4 and BRMS jboss-brms-installer-6.0.2.GA-redhat-5.jar.
Following link - http://developers.redhat.com/products/brms/get-started/
to replicate coolstore demo project.  I am able to setup the project but once i am a luanching the web app ( http://localhost:8080/brms-coolstore-demo )

JBWEB000065: HTTP Status 404 - /brms-coolstore-demo

JBWEB000309: type JBWEB000067: Status report

JBWEB000068: message /brms-coolstore-demo

JBWEB000069: description JBWEB000124: The requested resource is not available.

Any early response much appreciable.


We should discuss whether we want to use the (class  OnsNavigator .... {) syntax, or OnsNavigator = React.createClass, i personally like the second one better, because you can define functions like getInitialState and don't have to deal with constructors.


and i actually i change all the dofile to paths.dofile in my clone


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


Actually not from this changeset at all but it looks funny with the new if block - I would indent this at column zero...


And this at column 2.


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 9](https://teamcity.labkey.org/viewLog.html?buildId=307618&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 9 (1 new), passed: 38, muted: 4 Build time: 01:18:19
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilters: <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.verifyLogGutterPlot: <no details avaliable>

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

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

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

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

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

```


Good comments. A lot of things have been integrated; some were not (for example, the java -cp bit... since the classpath actually does need to include grappa itself and its dependencies.)


I remember one situation, which might make a case for putting the shortcut after DrillLogicalPlan is done.

Say, you have a false filter, such as "WHERE 1=0". Practically, this is a Limit-0 and we have an optimization rule for conversion during drill logical planning.


yasd


Is this needed? In this case on_hold will be FALSE so the new metric code below won't use the o_ref value.


You might want to push the logic from the `MasterViewController` to the `CoreDataEngine` here:
```
   ...
    MasterViewController *masterVC = (MasterViewController *)[[self.splitViewController.viewControllers firstObject] topViewController];
    if ([masterVC respondsToSelector:@selector(setStorageToiCloudIfTrue:)])
    {
        [masterVC setStorageToiCloudIfTrue:iCloudFlag];
    }
...
```


Done, not very effected


I think FindLimit0Visitor needs to be improved. What if Limit-0 is in one of the foot of inner JOIN?
In left foot of Left outer JOIN?


I'm also not seeing this setting 'take'... I can toggle it until the cows come home and the storage stays the same (and the switch remains on the selected option when I re-enter the settings).


By the way, in order to actually host the website on Github, you have to name the repository as your username. So the repository should be called spectrumsoftware.


De virker da fint :broken_heart:


The first new test in `halotools/mock_observables/tests/test_nearest_neighbor.py` passes, but the second one, `test_tight_locus`, fails. So the bug only happens when there are two sets of points that are tightly localized, so I'm not sure what sort of corner this pokes into. I've checked that it's not a relic of the points being too close together. Any ideas @duncandc?


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed (1296 of 1578 tests passed, 0 test warnings)

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


This is awesome. :+1:


Yes, thats a good idea for replacing the Dummy with LoadTags.
Regarding the value for aln_num_of_patterns I used 200 since which was the "default" branch or "ELSE".
I would imagine that one of the reasons that an alignment doesn't have the number of patters it could be because it was too big, I would say, since for relative normal length we haven't observed any issues.
If we set it to 0 we may be forcing a huge alignment into a small resource class, right?


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


We have no plan to do that now, though I want to move to 1.2.0 to ease compatibility issues.


There should be no spaces between your <p> tag and your text - could you please remove the extra spaces and returns? I'm going to close this pull request. Once you've made the changes, submit another pull request so I can merge your work. Thanks!


Can be done in SQL without having to iterate over every object using extract: Claim.where('EXTRACT(days FROM now() - created_at) = 7')


this finally closes #582; we just ignore tests failures under Plone 4.2.


I don't think so. This is the pattern for abstract methods


FixCrashWithZoneAndSystemERV (JasonGlazer) - Win64-Windows-7-VisualStudio-12: Tests Failed (1839 of 2134 tests passed, 134 test warnings)

 * 133 tests had: AUD diffs.
 * 133 tests had: BND diffs.
 * 8 tests had: MTD diffs.
 * 22 tests had: RDD diffs.
 * 3 tests had: ESO big diffs.
 * 3 tests had: Table big diffs.
 * 1 test had: ESO small diffs.
 * 2 tests had: MTR big diffs.

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


LGTM


@pdlun92 Since this is a fixed change, the correct way should be to change the hash algorithm name in the openssl.cnf.


Looks like this may have reintroduced the compatibility issues seen here: https://github.com/plribeiro3000/jazz_fingers/issues/3 is it possible pin it back at `1.3.3`?


What if request is null?


宗三，你简直太牛逼了！


Just a very minor thing here: you removed the explicit modal body component, so you should remove the `body=false` as well. Apart from that: :+1: :+1: (double thumbs up! :) )


I didn't modify that, I think that might be an artefact of me having submitted the PR after you updated.


Well, dang it, take credit or blame for it anyway. :)

On Thu, Mar 3, 2016 at 7:48 PM, ChrisBrenton <notifications@github.com>
wrote:

> I didn't modify that, I think that might be an artefact of me having
> submitted the PR after you updated.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ChrisBrenton/grappaexample/commit/34d1e4b3eaae62a029e46fa04594cefd2f0a5257#commitcomment-16489568>
> .
>


what editor should I use to do that?

Thanks.


Whoops, good catch!


read it


for the dofile thing, you probably get an error because you run this script with torch. Have you tried with lua?
for the other thing, i understand your point. In just case, I would prefer to check that  just after line 43:
if #chunk>1 then
-- do the rest
end


nit: why not use std::list encode/decode?


nit: Interval might be a better word than Window


Are you not able to install `jazz_fingers`?
Whats happening? Because it is working for me. I have it in several repos without any issue.

Can you explain your issue please?


this is kind of hacky, just delete this and in the constructor do:
`this.state = {  pages: React.Children.toArray(this.props.children) }; `


`count(*)` gets parsed to COUNTALL in Raco, which is why the first argument is `count(*)`.

I'm checking for count(something) because we don't have nulls anyway. For some reason (related to this)
Andrew seems to have wanted to generate the "wrong" SQL for COUNTALL:

![image](https://cloud.githubusercontent.com/assets/1979690/13514727/4f71fc64-e160-11e5-9746-33f33d425f2e.png)


晕，你这一系列算凑数么：）


add


Hii i'm Abderrahman Akdim


I've now documented this on the test.


Try to think of edge cases to test. What happens if the user enters a negative number for the output length, for example? Are you sure the code will function as you expect, or will fail gracefully? Prove it with tests.


this is not good, we need to think how we can communicate with the children from the OnsNavigator, see the dialog example, the problem is we cannot really update the page property, maybe we can push using an url and then have a function getPropsForPage : (url) => props and transfer it here.


internal key word = default access


Fixing the api key and the error due to the android target version.


a


Be sure to `new_file.close()`! You'll soon learn how you use context managers to take care of this for you, but not closing files can lead to nasty problems.


This looks great, though curious to ask that this was part of the eval ctors work - was there some interaction here?


read it


Only because the decode takes a version -- so I wanted to keep the encode/decode consistent.


Does this commit fixes #2 ?


OK, I was thinking of the old way, I guess you'll get an exception when you try to instantiate a subclass of an abstract class if you haven't implemented all abstract methods? Also, I haven't really used the `abc` module, but I thought that you had to declare your abstract class's metaclass to be `ABCMeta`?


This is actually intended and the check should be kept!

What is flawed is ref-counting with automatic `free`, while just a `struct *`
is being passed instead of `struct **` that would allow for setting the original
to `NULL` because at the current state of the affairs, one cannot distinguish
`free`'d and non-`free`'d object.


Yes it does. Sorry for not referencing it in the commit.


Yes, evalling ctors fails on ffis. Turns out libc++ uses some pthreads stuff in its ctors, so I optimized all the stuff I ran into, to not be ffis.


Found a bug:
1. go to any site and bookmark it
2. make a folder and drag the bookmark into it on the toolbar
3. click the folder and click the bookmark in the toolbar

After the bookmark is opened, it gets moved from the folder into the top-level toolbar instead of staying in the folder.


Hm.. and you both started doing it this way? Instead of

```js
module.exports = new App()
module.exports.App = App

module.exports = new Logger()
module.exports.Logger = Logger

module.exports = new CacheBase()
module.exports.CacheBase = CacheBase

module.exports = new Verbalize()
module.exports.Verbalize = Verbalize
```
meaning, main export will be instance, and also will be exported the constructor.

Okey, may be it's easier this way, because user will always know that he have `.create` method, but.. I'm curious what you both thinking.


Great commit @molinch ! ;)


You probably want the `assert_version_required` here


You don't edit it. You're not supposed to.


test test test


rs=me


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


OK, I understand you want to be conservative in this PR, but I don't see the distinction between `COUNT` and `COUNTALL` in the first place if it's not in whether null values are counted. I think we should fix this (i.e., emit `count(*)` for `COUNTALL`) in a separate issue; do you agree?


I'm also seeing a gap after the bookmark entry:
![screen shot 2016-03-03 at 5 04 33 pm](https://cloud.githubusercontent.com/assets/549654/13514951/1c2ce290-e162-11e5-9fcf-2790615ceac7.png)


Yes, you can define the const variable at the beginning of the file.


Default first commit.


I did this here because there are methods that modify the `prototype` which cause all instances to be modified. I wanted a way to create a "clean" constructor so I can test specific methods in unit tests.

In `base` and `cache-base`, the `.namespace` method is used to do something similar, but it also creates different functionality in the constructor that's returned (by use the namespaced property for `get`, `set`, etc...)

There may be other reasons to do this that I'm not thinking of right now.


well, that is okay for the checking #chunk stuff  , i got your point. Also the reason i am not using lua is because i have to install lua packages, but run th command is okay, so i modify it.


btw, this is a custom migration to create a faculty group on install.


Actually not, I don't have any advantage to point.. I will use json module instead.


this can be a local variable to the registerAPIHandlers function


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


`// Serve javascript files and favicon.ico from assets` comment needs to be updated for the new changes


You can just do:
```
class BootLintBear(LocalBear, Lint):
    executable = 'bootlint'
    output_regex = re.compile(
                r'^.+?:'  # filename
                r'(?P<line>\d+):(?P<col>\d+) '
                r'(?:(?P<error>[E])|(?P<warning>[W]))\d+ '
    #etc etc
    def match_to_result(self, match, filename):
        if match["col"] == " ":
            del match["col"]
        if match["line"] == " ":
            del match["line"]
        return Lint.match_to_result(self, match, filename)
```


It was like that in v2, but travis started to fail with external runner = `/usr/bin/env python`. I will give this another shot. Thank you.


Yea, for `base` and `cache-base` I know and I'm agree.


this is going to return true if there are no cohorts, it should return false to stop the destroy (unless the errors being added stops the destroy from happening? in which case we don't need this line at all)


I see, makes sense


looks good

utACK


This is ordered alphabetically btw


Wan't expecting this heading (see requirements doc). Would be better if this text is merged into the italics below


Four years ago, maybe five, I had considered not to draw terrain if nothing was changing in zoom, orientation, etc. So I had tested how many times this was happening, and it was practically never happening during flight. I came to the conclusion that I was only adding overhead by checking something that hardly happened .
So I am curoius to know if this is working or not..


We are planning with @czyzby to make a complete implementation of standard annotation mechanism to run in dragome (js). So I'll take this pull request as temporary patch but we need to improve it to use standard mechanism in a near future. thanks


Ah yeah, will add that for the upstream PR.


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


ᵔᴥᵔ


Ah sorry, yeah, installing `jazz_fingers` is no problem, but this version of `pry-byebug` doesnt actually work with `pry-remote`. These issues [on pry-byebug](https://github.com/deivid-rodriguez/pry-byebug/issues/33) and [on pry-remote](https://github.com/Mon-Ouie/pry-remote/issues/58) explain it better than i can.

Essentially, with any version of `pry-byebug` higher than `1.3.3`, when you use `pry-remote` you cant connect to the remote. It looks like in https://github.com/plribeiro3000/jazz_fingers/issues/3 you had pinned it to `1.3.3` but then updated it later. Because this now requires a version `3.1` or higher, I cant pin this in my projects gemfile either:

In Gemfile:
```
  ...snip...
  # This is pinned because of compatibility issues with pry remote.
  # See: https://github.com/deivid-rodriguez/pry-byebug/issues/33
  # And: https://github.com/Mon-Ouie/pry-remote/issues/58
  gem 'pry-byebug', '1.3.3'
  ...snip..
```

After running `bundle install`:
```
Fetching gem metadata from https://rubygems.org/........
Fetching version metadata from https://rubygems.org/...
Fetching dependency metadata from https://rubygems.org/..
You have requested:
  pry-byebug = 1.3.3

The bundle currently has pry-byebug locked at 3.3.0.
Try running `bundle update pry-byebug`

If you are updating multiple gems in your Gemfile at once,
try passing them all to `bundle update`
```


Commit success


Good point.  The constructor for ODataMediaTypeFormatter does check, but since I have made GetDefaultBaseAddress public and static, it should also check.  I will add:
            if (request == null)
            {
                throw Error.ArgumentNull("request");
            }


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


还是能看到哦,白帽子黑客


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


FixCrashWithZoneAndSystemERV (JasonGlazer) - i386-Windows-7-VisualStudio-12: Tests Failed (1847 of 2134 tests passed, 139 test warnings)

 * 138 tests had: AUD diffs.
 * 138 tests had: BND diffs.
 * 8 tests had: MTD diffs.
 * 22 tests had: RDD diffs.
 * 1 test had: ESO small diffs.
 * 3 tests had: ESO big diffs.
 * 2 tests had: MTR big diffs.
 * 3 tests had: Table big diffs.

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


Np. I'll close the issue and reference the commit :)


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


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


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


:trophy:


O arquivo 'DescriçãoProcessoInstalação.odt' criado pelo @MatheusFontesGama foi enviado para a pasta 'Documentos Gerados/Fase 1'


呵呵~我水平还太差，就是在期待大神们写。以后水平够的时候希望能和大神真正沟通技术和大神一起写。

和大神第一次对话，很激动。上周末刚把大神博客从第一篇到最近的浏览了一遍~有些只看的标题，有些看了内容，从宏观上对R的认识学习了很多。相比在封闭环境中学习快了很多，感谢各位开源大神的无私贡献。


发发


Official MVS2015 M_PI is equal to 3.14159265358979323846.
   https://msdn.microsoft.com/en-us/library/4hwaceh6.aspx
Probably doesn't matter though.


Yeah, I've just implemented this. It was quite a big change but it makes the code a lot cleaner. Functions can now be defined by implementing `\Transphporm\TSSFunction` e.g. https://github.com/Level-2/Transphporm/blob/master/src/TSSFunction/Attr.php  and then registered with Transphporm like this: https://github.com/Level-2/Transphporm/blob/master/src/Module/Functions.php


thanks - very minor point on the spec naming, these are each part of "describe" blocks that say whether they are dealing with "lat lon" or "region mapping", so you don't have to put that in here too...


This looks like a JS floating point issue: `-73.99047400000001`


Will this make it use the latest version of 0.6.x? Because 0.6.0 doesn't pass the integration tests.


Additions to support a local persistent docker-registry so that the rolled out instances can manage an amazon instance node stop / start after some app builds were pushed to the registry.


When running /deqp/functional/gles3/uniformbuffers.html, it does not print the results of every cases. Add the messages here.


这个项目还活着？


store/recover privileged attrs using xattrs


a


add slice lab 3 file


M_PI's use in the mixer doesn't need that much precision, but, thanks for the link. I'll update the code now.


okay, I did create a filteredList array, but I also have a filteredContentList NSPredicate. Hopefully we can iron it out tonight. @falkobuttler


Yes i know, i already did it :)


```
 ~/r/G/graphmalizer-core (master:5) » node utils/graphIsomorphism.js
[ [ 2 ], [ 3 ] ]
graph 1 (not isomorphic): isIso?= false
Match: [ -1, -2, -4, -3, 1, 3, 2 ]
Match: [ -1, -2, -3, -4, 1, 2, 3 ]
graph 2 (isomorphic): isIso?= true
fails 0 asserts 1 errors 0
```


I missed that, thanks


From the perspective of C/C++ language standard, any data race is equivalent to assignment to out-of-bound array element; they are simply undefined behaviors. However, it's harder to get explicit answer about if a compiler will (or will not) perform such transformation. C++03 compilers can definitely apply such transformation because there isn't any concurrency memory model formally defined. I am less certain about C++11 compilers though. C++ standards comittee paper N2338 titled "Concurrency memory model compiler consequences" (http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2007/n2338.html) is the most related reference. In a nutshell, it states that a compiler cannot introduce data races on shared memory locations if the original program is race-free. All the transformation examples in the paper fall into this category. But it doesn't say anything about program that already has data race (because it's already undefined).

I don't know std::atomic + std::memory_order_relaxed are implemented in details but they are the most portable way to express weakly ordered accesses (i.e., atomicity + minimal visibility ordering guarantees that prevents the most counter-intuitive behaviors). In other way, if the current code is fine in practice (even undefined w.r.t. language standard), changing it to std::atomics will likely generate equally efficient code. And the more verbose syntax of std::atomic and std::memory_order serves as a documentation to the tricky code.


Thanks!


testWithFuseCardShouldNotCounterCMCFive should be written. The test should pass when Disrupting Shoal + Fuse Card is properly implemented. Right now it is passing without proper implementation.

Change the last two lines to:
`assertPermanentCount(playerA, "Air Elemental", 1); // should not have been countered by Shoal
 assertGraveyardCount(playerA, "Air Elemental", 0); `

Also may need to change if Far//Away gets exiled here. However, from my understanding, Disrupting Shoal can still exile Far // Away, just it will never counter a 5 CMC spell, so the rest of the test should be fine.


I dont understand, why are we checking if IsDirty? If it is Dirty we SetDirty, but it is already Dirty, no?


Closes #58


Because the `format` method is declared `static`, there is no need for you to create an instance of the `DateFormat` class. You can replace `new DateFormat()` with `DateFormat` so that you are doing the static method invocation `DateFormat.format(...)`.


This right here is giving you a infinite loop


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


这个语言主要用在哪些场景中？

Linlin Yan <notifications@github.com>于2016年3月4日周五 上午9:26写道：

> 这个项目还活着？
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yihui/r-ninja/commit/def33434193fee9cd252526cfd56965adf37cb4a#commitcomment-16490136>
> .
>


q


Break is typically not really a good thing to have in your code. If you want to respond when something is done that isn't allowed.. print something, or throw an appropriate error.. But breaking gives me little information other than just shutting everything down.


Previous comment still applies; I think we can toss this sentence altogether since we are now going to a restart.


This is throwing me an error
'''E       TypeError: read_file() takes exactly 1 argument (0 given)'''


vb


This doesn't take into account table prefixes.


I don't understand why this is placed on the Flight object? Why is a specific type of data point on the flight?


Overall very well written code and very clean. Maybe incorporate a little more comments. Also there is a very weird long space on line 18.


Don't worry too much about this. Eventually we will want to integrate the py-mz-wof-export code in to this and that will take care of these things.


Ok got it


This is throwing an error
``` E       TypeError: make_a_damn_story_list() takes exactly 2 arguments (1 given)```


I see the HeaderRPMBlock added. I don't have a good grasp on what this block is doing. See my inline comments. Where does this block/data come in the stream?


I kind of see that the HeaderRPMBlock is used to calculate the rpm.


Can you add some JavaDoc to this class. I've been meaning to go back and document all the classes and public members of the other blocks. I'm trying to understand how this fits into the data model.


it work when
 - if map orient is NothUP and low zoom level and or low speed ( 3.5km zoom
for paraglider )
 - in pan mode all time
 - in most multimap top view ( mostly always low zoom & fix map orient )
 - all time where you are o ground (before takeoff, and after landing)

and for all other case, only very small overhead.

2016-03-04 2:13 GMT+01:00 Paolo Ventafridda <notifications@github.com>:

> Four years ago, maybe five, I had considered not to draw terrain if
> nothing was changing in zoom, orientation, etc. So I had tested how many
> times this was happening, and it was practically never happening during
> flight. I came to the conclusion that I was only adding overhead by
> checking something that hardly happened .
> So I am curoius to know if this is working or not..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brunotl/LK8000/commit/86680cf864feb36bd3b791bf08a64c2807bd2f19#commitcomment-16489951>
> .
>


Looks good, but how come you got rid of the illegal argument exception? It's probably better to keep `index %= imageSetCopy.size();` inside of Overworld Panel, since we don't want other classes to incorrectly access a sprite and silently end up with the wrong one. At the very least, make it log that it's changing the index


Looks good. One issue, when linking (in the "Learn More" and "More Testimonials" buttons), you should never link to concrete urls. Inside of the a's href quotations, you want to use @Url.Action("ActionName","ControllerName", new { param1="x", param2="y"}). In this case I don't think you need the URL parameters.

That will spit out what you have there. The idea is if you change the action name in the controller, it doesn't break the link (that you've hardcoded).


最近工作了，没空去维护这项目了，我以前是可以的，你如果感兴趣的话可以自己看看什么问题，后台用的 lean cloud，肯定不是后台的问题。


getting rid of `display: flex` in `.contextMenu` shows that the weird gap on the right is actually a scrollbar


First, the get started uses all the latest versions of JBoss EAP and BPM Suite, see the project you downloaded:  installs/README for the details.

If you need to use 6.0.2 then the way to do that is to switch to tagged version that matches,  JBoss EAP 6.4 does not work with JBoss BRMS 6.0.2 so see the RELEASED VERSIONS at the bottom to pick what you want to use: https://github.com/jbossdemocentral/brms-coolstore-demo

If you want to use the most recent then you need JBoss EAP 6.4 + patch 4 = EAP 6.4.4 and then JBoss BRMS 6.2.0.


:clap:


because if isDirty() = true no need to call UpTodate().
isDirty is trivial, UpToDate no.


새 텍스트 문서.txt

ccc


good boys


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


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


public abstract class Jogo {

  private int codigo;
  private String nome;
  private int versao;

  public Jogo(int codigo, String nome, int versao) {
    this.codigo = codigo;
    this.nome = nome;
    this.versao = versao;
  }

  public int getCodigo() {
    return codigo;
  }

  public void setCodigo(int codigo) {
    this.codigo = codigo;
  }

  public String getNome() {
    return nome;
  }

  public void setNome(String nome) {
    this.nome = nome;
  }

  public int getVersao() {
    return versao;
  }

  public void setVersao(int versao) {
    this.versao = versao;
  }

  public abstract void exibeJogo();

}


@javelinanddart already done it
but skip setupwizard will make trebuchet missing default widget/icon placement on home screen


Sorry could you tell me how could i patch this on cyanogemod 13 ?!


So it should be

if (!trenderer IsDirty && ..

From: Bruno de Lacheisserie
Sent: Friday, March 04, 2016 2:49 AM
To: brunotl/LK8000
Cc: Paolo Ventafridda
Subject: Re: [LK8000] DrawTerrain only if needed, improve change conditions. (86680cf)

because if isDirty() = true no need to call UpTodate().
isDirty is trivial, UpToDate no.

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


outcome[,11] <- as.numeric(outcome[,11])
hist(outcome[,11])


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


Unless I'm mistaken, the following
 else if (rank != RANK_1 && (!(b & mobilityArea[Us] & file_bb(s)) && (!(b & mobilityArea[Us] & rank_bb(s)))))

is equivalent to
 else if (rank != RANK_1 && no file mobiility && no rank mobility)

which is the same as
 else if (rank != RANK_1 && mob ==0)

Is this what you want to test ?


Replaces robots.dmi with the cogi sprites added.


What are your stats about UpToDate hits during normal flight or flight replay?
My stats says about 0% of hits!!
Then, during thermalling it will never work. The reason I did not made the UpToDate years ago was that it was only slowing down the software.
Please check the hits, mine says it is absolutely useless.

From: Bruno de Lacheisserie
Sent: Friday, March 04, 2016 2:42 AM
To: brunotl/LK8000
Cc: Paolo Ventafridda
Subject: Re: [LK8000] DrawTerrain only if needed, improve change conditions. (86680cf)

it work when
- if map orient is NothUP and low zoom level and or low speed ( 3.5km zoom
for paraglider )
- in pan mode all time
- in most multimap top view ( mostly always low zoom & fix map orient )
- all time where you are o ground (before takeoff, and after landing)

and for all other case, only very small overhead.

2016-03-04 2:13 GMT+01:00 Paolo Ventafridda <notifications@github.com>:

> Four years ago, maybe five, I had considered not to draw terrain if
> nothing was changing in zoom, orientation, etc. So I had tested how many
> times this was happening, and it was practically never happening during
> flight. I came to the conclusion that I was only adding overhead by
> checking something that hardly happened .
> So I am curoius to know if this is working or not..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brunotl/LK8000/commit/86680cf864feb36bd3b791bf08a64c2807bd2f19#commitcomment-16489951>
> .
>

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


I'm not sure I understand; I meant `map stave voices` instead of  `map (\voice -> stave voice) voices`.


Should not be also upgraded in server/pom.xml, mailbox/pom.xml ... ?


`recentf-mode` will be enabled, whether `helm-turn-on-recentf` is t or nil (to disable the mode, only zero or a negative number works).  Use `(when helm-turn-on-recentf (recentf-mode 1))` instead?


@mibby Thanks for the blueprint - i've already set up the class (without the setAs calls, but with the copy and past from the log :p). The problem is that the breaking times are somehow needed. Usually i take them from the wiki (https://www.minecraft.gamepedia.com/Breaking), however they haven't been updated.

So i'll likely not try to waste time trying each block with hand, finding the 'smallest' tool that breaks it faster than hand, and so on.

Maybe i'll set the block flags (rather easy) and set all to instant breaking, so the warnings are gone for the moment. Or maybe i try setting most of them like similar stone based blocks and do minimal tests.


tai sao thay o day


Thanks for fixing it - forgot to merge it into the fixes.


@jcalovski This issue about "hash pressure" in context of this TT-save patches has been discussed on
Forum topic "Appropriate hash sizes for TT testing". See posting from 14. Jan. and following ones.


The Japanese matches the text as it was before the change. (Shion - 詩音 first, corrected to Mion - 魅音 second)

This must be from when Keiichi is still convinced that Shion is an act put on by Mion.


В случае, если был открыт пустой файл.


Yes.


Why block ?


indeed, thanks !
https://github.com/melonjs/melonJS/commit/5cd24df8dc36488817fdd3b530ea7bb6567c6b76

I have to go now, but I'll have a fresh look as well tomorrow to see if the code can be further optimised/cleaned


this is new edit


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


Do you examples of inlined checkboxes ?


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


Actually, it was the "cute twice" in the commit message that piqued my interest, but I thought I would forego a sarcastic comment for something actually constructive. :)


I think it's quite difficult to find a project that has upper bounds for every dependnecy. Some packages are quite stable and don't change often. After all, once a real compatibility problem is discovered, I will fix it and put out new version if necessary. With lots of upper bounds I wound need to watch out and refresh deps regularily even if everything is fine.

That being said, you may be right. This is just not something I see other people doing.


There are also 2 *ConditionalDataflow runnables that are used in this pipeline. They can now all be replaced by LoadTags in it, and the runnables removed. Can you do that ?

OK, so the default value matches the default dataflow we want. Then please add a comment saying that 200 is chosen so that the decision selects the _8_cores analysis. Also, if the conditions cover all the scenarios and if there is a default value, we don't need an ELSE branch

Finally, don't you need to also load "gene_count" here ?


I figured out now working as expected


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


Its weird, because I am still can see appointments on all views

I am looking into it


```
if (((Actor) entity).getSourceCard() == null) {
	return false;
}
```

How can this ever happen? Actors are one of the following: Minions, Weapons, Heroes. All of these have a source card, so I am wondering which edge case you are covering here.


Hi Matt, I also had issues with the installer bulding but managed to get it working with a couple of changes to the AddFiles.ps1 script and the pre-build event call - see this commit on my fork (I'll create a pull request if you like): https://github.com/GeekJosh/SIMSBulkImport/commit/5df3661a1410a6fd66ba631fe2bcdf904c0d6d90


@bdunogier This is copied from platformui sysinfo, as I intended it to be a drop-in replacement for that, at least in early development. But it seems now it won't be, so I'll put it up for change.


Thanks James. (1) fixed and (2) attempting to offer a better error message. I think some people will be stuck with early scipy so it's potentially not very practical for them to upgrade this, and much easier for them to just install a Scientific package.


Not yet! still need partition information


need publish, it's just quick fix before your refactor :)


I actually realized that I introduced a bug here, which is that it will cause an error if `dir == nil` I made a fix in master, so please merge master to this branch in your fork.


Thanks. MrEfrem will look l8r


This how is done for torrents and size for nzb. To be consistent.


@nursix I tried changing the title_list as per your suggestions but I see that this never comes in here?


@nursix can i disable the represent for the component in the subscription?


Sorry, don't have any more time to spend on Pi today....

All I know right now is, latest "VC boot" firmware....

$ vcgencmd version
Mar  3 2016 20:00:47
Copyright (c) 2012 Broadcom
version dde4ad1fe6f2d1f0fde268be91c5faed754153bc (clean) (release)

Zero boots fine with the updated firmware, but the 2x Pi3B's I upgraded this morning, (this was only change, bootcode.bin, fixup*, start*), do not boot. Single flash of green LED activity when power applied, nothing more. (And I gave up with serial console on the 3B's yesterday. Was just outputting non-readable garbage. I'll attach a serial cable later today.)

Working (booting) firmware on Pi3B (for me)....
$ vcgencmd version
Feb 29 2016 14:25:00
Copyright (c) 2012 Broadcom
version dea971b793dd6cf89133ede5a8362eb77e4f4ade (clean) (release)


`address` avec 2 d.


put the person's ID into the error message. that will help debugging these errors.


see above


here as well


LGTM


And no sooner did I post that I realised I'd left the full path rather than relative, d'oh!
https://github.com/GeekJosh/SIMSBulkImport/commit/ed428220da398dd36b41659d138b9e1f5c7d73b6


You need to remove the checks you introduced before. They break the build.


Cheers :+1:  a pull request would be great - just exclude SimsBulkImport.wxs from the pull


You were very clear ;) It's just that it's not the type of function I need there. Something with placeholder code :)


What do you mean by this comment?


For KNIME, we have received some files, where the `controlByte` was `0x10`, `0x20` or even `0x30` were. They all seem to indicate the number of bytes to copy (for `0x30` quite a lot).


If Editor is initialized using existing `<textarea>` as the element, `this.textarea` is allready present and observers are not added.


This inc file is included by some recipes. You need to bbappend the recipes not rewrite the inc file.


If I have understood @kaniini at all, the point of server-time is to have the same times in logs so if something must be checked from the past, logs of two or more trusted parties can be compared and they both have the same timestamps.

At least three IRCds seem to implement it. http://ircv3.net/software/servers.html


Okay, I'll do it during the day, choose ".resx" for everything?


incode -> encode


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


Where did you get this recipe from?


Chunyang Xu <notifications@github.com> writes:

> recentf-mode will be enabled, whether helm-turn-on-recentf is t or nil
> (to disable the mode, only zero or a negative number works). Use (when
> helm-turn-on-recentf (recentf-mode 1)) instead?

Yes, done.

Thanks.

--
Thierry


Essayez de mettre les scripts vers les autres (dans la page finale). (Le '&' rouge est bel et bien une erreur dans l'HTML)


Don't think the virtuoso logo is in there


ok let me think about this a bit. just looking now how set_path works makes me want to change it...but in principle I like your fix, so hopefully I can merge it after I make some changes to set_path.


Sounds good


Sorry for being so boring... well, let say that we can overpass the 80 columns for a single string.
While, the following parameters which are for sure over 80 chars, these should go to new line...


Append needs a space in front.


is that indentation required?


LGTM


one change missing here in first filename


Same here, you can avoid to break the format string...


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


Kudos for that ;-)
I would suggest however to put this patch before the previous one... without the docs for the two new parameters, which should be moved for consistency in the patch that adds them.


Ok, this was actually final.

The Repository model must be compatible with the Database, so that requires a nullable bool. So that conversion would happen in any other place that fransforms from Repostiory to RepositoryModel.

Another option would bo to use 0,1,2 in the database and map directly to the enum. I like this idea. What do you think?


Could you please share how are we ensuring that change does not affect confirmed orders ?


I think the input param could be only `ServerMetadata metadata`. The instance of `Server` could be retrieved as `Server.refById(metadata.getId())`


FixTransition84to85EMSActuator (mjwitte) - x86_64-MacOS-10.9-clang: OK (2127 of 2127 tests passed, 0 test warnings)

![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)


Little Indentation Issue

Hound integration would  help here :)


It's fine if that's one of the points of server-time, but I think what's listed is a more compelling feature for end users, and makes more sense to put on the front page.


mainメソッドは_analytics.js.erbの後に実行される保証がないので
_analytics.js.erbで取得した内容が取れない可能性があります。

専用のAPIがあるので下記の処理で送信して
JS側API実行サンプル
https://github.com/OKSKY/embedded_skytart_chat/blob/analytics_sample/app/views/embedded_skytart_chat/widget/_analytics.js.erb

https://github.com/OKSKY/embedded_skytart_chat/blob/master/app/controllers/embedded_skytart_chat/analytics_controller.rb#L28

でtrackerに送信する処理を記述して下さい。


TeamCity ReOpenLDAP :: devel [Build 237](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1856&buildTypeId=ReOpenLDAP_Devel&guest=1) is now running


Can we change the equality to less than comparison as having negative quantity is actually data corruption issue but would shield us with more data corruption ?? :)


Sounds good to me.
We could use this update to reformulate this regulation in accordance with the style guide. I would suggest to change it to "If an extra attempt is granted...".


On Fri, Mar 04, 2016 at 01:49:12AM -0800, Jan Pokorný wrote:
> On 04/03/16 01:36 -0800, Dejan Muhamedagic wrote:
> > On Fri, Mar 04, 2016 at 01:20:41AM -0800, Jan Pokorný wrote:
> >> On 04/03/16 01:02 -0800, Dejan Muhamedagic wrote:
> >>> On Thu, Mar 03, 2016 at 10:01:14AM -0800, Jan Pokorný wrote:
> >>>> Anyway, if there's no better solution, we can resort to using conditionals...
> >>>
> >>> Right. We can merge this post the (a bit overdue) v1.0.
> >>
> >> So what do you propose for now?
> >
> > Testing!
>
> Yeah, getting to that point, if only this dependency on cluster-glue is
> optional so as to avoid the bumpy ride on Fedora/RHEL packaging front :)

Please do whatever's necessary to bring it up to speed there. I
suppose that replacing the glue parts is going to take a bit.
BTW, glue is known to compile and work fine on redhat too, just
in case you want to test booth there beforehand.


I think you meant to edit this line :smile:


Hi Haibao,

How can I see this message too? I tried logging in to Travis, but am unsure
how to see the message.

I looked quickly, but still have to figure out what the issue is...

We're there any other error messages?

Thanks,

Debra
On Mar 4, 2016 12:32 AM, "Haibao Tang" <notifications@github.com> wrote:

> @dvklopfenstein <https://github.com/dvklopfenstein> somehow this code
> breaks the build
>
> [image: image]
> <https://cloud.githubusercontent.com/assets/106987/13518883/6093406c-e187-11e5-846d-3848f34706e0.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tanghaibao/goatools/commit/52ea4199a57c3b161afa2b0aef59a5cf76b60f72#commitcomment-16492342>
> .
>


[I removed all that info in the past to make sure PMS would transcode](https://github.com/plexinc-plugins/Services.bundle/commit/634f6b0c7562be4a1d4c81064d24c1897c9c63f3) the Vidzi videos, because they wouldn't work in some cases otherwise (when client and server are not in the same network).


you should use the `BITNAMI_APP_USER` macro in the `chown` command.


> I, on the contrary, use the docs directly on github as it is faster for me than finding the documentation website (it's just 1/2 clicks, since I'm already inside the repo), so it's a matter of preference

I prefer to read the reST files locally, that's even faster :smile:  E.g. `SPC p f quick RET` when coding  anywhere will open the "Quick installation guide".   Besides, for online reading on rtfd.org, the browser history auto-completion may get there very fast too...  Anyway, N persons, M>N workflows, and it's good like that.

> As for creating an issue for github, that would be a good idea, but based on how long it took them to implement some basic improvements that people were asking since long time, we will probably move to a different documentation format before they can do this

Yeah, sometimes there are not enough commercial incentives for a commercial company.  And, WRT reST, people probably mostly use Markdown.


@bwmarrin Is there a better way to do this?


Good point, thank you @stof!


someone already posted an issue about this error (#1675), mine also affected by this error.
hope there will be some progress there.


Can you plz use Exception Rule to check also error message.


林文滨你不得好死,人渣,你会有报应的.你这个变态王八蛋,去死.


@Yive Hey yive! cheers mate


I see. Well, if it was like this before, then maybe it can wait. Is anybody else concerned with this, or not really ? @dpobel ?


Thanks for documenting it.


StringUtils.isBlank(). Also helps with space-only strings


Could not think of a better name


No, it's actually a mistake. I thought I had fixed it...


射程とはもちろん、装備および艦娘の射程のことですわ。艦娘の射程＝max(艦娘の主射程, 艦娘の装備1の射程, 艦娘の装備2の射程, ...)で、長い方が1巡目では先に攻撃できます。しかし同射程での攻撃順はランダムですので、事前にシャッフルする必要があるのです。


First commit


@dunixd haha


You need to handle InvalidArgumentException. Also it would be useful to have a field validation (I can do it after merging the PR)


LG
@kamilkoterba - want to take a look?


Missing `.view()`


> [ERROR] Sorry, unsupported distribution LinuxMint.

Mint is Debian based, so adding support should be easy.  It may be even enough to reuse Ubuntu Trusty provisioning commands...
https://github.com/inveniosoftware/invenio/blob/master/scripts/provision-web.sh#L176-L223


ok never mind..now I remember why I did it this way. =p


Ok


Could not think of a better name. Type erasure equalizes both combine signatures.
Nevertheless, this function is important for lists with immutable, thus non-observable, data.
As an example consider lists of measured values from some sensor.


if patch which I posted above doesn't help I suggest to keep mvc queue with 6*2 packets (in your log for one h264 packet it reads 6 mvc packets). this should be enough to fix small mvc packets issue.


I think here, just put it on 3 lines instead of using `and`.


Returns GMT if it is not understood. Maybe it's OK, but it's a topic for validation