oh crap, i only updated the merge test. sorry


Why do you need to send the Exception back, I don't think we do this kind of in JMS.


If I specify NoWait and Follow... What happens?


hashWeek はstatic変数として、メソッドの外で定義しましょう！


The `process` checks whether the `actual_get/post/put/delete` is None, so I can write one line of code instead of three in other functions. You can take it as a macro in C/C++.

All the get/post/put/delete methods provided in AbstractXXXBackend just check whether the requested/provided MIME types stick to the rfc, so other people developing a new backend don't have to do the same thing. Instead they can just call `retrieve_data` and put the data in the response.


I still see favicon as the paw print in FireFox.


You need add the ASL header on the new created Java file.


今日のように、１日毎に終わっていなくても、commit & pushのほどお願いします。
git commit -m "xxxx"　で記載するコメントについては、何を実装したのか詳しく記載してください。

日本語が使えなかった教えてください。


Nice! Lots of excellent ideas here.

The optionally instantiating the object thing is a great idea! Maybe the `require $class` should also be part of that? That way, you get the "container looks okay" version even if not all the requisite classes and such are installed, and the "all the objects inside the container can be instantiated" version with just a flip of a `-i` flag.

The `-a` flag looks like it's not being used, because it's the default?

Could you add a line or two of docs above the `validate` method?

In the script, you probably want the check for `$help` before the check for the config file argument, so `beam-wire -h` works without giving an error.

I love the `-l` flag too. That'll be incredibly useful.

Altogether, great stuff. Thanks!


형변환 선언할곳이 없는데?..


thx


@csfrancis this was a conflict when merging GHs 2_1_6_github1 seems related to https://github.com/Shopify/ruby/commit/7aa24f544fb5e896e47be2111e8d49bc63a75dca do you remember why this file changed?


Hmm.. Should this be `unshift @INC` to put `lib` first? Or do we care about the distinction enough? Would this be used as a sanity check in a Jenkins/Travis job maybe?


H


Wow Kalman, that update is pretty cool... it looks like you put a lot of hard work into it, like 3 days worth of effort....


yyyaaayyyyy <3


Thanks!


Crate local? You can check that with `ast_util::is_local(def_id)`.


협업 사용자는 코멘트를 답니다.


There is a trade off between readable/maintainable code and compact code, but I'm okay with it if you insist here since refactoring is also very annoying and time consuming.


93441220-LowTempRadiantWarning-4237 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1647 of 1647 tests passed)

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


Certainly there can be a `database backend` as well, the `static file end` is just a simpler approach.

Loading file to the odl datastore can be done in no time with the adapter you provide, it doesn't even need a server.


그건그렇다치고 이번주 토요일에 동적할당들을 대부분 손볼거임.
전부 allocater(gc)같이 내가 따로 알고리즘을 짤 예정임 STL같은경우도 따로 allocater가있음.
이게 문제가 내프로그렘같은경우엔 delete나 new을 많이 해야되는데 굉장히 비효율적임.
모든 동적할당은 이번주 토,일 요일에 만들 allocater에 모두 종속되게 만들거임.
그리고 nm 프로젝트가 커짐에 따라 좀 더 잘 나눠야겠음 한 파일에 담기가힘듬
nm_block같은경우에도 이제 인터프리터도 만들거라서 3개정도로 나눠야하고.
exception handler같은경우에는 말할것도없고.
factory인 매인 프레임워크는 좀더 세분화할필요가있음.
리스트, 큐 두가지 다 만들거임. 리스트 큐 같은경우에는 이 프레임워크에서 쓸 일도있겟지만
나같은경우에는 새로운 언어를 만드는 쪽으로 갈 것이기 때문에 가상으로 내부적으로 사용할일도 생길거임.

결론- 좀더 세부적으로. 이제는 진짜 가상머신처럼 만들생각.
준비는 끝났어!! 할시간이야!!


I think this is wrong step, as sun* properties are only supported in Oracle JVM, and not in OpenJDK. In my opinion, **os.arch** is the right one, which you had before.


Primeira linha


This line supposedly fixes #1637


Ah, poo. I should have been more specific. I only fixed it for the table view. Working on the grid view now!


[It works!](https://kanban-boards.herokuapp.com)


Primeita linha.txt


@jkeck Added tests to new helper methods


> Hm, isn't this kind of an odd way to go about this? I would think you should be able to control whether Staff badges get a printed name via a config option.

This is part of the discussion that I plan to kick off this weekend when I finally have a chance to document some of this stuff and summarize my preferred approach to customization.

At a high level, we can choose between two paths:

1) Everything that's built into Uber is customizable via config.  So if we have 15 form fields on our default prereg form, then it means we'll have ~15 different config options like ``c.ENABLE_COMMENTS`` and ``c.ENABLE_INTERESTS`` etc.

2) Have almost no direct configuration, and just let plugins dynamically insert and remove form fields as desired.  Provide an API on the backend for fiddling with database columns, validations, etc.

Between those two options, I greatly prefer the second one, since it results in less code and cleaner code plus it's generally more flexible.  (I've been keeping a list of all of the customizations that Anthrocon has made to ensure that we'll be able to support all of them with this new approach, of course.)

Obviously there's some middle ground, since we could make some things customizable but not others.  If there are some common things that we know for a fact organizations will want to do differently, then it probably makes sense to support all of them out of the box, so to speak.  (Exact birthdate vs age groups is a good example, which is currently still customizable.)

> Plus, won't just removing the field cause model_checks.py to complain?

Not in this case, because that field is optional.  Though there is an API now for turning off individual model validations specifically in case a plugin ever needs to do that :)


Shouldn't the message be handled in the `detailschange` above?


这个问题不大，
barCategoryGap:'0%' 就好了。

我再想一下bar 和 line plot那里，发现echarts可以有个magic参数直接切换，先暂时这样...以后可能要改。


Strongly =dt here


