In debug builds it shouldn't matter. I can imagine problems in CPU intensive parts of the release build. I'm partial to using macros or `#if defined(DEBUG)`s in certain spots.


Fix for #2 and #3


いいね


After deleting ```notify?``` method, I cannot open the student_task page. Because in ```student_task_controller.rb``` line#17 ```@notifications = @student_tasks.select(&:notify?)``` calls this method. I am not sure we still need this feature.


Discarded file


:dollar: :money_with_wings: :moneybag:


It would be useful to have a short doc describing what the bool actually does. In particular, I find it pretty confusing that we can return (true, error).


Getting better... Slowly... Would like to have a review session of the bank account part a la yesterday if possible. I could get the withdrawal part but not in terms of preventing a withdrawal of funds greater than the account. Also how would one delete an account from the array? I was trying to be smart and delete "Pete" from the Beatles by searching for his bankAccount.accountName but couldn't get it to work.

How difficult did you find this (out of 10)? 0 being no problems at all, 10 being impossible
7

Was there anything that you struggled with?
Not sure why for the totalSum function when I created a variable for accounts[i].balance it wouldn't complete the loop
 i.e.

+var totalSum = function(accounts){
+	var total = 0;
+ 	for (var i = 0; i <accounts.length; i++){
+     var accountBalance = accounts[i].balance;
+     total += accountBalance;
+ 	}
+ 	return total;
+}

Is there anything that you'd like some further information on?
Objects

Roughly, how long did it take?
6hrs


Might be useful to check dependencies too, like destroy.


I would also test methods from posts_helper : other_users_liking_post(post_id) , names_to_display(id)


