The Font included was supplied from a URL in the tutorial.  The font is available at http://www.dafont.com/perfect-dos-vga-437.font, and was created by Zeh Fernando.


:+1: :+1: :+1: Looks really good!


Is there a reason to create this file?


> Does that mean they stay in the title bar no matter what? :D

No, if you don't want to, just don't include `${cpuusage}` or `${gsdx}` at `TitleTemplate`.


seems like outerHeight() is getting wrong here, need to fix


No documentation comment !!


Why safe a reference to `this` if you never use it?


Unicode character names in comments should be updated together with characters themselves.


My documentation comment has been deleted. Why??
This is a role in RoR convention.


No documentation comment!!


I noticed that my average OBP for all of MLB seemed a bit low, especially when I compared it with official stats from that year (2004). The individual players' OBP was fine. So I went back and removed all the OBP of 0, only including an OBP greater than 0. This did increase the overall average MLB OBP, and also moved the average salary up a bit. So I  re-ran all the cells and it did end up affecting a few players. I changed out 3 players based on the new numbers. I'm pretty happy with this team, and I think the OBP_Sal stat that I came up with (as explained in the first dataframe comment) is actually a really good way to measure a player's value.


Implement view category list page


haha jk, wasn't a lighting change, this was me enabling the cloud build shit. whoops.


Issue already reported here: #5076


@hjmjohnson It is ok to remove one of the signature of `ComputeJacobianWithRespectToParameters` ?

I was thinking to list the function that are removed because of

```
Reviewing these changes also identified some dead code
that was intended to override a parent class an provide
exceptions or no action, but infact did not override any
parent virtual function.
```


Still not showing the contact details on the place page. Only the name is shown.


Well it's not really for me it's more just to make sure end-users don't remove them. Both of those are useful when troubleshooting problems on the forums.


102867486-FixRectangleShapeTest-5173 (mjwitte) - i386-Windows-7-VisualStudio-12: Tests Failed

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


@m-kuhn wrote:

> ... Just a random guess, is that compiled with RelWithDebInfo and that's an effect of this?

Bingo! That was exactly the issue, since I _am_ doing RelWithDebInfo builds. So, in the `qgis2.11.0.cbp` file produced during configure, it defines

```xml
<Target title="qgis_core">
   ...
   <Compiler>
      ...
      <Add option="-DQT_NO_DEBUG" />
      ...
      <Add option="-DQGISDEBUG=1" />
```

Looks like someone (@nyalldawson ?) ran into this before and [used the Q_UNUSED workaround as well](https://github.com/qgis/QGIS/blob/master/src/app/qgsapplayertreeviewmenuprovider.cpp#L268), but apparently for a variable used within a `QgsDebugMsg()` call, which is a bit more odd, in that that macro is not expanding to a noop. Of the 77 uses of `#ifdef QGISDEBUG` there are no others that have a nested Q_ASSERT.

Suggestions on the best fix here? Just `#include <cassert>` and use `assert(...)` instead?


h


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (836 of 836 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-24.31%25-green.svg)


Also added employees as one of the actors for the Login use case, seemed to make sense, Nicole updated the diagram to reflect this change but it still needs to be committed.


p + h4 should not work in SCSS. You should try to use p, h4 or p > h4


Where are the mixins?


compilable must return valid php code that's put into the compiled file, NULL results in broken compiled templates like:

```php
<?
// Rendering ViewHelper FluidTYPO3\Flux\ViewHelpers\Widget\GridViewHelper
$arguments22 = array();
$renderChildrenClosure23 = function() {return NULL;};

$output21 .= ;
```

we probably need to make that return:
```
return 'NULL';
```


same here


this s test


This is an interesting approach. I would not have thought to replace the methods with `noop`s. Does `this._closed` have any use or meaning with this approach?


Open to suggestions on styling this arrow.


I’m following the book in this assignment by using couple mixins which are already defined in: bower_components/bootstrap-sass-official/assets/stylesheets/bootstrap/mixins/

They are called:
border-top-radius(0px)
and
border-bottom-radius(50px)

and I included them in the _content.scss

I was asked the same thing by my peer reviewer, am I missing something?!

On Oct 7, 2015, at 9:28 PM, Vincent Ning <notifications@github.com> wrote:

> Where are the mixins?
>
> —
> Reply to this email directly or view it on GitHub.
>


@jheftmann :eyes:


Utilizando o casting para converter o número 2 em um float.


done


I've never heard of `Xnode_modules`. What creates this?


sdfsadf


Actually I may prefer the previous sorting ;-)
The ideal order would be full alphabetical ordering, for instance `t` and `T


This is how key bindings are sorted in the code and in the documentation files.


Ali, you're absolutely correct. I just double checked that and the mixins were provided for you by bootstrap so I'll go back and give you full credit for that part of the rubric.


Thank you!

On Oct 7, 2015, at 9:40 PM, Vincent Ning <notifications@github.com> wrote:

> Ali, you're absolutely correct. I just double checked that and the mixins were provided for you by bootstrap so I'll go back and give you full credit for that part of the rubric.
>
> —
> Reply to this email directly or view it on GitHub.
>


I've just pushed some changes in 5dd10ba0e7a943d4addc91bf2e6a8e3822485190 that should help with some your concerns. This change was primarily made to the `render()` method, which removes the data and escaping features altogether, requiring that `setData()` or `setVar()` be used, when not using the convenience factor. I believe this will provide the flexibility for your thoughts here, as well as for integrating with third-party libs.

However, this all hinges on NOT using the `view()` convenience method. All of the use-cases that you presented are outside of the "traditional" CI view handling style, so would be better addressed by using the View class (or another Renderable class) directly.

The `view()` method is simply a convenience method intended to hit the most common use cases, and mimic the previous $this->load->view() syntax reasonably closely. The escaping is a security feature that helps us devs to fight XSS properly, on output, without having to remember to manually escape all of the time. It's one of the biggest security features of any template engine. And, while this isn't a template engine, this bit of added security is a needed thing, I believe.

I'm definitely open to revisiting this down the road once it has been used a little more to see if it needs another change for usability but for now I believe keeping the `view()` method simple for the 80% use-case, and allowing further options in the View class directly is the best choice.


We should try switching this to `glob` or `globby`, I'll do a new PR :smile:


Typo


Do we really want to import all models and forms into here? There were only two specific imports removed from `django-threadedcomments`, for a model and a form. Maybe we should just explicitly import them, instead?


@MilosStanic can you pull request to fix it?


Why are you prepending "controller_" here when a) This macro is used for vision as well as controller and b) `controller_` is already part of `{ID}`?


---
layout: default
title: Hank Quinlan, Horrible Cop
---

Hi there, I'm Hank Quinlan!

I'm best known as the horrible cop from A Touch of Evil Don't trust me. Read more about my life...


I'm not. I removed the prepend.


@eduoda notice here the introduction of hook_online() and hook_offline(). With the online hook, app's can weigh in and say whether or not the app is online. If the app is not online, then we can invoke the offline hook, and then others can take action. Either way though, as you can see, if the app is reported offline by someone (e.g. https://github.com/easystreet3/jDrupal/blob/7.x-1.x-offline/tests.html#L86) then I think we should immediately queue the submission, and return. That's what I've implemented.

I do like your timeout feature, and I think that should still exist in case of timeouts. But in most cases, the app will be running in PhoneGap, so it's important to be able to check for a connection before proceeding, that way we can take action immediately if the app is offline.

Next up I guess we'll want to work on processing the submission queue at a later time when the app is online.

Thoughts?


I was too scared to go that far myself. Feel free to do that, I would really appreciate it :rose:


probably want to assert that len(phases) == 1 in this case, or else you might throw away args silently.


It's not the bot user who benefits- they see quickly on slack that their bot is offline. The potential time waste is for the bot admin who needs to investigate why their bot is flooding channels or whatever. You could make LIMBO_DEBUG turn up the ping rate if you wanted.

If limbo does well and gets used by just half of slack's 60,000 teams (stat from 8 months ago), this one design choice will cause 48 megabits/sec of traffic to slack (each ping is 200+ bytes, according to strace, which is where I first noticed them). At this rate you could just poll instead of using websockets at all and it wouldn't cost much more.

No action required, though. I'll run mine at 30sec and you can run yours at 1sec.


yeah, the call to Kill() doesn't raise the error... nevertheless, the resulting log spam is really annoying, so i've fixed it in my latest pull request :)


"That was easy."


`$(e.target)` can be used only when the user presses Enter key, not when he clicks on the send button. So I think it would be more consistent to use $("#input") in both cases.


did you git mv this file?  looks like it's going to lose revision history on it like this, dunno if you can do anything about that?


Added more context to in commit : a3089c3


can we start nuking cells now?  and have you only take one in?  that's next on the chopping block for me.


Hello


you can switch the outer ' to " so you don't have to escape the apostrophe (that's the convention within google3 at least, and i suspect elsewhere).


Hi,

Did you know if there is a way to open the PopUp without clicking it? This popUp: https://i.gyazo.com/bda10009d630a1cf22e3b54dd2933679.png

I am reading de code and i found this: plugin.playerTracker.onClickListener
But i wanna know if that function would open the popUp automatically on all "players" on my map without clicking on the player image

I am supporting @DuNord_Intel_Bot on Telegram and that's will be a great feature
Thanks a lot!


Ну как же так:
![image](https://cloud.githubusercontent.com/assets/13660819/10358143/d072fdbe-6d92-11e5-977c-c6db73d5688d.png)


#Potwierdzam, wyliczając z metody symulowanego wyżarzania to ktoś tu leci w chuja


I thought we don't use this any more, is't it part of the executors now?


LGTM


Do you mean merging sendMyMessage() with checkKey(), or sendMyMessage() with sendMessage()?


Да, Spring Boot 1.2.6 использует Thymeleaf версии 2.1.4.RELEASE


102867486-FixRectangleShapeTest-5173 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Well no, cause the executor is used when a service consume a node.
But the node.ssh itself don't use the executor, so we still need some code here to connect to the remote node and be able to execute commands.

What we could do to be more consistent and keep the code in one place, is to use the executor in here.
So we keep the 'node' logic only in the executors.


Swiftylib autoloader should not use sdd theme swiftlylib when not active


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


Sorry, take it back, there's no problem as the memory size increases with the inline `base < size - offset` test, but perhaps at the other end as the size decreases might the offset need to the less than the memory size.


I was just making functionality similar to the boostc python lbrary. That one allowed for motor.run_to_abs_pos(50). Why the different architecture?


#Greeeaaat work!!!!


# Greeeaaat work!!!!


Wow, you just added 16 bytes to ___processor_state_struct!  My life with bignums might have been easier if I had give myself another 8 or 16 bytes of temporary space to work with.


So much readability ! Great job ! :D


But can't you just set `box.x` and `box.y`?


Not available on windows and I'd rather not start defining code that runs two ways. I can assume we have a rule of lower case for mesh names and skip this, though.


Seems that it's a bug. No one has touched this since this commit AFAIK.
My guess is that it's copied from minion/init_script, by which i mean, that part is wrong as well unfortunately :-/
Will fix them both.


That's why I'm saying I should take adding children out of container:wrap altogether, and container:wrap can just become container:relayout.


102867486-FixRectangleShapeTest-5173 (mjwitte) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


local costmapに走行可能領域，未計測領域，走行不可能領域を表現するためsafe_layerを追加


Аа :D то трябваше аддед да е :D


Why would you want to define the layout externally?


The diff seems wrong, it seems you are testing against master: please check it.

I delete the test.


float -> uint16_t


Я не удержался и тоже заглянул в этот класс :) мне кажется, что причина в этом: `import org.thymeleaf.extras.springsecurity3.dialect.SpringSecurityDialect;` - третья версия вместо четвертой.


Not merging this breaks the library. Can't run it here.


Yes we want to translate with the %s because depending on the language, the order of words may vary. The translation is done before the string substitution, so no problem here. The translator just has to add %s in the translated string at the right place, depending on his language.

That's the way it is done everywhere else.


「移動できる(movable?)」ことを示すレイヤーですので，「安全(safe)」というフレーズを使用しないほうが良いと思います．
後で読んだ人が混乱するおそれがありますので，誤解を生じさせない名称を割り当てたほうが良いです．


no pasa nada, es porque el mío estaba minificado, hasta que no vayamos a pro no hay problema con esto


@HenriqueLimas I just realized this was merged into master. The website is based off the gh-pages branch so if you want to see your work reflected on the site please resubmit this correction against the gh-pages branch.


Actually, I don't think so :)
It's impossible to kill a minotaur without cheating, so I think I won't see it :)
Try to play on the server with a NON-diamond sword and with a NON-diamond complete armor, you'll understand.


If that is the case, then it was not by intention. I'll update the PR throughout the day.
Best,
P

> On 07 Oct 2015, at 23:57, Andi Kleen <notifications@github.com> wrote:
>
> It looks like you replaced the ucevent symlinks with real files?
>
> —
> Reply to this email directly or view it on GitHub.
>


This isn't a good practice. You should sync locally then push to origin.


Please also update documentation


Yes, django is from PyPI. It would have to be in collection in order to install it as rpm.


ouch. rebasing.


Sure. Makes sense.Fixing it


lol


@taion the only reason I had the above `if/else` was in order to prevent invalid html and block elements been printed inside inline elements.


f


@john9804 You're formatting your titles wrong, reread the wiki page and look at the template to see what you're doing wrong.


Hi 😊


This isn't a good practice, you should sync locally, then push to origin. Here's a script you can put in the Git Bash prompt that will do most things for you, make sure you have the working directory correctly set to your repo, and the master branch is checked out.
```git fetch https://github.com/tgstation/-tg-station.git master; git reset --hard FETCH_HEAD; git push --force origin master```
Then all you need to do is authenticate and you will sync up your github master with tgstation's master. You will lose all changes to master, but that is the point.


