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


saw approval from huonw
at https://github.com/akiss77/rust/commit/78c7faf5a61dbadb757122e35abed4ab9549898d


merging 1 batched pull requests into batch


status: {"merge_sha": "a5cec1ec4aff641a5bd29f176ea7de5b8aecc616", "rollup_pulls": [[20736, "78c7faf5a61dbadb757122e35abed4ab9549898d"]]}


Testing rollup candidate = a5cec1e

**Successful merges:**

- #20736 Fix: `libc::c_char` is not always `i8` (akiss77/pr-u8-c_char = 78c7faf5a61dbadb757122e35abed4ab9549898d)


貌似我们在这个地方没有考虑清楚。如下:

**index_controller** , 是不好去加载的。我们如果遍历了所有的 **class** 文件，势必会出现 **index_controller** 这种类相同的情况, 如果在 class_map 中尝试去加载, 肯定会出现错误的情况.

需要重新考虑解决方案。


Oh, *to /var of course


Will do.


Add div hello world


What this means "can't exec"?

On Mon, Jan 5, 2015, 12:01 PM KangDroid <notifications@github.com> wrote:

> Oh I see Thanks. But actually I have a lot of problem. First libbacktrace
> doesn't work(can't exec) So I am trying to change with Android's one becuz
> it is working fine on MacOSX. Thanks! @pbeeler
> <https://github.com/pbeeler>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KangDroid/sabermod-toolchain-build/commit/b58cf39db4169e7863780d2766e6ee20e8d580b9#commitcomment-9157882>
> .
>


I don't like the duplication (and the additional volatile) but there are already some code parts which are using the OperationThread.currentOperation as Operation. Since I didn't check them all yet I didn't touch that part. Maybe I can optimize this for the final version.


How do I turn off said warning?
On Jan 8, 2015 11:11 AM, "William Ferguson" <notifications@github.com>
wrote:

> As explained previously, you have been using a *feature* that was never
> intended by Aapt or android-maven-plugin. You can still attempt to use it,
> it may or may not work.
>
> You will get a warning about using it which you can turn off.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/simpligility/android-maven-plugin/commit/6247bf385c714332a90be7f463f2bd302315804b#commitcomment-9199222>
> .
>


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=6adb5760619243b18645a95e522c096a
 plone-4.3-python-2.6 [FAILURE]
plone-4.3-python-2.7 [FAILURE]
plone-5.0-python-2.7 [SUCCESS]


Looks great, but not on IE10:

![Screen](http://i.imgur.com/5EHdfSB.png)

To fix, inside `md-input-container .md-input` I've changed `-ms-flex-preferred-size` to `26px` and now looks fine. Probably it needs to be checked using IE11.


這一行多餘了, 因為上一行的 `params[:session][:remember_me] == '1' ? remember(user) : forget(user)`, 已經把 remember(user) 的邏輯考慮進去了


Example log output of the included test:

```
20:20:16,333  WARN [BasicOperationScheduler] OperationThreadWatchDog - [127.0.0.1]:5001 [dev] [3.5-SNAPSHOT] Long running task detected!
LongRunningOperationLog{task='PartitionWideEntryWithPredicateOperation{}', stackTrace='java.lang.Thread.sleep(Native Method)
java.lang.Thread.sleep(Thread.java:321)
java.util.concurrent.TimeUnit.sleep(TimeUnit.java:356)
com.hazelcast.map.LongRunningOperationTest$LongRunningEntryProcessor.process(LongRunningOperationTest.java:86)
com.hazelcast.map.impl.operation.AbstractMultipleEntryOperation.process(AbstractMultipleEntryOperation.java:273)
com.hazelcast.map.impl.operation.PartitionWideEntryOperation.run(PartitionWideEntryOperation.java:75)
com.hazelcast.spi.impl.BasicOperationService$OperationHandler.process(BasicOperationService.java:605)
com.hazelcast.spi.impl.BasicOperationScheduler$OperationThread.process(BasicOperationScheduler.java:598)
com.hazelcast.spi.impl.BasicOperationScheduler$OperationThread.doRun(BasicOperationScheduler.java:541)
com.hazelcast.spi.impl.BasicOperationScheduler$OperationThread.run(BasicOperationScheduler.java:515)', invocations='Invocation{duration=3781}, Invocation{duration=3783}, Invocation{duration=3788}, Invocation{duration=6793}'}
```


k, gonna test this on amami after changing 019E to 01A7, right?


r=cmr


這個 raise 是為了測試是否有沒考慮到的測試項目而加上去的, 所以當測試項目補上去的時候, 這個 raise 就要拿掉


FWIW, `__exposedProps__` is quite deprecated now, one should rather use something like:
```js
var protectedFunctions = XPCNativeWrapper.unwrap(Cu.createObjectIn(win));
for (var p in functions)
    protectedFunctions[p] = protect(functions[p]);
Cu.makeObjectPropsNormal(protectedFunctions);
win.wrappedJSObject.NetExport = protectedFunctions;
```


For all non-protected videos keySystem is always null. It causes "Unable to get property 'unsubscribe' of undefined or null reference" exception each time reset method is called. You can reproduce this by calling MediaPlayer.attachSouce twice.


Did
https://github.com/mono/mono/commits/2368176d0f90d53f26d33ad14734459aabcc5db6
fix this ?


there probably should be special handling if the operation is PartitionIteratingOperation - to reveal the actual operation.


This address #5


Yes, you are right. These functions need documentation. I'm working on that at the moment.


GitHub does't diff this very well, but it's just the removal of the `try..catch`, to allow any exception to walk up the stack to `Program.Main`.


@bors: retry


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.3.0.928](http://build.osmsharp.com:8080/viewLog.html?buildId=4096&buildTypeId=bt16) outcome was **SUCCESS**
Summary: Tests passed: 486 Build time: 0:0:0


some tests failed:
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3066
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3059
exception: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3060
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3051
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3051
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3054
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3047
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3042
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2719
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2717
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1555
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1547


Pikachu :zap:


It's queued for build here @vargaz. Will let you know. Thanks.

https://ci.appveyor.com/project/ajlennon/mono-817/build/1.0.604


saw approval from alexcrichton
at https://github.com/brson/rust/commit/1f70acbf4c4f345265a7626bd927187d3bfed91f


merging brson/rust/feature-staging2 = 1f70acbf into auto


status: {"merge_sha": "f255d26d1c0f3ee6379c454f22a4f8eb04fd326f"}


brson/rust/feature-staging2 = 1f70acbf merged ok, testing candidate = f255d26d


Fixed in c5b5bd5.


You will get a `KeyError` when `x = False`


Technically you are wrong. This prevents memory leaks.
Practically you are correct because this singleton will live almost until the program exits in which case the OS frees the program memory anyway.


Good point.


Why only this method catch google errors?


I understand you about `const int&`.
However for the rest I'll have to reread the code when I get home.


@matheusagcosta thanks!


You probably want to look at the Maven "replacer" configuration for form-modeller to prevent this re-occurring every time translations are pulled from Zanata by Michael B and the "replacer" plugin ran.


Any suggestions?


This font is not a copyright of Silicon Andhra.. We developed this for APSFKN


I was treating 0 as a special value to indicate "give me all the messages you have in the queue".
Maybe I should change it to "-1".


The && _gatorInstances[element.__gatorId]  check doesn't work.
The first time, the id is 0 (falsy). Thus, the first off() following an on() is ignored.


Issue is here, you deleted the layout file!


Just as a side note (this is written also in other parts of Feel++, so it's not specific to your commit): one has to be careful when using `auto`, especially in for loops. Here, you are calling the copy constructor, for each element of `vect_wn`. This can be costly (both in memory and CPU) depending on the underlying structure, so in many case, it might be better to use `auto const&` (if one actually doesn't need to copy the variable).


先搭建好protobuf环境,
然后运行nettyServer;
运行client1,client2,client3之一,可以自动连接服务端,服务端回包欢迎,对应客户端发送给服务端一句话,服务端push给所有在线玩家


Don't think this is needed as index.handlebars already exists for the page content


直接 `function` 吧


yes. oversight.


There's no need for partials at all actually, if you look at e.g. the jade templates it's just layout, index, error


The config on the master branch is right, it seems that these '' where introduced by error on a cherry pick from the 6.2 branch.

Now I'm checking the config on the 6.2 branch, it seems that it has the error that is causing this replacing problem :S


It is intentional to aid when searching for "newLogMessage" in the code when someone sees this a connect().


True. This was copied from the other singletons in the codebase and I didn't give it much thought.


We have just done sourceBuffer.appendBuffer ; how could it be expected that mediaSource.activeSourceBuffers == 0 ?

it will be at least 1 !


Yes. In case a worker thread (eg downloadthread class needs to add log messages).


I am gonna try it later today.


LGTM


Are you sure this is the correct syntax for windows? I thought it would have to be ```%LOCALAPPDATA%\pyfarm\agent\uuid.dat```


Rolled back order changes.


Not sure what this is....


My email got reject! :sob:

> Recipient address rejected: User unknown in relay recipient table


if we want to test for mediaSource.activeSourceBuffers to be 0, it has to be done after mediaSource.removeSourceBuff(sourceBuffer);


Noted Marco

On Thu, Jan 8, 2015 at 10:58 AM, Marco Mascheroni <notifications@github.com>
wrote:

> Hi nizam,
>
> can we give a default value to filename to None? Then we can parse the
> url and get the filename from there if the user did not provide one. For
> example we could do:
>
> from urlparse import urlparseimport os
>
> url = '//www.sitename:80/%7Esomethibng/Python.html?param=val' #urls can be complicated :)
> path = urlparse('//www.sitename:80/%7Esomethibng/Python.html?param=val').path
> filename = os.path.basename('/%7Eguido/Python.html')
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nizamyusli/CRABClient/commit/72d9878d7a8e2336392ac9ebea77f45c03410a37#commitcomment-9199072>
> .
>


good point, will fix that one.


What is that `or` just before `{% endif %}{% endif %}{% endfor %}`?  It seems to be dangling...


_(:з」∠)_ 少了个分号，下次commit再补上吧


use this in the head..

var slideimages=new Array()
var slidelinks=new Array()
function slideshowimages(){
for (i=0;i<slideshowimages.arguments.length;i++){
slideimages[i]=new Image()
slideimages[i].src=slideshowimages.arguments[i]
}
}

function slideshowlinks(){
for (i=0;i<slideshowlinks.arguments.length;i++)
slidelinks[i]=slideshowlinks.arguments[i]
}

function gotoshow(){
if (!window.winslide||winslide.closed)
winslide=window.open(slidelinks[whichlink])
else
winslide.location=slidelinks[whichlink]
winslide.focus()
}
</script>

------------------------------------------------------------------------------------------------------------------------------------
use this in the body

------


Came from a rebasing master onto the branch


@craigcitro httplib2 treats headers as native string.  It's unicode on Python 3.
See https://github.com/jcgregorio/httplib2/blob/master/python3/httplib2/__init__.py#L193-L195

```python
    if six.PY3:
        clean_k = clean_k.decode('ascii')
        clean_v = clean_v.decode('ascii')
    clean[clean_k] = clean_v
```


Did you mean to call `self.warning` before `self.update_headers`? That seems like it would make `warning` more prone to duplicated code and awkward architecture since it couldn't make its decision based on the results of `update_headers`.


What is the reason here?


0 cannot be a special value, as it is in the range of IDs. -1 will be as suitable as any other negative value. But you can not do a separate check for this case (at least in the code I gave).


不自动加载Controller就好了
Controller那边肯定是dispatch的时候找对应的文件的

> On Jan 8, 2015, at 7:10 PM, Rui Ma <notifications@github.com> wrote:
>
> 貌似我们在这个地方没有考虑清楚。如下:
>
> index_controller , 是不好去加载的。我们如果遍历了所有的 class 文件，势必会出现 index_controller 这种类相同的情况, 如果在 class_map 中尝试去加载, 肯定会出现错误的情况.
>
> 需要重新考虑解决方案。
>
> —
> Reply to this email directly or view it on GitHub.
>


What with, and how did u compiled it pls answer?


0.1.9 is based on master, not the nightly. Give it a few more hours and a
new nightly will be out, or go back to 0.1.8 which I believe to work
against the current nightly.
On 08/01/2015 9:49 pm, "Konstantin Stepanov" <notifications@github.com>
wrote:

> It doesn't compile with this line changed with latest nightlies. Do you
> know something I don't about Rust core lib moving of marker module up a
> level? Because at this moment it's still in core::kinds and so mucell
> doesn't work for me any more.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/chris-morgan/mucell/commit/a3a7121802f9c64112e75f4c53e406f565ad914f#commitcomment-9199751>
> .
>


@rha-odoo I have made the suggested changes, Kindly check.


Are we still using the `_name` convention for fields?


Ну що за трансліт?? )))


