"It is doing partial application". I'm not sure, but for some reason I'm not comfortable calling that a partial application - as in the way this term is usually used. Yes, I agree I a way it can be seen as a partial application (like you do not need to pass in the collection object again as you pointed out), but I'd still rather not use "partial application" here just because it usually has a different meaning.


Would this be better as a .startswith('<xml') ? A html response would likely have the < in it too?


Indeed, I'm just too tiresome to do it Meh :)
Btw, as you probably know, I'm getting a lot of SLUB failure kernel panic
on temasek cm-12.1 build for i9300.

This is actually my first experience with SLUB failure on all of my Android
development period.

Is it because of that 80+ commits I've done locally on temasek kernel repo,
or PM, L2 changes made on ArchiDroid repo?


you really put the f in the fun of coding. spaces added, i don't see a reason to roundtrip through std::string for those comparisons.


oo


Ok, disregard my comment on previous commit.


maybe it's better to phrase it like "...we know as object oriented (programming?) with higher-order functions only".


här fanns det mer än vad titeln lovade! granskad.


КурсовойПроект/project.txt

Посмотри, какие есть тулы для генерации кейсов с помощью этой техники. Помогают ли они решить часть перечисленных проблем?


Exactly, and I guess that's a general point of discussion among git users. At the moment I tend to the "only add what everybody will find useful" side of the argument, as it shouldn't be a big problem to modify the .gitignore to match your specific setup.
I actually recently found a pretty cool service to do that: [gitignore.io](https://www.gitignore.io)

Admittedly, the overhead is not too much.


I just made it exactly match the original code from MARS, the commit that actially fixes the problem by just one line is this one:
https://github.com/rock-simulation/phobos/commit/d1b851e1451a71f70958b1593e391a763d935938


Leaking bio and x509_str here


Вопрос про Pairwise Testing:
1. Почему проводится тестирование именно пар значений?


Yay, that was my suspicion. ;)


granskad


March 2013?  So you've checked out Github before :)


PHPCS: Expected 1 newline after opening brace; 2 found


PHPCS: Expected 1 newline at end of file; 0 found


Class DatabaseController does not have a Docblock comment.


Method DatabaseController::registerMenus does not have a Docblock comment.


Method DatabaseController::init does not have a Docblock comment.


haha not to worry! Not just yours ;) gitflow seems a bit weird when you first try to use it, but it really does pay off in the long run. I know this isn't "code" and it won't be "deployed", but sticking to best practice doesn't hurt. We gain huge benifit in our ability to make clear releases, hotfixes, and still continue development for the next release cycle, by using the gitflow branching and merging model.


深入共享元素 Transitions 底层


Yes, I am a fan of continuing supporting both convention and configuration and I considered this - we would then have to always ignore a file in folder if it's called "properties.json" - but that's fine I guess.


Thanks for your interest in our API client : )

FYI: I pushed a commit that allows giving the `os` parameter an UUID. `get_OS_UUID` would then figure out that it was given and UUID and not raise an exception on invalid OS.

https://github.com/UpCloudLtd/upcloud-python-api/commit/6d1918326783f41eb8f18df8b07fb57351497490

Now that I think about it, perhaps the best way would be to call the parameter `template` as the uuid points to a template which is not necessarily an OS template (strictly speaking).

ps. `uuid` is an attribute returned by the API and is different from the template's uuid that the storage was cloned from.


ok


I guess this is easier than having the backend send its data in a different format.


Forgot to comment...... -.-'
added "logga ut" to menu


Sorry but no sense in having a (rather lose) coding style guide if it isn't followed. I'm doing this to everyone ;-)
Thanks for the adjustments.

Concerning `strcasecmp` I actually wouldn't even have noticed if you wouldn't have used it for the `.mka` comparison. But I don't have a strong opinion on it. But it's actually not going through `std::string` but the other way around i.e. `std::string`'s are handled as `char*`.


should be register?


granskad!
![](http://i.imgur.com/etZlNw4.jpg)


And in case that sounded passive-aggressive, I meant that sincerely. Having only the front-end worryng about the representation would be better in general, but out of this ticket's scope.


Super nitpick, but, could you maybe join these two assertions into one so that we also know that there are exactly 2 entries in the "services" map in the body?  Or add an assertion that the count of entries in "services" is 2?


Ah, yeah, the count++ should be in the /* tag still exist, update it */ branch. The goal is to

* in the first pass count the elements left in the tree
* in the second pass count the newly added elements

and limit the total number of elements in the tree.


Why did you remove the IRoQQueueManager interface in favor of the IRoQQueueManagement interface, which does the same thing?
Also, the license is missing from the new file.


I thought about it but I wasn't as comfortable working with enums as static
int varibles, if you want to switch it up go ahead. And since the output
from addItem is an int it seemed easier.

On Tue, Apr 21, 2015 at 1:08 AM, Austin <notifications@github.com> wrote:

> I like this idea, but why not use enumeration? Something like:
>
> enum cateogry {beverage, appetizer, soup /* .. */};
>
> might be more readable and precise.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ECarpenter/tripping-octo-boo/commit/946c4c180e42204647e29785965685473871ff1e#commitcomment-10819250>
> .
>


No hello?


Looks like this broke Timeout:

```
        Testing betty
==> /usr/local/Cellar/betty/0.1.7/bin/betty speech on
Betty: Speech ON
Error: betty: failed
undefined class/module Timeout::
/usr/local/Library/Homebrew/utils/fork.rb:36:in `load'
/usr/local/Library/Homebrew/utils/fork.rb:36:in `block in safe_fork'
/usr/local/Library/Homebrew/utils.rb:265:in `ignore_interrupts'
/usr/local/Library/Homebrew/utils/fork.rb:24:in `safe_fork'
/usr/local/Library/Homebrew/cmd/test.rb:37:in `block in test'
/usr/local/Library/Homebrew/cmd/test.rb:10:in `each'
/usr/local/Library/Homebrew/cmd/test.rb:10:in `test'
/usr/local/Library/brew.rb:135:in `<main>'
```
http://bot.brew.sh/job/Homebrew%20Pull%20Requests/24550/version=yosemite/testReport/junit/brew-test-bot/yosemite/test_betty/


This reordering of imports should be reversed, see https://github.com/spring-projects/spring-framework/wiki/Spring-Framework-Code-Style#import-statements


@fritsch

done https://github.com/mk01/xbmc/commit/256e1a07c04209dc40ada0b2b0b319039bd3be19


I'm not really sure what you mean, the formatting of the label (including inserting the <br>s) is done in the front-end.


isn't it removed? my commit includes the removal of metadata.xml (which makes the testing profile directory empty) and i can't find a testing profile on master.


Looks like this release hasn’t been `git tag`ged?


Document不是解析Xml的么


Oh, I see:

> NOTICE: Skipped v1.8.0 due to problems with release tooling. See #1436 for details.


granskad
![](http://i.imgur.com/v06N8.gif)


Might be worth calling out here that we're compatible with only 4.x on the server, and either 4.x or 3.x on the agent side.  I don't think we're compatible with 3.x on the server side.


不完全是，根据具体场景有不同的定义，其实还是指一颗HTML的文档树

2015-04-21 16:35 GMT+08:00, lzw19940512 <notifications@github.com>:
> Document不是解析Xml的么
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Glacier759/Duplication-of-work/commit/2c70860a619047ab1721937db1ad88d85cc64cdd#commitcomment-10819706


Please read and review this again for latest update


I thought the idea was to remove the AFNetworking dependency completely. Did I miss something?


.htaccess jg diapus bos, orang server nanti nangis servernya dimasukin maling


I guess I should not comment on front-end stuff anymore :grimacing:. For some reason, I was under impression that the backend would send the header labels and they would be used as such. But that probably only stands for their text at most.


pinged you on irc about it!
```
i can revert it, but would like to find proper.
is there a way to tell willie (which uses pyenchant) to wrap its binaries with
what is defined in pyenchant?
```


The variable name `SCHACH_HOME_ORGANIZATION_HEADER_NAME` starts with "schach" which should be "schac". Perhaps useful to refactor to prevent confusion?


Weird, it works fine on local. Will do more dig.


with extension blocks upper size limit is 32k
what is relevant for us is that mxc_hdmi exports edid to /sys as 512 byte string, which is a 1:1 copy from monitors EDID info - if shorter, then padded wth x00 to size of 512.


Vi kan ändra det manuellt tycker jag.


@ta2-1 I would put here some example to let managers know about possible values, like in https://github.com/ta2-1/pootle/blob/3c0c706bd54850df0f4175f3096689a9ec82166e/pootle/settings/40-apps.conf#L35-L42


Sorry for this, I think whole file should be deleted


self.Table.Cell(*originCellCoordinates)


ok will do, but this actually is no parsing, this is only conversion from mxc_hdmi export format to binary array.
mxc_hdmi exports the info as human readable 0xXX one-byte blocks delimited with _space_ like this:

```
0x00 0xFF 0xFF 0xFF 0xFF 0xFF 0xFF 0x00 0x4C 0x2D 0x7A 0x0A 0x00 0x00 0x00 0x00
0x2E 0x16 0x01 0x03 0x80 0x30 0x1B 0x78 0x0A 0xEE 0x91 0xA3 0x54 0x4C 0x99 0x26
...
...
```


На практике коммитеть их хочется, но вот чтобы они попали в продакшен - нет. Т.е. вычистить их надо перед пул-реквестом в корневую ветку. Чтобы не забыть это сделать, можно помечать такие строчки комментарием вида
```java
System.out.println("style after 2 = " + style); // TODO delete me before pull request
```


@MichaelBarz is this import required?


Вообще никаких комментариев быть не должно. Тем более автосгенеренных.


> While I can understand the rationale, the hard-coded limit seem pretty
> arbitrary to me, did you have any metric for choosing it?
>
Nothing scientific - just tried with 5000 elements and the freezes were too
annoying when typing. Then tried with 2500 elements and typing was
completely fine (the complexity isn't linear so halving the number brought
more than double performance). But I have a fast machine so maybe even
something like 1000 elements could be used.

I think the tree is getting decreasingly useful with an increasing number
of elements - scrolling the tree to find the symbol will get slower than
using simple editor's "find". So I think limiting the tree size for files
with many tags (which will be most often something generated rather than
real code) won't impact the usability of the tree much as it isn't usable
already ;-).

> Also, note that the symbol reparse timeout setting will have an impact on
> the usability vs. speed.
>
Not completely - it just reduces the frequency of freezes but the length of
freezes will be the same and while having a 1s freeze every 10s is better
than having it every second, it's still bad.


So sneaky! Thanks for bumping and testing.


bgr and img should point to the same data format. OpenCV defaults to BGR its a bit confusing. Basically just mimic this:

```
@property
def bgr(self):
    return self.img
```


Yay. Was going to ask you about future support for iron-doc-viewer. This should help us hit parity with the 0.5.x series of the seed element.


printf();


This is a global permission check and should be an og_user_access.


I think `waitForResourceTree` would be a better name.


PHPCS: Expected 1 newline at end of file; 0 found


I think `_waitForResourceTree` would be a better name.


you might want to print a log message here. What do you think, it might be useful fopr operators


How about `_flushCachedEvents`?


Please delete allocated resources here.


Looks good. One minor adjustment is needed.


Тоже в шоке. Вот тут http://www.javapractices.com/topic/TopicAction.do?Id=23 есть ответ. Ну точнее гипотеза
>  final fields, parameters, and local variables cannot change their value once set

Чтобы это проверить, допиши еще одно присваивание сразу после первого
```java
dptNumber = header.getInt();
dptNumber = 0;
```
Собирается ?


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


According to the [wiki](http://mpv.io/manual/master/#environment-variables-xdg-config-home) of mpv.io, mpv now uses ~/.config/mpv/ as the primary config directory.


This is actually not quite right: it doesn't work for preinstalled compilers switchs which are not `system`.

I believe something like this would be more correct:
```diff
--- a/src/client/opamState.ml
+++ b/src/client/opamState.ml
@@ -706,7 +706,7 @@ let rec resolve_variable t ?opam:opam_arg local_variables v =
     | "switch"        -> string (OpamSwitch.to_string t.switch)
     | "jobs"          -> int (jobs t)
     | "ocaml-native"  ->
-      if t.compiler = OpamCompiler.system then
+      if OpamFile.Comp.preinstalled (compiler_comp t t.compiler) then
         bool (Lazy.force OpamSystem.ocaml_native_available)
       else
         bool (OpamFilename.exists
```
(and of course, the same should be done for `ocaml-native-dynlink` and `ocaml-native-tools`).

I would have done a PR but:
- I might have missed something
- I didn't know if I should apply the patch on master, 1.3 or both.
I can still submit one if you want though.


printf("coucou");


No, you're just emplacing here. The problem happens when the uninitialized memory is cast to T and assigned a moved T.


Тогда, наверное, лучше даже `FIXME` вместо `TODO`, чтобы удобнее было их находить и отсеивать, т.к. `TODO` у нас в коде используется и так, а вот `FIXME` нигде вроде нет.


I suppose we should fail or fallback to another signer here.


This commit introduced again some errors fixed by precedent commits! Please rollback!


Same here.


I don't think it's a good idea to setup the process only once, before (possibly multiple) tests. I also don't understand where is the speedup here, it seems to me that you are just moving the code from the test body to a before hook. Can you please explain?


Actually, leaving it as unset is better because we use the default value provided by the SDK itself and depending on the region and service type.


이거 그냥 자바 run 하면 돌아감 흐후.


> You might also simplify this by simply cutting in get_tag_list()?
>
I was a little worried I might damage some logic in the tree
creation/update by it but after looking at the code again it should work
and is probably better to do.

> Which BTW could probably be improved (removing the useless tag->type &
> tag_types check as it's always passed tm_tag_max_t, or maybe even doing a
> mean memdup() on the array and sort and use this instead of a list) --
> yet improving might require profiling to really gain anything, and it might
> be worthless.
>
I don't do optimizations without profiling - the bottleneck is _always_
somewhere else than I think :-). And this is just invisible in the profile.

By the way, from time to time I find G_LIKELY and friends in the code and I
kind of can't stand those guys - I have never seen any performance
improvements using these (I can imagine it could happen in some tightmost
loop where you are just adding integers or something similar) and IMO  just
add noise to the code. Would it be OK to remove these (when I have time for
that)?


I love polyserve. A little on the fence about us dropping the fallback note for SimpleHTTPServer, but we can add that advice back if we find folks run into issues at all.


This reintroduces again error fixed by 5035d77655e6f4166cac194b2a3c8f394169d3ff


Typo!


Manifest에서 최소 최대 버전 설정함.
나는 기회주의자라 허니콤 이상 버전만 상대함 후흐


Typo error!


Good catch, I've accidentally reverted https://github.com/c9/core/commit/dddeef97ae10a1fdb3d0ade35323cd7c13e8d3ed, only changes to updateCore are required in this commit.


Reintroduces bug fixed in f241e5ee64a8feafa02de5193d3c9645fbf38d9a!


Can this be camel case instead of underscore


I think I'm confused now. To clearly understand what used to work (and is getting restored) and what this patch is meant to prevent from working, can you provide example config(s) with before/after behavior?


@fritsch

better ? https://github.com/mk01/xbmc/commit/9f649819373469c8ed69cbc78f00bec897b4f383


nitpicking: want to use `utils.internal` in this file as well?


this change makes it incompatible with everything below.


OK, I success to reproduce the problem. This test is failed because the test is timeout(probably because `betty` is a TTS program and cannot be run in the headless environment.)

And I forget to `require "timeout"` in `cmd/test.rb`, so when the exception was propagated back it cannot be restored. Fixed in https://github.com/Homebrew/homebrew/commit/b1f532acfc0259469d99d5c3e4fbed7c3a9616c6


@xhaggi with removing steps > 10min, do you mean to simply not enable them by default or get rid of them entirely? Personally I prefer the way it is now, also because I don't get automatic repeated button presses via CEC and I really don't want to press skip forward 30 times by hand in order to jump a larger portion.
Regarding replacing 15s with 10s - well, doesn't make much sense when we also have 7s as step size. Right now we sort of double the time with each step size which IMO is fine. If we would like a 10s step then we would have to get rid of 7s and 15s so that we'd end up with 10,30,60,... - but the smallstepback users will be bitching then because their beloved 7s are not there anymore. 10, 30, 60 would be fine with me though.


Thx - fine with me. Needs testing though.

2015-04-21 11:01 GMT+02:00 Matus Kral <notifications@github.com>:

> @fritsch <https://github.com/fritsch>
>
> better ? 9f64981
> <https://github.com/mk01/xbmc/commit/9f649819373469c8ed69cbc78f00bec897b4f383>
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/mk01/xbmc/commit/31e16d105c32f12afd86c01f876c0237ff1caaad#commitcomment-10820097>
> .
>

--
                   Key-ID:     0x1A995A9B
                   keyserver: pgp.mit.edu
==============================================================
Fingerprint: 4606 DA19 EC2E 9A0B 0157  C81B DA07 CF63 1A99 5A9B


cool

kleine Anmerkung: die Klassen (hier Testmodel) sollten auf ViewModel enden (z.B.: TestViewModel) und die Branch das nächste mal bitte feature_#issueNr (z.b.: feature_#18) nennen

ansonsten top


Is there any reason you hardcoded some px in here ?


serve index directory listing
```bash
edev share common/trunk-one
```


это забыла убрать, не нужно вообще


Hey Emil,

We explain that later on :smile: Mostly, JS gets module wrapping, templates get transformed into JS functions before that, and "miscellanea" are basically static assets that just get copied verbatim.


Reintroduces bug previously fixed in 52cde80648690c8740971948cb74eb14b4f05b68!


I guess you committed the backup file accidentally?


Do this once not every time the property is accessed


I see your point. This whole post is based on the oversimplification of the different types of functions. Given that: saying that a constructor is another example of "partial application" is also an oversimplification. Like saying a method is a function. I'll try to express the idea in a way that is not that "black or white".


Also an accidental addition?


The same here?


(368c887) The title of the commit doesn't respect guideline ("type(scope): title (refs #<number>[, ...])")


Guess what? ;-)


