Yes, as far as I could tell it was never used.


realign?


LGTM


`enum34` is only available in Python 2.


Good implementation of all the required functionalities of the game


I pointed it out  because its not the part of game (atleast the one which i played earlier)


Well, unfortunately it's slightly more more complicated than that. Thanks to ballooning, ploop filesystem mount is two-step process: first we mount it r/o and read the balloon inode. Second, we remount it with option balloon_ino=N. If we do attach_ploop() and sys_mount(), we lose this and balloon file will be seen from inside container. For details, see ploop_mount_fs() in https://github.com/kolyshkin/ploop/blob/master/lib/ploop.c. The other thing you can find there is we can optionally fsck the filesystem just before mounting it, but I guess it's not as important as hiding the balloon file.


It *should* be optimized away by HotSpot, but it's probably better to be explicit.  I'll make the change.


s/failes/files

LGTM other than that
(sadly my last pr was still incomplete, despite being big... =/)


I thought you're going to put this into the `data_dir` module, but it's still better than nothing...


:D


PEP8 - Multiple imports in one line


should just return `None` as NetMHC does not implement any version information in interface


Missing white space after ","


Multiple imports on one line


:D

On Monday, September 28, 2015, dzhan008 <notifications@github.com> wrote:

> ..
>
> On Mon, Sep 28, 2015 at 9:31 AM, Rica Feng <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
> > np man
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/dzhan008/cheshiregame/commit/81ac88c4d7652e1ac078bfaad312a26cb67bd187#commitcomment-13473175
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dzhan008/cheshiregame/commit/81ac88c4d7652e1ac078bfaad312a26cb67bd187#commitcomment-13474178>
> .
>


Only allow TLS 1.2+


No queen is present,player cannot die.


Correct.  The 2nd uploadPart from the first call fails, so we'll see it repeated on the retry.


awesome :+1:


You have to very careful here. With the "offline" import feature, I am not sure this will cover all the supported cases now... I will have to take a second look at this one.


Specific entity information shouldn't be in the SDK.


Multiple imports in one line


Game does not meet any basic requirements.frankly this is not a game.


esto esta mal aca... el comentario para mi es correcto, ya que un driver no tienen lowerlayer...


Code smell : Too many blank lines. PEP8 Standard not followed.


@bmorris3 - No, I meant something a bit more complex, where you'd want to actually completely supress the "live" update mode so that you can e.g. write out a bunch of finding charts in a row without waiting for the GUI to update.

@cdeil - I disagree it "saves the user a little typing" for the use case I described above.  You have to do more than just a savefig call at the end - you have to actually fiddle around with backends if `draw` always gets called in astroplan.

All that said, I *do* see @cdeil's point that this adds quite a bit of complexity, as well as @bmorris3's point that it's not in the other plots.  So it's probably right the best thing is to issue a PR that just does this first and then a later PR might add that to *all* of the plots.  (Or we can just say too bad so sad and force the user to deal with matplotlib backends themselves.)


Wasn't the whole point to not expose `Session` to driver authors?


no me queda claro, se para todo o solo el device que se abrio?


Okay nice! Did you tell @remvee? :sparkles:


I added the following here

            $documentId = $file['id'];
            $obj = new \C_Document();
            $document = $obj->retrieve_action("", $documentId, true, true, true);

It downloads the document each time not the file name. (just the first document is processed), $document contains the file even it I set the last flag to false.


bug : player is going into the wall.


I find it interesting that the VIM user base still seems to be growing...


PEP8 - Indentation contains tabs


un comentario etaria muy bueno... :) no me queda clara la magia. el *100 es por 100%? deberia ser un macro?


yeah I thought about that. But i wanted to expose session to the parser.

On Mon, Sep 28, 2015 at 1:40 PM Tamir Duberstein <notifications@github.com>
wrote:

> Wasn't the whole point to not expose Session to driver authors?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cockroachdb/cockroach/commit/e0627d2d2fdf2f97b54434fa1155187770516510#commitcomment-13474723>
> .
>


The “multi-volume” is actually a scalar volume with many scalar components (there is a scalar component for each volume).


This code has been checked into the magnum ripper tree here:

https://github.com/magnumripper/JohnTheRipper/commit/91279267e2a8b0f508873f076950f3e5612c05b5

NOTE, the last if() statement which actually does the printout has been commented out in the magnum-ripper patch.  that is because the only things left are FP's.  However, we really should look at 'why' we are getting them, and if we keep this code, try to make it smarter.

NOTE, I am going to work on a generic method that is very similare to the one I used to find known binary() and known salt() values. But in the generic version, it will simply find 'valid' hex strings, and mangle them and then check that the string either is 'fixed', or fails valid.  This test will possibly find some new bugs, but 'may' end up having some FP's show up also.   Likely when that one is done, this blind case change test can simply be scrapped.


FYI @wdaher

This ensures that https://zulip.github.io/ → https://zulip.org/

It doesn't actually mean we have to host zulip.org via GitHub pages.


Sounds controversial. Let's revert this for now.


Some of this code comes from https://github.com/loverszhaokai/JohnTheRipper/commit/01abf3ff176d371441b979fb750c885d3c91f0d9


PEP8 - No white space after ","


loopi + 1? no entiendo por que se hace esto.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1360 of 1360 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)


I don't think we want to expose session to the parser.


donde esta declarada esta funcion? si es external esta bueno agregar extern aunque desde c es igual, da claridad al codigo.


Code Smell - Line too long


falta all rights reserved.


Score of the player doesn't increase after reaching the queen


aplica esto el driver?


Code smell : Bad white spaces.


PEP8 - Missing white space after ","


@jeremy-w I'm not entirely sure what you mean by "decompose the function"?


use arrow functions instead of w,s,a,d (or) represent your assumption in readme.txt


si todo esto es copy and paste de otro archivo, como parece debe tener el copyright anterior. en caso contrario debe ser copia de los templates.


bug:player cannot jump while running.


code smell - Line too long


Log /proc/net/snmp


si no es soportado lo indicaria, ya que devolves siempre -1


Code smell line too long


en read  write hay que explicar mucho, yo no entiendo qeu se lee ni en que formato.


Code smell - line too long


idem al read.


ack


no deberia ser null?


por que necesitas este archivo?


Think this line can be removed.


fire balls always follow a fixed path it would be better if it was not like that.


Coordinates should be declared as Variable name.


saca esta linea y te ahorras el archivo si contenido.


Meaningless for SEO :) "Adding a README using Markdown" or something.


"a little" -> just "context", no belittling


no deberia retornar null?


SEO: "Importing a README from GitHub"


use arrows as convention or represent your assumptions in readme.txt


According to inheritance logic, I think we should have
````
%w(three four one two).each do |value|
+      it { Girl.filterrific_available_filters.must_include value }
+    end
````
As a subclass integrate everything in the superclass, and add some new things.


"an icon" - little is not needed


Code smell : Bad indentation . PEP8 standard voilated.


Also add the images to the project file...


Wouldn't *any* also be deterministic, and say more? In any case just a nit. Not worth holding things up over.


Perfect ... if ready, send out pull request!


bug:q will not quit the game


Why was system shrunk?


bug:after hitting the fireball player will not start from the beginning.


Leave this commented


Fix conflict heads


Conditional Complexity. Used too many if - elif conditions.
Could use alternate OOP concept like decorate etc.


@neobuddy89 should it not resume as whatever the max freq set by user in cpufreq driver? That would make more sense


