**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/parsing/ConfParser.py`, lines 129 to 130.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/parsing/ConfParser.py
+++ b/coalib/parsing/ConfParser.py
@@ -126,8 +126,7 @@
                                     value,
                                     origin,
                                     # Ignore PEP8Bear, it fails to format that
-                                    remove_empty_iter_elements=
-                                    self.__remove_empty_iter_elements),
+                                    remove_empty_iter_elements=self.__remove_empty_iter_elements),
                             allow_appending=(keys == []))

     def __init_sections(self):
```


The text `"Choose a Planet"` must be placed on strings.xml.


fragment?


The creature should enter the battlefield tapped for this effect.


This line is throwing an exception, because `.len()` is unrecognized. You probably meant `len(CACHED_SELECTION)`?


Please also try to format your XML layouts in such a way `android:id` must be the first attribute in each view. Just hit `Ctrl+Alt+L` to let Android Studio do it for you.


Will that string contains syntax work? I'd expect to see something like `request.host.include?("bostonhash.com")`


Hi Feng
Please see this bug use kernel 4.3.3 and this patch :

[47327.351771] BUG: unable to handle kernel NULL pointer dereference at 0000000000000046
[47327.352682] IP: [<ffffffffa0052b31>] get_unique_tuple+0x3e1/0x4e0 [nf_nat]
[47327.353465] PGD 265cb2067 PUD 26643e067 PMD 0
[47327.354013] Oops: 0000 [#1] SMP
[47327.354407] Modules linked in: udp_diag unix_diag af_packet_diag sch_hfsc xt_recent iptable_filter xt_IMQ xt_mark iptable_mangle xt_nat xt_addrtype iptable_nat nf_conntrack_ipv4 nf_defrag_ipv4 nf_nat_ipv4 ip_tables xt_IAND(O) xt_IADN(O) xt_iar80(O) xt_IA(O) ebt_iabo(O) ebt_iabi(O) xt_IADB(O) IA(O) netconsole w83627ehf hwmon_vid imq bonding pcnet32 de4x5 tulip dmfe ne2k_pci 8390 b44 ssb natsemi 3c59x fealnx via_rhine sis900 e100 8139cp 8139too dl2k acenic tg3 libphy r8169 mii sky2 e1000e e1000 igb i2c_algo_bit ptp pps_core mpt2sas scsi_transport_sas raid_class nf_nat_pptp nf_conntrack_pptp nf_conntrack_proto_gre nf_nat_proto_gre nf_nat nf_conntrack mcryptd sha1_ssse3 sha1_generic arc4 ecb ppp_mppe ppp_generic slhc
[47327.362850] CPU: 4 PID: 0 Comm: BFS/4 Tainted: G           O    4.3.3 #1
[47327.363592] Hardware name: Supermicro X9SRE/X9SRE-3F/X9SRi/X9SRi-3F/X9SRE/X9SRE-3F/X9SRi/X9SRi-3F, BIOS 1.0a 03/06/2012
[47327.364786] task: ffff880276198ac0 ti: ffff8802761a0000 task.ti: ffff8802761a0000
[47327.365613] RIP: 0010:[<ffffffffa0052b31>]  [<ffffffffa0052b31>] get_unique_tuple+0x3e1/0x4e0 [nf_nat]
[47327.405707] RSP: 0018:ffff88027fc83510  EFLAGS: 00010286
[47327.426149] RAX: 0000000000000006 RBX: ffff88027fc835c0 RCX: 0000000002235745
[47327.446715] RDX: ffffc90002b4a728 RSI: 0000000000000000 RDI: 00000000274ddd74
[47327.466964] RBP: ffff88027fc83588 R08: ffff880059f891a0 R09: ffff8801e9c40d00
[47327.487030] R10: 0000000000000003 R11: 0000000000000000 R12: 0000000000000000
[47327.507250] R13: ffff8802238e9928 R14: ffff88027fc83598 R15: ffff88027fc836b8
[47327.527327] FS:  0000000000000000(0000) GS:ffff88027fc80000(0000) knlGS:0000000000000000
[47327.566871] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[47327.586726] CR2: 0000000000000046 CR3: 00000002665b7000 CR4: 00000000000406e0
[47327.606571] Stack:
[47327.625880]  ffffffffa0053cd0 ffffffffa0035b68 ffffffffa0035b60 0000000066199600
[47327.665675]  ffffffffa0053cc0 ffffc9000492a630 ffff880059e7e800 ffffffffa00bd4a7
[47327.705489]  6230000000000246 ffffffffa0039adf ffff8802238e9928 ffff88027fc836b8
[47327.745168] Call Trace:
[47327.764333]  <IRQ>
[47327.764574]  [<ffffffffa00bd4a7>] ? igb_xmit_frame_ring+0x327/0xb90 [igb]
[47327.801887]  [<ffffffffa0039adf>] ? nf_ct_invert_tuple+0x3f/0x70 [nf_conntrack]
[47327.838674]  [<ffffffffa0052c62>] ? nf_nat_setup_info.part.10+0x32/0x290 [nf_nat]
[47327.876034]  [<ffffffffa0052ef6>] ? nf_nat_setup_info+0x36/0x70 [nf_nat]
[47327.894989]  [<ffffffffa001b11b>] ? 0xffffffffa001b11b
[47327.913465]  [<ffffffffa002e468>] ? ipt_do_table+0x2f8/0x590 [ip_tables]
[47327.931845]  [<ffffffff8100452a>] ? do_IRQ+0x4a/0xc0
[47327.949755]  [<ffffffffa00356b6>] ? nf_nat_ipv4_fn+0x126/0x1d0 [nf_nat_ipv4]
[47327.967626]  [<ffffffffa0050010>] ? iptable_nat_ipv4_fn+0x10/0x10 [iptable_nat]
[47328.002605]  [<ffffffffa003580b>] ? nf_nat_ipv4_out+0x3b/0xe0 [nf_nat_ipv4]
[47328.020804]  [<ffffffff8145ca6b>] ? nf_iterate+0x6b/0x80
[47328.038791]  [<ffffffff8145d8e4>] ? nf_reinject+0xb4/0x140
[47328.056567]  [<ffffffffa0283194>] ? imq_dev_xmit+0x84/0xc0 [imq]
[47328.074266]  [<ffffffff81432d83>] ? dev_hard_start_xmit+0x93/0x300
[47328.091744]  [<ffffffffa02834d1>] ? __imq_nf_queue+0x281/0x6b0 [imq]
[47328.108934]  [<ffffffffa0283b77>] ? imq_nf_queue+0x277/0x39e [imq]
[47328.125944]  [<ffffffff81469350>] ? ip_fragment.constprop.5+0x80/0x80
[47328.142855]  [<ffffffff8145d7bb>] ? nf_queue+0x12b/0x1a0
[47328.159410]  [<ffffffff8145cac4>] ? nf_hook_slow+0x44/0xc0
[47328.175609]  [<ffffffff81469826>] ? ip_output+0x96/0xb0
[47328.191349]  [<ffffffff81469350>] ? ip_fragment.constprop.5+0x80/0x80
[47328.206858]  [<ffffffff81465df7>] ? ip_forward+0x397/0x420
[47328.221925]  [<ffffffff81465a00>] ? ip_frag_mem+0x40/0x40
[47328.236482]  [<ffffffff814648d4>] ? ip_rcv+0x2d4/0x360
[47328.250541]  [<ffffffff81464150>] ? ip_local_deliver_finish+0x1a0/0x1a0
[47328.264920]  [<ffffffff8142f915>] ? __netif_receive_skb_core+0x765/0x9b0
[47328.279565]  [<ffffffff8145ca6b>] ? nf_iterate+0x6b/0x80
[47328.293901]  [<ffffffff81431a34>] ? netif_receive_skb_internal+0x24/0x80
[47328.308307]  [<ffffffff81502f2c>] ? br_handle_frame_finish+0x20c/0x5a0
[47328.322425]  [<ffffffffa0052f72>] ? __nf_nat_alloc_null_binding+0x42/0x50 [nf_nat]
[47328.349550]  [<ffffffff81431a90>] ? netif_receive_skb_internal+0x80/0x80
[47328.363448]  [<ffffffff8150ec7b>] ? br_nf_pre_routing_finish+0x25b/0x3d0
[47328.377325]  [<ffffffff8145ca07>] ? nf_iterate+0x7/0x80
[47328.391039]  [<ffffffff81502d20>] ? br_port_flags_change+0x10/0x10
[47328.404801]  [<ffffffff81510145>] ? br_nf_pre_routing+0x3f5/0x410
[47328.418448]  [<ffffffff8150ea20>] ? br_nf_local_in+0x40/0x40
[47328.431895]  [<ffffffff8145ca6b>] ? nf_iterate+0x6b/0x80
[47328.444899]  [<ffffffff8145cae9>] ? nf_hook_slow+0x69/0xc0
[47328.457451]  [<ffffffff815034b2>] ? br_handle_frame+0x182/0x280
[47328.469974]  [<ffffffff81502d20>] ? br_port_flags_change+0x10/0x10
[47328.482604]  [<ffffffff8142f509>] ? __netif_receive_skb_core+0x359/0x9b0
[47328.495357]  [<ffffffff81432c66>] ? process_backlog+0x86/0x110
[47328.508032]  [<ffffffff81432ada>] ? net_rx_action+0x1ba/0x2c0
[47328.520624]  [<ffffffff81042107>] ? __do_softirq+0xd7/0x1f0
[47328.533167]  [<ffffffff8104236e>] ? irq_exit+0x9e/0xa0
[47328.545548]  [<ffffffff815388fc>] ? call_function_single_interrupt+0x7c/0x90
[47328.558222]  <EOI>
[47328.558457]  [<ffffffff81402233>] ? cpuidle_enter_state+0x133/0x1e0
[47328.583930]  [<ffffffff81402228>] ? cpuidle_enter_state+0x128/0x1e0
[47328.596857]  [<ffffffff8105fa83>] ? cpu_startup_entry+0x1c3/0x220
[47328.609763]  [<ffffffff815376a5>] ? _raw_spin_unlock_irqrestore+0x5/0x10
[47328.622975] Code: e1 48 c1 ee 20 48 8d 14 f2 4c 8b 02 4d 85 c0 75 16 e9 90 fc ff ff 4d 8b 00 4d 85 c0 0f 84 84 fc ff ff 41 0f b6 46 26 49 8b 70 10 <3a> 46 46 75 e6 41 8b 06 39 46 20 75 de 41 8b 46 04 39 46 24 75
[47328.665658] RIP  [<ffffffffa0052b31>] get_unique_tuple+0x3e1/0x4e0 [nf_nat]
[47328.680015]  RSP <ffff88027fc83510>
[47328.694167] CR2: 0000000000000046
[47328.708091] ---[ end trace 4c6fd70eae2bd46d ]---
[47328.722072] Kernel panic - not syncing: Fatal exception in interrupt
[47328.736077] Kernel Offset: disabled
[47328.749528] Rebooting in 10 seconds..

m.


Please also improve the commit statement.


Update


აუ ეს რა არი აზზე არ ვარ :დ


This implementation is okay. However, there's a faster and convenient way of doing this. Instead of getting the item selected by the spinner, you could just compare on your if statements the `position` since we know at hand the position of each planet on your list above.

For example:

```java
if (position == 0) {
    imageView.setImageResource(R.drawable.mercury);
} else if (position == 1) {
    imageView.setImageResource(R.drawable.venus);
} else if (position == 2) {
    ....
}
```


// TODO: See if we can figure out a way to set this to 'AES-256-CBC' for new installs only


Instead of also instantiating new `Drawable` resources, you could just directly set the drawable as an image resource to your image view. See example above.


Please format your xml layout as well in such a way that all `android:id` will be the first attribute in each view. Just hit `Ctrl+Alt+L` to perform the formatting.


We are going to go back and forth on this forever wont we?


[#11] Isn't what redundant?  If you are referring to the extra export after the main export, its more future proofing.  It would allow us to use the class without being required to initialize it or do a weird initialize with null.

In other words, you would be able to do:
```
var WinkAirConditionerAccessory = require('./accessories/air_conditioners').WinkAirConditionerAccessory;
```

Instead of:
```
var WinkAirConditionerAccessory = require('./accessories/air_conditioners')();
```

So its a little more like other node modules.  Really with the first option, we don't need the if check, but I would rather err on the side of caution.  But no, this is not used right now, I just didn't want us to have to do this later when we need it.


Please also improve the commit statements. Be at least be more descriptive. At least state briefly what you have done on this stage of commit.


Nice!


Fixed in latest commit to that branch


`first commit` seems vague as a commit statement. Please be more descriptive on your commit statements. At least state briefly what was actually done on this stage of commit.


Good point, we will need to handle that case. Alternatively provide a rake task that only sets the environment.


No worries @Wilfred, this commit broke something, and I prefer direct communication. :relaxed:

Assuming the PR gets merged, I think the only compelling reason to use our own company backend is if we want to make it async. If we just add a backend for debugging purposes, we just have more code, with more setup for users, without any additional functionality.


no longer true right


Hi Stefano-
Just giving you my 2 cents you can take it or leave it.

You can try something different...
If (history>hThreshold && cmh> cmhThreshold)

Where you will tune hThreshold and cmhThreshold...

Just a suggestion..


```cpp
using NativeVertexFormatMap = std::unordered_map<PortableVertexDeclaration, std::unique_ptr<NativeVertexFormat>>;
```


@DomT4 I think this has the potential to cause some unnecessary confusion. When determining the quantum depth, why should `--with-quantum-depth-32` take precedence over `--with-perl`, but `--with-quantum-depth-8` is overruled by `--with-perl`? (And what is the default quantum depth anyway? Does that change with every `imagemagick` release?)


test


Looking at this for the first time, I would think `group.playAll()`, `stopAll()`, `pauseAll()`, and `clear()` should be included.


unnecessary newline


Note the spaces with binary operators


Since there's only one variable, you only need one


You reassigned here


This line works for associations but not for embeddables.

It infers the field (default) name from a (presumed) association but when used with embeddables this break the datagrid filtering as well as model autocomplete.


Now we made the function


y is now a theano variable that comes from x


And we run the function with real values


I would go with `{:error, :rollback}` because I don't think this distinction is interesting for Ecto developers. Of course, DBConnection may return :closed, we can translate it to rollback in Ecto itself.


Read the in-line comments please :)


Смотри, тут интересный момент. Это вообще говоря уже довольно сложный код, чтобы тестировать не только «результаты работы», но и «процесс» (для чего собственно и нужно модульное тестирование).

Что у нас тут происходит, в принципе:
* запрашивается страница со списком всех фильмов, разбирается на отдельные фильмы
* запрашивается страница каждого фильма
* её данные превращаются в хеш.

Соответственно, чтобы убедиться, что твой код хорошо работает, поделён на маленькие логические кусочки, разумно оттестирован, осмысленно поступить так:
* тест «парсер запросит страницу http://www.imdb.com/chart/top»
* тест «получив такую страницу, он сделает 250 запросов к урлам отдельных фильмов (с такой-то общей схемой урла)»
* тест «запросив страницу одного фильма, он корректно превратит фильм в хеш»
* дополнительные тесты «запросив страницу фильма, где есть не все нужные данные, он тоже не взорвётся»

При этом чтобы тесты не выполнялись бесконечно долго, в настоящий интернет мы их не пускаем — используем библиотеку WebMock и имитируем общение с инетом, подсовывая заранее заготовленные странички.


Looks good


Тут имеет смысл использовать `map` вместо того, чтобы сначала создавать массив, а потом в `each` класть в него элементы.


```C
#include <stdio.h>

int main( ) {

// print the string "hello World" using arrays
   char message[ ] = "Hello World";
   printf( "\n%s\n" , message );

   return 0;
}

```


I am still using flowCore_1.34.11, and I don't get any error. I was simply wondering what would happen if delimiter is prefixed with "\\" as in the fixed version. If you don't get any error, it's fine. Could you let me know how to use the version you have just fixed.


Oh, very bad line. Reformat it, please!


oops! Thank you! :)