:+1: It could just be me, but I was unable to find the `-l` and `-p` short-flags mentioned in the wct README. Might be worth calling them out there too :)


Just testing


Small typo: ragister -> register


Oops. I was looking at an outdated commit. Sorry.


Because you have scrolled down to accesses multi edit icon on the browse page


`import assert from 'power-assert';` will work as expected if you replace espowerify with
[twada/babel-plugin-espower](https://github.com/twada/babel-plugin-espower). Thanks!


Does it fit 80 column limit?


manque le point.


Any idea why this commit does not have a clean diff, but rather claims to replace the full content? To see what has changed, it would be much better to have a correct diff.


try last commit, but note that in console default output uses inspect, which will double-quote the string and still include backslashes to escape the double-quotes. This is remove all backslashes. Could you grab raw json received on this server? I sent json to http://requestb.in/19ayymq1 and it parsed properly, may be pb not in backslashes. What kind of service is icq.affiliationfocus.com?


I know it seems a bit petty, but can we use the JS convention of leaving a space between function and ()?


Oops. Thanks, I hadn't notice that the example for `krige.conv()` was missing a couple of lines.
That's fixed. Now it works fine.


+1 for the trust_server class! Nice way of doing it. I would put a comment here though, explaining how it is set.


Looks good!


Same again, function (). My OCD again.


the .z file actually is not syncing since the symbolic file get overwrote everytime.


I think this change should be reverted from windows phone 8.0 silverlight, as now the application is redirecting to /<fbappid> instead of the page set in facebookconfig.xml and crashes.

1.0.2 -version works as expected.


Could we please have spaces around the operator?


Indeed, there's actually no permission called ``create group content``


the .z file actually is not syncing since the symbolic file get overwrote everytime. @lra


cex = 0.7 her


Can we have spaces after commas in argument lists, please? It's all JSLint stuff.


Better use C++11 initializers instead of a default constructor:
```
size_t super_period_us = 0;
size_t us = 0;
```


Can we have spaces around ===, please?


Same again, spaces between function and (), please.


idea: inside the pause button nandun yung *quit to main menu *resume etc. :)


Frederick perhaps knows more, but I think there was a test case that used empty arrays in a sequence computation. I think it was more a test of correctness with an edge case, rather than something someone would actually want to write.  Either way, it's not there anymore, so we can probably get rid of it. I don't see any particular reason to have it as part of the prelude.


Can you either pass the value of _GET['d'] in to this function as a param or access it via $this->request->getParam('d') so there is no reliance on global variables in here


_upd -> _update, _insuff -> _insufficient


Oh! Sure, well spotted. :+1:


Please remove this bit


Your error path leaves the mutex locked.  Best to use
```go
clusterAliasMapMutex.Lock()
defer clusterAliasMapMutex.Unlock()
```


@yairwein we have none of those things. They were edited in the renaming, I didn't check what they do, just that the app is stable.


By currentMembers, do you mean rendered members? If so, renderedMembers would make more sense as currentMembers implies the entire membership of the site.


A message is in lines 1719-1721. Or you mean something else?


Maybe add it in .gitignore so that it doesn't get added in the future by mistake again.


Thanks =)


@lechat these components have different names in Python3 from Python2. Six.moves is intended to make the demo compatible with both versions of python.


+ [Gregory Pakosz] on `commands_dirs` line ?

otherwise :+1:


Field should be readonly, not disabled! Try updating the admin user for example, this is now impossible because the name isn't posted.

Should probably be changed at other `if ($ro)` statements as well.


I would go with 1.0.0 to reflect the changes better.


I pushed this fix last night. @WD-93 can you check that it does the correct thing?


Thanks!


Alright, thanks!


This one is better: https://github.com/appium/appium-es7-jshint


Apart from comments in code, on the screen the title is **FormularyDrugs** and the title of the drug type column is Type_ID. These will need to be changed to be more human friendly titles. Ask me if you need me to show you how.


No problem. Out of curiosity: Is there a runtime benefit when doing that?


There's a lot of whitespace in the textarea, not sure where it comes from? Can probably be trimmed in Form_Textarea.


Hopefully I didn't screw up anything -_-


Can't actually delete a user.


.div1と.div4にfloat: left;とmargin: 0px;は不要。
.div2と.div3のmarginは不要。


