Because `ConditionTimedOut` is a `SignalException` ( `Celluloid::Interruption` ) whereas `ConditionError` is a `StandardError` ( `Celluloid::Error` ) ... unlike `Task` where only `Celluloid::Interruption` is effected immediately, `Condition` effects both... so it needs both, since they diverged.


굳이 toString() 메서드를 오버라이드할 필요가 있나요? 다른 메서드 이름으로 제공하는건 어떨지


@luckyh

It's better to use "r" prefix to tell  re module, kernel_re params is raw string, so that it's no need to use slash to convert it.

thanks,
Xu


lol ecks dee gaise MEMES


Is there some reason why they need to be different? Who raises these exceptions and where?


r+


gjioergjitro


how would i change "regionthree.dat" to "TechHacks.dat"


saw approval from graydon
at https://github.com/jedmccaleb/stellar-core/commit/cbb232a15e7a8da29e520ef415e0146f9c8ece45


merging jedmccaleb/stellar-core/master = cbb232a1 into auto


jedmccaleb/stellar-core/master = cbb232a1 merged ok, testing candidate = f1ea3554


93651760-RowPerObjectReport (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1272 of 1272 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-69.98%25-green.svg)


You could mark object as nonnull here, then compiler will error if anyone explicitly passes nil (which doesn't seem to be a valid use case)


Is there a time-out for PLL 1 before power down and if so, Is this adjustable?
My understanding is that it's just XOR'd if I read this correctly...


do not need use @synchronized(self) for that line


Sure, can make a helper function. Dunno how often we'd use it, but doesn't hurt.


need @synchronized(_date) only


need @synchronized(_IPAddress) only


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


need @synchronized(_changed) only


i think this commit itself does not solve the multiarch problem, but make the simple usecase broken.


The Drawer is initialized two times. Please, remove this line.


This was the right step for issue #11, however this change has broken python3 support for me because *stdout* is a binary stream (type **bytes**). Therefor, when we call *stdout.split()* in the line below, it raises a **TypeError**:
```
  File "..[truncated for brevity]../lib/python3.4/site-packages/svn/common.py", line 51, in run_command
    return stdout if combine is True else stdout.split("\n")
TypeError: Type str doesn't support the buffer API
```

I did a little research and I think we should call the **Popen** constructor with `universal_newlines=True` as described in these links (you have to read a bit to find the mention of the *universal_newlines* parameter):
* https://docs.python.org/3/library/subprocess.html#popen-constructor
* https://docs.python.org/3/library/subprocess.html#frequently-used-arguments

According to the docs, this should result in *stdout* being a text stream rather than a binary stream. I think this will be better because we will parse the results as text (rather than binary).

I can try to make a pull request for that in a few hours...


Thanks for the feedback. I actually added this example since I was not entirely sure where to place the ``gem jekyll`` and ``gem html-proofer`` lines. But now that I see this, it's pretty obvious actually. I'm still a Ruby newbie.


[bcryptjs](https://www.npmjs.com/package/bcryptjs) is so much slower. The JavaScript event loop will get overloaded during registration more easily, compared to using [bcrypt](https://npmjs.org/package/bcrypt) itself.


Thank you!


Thanks for the wording pointer. Will change this.


不知道DOM里还有个contains方法, 代码瞬间变长了 :(


Ok, gotcha.


Форматирование кода разное во всех файлах. Если будешь работать в команде — нужно единообразие. CSScomb в помощь!


write


Actually, the target `make parser` in `FStar/src/ocaml-output` copies the file from `src/support/ocaml/fstar-lib/` into this one.


+ 403 Forbidden


Ok, I willl rephrase and add in the advice not to check in the ``Gemfile.lock`` file.
:question: Should we add how to add an entry to  .gitignore, or will this take us too far?


@shadowfacts It's @coolsquid ... you can't change him...


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


yeah, i agree. it's far from a bottle neck. but, in short:

- syncfs is supported in recent kernels.
- this introduces more complexity.
- it has another condition statement in io path. (feel free to ignore this)

do these arguments make sense to you?

> I can remove this flag and directly copy all code of sync_filesystem.(do the compile work for inline function).

iiuc, the idea of having `sync_filesystem.h` is to hide system/platform specific details from the FileStore. that's also why we wrap some other system specific calls with their wrappers, among other things, `fgetxattr()` versus `ceph_os_fgetxattr()`.

yes, i understand, it's pretty much just my personal understanding. we should certainly see other opinions.


По id не стилизуют, это плохая практика.


False comment


Это тоже лишние стили, в продакшне всё должно быть чисто.


Can you remove this line


It's weird..
Storage location is /storage/emulated/0/OTAUpdates.
Addons are downloading properly to /storage/emulated/0/
but ROM is downloading to /sdcard/ + /storage/emulated/0/


По моему лучше хранить шрифты у себя, а не полагаться на cdn-ы.


프로젝트 모두 업데이트 완료!!


@jing3018
I believe you mean line 1934 and 1936? Could you post a .patch file for this?


I don't fully agree with you.
In an array, the order of the elements is fixed (unless you proved me wrong - but then a lot of things would have to be changed). That's the reason I choose an array over a hash. An I'm using this order to place my button automatically in the right order.

If I use a hash, the keys won't come in any precise order, so I'll need to hard-code the position to be sure they are at the right place, or precise that I want to treat the key `character`, then `inventory`, then ... so I am not sure it's better.


I was testing with `rbx` and `mri` is still hosed... sigh.


Как-то странно выборочно префиксованы свойства. Свобйоста флексбокса, например, не префиксованы, а дальше по коду есть префиксы. Autoprefixer в помощь.


x


No need, I'll fix this before leaving.


you mean *"at least one* **logical** *operator"* ?


@AkamaiDASH that's great with me.

@KarishmaBagga I don't agree that using the chunk type is the correct solution since the real question is what is the underlying type of buffer. There needs to be a way of differentiating whether it is the native version or ours. Now the function signature no longer matches for our source buffer this whole area seems pretty suspect.


Its designed to only trigger once for multiple changes on a single run of the runloop, I only ever use it on the main thread. But if I made that BOOL check atomic it should be thread safe yes? But then we get into what if I want the block to be called on the background! I would say, its like a completion, I have a rule with completions, always call back on main thread. Its then up to the user to send heavy work to the background.


Great! (and happy holidays;))


В продакшне стили нужно минифицировать и склеить в один файл. Тут надобно разобраться с системами сборки на выбор: grunt, gulp или лоадером webpack, например.


There is already method called `getFilename` in this class. They do not conflict (different scope) but for sake of readability it would probably be better if you have given the new method some other name.
And how about making a private method of `InputHandler` (as all other methods are done), not a subfunction in `_extractFilename`?


repo isn't with boilerplate currently, that's why i add istanbul, because the `use chorks/jscss` commit.


Вот за это мне не нравятся икончные шрифты. Несемантично )


doesn't the above call to addReplyError means that with the previous code, we return reply twice?
actually, i can't find the place where the first reply is returned, but in any case it seems that some paths return reply and some don't.
or maybe i'm missing something.


Oh. Either `call` or `[event]` here.


<!-- PMABOT:SOB -->
This commit is missing Signed-Off-By line to indicate that you agree with phpMyAdmin Developer's Certificate of Origin. Please check [contributing documentation](https://github.com/phpmyadmin/phpmyadmin/blob/master/CONTRIBUTING.md) for more information.


Can you please add this to your documentation string? Consider adding to the other API endpoints as well.

Thanks!


Using this method of initialising variables is fine so this patch looks fine to me.


Using a constructor is indeed the next step. And I will tend to this if (or when) I need an API with several functions.

Indeed I have now to hard-code the string `"Character"`, but you're doing the same for the constructor, so that semi-issue is no real argument :-)


Breaking out the main rotor and tail ramp code to this separate class is a nice way to reduce the complexity in the main heli class and also leads to some reuse of code even though the tail doesn't use about half of the features of the main rotor.

Note that our coding style normally doesn't include a space between the function name and the brackets.  I.e. "output_armed ()" should be "output_armed()".


```// Timeout is a hack to ensure they are loaded in that order. Sometimes they would load arbitrarily, causing issues.
+		setTimeout(function(){Loader.load('script/variables.js');},50);
+		setTimeout(function(){Loader.load('script/loading.js');},100);
+		setTimeout(function(){Loader.load('script/jquery.timer.js');},150);
+		setTimeout(function(){Loader.load('script/ui.js');},200);
+		setTimeout(function(){Loader.load('script/sound.js');},250);
+		setTimeout(function(){Loader.load('script/ai.js');},300);
+		setTimeout(function(){Loader.load('script/save.js');},350);
+		setTimeout(function(){Loader.load('script/animation.js');},400);
+		setTimeout(function(){Loader.load('script/main.js');},450);
```
Why `setTimeout`?
You should do it this way:
```
Loader.load('script/variables.js', function() {
    Loader.load('script/loading.js', function() {
        Loader.load('script/jquery.timer.js ... etc
    });
});
```


`trove` in OP should actually be `trv2`. Thanks, autocorrect.


Personally I think that's a story for another day much like the dependency locking.


Скрипты то надо сливать в один и минифицировать. За кучу запросов тебя никто не похвалит ) Но для разработки норм.


@prafullpcsx2
Do you have the 4GB patch/option? Or are you limited to 2GB of memory

@gabest11
I would like your opinion on the code. It is a bit hack-ish but it seems to do the job. However it is still strange that we need to flush nearly all input textures to keep nearly a single frame. Maybe others data consume too much memory.


I'm not sure this really improves the readability of the code.  It just ends up spreading out the initialisation so I'd probably prefer if we skip this unless there's some good reason to include it.


