@deMattin sorry but I have to disagree here.

> I would like it more, that he invests his time in coding and fixing and not in documenting and splitting pulls.

This results in only @libasys working on the app and the development process and progress being entirely opaque to other people. There are lots of other folks who want to contribute, among them @sahne123 @Henni @ChristophWurst @georgehrke @raghunayyar and me. If making the process more public allows even one more person to join in, @libasys wouldn’t need to crunch like he does now, or stuff could be tested better and reviewed by others.

For this to work, we have to work on the app as a team, not as just one person. Especially because for one person it is impossible to find many common issues – we need a broad developer base for such an essential app as Calendar so it’s properly stable and tested cc @DeepDiver1975 @LukasReschke.


OK, cool.


Grand.


It might be helpful to also comment on the second conditional test that filters out non-system suspend operations: presumably the TLK has nothing to do in those cases and the call to Secure-EL1 is relatively expensive.


Fair enough.


Тлько 26 кд не концентрации, а 10к копий. А для концентрации не будет лишним поставить кд 30


The cut and paste from TSPD is evident here. You might consider making the second sentence directly relevant to TLK and what it needs to do in this situation (information that is in the commit message...)


With this, an objectID can't be a Unicode or a number like with the other API client. It's not the good solution. What is the issue with the initial implementation?


yep, just to leave the repo in a clean state.
no need to re run `meteor publish` though!


Bug number please.


угумс, апчаткен вышло


feel free to do it by hands and use a different commit message if you wish ;-)


I was a bit careless and this breaks the apiserver if the server is restarted, because it looks for the apiserver.key but it's not there and it does not regenerate it. Also this is just a partial solution, because the ca.crt is still missing from the serviceaccount tokens. I figured out a solution but there is still one problem to be solved. On the master a ca.crt and a keypair has to be generated once, then it can be used permanently.


I presume you do the current CPU index calculation in a few places, and if there is ever a Tegra platform that exposes multiple clusters to the OS it would have been handy to encapsulate this in a macro or function within your port?


:ok:


Found this with Nelson too. I'd think last 2 versions would have been default.


Added the changes


There are two 11 thread machines available.


感谢你的贡献


I marked some methods as public and some methods as internal. I think it should work fine now.


This is interesting, and we might consider including it if it was made more optional, but as it is, there are a few problems.

What we tend to do in other projects is only call google analytics if that key is set - this allows dev and testing versions not to use it, and obviously any other users who don't want it. Perhaps the easiest way to do this would be to create a couple of functions on the FMS object that wrapping the calls to the analytics api, so that you can check the config setting exists first.

Also, it would be useful to add some documentation about this, for example - I think this means we need to add a new plugin to the app, so it would be good to explain that.


where will you put the limit?


I know, otherwise... what's the point. Seems a little too explicit


Þú geturn núna hent inn myndum og sett upplýsingar um sér hverja vöru í app.js skránna inn í ProductsControllernum.


If PSTATE_ID_CLUSTER_IDLE is a retention state for  the platform (which the name seems to suggest), i.e the cache at cluster is contents are retained and cache maintenance operations are not to be performed, then it is more efficient to inform this state as a retention state to PSCI generic layer. For this PLAT_MAX_RET_STATE has to be modified suitably so that it is >= PSTATE_ID_CLUSTER_IDLE.

All the power down states then has  to be > PLAT_MAX_RET_STATE and <= PLAT_MAX_OFF_STATE. If your state-IDs dont match this criteria, you could consider ORing a value (say 0x80) to the power down state-id when they are populated in the req_state[] array. Of course that means you would need to mask this value before using it in your suspend handler.


Happy this commit helped.


101615016CoilCoolingDXMultiSpeed-requires-Waste-Heat-Function (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1278 of 1278 tests passed)

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


@signifox I think variable `e` will never be nil. Would you remove this modification?


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


first commit


Please be safe, thank you


Nice :)


Added to code school path


That's Ubuntu specific. The nvidia-prime package does this https://wiki.archlinux.org/index.php/NVIDIA_Optimus#Using_nvidia which needs to be done manually on most other distributions.


https://github.com/shadowsocks/shadowsocks-windows

Thank you all so much.
and may the force be with you


nit: use PLATFORM_CORE_COUNT here ?


You are copying 32 bit variable to a 8 bit array element. Ideally you only want only the state-id to be copied to the pwr_domain_state[]. So this statement should be:
req_state->pwr_domain_state[i] = state_id & 0xff;


Ah, yes this issue has cropped up a few times before... That's the compiler trying to be more helpful than anticipated. Simplest is to merge all the files in `lib/sparse/` into a single `lib/image_sparse.h` header...


This is what get_av() does.


Abi yorum modelini aşağıdaki gibi oluşturdum:

class Comment(models.Model):
    comment_text = models.TextField(max_length=200)
    reply = models.ForeignKey('self', blank=True, null=True)
    comment_blog = models.ForeignKey(Blog, null=True, blank=True)
    comment_mail = models.EmailField()
    comment_name = models.CharField(max_length=200, null=True)

Self foreign key verdiğim için yoruma yorum yazdırabiliyorum ama farklı bir yolla yap dersen onu da denerim.


I will try to put this code into JogAmp's Ardor3D Continuation with the following changes:
- addition of a Font class into ardor3d-core, move of the AWT code into an AWT front provider in ardor3d-awt (or ardor3d-jogl-awt as it requires GLU), implementation of an AWT-free font provider based on the Typecast branch within JOGL
- addition of an API exposing some GLU methods into ardor3d-core, the concrete implementation will be in ardor3d-jogl. If the GLU tessellation doesn't work both with OpenGL and OpenGL ES, I will have to implement a similar mechanism without GLU
- replacement of all JOGL specific code in GlyphTessellatorCallback and Glyph by renderer agnostic code
- split of the example into 2 (one example per font provider)
- use of JOGL 2.3.2 RC


I agree. If not now, it might be useful when/if such a Tegra platform comes into existence.


You only need `assertEquals`


修改了URL


:3 why 400?


Same comment to the one here:
https://github.com/vwadekar/arm-trusted-firmware/commit/58bdcf68b6fffc6829358ff6c1d0936229eb966b#diff-c0f3dfc73bc47ce63ab71af1a9a7c0feR86


11 threads can exercise depths upto 32.
3 + 28/4 = 11

But for such high depths we may need more than 15+0.05


`Observable#takeUntil` ([source](https://github.com/ReactiveX/RxJava/blob/689e73ff2aa2c2715daf56ea3833dcc07fbf18d2/src/main/java/rx/Observable.java#L8231-L8233)) doesn't care about the type that is outputted, as long as every item is assignable from `E`

You probably want to mimic this signature.


The objectID is always a string for the backend point of view, so it doesn't matter if we convert is before or after the api call. I added the conversion to fix the call to the `safe` method with an integer.


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Maybe a dedicated object is overdoing it, I was just worried about accidental "forgery", but it is very unlikely, and it voids your guarantee :)


ah, right, thanks


Anyone who's in a group calling themselves ``` space niggeers space nigs ``` isn't someone nice to play with.

Almost all propkillers ruin the game of whatever servers they join, and have various cheats & bypasses, and are 12 year old skiddies.
I once ran a decoy propkill server in 2012 that allowed scripts to run, and more often than not they had tens of bad lua files, forced cvars, stupid troll names etc.


LGTM


Would prefer the PSTATE_ID_CORE_POWERDN macro and the function tegra_fc_cpu_idle() to match. Suggest tegra_fc_cpu_powerdn() ?


On Wed, Aug 26, 2015 at 03:18:03AM -0700, Caerbannog wrote:
> "Typically"
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/KiCad/kicad-doc/commit/e3677f78d2937f065cae890716ae719e4aef727a#commitcomment-12899580

Ooops thanks!

--

Marco Ciampa

I know a joke about UDP, but you might not get it.

+------------------------+
| GNU/Linux User  #78271 |
| FSFE fellow       #364 |
+------------------------+


Thanks for contributing this!

I think it can be easily modified to check if ``tg.hooks`` is available and only in such case rely on it. That way we can keep compatibility with previous TG versions. In the mean time I'm going to merge this.


ae @gpedro default plugin locales!


You can split it up like this:
```python
    >>> all_layer_names = [l.__class__.__name__ for l in get_all_layers(l_out)]
    >>> all_layer_names[:4]
    [['InputLayer', 'DenseLayer', 'BiasLayer', 'NonlinearityLayer']
    >>> all_layer_names[4:]
    ['DenseLayer', 'BiasLayer', 'NonlinearityLayer']
```


* Could you fix the sphinx test to copy data files to tmp directory like /tmp/pytest-301/test_sphinx_onedir_0/ and then instruct the test to use data files from that tmpdir?
* There should be already some tests that use tmpdir.


`bath_path` in the commit message :bathtub: :walking:


add blank line


This "passing the name and the value" seems awkward, and I think is just something being forced on you by the fact that the ImageWriter::WriteMode enum hasn't been bound to python (there is actually a todo in ImageWriterUI.py about doing that). Would you be willing to do that as well as part of this PR? If not, I can take a look once its merged...


Also, note  that the formula 3 + depth/5 has already been tested at 9 threads STC and showed some elo gain after 10K games.
http://tests.stockfishchess.org/tests/view/55d820120ebc595da9433856


[蜡烛]


No - both are done now


Hi
thank you for you sharing code, it's really great , I found that GPA0 and GPB0 could not turn on the LED , it's pin number 9 and 10 , do you have any idea ?
I use only your module : mcp23017.lua
my firmware is : NodeMCU 0.9.6 build 20150704
esp1


Thanks!


You would need to calculate a CPU index which would correspond to the PLATFORM_CORE_COUNT. So ideally you should return cpu_id * cluster_id. See further comments in topology description of T210 platform.


It's called from the cleanup functions per resource (CQ, QP ...) that call rxe_queue_cleanup function that finally call rxe_mmap_release function.


See comments for Line 260.


All the power levels in the target state should be set to RUN. You could do this by:

psci_power_state_t target_state = { {PSCI_LOCAL_STATE_RUN} };


You may want to modify the loop as following:

for (int i = MPIDR_AFFLVL0; i < PLAT_MAX_PWR_LVL; i++)
 	req_state->pwr_domain_state[i] = PLAT_MAX_OFF_STATE;

req_state->pwr_domain_state[PLAT_MAX_PWR_LVL]  = PLAT_SYS_SUSPEND_STATE_ID;


See whitespace.rpgle.  It now drops all of the blank lines (correctly), except for the o spec.


You may want to reconsider this function for T210 platform. See comments below for the topology construction.


@ccx-testerbot runtests feature-cea-708


That just wasn't made official in the composer.json so by default it was supporting 5.3.  But I don't see why we can't jump up to requiring PHP 5.5 (5.4 reaches EOL in a few days) as it doesn't seem likely anyone's using this in the wild - and even if they are, the existing library will still work in previous versions. I'll specify 5.5.


How come youdon't clear all IPCB, and don't call skb_scrub_packet, clear_hash, etc. here?


thanks!


This is overridden below.


Typo :)
```
"grey-blue": $clr-blue-grey-list
```