div2とdiv3を包むようにclearfixを設定してください。


Same as in usermanager, groups aren't deleted when I try to.


Please, revert this commit, it cause the paned all the time to be minimized, even if
I change the size, every time I switch album, and not matter which content has the album.


全体を包括するdivを指定し、要素全てを中央寄せしましょう！


Можно и так. Плюс TODO в том, среда находит их сама и может вывести в отдельном окошке большим списком. С FIXME тоже самое ? Если нет, то всеравно можно. Просто я за интеграцию с IDE. В общем кому как удобнее, главное, чтобы в конечном пулреквесте не видеть этого, а в промежуточных было бы понятно, что на эту строчку не надо обращать внимание. Т.е. если перемнная используется только в логировании, то рядом с ней тоже надо писать этот коммент.


Yes....I used the ESH formatter that replaced tab by spaces... Do you prefer to commit/squash again in order to see only the actual changes?


Merge


Debug code?


The onyl changes are the range check for setBrightness and setSat (of course the JavaDoc has been changed accordingly)


:bow:


I don't think so. It is just a matter of style (more concise and less bug prone).


You may want to re-dump without the option set that crams it all on one line and makes it a pain to edit again later. :P


I was trying to be funny!!


The hanging indents should be 4 spaces (looks like 6 here).


Better indentation, moved "this.modifiedEntries.Remove (key);" next to equivalent "this.addedEntries.Remove(key);"


@juanjmerono I think I've finished reviewing this PR now :)


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


I realize I'm a bit inconsistent in this file; but let's stick to it for now: would be good to add a blank line before this `case`.


Hanging indent (see above).


This results in
==> default: Error: Invalid parameter onlyif at /tmp/vagrant-puppet/manifests-893fcda21077f5a583c2dbbba8515c0a/nodes/Apache.pp:217 on node capacity4more.eu.vm


Да, насколько я знаю, эклипс автоматически добавляет в `task list` как туду, так и фиксми, а также еще несколько подобных отметок.


I realized that '<' might be better. With '.', you can't enter a floating point number beginning with '.'. You have to enter a space or 0 first, and it takes some training to remember.  But, choosing something different than '<' is a good idea. I can't remember what I tried and what problems cropped up... maybe '>' or '=' would work.

I stopped editing when the code first worked. For instance,

* I did not do anything about colors. Previously, I used blue for the prompt.

* In the older hacked code, you could pull up Julia input in the history, then enter '.' and edit the code as SJulia input. I'm sure this is possible in the new implementation, I just did not look at it yet.

* In a few of minutes of playing, I got some REPL error that printed about a thousand lines of stuff in the error message. If I see it again, I'll report it somewhere. The hacked version was 100% stable over a few hundred hours of use, except for this: https://github.com/jlapeyre/SJulia/issues/5 .

* Entering unicode with latex codes, like in normal Julia mode, is not possible now in SJulia. It was in the hacked version (except for the aforementioned bug).


Missing space after `gint`. It would also be good to surround the expression that `*` applies to in parenthesizes like so:
```C
*((gint *) (ctx->insn->pc + ci->detail->arm.operands[1].mem.disp))
```


@temasek thank you for your help. :) Btw does clear all button dissappear for you?


Second argument should align with the first (note: this does not apply to forward-declarations).


LGTM


I would have put "Lenitent" at the beginning or end in the identifiers, sounds better to me. Otherwise fine.


Thank you for your PR. Can you please update Readme? When you do so I'll merge it :)


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


fetch resources from escenic backend
```bash
escenic [cmd] -c rnd/5.7
```


Uck, these "special idents" are indeed troublesome, I've removed some (like the `pinned` version number) but there's still some legacy.

It _should_ work in that case though, because the __switch__ will be named with an alias but the __compiler__ will still be `system`. I do prefer your version though: having the real compiler version directly and checking for system status would be more sound than having a special virtual compiler version and needing to check to get the real one.


oh, what readded? was it deleted once?


What do you think about: "We could make those two `for` loops that iterate through the `eligible_driver_names` more generic by doing this:" ?


Ok je vais le faire. Mais juste pour ma culture personnelle, pourquoi on doit faire ça ?

      De : benTrust <notifications@github.com>
 À : GIorfindel/MasterMind <MasterMind@noreply.github.com>
Cc : ChristGood <christ.good@yahoo.fr>
 Envoyé le : Mardi 21 avril 2015 9h41
 Objet : Re: [MasterMind] MAJ personnaliserNiveau (94c62af)

Avant de faire la méthode personnaliserNiveau(...), il faut rajouter une nouvelle classe niveauPersonel qui hérite de niveau. ET la tu peut appeleé le constructeur de niveauPersonel—
Reply to this email directly or view it on GitHub.


Dear Cooper

Would you please send the C++ code of Jonker-Volgenant algorithm-LAPJV to me? Cause the original website is shut down. My email address is haiyang.liu921@gmail.com

Thank you very much.

Haiyang


OK


TeamCity xBIM :: XbimExchange :: Build [Build 244](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1463&buildTypeId=XBIM_XbimExchange_Build) is now running


create


TeamCity xBIM :: XbimExchange :: Build [Build 244](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1463&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:32


I'm getting:

```
Case mismatch on class name 'App\Modules\...\ProductFilterControlFactory', correct name is 'app\modules\...\ProductFilterControlFactory'.
```

What is this actually for?

So now I shall rename `app\modules` directory to `App\Modules`?


Forward declarations should looks like the ones above this one.


great --- ping me when final ... so we can add it on .org also - what do you think!?


Added error constants.


What do you think about disallowing the [0].[0] syntax? It's not valid in any C-esque languages I can think of (JavaScript, C/C++, Java) and it just complicates things to have two ways of handling arrays.


We only introduce another block scope when we need to declare local variables.


Second argument not aligned.


Added native send and receive functions.


I've noticed that too late, but there would be a problem with using json-path-assert 2.0.0 and Wiremock (which AFAIR internally is compiled against version 0.9.x). Nevertheless I don't know if anybody uses that feature, so we can just wait for the first report.


Documentation changes reviewed and appear to be OK.


Granskat


Already spotted that, fixed in SVN 5572


I have a feeling it may be included in GLEW, not sure though.


rebase & force push. I won't tell anyone! 😶


Align the operands of an expression in an assignment spanning multiple lines.


I think it should be `%/tmp` not `%/tmp/`

FYI, on OSX there is the `$TMPDIR` env variable, it is a per user temporary files location.

I suppose the main difference between using `$HOME + %/.red/` vs `$TMPDIR + %.red/`
is that the latter get cleaned periodically.

reference:
http://superuser.com/questions/581347/where-is-the-temp-folder-in-os-x


https://developer.mozilla.org/en-US/docs/Web/API/Event/originalTarget
Better add a check for that
    `var data = (event.originalEvent ? event.originalEvent : event).clipboardData.getData('text/plain');`


I am following this tool too, this topic interests me.


> It should work in that case though, because the switch will be named with an alias but the compiler will still be `system`.

I don't think so, not in my case that is.
I didn't create the switch using `opam switch foo -A system`, I have a bunch of compilers installed on my system (but only one in my `PATH` at any given time) and I've a local opam repo with compiler descriptions for each of these.
The description looks like this:

```
opam-version: "1"
name: "4.02.1+foo"
version: "4.02.1"
packages: ["base-unix" "base-bigarray" "base-threads"]
env: [
  [CAML_LD_LIBRARY_PATH = "%{lib}%/stublibs:/path/to/ocam/build/lib/ocaml/stublibs"]
]
preinstalled: true
```

Will the compiler still be `system` in that case?


shouldn't styles be in the repo that uses the lib? (So web-app and online platform)?
If you want a different style in both apps, you might mess up the styles in the other app without noticing


This section is never appended to the form.


Granskad.


yep, agreed. but this is just aligning the close button, which for some reason the author messed up. So you'll want this everywhere


Granskad


This is not a Lisp file, but assembly language.


Fixed in 0c170db


please add some description here and again raise the pull request.


Tested on Windows:

 > path.normalize('C:/path/to/a.txt').replace(/[\/\\][^\/\\]+?$/, '')
    'C:\\path\\to'


Changing this order wasn't intentional but a result from importing the package from NixOS/nixpkgs.


May want to say "...identification of the participant as a harasser to other ZA Developers members **and/or** the general public.".


This is done for all fonts mapped by `map_font`.


Granskat


const char *dlm = ";" This coursed path.protocol failed. "hdfs://" will be stripd to "hdfs".


WHY DO YOU DELETE THE SPACE


sorry, typo wrong, coursed => caused


Great, glad to hear it!

I was actually worried they'd get annoying while mining, like when you try to drop stuff and it sucks it back up :)


@da-anda I meant removing it entirely from the list, but we can use 10s,30s,60s,3m,5m,10m,(30m disabled by default) for both forward and backward. any objections?


Right, I need to remove the extra /.

Yes, we just want to avoid the useless cleaning process.


So is this a Android bug, or just our usage was incorrect? Could you leave comment here https://code.google.com/p/android/issues/detail?id=75000 ? Thanks!


It's boost::algorithm::contains you're using right? I can't find anything on std::contains... If that's the case then maybe it would be better to add the right includes and namespaces here?


function() inte function ()


Regarding [previous discussion](https://github.com/tkomstadius/bruse/pull/36#discussion_r28761922), `return false` is maybe used to prevent the default action for the `event` to happen? (Which might be redundant, since we run `event.preventDefault()`.)


Can we overall switch from `NSError` to a particular error class, like `DatabaseError`, that implements `ErrorType`?


comma?


Can we log loudly, too? This is a programming error.


This seems a lot like part of desktop's `mozIAsyncStatement` (in /storage/). Might be worth a skim of that code to see if there are any lessons to learn.


No need for the `else`.


I just noticed this line... It already worked for Debian. Not sure why you added this, but it would have failed in an obvious way (pkg_repo undefined) without this line if someone tried to enable ```postgres:use_upstream_repo```.


Doubles also pass `obj` as `Int`…


(y)


Why? We haven't been explicitly closing cursors in most places. We use ARC for closing connections and statements; I don't see why we couldn't do the same here.


Care to elaborate?


Clever way of preventing duplicate tag listings.


We could make it check 'os_family' for 'Debian' so that it catches all of them.


got log-style here, log_style in replayer()


Use test2.txt as the input file.


If you're totally confident that we have strong control of scope and when deinit happens…

It seems to me that the shorter the lifespan of a cursor, the fewer perf (and other?) bugs we'll have, so I naturally tend towards having belt and braces: yes, automatically close, but encourage ourselves to close-when-done.


Dead code, remove this


ello


done


@pletcher We were trying to prevent this (a few days after creating a new product):

![NH](https://assembly.firesize.com/https://d1015h9unskp4y.cloudfront.net/attachments/0fb978c3-6a19-4048-b80e-fd9379de0695/IMG_1743.jpg)


Can this actually just be expressed as
```haskell
cohesion :: Boid -> [Boid] -> Vector
cohesion self =
   let p = position self
   in . centre - p
```


You're reading and writing a non-atomic member from two threads. You need to ensure that the member is only read from one queue, so use a mutable local to do that.

https://mikeash.com/pyblog/friday-qa-2015-02-06-locks-thread-safety-and-swift.html , about half way down, gives a good example.


The `let` can also be eliminated, I just thought it looked nicer. In the other ones it's actually necessary to have the `let` so that the position isn't evaluated every time (although GHC may be smart enough to know this, IDK).


@janardanyri linking to our iOS app. Not trying to send you to this specific commit, but not sure how to create a comment on the overall app.


This function was pulled off the prototype as an attempt to make more stuff 'private', but the `fastdom` passed to it should always be a singleton.


prasad third


Concordo com Eduardo, é necessário checar ser o usuário está na página indicada.


Si encuentran algún error, no duden en corregirlo.
Pd: espero que no les moleste como molesto con la forma de escribir.
Pd2: el pasado me persigue https://github.com/keze87/Zombie-Wars/issues/7


Lift magic constant.


"I really like the idea of adding articles, but I think it needs more than just the models before I can make a decision. After all, Rails is all about MVC working together, not just the M portion in isolation. If the views and controller are in working order, then I'll pull this into my project."


I really like the idea of adding articles, but I think it needs more than just the models before I can make a decision. After all, Rails is all about MVC working together, not just the M portion in isolation. If the views and controller are in working order, then I'll pull this into my project.


I hear you (and was aware of why we made this change), just making sure we aren't throwing any babies out with the bathwater (so to speak).


License block.


Typo


In case when more than one projects are accessible by account, why do we return just the first one?


El makefile solo sirve para especificar las dependencias (si usan codeblocks no hace falta)


哪里需要保护没看出来…


i don't understand this code but if we do the same transformation everywhere then it should work right?

\W is non-alphanumeric characters and underscore iirc


Changed to this
echo text(round(abs($to_time - $from_time) / 60,0). ' '. xl('minutes'));


Márcio, na definição do "And" tu só está definindo 2 parâmetros e, pelo que estou vendo, no início na função tu espera espera receber 5. Outra coisa é que o parâmetro "Name" deve ser passado na definição dos cenários, nos arquivos .feature.


D'oh! that insidious comma ...


Removed in 3 places.


done


Could this have a good default instead of requiring entry?


A função não está verificando se o membro está lá, apenas se o usuários está na página.


@ranjitjhala you forgot this begguing


`id` must be unique. Since you're in a loop, it doesn't look like it will be unique.


no, the code from your first comment wont work. GHC will think you're trying to apply `centre` as the first argument to `(-)`, and it'll freak out.


Yeah, I think its pretty idiomatic to rely on `let` and `where` when it makes your code more obviously correct.


Hi. Did it work?


Oooh, okay I get it. I figured it probably wouldn't parse quite right.


`.nin` 是什么魔法？


Are we adding amo.xchg to PARC now?


Nitpicking, but should that colon be a period?


Noted and included in the newest commit. Thanks for the tip.


1) The change to src/theory/fp/theory_fp_rewriter.cpp is a bit ugly. I'd prefer to just ask for the kind twice to avoid the macro:
Assert((node.getKind() == kind::FLOATINGPOINT_MIN) || (node.getKind() == kind::FLOATINGPOINT_MAX));

You can also use CVC4_UNUSED.

2) Please delay the change to src/theory/arith/linear_equality.h . What that code does with d_end is a bit stupidly complicated. Let me take a pass on rewriting that class.

Other than that it looks good.


removed and changed


Found BROKEN by full.
j, getattr(obj, part)
AttributeError: 'module' object has no attribute ''

==============================
21. function.py
Ran for 0.001159 seconds
In python, function.py failed:
==============================
returncode:  1
<function test_jacobians at 0x2abc9c53d9b0>
<function test_hessians at 0x2abc9c53da28>

Traceback (most recent call last):
  File "function.py", line 1153, in <module>
    unittest.main()
  File "/usr/lib/python2.7/unittest/main.py", line 94, in __init__
    self.parseArgs(argv)
  File "/usr/lib/python2.7/unittest/main.py", line 149, in parseArgs
    self.createTests()
  File "/usr/lib/python2.7/unittest/main.py", line 158, in createTests
    self.module)
  File "/usr/lib/python2.7/unittest/loader.py", line 130, in loadTestsFromNames
    suites = [self.loadTestsFromName(name, module) for name in names]
  File "/usr/lib/python2.7/unittest/loader.py", line 100, in loadTestsFromName
    parent, obj = obj, getattr(obj, part)