There is a [build failure](https://registry.hub.docker.com/u/cgal/testsuite-docker/build_id/67256/code/b27vjbfh4sp6ww4pkipnww4/) of ArchLinux, related to ipe.

Do you receive mails from Docker Hub, by the way?


Форматирование прыгает


What is the purpose of viewStack ?


Looks like the type for this should be [`sorter()`]


instead of naming this function, "output_yaw" we should consider maybe "move_tail".  It's a part of the "move_swash" function, but it's name and placement make it appear like it's a peer of "output_disarmed()".


Upper case record fields hurts my eyes, is the QL case sensitive?


@rmackay9 The purpose of this change is so that the subclasses to AP_MotorsHeli can define their own reset_servos methods, that calls reset_swash_servo according to the specific motor implementation.


Вот эта обёртка `$(function…)` — это сокращённая запись  `$(document).ready(function(){`. В этом месте она не нужна.


Minor thing, we shouldn't have the "-" between the "0" and "1000" because it'll break the parameter description parser.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/74195056


@Klap-in This commit breaks the sitemap link for some reason (see current DokuWiki.org). What was "bds" and why does it work and "index" doesn't?


Removed spaces in line 16 and 17.


Where is `it` defined?


Hello world, i'm new on the this platform, but i'd like to improve more, i hope you can help me, thanks.


@rmackay9 Using yaw instead of tail in the name is based on that the compound implementation with two side-by-side rotors override this method.


Thanks for notifying me. I tried to activate email notifications but had no luck so far. I'll have a look at the interface again.


this seems to be related to #1278


Thanks, yes. (Actually I've started going through and fixing all these to use Python3 syntax which differs when there's a message to be retrieved)


I add an extra test case  ```test_swap_SharedVaraile_with_given()```, which is not very rigorous.
Please have a look and tell me whether it reflect the case in DLT logistic_sgd.py. I will improve it if your think this is ok.
For the test case of In with default value, it's tested in the original case ```test_copy()```


Ahh never mind. Your commit *fixes* it. I thought the wrong way round.


@rmackay9 Could I suggest that we rename the move swash method instead?


Drawer -> tab
Please check the debug issue


Ditto @borisrorsvort


Sorry for that, but argument used to have `$` prefix (`$content`), so I wouldn't expect anyone passing string there.
Is it difficult to fix it? It's super easy to restructure this function too, so it still works with strings. Just:
```javascript
var $content = $('<div>').append(content);
```
instead of:
```javascript
var $content = $(content);
```
I'm fine with both approaches, up to you, as I'm not familiar with sensor use case.


@rmackay9 Thanks. I'll do a separate formatting commit for the entire PR and squash it in once the review is finished.


Space between cast type and variable being cast.


Hmm, `dir.replace(/\\/g, '/');` converted the paths to `foo//bar//unicorn`, and that's what the problem was from what I can tell.


fast-forwarding master to auto = f1ea3554


@rmackay9 My bad, fixing.


Hello ,

I am sorry for my French spelling.
your project and really beautiful but I have a problem concerning the configuration.
How should I use JavaScript loader for it to work because it does not work for me
I am available at cedric.cecilia@yahoo.fr

Cordialy


Good to see you're moving psychopy to Python 3! But I encountered this issue already in Python 2.7. I think the old syntax only applied up to Python 2.5.


Translations for zh-rCN


we use err


I doubt this is gonna work properly. Once the settings file is loaded once these settings are saved to the addon_data folder. Say you want to change the API because of whatever reason it will still fetch the key from the addon_data and not from the updated settings.xml
(untested to know for sure)

Best would be to just have it somewhere in the code as variable imo


Hi. Are these the only changes needed to make dual sim work? (together with the patch for android_packages_services_Telephony). Thanks.


This would be more understandable to me if the name made it more obvious that it was an assert, like `assertTokensEqual`.


В данном случае $(touch) не нужно оборачивать в $(), т.к. в переменной touch уже лежит jquery-объект.


On line 1063, why did you clear the default value for 'index'? Clearly, on line 1076 it is being used as the index of an array (where undefined is evaluated as 0), but on line 1073 it is not so clear-cut. If self._onendTimer is an object, that object could conceivably have a property called undefined, and another property called 0.

I realize this is an edge case, but it was not a problem before, was it? Is there something I'm missing?


Магичесткие числа не есть хорошо. :boom:


http://www.thefreedictionary.com/interrogator

in·ter·ro·gate  (ĭn-tĕr′ə-gāt′)
2. *Computers* To transmit a signal for setting off an appropriate response.

Alas the same dictionary also defines *interrogator* as "one who conducts an official inquiry, usually with no regard for human rights" — so there's a negative implication, granted.


Не понимаю, что делает этот код.


Checkmarx scan results 2015-08-05T09:14:41Z

High
Medium
Low
Info

New
1
1
2
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud

ad58b128e2fede39820ae668970bbfab38b56195	@cxehud

6b05867d4fe80a4c3b55cb36e0b6f76abc0a59f9	@cxehud

accf7e51faac6fffc26f426e5afeeaf3ed561731	@cxehud

7917c1e1165bbb3f8b67a7c2943b68d3780e2a79	@cxehud

74262a0e47571dc6ed4aa4c3a03f759fee5bd42c	@cxehud

5b96953244ae55ea53f4983dc1a9fa4d4eb872a6	@cxehud

2166d1ea7164e801468b1b1f20ff0a3ecf4c67dd	@cxehud


Что из JS-кода бутстрапа у тебя используется?


Есть хороший тег <nav> для меню.


I have to admit that I've lost track of translation handling.


Yes, I don't remember a time when this particular error would have been correct! I think when an AssertionError occurred it would have been passed to the block as variable `ImportError` and an actual ImportError would have been ignored! Pretty hard bug to find!


@NedScott feel free to do a PR with added itunes library. I guess you would try it out before doing so - right? ^^


Hi. It would be more helpful if you could ask on the main tech blog article: https://techblog.badoo.com/blog/2015/07/22/jacoco-coverage-in-cucumber-on-android/ because the question is probably not about this patch. I'll respond to you there.


:clap: This is pretty clever!


test


Такие селекторы — обычно это избыточно. Чем просто `.massage` хуже, чем `textarea.message`?


fsdfsdf


this could've been on one line like [here](https://github.com/ironsmile/nedomi/blob/fe2c5e8b3382ea0a50ec67d43e1a6e6ebeb71e86/main.go#L91)


Fixed in https://github.com/meetecho/janus-gateway/commit/3a3cc0543effd53627217f165a6a9e710344a1b8


Презентационные названия кнопок нежелательны. ЛУчше называть по смыслу. Типа `btn-primary`, т.к. кнопка может стать не красной и название класса станет неверным.


Поля ввода надобно оборачивать в форму!


It was done by mistake. Has been fixed.


yep... fixing :D


another (and faster way) of filling a list is http://mirrors.kodi.tv/docs/python-docs/14.x-helix/xbmcplugin.html#-addDirectoryItems
addDirectoryItem -> addDirectoryItems

In case of small lists there's little speed gain however once lists grow it could save some time building the lists (i've seen massive speed boost when doing some stuff years back on PseudoTV which has a huge list)


@NedScott ahh sorry - just have read your second slack notification. Well no luck for you - preflight is not used on ios ...


Hmm, lol, that's true.


@axelaris Out of interest; if this exception gets thrown where does the user "see" it?


but does not hurt.


You're right, never mind.


whats that


Плохой коммит. /system журнал не нужен, т.к. она всегда в RO. А журнал место драгоценное занимает


У кнопок лучше писать type="submit", чтобы они работали в форме по умолчанию.


OpenMVG


Useless method, same thing in destination


please remove the first "introducing-" here – the permalink should start with the date:)


Блок секции подразумевает обычно заголовок в этой секции. Без заголовка — это уже скорее всего не секция.


this is a minor change, but I think you need to remove the spaces between [] and (), otherwise the link doesn't show up properly in Jekyll!


:+1:

Is there a good reason for avoiding SW1A 1AA? Is it a special case somehow in constituencies?


There is actually no need for contiguous camera IDs. MVE should be able to handle "holes" in the IDs.


missing package-info in ` org.sonar.api.ce.measure`


我之前以为这块空的 block 是遗留的空白代码块，所以就移除了。
请问这段的作用是什么？没找到相关资料。

另外，我看 archives 页面的年份也在……


That was indeed the case. I only spotted it because after reinstalling my system I no longer have pyo (couldn't be bothered to compile it under Linux specially for my conda Python), thus the `ImportError` should have been caught but it wasn't.


how do I know the value type of `Measure`? there is no `getType` defined in the interface


Зачем это чудо? )) Рамку же можно добавить и по фокусу на поля ввода.


Check if folder has been created?


Избыточный селектор. `.mixitup li` короче )


warning: unused parameter


and here


In general we shouldn't move the eeprom location of variables because it leads to corruption without any warning to users.  Normally we allocate a new eeprom location and simply stop using the old one.


NPE if arg is `null`? this deserve to be documented


Unused variable?


Названия классов должны быть осмысленными. `bg` – не очень хорошее название. Из названи непонятно что это и зачем.


Risky. May override the current geocoding job.


what happens if I pass a `null` in the String array?
can I call it without any argument?


No. Images can be of arbitrary size. Remove this.


"instantiated"


想有一套好的开发学习路线，目前上班没有老员工带。学习起来非常困难。


Do you really need it ? As you get a measure with context.getMeasure(String metricKey), you should know the type of the measure.


indeed


Risky. May override the current geocoding job. We talk about this.


Checkmarx scan results 2015-08-05T09:29:30Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
1
1
2
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud

ad58b128e2fede39820ae668970bbfab38b56195	@cxehud

6b05867d4fe80a4c3b55cb36e0b6f76abc0a59f9	@cxehud

accf7e51faac6fffc26f426e5afeeaf3ed561731	@cxehud

7917c1e1165bbb3f8b67a7c2943b68d3780e2a79	@cxehud

74262a0e47571dc6ed4aa4c3a03f759fee5bd42c	@cxehud

5b96953244ae55ea53f4983dc1a9fa4d4eb872a6	@cxehud

2166d1ea7164e801468b1b1f20ff0a3ecf4c67dd	@cxehud


Ok, so this is the big change so @Rob-Lefebvre what's your opinion on this?  Are you happy with the rename of Heli to Heli_Single?

The need to push the ramp-time, runup-time and critical speeds from the params held in the main heli class down into the RSC instances is a bit of a drag but can't be helped I think.


missing javadoc


hehe


And I don't understand the description. By definition, an interface can **not** be instantiated


Избыточный каскад увеличивает специфичность правил. Чем правила специфичнее, тем сложнее будет в будущем поддерживать такой код. По возможности используй как можно более простые и плоские правила.


Both PNG and JPG are supported. In particular, you want to also copy JPG files to save memory.


Для автоматизации работы библиотеки необходимо реализовать метод, который печатает информацию про каждое издание, которое находится в переданном каталоге.

На выходе ожидается конкатенация печатного вида всех изданий. Если количество элементов массива  = 0  вернуть пустую строку "".

(Примечание: В консоль писать нельзя ! ).

Original code:

class Issue {
    private String name;
    private int countPages;

    public Issue(String name, int countPages) {
        this.name = name;
        this.countPages = countPages;
    }

    public String toPrint() {
        return "Issue{" +
                "name=" + name +
                ",countPages=" + countPages +
                "}";
    }

    public int getCountPages() {
        return countPages;
    }

    public String getName(){
        return name;
    }

}

class Journal extends Issue {
    private String yearJournal;
    private String numberJournal;

    public Journal(String nameJournal, int countPages, String yearJournal, String numberJournal) {
        super(nameJournal, countPages);
        this.yearJournal = yearJournal;
        this.numberJournal = numberJournal;
    }

    @Override
    public String toPrint() {
        return "Journal{" +
                "name=" + getName() +
                ",countPages=" + getCountPages() +
                ",year=" + yearJournal +
                ",number=" + numberJournal +
                "}";
    }
}

class Book extends Issue {

    private String authorBook;

    public Book(String name, int countPages, String authorBook) {
        super(name, countPages);
        this.authorBook = authorBook;
    }

    @Override
    public String toPrint() {
        return "Book{" +
                "name=" + getName() +
                ",countPages=" + getCountPages() +
                ",author=" + authorBook +
                "}";
    }
}

public class Library {

    public String printCatalog(Issue[] catalog) {
              /*BODY*/
    }
}