{{ doc.id | remove: '/' }} just before injects the name of the input (happens to be the item id in _posts)
the `false` before `action:`  is added after the last `or` as there is no way to ensure you are at the last item in site.posts when looping through.


Well, that is what we currently have. The current compiler is very strict. We will still add as many workarounds as reasonably possible to get existing mods working. But these workarounds will be switched of when compiling in a higher version of the language.


I am not sure whether this was intended (i.e. you want to do it later), but this commit misses the table actually explaining what the different actions of the user mean (see [this comment](https://github.com/opf/openproject/pull/2401#issuecomment-69154952) of mine).


Did you forget Acidmaw? :dancers:


Ну да, просто не використовуй `System.arraycopy`


Found BROKEN by full.
ulmageMendelsohn.py
14. coloring.py
15. stronglyConnectedComponents.py
16. createParent.py
17. matrix_expand.py
18. veccat.py
19. expand.py
20. customfunction.py
21. parallelizer.py
22. limitPrinting.py
23. assertion.py
24. printme.py
25. dotdraw.py
26. callback.py
27. KinsolSolver.py
28. NewtonImplicitSolver.py
29. NLPImplicitSolver.py
30. exacthessian.py
31. ipopt.py
32. LinearSolver.py
Ran 32 tests, 0 fails.
python internal/test_cpp.py '../docs/api/examples' -memcheck
Compiling
make[1]: Entering directory `/home/casadibot/slaves/linuxbot/full/source/docs/api/examples'
cp SX/SXaddition.cc _temp.cc
ctemplate/compiler.sh _temp
ctemplate/linker.sh _temp
rm _temp.cc _temp.o
mv _temp.run SX/SXaddition.run
cp Function/customfunction_c.cc _temp.cc
ctemplate/compiler.sh _temp
ctemplate/linker.sh _temp
rm _temp.cc _temp.o
mv _temp.run Function/customfunction_c.run
cp misc/all.cc _temp.cc
ctemplate/compiler.sh _temp
make[1]: Leaving directory `/home/casadibot/slaves/linuxbot/full/source/docs/api/examples'

In file included from /home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/matrix.hpp:31:0,
                 from /home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/matrix_tools.hpp:29,
                 from _temp.cc:1:
_temp.cc: In function ‘int main(int, char**)’:
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:128:11: error: cannot convert ‘casadi::Matrix<double>’ to ‘bool’ in assignment
     is_ok = x; \
           ^
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:144:26: note: in expansion of macro ‘casadi_assert_message’
 #define casadi_assert(x) casadi_assert_message(x, "Please notify the CasADi developers.")
                          ^
_temp.cc:11:3: note: in expansion of macro ‘casadi_assert’
   casadi_assert(!all(d));
   ^
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:128:11: error: cannot convert ‘casadi::Matrix<double>’ to ‘bool’ in assignment
     is_ok = x; \
           ^
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:144:26: note: in expansion of macro ‘casadi_assert_message’
 #define casadi_assert(x) casadi_assert_message(x, "Please notify the CasADi developers.")
                          ^
_temp.cc:12:3: note: in expansion of macro ‘casadi_assert’
   casadi_assert(!any(d));
   ^
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:128:11: error: cannot convert ‘casadi::Matrix<double>’ to ‘bool’ in assignment
     is_ok = x; \
           ^
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:144:26: note: in expansion of macro ‘casadi_assert_message’
 #define casadi_assert(x) casadi_assert_message(x, "Please notify the CasADi developers.")
                          ^
_temp.cc:20:3: note: in expansion of macro ‘casadi_assert’
   casadi_assert(!all(d));
   ^
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:128:11: error: cannot convert ‘casadi::Matrix<double>’ to ‘bool’ in assignment
     is_ok = x; \
           ^
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:144:26: note: in expansion of macro ‘casadi_assert_message’
 #define casadi_assert(x) casadi_assert_message(x, "Please notify the CasADi developers.")
                          ^
_temp.cc:21:3: note: in expansion of macro ‘casadi_assert’
   casadi_assert(any(d));
   ^
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:128:11: error: cannot convert ‘casadi::Matrix<double>’ to ‘bool’ in assignment
     is_ok = x; \
           ^
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:144:26: note: in expansion of macro ‘casadi_assert_message’
 #define casadi_assert(x) casadi_assert_message(x, "Please notify the CasADi developers.")
                          ^
_temp.cc:28:3: note: in expansion of macro ‘casadi_assert’
   casadi_assert(all(d));
   ^
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:128:11: error: cannot convert ‘casadi::Matrix<double>’ to ‘bool’ in assignment
     is_ok = x; \
           ^
/home/casadibot/slaves/linuxbot/full/source/./casadi/core/matrix/../casadi_exception.hpp:144:26: note: in expansion of macro ‘casadi_assert_message’
 #define casadi_assert(x) casadi_assert_message(x, "Please notify the CasADi developers.")
                          ^
_temp.cc:29:3: note: in expansion of macro ‘casadi_assert’
   casadi_assert(any(d));
   ^
make[1]: *** [misc/all.run] Error 1

Traceback (most recent call last):
  File "internal/test_cpp.py", line 40, in <module>
    raise Exception("Was unable to compile.")
Exception: Was unable to compile.
make: *** [examples_indoc_cpp] Error 1

```


So this took me longer to understand than id like to admit. Was unsure how the session worked. But my assumption was this; I am created an object that I should be able to pass to a merge? I tested and it works, I'm just not sure if it is the correct way.

```python
		with db_session() as db:
			if preType == "NEW":
				try:
					p = Pre(**formattedPre)
					db.merge(p)
```

Was that too far off? :)


Also it is a shame but I would have liked to be able to put this liquid tag logic on seperate lines but the compilation produces horrible output that pat-depend (or more likely pat-parse) cannto handle.


reverted to 'updated befehle.txt'


Hi @weiguUL
You changed the marker selection analysis to receive the log projection instead of z-score from the db. However, the heatmap analysis which is called at the end was designed to only plot zscores. So either the zscores need te be calculated before forwarding the data to the heatmap analysis, or the heatmap analysis needs to be changed (it has value clamping, and perhaps some other zscore assumptions).


I think I was getting confused by its existence outside of the translator class, and assuming it needed to be a return value somewhere. It doesn't look like that's the case, my apologies for not being as thorough as I should've been.

I suggest adding a comment in the header to the effect that this should be an inner class but the static nature of the methods in the translator class preclude this, otherwise this intent is hard to derive.


what ever i do i get no route  .... and i am using VPN


boom


Исправим :)


I'm using neovim, and to enable python plugins like jedi, it's needed to install neovim python library through pip. Do you see any thing that I can do to not have to install neovim inside every virtualenv?


Can you make the copyright messages for Kinesis S3 Sink just (c) 2015?


OK. But in this case you must ensure that the instance of the singleton will be created before running other threads, because the current implementation of the method instance() is not thread safe.


Wrong sub-project for my comment but still applies to Kinesis S3 Sink


r+


Done.


```keySystems``` array must be cleared either in ```reset``` method or at the beginning of ```init``` method. Otherwise duplicated items are added each time init method is called. To reproduce this you can call MediaPlayer.attachSource several times.


Assuming we're moving to AWS I want to move away from RabbitMQ and use Redis as the queue. Storing results in the database is a good way to not overburden Redis and is an initial step towards fixing our use of Celery.


All right. Since we're browser-only, I think it should be safe.


Yeah, it's disabled in prod in a settings_local.py (sic!)


I don't know, do you?


Hello, I believe the PHPGraphLib examples are the easiest to follow out there.
I was just wondering if the sql code to create the database table is available to use?

PHPGraphLib Tutorial #2: MYSQL and PHPGraphLib
http://www.ebrueggeman.com/phpgraphlib/documentation/tutorial-mysql-and-phpgraphlib

Thanks for your help!


Makes sense. FWIW, the next step would be to stop using pickle for the serialization completely (since it'll be deprecated in some version of Celery in the future). First I'd need to double check all the tasks arguments if they can be serialized.


A comment.


Could you make a kernel for sm7?


This change increased the worst-case memory complexity of the algorithm from O(n) to O(n^2). The average case probably increased less, but definitely from O(n) to O(n*averageNeighborhoodSize).
This makes the algorithm less scalable.


This `if` seems pretty sketchy, not least because you can complete it and `dom` won't be set.

```py
    if isinstance(html_input, str):
       html_input = html_to_dom(html_input, default_encoding, encoding, enc_errors)
    dom = preprocessor(html_input)
```

Feels better and only needs a single call to preprocessor. `try ... catch` the last line as appropriate to deal with cases where `html_input` is not `lxml.html.HtmlElement` by that point.


try to use another one.


Oops, the whitespace sanitizer was switched on which makes this change look messy. Repeats.


Why ?? : ((


thats good


nope


я тут трохи з гітхабом натупив, а трансліт - так я писав на скору руку


This could be an ITransactionProvider only.


r=Gankro


Please remove that file from the patch. It's nice for testing but does not belong to the repository.


No Comment


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Can you revert this change? (should be something like `git reset HEAD^; git submodule update; git commit -a;)


I'll do that now!


Great, thanks. Hashing paths is an excellent solution.


Right


Could you please perform deployment  to maven central?


Oh, I am ashamed! Sorry for deleting the licence. Will add it back immediately!


This is a pretty big change to the parsing semantics. Could you add a commit message explaining more about the changes and why they're done this way? In particular why it requires `Elements` to shim around parsing restrictions. It'll be really useful to future developers.


getUrl - hmm i don't get it, isn't it a mistake?


Convert -> Converts


this is the reason for IOS failing


a wise note on the dependency...
I'll check the significance of that statement and will validate things on pyside...
I'll ping you when done...


`File.basename` would be better than `split('/')` (and more reliable)


Thanks for this patch! Took me a while to figure out why I lose all my large POST's payload. I see that the backport was merged. Is it possible to get a new `1.5.x` version that includes this patch?


フルネイムで記述できませんか？


Sollte das nicht "unverschlüsselt" heissen?


I meant organizations like the Dutch or German Django Association. I thought they were local chapters of the DSF. Asking the DSF to help with money transfer might not always work because I don't think they will help all Django Girls events all over the world with money transfer. Maybe we can say something like "If there's a Django Association in your country you can ask them for help as well." What do you think?


it's better we add context here


Shouldn't the placeholder have its own specific property instead of a generic one?


Hi @vargaz

This helps but we're failing on socket.h now

../../mono/utils/networking.h:18:24: fatal error: sys/socket.h: No such file or directory

https://ci.appveyor.com/project/ajlennon/mono-817/build/1.0.604

Regards, Alex


i thought it's in SOURCE=>DESTINATION format but it appears to be otherwise. do you think DESTINATION => SOURCE format is ok? counterintuitive for me. And, anyway, at this moment there are no mentions about right format either (in readme)


yeah - probably SMSG_CREATURE_QUERY_RESPONE and maybe ObjectCreate fields.
Also note that sometimes auras are used to toggle a model


Thanks for this! Made me aware that "computed accessors" was currently broken on 6to5... Has since been fixed in 2.8.1 and 6to5/6to5@e1c7584280a61034dd0f1cca400decd057a89730


Haha :D Thx.


ist das so richtig?


r=pnkfelix


Cash файл лучшевыкинуть из индекса, добавь в .gitignore строку *.cache


Inspire is for console request (like controller for http requedt. Nothing to do with CQRS


It works it's correct it's not URL it s the Host: header


Кодировку лучше конечно использовать utf-8


r+ rollup


Такиеоднострочникипотенциальное место прошляпить ошибку, лучше их писать в 3 полноценные строки:
for(int i=0; i<count; i++) {
    buttons.get(i).setVisibility(View.VISIBLE);
}


In Whydah 2.1, UAS should also send the mail (no UIB)  :)


We ensured that a minumum sql queries are node, if you rewrite make sure no more sql queries are done.


Таких однострочников лучше избегать


1>   Download a server eg. WAMP, XAMP, Zend (I use Zend Server).

     Download Zend server from http://www.zend.com/en/download/

-------if you use Zend server follow the instruction bellow-------

2>   INSTAL (ZendServer)

3>   Paste (php.ini) file in C:\Program Files (x86)\Zend\ZendServer\etc

4>   Paste (OIS) folder in C:\Program Files (x86)\Zend\Apache2\htdocs

YOUR TOOLS IS READY  FOR HACK THE OIS.

5>   Go to browser windows & type    localhost/OIS

-------else -----------------------------------------------------
2>   Directly instal the server and paste OIS folder into the hosting folder.

3>   Go to browser windows & type    localhost/OIS

Devloped by Aiswarjya Buda.
Contact me on http://www.facebook.com/aiswarjya

Thank you for watching......


I don't know. If it matches the generic one, should we create new label with the same value?


Notes from KB:
WASH I, WASH II, WASH III are programmes,
there is also a distinction between Area 1 and Area 2 which is geographic, and has some overlap with programming, but not 1:1. Here WASH I and WASH II actually refer to the geographic areas.


Читаемость эих 3 строк просто нулевая :)


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