also I'm getting the following invalid configuration when I try to use this change
with VIRTUAL_HOST=*/sort, */sort/*

Notice the extra acl path_rule_3 which duplicates path rule 2 but for non-existent backend causing the configuration to fail and haproxy to not work

frontend port_80
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   bind :80
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   acl is_websocket hdr(Upgrade) -i WebSocket
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   acl path_rule_1 path -i /sort
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   use_backend SERVICE_TST_SORT_WEB_GREEN if  path_rule_1
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   acl path_rule_2 path_reg -i ^/sort/.*$
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   use_backend SERVICE_TST_SORT_WEB_GREEN if  path_rule_2
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   acl path_rule_3 path_reg -i ^/sort/.*$
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   use_backend SERVICE_TST_SORT_WEB_BLUE if  path_rule_3
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z backend SERVICE_TST_SORT_WEB_GREEN
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   appsession JSESSIONID len 52 timeout 3h
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   server TST_SORT_WEB_BLUE_1 10.7.0.10:8080 cookie TST_SORT_WEB_BLUE_1 check
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   server TST_SORT_WEB_GREEN_1 10.7.0.13:8080 cookie TST_SORT_WEB_GREEN_1 check
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z   server TST_SORT_WEB_GREEN_2 10.7.0.14:8080 cookie TST_SORT_WEB_GREEN_2 check
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z INFO:haproxy:Launching HAProxy
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z ******************************
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z [ALERT] 270/174308 (18) : Proxy 'port_80': unable to find required use_backend: 'SERVICE_TST_SORT_WEB_BLUE'.
tst-multi-lb-1 | 2015-09-28T17:43:37.300098972Z [ALERT] 270/174308 (18) : Fatal errors found in configuration.


great suggestion @floatplane. Thanks for the CR!


broken ladders is not implemented.


if I understand it correctly the user action sets the trigger bool? In that case queueing would be solved by not calling the method again until the trigger is set back to false once resolution is changed.


It's for side-effects actions. Like timeouted sync or something like that. Dispatch must be passed in.


Užkraunu "list.json" failą iš karto, kai tik užsikrauna puslapis.


Great! :+1:


Large class. could break in smaller classes.


I'm sorry, but this seems to blow up my config :) I don't really know what this is changing, but could it be that the logic is flawed? if (l 78) and else (l 80) seem to make the same statement, not being an option than?

see here for the log https://bpaste.net/show/69d3fbc77f1f


time module has not been used


For anchors, always set the `href` to include the `https://` protocol (if you know it exists).

It's only for linked resources e.g. `src` attributes on scripts or images, where you can leave off the `http|s` so that the correct protocol is loaded, based on the parent (current) URL.

So, links: `<a href="https://www.facebook.com">` and e.g. images `<img src="//placehold.it/100x100">`

The _reason_ this is done, is to prevent a request for e.g. a non-secure (http) resource (image, script, etc.) when your site e.g. https://www.putterschool.com is serving secure content.

One of these days everything will just be `https`, but until then keep the above in mind.

@laurenwaller @mieliep :arrow_up:


I cant see this button at `Git commits` :(


This will be removed.


Missing module docstring.


propMerge in master had this.. Forgot to add it here.


This looks very good! I'd be happy to merge this if you'd make a pull request!


np, hope this works! :smile:


Those 3 lines edit are related to @5c99ecb8ba376e5fb47df47d1e87f946a6385f47


no fireballs,donkey cannot kill the player.


This will work and it's the x86_32 classic, use `call` to push the current IP on the stack and add the distance of 0 and 1 in order to determine the return address, which NOVA requires in edx.


@chexee Are there any other elements which might work here as well?


I think will be better to check here the type of error.


I forgot to use the new code actually it is:
```
after_create :generate_secure_key
```


See also this : See also this [comment](https://github.com/RCura/transition8/commit/954b522c1554c38485d9b7de32210506012904ae#commitcomment-13475073)


remove empty lines in places like this


This undid the LESS fix for Meteor 1.2


Excess Use of literals in class .. Bad code smell https://en.wikipedia.org/wiki/Code_smell
>Excessive use of literals:
>>these should be coded as named constants, to improve readability and to avoid programming errors. Additionally, literals can and should be externalized into resource files/scripts where possible, to facilitate localization of software if it is intended to be deployed in different regions.


should we handle NULL? I'd say better safe than sorry, so add a `SOL_NULL_CHECK(instring, -EINVAL)`


My goal for the theme and shimmie is to allow it to run independently also.. Without needing to download from external sites like a CDN for example. So I was trying to stick to that best I knew how. As for the images, we can replace them of course :) It was just an image that fits the normal rectangle size, so I thought it would be a good fit.


Reverted.


we make deadline the default, instead of timeout/seconds. That means, even if you set seconds, it gets converted to deadline and passed on down.... notice that we pop timeout above, but only get deadline.


mario  and donkey-kong functions in same class and code is unreadable


I will comment here for now. Could you see if can create an issue and mention @codepathreivew? This way your repo will be picked up by our software


don't add these parenthesis in python (only if there is the need to explicitly state the order with multipl `and` and `or` statements).


no parenthesis.


:+1: nice work. The primary challenge of this homework was to learn how to use UITableView to manage a complex form, establish communication between view controllers, and apply the correct Auto Layout constraints.

Here's a checklist of things that I'm looking for in this project. You should review the checklist, and make any changes to your project, if necessary.

* For the restaurant cell, note that the subviews should be private and only updated in the Swift property observer.
* All screens in Storyboard and nibs should contain Auto Layout constraints. Make sure to resolve all the Auto Layout constraint warnings.
* In the filter view controller, make sure the cells are being recycled and dequeued and appropriately linked to the model. See this [popular form library](https://github.com/nicklockwood/FXForms) for an alternative to implementing forms manually.
* The restaurants view controller and the filters view controller should be connected via a custom filter delegate. This is a popular pattern for handling events between view controllers.
* Note: Connecting the filter view controller directly to a global filter settings is convenient, but the downside is if you cancel the filters, the values have already been saved.

This was a challenging assignment, congrats on completing it successfully!

-- dirk


Unused arguments 'x' , 'y' , 'x_change' , 'y_donkey' , 'y_change' , 'x_donkey' .


You can achieve the something in a much more pragmatic way using `redux-thunk`.


Thanks. Fixed in v2.051.
https://github.com/apmuthu/LimeSurvey/commit/33382bd4305c4cca1bf6589bd5a4f70d298b77d0


You're right. I will add the check.


give meaningful variables not random since code becomes unreadable


I think as the number of these classes grows, we'd find it convenient to have it sorted alphabetically


here I'd use the slice operation as above:

```python
    return types[len("composed:"):].replace(",", "_")
```

and you can use this function there as well.


@boomstick8x Because the code was updated to the new backup system when you just updated, you won't see the restore buttons until you update again. Also, this is just the framework. I haven't implemented the buttons doing anything yet, but the hard part is out of the way (I think).


I had the same problem: https://github.com/trevorf/ti-android-streamer/issues/4
If it doesn't create a new instance of multiplayer again and just reuse the old instance and just set the playAsync url. I'll redo the changes and try to reproduce the error.


resort?


Bad code smell .. https://en.wikipedia.org/wiki/Code_smell
>Excessive use of literals:
>> these should be coded as named constants, to improve readability and to avoid programming errors. Additionally, literals can and should be externalized into resource files/scripts where possible, to facilitate localization of software if it is intended to be deployed in different regions.


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


I'm trying to remember why, but I changed _FONT_NAME_REGEX to a list instead of a string, so using join was necessary. I think this is a bit cleaner though so I'll submit another commit to do it this way.


Why not just submit this as a PR? :)


in python is easier to write:

```python
s += " %s%s out_value_%d," % (data_type_to_c(type), pointer, i)
```


time module not used


Add more details to readme. Better add per-requisites required to run your game . Like installing pygame module in your case. You may also add details like for what the a particular module is written etc


well you have this:
https://github.com/JACoders/OpenJK/blob/master/shared/sdl/sdl_input.cpp#L797
its basically the same I guess


Can use a object to pass in argument to prevent from too many parameters.


nice job making the changes! 10/10


Oh yes, it does. Not all versions, however. Anyways, both 3.0a and 3.4 sport the --version flag, and since these classes are rather version specific...


Why was this test removed?


Why were the done channels removed? Here and elsewhere.


Do not use wild card imports . Its a bad coding practice . better import what you need


#!/usr/bin/env node

'use strict';

var pg = require('../');

var argv = require('yargs')
	.usage('Usage: $0 [options] <message>')
	.describe('r', 'Specify the repository slug')
	.alias('r', 'repo')
	.describe('c', 'Specify the commit to comment on')
	.alias('c', 'commit')
	.describe('t', 'Specify a GitHub authorization token')
	.alias('t', 'token')
	.argv;

function makeComment(comment) {
	pg.comment(comment, {
		repo_slug: argv.r,
		commit_hash: argv.c,
		token: argv.t
	}, function (err) {
		if (err) console.error(JSON.stringify(err));
	});
}

var commentArg = argv._[0];
if (commentArg) {
	makeComment(comment);
} else {
	process.stdin.pipe(require('concat-stream')({
		encoding: 'string'
	}, makeComment));
}


Or we cure the root cause that is the render thread and main thread are the same thing.


That would defeat purpose of MSM Limiter.
Limiter is created to keep frequency fluctuations in control.
Many time frequencies are fluctuated by thermal driver / mpdecision / apps / scripts.
That's why MSM Limiter stores Max Freq in resume_max_freq_<core> attribute and restores same via calling cpufreq_set_freq(max_freq, min_freq, cpu);


spaces and not `()` around `"_"`!

```python
declare_packets(stub_file, t, packets, data_item["name_c"] + "_" + c_clean(t["name"].split("/")[1]))
```


Inconsistent names.
Can use standard terminology and stick to it throughout your methods


haha lol


I think if it's a raw function, this will throw an AttributeError exception. We should probably either try/except this or do a hasattr or other test check.


Ues thats it @pixelastic


Report for HW1


well then it is no problem because it is labeled for version 2.4... well have to cook up one for 2.8 eventually, though I don't have that version handy right now to test


I think you wanted `maven_repositories` here so the link actually works.


Ready for re review @Jerskouille


Compatible with injectDependencies? I don't think so since both middlewares are using isFunction test.


Why was the change to the loop needed?


Should we do that or should we handle http://www.gravatar.com -> https://secure.gravatar.com ?


@TheChymera please fix the checksums.


Too many branches.


Or should we get EMS to do a find/replace in the db to the secure URL format?


good layout of the game.


Use communicative variable names.


Ah, sorry. I don't use `injectDependencies` myself. Don't even understand what it does.


Make this case more specific.


ouch... this condition still makes me worry. we shouldn't have to deal with that.

Also, you should be calling `sol_flow_packet_get()`, not `sol_flow_packet_memory_get()`, some types can override their getters and you'd be missing that. The correct way is to:

```c
    void *data = calloc(type->data_size);
    const struct flow_packet_type *type = sol_flow_packet_get_type(in[i]);
    r = sol_flow_packet_get(in[i], &data);
   SOL_INT_CHECK_GOTO(r, < 0, err_get); // free data and goto err_exit
   array[i] = sol_flow_packet_new(type, data);
   free(data);
   SOL_NULL_CHECK_GOTO(array[i], err_exit);
```


Any chance someone could take a look at this pull request?


And please, always use repoman! ALWAYS!!!


Makes sense.

But I would still like the branch to be merged so that we have a code base that actually compiles in master branch.


It's not a critical issue, but there was no need to edit this line. `++.@i` is faster than `.@i++`, which is why we always prefer the prefix form, unless there is a special reason to use the other.


code smell : Unused variable 'pos'.


@albertotorresfoltyn there is a forgotten debugger;


Better to check for modelPath first before evaluating this huge line.

```
if (modelPath) return <huge line>
else return this;
```

Also it can return null in some cases, but I assume it is ok, isn't it?


Two line gap must be left between class name and it's first method.


FWIW, this all looks sensible and extremely readable to me.


@RyanBalfanz The containing section could be <main>, (or assuming we'll be adding more content, <main> should be a container around that section).


yep


That's OK - we can do that - just create a pre-xcode7 branch from current master as reference first.


Well, I did like this because I did not want to alloc more memory.
But you're right a packet can override the get function.
I will change the impl.


Why did this change from 1 to 2?


This could `<main>` or, could add another `<main>` to contain this and account for adding more `<section>` s


as I said before, you might leave it as is only add the `finally` block. It'll be evaluated even when `return` is called (in try of except block).


Too many if's used -> code smell


where is twoBinomialPost along with other data sets missing from the course


But is it webscale?


teste


Looks pretty good. I still need to try it out.


@JeroenDeDauw A sync with WMF will be cool.


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


@cboettig sounds good


well thank you VS... fixing.


Add yahoo minifier step previous to this.


Opinion, as long as you're asking:
1. I'd like to get rid of joint states soon.
2. DRY is more important. In a perfect world, either Rig or a JointStates (plural) class would define a getJointState(), such that there would be only one place that has this code, whichever way we decided to go on failing.
3. To actually answer your question, IMHO, we should fail hard, but it's best to fail hard with a specific message. So I would have getJointState() test and fail, rather than test and skip-silently. It's too much of a PIA to do that everywhere, hence #2 becomes important.
4. Aside: I'd rather have such tests exit the function (regardless of whether it fails hard or not), rather than causing everything to be nested deeper. I find such nesting hard to read, and prone to diff-bloat. For example:
if (index == -1...) {
  return
}
JointState& state = ...


Any code using this would be a helicopter currently, because this code is for helicopter behavior. We don't have the logic to properly distinguish at activity level yet. Do you want me to throw if it's a plane?


Also, combining #2 and #3 effectively forms a self-documenting spec for the API.


sure but on line 24 you set kwargs["_deadline"] and on line 19 you set kwargs["deadline"]
is that intentional?


`/dropbox` not `/docker`


well I would use `glob.glob(os.path.join(data_dir.BASE_TMP_DIR, 'avocado*'))` and `self.assertFalse(avocado_tmp_dirs)`, but I have no strong opinion about this...


the `Dropbox` directory should be created by the dropbox client. we should note this directory creation as optional.

also, lets pull the `mkdir` out of the Dockerfile since this needs to be done on the host volume as you've found.


Pushed a fix with better handling for non-method action functions: Seems to work well for me in Inquisitor (our only Pybald 0.4 app), though I haven't tried it with a raw function action...


I wasn't aware of it being Qt 5+.
@lordofbikes Can you add the conditions? I don't have experience with Mac.


1. Достатньо один Scanner на всю програму. Як я вже писав, Scanner — це просто обгортка над вхідним потоком даних (System.in), абстракція, яка ховає низький рівень (зчитати байти, перетворити на літери, розбити потік літер на окремі рядки і т.д) за високим рівнем (`next()`, `nextInt()` і т.д.).

До речі, +1 за `next()` і `nextInt()` замість `nextLine()` — схоже, так воно навіть стабільніше.

2. Змінні `rangmin` i `rangemin`, `rangmax` i `rangemax`... Треба докласти зусиль, щоб не заплутатись.
Окрім того, якщо у назві змінної кілька слів, кожне слово крім першого повинне йти з великої, напр. `rangeMin`, `rangeMax`.


fatal on fullXML suite (not running on kernel here):

```
PHP Fatal error:  Cannot use object of type Behat\Testwork\Call\Exception\CallErrorException as array in /home/travis/build/ezsystems/ezplatform/vendor/ezsystems/ezpublish-kernel/eZ/Bundle/EzPublishRestBundle/Features/Context/SubContext/User.php on line 81
```

https://travis-ci.org/ezsystems/ezplatform/jobs/82598241


This does not provide consistent behaviour. If I haven't been in the Hub today, the link in the daily page does not perform any action since it can not reference an element that does not exist. Consider replacing the "Find Me!" button with a text that the user does not exist in this timeframe, instead.


what about a method ``Warrior.resetLife()``? For that, besides the current life, a Warrior would need to remember its initial life value.


It definitely doesn't work :smile: so maybe it should just be deleted to stop confusing people.


@kizniche Ok, thanks, now I understand. I manually create a dir`/var/Mycodo-backups/CommitId` and see the button :)


@adg There is no point of polluting the APIs. Users must wrap the RoundTripper to cache. We must document it by providing a sample. If you would like to contribute a sample, I can review.


Figure out what the heck is going on with the build: https://github.com/kjur/jsrsasign/issues/57


That's ok, I stole my original contact page from you.
And BTW, this jsonresume sounds reasonable too I suppose :D


You want to rebase this out...


Окей. Хоча якщо виходити із програми через `System.exit` через помилку, то варто вже придумати і ненульовий код помилки (`exit(0)` значить, що програма закінчилась без помилок). Або ще краще — кинути exception. Але це так, на майбутнє.

Насправді, просто `return;` тут спрацював би (`return` — вихід з методу. Оскільки ми зараз в `main`, то це буде вихід із усієї програми)


Because it doesn't apply any more. When the syslog server does not come up, only the current message is pulled.
There is no guarantee that you would ever see these "undefined" messages which is tested by the TruncatingBuffer.

Futhermore, the fact that these overflow messages appear is a function of the TruncatingBuffer not Syslog Sink.


Let's not remove the HTTPs writer test. We had a bug previously in the code where the "Connect" would always return nil and the exponential backoff won't trigger. This test was written specifically to catch that bug.


PEP-8 standards violation
invalid method name getScore


The phone being locked should not prevent GcmListenerService from receiving messages. So if that is the case WRT the GalNote 2 that you have could you submit that issue via [this form](https://support.google.com/code/contact/gcm_dev_support).

GcmServiceListener is expected to work without power management by the developer. Thus PowerManagement can be removed from MyGcmListenerService.


Can you test with this url? http://www.dailymotion.com/video/k5PN3L5iRe9nm22I5CM

These longer urls were giving people issues, which is why I created this fork


Done was initially added as a crutch for Ginkgo and should no longer be used. Eventually came along to replace most if not all usage of Done.


nice, only the same note as before, it'd be more convenient to use `__name__` instead of some name.


#FishPala


Hm, this will fail in cases where next to `information_schema`, there are a number of different databases. So issuing `SHOW DATABASES;` would return with

```
information_schema
public
wordpress_installation
client_3498
my_random_tests
```


A wrapper for `TokenSource` can provide caching. +1 for a code sample.

`ReuseTokenSource` is a good example.

https://github.com/golang/oauth2/blob/master/oauth2.go#L310


Let's move this to a bug.


It turns out most numpy_expr do not implement the ``size()`` method, I don't want to modify this now ;-)


The first message is no longer lost so you end up with m1, followed by whatever truncating buffer will provide.
Prior to this m1 was dropped which is exactly what this fixes. You will receive the current message being processed rather than dropping it on the floor.


Do `SELECT` queries involving `phone_col ` work at this point in the test, prior to shutting down the client?


Розумне рішення. Якби було просто `(rangemin + rangemax) / 2`, то при дуже великих числах можна було би вилізти за межі значень int`а.


Nice catch on [the data model change!](https://developer.paypal.com/webapps/developer/docs/api/#store-a-credit-card) :+1:


Bad code smell:
   too few public methods


Actually looking at `injectDependencies` it does basically the same thing as `redux-thunk` and a little more.


I propose we move from alxndrsn/enketo-core to medic/enketo-core. I've forked enketo/enketo-core to medic/enketo-core; the `master` branch seemed to match alxndrsn/enketo-core, but we may need pulls/merges for any other branches that matter.


This test is superseded by [`GetCustomizationShouldMatchByExactParameterType`](https://github.com/AutoFixture/AutoFixture/blob/freeze-attribute-nunit2/Src/AutoFixture.NUnit2.UnitTest/FrozenAttributeTest.cs#L73-L90) and [`GetCustomizationShouldMatchBySeedRequestForParameterType`](https://github.com/AutoFixture/AutoFixture/blob/freeze-attribute-nunit2/Src/AutoFixture.NUnit2.UnitTest/FrozenAttributeTest.cs#L92-L109) which both test `FrozenAttribute` initialized with its default constructor.


Question: in this file, I started the `err` label with `err1` since that was already here.  However, in `tar/write.c`, we used `err0` to include
```
err0:
        /* Failure */
        bsdtar->return_value = 1;
        return;