This has the side effect of having the server exit when the screen saver goes inactive, which will disconnect the client(s) and cause the server to not restart for 2 seconds. I don't see any other easy way around this bug. Perhaps the server should check if the screensaver is active when it starts, since we know that is the condition that causes EnumClipboardFormats() to fail.


Why VFPS and not VPS or Hz? According to HyNES, which is the only other emulator I can find that mentions the term, "VFPS means virtual FPS, or frames that are being rendered, but not necessarily displayed". So it seems that the usage here is incorrect.


I double checked and the Gustafsson equations has no gas constants in their “Arrhenius type” terms.


awesome :+1:


It seems that the grammar only supports adding one column at a time:
ALTER TABLE <tableName> ADD COLUMN <identifier> <type>

I wonder how does TestHiveDistributedQueries.testAddColumn work?
ALTER TABLE test_add_column ADD COLUMN (a bigint, b double)
For me, it fails when running in IntelliJ, the error is:
com.facebook.presto.sql.parser.ParsingException: line 1:40: no viable alternative at input '('
	at com.facebook.presto.sql.parser.SqlParser$1.syntaxError(SqlParser.java:46)
Did I miss something?


Used ```tab``` instead of ```space```. Will push the necessary changes asap.


Please improve your GIT commit messages! Use `git commit --amend` to rework.


@JJRcop I only did this because I made the PR on my phone using github and I wanted to make sure it wouldn't conflict
when I'm actually at a computer I do it properly :v


Can you build two implementations properly?


let's just add JSON to `REQUIRE`


Blokay, my bad


You can use `git rebase`.


Since we already defined macro `IMPL`, the access to macro `OPT` is redundant.


I would delete "In order" and delete the comma.


Does it have to use `Hash`?


Oops, oversight on my part. Sorry.


Mahbe they don't run for Hive unless a profile is enabled, so we aren't actually running them?


"It takes approximately one hour for OpenStack to automatically deploy on the Open edX environment"


Shit


Yep, it's a thing in both Ruby and Python at least.  I like to learn all the tricks.


Oh :smile: I forgot


This also fixes a bug with the temple in the structures.yml file


Oh i have to revert power part then :(


Delete "should". Do they need Icehouse 2014.1 or later to run on OpenStack? If so, I would say "must". If not required but it's optimal, I suggest you explain this in more detail.


I am no go programmer so not sure what you mean but I have good friend Google that can probably help me, I will also look at the java implementation


这个作者咋不更新了，用最先的api编译，就挂拉。好像最新的API下边这些个类都没了。
我现在用的api23，为了不挂，就改回19了，没试过api多少开始挂
import org.apache.http.Header;
import org.apache.http.HttpEntity;
import org.apache.http.HttpEntityEnclosingRequest;
import org.apache.http.NameValuePair;
import org.apache.http.client.methods.HttpRequestBase;
import org.apache.http.client.utils.CloneUtils;
import org.apache.http.protocol.HTTP;


if we generate a cookie with a session id, use this sessionid to encrypt the username/password, and add a hash as well, would that be safe ?
maybe one year is a bit to much ;) a month or two would be fine as well


There's nothing wrong with the exe, please refer this comment

https://github.com/nishad/udemy-dl-windows/issues/5#issuecomment-145242296


Why you made space ?


Justt for fun


"You will require the following to run the Heat templates":


Delete "You have"


I'm flattered that you took the time to read through enough of this program to make this fix, so I'll take the request as a sign of respect to you.


doh!


replace "should" with "will", otherwise I want to know why it might not?


经测试，确认修复。非常感谢。


Delete "you know"


Delete "You know"


cpan上1.5.4版本也已经发布，如果之前是通过cpan安装的模块也可以直接升级下比较方便

发自 网易邮箱大师
在2015年10月08日 13:54，Finder 写道:

经测试，确认修复。非常感谢。

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


"The Nova keypair you created"


Note that I started writing https://github.com/open-mpi/hwloc/wiki/Upgrading-to-v2.0-API to ease the transition. If you see missing/wrong information there, please let me know.
Also note that there are other API changes coming in hwloc master soon (rework of topology flags vs ignore() functions, and possible rework of the distance API).
Early feedback about all these hwloc changes would be very appreciated.


Thank you!! :cake: :grin:


So, it's fixed in my branch b/c I didn't want to push anything to master, but the second condition should be `Application.platform != RuntimePlatform.OSXPlayer`


"An Ubuntu 12.04 image for your tenant that you have uploaded into Glance"


Delete "either"


isso anda um pouco atrasado...


I didn't really think about that possibility. I'm running this against OpenLDAP.
Should probably do some native tests against both AD and OpenLDAP to document the differences.

Great work with this project btw!


Delete "in order".


is this correct?


All other arguments in the periph/timer API is using the basic int types without width, so I thought we should follow that practice when changing this parameter


Why do we have a default value for ip argument here ?
no more question when doing ```ays init -n node.ssh```


sure thats what i meant !


It's the same logic here - I just omit the explicit `else` and return directly (since in both cases you just return `brand` right away).


nope, sorry I forgot to fix it before the push.


Can this be made into a procedure to make it easier to follow. E.g.
Set three mandatory parameters to invoke Heat:
1. Set `public_net_id` to the application servers you want to spin up.
2. ....etc

Secondly, set the name of the stack:
......
etc


:) my pleasure


No it doesn't but looks cleaner than `isset($config['applicationRules']) && $config['applicationRules'] === false`


True.


Great, let's wait what @shurcooL has to say about it :-)


"To enable the `git` command in the next step, ensure that `cloud-init` is completed by" (how do you do this?)


Hmm actually the `isset()` is not need, I have added default so will get rid of Hash::get()


Brad! Nice to know you're still watching these things. And you are of course correct.


"To verify that the stack reaches the `CREATE_COMPLETE` stack run:
`$ heat stack-show <stack_name>`


Made `_.words` more lenient. To solve the issue of casing rules vs total conversion I'll add `_.toLower` and `_.toUpper` as well which don't apply casing rules.


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


@jadb Err scratch that, isset() would be needed, by default lists don't have any options.


@sobinsunny  check this out


Better if it is patientIdentifier so that user knows that only string which could be found in identifier list in OpenMRS is valid


to include "bower install". git commit ate half of my commit message.


As usual.... Comes of rushing to get it committed before the POL2 meeting. Apologies.


@sobinsunny


Updated. Regarding saving verified number strip is done in ```apply_leniency``` so we don't have to worry about this (https://github.com/dimagi/commcare-hq/blob/master/corehq/apps/sms/mixin.py#L644, https://github.com/dimagi/commcare-hq/blob/master/corehq/apps/sms/mixin.py#L546)


không dùng được, tên file gửi không đúng!


Didn't work, as in the previous two commits. Command line is apparently the way to go on windows.


mine does not evn want to logcat anymore


This try is bad, but this is an information. I'll try to draw all three graphs (default and two of mine),
and we can discuss in the forum about the best formula.
I'll now stop this test.


fileCategory should be renamed to category. File category seems like physical files type, but here it is type of image (from business logic perspective)


Tam je jenom novej radek, akoratze ted stejne pouzivame jinej travis.yml, ktery podporuje service: docker takze aspon nektery docker prikazy tam jedou, ale stejne ne vsechny (psal jsem ti to do slacku kdysi), tak nevim kterou verzi travis.yml pouzit.


This is not needed as Spring wont allow null param pass through method. Rather make it empty/whitespace check


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


If possible replace it with a check that allows upload of image files only...


this what


su slepá piča!


Donot hardcode it...


Thanks. Will look at it once I have some (at least semi-)free time. :+1:


If `this.primaryKeyAttribute` is undefined, the condition evaluates to `-1` which I believe gives a false positive.


What would happen if there are 1 million patients in system!!


It is a good practice - separation of logic and view (see MVC - Model View Controler for example). As you can see, I can update view (layout) in love.resize() without even knowing which object are created, and how many of them are created. I can imagine a user configuring their own personal layout saved in external file. Then you could supply his layout to your function and that would just work - for any user.