Yeah, thx.


Unfortunately this does not work on Windows.
Proposed fix:
```python
        pipe = os.popen( '%s%s %s "%s"' % (
            valgrind_path, jsontest_executable_path, options,
            input_path))
```


Yes if you want website are like companies ony a few there max 1 website by domain,


Oh, hm, i just see now you did graphical changes on your personal branch. I just did teh same in the open-ideas repository based on all graphics of IDEAS ...


yes we should crash if not found, like if no company found


Indeed, and I realised this in making the change. We'll discuss over at #4066


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


beacuse new editor will be multitenant


@Underground78 @Armada651 @kasper93 @alexmarsev: so, what should I do with this? Do you think it can be of use for us (realistically)?


Why is there a `request` parameter, if it's not being used? Was it meant to be used instead of the `AccountController`'s `Request` property? Which would be a better option?


Что за магическая чиселка?


chm has real reason, that i forgot, it's to allow both custom footer and footer on multiple version


same reason to allow multiversion on footer


How were these generated. I'm not sure they are coming up as valid markdown for the tables.


@arnaudgelas
No, if this pans out nicely, we do not. To test viability of this
method, I just created packages for trusty and utopic and released them
on my ppa.
If package naming/versions diverge, manual editing of debian/control
might become necessary.
Additionally, I set the default distribution to debian "unstable". Upon
a new release, we will need to

a) change UNRELEASED in debian/changelog to unstable
b) run dch -v "version" to bump package version

Then we can create packages using the script.

Shall I issue a pull request?

Best
Stefan

On 08/01/15 10:43, Arnaud Gelas wrote:
>
> @zarquon42b <https://github.com/zarquon42b> do you still need to keep
> it on a separated branch?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/statismo/statismo/commit/44cf60d7599bf29dfede0f886436d6c3980da635#commitcomment-9198876>.
>


lgtm


@stof I think this must be applied to 2.3 branch also ??


we moved js code on top. so asset is not split anymore, not doing so resulted in a lot of annoyace for inline js. Ask jke,


some reason custom footer


I hate spaces, tabs were invented for this. But yes, using spaces! :cake:


idem


Start working on Udacity JavaScript basics project.


@ajlennon This is a work branch ("martin-ssl-integration"), I don't think it makes sense to build anything else than master. What do I need to change in the appveyor.yml?


What, using xmlid_to_res_id makes more queries ? Or even self.env.ref() ?


You return ids[0]. Why not putting a limit ?


no more license header please


Lets see if 16b2539bab67dcb37b6523d33a670e0d33ff79ab fixes it.


We talked about properties one day. We rarely use generic properties.


it's pure ascii


Why not raising a real exception or warning instead of having a crash ? Having a crash and traceback as a normal behavior is weird.


Given work is underway on #2391, I only want to make minimal changes here.

