Actually CR+LF can be recognized by ygopro, but it will prevent git to diff. So we should keep an eye on line encoding on saving, or use the [autocrlf](https://help.github.com/articles/dealing-with-line-endings/) feature of git client.


@pickfire @bucaran ah okay then. That's a pity.


:+1: This will fix the Coverity issue I assume.


I think you meant `(+1 (rand-int 4))`; I'll do that.


on - > with

Please add Dagger2


f.isAccessible() is not sufficient ?


@akrahdan do you mean in the keystone site project? maybe open a new issue with more information?


That's more an instruction for development.

I'll update it to reflect more general usage, thanks.

I do get notifications from this repo, so I think I've got it set up in a way that works for me.


fixed with commit 9f1d7ee


谢谢


Oh nevermind, it changed in Python 3.


hm, so there might be a problem when using the drag&drop feature git introduced a short time ago?
I don't recall any diff problems bevore trying that out...


remove the pragma now that you've got a test


maybe file an issue about obeying cache here?


Coursera folder


Not needed, `curl` is installed by PackageCloud script.
So after "setup repositories" step it's available in both deb/el systems.


同问readme要写什么- -


Personally, I don't think this makes sense for the general public.

You could add a new option to pass an attribute or something, instead.


Hmm, this is a bit incorrect: the condition for getting paused is when the next bytecode opcode is dispatched. This can happen if:

- The Duktape/C call returns into an Ecmascript activation. But note that all Duktape/C function returns won't cause this, if there's nothing else on the call stack, or if we return to another Duktape/C function.
- An eval() or a duk_(p)call() happens with an Ecmascript function as a target.

I'd suggest maybe:

> The target will pause the next time it enters the bytecode executor, i.e. attempts to execute Ecmascript function code. This may take some time e.g. if a Duktape/C function call is in progress, or if nothing is being executed right now.


yes, everything is warning in their case http://www.scalastyle.org/rules-0.5.0.html.


Similar wording change here.


dfgsdfgsdf


De esas 27.371 lineas 27.360 son de librerias incluidas en el bower para el autocomplete.


For me this is OK.
I think we should get this merged as soon as possible to get a full German version in.
Finetuning can happen later on.

Concerning the unknonw context of the following message.

-#, c-format
+#, fuzzy, c-format
 msgid "MAME %s ( %d / %d machines (%d BIOS) )"
-msgstr ""
+msgstr "MAME %s ( %d / %d machines (%d BIOS) )"

I'm not sure. I think this is a kind of summary. I wouldn't worry about the context too much, though.
It seems straightforward to translate to me as.

+msgstr "MAME %s ( %d / %d Maschinen (%d BIOS) )"

Please note that there still seems to be a "fuzzy" before the , "c-format" ?


Hi. Are you sure that it is a good idea to rename the pythonw exe, and why?


this would be nice.


ack


Every time when I think I understood half of what you're doing within M2 you're proving me wrong. I would have created a new SPMeta2.Validation.Validators.Definitions.BooleanFieldDefinitionValidator because I expect this to be the right place.

However this is way more than I would have thought about. Nice good and approach!
This is not available in dev right?


Itu commit message-nya "tra" maksudnya apa?

Btw ingat untuk merge ke master kalau branch DEV udah cukup selesai. Sebaiknya si sesegera mungkin.


I don't think you need a mysql setup, do you?


Do not use root. It's not secure.


no need to download the source it is already mounted in /vagrant folder


@webteckie might be worth including an inline comment when you do this, so it's clear when browsing the test code why it's been commented out.


To merge this pull request please add -
* Integration test that verifies this parameter
* Documentation (see Status/Routes)


Isso não pode gerar breaking changes?


OK. works now.... Thanks.


exactly that's where I tried that drag & drop out, that's why I'm asking^^
take the one I made previously to that
https://github.com/Fluorohydride/ygopro-pre-script/commit/4d0b34b891ec84ec7f7d0c9d90b2c9bebf902fdb
everything seems fine to me


shouldn't this line now be removed?
model.add(Dropout(0.5))


Please add "Fixes: #14402" to the commit message.


I believe AP_Parachute is common to both Copter (it's included in `ArduCopter/Copter.h`) and Plane, and this code is for Plane only (Copter has its own parachute emergency release logic), that's why I placed it here and not in AP_Parachute.

I was following the same logic as with some other plane-specific parameters (like `VTAIL_OUTPUT` etc.) that are located in `ArduPlane/Parameters.cpp` and also the same is done with Copter crash check parameter `FS_CRASH_CHECK` in `ArduCopter/Parameters.cpp`.


Isn't ml_flags long not size_t?


Well, which git client are you using? I'm using https://git-for-windows.github.io/ and I didn't found drag & drop feature in it.


ajout fenêtre statistique


I think technically the whole terminating NULL PyMethodDef should be added, as it would be indexed as an array in C and any member could be queried for NULL (although I guess in practice the current implementation of the interpreter only checks the first member, but it would be safer not to assume that).


oops,protocol version


Hello @jarekratajski, hat is this method for? I understand all the other methods, but not the idea of the method `but()`.


I'm just finished integration with Clickmeeting.
I need it for this function
http://dev.clickmeeting.com/api-doc/#delete_conferences_by_room_id

It works.
It would be great to improve this code for full DML SELECT,INSERT,DELETE. But I was limited by time.


In this case form.save() must be initiated manually. Probably UpdateView would take care of this automatically.


харош, дякую


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


yes, I'm using that too
but I don't think that is connected to the git client itself ... a short while ago I got the message here, that drag&drop is supported now, you can access that with the "upload files" button between the new file and find file buttons


шо за беда?
воно ж не дійде до клієнта а просто зїсться


This is for Plane only, that's why I didn't want to place it AP_Parachute.

I'm not sure what the consensus about library structuring is, but moving this to a library would make more sense if libraries for ArduPlane parachute and ArduCopter parachute were separate. But that would mean that manual parachute release for plane (that's already in this file) would likely go there too.

Also, since @tridge started this file, and did so having automatic emergency parachute deployment in mind, maybe he has some vision of how this should be structured.


Ми ж наче з АО говорили що той патерн що на фронтенді перевіряє не правильно робить, хочеш запитай
Він попросив юзати цю лібу, було ю добре повернути яку було


This causes the problem described in #18 (Which was opened after this commit!)


So either your browser or github fail to parse LF correctly..


я про це

в методі createKeywords(keywords, bookEntity); не буде нулу і помилка не впад


` if (websocketStream != null) {
websocketStream.close();
}
this.listeningChannels.forEach((channel) => {
workspaceBus.unsubscribe(channel);
});
this.listeningChannels.length = 0
if (channel != null) {
bus.unsubscribe(channel);
}`
can be moved to separate method and used both onsuccess and onerror.


Yeah, at least, a TODO or FIXME would have been good...my bad was late and wanted to shutdown my computer after a long day :-)  It's still fresh in my mind and plan to look at it.


I see. Is it enough to just do an apt-get install Moonlight-embedded after applying the script?


No worries :smile:


This won't work. Try yourself:

#### 1.) Create test.sh script

```sh
#!/bin/bash

set -e

fail() {
  echo "############### ERROR ###############"
  echo "# Failed on $1 #"
  echo "#####################################"
  exit 2
}

test() {
  non-existent-command
  echo OK
}

test || fail "test"
```

#### 2.) Run it
```sh
chmod +x ./test.sh
./test.sh
```

#### 3.) Output
```sh
$ ./test.sh
./test.sh: line 16: non-existent-command: command not found
OK

# success exit code
$ echo $?
0
```


We don't generally indent inside macros IIRC (see the astyle config)


Polskie znaki TYLKO w komentarzach, nigdy w nazwach funkcji, zmiennych.


The GPL-template goes here.


Szybciej stworzyć tablicę wg takiej konstrukcji:
```js
var tab = [5,7, 3, 1, 8, 2, 0, 4, 9, 6];
```


that seems to be the case :/
anyway, I've seen for myself what that does and I wont use it anymore, just pushing via the git client isn't too much more effort than that^^


Why not just add a virtual call from the super-class to a specialization here? (Make the super-class' implementation empty)


Jakiś problem z wcięciami ;/

Słabo czyta i analizuje się taki kod. Dbaj o poprawne wcięcia.


Not really neccessary, the loop below also checks it, and the return at the end will return nullptr then.


Nie ma czego takiego jak `Javascript`, poprawna nazwa to `JavaScript`.


const Common::String& might be preferable in general.


Do you actually need this?


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


Might be good to put this constant inside the class, as it's not used outside it, and well, the name is a tad generic for being in the "global" Wintermute-namespace.


The general style is
} // End of namespace Wintermute

See all the other files.


Any reason all of these are macros and not string-constants?


We do have smart-pointers available (see common)


Typealias bedeutet?


'*' aligns to the variable, not the type.


OpenSSL is only needed by the CsNetwork component, this can be move to https://github.com/copperspice/copperspice/blob/cs-1.2/src/network/CMakeLists.txt#L27 in else() condition to avoid massive rebuilds when the OPENSSL_INCLUDE_DIR variable changes.


what are you trying to fix with that ?


I like it too, but I've noticed it would make a dead link on this page : http://data.okfn.org/data/core/s-and-p-500-companies
Any idea to improve this ?


This should probably check for if(CMAKE_CXX_COMPILER_ID MATCHES "AppleClang") since this is a compiler quirk, not exactly platform.


hey zocker, can we chat over to the chatroom?

https://gitter.im/thirdy/durian?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge


OK, I figured it out. It's at https://github.com/SMSSecure/SMSSecure/pull/353


lets be consequent and use "let" instead of var. Also i would prefer to prefix jquery objects with '$':

let $qr = this.$(".qrcode")


$qr.empty(); seems to be faster then $qr.html("")

according to http://stackoverflow.com/questions/17543711/jquery-difference-html-vs-empty


Thx for the request!

In which cases does the re-rendering cause problems?


Changed from 20 to 50