Use a constant var and change de URI


It's not a serious group and was not intended to be public in any way, only to be used as a white list for a week or two while we played space engineers so no one else could join the server.

What about the PropKillers that don't take part in hacking/trolling and just PropKill because its a fun and competitive sport, and also something unique to be good at?


This looks like it should be placed in a test fixture. Needs to be removed as it now stands before merging. (as already noted in the commit message).


I have doubts on how this topology description is defined. If you refer the topology description guide [here](https://github.com/ARM-software/arm-trusted-firmware/blob/master/docs/psci-pd-tree.md), you will see that T210 topology tree needs to be defined as:
```
const unsigned char tegra_power_domain_tree_desc[] = {
	/* No of root nodes */
	1,
	/* No of children for the root node */
	PLATFORM_CLUSTER_COUNT,
	/* No of children for the first cluster node */
	CLUSTER0_CORE_COUNT,
	/* No of children for the second cluster node */
	CLUSTER1_CORE_COUNT
};
```

Since both the cluster are having the same number of CPUs, both CLUSTER0_CORE_COUNT and CLUSTER1_CORE_COUNT can be defined to PLATFORM_MAX_CPUS_PER_CLUSTER.

You would also need to redefine plat_my_core_pos() and plat_core_pos_by_mpidr() to return an index which is equal to cluster id * core id.

For more details, please refer [here](https://github.com/ARM-software/arm-trusted-firmware/blob/master/docs/psci-pd-tree.md)


Is there a reason you've standardised on right single quotes rather than apostrophes?

I think I'd prefer to standardise on apostrophes as I suspect that's what most people are going to use when writing content. In addition, apostrophes are converted to right single quotes when they go through Govspeak:

```
>> Govspeak::Document.new("Shouldn't").to_html.html_safe
=> "<p>Shouldn’t</p>\n"
```


### Missing semicolon.

```
return "Hello, World!"
                      ^
```


### Use '===' to compare with 'null'.

```
if (bla == null) return;
        ^
```


Thank you for the request. It has been added to the queue (id: 75). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/75) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


Don't use C++ style comments.


1. *shadow = size & KASAN_SHADOW_MASK;
2. This poisons only one shadow byte. I think, we should poison all shadow that doesn't belong to module allocation. IOW: [shadow, shadow_start + shadow_size]
3. This should be split into two patches. First fixes calculation of shadow_size, the second is poisons memory after module.


Why did you have to change from JDK_1_8 -> JDK_1_7?


Thanks for finding / fixing this. Minor remark: although the fix works OK, I would have preferred LogError over qDebug, as the pull parser tries to route all output through a few private functions. This will enable catching / displaying the errors in the GUI if we want to later.


*Route?


what is this file for?
Why did you name this file as shashankreddy509.xml?


Thanks! I am working on desktop workspaces right now too. They work, I just need a good keybinding for changing between them. :smile:


Can you explain where is used each authorization?


Why is misc.xml?
Proper naming for files is very important?


Why do you want to do two logically unrelated changes in a single commit?


what is this?
The app should be called as Bullion. why tracker and stuff?


This line conflicts with line 14. Now empty root password is no longer a valid option.


Can you please amend the commit where I forgot to add these free() calls?


Could be done given the reductions with the new syntax. Though part of the logic behind the format (templated sparse class, so new classes can be introduced to serve different needs) would be lost if it were all embedded in a single file.

Let's leave it since this seems to work, but if it trips anyone over it can be changed.


Why are line-6 & line-7 duplicates?


Thank you , sir. Take care of yourself.


Did you mean to move this onto a newline? Although the Govspeak output will have changed I don't think it's going to be rendered any differently in the HTML. You'd need another blank line between those two in order to have the two lines split into two paragraphs in HTML.


what does version '4' indicate here?
and reason for choosing that version?

Bullian-Tracker -> should it be Bullian?


:ok:


I think we should be eliminating race conditions by implementing the required dependencies between events instead of timeouts like this. Can't we do that in this case?


What's this change for? It doesn't appear to be used.


@moolitayer just make sure this works in both kubeclient and openshift_client.
This was added as a patch to an issue in openshift_client, which didn't initialize headers correctly.


For each dependency - please add a small comment why this is taken. one of the aims of the project is to keep it minimal size.


actually its node-sass < 3.3.0


What about to use ``continue`` instead of ``return``? In this case a single array entry is invalid but the next one can be OK.


Hey, that commit lacks the `getSongByElementNumber` it seems.


cc @yole


Thanks. But unfortunately is this google service not really robust.

Perhaps openuserjs could provide a little service by using https://github.com/sentientwaffle/node-favicon ;-)


Please rename ``errors `` to ``error``.


interface <-> Object (kind of contradictory)

All interfaces should be added enough description - provide a bit more detail.


Missing ``json_object_put(errors_array);``


Yes, I agree. I am new to GitHub and php. I tought after sending the pull
request that it was a bad idea but at least I tried :)

2015-08-26 12:03 GMT+02:00 SoftwareSpot Apps <notifications@github.com>:

> A couple of points in regards to your PR:
>
>    - 400 should be changed to one of the predefined constants used in the
>    library (see top of the page).
>    - Don't hard code language strings in the library, use one the
>    predefined language files instead.
>    - Whitespace is missing between the end bracket and curly braces, as
>    well as the fat arrows.
>    - Doesn't conform to CI3's style guide e.g. we use Alman styling for
>    curly brackets.
>    - Doc comment is a mismatch with the rest of the library.
>
> Then I have a question. Why is it restricted to POST only?
>
> Honestly, this is just my opinion, but at present this is to specific and
> something a user could easily/should easily create themselves when they
> extend the REST server library.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TarikHuber/codeigniter-restserver/commit/fa8dc0e527b8527cf22a7312fbcfa78438652c32#commitcomment-12899353>
> .
>

--
Ing. Tarik Huber
Sudetenplatz 5
Freilassing 83395
Mob: 0049 152 318 413 02
E-Mail: tarik.huber@ics-logistik.com


The ConnectionManager has a timer running whilst connecting, and when the connection attempt times out it transitions to `disconnected`. I think the `ConnectionManager` should then be the thing that disposes the transport. I don't think we should have a separate timer running here.


Как-то у тебя редактор так настроен что ли, что не добавляет перевод строки в конец файлов. Хорошо бы исправить. Кажется, в IDEA должна быть настройка по этому поводу.


I think another project was configuring this correctly for us. TeamCity pollution :cry:


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (757 of 757 tests passed)

![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-17.62%25-green.svg)


Why does the function need to clone every skb?


cool


you should not override the ESMailboxModule with inheritance, you should just create a module and add it as an override module


It creates `lib/smart_answer_flows/flow_name/partials/partial.erb`that is yielded as one of the files to be detected in the test above ("include the path of erb templates"). If you undo the change in `lib/smart_answer_files.rb` the test will fail because only the two files `lib/smart_answer_flows/flow_name/flow_name-*` will be detected.


This might have broken pypy: http://hydra.nixos.org/build/25054672/log/raw


The function's name suggests that the function returns the error message but the function does return nothing and prints out the error message.


Здесь видна попытка сделать форматирование, но в результате его нет: https://github.com/flexjoy/SpringHelloWorld

Когда коммит относится к какой-то задаче (issue), то желательно в комментарии указывать. Я обычно пишу что-то типа "Fix #N" или "Addressed to #N"


@CaseyCarter I had to change this for the sake of compile times. Another way to fix the compile times is by hiding this definition of `Boolean` behind a type trait. LMK if you find that solution more palatable.


The array is called errors. See Error response example so that is the reason why I named it errors_array.


Is it worth handling NET_XMIT_CN explicitly?


Will fix that behavior in https://github.com/mp911de/lettuce/issues/123 (releasing lettuce 3.3.Final tomorrow).


This is an interface -> and this method provides all the information about curreny to the world. It is very important to document the following:
1. What will this do?
2. What will it do?
3. What happens in failure scenarios?

Why is the return type void?


I think that the error checking should be done before you call ``bodhi_parse_json()``.


your bool values are:
```
    data.WriteBit(IsParty);
    data.WriteBit(NotifyUI);
    data.WriteBit(Joined);
    data.WriteBit(LfgJoined);
    data.WriteBit(Queued);
```


Den klagar på

Path: /Users/toffe/dev/mushbarf/Vagrantfile
Message: cannot load such file -- /Users/toffe/dev/mushbarf/vm/Vagrantfile


THX i'll fix it in a few days. Find7s?


see bindConstant here http://stackoverflow.com/questions/4165506/guice-difference-between-binderbindconstant-and-binderbind-toinstance


@gocursor - have a look at the PR that I just merged in. It deals precisely with the extra "\r" that Windows adds in. I think it should all be working now.


Same comments as above.


As far as I understand, rxe_rcv return value matches the convention of the encapsulated udp receive callback, which is not your ordinary negative errno value.


I don't see the point creating this here. Why it's not in a guice module ?


Haha. Awesome. You've been coding since I last spoke to you ?


how is it related to ES ?


This is an implementation - Does it have to be public?
Ideally we don't have to expose this (and use the DI framework to create this objects and give as required)


typo: _compute


Also, do you really need a special loopback case? Won't this be handled by the Linux UDP/IP layer?


``errors_array`` is OK, ``errors`` used as a name of variable holding a single item from the ``errors_array`` is not OK.


looool :D Ob da jemals schon mal irgendwer was eingetragen hat? :D


nvm, det beror på min sed version som krashar under make vm-install


How come you don't check the full address?


And the full GID table?


Fixes #1099


why ?


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **82/87** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/75) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


yes, but the order isn't easy to have... i ve only one sniff file for lfg...


This is an interface - In this case, these should belong to Implementations.


don't use inheritance, use Module.override


This does not effect the PCampReview or does it? There we agreed only to have one input selector im more than one MV exists.

But in general it makes sense to do it for the second MV as well.


Should give the same input as the UI class does


Actually, I think you can drop the av parameter and get it from the pkt_info.


I worked my whole evening. Now I have to run off to work, but basically code is all I do after work and on my days off.


These should be separated. A "Render" source or something for the rendertexture, since it's not Luma specific. And please try separate commits of luma specifics and non-luma to make merging upstream easier.


얼니ㅓㅏ닝ㄹ


why do you call bind here ?


Sweet :+1:


looks like the wrong import for jms


Gostei disso.


A classy commit message!


Isso eu nçao entendi para q q serve.


Should you open a pull request for this in the future, please drop the bugzilla link. This is a login-walled ticket and that's not something we want in the upstream history -- unless anyone can create an account with appropriate permissions to see the ticket, which I don't think is the case.


@buchslava remove width


bind twice ?


That looks to be the thing I was trying to solve. I didn't dig too deep, but I'm reasonably sure that's it.


OK


