Consistency improves the readability a lot and often simplifies debugging process. I don't think we should come up with our own style — it makes more sense to stick to PEP8. However, PEP8 is just a guide, there are going to be questionable cases. The best way to deal with this is to cross-check each other code. If you feel an urge to edit someone's code when you look through it, this only means that the code is not good enough.

If something works perfectly it is probably not the best idea to mess up with the code by reformatting it. However, it is a good practice to revise/refactor the code when the code needs to be changed anyway, and I encourage making the suitable cosmetic changes in such situations.

Here is an example of a server setting that is formated in a way that maximizes the consistency with the majority of existing servers.
```python
    @setting(101, 'Output State', os='b', returns='b')
    def output_state(self, c, os=None):
        """Get or set the output state, on or off."""
        dev = self.selectedDevice(c)
        if os is None:
            resp = yield dev.query('OUTP?')
            os = bool(int(resp))
        else:
            yield dev.write('OUTP %d' %int(os))
        returnValue(os)
```

Finally, I'm pretty sure that my coding style is not perfectly consistent with PEP8. However, peer reviewing should help correct the inconsistencies. In fact, I think code reviews are more helpful than any guides or documents: pointing out and directly correcting each other weaknesses is one of the most efficient ways to spread the knowledge. This also helps distill the best practices.


no empty line at the end


везде в ссылках не нужен атрибут `title`, если этого не оговорено в спецификации и в критериях


This was an oversight. No need to shout.


Thanks for the precision. I removed it because Qt5 on OSX apparently does not provide it (not found in Travis builds). Now I need to understand why Travis cannot find Qt5Core, which is probably for the same reason that some package is missing...


Told you 😉 don't forget to claim them as wcnss_service under file_context


agregado de sublime text


I think I'll use ``xClickInverse`` and ``yClickInverse``, and ``xNearest`` and ``yNearest``.


i thnik it should be named simply: it_logs_message


нет


I was having two minds when I did this but ultimately just chose this way. If you think about this data type as a test result measure then then this makes more sense. We perhaps would always want to compare this data type in terms of how good the match was rather than comparing it in the literal sense.

I don't mind using a separate function approach too if that looks clearer.


пока не _фокусируйся на этом_


update


@angelozerr  Are you sure this will work for any language set as system default?


Cette PR est fermée car le résultat final ne donne rien de concluant visuellement


Time for your first PR?


Yes I used it elsewere...


да, лучше всего сабмитить `button`


refactored


Je n'ai même pas eu le temps de tester :'(


breadctumbs-item, breadcrumbs-link


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


la branche est toujours là :)


make it working in both (bunny and march_hare). Otherwise travis_hub stop woring with this chnage.


лучше оберни блок.
`oninput="sum.value=price.value"` js в разметке быть не должно


PolyglotEngine isn't on the classpath (but javadoc sees it when --unified). You have to use FQN.


Okay thx I'll change it soon. Yeah it would be great to have an overview page for all projects.


What do you mean by not liking the context idea..?


обычно это оговаривается во время работа или на этапе проектирования интерфейса. можешь вообще убрать шаг и поставить только границы, например: https://jsfiddle.net/2kn150ss/


нужен класс


@liewegas , pg_missing_t is still a on-wire Message in MOSDPGLOG when dealing with MQuery.  Automatically traversing need to know the size of the map element in Bufferlist so that certain length can be decoded --> m[k]. But currently, decoder for map do not add this. ?


Sorry for that. Strange that it didn't cause an error for me.


I was talking about apps/system/test/marionette/lib/input_management.js :)
sorry for the confusion


There's a bug when I remove the second query - will need to do more research before eliminating it.


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


was detected by clang

On Wed, Jan 27, 2016 at 8:42 AM, ImJezze <notifications@github.com> wrote:

> Sorry for that. Strange that it didn't cause an error for me.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mamedev/mame/commit/9425c141de76ef9ee178af80731110e228fa89bb#commitcomment-15698589>
> .
>


Was done? I always use done. Maybe we could find another synonym. But realize may be wrong. That's right.


Even the filtered out points are still visible though, they just have a smaller opacity. Maybe ``defaultStyle`` or ``unstyled`` or something?


Actually it did change the behavior and I fixed it via da82ce9 . There was a special case for stack init for fixing another problem due to conflict between global and local args.


I would extract it on separate method (as the method does 2 distinct things) and rename the method "top" method to be more explicit. We would have,
- initialize_topicmodeller_and_db
- initialize_topicmodeller
- initialize_db


``redraw`` sounds good to me!


Just get rid of `this.`


Ok.


@gbiellem I do agree. There is an open issue about providing samples on how to write custom throttling. Just saying "implementing throttling mechanisms which work for everyone is hard, therefore removed" isn't a help at all. But saying "implementing throttling mechanisms which work for everyone is hard, see these samples how you can implement this yourself to match your requirements" is better.


+1 :)


я всем студентов сперва знакомлю с техникой предложенной здесь: [wtfforms] (http://wtfforms.com).
пока достаточно только разметки (!)


did you file a bug for this one ? you should have put the bug # in the comment ;)


We shouldn't ever call getIntent() outside of a function, it's likely to result in a null value. getIntent should always be called after onCreate() of an activity. We can go ahead and remove this since we also don't want to hold the intent as an instance variable, it should be referenced when needed with getIntent().


Ah gotcha, nice!


поменяй обратно на тэг `button`


I would like to put a boolean at the beginning, but it basically does not allow to use the Nvidia extension "ADAPTIVE' vsync.


Good practice to check if the cursor is null before closing


`h1` должен быть на странице только 1 раз
используй знания, который ты получила на barbershop


deleted


Thanks for the contribution!

Unfortunately, these two statements are not equivalent, as here the map might contain additional elements. But as we are checking this map several times in other tests I guess it's fine :)


Cool, JS's ridiculous truthy and falsy casting has bitten me before so I'm happy to make this a habit actually.


WHERES THE APP !! THIS IS JUST A SKELETON


`h1` на странице должен быть всего 1


PC? trodde du körde mac?


Is there any reason in particular why you think it's primarily for graphic designers, or is that just the impression you got of the language (maybe you have a different definition of "graphic design" than what I'm thinking of since you included casual use)? I had the impression that it's really there for whoever feels like making something that looks cool (and maybe that's what you meant), but I couldn't point to one or a few things that made me think that.


ни в коем случае section, слишком много разделов в разделе


Are you sure about this?


@EugeneButusov pay attention to copy paste please!


Because of d3/lodash conventions, I think of min/max as "extent" and a series of sequential numbers as a "range".. :P Maybe ``indexRange`` is less ambiguous?


I was told by an old friend and fellow programmer some time ago that you should commit every time you make even some of the smallest changes. If it's a change that you want finalized in the code and doesn't break anything, commit it: you never know when you might have to revert. It's nice to have several save points where you might lose up to 5 minutes of progress instead of several hours' worth!


This seems to work perfectly. `.xhtml` files picked up the default layout without having to specify `page "/*.xhtml", layout: 'layout'` and `blog.layout = 'article'` worked as expected.


Depending what you want to say by "main loop", it is true. Through, you can have multiples call to physics | logic by graphic update. See the current "doc" in time_event.rst.


Ah ok.

By the way it should be.
if (options->phosphor[0] == 0.0f && options->phosphor[1] == 0.0f && options->phosphor[2] == 0.0f)

Maybe you want to fix it for this release, or I'll make a pull request to fix for the next version.


That's basically what I found to be the case, too.


Yes it's correct. The after callback was never used before. It was unsuitable to replace nodes.


We don't want to hard core the indexes here as they can change. When you get the cursor, you should get the index of a column using the field name from it using Cursor#getColumnIndex("nameOfFieldHere")


The title of this commit is rather unprecise : Indeed you improved coverage but coverage should not be the goal of any commit in fact and as you fixed issue I would expect to see it in the commit comment.


Can just return the result directly (no need to store it in eventList)


это  `div` (блок) c `article` (предметами торговли)


@roberthorlings is there a specific reason for removing the sorting here? It is specifically required because otherwise the order is random. I understand it affects performance, but for a list of only 50000 items it shouldn't make that much of a difference, right?


да


нет. так можно


ah, release is under way so you need to send pull request for this for
after release

On Wed, Jan 27, 2016 at 8:50 AM, ImJezze <notifications@github.com> wrote:

> Ah ok.
>
> By the way it should be.
> if (options->phosphor[0] == 0.0f && options->phosphor[1] == 0.0f &&
> options->phosphor[2] == 0.0f)
>
> Maybe you want to fix it for this release, or I'll make a pull request to
> fix for the next version.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mamedev/mame/commit/9425c141de76ef9ee178af80731110e228fa89bb#commitcomment-15698701>
> .
>


Well, the only "reason" is that it is a configuration (vstnc == 'OFF') tested only on my computer. Through, it seems a "better" setting. Moreover, it allows to use Morse without morse_syncer until some reasonably high-frequency (but the threshold where we need it is more difficult to guess).


refactored


Yeah, this looks better. We got rid of the unnecessary Eq instance.


Hmm... but the difference is easily seen...