OKですね。
ただ一つ指摘事項としては、
1-7以外の数字を渡したら、""(空文字）が返ってくるように修正をお願いします。
これだと、"0"と"8"だけしか、""（空文字）が返ってきません。


I can't merge this.  This breaks the ability to reload things on the fly.
Most notably, plugins are read into a byte array and loaded from that on purpose - it allows them to be unloaded without creating and loading them into a new AppDomain.  If you load them directly from the file, as in these edits, the file remains locked and you can never upgrade a plugin while the server is running.
It'd be easier on me if you reverted those particular changes and push again. That should automatically update the pull request.


Why is this happening here instead of the canonical source, editor-imagery-index?


@Morphan1 pls 2 check your own commits for extra crap


바뀐거 없구나.


Boa fera!
agora começa a upar os arquivos de hmtl que vc msm faz no codecademy, pra praticar mesmo, com um código que ja seja útil!


That's true, but my point is that the "static file end" approach doesn't worth much efforts and we should not dig too deep in this implementation.


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


我的文章


Gg


looks good


あのですねっ


Might reuse this function if it makes sense


はひっ


But it seems your atomic insn, do uncached access while these will go thru cache. What if you have both atomic_t and atomic64_t in same cache line ?


Nice !


qqq


qqqq


Could we just replace the "**" with "**" + filepath.Separator + "*" here and have this case work?


s/Glob/GlobWithExcludes/


I would strongly recomned for using of c++11 style initialization, as stated e.g. [here](http://herbsutter.com/elements-of-modern-c-style/):
> In other cases, including especially everywhere that you would have used ( ) parentheses when constructing an object, prefer using { } braces instead. Using braces avoids several potential problems: you can’t accidentally get narrowing conversions (e.g., float to int), you won’t occasionally accidentally have uninitialized POD member variables or arrays, and you’ll avoid the occasional C++98 surprise that your code compiles but actually declares a function rather than a variable because of a declaration ambiguity in C++’s

Why don't you like the modern style?


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


Thanks, I feel a lot better :stuck_out_tongue:


@getaaron this line should have been commented out.


LXDE-Qt and razor-qt.org ?


same here...


why did you remove the cleanup?


Let's keep `initialize` simple and set these elements and the callbacks (lines 49-50) within `connect` method instead.


Thanks for the feedback.
I will implement it next week then.
By the way, would you allow to have several widgets at once in the bar?
I mean could a plug-in display a widget at a same than a map tool?


Why did you remove this line ?


Found another problem. In train.lua within the for-loop of the forward pass, this line:

embeddings[t] = clones.embed[t]:forward(x[{{}, t}])

throws an error:

/home/mark/torch/install/bin/luajit: bad argument #2 to '?' (out of range)
stack traceback:
	[C]: at 0xb7341ea0
	[C]: in function '__index'
	./Embedding.lua:29: in function 'forward'
	train.lua:91: in function 'opfunc'
	/home/mark/torch/install/share/lua/5.1/optim/adagrad.lua:30: in function 'adagrad'
	train.lua:141: in main chunk
	[C]: in function 'dofile'
	...mark/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:131: in main chunk
	[C]: at 0x0804d650

Looks like it's related to how the embedding got set up earlier:

protos.embed = Embedding(vocab_size, opt.rnn_size)

and the tensor being forwarded through clones.embed
x[{{}, t}]

Sizes don't seem right for Embedding:updateOutput?????


It is an extra space


好，我等的就是`barCategoryGap:'0%'`这句话。那你可以把直方图完善一下了。主要应该就是加个data_hist()计算一下数据在区间上的频数。

随着图形种类的增多，代码抽象的难度肯定也会越来越大。我已经尽量把散点图、条形图、折线图的数据生成函数抽象化了，这个直方图肯定可以用条形图的data_bar()函数，但可能存在其它不太好抽象的地方。你先试试吧。


Is this needed? If so, it should be initialized in `__init__` to make it a class variable.


This will get written in plain text in logs, which may need to be distributed/published. Remove the password and change the logging level to debug.


Just added a commit to address your comments and another one to pep8'ify everything:

https://github.com/zwhite/twilio-python/commit/8c5b81644809a837a7b12bf306899a0be1c76552

https://github.com/zwhite/twilio-python/commit/deb5aff125458487b630e589e8d700d443926b7d


Sorry for that... i was misinterpreting the Plugins class hierarchy :(


Range-checking every single access to MappedRAM isn't particularly ideal from a performance perspective. Do you know which part of the emulator was generating illegal accesses and necessitating this change?

Empty/unallocated MappedRAM objects should never be getting mapped on a bus (there's an explicit check for it in Bus::map()), and MappedRAM gets padded to a multiple of 256 bytes (the size of a bus page) when allocated, so access via a bus should never ever dereference a null pointer or go out of bounds. If I'm not mistaken that only leaves the debugger hexeditor view as a possibility... and in baseline bsnes-classic that's also range-checked (displaying a partial line of hex display if necessary), so if you've somehow broken that range checking, you should fix it there rather than in the performance-critical memory code.


还有，我不知道你对git有多熟悉，以下两个东西希望你能常用：

```bash
git checkout master
git pull --rebase https://github.com/yihui/recharts.git master
```

这是让你的master分支和我的同步，`--rebase`是为了避免在pull之后的合并时产生一个合并commit。

然后是我希望你能用单独的分支开发单独的功能，而不是一直在master分支上工作：

```
git checkout -b feature/histogram
```

这是因为有时候我可能需要你做一些需要篡改历史的暴力git命令，这种命令最好不要在master分支上进行。再一个也是因为这样你可以平行开发不同的新功能，每个功能用自己单独的分支，不用把所有东西都堆在master里。

再往后你可能还需要用到`git rebase`以及解决冲突，等到真正要用的时候我再告诉你。我平时常用的git命令也就三五个，没那么复杂。


revert.


Гифку запили в README


Would you like to send a PR to fix it?


You can make it cleaner by (as mentioned in comment for `initialize`) initializing the variables and callbacks outside the if condition and only initializing the client within the if condition.


`Private message` is not use anymore. Instead of plain `message` if I get it right.


Oh my goodness @pkhorrami4. Inline comments! With markdown! This is **crazy**.


I would rather not have the returnCode parameter. See comment below as well


One problem is that {{DEFAULT_TARGET_DELIM}} is quoted and should not be.


Thank you for pointing this out.

The error was in the `tcp_adaptor::resolve_host` function and also the `ssl_tcp_adaptor` version of the same function. It was calling the version of resolve that throws an exception:

    return resolver.resolve(query);

Resulting in the following error:

    HttpClient.exe host: www.boost.org method: GET uri: /LICENSE_1_0.txt
    Exception:resolve: No such host is known

It has been changed to call the version that takes an error code:

    boost::system::error_code ignoredEc;
    return resolver.resolve(query, ignoredEc);

 It now simply returns false when there is no internet connection:

    HttpClient.exe host: www.boost.org method: GET uri: /LICENSE_1_0.txt
    Error, could not resolve host: www.boost.org

The change has been checked in to the develop branch.


Hello, gimmeitorilltell. I was trying to build cm12.1 with your device and kernel sources but I always get 'no rule to make cm12.1/out/target/product/quincyatt/obj/SHARED_LIBRARIES/libqdutils_intermediates/export_includes''
this error. What should I do for it? I think GPU and CAF flags which libqdutils(qcom display library) requires are missing.


ok


@svmehta I removed the key from the pom.xml, can we revoke this key?


This is another problem that merits more discussion.  The previous behavior simply returned the naked value.  The new behavior stuffs the value in a ``dict`` with the key being the grain path.  This has two problems:

    1. The key/value dictionary will break any previously existing tool that relies on receiving a naked value
    2. If the returned data is to be structured inside of a dictionary then the key should not merely be the string of the grain path, but should actually be structured data as well.

    {
        'foo:bar' : 'baz'
    }

vs.

    {
        'foo' : {
            'bar' : 'baz'
        }
    }


Is "/tmp" a better choice?


These are minor structural changes for MVP.


I just double checked the behavior of JMS and it only does this if transferException=true is set on the endpointUri. I will add that option make it work the same.


This makes it return the returnCode if specified and the mqtt Client object otherwise. Not a good idea to return 2 totally different types based on certain condition. Probably better to return the object and use the rc within the method to validate that it connected without errors. If not, raise error.


I'm not sure what you mean here


I have absolutely no idea what I'm doing


Thanks for pointing this out. It is actually a very important point that is badly explained in too many places, and my explanation attempt here is probably not the best either.

If I understand it right, any **Angular expression** is allowed here, not only a callback function. Even non-assignable expressions are possible. For instance, you could do:
```
typeahead-input-formatter = "$model.hours * 3600 + $model.minutes * 60 + $model.seconds"
```
and you get back the calculated total seconds value instead of having to parse your `$model` (and having to write tests for your parser :)

This might be useful to save some extra code in your controller, and have the advantage of better visibility inside your template.

Of course, there is a downside - this might encourage folks to abuse the power of Angular expression in the Views, which you can only test with e2e.

I have tried to bring the possibility of any Angular expression here, especially since there is no example with this option in the Demo. How about this try:

> the typeahead-input-formatter is an Angular expression that receives the model's value through an optional $model parameter


I thing it should be `` `id`=187943;``


Probably not, but I think the class should already have been initialised at
this point, and I think I did it to be consistent with the old way (which I
think calls Class.forName(String) at some point and thus does initialise).

Anyway, I'll change it.

On 29 April 2015 at 06:57, Tibor Digana <notifications@github.com> wrote:

> Is it necessary to force loading static class context - second param with
> true ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/seanf/maven-surefire/commit/7552ea926a2df2869aead38a52f4bad49acaea8c#commitcomment-10951204>
> .
>


Alright I submitted the homework.
Good job, it was nice working with you.

Ruben


@bloodbare FYI.

I turned NavigationTile into a transient tile

https://github.com/plone/plone.app.standardtiles/commit/c719acbbe281330e8919bb925bf767474a4aa674

and faced a few WTF moments with Mosaic:

1. our overlays seem to reuse the same DOM elements and I had to registered events manually to avoid wrong events being fired (otherwise edit did fire also add callback)

2. since transient tile URL includes ZPublisher-marshalled query string (with type annotation) it cannot be turned into tiledata in a such naive manner as the code did, and I reverted back to just calling the tile edit view with the legacy query string (well, it works)

3. overlays steal focus, which prevented edit form updating the tile (needed to add special class for the tile being ediged)

4. at least "&" must be entity-encoded before adding transien tile url with parameters into DOM; otherwise the parameters may break in many ways (like &current turning into some weird character)

With these commits I'm now able to add and edit transient NavigationTile successfully.


Sure, but there's nothing to try yet?


Thanks, you did a good job also.
On Apr 29, 2015 10:02 PM, "Ruben Niculcea" <notifications@github.com> wrote:

> Alright I submitted the homework.
> Good job, it was nice working with you.
>
> Ruben
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RubenSandwich/CS420HW2/commit/9d17c4ee66e00bd42d84120acc3bd5a08365b4ac#commitcomment-10977874>
> .
>


IMHO I was right and the sanity here should remain as in `QAbstractItemView::setModel` the `selectionModel` is reset, see [here](
https://qt.gitorious.org/qt/qt/source/c2d9d7cdcaa59486864502ae54189ba64f5ef8e1:src/gui/itemviews/qabstractitemview.cpp#L683)

...and for correctness I think these objects should be deleted after the `set*` as it was before


:dancers:


We should not have this variable in Event object.Please remove


You don’t need to modify `sys.path`, it should work right away. This is only needed if you want to support some older *Vim* versions, and in this case I think code should be modified to be run on `if v:version < 704` condition.


I looked at this one carefully again , and I found it misses ```else``` before ```MVM_free(buffer);```, because if the buffer is added to decodestream, it can't be freed. And same as in latin1.c, thanks.


I don't actually remember the reason for this change; I think it was done preemptively for something I was doing at the time but never actually committed. I'll revert it later and make sure nothing is affected (which is likely since I haven't made any relevant changes to he hex editor or anything else).


to ligado.. mas aí sobe um browser só == anos pra rodar a suite toda...


Indeed ;-)

- Ending with period makes it clear, the explanation ends rather than being truncated. Also consistent with other explanations ending with `?`.

- The problem with "value" is that - there are two of them! One _value_ is used inside the **comprehension expression**. And then you have the `input` value, which is different! So I've tried to use bold to avoid the confusion. But maybe it is still confusing, and
 it would even be more clear to call it something else inside the comprehension expression - maybe _item_ as it is often called in such examples?


@mikejolley I also encountered this loop of death today... :(


A couple of additional problems:
1. I don't think it plays nice with the sonar code because the target_sonar_alt is updated from the pilot provided target-climb-rate but the take-off code directly updates the position controller altitude target thus by-passing the target-sonar-alt.
2. I think directly updating the position-controller target bypasses the jerk limits that are implemented in the set_alt_target_from_climb_rate method.
To fix these problems, I wonder if we could make the new take-off code calculate a climb rate instead and then combine that in with the pilot's throttle?


hmm, I found ```windows1252.c``` and ```utf8.c``` also miss ```else MVM_free(buffer);```, it would be nice you will add them together :P


oh cool. I see your using the colorpicker in appointment status lists.


I'd remove the demo script though.


oh I see... we should add something for that


@vijay100040
By default it will be long value in event itself. no need to cast as string & back to long type again


93441220-LowTempRadiantWarning-4237 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1642 of 1642 tests passed)

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


Thank you.  I get it.


Didn't need to add


rax retry tests


Sure!
On Apr 30, 2015 12:44 AM, "Peter Dave Hello" <notifications@github.com>
wrote:

> Would you like to send a PR to fix it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cdnjs/new-website/commit/9f09e152fa91af5e16cca760cb4b312b231b5991#commitcomment-10977763>
> .
>


When the exception is raised. So, the user know what to do to fix the issue.
An other possibility is to have a documentation page about how to install the optional dependency, and to give the link when the exception is raised. This latter option may be simpler, and ultimately more useful.
Le 30 avr. 2015 12:23 AM, Hub <notifications@github.com> a écrit :It will be quite easy to add it but when this string will be print to the user? With the --help option or when the exception argparse.ArgumentError is raised?

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


Do these need translating?


This is great.... but as the `addPluginDialog` is destructed in static deinitialization time... the destructor segfaults:
```
(gdb) bt
#0  0x00007fe98513f660 in QWidget::isActiveWindow() const () from /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5
#1  0x00007fe98530e0b4 in QDialog::setVisible(bool) () from /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5
#2  0x00007fe98530be31 in QDialog::~QDialog() () from /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5
#3  0x0000000000469aa8 in AddPluginDialog::~AddPluginDialog() ()
#4  0x00007fe980585bc9 in __run_exit_handlers (status=0, listp=0x7fe9808f15a8 <__exit_funcs>, run_list_atexit=run_list_atexit@entry=true) at exit.c:82
#5  0x00007fe980585c15 in __GI_exit (status=<optimized out>) at exit.c:104
#6  0x00007fe98056fb4c in __libc_start_main (main=0x444f60 <main>, argc=3, argv=0x7ffe6797ccc8, init=<optimized out>, fini=<optimized out>, rtld_fini=<optimized out>,
    stack_end=0x7ffe6797ccb8) at libc-start.c:321
#7  0x0000000000444e64 in _start ()
```


Add


Oops, my bad, good spot!


I'm impressed that you handled this! And that you knew that it was a good unit test.


The file.version+2 was added to see whether the server sends only one file version. The file version is not 2 versions behind.
Removing the +2 and sending only the file.version would still pass the test. We have removed the +2 and committed the code


I've just committed a fix that allows assert()s to actually get compiled in, so in future you can use them instead of this kind of safety check (which is effectively painting over bugs instead of fixing them)


Its not that easy.  a/** should match a/, because ** matches 0 or more path elements, but a/**/* will not match a/.


I couldn't find a more XPath-ier way to grab the `td` elements out of each `tr` element. I used the index of the elements under `tr` instead, where `tr[1]` refers to the date and `tr[2]` refers to the value. Let me know if you think there's a better way. The HTML I'm trying to parse is pasted below.

```html
<tr>
    <td class="first">Body Fat</td>
    <td class="col-num">4/29/2015</td>
    <td class="col-num">19.1</td>
    <td>
        <a href="/measurements/delete/170628596" class="remove" data-confirm="Are you sure you want to delete this entry?">Delete?</a>
    </td>
</tr>
```


沃尔沃而


You want this:

```ruby
params(:summary)
```

This is the parameter that will contain all the values needed to build our summary.


You actually don't need this since @post.build_summary creates the association for you. This method is given to you by the has_one relationship, and is similar to the @topic.posts.build method we used earlier in the check point with the has_many relationship.


The log is here: https://travis-ci.org/btb/dxx-rebirth/jobs/60503604

```
In file included from common/arch/sdl/digi_mixer_music.cpp:22:
In file included from common/include/hmp.h:16:
common/include/physfsx.h:161:47: error: unused parameter 'file'
      [-Werror,-Wunused-parameter]
PHYSFS_sint64 PHYSFSX_check_read(PHYSFS_file *file, exact_type<V> v, PHY...
                                              ^
common/include/physfsx.h:161:67: error: unused parameter 'v'
      [-Werror,-Wunused-parameter]
PHYSFS_sint64 PHYSFSX_check_read(PHYSFS_file *file, exact_type<V> v, PHY...
                                                                  ^
common/include/physfsx.h:161:84: error: unused parameter 'S'
      [-Werror,-Wunused-parameter]
  ...*file, exact_type<V> v, PHYSFS_uint32 S, PHYSFS_uint32 C) DXX_CXX11_EXPL...
                                           ^
common/include/physfsx.h:161:101: error: unused parameter 'C'
      [-Werror,-Wunused-parameter]
  ...*file, exact_type<V> v, PHYSFS_uint32 S, PHYSFS_uint32 C) DXX_CXX11_EXPL...
                                                            ^
common/include/physfsx.h:163:48: error: unused parameter 'file'
      [-Werror,-Wunused-parameter]
PHYSFS_sint64 PHYSFSX_check_write(PHYSFS_file *file, exact_type<V> v, PH...
                                               ^
common/include/physfsx.h:163:68: error: unused parameter 'v'
      [-Werror,-Wunused-parameter]
PHYSFS_sint64 PHYSFSX_check_write(PHYSFS_file *file, exact_type<V> v, PH...
                                                                   ^
common/include/physfsx.h:163:85: error: unused parameter 'S'
      [-Werror,-Wunused-parameter]
  ...*file, exact_type<V> v, PHYSFS_uint32 S, PHYSFS_uint32 C) DXX_CXX11_EXPL...
                                           ^
common/include/physfsx.h:163:102: error: unused parameter 'C'
      [-Werror,-Wunused-parameter]
  ...*file, exact_type<V> v, PHYSFS_uint32 S, PHYSFS_uint32 C) DXX_CXX11_EXPL...
                                                            ^
common/include/physfsx.h:166:47: error: unused parameter 'file'
      [-Werror,-Wunused-parameter]
PHYSFS_sint64 PHYSFSX_check_read(PHYSFS_file *file, V **v, PHYSFS_uint3...
                                              ^
common/include/physfsx.h:166:57: error: unused parameter 'v'
      [-Werror,-Wunused-parameter]
PHYSFS_sint64 PHYSFSX_check_read(PHYSFS_file *file, V **v, PHYSFS_uint3...
                                                        ^
common/include/physfsx.h:166:74: error: unused parameter 'S'
      [-Werror,-Wunused-parameter]
  ...PHYSFSX_check_read(PHYSFS_file *file, V **v, PHYSFS_uint32 S, PHYSFS_uin...
                                                                ^
common/include/physfsx.h:166:91: error: unused parameter 'C'
      [-Werror,-Wunused-parameter]
  ...*file, V **v, PHYSFS_uint32 S, PHYSFS_uint32 C) DXX_CXX11_EXPLICIT_DELETE;
                                                  ^
common/include/physfsx.h:168:48: error: unused parameter 'file'
      [-Werror,-Wunused-parameter]
PHYSFS_sint64 PHYSFSX_check_write(PHYSFS_file *file, V **v, PHYSFS_uint3...
                                               ^
common/include/physfsx.h:168:58: error: unused parameter 'v'
      [-Werror,-Wunused-parameter]
PHYSFS_sint64 PHYSFSX_check_write(PHYSFS_file *file, V **v, PHYSFS_uint3...
                                                         ^
common/include/physfsx.h:168:75: error: unused parameter 'S'
      [-Werror,-Wunused-parameter]
  ...PHYSFSX_check_write(PHYSFS_file *file, V **v, PHYSFS_uint32 S, PHYSFS_ui...
                                                                 ^
common/include/physfsx.h:168:92: error: unused parameter 'C'
      [-Werror,-Wunused-parameter]
  ...*file, V **v, PHYSFS_uint32 S, PHYSFS_uint32 C) DXX_CXX11_EXPLICIT_DELETE;
                                                  ^
```


This hooks calls `qubes-update-check.service` which performs an `apt-get update` to see if any updates are available.

It's primary purpose it to notify dom0 after an `apt-get [dist-]upgrade` is complete to clean the `updates available` flag in `qubes-manager`.

Although this hook works and seems to normally be called when the `upgrade` is complete I have noticed it also may be getting called after each `trigger` event.  In the case where `qubes-core-agents`'s triggers are initiated on initial installation, the post hook is being called many times I believe, each time running a time consuming `apt-get update`.

Possible solutions; looking for input:
- Don't call `apt-get update` at all after `upgrade`; assume clean; just send dom0 status to clear the flag.  As the post hooks does not get called too much I would not think it would effect performance.
- On the first call, touch a file `/tmp/apt-get-post-hook` (so file gets removed on reboot) then on next call, condition to `apt-get update` will fail since that file exists.  Problem with this scenario is that maybe the first call to `apt-get update` will still show updates exist so dom0 would not clear flag in that case.
- combine option 1 and 2 above; notify once only; no `apt-get update`


Just use `"jasmine": "karma start test/config.js --single-run",`; npm already looks in the `npm_modules` folder.


Could you give an example of how this can be exploited? The combination of withCwd and pathModule.join should already prevent this from being possible. `pathServer` will always be a string beginning with '/', so that when the root dir is joined to `pathServer`, the result must either be the root dir itself or a subdirectory of it. There might be a flaw in this logic somewhere but it's not immediately obvious.


update


Hi, this "if" will update facility description only if also name has changed. Please make it like
```java
if ((!dbFacility.getName().equals(facility.getName())) || (!dbFacility.getDescription().equals(facility.getDescription())))
```
so it can work with your modified sql update.


Hi


10/10 Great Job!


You accidentally removed the trailing newline in this file.

(I'd say your text editor is buggy: trailing newlines are required on Unix.)


In my opinion, we should commit this because release tag should clearly point the released revision.


It would have been better if the throttle behaviour stuff was in a separate commit from the take-off.


This file also lacks a trailing newline.  (GitHub helpfully highlights those, and so does `git diff`.)


안전한곳 추천
핸디1.90
사다리 로하리 1.97
달팽이2.80
첫충 10% 첫낙10%
크로스,단폴 가능
vipss77.com 코드:ss77


i see them all in my email bc im subscribed to repo

wtf, why is other jennifer sun seeing thinggs!?!

On Wed, Apr 29, 2015 at 11:38 PM, Jennifer Sun <notifications@github.com>
wrote:

> hey other jennifer sun, are you seeing this message?
>
> @jayzach99 <https://github.com/jayzach99> do you only see messages if
> you're tagged?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sunnifer/learningtogether/commit/4a7c0c8c13713cb5a497804c2c9dfe09b609de7e#commitcomment-10977344>
> .
>


@NitroXenon Oops, Kha'zix Exploit still works? in my case, it seems patched


> Shouldn't this be nprocs?

Could be done also using coreutils ```nproc```, ```/sys/devices/system/cpu/online``` and  ```getconf _NPROCESSORS_ONLN```, but IMO they are all equivalent solutions. I picked the one that probably works on busybox. :smile:


There was no code submitted in github by this person.


saw approval from shino
at https://github.com/basho/riak_cs/commit/885f839c4247eb87e478a5e0a3b2cb967d2f559a


merging basho/riak_cs/feature/offline-gc-tool-squashed = 885f839c into borshop-integration-1134-feature/offline-gc-tool-squashed


basho/riak_cs/feature/offline-gc-tool-squashed = 885f839c merged ok, testing candidate = 83a7eb2b


That's easy - L1 data cache is not used in NPS since it is not coherent  :-)

Noam, you can remove the .di from the ld/st. I know why you put it there but doesn't really have any effect and it just confuses people.


Thanks for the suggestion, I'll look into it.


嗯嗯，我会去建branch....

有个东西没想清楚，我在pull request里面详述。


10/10 good job! and it will be better if you can represent a pokemon using @pokemon instead of @x lol


I don't see it used in maths.h. If you want it centralized, the logical place would probably be in pstypes.h, which contains the typedefs for all different platforms. At this point I think only platforms with cstdint are going to be supported anyway.


If this is Bad then what is good? I keep running up against a lot of errors for code like the following:

```php
						if ( ! empty( $page_for_posts_content ) ) :
							printf(
								'<div id="page-content">%1$s</div>',
								$page_for_posts_content
							);
						endif;
```

`$page_for_posts_content` contains post content (HTML). What's the best way of outputting this and avoiding these errors?


I don't think so. the usual way would be to make a helper script to handle it.


The instructions are about how to build the julia debugger. I am not sure I understand.


Speculative enquiry really, but would there be a strong reason for not making signal trapping optional?

The reason I ask is that I have some code using EventMachine and defining a signal trap handler.

For my use case, having my application defined signal handler executing is preferable to the rspec one. Ideally ruby would allow access to previously defined ones so rspec could call previously defined ones, but it seems that is not the case.


Getting this on every message I send now
```
[27923:0430/005134:INFO:CONSOLE(111)] "Uncaught TypeError: Cannot set property 'rows' of undefined", source: /home/flet/code/cleanfriends/lib/elements/composer.js (111)
```


93441220-LowTempRadiantWarning-4237 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1647 of 1648 tests passed)

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


In this revision, I've confirmed that Zabbix(HAPI) works correctly:

```log
[INFO] <HatoholArmPluginInterface.cc:123> Try to connect (server). broker: 'localhost:5672', queue: 'hatohol-arm-plugin.2'
[INFO] <HatoholArmPluginGate.cc:735> Started: plugin (2) hatohol-arm-plugin-zabbix
[INFO] <hatohol-arm-plugin-zabbix.cc:27> hatohol-arm-plugin-zabbix. ver: 15.06_dev
[INFO] <HatoholArmPluginInterface.cc:123> Try to connect (client). broker: 'localhost:5672', queue: 'hatohol-arm-plugin.2'
[INFO] <HatoholArmPluginInterface.cc:947> [Init] Drop a message: type: 3 (expect: 1).
[INFO] <HatoholArmPluginInterface.cc:921> [Init] Drop a message: size: 8.
```
![screenshot from 2015-04-30 14 54 32](https://cloud.githubusercontent.com/assets/700876/7407099/da9a79a2-ef48-11e4-9a8a-04415f693d8b.png)


fsafs


fsd


Ah, right. The commit should happen between `git flow release start` and `git flow release finish`. Let's fix the wording and do it next time.


First, GOOD JOB!
Second, Isn't calling create in the create method gonna cause infinite recursions? Also the way flash works is flash is a hash that stores the error message into flash["error"] and when error occurs it's displayed on screen, and you can find where in screen it's displayed in application.html.erb (I believe it's above the <main> tag)

8/10


No run the application! I changed the way up my game to a file on Game.yava gamePath = "I: / Program Files / Origin / Battlefield 4";
Bat file still shows me
![default](https://cloud.githubusercontent.com/assets/11767892/7407140/227d8b24-ef17-11e4-9693-3a2da453b5a3.jpg)


rkt images is on master, so soon!


There is no `->` operator in Go. `<-` is used for both send and receive (as you demonstrate), and the operations are distinguished based on whether it is used in unary or infix fashion, similarly to `&`, `*`, `-`, etc.


Damn, that's some badass regex


We usually do constraint fixes in updateConstraints. I sometimes wonder if that means we end up doing a lot for views who's constraints never change then, but I do like having them all in one place (and its really not much code...)


I prefer first variant of Michael. I think other examples are easier to read for implementors but not the reader/reviewer.


Yeah, we could move them into camel-core.


93441220-LowTempRadiantWarning-4237 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1647 of 1648 tests passed)

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


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


Really neat


The [pre-repo history is still there in README.md](https://github.com/csells/FormatDesigner/blob/cae78a79cedbe10c944d05d041dade4f34854ff3/README.md#history). I didn't feel it was necessary to maintain it in two places but if you find it's important to have in code then I can revert to this.


Yeah, we could move them into camel-core. We could do it in camel-2.16.x


Looks like this would be fine:

```php
						if ( ! empty( $page_for_posts_content ) ) :
							printf(
								'<div id="page-content">%1$s</div>',
								wp_kses_post( $page_for_posts_content )
							);
						endif;
```


we could move them into camel-core. We could do it in camel-2.16.x


Seriously that switch statement made me so mad.


No, I just had to read the sentence "Use LLDB just as you would usually." about 5 times before realizing that's all this is. "The Julia debugger" at the moment means "run LLDB." That was not at all clear from an initial reading of this.


Looks good :+1:


What if filename is like "test.proto" (w/o a dir name)?

Try to run this to see whether it works:
// dep.proto
package proto;
message Dep {
  required int i = 1;
};
// test.proto
import "dep.proto"
package proto;
message Test {
  optional Dep dep = 1;
}

protoc --go_out=. dep.proto
protoc --go_out=. test.proto


I changed it in adc6b2261687f4f83e03118f41e9b8b5c9fb788a to use branches/release_34 - I'm guessing that is the branch one is meant to use with clang 3.4?


Sorry, it's been awhile since I wrote this. What is the rationale for removal here? Why is this 'redundant'?


Do we have any lining indentation ruled for js/css/HTML?


rax retry tests


What fixing this ?


Empty interfaces (`interface{}`) are rarely found outside of libraries, and even then, only those that are reflective, such as "fmt" and "encoding/json" (in particular, these tend to be packages used for external I/O). Use of empty interfaces in applications and non-reflective libraries is often a mistake, or there is often a better way (examples to follow in later comments)


The only reason is that no one's done the work yet to make RSpec's signal handling configurable.  It's been an open issue for a while--see #1439.


Account for all **three parameters** for iteratee(func)
```(el, i, list)```

Usage: ```func(el, i, list)```


qad


@mback2k Another thing is the loop [here](https://github.com/bagder/curl/blob/926cb9f/lib/vtls/schannel.c#L408) which may need to be bumped as well since according to [msdn](https://msdn.microsoft.com/en-us/library/windows/desktop/aa379814.aspx) the emptys can have information returned in them.


Update the server and client update methods


Putting this under CONFIG_ARC_PLAT_EZNPS doesn't make sense at all to me.

Either users readb and friends is supposed to only talk to little endian HW (in which case the serial driver is the things that needs fixing) or it doesn't (and than the generic implementation needs fixing.

Which one is it?


fast-forwarding develop to borshop-integration-1134-feature/offline-gc-tool-squashed = 83a7eb2b


Hello @radialglo


What are the 'security reasons'?

If `sysctl` levels of kernel access have been granted to the container then telling `openrc` to avoid using them isn't going to improve the situation.

As a completely separate and less important consideration, on the other hand if a containerization-aware kernel subsystem (like networking with namespaces, I guess) exports `sysctl` options (now or in future) then setting these options may have a legitimate use (eg. to set IP stack related options for the network namespace owned by the container, but not affecting the host). I have absolutely no idea if that's the way `sysctl` is set up, but it does strike me as plausible.


I also prefer the slash. :+!:


this is `listToMaybe` in `base`.


Account for all three parameters for iteratee(func)
```(el, i, list)```

@rarafon use ```[]``` instead of ```.```. Need to account for keys that contain spaces.
```
var o = {"weird key": 5};
    o["weird key"]; // 5
```


Good idea. :+!:


nodejs for beginners, deploy your blog to heroku, how to node   http://j.mp/1OTO0qv

perfect css hexagons http://j.mp/1DiyQV5

anatomy of a javascript framework http://j.mp/1IOyRou

the absolute beginners guide to node.js    http://j.mp/1GmtPiQ

best practices for building angular.js http://j.mp/1aSHUJt


Woah, hang on what's going on here. You have set an absolute path to `/var/lib/lxc/${NAME}/rootfs`. That's not something we have precedent for?


Update these functions to account for all case
// 1, 2 and 3 parameters, objects, array


`BRIDHE` typo should be `BRIDGE` presumably.


OK, absolute path weirdness resolved. :+1:


@bparees why were those triggers removed?


Beautiful, well done :-)


Try https://hasjob.co/?l=anywhere as what @jace mentioned does not appear to work for me.


Sorted :+1:


Not sure where you went here, but nice to know we weren't just missing slashes! :+1:


I wonder if we could do better here rather than having two separate implementations, one here and one in `customPackageSets` that runs at compile time just so we don't have to update 2 places if the lookup method changes. Then again I can't think of a way straight away either and it seems like it might be more effort in the end.


Ok, saw that you refactored ICTs


Need update the input parameter


@stoeffel Your doing!!! I opened it up in Inkscape, imported a SVG of the PHP logo, and it worked perfectly! :+1:


Lol wat?


Any specific reason? I find it to be a useful warning.


С чего вдруг? Несколько раз уже это обсуждали же


Yeah, sorry guys I'll fix that :)


Wrong link: https://github.com/boltdb/bolt


Папку метадата не заливаем в репозиторий.


Посмотрите сколько мусора в папке метадата, окторый не надо заливать!


Yeah, it's a bit weird to just have very similar implementations in the code and the tests. Should we just get rid of the test then?


Папку рекомендерс также не нужно заливать.
Заливаем только полезный код, папку src


Thanks, changing.


Отлично


1) Названия переменных userfig и compfig говорят за себя
2) Очень избыточно, не сказал бы, что становится понятнее. Труднее читать - да
3) Ну, сплошной копипаст же


Not sure how to best do this. Not sure if a separate encounter link is very useful. For example, seems like that if a encounter exists, then as a physician, I'd want to open a new complete openemr frame that shows the patient summary screen on the top and the encounter on the bottom screen when click on this; and if encounter did not yet exist then would just show the patient summary screen in a complete new frame on top and the encounters listing on bottom(basically mimicking what happens when select a new patient in other parts of OpenEMR). You could even do what Rod does in the Patient/Client->Patients search feature where there is a toggle box to either open a new openemr frame or to do it in the same frame(thus would leave the tracker board when click on it). Just seems more natural since this is what happens in other parts of OpenEMR. If you think another way is better, could also have several different options in globals to choose from.


Ок


Лишние импорты можно удалить


Папку тест-аутпут не заливаем в репозитрий, это тоже мусор


Thanks for the update.

I would like to ask to split these three separate fixes into separate commits, too. In this particular case, it is not just a style and a bisectability question, there is also a concrete concern: It has been reported that the WSH DropHandler [does not handle Unicode file names](http://answers.microsoft.com/en-us/windows/forum/windows8_1-files/wshvbs-drophandler-doesnt-support-unicode/6c74a306-0655-489a-a22c-99909bf74a02). If that is true, we need to reconsider this particular change, without dropping the others (and that is much easier if the commits are clearly separated by concerns).

Would you mind doing the honors of [splitting the commit](http://plasmasturm.org/log/530/)?


Thats cute computer....

I havn't reviewed these commits yet I just dumped by working copy as I was travelling between cities. Rest assured I'll clean this up :P


много пустого места


My bad. I removed the trace in js_objects.js. Thanks for mentioning, and for the bug report. The bug was a wrong result when the attribute of a JSObject is a Javascript Array.


Duplicate?


не соращайте слова, birthDay birthMonth. birthYear


Can you paste your trace back please. I want to be sure that I am fixing
the right bug.

On Thu, 30 Apr 2015 01:49 Dirk Thomas <notifications@github.com> wrote:

> If it is not intended it would be great to change it back and release a
> fixed version of the package. Thank you.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/salimfadhley/jenkinsapi/commit/045eb7572a1b3a043644c3829dbbf72bdfda3912#commitcomment-10976052>
> .
>


I don't know if I agree with that. This is doing a really generic teardown. It is covering up any unexpected state left behind by the test. Like, say for instance the "New tab" buttons opens one new tab instead of two. That bug has some (albeit slim) chance of being caught right now since the tests run in the same instance. But a "verifyState" function and specific teardown code in each test would be better at catching a bug like that. I think Desktop does something like that. i.e. if you leave a window/tab floating around after a test, the test will fail.


the second value `jpeg_quality` is never used, is it? A few lines below we have `$thumbnail->get($thumbnailGetType[0]),`


When all possible values are known, gophers will sometimes emulate tagged unions using structs:

    type OpType int

    const (
        Line OpType = iota // c and fill unused
        Rectangle          // a and b are top-left and bottom-right, c unused
        Circle             // a is center, distance between a and b is radius, c unused
        Triangle           // a, b, c are the vertices
        Background         // only fill is used
    )

    type DrawOp struct {
       op           OpType
       stroke, fill Color
       a, b, c      Point
    }

If the "unioned" types don't have much field overlap, than such emulation will be space-inefficient. In this particular case, however, it works out to have the same memory representation that a true tagged union would have, due to the field sharing.  The paintingLoop function would take a `<-chan DrawOp`, and the switch expression would be `msg.op`. Perfectly type-safe


Папку тест-аутпут не нужно заливать в репозиторий. Это мусор.


Portugu**ê**s


Sorry I was in holidays, I will release version 1.0.0 today


Often it's a useful warning, but this project makes use of name-shadowing in lots of places. So I think it's good to have it in here reflecting the project specific coding style.

Whether it's in general a good or bad idea to use name-shadowing is probably a longer discussion. :)


Quality commit


It is: https://github.com/osmlab/editor-imagery-index/commit/a517a58abab7b2458cdd80f579fbffaecc15d380. In this commit I did just run `make imagery` to have it in iD/master asap.


Всё же лучше так импортировать
```python
import nc_logic
```
Будет сразу видно, что вызывается из другого модуля. Тем более использоваться в коде функции из него будут пару раз всего


It will be reverted


this.name = name; - не дублируем слово group в названии и в полях, 1 на конце убираем


1 на конце убираем


Лишние импорты удаляем


Quality Commit


It probably doesn't hurt to leave it as it is.


Ref #5


@jpd002 do you know if there are any tricks to receiving DMA via destination chain tags (e.g. channels 5 and 8)?

I've mostly been testing channel 8, and I can read direct DMA from it fine, but I can't get dmatags to work (they just lock my PS2 up...)  Source chain dmatags on channel 9 work fine.

I'm basically doing this:

```c++
DMA::DstChainPacket dstTag(512);
dstTag.END(buf, 1);
DMA::SendChain(fromSPR, dstTag.Raw(), 512);
```

I also tried creating a `ReceiveChain` which just took an initial MADR/QWC as well as the tag, but that locks up too.

-[Unknown]


np; thanks :)


пустая строка?


Опечатка в слове Data


w00t! I will always celebrate the removal of tricksy middleware.


тест-аутпут удаляем


The instructions certainly need improvement. I am hoping to get the whole thing up and running and improve the instructions.


What is this line doing?


сделать проверку на null для text


这里主要是要传barCategoryGap这个东西进去。我在想如果不是传到data_bar()里面去的话，就得在下面params$series里面加了。我现在是把data_bar()里面加了很多参数...


i would expect a constant with a nice name here for the string value


加了一些参数。


As long as it works, it's not hackish! ;)


искать элемент можно 1 раз здесь


тест-аутпут удаляем, и больше не заливаем


This gets a special translation function:
``````
<?php echo text(xl_appt_category($appointment['pc_title'])) ?>


Yes, The OrderType is getting set automatically from Concept. Not setting it explicitly. Just had the Property in the Object to support when we edit the existing TestOrder.


This looks awkward IMHO


如果这些参数有指定的话，就需要传到series 那里去。json的格式是比如
"series":[{"name":"","type":"bar","data":[1,0,0,3,0,2,1,2,0,1,2,3,2,2,0,1],barCategoryGap='0%'}]
这样。我在想是从这里就加进去还是另外写一个function加参数。


Note 2


Note 3


Ok, I had to change this setting manually with my Intel and Radeon cards.


Shouldn't the apptstatus be coming from the tracker table (although they should always be the same)?


这个纯粹就是 bar  plot，只是告诉echart() type="hist" 但是实际上就是把barCategoryGap='0%'传进去就好了。any better idea?


Non-exported methods are occasionally used to prevent third-party implementation of an interface, but it's not especially common (http://golang.org/pkg/go/ast/#Expr is one of the only places in the stdlib that uses it), and you may note that Expr does export methods. If you're just trying to lock down an empty interface, it's probably more idiomatic to express that via a non-interface type.


@marcysutton adding line-height of 2.8rem is setting my icons to 28x28, shouldn't this be 2.4rem? Google material guidelines state system icons should be 24px


Thank you. Found the setting and disabled it. :)


поиск элемента можно сделать 1 раз в этом методе


I'd remove this and instead don't show the link in patient tracker main board when it's not pertinent.


очень хорошо, много тестов


![screen shot 2015-04-29 at 11 40 41 pm](https://cloud.githubusercontent.com/assets/175005/7407605/2d46e8e6-eec9-11e4-8a89-e322f0679c06.png)

现在画出来大概是这样的，就是code估计要重新写...


The binding was designed to observe a minimum waiting time (default 50 ms) when communicating to the  KNX bus to prevent flooding (KNXConnection.getReadingPause()). With this iterative call this waiting time seems to be circumvented.


Déjà fait ;-)


можно искать элемент 1 раз


сделать проверку на null для text


You are welcome! :)


Ok!


Thanks. Guess I should try it, but will probably stick with `tap`.


Or you could, you know, actually initialize the variable to a sensible value (such as `0`) instead of doing this little weird dance that requires an explanatory comment and still leaves the variable uninitialized (or non-obviously initialized)...


update name and url


note $res is not an array but is a string


lgtm


Suggestion: wouldn't it be better to add the failed datapoint back to the reader's queue?
The queue management would then need to be expanded to do the "retries" housekeeping".


the campus data is not available so we choose to make it default to main campus.  The schema links the course and campus so i should be able to populate that now. The courses added for each line of data are not the same. for instance boland college is entered five times in separate rows because of the different courses they have. We were trying to do the best we have with the data without actually overdoing staff as we do not know how much the complete dataset will change once we get the final spreadsheet


attr()


that sentence doesn't look correct...


So how do we extend blade?


regarding code conventions: the operator should be padded by a space, also a space after the `if`statement
```java
if (projection.getMasterVariant().getImages().size() > 0)
```

do we have a code convention defined for our Java open source projects?


Suggestion: move your change to the groupWrite() method, as this is the interface to calimero


If you want it initialized to 0 I can initialize it to 0, but despite the compiler's heuristic (which means that the compiler will not optimize out the assembly instructions to zero the variable) the variable is guaranteed to be initialized later. C and C++ programmers are often already familiar with this no-op, no-warning syntax, but I left a comment just in case.


This needed to be before the div, so it would show above the gallery, since it's the id that is getting injected to.


Woo!


css has to be loaded


Make it to where actual cutoff frequency values (Hz) can be used on the filter. This is done by taking the sample rate from the host and dividing the cutoff frequency by the sample rate.


Update on instructions text


Whoop whoop


Excellent !


Can get username this way $_SESSION["authUser"]


@lgiavedoni quick question: can you check if your git user (email) settings are defined properly on your local environment? Currently github shows you as "unrecognized author" (see header)

![bildschirmfoto 2015-04-30 um 08 51 55](https://cloud.githubusercontent.com/assets/131689/7407832/320d11d2-ef16-11e4-9693-141b61167e69.png)


The "unfinished" -T option was replaced with a more general -g (--group) option which allows testing with any of the standard SBo categories as well as the whole repo.


Disregard above comment. I see you are using for the link to patient board. nice.


LinearLayout is more effective for simple view hierarchies like this one.
https://developers.google.com/events/io/sessions/325615129


Actually, I overlooked something very crucial... `git-bash.exe` is a GUI program that does nothing else than calling `mintty` -- which is a GUI program, too, therefore the `SW_HIDE` does not do what we want it to do. And indeed in my test on 32-bit XP, I saw the flashing window. I opened [a new ticket](https://github.com/git-for-windows/git/issues/126).


Danke für den Hinweis! Ich geb Dir Bescheid, sobald ich das gefixt habe.


And '**pt**'?


это не правильно. Второе объявление связи перетрет первое и получится, что у нас только один курс для группы. Имя поля всегда должно быть унакальным


Is it ok to remove $scope.hideTooltip at all?


Will be safer to store it in the notes column. Then all you need to do is change the name of the Notes column (see how the language list_id does it). The mapping column is used for something specific (of which I don't know exactly), so best to avoid it.


/CC @cvrebert @hnrch02: seems to work now on Windows with Lucida Console.

Before
![before](https://cloud.githubusercontent.com/assets/349621/7407882/444518be-ef1f-11e4-8d6c-427563ccf5db.PNG)

After
![after](https://cloud.githubusercontent.com/assets/349621/7407883/49d70ba2-ef1f-11e4-93de-25989d6e60b3.PNG)

Travis output seems to be fine too.


Сериалайзер не верно написан. Чтобы передавать embedded records, нужно использовать has_one/has_many


- Added proper image upload functionality
- Display uploaded images on Gallery page
- Id of the user who uploaded an image is stored in the database
- Retrieve id of the user through use of sessions
- Corrected Sign In and Sign Up functionality


No reason to not show this column. Some crazy soul may want to assign the color term a SNOMED term.


Yes, that won't work (it'll generate `import dep "."`). This isn't designed for every case; you should either put different protos in their own directory (when they are in different packages), or compile them together (in which case the generated code won't have imports). The `M` flag can override import paths if there's truly a need for independent .proto files to be in the same dir.


D'ailleurs génial ton optimisation =D
C'est de nouveau fluide chez moi ^^


1. It was first time when I worked with perl, so not sure about that part. About objective-c part: I plan to check that codi is ok for new xcode and hight-resolution icons for retina.

2. On mac it works ok for me. I use it every work day. But I not remember, was my perl changes tested in windows and linux or not. But I can test it.
Also note that it is simple status icon without any menus, etc. One icon for all realsync processes. It useful for long updates(rsync) for small updates icon changes to normal state too fast(but in my understanding it's ok).

I will check/update code in this weekend and then you cam merge it if you want. Thanks


Should be changing this to getListItemNote()


The problem with two functions is that the status change in the calendar appointment screen needs to basically be the same as the patient board. (or else for example, after check out, the status changes in the appointment screen are not recorded).


Let me think about this some. I'll probably try to come up with a function that you can use in both places.


Awesome! I edited the comment to notice that.


these emails are annoying please disable Github

On 30 April 2015 at 01:20, dx <notifications@github.com> wrote:

> this causes dx errors
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/infoforcefeed/OlegDB/commit/dd80def63f913d4d468b9bdbd427fae4268ae6ae#commitcomment-10975214>
> .
>


There should be one empty line at the end of each file.


Awesome bit twiddling .  Can you suggest a better way to calculate the neighbors?
This is the last missing from otherwise complete implementation.

```java
 public static long getNorthernNeighbour(long hash)
 {
    long lat = unwiden(hash);
    long lon = unwiden(hash >>> 1);
    lat += 1;

    return  ( widen(lat)) | (widen(lon) << 1);
 }
```


@Hamcha Unwatch the repo


is it possible to have a real test on `ClientInstallationServiceTest` as well ?


May be useful for such things:
http://docs.oracle.com/javase/6/docs/api/java/lang/String.html#format(java.lang.String,%20java.lang.Object...)


ObjectId nie powinien się tutaj znajdować, Klasa jest generyczna - zostawmy ją taką.


Tej zależności nie powinno być


Not really, we can easily include the checker in `cidr.js` if I want to represent that range more accurately.


.split is the wrong method

use findall ie

[code]
  timelist = re.findall('(\d+[DHMS])', length)

    length = 0
    for t in timelist:
	t_field = int(t[:-1])
	if t[-1:] == 'D':   length += 86400 * t_field
	elif t[-1:] == 'H': length += 3600 * t_field
	elif t[-1:] == 'M': length += 60 * t_field
	elif t[-1:] == 'S': length += t_field

    if length > 86400: length=time.strftime("%d:%H:%M:%S", time.gmtime(length))
    elif length > 3600: length=time.strftime("%H:%M:%S", time.gmtime(length))
    else: length=time.strftime("%M:%S", time.gmtime(length))

[/code]


`ClientInstallationService` is not touched by the analytics, the endpoint class calls the `PushMessageMetricsService`


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


28b331a : apparently I missed a spot


Masz na myśli jakiś sposób wykorzystanie baseURL w BE?


Thanks, you are right!! I extended it, see my new commit.


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.0-DEV4](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=227680&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) is now running


Best Fixed Here


OKです。
次の問題に進めてください。


TeamCity .NET :: NET - Redis4Net :: Continuous integration build [Build 0.5.0-DEV4](https://teamcity.coolblue.eu/teamcity/viewLog.html?buildId=227680&buildTypeId=Net_NetRedis4Net_BuildAndPublishPackage) outcome was **FAILURE**
Summary: Compilation error: src\redis4net\redis4net.csproj Build time: 0:0:58


Did you mean something like "By simply **adding the** `weld-servlet`..."?


Color::COLOR is a bit redundant. If you use namespaces, you could drop the COLOR. Looks nicer.


space :P


This should all be moved to the library/js/jscolor directory


Will do.


Please save PHI->getBasicBlockIndex(NewBlock) in a local to avoid calling it twice.


In JS this should be:

```javascript
          slidesContainer.bind("mouseenter", function() {
            clearTimeout(_this.data.restartDelay);
            $.data(_this, "restartDelay", null);
            return _this.stop();
          });
```

I think it should be committed..


phi --> PHI for consistency


Thanks for the shoutout @jansenbe :)


@paddybyers yes, I think it is needed.  It is quite feasible someone wants to instance a library yet control when it connects.  I don't think it's urgent by any stretch and is a low priority feature, but I think we should have it given how easy it is to implement.  I'll change to `autoconnect`


Hi Thomas, I think this change caused the txn and address labels to become uneditable


Please add a comment.


How can we prevent this happening in the future?


You are able to use maven repositories here if you want. It may be more convenient.
For example, gson can be included using the following string:
compile 'com.google.code.gson:gson:2.3.1'

Here is where to get this string:
http://mvnrepository.com/artifact/com.google.code.gson/gson/2.3.1


Yes we can merge these 3 functions in one, which will return an enum.
https://github.com/mikrosimage/avTranscoder/issues/180
But we want it to stay private, because a user does not have to know if the process will transcode / rewrap / any action with tht stream (and it's more complicate than that, because of the offset...).


CodeReview: add annother line


I agree. However looks like this repo/plugin is no longer maintained.


As before


TODO: config this; it can by language


I had the same issue with the curl operator.


Do you have Battlefield 4 installed at C:/Program Files (x86)/Origin Games/Battlefield 4 ?
otherwise you have to change the source code of the program :)


I'd buy any other solution though... Any idea welcome!

Le jeu. 30 avr. 2015 08:39, Christopher <notifications@github.com> a écrit :

> Ok, I had to change this setting manually with my Intel and Radeon cards.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hevs-isi/gdx2d/commit/f24abbf537dab2eeb991dceef1b3c316e25f5a51#commitcomment-10978659>
> .
>


TODO: translate each language name


was sending twice intended?!


@robertmesserle I cannot longer use md-warn and md-accent on toolbar icons after this change


Resposition? :P


What about PowerShell?


Selectivity 0 is scary it messes up cardinality all the time. Would it work to just return 1.0 (i.e. ignoring the predicate) and rely on label cardinality = 0 instead?


Yes a generator is also created when we transcode from an existing stream.
The changes here concern the generator case: we want to have a pointer to the generator from _generator and from _currentDecoder. This is just a clean, to keep a symmetry between StreamTranscoder's constructors.


oh! MY fault. Thanks.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.10](http://teamcity.monogame.net/viewLog.html?buildId=6567&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 485, ignored: 6 Build time: 00:15:03


Small variable rename, but LGTM after that


Pull request: #58


got it.


Great overall, please double check that there is a matching *PlanningIntegrationTest for the index scans.


Ooops, thanks


It has the same issue, except instead of blocks I see questionmarks. With this patch, it works fine.


Perhaps just add another map/counter?


CodeReview: add annother line in test4.txt the second time


嗯那就这里直接传进去好了....好傻的感觉？


You have to recompile the program & export it, that you have the new .jar!


强行扔进去....喵呜。


不需要判断了...


Actually, I think we should review the purpose of this class. It appears to only be used in test code, and the indexSelectivity() method not only returns zero in this case, but returns a number that does not grok in the other case. So we actually need to fix both methods (both indexSelectivity and the new indexPropertyExistsSelectivity). Right now I believe the new method is more correct, but as you point out not completely so.


More typically (and contrary to the above tagged-union suggestion), gophers will use a non-empty interface, so that third-party implementations can be accepted in a type-safe way, modeled on behaviors. For example:

    type Drawer interface { Draw(Canvas) }

paintingLoop (and drawThingy) would simply accept a `chan Drawer` -- since neither bools nor strings implement the Draw method (specifically taking a Canvas as its argument), the compiler would prevent sending those down the channel.

Utility functions, like `NewCircle`, `NewRectangle`, etc., would return Drawer implementations that draw themselves, and additional Drawer implementations could be created. Assuming Canvas is something practical, like a pixel buffer (such as one of the types in http://golang.org/pkg/image), these third-party implementations would enjoy a great deal of freedom -- more than would be afforded by a non-extensible library that only exposes the set of shapes that the author thought of or deemed important.

I'd argue that this approach (entities that draw themselves on a canvas, rather than a canvas that draws predefined shapes) is a superior design, regardless of the language.


The added lines concern properties stored in the base class _StreamProperties_ (streamId, streamIndex, timeBase, duration). We call the function of the based class (_StreamProperties::getPropertiesAsVector_), and continue to insert specific audio properties to our result. I think this is the best way to have most factored code (the same code was done for VideoProperties).


Don't forget the include for `std::pair`.


I also prefer the spaces, but I want to spend my time more likely in discussing higher level stuff - what the customer needs and not discuss formatting.


Seems, in the end it's not needed since `instances` variable and the functions in question are in the same scope.


This approach has some downsides too. My major issue was with the vtable dispatch that happens regardless, but the other issue is that local variables get harder, and it gets more complicated when you have multiple possible receivers.

I do intend to add a section on this though, since this point got brought up countless times on proggit and HN

> I'd argue that this approach (entities that draw themselves on a canvas, rather than a canvas that draws predefined shapes) is a superior design, regardless of the language.

The message pattern is a rather good one IMO and sometimes works out better than the usual object oriented stuff


I think view is misleading since it is still just data and no visual representation.


LGTM.


Střední škola technická a ekonomická brno


CodeReview: add annother line in test4.txt the third time


duh! fixing...


So this really becomes `Directive` and not `Decorator` ?


There is a

    ",

missing at the end of the line ... please verify that the JSON syntax is ok for the rest of the file too!


Yeah, I did, it's pretty awesome!


Yes I think to do something else, but it don't work..


![uten navn](https://cloud.githubusercontent.com/assets/1188192/7408380/8af948b4-ef1c-11e4-8775-58af53878113.png)


The framework itself has provided the feature to complete branch name, but we have to implement the feature in PHP, e.g., the branch name.


I fixed it after merging here: https://github.com/BitShares/web_wallet/commit/f7259e231a393caec70fe502996ee08934ac001c


is it possible to pass the reference in parameter to this fonction ?
It will remove a copy of common properties :-)


Concerning the namespace std, I don't know yet the good practice. If you know you can improve it, but please create a ticket to explain why it's a bad practice.

Concerning the camel notation, at the end, the project will be more a C project than a C++ project. The idea is to use classes just to represent the geometric models. Everything else should be function invocations in the good old C style. In C, they don't use camel notation, thus better to keep the consistency in the entire project. In C++ we have this freedom. In Java, we are forced to write everything in classes, even when classes are not appropriate.

I agree with you about the bullshit details.


```
pry(main)> File.extname("google_base.txt")
=> ".txt"
```
I don't really like that . there. We would need to split again, as requiring the format parameter as '.txt' for the rake task is probably not a good idea.


Yes we discuss a lot with @valnoel to decide what is the best place to create and add stream properties to the FileProperties. We decided to move it from InputFile to FileProperties, because this last class can be useful without the current InputFile implementation.
Do you agree with our choice?


Dynamic dispatch of _this_ kind is rather efficient in Go; my impression was that your main concern related to the lack of compile-time validation using the empty-interface approach -- a weakness that this approach does not share (since it involves safely programming against a concrete _behavior_, which the Go compiler can fully analyze).


This has to be pulled. Worked fine for me.


I can make sense, no problem with that :+1:


Thanks my friend!


Right, will change this back.


Won't work on python3. use six


shouts to @javierasandoval


The problem of this constructor is that it would require set functions for the two coordinates. It breaks encapsulation because set functions are dumb functions. Those coordinates will change when we call functions such as `zoom`, `rotate`, etc.


2015, Ericsson AB


Either the `Close()` call should be inside `finally` or `tempFile` should be wrapped in `using` to ensure the file is closed if there's an exception.


I wouldn’t say C++ programmers are familiar with this — and if somebody is, it’s definitely not something any would consider *good* practice. Clarity is better than obscure hacks and obscure hacks that any static analysis flags (as Clang puts it, “assigned value is garbage or undefined”) are doubly bad.


Hmm, I definitely misunderstood this part. It indeed breaks blocks on HTML paste etc. Will revert this back,


Thanks for the response, looks like some Sunday afternoon reading for me.


You do not need those assertions here, a `FieldMatrix` is guaranteed to have a static size.


That solution works. Thanks.


`Delete()` should be in `finally` block to ensure the file is deleted if there's an exception.


Indeed, it is a typo. Sorry.


Change the color to #099909 instead of #000000.
Remaining code looks good.


A teraz poczytaj o klasie SimpleDateFormat :D


What if the user chooses not to install node4ords and/or wants to use tomcat directly?


If we put the rectangle up side down, what will happen with up? ;-) It will become visually down, but still called up. So, I suggest to represent all the lines in a fixed size vector.


We have a bunch of tests in LeafPlanningIntegrationTest for this. They were written during the original work on IndexScan a month back. See lines 70-141.


This condition is going to be called almost always, while the previous only rarely.


Why do TTJet and SingleTop appear twice?


I liked that `tomcat-webapps` gave a root webapp for tomcat.


We have Sonar 4.3.2, and in the plugin update center, there's a warning in the PDF Report update info: *"PDF Report 1.4 is not compatible with SonarQube 4.3.2. You need to upgrade to SonarQube 4.5.1 to perform the plugin upgrade."* According to this, we should update sonar first and the the PDF plugin, among other updates requires for other plugins we have. As it's a corporate server, I'm not sure we can do that **quickly**, can't this feature get fixed?


The vector for the rectangle would solve the naming issue in the triangle. What do you think?


I just tested you code and for me works fine!


Yeah, but it's not the messaging pattern :)

Like I mentioned, it gets complex with local variables and multiple receivers. It becomes harder to separate libraries logically because of this. Etc etc. The messaging pattern is a common one in Rust where you send a message but not the behavior across a channel -- it has its own benefits -- and I miss that in Go :)


there should be mInverse, not m_inverse
m_inverse is never used in the code


Even After doing the same i'm getting the same Error


Oh I was not aware of the plan to update a FileProperties, and use it to update the OutputFile. Here I update FileProperties to have a "read only class", which exposes properties.


var observer = new Observer(setting, callback);


I reverted this since this breaks the post link.


this.onsuccess && typeof this.onsuccess === 'function' && this.onsuccess();


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


this.onerror && typeof this.onerror === 'function' && this.onerror(error);


Please change the color as per @visa-git comments and create new pull request  for ewview.


For me it seems that those themes still got deleted after I started Emacs. They only came back after I included them in `themes-megapack/packages.el`


La CNIL autorise la non suppression des données ?


:+1:  for noticing this!


These 2 lines should be removed


This url must be the same as the one you use in navigator.connect

const SETTINGS_SERVICE = 'https://settingsservice.gaiamobile.org';


I think it is good as it is with 4, 4 tries. Having done *lots* of test-uploads, I only had one occurrence, where it still failed. When using 4, 2 there was a higher risk of failing, even when the upload did never start at the 3rd or 4th inner loop count. Maybe the slower outer loop reset repeat rate due to the higher inner retries improve the probability? Whatever. 4, 4 seems good.


remove commented lines, please


Good Suggestion, done.


#68 @yasseraboelsaad UML reviewing done


some lines are 8 space indented, but should be tab


Yes, obviously :) (it has an isolate scope).
But what was wrong with the previous implementation (2-way data-binding) as long as you don't try to "write back".


The first time we observe a setting, _observers[setting] is not defined, so we should check it

if (!_observers[setting])
      _observers[setting] = [];


![cuff](https://cloud.githubusercontent.com/assets/7131393/7408651/d65bb60a-eed3-11e4-8c15-b8d9f56c0959.png) who?


//============================================================================
// File: MicroSecondTimer.cpp
// Purpose: This class creates a timer which can be used to time events
//     to the nearest microsecond.  It is more accurate than using the
//     calls to standard time.h functions which only give millisecond
//     accuracy.
// Author: Adapted from code provided by Gregory A. Gibbs
//============================================================================
#include "MicroSecondTimer.h"

//---------------------------------------------
// Default constructor
//---------------------------------------------
MicroSecondTimer::MicroSecondTimer()
{
	// Get the query call execution time for calibrating out
	QueryPerformanceCounter(&m_iStartCounts);
	QueryPerformanceCounter(&m_iEndCounts);
	// Get the hardware base timer frequency
	QueryPerformanceFrequency(&m_iBaseFrequency); // current frequency
	m_iCalibrationConstant.QuadPart = m_iEndCounts.QuadPart - m_iStartCounts.QuadPart;
}

//---------------------------------------------
// Start the timer
//---------------------------------------------
void MicroSecondTimer::Start()
{
	QueryPerformanceCounter(&m_iStartCounts);
}

//---------------------------------------------
// Stop the timer
//---------------------------------------------
void MicroSecondTimer::Stop()
{
	QueryPerformanceCounter(&m_iEndCounts);
}

//---------------------------------------------
// Get the elapsed time in seconds
//---------------------------------------------
double MicroSecondTimer::getTime()

{
	//calculate the number of elapsed microseconds
	//seconds = (end counts - start counts - calibration constant) / frequency)
	//Return the number of Elapsed Seconds
	return double((m_iEndCounts.QuadPart - m_iStartCounts.QuadPart -
					m_iCalibrationConstant.QuadPart)
					/ double(m_iBaseFrequency.QuadPart));
}

//---------------------------------------------
// Handle the timer overhead. This is needed
//   only if multiple timers are nested, i.e.
//   running simultaneously.
//---------------------------------------------
double	MicroSecondTimer::getOverhead()
{
	//This is the overhead of the timer
	//this is needed only if multiple timers are nested
	return double(m_iCalibrationConstant.QuadPart / double(m_iBaseFrequency.QuadPart));
}


never do this.
šo risina iekš css


In fact, my icons are bigger too. :P


//============================================================================
// File: MicroSecondTimer.h
// Purpose: This class creates a timer which can be used to time events
//     to the nearest microsecond.  It is more accurate than using the
//     calls to standard time.h functions which only give millisecond
//     accuracy.
// Author: Adapted from code provided by Gregory A. Gibbs
//============================================================================
#ifndef MICROSECONDTIMER_H
#define MICROSECONDTIMER_H

#include <Windows.h>

class MicroSecondTimer
{
	private:
		LARGE_INTEGER	m_iCalibrationConstant;
		LARGE_INTEGER	m_iStartCounts;
		LARGE_INTEGER	m_iEndCounts;
		LARGE_INTEGER	m_iBaseFrequency;
	public:
		MicroSecondTimer();		//default constructor
		void	Start();		//start the timer
		void	Stop();			//Stop the timer
		double	getTime();		//return the elapsed time in seconds
		double	getOverhead();	//return the overhead that this time has
				//The overhead is only needed if you are using multiple times
				//  if multiple timers are used in a loop, the execution time
				//  of one time will add to the time in a second time. The overhead
				//  time can be used to remove the timer execution time from the
				//  first time.
};
#endif


I've done similar things to the Drawer example above, and don't know what you mean by the "local variables and multiple receivers" problems. It's possible that, in this regard, there is a slight terminological difference between these two languages.

I'll wait for your HN/etc response, as I'm not familiar enough with Rust to know how the term "messaging pattern" is specifically realized. Without that specific context, the term is rather overloaded and ambiguous, in my experience.


Hi Omer,
Given that the DateTimeAdapter has no state I'm pretty sure you can get away with using a singleton. (No functional diff, but should improve performance. Very slightly, but still.)


From what I understand NoWait applies only when you're starting a new build, where it takes precedence over Follow. In all other cases NoWait is ignored and only Follow makes sense. IMHO this is perfectly OK.


Oui. Personne n'est tenu à supprimer toutes les données d'un utilisateur : ça serait trop ingérable.


The above files are complete source file for Marathon Demo! :)


Extra empty line detected at block body end.


Is there a reason for 1e5? If so, can it be documented? "default is 1e5 like RMSProp because..."


Indeed, the latter will be more usefull. For weblogo, the link can be: https://github.com/pierrepo/PBxplore/blob/master/doc/installation.md


The previous implementation was running in the wrong scope. In a more
recent commit, I removed transclusion and handled it manually to address
the scope issue.
On Thu, Apr 30, 2015 at 12:57 AM Georgios Kalpakas <notifications@github.com>
wrote:

> Yes, obviously :) (it has an isolate scope).
> But what was wrong with the previous implementation (2-way data-binding)
> as long as you don't try to "write back".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/material/commit/973a2fcadfc0fb786635cabf7c6674e5027117c4#commitcomment-10979668>
> .
>


Ah, PEP8 my friend. Maybe split out the denominator to make this a bit prettier?  Also, should there be a space between `-` and `epsilon`?


Please have a look at this again, this seems to fail when you try to open the lightbox from script, see the ajax demo at the demo page http://wet-boew.github.io/wet-boew/demos/lightbox/lightbox-en.html

Can i suggest to change line 215 to: filter = (currItem.el)?currItem.el.data( "wbLbxFilter" ):undefined,


@DimaKoz  Please leave these comments in, as they serve as place holders for a future table.


According to ffmpeg duration's documentation:
>If a source file does not specify a duration, but does specify a bitrate, this value will be estimated from bitrate and file size.
And bit_rate can be equal to 0 if it is not available... Maybe the function _StreamProperties::getDuration_ should throw an exception if the duration is equal to 0. What do you think?


@popcornmix Thanks for this one! It really improves picture quality for SD TV playback. Especially tickers and stuff like that are way better now!


I find it less clear to write "trans = 0", because it makes it look like trans is initialized in one place when in fact it is initialized somewhere else. But it's not a big deal, and I'll push a new version in a few minutes. In fact, the Linux kernel developers had this same debate and eventually sided with you: https://lwn.net/Articles/529954/


Might be good to include a script in the repo that downloads the latest versions of web-platform-tests stuff.

It could even be some kind of prepublish check that fails if the versions in the repo aren't the latest.


I'm not opposed to this repo being written in TypeScript :). Probably a bit late by now though.


Ahahah fantastic!!! Alex Cruz, Alex Cruz everywhere!!


@EagleSixty6 I think link_directories is expecting paths not libraries. I think this line could be erased.


> They follow a different RFC than the URL spec references, so we'll have to lowercase ourselves first.

This seems kind of bad. @mathiasbyens, is there an IDNA.js or similar?


@EagleSixty6 I think WSOC should be lowercased otherwise it won't compile on case-sensitive filesystem


Normally I would say, we need no retries, because eibd shows, how stable a KNX connection could be!
But calimero still has some internal problems, which causes timeouts and dropouts. Often, not every time some commands will be lost. See for example: https://github.com/openhab/openhab/issues/2508


Like this?

Doesn't look like the JS files are checked in, so I assume it's ok that it's only the .ts file.

Btw, is there a way to run locally to confirm my change? I did it via fiddler intercepting the request on the real site, but I'd rather be able to run the site locally or something like that to double check my changes.


we should depend on "release" (not a shadow-fixed) version of the software


oh, I didn't know you could do that. :)


Frytiks, duplicate spellInfo->HasAttribute(SPELL_ATTR0_TRADESPELL)


Really ? I thought it was part of clang starting from 3.3 ? It is not really a big problem if you remove it. It just means that the code won't be thread-safe anymore (so I won't be able to use it to generate random numbers in the raytracer, for instance).


head


*SHOULD use txtpnabc.setText to use html tag*


FormatContext is ffmpeg dependant, so I don't think it's a problem (for now, with only ffmpeg/libav used by avtranscoder).
But we paid attention of the InputFile class, and so it doesn't manipulate ffmpeg seek flags: it only uses seekAtFrame and seekAtTime.
What do you think of this solution?


merge


And I just realised that this thing :

defined(__STDC_VERSION__) && __STDC_VERSION__ >= 201112L

Should not be there. It remains from an old change that I pushed by mistake. I am going to clean up all of this (by the way, it is better I think to check for posix_memalign first, because for the Katana render plugin I am limited to an old version of the glibc).


Can you add an `isset` check to avoid notice errors ?


@omaromar94 please fix the indentation here


/play drama


@omaromar94 please fix the indentaion here


After, it used to compute the complete stream. If the duration will be 0, it don't process nothing...
Some codecs return every time 0 (especially raw codecs).
For this reason it will return the duration from the InputFile because the duration can be estimated with file size and others stream properties. Which allow the process for the library during the transcoding.


Local variables -- What happens when you need to access and update variables local to the receiving thread/loop? You need a context object then.

Multiple receivers -- what if I want to send this message to different receiving threads, each which handles things differently? I need to create multiple impls. This gets harder if the receiving threads aren't mine to define.

By messaging pattern I mean you just send signals between threads (which contain data and nothing more) and the "logical control" of the data goes to the receiving thread. (So there is no globally defined "behavior", the receiving thread defines it locally. This lets the messages be reused, sent across processes, and sent across abstraction boundaries with ease.

My responses already are on HN and proggit :)


@omaromar94 please fix the indentation


What entity do sourceId and targetId identify? Please rename so that this is clear.


@rxin This can't be how users are going to call this method in Java, right? Because currently, even though I added `List[String]`, it says method not found (in IntelliJ, didn't try to compile).