@buchslava use `${['More', 'Extra', 'Lots of', 'Surplus'][this.slides.length % 4]} ${['Cats', 'Kittys', 'Felines', 'Cutes'][this.slides.length % 4] }`


use @Provides method for this instead of calling "new" on EmbeddedActiveMQ


remove


good to know, I will check.


?actual?


@buchslava remove `else`s


We shouldn't need this check - it dates back to a time when ImageWriter (unwisely) monkeyed with its own ReadOnly states.


> you can't trust this key even if the commits are signed with this key

The previous signed commit long ago will remains in the forks even after your account is stolen by big brother thus preventing them deliver an evil key. What I want to point out is that GPG encourage people validate keys they newly received by other trusted method, for example, telephone, IM etc. if we know each other. Otherwise it's better to upload the key on a trusted key server and have some friends sign it ( This is not a trusted method since people can arbitrarily sign the key.)


thanks,and could we do something special here?eg:catch the producer already closed exception,and if the producer is closed,we recreated one,and when the actor is restarting ,we could cache the valid one in a concurrernthashmap,and reuse it again?


simplify


This isn't necessay, few lines below, you reassign CFLAGS to the same value.


so it's "startSmtpServer"


Sorry, I don't really know C++/Qt, and it doesn't look like something that I would be able to fix quickly using http://doc.qt.io/qt-5/qdesktopservices.html .
Just so you know, this *fixes* 2 issues, and if someone knows how to make code better - they should do it.

For now, I'll be working on going through issues, tagging, etc, since that's what I can do.

After I'll get over the issues, I'll look into trying to "fix" this PR, so that it could adhere to what you proposed. (I can't promise though that I will be able to "fix" it.)


This is an interface class, and you are fixing the implementation specific here.
This should go into correct implementation class.


Shouldn't an ib event be dispatched here?


现在仅仅使用了html来说明创建时间，后期还会继续修改的！
:dancer:


I'm quite sure you can do everything with a single injector, just use @Provides methods


Dead code


[Citation needed]


please, replace spaces to tabs in makefiles


"sollten" statt können?


same as above


Also did you notice, public is not required here?
it is implicit on a public interface -> all methods on a public interface are public by default.


/usr/bin toffe@Anduril.local: /usr/bin/sed --version
ger
/usr/bin/sed: illegal option -- -

medan
/usr/local/Cellar/gnu-sed/4.2.2/bin/sed --version
ger
/usr/local/Cellar/gnu-sed/4.2.2/bin/sed (GNU sed) 4.2.2

satte in
alias sed=/usr/local/Cellar/gnu-sed/4.2.2/bin/sed
i min bash_profile


LGTM.


Dann können wir auch "müssten" (in quotes) nehmen :P


index should be always present, simplify logic and remove getCurrentIndex


new line, maybe?


Please port the account.financial.report model to the v8 api.


Yes! That was my cunning plan as well. It *should* work, and I really *should* make time to do it.


index always here


new line?


Shouldn't use magic numbers. Please add IB_PHYS_STATE_DISABLED


Ah, OK. I understand now.


There should be an event here too.


you should not keep the whole injector, only a Set<ProtocolHandler> and define a multibinder to populate it


可以，先暂时保存一个版本吧！
:game_die:


since this is an implementation, we can say the 'server' name


use DIRECTION enum const


Should rather expose a camera field allowing the user to set the camera to use.


g


Hmm, kommer ännu...

~/dev/mushbarf toffe@Anduril.local master: make vm-install
> Restart services if vagrant is already running ...
vagrant status | grep -qe 'running' && (vagrant ssh -c 'sudo service mysql restart; sudo service apache2 restart') || true
There was an error loading a Vagrantfile. The file being loaded
and the error message are shown below. This is usually caused by
a syntax error.

Path: /Users/toffe/dev/mushbarf/Vagrantfile
Message: cannot load such file -- /Users/toffe/dev/mushbarf/vm/Vagrantfile
> Start and provision the vagrant environment ...
vagrant up --provision && vagrant provision
There was an error loading a Vagrantfile. The file being loaded
and the error message are shown below. This is usually caused by
a syntax error.

Path: /Users/toffe/dev/mushbarf/Vagrantfile
Message: cannot load such file -- /Users/toffe/dev/mushbarf/vm/Vagrantfile


Sollte geschehen sein.


I tested this version on Windows and it builds now without problems...


this should return the data.


why ?


It is working now! Thank you! I am waiting for the release... :-)


`skillsss` fixed: 7bb1a3120e41d96707b288606d4bf249107da4f7


