You need to have both a case for Just and a case for Nothing. In this case you can add a line:

```
Nothing -> nil
```


Init


Yes there are, I hope to improve the GUI after KSP 1.1 which includes improved Unity GUI interface


I keep getting an error when trying to push HomeworkThree to gitHub?


Okay, sounds well-considered-- I especially appreciate the semantic restriction. Hopefully it will even be something with a drop-in return to std::function<> if the proposal proceeds. Thanks for explaining!


Same as above.


I think it's better to leave it for stylistic purposes.


`fiiter` typo?


It still makes sense; it just implies FIRRTL needs to conditionally not emit code for modules.


@philolo1 I think we can just put it in the `else` clause since we have already verified that the attribute exists:

```javascript
if (collapse === 'portrait' || collapse === 'landscape') {
  this._updateCollapseStrategy(new StaticCollapseDetection());
}
else {
  this._updateCollapseStrategy(new OrientationCollapseDetection(collapse));
}
```


Ok, a couple scenarios I think we'll need to run for the Word behavior section. Running a scenario here would mean constructing a document a certain way in Word and then seeing how the XML comes out.

1. Default document, add header, save. I think you already have this somewhere.
2. Default document, add header 'section 1', add text 'Section 1', then add a next-page section break, type 'Section 2' and add header text 'section 2'.
3. Default document, add text 'Section 1', *don't* add header, add next-page section break, type 'Section 2' add header 'section 2'. I think this one will be interesting as far as what it does for the first section, we'll see :)
4. Default document, add section 1 header, section break, then remove section 2 "inherited" header (not sure what the Word options are there, might be an explicit remove or just delete all the contents).
5. I think there's one more to fill out the set but it's not coming to me right now.

Btw, as I thought about it a little more, I don't think there are any explicit behaviors to add to make this work for as many sections as there happen to be or to take care of inheritance. All the code will be off of Section (aside from the even/first page bits), and the section sequence is already in there. So once we make it work for the "sentinel" section, it will work the same for any others. The only thing we need to do is assure ourselves that we don't screw up folks ability to have Word inherit in the places they want. It will be up to them to add and remove headers in the right places to get the look they want.

----

I think we should consider the alternative you suggest from the MS API, where headers is a dictionary and you get a new header by accessing that dict with the key, maybe `headers[WD_HEADER_FOOTER.PRIMARY]`. It does have certain virtues, so I think we should add it to the mix. Even if we don't choose it, it may add some insight to our thinking. The protocol might look something like this:
```python
>>> headers = Document().sections[-1].headers
>>> len(headers)
0
    # creates requested header if it doesn't exist
>>> headers[WD_HEADER_FOOTER.PRIMARY]
<docx.hdrftr.Header object at 0x10a26a870>
>>> len(headers)
1
>>> WD_HEADER_FOOTER.PRIMARY in headers
True
    # returns existing header if it's already there, basically like get_or_add_header_x()
>>> headers[WD_HEADER_FOOTER.PRIMARY]
<docx.hdrftr.Header object at 0x10a26a870>
```


xvv


Updated.


项目初始化