`x ||= something` — идиома для присвоения переменной значения по умолчанию (если в ней нет значения; а если уже есть — останется предыдущее). Поскольку переменная movies у тебя раньше не использовалась, это избыточно, достаточно `movies = []`


questo è stato tolto


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


@DjalmaPessoa i believe that this version does what you describe at [1] ?

this change is useful for timing, because it no longer requires the (big process) svydesign creation of `dom.design` -- i think it does what you describe because `v6531` is already on the file.  i removed the 272.5 chart, but just because these processes take a long time..  we can edit more if you need  :)  thanks djalma

[1] https://github.com/ajdamico/asdfree/commit/206e271e429895ef7fa3d8f27883e618c179503a#commitcomment-15355100


Well I'm not sure if this is a right choice.  Forcing a new thread to prevent deadlocks sounds not right.


This probably should call a method inside `StringUtil`.


Такие длинные строки лучше переносить, чтобы лучше читалось:
```ruby
puts movies.
  sort_by{|movie| movie[:duration].split(' ').map(&:to_i)}.
  reverse[0..5].
  map {|movie| "Продолжительность: #{movie[:duration]} Название: #{movie[:title]}"}
```


Hi Rupal,
Thank you for your review. I did try 'e.preventDefault()' and it is working to fix our problem. Thank you ! I missed this earlier. Also thank you for noticing the bug regarding adding choices button, I am sorry that I missed it. I am working on it too.

I am really sorry that I missed these crucial bugs!


Если здесь сначала извлечь всех режиссёров, а потом делать uniq и sort, кода получится намного меньше.
И, кстати, режиссёров хорошо бы по фамилии отсортировать (считая таковой последнее слово ФИО).


just saw this line: Morph has a function `getSubmorph(selector)`. You should be able to use it like `this.getSubmorph(...)`. It is defined in `templates/classes/Morph.js`.


Тут можно было бы сделать так: после group_by у тебя массив пар «режиссёр — его фильмы», и руби достаточно умён чтобы это понять:
`group_by{|movie| movie[:director]}.map{|director, dirmovies| ...`


тут нет необходимости инициализировать массив заранее, достаточно использовать flatten.


Тут хорошо бы СНАЧАЛА посчитать всех актёров, а потом вывести результат. А то ты получается на каждом шаге подсчёта выводишь весь total


@julien-f add `ws` in to the browserify ignore and your done.


Insanely great!


Subproject icecream


It looks like Sphinx can grab member docstring's, see this:

http://stackoverflow.com/questions/6060813/how-to-document-fields-and-properties-in-python

And:

http://stackoverflow.com/questions/6060813/how-to-document-fields-and-properties-in-python

On Sat, Jan 9, 2016, 8:22 AM Wink Saville <wink@saville.com> wrote:

> Completion, like on the command line, works better when words start the
> same way. In baremetal-hi you had Craftfile and Craftr.link.ld.template,
> IIRC and it was a little inconvenient. And it just seems to me Craftrfile
> is better :)
>
> On Fri, Jan 8, 2016, 8:47 AM Niklas Rosenstein <notifications@github.com>
> wrote:
>
>> The Craftr ("crafter") is *craft*ing using *Craft*files. :)
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/craftr-build/craftr/commit/470dc825565f1016975fd72af8e5456694382f70#commitcomment-15346194>
>> .
>>
>


s_bbox_upload_heap_data_pointer is never initialized


това не е добра идея. В модала нямаш последно добавените неща, защото той се генерира само като го отвориш. Т.е. преди да го отвориш е празен а реално има неща.
Ясно е че в момента извикваш функцията само при отваряне, но впоследствие може някой друг да работи по кода ти и да реши да я извика в друг случай.

Правилното тук е да калкулираш цента от `cart` обекта


update commit for issues #333


That equals sign is a typo


I have some thoughts about this.

First, setting our "base" `a` tag style is a good thing.
The `font-weight: 600;` property is used in most static text pages in the `blog` repository to provide a stronger amount of contrast since the blue-green color itself is not very contrasty from the black text itself. This is not used everywhere, so it's good if we figure out what should be our standard and establish it.

That being said we now have to be very conscientious about when `a` is used and `font-weight: 600` is not appropriate, so that it must be explicitly reset to `font-weight: normal` (or; `initial` or `inherit` or `unset`). Let's also look into where `.link` is used. As a class name, it's not very descriptive about where it's appropriate to apply that class, and I haven't looked into it, but it's almost as if there are certain HTML tags that are not anchor tags per se but behave like one. This is not a great practice but might have been introduced as a side effect of other circumstances, so we need to document this.


some: `asyncrhonous` there


Sounds good to me.

- sent from my mobile device
On 8 Jan 2016 15:15, "Paul Reed" <notifications@github.com> wrote:

> OK, to get 1=INFO, 2=WARN, 3=ERROR, 4=CRITICAL
> Setting to 1 also includes 2, 3 and 4.
> Setting to 2 includes also 3 and 4
> Setting to 3 also includes 4
> would need the following changes to EmonLogger.php (and of course the
> settings files updated)
>
> public function info ($message){
>     if ($this->log_level = 1) $this->write("INFO",$message);
> }
>
> public function warn ($message){
>     if ($this->log_level <= 2) $this->write("WARN",$message);
> }
>
> public function error ($message){
>     if ($this->log_level <= 3) $this->write("ERROR",$message);
> }
>
> public function critical ($message){
>     if ($this->log_level <= 4) $this->write("CRITICAL",$message);
> }
>
> Just tested this and it works fine, however are we agreed that we don't
> need a log_level =NONE as it duplicates $log_enabled = false;
>
> I've not committed the above - I think we need a consensus first :)
>
> Paul
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/emoncms/emoncms/commit/b27063dd2953a10d60afe9f504a1c23b15754b6f#commitcomment-15340195>
> .
>


