I'd have to check- but I'm positive the formula I used is correct. I checked with many different programs online and did the equation by hand several times. The only thing I would do differently if I had more time is work out a way to round the decimal value. I did but when I made the change it really messed up my test cases so I just left the decimal value long.

I finished my individual assignment yesterday- I am working on gathering information for the presentation.

________________________________
From: Aaron Keller <notifications@github.com>
Sent: Thursday, June 11, 2015 2:58 AM
To: buck-kristen/charcoaled
Cc: Buck, Kristen
Subject: Re: [charcoaled] 06/09/15 completed individual assignment and adjust/added the following: (1145433)

Got thinking about this formula. I'm wondering if it is accurate. Wouldn't it be:
double volume = (4 * Math.PI * Math.pow (radius, 3)) / 3; // calculate volume of a sphere

-
Reply to this email directly or view it on GitHub<https://github.com/buck-kristen/charcoaled/commit/1145433697b61b9cd8b17e10f862abf2c11ca6f5#commitcomment-11625588>.


I probably didn't understand what you are doing. I just assumed that MapValue was a data structure that could contain any type as keys. You are right that JSON associative arrays can only have strings as keys.


> I'll have to take a closer look at the blurriness you describe.

This is a screenshot taken from the gl-branch. If you look at it closely, you'll see that there is interpolation on each sides of each pixels on this picture. This makes the pixels less sharp and it is what causes the blurry appearance that I describe.