I would leave plugins and namespaces as they are for now. Not a user "visible" matter in any case.

In the end we still have a lot of classes named "QLCSomething" while they should be named "QLCPlusSomething" :wink:

(P.S. I still have to generate the 4.8.5 documentation and upload it online...)


I'll answer on the issue itself.


Why not use normal chaining?

```
$($o.timePassRevenueModel + '[value=' + passData['revenue_model'] + ']', $timePass)
.prop('checked', 'checked')
    .parent('label')
    .addClass($o.selected);
```


Fixed already. This is old info )


java,util.List - not scala list


I think it better to have this method:

void seek( const uint64_t position, const ESeekFlag flag );


The offset is stored in _offset attribute.
The output stream duration can be updated at each new call of _StreamTranscoder::processFrame_, depending of what was wrapped. How do you think we can skip the compute of the stream duration?


The temporary file will be left on disk unless `Delete()` is called which can never happen if there's an exception and control never gets here.


Oh, for example we should change this as well
https://github.com/mcallegari/qlcplus/blob/master/ui/src/app.cpp#L113
but it would break backward compatibility on OSX, since that is the base path used by QSettings

Maybe we can do some more "migrations" later on QLC+ 5


тут свічі приймають різні філди, зверни увагу на це. Мені потрібно проініціалізувати direction(напрямок мого танку) і enemyDirection(напрямок ворожого танку)