=> implement


documentation: context specific to the computation of the measure(s) of a given component


Don't like it, if we must make more room, then do it properly. I'm downloading the game, going to report in when I got it. Do I have to look for any particular scene to trigger this memory pressure?


"Return**s**"


Ещё раз скажу про автопрефиксер. Он позволяет тебе не заморачиваться с префиксами. Например, для флексбокса у тебя не было префиксов, а для box-shadow, который поддерживается уже везде префкисы есть )


Cannot be the same model as destination import ?


"is not list**ed**"


Does this declaration work for all files / modules in the crate?


It's good )))

I didn't make request yet, cause testing next modules in ShMcK demos. And found more errors with router module examples.

And more, when look at examples code in VSCode, intellisense can't find modules without ".ng":
import {PartiesCmp} from 'client/parties/parties'; -- error in VSCode, but working on client
import {PartiesCmp} from 'client/parties/parties.ng'; -- work in VSCode, but error on client


Аяяй так делать! Контент же может обрезаться из-за такого правила!


Fair point! Implicit ordering is quite useful in that perspective (position of the menu entries). It's still a bit brittle to rely on the ordering IMO, but as long as you know what you're doing in the code, it's fine.


aprendiendo a usar github


Can I know your comment on this?


Maybe have these variables like tvshowtitle,status and cast somewhere on top in the file and insert it here?


This a given instance of `MeasureComputerImplementation` is tighly tied to a given instance of `MeasureComputer`, it could be renamed to `Implementation` and be an inner interface of `MeasureComputer`. WDYT?


On some file i get this:

456	05.08.2015 10:50:40	WARNING	HOSTER EasybytezCom[5401]: Premium download failed | File size mismatch | Expected file size: 437465907.2 | Downloaded file size: 452969455


I have test this on my Android 5.0 device and Emulator, it works fine. Would you please provide more information? What device did you test on?


Wait, aren't you removing support for `@before` / `@after` here ? (not that I'll be the one to complain...)


There must be an easier way. `P.fill(-1.0)`? Is this Eigen?


I really don't understand - how can ShMcK run examples without errors with netanelgilad:angular2 module.


In addition, documentation of `MeasureComputerImplementation` should emphasis this coupling and explain why (by referring to method `getMeasure` for example)


Префкисы!


Well, actually what I meant by duplication is that you store `"Character"` in the `buttons` variable as well as passing it as an argument. If you were to store that string internally, you wouldn't need it in the `buttons` variable anymore, so duplication would be gone :)


what happens if there is no measure for a child of the component? null? just ignored?


(Can be "null: true")


What is this? The `pixelAspect` is usually `1`. Don't confuse this with the image aspect.


"list**ed**"


@fhedberg, ok but do we need both "reset_servos" and "reset_swash_servo" and similarly "init_servos" and "init_swash_servo"?  We're not overriding those individuals are we?

It's not a big deal either way really but in general, let's keep it simple and not think too far ahead.


@TekkiCommando ikr


No. no add this table.


"if a measure for the specified metric already exist for the current component"


@maropu @davies looks like this line doesn't compile
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/39848/console


If `create_preview_form` is not used outside `management.py`, I'd keep it in here. If you plan to use it outside, please ignore my comment :smiley:


The component also has "param" property. The same parameter object can be contained in more than one component. For example, parameter with name "Territory" can be contained in ParameterPanelComponent, TextComponent, SelectComponent at the same time. Probably we should set new parameter object to all according existing components.


Nice web!


So do I :)
Looks like you're switching some stuff back to RT though ?


no add this table


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: requirements


This seems wrong. Usually this is how it's done:
    const float flen = K(0,0) / static_cast<double>(std::max(cam->w(), cam->h()));


ewww, goto :P


might be wise to add "which measure type will be int" and do the same for each override of `addMeasure`


I'm fixing it now.  It looks like @davies might have removed an import during merge in a way that broke compilation.


https://en.wikipedia.org/wiki/Face/Off =D


why "first"?


If this field is always `1`, I would omit this line and let MVE assume the standard.


typooooooooooooo noooooooooooooooo


It was just that the constituency it's in wasn't in the fixture that's used to fake the results of `get_all_posts`, which is used to check that someone's selected a valid constituency. I'll reword the commit message to explain that.


why are you calling autogen.sh / configure on configure for the cmake based project ?


The translation is NOT the optical center. It is the `t` part of the `[R|t]` world to camera matrix.


@rmackay9 @R-Lefebvre Perhaps AP_MotorsHeli_Traditional is better? :)

I agree that the params situation is bad. Regarding the RSC instances I can't really see how to get around that without a pretty substantial rewrite of AP_Params, but I agree that it would be nice.

From what I've understood of AP_Params, the eeprom indices will have to change as we're now subclassing the motor classes and params arrays, so I thought I might as well reorder them cleanly. I've spoken with @R-Lefebvre previously about the param conversion that @tridge has used before, but I have yet to figure out a good solution for this case.

Perhaps it's not too much to ask of the trad heli crowd (specifically), to re-import your param file for the next major release?


Perhaps use the oscillators from https://github.com/opendsp/osc using `import { sin, tri, sqr } from 'opendsp/osc/1.1.0';`


If you want a more thorough refactoring, you can use [staticMap](http://dlang.org/phobos/std_typetuple.html#.staticMap) within the `RestClientMethods*` mixin template.


See comment at #7676


Are we sure about this? Without at least setting a `min-width`?


"This extension point can be used to register {@link MeasureComputer}(s) that will be able to compute when a batch report is processed by the Compute Engine", WDYT?


invítame a esta carpeta


Isn't this supposed to be the contiguous view ID?


SupSuper: why is the same translation used here as for STR_ALIEN_SURFACE_ATTACK? Mistake or intention?


Checkmarx scan results 2015-08-05T09:41:05Z

High
Medium
Low
Info

New
36
31
37
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

27ad63073e3c2aa12ea263fb4b9914826dc2dfc5	@dudug

c6c10e625db5f31ccfe15e2d7029e07135f69039	@dudug

5f913f602916b1a5751f4be2742e6cf7c70f906c	@dudug

1ba42ffbd647b4b765baf787f9c8ddcb3e72c014	@dudug

cc8fb54ec5c3d0840f90fbebe2f62d90390aaac9	@dudug

e5077af18528bac6d3f8c636353e8923f84d4707	@dudug

8c243f37f0aed799d586c32a5b9400dc4102d020	@dudug

059fc0173cbb8a05fe212a0162a21afb499cd454	@dudug

c4fd57c54da5a74526a4b92ab6b8845736279988	@dudug

d4d93b42ce2890e4e7973d279acd50daead22f8f	@dudug

dfc68329bd02eeca9274805825156ceb8a43deef	@dudug

9a3fdd0be88e1597aab1c0d785182bb0783890c0	@dudug

a8f36e99056cb2126abb9eaee0f87e2a28671a56	@dudug

909ad4704c846df3febb2b65c0929e5530d45613	@dudug

049d7eeb2975dd58ed65adc7c32042c0667d2f14	@dudug

e84bad447da69baaf9314b2b41d8aabd25ac571c	@dudug

41ed5e8e316e2ab7f48cbb5ca3f82843b6e75a85	@dudug

5bd601d6b8fa952106c1afe1a06bd226c0959bfd	@dudug

ef5f142de7c6fcc94a0adb3bd469f7db8f2fbe91	@dudug

d870498c7f84672c6228a965cfa83aeb5eef5904	@dudug

f829b67a2028c7bff706338de1a6766c5918930a	@dudug

26be59b1dc65eaa49cdc9b73d28bcb2fbd074150	@dudug

61447f9f8435f4498b269aa56383aad86e5d1caa	@dudug

fe7fa9367d2ded0a6af660c19dafb0c3688d0aef	@dudug

dd6707de1fda624f4a55a0ad951bc5a998f3cb5d	@dudug

0e60f69d71dae0871b2895fb544c424c6bfd5dae	@dudug

4a9bdd03a01b305137a050e608c1d8cd0a578a35	@dudug

0854a767335332668f16dcb5f79fd2b656d13132	@dudug

d8d128ba0754a6495609517e6a02a5b2c3ea0552	@dudug

ffa140c1dde768825897b73e59364f7895bc0525	@dudug


相当于一个占位符，在 _layout 中定义后，可以在页面里注入特定的代码到这块区域里。


OpenMVG


"defined by another **MeasureComputer**"
you could define in a single plugin two computers for the same measure


I don't speak Arabic, but according to Google Translatate, the old one meant "Yes" and the new one means "Yeah". So this change doesn't seem to be right. But either way the `.lower()` is unneeded as `u'نعم'.lower()` == `u'نعم'`.


The database storage object is now totally decoupled from translating queries to SQL!


can the returned `Set` be empty? what happens if so?


I thought about using it somewhere outside. Probably in places where we will need to present the form to users.


Thumbnails are really useful in UMVE though not required. But I suggest to generate them.


as for the linker issues i think those are a consequence of missing -lprotobuf not libstdc++.

you don't have to change to g++, it should be using it. use VERBOSE=1 (or whatever is the equivalent on windows) to see the build command.

if you have to force a specific linker language, that is available as a property on the target.


Oups, "synchronized" à retirer


would be wise to document that matching on key is case sensitive
isn't a `@CheckForNull` missing? otherwise exception should be documented


The test was failing before. I did the minimum necessary to get it to pass
again.

On Wed, 5 Aug 2015 10:41 am Sebastian Noack <notifications@github.com>
wrote:

> I don't speak Arabic, but according to Google Translatate, the old one
> meant "Yes" and the new one means "Yeah". So this change doesn't seem to be
> right. But either way the .lower() is unneeded as u'نعم'.lower() == u'نعم'
> .
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/judy2k/ish/commit/85da306385ca88a72dbaed0bb00479d697d115a9#commitcomment-12536511>
> .
>


This is also the translation vector `t` in `[R|t]`.


I'd prefer if it was the instantiator (?) decision rather than packed into the struct. I'm also fearing that this message might get printed more than once.


(Avoid french text)


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


So this is what you do at HP?


there's test.compute.TestCaseMixin for these basic tests


Why is --sort a special option? Should not be output sorted all the time?


Remapped view ID?


Still looking, est. looking time 22 minutes :)


Hmmm, not sure why we're using /bin/bash here. But the web interface uses /bin/sh with the scripts, and we should do the same when testing the scripts.


So yeah, tons of comments. We're getting closer to a working exporter!


Missing capitals ?


```
ewww, goto :P
```
For me it is a good solution at midnight ;)

```
do it properly
```
You mean with removeAll?  Anyway only prafull know the game but several of them suffer of those memory issues.


already on line 7


And I'd also expect sorted output when --resolve is specified, i.e. in self.show_resolved_packages().


why not using `@com.google.inject.Singleton` on `fr.alecharp.picshare.http.Session`?


I have done some Arabic

One of the consequences of the 'lower' call is that the letters have become 'disconnected'. Arabic uses different symbols for most letters, depending on whether they are in the start, middle, or end of a word. The left-hand version uses the characters as they would be written on their own, rather than part of a word.