@0xF013 not sure why removing `trigger.position().top` fixed positioning here – but it works.


He has reason you forgot ? Well, having a commit that explains that seems quite necessary to me then, if even you do not remember ...


You need to update `amber-dev` in `package.json` to `^0.3.0` as well.


@bazsi Something like this? Or did you have another design in mind?


I love that you cleaned this up! However, I do like having the `expect` blocks in the spec to make it really clear what we're testing. What do you think?


I don't think so it would be useful. I don't like the idea of scrolling through the comments `OMG it doesn't work!!!111` and such. We have trac for bugs, IRC and d9 thread for casual discussions. I mean we don't post any news that needs commenting, just release notes and some occasional events. But if you guys thinks it is worth it I don't mind give it a try.


If I delete the *savePom()* calls from the rules and call it only once at the end than you have want you want, right?


Since bearerToken is now set as a part of protectionData, ```setBearerToken``` method must be removed from MediaPlayer API. Also since ProtectionController has ceased to be a singleton, protectionController variable has to be removed from MediaPlayer class as well.


There 's always at least one website.


goToNextLesson should not be present at all. It is added dynamically during the lesson.


This should *not* be removed entirely, it should change to `Removed` (as with `quad_precision_float` above)


Hi,

Please replace:
renderer.ScaleTransform(bitmap.Width / this.Width, bitmap.Height / this.Height);

by:
 SizeF size = this.GetDimensions();
 renderer.ScaleTransform(bitmap.Width / size.Width, bitmap.Height / size.Height);


It shouldn't make a difference, but I think you could have saved yourself the (not ...   ).

file:  false or doc.id     --> doc.id
folder:  false and doc.id   -->  false

That way, you can get rid of the closing `)`.


some tests failed:
failure: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3061
exception: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3067
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3052
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3052
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3055
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3048
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3043
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2718
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1556
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1548


OK


ascii only


I agree with that, that is why we removed it (we had it in the beginning). I just wanted to hear what everyone else thinks before I ditched the patch :p

Also, CC @vBm @Jellyfrog


Yes, that sounds better :). I would even say Django/Python, since not everywhere Django is popular, but there might be some kind of Python group or foundation.


yeah i was thinking about that too

maybe we can do this

```
it "filters the entries on not billed" do
  expect(filter({ billed: entry.billed}).count).to eq(1)
end

def filter(params)
  EntryQuery.new(Entry.all, params).filter
end
```


oh, mmh, good point. Let me do that


This field is not used anywhere.


You are right it worked with the binary value of ffmpeg flags. But i think it's better to check this way, no matter the flag value: it always works. Don't you think ?


So every json route shoud be POST, let's do that in route not here. And you can have url parameter on a POST route, you can legally POST /url?withparam=1


This will be an option right? I mean we can use only a simple String (if it is unique) or a combination of 1+ String to create the key, am I correct?


This probably has nothing to do with this PR, but when I try it in http://localhost:4000/appraisal-masterclass/, the first item has no `document_type`.  I'm not sure what it is, but it looks like a todo.


ところで
[Re: Error: add-to-list cannot use lexical var ...](https://lists.gnu.org/archive/html/emacs-devel/2012-08/msg00450.html)
死ぬほどエラー吐いてるしadd-to-listはやめたほうがいいかも。
むしろwarningじゃなくerror吐いてるのに正常に動くしコンパイルも通るのがよくわからない


saw approval from pnkfelix
at https://github.com/huonw/rust/commit/3155b3101108dc9a3be32b6cb3cf33b9e1c45a0c


merging huonw/rust/fix-string-slicing = 3155b310 into auto


remainder from testing this, thanks. I'll remove it with whatever the next commit will be.


status: {"merge_sha": "2f99a41fe1a27a48e96bc2616ec9faa6de924386"}


huonw/rust/fix-string-slicing = 3155b310 merged ok, testing candidate = 2f99a41f


@dongqunxi  Do you what was in this file? Is it just hre by mistake?


I'd see this more as a feature, as this will not break inserting 'ordinary' unicode characters but allow to alternatively insert unicode codepoints, e.g. "»" This might be done faster than retrieving the unicode character if one knows the codepoint ;-)

d-o-e not surviving a second pass might be good argument against...

Maybe a good alternative would be to define the wanted character as glyph in the TEI header and then add a respctive XPath as submitted parameter value?


Buy a satellite dish and a DVB-S2 card and you're off again...


Indeed I saw this conversation, but i found no patch even in the ffmpeg git repository...


Excellent work. One question, with this approach (String as key) can you resolve your problem? I am talking about this one:

https://github.com/nicolasjafelle/Qachee/issues/1


OK


To have the right target for the button "Insert Blocks" because I added two other elements which are "Save and Publish" and "Save as New Version".


You're right! That works :)


This is only a personal preference, but I generally feel that the simplest rule is one that says "use the singular form nearly everywhere" instead of the plural form. I mentioned this over here also https://groups.google.com/d/msg/elm-discuss/IJkhU17-UqY/51x19Y-7-J4J

I also called the module in `elm-result-extra`:
```
import Result.Extra
```

There's no way I could argue this is the *right thing*, it's just the simplest rule I can come up with for keeping my own libraries consistent. For example, I already messed up using singular for `elm-html-shorthand` and plural for `elm-filepickerio-api-types`.


You could use the "disableTouchKeyboard" option introduced in #842, maybe that fixes your question?


you may not rpc it without the param it should crash


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


sorry here name and version are RPC parameters not route or url


ok


Súbor štandardného generovania podľa akého kľúča? Alebo to budete robiť?


So, @tsauerwein, maybe you can remove this commit and add a new one to your `webgl-point-hasFeatureAtPixel` branch.


Yeah I like that!


:+1: for using `//`


Probably missing from the front matter of the site post.
I think it was only developed with http://0.0.0.0:4000/open-market-committee/ in mind at the moment.
Can you raise an issue to look at this?


This commit produces this error for me:
> Error: The application is trying to load a file from the app_form plugin.
> Make sure your plugin app_form is in the /home/bernat/work/new-roomsday3/plugins/ directory and was loaded.

The file config/app_form.php exists but it's only used in AppController to load new templates in FormHelper as suggested in the book.

I'm not sure if I should comment here or rather open an issue.


this is a comment


'checkout/layout.html' should include css and JS scripts due to base.html not including them anymore


child 要设置 key 属性


To avoid some inheritance issues when we 've got several automatic footers.


Doesn't work yet. I'll probably have to fix the django-compressor manifest.json with `sed` :dancer:


Excellent! :)


closes #39


I think product_id should be in the parameters like the track_all.  I doubt these ids will work.


int is useless has it's jsonrpc


The POST wasn't about json or not, but because you alter the database (REST)


Jsonrpc i explained you


BTW: _otherIdRegex, _opponentPlayRegex, _powerListRegex, _zoneRegex and local variable *player* in *TagChange* (due to commented out logger messages) is also never used.


I think I understand what you are saying and you are correct in saying
folder: false and doc.id --> false
The logic is that if a non-folder is selected then the button is enabled if a folder is NOT selected.
As soon as a folder is selected it will always be false
file (true) & folder(true): false or file and (not folder) --> false

If you know what I mean. ;)


Ok you are right we should move it to model and rename it.


Is it intentional that you take any list whose title starts with "Blocked" or do you want to match on the exact title?


I think I need to set HOME to a writeable location but I'm investigating how other packages do that first.


OK


We can remove <data> also


ascii only no need for coding


It seems something was left out in the commit, as bcf.py refers to CCoverResults member "all_unique_bbs" on line 508 for example, which in effect breaks the bcf example. Reverting to the previous version seems to work ok.


asdasdad


PS people on hackage suffer from the same confusion: http://hackage.haskell.org/packages/search?terms=extra

Nobody can figure out whether plural / singular is the right thing, so it's random!


It a m2m it's normal we need to add some info on the m2m like in im_chat


Yes, this will apply to all resources.


877abd2dc4d7608060db58f2b85f29d51dc97ddb で一部修正


Actually I also updated the static.


Fp request... :) i dont like it either


it's website_version.goals it's the name on google


202d9245e1bfe4e910759e0a184b3bb85530ac78 で修正