aha, thx


@valnoel was assigned to the issue ;)
https://github.com/mikrosimage/avTranscoder/issues/180


jsk_tools 1.0.66 seems to be already out publicly http://www.ros.org/debbuild/indigo.html?q=jsk_tools


please add comments


On 30/04/15 10:01, Hub wrote:
>
> Indeed, the latter will be more usefull. For weblogo, the link can be:
> https://github.com/pierrepo/PBxplore/blob/master/doc/installation.md
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/HubLot/PBxplore/commit/386d2f3a20aed994648cd1cb95fb0afe28c88c58#commitcomment-10979724>.
>
This should be OK. Yet, could you make the URL an argument? This way we
can customize the URL and point to the weblogo help if it is better.


I do not see the corresponding code in `test_laplacian*`, please add it


w sumie nie potrzebne :)


See https://github.com/mathiasbynens/todo/issues/9.


Added the syntax highlighting to the README file... pressed enter accidentally when trying to add a comment and a pull request...


Hmm, isn’t `.toLowerCase()` still incorrect? It doesn’t perform full case folding as per the Unicode standard.


Map це тип делегату, не бачу можливості екзепмляри делегату робити пріватом


tape works cross browser as well, which is relevant for libs like levelup


а це евент...


sdf


where is the database located at?


