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

 TestSuite|TestCase | Pass | Fail | TIMEOUT | Skip
 ---------|---------| ---- | ---- | ------- | ----
 basic_test|suite_setup|1|0|0|0
basic_test|basic/test_create_vm|1|0|0|0
basic_test|basic/test_stop_vm|1|0|0|0
basic_test|basic/test_start_vm|1|0|0|0
basic_test|basic/test_reboot_vm|1|0|0|0
basic_test|basic/test_add_volume|1|0|0|0
basic_test|basic/test_add_vol_to_stopvm|1|0|0|0
basic_test|basic/test_add_volume_reboot_vm|1|0|0|0
basic_test|basic/test_add_multi_volumes|1|0|0|0
basic_test|basic/test_add_volume_negative|1|0|0|0
basic_test|basic/test_crt_temp_from_volume|1|0|0|0
basic_test|basic/test_crt_vm_with_volume|1|0|0|0
basic_test|basic/test_vm_securitygroup|1|0|0|0
basic_test|basic/test_create_iso_vm|1|0|0|0
basic_test|suite_teardown|1|0|0|0
virtual_router_test|suite_setup|1|0|0|0
virtual_router_test|__host/test_reconnect_host__|0|1|0|0
virtual_router_test|novlan/test_create_vm|1|0|0|0
virtual_router_test|novlan/test_reboot_vm|1|0|0|0
virtual_router_test|novlan/test_ping_vms|1|0|0|0
virtual_router_test|__host/test_reconnect_host__|0|1|0|0
virtual_router_test|vlan/test_create_vm|1|0|0|0
virtual_router_test|vlan/test_reboot_vm|1|0|0|0
virtual_router_test|vlan/test_snat_create|1|0|0|0
virtual_router_test|vlan/test_dnat_reboot|1|0|0|0
virtual_router_test|vlan/test_ping_vms|1|0|0|0
virtual_router_test|vlan/test_create_3vms|1|0|0|0
virtual_router_test|vlan/test_start_vm|1|0|0|0
virtual_router_test|vlan/test_3l3s_vm|1|0|0|0
virtual_router_test|vlan/test_remove_dhcp|1|0|0|0
virtual_router_test|vlan/test_2l3_vm|1|0|0|0
virtual_router_test|vlan/test_crt_vms_with_limited_ips|1|0|0|0
virtual_router_test|vlan/test_crt_vms_with_limited_ips2.0|1|0|0|0
virtual_router_test|vlan/test_crt_vms_with_limited_ips2.1|1|0|0|0
virtual_router_test|vlan/test_start_vms_simultaneously|1|0|0|0
virtual_router_test|vlan/test_crt_static_ip_vm|1|0|0|0
virtual_router_test|vlan/test_crt_vm_with_hostname|1|0|0|0
virtual_router_test|volume/test_add_volume|1|0|0|0
virtual_router_test|volume/test_add_m_vols|1|0|0|0
virtual_router_test|volume/test_add_volume_vr|1|0|0|0
virtual_router_test|volume/test_crt_vm_volume|1|0|0|0
virtual_router_test|volume/test_crt_vm_2_volumes|1|0|0|0
virtual_router_test|volume/test_crt_vm_24_volumes|1|0|0|0
virtual_router_test|volume/test_volume_ops_on_stopped_vm|1|0|0|0
virtual_router_test|volume/test_crt_vm_from_dvt|1|0|0|0
virtual_router_test|template_image/test_crt_temp_image|1|0|0|0
virtual_router_test|template_image/test_del_temp_image|1|0|0|0
virtual_router_test|sg/test_ingress|1|0|0|0
virtual_router_test|sg/test_egress|1|0|0|0
virtual_router_test|sg/test_icmp|1|0|0|0
virtual_router_test|sg/test_udp|1|0|0|0
virtual_router_test|sg/test_stopped_vm|1|0|0|0
virtual_router_test|sg/test_vm_ops|1|0|0|0
virtual_router_test|sg/test_2vm_egress|1|0|0|0
virtual_router_test|sg/test_2vm_ingress|1|0|0|0
virtual_router_test|sg/test_4vm_complx|1|0|0|0
virtual_router_test|sg/test_detach_l3|1|0|0|0
virtual_router_test|pf/test_2l3s_pfs|1|0|0|0
virtual_router_test|pf/test_attach_pf|1|0|0|0
virtual_router_test|pf/test_delete_pf_after_destroy_vm|1|0|0|0
virtual_router_test|pf/test_del_vip_before_add_pf_to_vm|1|0|0|0
virtual_router_test|pf/test_multi_pfs_sg_tcp|1|0|0|0
virtual_router_test|pf/test_multi_rules|1|0|0|0
virtual_router_test|pf/test_pf_after_destroy_vm|1|0|0|0
virtual_router_test|pf/test_pf_sg_tcp|1|0|0|0
virtual_router_test|pf/test_pfs_tcp_udp_same_vip_ports|1|0|0|0
virtual_router_test|pf/test_pfs_tcp_with_same_vip|1|0|0|0
virtual_router_test|pf/test_tcp|1|0|0|0
virtual_router_test|pf/test_udp|1|0|0|0
virtual_router_test|pf/test_vm_ops|1|0|0|0
virtual_router_test|eip/test_2_eips|1|0|0|0
virtual_router_test|eip/test_2l3s_eip_pf|1|0|0|0
virtual_router_test|eip/test_2l3s_eips|1|0|0|0
virtual_router_test|eip/test_attach_eip|1|0|0|0
virtual_router_test|eip/test_create_eip_vm|1|0|0|0
virtual_router_test|eip/test_del_vip_before_add_eip_to_vm|1|0|0|0
virtual_router_test|eip/test_eip_sg_tcp|1|0|0|0
virtual_router_test|eip/test_pf_eip_attach|1|0|0|0
virtual_router_test|eip/test_vm_ops|1|0|0|0
virtual_router_test|eip/test_vm_ops2|1|0|0|0
virtual_router_test|snapshot/test_backup_snapshot|1|0|0|0
virtual_router_test|__snapshot/test_create_100_snapshots__|0|0|1|0
virtual_router_test|snapshot/test_create_snapshot|1|0|0|0
virtual_router_test|snapshot/test_create_sp_after_delete|1|0|0|0
virtual_router_test|snapshot/test_create_image_by_snapshot|1|0|0|0
virtual_router_test|snapshot/test_crt_sp_for_both_root_...|1|0|0|0
virtual_router_test|snapshot/test_crt_sp_in_stopped_vm2|1|0|0|0
virtual_router_test|__snapshot/test_crt_sp_in_stopped_vm3__|0|0|0|1
virtual_router_test|snapshot/test_crt_sp_in_stopped_vm|1|0|0|0
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...|1|0|0|0
virtual_router_test|snapshot/test_change_max_sp_depth|1|0|0|0
virtual_router_test|__snapshot/test_change_max_sp_depth_l...__|0|0|0|1
virtual_router_test|regression/delete_sg_with_2_attache...|1|0|0|0
virtual_router_test|regression/rm_nic_sg_on_stopped_vm|1|0|0|0
virtual_router_test|regression/test_data_volume_with_vm...|1|0|0|0
virtual_router_test|vr/test_ping_vms_reboot_vr|1|0|0|0
virtual_router_test|vr/test_reconnect_vr|1|0|0|0
virtual_router_test|vr/test_start_vrs_simultaneously|1|0|0|0
virtual_router_test|__host/test_reconnect_host__|0|1|0|0
virtual_router_test|suite_teardown|1|0|0|0
Total|102|94|3|1|4


@AdsonCicilioti desculpa o atrasado para comentar haha. Acho que é mais interessante deixar a imagem `core/assets/images/placeholder.png`, só que tem que deixar ela realmente com transparência! Porque daí evita dos devs mexer nesse diretório **core**, com isso, se quiserem adicionar uma imagem *default* diferente, podem fazer isso via CSS através do `background-image`.


The printk's are useful for debugging but we'll want to drop them in a final version for the common case.  They could be preserved for unexpected error cases.  We don't want to spam the console of anyone who just happens to cause a lot of snapshot mounting and unmounting.


88737258-EPW-Ruby-Access (jasondegraw) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


No.There is no number between 9999 and 1000 that raised to the square would be a palindrome.
Breaking the loop would work only in case of 4 digits, but for 3 digits is a "No go".Is not a general solution

Proof: "The biggest palindrome  is 580085 determinated by the numbers 995 and 583"
and "The biggest palindrome is 906609 determinated by the numbers 993 and 913"


@yjun2 DWMR. Not a lot of code to add that functionality. Some things are simple!


ok, done


This hunk isn't needed when ismounted is initialized to zero.


[style] should be indented


Yes, it should be updated now.


Why do this?


+1
I will remember it.


?
Can we take money datatype?


Yeah, I definitely tried it, but the non-numeric references looked too goofy to me.  I'll think on it more.

Regardless, though, for maximum clarity and readability, I would want the references at the bottom to be in the same order as they appear in the text, so that wouldn't solve the problem _per se_.

The "real solution" here, is to just associate the link with the text itself and have the typesetting system automatically produce footnotes in the correct order... but that would be a different system altogether from markdown, which, in my view, is just a little bit of fanciness on top of what is fundamentally a text-only file.  In fact, the frequent complication of how to use markdown well is making me consider just abandoning it and going back to plain text.


this anger feature will lead into a smite feature in the future. If this gets too high, be prepared to kill an important NPC at any point in the story.


this is temporary. a function needed access that I could not immediately grant. Will revise the function soon.


Will fix it.


jjkjlkjlk


I think this specialization is redundant. when there are three or more parameters and the requirements of the specialization below aren't met, only the base template is viable and its definition is equivalent to this specialization.


Likely sometime this month.


Alright, I understand now. Thanks!


You are right that is exactly what is going on (sorry for the regression). You could also use `address.send(Runtime.Shutdown())` instead to avoid extra closure


已修改


I am absolutely sure there's a better way to do this. Unfortunately, for my experience level, this is the best option I can offer without redoing the entire setup process for MODX. If anyone sees a better way or implementing this fix, by all means, send my code to the great /dev/null in the sky and submit your fix.

For now, however, this seems to be the first hotfix offered to solve this problem, and it works. It may be sloppy code and not the most efficient means of doing this, but it works.


Could you please refine it refer with https://github.com/crosswalk-project/crosswalk-cordova-android/commit/efe8d24597579aa085e01f2d89c92d2565da8184


Accidentally pressed enter. It should say: 'Fixed showMessage'


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/63ad495e0c95ed315a2b21c6e5071f0537a420d1


merging basho/riak_cs/bugfix/default-multiple-choices-missing = 63ad495e into borshop-integration-1181-bugfix/default-multiple-choices-missing


basho/riak_cs/bugfix/default-multiple-choices-missing = 63ad495e merged ok, testing candidate = 1f954aaa


Needs work.


You don't need to mock email functionality, checkout `outbox` module[1].

With pytest, you can also use available `outbox` fixture[2], while in development django is set to use `consoleBackend` so not actual email sent out anyways.

Sample Test:
```
def test_some_function(outbox):
  some_function()
  assert len(outbox) == 1
  assert "sometext" in outbox
```

__Footnotes__:

[1] https://docs.djangoproject.com/en/1.8/topics/testing/tools/#email-services
[2] https://github.com/pythonindia/junction/blob/master/tests/fixtures.py#L49-L53


现在是有一个表存储收获地址信息，在下面，shopping_member_address这个表，
这个用户表现在还是需要的，考虑到以后如果做app的话，这个表可以用来存储用户信息的