AttributeError: 'module' object has no attribute ''

==============================
22. lpsolver.py
Ran for 0.001033 seconds
In python, lpsolver.py failed:
==============================
returncode:  1
[('qp', {'qp_solver': 'ooqp'}, False), ('qp', {'qp_solver_options': {'nlp_solver': 'ipopt'}, 'qp_solver': 'nlp'}, False), ('qp.nlp.ipopt', {}, False), ('qp.ooqp', {}, False)]

Traceback (most recent call last):
  File "lpsolver.py", line 357, in <module>
    unittest.main()
  File "/usr/lib/python2.7/unittest/main.py", line 94, in __init__
    self.parseArgs(argv)
  File "/usr/lib/python2.7/unittest/main.py", line 149, in parseArgs
    self.createTests()
  File "/usr/lib/python2.7/unittest/main.py", line 158, in createTests
    self.module)
  File "/usr/lib/python2.7/unittest/loader.py", line 130, in loadTestsFromNames
    suites = [self.loadTestsFromName(name, module) for name in names]
  File "/usr/lib/python2.7/unittest/loader.py", line 100, in loadTestsFromName
    parent, obj = obj, getattr(obj, part)
AttributeError: 'module' object has no attribute ''

==============================
23. ad.py
Ran for 0.001059 seconds
In python, ad.py failed:
==============================
returncode:  1
<function test_MX at 0x2af9abb16b18>

Traceback (most recent call last):
  File "ad.py", line 865, in <module>
    unittest.main()
  File "/usr/lib/python2.7/unittest/main.py", line 94, in __init__
    self.parseArgs(argv)
  File "/usr/lib/python2.7/unittest/main.py", line 149, in parseArgs
    self.createTests()
  File "/usr/lib/python2.7/unittest/main.py", line 158, in createTests
    self.module)
  File "/usr/lib/python2.7/unittest/loader.py", line 130, in loadTestsFromNames
    suites = [self.loadTestsFromName(name, module) for name in names]
  File "/usr/lib/python2.7/unittest/loader.py", line 100, in loadTestsFromName
    parent, obj = obj, getattr(obj, part)
AttributeError: 'module' object has no attribute ''

==============================
24. socpsolver.py
Ran for 0.000988 seconds
In python, socpsolver.py failed:
==============================
returncode:  1
[('sdp', {'sdp_solver': 'dsdp', 'verbose': True, 'sdp_solver_options': {'verbose': True}}, False), ('sdp.dsdp', {'sdp_solver_options': {'verbose': True}}, False)]

Traceback (most recent call last):
  File "socpsolver.py", line 248, in <module>
    unittest.main()
  File "/usr/lib/python2.7/unittest/main.py", line 94, in __init__
    self.parseArgs(argv)
  File "/usr/lib/python2.7/unittest/main.py", line 149, in parseArgs
    self.createTests()
  File "/usr/lib/python2.7/unittest/main.py", line 158, in createTests
    self.module)
  File "/usr/lib/python2.7/unittest/loader.py", line 130, in loadTestsFromNames
    suites = [self.loadTestsFromName(name, module) for name in names]
  File "/usr/lib/python2.7/unittest/loader.py", line 100, in loadTestsFromName
    parent, obj = obj, getattr(obj, part)
AttributeError: 'module' object has no attribute ''

==============================
25. sx.py
Ran for 0.00077 seconds
In python, sx.py failed:
==============================
returncode:  1
<function test_large_hessian at 0x2b1f12c37848>
Not available isSmooth, skipping unittests

Traceback (most recent call last):
  File "sx.py", line 1314, in <module>
    unittest.main()
  File "/usr/lib/python2.7/unittest/main.py", line 94, in __init__
    self.parseArgs(argv)
  File "/usr/lib/python2.7/unittest/main.py", line 149, in parseArgs
    self.createTests()
  File "/usr/lib/python2.7/unittest/main.py", line 158, in createTests
    self.module)
  File "/usr/lib/python2.7/unittest/loader.py", line 130, in loadTestsFromNames
    suites = [self.loadTestsFromName(name, module) for name in names]
  File "/usr/lib/python2.7/unittest/loader.py", line 100, in loadTestsFromName
    parent, obj = obj, getattr(obj, part)
AttributeError: 'module' object has no attribute ''

==============================
26. mx_jacobian_sparsity.py
Ran for 0.001146 seconds
Ran for 43.8835401535 seconds
Ran 26 tests, 25 fails.
make: *** [unittests_py] Error 1

```


These should be separated by blank lines, which makes them into separate paragraphs.


You guys. vcs-version.sh has been working pretty well on OSX vs Unix :)


You spelled color wrong! xP


Nice, yes I am familiar with rems, I guess I haven't been burned by `em`s yet so I hadn't made the switch to using them by default. Thanks for sharing the article!


To reset these for existing users of the daily or preview you may also want to change the name of the options subtly. If you do this you should also be sure to sync with any accesses of the pref from UserSettings on the C++ side.


Something to think about: instead of rewriting all the mask channels, you could change the Accumulators to support multiple mask channels. This might be more performant since you would avoid reconstructing all these mask blocks


Should the fd be open in RD/RW ?

Also this should be `O_RSYNC` ? since we want to ensure we are reading fresh ?


Good, just a couple of typos.


Because you want to always give a project if one is available. This is to handle the case where your default project was deleted. But you have access to other projects. This allows you to delete your default and only use another one. Eventually we will add the ability to set default to another.


Yes... I'm using "unsigned int" (on Arduino Uno R3). For example, when I write the eeprom with EEPROM.put(512, 65532) it overwrites position 514. I think that is because numbers higher than 32767 (with lower numbers than this I could use "int") the program use 4 bytes... and that happens because I'm not declaring the variable (numbers) first (I'm declaring it later for reading), so the program doesn't understand that numbers between 32767 and 65535 could be "unsigned int" and maybe it uses "long" (4 bytes) as default.... could it be? I solved the problem declaring first the variable with the number. Thank you!!


"I really like the idea of adding articles, but I think it needs more than just the models before I can make a decision. After all, Rails is all about MVC working together, not just the M portion in isolation. If the views and controller are in working order, then I'll pull this into my project."


This TODO can be removed now


For `newlen > hislen` the clearing should start at `hisidx[type]+1` and end at `hisidx[type]+(newlen-hislen)`.
Old code:
```c
i = hisidx[type]+1;
for (; i <= newlen - (hislen - hisidx[type]); ++i)
  clear_hist_entry(&temp[i]);