![screenshot from 2014-03-15 14 21 14](https://cloud.githubusercontent.com/assets/6194072/8099410/8a2f1ac0-0fca-11e5-8566-490e3e093822.png)

Close look:
![screenshot from 2015-06-10 23 45 59](https://cloud.githubusercontent.com/assets/6194072/8099443/00acc5ee-0fcb-11e5-9d40-c70fc5805b1b.png)


It's javascript. All numbers are float numbers whether you like it or not.

I think Math.floor() here would probably solve the issue.


looks good. you should use ul's for vertical navs as well, rather all navs!


:)

On Thu, Jun 11, 2015, 1:02 AM Ryan P.C. McQuen <notifications@github.com>
wrote:

> You are a wizard!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Gisto/Gisto/commit/34996416072cdd0b38080ae16671bdb99a480ea9#commitcomment-11622253>
> .
>


@mikejolley this change breaks Order Status Manager emails. The extension uses a common class for each custom email so the `$current_section === sanitize_title( get_class( $email ) )` check fails. Is it possible to change [this](https://github.com/woothemes/woocommerce/blob/60814f974a5eed9e61be3d9e3d8c20d9e87e77ea/includes/admin/settings/class-wc-settings-emails.php#L205-L209) to something like:

```
foreach ( $wc_emails->get_emails() as $email_id => $email ) {

	if ( $current_section === sanitize_title( $email_id ) ) {
		do_action( 'woocommerce_update_options_' . $this->id . '_' . $email_id );
	}
}
```

Thanks!


Sorry, of course you're right. This commit LGTM.


If we are going to do this, should we export `Control.Monad.Trans.Either` as well?


I'm curious what problem you are attempting to solve with this change?


It takes a little effort to decipher these dates which are essentially the key to why these tests pass or fail.

Perhaps using variables like this might reduce some of that mental load:
```groovy
def now = new DateTime() // Already exists
def yesterday = now.minusDays(1)
def dayBeforeYesterday = yesterday.minusDays(1)
def tomorrow = now.plusDays(1)
def dayAfterTomorrow = tomorrow.plusDays(1)
```

And then it might be easier to visualise where we are in relation to the time range between deployment and recovery. eg.:
```groovy
def deployment = createDeployment(receiver: receiver, initDateTime: yesterday)
def recovery = createRecovery(deployment: deployment, recoveryDateTime: tomorrow)
```


Woot, back on a laptop.

Ah right, so `recoveryParam = null` is set properly inside the DER import function. It will only ever be messed up, in the basically never case (the constructor isn't exported) you pass in an {object, ...} that doesn't have the recoveryParam.


Shouldn't this be `modifies the environment`?


Maybe `tryJoinRoom` should support leaving the `connection` parameter empty, like `joinRoom` does?


Indeed it would return nothing for other distros (Unless they add themselves to the list).

And yes, that was the intention, as this entry inside the menu was **added** and **not moved from elsewhere**.
ATM only OpenATV wants this additional entry, but feel free to add openvix to the list, actually its exactly your menu structure when it comes to the EPG.

So I'm expecting the first line to read at least
{{{if getImageDistro() in ("openatv", "openvix", "openhdf", "egami"):}}}
sooner or later, because those are the other teams that already made the EPG settings menu more prominent.
That's why I didn't chose a shorter variant.


I think we should handle permissions in `tryJoinRoom` itself.
That would be an step towards an IRC-like `/invite` that bypasses permission.


Indentation is totally screwed in this file


I think it would be better if this returned `&influxql.Row{Err: fmt.Errorf("show diagnostics has not been implemented yet"}`


@gsamokovarov  Thank you for checking. I will check your points later.

> Wooo, this whole task looks pretty complicated.

Yeah, I think so too. Can I split this task into sub tasks?


Partially reverted, yes.


`.each_with_object([]) do ...`


排序的参考
http://balinterdi.com/2014/03/05/sorting-arrays-in-ember-dot-js-by-various-criteria.html

attach to #1


attach to #1


This breaks coreutils for me, it never even builds.


Oh I didn't realize it was for new,the new path doesn't accept any arguments
rake routes to see accepted paths

On Tue, Jun 9, 2015 at 11:27 PM, LinnierGames <notifications@github.com>
wrote:

> tried this new_blog_comment_path(@blog) nope
> and new_blog_comment_path(blog_id: @blog) same thing the text field is
> blank
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/whothat/game_blog/commit/2c1699143a84be8227f5591868f458f7b56cfc7a#commitcomment-11604168>
> .
>


@TsudaKageyu, This conflicts with #567, it appears to be unnecessary in this PR, can it be undone for maintainer ease.


We could remove this :)


Can you explain it bit feature.

I followed ('zeppelinWebApp').controller('NavCtrl')

On Thu, Jun 11, 2015 at 9:32 AM, CORNEAU Damien <notifications@github.com>
wrote:

> Indentation is totally screwed in this file
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Madhuka/apache-zeppelin-gsoc/commit/595d3e9c31dd0720b69f972c9be09d8df7421ee7#commitcomment-11626019>
> .
>

--
Cheers,
Madhuka Udantha
http://madhukaudantha.blogspot.com


Sorry, but this didn't seem to fix it. Could you reopen this issue?


Right. This change has nothing to do with this PR. I will revert it soon.


I think that would give me the instance of the post instead of the integer
of the post ID. I can't think of any other way to set the comment's post_id
other than to do it manually.

On Wed, Jun 10, 2015 at 7:52 PM Bobbilee <notifications@github.com> wrote:

> disregard what i said. im playing around with it and looks good!
>
> i would just simplify...
>
> @comment.post_id = Post.find(params[:post_id])
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/voltaaage/Bloccit/commit/e0a64480899bd7d86222fc8b14dc3e49d4ea43cf#commitcomment-11624347>
> .
>


Weird, I just purged gcc and with the new gcc build it seems to work fine :/


Deleted copy


Should this include the number of readers?


Merge pull request #120


Looks good to me. One quick question: why recursive_mutex is required? Is it possible to just use mutex instead?


Please fix, It does not read right


You are correct, It's my oversight to assume that the keys would be strings all the time.
I'm correcting the implementation.
Is my understanding correct in the following case :
If the key is a *boolean*,*number*, The JSON would look like [("key" -> *Any structure than string*), ("val" -> *Any structure*)].


This is a necessary move it seems when using jekyll, in github. The info on the Jekyll website seems to be wrong. It lists it as /css.css instead of main.css which is what it's called in the "head" file (_includes).


that's a fine example

On Wed, Jun 10, 2015 at 6:22 PM, Lauren Boyd <notifications@github.com>
wrote:

> What about the example ps aux | "username" so they can see what programs
> they are running? And they could run it on their partner's username or
> something like that?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lboyd002/ucr-cs100/commit/dcf03e6af7a326729a03549ea61de483ecdc412e#commitcomment-11624623>
> .
>


no

On Wed, Jun 10, 2015 at 6:38 PM, ZhuLeon <notifications@github.com> wrote:

> Okay I got it, that fixed it! Thanks! Since execvp is called right after,
> I wouldn't need to then deallocate the memory would I?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lboyd002/ucr-cs100/commit/dcf03e6af7a326729a03549ea61de483ecdc412e#commitcomment-11624767>
> .
>


sentence fragment


Rc


Can be removed.


Needs to be updated.


I fixed this problem. Use props for Authentication mode


Looks good to me. Please merge


It seems like these specs are still failing: https://travis-ci.org/NetSweet/netsuite/builds/65077207


Доберусь до дома закоммичу нормально, забыл совсем про сборку.


Serious?:) Why do you need this method at all?  There are some legal cases when it might be: in our case there are DB A and underling DB B.


Why would you ever turn spell check off?


This is annoying because it moves the cursor to a different location


I'm not very enthusiastic about adding this field, but I couldn't think of another way to get the name of the function to the unconstrainedTest.String() method.


this deletes the first two characters, which very rarely actually uncomments a line


this duplicates `yy` which is standard vim and easier to type


I removed these, since it's possible now to just specify a Problem with Grad == nil


`>>` does the same and is standard vim and doesn't move my cursor from its current location


None of these examples are particularly usable in practice, and it doesn't seem like you put all that much thought into them.


we need to ignore the public folder in the root


Code Review:  Coke and pepsi are not descriptive variable names


TODO: Remove the TODO...


I'm giving you the 80/100 I mentioned before, but if you want to work on the examples some more you can resubmit before Monday.


... rework needed


git bisect points to 3242086028408b3e4eb1ac2f3fca44f6dbb20860 as the culprit.


This change seems to have broken the build: https://travis-ci.org/NetSweet/netsuite/builds/65077207


HTML tags are not allowed.  A blank line will have the same effect as `<br>`.


this should be a link to the later section


Looks good.


I really like this.


Yes, he does.


Sarchak - how do I get a hold of you to build an app similar to yik yak for me?

Ema
il me at eli@alibichat.com


The main use case for `tee` is within a chain of pipes to capture the intermediate output; you should have an example like this


Now I'm getting weird build errors when running tests for certain perl modules like net-http-6.06


I don't agree. In most cases the automatic calculation gives me not what i expecting.


s/pid/`pid`

anytime where you're quoting something in backticks, you need to place it in backticks


Vinny!


1024 :unamused:


1679-DualDuct (kbenne) - i386-Windows-7-VisualStudio-12: OK (2185 of 2214 tests passed)

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


My key value pairs were a JSON object with two fields, one called "key" and the other called "value".

So `{ "key": 1, "value": [1, 2, 3] }`, for instance.


Great, nice cleaning up. Microsoft should definitely merge this.


lookin good


=> https://github.com/yeoman/generator-mobile/issues/84


You should not call `thefuck_alias()` on every iteration.


Thanks, That is clear.

Earlier I meant, ```[{ "key": 1, "value": [1, 2, 3] }, { "key": 2, "value": [1, 2, 3] }]```


commit message should say "added CODEPEN variation of picker script", not 'codeine' -- thanks autocorrect!


I got it. Until upstream is done, let me update the wording so that "flag-is-true-by-default" is only applied to UA generated events.


@crhym3 Instead of these constants what's wrong with `callback(null, {})` and `callback(error)`. That's how basically every node module (and node core) does it. Or am I totally missing something?


**nice work**


# JEREMY ROCKS


```js
if (['github', 'heroku'].indexOf(reqs.hostingChoice) !== 0) {
```


Is this a bit weird having field.opts.name different to whats set in `mapper.fields`  Perhaps `_fields.update({name: obj})` seen on LN73 should in fact use `_fields.update({obj.get_name(): obj})` after the default has been set?


you are treating linux, windows and mac line breaks differently here ("\n" => "", "\n\r" => " ", "\r\n" => " ") - does this have a specific reason? "\s" does also include "\n"


hhhgg


shouldn't you add 'sm' modifiers to also catch linebreaks?


you suppose to use the pull spell on all creatures agro or not.

even when they are attacking other NPC or player.


jhgjjjgh


@rooque this is a test release， not public

I have not received the relevant report


为什么要添加domIndex呢？有什么应用场景需要吗？


Function needs error handling.


jjj


Ty Chris :+1:


also guys can you please include this in all versions of game cata and wotlk


Change this to double, currency is used to hold currency codes like USD or KES not amount.


@t4t5, The `routeObj` property doesn't exist.
It's breaking the Router component. Did you forget to commit something?


This commit introduced a regression:
https://issues.dlang.org/show_bug.cgi?id=14682


例えば
select_id　に　user.belong_id をセットしたい場合の記述はどうなりますか？

User.belong_id  ?
User[belong_id] ?
User.[:belong_id]  ?

お作法の問題かと思いますが、ごちゃごちゃになってよく分からないんです。


kakaha


new update


format your code to:
```java
public void methodName() {
}
```


That it is. Thanks. :P


@bredon I branched off our Master branch and pulled in the most recent bedrock stuff from Roots. Looks like a lot has changed.


this should be `@ManyToOne`


They cleaned house on this aspect of the environment config files


But as I pointed out with my XMLHttpRequest example, even that would be wrong. We need to identify which actions need to dispatch events with this flag set.


So this will be like this forever...?


This should be `@ManyToOne`. Many payments can be of one type


From my understanding, words not recognized will be highlighted as misspelled words. So toggling the highlighting would be particularly convenient while coding since most code terms would end up being highlighted, making the document look rather obnoxious.


With all these deploy configs removed, I'm not sure how it deploys yet.


Yes, a copy/paste error. Thanks for catching.

Fixed in bad2b37090722f7655e5aefd33d908b424925b4b.


Rename this to `Payment`.


The test net functionality, in terms of a different alphabet, hash prefixes etc, has been gutted from rippled. There's not much reason to keep this these days.


Just set vim to only spell check comments.

On Wed, Jun 10, 2015 at 10:21 PM, markasfour <notifications@github.com>
wrote:

> From my understanding, words not recognized will be highlighted as
> misspelled words. So toggling the highlighting would be particularly
> convenient while coding since most code terms would end up being
> highlighted, making the document look rather obnoxious.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/markasfour/ucr-cs100/commit/d93769ea937863b8d819f762816e57898d84b062#commitcomment-11626738>
> .
>


One thing that transaction.complete() does is set the canonical signature bit in the flags. This doesn't seem to?


since 1.2?


And there is no matching relationship in `Account.java`.


Add a line before the rule which does:
```
MPCONFIGPORT_MK = $(wildcard mpconfigport.mk)
```
and then use $(MPCONFIGPORT_MK) as the dependency instead ot mkconfigport.mk


ใช่


Should they be doc(hidden)?


I made a pull request (#33) in order to fix it.
Please, take a look


The table should be named `payment_type`.


I didn't understand the issue here ... does Postgres not preserve case, instead returning columns like "starttime", "gametype", etc. that must be renamed?  Or is it that we have a gameType that collides with a gametype elsewhere?


Format this code.


Thanks. I'm afraid that I don't understand what action need to be done for XMLHttpRequest.
I guess we need more investigation. Let's reopen #61.


That breaks step 4.


On both links for quality test used SSL Labs service.

With short cipher list some software can not connect to server:
* IE 6 / XP
* IE 8 / XP
* Java 6u45
* Java 7u25
* OpenSSL 0.9.8y

With long cipher list only (IE 6 / XP) and (Java 6u45) can not connect to server, same as cipher list published by [SSL Labs](https://www.ssllabs.com/projects/documentation/index.html).

Cipher list from SSL Labs is shorter than cipher list from https://cipherli.st/ it that matter.

Any way current cipher list is less secure than cipher list from SSL Labs or https://cipherli.st/: SSL Labs Overall Rating **B**


per #57


`description` should work fine.


[nit] I always like to take out conditions that don't matter (if tables.games.length > 0 isn't needed, if I read things correctly)


This field is not required


Should be an option (either through the environment variable, or maybe the time has come to have a bona fide options file (in JSON format) that we read in at startup).


Should be provided as a (command-line?) option ...


Revert -- or add || "".


Mathematically this is kinda wrong? Most mathematicians will consider `0` neither positive nor negative. Others will argue that `0` is both negative and positive. This is anyways inconsistent with `is-negative`.

Also it just sounds weird to me that this module return `true` when passing negative zero (`-0`)

@kevva @sindresorhus Thoughts?


> You're assuming it supports reloading config which, according to the init script (/etc/init.d/dnsmasq), it does not.

aren't we using upstart on the ubuntu systems?
They should interpret a 'reload' as 'send a HUP to main pid'


Why need to convert into string? Please specify why ```to_s``` is used?


Why need to convert into string? Please specify why to_s is used?


Move inline CSS into custom.css


Remove logger


Yes.

This went to my home email for some reason. Sorry for the late response.


@SiGe The VM delete fails because of the option "-b". The whole process was blocked by the deletion of storage. We might consider skip it for now.


LGTM


I think no need to check for current user. Please specify about this code.


I guess consensus is that best-effort semantics is appropriate here, so looks good.


Isn't this method unnecessary, because setChoices() always removes existing choices anyway?


Probably nitpick, but how about checking for `>=2` similar to how you call `socket.end()`?


call this mesos_runtime or something like that


Ah, I mixed up JRuby's implementation of `File.exist?` with MRI's (JRuby's treats a broken symlink as file that exists).

Fixed in 1e5994ed73.


public or private ??


Unfortunately this breaks due to a bad interaction with a langmap workaround needed for vim <= 7.4 patch 502: https://travis-ci.org/SirVer/ultisnips/builds/66328567


Don't care how minor the comment is as long as it's constructive. ;)

Though don't understand what you mean by checking `>= 2`. Only use that above to determine whether more data should be written or should end. At the end of the test exactly 2 chunks should have been read. I intentionally wrote only one byte (i.e. `'.'`) to make sure the packets of data couldn't somehow break up the data transfer.


If I will not specify current_user then after sign out also index page loads. That will give error.


This broke long options, i.e., --apple, --extension, --mime-type, and --mime-encoding are now 11, 12, 13, and 14 respectively.
```
--- src/file.c
+++ src/file.c
@@ -187,16 +187,16 @@ main(int argc, char *argv[])
                        case 0:
                                help();
                                break;
-                       case 10:
+                       case 11:
                                flags |= MAGIC_APPLE;
                                break;
-                       case 11:
+                       case 12:
                                flags |= MAGIC_EXTENSION;
                                break;
-                       case 12:
+                       case 13:
                                flags |= MAGIC_MIME_TYPE;
                                break;
-                       case 13:
+                       case 14:
                                flags |= MAGIC_MIME_ENCODING;
                                break;
                        }
```


Why you have used 'install_with_xfce' in old commit and declared it latter?


? we should probably deny `unstable since`


doc(hidden)?


комментарий считается к линии под которой он написан. Так мы обсуждаем строку 61.


I want to ask you ,Is there full text index kind of functionality available in Apache Hive for searching, matching & ranking keywords of the text?

If any functionality will be there then let me know.

Thanks,
Jalaj T


since 1.2?


You have used xfceStart() in previous commit and defined here ??


@fxmzb123 It would be nice not to have blank-space changes like this, just to keep the diffs clean.


My bad.


Чтобы список элементов в карте сайта можно было легче отличать от других списков в процессе стилизации.


@fxmzb123 Here too.


no facets?


The bug is not here :(


I will change it.


@romainhild Can you correct it ?
[See here](http://my.cdash.org/viewConfigure.php?buildid=778693)


good job


I thought, it can be used when need to get all members of the cluster using the map. As it is not necessary I will remove it.


This function is only runs once during init.  Once the user requests some refactoring depending on the AST we'll make another attempt at building it and this time the error will be user-visible.


I would do that with templates. Either you do it or I do it when I am back.


RIP :ghost:


put the last 'if statement' at line82 ahead at the very beginning.
to make sure each entry will be only counted once


This is of course the beta website.  I hope to extend this project and gather insights from talented coders to improve the aesthetics and interoperability. This is a timeline function and hopes to demonstrate the growth patterns and histories of an understudied community in Los Angeles and, more specifically, in the Jewish community.


No description?


When I remove that line, it builds on my local machine. Apparently something else is broken though.


Ah yeah, we probably can do that now :)


I prefer explicit second arrow: `$self->{id_gen}->($self)`. Same below


https://github.com/fmars/Differential-Privacy/blob/4eb0c8f3743992035ccdccf65744bf02cd00bea4/CodeBase/lib/NumericaltoFloat.py#L6

when parse string into float, please check if the string is in valid format


Sorry Adrien ! Je n'étais pas très inspiré sur le second degré ce jour-là :(
C'est update ;)


Typo: that → than


Maybe add `cats_dir` to `@CATS::Config::EXPORT` instead?


```c++
return sGuildColorBackgroundStore.LookupEntry(m_backgroundColor) && sGuildColorBorderStore.LookupEntry(m_borderColor) && sGuildColorEmblemStore.LookupEntry(m_color);
```
? O.o


Would it be possible to have two independent progress bar colors for frame title and sections? Some time ago I changed the section progressbar to dark teal/gray but that was lost by changes done by others. Alternatively, I would (nowadays) completely refrain from using alerted for the progress bar, it's almost too much contrast.


【コメントテスト】headタグは田場くんが編集したのを上書きお願いします。


@archermind 关于“yet to be created”如何翻译，第一遍草拟翻译时我就斟酌良久。你既提及，我们恰可细细玩味。

“yet to be”一般翻译为“尚待”、“有待”。与你思路类似，为了突出“yet to be”，我曾考虑译成 “此处仍未创建”、“此处还未创建”（这个与你的提议有点类似）、“此处尚未创建”、“此处没有创建” 等等。最后通读时，考虑上下文语感，（纯凭个人感觉）选了个“此处并未创建”。

现在回头审阅，该处原文为：
> and (yet to be created) `index` files

私以为更好的翻译方案是：“（尚待创建的）`index` 文件”。

这样既突出“yet to be”，语序也与原文更对应。不知你意下如何？


update comment too :P


Is this condition duplicated somewhere?


you can just
```
return p - scalarMultiplication(n, 2 * (scalarProduct(p,n) / scalarProduct(n,n)));
```


Commit message should reference issue #98


Right, I'll post an update later today


the return type prpobably shouldn't be ```float```


For #124. added -d flag to command line. works on Linux,  Windows and OSX.


surely your code doesn't compile?!


use ```const auto&```


Oops, I didn't go further than check the title of the badger including cobertura in its name ;)


your code is incomplete.


Styles Panel bug shows up at this commit.


please only submit pull requests that at least compile!


Every time I see this copypasted UTC class I die a little inside. Someday we'll all adopt python 3 and this madness can stop.


You can use the option -DskipTests when executing mvn install.
Maybe @lveci can explain what you have to do to get the tests working.


$ 2


please use formward slashes!


h


Nah, I give up for now. Looked at the dump once again, and it's a portion of evr.dll that is suddenly not-executable. Like that was done by some rogue hook or on purpose.


This is actually quite the performance hit :(


enumset is actually basically rustc internal. It isn't re-exported out of libcollections.


you shouldn't hard-code ```double```, but use the number type you ```typedef```ed. This avoids unpleasant surprises when you change that type.


Okay


Our company - Xiamen Finehope Polyurethane Co., Ltd devoted in promoting widely usage of polyurethane since 2002. During the past 12 years, we had worked with many prestigious companies.
http://www.polyurethanesupplierschina.com/jp/products.html


githalytics link now points to a spam site, apparently the DN is picked up by other party.


you could combine some operations here, but that's a matter of taste.


you intentionally don't store ```normal_vec``` in this constructor?


Follow-up to #102, to match the DDL fixes/updates @horsburgh made.


put this declaration to line 20, where it's used


put this declaration to line 18, where it's used


Calculate the upper bound for j and use that!


Es jau dzēsu nost pirmstam, bet man izmeta kļūdu:
  "Indenting at the beginning of the document is illegal."
Internetā skatijos, ka visos thread raksta, ka tā ir Hamla versijas vaina. Nesaprotu kāda problēma, ja man principā ir viens no jaunākajiem (varbūt pat jaunākais).


> Meanwhile, NetBSD and OpenBSD don't seem to offer a method to poll for high priority data via kqueue at all. The only way seems to be falling back to poll().

I emailed their respective mailing lists.

For now would you be okay with just adding:
```C
#ifndef EV_OOBAND
#define EV_OOBAND 0
#endif
```
so that this code can be merged?


@archermind 感谢。
转换已完成，见 https://github.com/oldsharp/progit2-zh/commit/b849fbd6a647e2c9e26567aaf5d967d2f2214e36 。


why do you need that?


I do.


The repository link can cause 404 page error if repository is set in another project (not in the project current viewing).
`:id => @project` should be changed to `:id => The id of project owned @repository` (sorry i thought for a while but couldn't figure out how to query that)


throughout, replace ```double``` with a ```typedef```ed  number type


Replace


replace


Unclosed? It looks closed to me


np, glad I could catch it :P


my suggestion would be to include both - default on the more secure one. Commented out the less secure version including older clients.

In my opinion the default should be more secure and only if it matter the lesser secure version should be used.


Is there a repository that I can fork to add that information to the documentation ?


No, think it's because i'm in a subdirectory. assetUrl is set to '/tmp/assets' . I look , but seems core/WebApplication.php are not called. I can fix it in InstallerController (using beforeAction), but did it needed for all controller (i think, no ?))


Same issue as https://github.com/tleish/redmine_revision_branches/commit/e2cf37a16109de6000fdb69af947cde4c26efa62#commitcomment-11627247


This is bogus code, because we don't recognize the video surface ...

Will be removed again.


I'm partial about this one. On the one hand, this makes sense when calling `.isFilesEditable`, but it doesn't make as much sense when setting `true` for `.setFilesEditable`. The alternative is, of course, that we always check `isEditable` before checking `isFilesEditable`


Should we use `-o` instead of `--css` to specify the output bundle.css?


Removed it, wasn't needed.


Alatis i have said to the people who wanted to join dragoncanister  this ;'Sorry you cant join dragon canister now because we have not properly started :)' After we set up we will notify the people what they have to do in order to join Dragoncanister :)


By the way, our IPinHook is fucking dangerous - we take away execution permission while we're modifying the page. PAGE_WRITECOPY should be replaced with PAGE_EXECUTE_WRITECOPY.


should be ok now. see 003520120838fe4c7c87db7fdaa84adc6dbf10b3


&Self should work here, yeah?


I'll give it some thought. Off the top of my head, it should be possible if we give `\progressbar` an optional argument for the beamer colour to use, but I'll see if I can find a clearer solution.


This is what tests_require is for :)


I'm eager to show you, i downloaded the whole map from the server's, which is the API V3 was holding. And i've made a few changes in the map with create.io . And also added more exciting and useful features to the map, if you wish i'm ready to mail you or make it as a new repo. This can be turned into buisiness in India, that is why i'm not uploading it. Please mail me. vishnu@codemazk.com .


Get сейчас просто ссылку возвращает, новый List не создается.


this and the two functions below are actually not currently used and I forgot to remove them for this PR, but they will be essential for upcoming PR, where I'll write tests for them


Awesome - fewer lines of code + better.


kebaiwei
![e kdsi y86 v7vfaz15 t8](https://cloud.githubusercontent.com/assets/10810285/8101115/a99edac2-1046-11e5-8238-a333136674b7.jpg)


same here and `updateFeeds`


Great!


Sorry for typo in commit message


2333。。不过打印店硬件各种省成本


yes! :+1:


Bah, apparently some Chrome extension ate the ending tag... yes it is correct.


Ah, you are right. I totally forgot that they use the same definition in `beamerthemem.sty`. It is little bit more work than I though.


can't you simply do `for elem in limit(strings, max_matches):` instead of storing the generator in the strings variable?  find that better readable and thinkable.


Thanks, I never used those functions so I missed it


Suggestions for the title:
Docker at ePages
Dockerising ePages services
Docker power at ePages


ack


you can explain why the buffered output is prepended instead of appended?


Hi, yeah the controls I made are just for testing purpose's an separate view/tab for these demodulation activities would be much nicer. I was also thinking that some programs could use the demodulated output over tcp/ip or udp streams, I've seen an a program called LRPT analyzer (used to decode NOAA and meteor-m2 weather satellites ) as an example although I've no experience using it.

I don't think Ill be able to create a function to determine the number of constellations and automatically choose the right demodulation scheme, I'm thinking about whats the best way to store this "bitstream" and after that I think its wise if I submit a pull request so we could take it from their. I would like to continue to help you as best as I can. Let me know what you think.

Cheers.


But ` master/load-shiny-data.R` seems to be empty. Deliberate?


What is the error displayed in the logs?


added inline css


Screenshot of how this looks on mobile?

Screenshot of code snippets inside the docs?

Can we use roboto?


add mirror server and another low bitrate stream


Wait what, I thought this was all stabilized.


Oh my god! Finally :3


Change to:
... is one of the hottest...
In a nutshell, one can describe Docker as Linux containers with ....
Docker can be classified somewhere in between fully fledged virtual machines and a simple file system ...
Since it is not yet clear, if Docker..., this post only concentrates on how our development workflow benefits from Docker.


Change to: The challenge


Are you familiar with context managers? They can make this sort of before/after pattern cleaner.


Damn, I guess that's what happens when you start developing on two different platforms at the same time! Though if anything, it should mean that I'd pick it up sooner...


不要なため削除


initで初期化


@oldsharp it's perfect !


NOOOOO

I'd happily continue browsing Reddit insecurely to keep the toolbar.


That has nothing to do with Google Maps Engine API, as near as I can tell.


OK - absolutely last nitpick: `std::array<size_t, 3> ijk1`


リセットフラグが飛んできた時はセッションからキーワードをリセット及びtarget_keywordを空にする


検索状態かの判別はこのメソッドで可能


`log.exception` will include the exception stack trace for you


検索キーワードを無害化するためbindにして対応


keywordにパラメータをセット


キーワードをパラメータから取得してセッションに登録


セッションからキーワードを削除


リセットフラグのパラメータが飛んできているかチェック


good


リセット用のURL生成。グループについては
"target_group_name"に"all"をセットすれば良いためパラメータに指定


It'd not complete - I'm pushing it to a branch here so that I don't lose it if something happens to my machine. But it's not likely that I'll take this much further. A completed version might look pretty similar to http://primercss.io/scaffolding/.


I'm pretty sure it does. It connects to `org.kde.kglobalaccel` for all the shortcut handling to handle screen locking etc.

See also `plasma-workspace/ksmserver/screenlocker/globalaccel.cpp` or `plasma-workspace/ksmserver/server.cpp`.
Do you agree or do I get this somehow wrong?


1画面に複数タイムラインを配置した時にidを一意にするため、idにはportlet.IDをつけます。


Please, write every reports on a .md file named after the date (so move that one)

Do you have an atlas account ?
Have you managed to clone/compile/execute Feel++ examples ?


@jcppkkk... fixed https://github.com/tleish/redmine_revision_branches/issues/3


These links seems wrong.


リセット時には「すべてのグループ」の選択に切り替える処理を流用


LOL, that's true 😕


Fixed


Rather than defaulting here we should simply make it default to the empty string in the dictionary definition if that is the no-op state.


The next thing I plan to do with it is attempt to split out all the data tables and such into separate files. Once that's done it should be easier for collaborators to add content. Currently there's no easy way to merge changes made by different users. That said, letting me know what cells were changed and what you changed them to means I can copy them in a lot faster than working out what needs doing myself.


Thanks for the feedback.


Change to:
... So far we deployed the backend service...
It worked but still could be improved: Changes that were not backwards compatible to the backend ...
Comment: I'm afraid that this sentence sounds too negative... Can we have it sound more positively?


lgtm

if you super duper love this design, we should push to switch over to it.

One plan could be: close out all the bugs, which are mostly due to analyzer (i.e. ensure we display correct information), then automate publishing the docs somewhere, and then we can use remaining time and efforts to work on UI.


Why not

```cpp
PSendSysMessage("Provided spawn radius for creature '%s' is too small. Using 5.0f instead.", unit->GetName());
```


truly nitpicking here: organize this with the other imports at the top


Change to:
To  improve this, we decided use Docker to...
Comment: the end of this sentence sounds a bit bumpy. See if Matt has a suggestion for that during the native proof-read.


Wouldn't just replacing "http://" with "//" work? That's called protocol relative URL IIRC.


Change to: Dockerising
Comment: I know, it looks strange, but as we go for British English, we should use "s" instead of "z".


this comment seems out of date?


Unfortunately, we still need an explicit binding in [KillbillServerModule](https://github.com/killbill/killbill/blob/master/profiles/killbill/src/main/java/org/killbill/billing/server/modules/KillbillServerModule.java#L165).


过滤选择器


oops, something bad happened here. looks like html elements were removed and linebreaks were added to the string (real ones, not the string "\n") were added


Change to: First step towards this goal: dockerise our services. We had to provide some kind of built description, that...
With regards to Docker, this is a file called `Dockerfile` that we added to all of our ...


here too


Well I don't enable null or empty collections, so the field won't appear in Kibana at all for that event.
The `put(String,Map<String,Object>)` method filters the null and empty collections.


Do you need to do something similar to https://github.com/killbill/killbill/commit/07fa3c716926c056dbc5d7f6d0a59178e0e17bf9?


Change to:
Add a current....


:+1:

Are you planning on adding support in the Java client? That way, we could add a server test.


i have an atlas account yes.
But i haven't been able to clone/compile/execute Feel++ examples yet.


The Google Analytics plugin need p12 support.


No need for account event too?


Change to:
Install all of our...
Comment: I'm not sure it it must be "built" instead of build here. See also "build description" vs. "built description".


these magic suffix strings might not want to live on github


Good luck ;-)


Done (ae88615a). It may be reactivated later.


These are just defaults, the GA plugin will need to specify in the field config:

```
fileTypes: p12
```


We should talk about this.


Uppss you are right, I'll fix it!


Change to:
Define what it means to run this image...


Oops. Maybe look at the other callbacks to see if we didn't copy/paste the mistake?


我看就这个样了


and for a .md file, do i do it using an external editor or there is a way to do it directly here?


Second sentence is a bit too long.
Suggestion to end with ... running Docker.  Then start a new sentence.


@klinovp How will isObjectProperty() turn out if predicate is ANY?


// FIXME: Change so it only sends the actual date not timezone and timezone_type


It's unnecessarily complex. Doing three least-squares iterations from the
center of the Earth is enough.

2015-06-09 11:06 GMT+02:00 Emanuele Ziglioli <notifications@github.com>:

> What was the problem with Bancroft? Just curious
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/goGPS-Project/goGPS/commit/c3a63fc981f07b2f2f2cffb7e814599be910adce#commitcomment-11583770>
> .
>


Worked around by 8b2f226a (or maybe fixed... I don't directly see if the phase does what it was supposed to do).


Андрей Витальевич, а почему ветка `release/current` находится в вашем репозитории? Тут сказано, что она должна быть у Кирилла. Или я чего-то не понимаю?


hah, I was going to comment on this is the previous commit :)


Why not just set ``doCheck = false;``? That would stop the dependency and left it easier to re-activate checking...


Thanks @svaksha


Anyway, https://github.com/alexmarsev/mpc-hc/commit/45b1b02d3869df6a621fb693f7e6858b403e6edd. Please push it in the next batch.
I would also revert IMFTrackedSample commit, but if anyone wants to keep it, be that way.


Thanks


Dat is a typo. It has to read validate-dist with a hyphen corresponding to how it's in validate.mk


That's nice but the bigger problem is that the change doesn't actually do anything unless we move the `PyTango.DeviceProxy(uri)` in between the host settings...
The biggest problem is, however, that the `TANGO_HOST` setting doesn't have any effect, it seems to me that it reads the env variable in the beginning but then it doesn't pick up the changes. I found a multi tango host option but that doesn't seem to work either, I will write to Tango guys, until then I will revert this because it's useless...


`days and months are always two digit` but that's not what I was seeing with your initial patch... is there a bug in our Intl implementation ? Or am I doing something incorrectly ?


Change to:
One can deploy custom Docker images to the official Docker Hub for free as long as these images don't need to be private.
Whenever we...., our continuous integration systems take the ...
As a result, every developer can just use these ...


No, because `this.childNodes != this.children`, `this.children` does not contain Text nodes.
![child-nodes](https://cloud.githubusercontent.com/assets/6435556/8101685/8ff89b9c-102a-11e5-8944-2c84a5cca181.png)


MariaDB :-)


Thanks!


Change to:
The last step is to...
This container receives a...


Cool, I had some trouble figuring out what was up with the code for this one. I guess the order part isn't implemented yet?

Also, this is bringing the use of the permission system to levels I don't think I had ever dreamed of, pretty wild!


BTW, after this commit, TestGoGPS process get below error.
Only me?

java.lang.StringIndexOutOfBoundsException: String index out of range: -1
	at java.lang.String.substring(String.java:1918)
	at org.gogpsproject.ReceiverPosition.codeStandalone(ReceiverPosition.java:312)
	at org.gogpsproject.GoGPS.runKalmanFilter(GoGPS.java:457)
	at org.gogpsproject.TestGoGPS.main(TestGoGPS.java:157)


I'll fix the changelog in a second, commits after release were just cosmetic code cleanup, nothing new feature-wise.


Good finds. :-)


There is a lot of solution, it's actually up to you. But you should consider using the same editor for everything, that is Vim or Emacs.


Change to:
The Docker image to be used is defined for every service as well as the links, i.e. the frontend needs the backend and the backend needs the database.


Yes, if call `filterNodes` on `this.childNodes`, but not on `this.children`.


ok let me try it.


Whoops

On Wed, Jun 10, 2015, 23:00 Bjørn Forsman <notifications@github.com> wrote:

> No description?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/aece1407d5a568e90d0c680fbc58266759cf6db4#commitcomment-11627065>
> .
>


Comment: I don't understand the "Since Docker is changing..." sentence. It is too long and I don't get the content.
Please rewrite.


It looks ok :)


I'll skip Closures for default value, using protected method on form is enough, IMO even better and cleaner than Closure


@shakuzen That was quick!


Fixed just now! :D
Please check.

2015-06-11 9:16 GMT+02:00 Daisuke YOSHIDA <notifications@github.com>:

> BTW, after this commit, TestGoGPS process get below error.
> Only me?
>
> java.lang.StringIndexOutOfBoundsException: String index out of range: -1
> at java.lang.String.substring(String.java:1918)
> at
> org.gogpsproject.ReceiverPosition.codeStandalone(ReceiverPosition.java:312)
> at org.gogpsproject.GoGPS.runKalmanFilter(GoGPS.java:457)
> at org.gogpsproject.TestGoGPS.main(TestGoGPS.java:157)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/goGPS-Project/goGPS/commit/c3a63fc981f07b2f2f2cffb7e814599be910adce#commitcomment-11627907>
> .
>


The crazy ideas are great... we'll just have to see which ones work best in practice...
I've heard about something called "Holy Traits"... I'm going to have to look into that, but maybe that's what is really needed for some of this...
At least for immutable strings, I think it makes sense for the types to only allow validated strings.
For mutable strings, you frequently have to wait until the string is fully constructed before it is valid (if you are building it using code units as opposed to characters [if the characters are always valid]).
Null terminated strings are almost never needed, only for some antique APIs, or APIs that are built upon those antique APIs... I had been thinking that for convenience, you could store a byte/word of 0 after the last code unit,
*space permitting*.  You can have a function then that does the checking for embedded nulls and for a 0 past the last code unit... which is what Cstring and Cwstring need for ccall... they give an error on an embedded null, and copy if there isn't a 0 past the last code unit (which shouldn't be that much overhead... these strings are usually < 256 characters, things like file names, directory names, etc.


It might be a bug when using Unit.getUnitsInRadius() in melee mode. The function works 80% in use-map-setting mode while some units inside the range may corrupt the result unit set.


Sīs metodes pasargā no tā, ka lietotājs manuāli mainot linka { http://localhost:3000/diary?direction=asc&sort=date } direction nosaukumu vai sort nosaukumu, if automātiski aizmetīs viņu atpakaļ, savādāk tiek izmests error, ka tāda nav. Piemēram, { diary?direction=asc&sort=date } nomainīs uz { diary?direction=asc&sort=like }, izvadīs ārā kļūdu, ka nav tāds "like" kolonnas.


A regex match definition has match: regex sometimes. Can we be consistent?


Please, place closing bracket and `catch` on the same line. This is a common JS style.


Small request: could you set Git's `core.autocrlf` to a value that prevents it from normalizing line endings? It makes diffs really hard  to read, on Github at least.

For anyone interesting in a quick glance to this change, here's the same diff ignoring spaces:
https://github.com/boogie-org/boogie/commit/9c97c7c52776575485c3b131202addb1d864e3d9?w=1


The token is not necessarily revoked, but it is definitely invalid. Cause could either be the clock or that the token has been revoked.


That's why I got messy video stream using VLC when playing WVX stream, without such patch.
https://a.disquscdn.com/uploads/mediaembed/images/1942/867/original.jpg (from https://launchpad.net/bugs/1449892)


Why not to check if `fs.existsSync()`?


Nope, didn't work.

On Wednesday, June 10, 2015, posixphreak <notifications@github.com> wrote:

> Wouldn't just replacing "http://" with "//" work? That's called protocol
> relative URL IIRC.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Mevin1/agarmods/commit/c7c6c88d67e9f62e1297905be644b43036e093f6#commitcomment-11627645>
> .
>

--


Should we log what happens after as well? Did refreshing the access token work? Or did the user get logged out?


Fix #18


read this


Confirmed. Thanks!


sweet :+1:


下次记得把summary写一下
[WG]汉化更新


PHPCS: There must be one blank line after the last USE statement; 2 found;


HTML cares not for white space so those lines, so this will interweave them (male, replace, female, swap)
You are better off putting them at the bottom or looking up bootstrap's colums.
Additionally it

 should be used for the title (which here I'm using for the icons) then

 for the description (shortcut). I actually messed up here and used dt on the closing tag.
The html pages can be tested (roughy) by opening them in a webpage. But only the display not the saving or anything else.


I don't need logging if it works, but there is a log event if the user was forced out


@wkyoshe You can do it both by editing a local file and commit it or directly by creating a file on github (You have a preview tab on github to seek what the report will look lke).
And also when doing a commit from local files, please put an informative commit message about what you did: e.g.:
```
Added new report, updated
```
```
Updated code:
- added feature 1
- fixed bugs in section
```


The api was changed for it


// FIXME: Change so it only sends the actual date not timezone and timezone_type


+1 Don't know why this hasn't been merged yet. More focus on the Constitution right now I suppose. :stuck_out_tongue_closed_eyes:


Hi Vivek

i am still awaiting the code above and i have problem reading images for each air flight mybe you can assist me?

Please reply urgently because my client is already being charged by travelport


really odd, that migration. :/
thanks for the cleanup!


> I think we should contain hit_set_count items when pg do not creat new hitset?
> reload -> trim -> reload -> trim -> reload is not necessary in ReplicatedPG::agent_work.

oh,i see. but why not set "size = pool.info.hit_set_count" before call in_memory_trim in hit_set_persist side?and then it seems no need to call trim in agent_work.


fixes #645


I like the `codeunits` stuff... just remember to keep them straight from logical characters, code points, and encodings...
Where you say `codeunits(s)*sizeof(E)`, I think that maybe should be `codeunits(s)*sizeof(codeunit(E))` (yes, I know you have `sizeof(E)` defined to be `sizeof(codeunit(E))`, it just feels wrong to talk about the "size" of an encoding...).
Also, it assumes that you can only have an encoding stored with a particular size code unit... (not necessarily true,
you can store binary strings or UTF-8 encoded strings in the lower bytes of uint16_t, for example)


This is some really opiniated commit... Breaking aliases with no deprecation warning, no explanation, no real update....


Thanks, fixed in https://github.com/almende/vis/commit/eef693113e54c7acb94dd9db187a31be8e305c47


merged gh-pages


commit: d784a6ec7fff2d9592fb23830985ba3adc2fb411


@marutosi The whitespace is intentional, because this is an ASCII art banner.

Otherwise the stars on the right-hand side would not be aligned.


quick nitpick: I'm not sure about the `MESSAGING_CAPABILITY` name: `org.wildly.extension.messaging`. To me, the value sounds like the name of an extension, not a capability. Why not `org.wildfly.capability.messaging` instead?


`ggpnp`does NOT require a parameter, even in its new form. I takes as an optional parameter the name of the branch to push to; when the branch name is omitted, it should take the current branch by default.

You may have issues with the `current_branch` function, which would explain why you need a parameter for `ggpnp`; can you please check your `current_branch` function?


Thanks for the code.   Yes, it simply warns the error.  I'm afraid the message level should be ERROR or something like more severe?

Please let me know your thought.

Regards;


Why use quotation mark? it is not a valid rule.


The commit is not pushed to mainstream, because Tp::BaseConnectionContactsInterface changes is not reviewed yet.


Μεταφραστική μνήμη ελληνικών όρων που δημιουργήθηκε από τα Έργα Mozilla , OpenOffice kai Gnome


Thanks. Let me update the spec accordingly later.


The kind of thing that proves that it should have not been pre-implemented before the real implementation, as you didn't know yet how you would proceed :stuck_out_tongue_winking_eye:


TODO(krasin): set learningRate back to 0.0001; it was an accidental change


这只是一个测试


How do you handle race conditions ?


测试一下


When the fullName var contains the path and the some file name it puts the pair (agent id and assembly name) into the internal dictionary


It's considered a bug to define a hook with a default value because hook variables can be added to before being bound.  See http://lists.gnu.org/archive/html/emacs-devel/2015-02/msg01271.html:

> All hook functions have always treated "unbound" as equal to nil, so no,
it's not considered a bug to run a hook that's still unbound.

> Basically, given that "unbound == nil", what is a **bug is to defvar
a hook with a non-nil default value**, unless that hook is predefined
(i.e. is never unbound).

If the user calls `add-hook` for this hook *before* firestarter.el is loaded `firestarter-report-to-buffer` is never added to hook, because the variable becomes bound before `defcustom` is run, and `defcustom` (and `defvar` generally) does not change the value of bound variables.

Instead the recommended pattern is to always `defvar` hooks with `nil`, and then explicitly call `add-hook` at top-level to add the pre-defined hooks.


not clear method name.
you maybe mean authorization method?
it'll look better if named log_in_as(login) I think


PHPCS: Expected 1 newline at end of file; 0 found


* FIXME: This does not appear to work


li hhhhh


Thanks for catching that.


here I see wrong method implementation
let's try to divide this method into few logical parts (with appropriate names)
for example
open_home_page
log_in_as(...)
open_projects_page
create_project(project_name)
etc


@yinghau76 Hmm, not sure how it got there... Thanks.


examples?


Indeed, that is a question that I was also asking myself. For each approach
there is a scenario that has problems.
Approach 1: only copy active UV layer
Problem: only changes in one layer are saved, so user should not make
changes in more than one uv layer
Approach 2: copy all UV layers
Problem: objects might get unneccissarily many uv layers
Approach 3: keep list of existing uv layers, only update these if the
original object has them, and add the new layers to the object
Problem:
Object 1 has 2 UV layers: UVMap and Lightmap
Obejct 2 has 1 UV layer: UVMap
You edit them using the plugin, adding Object2 to Lightmap. when you go
back to object mode, these changes are not copied.

The last scenario could be solved by using my approach, but only
transfer/copy a UV layer if any of the UVs in that layer is not 0,0
I think that would be the best approach, though I'm not sure if it covers
all possibilities.

On Wed, Jun 10, 2015 at 6:35 PM, ndee85 <notifications@github.com> wrote:

> Hey jonim8or, first of all, thanks for your additions and pull requests!
> Greatfull for your contributions.
> I am looking through your pull request.
> One thing I am not sure about yet. Maybe we can talk a bit about that. You
> have added the part that every UV Layout in the merged object gets copied
> to the original objects. Before it was that only the active layout got
> copied over. Don't you think that this way we can get some uneeded
> overhead? Your method will result in that every object will share the uv's
> later, but that may not be wanted. Before it was that only the active uv
> that was edited got copied to all objects, so objects still may have uv
> layouts that are not shared with others.
>
> What do you think?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jonim8or/Multi-Object-UV-Editing/commit/731cff7d4fc4a3b8629abb244f82cbcc81f1fb20#commitcomment-11615805>
> .
>


@yinghau76 I mean, I overlooked it, but not sure why @ishidawataru changed it like that :-)


1682_waterheater_stratified (mbadams5) - i386-Windows-7-VisualStudio-12: OK (2184 of 2214 tests passed)

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


Thanks so much, Chris! After quite a lot of battling (I guess I'm not as
comfortable with link_to  and its syntax as I'd like to be), I was able to
get the errors to go away but I haven't been able to write the permissions
to make it so that users can't view items they've been excluded from.
Hopefully we can get it figured out at our meeting tomorrow.

I've also been working on the birthday input like we talked about. It still
needs a little work though - I think the column should be date, not
datetime, and the discard_year doesn't seem to work quite as we expected it
to. Will see if I can make any more progress on that before our meeting.

Thanks again for all the help, and talk to you soon!
Rachel

On Wed, Jun 10, 2015 at 6:36 AM, Christopher Beck <notifications@github.com>
wrote:

> Once you eventually add pundit (if you do), then you can restrict the
> deletion at that level instead.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rachelcolby11/Wrapt/commit/a8ad6ae368b5763a9fa59bad2953f630be504cdf#commitcomment-11611419>
> .
>


wds test first commit


to be more consistent, probably, we can use `%{_mandir}` instead?


why you commented the assertion?


s/analisys/analysis/


Thanks for your answer.
Using `snap-installer/updateall.bat` instead of the `mvn installs` seems to be working, but I don't know if it's the correct way to build st1tb.


@waneck: This strikes me as weird: The function skips (non-core-type) TAbstract but retains TType. That can't be right, right?


if you decided to divide all tests into few files then each file(task) should include its own assertions otherwise it's not test actually


s/erronous/erroneous/


so we can dispense the `%{_mandir}/man8/*.8*` lines? and i'd suggest use `%{_mandir}/*/*` instead.


I'd like to ask RMs (@weltling, @KalleZ) to reconsider **reverting** this change because it has been done without RFC and creates **BC break** which is very difficult to deal with.


why assertion is commented?


I think it is because Vector3d_fx is an unaligned object (defined via eigenpy::UnalignedEquivalent) and Matrix3d is aligned. So, this case is not supported by your template. You should use Matrix3d_fx for the return type

If you need another example, you might have a look at src/python/se3.hpp


s/occations/occasions/


Predicate.createIfNotEqual


Not sure what ANY is in this context, but it's not a URI thus it can't be a predicate in RDF. Therefore it can't be an object or data property. Where do you see it?


my only comment is if ```out_summary``` and ```out_signatures``` should return a ```GVariant``` with the requested data instead of the raw ```GBytes```.

Either way it looks fine to me: :+1:


I left it as just // so that anyone browsing via https doesn't get the silly broken padlock thing, is it Sculpin that's added this in?


Oh you did this already, neat, I was just checking it!


> Use popwin for *quickrun* buffers.

I guess


in line no.65, must change
int pushlengthset = 2-000; // value for a long push in mS
in
int pushlengthset = 2000; // value for a long push in mS


@yinghau76 Fixed.


Clears All Drupal Cache and adds a menu item on the main admin menu,to clear cache based on user permissions.
* Adds  permissions for clearing Drupal cache
 * Note: Clearing caches is relatively harmless. Sites might slow down for a bit afterwards while the cache fills back up.
 * Note: Note that there are sometimes caches other than the Drupal cache that can affect website performance. For example, some hosting services.
 * Note: Note that this does not clear web browser's cache, this will need to be cleared separately. :p
 * Enabling this module will add a menu link titled "Clear Cache" in the administrative toolbar at the top.


Thanks for the info.
Implemented in version 1.1.1 (http://www.spigotmc.org/resources/playertabcomplete.2631/update?update=29475)


should we check signatures if present?


Because unless I'm mistaken, `ctr_encrypt()` (in `core/lib/libtomcrypt/src/modes/ctr/ctr_encrypt.c`) handles the first block by itself, i.e. it will not call `accel_ctr_encrypt()` (this function) for the first block.


Clears All Drupal Cache and adds a menu item on the main admin menu,to clear cache based on user permissions.
* Adds  permissions for clearing Drupal cache
 * Note: Clearing caches is relatively harmless. Sites might slow down for a bit afterwards while the cache fills back up.
 * Note: Note that there are sometimes caches other than the Drupal cache that can affect website performance. For example, some hosting services.
 * Note: Note that this does not clear web browser cache, this will need to be cleared separately. :p
 * Enabling this module will add a menu link titled "Clear Cache" in the administrative toolbar at the top.


This could just be `Ted Yu`


That's what I meant. Is there any problem with running on `childNodes`?


1，注释 2.主要思路是采用持久化存储存储数据 不同的操作用类不同的方法这是思路3.如果程序作为主程序执行4.执行myclass实例的用户输入方法 5.类 6.持久化存储，存储用户资料 7.类实例化。老师，我俩努力学了好多，非常想把这个东西学好，因为他非常有用，老师麻烦您了啊


I'll figure out how to store the token.  Accepting this commit so I can include the code in the deployment effort; we can use it this way for now.


I fixed some typos in this file


It is ok to use. It uses the -DskipTests option.
The script does more than just building S1TBX, it also builds S2TBX and S3TBX.
Is a bit more time consuming.


Added an info file (needed by drupal)


This version is OK to read.


@Hideaki1 this "slug" parameter needs to be in English. Please fix.


Yes but it is tuned very aggressive. I am happy with my current configuration.


Looks good


This should probably just be `id`.


I think I saw a couple of triple patterns during debugging some time back asking for all predictes and objects for some specific subject.

Sent from my iPhone

On 11 Jun 2015, at 3:55 pm, Pavel Klinov <notifications@github.com<mailto:notifications@github.com>> wrote:

Not sure what ANY is in this context, but it's not a URI thus it can't be a predicate in RDF. Therefore it can't be an object or data property. Where do you see it?

-
Reply to this email directly or view it on GitHub<https://github.com/klinovp/pellet/commit/16e3ab9ace507a3de7f1c7706e2f91ec3a9afacb#commitcomment-11628466>.


@chelmuth: please add to staging


Attention tout ce qui est "payé" par les ménages est négatif !


Add class level comments.


@pasinskim: I had to revert this because it broke the acceptance tests:
* 17_packages/unsafe/timed/inventory_packages_update_frequency.cf
* 17_packages/unsafe/12_packagesmatching.cf

The first one can probably just be removed, since the old way of testing inventory is obsolete. But do we test it in a different way?

The second one needs to keep working, although the test probably needs changing. It's making sure that the ``packagesmatching`` function call returns the packages on the system, but it's still assuming the old way of doing package inventory.


Ah thats nice did not know that. I have taken these flags from chrisopher83. On my i9000 ffast mast increased the benchmark scores about 5 - 10%. Where can i find this article?


還不是靠劉備，哼！
好啦其實雙體表婆哥也可以用啦


A thread sleep for these type of operations are not recommended (cos there's a probability that this time is not enough), isn't there a mechanism to check by polling maybe if the deployment/generation is done? ..


No problems so far. Pushed.


Fix code formatting, indentation etc..


Googling around I am pretty sure, that is is "build script". Let's see what Matt says.


Dang. Not much usage coming from IE6/7/8, but it would be a nice to have. http://www.sitepoint.com/browser-trends-june-2015-battle-second-place/


wenn dann, entityRef == null || !.....


the commit message was supposed to be "install sass in vagrant bootstrap file"


wobei der fehler "nur" ein folgefehler war - sollte jetzt wieder gehn


/cc @jacekjursza


Noo , don't add more! :P


> Also: observed that..if creature has aggro on another npc.... "spell" wont pull him to you. ..only aggro.(i dont know if this is how it should be)

That doesn't work in @kvipka version as well.
I will investigate and see what can we do about it. Once fixed, I will push along with the patch above.


To let us quickly find a file, please use the format
yyyy-mm-dd.md
That is, for this particular file
2015-06-10.md


ok.


Maybe format it like this?
```cpp
#warning "==================================================================="
#warning "Ignore this warning if a full batch test is being run!"
#warning "A flaky test comparing against wall time is disabled for batch run."
#warning "==================================================================="

```


What's the point of adding this dependency ?


Do we need to exclude the compile messages from github?


variables do not start with upper case letters ..


This test is doing nothing. Did you forget the body somehow ?


this commit should not be part of the PR


Put proper class comments, we do not put author tags in code ..


thats it


@eishikawa change should be 変更？


Rick san,

I will check it tomorrow.

Xperia™から送信

Rick Cogleyさんは書きました :

>@eishikawa change should be 変更？
>
>—
>Reply to this email directly or view it on GitHub.￼
>


Please change  ``max_connections=300`` in ``my.cnf`` if you are using mysql.


Yes, copy/pasted from DefaultEntityValueProviderTest. Since I usually don't put any @since in test I did not tough about checking it.


These have been fixed in the latest PR.


Which deprecated messages are avoided?

At least, Travis is configured to ignore it on required tests.


Bad reflex index.


These have been fixed in the latest PR


test test


Just one minor thing -- function calls should have a space before the parenthesis. At least that's what I'm trying to stick to.


you can make a comment on a single line, in a given commit, so check the commit tomorrow for the exact location.
R

--
Rick Cogley
Mobile: 090-9959-5452 (Outside Japan, remove "0". +81-90-...)

> On 11 Jun, 2015, at 17:22 PM, eishikawa <notifications@github.com> wrote:
>
> Rick san,
>
> I will check it tomorrow.
>
> Xperia™から送信
>
> Rick Cogleyさんは書きました :
>
> >@eishikawa change should be 変更？
> >
> >—
> >Reply to this email directly or view it on GitHub.￼
> >
> —
> Reply to this email directly or view it on GitHub <https://github.com/eSolia/eSolia/commit/f4d9f56b701c588bba2d17c7bb9e809885152942#commitcomment-11628900>.
>


Comment above looks outdated.


... in between fully fledged... and a simple file system ...


Why do you want to move it?


LGTM :)


@Soullivaneuh while running qa_client_ci.sh from the sandbox (the code is used in codeship)


You don't need to change any shema to support spaces, that's the whole point. It's reusing the already existing space field as you can see in XWikiDocument#getSpace/setSpace.


@JanTvrdik Almost all bug fixes are done without RFC.


My impression is that the batch scripts provide a simpler interface -- see
the contents of the re-written batch scripts for what they would be
replaced by. Of course with 3.0 we can further simplify or change that.

@glennsarti: what do you think is the best way to provide a simple UI for
just starting and stopping a local server?

On 10 June 2015 at 15:38, Andreas Kollegger <notifications@github.com>
wrote:

> Does this work completely obviate the batch scripts? I *think* it
> probably does, so we could instead use phrasing which deprecates the bat
> files. One reason to *not* do that would be OS coverage. For instance, we
> cite Windows XP in our system requirements
> <http://neo4j.com/docs/stable/deployment-requirements.html>. If users on
> that platform can't (without installing extra stuff) use this, then the
> bat files aren't a legacy but a viable alternative.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/glennsarti/neo4j/commit/bc9fab2d6a88ea75f0f00a22f6e95300e012283b#commitcomment-11612895>
> .
>


podoba mi się nazwa commitu :smile:


I wonder how this made it through all the runs of tests even an improved version I just pushed (see 86d5fe31).


I see. Could be not use the same way for sandbox ? phpunit-bridge with weak env.


@lattera more code to cope with underlying directory structure as needed :)


This introduces conflicts with previous caja and caja-common packages built using this repo...

I thought arch-independent files should go into -common packages, no?


@Soullivaneuh just trying ;)


... Using Docker, the services receive IP new addresses on every (re)start.


According to the RFCs (2445 resp 5545) the startDt date has to be set without any timezone information and is to be interpreted as a timestamp in the timezone that is being defined. On the other hand, a possible `UNTIL` date must be defined in UTC according to the RFC.
We should not use a non-UTC timestamp as endDt and we should not try to transform the given date into UTC by applying either `TZOFFSETFROM` or `TZOFFSETTO` (we do not really know which one it is...)
A more tolerant implementation could just interpret the timestamp as being in UTC even if a non-UTC timestamp has been provided. Do you think this is preferable?


Line 22: Can you give 'herb garden' capital letters: 'Herb Garden'


@flexiondotorg: this will get in the way in GTK+3 build...


Improving the development workflow with Docker


Change date to 11th


Change


网络事件回调以 lambda匿名函数对象作为回调，它可以绑定会话对象（就无需通过id去进行某种查找--效率更高），并且将它传递给链接建立的回调。
并可在其中设置它的其他事件回调。 相当于每个会话对象都天然的拥有各自的回调。
（而原先基于id的回调设计，则必须由用户去构建对象设计和管理器，相对麻烦）


I think this solution is to simple, while the other one (lexer+parser in antler) is to complex.

What about using one of the many available JSON parsers? Jackson seems popular and friendly:
http://wiki.fasterxml.com/JacksonInFiveMinutes


I'm okay with "build script" - could you change then build description to "build script" here?


Ok but this is not efficient :)


@derickr Changes that break user code in a way that cannot be worked around (and as @bazbarfoo pointed out, this seems to be the case here -- I initially thought conditionally declaring the class would solve this, but it doesn't) are not usually done without an RFC. This change will require Carbon to change it's API and everyone using the library to change their use of this API.

As such I'd also like to have this reverted for PHP 7.0. For PHP 7.1 I can make an RFC for the underlying issue in the LSP checks (I think we have a pretty good basis here given that return types are already weakly covariant, so making parameter types weakly contravariant is quite reasonable) and we can reintroduce this and other typehints then without any concern for BC breakage.


Install the profile


Корневой репозиторий переехал ко мне.


Agreed, but most bug fixes don't result in heavy BC breaks. Also I would prefer entirely different approach to fixing the bug – **allow both string and `DateTimeZone`** instance in `DateTime::createFromFormat`, which is what most userland libraries are [currently doing](https://github.com/nette/utils/blob/v2.3.1/src/Utils/DateTime.php#L116-L117).


add master?


@flexiondotorg: also building python stuff here requires explicit ```--enable-python``` so the build-dep alone is useless.


spanserver在每个tablet上实现了一个Paxos状态机来管理副本 --->
为了支持复制，spanserver在每个tablet上实现了一个Paxos状态机


最初的Spanner在每个tablet上实现了多个Paxos状态机 --->
最初, Spanner在每个tablet上实现了多个Paxos状态机


Why is evaluate() in Search and not in Eval ? If you put it in Eval you have easier access to the private constants. And if it is static, we do not care to which class it belongs


cool


OK, fixed.


Hi @maxime-petit, can you add a check whether `vResume.asString()` is either `agent`, `no` or `yes` and otherwise throw an error? :+1:


@flexiondotorg: why reintroduce this build-dep? I didn't see any glib stuff used during the build, so I've removed it.


I'm using it to convert arrays of type int[] to Integer[] arrays. In order to use it with Assert.assertArrayEquals() method when performing tests.


Unused variable


Hi, and thanks for voicing your opinion.

To be fair, the changes in the aliases names were long discussed in the several months during which the PR was developed. Keep in mind that OMZ is a community-driven framework: you have to get involved!

Despite the philosophical argument above, I reckon it was quite narrow-minded of me to forget about the many OMZ users potentially relying on those aliases for their workflow.

Once this is resolved, we will work on improving the updating process so that kind of things does not happen again. You are welcome to suggest any improvement you think would have helped here.

Also, the aliases were indeed improved: by making them into functions, we are able to take the branch name as an optional argument defaulting to the current branch.


@flexiondotorg: using ```--parallel``` won't hurt :)


Speed up to 50%


According to the spec this is optional. There should not be an exception here.

From https://github.com/bitcoin/bips/blob/master/bip-0070.mediawiki

> optional uint32 payment_details_version = 1 [default = 1];

and from https://developers.google.com/protocol-buffers/docs/proto#optional

> A well-formed message may or may not contain an optional element. When a message is parsed, if it does not contain an optional element, the corresponding field in the parsed object is set to the default value for that field.


Could you maybe preserve a more informative error message using
```haskell
  traceSLn "impossible" 10 ...
```


Ultima actualizacion


As I was the person who added those aliases, I consider myself "involved".


`biggestRect.br()` Bottom right point
`biggestRect.tl()` Top left point


@flexiondotorg: using --parallel won't hurt :)


@Soullivaneuh don't know why I didn't done that before ... the sandbox build is now green for unit tests. Now I will start the behat test.


Remove `std::`


(译者注：如何定义足够新？)


Assignments to default values is done at definition time, therefore all invocations get the same object and not a new copy. List is a mutable type, therefore it's dangerous to use it as default value. In this specific case, it cannot do any harm, but it's a good practice to always avoid using mutable objects as default values, because someone might change the code later without noticing it. Tuples are immutable, so empty tuple -- () -- can be used here instead


you don't need any "\n"


This works nicely :+1:

The resulting email shows:

A request has been made to reset the password for the following site and account:
http://breakspear

* Email: ben@meredevelopment.co.uk

To reset your password visit the following address, otherwise just ignore this email and nothing will happen.

[http://breakspear/shop/account/?rp=rLSeshKtN7sqX0nnMY8w](http://breakspear/shop/account/?rp=rLSeshKtN7sqX0nnMY8w)


use ``findAll`` instead.


It turns out that reldeps cannot be constructed directly from all provides (namely file paths). Fixed in d1e23e4


test


I tried:

```C++
static Matrix3d_fx exp3_proxy(const Vector3d_fx & v)
{
   return exp3(v);
}
```
but I still have the same error.

In src/python/se3.hpp, to my understanding, we build a python class which is equivalent to ```SE3```. By design, it holds a ```rotation``` property of type ```Matrix3_fx```. It is then normal to have a function ```getRotation``` which returns  ```Matrix3_fx```.

My first implementation is based on src/python/algorithms.hpp. ```rnea_proxy``` takes unaligned vectors as arguments, and returns an aligned vector.


Change to `bool sendSteeringMessage(RoboyAdapter& roboy_adapter, Rect& rect) { }`
- Pass by reference
- Avoid global


Shouldn't this be 4 instead of 5?


THIS IS NOW OUTDATED!!!


Damn! I hope it's not to late to include #39???


I made some annotations. I don't have my environment running, so I can't change it myself. Altering it without testing it first is probably a bad idea.


@flexiondotorg: I thought symbols are auto-generated? And we don't need ancient symbols from 1.6 anyway.


Any reason to uppercase here and not in the other case?


That's because the output string is built in reverse order: from the top level buffer to the bottom one.
For example, `ob_start();echo 'A';ob_start();echo 'B';` gives `'B'` then `'A'`, and should output `'AB'`.


Однако с точки зрения гитхаба он все еще считается форком. Мне кажется, это не очень удобно.
Может быть, нам нужно перевести репозитарий Кирилл на вас (https://help.github.com/articles/transferring-a-repository/) или сделать ваш самостоятельным и сделать, чтобы наши форки указывали на него? (


Ok. No matter. I have just repushed the new tests and they are well appearing now.


It's better for user to select his own startup delay value, but "quickstart" works better when startup_delay is set to 0.


Thanks, I'll take that out when rebasing (also I see that this module's missing an import of `settings` at this point in the history, d'oh.)


Indeed you did add `ggpur`.

What I meant by _"get involved"_ is not _"be involved"_, I meant all interested users could have taken part in the discussion. Sorry, English is a second language.

Anyway, I did not meant to say you were not involved in OMZ; just that you could have voiced your concern (for the aliases staying the same) during the months of discussion on this PR.


We'll just use a different command :)


Are we going to be doing any look ups on this field? Is it worth adding an index?


OK yea I see what you mean with the Class Weapons And Armor sheet. It looked different that's why I though you changed it.

Out of curiosity is there a way for you to add Casting info (ie.. how many spells know and/or able to prepared per lvl (including bonus Spells), Total caster level (raised by feats and/or buffs/debuffs), Current total bonus to overcome SR) on one of the character sheets. I think it would kinda make things a little easier if people can just look at their character sheet and know the info they need. That way say someone creates a 10th level wizard with and Int of 26 that they can prep 0's:4, 1's:6, 2's:6, 3's:5, 4's:5, 5's:3 but because of feats and/or buffs that raised CL to 12 and they can cast 1 more 3rd and 5th and the DC's goes up by 1 or 2. and their bonus to overcome SR is +2 Because they have Spell Pen.

I mean its just a though about a little bit of extra info that should mostly be calculated already in Heroforge. It just writing the code or formula to show the info. The only info that might have to be added is the info for how many bonus spells you get for your high casting stat.


Consistent visibilities? Package vs. protected?


@flexiondotorg: I don't see any cdbs usage in ```debian/rules```.


РЕШЕТО


Minor convention problem - according to https://fedoraproject.org/wiki/Infrastructure/AppBestPractices, infra apps should try to follow PEP8. Argument names should use snake_case


@TSC21 @jgoppert The main reason nothing of this has gone into master is the unclear testing situation of this. Although the code makes sense, based on the comments its really hard to judge if this is ready for mainline or not.


I have no idea!! I'll see what Sqlest does and copy that...


You're right - they ignore the entire /target/ folder, I'll change that.


Consistent 'this' usage :-)


Left a few comments, looks good!


Yep! I copied that style from your (or Maxk's?) new svgb implementation


This should be [-branch branch | -revision rev | -tag tag], you cannot specify branch and tag at once. You either clone specific branch and tag at once.


banch -> branch


@markus9999: you found the fix, so I committed it on your behalf. Hope it's ok \o/ :)


Same as line 31.


supplie -> supplied
banch -> branch


alldocs.go is autogenerated, this is the trouble here.


I'd suggest doing this as part of this first commit; it makes things a bit difficult to parse when I need to figure out whether `referrer_policy` is just a variable that hasn't been renamed, or whether the function is entirely unused. :)

Alternatively, all these tests run on the same server: if you're reusing 90% of the code, you can just rely on the file in the `/referrer-policy` directory (e.g. ` <script src="/referrer-policy/generic/common.js"></script>`) as a first-step towards extracting something common.


it would be better to fix the indent here.


Cześć, czy mógłbyś napisać na czym polega ta zmiana? Po opisie widzę, że chodzi o wyświetlanie simulation runów z tabeli progress - a już takie coś mamy.


class has 2 variables: key, text and 2 methods: encrypt, decrypt


(Can you give me a hint about what portions of this file are new? :) )


I think it's better to put this outside the if-else block. i.e. print the download origin whenever a redirection happens.


It seems I have a bigger problem here. I can go straight to $Bio::Root::Version::VERSION but that only appears as of 1.5.0 (or thereabouts), and Bio::Perl exists in older modules but has no version either. I don't think it's possible for me to introspect the BioPerl version available if it pre-dates 1.5.0. Can I cut off the remaining 1.2.3 users?

Bio::Perl is always present but doesn't always have a version, code should do its job anyway?
$Bio::Root::Version::VERSION is only present after a certain release. Code works properly, but not on old libs.
$BioPerl::VERSION is only in 1.6 . As above but less tolerant.


Right, `matches()` was just new, but I'm not seeing an issue with it otherwise.


@flexiondotorg: same as in caja - introduces conflict with previous dev version...


Twitter Sign in


Ok, no idea how to change it. The .gitignore file already seems to exclude the entire target folder. Any ideas?


Ah yes. I tried various ways to do this. The problem is that `.done()` does not return the promise - it returns `undefined`.

So you have to create a new promise which is left in a deferred state and then resolved or rejected asyncronously depending on whether the `expect()` statements throw an error or not.

Does that make sense?


@flexiondotorg: I thought deps between main and -common package should mention exactly the same version...


As you need a Parameterset object and the specified loss function as parameters to pass to UnsupervisedModel.__init__  you would need to create those prior to the call.
That would mean either moving the whole init_exprs method into __init__
or (as we call init_exprs in __init__ anyways) store all necessary variables that are created in init_exprs as the object's attributes and then call UnsupervisedModel.__init__  after calling init_exprs  withing this __init__ method


@cstrahan probably due to https://github.com/ghcjs/ghcjs/issues/348


If it's a 3 byte image, the 4th byte will be overwritten in the next run of the loop anyways, so you don't actually need the additional &-operation, right?


Good point, I'll do that.


Ah yes. I tried various ways to do this. The problem is that .done() does not return the promise - it returns undefined.

So you have to create a new promise which is left in a deferred state and then resolved or rejected asyncronously depending on whether the expect() statements throw an error or not.

Does that make sense?


Oh, sorry, I misrepresent what I meant. I meant that the before the patch it should not always be double digit (as per %b definition). With the change I made it is always 2 digit.
Now, hour numeric should be two digit in fr and one digit in en-US.


1682_waterheater_stratified (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (2186 of 2214 tests passed)

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


@flexiondotorg: no need to reintroduce this - see https://github.com/mate-desktop/debian-packages/commit/5a6d8ecdeebe730519565c796869892d45b4c3ff.


ahh I missed my mail notification. Yeah I think `python:2.7` is a smaller box.


Please don't get me wrong, but I strongly believe methods like this serve only to clutter the framework's base-code. It's clearly a debugging method which shouldn't be here, instead the developer himself should use proper tools to extrapolate returned values. I see it is already merged here, and doesn't particularly bother me besides the fact I just stated. Respect. :)


@k-alexandrovsky

Сначала я решил не заморачиваться на счет форка. Теперь тоже получил несколько неудобных моментов. Но я не знаю как поступить лучше. Репозитарий Кирилла уже отстаёт от моего. Я бы видел решение таким

- Кирилл подливает все изменения ко мне
- грохает свой
- делает от форк от моего

Но получается, что все тоже должны поступить аналогично.

Сейчас сделать https://help.github.com/articles/transferring-a-repository/ кажется единственно правильным. Для него необходимо
- забрать всё моё к Кириллу
- грохнуть репу у меня
- сделать перемещение

Кирилл ?


a name was given to the y-axis label not the x-axis.


The solution would be to create a chrome plugin that does this :)


Thanks!


Done


Jose, porsiacaso he cambiado los datos del saludo, no los vuelvas a cambiar


Please don't get me wrong, but I strongly believe methods like this serve only to clutter the framework's base-code. It's clearly a debugging method which shouldn't be here, instead the developer himself should use proper tools to extrapolate returned values. I see it is already merged here, and doesn't particularly bother me besides the fact I just stated.  Instead we could just have an accessor ```getResponse()``` which is much needed (I just created #9202).

Respect. :)

CC: @taylorotwell


I think gitigore is a local thing... did you have the new one on your PC when you pushed back to the repo?


Gracias! lo tendré en cuenta


With Mojo's Maven Plugin for GWT, you'll want to "override" the plugin's dependencies to use GWT 2.8.0-SNAPSHOT, or use the 2.8.0-SNAPSHOT version of the plugin (which depends on GWT 2.8.0-SNAPSHOT).


This one is allowed. And I'm afraid it will remains. However it might be possible to factorize this hack for various games. (half of the OI_* hacks are related to this effect).

The GS has an optimization that can't be done in GPU. The change the Fb pointer to bottom half of the depth buffer (or the contrary fb points to start, and depth to bottom half of the depth).
This way you only need to render half of the pixels. Color will be written in the top and Depth will be written in the bottom.

It will be cool if we manage to find a hack that work for all games. The issue is that we don't know the half size of the framebuffer. Otherwise it would be too easy. Maybe we can infer somethings close to it with the buffer width.


It only appeared on the constituency detail page (unless there turns out to be a validation error) so the constituency was implied.


Shouldn't those dependencies use `<scope>provided</scope>`?

Also, instead of copy/pasting the `<version>`, you can just use a `<dependencyManagement>` on `com.google.gwt:gwt` with `<scope>import</scope>` (and `<type>pom</type>`)


Wouldn't be `!atMostOne(branch, tag, revision)` be better?


test


What does this test? I don't think the spec says anything normative about forms.


At the moment, we don't think we'll have a lookup. If it changes to be the case in the future we'll add one in.


Yeah, point taken - this is a bit horrible. I'll add some comments, and think about how to make the "party set" code more obvious.


@MineStarPE https://github.com/open-source-parsers/jsoncpp


But you can do -branch bbb -revision yyy ? if not, then I'll do
atMostOne(branch, tag, revision)

On Thu, Jun 11, 2015 at 7:12 PM, Dávid Kaya <notifications@github.com>
wrote:

> Wouldn't be !atMostOne(branch, tag, revision) be better?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/constabulary/gb/commit/897092c1a04350770c970e17672e4662a291efb7#commitcomment-11629637>
> .
>


Is this really separate from `<audio>` and `<video>`?


Yeah, I hope there's a faster way to get correct text rendering. This is going to be my next area of research probably.


I might be completely misunderstanding, but aren't we using the Jinja version of the template...?


Great.
Since this is used only in tests, you should limit the scope to test only.


std::string::back() was introduced in C++11 standard.
We should avoid pulling C++11 requirements for single use of std::string::back().


I changed it slightly, thought it might be a formatting error but don't think it was. Tried a commit, but it didn't wipe the /target folder form github.


Should be `provided`, not `runtime`.
And if it works with `runtime`, that probably means you actually don't need it (you might very-well have it transitively –e.g. from commons-fileupload, and/or appengine-tools-sdk–, and then you'll want to check the scope and possibly override it to `provided`)
`mvn dependency:tree` to the rescue!


