Whether or not the OOME actually occurs really depends on the configuration and usage.  Consider a broker with 1 address and 1 queue where the max-size-bytes is 1GB but the heap size is only 512MB.

I'm not sure what you mean when you say, "This could be a valid configuration for certain users who monitor outages."


A little note for later: while a strategically places `abs` here and there may be useful, sprinkling it *everywhere* may not be the cleanest way of going about this. Let's look at ways of improving upon that in the not too distant future.


Change the doc to tell this. doc/installation.rst


and your filename... it should be src/pocketmine/block/GoldenCarrot.php , not src/pocketmine/block/GoldenCarrot


Strange, looks like I never wrote the end of that comment, since this is the first commit in which it appears. I might have wanted to say something like 736585e


Are you positive? I understand that this.userId would be defined in the publication as the context is the publication at that point, but once you've reached the function scope for the participants function this.userId should be undefined as the context switched from the publication to the conversation.


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


Hi Chris,

Just wanna start off by saying I really appreciate the effort and the time spent. Its hard to find people to help out. A few comments regarding your code.

1. For save(), I was thinking the server will return the HashMap representation of the events list (or sheet as you call it) to the front end, and it will appear as a download for the user. User will upload the project file everything the server restarts. Basically, no need to save the project file to disk. We let the browser and user handle that.
 1. Every time the server starts and the user loads the app, it will ask to upload a project file (we will use a file reader on JS and send it over as JSON) or create a new project
 2. The only file we need on the computer is the music file and we only save the path to the music file (absolute path is fine)
2. I think the use of Timer is not feasible. If you read the Java API, the delay is not guaranteed and it may chalk up if the background thread is chalk full of requests. Also, the delay doesn't take into account the processing time of the actual command. So, the longer the timer runs, it will become more out of sync.
 1. On this topic, I want to discuss with you a way to do an [observer-observee](http://www.tutorialspoint.com/design_pattern/observer_pattern.htm) approach to firing off commands to the LEDs.
 2. The idea is to register each LED device to an events controller. The events controller will have the master list of events and create a duplicate just before the music starts. This will run on a thread (events thread)
 3. There is a separate music controller that will playback the MP3/WAV file. (this will run on a music thread) When the music plays, the events controller will spam poll the music player for the [current playback time](https://docs.oracle.com/javafx/2/api/javafx/scene/media/MediaPlayer.html#getCurrentTime()). Each time it polls, it will check if the current playback time >= first event. If so, run the command --> This should be a runnable (this can be dynamically created when the project file is uploaded to the server) and spin off a new thread so it doesn't block off the events thread. Remove event from list. Check the next event if current playback time >= first event, if so run else sleep for a while and poll again.
3. The project controller is only for reading -> push data to memory then initialize the music and events controller, and saving -> push data to client
4. We can spin off the SheetHandler to become the events handler listed in 2iii.


@msmollin I added a test implementation for getSearch(...) using a focus lat/lon point. The other methods I just formatted the params.


Question: How come we close the db after moving and not before?


extract a def, and prefer pattern matching to if/else, and to be perfect it should be tested


Let's remove the match so we don't crash on completely garbage data.


What's that spamming?


with this change the design view on android studio works


Should this be read with the mutex locked?


Why?


Alternatively, could introduce another function head for garbage data and throw a different error


Progetto interessantissimo, complimenti! :wine_glass:


The reason I didn't is that I added a bunch of additional regexps for more specialized categories, and figured that providing boilerplate accessors for all of would be noisy.


Please revert, I'm not ready for Rdio to go away.


The call visit(context.privilege()) ultimately results in calling the method visitPrestoPrivilege() (through the registered callback function) which constructs an instance of PrestoPrivilege class, which is passed as the first argument to the Grant() constructor.


Adding a new function which hardly anyone knows won't change much, 99.9% of all add-ons would keep causing warnings. I'd merge a change to add `Package:isInstalled` if a function like that would exist in 5.7.


Agreed, that's a better name.


It's already in base.yaml


Thank you


Will, the code layout should be updated like something as below:

$ tree -d
.
├── common
├── resourcemgr
├── sysapi
│   ├── include
│   ├── sysapi
│   └── sysapi_util
├── tcti
│   ├── localtpm
│   └── tpmsockets
└── test
    ├── common
    │   └── sample
    ├── tpmclient
    │   └── good
    └── tpmtest


This fixes #3


return back


Fixes #2


references on type class technique
- http://danielwestheide.com/blog/2013/02/06/the-neophytes-guide-to-scala-part-12-type-classes.html


void saveBackup() Сохраняет бэкап
void loadFromBackup(String fileName) Загружает из бэкапа
В FieldChecker изменилась сигнатура checkTrainSearch - добавлен параметр со второй датой, появилось исключение TimeCollapseException


It seems that changing the controller has no effect for the line:

`get 'profile', to: 'users#show', controller: 'users'`

For instance, if I change the line to this:

`get 'profile', to: 'users#show', controller: 'pets'`

The output of `rake routes` still includes `users` as the controller:

`profile GET   /profile(.:format)                          users#show`

But if I change the `to:` option to use the symbol form:

`get 'profile', to: :show, controller: 'pets'`

Now I get my pet profile in the `rake routes` output:

`profile GET   /profile(.:format)                          pets#show`


We'll probably need a lock per ID in many cases rather than locking the entire store... or at least in addition to locking the entire store, where the only time we lock the entire store is for creating/removing dirs and the walk function...

Or at least an RWMutex.


Thx! What are the rules for a package being in community vs testing?


INTEGRATION TESTING_2015-11-23 17:23:02


INTEGRATION TESTING_2015-11-23 17:23:05


Server is now sending back attacker and defender, while front end is updating only these two on attack. Every 30sec, btw, the client request a full fetch.


No i did not. But i feel this ticket should not be part of this week's
sprint because we agreed that when one uninstalls the legacyui module, they
should restart tomcat. In other words, this is not a high priority one. :)

On Mon, Nov 23, 2015 at 3:32 PM, Rafal Korytkowski <notifications@github.com
> wrote:

> @dkayiwa <https://github.com/dkayiwa>, did you have a chance to test it
> out?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openmrs/openmrs-module-legacyui/commit/1fea16d509f122a1c99dd289542f56440b6bd2fe#commitcomment-14558764>
> .
>

--
If we keep uppermost in our minds the unkind and unjust acts of others, we
shall find it impossible to love them as Christ has loved us; but if our
thoughts dwell upon the wondrous love and pity of Christ for us, the same
spirit will flow out to others.


> Also, we should probably just force it, since it crashes otherwise.

Agree, although it'd be nice to find out why it crashes and fix it.


;c


whoops


The problem was, if I remember correctly, that we use camelCase in the JS environment (most commonly used) and snake_case in the Rails environment (most commonly used). Therefore we couldn't just use the railsy version, but had to `camlize(:lower)` it in order to use it in the react environment.

The "obvious" thing you are missing is that different environments have different best practices regarding casing (compare [React](https://github.com/facebook/react/blob/master/src/React.js) and [Rails](https://github.com/rails/rails/blob/master/activesupport/lib/active_support/dependencies.rb#L29) as great examples for each environment)


If we want to protect ourself from bad clients ever sending us a malformed `#tscell{}` (which is still a `#tscell{}`), then let's extend the protection to any term? Just omit `#tscell{} = ` in the expression in the head of the list. It comes at no additional cost.


*not* supported


Yes, you're right about ```QualifiedName```.  ```name``` is the name of the identity -- either user or role name. It is yet not clear whether this would be a single identifier or would consist of a hierarchical name. For ex., a username can be ```hive.admin``` to specify a hive specific user. I have used hierarchical name to be on a safer side (any hierarchical name can also be a single identifier).


Since it's not Unix and this part of the code is for precisely this purpose, why not make it a more "normal" layout?


Fixed now. The warning is of course just a result of the size of the initial design. The error was coming from a stupid mistake on my part (sqrt of a negative variance, positivity was enforced later than was necessary). Bernd where should I put said tests? I have the one earlier that @giuseppec posted above, which the code absent the above fix passes. The above example is pretty long for a test. If we can come up with a way to generate a negative variance estimate reliably just from the learner/newdata args then that would be what we would want, no?


Yes


2.6 compatibility


dig it


This is how I'm using it: https://gist.github.com/Neobii/9543721397b1a5f68491 with a package called publish composite I mentioned earlier which really seems to fit what you are trying to do here.


I think you are not removing the plugins from the mainController. To test this try:
1- insert addictive drums
2 - start a drum groove in addicitive drums
3 - click in reset - the plugin will be removed from GUI
4 - I'm suspecting you still will hear the drum groove because the plugin is not removed from mainController.


Meh, this and below were just found by trial and error, can easily remove or adjust as needed. I never said it was 100% perfect :)


+DEPENDS+= python


@glogiotatidis would you mind replying with a link to the github issue that has more details?


:ok:


Why gettext but not intltool? Or did you just forget to spam one of the items? :)


+DEPENDS+= python3


This isn't bad. Seems like we should be able to tweak the RE only though instead of what you did. I'm willing to merge this if you need it right away, otherwise I'd like to take a second look


You missed this one, it was from when I was downloading Geany's source code as a .zip file from Github.


And here (see I can do it too :)


I find this hard to read.

if 'Version' in layer \
         return layer['Version']
else
         return "Version unavailable"


Probably. It was added when I was having problems with 64-bit build which mysteriously went away (PEBKAC no doubt).


Take `port` and `path` from the `settings` file.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1433 of 1433 tests passed)

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


Yes, per the comment in the file, it was added to help prevent autoconf from getting called.


I like it, will change.


ptyhon3?
print(bar.images())


300 seems arbitrary. What about:

signconf_del = malloc( strlen(".ZONE_DELETED") + strlen(zone_signconf_path(zone)) + 1 );
/* check malloc here */
strcat(signconf_del, zone_signconf_path(zone));
strcat(signconf_del, ".ZONE_DELETED");


Should this be WriteOut?