I rebased on current develop and
```
$ ./bar.sh --all
uname=Linux j 3.13.0-66-generic #108-Ubuntu SMP Wed Oct 7 15:20:27 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
gcc=4.8
Run test gcc -m64
Run test gcc -m64 -DMP_8BIT
error 1 while running tests
$ git bisect bad
093b5b33abb0e8337033bfc8401b17f786225b34 is the first bad commit
commit 093b5b33abb0e8337033bfc8401b17f786225b34
Author: Francois Perrad <francois.perrad@gadz.org>
Date:   Thu Nov 26 22:09:32 2015 +0100

    refactor cast
```
:-(
`bar.sh` is a copy of the latest version of the `testme.sh` script


https://github.com/ShareX/ShareX/blob/master/README.md still contains reference to irc client.


Ok, that's how I normally use `racket`. But here I followed the style from `renderer2d.scrbl`.


@UniqMartin Apologies. I can reword the block if it's confusing. The Perl module expects at least 16 `quantum_depth`, which is the default. It seems perfectly content building against 32 QD, but would encounter problems built/ran against 8 QD. The addition of this line here was just designed to ensure you picked a compatible QD.


either works for me! testing that each method is using .name() indirectly might be a little annoying, I try not to worry about stuff like that too much but coverage-wise it would definitely be good


yes, I have just re-checked and indeed, only 1.3% of our users have android 4.0.x


Makes sense. Done!


Since when are there these useless classes? Andesite, granite and diorite are stone.. just with a different meta!


asdasd


Why this change? I just see no point of this modification for your enhancement.


asdasdasd


Yea, it is buggy, glichy, but for the time being this should do it. Better than the wrong rotating once currently (which only happens because i don't calculate when the rails update)


Tested? I don't think this will actually work. The NoteBlockSound is not updated to support this


Also added title textfield resign first responder and pitch textview become first responder


nice


pygame müsste ne getRect Funktion dafür haben


Ok


Does this pass tests? Duktape will always pad arguments up to the total specified for the function. That means that the 'required' mechanism in argument parsing doesn't really work, and potentially missing arguments should be marked with ? so that undefined is acceptable.


Good to know. I guess this is again one of those instances where I would prefer that conflicting options are pointed out to the user instead of automatically doing what the user might or might not have intended. I think [`graphicsmagick`](https://github.com/Homebrew/homebrew/blob/1b718760bda998df401920ba29b013f27647c744/Library/Formula/graphicsmagick.rb#L50-L54) solves this quite nicely for the quantum depth options themselves. But I suspect I'm on lonely ground again …


I really would have hoped so but I guess there isn't. However, the only exception that `os.mkdir` is documented to raise is `OSError`. Should you be catching only `OSError` rather than having the bare `except`?


Also, **row** is enabled by default.


Seems sensible.


I was just committing whatever was in the `ELPA` directory. Please remove.


Nice work.

I am just curious on why you are accessing the diff by index instead of doing:

```go
for _, diff = range diffs {
    // ...
}
```


Is it enough to check for an equal shape on the dispatch guard? I think the previous layouts had the class of which they are layouts from, but that is not the case of the shapes, is it? If not (that is the case in truffleMATE) just checking for the shape is not enough i think...


Translator Fixed


The cases, which were not covered by the RearBlockedIso test
are backward pawns which have no contact with the side pawns.

It seems that backward are already penalized enough.

LLR: -1.97 (-2.94,2.94) [0.00,5.00]
Total: 9701 W: 1763 L: 1805 D: 6133


Probably whoever wrote taht didn't know about the full glory of the `@` reader.


merged by rebasing. thanx!


@NikoKrause
You've got my full attention. Interesting. Of course, I'm looking at this in the context of CinnXP as well.


This part of the code was implemented by @sebpiq quite some time ago, and after a while I was indeed wondering if this is handy. I myself only have experience with normal distributions where you provide a mean and variance, not by providing a min and max value. Therefore I've disabled the normal distribution functions in the library for now, until we've given this some more thought.


m_tex_width and m_text_height can be moved to the class body as well.


This line (50) introduce a bug. Please see the bug fix 803185a.


Don't merge this yet.

Creates items from integrations in the new integration repo. slow for now, but room to improve


```c

#include <stdio.h>
#define SIZE 100

int main(void) {
	/*This program is converting unsigned integers (up to 9 digits) to binary*/
	int binary_form[SIZE];
	int number, remainder;

	printf("\nInsert an integer:\t");
	scanf("%d", &number);

	while(number < 0) //Checking for negative inputs
	{
		printf("\n\nNegative input.\nInsert unsigned integer:\t");
		scanf("%d", &number);
	}

	int quotient = 2*number;
	size_t counter = 0;

	if(number == 0)
	{
		printf("\n\nByte 0 in binary is 00");
	}
	else
	{
	    printf("\n\nQUOTIENT\tREMAINDER\n\n");

		while(quotient != 0 && counter <= SIZE)
	    {
		    quotient = quotient/2;
		    remainder = quotient %2;

		    printf("%d\t\t%d\n\n", quotient, remainder);
		    binary_form[counter] = remainder;

		    if(quotient == 0)
		    {
			    break;
		    }

		    counter++;
	    }

        printf("\n\nByte %d in binary is ", number);
        /* printing array's elements in opposite order to form
           form The binary form of input */
        int pos = counter;
        for( ; counter>=0 && counter<= pos; --counter )
        {
	        printf("%d", binary_form[counter]);
        }
	}

	return 0;
}

```


Closes #1


I've just added a new Localizable.strings with "in code messages".


Yes, that was a silly mistake. I thought "all" had priority. Well, one learns something new some days :)  Fixed!


Formatting, Int values don't need to be in quotation marks, an error output method is imho not needed, the alias stuff didn't work for me and the shebang should be with /usr/bin/env.


I'm not fond of introducing new static returns


:+1: Nice!


could use `strchr()` and `strrchr()` instead


dangerous if called on a tag that don't actually have a `_` in the name


sudo_users = {146480433},--sudousers


Yep, I figured we never want undefined to be passed as an argument to any of our APIs, and adjusted the argument parsing like this:
https://github.com/frida/frida-gum/blob/8365b056518ea0796f8410ad613f1ad131ac218d/bindings/gumjs/gumdukvalue.c#L43


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


and length would expand to `0 - 1` (-1) in case there is only 1 `_`.


same deal here


assertion removed https://github.com/emberjs/ember.js/pull/12793, their will be a patch release to 1.13.1 today.


I'm not sure the ratio usefulness/(complexity*dangerousness) warrants such a change… do you think it's really useful?


**Warning!** This line introduce a bug at compile. Please see bug fix 803185a.


Thanks a lot , i will test it tomorrow.


Hi Kushank,
Please don't apologize! We are all here to learn, and it happens quite often with all of us that something small is missed while taking care of other bigger issues.
Thanks for your contributions!


needs testing, might be buggy :)


The `has-filter` class is correctly applied and removed, but the funnel always look exactly the same with or without the class?


Please PR @ablwr


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


This looks good, though I wonder if we could prevent having to re-read the import lines. That should be fairly fast, though it would feel better if we didn't grab lines again. I've felt before that the hash that we use to read old imports into is a bit limited. I think we could introduce a class to

1. read current imports
2. import new imports
3. remove unused imports

When we have that, it should be fairly simple to just add a `is_dirty` flag whenever the imports change. And then we could use that to decided whether to rewrite imports or not.


@NikoKrause
You've got my full attention. Interesting. Of course, I look at this in the context of CinnXP as well.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1193.e3ea1677f25f24950ecebd117b8aeeb3c234b1d6](http://build.palaso.org/viewLog.html?buildId=67742&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1 (1 new), passed: 455, ignored: 4) Build time: 00:03:11
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


ack


Whoops, I didn't realize I was committing to master. Too late now, I guess.


Yes and thanks for the details!


Hi all.


Not funny xD


Why not %5u/%5u?


I think this is a good one. I will try it, if the current tuning fails.


rebased the renaming onto this, not what I meant to do, but it's ok.


If this function is not implemented, `4 * sizeof(float)` is assumed.


Suggested by @Bianott


was going to write some cov for project but mocks make me sad :( haha. Ideally we'd mock out Function there but then we need an interface and a lot more clutter


That is okei, the problem was related to the use of the command palette and the cache logic, I fixed that on latest commit, thanks!


Please include the full license too.


Commented and tested


you don't reach this code


Hei Robin, für was bruuchsch die klass error genau?


I don't want this here, surely there has to be a sensible function to use from JaSerializer that doesn't take a connection.


Ah, you just hadn't committed the rebuilt scss!


Sam da ves, v C-ju in C-ish jezikih se if z enim statementom pise v isti vrstici kot pogoj in to brez curlyjev


Oh, yes, unquestionably (I'd forgotten about the nonsense Python has with e.g. `KeyboardInterrupt` and the like).


An enormous work pivoq. Thanks!


自分が書いたコードだと思って消しちゃいました、というか自分も実際に同じように書いた記憶がｗ
何か問題があったはずなんですが、思い出せないのでいったん戻しますね。特定の条件でプレビューが空白で表示される状態になってました。


How many test cases did my regex work on?


\cc @josevalim


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1190.e3ea1677f25f24950ecebd117b8aeeb3c234b1d6](http://build.palaso.org/viewLog.html?buildId=67707&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1, passed: 24) Build time: 00:03:05
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


It worked pretty well but on links and language names with `.`'s and `,`'s it tended to mess up. I've pushed in the last commit a regex that (seems to) work perfectly.


Your mention is the right way for us. Try it.


WoW ! Nice Jack !


OK, sounds good. Does it work properly on bonuses too, e.g. on [this question](http://codegolf.stackexchange.com/questions/68852/hexadecimal-of-a-character)?


Merged from upstream


I'm pretty sure Raptor can make this READ_COMMITTED


그전에서부터 화면으로깨우기를알아본게있습니다.
https://github.com/flar2/bullhead/commit/2db51a65f1ebd890744afb08e0a2fe28da3e9984
웨이크제스쳐라고있더군요. 분리방식보다는 웨이크제스쳐를한번고려해보실생각없으신가요.?
아그리고 6.0.1 보안패치버전나왔습니다.
거기서부트이미지패치가있더군요.한번확인부탁드립니다


Yeah, it does. I have yet to find a case where it doesn't work


Is the 'air_above' here needed? it's a condition for this block at 979.


'and mapgen_v7 by kwolekr and paramat.'
as kwolekr's full name and email is above.


@aturon I put it in a module so as to form a privacy boundary (and because more modules is almost always better from my POV :)


Where is the found variable set in this case? Don't think it is necessary though since it's of type interface


didn't even know that function existed


ack


Should these 2 profiler lines be commented out?


Perhaps it would be wise to cache this to make this more efficient.
After all, the chunks shouldn't be changing bytes between these calls.
```java
byte[] bytes = chunk.getBytes();
buf.writeInt(bytes.length);
buf.writeBytes(bytes);
```


If sudo is set to true for this task, it will break since the home directory for root is 'root'.  Where as the .henchman directory getting made is in (in the case of a vagrant box) /home/vagrant/.henchman


When switching to latest fred next, I used bcprov-jdk15on-154.jar as filename when compiling fred itself, and it didn't complain. Maybe someone changed fred to not require bcprov.jar anymore instead?
So this will fail then if people don't use bcprov.jar at fred anymore.


Beta BOFC


111303866_ATSingleDuctReheat (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2069 of 2080 tests passed)

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


This doesn't match what fred nowadays uses (154).


Isn't a return missing here somewhere?
It seems to me if mmap(NULL) succeeds,
you will still map lowram once again a few
lines below.


![ddns](https://cloud.githubusercontent.com/assets/1783131/12213830/4e28a1c4-b680-11e5-947d-e86fb9c05e71.jpg)
I have five domain ,and the program ddns updater only one auto update or I…
Thank you


Looks appropriate.


111303866_ATSingleDuctReheat (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Where is this getting installed without your change on Windows?

I'm a little surprised that the .so is being put in the same place with and without on Linux.


Mykodi is a beta test of BOFCproductions configurations and settings


Hmmm, it might be better if the Maya symbols are explicitly exported instead of trying to reuse ALEMBIC_EXPORT.

I'm worried about side effects when mixing these Maya plugins with static versions of the library.


Update perf data output


lol @typo


需要空行


Thanks @mikemcquaid!


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1191.e3ea1677f25f24950ecebd117b8aeeb3c234b1d6](http://build.palaso.org/viewLog.html?buildId=67709&buildTypeId=bt431) outcome was **FAILURE**
Summary: Execution timeout; tests failed: 1 (1 new), passed: 811, ignored: 7 Build time: 00:36:04
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData: <no details avaliable>

```


This can't happen here so no worries :)


Thank you for your issue and solution.  I had the same error.


They won't actually behave as expected with this commit because a line got left out. Commit f38e2b1c2392f75b36b43792a07d897c88510c2c fixes this.


@nhoizey True, but Firefox dropped support for that in 41.


Do we want to include our own copy or use a CDN (e.g. [Google Fonts](https://www.google.com/fonts#UsePlace:use/Collection:Open+Sans))? Open Sans is used a lot and we can take advantage of using a central file for caching purposes.


for some reason, smartMotor and smartServo became inherited by general. I undid that.


Might've pasted it over but the change is still just added on so there's really no "deletion"


why?


why are you deleting  ENV.delete "PERL_MM_OPT" ?  my local perl lib is in a  non default location and that's where i want all my user libs.    typically my perl ENV is this unless i'm in a dev shell env

export PERLHOME="${HOME}/perl5"
export PERL_MB_OPT="--install_base \"${HOME}/perl5\""
export PERL_MM_OPT="INSTALL_BASE=${PERLHOME}"
export PERL5LIB=${PERLHOME}/lib/perl5:\
${HOME}/.opam/system/lib/perl5:\
${HOMEBREW_PREFIX}/lib/perl5/site_perl
export PERL_LOCAL_LIB_ROOT="${PERLHOME}:${PERL_LOCAL_LIB_ROOT}"

i don't see a switch or a recursion of how you're going to build the 32bit and 16bit libs


110568302-FixCrashInZoneComponentLoadSummary (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1499 of 1499 tests passed)

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


`result` is of form `{:ok, _}`. Is this the correct return when closed? Should we return `{:error, :rollback}` or something similar? Maybe `{:error, :closed}`?


> result is of form {:ok, _}. Is this the correct return when closed? Should we return {:error, :rollback} or something similar? Maybe {:error, :closed}?

@josevalim


We sandbox, and _eventually_ that sandbox will be enabled globally by default, and that is incompatible with Perl modules being able to install willy-nilly. We _generally_ believe during install formula shouldn't be writing outside the `HOMEBREW_PREFIX`.

There's also often this issue with Perl stuff when the environment contains that method:
```
Only one of PREFIX or INSTALL_BASE can be given.  Not both.
```
Which kills the build, very quickly.


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


Shoot. I wish I'd had some feedback on #2541 and #2590 when I asked for help from a windows dev to verify my changes. I really do find the previous version easier to understand. Too bad it didn't work =P


Good point. The issue with using a CDN is that we are then reliant on the CDN and are loading resources from a third party, sending user info along as well; all of this is more a matter of principle than anything else.


Hi, your builds are broken, please fix them.


ack
though I don't understand what's done here^^


a


Hi,

Thanks!!.. Is see you are busy :+1: enhancing  SatPI nice to see :-)
I could use the some help.

The problem with this is at the moment, that it can not be dynamically changed. For that it should close the port and reopen the new one.


@copenhas this doesn't test every combination of errors just sort of the username duplication, an invalid doc one that is sent if you pass garbage in and then for like password mismatch simply check that "detail" is in there, that probably should be errors not detail...

perhaps the next step would be ensure the structure for errors but the actual errors I'm not concerned with, because we test that out in the user tests and I feel like it's a lot of duplication.

It does of course check to ensure the user created was returned as expected and validate the token etc.


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


Dr Rest schiint z'funktioniera :+1:


unack
see scrutinizer


Why is stride 0 by default?


Hi,

Yes it needs a restart of satpi right now, I will look into that dynamic part :)


This is like an exact copy of session view, perhaps pulling this out to a standalone serializer would be better and ditching the view is best like the docs show [here](http://hexdocs.pm/ja_serializer/JaSerializer.Serializer.html)


Hi Marius, thank you for feedback
i will test it once more time. I think "nearby" is tricky event as it depends on distance reading of the beacon, and for some beacons distance could be wrong calculated, say 3 meter and the app expect nearby as 2 meter or less. By the way what beacon vendor do you use?


If you can, it would be nice :+1:   Thanks!


ack


but you do understand what happens here, do you?


not really, looks just sense-making :O


Why do you still use the minimum enclosing circle? I thought we use the least square solution.


Oops, includes config files in /etc/ice/ I see.


See also: https://github.com/wwmayer/FreeCAD/commits/py3-5


What is the reason for the hard coded values (230, 70) etc. here?


no bracket necessary


Good catch. Thanks Shark


damn edvin thats good


Hue Hue Hue


@Makman2 the processing module passes those results to the main function so this commit removes the results from those, previously they were only ignored when printing them, now they are globally removed


gsehni das richtig das dä Fall aigentli garnie uftrette sötti? Input wird ja immer grad uf öppis valids zruggsetzt wenner nid valid isch?


This file wasn't written by 'Skurydin Alexey', it was initially written by hrsavla and reworked (quite a bit IIRC) by me. Although many of the core sming files started off being written by anakod, most if not all have had some degree of work or new code by others, so I don't think listing a single author or copyright owner is very accurate.


Fileformat still needs to change.


Can't add things to these files, they are a separate module pulled in from upstream. If you add anything to these it will be lost on the next update. Only option is to get it added upstream, which reminds me...


wouldn't you want to redirect to main after they signed up?


This should be

```js
socket.channel("game:lobby", {})
```

now same for the player channel. No need to pass in the token, the joining of the socket takes care of the token.


### Re: patch
[patch 7.4.954](https://github.com/vim/vim/commit/f609dcf8c1094f6fc95f4fc36321a1fb08a7110c)
worked like a charm,
```
The new issue only concerning to cursor "freezing" (some milliseconds)
Cursor "freezing" reproduced during cursor moving like jjj kkkk lll hhhhh
```


This patch isn't required, the proper solution is the changes of lg_chunk settings.

ln -s 'lg_chunk:14' /etc/malloc.conf or change this property in source code.


I thought you use +- 90


The this can be written much more compact as `samples=contours1[maxj][(ang > i-10) & (ang < i+10)]`


I think @timdorr was trying to fix https://github.com/rackt/react-router/pull/2864, which looks to just be buggy.

I am actually a little concerned about https://github.com/rackt/react-router/commit/a94cb1c393b2bd54e5a9cbbeb08ea6044f2530d3 – if we've dropped e.g. Safari 8 and Mobile Safari 7 from our test suite, are we even claiming that we still support them?

It's asymmetrical, right? Old browsers are much more likely to be quirky/buggy than new browsers are likely to introduce regressions to our code.


100 should not be hard coded but an argument to the script.


@fishcakez On checkout, I am sending the pool_mod and pool to the client so the client is the one issuing the checkout. However, this is non-ideal as it would be preferable if this process was the one doing the checkout. I have done this in the next commit and you can see how it breaks the build. You can see the changes in the next commit, as well as the build break and a TODO: https://github.com/fishcakez/db_connection/commit/d98f3f91b889105c97e4eab520b0c31e8e9c9652

Thoughts?


Especially bmp images will fill up the disk. We should really rethink storing those images.


Nice! :+1: :-)


I feel like this should be in the game channel the user talks to the game channel and then requests to join_game, but nit picky I guess. I feel like the player channel join/2 should be what joins the game, instead of having the client explicitly join_game.

of course this runs into the issue of not being able to broadcast the welcome message I suppose. So I don't know, I can see why having it here is good just feels out of place, but I think it's probably best for now. So I'm good with it.


In general
* code needs to be cleaned of commented lines that are not necessary
* code needs to be PEP8 compatible
* it would be good to split the main script into smaller functions to make the code easier to understand


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1194.e3ea1677f25f24950ecebd117b8aeeb3c234b1d6](http://build.palaso.org/viewLog.html?buildId=67744&buildTypeId=bt431) outcome was **FAILURE**
Summary: Execution timeout; tests failed: 2 (1 new), passed: 810, ignored: 7 Build time: 00:35:49
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.Constructor_CreateNewProjectFromBundle_BundleHasNoLdmlFile_ProjectIsCreatedSuccessfully: <no details avaliable>

GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


On your travis profile page you can also force a refresh of your repo’s, doing that ensures the new badges work immediately thereafter!


Ah, thanks for the tip!


You should use `add-hook` instead.


I would rather test against browsers that are actually in use. Regressions happen all the time, often very subtly. Safari usage is already 75% version 9, iOS is almost entirely 8.4 and 9.2 at this point, Chrome auto-updates so well that you'll run into sub-0.1% usage on a version only a few months old. The web moves quickly!


mere ras pushav :D reverts vuzam


You copied too much ... Just a small part is enough


What


Do


They


Used


@aodzip I need you on Gitter XD


For


just to make sure that it excludes non results correctly at the same time.


@keithkfng  Ok


the hidden result is there for the same reason, this stuff is closely related to the exclusion so it's vital to check implicitly that that works too IMO


why is this correct?


I don't understand the point of this commit, it just re-moves a block of code that was moved for no reason in an earlier commit.


this is a copy paste


why? at least add a comment so future coders will know why this sort is correct.


copy paste


should be reuseable


DAwit


DAWIT


copy paste.


The config.h files should use /* */ as comments rather than #'s


Sudo apt-get install sweet@:-D
On Jan 9, 2016 3:20 PM, "practicalswift" <notifications@github.com> wrote:

> Nice! [image: :+1:] :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apple/swift/commit/11fda96b850d01816752ecb230f9e980c61eb7eb#commitcomment-15356580>
> .
>


String equality check not working? Really?
Я даже когда .bat'ники под шинду писал (первый опыт в "кодинге"), там и то это есть и ворк норм.


could you align that correctly? so putting the `5` into an own line^^


The change lacks tests.


I'm not sure if I understand why you're doing this. All of these types used to be defined in the SVM crate but were factored out to `seat_util` for modularity reasons, so that the Scheme compiler crate (and other future compilers) _don't have to_ depend on the entire VM runtime. All you've done here is undo commit 1ee4495740278682be2c096168d9b4a915f83aea.


the change lacks tests.


@ajlennon This should be fixed with https://github.com/mono/mono/pull/2414. Thanks for your workaround, it pointed me in the right direction.


but the result shouldn't go away, since we don't auto apply it should still be there!


like I said in the other review, just get rid of the catch handler completely.... I explained the reason in the other review. Having a catch handler is pointless because you're simply re-rejecting the error.


Thanks, using %windir%\Sysnative worked fine for launching the x64 version.
Guess I'll just have to add the architecture check and implement a working convertUriToPath function now, and it's starting to look like something usable :)


assert*Not*Equal


yeah but the comment irritates^^


Same with these two things. You're supposed to get rid of them. At least what you had before was unnecessary but correct, while this is unnecessary and incorrect (you're not wrapping the result in a promise. Get rid of the `then` and `catch` handlers entirely. You have to be sure you understand why.


hit,
thx,
yes its not a bad idea. i do check it soon. seems interesting.

maybe you can set the favorites box top padding very high and the
padding for boxpointer/ hover-icon / user low.

just an idea that might work or not.


Logging this here as well, I believe this PR is the cause of issues in 8.3.0 and 8.4.0

https://www.reddit.com/r/TronScript/comments/402cvo/tron_v840_20160108_critical_bugfix_dont_remove/cyr7oo7
https://www.reddit.com/r/TronScript/comments/405zp0/tron_stuck_on_kill_microsoft_telemetry/
https://www.reddit.com/r/TronScript/comments/40740s/tronscript_crashing_out_with_no_error_or_logs_on/


Good, but you _also_ should get rid of the `then` handler for the ***exact*** same reason.


Type in Hello.vue
"changing this line won't causeS changes"

Vue.js<3


ack


(BTW, yes, I'm aware that `tm_tag_is_anon()` will ensure that (currently), but am somewhat afraid it can easily get updated at some point and change the "there will be 2 `_`" contract


Oops, thanks!

Placing this in a `finally` block may be a good idea:
```java
BufferedReader reader = new BufferedReader(new InputStreamReader(new FileInputStream(configurationFile)));
try
{
    for (String line; (line = reader.readLine()) != null; )
    {
        // ...
    }
}
finally
{
    reader.close();
}
```


On `:DOWN` we want to disconnect. We can simply send the same `DBConnection.Error` as `DBConnection.Connection` would use on receiving a `:DOWN`. It is perfectly fine to send the old state in the case of disconnect as the connections have to be able to handle this when a timeout occurs or direct `:DOWN` occurs.


this field is now no more initialized in the ctor


I think a less conventional approach to the GUI image would be good. Take a look at Witchery's Witch's Cauldron GUI to see what I mean. It doesn't imply any sort of actual GUI for the machine, or it having slots. Instead, it implies in-world recipes.


Yes?


Worth noting that this functionality increases the turn count by one. This seems a reasonable behaviour for this function and without it then @last will not function properly. Should probably be mentioned in any accompanying documentation however.


@ffcb ich denke du hast deinen test mit kathleen build #738.
Ich kann ja mal ein build mit dem merge https://github.com/openwrt/luci/commit/3bae3c7f5e8c383e481667fcfeae7765a1dacc75 und 90bc08048925133f1522c347e3239e3bf4b4fa52 drinnen anstossen


Does the "XXX" in this comment denote a TODO? just pointing it out in case there was an oversight. I trust you completely.


We did it!


typo?


Hm... you make me doubt things. Actually, I am not sure.
I think, there could be cases where different classes use their objects absolutely identical with respect to their fields (store same value types in the same slots, etc.) which could result in them sharing shapes.
In my own object model, this was not the case, because object layout was always specific to a class. Here, it could actually be happening. Hmm...
Need to investigate that.


@bluedxca93
There are 3 changes around here. The above is 1 of 3. Please check the other 2 as well 'cause I don't do anything before knowing your opinion.


Yeah, so in JRuby+Truffle, they store the class as part of the shape's object type object. Think, I'll need to do that as well. Thanks, very good catch! Didn't think of that.


Вообще в данном случае проверку while(true) лучше заменить на while(!currentThread.isInterrupted()), чтобы не игнорировать прерывание потока методом interrupt(). В остальных случаях(когда JVM сама прерывает interruption points) прерывание игнорируется.


Fixed in next commit.


Excellent work, does absolutely not look like my GPL2 work.

https://github.com/tpoechtrager/wc-ng/blob/master/src/mod/extinfo.h#L44


habe verschiedene weekly-builds aus http://buildbot.berlin.freifunk.net/buildbot/unstable/ramips genutzt, parallel dazu per ssh das profil manuel auf nen 4300er mit kathleen 0.1.2 aufgespielt
bisher konnte ich keine probleme feststellen


sub directory => subdirectory


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.447.0](http://build01:8888/viewLog.html?buildId=13789&buildTypeId=ConfigInjector_CI) is now running


This should likely be pulled out into a helper or another (better) way of testing the inclusion of a substring should be devised.  The idea is that the acceptance test shouldn't be bound so tightly to the markup, ie: don't require the test to use very specific selectors, instead just see if the text is included in very general selector, or just the page at all.


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.447.0](http://build01:8888/viewLog.html?buildId=13789&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:32


maybe it's worth caching the lookup not to perform it twice, as apparently this is part of a (sometimes) hot spot in the code


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/parsing/ConfParser.py`, lines 118 to 119.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/parsing/ConfParser.py
+++ b/coalib/parsing/ConfParser.py
@@ -115,8 +115,7 @@
                                 value,
                                 origin,
                                 # Ignore PEP8Bear, it fails to format that
-                                remove_empty_iter_elements=
-                                self.__remove_empty_iter_elements),
+                                remove_empty_iter_elements=self.__remove_empty_iter_elements),
                         allow_appending=(keys == []))
                 else:
                     self.get_section(
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/parsing/ConfParser.py`, lines 129 to 130.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/parsing/ConfParser.py
+++ b/coalib/parsing/ConfParser.py
@@ -126,8 +126,7 @@
                                     value,
                                     origin,
                                     # Ignore PEP8Bear, it fails to format that
-                                    remove_empty_iter_elements=
-                                    self.__remove_empty_iter_elements),
+                                    remove_empty_iter_elements=self.__remove_empty_iter_elements),
                             allow_appending=(keys == []))

     def __init_sections(self):
```


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1195.e3ea1677f25f24950ecebd117b8aeeb3c234b1d6](http://build.palaso.org/viewLog.html?buildId=67746&buildTypeId=bt431) outcome was **FAILURE**
Summary: Execution timeout; tests failed: 1, passed: 811, ignored: 7 Build time: 00:35:19
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


some stuff here that could help a little too much with assignments :)


HDHT support => HDHT supports
a abstract => an abstract
interfaces HDHT.Writer and HDHT.Reader => interfaces `HDHT.Writer` and `HDHT.Reader`
HDHTReader, HDHTWriter classes => `HDHTReader` and `HDHTWriter` classes


@marcshilling if you can publish ! <3


this one instance of `pkg` needs to stay :)


Test comment


returns => return


methods takes bucketKey as as => methods take `bucketKey` as the
(should have spotted this on the earlier version, sorry)


refer => refer to


Hello. I will respond on your request in two days. Today I just released a new version by another request. Your point is absolutely valid and I will support this case, but first I need to crease a unit tests and then I will accept your pull request. Give me a couple of days.


This seems like it could be really annoying if anything outside dolphin needs to know the mac


Great minds debug the same.


in a application => in an application


Looks ok to me, but I am not the expert on this.


@contra here you go...


Lol!  The diff never lies!


Delete the commented lines


This should be `rake db:seed`


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1196.e3ea1677f25f24950ecebd117b8aeeb3c234b1d6](http://build.palaso.org/viewLog.html?buildId=67748&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1, passed: 234) Build time: 00:02:43
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


It works great. Espresso waits for network operations.  But tests fail when network error occurs


I think in RecipeDao.js in function _getRecipe() is a better place to add The toString() function.


C# 6???


The partial count are => Each partial count is


previous count and write back incremented => the previous count and writes back the incremented


Should be apply lmao, will fix later


Do we need to use a data.table here? It's a pretty big dependency.


Why not?


@fernandog you can't get rid of the try - except or it will bork if 'total_results' isn't present.


this is the beginning of the new files


Temporary fix. See thoughtbot/ex_machina#78


good shit jq


Remind self - remove this.


And this.


Either backquotes or double stars on the class names starting each bulleted item.


I agree - note thought that you still need an install of scons, as the generated project is really just a way of showing the files in VS. The build still occurs within scons.

Still, it makes developement much easier to have a full IDE to work in.


AbstractSinglePortHDHTWriter => `AbstractSinglePortHDHTWriter`


never mind, the reason I was using submodules was so rollup would properly bundle lodash


as below => as shown below


getCodec => `getCodec`
processEvent => `processEvent`


BTW can you test if the CHECK_ADD_OCE_OCC_DEF() biz can be removed for good in CMakeLists.txt?


Yes, having to press TAB twice is annoying. Unfortunately if you don't have the `echo {}` completion in, something like `ssh myserver` doesn't work because the completion from `compgen` will only return `ssh`. Maybe it works if we change the parameters to `compgen` so that they can complete commands along with their arguments?


ups, wrong line for the comment closing


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


cool, would you mind add another line answer?? :question:


this makes me uneaesy


Yeah, I tried getting rollup to automatically bundle lodash-es from npm, but couldn't get it working.  I need to familiarize myself with all the plugins and options more.


where is this?


It seems like the kodi pvr plugins are still using platform packages with the old name and want to keep this for Jarvis:
https://github.com/kodi-pvr/pvr.vdr.vnsi/pull/43

So, to keep things compatible, would it make sense to maintain a Jarvis branch of kodi-platform without this change? Or a more general question: Are there stable versions of kodi-platform that correspond to a kodi release? There are no tags or releases available on github.

I'm asking because I'm providing different kodi packages for Arch Linux and until now just used the latest git version for kodi-platform which currently is not possible anymore.


I mean herecool, would you mind add another line answer?? ￼


For example => For example, suppose
(or: For example, assume)


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1197.e3ea1677f25f24950ecebd117b8aeeb3c234b1d6](http://build.palaso.org/viewLog.html?buildId=67750&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 812, ignored: 7 Build time: 00:16:37


This seems relevant: http://stackoverflow.com/questions/3520936/accessing-bash-completions-for-specific-commands-programmatically


nah


There's a space after `()`


You should be naming your commits properly


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `setup.py`, line 11.*

The issue can be fixed by applying the following patch:

```diff
--- a/setup.py
+++ b/setup.py
@@ -8,7 +8,6 @@
 from coalib.misc.BuildManPage import BuildManPage
 from coalib.output.dbus.BuildDbusService import BuildDbusService
 from coalib.misc import Constants
-

 class BuildPyCommand(setuptools.command.build_py.build_py):
```


Danke! Korrigiert.


@Nadahar I checked the log the user provided but it seems OK from the UMS side. UMS offers the subtitles to stream but the TV is not asking for them. So I asked user to test the previous working UMS version and provide the log to be able to compare it.


I'll rebuild all the kodi packages tomorrow with the legacy platform. That is what I am going to do until upstream kodi changes.


yes, you need to change the if command, not add it.


Why did you drop this linebreak? It looks nicer one line above


@valib Great, I just wanted to let you know about it as I know nothing about the subtitle system.


These should not be mixer_ctl functions because they operate on the mixer, not the control. In other words the ctl pointer is only used to access the mixer pointer. Instead, use these function prototypes:

int mixer_subscribe_events(struct mixer *mixer, int subscribe);
int mixer_ctl_poll(struct mixer *mixer, int timeout);

Please let me know if there's any other reason to specify the ctl here.


As I mentioned at our class, stack is a deprecated collecction — use dequeue instead.


You can do `|(key, value)|`.


will be resolved with corrections from https://github.com/OpenDevelopmentMekong/opendev-wptheme/issues/363


Are you sure about this commit message? :-)


I agree with this API, though the naming suggests that it is used to poll, which it is not. Internally it calls poll, but externally it waits for an event.

Suggest calling this mixer_wait_events() or similar.


Using map for operations is a good idea. However, why don't you incorporate the operation characteristics into one class? Like you have priority and operation itself (command pattern)


I know anything actually is worthy mentioning but sometimes I'm too lazy to come up with a good desc


wieso erstellsch du .date-error, bruch doch eifach ng-touched.ng-invalid womer scho hännd und genau glich isch


haha i was going to post a question about this earlier today


Very nice!

There is a predicate called `sat_count/2` which allows you to count the number of admissible assignments without enumerating them, I hope you find this useful. For example:

?- time((main2(4, Vs), sat_count(+[1|Vs], Count), writeln(Count))), false.
13630
% 170,397,238 inferences, 18.546 CPU in 18.677 seconds (99% CPU, 9187691 Lips)
false.

I am using `false/0` to avoid the display of residual goals in this case.


Please revert this file.
Year is wrong now, team name/site is okay, but we prefer to update the copyright notice when a file is actually modified.


Indeed, you it is hard to get rid of bunch of ifs when you are programming a state machine, but still moving some logic into sub-methods will make code more readable.


The high level idea is explained in this article http://www.codeproject.com/Articles/28636/Minimalist-Meta-Model-for-CRUD-Applications


This swich could and should be avoided by creating the operation class.


This switch have to be removed and replaced with OOP design.


---
- name: Warn users if the server's Linux distribution is not Ubuntu 14.04
  pause: prompt="Ubuntu 14.04 is the only fully tested distribution; the setup will probably fail. Press Enter if you still want to continue."
  when: ansible_distribution != "Ubuntu" or ansible_distribution_major_version != "14"

# Set default variables
- include: set-default-variables.yml

- name: Ensure the APT cache is up to date
  apt: update_cache=yes
       cache_valid_time=3600

- name: Install Streisand common packages
  apt: name={{ item }}
  with_items: streisand_common_packages

- name: Copy the HTML header and footer templates that are used during documentation generation
  copy: src={{ item.src }}
        dest={{ item.dest }}
  with_items:
    - { src: "header.html", dest: "{{ streisand_header_template }}" }
    - { src: "footer.html", dest: "{{ streisand_footer_template }}" }

- name: Copy the unattended-upgrades templates to enable automatic security updates
  copy: src={{ item.src }}
        dest={{ item.dest }}
        owner=root
        group=root
        mode=644
  with_items:
    - { src: "20auto-upgrades",       dest: "/etc/apt/apt.conf.d" }
    - { src: "50unattended-upgrades", dest: "/etc/apt/apt.conf.d" }


Not in the long run. This was just stage 1 of getting my vis thing in
there. I can move it all to C++

On Sat, Jan 9, 2016 at 4:23 PM, Oliver Keyes <notifications@github.com>
wrote:

> Do we need to use a data.table here? It's a pretty big dependency.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hrbrmstr/iptools/commit/30e8a4545a087cd96447dcca342fb90e88c0e02e#commitcomment-15356831>
> .
>


Tak CORS pořešeno a teď se mi ty rozcestníky chovají zajímavě - při každém posunu mapy se znova dotáhnou a po chvíli mám jeden rozcestník klidně i desetkrát :-D

![duplicity](https://cloud.githubusercontent.com/assets/155542/12218573/a1e9e4fa-b723-11e5-97ac-c5510f1955ab.png)


Item id="6261" is used also for "Invalid path", it's currently hard coding so I prefer to keep it untranslatable.

For adding ```<Item id="2036" name="Repeat :"/>```, I updated english.xml here:
https://github.com/notepad-plus-plus/notepad-plus-plus/commit/4c53179c52ef2712875797152e231f18b946f6f8

Thank you for your contribution.


Here we need to explain how the addition of more partitions affects the above expression to bring it within desired range.


Why everything is from genisys :V soon everything will be from genisys :/


Commit is for checkpoint 30, not 29.


For example, suppose with 128MB ...
This sentence feels a bit off since the prior sentence also begins with "For example ...". Suggest rewriting as follows:
If `S`, `T` and `B` are 128MB, 30 seconds, and 40MB respectively, this expression evaluates to 10, so if your keys span more than 10 files with 30 seconds, the write cannot be sustained.


Item id="6261" is used also for "Invalid path", it's currently hard coding so I prefer to keep it untranslatable.


Then fix this properly:
See [MixinEnchantment](https://github.com/SpongePowered/SpongeCommon/blob/master/src/main/java/org/spongepowered/common/mixin/core/item/MixinEnchantment.java#L105)


I think this should be ``n < x`` (otherwise, this function always converts to YiB).


This will be a NPE if mCameraDevice == null


For adding ```<Item id="2036" name="Repeat :"/>```, I updated english.xml here:
https://github.com/notepad-plus-plus/notepad-plus-plus/commit/4c53179c52ef2712875797152e231f18b946f6f8


This is not a pull request. I followed along and understood what Brian said to me about GitHub, but the website itself didn't show me the comment section or the second green button, "Make new Pull Request." Instead, it showed me some work I made on Thursday, and I have the ability to add more to that Pull Request which I believe will interrupt the grading process. I'm sending you a link to the commit, since it seems to have the same result as the pull request.


@andrewgioia the minified CSS needs updating


DTFile => `DTFile`
TFile => `TFile`
get => `get`


I/Os => I/O


What is study mode?


Fixed to return local time in timerEvaluate();  Time.now() returns GMT


Yeah. To be honest, I look back at it now and I groan inside because it was poorly made.


@csadilek yes, it would be sufficient. My intention was to enable logging during test execution.

On the other hand, I think the `runtime` scope is also valid because logging is done in `src/main/java` code. In other words the project needs some logging implementation on runtime classpath for logging to actually work. @psiroky - do you have an opinion on this? Is it better to declare runtime dependency on logging impl in each project that uses logging API or does it cause some maintenance complications?


Ha oui ? Bon ca je comprends pas trop pourquoi, tu m'expliqueras.


Whoops, just pushed that up @ardeay


C'est top ! Merci !


// Change the login email - FIXME: Shouldn't this be in the auth section?


@andrewgioia dude you rock!


The fix is pending in #510 by the way


I love the style of our coafile :D


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


This shouldn't be hard-coded


Somehow fixed normals in oldlib. I was using LaplareRegressor, while @carlosjoserg was using GaussianRegressor, the latter seems to work:
![screenshot from 2016-01-09 16 00 06](https://cloud.githubusercontent.com/assets/1950251/12218598/40983a48-b724-11e5-9910-887e53c92323.png)

Newlib is still plotted and since it was not modified it's still wrong.
However for the mug the same sigma and length that are good for the sphere, produce wrong results (normals), this leads me to believe we have to tune them somehow.
For instance I was thinking of making them a function of point density, if points are for example 3cm distant, length could be three times that. (We can "easly" measure point density with pcl)

Or the mug wrong normals could be unrelated to sigma,length and instead be related to the fact that the object is not convex (like a sphere), but appears like cut in half. To validate this I'll make another test with an half sphere.

Cheers


> but normals look ok

Do they?

> Todo find sigma and length to be dependant on input point density.

Don't really know how to do that... remember we shouldn't know the object...


ano to presne delaji:), vrstva se musi po presunu vymazat, coz ted prestala delat, to se brzy vyresi, dnes uz asi ne


where did `use_spaces` go?


Sure about this?

I was passing that to make the projection independent of the covariance function used in the regressor.

But if you see it fine, good then.


This code does halfway what we need:
```bash
#!/usr/bin/env bash
source /etc/bash_completion
__print_completions()
{
  for ((i=0;i<${#COMPREPLY[*]};i++)); do
    echo ${COMPREPLY[i]}
  done
}
COMP_WORDS=(git l)
COMP_LINE="git l"
COMP_COUNT=6
COMP_CWORD=1
_completion_loader git
_git
__print_completions
```

Unfortunately it seems to complete as if called with just `git` instead of `git l` and it would have to be made generic enough to do the counting and completion loading for all commands.

Phew, I would have though there's a command where you can just call
```dear_bash_please_do_complete 'git l'```
and it will do that.


These should not have "www.".


Splitting at whitespace isn't quite right for 'versioned' packages. Split only at comma and purge whitespace at left/right.


oder wenigstens

    .ng-touched.ng-invalid,
    .date-error {
       ...
    }

anstell vo code copy pasting. witeri informatione zum DRY-konzept findsch susch unter http://www.itwissen.info/definition/lexikon/DRY-don%B4t-repeat-yourself.html


If I prefer to skip the not yet implemented features, it's for clearly detect when a feature that existed previously is broken and differentiate them to remaining ones. We can remove them one by one when you develop or use ```php tests/test.php feature-file-name```

Maybe you have a counter argument?


looks sane to me, one weird thing but it's ok.


logo commit without alpha I mean ^^


@jakeczyz happy to merge, but please update your code so quadmetrics is listed alphabetically first


@piranna Something like this?


into the default section because we use it in multiple sections


dying*


that looks weird, then we should put the comment above that statement if it's too long^^


für was genau bruuchsch da die error class :question:


ah, PEP8Bear didn't know that they're just one comment it seems ;)


Indentation in this fn seems off, did you use tabs instead of 4 spaces?


Isso aí foi só pra me lembrar o host e a porta do servidor DNS


Хз, оно как-то странно работает: то ок, то не ок. Вывожу значения переменных, а они то норм, то как-то покарябаны. Короче, какая-то черная магия, не знаю, что происходит и что я делаю не так.


bbc


Thanks for mentioning my humble project.
Might be a bit early for so much attention - but then, you never know :-).


pushed a new fixed version


Configuring `git` to use an editor (such as Notepad++) is part of the `git` lesson - but given that Notepad++ doesn't seem to be in the `$PATH`, and there are other options required, there's no harm getting it sorted before we start.


It would be nice to use %zu/%zd, but I seem to recall there being portability issues with it (maybe windows port? maybe some embedded systems not supporting %zu in their printf?).  Using %zu requires a C99 conforming std lib, not just a conforming compiler.


this one is overindented


this one should also indented a level back


Sorry, I'm a noob with github. I change everything in the browser. When I edit a file, I have two options:
1. Commit directly to the master branch.
2. Create a new branch for this commit and start a pull request.

Which one should I choose?

I don't have much time, so installing git is currently not an option.
Someday I am going to learn it to work with git in the terminal, but for now working in the browser is the fastest option for me.


reklamu klidne zrus
ga je (UA-308728-3) na stary openstreetmap.cz, je aktivni nekdy od pulky 2008.

Nekam sem te tam pridal s pravama read&analyze ... kdyby to bylo blbe tak to zkusim znova


also indentation level


Yes, I agree the patch is a bit ugly.

Maybe an idea - what about dropping the ugly tm_tag_get_anon_name() and just using tm_tag_type_name() instead? It would have to be moved outside TM_DEBUG of course.

Even though I don't like the numbers much, I'm not too attached to this patch so if you don't like the idea with  tm_tag_type_name(), just tell me and I'll drop the patch completely.


Shouldn't that be at the end of the list directly and not in a new line?


Seriously, you should use some `-l`…


@labrys
API will all always show the '0'
with result:
https://gist.github.com/duramato/2ce34a202c9089d0e197
without results:
{"0":{"total_results":0}}

if no results I handle with `if not '0' in jdata`

from @duramato


I think PEP8 wants yo uto indent two levels at a continuous indent, that solves the if problem


obsolete comma


Yes, that's it :-) A better alternative is

```Javascript
function inspect(depth, context)
{
  // ...
}

Object.defineProperty(info, 'inspect', {value: inspect});
```

This way `inspect` is a non-enumerable attribute of `info`, that's cleaner. Also, I would enable the different params when using the `all` flag or when there's no arguments instead of consider them as special use cases.


I think we should indent all comments then


You can use Groovy truths here...

```groovy
if (grgit.branch.current.trackingBranch && grgit.branch.status(branch: grgit.branch.current.fullName).behindCount)
```


Wg. mnie trochę za duży i oczojebny ale to może tylko ja jestem zbyt nadgorliwym zwolennikiem minimalizmu.

Zmniejszyłbym go tak o połowę.


Dear reviewer, I removed this code because a node's key is never settable to null.


Dear reviewer, I removed this check, because it is already covered in line 180 (old) and 177 (new) - this creates parity between Debug and Release


hm I think pep8 allows both, not fully sure, would have to read it up


if default `causes` would be an empty tuple (or an empty `frozenset`, or an empty whatever, really), then this if statement could be removed.


Let's see if this is closer to where we want to be


Yes, i have read through quite some articles back then, but i did not find a convenient way to let bash complete something. I would even do some C-call to a library but i have not found something.
The completion stuff is quite complex :(


No problem. The PR isn't perfect either, and it needs a little more work.


тут не потрібно цей клас вказувати


Read the readme wrt memory.


I'd suggest moving the banner to "usage" mode. Right now it prints whenever it's started successfully, but not when the command line usage is shown. Winds up printing a lot of banners when batched. That would be more like most command line utilities, where the default is to be relatively silent unless information is requested (in this case, implicitly, by failing to provide correct parameters).


overlay1, overlay2, overlay3, etc. These names are unclear in meaning and should be re-specified to improve readability.


ack


> ttps://img.shields.io/pypi/pyversions/remindme.svg?style=flat-square

typo


*miep* typo: PEP8 codestyl***e***s


@adnnoky veritabanı ayarlarını yaptınız değilmi ? (.env dosya ayarlarını?)


Question:
I want to have the menu in another language.
Some text in the menu are translated by default (for example "Computer" -> "Rechner") others are not, for example "Shutdown" stays "Shutdown".

I could directly change this in the settings-schema.json file. But is there a nicer way to do this?


Also, I'd suggest at least reverse alphabetical order on the names. I'm mostly tweaking, you folks wrote it. :) I seem to have pretty high billing here.


nawet o 3/4


didn't you correct that value to 80 for PEP8Bear? If so I would say 80 characters including newline (\n)


> you can let coala correct your source code via `coala`...

hmm... I think just "Invoke coala to let it correct your code according to PEP8."


unless jdata is a string with '0' in it.


Because it works


LGTM, thanks!


In this case the logging impl should be in `test` scope.

Logging implementations should be specified only by end-users of these different modules. In our (KIE) case that means we usually do specify them for distribution zips, etc, but not for individual modules. If we bundle logging impls with the modules we may easily end-up with clashes.

There are several different logging implementations for SLF4J to choose from. So we should in ideal case let the users pick the one they one want. We generally use logback, but there is also log4j, jul and jcl.


@aodzip that are all settings from their advanced config.


so how to check a '0' key instead?


@ilkosta Do you think we should generate a new release now, with the Windows fixes?  I'd like to start using an official npm package instead of my GitHub branch for my project.

Thanks for the merge! :)