Have you read the announcement in Community Group about this? There is explained why


This function is only added in dab90a0ff2c4652e92162e9de8568aa9ca8539ae


let's discuss on IRC, I feel like we can't understand each other :D


Added Entity Classes


Does it write it in such a way that in memory it crosses the barrier in to the depth buffer area? because we could check if the position overlaps the depth buffer pointer position.


@flexiondotorg: ```dh $@ --with python2``` in ```debian/rules``` implies build-dep on dh-python...


Yeah, and I suppose `election=election` actually would make no difference anyway :)


You can't, you either
`git clone --branch <branch | tag> $repo --single-branch`
or
`git clone $repo` && `git checkout $revision`.


@flexiondotorg: not only headers - also libmate-menu.so and libmate-menu.pc.


This list doesn't match the list of resources in `spec.src.json`.


I'm not sure if territory is really the right word, but everything else looks okay.

TODO (just a remark for core team)
- [ ] German translation


@flexiondotorg: why remove this field? It's a library package...


Fair enough, that makes things easier

On Thu, Jun 11, 2015 at 7:16 PM, Dávid Kaya <notifications@github.com>
wrote:

> You can't, you either
> git clone --branch <branch | tag> $repo --single-branch
> or
> git clone $repo && git checkout $revision.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/constabulary/gb/commit/897092c1a04350770c970e17672e4662a291efb7#commitcomment-11629701>
> .
>