Remove it complettly just keep it on the selection


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1806](http://teamcity.monogame.net/viewLog.html?buildId=4237&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 304, ignored: 6 Build time: 00:16:09


```initProtection``` method is called for each Stream (Period) in the mpd. At the same time ProtectionExtensions is a singleton. This means that in case of multi-period mpd after a new Stream object has been initialized it will call ```protectionExt.init``` and override the data that was set by a previous Stream.


Unfortunately this does not work on Windows.
Proposed fix:
```python
        pipe = os.popen( '%s%s %s "%s"' % (
            valgrind_path, jsontest_executable_path, options,
            input_path))
```


Issue has been fixed in 4b88ef58910c63cfef0fe6a1dac471980813f00c.

And coming back to a question in the original posting: Yes, we have two different methods of parser the script. One for compiling it and one for extracting the local variable declarations. --script-all does not always make use of the later.


Yes but make sure we using the exact same wording as google


Use Optional because the interval is simply not availabe. This way you avoud any magic numbers.


When it is an existing record, these ids will be set, so you should check ids is not empty and also, these fields are put on product_product and product_template so ids could be both?  You could define onchange_track_all in both objects?


They need to be syncrho with google we need to do it in create


same for syncro with google


We should still ship this class with an obsolete attribute until 2.0 for bwc reasons.


Long commit line.


I asked to remove it too and rename to google anayltics the key in ir_config_paramm


I see that in this situation (using a TemplateHandler for `/contacts` ) we cannot use @Metered.


ok


フォーク元への取り込みに伴い、7d9370c3314c9366a95e317d63874c8df0f09213 でrevert


just because using string interpolation is fun :balloon:


Here the method could maybe use a common method for rpc call but jke tried and it didnt succed to make it shorted


setting HOME = $TMPDIR works but now I'm hitting https://github.com/elm-lang/elm-reactor/issues/54 - how did this ever work?


フォーク元への取り込みに伴い、[b04d31cc58f911d1336ee960bf7ffab585dcc8a9] でrevert


A components message isn't required. You shouldn't have to describe what a component is.


Typo `partitionAall`.

Let use the names `toJs` and `toClj`. Thanks.

David


is this data base related..?


You could do [('product_id.product_tmpl_id','=', ...)] in the quants search as here the '=' should be 'in' and this in is slower.


Does the native PLATFORM macro equal "win32"?  Maybe the commit log and the comment can mention that.


You can omit the `File.new` if I'm not mistaken, `File.write` should take care of it.


I think you probably forgot to change the name of this test?


Maybe I'm misunderstanding, but I think this should be reverted?

First off, the commit log should be descriptive.

Second, it looks like this commits a bunch of settings for a specific printer into the stock Configuration.h?

Third, what is this change *supposed* to be for??  Maybe split out just the thermistortables.h changes and put them in a commit with a description like "add support for the Foobar printer's XYZ33 thermocouple" ???


a version is a set of modifed views


module va written before the new api, we may convert it


フォーク元への取り込みに伴い、[33133f7](33133f7d7e0397bdefd1fd5a65e24686d3e828f5) でrevert


Guys, if I may: why are you doing code review on a commit? These 130+ comments should really be comments to a pull request.


フォーク元への取り込みに伴い、[1adad85](1adad857de0840717a4df91ce653d179e3d4859c) でrevert


Yes. If google use goals, use goals everywhere, if google uses objectives, use objectives everywhere. We can still rename fields if necessary ;) .


Warum genau?


:+1: :clap:


git status


Lecture exercises are labeled LecX exercise, lab exercises LabX exercise, homework HWX.


It should probably be factorized in a method (because code seems to be twice the same) and called in create + write as hooks. Easier to understand, one method to review.


Well done :+1:


Hi,

Since this commit I see a lot of indexes are duplicated in my application when using the ``doctrine:schema:update`` command. For example if I have an entity defined like this:

```php
class Participant
{
    ...

    /**
     * @var File
     *
     * @ORM\OneToOne(targetEntity="PTC\CoreBundle\Entity\File", cascade={"all"})
     * @ORM\JoinColumn(name="photo_id", referencedColumnName="id")
     */
    private $photo;

    ...
}
```

When I run ``php app/console doctrine:schema:update --dump-sql`` I get:
```
ALTER TABLE participant ADD CONSTRAINT FK_D79F6B117E9E4C8C FOREIGN KEY (photo_id) REFERENCES file (id);
CREATE INDEX IDX_D79F6B117E9E4C8C ON participant (photo_id);
CREATE UNIQUE INDEX UNIQ_D79F6B117E9E4C8C ON participant (photo_id);
```

Before this commit, just the unique index was created (it seems to be more appropriated):
```
ALTER TABLE participant ADD CONSTRAINT FK_D79F6B117E9E4C8C FOREIGN KEY (photo_id) REFERENCES file (id);
CREATE UNIQUE INDEX UNIQ_D79F6B117E9E4C8C ON participant (photo_id);
```


Should'nt that be factorized in a method, because code seems to be duplicate ? It seems better to me that create and write call create_google_experiment: one method to review, one method to understand, create and write cleaner.


Coding style remark: you may directly create exp = { key: value, key: value, ... } .


I don't think you handle all 2m commands.


@scytacki Thanks,

We have looked into this again. author_email  is not required in Investigation model. Please refer to the changes we made in this commit https://github.com/concord-consortium/rigse/commit/1ac6dd1a668df9574bc7c4df10a24b6cfde242fb


That one will crash with some commands I think.


Those CAPS tho