```

Or is the difference that this `err1` included a `bsdtar_warnc()` ?


@limorui thanks for contributing! lets talk about this addition when we meet later today. I'm interested to get a better sense of your objectives in adding this section, as well as provide help in clarifying certain language.


No ...


Maybe this is failing because you need to keep the `return index` line here? (Not sure if that's a good lead, just looking for implementation differences.)


Before we looked for the messages the truncating buffer injected, now we have the original message plus whatever gets injected.


Note for JJ:

This obviously isn't fully implemented logging (haven't made an enum for severity yet, not using my "int sev" parameter, etc...). My goal was to implement something with variable args :)


PEP-8 standards violation:
Unnecessary parentheses after while keyword


Хех. Бонусне завдання «на засипку» на майбутнє — написати такий код, що вертатиме правильне слово залежно від числа. [Підказочка](http://www.unicode.org/cldr/charts/latest/supplemental/language_plural_rules.html#uk)


Good idea


good one. CMAKE has interesting differences to what we think ;-)


yes this is correct


Neither did I, thanks [2]


maven-checkstyle-plugin будет ругаться на отсутствие форматирования и отсутствие пустой строки в конце файла.


My other guess is that this has to do with https://github.com/mapbox/mapbox-gl-js/pull/1568#issuecomment-143833531


Warning:
 bad indentation Found 8 spaces, expected 4


Great idea! :)


OKay


what is stash?


PEP-8 standards violation:
 exactly 1 space required after comma


Why this revert, are you getting an error with this commit?


@suriyanr, @mdrjr is that possible to actualize this for latest 3.10.y kernel version?


PEP-8 standards violation:
 Wrong hanging indentation


Never mind; after looking around some more, I'm 99% certain you want `err0` for the `bsdtar->return_value` and `return`.  Adjusting commits accordingly.


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/1b40b7e6d8f970bf478510f89804228522ad16be


merging basho/yokozuna/better_list_comprehension_and_dry = 1b40b7e6 into borshop-integration-563-better_list_comprehension_and_dry


basho/yokozuna/better_list_comprehension_and_dry = 1b40b7e6 merged ok, testing candidate = e525e1a7


@Galixte Because that is the way they were from the 3.0.x mod.


TeamCity pull requests :: netty [Build 323](https://garage.netty.io/teamcity/viewLog.html?buildId=1795&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5619, ignored: 21 Build time: 00:38:08


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


Just curious: what is the approach to time code issues (besides changing `vcr` to `vcr4j` ;^)  How do video files store time code equivalent?


This does not compile there is no class to be found at ic2.api.item.IWrenchHandler

Its not on your repo, its not in the ic2 api.


No such class, that I can find.


Oh, alright. Thanks!

On Mon, Sep 28, 2015 at 1:28 PM, Arthur Thompson <notifications@github.com>
wrote:

> The phone being locked should not prevent GcmListenerService from
> receiving messages. So if that is the case WRT the GalNote 2 that you have
> could you submit that issue via this form
> <https://support.google.com/code/contact/gcm_dev_support>.
>
> GcmServiceListener is expected to work without power management by the
> developer. Thus PowerManagement can be removed from MyGcmListenerService.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jesdynf/google-services/commit/44e46b5a6e3b1f22cbf4ee9c30efaa816c536d60#commitcomment-13475816>
> .
>

--
Asher Densmore-Lynn <jesdynf@gmail.com>


@AhmedMustafa Thanks! Let's get that fixed? Please, could you submit a PR fixing this content? Thanks in advance.

@ManrajGrover Ah! Sorry. Thanks.


This is an integration test not a unit test.
It sounds more like an issue that should have been tested on http_writer.Connect().

I don't mind moving it elsewhere but it makes little sense here.


PEP-8 standards violation:
 Line too long -max 80 chars here 133


PEP-8 standards violation:
  Exactly 1 space required after comma


@sullivanm I don't think this is one of the right places for this.


It appears this caused #3094.


IMO it would be even better to define missing links explicitly to let us know what parts are to be written later.

Something like this:
```
[edge cases you should take into account][wrapping]
...
[wrapping]: http://404.html
[reusing]: http:// 404.html
```


PEP-8 STANDARDS VIOLATION:
  Exactly one space required around comparison


Since it would require a new repo, and i wasnt sure if you wanted to go with it :)


Sorry I don't see why this is required. CMake should not create any folder in the cmake folder.
Could you elaborate why this is necessary?


lol


You should still null check even for API 11+.


@chrisndodge @jhendersonedx @griffresch Added this note to course staff doc re modifying policy on a per exam basis.


Thanks


Bad indentation. Found 12 spaces, expected 8


========== Build: 5 succeeded, 1 failed, 6 up-to-date, 0 skipped ==========


And it uses another kernel repo, since zeroltexx is the s6-edge


here's another email


@chrisndodge @jhendersonedx @griffresch Added this section to course staff doc re requests for special allowances.


hiokaslkdnalskd h


Fireball not working and jump feature not implemented


@chrisndodge @jhendersonedx @griffresch  Added mention of "special allowances" (used to be "Request Additional Time" only) to student-facing doc.


We should be careful to say that "standard rules" means to Software Secure as "Closed Book". That is the strictest set of rules and anything (e.g. use of calculators, reference materials, pencil & paper) will get flagged as failing.


Whole code is written in a single file and is not distributed into modules-Absence of modularity


Video files in .mov. or .mxf containers can have timecode tracks for storing SMPTE timecodes. That track would function exactly like a tape's timecode track. The timecode track is frame-rate dependent, so transcoding the source video to other formats can result is problems with the track.

All other video containers would be indexed using run-time (i.e. elapsed-time). Some video containers support a header atom that contains the date/time that the video file recording was started. That atom, coupled with run-time, provides a date/time of each frame.


This doesn't solve the issue of unit tests that have hardcoded decimal separator on non-US systems:

![unittestbreakage](https://cloud.githubusercontent.com/assets/944894/10145156/74e0d9d0-65ef-11e5-9b3a-66913c3dd216.png)

Culture should be optionnaly exposed to force it in unit tests and in other scenarii where current culture is not desired.


I saw this commit, but 1 min is still not super efficient I guess ;-)
My typechecker takes an option of an expected type `pt`, calculates the most precise type, and then calls [adapt](https://github.com/samuelgruetter/dot-calculus/blob/303c571771bc070da6497d45464da2f723eeb329/dev/mylib/experiment.v#L1240) to check if the precise type is a subtype of the expected type.


@chrisndodge @jhendersonedx @griffresch Added this note to the proctored exam rules that we publish for students and course staff.


Is the return inside the if statement really needed?


It does boot with the old defconfig, not sure about this one.
Didnt test it either and idc if it works tbh, since i quit development for this device :)


all _includes styles are still compiled into global.min.css, the css/includes folder is for reference.


ERROR/BUG: player cannot jump if he goes beyond  the floors in air


If __name__ == '__main__' not written in main.py. Needed in order to prevent it from running when imported.


We should note that this ^ needs to be completed *before* said student interacts with a proctored exam. It cannot be retroactively applied to a proctored exam which the user has already started. We can delete the previous attempts, but that will delete any work the student might have already started.


### testing trick: mock content from the filesystem


Thanks.


This commit allows to execute 'push-in' feature from Spring Roo Shell. The only problem is that some other add-on Metadatas needs some refactor to validate that method exists on .java file and remove it automatically from .aj file.


Mmmh you broke the find command here.


When attempting to connect I get the message "connection error - error setting version of protocol (also check console)" but there is no output in the console. Is this because of the definition of the UDP port?


1. Не ставте інструкції після `{`, а починайте вміст блоку з нового рядка.
Єдиний виняток, коли таке роблять — коли оголошують метод, що складається із одного рядка, наприклад
`public static int getSquare(int x) { return x * x; }`

2. Та думка, яку я так і не закінчив. Тут і на 32-му рядку мені не очевидно, що означає число 2. У програмуванні такі числа називаються [магічними](https://ru.wikipedia.org/wiki/%D0%9C%D0%B0%D0%B3%D0%B8%D1%87%D0%B5%D1%81%D0%BA%D0%BE%D0%B5_%D1%87%D0%B8%D1%81%D0%BB%D0%BE_(%D0%BF%D1%80%D0%BE%D0%B3%D1%80%D0%B0%D0%BC%D0%BC%D0%B8%D1%80%D0%BE%D0%B2%D0%B0%D0%BD%D0%B8%D0%B5)#.D0.9F.D0.BB.D0.BE.D1.85.D0.B0.D1.8F_.D0.BF.D1.80.D0.B0.D0.BA.D1.82.D0.B8.D0.BA.D0.B0_.D0.BF.D1.80.D0.BE.D0.B3.D1.80.D0.B0.D0.BC.D0.BC.D0.B8.D1.80.D0.BE.D0.B2.D0.B0.D0.BD.D0.B8.D1.8F). Замініть `< rangemin + 2` на `<= rangemin + 1` і `> rangemax - 2` на `>= rangemin - 1`, і все стане значно зрозуміліше (а ще краще — додатково до цього написати коментарі)

3. В даному випадку неважливо, бо `answer` ніколи не буде `null`, але на майбутнє: коли порівнюються об’єкти, краще викликати `.equals()` на тому об’єкті, який точно не буде `null`. У даному випадку, `"<"` точно не `null`, тож краще писати такі умови як `"<".equals(answer)`. P.S. Поставте курсор на `.equals` і натисніть Alt+Enter — ідея виведе список дій, що тут можна автоматично зробити, зокрема запропонує поміняти місцями об’єкти (Flip equals).


Initial commit


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1361


こめんとこめんと


How?


необов’язково брати в дужки. можна просто `middle = middle + step`. або ще простіше — `middle += step`


ERROR: after reaching queen ,queen doesn't disappear and player's position also doesn't change.Also lives are not shown


again as before, I'm not sure if target `clear` should touch files not created by the installer`. Now when I think about it I'd probably movie it into `check` part.


`counter++` є у кожній гілці — замість дублювання коду варто винести це взагалі назовні (всередині циклу, але зовні розгалуження)


Colored output is implemented


Might be able to make this is a single query by using `Q(layer_tags__name__icontains=value)`.


Тут у вас поповзли відступи, виправте.


Hi @MPagel sorry I'm just getting around to this.  Please feel free to send a pull request through GitHub.


І тут теж.


sure


Fixes #115


If __ name__ == '__ main__' needs to be mentioned in main. Prevents from running on imports.


gradle build was failing at this line. [Commented now]


Consider using `self.logged_in_user`.


Add comments here like the one on Scala implementation
Example : **get censorcol value**


It booted on CM with this configuration. But it doesn't boot on stock with a modified kernel. There's something missing in the defconfig. Motorola botched this one.


same here


I cover that a little in the automation section, or you use a tool that supports direct inclusion of AWS or Azure via API. Check with your vendor...


I played with it for a while and yes, this is better (it's slightly slower, but has lower memory footprint). (I'd expect the opposite)


Why is this file required ?


Won't the 2.* use the latest one automatically?


Had to manually build this due to the "new" build server being fixed :P


Hey what name do you want to use for the package? Mine is hosted as Java_8_Update_60.pkg


This should go in the `init` task in tasks.py


Note that you will probably also want to handle
- `rgba` brackets
- `hsl`/`hsla` brackets


This should be two separate imports.


Як я вже казав, у цілому рішення приймається. Проте його можна спростити ще далі. Позбутися `step`у, а натомість звужувати саму вилку `[rangeMin, rangeMax]` — тобто, кожної ітерації підсувувати якусь із цих змінних ближче до іншої на половину шляху


I also found it easier to arrange the page with Pure.CSS. However, I did find the site more or less confusing because there was not enough examples of how to place the CSS within the html. I had to do a lot of experimenting, which you said you did with the first part of the assignment. I think once you know what you're doing with CSS, Pure.CSS would be very helpful.


I doubt its source related, i have a pretty big feeling something is wrong with the dt you used, or the ramdisk since a friend of mine boot this kernel fine on stock with the defconfig used before this one^


Why do we need this as well as PULSE_DATA_INGESTION_CONFIG?


Apropos: I see you are using my `if True:` trick.
Just as a note: it is a corner case that works on python 2 and 3, although we do not use `from __future__ import (a lot)`. This would be very ugly because we would create a huge line that ends in an `if`. But because we only want to print, and tuple braces around the one argument to print are ignored by python2, this works.
(It looks cleaner than it is...)


It's not working. "substring not found" error on br Title line.


yes, but imo better to pin it to a version forever bc latest is mutable


Could also use target_link_libraries here. (which qt5_use_modules does anyway)


This fails to even bind.  Address (below this diff) defines the map-key as `@MapKeyEnumerated(EnumType.STRING)` which is obviously wrong here.

Please show me a complete listing that reproduces this problem


Szerintem mehet a release az lxqt és MATE isókkal egyetemben, és jó lenne eztán egy kis hírverés valahol, ha jónak bizonyul, hogy többen kipróbálhassák.


This is my last one :)