it stills throws an exception. are you sure that it will break existing workflows? can you provide an example?


I actually think this doesn't need to be changed; Feedback needed ;)


I'd prefer we keep this out of this branch so that all the regression test branches are focussed on doing the minimum required to add the tests.


I note this is replacing election name with post label, not constituency name like the other changes. Presumably this is because an election could have more than one post, if I've understood correctly, but seemed a bit odd with the other changes in the commit and could be explained.


@flexiondotorg: that essentially removed ```--parallel```.


to be honest, the pom files (and other configuration files) was auto-generated by the archetype I've used :-)

I will do some cleanup by following your advices


This will be a cross-origin request when sent from an HTTP origin (which will cause problems for XHR unless you're sending CORS headers, which I don't think you are?). I'd suggest using the same origin on which the test is executing (as we'll always be able to make requests there).


Ditto.


How about using `com.google.gwt.xhr.client.XMLHttpRequest` and just use JSNI to call the `send()` method with the `FormData`?

Also, isn't all this already available in Elemental? I understand the appeal of JsInterop, but this is screaming for "Elemental 2.0".


Personal opinion: `(propertyId < 3)` should either be directly in `getMPbalance`, or nowhere at all.


How to use the project in daily life?What is the detailed means to make use of this application?Could you please give me some instructions?I sincerely need your generous help.