Die Anforderung ist Niveau B2 (wobei unter https://tu-dresden.de/die_tu_dresden/fakultaeten/fakultaet_informatik/studium/studiengaenge/master_informatik noch B2+ steht, das sollte doch geändert werden?). Ob man das in 4 SWS macht oder anders nachweist, ist egal.


simplify and remove


no 'state' (fields holding data) in Interfaces.


I assume you are putting a new Connection object to the store in order to account for the check if the client still has the same IP. I think this is the correct approach if the client has a different IP address. However, if the client still has the same IP address, the already established session will no longer be usable. This might be unlikely anyway because the client tries to resume the session (usually meaning that he has lost the connection), however, I think we should re-use the existing Connection object in this case instead of replacing it with a new one.


where are the methods?
and why is this a Dao?
Dashboard I think is for graphing - why does it have to be Dao?


useless, it's the default behavior of guice


Sorry, @rxaviers . I meant for you to combine the two sentences, because currently the first one is a fragment.

> If you're using a different tool than the one above, but you're comfortable


LGTM


no new, please, use @Provides method instead


1025 is not ok ?


Oh! Right. My bad, thanks.


why techradicle?
why not shashank?
Why is the package name 'DAO', java standards, the package name should be small letters always. (i can provide more pointer with problems of using 'CAPS' if you want me to)


Why do you need to release the spinlock? Does rxe_remove() block?

Can another device register here while the lock was released?


ok, this didn't work, see https://travis-ci.org/ismail-s/locust/builds/77307282


why empty line?
it can effect readability.


monetary widget


It could also be c234f37b5995457e2a5b518648e6735612cb3c27


should we call this method? getHistory?


not need for ^ in event handler


public interface.


谢谢你。这是我用过最好用的科学上网工具。虽然项目不再更新，但是已经部署好的还可以使用。谢谢你。


remove


DAO -> dao


If register_netdevice_notifier fails, you don't release the tunnels.


why there's lmtp module introduction in this commit ?


this and above lists are implementaion specifics.


!slides.length


you should not need the version here


Hmm, this might've caused pypy to fail: http://hydra.nixos.org/build/25054672/log/raw


And added a timestamp


Testing build with this reverted.


Where is it used ?


all private


Please, drop those comments.


Thx :+1:

Dann können ja mal wieder Mails von Matthias Peukert kommen :P


documentation is imp.
lets call it getCurrentPrice (it is GoldPriceDao, can it fetch anything other than gold? No right.. so gold is unnecessary in method name)


remove private el:ElementRef


lets call it -> getPriceHistory()


Should be `grunt` not `git` I presume.


The order can also be reversed:

complex double x;


pypy takes a long time to build, also try reverting https://github.com/NixOS/nixpkgs/commit/c234f37b5995457e2a5b518648e6735612cb3c27


I believe this fixes issue #308
https://github.com/Microsoft/nodejstools/issues/308


I think you can delete this commented code.


Couldn't this be done in the newly introduced constructor of `DTLSSession`where we copy the session state from the passed in session (the already established session)?


Matan had similar code for the cma configfs that didn't require copying the string.


Yup almost likely: http://stackoverflow.com/questions/27323264/create-pypy-process


what is this class for ?


Good catch!


ok, no worries I will revert it back, as I had no chance to test it myself.


Good point, I'll first try with both reverted.


Yes it's almost likely the other commit: http://stackoverflow.com/questions/27323264/create-pypy-process


lets call this QuandlGoldDao


Please improve formatting here and add the missing space before "(".

I also think that a better name for the option would be `--sdk-ckeditor-version`.

And the last thing is that we don't have to limit this to `master` and `major`. A tag could also work I guess. Won't it?


similar comments in other implimention also applicable here..


This is covered by net_to_rxe()


Please, add `@since: 2.2.1`


Okay, I am working on a systemd unit file to generate the certificates. I think it'll be ready soon.


bad idea


No problem. I'll change it.


Isn't this done by the notifier block already?


don't use an injector but a multibinder


Found reason of this problem (check the `StackOverflow` link).


Also added getEKin() and getEPot() for later graphing


please restrict new usage


why does the gold implementation require appContext?


修复解析响应数据时的BUG


Lycka till min kompis :+1: :+1:  :+1: :door: :door: :hurtrealbad:


Oh, the other one isn't a stdenv rebuild, so it should be much faster to test.


you probably to introduce a way to register mailets with guice modules


I feel like this class is doing lot of stuff..
see other good projects and see how the activities there are.

this is 'Home' - what is it? Activity/Dao? -- lets call it HomeActivity (unless it is a bad practive to include Activity in name


What's new with this Beta 8 ?

I'm using it's Profile 5 when my battery is <= 25%
(it gives me AMAZING extra battery)


Add new line and this note:
```
Creates num_devices zram devices.

No additional options are currently defined.
```


the commit title doesn't match the content


I think this looks quite correct. However, we definitly need some test cases verifying the correct behavior. Currently, the `DTLSConnectorTest.testConnectorResumesExistingSession()`only checks, whether the server responds with a Handshake message to a CLIENT_HELLO with an existing SessionID...


This value should be read from CKEditor's package.json or ckeditor.js or some other place when building CKEditor. Not when updating it. In other words - this value should not exist here.


reverted back the changes


50px is the absolute size - should we use percentage instead?
otherwise.. the screen will not be uniform across different device resolutions.


Add a parameters list.


Genius commit :+1:


this I think is ok.. (doesn't matter much)


BTW This is placed in wrong section of Kconfig - in CPU configuration, which it is really NOT.
It needs to be after LINUX_LINK_BASE !


similar


Address all the comments here.


You can't really know that, because another remove may have removed it after it was successfully added. If you want to be sure, add a return value to rxe_net_add().


Great!

Not a bug but.. there is no need to escape . and @ in [] inside regexp. From [py3docs](https://docs.python.org/3/library/re.html#regular-expression-syntax):
> Special characters lose their special meaning inside sets


I'm referring to just the line `resources :advertisements`.

Run `rake routes` in the terminal to see a list of generated routes.  Do you have controller actions implemented for all of the generated "advertisements" routes?


Forgot a ';' - will be fixed soon.


Add the previous note for additional options.


Please, add `@since: 2.2.1`.


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1281 of 1283 tests passed)

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


Check out without fetching changes is not enough.


@OsuKrafter A merge with upstream.


Capital 'E', formatting basically.


MailRepositories are for storing mails that goes out of the mailqueue.
User specifies when a mail is stored via the mailet/matcher/processor configuration (mailetcontainer.xml)
The point is that he can choose between different storage backends. A StorageBackend iys a mail repository.

For instance Spam can be stored on local disk, but undelivered user e-mails can be stored in Cassandra. This is what this configuration trick allows.

So we need to dynamically load MailRepository, following the content of the file mailrepositorystore.xml. This is what this class do.


Add the previous note about the additional options.


Make sure `.DS_store` is your `.gitignore`, and congrats on getting your Mac up and running.  (These are Mac OS generated files)


The URL inside `simplesample.js` should include some version placeholder. We should look here for `\d\.\d\.\d` but rather for `<version>`.


Add a "doc" string for this method.


Good point. Tweaked!

On Wed, 26 Aug 2015 at 12:46 Michael Nacharov <notifications@github.com>
wrote:

> Great!
>
> Not a bug but.. there is no need to escape . and @ in [] inside regexp.
> From py3docs
> <https://docs.python.org/3/library/re.html#regular-expression-syntax>:
>
> Special characters lose their special meaning inside sets
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/etianen/django-python3-ldap/commit/bed01c7317e68e1efa20f036bc8e84af0777dd7c#commitcomment-12901052>
> .
>


Capital 'D' and note about the additional options.


@lewiscowper em in that case doesn't work well as a unit. We want the cross stay at the same measurements across breakpoints. But overall we're changing font-sizes between breakpoints.


a simple @Provides method should be enough


@adriancole FYI


Add parameters list.


Anyone using this should probably also replace the root variable with this, at least that's what I had to do, since `$(window).scrollTop()` will return `0` if they are at the top of the page anyway, but it just means it'll get initialized correctly if they are half down a page.

    root = $(window).scrollTop(),


I've changed the positioning of the cross to absolute and vertically aligned it with a little hack (top: 0; bottom: 0; margin-top: auto; margin-bottom: auto;). This also works horizontally.


How is this project coming along? I am interested in finding what milestones you've achieved here. Would love to help.


Hence the new positioning, margin and floating is obsolete.


`unless user.admin? || user.in_group? group.to_sym`


I think the description should avoid mentioning rxe_cfg, and just point to the wiki page that explains how to set the whole thing up.


::before and ::after are elements totally valid to be nested.


Edited index.html


`^/present/order$` ;)


Like the one before (measuring in em), setting these dimensions in percentages had small visual problems (we're talking about 1-2px difference) made the cross look uneven.


@dlazar I'm just wondering, did you add `sort_by_value` [to get around this bug][1]? If you passed an empty string then the `most_occurring_words` would fail. This has since been fixed.

[1]: https://github.com/abitdodgy/words_counted/pull/15


This paragraph has some "here" words that I would assume to be links.


Yep you are right. ```DELETE FROM `spell_script_names` WHERE `spell_id`=49575;``` should be enough.


@hoholee12
facepalm* xddd but is ugly to see ___________________________done. for me :/

I knew the existence of dat random colourful zeros in the script xd, was amazing when tried it, I was like O_________O okay bitches... xD dat method would be used to sensual title at first run or other sensual stuff D:


The sold out banner doesn't need a height declared. The content + padding itself dictate the dimension (+ you've used a line-height variable which is unitless = no effect on height).


Written in the middle of a Camel related commit ? Without any tests ? Really ?


Correct, removed.


Ok
> On 26 Aug 2015, at 00:28, Ole-André Rodlie <notifications@github.com> wrote:
>
> will also auto generate the manifests in build-installer.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/MrKepzie/natron-mingw/commit/295e470ed5f4e5ba20df5890d27511906e5fc42a#commitcomment-12891517>.
>


Using a global variable :+1: With the new approach (positioning the cross absolute) we need space for it on the right side = higher padding value.


Setting position: relative; to position the cross in context of the banner.


I think it'd be clearer with a symbolic name for the algorithms, e.g. `NAME_SET_INTERSECTION` for the new one. (Or since there are only two at the moment, this could just be a boolean option. It's just awkward to read if you have to remember what `2` means.


Well, on the mailets, I tried to keep the current James behaviour : the user just need to add the JAR to the classpath, in the conf/lib directory for instance. He can then simply access his matchers / mailets without a single line of configuration.


Works well great fix :)


Hello,

I have asked on the google's building group to explain a bit the difference betweens the tags and branches.

I got an interesting explanation:

https://groups.google.com/forum/#!topic/android-building/ETO-zhHxS3s

QUOTE:
"You currently have 3 main parallel branches for LP:

    lollipop-mr1-release -  tags:  android-5.1.1_r4 android-5.1.1_r6 android-5.1.1_r9 android-5.1.1_r10
    lollipop-mr1-fi-release -  tags: android-5.1.1_r4 android-5.1.1_r8 android-5.1.1_r13
    lollipop-mr1-wfc-release -  tags: lollipop-mr1-devandroid-5.1.1_r5android-5.1.1_r12

The wfc branch was forked a long time ago, and it's older, hence why it'll report many changes. when you compared from r12 to r13
It might also have gotten a bunch of cp, with commit ids changed it seems

But basically if you're currently at _r6 (mr1-release )  you should switch to the _r10 not _r13

It's very confusing, not sure why the tags are not named accordingly with their respective branches like add a suffix -wfc -fi
it would make it easier to identify.

END OF QUOTE.

Therefore you did the correct thing by reverting to r9.

Dunno if tracking r10 would be easy, however.

What do you think of this guys?

I am gonna try building r10.

The difference however between r9 and r10 is very light: most of  the time one or two commits.


Obviously this isn't a problem you introduced, but shouldn't there be a `cache[cache_key] = speakers` here (or somewhere?)


ah alright ill fix that later, had som troubles with
<code> unless user.in_group? group.to_sym || user.admin? </code>


The comment and class expectations are still pending.  I've got more details about the JSON that I created below.


I don't think you want to unskip all the known failing tests now. Please get all those changes reverted.


Just saw this function now, propably does exactly the same but maybe nicer:
replace this line with
IPython.version_info[0]


I don't think we can get this behaviour using the multibinder


Dynamic loading of JAR is not relevant at this point, we don't have to rewrite everything spring do right now.


LGTM.


This broke a few dozen go packages, see http://hydra.nixos.org/eval/1217479?full=1#tabs-still-fail


不错


This actually wasn't a 'real' result (I copied that comment from the previous test by accident).  I was thinking of a 'what-if' scenario with an array.  But, I don't think an included doc would start as an array - it should be a JSON object.


LGTM


ИМХО, сокращение на одну букву `index` -> `indx` большого смысла не имеет, лучше `index` или `i`.


Looking forward to a nice translations update in the next release ;).


So if I wan't to add a mailet, I need as a user to modify the GUICE module list to add mines in the multi - binders ? The mailet and matcher loader would be given the full mailet / matcher list registered in the given multibinder, and on load call would iterate this list until it finds the corresponding one, is that what you mean ?


Thanks, I stumbled over it by chance. Maybe next time you update `rebase.exe`, you want to mention that URL in the commit message? Just a suggestion...


谢谢，用的第一个翻墙工具，用得最久的翻墙工具。


Yes, it's the QHD Find7s. Thank you.


Done


