# Emily Pilloton: Teaching design for change
---
## 1 TED talk link
[Teaching design for change][1]

## 2 Speaker
[Emily Pilloton][2]
## 3 Education Story
From TED
> Convinced of the power of design to change the world, at age 26 Pilloton founded Project H to help develop effective design solutions for people who need it most.

Emily and her team give things to the poorest students, they are hope and passion.

## 4 Inspiration
### 4.1 Design is Education
As Prof.Yang put it,
> 专家级别的开发者善用设计模式；
> Translation: Excellent developers are good at design.

Right now I am learning education system with a designer perspective, this is also the reason why I love this TED talk.
**Designing Model** does help, cause it can put yourself in other's position. Thinking from a designer view, you'll learn things in depth, in addition, you'll keep it for a life long time.
Next time when you fall in love with sth, be sure to get involved in it, and keep doing exercise while you are studying(练中学).
### 4.2 Make positive response in time
Why what happens after the training is as important as the training itself? According to a business white paper by Kevin Lovell[^1], what really matters is the response after training.

> #### Reason#1
> 13% of learners don’t use what they learn, don’t experience performance improvement and also lack line manager support.
>#### Reason#2
> Around 10% of learners receive training that is ill-suited to their needs.
>#### Reason#3:
> Around 5% of learners don’t use what they learned because of bad timing.
>#### Reason#4:
Around 3% of learners complete training, knowing they will not use what they learn.

Just take an example of learning English by most Chinese students. We started to learn English since the primary school, we've spent nearly 15 years on it. However, when it comes to speaking English or writing in English, we still feel embarrassed.
Why? I think the reason lies in the education system, we learned so much of useless knowledge, but ill-suited to our needs. When we got a red-colored score, say 59, that's the time we finally realize the learning outcome, still bad timing and ill-suited.
## 5 What are the lessons?

 - **Ensure training is properly targeted**
Not too hard, not too easy, but teaching students the life-long knowledge, such as *self-learning, designing perspective.*
 - **Teachers need to support learners after training**
 Making positive response, let students be proud of their work.
 - **Capture post training data**
> The best training course in the world is worthless unless learners subsequently use what they learn – so you need to know what happens after they go back to the workplace.

  For instance, the last 5 minutes of class is the best time to review newly-get knowledge.

  [^1]:[https://cdns3.trainingindustry.com/media/2624955/knowledgepool-they%20think%20it's%20all%20over.pdf][3]

  [1]:https://www.ted.com/talks/emily_pilloton_teaching_design_for_change
  [2]:https://www.ted.com/speakers/emily_pilloton
  [3]: https://cdns3.trainingindustry.com/media/2624955/knowledgepool-they%20think%20it%27s%20all%20over.pdf


> Which is why suddenly it's okay to not use `esc_html()` when the string already includes HTML seems weird, if someone is going to inject bad data they might as well just target all the strings which you don't escape - because they certainly exist and human error is just as likely.

For strings that cannot be escaped because they contain HTML, you secure them with `wp_kses()`. As this code did.

> The fact everything is filterable is a good example of why you shouldn't over escape, [...]

I think it's about keeping the API intact. If I call a function that by default returns a URL, I will escape it with `esc_url()`. Now if some plugin finds it convenient to add HTML in there, the plugin is doing it wrong, because now suddenly everywhere this function is called there's no longer a valid URL returned, but a string containing HTML.

This hasn't caused any problems with plugins so far on all of WordPress.com, as well as the WordPress.org themes that Automattic has, which all follow the same coding standards.

> Which is exactly the same as someone adding a rogue .php file to their site - a .mo file contains executable PHP and can do everything a plugin can do, just silently without people realising what's happening.

Valid. But I'm talking what escaping can protected against. For someone generating their own `.mo` files from `.po` files still is vulnerable to injections, unless that developer reviews all the strings manually or automatically for correspondence with the initial texts. Which is what escaping protects against.

> So, you're saying that since this is indeed a theme targeted to core, that having the extra escaping does indeed go against our established best practices (and so, should be removed)? I don't think this is the right venue to argue for a change.

I'm saying Core trusts translations, but everybody else shouldn't. Which is why the Core best practice is to not escape, but for the rest of the WordPress ecosystem it is.

So I'm saying escaping doesn't hurt, so let's add it even if the Core developers think it's not needed.


TeamCity mobile-music-automatization :: PR_BUILD_CONFIGURATION [Build 39](http://teamcity.mobile.dev.yandex.net/viewLog.html?buildId=1190103&buildTypeId=MobileMusicAutomatization_TestBuildConfiguration) outcome was **SUCCESS**
Summary: Running Build time: 00:00:03


@jasondegraw Thanks for your efforts to fix the equations and revising the content. I am OK with the changes. Please merge it and accept the doc ticket.


@shvburade don't use virtual phone numbers


I think the `tapAtPoint` call should be in an else clause. The code as-is works on iOS 9 but it breaks our tests when running on iOS 8. It seems to be closing the dialog and then performing an additional tap on the view that was under the dialog.


Maybe, but you didn't remove it?


@bwidtmann, I periodically see "Invalid argument" exception at this line of code. Seems like that sometimes the buffered.length changes (reduces) after video.buffered.length is read, but before buffered.start/end is read. It can be reproduced with the following stream:
http://dash.edgesuite.net/akamai/streamroot/050714/Spring_4Ktest.mpd
Could you please take a look?


Can we get an `npm publish` here?


Looks great to me: thanks!


Because the default isn't `0`, it's `false` -- and `false` may need to be passed in by something specifying `$digits` or `$hash`.  As the passed in value could be either an `int` or `boolean` I condensed it to mixed -- though it would be just as correct (potentially more so) to do `int|bool`


Trailing comma, this is invalid JSON.


Ahhh, I see. That makes sense.


This looks good


So, are you going to pay after all?
If no, you've no right to revert that commit.

It's not about using till the 20th of September, it's about we had a deal which you cancelled by your "have no money to pay".
Anyway, I'll wait till the 4th of September. By then I need an answer and appropriate actions (either removal or paying)


And this was part of the original code that got remove, I think it's pretty important to keep.


change the line with the following
 `return cls.status_alias_mapping.get(status, '')`


:pizza


Good question; It's inspired by the different association types in ORM, but I guess it doesn't really need to be distinguished at the storage layer. I think I'll remove ARRAY and map array-type to LIST as well.


why is the active assertion skipped?


Illegal one.

Let it be here until I'll be sure MoNTE48 will act like he should - e.g. according to the law.


I answered the email. I am against public discussion of these issues.


my original intent was to transfer const like that 'HELLO_WORLD' -> 'Hello World'.
That is why I didn't change the join. I suggest to keep it the way I did.


Hello, I cannot seem to get this fork working. I have replaced the files in react-select folder with the ones provided here and run gulp build. However, afterwards, when I include it in my web page, I am no longer able to interact with the select field. I can get the demo working just fine. Am I missing some steps to implement this fork?


The one-to-one associations which are represented to dialects as associations, too (see Husband/Wife example). There used to be a dedicated method `boolean isOneToOne()` which I removed in the favour of this enum.


I don't really follow what you want me to look at, nothing mayor changed here?


typeNames should be space separated instead of comma see https://github.com/geonetwork/core-geonetwork/issues/1163 for details.


thanks it is working now


You've removed tap_to_click_fix_filter_interpreter.h which is still included in several places.


I think you just received notification because you are watching the projectara/greybus repository on github.
You can always unwatch it. Sorry for the confusion.


I removed the `null=True` argument. Let me know if you find any others issues.


You missed a quote here.


The Entry API is so good.

> On Aug 31, 2015, at 08:31, Manuel Wiedenmann <notifications@github.com> wrote:
>
> Ha! Couldn't make it work.
>
> ―
> Reply to this email directly or view it on GitHub.
>


Kurze Frage - also das Diagramm beschreibt den Ablauf etwas anders. Erst CPR und dann Verletzungen behandeln.


Any side. It's meant to determine the embedding entity on each side of bi-di associations, so e.g. the schema can be prepared as needed in case of C*.


Well what nemes just said was my train of though at 4am or so.
Anyway the password and user are the same so i guess that could already be there.

Also dico, if i understand correctly you want to be able to pass other arguments to the target along with fetchall?


Because you cannot assume here that it is `unpaused`. I.e., another master switch could have happened in the meanwhile and `paused` the `updatePuller` again. In such a case we simple bail out since we that event is gonna rerun this code again.


Unhandled Fundamental Failure


I like the automagic but that's also a problem, it's magic to the complete Vagrant newbie (and yes we exist still :) )

It looks like everything after ln 53 "Launch the Atomic Host" is the same for both configs.  What if we have a advanced section with an internal target link that allows folks who know Vagrant to skip to the more complex example file from the Project?  This means adding new providers, adding more complex configs, etc. can be done in the maintained advanced config but there's always a basic guide that follows the vagrant guides.

Generally, I'd like to avoid direct links to artifacts in the docs, it's one less thing to have to police.  It's not a problem in the Vagrantfile as long as it's clear what need to be edited.


what is the benefit of this inversion?


Hmm, instead of first spliting things into an strv, and then iterating through the strv, shouldn't we just iterate through the string right-away? i.e. read one word, process it, read another word, process that, and so on?


This one looks great.


create project


현재 발견된 문제 : 가까운 곳 보다 멀리가 좋으면 아무리 멀어도 멀리 뛰어감 (map maze 참고)
가끔 의미모를 행동을 함


Fixed.


TeamCity ReOpenLDAP :: devel-2.5 [Build 106](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=238&buildTypeId=ReOpenLDAP_Devel25&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:36:11


It is needed since `HighlyAvailableGraphDatabase` constructs stuff in `createIdGeneratorFactory()` that is used in `createLockManager()`.


Never mind this Andy, your change is fine I think.


I deleted this line but it failed again.


Nit: needs to *be* added


今まで気付かなかった！


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


Go sucks, reloaded :-)


또한, 맵이 지나치게 커질 시 속도가 상당히 저하됨. fight 맵 기준 평균 150ms의 연산시간을 가짐.
더 큰 맵이 없다고는 보장 못하는 바, 대책을 마련하거나 타일이 많은 맵에 대해서는 다른 알고리즘 사용 고려


Fixed, thans.
https://github.com/mewebstudio/Purifier/commit/83dd3c9403ab732147c6c030a297f19e442d0534#diff-04c6e90faac2675aa89e2176d2eec7d8R49


In the case that the model (class Topic, in this case) does not include
the project_id, and it does not come in with the params, where *does* one
get it?

_______________________________
James Tucker, Programmer
Species File Group,
Illinois Natural History Survey,
Prairie Research Institute at the University of Illinois, Urbana-Champaign.
TuckerJD@UIUC.Edu
217-244-9416

From:  Matt <notifications@github.com>
Reply-To:  SpeciesFileGroup/taxonworks
<reply+001c0e126e861a8081f600d1e7457b4c799757f88ce7903492cf0000000111f8f12a
92a163ce00c5c0a0@reply.github.com>
Date:  Friday, August 28, 2015 at 23:03
To:  SpeciesFileGroup/taxonworks <taxonworks@noreply.github.com>
Cc:  James Tucker <TuckerJD@UIUC.Edu>
Subject:  Re: [taxonworks] Topic finder for otu_page_layout. (518c06c)

This is the right line, don't reference $project_id, it should only be
referenced in the models except through housekeeping.
�
Reply to this email directly or
view it on GitHub
<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_SpeciesFil
eGroup_taxonworks_commit_518c06cce4eb7f65e79f0ee53d1a12b5424b401b-23commitc
omment-2D12959904&d=AwMCaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=87FiFUsLQn0J2UnE2s49r
k_Ml7AdI6AGU3DZpzKZL1o&m=LjfLusMuvcakqu304mLQ5nKwi87XcqqGQMsTwhPkgXU&s=U-tJ
A4U6Fpzj5f6k1nI492vAtuS0cnPkLYUlk4sb29s&e=>.


Then LGTM. :-)