`git clone --branch <tag> $repo --single-branch` puts repo into `Detached HEAD` -> branch will be "HEAD" in the Manifest.

`git clone --branch <branch> $repo --single-branch` clones specified branch and `tag` will be empty in the Manifest.

`git clone $repo` && `git checkout <flags...> $revision` puts repo into `Detached HEAD` -> branch will be `HEAD`, tag will be empty


yaha pe te.getExceptionCode karna , exception code dekh kya hai

try {
accessToken = twitter.getOAuthAccessToken(requestToken);
} catch (TwitterException te) {
if (401 == te.getStatusCode()) {
logger.severe("Unable to get the access token.");
} else {
te.printStackTrace();
}


Did you test this?


@tchap Why is this check even here? Shouldn't there always be a message about what's happening?


As #2005 should have fixed it here...
Left the chnge commented


Since this is the only return path, and no "meaningful" (literally) information is returned, `set_wallet_totals()` might as well be `void`.


Nit: `std::map`, `std::string`


I'm still have an issue with Propel where every object has a __call method.

If I use a column with the template option, the exception below is thrown because of the condition method_exists($object, '__call'):

    Twig_Error_Runtime: An exception has been thrown during the rendering of a template ("Call to undefined method: Name") in "SonataAdminBundle:CRUD:base_list_inner_row.html.twig" at line 18.

    /**
     * Fields to be shown on lists.
     *
     * @param ListMapper $listMapper
     */
    protected function configureListFields(ListMapper $listMapper)
    {
        $listMapper
            ->add('Name', null, array('template' => 'TaladOrderBundle:Admin:list_order_line_label.html.twig'))
        ;
    }

I think columns with custom templates could be considered as virtual too.


Think it's sorted now.


Earlier, I was wondering about this one. As far as I can see, it's only used by one UI view. Could it be moved there?


I submitted a separate PR with this commit https://github.com/alphagov/smart-answers/pull/1696.


How about just using `HttpServletResponse#getWriter()`?


"Make static_data election specific" - it doesn't cover PartyData.


Wonder if we should keep the greeting hidden on mobile. What does it add?

If we're trying to show the user is signed in, then could move their name to the "Sign out" link?

![screen shot 2015-06-11 at 10 24 51](https://cloud.githubusercontent.com/assets/282788/8103893/2d2010fa-1024-11e5-9ddc-b6d0699217b2.png)

Equally happy to keep it – just thinking out loud.


Thank you for your review!


@ksooo sorry about that, go ahead and merge your PR and update the changelog. We don't need to bump the version twice since this latest bump hasn't been released yet.


please don't commit this kind of modification


Those two comments are two excellent catches, thanks a lot for those! I've updated the code, and added/improved the tests, please let me know if it's good now.


That was only needed for IE6/7 and they're no longer supported in the version of GWT we use here.


@mbernat Yeah, this is a bit weird, but the thing is that the rollback function is sometimes returned from an interface and there is already a log message printed as a part of the rollback function. That cannot be really pulled out because these messages are specific to the interface implementation. Of course, I could specify the task even though there is already a message in the rollback function, that that is only polluting the output...


> Traditional '-h' / '--help'?

I think displaying --usage when no args were present is just a thing I like about the programs that does it, because I frequently forget what arg to give:) Both tar and unzip shows usage or show a line referring to it. bc and testdisk simply runs the program without arguments if no argument is given.

There is no definitive answer here, and we do have --help | -h | --usage | --version and -v, so to my mind we're already following the standards that are put in place, meaning it's up to us:P


Merging


Hi @rfeathers, I have made those changes for you.


nice idea :-)