Is it known what are vdv computers? Do they use real cores? Have they been used in any other test?
Maybe this test should rerun on computers confirmed to have real cores


OK for me.


@gcapalbo Yes, before we have converted WebUser with phone number to CommCareUser during migration. Here is the change: https://github.com/lwyszomi/commcare-hq/commit/98ba0cb51da7d2a33fc192fd3059dc1f834f6ce4#diff-1ef4aabc982e7f8631c9b001830ea163L605


Is this done?


確かにsafeは避けたほうがいいですね
safeからmovableに変更しました


The second formula was worse, but this one works good so far, and was good on 4 cores too.
Let's continue the test, and let's discuss on the forum. I'll try to prepare some graphs for easier conclusions.


Merged it with `Hash` for now.


The code is almost same as addImage... why two separate methods with same code


The advantage is to ensure a consistant color scheme


closed #2


Hi Tony, I think we are using the wiki for information like this. You can access by the left tool bar, with an icon that looks like an open book. You can edit any of the files (3 so far - pertaining to each of our meetings). All previous versions before editing can also be browsed as well by clicking edit, and then clicking on history


Yes, probably (for resize() it would be .w and .h, .x and .y would be for setPosition() or similar). It is not critical, so yes, it is for convenience (writing one line of conde instead of 2 lines)


把bower_components的地址硬写到脚本里，发布生成index.build.js时会导致路径错误。


実装例いりますかね・・・ｊ


Great job :)


Did not understand the use of this..


Is there a way to make it so I can use spaces in the limiter string? Right now if I have `LimiterTurbo= (Turbo)` the space won't show up in the title bar. Obviously I could just add the space in the TitleTemplate string but then the space will always be there, even when the limiter is on normal.


Why? :smile:


Donot hardcode patient_images here...


Thank you! The RenderableInterface is flexible now, and this is the important thing. I have no problems with view() then.

One tiny thing about the escaping contexts: They are (uppercase) ['HTML', 'JS', 'CSS', 'URL']. Maybe another non-default one 'RAW' or 'NOESCAPE' or 'PURE' should be added for turning off the escaping at will.

Thank you very much, again.


Yep @snail23 too :) Sorry for that, you dont have to instantiate bundle::strings at all, plain std::strings would work.


Exactly, we want to send alerts to users that are assigned to administrative location and in ewsghana v1 it's only possible to assign location to webuser (mobile worker can be only assigned to non-administrative location)


Please don't use absolute home directory path. You should use `$ENV{HOME}` instead of it or install `fast-rcnn` under not home directory such as `/usr/local`, `/opt` etc.


Please use a more meaningful variable name for `splientry`.


where it is saving attribute incase doesnot exist?


@nmittler Is this correct? It seems like it returns `false` even if the stream with the specified `streamId` exists because server's last local stream ID is supposed to be smaller if the server received a request but did not send a response yet.

For instance, I get the following exception:

```
io.netty.handler.codec.http2.Http2Exception: Stream does not exist 3
	at io.netty.handler.codec.http2.Http2Exception.connectionError(Http2Exception.java:82) ~[netty-codec-http2-4.1.0.Beta7.jar:4.1.0.Beta7]
	at io.netty.handler.codec.http2.StreamBufferingEncoder.writeData(StreamBufferingEncoder.java:198) ~[netty-codec-http2-4.1.0.Beta7.jar:4.1.0.Beta7]
	at io.netty.handler.codec.http2.HttpToHttp2ConnectionHandler.write(HttpToHttp2ConnectionHandler.java:112) ~[netty-codec-http2-4.1.0.Beta7.jar:4.1.0.Beta7]
```

and the problem goes away when I disable `encoderEnforceMaxConcurrentStreams` when building `HttpToHttp2ConnectionHandler`.


Kirill, WTF? Why we need to hardcode IP addresses in rb file?


You can run mima:
```
> mima-report-binary-issues
```


manups4e fixed graphics problem on g3 but it still not boots up. maybe his sources will help you


use uuid or a system id that is unchangeable... what if attribute is change or removed or voided?


I think the case where `keyValue.length != 2` should be managed, otherwise an ArrayIndexOutOfBoundException may be raised if a user use a definition like `var=`, `=` or `var==`. It would be also nice to unit tests these cases.


Same comment as before. Since the code is the same, it would be nice to factorize it in a method.


try avoiding special characters in message


Same as above.. What is the use of this variable?


You can use scoped enumeration.

See also
- http://en.cppreference.com/w/cpp/language/enum


First of all, it is because I wanted to do the first work of removing the Cookbook of Apache.
Bad Practices?


Is it not possible to rely on `com.eclipsesource.minimal-json:minimal-json:0.9.4` (which is already imported for tests) to prevent to use yet another library?


`break`s after `return` make no sence.


attribute type name should be prefixed with "Patient Image"


That's nice.

Now we need a gui element for that :P


Why did you use `enum` type(enum RcnnDetectorClasses) ?


Missing space around `!=`. There is a formatting profile available at [1], are you using it?

[1] https://github.com/ow2-proactive/coding-rules


hint hint @turtleli ;P


@navitronic @pda this is where things are getting tricky. We need to return the request back the to handler stack.


Done.


@Ocramius I use it without on 2.5 and still got no issue, that's why I'm confused.


Thanks, @nathansobo.


Hmm. I tend to regard those as optional (there's always `git blame`), but yeah, I guess so.


Missing spaces around `=`.


I just realized this.  I'm off Friday from work...I just have a 9am class and after that I'm free.


I suppose using last value of enum is better than such macro. With following code, we need not to count class numbers.

```c++
enum RcnnDetectorClasses
{
   BACKGROUND,
   PLANE,
   ...
   RCNN_CLASS_MAX,
};
```


Function definitions in `class` declaration are always inlined.


done


Hm. It'll only be an issue if they're on the same webpage, right?

Sounds like something we should do a backend test for when we have the new directory structure -- it doesn't seem that hard to check. If we go for custom naming now then we're basically breaking the convention that we wanted to put in place with the new file structure.


这里是什么意思


Can we make the placeholder `14px` as well to be in line with the other placeholders on the page ?


Alphabetic ordering


This _apply method will always overwite the service.hrd with the content of the templates.
if this function is called on a service instance, all instance value will be deleted. Don't know what's the intention here...


実装例いりますかね？`<spring:mvcUrl>`タグ使いたいだけなのにここ見ると
拡張例が目に入ってちょっと面食らうような。


This was as long correct as there were no single values (non-dicts, as `sysconfig['foo']` in the example above). The code was correct with this assumption.

I pushed a fix correctly handling dict and non-dict values.


2nd argument of `ConvertImageToBlob` is reference, while 2nd argument of `ConvertRoisToBlob` is pointer. why ?


Like we discussed, lets put the onus of deleting the manually created ports , on the user who created them and not do this as part of the fixture cleanup


Those are much better names. Thanks! Fixed.


> This sentence seems weird: "a small number of observations from the
> information"?
> I would say it oppositely, "a small number of information from the
> observations".

That's correct. Do you want to make a pull request fixing that?

Cheers


What about adding tests for the following cases?

  - Empty key but non-empty value
  - Non-empty key but empty value
  - Empty JSON payload `{ }`
  - Two entries separated with a wrong delimiter (not comma)


Please don't add python compiled file(`*.pyc`).


@rhysd that would be great. I spent a couple of minutes trying to see if their was anything obvious, but realized I would need to do some learning/research first.


It still looks a bit odd for tenancies without a flag:

<img width="641" alt="screen shot 2015-10-07 at 23 44 39" src="https://cloud.githubusercontent.com/assets/109850/10359423/6c8c8058-6d4d-11e5-98f5-53ac723e8333.png">

Maybe we should just remove the space for the flag for tenancies without a flag, and accept that not all tenancies in the returned might be perfectly aligned because of it.


Oh yeah, of course, just noticed! I really like the simplification especially in `Navbar.js`, it's more clear that way.


forget to change AuthorShip sorry @xfurry


I disagree, because you can totally remove blocks in Snap that are *in use*, by explicitly deleting their definition, and this will also remove all of their instances. Naming this particular menu entry "remove blocks" will confuse users into believing that's the *only* way to remove blocks.


This will be merged this evening ;)


Inclusion will just fix asserts in this file, no?

What about changing it to `#ifndef QT_NO_DEBUG` and

 * build with built type DEBUG
or

 * Make sure that RelWithDebInfo undefs QT_NO_DEBUG (or is having QT_NO_DEBUG defined the whole point of the RWDI configuration and there's some problem with this on mac)?


the cluster is still using the `140 - 3 * level` version at the moment.


The mask is not preventing a user from tabbing to the masked buttons/input fields. It might be better to add an opacity to the area and make them disabled through JS.


The opacity can just be set directly on the `signup-options-container` container


Ideally, each commit should be self-contained and work. It would be nice to squash (merge) this commit with the one that introduces the test.


Потому что в стабильной версии Spring Boot используется 3-я версия Spring Security.


No problem, but you should see these articles.
http://easyramble.com/tdd-chec-cookbook-with-serverspec.html
http://critical-alert.hatenablog.com/entry/2015/01/26/025414

You can test centos and ubunt at one command.


P.S. Если уж ты используешь  1.2.6 то и Spring Security тогда тоже используй третий.


It required some changes In the DNS configuration


Relleno de treeView con bucle de lectura


Nice catch


No new line


True Lua coders would do it like this:

```
    FindTechLevel = function(self)
        return EntityCategoryContains(categories.COMMAND, self) and 'COMMAND' or
            EntityCategoryContains(categories.EXPERIMENTAL, self) and 'EXPERIMENTAL' or
            EntityCategoryContains(categories.TECH3, self) and 'TECH3' or
            EntityCategoryContains(categories.TECH2, self) and 'TECH2' or
            EntityCategoryContains(categories.TECH1, self) and 'TECH1' or
            nil
    end,
```

I don't think "early return" should ever be used unless it's absolutely necessary to stave off death by indentation or massively confusing logic.


<title>Hardyweb Amp Page</title>


still flickering for me


actually patient image attribute is only used for Display images by default . so i thought that it would be every other category would be a new attribute type
@maimoonak


It looks like `type_type()` is meant to be `table_type()`


Haha I got schooled.


You can use `std::swap`.

http://www.cplusplus.com/reference/algorithm/swap/


Thanks for the fix Peter, I just hit the mclapply/bplapply issue yesterday.


lol :)

"The answer to that is that if you need more than 3 levels of indentation, you're screwed anyway, and should fix  your program."  From https://www.kernel.org/doc/Documentation/CodingStyle :)


Analyzer Tuner should work with any kind of 3a analyzer, decoupled tuner with Aiq Analyzer.


Why was UX - Interaction Design Specialization removed? AFAIK it is not free as a whole, but consists of courses that are free by themselves. Or is it just not good enough?


Looking...


if attribute does exits, then it creates it first


**README - Nitpick**
Commands should be written within back-ticks, i.e. `composer require burayan/evangalize`


