Crash on startup:
16:29:50 T:2889618464   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
                                             - NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
                                            Error Type: <type 'exceptions.ImportError'>
                                            Error Contents: No module named requests
                                            Traceback (most recent call last):
                                              File "/home/osmc/.kodi/addons/script.trakt/default.py", line 6, in <module>
                                                from service import traktService
                                              File "/home/osmc/.kodi/addons/script.trakt/service.py", line 9, in <module>
                                                from traktapi import traktAPI
                                              File "/home/osmc/.kodi/addons/script.trakt/traktapi.py", line 9, in <module>
                                                from trakt import Trakt
                                              File "/home/osmc/.kodi/addons/script.trakt/trakt/__init__.py", line 1, in <module>
                                                from trakt.client import TraktClient, __version__
                                              File "/home/osmc/.kodi/addons/script.trakt/trakt/client.py", line 2, in <module>
                                                from trakt.core.http import HttpClient
                                              File "/home/osmc/.kodi/addons/script.trakt/trakt/core/http.py", line 2, in <module>
                                                from trakt.core.request import TraktRequest
                                              File "/home/osmc/.kodi/addons/script.trakt/trakt/core/request.py", line 1, in <module>
                                                from requests import Request
                                            ImportError: No module named requests
                                            -->End of Python script error report<--


Successfully cloned the GitHub repository containing Assignment2.Already did the coding in RStudio. Will reformat on GitHub abd recommit.


@tdragonite
just install requests and probably six like mentioned before


Test


hihihi


The only difference is that it uses the mac_file function and not the nvram function, and the TREBON flag was removed. I don't think this is the file with the issue.

Here's a new logcat for a new experimental build using your kernel (I ported yours to https://github.com/cmtrebon/android_kernel_samsung_msm7x27a and branch 3.0.101 now builds successfully): http://pastebin.com/LmnwCwHX

Problem with that is that the phone does not show the CyanogenMod logo, it shows the GT-S7500 logo then turns to a black screen. It uses the new device tree in the same GitHub account (cmtrebon), the same problem is there using the old device tree, so I think this is a problem with the kernel itself.


Resolves #6


It does do what I wanted it to do.

I don't understand why you let now check for a zero string, which won't trigger blacklist naturally.


I was gonna commit a fix like that! XD


Hello World


@ramako @JCristobal
Os expliqué que eso no debemos hacerlo así.
Primero paramos el servicio, después actualizamos el repositorio y, finalmente, iniciamos el servicio.

¿Por qué? Por que al no pararse el servicio al actualizar el repositorio, puede que se haya cambiado algo de los ficheros del repositorio que no tenga nada que ver con la aplicación que está ejecutando. Cuando la aplicación que está ejecutando, vaya a ese fichero a coger algo que necesite, no encontrará lo que debe y habrá un error.


Are you sure this will give the very same results and won't turn into a gimbal lock?


Oh, ye.. sorry, forgot you have removed them :-) ok, now start.
I report also an error (maybe you know it already) during sync: http://pastebin.com/mS6QHF2U
Tnx for everything! :-)


I've opened an issue: https://github.com/tcoulter/jockeyjs/issues/28


I did not understand what you were trying to do previously.

If ```blacklist_trigger``` was not empty (```-n```), why modify it at all?
It was that unnecessary (and wrong) modification that was causing troubles.


I'm sorry to have to tell you that but most of the spaces you replaced were there on purpose: to make sure that the line was at the same level on every text renderer (without caring about the number of spaces taken by a tab).

Like those two.


Do you want to send a pull request with that, as it's you that found the solution.


Same.


On my gear this slowed down loading of rockyou-with-dupes with 2 seconds, and memory use went like 5x (although that might just be virtual, not sure)


Same here.


I guess I solved it, working fine with Gmail,
will upload the fix in a few hours


Okay from now I will only comment the replacement you did right.


I get this when I run the tests:

    Assertion failed in test/testunusedfunctions.cpp at line 437
    Expected:

    Actual:
    [test.cpp:2]: (style) The function 'f1' is used in this file only. It can be made static.\n


Yup, this file is good.


This file is good too.


This file is good so far.


But not there.


Wait, what?


So far so good.


And nope.


This file is good.


Это ты оверрайдишь `.jumbotron` бутстрапа?


Oops, I left in `.swift` from where I copied it from.


After consulting with the team (@grozen, to be more specific), it seems more appropriate that the error code will be related to its specific activity, i.e.: `PreferencesActivity.RESULT_ERROR` or `RegistrationActivity.RESULT_ERROR`.


Oh sweet, thanks.
On Jan 25, 2015 10:41 AM, "XXMrHyde" <notifications@github.com> wrote:

> I guess I solved it, working fine with Gmail,
> will upload the fix in a few hours
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/XXMrHyde/android_frameworks_base/commit/111ba77f662be2a9f3bf4d761cff9766a1b0c116#commitcomment-9432088>
> .
>


These options are needed if you want to specify the host/port/user etc needed for running the unit tests. I'm not sure why you think this is not supported by MariaDB?


I think `hook` needs to stay here, otherwise it's not clear enough.


Hum, I don't like to have that much copy/paste.


Look great!


This new 'droppy' theme is basically Twilight, but better than the included theme. I will contribute it back to CM once I've tested it enough.


For comparison on the security side, there was a [widely publicized git bug](http://arstechnica.com/security/2014/12/critical-git-bug-allows-malicious-code-execution-on-client-machines/) last month that allowed arbitrary code execution.


Great :+1:


What do you mean by this comment? Oh, and if you have a suggestion on how to improve the code, add it as a line comment on GitHub like this instead of an inline comment; that way, we're sure to give it the attention it needs. :)


:+1: way to push logic down the stack


Thank you @thomas-mangin for all the help! :sparkling_heart: We're going to continue updating this cookbook to eventually release it to the Chef Supermarket for general availability. Plus this will help us learn a lot more about the options we have with ExaBGP in general.


What is this file?


@joshuaclayton, @mshytikov - that rspec example would be perfect as a default generated test in factory_girl_rails


Would you elaborate on why this test case isn't valid MiniJava? Yeah, `javac` won't compile it, but the whole point of this test case is to test expression statements, which aren't allowed in Java, but are in MiniJava.


Are we doing something with .json?


I recommend urgently to revert your changes. Checking for a zero string is nonsense.

The blacklist will be triggered if the blacklist_trigger array is not zero!


I'd prefer to rename it into `_template`, because:

* *feed* word is unnecessary here, since it's obvious that all in Feed class are related to feed;
* *_ (underscore prefix)* marks that it's for internal usage only.


Type /kill into chat. You will respawn in the nearest visited town and it will take 2 soul points.


i think line 41 just change color hex


With a smaller file it's even worse.
```
$ time ./pp64-b4.app < rockyou.lst --keyspace --dupe-check
1584345389899053977143859

real	0m5.566s
user	0m4.907s
sys	0m0.649s

$ time ./pp64.app < rockyou.lst --keyspace
1584345389899053977143859

real	0m9.070s
user	0m5.961s
sys	0m2.509s
```


Thats what I thought, your whole logic is broken and this check should simply be removed in my opinion.


I'm not sure here, but maybe remove this function at all and use two lines explicitly in `generate` method?


line 41 sould be fix it .but not sure

https://github.com/FusionSP/android_packages_apps_Dialer/blob/c824550da0ff0dadf195b76634b206ec43a2d24a/res/layout/call_detail_history_item.xml


hehe, `attachStenilBuffer` should probably be `attachStencilBuffer` (misspelled Stencil)


Shouldn't the .data() here be replaced as well?


It's not a *package*, it's a *module*, actually. Please fix here and in other generators.


My speed problems are probably due to memory use exceeding physical memory. I probably only have 4 GB and memory use went from 800 MB to 4.5 GB.


update


s/bot/both


It's important to note, I think, that it will be available in the root of output folder.


Awesome thanks :+1:


You might want to check for an empty blacklist where you remove blacklist packages, cause i removed the default blacklist code.

Commit https://github.com/manjaro/manjaro-tools/commit/d7ab4cd4467dc61f120ef830899a54f6d823c57a


## This is a markdown file


That was unexpected behavior that isn't documented.
I guess it much depends how you envision peru as a tool.
I think I see it more like make and you see it more like git.
As I said #83 would force a lot of security work on the project like limiting into where all of these commands are allowed to run. (only on directories? not allowed to fallow links? can't rewrite it's own peru.yml....)


в таких местах async/await можно убрать


I think it could be removed. IIRC, we do sort in `document-list.html`.


see #203


I just tried the link and it looks like it is a dead link.


I was´t really sure which path I should use there, as the left data is added to the widget_container and the replaced data is from the widget itself. Any hints are welcome.


на фон попробовал картинку воткнуть, потом заменю на ссыль с дропбокса


This is that `kwargs` trick I was mentioning before: declare all of the parameters we need, and leave the rest to `kwargs`.  Let me know what's different about what I did, and what you did, so we can think about why it didn't work previously!


i like it


The logic is not broken.

You need a blacklist and a trigger.
Since we want the blacklist triggered by one of a set of packages, you need to check for non zero blacklist trigger first, and then iterate the elements if they match your package you want to build.


hmm, man müsste am besten irgendwie herausfinden, ob gerade vielleicht doch der Kernel gemeint ist


Лучше не оверрайдить класс фреймворка, а дописать свой и добавить его к элементу


Logging a backtrace of a the offending thread before abort is absolutely something this code should do, great call.

@schneems Your assessment is correct. Ideally we'd like Rack::Timeout to always work BUT it has problems because `Thread.raise` has a bad habit of interacting badly with some C extensions. This at least gives those users that are having that kind of problem an out, with the knowledge that they probably need to tune threads = 1 or suffer the consequences.

Something to consider, @schneems and @texel: It sounds like the most common times that a request times stalls is a) an external service (database, etc) stalls processing the request or b) the server is overloaded and the request thread simply isn't getting any cycles to run. In both of those cases, aborting the process and restarting it won't solve the problem, it simply churns the application layer to try and push the timing errors upstream.