Nice :+1:


Update to something better, even if just "Css2Gss".

BTW, you can strip the meta charset down to just `<meta charset=UTF-8>`, and the `type="text/javascript"` on `<script>` is useless as well.


I see it this way.
rt    = new array[32][32]
dss = new array[32][32]

rt = db[0][0]
dss = db[16][0]

However you right is is possible to do
rt = rt[0][0] and do a clean of [64][32] pixels to clean both the RT and dss in one pass.

In my opinion the purpose is to reduce the number of fragment shader execution. Much more costly than geometry. So I guess they use the first way.

Maybe we could check that the draw call is a sprite which a color of 0 and a depth of 0 (so at least we are sure it is a clean). Then we can nearly compute the height. I mean if you have a 640 width pixel buffer. There is a high probability that half size will be around 512/2. Hum buffer must be aligned on page boundary, that mean height must be a multiple of 64.


How about `rename-to="css2gss"` instead?


My bad on the nextup - it's something I've been playing with for a few weeks, but as I haven't managed to get anything like a decent UX I (thought I) removed the code after I branched - obviously I just removed the UX code, not the code to actually build the widget. Sorry.


I don't get why Guice is used here: there are only 2 servlets and they don't use DI; just declare them in the `web.xml` and save the Guice overhead.


My goals were:

- To play a bit with JsInterop :)
- Provide an example of a running application using JsInterop to the community

Now we can maybe discuss the fact this is a bad example because people will be tempted to use JsInterop everywhere when Elemental makes more sense. (adding comments on classes could fix that)


In the docs of reveng, it say ARC is often the definition of CRC-16.  But its name is still ARC?


This is the documentation repository.
https://github.com/serverspec/serverspec.github.io


绍兴长宽高程序


ss


LGTM


Hi,  when  I hook  OpenPrinterW, I find  it does not work  under  x64 .
Test Code:
typedef BOOL  (WINAPI* PFNOPENPRINTERW)( LPWSTR             pPrinterName,
										 LPHANDLE            phPrinter,
										LPPRINTER_DEFAULTSW pDefault);

PFNOPENPRINTERW fpOpenPrinterW=(PFNOPENPRINTERW)GetProcAddress(LoadLibraryW(L"winspool.drv"), "OpenPrinterW");

BOOL
WINAPI
MyOpenPrinterW(
			   LPWSTR    pPrinterName,
			   LPHANDLE  phPrinter,
			   LPPRINTER_DEFAULTSW pDefault)

{
    BOOL Status;
	char temp[MAX_PATH]={0x00};
	GetModuleFileNameA(NULL,temp,MAX_PATH);
	printf("%s[%d] m_sProcessName=%s \n",__FILE__,__LINE__,temp);

    Status = fpOpenPrinterW (pPrinterName,
  			   phPrinter,
			   pDefault);

    return Status;

}