@orbeckst Cython is needed for PyTables, but not for MDAnalysis. I admit I lumped my grumbling for both in one commit without specifying which one had which needs.


I had very little to none experience with CSS before this assignment so I also started out as a beginner. I think what makes Duckett's chapters so helpful is the detail the book goes into on each little subject such as boxes. It gives several examples and different ways to use it.


What is "STDOUT", you meant sout - printing into console ?


There's no periodic celery task scheduled at the moment, so this worker isn't going to do anything. Should we wait to add it until we have a periodic task?


Ooops... for any reason the DirectX9 Dependency file is includes twice.
This is not a big issue, but it is also not nice. I will fix it together with the next release...


Yeah, it inject dependencies to have actions testable.


The `try` and the `if` here are redundant.


Can you please add a comment saying that this index is for deleting all the messages of a particular tenant when the tenant is deleted from BOP?


This needed a lot of massaging to get working on OSX Yosemite for me.  Once I got `diff-highlight` in my path,
```bash
function strip_diff_leading_symbols(){
	color_code_regex=$'(\x1B\\[([0-9]{1,2}(;[0-9]{1,2})?)?[m|K])'

        # simplify the unified patch diff header
        sed -E "s/^($color_code_regex)diff --git .*$//g" | \
               sed -E "s/^($color_code_regex)index .*$/\
\1$(rule)/g" | \
               sed -E "s/^($color_code_regex)\+\+\+(.*)$/\1\+\+\+\5\\
\1$(rule)/g" | \

        # actually strips the leading symbols
               sed -E "s/^($color_code_regex)[\+\-]/\1 /g"
}

## Print a horizontal rule
rule () {
        printf "%$(tput cols)s\n"|tr " " "─"
}
```

The line breaks are escaped and intentional.  But they're certainly not pretty!  I'm sure someone can improve this.


- do we need to really use apache io library ? [ Can use java.nio.file for file operations. Or we will have some use later ? ]
- Pls  include a brief summary for checkins.


Why was deleted "cached"?
This is a very useful metric.


@jorgebay please see https://gist.github.com/cjihrig/54b874213dd8367529fd


fast-forwarding hotfix/2.0.6-2012.08.25-solrq-per-index-updates+fuses to borshop-integration-563-better_list_comprehension_and_dry = e525e1a7


I think the Pure.CSS website was not as detailed as I wanted it to be. There was a very small section on changing the font in the webpage with Pure.CSS, but I found it easier to just customize through regular CSS. I decided to do Pure.CSS based on the simplicity of the website. I am not sure about the differences between Bootstrap and Pure.CSS, but if I knew there was more stylistic design choices that could be made using Bootstrap I may have picked that as my framework instead.


assert() calls are gone. Replaced with SALDL_ASSERT() macro. 7b1ff31c78c7c8f8e6f33815a7a0bb4c5d320a45 and  8047d4b8093b532b6fee740a0eb29d4896f294de


There's a stray trailing newline here and in some of the other docstrings.


I'm not sure.
But shouldn't be rule 100 in apt.h be:
Result APT_GetAppletInfo(Handle* handle, NS_APPID appID, u64* pProgramID, u8* pMediaType, u8* pRegistered, u8* pLoadState, u32* pAttributes);
instead of:
Result APT_GetAppletInfo(Handle* handle, NS_APPID appID);

I tried to build the latest version, and by doing this i fixed the errors.


try to set Boost_DEBUG to True and rerun, then you will see where and what boost searches for


okay, then I'll have to rewrite the tests.


userid can be extracted from PULSE_DATA_INGESTION_CONFIG


Is this important now that #1240 has been merged. There, @sgotti has decoupled the ACI images from the pods.


Double slash in the help text


that was fixed in https://github.com/smealum/ctrulib/commit/56b56955c0eb00d7556d15a099747a93fd0c1f0a


"substring not found" error still showing on Title line.


Sorry for my lack of understanding, but is "this" yours or my version? I'm happy to go with the one that performs better in your opinion and tests.


Please remove the additional newlines here and below :-)


This is something I wanted to do with my webpage, but I could not figure out if you could do it on Pure.CSS or not. I checked out your webpage and see you did what I wanted to do with the two paragraphs next to each other but when you minimize the browser more they go on top of each other. Mine ended up overlapping when you close in the browser.