back to basic


**README - Nitpick**
You could add `\`\`\`php` to enable syntax highlighting.


This line is probably not correct ;-)


it would take more time  or it might get hanged . is there any better alternative??


This will probably need plugin-infinispan80. See my later comment below.


this was a breaking change from 0.6.11, the search index can no longer be initialized as
```
var si = require('search-index')();
```
like listed in some of the docs but instead must now be initialized as
```
var si = require('search-index')({});
```
which is frustrating


This should probably also be added in on-rpi-boot.sh


Yes


Why mix await and continuewith?  ContinueWith is used when you don't have await available.  This should just be Try / Catch with await, I think that is more simple.


Oh, that was the cause for the derpiness. I was wondering for some days why they were broken, but I somehow didn't notice that this was related to the third batch of getTileEntity changes -.-


Just use await try / catch, no need for complex error handling logic with ContinueWith


ah okay, and no problem.


Can you change this to Infinispan80HotrodService please? (assuming my PR https://github.com/radargun/radargun/pull/279 is integrated) I need this so as to be able to test expiration events which are only available in 8.0


This is note from wiki
```
A list of domains that have a shared presence in the database specified in dbname. People who use multiple domains per profile can't use this feature anyway, so you'll want to set it to something like "_DISABLED_" in this case to avoid getting users from similar domains all mashed together. For multiple domains also known as multi-tenant calling 1001 would call all matching users in all domains. Don't use presence-hosts with multi-tenant.
```
So I am not sure that it should be `enabeld='false'` instead of `value= "_DISABLED_"`


I think he removed all of the specializations because it's too much material to be required in the curriculum.


ok


This can be a plain `char *`. The contents of `__FILE__` is always going to reside in static memory.


**Code Quality - Awesome**
This is good, going beyond what was required!


Tu laisses tomber les reformes ?


**Code Quality - Awesome**
Good application of _constructor dependency injection_.


Please rename these! Symbol names beginning with `_` followed by an uppercase letter are reserved for the language implementation.

I would consider something like `Z_MallocDebug` and `S_MallocDebug`..


`RAILS_ENV="production"` missing, right?


Sipariş eklendikten sonra bağımsız malzemeleri direk olarak işleme alıyor. Ama kontrol yapılmamış oluyor. Kontrol yapıldıktan sonra işleme alınması daha faydalı olabilir.


HOREEEEEE


**Code Quality - Nitpick**
The filename should be _ucfirst_ i.e. `GithubApiClient.php`


On est obligé de toujours mettre sa réforme dans le cache ? Si oui c'est un peu relou...


Я убираю определение этого бина:
```
    @Bean
    public SpringSecurityDialect securityDialect() {
        return new SpringSecurityDialect();
    }
```
но приложение работает, только если в явном виде указать зависимость:
```
        <dependency>
            <groupId>org.thymeleaf.extras</groupId>
            <artifactId>thymeleaf-extras-springsecurity3</artifactId>
            <version>2.1.2.RELEASE</version>
        </dependency>
```
Разве Spring Boot не подключает `thymeleaf-extras-springsecurity3`  автоматом?


Would it be relevant to include the fix in the original code (piletools?)?


Checkmarx scan results 08 Oct 2015 09:58 UTC +03:00

High
Medium
Low
Info

New
36
31
39
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

19a974971b960a93fd9db110981376e4a81c8bfb	@dudug


what is your reason for this? it is also a breaking change for 0.6.11


@rkday : Is it time for a new release?

There haven't been many commits recently, and I haven't seen critical bugs that require immediate fixing. (And there have been 270 commits since 3.4.1.)


Änderung vorgenommen.
Blubb entfernt!!


Checkmarx scan results 08 Oct 2015 09:59 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
36
31
39
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

3f348b8cf3177fdd35260d941fc6d2143cbb428f	@dudug


On Oct 8, 2015, at 8:44 AM, Sebastian <notifications@github.com> wrote:

> This was as long correct as there were no single values (non-dicts, as sysconfig['foo'] in the example above). The code was correct with this assumption.
>
okay, i can show you the stack trace then ;)

> I pushed a fix correctly handling dict and non-dict values.
>
> —
> Reply to this email directly or view it on GitHub.
>


hey @thiagopnts are u planning to update https://github.com/clappr/generator-clappr-plugin/ with these changes?


**Comments - Awesome**
Well structured `docBlock`.


cc @dayvson


"Massively confusing logic" such as the expression `EntityCategoryContains(categories.COMMAND, self) and 'COMMAND'` which is, in a sane language, is of type `bool` not `string` ;)


Each commit of refactoring makes me cry ~~


Ask Christophe why and how

Le jeu. 8 oct. 2015 09:09, adrienpacifico <notifications@github.com> a
écrit :

> On est obligé de toujours mettre sa réforme dans le cache ? Si oui c'est
> un peu relou...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/benjello/openfisca-france/commit/fd19a5e69618ebccf175b4b1dbd306e12d9845b2#commitcomment-13656043>
> .
>


Sorry, it must be lt_LT.inc, not lt_LT


Should I add a PR for this? I don't know how this kind of refactorings are approached.

This is also a quite general function and not all branches are covered by tests. Should it be extracted somewhere else?


Dass ist bloß Eine. Das rot markierte ist alter Code den ich gelöscht habe.


Riot is cool too :-)


Im ERM sieht man das es eine Beziehung Veranstaltung - haben - Gruppen gibt. Das heißt der SQL Befehl um alle Guppen einer Veranstaltung zu finden müsste so lauten: "select * from Gruppe where GruppenNr in(select * from haben where Veranstaltungsfach = fach)".


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


First point


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


I guess this works too :)
Although generally we use isinstance(style, basestring)


**Code Quality**
- See inline comments above.
- Your code is neat, following the PSR guides well!

**Functionality**
- It is satisfactory

**README/Comments**
- Well commented code.
- A little corrections to note on README, see inline comments above

**Testing**
- The test coverage is satisfactory.


See discussion here : https://bugs.limesurvey.org/view.php?id=7528 .

And @SamMousa is rigth here : it's not Question specific, then adding it to whole Question->attribute seams better.


This line isn't necessary as `map` will automatically wrap the object into a function


Der SQL Befehl müsste so sein: select * from Student where MatrikelNr in(select * from teil_von where TeamNr = teamNr)


Correct, fixed.


In Python 3 dict.keys() method returns 'dict_keys' object not a list.

Python 3 documentation here:
https://docs.python.org/3.5/library/stdtypes.html#dict-views


Primary Key wird denke ich "FACH" Sein da unique vergeben. Es gibt keine zwei Fächer mit gleicher Bezeichnung, egal wo


Wieso ist das hier unten? hmm


Will do.


Mean line 7, `.map(function (x) { return object; });` this one?


Hi, what is the ETA on this change to be available thru npm? Currently apps generated using xcode 7 cant be deployed on iTunes Connect.


@kwonoj correct


Not my code. SetCanBeKilled is already there, I just moved it when I rearranged things.


Thanks! I should've checked, will update PR as suggested.


Sorry, yes...the validator has a tuple returend...you want just the 1st part - the 2nd part is the error (or lack of hopefully!), so:
style = IS_JSONS3()(json.dumps(style))[0]


Originally had it in OnCreate. However, doing that meant later needing to check FractionComplete on the unit which dies. Doing it here means that only units which are finished ever get given the values for tracking this stuff, so I don't need the check. Cuts down on a check every single time anything takes damage = good?????


ооо, спасибо!


Die Auswahl sollte noch weiter limitiert werden, so dass nur aktive Ordnungspositionen ausgewählt werden könnnen.


Oh my, and I was wondering why those tests fail on Travis. Of course that `phantomjs` is not in `.`


hier müssen wir wahrscheinlich noch ein paar Argumente mehr an der Schnittstelle bereitstellen, Gruppe allein reicht nicht aus. Der SQL-Befehl passt nicht zum ERM. Wir müssten in etwa sowas haben:

select * from team where GruppenNr in (select TeamNr from enthaelt where GruppenNr = gruppenNr)

Dann haben wir schonmal alle TeamNr die in der Gruppe sind. Danach können wir alle Studenten mit:

select * from Student where MatrikelNr in (select * MatrikelNr from teil_von where TeamNr = teamNr)