```
`hisidx[type]+1` does not need to be equal to `hislen`.


good suggestion. That would be way more semantic.


Esse ".add" faz parecer que apenas insere o texto e que não clica no botão de login.


I see just three files:

env.tex, eq.tex, zero.tex.

Should I not see many more?  I have 28 files in adoc/examples.

> On Apr 22, 2015, at 7:56 AM, Jakub Jirutka <notifications@github.com> wrote:
>
> What do you see here <https://github.com/asciidoctor/asciidoctor-latex/tree/master/test/examples/tex>?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/asciidoctor/asciidoctor-latex/commit/f6e44ef5dd27475bb49d8b0482834f4dfba5d604#commitcomment-10836962>.
>


A linha deveria ficar da seguinte forma: And(~'^I select the "([^"]*)" option at research group list page$') { String option ->


Did QA uncover problems with this?


I use "Sip". Purdy nice OSX tool and I like it more than a Chrome plugin I used to use called "ColorZilla"


I like the whole `@NonNull` thing that got started here. One could call the other but only if the Locale argument was left without `@NonNull` *or* defaulted to `Locale.getDefault()`-- three non-ideal situations, any opinions on one vs the others?


Haha, I thought about googling and figuring out how to get to the 100% solution but it's probably just a big time suck and I don't really care if the whole world can see my sass-cache.


I don't have this problem. Are you selecting the region with the mouse? If so then it could be cause by `mouse-drag-copy-region` being `t`.


"Nome" deve ser passado nos arquivos .feature


@robsongajunior is this "." supposed to be here?


`mCurrentMSeconds` should have type `std::chrono::milliseconds` and be renamed to not have the type in its name, perhaps `mCurrent`.


JSRocksHQ.github.io -> jsrockshq.github.io
Also please replace `&` with `&`


`std::chrono::steady_clock::time_point` would be a slightly more concise way to spell/specify this type.


Correct, so how do you explain: 240 runs, **199 assertions**, 0 failures, 0 errors, 41 skips (assertions basically means examples in this case, this will be fixed in next version)? You have only 5 output examples for latex in the repository and yet your log says that you have 199 of them…


Acho que o Then poderia incluir esta ação também. Ficaria da seguinte forma: Then(~'^I can fill the fields "([^"]*)" and "([^"]*)" with the values "([^"]*)" and "([^"]*)"$') { String field1, String name1, String field2, String name2 ->


INSERT INTO warden_checks VALUES ('783', '243', '', '', '6356849', '1', '74688B87100100008B', 'Unk');

INSERT INTO warden_checks VALUES ('784', '243', '', '', '8153504', '4', '558BEC56', 'Unk');

INSERT INTO warden_checks VALUES ('785', '243', '', '', 6211360', '8', '558BEC83EC0C8B450C53', 'Unk');

INSERT INTO warden_checks VALUES ('786', '243', '', '', '4847069', '6', '741868EF000000E8', 'Unk');

INSERT INTO warden_checks VALUES ('787', '243', '', '', '6380789', '6', 'F8260F8754010000FF', 'Unk');

INSERT INTO warden_checks VALUES ('788', '243', '', '', '8151657', '4', '488C11C17507C7', 'Unk');

INSERT INTO warden_checks VALUES ('789', '243', '', '', '8145237', '3', '8B4908894A085E
', 'Unk');

INSERT INTO warden_checks VALUES ('790', '243', '', '', '6501616', '4', 'FCDF8000DFE0F6C4417B', 'Unk');

INSERT INTO warden_checks VALUES ('791', '243', '', '', '6511674', '2', 'FCDF8000DFE0F6', 'Unk');

INSERT INTO warden_checks VALUES ('792', '243', '', '', '6356837', '5', '7474568BCFE8316CFFFF84', 'Unk');


Hey guys? The carrot block and apple block use the missing texture when you hold them in your inventory, but when you place them down, they show the correct texture. Does anyone know what I missed?


"Descrição" deve ser passado nos arquivos .feature


When the type of `mCurrentMSeconds` changes to `milliseconds`, you will need to drop the `.count()` here.


@nehaljwani : This URI is currently not accessible, and hence you get `403 Forbidden` errors when the clients are trying to access the readboard.


Would make sense to handle other common errors from urllib2


`mPreviousMSeconds` should have type `std::chrono::milliseconds` and be renamed to not have the type in its name, perhaps `mPrevious`.


Rock! Don't forget the [patches on master](https://github.com/kripken/misc-js-benchmarks/commits/master) too.


This expression will just work *as is* when both `mCurrentMSeconds` and `mPreviousMSeconds` are changed to type `milliseconds`.


falta agrega en el 2° strtok del FOR los delimitares "\n\t,."


I wonder why no unit test picked this up?


What do you mean by everywhere? This is the only place I'm aware of that needs this type of escaping. \W seems to not cause any problems, so I've updated #904 to use it.


`MAX_TIME_COLNAME` instead? Might be more appropriate.


Yes, perfectly and easily — thanks!

On 22 April 2015 at 10:20, Zeke Sikelianos <notifications@github.com> wrote:

> Hi. Did it work?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/callumflack/bellflower/commit/d57681ec3aa2f9445485fb5f028257e99559e14b#commitcomment-10837295>
> .
>


less than 3 minutes old rotates


This would be one nanosecond right?

How did you do that? :)


I am both impressed and repulsed by your suggestion.  It avoid some code repetition and ensures that the file is always closed, but allowing `return nil` to return something other than `nil` horrifies me.  And I think named return values remind me too much of Basic.


Como esse Then executa uma ação igual aos dos próximos 3 ANDs, acho válido colocar tudo na mesma ação. Dessa forma:  Then(~'^I can fill the fields "([^"])", "([^"])", "([^"])" and "([^"])" with the values "([^"])", "([^"])", "([^"])" and "([^"])"$') { String field1, String name1, String field2, String name2, String field3, String name3, String field4, String name4 ->


@UniversalMediaServer/developers just making sure you guys all see this http://www.universalmediaserver.com/forum/viewtopic.php?f=16&t=3329 :)


"Twitter" deveria ser trocado por "([^"]*)".


Seems like you hadn't tested that previously ;)


We should give `PROPOSE_INTERVAL` and `PROPOSE_FRESHNESS` type `std::chrono::seconds const` in LedgerTiming.h.  When VS allows (gcc and clang already do) we should make these `constexpr`.


@paltman Your comment makes sense now. Maybe we can refactor the comment on this line to be <yourprojectname>.wsgi:application to be more clear?


Concordo com Luís, deve-se colocar o "$" no final da definição da ação.


Também não achei o  "selectMember"


I thought it was already in PARC? It's in the ISA spec (https://github.com/cornell-brg/maven-docs/blob/master/parc-isa.txt#L1081,L1118) and in our apps (https://github.com/cornell-brg/maven-app-misc/blob/master/bfs/bfs-mt.cc#L42).


`std::chrono::steady_clock::time_point  mTime;`


`std::chrono::steady_clock::time_point cutoff`


Não está verificando se o erro apareceu.


Changed the title of the pull request. This issue came up because I was/am
using the histogram to test the random distribution, which sometimes was so
low as to round to zero.

On Tue, Apr 21, 2015, 5:44 PM Craig Tiller <notifications@github.com> wrote:

> This would be one nanosecond right?
>
> How did you do that? :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vjpai/grpc/commit/6db722453fe10179c7a6cebfb6fe027caaae8381#commitcomment-10837538>
> .
>


`std::chrono::steady_clock::time_point getCreateTime ()`.  Loose the `const`, it doesn't buy us anything.


FacilityTabGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2173 of 2196 tests passed)

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


done


Makes sense! Going to wait for Travis to be safe (which means probably AM tomorrow because Travis is still recovering from it's issues)


I debated about that and thought it was cleaner to have it all in the same location. It is an odd piece to be sure. I even debated where the lambda function code should live. If we have a bunch more lambda functions then it makes sense to have it here, but if it is just this one-off, I am not sure it makes sense to have even that here. I am happy to discuss restructuring.

The one technical note is that the lambda function must exist in order to remove any stacks that rely on the function. In this case that seems a bit silly, however it is a cloudformation limitation that makes sense for other use cases. All this means is that wherever the function is created from needs to be stable. I think it makes sense to have it with the stacks as the stacks are the only things relying on it. I simply extended that logic to the roll as well.

I will admit however that it seems a bit odd to have an entire stack that basically creates one role.

At the end of the day, I do not really care where it lives as long as it is stable, obvious and well documented.


The patch is taken from https://www.drupal.org/node/1244072#comment-7556267


I don't really understand your concern. Injecting implicit behavior on `return` is exactly what defer is supposed to do. If for some reason your precious sensibilities forbid this pattern, there are plenty of alternatives that preserve the error while deferring; for example you can defer file.Close() and return file.Close() in the same function. The second return (from the defer) will return an ignored error.


pass null, don't overengineer. nonnull is an advisory annotation that only serves a purpose in your ide for a circumstance that doesn't apply to this case. it's ok to pass null if you're expecting null.


This code is looking hideous, but will tidy up after rendering is certain


@unstoppablecarl: Can you please increment this package version and publish it fix to npm? I just spent a half hour finding and fixing this issue, only to learn that you fixed it already. :)


At least we should call `unique` here -- see https://github.com/hadley/devtools/pull/769/files#diff-f458be81102f7b666fa185cdf846f8bbL27


    $trow = sqlQuery("select roomnumber , origappt , pid " .
      "from patient_tracker where id =? LIMIT 1",array($_GET['record_id']);

This is what I put in place will it work?


falta passar por parâmetro e salvar as variáveis da própria classe.


Thinking back, I think this data was to map fxo parameters to xna effect parameters. Maybe related to the data at the end of xna effects files too?


結合してしまうとテストに失敗した時にどこで落ちたかと言う情報が抜け落ちてしまうので、少なくともここはやめた方がいいと思います。


あとは `Throws` は使ってもいいんじゃないかな。


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (339 of 340 tests passed)

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


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Hmmm ... ok -- I guess I just didn't realize we were using it ...


That's because old code didn't reorder entries when newlen > hislen, but new code does.


I think you can get it back down to 4 LOC:
```php
<?php class λ{private$r=[];function __construct($a=null){global$argv;$this->args=$a===null?array_splice($argv,1):$a;}
function cfg($k,$v=null){$c=&$this->$k;if($v===null)return is_callable($c)?($c=$c($this)):$c;$c=$v;return$this;}function
add($r,$c){$this->r[]=['r'=>explode(' ',$r),'c'=>$c];}function run(){foreach($this->r as$e)$m=[];foreach($e['r']as$r)
foreach($this->args as$a)if(preg_match("#$r#",$a,$h))foreach($h as$k=>$v)!is_int($k)&&$m[$k]=$v;$m&&$e['c']($this,$m);}}
```


ok sounds good. I just meant if there was some name mangling to convert task id's into something that can be use to identify elements. but if it works let's do it!


You could get around this by making default_options and dirty_options into HashWithIndifferentAccesses instead of Hashes


Right... merged already however and probably not worth an extra push right now.


changing container dimensions is necessary to change graph width. To be consistend, the graph height is now also changed when toggling


This looks like a bug to me.


I wonder if we can un-intrinsify it, now that we have inline + reified, and make it a normal stdlib function?


@nerzhul: Or use `is_local_server` or equivalent...


Take a look now.  I need to review my .gitignore -- it was blocking inclusion of *.tex files in the test directory.


Interesting, seems like we could use this. @davidlehn, thoughts?


Done


I would make it more obvious that this only happens if the file doesn't exist when the hook is executed.


Care should be taken to not depend on two containers (eg: The addon under test depends on simple but the test declares a dependency on cdi on test scope).


I wonder if this should just be `rpm_ostree_db_query_all()` and get rid of the `query` argument until we figure out how that will actually work.

We can always add `rpm_ostree_db_query()` later.  Best not to lock ourselves into a placeholder.


You can only put `if` constraints on template functions, not on normal functions.


number = number % arabic seems obvious now that you point it out. :)

Thanks for the tip on divmod, I'm checking it out in irb.


fghfgh


done


There are two issues with this:
If I move print functions to utils.h it will mean that utils.h should include PositionState.h, which itself includes "utils.h", so we get convoluted header file. Is it ok? Also should I add additional utils.cpp file for these?
Second, if I move these functions out of class, then I should add Bitboard get_white_pieces() and similar methods in PositionState class or make print functions friend functions. Please let me know how should I proceed.


This was @k-joseph's initial commit like 1.5yrs ago, they are set correctly in a follow up commit, see 5e645fc95bf1a7d2031665c56af1c3b05e9f29d8


Yeah, you `.gitignore` is total mess, I’ve gave up trying to read it…

Well, now I have to remind [step 2](https://github.com/asciidoctor/asciidoctor-latex/commit/f6e44ef5dd27475bb49d8b0482834f4dfba5d604#commitcomment-10836137) – check if all the generated output is corrected (not just syntactically, but also semantically) and fix incorrect ones.


@javierpena - looks like master-patches was not rebased so I'll push this as a cherry-pick to there to unblock packstack --allinone


         sqlStatement("UPDATE openemr_postcalendar_events SET " .
            "pc_apptstatus = '$status' " .
            "where pc_pid =? AND pc_eventdate =? AND pc_startTime =? ", array($patient_id,$today,$appttime));

This is what I used


ghjghjghjghj


ghjghjghj


    $row = sqlQuery("select fname, lname " .
     "from patient_data where pid =? limit 1" , array($patient_id));
This is what I used


i think jdiff might be confused here -- many of these existed in 3.0.   see: http://google.github.io/guice/api-docs/3.0/javadoc/com/google/inject/spi/ElementVisitor.html


github organization*


What is the purpose of this example? This is already covered in block_table from the reference examples.
Beside that it’s invalid ’cause it doesn’t have DocTest header. It’s actually a bug that it generated it anyway, I should add some check for missing header.


@orangejenny I had to add these to make [mug messages](https://github.com/dimagi/Vellum/pull/373) display correctly since you [switched to absolute positioning for `fd-edit-controls` and the edit button](https://github.com/dimagi/Vellum/pull/376).

If I'm reading this [SO explanation of `clear: both;`](http://stackoverflow.com/a/12871734/10840) correctly, this should not work. `clear: both;` should only affect floated elements, not absolutely positioned ones. However, both Chrome and Firefox render messages correctly under a widget with and without edit button; I'm not sure why.

I believe the correct thing to do would be to change the styling to use floated elements rather than absolute positioning. I'm open to other suggestions as well if you have any.


No había visto que le habias puesto el cometnario arriba aclarando, mala mia :P


@javierpena feel free to rebase master-patches when ready, this cherry-pick was just a quickfix


GroundDomainUpdate (sushobhit) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (617 of 617 tests passed)

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


Only 17 additions, but holy cow, the headache and testing it took to get it to work....


:laughing:


deprecated?

https://developer.mozilla.org/zh-CN/docs/Web/API/KeyboardEvent/initKeyboardEvent


Thanks very much for this fix!


exit(0); abruptly shut down / terminate the whole program??????  If misused I get it but wow,  return 0 is good for a failure. an exit inside a function inside of windows is not friendly way to say goodbye.


I believe the call to removeAttribute is missing the attribute name. Otherwise, this looks awesome.


Not used removed


Obrigado! Funcionou!


develop (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (340 of 340 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)


@emord we'll need to cherry-pick this commit whenever [itemset-connector-2](https://github.com/dimagi/Vellum/pull/365) and [copy-paste](https://github.com/dimagi/Vellum/pull/380) make it to master, or rather when the last of those two gets merged.

We'll also need to redo these merge conflict resolutions:
- [src/logic.js](https://github.com/dimagi/Vellum/commit/0416579da2ea560d4d54aaab3058ee516b4fd85e#diff-e8c10ecfceaf2d2a578b7321207927c9L151)
- [src/mugs.js](https://github.com/dimagi/Vellum/commit/0416579da2ea560d4d54aaab3058ee516b4fd85e#diff-fb1b9e010259d16ab20a8186e2d67b23R274)
- src/widgets.js (very simple)
- [tests/main.js](https://github.com/dimagi/Vellum/commit/0416579da2ea560d4d54aaab3058ee516b4fd85e#diff-494b0153451a4e8f44d4cca8787017fbR110)


"WHERE id =? AND date =?", array($record_id,$today));

used


This isn't going to work as the line it's now looking for is `test: "value"` but sysrc outputs `test: value` (at least on 9/10 systems)


dsad


done


Updated and amended commit per maintainer's comments.


? Begguing?

> On Apr 21, 2015, at 5:18 PM, Niki Vazou <notifications@github.com> wrote:
>
> @ranjitjhala you forgot this begguing
>
> —
> Reply to this email directly or view it on GitHub.
>


If I may also chime-in, I don't like the fix made to didyoumean.cpp

Is the issue is that the array is being initialized on the stack? The one-line change:
  int C[3][len2+1];      //cost
to:
  vector< vector<int> > C(3, vector<int>(len2+1));   //cost
should be enough if, for whatever reason, using variable-length arrays is an issue.


Glad to help!


this is only required for hdf5 right? there's no reference of numpy or numexpr versions in the rest of the file. I have the impression we are checking and rechecking versions everywhere ...


Thanks for the help! =)


Thanks Bill - this is a fantastic resource! We're working on starting up an R user group  at U Maryland and will probably use this setup going forward. Will let you know how things go and if we run into any trouble!


OH GOD THE PAIN! This is a terrible change.

The problem with this is that company pops up frequently and most of the time you can just ignore it, but now it completely breaks muscle memory for all of vim. In insert mode and want to go down 2 lines? Well if company happened to pop up you now need to press `ESC ESC` instead of `ESC`. At first I didn't even notice it was company mode I just thought escape was broken and finicky and it annoyed the shit out of me. The presence or absence of a company popup should never change how you use vim editing commands.

Before the behaviour was that escaping to normal state wouldn't exit the company popup but it would close whenever any other command was executed. This could be fixed by just exiting company *and* going to normal state on pressing `ESC`.


OK, I think I am beginning to get the picture and to see what I need to do.  One more question.  Where is the source for the reference examples?  I don't see it in the `test` directory, but do see it, for example, at

```
https://raw.githubusercontent.com/asciidoctor/asciidoctor-doctest/master/data/examples/asciidoc/block_admonition.adoc
```

When I view it as `raw`.

I will work on that terrible  `.gitignore` file


maybe wrap these parameters


really cool idea


Hmm, you're right. I'll remove these and double check the rest


rax retry tests


ping @eregon


Very good reasoning! Figured you had thought about it. And works for me just fine!


these requests functions should work


Yeah, I can't think of a way to cast it back and forth without going through the code again.
Let's keep it as is for now.


This is two changes.. the NULL fix and then intellij warnings


Oops.


While you're culling, it is safe to also remove bIndex and iIndex and use `0` for `iIndex`. Correct @joelkatz?


Sorry, I don't think I understand. `previous` should hold the attribute name to be removed. This handles the case where the bound `attribute` value changes and we need to remove the previous attribute as we add the new one.


It would be really great if you can squash this commit into the last one =)


LOL. My mind is somewhere else right now. Apologies. You are correct (no doubt). This looks great. Merging shortly.


Thnaks, Kyle.
With some modification, I've managed to build your code sucessfully.
I use Boost 1.54 and the `boost::range_difference` changed back to
`boost::range_size` in Helper.cpp as the boost doc
<http://www.boost.org/doc/libs/1_54_0/libs/range/doc/html/range/concepts/random_access_range.html>
indicated.

On Tue, Apr 21, 2015 at 2:40 AM, Kyle Kastner <notifications@github.com>
wrote:

> When I last used this, it seemed to depend on this specific version of
> boost. Also, specifying a particular version seemed useful to make sure
> everything will compile if boost updates again and changes its API. I don't
> use RNNLIB anymore though, so maybe the current version of boost is a
> better choice.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kastnerkyle/RNNLIB/commit/426a940bde0e890d340d765bfb69e2985b74ec76#commitcomment-10809405>
> .
>


No worries :-)


Não achei "def clickOnCreate(){}" na página "MemberCreatePage".


best just not to add more "Debt" we can add new "concepts" instead of colors and I think all will be well


What is that for? afaik nothing in artiq uses cython.


would prefer `yAxisDomain` instead of `yDomain`


Não achei "def fillMemberDetails(String value){}" na página "MemberCreatePage".


> Although ugly, actually, this is a reasonable workaround. I'll test this tonight and see if that works.

I tested this, and it seems to work great! Thanks @cgwalters for the idea.
See:
https://github.com/purpleidea/oh-my-vagrant/issues/78#issuecomment-94996537
https://github.com/purpleidea/oh-my-vagrant/commit/d8d1cb140f92fb16693a42d4728fe97800c11180

We still need the other fixes, so I'll submit a new patch in one sec...


nothing, you get multiple captures in the split. I kind of say that below by defining it as a collection of strings that contain the matched url segments...


I am a nice comment


Derp here


Seems travis is already getting the submodules correctly anyway: https://travis-ci.org/m-labs/artiq/builds/59432246#L1253


Really should be in a wrapper, otherwise people will start to heckle us about not being uniform and tell us to change back to the old way


This PR has been updated, but based on Colin's comments, I'm not sure if it's correct.
Perhaps someone more familiar with this build process can add the root/vagrant user/password combo, as well as the insecure key for root too.

Thanks,
James


As mentioned in IRC, it is a very nice approach to make your statechart the defaultResponder. This allows you to just define action strings on views, and this action will automatically end up on the state chart. The only requirement is that on the current state, you'll need a function named exactly like the action.

You can make your state chart the defaultResponder by doing the following:

```
  var sc = Sites.statechart;
  SC.RootResponder.responder.set('defaultResponder', sc);
  sc.initStatechart();