Yes, saw it and directly decided not to reinvent the wheel. I was sure
there is a good reason for this :)


Since I never did this quest, I don't know how close (or far) this patch is from the retail version.
Maybe can @TheTrueAnimal or @scotty0100 provide some feedback on the matter, please?

Also poke @Tobschinski in case he has the ID of the two handed sword NPC 9398 is supposed to wield, hidden in his data.


redid the way the AssignStatus enum was handled. Works better now I think.
Also added a function to return the Status as a string value.


Should be able to share the kernel with zeroltetmo ultimately i would think. In the end, all S6 variants should be able to share the zero-common repo.


Additional newlines can be removed :-)


The previous server wasn't being closed or what?


I am wondering if this should be defined in `RigidMotion`, i.e., `RigidMotion::Transformation`.


Why is this public?


Now it'll finally stop judging me for standing on it!


This looks like a reasonable timeline.


Aquí tienes un conflicto que no has resuelto. Recuerda que dejar esto provocará un crash. Para cada conflicto elije una de las partes (HEAD o remotes/upstream/master que están separadas por =======) y borra la otra. En este caso quédate con HEAD.


Platofrm :open_mouth:


Pushed update to Journal


ок


Ah, yes, we definitely don't want this. And unfortunately, there's not a good way around this: PhpUnit combines our message with its "Failed asserting" message into the Exception (it doesn't allow us to fetch one independent from the other in the Exception that it throws). So, I guess we should stop using these.

If there are some that are convenient, we can add them to the CodingExecutionResult


test


This design looks more promising. I find it interesting that this macro addition was applied only after another intermediate source code change.


No.


that doesn't work in Python3. I've got a very simple fix coming that handles both Python 2 & 3


Done.


Good call.


@shane-tomlinson does not look right, wrong branch?


Confirmed the location had changed


**str** will change the **int** into string


rax retry tests


@manipalsingh013 the files uploaded in github have a different name. ALL the files needed are present


Agreed. Let's move it to an integration test.


Nope.


For the initial load, what will the series of requests be?

Thinking through it, it seems like it should be something like you first get ALL contacts (with page=0), then after that you do before= the last contact returned by the previous call.

After that you just do after= with whatever the time that you started the entire fetch was.


We cannot change in string type: cannot do a[0: 2] = b[0: 2]


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


TeamCity pull requests :: netty [Build 325](https://garage.netty.io/teamcity/viewLog.html?buildId=1797&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5610, ignored: 21 Build time: 00:38:10


s.replace(stra, strb) returns a version of string s where **all** instances of stra have been replaced by strb.


I think we should be able to remove this checkstyle plugin since it would inherit the plugin from the top-level pom


Prime III


Same comment for the contrib-check profile, should be able to remove this and inherit from top-level pom


Isn't this relating to `hash` field below?


@ChaksChigurupati : done.


Do you plan to leave in these many TODOs?


done

Date: Mon, 28 Sep 2015 10:05:01 -0700
From: notifications@github.com
To: SoulMeet@noreply.github.com
CC: manuel.buerge@hotmail.com
Subject: Re: [SoulMeet] Removed Dob from register activity. Its an unconventional requirement for a registration (0777c45)

https://github.com/SteveMcGregor/SoulMeet/wiki/RegistrationModel-JSON

It is a requirement of the domain. Which means it has to be committed when creating the account in the api.

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


Why does this use a `map` and the others `repeated KeyValue name = N;` Aren't they equivalent?


dodalem OrderedDict :)


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


Though, ttgmichael's wiki says Chupaluka.


Prueba Initial commi


Ohh, it really make sense!!
Just get a little "dzen" :D


So... a missing `\0` that was causing trouble on my machine :angry:


fixed: https://github.com/owncloud/documentation/issues/1702


Fix für Probleme Mit dem Website Manager https://github.com/joachimdoerr/mform/issues/58


should I set

(Boost_DEBUG ON)

somewhere in the CMakelist.txt, or what would you like me to do?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


OK, so far so good. Now, to the test.


Looks good, but most of this change isn't what the commit message says!  Maybe just amend commit msg to mention logging rework and in in-decl inits?


Update instead of deleting the record.


@otoolep shouldn't this be rc${rc} ?


Currently the `sleep` function does 2 things:

- Calculate the sleep duration
- Pass it to `time.Sleep`

The decomposition is to yank out the duration calculation and then just call `time.Sleep` directly here.

If you move to accepting a `Duration` from the user, you won't even need the calculation bit any more, though. :)


seems to be as if. here is the log

 https://bpaste.net/show/267eca84722e


FWIW, My benchmarks show that xorshift library ~2x as slow as my pcgr library.  I belive it's due to the fact that the call to r.Next() is an indirect call via an interface.

```
<dgryski@kaepora[go-pcgr] \ʕ◔ϖ◔ʔ/ > go test -test.bench=.
PASS
BenchmarkPCGR-4    	300000000	         4.70 ns/op
BenchmarkRand-4    	100000000	        18.4 ns/op
BenchmarkXorshift-4	200000000	        10.2 ns/op
ok  	github.com/dgryski/go-pcgr	6.779s
```


Oh... I like this. Had some plans to read the rebar.config and also erlang.mk.


@taylordowns2000, turns out it was the `ng-include` causing the problem. Now [resolved] (https://github.com/OpenFn/OpenFn-Site/commit/2f7eb5cc3dd556669fa1fc3089cd6fc1cc68f49f). Let me know if there is anything else you'd like me to look at.


Löst das bekannte Login Problem https://github.com/joachimdoerr/mform/issues/32


(You'll still need a calculation because the sleep duration is randomized. Derp.)


Would it simplify things to have one field called "Document Id" that supports expression language? It seems like that would still support the static vs dynamic case. You would just call

    context.getProperty(DOC_ID).evaluateAttributeExpressions(flowFile).getValue();

and if the property was a static value you get that, if it has expression language it would evaluate that.


maybe `valuesAreUnique`


> Hmm, maybe something else broke the toggle then, because I'm not sure its working now.

Appears to be an issue with opal-jquery, as I'm getting a `NameError: uninitialized constant Object::Document` in my console. Any changes to the loading order in application.js.rb or anything like that?


Nice option! Thanks :)


this is not clear to me


I needed a 64 bit random

—
Sent from Mailbox

On Mon, Sep 28, 2015 at 9:28 PM, Damian Gryski <notifications@github.com>
wrote:

> FWIW, My benchmarks show that xorshift library ~2x as slow as my pcgr library.  I belive it's due to the fact that the call to r.Next() is an indirect call via an interface.
> ```
> <dgryski@kaepora[go-pcgr] \ʕ◔ϖ◔ʔ/ > go test -test.bench=.
> PASS
> BenchmarkPCGR-4    	300000000	         4.70 ns/op
> BenchmarkRand-4    	100000000	        18.4 ns/op
> BenchmarkXorshift-4	200000000	        10.2 ns/op
> ok  	github.com/dgryski/go-pcgr	6.779s
> ```
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/seiflotfy/count-min-log/commit/2e7f2fb42df58557ba8fea561d5ce1860496c335#commitcomment-13476873


Esto es un comentario :v


Your email went to the wrong person.
I don't know anything about base16-tomorrow-dark-theme.

On Sat, Sep 26, 2015 at 11:27 AM, simurai <notifications@github.com> wrote:
> :art: Rename @dark-red -> @brown
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/atom/base16-tomorrow-dark-theme/commit/93878f286e096a13645b14392179a8aba7064dd8


========== Build: 6 succeeded, 0 failed, 6 up-to-date, 0 skipped ==========


TeamCity pull requests :: netty-tcnative [Build 8](https://garage.netty.io/teamcity/viewLog.html?buildId=1799&buildTypeId=pulls_netty_tcnative&guest=1) is now running


TeamCity pull requests :: netty [Build 324](https://garage.netty.io/teamcity/viewLog.html?buildId=1796&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5619, ignored: 21 Build time: 01:11:54


"describes language" - right, I wrote it a lot ago. Actual :
https://github.com/BarredKnight/TD/blob/building-up-%230/src/com/td/model/Word.java
About Words and Text. They have different purpose. Words contains only words, instead Text contain all specter of characters - like commons, dots, !, ", - and etc. Also in future Words wouldn't contain duplicates.


Reading the commit message I'm not sure you understand exactly what is the behaviour.

I think the logic is that only one swapchain can be fullscreen.
Since via a Reset (or ResetEx) the device swapchain can be put fullscreen from windowed mode,
then they enforce the 'max one is fullscreen' by disallowing any additional swapchain to be fullscreen when created (if I remember well the mode of additional swapchains can't be changed).
Your patch implements that behaviour.

However the comment and the commit message say that as long as there is no swapchain yet in fullscreen mode (the device one in windowed mode for example), then you can create an additional swapchain in fullscreen mode.
This is not what you implement, and as Reset doesn't have failure cases to handle when an additional swapchain is fullscreen (then preventing the device one to switch to fullscreen), I believe this is not the expected behaviour.

Am I wrong ?

Can you match your comment and commit message to the correct behaviour


So the key is we never want to page.. because that is too slow.. so we never want to do a call like /contacts&page=10000

Instead, we want to use before & after to do that, especially on the first load.

So on the first load, you can use &before, which starts at now, and then gets earlier according to the last contact you see in the result.. that way you always stay at page 0.

Make sense?


TeamCity pull requests :: netty-tcnative [Build 8](https://garage.netty.io/teamcity/viewLog.html?buildId=1799&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:57


Needs comment `# bintray.com is the official download host per the vendor homepage` before the `url`.


decodeURIComponent 함수를 추가함으로써 한글태그도 검색 되도록 수정


Sí, yo creo que mi problema, entre otras cosas, es que empecé a programar sin tener cuáles son las reglas de para jungar los enclíticos con la forma verbal.
Al prinicipio la cosa efectivamente se planteó como algo parecido a lo que dices tú: identificar la forma verbal y ver si la última sílaba o las dos últimas sílabas son enclíticos y si son dos asegurarse de que el orden es OI- OD.
Pero cuando empecé a programar me dí cuenta de que esto solo era la punta del iceberg y de que la mayoría de las combinaciones de dos enclíticos nunca se pueden dar por razones muy variadas, que en algunas combinaciones el orden es OD-OI, que el complemento indirecto se puede confundir con el pronombre reflexivo, impersonal, dativo ético, que en algunas casos sí puede haber ambiguedad y en otros no etc. etc.
Además, luego me dí cuenta de que los resultados no debían ir por palabra si no por cada posible combinación forma verbal + enclíticos (llamé a estas combinaciones 'estructuras' aunque el nombre es muy explícito y lo tendría que cambiar. Por ejemplo, en una palabra como 'vélale' tenemos dos combinaciones: imperativo del verbo 'ir' o 'ver' con una combinación, incorrecta, de dos enclíticos, por un lado, y el verbo velar + un enclítico. Y el análisis lo tenemos que hacer para cada una de las combinaciones...

Por eso digo que si yo hubiera planteado un test para la primera versión de mi código, me hubiera basado en que el resultado tendría que ser parecido a tu json. Y este test no me hubiera valido para nada para el código que tengo ahora.
Tal y como tengo el código ahora ya se acerca más a lo que podría ser un detector de enclíticos (aunque todavía no he encontrado la arquitectura ideal y sigue siendo bastante desordenado) por eso ahora sí se puede diseñar la mayoría de los tests, sabiendo que habrá que revisarlos cada vez que se integre una regla nueva para las formas reflexivas, las combinaciones de 3 enclíticos y a lo mejor alguna más.

Vas a estar el miércoles? Es que te quería hacer un par de preguntas y creo que hacerlo en persona va a ser más fácil que por escrito.


nit-picking here since this will end up in the documentation - spelling of Expression


decodeURIComponent 함수를 추가함으로써 한글태그도 검색 되도록 수정 하였습니다


How big is dataSize?  It appears you are using you are finding all the points in the map and then setting the MPlotSimpleImageData to that.  For instance, if you have 50 scans and each scan is 120 points, then you would put in

```c++
data2D_ = new MPlotSimpleImageData(50, 120);
```
not,
```c++
data2D_ = new MPlotSimpleImageData(600, 600);
```.

That's just my guess.


Variable name does not describe the use of the variable


Could we possibly have different versions of `cpp_rockbot.pro` as separate files? Like say `cpp_rockbot.pro.linux` and `cpp_rockbot.pro.windows`. I just created a PR to fix the typo `cpp_rockbot.pro` in `.gitignore`. We can avoid merge conflicts in the future if we can stop tracking `cpp_rockbot.pro`, since this file [gets changed frequently](https://github.com/protoman/rockbot/commits/master/cpp_rockbot.pro).


Needs comment `# bintray.com is the official download host per the vendor homepage`.


saw approval from fizzy123
at https://github.com/fizzy123/test-repo/commit/929b0d07c6f702498cfd07c4314e9d2966b1047c


merging fizzy123/test-repo/test2 = 929b0d07 into testing


fizzy123/test-repo/test2 = 929b0d07 merged ok, testing candidate = 658b590d


saw approval from fizzy123
at https://github.com/fizzy123/test-repo/commit/589ad8eddfc8830aeccfe6657c2142d6d501fc30


merging fizzy123/test-repo/test1 = 589ad8ed into testing


fizzy123/test-repo/test1 = 589ad8ed merged ok, testing candidate = a53ca951


@mstembera: that was an interesting experiment, thanks for doing it.


Annoyingly, it looks like `newSerialNumber` has been deprecated in favour of `newSerialNumberString`, at least according to [Apple's docs](https://developer.apple.com/library/mac/documentation/Kernel/Reference/IOHIDDevice_reference/#//apple_ref/cpp/instm/IOHIDDevice/newSerialNumber/OSNumber) and the [latest IOHIDDevice.cpp source](https://opensource.apple.com/source/IOHIDFamily/IOHIDFamily-606.40.1/IOHIDFamily/IOHIDDevice.cpp) I could find.


unused import Person


The comment doesn't make sense unless you add that we should return D3DERR_DEVICELOST "if the device is lost".

I think it is ok to drop this patch.


OK, I was able to adapt `avocado-vt` by simply adding the `extra_params` to the plugin's constructor:

    diff --git a/avocado/core/plugins/vt.py b/avocado/core/plugins/vt.py
    index 264efe9..01dc4d2 100644
    --- a/avocado/core/plugins/vt.py
    +++ b/avocado/core/plugins/vt.py
    @@ -155,8 +155,8 @@ class VirtTestLoader(loader.TestLoader):

         name = 'vt'

    -    def __init__(self, args):
    -        super(VirtTestLoader, self).__init__(args)
    +    def __init__(self, args, extra_params):
    +        super(VirtTestLoader, self).__init__(args, extra_params)
             self._fill_optional_args()

         def _fill_optional_args(self):

And I can run the tests so far. Nice.


Hi,

At the point I made these changes we were still running an older version of Sequelize that didn't properly support json comparisons. We're now on 3.9 so you're probably right that this isn't necessary anymore.


import java.util.Random;
import java.util.Scanner;

public class BattleShip {

	public static void main(String [ ] args){
		int [ ] [ ] board = new int [ 5 ][ 5 ];
		int [ ] [ ] ships = new int [ 3 ][ 2 ];
		int [ ] shoot = new int [ 2 ];
		int shotHit = 0;

		initBoard(board);
		initShip(ships);

		do{

			showBoard(board);
			shoot(shoot);

			if (shotHit (shoot,ships) ) {
                shotHit++;
            }

            changeBoard(shoot, ships, board);

		} while(shotHit != 3);

		System.out.println(" Game over! You win! " );

		showBoard(board);

	}

	public static void initBoard (int[][] board){

		for ( int i = 0; i < 5; i++ ){
			for ( int j = 0; j < 5; j++ ){
				board[ i ][ j ] = -1;
			}
		}
	}

	public static void showBoard (int[][] board){
		System.out.println();
		for ( int i = 0; i < 5; i++ ){
			for ( int j = 0; j < 5; j++ ){
				if( board[ i ][ j ] == -1 ){
                    System.out.print(" \t " + " O ");
                }

                else if(board[ i ][ j ] == 0 ){
                    System.out.print(" \t " + " X ");
                }

                else if(board[ i ][ j ] == 1 ){
                    System.out.print(" \t "+ " ! ");
                }
			}
			System.out.println();

		}
	}

	public static void initShip (int[][] ships){

		Random random = new Random( );

		for ( int i = 0 ; i < 3 ; i++ ){
			ships[ i ][ 0 ]=random.nextInt(5);
			ships[ i ][ 1 ]=random.nextInt(5);

			for ( int j = 0 ; j < i ; j++){
				if ( (ships[ i ][ 0 ] == ships[ j ][0] )&&( ships[ i ][ 1 ] == ships[ j ][ 1 ] ) ){
					do{
						ships[ i ][ 0 ]=random.nextInt(5);
						ships[ i ][ 1 ]=random.nextInt(5);
					} while ( ( ships[ i ][ 0 ] == ships[ j ][ 0 ] )&&( ships[ i ][ 1 ] == ships[ j ][ 1 ] ) );
				}
			}
		}
	}

	public static void shoot (int[] shot){

		Scanner input = new Scanner(System.in);

		System.out.println( "Row: ");
        shot[ 0 ] = input.nextInt( );
        shot[ 0 ]--;

        System.out.println(" Column: ");
        shot[ 1 ] = input.nextInt( );
        shot[ 1 ]--;
	}

	public static boolean shotHit (int[] shot, int[][] ships){

		for ( int i = 0; i < ships.length; i++ ){
       			if ( shot[0] == ships[ i ][ 0 ] && shot[1] == ships[ i ][ 1 ]){
					System.out.println(" You hit a ship located in "  + (shot[ 0 ]+1) + " , " + (shot[ 1 ]+1) );
						return true;
				}
        }

        return false;
	}

	public static void changeBoard (int[] shot, int[][] ships, int[][] board){
		if (shotHit( shot,ships ) ){
            board[shot[ 0 ] ][ shot[ 1 ] ] = 1;
        }
        else {
            board[ shot [ 0 ] ][ shot[ 1 ] ] = 0;
        }
	}
}


pep8 - Should leave a space after "#"


There's no need for the trailing `$`, it will always be ignored.


decodeURIComponent 를 추가하여 한글검색이 되도록 하였습니다.


Added in pull request now.


Is this URL correct?


Nope, but `twitter-bootswatch-rails` tries to make an `application.js` when you run `rails g bootswatch:install` -- maybe it expects a different load order?


Makes sense.  And yeah, I should add one of the 64-bit variants to my pgcr code..


mange tak


Hi, can you please open this as an issue? Thanks


I dont like GRADLE here


Import Error
Unable to import 'Person'. (Fatal Error)


I'd rather like if this just queues the plane activities then.


Old-style class defined. (old-style-class)


LGTM


Double slash in the URL


This doesn't conflict with my patch https://github.com/lmr/avocado/commit/a0532382aa9686cf864cc0b8862a51bf8f7ac69f. This patch is fine.


Modules could have been implemented better


yep, we fetch from new to older one page at a time always pass &page=0


@palamago Este cambio rompió el template que se carga en la parte superior ('.context-menu'). Podrias revisarlo?


comment from js


Closes #12


I reverted it in https://github.com/knpuniversity/oo/commit/ca8643864437a8fd8b1fd4369f4664706352740a


OOP concept, Polymorphism, has not been implemented


randapplex,randappley are not used at all.


Inheritance Implemented, incorrect comment.


That's fine.


It might be helpful to mention in the CapailityDescription that this processor can be triggered by an incoming FlowFile, or it can be scheduled on a timer, just so that the documentation is clear for the end users.


They were used in the example #3 in http://php.net/manual/en/function.crypt.php
But I can remove it


So go test it or go fix it :)


Which color and which kind is the symbol?


Modularity Implemented, incorrect comment.


nope.. it's not.. thanks.. I will fix it


PEP-8 Standard Violation
Invalid class name "fireballs"


Por favor quien arregle esto, que lo haga creando un branch desde master y pidiendo pull request asi queda como bugfix de la version en producción. Gracias !


Code not duplicated, requires multiple imports, incorrect comment.


Class has no __init__ method (no-init)


Seems to be working fine.


@i2plover I realize why you've done it. But I also think there's a difference between dev and publishing.

All the tools we are using are either Apache, or MIT, but both of those licenses **require** that we bundle the licenses and the copyright notices. Which I've seen you've only done with Material Design Icons.

When it comes to the images, I think we should replace them with images that are legal to use. I have some images that we can use, but these ones definitely need to go (And preferably be scrubbed from the git history)

I also suggest using a different repo, (perhaps a fork of Shimmie?) for the theme and php port itself. As this repo was mainly meant for the design portion of the job.

I have added you as a collaborator to my fork of Shimmie if you'd like to do some of the php work there.


There are too many mixed indentations, trailing whitespaces, missing docstrings


left and right movements are not implemented with a,d.


Code Smell: Too many local variables, without reference to there function.


TALKING ABOUT BROSCIENCE TAKEN TOO FAR


up,down movements are not implemented with w,s.


Spelling of Expression


Warning, Wildcard import main


@aancel @prudhomm [CMAKE_CXX_COMPILER_VERSION](https://cmake.org/cmake/help/v3.3/variable/CMAKE_LANG_COMPILER_VERSION.html#variable:CMAKE_<LANG>_COMPILER_VERSION) is not a reliable variable. It generates error with clang/travis (at least)


@taniaka , **¡el jueves!** Se ha movido al jueves, que el miércoles es el día del traductor y han organizado un Lengua & Té. Y sí, el jueves estaré, y también el sábado.

La verdad es que la complejidad del tema de los enclíticos me ha parece brutal, tengo ganas realmente de que me lo contéis con calma porque a mí desde mi perspectiva ingenieril me parecía algo mucho más sencillo (¡pero la lengua siempre sorprende y escapa a nuestro control!)

Cuando nos veamos planteamos algunas cosas sobre papel, que yo también tengo muchas ganas de entender cómo lo estás enfocando y cuáles son esas dudas que te surgen.


more_doc_changes (mmatts) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Isn't it Ivy?


This call does only fetch 20 categories.


Code Smell: Too many if-conditions, can be written in lesser statements.
Should be avoided lesser code length.


if we want to change the the game slightly also we have to change the values at so many places.it is bad practice to hardcode.


PEP-8 Violation Line too long


new line here


new line


new line o nevery new function


PEP-8 Violation Exactly one space required after comma


also swapped to a map instead of a list of tuples for shared strings


jg


Code Smell: Code repition. Shoild be avoided (maybe by using method a new method)


Hi


not necessary, use CategoryTree


not necessary, supported by category tree


PEP-8 violation, missing class docstring


Code Smell: Too much hardcoding (Bad practice)


y como hace la implementación actual? :)


I see, thanks!


Sunrise Dmexco has a cool implementation that fetches ALL categories and not the first 20. It also can refresh the tree:

https://github.com/sphereio/sphere-sunrise-dmexco/blob/59283361da7bf48b4170674f32e27d4dd2b08e1b/app/categories/RefreshableCategoryTree.java


Invalid argument name "x"


PEP-8 violation: Extra whitespaces


is it 4 MBytes or 4MBits=512Kbytes?


Multilevel not implemented


@Optimusprimeums thanks!

@Nadahar our versioning doesn't really work like that, we just increment based on the feature changes. So if there's a change we think needs to be released for Java 6 it would just be the next increment. The exception is if we have alphas or betas running between major versions

I think there are probably still a lot of people stuck on Java 7 for whatever reason so it will be important to release versions for them for a while longer. I wonder if there are reliable statistics on that.


Restored priority to 0

Please don't touch it. Thank you.

Result is looking good and torture test passed, so we are going to apply
this patch.

On Mon, Sep 28, 2015 at 5:38 PM, VoyagerOne <notifications@github.com>
wrote:

> Changed Priority to -1. As the results seems inconclusive and don't want
> to burn through resources...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/f91639251802eeb7c07907cb316fbc8780e79d70#commitcomment-13471880>
> .
>


mejora pero sigue sin ser el formato del template de firmware :(


un comentario de que hace cada macro no seria nada malo :)


Global variables used, leads to loss of encapsulation.


Hola sadsadasd


no deberia comenzar todo con USB_... va solo una consulta


I think this should not be CompletableFuture but CompletionStage, since you only should consume values


Who I? What have I done?
Ha ha ... Please ...


este enumeracion no se usa? ya que no tiene nombre


Encapsulation lost due to use of Global Variables


estas mezclando definiciones de enumeraciones y macros... :(


esto iria en un archivo de configuracion?


Please do. I also noticed the difference

—
Sent from Mailbox

On Mon, Sep 28, 2015 at 9:38 PM, Damian Gryski <notifications@github.com>
wrote:

> Makes sense.  And yeah, I should add one of the 64-bit variants to my pgcr code..
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/seiflotfy/count-min-log/commit/2e7f2fb42df58557ba8fea561d5ce1860496c335#commitcomment-13477079


1) Basically because we want to avoid re-creating temporary directories. `utils_path.init_dir` will re-create the entire directory structure even if `data_dir.get_tmp_dir()` gets deleted in the meantime (which could happen should anyone try to put a cleanup function that runs before the test class is being initialized).
2) See 1)
3) By handling it at the data_dir module there's no guarantee that the temp dir won't be deleted later by some cleanup code inserted incorrectly in the code. The best time to check for it is right before `utils_path.init_dir` is called.


It may be useful to actually point to the issue as someone checking if 2.6.2 fixes there issue might find that thebug report is the same as their.  Otherwise they simply don't know.


TeamCity pull requests :: netty [Build 326](https://garage.netty.io/teamcity/viewLog.html?buildId=1798&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5619, ignored: 21; resource leak detected (new) Build time: 00:37:43


So with this I'm ensuring the base temporary directory will not be re-created, which is one of the points of the original card that originated this PR (temporary directories should never be re-created, and yadda yadda).


Bad UI, the grid gets printed twice,Modularity not implemented


Good call.


Probably should not add this. There is a memory problem somewhere here.


Uh, did you test this? The use of `#{...}` in [my comment](https://github.com/geometalab/osmaxx/pull/208#discussion_r40444228) was Ruby-inspired. It seems like [Python's `str.format()`](https://docs.python.org/3.4/library/stdtypes.html#str.format) uses [curly braces without the `#`](https://docs.python.org/3.4/library/string.html#formatstrings).


If you strongly believe this assert is wrong, I'll remove it and we can call it a day.


toast not implemented


Looks like all that work payed off.  I do seem to be getting time losses at fast time controls and 7 threads.  Not a big deal, but does make it hard to measure strength.


@vladikoff - yep, I merged the branch which was based off of a post-46 master instead of using cherry-pick. New tag created https://github.com/mozilla/fxa-content-server/releases/tag/v0.46.1


Don't you see something wrong ?

<option data-i18n="Power day graph value="1">Power day graph</option>


It is better to have just one .pro file as they share a good part in
common. If you want to changed between types, you can use qmake in command
line passing it as a parameter.

Also, it should not be in gitgnore, as this is our real makefile and yes,
we need to track it.

--
Iuri Fiedoruk, Software Developer

On Mon, Sep 28, 2015 at 4:33 PM, Farley Knight <notifications@github.com>
wrote:

> Could we possibly have different versions of cpp_rockbot.pro as separate
> files? Like say cpp_rockbot.pro.linux and cpp_rockbot.pro.windows. I just
> created a PR to fix the typo cpp_rockbot.pro in .gitignore. We can avoid
> merge conflicts in the future if we can stop tracking cpp_rockbot.pro,
> since this file gets changed frequently
> <https://github.com/protoman/rockbot/commits/master/cpp_rockbot.pro>.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/protoman/rockbot/commit/5f1e70707a4eaff6ee380d59d1b2c13bf10e4197#commitcomment-13476985>
> .
>


третий параметр - обязательный?


Just thinking here, are there any exceptions that we would want to handle differently than others? I'm thinking of the difference between an exception like an invalid document vs. an exception where the client couldn't communicate with the couchbase server. In the first case you would probably just route to failure, in the second case you might want to have a retry relationship.


@mbootsector
Just curious...why did you reference history but not cmh with th->?


`des` should fall here as well, since it uses `crypt()`. I'd rewrite this block somewhat like this:
```
                   if(empty($salt)) {
                     if($cryptscheme === 'sha512') {
                       $salt = '$6$';
                     } elseif($cryptscheme === 'bcrypt') {
                       $salt = '$2a$10$';
                     } elseif($cryptscheme === 'md5') {
                       $salt = '$1$';
                     } elseif($cryptscheme === 'des') {
                       $salt = '';
                     } elseif(preg_match('/\$[:digit:][:alnum:]?\$/', $cryptscheme)) {
                      $salt = $cryptscheme;
                     } else {
                      die(_('The value of $cryptscheme is invalid!'));
                     }
                     $salt .= get_random_bytes(CRYPT_SALT_LENGTH);
                   }
                   $cryptedpass = crypt($clear, $salt);
```

It could even be turned into a `switch` with the last two cases merged into the default case with an additional if.


I love this change :)  :+1:


saw approval from fizzy123
at https://github.com/fizzy123/test-repo/commit/bdcbe42601d75a85230e902d33db7d13a2e51d43


merging fizzy123/test-repo/test1 = bdcbe426 into testing


fizzy123/test-repo/test1 = bdcbe426 merged ok, testing candidate = 32269b5a


saw approval from fizzy123
at https://github.com/fizzy123/test-repo/commit/b1a922b366e4e8d74500e0bf23b848039aa592a0


merging fizzy123/test-repo/test2 = b1a922b3 into testing


fizzy123/test-repo/test2 = b1a922b3 merged ok, testing candidate = 87710882


yay!


the `verifyResolutionFails` check should be here, else the `AddName` could be causing the resolution to work, and the republishing not be happening at all.


Code Smell :Method-Level Smell :  Long method
The method is very long , should break up the method into more funtionalities


Yeah, thanx, I just read on framework.zend.com that it is 5.3+,
Than I went to "get started with skeleton..." and there were no mention
about 5.5+ as well as on git mainpage... so...
Ok, pity, there's no way to download 5.3-compatible skeleton...
Sorry for issue in that case!

2015-09-28 4:53 GMT+03:00 Marco Pivetta <notifications@github.com>:

> @Jokersoft <https://github.com/Jokersoft> the requirement to run the
> skeleton is now PHP 5.5. Please note that existing apps are unaffected by
> this change.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zendframework/ZendSkeletonApplication/commit/6ba430f6a2006d9f37c24b039fda3488e8681c91#commitcomment-13460150>
> .
>


Tem que remover


It looks like `pick_up_secondary` prior to your change only uses `SOrderList(255)` once, and then only when another conditional is satisfied.

```c++
		if (!(Controls.state.fire_secondary && PlayerCfg.NoFireAutoselect) &&
			weapon_order < SOrderList(255) &&
```

I probably did not copy `SOrderList(255)` to a temporary since (1) it may not be used if the first part of the if() is false and (2) it is used at most once, so there is no need to cache it and no code size savings from eliminating extra calls.


@Jokersoft which page? That needs updating: versions of zendframework below 2.5.0 are PHP 5.3 compatible.


right now after my try that error gone and another error show

drivers/built-in.o: In function `mon_recv_decrypted':
mon.c:(.text+0x2849a3): undefined reference to `ieee80211_hdrlen'
Makefile:916: recipe for target 'vmlinux' failed


This is causing us grief as well. Any input element in a parent container which has an ng-click directive is now unclickable on mobile.


Not needed. https://nodesource.com/blog/semver-tilde-and-caret


Keh
On Sep 28, 2015 7:28 AM, "srisurya95" <notifications@github.com> wrote:

> @akhilnarang <https://github.com/akhilnarang> :3
> Favicon generator
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/srisurya95/srisurya95.github.io/commit/171a52846fd07902ff1e5d494700cc706b3cab8c#commitcomment-13460176>
> .
>


Thought I did that, but apparently not. Done now.


Code Smell : Method Level Smell : Excessively long identifier: reachPaulineWin


Code Smell : Method Level Smell : Excessively long identifier : checkTrailCollision


Code Smell : Method Level Smell : Excessively long identifier applyGroundForce


Be nice to add the other possible props here with some reasonable defaults. Would make it nice and clear to read & use the component.


Unused import sys


Missing end of line at the end of the file


Bad whitespace error


this test


It is kind of inverse - I did NOT meet any warning about <=5.5
incompatibility, until I opened composer config.

But ok, let us go step by step
First obvious way you go when on zend mainpage
1) http://framework.zend.com/downloads;
2) There you see "Get the Skeleton Application", you click it;
3) http://framework.zend.com/downloads/skeleton-app - no mentioning about
version;
3.1) if you try "http://framework.zend.com/about/faq/" - "Zend Framework 2
is based on PHP 5.3. We expect to add support for PHP 5.4 once it starts to
be adopted by the PHP community."
4) Now, I am on github readme.md, still unsuspicious
https://github.com/zendframework/ZendSkeletonApplication
..no php version mentioning.

So, may be I slept to little today, but it took quite an amount of time to
find a problem.

Also, there were no warnings or something about my phpversion when I opened
public\index.php
May be there should be?

2015-09-28 5:03 GMT+03:00 Marco Pivetta <notifications@github.com>:

> @Jokersoft <https://github.com/Jokersoft> which page? That needs
> updating: versions of zendframework below 2.5.0 are PHP 5.3 compatible.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zendframework/ZendSkeletonApplication/commit/6ba430f6a2006d9f37c24b039fda3488e8681c91#commitcomment-13460203>
> .
>


I've reverted this update incase it's compounding #87.  The ide-backend changes work fine for me, but the more recent descriptions of issues with it are very close to something that could have been caused by https://github.com/fpco/ide-backend/commit/48459c6856651ea4a2b92256383841229df35f5e


Being overly cautious here, but I don't like that server switch, I don't know how secure that server is and how (little?) effort it would take for someone to get control of the script.


Yes, that was where the error was. I had tried using Google's original fork, but was getting the same crash. I'll work on uploading the code, but if I had set up Dagger incorrectly, wouldn't that just be in InjectedApplication? I believe I've done everything correctly in setting this up.


Ahh, got it. Thanks for clarifying, @jondkinney!


ｗ


 > 3.1) if you try "http://framework.zend.com/about/faq/" - "Zend Framework 2 is based on PHP 5.3. We expect to add support for PHP 5.4 once it starts to be adopted by the PHP community."