Dunno, it's a bit too hacky for me...

I don't really see a good short-term solution other than removing the trace messages. Long-term solution would be to have a separate attribute set containing the packages for nix-env.


Obrigado pelas correções! =)


I'd suggest `'nodeenv/' + nodeenv_version + ' (https://github.com/ekalinin/nodeenv/)'` which would match convension for user agent strings[1].  While the link below is for automated agents, the convention is typical for utilities and scripts to provide a generic name/version with a url for the utility in the parameters section.

[1] https://en.wikipedia.org/wiki/User_agent#Format_for_automated_agents_.28bots.29


@dcrowell77 - Can you answer this?  I don't know what the intended use is anymore.


@wluyima i see that you moved these into _base.scss. How do i reference these? Because i do not see your changes reflected in the UI.


Weshalb ist es relevant, ob die Ressource ursprünglich woanders eingebettet war? Vermutlich möchtest du unterscheiden zwischen "unfetched", "partially fetched" und "fetched"?

Wie bereits besprochen, halte ich "partially fetched" für bedingt sinnvoll bzw. kaum umsetzbar: Sofern die URI nicht aufgelöst wurde, weißt du nicht, ob die eingebettete Ressource nicht vielleicht bereits vollständig war.


first commit


@paulohenriquesi Existe no roadmap de vocês a implementação de transações incluindo antifraude ?


Did you try a   here? I wonder if that would resolve the warning in a slightly more attractive way (at least preventing the possibility if showing the user the word "void").


Ja, hast recht


This change break git usecase for multiple files diff (it ignore \n as whitespace). It should be at least
match = re.search('^\+\+\+\ \"?(.*?/){%s}([^ \t\"\n]*)' % args.p, line)


In commit message replace "Added" with "Add"


This is a client class thus is not used in the code. I think you only wanted to delete `RemoteProcessorServer::isStarted`


idem


- [ ] the function was only updating in case the ```transport_metadata``` is not empty dict. By calling directly the manager, you have to ensure the consistent behaviour.


I tried getting rid of the ng-app for soundcloudify.background and it's removal caused everything to fail. I'll play around with it and see what I can't figure out.

And sure, I can throw that in a run block.


maybe group function using functions() method.


its.  no apostrophe. :)


The other two were removed from the game, I have no information about them.


- [ ] remove unnecessary space on empty line


why set to null ?


Yeah, you're right. Will fix it. Thanks.


value에 값을 넣지 말고 placeholder을 사용해보면 좋을 것 같아.


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 basic_test|__suite_setup__|0|1|0|0
basic_test|_basic/test_create_vm_|0|0|0|1
basic_test|_basic/test_stop_vm_|0|0|0|1
basic_test|_basic/test_start_vm_|0|0|0|1
basic_test|_basic/test_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_volume_|0|0|0|1
basic_test|_basic/test_add_vol_to_stopvm_|0|0|0|1
basic_test|_basic/test_add_volume_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_multi_volumes_|0|0|0|1
basic_test|_basic/test_add_volume_negative_|0|0|0|1
basic_test|_basic/test_crt_temp_from_volume_|0|0|0|1
basic_test|_basic/test_crt_vm_with_volume_|0|0|0|1
basic_test|_basic/test_vm_securitygroup_|0|0|0|1
basic_test|_basic/test_create_iso_vm_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account2_|0|0|0|1
basic_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
basic_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
basic_test|_suite_teardown_|0|0|0|1
virtual_router_test|__suite_setup__|0|1|0|0
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
virtual_router_test|_novlan/test_create_vm_|0|0|0|1
virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_vlan/test_create_vm_|0|0|0|1
virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_snat_create_|0|0|0|1
virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
virtual_router_test|_vlan/test_start_vm_|0|0|0|1
virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
virtual_router_test|_volume/test_add_volume_|0|0|0|1
virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
virtual_router_test|_sg/test_ingress_|0|0|0|1
virtual_router_test|_sg/test_egress_|0|0|0|1
virtual_router_test|_sg/test_icmp_|0|0|0|1
virtual_router_test|_sg/test_udp_|0|0|0|1
virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
virtual_router_test|_sg/test_vm_ops_|0|0|0|1
virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
virtual_router_test|_sg/test_detach_l3_|0|0|0|1
virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
virtual_router_test|_pf/test_attach_pf_|0|0|0|1
virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_multi_rules_|0|0|0|1
virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
virtual_router_test|_pf/test_tcp_|0|0|0|1
virtual_router_test|_pf/test_udp_|0|0|0|1
virtual_router_test|_pf/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_2_eips_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
virtual_router_test|_eip/test_attach_eip_|0|0|0|1
virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
virtual_router_test|_eip/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
virtual_router_test|_lb/test_create_lb_|0|0|0|1
virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_suite_teardown_|0|0|0|1
local_ps_virtual_router_test|__suite_setup__|0|1|0|0
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_ps_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_ps_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_udp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_ps_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_ps_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_ps_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_ps_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_ps_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_udp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_ps_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_ps_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_ps_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_ps_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_ps_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_suite_teardown_|0|0|0|1
local_nfs_virtual_router_test|__suite_setup__|0|1|0|0
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_nfs_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_nfs_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_nfs_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_nfs_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_suite_teardown_|0|0|0|1
Total|304|0|4|0|300


잘했다.
근데 다음 주는 로그인/아웃보다는 html5의 기능을 구현하는데 초점을 맞추면 좋겠다.


Memo to self: https://github.com/eklem/life-indexer/blob/master/li-flickr-uploads.js


Decided to separate that on higher level - separate VisitSource instance.
Reasons:
* simpler usage
* Separate cookie name

If you'd like to store both, just create 2 separate instances of VisitSource.


Thanks for explanation :smiley:


This looks like a nice clean solution.


Interesting info, thanks :)


The attached patch enables a Habanero pass1 and Palmetto we have to boot with the latest 1.6 release.


I'm fixing that now, once I pass the Teuthology smoke test I'll submit a pull request. Basically, I had cast/constructor operators in utime_t that made all these things just work. I removed them since I realized it might be a Very Bad Idea to require a file in src/include to include a file in src/common.  Once I deal with the fallout of that I shall be set, I think.


Maybe it's possible to generalize somehow, but I didn't spend time cause not sure if we live with this workaround long enough.


BAD NEWS: this will totally break everything
GOOD NEWS: the fix is REALLY simple


fix localtime sh4


Yes, looked easy =)


To make it similar to what happens in real API


@BlaXpirit could you try this and tell me what you think? :-)


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1813 of 1813 tests passed)

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


`it's` -> `its`


@BlaXpirit actually try this one :blush:


LGTM


You have excellent English (see commit message)!


93651760-RowPerObjectReport (JasonGlazer) - 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)


destructor ! Consider using an async ?


__question:__ Do you think we should move this method to single place like ```utils.js``` or it's ok to duplicate it?


Do we need `tb` passed to this function?


I believe the type argument is not needed here. `return negate.call(x);` should work just fine. Same thing below...

Also, consider adding an `operator()(Args args...)` to `bind::cbFunction` making it

````
return negate(x);
````

:D


Closing double quote not escaped at the end of the sed expression: `...|g"` -> `...|g\"`


Such Checks are matter style, it is my preference to not have them in list of active Checks in our config.

As we discussed before, in xdoc we will have link to checksty_checks.xml as example how we use our own Check.


Hehe, just thought it might be a good idea to document the few steps for upgrading. It's very likely I won't have any idea what changed in a few weeks :D

Unless we have some kind of upgrade script, maybe it would be a good idea to put infos for each release and it's update into admin/upgrade/release-x.x.x.info ? So one could check current version (would be a good idea to add that info somewhere, maybe in the config-default as $cfg->sys->version) and do each update step with those files?

On the other hand. I really hope, the current changes are good enough, so you'd never want to move stuff around again. So it would be best to get the structure for existing (and thinkable future additions) right as soon as possible.


93651760-RowPerObjectReport (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1807 of 1807 tests passed)

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


Looks like a linger `tarfile.open(fname)` reference...


@hdinger can you please have a look. this fixes the problem. not sure if it introduces a regression somewhere else.


You delete the message. It is not right


Not sure about the debug one. There's also `applications-debug` in `categories`


Could we get the list stuffed into a temporary file and read it from there?


Moet dit niet ook in de compressed ol.js?


I don't understand why the 1 is quoted ('1') but the zero isn't.


init


Got it in the latest push.


final ?


Github correctly contains Vagrantfiles for latest base box and kaldi base box,
but finding their URLs means peeking Vagrantfiles of other VMs that use them,
which look like:

   http://speechkitchen.org/boxes/mario.box

   http://speechkitchen.org/boxes/mario-kaldi.box

I have just one pesky thing to fix so that IVW3 comes up, not only with Ubuntu
desktop, but with VirtualBox guest additions compiled in, so that we get bigger
window size and cut-and-paste right out of the box. Something really clever
would be to launch lightdm while Vagrantfile shell commands are
still running, rather than after a reboot. We'll find out...

On 08/29/2015 12:04 PM, Florian Metze wrote:
> cool, can you send me a pointer to the latest basebox and kaldi-vms, or should i able to find them myself not he site? thanks, F.
>
> > On Aug 28, 2015, at 1:52 PM, riebling <notifications@github.com> wrote:
> >
> > Also does extra steps to set up ubuntu desktop
> >
> > virtualbox guest additions
> > automatic login
> > cut & paste enabled
> > auto resize works
> > —
> > Reply to this email directly or view it on GitHub <https://github.com/srvk/IVW-v3/commit/3e8477b1dc87d3a4e8154c1c2660088551448313#commitcomment-12953017>.
> >
>
>
>
> Florian Metze <http://www.cs.cmu.edu/directory/florian-metze>
> Associate Research Professor
> Carnegie Mellon University
>
>
>
>
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/srvk/IVW-v3/commit/3e8477b1dc87d3a4e8154c1c2660088551448313#commitcomment-12961745>.
>

--

Eric Riebling      Interactive Systems Lab
er1k@cs.cmu.edu    407 South Craig St.


See comment above ... it does not appear to be the case that `tb` needs to be passed.


TeamCity pull requests :: netty [Build 158](https://garage.netty.io/teamcity/viewLog.html?buildId=1021&buildTypeId=pulls_netty&guest=1) is now running


I figured, this was a reminder, nothing more. :)


The min-width is necessary for the right calculation of the headers, but this problem is already fixed in: https://github.com/opf/openproject/pull/3432


empty string vs. string with a '1' in it == no attribute file vs. one with a '1' in it


@asterite It's great! Better than any ideas I had.


self.assertFalse


Rename to run ? As this is now blocking.


@asterite
It's not so great to have "Methods inherited from module Spec::ObjectExtensions" on every page.
And maybe it's worth deduplicating the method names and linking to the first overload?


@winstonf88 o certo aqui é usar `self.assertTrue()`


Can you add spacing around the assignment?


I was thinking..

couldn't it be "simply", that, due to the fact get_free_port() could really
return the same port for 2 consecutive(or almost) calls , that is :

get_free_port() -> 33275  # say
(get_free_port() -> eventually another port)*
# but then:
get_free_port() -> 33275 # ..

and when these ports are given to the server instance, if 2 of them are
equals ... boum server start will fail obviously..

so maybe the solution is "simply" to implement a
`get_free_ports(num_ports=1, ..)` which don't close the sockets up to when
the num_ports have been obtained..
if you see what I mean.. ;)

2015-08-28 15:58 GMT-04:00 Grégory Starck <g.starck@gmail.com>:

> > I don't think so. Travis builds have individual containers. I can't be
> sure tho.
>
> I was rather sure about that too.. but I don't see what could be
> eating/using the free ports obtained the way it's done here.. if we are
> really in a container (with own network namespace) then I really don't see
> what could do that :s damn.
>
> 2015-08-28 15:33 GMT-04:00 Alexandre Viau <notifications@github.com>:
>
>> build is shared with others builds that would some time to time re-use
>> the open ports we discovered with get_free_port()
>>
>> I don't think so. Travis builds have individual containers. I can't be
>> sure tho.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/influxdb/influxdb-python/commit/78e5adad441767361bcc1a85393140a984394f1a#commitcomment-12955013>
>> .
>>
>
>


Oops.. seems like I was drunken when creating this PR :smile:
This state is not what it was supposed to be.


Was wondering whether you had been looking into the abyss again :)


@BlaXpirit I know about `Spec::ObjectExtensions`, but it's better than showing no inherited methods at all (or put differently: it's an unrelated issue). You can hover the name to see the different signautres, but yes, maybe we should just select the first name to reduce the list. On the other hand, showing it multiple times let you know that there are overloads, otherwise maybe you click on the link, see the first overload but miss the other ones.


Now all we need is a 3+2 laned road and proper turning lanes will be possible!  I just wish there was a way to add custom transitions like SC4 NAM had.  The vanilla double-zebra ones are quite frankly awful.


HI Ritesh, thanks for replying, that helped me to resolve the error.

However, I want to use this on Mysql database instead of mongodb, can you please help me do that. I am really sorry for troubling, but this is really making me pull my hairs.


Typo: `samba`


update from KAFKA-1878 and KAFKA-1999


Typo: `everything`


Really don't like this exception-driven logic, can we avoid it somehow?


The commit message suggests that `Genome::Model::ReferenceAlignment::Command::AdspQcSummary` is replaced by this new module, but the removal of that tool was not included in the commit.


Typo: `requests`


https://www.reddit.com/r/csharp/comments/3ikzm5/material_design_in_xaml_toolkit_new_homepage/cuie9fw ;)


Why didnt we fix the 32u4 bootloader with this update? If you start producing new boards with new boards.txt entries you could fix all the bugs in the 32u4 bootloader. It wouldnt hurt but make things easier.


Also, Javadoc shows them duplicated, that's why I thought it might not be a real issue. Maybe we could show them once but when you click them show a popup to select the overload... but that involves JS so it's a bit more complex, but definitely an improvement.


Une petite typo détecté par @VincentCATILLON "seasion"