I don't know why the test was failing - as you say, it shouldn't make any difference. But I wanted to get the test to pass, and this worked :)


What about **//stores?** ?


commit on the same branch with the fix


@fhedberg, yes, if we're going to re-order the eeprom, now is the time to do it because as you say, we've had to do it for AC3.3 anyway so only those who have been testing the AC3.3 release candidates would need to be careful to avoid the corruption.

Re the name, yes, I think _Traditional or _Trad or something like that might be best.  I leave that decision to Rob.


I don't know yet, something smarter than IncAge.


I implemented a System.arraycopy method - we used it in the legacy version also.


Done.


Template constraint ?


Should be better:
```
+``--arch <arch>[,<arch>...]``
+    Limit the resulting set only to packages of selected architectures.
```


other tests seem to use `else:` here


There is an explicit mention in the dev docs. http://click.pocoo.org/dev/python3/#unicode-literals


>this this
typo


this won't work: see "[Patch] use mkstemp for linux and osx (v2)" in "Developer's Discussions" ( well it does work but also adds a memory leak) There is also a discussion about this in the "Help wanted" thread in the same forum


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: docs


IIUC, I'll add a `assert(__ctfe)` to those functions.


This number is also invalid, right? So it should be reflected in test name.

BTW: Why accept integers when it cannot be used to represent all valid numbers. Any integer than should throw an exception - InvalidArgument (type)


It's a kotlin thing (http://kotlinlang.org/docs/reference/lambdas.html)

```One other helpful convention is that if a function literal has only one parameter, its declaration may be omitted (along with the ->), and its name will be it```


Looks like its only in the betas, will leave this one for now


@gabest11 : You will get the error as soon as you get ingame. Also HW mode got no colors (may be related?).


indentation


I think that *mangle* is correct in this context https://en.wikipedia.org/wiki/Name_mangling


このコミットはなんでしょう？


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


maybe this should have been part of another PR..


"immediately after each's initialization" was a little confusing. But, I've comeup with the solution, committing it now.


harr harr :D

2015-08-05 11:38 GMT+02:00 Clemens Kaposi <notifications@github.com>:

> https://en.wikipedia.org/wiki/Face/Off =D
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tkaposi/ubercouple/commit/70ca7cd6d6a7b365ed714cb8b06fea5b2bb2d9f9#commitcomment-12536445>
> .
>


The time displayed for 'Start Time' and 'End Time' in the Timer Settings dialog was one hour ahead of what was reported by tvheadend. As a result if you set start time to 17:00 and press save, although tvheadend correctly sees 17:00, what appears in Kodi after refereshing timers (strSummary/start time) is 18:00. As a result every time you press 'save' after initial creation another hour gets added to startTime/endTime.

Yes it isn't strictly related to API 3.0.0, but without it, the changes I made don't work. Happy to split it into a separate PR if you prefer.

