While this importers are very fast indeed, they have the major drawback of being highly fragile. I'm pretty sure they already are broken since @SamSaffron extracted some fields off the users table. Because of that, I don't think they'll ever be a new base importer. I'd rather we keep on improving current importers and stay away from writing raw sql queries.


Happy to commit...


**HEAD commit has no message.**
*GitCommitBear, severity 1, section `commit`.* *Project wide.*


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/__init__.py`, line 9.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/__init__.py
+++ b/coalib/__init__.py
@@ -6,6 +6,6 @@
     if not sys.version_info > (3, 2):
         print("coala supports only python 3.3 or later.")
         exit(4)
-
+

 __version__ = Constants.VERSION
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `coalib/__init__.py`, line 9.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/__init__.py
+++ b/coalib/__init__.py
@@ -6,6 +6,6 @@
     if not sys.version_info > (3, 2):
         print("coala supports only python 3.3 or later.")
         exit(4)
-
+

 __version__ = Constants.VERSION
```


Does this behave as expected if the `impl` block is defined inside a function? I'm not sure if `parent().parent()` returns the function or the containing module.


Seems like this package would be more useful directly in the GNAT runtime (there are several pools there, already), or perhaps in GNATCOLL. It isn't specific to the containers hierarchy (otherwise we should put it in the temporary Conts.* hierarchy).
How about improving composition here ? If we put this package in the Conts hierarchy, it should be a generic package, that takes another pool in parameter (Conts.Pools). Then, instead of calling the services from System.Memory or the global pool object, just call the services from the Conts.Pools you received. That would allow wrapping any storage pool to make it thread-safe via protected objects, and thus make the code more reusable.


Hi @FaisalAccion - I can see you have outcommented `stdfn.CHAR = String.fromCharCode.bind(String);`

If you can explain your ideas behind it we can look into how to get the change into the original repo?


今は精度より終わらせること重視で！


This can be simplified to

```ruby
Timecop.freeze("2015-12-18 10:12:26 UTC")
```


I have forked and will be sending some improvements as pull requests soon.
Also making issues that inform what I am working on. If I happen to choose to work on something that someone has already claimed, please let me know.

Thanks :) Looking forward to a better web portal for BRL-CAD.


The same question as above - is `ancestor_module` necessarily a module?


@nehalecky, looks like you're missing the `$` in front of `JUPYTER_IP`...


Er, Crap. No. :D


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


I think you should set the product_id by default instead.  As you can have views with different products in it.


What's the general feels for this type of alignment? It doesn't seem more readable, at least for me anyway.


这个地方，没有判断 /var/log/dmesg 是否存在，直接读取，导致异常，在 aliyun 的 vps 上很奇怪，就没有这个文件。所以首页就报错了


This function is still very clunky, I'm sure you could do better by rewriting it from scratch. Also, what does `authorized` mean in this context.


Thanks for taking time to adapt and apply this patch !

It effectively breaks compatibility somewhere (older v2 servers with newer v2 clients) but I'm not sure if anyone uses v2 servers in test/production environments (since the bug I reported last time was blocking backups with index > 10, anyone using it for a long time enough would have reported that !)

If you want complete compatibility it would be possible to detect PROTO1 or PROTO2 and adjust this command as necessary, though I didn't figure why the command was changed between versions and if this fixes or avoids something.
Personnaly I would much better benefit from a transition path from stable 1.4.x to 2.x(when ready) rather than 2.0.x(beta) to 2.x(production ready)

BTW, apart from these small fixes we are having very good results in our pre-test environment (especially coming from the very high inode consumption in PROTO1). I am considering going forward soon for hosts that don't need xattr/VSS ; with them it would be perfect :smile:


Does the code even work? It should be `dependent: :nullify` (note the "e" instead of "a" and a colon).


C'est surtout pour PHP7 qui ne supporte plus le nom de la classe en méthode constructeur.


just for completeness:

2.7.4 seems from 2015-12-21
2.7.6 seems from 2016-02-01

see http://ffmpeg.org/releases/
so it seems 2.7.6 is the latest from the 2.7.x

any ideas to go for release 3.0?


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


@sils1297 You mean the `use_stdin` flag? Though important, it isn't specifically related to this kind of bear. That should be done elsewhere imo. But do tell me if feel otherwise.


thanks for the commit


traditionally, people tend to write `word.length == n + 1` instead of the other way around, that is you variables on the left hand side


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


Good spot


Great!

I've been very happy with the CommonJS pattern and it's what many people are now familiar with (based on it's prevalence in npm modules). So perhaps we can create a new issue to migrate to it (in some form or fashion).


You could try `case ... when` when you have multi-way if comparisons


Forgot `.` in $(). With the `.` in selector, we will correctly query for existing class.


Delete the files as well in this commit.


In Ruby, programmers tend to use symbols when they use this kind of static values


@akallabeth: I'm not a deep expert, but I did a bit of comparison with your code.
Here is the comparing program
```
#include <stdio.h>

int main()
{
        unsigned char data[2] = { 248, 255 };
        int offset;
        int unsigned_size;

        unsigned_size = sizeof(unsigned);

        offset = (int)(data[0] | (data[1] << 8));
        printf("Shifting left an unsigned char does not create sign bits: %0*X\n",
                unsigned_size * 2, (unsigned)(data[1] <<8));
        printf("offset=%d\n",offset);

        printf("\n");

        offset = data[0] | ((int)((signed char)data[1]) << 8);
        printf("Shifting left a signed char does create sign bits: %0*X\n",
                unsigned_size * 2, (unsigned)((signed char)data[1] <<8));
        printf("%d\n",offset);
}

```
The correct result from the WORD { 248, 255 } should be offset = -8.

Output on 32bit platform:
```
Shifting left an unsigned char does not create sign bits: 0000FF00
offset=65528

Shifting left a signed char does create sign bits: FFFFFF00
-8
```

The problem in your suggestion is that when you shift an unsigned char (BYTE) left by 8 bits, the C compiler implicitly converts it to a unsigned int (32 or 64bit) before shifting, without extending the sign bit, because the source data is unsigned.

And yes, I'm sure my code can be optimized and made more readable :smile:
Maybe my cast to `(int)` is reduntant, because `(signed char)data[1] << 8` is already an int due to implicit conversion.
I will be happy to see other improvements :smile:


Na didn't seem to work, still had the delay. I ended up wrapping Email.send() in Meteor.defer() to solve the delay.


See #2


spaces around `&&`


I thought so, and I should've explained it somewhere :) but now thinking about it, seems moot so I'll reject unknown types.
My idea was that for containers, we will add interfaces speculatively (i.e. as soon as we add the machine doc for the container, we'll add the same interfaces to it as its host machine has), and that might mean we don't necessarily know whether each of the container interfaces will have the same type as the interface of the host which was used as a "template" for its creation. For example, on MAAS we have eth0 and br-eth0, as well as br-eth0.42, for the container we definitely don't want "br-eth0" of type Bridge to be the name and type of "eth0". So, during machine document creation for the container, we can add an Unknown interface (in advance) with name "eth0" (as in, strip the bridge prefix, not hard code "eth"+device index). But then at that point we'll know that new "eth0" has to be an EthernetInterface, so using type Unknown as valid makes no sense and will make it an error.


Update Project Progress


@dburles Na didn't seem to work, still had the delay. I ended up wrapping Email.send() in Meteor.defer() to solve the delay.


I am unsure whether it is clever to *start the scan first* with the call back wrapper and *afterwards* set the callback in the wrapper and the internal variable.


Just remove it, if we require 0.4 we're unlikely to use 0.3 again (and anyway it's easy to re-add).


Just an opinion, but I would not force it to PARTITIONED here, but would rather check for the mode set by the tests inheriting this.


Instead of worrying calling `spawnNewTile()` twice with a race condition, simply make another much more robust `spawnTwoNewTiles()` function! Not the most elegant solution of course, but certainly get the job done robustly!


I had this in the pipe that I forgot to commit. However I don't know if it is something we want to handle.


`res.vent.dis`
Te cobran por caracter?


常にx1 < x2とは限らないので逆の判別式が必要


of **the** effector


`{{}}`
un abruzo


`(from 0.0 to 1.0)` is more clear


actually that was my bad, as I didn't clearly/properly update it when I made the change.


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


Use this `tilesMoved` flag to tell whether we should spawn a new tile. If we press an arrow key, but no tiles are moved because they are already at their respective final position, we don't want to spawn anything!


@KorvinSzanto #3578


Fixed in https://github.com/qgis/QGIS/commit/b97e86da413295b7ff9f3214af638bb2fd6f3c40


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


@karasu Yep..let me know if you have any questions about the changes I made for the new UI. Its still a little rough around the edges but its close to being ready I think :smile:


`maximum_depth_of_path`, use quotes so that it appears as a variable (not in plain text)


インスタンス名 rainCountはやめよ笑


What to do with this TODO?


Why do you use the git repo rather than the release tarball <http://erratique.ch/software/tsdl/releases/tsdl-0.9.0.tbz>?


Typos: `Admitedly` -> `Admittedly`.


Same here for `extrication_frequency_`


And this TODO?


**Nitpick - Code Quality**
There should be a newline before this line. This will aid code readability. Please fix all instances where this occurs in your code.


This should not cause a privacy error, I suppose.
```
struct S;

fn f() {
    impl S {
        fn method(&self) {}
    }
}

fn g() {
    // g is not an ancestor of method in the AST,
    // but they are still in the same module
    S.method();
}

fn main() {
}
```


hello there


とりあえずこれでいいけど、ループが逆の方が効率いい


Add the framework ID to the log message


If I am interpreting the [docs](https://manishearth.github.io/rust-internals-docs/rustc/front/map/struct.Map.html#method.get_parent) correctly, `parent().parent()` returns the function, so it behaves correctly.
While a function isn't technically a module, it behaves like one for visibility purposes -- a private method in an impl in a function should be visible only inside the function.


Yeah, it's OK here, but please note that `persisted?` is not a strict opposite of `new_record?`, as it also takes deletions into account. Full definition below:

```ruby
def persisted?
  !(new_record? || destroyed?)
end
```


I find this function name confusing. You are basically comparing the uri from the request with a given string. I prefer the previous approach where the function explicitly compares the uri with the callback uri.


Your change doesn't work. Just tested it (took me a bit longer get used to git and get my files on pc updated from this )


This fix is just trading one exception for another.
I really hope the IOException is handled nicely.


We only spawn a new tile if:

1. At least 1 tile has been moved, and
1. We have less than 16 tiles.


Shouldn't this be pwm_params.multiShot instead ?
Also, that parameter could perhaps be changed to useMultiShot instead to be more similar to other (e.g. useOneshot42, useOneShot)


Please rename the file `robot_trajectory.png` and rename the image file name here too.


Enemy only takes one "n" in english! :P


Please comply with the (local) alphabetical ordering of the package names.


Sure, since there's no map involved for preparing and delivering the results, they should be in the same order as the args. I'll change it as you suggest. This is one of the earlier tests where I wasn't sure a map wasn't needed for the implementation and decided to play it safe.


yep, even if we haven't moved this tile, we still need to return its current `rowNum` or `columnNum`, otherwise we'll get tiles with class like `3undefined`


Hmm, I think that should be a privacy error, but it isn't right now.


这个这个真麻烦


qibo second


这个有点麻烦


@vaa25 Test name tells that it checks finding of events in the future, but test logic doesn't do it. The test checks only calling dao method. I mean that you have incorrect name for this test, the test isn't wrong.
P.S. Don't forget write name mention with `@` of communication participant, please.


Is this `(.getPath (uri/make ...))` necessary?


Hello, its possible to add some features to the bot, something like
auto-attack with accuracy?

On Wed, Feb 10, 2016 at 4:07 AM, BrandonWill <notifications@github.com>
wrote:

> I just started a new account and tested the building queue. It works for
> me.
>
> What's not functioning for you?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BrandonWill/GrepolisBot/commit/2806b2527daf7e7dbf5f455115ca8fda7118dacc#commitcomment-15994934>
> .
>

--

Cu stima,
Patriciu Anton


Another issue, if "pam_enabled: True" then ansible-role-pam will overwrite system-auth, and this module will add it back. Whoever wins depends on the order in which the roles are evaluated.


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


是的、


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


오타 주의!! ㅎㅎ


Why was this added? @tjhcb


I guess established practice in state - possibly coming from the way a txn.Op with Remove: true and no Assert: txn.DocExists works. Not every state type's Remove() method works like this, but I've checked most of them do and went for consistency.


Will change, sorry for the confusion :)


Please use UpperCamelCase for filenames whenever possible.


>a private method in an impl in a function should be visible only inside the function.

This is not true in the current implementation - only modules set the privacy boundaries, everything else is transparent.


Glad to see that you use jscs :) I plan to integrate it soon with the build task