TeamCity DNN JavaScript Libraries :: CI [Build 72](http://teamcity.engagesoftware.com/viewLog.html?buildId=340&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 72](http://teamcity.engagesoftware.com/viewLog.html?buildId=340&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:8


Isn't this about #1325? I suggested setting `PYTHONIOENCODING=UTF-8` and codewarrior had some insights on this.


I may be misunderstanding this, does this preclude seednodes from bootstrapping from others seednodes?


Nice, thanks for tackling that!


I see now. And what would be the use case? Because doing it like that I think it would change the meaning of the filter.


I found it really hard to parse, since I assume I will never have `browserInfo.winOpenCmdTemplate` on non-win systems.


This patch is a bad workaround, as the issue still exists. In case of getResultCount() which will not call applyOrderBy() i get the same SQL error


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


I think `open` has grown too much and requires refactoring. Can we move `exec` params gathering to the separate methods for each os?


Hi Imahendra, if you want to use only MySQL with Rails 4.0 app with Devise, CanCan, OmniAuth and Twitter Bootstrap, then you can clone and work on https://github.com/Ritesh-Kumar/rails4-bootstrap-devise-cancan-omniauth.


Would it hurt to show a compacted signature, similar to how you get function pointers to overloads? `#to_i`, `#to_i(Int32)`, `#to_i(int32, Bool)`


This is cool! I didn't realise ScalaTest has a `compile` matcher!


Что за старье? На дворе уже 5.2.1, даже когдя я начинал его использовать уже была версия 4.3.0


> Long-term solution would be to have a separate attribute set containing the packages for nix-env.

I'm not so sure about that. The behavior with patch above is that `nix-env -q` shows no warning, but `nix-env -i` does (i.e. it warns iff you instantiate through a deprecated name).


lol... this is _so_ similar to n2o:

````
N2O_MODULE(object exports) {
    exports["World"] =
        class_<world>("World")
            . function("greet", &world::greet)
            ;
}
````

Constructors are still under construction, though. The only difference is that I use explicit exports like commonjs. It looks very nice in a nodejs add-on, because it's so similar to a JS module. But it obviously is a bad fit when targeting web browsers.

BTW, do you support overloaded functions, yet?


This is Spring project


Can we refactor this huge loop's body into the set of methods with the meaningful names. I find it really hard to understand.


Hey @MeltedFruit, this is a repository for our lua-practices. Just put your practices in this repository, so we both can see and comment or improve from different perspective.


We need a comment here which explains why we do this.


@jhass I tried that initially, but it bloats that section and makes finding something harder.


Why we need this array wrapping here?


Thanks Michael, that did fix the issue.

Thomas Tomlinson
thomas_tomlinson@me.com

> On Aug 29, 2015, at 10:06, Michael Brown <notifications@github.com> wrote:
>
> Should be fixed as of http://git.ipxe.org/ipxe.git/commitdiff/2ef04f0
>
> —
> Reply to this email directly or view it on GitHub.
>


\r-


Breaking how?


Fix small error at:

```
    @classmethod
    def sweep(klass, privkeys, network, to_address, fee):
        inputs = []
        keypairs = {}
        for privkey in privkeys:
            pubkey = public_key_from_private_key(privkey)
            address = address_from_private_key(privkey)
            u = network.synchronous_get(('blockchain.address.listunspent',[address]))
            pay_script = klass.pay_script('address', address)
            for item in u:
>               item['scriptPubKey'] = pay_script
E               TypeError: list indices must be integers, not str

lib/transaction.py:540: TypeError
```


Should we also add another router for LuaScript? At the moment I think Lua's loader is configured to first look for *.luc then *.lua (which is, unfortunately, exactly the other way round than AngelScript), by calling the GetResources() twice.


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


I should have used -a with my last commit.  Deletion of AdspQcSummary added.


Instead of moving these lines here, let's move them to just before the make check-$SWIGLANG-examples command, otherwise it clutters the start of the script. Also, let's just drop 'export' only and keep as was:
```
- if test -n "$SWIGLANG"; then cflags=$(Tools/testflags.py --language $SWIGLANG --cflags --std=$CSTD) && echo $cflags; fi
- if test -n "$SWIGLANG"; then cxxflags=$(Tools/testflags.py --language $SWIGLANG --cxxflags --std=$CPPSTD) && echo $cxxflags; fi
```


@koistya ... I think that should work for you, I usually create a connection variable, and use that from a separate module that I will wrap in a promise.

Actually, lately I just use my `mssql-ng` wrapper for most interactions with async/await syntax via babeljs.


слишком быстро выполнил mvn versions:display-dependency-updates и не нашлось обновлений.


Not sure on this one ;)