So, if this mechanism doesn't actually solve anything related to the true problem, why do it? Am I missing something? Why not just let the requests stall? Do you want some kind of upstream to see an empty response and show an error page to the client and thats why it's important to just abort requests?


Added a basic check that there are enough images to do what you want.


Btw this file used tabs for indentation change uses spaces.
What exactly are the coding style guides we follow?


I forgot about:  />


I am not talking about blacklist and trigger, I am talking assigning default values to these variables.

If blacklist and trigger and empty, do nothing.


IMHO it would make it more usable, if it was deployable directly from a git checkout, without having to have anything besides the bundled environment in place. (I like https://github.com/errbit/errbit as an example, which can directly get deployed via capistrano, and even has dotenv integration, that makes it quite easy to tune settings like SMTP, etc. outside of what is commited to git.)


I don't know if I understand it correctly, but generally speaking: the widget_container carrys the ID / path which is also used for the widgetData lookup. Anything below (except for containers like group or pages) will be identified by exactly this ID - so there won't be any extra entry in the widgetData anyway.


You were keeping track of an 'i' variable yourself; `enumerate` does that for you.  Also, I tend to call iterating variables with two letters `xi`, as some languages (like matlab) use `i` or `j` as a primitive (for complex numbers), and I find it a better way to keep track of iterating variables (`ii` is a generic iterating variable, `xi` can mean iterating over x, `yi`, `zi`, etc)


`dtype` was declared above; use it!


Thank you a lot!


Tests?


Why are we changing things back to `console.log` here?
`log` should work with the `util.format` right?


Can you explain the trick here?
I think you add an extra value to have a real transposition but I have :
>>> numpy.array([int]).repeat(2, axis=0).T.flags
```Python
  C_CONTIGUOUS : True
  F_CONTIGUOUS : True
  OWNDATA : False
  WRITEABLE : True
  ALIGNED : True
  UPDATEIFCOPY : False
```
>>> numpy.array([int]).flags
```Python
  C_CONTIGUOUS : True
  F_CONTIGUOUS : True
  OWNDATA : True
  WRITEABLE : True
  ALIGNED : True
  UPDATEIFCOPY : False
```
so except for the OWNDATA flag, others are the same.
Indeed, I don't know why but I have:
>>> numpy.array([int]).repeat(2).T
array([<type 'int'>, <type 'int'>], dtype=object)

Can you check it on your computer?

If you have the expected value on you computer, (`numpy.array([[int], [int]])`), It means you have two value in your array. You should change the type_dependencies file to 'check forward' on t[0][0] in these case or use the flat attribute likes in types/conversions.py


better to avoid copying more complex code, and duplicating only the most simple code.

This entire function duplicates plenty of code unnecessarily; we should rewrite it to be shorter and cleaner.

It also loads the entire image data every time `test`, `train`, or `valid` is created (rather than just the images that are needed, making creating the dataset 3x slower than needed.  A good goal for the future would be to fix all that.


Ideally we'd add an `else` to raise an exception if another set is passed.


@iblancasa @JCristobal

De acuerdo con la documentación (http://docs.ansible.com/service_module.html) la sintaxis correcta seria
`- service: name=httpd state=started`

Pero es de esperar que así también funcione.


The shapes were transposed for what `pylearn2` needed.  You didn't see an error because of another bug below.


Without adding this line, I didn't see the bug above exposed.  With a super class that you didn't write, never assume you know that it's returning--always make sure to call it and return whatever it gives you (even if it is `None`).


Looks fine to me


Take a look at the code and let me know if it needs further comments.  I don't want to add comments in github, they should be in the code!


Saving is the slowest thing that happens; saving frequently is only useful for slow, long-running simulations.  For us, this really should be something large (1 million)--`pylearn2` will always save at the end.


Ah, gotcha!


s/stars/starts/


looks like you're missing '#' here


@ramako me fijé, pero como en tu [commit](https://github.com/iblancasa/BackendSI2-IV/commit/0e731918b0345d850f5391e96cfc70af73c6653f) lo pusiste así lo he dejado. De todas maneras, así funciona también


Note that this value can't be arbitrary anymore; the selection code I write requires `nvis / nhid` to be a square.  So, `32 * 32 / 64` is `16`, so the code works just fine.


dead code


put the stylesheet link in the head section


Very nice, :+1:  @ahmadsyarif

Makasih ya boss @vicknzr83

Pak @Marzuki001 trus ini diapain sekarang? :dancer:


Lost a few commit messages because I deployed the update from another machine and it somehow reverted those commits :sob:.


That's a good point. @olson-sean-k what do you think?


This will need a `try...catch` around it in case `nativeStringify` throws. We can probably just pave `Date.prototype.toJSON` in the `if` block to avoid that. (This will also let folks set `Date.prototype.toJSON`, which wasn't possible before).


Fixes shamans learning rogue spells...


I hope this method works.


Megaman & Bass was mistakenly ID'ed as using FLASH RAM, however, it really uses EEPROM.

But more importantly, Pokemon RSE boot!!!


Ok, but then I don´t understand why date.type is replaced there. As far
as I unterstood the code, data.type is set in the widgets create method
(btw. it is missing in web and wrong in group) so there should be no
need to set it again in this place right?

Am 25.01.2015 um 17:07 schrieb Christian Mayer:
>
> I don't know if I understand it correctly, but generally speaking: the
> widget_container carrys the ID / path which is also used for the
> widgetData lookup. Anything below (except for containers like group or
> pages) will be identified by exactly this ID - so there won't be any
> extra entry in the widgetData anyway.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/peuter/CometVisu/commit/28733c596761788c0b6e6974f4a8b5a6cff990cb#commitcomment-9432195>.
>


Nice catch


On Jan 25,  8:15am, notifications@github.com (=?UTF-8?B?RWxhbiBSdXVzYW3DpGU=?=) wrote:
-- Subject: Re: [file] add lz4 bookmarks from wiz (8595a9a)

| looks like you're missing '#' here

I meant to delete it, thanks!

christos


Committed as "Beam1, H-pol" but this is "Beam4, H-pol" ...


History... I got annoyed by setting it everywhere - especially as that information is available and by manually setting it got wrong (as you have discovered...).
It should be removed at the widgets.


This is a trick to avoid recomputation of this value? Compiler are not smart enough with it for now? :)


Should we just move to using Gradle? :)


I dont understand the logic in this:
~~~~
    if [[ -n ${branch} ]];then
	branch=${branch}
    else
	branch='stable'
    fi
~~~~

Does this not make more sense:
~~~~
    if [[ -z ${branch} ]];then
	branch='stable'
    fi
~~~~


Should `<scope>test</scope>` also be included?


Thanks for explanation.


PEP 8 suggests spaces. PyCharm out of the box suggests to follow PEP 8.
The way the file is displayed in PyCharm gave me the impression that the files used a mixture of tabs and spaces, but now i see this was not the case.

If we agree on PEP 8 i could commit how PyCharm would fix the format.
Otherwise i would reintroduces tabs.


CC @hartsantler


а можно и оверрайднуть, если мешает


I used the template format because I found that exact code in the D forum. I think it's used because enum ensures that it's a compile time constant. That way the function call gets replaced by just true/false.
I tried to use your format in code I wrote later, but I couldn't get that to work. I think it's because the compiler writers haven't prioritized making alias and symbols as template parameters easy and good looking. Those who use it can live with having to write the code in a bit strange way.


You should commit this through anke. Now this is going to be overwritten the next time someone pushed a commit to anke.


## This is a markdown file


Yeah, that seems like it could be useful.  Although, the BCD values are readable and easy-to-edit in their current form.


機能: コンタクト先3件(データは決め打ち)を画面に表示するだけ

CollectionViewを使っている。


https://github.com/g-truc/glm/blob/master/glm/gtc/quaternion.inl line 171 and the correction for YXZ seems strange because in the example "FirstScene", the camera acts normally but http://www.euclideanspace.com/maths/geometry/rotations/conversions/eulerToQuaternion/index.htm says that it's for y,z then x.


Though I started using requireJS, I ended up just using grunt to concat all js files into one.


Rearranged code a little in FtpConsumer to make it more similar to FileConsumer


[Only because the compiler is so old in Travis CI](https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html). But I'll change it.


Why types are not merged?

In [25]: a + 1.
Out[25]: array([ 2.])


GNU General Public License, version 2


A small nitpick, but the image for this one actually does show the closure under the Library Calendar section in the bottom left. It's definitely not very visible, but it's there.


SHouldn't we crash in this case?


i think i have to do a commit, this shows on the frontpage @jundl77


Great post.  You highlight an issue that really worries me.  While I am very committed to the idea of open government data in theory, the data obtained can certainly be used for ill purposes--even if unintentionally--and the government's use of the data is actually the least of my worries.  Even Harlan Yu's treatment of this issue at the end of the last class left me feeling somewhat skeptical.  What to do?


Moze moze, samo je ubaci u onaj fajl funkcije.php , da ih cuvamo sve na jednom mestu.


```json
{
  "version": "0.6.11",
  "approved": true,
  "approver": "JayGray",
  "approvalDate": 1422203862867
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


Regenerate these keys if you haven't already been pwned...


so it does! Good catch :)


test


I just made a sync on my desktop. I am expecting that my assignment is pushed onto GitHub successfully.


Well, if it works the same then I'm okay with it.


Yes, can do. Give me a couple of days.


actualizacion


Should now be also on anke.


```json
{
  "version": "0.6.11",
  "approved": true,
  "approver": "JayGray",
  "approvalDate": 1422203980552
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


You accept list as array like? :-)


а потом тратить часы на поиски, что же это за херня такая и почему у обычного жаботрона вдруг фоновая пикча


what is kodiplatform?


Updated on Jan 25th


"Using a CompositeView"の完了時点 (p49)

* CollectionViewの代わりに、CompositeViewを使うようにしている


don't work in django 1.7


This could be replaced with `c, err := base64.StdEncoding.DecodeString(auth[6:])` since the prefix has already been verified by `strings.HasPrefix`.


I found this miss the odin config.


The comment


Pruebas de funcionamiento de gitHub


OH YES THERE WAS


Is there a real reason to not perform SIMD with float and int addition?


Yeah, right!


Sure, I'll do that as of tomorrow


@amypotvin hahah [yes](https://github.com/rsalmond/concrete-jumble/tree/master/backgrounds)!

We gotta get you committing ASAP. Let's do a git jam session!


Ich hab das tatsächlich schonmal gehört, hab dann aber trotzdem nochmal gegoogelt und das dabei rausgefunden: http://de.wikipedia.org/wiki/Makro


I also recommend to use the -z way ...


Following the example verbatim gives me the following error at the call to configure:

I am running from the VS2012 x86 Native Tools Command Prompt as indicated in the directions.

c:\php-sdk\phpdev\vc11\x86>configure --disable-all --enable-cli
Saving configure options to config.nice.bat
Checking for cl.exe ...  <in default path>
  Detected compiler MSVC11 (Visual C++ 2012)
ERROR: 64-bit compiler required


As long as it works fine with MSYS :+1:


Shouldn't you use {}?
If std::get return 1, 2, 3, you want shape({1, 2, 3}) and not shape(1, 2, 3) no?
(I assume I am wrong but I would like to understand ;) )


OK, I guess.


@rsalmond I don't know what that means, but I'm IN!


Make sure everything builds normally.


do you really use that crap? I kicked this out of kodi when I implemented ActiveAE. Please use ffmpeg or soxr.


Okay with the asking that's quite nice.
I'll test coala on our project next time, but now I have something to do for my studies since there's only a week left :)


This line is redundant for this batch.


I have the captcha code in this repository https://github.com/SiegWahrheit/Captcha-code
and it's ok when it runs alone.(You can test it in the repository above), but when i add the code for the captcha control in the view directory it's not working.

The code right now creates the captcha image and is putting it in the right position both at the first login and the login-fail page.
The only problem is that it's not working like a captcha(it's always true when it's not empty)

So the only problem is that i don't know where to put the captcha-control code.


contains 2 files - HTML and R


Beautiful!


This line is redundant too here. I don't know, maybe just keep this subroutine as it is anyway.


Is there a reason for using `new UUID(long, long)` rather than `UUID.randomUUID()`?


Sorry!
Get also something like this:

```
 You have a problem! 'che/' is not writeable. Please ensure that the permissions are set correctly for the installation process.
nfig/ is writeable.
nfig/dynamic is writeable.
gs/ is writeable.
/userdata/ is writeable.
/config/ is writeable.
```
The folder names are partly incomplete.


Mantap!!!


Also, I understant select_shape as : "Indice of the first other array with the same dimension as the first array". I don't understand how does it impact final shape.


typo here


You seem to miss some commits for ca. one week, please pull )

Thanks


close #14


I think we have to many accessors. Do you know what operator() is use for? Why we don't just use operator[]?


## ”Using Events”の完了時点 (p51)

* テーブルの行をクリックするとハイライトがオン/オフされる。
* 実装は、ItemViewでclickイベントを拾って、warningクラスをトグルさせている。

画面はBobの行をハイライトさせた状態
![image](https://cloud.githubusercontent.com/assets/511613/5892342/b6b10efa-a4fe-11e4-8a14-eced56057cff.png)


typo


fixed


I have removed the spaces and put back tabs in this commit:
https://github.com/rusthon/Rusthon/commit/313e9ab23aa4697af47db653195399dff7405aaa

The coding style is use tabs, and generally use the style you see which is basically lower case `snake_case`.  PyCharm won't auto switch tabs vs spaces based on the file?


I assume you will squash this with the other cov patch if everything goes well, right?


What's wrong with samplerate?
Yeah sure if ffmpeg is better we can implement wrapper functions in library.xbmc.adsp. So we can directly access it from kodi.


Commented in some stuff, wait for the others too.

I can delay tonight's nighlty if you tell me so, so that you have more time.


I dont know how these variables are used afterwards, and @udeved said that my commit was [somehow wrong](https://github.com/manjaro/manjaro-tools/commit/a333e01c51216afd9a977265a5c3d2c21b6213d3#commitcomment-9432131) so I removed it..

If my [earlier commit](https://github.com/manjaro/manjaro-tools/commit/a333e01c51216afd9a977265a5c3d2c21b6213d3) was correct can revert this one.


Here there were 2 typos BTW, preset and decrease


aaaaa


kodiplatform (https://github.com/Montellese/kodi-platform) is a library that comes from pvr-addons (tinyxml, threads, ...).


I guess it is not needed. I will remove this part and the other one.


bbbbbbb


Should be fixed with the last commit


OK, it's your call. Though I wouldn't mind keeping it as it is.

BTW, we could really move some common stuff to a common file.


Is this supposed to work on apple trailers? Doesn't seem to here, maybe user error/ (mine
Ex. choosing a trailer at random -
$ mpv http://movietrailers.apple.com/movies/independent/amirasam/amiraandsam-tlr1b_h480p.mov
Playing: http://movietrailers.apple.com/movies/independent/amirasam/amiraandsam-tlr1b_h480p.mov
[ytdl_hook] WARNING: Falling back on generic information extractor.
[ytdl_hook] ERROR: Unsupported URL: http://trailers.apple.com/
[ytdl_hook] youtube-dl failed, trying to play URL directly ...
[ffmpeg] http: Invalid cookie found, no value, path or domain specified
[ffmpeg] http: Invalid cookie found, no value, path or domain specified
[ffmpeg] http: Invalid cookie found, no value, path or domain specified
[ffmpeg] http: Invalid cookie found, no value, path or domain specified
[ffmpeg] http: Invalid cookie found, no value, path or domain specified
Failed to recognize file format.
(- using latest youtube-dl update

Works fine as before if setting agent in command, ie. -
 mpv -user-agent 'QuickTime/7.6.2 (qtver=7.6.2;os=Windows NT 5.1Service Pack 3)' http://movietrailers.apple.com/movies/independent/amirasam/amiraandsam-tlr1b_h480p.mov


@udeved told you to revert your changes, not to remove the complete code ...


prvi


?? too


@Shoozza really nice work man!
This now works `./rusthon.py ./examples/call_c_from_c++.md`
and correctly outputs
```
running: /tmp/rusthon-c++-bin
hello world
hello world from c++
Hello world from C
```


Note that this doesn't cover the case where one has defined `MPCHC_MSYS` but the path doesn't exist.

It's something I generally want to cover eventually.


Can you please explain why this change was done? after this change MySQL on my system can't start and I get this error message:

    Fatal error: Please read "Security" section of the manual to find out how to run mysqld as root!


Suggestion on the above error? I'd like to get this figured out for Windows as it works perfectly in Linux...  The directions don't seem to work well on Windows 7 and VS2012.


> BTW, we could really move some common stuff to a common file.

I will look into it, but another time.


Should I switch to the x64 command window as I did before?


Damn, I can't type when it is late.


Why did this file get removed?


While the extra sourcepath links are useful, they aren't relevant to this pull request.  They should go in a separate pull request.


Looks like a step backwards to me.


does it work with new GOOGLE RichSnippets???


reeceyang@live.com or liuyoyo999@gmail.com


His earlier commit is wrong in my opinion, firstly he is checking non-empty an array, and then he is assigning an array to a variable, leading to unpredictable result...
~~~~
 if [[ -n ${blacklist[@]} ]];then
 blacklist=${blacklist[@]}
 else
~~~~

Thats why I had the same value repeated twice or thrice in the blacklist output...
~~~~
==> BLACKLIST:
  -> blacklist_trigger: eudev upower-pm-utils eudev-systemdcompat upower-pm-utils eudev-systemdcompat upower-pm-utils eudev-systemdcompat
  -> blacklist: libsystemd
~~~~
I will revert to my [earlier commit](https://github.com/manjaro/manjaro-tools/commit/a333e01c51216afd9a977265a5c3d2c21b6213d3) as it tries to achieve same thing which @udeved was doing earlier


The issue with 64-bit compiler error was fixed about a week ago, so you shouldn't have that error, that's why i meant you'll get it fixed by a pull. If you can't pull for some reasons, then yeah better switch to x64. But then you also need the x64 deps.

Thanks.


## "Animating the Removed ItemView"まで完了 (p59)


@fritsch found out that it is extremely resource hungry. In the old SoftAE it was configured at lowest quality level in order to run on all platforms.


Only YASM.bat case does not. Probably because it is internal script and at this point we know that compilation will work. But sure I will add missing check.


That's something used by some pvr addons. Is that used here?


I got you on this one. I'll try to simply the code. I don't know why he added this [@] in it ...


@AchimTuran see the different quality levels we have for the ffmpeg resampler and how we configure it concerning Nicquist, width and so on.


According to the boost docs, copy_file will throw if the source file doesn't exist. So this would still fail when there aren't any files yet.


please try with #define ALLOC_NEW_DUPES  0x10000


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=ad91b02e7cfe4bcf91a4b1ec62072f44
 plone-5.0-python-2.7 [SUCCESS]


для этого надо всего-лишь положить оверрайды в удобное место


Thx @manjaro


I think i know now what is meant with "streaming" ;)


I have no clue why you try to change something you stated you don't understand aaditya.

@manjaro
I didn't get you with your comment.
I have nt changes anything, aaditya did which screws blacklisting entirely.


Pas de soucis


Whatever, do what you want, I am slightly annoyed.

if value are uncommented in manjaro-tools.conf, they will be used, else it used default values in code(util.sh)


@udeved thats why asked for your comment [here](https://github.com/manjaro/manjaro-tools/commit/a333e01c51216afd9a977265a5c3d2c21b6213d3).

Your logic for the array checking and subsequent assignment is broken though.


Output does not need to be in parenthesis, make sure the interpreter works with and without them.


Hugh, thank you for playing and extending!


Check if it works now: e5d89f7ee3ddb7c0cc576d79b9d0714cf63013f5


thks I fixed it


Push to **master** also?


aaditya

Please give more details than just stating "its broken".
It works perfectly for eudev stuff.


Yes, the includes don't work inside a module.

-Samuel E. Giddins

On Jan 25, 2015, at 1:44 AM, Arne Harren <notifications@github.com> wrote:

@segiddins it would be cool if we could leave the lcl_RK.h/m files untouched as they originate from the original source code by running lcl_embed (cf. https://github.com/aharren/LibComponentLogging-embedded). Modifying them inside RestKit has 2 drawbacks: 1) we cannot easily update (by just checking out the latest version and running lcl_embed), and 2) it will break un-embedding in conjunction with CocoaPods (cf. https://github.com/aharren/LibComponentLogging-configure 2nd example).

Are the #includes problematic?

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


You have to load defaults when they are uncommented. You don't have to set them twice. That's all.


Yeah it is used for XMLUtils.cpp, to store the settings in xml-files.


do you know how to fix screen blink issue when ambiant display enabled ?


```io/file``` is variadic and already handles platform specific paths. Could simplify this and others.

```clojure
(io/file (util/output-directory opts) "goog" "bootstrap" "nodejs.js")
```


@udeved I think I have reported this issue many times before, there are same multiple entries in blacklist as I commented a few posts above.

https://github.com/manjaro/manjaro-tools/commit/d7ab4cd4467dc61f120ef830899a54f6d823c57a#commitcomment-9432416


Aaahja... da hat wohl Bootstrap bei dem Release mal Mist gebaut - und den Fix https://github.com/twbs/bootstrap-sass/commit/daeb43dcc7b0ab06328acaca0549ee68c039aaa6 noch nicht ins Release ausgeliefert. Hab das jetzt mal bei uns eingespielt, damit funktioniert es jetzt wieder.


Ah yes, this is much better. Thank you. Some thoughts:

* I believe the `0 \dots k-1` is correct, because we want *k* bits (assuming the `\dots` notation denotes an inclusive range). Or am I missing something?
* To reduce the bias of the top bit, we could encrypt *d_a* * 2^*k* / *n* rather than straight *d_a* (i.e. renormalise *d_a* from the [0, *n*] range into the [0, 2^*k*] range). However, that won't entirely remove the bias (*d* < *n* < 2^*k*) and would require careful checking of the bigint maths to ensure those multiplications and divisions are lossless, so I'm not sure it's worth the effort.


@thheller heh thanks, I did not know that or had forgotten.


:wink: Thanks!


Hi,
have you tested this further? Are there any regressions with different resolutions?
If all is good, I would like to ask you to pull this change: https://github.com/Mr-Dave/motion/issues/28


Yes, and it works, we talked about it extensively on irc and I supplied you with the build log.


Bro, kalau kata _binding_ untuk pemula yang baru belajar pemrograman gampang dipahami ga? Ada pilihan kata lain ga?


Ok didn't know that, but when I use it for my FIR-Filters it toke very long to resample it and that's one problem when creating a audio stream.

@fritsch what's about the SNR?


https://github.com/manjaro/manjaro-tools/commit/e5d89f7ee3ddb7c0cc576d79b9d0714cf63013f5 should fix it. Needs testing. I don't get the **[@]**  within **${blacklist[@]}** and **${blacklist_trigger[@]}** so we have to check it. I think it is not needed ...


I have to admit, I got a little help from the interwebs on this one but I do understand what the code is doing here and like how concise it is. It's good to know (as you mentioned Thursday evening) that the `#send` method can have access to private methods and to be careful with its use. It just seems okay to use it in this small piece of code and not have to define the operators in separate methods. It would be kind of cool if methods converting operators in string format to actual operator methods was added to the Ruby standard library. It could be something like `#to_o` although this looks funny and the "#to_something" convention usually converts an object to a different class object, not a method. Maybe the method could simply be named `#operator`.


I don't set them twice.

if we use -z or -n is basically trivial and amounts to the same logic just differently asked.
I don't get the issue here, except aaditya's changes broke blacklisting.


Don't set variables twice. We already loaded them from our conf file.


合并原作者


@manjaro will test.

@udeved it may work for you but it does not work for me most of the time unless i clean chroot.


Have you checked https://github.com/Mr-Dave/motion ? Some RTSP work has been made there, and I rememeber auth also added, wasn't it? (so we all are on the same track)


Whatever, I am out for this evening. Test you changes, if it won't work, I will revert them or fork the repo.


Okay, that makes sense. I will change and commit. I think since `puts` is located in the separate executable file, I wasn't thinking about how it was outputting in the terminal. I was thinking it was simply returning the evaluated number, not "printing out" the evaluated number.


Most have this: http://src.infinitewave.ca/ in mind - but they don't understand that this tests ffmpeg with default settings, that are optimized to run on the slowest platform available (cutting Nicquist, very few samples and so on) - therefore I mentioned our quality levels.


The original problem of sending long command string was when the cmd sent
is echoed back . The echoed command adds some line feed characters which
conflict with the matching pattern in the expect statement.

On Saturday, January 24, 2015, hakrishn <notifications@github.com> wrote:

> Hi Jon,
>
> I have tried building pexpect from latest source and it seem to work even
> with large command strings sent to remote SSH session. This may avoid
> adding too many work arounds in the drivers and any function overload of
> sendline() in TestON.
> Note: I have char 'z' at beginning and end of the string.
> ************************With latest pexpect (sent 503 chars with out
> stripping)*******************
> echo = False >>>>it seem to work only with this value set to False<<<<<
> child = pexpect.spawn('ssh admin@10.128.40.50
> <javascript:_e(%7B%7D,'cvml','admin@10.128.40.50');>', echo=echo,
> timeout=5)
>
>  child.isalive()
> True
> child.sendline(t)
> 503
> child.expect('\$')
> 0
> print child.before
> zaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaz:
> command not found
> admin@chONOS-MN:~
>
>  *****************************With pexpect 3.1********************
>
>  child = pexpect.spawn('ssh admin@10.128.40.50
> <javascript:_e(%7B%7D,'cvml','admin@10.128.40.50');>', timeout=5)
> child.isalive()
> True
> child.sendline(t)
> 503
> child.expect('\$')
> 0
> print child.before
>
> aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaazaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa
> zaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaz:
> command not found
> admin@chONOS-MN:~
> print child.after
> $
>
>  —
> Reply to this email directly or view it on GitHub
> <https://github.com/opennetworkinglab/ONLabTest/commit/aea67aa184bad560e8d4869e5115a5d81cde95ec#commitcomment-9430173>
> .
>


{% load i18n %}


No, because 1) it changes a thing that doesn't exist in master, 2) the other thing that is changed is essentially refactoring since AutoHotkeySC.bin still supports reading `>AHK WITH ICON<`.


Does this take into account the following:
RCS "torque" shouldn't be added to the calculation when it's not really present because:
- RCS is toggled off (global check once - applies to all parts) , or
- RCS partmodule is disabled (user can do that on the right click menu or via action group.) (check per part), or,
- Monopropellant resource is empty (probably just check once globally, not once per part, because this resource isn't pathed like liquidfuel or oxidizer. It ignores crossfeed),


@udeved if thats your attitude i will simply stop filing reports or making any changes.


ahh. mis-understood your comment above...


# Matrix inversion is usually a costly computation and there may be some benefit to caching the inverse of a
# matrix rather than compute it repeatedly. The following two functions are used to cache the inverse of a matrix.
# makeCacheMatrix creates a list containing a function to
# 1. set the value of the matrix
# 2. get the value of the matrix
# 3. set the value of inverse of the matrix
# 4. get the value of inverse of the matrix


My attitude is, that I dislike changes made when it is stated that the one who made changes doesn't understand the code.

I find this annoying. You can't simply change something you don't self admittedly
 self admittedly understand.
Your initial change broke blacklisting.


make const reference?


I have a question about the regex on line 18. I noticed in the session, you used `/\d+/` instead of simply `/\d/`. I don't fully understand the benefit of '+' in this case. When should it be used?


## This is a markdown file


I guess I will do this now.


sadly this doesnt do the job.


no sorry mate, for me ambiant display, lift to wake and adaptive display dont work at all.
still have to look whats wrong there


See ShadowNinja/LuaIRC@b1382c9885d2d3e00debc929e4e03e3c88867f0d (#25).

Your method looks cleaner though.  I'll try to implement it.


also note that ffmpeg allows usage on soxr if desired.


@Fyrd should this stay as http2?


ok mate thanks anyway


Please see my response to these Qs in the other thread.

If there's an autogenerate tool then perhaps there's something wrong with the original TTF.  Either that, or the tool had a brain fart.  Either way, I doubt that my manually-created fonts for ! ' and | are exactly what the autogenerate tool would've come up with.


@kyleboon did some performance tests on this. Specifically `randomUUID()` uses `SecureRandom` which is blocks.


For simplicity of having a single way to write the name. I've seen (even like 3 minutes ago) devs write it as LoDash. The hyphen throws people off and this way it's consistent with the package  as well.


I like this


The commot message doesn't seem correct.  This saves the nick, not the access.


Thanks!


Ahhh ok thanks

-joedf

On 2015-01-25, at 12:30 PM, "fincs" <notifications@github.com> wrote:

> No, because 1) it changes a thing that doesn't exist in master, 2) the other thing that is changed is essentially refactoring since AutoHotkeySC.bin still supports reading >AHK WITH ICON<.
>
> \
> Reply to this email directly or view it on GitHub.
>


Ok I wanna look deeper into it. I think using samplerate was more an proof of concept.
Alwin did the most part of this addon.


Hold on a sec. We had followed code:

 if [[ -n ${blacklist_trigger[@]} ]];then
blacklist_trigger=${blacklist_trigger[@]}
else
blacklist_trigger=('eudev' 'upower-pm-utils' 'eudev-systemdcompat')
fi
if [[ -n ${blacklist[@]} ]];then
blacklist=${blacklist[@]}
else
blacklist=('libsystemd')
fi

which I changed now to:

    if [[ -z ${blacklist_trigger[@]} ]];then
	blacklist_trigger=('eudev' 'upower-pm-utils' 'eudev-systemdcompat')
    fi

    if [[ -z ${blacklist[@]} ]];then
	blacklist=('libsystemd')
    fi

With this it should work as intendend when blacklist_trigger and blacklist is empty. Else blacklist_trigger and blacklist are already set and don't need to be set again.

Regarding [@]. This is needed to have no globbing and each element will be its own argument.

If there is now a misunderstanding within these changes we can simply revert them. In the end it should work and do the job. And there are always more ways to Rome ...


I tried to add the ability to specify the phenotips id inside the pedigree, but this is as far as I got.

One thought:
If free text entry is allowed in the external id field, then there is no way to tell for sure if the id is a phenotips id or just an id that matches a phenotips id. What is the external id currently used for?


@wcandillon thanks. You just forgot to replace 'form' with 'formData' :)


I'm not sure either, but testing this code is much easier if it is a function. Probably this is the reason I used the function.


Right Phil, you need the @ to iterate each element, otherwise you have to split the string.


# There are some benefits of Matrix inversion like caching the inverse of a matrix rather than compute it repeatedly. # The following two functions are used to cache the inverse of a matrix


hi


No, it doesn't. I didn't really think about it, as the previous code didn't check for these conditions either.


You just don't know what it's like to be me.


The reason why I didn't use a for-loop here is because for some reason the C program went into an infinite loop and I didn't find a solution for it.


Das Pferd von der anderen Seite aufgezäumt @phil fällt mir dazu ein. :)


should make multi-threading easier (carter suggested)


I was able to register an account without the faucet using one of my accounts as the pay-from account.


Here's hoping this is in some way helpful.
Have to restart kodi when it tries to scrobble shows or movies before the script will work again - http://pastebin.com/N1tpVipN
The scrobbling did work once with one of the very recent commits but can't seem to get it going again.
Manual sync for movies went smooth as hell but crawls on episodes (took about 20 minutes) but it completed at least :P

"03:34:15 T:36644   ERROR: WARNING:trakt.core.http:Continue retry since status is 504"


@aadityabagga: And now the conclusion why you get sometimes double lines in your log:
You have followed triggers: 'eudev' 'lib32-eudev' 'upower-pm-utils' 'eudev-systemdcompat' 'lib32-eudev-systemdcompat'. For example you have 'eudev' and 'upower-pm-utils' in your list it will be triggered twice. So with these triggers it can be triggered maximum 5 times.

Your issue is some different. From the user perspective you want to have the triggers listed only once in your log and not double or more.

To handle that, a loop routine will solve the issue. On the other hand it is just cosmetics and can be ignored.


o0o0 fancy ansi colors!


just found it in styles. CallLogSpinnerStyle line 188. but thanks anyway for help searching

Cheers


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 149](http://192.168.30.225:8111/viewLog.html?buildId=652&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 149](http://192.168.30.225:8111/viewLog.html?buildId=652&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 127 Build time: 00:00:01


TeamCity Ylab :: zmactep/ylab [Build 186](http://192.168.30.225:8111/viewLog.html?buildId=653&buildTypeId=Ylab_ZmactepYlab&guest=1) is now running


pokemon combat


If you get the last error it's trakt not answering your request in time :/


Can't wait to check. I was fighting it for a few hours. Left it thinking it was some kind of an issue with Flot.


Gamea.clashofclans.com


tár


Thank you very much for your response!


siema


@manjaro thx for the explanation.

I will stand by my comment though and will not involve myself too much in @udeved's work as does not like my involvement. I have been quite most of the time for the issues I am having.


need to capture the rest of these values

also add a `transactions` [object] to the payment schema, so you can just add the `transaction` response object as you get them to that array,  to have a record of the raw response objects.


```json
{
  "version": "0.6.11",
  "approved": true,
  "approver": "JayGray",
  "approvalDate": 1422208850478
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


beautiful thank you


Build error need / not \


Looks cool!

:bowtie:


New line needed


```json
{
  "version": "0.6.11",
  "approved": true,
  "approver": "JayGray",
  "approvalDate": 1422208985962
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


Tabb


@aadityabagga: You don't have to be quiet about issues you have. Else we won't see them. We are not perfect. @udeved sees the technical side. You on the other hand found it quite unusual to see some lines double. As it is just cosmetics your the application is not broken due this. We might find a solution for this issue but changing the code as you did simply broke the code. Please test https://github.com/manjaro/manjaro-tools/commit/e5d89f7ee3ddb7c0cc576d79b9d0714cf63013f5 and tell us if it fixes your issue as I think it should ...


```json
{
  "version": "0.6.11",
  "approved": true,
  "approver": "JayGray",
  "approvalDate": 1422209022210
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


Other looks good


In this line, you specify a plan_id. This is generally a bad practice, as depending on how your database is created or maintained, the id could change. Instead, you should have some class method on the Plan model that returns the plan object based on its name or something that allows you to make sure you are getting the right one, but not being restricted to a certain id.


Excellon uses "zeros" to indicate the presence of zeros, where RS274 uses "zero suppression" to indicate the zeros that are omitted. I think having the internal representation match the spec, i.e. using zeros in excellon files and zero suppression in gerber files is best. I used zero_suppression in the excellon code originally so that FileSettings would be useable for both filetypes.

I've started a patch which adds a "zeros" attribute to FileSettings, that always takes the opposite value of "zero_suppression" and vice versa. I would propose using this as a solution, so that the file settings can be specified using the appropriate parameter, zeros or zero_suppression based on the file type, and have the library Do The Right Thing (TM) in all cases. I'll update all the references to "zero_suppression" in the excellon code to use "zeros" per the spec. Let me know if you agree with this approach.


prvi


Same issue here as in charges_controller - you shouldn't specify a plan_id here.


Hi Andrey.
I'm interest why do not use `const` for this value`? We read and speak often about globals are bad and on coding, it depends on php 5.3 namespaces is a const possible.
Thanks for feedback!
Best regards Frank


For the `+1` in [Line 93](https://github.com/Runalyze/Runalyze/commit/be3deca49433f846d582d4e3930fdc0da6b8091f#diff-ca5d15c5b6ee35b3d288f10dc5c267feR93) I'm not sure where it comes from. If I remember right, it already changed a few times from 0 to 1 to 2 and backwards. It'd be good to know if it matches for you as well.


No, didn't merge Chani's changes yet. Pushed my changes.


TeamCity Ylab :: zmactep/ylab [Build 186](http://192.168.30.225:8111/viewLog.html?buildId=653&buildTypeId=Ylab_ZmactepYlab&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 132 Build time: 00:11:49


Since none of us are quite sure what this means / what the right terminology is, I'd suggest the right answer is not to use "disequality" or "inequality"...


```json
{
  "version": "0.6.11",
  "approved": true,
  "approver": "JayGray",
  "approvalDate": 1422209273201
}
```
approved with [gh-review](http://gh-review.herokuapp.com/)


Good changes, I've merged these into the main repository.


Thanks for all your feedbacks,
here you go:
https://github.com/XXMrHyde/android_frameworks_base/commit/8706b1f8aeacad346bdb6cc2dfd17f853658cc45


In four of the methods in this controller, you do this:

```
@wiki = Wiki.find(params[:id])
```

Instead, you could have put a before_action and a method that populated it, so you don't have to call it from each of those methods. Not saying this is wrong, it's just a DRYer alternative.


As discussed there is not real merrit in a special case handling for '....'; just replace the '...' by an ellipsis as done in line 421..


You use pretty much the same form in wikis/new and edit. Why not make a _form partial, put the code in there, then just render the form from each of those pages?


I personally still don't understand the issue.

Anyway, as long as changes work its all fine.

However, blacklistling wasn't triggered unless the package you want to build was in blacklist_trigger array.
So it did not matter if the blacklist_trigger array was empty or not. It was only triggered if it matches the package to build.


The external ID is still used for associating phenotips records with other
databases, e.g. FORGE. It's important for the users to be able to store the
label that they associate with each patient. I think we should probably
think of it less as an "external ID", which implies uniqueness, versus
something like "label" for an identifier that the user can use to remember
which patient is which.

On Sun, Jan 25, 2015 at 12:44 PM, Anton <notifications@github.com> wrote:

> I tried to add the ability to specify the phenotips id inside the
> pedigree, but this is as far as I got.
>
> One thought:
> If free text entry is allowed in the external id field, then there is no
> way to tell for sure if the id is a phenotips id or just an id that matches
> a phenotips id. What is the external id currently used for?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/phenotips/phenotips/commit/abc15b85c59fb2dda0dfad41ba08e7c66138716d#commitcomment-9432552>
> .
>


Add comment: # TODO: Consolidate with `` IntegerRange,__init__()``.


@manjaro to give my honest opinion manjaro-tools never worked well for me...

Half the time it fails to make packages, and I have yet to be able to make an ISO with it that works correctly (tried 5 or 6 times at various points in its history).

For testing https://github.com/manjaro/manjaro-tools/commit/e5d89f7ee3ddb7c0cc576d79b9d0714cf63013f5 you might have to ask @udeved as blacklisting has not worked correctly most of the time for me..

After applying your commit or with @udeved old code, does not seem to work here.
~~~~
$ buildpkg -p packages-openrc/upower-pm-utils/ -b unstable
==> Updating chroot for [unstable] (x86_64)...
:: Synchronizing package databases...
 core                     134.6 KiB   181K/s 00:01 [######################] 100%
 extra                   1917.6 KiB   251K/s 00:08 [######################] 100%
 community                  2.7 MiB   101K/s 00:28 [######################] 100%
:: Starting full system upgrade...
 there is nothing to do
==> Making package: upower-pm-utils 0.9.23-4 (Sun Jan 25 23:18:33 IST 2015)
==> Retrieving sources...
  -> Found upower-0.9.23.tar.xz
  -> Found upower-pm-utils-0.9.23-clamp_percentage_for_overfull_batt.patch
  -> Found upower-pm-utils-0.9.23-create-dir-runtime.patch
  -> Found upower-pm-utils-0.9.23-fix-segfault.patch
  -> Found upower-pm-utils-0.9.23-always_use_pm-utils_backend.patch
==> Validating source files with sha256sums...
    upower-0.9.23.tar.xz ... Passed
    upower-pm-utils-0.9.23-clamp_percentage_for_overfull_batt.patch ... Passed
    upower-pm-utils-0.9.23-create-dir-runtime.patch ... Passed
    upower-pm-utils-0.9.23-fix-segfault.patch ... Passed
    upower-pm-utils-0.9.23-always_use_pm-utils_backend.patch ... Passed
==> Making package: upower-pm-utils 0.9.23-4 (Sun Jan 25 17:48:33 UTC 2015)
==> Checking runtime dependencies...
==> Installing missing dependencies...
resolving dependencies...
looking for conflicting packages...
:: eudev-systemdcompat and libsystemd are in conflict. Remove libsystemd? [y/N]
error: unresolvable package conflicts detected
error: failed to prepare transaction (conflicting dependencies)
:: eudev-systemdcompat and libsystemd are in conflict
==> ERROR: 'pacman' failed to install missing dependencies.
==> ERROR: Build failed, check /mnt/datalinux2/manjaro-tools/buildpkg/unstable/x86_64/aaditya/build
==> ERROR: Aborting...
~~~~~

~~~~
$ buildpkg -p packages-openrc/upower-pm-utils/ -b unstable -c
[sudo] password for aaditya:
  -> Deleting chroot copy 'aaditya'...
  -> Deleting chroot copy 'root'...
==> Creating chroot for [unstable] (x86_64)...
==> Creating install root at /mnt/datalinux2/manjaro-tools/buildpkg/unstable/x86_64/root
==> Installing packages to /mnt/datalinux2/manjaro-tools/buildpkg/unstable/x86_64/root
:: Synchronizing package databases...
 core                     133.7 KiB   117K/s 00:01 [######################] 100%
 extra                   1909.0 KiB   178K/s 00:11 [######################] 100%
 community                  2.7 MiB   256K/s 00:11 [######################] 100%
:: There are 25 members in group base-devel:
:: Repository core
   1) autoconf  2) automake  3) binutils  4) bison  5) fakeroot  6) file
   7) findutils  8) flex  9) gawk  10) gcc  11) gettext  12) grep  13) groff
   14) gzip  15) libtool  16) m4  17) make  18) pacman  19) patch
   20) pkg-config  21) sed  22) sudo  23) texinfo  24) util-linux  25) which