Yeah, I think it was working. @riz-hossain didn't we check this one already?
I would love to test it again, but things are already broken for some other reason :(


Actually was Issue #1


Coding style remark: here you could write "for version_command in vals.get('experiment_version_ids', list())". Less indentation, one line, code easier to read.


这还有license，我勒个去。。。


It is still breaking. This issue came up post making changes in the controllers, related to package


Tagit bort


Might be good to avoid the 2014 date and say something like "Although some critical issues may be patched in the future, most open issues will remain unaddressed."


The interesting thing to illustrate here is using EditorActionHandler as a base class for your own action, not invoking an existing action through the EditorActionHandler API.


Shouldn't google management directly manage the errors and warnings ? This way you can always try / catch them if necessary. Here you suppose that nor receiving an id is an issue with google analytics, seems broad to me.


This comparision is wrong, I think: if the status is 200 this if statement will still be true and the null will be returned, which is wrong. I.e. maybe it should be "&&" instead of "||"?


As it is still breaking.  I will look into this issue and fix the packages.


Performing any kind of initialization in a static initializer block is very bad style. The correct API to use here is the <editorTypedHandler> extension point.


On 2015-01-08 13:56, Cornelius Schumacher wrote:
> Is it intentional that you take any list whose title starts with
> "Blocked" or do you want to match on the exact title?

It is intentional as the actual title is "Blocked on 3rd party"
to discourage people using it for cards waiting for team-internal reviews.


This is a known issue that needs fixing in the ORM. Duplicate index is less dangerous than the previous behavior.


it should be possible to parse directly into proto_options, e.g. by adding to the msg_format_option rule in cfg-grammar.y, then the reader wouldn't need to pass that value along.

it shouldn't be needed to add this to proto_options. Are there any other patches on a series? maybe I could have a look at the entire patchset, which would be easier to help. I have some trouble getting the entire picture right now.


I think that a typed handler that inserts some text when any character is typed is not a very realistic example. It would be more interesting to show how a certain abbreviation can be expanded.


Coding style remark: we generally put order and constraints at the header of a class (name, description, order, column definition, CRUD hooks, then model methods).


Might have a look into this this evening.


tror man kanske måste skriva någonting annat på type=
vet inte vad än bara!


Sounds good, I've replaced that sentence with yours. See the PR for the new version.


I took a slightly closer look at how the sticky applet works - I'm not sure it's not just a matter of cleaning some things up there to get it working properly - it seems to work perfectly under certain circumstances, but then some state or other gets confused, and that's where the problems begin (and we no longer trigger focus-in signals, etc..)  I'll mess with it more in depth when I'm not at work, along with the desklet.

Perhaps I'm taking a bit of a simplistic approach (not having written these applets,) but I don't see how they're much different, at a basic level, from the menu applet, or any other modal popup that contains a text entry box.  And I'd prefer to approach it from that angle as well.


Can you adjust the size and put a new line after the button?
![screen shot 2015-01-08 at 14 22 24](https://cloud.githubusercontent.com/assets/642544/5663043/d3b7520e-9741-11e4-933c-6ebbea99a798.png)


Should probably be increased to somewhere around 12 or maybe even higher. Up to you guys and that's considering if you even pull this haha


If you look at the column headers on the table, the second one is "Undefined", so the check is *meant* to check against an undefined variable - that's why the error reporting is `E_NONE` in the original code.


@pawelkaczor
Thanks. Released v0.3.1.


Well, yes, `/joinpassword` is for the game client. But we are talking about the game server here... :yum:


随手点的……感觉还是英文课件能看明白……


I'd personally let this be a blank line according to [PSR-2 2.2 Files](http://www.php-fig.org/psr/psr-2/).


Oh, nice!


+1. It's always a matter of preference IMHO. As long as it's consistent, it'll do fine.


Looks like the descriptions have been copy/pasted. Please give a unique description to each action.


fixed in commit e44ecca5f05df664d1fed3f0cdab54c7b29ee27d


觉得他的英文课件做的还是不错的。。。。


WP code standards also state that constants like boolean true and false should be lowercase `true` and `false`, not `TRUE` and `FALSE`.


I think it's better to get a VirtualFile from data context directly. You can check its file type if you want the action to work only for Java classes.


What is this?!


正经不错，就是讲的让人听不下去啊


yeah that method is a huge mess and needs some serious refactoring


@salvatejero added your project to the list!


Defining onchange in product_product does not work, gives error product.template object has no method onchange_track_all as the field is defined only in product.template. And the view is of product.template so the ids will be of product_template


Are you saying it is working like this for product template and product product form?


And where are you disabling those monsters? ^.-


                                                                                I think a lot of things went wrong. I cannot create test cases any more. Seems like result page is broken or dB died...I couldn't do much. We need to fix all these crazy issues.                                                                                Sent from my BlackBerry 10 smartphone.                                                                                From: Mohammed Sazid-Al-RashidSent: Thursday, January 8, 2015 5:16 PMTo: AutomationSolutionz/Framework_0.1Reply To: AutomationSolutionz/Framework_0.1Cc: Riz HossainSubject: Re: [Framework_0.1] Fix 'Assigned User' field in 'Create/Edit Task' page (4e7313e)Yeah, I think it was working. @riz-hossain didn't we check this one already?
I would love to test it again, but things are already broken for some other reason :(

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


I'm confused now: was this a bug in the commit of the PR branch?
IIUC, this will change cosmic reco performance significantly.
Were the plots posted in the PR made with ``` intime_cut = -1 ``` or without?


LGTM


Please note that __noChroot is an undocumented hack that absolutely should not be used in Nixpkgs!


Am I missing something or is a file missing from this commit? What is fileURL('resource-errors') ?


I think you need to define also in product_product and then the ids will be
of product_product.

On Thu, Jan 8, 2015 at 2:27 PM, dharti <notifications@github.com> wrote:

> Defining onchange in product_product does not work, gives error
> product.template object has no method onchange_track_all as the field is
> defined only in product.template. And the view is of product.template so
> the ids will be of product_template
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/odoo-dev/odoo/commit/05023fb57ea0d74f8eb54ef5dbcbf1ac0a58b674#commitcomment-9201599>
> .
>


ok


@Ocramius Ok thanks you for your quick answer and the explanations.

@deeky666 Thank you very much for your hard work!


this resulted in a white admin page after connecting to addons. it's still in the public archive
I think it's important to update


Awesome! many many thanks!

I'm setting bower to hit this repo and install it in my frontend and that's good enough.

However!

If you want to take it to the PRO level you can publish it in bower ;)

[Here](http://devhike.blogspot.com.br/2013/06/how-to-register-your-library-or.html) are some friendly instructions on how that's done (very easy)


Does brewbot honor `[ci skip]`/`[skip ci]` commit notes?


There is other modes than `w` that would skip the truncation of the file just place the pointer at the end of the file, see http://php.net/manual/en/function.fopen.php. By just changing the mode you could skip the usage of `file_get_contents`.


Dang, sorry, same exact errors. There must be something on my end that is goofed up.


Not for components for incidents

```
dashboard/incidents/add.blade.php
```

The component view is this one (unchanged) [Component Add View](https://github.com/cachethq/Cachet/blob/master/app/views/dashboard/components/index.blade.php) Right?


Are we able to submit a pull request for this fix please?


## Programming Assignments
- [Tutorial for those struggling with Programming Assignment 1](https://github.com/derekfranks/practice_assignment)
+
+
+## R Language
+
+- [Some notes on the R Language](http://lopezrj.github.io)


:thumbsup:


then its still /joinpassword ^^ /servername /serverpassword /validation /joinpassword


I implemented this in #43.

The `_input_output_pair` is indeed gone, but now there is an `_InputOutputDescriptor` class ...


So, I did regression while doing regression tests. Will fix this asap.


I should mention, this should only happen on cURL versions prior to 7.20.0. In testing, however, this fix is working great so far for a variety of versions on Mac, Linux and Windows!

>This is not really an error. It means you should call curl_multi_perform again without doing select() or similar in between. Before version 7.20.0 this could be returned by curl_multi_perform, but in later versions this return code is never used.


OK


Duh. Sorry. I shouldn't look at stuff when I'm tired!  I'll delete my note...


Hi timdo,

Z-score is not a requirement per se for hierarchical clustering (called here heatmap). One can perform z-score scaling for the heatmap data but it is not required to perform a “correct” hierarchical clutering. It depends on how the input data is. The previous version that uses pre-computed z-score is to our understanding quite wired, given the way the pre-computed z-scores are generated, this means they have been calculated on the complete study data available even though a user might only select a small subset of the samples available for defining the two cohorts. Such approach is problematic as z-score is pre-computed on a different data set then the one displayed using heat map.

One thing one may consider to add is to scale the input (e.g. by re-compute z-score calculation row-wise). For this there is a new commit available on Github.

https://github.com/transmart/Rmodules/commit/ec4e733bfccd52a0c1a2221c03b0437b59916358

Another thing is whether one want to truncate extreme values like those in the pre-computed z-score as it was in the previous MarkerSelection, this means one just truncates values where the z-score is above/below a certain threshold.


LGTM


What form?


The heartbeat is automatic.  The server will ping you and the ddp client automatically responds to it.
If you want to ping the server, you need to issue a ping command and handle the response back...


> The VM is not the problem. Using the original would fix nothing. The problem is entirely at the compiler level.

Having a MW-compatible VM would allow you to use the compiled bytecode stored in the script records. That is what I meant. And therefore you can bypass the compiling stage entirely (for content that was made in the TES-CS anyway).


nice


Perhaps, I don't really understand the context.
``Content-Type to 'text/csv'`` is not really clear as well.


My mistake, there is a typo here. it should be removeClass.


It's clearer to use CommonDataKeys.PSI_ELEMENT instead of NAVIGATABLE.


Do you know for sure the syntactic sugar of this type is okay with the GCC version they are using?


Seems that CurrentProject.get() will return either a Project or {}. Don't see the reason for these two lines. Can it be simplified ?


Doh. I completely misread.


I also was going to add 0x10 and 0x13 but I was "interrupted". Will finish
today.

On Thursday, January 8, 2015, PEMapModder <notifications@github.com> wrote:

> [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/BlockServer/commit/08f3c5ae036b92ea4b1224ff1004b34ee918c690#commitcomment-9199331>
> .
>

--


Hmm... tables indeed look dodgy. I just used `pandoc -t markdown`. I may have to use another option.


xD.

On Thursday, January 8, 2015, PEMapModder <notifications@github.com> wrote:

> idk. But its inspections want me to do so.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/BlockServer/commit/6c6881d6981c870c416bbd5342b0cc16c057bf97#commitcomment-9199381>
> .
>

--


Case analysis on old code:

`length xs < l`: `clen = 0`
`length xs == l`: `clen = 1`

Remaining case is `length xs > l`, but plugging in a concrete number for simplicity:

`length xs == l + 1`: `clen = 1`

The old code fails when the length of xs isn't an integer multiple of l. Case analysis on new code:

`length xs < l`: `clen = 1`
`length xs == l`: `clen = 2`

Remaining case is `length xs > l`, but plugging in a concrete number for simplicity:

`length xs == l + 1`: `clen = 2`

The new code fixes the failing cases. I have written code which does exactly this so I might be the author of the code in question. My reason for writing the code was to experiment with work distribution across cores and I assumed that l would divide the length of xs. Calling `segment 2 [1,2]` and getting `[[1,2], []]` as result is semantically correct but not the desired operational behaviour.


@pravsripad it is an unrelated file, you can delete it.


Route handlers should be moved to `artexin_web/routes`, but routing should be done in `app.py`.


比徐晓飞的做的好多了，起码条理很清晰


This test changes was made in a way that prevents the test from running on old versions, which won't work going forward. This is why the GKE tests are failing.


I added my favorite Path, but it's not registering with Code School through Git.


И, кстати. scrollPage для постов должен выполняться после полной загрузки страницы (со всеми картинками), иначе может проскроллить не туда.


Don't commit hello world type code. Make a separate branch for your sandbox.


This is wrong. Line above already provides the correct dir.


Is there any benefit in using a compile function ?


This code is somewhat weird: if you've been able to resolve the PsiClass reference, this means that the file is already in the project as part of some library, and you're just adding a duplicate library, which doesn't make much sense. A more interesting sample would be creating a library with different root types (classes, sources, javadoc) from code.


Hello, you forget to add WOOD_PLATE


Views should be in `artexin_web/views` not in the project root. Also, think about making base views and rebasing them. DRY.


If you need a VirtualFile, get it directly from DataContext, not through an Editor instance.


`giveItemRandomly(killer, npc, UNNAMED_RELICS, 1, 20, 1.0, true);`
Also, drop the `Util.contains(MOBS...` check, those are the only mobs registered for onKill so it's a redundant check.


This is an update script, I've just ported @garbas work. Note that the same is used for gitlab, because that's the only sane way for packaging gems at the moment.


It's better to handle *all* static files under `artexin_web/static`, not just css.


Watch the whitespace.


Looks good thanks...

I should have bower updated sometime today along with an updated demo page.


https://github.com/L2J/L2J_Server/commit/3443a3646d673ee552bcde4cf76e6f537de71497#diff-c12786e7341a062f3acf1a70239bec11R187
Your commit.


+                || material == Material.WOOD_PLATE;


You wouldn't need a MW-compatible VM. Just a translator that can interpret the original bytecode and generate something for OpenMW's VM at load time (kind of like how wined3d reads D3D shader bytecode, and generates OpenGL shaders from it). In essence, a decompiler. Then OpenMW's compiler can be as strict as you need it to be since you can ensure that the code generated from decompiling is "clean".


> as far as I understood, correction of a script is always possible, and so if it doesn't wish/won't be able the author of the original mod to make times, the community will be able

Not really. A lot of mods have "no redistribution" or "no alteration" clauses in their license.

Look at this mess for instance: http://mw.modhistory.com/permissions


you're right Bozana, will fix it. Thanks.


There is `$color-white: #ffffff;`


no _this select 1 is correct, a pvEH is always [PVNAME,Content of PV]


@dongqunxi Can you please delete it and push it along with the other changes? I cannot push any changes to your branch.


Nice one. ;)


Is histogram and mode really the same?


Sorry but I'm doing setup troubleshooting help for four years now for Diaspora, keeping the steps at a minimum is worth a real lot, so if we can avoid such requirements,  I'd love to.

What error does `ROW_FORMAT=DYNAMIC` solve?


würde ich auch so machen


Hmm, I think the URL *is* used here. Try making a page named 'something with spaces.md' and navigating to it. In that case, `$data['uri']` be something along the lines of 'something%20with%20spaces.md'. I do agree that it makes more sense to apply `urldecode()` to `$data['uri']` instead of to `$currentpath`, but you are actually more right that this should have been done by PhileCMS itself, and not by a plugin. I've submitted a pull request (here: https://github.com/PhileCMS/Phile/pull/175) to have this fixed.


This line might have been wrongly pasted here by an accident.


Please use `{}` around `++count` -- even for single line statements, as a) this makes the code more readable and b) prevents issues if someone accidently removes the `count++` after which the syntax would still be good.


typo - "is" not "if"


The key values from the pseudo form:
https://pyramid.readthedocs.org/en/1.0-branch/api/request.html#pyramid.request.Request.str_POST


VirtualFileAdapter.this


You need to handle moves/renames as well.


The header 'Content-Type' will have the value 'text/csv' ?


This also looks like a case for the "x${VAR}x" should be used too, I'll update...


This is a really inefficient way to update the project view pane. I'd rather not show any solution to a problem than show an unscalable one.


I'm still confused, haha. Are you telling me I'm wrong or right?
Are you saying that `/joinpass` should be `/joinpassword`?
[Client command line][1] lists `/serverpassword`. Should it be the same for the server? If so, then the [relay server guide][2] has to be updated as well.

Can you be a little more clear?

[1]: http://doc.maniaplanet.com/client/command-line.html
[2]: http://doc.maniaplanet.com/dedicated-server/basic.html#Relay-Servers


Removing `trigger.position().top` breaks positioning of the query menu :disappointed:


Why do you have the default parameter names here? Please setup the project so that all overriding methods have the correct parameter names from the IJ source code.


Iam new use could you please interdouce for me the feachers and the benefits of github please?


Are you using tabs for indents? If so please use 4 spaces instead


NSA commit !


This is a proper package name


1: M111 unrecognized component name: BibConvert


Without knowing what this class is about, I am not sure if `-o` is a suitable suffix for the class name.


I didn't mean client-side, I meant device-side. But ok. I checked and the amrsrc copies the full length of the USB_DATA_SIZE.


Devtools specific examples are moved into a separate repo:
https://github.com/firebug/devtools-extension-examples

Honza


> We are making enough progress and some people have already started to fix their broken mods. If it turns out that a few more mods need fixing that wouldn't be the end of the world.

Well, I have a few problems with this approach.

1. Sample size isn't big enough. There are too many mods for us to accurately say how many of them are working at a given time and how much "progress" we are making.
2. Due to the nature of these workarounds adding one seems to break others. This keeps happening, and as no one has a "master DB" of all mods for testing it is difficult to catch these regressions.
3. Scripts that seem plain impossible to workaround. Would you say we should add a workaround for [this](https://bugs.openmw.org/issues/2030)?
<pre>
; Collision Detection Check
-------------------------------------------------------------------------------------------------------------------------------------------------------------
if ( KS_SR_Boat_Sailmode == 1 )
</pre>


Nice, I didn't even think about being able to do this when I sent the PR to linguist. I was just thinking about highlighting in .elm files.


Accordingly, ManiaPlanetServer.exe says: /servername /serverpassword
/validation /joinpassword there is no joinpass in the code to be seen.

2015-01-08 14:56 GMT+01:00 Kristjan Broder Lund <notifications@github.com>:

> I'm still confused, haha. Are you telling me I'm wrong or right?
> Are you saying that /joinpass should be /joinpassword?
> Client command line <http://doc.maniaplanet.com/client/command-line.html>
> lists /serverpassword. Should it be the same for the server? If so, then
> the relay server guide
> <http://doc.maniaplanet.com/dedicated-server/basic.html#Relay-Servers>
> has to be updated as well.
>
> Can you be a little more clear?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MiniGod/documentation/commit/9315eaac491de2ebcdc5b967f3968c1d14d2db43#commitcomment-9201962>
> .
>


LOL I didn't even realise ```behaviour``` was a correct spelling. Sublimetext just highlighted it :P


need to remove a `%s`


Oh, and I don't know if you noticed, but I also changed Elm's color in the language statistics bar from an uncolored grey to the bluish green from Elm's tangram logo.


Why didn't you just refrain from allowing the exception to raise rather than creating a new Job?


maybe i'm missing something, but why don't you use titleHint/setTitleHint for that ?


Could you please continue? I think this will be a nice example of file I/O. Can I contribute with whatever I can? I'm beginner but have some experience.


Thanks! https://github.com/yiisoft/yii2/commit/23279fafc9efe3b80e296012b6f7694b4a79d23d


OK. I'll change it then.
Are you looking at the source of the server, or is there some way to see which parameters are accepted?


In order to use the HW UART an Rx an Tx Fifo buffer must be defined before calling the UartConfig function.

For I2C there are some working examples on the other supported platforms. Please look at the LoRaMote platform as it is also based on the iM880A module

Please find below a working UART echo example:
```c
#include <string.h>
#include "board.h"
#include "uart-board.h"

/*!
 * FIFO buffers size
 */
#define FIFO_TX_SIZE                                128
#define FIFO_RX_SIZE                                128

uint8_t TxBuffer[FIFO_TX_SIZE];
uint8_t RxBuffer[FIFO_RX_SIZE];

int main( void )
{
    uint8_t d;

    BoardInitMcu( );
    BoardInitPeriph( );

    UartInit( &Uart1, UART_1, UART_TX, UART_RX );

    FifoInit( &Uart1.FifoTx, TxBuffer, FIFO_TX_SIZE );
    FifoInit( &Uart1.FifoRx, RxBuffer, FIFO_RX_SIZE );

    UartConfig( &Uart1, RX_TX, 9600, UART_8_BIT, UART_1_STOP_BIT, NO_PARITY, NO_FLOW_CTRL );

    while( 1 )
    {
        if( UartGetChar( &Uart1, &d ) == 0 )
        {
            while( UartPutChar( &Uart1, d ) != 0 );
        }
    }
}
```


@peti I see some packages in `hackage-packages.nix` have version suffixes (e.g. `array_0_5_0_0`). How does `hackage4nix` decide that? Or, to ask the question I really care about, how do I add an old version of a package?


just wondering if dpkg will bitch if that file's removed when it comes to upgrades?

```
dpkg -S nginx | grep default
nginx-common: /etc/nginx/sites-available/default
nginx-common: /etc/default/nginx
```


source through notepad++.exe

2015-01-08 15:04 GMT+01:00 Kristjan Broder Lund <notifications@github.com>:

> OK. I'll change it then.
> Are you looking at the source of the server, or is there some way to see
> which parameters are accepted?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MiniGod/documentation/commit/9315eaac491de2ebcdc5b967f3968c1d14d2db43#commitcomment-9202071>
> .
>


is `parse` thread safe? I would prefer to not rely on a root var for this, i think is safer to create the `Tika` instance within the method?


Ну офигеть..


There's a lot of stuff in this commit, I think this will conflict with my ongoing work with annotation (I've also updated with aspers info)... And you've removed a lot of close() on files - these statements were just before return() - so they should be closed before return is called.

And if you want to change whitespace, it is better to do a separate commit with *only* formatting changes? It's easier to 'vet' a commit knowing that it's all functional changes or all formatting. I'll try to get this in, but it'll take a little while...


contextBinderMark.length


r+


:p


Sometimes \n sometimes not :D


no need of interpolation

```python
raise UserError(tools.ustr(e))
```


То есть, грузится страница-грузится, а потом внезапно должна скакать к посту....


Yeeeaaah !


I read explanation. I understand why app is crashing when UNICODE is defined. Maybe you have some very specific needs which here is not described, but I don't get why my solution is not working.
I refer to MS docs and what seem conventional wisdom for me. See http://msdn.microsoft.com/en-us/library/dybsewaf.aspx

Maybe you miss **_T** macro which expand string to different literals based on presence of UNICODE define.

So basically when UNICODE not defined  you will have

    LoadLibraryA( "advapi32.dll" );

With UNICODE defined you will have

    LoadLibraryW( L"advapi32.dll" );

Your solution should crash on 32-bit Windows I believe.
Again this is so regular stuff, so maybe I miss something very obvious to you, but not visible to me?


Change to 7.0


Fixed.


Ты на гитхабе посмотри, так и делается. Можно смотреть, скроллил ли пользователь страницу сам и если да, то не скроллить автоматом к посту.

https://github.com/SthephanShinkufag/Dollchan-Extension-Tools/blob/master/Dollchan_Extension_Tools.user.js#L666


Haha, I like the commit message :-)


in Controller.php:392: ```$this->addCSS($ui_path['css'], null, false);```
the "null" prevents jquery ui css files from getting added because they get ```$type = 'js';```


TODO: We can't use ":" as property value.


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/3248/console
Build was first broken at http://ci.cgeo.org/job/c-geo/3246/


Ok, please fix the copyright :)


I think this is repeated.


Fixed it with a patch. It feels like the elm-reactor `Setup.hs` is wrong but I'm not good with haskell/cabal.

https://github.com/NixOS/nixpkgs/pull/5641


This fall through is only correct for a) one data point which is not always true and b)
only for TOP(name,1) which is probably FIRST(name).
For more then one point in the list (which does not even need to be sorted by size), this is not the same