(plus, tinting icons only works on Android L :()


Thanks ! https://github.com/CanalTP/MttBundle/pull/54


@Fevix cinematics/airupgrade/avali/avalifemale.png
The first two frames contain the outlined body. The left one was adjusted lightly to match the sketches and Starbound design for face while the right one remains unchanged. Feedback?


I think we should also refactor the metadata.log config parsing code, as we see this pattern at least three times in the code.


@mahangu could this be changed into a variable and then filtered before the if statement. It will be easier to read and understand that way. The variable name can be the same as the filter if you like.


@cehallen DWMR. Press on!


I can add a prefix for the user name lock string like "PREFERENCES_LOCK_" so that it will be used for preferences only (if the same solution will be used elsewhere this is certainly a good option). It doesn't make it explicit but it greatly reduces the chance that other places in the code will use the same string for locking.
There is also an option to use a WeakHashMap for saving a string-lock pair by user name (it will have a similar effect with many more lines of code - a solution I found but haven't tested can be found here near the end: http://stackoverflow.com/questions/12450402/java-synchronizing-based-on-a-parameter). In this case the lock is explicit but I'm not sure it's worth the added complexity.
For a lock cache like you suggest we need to define the number of available locks, and since I don't know in advance how many users there are or what system resources are available, every value will be just a guess (as the use case could be different for each implementation of Che).

As for the API change - we are not currently planning on moving to 4.0 soon so this change is not relevant for us yet. In any case, if the code in the service changes in a way that doesn't require my change, of course it shouldn't be merged in the newer version but only in versions that require it.


Should be node['monitor']['default_handler_timeout']


Hi Ritesh, thanks for your time.
Actually I downloaded repository and installed bundle on my server, later I updated database.yml with my database credentials. when I rake db:migrate, I could see movies and policies table, but not the user or roles which I presume went to mongodb which I am completely unaware of.
So wanted to make it work with MySQL, but could not do much, if you can help me how to get started with, I will be more than greatful to you.


should we be linking both commands (`openshift-router` and `openshift-f5-router`)?


It's not an array wrapping. Microsoft Edge truncates window title to hostname, if document.title set to plain URL ( e.g. `document.title = http://localhost:1334/page321331` gives just `localhost` in window title). So we need to add some characters before URL in order to get it in window title.


Doesn't $window.location.origin make this incompatible with Internet Explorer (which does not support this property)?


Activity -> Activities


convention de codage ! espace entre les parametres


너도 알다시피 이렇게는 안된다 ㅎㅎ


Merge it in to the params in the controller -> (params.merge(project_id:
$project_id) (same for $user_id if you need it).

 $project_id and $user_id are only accessible in controllers or rake tasks,
they are set before actions/tasks, and cleared after actions or tasks.

M

On Mon, Aug 31, 2015 at 8:27 AM, James Tucker <notifications@github.com>
wrote:

> In the case that the model (class Topic, in this case) does not include
> the project_id, and it does not come in with the params, where *does* one
> get it?
>
> _______________________________
> James Tucker, Programmer
> Species File Group,
> Illinois Natural History Survey,
> Prairie Research Institute at the University of Illinois, Urbana-Champaign.
> TuckerJD@UIUC.Edu
> 217-244-9416
>
>
>
>
>
>
> From: Matt <notifications@github.com>
> Reply-To: SpeciesFileGroup/taxonworks
> <reply+001c0e126e861a8081f600d1e7457b4c799757f88ce7903492cf0000000111f8f12a
> 92a163ce00c5c0a0@reply.github.com>
> Date: Friday, August 28, 2015 at 23:03
> To: SpeciesFileGroup/taxonworks <taxonworks@noreply.github.com>
> Cc: James Tucker <TuckerJD@UIUC.Edu>
> Subject: Re: [taxonworks] Topic finder for otu_page_layout. (518c06c)
>
>
> This is the right line, don't reference $project_id, it should only be
> referenced in the models except through housekeeping.
> �
> Reply to this email directly or
> view it on GitHub
> <
> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_SpeciesFil
> eGroup_taxonworks_commit_518c06cce4eb7f65e79f0ee53d1a12b5424b401b-23commitc
> omment-2D12959904&d=AwMCaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=87FiFUsLQn0J2UnE2s49r
> k_Ml7AdI6AGU3DZpzKZL1o&m=LjfLusMuvcakqu304mLQ5nKwi87XcqqGQMsTwhPkgXU&s=U-tJ
> A4U6Fpzj5f6k1nI492vAtuS0cnPkLYUlk4sb29s&e=>.
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SpeciesFileGroup/taxonworks/commit/518c06cce4eb7f65e79f0ee53d1a12b5424b401b#commitcomment-12975854>
> .
>


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (759 of 759 tests passed)

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


너도 알다시피 이렇게는 안된다.


>couldn't it be "simply", that, due to the fact get_free_port() could really return the same port for 2 consecutive(or almost) calls , that is :

This is what I said, see above:

> There are chances that we get the same random port two times... Maybe it would be worth testing.


oh my goodness.


이렇게 하면 CORS로 동작하는데 이렇게 많이 하네?


Popups is one option. And it's not difficult to make. I may look into this.

Another option is to just make these sections expandable to full-fledged normal views.
But gotta be careful not to bloat the file size too much.


effectively, I had not understood directly what you said.. /o\

go on it if you would like, I'm rather busy ;)


I think I got it wrong, I meant the status bar color, which gets green too.


Ага, хорошо, что ты с этим поигрался и почти полностью реализовал `AbstractAnnotationConfigDispatcherServletInitializer` самостоятельно.

Теперь у тебя стало больше кода и все бины переместились в корневой контекст.

Теперь:
- читаем вот эту ссылку http://stackoverflow.com/questions/3652090/difference-between-applicationcontext-xml-and-spring-servlet-xml-in-spring
- переносим наши бины из рутового контекста в контекст сервлета (кажется, я так предлагал с самого начала)
- возвращаемся к реализации `AbstractAnnotationConfigDispatcherServletInitializer` (просто сделай так, чтобы `getRootConfigClasses()` возвращал класс без каких-либо бинов.


Hm, I see... this example is outdated, right?


How do you think about to support an autoconf software version which is a bit older than "2.69"?


Should be:

```
{ grep -I -H -l -R "\(INSTALLROOT/$PKGHASH\|[@][@][^@]\+[@]$PKGHASH[@][@]\)" . || true; } | sed -e 's|^\./||' > "$INSTALLROOT/.original-unrelocated"
```


No worries. :)


Sounds good to me :+1:

Should be updated now.


TeamCity pull requests :: netty [Build 158](https://garage.netty.io/teamcity/viewLog.html?buildId=1021&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 4759, ignored: 21 Build time: 00:28:37
### Failed tests
```
io.netty.channel.epoll.EpollSocketTcpMd5Test.initializationError: <no details avaliable>

```


I know for a fact some bans are unjustified, he added the entirety of cdizas friendslist, note some of them DON'T EVEN OWN GMOD. He also added steamids located inside of some really old files of mine, none of which were hackers, just people I knew or disliked.

He also adds things from a google search for your steamid, even if they are from un-reputable or unjustified sources. (EG, I am listed for 'AC truth engineering', which I have never done, the only thing similar was debugging related tests and attempting to bypass my own private stuff. And ghook/nospread is because I am credited in something that i wasn't even involved in the making of.)

This is a good addition because the amount of times I've been banned from servers for the only reason that I am on skidcheck (People literally turn kicking off and manually ban you, these idiots trust it that much.) is just not even funny any-more...


I see it as huminize for strings. humans prefer spaces.
On Aug 31, 2015 17:03, "Pablo Villoslada" <notifications@github.com> wrote:

> I see now. And what would be the use case? Because doing it like that I
> think it would change the meaning of the filter.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Puigcerber/angular-capitalize-filter/commit/9af958f18f1c229a63909da8847bd492d2c3efeb#commitcomment-12976653>
> .
>


Is this fix really related to the UIAppearance PR? It seems not .. or am I wrong? Whats the problem here?


Ian's changes.


We don't need to do this. hang on


I was intending to modify travis.yml to do an Android compilation to, however, that doesn't seem to neat now. For one, the language is set to `cpp` but only `android` provides nice clean installation of dependencies.

Secondly, our build matrix does a clang and gcc compile, but Android will only do clang (gcc is bugged) so that's an extra unnecessary step.

I was thinking of splitting this into a separate project, but if some change breaks it this won't be immediately apparent.

halp :(


nope, it should actually be PKGREVISION, no? Not sure what happened here..
Will fix it.


hi, wojciech,
you might be interested to take a look at my long-time project [cwebbin](https://github.com/ascherer/cwebbin), parts of which date back to “march 1993”. it ships with a set of <code>*-ansi.[h]ch</code> “change files” that make “cweb-3.64ad.tar.gz” compile flawlessly with g++ 4.6.3 with “-wall”, plus some convenient extensions.
cheers, andreas


Hi, this appears to have caused this:-
https://github.com/kickstarter/rack-attack/issues/136


@abnero DWMR. Good work here. Press on!


I wish we could find a nicer name. But I can't think of any atm.


I recommend to pass an existing source file name for [this macro call](https://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.69/html_node/Input.html "Finding configure input").


I think we'd need a third CI, so ffor now you should avoid duplication where possible.


You need to add retired or voided class to TDs. As for the angular error classes those are added by angular so there is nothing you need to to


@jgarboza813 DWMR. Nice work here. Good.


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


@jgarboza813 DWMR. Good good. You are getting it and moving quikcly. Ideal developer habits/skills/behavior.


@kilahimm Interesting - where does the app use libreoffice?


Could you maybe rename `results` to `hash`?


Text:
"Google Maps geocoder is not configured. Please contact us at
sales@cartodb.com"
*I like the original copy. It is succinct and to the point. I would not
change it except to make the first part of the clause a full stop (add a
period).*

*Can we drop the "column" in #1 and #2? It would read:*

1) Select your Longitude and 2) Select your Latitude

"Select the columns containing your Lon/Lat"
"Choose and transform any column to numerical data for georeferencing."

"Override all values. Geocode all your rows, even those with geometry data."

On Mon, Aug 31, 2015 at 8:17 AM, csobier <notifications@github.com> wrote:

> Regarding the new text:
> "Geocoding quota is not defined for your account, please contact
> sales@cartodb.com"
>
> Additionally, the dialog box as is needs attention. (I'm not sure what
> exactly the override option does but it is not clear. Let me know if the
> text below is applicable).
>
> "Select the columns containing your Lon/Lat columns
> The selected columns are transformed into georeference coordinates.
>
> Override all values and transform your entire dataset to georeference
> coordinates. This overrides any previously transformed georeferenced data.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CartoDB/cartodb/commit/bda38366e7cbeecc28a2865e0bd76663602cd5d4#commitcomment-12974664>
> .
>

--
*Brittany Micek*
Content Writer

*CartoDB—Map Your World's Data*
143 Roebling St 2A, New York, 11211, USA
T: + <%2B34695497365>19174633232
W: www.cartodb.com

Live sessions to learn CartoDB <http://cartodb.com/webinars/>


It's a builder. If nobody uses these method I would delete them at all.


@jgarboza813 Never seen this solution before. I love it!! There are so many ways to solve any one problem. Just solve it and move on. Good.


Ok, let's remove this code.


best commit ever ;)


I don't think this change is correct.
It will change the order of the output list and thus values won't be overridden in the correct order.
What's the reason for the change ?


@jgarboza813 What would happen in this code if `number > otherNumber` ? It would not work as expected. Test assumptions - that's the rule.


@jgarboza813 DWMR. Press ahead!


Все сообщения нужно вынести в properties-файлы. Почитай для начала тут: http://docs.jboss.org/hibernate/validator/5.2/reference/en-US/html/chapter-message-interpolation.html


"right" not "pull-right" to pull the buttons to the right


@jgarboza813 DWMR. :+1:


hey. git blame showed your email address so i added you.

**opting out** is the only way of getting away ;-)


Смысла выводить все ошибки разом и потом еще рядом с каждым полем я не вижу. Но если нравится, то путь будет.


This does not works on my side:
```
[ERROR] .../guiproko/list/ConfirmDeleteMapGui.java:[38,19] error: cannot find symbol
[ERROR] symbol:   method sendMessage(String)
[ERROR] location: interface HumanEntity
```
(Compiled using Maven.)

Maybe we should use simple `Player` objects here? Avoiding a cast.


Кажется, что тут было бы уместным использование `<form:label>`


When using UIAppearance, the background color property can't be used at the mask background. Background color is used as the HUD's background color.

I converted the gradient layer to a general CALayer to preserve the mask.


It's been reported than in 6.2 profiles were no longer automatically extending `default` profile.
We have included that but with that fix this bit of code was entering in a infinite recursion.

I wasn't aware of the relevance of ordering of profiles though.


@avigad , I don't mean to nitpick before you've finished writing the chapter! But I noticed a couple other minor things that I wasn't sure if/how I should fix.

- It's shorter to enter ℕ using \N instead of \nat. Did you choose the more explicit one on purpose?
- At least in the pdf, the typesetting for  ∀x ∀y ∀z (x|y ∧ y|z → x|z) looks off. I read it as |y ∧ y| (or |z → x|) surrounded by other things. Is that just my brain being weird?
- When I compile, I get red boxes around | and ^ in the Lean style sections. A bug, or something wrong with my tex setup?


sorry, didn't spot this last time. `dev-repo` needs to be `git://github.com/hakuch/Nebula.git` or else only you can clone it


Если не ошибаюсь, то можно вместо `hello` вернуть просто `null`, чтобы остаться на той же странице (т.е. отображить тот же самый view).


P.S.: This is some Java usage stats from pagina EPUB-Checker (retreived from server logs when checking for updates) for last month:
```
1.6	  7434	24%
1.7	  7932	26%
1.8	 15000	50%
=================
	 29935
```

Java 6 is still in heavy use in epubcheck production environments...


Можно упростить -- пробрось во view весь объект `person` и там обращайся к его свойствам.


Ok, seems legit. However I see some code duplication (´gradientCenter´). This needs to be optimised, I will do so during merge if this is OK for you.


whitespace trouble


i mean command >.<


That sounds great! Thanks.


ขอบคุณมากเลยครับที่แก้ให้ :+1:


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


Things like `for-of` or `Array.from` call `wrapped[Symbol.iterator]()` which then just returns `this` (`wrapped`) and because `wrapped` has `.next` it works.


I think a better fix would be to always add the {{default}} profile at the beginning of the list and keep the code the way it was.


This should log the serviceUrl as well now.  It wasn't ambiguous when it was shell-command, but now it would be nice to know.


Looking forward to testing that!


Did it.
#14 and #15


Since this is now configurable we should expose it to the template since the template assumes you'll write certs to the `/var/lib/containers/router` path


Why not a reference then ?


duplicated error output? (SN_CAUGHT() vs. cerr)


ok! i'll change it, thank you


Pre-allocated the slice for 10 items. Just a random guess, should be enough rather often I guess.


@heyMP is this something we can merge in, don't see a PR for this and dev froze awhile ago; want to get the branches all merged in where it makes sense


Oops, this means no rollbar notifications! Noticed the latest was 2 months old so went looking, I think this is probably it.


replace with DecimalType.isShort()


Added the Printed-Printer-files folder containing all of the printed parts used to build the 3D printer.


Ah, interesting

On Mon, Aug 31, 2015 at 10:40 AM, John-David Dalton <
notifications@github.com> wrote:

> Things like for-of or Array.from call wrapped[Symbol.iterator]() which
> then just returns this (wrapped) and because wrapped has .next it works.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lodash/lodash/commit/9e86efadbcc2a0720282bce10924e6d9ea155077#commitcomment-12977565>
> .
>


@heyMP is this a branch we can kill cause I think you ruled out using this method one day as far as accessibility is concerned


Комментарии к коммитам уже лучше, но всё равно еще не идеальны. О чем нам говорит "изменил содержимое файла"? Да ни о чем. Просто ты что-то поменял. Старайся писать так, чтобы сразу можно было понять, что ты в коммите поменял.

Например, `README.md: update.` Здесь уже понятно какой файл ты хотя бы обновил. Или: `README.md: update for the latest changes.` Не сильно лучше хотя как бы намекает, что остальные изменения где-то в соседних коммитах. Еще вариант: `README.md: describe how this project is supposed to work.` Тут еще больше конкретики -- сразу ясно почти все -- где поменял и что поменял. Если остальное не интересно, то можно даже весь коммит не просматривать, потому что все описано в комментарии.


:smiley:


Does anybody know if activating these flags`-fprofile-arcs` `-ftest-coverage` `-lgcov` has any performance impact?
I would say they should only be active while testing on travis, right?


@wluyima we had them in adminui.css and you moved them to uicommons. Not so?


Yes, one log to the log (wherever that is) and the other to cerr.


Debugging enabled on the last commit.


mdp更好？


Can we remove the above wild card import?


git is already installed when following libclangmm-installation document.


Scrum Master是老师来做，不需要我们做的吧


What version of Ruby are you using? redis-store 1.1.5 should support the extra argument:

https://github.com/redis-store/redis-store/blob/master/lib/redis/store/interface.rb


LGTM


This should be "verify that you have installed X as described in LINK", if we have the link :)


We should ignore all migrations as changing the line to migrations/


looks good


Oh, I needed the ng-app for soundcloudify.background because I needed to load background controller. If we move the logic from background controller to a run block, getting rid of it should be easy.


Что за `perponse`? Выражай свою идею в коде ясне. Что делает этот метод? Обрабатывает форму, так? Можно назвать `handleForm`. Уже лучше. А что если у нас таких форм и обработчиков несколько? Тут два варианта -- либо разносить их по контроллерам (и тогда везде можно называть `handleForm`), либо называть еще более конкретно, например, `handlePersonForm`.


Re-add space


-> deIsobar


yhy ;)
thank you.


I :heart: this commit


can you remove these requirements lines?


@claudiob thanks! ^__^


> Any of the regular comparison operators (=, !=, >, >=, <, <=, IN and NOT IN) have the property that when compared with NULL they return NULL. So it isn't quite accurate to say a <cmp> NULL will be false, but it won't be true and for the purposes of a WHERE clause not-true is the same as false. See https://en.wikipedia.org/wiki/Null_(SQL)#Effect_of_Unknown_in_WHERE_clauses and https://en.wikipedia.org/wiki/Null_(SQL)#Common_mistakes.

Can you add a comment to this effect to the commit message or somewhere in the code, or both?


After later changes, I don't have `old_abilites` just `old_abilities_downcase`. So above line is faster then: `new_abilites_downcase - user.abilities.pluck(:name)`


What is the use of this loop?


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1287 of 1287 tests passed)

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


getMaxPrecision function could be fully internal to WebGLCapabilities and all results cached (as there are only three different possible inputs.)  So we can cache on startup once and then never expose this function.


On compiling libx265 followed by FFmpeg, the configure tests in FFmpeg show the new `bool` for `bScenecut` in x265.h to be an "unknown type name". Adding <stdbool.h> to the headers fixes this for me.

Environment: cross-compiling using mingw32-w64-x86_64-gcc version 5.2.0 with mingw-w64 version 4.0.4

```diff
--- source/x265.h.orig  2015-08-31 14:39:07.197126527 +0000
+++ source/x265.h       2015-08-31 14:39:26.953126527 +0000
@@ -25,6 +25,7 @@
 #define X265_H

 #include <stdint.h>
+#include <stdbool.h>
 #include "x265_config.h"

 #ifdef __cplusplus
```


I'd prefer not to have derived parameters in Capabilities (like supportsBoneTextures) if possible as that is an implementation choice.  I'd rather have WebGLCapabilities be purely reflective of the properties of WebGL.


Hello,

A new driver released by Nvidia;
http://www.geforce.com/drivers/results/90393

VDPAU related notes are;

1)Fixed a bug that caused VDPAU to only display the top half of a video frame when decoding and displaying H.265/HEVC encoded video streams.