Before:
![dict-before](https://cloud.githubusercontent.com/assets/5138986/12607234/6ecadaec-c4e4-11e5-9344-c2e9532baf23.png)

After:
![dict-after](https://cloud.githubusercontent.com/assets/5138986/12607237/73305a26-c4e4-11e5-80e9-b5f667775ae4.png)


TeamCity pull requests :: netty-tcnative [Build 46](https://garage.netty.io/teamcity/viewLog.html?buildId=5087&buildTypeId=pulls_netty_tcnative&guest=1) is now running


нет, это декоративные картинки из ставим через `background` в css


Whoops, you didn't take the change from minus to plus along here.
let x = res.location.value.x - (res.size.value.width / 2)
let y = res.location.value.y - (res.size.value.height / 2)
Needs to be:
let x = res.location.value.x + (res.size.value.width / 2)
let y = res.location.value.y + (res.size.value.height / 2)
...or it will move to a location to the upper left of the element :)

For reference:
https://github.com/webdriverio/webdriverio/commit/b73635f64f701d2efeee607c3b169a7f360f3dc2


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


насчёт action – не важно куда, а вот метод здесь будет post.


TeamCity pull requests :: netty-tcnative [Build 46](https://garage.netty.io/teamcity/viewLog.html?buildId=5087&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:01:01


@kraban @kraban @kraban @kraban


打开这个comments, 让我的笔记本风扇狂转啊,
[这个bug出现的提交](https://github.com/MrMEEE/bumblebee-Old-and-abbandoned/commit/6cd6b2485668e8a87485cb34ca8a0a937e73f16d)

另外自此以后,这个项目好像停止维护了??


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


Snyggt! :)


@TheStrix  luttuce!!. lul


知乎观光团 :+1:


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


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


Did you mean to remove `"loadeddata"`? Although with timeupdate, you might be able to detect stream and element size changes?


Yay! Thank you!


Okay, I think it's just a minor bug and shouldn't cause any other problem.


Yeah; I put it in the same commit because hound made me write the new code that way and I wanted to be consistent between the two validations (I initially used the same code as the old one here for the new delegator validation).


It is more a cmake, homebrew, qt5 issue I think. Something along the line of adding

    -DCMAKE_PREFIX_PATH=`brew --prefix qt5`

to the before_script cmake command should fix it.


Phân trang


Xóa file này đi


bỏ space trước và sau dấu =


End of files, thêm Enter ở cuối


Thêm enter ở cuối


The last two commits in branch https://github.com/practiceprovider/openemr/commits/php7-testing are related to GACL

gacl/gacl.class.php (changed adodb path and fixed php4 style constructor)
gacl/profiler.inc (changed split method to explode as split is removed in php7)


Very nice! Thank you


Can you show me where this start_date in TL table ? is it changed ?


Please let me know should I commit new Smarty update


Things on the internet have a habit of staying around for a very long time.  Some day, you might give your github account to a prospective employer... Or your colleagues may search you.  Maybe even you'll start your own company some day and many people will Google to see what tjhorner is like.


why do you use matchCONTINUE here?


please check this line once


@grempe Good catch, thanks! Do you want to send me a PR? (It's trivial for me to change it too, let me know!).


Suggestion: I would prefer Removing the `else` and assign to `namespace` before `if`.


@olefredrik this right here was the problem!!! It was the missing `@link`   :wink:


ok done every thing fine


Does this call works with ARM containers?


TeamCity pull requests :: netty [Build 1116](https://garage.netty.io/teamcity/viewLog.html?buildId=5094&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Out of memory: unable to create new native thread (new); tests failed: 1 (1 new), passed: 3390, ignored: 6 Build time: 00:10:49
### Failed tests
```
io.netty.buffer.PooledByteBufAllocatorTest.testConcurrentUsage: <no details avaliable>

```


TeamCity pull requests :: netty-tcnative [Build 47](https://garage.netty.io/teamcity/viewLog.html?buildId=5099&buildTypeId=pulls_netty_tcnative&guest=1) is now running


ok, now i understand, its better to handle disconnection in Cluster class. It's supposed to do such things in this class.


Does the cmdlet "New-AzureStorageContainerSASToken" works with ARM container resources?


TeamCity pull requests :: netty-tcnative [Build 47](https://garage.netty.io/teamcity/viewLog.html?buildId=5099&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:01:00


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


@Kikobeats What is the reason behind this change? This will result into everybody has to edit (and handle those changes on updates for) the partials/aside.hbs. Before this change the 'default' @blog.title set by ghost itself was used if not overridden.


Ha, so previously the specs were creating posts with the slug set to the actual string "slug", and then searching for posts with that string. Glad you fixed it, @sareg0!


Leaving this alone for v0.12, but need to revert it for Sails v1.0 (added inline comments w/ more info)


We're on 2016 ;D


@aaronra Thanks!


This and the following append_arg seems to be full of whitespaces instead of tabs


Yes, definitly a good idea. I think now its time to document the api :)


Leaving alone for 0.12, but blueprint hook should expose this for Sails v1.0 (added more info in inline comments)


I don't release this yet, but I'm experimenting.

Based in the javascript code:

1) Bad for SEO
2) Bad visual effect. When you enter a site that inject this, you see a no immediate change in the frontend and looks terrible.

Needs Ghost Apps to be resolved successful :(


TeamCity Civic Assembly :: Build all [Build 29](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=476&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 130 (7 new), passed: 1609, ignored: 7 Build time: 00:18:08
### Failed tests
```
Basic Pages Categories CRUD Actions: Should navigate to  Category page: <no details avaliable>

Basic Pages Categories CRUD Actions: Should Not Add a  Category if Cancel button pressed: <no details avaliable>

Basic Pages Categories CRUD Actions: Should Add a  Category: <no details avaliable>

Basic Pages Categories CRUD Actions: Should Not Edit an  Category: <no details avaliable>

Basic Pages Categories CRUD Actions: Should Edit a  Category: <no details avaliable>

Basic Pages Categories CRUD Actions: Should Not Delete a  Category: <no details avaliable>

Basic Pages Categories CRUD Actions: Should Delete an  category: <no details avaliable>

Document tests: Should take a alert when try to upload new version for locked document: <no details avaliable>

User profile tests: Should change user profile info: <no details avaliable>

User profile tests: Should cancel upload new user avatar: <no details avaliable>

User profile tests: Should upload new user avatar: <no details avaliable>

##### there are 120 more failed tests, see build details
```


@Nagriar should be dump a version to make it more obvious? recently we got some new SNI issues, which is weird


This is going to cause tests to fail until either tealeaf is a public repo or we give the circleci access to it somehow.


実はMilkcocoaのMQTTはconnackのパケットを少し改変していまして、pythonのpahoだと正常に接続を確立できていませんでした。その部分を修正しています。

paho/mqtt/client.pyの1981行目の以下をコメントアウトしています。

```
        #if len(self._in_packet['packet']) != 2:
        #    return MQTT_ERR_PROTOCOL
```


Tu veux VRAIMENT utiliser une logique dans un constructeur....
Un constructeur n'est pas sensé (ou en de très rare cas) utiliser une factory !
C'est l'inverse... Une factory utilise des constructeurs ou d'autres factories pour créer une instance d'objets cohérents.


Tut mir ja leid!


Maybe it is developed? Created? Initiated?


Set the TM pointer to null after destroy


BTW https://interface.fh-potsdam.de/eingabe-ausgabe/2015-2016/ its online.


And seems like we need to guranty the start/stop atomics at all.
Beacause it's possible to call start in the one thread and stop in another.


to be more clearer I did a list (where are ?? I am not sure it is usefull to have it in maven plugin, but it is worth it anyway ;) ):
-keepWorkDirs ??
-explodedApps ??
-exportCSV
-enableTattletale
-excludePackages
-enableCompatibleFilesReport


They don't support IE8. See at the bottom of [their site](https://lodash.com/):

> Tested in Chrome 46-47, Firefox 42-43, IE 9-11, Edge 13, Safari 8-9, Node.js 0.10, 0.12, 4, & 5, & PhantomJS 1.9.8.


you can omit the first prams's type: string


あーーーなるほど！！どうもどうも！


please take another look


Hmm, bah en même temps je voyais pas trop où le mettre non plus...


fyi: i was just trying the lib with
```
void setup() {
  Wire.begin();
  Serial.begin(9600);
  Serial.println(Plugin_014_si7021_begin(SI7021_RESOLUTION_13T_10RH));
  Serial.println(Plugin_014_si7021_readValues(SI7021_RESOLUTION_13T_10RH));
  Serial.println(si7021_temperature);
  // put your setup code here, to run once:

}
```
it was returning -1 for readValues
then i just fixed the delay to 75ms and i got the reading. i haven't investigated too much in the timing though.. but looking at the coding - apparently a lot is being done to compute the correct delay.. for me 75 ms is fine


@roman-melnishin, здесь всё-таки лучше article http://html5doctor.com/downloads/h5d-sectioning-flowchart.png


@dkayiwa Sorry!! seems to be the same uuid. :)) will remove the line :))


Should we add such a TODO to AggregateRepository? It still implements CommandHandler.


Done or one of its synonyms would probaly be better. http://www.merriam-webster.com/thesaurus/done


Duplicated log


а где закрывается `section`?


Needs removal in other repositories as well.


Thank you @kucharovic


Or something from made http://www.merriam-webster.com/thesaurus/made


Почему по-английски?


I see this file is mentioned here now, but no such file added as part of a commit. I presume file was always there, but wasn't mentioned as part of the package.


style.css


Okay ... so injecting vars via JS for this purpos is bad in general. this also counts for all the other hacks actually can be used.
Anyway ... I don't understand why you don't set 'Kiko Beats' as your "Blog Title" in ghost and use '@blog.title' as before. Or is the technical result the same as injecting the var via JS?


I addressed various of those concerns here; (reddit) bit.ly/1WNnEM9

Please consider continuing the conversation on reddit or anything not github. Github is not for debate.


Are you sure this is a good solution? The state and the potential already use the halo size, so changing it later to 8 does not sound safe. I would rather move this whole logic to the constructor of the lattice.

The main problem is that it is exactly the single-process execution that suffers from a fixed halo length of 8, so the Python version would be slowed down. Anyway, I suggest dropping the optional argument of the lattice constructor, and live with the slow down. Interesting systems have a nonzero angular velocity anyway.


ok good go a head


![vpc us_east-1](https://cloud.githubusercontent.com/assets/13885004/12607884/86de0ee4-c4d8-11e5-97be-794486491ccc.JPG)


Yeah, but this is consistent with the other format classes, AFAICT. We should do some mass change getting rid of all properties defined like this.


This now differs from 3.x branch. Hopefully you can merge them.


Good catch! However, it is just a comment moved from the other file.


I tested this patch. I found strange thing. I load cached data from file and reuse it when node process restart. The problem is that the cache data which size is under 8K is always rejected with sanity check error 'MAGIC_NUMBER_MISMATCH'. cached_data_buf is wrong completly from GetCachedData.


:tada:


ошибки нет
может показаться интересным: https://developer.mozilla.org/ru/docs/Web/CSS/font-weight


TeamCity pull requests :: netty [Build 1115](https://garage.netty.io/teamcity/viewLog.html?buildId=5093&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4910, ignored: 23 Build time: 00:36:38


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


Зачем числовые обозначения, если можно ввести string-поле и работать с ним напрямую с гемом aasm?


What if the partition doesn't have a format? I think this should check ``self.format`` first.


@blog.title is used in the title and makes sense use it. For example:

`Kikobeats - Software Engineer, UNIX, JavaScript & Open Source`

This is ok and good for SEO.

But int the aside I only need the name 'Kikobeats'.

What happens?

- I can't inject using handlebars helper because Ghost don't have one and I can't write one because I can't extend Ghost core without Apps supports.

- Inject using javascript is an alternative but is bad as I said.

so, fuck, yes, I don't want to force all people edit a file but Ghost don't support this 'feature' yet.


Yes, the problem is that I don't know the reason and @schubi is on vacation. I'd add a new task to the PBI to clarify this.


Would there be any issue in also adding `.xml` and `.xhtml` to the default extensions. They can be easily added via the config file but is there an issue in gzipping them by default?


Which board/device are you using ?


Task.find(params[:id])


у всей типографики должен быть прописать межстрочный интервал:
https://developer.mozilla.org/en-US/docs/Web/CSS/line-height
https://css-tricks.com/almanac/properties/l/line-height/


Can you add a newline here?


or TRUE?


лучше line-height: 1


Holy cow!
That was fast :running:


There is no need to match the return value of the 'case' to 'ok'.


Isn't there a pitfall here ?

If you add items blindly with no knowledge of if you gonna have a simple sitemap.xml or sitemapindex, the $path parameter becomes irrelevant because you're not controlling where are stored the sitemaps files.


POST используется для создания новой сущности, в данном случае будет происходить изменение атрибутов существующей, поэтому PUT либо PATCH


I've had somewhere in my mind that custom exception *shloud* inherit from `ApplicationExcpetion`, not from `Excpetion`, so the opposite. I've used it in my projects :worried: It's wrong and I have to correct it.


Your description sounds more like a stand-alone (as opposed to fragmentary) DSL, rather than an external DSL. I've had a look at the syntax, and it is _very_ reminiscent of Lisp or Racket, so I would guess it's either an internal DSL hosted in Lisp or the designers chose to very closely mimic it. I'd have to ask the developers to be even kind of sure which one of those it is, though.


Версия же должна браться из `package.json`


@jeckman Hi. @mega94 don't know english, but have a question: why script doesn't download this url: https://www.youtube.com/watch?v=0AqnCSdkjQ0 ?
Other services like SaveFromWeb.net download it successfully. Can you help with it?


LGTM


TeamCity pull requests :: netty-tcnative [Build 48](https://garage.netty.io/teamcity/viewLog.html?buildId=5102&buildTypeId=pulls_netty_tcnative&guest=1) is now running


TeamCity pull requests :: netty [Build 1117](https://garage.netty.io/teamcity/viewLog.html?buildId=5100&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 5088, ignored: 18 Build time: 00:28:02
### Failed tests
```
io.netty.handler.codec.http.websocketx.extensions.compression.WebSocketServerCompressionHandlerTest.testServerNoContext: <no details avaliable>

```


In the file it's not actually like it's showing here. Prob the weirdness is due to the way changes were staged for the commit.


Hi sir, as the the issue in this page [GitHub](https://github.com/opensourceBIM/bimvie.ws-viewer/issues/3#issuecomment-136335282) , i am use your "bare metal" code to download. Could I load object/geometry by the download component ? If yes, how can i work? thank you so much!


Please indicate the shape of the returned array.


TeamCity pull requests :: netty-tcnative [Build 48](https://garage.netty.io/teamcity/viewLog.html?buildId=5102&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:55


May be it related to #117 or #111.


“slightly more complex chart”.

I don’t know what’s the point of this section though.

If you want to explain that it supports row channel as faceting then it should be explained, but I figure that a duty of a missing “facet” page.


E Z grondstoffen


I would even go as far as to say that, except for the fact that this language does seem to actually be runnable, it's almost as restricted in use as a configuration file is.


Ошибки теже, что и в Барбершопе


ok


Ah ... now I see you use @blog.title to set your html-page title and and using the aside to have a short form of that. Now I understand why you divert this. This makes totally sense for your use case, even if most people won't need that. But he ... others use your (great) work, I think those can handle that.


Nope, I was just an idiot and forgot to commit it. Thanks for picking that up. That's twice this year I think :)


Maybe you didn't want it to show like this but it does.
https://github.com/electricimp/APDS9007/blob/c6b9bef9d1af73e697c44753b00a8e12b02e57d3/APDS9007.class.nut


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


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


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


:heart:


Is this really necessary? We should test if source maps works better...


3.x was always different because it requires you to pass a `-` if you want PHPCS to wait for STDIN. That stops PHPCS looking like it is hanging when you forget to pass a file to check.

But 3.x works the same in all other ways. You can pass files via xargs and you can pass STDIN directly into the command.


@cr0sh it's already mentioned:

> `// #blameshoghi`


Don't worry, since I'm watching the repo I will pick that stuff for you as long as you mention commit in an issue/PR because GitHub doesn't send e-mail notifications on pushes.


It's unrelated, but yes :)


There's also the fact that the engine itself is proprietary; while they say in the FAQ that you could sell your .zrf to people if you wanted to, I don't think that's how you would want to use this. What you said might apply to the people working at Zillions of Games, though.


What for ?


@task.user == @user
Надо оперировать объектами, а не их атрибутами.


Do we have the chance for returning with FALSE?


Yep, ok, https://github.com/electricimp/APDS9007/commit/eb6d3439ea8aededf87f3cd3f50d55da70007c6f...


If it let you specify arbitrary games, it would also basically be Unity or Unreal Engine, which don't use DSLs since arbitrary games are, well.... not very specific a domain.


I can put 'Kiko Beats' as default and inject by javascript if you declare the variable :P


Oh, thank you! I was so happy to find this commit after a week of digging for solution of my errors, so i was dare to ask you for PR :) Just a few weeks ago introduced chef-solo in my life. Of course, I agree with you, this commit must not break previous versions.
In addition, this commit may be interesting for you: https://github.com/leonid-shevtsov/knife-solo/commit/a7efa0fed2d25fd7ce796d2ba7fbd5f4213ced7c


По идее можно было хранить название файла и в модели Task, а все остальные настройки (допустимые типы, и т.д. - это уже логика Uploader'а)


LGTM


sorry
I tried on a wemos d1 mini
On Jan 27, 2016 10:35 AM, "Charles" <notifications@github.com> wrote:

> Which board/device are you using ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ESP8266nu/ESPEasy/commit/9d3094f8eaacd7755f5c690760b44aa7b8f9f120#commitcomment-15699318>
> .
>


developed is good I think. @miduku We use made just the sentence after. I'll change it to _developed_ if you don't mind.


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


Hi, I got SMPTE-TT captions here and some captions are empty. It seems valid to me even though it doesn't make much sense but this code explicitly forbids empty Cpations. Is there a specific reason for that?


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


Hmm... `serialize_read_TYPE` should be renamed to `deserialize_TYPE`
and `serialize_write_TYPE` to `serialize_TYPE`.


This is a moment to discuss: I see only two evidence ways to decompose this score, first is to re-count values four times, and second one is to cache one more matrix with Phi size. Both ways are unacceptable, so this change is under the question.


I can't reproduce what you're saying.  Are you sure you're not mangling the data when writing it to disk, e.g., by passing an encoding to `fs.writeFile()`?  If it's not that, can you open an issue and include a minimal test case?


Это старый код, просто перенёс сюда. К тому же он будет выброшен в следующем релизе. Предлагаю оставить как есть.


Missing @since.


Here it is. The three changes are:
- realized > developed
- https://github.com/FH-Potsdam/steel-ant-input-output > https://interface.fh-potsdam.de/eingabe-ausgabe/2015-2016/
- http://fabianmoronzirfas.me/ > https://github.com/fabiantheblind


@miduku @fabiantheblind


@Nadahar are they worse than 5.5.0?


@mcallegari I don't remember, I guess this was a temporary include. It's gone now.


Формы лучше делать с помощью гема simple_form, у него есть много встроенных хороших возможностей и отображения ошибок без изобретания велосипеда


Åh, en sån tabbe! My bad.


Python byte code. Don't include it the repo.


И labels тоже гем simplef_form сам формирует


Thanks. Gonna merge


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


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


done


Can't see required header's include


I even threw in documentation


The comparison was wrong but it worked because we never compared Ok with Fail or Partial. We break out of the loop as soon as we find an Ok snapshot. The code is consistent now. Thanks!


Vmaps4 ofc


Missing @since.


Ahhh So this is the reason why it went back... Why not load the CSS files via import using less or sass? like there's an index.less per module and that will be load in the /client/main.less?


This is buggy.  find(1) returns the record with id 1, which is not necessary the "newly created" record.


ASSERT (EXPECTED, ACTUAL)


Cool beans. I've still to add the missing files to the make dist. I'll get onto it.


@qkmiao @Richard-Yunchao , pls help check it, thank you.


Commit inicial con el readme básico


Oui bonne question et j'avais zappé la collection — j'y repensais car il me semblait que t'avait fait un truc à ce sujet.

En l'occurence, ce qui m'a poussé à faire comme ça c'est qu'on affiche plus souvent les speakers que le programme. Donc y'avait besoin de pouvoir itérer sur les speakers plus simplement qu'en allant les extraire des talks.

On peut très bien avoir les deux, ça ne gêne en rien et j'ai poussé un autre commit qui montre comment lier l'ID du speaker avec ses données.


To be honest, the old unstyled menu looks better! ><"


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


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


With next commit (https://github.com/electricimp/APDS9007/blob/9a7fa0c4ed7cff9a104327e4d71f8730da7ff8c5/APDS9007.class.nut) things fall in right places. Staged the difs related to parameters update while having some other edits around those lines. In the end it's all good.


some didnt which is why i created them, some still need to be created. if the config pages werent versioned via the url we could probably get away with reusing some of the links (like we used to) but then it became a pain to update pages on changes.. thus ideally we should just have -0-8 pages so that people dont think we linked to the wrong page on accident and if we/someone makes changes it applies to the correct version of the app.. as some of the config pages recently are getting touched..


didn't seem fix the issue :(


`tags_deserialize` -> `deserialize_tags` ?


:clap:


Just speaking for me, this would be better then working with (one more) diff.
It should be still possible to do something like this via injection to get the old behavior back:

<script>
var my_aside_blog_title = blog.title;
</script>


I will work on this next. Admins will be able to fill an email adress out into a form and an email will be sent to that email with a randomly generated link. This link can be used to register an account.


Effect index is always less than 4 the only possible values are 0,1 and 2


If I recall correctly past entries should have negative endSeconds. I'll
check.

Am Di, 26. Jan 2016, um 22:17, schrieb The-Flix:
> Great work, thank you! I believe you should change "else if (endSeconds
> >=0)" "else if(endSeconds>0" to not show events that last 0 seconds.
>
> I have entries in my calendar which last from 24.10.2009 10:00 to
> 24.10.2009 10:00. I don't want to see them years later. :-)
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/bitte-ein-bit/MagicMirror/commit/579ef5fe75ea6800f601ace7560dc24b925b5ef7#commitcomment-15689122


copy and paste error. Thanks :)


**BTW**, that should closes :point_right: #1573 (i think :wink:).


Oh sorry I meant the sensor One ;)


should we really name it `ReverseProxy` ? atm the name is not explicit?


Sure , we'll write sepolicy from scratch


Currently meteor support it's way of CSS loading. And I don't want to focus on a CSS framework for now. It's too much in this stage.

And putting CSS inside the components really make sense.
So considering all these, it's not worth address this now.We'll do it later on. May be after modules.


@kadowa  ist die struktur ok für test-script ?


Naming convention around serialize/deserialize is not consistent... (one is following `class_method` form, like `nv_table_deserialize, other is method_class`, like `_deserialize_sdata`... ).


Ok... We'll go then to a minimum viable product :) onwards

On Wed, 27 Jan 2016 17:03 Arunoda Susiripala <notifications@github.com>
wrote:

> Currently meteor support it's way of CSS loading. And I don't want to
> focus on a CSS framework for now. It's too much in this stage.
>
> And putting CSS inside the components really make sense.
> So considering all these, it's not worth address this now.We'll do it
> later on. May be after modules.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mantrajs/mantra-sample-blog-app/commit/1a2ecbc38e98fb78f0870d99f69d00b83b38c715#commitcomment-15699778>
> .
>


aahh.. i am using one of the aliexpress ones GY-21 - Si7021

On Wed, Jan 27, 2016 at 11:02 AM, Charles <notifications@github.com> wrote:

> Oh sorry I meant the sensor One ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ESP8266nu/ESPEasy/commit/9d3094f8eaacd7755f5c690760b44aa7b8f9f120#commitcomment-15699767>
> .
>


twenty-six... I like this number :-)


7.3. The SOA.MNAME field

   It is quite clear in the specifications, yet seems to have been
   widely ignored, that the MNAME field of the SOA record should contain
   the name of the primary (master) server for the zone identified by
   the SOA.  It should not contain the name of the zone itself.  That
   information would be useless, as to discover it, one needs to start
   with the domain name of the SOA record - that is the name of the
   zone.


I undid this commit in eff4d37d6466d4598921f5138cd32bdd31072830. The bug is fixed and we need to ensure we show an **Exception** if it happens again.


Not sure, but is there anything else _reverse_ in webservers usually?

I think shortness and memorability is also a factor ;-)


New repository


А почему не через фикстуры или фабрики создаются данные?


thanks.


By adding this to the enum, the value of DETECT_SM_LIST_MAX is one higher than necessary. Which means various arrays in the Signature structure are bigger than needed.


И в случае с объектами надо фабриками пользоваться. Как вариант, factory_girl


For tests we prefix the lines with \test


I should probably move this check somewhere else. An empty collection in "this case" basically means that we got empty result, which is bad. But you are right, it does not belong here.


It's ok to use `const` here, even if you mutate the object afterward. `const` means the variable can't be re-affected, not that the value is frozen.


Дублирование описания дальше по коду. Надо в отдельную переменную выделять и использовать дальше по коду.


Based on the error message, this is a deserializer, but this is a checker function. This may conflicts with SRP.


Советую ещё посмотреть гем site_prism.


TeamCity pull requests :: netty [Build 1118](https://garage.netty.io/teamcity/viewLog.html?buildId=5101&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 4910, ignored: 23 Build time: 00:35:27
### Failed tests
```
io.netty.channel.epoll.EpollSocketTcpMd5Test.testKeyMismatch: <no details avaliable>

```


Sind 00-03 die verschiedenen Kapitel? Für das Testskript dürfen unter ```./xml``` keine Unterordner mehr liegen, besser wäre daher

```
 static/tests/sander/src/xml/snd00.pt0.nlm.xml
 static/tests/sander/src/xml/snd01.pt1.nlm.xml
```


Maybe, it will be useful to rename some fields with same sense in similar way.Examples:

num_tokens,
num_entries,
num_items_per_batch

but

min_total_count,
inner_iterations_count,
processors_count.

Also these variables should have 'repeated double' type:

 optional DoubleArray kernel_size = 1;
  optional DoubleArray kernel_purity = 2;
  optional DoubleArray kernel_contrast = 3;
  optional double average_kernel_size = 4;
  optional double average_kernel_purity = 5;
  optional double average_kernel_contrast = 6;
  optional DoubleArray coherence = 7;


Hi @FuadEfendi ,

You may need to set the `core.longpaths` to `true` in your git config in case of Windows OS.

Reference:
https://github.com/msysgit/msysgit/wiki/Git-cannot-create-a-file-or-directory-with-a-long-path


This looks mostly fine to me. I'm not sure about the status code, though. If the key is missing, it gives 400. If the key is present, but the compose does not exist, it gives 404.

Grammar note: `Compose %s does not exist.`


Heh, you are right, my bad  :/ But I'm on it right now in https://bugzilla.mozilla.org/show_bug.cgi?id=1233552  and dont plan to keep it skiped for too long :P


Good catch, indeed the test does not trigger the restart button, maybe because there is no easy way to determine if a iceRestart has been performed from Javacript directl?. But indeed the page works, try it out.

@fippo any comments?


okay


php contact form not working either


Please write better commit messages. No commit is ever temporary! Try something like this:

Removed helpers from AR::Base because the the "tag" method from TagHelper was polluting the ActiveRecord classes and causing a conflict when upgrading to Rails 4. Rails now is more vigorous about checking for conflicting methods. Plus this include is potentially a security issue and not good practice.


哟，不错嘛，给我API我要写客户端


Let's do 2pm then. Only respond if that doesn't work for you.


Also, don't comment out lines of code. Delete them, or no one in the future will ever know why they are commented out and might just bring them back.


和哈哈哈哈哈哈哈哈哈


mismatch tags < > </>


[Hai] Mismatch tag < > </>


yep! =)


@tjmonsi MVP. That's the best way rather working on all the stuffs at once.
We started with an empty repo  :P


this is how I would love to run it. sad thing is, that id doesn't work. I don't know why, but if we run the script by piping it to /bin/sh, it stops after the first ```vagrant up``` and the rest is simply skipped.
I don't know why. Do you have any ideas?


In your commit messages tell me *why* you did something, not what you did. I can see from the commit that you added "self.class", but why did you do that? Did it fix a bug?

Also, don't do two unrelated things in one commit.


e.g. ReverseDns


I mean it's one step removed from a global variable, which makes evaluating what the function is doing a nightmare. State should generally be minimised. In this case, it seems really easy to minimise, so might as well?


There is a lot of noisy whitespace changes here. Do you have a tool (Redmine, etc) which might be able to do this across the entire project in one giant commit so that we don't get them in each commit in the future?


Now we have to add to docs that `handlingResult` could be of the void type. Also, note that `commandHandlingResultToEvents` method is overridden in `PmCommandHandler`. And now we can remove it from there.


yeah both those options are good! Another option is filteredIn. Your call. But double negatives are bad variable names. =)


Just for the readers here. We discuss at https://github.com/JabRef/jabref/issues/705#issuecomment-175083096 that there is `EntryUtil.getSeparatedKeywords(String)`.


Yes it will work. Tern error message are hard coded. See https://github.com/angelozerr/tern.java/blob/master/core/ternjs/node_modules/tern/lib/tern.js#L841


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 100](https://teamcity.labkey.org/viewLog.html?buildId=299589&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:14:12


I think they should be added--go ahead and raise a PR.


Maybe we should introduce a constant for `","` too, to make the code more readable.


vip3 test migrate


How about "allIndices"?


Looks okay to me, but probably worth a second look by someone more familiar with `supervisord`


It returned `nil` anyway if the entry was not present in `linuxrc`. It should work as far as when processing rules those values are converted to strings (with `sformat` or `tostring`): https://github.com/yast/yast-autoinstallation/blob/master/src/modules/AutoInstallRules.rb#L355-L390

Anyway, I'll do some further testing about this.


`ReverseProxy` :+1:


Please update this parameter name to 'requiredVersion'


IOP test on hi/lo move
```
   45048:       34180010        li      t8,0x10
   4504c:       03000011        mthi    t8
   45050:       34180020        li      t8,0x20
   45054:       03000013        mtlo    t8
```

```
3406006e:	mov    eax,0x45048
34060073:	nop
34060074:	mov    eax,0x4504c
34060079:	mov    DWORD PTR &hi_B0,0x10
34060083:	nop
34060084:	mov    eax,0x45050
34060089:	nop
3406008a:	mov    eax,0x45054
3406008f:	mov    DWORD PTR &hi_B0,0x20
34060099:	nop
```
without tags
```
34060079:	mov    DWORD PTR &hi_B0,0x10
3406008f:	mov    DWORD PTR &hi_B0,0x20
```

It seems mtlo is compiled as mthi ...


The charger patch is missing the addition of `bool ext_control;` in include/linux/power/bq24735-charger.h


Please update this local variable name to 'currentSysVersion' (or some close variant)


why we need this line


Yay... this has now worked.


@DV3D thanks for the feedback, will fix that


Please update this code to read from requirements.txt, rather than hard-coded


cloud it have a [ wrong?


Perfect, thank you! Feel free to merge.


Hast du hierfür eine andere Lösung?


Well after a quick grep
```
void rpsxMTLO()
{
	if( PSX_IS_CONST1(_Rs_) ) {
		xMOV(ptr32[&psxRegs.GPR.n.hi], g_psxConstRegs[_Rs_]);
	}
	else {
		_psxDeleteReg(_Rs_, 1);
		xMOV(eax, ptr[&psxRegs.GPR.r[_Rs_]]);
		xMOV(ptr[&psxRegs.GPR.n.lo], eax);
	}
}
```
Ouch the typo. However again the test suite only check the constant path and not the general path.


Use internet while watching this page on browser. Or else some portions won't be visible.


:+1: as is, `$http.get(...)` doesn't work


it's not a matter. You can do that too


Declare variables as much as possible close to its first use.
RawMessagePtr res = prot_packet.serializePacket();


karatsuba's recursive multiplication


I made the other missing Wiki-pages to complete it all.
Would be nice if/when we move the website to GitHub pages so we can make some proper layouting of the wiki..


@s-hinse Achte auf den Codex, Leerzeichen nach dem `.`, also `. esc_html__(`
Wenn du PhpStrorm nutzt, dann kann das die IDE für dich erledigen.


Оу. Прошу простить, не знал. Тогда всё ок.


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


isn't that service_instance_name_or_id from the manila proposal?


isn't this service_instance_user from the manila proposal?


теперь это не так делается, а через _sessionProvider.GetCurrentSession() и без using


isn't this service_instance_password from the manila proposal?


I neither have a strong opinion on this. Regarding compute, I don't see the tags and metadata fields relevant to determine a node/image equality. Could it make sense to add then the user metadata just to the StorageMetadata subclasses?
In any case, I've opened [JCLOUDS-1070](https://issues.apache.org/jira/browse/JCLOUDS-1070) to track this.


Done: https://github.com/opengapps/opengapps/commit/4376e8481fd687d56c9cc9e679266724c7414236


okay, okay. pushed.


personally I'd call that manila ..


Это лучше верстать блочным элементом, т.к. нам не нужно переноса на первую строку при коротких названиях групп товаров или компаний.


От и до в этом случае будут лейблами (label), при нажатии на которые ставится курс в поле.


it seems logical to change this:
--- a/src/main/java/org/openpnp/vision/FiducialLocator.java
+++ b/src/main/java/org/openpnp/vision/FiducialLocator.java
@@ -69,11 +69,6 @@ public class FiducialLocator {
         Location aLoc = a.getLocation();
         Location bLoc = b.getLocation();

-        if (boardLocation.getSide() == Side.Bottom) {
-            aLoc = aLoc.invert(true, false, false, false);
-            bLoc = bLoc.invert(true, false, false, false);
-        }
-


@sevoku: Does it mean he removed support for md-*-enum altogether?


Ты сделал активный список, зачем?) У нас уже есть общий класс navigation-menu, тебе надо определенному li сделать класс link-active, например. Который уже будет применять для него стили по гайду (подчеркивание например, когда мы находимся на этой странице).


I suspect any measurable difference as most of the methods here are not in 'hot path' of regular pool operations (get connection and close connection). However, I like the change conceptually.


I can see almost the same piece of code in every test case. Try to think : May be create some function and pass args to it.


sorry, the diff displays wrong, inside fiducialLocator there is this old code that should simply be removed.
the check for getSide() and inversion of coordinates.


many thanks！


А вот этот див, что он закрывает? Так как в контейнер тоже должен слайдер. Раньше не было)


#Learning Share And Notes
- ### 从今天起不定期分享一些对于iOS的学习心得或者知识点，以及开发遇到的难点解决方式等等#


Чаще всего такую связь реализовывать через отдельную модель, где можно навешивать дополнительную логику.


generator-license was added to Travis in 2013 so we need the sudo flag.


开博初衷


Use git cherry-pick man!


А вот этот див, что он закрывает? Его раньше не было. И контейнер должен в себя включать след. блок слайдер.


这里打错了.....composer的这个require关键字是""


maybe say `.orig backup files before applying patches` so users understand what those files are for


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


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


Этот метод нарушает принцип Command Query Separation


s/do/to


AAH WHAT IS THIS CODE STYLE VIOLATION. How on earth did that happen. Probably vim, and tabs vs spaces. Jerk tabs.


@PeratX yes we copied some code bugfixs becouse its very hard fix some bugs.


Переводы ошибок из контроллеров надо выносить в locales


Здесь один из тегов лишний достаточно или div или p


hey, it'd be simpler if you'd leave out the default, just do action store_true and then you use this setting later you access it with `section.get('no_orig', False)`, this way we don't have unneeded default values swimming everywhere.


unack commit message again :(


see, this shouldn't happen :) it's a redundant value.


I've reported the issue upstream: https://github.com/jbeder/yaml-cpp/issues/361


Полезно использовать гем simple_form для построения форм. Там и отображение ошибок в комплекте и много чего полезного


I rebased, squashed & finally reviewed. Can we merge this PR?


@weitsang This is currently hard-coded because currently there is only 1 guildmaster. We will fix that when we integrate the sign-up/login system into the game.


+1 for @nitincchauhan
So you do not do benchmark.


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


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


thanks :) was such a headache!


hey, you don't want to reparse the CLI. The thing is we have built a configuration management system so you don't have to deal with this. The neat thing is that this way the user will also be able to give the value via a project, user wide or system wide configuration file.

So we wanted it simple. We've written a little component that automatically gives you values you demand in your function signature, like it's done for the bears. So you can simply add a parameter `no_orig: bool=False` to the function signature, best to also add a documentation sentence to it, and coala should automatically retrieve this value for you.


Для создания объектов надо использовать фабрики Factory_girl


repositories {
    jcenter()
    // add the following lines ...
    flatDir {
        dirs 'libs' //locate the lib-release.aar file in libs folder
    }
}


Поправь тут разметку (отступ и перенеси закрывающий тег ссылки на следующую строчку)


Во-первых, создание через фабрики.
Во-вторых, предварительно надо user'а создавать, а потом его использовать при создании task. Не верно назначать user_id = 1. Откуда это число взялось?


please read my comment on your previous version of this commit ;)


ok, work with 275fe857fe0e1586fc3267331804c72d5f8b8276


![image](https://cloud.githubusercontent.com/assets/576184/12609553/c19b6e46-c4e2-11e5-82c6-35ef535e6ab9.png)

I just tried this, I see the private/public etc modifiers are all "identifier" (same as variable names) which makes the code look different than it was.


Spaces


Not used, YET!


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


Spaces in all function calls


@ligi Is there a particular reason why you do not add the flag in `onCreate()`?


Posted my comment to #2157


I would rather add such hack in the SDL library as it's maintained downstream. Also I think this should be configurable in order to let the user choose between battery life and performance.


Since we are doing this, should clean up the adodb stuff that we are not using. Rec the following:
1) In the interface/main/calendar/includes/pnAPI.php file define ADODB_DIR to the library/adodb directory.
2) Remove the interface/main/calendar/pnadodb directory
3) Remove the gacl/adodb directory