This project existed a while ago, but I created and deleted a few different repositories while I was deciding how I wanted to host it for myself. It was hosted on Heroku originally, but once they changed their free plan and added uptime restrictions, I thought I would try moving to OpenShift. That was a bad decision, as I couldn't get it to work for the life of me. It's now back on Heroku for the foreseeable future. I don't see the uptime restrictions being a problem since I'm the only one that uses it, but I could always host it on my own server if needed.


不是，我们在后台管理的话，需要根据针对不同期数进行商品的增加，这个和公众号推送没有什么关系


test


这个商品表你根据你的意见修改一下吧，不用直接评论了


这个就是规格，按照一个字符串的话，长度可以再加长一些，45有点短了吧


回复建议，有时间看看


You've named the repository "HelloWorld.md" and have not created a true markdown file. You need to create a markdown file, add it to your *local* repo, commit it with a message, and git push origin master


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1324


Could you please import the class?


fix issue #2


预留的自定义字段


I'm busy tonight as well :) we can look at it tomorrow.


This closes #74 and #75.


Thanks, well cought


I am curious to know if creating interim state is really necessary (this is just a question). If Flux is designed around the core idea of finite state machine and the state changes triggering actions, why does user have to do extra bookkeeping for tracking previous states. Is there better way to do this that is more idiomatic to flux?


Getting this?

;

Starting
/home/graeme/git/mandelbulber2/mandelbulber2/build-mandelbulber-Desktop-Debug/mandelbulber2...

Log file name: /home/graeme/.mandelbulber_log.txt

Detected 8 CPUs

Default data directory: /home/graeme/.mandelbulber/

Warning: slotLogLineEditChanged() error: value from "logedit_flight_speed"
is not greater zero

(../src/render_window.cpp:144, void
RenderWindow::slotLogLineEditChanged(const QString&))

Warning: slotLogLineEditChanged() error: value from
"logedit_formula_scale_1" is not greater zero

(../src/render_window.cpp:144, void
RenderWindow::slotLogLineEditChanged(const QString&))

Warning: slotLogLineEditChanged() error: value from
"logedit_formula_scale_2" is not greater zero

(../src/render_window.cpp:144, void
RenderWindow::slotLogLineEditChanged(const QString&))

Warning: slotLogLineEditChanged() error: value from
"logedit_formula_scale_3" is not greater zero

(../src/render_window.cpp:144, void
RenderWindow::slotLogLineEditChanged(const QString&))

Warning: slotLogLineEditChanged() error: value from
"logedit_formula_scale_4" is not greater zero

(../src/render_window.cpp:144, void
RenderWindow::slotLogLineEditChanged(const QString&))

On Sat, Jul 11, 2015 at 12:55 PM, carbuncle grim <mclarekin@gmail.com>
wrote:

> Yes, I like the equations in groupbox titles too! Educational for the
> newbie user.
>
> BTW the "weight" tweak works  with mainRotations, you can "grow" the
> transformation (so is good for animation)
>
>
>
>
>
> On Thu, Jul 9, 2015 at 4:18 AM, Krzysztof Marczak <
> notifications@github.com> wrote:
>
>> I like equations in groupbox titles. It says much more than some weird
>> name of transformation.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/buddhi1980/mandelbulber2/commit/9620b313cfc8011a7f9d5620dcc5e9f395c9bbca#commitcomment-12065857>
>> .
>>
>
>


I did this to handle if the ```operation.produces``` was defined, and if not, take the ```swagger.produces``` set and if that isn't defined, it will be set to ```undefined``` and default to ```application/json```

is that line of thinking justified?


Pascal,
I found that using an .xls format would try to open in Excel but would prompt the following message.

![image](https://cloud.githubusercontent.com/assets/4065727/8642306/687d18dc-2965-11e5-842c-eed28bcac290.png)

If I select Yes, then the file would open.  This may be more intuitive than naming the file Excel.Downtime.xml

What do you think?


any specific reason to write at the offset of 10?


This seems like a pretty weak check to me. Couldn't someone just use `./../../potentially_malicious.sh`?


We might consider rewording this a bit since users completing the exercises won't have any `example.lfe` files.

I'm not sure if this document is targeted at them or contributors. My guess is that it would end up somewhere like [this][1].

[1]: http://help.exercism.io/getting-started-with-erlang.html


@dannon feel free to also have a look at: https://github.com/bgruening/galaxy/tree/lint-additions


fast-forwarding develop to borshop-integration-1181-bugfix/default-multiple-choices-missing = 1f954aaa


Perhaps stating the obvious, but there are currently no machines with 16 cores in the pool so your test will not run.


"attr2" is never read after been written. is this intended ? sorry i am just not sure about the intention of this.


Oh, I see ... yeah, we'll want to reword this so that it makes general sense for the user.


I have run `make test`


I would add this to fourslash itself, it seems generally useful


It's calling a hook.  See this article for more details on hooks:

http://codebyjeff.com/blog/2013/01/using-hooks-with-ion_auth


may we trade the magic number `5` for `large.length()`?


But people will never notice silent failures and

    ohnobinki@gibby ~/discount $ ./configure.sh --blah
    Bad option --blah.   Use --help to show options
    ohnobinki@gibby ~/discount $ echo $?
    1


I am perplexed why changing state and emit are decoupled in flux. I don't see any immediate benefits, and it may be error prone? It's ok if the answer is "it's just the way it is".


I would move this up, before you set isMemberCompletion.


What does that do?


Though, I guess this behavior is consistent with autoconf. OK.


Thank you for the fix!


By downloading RO. It's part of RO.


existingImportsOrExports


收获地址表有创建的，shopping_member_address就是这个，用户表可以保留，以后如果是app的话，需要有此表


Actually, I think `exports` represents the existing ones. `namedImportsOrExports` are the new ones. So why are the `namedImportsOrExports` being called the existing ones?


as of 2015-07-13


这个期刊表是为了后台增加哪一期商品而设置的，和公众号推荐没有关系，相当于分类的意义了


Don't do lookAhead


商品表按照你的意见你直接改就行了


just git practice session


It looks like [`INSTALLATION.md`][1] is actually the file that previous link points to, so maybe it's cool to keep this one aimed at contributors.

[1]: https://github.com/lfex/xlfe/blob/master/docs/INSTALLATION.md


这些是预留字段


I like 99% of the config-falls-over-dead-on-bad-option behavior, but the last 1% is that I lose any existing config.cmd when this happens.   On the mac it's not so bad because I can easily pull my last config.cmd off the backup, but on standard unixes I don't have time machine and it becomes A Chore :-(


是的，如果按照一个字符串的话，字段长度可以增加


I think these little helpers are overkill.


Later I need to convert this into Object Oriented solution


Very nice


现在考虑的是，库存问题，可能一批货放在北航，一批货放在北邮，所以方便管理增加这个表


Lose because of it not public an update of my kernel Phoenix for the lg l1 the IInd:p will begin denuevo with the updates it will take me time:P


Speed will be an issue unless we find someone that wants to Rcpp-ify `gridSVG`.  I don't think it is a huge dealbreaker since I envision the interactivity that we are implementing here as an enhancement in the final stage for communication/distribution rather than a necessity in the exploration/analysis phase where latency is a much bigger problem.  `dev.svg` is much faster, but introduces all sorts of other issues.

I made just one change to demo why the `stroke-width` change was not showing on mouse events.  Unfortunately, it only currently works with fill/polygon, and not other shapes, but it can be made more generalizable.

Guess I need to work on getting `svgPanZoom` ready for CRAN if you think we'll use it.


This is the token *before* the current position?


回复已经加了，订单表没有什么意见？


Instead of using the coordinates to store pairs of pixels, I am using the corresponding indices of the pixels to store pairs.


What does it mean for the completion list to be empty, but allow new identifiers?


Not too fond of that yellow tone, it's kind of hard to find a nice-looking dark yellow :wink:


@xinxinsh could you put some words in the comment w.r.t. what is improved in this change? thanks.


@karelmikie3 Dude look at this shit it makes mandrake look like a genious and calclavia look like someone who graduated school. You will die mod wasn't even this bad hell peripherals ++ looks fair in comparison... FOR LOOPS DUDE FOR LOOPS


Adding a chest section in the XML seems wasteful when it's already fully defined in the tile for chests


I don't believe so. This is a generic template describing how to apply Apache License to your work, so I think the intent is that here this remains as is, to be filled in by someone copying and pasting this template. I've looked at the LICENSE file for several other Apache projects, and they all leave this as is.


Correct on both accounts. I'm not sure if that'll remain the case WRT running in sequence, but there's a test now that captures the order.


Use members to hold pointer in RealmQuery still might have problems.
Here, if user call `findAll` in the UI thread twice before the ExcutorService run, the second one has a chance to pass a `-1` to `findAllWithHandover`.
And it really makes the RealmQuery Class become quite complex.
What about using the subclass or Runnable to manage those pointer's life cycle? One thing is if user doesn't hold the reference to Request, anyone would have a reference to the Runnable before the worker thread finished?


`Type` is the type for type in Dart.


eheh, might not be a horrible idea


@curiousdannii https://github.com/pid/speakingurl/commit/fb026793f1c1eb8a0cf7d934fe1c038cfa80a3ce#commitcomment-10854844


Danke für die Antwort

Von: Markus [mailto:notifications@github.com]
Gesendet: Donnerstag, 9. Juli 2015 21:30
An: markusk/StarterKit-Robotics
Cc: orje
Betreff: Re: [StarterKit-Robotics] Added a .svg file for convenience. Can be used easily i.e. at http://formulor.de on a 181x181 mm panel. (84b4401)

Hallo Orje,

1.	Die Motoren haben keine Bremse. Die Pins auf dem Motor-Shield heißen aber "brake". Und wenn diese auf HIGH gesetzt werden, wird "gebremst" (der Motor ausgeschaltet).
2.	Das Design des Roboters und auch das Listing sind nicht von mir. Im normalen Tageslicht ist das mit dem IR kaum ein Problem. Ein Ultraschall-Sensor wäre genauso denkbar. Hintergrund war vermutlich, den Roboter so günstig wie möglich für Einsteiger zu bekommen: Ultraschallsensoren sind i.d.R. teurer als IR-Sensoren.

—
Reply to this email directly or view it on GitHub <https://github.com/markusk/StarterKit-Robotics/commit/84b440199ba9e640b31987680c677874a6842b01#commitcomment-12092055> .Das Bild wurde vom Absender entfernt.

---
Diese E-Mail wurde von Avast Antivirus-Software auf Viren geprüft.
https://www.avast.com/antivirus


in reference to #46


Without the underscore, the compiler complains about an unused variable. You can get rid of the variable if you want


Are there any other similar codes, at least in GTM?   I think this kind of improvement should be done systematically.


True


To answer the question myself, I guess there may be cases where the store does not need to propagate changed state to controller-views. For example, transition from nothing to initial state. I think I am good.


Could you give more description about the following?

1. More specific background of this change,
2. If there's no serious side effects, because this removes some internal piece of planner code.


Why are these checks necessary? I thought emitEs6ExportDefaultCompat only gets called under these circumstances to begin with. I suppose you can assert them.


Why did you give the same value to OCLASS_PGXC_CLASS, OCALSS_PGXC_NODE and OCLASS_PGXC_GROP?


I may be too paranoid, but according to json.org's railroad diagram for ```object```, key is defined by ```string``` which can only start with double quotes. Some parsers do refuse to consider single quoted key as valid one.


Yeah. I agree that dependency really matter for users and `rocon_tool` isn't the right place to host script that brings heavy dependency. I wasn't aware of embedded world.

Anyway, there are three options to resolve the issue.

* Create a new repository for vision utilities and move this script out from `rocon_tools`. maybe `rocon_vision utilities`? But it seems overuse to create repo just to keep one script. Would `rocon` need vision utils later?
* Move the script to where it needs([rocon_nao](https://github.com/robotics-in-concert/rocon_robots/tree/nao). It was developed for image streaming for nao cam feed.
* Keep the script in `rocon_tools` but remove opencv dependency. So that can be used for systems where has already installed opencv.

I will go for option 2 unless there is good reason for option1. I am not convinced yet if we need `rocon_vision_utilities`.


I don't think it's important to use the erased signatures for the abstract check.


@ahomu what a nice feature! Would you give me this as a pull-req?


Finally :)


Aquí no quieres que el orden sea DESC para que el que salga sea el último que aprobó/rechazó?


`$(CMAKE)` so it doesn't have to be on the path


test git


the throttle_min method's default implementation returning the parameter default like this is a bit dangerous.  I think what's actually wanted is to ay that the throttle_min is not valid for some vehicle types (i.e. helicopters) in which case we should probably move the method down to the multicopter class and make sure that any callers (only the AC_AttitudeControl) library are working with a multicopter class.


`$(CMAKE) --build .` would be more general here


using Java


If the FindSDL*.cmake scripts do their jobs well (which I suppose), there's no need for this `if` anymore.


yay


Maybe `CMakeCache.txt` instead of `build.ninja` for generality's sake? How about adding a new variable `CMAKE_GENERATOR` that's easy to set to `-G Ninja` in your `Make.user`, and appending that to `CMAKE_COMMON`?


Please only use features we've learned about.  Exceptions and even functions have important subtleties, and this isn't a very good way to abstract out the error handling.  For now, don't abstract it.


Always use `auto`!


no need for set_yaw_headroom to be virtual because it's never overwritten (this was an existing problem, it was not introduced by your patch)


Would it be more or less efficient to just keep a context of variable-pairs which we are identifying? This would at least let us avoid calling `subst`, but perhaps lookup would be more expensive?


Thanks :)


We don't need to handle errors according to the assignment.  Let's avoid using features we haven't covered yet.  Also, note that this handling isn't correct: any sort of exception that is thrown doesn't mean the input wasn't an integer.  `catch (...)` is almost always a bug.


By convention, we put newlines at the end of output.


All the transforms that were speicified using the `transform` option were always applied before babelify and partialify, which is not always desirable because the parser will throw errors if you use ES6 features.


Usually you don't want `endl`, which is why we haven't covered it.  In short, it's a needless pessimization, when a simple newline will do just fine, especially here at the end of the program.


We haven't covered the tertiary conditional operator yet; please don't use it.


Hm, I suspect we should do it either way @jonsterling, otherwise I think we could unify `lambda(x. x)` with `lambda(x. DefinitelyNotX)` which is a little annoying. The idea being you can only unify variables free in the original toplevel terms


YW :)


There is no need for a trailing `/download` (which was specific to SourceForge), but if it is working no issue :smile:


Could these be preconditions?  We haven't talked about wide-vs-narrow contracts here.  It is a design decision, I suppose.


`0` isn't positive either; probably want `< 1`


I have had to write similar code in a number of places in JonPRL. Looking forward to having it factored out here! :)