2)Added support for VDPAU profiles added in VDPAU 0.9:
VDP_DECODER_PROFILE_H264_BASELINE
VDP_DECODER_PROFILE_H264_CONSTRAINED_BASELINE
VDP_DECODER_PROFILE_H264_EXTENDED
VDP_DECODER_PROFILE_H264_PROGRESSIVE_HIGH
VDP_DECODER_PROFILE_H264_CONSTRAINED_HIGH

I will try to check if vdpau decoding of HEVC fixed tomorrow.


Was thinking that we did not need those requirements that were already installed through libclangmm, however, if you want them here as well, we need to add more packages (and it becomes somewhat bloated).


This is fixed now https://github.com/kokarn/wifi-unlimiter/commit/f73bdfe202ef37565cf273518166030945ef8f69


This might be a bit picky, but I wrote the efergy plugin to work with the energy monitoring hub, which monitors whole-house power usage via clamps on the lines themselves.  The company does actually make plugs in the UK market, but I have no idea if they would actually work with this, as I don't have any - my point being, referring to it as supporting "efergy plugs", might be misleading.


Unfortunately as noted in another issue someone else took control of the npm


Wow this is ugly, is there no system call to do this ?
Reading /proc is generally racy, which is why I use getpeercred for pid/uid/gid


Btw, do you have a Ubuntu 14 machine? See the new issue. No idea yet how to fix it.


Not sure I completely get the point. It seems this method can be called (with an effect) exactly once. Is there a reason to prevent calling it multiple times?
I.e. is there a reason to keep the `mSetPageOrientation` variable?

And a good thing to do in general is running `scripts/prepare-commit.sh` before making a pull request :-)


I used `find_or_create_by(name_downcase: ...)` in case, when there is some Ability with `name: 'ruby'`, and `user_data = ['Ruby']`. In this situation `find_by(name: ...)`, would create another record, which is not necessary.

But I changed code as your suggested in second comment :)


Please explain what these numbers represent, and what they are for.


We also need to add libboost1.55-dev-all-or-something and g++4.9 most likely to Ubuntu 14 and Mint requirements.


also incorporated cleaner method of getting timestamp


I also care about these:

```
gl.getParameter(_gl.MAX_VARYING_VECTORS)
gl.getParameter(_gl.MAX_VERTEX_ATTRIBS)
gl.getParameter(_gl.MAX_VERTEX_UNIFORM_VECTORS)
gl.getParameter(_gl.MAX_FRAGMENT_UNIFORM_VECTORS)
```

Especially relevant with regards to blowing out the uniforms when having too many lights.


Please add a newline to the end of the file.


Actually no packet contents are modified, only firing less of them---- But should we allow accelerated mode in multiplayer game at all?


This line doesn't make any sense.


Again I think this is derived rather than a true WebGLCapability.


@Dwar3xwar DWMR. You are flying though this. And it's done well. That's the perfect 1-2 combination. Your energy is back!