In this case, ok to do the constructor fix, but avoid mixing constructor fixes and above fixes in the same commit.


No. This is a image which needs to have nfs/cifs tools installed. The image is used to test the share service. So ideally it would be the cirros image but afaik that image doesn't contain the nfs-client and cifs-utils.


the formatting was on purpose, since it's one sql-expressed (SELECT * From ExperimentsPo....)


I like the simplicity in this.


No. It's the username of the image from ```image_with_share_tools```


Also nice and simple.


same here


Space after if and (


@mkrueger ^^. Is it removed?


Not quite sure why you get this error as it works here for me if I do not create the stats object.
The stats object is my profiling class.

I'm not too familiar with javascript and all it's magic but somehow that stats thing work here.
Maybe you can get around it by defining the object and those two functions to do nothing?


In tempest, the section is called ```share```. But I'm fine with calling it ```manila```


There is no need for the upgrade variable?


this makes sense.


Move pointer sign to type


If you do the refactoring before adding the new validation then it would be consistent?  Not that it matters hugely for this time, but maybe worth bearing in mind for next time.


Can this be moved inside the only_if? It's more "Cheffie"


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


Debugger didn't work in IntelliJ otherwise. Is there an alternative solution?


If you're touching this file -- we have a different writeall() function you added. Can you remove the one from this file?


@mkrueger: We still have these -enum stock icons defined:

* md-internal-enum
* md-InternalAndProtected-enum
* md-private-enum
* md-protected-enum
* md-ProtectedOrInternal-enum

And mainly:

* element-enum-16.png

Should I remove them?


May be const here and above?


Backslashes aren't a valid path separator for URLs. This causes broken images in some browsers and platforms, eg. Firefox.

For reference:
http://lists.w3.org/Archives/Public/www-validator/2011Feb/0024.html


Hm, now I'm more confused :P @cmr what is you opinion here?


thanks, didn't know about ,it makes it so much easier :)


Thanks I have now

> On 27 Ean 2016, at 09:20, Johnny Shields <notifications@github.com> wrote:
>
> I think they should be added--go ahead and raise a PR.


Nope. The KVM node has no webapp, and compiling the recipe fails. Either this, or add a dummy webapp name for KVM (which is odd too).


It's in WildFly process (using CassandraDaemon) https://github.com/hawkular/wildfly-cassandra/blob/master/src/main/java/org/wildfly/extension/cassandra/CassandraService.java#L119


@AMikhniev , this and next ~10-15 lines is duplicates in next matcher.
Can you refactor and move out this section to method?


Uncomment this?


sure, i actually noticed it thinking this was the one i added.


Well, that's WIP still. As the resource does not actually run, but instead waits on notications, I need to see how to check if the correct parameters are set on the resource.


ah true, I'm always unsure when those ycp -> ruby wrappers takes care of nil and when not.


For what do you need this header?


On Tue, Jan 26, 2016 at 10:56:55PM -0800, pbrunet wrote:
> Or you can use the onle line template evaluation:
> ```
> a = 0
> int _[] = {(a += std::get<I>(from).shape()[axis],1)...}
> return a;`

Yup. Aesthetically, I prefer mine, and the folding operation is the way
to go anyway!


:tada: already updated my `composer.json`


看不到a, 你的a在迴圈裡宣告, 所以迴圈外看不到, 請拉到迴圈外


I meant ... why is such setup needed at all? If you don't read IP from system, why it is correct to setup some magic one?


User.find(params[:id])


Doh, silly typo's!


```rb
module Roman
  def to_s
    # unicorns and rainbows
  end
end # module Roman

7.extend(Roman).to_s
```


А гем aasm не обрабатывает назначение некорректного стейта?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 12983](https://teamcity.mesosphere.io/viewLog.html?buildId=114161&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:03:03


@AMikhniev , don`t use using namespace directives, please write where is this variable/enum defined.


NP: ```AutoInstallRulesClass::DEFAULT_IP```


can you remove the ( ...)
`t = (pt.datafiles.load_dpdp()[:].superpose('@CA'))` should be

`t = pt.datafiles.load_dpdp()[:].superpose('@CA')`

Python does not allow to break line for chain methods, so I use ( ... ) to do the trick

```python
(pt.datafiles
     .load_dpdp()[:]
     .superpose('@CA'))
```


Small thing: I guess this can be moved outside at the top.


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


thanks for `show_pytraj`. It's much easier for users.


`repr_type` is nicer name than `reptype`. thx.


hey please do not add the option *and* the check if the file exists in one commit! Those are two different changes


Mmh. If we are not in the context we shall not set the global variable `valid_keywords`. Otherwise, once finally we are in the context that would be already set from previous calls but without the whole configuration (I know this is more of a theoretical issue :dancer: ) So I guess you can add a flag such as `in_context = True` in the above `try..except` to ensure this behavior?


Commit message is not good. For future please try to describe the reason of changes but not actually changes. `Fix incorrect behavior` is not acceptable description.


@ramapcsx2
```
Also, the ideal way to run irx's is by using a ps2link.elf. This is what people use on the real hardware, when fetching .expected results.
Ps2link.elf runs irx's from host0:
```
Maybe, but my hack was easy to do. But you're free to implement it ;)

@refractionpcsx2
Yep. However, I'm nearly sure code wasn't never hit. It is quite unlikely that game moves some constant in HI/LO register.


А куда пропал ul? Лишки без него не могут существовать)


woot woot.


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


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


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


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

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


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


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


I don't want to beat this up any more though.  Let's leave it.  I can change it at a later point if warranted.


this should be `string.Join(" ", ...)` otherwise you'll get "isan" rather than "is an". First argument is what joins the two strings. ie: a + " " + b


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2054 of 2056 tests passed, 2 test warnings)

 * 6 tests had: AUD diffs.
 * 2 tests had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: Table small diffs.
 * 2 tests had: ERR diffs.

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


Maybe capitalise variable names so the distinction between function names and variable names is clear and consistent with the rest of `brickstrap` code.


Idem.


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 tests passed, 0 test warnings)

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


new line should be added


new line!!


This change is not part of "Fix assert_select assertions in ImageControllerTest", so it would be better to move this change to the "Add the ability to delete a saved image in index page" commit (https://github.com/AppFolioOnboarding/Li_Image_Share/commit/cbe9e3fe61df9f590cccdef4ec505b4cd9593e0a).


is declaring length needed or can either be used without declaration?


The names of these enum values sound a bit too generic to be made public. Should I rename them to `POS_BEFORE`/`POS_AFTER`?


Thanks @Evisceration


@jkasten2 - The library is open sourced! https://github.com/googlesamples/unity-jar-resolver  Let me know how things go!


Your code sucks.


Typically in Rails, we add the methods to edit or delete a resource on the show page for that resource, rather than on the index page.


partial :)


Again though, this change belongs with the "Add the ability to delete a saved image in index page" commit (cbe9e3f).


typo "subscription"


Or maybe make scope a hash set is prob better


I understand why you did that, but can't really rely on that conditional behavior, also we need to set the captcha field in the Meta.fields , this is required for django >= 1.7.


Why this was reverted?


It didn't have nice things to say about you either :-P


Hi m,
you could try the patch https://github.com/gfreewind/linuximq-HighPerf/blob/master/kernel/v3.x/linux-3.2-imq.diff


I do not have a Mac on my hand. But from my understanding `make csharplib` should output a `libpdcsharp.dylib` which can be used on OS X. If so, then please uncomment this line.


The problem is due to the use of different Design Coil Capacity/Load for auto-sizing water flow rate and the UA-Value of the hot water heating coil in ZoneHVAC:UnitHeater object. The Design Coil Capacity for zone equipment such FanCoil and Terminal Single Duct Objects is recalculated using the design water flow rate and the hot water loop design deltaT. Thus the hot water heating coil UA-value is sized using the same load/capacity as the one used for design water flow rate sizing.

On the contrary the unit heater's water heating coil recalculates Design Coil Capacity based on the design supply air flow rate and entering and leaving design air conditions just like an air loop equipment, which accounts for the load due to outside air. As the result when there is significant outside air fraction (like this defect file case), the unit heater hot water coil entering design condition is skewed toward outside air, which leads to very huge Design Coil Capacity compared to the Design Coil Load and became a bad starting condition for COIL UA-value iterative solver, Regula Falsi. Consequently the UA-Value auto sizing calculation failed for one of the zones, which had large amount of outside air.

This can be corrected by allowing the unit heater hot water coil UA-value sizing to recalculate the Design Coil Capacity based on design water flow rate and the hot water deltaT just like the other ZoneHVAC equipment (e.g. FanCoil). Now, the defect file has completed the sizing calculation successfully and for annual run. The auto sized UA-Values look reasonable compared to Design Coil Load. I pushed the branch to remote.


While it won't make a difference right now, it'd be better to use `destroy` instead of `delete` here because `destroy` will run any attached callbacks (which we may add in the future without remembering to change this line).


Also see [Issue 122](http://github.com/weidai11/cryptopp/issues/122).


Reading the code it looks great, I'll give it a go tomorrow :)


@lagbox whose side are you on, anyway?!


Maybe we should store this in `flash[:error]` to communicate that we _failed_ to delete the image.


Tras la revisión con el tutor, se ha realizado una nueva versión de la memoria con las siguientes modificaciones:
- Modificación de portada (se incluye todo lo mencionado en los Comentarios para mejorar los trabajos 15-16)
- Mejora de estructura  y organización del documento.
- Se han añadido apartados que no se incluyeron en la entrega del día 26 de Diciembre tales como: Indice de figuras, indice de tablas, una página al inicio del documento donde se especifican enlaces importantes (por ejemplo, a los repositorio/s de código del equipo), gestión de la construcción e integración continua, conclusión y bibliografía.
- Se han añadido a todas las figuras una leyenda y sus correspondientes referencias.


Lorem ipsum dolor**** sit amet, consectetur adipisicing elit.** **Odio reprehenderit quaera


good change


Woops, added the spaces within the string in other file, forgot to add them here. Good catch btw! =)


> 所有人在phaseChanging都有机会触发，只是在回合结束后执行时按座次，有的人并不能真的执行额外回合。
> 那么 就算 已经hasExtraTrun， 但在phaseChanging时，这个 回1血和摸1牌 还是应该执行的？

其实这个我的理解是，所有人都可以触发，不过当存在额外回合时，只是不执行额外回合的效果。
可以看我之后的一个commit。

除了死欲妖梦之外，其余的额外回合效果都是被干掉的。


# Assignment 3 Grading Rubric (15 points total)

## Task Set 1

- [ ] Regex matches entire dinucleotide repeat (AT)5 and only the repeat (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different from above (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different both above (1 pt)
- [ ] Regex correctly matches all instances of `R` or `S` (1 pt)
- [ ] Regex (find and replace) matches all instances of `R` or `S` and replaces them with `N` (1 pt)

## Task Set 2

- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 4,3,2,1 (1 pt)
- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 1,2,4,3 (1 pt)
- [ ] Regex (find and replace) correctly replaces commas with tabs (1 pt)
- [ ] Regex (find and replace) correctly gives us the horrible format requested (1 pt)

## Task Set 3

- [ ] Regex (find and replace) correctly parses out only the lane number from the header (1 pt)
- [ ] Regex (find and replace) correctly parses out only the flowcell ID from the header (1 pt)
- [ ] Regex (find and replace) correctly parses any flowcell ID from the header (1 pt)

## Task Set 4

- [ ] Regex (find and replace) correctly extracts only the serial number from the fastq sequence (1 pt)
- [ ] Regex (find and replace) correctly turns the fastq into a fasta having the requested format (2 pt)

## Task Set 5

- [ ] Regex (find and replace) correctly extracts the (AT)5 repeat (Regex bliss)


In general, it's preferable to use the human-readable symbol equivalents for HTTP status codes (for `303` that would be `:see_other`.

However, we typically don't use `:see_other` when redirecting after a `destroy`.


Are there plans to make this package work with more recent versions of Julia ? (i.e. > Julia v0.3) Thanks. @stevengj


01-27 17:10:58.948 21389-21389/michael.mobilecomputing.com.ereca E/AndroidRuntime: FATAL EXCEPTION: main
                                                                                Process: michael.mobilecomputing.com.ereca, PID: 21389
                                                                                java.lang.IllegalStateException: Could not find method insertSpeech(View) in a parent or ancestor Context for android:onClick attribute defined on view class android.support.v7.widget.AppCompatButton with id 'btn_insertSpeech'
                                                                                at android.support.v7.app.AppCompatViewInflater$DeclaredOnClickListener.resolveMethod(AppCompatViewInflater.java:307)
                                                                                at android.support.v7.app.AppCompatViewInflater$DeclaredOnClickListener.onClick(AppCompatViewInflater.java:266)
                                                                                at android.view.View.performClick(View.java:5191)
                                                                                at android.view.View$PerformClick.run(View.java:20916)
                                                                                at android.os.Handler.handleCallback(Handler.java:739)
                                                                                at android.os.Handler.dispatchMessage(Handler.java:95)
                                                                                at android.os.Looper.loop(Looper.java:145)
                                                                                at android.app.ActivityThread.main(ActivityThread.java:5972)
                                                                                at java.lang.reflect.Method.invoke(Native Method)
                                                                                at java.lang.reflect.Method.invoke(Method.java:372)
                                                                                at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1399)
                                                                                at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1194)


Done


Ultra picky, here, but with the following `JLOG` do we even need the `if (j.trace)`?  I'm guessing we can remove the condition and the braces.


_改版本号的用途是？_


Well javascript talks, we were hanging out last night and that is what it said.
It was a one time thing, wont happen again.


Instead of calling `Image.first` here, you could store the result of `Image.create!` above into a local `image` variable and use that here.


Whats this supposed to do?
It will write wrong value.
There is no greeter.conf, the ls command is supposed to detect with the greeter.


@DanielKehoe  hi can u see this ? XD


这个地方应该先moveToThread？？

```

	p_ptr->effect = new CSoundEffect(fileName);
	p_ptr->effect->moveToThread(p_ptr->thread);

	connect(p_ptr->effect, &CSoundEffect::started, this, &CSound::started);
	connect(p_ptr->effect, &CSoundEffect::stopped, this, &CSound::stopped);
	connect(p_ptr->effect, &CSoundEffect::loopsChanged, this, &CSound::loopsChanged);
	connect(p_ptr->effect, &CSoundEffect::loopsRemainingChanged, this, &CSound::loopsRemainingChanged);
	connect(p_ptr->effect, &CSoundEffect::isPlayingChanged, this, &CSound::isPlayingChanged);
	connect(p_ptr->effect, &CSoundEffect::volumeChanged, this, &CSound::volumeChanged);

```


Instead of creating an image and then trying to delete an image with the subsequent id, you could simply pass `-1` for the id.


Minor, but I'm not sure that "required" adds value to this method name. I would've gone with `find_image`. But it's not a big deal.


What does this button do?


sorry, my fault :(


I tried sending duplicates to Google, Facebook, and AAD and they didn't actually notice. Have you seen duplicates cause problems with any providers?


running npm update (which updated to webpack@1.12.12) fixed this; i'll add it anyway for others who haven't updated


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


nice refresh button ;)


Ah, I see that you switched to `destroy!` in a later commit :+1:


I was just wondering where this went. Is this something we can push back to proper?


This is correct and works. **lightdm.conf** is the basic config and **greeters.conf** is used for settings not covered by **lightdm.conf**. There are manuals for it. Did you check some content of **greeters.conf** ?!?


Generated javadoc


You could rescue the `ActiveRecord::RecordNotFound` error here and display the error flash that you've deleted in this commit.


We need to look in /sur/share/xgreeters instead, to detect what greeter is used to write the greeter to lightdm.conf


echt schön, hätte es nicht besser machen können ;)


Somewhere in this commit is a change which breaks the pipemenus.


changed to trace level


One example file (UnitHeaterAuto) with auto-sized input fields showed diffs. The fix forced the hot water heating coil to use a Design Coil Capacity/Load calculated based on the design water flow rate and the hot water loop deltaT.  This design coil capacity is different  from a value that can be calculated from supply air flow rate and design inlet and outlet air temperature of the coils. For this reason, the Coil design capacity and UA values after the fix will slightly change for autosized coils in UnitHeater objects.

I will add unit test for this fix.


Why did you unintentionally  delete code from the LoaderAngle.cpp file.
Make sure you  sync before you push.(I think those are the right words)


Prefering /usr/bin/systemctl to /bin/systemctl seems reasonable to me.

Again reminding me that it would be convenient to be able to define a search path within the for commands.


Do not think it's a good idea to log the password at any point. BTW this is also an info log.


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 tests passed, 0 test warnings)

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


sweetness, just get @gregllong and @jayich  to check this and were good to go..


i can destructuring even this one? If yes how?
I wasn't able to do the same on `NavBar.js` cause that `getLocals()` is a bit to complex.


You misunderstand the code I think.
again, the ls command is supposed to list a greeter.conf, and it will use the filename without extension to write the necessary lightdm value entry, eg to set it to lightdm-deepin-greeter.
With your changes, it write a wrong value, simply 'lightdm' which won't work.


I dont think this one would go into the MPF, probably needs different re factoring to support this restart using different service names. something from def might make sense.


hehe


I'm stuck right here!


Idk what happend here


Is it possible to add a text box next to the Path to add command line options?  I am using the Windows version and would like the place the -home directory on the D: drive so I can image restore the C: without wiping out the sync database


yes, otherwise you get unstripped libc.so in initrd


@leudanielm provider is missing from component
```
@Component({
...,
providers: [MyAppService]
})


ConcurrentHashMap instead of `Collections.synchronizedMap()` ?


@pestOO I can do you say if you insist. But IMHO impicit cast is bad practice


@gabrielamatias look not looks


I'll change for `utf-8`, thanks for pointing this out. In fact this encoding is quite transparent for end user, because we call decode just after, but `utf-8` would support more chars that won't be supported with `latin-1`.


Reflection always gets you in trouble, I suppose. Surprised no unit test was present there. I will add a unit test for `Project_v2` to capture the regression.


@leudanielm contructor should have the injectable service as arguments
```
constructor(myAppService: MyAppService) {
this.myAppService = myAppService;
}


On RhiotCloudPlatform class I totally agree ... I'll remove it.
I'm thinking about a simple but meaning example. To show the service binding in all its features, the example should have request/reply style. In IoT, from a device point of view it's the "inquiry" pattern (used by devices to get useful information from the backend). We need to have a meaningful example or just an "hello world" ?


Ну ок.
> On Jan 28, 2016, at 12:57 PM, Alex Zolotarev <notifications@github.com> wrote:
>
> Чуть-чуть должно быть отражено в теге - [save_chut]
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/therearesomewhocallmetim/omim/commit/e47ec7d892d97c2fc8d6d210d5eae1d136b67763#commitcomment-15728294>.
>


@leudanielm here, you'll use your private myAppService instantiated in constructor


RE https://github.com/andytill/erlyberly/pull/68#issuecomment-175512190

erlyberly already supports reapplying traces when a node reconnects. The traces are stored in `DbgController#traces` which are being cleared here.


is this required


Snx for attention! =) Podfile.lock must be placed under git due to all contributors should knows what is accepted version of each pods was in last working commiti for example.
And you probably need to check official guide (http://guides.cocoapods.org/using/using-cocoapods.html) and gidhub gitignore cookbooks (like https://github.com/github/gitignore/blob/master/Objective-C.gitignore) and means of others (http://stackoverflow.com/questions/9446644/what-goes-into-your-gitignore-if-youre-using-cocoapods). Good days!


TeslaCrypt generates an ECDH shared secret (ECDH itself isn't an encryption method, only a key exchange), then multiplies it with the AES key. This multiplication-as-a-cipher is certainly inspired by RSA, but it's weak precisely because it's not true RSA.


@gabrielamatias the hyphen should be written as "–" instead of "-"


Ah, yes of course.  This has been fixed.
Check [latest](/ankostis/TeslaCrack).


In fact we don't support explicitly oauth. I would have said:
Support only password. The client can also use OAuth by providing a header in the request such as this:
Authorization: Bearer JWT_TOKEN


Wow, I never thought this could be implemented in so few lines of code.


> Your site is a good case for the ones that need long timeouts for request processing, as it is using long polling where requests do not produce data for a minute. How would you prefer to configure this?

The less directives and options the better.

Unless it really breaks things.

So far it seems that equalizing Timeout and H2KeepAliveTimeout really worked, no issues visible right now.


@gabrielamatias same as above, the hyphen should be written as –


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13168](https://teamcity.mesosphere.io/viewLog.html?buildId=115417&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:18


@gabrielamatias what is the <strong> bit for?


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 103](https://teamcity.labkey.org/viewLog.html?buildId=299765&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 18 (2 new), passed: 275, muted: 1 Build time: 01:44:18
### Failed tests
```
org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.tests.CDSLoginTest.testSessionTimeoutIntercept: <no details avaliable>

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

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

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

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

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

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

##### there are 8 more failed tests, see build details
```


1. Я против добавления не используемых строк. Они никем не ревьюятся и забивают место. А к следующему релизу Наташа заново их переводит, потому что все забыли, что они уже были вставлены в код.
2. Где конкретно запланирована эта строка в нашем UI? В каком кейзе в будущем мы ее будем вставлять?


changed the name to runPolyEvalAlgorithm


@gabrielamatias can these bullet points have full stops at the end? Just checked the support ad and the daily tasks there have them, so we should probably make them all the same - sorry!


From when and what version is NOW() supported?, I see a clear risk of us not supporting older versions instead here.


Can someone please give me the explaination for calculate sigma function asap? from where did you get this formula for computing sigma?


I would make IoT-flavoured hello world :) Can can have two operations in service. One void for semi-telemetry example (like `void storeTemperature(String device, double payload)`) and the second one for some-inquiry, for example `double temperatureTreshold(String forDevice)`.

What do you think? :)