Opened zendframework/zf2-documentation#1516 to fix thi

 > 4) Now, I am on github readme.md, still unsuspicious https://github.com/zendframework/ZendSkeletonApplication ..no php version mentioning.

Yeah, this is probably something that could be fixed in the README, but in general, `composer.json` dictates the dependencies in modern PHP packages - you can always refer to that file as authoritative source of truth when it comes to dependencies (what isn't in there is to be considered missing or a dependency-related bug)

 > Also, there were no warnings or something about my phpversion when I opened public\index.php

`index.php` acts post-installation: this sort of error is caught during composer installation.

I've opened zendframework/ZendSkeletonApplication#313 to address this, as it seems that you installed ZF2 without composer.


I'd prefer this on the line before it. If the path ends up being super long the end-user can wrap however they'd like.


Just mixed the case here, you were closer the first time :-)


I'd prefer to keep the `display: block` in the element style because I've found in over a year of Polymer development that it's usually what you intended. For a beginner it can be very frustrating trying to figure out why your elements are laid out funky without it.


👌👀👌👀👌👀👌👀👌👀 good shit go౦ԁ sHit👌 thats ✔ some good👌👌shit right👌👌th 👌 ere👌👌👌 right✔there ✔✔if i do ƽaү so my selｆ 💯 i say so 💯 thats what im talking about right there right there (chorus: ʳᶦᵍʰᵗ ᵗʰᵉʳᵉ) mMMMMᎷМ💯 👌👌 👌НO0ОଠＯOOＯOОଠଠOoooᵒᵒᵒᵒᵒᵒᵒᵒᵒ👌 👌👌 👌 💯 👌 👀 👀 👀 👌👌Good shit