same same.


Well, I hadn't written a separate visitor function in ``AstBuilder.java```. But thinking about it, I think it's a better idea to have a ```visitPrestoIdentity()``` function instead of the current ```processIdentity()```function since ```processIdentity()``` is basically a visitor for ```PrestoIdentity```.


Bad exception call here


@HenocKA If you look at the current version of the document, you'll see that the link has been fixed: https://github.com/librenms/librenms/blob/master/doc/Developing/Dynamic-Config.md


We already have such stuff for OSX, and it was needed to disable the file dialogs and use normal install dir paths. It's hardly a massive amount of maintenance effort to have a couple `#ifdefs` to allow a more natural feeling install, and it will be used more later for stuff like restoring a decent native-theme depending on the Windows version and such.


test


@LeeKamentsky :doughnut: :doughnut:


Meh


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


Those tests are fine, but they belong to `engines.gdb.lookups.neo4j`, thus, to the `engines` application.


Yeah, not needed anymore. I removed that so we have adb now


@puffnfresh should it be `f(j)` instead of `Applicative[F].point(j)`?


yay


Yes. But next commit has everything that was missed. Sorry

2015-11-23 15:52 GMT+01:00 pierreganty <notifications@github.com>:

> The message claims more than the actual commit. Did you forget to include
> something?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/elenagutiv/Linearisation-2015/commit/3b98b3daacfc8b713dc76f6c7179d3134de58593#commitcomment-14562461>
> .
>


add a \n after 'vs' ?


@Utumno I can't read all the Python code.  However, now that you have moved that are you accounting for the header differences in the records?  Oblivion is 20 and Skyrim is 24 bytes.  Is there a check based on the game mode for that?


I would say maybe `model_objects` to be a bit more precise. I'll try to change it in another PR.


holy cow


more clearly perhaps do an elseif before the next case, and just a single return at the end?


Good points here.

Joe Murray, PhD
President, JMA Consulting
joe.murray@jmaconsulting.biz
skype JosephPMurray twitter JoeMurray
416.466.1281

On Mon, Nov 23, 2015 at 9:47 AM, Edsel Roque Lopez <notifications@github.com
> wrote:

> We could do a COUNT(sid) instead of COUNT(nid), but it would essentially
> be the same thing.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JMAConsulting/biz.jmaconsulting.nrm/commit/0e6705ffab738b39052bff23ce705f7d97f12558#commitcomment-14562286>
> .
>


? that one one surprises me a big


Here


While I remember, it'd be good to avoid passing `arguments` objects around as it [kills v8 fn optimization](https://github.com/petkaantonov/bluebird/wiki/Optimization-killers#32-leaking-arguments). When I get a spare min I'll have a look and see if I can make any little improves like that.


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


Hi,

Thanks for the hint :) I'll fix it asap.

Best,
Eric


This fixes are needed for the following directory structure:
root_dir/CMakeLists.txt    ... that is the starting point of MyProject
root_dir/build        ...here is the out-of-source-build
root_dir/MyProjectSrc/CMakeLists.txt
root_dir/ext_libs/FreeRDP/CMakeLists.txt
root_dir/ext_libs/RemoteDisplay/CMakeLists.txt


I'm very much aware of the best practices as far as casing. What i'm not getting is how people manage to deal with this difference.

The assumtion here is that data is coming from Rails into React. Is the idea that we change all of our json model serializers to output camelCase just so that React can read it properly?

I'm guessing theres a lot of people out there using APIs with snake_cased property names with React.
And yet I'm not aware of a simple way to transform the snake_case keys to camelCase.

That's the "obvious" thing i'm referring to. Maybe people just manually convert in their js API wrapper code like they seem to do in the React examples.


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


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.826](http://teamcity.monogame.net/viewLog.html?buildId=11659&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 552, ignored: 6 Build time: 00:20:51


The indent in above two lines is still not good enough. More of the code is using 4 spaces as one level of indent, here is one tab instead.


yeah some of these are crazy and make me want to clean up the dependencies in the code (which i think is a great thing)


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


I see the change now thanks for the heads up.


we need to remove cr0, the last param.


sinto que estas mudanças foram muito pouco eficazes.


Esta correto escrever "Gera suspeitas" mas acredito que o termo "traz suspeitas" encaixaria se melhor na oração.


more like update you gayness


Only if we bump the duktape version, also the docs states that duk_push_external_buffer is experimental.


I still don't get how it's supposed to work - `object` is `const` while `optional_object_` isn't.


I've decided to keep using it for the gradle plugin. So I had to enhance it
a bit.

2015-11-23 15:23 GMT+01:00 Sam Brannen <notifications@github.com>:

> Wow: the ConsoleRunner is becoming very feature rich.
>
> Nice work! [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/junit-team/junit-lambda/commit/884efc71012d74ccab55c8f98b823e067a6d0e29#commitcomment-14561553>
> .
>


Yea, its because when `user.delete()` is called Django kindly goes and
tries to delete from every table that has a foreign key related to the
model. The way to fix it is to use raw sql for the delete but that is
arguably as bad:
https://github.com/dimagi/commcare-hq/blob/64f0f67a97f7605bc0a7e240c5665136fe3ad635/corehq/apps/domain/models.py#L920-L920

On 23 November 2015 at 17:35, Ben <notifications@github.com> wrote:

> ? that one one surprises me a big
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dimagi/commcare-hq/commit/278656cb38a01162d9856d48dfb54c7c0716ff6d#commitcomment-14563961>
> .
>

--
Simon Kelly
Senior Engineer | Dimagi South Africa


basic dick handling completed. what a faggot project


it's not just history but also movecount, futility and SEE


If init is called twice, we end up with two bookshelf objects anyhow, which I'd imagine will usually be a bug. As it stands, there's no way I can see for the user to hit a duplicate model error, since init() creates and returns a bookshelf object with the models registered to it.

If we want to handle the case of init() being called twice better, I think the way to do it is for init() to check if it's already been called. I'd like a better way of structuring that function anyhow (I'm not entirely happy with enumerating individual models through requires), but as it stands, this commit shouldn't have any externally noticeable effects.


mmm, sorry but I still can't see it. Can you give the SHA1 of the commit
adding the shell script running mustache?

On 23 November 2015 at 16:34, elenagutiv <notifications@github.com> wrote:

> Yes. But next commit has everything that was missed. Sorry
>
> 2015-11-23 15:52 GMT+01:00 pierreganty <notifications@github.com>:
>
> > The message claims more than the actual commit. Did you forget to include
> > something?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/elenagutiv/Linearisation-2015/commit/3b98b3daacfc8b713dc76f6c7179d3134de58593#commitcomment-14562461
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/elenagutiv/Linearisation-2015/commit/3b98b3daacfc8b713dc76f6c7179d3134de58593#commitcomment-14563924>
> .
>


ok , i'll check that .
It is difficult to make it work because of the model view framework , nothing is centralized .
I am working on that at localTrackview :
//PRESETS NEED THAT TOO
    inline FxPanel* getFxPanel(){return fxPanel;}
    void mute(bool b);
    void solo(bool b);


hmm. i wonder why django knows about that foreign key. there may be another thing we can patch in the decorator to prevent this.


The only issue I see is that it would break GCE, as we don't specify the pool.

Also, in the disk_pools definition, maybe we can add a `<<: (( merge ))` to merge other potential definitions of the disk pools.

But as all of this will be replaced soon and GCE is not considered anymore, it is fine for me.


is it "file each node to the root user" or "file _of_ each node"? (I may not be the best person for that, but no harm on asking anyway)


Не сабмитится потому, что не выполняется условие. Ты можешь сам понять, почему не выполняется?


oh. i see. that's our code.


Thanks for this @czue. Its SO much better than having `couchformssettings.py`, `casesettings.py` lurking in my code folder!


The simple theory is that this piece of code is just not being compiled. Let's pretend it's commented out for now? :-)


It may seem to have no effect because the FormIDs are probably already sorted.  This is Valda's addition and those should be sorted.  Not to confuse things but just in case someone tells you not to sort FormIDs.  You can't always sort lists like those in the plugin with xEdit because a script might need the FormID to be the 5th FormID.  However, that is talking about the plugin no the Bash Patch.  Plus if needed there is the NoMerge Bash Tag.


This would work only without WebDriver, as it unconditionally sends a DevTools command directly. Appears to break with WebDriver, see https://github.com/WPO-Foundation/webpagetest/issues/529.

WebDriver has an executeScript() API, already called elsewhere in wd_server.js.


(i think this is surmountable, will put on my backlog)


The reason I have written it this way is to catch all violating issues in one error message. It will return a bulleted list of offending issues, an if/elseif flow will just catch the top one. So for example, it's helpful to distinguish the tensors being completely different (e.g. storage, offset, sizes different) vs identical but cloned (e.g. storage is different, but offset and sizes are same).


`exit_code` is already an attribute of `GalaxyTestResults` class, defined in the `__init__()` method. Now instantiating an object of this class fails with `AttributeError: can't set attribute`.


Ok, but under what circumstances was autoconf being called. I haven't seen it get called yet, ever. Do we need to change the prerequisites for this target to avoid the invalidation cycle?


Right... thanks


this is potentially a 'tm'.


True. I hasn't realize it was experimental.


cool, +1


but not critical at this point


About not ABout


Missing a comma here.


this babel plugin will be very helpful thinking towards new added files.
http://babeljs.io/docs/plugins/transform-strict-mode/


Também tive essa sensação. Mas o kepa grande soou estranho. Mas n tem jeito msm, é a tradução mais correta.


> It may seem to have no effect because the FormIDs are probably already sorted.

It does not sort cause it should be `self.formIDInList.sort()` ;)


Okay.


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


is that correct?


BTW, I change nothing in this line


Could just pass this to these.


Hello,
I've been traveling and have just finally uploaded a copy of my source code.
Please feel free to use it.
https://github.com/mac1usa1/AgilentU3606/tree/master


you know with @computed you can get these as parameters to the function right? Saves you two get commands!


too bad, the formatting was nicer before :)