sorry for the double paragraphs lol >_<`


was meaning to fix this for forever. thanks!


Je propose que ``__init__`` accepte des ``**kwargs`` correspondant à des paramètres additionnels de l'événement à tester. Par exemple, ``Consume('floorSelected', floor=4)`` ne serait vrai que si l'on consomme `floorSelected` **ET** que l'événement a un paramètre ``floor`` qui vaut 4.

Si tu stockes le contenu de ``kwargs`` dans ``self.parameters``, on aurait alors quelque chose comme ceci :

``guard_parameters = ' and '.join(['event.event.{} == {}'.format(k, repr(v)) for k, v in self.parameters.items()])``

Il faudrait alors construire le garde comme étant le ``guard`` actuel si ``self.parameters`` est vide, ou alors le ``guard`` actuel + `` and `` + ``guard_parameters`` sinon.

Il faut par contre discuter de la pertinence d'avoir un tel support pour les paramètres, car autoriser les égalités sur des paramètres ouvre la porte à autoriser d'autres opérations, qui ne peuvent **pas** s'implémenter de la même façon.


I guess the thing I'm thinking of that could happen here would indeed be bubbling-up to std::terminate and stopping a daemon. (Please bear in mind that I'm not currently familiar with whatever handling happens above this point in the code.) You don't need to re-throw after catch(...), but you're right in that it's often what you want to do since by definition you kinda can't handle the mystery exception. In this case, I was mostly wondering if that's the right thing to do because our handling only consists of printing a message and not taking any action (I don't see much we could do here besides that!).


In the case err != nil, the user can't get useful info since the log didn't print err.


Could you please bump the addon version and update the changelog? Thanks.


牛逼,你入了个电信机器啊,我以为你入了个zopo呢


belongs in the `Deprecated or removed` section below


There are a couple popular implementations, one from Herb Sutter and one from Stephan T. Lavavej. One thing that has worked okay for me is to stash the implementation in a detail namespace and use the standard implementation if C++14 is detected. Admittedly a bit of a stop-gap, but I think it's worth it. (I can also do that myself as a separate change if you'd like to move on; I'll be travelling before I can get to it, but would be happy to do so.)


`"formAction"` ?


Absolutely agreed. Some types like std:;set also turn out to generate surprisingly large nodes in the underlying data structures sometimes. Sounds good to me.


Les noms devraient être définis comme variables de classes (donc directement sous ``Condition``) et y faire référence via ``Condition.ENDSTEP_EVENT`` par exemple.

Je suggère par contre que les noms (de variables) soient uniformisés dans le code (ici, on a par ex "CONSUME_EVENT" au lieu de "CONSUMED_EVENT").


Yes, well, almost. I want to eliminate all these cases for 3.3 because it causes terrible code on all platforms. There are currently two situations where it doesn't quite work yet:

* Complex member field inits: https://github.com/HaxeFoundation/haxe/issues/3704
* Code that uses untyped is ignored by the analyzer. I plan to address this in https://github.com/Simn/haxe/commits/untyped_vs_analyzer


maybe you can do in the way
```go
pods := []*api.Pod{pod}
```


It might not be necessary to distinguish python2 or python3, at least on Red Hat
there's a global variable name $python_cmd, you could use this command instead


Can you document the purpose of this type?


I did answered to your comment which I understood as "#dispose is never called" and again it is called by the ComponentManager when the component (bundles are components) is unregistered.


@argelius i don't understand what you mean? There are three conditions, what do you want to put where?


s/currpod/existingPod/


Wrong


do we really need to run the whole setup script? It does a lot of things, which is not needed in this case. what is needed here is to install the PrepareRHUI package only


Char vector created


is it necessary to add an xml for a single package?


PHP Fatal error:  Class 'Sonata\AdminBundle\Admin\Admin' not found in /var/www/html/vendor/lexik/mailer-bundle/Admin/EmailTranslationAdmin.php on line 15


No . memcpy has the same flaw. the memmove which uses a temporary is the way to go.


It's not something urgent. I was just curious. ;)


No. The obtains message comes with the mining event which is started on line 381.


If `request.delayLoadingTime` is `NaN` (its default value, which it will be for requests of type HeadRequest and TextRequest) this will return false.

It's not clear to me what setTimeout(() => {}, NaN) should do. In Chrome, it seems o, but it's probably not what you intended.


you erroneously replaced config with confiig, note the two 'i'.


Thanks! :)


The gnome/xterm scrolling thing, I think.


Should this be `python-mock` and `python-nose2`?


Please use new lines for each option and for the terminating ';;'.
Makes it more readable, especially when further options are added.


color.xml에 색상 ID를 추가하고 임의로 구현
로그인 및 회원가입 ID는 미구현


Pour les tests ci-dessous, tu testes toutes les possibilités.
Je te suggère d'utiliser ``itertools.combinations_with_replacement`` (ou ``combinations`` tout court si suffisant) avec une liste du genre ``[TrueCondition(), FalseCondition(), UndeterminedCondition()]``.

Si tu as besoin d'instances "fraiches", stocke simplement le nom du callable ou l'objet:
``[TrueCondition, FalseCondition, UndeterminedCondition]`` et ensuite tu fais un ``map(lambda x: x(), la_liste)`` par exemple (ou via ``methodcaller`` du module ``operator`` si tu veux éviter le lambda).

La classe ``unittest.TestCase`` expose ``subTest`` comme context manager, tu peux donc facilement faire:
```python
for a in ...:
  for b in ...:
    for c in ... :
      with self.subTest(a=a, b=b, c=c):
        self.generic_temporal_test(FirstTime(True, a, b, c).....
```

Les paramètres fournis à subTest servent de documentation en cas d'erreur du test.
Si tu as besoin de stocker l'outcome du test (ici le ``True`` ou ``False``, premier paramètre de ``FirstTime``), cette solution n'est peut-être pas la plus idéale (ce n'est pas une bonne idée de génériciser de trop les tests) sauf si tu peux déjà gagner beaucoup en groupant les tests avec "True" et les tests avec "False").


Hum, you're right. There is likely a bug somewhere. @ramapcsx2 do you have any info on this DECI2 code ?


Please
 1. rebase on top of eos/master
 2. fix typo and the style above
 3. use a commit message '[infra] Fix libtoolize ...'


This is a test!


better not have such a dependency on CI code.
The src storage account information could be from the Global section in Azure_ICA_all.xml.
The dest storage account name and contianer name could be defined in the parameter of this case in the case definition. And in CI, you could set the value of the parameters according to the Env config files


I prefer a subclass
but then the bezierPath will lose `moveto(10, 10)` support, I don't really care but yeah

`endPath` is already supported https://github.com/typemytype/drawbot/blob/master/drawBot/context/baseContext.py#L141

have to add support to set a `glyphSet`

it would idd be nice to be able to draw it again in a other pen and just call it `draw(pen)` and `drawPoints(pointPen)`


Ayan, this one is in the wrong place. We should replace the entire libgsl check
by checking for gsl-config instead. I'll see what I can cook up.


Showing up in tvheadend now

[more info](https://github.com/crazycat69/linux_media/issues/1#issuecomment-188617636)


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


I had already pushed a similar commit to eos/master. You can remove this upon your rebase.


changing this to zero keeps the source open until it's closed in zip_discard


Wasn't sure which way to go. I wanted to use the LOGGER but are fine to switch to the EnsureUtil. I will change this on the branch.


ok now i see it, maybe we bring also this parts to a seperate function so we don't need to duplicate the code...


A perl-text-diff nem, csak véletlenül csúszott be a perl tengerbe.
Pótlom.


不是知道这是干嘛的啊


See also: https://github.com/grpc/grpc-go/issues/106


freed all mallocs (At least all that could be found by valgrind)
changed from `zip_close` to `zip_discard` to be sure, that the file on disk is not touched


@YDMDFGOFIGHTING  第六期整理的很好，无论是内容和格式都不需要我再改什么了。一句话介绍也有了很大的进步，继续加油。：）


This was a sync with the internal code as the code is common now between RHEL and CentOS 6 and 7.


whoops, also add a link to this PR and run `doc/NEWS-update.jl`


At least here the constexpr isn't even needed. I just tested it without, and everything compiled and passed 'make check'.


I agree with this change.


This is not added to the `package.json` and thus is not installed with the whole package.


OK, I've updated postgis_transform_geometry to accept and produce
SRID_UNKNOWN and removed the 99 business from the SQL wrapper function.  I
don't think postgis_transform_geometry itself requires documentation since
its public API (ST_Transform) is documented.  Does this take care of things?

On Wed, Feb 24, 2016 at 3:28 PM, Mike Toews <notifications@github.com>
wrote:

> If we are keeping postgis_transform_geometry, there are several fixes
> needed:
>
>    - Correct typo "tranform" in error message
>    - Allow geometries with unknown SRIDs (0) (I had to use a hack with 99
>    that shouldn't be required)
>
> Additionally, the function is not documented. Does it require
> documentation?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dbaston/postgis/commit/6edddb59958bbb2c57aeac0f58c4cc45e7942f7d#commitcomment-16308519>
> .
>


@ccfxny 233我换了电信机，之前用的华为C8813年事已高，完全不能再战了，又换了个千元机（电信机只打电话用所以要求低一点


Shouldn't that be "Relay" without the trailing s?


Two new tests included:

1. The first test creates a discrete sine wave of 10.000 points with amplitude `a = sqrt(2)` and calculates the RMS. For a sine wave of the form `a sin(2πft)` should equal `a / sqrt(2)`. Which evaluates to 1 within a certain precision limit chosen at `1E-3`.

2. An array of 10.000 points with value `np.int32.maxInt` are passed. This implicitly caused an overflow in the RMS calculation (and the RMS became negative...) since we're squaring the array. This is fixed in these new commits where we convert the np.int32 to an array of Python long types. In this case, the RMS should be equivalent to `np.int32.maxInt` because the trace is constant.


a gently reminder. AFAIK nose2 will be not ported to python3


Hi hhokawa777,

I installed it on redmine 3.0.2.stable and its not working! but on the 3.0.3.stable its work well, What I can do to be run on our current version?

thank you,
best,
Lubna


Érdemes lenne átnézni ezeket a perl modulokat. Tegnap csináltam grep-pel egy listát, hogy milyen perl modulok kellenek a depends fájlok alapján (ezekre ugye az autodepends nem működik).
Nálam az jött ki, hogy 122 perl modul felesleges, mert 224 van használatban a meglévő 346-ból.
Jó lenne, ha erre te is ránéznél, nehogy szükséges modulok legyenek törölve.


@syohex ,
コメントありがとうございます．
今回はmasterへのマージ時にコンフリクトが起きそうでしたので，masterを一度マージしようとしてしまいました．
次回以降はご教示いただいた方法でブランチを作成後`git cherry-pick`でコミットを拾っていくように気をつけます．

また，conflictがなるべく発生しないよう，今後は小さい変更量で，早めにPRするよう気をつけたいと思います．


gem 'better_errors'
エラーを見やすくするgem


百毒不会有各种火星坐标的问题么……？百毒好像用的是自己的座标系来着……


Line 4: `curl curl is my baby`.
Should read: `curl is my baby`, no?


Not necessarily. Maybe some modules work as is, but most modules won't do it, and you have to adapt them.


As I understand this is test code so it's fine.


`Math.sin` and `Math.cos` are in radiant. You use degree which leeds to thick circles.


Since the constexpr is only needed for the initialization, I will rather replace this with a static function '_tau_0()'.
Please remove this commit from your patchset upon rebase.


glyphSet: a new glyphSet keyword arg to __init__ that defaults to None.
I suggested to call it drawToPen() to make sure DB users don't guess that is the method to draw the path in the canvas :)
I'm not sure why you added moveTo(x, y) (as well as moveto) to begin with, I don't think you needed to be b/w compatible there with anything...


@LordUsagi good work!


>>> origin country resolution using the following web service,
you can choose any rest service available on internet


gfhffmnhg


It is "the wrong way around". You should use "https" especially if you are behind a firewall which blocks everything except http and may use "git" as a default if that is not the case. Also please note, that the explanatory parenthetical is added in line 10. Furthermore: note that in line 16, https// is specified, not https:// (the colon is missing).


LGTM


卧槽我逗比了:scream:，之前看了一下还是对的，再一看真的不对！


done at https://github.com/WsdlToPhp/PackageGenerator/commit/43c0e72f9bcc586c67cb4578d4b441c6079666d1


Well, that's how the composite field type is defined...


Is the `Bucket` a singleton (i.e. if another instance is asking for that bucket, will it get the _exact_ same instance?)


Excellent, thanks Danny! Yes, I think having this as the default OR procedure is best as it is the SUSY recommendations. In this case, there is little point in having a repeat of all this for 4Lep. But we will want to keep our own perform_overlap, since we want to have changes for our control regions.


mapbox 用的是 OSM 的话，所有座标应该都使用 WGS-84。国内服务的座标基本都是 GCJ-02，百毒是 BD-09，直接用的话估计都会偏不少……


**Nitpick - Code Quality**
You should avoid object types in your variable names in this case you have `columns_array`.

Please check out https://github.com/andela/code-review-guidelines/tree/master/coding-style-guides for more style guide info.


This does not seems to be correct fix
Same problem affects #10823, for example.
'binary' type should be used for PostgreSQL session data!


## Task 1

Previously, you wrote a program that used a recursive function to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `while` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `while`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `while` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 2

Above, you wrote a program that used a `while` loop to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `for` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `for`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `for` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 3

In Python, we tend to use `for` loops over different types of iterators much more than `while` loops - they're more flexible and iterators are **everywhere** in Python.  Write a program that includes 5 different functions, each of which should demonstrate the use of `for` over **different types** of iterators. Your program should include these five functions, a `main()` function that calls all the other functions, and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program includes `main()` function and "ifmain" statement (1 pt)
- [ ] Program contains 5, correct subfunctions that use `for` over **different types** of iterators (5 pt)
- [ ] Program's `main()` function calls all subfunctions (1 pt)
- [ ] Program runs and correctly demonstrates use of `for` (1 pt)


oops


aoa u see


음... 히스토그램은 왜 필요한걸까용?


This will get you into bootloop
Sammy used specific ion lib to use it. It's pretty hard to port ION to specific binary because it's related to binder


Why does it need to be public?


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


原来是这么回事，确实偏差很大……

请问除了 Mapbox 以外还有什么能够获得比较准确的座标的服务嘛？我这里 Mapbox 死活得不到座标


Still think this should be renamed to something more descriptive, such as `isImplicit()`


For the reference, back in Yii1 we used separation for different SQL engines:
https://github.com/yiisoft/yii/blob/master/framework/web/CDbHttpSession.php#L146


The change of the signature of the FillTable() here results in compilation errors of the tip of the branch. Did you push updated CutsTable::FillTable() or did I miss something?


interesting - in what situations do you use the browserSniffer?


psst.  otherwise they'll probably remove this too ;)

<pre>
$ apt-cache show python3-nose2
Package: python3-nose2
Source: nose2
Version: 0.5.0-2
Installed-Size: 443
Maintainer: Barry Warsaw <barry@debian.org>
Architecture: all
Depends: python3-pkg-resources, python3-six, python3, python3:any (>= 3.3.2-2~)
Suggests: python-nose2, python-nose2-doc
Description-en: next generation of nicer testing for Python (Python 3)
 This is the Python 3 package.
Description-md5: f3d7e24837af5a3d7f9e07bf62d44b7b
Homepage: https://pypi.python.org/pypi/nose2
Section: python
Priority: optional
Filename: pool/main/n/nose2/python3-nose2_0.5.0-2_all.deb
Size: 75558
MD5sum: 979b7c27c6057132d5e949d48efdb25b
SHA1: a691123cf07192d6ca823573b922bf4e33b5af17
SHA256: cf78ea62314afb86eee3b57c57d020a51f2109abeab7f74affb7e384c00b5e8b
</pre>


@yoanmalie hum j'ai un doute, tu es sûre ? En anglais il n'y a pas d'espace devant deux-points.


Dans le guide Google non plus : https://google.github.io/styleguide/jsoncstyleguide.xml


Function from "Cook book for R"


This rule seems conflict to external/sepolicy/domain.te
```
# Only init should be able to configure kernel usermodehelpers or
# security-sensitive proc settings.
neverallow { domain -init } usermodehelper:file { append write };
neverallow { domain -init } proc_security:file { append write };
```


Name it ICA-VERIFY-RDMA?


then:

                case 69891:                                 // Copy Weapon (No Threat)
                {
                    if (!unitTarget || !IsPlayer(unitTarget) || !IsUnit(m_caster))
                        return;

                    // Main-hand
                    m_caster->SetVirtualItem(0, unitTarget->GetUInt32Value(PLAYER_VISIBLE_ITEM_16_ENTRYID));
                    // Ranged
                    m_caster->SetVirtualItem(2, unitTarget->GetUInt32Value(PLAYER_VISIBLE_ITEM_18_ENTRYID));

                    m_caster->CastSpell(unitTarget, m_spellInfo->CalculateSimpleValue(eff_idx), true);
                    return;
                }
                case 69892:                                 // Copy Off-hand Weapon (No Threat)
                {
                    if (!unitTarget || !IsPlayer(unitTarget) || !IsUnit(m_caster))
                        return;

                    // Off-hand
                    m_caster->SetVirtualItem(1, unitTarget->GetUInt32Value(PLAYER_VISIBLE_ITEM_17_ENTRYID));

                    m_caster->CastSpell(unitTarget, m_spellInfo->CalculateSimpleValue(eff_idx), true);
                    return;
                }


Reminder to wipe `$GOPATH/pkg` and run `gomobile init` after build new binaries of gomobile.


The WebUI allows you to search for programmes (only a channel at a time
though) and set timers (including series links) for them.

Not had any need for the EPG on XBMC as I rarely watch live TV, only really
did it so I could watch recordings in the bedroom/kitchen.

On Thu, 25 Feb 2016 at 01:25 fraser1978 <notifications@github.com> wrote:

> Hello again Kev,
>
> That's fantastic, thanks for that! Excellent work mate, exactly what I
> needed. Is there any plans to build on the catchup section (if its actually
> theoretically possible) and maybe a record setting option? Also, is there
> any way to add EPG and choose programmes as required?
>
> I'm definitely going to consider looking at the webui stuff in the future
> as I have some experience helping a friend build his website with some php
> code. Its certainly an interesting subject for me.
>
> I honestly highly commend you for your work on this, as after many searches
> on google you seem to be the only person who is expanding on an already
> lacking service. So hats off to ya pal!
>
> Thanks again.
>
> Regards,
>
> Fraser
>
> Hi fraser1978.
>
> I've updated the readme for the XBMC plugin at - should allow you to get
> started
> https://github.com/kevjs1982/xbmcplugins/blob/master/README.md (Alas my
> Kodi/XBMC installs are a couple of years old so these haven't been tested
> with the latest ones).
>
> The web-ui requires knowledge of setting up webservers and PHP which is a
> bit on the complex side.
>
> —
> Reply to this email directly or view it on GitHub
> <
> https://github.com/kevjs1982/eetv-webui/commit/8cd482dbbab10c7b93363c0af4355c6ef582628c#commitcomment-16298393
> >
> .
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kevjs1982/eetv-webui/commit/8cd482dbbab10c7b93363c0af4355c6ef582628c#commitcomment-16314954>
> .
>


Why is this here?


Isn't it a breaking change?


Good catch!


umm just make sure that you two work together effectively, Ie don't kill each other unless the sacrifice has a major upside.


I just committed to the codebase the removal of the examples directory from the TCPDF package(it's ok if it gets added back when update it; can just keep removing it).


My understanding is make2cmake.cmake is only used to convert makefile to cmake. Where would be the correct place for this ?


We can use GetPreparedStatement for this


直接从 [OSM](http://www.openstreetmap.org) 上获取座标就可以（前提是上的去），但是 OSM 本身的 POI 比较少，一般需要自己知道具体位置，比较难直接搜到某个店一类的……


good )


I don't think you can run CONFIG[] keyless.


nice. one more bhoma relic out of the codebase! :skull:


You can't pass Guid.Empty into the Create function, it doesn't let you (see line 69 below this comment)

You either call the version which takes a guid, in which case you have to pass a non-zero Guid, or you call the version which doesn't take a guid.


hehe,. a Typo :sweat_smile:
Thanks, will change it :+1:


Hehe <3


Is this condition done on purpose? Shouldn't it be  ```(nextProps.children.length !== this.props.children.length)```?


TeamCity Civic Assembly :: Build all [Build 79](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1372&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


All of which can be read about in the wiki section: "Theoretical Ideas".


@shahmishal Awesome! Sorry again for forgetting to pass it along to `build-script-impl`...! :joy:


The test images are prepared by us, we cannot guarantee the name contains "HPC". Actually it's controlled by us, to run the test agaisnt which image. So it might not be necessary to detect whether it's HPC image in the case


also need handle emotion:

                case 70733:                                 // Stoneform
                {
                    target->ApplyModFlag(UNIT_FIELD_FLAGS, UNIT_FLAG_NOT_SELECTABLE | UNIT_FLAG_OOC_NOT_ATTACKABLE, apply);
                    target->HandleEmoteState(apply ? EMOTE_STATE_CUSTOM_SPELL_02 : EMOTE_ONESHOT_NONE);
                    return;
                }


Thanks


Is this changed back or just merged? Sorry, i don't get it

> Am 25.02.2016 um 10:09 schrieb PEMapModder <notifications@github.com>:
>
> Added ACTION_JUMP
>
> Thanks @thebigsmileXD
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/legoboy0215/PocketMine-MP/commit/5e0aeea8c32e531066b5b3da4f4ef3f9e8633a6f


FInal


not selecting bitmaps automatically when recording
>>>
exec: import fmbtandroid
time: 0.000 s
>>>
exec: sut = fmbtandroid.Device()
exception: Could not connect to device(s): emulator-5554.
  Traceback (most recent call last):
  File "/usr/local/bin/fmbt-scripter", line 1919, in _fmbtExec
    exec statement in self._fmbtEnv
  File "<string>", line 1, in <module>
  File "/usr/local/lib64/python2.7/site-packages/fmbtandroid.py", line 470, in __init__
    ", ".join(potentialDevices)))
AndroidConnectionError: Could not connect to device(s): emulator-5554.
>>>
time: 12.626 s
>>>


@alonsogarciapablo I think this should be dynamic


should better be `$this->assertFalse(ArrayHelper::isTraversable(false));` to have some sense ;)


do you think we should have a setting allowing you to use a hash of the grain key, to avoid invalid names?


## Task 1

Previously, you wrote a program that used a recursive function to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `while` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `while`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `while` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 2

Above, you wrote a program that used a `while` loop to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `for` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `for`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `for` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 3

In Python, we tend to use `for` loops over different types of iterators much more than `while` loops - they're more flexible and iterators are **everywhere** in Python.  Write a program that includes 5 different functions, each of which should demonstrate the use of `for` over **different types** of iterators. Your program should include these five functions, a `main()` function that calls all the other functions, and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program includes `main()` function and "ifmain" statement (1 pt)
- [ ] Program contains 5, correct subfunctions that use `for` over **different types** of iterators (5 pt)
- [ ] Program's `main()` function calls all subfunctions (1 pt)
- [ ] Program runs and correctly demonstrates use of `for` (1 pt)


Problem is if you directly create the pointer without assigning the object first to somewhere else the GC could delete it before you access.


More domains:
https://hg.adblockplus.org/easylist/rev/98aed4421ab0
https://hg.adblockplus.org/easylist/rev/ae10a729df9f
https://hg.adblockplus.org/easylist/rev/130945f62599


0038b8ec8c3f59358445ea46bbfe1b2e1ae412b5 seems to work right


Do you mean adding some inline CSS programatically to the layer with labels?


instead, we could check the HPC related configurations in /etc/waagent.conf, to decide skip the test or run the test


fgfgfg


fdhgfh


fghgggggh


yes


this cannot  be done with script for the creature it self? creature > on spellhit > add flags?


fgggg


Indeed. It's maybe too early. I remove this commit.


fdggfhfgh


clone is gone from the addon code now so this can be removed


Typo: transtation

https://translatewiki.net/wiki/Thread:Support/About_Mantis:Codev-1c1449-You_can_help_on_transtations:/inh


remove the commented line


**Nitpick - Code Quality**
Avoid bang (!) method names. Prefer descriptive names.

Please take a look at the ruby style guide.
https://docs.google.com/document/d/1V1YFusbUsG8rBd-XcS9gc31xI45s4DMQIrheMIeRqb8


can we have a test for this? In which situations does this fail?


You are right, sorry i did not notice line 69 below.


The default is already set through setAgentsOptionsDefaults() in CoreMain.cpp.


ここのオブジェクト生成の箇所、長くなっちゃうけど改行したほうがいいね。


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.5 [Build 111](https://teamcity.labkey.org/viewLog.html?buildId=305539&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 10, passed: 40 Build time: 01:10:27
### Failed tests
```
org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

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

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

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

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

```


Hi! How about avoiding allocation like so? Storing pointer to object in C and return acceptable for iconv **char.
~~~.go
/*
char* c;
char** store(char* inp) {
	c = inp;
	return &c;
}
*/
import "C"
func byteSliceToCPtr(b []byte) (c **C.char) {
 	c  = C.store((*C.char)(unsafe.Pointer(&b[0])))
 	return c
}
~~~
Also, isn't passing var instead of pointers into func is really passing a copy of var, that is allocating memory for this copy?


Was soll die Klasse tun?


TeamCity Civic Assembly :: Build all [Build 79](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1372&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 10 (7 new), passed: 198 Build time: 00:11:04
### Failed tests
```
Venue tests: Venue type: Should Delete an Venue: <no details avaliable>

Venue tests: Conference place type: Should Delete an Conference place: <no details avaliable>

Venue tests: Parking type: Should Delete an Parking: <no details avaliable>

Venue tests: Concert hall type: Should Delete an Concert hall: <no details avaliable>

Venue tests: Picnic area type: Should Delete an Picnic area: <no details avaliable>

Venue tests: Hike trail parking lots type: Should Add an Hike trail parking lots and CONTINUE: <no details avaliable>

Venue tests: Hike trail parking lots type: Should Delete an Hike trail parking lots: <no details avaliable>

Basic Message Campaigns CRUD Actions: Should Add Message Campaign: <no details avaliable>

Basic Message Campaigns CRUD Actions: Should Not Edit Message Campaign: <no details avaliable>

Basic Message Campaigns CRUD Actions: Should Edit Message Campaign: <no details avaliable>

```


In task 1 the value it gives is 1001 instead of 500500. In task 2 there is a small mistake: "for i in range(0-1001):" should have been "for i in range(0,1001):". I tried several times but Task 3 does not run in my laptop.


ここのオブジェクト生成は長くなっちゃうけど、フィールドごとに改行したほうがいいね


If two caches are instantiated in the same JVM and are supposed to store data in the same bucket, then they *should* use the same `Bucket` reference. Come to think of it, it may be too coarse a unit of synchronization (blocking other keys and even other caches)...


## Task 1

Previously, you wrote a program that used a recursive function to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `while` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `while`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `while` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 2

Above, you wrote a program that used a `while` loop to compute the sum of the integers from 0 to 1000 (inclusive).  This time, I want you to accomplish the same thing, but using a `for` loop. Compute the sum of 0 to 1000 (inclusive) in a function using `for`. Return the sum of these numbers to the main loop and print the result. Write the program to include a `main()` function and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program computes sum using a `for` loop in it's own function (1 pt)
- [ ] Program correctly computes and displays sum of 0 to 1000, inclusive (1 pt)

## Task 3

In Python, we tend to use `for` loops over different types of iterators much more than `while` loops - they're more flexible and iterators are **everywhere** in Python.  Write a program that includes 5 different functions, each of which should demonstrate the use of `for` over **different types** of iterators. Your program should include these five functions, a `main()` function that calls all the other functions, and the `ifmain` statement.

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program includes `main()` function and "ifmain" statement (1 pt)
- [ ] Program contains 5, correct subfunctions that use `for` over **different types** of iterators (5 pt)
- [ ] Program's `main()` function calls all subfunctions (1 pt)
- [ ] Program runs and correctly demonstrates use of `for` (1 pt)


Very nice :)


I could make it protected probably if it's going to be used only in tests, but sometimes users may want to know the underlying couchbase id (since they can access the native cache implementation, why not give them access to the native key?)


LGTM


rebase leftovers?


Unlikely, it is environment specific.


Thx :)


OK, I get `This is an test.\n\nThis is a test of of language tool.\n\nThis is is a test of language tool.\r\n' (notice `\r'). This is because there's no EOL in the input string.