Changing the import path for Polymer on new elements deserves some discussion.

It really depends on the context. If you're writing reusable components you intend to share and are using [polyserve](https://github.com/polymerlabs/polyserve), it is best to use the Polymer version at that path currently there. But if you're writing new elements in your private project you will probably be importing from `bower_components`. Hmm. I'm not sure which is more likely.

I suppose it might be more likely that you're working on private stuff. I'm probably biased here because I tend to work on open-sourced stuff most. Also at Google the project I work on packages itself as a portable component for deployment in this way. I guess developers working on public components would know what they were doing and willfully disobey the snippet if we defaulted to the bower path. What do you think?


Rather than having a separate handler here you could just pass the event into the `changeBackground` function and do it all in one.  It doesn't make much sense to have an event handler just for the sake of preventing the default action.

    function changeBackground(e) {
        e.preventDefault();
        /* Your code as before */
    }


okay


I like the SASS syntax too, but I'm all on the SCSS train these days.


That's a good question. I was following the example of the existing code here, but this is definitely not ideal. I think the previous code swallowed errors because the only way a Propose can fail is due to a concurrent shutdown, but now that there are other ways to fail it should pass the error up the stack. We shouldn't write to the channel directly because by the time this function is called we no longer own it, but `proposal.fn()` can and should return an error.