Enter a selection (default=all):
resolving dependencies...
looking for conflicting packages...
warning: dependency cycle detected:
warning: ncurses will be installed before its bash dependency

Packages (98) acl-2.2.52-2  archlinux-keyring-20141218-1  attr-2.4.47-1
              bash-4.3.033-1  bzip2-1.0.6-5  ca-certificates-20140923-9
              ca-certificates-cacert-20140824-2
              ca-certificates-mozilla-3.17.3-2
              ca-certificates-utils-20140923-9  coreutils-8.23-1
              cracklib-2.9.1-1  curl-7.40.0-1  db-5.3.28-2  diffutils-3.3-2
              e2fsprogs-1.42.12-1  expat-2.1.0-4  filesystem-2014.07-1
              gc-7.4.2-2  gcc-libs-4.9.2-2  gdbm-1.11-1  glib2-2.42.1-1
              glibc-2.20-6  gmp-6.0.0-1  gnupg-2.1.1-1  gnutls-3.3.12-1
              gpgme-1.5.3-1  guile-2.0.11-2  iana-etc-2.30-4  keyutils-1.5.9-1
              krb5-1.13-1  less-471-1  libarchive-3.1.2-8  libassuan-2.1.3-1
              libatomic_ops-7.4.2-1  libcap-2.24-2  libffi-3.2.1-1
              libgcrypt-1.6.2-1  libgpg-error-1.17-1  libidn-1.29-1
              libksba-1.3.2-1  libldap-2.4.40-2  libmpc-1.0.2-2
              libsasl-2.1.26-7  libssh2-1.4.3-2  libsystemd-218-1
              libtasn1-4.2-1  libtirpc-0.2.5-1  libunistring-0.9.4-1
              libutil-linux-2.25.2-1  linux-api-headers-3.18.1-1  lz4-127-1
              lzo-2.08-3  manjaro-keyring-20141220-1  mpfr-3.1.2.p11-1
              ncurses-5.9-7  nettle-2.7.1-1  npth-1.1-1  openssl-1.0.1.l-1
              p11-kit-0.22.1-3  pacman-mirrorlist-20150105-1  pam-1.1.8-5
              pambase-20130928-1  pcre-8.36-2  perl-5.20.1-1  pinentry-0.9.0-1
              pth-2.0.7-5  python-3.4.2-2  readline-6.3.006-1  shadow-4.2.1-2
              tar-1.28-1  tzdata-2014j-1  xz-5.2.0-1  zlib-1.2.8-3
              autoconf-2.69-2  automake-1.14.1-1  binutils-2.25-1
              bison-3.0.2-1  fakeroot-1.20.2-1  file-5.22-1  findutils-4.4.2-6
              flex-2.5.39-1  gawk-4.1.1-1  gcc-4.9.2-2  gettext-0.19.3-1
              grep-2.21-1  groff-1.22.3-2  gzip-1.6-1  libtool-2.4.4-2
              m4-1.4.17-1  make-4.1-1  pacman-4.2.0-6.4  patch-2.7.1-3
              pkg-config-0.28-2  sed-4.2.2-3  sudo-1.8.11.p2-1  texinfo-5.2-3
              util-linux-2.25.2-1  which-2.20-7