Why not just do it the way it done in tc and tc-browser-natives? (https://github.com/inikulin/testcafe-phoenix/blob/master/src/messages.js)

\r-


I wasn't aware of the @Provides, it solves most of my problems.


change it to the english name: litigiated


@mcharleb I think this needs quotes around the $1.


Shouldn't this be `/srv/meetings` as in https://github.com/daniel-beck/robobutler/commit/687e4a700b67eec13d3485c341de9ed81aae3f4f#diff-b67911656ef5d18c4ae36cb6741b7965R7?


I think it would be good to have more description in this comment of how the name set intersection algorithm works, and how the old Kenyan one works. (As mentioned in the other commit, I'd prefer a more meaningful value than just an integer, whether it's a string, a defined constant or a boolean switch.)


@kohsuke Shouldn't this be `/srv/meetings` as in https://github.com/jenkins-infra/robobutler/commit/687e4a700b67eec13d3485c341de9ed81aae3f4f#diff-b67911656ef5d18c4ae36cb6741b7965R7?


This first project was thinking like this:

Library-> Sections-> Shelves-> Resources (could be digital or 'pages')-> hardResources-> Books/Magazines/

(Yes, I'm going to create the java doc tonight!!)
(And yes, yesterday I remembered that one of the requirements was to return the search alphabetically)

RESOURCES:
So, the first thing was to create resources, example:
hardResources extends Resources
books extends hardResources

every resources has: name, author, category
*Digital and hard resources will have the same labels, and also books, magazines and newspaper; that's why I did it this way

SHELVES
After that, I created the shelves. At the beginning, one shelve = one category; but then I remember that in my university and some libraries have shelves with: 3/4 mathematics and 1/4 physics (and another shelve with more physic’s books). They usually don't leave blank spaces in shelves, combining a little bit with the next shelves.

SECTION
Same with shelves, section contains shelves (none, one, two....) and each shelve contain 'n' number of resources.

The main allows the user to create sections, shelves and resources.
create section
create shelves
add resources (to one shelve that belongs to a section)
remove (except for news paper)
search by name (don't return results alphabetically, this is the next implementation)
search by name and author (don't return results alphabetically, this is the next implementation)
help (show the commands of the app)
quit (quit)

To implement:
not checking if the section already exists
not checking if the shelve already exists in the section
not checking if the resource already exists (yes, an improvement)
add the name and phone contact when some resource doesn't exist


You have to define a combination of module that match your needs, probably by using the great guice module we are writing + a module that adds the mailets you need.
For the loaders, you are rights.


@domenkozar If wrapped Python programs need the full path in `argv[0]`, shouldn't they have their own wrapper, since this differs from most other programs?


Oke bro...sekarang uda muncul shopping cart nya. Cuma saat process check out, selalu muncul
message :"Please accept the Terms of Service to proceed"

bahkan setelah di check term of service nya...rupanya 'ngeloop' disitu.

Bisa kasih masukan ? Thanks


she is a girl。
2015年8月26日 18:06，"Milan" <notifications@github.com>写道：

> thanks man, good luck!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shadowsocks/shadowsocks/commit/938bba32a4008bdde9c064dda6a0597987ddef54#commitcomment-12899385>
> .
>


If it's causing trouble just revert it.


Lines must be 79 characters or less. Most of our tests don't have docstrings, in part because docstrings can make nose output harder to follow.


Please use PEP8 indentation


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


keep this on one line


Please, use either `<blockdev/swap.h>` or `<sys/swap.h>; so that it's more readable.


You can drop this `priv` pointer.


> Just saw this function now, propably does exactly the same but maybe nicer:
> replace this line with
> IPython.version_info[0]

OK, just fixed that ;) thanks.


That's what I get for testing on windows and OS X...

I had wanted to make this part of the build process, but I can't guarantee that Python is available...  Perhaps I'll do it conditionally...

----
Brian Kelley

> On Aug 26, 2015, at 5:19 AM, Axel Pahl <notifications@github.com> wrote:
>
> Brian, thanks for the changes.
> The path to the wehi_pains.csv is: ../../../Data/Pains/wehi_pains.csv, not ../../../Data/PAINS/wehi_pains.csv (line 18)
> -Axel
>
> —
> Reply to this email directly or view it on GitHub.
>


@ttuegel I'm OK if we make the fix for pypy somewhere else, as long as it works.


this is a comment


OK you just ignore the tiny pull request :)


Thank you! <3


paper下面default会爆炸，一片白看不到字


Some quick time-to-depth measurements on 20 cores suggests that 3 +
depth/4 is indeed better than either 3 + depth/3 or 3 + depth/5. But I
haven't done any rigorous testing.

Used ./stockfish bench 8192 20 34 default depth, so fairly high depths
were reached.

On Wed, Aug 26, 2015 at 6:52 AM, ajithcj <notifications@github.com>
wrote:
> Also, note that the formula 3 + depth/5 has already been tested at 9
> threads STC and showed some elo gain after 10K games.
> http://tests.stockfishchess.org/tests/view/55d820120ebc595da9433856
>
> —
> Reply to this email directly or view it on GitHub.
>


monetary widget


This doesn't actually work correctly. Since if there is more than one function nested it pops the arguments of the parent function if it can't find the necessary argument in the child.


Just wanted to understand the meaning of this rule below
 ||c.speedtest.net/javascript/speedtest-main.js$important,inline-script,script,image,domain=c.speedtest.net
If i understand correctly, the first part  ||c.speedtest.net/javascript/speedtest-main.js$important  stops the specified js resource.

And the second part of the rule inline-script,script,image,domain=c.speedtest.net stops all the resources of type=incline-script,image,script from domain=c.speedtest.net ?

I know gorhill will be busy. Maybe some once answer. Thanks!


@AndreasAntener Sorry, I missed this in the review. This change is wrong. @mcharleb This fires up the estimator at 1 KHz.


I think we should use a single CDN for all our files for performance and because this breaks `--offline`.


It would be nice to document why we ignore Rawhide (even though all members of ABRT team know that).


(Note from a Lasagne developer who just had a brief look around:) Careful! `nn.dropout` already scales by 1/(1-p), you're now scaling it twice. That won't work.


For Birna:
![image](https://cloud.githubusercontent.com/assets/3079749/9493006/0ee76080-4bec-11e5-9998-22c48b74bd8c.png)


I would propose to always have a floating header with at least the line "Run set" and the column names, and provide a switch for showing/hiding the remaining header rows.


Did you forget to update the "else branch" too?


What about embedded pictures in audio files ?

2015-08-26 7:57 GMT+02:00 Volumio <notifications@github.com>:

> We could use as a primary service the album art (if any) in the album
> folder, if this is not present we then make a call to last.fm apis...
> The album folder albumart will need to get exposed trough a webserver,
> don't know if that's feasible but we could use what we already have with
> express...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/volumio/Volumio2/commit/c2bc3f218a4403df22b1fc1899bc8998444d83f1#commitcomment-12895551>
> .
>

--
--
Jérémie Tarot
http://about.me/silopolis


The placement of the header is off in my browser (Firefox), for example there is a blank line between header and table after toggling the switch twice.


`((pos + blocklen) > chunklen))` seems to be responsible of making `__getitem__` in the carray class to run into fallback situation https://github.com/Blosc/bcolz/blob/master/bcolz/carray_ext.pyx#L1901-L1902, when retrieving a single item it returns a ndarray instead of a single int.

Do you think removing this check would it it be a sensible solution? Are there situations in which one would like to check this?


1. ListView has been replaced with ListBox for better performance and styling.

2. A new property IsPanSelectorEnabled (well proabably badly named :p) has been added to allow users to select a menu item by panning up/down on the bottom area of the left pane. This is an experimental UX design that I came up with a few months ago and hopefully it could be helpful for users with big phones since they don't need to stretch their fingers to reach the top part of the screen.


See https://github.com/phpmyadmin/phpmyadmin/issues/11431


Please use WaitForSingleObject() for each thread instead of having a while loop.


Thanks!


I meant "not that you'll use all these arguments at once as positional arguments..."


The `md-virtual-repeat-container` is not being removed after an item has been selected.


exact, i will search


I think that this logic checks that the indicator actually has data behind it.  Yes, it will display the indicator in the catalog, but the visualizations/download will give a 500 because the data will not be found.  Maybe we should see why indicators without mapping are not loaded and either remove them from the system or load the data.


Of course we should, but jsdelivr does not have this file.
I agree that this breaks offline, but this is fixable (e.g. by having lib_url / alt_lib_url)


Do we really need to check ``http_resp_code``?


This is in the async callback, at this point both enter events have been received successfully. And yet presence set still occasionally returns only one without the timeout - eg https://ci.ably.io/job/ably-js/188/consoleText - so presumably a bug in the code rather than the test. Mostly put this in to get it out of the way so I could focus on the IE11 failures, but yeah, it's its own bug (just annoyingly hard to reproduce). I'll remove this commit and file a new issue.


I really don't follow, could you elaborate?


I see. Do you disable console log in production?


OK I'll take a closer look @ Firefox.


When I viewed this page with `jekyll serve` locally, the code snippet below wasn't shown correctly (everything on a single line without newlines). The fix was to add a newline to the source markup between lines 34 and 35 (similar fix was required for two other code snippets below).


> And the second part of the rule `inline-script,script,image,domain=c.speedtest.net` stops all the resources of type=`incline-script,image,script` from `domain=c.speedtest.net` ?

Many parts in the filter. All the conditions must be fulfilled:

- URL matches `||c.speedtest.net/javascript/speedtest-main.js`
- `domain=c.speedtest.com`: context must be `c.speedtest.com`
    - It's because the offending resource (ad) is within an `iframe` which comes from `c.speedtest.com`.
- `important`: disregard exception filters (present in EasyList).
- `inline-script,script,image`: block inline script tags, scripts, images.
    - If I remember correctly, filter worked fine with just blocking inline script tags, but went a bit further just to cut down on preventing useless resources from being fetched anyways.


.看来要指定默认字体颜色


Made a mistake here, should have done this on the develop branch. Master branch should only be updated after a group code review of develop.


LGTM


I guess it doesn't right now. We could add a parameter node that would be used by MultiVolume logic to get multivolumes used to make the plots. In this case, selectors would update the parameter node both in the Explorer and PCampReview, and they would better reuse the same logic.


@jonpas you bad banana
![Don't comment on commits](https://farm3.staticflickr.com/2876/9681688504_b5e8689026_z.jpg)


Shouldn't the method here be PUT as well?


Good ones :+1:


Thank for comment


I don't think anyone would want to switch off the floating header, what people would actually want is to save screen space while still seeing the most relevant column headings. Thus I propose a different switch instead that toggles between the full header as it is now (default) and a reduced header with only two row (the "Run set" row and the last header row). This feature is actually independently useful from the floating, e.g. when printing tables.


We don't use `getText` in tests, but compare with the actual text, see other tests.


You'd better include `<blockdev/kbd.h>`; same as above.


Space missing.


seconded


OK, do it your way.


@chikamichi It's here I think

Line 270 of the `master` branch.
```
var getSongElementByNumber = function(num) {
	return $("[data-song-number='" + num + "']");
};
```


Also in QA_4_5 please.


Oh sorry, I didn't mean to notify you --- I was referring to my colleague Michael who goes around as mg.
regards,

Karim


I mean 7 threads at LTC (60 seconds), fixed 20K games

On Wed, Aug 26, 2015 at 9:47 AM, lucasart <notifications@github.com> wrote:

> so maybe just open a pull request. there is no 15 thread machine
> available, so we'll have to wait and verify on 15 threads later.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/19eeaf272393613bf10682a444771fe0da9d9f20#commitcomment-12896776>
> .
>


You might prefer to use 'application/json-rpc' here. It would be more accurate.


i have just hit this exact bug myself! the problem  with this fix is that it doesnt work for python3, because they got rid of basestring. (wtf!)

so if we want skrf to support python2, which i think is worth it, we have to use some sort of 2/3 library like `six` or `future`. i think i am going to go with future.

also, this bug made me grep for all instances of calls like this, and i found a handful, which stopped me from fixing this one alone. ill try to get to it asap

alternatively you can just switch your environment to python3.3, i am doing this currently


This is reasonable and should be easy to implement.


Then I would need to copy the `throw new JobExecutionException(jobGraph.getJobID(), ..." into both blocks --> duplcate code.


Mhh, irgendwie verschwinden die crop marks hinter dem Bild....


fountainhead_name = node.name


This should include a filter so more countries can be added without a WC point release


StandardCharsets is supported in Android API level 19 and above.
According to Google's recent [version chart](https://developer.android.com/about/dashboards/index.html#version-chart) (August 3, 2015), this leaves out:
1) 37.7% of users who use 4.X versions below 4.4
2) 4.9% of users who use Android 2.X

Please consider reverting this commit.


Looks like this breaks `st2cd.st2_deploy_test` task:

```<?xml version="1.0" encoding="UTF-8"?>
<Response><Errors><Error><Code>InvalidParameterCombination</Code><Message>Virtualization type 'hvm' is required for instances of type 'm4.large'. Ensure that you are using an AMI with virtualization type 'hvm'. For more information, see http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/virtualization_types.html</Message></Error></Errors><RequestID>31a9c7d3-805e-441f-9107-17a63473a8a8</RequestID></Response>
```

For some reason it only started happened today. It sounds like we didn't pull down an updated pack or?

In any case, I will look into adjusting the action.


Read up about function parameters in ruby!

`unless user.in_group? group.to_sym || user.admin?` will execute as
`unless user.in_group?(group.to_sym || user.admin?)`


In my opinion it would improve readability. And you don't have to cast it yourself.


With this form we have to specify context for each string.
Translation is made at import time (strings are effectively tranlated in QGIS because language is set before processing is loaded).


Ok, I found it, inside a254231.

I used the *blame* feature from git, available as `git blame [path-of-a-file]`, or directly on GitHub through the *Blame* button. For instance, [for the album.js file on the master branch](https://github.com/kusera/bloc-jams/blame/master/scripts/album.js).


Maybe use of 'str.format' would be better choice.


Стиль: в некоторых местах есть пробел перед открывающей `<`, а в других – нет.


Yes, that should work! Missed that.. :smile:  :+1:


Just start playing your dungeon 'lute' and the horn section will join in.
It's dungeon loot.


The name of this commit though, references `getSongByElementNumber` but don't actually contains anything related to that.


Are `/order/:id`, `/order`, `/orders`, `/orders/:id` not relevant anymore or not part of the api?


Adding this logic means we need to modify the code to stop the RX schedule, which is not there. Can this be done as a separate commit?


Build seems to fail for mips platforms, e.g. ar71xx, lantiq etc.
The reason looks like a mips16 incompatibility. You might need to disable MIPS16 for the package.

E.g.
http://buildbot.openwrt.org:8010/broken_packages/lantiq/reaver/compile.txt
http://buildbot.openwrt.org:8010/broken_packages/ar71xx/reaver/compile.txt

```
mips-openwrt-linux-musl-gcc -DCONF_DIR='"/etc/reaver"' -DREAVER_DATABASE='"/etc/reaver/reaver.db"' -Wall -Os -pipe -mno-branch-likely -mips32r2 -mtune=34kc -mdsp -fno-caller-saves -fhonour-copts -Wno-error=unused-but-set-variable -msoft-float -mips16 -minterlink-mips16 -Wformat -Werror=format-security -fstack-protector -D_FORTIFY_SOURCE=1 -Wl,-z,now -Wl,-z,relro  builder.c -c
{standard input}: Assembler messages:
{standard input}:79: Error: unrecognized opcode `wsbh $2,$2'
{standard input}:90: Error: unrecognized opcode `wsbh $3,$17'
{standard input}:208: Error: unrecognized opcode `wsbh $2,$2'
make[3]: *** [builder.o] Error 1
```


If you like, you can have a look at or link to https://github.com/eddyson-de/tapestry-webjars.


I don't think either alt text or a tooltip is useful here. In particular, the ALT text for a logo should be empty (it's decorative, and there's no reason to repeat the title above).


I agree. Can this be done separately?


Should this be `List`? I have not looked at the other Moltin server client libraries but it should match those if possible.


Actually, we need to use an image which uses `hvm` virtualization type.

For now, I will revert this change.


@domenkozar Well, I'm not sure how to fix it because I'm not sure what the problem actually is. It's just using the path to find the standard library, right? In this case, the difference is just in the last component (the filename), but the Nix store path is the same.


In my case the ctkDicomBrowser crashes, if either one of the fields (patient name, study or series description) is empty (see stacktrace below):

```
 Crashed Thread:        0  Dispatch queue: com.apple.main-thread

Exception Type:        EXC_BAD_ACCESS (SIGSEGV)
Exception Codes:       EXC_I386_GPFLT

Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   QtGui                         	0x000000010c56cbb4 QLabel::text() const + 4
1   libCTKDICOMWidgets.0.1.dylib  	0x0000000109e747d7 ctkDICOMBrowser::addSelectionLabelsToContextMenu(QStringList, QMenu*) + 1463 (ctkDICOMBrowser.cpp:710)
2   libCTKDICOMWidgets.0.1.dylib  	0x0000000109e75662 ctkDICOMBrowser::onSeriesRightClicked(QPoint const&) + 338 (ctkDICOMBrowser.cpp:814)


I'd prefer to stick with more than Boolean. I guess the first algorithm is SUBSTRING_MATCH. Effectively the'd probably me 1 and 2 anyway, unless we use a symbol library.


can we change this to is_null() ?


You might use %I for the sequence name here and below, to try and handle cases that require quoting, etc.


Maybe, we can add a method for `StoragedDaemon *` retrieval. It's up to you.


That is an option as well. It only depends on how efficient is retrieving it...


New line?


FWIW: my reason for putting it in here is that, at this point, `ConnectionManager` doesn't have a reference to the transport (it gets that in the callback in line 27, after the websocket has been opened). So putting it in `ConnectionManager` means that that has to keep track of transports before they becomes a `pendingTransport` (`prePendingTransport`? or maybe `proposedTransport`) (and also having to do the same for comet transports, which don't really have this problem), which seemed needlessly complicated. But happy to do it that way if you think it's better.


Ok


Fixed this case, thanks. But what's the point of allowing such private function being open? Can I override it somewhere?


we can, but `null ===` is a smidgen faster :)


`users.coffee` and `users.scss` can be safely deleted.

If you get bored of manually deleting these blank generated files, you can add some configuration to `config/application.rb` that'll turn off the generators you don't want. This will stop these blank `.scss` and `.coffee` files from being created.

Should you want to do that, you can add the following `config.generators do ...` block towards the end of this section of `config/application.rb`:

```ruby
module YourAppNameIsAlreadyHere
  class Application < Rails::Application
    ...
    # These are the lines you add:
    config.generators do |g|
      g.assets = false # Prevents generation of blank asset files
    end

```


As it does some (not-so)obvious calculations, please do add some documentation. It is a pity that the lack of internal documentation is questionable. But having *some* free-form description of functions greatly reduces the mental burden of understanding things later on.


I'd add an extra check to ensure len(parts) == 2


With this form, we use the self.tr method at runtime on each defineCharacteristics call, like it is the case for name, group and parameters names.


Hey @martinmoene

I'm planning on pushing fixes to another chapter and opening a PR by the end of the week :smiley:

I'm really enjoying the book btw! Thanks


Additional space right before the last parentheses.


doesn't satisfy my ocd, ill live with it.


Memory leak for every call. Please fix.


Do we need this empty NULL-pointer?


@vpartington Should this be `WsmanSPNegoSchemeFactory.create invoked...`?


Consider using `resource` instead of `@user`. The `resource` argument is the logged-in user. Also consider renaming `resource` to `user` so its clearer what that argument is, `resource` is a bit general.


Email duplicated on this view.


no need for .keys() when iterating over dict.


Can we delete edit?


This controller test can be deleted. Favor using RSpec for now as that's what we've used on the previous app.


어쩐지 매우 훌륭한 코드네요. ㅋㅋㅋㅋㅋㅋㅋ


Frankly, I don't remember from the top of my head.  Does removing this make some tests to fail?


no need for keys here too


왜 git push origin master 가 아니죠?
기본적으로 origin으로 설정되어 있을텐데 kim으로 바꾸셨나요?
git push 시 origin과 master가 무엇인지 검색해보시기 바랍니다.


what sense of adding empty separator?


hm I thought this was due to some kind of limitation in `pbgzip`. Thanks for fixing it :)