Woops =)


please comment out `RequiredDuringExecution` related logic and add comments with "todo: ****", you may refer to the node affinity for a detailed todo description


sorry I was confused by the old nose


Maybe, should be (linEnd - linStart)/16


@chelmuth: Could you apply this to staging?


I got even closer to that in the my other branch, which continues refactoring. However, there's some vtable-specific stuff I'd like to keep in its own module.


Yeah I spent whole day today fixing the site. It was intermittent for hours. Can you try again now?


Thanks. Btw I tried getting a slice out of a literal array;

    offset_cache.insert(MASK_OUT_X, &[0, 512, 1024, 1536, 2048, 2560, 3072, 3584]);

but that failed on lifetime issues, the array didn't live long enough, so I hade to move creation of the array to before creation of the offset_cache. Do you see why?


Hey thanks for the tip, I was trying various syntaxes before figuring out it was the *.cc file type that Todofy wouldnt detect it :)


probably more consistent to have an `int` there, not an `Integer`?


不要用 `h` 這樣的縮寫，直接用 history，變數名稱不要縮寫


mmm, I prefer another way to implement it, try to add a new rule:
7 => "bang"
and maybe you will understand why.


@jreback I just want to make sure you were notified of my comment above.  I am not sure if GH notifies all participants, if we're just commenting on code.