finden (for-Schleife für jede TeamNr die wir vorher gewonnen haben.


Analyzer tuner should work independently with AIQ analyzer, decoupled it with AIQ.


For True Lua Coders[tm] it's perfectly readable and **the** way to solve this problem.


How to merge this commit into my fork? I have set upstream repository to https://github.com/bchavez/BugTrap.git and then follow the instructions from there: https://help.github.com/articles/syncing-a-fork/. In the last step (git merge upstream/master) I get many errors like this:

D:\Projekte\BugTrap-Source>git merge upstream/master
Auto-merging source/zlib/zlib.vs2008.vcproj
CONFLICT (rename/rename): Rename "source/resource.h"->"source/Win32/BugTrap/resource.h" in branch "HEAD" rename "source/resource.h"->"source/Client/resource.h" in "upstream/master"

CONFLICT (rename/rename): Rename "source/res/Upload.avi"->"source/Win32/BugTrap/res/Upload.avi" in branch "HEAD" rename "source/res/Upload.avi"->"source/Client/res/Upload.avi" in "upstream/master"

CONFLICT (rename/rename): Rename "source/res/SortArrows.bmp"->"source/Win32/BugTrap/res/SortArrows.bmp" in branch "HEAD" rename "source/res/SortArrows.bmp"->"source/Client/res/SortArrows.bmp" in "upstream/master"

CONFLICT (rename/rename): Rename "source/res/KeyPair.snk"->"source/Win32/BugTrap/res/KeyPair.snk" in branch "HEAD" rename "source/res/KeyPair.snk"->"source/Client/res/KeyPair.snk" in "upstream/master"

CONFLICT (rename/rename): Rename "source/res/ImageToolbar.bmp"->"source/Win32/BugTrap/res/ImageToolbar.bmp" in branch "HEAD" rename "source/res/ImageToolbar.bmp"->"source/Client/res/ImageToolbar.bmp" in "upstream/master"

CONFLICT (rename/rename): Rename "source/res/Checkmark.bmp"->"source/Win32/BugTrap/res/Checkmark.bmp" in branch "HEAD" rename "source/res/Checkmark.bmp"->"source/Client/res/Checkmark.bmp" in "upstream/master"

CONFLICT (rename/rename): Rename "source/mkdoc.cmd"->"source/Win32/BugTrap/mkdoc.cmd" in branch "HEAD" rename "source/mkdoc.cmd"->"source/Client/mkdoc.cmd" in "upstream/master"
CONFLICT (rename/rename): Rename "source/XmlWriter.h"->"source/Win32/BugTrap/XmlWriter.h" in branch "HEAD" rename "source/XmlWriter.h"->"source/Client/XmlWriter.h" in "upstream/master"

Here is a screenshot with a fresh cloned repository:
![bugtrap merge problem](https://cloud.githubusercontent.com/assets/8649051/10360235/743ab00c-6d9f-11e5-8a92-8024c377e45d.png)


@TomasVotruba in 2.4.x (and before) the discriminator map didn't require abstract types.


Sgtm!


Hi Ahmad,

cool, dass das so schnell geklappt hat. Ich habe oben Code ein paar Zeilen markiert und kommentiert.
Wäre cool, wenn du das nochmal korrigierst und mit dem nächsten Pullrequest wartest, bis ich die POJO-Klassen hochgeladen habe und deine POJO-Klassem noch einmal löscht. sonst haben wir naschher alles doppelt und dreifach. Ansonsten finde ich es richtig gut, dass das über Gihub so gut läuft.
Besten Gruß
Jan


Given this commit, I think git blame won't work correctly after moving a file. Author tags at least have some written record of it (though I prefer auto author tags, but either way).


Lgtm


No, I believe it doesn't, because it's optional dependency and not all the Spring users need it.


Hey @Tatipiment! If you haven't noticed it already: This account is now owned by a different person. :blush:


Change `edit-mode` to `insert-mode`


Вообщем чтобы этого добиться нужно много переделывать в самих сервисах, так как там в handleChatMessage все notification message обновляют текущий диалог с параметрами, которые берут из того самого messgae.dialog


@berak those *.pyc files, did you actually change anything in it? Because I cannot open it using a text editor without seeing rubbish


Change $baseUrl in TestCase


T'as fait comment pour trouver les bons montants ? T'as juste calculé CSG non déductible et CRDS en plus ?


Спасибо что не на Чешском :)


Ça change quelque chose de tout mettre sur un plutôt que de repartir sur les deux pour les retraités ?


wtf happened, github desktop is weird


Hello World Only


Oh... they are all packed into full name... not sure if I need either of those.


You're welcome ;-)


typo. Please change "and then transmits" to "and then transmit".


One more close parenthesis is needed.


Autrement je crois comprendre plus ou moins les contraintes, mais on pourrait faire un backward_revenu_net_to_imposable.py , ça pourrait être utiles pour certains test ?


How about using "replica" instead of "partition"?
In contrast to model partitioning in which each evaluator works on a partition of the whole network model, in data partitioning, evaluators are NOT partitioned, but just train their own partition of input data.


typo "and provide new parameters".
And, it will be better to add the link for 'Parameter provider'.


Please remove the article 'A' and use plural for the consistency.


Using the example code is good. But, is the previous format better for the consistency with other explanations such as 'Common Fields'' and 'Layers'?


It'll be better to add the link to `dolphin-ps`.


typo "also contribute to"


e.g.:
broomweed 12:37 AM @mettatiny take my foot
mettatinyBOT 12:37 AM I'm now holding broomweed's foot
broomweed 12:37 AM @mettatiny take a piece of candy
mettatinyBOT 12:37 AM I'm now holding a piece of candy
broomweed 12:37 AM @mettatiny inventory
mettatinyBOT 12:37 AM I'm holding broomweed's foot and a piece of candy
broomweed 12:37 AM @mettatiny drop my foot
mettatinyBOT 12:37 AM Dropped broomweed's foot
broomweed 12:37 AM @mettatiny inventory
mettatinyBOT 12:37 AM I'm holding a piece of candy


Removing `Hadoop NameNode` is better since `Hadoop NameNode` does not direct requests to `DataNode`, but instead lets clients know information of `DataNode` for they connect to `DataNode` directly, as far as I know.
I know I introduced `Hadoop NameNode` concepts. Sorry for confusing you.


@jsjason This looks good. I left some comments.
Also, resized images are too small. It is hard to read texts inside the images. Please resize them larger.


Christophe is not very reactive this days... Je crois que c'était pour accélérer le build travis cette histoire de cache.


missing one bool?


no else


char[] isch eigetli en technischeri bezeichnig für en String. es macht aber meh sinn, char[] nur innerhalb vo de methode zbenutze und als return wert String zmache. String isch eigetli de standard für text wert, wo vollständig im memory sind..


@dkayiwa I am setting it to the exact same as a name that is already voided. (The name object above.) Make sense?


All bools are in place :)


Well, this one ought rather to stay a cout as there is a cin just bellow related to it. Anyway the interaction scheme is to be changed in the end.


if( isEnum )
   type = uint32;
addValue( .. )

for line 48..56


poor me...How did I missed it :(


any news on this patch ? a few of us have had this problem on older systems still in use. Is  there a mailing list on which I could send this patch ? or should I  open a github issue ?

Thanks,


idem above


indentation and/or line length


I understand u kept the patch here as part of the 3 buckets (upstream, ARC, eZChip platform). However it seems we need to add this - as an optimization - after EZChip platform introduces it's cpu_relax() - NO ?

Also with your change u r adding a barrier inside the loop - which might not be desirable for other platforms. FWIW, a failed trylock need not be a barrier - I need to check that again and get back to you !

Anyhow this patch is contentious and needs to go at the end !


@t4deu Thank you :smile: :+1:


Either the whole list of these checks are unnecessary (especially the a7lte one), or do not set TARGET_BOARD_PLATFORM in BoardConfigCommon!


The minotaur will be lose power because of the new bosses


wahrschinli hesch throw new ServiceException mal bemene code snippet vo mir gse. eigetli ischs aber so, dassmer i jedem fall en exception us em BusinessException-Tree verwendet. Also i dem fall müsstisch en neui Exception mache, wo vo BusinessException erbt (oder vonere generische exception unde dra). Ide TypeHierachy im eclipse (rechti mustaste uf en bestehendi exception im package model.exception, denn show in type hiararchy) gsesch euse exception tree.

Also machsch da am beste en Exception QrCodeGenerationException wo vo BusinessException erbt und di glich struktur het wie z.b. en ExpenseDeleteViolationException. und lueg na d type hierarchy vo ExpenseDeleteViolationException ah fürs verständnis vom exception tree


I guess it would be cleaner to create a map on top of the file

```js
var env = {
    CI: true,
    TRAVIS: true,
    CONTINUOUS_INTEGRATION: true
};
```

And iterate through it when building the args. This way we can also support defining your own environment variables in the future. http://docs.travis-ci.com/user/environment-variables/#Defining-Variables-in-.travis.yml

But we can always refactor later on. Thanks for adding this :).


Oh the type_id is the "index" number. Yes, for printing purposes probably neither is needed.


Yeah changed it already


``````
attr($pid)


Good Job


Test comment


Blabla


This change should be undone. It is a regression on the computation of the travelled distance.


=스켈레톤이 쏘는 화살에
유저의 접속이 풀리는 상황을 임시해결


yes


Should not be using sqlStatementNoLog(). That is only used in special cases to skip logging. shoud use sqlStatement() .


It is not longer than your line 30 ! ;)


Happily we're all C-programmers :+1:


Also can be more concise be selecting just lname, fname, and mname in the query.


This commit deleted the `api/diff.txt` file.

Please make sure when you commit something or review/merge a PR that this file is updated.

BTW, have you installed the commit hook into your repository? If you did I'm not sure how this could happen.


für de type png gits sicher en java konstante?

z.b. git MimeTypeUtils.IMAGE_PNG_VALUE en string mit image/png zrugg


I just realized the two functions above are the same with different names. So, would remove one of them.


Je suis à la bourre mais tu peux finir le travail ;-)

Le jeu. 8 oct. 2015 09:06, adrienpacifico <notifications@github.com> a
écrit :

> Tu laisses tomber les reformes ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/benjello/openfisca-france/commit/fd19a5e69618ebccf175b4b1dbd306e12d9845b2#commitcomment-13656016>
> .
>


This hardcoded IP is needed for obtain IP addresses for all AWS nodes. Read here -- http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/using-instance-addressing.html


Yeah, good point, I think I need a coffee.


``````
xl('Unknown or N/A')


@jamesallsup Do we get a quick hotfix release of `OC 2.1.0.2` or do we have to wait for the next big release?


ok


I actually think `ReactDOM` thing is looking weird :-)


Also can be more concise be selecting just title in the query.


I don't think the default should be to snap geometries to a grid. I now of corporate users that would be very disappointed by their geometries being modified with no notice.

Also it might be wise to use an interface that allows for specifying whether or not the PrecisionModel should be changed as that's what will happen by using the upcoming GEOS version (which might be still improved to allow for also requesting not to change PrecisionModel, now that we talk about it).


you need to add an overview section in read me abt you project!! then other :dancers:


Since the outer for loop does not have any state it has the same effect as a goto, or am I missing something again?
Also what about
http://play.golang.org/p/Q5E3mePOKA


Wow!
Nice Job bug hunting
Thanks @jan-hudec


Fix the `@constructor` definition to match the exposed function name, in this case `@constructor TA.TimelineAction.Start`.


"This allows ...?" -- this branch's docs are live only after 5.2 is released, I think? (Part of confusion here might be that I was trying to read the tenses as though 5.2 were already with us.)


Was about to submit a PR for this issue but see it's been fixed on master - so thanks!

Not sure it matters but the `m` option has never worked as it should have been:
- Part of the `q` hash rather than a separate query param
- The value must be lowercase 'or' not 'OR'

As it's redundant is it worth removing this line?


It's not really necessary to attach `TimelineAction` to `TA` at this point. Just use a local variable and attach it at the end.


The `/opt/jboss` dir is needed what for ?


There is still a significant ...


When click on Ledger in the Patient Summary screen, does it make sense to run the report automatically with the default dates initially rather than need to click Submit?(since users can select the default date settings)


typo: `privileged`


Try and error


they are good, the comment here was just useful to copy paste something.

I've adopted a convention like: "less than 20 = admin, more or 20, Receiver", are defined in the top of globaleaks/utils/logger.py

thanks for the check!


attr()


Crois pas


@gaearon do you mean with ReactDOM or without it ;p


It is redundant indeed. It is worth it : )


How does this show up?


this was part of the change done for the convention. Instead of using incremental numbers (and mixing Receiver and Admin messages), now is easier extend one list without having to change all the following numbers.

For example, if you've to create another admin event, you append LOGIN_4 without changing number of the others before _20


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


@kizniche I try new note system and get this error:
PHP Fatal error:  Call to a member function bindValue() on a non-object in /home/pi/Mycodo/3.5/incl
udes/restricted.php on line 2620, referer: https://192.168.0.10/mycodo/?tab=data


attr() $type_form and $form_pid


@knowbody

I think this is the way forward considering ES6 supports named imports:

```js
import { render } from 'react-dom';
```

That we even *had* `import React from 'react'` is just an artifact of it being pre-ES6.


It is better to have meaningful text for links. Words such as "this", "code lirary", "article" are not good from SEO perspective, and do not help the reader understand where they will go by clicking the link.


Blank lines?


it's fixed now


Fehlt da nicht das + für die string concatination ?


Ah, I guess I should finish writing that docstring...