[Uploading emptySeleniumAndCucumberProject-master.zip…]()


В finished наверное


"Locked" is a misleading term, as it sounds like locking a file for exclusive access (a common Windows annoyance sometimes exploited by malware). I would go for cipher/decipher/ciphertext, if we already used plaintext once.


Mind you, I have no idea how common NOW is vs TIME, but historically, Fusion always used TIME and I have never heard an issue with that. We need more info.


TeamCity Civic Assembly :: Build backend + frontend [Build 32](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=525&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 21 (10 new), passed: 1719, ignored: 7 Build time: 00:20:40
### Failed tests
```
Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blog categories tests: Should delete blog category new blog category 1453975470767: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

Blogs tests: Should delete blog: <no details avaliable>

Document tests: Should take a alert when try to upload new version for locked document: <no details avaliable>

##### there are 11 more failed tests, see build details
```


Based on @remialvado's PR on the old client repo (https://github.com/neoxygen/neo4j-neoclient/pull/76)


don't import from sickbeard for a library... it's either a library or part of the program.


Wrong не катит уже?


TeamCity pull requests :: netty [Build 1127](https://garage.netty.io/teamcity/viewLog.html?buildId=5140&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6390, ignored: 23 Build time: 00:42:22


@scottdavis Sorry if I've got my wires crossed and you're not the maintainer of this project !


Maybe I'm misunderstanding the point of this change, but wouldn't it be better to just update every instance of `fedora-21` to `fedora-23` in the text?


Its fine now :)


ok


Some fixes and refactoring
@Zoltrix @MohamedBassem @saherneklawy


Seems like this section-1 is a hidden system file. Renaming it prevents code from showing up.


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


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


@dotsdl

1) So with the `.ts` attribute, it's just like any other property you can slice out.  Once I create the array `ag.names` it's just a snapshot of the names at that point, and if I change the array it doesn't go back to master, or if master changes my snapshot is stale.