Kommandot är `make install`, jag antar att du inte har laddat ner drupal-vm och därmed inte har en `vm` folder ännu.

https://github.com/generoi/genero-conf/blob/master/make/vm.mk#L66


Typo :)
```scss
"light-blue": $clr-blue-list,
```


주소를 저렇게 설정해두면 이 경로 밖에 못쓰니깐 같은 디렉토리에 있다면 단지 '*_DATA' 만으로도 충분합니다.


I think it's valid to have a test that crashes if it fails, for this specific case.
However, idiomatic use would be to say:
  tt_assert(!s)


Also, för att inte ha prefixed versioner av gnu tools se http://apple.stackexchange.com/questions/69223/how-to-replace-mac-os-x-utilities-with-gnu-core-utilities


Hi,

    Just a question, why are you using HttpSession instead Dpsace Session service? Is it deprecated? https://wiki.duraspace.org/display/DSPACE/DSpace+Spring+Services+Tutorial#DSpaceSpringServicesTutorial-TheDSpaceSessionService

Thanks
Luiz


我不曉得怎麼從data物件取得屬性(ex. User/Shipment...etc)傳給purchaseResult，目前串接的部份大概就差在轉換資料格式的環節


Just a random thought: has this work in general been tested against views?


Couple more nitpicks:

- `stride` is unnecessary and should be replaced by `size`
- `if` is unnecessary if `ssize_t` will be used in `for` instead of `size_t`


I guess the two (to string, and parse) JSON utilties were moved here to be used by the ajax code below?


HI, thanks a lot for making this makefile available. I am facing an issue after I transfer the binary (helloworld) to my Galileo Gen 2 board. When I try to run the executable, the bash doesn't recognize the file.
root@galileo:~/c_codes# ls -al
total 28
drwxr-xr-x 2 root root 4096 Aug 26 18:03 .
drwx------ 3 root root 4096 Jan  1  2001 ..
-rwxr-xr-x 1 root root 4854 Jan  1  2001 a.out
-rwxr-xr-x 1 root root 7060 Aug 26 18:03 helloworld
-rw-r--r-- 1 root root   64 Jan  1  2001 hi.c
root@galileo:~/c_codes# ./helloworld
-bash: ./helloworld: No such file or directory
root@galileo:~/c_codes#

Why is this happening? Any solution?


Hello,

thanks for you input, but currently we do not accept pull requests.

BR,
Wiktor Dolecki


An extra newline is still here.


This seems to be a commit that we want to have in master. Could you submit a pull request for this one? It is the only thing that is not already in master somehow:
- Toplevel definitions of `local` variables are gone
- Customization of Multiline-Prompts can be done by specifying `POWERLEVEL9K_MULTILINE_FIRST_PROMPT_PREFIX` and `POWERLEVEL9K_MULTILINE_SECOND_PROMPT_PREFIX` in your `~/.zshrc`

Thx and best regards


I'm not sure that I like the idea of pushing all of this data into Splunk; it's a lot of data. Previously, we've written to file, then parsed the file to get the data that we need (the log file parser is in an internal repository). Is there a reason that pattern wouldn't work for you?


@movibe We're using Twitter username here, please change to @movibeoficial :roller_coaster:


Hi,

    Just a question, why are you using HttpSession instead Dpsace Session service? Is it deprecated?

Thanks
Luiz


Have any idea? @sirikfoll


預留[訂單建立時，若後端有錯誤要讓訊息出現](https://trello.com/c/ey0kaXIE/57--)的處理


nahh ı guess ı found it...


为什么要把空格换成 Tab 呢？


Looks good to me!


The previous execution of `forceGC()` every 5th file sounds good but only for large files. I think if they are small, cleaning GC every 50 files is more than enough. If sum of processed file lengths will exceed 1 MB (1e6) then `sizeCheck` condition will trigger GC cleanup sooner.

The previous  `buff.length > 1e6` condition was dead since the `buff` variable was not coming from inside-loop context, hence the `sizeCheck` replacement.

This commit reduces the duration of `jx package` of project #499 (containing 10k files/ 150 MB) from 100 secs down to 20 secs (with same memory consumption).


@airglow , this repo is protected by a password and makes the doc jobs fail.


Idek


Hi Spyros,

Which other chapter do you plan to create a PR for? I ask to prevent creating merge conflicting PRs as a result of making changes to the same chapter at the same time.

cheers,
Martin


Tested: very good :)


I'm sorry Kevin, I'm new in the Dspace development, but doesn't it make sense return ArrayList<EPerson>?

regards
Luiz


Hi,
Did you take a look at #3? The error you got may be related to this issue.
If it does not solve your problem, you can open a new issue so that everyone using this code can get the workaround, information, etc :smile_cat:


this should be the reason, I should only add the new data, instead of add all the data, the clear seems to got problem.


I'm not sure on why you would need programs_to_packages? Why not just check for hmmer? or ktImportText in package_urls?


tabs! evil, soul-eating tabs!

;)


cool but I def prefer that this be in the settings.json file so it's more explicit rather then auto-detected...  I wonder if it can be part of the gulp dist process to change this out.


fileNames is local var for start function (see js output), but it seems it should be local for exported parse function, right?


Hi , I found that my IC chip's leg got blend, so every thing is fine now
thanks