It's after the parenthetical.


I can't say for sure. I just guess from the error. Dagger creates dependency graph and inject dependencies, I just think `context` might not be injected so you got "is no virtual method getSystemService".


Updated v2.051 as well:
https://github.com/apmuthu/LimeSurvey/commit/c3bf6a8bcc1e78e7889c2eb95df32386937502f5


yeah, I know, what "composer.json" is and for
But I didn't thing I have to look there until I realized I have a problem.
Again - I wanted to do all quick, as I did a few times before. Imagine.

I have composer, I launched it, it downloaded stuff...
No errors during installation.

That what is weird - everything installs if you follow steps, but no errors
you see. Just a blank white tab in your browser.
Than you go to that line 78, edit it - and everything works just fine. (for
now)

2015-09-28 5:19 GMT+03:00 Marco Pivetta <notifications@github.com>:

> 3.1) if you try "http://framework.zend.com/about/faq/" - "Zend Framework
> 2 is based on PHP 5.3. We expect to add support for PHP 5.4 once it starts
> to be adopted by the PHP community."
>
> Opened zendframework/zf2-documentation#1516
> <https://github.com/zendframework/zf2-documentation/issues/1516> to fix
> thi
>
> 4) Now, I am on github readme.md, still unsuspicious
> https://github.com/zendframework/ZendSkeletonApplication ..no php version
> mentioning.
>
> Yeah, this is probably something that could be fixed in the README, but in
> general, composer.json dictates the dependencies in modern PHP packages -
> you can always refer to that file as authoritative source of truth when it
> comes to dependencies (what isn't in there is to be considered missing or a
> dependency-related bug)
>
> Also, there were no warnings or something about my phpversion when I
> opened public\index.php
>
> index.php acts post-installation: this sort of error is caught during
> composer installation.
>
> I've opened zendframework/ZendSkeletonApplication#313
> <https://github.com/zendframework/ZendSkeletonApplication/issues/313> to
> address this, as it seems that you installed ZF2 without composer.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zendframework/ZendSkeletonApplication/commit/6ba430f6a2006d9f37c24b039fda3488e8681c91#commitcomment-13460262>
> .
>


Looks like I misspelled "Palm"


Really Cute. :)


@Jokersoft that seems weird: it should simply refuse to install, given that `composer.json` requires `5.5` or more...


哈哈 你们随便搞~


Binary file not shown


Definitely. There's a lot more impactful sources of allocations in MVC right now. We'll continue to iterate and improve.


This and the following line should be indented, otherwise markdown doesn't recognise it as code.


Good idea trimming the input, well done!


That what I'm saying)
(was trying to say:))

2015-09-28 5:29 GMT+03:00 Marco Pivetta <notifications@github.com>:

> @Jokersoft <https://github.com/Jokersoft> that seems weird: it should
> simply refuse to install, given that composer.json requires 5.5 or more...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zendframework/ZendSkeletonApplication/commit/6ba430f6a2006d9f37c24b039fda3488e8681c91#commitcomment-13460315>
> .
>


The space is actually intended, because it enforces a newline in markdown.


It's because your fork is out of sync with the master branch and so it cannot be fast forwarded but has to be merged. Easiest to fix this is:
 - delete existing fork
 - fork again

And from now on, whenever you want to make a change:
 - create a new branch based off `balloob/master`
 - apply your change, publish
 - submit PR

Do not edit the master branch yourself but just sync it with upstream regularly.


I'd consider giving the user some feedback in the default case which indicates a not found city.  It wasn't in the brief so it's cool to do nothing.


@Jokersoft is your PHP CLI the same as the one used for your http requests (presumably apache2/nginx + mod_php or fastcgi)?


test case??


Signature algorithm should be passed to this object rather than letting the object to look up.


I dunno, It could be some strange issue of my strange provider likely...

2015-09-28 5:30 GMT+03:00 Ярослав Склабинский <jokersoft.net@gmail.com>:

> That what I'm saying)
> (was trying to say:))
>
> 2015-09-28 5:29 GMT+03:00 Marco Pivetta <notifications@github.com>:
>
>> @Jokersoft <https://github.com/Jokersoft> that seems weird: it should
>> simply refuse to install, given that composer.json requires 5.5 or
>> more...
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/zendframework/ZendSkeletonApplication/commit/6ba430f6a2006d9f37c24b039fda3488e8681c91#commitcomment-13460315>
>> .
>>
>
>


Seriously? 40k lines of bloat to gain 2 elo? You really think this has any chance to be merged into the master branch?


http://raphaeljs.com/
http://raphaeljs.com/tiger.html

http://cancerberosgx.github.io/raphaeljs-tutorial/

three.js - Javascript 3D library <http://threejs.org/>
*threejs*.org/
<https://www.google.co.id/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#>

   -
   <http://webcache.googleusercontent.com/search?q=cache:5jgno687mJEJ:threejs.org/+&cd=1&hl=en&ct=clnk&gl=id>
   -
   <https://www.google.co.id/search?espv=2&biw=1440&bih=798&site=webhp&q=related:threejs.org/+3d.js&tbo=1&sa=X&ved=0CCIQHzAAahUKEwjk9KaU2JjIAhUHHY4KHSypApQ>

A JavaScript 3D Library which makes WebGL simpler.
Examples <http://threejs.org/examples/> - Documentation
<http://threejs.org/docs/> - Cloth simulation
<http://threejs.org/examples/webgl_animation_cloth.html> - Animation -
skinning
<http://threejs.org/examples/webgl_animation_skinning_blending.html>
D3.js - Data-Driven Documents <http://d3js.org/>
*d3js*.org/
<https://www.google.co.id/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#>

   -
   <http://webcache.googleusercontent.com/search?q=cache:fCkLq1Kw-T8J:d3js.org/+&cd=2&hl=en&ct=clnk&gl=id>
   -
   <https://www.google.co.id/search?espv=2&biw=1440&bih=798&site=webhp&q=related:d3js.org/+3d.js&tbo=1&sa=X&ved=0CC0QHzABahUKEwjk9KaU2JjIAhUHHY4KHSypApQ>

D3 is a JavaScript library for visualizing data with HTML, SVG, and CSS.
You've visited this page 2 times. Last visit: 6/17/14

http://www.pixijs.com/ - Canvas / WebGL


Don't store `dispatch()` methods catch *all* actions sent throughout the entire app? I've thought about this same refactor, but wouldn't it then trigger a change for every since dispatched event? I know React would know that the state hasn't changed so not trigger the redraw of the components, but I wonder if would be making it do extra work...


Shouldn't we add stem here, too?


nit---maybe no colons here or below?


nit?


Updated v2.051 as well:
https://github.com/apmuthu/LimeSurvey/commit/740d2be77a890b2340068f646d5e20abbef88642


@pomu0325 You are right! I removed `isSupported()`. Thank you :+1:
https://github.com/koogawa/InteractiveMessagingSample/commit/3e41e9516167a12f30ad62f0672e218b9fbe54ed


Uh?


wouldn't this be true given line 26 above?  `vm.taskAssignment = data;` would imply that `vm.taskAssignment.is_reviewer === data.is_reviewer;`


修改


sanity check---no one else used this?


Bad Code Smell : Excessively short identifiers
The method name does not give any hint about what the method do.


I'm also not sure how to do this.


Quotes are a problem for at least windows


thx


création d'un fichier readme