1: M111 unrecognized component name: BibConvert


> I see some packages in hackage-packages.nix have version suffixes (e.g. array_0_5_0_0). How does hackage4nix decide that?

The logic is a little involved. I'll explain that later in more detail.

> How do I add an old version of a package?

Add it to this list: https://github.com/NixOS/cabal2nix/blob/master/src/hackage2nix.hs#L153.


Remove unused/stub/generated/noise methods


Use department level coding style.


OK, but the sentence is not explicit enough.


А ч


If you are asking the difference between GetRawAxis() and GetY(), I think GetY() is based on the mapping in windows, while GetRawAxis is the actual axis built in the xbox controller.


Great to test the UTF grid, but we won't need it for the progress page, right?


Есть более быстрые методы определения хеша?


Ah, I totally noticed and was delighted, but I didn't know it was you. I always assumed the colors were somewhat random. In any case, great idea, it looks really nice! :D


Or simply
The 'Content-Type' header is 'text/csv'.


I moved the fileopen and with that I can remove alot of extra closes..
The isssue with whitespace I didn't know.  I guess that just comes when I made changes in a row..


Или это все вынесется куда-нибудь в асинхронную функцию,


Tested?


better? `use_lzma = (args[1] == '1')`


Thank you for fixing this!


Would be nice to have app icon back too:  https://wiki.libsdl.org/SDL_SetWindowIcon