No, it`s bad idea..


The definition for the Mac backend sets QT_NO_OPENGL, are you sure this is needed and it's not other problem? If it isn't and OpenGL component is absolutely needed then QT_NO_OPENGL should probably be removed from https://github.com/copperspice/copperspice/blob/cs-1.2/src/phonon/CMakeLists.txt#L430.


I would want node_modules in .gitignore. It's a more reasonable default because I don't think anyone should want to add all files in node_modules to git.


Yeah, that's my bad ^_^


@pitr-ch this is failing so I'm moving it out of the way while JRuby try to do their 9.1 release.


@jruby/truffle MRI tests now use (not test, actually use) refinements :(


Why did you move that?


fixes #1590


> **Naming**
> * Use `snake_case` for methods and variables.
> * Use `CamelCase` for classes and modules. (Keep acronyms like HTTP, RFC, XML uppercase.)
> * Use `SCREAMING_SNAKE_CASE` for other constants.
>
> https://github.com/styleguide/ruby

Et faire un `Stage.all` avant un `redirect_to` est un réel gaspillage.


Please, move that before or after the the other cmake_policy() and leave a comment why it is required. Also, can those warnings be fixed rather then setting the policy to old behaviour?


Thank you for the comments. It's very very early to be thinking about those (or, for that matter, indentation), though :)


I am in favor


> Returns true if the collection is empty. If the collection has been loaded it is equivalent to collection.size.zero?. If the collection has not been loaded, it is equivalent to collection.exists?. If the collection has not already been loaded and you are going to fetch the records anyway it is better to check collection.length.zero?.
>
> http://api.rubyonrails.org/classes/ActiveRecord/Associations/CollectionProxy.html#method-i-empty-3F


What about `disableSync` option and ask community to test it?


when will be next release?


Is this for voltage?
https://github.com/koquantam/android_kernel_samsung_vivalto3gvn/commit/c34e5145970f41d6a268b22263569fb3c104133b#diff-193a33780e58edcb1777afecd69c7513R205

You can also implement io shedulers and governors from here. Working good.
https://github.com/Y300-0100/android_kernel_samsung_SM-G355HN_XEC/commit/18149dd80989de42e40e556e1d708ffd2435b049


I toyed with the idea, then decided to play it super safe and keep the original class untouched.
I was also considering that this way we avoid a few indirection steps in the interpreter.
You think it's much better the other way around?


and... why?


It's considerably ugly.
On the other way, WME wants a char* (*not* const char).
What would you do, please?


ændring af filen


This is stock kernel. I've commited wrong tree
I've added some IO schedulers in this kernel: https://github.com/koquantam/DumbKernel_vivalto3gvn


Ok. I can grab a copy of the older image and get the size.


Thanks for the heads up.
Don't mind too much these (UI-side) files for the time being, though, they are basically still the old and ugly ones.


skriv


Maybe just use class QPair { public: .. } in https://github.com/copperspice/copperspice/blob/cs-1.2/src/core/tools/qpair.h#L34?


Thanks; I'll simply feed them into astyle when the time comes.


Well done for this tedious work!


Not best solution for smaller screens... Items don't hide after 100%width (site width is now ca.600-700%)


@somaen thank you very much for the comments. Anyhow, I'm redoing the way watches are handled with a publish-subscribe relationship (or listener, or what you like to call them) from the engine to the script which should end up being less... ugly and spaghetti-ish.


ok, added your changes with TODO comment for PUT/POST.
Thanks for the patch :)


@somaen P.S.: Apparently my provider is down and my landline will be fixed no sooner than Monday, so forgive me if I don't get around to answering you before then


Why would you want null contexts?


Ok. How about the FIXME:?


By the way - in the end I'd like to enable/disable this in the ./configure or something, so that cowar^N^N^N^N^N regular users don't get it if they don't need it.


I think there is a typo double slash in the URL. If you fix that and squash all PRs, I'll merge and backport this docs to old documentations


Files added


Great :wink:


Yes, I meant R1 and D2.

I have updated the example such that it respects the outcome of the initialization handshake, this might have been a cause of the problems you were having. Please update and try again, I just tested the updated example on an Uno. The `SN65HVDA100` is an automative grade chip, these are usually quite robust. So it is quite likely your chip is not damaged. :)

For more information see issue #3, please also continue discussion there if the problem persists so that others can find the information more easily.


There is no stable release of ^0.17 version on github/packagist - se bundle is not installable now ;/

//cc @pamil

```
$ composer update sylius/theme-bundle
Loading composer repositories with package information
Updating dependencies (including require-dev)

  [Composer\DependencyResolver\SolverProblemsException]
    Problem 1
      - Installation request for sylius/theme-bundle 0.17.x-dev -> satisfiable by sylius/theme-bundle[0.17.x-dev].
      - sylius/theme-bundle 0.17.x-dev requires sylius/resource-bundle ^0.17 -> no matching package found.
  Potential causes:
   - A typo in the package name
   - The package is not available in a stable-enough version according to your minimum-stability setting
     see <https://getcomposer.org/doc/04-schema.md#minimum-stability> for more details.```


может тогда уж \label{ch2n1} ??


We need to fix the chat and configs before making a release.


Snake? is there an entity called snake?!


you may wan to log, or better check, the exit  code?


@NiclasOlofsson Would u like to join us or have a friendship with our project?


Note: No translations yet for any of this stuff. (Should probably be in the commit notes, but still)


Тут, вобщем, затык такой: ansible рекурсивно всем даёт права 755. И файлам (что не верно), и папкам. Нужно переписать в этом yaml так, чтобы 755 было на только папки, и 644 на файлы (точнее файлы вообще не трогать, они по дефолту правильные). А то из репа идут 644 файлы, а ансибл их передёргивает в 755 (они становятся исполняемые), это вообще дырень по сути в безопасности.


Постарайся не использовать скриплеты. Разберись как использовать JSTL


ok


@dillaman I have created a separate pool request for this: #7832


You're still adding the virtua to the super class, and I don't really like code duplication.


Branches


Trunk


Refactor WME to use Common::String... but as far as I remember that's a rather involved task (which would be quite helpful), as the pointers are passed EVERYWHERE.


Can't understand!!!???


i have not tested this, but it should work.


apple/ios has its opengl headers in
OpenGL, not in GL (see ExtensionList.h)


Tags


Agreed


Also, you knew this would happen... but could you slice this up into a nice set of smallish commits before PRing?


Creating Folder Structure


Ok


Added folder


I agreed


This change was merged as part of #10800, so can you rebase away this commit?


Looks legit to me too.


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


По правилам типографики после запятой ставится пробел.


Hi reger24, maybe some other files could be updated the same way, like startYACY_debug.bat, statYACY.sh or build.xml?


That's for preventing overfitting in the logistic regression layer.

On 27 February 2016 at 03:52, eyaler <notifications@github.com> wrote:

> shouldn't this line now be removed?
> model.add(Dropout(0.5))
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fchollet/keras/commit/abca83373d6d7df2747c5cdcc2bb7234395604e6#commitcomment-16370138>
> .
>


Jep, already noticed. I'm currently testing the fix


Hi I am a techie but I am new to OpenHAB and trying to get my Raspberry Pi and RaZberry z-wave plugin board to work.
I found some code...  openhab/bundles/binding/org.openhab.binding.zwave/src/main/java/org/openhab/binding/zwave/internal/protocol/commandclass/ZWaveBinarySensorCommandClass.java

I think this code will help, however I do not know in which directory to put this code.
I would be nice for beginner as part of the codes comment to let people know where the code should go.
I just choose you because you were the first name.


Is it correct, @motezazer?


As I remember MiLo after athoik post in the forum has already started some changes.
Unfortunately, I do not know why he did not continue.

I think to the correct only one file makes sense only if something doing with her.
So I really sorry if I have just changed it, and immediately after this, someone break this again with a bad code style.

It's that you do not like two empty lines are only your habit.


I don't think truncating the message to one line for the status bar has any effect, since it seems to do it itself anyway.

So this shouldn't cause any regressions.


What a fuck mother pfuck...


This line is duplicated. You could at least have tried compiling before committing.


@hannesmuehleisen fix for current swmap_acs crash http://monetdb.cwi.nl/testweb/web/eanthony/fedora20-3/1456554009/swmap_acs.log


we should use the oglIncludes.h (also in ExtensionList.h) I change that now


This should work:

```yml
language: java
script: cd java/pro2 && mvn install
```


Thanks for the request. I will close it since it is failing on travis and scrutinizer, then will merge to the develop branch in a different pull request.


Ok, gut. Das Problem war gestern allerdings, dass mit dem neuen system die tiefe um 2 reduziert war. Und eine sichere Überprüfung, ob andere graphen geschnitten werden, hab ich noch gar nicht eingebaut. Nur die iteration über die graphen...


@hannesmuehleisen fix for swmap_pnad crash http://monetdb.cwi.nl/testweb/web/eanthony/fedora20-3/1456554009/swmap_pnad.log


Thanks for your contribution!


Very good, we should use American English in project not British English.


Wait! What do you mean? Extensionlist includes oglIncludes but also has a few more defines. These defines are required in Window.cpp


a


Hello world!!


dsf


Here's an accepted solution of the problem mentioned in the todo list: http://codeforces.com/contest/633/submission/16383331


Optimize: should also use tag arrays for filling the form


Hey @Inspiron17R why do you need to remove it? I see you've added the static accessors to Links and Features, what's the motivation?


1. A geographical names database system.
2.It is a desktop system
3.based on Adobe AIR technology
ActionScript, sqlite


@somaen but of course, that's a given. I'm not nearly done, though :)


I fixed this upstream: https://gitlab.com/m2crypto/m2crypto/merge_requests/50