なんでここってラップ関数っぽい実装になってるの？


@springmeyer - hmm, we shouldn't include *.cpp, should implementation go to *_impl.hpp ?


This is now dead code, because variable exports are printed for eval instead of reaching here.  If preserving this behaviour is still desirable, maybe add an option flag (`--shell-eval`) to switch the eval functionality on, so the default behaviour with no `command` can remain running a shell.  I will add it to my patch set when I get back to it.


Change font size and color.


Oh yeah, definitely! Was a bit too lazy here...


fetchStatusList()はオブジェクトを直書きしないで、画面ロード時にサーバーから値をうけとったほうがいいね


Can you do a minimal repro over here https://play.rust-lang.org/ and send me the link (Press the Gist button) and I can take a look.


dfgdfgdfgdfg


I'm not sure this is quite the right fix. Once we set current_tab = NULL then if redraw_ui gets called then it can switch to the next pending tab while the previous tab may not have closed yet.

I've made a PR [here](https://github.com/rib/gputop/pull/108) which, based on your description here and IM the other day I think maybe should also address this issue. My local driver/gputop setup is shot a.t.m though, so can you maybe check it works for you if you get a chance?


Task 3 does run last two functions. it shows 'numpy is not defined' , you could have simply done 'import numpy'. Otherwise, all are fine.


think we can remove L116-L122, those are ManageIq specific


focusedValue変数の使い方について後で教えて下さい


Thank you :)


grayOutClosestTableRow（）は未使用かな？


I don't like it, I prefer separated test with more clear list of number: es. (3 * 5, 'FizzBuzz')


Ah no, that was accidental.


We can remove L69 and L70, ManageIQ specific


> git pull AliceWonder -u master
git checkout master
git merge --squash AliceWonder
git commit

> seems to have worked but I can't seem to make that appear here.

You'd have to do a `git push AliceWonder master -f`, still. Make sure to open this pull, first.


test


remove useless empty lines.


we can remove L26 - L39, ManageIQ specific


use tab here.


You can also try `git rebase --interactive a08c41dfc23234064da322159652b44684df9375` and then do the force push.


as mention above.


as mentioned above.


wqeqw


Not bad !!

Two comments though

- The script fails if there is any nested directory in the directory you are processing. The reason is glob.glob("/foo/faa") returns ALL the content of that directory, including nested directories.

```
if os.path.isfile(text_file):
   ...
```

does the trick

Well done !! Keep working !!


God bless this fix, i was going crazy!


It would be better to have:

```swift
    ///
    /// Predicts the next labels and duration for the upcoming exercise
    /// - parameter exerciseId: the upcoming exercise id
    /// - returns: the predicted labels and duration
    ///
   func predictLabelsForExerciseId(exerciseId: MKExercise.Id) -> MKExerciseLabelsWithDuration?
```

I know this is pedantic, but if we don't stick to good code style & quality, we will never have good product.


Thanks for that


This commit is not causing the problem, this one is: https://github.com/OregonCore/OregonCore/commit/5243aebeee4d47f2b4e408e3b51a5a5a3515ae69
Solution: put SpawnObject functions below the Spawn1Object and Despawn1Object functions.


`make riot` still does run clean, but `make raw` does not. This is to work around karma's file watching system being stupid. When clean removes riot.js, karma detects this and stops watching. Now refreshing the browser won't pick up changes to riot.js. Being able to reload the page to pick up changes is much faster than having to restart karma every time.


Reading this doc tells me nothing about the values in the dictionary. What happens when the ``state`` is malformed, empty; what values are expected?


1. Yeah, I think it is a good idea :-)
2. Cool!
3. That sounds good! I'll work today on the proxy, so the needs for that should soon be clearer.


Best to keep an upper bound that is known to be compatible.


We must use one approach to initialise from JSON. Either a static method, but then use same naming convention, or convenience initializer.


Either describe or remove.


@bbottema I'm not sure what the process is, but is it possible to tag this release so the version get's picked up automatically by bower? Thanks


please use make_unique - it's simply more consistent and IT HAS PERFECT FORWARDING XD XD


coala is all in english, we should keep all output in english consistently
IMO

Sincerely,

Lasse Schuirmann

contact@viperdev.io
http://viperdev.io/

2016-02-25 0:29 GMT+01:00 AbdealiJK <notifications@github.com>:

> I thought we force set the locale somewhere in coala - will change it if
> we dont do that
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala-bears/commit/2d0a1696a14c2765a5c29b6be4eae0fb47719bd9#commitcomment-16312806>
> .
>


Yup.


Suggestion: since it is a factory - `buildTaskOp`?


tip: Use os.path.join(directory, file_name) to build complete paths without taken care of last "/" ... this is good for your recipes file we are waiting for too !!