Incorporate the new print log code here. See one of the report examples in this commit:
https://github.com/openemr/openemr/commit/9c4a08cc76809bf8b0b19ad73e0eaa1c0cb3ae1b
(note your code will need to be based on most recent codebase for this to work)


Для чего id?


This appears to be a full path nowadays, which seems slightly silly
http://php.net/basename


This is CI startup options, it shouldn't be passed to the DB at all. Only the build result struct is sent to DB, and should be built from within the CI.


text()


I tried to use Try / Catch first, but it completely killed the performance tests for some reason.

However I also discussed this mixing with our resident async expert, going forward we will no longer mix await & task, but go for returning the tasks and keep those tasks hierarchies linear (no more conditional OnlyOn...)


указывай name а не id


Я бы все-таки оформил все это таблицей


Hi Ralf,

Ob es einen negativen Einfluss gibt, wenn die 3 zwei Mal vorkommt,  weiss ich nicht. So wie ich das verstanden habe,  ist es nur zum Festlegen Einer Reihenfolge.

Wir können die Reihenfolge beliebig ändern denke ich. Jetzt weiss ich auch,wofür man das einsetzen kann.

Bezüglich Fingerprint. Eine Doku konnte ich nicht finden. Nur was in der Fhem.pl dazu steht.
Ich habe es so aus Beschreibung und dem Code verstanden:

Die Funktion Dispatch über gibt den Inhalt von dmsg an eine checkdup Funktion. Diese ruft die Fingerprint Funktion des Moduls auf.
Der Rückgabewert wird gegen einen Hash geprüft.
Ist der Rückgabewert bereits vorhanden und nicht älter als "duptimeout" wird er verworfen.
Andernfalls wird er verworfen.

Das sollte doch den gleichen Effekt wie minsecs haben ?  Duptimeout ist aktuell ein global Attribut.

Grüße Sidey


[CodeNarc](http://codenarc.sourceforge.net/) is the main tool that people use. There are plugins for Grails and Gradle (probably Maven too).


attr


Hi.

Using this patch the added tests pass but I've found other test that doesn't work.

assert.equal(jDataView.Int64.fromNumber(-1618194002072370894).toString(), '-1618194002072370894');
AssertionError: expected '-1618194002072370944' to equal '-1618194002072370894'

Regards


squash this with patch that introduced all this?


So delete it.


Yeah, that is slightly silly, it would be better as just the filename itself.


Do we really need to explain this?


Extra blank line detected.


I think we don't need database create, because schema:create do the same + generate db schema


А здесь использовал какие-нибудь более легковесный (в визуальном плане) кнопки. Вроде так, что в списке транзакций используются, без темного фона


то же по поводу id


Review complete.


Ok, I wasn't sure what it was used for before, so I didn't want to remove something that might be used in the future, but if it's fine to delete then I'll do that.


@enlait , @elilev3 is totally right. Specializations will be chosen by the students. We'll keep this curriculum as short as possible.


We have the power of git, it can be resurrected at such time.


I think that GitHub (and text editors) prefers spaces than tabs.


invalid_attach ?


I know I would probably ask why someone is using hasattr instead of catching AttributeError when seeing this code. I even used to use the exception myself in the old code (since it'd be indeed cleaner if there wasn't the drawback of handling an exception from inside the method).


используй автоинкремент https://www.sqlite.org/autoinc.html


I suggest a comment like this:
Interceptor adding the full request url (with parameters) to the intercepted request. The url is kept as a <b>requestUrlWithParameters</b> request variable.


Good point, I'll change it then.


Oops. I meant to fix that. Thanks

On Thu, Oct 8, 2015 at 2:11 AM Chris Forbes <notifications@github.com>
wrote:

> invalid_attach ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/engineers-nightmare/engineers-nightmare/commit/aa9d3fe0fec143ac077636066c6c1a78340b69e6#commitcomment-13656935>
> .
>


Does it make sense to just use Patient Ledger in the name?


Leave a comment coffee ~~this~~ NOT cat


Тут намешан ui и логика разнеси по файлам


Oops. Lost my comments because of the commit. :)

So, yeah, you're right. If we'd like to strictly follow the specs, that's the way it should work.

Btw that would be really great to know what was the original motivation of @sindresorhus when he created this module and what he uses it for. Waiting for his response...


Actually, disregard above comment.


Why can't we check for does Exist under maintenenace ref as well, be fore creating it actually ?


Adds a client request url string (with parameters) to the request as a requestUrlWithParameters variable.


Это вообще пиздец, разноси


you can use `hasOwnProperty` here. Check line `3`


разноси


this rank calculation is already done in another API.

all this method should be replaceable with simply:

```python
        for r in api.character.rankadv.get_all():
            tech_ = api.character.schools.get_tech_by_rank(r.rank)
            if tech_:
                rules_.append(tech_.id)
```


You are right. Some how my formatter does different things when using two different shortcuts, although it shouldn't. Gonnna change this back.


This was nasty, but it was the only way I found to overcome the problem of variable scoping.


That's gonna get old pretty quick when it will be used more. Maybe it's time to thing about alternative solutions for secrets... https://www.vaultproject.io/ seems interesting, but it would need an Ansible lookup plugin to work with DebOps. But its a debate for another time.


@vilchik-elena yes so I tried something let me now what you think of it.


I agree that things must be prioritized. Like getting your education, work as expected of you, etc. (wait a minute, am I too old or what...)
With this bat cave project and the to do list you mentioned, I would think that the sensor handling may needs to be redesigned to allow e.g. for sensors grouping, may be more than two of those. There is good saying for this in Hungarian, I guess it is true for any scientific experiment: "Egy mérés nem mérés, két mérés fél mérés, három mérés egy mérés". Don't try Google translate (or try if want a good laugh): On measurement is no measurement, two measurements equal a half measurement, three measurements equal one measurement. I'm sure there is an English equivalent, but yo see my point.
So if this is the direction of your development, please keep in mind my suggestion on reordering, only if it fits, of course.


I checked yesterday for the same purpose, it already seems included (I
didn't verify by running piletools on the mpileup files, though). I
preferred keeping the same script as we had always used, as the fix is
pretty minor (but slightly differently implemented).

On Thu, 2015-10-08 at 00:09 -0700, Martijn Vermaat wrote:
> Would it be relevant to include the fix in the original code
> (piletools?)?
>
> —
> Reply to this email directly or view it on GitHub.
>
>


@vilchik-elena indeed.


maxSize is used to calculate the size of the output. The cipher will output encrypted data whenever the buffer goes above the block size. We first round ulDataLen down to the modulus of the block size. We then add a block to the output if any previous remainder in the buffer and this remainder goes above the block size.

If maxSize equals zero then ulDataLen is less than the block size and the cipher will not produce any output. So I do not understand why we need to round it up to the block size. The only case it would produce an output in this case is when there is data already in the buffer and this remainder makes it go above the block size. But we have a case for this below.


Thanks, @adrianbk :)


Can someone please latest library version to NPM repository?


fixed


The unique ID for groups are now a int. If we want to get the group from the DB we then need to convert the name to a int and then look it up, instead of just looking it up.


good job


2573 KB/s (39531 bytes in 0.015s)
        pkg: /data/local/tmp/intall.apk
Failure [INSTALL_FAILED_INVALID_APK]


Yes I suppose the precision to be used could be exposed in the settings. My experience is that by the time geometries come to GEOS (after reprojections etc etc), the geometries are already altered to a point where snapping only helps.


so when I pull your change to NN_SURVEYOR and NN_RESPONDENT the survey test in the examples directory mysteriously stops working.  I think I should be using the right system version of nanomsg beta.

can you double check, as I don't know if the change is correct.


Do we really need the __ here? We are already inside a namespace.


@Urigo your missing the chat-detail.scss file


ok. this was mostly a comment to check later. so if it does not work well, then we don't change it.


Will using does exist on mmodeRef make sense, like double check locking thing.


@vilchik-elena ok!


ah ah ah, just this.... So it must match the name defined in the file.


Det är lugnt med app ID! :) den är publik ändå!


Looks awesome Andrii, nice clean and simple.
should this not be a separate new test instead of modifying the existing test?


ok. got it.


D’accord avec les deux.

Juste un point pas clair pour moi :

> tu fais ta propre réforme

Qu’est-ce que ça signifie exactement ? Que tant qu’on n’est pas sûr à 100% de la date de début d’un coefficient, on l’isole dans une réforme ?


grunt-contrib-sass est plus stable.

J'ai pas l'impression que le css a été minifié par contre options: {style: 'compressed'}


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


instead of @date+1, you could have

    1.day.from_now(@date)

it's way more clear


Fixes #904


Yep, thanks!


I would use the following pattern for all ```export = ``` statements:
```
import cloneWithProps = __React.__Addons.cloneWithProps;
export = cloneWithProps;
```

If you inline ```export = ```, the compiler will export only the variable and not it's type so you won't be have access to it unless you use the global namespace.


I'd remove it, at least for now. I have only added it because I had seen in other form-related code. But it is in fact not used when browsing with IE. I think the details of how to visualize errors in form should be added later, once the actual form is created. And this class is maybe not the place where such things should be done. And I think showing errors with some js/css would be the way to go instead of using HTML5 validation.


yes it was a find and replace bug, the way AMD works is that the script must define the name of the module and for some reason i missed it during the refactoring of the client code, for 3.2 we should export `vertx-eventbus` to keep the same name as the other loaders (commonjs and webpack)


Here's a comment...


It does not crash on Ubuntu, but maybe there are other systems that do crash. What system are you running on?

Maybe this is just to avoid issues from potential memcpy implementations that touch the memory area even when the copy size is zero.


Switching the DB encoding to JSON will make the current live DBs unusable for the new implementation. This either means we cant update the live instance of AG with code from this branch until the semester is over, or we need a migration implemented for it to go from gob to json.


Why remove it?


The website's link ( http://download.magephp.com/magallanes.latest.tar.gz ) still leads to the unpatched version


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


So this fork is for BIG ENDIAN only ?


Here's another comment on this thing


klingt interessant :)
Viel Glück!


And another


Because it's ok for a file to have no parts on the storage, just metadata


OK then, so I guess I'll try to make it work in MariaDB. :-)


@vilchik-elena it is mainly for readability, as both are used and argument multiple times, it avoids to store it to a local variable or to have 3 times a chain call of getters.


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


I'm looking forward to the next release series as Brice has been kind enough to address many of my growsing complaints. @ggouaillardet I'm thinking it would be best if we create opal_hwloc_base functions that abstract the impacted functions, ensure that only those are used throughout the code base, and thus we concentrate all the version #if's in one place.


I followed the current style of test, that is written in a dirty way without test runners with just using `process.exit(0);` and global timeout)


Tnx dude ;-)


Prints through fmt to separate it from log. This is not important log material, and was on fmt to not let it be printed to files if the log location changed.


viel gück!


woooow krass voll gut!


I know. The gob version is easy to activate again (two lines commented in score.go). But we will want to switch to something that is faster at some point, e.g. for next semester. Currently, I think there may also be other incompatible changes with the `NextSequence()` thing as well.