2) Can just hard code them for now and do something elegant later

3) Yep not a problem

4) It would be nice to have something like `from MDAnalysis.extensions import proteins` which loads in a lot of extra stuff to your namespaces.  But for now, like 2) let's just get things done


Changements à faire sur le serveur pour lancer puma correctement en plus des deux fichiers de config de base (`/var/www/config`).

```diff
#/etc/service/puma/run

- cd "/var/www/app"
+ cd "/var/www/app/current"

- PUMA_ENV="../config/puma.rb"
+ PUMA_ENV="/var/www/config/puma.rb"
```


You can use explicit concatenation for the same thing.  And you probably should, since with any luck, implicit concatenation will be removed from the language at some point.


needs to be 2.7.3!


The need is this:  we must have a nice way to name the `.vvv` files and their respective undecrypted ones.
"Undecrypted" does no sound good, at all good, and "deciphered" sounds a bit alienating for the user, although agreeably is better.

If you read the docs + source, they all are read much more natural now (no wonder they had named CryptoLocker this way :-)
Try mentally doing the same with ciphered/dechiphered, and if ok, I will change it in the afternoon.


Why use `transaction-` prefix on property names here?

Why not just `type` and `amount`?


@anosach-luxoft, have you followed provided link?
There is a `bool operator== (const string& lhs, const char*   rhs);`. No casting or converting to `std::string`.
And No, I do not insist.