```


90946736_WSHP_#4567 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Hmm....it's possible squeeze it to 1-2 lines, bu this looks more clear for me. However, probably it can be solved with substrings only. Something like this:
*return [i for i in ["python", "perl", "csh", "sh", ""] if i in line][0]*

Where is problem with RHEL-5?


Yeah thats a nice idea; I'll whip something up and stick it in weave info (where I think we already stick a bunch of IPAM stuff)


Given the lack of other debugging tools for golang, I seem to find myself resorting to printf-debugging quite often.  So the potential for blocking in a `String()` function makes me nervous.  I'd be far more comfortable if you had a simpler `String()` function for `Allocator` (which might not say very much), or perhaps no `String()` function at all, and a separate function that does the work to provide more elaborate information (`Inspect` ?).


hva tycker ni? dekorerte gulp med $log og $config istedenfor å bruke _config og $log osv som vi har brukt i FE-prosjektet.


司库监管资金和采购，从制度上来看，是否合适呢？


Of course, you're right!


Found BROKEN by quick.
Here's the log file for step 'all_1':
```
[ 11%] Linting casadi_sdqpsolver_sdp
[ 11%] Built target lint_casadi_sdqpsolver_sdp
[ 22%] Linting casadi_linearsolver_lapackqr
[ 22%] Built target lint_casadi_linearsolver_lapackqr
[ 22%] Linting casadi_linearsolver_lapacklu
[ 22%] Built target lint_casadi_linearsolver_lapacklu
[ 22%] Linting casadi
function/external_function_internal.cpp:210:  Missing space after ,  [whitespace/comma] [3]
Category 'whitespace/comma' errors found: 1
Total errors found: 1
make[3]: *** [casadi/core/CMakeFiles/lint_casadi] Error 1
make[2]: *** [casadi/core/CMakeFiles/lint_casadi.dir/all] Error 2
make[1]: *** [casadi/CMakeFiles/lint.dir/rule] Error 2
make: *** [lint] Error 2