Perfect. Thanks so much for your work and the explanation!!


Thanks so much for fixing this!


Is this implemented in 49? I'm usually hesitant to demonstrate support for something that isn't implemented anywhere just yet, but it isn't a lot of code. Mostly curious :)


Please use size_t instead of unsigned int.  This is a more appropriate type for array sizes, and unsigned int will not be large enough for large images.


Good spot @mpgomez

This is a typo - it should read:
> In order to do this, append the contents of the `/root/.ssh/id_rsa.pub` file on each node to the `/root/.ssh/authorized_keys` file on each node.


No did not even seen there was a @since in the test class I copied since it does not make much sense.


typo:  multidim.wavecrec -> multilevel.waverec


verifiy that every bam file has an index .bam.bai in the same folder
fixes #66


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


I didn't really CARE about this.  I was just saying it wasn't really a typo :)


@elieserdejesus  my updates are in congigurator , so you have to pull it to test my new work .
I'll upadate frankie when done .


diamond operator?


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


Thanks, fixed :)


Not sure if I can "listen" for `poll.options.[]` and still get `poll.options` as a parameter.


@stof indeed, but @Seldaek solution (`php -n -dextension=... composer`) is painful because you have to load all extension except xdebug. If not, your project requirements will cry on update.


Having looked at the merged pull requests and logic behind the nested JPA transactions I am 95% sure we **still** need the blocking code. I am pretty sure the need for blocking and nested JPA transactions are two independent issues. I will try to set up some demo code/project this or next week when I find some time.


@jason-rgl Are you able to correct some of the spacing changes that were made in this commit as well?  I believe your IDE must of have switched some of the indentations.  Please set it to indent using 4 spaces to keep consistent with the rest of the project.  Thanks again!


Okay then yes that should be changed then.  :wink:


Не нужно oninput, есть onchange, который подходит для всех случаев.


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


Done


Updated the path argument. My B. Thanks for the PR @ericcf


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


Hm, littering Nixpkgs with Darwin-specific sandbox configuration seems like a bad idea to me. For example, why do we need `lib.sandbox.allowNetwork`? We don't need that on Linux - we just grant network access to fixed-output derivations, which is what Nix should ensure on Darwin as well. It shouldn't be configured in Nixpkgs.


Sorry -there is some wrapping (topTypes etc) that should be in a dedicated noop commit - nothing there - it's moving the `RecordHeader` and changing the imports that matter :)


and this with disk_type so we can remove duplicate code


LGTM


@Shevchik I tested this new commit a bit.

No desync issues so far :+1:

(Also, players in survival mode can break blocks, however players in creative mode can't)


memory leak here?  to free (rp)?


True, if the DIM0 is desired to be copied, we should only copy what it is @simon816 .


@MiJyn are you lkjoel?


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


I have no clue - it was just an IDE warning that pointed me to this - I leave it up to you ;)


You forgot a `dlo(o)` at the end here. I think it's a memory leak otherwise.


Works for me. Might not work for everyone. It was just a pointer.


I couldn't figure out the reason behind the above change. Can you help explaining it?


ah, yep. sure does.


should line 62 below also get a trace: false?


Trailing whitespace detected.


Now another warnbing here : should  'rank' be a string as it is or a tuple ? In the second case just adding parentheses doe snothing - should become `('rank',)`  - note the comma


TeamCity pull requests :: netty [Build 723](https://garage.netty.io/teamcity/viewLog.html?buildId=3529&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5303, ignored: 21 Build time: 00:16:11


touch -c $(SRCDIR)/srccache/$(LIBUV_SRC_DIR)/aclocal.m4
and
touch -c $(SRCDIR)/srccache/$(LIBUV_SRC_DIR)/Makefile.in

are expected to cause it


I don’t know what this means.  Which lines?

Will Arthur
Intel Corporation
Server Security Firmware
803-216-2101

From: Gang(Jimmy) Wei [mailto:notifications@github.com]
Sent: Monday, November 23, 2015 10:38 AM
To: wcarthur1/TPM2.0-TSS <TPM2.0-TSS@noreply.github.com>
Cc: Arthur, Will C <will.c.arthur@intel.com>
Subject: Re: [TPM2.0-TSS] Fixed bug with _Prepare tests. Needed to reset SYS_CONTEXT->rval at the right (d71ab7d)

The indent in above two lines is still not good enough. More of the code is using 4 spaces as one level of indent, here is one tab instead.

—
Reply to this email directly or view it on GitHub<https://github.com/wcarthur1/TPM2.0-TSS/commit/d71ab7d558b81a25f9410a8e9069771d5450f5af#commitcomment-14564028>.


@wluyima claims it doesn't even work when you install the legacyui module when OpenMRS is running. It works only after you restart the server. I'll test that.


ok thanks :)


I think that the ordering passed from the command-line is lost here


Je sais que CsrfPreventionFilterHttpHeader.java fait un truc comme ça, mais je pense que ce serait mieux d'utiliser Helper.jsonErrorResponse, voir même créer une exception et faire un CreateMessageExceptionMapper.java


That's a good question!

I think that on these filesystems it doesn't matter, but if the close triggers an fsync we might want to finish the move first. But I think it makes sense to flip these around!


I think there should be only trainer ID.


I'm not sure why it's there either.

Maybe we just need to add one more case. `if resource.get(field_name) is None` muddles two cases together; it's true if (1) the key exists and the value is `None`, or if (2) the key doesn't exist at all. I imagine we want to skip the field for (1) and treat it as a null for (2).


should it be function_in_register_r3?


Dax -- this is David from Change My Path.  We just wanted to see how you're doing.  Are you back in New York or still in India?  Thanks, -David (Cofounder & Chairman)


lgtm


I think Dain's message here provides a clue as to what the principal is https://groups.google.com/forum/#!topic/presto-users/WXyHwmUUO6k.


And I never implied you did, but didn’t you asked for review?
I just share what solutions I found for errors I have hit too. ;-)


I've rolled back this one


ooooo versions


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


Зачем оборачивать в функцию, если не собираешься переиспользовать и функция все равно не изолирована (использует объявленный выше массив)?

Если тебе просто цикл кажется скучным, сделай через radios.filter, но а вообще не усложнял бы.


Because gettext has a library, intltool is just a build-time tool.


Also maybe again read through https://github.com/schaloner/deadbolt-2-java/issues/7 to understand whats the problem with blocking here.


@agabrys Thanks for PR! It's merged in https://github.com/SonarCommunity/sonar-javascript/commit/570f490a72c871296abd057b1d3191dd69e2442a


but you still call ``basic-block off``... it's a bit weird to act on a block that might not be ``basic-block``, no?


Hey Barron,

I flowed the instructions,

[jiaoyanh@borah GEOS]$ ls
geos2cmaq  testrun

here is the error message:

[jiaoyanh@borah testrun]$ make -f Makefile
rm -rf ./test
mkdir -p test; python -m geos2cmaq -t tracerinfo.dat -s smv2.log -m ./ -c cb05cl_ae6_aq ./test
/usr/bin/python: geos2cmaq is a package and cannot be directly executed
make: *** [src] Error 1

Any suggestions?

Thank you


You removed them in your other commit, and I just pointed out msys/mingw issues that could be really bad for the build itself. :-)


not used


I suggest you request this merge to the original repo :)


Yes, you are right, we can use `NULL` pointer.
It's just like casting `malloc()` result to pointer type, and I think that `NULL` pointer is defined as `((void *)0)`.
It will useful if we are using a compiler that disallows the implicit conversion from `void*`.


Why are you assigning a constant to another one ?


Faudra que je vérifie ça... Pourquoi j'avais mis /* ??
:-(


not very useful


I'd log here regardless of the autostart's value


It would be nice to put handwidth in here, and then we can have a 'total capacity' figure in the footer, which will lend itself towards a "network load" metric (used bandwidth / available bandwidth).

I'll try it out and maybe put together a PR at some point.


> It's working. Nice Work

It works sometimes, but LWJGL usually freezes the report window.


Préciser que c'est comma separated


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


if ``quot`` always takes a block, then we should add stack effects to it: ``quot: ( block -- )``


This probably should have removed the docs too, no?


This is supposed to be the module version, not Thelia version.


unnecessary change.


Ca me parait bien sinon :)


`else` is unnecessary here


MSYS2 is using a UNIX-like layout, and I *really* love the fact that I can move `geany.exe` from msys to my regular install, to test stuff. Also, using the direct `PKGBUILD` from msys to build the Geany installer is a really nice feature too. That more or less removes half of your script (to `PKGBUILD`).


This way, the Carousel module could no longer be installed on Thelia 2.1 and 2.2.

Is this really want you want to do ?


Good thinking here with regards to error checking!


ResortedSurfaces (amirroth) - x86_64-MacOS-10.9-clang: Tests Failed

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


Hi@Juan,

you were to fast ... i already updated it in "my repo" ... can you pls add it again? ... i you want then i can "maintain" it ...

Cheers
  Thorsten


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


Be careful.  In my code I use single quotes.  Im note 100% sure that this could make a difference when sending the url over to my end but it may be worth looking into.


Hey, could you please test your code on another website instead of changing everything one line at a time. This just helps all of your commits be more organized.


У элементов формы с `disabled` на мой взгляд проще работать напрямую через свойство, а не через `set-` или `getAttribute`. В HTML, чтобы элемент был `disabled`, можно добавить этот атрибут без значения. `setAttribute('disabled', '')` похоже на то же самое, даже если может по факту этим не является.

Предлагаю использовать `this.getSubmit().disabled = true|false;`


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


You may want to add your name and email to the list of contributors :)


Maybe we can just install the correct GTK+ configuration files in geany `etc/` directory to do your Windows theme magic?
And as I just said on the other commit, I really want to shared the same layout for installer and msys Geany, to switch them and test stuff more easily.


If the process hard-crashes, this implementation will not release the lock. That's why I was using a more complex but ultimately more resilient flock syscall


@alexgibson Done!