@rogerhutchings what's the difference getCenter and getPosition?


damn copy/paste..


**Code Quality: Nitpick**

Lots of commented out code within your codebase. I would actually prefer less comments overall.


For the files without read-access, I wouldn't call them "locked" but just "no access" or something similar.


Repo


Test


@nebhale Nice catch. Also spotted a missing test of a similar case.


My view is on the opposite: I think the dependency on `react-native-system-notification` isn't necessary for `react-native-gcm-android`, but `react-native-system-notification` should rely on `react-native-gcm-android` for its push-notification feature, as sending notifications isn't necessary on using GCM, but using GCM is required on sending push-notifications.

@oney how do you think?


**Code Quality: Critical**

It's surprising to see that you didn't split your tests into the respective contexts. This is not very good practice especially since we have stressed on how important this is so far.


love this new sentences


All good, issue was resolved, will be put on prod ASAP.

> On Jan 28, 2016, at 10:39 AM, Emiliano Busiello <emiliano@relayr.de> wrote:
>
> So, both AtomiLong and synch make rabbit work as expected, now the problem shifted to babylon not processing the request somehow.
>
>> On Jan 28, 2016, at 10:33 AM, Mike Bush <notifications@github.com <mailto:notifications@github.com>> wrote:
>>
>> It would be a problem if there were other uses of the variable which were not synchronized - but as this is the only use the synchronized block guarantees that other blocks synchronized on the same object will see its changes.
>>
>> https://docs.oracle.com/javase/tutorial/essential/concurrency/syncmeth.html <https://docs.oracle.com/javase/tutorial/essential/concurrency/syncmeth.html>
>> when a synchronized method exits, it automatically establishes a happens-before relationship with any subsequent invocation of a synchronized method for the same object. This guarantees that changes to the state of the object are visible to all threads.
>>
>> Random would do - but it still uses AtomicLong - http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/6-b14/java/util/Random.java#Random.next%28int%29 <http://grepcode.com/file/repository.grepcode.com/java/root/jdk/openjdk/6-b14/java/util/Random.java#Random.next%28int%29>
>> —
>> Reply to this email directly or view it on GitHub <https://github.com/relayr/rabbitmq-scala-client/commit/f27cf43a51546695d0414c0698d31f0b4e26df65#commitcomment-15727728>.
>>
>