It doesn't matter...


що це за файл взагалі. видалили. його утт не повинно бути. в корені.


reack


@PeratX right. They are just unused variables, that wouldn't cause any problems


I shoved it in the Usage. But that only means I'm old school MOST of the way.

As for name order, I had put it in alphabetical by last night. But fine! ;-)


I'm not really sure about this one and the `EVENT_REMOVE`, shouldn't we put the service name in an object we will emit?


Shift that bit into the git lessons then.

Still haven't sorted getting the lessons to build with jekyll - appears broken.

..d

Dr David Martin
Lecturer in Bioinformatics
College of Life Sciences
University of Dundee

________________________________
From: Leighton Pritchard <notifications@github.com>
Sent: 09 January 2016 22:13
To: widdowquinn/2016-01-11-dundee
Cc: David Martin (Staff)
Subject: Re: [2016-01-11-dundee] added dundee specific setup notes (db693ba)

Configuring git to use an editor (such as Notepad++) is part of the git lesson - but given that Notepad++ doesn't seem to be in the $PATH, and there are other options required, there's no harm getting it sorted before we start.

-
Reply to this email directly or view it on GitHub<https://github.com/widdowquinn/2016-01-11-dundee/commit/db693ba5bdffadd021971ed44c2821c1d62084c9#commitcomment-15357048>.