unnecessary change.


`this.getSubmit().disabled = !this.validUsername();`


merge in judge restructuring


please add details and a title to all your commits, even if you are changing very little. Thank you.


> I guess when you bump react versions you should release a minor instead of a patch bump (I guess it's unnecessary to remind that it's major.minor.patch for SemVer)

Actually given how Facebook has proven to be terrible with SemVer on React, I would recommend keeping this kind of “fix” for major releases as it can literally break a piece of software.

One of our platforms at work is using React 0.13, and this *minor* change actually broke it. I would argue it is not really your fault if React is doing shit with SemVer though.


there is no need r0 at the end.


Thanks, I can make it into a pull request for upstream if this is something that they want to include.


Good modulation.  This is way better than repeating this code in every method you have!!


fdf


:+1: good cleanup, and :+1: for collecting in a separate commit


don't use single character variables!!!


@zimbatm you are right if the process hard crashes (or is `kill -9`) then the lock will not be cleared, this is a very common and well understood problem with a number of applications (including `apt`).


I have excluded  the 1 and Number it self from the list, to decrease the simulation time.


```
 nixos-rebuild switch --upgrade -I nixpkgs=/home/freddy/Code/nixpkgs  --option extra-binary-caches https://hydra.nixos.org
```
with a checkout of master resulted in
```
error: attribute ‘oxygen-icons5’ missing, at /home/freddy/Data/Data/Code/nixpkgs/nixos/modules/services/x11/desktop-managers/kde5.nix:111:34
```


I like the new `ProcureState`. Reviewing this ports procurement func.. I'm wondering if we need a comment here along the lines of "the ports resources aren't actually removed from the offer". Compare to scalar resources, the port mapper/allocator doesn't actually remove the ports from the mesos.Offer so it's a bit .. inconsistent. And that inconsistency may be worth noting. Especially once we need to implement support for port allocation for the *executor*  (executor driver port, kubelet ports, proxy ports).

Thoughts?


You don't have to have this btw. All that's asked for is the badge, and even that is optional ;)


Jimmy, I’ve got about 20 of these emails that all say the same thing.

Will Arthur
Intel Corporation
Server Security Firmware
803-216-2101

From: Gang(Jimmy) Wei [mailto:notifications@github.com]
Sent: Monday, November 23, 2015 11:04 AM
To: wcarthur1/TPM2.0-TSS <TPM2.0-TSS@noreply.github.com>
Cc: Arthur, Will C <will.c.arthur@intel.com>
Subject: Re: [TPM2.0-TSS] Fixed RM error codes. (2efe291)

unnecessary change.

—
Reply to this email directly or view it on GitHub<https://github.com/wcarthur1/TPM2.0-TSS/commit/2efe29185172c7f5aae583fa3e150538dcc7a74b#commitcomment-14564910>.


unnecessary change.


There may need to be more lines added here if more categories are required in the future just as a heads up


This removal is causing regression #210


@zimbatm could you provide an example of it disrespecting ordering? The array is sorted prior to this method, and the returned array is the result of iterate+append so should respect the previous ordering.


Зачем такая реализация? Функция не реализовывает супер логики, не ограничивает видимость локальных переменных, вызывается один раз.


change the release to match next mx release


@lcalvet this can be just `localizationResults[iView].isValid()` no?


please use single assignment to whole condition instead of within condition assignments


the comment should be below this line


In the future, It may be worth creating a parser object that you can instantiate rather than copying all your code.  However, that is just as a suggestion in order to refactor your code.

As far as functionality goes everything looks sound!


In 2.3.0, the version of the core modules will be the Thelia's version.
I will change it, it will be done in an other PR.


Shouldn't the scaling be shown too?


Wohoooo! Thanks for using Fresco ;)
Amazing and very meaningful app! I'd love to hear your feedback for using Fresco.


@lcalvet same here, `localizationResults[iView]` instead of `_vLocalizationResults[iLocalizer][iView]`


Sorry, I was not aware of this point.


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


So does this line only do POS tagging? Does that string control which analysis is being performed by the library?


Should be `InvalidArgumentException`


That's a valid comment and a very good observation!

I'd be even inclined to implement "removing procured ports from the offer" (in a follow-up PR?) instead of leaving this inconsistency.


Missing sanity check to avoid invalid pointer crash on dereference.


Removing this isn't necessary to use the in-tree devtree.  Would like to keep it around for future use.


по поводу этого "фикса", отдельным таском посмотрю где происходит конфликт джоинов и почему к руме переодически не джоинится.


ok


Fixed that :+1:


class MelCoed should be handling that however this is one of many records like this.  I have mentioned these types of cases to @lojack5 in the past.  The reason there is an exception is that xEdit code uses what we all a Union decider.  So based on the value of another field that can be an integer or a string, or a FormID.

So I put that in the comments under class MelCoed.  When Wrye Bash uses MelCoed, MelCoed needs to know if it is an NPC_ record or a FACT (Faction) record.  When it's an NPC_ it's a FormID.  Then my comments are not very clear.  When yt's anything else except an NPC_ then it's an Integer.

I just realized that I can rewrite that myself.  I can have two separate routines.  One for everything except NPC_ records and another for just NPC_ records.  I'll make the change myself I'll update the comments so that it just says don't use this for MPC_ records.


I went ahead and bumped the kernel version here: https://github.com/openbmc/openbmc/pull/89 so you can probably close this one.


don't use open method when you already have html page via page.source


follow-up PR sounds fine to me.

On Mon, Nov 23, 2015 at 11:10 AM, Sergiusz Urbaniak <
notifications@github.com> wrote:

> That's a valid comment and a very good observation!
>
> I'd be even inclined to implement "removing procured ports from the offer"
> (in a follow-up PR?) instead of leaving this inconsistency.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mesosphere/kubernetes/commit/f37842c405687e939299b6780c2f654ff335cfe5#commitcomment-14565159>
> .
>


Should we use both `.r-list` and `.List` class?


this is my first repository


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

Если используешь `classList`, то у него есть метод `toggle`. Лаконичнее одной строкой так: `currentControl.classList.toggle('hidden', action);`


already updated in the PR, this is an outdated commit


I don't understand this code.

Gen 2 egg moves are `2E` and Gen 2 event moves are `2S`, so this should already lead to an empty source intersection. Why does this need to be special-cased?


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


Use `$base-fw` but I can't figure out why you need to reset it


@tcustodio, I am sorting out a merge of master into dev to rebaseline a new master base on the SDK 1.4.  I am reviewing the modules that were merged into master without being reviewed and promoted through dev.  And when I look at this, I have quite a few comments:

*   `mpr121-example.lua` - `while` loop.  Doing this in an example is a no-no, because this type of blocking code _will_ cause the wifi stack to fail.  In general if you are needing to do `tmr.wdclr()` then you are approaching the problem wrong.  The example should use a repeating `tmr.alarm` to implement this.

*    `M.setRegister()` needn't take the argument `address` as this is only the address specified in the `M.init()`.  It should return a status depending on whether the set works or not.  i2c.write() accepts multiple byte arguments and returns the number of bytes written  so the end of this would be better:

```Lua
     local bytes_written =i2c.write(id, r, v)
     i2c.stop(id)
     return (bytes_written==2)
```
*    `M.init()` should not assume that writes will succeed.  If should test the write status _at least_ once to ensure that the device exists.  In this case it's also worthwhile using a local value for the `setRegister()` call to avoid the repeated table lookups:

```Lua
function M.init(address)
   devAddr = address
   local setRegister = M.setRegister
   if not setRegister(0x5E, 0x00) --[[ ELE_CFG ]] then
      return false
   end

   -- Section A - Controls filtering when data is > baseline.
   setRegister(0x2B, 0x01) -- MHD_R
   -- . . .
```

*    `M.readTouchInputs()`.  Given that this is an example, I do feel that we should be teaching how to write efficient Lua so I would code this as:

```Lua
function M.readTouchInputs()
   i2c.start(id)
   i2c.address(id, devAddr, i2c.RECEIVER)
   local c = i2c.read(id, 2)
   i2c.stop(id)
   local touched = c:byte(1)*256 + c:byte(2)
   -- . . .
```

I will defer raising the pull until have some responses.


You need to update your Python. I recommend installing anaconda

Barron Henderson
Asst. Professor, UF ESSIE EES
420 Black Hall; 352-846-2695

> On Nov 23, 2015, at 10:58 AM, JiaoyanHuang <notifications@github.com> wrote:
>
> Hey Barron,
>
> I flowed the instructions,
>
> [jiaoyanh@borah GEOS]$ ls
> geos2cmaq testrun
>
> here is the error message:
>
> [jiaoyanh@borah testrun]$ make -f Makefile
> rm -rf ./test
> mkdir -p test; python -m geos2cmaq -t tracerinfo.dat -s smv2.log -m ./ -c cb05cl_ae6_aq ./test
> /usr/bin/python: geos2cmaq is a package and cannot be directly executed
> make: *** [src] Error 1
>
> Any suggestions?
>
> Thank you
>
> —
> Reply to this email directly or view it on GitHub.
>


拜托不要提交奇怪的折行和奇怪的CRLF上来啊........


Нет большого смысла в том, чтобы определять переменную, которая далее используется единожды.
Если разумная длина строки позволяет, то тожно объединять вызовы например так: `error.appendChild(document.createTextNode(errorMessage));`.


i meant that line 36 and 38 are the same...


Why do we need it?


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


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


This is not an unsigned long, please use PRId64


If the argument isn't provided then the token will also be consumed.  Maybe something more along the lines of `{boolean} [consume] if true, the expected token will be consumed.  The default behavior is to consume the token.`  Putting square brackets around the parameter name indicates that it's optional.


I have tested this and I am relying on this particular fork of the plugin to use with Jekyll 3.0. Are you planning on issuing a pull-request?