Total Installed Size:  480.04 MiB

:: Proceed with installation? [Y/n]
(98/98) checking keys in keyring                   [######################] 100%
(98/98) checking package integrity                 [######################] 100%
(98/98) loading package files                      [######################] 100%
(98/98) checking for file conflicts                [######################] 100%
( 1/98) installing linux-api-headers               [######################] 100%
( 2/98) installing tzdata                          [######################] 100%
( 3/98) installing iana-etc                        [######################] 100%
( 4/98) installing filesystem                      [######################] 100%
( 5/98) installing glibc                           [######################] 100%
( 6/98) installing gcc-libs                        [######################] 100%
( 7/98) installing ncurses                         [######################] 100%
( 8/98) installing readline                        [######################] 100%
( 9/98) installing bash                            [######################] 100%
Optional dependencies for bash
    bash-completion: for tab completion
(10/98) installing gmp                             [######################] 100%
(11/98) installing mpfr                            [######################] 100%
(12/98) installing gawk                            [######################] 100%
(13/98) installing m4                              [######################] 100%
(14/98) installing diffutils                       [######################] 100%
(15/98) installing autoconf                        [######################] 100%
(16/98) installing gdbm                            [######################] 100%
(17/98) installing db                              [######################] 100%
(18/98) installing perl                            [######################] 100%
(19/98) installing automake                        [######################] 100%
(20/98) installing zlib                            [######################] 100%
(21/98) installing binutils                        [######################] 100%
(22/98) installing bison                           [######################] 100%
(23/98) installing attr                            [######################] 100%
(24/98) installing acl                             [######################] 100%
(25/98) installing sed                             [######################] 100%
(26/98) installing cracklib                        [######################] 100%
(27/98) installing libutil-linux                   [######################] 100%
(28/98) installing e2fsprogs                       [######################] 100%
(29/98) installing openssl                         [######################] 100%
Optional dependencies for openssl
    ca-certificates [pending]
(30/98) installing libsasl                         [######################] 100%
(31/98) installing libldap                         [######################] 100%
(32/98) installing keyutils                        [######################] 100%
(33/98) installing krb5                            [######################] 100%
(34/98) installing libtirpc                        [######################] 100%
(35/98) installing pambase                         [######################] 100%
(36/98) installing pam                             [######################] 100%
(37/98) installing shadow                          [######################] 100%
(38/98) installing libcap                          [######################] 100%
(39/98) installing coreutils                       [######################] 100%
(40/98) installing bzip2                           [######################] 100%
(41/98) installing pcre                            [######################] 100%
(42/98) installing libffi                          [######################] 100%
(43/98) installing glib2                           [######################] 100%
Optional dependencies for glib2
    python2: for gdbus-codegen and gtester-report
    elfutils: gresource inspection tool
(44/98) installing libgpg-error                    [######################] 100%
(45/98) installing libgcrypt                       [######################] 100%
(46/98) installing lz4                             [######################] 100%
(47/98) installing xz                              [######################] 100%
(48/98) installing libsystemd                      [######################] 100%
(49/98) installing util-linux                      [######################] 100%
Optional dependencies for util-linux
    python: python bindings to libmount [pending]
(50/98) installing fakeroot                        [######################] 100%
(51/98) installing file                            [######################] 100%
(52/98) installing findutils                       [######################] 100%
(53/98) installing flex                            [######################] 100%
(54/98) installing libmpc                          [######################] 100%
(55/98) installing gcc                             [######################] 100%
(56/98) installing libunistring                    [######################] 100%
(57/98) installing gettext                         [######################] 100%
Optional dependencies for gettext
    git: for autopoint infrastructure updates
(58/98) installing grep                            [######################] 100%
(59/98) installing groff                           [######################] 100%
Optional dependencies for groff
    netpbm: for use together with man -H command interaction in browsers
    psutils: for use together with man -H command interaction in browsers
    libxaw: for gxditview
(60/98) installing less                            [######################] 100%
(61/98) installing gzip                            [######################] 100%
(62/98) installing tar                             [######################] 100%
(63/98) installing libtool                         [######################] 100%
(64/98) installing texinfo                         [######################] 100%
(65/98) installing libatomic_ops                   [######################] 100%
(66/98) installing gc                              [######################] 100%
(67/98) installing guile                           [######################] 100%
(68/98) installing make                            [######################] 100%
(69/98) installing expat                           [######################] 100%
(70/98) installing lzo                             [######################] 100%
(71/98) installing libarchive                      [######################] 100%
(72/98) installing libtasn1                        [######################] 100%
(73/98) installing p11-kit                         [######################] 100%
(74/98) installing ca-certificates-utils           [######################] 100%
(75/98) installing ca-certificates-mozilla         [######################] 100%
(76/98) installing ca-certificates-cacert          [######################] 100%
(77/98) installing ca-certificates                 [######################] 100%
(78/98) installing libssh2                         [######################] 100%
(79/98) installing libidn                          [######################] 100%
(80/98) installing curl                            [######################] 100%
(81/98) installing pth                             [######################] 100%
(82/98) installing npth                            [######################] 100%
(83/98) installing libksba                         [######################] 100%
(84/98) installing libassuan                       [######################] 100%
(85/98) installing pinentry                        [######################] 100%
Optional dependencies for pinentry
    gtk2: for gtk2 backend
    qt4: for qt4 backend
(86/98) installing nettle                          [######################] 100%
(87/98) installing gnutls                          [######################] 100%
(88/98) installing gnupg                           [######################] 100%
Optional dependencies for gnupg
    libldap: gpg2keys_ldap [installed]
    libusb-compat: scdaemon
(89/98) installing gpgme                           [######################] 100%
(90/98) installing python                          [######################] 100%
Optional dependencies for python
    python-setuptools
    python-pip
    sqlite
    mpdecimal: for decimal
    xz: for lzma [installed]
    tk: for tkinter
(91/98) installing pacman-mirrorlist               [######################] 100%
:: Querying servers, this may take some time...
Africa
-> Error!http://manjaro.mirror.ac.za/stable/$repo/$arch
-> Error!http://mirror.is.co.za/mirrors/manjaro.org/stable/$repo/$arch
Australia
-> Error!http://mirror.ventraip.net.au/Manjaro/stable/$repo/$arch
-> Error!http://manjaro.mirror.serversaustralia.com.au/stable/$repo/$arch
-> Error!http://manjaro.mirror.uber.com.au/stable/$repo/$arch
Bangladesh
-> Error!http://mirrors.ispros.com.bd/manjaro/stable/$repo/$arch
Belgium
-> Error!http://ftp.belnet.be/manjaro/stable/$repo/$arch
-> Error!http://manjaro.cu.be/stable/$repo/$arch
Brasil
-> Error!http://manjaro.fis.unb.br/stable/$repo/$arch
-> Error!http://www.las.ic.unicamp.br/pub/manjarolinux/stable/$repo/$arch
Chile
-> Error!http://manjaro.dcc.uchile.cl/stable/$repo/$arch
China
-> Error!http://mirrors.ustc.edu.cn/manjaro/stable/$repo/$arch
-> Error!http://mirror.bjtu.edu.cn/manjaro/stable/$repo/$arch
-> Error!http://ftp.cuhk.edu.hk/pub/Linux/manjaro/stable/$repo/$arch
Colombia
-> Error!http://mirror.edatel.net.co/manjaro/stable/$repo/$arch
Costa_Rica
-> Error!http://mirrors.ucr.ac.cr/manjaro/stable/$repo/$arch
Denmark
-> Error!http://mirrors.dotsrc.org/manjaro/stable/$repo/$arch
France
-> Error!http://manjarolinux.polymorf.fr/stable/$repo/$arch
-> Error!http://mirror.finn.lu/manjaro/stable/$repo/$arch
Germany
-> Error!http://ftp.halifax.rwth-aachen.de/manjaro/stable/$repo/$arch
-> Error!http://ftp.tu-chemnitz.de/pub/linux/manjaro/stable/$repo/$arch
-> Error!http://ftp.rz.tu-bs.de/pub/mirror/manjaro.org/repos/stable/$repo/$arch
-> Error!http://mirror3.layerjet.com/manjaro/stable/$repo/$arch
-> Error!http://mirror.netzspielplatz.de/manjaro/packages/stable/$repo/$arch
-> Error!http://mirror.intercolo.net/manjaro/stable/$repo/$arch
-> Error!http://mirror.netcologne.de/manjaro/stable/$repo/$arch
Greece
-> Error!http://ftp.cc.uoc.gr/mirrors/linux/manjaro/stable/$repo/$arch
India
-> Error!http://manjaro.mirror.net.in/manjaro/stable/$repo/$arch
Indonesia
-> Error!http://kambing.ui.ac.id/manjaro/stable/$repo/$arch
-> Error!http://kartolo.sby.datautama.net.id/manjaro/stable/$repo/$arch
-> Error!http://jaran.undip.ac.id/index.php/manjaro/stable/$repo/$arch
Ireland
-> Error!http://ftp.heanet.ie/mirrors/manjaro/stable/$repo/$arch
Italy
-> Error!http://manjaro.mirror.garr.it/mirrors/manjaro/stable/$repo/$arch
Japan
-> Error!http://ftp.tsukuba.wide.ad.jp/Linux/manjaro/stable/$repo/$arch
-> Error!http://ftp.riken.jp/Linux/manjaro/stable/$repo/$arch
Netherlands
-> Error!http://ftp.nluug.nl/pub/os/Linux/distr/manjaro/stable/$repo/$arch
Philippines
-> Error!http://mirror.pregi.net/pub/Linux/manjaro/stable/$repo/$arch
Portugal
-> Error!http://ftp.dei.uc.pt/pub/linux/manjaro/repos/stable/$repo/$arch
Romania
-> Error!http://mirrors.xservers.ro/manjaro/stable/$repo/$arch
-> Error!http://mirrors.serverhost.ro/manjaro/packages/stable/$repo/$arch
-> Error!http://ftp.lug.ro/manjaro/stable/$repo/$arch
-> Error!http://manjaro.mirrors.linux.ro/repos/stable/$repo/$arch
Russia
-> Error!http://mirror.yandex.ru/mirrors/manjaro/stable/$repo/$arch
South_Korea
-> Error!http://mirror.premi.st/manjaro/stable/$repo/$arch
Sweden
-> Error!http://ftp.lysator.liu.se/pub/manjaro/stable/$repo/$arch
-> Error!http://mirror.zetup.net/manjaro/stable/$repo/$arch
Taiwan
-> Error!http://free.nchc.org.tw/manjaro/stable/$repo/$arch
-> Error!http://free2.nchc.org.tw/manjaro/stable/$repo/$arch
-> Error!http://ftp.yzu.edu.tw/Linux/manjaro/stable/$repo/$arch
Turkey
-> Error!http://ftp.linux.org.tr/manjaro/stable/$repo/$arch
United_Kingdom
-> Error!http://repo.manjaro.org.uk/stable/$repo/$arch
-> ..... http://www.mirrorservice.org/sites/repo.manjaro.org/repos/stable/$repo/-> Error!
-> Error!http://mirror.catn.com/pub/manjaro/stable/$repo/$arch
-> Error!http://manjaro.mirrors.uk2.net/stable/$repo/$arch
United_States
-> Error!http://mirror.dacentec.com/manjaro/stable/$repo/$arch
-> Error!http://www.gtlib.gatech.edu/pub/manjaro/stable/$repo/$arch
-> Error!http://mirror.jmu.edu/manjaro/stable/$repo/$arch
-> Error!http://mirror.solarvps.com/manjaro/stable/$repo/$arch
-> Error!http://mirror.nexcess.net/manjaro/stable/$repo/$arch
-> Error!http://vm1.sorch.info/manjaro/stable/$repo/$arch
:: Generated and saved '/etc/pacman.d/mirrorlist' mirrorlist.

hint: use `pacman-mirrors` to generate and update your pacman mirrorlist.
Optional dependencies for pacman-mirrorlist
    gtk3: for GTK UI
    python-gobject: for GTK UI
(92/98) installing archlinux-keyring               [######################] 100%
(93/98) installing manjaro-keyring                 [######################] 100%
(94/98) installing pacman                          [######################] 100%
==> To import the data required by pacman for package verification run:
==> `pacman-key --init; pacman-key --populate archlinux manjaro`
==> See: https://www.archlinux.org/news/having-pacman-verify-packages
Optional dependencies for pacman
    haveged: for pacman-init.service
(95/98) installing patch                           [######################] 100%
Optional dependencies for patch
    ed: for patch -e functionality
(96/98) installing pkg-config                      [######################] 100%
(97/98) installing sudo                            [######################] 100%
(98/98) installing which                           [######################] 100%
Generating locales...
  en_US.UTF-8... done
  de_DE.UTF-8... done
Generation complete.
==> Creating clean working copy [aaditya]...done
==> Making package: upower-pm-utils 0.9.23-4 (Sun Jan 25 23:39:10 IST 2015)
==> Retrieving sources...
  -> Found upower-0.9.23.tar.xz
  -> Found upower-pm-utils-0.9.23-clamp_percentage_for_overfull_batt.patch
  -> Found upower-pm-utils-0.9.23-create-dir-runtime.patch
  -> Found upower-pm-utils-0.9.23-fix-segfault.patch
  -> Found upower-pm-utils-0.9.23-always_use_pm-utils_backend.patch
==> Validating source files with sha256sums...
    upower-0.9.23.tar.xz ... Passed
    upower-pm-utils-0.9.23-clamp_percentage_for_overfull_batt.patch ... Passed
    upower-pm-utils-0.9.23-create-dir-runtime.patch ... Passed
    upower-pm-utils-0.9.23-fix-segfault.patch ... Passed
    upower-pm-utils-0.9.23-always_use_pm-utils_backend.patch ... Passed
==> Making package: upower-pm-utils 0.9.23-4 (Sun Jan 25 18:09:11 UTC 2015)
==> Checking runtime dependencies...
==> Installing missing dependencies...
resolving dependencies...
looking for conflicting packages...
:: eudev-systemdcompat and libsystemd are in conflict. Remove libsystemd? [y/N]
error: unresolvable package conflicts detected
error: failed to prepare transaction (conflicting dependencies)
:: eudev-systemdcompat and libsystemd are in conflict
==> ERROR: 'pacman' failed to install missing dependencies.
==> ERROR: Build failed, check /mnt/datalinux2/manjaro-tools/buildpkg/unstable/x86_64/aaditya/build
==> ERROR: Aborting...
~~~~

I am not interested in reporting issues and wasting my time if I feel I am just being a burden.


In this case I propose a separate "Phenotips id" field in the pedigree menu (only for single nodes, not group nodes)


it's not fully tested right now


Class `const` must be constant expression, it cannot hold an object (as far as I remember). Simply put there is _no_ “good” place in WordPress architecture for main class instance.

Some people prefer singletons, but I don't like the pattern and it complicated the class itself. Globals are used anyway (let's be honest, they are never going away in WP) and (unless it gets better mechanism) global is compromise to hold instance, without having the class deal with the problem in its implementation.


In virtually all of these collaborators controller methods, you are looking up the wiki. This is another great candidate for a before_action.


Typically, collaborator_params (and methods like it) would be private methods, because only this controller should be able to call that method.


Why easier? It shouldn't be. I'm sure that it's better to test public interface. I mean that we have to:

* test that `generate` method generates feed in valid xml
* test that `generate` method generates feed with received posts
* test that `generate` method generated feed and use posts in reverse order (just last N posts)

See? We don't need to test how `_extract_posts` works. We just want to be sure that `generate` method produces correct output. How it was achieved is unrelated question here


I would probably make this a helper or a method on one of the models rather than putting the query in the view.


let's sort generators it by alphabet. :)


TeamCity Ybackend :: Feodorov/ybackend [Build 79](http://192.168.30.225:8111/viewLog.html?buildId=654&buildTypeId=Ybackend_FeodorovYbackend&guest=1) is now running


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 150](http://192.168.30.225:8111/viewLog.html?buildId=655&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 150](http://192.168.30.225:8111/viewLog.html?buildId=655&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 127 Build time: 00:00:01


If you fill the queue before starting the threads then the wait is not needed anymore. And even if that is very unlikely to happen, that is still a race.


TeamCity Ybackend :: Feodorov/ybackend [Build 79](http://192.168.30.225:8111/viewLog.html?buildId=654&buildTypeId=Ybackend_FeodorovYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:16


TeamCity Ybackend :: zmactep/ybackend [Build 51](http://192.168.30.225:8111/viewLog.html?buildId=656&buildTypeId=Ybackend_ZmactepYbackend&guest=1) is now running


You have right, in WP we have always globals, I think also. It was only interest for me, because I like you source and will always learn.


TeamCity Ybackend :: zmactep/ybackend [Build 51](http://192.168.30.225:8111/viewLog.html?buildId=656&buildTypeId=Ybackend_ZmactepYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:20


@Aaditya, you are not a burden, you are a valuable member of team.
But, as said, you should not change stuff which breaks functionality while stating you don't understand what the code does. Just think how this comes across if one look at the changes is enough to say it will break something. It implies someone has to fix it.

I am known for clear and straight forward talk, not everybody likes that or may misunderstand it.
I am always pretty straight forward, not meant to hurt, its just how I tick.


Hi,
With the previous discussion on mailing list, is this gonna get effected?


what do you think about adding underscore to all private things?

btw, maybe it's better to call `_template_name`? we know that it's about index and name could say us that it's a template name, not a template itself.


NB: such declarations go to global space.
Please, consider this module https://github.com/dvv/nodemcu-firmware/commit/b8522cbb788464ed0b9b320ab838a002861508b5 instead -- it supports both DHT11/22.


`_save_as` ?


Genial. Si me llego el pull request. Lo miro a la noche. Abrazo. JP

Ya me estoy poniendo a hacer las correcciones. Cuando termine de hacer las
high te aviso. Tengo algunas dudas de las correcciones que también te las
hago después.

Gracias por las palabras de motivación!!! A seguir metiéndole!!

—
Reply to this email directly or view it on GitHub
<https://github.com/iberras/java-bootcamp/commit/bc0a5abbecb3f18a36d38716bdf04818aac59fdf#commitcomment-9427802>
.

--

The information contained in this e-mail may be confidential. It has been
sent for the sole use of the intended recipient(s). If the reader of this
message is not an intended recipient, you are hereby notified that any
unauthorized review, use, disclosure, dissemination, distribution or
copying of this communication, or any of its contents,
is strictly prohibited. If you have received it by mistake please let us
know by e-mail immediately and delete it from your system. Many thanks.

La información contenida en este mensaje puede ser confidencial. Ha sido
enviada para el uso exclusivo del destinatario(s) previsto. Si el lector de
este mensaje no fuera el destinatario previsto, por el presente queda Ud.
notificado que cualquier lectura, uso, publicación, diseminación,
distribución o copiado de esta comunicación o su contenido está
estrictamente prohibido. En caso de que Ud. hubiera recibido este mensaje
por error le agradeceremos notificarnos por e-mail inmediatamente y
eliminarlo de su sistema. Muchas gracias.


But you have right, `const` can't array, object -- http://php.net/manual/en/language.constants.syntax.php
The alternative is a `public static $var`.


PATHS_FROM_ECLIPSE_TO_PYTHON should be a list(tuple(str, str)), so, this change isn't really correct... I think the issue lies in how you've filled PATHS_FROM_ECLIPSE_TO_PYTHON in your installation (which should usually be an empty list unless you're doing some remote debugging on a different machine).


It's not actually *default*. You know when I see *default* I start thinking that it could be overridden somehow by user. In our case it's could not be overridden since it makes no sense. So I propose to rewrite it. Perhaps something like this: *a template name to be used for rendering index page*


Same as above. It's not *default*. :)


Use a more descriptive name, e.g. `decimal_value`


Same as for previous generator.