Same comment as above with `endl`.


It's not clear that you're doing an optimization here to avoid all even numbers except two.  The code reads like 3 is the special case, but 2 is, isn't it?  See if you can rework it and make it clearer what you're doing.


Make sure you write a loop invariant.  This loop is especially tricky to figure out what the loop invariant is, because you're mutating `n` and `i` and implicitly doing a prime factorization.


A loop invariant comment would make it easier to see why this is needed, and why it's not covered by the above loop.  The code has all the information to show why, but it doesn't immediately tell you; thus, someone is going to come back in two weeks on a future assignment and have to modify this code, and they could introduce a bug without realizing it.


Always use `auto`!


This here is good: even though we have to do the branch twice, we're minimizing code duplication here.


Really useful comment.


sure, but we could do the following:

  - compute score without suffix & distance
  - if equal scores, try to break the tie with 'distance'
  - if same distance, try to break the tie with 'suffix'


This error handling is incorrect: std::vector<T, A>::push_back() can raise an exception, and you're printing the incorrect value (and also unknowingly potentially raising a second exception, which is undefined behavior (`==` bad thing)).  Hold off on exceptions for now.


See above for comment on `endl`


looks great!  the tests are awesome too!


looks good


Slightly subtle logic here.  The code reads: “Print everything in the stack.  When you are finished doing that, print out `" [empty]"s` if there is nothing in it”.  We're relying on the fact that the first step doesn't do anything.  It's clearer to make that explicit: “If there is nothing in the stack, print `" [empty]"s`.  Otherwise, print everything in it.

Alternatively, don't print anything at all.  This satisfies the spec, simplifies your code, and simplifies anyone who's writing a tool to parse the output of the language.


And here `auto`!


Make sure you use `auto` here.


By convention, we add a `\n` at the end of output for commands.


If we need a comment to say what it is, is `flag` really a good name?  Wouldn't `seen_decimal_point` be better?


Would it be better to explicitly check that the file is in the Git index?


Then, we could say “If `c` is `'.'` and we have not `seen_decimal_point`, which is clearer and easier to read.


d


I guess the faqpage was redundant. Javascript/jQuery in process.


Hi,

I test it and the _tasks_ list disappear (is never loaded).

Another thing was :

>Note the _**ObjectId**_ disappear

When I insert a task from **Mongo** command line:
```
{ "_id" : ObjectId("55a34366097e82c02a27226b"), "text" : "hola", "createdAt" : ISODate("2015-07-13T04:49:42.874Z") }

```

When I insert a task from _Todo_ form:
```
{ "_id" : "D8zeQhcS9MxRkPpcD", "text" : "From meteor todo listo", "createdAt" : ISODate("2015-07-13T05:01:43.866Z") }
```

Is that ok?

PD: I'm learning, sorry.

a JsFiddle of my code (Jade+coffeScript) : https://jsfiddle.net/ayv3eu6z/

Thanks.


if `run_name_p` is NULL, `ObjBencher::clean_up()` will try to construct a temporary `std::string` with a zero pointer. this will crash `rest_bench`. am i missing anything?


Empty statement blends in here and looks a bit like a typo.  Usually we separate it by whitespace or put it on its own line to make it clear it's not a typo.  The latter would look out of place in this code.


Cider -> CIDER
cljs -> ClojureScript


Add a newline before the code block.


Weren't rifles purposely un-critboosted because of the fact they got triple damage on non-crit hits instead? Crit rifles gave lots of knockback for pretty much no effort.


@Download So the callback now works as expected for the different ways of calling flip. Turns out your suggestion of temporarily disabling the transition was the best solution for setAxis. If you take a look at line 148 you'll see that the setTimeout with a param of 0 stopped working stably for me so i've temporarily got it set with a short delay until someone can figure out what exactly is going on there.

Also if i'm not mistaken nnattawat has made you a collaborator on the the original project (https://github.com/nnattawat?tab=activity)? So i assume the new code will all be moved there soon?


It seems everyone has their own input formats for this program, since the spec wasn't clear.  A good thing to do when the spec isn't clear is to document assumptions, especially if the user has to deal with the effects of those assumptions.  For example, this solution deals with decimals (which mine does not), but not with exponentials.  It would be good to document this.


Cider already depends on `tools.nrepl` 0.2.10, so I don't think this is needed.


Try not to use `endl`.


Add a newline before this.


rhino


@jonsterling This is what you had in mind I think? Now we don't call `subst` for every binder and we can only unify free variables. Needs some test tomorrow when I'm properly awake though :)


cider -> CIDER


I think this should only be done for ```$TRAVIS_OS_NAME = linux```. The docs are cross platform.


Looks like an inadvertent delete?


Always write a loop invariant, except for trivial loops.  Here, it's not immediately clear what you're doing with `n` and `frac`, and so a reader of your code has to figure out what your algorithm is.  We never want the reader of your code to have to stop and think hard about a certain segment of code.  Additionally, a loop invariant shows that you have; it should be simple enough that corner cases are obvious.  If you find your loop invariant getting too difficult to describe, you probably want to restructure your code, because that means it's getting too difficult for you or your reader to reason about (and thus maintain or extend).


You should also add the extra info to the connection-info command and to the connection browser command.


We haven't learned about postfix `++`, and it's probably doesn't do what you mean, because of its subtle behavior.


Always use `auto`!


It looks great when inspecting your hosted version. The problem is when
I pull and run locally.

When I pull from upstream, and run `npm run start` on `master`, I get
the attached errors. I get the same behavior when I run the server from
the `gh-pages` branch. This is running in firefox 38.0.5.

I also ran `npm install`, in case there were version updates for
dependencies. This still didn't help :-/

On 7/11/15 10:14 PM, Prakhar Srivastav wrote:
>
> Just deployed the latest build on my website
> <http://prakhar.me/react-surveyman/>. Seems to be working fine on
> Firefox.. are you seeing any errors in the |console|?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/prakhar1989/react-surveyman/commit/916dccebdf6408bbac2378cb6ba41d745d3e9888#commitcomment-12121547>.
>


LGTM.


![screen shot 2015-07-12 at 10 19 12 pm](https://cloud.githubusercontent.com/assets/508630/8643180/ffca44e8-28e4-11e5-89fb-a3245755a8ab.png)

(since this didnt work by email)


comment comment comment


Variable *parsetree_list*  is always been NULL so it's deadcode. Because commit 8a05756 remove partial of it and make the code dead. It's OK If not remove. But migration to 9.5 will be complicated with such code.


Here, a `while` loop would be clearer: the for loop starts out saying “my iterator is `s`”, and “to check whether the iterator is valid, read from `cin` into the iterator, ignore the iterator, and see how `cin` is doing“, and then “to update my iterator, ignore it and increment a different variable”.  The `for` loop is terser, but doesn't say what you mean, so people will get confused when reading as to why you phrased it that way.


This bit of code requires some explanation I think. Either a comment or an expanded docstring.


Same with `endl` here.


Hmm.. wierd. Can you try doing a fresh git pull, removing the old node_modules folder and running `npm install` again?


@rschmukler it looks as if this change has removed the ability to monitor `required` for an input. All of my 'ng-messages' are no longer firing in $error?


I like that you're sending other builds to transfer.sh, but you don't mention this anywhere. Please add a note about this in your commit and pr descriptions.


Your mobile detection is not necessarily a good implementation since many laptop and desktop PCs are now integrating touch screens and Google Chrome has long since reported touch-enabled devices.

http://www.stucox.com/blog/you-cant-detect-a-touchscreen/#touch-apis


@gpleiss - can you look at this? Is this what you were hoping for?


@JavaJens Forgive me for not understanding, but what purpose does this serve? Is it simply to send info about the install to the server for stats?


first try


用来缓存rowChooseChecked，方便在componentShouldUpdate中判断是否应该更新。


I wish past-me were considerate enough to let present-me know exactly what this did work around, so I could test if it's needed anymore..


@friedger Yes I just followed the same steps which I was following before Push rights are assigned to me. Like creating a separate branch and committing my code into it and then sending a pull request.

But my mistake I didn't selected origin and it was "gdg-x" selected by default in Android studio.

Will make sure I will check the proper repository and branch selected next time onwards.


This is really nifty, but took a while to figure out what you were doing.  `frac` especially isn't a good name for what it's doing.  Could this be simplified at all?  You're doing it in a one-pass way, so you don't have all the information you need at the start; only when you get to a `.` or the end of a string can you really normalize it and say “this is where the units place is”.  If the algorithm can't be simplified, I think the code can still be made clearer.  At first (except for loop invariants), don't write comments.  Try to let the code say what it means by itself.


I just found there are duplicated declaration of node_register and bkup_node_register. so clean it with add const due to warning in my case.


This isn't very idiomatic; see below for details.  Prefer a `while` loop here.


@tasomaniac no problem :) I enjoyed weekend by integrating design support library but the sad thing is I can't see my weekend enjoyment into the action :P no worries, will integrate those changes again and will send a pull request separately!