Тому що тупо убирать весь скроллинг тоже как-то непраильно.


Не в хэше дело. Сама функция `scrollTo` очень медленная + определение верхней координаты поста `pByNum[val].topCoord` выполняется тоже очень медленно. У меня всё это вместе занимает вплоть до полторы секунды (при том, что всё остальное выполняется за 0.180 секунд).


:-1: As indicated in the [comment in the spec](https://github.com/mysociety/alaveteli/commit/5d61441cc3dbb468a207e714db64ff2d2ee84bfc#diff-851596e66f907d9ac8cec23f2f40d677R1071), the /new url is used to record the completion of the 'Make a new request' goal in google analytics. More details in the [original commit message](https://github.com/mysociety/alaveteli/commit/bd2472e13a03f4f30c96a5d694efdc5c76798662).


вы с заглавной


Whoops... need to add each one explicitly. Will add that in the next version.


Нужно выполнять по событию `window.onload` как я и предлагал + следить за пользовательским скроллом.


Works for me!


What's Polar doing here?


nit, `if (exists)`


This service returns a file containing data submitted in the POST request as the "file" field.


Naughty!!!


Hi,
You're right. Z-scores are not necessarily needed for a heatmap visualization (by the way; marker selection forwards its data to HeatmapLoader.R, which does not cluster (or scale) at all). However, the heatmap code does assume z-score data on some level (if only because it clamps the values by default to [-2.5, 2.5], and all of the three heatmap analyses always get the z-score projection data). I understand that this is a problematic assumption, because as you point out, most of the time it will only receive a subset of the original data upon which the z-scores were calculated, and perhaps it would be better to calculate z-scores on the eventual subset. Unfortunately, this does mean that either those faulty assumptions need to be rectified when log values are supplied instead, or the data needs to be scaled as per your suggested commit.
I am bringing this up because we encountered this issue by comparing the latest 1.2 analyses results with an older build (1.1), and it is a rather large change that introduces a sort of "bug" in the heatmap visualized for marker selection.


I still get the issue when the build fails somehow at the `do_img` part:
```
+ truncate -s 1200000000 xbian-image-rpi-20150108.img
+ '[' -f /etc/centos-release ']'
+ parted -s xbian-image-rpi-20150108.img mklabel msdos
+ '[' -z ']'
+ config_offset_root=141312
+ '[' -z vfat ']'
+ echo '2048,69632,b,*,'
+ sfdisk -u S -N1 -H 4 -S 16 -q xbian-image-rpi-20150108.img
+ echo 141312,+,83,,
+ sfdisk -u S -N2 -H 4 -S 16 -q xbian-image-rpi-20150108.img
++ kpartx -s -l xbian-image-rpi-20150108.img -av
++ tail -1
++ awk '{print $3}'
+ loopd=loop0p1
+ loopd=loop0
+ echo loopd=loop0
+ '[' -z vfat ']'
+ rootpart=2
+ case $config_fstype_boot in
+ mkfs.msdos -F 16 -n xbianboot /dev/mapper/loop0p1
+ case $config_fstype_root in
+ mkfs.btrfs --help
+ grep -q '\-O'
+ mkfs.btrfs -m single -d single -n 16384 -L xbian /dev/mapper/loop0p2
check_mounted(): Could not open /dev/mapper/loop0p2
error checking /dev/mapper/loop0p2 mount status

WARNING! - Btrfs Btrfs v0.20-rc1 IS EXPERIMENTAL
WARNING! - see http://btrfs.wiki.kernel.org before using

+ clean_tmp
+ set +e
+ umfail=1
+ '[' 1 -eq 1 ']'
+ umfail=0
+ for d in rootfs/var/cache/apt/archives rootfs/tmp/tmp
+ mountpoint -q /opt/xbian/xbian-build-img/build/rpi/working/rootfs/var/cache/apt/archives
+ for d in rootfs/var/cache/apt/archives rootfs/tmp/tmp
+ mountpoint -q /opt/xbian/xbian-build-img/build/rpi/working/rootfs/tmp/tmp
+ '[' 0 -eq 0 ']'
+ '[' 0 -eq 1 ']'
+ set -e
+ return 0
+ clean_all
+ set +e
++ mount
++ grep 'mapper/\|build/rpi/working/rootfs'
++ awk '$5 ~ /btrfs/ {print $3}'
+ umfail=1
+ '[' 1 -eq 1 ']'
+ umfail=0
++ mount
++ grep 'mapper/\|build/rpi/working/rootfs'
++ awk '{print $3}'
++ sort -r
+ for d in '$(mount | grep "mapper/$loopd\|$dir/working/rootfs" | awk '\''{print $3}'\'' | sort -r)'
++ mountpoint -q /home
++ echo 0
+ '[' 0 -eq 0 ']'
+ '[' /home '!=' / ']'
+ echo /home
+ for d in '$(mount | grep "mapper/$loopd\|$dir/working/rootfs" | awk '\''{print $3}'\'' | sort -r)'
/home
++ mountpoint -q /
++ echo 0
+ '[' 0 -eq 0 ']'
+ '[' / '!=' / ']'
+ '[' 0 -eq 0 ']'
+ '[' 0 -eq 1 ']'
+ set -e
+ return 0
+ clean_loopd
+ set +e
+ cd /opt/xbian/xbian-build-img/build/rpi/working
+ kpartx -dv xbian-image-rpi-20150108.img
+ rm -f /run/xbian-image-rpi-20150108.img.loopd
+ set -e
+ return 0
+ rm -f /opt/xbian/xbian-build-img/build.~lock
```

As you can see it wanted to umount `/home`


Consider renaming `tiles` to `tileUrlTemplates`


1: M101 signature is missing
2: M102 unrecognized bullet/signature
3: M102 unrecognized bullet/signature
4: M102 unrecognized bullet/signature
1: M110 missing component name
1: M190 line is too long (66 > 50)


This broke the Python 3 builds.


Side note: lololol I just realized GitHub tagged the user `@default` in the commit message


Hello Oleg,
I do not think that this will look good. Did you have tested this with a bootstrap?
In the border-right property there is no property inherit:
border-right: 1px solid inherit
is not correct (you can check this)
For me this add a extra vertical border which does not look good.
I have try to support bootstrap with very minimal changes where the important thing is border colapse property set on the both tables.
You can view my demo here: http://www.guriddo.net/demo/guriddojs/integrations/bootstrap/index.html

Kind Regards
Tony


Okay, that makes sense. My initial reasoning (https://github.com/catharsis/syslog-ng/commit/dd5de7ef496d1b748445dbee4c24d9c5b91f3a94) was that the encoding() option lives there, and this is fairly related. Especially since there are interdependencies between the encoding and the encoding mode. But of course, I might be completely off. The msg_format_option rules lie on the same level config-file-wise anyways, so it wouldn't be much of a problem moving it, I don't think, but it'd still be necessary to pass along the encoding unless we move that as well. What do you think?


Yeah noted. Looking into it. Probably we have a minimum version requirement we've not noticed.


Good catch.  I made this change in response to someone else's feedback and didn't really give it my full attention.  Also there is no test setup on this project where I probably would have caught this normally.


**SUCCESS** - TeamCity Infinispan :: Pull requests monitor [Build 8254](http://ci.infinispan.org/viewLog.html?buildId=22681&buildTypeId=bt9&guest=1) for 79910072e95d98c9dbdd35294d8d749581bbb18f
Tests: 11231, 0 failed (0 new), 23 ignored. Build time: 01:41:34


Related: http://stackoverflow.com/questions/27497676/django-rest-api-urlsplit-error


Is this really right? Doesn't this break the build-for-publication?


Thank you!


We're almost there! Unfortunately `reduce` is not available in Python3 anymore, so the unittests will fail in Python3. Can you replace `reduce` with `functools.reduce` (which will work in Python2 and -3), an explicit for-loop, or something similar?


With `-t markdown_github`, `pandoc` generates normal HTML tables, which should always work.


@pmall +1


:-1: You've created a partial for the text used on the creation of a new request here, but you've only removed the text from the controller action which is called when creating batch requests. In addition the text for the two actions is different e.g. they start

Your {{law_used_full}} requests will be sent shortly!

vs

Your {{law_used_full}} request has been sent on its way!


c'est bon, tu as modifié le fichier


I think there is bug if libDir is exists but not a directory.


logging?


Just pushed 0.3.4


Yeah, sorry :sob:

The alternative is to change all the generated code to me more explicit or to change `DefaultProduct` (I don't think the `isIncluded` method is actually used).


@pravsripad I will push it, after I finish all changes.


Actually the designer was not at all present beforehand. It was completely added in this.


Reset the duration to 0 is weird from the user's perspective. I would prefer to keep the finalization of the custom period in a separated property and provide the user some way to reset the custom plan for the same parameters.


:+1: nice, simple, readable test.


I'm a little confused about the difference between the `simpligility-groupid` and `android-maven-plugin-4` projects: why does `simpligility-groupid` have an invalid project layout?


тут лажа


1: M111 unrecognized component name: WebBasket