Good point! -> UI test for Rows count picker #7794


idem for @romainhild ;)
thanks in advance


s/implemets/implements


try not to use this built in function


якщо язроблю приватний сет, я не зможу в танк енжін зробити екземпляр цього класу


@marshmellow's changes found here.
https://github.com/marshmellow42/proxmark3/commit/f168b2633b28e52e8ca08ace5f3e62f1767d9df2


I guess that a "CassandraDefinitionsServiceImpl" is a big name also just for usability in the IDE.
I thought Db for database or Cass for Cassandra could work, anyway naming conventions have always these kind of discussions, we can review them later just to follow a single pattern.
It's not a big deal to change them in the future.


Hello,
    I am using Eclipse on my Laptop with windows8 os and I downloaded Mingw and Cygwin as well. Now i am building the project. But unfortunately i got the following error.

'_byteswap_uint64' was not declared in this scope	 booost_test		line 100, external location: C:\Users\Jay\Desktop\Implementation\DeviceHive1.3\cpp\include\hive\swab.hpp	C/C++ Problem

And i didnt get the reason. Can you help me with this??


The following abbreviations are recognized by ffmpeg for video size:
```
‘hd720’
1280x720

‘hd1080’
1920x1080
```


Is renaming this public method to FindDependencies possible for consistency's sake?