Left: Before, Right: After
![elcamino-bef-aft](https://cloud.githubusercontent.com/assets/7337916/13373066/30993016-dd5d-11e5-98c7-13e9b5f40f82.png)


ah yes I think we could replace oglIncludes.h completly with ExtensionList.h, since we need most of those opengl definitions if we have code with opengl


Tu fais pas de setup redis object ?


This will break a clean build. Try to empty the workspace and build again...
The symlink creation was moved from the shell script to CMake so the folders will be missing. In a test, I cleaned the build directory (for apple on "my" build-project) and rerun cmake. Symlinks were created as they should. Maybe the problem is something else?


pas de setup de redis ?


The type information is stored in the typename. What is a type then? it looks like 'h' or 'v', but what happens if we have a road joining at 30˚ to the main road - will it be 'v' or 'h'?

Also probably `numberOfLines` should be `numberOfLanes` - lane is a single pathway within the road.


Well, I'm totally confused. You seem to be commenting on a very old PR (ie 1 year ago) but I've absolutely no idea what you are trying to do.

> I think this code will help,

Help to do what?

> I would be nice for beginner as part of the codes comment to let people know where the code should go.

For what purpose - you don't say what you are trying to do.


Les messages pourraient être mis dans un fichier de traduction cf https://docs.lita.io/plugin-authoring/localization/


This is possible. However I just did a great deal of work, to try to minimize the includes. No making another header-composite just for "often required together" stuff counteracts that ;)
I was also hesitant leaving the `glext.h` include in the oglIncludes...


Again, probably `Lane`, not `Lines` - it should be singular and lane (i guess)


rajoute standup dans toutes tes routes histoire de namespace tes commandes. Surtout avec ignore


no, the problem is, that the jenkins has problems with recursive symlinks (thats why it was so slow and needs so much memory)

but you're right, the RTTR symlink is necessary, the rest not


Enumerable#inject


:o does that compile at all?


What does feature within feature mean?


@PeratX )))


Why?


good catch, fixed


yes but when do we change the opengl-basic header, I think thats less duplicate code to move these 10 lines into a single file and use that, instead of copying everything into 30 files


What is type of road for character 'å' for example?


@An7ar35 can you look into this? This probably works out somehow with your loading code.


Là du coup (comme c'est moins complexe qu'au moment du saut de ligne) je pense que .find peut revenir a la ligne du dessus.


WIP
still building base for utilite
do not work right now for i386 (systemd ends with <ILL> while booting after switch_root )


Ton identation a pas l'air correcte


ligne en trop