Yes, masking isn't enough. But it was easy to do and it does help some DMA tests. I was afraid that both source&destination need wrapping. But actually I guess only the spad memory need it. I guess they just wire the 14 lsb bits for the address. I hope DMAC can't do transfer bigger than 16K.

We also need to handle a special msb bit of madr for SPR channel.


@gabrielamatias fast-paced still needs a hyphen in it


All mem(bers, ory) is zero-initialized. Needs to be documented somewhere.


Hi Nicole,
why don't remove that switch completly - it was intended for different cases which never occured.


Just a note, this bump of patch version broke our build. We're using a module loading and were checking `window.deferredBootstrapper`, this change fixed AMD and now it's null.

Maybe our case is rare, though it might be worth reverting, releasing as `0.1.10`, then revert-revert and release as `0.2.0`


Hi, I see that you implement a sequential SfM toolchain.

You know that you can simulate a similar behavior by using main_ComputeMatches with the contiguous pairs option => -v https://github.com/openMVG/openMVG/blob/master/src/software/SfM/main_ComputeMatches.cpp#L105

And run classic IncrementalSfM on the resulting geometry filtered file.


@Strawhatfy i had reviewed your code, and has some questions.

### First.
 What if there is a chance `self.on_result` get called twice?

Imagine a use case, celery worker just set the task result in redis backend, and publish it.