That's definitely WAY better on almost every axis.

100% agree that this is more idiomatic for creating a custom-configured processor.  Also, so THAT'S where FeatureKeys live. I somehow left "lib" off EVERY SINGLE TIME until I gave up on using them.

Would you be opposed to merging the two approaches?  Having processor.config[] and config[]= be forwarding functions to the processor's get/set methods?  It reflects the Saxon API, rather than Noko/Rails, but I still think there's convenience value in being able to reconfigure the default proc (or just, say, toggle line numbering for same)


jsとcssの配信初めてみて感動している


Do you still need this git reset since another one is done 3 commands later?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.927.c1faadfd3e8a072f5ca1b5ff5af4f7b93244e8cb](http://build.palaso.org/viewLog.html?buildId=61903&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 464, ignored: 6 Build time: 00:03:34


@ccx-testerbot runtests master


"I'll fix formatting if there are no other problems." :)


Yeh this was poor programming soz


return => returns or returned


Interesting! Which font do you use?


`items.coffee` and `items.scss` can be removed.


the same fileNames is local for parseFile function, but it seems it supposed to be reused in exported parse function. BTW you could move parseFile and paseFileFallback functions inside parse function.


No, it's not that


The signs look correct to me compared to wikipedia.


Good Job


ИМХО, `scheme` -> `schem` тоже не имеет смысла сокращать.


@albertosouza boaaaaaaaaaaa


Typo : `http_resp_code`


This route on line 3 was added by the generator - it can be removed. The routes for items are defined later in `routes.rb`.


@DeicPro
well there is one good use for this:

___________________press any key to continue...

looks neat now aint it? :3


thanks :+1: :)


ИМХО: `for (const auto& i : db)`


I think there's something not quite right about the bounding box conversions here - they don't seem to round trip properly. If you stick this at the end of FormatTest.testCoordinateSystemTransforms() it fails :

```
for i in range( 0, 1000 ) :

	b = IECore.Box2i()
	b.extendBy( IECore.V2i( int( random.uniform( -500, 500 ) ), int( random.uniform( -500, 500 ) ) ) )
	b.extendBy( IECore.V2i( int( random.uniform( -500, 500 ) ), int( random.uniform( -500, 500 ) ) ) )

	bDown = f.formatToYDownSpace( b )
	self.assertEqual( f.yDownToFormatSpace( bDown ), b )
```

Do we have a bit of a conceptual problem here as well? The function sounds like it only deals in changes in Y, but if it's adjusting for inclusive-vs-exclusive, perhaps it should also be doing that for X? Maybe the methods should be called `formatToEXRSpace()`?


Sure, but what does unless do? I'm gonna stick with what I know and do:  ``if not self.args.no_clustering``


Eindel program


Лучше `&` чтобы не копировать и `const`, так как не нужно менять: `const auto& el : ...`.


インラインにcss書くと後々めんどくさそうだから今のうちに削っておくと良いと思う


What do you think about lines 13 and 14 given the flash handling code on lines 16+?


加个颜色？


### Missing semicolon.

```
return "Hello, World!"
                      ^
```


### Use '===' to compare with 'null'.

```
if (bla == null) return;
        ^
```


ИМХО, `for (const auto& i : db_util)`.


Also long double complex is missing.


Yeah, Transifex uses the source file as a template for the rest, and simply swaps out strings, so these can't be used for any other changes in the files anymore.

I'm always good about making sure translators are given credit in the commit messages for what it's worth though, and I'd even add them to the README, but I'd rather stick with full legal names in that case, as well as email addresses instead of websites. Not only do I rarely have either of those from the translators, but I'd feel terrible about publishing that info without the translator adding themselves to it. That might be easier to do with Transifex though.


Moved this method to controller.


Sorry! I don't remember exactly. My use case was a bit bizarre. Given a string of N words, have it split over one or more lines based on character count. Something in the sort bugged me.


@pumanitro , Find a way to print errors nicely


The <> ensure that the returned list is an ArrayList with type EPerson. This is called the Diamond operator & it is available in Java 7. (http://www.javaworld.com/article/2074080/core-java/jdk-7--the-diamond-operator.html)


This check is different from the checks for CPU0 in other handlers. Could you explain the reason better in the comments?


Sounds interesting, where would we keep the cache?


Ok, Thanks. :)


This was existing code, I just changed the casting & corrected the if query.


good.


ИМХО, `for (const auto& price_val : prices)`.


Oops I meant to push this on a separate branch. Anyway I think that deals with #1349
I did not change the tests in the file `uff-8.R` as I understand they are not run as part of the test suite.

I currently don't have a windows machine handy, @hadley can you run another win builder; hopefully we won't have too many iterations of this.


There's a vulnerability with the app that means any user (logged-in or anonymous users) can create todo items for any registered user.

What do you think about writing a feature spec to test this?


Is it better?
```python
msg += '[{}]'.format(', '.join(gcontext['graded_sections'].keys()))
```


Ok, so I (locally for now) fixed things to include the plugins as well as work for Brew. However, the problem you discovered remained. The reason for this is that the Qt libraries themselves load these plugins, and they must have the path hardcoded into themselves. Merely putting them into the standalone directory does not fix it, and they aren't mentioned anywhere in the library's load paths as they are only loaded conditionally at runtime of a program. However, fear not, as there is a fix: [QCoreApplication::setLibraryPaths](http://doc.qt.io/qt-5/qcoreapplication.html#setLibraryPaths). In order to make use of this, there's two necessary changes: First, qt-libs needs to override qt:make-qapplication. Second, Qtools needs to catch this crap during deployment and adjust the variable appropriately upon startup. I'll write again once I got that part all worked out.


`Item.new` can be replaced with the local `item` variable.


Can we perhaps have a comment here what's the difference between plugin_can_fail (or allowed_to_fail per #92) and keep_going (after failure) as I don't see it.


Much, thanks, but no need for keys() here too.


to make it clear that they need to download hmmer, not hmmsearch.


ИМХО, `void print_price() const`.


Nice.

Would you mind to remove the nine remaining occurences of `NotificationsService.addError('something', [])`?


ИМХО, `void print_scheme() const`.


Where did you add `fattaholmanan-venice` in Gemfile?


ИМХО, `void print_result() const`.


Lo tendré en cuenta, gracias


~okeanos authentication token. If you have an account you may find the authentication token at (Dashboad-> API Access) https://accounts.okeanos.grnet.gr/ui/api_access.


It is better to put Authorize attribute above controller.


`unless` doesn't compile, is what it does. Works in Ruby..

It's really the same as `if not`. In this case the code reads `if not not clustering`. But that seems to be the python way, I guess


list or retrieve


좋은 시도입니다. 실제로도 이렇게  많이 사용하지요.


ИМХО, `for (const auto& part : result_schem)`.


Or maybe a bad reaction for the new changes made


from the database.. i think it is not necessary ..


The `_item` partial template ought to be responsible for rendering a single `Item` only. Investigate moving the `.each` loop out of this template and how that affects the issue you're seeing with multiple items being rendered when you expect only one to be rendered.


Effectively I don't think that's needed :-)


It's not in Gemfile. It's only in the `purchase.gemspec` file, as you see above.


Стиль: из-за копипасты поехали отступы.


@miguelaferreira in my env, the test fails at this step. It might be because the 443 port is open.

  <testcase name="testExecuteRequestWhenClientThrowsIOException" classname="com.cloud.utils.rest.BasicRestClientTest" time="0.6">
    <failure message="Expected exception: com.cloud.utils.rest.CloudstackRESTException" type="java.lang.AssertionError">java.lang.AssertionError: Expected exception: com.cloud.utils.rest.CloudstackRESTException
        at org.junit.internal.runners.statements.ExpectException.evaluate(ExpectException.java:32)
        at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:271)

it works if I change localhost to other name..


대기 시간이 너무 적습니다. 1/1000 정도가 적당합니다.
이 프로그램이 대단한 일을 하는건 아니지만, 1/10000는 파일을 쓰는 시간이 부족할 수 있습니다. (ARM에서)


Just refer to authentication documentatiion page.


Oh, I see :)


I got it, thanks!


"predifinidas" es erróneo, es "predefinidas"


Wasn't sure about using < or <= here. It appears to be meant to function when the previous effectsgroup's upper limit is not met, so < seemed better.


Checkpatch script with flags "--subjective --strict" will fail if there is no comment after spinlock_t


even shorter `%w[I i l L O o 0]`


first merger


Note, the application.html.erb layout template is not re-rendered for AJAX requests that render `*.js.erb` files. This will likely be causing the no-flash message issue you're seeing.

Refer to the flash-related code in the `destroy.js.erb` template from the foundation for a way to get flash messages and AJAX working together.


We should explain the keywords (uuid, name, id)  in the response

Just add a new table
### Keywords in response

Name | Description |
---------|----------------|


Thanks.  The issue was also reproduced on my build environment.  I will disable MIPS16 for this package.

P.S. is it possible that this is a toolchain bug?


Содержимое этих скобок нужно вынести в отдельную функцию, чтобы не копипастить этот же код ниже.
Копипаста плоха тем что
1. провоцирует ошибки, особенно если нужно в копии что-то поменять;
2. для внесения одного логического изменения в код приходится однотипно править несколько мест;
3. с течением времени код разных копий отдаляется друг от друга;
4. больше кода труднее читать.


README.md UPDATE


This still have the potential to taint the default configuration. What if the old value was "no"?


I know, and it still is defined by the `settings.json` file.
Just if there's no `settings.json`, it uses these lines of code to generate one.

I'm still concerned about github repos. It's better then having "Unknown" like kyle stated.
If someone sees it's the wrong platform (in the UI) he can always dig into the project files, find settings.json and change it. :)


Nice work


Indeed that could be the case because I'm not actually mocking `httpClient` here. I create a real instance and then partially mock some methods with `spy()`. In this test case I don't even mock anything at all. So the fact that you have port 443 open will make the client connect, and hence no IOException.

I guess that your suggestion of changing the target host (to something that does not exist) would be best.


New comment


might be my lacking context, but I don't understand why this is cheating.


#approve


ahh and i thought it was only me hehe. So this one build sucess ?

I'll check that about gallery


Just for the record, the following was emitted by gcc with `-S` flag.

```
  75 #APP
  76  # 21 "/home/yousong/trunk-openwrt/staging_dir/toolchain-mips_34kc_gcc-4.8-linaro_musl-1.1.10/include/asm/swab.h" 1
  77         .set    push
  78     .set    arch=mips32r2
  79     wsbh    $2, $2
  80     .set    pop
```


I'd actually prefer if it were explicit that you need to set a region.


@resistorsoftware no worries. FYI, I hard coded sort direction because it makes sense that the default is descending, and because reversing it is as easy as calling reverse on the result. Cheers!


I was going from memory... obviously flawed.

Point is... not my code/decisions.


I think it's possible, trying to implement.


it is possible to release 3.1.2  to have this fix ?