この辺も改行したほうがいいね


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 143](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=823&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


ack


could you please use english?

This class is just for testing, whether the test setup is working properly.


Will there ever be another factory? Or is it simply `TaskOpFactoryImpl`?


Because the key has already been "processed". Assume

```java
@Cacheable
public Foo getFoo(String id, String anotherId) { ... }
```

Your `key` argument is a combination of `id` and `anotherId` so it's not something you'd find "right the way".

(Unless I am missing something)


You forgot the release tag.


the export has to be in `circle.yml`
Follow the examples for node modules or cabal. The export command must be in .circlerc


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 143](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=823&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:01:17


@drewzh ohh, sure can do. It's better to just switch to NPM dependencies though, Bower is becoming a thing of the past. In the meantime I'll tag the release!


These classes are not equivalent though, but for our current purposes we could possibly ignore the edge cases.


sakráá :D


With this set to any value over 660 ( and even 660), the design looks really strange at a screen size of 660 - 700.


Relevant: https://github.com/dotnet/corefx/issues/1784


Can we do `typeof<Suave.Utils.YoLo>` perhaps?


@javisantana please take a look at [a8885a73bb7f49b37dbf1213a2878252920acbdd](https://github.com/CartoDB/cartodb.js/commit/a8885a73bb7f49b37dbf1213a2878252920acbdd). I reverted this change and made if more dynamic...


document in the javadoc in the setRouteId method instead.


Can we move this to the `case ActorOfferMatcher.MatchOffer(deadline, offer)` match?


@boxa thanks. Will check that in sniff.
@Rushor no, it can't. The idea is that we need to remove flag and possible emote state when aura is removed. That can't be done with spell hit


This adds a scrollbar to the header...


The script is named ``Populate Metadata`` and as such is displayed in Insight.


Vous êtes sûr que CBI est ok d'avoir son support de cours sur l'interweb?


ajsdfklj


LGTM


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


Question: now that we extracted `handleTaskOp` define this method here?


opraveno


In languages where the heading is longer than one line, this icon will no longer align with the top of the text.


Should possibly restrict originationBranch to type branch and branchBase to type prefix (gitFlowConfigAttribute), but it's only used internally so maybe we don't need to worry about it


ggggg


updated the new_app method for uploading icons(optional parameter)


@vilchik-elena Are lines 24-35 relevant to the test?


Invalid indentation


:+1: That's a nice and clean test class!


This needs a better message "Formatting can be made better" ?


круто! 3 волны всегда


@pynicolas I think so, they show that we don't raise issue on exports, only on imports


ruby n'aime pas les `get_` comme les `is_`.

rtfm: http://apidock.com/rails/ActionController/Helpers/ClassMethods/helper_method


If no zone is supplied (i.e. `zone = None`), baton will do the query will be done in the default zone.


Although I can easily believe that this works faster than a forloop, I would appreciate if you could include some timings.


This method has enough responsibilities to consider splitting it into one or two functions. If this pattern is used across a number of services (which I believe it is) it could be moved to a library to help make this code more readable.

This is an iterative process so will not block this pull request.


Yes, this indentation has been bothering me for a while.


Nice!


This looks strange:
<img width="538" alt="screen shot 2016-02-25 at 09 50 18" src="https://cloud.githubusercontent.com/assets/275961/13315966/b0c682f2-dba5-11e5-8a93-784f79fa790d.png">


Nice tests! Thanks!


You need to have spaces between the dash and brackets, between the brackets, and after the brackts and the following text.

Like this (view the raw text):

- [ ] Checkbox


looked a bit into the keystone v3 patch, as I understand, we first try the value of `keystone admin project` and if not configured we fall back to `keystone admin tenant`. I believe this might be to support a use case where we're upgrading from from an older to a newer version of radosgw, then we can configure the new `keystone admin project` where a tenant can be configured with more stricter restrictions (as I understand the v3 keystone has more richer support for rbac, scopes etc) @rzarzynski can you confirm?


Cannot confirm that. Current master scp and sftp tests work both for me locally and Travis. https://travis-ci.org/scop/bash-completion/builds/111702925


This is awesome! :+1:


We don't use these rounded corners elsewhere in Calypso. If we want to introduce something like this I wonder if it should be a wider design decision?


Ändrade version och datum


oh k thanks for the answer


ok


Not using the `Card` here, means we get inconsistent shadows in mobile:
<img width="79" alt="screen shot 2016-02-25 at 09 56 00" src="https://cloud.githubusercontent.com/assets/275961/13316057/0f50e268-dba6-11e5-8473-8afba18587a8.png">


Done.


je comprends le concept "human readable" de la référence, cependant je n’arrive pas à comprendre pourquoi il n'aime pas le `get_` et `is_` c'est readable pour moi 😅


Thanks a million @eviltrout!


dasdasdas


I've changed the subnets but not the sizes for company privacy, but here is a copy of the input.
I'm running multiple kernels across a bunch of boxes, but here is an example:

Linux debian 3.16.0-4-686-pae #1 SMP Debian 3.16.7-ckt11-1+deb8u6 (2015-11-09) i686 GNU/Linux
ipset v6.23, protocol version: 6

```
 - "10.33.16.0/27"
 - "10.33.0.0/21"
 - "10.33.15.201/32"
 - "10.33.15.200/32"
 - "10.31.5.0/29"
 - "10.32.5.0/24"
 - "10.33.15.30/32"
```


need to remove namespaces like this?


develop (energy-plus) - i386-Windows-7-VisualStudio-12: OK (1569 of 1569 tests passed, 0 test warnings)

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


[![St. Stephen Borne Away by His Disciples](http://api.artsmia.org/images/123796/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=123796)


Grammer and typo changes, still errors at end which need correcting


This blew up with:
`fatal: [spoton2-hubhotspot] => {'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'address'", 'failed': True}
fatal: [spoton2-hubhotspot] => {'msg': 'One or more items failed.', 'failed': True, 'changed': False, 'results': [{'msg': "AnsibleUndefinedVariable: One or more undefined variables: 'dict object' has no attribute 'address'", 'failed': True}]}`


Actually these are file permissions changes - made in error. Will revert.


Do you think adding boolean member of `NamedBuffer` for `alreadyTemplated` and `alreadyEncoded` and gating the Encode() and Template() functions? Not quite idempotent, but at least we're explicitly managing buffer state.


I will correct that globally.


About still existed copeis **handle_p, handle_music, handle_m, handle_add, handle_q, handle_rem, handle_u**
I try using something like:
```
async def handle_p(self, player, channel, author, song_url): #X4: Add shortcut command
        """
        Usage {command_prefix}p [song link]
        Adds the song to the playlist.
        """

        await self.handle_play(player, channel, author, song_url) #X4: Use shortcut to main function
```
But it don't work property (only partical). So, if I find metod to just jump to original function, I will delete full-code versions.


Perfect. This feature is a little on the complex side, which is probably why it didn't get done before :)

But I think we've got it on the run now. A couple more iterations and we should have it all laid out :)


I suspect g_assert() will print everything you passed there. Either way, you need assert when debugging (and you have gdb there to stop at assert) as final binaries are compiled with NDEBUG and assert() is no-op.


This is the ENTIRE REASON people could acess the wagering page without logging in


TYPE_ASPEED_TIMER is but not AspeedTimerState, AspeedTimer, ASPEED_TIMER


@MichaelKohler

1. Now you have the same title twice in the footer if you want to keep this i'd use two differnet words as in the en version
2. Its almost the same than the "getinvolved" section
3  It does make sense to have no translation part in the English version but wouldn't it make more sense to link the "getinvolved" title with the contribute link instead of replacing the l10n section
4. Now you have an inconsistence between the de and the fr/it version of the page
5. (If you want to change the content you should change the key from l10n to contribute for consistence)


Heyho @erebusnz,

you know, you can specify something like `Cape Town, South Africa` in your config file or on the commandline to resolve the ambiguity? ;)


Apologies, I accidentally had this other file in IntelliJ (.idea) checked in addition to demo.txt when I pushed it therefore this file showed up. I cannot emphasize enough how much I know .idea file should not be in the repository and I cannot figure out how to delete it. Thanks


Non-functional, just indicates the function is only interested in calculating file size. Comment may be sufficient, so I could remove this


ah, no, I missed that. If you needed AspeedTimer struct defined in a header, then you would have to keep ASPEED_TIMER_NR_TIMERS here as well. I just do not think you need to make AspeedTimer* visible yet.


Lolka


@sonja #35 is done. Let me know what you think, and if I should make any additional changes.


Wait the transformers are items?


I do the search match by checking first for a substring match of path, then a separate substring match of filename. So /usr/lib will match /usr/lib64 as well, and this covers both cases


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

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


Would recommend using the gcc unused attribute then. There is a #define UNUSED_PARAM  __attribute__ ((__unused__)) for it in swupd.h.


This makes the taurs show up.


Job done in latest commit!


Job done.


SEWEEEEEEEEEEEET BABY JESUS


It is not working for Ubuntu 15.10 Desktop, using qt-opensource-linux-x64-5.6.0-rc_345.run on ASRock Beebox NUC with N3150, I had to append -fPIC to QT5_CFLAGS manually inside the COMP ${components} foreach loop for it to compile.

On the other hand it is working now for Ubuntu 14.04 Server, using the same qt-opensource-linux-x64-5.6.0-rc_345.run on a homebrew server, without this fix, OTB.


Can you please add discription to config.ini:
#Specify path to override_rules.yaml file which specify override rules. Default override_rules file
override_rules = None


All the values in `ST_TabJc` need an enumeration value. Like `WD_ALIGN_TAB.CLEAR` for example.

Likewise for those in `ST_TabTlc`.

Otherwise if one of them shows up in the document it will throw a KeyError when trying to access the value.

It would be really good for us to know what they mean, so we can put that in the documentation for the enumeration. This is why we need the hand-edited XML file to experiment with and see how Word behaves with those values (including maybe puking up).

It's not entirely uncommon for this to happen, that the XML allows more options than can be placed in a document using the Word UI. That's why the hand-editing comes into it.


Looks perfect.


Aren't /sbin, /usr/sbin, and /bin all linked to /usr/sbin? So searching one should be identical to searching all.

Also, if someone wishes to search in alternative paths, they should use the -e option


/cc @nathansobo @maxbrunsfeld


Yes indeed. I'll swich them


That's a good idea.  I'll make those two commits consecutive at a minimum, but I'm currently leaning towards just making them one commit as you originally suggested.

:+1:


No worries, I'll get to more later :smile:


Should we worry about overflow? I guess one could supply a really large chunk_size, even if the slice itself has a reasonable length...?


Agreed about the warnings.

I don't remember the details off hand, but shouldn't there still be
templating so that the inputs can have fixed/variable #rows/#cols?

On Thu, Feb 25, 2016 at 6:17 PM, Daniele Panozzo <notifications@github.com>
wrote:

> There is no standard and cross platform way to issue these warnings. I
> propose we keep them like this for now, this code is used in a few
> projects. It is already in the todo list to clean them up. The proper
> solution here would be to drop all templating and go back to plain Eigen
> types. It does not make any sense to support floating point numbers here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/libigl/libigl/commit/c50570af1c1954b270d70c1e70bb82f05ed7c52a#commitcomment-16340615>
> .
>


蛤蛤，为什么不直接merge过来


And yet...feels like maybe we don't need to worry about overflow here. :)


To be honest, I'd think most distributions would do something similar, but I don't know for sure. I agree it's bad in this particular case, but similar things are done for many other code-bases, may just not work for imagemagick (and I'm not sure if the Debian/Ubuntu folks were aware of these version string kind of checks).

Understood completely about your position -- thanks for your patience with my questions/comments!


Performance encapsulates bugs as well speed for me - if it breaks it's not performing very well! Could change it to "critical for robustness" but I think there's an argument for sticking with "performance" as, as you show, integers can be faster than numerics. Factors can also be faster than characters. And data.tables can definitely be faster than data.frames.


Oh yea your other comment makes perfect sense for Clear Linux. Just if it is another distro who doesn't symlink /bin -> /usr/bin and all their stuff to /usr/bin the feature will not work as expected.


This works, just tested.


I think that's referring to how the map refers to it's values as pointers.

https://play.golang.org/p/wEL8OzWbwz


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


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


I just realized the way to fix this without a patch is to remove /32 from the input so host IP are treated as hosts instead of subnets.  It would still be nice if it could handle comparisons even with the subnet mask, but the functionality works without this patch.

My new YAML looks like this:
```
 - 10.33.16.0/27
 - 10.33.0.0/21
 - 10.33.15.201
 - 10.33.15.200
 - 10.31.5.0/29
 - 10.32.5.0/24
 - 10.33.15.30
```


We could do that. But then, that only solves the `GroupVersion` part of the problem. We still need the mapper. We could probably call `registered.GroupOrDie()` with the group version of the storage.New() returned object to get the `GroupMeta` and then the mapper from that. But that sounds even more hackier.


Tested and it breaks lodash for me.

It says

> lodash/array/compact module cannot be found


It would just go around in a loop. There should _definitely_ be a default case.


Well spotted, thanks for that.


Does this work?


There is no reaping currently done, though this is something I've given some thought to.

The best approach I've come up with is to compare the manifests on the fs to the list of complete manifests, and remove those no longer needed.  Does that sound fair? I don't foresee any conflict with other swupd operations.

Also, I'd like to tackle this in a follow-on patch. This is big enough


I've since graduated to using the coreos hyperkube images from quay and kubelet wrapper. (subsequent PR)


I think we would have to create another subnet in the same VPC and attach the virtual NIC's to it.


not for cross language p2p reference


:joy_cat: Great name!


Why do we need it to store the encoded state at all? Why not store all of our raw data (config + templates) then have some output method that takes that raw data and then spits it out in the format we want?

I'm only looking at this from a high level, but it seems a natural separation would be templates and config:

```
config := foo{}
template := bar{}
out := templates.RenderEncoded(config)
```


Again, force of habit. The `./` prefix triggers the "this is a relative path" circuit in my brain. Should we nix this convention?


Why is this check required? The function checks if a bundle name is present in the MOM, but since you are iterating over the bundle names in the MoM itself, this will always be true. Seems like a redundant check.


ya, could re-order and expose that bits.


That actually sounds less hacky to me :)

Just use Group() instead of GroupOrDie().


This block is identical to the above, right? Maybe just return the result of the `getRemote()` call (L67) and remove the catch (L72) and let the outer catch handle it?


Shouldn't it be in UTC already?


Sounds good to me if you have the lock.


never a good idea to leave db credentials in the file. Not that the database exists anymore.


Shouldn't it be `/credentials`? The only credentials directory with pem files is at the same level as the `.gitignore` file. Does `/` in `/credentials` refer to the root of the git repo or the folder containing the gitignore file that lists the rule?


Crap. Nope. `User.objects.get(email=email).is_active` should though


Oh, `LocalDate`. Nevermind.


yup!


Cyrus asked me to rename IsComplete to HasAllInformation.


Do you really want to be non-strict here?


@mrohban , the newest version of CPA now allows you to fetch and save thumbnail images as JPG


That was alot easier than I was expecting.


You're very right... I'm bad at cross platform. I will find/destroy the nix-specific path separators elements wherever possible.

The convention is that the buffers know their own name, and the `WriteFile(s)` functions take a directory. That way you can treat a single NamedBuffer or a group of NamedBuffers the same way.


Don't think you meant to add 1 here.


Could you move this into the Score model and make that implement `JsonSerializable`? `Insights` subclasses that.


'represented'


@tmat am I okay to call GetMetadata on metadata reference for perf? I can call GetMetadata rather than checking this if that is fine.

this is where new metadata is just added to workspace or an existing metadata is removed from workspace.


we could.


Making the buffers immutable would add a (relatively small) amount of complexity, as we'll have to swap out buffers at each transformation step. Let me know what you think about keeping track of whether the buffer has been templated and encoded and gating the transformation functions( Template() and Encode())


Nice!


If you want to worry about it, dividing then checking for a remainder is not too bad.

`chuck_size==0` is another fun consideration.  The standard library asserts against that up front.


Dafuq


This line's redundant, if result == RESULT_CRIT then this will always be true.


Not sure what you mean with that example. The values can be pointers or not:

```
foo := make(map[string]*myType)
bar := make(map[string]myType)
```

But why do you need a pointer to a map? e.g. `stackHolder := &map[string]interface{}{}` ?


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


Yes definitely. Honestly we could annotate the config struct with a "description" tag and generate the whole cluster yaml wholesale and get rid of one more stupid book keeping job of keeping this blob up-to-date with the Config struct.


because this check things when first happen rather than doing same thing repeatedly.


$options = new stdClass(); 해줘야 Warning이 안나겠습니다.


Kkkkkkk tudo pra não perder o streak..


I really don't think closure should be a type, if you pass a context to a function, then you have a function with extra data, and wont do anything with it, and if you don't give a closure a context, then you have undefined behavior, why not fix both this issues by defining closures as functions that have been given a context?


I do need to fetch a user from the DB, I'm not quite sure what I was thinking there. This returns 'True' by default, which is definitely not the intended result


pic of a broken weapon:
http://imgur.com/PzyZW3M


I agree. The way that the AMI / release channel stuff works has been changed in a subsequent PR.

https://github.com/colhom/coreos-kubernetes/blob/master/multi-node/aws/pkg/config/region_map.go

No, we build the region-ami map and only include AMIs that are HVM. We then simply error if there's not an AMI for a region/release channel combination.


great job


gzipReader needs to be Closed() _before_ we compared the bytes. I could extract the inner loop body into a separated function?


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


Ah, good catch. I was assuming this filtered components from other entry types in the Manifest, but it seems components are the only entries


This is the actual fix all other changes are clean up for unneeded steps in the start process.


Fields' `DefId`s are not in the ast map


Since you are actually downloading a bundle manifest here and not Mom, the variable name tmpMom is confusing. Maybe component/bundle/sub would be a better name.


Definitely - a very useful little function in certain situations. Added.


This is a proof-of-concept for skipping an issue, when the user does not want to vote on that issue. However there is a big flaw: the new issue returned is not guaranteed to be different than the one that the user currently sees.

@sonja @huynick @rhwilk I would like to have your opinion on how we can implement skipping an issue in the backend. This may involve additional logic in `getNextIssue`, as well as additional fields in the database.


I don't see errbuf used after this point. Perhaps you could print the error string if curl_easy_perform() fails?


This should be #include "swupd.h"


looks good


Remove `virtual`?


Fair point - comment added.


Checked long ago, I have not noticed side effects, since no package or library needs it.


- This is not no perm message.


This doesn't look right. How are you testing this code?


- This is for no perm message.


Well the size isn't the key thing - the fact it's close to the system's RAM capacity is. Updated.


Also, shouldn't the order of the arguments current_version and file->last_change be reversed here? current_version is always going to be greater than last_change and the functions expects arguments in the order old version followed by new version (to be able to use deltas if possible).


A quick Google search [found me this](http://developer.android.com/reference/android/test/AndroidTestCase.html).


> https://github.com/hbarroso/Co-Authors-Plus/commit/bc5efce499eab882dae68f6f98af171b1b8901a3#commitcomment-16341952

Ah - I see.


This option is not listed in print_help()


@azhornyak I'm not sure I understand the problem. Could you please raise a new issue if you think we need to change something. Comments here are just likely to get lost. Cheers.


https://github.com/dbenage-cx/freeorion/commit/d455b7eeeffd54bc236690b1a1a65714bff0c7df#diff-6ce52f5b90a662818a62fbd02ba6de8fR436 Unsure why ListScripts() is supposed to be camel case, I've changed accordingly from list_scripts() but wanted to verify.


this is wrong. read up on enablevertexattribarray in the documentation.


Can you please make separate commits for coding guidelines/whitespace v. functional changes?

Mixing them together makes it harder to review.


Please don't litter the code with log statements :(


new element MAX44006 & MAX44008


probably need to weak here


Ah yes, I'll remove this. Used during development, but not worth outputting since a failure here is non-critical


develop (energy-plus) - Win64-Windows-7-VisualStudio-12: OK (1569 of 1569 tests passed, 0 test warnings)

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


I was trying to be all fancy
_** hangs head in shame**_


Should the option be --display-all or --display-files? print_help() describes the latter.


** tips fedora **


先簡單說一下結論。storyboard的ViewController是紀錄之前ViewController.h的資料，但因為名稱改成CheckFighterViewController.h了所以找不到。（storyboard上開開第二次會變成空欄，也就是參照基本的UIViewController，所以沒有fightersArray）。再從新設定一次ViewController就可以了。


wtf?


Thanks for the tutorial Barry.

Quick question- should I have issues I can’t troubleshoot via the resources you provided while I get the repository for my capstone up and running, may I shoot you an email? Also, do I have permission to share your slides to my intern and study staff?

Thanks again!
Emily

PS. Richard Feynman is basically awesome :). He once referred to a feline anatomical diagram a cat map.

Emily Menard | MPH 2017
Department of Epidemiology
School of Public Health
University of Michigan

> On Feb 25, 2016, at 11:37 AM, Biocomputing Bootcamp <notifications@github.com> wrote:
>
> Nice job today Emily!
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/emilycmenard/demo2-github/commit/dd5908ff2d7a48823213032f2386d7d3ec7c0ef6#commitcomment-16331330>.
>


The 'V' option is not handled in the while loop.


I agree.


Field `DefId`s are not in the ast map


That's because mruby-1.0.0 does not support that syntax.  Do you need it? Or is it just a hint for something?


根本的原因是，檔案名稱變更時（ViewController.h -> CheckFighterViewController.h），裡面的interface和implementation也要跟著改。因為有些compiler參照class時是和file name綁定的。


This commit is generating false warnings for any JSNI function that calls an Object API - even if the class in which the member API is defined has nothing to do with String or Array or JSO:
https://groups.google.com/forum/#!topic/google-web-toolkit/heCTXpr-aQs


nice diff :^)