I used a pattern I saw here: https://github.com/angular/angular/blob/master/modules/angular2/src/core/compiler/directive_lifecycle_reflector.dart#L14-L24

Honestly switch/case would be more code and I bet the compiler is smart enough to turn this into a jump list even if this isn't using switch/case.


Sorry, It is a typo error. It should be PgxcNodeRelationId and PgxcGroupRelationId. I will fix it


This loop here is the crux of what makes the entire algorithm work.

Could the algorithm be simplified at all (maybe not without giving up performance)?  If not, could the code be simplified and made clearer, so a reader doesn't have to dig for the algorithm?


Ah, scratch that. Just occurred to me to look in TextSecure-Server for the important details :/ Sorry for the noise

https://github.com/WhisperSystems/TextSecure-Server/search?utf8=%E2%9C%93&q=fetchesMessages


This is not terribly idiomatic; see below.


Sounds good :)


seems we do wait for the completion of all ops in `ObjBencher::*bench()` methods. but it will bail out if any of the ops returns non-zero. did any of your ops fail? if that's the case, personally, i guess it's also fine to crash the rest_bench instead of waiting for all existing ops returns.


Now with extra recursion on children.


Good to know, thanks!


At first, my solution used `getline` too, but it introduced a really subtle bug.  Test the program with `abc^D` as input (`^D` being `ctrl-D`).  Notice how the input does not have a newline, but the output does.  `getline` looses the information about whether `\n` is really there or not.  We tend to use `\n` as a line terminator in UNIX, but this behavior is a strange corner case.

There's another way you can do this which might even be simpler, and doesn't suffer from that bug.


I have just added an intermediate RGBA buffer that we perform an integer scaled blit into and that is then uploaded into the texture and then rendered to screen by means of linear scaling. If this is still not good enough, I am out of ideas, sorry.

Another remark: I have just found that we are not yet applying the SDL_RENDERER_ACCELERATED flag to the renderer. Does this mean we are not yet taking advantage of hardware rendering at all or what?


An interesting thing to note: what constitutes a “word” here?  We're using `cin`'s internal parsing, which may or may not be what we want.  The spec didn't say, of course, and this is a reasonable interpretation.  But it may give unintuitive results at times.  Depending on the intended usage of the tool this may be important, or may be unimportant.


I think checking that the file is tracked by git is making a stronger statement.

There is of course the situation where your script executes another script in the repository. Since Overcommit doesn't (and can't) traverse the chain of dependencies of other scripts your script calls, there's only so much security you can squeeze out of this signature-based approach.


Good testing here.  We're getting closer to testing what we really want to test, but as you can see, we're straining this simple part of `cmake` to its limits.


OK, thanks a lot for pointing out.


Here is what I said in above comments :)