The call `self.io_loop.add_future(gen.Task(self.client.get, key), _check_subscribed)` comes, and get a result back.

Then redis send out the pub event, waking up the subscriber, which you have defined in your code like ：
````
        self.io_loop.add_future(gen.Task(self.subscriber.subscribe, key, current_subscriber),
                                lambda future: _set_subscribed(future.result()))
```

In your code, such race condition would happen and make the call `on_result` got called twice.

#### Second.
 when the timeout fucntion get called, there is no way to notify upper logic that the request to celery has broken.Would that be better to provide a `raise` like exception?


@osaris it's alive, _alive!_

Mais j'ai un problème avec le mot de passe de la base de donnée qui provient des variables d'environnement. Et ce coquin de `cap` ne semble pas y prêter attention. Tu as une meilleure idée que le `set :default_env` de ci-dessus?

**NB** il existe également un utilisateur pour le groupe qui pourrait être partagé par les membres à condition de mettre les bonnes clés SSH aux bons endroits.


because otherwise I have to start from zero with the others. (This reverts a substantive change)


Интересно обсудить совместно с декораторами.


Boas...
Só vendo sexta feira...


First, you are mixing spaces and tabs. Second, in Python3 str doesn't have the method "decode" any more. The line must be changed to: https://github.com/wwmayer/FreeCAD/commit/709718d1e9f1389f7a35814ece9d41b66325b30e


Well, mac ports != homebrew, it is a different package manager.


hm, How to add the new Item in the creative economy?
ex) flower codes-38:1 38:2 ~ 38:8


Hier aber dann auch len + 1


Genau so hier len + 1


Seems you accidentally added the log to the commit


I'm getting `File mode specification error: (void-variable flycheck-swift-sdk-path)`. I did some investigation and found that only byte-compiled by cask version of the swift-mode has this error and manually recompiling with flycheck loaded solves it.


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


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


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


I guess the question is whether we go for the packaged versions in Debian and Ubuntu, or for the stable versions - I'm a bit inclined to the former as practically, that's what we recommend people use.


det er ingen spesiell grunn til det.
Godt forslag


@gabrielamatias should be a full stop after 'world'


TimeUnit.SECONDS.sleep(2 * verySmallTimeoutInSecond)


I'm not sure this is a good name for this. This is misleading. The EXPORT_DEVICE can be any of the N disks:

>  EXPORT_INDEX  which  denotes  the index  in  the  instance  disks  structure (and could be used for example to skip the second disk if not needed for backup) and EXPORT_DEVICE which has the same value as DISK_N_PATH but is duplicated here for easier usage by  shell scripts (rather than parse the DISK_... variables)

 Naming it disk0 seems as if this is the first disk. I would suggest completely removing it and losetup-ing EXPORT_DEVICE in the export script


Yes, that was actual for the previous version of CocoaPods. Nowadays you can point it right in the `Podfile`:

```
pod 'SDWebImage', '~> 3.7'
```


This is wrong.
Weapon ammo is serverside. As a result, server thinks you still have your old ammo value, but you think you have ammo = GetClipSize(). So your client might think that it is shooting, but actually it is shooting ghost bullets, server is just going to ignore them.

Do:
Shoot your whole clip. 0/120
Go to tent. 30/120
Shoot a single bullet. 29/120
Reload and look at your ammo value server sent you. 30/90

If this would work, you would get 30/119.


Same as above. (see EXPORT_DEVICE)


this is a test


TeamCity Civic Assembly :: Build backend + frontend [Build 33](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=527&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


local variables path and uri are missing


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