**time_t and struct tm explanation**
time_t is number of seconds since 1/1/1970 UTC (http://www.cplusplus.com/reference/ctime/time_t/) although it could be implemented differently. The important point is it always assumes a UTC/GMT reference.
localtime(&t) and gmtime(&t) convert to a tm struct which can be either local or UTC/GMT. (http://www.cplusplus.com/reference/ctime/tm/). localtime() uses the computer's timezone settings to add hours (UTC+1 for Hamburg) and if in the right part of the year (i.e. summer), add an extra hour for daylight savings time.
mktime will use tm_isdst to decide if it should subtract the extra hour on conversion back to to a time_t.
In this particular example, setting tm_isdst = 0 will result in the mktime(tm_time) conversion being out by one hour in the summer (it will work fine in the winter).
"= mktime(gmtime(&t))" effectively does exactly the same as "= t"

**Conclusion**
When converting from time_t to tm to compare times (e.g. end-start) always use gmtime(time_t). This works across dst boundaries (```localtime(1/1/1970)->m_isdst==false``` , ```localtime(26/7/2015)->m_isdst==true``` which is why I found all this out last week ;-).
Use localtime(time_t) when you want to display the result, know what 'day' it is in your location, or (as in this case) if you have a 'seconds since midnight' value w.r.t. local time.


I believe we should check and sanitize the username to avoid having special characters forbidden in usernames.


ここ書き換えるなら新しいマイグレーションを用意しないとダメですね


I don't think that Module::Runtime should be an *optional dependency depending on the activation of an feature*. Instead it should be a *recommended dependency*.
*`Module::Pluggable` working without Module::Runtime* should just be a convenience for users who don't want to install it. That should not be the default.


ok, I shouldn't have used tabs in html/js but here..


@bbcrddave There doesn't seem to be a clean solution either ways. The problem is native version is not same across browsers + relying on function existence is flaky too


@gabest11  read your email please :)


Done.


could be removed


九九乘法表


You could remove "!= null" at line 45. Groovy use truethy assertions


I have a question, in `normalizeData(data || {})`, sometimes the `data` is `undefined`, and why it happened


could be replaced by `Materialize.toast(e.data, 3000, 'red')`


Took a few googles for my brain to understand this.
Best summed up as: If a convenience initializer you add does not call a designated initializer you will get a warning.
Genius


明白了。我这边年份正常显示是因为把动画关了。


That IN is valid and will pass the validation if padded with zeroes from the left (ARES will take it even without them). See the next test.

The point about integers is a valid one, but I'm against rejecting all integers without question. The only problem are INs with zeroes, because PHP will transform that to some random gibberish:

```
$ php -r "var_dump(00216224);"
int(72852)
```

So I'd maybe go and issue `E_USER_WARNING` when integers are passed, because in a majority of cases the result will be correct. Just to warn the user.


better used `'` than `"` in JS


色々つまづいて、行削除したり追加したときの。余計なものが紛れてました。すいません！　本筋とは関係ない部分です。


Looks good.


@fernandog no, skipped is bypassed above this unless manual search. In manual search skipped is allowed to be downloaded.


Sista stycket var bra fomulerat. Och det är nog det bästa vi har att komma med på RQ5


ok. sorry


98318570_RemoveRadianceCheckbox (macumber) - i386-Windows-7-VisualStudio-12: OK (2196 of 2215 tests passed)

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


Yes, from the appearance of the current source code, this change was merged in, but I believe the canonical source code is in CoffeeScript not JS, so this PR didn't get pulled in directly.


GUI or CLI*


Thanks so much Rory, that is awesome.


Actually, forget all this noise.  iptables-restore's man page says it can't read a file, but the source code says it can, since 2000.

https://git.netfilter.org/iptables/commit/iptables-restore.c?id=e6869a8f59d779ff4d5a0984c86d80db70784962


I thought I had followed the mailing list, but I must not have. I don't
know if I have 'expertise' regarding cardboard, but I have spent a lot of
time wrapping things between languages. There is an odd issue of circular
dependence though, most JNI bindings expect the Java program to make calls
against the C/C++ program. We have an odd edge case where we want a Java
program to start  C++ program that loads a library that calls back into the
Java program. There's probably an easy way to do it I'm overlooking, but in
the mean time chat seems to be the solution.

Have you seen the Trinus VR thing though? http://trinusvr.com It's
basically a wrapper for the Oculus API that pushes it wirelessly to an
Android device on the same network. Using something like that, it may be
possible to deploy OSVR desktop apps through Cardboard, to let people who
can only afford the $4 cardboard viewer have a taste of rotational
tracking.

I have yet to try it out though, I imagine the latency is immense.
On Aug 5, 2015 19:27, "Ryan Pavlik" <notifications@github.com> wrote:

> Setting aside for the moment binding method as an implementation detail...
>
> So, I think the solution is actually have to do what you said at first:
> having a plugin that talks through JNI/other binding method back up to
> Cardboard (or conversely have a JNI binding for PluginKit with a small stub
> to load plugins written in Java, but I don't know how this would interact
> with dynamic loading of plugins). This is based on the architecture of
> OSVR, where plugins have a very small, simple (and fairly stable) API
> (PluginKit) that they build against, and they remain ignorant of the rest
> of the system, while the server is responsible for loading these plugins
> (typically dynamically), providing them with the opaque context objects
> they interact with, and running a main server loop that calls the various
> callbacks registered by plugins as needed. (There's a diagram early in the
> "Path Tree" slides on http://osvr.github.io that might give an idea of
> this architecture better than this wall of text)
>
> The cardboard sensor data plugin is something that's pretty interesting to
> me and others in terms of being able to provide an easy-start Android
> experience that doesn't require root or an external tracker, and at the
> same time you have expertise in this area that I don't have. Is there
> anything you need from me or one of my connections that would help you out
> here? (some documentation, voip or IM Q&A/architecture discussion,
> something else?)
>
> If you haven't already, consider joining the OSVR dev mailing list -
> http://osvr.github.io/mailing-lists/ - it's probably the best place to
> continue the technical discussion (there or on a github issue), because
> I'll eventually forget which commit has all the GitHub comments on it and
> lose the email that I followed here :D (and there are others on the mailing
> list who might be able to help, and it's archived for future reflection,
> etc - the typical reasons why open source projects have mailing lists, you
> know.)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OSVR/OSVR-Android-Build/commit/7407c1015389e2af8cac0c27dc7b417a6cd505df#commitcomment-12553974>
> .
>


remove and just mark class as static


we should check for valid input here as well. extensions could call this directly


I meant can this method be tested directly.


i've updated the text, and added to the other tables as well with f991f58cd7ed121bcfb5e137e27a6673a0ab3cfa and fd7ac7742c287227e1bb8d126c93e6a0ba4756cd


we can't do this. that's PHP 5.3+


Do we still have StyleCop in release mode?


I don't understand the point of using alloca. Why not instead use malloc and then register an atexit handler?


Confirmed


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 268](https://teamcity.labkey.org/viewLog.html?buildId=264827&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:09:52
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

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

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

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

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

```


YES!


Can we update to a latest version, such as 6.1.3.  6.0.0 is about 2 years ago.
https://www.nuget.org/packages/EntityFramework/6.1.3


Thank you. I will move this target to Makefile.in.


+1 for mentioning.


Done (pushed https://github.com/grpc/grpc/commit/83b7971c10d18fcc1c04056a028860283b2be051 to this pull request).


this file should just be called "restore.go".


Can we update to a latest version, such as 6.1.3. https://www.nuget.org/packages/EntityFramework/6.1.3


ah the old coffeescript trick. thanks. It turned out I just needed ng-form.


Hey @ericyanLr,

This looks much better.  I think we can use this for now.  One thing we can maybe consider is something like this, but I do not know how accepted of an approach this is since normally we

```The folder name cannot have the following:
* Be blank or a reserved word: null, con, prn, aux, nul, com1, com2, com3, com4, com5, com6, com7, com8, com9, lpt1, lpt2, lpt3, lpt4, lpt5, lpt6, lpt7, lpt8, lpt9.
* End with the following characters: .
* Contain the following invalid characters: \ // : * ? " < > | ../ /..
```


Please use naming of commits as we defined here. http://hydra.nixos.org/build/24422016/download/1/nixpkgs/manual.html#chap-submitting-changes


Were these not working without bind or is this just for best practice?


Remove these two empty lines


Can we make it configurable?

sth like this:

```javascript
filter: config.livereload.filter,
interval: config.livereload.interval || 500
```


FYI: before I saw this PR, I also saw this issue and solved it slightly differently, not sure which was is better:

I figured we shouldn't even return the result from the find_servers method in the RedisCheckAggregate class, so the initialize method becomes:

```
def initialize(redis, check, logger, cluster_name)
    @check  = check
    @redis  = redis
    @logger = logger
    @cluster_name = cluster_name
  end
```

and then reject the result from the returned find_servers array:

```
def find_servers
    # TODO: reimplement using @redis.scan for webscale
    @servers ||= @redis.keys("result:*:#@check").map {|key| key.split(':')[1]}
    @servers.reject {|s| s == @cluster_name }
  end
```


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.465](http://teamcity.monogame.net/viewLog.html?buildId=9443&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:22:55


this can be in the function


Exceptions aren't the only way. There are times when cerr/exit are acceptable, but personally, I almost never do that. The answer to your question is kind of long.

I tend to think like a library developer (i.e., code reuse is always on my mind). If your function calls exit(), that makes it impossible to use in other contexts that might not care about whatever error condition is causing you to want to exit. "But what about contexts where the error is truly unrecoverable?" In those cases, exit() inhibits debugging, use abort() instead. This should be reserved for truly egregious errors--those which you'll want to get a core dump for (exit won't give you a core, abort can).

This commit wasn't so much about exit() or preferring exceptions to checking return values, but rather eliminating return values from functions that have no defined failure path. If a function always returns true, then its return value has no meaning. Returning true/false when there is a defined failure path is a perfectly valid thing to do.


Is this due to Web API depends on json.net 6.0.4?


@Evairfairy I'm a little bit confused... why do we handle banaccount, unbanaccount in worldserver? Should we handle all in worldserver or should we handle account things in logonconsole and char related things in worldserver?


El porque de utilizar Rtcs_Port.h es el de no editar ningun fichero de la biblioteca para portarla a otro firmware, salvo este fichero en cuestion.

El Rtcs_Port.c quedaría vacío (solo copyright y breve documentacion de su uso en doxygen) porque no se necesita para el CIAA Firmware dado que la biblioteca de control es parte del firmware con todo lo que eso implique. Solo se utilizaría si alguien quiere portar la biblioteca de control a otro Firmware.

De esta forma, intento dejar dos alternativas para los potenciales usuarios finales de esta biblioteca:
. Utilizar la biblioteca de control en conjunto con el CIAA Firmware.  El usuario no necesita portar nada (porque ya es parte) y aprovechará todo el know how, el testing involucrado y la integracion continua del proyecto.

. El usuario ya tiene su propio firmware y desea utilizar solamente la biblioteca de control. Para eso, debe editar solo el par ficheros en cuestion. No es recomandable por todo lo dicho, pero sigue vigente uno de los objetivos de la CIAA de no reinventar la rueda a la hora de un desarrollo con embebidos (en este caso para el desarrollo de sistemas de control en tiempo real).

Es solo un punto de vista totalmente discutible. Si les parece lo vemos en la lista.


The intention here was to provide a default port when the user didn't specify any (specifying port 0 should have us keep 0). I've added a comment stating so: https://github.com/grpc/grpc/commit/cceeb515927d72e3cfb24ecce4af89a9eed3b42b


or better, call this save_restore.go and make Save() and Restore() functions that produce and consume []byte.

Save(table Table) []byte
Restore(table Table, data []byte, flush bool)

also define a new TableAll = "" and test for that


Increase the scroll size to 800.   Also change it to only trigger on Fridays.


I'll leave it to you to decide which is easier - writing []byte to a temp file and calling restore on it or adding stdin to util/exec (which is useful, too).


Don, that was woot... Please
```cpp
				if (isSnapshotMode) {
					::LockWindowUpdate(_pPublicInterface->getHSelf());
					MainFileManager->backupCurrentBuffer();
				}
```


I don't have a strong opinion. I don't feel like either name is sufficiently descriptive ¯\\\_(ツ)\_/¯


Bra skrivet!


Suggested alternative: checkdate


Woo :+1:


you have to test if filename is in radar: `if hasattrs(self.Vradar.value, 'filename')`


98812358-IDF-SwimmingPoolSurface-4844 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1757 of 1795 tests passed)

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


Cool! Thanks. Done: https://github.com/grpc/grpc/commit/82fb883bec8eef4f396723d259ceea06fa2771ea


That looks like it should do the same thing assuming that `system` invokes `sh -c` in vimscript.


The Testing repo is now changed to `oct`. It tests both configuration and runtime.
+* [huawei-openlab/ocp-testing](https://github.com/huawei-openlab/ocp-testing) - Testing framework for OCI configuration and runtime


Bra text, lite små fel men sånt som vi kan ta på grovläsningen :)


Hey @jonmak08,

Thanks for your input, I will use this approach for now.
The approach you mentioned would be great, but I'm not sure how to implement that formatting using the language keys without including html code.

Thanks for all your help, I really appreciate it!


Comment these methods - what are they supposed to do.  Does SyncLoop ever return?  What does it mean if it returns?

Next summer some poor student is going to be charged with adding some new implementation of this, and they are going to run 'git blame' and see that YOU wrote no comments and they will curse your github handle.

http://thecodelesscode.com/case/116


Might be threading issues with this calling SendMessageToServer from the Timer thread.


An assertion is worth twenty lines of comment.

```
var _ ProxyProvider = &Proxier{}
```


Question posted on TeX-StackExchange: http://tex.stackexchange.com/questions/258951/position-includepdf-to-top-of-page-when-input-is-of-varied-size?stw=2


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 269](https://teamcity.labkey.org/viewLog.html?buildId=264830&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:48
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

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

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

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

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

```


Ooops, 你的@cards=Card.all 那行的indentation其实还是多了（应该还是2个空格）。不过我take care of it。


Huge improvement to simplify this!


Yes. I want to pull all of this messy traversal logic out of the outlet and have the Router fully own that responsibility.

This will be important for projects like Ionic that want to implement their own outlet, but piggyback on the rest of the component router.

I figured I can tackle that when we come to adding tests for aux routes of aux routes, etc.


The logic of `MaxFreeBalancer` doesn't correspond to this.


Fixing, I meant to doc this from the old Proxier as it is a subset of the methods it provides and the new ProxierIptables mimics it. Sorry!


@jedmao this change was made for next `precompile` step.

I think we need to report it to Gulp, I hate this red line too.

Or maybe we can load `babel` and `babel-core` in npm, but it will be dirty.


yes indeed


It seems you've removed the logic I added in this commit: https://github.com/quantopian/zipline/commit/2c8d5f676d7214ebd55f1d0e8f302fbdf2bd05d2, probably because you initially checked out an older commit and then these updates were lost in a rebase. You ok with me adding this back in?


hua hua huahua huahuahau hauahuahauahuahau


Don't docstrings have to be the first thing in the file?


comment code 1


comment on file 1


Shouldn't Args be Func?


What about changing the interface to `NetAddress getWorker(String host, Map<Long, MasterWorkerInfo> mWorkers, Map<NetAddress, Long> mWorkerAddressToId)`? If `host` is not empty or null, select a worker with certain strategy depending on the implementation for classes implementing `Balancer`, otherwise, select a worker on `host`.


What were you trying to achieve with this change?


Explanation of the bug, for posterity:

> 17:31 <+halfak> Yeah.  So it had to do with re-writing the module.  It's the same problem we have
                when I tried to use function decorators around pickled things.  If you set a name
                in a module to one thing and then change that later, pickle gets confused.
17:32 <+halfak> So, pickle is all like, "I heard there was a function called
                revscoring.languages.english.is_badword_process" and our language utility is like
                "I've never heard of 'is_badword_process'.  I just have 'is_badword'."


+ updates `LICENSE` with that which comes from Github UI


Build files must have same path as on GitHub. So `require('postcss/lib/css-syntax-error')`should continue to work even if we use PostCSS master from GitHub.

I can add `lib/*.js` to `.gitignore`, but why somebody will run `prepublish` on local copy of repo? I will be publish PostCSS by old way: `n gulp build; npm publish ./build`.


What about `LOCAL_FIRST` ?


Missing semicolons at the end of var declarations


Sounds good @ericyanLr!  Thanks!


Missing semicolons after var declarations


@zhangkun83 LGTM


Dug more into template compilation code. It appears that the compiled template is attached to browser’s render tree after every node in template has its properties reflected to DOM, meaning:

```javascript
// 1st `refreshRendering()` call happens with default widget property values
var c1 = register.createElement(“my-widget”);
c1.foo = “Foo”;
// 2nd `refreshRendering()` call with `foo` update
c1.deliver();

...

// c1 is attached to browser's render tree
// After then any changes to c1's DOM may cause re-layout/re-paints
this.appendChild(c1);
```

Therefore above code would just be with multiple calls to `refreshRendering()` causing updates to inert DOM. Such multiple calls won't cause change in browser’s render tree that would cause browser’s re-layouts/re-paints. Avoiding such re-layouts/re-paints is what our “debounced DOM changes with widget property observation” approach is for.

Of course it's (slightly) better if all above can be done with just one `refreshRendering()` call, but I don't see a good way unless above code uses an API that creates widget and mix-in properties at once (`tagConstructor()`?).

Wrt other topics:

* It appears that you think there is a case where my new code throws? Could you elaborate this?
* Browsers with native Custom Element support calls `createdCallback()` right when native `document.createElement()` is called.
  Concerning direct call to `tagConstructor()` (`new MyWidget({foo: bar})`) case with native Custom Element support in browser, `register.creatingElementInTagConstructor` is set before/after call to wrapped `createElement()` which eventually calls browser’s native `document.createElement()`. `delite/Widget#createdCallback()` happens right at then.
  Given `register.creatingElementInTagConstructor` is set, `initializeRendering()` call from `delite/Widget#createdCallback()` won’t happen in this case.


Does this pass PoliCheck?


So, I didn't know this yesterday when we were setting things up, but looks like they (polymer team) have a handy local webserver that plays nicely with bower, which would mean you can move these things back into bower_components and revert the .gitignore.

https://github.com/PolymerLabs/polyserve
https://www.polymer-project.org/1.0/docs/start/reusableelements.html


Suggest for change later: 我们想要url长类似这样 :user_id/decks/:deck_id/ 所以或许需要参考：http://guides.rubyonrails.org/routing.html#nested-resources


Yes, I can see that this test and the following your removed are redundant with ```check_variable_attribute_type``` defined above.


any demos with no position?


I'm guessing this is redundant with what is currently implemented in CF tests.


why?


Done and noted. Also in ProxierIptables of course.


Nuts :)

Thanks for looking into it anyways.


What can you tell me about your freebsd environment that can help me reproduce the issue you experienced?