TeamCity pull requests :: netty [Build 724](https://garage.netty.io/teamcity/viewLog.html?buildId=3530&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5303, ignored: 21 Build time: 00:25:49


I guess those more specialized categories will be rarely needed by plugins, in opposite to the common ones (`isKeyword` / `isReservedWord`) which are part of API we might break more than once.


I'm not sure I know how to use noop yet for interactive rebasing.  For example I see this quite often

pick <SHA> Title Of Commit
pick <SHA> Title Of Commit
pick <SHA> Title Of Commit

So I don't know where to put noop, why and all that.  So I should learn that at a later date more then likely.  Your help getting the interactive rebasing going is going to help me anyway.  So I got it it's just still a bit abstract.


Comment meant for block above. Here we print a size_t, priS


Well, Chrome denies setting Content-length so the whole thing collapses into nothingness. At least this way the JS console is clean. ACK, will keep this on devel for 15.7.20.


I'm biased about `model_objects`, but at least there will be distinction between models and objects.


Don't use conditions in views, realize that logic via helper.


You can use file.**contents** && file.contents.length instead of path


To double-check:  Can we expect `jps` to be in the `PATH` of the user running the script?


不是哦，您说的人我不认识。
On Nov 23, 2015 4:01 AM, "Jeff Chung" <notifications@github.com> wrote:

> 杜宇类?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/giantray/stackoverflow-java-top-qa/commit/44760d468fa50254299d227dfb4ea1e13e92e354#commitcomment-14557049>
> .
>


What is the reason of writing row above?


а сколько будет слов в такой строке:
```
var z = '  a  b  c  ';
```


"Objetivo".


+1.

At some point it might make sense to make the external urls a non failing check to avoid this.


"poderá" com acento.


попробуйте воспользоваться функцией filter  у массива


@aboe76 that would make the whole grain system very confusing on SUSE. I think this should be fixed.

@aboe76 family is a "collection". It is clear that SLE,SLED and openSUSE are a family that share similar traits. Changing the meaning of such basic concepts because they were already wrong would make the whole grain system confusing.

The example code search you cite seems mostly correct, os_family is checked against "suse". salt/grains/core.py seems correct as well:

```python
    'SLED': 'Suse',
    'openSUSE': 'Suse',
    'SUSE': 'Suse',
```

`salt/modules/rh_service.py` seems wrong:

```python
if __grains__['os'] == 'SUSE':
            if str(__grains__['osrelease']).startswith('11'):
                return __virtualname__
            else:
                return False
        try:
            osrelease = float(__grains__.get('osrelease', 0))
        except ValueError:
            return False
        if __grains__['os'] == 'Fedora':
            if osrelease > 15:
                return False
        if __grains__['os'] in ('RedHat', 'CentOS', 'ScientificLinux', 'OEL'):
```

You can see the inconsistency here: if `os` can be 'Fedora', 'RedHat', 'CentOS', 'ScientificLinux', 'OEL' for `os_family` 'RedHat', then it has to be 'SLES', 'openSUSE', etc for `os_family` 'Suse'.


Quick a lot of unnecessary changes, suggest to remove them before commit.


Don't use single character definitions


set_message_done takes partner_ids that you do not seem to handle anymore.


`.is-active` is not related to any CSS or JS


It's been six months so I don't remember right now, but there was some problem with egg and event moves.


:hocho:


Yes, having the same code repeated is not a desirable scenario, but I haven't found the best approach to deal with this. The two places here are the SPI interface and the parser module. I don't think that it's a good idea to have a superclass-subclass hierarchy between SPI interface module and the parser module. The SPI interface is meant for connectors to override. So if you look at the class hierarchies, a class in Parser module extending a class in SPI interface doesn't make much sense. I looked in Presto code to see whether modules other than the connectors have dependencies on SPI module. I see imports of SPI interface in ```presto-main```module, but didn't see non-connector modules extending classes from SPI interface.

I think a better solution might be to make a superclass and add the sub classes in SPI interface and parser modules.  What do you think?


Are you using the case class here instead of the prism for performance reasons? I’m a bit nervous about it because I imagine contributors looking here to see how to use `EnvironmentError`, and coming away with the wrong idea. Also, consistency – we won’t refer to the case classes in other places where we need to “pattern match”, so why do it here?


Se traduziu "enhancement" como "aperfeiçoamento", tenta manter a consistência aqui também.


Thanks Matt! I have made this change.


You don't need to make this public. In fact, you only have to call `doWithSession(sessionId) {}` and catch `NoSuchResourceException` (though it could make sense to add another method like `touchSession`, so it could be more explicit). But exposing the `sessionContext` (now only an implementation detail of `SmartRSessionSpringScope` and `SessionService`) is not the way to go.


If you throw here, the task will never be deemed finished!


108453024-FixLeedNumberOfHoursNotMet (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1957 of 1969 tests passed)

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


Could you try with a button, please?


Plus, there's no reason not to touch the session also when the task fails.


end line...


Sure although maybe we are just using different terminology too. I think of "Model classes" from which we make "Model objects" and when I say/hear just "Model" my default interpretation is "Model objects"


It's a bit weird; 'pos' in this case allows the tokenize, ssplit, pos, and lemma annotators according to their Python code.  We can make a configuration file even with this Python wrapper that allows us to specific what seem to be annotators, so we might just want to end up using that and specifying all of them.


@elieserdejesus can you test please ?


this was set in wrong function. found comparing 3.10.y code.


Could you please check if unit tests pass?
Tx


Eu tenho a impressão que esse "Crave" era pra ser "Carve"...


ion-content.content(scroll="false") mean???


This one seems particularly bad – we should be treating `Case` as though it were private. The only reason it’s not is to allow Shapeless deriving.


This should be `@param`.


updated in last commit thanks


Good luck on this endeavor! :+1:


Is there some way to get a filename in nw?


108453024-FixLeedNumberOfHoursNotMet (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Put another way, you need a good reason to make an object as complex as `SessionContext` an internal API. We should try to avoid a scenario where every object communicates with every other.


`= set(recast.values())`


What's gonna happen when we also have a `TimetableEntry` linked to the `Contribution` and the `SessionBlock`?


Another omission that is causing #210


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


- why are `flush` and `close` no longer abstract methods?


@tcustodio, I am sorting out a merge of master into dev to rebaseline a new master base on the SDK 1.4.  I am reviewing the modules that were merged into master without being reviewed and promoted through dev. (see #745) And when I look at this, I have a couple of comments:


Ah. Sure enough. Shouldn't break anything, so I won't bother with another
release right now, but I'll make the change.

On Mon, Nov 23, 2015 at 8:15 AM, Vijay Raghavan Aravamudhan <
notifications@github.com> wrote:

> i meant that line 36 and 38 are the same...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/danielsomerfield/gradle-cve-dependency-check/commit/6c2b82481086a2a72addfcd8fca4050ec2f21b9d#commitcomment-14565297>
> .
>


Len;
Thanks again for this. You've got it exactly right, I'm trying to control resource usage on shared systems. I did adjust this up after running into some problems with the 2Gb limit. Do you have ballpark numbers of expected memory usage per thread for validation? I can set threads and memory to something reasonable based on this.

Does RTG have built in logic to set the ROC scores based on inputs? If I fed it the name of a caller could it automagically grab the right thing? I'd definitely lean on your experience here as I haven't made a ton of use of ROC plots in the past but am keen to do the right thing here. Thanks again.


I want to make some contribution with this kind of HTML5 game.


他好像要用__bootstrap-combined.min.css__不然有些東西會出不來


How does this work? I would guess that the `ioclass` needs to know the gp it is writing.


Does this fail when a coupon isn't provided?


Ahh asking about something similar here https://github.com/cfpb/cfgov-refresh/pull/1162#discussion_r45617599

So what's happening here? Will the macro be removed and this first part of the if statement block will remain? If not can this still utilize the macro? Something like:

```
{% if value %}
    render( value )
{% else %}
    {% macro render( settings, options={} ) %}
    …
```


develop (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1432 of 1432 tests passed)

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


OK


__window.projectsArray__這個變數在angularjs.org首頁有，我把他複製過來了


we'll have a `timetable_entry` backref here. unfortunately we can't enforce consistency :/


Does `if: :coupon?` work here?


@zevarito Es este e método que se encarga de sincronizar cada video con la linea de tiempo.
Desde la directiva offlineStreamCarousel se le notifica a cada video que debiese estar reproduciéndose en ese momento, que debe estar corriendo en un currentTime que se calcula en base al currentTime del video principal. Cuando los videos reciben esta señal modifican su currentTime si es que están lo suficientemente retrasados/adelantados (diffTolerance).


@AnriShitko


This broke stuff. See #301.


Sorry, my .gitignore was "gitignoring" the file.. now you should see it in plot-scripts/


Thanks! I'll be changing this to use the [SQLAlchemy declarative API](http://docs.sqlalchemy.org/en/rel_0_8/orm/extensions/declarative.html) at first because it's more similar to the Django extension and then possibly adding support for straight SQLAlchemy Tables.
Off the top of your head, do you have any tips?


danke :+1:


Well spotted, thanks!


Will, I am reviewing via the web interface, so can you just open below urls to see my review comments in browser?

https://github.com/wcarthur1/TPM2.0-TSS/commit/d71ab7d558b81a25f9410a8e9069771d5450f5af
https://github.com/wcarthur1/TPM2.0-TSS/commit/2efe29185172c7f5aae583fa3e150538dcc7a74b

Jimmy

-----Original Message-----
From: wcarthur1 [mailto:notifications@github.com]
Sent: Monday, November 23, 2015 10:07 AM
To: wcarthur1/TPM2.0-TSS
Cc: Wei, Gang
Subject: Re: [TPM2.0-TSS] Fixed RM error codes. (2efe291)

Jimmy, I’ve got about 20 of these emails that all say the same thing.

Will Arthur
Intel Corporation
Server Security Firmware
803-216-2101

From: Gang(Jimmy) Wei [mailto:notifications@github.com]
Sent: Monday, November 23, 2015 11:04 AM
To: wcarthur1/TPM2.0-TSS <TPM2.0-TSS@noreply.github.com>
Cc: Arthur, Will C <will.c.arthur@intel.com>
Subject: Re: [TPM2.0-TSS] Fixed RM error codes. (2efe291)

unnecessary change.

—
Reply to this email directly or view it on GitHub<https://github.com/wcarthur1/TPM2.0-TSS/commit/2efe29185172c7f5aae583fa3e150538dcc7a74b#commitcomment-14564910>.

—
Reply to this email directly or view it on GitHub <https://github.com/wcarthur1/TPM2.0-TSS/commit/2efe29185172c7f5aae583fa3e150538dcc7a74b#commitcomment-14565036> . <https://github.com/notifications/beacon/ABguKB4cgHV8M9V2_Dzm1eWoDhY3ZTUvks5pIzEqgaJpZM4Gnn_l.gif>


Cool, this will go in as part of a clean up effort I'm embarking on, cleaning up various suggestions by things like Rubocop and the like.


@danilaml Must have read it wrong. But it currently [is](https://github.com/tambry/rpcs3/blob/db4c0592d5db22201d8fe1a1c3282a9cbd106d94/.travis.yml#L91) in a separate branch...


Wouldn't `memset()` be a cleaner approach? It would also initialize all those `str` fields.


Code is wrong.


First I want to make clear I'm not trying to convince you guys we should integrate the ninbot gold feature specifically. That was only an example but it's not that important. It will go it's own way and I think it has much potential.

 I've worked in DAW environment since the first cubase VST back in '96. In general DAWs are just the emulation of analog equipment in every form, that's why most producers who still have the chance to work with real consoles and tape prefer them.

And the multitrack environment of Recording studios is the polished version of Les Paul's recording method of the 50's, nothing more.

https://www.youtube.com/watch?v=cFKLwJzMoIs

So nothing new here..... the 80's brought Djs and hiphop and now there come the internet, live looping, ninjam and here we go again .......all this is just the expansion if this same idea.....in my opinion....

No problem with that and this is where jamtaba stands I think.....Some fresh air to this old wishes.....

Let's keep the dream alive......


Не коммить xml-ки - они никому в репо не нужны!
И второе - ты с кем в команде? Кто у вас главный?


I understand that O_EXCL is more cross-platform and it's why you went with that solution (even if in some cases they system just pretends the lock is atomic). Since the tool is mainly for POSIX systems I would rather use the syscall and not have to cleanup in case of failure, but that's more like a soft preference.


Added few comments that may be worth considering.


This seems to be messing up some of the tests. JIRA1961 for example relies on being able to do `k.application.is_prepared != None`. Does it need to be hidden?


Merged.  @lsm5 could you build a fedora version that @jeremyeder can play with.


This method invokes the appropriate connector's method. So if hive connector is being used, it will call the ```grantTablePrivilege()``` method in ```HiveMetadata.java```


Если в описании коммита приходится писать and, в 90% случаев стоит сделать два коммита


So we definitively need to define the terminology here (and other areas), because my understanding is exactly the opposite, i.e. `model` is a class (and we don't use `class` or `Class` simply for technical reasons).


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


@brickZA  Ready to merge.
Please review and merge


Autant, y'en a qui sont doués qui sont doués pour le français, autant du côté de l'anglais c'est dégeulasse. "Qwery" ça n'existe pas, la fonction est définie dans ce fichier https://github.com/Florent8/Projet-S1/blob/master/functions.php


What are the warns in __init__.py?

My naming structure is this all the records and the attributes that Wrye Bash should be looking at are graphicsTypes for example.  The set in the patcher that the patcher uses to for converting LogFids always has the suffix 'LongsTypes ' for example graphicsLongsTypes.


I realize that I haven't actually told autoprefixer to use the options, and there might also be a typo in line 34 ('last 2 version' - should probably be 'last 2 versions')
I will check the fix, however. :)


I thought a simple implementation of `flush` and `close` can also be a method doing nothing. Do you think it is better to force users to implement it?


@lcalvet since we return inside the if, we can remove the else branch


nazwy zmienny w C# piszemy z dużych liter


object inheritance is not required in 3.5


I've finally had time to code review this.

Are you sure you understand the validator algorithm? We intersect the set of possible sources for each move, and if the intersection is empty, the moveset is illegal, otherwise the moveset is legal.

For instance, a source of `2Edragonite` means that the move is legal from an egg where the father is Dragonite.

You seem to be keeping track of egg parents separately, which leads to bugs. For instance, your `hasEggMove` check ignores that egg moves might not be the only way to learn a move. Many egg moves are also TM or tutor moves, for instance, so you can get false positives for incompatibilities.


I tried to test it, but unfortunately there is no Polish translation in /usr/share/qt5/translations for qt_base
I only see that it can load qt_pl.qm file.
What I can do is to send you files from /usr/share/qt5/translations folder


@kolodny, I've forked the repo, maked a commit and then removed it. When I get some time I will make a PR. ;)


The `super()` call without arguments doesn't work on Python 2, which we are still trying to support. And I think I do prefer a try-except rather than version testing.

We'll be upgrading our own code to Django 1.8 soon, so we'll probably have to patch this up ourselves in a while. I'll probably get around to doing the patch the way I want it then. If you're in a hurry, then please update the PR yourself so I can one-click merge it.


if you come up with a better fix, feel free to do so. I have never worked with these tools before :)


it works for generic io, because it assumes a field `gpi` inside of `data`. let me fix it and make it more general by passing also `gp`


Updated in new commits.


"West Siauliai Woods" é o nome de uma localidade, e você pode traduzir de acordo com a convenção da wiki em https://github.com/Treeofsavior/PortugueseTranslation/wiki/Tradu%C3%A7%C3%A3o-de-Lugares


I'd advise against using again `*_data` for things that return a list of many.


what would you call it?


It can't hurt to have that fix, though - I haven't come across it but that is probably because I am using an older nodejs?


@fazibear hey, thanks for this pr :+1: , out of curiosity why you didn't define the if statement inside one initialize ?


### :+1:


If you're going to be condescending as usual, I'll pass on any communication. Go ahead, do it yourself.
PS: I'm fairly sure this is outdated as the code has been changed, at least minimally.


... in contribution fields


The dream is still alive , the dream keeps us alive !
I'll work on that specifically in this branch Jon .
But as elieser said , we have to deal with some priorities .
I am able to work on many branches at same time , it is the way a tree grows ...
But first the grow is underground , the roots ...
https://www.youtube.com/watch?v=qm7Xt2Qsjcg


Yeah it will never change on this page as that's in git history now but the actual file is updated


I define class MelModel in records.py for Skyrim.  So I can see why you would not want/need to import that from brec.  That makes sense.  So you are saying that MelModel is unused from brec because i define a completely different version in records.py?


Super Kod, świetnie zrobione zadanie, brawo!


nazwa zmieniej refs nieczytelna.


The problem with me finding a better fix is that I haven't got anything that needs fixing :)
So if this makes you able to build, then I think that it's appropriate.


nie nazwy zmiennych, tylko nazwy własności. Nazwy zmiennych małymi.


"Monstros que estão confusos"


I have forced locale setting directly in program by adding 	locale = "de";
Now I see correct German texts:
Optionen:
...
Argumente:
....

Great job! It works!


Hostname is created but it is the same in every deployment.

can we change it to use a different name per deployment? for instance:
```
"${var.env}-bastion"
```
Actually that makes sense and it is consistent with the DNS name below.


Should this be based on a feature setting instead?


It would be great if this could patch would get cherry-picked into at least 4-2-stable branch?


LGTM


słaba nazwa klasy


okay, will merge it then.


actually it seems that the best is define both hostname and "fqdn":

```
#cloud-config
hostname: mynode
fqdn: mynode.example.com
manage_etc_hosts: true
```


czy nie lepiej rozdzielić śmianie się od wakacji? Czemu jest to w jednej klasie?


The autoprefixer line needs to be:
~~~
.pipe(autoprefixer(autoprefixerOptions))
~~~
obviously :p
If you change that, it will do what we intended it to do.
And you can merge this in :)


Walker mógł być dalej dziedziczony ale TexasRanger nie musiał nadpisywać jego metod. Można je było albo usunąć  z Texas Renger albo zmienić nazwę metody Walk


I could completely do away with this commit if I knew how to use game.masterFiles[1].lower properly.  How can I say read the variable masterFiles and since it's set __and__ the main ESM file is always the first one, then get the first one and then convert it to all lower case for comparison.


reszta spoko


brak zmian


FWIW this is what has been in the documentation for some time: http://bokeh.pydata.org/en/latest/docs/user_guide/concepts.html


Do we know why we have `--no-check-certificate` here? Seems not the best


Hi!

Just wondering why does the drush make command need to be run inside the **profiles/wetkit** directory? I am guessing this is because you do not specific the build path at the end of the command. That is why it asks:
**
Make new site in the current directory? (y/n):
**

  The following run in the Drupal webroot will install the same.
``drush make --no-core profiles/wetkit/modules/custom/wetkit_deployment/wetkit_deployment.make profiles/wetkit/modules/custom/wetkit_deployment/``

God Bless!

Frederick


Just to clarify...I'm not pushing the DAWtaba branch in this post, just making some history of where it all came from. Live jaming is for me part of the same thing...


OK


@lcalvet unused


bardzo ładnie


@dgonz7 I'll be removing this section from the gradle file that makes the cdvCompileSdkVersion = 23, as you've suggested.


Can help you, if you would provide us with what your program should actually do and an example input/output. Don't just say "Logic gate simulator" (its too general)


hmm... numbers seems quite strange.  Especially for improving...


actually not during deserialization


How `ofApp.h` knows about `BrushBase` class? It's not included in header.


Sure, I'll look for the other places too. This one was the first that came to my mind.


Good move!


it not add content ex. toggle


Dobrze zrobione!


is this a place where ``untag<byte_array>(relocation)`` is cleaner looking?


Looking at the header file this file was created around the time that string.split entered python (python 1.6), so it might have been the canonical way to do this back in the day...


That was my initial thought, but the route is included/excluded based on `AccountManager.using_statements?`. This object only ever really gets used if statements are on, so the the presenter logic doesn't even really apply if statements are off. It just didn't seem worth the refactoring at this point.


This may fail if using `gzip.open` in Python 2.6, which doesn't support context managers.


Ezee, solo and mute in the view are automatically changed if you change the model. But now I don't remember if this is working only in the midiMap branch, but I remember I code this updates from model to view.


Got backported to 8.2 in https://github.com/owncloud/core/issues/20193 so thats fine now


Good idea, `rubocop` is great and it takes no effort to get started. I couldn't help it and run it against this repo! :wink


This needs to be a relative path


let's add parameter which will be used as default result


Mmm I'm sure it would be handled at the moment as we see POT-Creation-Date header changes.  Here is likely the right place for now since a PO header is actually just a unit. But just wanted to flag it as I'm concerned we won't be able to sync well if we're not picking up those changes


yeah , but that branch is dedicated to that kind of stuff , so ... ?


i not forget but jade compile not end line


Hey Bob,

Thanks! I'll get it imported into Eager right away!

Zack


and maybe with default value = 0


Is this extension available on RDS?


I added these here instead of checking the extension during `selectTiles`. I also renamed `tilesLoading` to `subtreeLoading`.


maybe it should be `\code{Tag} (see \link{Tags})`?


same story here


@tambry ah, ok. Didn't notice. Have you looked at coverity logs to see why no files are generated?


At first, I thought grantPrivilegeOnTable is a bit longer name where as grantTablePrivilege is compact and is in line with other names being used. But yeah, after reading your explanation, I am leaning towards the former; it is more clarifying .


Review complete.


Organized this function some more, and I make the assumption that if it passes in a parent tile, then that parent is the head of the tree.


Works great as a fix - you should send Auth0 a pull request.


Python 2.6 does support context managers, just not more than one on the same line.


@danilaml The files are normally generated, but sometimes there aren't enough "compiled units". The thing is very useful, but it's quite unreliable. Anyways, not very sure why it fails. Might need to add an echo for the Coverity logs.


@misterdjules What does this actually do? (And how? The domain swap is lost because `nextTick` is fired later..)

The test passes with or without this...


Ah ? must be in the midi branch so .
I hesitate to merge midi with configurator , it's better to have separated stuff to merge to master .
If you could work on midi branch and finish the work , i'll update configurator then , and possibly
remove funks if needed .
thx


Thank you @stebe.
I've had the same issue.


Thanks - I don't quite get all that cause I am not familiar with records (yet). What I point out is that the `MelOptStruct('COED','=IIf', (FID,'owner'), (FID,'glob'), 'rank')` probably expects a tuple as the 5th argument (and not just a sting, namely `rank`) ?

But actually looking at MelBase there is:

`if not isinstance(element,tuple): element = (element,)` so no worries - it will turn 'rank' to ('rank',) fine - so this is probably a false alarm (the rest about the records is beyond me) - except if I miss something


oh ok, thanks!


That is exactly what this implementation does.  It add 'P' as a width modifier.


Good - this was how I intended the parser changes to be used; hope it felt reasonably natural.


Is this line a bug/typo? I don't understand it.


Because the support won't change during runtime, so you avoid a check at every initialization.


Ah, I see.  I misread it.


@mangin  Три? Она же работает? Но могу переделать через filter


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


I removed the loading finished code since this should all be handled in `selectTiles`


It definitely isn't commented, which is kind of a big deal considering how dense the validator is...


i love you @Romney81


Python 2.6 does, the `gzip.open` function does not.


The tricky thing here is we can't safely figure out how to strip the "On x, y wrote:" part which really uglies up the emails. I honestly think the quote at the top of an email is largely useless. People will be able to figure out what they are replying to.


do you really mean `xSS` and `xES` here? they are computed by `HKDF-extract(0, SS)` (and `HKDF-extract(0, ES)`) -- thus if `ES` and `SS` are the same, `xSS` and `xES` are the same...


This is one of those ambiguous language cases that my high school English teacher warned me about.


Added the subtree check here as well.


Maybe add some temporary MPI protection:
```C++
if (mpi_size > 1)
   ....
```


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


In case anyone wonders, what changed here, nothing except line endings: https://github.com/yiisoft/yii/commit/53d9f95ff236fcad1580ab75db83d60fc2fa4a6c?w=0


Oops sorry - I meant that the wrapping should be in a dedicated commit that basically does nothing important (hence noop). there is no noop in rebase (except if you ask git to do something that will not change anything so it says 'noop')


@gabizou What's with all these formatting changes?


No, we definitely do not want to allow refinements that are not mentioned in parent. The type lambda in Set is a workaround for binary compatibility, we can easily replace that with a simple type constructor.


Oh right! But you could get `poll.voters` and `poll.isMultiple`


wew


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


`game.masterFiles[1].lower()` ;)


commas not required


Review is complete.


Actually if its the first `game.masterFiles[0].lower()`

masterFiles is list of strings right ?


Фигню запостил, дурашка


Sry. It is fixed with 0.2.15


Meant to say "....That now sends attachments..." whoops


De nada, @TomazAlexandre !

Um abraço!


108453024-FixLeedNumberOfHoursNotMet (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (895 of 895 tests passed)

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


Code was copied from decompiled source and when IntellIJ touched the file, it applied Sponge's formatting style.


Pycharm has a python console you can use to type things like that and experiment:

```
>>> 'RRR'.lower()
'rrr'
```


@Renkas Fixed: 2b449c7b7dcb60727406504f8c0a92e59dd1de79


Yep !


Looks like we can just use `it.second.get()` instead of update opts


The main objective of this program is just to simulate a series of logic gates, and therby get the required output based on the given input.

In reference to the current version of the gates.cpp  program, I have put the format as to how the digital version of the circuit would be, in the build function in the class gates_2.

The input will be given In the form of bool's(0,1) and the corresponding output will be given by the gate which is also  0 or 1 based on the gate.

I have solved the issue related to this commit, but is there another way to impliment this, like instead of overloading the function, are there any other methods which can be used to impliment this.

Also, this version of this code requires a programmer to understand and build the circuit. But can we improve the code in such a way that the used selects the gates during the runtime and then runs the code,  by this i mean, instead of creating the logic circuit in the program, is it possible to build the circuit during runtime, on the fly.


Yes, this check is done in ```checkCanGrantPrivilege()``` method.


You're welcome 😊


I'm going to remove this in my refactor, feel free to add it back in if you think it's not a typo (but please add comments so we know what exactly it's doing)


@VoyagerOne : I have problems with git fetch upstream.
I receive a message: fatal: 'upstream' does not appear to be a git repository.
I also need to delete my local Stockfish directory and clone again each time to work properly.
I have installed GitHub and Git Shell.

Do I need something else? Do you have some clue about my problems? Thank you in advance.


Oh, would allowing that cause problems? Disallowing it makes it a bit harder to port some code from Scala 2 and forces the use of boilerplate code.


It's the warns in the commit message - better use a set literal than a set call - that is

```
s = set(1,2)
ss = {1,2} # better

s==ss # True
```


The source of these changes here: http://www.smogon.com/forums/threads/gen-2-ps-development-post-bugs-here.3524926/page-3#post-6165333


Nothing to do with your notation


Yes, all top-level assignments need to be saved for things to work properly.

VerifyTools[AddVerification] can be added to an export called ModuleLoad (see ?ModuleLoad) procedure, which gets called whenever that module is read in from disk [there's ModuleUnload as well, for symmetry].


Good point. I think for now, caching table privileges and user roles should suffice. But we should certainly keep in mind to add more if needed.


I started with a CSV but Gumeric kept interpreting some version numbers as dates. But yeah, a README.md in the build subdir would work better.
I'm trying to minimize effort needed to upgrade libraries ideally to 1) download new source, 2) delete old folder, 3) extract to new folder with old name, 4) run build_thing.bat, done. How many (like what percentage) of our dependencies would work this way (i.e. don't require any hacks)?


but the best strategy is to merge midMap to master first , then  the branchs are updated from master .


you mean the code above?


Colorschemes shouldn't modify terminal settings.


Working Ezee. I notice two things:

1 - The boost buttons are not reseted.
2 - I switch to full screen (very good in my laptop small screen) and quit using ESC. Started Jamtaba again and the settings works, Jamtaba is reopened in full screen. But I notice the bottom of my Jamtaba window is cropped (see the image in bottom). When I press F11 the crop effect disappear.
![image](https://cloud.githubusercontent.com/assets/1012741/11343711/64a7721e-91f4-11e5-86c0-fc9194f56b03.png)


Yes, the message is JSON and is parsed as such.


You may want to check if on OSX the build directory contains something other then "lib.linux-${CARCH}-${PYTHONVERSION}" (as it contains the word linux).
If so, I could help making this auto-detected if you tell me the directory name on OSX


Neither `composer self-update` nor `composer install` are required. `dev-require` dependencies are installed by default.


In that case, if the message is not a string, it will throw...


Sorry for my many spelling errors and formatting problems, :+1:
But doesn't it look a-bit overwhelming now,
I kinda added the numbering because it would of looked better,
Now the part you made looks really neat but what has been added to you'r part looks like a wall of text,
Stuff like that turns off the people that want to do simple stuff and use simple stuff,
https://www.joduska.me/forum/index.php?app=infotickets&page=article&id=16#/article


Should it be
byte_3 = byte_3 + 2 ^ (8 - i)
?


This would be checking a different path (full path to `brew` shell wrapper *after* resolving any directory symlinks), which is why I didn't change it. But right now I can't think of any scenario where this could give a different result, so I think it's safe to replace `$0` with `$HOMEBREW_BREW_FILE` – and thus make the code more readable as you suggest.


NotificationARNs must be a list of strings or not set


@Zuchens - ten commit psuje dwa testy:
* SensitivityAnalysisMorrisStartingTest#test_proper_response_on_error_while_starting_sensitivity_analysis_morris_script_with_cleanup
* SensitivityAnalysisMorrisStartingTest#test_successful_start_of_sensitivity_analysis_morris_supervisor_script

Sam czasami zapominam uruchamiać testy przed pushem, ale pamiętajcie o tym!


added user in takers section

![Uploading Screenshot from 2015-11-23 22:39:34.png…]()


The `if` is neither necessary nor does it ever evaluate to true. The `deploy` section is executed with PHP 5.3 only (see `deploy on php: 5.3` below). There are some `phpdoc` options missing, but that shouldn't be a problem.


@kkrieger85 lol. So we where both half wrong.


LOL


Чётко


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


why ;-;


Passing the `phpdoc` target directory (instead of changing to the directory) and renaming the script to something like `deploy-phpdoc.sh` is imho reasonable.


Saw this 21 days ago then totally forgot to reply. :|
One could try to provide ad-hoc methods for things like double-clicks and similar on top of the existing interface. I fail to imagine a nice API for managing input events, polling is simply too much comfortable for such use case. Do you already have any concrete ideas?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 116](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=512&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:05


Hmmm, as it is, it seems a counter-productive added check... Could be a typo indeed.


you mean

```c++
if (mpi_size > 1)
  // throw something
```

? that's a good idea, and I'll amend the patch by this tomorrow, but I'd like to have my suspicion that updating the transmissibilities during the simulation breaks the parallelization confirmed or refuted. (that would happen if the transmissibilities would need to be redistributed. it could be either way, depending on the approach chosen. @blattms?)


Yep :)


Remove this


remove this


Once I got the hang of it it was pretty straight-forward. thanks!


Commits where you touch 11 files deserve a commit body (2 min brain dump).  Also during refactors it communicates/reinforces good patterns.  FWIW it does read much better now!


Remember that we used to have some crashes here: https://github.com/Zarel/Pokemon-Showdown/commit/46a5fb0d3b46a062c9fd887a587826c70e55947d


This probably should be `${GITHUB_OAUTH_TOKEN}`


You are right. I missed the directory resolving part. I think we should ignore my pervious suggestion and keep `$0` as it is. So we don't accidentally break someone's workflow.


I see, so I'll use try-finally instead.


> How many (like what percentage) of our dependencies would work this way (i.e. don't require any hacks)?

I'm pretty sure zero percent work out of the box. I had to hack every single solution to make it build the way we want it to.

This is why I feel it's important to be able to isolate and track the specific changes made to a version -- which is easiest when they are in separate folders since you can git log the folder.


@Bubu, ah yes. I'll update it in a new PR.


Okay then. Nevermind me and feel free to delete the spreadsheet. :)


Fix #15


108453024-FixLeedNumberOfHoursNotMet (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1957 of 1969 tests passed)

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


<li class="has form"><link_to class="button">


Can probably mention this as well in the changelog


Can we just `return metaProcessor->process(...)`?


Yeah that's an unfortunate problem with our world structure. Should it copy anything that isn't a directory with `level.dat` inside, or copy only known file/directory names?


Done


SockJS documentation says "A message must be a non-empty string."


Shouldn't this be:

`U = [U1, U4, U5, U6, U7, U7, U9]`

Instead of:

`U = [U1, U1, U1, U4, U5, U6, U7, U7, U9]`

Mind me, I do not know Python, at all. :-)


Niezamierzone. A później wszystko trzymało się kupy dzięki podpowiadaniu
nazw metod przez IDE :)

W dniu 23.11.2015 o 13:49, nicraMarcin pisze:
>
> Działało, stąd moje było pytanie czy to zamierzone :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lmsgit/lms/commit/190511aabaca1743fe184eebcce4ddc641f56d78#commitcomment-14559197>.
>

--
lion.net.pl - wdrożenia i rozwój Lan Management System


Do we need that `Info`? What does it add? Maybe instead state that execution was aborted?


Just curious if this `Content-Length` comparison should be case insensitive.


ok, I will try merge midMap and comment out the not finished stuff.


its a fix for a weird bug that we can't trace down where the chip replies to 0x10 (shrug)


I have resolved the issue in https://github.com/Sharlikran/wrye-bash/commit/145649ba6b3fd7721c8b8cc7e24d445bdedfffea


Is this (still) needed?


@onegulin.  Durrrhhh yup.  I'l code it up more efficiently / simply when I put the change in `dev`.   thanks.


Added folders and files


Do we need it? If not, can we remove the line?


Everything without level.day inside. Cause mods :/.


Thanks - fixed it. (Wonder why it compiled anyway?!?!)


Renamed and added comment.


Where does this block come from? I don't think we need it, right?


Why not use Accounts.findUserByEmail() which is case insensitive?


This is for [home](http://localhost:8080/home/) page.


This can indeed go away. Are there no more references to m_JumpBuf in all of cling?


I know this is just a sample, but I got curious: can one cancel the connection at this moment?


Can you think of a scenario where this would make a difference? I think your original suggestion is a good one (even if it is a subtly different check) and I fail to imagine a case where this would break something that's currently working. And because of this I'm leaning towards picking up your suggestion.


Hi. Can you please issue a PR if this issue still exists?

Thanks.


> write(message)
Sends a message over opened connection. A message must be a non-empty string. It's illegal to send a message after the connection was closed (either after 'close' or 'end' method or 'close' event).

This is for the server sending messages to the client.

I don't know if there are limitations imposed by the SockJS client, but I have been able to crash a test server by using other WS libraries.


you're rigt, it is case insensitive, i'll fix it before commit


Use `^2.8` as version constraint instead.


eh who needs PRs:
master: 627e2355acb372fc4195d16b0387ce22fcd4fce0
stable8.2: 45e9801a44fb14d71c6949cab143561d454967a


Avoid using id selectors


Is it a problem to cancel the connection from inside this callback? Or from write one?


Good point. I think this may have been right at one point but it's not right now.


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


I didn't see this interface being removed? I think we can and should.


@CorainChicago styling issue in nav bar that needs to be addressed. do we need commented out code on the config file?


Added comment


yeah , that make that the first time you use it , then disappear .
It is related of the behavior in the intWindowStat() if i remember well .
That could be improved .
Test again please ?


This is used only for one unique element on page.


yes, calling ``sol_http_client_connection_cancel``


**1 - The boost buttons are not reseted.**
forgot that eh eh
thx lol


How annoying.


why do not use g_spawn_command_line_sync? https://developer.gnome.org/glib/stable/glib-Spawning-Processes.html


Please do not use Q* namespace for non-Qt classes. This can be named i.e. ReTextPreview.


We don't have a spec or implementation for the singular form of this model, so I don't see the point to adjusting the `.get` action (when users can just use `.query`.  You should also document the lack of support for the singular endpoints in the README around “Each model currently supports the following actions” section.


I guess that the actual version got it covered. If not, can you please tell me or issue a PR ?

Thanks


On 11/21/15, ondrokrc <notifications@github.com> wrote:
> Back on sourceforge, I always requested a pull request with the changes to
> master only and somebody else merged it to the branches (I think it was
> Jerome)? Should I do it by myself and file a pull request to maintenance
> branches?
>

> Reply to this email directly or view it on GitHub:
> https://github.com/ondrokrc/gramps/commit/8f70acfb90c63e0791bf46d360e6fa6ec590f6bb#commitcomment-14546041

This is not an "official" answer (as I am not Jerome),
but in my opinion, very few people (nobody?) is running
"master" so I don't think the xx.po files there get
updated very often.  If I were you, I'd revise the sk.po
file in the "current" gramps, which is gramps42 at the
moment.  Any revisions there will make it into the
next "maintenance" release (4.2.2 is the next one),
and if Jerome feels like it, he can merge any changes
into the "master" sk.po file.  But given that strings
there will probably change before it is released (as
gramps 5.0.0 sometime in 2016), it doesn't really
matter, IMHO.  The "master" branch is only for
development.  Not many users are running it.


yes, something went wrong during the spsa...
Probably ck=10 was too high for tuning such sensitive parameters.


Can we move 'host' var to properties file ?


Okay, `composer self-update` is out. Then, is `composer install` ran automatically by travis? If not, how do we include development dependencies/phpDoc+ before we generate the docs, then remove phpDoc from our Pico package after their generation and upload i.e the `composer update --no-dev` later in the script?


Done


    QTextBrowser.__init__(self, tab)

This way you will set the parent on the widget too.


"Void elements area, base, br, col, command, embed, hr, img, input, keygen, link, meta, param, source, track, wbr only have a start tag; end tags must not be specified for void elements."


It should be fairly easy, most of the code is there,


Here and below: `re_text_tab` → just `tab`, for consistency with window.py.


Was this an unused variable?  This change doesn't seem relevant to any of your other work here.


 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/7507223c8be18a51252db6279fa2966c65653617/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:683](../blob/7507223c8be18a51252db6279fa2966c65653617/include/chaiscript/language/chaiscript_parser.hpp#L683) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/7507223c8be18a51252db6279fa2966c65653617/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 681. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:685](../blob/7507223c8be18a51252db6279fa2966c65653617/include/chaiscript/language/chaiscript_parser.hpp#L685) Expression is always false because 'else if' condition matches previous condition at line 683. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/7507223c8be18a51252db6279fa2966c65653617/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:0](../blob/7507223c8be18a51252db6279fa2966c65653617/src/main.cpp#L0) Too many #ifdef configurations - cppcheck only checks 12 of 13 configurations. Use --force to check all configurations. (toomanyconfigs)
 - __information__: [src/main.cpp:1](../blob/7507223c8be18a51252db6279fa2966c65653617/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/7507223c8be18a51252db6279fa2966c65653617/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/7507223c8be18a51252db6279fa2966c65653617/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/7507223c8be18a51252db6279fa2966c65653617/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/7507223c8be18a51252db6279fa2966c65653617/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/7507223c8be18a51252db6279fa2966c65653617/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Fixed, now raises an exception


Yes can do :D But just a lil more work form the coder side :p


What more work are you expecting