I vaguely remember a content designer telling me right single quotes are the way to go, but given they are more difficult to type in and the end result is the same I am leaning towards using apostrophes.

However, rebasing this would cascade through all the changes in further commits, so I'd prefer to do it in a separate commit. Perhaps updating all smart answers at once?


Is it viable to setup [travis-ci](https://travis-ci.org) with [shellcheck](http://www.shellcheck.net) for `.sh` linting?

For example [SC2086](https://github.com/koalaman/shellcheck/wiki/SC2086):

`${BUILDER_TOOLS}` should be `"${BUILDER_TOOLS}"`


how about metadata[key[len("x-pghoard-"):]] = value


buuuuuh


Oh, i see for which to work you need the proper bash command.


@ccx-testerbot runtests master


Ok, thanks for informtions. ;)


In case of memory error, simply write an error message an `exit(1)`.


Nice!


Move inside `parse_args()`.


Hello @GarrettHoffman cc: @jonhanke @julialintern
typing "python" will begin interactive session.
This will give version:
'''!python --version'''


why not simply this.status = update.status? GC avoid >> code simplicity?


:-) https://github.com/tnataly/url-shortener/commit/a6408b1af87183d58f315a3434d7d865f4c2b9f4


(ditto above: could be direct assign)


This tests `get Obj:X field` but a test for `get Obj:X field index` would be good if someone can suggest an example object and field for which this is used.


I added it because when I was skimming over the class, I couldn't find the abstract function/s just by looking at it. So, moved the function below the declarations and added a comment. Nevertheless, I'll remove it.


No need for the `if` here, just ensure `path` is either NULL or a valid one.


Every unix should come with a /usr/bin/cc, even MacOS does (apparently).

/usr/bin/cc will be gcc most of the time, but for FreeBSD and MacOSX I guess it's actually a link to clang.


@GarrettHoffman
Ok with Vim and Sublime !


change to english


Not currently. My first concern was making RW work for CartoDB datasets (and then maps). I guess views and materialized views support will come later. Am I right, @javisantana ?


Leaking strings for invalid paths.


Thanks! Running now


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


Hi Dori. You do not have holidays?  :p
A Question, is it safe to enable adaptive_lmk with LMK_ADJ_RBTREE from motorola?

Thanks and good work.


No point, I'm just spreading the knowledge about `Visibilities.isPrivate` :)


Fix ich nach meinem Urlaub
Am 25.08.2015 4:34 nachm. schrieb Stefan Nuernberger <notifications@github.com>:In the latest aospal release 20150821 the camera of my find7 does not work anymore. Is there a bugtracker anywhere? Previous release 20150704 was still good, but it suffered stagefright.

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


The "s" added here is a typo?


Could do with an explicit test for the error message.


Or use individual arguments instead of tuples.


:tada:


Cannot accept any battery patches without native DSDT.

The u2442N patch looks same/similar to the one you propose to add.  They should be merged, if anything.


good work :)


Hi @GarrettHoffman
a.  Would be helpful if cheatsheet were in list format; easier to read
b.  xargs, explanation and info below:

xargs is a command used to execute commands based on arguments from standard input
#Recursively find all Python files and count the number of lines
!find . -name 'c*.py' | xargs wc -l


this could be more readable, if:

r := ctx.Request()
if r == nil {
        return
}

url := r.URL
if url == nil {
        return
}

if !f.preserveAsHeader {
        url.RawQuery = ""
        return
}

q := url.Query()
for k, vv := range q {
	for _, v := range vv  {
		if r.Header == nil {
			r.Header = http.Header{}
		}
		r.Header.Add(fmt.Sprintf("X-Query-Param-%s", sanitize(k)), v)
	}
}

url.RawQuery = ""


+1 :D


To sum up, to make `gulp-compass` work, we need to
* Install `sass` gem on the machine (this step is optional as `sass` would be installed automatically as a dependency of gem `compass`)
 * `$ gem update --system`
 * `$ gem install sass # It is better to use "gem install sass -v 3.4.16"`
* Install `compass` gem on the machine
 * `$ gem install compass`
* In the project directory, install npm module `gulp-compass`
 * `$ npm install gulp-compass --save-dev`
* Import `gulp-compass` module in `gulpfile.js`
 * `var compass = require('gulp-compass');`


TeamCity Azmq :: Build (OSX) [Build 114](https://78.46.71.115/viewLog.html?buildId=593&buildTypeId=Azmq_BuildOsx&guest=1) is now running


TeamCity Azmq :: Build (Windows) [Build 161](https://78.46.71.115/viewLog.html?buildId=592&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 131](https://78.46.71.115/viewLog.html?buildId=591&buildTypeId=Azmq_BuildLinux&guest=1) is now running


well, with leading whitespaces of course


TeamCity Azmq :: Build (Linux) [Build 131](https://78.46.71.115/viewLog.html?buildId=591&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 27 Build time: 00:00:09


umm... how many milliseconds did you say it was?

Also, I'd recommend to not refresh this that often. Consider the case if the review page is not cached, then having x users update that page 4 times per minute may put some extra pressure on the poor Stack Exchange servers. Is it really necessary to refresh this four times per minute? I would think that once every two minutes is more than enough.


TeamCity Azmq :: Build (OSX) [Build 114](https://78.46.71.115/viewLog.html?buildId=593&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 27 Build time: 00:00:19


TeamCity Azmq :: Build (Windows) [Build 161](https://78.46.71.115/viewLog.html?buildId=592&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 27 Build time: 00:00:21


Maybe we should prepend here.


@Agontuk my gps in high accuracy mode doesn't work, in particular I have Google Play Service fc when I try to get location (ie in google maps)


Hi @aembler, thanks for fixing this. We just noticed the problem when running some upgrades on a few beta sites. Will this be rolled out in 5.7.5.2 and, if so, any idea on when we'll see that release? Thanks!


FBP in uppercase here.


Use same name as in runner.


Lines 6-15 could be optimized as:

.container-pf-scroll {
  height: @container-nav-pf-vertical-alt-height-calc;
  overflow: auto;
  .container-pf-sticky-footer  > & {
    height: @container-nav-pf-vertical-alt-with-footer-height-calc;
  }
}


And also that innocent C++03 code is mistreated:

```
float complex;
float foo(float complex) {
    return complex;
}
void bar() {
    float complex;
    foo(complex);
}
```


test for github


In the case where the cluster becomes unhealthy, where would we update the allowed operations?


First, I find this pretty readable. Second, there's a lot going on here, that is precisely gated by the specific incarnation compares (>, >=). Would this be worthy of a higher level english description of the intended logic? Thought?


@WaleedAshraf you messed this up


Yes. :)


It seems like the original is much stronger.


Lines 21-36 could be optimized as:

```
.container-pf-nav-pf-vertical-alt {
  margin-left: (@nav-pf-vertical-alt-width + 5px);
  transition: @flyout-transition-pf;
  &.collapsed-nav {
    margin-left: (@nav-pf-vertical-alt-collapsed-width + 5px);
  }
  &.hidden-nav {
    margin-left: 5px;
  }
}
```


I'd get it faster if it was:
[conf....
 for conf in plugin
 for plugin in plugins.values()]


Yeah I thought I fixed it in 5.7.5.1 but missed an instance of the problem. If you apply the develop branch to these same servers the problem should be fixed. Could you try that and see if it works ?

Sent from my iPhone

> On Aug 26, 2015, at 6:34 AM, Corey Reed <notifications@github.com> wrote:
>
> Hi @aembler, thanks for fixing this. We just noticed the problem when running some upgrades on a few beta sites. Will this be rolled out in 5.7.5.2 and, if so, any idea on when we'll see that release? Thanks!
>
> —
> Reply to this email directly or view it on GitHub.
>


(Follow up - this thought is not new to this diff - just reminded of it, since you're already in there.)


Sure


Yes, it probably should be done separately.


no idea, the only place I though about mat views is to deal with FDW.

What are the problems with views @pramsey ?


i prefer you to use the default optimization, it more stable than archi optimization. And to avoid errors and bug...
Even CM dev never use what archi did


@jwolski only questions/nits; otherwise lgtm


monetary widget


Modulo some minor restructuring which can be done separately, this seems fine.


There are a regular 11 core and two occasional 23 core (= 2 × 11) machines in the pool so an 11 core LTC 60s test will use those machines efficiently and free up the 7 core engines for other (SMP) tests, but it will run longer.


This is indeed the line of the problem, however only KML data will encounter this, not all data (e.g. SHP). I'll refer the commit and close this PR when the hotfix commit has been pushed.


Problem with views is just "are then handled". Once people can grant group roles on tables, they will naturally want to do so on all the objects in their database, and enterprise customers will have a higher than normal likelihood of having "weird" objects in their databases, like views or matviews.


最早的时候 是指定暗灰色的 但是某个主题的alert比别的主题宽和高1px左右 就会看到一个蓝色的底边


Lines 1-18 could be optimized as:

```
.nav-pf-vertical-alt {
  .backface-visibility(hidden);
  background: @nav-pf-vertical-alt-background-color;
  border-right: 1px solid @sidebar-pf-border-color;
  float: left;
  height: @container-nav-pf-vertical-alt-height-calc;
  overflow-x: hidden;
  overflow-y: auto;
  transition: @flyout-transition-pf;
  width: @nav-pf-vertical-alt-width;
  z-index: 1000;

  .container-pf-sticky-footer > & {
      height: @container-nav-pf-vertical-alt-with-footer-height-calc;
  }
```


I don't think we need to start the sensor detector from here, as it should already be started. In fact the whole setup looks quite weird. Why do we have a sensor detector that manually sets the orientation of the `Activity`? I guess it was implemented to catch when rotation is disabled, and then exit full screen.


Basically, add `.pipe(plumber())` to the `gulp.src()` of each task that we want to prevent from breaking the `gulp.watch` task.


I should make it work with both but I cannot test with valueString. Can somebody point me to  specs about the "valueString" possible formats/values?


The "s" is a typo?  Sorry for the duplicate comment.  I realized I commented on the compiled CSS after the fact.  :)


It is good practice (if not required) to add `.pipe(plumber)` before any other `pipe`.


Are the `test:tinyCollection` and `test:smallCollection` used? I don't see them in these changes. They seem to be invalid, since they are `orderable` and allow same-name-siblings. If they are used to verify those attributes as being invalid, then perhaps rename them to signify that they are not valid.


@malonecj  as we discussed it on slack , returning the underlying value here, so that if the value is false is not replaced with an empty string.


This allows same-name siblings - how can it be used for an unordered collection?


I'm afraid the situation will not be worse than now, I'd add that as a todo.

In any case, how do you deal with a mat view created from a join using 2 tables?


Thanks a lot ! Take care of yourself!


Please remove this empty line and the following comment.


Move this check inside `handle_include_dir`.


Thank you for the request. It has been added to the queue (id: 76). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/76) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


Please remove the empty line and the following comment and correctly alphabetize these vars (around line 50).


The line looks like it should have been removed