Ah thanks, I've changed the password ;). It's the first time I'm using Github so I was not aware everyone could see it.


They shouldn't. It should be
- ttjet + single top (combined)
- ttjet
- single top

> On 30 Apr 2015, at 08:46, EmyrClement <notifications@github.com> wrote:
>
> Why do TTJet and SingleTop appear twice?
>
> —
> Reply to this email directly or view it on GitHub.
>


do you need count>0 here?
Also, what if the string ends with spaces?


@noonereally
How about this commit?
maybe @Phr33d0m is interested? :D


lgtm


ок, але потрібно враховувати напрямок кулі.....тоді в запропонованому методі все одно буде свіч з 4-х елементів


1: M101 signature is missing
1: M111 unrecognized component name: template


Assignment 6.1 Kruskal & Bellman Ford.


New issue: https://github.com/mikrosimage/avTranscoder/issues/181


Oof, OK, that sounds like a lot of work. And @jcranmer's comment at https://github.com/mathiasbynens/todo/issues/9#issuecomment-68493815 is not very heartening either.


try to avoid return value with void. the simple return is bool


Isn't there a new Ecma 402 API that does that?

Also, if we fix this, I insist that we add a test case before doing so.


Easier to update / maintain. Don't you think?


s/Can't/can't/

go recommend lower-case for the error message.