According to this PR, it will set "PROC_PATH = '/proc'" only after it fails to identify '/proc/uptime' and '/usr/compat/linux/proc/uptime'.  However, it should *not* be pointing to /proc on FreeBSD, as BSD uses an incompatible procfs that MineOS is not compatible with.

Though this page is for the Python-based MineOS, the specific 'linprocfs' area is relevant to your install on FreeBSD even for mineos-node: http://minecraft.codeemo.com/mineoswiki/index.php?title=MineOS_(pkg_add)#linprocfs

Add the linux-compatible /proc filesystem as shown in the wiki and see if the web-ui works as expected.  If /proc/uptime isn't found (and it shouldn't be), that is one of many different functions of the webui that won't work (but have been tested to, under linprocfs)


#464 に対応する修正です(コミットコメントのリンク間違えました)


You know I've already added those wakelock sliders/knobs, right? No idea what happens if you have two of them.

https://github.com/BenzoRom/kernel_moto_shamu/commit/4154b383145fc3d1d2822051e949c3ea72ecab89
https://github.com/BenzoRom/kernel_moto_shamu/commit/1a75a0d1233ac0ac55d948c8c4aade2a4f7c7b65
https://github.com/BenzoRom/kernel_moto_shamu/commit/c70803f480ec9adfeca7dd1e6f497191991c3cbb
https://github.com/BenzoRom/kernel_moto_shamu/commit/31606ca62dc68677bf55f26fbaae433f429eeea2
https://github.com/BenzoRom/kernel_moto_shamu/commit/6ec26cbf44e5607927da82aa2fabf02f5606166d
https://github.com/BenzoRom/kernel_moto_shamu/commit/9f47eb2d59ebc95b2315cba66d182e1a5c5e2a9b
https://github.com/BenzoRom/kernel_moto_shamu/commit/8a55661dfa63becf46b5ad4477b83d73df658651


There is no specific scenario related to this on application side but I found the symptom that you mentioned while I am working on this so I added.


this is always the same, right?


line break


I need to migrate out buggy data, but yeah I should probably register it as an int with custom fields and have it take care of it


setPageTransformerにはtrue渡すべきだった


git bisect tells me this commit caused a bunch of settings-view spec failures (that pass when individually focused) but I'm having a hard time figuring it out.


Hah, I didn't know that. They weren't/aren't showing up in Kernel Auditor so I just assumed they weren't there. This is one of those "just get the guy to shut up" commits and I just assumed they weren't there. Thanks for the heads up.


This is a bad idea because trim isn't well supported.
cf https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/Trim#Browser_compatibility


revert this


cool!


Np, wi-fi can be tweaked by /sys/module/bcmdhd/parameters/wlrx_divide and /sys/module/bcmdhd/parameters/wlctrl_divide. The paths are a little different, so they won't show up in kernel adiutor. Maybe he adds them in the future, too.


I don't think so :(
master can only be used by user and org repos

https://help.github.com/articles/user-organization-and-project-pages/

But, we could delete master and make gh-pages the default branch.

Note that things being output to /build would need to be output to the root instead... If we do this, we may want to move all the stuff to generate into its own directory to keep things tidy...


Well in my rom I use a modded KA so I'll just add them in myself. Thanks again, I appreciate it. This will probably save me some headaches.


Thanks, I wasn't aware -- will do in future.


why'd you change the name of this method? Convention is that *_setPosition* is meant to be private but *setPosition* would be part of the element's API.


top --> edge


OOPS


Good work! Some minor comments. :)


side note: cmdline parsing code that uses getopt is an example of something where std::exit(1) doesn't bother me that much (you probably don't mean std::exit(-1)--you'll get an exit status of 255). I still don't do it myself (I prefer boost::program_options to getopt/getopt_long, and I throw), but options parsing code is less likely to be reusable than most other things anyhow.


The problem here could be that regex's with a `g` flag set state on exec (so you can call exec several times, and walk find all the matches). Try without a `g` flag, or recreate the regex every time this is called.


@chriscct7 is correct. DateTime is 5.2+ but `createFromFormat` is 5.3+ only. No bueno here.


Actually, double thanks for this: I haven't paid much attention to the
guide before and found a lot of useful information. I should contribute
to this later ^_^


По-моему ты не понимаешь сути prefs_meta. Разветвление нужно создавать только в исключительном случае, когда настройки одной версии будут **мешать** настройкам другой. Совершенно ни к чему делать это на каждую новую версию, в которой добавили или удалили настройку. Это лишняя морока при сопровождении, а пользы никакой, ведь создание несуществующих в данной версии настроек ничем не вредит. Я вообще собираюсь убрать единственное имеющееся разветвление (см. The-OP/Fox#20).

Кроме того, файлы с суффиксами fx<номер версии> не пишутся вручную, а генерируются из meta при помощи make_prefs.sh.

Алсо, не нужно удалять устаревшие настройки. Мне все равно придется тщательно проверять каждую, так что лучше уж я сам.

И да, я очень надеюсь, ты не собираешься создать гигантский пулл-реквест со всеми этими спорными изменениями сразу. Лучше по частям.


exit() is also bad for testability. Although my preferred unit test framework (gtest) supports "death tests" (tests that assert that a program exits), it's much more convenient and portable to test that an exception is thrown.


I know how to do the only on Fridays thing in Node, not sure in php.


I also added the miles calculation since it said measurements should be in Kilometers and miles


98812358-IDF-SwimmingPoolSurface-4844 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1784 of 1795 tests passed)

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


Done: https://github.com/jcanizales/grpc/commit/d7f2ab31258a0972eb08e9fac0cac5b64b2a7919


Are we saving a hash allocation in this commit or is this paving the way for saving a hash later?


Good idea. Done: https://github.com/jcanizales/grpc/commit/d7f2ab31258a0972eb08e9fac0cac5b64b2a7919


Maybe use udemy-original.png that you added to the repo ?


That's a very long url :)


Since fixed in https://github.com/10gen-labs/mongo-php-driver-prototype/pull/77.


We did saved some allocations in later commits, this commit only moved the code.


Also, this could very likely be the cause of your randomly failing specs.


According to `php_phongo_log()`, only error and critical messages throw. Warnings and below simply get logged. I think this is alright as-is (I was really just mimicking the same logs we'd get during URI string parsing).


I wouldn't link to frozenlock,  I'd link to https://github.com/Day8/re-com/issues/49


Test comment plz ignore


for some reason `node_modules` is still being pushed to github. Any reason why it is still happening?


by gum you're right. life is good.


Great idea! Thanks for the help!


Note: fixup here https://github.com/grpc/grpc/commit/d13bbed8becc2f3564734399012c09660ceed303


Lambda_MemberArray_Reduce (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1764 of 1777 tests passed)

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


And you did it here anyway ;) `if (!parentExpression.OperatorToken.IsKind(SyntaxKind.EqualsEqualsToken))`


logs should not be included in Git or github; it should be ignored as well I also noticed that your `.gitignore` file is inside another folder, it would work best to put it at the root folder.


yes, I should have used it, I'll fix it now In shaa'ALLAH


@aleffert: We can set an `UncaughtExceptionHandler` on the UI thread, but I'm not sure what we can accomplish there. All the throwables caught there would either be unchecked exceptions, which are not meant to be caught even in their own scope and would be meaningless in a global scope, or errors, which are not meant to be caught in any circumstances. The only thing we might sensibly deal with would be an out of memory error, in which case we could attempt to release any memory caches that we might have, although I am not sure if we are doing anything of the sort.

The existing handler for uncaught exceptions in Android just displays a generic dialog message, while the framework restarts the thread with the previous Activity from the stack. We might be able to get rid of the error dialog with a custom handler, but I am not sure if that is a desired behaviour.


I would use an explicit cast when it's already certain it's a `BinaryExpressionSyntax`. Same goes for line 47 then.


so this appears to be used for redirected the reference artifact from the live log container to the backing log.  The expiration of this should be the same as the expiration of the bulk log artifact.  bulk_log.js should be changed as well.


the lack of capitalization is intentional


any ideas?


This is tempoary


>Это лишняя морока при сопровождении, а пользы никакой, ведь создание несуществующих в данной версии настроек ничем не вредит.
Не люблю лишнее, поэтому и подчищяю.
>Кроме того, файлы с суффиксами fx<номер версии> не пишутся вручную, а генерируются из meta при помощи make-prefs.sh.
Да, это я заметил уже после коммита.
>Алсо, не нужно удалять устаревшие настройки. Мне все равно придется тщательно проверять каждую, так что лучше уж я сам.
Эту ветку я скорей для себя сделал, для остальных полезного тут мало.
>И да, я очень надеюсь, что ты не собираешься создать гигантский пулл-реквест со всеми этими спорными изменениями сразу. Лучше по частям.
Не собираюсь. Да и какие коммиты перенести в основной то можно?


Lambda_MemberArray_Reduce (mbadams5) - 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)


Correct me if I'm wrong, but I think that logic is now handled in upload_to_s3.js now.


Okay


Sorry I'm slow, but what's the purpose of the bitwise 'and' here? Should it be an 'or'?


Looks nicer with caps. Like how Tim Cook would do it


感觉这里应该用TryLock，可是目前ins中的TryLock实现有点问题，我已经在那边提issue了


Every time I run `npm install` it runs the `prepublish` task.


姚沛然！！！！缩进！！！！！！！！！！！！！！！！用TAB！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！！


This was a bug -- I'm trying to track the local variable `snap_min`, but setting `oldest_id`.


@rvagg uh, should this be here?


>Эту ветку я скорей для себя сделал

Тогда никаких претензий.


Looking around, It will probably be easier to use a temp file than add a WriteCloser everywhere we implement util/exec, E.G. kubelet has `execInContainer`.


@bnoordhuis Should this also go on master?


:scream: wrong hanging indent!


98318570_RemoveRadianceCheckbox (macumber) - Win64-Windows-7-VisualStudio-12: OK (2197 of 2215 tests passed)

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


The Save/Restore route seems most elegant, I am working on writing these, should be in soonish.


Ah right, because we are switching to https://github.com/nodejs/dev-policy/blob/master/releases.md :)


cc @one-more-minute @MichaelHatherly is there a way to filter docstrings from files which don't belong/weren't defined here (in LightGraphs). I don't see anything in the meta. :(