@Ede123 Sorry, I didn't see your comment, until now.
Let's talk about this issue in #1018


This change is not yet reflectd in the configuration documentation.


Please remember to do updates with `git pull -r` (rebase) instead of just `git pull`, otherwise we get merge commits like this with no value.


Nice!


Boooo-yaaaa! Nice!!


That depends on the the parameter of the member function `detect_sharp_feature`. The functionality that detects the borders should be merged into it `detect_sharp_feature`, maybe with an additional Boolean parameter. Or we should document that `detect_borders()` must not be called if `detect_sharp_features` is used.
CC: @janetournois


Fuck ya!!


Ok.


Thanks for this. But what if we wanted to use some of the inputted information in order to do further verification?

For example, how would we access the inputted email address inside getResetValidationRules()?


Here I moved the first two bullets to the end of the list, so that we define the terms "owning side" and "inverse side" *before* using them.  I also reorganized and reworded the other bullets, so that they flow from a discussion of one-to-one, many-to-one, then finally many-to-many.


Bug, nice find.


[![Our Homes Ourselves](http://api.artsmia.org/images/200/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=200)


[![Woman and Little Girl of Constantine with a Gazelle](http://api.artsmia.org/images/123758/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123758)


[![Shipwreck on the Coast](http://api.artsmia.org/images/123759/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123759)


[![Bathing Women](http://api.artsmia.org/images/123768/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123768)


[![Arabs Skirmishing in the Mountains](http://api.artsmia.org/images/123771/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123771)


[![Young Woman with Peonies](http://api.artsmia.org/images/123772/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123772)


[![Alexander and Bucephalus](http://api.artsmia.org/images/123773/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123773)


[![The Death of Sardanapalus](http://api.artsmia.org/images/123774/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123774)


[![Still Life with Flowers and Fruit](http://api.artsmia.org/images/123775/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123775)


[![The Trellis](http://api.artsmia.org/images/123776/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123776)


[![Christ in the Garden of Olives](http://api.artsmia.org/images/123782/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123782)


[![Weislingen Captured by GÃ¶tz's Men](http://api.artsmia.org/images/123786/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123786)


[![The Martyred St. Sebastian](http://api.artsmia.org/images/123787/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123787)


[![The Entombment of Christ](http://api.artsmia.org/images/123789/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123789)


[![Christ Asleep During the Tempest](http://api.artsmia.org/images/123791/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123791)


[![Study for "Luxe, calme et voluptÃ©"](http://api.artsmia.org/images/123793/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123793)


[![The Camel](http://api.artsmia.org/images/566/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=566)


[![Saints in Glory](http://api.artsmia.org/images/1590/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=1590)


[![Our Homes Ourselves](http://api.artsmia.org/images/200/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=200)


[![Woman and Little Girl of Constantine with a Gazelle](http://api.artsmia.org/images/123758/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123758)


[![Shipwreck on the Coast](http://api.artsmia.org/images/123759/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123759)


[![Bathing Women](http://api.artsmia.org/images/123768/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123768)


[![Arabs Skirmishing in the Mountains](http://api.artsmia.org/images/123771/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123771)


[![Young Woman with Peonies](http://api.artsmia.org/images/123772/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123772)


[![Alexander and Bucephalus](http://api.artsmia.org/images/123773/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123773)


[![The Death of Sardanapalus](http://api.artsmia.org/images/123774/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123774)


[![Still Life with Flowers and Fruit](http://api.artsmia.org/images/123775/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123775)


[![The Trellis](http://api.artsmia.org/images/123776/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123776)


[![Christ in the Garden of Olives](http://api.artsmia.org/images/123782/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123782)


[![Weislingen Captured by GÃ¶tz's Men](http://api.artsmia.org/images/123786/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123786)


[![The Martyred St. Sebastian](http://api.artsmia.org/images/123787/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123787)


[![The Entombment of Christ](http://api.artsmia.org/images/123789/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123789)


[![Christ Asleep During the Tempest](http://api.artsmia.org/images/123791/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123791)


[![Study for "Luxe, calme et voluptÃ©"](http://api.artsmia.org/images/123793/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123793)


Hi! Yep it's any non-whitespace character, so dashes and underscores count.


I think you are right here. It escaped my notice on review. @dicko2 A cast would have been sufficient here, since we know the value is an int.


Is it even safe to use `@var Bug[]` here?  `$reportedBugs` is an object of type `ArrayCollection`.


This is my hello-world readme file.


Adding CLI arguments in the comment for consistency with previous code snippets.


Right - there's a load of discussion today on https://github.com/jakubgarfield/Bonobo-Git-Server/issues/368 which might affect this too - the username@domain.local thing ("Upn") is just one of several ways in which we might have 'username' presented.  All a bit beyond me at the moment, I have to say, and extremely difficult to test.

I think one possibility is that we might move away from the username@domain.local format, to just 'username', which would then perhaps make this a bit easier, though I don't know enough about AD to know if it's an issue that you then wouldn't be able to to distinguish joe@domain1 from joe@domain2.


We'll have to fix this.


The statement "This is the first contact with the read API of the EntityManager" is not accurate, as we were previously introduced to `EntityManager::find()` earlier in the guide.


Yep. :/  Thankfully you don't really need to understand it to understand my change. <_<


 David Anderson, CS 270, Assignment 3
Description: Create the models and database migrations necessary to form the core of the application.
Input: N/A
Output: N/A


From http://zsh.sourceforge.net/Doc/Release/Zsh-Line-Editor.html#Zle-Widgets:

> Normally, calling a widget [with zle] does not set the special parameter WIDGET and related parameters, so that the environment appears as if the top-level widget called by the user were still active. With the option -w, WIDGET and related parameters are set to reflect the widget being executed by the zle call.

Can you give a specific example of something it breaks?


Are you using Windows?  I'm pretty sure that this is an unnecessary for me on Linux and OS X.  Please call it out as such.


Yes, verified the fix doesn't break with a screen rotation.
When a screen is rotated, new surface is created matching the rotated screen size & it works.


That comment doesn't make sense to me.  Isn't replicating exactly what you are doing then? :-)  Never mind.  It seems easier to handle this in 8's HotSpotVMConfig than having some code in the compiler.


In that case, it would make more sense to match on [^\s]+ instead. Could you make that change please, @jgillis01? :)

Thanks for the help, Gina.


We are trying to get away from React.createClass, use the ES6 style.  One note- you will have to bind your callbacks to this.   Otherwise the class looks very clean. rowStates could probably const object outside of the class.


Added.


Let's revert then.
(our repos are such a mess that I can't distinguish what's new from what's old...)


this change seems to be confusing to me. will it be clear to get necessary tp by language and project codes here?
```TranslationProject.objects.get(language__code="language1", project__code="project1")```


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


make it happen :+1:


Instead of creating and catching your own Exception, why not just handle this case above when the `modelFilename == null`?  Then any other real IOExceptions can be caught and logged here


The mention of modal boxes, I guess. There was a point in time where that was a big deal to the community.


I might be wronte about rowStates. I have not studied it enough.


both open source. again though you would have to test because i dont remeber the sources i used before...which is why im going to cleanup this repo and only leave the semi-working rild/libril in device tree that way we can test and work on it easyer. So yes it should have that working with whats in my current device/vendor however verification is needed. And when i say calling works that only includes regular calls not emergency. I think we still are going to have to convert our java ril emergency calling code over to our open source libril code.


I'd somewhat prefer to keep the additional symbols at the bottom.  As a first pass, I added a comment referencing `typedargslist`.


Looks good to me then!


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


I have created a user-defined widget to do something with vi-put-after. It was then wrapped to _zsh_highlight_widget_vim-put-after using the zsh-syntax-highlighting. Without the -w it used to work well, but now zsh reports sth like "no such widget _autosuggest_orig-vi-put-after".


This was the compromise we decided upon when this dependency injection thing was introduced (cant find the PR with the discussion right now). The problem is that the class is used in the GUI everywhere and there is no clear chain of injections but a whole net. Thus we decided to wait with the final removal of the global variable until the GUI code is cleaned up.


Can you backtick these in Markdown format?  Also, they didn't appear as an ordered list in the View.  You might need another newline before the first entry?


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


Does this block make sense now?
Doesn't look like you do anything with `pathContent`.


Instead of creating and catching your own Exception, why not just handle this case above when the `modelFilename == null`?  Then any other real IOExceptions can be caught and logged here

Also, typo in "directory"


No, I was hypothesizing the creation of such a constant for our own use :)

I think there are some predefined constants like that however.  I will research.


Why is one instance of `IdentityToken` marked omitempty and the other not?

Also kind of weird that the character casing is different, but that's not your fault.


es6 new function `cell.includes(newline)`

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/includes


white space alert!

fun({rep_start, Pid}) -> on filtermap line, only indent another four spaces.

Turn off electric mode.


Also executed several test on the app level.


react default to px when you specify a number, so you could have `padding: 5`


Yeah, that's part is just a copy of the AST module's documentation.  Should definitely change to specify the Python version.  Sounds like it may be worth waiting for a more comprehensive doc pass?


white space, end should be out at the same level as NewPids, and State#state.rep_start_pids), should not be on a new line after end.


Actually, fixing this eliminates the NETCF issue.


scan_pid on new line four spaces in, other fields at same indent, closing curly brace `}` on its own line vertically aligned with S in State.


You are getting better !!

- KISS !! wtf is that "env" in the interpreter ? #!/usr/bin/python is all you need
- Don't waste your time writing comments ... and even worse not needed comments. ( III )
- Use absolute paths ( III )
- Not important at all at this point, but variables should be lowercase and underscore ... enc_pass
- Probably the use of %s or {} and format is less propense to errors for strings substitution. Choose one and use it.

Well done !


you can write `LocalBear` together since it resembles an actual class :)


same here with `LocalBear`


this is fine. Know that we also have http://numeraljs.com/

see: DistanceTool.js


in this case I would also replace `[...] files, which makes [...]` with `[...] files, but that makes [...]` so one understands that this limitation actually has its benefits


Looks good. +1


typo -> `parallelize`, only a single `r`


What do you think about this? I don't know if there's a clean way to handle it in CocoaPods.


Would you be willing to share the implementation of the user-defined widget that does something with vi-put-after?


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


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


`Global Bear` -> `GlobalBear`


Overall this looks really good. UI looks nice too.


Yeah, it looks like that should be fine.  My mistake.  I added it back.


And I think you could be a bit more concrete on the benefits of a `GlobalBear`. If you have questions about the details here, ping us at gitter, there we can help you :)


This is a link: http://www.something.com


How do you feel about removing this entirely? Until Apple improves these headers, there's no clean way (that I could find) to make this work. You can create a custom module, but it involves listing an explicit path into one of the platforms (Xcode/Contents/Developer/Platforms/iPhoneSimulator.platform/) which is impossible in the pod.


Don't bother checking this repository its just full of junk.


I would just write the classes itself here and summarize this a bit, I would do it like this:
```
There are two types of bears, the GlobalBear and the LocalBear.
```


@blambeau @anne-france , besoin de votre vérification. Je suis un peu moins sur pour l'optimization avec restrict


In general, I'd rather leave the modifications to this source code to the minimum necessary to get the methods to inline (since the source was imported from elsewhere).


We use NET_n_m constants for the targeted runtime. In the past, we used CS_n_0 to indicate the C# level being used. The problem with that was that the constants are embedded in the project files and are therefore always the same no matter what compiler is being used. If the compiler provided constants it would solve the problem.


ASD


Ah j'ai oublié de répondre au commentaire sur ma PR.
En fait, index.transactions.home et index.transactions.details ont un indent de plus que index.transactions parce que c'est ses fils.

cf : https://github.com/angular-ui/ui-router/wiki/Nested-States-&-Nested-Views


Nice, clean code! Well done.


`Don't bother reading this repository`


Yes. Two for the price of one. :-)


Why does it need to be a string literal? It's totally valid in node to run:

```js
require(path.join(__dirname, 'file.js'));
```


Maybe I'm saying something very stupid, but I have the feeling that this breaks strict aliasing rules. The change from `char*` to `void*` is fine (although it introduces the need for annoying casts...). But the change from `char` to `uint8_t` for `storage` means that we will now access the memory as `uint8_t`(for example when it is initialized) and through a pointer of different kind at some other times.


Nice, clean code! Good work.


http://coala.readthedocs.org/en/latest/Getting_Involved/Writing_Good_Commits.html
Your commit message needs a tag, otherwise the shortlog is quite good, it resembles what changes you make. A tag is normally the file where changes happen so in this case I would suggest `Tutorial.rst` :+1:

If your commit message exceeds the 50-char limit because of the added tag, you could shorten it for example with `Explain bear kinds`

Also I assume you picked up an issue this commit fixes, please add it inside the commit body with `Fixes https://github.com/coala-analyzer/coala/xxxxxxx`, so use a hardlink instead of the plain issue ID. See the Writing-good-commits guide for details.


Already updated, they should both be omitempty


This is the default behavior. While I too like to hide "http..." folder(s) in their own subdirectory, implementing that while taking note of existing installs would have been more lines of code than it is worth.


RAS


you should put a full colon : after the word 'is'.


Space missing after dot. `created.Here` -> `created. Here`


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


you won't say `a code`, would you? :) --> `code`


Again, looks good. One minor nitpick: when you're done with the site, you will probably wan tot delete the code that is commented out.

Good work!


Replaced with Install-BinFile that works on pre-existing installations


For windows auth we should store the username as DOMAIN\username


And `A bear` needs to be `a bear`, the sentence continues after a comma not a dot


Même remarque que précédemment.


@dzknj try to be a bit more clear with your naming conventions and leave off elemental or code based naming conventions, like: "header-div", "my-id", "this-span", etc.


Fix me.


Please use whitespace after the `if` keyword and around binary operators (see https://google.github.io/styleguide/javaguide.html#s4.6.2-horizontal-whitespace)


Même remarque.


@buzzdeee would you be willing to open a pull request with your OpenBSD changes back to the mayflower repo?


Can also use `scalaVersion in ThisBuild := "2.10.1"` and add that just to the root project. I believe that results in a leaner internal structure in SBT.


I'm not sure this line is very helpful in this case since the example is quite good readable. We should discuss that on gitter with the others^^


Fix me


Previously, the script downloaded packages to a special folder on the first install, and then didn't direct them anywhere on if the above registry key existed (eg, all subsequent installs/updates). For whatever reason the setup.exe would dump them in `ChocolateyInstall`. Added this `Else` to properly handle the situation.


**Excellent Work!!!!!**


This isn't necessary


RAS


**Line is longer than allowed. (81 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 19 column 81 to line 19 column 82*.


This is only stylistic stuff from my point of view, maybe you want to change `created` to `utilized`? That resembles more on-demand mood :D


I'll have to revert this as I just ran into a problem because of it.. On our systems tophat is a symbolic link to the tophat2 wrapper script which is in a separate directory along with other tophat programs. This patch would get the wrapper into an recursive call of itself in this case. Obviously it should be dirname(readlink () ), not the other way around. The original script was doing that just fine, there is just no need for an absolute path there at all.


@alexshenoy : This is the model I was trying to describe earlier.  It works, not sure I like the pattern though.


newline too much


113837367-Fanger-PMV-Equation-Bug (RKStrand) - x86_64-MacOS-10.9-clang: OK (2113 of 2113 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)


I wrote a small improvement on dbg_mean, in calculates the variance as you go on.
To get the standard deviation, just sqrt(variance)

/// Debug functions used mainly to collect run-time statistics
static int64_t hits[2], means[3];

void dbg_mean_of(int v) { ++means[0]; means[1] += v; means[2] += v * v; }

void dbg_print() {

  if (hits[0])
      cerr << "Total " << hits[0] << " Hits " << hits[1]
           << " hit rate (%) " << 100 * hits[1] / hits[0] << endl;

  if (means[0])
      cerr << "Total " << means[0] << " Mean "
	  << (double)means[1] / means[0] << " Variance "
	  << ((double)means[2] - ((double)(means[1] * means[1]) / (double)means[0])) / (double)(means[0] - 1) <<  endl;
}


just randomly picked `if (err)`. why not just use `assert.ifError(err)`? i noticed that you using this style everywhere.


This example is in fact a copy of the `LocalBear` :O This needs to be changed accordingly^^


There appear to be comment about grow_memory on this branch?

White space at the end of some lines.

Is this also intended to add the `call_import` operator?

Is this also intended remove the import flag from the functions section, or are both methods usable in transition?

Is this also intended to change the index spaces, to make the index space for `call_function` and `call_import` separate?


I guess the idea here is that you have a copy of setup.exe not buried in a temp folder...? Since this is unnecessary (and would download the setup file to Bin-Root\cygwin regardless of whether cygwin was installed there or not), I've removed it.
The following command easily gets you what you want:
 `cinst cygwin --force --not-silent`


would this work using the `@test.utils.wait.until` function instead?  that way, this will have verbose logging if it fails -- like this:
```coffee

    doneGenerating: =>
      @test.utils.wait.until 'export url is set', =>
        @test.driver.isElementPresent(COMMON_ELEMENTS.doneGenerating)
```


If it weren't for the `/home/User` directory, this would probably be fine. Gotta keep user files out of ChocolateyInstall though.


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


Esse commit podia ter sido melhor dividido tornando a análise um pouco mais fácil mas tá muito bom pois implementa adaptação ao tema padrão do wordpress de 2015 do delibera o que faz muito sentido.


Please add spaces around keywords (see https://google.github.io/styleguide/javaguide.html#s4.6.2-horizontal-whitespace)


@mlubin https://github.com/JuliaLang/METADATA.jl/pull/4644


You didnt replace `getPost` with memoized selector because it would have cache depth of 1?
So that way we get bigger cache ? (whole list) ?


Please keep the closing `)` with the last parameter and void unnecessary line breaks. I would do it like this:
<img width="841" alt="screen shot 2016-02-23 at 7 33 23 pm" src="https://cloud.githubusercontent.com/assets/6458867/13271637/54e715c4-da64-11e5-88ff-fe9ba4e23ccd.png">


this `base` is never used? it seems it is used `app.base` always


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


Done.  I specifically avoided the use of the word "closed", since there's a few different cursor states in the mongo driver (killed, closed,...) and I didn't want anyone to read into this to pedantically.


done.


Please avoid wildcard imports (see https://google.github.io/styleguide/javaguide.html#s3.3.1-wildcard-imports)


Dont we want to actually memoize this? (instead of `getSitePosts ` ) ?


>I **purposely** chose these names

Ok then. Leave it as-is.


we only import this package to call `GetErrorMessage`, which makes me very sad.


interface?


it should become a package, unified with `docker/distrubution/reference` probably.


that check should probably not be at the api level.


QuickFixIDD-Memos (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2151 of 2151 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)


I actually followed the "unprotected" code and you are correct. The changes you made make the protection unnecessary.


operator spacing (and I would split it onto two lines)

Also: I would move `getCanAddImpactTabAuthorComment` `getCanAddImpactTabIAFComment` to `ProposalPermissions`.


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


spacing around `||`


113837367-Fanger-PMV-Equation-Bug (RKStrand) - 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)


Sorry, I didn't write "disregard" here also as above.


learning


Logging should be done with slf4j API, could you fix that @larsonmattr ?


middlewares should become structs that accept arguments as initializers and have a method that implements the middleware interface, that way they'd become generic and we could move them to the engine-api package.

```go
type versionMiddleware struct {
  baseVersion string
}

func (m versionMiddleware) (handler httputils.APIFunc) httputils.APIFunc {
  return func(ctx context.Context, w http.ResponseWriter, r *http.Request, vars map[string]string) error {
    if m.version == someOtherVersion {
    }
  }
}
```


Ummm... noexcept + CURRENT_THROW?


@deltakosh you forget to remove the 798 statement...


i did´t have time to look into this but i get the error
Public DNS (nsd4) is running and available over IPv4 but is not accessible over IPv6 at .......


Java 7 has now a Files.copy, this could be replaced by that


++ what are the main things we're using new relic for here? up time?


lets remove this client after the test is done.


Adding esc for exiting the placing mode, would be good too.


Wait, The 6.0 version of nethunter is for CM? Every time I installed NH on stock 6.0 ROM on my Nexus 7 (2013 wifi) keeps crashing, such as settings etc,


Up time / performance / database connectivity


LGTM


It's a bad idea to catch a NullPointerException... if you want to check for a null you should do it explicitly wherever is needed. This will catch a null anywhere within the block and can go wrong if a bug is introduced somewhere within the code block that produces a new null


Due to a [**Qt bug**](https://bugreports.qt.io/browse/QTBUG-16372) i must use **8.3 filename** because "**++**" in **long filename** cause an error.


QuickFixIDD-Memos (mjwitte) - 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)


Good, but do you know what does that "owner => '0'" mean ?

It's ok, but KISS if you can use username or groupname ... don't bother with uuids and guids if not needed.

https://docs.puppetlabs.com/puppet/latest/reference/type.html


Pas convaincu : de toute façon on est dans transactions.route.js ; donc les seules routes possibles sont des routes en index.transactions. Du coup, fils ou pas fils, ça me semble logique de tout mettre au même niveau. À la limite le jour où on aura des petits-fils ça serait logique ; mais le jour où on aura des petits-fils on aura déjà perdu la partie contre le code sale.


Please catch specific exceptions (I guess you want IOException here?). Also the error message should be more explicit and definitely more than info level.


Please always use braces even when they are optional (see https://google.github.io/styleguide/javaguide.html#s4.1.1-braces-always-used).

Also: spacing around `if`


Again check for nulls explicitely


Testing


This looks suspicious.  Pass by value, then swap?  Are you certain this does what it says it does?  Also, it kind of defeats the purpose of the move ctor.


怎么在这么早的1.0版本commit上进行comment 哈哈


QuickFixIDD-Memos (mjwitte) - x86_64-MacOS-10.9-clang: OK (2113 of 2113 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)


```python
cxn = getattr(self.connection,'__wrapped__',self.connection)
```

For newrelic>=2.10.0.8.


Not too sure why gradle downgraded its selft. :/ it won't matter thought.


merging pull request


Hmm, I did not realize.  I suppose keeping in sync would be good...


113837367-Fanger-PMV-Equation-Bug (RKStrand) - i386-Windows-7-VisualStudio-12: OK (2119 of 2119 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)


The `relative_charge_level` and `absolute_charge_level` is only available for MegaRAID yet as HP SmartArray does not
expose these info.

Besides those two, both MegaRAID and SmartArray supports well.

I will check with other SAN array's support status.


Este boton no debería encontrarse aquí. Dentro del template BusetaBodega/NecesidadMaterial/form_template.html.twig se encuentran los botones para el formulario de esa entidad. Ahí dentro es donde debería estar ese boton.


Your comment says this handles convert_function, but the implementation does not actual appear to do so.

Also, this should be consistent and use the proper SIL-printed form for "thin_to_thick_function".


Sure. The code was from the X clipboard support. [vi-mode](http://zshwiki.org/home/zle/vi-mode).
```
[[ -n $DISPLAY ]] && (( $+commands[xclip] )) && {

  function cutbuffer() {
    zle .$WIDGET
    echo $CUTBUFFER | xclip
  }

  zle_cut_widgets=(
    vi-backward-delete-char
    vi-change
    vi-change-eol
    vi-change-whole-line
    vi-delete
    vi-delete-char
    vi-kill-eol
    vi-substitute
    vi-yank
    vi-yank-eol
  )
  for widget in $zle_cut_widgets
  do
    zle -N $widget cutbuffer
  done

  function putbuffer() {
    zle copy-region-as-kill "$(xclip -o)"
    zle .$WIDGET
  }

  zle_put_widgets=(
    vi-put-after
    vi-put-before
  )
  for widget in $zle_put_widgets
  do
    zle -N $widget putbuffer
  done
}
```


Fix pls


I'm not sure I follow, the posts list is already stored in the the redux tree. You can compute getSitePosts from a list of posts, but not the other way around since getSitePosts only contains ids.


WTF?!


I think you're wrong:)
https://github.com/BabylonJS/Babylon.js/blob/master/src/Materials/babylon.standardMaterial.ts#L798


No worries, I was wondering why travis-ci failed. :)

As no other package is using this, I identified that the docs are a bit misleading on this topic. See #13409.


For Test purposes....If you guys dont find this ok. we can revert but lets test for a couple of days :+1:


@pdeboer what should we do with this branch? It is 2 commits ahead of master - this one and the pervious one.

Are we still using these files? And more importantly: does this change work? the .properties file will be evaluated at runtime and the path seems to be relative to our repository.


fix this typo


No content altered in this commit, just reorganized to match the template
And added the utf8 test line and the template date


When a stream ends during a delay for loading a fragment, the fragment might then not be loaded correctly. This can be resolved by moving the delay from the FragmentModel to the FragmentLoader. It works by allowing the xhr request to be generated right away, because if the stream ends then there might be a problem generating the request. The delay is then inserted just before sending the xhr request.

After this change, there will no longer be a need for hooking the PLAYBACK_SEEKING event in the FragmentModel.


I suppose we could, though it seems a bit like overkill.


that's makocm/flocm, not mako or flo


If I'm not mistaken doing a handler call like this is subject to race conditions. I believe assigning the handler before calling it is best practice and the secure approach.


The module commands do nothing, NOTHING !!!!! MUWAHAHAHAHAHA.... =p


Nitpick: I'm not sure why you rewrote this in that form. It's no shorter, no clearer, and no less indented. I'd argue in fact that it's slightly less readable both because of the extra line and the negation of the result of dyn_cast<>().


@MrAndMrsK

There was a problem that we could not use type definitions for both main process and renderer process because some definitions causes collisions.  `tsd` gathers all definitions at the same place and provides only `tsd.d.ts`, but `typings` provides `browser.d.ts` and `main.d.ts`.  So the problem may have gone.


FragmentModel no longer keeps track of delayed fragments, so it doesn't need to clear list on seeking.

FragmentLoader is now updated to handle the delayed-and-not-yet-sent fragments when aborting.


Nit: could you remove empty lines?


NICE. What exactly does the plot mean?


@pdeboer What should we do with this branch? Are these changes still needed?

I'm not sure how we are handling it right now, but this would need to be adapted for other colabs.


I don't know why someone would include an offset in a Bernoulli model, but
I wouldn't rule it out. I think it is fine to have the code coverage tool
overlook code that can't be reached but it would also be fine to not
comment it out so that we have a reminded to do something about that code.

On Tue, Feb 23, 2016 at 7:41 PM, Jonah Gabry <notifications@github.com>
wrote:

> stan_glm.fit.R: don't include unused code in line coverage
>
> @bgoodri We can probably go further than just not including this in
> line coverage stats and get rid of these lines. I don’t think we need
> the code for weights here since any weights specified will be
> interpreted as number of trials and won’t be passed to Stan as weights.
> Also, is there ever a reason to specify an offset in a bernoulli model?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/stan-dev/rstanarm/commit/95030d953560d2173bee3d8f680634580c7b3b20


Yup, you're correct, we are doing that check in [`OnPropertyChanged`](https://github.com/Caliburn-Micro/Caliburn.Micro/blob/e59223f3d00825e5923096c5f435f1f0bf2090d8/src/Caliburn.Micro/PropertyChangedBase.cs#L65).

This change essentially just stops us doing the dispatch and unnecessary creating the event args.


Sorry, needed a local merge to clear the grow_memory branch.

Is a parallel change expected to the ml-proto spec to use a separate text format imports section to group them all, for example:

```
(module
  (imports
    ($import1 "mod1" "name1" (type <n>))
    ...))
```

If there are plans to use the same section to import other objects then could there be provision for it now, or would that clearly be a matter for follow up issues.


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 1.2.33](http://build.clear-measure.com/viewLog.html?buildId=5565&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


@pdeboer Any idea if we still need this?


Nit: better function name than "generate"? How about generateAssignment (overload)?


Terminaste la primera **sección** del tutorial. Esa imagen es del ejercicio 5/18.


I'm afraid it won't be possible right now to cope with non-convex objects.
The only way I can think of it is to encode the normals in the gp and
eliminate the internal point.
That's gonna be a future work anymore.

Let me know if there are gonna be major changes in the algorithm, so I can
fix the paper.

Carlos, I'm very behind with the DR3.2 report. ermano is gonna give me his
report in a couple of days and this afternoon I got Alex's.
Tomorrow I'll push into git a draft. If you can have a look at it, would be
great.

Nite
C.
On 23 Feb 2016 22:41, "Carlos J. Rosales" <notifications@github.com> wrote:

> ok, just to be sure, cause I'm writing the ending condition (the main one,
> not by number of charts) to be that
>
> thanks
> On Feb 23, 2016 23:37, "Federico" <notifications@github.com> wrote:
>
> > Means the atlas covered the whole estimated object, right?
> >
> > Not the whole object, it reached max number of atlases, but most of it
> was
> > covered. I can try raising max charts, but sometimes rviz crashes because
> > there's so many goddam markers being displayed !!
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/pacman-project/gaussian-object-modelling/commit/d420f0a344027fe37eeb4fb6097d93f26fec996d#commitcomment-16282312
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pacman-project/gaussian-object-modelling/commit/d420f0a344027fe37eeb4fb6097d93f26fec996d#commitcomment-16282409>
> .
>


Nit: how about executeAssignment? (overload)


BTW ... git rm <file>, git commit, git push ... and extra .project is gone


I don't think we will ever want to manipulate strings at runtime: I expect `terminal` to have a type like `u32` or maybe `struct TerminalIndex(u32)`


Why did you do this? Don't hard code your URL, use the refer method!


This is hilarious.


Nit: consider removing ;) Nice touch though.


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 1.2.33](http://build.clear-measure.com/viewLog.html?buildId=5565&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 85, ignored: 13 Build time: 00:02:25


QuickFixIDD-Memos (mjwitte) - i386-Windows-7-VisualStudio-12: OK (2119 of 2119 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)


This will break - not every node has an id attribute


Not directly related to this commit, but why do we have this? can it be as simple as ```QueryOver<TimeoutEntity>().FirstOrDefault(x => x.Id == id)``` think it works the same way?


Oh yeah,  I went too far :) Reviewing prs from a conference is maybe not
the best idea :)

On Tuesday, 23 February 2016, Kerry Liu <notifications@github.com> wrote:

> I suppose we could, though it seems a bit like overkill.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Automattic/wp-calypso/commit/cfd1ca4d35a75905c5da64382f4a2915984cf1df#commitcomment-16285276>
> .
>


Nit: if there is something specific about zookeeper here, perhaps consider "formatAsJsonForZK"?


La idea es tener el botón procesar también en el Modificar, por lo que no es necesario eliminarlo del Ver Datos. Debes dejar también el botón procesar en el Ver Datos de Necesidad Material.


113837367-Fanger-PMV-Equation-Bug (RKStrand) - Win64-Windows-7-VisualStudio-12: OK (2119 of 2119 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)


Is this the problem we already discussed about qt4, c++11 and msvc2013+ ? Namely that it doesn't enable the initializer lists for that compiler(and newer)? And that it is just a configuration issue with qt4?


I think we should escape whitespace.  Could you use `shellescape()` as below?

```vim
shellescape(expand('%:p'))
```


@benstopford LGTM functionality-wise, have a look at a couple of minor comments above. Thanks.


Woops


Yeah, I would think it would be preferable, in the worst case, to have two formal projects to maintain that isolation/avoid the conflict - that's easily appropriate since they really are independent scripts running in different environments and in different processes.

But now with TypeScript 1.8 you can have multiple .tsconfig files within a single project.  :+1: (But see here where I'm trying to get some clarification on this: https://github.com/Microsoft/TypeScript/issues/4714#issuecomment-187952476).


QuickFixIDD-Memos (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1005 of 1005 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-29.0%25-green.svg)


Pretty sure I'm the wrong @carey.


113837367-Fanger-PMV-Equation-Bug (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2151 of 2151 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)


I think using [Spacejam](https://www.npmjs.com/package/spacejam). It has not yet been updated to work with `test-app` but it will be soon I hope! Coordinating that with @rbabayoff.


I'm not sure 100% what you have in mind here but input from the user comes in as a sequence of tokens of some user-defined type (returned by the function `terminal_token_type` [here](https://github.com/nikomatsakis/lalrpop/blob/master/lalrpop/src/grammar/repr.rs#L243-L245)). We then have to apply various patterns in a match statement to decide which terminal it is (defined by the field `conversions` [here](https://github.com/nikomatsakis/lalrpop/blob/master/lalrpop/src/grammar/repr.rs#L57)). This is all generated code of course. For recursive ascent, it looks kind of [like this](https://github.com/nikomatsakis/lalrpop/blob/master/lalrpop-test/src/expr.rs#L58-L75).

So I imagine that we could supply `TOKENS: Iterator<Item=u32>` here -- but we'll still have to return after each iteration of the loop so that we can invoke the action code (which will be a bunch of generated action fns [like this one](https://github.com/nikomatsakis/lalrpop/blob/master/lalrpop-test/src/expr.rs#L1360-L1369)).

Invoking an action fn probably looks like:

```rust
let sym0 = match data_stack.pop().unwrap() { SymbolType::AppropriateTerminalOrNonTerminal(v) => v, _ => unreachable!() };
...
let symN = match data_stack.pop().unwrap() { SymbolType::AppropriateTerminalOrNonTerminal(v) => v, _ => unreachable!() };
super::__action22(sym0 ,..., symN)
```

where the data_stack is a `Vec<SymbolType>` where `SymbolType` is a generated type looking vaguely like [this one](https://github.com/nikomatsakis/lalrpop/blob/master/lalrpop-test/src/expr.rs#L40-L46) but also with terminals.


hmm that is weird


successful


You don't map these. But again these weren't ever released. They are introduced in the 2nd commit.


Please excuse my lack of scientifik rigour.


@yosifkit, why did you change this? I'm extending this image and it fails on [this line](https://github.com/nazar-pc/docker-webserver/blob/948629f370e406fcdf7cb8ff099481d57afbc8c3/mariadb/webserver-entrypoint.sh#L59) now, on `awk` actually, but without any error messages or something.


113837367-Fanger-PMV-Equation-Bug (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1005 of 1005 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-29.0%25-green.svg)


your _mom_ is successful


How is is giving an error? the xml is fully valid


Point of Sale specifications

•	Store product in database
o	Product ID
o	Product Name
o	Price
o	Current Discount (percentage)
o	Category
o	Number in inventory
o	Minimum number to keep on hand
•	Store records in database
o	Organized by date
•	Handle  a Sale
o	Allow cashier to enter any number of products by ID, or choose from list
o	Look up prices (adjusted for current discount) of chosen products, add them to receipt
o	Allow cashier to enter a global discount ( from a coupon) and adjust receipt accordingly
o	Interface to confirm/finalize a sale
o	After a sale is complete, update inventory in the database
o	Store all information about the sale in the records database
•	Monitor the inventory numbers
o	When an item goes below the minimum number, generate a purchase order
o	Store purchase order in the records database
o	Email the purchase order to a specified email address
•	Receive incoming products
o	Allow entry  of products by ID or from list- same interface as for a sale
o	Update inventory numbers
o	Store all information about the shipment  in the records database
•	Administrative tasks
o	Export product database to printable format- including total inventory worth
o	Export printable audit trail data from records database- all, by day, month, year
o	Return values calculated from the records database
	Inventory worth
	Income from sales: total, by day, by month, by year
o	Add a new product to the inventory
o	Remove a product from the inventory
o	Manually update all inventory values for a product
o	Enter a discount for a category- overrides current discount for all items in category
	No Limit
o	Price changing over time (average cost)
•	No Returns!!


113837367-Fanger-PMV-Equation-Bug (RKStrand) - 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)


I think this is one duplicate and need deleting.


Mom and Pop Pet Store


QuickFixIDD-Memos (mjwitte) - 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)


Nice. British English patterns... I didn't know it. If I assume that
right, a bunch of repo at github makes that 'mistake' and other locals
too. Anyway, althoug 'license' points to verb, makes some sense yet. xD

On 02/23/2016 09:57 PM, Vitor Rodrigues wrote:
>
> This
> <http://www.future-perfect.co.uk/grammar-tip/is-it-license-or-licence/>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tkovs/cbrainfuck/commit/b1b9f7a42c04bb6ad1f6939401a8db5e5d1f3edf#commitcomment-16285332>.
>

--
As lembranças não são só do passado. Podem ser de agora e até de amanhã.
— Serial Experiments Lain
Manoel Vilela,
Discente em Engenharia da Computação[UFPA] - Turma 2014.


I don't see how - these thing is we use `gather_configs` there and here we do not want the local/global bears to be loaded if I ask for show_all_bears (As I anyway need all bears - no point in loading everything).
Also, when doing `show_bears` we dont want to ask for missing settings - because the user may want to actually see the settings of the bear using it.
Hence, i think these are two cases currently not happening in `coala_main` - and wouldnt be considered as duplication


Cool. Will this directory get cleaned up by `medic-clean.js`?


Cool. Thanks for clarifying.


Sounds good.


いままでは、def root の中に `layout 'application'` を書くことで、この**action**では application のlayoutを使うよ、っていう宣言だったんだけど、

この**controller**では layout 'application' にするよ、っていう宣言に変えたっていう感じ。


Please rebase anf issue a new pull request as I just committed and pushed a patch to this file from Mike, W9MDB.

Thanks!


check over commented out function.....we may not need if we are plotting doggy location using shelter.find API call


Thanks for this. Looking into it more, it looks like the -w flag isn't actually necessary, and does in fact break things... I'll revert this commit and push a new version. Sorry for the inconvenience.


Due to this you need to remove `Preferences::appendTorrentLabel()` and `Preferences::setAppendTorrentLabel()`


Or atleast I cant think of any easy way to avoid duplication


Thank you for the update.


Gracias @joecohens, no me había percatado que no lo había hecho. Ya lo actualicé!


>Due to this you need to remove `Preferences::appendTorrentLabel()` and `Preferences::setAppendTorrentLabel()`

You removed the get in the 2nd commit apparently.


because otherwise, the `gunzip -c "$f" | "${mysql[@]}"` below will _not_ fail if `gunzip` fails :disappointed:


In your specific case, I think that means your `grep` is actually not matching anything, so it's returning a non-zero exit code (failure).


yep. thanks for the fix.


If we are going to go that way, then I should really also do partial_apply ; ).

The convert_function was a thinko.


Sounds like frac_second is deprecated in MySQL 5.5. In stead, MICROSECOND is used [1]

[1] https://dev.mysql.com/doc/refman/5.5/en/date-and-time-functions.html#function_timestampadd


I think so, but in my case this is completely fine, I'll just get empty string there.
How can I overcome this?


built app from your source and tried changing any tunables for hotplugs causes fc


I'd recommend adding `|| true` to the end of that line as a fix, ie:
```bash
first_node="`grep -P \"[^_\s]_${SERVICE_NAME}_1$\" /etc/hosts | awk '{ print $2; exit }' || true`"
```

(basically, just being explicit about telling Bash that you don't care about failure on that line)


Because it could be tweaked by the end user, e.g. if they provide custom NH configuration object? There's no way to enforce them to use Guid / Comb key generation strategy AFAIK.


TBH, I lost the original method while editing so I just rewrote it. I am fine restoring it to its previous form if you wish it so.


Test unitaire:

    Assertions:
    ┌────────┬──────────┬───────────────────────────────────────────────────┬────────┐
    │ Module │ Test     │ Assertion                                         │ Result │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │ carts  │ Carts()  │ getLength method                                  │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ 4 symbols                                         │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ 13 names                                          │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ each value is in [2;14]                           │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ pull method, length                               │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ pull method, carts                                │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ add method, multiple carts                        │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ get method, type                                  │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ get method, keys                                  │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ get method, carts                                 │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ add method, one cart                              │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │ Whist()  │ Le jeu est distribué dans son intégrité           │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ Chaque joueur a 13 cartes                         │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ Chaque joueur a des cartes différentes des autres │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ Le joueur courant peut jouer                      │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ Les autres ne peuvent pas jouer #1                │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ Les autres ne peuvent pas jouer #2                │ ok     │
    ├────────┼──────────┼───────────────────────────────────────────────────┼────────┤
    │        │          │ Les autres ne peuvent pas jouer #3                │ ok     │
    └────────┴──────────┴───────────────────────────────────────────────────┴────────┘


I suppose this isn't now because you converted from `enum class` to just `enum`, right?


:beers:


Yes. I intend to separate SSD cache in another functions. There are big difference between RAM cache and SSD cache.


This commit must be ignored. This is actually the next project, named weather.


You put SQL_TSI_SECOND, etc in the keyword lists, yet here model it as a Literal. I'm not sure if that's the right way to do that.

Also, timestampadd(SQL_TSI_SECOND, ...) is same as timestamp(SECOND, ...).  Will this patch work for the late format, if we use the same approach, and treat "SECOND" as literal?


Sorry if this has already been asked, but is there an approx. release date for 3.0?  I'm planning a new product feature that will use the SDK, though not really any of the charting features at all, so wondering if there are compelling reasons to hold off a bit until 3.0 is out.  Our markup will typically rather routine -- perhaps some lists and tables and hyperlinks as well as the usual formatting, but not really much fancy content.

Thanks in advance.


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


This is what I was exactly what I was getting at. Now we just need to be able to feed this data from the backend.


good


Thanks for quick answer, it helped!


Can you please submit PR?


read it


:+1: :metal:


@bberenz perhaps we pass it as an optional [`options`](https://qz.io/api/qz#.print) parameter?  The URL encoding fallback seems relatively benign, but it seems inconsistent with our other base64 portions of the API.  `MalformedURLException ` may catch "AAABBBBB==" 100% of the time, but it has the possibility of also catching "foo://myimage.jpg" as well, which could in turn result in a confusing base64 decoding error.

I'm not all that excited about it, but how about this?
```diff
{
     type: 'raw', format: 'image', data: 'AAABBBBB==',
     options: {
       language: 'ZPL',
+      format: 'base64'
     }
}
```


@singun
12 / 6 = 2
2나옴 ?


These method names are slightly misleading/confusing. The helpers return the _number of_ error messages and the _number of_ images, not the error messages themselves or the images themselves.


Don't know I check the file is the same as on git, that way I ask about my machine my linux OS, maybe a language missing if that is possible.
I building with clang. in the build log there is only that about the error, remove the file build successfully...

I try again tomorrow If is ok if I fix, I post here, is probably my machine. :+1:


You are right. we should not limit this to a HW RAID only. Will fix it. But I am afraid I might no be able to add support of
any SAN arrays in a short time.


제출할땐 package 들어가면 에러남


read it 睡着么晚啊！


이런건 당연 제거 했겠지 ...


OK.


@Kurami- HELP!!!


Maybe it is because I am sleepy, but can you explain what you do here? I can't wrap my head around it.


Also, for `diffie-hellman-group-exchange-*`, if `p_bits+1` is not multiply of 8, `group_order*8` can be larger than `p_bits` (by up to 7 bits); thus, generated `group_order*8-1`-bit random value `x` can fail `1 < x < (p-1)/2` test, see [rfc4419](https://tools.ietf.org/html/rfc4419#section-3)


Novos doubles para a função de linearização do resistor linear por partes.


I agree with @mymedia2. Repeated code here.


Will do.


Nova variável de controle.


Address the issue in https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53839993


read it


Rather than adding another repo argument to this constructor, I'm just passing a dbMap instead


Função que resolve o sistema para o Newton-Raphson.


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


If the condition has the `public_key` in it it might be redundant to require it in the fulfillment


sorry, the variable name here I think `weightedTerm` is better than `term`


Delete - me


Seems like a description copied from a more general field. There's no public key involved here


Okay, I was merely the caretaker, merging things that had been approved.
Please send the revert to whomever gave LGTM on this PR

On Tue, Feb 23, 2016 at 8:42 AM Pawel Kozlowski <notifications@github.com>
wrote:

> @alexeagle <https://github.com/alexeagle> I don't think that this change
> is correct, IMO it should be reverted, see: karma-runner/gulp-karma#18
> (comment)
> <https://github.com/karma-runner/gulp-karma/issues/18#issuecomment-116766251>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/angular/commit/27daeaff5e5fba05e6bdd7c36273d47646d69a42#commitcomment-16272915>
> .
>


Address https://github.com/AppFolioOnboarding/Haomin_ImageUploader/commit/1cfde6c1a4db1d9b7c45973e3cdf36e6dd3c8620#commitcomment-16276091
But I still have to add above two lines here otherwise the constraint on format doesn't work


'datahub dp create' 是datahub的命令


Tomada de dados dos valores do resistor linear por partes, e automaticamente já desenvolve o I0, G0 etc.


read it 应该再找一个人和你review的。


Yeah. That is better. Changing.


There's some debug code that remains in AActor::CheckPortalTransition().


I still think the append operation makes it look confusing even I know what it's for.
Do you think the following is better?
```go
termsToValidate := []api.PodAffinityTerm{term.PodAffinityTerm}
```


This was my change


Fixes #16


Just heads up: This isn't PHP 5.3 compatible. Using `$this` in the function context doesn't get bound like this until 5.4; it'll throw an error. If you're not going to use the logic outside of this closure, then you don't necessarily need it to be a class method; you can just put it right here. If you need it later in this method, e.g. for other scripts, you can just assign it to a variable and pass it in to a few different `register_X` params.


Variável de controle.


This .jscsrc file is used for the linter shouldn't have been added. I removed it in a following commit.


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


In the SignedMessageTemplate this field is called `state`. I'm still a little confused about what this field is for but at the very least the name should be consistent


Estampa do N.


Error:please input a valid datapool and path.d  多了个.d


Newton-Raphson versão Alpha.


> On a related note, it might be worth setting the `:stubbing_non_existent_method` option in Mocha::Configuration to `prevent` globally and see whether any tests fail.

We already have this option set in our [test_helper.rb](https://github.com/alphagov/smart-answers/blob/master/test/test_helper.rb#L18).


这个需要san来review一下。似乎没有完成呢。


I think these should be called `__SignedMessage` rather than `__Receipt`, because the former is the more general type.

`Ed25519SignedMessage` and `Sha256SignedMessage` are arguably instances of the `SignedMessageTemplate`. And the `TransferStateReceipt` happens to be an example of a SignedMessage


I got this to work, but for some reason when using the react components, I run into the Uncaught Invariant Violation issue. Does this have something to do with my webpack configuration?  FWIW, I used https://github.com/banderson/generator-redux to generate my projects scaffold.


I don't seem to be the only one: https://facepunch.com/showthread.php?t=1507496


If this is a general `SignedMessageTemplate`, I think the transfer state should go elsewhere. If that needs to be a required field for the `TransferStateReceipt`, it should arguably be a property on that schema


添田です。
課題4については、yesとnoがルート逆になってますね。フローとしては問題ないので、ＯＫです。
課題9については、脱出条件が20のため、19回しか回らないですね。確認してください。
課題10については、奇数だった場合に即ループを脱出しちゃってますね。確認してください。
課題11のその2については、入金待ちのループ図は無くても良いかもしれませんね。


Please add a description of why you're taking out these functions to the commit message


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


添田です。
課題13のサーチアルゴリズムのランダムで考えた方は、昨日も話しましたがよくありません。
1回でいきなり当たる可能性もありますが、無駄に試行回数が増える可能性もあります。
また、システムに問題が発生しても再現をさせるのが難しくなります。
（そもそもランダムを採用しているため）


@leninkp `#learnContent(contentId, neuronId)` ^


@leninkp `#learnContent(neuronId, contentId)` :wat:


It's an interesting issue which I'm not quite sure why this is happening. I can reproduce it in a production app I'm working on, but I'm still tracking down the cause of the issue. Initially I thought it was a Babel 5 issue which is why I made the `babel6` branch but it leads to the same result.


Prefer to use `if callable(callback)`


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53830966


Shouldn't. try using mine as built. http://bit.ly/1TxU6C9

If it still force-closes, get a logcat.

Otherwise, it's on your end.


no falta `/`? -> `/contents/`


Should we also be doing this to add dependencies?

```
godep save $(go list ./... | grep -v /vendor/)
```

Instead of the previously used `godep save ./...`, since `./...` may expand into things that are inside /vendor/ and I assume you don't want to vendor dependencies... of your dependencies... that are not used by your actual project (i.e., the `go list ./... | grep -v /vendor/` packages).


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


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


Oh snap that was supposed to be on my private fork.
http://review.cyanogenmod.org/#/c/133956/ Fixes that too
and I actually enjoy this questionable feature
This either locks the home button on alarm or is related to the shut off power dialog that pop-up up, both which I love


I changed this branch because I wanna try this feature.


That's the tricky part for MegaRAID cards. The MegaRAID card I have is forcing enable SSD physical disk cache and does not allow change that. I will rephrase that to "check with the vendor of your SSD disk and hardware RAID card".


Fix
https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53826540


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53826748


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53830410


`state`=transfer's current state; `condition_state`=state user requests transfer be in when computing the hash.
https://github.com/interledger/five-bells-ledger/blob/master/src/models/transfers.js#L67


This is too simple of a solution. What I tried to do was set n (i.e. 100) reviews to pick, and a percentage of occurrences of tags in the reviews (i.e. 0.8). We would pick the percentage of n (i.e. 80) reviews that contain any of the tags, from the 60 thousand-something reviews in the reviews.csv.backup file. The rest (i.e. 20) reviews don't have to contain the tags.


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53832080


Just wanted to let you know that the pager (the dot navigation) doesn't work while using touch swipe.
They stay in the same position. Is there a way to fix it? Please let us know. Thanks!


kala sang idolaqq.com [agen domino qq](http://www.idolaqq.com) sedang galau mencoba buka Fb chat sama temen yg tidak jarang maen pun eh ia ngasih tau web [agen poker indonesia](http://daunpoker.org) yg baru dgn penampilan yg lebih keren & mutu server yg tinggi, udah gitu bank yg di pake serta gak cuman BCA yg lain serta sanggup MANDIRI, BNI, BRI.
iseng ane datar tetap deposit deh tuh gocap maen, masuk room masihlah sedikit yg maen room yg terisi cuman 3 udah gak full table itu room menjadi peluang menang tentu lebih agung, main-main sabar main-main lagi bermain & bermain alhasil ane WD jg 750000. ehmmm... gak galau lagi deh gan : )
Sejak Mulai sekarang ini poker88.asia ane nyatakan PENIPU. & ogah untuk maen lagi, mnding di elangpoker.net aja deh, oh iya gan emoticonnya seru pula gan emotion roomnya serta gak bikin ganggu permainan, keren - keren bikin nafsu orang tuh...hahha ayo pindah kesini seluruh janganlah di lapak penipu sayang uang kalian.
mencoba list serta-merta.
sedikit penjelasan gan ane penasaran lacak ipnya eh nyata-nyatanya & nyata-nyatanya tetap di singapore, dalam hati ini tentu sama tajir poker88.asia atau dewapoker atau yg yang lain lah, namun jelasnya sih nggak nih beda casino yah gak tau benar atau salahnya itu pula kata orang, menjadi semangat lagi utk ngulik flash pemogramannya nih kala ini masihlah belum pun liat tajir dengan cara apa dalemya, agar yg list melalui ane dapet card keren & menang konsisten..hehe
oke gan segera teliti dimari supaya menang serta-merta list, masuk menu deposit transfer confirmasi deposit tunggu 5 menit utk bikin kopi tuh di kasih kala, kembali ke monitor layar & mainkan, jangan sampai lupa minum kopi diwaktu kita galau liat chips sejak mulai turun supaya santai maennya & naik lagi,hehehe menjadi pingin ketawa liat pengalaman tadi maen.

sekian banyak kelebihannya di banding website poker yg lain gan :
1. bank yg di gunakan gak hanya BCA, MANDIRI mampu BRI dapat, BNI serta mampu
2. minimal deposit 25000
3. bonus referral 10%
4. kabar adminnya komplit disertai nomer hape & pin bb, menjadi klo deposit gak masuk tinggal telpon aja atau bbm.
5. lebih enteng maennya lebih kumplit beritanya lebih serta-merta servernya
6. flashnya paling tinggi pokonya maknyus
7. ada chat serta-merta di websitenya utk ke administrator lewat yahoo massanger chat
8. emotionnya seru seru

oke lumayan terang gan saat ini tinggal berfikir ingin masuk referral aku atau nggak boleh boleh aja klo ingin masuk referral tinggal klik link di bawah ini
http : //referral.elangpoker.net/ref.php?Ref=CIWO77
ingin serta-merta ke situsnya pun dapat tinggal ikuti aja langkah pendaftaran dibawah ini

Arahan List :

1. Mari klik link ini : www.elangpoker.net
2. Sesudah itu sehingga dapat muncul gambar di bawah ini

Sesudah kamu menonton gambar diatas, mari klik "DAFTAR" di pojok kiri atas.
sesudah kamu klik "DAFTAR" sehingga dapat muncul form pendaftarannya...
isikan from pendaftaran yg benar menjadi bila kelak ada kesalahan administrator mampu konfirmasi sama kita lewat sms atau email, satu lagi kata sandi mesti huruf & angka.

Kalau ada yg masihlah bingung secara menambah account balancenya aku jelaskan dengan cara singkat nih yah gan kan tidak sedikit nih pemula yg baru maen soalnya kalo di website lain mesti pake bca aja klo disini nggak cuman bca bank lain serta mampu, so... tentu tidak sedikit pemula yg maen, serta-merta aja penjelasannya :
sesudah list dulu login masukan uerid & passwordnya tetap masuk ke menu deposit perhatikan disana Rek. administrator dgn banknya klo agan memanfaatkan mandiri berarti kita transfer ke mandiri pun, klo BRI yah ke bri pun, dulu ambil hpnya untuk transfer lewat m-banking atau ke atm tulis lalu no rekeningnya, klo telah laksanakan trasfer kelak perhatikan distruk ATM ada keterangan dikala transfer jumlah trasfer, masuk ke menu deposit isikan fromnya cocok struk atm, & mengisi keterangannya dgn apa aja terserah agan ingin, ambil yang sevty aja agan isikan bersama tulisan 'Deposit bos' dulu klik kirim.

udah sukses proses confirmasi deposit tinggal bikin kopi lalu untuk nunggu chionya masuk, klo umpamanya lama masuknya agan logout dahulu tetap login lagi & masuk lagi ke menu deposit kalau tetap sanggup melakkan confirmasi ulangi langkah diatas, kebanyakan seandainya kita udah confirmasi kita tak jalankan langkah di atas 2 kali kebanyakan ada pemberitahua gini "anda memiliki deposit pending, tak dapat lakukan deposit ketika ini" nah kalo udah ada tulisan begono tinggal tunggu aja klo lama masuknya mencoba ke luar & masuk lagi ke menu deposit konsisten aja konfimasi klo tetap mampu, konsisten perhatikan di memo outboxnya udah ada belum permintaan deposit klo udah ada tentu masuk chipnya.

Aku bakal memberi penjelasan di sesi berikutnya. Selamat main-main mudah-mudahan beruntung.


Do you think it has something to do with having react 0.13 as a dependency?


could you please tell me what is generic voltage?is which part of soc or something elase?


Yikes that is confusing


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


Every node that is an <entry\> has an id attribute, every file that inherits from entry_id.py has entry nodes.


One more thing, the root cause making it ugly is you are calling `validatePodAffinityTerms` to validate a single `term.PodAffinityTerm`. Please note that in the case that the data is invalid, the actual Err msg would be `fldPath.Index(j).Child("podAffinityTerm").Index(0).Child("matchExpressions")`, while `term.PodAffinityTerm` is actually not an array.
So, probably you need to add a new function `validatePodAffinityTerm` which accepts only one `api.PodAffinityTerm` each time called, and in `validatePodAffinityTerms` you can just call `validatePodAffinityTerm` inside for-loop.


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53830966


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53830665


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53830579


> I'm not sure I understand why you've used `quacks_like` in these tests...

I wasn't initially, but rewrote the commit to use it after using it in a later commit. I think it might've been in [this test for `#country_name_partner_residence`](https://github.com/alphagov/smart-answers/commit/d49dd6f706bf2658037846dc1d872b502d0b7878#diff-20aacf6cc6d132ec2007bda1c5877fedR392). I'd initially used a real `MarriageAbroadDataQuery` before realising that some of my tests were relying on the actual implementation of that object. I then switched to a standard `stub` object before using `quacks_like` to ensure that my stub conformed to the interface of the real object.

Do you think this is a valid use of `quacks_like`, or should I stick with using either the real object or a basic `stub`?


yep.. still fc.. here is a log opening app, selecting hotplug tab, then trying to change tunable
http://hastebin.com/obogixocul.avrasm


I'm open to putting this elsewhere. The reason it is in the `message` is because there is already code on the ledger that puts this field inside the message of the transfer state receipt.


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53831417


Hi. I was just browsing the code and noticed the same typo appears in the preceding function:
```C++
void stdcall i64_srl(unsigned long long* a, asDWORD* b, unsigned long long* r) {
	*r = *a << *b;
}
```
I didn't entirely familiarize myself with the rest of the code, or use the tool, but I interpolate that "srl" stands for logical shift right, and those arrows are certainly not facing right. Cheers.


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53846628


Not your fault, just some conflicts with different versions of the background downloader that I merged.

Enviado do Yahoo Mail no Android

  Em 16:39 seg, 22 de fev de PM, Errietta Kostala<notifications@github.com> escreveu:
Oops

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


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53831758


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53831841


oh and its blu_plug hotplug that was tried


It never becomes false.


That sounds like a good idea. I'll add it to my todo list.


上面有两条评论需要修改


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53832364


Can please use "with prepare_file" like other tests do? Makes it easier read


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53832479


Unfortunately it does not in this case. I updated it to React 0.14 on my local build and it's still not working in a more complex example (that is not this example).


Please comment out `RequiredDuringSchedulingRequiredDuringExecution` related logic, since currently it can't be complete -- RequiredDuringExecution not implemented.


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53833204


It is a temporary behaviour.


Prisoner416 is going to murder you for this. "Alright" is generally considered informal/incorrect by most.


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53833304


This ``pass`` means path exists. A ``pass`` in an ``except`` should (there are exceptions) lead to a further negative return. Not the case here, where the ``except/pass`` leads to a ``return path``. I'd suggest to test if the path exists in first place, then (if not) try to create and fails (``except``) on creation error.


wrong injection type


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


so what is the right place to file issues?


s/Podffinity/PodAffinity/


On Line 154, 165 and 172 I created three separate test to fix:
https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53833610


Looks Gross To Me!


Doesn't matter I made a mistake in one of my arrays... I didn't check if it was null first.

I'll have to look at it tomorrow probably.


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion-diff-53833674


Ah, this is more clear with the visual. I don't think any of us are 100% clear. I'm probably 80% and I think Joe is 85%.  Anyway, I think we're making the Add-Art logo the red +? Let's all try to discuss – I'll send JDP an email...


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53833797


This didn't get re-added


I can't seem to drag-and-drop a pinned tab to unpin it when there are no unpinned tabs. otherwise lgtm


k.. Thanks man..


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53833883


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53833945


这是一个测试


My only feedback is to close the PHP and treat this output as HTML. You should also get in the habit of making any text translation ready (https://codex.wordpress.org/L10n).

For instance, after the closing comment block `*/` I'd change it to the following instead:

```
?>

<?php _e( "Hahaha. You just got Julian'd!", 'gravity-forms-pdf-extended' ); ?>


ça aurait été cool de mettre le .filter à la ligne, j'ai eu du mal à lire (mais c'est ptetre parce que je suis bourré)


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/pull/49#discussion_r53833992


@atemo, why did u delete the woe_te branch? D:
I'll merge "the final changes" of branch after 'everyone' test these restriction to woe_te branch, so it won't mess the woe_te branch.

but seems, no one even care to test, right @aleos89 @Lemongrass3110


Sounds good. I'll make that change.


Should lodash-es be a devDep?


We discussed it over in https://github.com/jpetazzo/docker-busybox/issues/19, but that'd be https://github.com/docker-library/busybox (which will be pushed to the image readme via this change through our CI within the next hour or so).


Skip 4 for now just. Add back later once stable.


I feel like using a new vendoring tool should be left to a separate PR.

I'm fine with using the vendorexperiment in general, but we have so few dependencies this should be as simple as adding them to /vendor and we can figure out tooling later.

aws-skd-go already uses /vendor (so deps will be recursively included).
coreos-cloudinit uses pathrewrite (so deps are already managed)
Just means you may need to manually add:

github.com/spf13/cobra
github.com/spf13/viper
gopkg.in/yaml.v2


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


> if I have HMR enabled in my iOS app, the android "Live Reload" option seems to stop working

That's expected. When HMR is enabled we don't do live reload to optimize the HMR code path. Do you develop both iOS and Android at the same time? Maybe we should envision supporting Hot Loading enabled on more than a single client at the same time.


If we just use `postgres` for user and database (which are created by default) we don't need to create new users and databases etc. Simpler.


QuickFixIDD-Memos (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1563 of 1563 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.71%25-green.svg)


We want vendored code to be added...


init


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/commit/3d1eab761018c8c92b759fa83affbd376ea356fd#commitcomment-16278265


... Grave Robber ...


This isn't super helpful


Fix https://github.com/AppFolioOnboarding/Haomin_ImageUploader/commit/765e8bf6f1a57f374a2d6a49ca0c694f923f6ba3#commitcomment-16278281


I guess that line would need to be modified to something like:

> To remove this welcome message, remove the `ember-welcome-page` package from your `package.json` file


This if is duplicate of a previous one.


eh ? There's no difference between this and before except the name


ack


> A word is missing here

Fixed, thanks.


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


is that necessary?


Project Instructions


есть ли смысл продолжать если user == null ?


Removed.


s/much match/must match


is `addon/styles/ember-welcome-page.css` automatically included in the `vendor.css` file?


I agree, this operator doesn't make sense. I think it can be removed.

Yves


removed


LGTM


the new one uses hanging indent


> Imo genericType should only be used by generated code. Clients should never use it because we cannot trust clients to ensure that eg Java and Dart will always stay in sync and use the correct type.

That's how we'll show it in the examples, yes. If people want to do more, that's up to them. The most reasonable use case would be if people want to plug in some other serialization framework.


< DIV  класс = " телефон " > <  HREF = " тел: // 7-347-287-1187 " > +7 (347) 287-11-87 </ > </ DIV >


**HEAD commit has no message.**
*GitCommitBear, severity 1, section `commit`.* *Project wide.*


@Turbo87 I believe so. It got added to the dummy app, at least.


#OC


does it work?


Три замечания:
1) Где проверки?
2) Все в одном фпйле?
2) В данной реализации стек может расти, но не сжиматься. Причем растет он сам, когда места не хватает, но сам не сжимается. Исправлять не обязательно, но стоит иметь это в виду.


Sorry for not noticing this at once, when riding an elevator one can see
the projected crosshair drag vertically. You fixed the horizontal drag but
there is vertical as well.