@Inspiron17R `static` has a totally different meaning in java - [static keyword in java](http://stackoverflow.com/questions/413898/what-does-the-static-keyword-do-in-a-class)


Tá funcionando @Meticore !


Si y a un saut de ligne ici, pourquoi y en a pas un après le send_message?


indentation


![](https://media.giphy.com/media/xTiTnndUFybun5rqoM/giphy.gif)
:wrench: :woman: :slot_machine:


basically that's a big logical issue; in this way the code would work like this:

every time you are creating a link, these static vars will get reassigned. So in the end of the day, all links you have added will point to the same two features.


again, 30˚ angle?


ligne


Debug code


That's bad. That stops the thread for 5 seconds (in physical time). I.e. nothing happens for these 5 seconds. We have a simulated time for this. Wanna talk?


Tools::getLink() 已弃用，以后将使用$link->getPage()代表


hash_including (on se fiche du as_user => true)


drop it


asdasd


asfasfasf


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


I think it's better if we replace the lines 317-320 with the one I wrote, line 321.


met ça dans un before


@aserratorta check this to enable datepicker form templates


but but


I've put the info back in the doc.

I don't really like it though, because the day we might need to remember this, every thing around will have changed. And to my experience, if S&P brings the data back it won't be exactly as it used to be. And if wee need this, we can look back in the history, that's what is it for.
And on top of that, the lack of simplicity is bad for the user.


Still the compiler has to parse a header that is not required... If a cpp file needs the extension, it can include `ExtensionList.h` if not, it includes `oglIncludes.h`
Better would be to make oglIncludes only include `gl.h` and `ExtensionList.h` include the `gl_ext.h` and does the definitions. But well, it doesn't really matter in this case. The name `ExtensionList.h` is also bad, so if you like, merge them into oglIncludes but also rename the cpp with the definitions.


Et si le user report pas ? IMO ce context sert a rien


is* onvoltooid.
Damn typos.


dans un before


before


ligne


BLOG


Yes, copying the data should fix the issue, thanks @pscollins @phhusson


Teste commit


Hi, I recently bought a Raspberry Pi 2 computer and added the RaZberry (
https://www.buyapi.ca/product/razberry/#tab-description) Z-wave plugin
board to the Raspberry Pi.
I installed Raspbian and downloaded OpenHAB runtime, the OpenHAD addons. I
moved the org.openhab.binding.zwave-1.8.1.jar to my /opt/openhab/addons.
Now I figured I still needed something that targeted my board the RaZberry.
I did not think the org.openhab.binding.zwave-1.8.1.jar would be specific
enough for my RaZberry board.
I then found the code I commented on, I did not really want to comment on
the code, I really just wanted to asked a question, but did not see a way
to do this so commenting seemed like the best approach.

Now I see the code that starts with...

Copyright (c) 2010-2016, openHAB.org and others.
 *
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Public License v1.0
 * which accompanies this distribution, and is available at
 * http://www.eclipse.org/legal/epl-v10.html
 */
package org.openhab.binding.zwave.internal.protocol.commandclass;

import java.util.ArrayList;
import java.util.Collection;
import java.util.HashMap;
import java.util.Map;

import org.openhab.binding.zwave.internal.config.ZWaveDbCommandClass;
import org.openhab.binding.zwave.internal.protocol.SerialMessage;
import
org.openhab.binding.zwave.internal.protocol.SerialMessage.SerialMessageClass;
import
org.openhab.binding.zwave.internal.protocol.SerialMessage.SerialMessagePriority;
import
org.openhab.binding.zwave.internal.protocol.SerialMessage.SerialMessageType;
import org.openhab.binding.zwave.internal.protocol.ZWaveController;
import org.openhab.binding.zwave.internal.protocol.ZWaveEndpoint;
import org.openhab.binding.zwave.internal.protocol.ZWaveNode;
import
org.openhab.binding.zwave.internal.protocol.event.ZWaveCommandClassValueEvent;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import com.thoughtworks.xstream.annotations.XStreamAlias;
import com.thoughtworks.xstream.annotations.XStreamOmitField;

I thought this may help getting my board recognized.

Walter

On Sat, Feb 27, 2016 at 8:24 AM, Chris Jackson <notifications@github.com>
wrote:

> Well, I'm totally confused. You seem to be commenting on a very old PR (ie
> 1 year ago) but I've absolutely no idea what you are trying to do.
>
> I think this code will help,
>
> Help to do what?
>
> I would be nice for beginner as part of the codes comment to let people
> know where the code should go.
>
> For what purpose - you don't say what you are trying to do.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openhab/openhab/commit/c7f88b81201ddaf69e5244401515a1090c6a660e#commitcomment-16370440>
> .
>


Teste commit master


The explicit string cast shouldn't be necessary since the type is specified in the call. Is there a reason this is here?


Left-over from the merge conflict @sorvell


Perso je l'aurais mis dans un espèce de before dans toutes tes methodes non private


GITHUB BLOGS


мало бути окремим комітом.
Зараз прийму. Але це в перший і останній раз. В коміті мають бути пов'язані зміни. А видалення іконки твіттера ніяк не пов'язане із переписуванням на Markdown.


Not sure we usually do that, after all the debugging _console_ is always available. Then again, we can consider it.


I don't think it really matters. If it's locked, it'll either have changed when we verify at commit time and we'll abort, or it'll still be locked at commit time and we'll abort. Or that transaction will have aborted and unlocked and our commit will actually succeed.

From a perf standpoint we should probably just abort right away though.

I think from an opacity standpoint it's okay (assuming we're actually doing an atomic read to make sure we're getting a consistent value read). But again it's probably simplest/fastest if we just abort.


>An idea that might be interesting, would be to always have a spare syscall thread.

Yes, having a dedicated syscall thread to spawn syscall threads sounds like it should avoid the deadlock.

@pscollins @phhusson @tommie-lie

>On top of that, we can add that on every syscall, it checks whether it's on the default queue, or thread-specific queue.
>If it's the default queue, then start a new thread.
>...
> I'd say having an option for that gives the best modularity.

Sounds reasonable to me, I am going to give it a try.


Please add BSD 3-clause license to new files (can be copied from mouse.py).


Admittedly I thought this would be helpful in the beginning, but then I saw myself deleting everything and starting from scratch anyway. Let's remove them.


mk2man => mkd2man


"of the button"


Il branch e' pronto? Se si' prepara PR .


Please use fetchFromGitHub, since that uses fetchzip internally

```
# This function downloads and unpacks an archive file, such as a zip
# or tar file. This is primarily useful for dynamically generated
# archives, such as GitHub's /archive URLs, where the unpacked content
# of the zip file doesn't change, but the zip file itself may
# (e.g. due to minor changes in the compression algorithm, or changes
# in timestamps).
```


@DVigneault Very nice. There is 1 new test failing.  It may not be related to this patch set.  Please investigate.


My pdb has a loud "lost aircraft buzzer" that is connected to the servo outputs and mapped to a switch using CHANNEL_FORWARDING. Unfortunately this does not work anymore in the newest release. Would you mind putting this feature back in?


English is to be provided when everything is coded completely.
You may also help with translation as well.


Why include the version in meta? Normally it’s put directly in the `mkDerivation` attrset.


:clap: :sparkles:


I like the old one better. Suited the dock and OS X feel.


请问两个有啥不同吗


Yes, it still happens for fields mixed in from Scala2 traits ... despite
```scala
private def suspectFieldMutable(sym: Symbol) =
  sym.is(Mutable) || sym.is(MixedIn)
```
so I guess `sym.is(MixedInt)` does not do the job I thought it would.

[The *other* case](https://github.com/scala-js/scala-js/commit/619f264e431b079e8cfc3393a8744514c8a1577a#diff-eef8366df9f9e182656fd69928a3f7f1R301) where Scala 2 would cause this to happen, however, does not happen in Dotty.


Sorry Walter, I'm still quite confused. What do you mean by 'getting the board recognised'?

openhab runs under Java, so it doesn't care about the hardware - it should just work. If you can't get it working, then please ask a question on the openhab forum (community.openhab.org). You should not need to edit the code to get the system running.

Many people are using the Pi, so I'd suggest asking a question on the forum - someone will I'm sure be able to help, but I would suggest trying to describe your problem better as I still don't know what it is.


В целом - молодец, все сделал хорошо. Пара замечаний, если исправишь - будет здорово.
Интерфейс немножко запутанный, непонятно, то ли все зависло, то ли просто не нашло ничего. Какие-нибудь сообщения, например, что ничего не найдено - было бы удобнее. И поиск по регулярным выражениям пожоже не работает. В остальном - все хорошо. Лабораторная защитана.


@Glenn-1990 any reason for making the values hardcoded English strings? Please fix this.


such code
> On Feb 27, 2016, at 3:22 AM, Jack Serrino <notifications@github.com> wrote:
>
> wat
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/techx/event-checkin/commit/d4eca39387aa06086125381730a67d3272380e6f#commitcomment-16369556>.
>


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `docs`.* *Related to file `docs/Users/Install.rst`, line 164.*

The issue can be fixed by applying the following patch:

```diff
--- a/docs/Users/Install.rst
+++ b/docs/Users/Install.rst
@@ -161,7 +161,7 @@
     -  Use sudo -E pip install coala(-E flag takes in the env that
        you have set in the earlier step).

-    Check
+    Check
     http://stackoverflow.com/questions/14149422/using-pip-behind-a-proxy
     for further clarification.

```


@bhouston `THREE.CubeTexture` extends `THREE.Texture` and `THREE.WebGLRenderTargetCube` extends `THREE.WebGLRenderTarget` so I believe these checks are redundant.


Yep, opened PR. Needed this for creating a [generic React wrapper on (mostly D3) components](https://github.com/pemrouz/vanilla/blob/master/README.md#fn6-more), but needs more work..


in all the specs above, below, and moving forward, put a space above the expect
```
it "adds the correct information to entries" do
 +      book = AddressBook.new
 +      book.add_entry('Ada Lovelace', '010.012.1815', 'augusta.king@lovelace.com')
 +      new_entry = book.entries[0]

 +      expect(new_entry.name).to eq('Ada Lovelace')
 +      expect(new_entry.phone_number).to eq('010.012.1815')
 +      expect(new_entry.email).to eq('augusta.king@lovelace.com')
 +    end
```


We are waiting for new drivers that will remove vumodel.

The correct "work-a-round" you be the following until then...
```
-       elif fileExists("/proc/stb/info/vumodel"):
+       elif fileExists("/proc/stb/info/vumodel") and not fileExists("/proc/stb/info/boxtype"):

```

Anyway I will ask for revert when new drivers are out.


2 typos here


let's call this `specialUnitsMap` and call the other one `exportedMap`


Sorry, I misread what you were proposing.

Yeah go ahead and use the .Net 3.5 NuGet package, thanks for letting me
know about that, I never knew it existed.

On Friday, 26 February 2016, Sasha Goldshtein <notifications@github.com>
wrote:

> Why update the binaries and not take a dependency on the NuGet package?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PerfDotNet/BenchmarkDotNet/commit/ae71f296843b2d9bd05892477241c4193573b42e#commitcomment-16358159>
> .
>


it was the one before this, this one resolves it.


True that. Well we can also do this. This way we are touching only the required attrs. Can we go with this? :)
` _dict = get_public_members(self)
   _dict['file'] = os.path.relpath(self.file)
   return _dict
`


:D look at snake class

> Am 27.02.2016 um 13:57 schrieb PocketKiller <notifications@github.com>:
>
> Snake? is there an entity called snake?!
>
> —
> Reply to this email directly or view it on GitHub.
>


##### These commands assume you're using my toolchains, found [here] (https://www.androidfilehost.com/?w=files&flid=50435)
### __edit these as needed__
#### for ARM
$ ./configure --host=arm-linux --disable-shared --enable-static CFLAGS="--sysroot=/path_to_toolchain/arm-uClibc/arm-rorschack-linux-uclibcgnueabihf/sysroot -Os" LDFLAGS=--static
$ arm-linux-strip -s src/xzdec/xzdec
#### for x86
$ ./configure --host=i586-linux --disable-shared --enable-static CFLAGS="--sysroot=/path_to_toolchain/x86-uClibc/i586-rorschack-linux-uclibc/sysroot -Os" LDFLAGS=--static
$ i586-linux-strip -s src/xzdec/xzdec


as.as014.wpt has an extra line of whitespace at the end. I don't imagine this should cause any problems at all, but just so you're aware of it.


I was able to test the changes made by @fdp1nm. Yes, the sleep command does exist on my AVR as well, but it seems to work differently. It does not immediately suspend the AVR but sets a timer to 10 minutes, after which it turns off. When sending the request multiple times, it increases the timeout by 10 minutes each. So I guess there's no problem merging it, although it may work different on each device.


What has happened to boost on github? I can no longer click on folders and get into them. Did boost go into private repo?


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


sry dafür, ich glaub das gibt nen fehler


我改错了……… add main content 之前才可以背景音乐


Changed package on actuality


Somehow **updater** seems to get into a loop in this new **3.2-series**. Can you check that again? **manager** seems to be fine.


It'll be better to [inject the document](https://github.com/angular/angular/blob/3c8fa8c50d820e3317bc40292c73df03c8b23e86/modules/angular2/src/platform/dom/shared_styles_host.ts#L35), instead of using any global references.


This uploaded oddly. I have uploaded the project to Drive.


Correr programa en .. [https://repl.it/Bqop](https://repl.it/Bqop)


数据库在哪里


The maximum viewBox values in Chrome seem to be related to the width/height attribute.  For 350x296, this is the maximum: viewBox="0 0 5600 4736"  As soon as I increase the width or height, I can use higher values


I made Xfce use gvfs without samba: 9d092e3.


Please choose plugin relevant class name.


魔术常量和字符串是没有冲突的


why `Account::User`? can't we use `Group::User` here?


This proves harder than expected.

* At the time `ContextBase` is created, the settings have not been parsed yet, so we don't know what type of `platform` to instantiate. This is fixable, it could be a `lazy val`, but it's unclear whether `platform` will be accessed before the settings have been populated or not, and would be fragile even if it isn't.
* When creating the `platform`, we don't have any `Context`, but we need one to instantiate `JSDefinitions`. Even if we used the `implicit var` trick (which I still find awful), there would be nothing to `init` it. This is fixable too: we could add an `init(ctx)` in `Platform`, and `SJSPlatform` could override it to `init` its `jsDefinitions`. But it gets uglier ...


Wrong spelling!


Only problem with config is that it doesn't generate a new one if file is missing.


block = 0 does not work now anymore, but it was not used in the code and should be disencouraged because it breaks type stability.


@p0psicles please make PR! this is wrong!


I offered several patches, they forgotten, I lose interest. ;)


This should be reformatted using multi-line sting `"""... """`, avoiding the need for explicit `\n` characters.


this is wrong


also this


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


you can't set default as NONE


cant set default as none


I think xdebug is necessary in a PHP driven development environment. By facing the problem of complexity we should install xdebug anyway. IMHO the only one argument against xdebug is the lack of performance if enabled.

So I think it would be better to remove the additional param in Vagrantfile.


This must be in `org.xwiki.locks.script`.


Thanks for contribution! It was you that make me be able to see the world outside.


Hah, apparently a Chrome bug.  I posted this on stackoverflow and someone filed a Chrome bug in my behalf: https://bugs.chromium.org/p/chromium/issues/detail?id=590447


The script service is listed in the wrong package.


What env? It sounds confusing, that the `-E` flag is supposed to take something as an argument. You could write it like:
```
-E flag preserves the existing environment variables
```


pour l'exemple ci dessus : a={1: [1,2,3,4], 2: [1,2,3,4], 3:[2], 4:[1,2,3,4]}
prune à partir de 3 : a={1: [1,3,4], 2: [4], 3:[2], 4:[4]}
puis prune à partir de 4 : a={1: [3], 2: [ ], 3:[2], 4:[4]}
c'est bon ou alors il faut empêcher le prune au niveau de 4 ????? :octopus:


            (title, url) = self._get_title_and_url(item)
            (seeders, leechers) = self._get_result_info(item)

revert entire PR and just need to do:
ci = self.cache._addCacheEntry(title, url, seeders, leechers, size, parse_result=parse_result)


Also escape the code blocks in double backticks \`\`code here\`\`


Read could be a word.


We could actually work around this by doing the math ourselves and if the pattern bitmap were < 0.5 pixels in any dimension, manually double the pattern bitmap.  Unfortunately, I don't have time to do this today, but it should be doable.


Hello @krischer, I think you did the commit to my branch by accident. Sorry, I hadn't given my branch a different name until just now. I just did. Hopefully it didn't cause a big problem.


study


Nowhere in the cmake or autoconf files is LOGGING ever added as a compile definition. That means this source file is always empty when used, which is certainly not what was intended.  It is added as a source file when USE_LOGGING is true.  Two possible solutions: 1) add LOGGING as a compile definition when USE_LOGGING is true; or 2) just remove the #ifdef.  Since the file isn't even used when USE_LOGGING is false, the #ifdef wasn't serving any purpose.  That's why I chose option 2 in my patch.

How USE_LOGGING gets set in relation to logging in netcdf is a completely separate issue.


@SanketDG OK!


Can you revert? And fix please. The old implementation was wrong. You cant create a function with 4 mandatory params and then call it with 4.


Присваивание можно оставить в этой функции. Они не имеют никакого значения для "внешнего мира" этой функции.


What if maintenance is false?


Maybe in a future version a nice results filled database would come in handy :D


Yeah, the Readme is really outdated and need some informations about the great packages/features in this box and xdebug is also a nice essential tool, but for a guide using it with PhpStorm (one of thousands of IDEs/Editors) is this the wrong place. Only a small hint, like a link to the Jetbrains man page would be better.


But fetchFromGitHub downloads a zip file or the raw source code as a git command?


Whoops hehe, that would be really bad.


Nice but, please, write yours commits in english next time, we are an english server.


added HelloWorld file


At this point mysql will be added, don't bloat VM with unused service.


I think that the Browser crash problem on WebView 40.0.2214.114 prebuilt APK is caused by the Chromium WebView library (I tested the Chromium WebView library from pawitp's WebView but I need to adapt the code to use that) (I am receiving ClassNotFound errors, by the way)


Can you please make a 1.7.1 release with this fix? I'd like to `pip install` the module. Thanks!


At this commit, I just added the link to the page github.io


Ok will do it ASAP


Very nice!


<img width="564" alt="screen shot 2016-02-27 at 9 48 31 am" src="https://cloud.githubusercontent.com/assets/4250521/13373491/4906a8a2-dd37-11e5-9854-bae4f90fd40d.png">
how to get rid of `vendor` complains?


Did you mean ``-P`` or ``-p``?

> -P, --no-dereference
              never follow symbolic links in SOURCE
       -p     same as --preserve=mode,ownership,timestamps


@sandlerben Why removed from gitignore?


Cheers for this! Been meaning to fix that up but kept forgetting.


@gimite Any idea when there will be version [1.0.6](https://github.com/gimite/google-drive-ruby/blob/master/google_drive.gemspec#L4)?  I need this commit (to use env vars for refresh session creds), but prefer not to be referencing master in production ;)


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


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


Don't bother testing yet. The search paths still need some attention to find EQDefaultCurves.xml.


According to http://twoears.aipa.tu-berlin.de/doc/latest/binsim/usage/#configuration-using-a-matlab-script it is not recommended to create multiple instances of the `simulator.SimulatorConvexRoom()` class. However, this should work in the particular example.


sorry for commiting the mapper...


Reset humidity oversampling settings when calling `bme280.startreadout()` - if the sensor is reconnected and readouts are in forced mode there is no need to re-initialize the sensor (`bme280.init()`).


Signal incorect reading when the sensor is disconnected.


I tested both #938 and @jnpkrn's suggestion, they both work fine on FreeBSD-11-CURRENT.


zip file, as the name `fetchzip` indicates. ;)


No matter of the final values I would try 2.05 and 0.78 here.


*sigh* I appear to be entirely incapable of choosing one file naming schema and sticking to it.


It is not necessary to save the frame onto the hard disk. There is a `appendData` of the sound source, which you could use as follows:
``
simWFX.Sources{1}.appendData(frame_k)
``


@AbdealiJK

- I might need some thoughts on what to test in `coalaJSONTest`?   We can certainly test if we are getting a valid relpath or not. Is that what you meant ?
- So I assume if I run `coala --find-config ` , it will return the config files? I ain't getting any. Am I missing something here?


Это еще не Clean Code вариант, правильно?

Исправления к последнему замечанию, чтобы не нужно было вводить дату - здорово, когда люди обращают внимания и исправляют. Молодец.
Было бы неплохо еще в пакеты код поместить.
И как совет, на будущее, для clean code:
а) Когда Вы отлавливаете исключения, это хорошо, когда Вы выводите на консоль информацию о проблеме, но иногда этого недостаточно, возможно нужны какие-то дополнительные действия, по обработке ситуации (например - освободить память или ресурс). Это не всегда, но бывает. Например, у Вас в Map складываются сообщения в parser, но дальше отлавливается IOException. Вот тут можно и освободить память почитстив Map. Это не обязательно, приложение и так не сломается, но Вы же в будущем будете писать большие и сложные приложения.
б) Никогда не отлавливайте NullPointerException. Если Вы понимаете, что может быть null, то лучше проверить If (val  == null) { // some actions}.
Когда вы отлавливаете null, Вы получется прячете проблему. Особенно если в catch блоке ничего не выводиться о проблеме. Получается, что критическая ошибка была, было null значение, где его не ожидали, а приложение дальше работает и ничего не сообщило. На занятии можно будет обсудить этот вопрос, если непонятно.

В остальном - все неплохо, ждем Clean Code вариант


athoik
just add them in pull request
remember everything in the mind is impossible


Gauss-Seidel method has been added, but it isn`t work, because matrix[][] isn`t include B column.


The graphs are similar to my tuning benchmarks of QueenValueMg where it went close to an optimum, but was unable to be stable at it.


Pls explain why not? Alternative could be kwargs. But you still need to handle cache items that dont have seeders, leechers.


rename the variable "lessons1" to "lesson1". It is only one lesson object, not many lessons.


A recomendation maintained in the manual.

Also, I believe the version should be parametrized, and not guessed. But it is another story.


The query model for that is (using ToString):

    from Int32[] e in TestQueryable<Int32[]>() select {[e] => Count()} => Aggregate(0, acc => (acc + Convert({[e] => Count()})))

The fact that the {[e] => Count()} appears twice up there is probably a clue that the QM is being built incorrectly (???).

The crash happens b.c. the main QM.MainFromClause is the {[e] => Count()} and so that gets cloned once, and then during the result operator cloning phase, it tries to clone the [e] => Count() again, and that second time causes the bomb.


remove those extra spaces.


blog:http://www.cnblogs.com/wvqusrtg/
QQ:1911398892


Not passing all tests yet but FYI @SimonCropp @SzymonPobiega


Rename this method. It doen't read anything, it outputs the text representation of the object. You can call it toString()


:fireworks: Could you go through the core and util stuff to see if it needs some updating? :)


This is still not ideal - need a final check for system to report all overlaps, I think


Would be nice to add a `--aux-directory` flag for those weird and annoying files as well.


There's a bunch of ways to skin this cat, but, an interesting side effect of your approach is that now your feature file is responsible for knowing the ID of your text field.  The concern would be that you could have many different feature files using that step and if the ID had to change for some reason, you'd have to change the ID in all the features.  If you called a method instead, the name of the method could stay the same and the ID would change in one place (the page object).

Like all things it's give and take, because the down side of having the page object own that information is that you can't have the highly reusable step that you currently have.  No free lunch :)

Here's a really interesting blog post about this from one of the creators of cucumber:
http://aslakhellesoy.com/post/11055981222/the-training-wheels-came-off

So, with the disclaimer out of the way, ruby actually has a really concise way to do what you asked about:
```ruby
class SimpleHandlingPage
    include PageObject
    text_field(:message, { id: "message-input" })
    text_field(:delay, { id: "delay-input" })
    def enter_text(field, text)
        self.send("#{field}=".to_sym, text);
    end
end
```
I also changed the feature file to pass either "message" or "delay" as the 'field'.

What that's actually doing is, building a string that matches the setter method name for the specified field (:name with an equals after it), then converting it to a symbol and invoking it on self.  It also passes the text variable as the first argument to the message= or name= function.


hey, you should take a peak at http://coala.readthedocs.org/en/latest/Getting_Involved/Writing_Good_Commits.html and amend your commit, make the message fit.


is audience an integer number? Is it the number of students?
Can you create a class Student and convert the Lesson.audience field to array or ArrayList of students? It will be your second homework


My commit message should read:

[...]add entries on `$g_html_make_links` and `$g_html_make_links_new_window` to manual[...]


Thanks much for updating these.


No need to wrap this in an else, as the return in the if will exit before reaching it.


"build": "rimraf dist/* && ncp static dist/static && cross-env NODE_ENV=production webpack --progress --hide-modules --config build/webpack.prod.conf.js**s**",

后面的多了个s, win上面改成这个了:
"build": "rimraf dist/* && mk dist\\static && ncp static dist/static && cross-env NODE_ENV=production webpack --progress --hide-modules --config build/webpack.prod.conf.js",


Looks good, tests shouldn't need code to be refactored by the looks of it


I think ReadMe file content is wrong and not related to this project. :smile:


Thanks for testing Andrew, 2) is fixed and the transposing will be added
next week.

On Fri, Feb 26, 2016 at 07:51:58AM -0800, Andrew Champion wrote:
> Very cool -- only noticed two things:
>
> 1. It doesn't appear to check `TileSource`'s `transposeTiles`, which is necessary for some sources like DVID that have a different axis order than expected. Should be as simple as swapping u and v for these orientations.
> 2. Sometimes the overflow/tile cropping at the edge isn't working quite right at the corners, like in this case from the L1 ZY stack:
>
> ![screenshot 2016-02-26 10 44 19](https://cloud.githubusercontent.com/assets/460206/13357096/ca7f1ac0-dc76-11e5-8655-1a40758465ca.png)
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/catmaid/CATMAID/commit/bea312cac5d29fa55f840a544f9183ef590617a6#commitcomment-16355594


CircleCI はデフォルトだと、 1.9 が使われているようで、日本語が入った spec が ASCII だと言われてテストが通らなかった
http://shgam.hatenadiary.jp/entry/2013/06/07/161457

よって、Ruby のバージョンを 2 の最新を使うように指定しました。


Nit: missing period (full stop) at the end of the sentence.


诶你们还在弄这个啊 可以可以


This is imho the mostpropable reason, why the non-segmented and the segmented versions do not lead to the same result. The signal is devided into N parts, whose invidiual lengths are not guaranteed to be a multiple of the simulators blocksize. As the binaural simulator always expects signals with their length being a multiple of the blocksize, zeros are padded to each of the N parts. This lead to discontinities is the input signal as a whole.
Workaround:
```MATLAB
signal_length = ceil( length(signal) / (N*simWFX.BlockSize) )*(N*simWFX.BlockSize);
pas_son = signal_length/N;
signal_alt = zeros(signal_length, 1);
signal_alt(1:length(signal)) = signal;

for k = 1 : N
   son_frame = signal_alt( pas_son * (k - 1) + 1 :  pas_son * k , : );
```


I especially dislike how this transforms all errors to 500s.


Can see it happen as the QM is built - but I don't know re-linq well enough to know how you should refer back to the select like that...

OTOH - debuggers are amazing tools. :-)


No I haven't, I'd just write everything in Swift instead of depending on C libraries.


I think can remove the `install.sh` and `test.sh` now, since we use Makefile (btw: make does not work on windows and cygwin .....


Nit: Why is every line indented? Just makes line wrapping more likely.


Is this the real life, or is it just a fantasy ?


If Lombok stays it's important to add to the readme under Compiling Requirements(?) that you need to install the Lombok plugin. Same for RetroLambda if it requires something


why some people get some ddis were out of the game?


This is Number one change in the ZodiacSigns project


@aodzip


Hi there @kumarbabu - thanks for this update.
I've got a couple of request please:
* could you break the tests down into several methods
* could you avoid using *iter.next()* to find the relevant operation?  It feels fragile, since it will break if the order/sequence of the returned operations hcanges.


I might suggest that we just link to Docker's install page (much more informative) and remove the explanation in parenthesis.

Eg:
1. [Install Docker](https://docs.docker.com/engine/installation/)


以前的一个错误忘记改了_(:з」∠)_


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


Preface: This version is edited by me, but is not tested, so may be not functional yet.
Kosuke, below i put some of the comments I sent you in the email - perhaps these pointers may be useful to future/other developers.

**Comment 1** Looking into the code for TD-SE. It is not exactly what we want.
The points are:
a) The idea is to use symmetric splitting scheme when integrating electronic DOF, like for the nuclear DOF
b) The initial conditions are defined both by the nuclear and electronic variables. This means, for each set of electronic DOFs you'll need to have an additional set of nuclear DOFs. Well, it is possible to do additional averaging over phase of the wavefunction, or over initial excitation, like you do, but we also want to add averaging over initial nuclear DOFs.

**Comment 2** looking on how you iterate over the initial condition - i think this is incorrect, will try to fix it

**Comment 3** in the variables' documentation - when the variable is changed in the function (even if not returned explicitly), it should be marked as [out] or [in,out] - this is the case for many variables given to runMD function

**Comment 4**
when you do:
        tmp = "prop_ele_file" + str(ic)
        ftmp = open(params[tmp],"w")

You create a key-value pair that is stored in the params dictionary - you don't really want doing that - the "tmp" is temporary variable, which you know how to reconstruct, so you don't need to store it and increase the params dictionary. This is actually true for many other parts of the code. I think you store a lot of such relatively useless info in this variable. This is time and memory consuming and is a bad coding style, because there is no way for other users/developers to know what kind of pairs one can expect in that dictionary (so they may either override some of the variables or use something wrong). So try to avoid this practice, and use it only when is absolutely necessary. By the way, because you always push some new stuff into params dictionary, I note it as [in,out], because the params variable after calling the run_MD function is no the same as it was before calling that function

**Comment 5** it looks like we already have a pretty large number of options and keywords to pass to the parameters dictionary - please prepare a some manual about it - e.g. a table with 4 columns: 1 - key name, 2 - type of value expected (string/integer/float, etc, and perhaps some limitation on the possible values (e.g. all positive)), 3 - default value (if any), 4 - explanation of the meaning of the variable.

**Comment 6** the ".xyz" extension is a standard extension for the file format containing atomic coordinates. it is misleading to print SE populations in the file with such extension - like in your run/run.py

**Comment 7** in vibronic Hamiltonian, we actually need *relative* energies of the configurations, not the absolute values, so even though we compute E_GS, E_S1, etc., we use E_GS - E_GS, E_S1 - E_S1, etc.

**Comment 8** we need big conceptual change: use "excitation" objects (see in Hamiltonian_Atomistic/Hamiltonian_QM/Control_Parameters). Eventually the interface to define excitations must be almost as simple as in test_hamiltonian_qm/test_qm17.py
So, we don't want to use the old-style excitations from Pyxaid, nor we want to use lazy module, at least in the beginning development. Few excitations for tests we can define manually.

**Comment 9** i sketched the new concept, but haven't changed the vibronic_hamiltonian function - please adapt it to the new formulation. Also, the "excitation" object has the properties "Energy" and "f". You can use them to store absolute or relative energy of the configuration and the oscillator strength for the transition (please include such new functionality). The former can them be used to conveniently compute Hvib, while the latter will be used to compute electronic spectra. For computation of "f", you will need the qhem/molint  module - see the Moments.h and Multipoles.h files.


ho visto il codice, manca un comando o qualcosa altro per non farlo rimanere sempre in gestione  automatica


Ok, I'll change it. FWIW, your previous comment was about false positives/negatives, not about time to run the test :)


ddos?


But we are not a script. This should be done with a proper system call.


includes/modules/header_tags/ht_manufacturer_seo.php

In configuration, using the word Category when Manufacturer is probably intended:

'Do you want to allow Category meta tags to be added to the page header?


No, we should find the problem. When jtr exits from a cygwin build where -fork was used, the tty is fuk'd up.

But this work around does exit jtr leaving tty sane, and will ONLY be run on a cygwin build, so should not impact any other environment at all.


5499-Electric-Service-Follow-Up (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


what ? @PEMapModder


Oops. thanks, that's clearly a bug


This is fantastic!


What is `/32000` supposed to signify here?


Git can always tell us which files were modified. So, please don't mention the filename in GIT commit messages. Instead, you should write down the reasons and motivation!


y u do dis :(

It makes it impossible now to actually set this property if the `enum` is private.


增加一个功能


Use plain English instead.


I thought it would be easier to read rather than inlining everything. But I can change it if you think it's better. I won't change it on this branch though because I know I'll get merge conflicts with my other PR branch.


test?


Why even ;_; Just allow the poor child to set the `WatchMode` directly.


does this need to be async?


I see your point. It's not too much code to inline everything. It will still look readable. I'll update it.


:+1:

Talk to Dan about making a Unit Test for this and put 1-3 in there.


What do you mean?


this should probably depend on all of the other ones so it runs last, right?


Java는 대개 느리므로, C++위주로 그리고 최대한 STL활용해서 많이 풀도록 하기, RTE를 최대한 방지하자.


if it is a bug ,report it in issue tracker .


this test needs to only happen with DRF 3.0+


None. Not sure what my thinking was there.


This doesn't seem to work with multiple profiles. Actually you have to have profile_id instead of profile_uid to form a valid profile edit link. But changing to here is not enough, so don't know how to fix that.


bah non ******** ce n'est pas mon passwd .

le vrai est dans le fichier config.php qui est exclus par le gitignore


hey, can you please read http://coala.readthedocs.org/en/latest/Getting_Involved/Writing_Good_Commits.html carefully and look at the examples? There's some details not right yet. (spacing, capitalization)


plus the shortlog content is also not so done well I think you can do it better ;) the content is good!


A copy from PM? Waiting for your modifications and that will be good work. xD


none of it seems to, anything you notice that is bad?


Para crear inicialmente el archivo package.json debemos ejecutar el
comando `$ npm init`, este comando nos hará algunas preguntas iniciales
sobre nuestro proyecto y es aquí donde definiremos las dependencias de
nuestra aplicación.

En el archivo .gitignore vamos a ignorar inicialmente la carpeta
node_modules pues no es una buena práctica subir las dependencias al
sistema de control de versiones.


:+1:  I like the entire sentence check


Why this called bug?


Please remove the commented-out code.


:+1:  OK, lets check this out with Gayla's HTML....fun times!


Please wrap the lines at 80 characters. For example:
```
  credentials = ServiceAccountCredentials.from_json_keyfile_name(
      args.json,
      scopes=SCOPES).create_delegated(username)
```

Thanks!


good, this is only you need to do


Missing `=`


The time correction for particle data is already implemented in mms.variable2ts in the devel branch.


these lines are wrong... next week I'll explain it


; add semicolon;


Good job :+1:


Здорово, что Вы реализовали больше возможностей, чем было раньше. Это плюс.
С кодом, у Вас тоже все в принципе неплохо. Но, есть несколько замечаний, если исправите, будет плюсом.
а) Сделайте, пожалуйста, если не трудно, в Git-репозитории папку для заданий 2.1 и 2.1 и в нее ваш код со структруой пакетов. Например, у Вас имя репозитория upHapaeva. Это корень. В ней файлы .gitignore, log.txt и status.txt. Это у Вас есть. А вот папку up1 переделайте в папку CleanCode и в ней src/up/ и дальше ваши файлы с исходным кодом.
б) Было бы еще круто, если бы пакет Up стал by.bsu.up.cleancode :). Это Вам очень пригодиться в будущем, потому как самое сложное в написании кода - это придумывание имени класса и пакетов. :)
Тренируйтесь
в) Метод main можно тоже сделать более красивым. Выделить части для
* вывода инструкций (метод printHelp например)
* проверки ввода
* выбор и выполнение действия.

Но в остальном - все работает, работа принята. Молодец.


You need to check if there are actually valid components found (check using has_ignore_reason), because only if there is a valid component in there, the metadata will change.
You should, however, always rebuild the hints information then.


In Elixir it is the `when` keyword :)


So you are suggesting writing four zero pointers here? The Python C-api doesn't seem to require this, does it?


The validator wants this to be "Time" - not sure which one is better.


5499-Electric-Service-Follow-Up (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (2113 of 2120 tests passed, 2 test warnings)

 * 9 tests had: AUD diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 1 test had: ERR diffs.
 * 7 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


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


You sure about this change?

Ignoring the merits of any particular linter, this breaks some people's configs, which isn't nice.

The merits of linters taken into account, JSHint provides basic quality checks without any configuration, whereas ESLint does nothing except detect parse errors. So JSHint feels like a more useful default. On the other hand, JSHint is more opinionated than ESLint for the same reason.


@tushar-rishav
1. yes, check if the path ifs relpath
2. `coala --find-config` is meant to go down one directory at a time till it finds your .coafile. So, in editors we have the file `/a/path/to/coala/project/file.cpp` where we run coala from `/a/path/to/coala/project/` and let it find the coafile. So, if coafile is in `/a/path/to/.coafile` it'd find it automatically by searching in `/a/path/to/coala/project/` and `/a/path/to/coala/` and `/a/path/to/` until it finds it there.
In such cases will this relpath thing give `../../` for every file ?


5499-Electric-Service-Follow-Up (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1012 of 1012 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-29.4%25-green.svg)


5499-Electric-Service-Follow-Up (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


It's `int`, good catch. It works because of the endianess of x86. This is wrong in `MethodWrapper`, too.


Oh, and ESLint is also still dedicated to formatting-related linting, which could be another motivation to move it up. Still worth mulling over a breaking change, though.


had to change this back to using the where clause because the data that would be passed through to this would not have all of the fields in the database, making it a bit complicated to find a specific entry


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


Changed .find() to .findAll() for the purpose of finding every orderedItem an order may or may not have instead of just one orderedItem


Background: The message id (and query ID) is only unique for each session, but the centralized query tree assumes that they are globally unique. If two identical query ids show up, this means the assertion in the tree will fail on insertion of the second query.

Instead of fixing this by creating a query tree for every session (and changing lots of code in different places), this approach assured that the client never creates duplicated IDs. The protocol says that message ids only need to increase mono monotonically in each monotonically which means that we are allowed to do so.  (see https://core.telegram.org/mtproto/description#message-identifier-msg-id)


link does not work: https://goo.gl/Yl4uoN


@Inspiron17R - By `type`, do you mean to specify whether a line (I'm assuming its a lane on a road here) goes across or along the road? If you mean it as the physical rotation of an actual road then we're going to be unable to model anything that doesn't fit at 90 degree angles to each other ( -- or |- ).

- Other point is avoid using char types for things that can easily be translated to Enums with the added bonus that it offers no need to check for non expected characters. e.g. what if you pass a 'q' as a type; what happens?


Fazlaliktan Yazmışın Kırmızıları Sil


5499-Electric-Service-Follow-Up (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2151 of 2158 tests passed, 2 test warnings)

 * 9 tests had: AUD diffs.
 * 1 test had: ERR diffs.
 * 7 tests had: ESO big diffs.
 * 9 tests had: MDD diffs.
 * 9 tests had: MTD diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: RDD diffs.
 * 1 test had: Table big diffs.

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


I see, ok.

> I do get notifications from this repo, so I think I've got it set up in a way that works for me.

Do you mean to say you'd get a notification for this comment even if I hadn't explicitly @-mentioned your username?


Packages added: `icon-fonts`, `atom-bootstrap4` and `language-fish-shell`


already did


Good catch. :laughing:


Hum, I don't understand. When the option is *not* defined, it doesn't use the secrets ?


Wouldn't it be better to have the functions YSF_Get(Player)NetworkStats or something like that for the advanced functions ? Maybe we want both or something like that, lol.


Careful with the commenting inside the code. For such small things if you feel the need to explain that much then you might want to consider naming your variables more descriptively. I will make for easier reading.
E.g.: instead of `j` you could name that `lineNum` or something along these lines.


How/why is this branch not merged to master, and instead the commits replicated?


Thanks!


I just logged-in in your gmail account with "********", I didn't know you loved knitting so much !


Hi kumar, that's actually quite good.
Happy to see you have found a use for `Graphics2D`.

Does it only work out properly for horizontal/vertical crossroads? I.e. what happens if crossroad is rotated a little?

Anyway, in terms of signatures, I think you are on a right way - you obtain `Graphics` as a parameter.
What I think is that probably you may want to ask for `Graphics2d` so that the calling code (external) does conversion for you. It will make the overall thing a bit more smooth (less points to fail)

Anyway - you mentioned there was an issue with passing Graphics, what's the issue?


Yeah but that is most likely a Cygwin bug so maybe this is the only way to work around it.


Error:

system/core/liblog/tests/liblog_test.cpp:1066: error: undefined reference to '__android_log_error_write'
system/core/liblog/tests/liblog_test.cpp:1246: error: undefined reference to '__android_log_error_write'
system/core/liblog/tests/liblog_test.cpp:898: error: undefined reference to '__android_log_error_write'
system/core/liblog/tests/liblog_test.cpp:979: error: undefined reference to '__android_log_error_write'
collect2: error: ld returned 1 exit status


- I thought we moved beyond cell automata because of the restriction it imposes.
- I feel you are channeling the spirit of the C language of old here  with this `int[][]`. Remember: Java. :)


Here's tty_done() from tty.c.
```c
void tty_done(void)
{
#if !defined(__DJGPP__) && !defined(__MINGW32__) && !defined (_MSC_VER)
        int fd;

        if (tty_fd < 0) return;

        fd = tty_fd; tty_fd = -1;
        tcsetattr(fd, TCSANOW, &saved_ti);

        close(fd);
#endif
}
```
A cygwin build should get that code, right? Is `_MSC_VER` only VC?


I hope this wasn't a large hassle.


I know that @WilliamHester and I use code linters (such as pylint) on our code using plugins for our editors. There are a few known issues with this (for instance, pylint doesn't like `flask.ext` imports), and a few spots in the code that don't follow these rules, but the standards are defined in our [docs](https://github.com/AuburnACM/auacm/blob/master/docs/contributing.md#style-and-additional-considerations) and an open issue (#22) exists for cleaning up legacy code.


Perhaps a switch can be used instead? This will allow for better additions to be made later on down the track...


s/scope1/notifier_scope/ or just `scope`?


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


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#cYJXsUFJTzK4LPzhwCqI7g


BTW here's tty_init().
```c
void tty_init(opt_flags stdin_mode)
{
#if !defined(__DJGPP__) && !defined(__MINGW32__) && !defined (_MSC_VER)
        int fd;
        struct termios ti;

        if (tty_fd >= 0) return;

/*
 * If we're in "--stdin" mode (reading candidate passwords from stdin), then
 * only initialize the tty if stdin is not a tty.  Otherwise it could be the
 * same tty, in which case we'd interfere with the user's ability to type
 * candidate passwords directly to John.
 */
        if (stdin_mode && !tcgetattr(0, &ti))
                return;

        if ((fd = open("/dev/tty", O_RDONLY | O_NONBLOCK)) < 0) return;

#ifndef __CYGWIN__
        if (tcgetpgrp(fd) != getpid()) {
                close(fd); return;
        }
#endif

        tcgetattr(fd, &ti);
        saved_ti = ti;
        ti.c_lflag &= ~(ICANON | ECHO);
        ti.c_cc[VMIN] = 1;
        ti.c_cc[VTIME] = 0;
        tcsetattr(fd, TCSANOW, &ti);

        tty_fd = fd;

        atexit(tty_done);
#endif
}
```
I wonder what's with that close(fd). Why do we do that, and why do we not do it on Cygwin?


Like i said bin, kenapa ga pake menubar yang kamu buat aja buat blog? Soalnya, menubar yang kamu buat di jumbotron ini butuh javascript hehe, ganti pake menu bar yang buat blog aja ya


`return unit_celsius == unit || unit_fahrenheit == unit` ?


Won't it work with the variadic template ctor only?


Maybe use variable variables? `$row->$fieldName`, where `$fieldName` is set in a `switch` statement earlier on.


> A cygwin build should get that code, right? Is _MSC_VER only VC?

Yes, VC is only env that does _MSC_VER define.


Выводи Прогресс бар


play against pc


Закончили выводить прогресс бар


Mmmh why restrict at max 4 neighbours? An intersection could have more than 4 roads connected to it (when some are forking out - think of an underpass to get onto a highway running above another road where the intersection is for example. That would be 6 roads connecting into the intersection below).


> I wonder what's with that close(fd). Why do we do that, and why do we not do it on Cygwin?

Not sure, but not 100% sure cygwin ahs tcgetpgrp() function.  I can remove that and see if it builds.


@AbdealiJK Umm I think it won't. Because so far we have made changes keeping in mind the requirements of `JSONEncoder` allowing `coala-json` to return relpath for files. We aren't making any changes in core modules are using files. But coala-json would certainly return relpaths. What do you think?


Начали выводить прогресс бар


Functions read_unit_from and read_unit_to are almost identical. Maybe you should move this code to single function.


@lexaux Yup, looks like it.


Once again, there's a waste of lines here, just set a variable `$sqlLoc = ""` and `switch` to set it to the right sql file.


The requirements state that you need to pass an object into the tree function, which you do on line 20, but the function itself isn't written to accept an argument. You code works because `treeObject` is declared on the global scope on line 4. But you need to rewrite this to actually pass the object into `buildTree`


Unnecessary spacing


Delete commented out code


Sorry - I did not mention ..
In sublime we use `coala-json --find-config` - so we do use the config file. coala-json began because i needed it for linters - it was an easy and good method to get results


It compiles with that code, AND the tty is in echo mode.  That's the bug.  I will revert this commit and make the TTY.c change


Why setters here when the Features (`one` and `two`) are public? (also, for `static` - see what Alex wrote)


`char unit{}` ?


Same as above.


@sckott just a heads up, not sure if this was unintentional or not?


I'm investigating the origin of that #ifndef


Just a few optimisation issues, but they can get fixed later on down the track. It's just me being picky.

Overall, without trialling and just reading through all of the changes, I concur that this is a good PR. :+1: Good job, solid effort!


it may be that at one time, this WAS a bug in cygwin (since it uses the getpid() and that may have not been working at one time in cygwin).   It works today, and fixes the problem.


Isn't that still the wrong way around ? Aren't the real secrets in `/tls` ?
(Also, this multiline comment is terrible to read, I was trying to associate each line to the comment ...)


was trying to learn commit/revert and this is the "re-commit"


This does not have a namespace prefix and is externally (as it's in a header) visible. Might not be a good idea.


Yeah I see it's older than proper john 1.6.31...


This is not needed - self can never be nil in ObjFW. Rather, an exception is thrown.


You can declare variables right here, with assignment. Pre-defining is a bit C'ish.


ajout d'un commentaire depuis GitHub


Use the Logger.


@sabanggraini Tambahkan class dengan nama clearfix di sini, Jadinya <div class="container-3 clearfix">
 clearfix itu bawaan bootstrap, gunanya untuk menetralkan col ketika bawahnya bukan col biar tampilannya bagus.
Buat suplement, baca ini deh : https://www.youtube.com/watch?v=GIC6rMlZwwg


I bet older cygwin (32 bit) likely did not have getpid 'working'  This code seems fine with 'current' version of cygwin.


Using a string as an argument to an exception is quite unfortunate, as this does not allow programmatically looking at it. What ObjFW does is to not pass any description, but have a description per exception type.


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


Hm, there's already OF_STRING_ENCODING_* - what about those?


:+1: For SASS


Yea, we definitely need to change this, do you want to just add getters and setters to the cubeprop object so we can manipulate it without global options?


@sabanggraini
di kelas .container-3, tambahin :
margin: 0 auto;
max-width: 960px;


This should be in the README


mn


Sounds like using regular names for the TMs and HMs could be the solution to all the problems. What would it take? 300, 400 more bytes of text? Looks affordable. I believe there's enough space in bank 1 for that.


But doing a fork, and when done, the keyboard simply does not echo sux, and for a user that does not know wtf to do to 'fix' the situation.  That is why I added this commit anyway.  I have been doing 'tty echo' after running a -fork cygwin ever since we got -fork running. Seeing other users with same issue made me make this commit.  FIXING the real bug would be so much better.


Done in subsequent commit


I would avoid hard-coding traffic light rules here. Better to link them up as a sets into a `TrafficLightManager` class which does all that stuff based on rules that we can inject. Think of it as a programmable rule box, it will be much more scale-able and adaptable to change in the long run (-meaning less coding down the line when we expand the system which is always a :+1: ).


Import render related commit 39a1ce3


thanks a lot!


Should we flip the input so that it is `(wfn, filename)`. This would make it more consistent with other calls.


Looks like I am the author for this bug, but I have no idea how you change is fixing it. Could you be a little bit more verbose on the root cause in the commit message?


This is still not taking an object as an argument. This is one of the requirements in the instructions. Your code works because thr tree object has been created on the global scope on line 18. But in order to meet the requirements you need to pass that object into `buildTree`. Rewrite this function to accept an argument, then when you call buildTree on line 8, pass the object in. Be sure to add comments throughout this code to demonstrate that you understand what it is doing


Thanks for this patch!

Could you do all log messages in

#ifdef _DEBUG
...
#endif

because all the logging is create while debugging, but way to much information for the end-user.
"Stopping webserver(s)" should be enough ;)


Example to access variable or attribute.


[R/parse-rtf.R:138:41:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/R/parse-rtf.R#L138) *style:* **Commas should always have a space after.**
```r
         "(", re_icd9_major_strict_bare,")",
                                        ^
```
[R/parse-rtf.R:139:43:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/R/parse-rtf.R#L139) *style:* **Commas should always have a space after.**
```r
         "(-(", re_icd9_major_strict_bare,"))?",
                                          ^
```
[R/ranges.R:305:27:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/R/ranges.R#L305) *style:* **Commas should always have a space after.**
```r
  majors <- icd9(paste(c[,1], sprintf(fmt = fmt, c[,2]:d[,2]), sep  = ""))
                          ^
```
[R/ranges.R:305:53:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/R/ranges.R#L305) *style:* **Commas should always have a space after.**
```r
  majors <- icd9(paste(c[,1], sprintf(fmt = fmt, c[,2]:d[,2]), sep  = ""))
                                                    ^
```
[R/ranges.R:305:59:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/R/ranges.R#L305) *style:* **Commas should always have a space after.**
```r
  majors <- icd9(paste(c[,1], sprintf(fmt = fmt, c[,2]:d[,2]), sep  = ""))
                                                          ^
```
[R/score.R:111:8:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/R/score.R#L111) *style:* **Commas should always have a space after.**
```r
    x[,"DM"] <- x[, "DM"] & !x[, "DMcx"]
       ^
```
[R/score.R:327:8:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/R/score.R#L327) *style:* **Commas should always have a space after.**
```r
    x[,"DM"] <- x[, "DM"] & !x[, "DMcx"]
       ^
```
[tests/testthat/test-icd-class.R:184:46:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-class.R#L184) *style:* **Commas should always have a space after.**
```r
  expect_true(is.numeric(pts_invalid_mix[c(T,T,T), "visit_id"]))
                                             ^
```
[tests/testthat/test-icd-class.R:184:48:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-class.R#L184) *style:* **Commas should always have a space after.**
```r
  expect_true(is.numeric(pts_invalid_mix[c(T,T,T), "visit_id"]))
                                               ^
```
[tests/testthat/test-icd-class.R:185:45:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-class.R#L185) *style:* **Commas should always have a space after.**
```r
  expect_false(inherits(pts_invalid_mix[c(T,T,T), "visit_id"], "icd9"))
                                            ^
```
[tests/testthat/test-icd-class.R:185:47:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-class.R#L185) *style:* **Commas should always have a space after.**
```r
  expect_false(inherits(pts_invalid_mix[c(T,T,T), "visit_id"], "icd9"))
                                              ^
```
[tests/testthat/test-icd-class.R:186:43:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-class.R#L186) *style:* **Commas should always have a space after.**
```r
  expect_equal(sapply(pts_invalid_mix[c(T,T,T),], class),
                                          ^
```
[tests/testthat/test-icd-class.R:186:45:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-class.R#L186) *style:* **Commas should always have a space after.**
```r
  expect_equal(sapply(pts_invalid_mix[c(T,T,T),], class),
                                            ^
```
[tests/testthat/test-icd-class.R:186:48:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-class.R#L186) *style:* **Commas should always have a space after.**
```r
  expect_equal(sapply(pts_invalid_mix[c(T,T,T),], class),
                                               ^
```
[tests/testthat/test-icd-comorbid-calcs.R:49:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L49) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,31))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:60:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L60) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,30))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:74:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L74) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,31))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:87:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L87) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,30))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:101:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L101) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,32)) #longer because 2x htn
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:121:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L121) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,32))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:130:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L130) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,31))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:142:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L142) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,31))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:155:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L155) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,30))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:170:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L170) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,31))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:200:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L200) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,32))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:211:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L211) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,31))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:224:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L224) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,32))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:233:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L233) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,31))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:256:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L256) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,29))
                                       ^
```
[tests/testthat/test-icd-comorbid-calcs.R:289:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-comorbid-calcs.R#L289) *style:* **Commas should always have a space after.**
```r
            expect_equal(dim(res), c(1,31))
                                       ^
```
[tests/testthat/test-icd-condense.R:99:41:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-condense.R#L99) *style:* **Commas should always have a space after.**
```r
    icd_condense.icd9(short_code = TRUE,c("1000", "10081", "10089", "1009")),
                                        ^
```
[tests/testthat/test-icd-condense.R:106:41:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-condense.R#L106) *style:* **Commas should always have a space after.**
```r
    icd_condense.icd9(short_code = TRUE,c("1000", "10081", "10089", "1009", "101")),
                                        ^
```
[tests/testthat/test-icd-condense.R:113:41:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-condense.R#L113) *style:* **Commas should always have a space after.**
```r
    icd_condense.icd9(short_code = TRUE,c("1000", "1008", "10081", "10089", "1009")),
                                        ^
```
[tests/testthat/test-icd-condense.R:122:52:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-condense.R#L122) *style:* **Commas should always have a space after.**
```r
  expect_equal(icd_condense.icd9(short_code = TRUE,othersalmonella, defined = TRUE), "003")
                                                   ^
```
[tests/testthat/test-icd-condense.R:214:52:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-condense.R#L214) *style:* **Commas should always have a space after.**
```r
  expect_equal(icd_condense.icd9(short_code = TRUE,"003"), "003")
                                                   ^
```
[tests/testthat/test-icd-explain.R:79:40:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-explain.R#L79) *style:* **Commas should never have a space before.**
```r
test_that("explain a single leaf node" , {
                                      ~^
```
[tests/testthat/test-icd-filter.R:22:44:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-filter.R#L22) *style:* **Put spaces around all infix operators.**
```r
  expect_error(icd9FilterPoaNotYes(visit_id=c("1", "2"),
                                          ~^~
```
[tests/testthat/test-icd-filter.R:122:57:](https://github.com/jackwasey/icd9/blob/03faaa3e4524480963a06cf6c48b675ef1f6932c/tests/testthat/test-icd-filter.R#L122) *style:* **Commas should always have a space after.**
```r
  dfrm <- dfrm[dfrm[["poa"]] %in% c("Y", "N", "y", "n"),]
                                                        ^
```


I believe its plugins.istanbul otherwise it didn't work for me anyway.


Example to call method(function)


Add comment.


Modify indentation levels.


https://github.com/magnumripper/JohnTheRipper/commit/5aebb0bc4f94e778be5f95599cd47313604a64a9 reverts this commit, and edits the ```#ifndef __CYGWIN__``` out of tty.c so that the tty is left in a sane mode.


I agree about streaming commits and tags not really being necessary. Trees I want to be able to stream, since they can be very big ([example](https://github.com/EFForg/https-everywhere/tree/master/src/chrome/content/rules)), but in this code it was going to be more complex to handle that. It could be done if the objects stream allows multiple objects to be read concurrently instead of serially, but I haven't looked further into it.

If objects data is stored in buffers instead of streamed, then other tools like [git-object-commit](https://github.com/chrisdickinson/git-object-commit) and [git-to-js](https://github.com/chrisdickinson/git-to-js) could be used more easily. The main reason I used streams was to allow them to work nicely with `sbot.blobs`, since that is the backing store for [ssb-git-repo](https://github.com/clehner/ssb-git-repo).

Having the same API for each object type is supposed to keep the [repo API](https://github.com/clehner/abstract-pull-git-repo) simple as it does not have to do different things for the different object types. I also added some higher-level utilities in [pull-git-repo](https://github.com/clehner/pull-git-repo) which make it easier to work with the streamed objects, like `getCommitParsed` and `readTree` which streams a tree's entries.


But why tho


Yes. f552e51


id


:+1: f552e51


This commit 'did' get the job done, but yes you are right, the correct fix was the right fix.  NOTE, I do not understand much of the tty.c crap.  Very unix specific stuff to me, and doing true debugging on cygwin is not my forte.  Thanks for pushing on this issue to get it right.


:thumbsup:


Looks great, Dhru! Nice work. You are ready to merge.


Code is fine.


Please change this to Fedora 22 and newer, as DNF on F22 is capable of processing weak dependencies.


Since this is a source processor, it doesn't need this tag (TriggerWhenEmpty)


`ffdf`


Nah, it doesn't.

This is the Codepen license which needs to be combined with any Codepen stuff.

This is the only substantial thing we borrowed from Codepen, so this is the only thing that needs this license.


Yeah, I figure there would be enough room in there for that. I just hesitate because I don't want to shift too much around in a minor release and accidentally break saves. I figure people who are currently playing would be more likely to switch to the newer version if it won't make them lose progress. But I've got it on my todo list to get them all properly named in the list and remove that extra check altogether, along with tracking down anywhere that might be calling the routine without properly defining a list type like TMs were originally.


Oh right - we can't put it under our copyright then, except you told me Codepen stuff was all MIT?


The syntax of the `suggest` property is different. https://getcomposer.org/doc/04-schema.md#suggest


Done in subsequent commits