//=========================================================================
// This is where the work gets done.
//
int wmain(int argc, WCHAR* argv[])
{
	HANDLE hProc = NULL;
	HANDLE m_handle=NULL;
	if(Mhook_SetHook((PVOID*)&fpOpenPrinterW, MyOpenPrinterW)){
		OpenPrinterW(L"Brother MFC-7860DN Printer",&m_handle,NULL);
		if(m_handle)
			CloseHandle(m_handle);
		Mhook_Unhook((PVOID*)&fpOpenPrinterW);
	}else{
		printf("%s[%d].OpenPrinterW hook fail\n",__FILE__,__LINE__);
	}
	return 0;
}


This commit version need to be reviewed because of issue.


espeically this part


bottom also


Thanks for the pointer.
And thanks a lot for serverpec.


Did you mean to have the `!math.IsNaN(actual)` condition repeated?


I haven't looked in detail at this, but, depending on whether `set_wallet_totals()` potentially inserts *all* balances, even those, which are sort of unrelated to the wallet (= zero), or only those, which are relevant, then it might be possible to get away without an extra list for properties.

Say for example the wallet is associated with the following balances:
```c++
global_balance_money_maineco[1] == 100000000
global_balance_money_maineco[3] == 50000
global_balance_money_maineco[4] == 129322423
// ...
```
Then iterating over `global_balance_money_testeco` should be sufficient (also for retrieving only the properties):
```c++
for (std::map<uint32_t, int64_t>::iterator it = global_balance_money_maineco.begin() ; it != global_balance_money_maineco.end(); ++it) {
    uint32_t propertyId = it->first;
    int64_t balance = it->second;
}
```