good job


Hey,


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


This particular line would no longer be a problem as I intend to remove `setCustomValidity` calls but making such messages i18n-able is definitely the thing to keep in mind. I think as long as only messages such as "Field invalid/Value missing" are to be displayed, the translation could be done on client-side. But if we wanted some more detailed error messages (e.g. what is the expected format of the value in the given field) than the message would probably come from the server-side validator?


Lass dich überraschen wo dein Weg dich hinführt kannst du ja mit deiner Gruppe dann festlegen.


@vilchik-elena no it not. PHP fails with the error: ``fatal error: Cannot redeclare setlocale()``


```
if (!cookieSessionId || !store.local) {
    // может так?
}
```


All right :-)

I have only MariaDB running and so I need to change every role to `debops.mariadba` ;-)


braucht man die Zuweisung? es wird ja bereits vor der schleife der double y = 0 gesetzt...


First I like the idea ... but:
I'm not sure it's good idea to check all the errors at the same time. If there are 20 calls and the 5th fails it might be hard to figure out that it was the 5th not the 3rd. And chances are that in that case most of the following ones will fail either way.

Maybe [this](https://github.com/ironsmile/nedomi/commit/bafe0e3d37dbe9e5cfb6fab9f7f7396508b3ac17#diff-b48018d21ffa1ff272a5cb589fb69ef5R16) could be moved in `testutils` and used (with some changes) as well as making a similar for `Write`.

Versions with `Fatalf` and `Errorf` for both?

Maybe we should try some already done assertion library ?


nice nice, good Luck!


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


`:input` also selects `<selects>`. From https://api.jquery.com/input-selector/
> Description: Selects all input, textarea, select and button elements.


License is spaces instead of hyphens.


ah ... yes I agree


hum... this is breaking the API, no ?

I think we should change the name, but write it down somewhere (so we don't forget)


Here @andreaturli


Klingt spannend, aber auch etwas kompliziert. Bin gespannt wie das Ganze umgesetzt wird, viel Erfolg! :)


The correct way to use the docker connection is to attach to the container through the implementation below. Each time this was attempted earlier, the container got stuck. This implementation is still here for use when docker has fixed its issues and we can do it the correct and faster way.


naja man muss ja nicht gleich so gut wie locoroco werden aber ne vereinfachte version wieso nicht :)


sag bescheid wenn mans spielen kann :) Bin gespannt!


Could we make this configureable?
On 8 Oct 2015 07:51, "cyberium" <notifications@github.com> wrote:

> Oh i have to revert power part then :(
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cmangos/mangos-classic/commit/285a800e9d443c4380b218a87a2df0e288542ce4#commitcomment-13655261>
> .
>


Great. Then we leave it be, perhaps adding a comment to this effect.


Klingt interessant. Viel Erfolg dabei :)


No Problem, I translated it to english.


Lets start a wiki page with API changes for 3.2 like you did for 3.1


hmm und wo bleibt der Spaß mit Freunden und Bier? Bin gespannt auf die Umsetzung!


Don't leave the blank lines.


@gossen Shouldn't the $var-variables be declared as discrete?


This is used for giving the timestamp for when the build was executed. Used a lot in html files to give information out to the users.


You should summarize the changes in the first line of GIT commit messages.


Süße Idee. Das weckt Kindheitserinnerungen :)


Yes,

we also need a 3.2.0-SNAPSHOT branch for the examples.


I was just bitten by the same problem (luckily just on staging, not production). Might I suggest to follow strict semver? This is a breaking change, so should've been a major version number increase (2.0.0). To quote [semver.org](http://semver.org):

```
Given a version number MAJOR.MINOR.PATCH, increment the:

MAJOR version when you make incompatible API changes,
MINOR version when you add functionality in a backwards-compatible manner, and
PATCH version when you make backwards-compatible bug fixes.
```

A changelog would also help a lot.


Ein modernes Tamagotchi wäre echt klasse :D Coole Idee! :+1:


You should specify the length of the keys you are about to use.


TeamCity pull requests :: netty [Build 383](https://garage.netty.io/teamcity/viewLog.html?buildId=2135&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5618, ignored: 21 Build time: 00:40:50


Yes, the customer autoloader is a pain in the ass that I should have get rid of long time ago. Let's PSR everything. If you want some help on this maybe we can create an issue (or several) and split the work between us.


Use `uint32_t` or `int32_t` to replace plain `int`.


<3 ich hoffe ich kann bald meinen eigenen haben :)


sure !


> I've made it better (i think) yet again.

Nice! Still at http://codepen.io/anapsix/pen/epWwyr, or elsewhere..?


The idea of `ShouldntFail` is that it will be used for calls that return an error but are very unlikely to fail. So in the unlikely event that something fails and it's not understandable exactly what just from the error message, someone can go and modify the test to find out.


Although GitHub redirects properly, that is not a clone URL.  I think https://github.com/rhysd/Trendy.git is better.


This should be changed, yes, and its also Go specific. It is only for detecting the number of failed/passed tests and as long as the teacher use score object it wont effect the score.


@levackt +1 I like this approach :) .

Actually if we detect RPi device, with 2947 port open, we can automatically assume that tests should be executed. We could use `RUN_GPS_INTEGRATION_TESTS` only to enable tests when no Rpi is detected (i.e. you run tests against GPSD on your local laptop for example) or to disable tests even when RPi is detected.

BTW I think you can use our `io.rhiot.utils.Networks` to detect if given port is open.


Revert to Hunan, Zheng Zhou.


before_save { email.downcase! }


Da kann man ja nur auf die umwerfende Storyline und Umsetzung gespannt sein :D Coole Idee


Klingt gut, kann mir aber auch vorstellen, dass das viel Arbeit ist. Viel Erfolg auf jeden Fall dabei!!


Coole Idee und so :D :heart:


how to test


@Download sorry for the misunderstanding. Probably you should state on top of this list "Dependencies". Thanks.


I have inserted a dbg call in line 434 of the current master to count the average number of safe checks by Queen when at least one such safe check exists:

   // Enemy queen safe checks
   b = (b1 | b2) & ei.attackedBy[Them][QUEEN];
   if (b)
        {
            attackUnits += QueenCheck * popcount<Max15>(b);
            score -= Checked;
            dbg_mean_of(popcount<Max15>(b));
        }

And it's possible to also count the average number of safe Rook, Bishop and Knight. Here are the results after running a bench of depth 16:

Queen : 2.14
Rook   : 1.17
Bishop : 1.01
Knight : 1.12

The 1.01 result for bishop shows that the popcount in line 449 almost always evaluates to 1, so maybe it's possible to simplify it? And similarly, maybe we could try to get rid of some of the other popcounts, using the averaged values instead?

Just ideas,
Stéphane


I didn't really check, but it seems the entire 7-series (artix/kintex/zynq) share the same "clocking architecture" since there is only one Xilinx PDF called "7 series clocking resources" ( http://www.xilinx.com/support/documentation/user_guides/ug472_7Series_Clocking.pdf ).
I'll have a closed look.


Echt coole Idee, kommt mir irgendwie bekannt vor :P


That's why it's always `Fatalf` - we never expected the thing to fail. I am ok with having `read` and `write` helpers in `testutils`, just not in this commit - this was just to fix all the `errcheck` warnings so the metalinter output can be more useful


If you need help in testing, you have my hand ;)


Die Verbindung zu einem größeren Projekt finde ich sehr gut und sinnvoll :) "Farbfresser" klingt nach einem interessanten Titel. Viel Erfolg dabei!


Ich würde es auch gerne spielen :D


If the Score struct is changed at any point in the future we need to remember to also update this function.


You can rewrite this without using `findDOMNode`

```
var fileInput = this.refs.fileInput;
```


Yes I think these can be removed.

There is no need to implement a function that only throws an exception, UNLESS the function overrides behavior in a base class.


Ah, but IODELAYE2 does not seem to be in the clocking resources... my bad.


Ca veut dire: dans le cas limite où tu trouves que c'es trop lourd de faire un changement qui n'induit pas d'erreur dans un passé que tu ne maîtrises pas parce que que tu n'en a rien à faire du passé, ben tu fais ta réforme chez toi pour pas déclencher des effets de bords sur les autres. Je pense que le niveau 2 est acceptable pour tous. Mais bon si vous êtes très pressés vous n'êtes pas contraints, vous pouvez faire votre propre réforme ;-)


These variables are regular clocked variables.
I think that all non-continuous clocked variables should be treated as discrete.


This changeset does much more than just "Includes tests for both incr and decr". The refactorings should be on a dedicated commit.


Ah yes, I was wondering where to put it, was thinking on a misoc branch but in the target file it is fine for now :)


@sesom42 Your fork contains commits that have been reverted here, so in order to merge without conflicts you should revert your fork to `85c8c2256d38ff9a49d18ef9e8fb19aaa`.


Do we expect this to ever be ``False``?