FourPipeBeamAirTerminal (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (1750 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.49%25-yellow.svg)


Merge with master


Do we support `$expand=*`?  OData Web API does not support it: https://github.com/OData/WebApi/issues/463


These three methods still be called at ```response_controller.rb``` line198. Do we still need this feature?


Yeah, I should merge this to master. There were a couple tests not passing still so I wanted to hold off until I was sure


The infamous "Gamer-S"!


Éstas son las pruebas JUNIT


tested you submitted merge. works on both binaries and binaries_TPG


teste ranier ok


Teste ok


actually milipi fixed in his "cleanup"


Is this because you use ``V == AffExpr`` in JuMPeR?


感觉分类的话比较好？

------------------ 原始邮件 ------------------
发件人: "bianzhifeng";<notifications@github.com>;
发送时间: 2015年8月6日(星期四) 晚上6:36
收件人: "bianzhifeng/customTabbarItemBadgeValuebackground"<customTabbarItemBadgeValuebackground@noreply.github.com>;
抄送: "Malaimo"<345811279@qq.com>;
主题: Re: [customTabbarItemBadgeValuebackground] Delete README.md (c4668b7)

我的项目里面就有一个CustomBadgeValueBackgroundImage 继承自UITabbarItem 你可以参考里面的代码 有什么问题可以联系我

 ------------------ 原始邮件 ------------------
 发件人: "malaimoo";<notifications@github.com>;
 发送时间: 2015年8月6日(星期四) 下午4:19
 收件人: "bianzhifeng/customTabbarItemBadgeValuebackground"<customTabbarItemBadgeValuebackground@noreply.github.com>;
 抄送: "＿祢还不懂＿"<1107733919@qq.com>;
 主题: Re: [customTabbarItemBadgeValuebackground] Delete README.md (c4668b7)

 Hello?代码呢？能不能贴上来参考一下？Thx

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


Current codebase (effectively) assumes all initial widget properties are set after observers for `computeProperties()`/`refreshRendering()` are set up. As far as I see there are several cases where such assumption not true; Typically properties with default values set in widget's prototype and properties with implicit default value (e.g. `false` for undefined widget property) that needs to be reflected to DOM. As far as I see they are common for many widgets.

My latest change is primary to make `computeProperties()`/`refreshRendering()` workflow more intuitive for widget authors. It lets widget authors deal with initial widget property values as well as their subsequent changes without being worried about where the observers for those are set up.

But myself is more for pushing for [`node.bind()`](https://www.polymer-project.org/0.5/docs/polymer/node_bind.html)-compatible API rather than `refreshRendering()` workflow, and therefore, it's fine for now if you want to go without my latest change.

Wrt exception being thrown due to widget properties not being ready, it's widget authors who are responsible not to introduce that condition. Meaning they should check `this.foo` existence before running something like `this.innerHTML = this.foo.bar`. Same for compiled template code. BTW this is something the [liaison's underlying API of `node.bind()`](http://ibm-js.github.io/liaison/docs/0.1.6/BindingSource.html) (e.g. ObservablePath) automatically deals with (e.g. http://jsfiddle.net/ibmjs/UAyFy/).


timeForNextShot was not working, replaced settingShootRate to 500 for testing. Fixed this issue (#22) in 8a50814dbe


interface


No, this does not fix the issue above, I can still see it happen.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 280](https://teamcity.labkey.org/viewLog.html?buildId=265149&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:55
### 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>

```


@mlubin - Sorry about this repost if you saw it earlier in the day.  This (updated) commit fixes the ``to_dot(::Pair{UTF8String,Any})`` method error when testing the *Graphs* package in julia 0.4.

The issue is that the ``collect(attrs)`` call in the code returns an array of ``(UTF8String,Any)`` tuples in julia 0.3 and an array of ``Pair{UTF8String,Any}`` in 0.4.  The two types are different.  Hence, the accompanying call to ``to_dot(attr_tuple::@compat Tuple{UTF8String, Any})`` fails in julia 0.4.  Moreover, the ``Pair`` type does not exist in julia 0.3.

The solution in this commit, which works on both julia versions, is to define and call a new two-argument method ``to_dot(attr::String, value)``.

Note that the old ``to_dot`` method is no longer called internally, and could be considered for removal  or deprecation.


this is selection sort in reverse...


Can I ask what this is useful for? My guess is it for inventory fetching?


```
phase-impute-pipe git:(master) ✗ bash app/sub.sh 1 10 20 assets/data assets/ref_data bin home/ linux out help
: command not found
: command not found:
.integrated_phase1_v3.20101123.snps_indels_svs.genotypes.nomono.haplotypes -l ho helpte1.chr1 home/ -int 10123.snps_indels_svs.genotypes.nomono.legend -g home/
: command not found:
```


So, the behavior here is different than what was here previously.  Not to say it was correct before, but...

In the case where the cache metaData was not valid, the previous code would call refresh(), this new code does not.


what's this todo?


Please rename this to be replyUrl instead of just url.


Gonna show this to the guy that told me the exploit.


When I've merged it with #778 all tests were passed.


Why does this ignore some of the incoming parameters from the redirect?


It looks like this JSON originates from [`mesosphere.marathon.api.MarathonExceptionMapper#entity()` where it translates a `JsResultException`](https://github.com/mesosphere/marathon/blob/33190122bc75faf08928c003ddc6fa3a4ca47f17/src/main/scala/mesosphere/marathon/api/MarathonExceptionMapper.scala#L67).


develop (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (2215 of 2232 tests passed)

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


I'm not a big fan of those pop-ups; they aren't very user friendly. Imagine you are the client: you see this sleek webpage design (handy of bootstrap) and everything is so nice and colorful. Then, you mess up on your login form with a small type and right away you are shown a bland, non-colorful, and annoying pop-up. I think it would be better to have a small, hidden element somewhere near the bottom with red text that will show the error upon receiving one.


So, if these urls are not equal, this means we will trust the cache unconditionally for fbx files, even if the lastModified date in the header is newer then the cache.  This doesn't seem right.


Where are you downloading google_maps_api_v3_20.js  from?


What's all the FreeRTOS stuff for?  Is this in place of the builtin mbed RTOS?

Also, it's best to include large chunks of external source code with git submodules so we don't track their content in our repo.  I can show you how to set it up if you want.


This is a known problem that seems to be specific to the version of HIDAPI. See a short discussion [here](https://groups.google.com/forum/#!searchin/psmove/PSMOVE_BTADDR_GET_SIZE/psmove/NA8oHT2TV_k/V0xT1rQadzMJ). I'm guessing updating hidapi, which requires this change for Windows, breaks cross-platform compilation. Though I haven't tested that explicitly.


unnecessary println


This looks pretty much right, but I think it should be named "SkipDeleteDataFiles" and "SkipDeleteDataDir"?


Closes #1.


Nvm found it https://github.com/google/closure-compiler/tree/master/contrib/externs/maps


@cshenry , do you remember what  these changes fixed?  It looks mostly like syntax changes and now theres this
refhash[item[col.key]] = 1;


fix-run-without-permissions-5062 (nealkruis) - i386-Windows-7-VisualStudio-12: 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)


re: #476 @shadycady re: your question about how often the new episodes are checked for available  download: the answer was every 2 hours. But this change just implemented changes that to every 15 minutes.


this is just how babel is compiling modules

http://babeljs.io/repl/#?experimental=false&evaluate=false&loose=true&spec=false&code=import%20foo%20from%20'foo'%3B&playground=false


Does it not have any case of `git://git@github.com/blahblah`?
I have no idea.


asasas


logo


Well nothing, I could do that as well :)


Great change!! :+1:


close #7


This should fix [this bug](https://github.com/Jesse-V/OnioNS-HS/issues/7).


Why is this line like 100 characters wide


Why is this getting called four times?


what do you think if Parsel provides this functionality in a public function that we import from here?

Another option is to add a known "extended-html" parser type for lxml.html.HTMLParser in parser ctgroups, force its usage here by calling `root = Selector(response, type="extended-html").root`


sorry for going back and forward, I am just trying to come with the best solution while brainstorming on your code.


contact me :)


Hello All,

Really Great to see NLM in the repository. Thank you ReasearchGate.

Cheers
Senthil

On Fri, Aug 7, 2015 at 5:07 AM, Deyan Ginev <notifications@github.com>
wrote:

> Woohoo! Really great to see the JATS output enter the master repo! Thanks
> a lot @lkohlhase <https://github.com/lkohlhase> and @brucemiller
> <https://github.com/brucemiller> ! It would be awesome if we also put a
> visible note of gratitude to ResearchGate for donating the code, as they
> both took the initiative and contributed most of the XML mapping;
>
> That would also be a visible incentive for other external vendors to
> donate LaTeXML enhancements, which is a great trend to keep up! (I'm
> thinking mostly of the code files and the Changes log file)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brucemiller/LaTeXML/commit/2922e0c129e2ade78a52aaa233f0693f7af7baee#commitcomment-12577974>
> .
>


Thanks @marshall-lee .

Yes, it is grape-swagger.


Hi, I found this section(298~310) of code would trigger  'TypeError: can't concat bytes to str'  in py3.3+ environment.

I made some modification even use the old version before your modification but just can't solve it...
Could you give some hint to me ? Thanks!


haha


You'd need to ask @bloodmc for why they're not needed, it's his patch. An \@Redirect method is required to take all of the redirected method's arguments, though.


hahaha


Tak jsem to právě teď vyzkoušel a u mě to funguje. Jste mi schopen dodat trochu více diagnostických informací?


As it's merged, probably easiest to raise a new PR now?


Quick question: is it NodIO or NodEO?


you really should name your commits properly...


There's still a bunch of tabs in here that should ideally be removed


fix-run-without-permissions-5062 (nealkruis) - 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)


@J0n8s don't have to, doesn't compile under Travis-CI, I will fix it online.
Also, the includes for WSApi/sockets were in UDPWrapper on purpose, please revert the change.


If you have any suggrstions for  improvementin the resume, don't hesitate to share :P


removing changes and rebasing


Introduce a regression on WebGL renderer


我的使用场景：在 nginx 下放了一个名为 elastic_words.txt 的文件来保存热词。

如果去掉这里的304处理，当 elastic_words.txt 没有更新时，会持续往 elasticsearch 日志文件里写入如下的信息，每分钟一条：
[INFO ][ik-analyzer] remote_ext_dict http://HOST_NAME/elastic_words.txt return bad code 304

之所以加上 304 的处理，是因为：根据代码逻辑，返回200和304，都会检查 Last-Modified 和 ETag，而 web server 返回304时，其 Last-Modified 和 ETag  都是没有变化的，因为不会发生更新行为，也不会输出日志。

当然，如果不是用静态文件，而是通过 PHP 或其他动态脚本动态返回热词和控制 header，倒是可以返回 200 并使其 Last-Modified 和 ETag 保持不变，以避免返回 304 导致大量日志信息的问题，但是这样就丧失了对使用静态热词文件这一场景的兼容。

所以，建议还是加上304。


`rc` here still has the old fields if they were updated in this match block.


Ditto about old functional value of `rc`


moved


You could leave out `!==0` here, same us done in checking the length of `$attribution`.


discoverd it during the #230, it's not specific to the webgl renderer I think but to offsetted canvas. This deleted code make the tooltip take into account the offset between the canvas and the window.
A similar code is in captor.mouse.js


@garvinhicking
Did you ever test this Garvin?
It seems the getTemplateVars() we are looking for can not get access at this point in genpage. They are not available....


done


This statement is useless, `allErrs` is rewritten on the next line.


I think we should refrain ourselves to try to improve on commonly used api, besides promisifying them,  it makes it very unintuitive to work with them.


Ok, test done


done for all. thanks


The method should be `computeIfAbsent`, right?


We've tried to do this before but ran into the issues described in https://github.com/mysociety/alaveteli/issues/903 - can you confirm that these are no longer an issue?


You have removed the portion `url = url || ''`. Will that create problem in any platforms?


NO!
You cannot change the interface.
A flow stub is an object with 1 property "window" and two functions "open" and "close".
What we call "window" or "flow" is an abstract concept that shouldn't be defined in the app itself bu encapsulated within the implementation.

Example, for the drawer, a window could simply be a view placed in the left side. BUT, for everyone external to the drawerManager, they just ask for a "window". It's all the same for your TabGroup; If your implementation does not match the interface, you don't have to move the interface, but rather rethink your implementation.
We could discuss about the interface for sure if something is wrong or impossible; But in that case, there is no issue.

Also, it's not just about the name, the works done outside, in the app should be different for the implementation. Except for the configure and init methods.


This ain't gonna work, no FPU on F1.


But it was causing shift when css margin properties where used.


It was computeIfAbsent but there will be confilict with Doug's computeIfAbsent - if we will use it on JVM8 and lambdas.

That is why we renamed it to the applyIfAbsent.


これは？


text подойдет?


Nope again.
All this works should be done inside the implementation of the tabManager. Nobody outside needs to know how it is actually handling its tabs and window. Use the init function to supply everything you need (arguable) to the manager, and don't let people see what you are doing. People are means, don't let them hurt your functions <3


performed


Sorry about that, I misunderstood what the assert was trying to do and thought that the precedence was a mistake.


looks good


Yes of course. Worked with Smarty2.

On 07.08.15 12:09, Ian wrote:
> @garvinhicking <https://github.com/garvinhicking>
> Did you ever test this Garvin?
> It seems the getTemplateVars() we are looking for can not get access at
> this point in genpage. They are not available....
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/s9y/additional_plugins/commit/92ffe170e351b0d11d79ba5f7b798d33c11aacaf#commitcomment-12583935>.
>


This merge actually looks good to me now @OkayDave


Could you make a child ticket of CP-12983 requesting SMAPIv3 support for cluster stack constraints?


I'd like you to revert this change now you should understand why ;)
Let me know if the comment above makes sense to you


0x80000000 is higher than INT_MAX (0x7FFFFFFF) so I think it will be unsigned by necessity (it won't be promoted to a 64-bit value unless you add ULL?)


Then I'm missing the background story... I thought the goal was for this to be retrofittable to Java8 interface.


fixed in 62e069557b4a64a5da2ecab9f858d30d1adaef5b.


The goal is to refactor this code:

        V value = map.get(key);
        if (value == null) {
            synchronized (mutex) {
                value = map.get(key);
                if (value == null) {
                    value = func.createNew(key);
                    map.put(key, value);
                }
            }
        }
        return value;

to

map.applyIfAbsent(key, func);


Why are we passing in two lists to these functions? We could just pass one list of all 7 cards, since there's no practical difference between them in determining whether a thing has been scored.


It should not be a plugin, as event collection plugins should only implement protocols.
The “good” way to do that would be a daemon that uses libeventc, the same way [git-eventc-webhook](https://github.com/sardemff7/git-eventc/blob/master/src/git-eventc-webhook.c) is doing that.

I am not sure I want to *enforce* that, as it means making core interactive (`eventd_plugin_core_push_event`) private API, and that would block any non-official protocol plugins.
But that would be nice not to have “random” event collection plugins in the wild. ;-)


Things like this should be logged with a 'normal' level.

```
LOGGER.log(Level.normal, "Platform does not support X", e);
```


#1209 isn't merged yet?

i was thinking we'd need to `block.nVersion & 0xffff < 3` to not compare the top 16 bits where chainId is stored? `0x00620002 < 3` is not going to work?


It is a cure for this:
![bmesh_sockets2](https://cloud.githubusercontent.com/assets/7894950/9133887/e7fcd538-3d07-11e5-97d7-5ceda8b9ca62.png)

Now there are errors when updating the individual nodes, but when the entire  node-tree is updating, all nodes are operating normally.

![bmesh_sockets1](https://cloud.githubusercontent.com/assets/7894950/9133810/3d76bfb6-3d07-11e5-8cd6-899d38ed92ef.png)


我本机高版本没法通过


To create a batch file for running python programs, use below command and save in a .bat file:
@py.exe C:\<address for the file>\<file name>.py %*
@pause


Not a correct usage of Gradle


IMO it's not a good idea as SF 2.3 is still maintained.

However, if you want to keep it, a `1.1` stable branch for maintenance would be great (BTW, I have a fix for you :-) ).


Oh, I was looking in the wrong patch!

Yeah, that's a good point. I'm busy tonight, possibly all weekend, but will try to get this fixed up (or raise a PR and I'll merge it)


a


Operatory oddzielaj zawsze od argumentów spacjami, czyli:
```
t = [[]]
```
zamiast:
```
t=[[]]
```


Pliki otwierane przy pomocy *open* powinny być zamykane przez *close*. Czyli:
```
f = open("abc")
...
f.close()
```
Można też na skróty przez tzw. blok kontekstu:
```
with open("abc") as f:
    ....
```
Plik zostanie automatycznie zamknięty po wyjściu z bloku.


Oh yes, I was surprised because I know how much you love asm ;)

```
Hey, can actually comment on the code!
```
 Just click on the + on the left :)


Not sure if the change in getDropboxDisplayName(String siteId) is required as its got to have adminId


<!-- 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.


<!-- PMABOT:TAB -->
This commit is using tab character for indentation instead of spaces, what is mandated by phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: libraries/plugins/schema/svg/Svg_Relation_Schema.class.php


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: libraries/plugins/schema/svg/RelationStatsSvg.class.php, libraries/plugins/schema/svg/Svg_Relation_Schema.class.php, libraries/plugins/schema/svg/TableStatsSvg.class.php


Bah sorry, not sure how I managed that. Fixed in https://github.com/mysociety/alaveteli/commit/e03dad3c3ef21bf688e727c20a216cccf86ecc54


Agreed.


Actually no there are two `defaults()`


I think we can, yes. I've got some more work to make GVT-g types appear dynamically in the same way NVIDIA types are generated, so I'll make this change along with that work.


No because we are doing a type check for string above so it has to be set.


Ah, so its a Smarty 2/3 version issue then.
I thought in Jan 13, 2013 we were already testing at least 1.7-betas with Smarty3...
So then it seems I'll have to move this for 1.7 up to another hook, maybe frontend_display?!


commit succesfull


Powyższy kod wczytywania pliku powtarza się w skrypcie wielokrotnie: można zrobić z niego funkcję. A jeszcze lepiej użyć gotowej biblioteki pandas do pracy z tego typu danym. Pozwala ona wczytywać dane tabelaryczne i wygodnie je obrabiać.

Przykładowe metody:
read_csv -- wczytuje dane z pliku tekstowego
to_csv -- zapisuje dane do pliku tekstowego
read_excel, to_excel -- analogicznie dla plików Excela

Przykładowy kod:
```
import pandas as pd

data = pd.read_csv("dane.csv", sep=" ")
col_mean = data.mean(axis=0) # Średnie w kolumnach
row_mean = data.mean(axis=1) # Średnie w wierszach
col_mean.to_excel("results.xlts")
```


BenchResult, GroupResult?


FYI this file now has wrong line endings causing git to always mark it dirty on Linux.


BenchException?


А это зачем нам?


This was required as an update to assert means that `uint` and `uint32` don't match anymore. I could have changed the cast to a `uint32` but this is what `EqualValues` is for.


No, it should be `pferdestärke` instead of `pferdstärke`. Don't forget to update the unit test.


ino chera hazf kardi?


Zdravim ano dodam vam informacie ake potrebujete...
A co konkretne potrebujete vediet .
par screenov v prolohe...

> ** Fri, 07 Aug 2015 03:01:15 -0700 - "feixm1/fastshare.host" **

>
> Tak jsem to právě teď vyzkoušel a u mě to funguje. Jste mi schopen dodat trochu více diagnostických informací?
>
> —
Reply to this email directly or [view it on GitHub](https://github.com/feixm1/fastshare.host/commit/a2f38f103bf5d0e2c84253c471c3b32bd86a5c2a#commitcomment-12583829).
>
>


@shadysamir almost. It should allow all content from embed html. Usually it doesn't contain iframe. For example for twitter embed it will looks like:

``` javascript
config.extraAllowedContent = 'blockquote[lang](twitter-tweet); p[lang,dir]; a[!href]; script';
```


As far as I know google maps API key is free.
If you mean that you want to include google maps as a base layer in your maps you can do it by selecting the google maps checkboxes in the base layers config. located in the Tools tab in Lizmap Plugin (QGIS)


@divaykin We can use them if they actually improve the generated code. IIRC, they're not very useful with GCC on x86-64.


Thanks. Got it.


@shadysamir btw, your [regex](https://github.com/meedan/checkdesk/blob/master/drupal/sites/all/modules/custom/checkdesk_core/ckeditor/plugins/mediaembed/plugin.js#L88) is not secure, it allows host names like hacker-twitter.com


hihi


Fixed in https://github.com/mysociety/alaveteli/commit/557b778af924226e62cdbb48e239e3a21159914e#diff-99cb709286f806f8c8b24ea88bc311afR648


24-> magic number. Co to jest to 24 ? moze wrzucic w jakas stałą ?


@kunal9007 lots of conflicts lol
I think you merged adrian into mrcl lol


Is this ">1" because at this point you have not removed yet the current shim? (so it's checking if the current one is the last shim)?


Why this? You aren't supposed to delete those manually I think..


``` test ```


Jak czesto zmienia sie liczba pi ? Nie lepiej bylo by zeby byla stałą ? Czy double to na pewno najlepszy typ ?


+1 I will take care of it.


Should be setup to -1 by default.


Czy  plik zawsze bedzie sie nazwyal someFile.txt. Co lepiej counter++ czy ++counter ?


some_word - > SomeWord. Preferujmy CamelCase


Why did you use an `unsigned short` in this file? (Some other places are still unsigned short.)
It can works much slower (Typically, CPUs are fastest at operating on integers of their native word size) and this type can handle just ~65k.

P.S.Iterators are great;


Done in d42aeb77216a040bdf353faa815b872e7714bc4b


This commit is being reviewed as a part of review issue #51.


I think it's worth explaining in the commit message here that it's only been working by coincidence so far, but when we make manuals appear on topic pages the links to the manuals will be broken unless we fix this.

That's because so far the pages which fetch their content from Rummager and include links to manuals have only a single path segment (eg `/search`), so treating the manual link as relative to the current URL does the right thing from those pages.

Topic pages have longer paths (eg `/topic/business-tax/vat`) so a relative manual link from those pages will be incorrect (eg `/topic/business-tax/hmrc-internal-manuals/vat-government-and-public-bodies`).


Could have commented on this. It's the list of states that can be passed to `-state`.


What, why ...... ? You need to provide better changelog to explain what exactly you are trying to do.
Long calls leads to pathetic generated code and thus needs to be avoid unless really needed !


MMM, but don't you need to register the Hooks as a service too by calling "registerService"?
https://doc.owncloud.org/server/8.2/developer_manual/app/hooks.html


1: M111 unrecognized component name: dojson


The "-Value" parameter has a weird - symbol.


Worth just adding a `-` to the closing erb tag so that if no content is rendered there's no extra newline in the markup. Just thinking about debugging this years in the future and missing the extra properties for a while

```diff
- <%= content_for :open_graph_tags %>
+ <%= content_for :open_graph_tags -%>
```


you are perfectly right.


Build failing is caused by the following:
```
[ERROR] Failed to execute goal on project glowkit: Could not resolve dependencies for project net.glowstone:glowkit:jar:1.8.8-R0.1-SNAPSHOT: Could not find artifact net.md-5:bungeecord-chat:jar:1
```


괄호 포지션의 상태가..?


interesting


yes I mean google maps as base layer, so can I use google maps foor free in
my webgis projects business type, without key?
tks jaitor


@dennisnderitu254 the name of a default site is usually index.html


This is added to a previous release.

Should add a new `## Highlighted Features` heading under `#develop` and add this entry there. If you rebase with develop I think the new heading has already been added.


pull request submitted for merge from forked branch to parent jmrocela/munchjs


Haha, nice


I think it would be safer to delete the old document after creating the new one, in case the script exits unexpectedly halfway through and we have to run it again, having already deleted something without saving the new version.


Ok, angenommen du hast eine Instanz einer Komplexen Klasse, zum Beispiel ein Layouter einer GUI. Und du ruftst die Methode zur layoutneuberechnung auf. Demnach würdest du dazu sagen, dass du den Wert des Layouters verändert hast?

Also meinst du man kann Zustand und Wert als synonym verwenden? Ich müsste darüber nachdenken, aber ich denke, dass man nur bei imutables das machen kann.

Zustand sagt aus dass es sich bei einer Änderung immer noch um das gleiche Objekt handelt, sich die Identität nicht ändert (a_t1 === a_t2, wobei t Zeit ist und zwischen 1 und 2 das Objekt mutiert wurde, entweder durch setzen eines Wertes oder den Aufruf von Methoden mit Nebeneffekten)
Wären nodes immutable dann könnte man sagen der wert hat sich verändert da a_t1 !== a_t2 gelten würde, sie wären nicht mehr identisch.

Also wie wäre es wenn ich den satz zu
`ability to observe the tree node's state changing`
:question:


Gets resolved in later simplification


好像有点问题,应该是只有200才会继续检查header信息,如果head信息和上次不一样才会开始进行加载


Is it correct that this `innerloop_ext` module was not pushed to GitHub?
I am very interested in learning from the Cython version of the inner loop.


Я понимал, что не особо правильная и приятная глазу конструкция, но дело было в том, что методы модуля FSCommunicator используются как методами класса, так и методами экземпляра класса Database.


Not if you are using automatic resolution. You can basically skip the registerService step if it follows the defaults, meaning that the following is redundant

```php
// application.php
$container->registerService('OCA\YourApp\TheClass', function ($c) {
    return new OCA\YourApp\YourClass($c->query('OCA\YourApp\TheClass2'));
});
```

PS: I've written roughly all of the dev docs but I didn't (and probably won't, will start to work at a software company come September) find the time to update everything to automatic resolution.


yeah you aren't... it was put for testing something with DDOS but i forgot to take it out obviously.


Not sure if this is still relevant to 2.1.x version given that strip_tags() is no longer there. This fix was pertaining to 2.0.14.


It looks like a 0.2.*4* delivery instead


Fixed in 3a50a7e2e53a738e0476f98a4a1002aae9f9cfda


read some more about Junits:


If I'm no mistaken long calls are "on" by default for uClibc toolchain?


commit synopsis: look for usage file in munchjs directory instead of current


    # Hacks to enable final link due to absence of link-time branch relexation
    # and gcc choosing optimal(shorter) branches at -O3
    #
    # vineetg Feb 2010: -mlong-calls switched off for overall kernel build
    # However lib/decompress_inflate.o (.init.text) calls
    # zlib_inflate_workspacesize (.text) causing relocation errors.
    # Thus forcing all exten calls in this file to be long calls

As I understood this comment says that `decompress_inflate.o`, `initramfs.o` and `core.o` must be compiled with `-mlong-calls`. Or just `decompress_inflate.o`. Actually now I'm confused about it.


In order to not have this PR sprawl any further, I'm going to leave it as `if @xapian_requests`


당신이 요청한 사항을 받아주겠습니다.


I don't see test using this function. Did you forget to add a file ?


Logobox is View helper. So I think better/common usage example is in View or Decorator.


That means that given these 2 classes:

```php
// yourapp/class2.php
namespace OCA\YourApp;
class TheClass2 {}

// yourapp/class.php
namespace OCA\YourApp;
class TheClass {
    public function __construct(TheClass2 $class2){}
}
```

both of the following things will produce the same result:

```php
// yourapp/appinfo/application.php
$container->registerService('TheClass2', function ($c) {
    return new OCA\YourApp\TheClass2;
});
$container->registerService('TheClass', function ($c) {
    return new OCA\YourApp\TheClass($c->query('TheClass2'));
});

$container->query('TheClass');
```

```php
// yourapp/appinfo/application.php
$container->query('OCA\YourApp\TheClass');
```


look, even GitHub syntax highlighting sees how stupid that was


Он гениален


remove pls


Hid server conflict with adb server:

After apply kernel-3.4.patch, adb server can't use.
$adb shell  print "error: device not found"
Revert kernel-3.4.patch back to normal.


Students/Onaciu V. Andreea/Assignement 2/MyAssignment2/src/Polynomial.java

Your design needs more work.
You want to model a `Polynomial`. Now this by default should give you the idea that you need a `Polynomial` class. Now what this class contains is all the attributes which `one` polynomial has, for example, its *coefficients* and *degree*. You got that right, however, your `Polynomials` class also contains methods for adding and subtracting which are not specific to a polynomial. So:
- Create a `Polynomial` class which has a constructor with a `degree` and `coefficients` parameters.
- Keep that `Polynomial` class simple. Create getters/setters for all the fields and place a good `toString()` method in it.
- Your `Functions` class should look like this:
```
public class Functions {

    public static Polynomial add(Polynomial a, Polynomial b) {
        // ...
    }

}
```
See how natural it _sounds_? The _result_ of adding _Polynomial_ *a* with _Polynomial_ *b* is another _Polynomial_ (the return type) ... and so on, and so forth.

If you have any questions, feel free to ask!


Thank you for documenting methods. I suggest to write methods documentation right above particular methods.


typo: logo => logos


Hid server conflict with adb server:

After apply kernel-3.4.patch, adb server can't use.
$adb shell  print "error: device not found"
Revert kernel-3.4.patch back to normal.

How to solve this problem ?


Following concerns that the original suggestion may cause problems of unintended re-arming, have changed it so that the prevent_arming flag will be active after failsafe landing is complete, but can be cleared by disarming before trying to re-arm.


@cristian-ungureanu tre licenta la imaginile alea


My mouse told me she is afraid
Of an ultimate mouse killer
But so useless since it get paid
So forgotten by the seller

And suddenly the rebirth
Of backers and buyers hope
From all around this old earth
Faraway from tv soap

Silent sales's boss is named Darren
Restarting com could be sudden !


Students/Huruiala V. Vasile Alexandru/eclipse/assignment3/assignment3.1/src/javasmmr/zoosome/models/animals/Aquatic.java

I am curious, why didn't you implement this default initialization like in the other classes, using `private static final` constants?
I see that the "Species" classes are all somehow disobeying that rule, but why?


okay.
got it.


Lovely work!


Sorry, but i think that you have an error in this code.
Problem with the  line 886: if (location.href.indexOf("#messages/conversation/" + userId) > -1) {
Example: if  in your active conversations in moodle mobile have an user with userId = 2 and other with userId = 23 then mixed the list of messages. The conversation with the userid = 23 mixed messages with the userid = 2.

You have a possible solution, compare with the url userid:
 if (location.href.indexOf("#messages/conversation/") > -1) {
     var userIdUrlAux = location.href.split("/");
     if(userIdUrlAux.length >0)
        userIdUrlAux = userIdUrlAux[userIdUrlAux.length - 1];
              if(userIdUrlAux == userId){
                        conversationArea = $(".conversation-area");
                        conversationArea.find(".removeMessage").remove();
                        conversationArea.append(html);
                        conversationArea.scrollTop(conversationArea.prop("scrollHeight"));
               }
    }
(...)


Am 07.08.2015 um 13:09 schrieb Danny Arnold:
> Ok, angenommen du hast eine Instanz einer Komplexen Klasse, zum Beispiel
> ein Layouter einer GUI. Und du ruftst die Methode zur
> layoutneuberechnung auf. Demnach würdest du dazu sagen, dass du den Wert
> des Layouters verändert hast?

Ganz genau genommen ändert man den Wert einer/mehrere Eigenschaften des
Layouters. Ich denke aber wir reden aneinander vorbei. Meine Betrachtung
bezieht sich konkret auf einzelne Eigenschaften eines Objektes. Bei
einer Änderung dieser Eigenschaft wird selbstverständlich auch das
Objekt als solches geändert. Aber: Man ändert in erster Linie nicht das
Objekt, sondern dessen Eigenschaften, oder konkret: Deren Werte.

>
> Also meinst du man kann Zustand und Wert als synonym verwenden? Ich
> müsste darüber nachdenken, aber ich denke, dass man nur bei imutables
> das machen kann.
>
> Zustand sagt aus dass es sich bei einer Änderung immer noch um das
> gleiche Objekt handelt, sich die Identität nicht ändert (a_t1 === a_t2,
> wobei t Zeit ist und zwischen 1 und 2 das Objekt mutiert wurde, entweder
> durch setzen eines Wertes oder den Aufruf von Methoden mit Nebeneffekten)
> Wären nodes immutable dann könnte man sagen der wert hat sich verändert
> da a_t1 !== a_t2 gelten würde, sie wären nicht mehr identisch.
>
> Also wie wäre es wenn ich den satz zu
>
> |ability to observe the tree node's state changing|
> :question:

check

>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/despairblue/furry-octo-dangerzone/commit/36d2576511d93faaf4b94ccd360650156beaef1c#commitcomment-12584704>.
>

--
Matthias Lenk, M.Sc.
Technische Universität Bergakademie Freiberg
Universitätsrechenzentrum
Bernhard-von-Cotta Straße 1
09599 Freiberg

phone: +49 (0)3731 / 39-4789
mail: lenk@tu-freiberg.de


I think nobody will use more than ~65k perimeters :-)

If it's slower, we can use int.


Hey, @guilhermef . Why did you add `<4.0` for tornado's version? Was there anything that didn't work with Tornado 4? Thank you.


Students/Farcas Silviu Vlad/Assignment 4.1/javasmmr.zoowsome/src/javasmmr/zoowsome/models/employees/Caretaker.java

Usually, the fields go before the constructor.


Fix the comment to say medium calls ?
In long past (when dinosaurs roamed in ARC office), we only had long-calls, which generated absolute 8 bytes jumps (no branches) and full kernel was built as such. With one optimization driv, I managed to build kernel with only fewer files built with -mlong-calls (henc eorig comment/code)
With 4.4 we got medium calls - so i switched to them (but failed to update the comment).

Makes sense ?


<!-- PMABOT:SPACE -->
This commit contains trailing whitespace, what is prohibited in phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: libraries/plugins/schema/svg/Svg_Relation_Schema.class.php


@kkrampa Wouldn't it be better to always exclude this rather than doing two queries?


Без цього можна обійтись, то я негарно зробив)
Глянь файли *.js.jsp


Just noticed, wagtail/contrib/wagtailstyleguide/__init__.py needs to contain: default_app_config = 'wagtail.contrib.wagtailstyleguide.apps.WagtailStyleGuideAppConfig'.


Students/Farcas Silviu Vlad/Assignment 4.1/javasmmr.zoowsome/src/javasmmr/zoowsome/controllers/MainController.java

I would try using `for each` loops here


@kkrampa this should just be `return qs.first()`


Or `stringByEscapingXMLEntities`


I got it. Sorry for my misunderstanding. So what this comment must actually say here?


Fix


Nicely done! kudos


If you are using PHP 5.5 you can also use this BTW which is better

```php
// yourapp/appinfo/application.php
$container->query(\OCA\YourApp\TheClass::class);
```

This is better because it fails on the lanuage level if the resolved class does not exist


@anthony-kolesov AFAIKR, long calls can not be defualt specially for uclibc tools as that will break PIC code right away !


<!-- 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.


@kkrampa this should be moved inside the `else` statement so that it doesn't get evaluated if it doesn't have to.


This patch looks good assuming there isn't a performance regression. I couldn't test this locally today and I'll be OOO until the next Tuesday, but I doubt there's a significant performance change, Let me merge this as is.


You could try using the `Random` class for additional 'randomness', however everything else is quite alright


@snopoke This code is moved from v1 but you're right.


Good


This `else` is not needed


Thanks for the hint! I refactored it and published a new version :)


ah I see. may be add ``include syslog`` in the monit module ?
Thanks


dstat recordings in general, Marcelo's dmon script doesn't do anything special to the csv output


just keep the top 2 lines which describe the linker relaxation and delete the lower 5 lines !


`for (auto child : m_children)`


This is because the shim-eth-vlan can have multiple instances (vlans) over the same physical device (NIC). The qdisc is atteched to the phy_dev so if we restore the old one (the original) we should do it as long as there are no other shim instances (vlans) on the phy_dev.

btw, you made me loose 5 minustes looking for a ">1" in the code ;)


Not


There's #10851, and I'm slightly apprehensive about what will happen to the Compat version-checking mechanisms when we bump to `-pre` and start making RC tags which will restart the `version_git` numbering. Hopefully will be fine, but I don't think people were checking against `VERSION` as carefully during the last release cycle.


@JoeBengalen What is the purpose of this? When does the path start with `//`? The `ltrim` function does not remove any slashes too.


@kkrampa this function is only used here so we could pull it into this app and apply caching.


Yes, @scribblemaniac. You are absolutely right. Thank you! Fixed in https://github.com/caskroom/homebrew-cask/pull/13064.


Sounds good.  Yes, I was referring to the defunct Unicode+TeX project called Omega.


remove empty line


@TobiasRoikjer `login_steps.rb.rb`


ok. I get a better understand now. Let me think again about the implementation :)


this will never happen. I mean, why would the `animalFactory` be null after being instantiated? if you're thinking about avoiding an out-of-memory problem, this won't solve it. If Java runs out of memory it will throw an exception


There is problem with 'reads' tuple from string when compiled with hastec-0.5. So representing muxshape as '[Int] [Int] Int]' works. This is temporary solution (I hope so)


why resp.data?


@kkrampa these queries to get `products` are basically identical (I think they should be identical) to the ones in self.headers. We could extract them to a property on `self` and memoize it so that we don't do the same query twice:

```
if not self.config['products']:
             products = SQLProduct.objects.filter(domain=self.config['domain']).order_by('code')
         else:
             products = SQLProduct.objects.filter(product_id__in=self.config['products'],
                                                  domain=self.config['domain']).order_by('code')
```


This still needs to be fixed.


I've fixed it in https://github.com/foss-for-synopsys-dwc-arc-processors/linux/pull/4 (now it's just another commit).


This commit doesn't work with local/remote pending members and works in counterintuitive manner: this setMemberIdentifiers implementation is not a proper "MemberIdentifiers" setter.

-2. Sorry.


In the future, if you want to introduce the wallet feature, you'll have to rethink about CreatedBy filter.


What about something like this:
> The installed base operating system is pre-configured with approved software repositories (like YUM or Debian). These software repositories allow for a consistent install, configuration, and upgrades for various common components like PHP or Java.

The basic premise we (my co-workers and I) were thinking is that we need to know that we will be working with a *particular* operating system (windows, linux, etc) and that commonly used software (python, java, gradle, etc) are available for the developer to install in a *consistent* manor. That way we know we can update them to the latest OS patch or upgrade the components using the provided system configuration management software.

This comes from having to deal with many operating systems and software that are all configured _slightly_ different and thus make updating/upgrading really difficult/non-automated.


Я их использовал лишь для простоты получения путей в тестовом задании при учёте, что в один момент времени используется одна база.
Так как таблица не знает, в какой базе он хранится, то я не могу запросить имя базы из какого-нибудь класса Person, который соответствует данной таблице, по той причине, что неизвестно имя экземпляра Database, к которому можно обратиться за этим.
```ruby
class Person
  def self.db_path
      [$default_path, $db_name, "/"].join
  end
end
```
Классовые переменные для этих целей тоже не подходят. Самым "железным" решением было создать и хранить конфиг с путями и именами баз где-то в файловой системе. Но опять же для тестового задания я решил упростить это до глобальной переменной.
Я полностью согласен, что это приведет к проблемам и нежеланным результатам при попытке работы с 2-мя и больше базами.


Logobox is not view helper. At least not in Rails terms. And also who just begins with Rails might not know anything about decorator but definitelly will recognize rails model. Who already knows rails knows how to put it into decorator.


There should be only one argument, the hostname as a uri. so ./stove localhost, or ./stove localhost:12345


@kkrampa this should use `.first()` since doing `if st` will evaluate the entire query.


This looks like a good place for overflowing_sub, but it would require using unchecked indexing instead of `.get()` to get the same end result.


Ok for me, but I would add a print of the error in the setError. I agree that in general a library should not print anything, but in reality most of the users of the library will not properly check for error codes and in case of error the software will fail in a misterious way. Adding a error code print could save us hours of debugging in the future.

Given that this modifications in the yarp-wbi requires a specific commit of wholeBodyInterface, it would be nice to bump some minor version number of wbi and require the latest version in yarp-wbi.

Other lower importance comments:
  * std::string are ok for me, for the API it may be more convenient to use char * for DLL compatibility (see https://github.com/robotology/idyntree/issues/33 ), but it is not so important because we have std::string anyway all over the API.
  * I am not a lover of following a strict indent style, but at least in the same file we should try to be consistent for readability.


As far as I can remember pseudo tags are not needed in expected.tags. They are trimmed by the test harness when comparing actual output.


LGTM.


You shouldn't silently create a folder + this is not recursive, so it won't work most of the time anyway.


Yeah, but once there's a 0.2.5, it will all be ancient history.


PHP naming convention is snake_case


Thx - pushed to 3.18!

For kernel patches I prefer a simpler workflow - where u send me a cleartext patch using git send-email - no need to create branches / pull requests !


Dodat paket primeri i svi uradjeni primeri su premesteni u taj paket


Don't do this is you can determine that etag is correct (i.e. store hash somewhere).


Agree @seixasfelipe :)


Variable here won't be replaced.


Space.


@emileighoutlaw have to give credit where credit is due, this came from @cadams


Exakt weil MVC eher locker definiert ist hatte ich Anfangs vermieden es als Analogie zu verwenden, da jeder seine eigene Interpretation, abhängig von Frameworks die er genutzt hat, Anwenden würde, und es damit als Domainterminus eher untauglich ist.

Zum Beispiel habe ich angefangen Teile des DOMs als View zu bezeichnen. Normalerweise wird das Template oder der Code, der die DOM Elemente verwaltet, View genannt.
Man kann bei GUI ToolKits auch Introspection benutzen um sich die Struktur eines Fensters ausgeben zu lassen oder beliebig Werte von Labels und Buttons ändern zu können. Aber ich denke auch in diesem Fall würde man das was einem die Introspection liefert nicht als View bezeichnen. Klar man kann, aber wie nennt man dann dieses XML templates dass die Grundlage für das Fenster war? Kann man auch View nennen. Man kann das Fenster auch programmatisch erzeugen. Wie nennt man die Klasse die das Fenster erzeugt? View?

Es ist leider alles nicht eindeutig definiert durch 3 Worte.

Das DOM ist das worauf man über ein Introspection Interface zugriff erhält. Es View zu nennen, sollte helfen es von dem JavaScript Code zu unterscheiden der es Manipuliert. Tat es scheinbar aber nicht...


missing a parameter.


Configurable somewhere?


Thanks -- I'll keep it in mind next time.


@kkrampa `f.exists()` will be much faster than `f.count()`


Commenting the last two points:

- you mean as function parameters?
- I don't understand what you are referring to.


Why LogType = 1 and not 4 ?

```Csharp
TempData["LogType"] = 1;
TempData["LogMessage"] = "An error occurred, please contact site adminitrator, no transaction has been created";


set the data path crate on their mountpoints ?


I see I have omitted having constants for the constructor in all classes that have extended Animal including itself. A bit of a mistake on my part but since they are abstract classes I would have set them to 0 or null (maybe with the exception of WaterType).


Since I don't trust firmware to stick to a case for "human readable" wording anymore: better make this a `if "resend" in line.lower()`


Perfect! Thanks @Stasik0


Ok. I've got it.


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


you MUST provide
otherwise it won't work!


изменил тут


why are you talking about yaml syntax here? it's just confusing


fix-run-without-permissions-5062 (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (747 of 747 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.18%25-green.svg)


думаю, было бы хорошо валидировать наличие продукта у картинки


misspelled "canola"


are called the pre-installed  ?


Since we're going to evaluate this anyway we might as well just evaluate it here and then check if its empty rather than doing a separate DB query:
```python
if not group_summaries:
    continue
```


when a cluster ... are set up ?


I will delete these lines completely, they are in the git history.


"i" before "e" except after "c"!


I don't understand. You mean first write description and then method name?


Am 07.08.2015 um 14:01 schrieb Danny Arnold:
> Exakt weil MVC eher locker definiert ist hatte ich Anfangs vermieden es
> als Analogie zu verwenden, da jeder seine eigene Interpretation,
> abhängig von Frameworks die er genutzt hat, Anwenden würde, und es damit
> als Domainterminus eher untauglich ist.
Es gibt strenge Definitionen, die werden nur selten umgesetzt (s. dein
Anwendungsfall). Das Muster MFC ist aber jedem Informatiker ein klarer
Begriff und sollte in der Beschreibung genannt werden.

>
> Zum Beispiel habe ich angefangen Teile des DOMs als View zu bezeichnen.
> Normalerweise wird das Template oder der Code, der die DOM Elemente
> verwaltet, View genannt.
> Man kann bei GUI ToolKits auch Introspection benutzen um sich die
> Struktur eines Fensters ausgeben zu lassen oder beliebig Werte von
> Labels und Buttons ändern zu können. Aber ich denke auch in diesem Fall
> würde man das was einem die Introspection liefert nicht als View
> bezeichnen. Klar man kann, aber wie nennt man dann dieses XML templates
> dass die Grundlage für das Fenster war?
Layout beschreibt die View. Dargestellte Daten sind das Modell.

Kann man auch View nennen. Man
> kann das Fenster auch programmatisch erzeugen. Wie nennt man die Klasse
> die das Fenster erzeugt? View?
Ja, View. Bzw. wenn die View durch eine andere Klasse erzeugt wird, dann
ist die andere Klasse eine factory.

>
> Es ist leider alles nicht eindeutig definiert durch 3 Worte.
>
> Das DOM ist das worauf man über ein Introspection Interface zugriff
> erhält. Es View zu nennen, sollte helfen es von dem JavaScript Code zu
> unterscheiden der es Manipuliert.
Ist der DOM nicht das Datenmodell? JS ist das Control. Die
View-Komponente kommt durch den Browser, der den DOM rendert ins Spiel.

 Tat es scheinbar aber nicht...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/despairblue/furry-octo-dangerzone/commit/36d2576511d93faaf4b94ccd360650156beaef1c#commitcomment-12585402>.
>

--
Matthias Lenk, M.Sc.
Technische Universität Bergakademie Freiberg
Universitätsrechenzentrum
Bernhard-von-Cotta Straße 1
09599 Freiberg

phone: +49 (0)3731 / 39-4789
mail: lenk@tu-freiberg.de


Wrapping in jQuery breaks coffeedoc :/ Letting it go as the noconflict is more important.


Yeah thought about that too but didnt had the time to change it. Actually have some spare time will do it through the mobile site. Still have to think about the Format error because octoprint closes connection but I'm not sure if the format error still gets thrown when a successful Resend was done.


thanks for your help with the review. should be corrected now

/m


моунтить нужно один раз без указания версии
```ruby
mount_uploader :picture, ProductPictureUploader
```


Mario, is this a work in progress? I see the option being introduced - but no code actually using it.
(Assuming that this is a new option, never before implemented in some way.)


ref #262


нужно только одно поле
```ruby
t.string :file
```


First off: this is already great work :+1:

I do have a few suggestions though:

* i do not like the `user_measurements` method in the application controller because I think it is only used in one controller?
* I think `current_user.measurements` is very clear and informative and should not be memoized (rails does that for you) (so there is no need for this method)
* I would use a instance variable `@current_measure` which I would set in the controller, and in the views you can then test if it is set or not (instead of doing `user_measurements[-1]` or `user_measurement.last` or `user_measurements.any?`)


@wesleytodd Can you explain the purpose of this? That doesn't `return` does it?

```
if(ended) {
    false;
}
```


Oh, sorry. I had problems because I had forgotten to add
this.factories = new SpeciesFactory[Constants.Species.SPECIES.length];
so some constructor was issuing an exception and since it was being in a try clause took a while to trace. I probably should have used the print call stack then.
For some reason I used prints to debug and forgot to delete them.


Well, I had named Tattoo's dart game file as tattoo_darts.cpp, so it seemed to fit in better with the convention of having sclapel_* and tattoo_* prefixes for file/classes present in both games.


LGTM. Did you try "make noise"? It is useful to hunt this kind of bug.


Note: this still skips the binder :) it just does so in the implicit way I would eventually like to remove altogether (by making the `0` field private). But there is a comment and assert so...good enough.


@Sunilkumarr remove the todo statement also


Why are you ignoring CaptchaParser.py? The repository is supposed to be independent, unless it is a [Git module](https://git-scm.com/docs/git-submodule).


btw, `date -u` exists instead of setting `TZ=UTC` (e.g. i consider it more readable)


@alexsoble: :+1:

On a related note, both Uri and Sam said that the `placement` column in the "SPED & Disability" category is not useful since as a teacher you know where the student is placed: *"S/he is in your classroom."* Sam did say thought that it might be useful for a special education specialist who is assigned students from different types of classrooms.

Uri and Sam separately said that they would exclude `placement` and keep {`disability`, `level`, `504`} as the key default variables for the following reasons:

- `placement` not particularly useful for a classroom teacher
- reduce clutter


It is a cure for this:
![bmesh_sockets2](https://cloud.githubusercontent.com/assets/7894950/9133887/e7fcd538-3d07-11e5-97d7-5ceda8b9ca62.png)

Now there are errors when updating the individual nodes, but when the entire  node-tree is updating, all nodes are operating normally.

![bmesh_sockets1](https://cloud.githubusercontent.com/assets/7894950/9133810/3d76bfb6-3d07-11e5-8cd6-899d38ed92ef.png)


Be careful to read first the terms of use Google Map licence. Free does not mean with no rules !
https://developers.google.com/maps/terms


Also ignore `.DS_Store`, and other unnecessary files generated by the OS.


Thanks :)


@kkrampa there is no need to do a count here. You should just run the query and if there are no results then return 0.


I think that this is not much safe... I had to do hard reset my windows (CTRL+ATL+DEL was not working) and I guess this is the reason.


GRRR


* yes, but it is really ok. I don't even know if is an issue anymore on recent Visual Studio version, but for now we are always building yarp-wbi as a static library so the are no issue.
* Different bracing style  in src/yarpWholeBodyActuators.cpp , but again feel free to ignore.


@micheh It is to fulfil this rule: `If the path is starting with more than one "/" and no authority is present, the starting slashes MUST be reduced to one.`

You are right though that the `ltrim` is wrong. It should remove the slashes instead of whitespace.


not right


I mean standard ruby code documentation - Rdoc
http://rdoc.sourceforge.net/doc/index.html
http://inch-ci.org/github/ucetnictvi-on-line/logobox
http://trivelop.de/2014/01/26/about-documentation/


@kkrampa the `org_summary` and `soh_rows` queries only need to be done if `facs_count > 0`


There is still a problem with recordings claiming to have a certain fps that actually have a different fps because of the spacinf in the used pts


added rangecard to rangefinders
added maptools to basicitems
separated machinegunners into GhostMachinegunner and GhostAutorifleman
added Pistols
added F2000 to AssaultRifles, AssaultRiflesC and AssaultRiflesGL
added RPG32 to Launchers
added mtp bipod to Bipods
removed special carrier and GL vest
added ReaperVests
added UAVBackpacks
added UAVTerminals


Unless you think it is crucial for some debug reasons (when writing MetaDefinitions) I figured we would be processing intermediate code. If a code is generated when being a child of a MetaDefinition that means that that code is not actually generated as part of the real program's tree.


Looks like an unresolved merge conflict. Is this still there in your last version?


made a mistake?

----- Messaggio originale -----
Da: "Felix Obermaier" <notifications@github.com>
Inviato: 07/08/2015 14:17
A: "NetTopologySuite/NetTopologySuite" <NetTopologySuite@noreply.github.com>
Cc: "Diego Guidi" <diegoguidi@gmail.com>
Oggetto: Re: [NetTopologySuite] fixed solution build errors (231ffd4)

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


It's completely valid, only the first set of parenthesis is abundant.


Let's keep `*-test.js` glob, since where may be non-test `js` files


@matthijskooijman
Thanks for the response.

First of alll
> @ffissore, would it perhaps be an option to store the source and .o files inside sketch, but put the final linked .elf and .hex files directly in the build folder? This might fix most backward compatibility problems I believe, and it would still make sense (and not cause problems)?

I completely agree with this.

I understand the reasoning why you changed the platform.txt. However I think the solution you propose does not fix the problem completely. The problem is only completely fixed when the "build tree" maps the "source tree". An example which will not be fixed with the described solution
libraries/IRDistenceSensor/calculateDistance.cpp
libraries/SonarSensor/calculateDistance.cpp
There will be 2 files called libraries/calculateDistance.o which means one gets overwritten
This is so because
{source_file}=libraries/SonarSensor/calculateDistance.cpp
{object_file}=libraries/calculateDistance.o
If you change the definition of object_file so it becomes
{object_file}=libraries/SonarSensor/calculateDistance.o
you never have conflicting file names in the build as that would mean conflicting file names in the source.
This is how CDT and as such the arduino eclipse plugin works. Note with the existing platform.txt.

**What I'm trying to say is that what you are trying to achieve is a good thing, but it does not need a change in the platform.txt file.**
See the screenshot of my Arduino eclipse workspace which maps the source tree in the build tree(without weird things - I swear-)
![screenshot-arduino - eclipse arduino ide](https://cloud.githubusercontent.com/assets/1175841/9135541/01be0ec2-3d0f-11e5-86bb-de18ded1d912.png)


**Rick:** Cute. Your sister's boss gave me a microscope that would have made me retarded.
**Morty:** Ooo, oh boy Rick, I-I-I don't think you're allowed to say that word. Ya know?
**Rick:** Uh Morty, I'm not disparaging the differently abled. I'm stating the fact that if I had used this microscope it would have made me mentally retarded.
**Morty:** Ok but yeah, I don't think it's about logic, Rick. I-I think the word has just become a symbolic issue for powerful groups that feel like they're doing the right thing.
**Rick:** Well that's retarded.


:P true.


Thanks!


:D :+1: :clap:


^ continued..
**Morty:** : Oh you Git!


Don't like this one-liner. Can we split it in two lines. One for ternary assignment and the second with the `push`?


Is there any meaning to test this once again?


upload**s**Path?


I cannot push it as part of the repo no? That doesn't make sense.

-----Original Message-----
From: "Aneesh Neelam" <notifications@github.com>
Sent: 07-08-2015 19:14
To: "kishore-narendran/FacultyInformationScraper" <FacultyInformationScraper@noreply.github.com>
Cc: "Kishore Narendran" <kishore.narendran09@gmail.com>
Subject: Re: [FacultyInformationScraper] Fixed scraper and updated .gitignore(9e54028)

Why are you ignoring CaptchaParser.py? The repository is supposed to be independent, unless it is a Git module.
—
Reply to this email directly or view it on GitHub.


Was ist ein radiobutton-view?

Ich glaube das alles hier führt zu nichts. Es findet kein beidseitiger Informationsaustausch statt. Du versuchst aufs äußerste die dir geschilderten Funktionalitäten, in dein Konzept von User Interfaces, Graphikbibliotheken (was wahrscheinlich alles aus der Javawelt stammt und mir völlig unbekannt ist) und MDD (bezogen auf deine Projekte) zu pressen.
Es ist ja per se nicht Verkehrt sich zu fragen wie Sachen in die eigene Weltanschauung passen. Aber wenn man was neues lernt ist es vielleicht besser wenn man sich erstmal darauf einlässt und später überlegt wo parallelen sind, anstatt es die ganze Zeit umzumünzen in seine eigenen Definitionen und sich dann über Inkonsistenzen zu beschweren.

Nach dem das gesagt ist, bin ich fast soweit das ganze MVC Zeug wieder rauszuschmeißen da es zu sehr verleitet. Das Model ist zu vage und es hat dich scheinbar noch mehr in die Richtung "Ich kenn das, aber es ist anders, dass kann nicht sein" zu bewegen.

Wir sollten uns mehr darüber unterhalten. Kann ich bis nächsten Freitag bei dir einziehen?

Grüße!


One more remark looking at the screenshot
In the plugin (and cdt in general)
{object_file}={source_file}.o
The reason for this is to cater for
{source_file}=libraries/SonarSensor/calculateDistance.cpp
{source_file}=libraries/SonarSensor/calculateDistance.c
{source_file}=libraries/SonarSensor/calculateDistance.S
{source_file}=libraries/SonarSensor/calculateDistance.ino
....
And it is just lots easier to add ".o" than replace ".c" or ".cpp" or ".S" or ... with ".o"
As gcc caters for creating the folders this would be a simplification of the arduino IDE code.
Best regards
Jantje


@dotta will review/reply soon, frantically prepping slides for webinar this-evening now :)


It better refactor this arrow function in the separate method, so we will have:

```js
var processTasks = fileNames.map(fileName => this._createProcessTask(fileName));
```


nit: "in a consistent..."


OK. You can use logobox helper anywhere you need. Thenk I suggest to make pure ruby example:

```ruby
Logobox.generate_logo_url(27169278)
=> "http://www.logobox.cz/getimage.ashx?id=CZ27169278"
Logobox.generate_logo_url(27169278, :medium)
=> "http://www.logobox.cz/getimage.ashx?id=CZ27169278&size=medium"
```

What do you think about it?


so so


nit: "operator's" => "function's" ?


@janfriedli I made the sidebar show ob on ``/`` on purpose. Otherwise the contents start at a strange point. I propose leaving this as it stands since we should redo the start page somewhat soonish anyhow (in another PR).

I think this is probably more or less ready to merge.


I have refactor the code #604


304代表文件没有变化，这个时候应该不执行任何操作，也不打任何Log, 这个问题怎么讨论了两天。。。
而且当返回304的时候其实也不用验证任何哈希信息，因为服务器已经验证了，304就代表没区别了。。。


Can be rewritten as:
```js
var result = await Promise.all(processTasks);

result = result.filter(value => !!value);

return result.length ? result : null;
```


Do we need these changes? (same below)


Hmmm... that doesn't work well with my allocator-related changes because not all packages have a `package.d` file.


FYI, you don't need any `@Inject` on constructors in restx : as soon as you're working with `@Component`, injection will be made through constructor

You can keep the annotation though if you think it makes things clearer for people used to CDI


Why these are instance methods?


+1 Thanks for the fix.


Of course you can, put the same exact file in this repository as well. After all, it is supposed to be self contained, or there should be a script that sets it up anyway. However, a better solution would be to ask @karthikb351 to upload that on [PyPi](https://pypi.python.org/pypi) and similarly, on [npm](https://www.npmjs.com/).

For example, there is a logically similar but syntactically different captcha parser in the [VITacademics API Server](https://github.com/aneesh-neelam/VITacademics) respository [here](https://github.com/aneesh-neelam/VITacademics/blob/release/api%2Flogin%2Fcaptcha-parser.js). Compare that to [this](https://github.com/karthikb351/CaptchaParser/blob/master/CaptchaParser.js).


Should not the padding be a uint64_t, so that the struct is aligned of 64 bits, and not 32 bits?


There is a method in my madness - the upper case fields are all the canonical sections of an SQL select query - and as it goes through the various processing steps various things are added/subtracted to the record and fields (like the 'WHERE' field are rewritten/restructured - using upper case matches the 'style' of SQL and makes clear where various things came from in the processing pipeline


Should not `int fd` also modified?


I don't get it, `_processFiles` should return array or results. BTW, what are `results`?


Edited


Изменил.


I'm having some issue with the aux tag parsing on some data. Say I have a read alignment like this:

    SRR1545849.11:CELL_A12:UMI_TCTCTT\t0\tENSMUST00000188299\t1\t255\t41M\t*\t0\t0\tATCAACCTGTACACTGTTATCTTCATTAATATTTTATTATC\tggggiiighghhiiiihiihiiihhiiiiihiiiiiiiiii\tNH:i:1

Firstly, the `libhts.bam_get_l_aux()` method returns 4 for alignment to `l`, so it does not satisfy `0 + 4 < 4` and will not enter the loop.

Secondly, I do get the correct beginning of the tag from `s_aux_ptr`, i.e. `NH`. But `chr(s_aux_ptr[2])` return `'C'` rather than the expected `i`.

Similarly `hts.bam.ffi.cast('uint32_t *', s_aux_ptr + 0 + 3)[0]` return 21186561 rather than the expected 1.

(Actually `hts.bam.ffi.cast('uint32_t *', s_aux_ptr + 0 + 3)[1]` returns 1, but I'm not sure of this is a coincidence)


With the change on [line 35](https://github.com/camon/Flexget/commit/30f4ddf845e18d2af1711a2ab4cb31ae9ef9cb9f#diff-1f673d02eaa31c3375dfb13fb9336507R35) this import is unused.


Wie du siehst geht es nicht.

Entweder ich schreib das kurz auf und der Leser denkt sich dass er gerne eine Begründung hätte, oder ich schreib es ausführlich und der Leser denkt sich "Wen interessiert das?".

So geht's mir übrigens bei vielen Papers. Ein Haufen Behauptung wo ich mir Begründungen wünschen würde, aber dann wäre es ja wieder Prosa und kein Paper....


@JoeBengalen I see. Thank you!


No need to change `int fd_sess` ?


This is not indented, so it's a top-level function and not a member of the SearchView class...


Looking at it that record should be riak_sql_v1 but riak_sql_select_v1 - will raise a JIRA to that effect


Would prefer to have the change in library part of another commit if possible (same in Makefile)


This is an unused import


Nevermind, I see the other commit now. :-)


*ahem*


Shouldn't this (and the following) be `convert(Cint, 0)`? I get an error here.


Yeah, you're right.


Да, я обращал на него внимание при разработке.
Но это ведь метод рельсов, а не Руби.


Nice spotting.


Shouldn't we use the constant `DEFAULT_TRANSPORT_CLASS` here?


Argh, just figured it out. Nevermind. However it would be nice to have a variable for the processor, instead of direct passing.


I think [this is LuaJIT failing to build](http://build.voidlinux.eu/builders/armv6l-musl_builder/builds/2807/steps/shell_3/logs/stdio), not efl.


And -1 for `fixtures` dir.


clang has keen eyes.


BDD test names should start with the `Should` word and have descriptional form.


Changed it to sql_compilation_assert


Use split chain by lines.


nit: missing space after comma


nit: "operator" => "function"?
Is PE termination really the most desirable / helpful behavior if the runnable throws?


So, it's `fake` or `non-existent`? Typo in `non-existen` BTW.


Where did this come from?


I suggest to use some gibberish for this puproses, e.g.: `this/path/will/not/exist/ever`. Since, 'test/server/data/upload/none-existent/' maybe used one day.


We should use temporary dir for this purposes (using [tmp](https://www.npmjs.com/package/tmp)).


\r-


This fixes the original issue I raised, #174 , so that one can be closed.


nit: use the naming pattern used elsewhere in the api: FunctionContextImpl?


The origin is in https://github.com/mysociety/alaveteli/commit/80bb95a68258c76b8d4c9af66a1105c96e410424 which just talks about a broken xapian setup. OK, I'll reintroduce it in the limited form, particularly as we still have logic for the nil case in the view, although I don't really like that we don't know what specific error we're rescuing from.


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


saw approval from gordonguthrie
at https://github.com/basho/riak_kv/commit/13815ed7322522161cf8957fef77544c983f6128


merging basho/riak_kv/bk-rts-232 = 13815ed7 into borshop-integration-1160-bk-rts-232


basho/riak_kv/bk-rts-232 = 13815ed7 merged ok, testing candidate = 507c6696


Thanks for the patches. Initially, I liked to limit to usage of set_config_param, as it not easy to parse and modify a bareos config file from shell. However, as the other settings did work well for a while, I agree that it an advantage that also these parameter gets set.
I will run some tests on it and push it then to master, together with the other patches.


bad commit...


Cool to know. Thanks @fcamblor !


Fixed


see also #106


Maybe do ` -Dffmpeg_soname_version=$pkgver`?


Yeah, if we can find out what to rescue then great; if not, we'll review it in #2704.


related to #106 (opened later than this commit)


This can be removed since PlatformTarget will have the value `null` by default. If you want to be explicit, you can set it to `null` directly.


I see what you mean. I realize, by doing this, I also violated the initial assignment and I introduce a not very useful piece of code. I will look into this again. Thanks


Isn't the default hierarchy mode "reference"?


Thanks for the catch, fixed.


afaik many irc networks block tor users as tor makes it harder to ban ppl ^^
i would vote yes to it :)


I think just changing [^:] to [^:@] here will do what you need (without needing the extra sed below).

(Also, the indentation of the sed lines is weird...)


It's nice to use regex, but this doesn't work since it doesn't return anything...


Very nice! Is this line here on purpose, though?


Thanks for reporting. Can you try the latest commit and let me know if it works for you? It may need more change, but fixes the issue you found.


You just have to change 'it' to 'fit' for your test in order to run only that test. No other changes on config are required.


I already post them as post https://iamstarkov.com/fav-npm-scripts/


I think the `[]` around the parameter `file-name` is confusing. It suggests either that the parameter is an array, or that it's optional.

I think replacing it with an example filename, like `my-shape` (does it need to be a string: `'my-shape'`? test to find out), would be better. You could then explain that `my-shape` looks for `images/svg/my-shape.svg`.


@spocke: My `<body>` has relative positioning and the following CSS:
```css
  position: relative;
  min-height: 100%;
  height: auto;
```
The fix (adding `$(body).css('position') === 'static'`) broke the positioning of all tooltips and menus.
I'm initializing the TinyMCE when certain textarea is clicked. So when I click one of the textareas when I didn't scroll the page - everyhing is fine. But when I scroll down a bit and initialize another TinyMCE from another textarea (that is somewhere at the bottom of the page) - the tooltips and the menus are totally dispositioned and remain at the top of the page.


I think the examples should be above the mixin. In most documenting systems, all documentation about a thing comes before its definition.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2264


I don't think `display: inline-block` on the parent element is necessary for these examples. Perhaps they could explain how `svg-icon` sets `display: inline-block` because its usage is meant for icons, which are usually inline.


YES! It is functionally necessary :stuck_out_tongue_winking_eye:
Thanks for the catch!


We don't need this.  Returning false should make it's way back to the where all failed annotations are logged.  You need to turn on detailed logging to get the messages.


I thought the rule was only on same line if there are no arguments in the annotation


I was not done it seems...
Now I am https://github.com/NetTopologySuite/NetTopologySuite/commit/7954030cc98c90c88860a20ec79b767a33fe9ea1


We don't really need this variable. We could either a) set aux == NULL to mean "do not free" or b) do a Py_INCREF(aux) below and then always do DECREF(aux). I prefer (b) by the way.


Should be able to construct a unit test for this


Py_INCREF(aux) here.


Why are you not logging this error anymore?


Thanks for the patch. Added some comments.


Should be able to unit test this too


Interesting indentation here. Is the comment still relevant too?


The ticket is missing here.


Tempfile добавляет постфикс в виде псевдослучайного набора букв и цифр определённой длины. Мне это не подходило, так как есть соглашение, что название создаваемого класса = имени файла таблицы в единственном числе и с прописной буквы например, таблица persons - класс Person.


perfect, squash to "Extended MasterConfig for feature flags" for merge


Yes, I'll do that, it also makes it easier to copy for the other opera streams :p


We used double space after `.`. (old Emacs tradition)


Thanks @bdmock... does this commit have all the markups picked up?


it's there also. You mean delete the rails one?


I don't really know if OTHER is right in this case, probably CLIENT_DOES_NOT_HAVE_PERMISSION?


Для удобочитаемости?


Should be from 0 to 3 (1 is currently missing).


I set row level and didn't find the message


Ок, тогда mktmpdir из http://ruby-doc.org/stdlib-2.0.0/libdoc/tmpdir/rdoc/Dir.html#method-c-mktmpdir


@flavour Please review


Справедливо.


Are the parentheses intentional?


I added additional line to make the code more readable, has no performance impact afaik.


No.  There are still some sweeps and families.   I have made all the markup corrections and was just waiting on the word on the sonotubes before syncing again.

I'm walking into a meeting but, as soon as done, can re-sync either with piers or with a floating slab.

Sent from my iPhone

On Aug 7, 2015, at 09:10, Ryan Schultz <notifications@github.com<mailto:notifications@github.com>> wrote:

Thanks @bdmock<https://github.com/bdmock>... does this commit have all the markups picked up?

-
Reply to this email directly or view it on GitHub<https://github.com/OpeningDesign/Sports_Complex/commit/9fea4a237cdc2c36c8f6cae812c2249421445063#commitcomment-12586573>.


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


3: L102 copyright year is outdated, expected 2015 but got 2014


Hi, yes in the latest commit it returns the tag with the correct number!

It still has the type 'C' though for some reason, but the NH tag is there and the numbers are correct so I can parse them.


Now that you have LinkDimension returning false, that's the one that should cause the message to show up in detailed logging.


Your're right :)


:) Its really great to have this in cmake now rather than in the the support scripts!


Yes, this one I probably forgot to fix. Just removed the underscore.


Let me check and fix. Sorry for the mis


Test


fixed by method removal


```ruby require 'yaml'``` вынес за модуль.
Спасибо за совет, но ActiveSupport не использовал.


The new component should not inherit from MenuView from my point of view - it uses a different system of options, which are Views unlike the MenuView items, which are javascript objects, and also MultiselectMenu does not have any submenus, and it should not inherit this from its parent, so i decided a new component extending the panel would be the best solution.

I have rewritten all methods completely because i changed the class functionality once again which now relies more on itemViews. The only method intact is the menuSize method which would remain the same anyway - the menu needs to be of the same size as other opened menus.

fixed


n -> nic


> Es gibt strenge Definitionen, die werden nur selten umgesetzt (s. dein
Anwendungsfall).

Ein Link zu einer Spec oder einem Design Document wäre cool. Ich vermute eher, dass jegliche Definition zu Abstrakt ist als, dass sie eins zu eins umgesetzt werden kann und es eben an die jeweilige Domain angepasst wird. Sollte es diese strenge, offizielle Definition geben wäre ich daran interessiert sie zu lesen.

> Layout beschreibt die View. Dargestellte Daten sind das Modell.

Das ist eine idealisierte Ansicht. Normalerweise ist z.B. der Text in labels ein primitiv. Also kann man den ändern und sein Model, was eine Datenstruktur im Speicher ist, bleibt unverändert. Im Endeffekt sind dann View und Modell inkonsistent. Genau das Problem was ich beschreibe.

Allerdings finde ich die Idee interessant z.B. den Text von Labels direkt mit dem Modell zu koppeln. Man ändert programmatisch den Text im label und automatisch (über einen setter z.B) wird das Feld im Model oder gar in der Datenbank aktualisiert. Das sind Funktionalitäten die angular, oder meteor mittels data binding umsetzen.
Leider funktioniert das nur bei primitiven Views. Hat man zum Beispiel computed properties, oder noch andere Logik die das Model transformiert bevor es dargestellt wird, geht das nicht mehr. Man muss eine Rücktransformation definieren. Das ist zum Beispiel genau das was bei scegratoo mit den Transformationsattributen passiert: https://github.com/despairblue/scegratoo3/blob/master/scegratoo/public/scripts/services/treenodeattributetextbox.js#L27
Ein String wird als drei oder vier inputs dargestellt. Data Binding ohne die Möglichkeit Transformationen zu definieren würde das nicht zulassen.

Also zu sagen dass die Daten die im DOM sind das Modell sind, ist eine schöne Theorie, die Realität verlangt aber leider etwas komplexeres.

> Ja, View. Bzw. wenn die View durch eine andere Klasse erzeugt wird, dann
ist die andere Klasse eine factory.

Und da sind wir wieder bei einer Definition die das DOM nicht View nennt sondern den tree-view (den javascript code). Merkste dass wa uns im Kreis drehen? :)

> Ist der DOM nicht das Datenmodell? JS ist das Control. Die
View-Komponente kommt durch den Browser, der den DOM rendert ins Spiel.

Das Datenmodell ist der Scene Graph, meiner Meinung nach. Der JS Code erzeugt DOM Elemente nach diesem Model und packt sie ins DOM, und erfüllt damit deine Definition eines Views. Der Controller, also der Teil der Events behandelt und das Datenmodel anpasst ist auch JS Code. Die zwei JS-Teile auseinander zu halten ist, schwierig. Und man sieht auch vermehrt Ansätze, wie React, die das gar nicht erst versuchen, weil es eh nur Verwirrung stiftet.
Bleibt der Browser. Der Browser ist dann wohl eher etwas wie Qt oder GTK oder Windows Presentation Foundation.

Ich glaub die Differenzierung gefällt mir. :relieved:


Is there any difference in privatizing all together, eg. if you don't need some privatized is there any perf degradation if you don't do them at all vs privatizing all at the start?


I agree with you about the spelling of colour but the machine seems to insist on American spelling.


:ok:


Well, yes and no. I thought we have a recursive procedure, which will gradually process all such code, attach it to the meta call (regarless where that meta call is) and then go on to process the meta call's parent (if its a meta def).


you need to revert this - the point was that the Client handles timing, and we did not want a user to mistakenly not include timing information or include timing information and think it would be saved. directly putting in the runStatus like this is confusing, which is why it wasn't in the API


No worries, this is not urgent. But I'm definitely interested in hearing your thoughts. If simpler, we could also do a conf call.


This looks particularly useful. It's been a while since I used this, but were the instrumented sig functions filtered elsewhere?


I would never have guessed at tmux.


I would recommend continuing to output system.err messages that concern file write errors.


this all needs to be reverted once the api is back


Note taken, thanks. Wasn't exactly clear from the chat, seemed like some kind of a typo. :)


Thanks, fixed.


yes, I did see an issue with duplicate highlight after a new game and this looks good. please merge or let me know if you want me to merge. thanks, Scott Lee


I don't understand how this can happen.  If backfill is in progress with the wrong sort order, we actually do need to restart backfill even if we have a full log.


Meant to add that after making the pull, whoops...


This should go before `row` in alphabetical order.


Huzzah! Thanks @vargaz


what happens if we do this call twice? ie if everything always exists (the normal case)?


I don't think so, but it's been quite a while since I committed this. But AFAIK the patch made everything work as expected.


Скорее, для традиционности. Ну, и раз уж коснулись скобочек у методов, у `{` и `do .. end` разная ассоциативность, что иногда приводит к конфузам, типа как http://stackoverflow.com/questions/5513531/ruby-do-end-vs-braces, и иногда `{}` на конце могут быть блоком вместо хэша и наоборот в каких-то случаях. Короче, лучше использовать скобочки :)


Остальные require тоже надо бы.


Since `private` is an operation it consumes time executing, so batching them up can save some execution time. Might be minor though.


Upgrade guessit to get rid of the "IndexError: string index out of range" that prevents found subtitles to be saved.


It is probably good to use such an extra bonus not only in RRQ but also QQ endings.

And even more general, to all endings without minors.


Ok, I think it's best if all privates are always in the beginning of the file either way, maybe if there is a huge part of the function that is expected to execute rarely it might be better to separate. Just a thought.


Thanks for your work David, there are quite a few terms I have not come across yet in this one !!


ok tks very much

On Fri, Aug 7, 2015 at 2:15 PM, mdouchin <notifications@github.com> wrote:

> Be careful to read first the terms of use Google Map licence. Free does
> not mean with no rules !
> https://developers.google.com/maps/terms
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/3liz/lizmap-web-client/commit/8d1412f0ead31a191ae2f4d214bd1269b1812a71#commitcomment-12585633>
> .
>


I agree and that's actually the policy I try to follow myself.
Obviously sometimes it's hard to judge, so when in doubt just throw it together in the beginning.


Have you tested this? There was a brief discussion if parentheses are needed or not, but don't think anyone actually tested it.


While this will work for now, it also forces the user to answer every question. (Even though they should) If they don't it will count (likely) every following answer wrong.


@jwinder How about a pull-request command too:

```lisp
(defun hub-pull-request ()
  (interactive)
  (async-shell-command "EDITOR=/usr/local/bin/emacsclient hub pull-request" "*hub pull-request*"))
```


there is no need for that - there is already `jazz_hands` added


This commit is being reviewed as a part of review issue #52.


Okey


thanks, good catch.


密码明文啊。。你傻吗


*296 :P


why make it an option? i think
```
set(HAVE_BETTER_YASM_ELF64 1)
```

would suffice.


Описание задачи - https://github.com/ideals/Ideal-CMS/issues/353


that's not an optimization ... that makes the code less readable, imo


sorry sebv i'm not sure exactly what you mean by this comment. are you saying i shouldn't put rimraf in fs? it seemed like appium-support/fs was where we were putting file utils.


Yup, that is the change I made.


Made an error with two lines. I've updated the process I use to do these so hopefully wont be a repeat. Sorry. New pull request coming up to correct issue.


@tomchentw thank you so much for including this functionality in the main repo!

Did you keep the functionality of setting a class name on the DOM element where the infoWindow/infoBox child is mounted? I love your new code structure, but I think it's important to also have this in. (because there most likely will be the need to style the parent element of whatever content I put into the InfoBox, given that the parent element can be an empty div with no css class to reference it by)

For reference, I used to do that [here](https://github.com/tomchentw/react-google-maps/pull/69/files#diff-ed889797ca9b724744c832876c6f80c8R63). Thanks!


I was meaning to do that. The initial version works well for `find`, not so well for `update` and `remove` :)


yeah, I changed so I tap on it after it's visible.
That and I was referencing a wrong locator too.


if the google maps use is only view? in other words no legal disputes, no
print, no evidential use, but business webgis yes.
why the key field in lizmap plugin baselayers tab if is it for free (with
rules)
thk again


Good idea. :+1:

I just tossed this together. I plan to revisit more commands later. `hub browse` for example should allow for reading a repository name.


This needs a similar logic from the one in `_reduceNeighborColsWidth` for handling the smaller column first, but handling the greatest column first instead. If you don't do this and a much bigger column is passed last, the first won't get as much from `size` as it would before with the loop.


and for simplicity, could also use
```
if(NOT rc)
```


lowercase command for consistency, so
```
execute_process()
```


Ich habe gesehen, das einige Traps in Natura keine Rezepte haben. Vielleicht nochmal anschauen.


Can do. Personal preference I guess.
And yes I tested this, since all of them are right-hand operators, this works just fine.


Sub StartSpelling()
   'Set up your variables
   Dim iRow As Integer

   'And define your error handling routine.
   On Error GoTo ERRORHANDLER

   'Go through all the cells in column A, and perform a spellcheck on the value.
   'If the value is spelled incorrectly, write "Wrong" in column B, otherwise write "OK".
   For iRow = 1 To WorksheetFunction.CountA(Columns(1))
      If Application.CheckSpelling( _
         Cells(iRow, 1).Value, , True) = False Then
         Cells(iRow, 2).Value = "Wrong"
      Else
         Cells(iRow, 2).Value = "OK"
      End If
   Next iRow
   Exit Sub

    'Error handling routine.
ERRORHANDLER:
    MsgBox "The spell check feature is not installed!"

End Sub


Exactly :+1:


I like my book titles the way I like my variables - self explanatory :smile:


Lovely!


I guess you want to use ${CMAKE_SYSTEM_PROCESSOR}-${CORE_SYSTEM_NAME} here.
see http://www.cmake.org/cmake/help/v3.1/variable/CMAKE_SYSTEM_PROCESSOR.html


Shouldn't the values be 1 and 2 instead of 0 and 1?

If they are both in the same branch, then the index in the array can not be the same for the winner and loser.


Il commento è stato eliminato, spero di non averlo fatto io l'errore, comunque i commenti sui metodi metteli prima in javadoc, così lo vediamo ovunque il commento
```javascript
//pone le pedina nella posizione iniziale
```
tipo:
```javascript
/**
 *pone le pedina nella posizione iniziale
 */
public void InitialPosition(){ ...
```


should be static


This check is added


> I can't find a reference to the conversation ATM, but the Geany maintainer has in the past refused to allow Python to set spaces despite it being the PEP8 recommendation. And indentation is significant in Python and tabs are assumed to be 8. His reason is, IIRC, that filetype settings override user settings, so if we set it in the filetype it will not allow the user setting for that language.

Exactly.  If we had some kind of UI for overriding these filetype-specific settings I'd probably be happy to have more natural per-language defaults.

> Although to my mind that is the point of per filetype settings.

It is;  but until it becomes clearly discoverable I think we should not override the global preference unless absolutely needed (as @kugel- mentioned, i.e. for Make).

(maybe a UI could start by showing a warning that languages X and Y have specific settings next to the global setting, but it would be better to be able to edit those specific ones easily)


:+1: Thanks


We could definitely keep them but only if they are used someplace - I grepped and noticed it wasn't ever used in the code, and it wasn't obvious how to load a device for the BGAPI backend with a vendor and product ID.


我擦忘记了我擦我擦我擦


check if players vehicle earlier to avoid wasting CBA_fnc_inArea


delete please


Yeah, I know he has to do that, I am just going to upload the parser as part of the repo for now. :stuck_out_tongue:


Спасибо за полезную идею! Это дал возможность не использовать include вместе с extend, где он требуется.


Hrm. That's a little annoying as this is Yet Another Daemon to run. It makes sense for bridges to other applications, but not for direct event collection functionality, IMO.


> Just a note, the keyword sorting is meant to be alphabetical, so i8 and u8 were technically correct, though your position is more logical.

I'm fine with "human sorting"


Mach ein Issue auf hier geht es nur um Pams Fish/Animal Trap ^^

Am 07.08.2015 um 15:33 schrieb Uwe Dörl:
>
> Ich habe gesehen, das einige Traps in Natura keine Rezepte haben.
> Vielleicht nochmal anschauen.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/combak/ShatteredWorld/commit/932daa6513610378648ec3d720c9aaac11e3e3d3#commitcomment-12586971>.
>


Note for self, should be `!`


Do you have recent benchmarks for that "8x faster"?

I know this trick from Nooku's object https://github.com/nooku/nooku-platform/blob/develop/library/object/object.php#L395 where is stated: "Call_user_func_array is ~3 times slower than direct method calls" and I know benchmarks from years back on php5.2 and 5.3 that also come to a max of about 3 times faster conclusion and often less, for instance http://www.htmlist.com/development/function-references-and-runtime-functions-in-php/ (2008), https://mwop.net/blog/121-Benchmarking-dynamic-functionmethod-calls.html (2006), http://paul-m-jones.com/archives/182 (2005). And to measure it well you have to take the time for the switch statement into account as well, so the total gain will be even less. It is also a gain in microseconds, not called in some loop. PHP 5.3 does call_user_func_array already much faster than 5.2 http://akrabat.com/php-5-3-is-quicker-than-php-5-2-2/ but it can even be that later versions make the difference with direct function calls even smaller.

Here is a more recent test from a year ago (don't know which php version), including the switch: https://gist.github.com/nikic/6390366 where direct call is almost twice as fast. It was done with 1 million iterations and the total output is in microseconds (1 millionths of a second), so you get differences for instance between 0.00000000000037865686416626 seconds (call_user_func_array) and 0.000000000000015234017372131 seconds (direct call) , that is 0.00000000000022631669044495 seconds difference for one call with 2 arguments.

How many times is it called in one program execution? First guess something like the number of plugin-methods that are called = number of plugins times the average number of methods by which the plugins are called. Lets say 100 plugins with 10 called mehods each, so 1000 calls. Then we have gained 0.00000000022631669044495 seconds (about a fith nanosecond), wow. As stated in the comments of that test http://www.reddit.com/r/PHP/comments/1m8g34/benchmark_of_call_user_func_array_vs_switch/ not realy worth it. Not even if you have 1000 times as much calls (10000 plugins with 100 called methods each, then the difference would be 1/5 microsecond, 1/5 of a millionth of a second). I would even say: not worth that ugly switch.

Or am I interpreting those tests wrong and do you have tests that back up your claim of 8 times faster (and examples that even that would make a relevant difference).

Even if you want to avoid call_user_func_array there are some alternatives, for instance in php5.4 more things are callable. But you are in good company to avoid call_user_func_array, for instance https://www.drupal.org/node/329012 (maar ook dat was al weer 2008).


Why is this in the model reader code?


Without changing the order of the inserts or ordering, I assume so. I agree that it's a lazily designed test. I like your idea better! :+1:


Missed `UTS_NS`


Oooh I like this! It's a nice feature to have


I guess the host must also join the IPA domain, too. Otherwise the ipalib API doesn't know how to connect to the IPA master. Right?


right, yes.

i guess this issue isn't closed. give me five minutes.


* move initialization from header to initializer list in source file
    int maximumBlockCount = 1000;
    QColor mColorBase = Qt::black;
    QColor mColorInputText = Qt::green;
* make sure all members are properly initialized in initializer list in order they were declared
* remove from header:
    class QColor;
* rename member localEchoEnabled to mLocalEchoEnabled, and other members you can find
* search for other solution instead of mCursor.positionInBlock() > 2 (you may do it in other branch as a new task, if you want - tell me)


2015? And I did not check the other files, I noticed only this one.


You should do all this stuff at runtime, not buildtime


no need to commit this one? :-)


format commit can be removed.


We should try to stick with the 90 character width limit where possible. It's nice to be able to see all of the code without needing to do horizontal scrolling.


It built and ran fine for me on this commit, and CTRL+F pulled up the search bar.


Good catch


Why this was commented out?


Thanks for doing this. Just curious... what was the use case for this? since you said it doesn't apply to OPNsense :)


nothing else to complain. looks amazing! :+1:


Yes, to my opinion, it is better stick to straightforward promisification of fs/rimraf/ncp , even if the api is not perfect , easier to read/write code.


clean code