```


Før denne blir helt jævlig, så bør vi ta en runde på hvilket pattern vi skal ha! Har du noen tanker rundt det @kennethlynne ?


d


e


Hi Harold

I have download the LAPJV C++ code from the website you mentioned. Thank
you very much and I really appreciate your help.

I'm a PhD candidate in Harbin Institute of Technology, China. I'm working
on a Traveling Salesman Problem solution agorithm called COP that is designed
by Alexei Zverovitch. COP uses the LAPJV to slove the linear assignment
first. The C++ code of the COP can be found at
*http://www.cs.rhul.ac.uk/~zvero/thesis/cop_ref/
<http://www.cs.rhul.ac.uk/~zvero/thesis/cop_ref/> . *I'm using Visual
Studio 2012 to compile all the C++ code for the COP. If I can make all the
codes work in VC++, maybe I can send them to you so that you can share them
in the GitHub or elsewhere.

Keep in touch and hope to talk to you later.

Best regards

Haiyang

2015-04-21 23:51 GMT+08:00 harold cooper <notifications@github.com>:

> Haiyang 你好！
>
> I used the Internet Archive's "Wayback Machine" to find an old copy.
> Click the most recent date, "July 6th, 2007" here
> <https://web.archive.org/web/*/http://www.magiclogic.com/assignment/lap_cpp.zip>
>
> I hope that helps -Harold
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hrldcpr/Bezier.hs/commit/723ef8c0887656538edc33aea8da37fd3415027f#commitcomment-10828364>
> .
>


There is one good practice, which is that any view you are going to use on the mainPane is defined as a property on the mainPane. You have been defining views in /views/ using .extend(), which is correct. In order though to manage this properly, anything that will be required to popup in some way, or moved into a container, needs to be referred here. This allows SC.Page to instantiate your view in place, meaning that the link to the view class will be replaced by its instantiation as soon as it is used for the first time.


ups!


It is a bit unclear why this data is in here. Is it part of data which needs to be in the store? Because if it is, it should be defined in fixtures.


We don't want this to be an ng-click, because we lose the "Open in new tab" ability


It is good practice to better describe what happens in each state. Your generalState is better called MAIN, (capitalized on purpose to stand out).


I think this will mean cython iterprets every molecule with a symmetry number of 1 as having raised an exception?


I agree; I've attempted to dedupe as much as possible into doOperation, but we're hitting the problem of not having any polymorphism and these methods return different types.  If you'd prefer, I could make Claim return the IP, in which case these could be unified out of existence I think?


Can you try this on a buffer that has only a "." in it as the sole character?

The point is that `char-after` and `char-before` may return nil if invoked at resp beginning and end of buffer. Something bad may be happening in such case.

Also the `forall` test case is:

    xyz :: forall m a b . (Monad m) => a -> m b

the "." in above should NOT be a composition even though it has spaces around.


у тебя там точно частота нормальная? Xtal=168MHz, т.е. частота кварца, а PLL он может сконфигурить незаметно так, что проц будет работать на частоте 168/26MHz незаметно для тебя.


Going to haulingState is now leaving your main state. Is that what you want? I can imagine that ISHAULING might be better a substate of the main state.


@wkennington there are couple critical xio patches not ported to hammer, and they won't, i think.

see https://github.com/ceph/ceph/pull/4099


Let's use overloading. It can be done with the number of the parameters.


commit message should contain #9


yeah naturally, atm, idk where it is used, just tuning and trimming.


change width


Mathieu, let's keep the first commit message line short and then add detailed info on the next line for future commits.
https://gist.github.com/matthewhudson/1475276


What are you exactly trying to achieve with Sites.LAYOUT_LBL?


没啥问题, 你建PR吧


Actually.. shouldn't it be enough to do that for `@default_options`? If the validation doesn't pass (i.e. if the key is not already present in `@default_options`, the user-provided key will never be `to_sym`'ed, so we'd be safe – and this way, we make sure that the keys are always stored as symbols, which I think is good.


Fixing this thing -> ![screen shot 2015-04-22 at 12 10 06](https://cloud.githubusercontent.com/assets/132146/7272108/9f31ec80-e8e8-11e4-9f3a-42f046c6d9c6.png)


cc @viddo


why do you need nano in a container?


Yup its a bit tortuous.  We're trying to do two things:
- If the operation is taking a long time, the user might go away (on the other end of the http call).  In this case we want to remove the pending op and tear this all down, so we don't allocate an IP which will never be freed.
- We reuse this infrastructure to tear down and unblock any pending operations when the allocator terminates.

We're kinda hamstrung by go's cancelChan implementation; an alternative I'd like to consider is wrapping this in a function which does a non-blocking read from the socket (is have an hasBeenCancelled function) and then just calling that in the various operation's do function.  Then we can get rid of all this cancelChan stuff.


typo envOronment


and i doubt there is memory leak problem after i tried some test.(though i cannot locate where now)
Thanks for your work!


The beautifier indeed helps a lot, I would put the mixin reference on the previous line though.


Awesome!


NextInsideOptionWalker


This is actually not a clever idea. The SC datastore framework provides a generalized approach to binding to the primaryKey of a record, through the SC.Record#id computed property.

Better do:
```
  siteIdBinding: 'Sites.siteController.id'
```


LIKE


PHPCS: Expected 1 newline after opening brace; 2 found


Why aren't you using the existing m_settings member var for this?


PHPCS: Usage of ELSE IF is discouraged; use ELSEIF instead


PHPCS: Expected 1 newline at end of file; 0 found


Do we need helper_buffer to be a member of RecvInfo? I guess the whole content is constrant so it could be a global (of class) variable, couldn't it?


I wouldn't use function at all, just "loss expression". And I would explicitly distinguish the different cases:

Update functions take a loss expression or a list of gradient expressions and a list of parameters as input and return an ordered dictionary of updates: sgd(), momentum(), nesterov_momentum(), adagrad(), rmsprop(), adadelta()
Two functions can be used to further modify the updates to include momentum: apply_momentum(), apply_nesterov_momentum()
Finally, we provide a helper function to constrain the norm of a tensor variable: norm_constraint(). This can be used to constrain the norm of parameters (as an alternative to weight decay), or for a form of gradient clipping.


do you want to use docker in production? if so, better not use app.run()...
you could use gunicorn. it can be installed via pip too. see https://github.com/flurischt/dockerfiles/blob/ac0b405525126e8265672371386fa8b8cb9a47cb/images/image_resize_service/scripts/run.sh for an example


Librt should only be needed for the examples and the test code.  I'm pretty sure that we don't clock_gettime anywhere in the main library.  I noticed that it is in the cmake configurations for the tests, but not for the examples.  I'll fix this.


Don't do this: you are creating a memory leak here, which is never good, also not for debugging purposes. If you want to check on what the store returns, you can easily do  ``` Sites.visitsController.get('content')  ``` in the web console.


ostree/rpm-ostree approach to error handling is a bit different, should the function accept a GError **error and let the caller handle it?

The rest of the code use a gboolean to handle the return value, as:
```
   gboolean ret = FALSE;

...do stuff that can set error and "goto out"...

  ret = TRUE;
  out:
    return ret;
```


To have that covered, just write "scalar loss expression" where I suggested "loss expression".


I think you better set this on the collection view bound to this controller instead.


I may not understand how this is supposed to work, but it seems to me that this function returns the correct result only if it is called in the right order. Can this (and all other methods) be implemented so that they look at fixed offset in helper_buffer and use the value there instead of calling .getInt()? I.e. I don't see the reason why to use byte buffer instead of a plain array or class.


I'd still prefer giving this directly in the method docstrings. For example, nesterov momentum is not explained any more now, and the code doesn't give the formulas as clearly as my docstrings gave them (because of the `apply_momentum()` or `apply_nesterov_momentum()` indirection).


True :) I forgot to reset it after ecperimentibg with it s bit ^^


Should it be `'session-tags` or just plainly `session-tags`? If you quote then symbol gets pushed, if you do not quote then symbol value is pushed.

Which one should it be?


This is not a clever idea. The ObjectController is a very special controller, and should generally not be extended through create, except for bindings. The problem here is that whatever you will set (sectionIsEnabled for example) will actually leak onto the object held by the controller, which is generally not what you want.


应该再后面加上123


OK, great.


Better move this to the statechart.


Fixed at https://github.com/fpiot/arduino-ats/commit/f0bdd5c2677277ee01ffea90609ec349bba0f999. Thank's.


Fixed in 77f9ae83457450646bd5f7cddbfc399ceef67f43


by "There is not spoon" I mean "Merge"


@wyuenho , could you, please release latest version? latest release is 0.3.5 from January, 21, 2014.