![check](https://cloud.githubusercontent.com/assets/888080/8643298/b28fd4ca-294f-11e5-961e-6f4cb9b7e1b5.png)


Is this really enough to test the behavior of our `echo`?  It's only a single line of input, it doesn't contain any funky spacing, and so forth.


Thanks.   As you see, there are many more warning messages in the make.    It will be welcome to have such improvement.


Same with this.  Does the program work with multiple lines?  With large inputs?  Etc.  I can look at your program and know it does, which is incredibly powerful, but having a good test suite is useful as a safety net for modifications, in case an important detail slips by.


this commit doesn't look right to me – this function doesn't do anything to the shadow module (other than copy it), so why does the commit message say otherwise?


hm,  the links you are refering to is using a apdu styled frame,  like in iso-7816,  which doesn't apply at all for the tnp3xx. (what I know of at least).
I never seen the reset behavior in any of the tokens I tried with this script. The token behavies like a normal mifare classic tag.


Since this commit it broke multigroups users, had to sniff in the older builds


@shopware/shopware-webdev can you please check. /cc @Phil23


A good test here would be to try `print` in the middle of the expression.

Another thing which a test would have caught here is something that slipped by me as I was reviewing the code below: this interpreter accepts `1 2 3` as a valid `PROGRAM`, but `EXPR EXPR EXPR` isn't a valid `PROGRAM` according to our productions in the write-up.


Really good tests here.


@tntup I will look into reverting this commit. What version of PEX are you using?


@MarinaSD я смотрел ссылку от @Lenchik, но в целом шаблон сыроват (тем не менее огромное спасибо @AndreyAkinshin за публикацию оного). Я уже писал месяцев 6 назад, что в целом шаблон весьма далек от того, что может принять нормальной строгости секретарь ученого совета (например Бауманки). Я взял данный шаблон и достаточно глубоко его перерабатывал пару месяцев (после первого отлупа в совете). Из того что на памяти: содержание не соотв. ГОСТ, заголовки, списки, межстрочные интервалы до и после абзацев (вы что-то можете почерпнуть из моего репозитория, но он сильно кастомизирован и не претендует на какую-то правильность и уж точно не замена текущему шаблону, но если там посмотреть по истории изменений файлов шаблона можно почерпнуть много нужного). В целом, если у вас секретарь строгий, готовьтесь к глубокой переработке шаблона...


不用去掉这个字段吧，而且应该是token


Interesting,  your basic idea could be used for the proxy-rely attacks mentioned everywhere but not to be seen in the PM3 world...   Needless to say, the token lookup could be done from an externa source..
There exists better sources for token definitions then what I put in the scripts.  I never got around to update them.  But if that lookup can be maintained without updating a luascript all the time it would be better.


Thanks!

Actually, I'm talking about how GitHub is rendering the Markdown (e.g., the README). It looked fine in Atom's preview, but sucked in GitHub.

![](http://i.memeful.com/media/post/odg36dy_700wa_0.gif)


@forth32 а так пойдет 22d5dbd6db32811ccc1631f0010b7f04cb6d0734?


It's at least better to have an assembly version of mem* or str* function.

What I'm complaining is that my commits are not coded for Linux kernel and
thus can cause weird issues.

I'm aware of ARM64 having those, but that came directly from Linaro - coded
for Linux kernel.

Too bad ARMv8 assembly isn't compatible with 7.
On Jul 13, 2015 9:19 AM, "Jake Weinstein" <notifications@github.com> wrote:

> Interesting that there's enough for it to be worth it in ARM64.
>
>
> https://www.codeaurora.org/cgit/quic/la/kernel/msm/commit/?h=torvalds/master&id=d875c9b3724083cd2629cd8507e424cd3716cd28
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_kernel_samsung_hlte/commit/3741ca212082f58e4406d37a1f9f7a45e0cbf0af#commitcomment-12124874>
> .
>


I would replace it with the mysqli counterpart, but it seems that it produces the argument invalid error.

I'll look into possible workarounds.


>I have just added an intermediate RGBA buffer that we perform an integer scaled blit into and that is then uploaded into the texture and then rendered to screen by means of linear scaling. If this is still not good enough, I am out of ideas, sorry.

Interesting! Are you able to upload a zoomed in pic to show the difference between your method and SDL2's "linear" filter?

> Another remark: I have just found that we are not yet applying the SDL_RENDERER_ACCELERATED flag to the renderer. Does this mean we are not yet taking advantage of hardware rendering at all or what?

Without SDL_RENDERER_ACCELERATED, SDL2 will still choose hardware rendering, falling back to software if not available. If SDL_RENDERER_ACCELERATED is used and the user doesn't have DirectX or OpenGL drivers installed, then the renderer won't fall back to software and exit with an error.


這邊可以改成

```
`/rest/post/'${SinglePost.dataValues.id}`

```


Looks like this breaks mobilespec on all platforms. Test failure log:

https://ci.apache.org/builders/cordova-android-osx/builds/31/steps/running-tests/logs/stdio

Logs here: https://ci.apache.org/builders/cordova-android-osx/builds/31/steps/gathering-logs/logs/stdio

W/PluginManager(  970): THREAD WARNING: exec() call to Media.release blocked the main thread for 18ms. Plugin should use CordovaInterface.getThreadPool().
I/chromium(  970): [INFO:CONSOLE(115)] "Error in Success callbackId: Media1993552855 : TypeError: Wrong type for parameter "src" of Media: Expected String, but got Undefined.", source: file:///android_asset/www/plugins/cordova-plugin-test-framework/www/main.js (115)

Looks like you need to `new Media("dummy")` to prevent failure.


Using less than 8 characaters makes it alot easier to spoof the generated name, which is something to keep in mind.


In addition, such code is for statement "insert into values(xxx)". There is a new machinisam implemented by commit 8a05756. Actually, the code should be removed by commit 8a05756 which not did. And I test the statement "insert into values(xxx)" after clear the code(this commit), it works well.


Cool. Did you end up running the vis? Here are the figures. It looks like
the algorithm IS indeed now picking up finer grained communities in the
patents (though with some mistakes). The wikipedia run looks perfect to me.

On Sun, Jul 12, 2015 at 5:15 PM, Alex Ledger <notifications@github.com>
wrote:

> the change could be this. if this line is not removed, then the algorithm
> returns one community. But I'm pretty sure that this line should be removed
> because np.sum(self.A) should be preserver the entire time. If we don't
> multiply by 0.5, then np.sum(self.A) is preserved.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jacobmenick/community/commit/c3fd72d0de37a19352e2ed81b49ca3d56c13fe9f#commitcomment-12124850>
> .
>


I didn't change the stderr output from 8-12 characters to 7-12, which was a poor choice.


Addressed in 74435290f7876af11cda0ec76ad1ca79f570f25d.


I noticed that is set before used. Nevertheless it should be initialized in
the constructor with a well defined value. I would use the invalid value 0,
just because it must be initialized from the file.

On 07/13/2015 03:02 AM, Neale Pickett wrote:
>
> It is set unconditionally in line 113 of soundsourcem4a.cpp, before
> anything else uses it; what value do you think this should have as a
> default, considering it needs to be read from the file?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nealey/mixxx/commit/171b8f44aa4ce637259e8f7075bcd634d23be3a7#commitcomment-12124994>.
>


Just a stab in the dark...  Check social media blocks...
Some blocks can not be implemented unless the current RFC is rolled back...
Hence 'confused' json
By roll backs creates an integrety problem, to put mildly...

My opinion...  You have a mole


Oooh, cool, and it's revealed to be even better if you run the graph layout
algorithm for more iterations (it's a "force-directed" graph, essentially a
n-body simulation where nodes repel eachother but links have some sort of
elastic pull. The number of iterations is how long the simulation is run
before the layout is displayed)

On Sun, Jul 12, 2015 at 11:01 PM, Jacob Menick <menickjl@gmail.com> wrote:

> Cool. Did you end up running the vis? Here are the figures. It looks like
> the algorithm IS indeed now picking up finer grained communities in the
> patents (though with some mistakes). The wikipedia run looks perfect to me.
>
> On Sun, Jul 12, 2015 at 5:15 PM, Alex Ledger <notifications@github.com>
> wrote:
>
>> the change could be this. if this line is not removed, then the algorithm
>> returns one community. But I'm pretty sure that this line should be removed
>> because np.sum(self.A) should be preserver the entire time. If we don't
>> multiply by 0.5, then np.sum(self.A) is preserved.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/jacobmenick/community/commit/c3fd72d0de37a19352e2ed81b49ca3d56c13fe9f#commitcomment-12124850>
>> .
>>
>
>


오오오.. 드디어 반영이..


가 아니라잠깐 이거 뭐에요? Ex-core?


@rikukissa Kiitos vinkistä, noissa vanhimmissa koodeissa on jonkin verran turhia div-tasoja, jonne oikeustarkastelu on laitettu.


I'm not sure if this is correct. Did you check the Kernel?


Really, dude?


I presume this was coming from running gofmt, right? If yes, can you please send this in a separate pull?


add files


PermissionsEx 1.23.3. Devbuild of Autorank 3.4-Dev-355 works with multigroups. B356 not. Thanks for replying fast, had to update Autorank, had a severe ratelimiting (have 6.4k of records) and today I noticed the multigroups were not compatible with 3.5.1. Hope it helps you!


Check it please.


There is no way to disable just the host name verification? We do want to check that the server presents a cert that is signed by the RootCAs we configure above.


There is already a pull request to fix this. #17490
Feel free to comment on this pull request.


Note that the comment for this commit contains some accidental text ('# Please enter the commit message for your changes. Lines starting').


Little nit:
missing space after the ",".

You can run "scripts/checkpatch.pl --strict --ignore LONG_LINE [patch]" to check for style-errors.

Thanks!


1st commit


Minor nit:
There is an extra "tab" here.


It shows that the MTI gem directly selects the "subclass", i.e. follows the association to the pseudo-child.


Did this change get regressed or skipped in 4.2.3?  I'm seeing this very problem trying to create a new rails 4.2.3 app on rbx 2.5.7 - it tries to install/bundle byebug and fails


Yeah, I messed up with the comment.  I'll do a brand new pull request which
fixes formatting and commit message.
I'll also check if setting ServerName in tls.Config can actually workaround
the weird CN returned by Cloud SQL instances:

On Mon, Jul 13, 2015 at 3:15 PM, Razvan Musaloiu-E. <
notifications@github.com> wrote:

> Note that the comment for this commit contains some accidental text ('#
> Please enter the commit message for your changes. Lines starting').
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/80nashi/cloudsql-import/commit/14bcac39dd823eafcace2a1a643a84dc18781574#commitcomment-12126591>
> .
>

--
YAMANASHI Takeshi


yes


test for commit


As mudanças funcionaram (ao que parece) e o sumário está completamente alinhado agora


Interesting, okay.  It's possible I was misinterpreting the errors I was seeing.  I'll do some more testing; I have a whole pile of tokens to go through.  Thanks for the comment.


这里创建了错误的全局变量


I didn't find the latter removed lines from the master.


There was 2 files with their names in another case, but github decided not to include renamed files into commit.

Fixed in https://github.com/darki73/FreedomCore/commit/c3eca242b4509a9dd54958a230cd813d57b50d5e


The variable name should be consistent throughout assignment. Ex., if you are using fname then last name variable must be lname not l_name


- Commit one by one.
- Commit Project file.
- Singleton pattern not implemented correctly.


- Commit project file.


+1, Will Take care of this in future,


Nope...after blasting the node_modules folder, I even tried with a fresh
clone directly from your repo. I am seeing these errors. :-/

Will have to take a closer look tomorrow evening.

On 7/12/15 10:28 PM, Prakhar Srivastav wrote:
>
> Hmm.. wierd. Can you try doing a fresh git pull, removing the old
> node_modules folder and running |npm install| again?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/prakhar1989/react-surveyman/commit/916dccebdf6408bbac2378cb6ba41d745d3e9888#commitcomment-12126289>.
>


Try to take Id's as auto generated if explicitly not specified to take it some value.


May I ask what errors you got?


Damn, 'forceful', not 'forcefull'.


I think it not entirly correct to use {@inheritDoc} here - this doc is a little different to similar at poulpe user. So, If you don't mind, could we leave this one as is?


Reviewed, try to not repeat the mistakes in other assignments.


Try to give some descriptive name not PROC1


Just `ERROR:         Couldn't read block..` sometimes from tnp3dump.  Adding in an additional retry didn't seem to help.  Adding in this "reset" seemed to help most of the time, although not all of the time.  I have at least one token that the game itself won't read any more due to read errors.


@rozman1234, @telaviv1 - I think that "find" is better than "locate" in this sentence. WDYT?


[With MySQLi you must pass a connection.](http://php.net/manual/en/mysqli.real-escape-string.php)


This covered by #1304 correct? If so I've added the spec in https://github.com/sass/sass-spec/pull/439


Reviewed.


It would be worth while to determine why this global lock is needed. At the script-compile moment (`BundleCreatorComponent->compileAndCreateBundle()`) and at the install moment (`BundleCreatorComponent->installBundle()`) locking occurs using the RPC name - allowing for multiple different scripts to be compiled simultaneously. If we cannot determine why exactly this global lock is required, then those locks are both unnecessary and should be removed.


The commit was meant to be called *Completed Checkpoint#49 Assignment*.


Implementing this is a very good idea!


Thanks for your help.


new data to commit


This is great, but can you change the 236 to "originals" to show the full resolution pic?


We may want to check if the number of attempts made is greater than the maximum attempts allowed.


Don't use u, p , follow naming conventions


Thanks. I've created the specs.


meron pala tayong non-yser people? meron ba tong significant something?


No sender/tags?


could you help point out what i am missing then? thanks!


Incomplete , no checks are found.


This commit (and this bit of code, in particular, I think) introduced some error messages in the jasmine tests:

<pre>
GET http://images/openlayers/north-mini.png net::ERR_NAME_NOT_RESOLVED images/openlayers/west-mini.png:1
GET http://images/openlayers/west-mini.png net::ERR_NAME_NOT_RESOLVED images/openlayers/east-mini.png:1
GET http://images/openlayers/east-mini.png net::ERR_NAME_NOT_RESOLVED images/openlayers/south-mini.png:1
...
</pre>


This breaks Windows support.


This project is the bridge between the ehcache-entity and the ehcache.


you can try the normal "Hf mf rdsc 0" to read sector 0..  Usually the tag needs some re-positioning over the antenna, when I gotten that error.
how does yr "hw tune" output look like?


done


Все действия необходимо вынести за пределы парсера. Завести отдельный класс под них. Здесь - оставить только построение AST.

В дальнейшем надо рассмотреть возможность дополнения арифметическими действиями.


Full name is asked, not first name


Emp_Id not asked


Could reproduce. Created ticket: http://jira.shopware.de/?ticket=SW-12022


`= {0}` should have worked.


Typo here :/


Missing hours worked in each activity.


Reviewed


+1. Forgot to add a check. Will add it and commit


Тесты-то, конечно, замечательные, но вот в чем беда: JavaScript замечателен тем, что это динамический язык. У вас ни один из тестов даже undefined на входе не принимает, не говоря уж о случаях, когда вам вместо массива передали объект.


@mrehanali Try to use <?php ?> instead of <?=


Hey Guys, I will be back in a week from Vacation. We can all sit down together and brainstorm.

As always I am with XFurry and we must be careful on how we handle some things.


Could you post this as new pull request?


Nice one! Watch out, @sindresorhus. :)


It's possible, because those images are not overriden?


Perhaps making use of /var/state (http://mail.savs.hcc.edu.tw/~chuavv/fhs/fhs-5.11.html) At first glance it does seem to fit with that you are using the marker files for.


:+1: Thanks!


testing


I think it would be clearer to put the lazy val inside `closureCallsites`, but that can come in a followup PR if we want to get this into M2.


Instead of the big blob of bash commands that follows this line, we can do something like
```
brew tap grpc/php
brew install php55-grpc
```


It is from this PR: https://github.com/WhisperSystems/TextSecure/pull/3530
I'd guess for the final merge request I wouldn't submit this commit.


Merging from Joyent


@joseconsador Please see this. I wanted to use anonymous functions because I wanted the dev to be able to see and control what is being called without having to look at basicTest. This is also more flexible.


Merged with locally squashing.


This doesn't depend on the kernel afaik.
Samsung's LSI department seems to change this flag for all the Exynos devices/boards, I saw them doing it on Exynos5420 as well, and brought the change to my Galaxy Note 10.1 2014, which didn't seem to make a difference either.
The Hardkernel guys don't have any special commits in their kernel which might be related to this either iirc.


I didn't know that was possible, @mariorj. I'll change that today.

Thanks!


Nope, because the clear on reboot is expected by the "subsystem" code, and /var/state one also doesn't allow it:

    "State information should generally remain valid after a reboot,"

Which is also sensible, further indication of weird "subsystem" code behaviour.


Thank you @SvenHerrmann for reporting and @Phil23 for verifying.


The brackets here in SocketHandler are useful because they highlight that this is a mutable call that can change.


This is causing build failures on x86:
```
mini-x86.c: In function ‘mono_arch_emit_epilog’:
mini-x86.c:5468:23: error: #if with no expression
 #if MONO_HAVE_FAST_TLS
                       ^
```


Ah I mis-read the code then, I though it should remain after a reboot. On hind sight that makes no sense since you moved it to /tmp.


@s0undt3ch I've removed thiese lines because script_retcode() calls script() that already creates python_shell and has deprecation warning.


But I suppose we don't actually effect anything by calling it so OK to remove


well poo


We'll need to do some research first about using connection time as a metric to kick or not.
A player who dc'ed with -21 yesterday will have -21 with a low playtime when he reconnects the next day if the mission has not been reset.

No objection to at least recording it so an admin can use it to make a decision, but we'll need to be careful with automation of this.

We might want a 'rule_category' in the rules - what applies to a Domi instance does not apply to an Evo or Wasteland instance


it should be:
```
#if defined(MONO_HAVE_FAST_TLS)
			supported = TRUE;
#endif
```


Found design patterns not implemented according to the language, cannot comment on understanding level


i am changing.


Isn't this being handled by https://github.com/AmpersandJS/ampersand-state/blob/master/ampersand-state.js#L43
?


Yeah, that caused the reboot loop to happen in the first place. What is mind-boggling is that there is no explicit state setting for the initial state, so when you change something unrelated it starts breaking...


I think there should also be `IRCSOCKMODULECALL(OnRaw(Message), &bReturn);` here, to not require modules to parse raw line themselves again and again, in every module.


This will return lowest five salaries not highest five


Nightly keeps failing for me. I had to go back to 4.9.2 and now it seems to work fine, building as we speak. I need to go, but I'll try to see why it fails with 4.9.3 when I'm back.


aasa


Need to select top 3 not top 5


Hi Olalekan,
actually I never look into the essential and fundamental matrices. They are created by the calibration method from OpenCV, but are not used anywhere. They are just stored, because they might be of some use for someone later.


So how can a module provide some new capability which sends tags, and take advantage of these new functions, if all tags are stripped away?


gute Idee - eventuell alle vier Ports per Config definieren lassen?
Wie sind die Standard-Ports?
Eventuell danach mit sip_transports_used abfragen und ausgeben, welche Ports wirklich genutzt werden?

http://pythonhosted.org/linphone/api_reference.html#linphone.SipTransports

class linphone.SipTransports¶
Object representing the SIP transports: its UDP, TCP, TLS and DTLS ports.

dtls_port
[int] The port used for DTLS SIP transport

tcp_port
[int] The port used for TCP SIP transport

tls_port
[int] The port used for TLS SIP transport

udp_port
[int] The port used for UDP SIP transport


Could not see some part of the induction implemented, and not sure about some other parts


for the sake of consistency, can you change "2" here by "2u", please?


You can use alias for table names also
like

From EMPLOYEE_SLAB es INNER JOIN Employees e ON e.employee_id = es.employee_id.


can you remove this tab, please?


can you remove the extra tab, please?


If you want...


v << pixel[0], pixel[1], pixel[2], pixel[3];


v << pixel[0] , pixel[1], pixel[2];


Could you please provide me your ramdisk on XDA?


v << pixel[0] , pixel[1];


Why was this necessary? For the context we had decided on semver for libs. If api-libs got fixes it should get a stable release tagged (`1.0.1` I would guess if fixes only).


@matsko Will this fix be incorporated in 1.2.x branch?


:+1: looks awesome!


see #10


Employee_Backup table should be in another database.

You used it in same database.


see #11


And is the deprecation related to saltenv? Of so, ignore me, if its related to a different deprecation warning, this one should stay.

We should enforce the use of `saltenv` over `__env__`.


Column Sex should be one char.


@VasilyShelkov using a `default` value in this case is not optimal for the following reasons:
- a Models should represent the real world data as close as possible having "testPlace" as the UUID is not a real-world requirement, we want the UUID to be a real UUID in all cases
- since the field is unique, only the first ever Place would have a default value, the ones after that would raise an integrity error if created without a UUID when trying to use the default value; using `unique` and `default` together is rarely a good idea.


Related PR: #266. Thanks.


je les ai vu en concert en Juin :3


Thanks @georgebrock! You are, without doubt, the king of top tips.


yes please do, it is more helpful this way.

I use your tool with non-IT engineers and it will surely be more friendly
for them

Ciao

2015-07-13 5:46 GMT+02:00 Joseph Plant <notifications@github.com>:

> Pascal,
> I found that using an .xls format would try to open in Excel but would
> prompt the following message.
>
> [image: image]
> <https://cloud.githubusercontent.com/assets/4065727/8642306/687d18dc-2965-11e5-842c-eed28bcac290.png>
>
> If I select Yes, then the file would open. This may be more intuitive than
> naming the file Excel.Downtime.xml
>
> What do you think?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ampla/ProjectAnalysis/commit/eb2691b9b11a1d4dcfd7ecd6947f03e30f09b21b#commitcomment-12125710>
> .
>


We should say that it's not a _node_ version (NPM is often melt with node), But really a way to run a vertx application.


Naming conventions should be followed.


I think this is a nice way of doing it. Can't test it here though, but it looks like it should work.


Create a table of Designation and drop it


Hi @bradharding ,

> Are you able to upload a zoomed in pic to show the difference between your method and SDL2's "linear" filter?

Sure, here are the comparison shots (well, they are not zoomed-in, but I guess you can handle that):

This is the current state (https://github.com/chocolate-doom/chocolate-doom/commit/777148cae2032bc2a33754329e7ecb52017a21cc):

![chocolate-doom_777148ca](https://cloud.githubusercontent.com/assets/6449237/8644745/f00885f8-2941-11e5-8771-46a59f7a3272.png)

This is the previous state (https://github.com/chocolate-doom/chocolate-doom/commit/970a54d1cf2a980fd3bb5412d2e0cfc9ae8898ec) with "nearest" filtering:

![chocolate-doom_970a54d1](https://cloud.githubusercontent.com/assets/6449237/8644752/06b2716a-2942-11e5-9c7f-00553db7a61f.png)

And this is the previous state ((https://github.com/chocolate-doom/chocolate-doom/commit/970a54d1cf2a980fd3bb5412d2e0cfc9ae8898ec) with "linear" filtering:

![chocolate-doom_970a54d1linear](https://cloud.githubusercontent.com/assets/6449237/8644763/166ff276-2942-11e5-9069-3069e6b838eb.png)


newline missed


Create a unique index on the first name and last name  and a full text index on first name of the employee table


newline


Hi guys,
I am dealing with tons of time series data (mostly environmental sensor data for my phD) and thus I started to play around with influxdb. Usually, at our institute, all of our analytics and modelling tasks are empowered by R scripts. This is also the reason why I started my influxdbr project at https://github.com/dleutnant/influxdbr. It is still under development (and therefore more or less "beta") but already provides basic functions to let R communicate with influxdb (>=0.9.0). It especially deals with the new line protocol and uses R's famous xts object to handle time series data.
Of course, I would be very happy to contribute to a general and active repo for an influxdb R library.
Please let me know what you think!
Cheers
Dominik


Must be auto incremented.


Complete this:

1.Get current date in the format specified in example -Mon  20th sep 10, 1:30 pm
2. Get unix timestamp of date 1st Jan 2010 also display this date in the same format.
3. Add 2 days in current date and display in format specified in 1st point


remove branch test.


Great


Complete this:
Count sum of a column in which one of the values is Null


Reviewed


For the record, http://hackage.haskell.org/package/base-4.8.0.0/docs/Data-Fixed.html#v:MkFixed says it's only available (i.e. exported) since `base-4.7`


The main difference between the first two shots is the rendering of the ammo table in the lower right corner. The third shot is out of the question, IMHO.

> Without SDL_RENDERER_ACCELERATED, SDL2 will still choose hardware rendering, falling back to software if not available. If SDL_RENDERER_ACCELERATED is used and the user doesn't have DirectX or OpenGL drivers installed, then the renderer won't fall back to software and exit with an error.

This is very valuable information, thank you!

So, the SDL_RENDERER_ACCELERATED merely enforces usage of hardware rendering, right? Do you happen to know the purpose of the SDL_WINDOW_OPENGL window flag?


Wrong alias


Why do we need to use this line?


I'm not sure I agree here dude - we don't yet have any payloads/messages that directly contain a blockheight, so with this being our first usage in an Omni transaction it's essentially precedent setting.

My own feeling is that using 8 bytes of payload instead of 4 bytes for a blockheight is overkill (I think if we hit 4 billion blocks on the Bitcoin blockchain we'll have bigger concerns hehe) and we're better off setting the precedent (and perhaps noting in spec) that a blockheight field in a payload is of the following type:

Field: Integer-four byte

    Description: used as a multiplier or in other calculations
    Size: 32-bit unsigned integer, 4 bytes
    Valid values: 0 to 4,294,967,295


Use proper naming conventions for cursor, procedures, triggers, common table expressions


You can take aliases also, ex. Employee as emp


This was committed to master (197f0c06195b87efd5857ed0c143ed7f05a9de8d), REL1_2_STABLE and REL1_1_STABLE branches.


This would also work right ?
$command = '-c core.quotepath=false diff --name-status '.$remoteRevision.' '.$localRevision;
Space between remote and local revision

Thanks


Clever!


c is not appropriate alias


Wrong alias b


LOL. This said "Looking for people nearby..." fwiw.


Can you please remove this outright? Content-type was in the last version of the spec, but Francois thinks it will be gone for good (as it is considered out-of-scope -- however there is also a potential exploit were this left in the spec, which would add even further friction to CT re-introduction).


Expunge any mention of it whatsoever :)


This is doing the same thing as the scheme function purge-hypergraph (except that remove_descendants removes instead of purging).

I wasn't sure if the term descendants is right, looks like I should use hypergraph instead.

Are the notions of ancestors and descendants not used in the atomspace? Would it be appropriate?

I looked at the glossary http://wiki.opencog.org/w/Glossary but it doesn't say anything about those notions.


that's a lot of code duplication


можно переехать на битбакет + TortoiseHG. просто исходники текущие туда закинуть


I can't see it @BobRay


Oh, I was mistaken (thought you commented on another line). This is because there will be many different `AxiomSelection` "subclasses" and this test prepares for those.


//读程序
class ForLoop
{
	public static void main(String[] args)
	{
		int i=0,j=5;
		st:for(;;i++)
		{
			for(;;j--)
			{
				if(i>j)
					break st;
			}
		}

		System.out.println("i="+i+"j="+j);
	}
}


Reviewed


As far as I can see `defaultLogger` and `vhistLogger` are constructed but never used. Why is that?


Just fixed the issue in the development branch https://github.com/hulken/temploggning/commit/8a5384199b5aba1612c1722af2cd655f2bb6af65. Not fully tested but giv it a try.


> reusable modular pieces

that’s called modules, and you use them by importing them, not copy/pasting them :laughing:


Done, see f60729b.


Thanks: 68688512e00f386a7f9fd2136be7658c321a4d6d


No go. Domain must be validated here.


You don't want to drain  user battery and make its GPS that crazy =) a value that  worked me in the past is 3000


ditto here, I guess even 1 meter will make a difference on battery drain, etc.


The test name is not matching the test.
The request should provide a non-existing uuid as a filtering option, and the assertion should be that `response.status_code` equals 404 not found. (I think I provided bad specs by saying it should return Bad Request, it should actually return Not Found, so the test name should actually be `test_should_return_not_found_if_place_does_not_exist`)


You mean the unnecessary `fmt.Sprintf`? I haven't noticed it before.


Yoohoo!


Just a small note:
All the other extensions don't have the escaped quotes. I would suggest to follow the form of the existing ones if possible


```bash
_ans=${BASH_REMATCH[0]##0}
_ans=${_ans:-0}
```


I have installed the Rasptimer-1 and all works OK except the various day Intervals are never detected. This seems to be because when a schedule is triggered it never calls the runGpio function, it just bypasses that somehow and switches on the pin thus runGpio function is unable to trap the pin write to test for days to be switched on. I am particularly interested in this section as I am trying to extend the schedule to individual days and adding more schedules. I have already added +bump -bump buttons to bump the schedules up and down in 15min jumps to save having to reschedule and that works well with Johannes Ernst Rasptimer and Rasptimer-1. Any ideas? Rgds Dave
larapenta   at   hotmail   dot   com


And what happens if the user have the services deactivated?
It should show a settings dialog to warn the user about that and offer the possibility to change it.
Also remember that you have to check for both GPS status and network status.


If the GPS is disabled you can use the network provider.


ACK for everything else :)


I purposely listed all the options so that if someone adds a new message to the MessageType enum, the compiler will warn us that it needs to be explicitly checked here.


Checking the count only can be misleading sometimes (e.g. someone changes `queue_count` to 1 for whatever reason, and changes the code so that it returns an error, this test would still pass...).
On line 35 I would assign the return value of `create_batch` to a `queues` variable, and here I would loop through them checking that each of them is in the response.


as above


I'm not sure this is "okaish"... it can at least have a location near from where you are. But also, this kind of location should be used only if neither GPS provider or network provider is available.


Your patch is interesting because, although per thread history as already tested in the past, it is the first time that is bounded to splitpoints. I have rewritten your patch, focusing only on this and removing all non-related stuff.

I have submitted a couple of tests.
http://tests.stockfishchess.org/tests/view/55a36db70ebc590abbe1ba39
http://tests.stockfishchess.org/tests/view/55a36dbc0ebc590abbe1ba3b


Thanks for the comparisons. I'm playing around with DR now to see if the feature "fits".

> So, the SDL_RENDERER_ACCELERATED merely enforces usage of hardware rendering, right? Do you happen to know the purpose of the SDL_WINDOW_OPENGL window flag?

`SDL_WINDOW_OPENGL` doesn't appear to be necessary. The docs say "window usable with OpenGL context". Maybe its necessary for when using OpenGL specifically?

While I'm here, a few things to mention that may improve the hardware scaling for you.
* Under Windows, Direct3D is used by default, which I've been found to be slower than OpenGL. Might be worth using `SDL_SetHint(SDL_HINT_RENDER_DRIVER, "opengl");`.
* The calls to `SDL_SetSurfaceBlendMode()` aren't necessary as `SDL_BLENDMODE_NONE` is the default.
* Despite what the documentation says, I've found `SDL_UpdateTexture()` to be faster than using `SDL_LockTexture()`/`memset`/`SDL_UnockTexture()`.


Same here as the previous comment, imagine someone adds new Authority to the enum list, and by chance that enum has group as a source. The person who adds it will most likely forget to look at this `from_group` function and modify it. And we have a bug.


Что я не понимаю: Вижу, что удалена строка `SerializedObjectsView.commonUpdater.fireChange();`. Это была лишняя строчка кода ? Если нет, то где компенсация этого удаления ?


Is there a reason values are hardcoded and not read from a file that holds the defaults?

So the logic would be, load a baseline config unless a actual one is available?


You don't save the location? Probably I'm missing something, but how do you get the location from this class?


The way you connect the callback it is unexpected to get a NULL user_data. Therefore this should be an error condition (catched by an assert (g_return_if_fail()) instead of choosing the current doc.


Link to original https://github.com/TE-xTsuyoshiATanabe/testrepo/commit/66a5b6a728adc800c1404e72ce938d8b9d10a671


`"false" -> false` (no quotes)


I am not really sure why do we need the user to specify service headers list. IMO if the original thrift file is untouched, all of its deps are untouched and this rule configuration is same, we skip the build.


Why is this here?


Once again, you broke the database manager :) . Seems it counts to three now; time for breakfast !


/usr/local/etc/config.xml is being probed and only rejected on mismatch. The OPNsense defaults are shipped in /usr/local/etc/config.xml.sample, so you are free to modify config.xml according to your needs. That is also the file that is used on factory reset.

If all fails, the code above is executed. We can't assume fuzzy matching the LAN and pull the settings from the file even on interface mismatch. Setting ports is always the first step and it cannot be more clever (unless you override the factory defaults).


When you handle both GPS provider and network provider, you can change here the one used. In case GPS is enable always use GPS provider, if not, network provider.


`Babel` and `ES6`

Or even better, just write `ES2015`.


Thats better!! you move quick!... thanks


Please follow url naming conventions: http://www.django-rest-framework.org/api-guide/routers/#simplerouter `name="queue-list"`


@jerrinot I don't care about the code duplication at the moment. Lets focus on the idea


I'd recommend to list all de dependencies in the section just above this one (in [..] ++= Seq ( [..] ). This way all of them are together


Same as previous function, and showing the warning dialog with the possibility to change gps settings here as well if any service is available


I will be taking a slightly different approach here. Let the user not specify any headers etc. The pretty message of building cc_thrift with libs <here>, headers <here> can be split into two and the headers section can be printed after thrift command is done (btw, they would still appear at the same place to end user).


I also went with explicit type declaration of variables when declaring variables that were dynamically initialized... Just because it felt so alien to initialize then change right away. I did read your point about using `auto` as much as possible, initializing something, and then how to change the value, but it stuck in my mind as just what is _ideal_ and not necessarily do it even when it leads to dummy values. Re-reading now after seeing your responses here about `auto`, I realize it really is better to do a dummy value then change right away?


ok, fair enough


Yes, I have since updated my copy of travisify to include 0.12 (and made some other tweaks as well). We're currently in the process of going through all scijs packages to make sure everything is consistent (and adding travis support is one of the steps), so I'll probably leave it until the rest has been done though (unless you want to go through and create pull requests for all packages that still have the older .travis.yml, see https://github.com/scijs/ndarray-gaussian-filter/blob/master/.travis.yml for an example).


just fyi: you can use s"" for this. Makes the code look nicer.

print(s"${highlighter._2} highlighting combination of ${highlighter._1.instructions}")


I know. It happens cuz settings doesn't have permissions to write to that file. I'll fix it with a sepolicy


Good work :)

Can you remove the app/views/.columnsfunction.html.swp file? You can also filter *.swp files in the .gitignore file so it will not happen again.

I will then merge your branch.


Не, все такие случаи учтены. Например, когда передан не массив объектов:
it('must throw exception when array is not an array', function () {
    isThrows(render, '{{#each people}}{{+lastname}}{{/each}}', {
      people: ''
}).must.be.truthy();


@JoanaDalmeida please do not modify the .gitignore to achieve an update on ignored files. See [here](http://stackoverflow.com/questions/8006393/force-add-despite-the-gitignore-file) for more details on how to commit ignored files.


Code looks good. Thanks for doing my job. :)


+1 for following and mentioning PEP 8 style conventions.

For future reference:
http://legacy.python.org/dev/peps/pep-0008/


good! needs to make sure "convert" is in PATH variable of your BASH. But it's good.

consider working on this in parallel:

new File(outputDir).listFiles().par.foreach(pdfFile => {


# [Adding Action Buttons] (https://developer.android.com/training/basics/actionbar/adding-buttons.html)
The action bar allows you to add buttons for the most important action items relating to the app's current context. Those that appear directly in the action bar with an icon and/or text are known as action buttons. Actions that can't fit in the action bar or aren't important enough are hidden in the action overflow.

![An action bar with an action button for Search and the action overflow, which reveals additional actions.](https://developer.android.com/images/training/basics/actionbar-actions.png)


this is nicer, I like the push of the memset down from chunk into fixed_cube.


You should probably just use something like [generator-nm](https://github.com/sindresorhus/generator-nm) to scaffold your modules.


when I set the interval to empty string this fails on an exception. So you need to catch the NumberFormatException here.
Also please be careful what happens if the user sets "0" - you need to return some default value or handle it somehow.


use `django.utils.timezone` instead


A nitpick - if you chose isWikiaMobile below, you might consider using isInvalidImage here for consistency. It also highlights that it's a boolean var (thank you for the lack of static typing, PHP :imp: )


Minor one: I would prefer to just return a `bool` as the return type is really binary (success: (), error: FailedSignature), and I would also prefer to call it `verify` as opposed to `check` (appears to be standard in crypto world).


good!


Why do this?


Не хватает еще запрета доступа к свойствам вида prototype и constructor. И добавьте тест на вложенные циклы, внутри которых будет еще условие.


Еще пары-тройки тестов ошибок парсинга недостает.


That was 2 years ago...
If you want to add this extension back, you should take a look at the [requirements and steps required](https://github.com/github/linguist/blob/master/CONTRIBUTING.md#adding-an-extension-to-a-language). If it meets the requirements, we'd welcome a pull request!


All action buttons and other items available in the action overflow are defined in an XML menu resource. To add actions to the action bar, create a new XML file in your project's `res/menu/` directory.

Add an `<item>` element for each item you want to include in the action bar.

The `icon` attribute requires a resource ID for an image. The name that follows `@drawable/` must be the name of a bitmap image you've saved in your project's `res/drawable/` directory. For example, `"@drawable/ic_action_search"` refers to `ic_action_search.png`. Likewise, the `title` attribute uses a string resource that's defined by an XML file in your project's `res/values/` directory.


This is really not tied to infobox, it's something along the lines of normalizeHTML


@VasilyShelkov Good work! Nice use of the rest framework (filter), factories. The tests are also almost there.
The one thing I would point out is the `test_should_return_bad_request_if_place_does_not_exist` test, which is a type of mistake that is difficult to track down, therefore can be somewhat dangerous later on. (See the inline comment.)
For some small fixes please see the inline comments.


Perhaps you want to use |ALOGE_ERRNO| for consistent style?


Hello
I'm following your guide https://github.com/umiddelb/armhf/wiki/Installing,-running,-using-docker-on-armhf-(ARMv7)-devices to install docker on my arm device, but I got one error like as followings. May I know your comments?

root@localhost:~/docker.io# apt-get install docker.io
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package docker.io
E: Couldn't find any package by regex 'docker.io'


Дмитрий Петрович, это все не сложно сделать, тем более я уже подобное делал. Только смысл какой, раз уже все работает. Кто этим пользоваться то будет?))


Code looks good at first pass.

Need to try that in game. :)


@at15 感觉好像不好测试？


To place the menu items into the action bar, implement the `onCreateOptionsMenu()` callback method in your activity to inflate the menu resource into the given `Menu` object.


why didn't you use standard what() ?


It's better now, but I'd use an IF statement here to check if findByTurkerId returns an empty result and only then create it. More readable code


https://github.com/bogachev-pa/rdo-xtext/commit/7da840ec62550639950352ee7de018e30d96decd#commitcomment-12124090

Внутри `notifyChange()` вызывается `fireChange()` у всех подписчиков.


I'll document `filterModuleExports`


Why it's unknown? You know the reason.


This line has to go outside `HAVE_BZ2`.


What's changed? You still duplicating error message...


When the user presses one of the action buttons or another item in the action overflow, the system calls your activity's `onOptionsItemSelected()` callback method. In your implementation of this method, call `getItemId` on the given `MenuItem` to determine which item was pressed—the returned ID matches the value you declared in the corresponding `<item>` element's `android:id` attribute.


s/Google/bluedroid library/


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/17832/testReport/) for this commit**


Sorry, I haven't understood the comment here...is it about about the spaces around the comma?


What's changed? You still duplicate error text.


I'm used it for extended info. But other function is for general info about
some kind of exception.

2015-07-13 11:05 GMT+03:00 Maxim Kot <notifications@github.com>:

> why didn't you use standard what() ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/libproject/libproject/commit/8a85e48a3d7f801310e72667d327b0c3d497e9a8#commitcomment-12127804>
> .
>


`exports` represents the symbols exported from the module, and would be displayed unconditionally in a naive approach. `namedImportsOrExports` are the ones that have already been named by the user in the import/export clause.


Что значит, кто этим будет пользоваться? Это вопрос качества выполненной работы и вашей оценки за практику, соответственно.

Заодно и сделаете нормальные коммиты в репозитории - потому что сейчас у вас сообщения представляют собой нечто убогое.


That is true - I never implemented that.

I think we need more settings because currently we can't have logger on a per module basis - this for example is logger for the whole vhost not only for the storage module or the cache one.


Whoops, thanks for catching that.


A new pull request created


Let's upgrade this to the latest Rails version: https://rubygems.org/gems/rails/versions/4.2.3

On a new branch, in the project root, try running `bundle update rails` to update the rails gem. This will also update the gems it is dependent on. Once the update is complete, run up the `rails s` server and make sure things seem to be working OK. If you've written any specs also run those to make sure they are still passing after the update. If the update seems to have gone OK, then go ahead and merge the branch into master.

*For some bonus material, read the Gemfile.lock file and try to understand its format and see how you could figure out rails dependencies from it, the section of interest looks like this, note how the dependencies are nested and tabbed under rails:*

```
    rails (4.2.1)
      actionmailer (= 4.2.1)
      actionpack (= 4.2.1)
      actionview (= 4.2.1)
      activejob (= 4.2.1)
      activemodel (= 4.2.1)
      activerecord (= 4.2.1)
      activesupport (= 4.2.1)
      bundler (>= 1.3.0, < 2.0)
      railties (= 4.2.1)
      sprockets-rails
```


I don't necessarily agree; they're reusable, descriptive, and avoid allocation at use sites.


> Need to try that in game. :)

If everything goes well, you should not see any difference ^^


This is the parent of the prior token.


Вот теперь понятно, ожидал ссылку на строчку с вызовом `fireChange()`


Do we really want to lose the relay information here? I don't think the name `create_send_on` suggests that, to me it only suggests that the destination will change, while all the other information is preserved. If in the code we do want to lose it, I think we should be explicit about it, something like:

```
let msg = message.create_send_on(...).make_non_relayed();
```

Again, a minor one: I think _forward_ is the standard word for resending messages (as opposed to _send_on_).


I will create new pull request for different purpose fixing next time.


Please use 2 spaces for indentation in JS


@dominikl: we never really use the keywords. This will be removed from the DB at some point.
That can be cleaned up


Okay, I committed this to master (942e1e37b627d4690e05ec0ebb6e08bf0a89f67f) and REL1_2_STABLE.

Also I found the original commit  8a05756 was applied to REL1_1_STABLE, but because of the difference of the original code, this patch cannot be applied simply.   This can be a bit harmful so I postponed to apply this to REL1_1_STABLE branch.


This change results in the default Cordova icon and splash image files beeing left inside the "res/drawable" directory.
Maybe Android does not actually use them, but since the "cordova platform" command creates the "res/drawable" directory, shouldn't the specified icon and splash image files be copied into it anyway?!?


you miss the type "EXPORT"


With using current ObjectFactory I reduced the test method only by 2-3 rows.
Could I write additional ObjectFactory' method or use only existing?


Yes, that's better. Fixed in [4b4a16b3309d35b32](https://github.com/odoo-dev/odoo/commit/4b4a16b3309d35b32fc55bf7d1726cb990c92bdb).


@mrehanali If this code is not using anymore, then please remove this


|  Empty | Items Present
---|-----|------
Disallow new identifiers
 (no builder) | Nothing else should be typed here (e.g. an import clause that exhausts all of its module's exports). | Standard completion on all scoped items.
Allow new identifiers
 (builder present) | This is solely an identifier definition location. | We're not sure what you're typing! :smile:


Shouldn't that then be reflected in all the other poms? Else I get the error:

    ren@~/.../dkpro-core$ mvn clean install
    [INFO] Scanning for projects...
    [ERROR] The build could not read 2 projects -> [Help 1]
    [ERROR]
    [ERROR]   The project de.tudarmstadt.ukp.dkpro.core:de.tudarmstadt.ukp.dkpro.core-asl:1.8.0-SNAPSHOT (/Volumes/HDD2/ren_data/dev_hdd/uima/dkpro/dkpro-core/de.tudarmstadt.ukp.dkpro.core-asl/pom.xml) has 1 error
    [ERROR]     Non-resolvable parent POM: Could not find artifact de.tudarmstadt.ukp.dkpro.core:de.tudarmstadt.ukp.dkpro.core:pom:1.8.0-SNAPSHOT and 'parent.relativePath' points at wrong local POM @ line 21, column 10 -> [Help 2]
    [ERROR]
    [ERROR]   The project de.tudarmstadt.ukp.dkpro.core:de.tudarmstadt.ukp.dkpro.core-gpl:1.8.0-SNAPSHOT (/Volumes/HDD2/ren_data/dev_hdd/uima/dkpro/dkpro-core/de.tudarmstadt.ukp.dkpro.core-gpl/pom.xml) has 1 error
    [ERROR]     Non-resolvable parent POM: Could not find artifact de.tudarmstadt.ukp.dkpro.core:de.tudarmstadt.ukp.dkpro.core:pom:1.8.0-SNAPSHOT and 'parent.relativePath' points at wrong local POM @ line 23, column 10 -> [Help 2]
    [ERROR]


No test, not red, no breakfast :p


Maybe so, I'm not sure. Anyway it's not the case of this commit. Here I've just found that script_retcode() creates shell, checks for deprecated usage and calls script() which does the same: creates shell and checks for deprecation. So it creates shell twice and will print deprecation warning twice if __env__ passed. I've just removed redundant code.


this will give you an error if clim is a numpy array. use ``if clim is not None`` instead


good catch @poolio


@mrehanali Logout label should come from translation function


i had to checkout the hotness and instead found a :bug:


that's a good idea, filing a bug


also we forgot to include `clim` as a argument to the function

silly silly


see https://github.com/coala-analyzer/coala/issues/755


Oh man, this is embarassing. Sorry.


move this to some function. or explicitly return in the "if"s


This is a trial project for evaluating bluemix


Should take `const Data *data` - we don't modify the data and we don't need to keep a reference to it after the function has returned.


Add HelloWorld


I would like to not use Models in database service,

when I call database service I want to deal with database objects only, because only this layer is known for me


Good spot :+1:


how much slower is this than the blas_ger_fnc you were using before?


as message from above, it's prefered to map model to entity and then send entity to db service


This won't compile, `addr` is of type `sign::PublicKey`.


Should take `Data *data` - we don't need to hold a reference to the data after the function returns. Perhaps the argument types are motivated by issues with the interface for `despatchTypedData()`? If that's the case I'd rather use casts behind the scenes so that the API is right here, and then fix up `despatchTypedData()` in the future.


You should not write code inside a Merge commit.
Merge, resolve conflicts and then create a new commit for new features.


close #51, no #44


oh ya I forgot what the improvement was with calling `blas_ger_fnc` ... it must not have been big otherwise I would have remembered ... I should dig up that code again.  (@poolio I remember you had a gist with this I think?)

probably doesn't hurt to use the blas op


:thumbsup:


All screens in your app that are not the main entrance to your app (activities that are not the "home" screen) should offer the user a way to navigate to the logical parent screen in the app's hierarchy by pressing the Up button in the action bar.

you do not need to handle the Up button's event.


Caused: https://issues.jenkins-ci.org/browse/JENKINS-29376


Hi,
what is the output of
- uname -a
- cat /etc/os-release
- cat /etc/apt/sources.list


`IMappingService MapService` it would be better when it looks:
`IMappingService mappingService`
Mapservice - is it about maps? geo location? cities?


libarchive is keg-only, by default Homebrew will not link files to /usr/local/include and /usr/local/lib.


and starts with lower letter


![Uploading 图片1.png…]()
![Uploading 图片2.png…]()
![Uploading 图片3.png…]()


great job!


@xsfour 可以测试的，本来要at达达，结果发现他还没有接受邀请.... 简直日了达了。


Je
te
DETESTE
:(

Mais plus sérieusement, s'ils passent pendant qu'on est à Helsinki, ça te
dirait d'aller les voir ?
Ma copine est pas métaleuse du tout et j'ai pas envie d'aller à des
concerts toute seule :)

2015-07-13 9:29 GMT+02:00 ArmandDu <notifications@github.com>:

> je les ai vu en concert en Juin :3
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TeksInHelsinki/TeksInHelsinki.github.io/commit/ba4afc8b69f67205e27583cae0b8c89812df781d#commitcomment-12127322>
> .
>

--
Coline MARION
Epitech Paris - Promo 2017
Assistante Groupe IA @ Epitech Innovation Hub


This causes a JavaScript error if the server is unavailable.
Should probably handle errors other than 400 as well (with a general error message).


Please use `load_all?` instead of checking admin role. Admin are treated as normal user in Atmosphere REST API unless (s)he asks to load all entries (`?all=true` in query param).


Yes, I'll change that.


This should be `kbd` not `kdb`. I tend to do the same typo a lot myself :)


@Snger the 1.2.x branch is open only for big regressions and security fixes. This fix does not fit into these categories.


Thanks, I was indeed on an older GHC.


fixes https://github.com/fabric8io/fabric8/issues/4455


Thank you for the pointer. Issued a PR: https://github.com/github/linguist/pull/2498


saw approval from shino
at https://github.com/basho/riak_cs/commit/51374e9e96dd6388ac78c3b32d52f2da2f096068


merging basho/riak_cs/feature/logging-block-server-errors = 51374e9e into borshop-integration-1177-feature/logging-block-server-errors


basho/riak_cs/feature/logging-block-server-errors = 51374e9e merged ok, testing candidate = 24eba522


What do you mean broken?


@mrehanali Try to remove this code and see whether its working or not, as its violates rule for using db abstraction layer


They do not setup the data they use. Thus they don't work out of the box.
I am working on this already ;)


Please move `close(fd);` right after the for statement.


can `configurations` HashMap be replaced by ConcurrentHashMap?


Yeah, we know already https://github.com/sphereio/sphere-product-import/issues/8

/cc @panshul007

PS: I've invited you to the project we use for testing `importertest-42`. Get the credentials from there


fix issue #2


@mbernat No idea really, but who knows what can change. Better not to have a panic happening here. Or should this simply panic in case the dash is not there?


Maybe you want to get `all()` results from the query, right?


elseif


This isn't my preferred way - I am already have some code for product type + product.
Will finish my integration tests and @panshul007 may reuse this for the others.


(go fb)?

Bien sur que je t'accompagnerai ;)


I was thinking of having an enum which would be either NameType or sign::PublicKey, @mahmoud created such type in relay.rs named [IdType](https://github.com/maidsafe/routing/blob/master/src/relay.rs#L33). I haven't exported and used it yet because I don't like the name (too many Id/Identity types already) and I couldn't think of a better one.


If you can fix #8 then all the better! ;)


@mbernat Fixed in 9e46e67


I've taken the code from https://github.com/reactphp/promise/blob/34b77b1bb08f9bbd38de337c4e4b7c6e7ae34527/src/Queue/SynchronousQueue.php#L16-L36
To be honest, i'm not sure anymore why i've taken that approach. I've changed it here and over at the global-queue branch.


I like this localisation of `t`


Only the top-level POM in DKPro Core inherits from the DKPro Parent POM - the others inherit from the DKPro Core top-level POM:

    DKPro Parent POM v12
    +-- DKPro Core Parent POM v1.8.0-SNAPSHOT
        |-- DKPro Core ASL Parent POM v1.8.0-SNAPSHOT
            |-- ASL modules...
        +-- DKPro Core GPL Parent POM v1.8.0-SNAPSHOT
            |-- GPL modules...

The problem you report shouldn't happen. We tell Maven explicitly where the DKPro Core ASL/GPL Parent POMs can find the DKPro Core Parent POM using the "relativePath" setting:

	<parent>
		<groupId>de.tudarmstadt.ukp.dkpro.core</groupId>
		<artifactId>de.tudarmstadt.ukp.dkpro.core</artifactId>
		<version>1.8.0-SNAPSHOT</version>
		<relativePath>..</relativePath>
	</parent>
	<artifactId>de.tudarmstadt.ukp.dkpro.core-asl</artifactId>
	<packaging>pom</packaging>
	<name>DKPro Core ASL</name>


Removing the `ignore` fn will cause compilation warnings I believe (it is the only purpose of the `ignore` function).


the navbar collapse worked after installing nodejs from https://nodejs.org/download/ and using the word application instead of using the word default,


this is how I was able to build, have a look


To elaborate, Rust doesn't like when Result<_, _> is implicitly ignored.


I think you mean `pip install coala` ?


Thanks a lot for trying the pre-release. :smile: I really appreciate that. Please submit an issue for that. I must admit, that I copied code from another project and thought that `win32/file` is part of ruby core.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1335