Well, I se know. Can we add comment here which describes this behavior. We need to update [note in readme](https://github.com/superroma/testcafe-browser-natives#api-reference) as well.


This is not going to work. @key_personnel doesn't make any sense in a page class.


Aaron, this adds code to the wrong file. The code in nodeSource/nodeJS_serialToWS.js has moved to serialApp/serialApp.js, and that file has been updated.

I'm not sure if you added any changes in this re-addition of the file since the commit message is empty. I'll work on manually merging it back into the right file. But I just wanted to explain so that you know why this file disappeared.


Why return the default of "4294967295" here ?
I would rather return an empty string or even better None in this case.

Actually I would like to return a number here (not a string) for a valid lookup and None is ENOENT is hit.


This needs to be:

```c
netmask = htobe32(0xffffffffLLU << (32 - prefixlen));
```

...otherwise a prefixlen of `0` will have undefined behavior on 32bit.


please do:
loginuid = self._get_loginuid(pid)
then
return {'pid': pid, 'uid': uid, 'gid': gid, 'loginuid': loginuid}


This line fixed the issue where first person hand looked weird when the same instance of player model gets rendered when the player(its rendering) is riding an entity.


Because you removed this -
if (domain_count > 1) then
		recording_archive = recording_archive.."/"..domain_name;
end

Single tenant will be broken. I do want to move single tenant so it will use the same domain path as time permits but will have to write code to move it. And then change ivr menu so that it works the same.


Please add a comment explaining why the syntax property is added here, pointing to the function you added to explain its role if necessary. The idea is to make sure that anyone modifying this code in the future knows why this code is there, and that it needs to be preserved through refactorings, replicated in other places that generate units and do inference, etc.

Also, this information is not inherently TR-specific. Conceivably, some other tools could use that information, e.g., for tooltips. So you may want to use a key that reflects that.


The first part of this line should be removed; its wrong and unneeded.  Just test for [0] == '#'


我看没什么吧。。


We could just ask tj to transfer the rights.

Arnout Kazemier

> On Aug 31, 2015, at 4:59 PM, Tom Moor <notifications@github.com> wrote:
>
> Unfortunately as noted in another issue someone else took control of the npm
>
> —
> Reply to this email directly or view it on GitHub.
>


It can be performance bottleneck. We should read it directly to Slice for LongDecimalType and to long for ShortDecimalType.


![ios simulator screen shot aug 31 2015 11 03 41 am](https://cloud.githubusercontent.com/assets/1080613/9581959/8878aed0-4fd0-11e5-8ae3-7dd4f6b8b10d.png)


You need to add db upgrade logic for existing installs to the `GmaDatabase.onUpgrade` method


I would suggest using the old internet adage; be correct in what you send, be liberal in what you receive.

In this case, please don't return when a line fails to parse. Just skip it.


We need to add this back, en has not translated language code and therefore shows up as "en" now


Isn't `size_max` wrong if you pass `offset != 0` but `size == 0`?


@MorrisJobke ^


Nothing in this paragraph uses the word "size" or "offset". Could it be made a little more clear what effect the Pool{Offset,Size} options have on the resulting pool of addresses?


This is my first comment on my repo at guthub.


Yeah, I appreciate that you describe the behavior, but it lacks a description of the syntax. Does it take IP addresses? Subnet masks? Offsets?


Turns out, was too tired. :( `get_collectors_from_module` yields tuples, it doesn't return a dict.


I'll rename the variable to supportsFloatVertexTextures. that is a more accurate description anyways.


Merci c'est corrigé


This caused a UI test to fail because the casing of the label changed due to the xform styling. http://ci.xwiki.org/job/xwiki-enterprise-test-selenium/org.xwiki.enterprise$xwiki-enterprise-test-selenium/1435/testReport/junit/org.xwiki.test.selenium/UsersGroupsRightsManagementTest/testCreateAndDeleteGroup/

I`ll update the test to now check for the uppercase version of the label.


This could be rewritten using `for/list` with a `#:when` clause.


I agree. I jsut added what as minimum required to make it work again.


@tchap It looks like the bug here is that io.EOF is returned along with a valid http.Resonse object? This was done to be consistent with how EOF is used in the io package (specifically with io.Reader), but I can see how that would be unclear in this context. Sorry for any confusion this might have caused!

I do test the code with my existing pivotal project, but I would like to add unit tests that mock http.Responses soon.


Remove this line


Just a stylistic thing for me, but I'm not a fan of this sort of page element definition in the page class.  It's basically redundant code.  There's no real savings to be had. You've just changed the syntax of the line of code that's going to be invoking this from this:

`@thing.lead_unit_radio.exists?`

to this:

`@thing.lead_unit_radio_exists`

And you've done it by adding this extra line of code in your page class.

My suggestion: Get rid of this element definition and just use `#lead_unit_radio.exists?` where you need it.


We have two ChainStore files now and some var references are still chain_api...


Ugh, again, nevermind. Next commit does it.


:+1:
Thank you!
You really look rested :smile:


하나는 했다가 멀티로 다시 바꾸느라 코딩 덜 한 상태였어요 ㅠㅠㅠ
이건 안 되는 코드 맞습니다 ㅎㅎㅎㅎ
하나 첨부 되는건 다시 커밋했어요~


aaa


spaces around "<<"


서버가 없어서 일단 급한대로 넣었습니다 ㅎㅎㅎㅎ
아직 작동 안 합니다ㅠㅠ 갈길이 머네요


Is this really necessary? If people configure their server to be 192.168.0.128 with a subnet of /24 then i think it should be fine to hand out all of 192.168.0.1...192.168.0.127 as well as 192.168.0.129..192.168.0.254...


 disagreed but adding https://github.com/gero3/three.js/commit/08dcc556e590111583c37ca16e4195d4940b5428#diff-c0e88b98497597a015ecf238e91ac3a0R211

to the capabilities makes sense.


TeamCity mobile-music-automatization :: PR_BUILD_CONFIGURATION [Build 40](http://teamcity.mobile.dev.yandex.net/viewLog.html?buildId=1190232&buildTypeId=MobileMusicAutomatization_TestBuildConfiguration) is now running


TeamCity mobile-music-automatization :: PR_BUILD_CONFIGURATION [Build 40](http://teamcity.mobile.dev.yandex.net/viewLog.html?buildId=1190232&buildTypeId=MobileMusicAutomatization_TestBuildConfiguration) outcome was **SUCCESS**
Summary: Running Build time: 00:00:02


Shouldn't this rather be explicit `!lease || lease == &server->invalid_lease`?


It's actually an Array of Hashes. Your comment shows a Hash that contains an Array--which is not actually possible. :)


TeamCity mobile-music-automatization :: PR_BUILD_CONFIGURATION [Build 41](http://teamcity.mobile.dev.yandex.net/viewLog.html?buildId=1190235&buildTypeId=MobileMusicAutomatization_TestBuildConfiguration) is now running


TeamCity mobile-music-automatization :: PR_BUILD_CONFIGURATION [Build 41](http://teamcity.mobile.dev.yandex.net/viewLog.html?buildId=1190235&buildTypeId=MobileMusicAutomatization_TestBuildConfiguration) outcome was **SUCCESS**
Summary: Running Build time: 00:00:02


i think it would make a lot of sense to make the type of offset and size an uint32_t, after all ip addresses are 32bit too.


ditto - use of UIS here kills performance.


Why `+`? Below you use `|` here your use `+`. *confused* Sure, it has the same effect as the masks are cleared, but maybe try to be consistent?


Looks very clean here. Yay testing! :+1:


It actually get used too when material define a different precision [here](https://github.com/gero3/three.js/commit/08dcc556e590111583c37ca16e4195d4940b5428#diff-c0e88b98497597a015ecf238e91ac3a0R1415).


Okay, our bad, I'll update the readme.


:heart:


doesnt it work to do just this? https://github.com/pentaho/pentaho-platform-plugin-common-ui/blob/5.4/package-res/resources/web/prompting/pentaho-prompting.js#L396-L397


@krivera-pentaho please check


So this should work now by using

$ionicConfig.backButton.text('') in controller? It's not working for me, I uploaded ionic to 1.1.0.


How so? This code never actually made it into a release i think, but i'd still like to know more about the problem.


Can you remove these commented lines?  They're unnecessary, given that we're using a source control tool with history. There's never any need to save commented code in the repository. :) Sure, use it locally all you want, but don't include it in your pull requests.


OK, easy enough.  I'll push a revised patch.


I would suggest rewriting to use an iterator instead of a for loop. That makes the whole a bit cheaper.


Fixed 7c4431320324d65b54338b93bbc453256ab29981


行コメント


全体コメント


indeed, fixed


I don't think it's that simple. Before this change points were wrong or right depending on chosen period.


Ok, doesn't matter, patch #3 fixes it, anyway.


It's an equivalent to no-UID available - so int / None ?

http://stackoverflow.com/questions/22914627/some-uids-in-proc-pid-loginuid-are-strange


Fixes issue: #2


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.480.772ca7bca26be3e9fadb9889136a0ea7ec944800](http://build.palaso.org/viewLog.html?buildId=62310&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 140 Build time: 00:05:53


May want to think about using Rdoc docstrings to help clarify each class and def.
https://github.com/rdoc/rdoc


Hmm, I think so. If I make a request with a bad ID or non-existent endpoint, it still returns JSON describing the failure.


Unhandled Fundamental Failure


@owaism The quickest way to resolve RND errors is to ignore SSL errors inside the Java buildpack. Thoughts?


The parameter value is not the issue. That is all being handled well. What is not being handled is setting the selected HTML value back to its previously selected value. The 'null' is being handled fine, and when we get parameter values back, the 'null' has not been stored, but we already have selected the value in the html, and since we are not calling an update with the new lifecycle fixes, then the HTML stays the same. My fix puts the HTML back to the previously selected value.


Better to comment in the imperative tense. "Create array..."


I feel like:
>Over escaping certainly does have performance and maintenance burdens (although small)

Is accurate, it's a very small maintenance burden, the performance burden is near nil since I have yet to see a site where all of the escaping on it amounted to 1 so-so SQL query.
I'm not sure I understand the downsides in relation to the upsides. It seems that there have been a series of escaping security fixes in the community plugins that an escape all the things mentality would of fixed. If you are putting html in a translation and not willingly intending for it to happen you're doing it wrong.

I feel like the defaults should always be overly safe and you need to willingly and intentionally do something that is not sure to be safe. We should be in the mindset that everything starts as secure as possible all the time and only make exceptions when needed.


This is dropped in patch #3..


For now I'm leaving alone all the stuff that's at the higher levels (stepdefs and Data Objects) since we know that you're actively working on those things.


Убрать из адресов элементов рабочего стола: http://10.61.2.63/asu/


1786_fix_python_bindings (elainethale) - x86_64-MacOS-10.9-clang: Tests Failed

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


=) Та это я учу ++ и смотрю примеры, как люди пишут. Так что от смотрения толку мало =). Но, я у других видел, что они используют explicit для конструкторов с одним параметром. Может и Вам использовать? =)


It looks like the `set_a` and `set_b` variables could be eliminated, just using `chord_a.intervals` and `chord_b.intervals` instead. In this case, either just use `chord_a.intervals`, etc. each time so it's easier to read at first glance.

Better to name variables after what they are semantically rather than syntactically. So, `intervals_a` is better than `set_a`.


You can use self.get_by_key here


Updated & rebased.


Hi Niels,

Why do you want to include support for PHP 5.3 and (to a lesser extend) 5.4? This version is EOL and 5.4 will be EOL soon. The main reason I'd suggest dropping support for 5.3 and 5.4 would be the new PHP 5.5 ::class (http://php.net/manual/en/language.oop5.basic.php#language.oop5.basic.class.class), which is particularly useful for improving readability and stability of mock classes in unit tests.

I'd also want to suggest you add phpcs to Travis to ensure the project will stay PSR 2 compliant:

```
script:
    - vendor/bin/phpunit --configuration app/
    - vendor/bin/phpcs --standard=PSR2 app/
```

Ron


Should this say "are not" the cause?


Great formatting.


Actually, this range was specified because it is defined by IANA as the dynamic (or private/ephemeral) port range. All other ports are assignable for specific purposes by IANA and should only be used if assigned. When choosing an arbitrary port for a service, it should be from this range.


Some ssh use username/password. so prompt is expected and handled by expect script. BatchMode is not needed


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.479.64e25ff9cdb4d7b128317e284cad6bb692863382](http://build.palaso.org/viewLog.html?buildId=62309&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 140 Build time: 00:11:46


What does this do?


Yes they are in uicommons module and they retired class should be on TDs


ok! Rush the pr!


“with pages and feeds”?  Doesn’t make sense.  I’ll fix it.


with #59


preclass question for 8-27-15


Yes int() or None is good.
We should probably also check if the returned value is (uint32_t)-1 and also convert that to None.


I don't know why you talk about scaling. Just asking for a different width
and height from a rendered frame via MLT frame is not going to scale it.
You can request a different pixel format, however.
To use it as a gl texture it needs to be planar. The scopes should be a
different set of shaders for the same data.

On Sun, Aug 30, 2015, 8:13 PM Brian Matherly <notifications@github.com>
wrote:

> Sounds good. Perhaps you can upload the texture from the original frame
> somehow. Keep in mind that I aspire to enable the video scopes eventually
> and they will need safe access to the original frame data (unscaled). So at
> some point it may be required to create a copy.
>
> Keep in mind that there is no technical reason that the original frame
> data must be cloned before it can be resized. We could add a function to
> mlt_frame to resize the image into a new frame without modifying the
> original frame. It could be a variation on the clone function. Just
> thinking out loud.
>
> Let me know what I can do to help.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mltframework/shotcut/commit/04187163cbdbe99f97b720c4f40e5d53dfb56f08#commitcomment-12968704>
> .
>


Fixes issue #2


Although, a more succinct way would be to use Hash and zip:
```
part_names = ['bass', 'tenor', 'alto', 'soprano']
part_members = Hash[part_names.zip(members)]
```
This style is a bit more declarative / functionally inspired.


Not sure which is more clear, you be the judge.


It's one of anti-patterns: https://www.quantifiedcode.com/app/pattern/6803rnFX
Named function is better style.


s/requested/specified

since user specifies the file on the command line.  Requested may cause him to misunderstand the message.


oops! thanks


@lakshmivyas This breaks gitbot S3 uploads with errors like:

```
UnicodeDecodeError: 'ascii' codec can't decode byte 0x80 in position 105: ordinal not in range(128)
```

and the error is raised on this call:
```python
hash.update(chunk.encode())
```

Should we update our/gitbots code somehow or is there a bug in fswrap?


I saw it in another `test_somethig.py` where it was used `test_source`. It looks like `test_source()` does not work without `unicode_literals` with Python 2.7.


Awesome! :goat:


Important! Addition of y_adj, as in Guvenen code


pid/uid/gid are also per process (well technically threads could run under different credentials but that's a different thing) the problme with using proc is that by the time you get there and read it the original process may be gone and a new process with a completely unrelated loginuid may be found there. So if this is the only way to obtain the loginuid then we cannot use it for authorization purposes, at most we can use it in logs, is it worth ?


Thanks Vašku,
would /usr/sbin/ be better than / ?

Check my new commit.


The "$var != 0" expression is extremely confusing. It should've been
```php
if ($params['enable']) {
```
or even better
```php
if ($params['enable'] === true) {
```


@codewarrior0 Could you please take care of a proper fix?


slow slow slow


I made a mistake here by forgetting to fix the dependency constraints so that there are no upper bounds. Discussion in the original PR at https://github.com/ocaml/opam-repository/pull/4740. Please advise on the best way to fix.


Why did you add a trailing slash to this one?


Looks like it's broken now.


Looks like the documentation got flipped on these two.


The soname contains the ABI version. That's what the linker uses for matching other NEEDED libraries.

So you need to change the major whenever you have incompatible versions. The main issue is when people try to tie the project version to the major. AFAICS it's not what is happening here since major is being set separately and manually.  However there are some differences on how Linux and other systems use the major (if you want compatibility with others). See this explanation: https://autotools.io/libtool/version.html

And this is what I follow while updating the libraries: http://www.gnu.org/software/libtool/manual/html_node/Libtool-versioning.html


Arggg, you're right.


It could just be empty...


Just as a general comment, don't forget to remove all this duplicated code from NewTabUtils.jsm!


I went ahead and updated this in the GUI port.


I do not like the fact that you read data through BigInteger. It will ruin performance.


database tables?


I was just about to sort that, thanks!


Does this work? Shouldn't it go inside of the `switchTo` promise, like after line 84?


isShortDecimal


No, it will only generate an additional jar file with the sources that you then can upload alongside with the binary jar.


Worth to place comment what this function does ("Make sure map paths are relative to processed file"), that's not evident.


Comment added to `simplifyComparisonExpr`.


This was added in 592e857102e637120aeca42058d1f59be7705fc2.


Ops , I commented in the wrong commit. I will open an issue for it.


I already see this in uicommons. Could you be talking about something else?
td.retired, td.voided {
  text-decoration: line-through;
}


Now it checks 'EXT_frag_depth' too in capabilities.


Why not
```java
Validate.notNull()
```
?


This is renamed in 592e857102e637120aeca42058d1f59be7705fc2


Just for you!


If someone typed in the URL or manually linked to it and capitalized a letter in it -- sure.  To the best of my knowledge, `$_SERVER['REQUEST_URI']` isn't cast to lower-case anywhere.


:+1:  though I kinda liked 'sod off'


I don't yet understand completely what it is for.
The mode is

 * Saved in QgsMapToolAdvancedDigitizing
 * Only changed in QgsMapToolCapture
 * Only evaluated in QgsAdvancedDigitizingDockWidget

Do other tools not need to change it? Maybe we can have a short call to explain the scope of this :)


Still missing:
2 -> errno.ENOENT


Fixed ;)
Intent launchIntent = pm.getLaunchIntentForPackage(appPackageName);
				if (launchIntent != null) {
.....

https://github.com/ezio84/frameworks_base/commit/dcfa0226924ce0c6e0f183b39acb1334f73c09f1


\o/


ok


I had to remove this entry to compile


Unless fiddling with a low-level C API, we don't use any explicit `new`/`delete` in VAST. This one could be replaced with a `std::string`.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.459.64e25ff9cdb4d7b128317e284cad6bb692863382](http://build.palaso.org/viewLog.html?buildId=62308&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3356, ignored: 134 Build time: 00:22:14


Since this is universal and not Signal K specific I would like to leave this out. If you would like to keep it should we add a pointer to a relevant resource as a footnote?


*.orig files are generated when there are merge conflicts, we don't commit them. You can add *.orig to gitignore file


I know Clara.io will prefer half textures in some cases rather than full float textures (for memory efficiency and speed), but it will use float if half isn't available.  I would think we will likely offer this functionality in ThreeJS proper eventually.  Not sure how best to reflect this.

My preference, but it isn't that strong, is on each use case in the engine (bones may be solved differently than lights and HDR storage formats) one could write custom search functions to figure out what format to use, whether it is float or half textures or int textures.  And then it sets a properties flag that for this use case that lets everyone one know that the engine will be using this particular solution.  In some cases I found the search function to be fairly complex, especially to get around mobile browser bugs I found in half/float texture support.

It is because the results of these use case solutions are use case specific I'd prefer to not have them in WebGLCapabilties.

BTW I do not have strong opinions on an fairly internal class like this.  So do not take my feedback as super important.  I care more about functionality than internal design.


You should use `if choose not in`.


`elif`


We don't want to throw an error probably. Though, it might be better.


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


This fix is contained Twice inside this file (see line 75 above)


Yes, I already asked.


@nnarhinen - This is probably a unicode related bug in `fswrap`.  I will add a test and fix this tomorrow.  For now, you can fix `fswrap` to `0.1.2` in gitbot's requirements.txt to get around this issue.


*.js.map files are generated. In general we don't commit generated files. You can add entry to the gitignore file


Should be more specific, `"Character can't attack <target>"`


Using GitHub Page theme


Unhandled Fundamental Failure


You don't need this block, it will error further down.


Do you happen to have any unit tests for these functions? The other escaping functions also come [with some sanity checks](https://github.com/mavam/vast/blob/57b483aba012e345883cea1e87326da771946e72/test/unit/tests/string.cc).


Use `"%r requires a target." % (self)`


Sorry, it looks like my IDE cleaned up some whitespace issues.


93651760-RowPerObjectReport (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 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)