Nice work @jfoos !  Note that you're able to change the site title as you deem appropriate.  Try changing it to "Jude's Django" or something and see what happens.  Note that you might need to create an index page before the front end is able to load.


@nlf Can we increase `ulimit -l`("The maximum size that can be locked into memory.") as well? By default it's 64, but varnish uses much larger values (e.g. 82000) to lock the circular log buffer into memory.


Perhaps you can get the Perl assembler to be just good enough to assemble "as" (page 32 of 05-1-4.pdf).  Modifying the "as" where needed to meet the needs of the simple assembler or even hand assemble parts of it.  Run the result on the original "as" and compare the binaries to verify the bootstrap.


sure


The trick is to not break compatibility with the command line api


LobbyViewControllerのエラーは無くなりました！


This is also outdated since Multi-Shot generates focus now, just haven't gotten around to implementing that yet. I was just moving around code here.


sure! : )


I've looked to discussions about this on links at https://wiki.gentoo.org/wiki/Future_EAPI/EAPI_6_tentative_features#New_features (not deeply). This is good idea (epatch code is really magic in some places). But I don't think this ends with something simple and stupid, because portage devs try to create "userfriendly" tool which silently rounds all corners.


ack. Fine as-is


HTTP-redirect to HTTPS frontend-wise must naturally be implemented with web server logic hosting the frontend.