This is somewhat unusual to understand.
End the statement with semi-colon and create a new variable below.
var holdingsSection = ...


Ease the thing a little bit:
objectPage.find('.group-holdings-available .field-group-format-title')


Should `resourceURL.openStream()` be cleaned-up too?


1: M101 signature is missing


good game


cl_image_handler.h is the basic CL handler, should not have any specific structure there.
The only thing you need to to is add a enable function.
void set_enable (bool enable) {
    _enable = enable;
}
bool is_enabled() const { return _enabled; }

other control should be in each derived image handler.
and don't use 'isEnable', try '_' and lowercase words to define functions.


Why do we need Unquote? why not just use Assert.AreEqual?


dd


move to cl_3a_image_processor.h


Yes, as it has been discussed, Cassandra is not designed for traditional joins and queries as we have in a SQL database.
But from a business perspective, we have the business requeriment to query information.
So, after to catch up/learning with modelling techniques, the client should be responsible of part of the query computation, so, yes, in this first approach, the alert info is stored in a main table and we use aux tables to match possible combinations of clauses conditions.
I've got some areas to improvements, as I need to update the alerts table with a triggerId partition column, but to not over-architecture in this first approach I think it will be good to merge it, prepare performance tests that can give us solid information about what are best steps.
In Cassandra, there are common best practices/patterns/anti-patterns, but after 2 weeks reading, at the end, everything is a tradeoff between your model, the logic that you put in your client and the logic that are required in the Cassandra backend to work.
I think we have space for improvements, but before to "optimize" I am going to prepare perf tests to analyze and get better measures and useful info about what has sense and where to optimize (at the end it's a balance between the client+modelling+backend, not only one of the layers).


Please refactor into submethods.


I suggest "Media Session" which together with the subtitle "Living Standard" will read nicely.


Do you mean defining this function as :
```c++
void StreamProperties::getPropertiesAsVector( PropertyVector& propertyVector )
```
?


Travis has recently updated its VMs, and default python version is 3.2.3. Python 2.7 is still available, but we need to inform CMake of what python version we want:
```
find_package(PythonLibs 2.7)
```


LS for Living Standard


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


I don't think Level is needed.


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


https://github.com/whatwg/dom/blob/master/dom.bs doesn't have this, probably not needed.


https://github.com/whatwg/dom/blob/master/dom.bs has "No Editor: true"


we only need a simple change. don't make it too complex.
The only thing to add here is that

if (!is_enabled ()) {
    output = input;
   return XCAM_RETURN_NO_ERROR;
}


bb


Yes, we can create a new story when we'll have some experience on it?


gg


You should not watch `ngDisabled` directly. `ng-disabled` sets `disabled` for you (either `disabled="disabled"` => true ; or nothing => false). It is intended for the users not for the directive implementation. The user can use `ng-disabled` (`<fooBar ng-disabled="ctrl.myBooleanVariable">`) or directly `disabled` (`<fooBar disabled>`).

A better way (taken from https://github.com/tkrotoff/ui-select/blob/master/src/select.js#L220) is to simply watch `disabled` attribute:
```JavaScript
attrs.$observe('disabled', function() {
  scope.disabled = attrs.disabled ? true : false;
});
```

See also my remarks on #2619: ng-checked, ng-disabled, ng-src... they all work the same way.


Huh? Is the default autocomplete 'on' for type 'password'?

I'd expect this in core.


It really wasn't obvious to me what `request.header("Accept", "text/html")` would be doing. After checking the source code I could see that the second value is actually a default in case the header is not found.
I humbly suggest a rename to headerOrElse or headerWithDefault ...


@sshumake generally we try to avoid overriding style attribute in jade, right ?
I think for now its ok..as its related to IE :)


I don't think it is a good thing to return an estimated duration value to the user without specifying that it is not the exact value.
Perhaps we should implement an new method _getEstimatedDuration()_, which would compute this estimated value from the file and other streams, don't you think ?


Missing javadoc


This is redundant now, SF.net bugs are gone.


@winterbe Very interesting, thanks for the link!


Yes this is a problem.
2 solutions:
1 - Add new function _InputStream::getEstimateDuration_
2 - _StreamProperties::getDuration_ throws an exception if duration is unknown (0). _InputStream::getDuration_ catches it, and computes an estimate duration based on file size and others stream properties.
Some advice?


I guess you need to rebase. This diff is misleading.


This was to resolve a bug I introduced... also to fix my git


fix the translation of payment follow-up.


Should be  $ssh_target (with a 't'), right?


This commit will likely give me troubles.
The problem? Here's some data I've observed on my 7750 while hashing grøstl-myr at LI768 with device replication:

  Number of queues
  Total hashrate (last)
  Total hashrate (avg)

  1
  8300-9100
  same

  2
  ~12k
  7600

Of course it would have been best for me to keep the biggest number, too bad it probably does not reflect reality. I expect users to complain for hashrate reduction while in practice the processing is mostly unaffected.

In practice, I even seen some performance reduction in using multiple queue, which is quite surprising. I'm pretty sure summing up the instant values didn't make any sense.