*(Here the docstring for reverse of a graph is appended to the Base.reverse function, it's still a separate MD.)*


How do you know next version must be 2.2.0 but not 2.1.1 or 3.0.0?


Does that work?


I don't think so, This kind of username is extracted from a validated email, and almost email provider already validated the email address. Therefore, I think the username is safe and we don't need re-format it again.


:/. :( :-1:


This also doesn't work with mk_mvc.mak.
nmake doesn't support `$(shell)` function.


But is it valid when message fragmented but not encoded?
In section Fragmentation:
> Some networks may have a maximum message size that is too small to contain an encoded OTR message.
> In that event, the sender may choose to split the message into a number of fragments.


sounds good


pipelines to generate our own library


BTW, use Picasso plz.


Is this true? "once all the tests have finished, it will clean up the fixture object by calling Dispose, if present."  http://xunit.github.io/docs/shared-context.html


:tada: 2 :+1: 2


@jedmao you run `npm install` in `postcss/` dir or in other project that depends on `postcss`?


well, my expectations of `include_external_msproject()` working are low, because not only does it do nothing, but it does nothing and then throws a fatal error :)


LOL RIP Little Women


ah, so it doesn't. anyways, i would have to adapt my protobuf 2 messages, and then deal with the headache of getting cmake to use protobuf 3 instead of the distro's protobuf 2 on linux.

atm i've lost hope on getting this working on OSX and windows. what options do I have? have netplay be linux-only? is there another binary serialization library? maybe BSON?


At this point, I think stability and clarity are far more important than answers to these somewhat delphic questions.   (In the case of STRING*0, both "" and null are defensible, not least because both these values are additive zeros for strings.)


I think the reasoning was that one function could use multiple memotables to differentiate parts of a computation that might have conflicting art names. It might be useful since we don't yet have a standard for generating new names.

It's also likely that it wasn't fully implemented.


Add a note like this:

```rst
See :term:`UI_LLCC` for more information regarding the format of this string.
```


test-s3-fixes (lefticus) - x86_64-MacOS-10.9-clang: OK (1789 of 1789 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)


Is this due to a behavior change in OData Web API 5.7 beta: https://github.com/OData/WebApi/issues/447


@jedmao done aa65078


Type should be string.


Lambda_MemberArray_Reduce (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (743 of 743 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-7.12%25-green.svg)


For future reference: always limit first line of commit comments to 50 chars or less

![scrnshot1](https://cloud.githubusercontent.com/assets/90030/9101973/471bae48-3ba2-11e5-9ad8-aa493fa67b1c.png)

Learn more here...
http://chris.beams.io/posts/git-commit/


I'd like to cross-reference this with the troubleshooting info on session IDs. Maybe it makes sense for me to sync these commits with my own fork and make those additional changes, then merge the final thing to master? Or if you want to make those changes yourself, that's fine too.


漢字名: (null)
ひらがな: じょん

こちらを考えてました。
漢字名の部分はアプリとして必須ではないので（回答はひらがなでおこなうので）nullを許可しましたが、今後必要によっては not null に変えるかもしれません。

要件作成の時は、英語名にも完全対応するのは時間的におそらく難しいので、妥協しても良いという流れのお話になっていたと思います。
要件再確認してみます。


Why are we returning `io::ErrorKind::WouldBlock`? Why not a raft specific error. If this were a would block then mio would return an `Err(..WouldBlock)`. This is an `Result<Option<Stream>, io::IoError>`. I'm yet to check what causes the Ok(None) case - which would be a suitable name for the raft error - or simply ignore it.


If we keep the `ParallelizeTestCollections`'s default value `true`, which test cases fail?


> Therefore above code would just be with multiple calls to refreshRendering() causing updates to inert DOM.

OK that's good.

> Of course it's (slightly) better if all above can be done with just one refreshRendering() call, but I don't see a good way unless above code uses an API that creates widget and mix-in properties at once (tagConstructor()?).

Calling `new MyWidget()` is definitely one way to avoid the inefficiency, although that's a big change.

The other way is just to not include your latest changes.   The code was working fine before.  We just needed to occasionally calling `notifyCurrentValue()` on creation.   So the question is really: does your change make things better or worse?  I'm not sure.   What's the advantage of these changes?

> It appears that you think there is a case where my new code throws? Could you elaborate this?

```js
register("my-widget", [HTMLElement, Widget], {
    foo: null;
    refreshRendering: function () {
         this.innerHTML = this.foo.bar;
   }
});

var myWidget = register.createElement("my-widget");
myWidget.foo = {bar: "hello world"};
```

With your latest changes, `refreshRendering()` gets called before `foo` is set, causing an exception.

I think there's a similar situation with templates too, because `initializeRendering()` will be called before the user properties have been mixed in.


So it's sort of a summary line

On Wed, Aug 5, 2015, 6:47 PM andyl <notifications@github.com> wrote:

> For future reference: always limit first line of commit comments to 50
> chars or less
>
> [image: scrnshot1]
> <https://cloud.githubusercontent.com/assets/90030/9101973/471bae48-3ba2-11e5-9ad8-aa493fa67b1c.png>
>
> Learn more here...
> http://chris.beams.io/posts/git-commit/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ctaylor451/BAMRU-Org/commit/e9a099b706ea913830d8d44d9189899a20b1a235#commitcomment-12555615>
> .
>


お二人から指摘をいただいたため、

kana_first_name
kana_last_name

に変えようかと思ってます。


The instruction page works! The "bug" was that there was a script that was started but never closed.


Button is now title 'add histogram'. There is a sentence at the top that says Analysis is about displaying histograms and copied the examples from the notes. When you click 'add histogram' button there is some text telling you to search or browse for the layer you want to use. I'm thinking any additional text is redundant. Happy to change any text. The issue said an (i) with collapsable additional text instead of a notes area for anything else.


Вот и порешали.


Perhaps, `(_codePage == CodePageCyrillic)` ==> `(_codePage == CodePageThai)` ?


Add this to a common assembly info class, and link to each test csproj.  Which runner does this setting apply to?


#reading the data
datos=read.table('C:/Users/Martin/Downloads/household_power_consumption.txt',h=T,sep=';',fill=T)
#subsetting
dato=rbind(datos[datos$Date=='1/2/2007',],datos[datos$Date=='2/2/2007',])
#pass to character
for(i in 1:2){
dato[,i]=as.character(dato[,i])
}
#pass to numeric
for(i in 3:ncol(dato)){
dato[,i]=as.numeric(as.character(dato[,i]))
}
#plot 1
png(filename='plot1.png',height=480,width=480)
hist(dato[,3],col=2,xlab='Global Active Power (kilowatts)',main='Global Active Power')
dev.off()
![plot1](https://cloud.githubusercontent.com/assets/13001313/9102026/719db3c6-3bb3-11e5-929b-f0a3411a52b2.png)
#plot 2
png(filename='plot2.png',height=480,width=480)
plot(dato[,3],type='l',xaxt='n',xlab='',ylab='Global Active Power (kilowatts)')
axis(1,c(1,1441,(2*1440+1)),c('Thu','Fri','Sat'))
dev.off()
![plot2](https://cloud.githubusercontent.com/assets/13001313/9102032/7b78bbd4-3bb3-11e5-84a6-52bf0c309470.png)
#plot 3
png(filename='plot3.png',height=480,width=480)
plot(dato[,7],type='l',xaxt='n',xlab='',ylab='Energy sub mettering')
axis(1,c(1,1441,(2*1440+1)),c('Thu','Fri','Sat'))
lines(dato[,8],col=2)
lines(dato[,9],col=4)
legend('topright',lty=1,col=c(1,2,4),legend=names(datos)[7:9])
dev.off()
![plot3](https://cloud.githubusercontent.com/assets/13001313/9102040/831c6bd8-3bb3-11e5-969d-5a5aa1164e7b.png)
#plot 4
png(filename='plot4.png',height=480,width=480)
par(mfrow=c(2,2))
plot(dato[,3],type='l',xaxt='n',xlab='',ylab='Global Active Power (kilowatts)')
axis(1,c(1,1441,(2*1440+1)),c('Thu','Fri','Sat'))
plot(dato[,5],type='l',xaxt='n',xlab='datetime',ylab=names(dato)[5])
axis(1,c(1,1441,(2*1440+1)),c('Thu','Fri','Sat'))
plot(dato[,7],type='l',xaxt='n',xlab='',ylab='Energy sub mettering')
axis(1,c(1,1441,(2*1440+1)),c('Thu','Fri','Sat'))
lines(dato[,8],col=2)
lines(dato[,9],col=4)
legend('topright',lty=1,col=c(1,2,4),legend=names(datos)[7:9],bty='n',cex=.8)
plot(dato[,4],type='l',xaxt='n',xlab='datetime',ylab=names(dato)[4])
axis(1,c(1,1441,(2*1440+1)),c('Thu','Fri','Sat'))
dev.off()
![plot4](https://cloud.githubusercontent.com/assets/13001313/9102043/89bba044-3bb3-11e5-9687-ecc9bea07200.png)


Wow, so beautiful now! :smiley:


使用js去给li赋值。不要自己手动每一个赋值。


Actually, this needs to be fixed. (In my repo I've renamed the file):

```shell
➜  test git:(adamnovak-cycles) ✗ vg index -g -k 16 cyclic/two_node.gfa
GCSA::prefixDoubling(): Initial path length 16
  mergePaths(): 2 -> 2 paths
  mergePaths(): 2 unique, 0 unsorted, 0 nondeterministic paths
GCSA::mergeByLabel(): 2 -> 2 paths
GCSA::build(): 2 edges
GCSA::sample(): 2 samples at 2 positions
➜  test git:(adamnovak-cycles) ✗ vg index -g -k 16 <(vg view -v cyclic/two_node.gfa)
GCSA::prefixDoubling(): Initial path length 16
  mergePaths(): 117 -> 112 paths
  mergePaths(): 112 unique, 0 unsorted, 0 nondeterministic paths
GCSA::mergeByLabel(): 112 -> 112 paths
GCSA::build(): 117 edges
GCSA::sample(): 14 samples at 14 positions
```

So the test based on this in `test/t/06_vg_index.t` is broken. It doesn't fail but it also doesn't produce output. It's interesting that the GFA file can be opened by vg without error, and maybe not the right thing.

I'm fixing this in the merge.


The coding standard is to put the closing brace on a new line


将列表数据拉到全局去取。这样不会被限制住区域。


这个代码还是有问题。因为拖动的时候targetIndex会改变。会引发问题。


Вот что я нашел:
1. парент заменяется на
var newInpVal = $(this).closest('div').find('label span:last').text();
2. Ресурсы
- http://jquery-docs.ru/Traversing/closest/
- https://api.jquery.com/closest/


Missing lock


glad you found it useful :)


I confess to moderate confusion re: the new merging scheme.  Do patches go into v3.x, then up-merged into master?  Or do patches go into master and are then cherry-picked into v3.x?


we are not maintainers on this plugin
cc @tux2323


the data is processed


there


Might be best to replace " - " rather than "-" so you don't hit hyphenated words


My system is a FreeNAS 9.3 system (based on FreeBSD 9).
And MineOS-Node is running inside a jail. (standard jail, as in a freebsd jail).
I missed that step on the wiki actually, so i have mounted linprocfs on /usr/compat/linux/proc now, reverted my changes and it seems to be working now.

You might want to link to the wiki page for the different OSes installation guides in the Readme cause i have only been looking at the github Readme when i installed, or link to the wiki somewhere in the readme.

As of FreeNAS 9.2.1.8 linprocfs is already loaded in the kernel of what i could find but its not mounted in the jails automatically so i ran these commands outside of the jail (replacing the capitaliest words with correct info):
```sh
mkdir -p /mnt/POOLNAME/JAILS/JAILNAME/usr/compat/linux/proc
mount -t linprocfs linprocfs /mnt/ZPOOLNAME/JAILS/JAILNAME/usr/compat/linux/proc
```
And added a pre-init script to FreeNAS with the mount command.


Can you standardize the tabs??


are there any add-ons that do something similar? I'm afraid I don't have enough knowledge of our build system to figure this out. what files needs to be installed on each platform? where should they be installed to, what variables do I need to use for the proper directories, and what install commands do I use for each kind of file? is there a reference you used to build this build system, or is it all just standard cmake?


is it buffer profile list or priority groups list?


Sorry, I wasn't very clear. I meant you don't need the `.*` at the start or `*` at the end of your regular expressions, so you can do:

``` js
 if ((/Windows NT 6.1; WOW64/.test(navigator.userAgent)) && (/Chrome/.test(navigator.userAgent)))
```


NAVER - http://www.naver.com/
--------------------------------------------

heralder@naver.com 님께 보내신 메일 <Re: [frontend] modified if/else condition to make it more efficient. Will trigger on all chrome and opera users on windows 7 - but this is no different to before. (9e3e571)> 이 다음과 같은 이유로 전송 실패했습니다.

--------------------------------------------

받는 사람이 회원님의 메일을 수신차단 하였습니다.

--------------------------------------------


You shouldn't change the default commit message for merge commits


@johndyer Again, maybe you did not notice it in the original commit where I posted the comment (are you getting line comments?), but this is incorrect. There should either be `.call(this, event)` or `.apply(this, [event])`.


I would have name it with the full pathof the component `list.action.selection.all`


i am not overly happy with this approach, but did not have other ideas (except for editing backbone.js directly…). Tested with Mail app, OC.Backbone will always have shipped version if included.


@vilchik-elena this is breaking it-ruling


@jdalton Is this version was tested with Backbone.JS by replacing Underscore?


Oh, I know, but there's several issues with that, unfortunately.

SHGetKnownFolderPath needs Vista or above. I.e. no Windows XP, or any of the earlier Windows versions. I'd really not jetison XP support so easily. Also, SHGetKnownFolderPath is not available at all when compiling with MinGW.

There's an older version of that function, SHGetFolderPath. But you need to #define _WIN32_IE to some value. I see conflicting answers on which value to use, and I think the value might even be different in different MinGW releases. No idea what MSVC wants there either. And SHGetFolderPath is marked as deprecated.

So yeah, I'd rather avoid using those functions...


Checkmarx scan results 06 Aug 2015 13:12 Jeru0ale12 SPan6ar6 Ti12e

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud


I suggest we just always load "backbone.js" followed by "oc-backbone.js" in base.php


можно var newInpVal = $(this).parents('div')


- Now Prevent student to switch to another question by refreshing browser
- Bug fix at Initial test button


Is an adviser's reference number case-sensitive?


@herrniemand I feel like your comments should be directed primarily at @krainboltgreene, not at those criticizing him.


I'll do one more read to find any possible mistakes, but this looks good to me.


That has also the advantage that all the apps may use it. adding vendor script is not possible by API right now, unless i oversaw a thing.


멤버인 m_Ships가 public으로 공개되어 있습니다. 클래스가 내부적으로 가지는 멤버 변수들은 모두 private이나 protected로 지정하시는 편이 객체지향으로 좋습니다.


I just tried this and have the impression that exception handler is never reached when switching sessions (or restarting the application). The reason is that before actually *opening* the list of files is *loaded* - triggering the other dialog that immediately removes the files.
So, when I rename a folder and switch to that session the respective files are removed from the combobox, so the user can't access them to "open" anymore.

OTOH, when I'm *in* a session and remove the files and *then* switch to a removed file, your dialog appears. In *that* context multiple dialogs are not an issue. Except that when all files of a session are missing and I click on "Yes" to remove the file from the list I'll immediately get the next dialog.

OTOH, If I already had all files in a session open and *then* remove them I can still switch back and forth between them - because they had already been opened.

So the first thing that should be done is suppress/remove the check while loading the files in from the session handler.
Actually what seems like a good user experience for me would be:

* Session handler detects one or more missing files
* Session handler triggers a dialog informing the user about the list of missing files
* Instead of removing the files it (somehow) indicates the entries of the combobox.
  Best would be to make them appear grey (but not disable them)
* This gives the user the opportunity to restore missing files (if they were removed by accident)
* When the user *still* tries to activate a missing file *your* new dialog appears, giving the option to remove or keep the file in the session.
* When the user restores a file in the meantime and *then* opens it in the viewer the marking (as missing) should silently be removed and the entry be black again.

This requires adding a "isPresent" flag in the documentChooserAction and some housekeeping around it. But somehow this looks like a consistent approach.


you can use gettext to do this i guess


Hi Krzystztof,
is it something I should worry about? How much the performance has degraded and why? Can you elaborate on this? Is there a way to avoid this performance degradation?

Thanks!
Andrea


Should be possible, see https://github.com/owncloud/core/blob/master/lib/base.php#L409


Our advanced strategies got it at last!


if you don't know how standard software is installed, you should start there ;) follow unix standards.

install(FILES <headers> DESTINATION include/<something may go here>)
install(TARGETS <lib> DESTINATION lib)

it's all built on vanilla cmake. or, on vanilla standards for software installation really. random choice of example doing the same https://github.com/kodi-pvr/pvr.argustv/blob/master/depends/common/jsoncpp/CMakeLists.txt


Yes, ain't a problem at all.


Hi,

on my box:

     Linux server66 3.13.0-37-generic #64-Ubuntu SMP Mon Sep 22 21:30:01
UTC 2014 i686 i686 i686 GNU/Linux

the RPM build failed with many undefined references to __atomic_fetch_add_4.
libatomic provides them, but some gcc versions seem to resolve them as
built-in functions.
I therefore added a check for libatromic in ./configure and add the
-latomic, but only if it is present and if
it provides __atomic_fetch_add_4. That should not do any harm.

My gcc is 4.8.2. Interestingly these undefined references only show up
for the RPM build but not for
the "normal" build. I have browsed the web for this (apparently known)
problem, but none of the
solutions proposed there did really work.

If there is a better solution for this problem, please let me know.

On 08/05/2015 09:57 PM, Othmar Truniger wrote:
>
> Is there any special reason for additional dependency on libatomic?
> Now all link commands add a "-latomic" to the line, but in the end ldd
> does not show this library for any binary. So does this do any good?
> This is on Ubuntu with gcc 4.8.4.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/knxd/knxd/commit/da5f57832d33628c232f3d1f2526748a02dbddd1#commitcomment-12550079>.
>


TeamCity YaccConstructor :: YC.Core General Build [Build 487](http://teamcity.codebetter.com/viewLog.html?buildId=196691&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Hi, is there any update about this project recently? I found the Google APIs have updated for several times and I'm studying ur codes. It's so nice of u to make this project and could u tell me if I can run it with the latest Google APIs？


Checkmarx scan results 06 Aug 2015 13:19 Jerusalem Standard Time

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2df19aa46451ad26f4dc72164d23cf438903565b	@cxehud


Don't confuse # LoC with complexity.  6 of those extra lines are blank, one is due to line-splitting/reindentation, 2 are `def` function headers, and 2 are calls to the new functions.  So that accounts for 11 of the 12 extra lines, for a substantial improvement in readability and maintainability.


Checkmarx scan results 2015-08-06T10:08:53Z

High
Medium
Low
Info

New
26
17
30
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2870bb655123777502e2b4efe7d683026d07a793	@amitchen22


expModField = frameContents.find('#ExperienceModificationCustCA').val(items.ExpMod);
}else if (1 == frameContents.find('#ExpModPlaceholder input').length) {
expModField = frameContents.find('#ExperienceModificationCustCA').val(items.ExpMod);


:shipit:


Lets make the thumbnails 60px in height. Width should be whatever it is when image is scaled down to height 60px.

double size for supporting good screens like on retina and mobile devices


Перечитай выше


Good catch @slycoder


@joepie91 my comments are directed to the community as whole. We are so concerned about what people say, rather they do and what they can do, that we are simply happy to remove developer because his opinion is wrong. We stopped being able to accept criticism from others, and we are happy to use rules that were made for good, for the removing people that are offending us.

As I said before: I do not agree with @krainboltgreene opinion. But I am neither agree with "Piss off Ghazi, you're not welcome here.". If you think that you are better then @krainboltgreene, then maybe you should act like that.


нет, если бы заметил, написал бы по другому


For hadoop-1, with older versions, there was no hadoop-client, it was just hadoop-core. But hadoop-2 had hadoop-client. Hence oozie had hadoop-1 and hadoop-2 which had dependencies on on hadoop-core and hadoop-client. But newer versions of hadoop-1 have hadoop-client. Hence there is no need of hadoop-1/hadoop-2. However, hadooplibs has hadoop-auth-1 and hadoop-auth-2 and so on


@slycoder there's a prepublish script in the package.json that doesn't run automatically unless npm install is run without arguments. It populates */lib*.


Maybe I do not understand the --dbsnp flag but I'm not sure if this change is correct. I need to test.


@herrniemand I'm not arguing that @krainboltgreene should be removed. My point is that he started out with the exclusionary "go away" behaviour (repeatedly), and others have simply been dishing out the same in return.


ok, `install(TARGETS...` is what I want. however, I can't access the targets because they're defined in a subdirectory's CMakeLists.txt: `add_subdirectory(cmake)`. is there a way to access targets defined in a subdirectory? or should I patch the subdirectory's cmake files to install their targets?


It is in terms of this lookup


test-s3-fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1795 of 1795 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)


You don't need to use Redirect Facade, just use the redirect helper function like so: `return redirect(route('chops.index', $message))`


you cannot access targets in a different scope (subdirectory). you have to add the install commands at the spot where the targets are defined, or hardcode paths.


@joepie91 yea, I know.
P.S. Using my right to criticize:
@krainboltgreene stop behaving like a fucking dick.


Checkmarx scan results 2015-08-06T10:16:13Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
26
17
30
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

2fa6d7983fe8a58599f5121c9c037083bd2b49da	@amitchen22


why are you assigning 1 to user_id here?


Please put our last


If ReverseCompareNodeTimeConnected lets us preserve the longest-connected nodes, doesn't ReverseCompareNodeMinPingTime preserve the highest pingtime nodes?

And I'm not sure about the magic numbers?  They seem a bit weird if nMaxInbound is configurable...


@joepie91 I guess I just generally went out about the fact that people want other people to be gone and used @krainboltgreene as example.


Your indentation is not consistent here. Fix that


Checkmarx scan results 2015-08-06T10:17:52Z

High
Medium
Low
Info

New
0
0
0
0

Recurrent
26
17
30
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

91b93c8bc7000fb7e3d8b4e01f248b9169790b37	@amitchen22

2fa6d7983fe8a58599f5121c9c037083bd2b49da	@amitchen22


test-s3-fixes (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)


oki


oki 2


structure


good,this solve my problem.when i first run app on my device,it tells me that can not find com.github.mobile.GitHubModule class


oki 3


oki 4


OK


oki 5


Yep