[https://avatars3.githubusercontent.com/u/1867686?v=3&s=200]<https://github.com/widdowquinn/2016-01-11-dundee/commit/db693ba5bdffadd021971ed44c2821c1d62084c9#commitcomment-15357048>

added dundee specific setup notes · widdowquinn/2016-01-11-dundee@db693ba<https://github.com/widdowquinn/2016-01-11-dundee/commit/db693ba5bdffadd021971ed44c2821c1d62084c9#commitcomment-15357048>
github.com

The University of Dundee is a registered Scottish Charity, No: SC015096


Mi się podoba, tylko chyba lepiej by było, jakby był to na górze okna.


ah wait is that also described in PEP8?


You should probably send HTTP code [500](https://developer.mozilla.org/en-US/docs/Web/HTTP/Response_codes). See [docs](http://expressjs.com/en/guide/error-handling.html).


@ffcb ich würd jetzt einfach mal einen bauen lassen, der enthält:
- alles was aktuell im branch https://github.com/freifunk-berlin/firmware/commits/unstable_SAm0815 ist (anpassung am Makefile, support für TP-link WT1043NDV3 und DLink DIR505)
- neues Cottbis-profil
- die Änderungen durch das upstreamen der Berliner-profile

für ramips wird das dann wohl die build-ID 61 oder 62  --> feedback sehr willkommen


`10 * time.Second`


Set it to `:` you mean.


I don't know if it is worth the breaking change or not, but I kind of like `stop_action` better than `stop_command`. With the regular `command` attribute in both motors and sensors, when you write to `command`, something happens, so it is logical to thing that writing to `stop_command` should make something happen too.


This change makes this commit a `semver-minor`, at least.


`if !bytes.Equal(body, old_body) {`


Solve all the TODOs within this commit


@Electrical-Age/admin Thanks, but can you advertise the next time you move all pictures on the server ?
Some forum content are linked to them....


What do you think of adding a slightly more sophisticated argument parser using argparse? https://docs.python.org/2/library/argparse.html#module-argparse

I'd like to add a switch to allow it to output raw (untranslated) values (like just the number for the manufacturer ID), it'll be useful for feeding to the Drop /// Inches driver page. Adding the switch would probably be easiest using something like argparse. I can put it together pretty quickly I think, though I can leave it for you if you'd like to work through it.

I guess this is actually two separate proposals. One to use argparse, the other to have a switch for raw values. Maybe other switches later as well. (In particular, I'm thinking about a way to have it call cppo if it gets a disk and cppo is on the path.)


[Introduction.docx](https://github.com/damicomom/dissertation/files/83870/Introduction.docx)
[results.xlsx](https://github.com/damicomom/dissertation/files/83871/results.xlsx)


Ui, desculpa.


Very nice!


@mmun I believe so, thank you


It looks like this might traverse too var up the tree - see #4900.


looks good except that the
@todo not implemented yet because Nd4j lib does not provide map yet
lines need to be removed


Please review this, @casey-collab


Extracted here: https://gist.github.com/john-kurkowski/d3869575c83d1a719d0a


I am not sure if this was intentional but I really don't like the black font or the spacing on the sides of the banner


This splits on whitespace followed by a comma, which is wrong because there is no whitespace before the comma.

Example (client ip is 1.2.3.4):
```
X-Fowarded-For: 1.2.3.4, 5.6.7.8
```

If you for some reason want to split on either (white)space or comma, a matchgroup should do the job: `/[\s,]/`


OK (though the ratio of anon tags to all tags will probably be relatively small so it won't make much difference).


nice!


This isn't incorrect. If anything, the property names are very wonky. If you really wanted to do it this way (an object literal with your properties), you can clean it up by doing something like: `{type: 'Steam-Vanity-Unresolved', message: 'the name ...', log: 'failed to resolve ...'}`. The object itself should act as the namespace, i.e. you shouldn't be ad-hoc namespacing names with the `SSig` prefix. Then just check if it's your own error with `if (err.type == 'Steam-Vanity-Unresolved')` etc.

However, the more common way to do this is to create your own [`Error`](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error) type. This is the built-in error type, which has functionality such as containing a stack trace representing the context of the error (helps for debugging) and other functionality.

You can normally just create your own instance of `Error` with `new Error("a message")`, but if you want to add additional properties/information, (or even if you don't), it's a good idea to create your own kind of error that derives from it. Say you create an error type called `SteamVanityResolveError`, you can generate one with:

``` js
throw new SteamVanityError();
// you can also do resolve(new SteamVanityError());
```

then using the [bluebird's .catch](http://bluebirdjs.com/docs/api/catch.html) you can handle it specifically:

``` js
// specifically handles SteamVanityResolveError
.catch(SteamVanityResolveError, function(e) { /* handle it here */ });

// your regular .catch(function(e) { ... }); is a catch-all error handler,
// you can add one after specific handlers, kind of like `default:` in switch statements
```

See the `.catch` documentation I linked above for an example of how to create your own error type (in those docs it's called `MyCustomError`). Be sure to read it carefully, I'm expecting you'll just jump to first codeblock and pasta it, which will work but will be wrong.


You could've also `return template(this)` instead of placing it in a new variable 'html'


Same here.

Notice that you could (and probably should, for context) attach the original error to your own error type. Simply add another parameter to your error's constructor and attach it to `this`, so you can do something like:

``` js
throw new SteamVanityResolveError(err);
```

And then in the handler, if you would want to (even if you don't currently; the point is developing for the future), people could access the underlying error with something like `err.originalError`, so they could see that for example `err.originalError.code == 'ETIMEDOUT'`.


Sharing codes to Sherilyn, once copied this branch would be deleted.


Hi suborb,
What is the status of the satip plugin? Does it work with the 1.7.x vdr? Do you think it works with the stable vdr 1.7.21.6?

See my contribution to minisatip 0498c21aff787ec1337bcb06b7a8b076fa4a4cef

Cheers


No, it's `semver-major` and is a breaking change on `util._extend(undefined)`.
Please revert this change.


fixed


Yes


This Handlebars function allows you to return what you pass into the appTemplate variable, avoiding the need to use a for loop.


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


[This](https://gist.github.com/justmoon/15511f92e5216fa2624b) is a nice concise guide for creating your own error, if you can't understand the `catch` docs at first. The first code block there. Don't just copy paste, notice that `extra` is an example; you don't have to have it (though you can change that to e.g. `originalError`; point is you can add additional properties to your own error type).


Great!  It would also be nice to add a constructor for the new SoftBreak element, and update the examples so they still work.

And we should probably have a large version bump and make it clear in the README which versions of pandoc each version of pandoctypes works with.


По-хорошему, тут бы ещё количество аргументов проверить, но да бог с ним.


You don't really need `true ==`


(кстати, self в таких случаях не нужен)


narf


Looks good!
Out of curiosity, does the speed actually matter on your usage, or was it just for fun? :)


The Handlebars.compile function allows you to `return compileTemplate(this)` to return your template


Ach, pardon. Tak to je moje vina!  Když jsem dopisoval to "vypínání", tak se tam tahle chyba vloudila.

Mariáne, když se něco takového stane - rovnou tvoř issue :-)
Walley, vytvořil jsem #23. Jestli se na to podíváš, tak si ho prosím assigni na sebe.


Разбираюсь в гите, вот и создаю ветку на каждый модуль с мастера - кажется это верное решение. нет?


Programlama Dilleri


I like the new UI <3


var functionsDef = new Array[BaseElemDef](0)
          funcList.foreach(fun =>{
            functionsDef =functionsDef :+ fun
          })

Constructs like this should be done with map functionality.


@techee why not, it would indeed yield almost the same result.  I don't really mind either way so long as the code is solid enough not to crash sometime in the future :)


Note that this shouldn't be used right now unless you have a machine to
yourself. Jiwei (and possibly others) are running jobs that take a long
time (12h+) to start, and that die if they run out of GPU memory. Plus,
they regularly allocate and free memory, so they may wind up needing more
than nvidia_smi says they do. So, without either manual inspection or a
smarter PBS, there's no way to be sure that you won't wind up on the same
GPU as one of those jobs and kill it.

On Sat, Jan 9, 2016 at 3:07 PM Will Monroe <notifications@github.com> wrote:

> Add automatic GPU picking
>
> Thanks to Sam Bowman ( @sleepinyourhat ) for an earlier version
> of pick_gpu.py:
> <https://github.com/hans/rembed/blob/master/pick_gpu.py>
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/futurulus/coop-nets/commit/a9e05c821dc4cdce185621281901d546e38585c5
>


You can also pass `$('#blog-post')` right into the Handlebars.compile function to save a line.


jlskdjfa


Parfai Kolvasna je copie le tout du view file et je  vais coller et tester


// Change the login email - FIXME: Shouldn't this be in the auth section?


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


new chat created *IF does not exist :D


You can also simplify to one line with `var converter = Handlebars.compile($('#article-template').text());`


ack


Can also simplify to `return converter(this);`


separate


Díky. Na návštěvnost jsem se dostal. Před červnem průměr 40 návštěv za den, potom 30.

Což odpovídá - já mám od června průměr 10 :-)  .. akorát samozřejmě peaky kolem těch školení a konferncí.


submitting


submitted


ass all tests


pull reqested


I know that Michael S. would like the command-line switches implemented which put this on the to-do list for some time. I used argv simply because I didn't know any better. argparse looks like something more sustainable and flexible. Let me try out some things with it first if you don't mind.

When you say 'raw' option, what do you mean? Can you give me an example?


pull requested


@NikoKrause
For given situation: option 2
As for the translation: I'll think about it.
By the way CinnXP is a paradox: latest gtk + .scss + vintage theme (I like it though)
So when do you think you can finalize your version of stark? It was initially conceived to emulate Win7 menu not XP menu. Wouldn't you like to try out a Win7 theme? I can provide a link to a really nice implementation + really nice bootsplash.


@michaelklishin, I would like to ask for further clarification, if I may regarding the following excerpts from your prior comments here:

> Other synchronous protocol methods must be responded to in the same order they were received on a channel

and

> @awelzel what you describe is pipelining and for synchronous protocol methods it is NOT something that RabbitMQ guarantees is going to work.

A `nowait=False` method is considered "synchronous" in the context of the excerpts, right? So, if (per first excerpt) they must be responded to in the same order they were received on a channel, then I think that a client library should be able to send multiple of such synchronous methods (`nowait=False`) back-to-back on the same channel, then receive the corresponding responses in the exact same order as the requests, and use the order to match incoming responses with the original synchronous requests. I am inferring from the first excerpt that this should work.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1142](http://teamcity.monogame.net/viewLog.html?buildId=13483&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:27:32


thanks


:cool:


You can use the defconfig flag CONFIG_MSM_CPU_FREQ_MAX to make it device-independent.  Might be useful for overclocking by default ;)


yeah I saw smart people use XXX in place of TODO and FIXME so I'm using it some. It's lower priority


My naming of the module was a bit confusing... The `SYMBOL` signature is meant to be used for all symbol-like things, including symbols, variables and metavariables. The symbol-variable distinction manifests only in the ABT area of the library.


oops, thank you!


that particular one was handled a few commits after though


104923612_EnhanceReportingToGithub (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


USELESS???


the composer vendor folder should be added to gitignore


Meh, we could but meh

On Sat, Jan 9, 2016 at 5:28 PM, Dan Cartier <notifications@github.com>
wrote:

> You can use the defconfig flag CONFIG_MSM_CPU_FREQ_MAX to make it
> device-independent. Might be useful for overclocking by default ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CandyDevices/kernel_samsung_msm8660-common/commit/701bc222e72d7568600c76436a596e3869cb82a8#commitcomment-15357266>
> .
>


@jorgenschaefer Is this message intentional or just a left-over from debugging?


Sure, no problem. As for "raw" I mean just sending out 35 instead of trying to look up 35 and determining it to be "Quark", or sending out "None" rather than blank, or "Unknown" for any device type not in the built-in dictionary. I used a patched version of the Driv3rs script to dump those raw numbers out for what I already ingested into the D3I driver list as a test (so, the version number is abbreviated there, manufacturer is just a number, comment fields can be entirely blank). It may be less necessary to have access to the raw data once the lookup dictionaries are more mature, but at the moment it's better not to put everything "unknown" into the same undifferentiated pile I think. I do understand that part of the fun of the script is having it do those lookups, but for the initial sleuthing while we're trying to figure out what goes in the lookup tables, it'll be useful to be able to see the underlying numbers.


fixed


@jonsterling This seems like a good place for this then, people will still see it when trying to use ABTs even if it's a bit nonlinear


Put the mixins before NodeNG.


In PyGObject implementing an interface vfunc is done using "do_${vfunc_name}". So this should be "do_set_background".


Whops yeah. Though seems like it's different from file to file. I'll change this to conform to this file at least


Perfect!


It _should_ be compatible with all the actual usages of it, but that's a more than fair request. I'll revert that.


Dreams About How You are Expressing Yoursel


I'm not sure what you mean about building the lessons. The notes/slides I put there aren't supposed to be part of the site - are you trying to incorporate the `bash` and `Python` lessons?


You know... I used "Unknown"  or "None" and Michael S. suggested I just insert the raw data into the CSV. I agreed with him, but apparently didn't do that. I think it's more useful to have the raw values there for various reasons.

If you want to, go ahead and change the script so it puts the raw values in the CSV now. Later, if we want it to be an option, we can make it an option with argparse.

What do you think?


Mistakenly pushed. See c26357a06cd2c5f8c0b215c0a4b74226f18955ee.


what's this IP address?  your private openHAB server?


@flibitijibibo does this mean you are now linking mono into the kick binaries?


@soyuka If I call process.setuid inside my code then that line gets into source control. The app might be deployed on different environments, one has www-data the other has www. This is something I would like to from the _configuration_ side, not from the _code_.


I was just snagged by this. Can you update pypi please?


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


This was pretty freaky. Github Desktop and Intellij IDEA both failed to commit this, I had to resort to the *shudder* command line. I have no idea how a directory got duplicated, a folder concatenated to another, and delimited with a Latin ASCII character.


Is there such a thing as a built-in IsNumeric() function?


P.S. I adjusted the csv.write section that writes out the csv header line. I noticed that using the \ to break the long line up in the editor was introducing a space on the very next header added. So, "dib_start" might be " dib_start." I just changed it to a concatenating line. Works now.


Fixes #1112


what does this comment do to prevent code warnings, and where are these code warnings coming from?


likewise, what does this comment do to prevent which code warnings?


Possibly fixing #3


Over-indenting. Indent 2 spaces. smh


excessive whitespace


Ah, is this something coming from running jslint over the code?  It doesn't know about the pebblejs globals?


id suggest `'!**/*.{ttf,woff,woff2,eot,otf,svg}',`


This change also implemented a routine for Archons to build soldiers in random positions around them.


- [ ] I would say that we annotate things so that we can query what the sort of a term without a context.


:+1: :heavy_plus_sign:


I think SVGs should be covered by the images rule on the next line, but `otf` sounds like a good addition.


Note: new cryptominisat is LGPLv2 to make it more compatible. So saying LGPLv3 is not a problem (LGPLv2 is compatible with v3) but CryptoMiniSat is more lenient now :)


oh hah, my bad


i wonder if its worth adding psd to the images list


Whats needed to be done on this feature to make it safe, is that files that is deleted will be committed in the vfs if the master crashes or if slave goes offline before the change is committed


Would it be more efficient to just merge those two lines instead? I'm not sure how the matching works exactly.


Idk whats up with the hbs~ file. If thats uploaded it shouldnt be, thats the old one.


2016-01-09 upload final ticket counter program
Project Name TicketOrderingYang


geography (macumber) - i386-Windows-7-VisualStudio-12: OK (2350 of 2385 tests passed)

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


good catch, thanks! Integrating now :)


- [ ] When I read `description of sorts`, I was thinking of "sorts" in the technical sense; maybe we could rephrase this?
- [ ] `descripes => describes`