Need to remove these


74715752-Constant-Flow-Radiant-Heating-Not-Turning-On-Bug (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1569 of 1569 tests passed, 0 test warnings)

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


@jef-n , thanks.


2097-EP-8-5 (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


2097-EP-8-5 (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


> because portage devs try to create "userfriendly" tool which silently rounds all corners.

Not validating input is not userfriendly.


Yes just look to the commit above and select the color you like. I'll revert back for now. Can't understand why cm accepted this change.


Thanks for point this out, missed it when I was cleaning up.


FixGSHPweirdOutputOfAsterisksEtc (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (2126 of 2126 tests passed, 0 test warnings)

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


Done.


Moved the social.html into the _include file


OctreeTestEioMathDiff (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (0 of 0 tests passed, 0 test warnings)

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


The help information on Modal hasn't finished yet. To do it later.


OctreeTestEioMathDiff (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 tests passed, 0 test warnings)

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


@egranata does anything stick out for you?


read it


revert


You could add a check for `hdr_->version == SmxConsts::SP1_VERSION_1_0`?


Does t.deepEqual() do what you need here?


LGTM


It now describes why it's a string and not a real datetime type - analoguous to the comment about ``time_f`` ;)


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


`import`はpackageの先頭に持って行っていただきたいです。


Estavel


I would prefer to have things use the name "bundle" instead of component/subscription throughout the codebase to keep things consistent.


That would be good and proper except we are not a libc, we are trying to be as fast as possible. And we do have formats that alloc and free in inner loops. I dislike unneeded checks. Each check take absolutely no time, I know, but if we go that road we'll end up with a gazillion of other similar checks in code and end up slow for no real good. Another example is the silly code many people write along the lines of:
```c
    memset(foo, 0, sizeof(foo));
    memcpy(foo, bar, sizeof(foo));
```
While this may be good practice if you are coding pacemakers, I don't think it is good practice for a password cracker trying to be world's fastest. One can only hope the optimizer drops the memset.

For this actual case I'd rather have this. It will show up immediately for a debug but only then.
```c
    if (align < sizeof(void*))
        align = sizeof(void*);
#ifdef DEBUG
    assert(size);
    assert(!(align & (align - 1)));
#endif
```


fixed in 76fcb1f


Я может неправильно понимаю реализацию этого метода, но почему статический метод структуры считается Pure?


この変数一度しか使わないなら不要ではないでしょうか。


I have this deprecation warning showing up in my tests too, and I came across this PR in looking for a solution. I like the idea of abstracting the concept so it's not as explicitly tied to Capybara's API! I was just wondering if maybe the `BestInPlace.default_max_execution_time_in_seconds` value aught to be the catch all case, even in the extremely unlikely case that `Capybara` is defined but doesn't respond to either `:default_max_wait_time` or `:default_wait_time`. In that (albeit unlikely) case, I think the proposed method would return `nil`.

I don't really know what a great solution would be, but maybe something like this: (I know it's even more code with even more conditionals :/ but it's fairly explicit and I think would handle the cases I can think of).

```ruby
def default_max_execution_time_in_seconds
  if capybara_default_wait_time == :unspecified
    BestInPlace.default_max_execution_time_in_seconds
  else
    capybara_default_wait_time
  end
end

private

def capybara_default_wait_time
  return :unspecified unless defined?(Capybara)

  if Capybara.respond_to?(:default_max_wait_time)
    Capybara.default_max_wait_time
  elsif Capybara.respond_to?(:default_wait_time)
    Capybara.default_wait_time
  else
    :unspecified
  end
end
```

In any case, thanks for submitting the PR, and I like the idea.


`getOrElse`の上に改行があるといいます。


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


*heavy breathing* - Niiice

What about a GetVehicleCount ? (the number of all current valid vehicles) Without looping if possible. xD


Perfect, thanks!


:fire:


OMG just make a tutorial on how to disassemble MCPE game files. Win-win!


This is less than a nit, really only an opinion.  It looks like only two classes derive from `TOfferStreamBase`.  These defaults are used by one of those classes.  I'll suggest that you can improve locality by removing these defaults and using the explicit types on the class that is currently relying on the defaults.  That way when I look at that class I can see the types used for the instantiation.  Otherwise I have to split my attention to two different places.

If the defaults were used in more than one place, then I'd feel differently.  Anyway, just a thought.  Do what you feel is right.


LGTM


到 https://github.com/audreyt/archive.tw/blob/gh-pages/2016-02-04-%E5%85%AC%E5%85%B1%E6%94%BF%E7%AD%96%E7%B6%B2%E8%B7%AF%E5%8F%83%E8%88%87%E5%B9%B3%E8%87%BA%E5%85%AC%E6%B0%91%E5%8F%83%E8%88%87%E6%83%85%E5%BD%A2%E8%AA%BF%E6%9F%A5%E7%A0%94%E7%A9%B6%E5%BA%A7%E8%AB%87%E6%9C%83.an.xml 按筆形按鈕 `edit` 就可以改了~

<img width="291" alt="screen shot 2016-02-26 at 9 07 21 am" src="https://cloud.githubusercontent.com/assets/20723/13340079/63bf402a-dc68-11e5-819d-6ef26a151d68.png">

這次我先手動改。


I used ExecStart{Pre,Post} because this service configuration was copied from what is installed by libreswan normally. I'd prefer to keep it that way and use those, unless there's a particular advantage to making a separate script and using preStart/postStart.


In current version, HaarEvaluator only support AMD and Intel devices. Add minor change to allow HaarEvaluator to support Nvidia devices.


This wouldn't be problematic, would it? I think I could massage the function signatures back to what they were before (maybe).


Oh, is that all?
I think if I need to put up a blog post, I will email a word doc.


I accidentally used git apply on the last commit and it didn't show up as kevin thorley's changes


I'm a fan of putting the template implementation in a cpp and using explicit instantiation.  I like it.


This line should probably read
`$('body').css('background', 'url(images/citipix_skyline.jpg)');`

-Stephen


Interchangeably, you can also just do `$(body).addClass('sydney')`


修改了api记得通知到大家


Possible weirdness here.


That was actually @AmrMorad.


I believe so, am I missing something?


read it


Looks good to me.  I had to look at sync.Once(), that's a pretty interesting thing, good to know it exists.

I had to look into the select/case/default thing.  Also pretty useful, so if the channel is ready then it writes to it, otherwise it writes to stderr.  Very compact code, even if there's a certain amount of head scratching.


Okay, but is `Scenario: controller is extended with a helper module` an incorrect description?

If I try to do `controller.my_variable` it raises `no method` error.


For reference, here's how a simpler implementation of `globSequence` check compares:

```Go
func globSequence(pattern string) bool {
	return pattern == strings.Repeat("*", len(pattern))
}
```

```
BenchmarkGlobSequence-4	 3000000	       580 ns/op	     224 B/op	      10 allocs/op
```


@ryukinix was this supposed to be `grua-anjo`?


Thank you for reminding!


Hey @mweimer I see your adding a lot of functionality to the website.  That's awesome! I tried taking a stab at it with rails and was actually going to add the braintree functionality today to the rails version, but if you are this far along I figured we could continue to work together on this version of the app.  I chose to write my own in rails because I'm much more productive in that, but I'd be happy to work on this together. Do you think we should continue to use mongoDB? It seems like our app would need a lot of relational data and I think a SQL database would be more fitting.  Let me know what you think!


If you think the commented test code is worth keeping in-tree, I would prefer to surround the code with:

#if 0
...
#endif


I think it would be fairly straightforward to have a list of skipped issues for each user.


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


Correct, this has nothing to do with rails helpers


Done 3873bfc.


Ну, мне сложно себе представить структуру, статический метод которого возвращает результат, но при этом делает что-то еще.

Тут весь этот метод должен называться не IsPure, а NeedObserveTheResult....

В общем, ты прав, это эвристика, но я не нашел кейсов, когда она неправильна (а запускал я этот анализатор на куче кода уже).


The errbuf buffer was removed...


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


I was wondering what the best approach for that would be. I knew putting stuff in /etc wasn't right, but this is the first contribution I've made (and my first experience with Nix expressions in general), so I wasn't aware of the existing facilities. Thanks for the advice!


Never mind, you addressed this in the third patch.


@atom/core We backed out of these (and similar) changes for now in an effort to keep the public API changes as minimal as possible. Do you think they should be included?


代码质量越来越高


Now that this is abstracted out, it makes an implicit assumption that `initial_size > 0`, for the realloc logic to work successfully. Worth addressing that -- the simplest way could just be to grow `size -> 2*size + 1` on realloc, or here you could clamp it to >= 1 or assert.


I assume the reason we're passing the unused `issue` is for generic purposes?


oh, yes, sure


Ahh, it was in the other repo :)


works! thx! :D


yes, I will enable ParquetPageSource testing in TestHiveFileFormats


yes, sure


你忘記處理跳脫字元了


I will enable testParquetPageSource in TestHiveFileFormats.
No, this commit just add partition key support for date type, no support for date in parquet main code yet


shouldn't this be `entityStateName` ?


shoudnt we add a `-` befor appending the suffix or are we expecting user to pass `-` as well?
IMO its better that we add the `-` and ask user to pass only the sufix word


Please revert this.  `Optional` is a back-ported from a type that does not have this method and adding it means that we can never move to the JRE-native version.


same for these `-` needs to be added


@JBYoshi
Confusing, devs would mix that up with actual respawning from death (this change isn't for that)

@kashike
Not too bad but maybe a bit generic...I think Player should be in the event name somehow.


Hi @ulferts,
I have refactored the empty promise method, as Angular's `$q` provides a method `$q.when` for this use case.


shouldnt this also have suffix?


I was under the impression that the atoms created in the transient atomspaces were going to be discarded and only used by the pattern matcher for these potential groundings. And only after it was determined that the groundings made sense then they'd be added to the real atomspace wherein the duplicates would be checked.

I can certainly put back in the code for indexing in the transient spaces as that's fairly orthogonal to whether or not the spaces themselves are cached. I don't know what the performance impacts will be.


Have you gone back and updated every similar test that expects an error so that this matches?  I'd prefer this to be consistent.


How nasty would it be to pass in a `Journal` instead of a `Logs`?  that would be more in keeping with most of our signatures.  I don't feel strongly about it.


Encapsulate in a private method to remove complexity at the top-level


shouldnt this be camelized with first letter upperCase unless we are expecting users to pass `-suffix`


#1 fixed.


74715752-Constant-Flow-Radiant-Heating-Not-Turning-On-Bug (RKStrand) - i386-Windows-7-VisualStudio-12: OK (2122 of 2125 tests passed, 0 test warnings)

 * 3 tests had: EIO diffs.
 * 3 tests had: ESO big diffs.
 * 1 test had: MTR small diffs.

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


The conventional modern-C type to write here is `size_t` -- usually won't matter, and in particular I sure hope nobody finds themselves in need of more type-ignores in a file than an `unsigned int` (even a 16-bit unsigned int!) can represent, but in general it's the most portably correct type to use because it's guaranteed by the standard to handle any size you can allocate or array index you can dereference.

Google finds one [actually pretty good article](http://www.embedded.com/electronics-blogs/programming-pointers/4026076/Why-size-t-matters) about this, as well as some terse references like [this one](http://www.gnu.org/software/libc/manual/html_node/Important-Data-Types.html).


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


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


++ thanks for taking this!


The discussion is in https://github.com/neutrinolabs/xrdp/pull/333

I suspect you want `exit 1` if xrdp-keygen fails. `make` it not calling `set -e`, so only the final error code matters. In other words, all cases should be tested.


Who says I didnt ;)

On Fri, 26 Feb 2016 9:50 am Simon <notifications@github.com> wrote:

> [image: :scream:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/UQComputingSociety/uqcs-hubot/commit/4da76ac90906bb156e630a5b0d22ad410ce2039f#commitcomment-16341217>
> .
>


can we have a shorter option name like `angular-suffix` or something so that its easier to use. The word entity is redundant given this is passed for entity sub gen, and js also seems a bit redundant to me


The only unfortunate part is that this will store a fn ptr for `clone`, which means we'll get virtual calls. That could be expensive for simple stuff like `&i32`, though LLVM may see through it in common cases anyway. But there's no super convenient alternative that is also stable, I fear, except writing a special `Cloned` variant of `map`. So let's do it this way for now.


am adding more tests


@jdubois can you take a look at my comments please. If you are ok I would like to improve this?
cc @andidev


I think best practice for these tests is to call `weight_max` to force lots of splits -- can you add that? (Even though it's not particularly imp't in this case, since we're just building on clone)


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


Huh. Can one of these branches not actually happen, or do some of them not actually differ in behavior?

In particular, what if the file has `# coding: utf-8` -- does `PyUnicode_FromStringAndSize` do the same thing as `PyUnicode_DecodeUTF8`? (@gutworth, should the corresponding logic at the end of `parsestr` be simplified upstream too?)


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


Yeah, `Iterator::cloned` does use its own `Cloned` type, but it requires a lot more boilerplate here.  I'm willing if you think that would be better though.


`PlayerRedirectEvent`?


Sure, I'll add it.


@rluble FYI


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


Sorry about that, I was in a bit of a hurry


Yep, in fact from the implementation of those two functions, it's very clear they do the same thing as long as `s` is non-NULL, `size >= 0`, and that last argument to `PyUnicode_DecodeUTF8` is NULL.


I like the way you think.


github记录android开发之路


How about ask user copy the whole `mattermost_webook_url` to gitlab web hook? 2 cents for this:

1. no need teach user how to cut the hook identity from the total mattermost web hook copied from mattermost, just told them copy the whole web hook :-)
2. no need to re-construct ourself then.
3. no need to configure MATTERMOST_URL below


Don't you think this will make the code base unnecessarily huge? I mean, if we are going to write a new struct and validation method for `AddCard` we are easily going to write custom structs and validation methods for almost all Trello API calls. Wouldn't this be a mess?


In addition, how about change arg `webhook_url` into `mattermost_webhook_url`?


Database looks good! Ready to merge!


Why do these three happen sequentially rather than in parallel?  I think a `Mono.when()` with the three requests in it is appropriate here.


PlayedLoggedIntoUnloadedDimensionEvent

Basically its got to be that but without all those words.


I don't know if this is a stupid idea, but I can't stop thinking we should just accept a `map[string]interface{}` everywhere, serialize it to JSON and make the request with it. Wouldn't it be an easy solution?

(Instead of https://github.com/VojtechVitek/go-trello/issues/24, for example)


ahh yes, i forgot I modified that on the website as opposed to locally


Could have `PlayerRedirectEvent.InvalidDimension` or something? I could see redirections having future uses.


Found a leading paren ( and no closing paren )
You could change it form
(Note ...
to
Note:  `%spark.r` and `$r` are two different ways of calling ...


удалён тестовый метод testGetProject


I don't see curl_multi interface being used here. This function makes calls to load_manifests which calls swupd_curl_get_file thats downloads a file synchronously.


I removed the consts so that I could use the implicit copy assignment operator in `Framework/API/inc/MantidAPI/LogManager.h`. Should I revert both changes?


I can detect the difference even with my basic speaker set from amazon. :)

The filter seems to produce more natural sounding audio and I can detect the difference at the higher frequencies. It's amazing work. It doesn't seem to have any impact on the performance either. If I had speakers with a real tweeter then the difference would be even more obvious.


**Shortlog of HEAD commit is too long.**
*GitCommitBear, severity 1, section `commit`.* *Project wide.*


**This file contains unused source code.**
*PyUnusedCodeBear, severity 1, section `python`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 7.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/bearlib/abstractions/Lint.py
+++ b/coalib/bearlib/abstractions/Lint.py
@@ -4,7 +4,8 @@
 import tempfile

 from coalib.bears.Bear import Bear
-from coalib.misc.Shell import escape_path_argument, run_shell_command
+from coalib.misc.Shell import escape_path_argument
+from coalib.misc.Shell import run_shell_command
 from coalib.results.Diff import Diff
 from coalib.results.Result import Result
 from coalib.results.RESULT_SEVERITY import RESULT_SEVERITY
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 40.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/bearlib/abstractions/Lint.py
+++ b/coalib/bearlib/abstractions/Lint.py
@@ -38,6 +38,8 @@
     # TODO Look for existing classes inside python...

 # TODO Dynamic kwargs usage. Context depending signature.
+
+
 def Linter(executable, **kwargs):
            #shell=False, #TODO ??????
            #output_regex=r'(?P<line>\d+)\.(?P<column>\d+)\|'
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, lines 42 to 43.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/bearlib/abstractions/Lint.py
+++ b/coalib/bearlib/abstractions/Lint.py
@@ -39,8 +39,8 @@

 # TODO Dynamic kwargs usage. Context depending signature.
 def Linter(executable, **kwargs):
-           #shell=False, #TODO ??????
-           #output_regex=r'(?P<line>\d+)\.(?P<column>\d+)\|'
+           # shell=False, #TODO ??????
+           # output_regex=r'(?P<line>\d+)\.(?P<column>\d+)\|'
            #             r'(?P<severity>\d+): (?P<message>.*)',
            #use_stderr=False,
            #use_stdin=False,
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, lines 45 to 51.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/bearlib/abstractions/Lint.py
+++ b/coalib/bearlib/abstractions/Lint.py
@@ -42,13 +42,14 @@
            #shell=False, #TODO ??????
            #output_regex=r'(?P<line>\d+)\.(?P<column>\d+)\|'
            #             r'(?P<severity>\d+): (?P<message>.*)',
-           #use_stderr=False,
-           #use_stdin=False,
-           #provides_correction=False,
-           #diff_severity=RESULT_SEVERITY.NORMAL,
-           #diff_message='',
-           #severity_map=None):
+           # use_stderr=False,
+           # use_stdin=False,
+           # provides_correction=False,
+           # diff_severity=RESULT_SEVERITY.NORMAL,
+           # diff_message='',
+           # severity_map=None):
     kwargs["executable"] = executable
+
     def create_linter(cls):
         class Linter(LocalBear):
             handler = cls
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, lines 102 to 103.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/bearlib/abstractions/Lint.py
+++ b/coalib/bearlib/abstractions/Lint.py
@@ -99,8 +99,10 @@
                     stdin=stdin_input,
                     shell=self.shell)

-                self.stdout_output = tuple(stdout_output.splitlines(keepends=True))
-                self.stderr_output = tuple(stderr_output.splitlines(keepends=True))
+                self.stdout_output = tuple(
+                    stdout_output.splitlines(keepends=True))
+                self.stderr_output = tuple(
+                    stderr_output.splitlines(keepends=True))
                 results_output = (self.stderr_output if self.use_stderr
                                   else self.stdout_output)
                 results = self.process_output(results_output, filename, file)
```


**Line is longer than allowed. (84 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 91 column 81 to line 91 column 85*.


**Line is longer than allowed. (83 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 102 column 81 to line 102 column 84*.