This should be inside the if: ;)
(won't hold up merge)


Don't mix French with English.


D'accord, c'était trois solutions indépendantes et non une gradation (1 puis 2 puis 3), c'st ça ?


Since we're testing the result later, I don't think we need this any more?


Something I needed multiple times already, thanks!


I prefer a trailing comma & then close bracket on next line (more easily extensible)


I agree with the argument that it's unlikely to fail and the test could be changed in the unlikely event. I still prefer to not change tests to find out why they fail, fortunately I came up with a solution:
We could print the index of the error that occurred in the errors list.


Add a category to your commit message ex. Front-End login.


Missing ``(object)`` here.


isRecentHeartbeat -> isHeartbeatRecent


No. XS does that automatically for you. Check the C.c file


Переименуй QMPopoversFactory в QMViewControllersFactory


`python3-javapackages` dependency is fine. XML could be parsed by [xml.etree](https://docs.python.org/3.4/library/xml.etree.elementtree.html) - I know the module is little bit clumsy but no additional requirement is needed then,


Please provide a more descriptive readme


@13307130498
@NoobCak3
@LimYH
do pull the lastest code as the structure and afew class is wrong


Hmm, what is the actual exception that's thrown?

If this stays, then need to add braces.


Aha, I think it`s because my notes.db size is zero and no tables there...


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


Before using `uint32_t`, you have to do `#include <stdint.h>`.


J'ai mis à jour mon commentaire ;-)


:+1: fix incoming :)


@klauszhang this is how to search over multiple associations. Not very intuitive, thought…


asd


I'm not familiar with it.
I need more time to get used to it  ..............
Thank you !

2015-10-08 13:39 GMT+08:00 Jim Huang <notifications@github.com>:

> You can use git rebase.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/poa1006/phonebook/commit/268773c855d9ecee64c1464b9c7fad7677e8457e#commitcomment-13655198>
> .
>


Great! Thank you guys.


Please remember to remove information of merge conflicts. <layout


I sent a direct email to @dcrowell77 to follow up.

@legoater - I think Habanero might have been intentionally disabled, but we might be able to re-enable Palmetto.


LGTM


;[,oknm


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


Could you format this like https://github.com/kraftwagen/kw-itemnames/blob/master/entity/kw_itemnames_entity.kw_itemnames.inc#L14-L29


bbbbb


Will this work if I exclude a foreign key while doing nested `include`?
I'm getting `ER_BAD_FIELD_ERROR: Unknown column 'foo.bar_id' in 'on clause'`.


> things like displaying the output mode (p or i) next to the resolution

I said earlier that I have plans for it. Patience.


I think we should just merge this while we're figuring out HPC-GAP performance.


It works for me, thanks


testing


Проверяй completion на nil!


Done: 391dbc6.


Pas vu…


Have you tried [toJson](http://dlang.org/phobos/std_json.html#.toJSON) with pretty=true?


if use nanomsg-0.6-beta:

./test_survey
core.exception.AssertError@survey.d(21): Assertion failure
----------------core.exception.AssertError@survey.d(47): Assertion failure
----------------core.exception.AssertErrorcore.exception.AssertError@@survey.d(survey.d(47
)./survey() [0x45dd4f]47: Assertion failure):
----------------Assertion failure
----------------


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


[Uploading 带线程库的编.txt…]()


这里用到 Action Book Haoduo 这几个组件，但是前面没有定义，所以是有问题的，需要添加几个 require 语句进来才可以，https://github.com/happypeter/qd/blob/master/src/components/App.js#L1-L5 类似于这里这样。


@flexiondotorg
Yes, i see this warning too in .xsession-errors


It still does, as the Req.JS error has a meaningful message set. I started working on this based on real issue that I had no idea why some test fail because all I knew was that "plugin could not be loaded". Req.JS error contains all the necessary information.

Anyway, I'll add one more test, for a plugin which code threw. I could also check a message of this specific error, but I'm worried that with future versions of Req.JS this message may be changed. I'll think about it.


I love :coffee:  :pizza: ,and :dancer:


Well, it almost worked. It run on my laptop before getting a null reference exception (as expected) from not having an accelerometer, but still has the deployment issues on my surface.


另外，App 本身也是一个组件，所以最好在另外创建一个 main.js 把 React.render 语句写到那里面。类似：https://github.com/happypeter/qd/blob/master/src/main.js


dsdsdfdfdfd


Je viens de le faire ...

On Thu, Oct 8, 2015 at 11:08 AM Matti Schneider <notifications@github.com>
wrote:

> Pas vu…
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openfisca/openfisca-france/commit/060483f078c13681ab06b9b3597464c8b23ae176#commitcomment-13658015>
> .
>


:question: Why is the address limited to 70 characters


@zachguo Depends on the type of include, it could break in certain cases, especially if there's a subquery involved.


Tu as entièrement raison, j'ai été négligeant…


Keeping the comma. :)


This means I'll have to modify the code before I can use it, i.e. `dub fetch digitalocean_api && dub run digitalocean_api` won't work. It would be nicer to pick this up from the environment and/or read a config.json.


Unused import.


Should the user also have permissions to delete the document from the current collection in order to move it to another?


Rephrased, albeit slightly differently.


Maybe. I tried that many times, but it always seems to revert back to Chris Searle for some reason;) I´m also not sure how that will work if we want to use features that requires app-specific certificates.


Please show mean example. I did a very minimal job on this readme :(

Ratul Minhaz
http//ratul.xyz

On Thu, Oct 8, 2015 at 1:57 PM, Asif Saifuddin Auvi <
notifications@github.com> wrote:

> you need to add an overview section in read me abt you project!! then
> other [image: :dancers:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mnzr/PyShapes/commit/a1007e7d2d48b4e3e6c156f247f1c8afe21f0d48#commitcomment-13656720>
> .
>


if you index ``collection``, it should automatically convert it to ``collection_id`` for you


"The templates support an OpenStack environment running OpenStack Icehouse (2014.1) or later" is unambiguous, I hope.


where is the pdf file???


Yup, switching this to present tense and turning it into a procedure.


@SophieDeBenedetto This is amazing, Thank you so much!!
:100: :pray:


"... or to the CMAKE_PREFIX_PATH VARIABLE ..."


Added editing section


Replacing with "retrieve your OpenStack credentials", because knowing something is not an action, therefore it wouldn't belong in a procedure.


Look at releases: https://github.com/SquareBracketAssociates/ArchiveOONumericalMethods/releases


As above.


redundant


Changing to "create a Nova keypair".


Changing to "upload an image".


You might release a new version with only the Smalltalk part.


Some addresses are very long and will extend the containing element, thus
leading to non uniform display cards on the home page. Hence the need to
limit the number of characters. To view the full address users can click
and view the event details
On Oct 8, 2015 10:13 AM, "andela-toladele" <notifications@github.com> wrote:

> [image: :question:] Why is the address limited to 70 characters
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andela/roots/commit/b31b2d8fff5c8616e0358e4683eb6ce822f6f744#commitcomment-13658101>
> .
>


This seems moderately gnarly to me.


Keeping this one. It's an "xor" rather than "or" proposition, meaning you can have one or the other, but not combine them somehow.


Cloudinary configuration to be moved to environment variable


For Python 3.x compatibility you might want to start to use `except XYZ as ex`. Comma will no longer be allowed.


maybe add a TODO: if/when we switch to CMake 3.0, we will be able to make this lib an "INTERFACE" instead of a "STATIC" (interface libs are empty libs that are just used to declare usage requirements).


Heh, I couldn't help but tinker with it...

I know this isn't a big upgrade, but a complete guide should eventually appear in the docs folder!


tu si to mohol ukoncit returnom a nebol by tam zbytocne else blok


Boo!!!
Went stgermain? right!!!!


Could we perform the ``issubclass`` check here against the passed in image model?


@kizniche, did you receive notifications about pushing into my branch of Mycodo project?


Done.


No, this doesn't really make sense. You're only supposed to set the parameter when invoking the stack. That's what `heat stack-create -P` does. Your suggestion would imply that people edit the template in-place, which is *not* what we want them to do.


I know for sure some that would just like to keep using the floating
point grid, so please keep that as an option too, and the default one.

To be honest I don't really know all the effects of the change you
introduced, are them described on the mailing list or in some other
place ? Like, would it affect all geometries of a layer being edited ?
Or, the ones that are ever edited (like moving a single vertex) ?
Or none of that and only affect output ?


I think you should use `https://github.com/cucumber/gherkin-javascript.git` on this branch instead - you'll be able to use the latest version of the compiler. I pushed it yesterday.

/cc @jbpros @samccone


Removing (in the para about single-node deployments, too).


Until now I kept links out of the description and added them at the bottom instead. Not sure what's the best approach here.


Why is "domainResponseMessageSender" renamed to "domainResponseSender"?
Why is "ProtocolResponseMessage protocolResponseMessage" renamed to "ProtocolResponseMessage domainResponseMessageSender"?

Is JMS_TEMPLATE_NULL a functional or technical exception?


OK , I'll remember it and revise it next time!


What is the purposes of the |\/\/ in the end of this regex. Currently it breaks images hosted on a different domains in two cases (assuming http://example.com/ as the home_url):
1. When matching on a URL with protocol it will match // in http:// turning http://lorempixel.com/400/300/ in to http:http://example.imgix.net/lorempixel.com/400/300/

2. When matching content it will match urls with same protocol, turning  in to


No "neither" :)


学弟学的真快，在下自愧不如


Changing (likewise, also in para about single-node deployment).


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


You can write it like this:

    id_ab = 6 if rp.MOOG_VERSION == 2014 else 5


Thank you. It worked. I am relatively new in terms of git. Please have indulgence;)


Thanks. Looks as though here "paramnames"  can never be None (either it is set or ''). So I don't think the "if" is actually necessary, can just always pass "paramNamesFile=paramnames", and it will be ignored if either empty or None.


That's what it says in the parenthesis. :) You look into `/var/log/cloud-init.log` to see that `cloud-init` has completed.


which code calls this m68k_read_disassembler_8() function? i cannot find how it is used.


Hi!
Just tried it, but it produces the same result for me, as with the head branch => It stops working after ~2 secs (aka when the GC) did run.

P.S. I [created a patch here](https://github.com/ghcjs/shims/pull/24) for the ghcjs runtime (shims) which fixes it by trying to restore the ghc seamntics, but it might be stil buggy and/or memory leaking (haven't worked on the ghcjs rts so far), but at least it seems to solve the problem for me with that FRP event propagation stops after GC run. But I also like the idea of exploring if there could be a fix on the reactive-banana side which works with the current semantics the ghcjs runtime implements for weak pointers.


Looks like OSX has libresolv but I'm not an OSX user and no access.
I'll work on a common solution. I have a diff fore profanity configure.ac to (https://github.com/jasperla/openbsd-wip/blob/master/net/profanity/patches/patch-configure_ac) not pull requested yet.

Which areas you'll see more common? Only libresolv? Is the rest of the request okay for you? Any wishes? ;)


`ENV['SECRET_KEY_BASE']` はRailsがセッションの暗号化とかに使う環境変数だから、DBパスワードに使う値じゃないよ
環境変数の名前は自分で決められるから、一意な環境変数名であればなんでもいいよ
（例: `TODO_DB_PASSWORD`）


accidentally left it in after debugging


`self.send(request_method, body)`?


object should be created like this

cal = Calculator.new


:ok: :1234:


Are you sure it is __WINDOWS__ only? What about __CYGWIN__ and __MINGW__?


I tried, the a = cond ? d : e sintax, but it did not work. That is fine ;)

Cheers,
Sérgio

On 08-10-2015 10:24, Daniel Thaagaard Andreasen wrote:
>
> You can write it like this:
>
> |id_ab = 6 if rp.MOOG_VERSION == 2014 else 5 |
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sousasag/SPECPAR/commit/166877f299443e3d6cfc16250be54d95edb7466b#commitcomment-13658313>.
>


Can we come up with a shorter description?
*Ensures consistent parameter names in blaze event maps*


Same here, still valid for __CYGWIN__


can you explain how this trick about returning "bad data" works?


You may use `${CMAKE_DL_LIBS}` instead of manually using find_library()


oups trailing comma


is this approach OK?


@juliemr should we use `useAllAngular2AppRoots` or `rootElement` to tell protractor this is an Angular 2 app? Or should we use both?

Or does the latter mean that there is only one app running and there is no need to check for many apps and wait for them?


hmmm I think it's not used at all, as m68k read always 16bits ... @emoon could explain why it's here ?


To be consistent with the error reported [here](https://github.com/dferber90/eslint-plugin-meteor/blob/master/lib/rules/no-blaze-lifecycle-assignment.js#L29-L37), maybe the error should report:
*Invalid parameter name. Use ${shouldBeCalled} instead.*


The ARTY schematics seem to be missing the FT2232 part :/
I'll try to ask digilentinc support (since they host the schematics I suppose they did the design for Xilinx)