Looking at [OpenBlocks Vacuum Hopper](https://github.com/OpenMods/OpenBlocks/blob/master/src/main/java/openblocks/common/tileentity/TileEntityVacuumHopper.java#L200-L219) We probably should check if the itemEntity we want to pick up is still alive.


I think  this rule is only exist for the report contact list that contains status icon. Sorry about the misleading rule


Found BROKEN by quick.
Here's the log file for step 'all_1':
```
[ 11%] Linting casadi_sdqpsolver_sdp
[ 11%] Built target lint_casadi_sdqpsolver_sdp
[ 11%] Linting casadi_linearsolver_lapacklu
[ 11%] Built target lint_casadi_linearsolver_lapacklu
[ 11%] Linting casadi
function/external_function_internal.cpp:210:  Missing space after ,  [whitespace/comma] [3]
Category 'whitespace/comma' errors found: 1
Total errors found: 1
make[3]: *** [casadi/core/CMakeFiles/lint_casadi] Error 1
make[2]: *** [casadi/core/CMakeFiles/lint_casadi.dir/all] Error 2
make[1]: *** [casadi/CMakeFiles/lint.dir/rule] Error 2
make: *** [lint] Error 2

```


awesome!


When creating this system; I tried to keep it simple to the end user. It appears more difficult to create a similar interface when it comes to the HXSL side, if anyone could give me some pointers that'd be great.


Erm… Now there is. But it haven't been pulled to the host?


Python 2.6, 3.3, and 3.4 must also be removed from `tox.ini`.


Just curious, are there any plans to move back to using `bswap` at some point? I'm asking because this revert made the ARM-specific rustc bug reappear that was addressed by https://github.com/BurntSushi/byteorder/pull/11. The `bswap` variant did not suffer from that issue.


siehe auch https://github.com/swissbib/sbvf2/commit/d8ef397dc8963f10d81f6b817b0a3c01ef9fb2ca


intéresserons


Ah, thanks for the note!  Should I be making any changes to my Processing versions?


Hi @pattacini, I noticed a small typo: "tocuh" instead of "touch" in function getTocuhInTargetTol.


Merci d'ajouter une directive sectionauthor avec votre nom


I revert this commit.


puavo-rest can modify entries but not create because id pool is in puavo-web.


If the composer.css will represent the compose view in the future, I would say it's reasonable, but I think the composer.css now is only represent the composer widget?


Imaginez


Merci de mettre un autre user que obo


La phrase garde en mémoire est ambiguë. Je pense que vous voulez dire que make vérifie les dates de dernière modification des fichiers et les compare à celles des binaires etc pour décider si il faut recopier


TeamCity xBIM :: XbimExchange :: Build [Build 257](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1480&buildTypeId=XBIM_XbimExchange_Build) is now running


Collision is still possible, if both nodes have the same syntax. But I guess that in that case collision probably doesn't matter.


Ce serait mieux de citer ces deux documents via des références explicites dans la bibliographie


TeamCity xBIM :: XbimExchange :: Build [Build 257](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1480&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:08


I think you meant "BAD_CIGAR" instead of "BAR_CIGAR"


Merci pour cette contribution utile. J'ai quelques commentaires de détails que vous devriez pouvoir fixer facilement.


Small convention thing, when you render do it using this syntax

```
render :edit
```


OK


This is better written as

```
redirect_to @task.user, notice: 'Task has been updated'
```

It's just shorter really


I did let it generate by eclipse and simplified it through the MUtil.equals method, since it did the same thing, just in one line except two.


The description of the Git repo should also be (manually) updated ("RunDeck" -> "Rundeck")!


When you are setting the flash for a render you should always use the flash.now syntax

```
flash.now[:alert] = "Task has not been updated."
```

http://guides.rubyonrails.org/action_controller_overview.html#the-flash


Maybe it would be better if we could remove it: https://github.com/mozilla-b2g/gaia/pull/29659


Can we download it in `update_dependencies.xml` instead?


We should always make sure that a patient has a name (in model validations) so no need for the .try


召集人：发布《启动公告》并成功指定第二报敌人，则成为召集人（兼第二保底人）。第二保底人笔误。


Information about how many available addresses are owned by the current node is in `weave status`, albeit cryptically.


The pattern of calling `String()` from `router.Status()` leads to this design; if we're going to fix it for `Allocator` then I'd recommend fixing it everywhere.  Currently `String()` is blocking while `string()` is not, so you have to remember to call the latter when printf-debugging.


This change should be applied in line 909 too


Can confirm this fixes our issues with default values per-company in a multi-company environment, with users that have access to multiple companies.


u


yy


y u so gud, pls tch me


Vou utilizar o DateAdd().


yes, i thought also a bit about that and is on my mind when doing a ganttchartinspired view.

for the task list view it is probably not so easy to make it consistent , because you have so much task properties to filter and 'order by'.

For the case u have no filter restrictions (no privates,closed,..) and a nonexistent "sort default by supertasks" this could be done by a css-padding or margin on each level of sub tasks.

With the adding of sub tasks all gets a bit harder, remember the currently for consistency disabled mass operations.


Interesting , I did not know that this is even possible. Has this always been there or is that a new Asciidoc(tor) feasture?


I'll insert them again.


Since it is quite a bit of effort, I am wondering whether we have the time (and will) to do it now. We could create a new issue for this and keep any related commits separate for now.


I made that change in response to the conversation below.  But I can make it a static global variable, if that suits better?

DH>>>    * The sops instantiation of timer_ops_t in timer.c is unnecessary. If you continue with using a timer_ops_t in the init function then the passed arg can be constant so sops can be a ptr (instead of a deep copy). The sops ptr (or separate get_timer_value func() var) can be used to assert whether the timer is initialized in udelay().

RH>> Would that work if someone called timer_init() with a const cast on a non-const ops variable?  Doesn't a const function parameter tell the compiler the parameter is read only, not that it's fixed in memory?  Or is my K&R knowledge too rusty?? :-O

RH>> A const global variable is allocated in fixed memory and therefore lives for the life of the program.  But I've seen quite a few hacked programs where an API takes a const so the programmer just casts their local variable; which often works in basic testing and fails much later.

RH>> So I guess I'm saying that I think it's safer to take a copy:  assuming we keep the timer_ops_t.

[...]

RH>> Would that work if someone called timer_init() with a const cast on a non-const ops variable?

DH> Yes. GCC puts const data is put in the .rodata section. We make use of this in lots of places in the TF. It's more efficient than taking copies at runtime and means we don't need a .data section. We'd already be hosed if this was not fixed in memory. Casting a non-const var to const is a bug.


Applied, it'll be in the next pull request.


:warning:  **REMOVED FROM HISTORY** turns out to be docker-compose issue :warning:


It looks like this warning is currently being triggered way too often, e.g. even when I use the default initializer, which is GlorotUniform.  That's because GlorotUniform wisely uses Uniform under the hood.

There must be a better way than to print a warning for everyone, even when they're not affected.


Applied, it'll be in the next pull request (along with the same for the line below)


这个操作是来自于git软件的，git pull是更新，git add是增加一个修改(-A参数可以增加'删除文件'的修改，如git add -A test\ git)，git commit是提交修改。
"修改檔案 => 加入 stage (git add) => 提交( git commit )=> 繼續修改其他檔案"
git push -u origin master是合并分支，git里的origin是干嘛的?
"origin 是默认的远程版本库名称
你可以在 .git/config 之中进行修改
事实上 git push origin master 的意思是 git push origin master:master （将本地的 master 分支推送至远端的 master 分支，如果没有就新建一个）"


## Test Markdown
### this is new to me
* how to do this
* Am I correct?


Applied, it'll be in the next pull request.
That'll teach me to copy/paste ;-)


Looks good to me code-wise.

But the correct English term for "not even" would be "odd".
Besides, that function is never used anywhere in that code anyway.


Une raison particulière a ce changement ?


0 means all the time?
10000 means every 10 second?


> Has this always been there or is that a new Asciidoc(tor) feasture?

I don't know :)

> We could create a new issue for this and keep any related commits separate for now.

Yes same here. I think the commit at hand can be integrated as that work has been done now anyways. And then let's update other existing listings under a new issue.


做为我的第一个Github库，这是一个测试


libcxx_ubuntu_14_04 (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (336 of 337 tests passed)

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


phpdoc needs to be adjusted


please check that we also bump version on iOS


nice


fixes #224


I was originally passing in the timer_ops_t and reworked it along the way.  I'll add that back in again, but as recommended, I'll keep the base address as a parameter saying as that's driver specific.


Pour que nous puissions bénéficier des modifications si quelqu'un d'autre décide de reprendre Assembl. C'est arrivé avec la plateforme projet. La vraie question était: pourquoi Apache au départ (et c'était pour des raisons contractuelles, heureusement le passage apache->agpl est permis.) Nous avons eu l'accord du CA pour ce changement.


0 means "as frequently as possible"
10000 means "not more frequent than every ten seconds"


Will look at this right away.


/cc @garuma @chkn


keep as is


ok


Yes, cc/@duemunk


The singleton pattern is not complete. You forgot the copy constructor and the assignment operator. Also there's no reason to use protected instead of private. Can you rewrite this to be complete and also use the reference and static local instance method as depicted here(which makes smarter use of C++ capabilities):
http://stackoverflow.com/a/1008289


asef


I think if I bring back your patch here then it should be settled? https://github.com/BurntSushi/byteorder/pull/11/files

Sorry about the regression (I wish we could have a test for this). I must have just reverted to an older version of the code?


This line converts RUBY_VERSION to a float. Since we wanted the minimum version to be 2.2.0, RUBY_VERSION.to_f returns 2.20. If it's < 2.2, ruby will throw an exception reading the version of Ruby needs to be higher.


An by that you mean 0.1?


test comment


素晴らしい！ :smile:


These 3 lines are fine to change.


keep as it was.


This line doesn't work because it sets in stone the version of Ruby that must be running, which hinders development on updated machines. I replaced it with code to check for a minimum version of Ruby instead.


del


よく思いつきましたね。ここ。


keep as it was


Whoops? Weird how this could compile with no problems.


Sure, re-applying the patch would do the trick as well. No worries, I will report back whenever it breaks again, at the risk of being annoying :)


完璧


素晴らしい！


リアルタイム


@SubJunk OK my bad. I will revert it and make new PR.


da


OK, applied and uploaded as `0.3.8`.

A human regression test is fine by me! :P


It is a Ruby convention to name such constants all caps. So rather `NUM_OF_BULK_POSTS` than `num_of_bulk_posts`. As a bonus you will get a warning if you try to reassign such a constant.

See https://github.com/rails/rails/blob/master/railties/lib/rails/backtrace_cleaner.rb#L5 and https://github.com/rails/rails/blob/master/railties/lib/rails/generators.rb#L26 for examples of defining constants in the Rails code base.


Yes gcc complains about, multi-byte character literals for the TriFile changes.  (They can't really be used in any sane way.  The C++ standard allows them, but says they have an implementation defined value, so you can't really rely on them or their byte order.)


method_missing (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (341 of 341 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)


trivial: mixed brace formatting


wtf?


Если ЧИСЛО равно нулю, вернуть 0. Как я понимаю, чтобы убедиться в том, что число равно нулю, надо проверить каждую его цифру, ну и если все они равны нулю, то само число тоже равно нулю. Ваш КО


`singe_post_title` and `single_post_body` are already strings, so there's no need for string interpolation here.


This only affects the OutfitStudio wxTextCtrls.  It works for me on Linux.  I'll try to test it on Windows in the next day or two.


hey, @maximilien.

Are you really want to remove this option? Currently I rely on using it, since there is no good way to use server name with file metadata service [[link to code](https://github.com/cloudfoundry/bosh-agent/blob/master/infrastructure/file_metadata_service.go#L45)].


Flag is a concept, but it's too generic as a name of a variable. Instead, describe what the variable represents, i.e. `distinct_post_created`.


So the var name `--stacking-system-message` is going to be applied to all `OverlayModeless` components?


Here, as well as in conditions above, the parens are not necessary.


No need for string interpolation.


That's probably it. Old version of Marathon that does not support force_pull_image.


anything wrong with 1.0.0 ?


This is a good use of string interpolation.


when is `global` defined, but `module` is not?


String interpolation not necessary here and in the line below.


`moving-column-expand` classes should also be added to columns in mercator.


Update


Aleksey, did you comment this parts of the xml yourself manually?


実ゲームでのテストを行って、「つい長押しして起動させてしまう」人が居るようなら、2654ffb への差し戻しを行います。


tu peux laisser certains messages de debug si tu veux en utilisant `logger.longdebug()`, ça peut être pratique par exemple pour débugger plus tard, e.g. pourquoi une clef n'est pas reconnue correctement. (mais comme tu veux)