@TheBigBear, thanks for the feed back! I did test this locally using salt 2015.5.6, both installation and removal, and can attest that it works as submitted. I also used this SLS to roll ossec to all of our windows systems in our data center, so it's been used in the real world.

However, your feedback is well received and I've made the changes requested. Some comments below:

1 & 2: We've had problems if the sls `name` of the package, not `full_name`, does not match exactly the output from `pkg.list_pkgs`. Specifically, salt will indicate that the package is not installed even when it is. We encountered that issue here and changed the name so the install would work, but we were running an older version of salt at the time.

After your comment, I went back and made the `full_name` match the `pkg.list_pkgs` output and changed the `name` attribute to `ossec-agent`, and it still seems to work. My take away here is that either the `full_name` or the `name` attribute has to match the output from `pkg.list_pkgs`, preferably `full_name`,  and whatever issue we saw in the past doesn't appear to be the case anymore.

3. Sure thing, I'll add quotes for the version string.

4.  Leading spaces for the /S removed!

5. We've previously had issues with the single backslash in various context, so we always double backslash as a convention. I'll remove it here since it's a bit more verbose.


It is completely undefined as is a TT access... Nevertheless it works.

Even if in the counter ends up to be filled with complete random garbage,
the worst it can happen is to have a spurious check time call, and counter
is reset to zero at the next tick.

I will try to write something with atomics, hoping (and testing)  it does
not regress, but in this case it is really overkill and IMO not needed.

Ogn Sunday, November 1, 2015, Joona Kiiski <notifications@github.com> wrote:

> @mcostalba:
>
> - Until a month ago we used volatile boolean flags to pass some flags
> between threads. For splitpoint shared variables we always used lock
> protection to guarantee the expected behaviour.
>
> - What we are trying to do here is completely unorthodox... Sure it works
> well with a spinlock or mutex, but without them the behaviour is completely
> unspecified.
>
> On Sun, Nov 1, 2015 at 10:01 AM, Marco Costalba <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> wrote:
>
> > @zamar <https://github.com/zamar> timeCounter is (mostly) a local thread
> > object, it is incremented just by one thread and only very occasionally
> it
> > is reset to zero by another thread. If we set it atomic, we force a sync
> > after every increment, completely giving up the advantage of a local
> > counter. Moreover until one month ago, we always used shared variable in
> > this way and it worked well (as it works well in C where there is no
> > atomic). Said that, we *could* use
> >
> > std::atomic_fetch_add(&timeCounter, 1, std::memory_order_relaxed)
> >
> > instead of
> >
> > ++timeCounter
> >
> > But it seems to me an (unnecessary) overkill.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/mcostalba/Stockfish/commit/edac860627b6706e1c0722812c72731dc02d463d#commitcomment-14113328
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/edac860627b6706e1c0722812c72731dc02d463d#commitcomment-14114758>
> .
>


There seems to be a disconnect about the underlying assumptions of these tests, and what unit testing is for. We should have a meeting to clarify


opauth/facebook

Facebook strategy for Opauth


Readded in https://github.com/andygrunwald/sortinghat/commit/668748c5bf471ee82e7e45832fd7e734cc2ca9c9


Ahoj,
šla by ta příloha na mail calvera.jarda@seznam.cz? Nějak ji na to github
nemůžu najít. Jsem tam ted poprvý :)

Díky

Jarda

---------- Původní zpráva ----------
Od: peter8205 <notifications@github.com>
Komu: feixm1/fastshare.host <fastshare.host@noreply.github.com>
Datum: 1. 11. 2015 16:10:08
Předmět: Re: [fastshare.host] Initial commit (a2f38f1)

"
Cau
v prilohe mas ten subor tak to vyskusaj.
A daj mi potom vediet ci ti to islo alebo nie.
Diki

> ** Sat, 31 Oct 2015 09:57:40 -0700 - "feixm1/fastshare.host" **

 >
> Ahoj, zkus to zkusím to :)

jinak na stahování ty hosty moc nejsou co? myslím český...

Jarda

---------- Původní zpráva ----------
Od: peter8205
Komu: feixm1/fastshare.host
Datum: 31. 10. 2015 17:56:20
Předmět: Re: [fastshare.host] Initial commit (a2f38f1)

"
ahoj
ja som urobil ten fastshare.host kompiloval som si to sam ale ,nefunguje.
Borec tvrdi ze to ide ale hotovy subor fastshare.host tam nechce dat.
Tak ze neviem , kto koho taha za nohu.
Ak chces mozem ti ten moj poslat ale mne to nefunguje.
S pozdravom Peter.

> ** Sat, 31 Oct 2015 09:13:40 -0700 - "feixm1/fastshare.host" github.com>,
"feixm1/fastshare.host" **
>
> Ahoj, má někdo hotový ten host na fastshare? nemám linux, kdyby šel
funkční poslat...
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/feixm1/fastshare.host/commit/a2f38f103bf5d0e2c84253c471

c3b32bd86a5c2a#commitcomment-14109792
—
Reply to this email directly or view it on GitHub
(https://github.com/feixm1/fastshare.host/commit/a2f38f103bf5d0e2c84253c471c
3b32bd86a5c2a#commitcomment-14109948)
.

"
=
>
> —
Reply to this email directly or [view it on GitHub](https://github.com/feixm
1/fastshare.host/commit/a2f38f103bf5d0e2c84253c471c3b32bd86a5c2a#
commitcomment-14109958).
>
>
—
Reply to this email directly or view it on GitHub
(https://github.com/feixm1/fastshare.host/commit/a2f38f103bf5d0e2c84253c471c3b32bd86a5c2a#commitcomment-14114661)
.

"
=


Done in https://github.com/andygrunwald/sortinghat/commit/9423236466672bedd5a57f64d83970db8f169adf


Is this being used?


那个词叫适销性啊……


and


`toxcore-git/PKGBUILD` missing?


Is this some kind of a joke? What is #whitelist?


I think this is right.

My main concern would be the actual range of volume increase for thunder, it may need adjusting.
@onox please check the filter and make sure this is what you had in mind.

@gilbertohasnofb you can also check and make sure I didn't make a big mistake somewhere.

I tested this (briefly) and the numbers for volume appeared to be doing what I wanted, I'm just not sure what I wanted is correct.

I'm out of here until this after noon :(


Merci !


Looks alright


@SanketDG Are you sure markdown is supported? I get mixed messages about that. I've heard others say only reStructuredText is.


Did you cloned your repo (mirror) or the original one? If you correctly mirrored your mirror,
did you config you local mirror of your mirror on git-hub using the correct ssh key pair you have to create previously? See this guide: https://help.github.com/articles/set-up-git/#platform-windows


let... good to know... I was wondering already how to do something like that :)


Probably a good idea. Can you open a PR?


Que bien que hallas podido añadir los Emoticones y sus dependencias, crei que no lo lograrias


I fucked everything up good catch, rewrote history. Tell me if I missed anything!


pull the strong parameter to a private method will be better and easier to maintain.


Doing `screenshot.pixels.join(' ')` is faster than `.to_s` but it still takes ~53s for a baseline run so I don't think this is worth doing.


I want to change the default text of several Text Field of the struct preferences setting dialog(such as the default text of while segment), because the original text is in French. But… Seems my edition is wrong.


don't think you need to do this include. let's review it today.


I saw you made this change, but didn't see any pull request. I went ahead and pulled it: 30f53a9015ec40b47b7a9df6908699818c6ba3b5

Cheers!


@post.summary.nil? is better


don't need null: false for timestamps


Could you add the case using `java -jar`?


Are you implementing D* Lite on a grid-based manner? If you are implementing in a grid-based manner, is the grid in this entire project 4x4 (by default)? How can I modify the dimensions of the grid. I am having a hard time in understanding and using/testing the program.


It looks like that this isn't always calculated correctly. My old code fixes this.


#LRU #NeverForget !


Adding original datafile name when and treat it when os or app sends it with a "df:..." start...


Happened for me with the release build.


Is there any convenient way to change the default text according to the language setting?


:+1: This has been on my todo list for a while now. Btw, astrolabe was written by one of RuboCop's most prominent contributors.


No,
```mode <- match.arg(mode)```
will default to "augment"


On a train home from the north of Portugal… shall review properly tomorrow.


acordate de usar palabras en ingles


Facebook strategy for Opauth


Nice to see the new scheduler system in action. This really looks a lot more comfortable than the old events / timers.


Can squash this into previous commit


Fixed in 17d753f, thanks.


I know that @ShadowNinja will want to change this name in his next "cleanup".


Fixed here : https://github.com/cappuccino/cappuccino/commit/0bc02de0129e912e4f040d1f16c582ec843a4924

Thanks for reporting, it was due to a bad refactoring of myself ....


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 3179](https://teamcity.mesosphere.io/viewLog.html?buildId=66677&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 195 Build time: 00:08:20


@wlbragg hmm, they all seem to have a very similar volume to me.


If I really wanted to ignore cmdliner's API, the changes would be much larger than that. I made the smallest one that I felt would be both beneficial and not compromise similarity to cmdliner. With this change, `opt`, `required` and `flag` have the same argument excepts the extra one, which is labeled.


Favicon.

I forgot to comment this commit ^^


And why not:

```
function fizzbuzz($n)
{
	$data = $n;
	if (($n % 3 == 0) && ($n % 5 == 0)) {
		$data = 'FizzBuzz';
	} elseif ($n % 3 == 0) {
	    $data = 'Fizz';
	} elseif ($n % 5 == 0) {
	    $data = 'Buzz';
	}
	return $data;
}
```


Why do we have a path like this `src/Common/tests/Tests/System/CharArrayHelpersTest.cs`
that has 2 "tests" in the path?


この lazy は「遅延」でしょうね。
あと、「対象のコード分岐…のみJITする必要があることから、」は、それ以外をJITしてはいけないように読めてしまうので（実際にはJITしても無駄なだけで間違いではない）、「実際に実行されるコード分岐（たとえばif文）の依存先のみがJITを必要とするので、」のようなニュアンスで訳すのが良いと思います。


この atomic は atomic operation を意味する atomic でしょう。アトミック操作で置き換え、あるいは 不可分操作によって置き換え と訳すのが適切だと思います。
https://ja.wikipedia.org/wiki/%E4%B8%8D%E5%8F%AF%E5%88%86%E6%93%8D%E4%BD%9C

（invariant は 不変性 = invariance ではなく、「不変性を体現した実体」であるところのエントリポイントそのもの、のことだと思うのですが、まあ意味的には不変性でも大した齟齬はないでしょう。）


ここの can は「出来る」ではなく「可能性がある」と考えたほうが自然な気がします。


「その状況下」だと「どの状況下?」と考えてしまうので、givenは「所与の」「特定の」「現在の」などの、状況を特定するけどあくまで一般性を示す語句、にすると良いでしょう。

keyは鍵ですが、「鍵となる違い」はあまり日本語に馴染みがないので（「キーコンセプト」くらいまでいけばkeyに馴染みがあるといえるでしょうが）、「ここで重要な状況の違いは」くらいの訳し方がベターかと思います。they key differenceは of situation なので、これも補足したほうが良いと思います（英語では単語間の距離がほとんど無いですが、日本語訳では大きく開いていますし）。


「AならばBなのでCである」を「BなのでAならばCである」と書くと（特にA/B/Cが複雑だと）読者は混乱します。
「たとえば、複数回メソッドを呼び出すのに仮エントリポイントを使用すると、呼び出しのたびに事前スタブを経由することになるので、これはよくない考え方でしょう。」という語順にすれば、「えっ、呼び出しのたびに事前スタブを経由するの?」と考えながら読まずに済みます。


not every caller will have **slotMapping** to be called for ajax returned data. need to have a *callback* function provided by the caller and pass the data to that callback function here.


原文が残っているみたいですね。


FYI, this is the failing test.  Curiously, it passes if there is any data to decompress, we read off all that data, and then do a further zero-length read  So the test name is perhaps slightly misleading - it's not _any_ zero-length read at eof that failed, just specifically for an empty file.


fild should be field?


this function should have 2 more parameters
**cgi**: this will be the server cgi program to be called
**callback**: this will be the function that will be called by the ajax success/error handler. also the ajax data will be passed to this callback function


shoot


change this to match the **cgi** parameter from the function parameter


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.55](https://pixelated.design/build/viewLog.html?buildId=183&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 275 Build time: 00:00:17


add migrations path to gitignore for every new app... migrations from different systems create chaos later in other systems.


You are right, border is setup by configuration file only in xxxxxxwindow section by border command.
Support for UNIX terminals is limited and need improvement.
Home directory by default is $HOME, but you can customize it by creating ed.cfg file in home directory where you can switch on/off border for each type of windows and your preferred home directory.
Now default configuration file is bind to vi executable that if you remove ed.cfg file then this default is used. I don't want to change vi default setup I prefere customization of ed.cfg by user, because it is global for all supported platforms.


Is this file a symlink? If so, git doesn't handle symlinks soo well. Looks like this link wouldn't work for anyone else unless their username was afrazier

Maybe consider excluding these.


is **field1** required?


> D'autant qu'actuellement, dans master, patacrep/data est un datadir (au sens songbook).

C'est aspect est très intéressant et peu justement permettre une grande personnalisation: avec un système de liste (ordonnée) de datadir. Pour tout ce qui est type 'data', `patacrep` cherche successivement les datadir jusqu'à trouver le fichier requis (jinja est capable de le faire nativement).
Pour les utilisateurs avertis ça me semble assez puissant: il suffit de créer un dossier `my_datatir` et de personnaliser justes certains fichiers (templates jinja/templates tex/images html...).

Proposition: création d'un dossier `internal` (ou `patacrep` ou `resources` ou `internal_resources` ou `tex_generation` ou ...) dans le dossier `data` et de regrouper tout ce "bazar" dans des sous-dossiers (si possible bien rangé).

Du coup c'est assez propre (seulement un sous-dossier "exotique" dans `data`), c'est facile à inclure dans l'installation, cela regroupe tout ce qui n'est pas `.py` et cela peu offrir des grosses possibilités de personnalisation sans toucher au code (modifications de code sont nécessaires)


基本的に誤解の余地が小さいほうが良いと思いますが、原文レベルでもinstantiationはオブジェクトについてもジェネリクスについても使われる語句ですし、この辺の議論のように、混乱が生じる具体的な懸念がない限りは、genericの接頭辞無しで使ってもまあいいんじゃないかなとは思います。
日本語にしないほうが良いというのは賛成です。


I don't think the table names should be plural. The convention that I have seen before is that all table names are singular. So we should have 'feature' and 'batch'


Some general thoughts-
The stated goal of this project is to show how print type fonts have been made usable in web design. From the content, however, I’m having a hard time seeing the connections. Consider answering the question: How did early web developers deal with fonts? How has this process evolved to where it is now? You do a good job of describing what current developers can do. I would assume one of the first challenges was making the different fonts available in word processors. How was this done? From there, it would seem easier to have them rendered in HTML.

Overall, the design is fine. A few minor suggestions:
-create a link to the terms page whenever the term is used.
-please don’t use the red background. There’s nothing inherently wrong with it, so you can keep it if you’re really attached to it, but it seems harsh, and I have a hard time looking at it.
-keep all the links across the top even when the link is the active one. Having all of them always present across the top helps me stay oriented as to where I am.

Please give a summary of the “fascinating process through which characters are translated into a font by the server”. I would really like to know, but the information in the link is difficult to understand.

I’m a bit confused about the role foundries are playing today. I understand casting blocks for different type faces for use in printing presses, but is this technology still in use? Could you explain how?

You may want to make a connection between what kerning means today (the space between letters on a line that some software programs allow you to adjust) and the process you describe on the history page.

Which of today’s measures is parts out of 72? I can look it up in Duckett and learn that it is "points" or just guess that is what it is, but I think it would be better if it were made explicit.

You may want to note that monospaced fonts have practical uses. If you want an “M” on one line to align with an “I” another line, it’s helpful. Monospaced fonts allow this to happen. A biologist who wants to align a protein sequence from one organism to a related sequence from another organism will find this useful. (I know because I have done it. The alignment doesn’t look at all meaningful using a font that isn’t monospaced.)

Please include “glyph”, “Unicode range”, SVG, and sIFR on terms page.

I am a bit confused about the “format” concept. Does it refer to a website where the fonts can be obtained, or does it refer to the appearance of the font?

Code Review and some details-

You need a comma between “skills” and “people” in your first paragraph.

The “Many suggest… “ paragraph appears twice under the “Two” heading.

Consider using the quote and block quote elements for the “inventor of sIFR” (quote) and “Many web fonts differ” (blockquote) statements.

The apostrophes weren’t rendered in my browser.

The “kerning” link and the “formats” link on the terms page don’t go anywhere. I have some links that don’t do anything yet as well, so maybe you knew the links were incomplete, but if not, please remove the link formatting or add the content.

Remove the padding from SVG so it doesn’t block the surrounding text.

Consider adding links to the open-sources on the @font-face page.

I hope you found these suggestions helpful.


- added dup_movies table - This table will have all the duplicate movies present with required info.
- failed_movies table will have details about movies for which we failed to get imdbid [ This could be displayed to user/can do some advanced search]
- now directory can be passed as command line argument [ for easier tesing]
- TODO: can't execute from eclipse. config.properties isn't getting read coming as null


Looks very good in general. There are a few things to clean up
- ajax calls made from ALL views(like robot,dashboard,confg. etc..) should go through window.BaseView.ajaxCall.
- So have to make all views other than input views extend from window.BaseView
- window.BaseView.ajaxCall should take additional parameters **cgi<string>** and **callback<function>** so the cgi program on the server is provided by the caller. Also the caller will provide the callback function which ajax success/error will call to return data and/or error msgs.


just trying to make mormal commit :+1:


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.54](https://pixelated.design/build/viewLog.html?buildId=184&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:12


i don't think this gob is in 3.3.5 or is it?


I just used the same path as other Common tests, e.g. https://github.com/dotnet/corefx/blob/master/src/Common/tests/Tests/System/StringExtensions.Tests.cs


in URL, we try to avoid case swapping.
The url should be simpler, you don't need 2 levels (newAchetype/add).

Something like /archetypes/new


It's not out of the question that we want to do out-of-plane velocity models (e.g. glacier channel flow). This can be faked with 3D periodic bcs if the elements are trilinear but that may not be the best way.

       if velocityField.nodeDofCount != velocityField.feMesh.dim:
            raise ValueError( "Provided 'velocityField' must be a vector field of same dimensionality as its mesh."


Since the view is also shared with the update method, maybe addArchetype isn't a good name. form.html.twig may be enough ?


Copy command info


I still don't see why you have to change the argument order. You can force users to write more readable code and you choose not to do so.


How do you remove, with a GET or with a POST ?
To avoid XSS attack, POST only.


:+1: :D


>Lets not have any trusted apps part of the default settings. Why not? Where else to put it then?And yes, we just have one app right now, so it would be better to store it in dictionary.

Date: Sun, 1 Nov 2015 08:32:35 -0800
From: notifications@github.com
To: cleansweep@noreply.github.com
CC: Raghav_ftw@hotmail.com
Subject: Re: [cleansweep] Added a yaml file for trusted-apps (8b1ef81)

Lets not have any trusted apps part of the default settings.

Also, YAML was just an example. Probably it would be easier to write it directly as dictionary in the production settings file.

We may consider switching to YAML for config, but that thought for another day.

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


Ah sorry, didn't notice this was actually commited and not a PR. See #1572


Yeah, I wondered about that when I added the label. I decided to choose the general rule of "put the label before the rest", but I could change that.


Probably we need specs using `not` as well.


Your missing boringssl compatibility library https://github.com/CyanogenMod/android_external_sony_boringssl-compat/commits/cm-13.0


'.docx' ? I'm sincerely chocked.


the symbol "-->" and "<--" makes netbeans do some crazy shit


typos?


lol oops


Oh. .WithSize was necessary?


Why this site is not added?


It seems you do not need this check.


y, I saw this in the commit notes/API docs as well but was not able to get this property populated via our testing mocks...


@inpercima It would be better to see this as separate commits b/c getTimePart has nothing to do with logging or the routeProvider. What do you think?


If you don't mind me asking, when are you planning to add NeoVim support?


Looks good!


The [commit](https://github.com/hammerlab/ketrew/commit/a4a2e1b7697a80e183d28ba695f3db0bfeb30794) reworks the `read_loop` logic. Let me know if it addresses your concerns. For me, at least, it removes all of the original confusion about which monad is doing what.


Removed unecessary brick image reload


Thanks man! This worked like a charm :)


Append _lego instead of prepend so that generated files are next to the original when sorted alphabetically.


String formatting for zero padding


i think its a left over actually, il have to check into it as i copied it from a recent project, thanks for pointing it out


Print the number of frames before converting to have an idea of how long it's going to take and when it will end.


This commit changes the path to the jquery file on the index page.


Snyggt!


Hey dude seems Emby forum is down too. With regards home screen, I meant it is hard to read when blur is not turned on so maybe we blur the bottom in that case?


It occurred to me that you may have used the quote element and blockquote elements. When I went back and reviewed it, there isn't really any advantage to using the quote element instead of just quotation marks, so you can disregard that comment. As for blockquote, I expected the text for the quotation to be indented. I don't know if that is a default for the element or if there's something in the css file you need to add. So if you did use the element, you can ignore that part of my comment and see if the css needs some adjustment.


All would br very nice. :)


I cannot see your results (no image), but my graphics card is an Intel HD Graphics 4000 [from April 2012](https://en.wikipedia.org/wiki/Intel_HD_and_Iris_Graphics#Ivy_Bridge_2).


why wasn't this done through a PR? the `jl_ExecutionEngine` object is not intended for external use


Not necessarily in exactly this case, since all menu icons are bitmaps, but an Xwt Image can be a vector image without a fixed size. So in general we should always set a size requirement (we could check for zero size first, but since the menu icons should always have the same size, its ok to request it directly).
And the null check is important, because not all commands have icons.

But this doesn't fix the actual issue with Gtk stock icons. This needs a fix in Xwt, which I'm working on atm. This patch simply prevents MD from crashing, and logs an error for broken icons.


Proposed change from "." to "%" resolved issue I raised in my previous comment. Taken from http://sogo.nu/bugs/view.php?id=3373


"Streaming IO" is common in most programming frameworks, and is usually very useful. I mean, usually you don't really want to load a 100 MB file into a `String` in memory.
Of course `Stream Char` may not be a very good idea, and there might be a better interface that works well with resource tracking and strictness. Perhaps allowing some kind of location, along with rewinding and a read more function might be better, although that is somewhat more imperative.


+Valerie Hellinghausen


NeoVim support will take require rewriting a good deal of the bridge work between Vim and color_coded (a mixture of VimL and Lua). I don't have it planned out, currently, since it'll involve some more research into NeoVim's asynchronous architecture.

[Some work has been done](https://github.com/neovim/neovim/issues/719#issuecomment-147191109) on this front, by members of the NeoVim community. Unfortunately, it was on a much older version of color_coded and it wasn't (meant to be) and ideal implementation.

I've just opened #89 for this; feel free to comment there if you're interested in contributing.


Fixed in another commit.


how do you feel about this? do you think we always need to in initialize like this?


我建议你看看原版程序的翻译……


oh yeah... somehow this gets swallowed after reading "filename as key" :P


WTF


I checked OW vi documentation and for saving configuration file you must riun GENCONFIG vi command.


hmm okay... :D


This is not acceptable. Just say:
```
gnuplot-py, scapy, pyrit: import

pyrit depends on scapy, which depends on gnuplot-py.
```


Could you write a comment above code?


that really needed? :/


I also updated it so that it would add the entry to _entityTypes when not already there, but type was already in memorycache: https://github.com/SparkDevNetwork/Rock/commit/462135e4c4d61800b2b5ff907c688bc0e4ef3200


We really should avoid modifying the raw js files, especially since this js isn't even minified for production use.


I'll probably want an explanation why we needed our own version of std::make_unique(). Remind me please on Monday (I'll ask).


Fix exit codes : non-zero means there was an error


Ugh ceylon/ceylon-ide-eclipse#1583.


if I would look greedy it would stop at the last comment in file. Also `search_in_between` does not allow such behaviour (okay maybe it does, but the regex would be quite hacky I believe). A new doctype is much simpler in this case^^


fixed locally


Fail early on Windows. Turns out there is a default program called convert already in the path. Without this environment variable we can't do much.


Élégant !


Hi. looks great! Can you remove the commented out code for me? thanks!


```python
def A():
    print("hello")
    yield 1
    yield 2

def X():
    print("hello")
    def it():
        yield 1
        yield 2
    return it()

A()
# Nothing happens, because we only set up a generator.
X()
# prints "hello", and then it sets up a generator we can iterate over.
```


type は、これ以降の使われ方を俯瞰した印象としては、「型」より「種類」と訳したほうが良いような気がしています。型はCLR型の話と混同する可能性があるためです。たとえば、以降で次のような一文が登場しますが、

> 本エントリポイント用のプリコードの割り当ては、一致する型のものが存在するならば、割り当て済みの仮エントリポイントのプリコードを再利用しようとします。

この「型」は、パッと見、どの意味で使われているのかわかりません。


「パッチ処理」よりは「上書き」みたいな日本語にしてしまったほうが意味が通じるかもしれません。以降も同様。


They are trading off speed for size. が抜けているようです。

ソースまで確認していないのですが、FuncPtrStubs は識別子で FuncPtrStub としてはいけないのではないでしょうか。あと、もしこれがテーブルそのものだとしたら（FuncPtrStubs）は「テーブル（…）内のスロット」と書くべきかと思います。ちなみにここ以降に「テーブル（FuncPtrStubs）」と訳した文も登場します。


ここも 遅延バインディング でしょう。


you're introducing a whole system of doctypes just because you're to lazy to implement one type properly? That still leaves you with bugs because I *do* expect the standard doctype to work in all cases and there's nothing that keeps users from using it for the special cases. Adding more algorithms adds another layer of complexity.


maybe needs another fixme or note to relativate
> contains **all** the data that the base Result has.
since that is not the cast atm?

see line 124


> here appears to be a bug in current (4.1.3) smack, related to trusting certificates arbitrarily

I'm not aware of any kind of such a bug. If you want to trust arbitrary certificate, i.e. perform no validation at all, then you could either just disable TLS (because there is no gain in using it then) or use
https://www.igniterealtime.org/builds/smack/docs/4.1.4/javadoc/src-html/org/jivesoftware/smack/util/TLSUtils.html#line.95
and
https://www.igniterealtime.org/builds/smack/docs/4.1.4/javadoc/src-html/org/jivesoftware/smack/util/TLSUtils.html#line.121

> and XEP relating to capaility-agreement-verification
What is "capaility-agreement-verification"?


see #2


Yeah, I had the IsNull check. Great that it works!


This was already fixed in the fixes-branch (b0c9f2b8a13951adc207dedbe2e509629a8f7717).
But thanks for reporting!


ahh you mean the inlined function...
Maybe I could try to chain them via `functools.chain`, this would require the same amount of code as using a list^^


Thanks!  Well that's weird.  It looks like they made that change in February and pushed a maven release in April.  I presume the release contains that fix, but I haven't been able to get that 1.0 version to work, only your patched version.  I'll keep playing around with it.  Cheers


okay how is this property important here?


@Sepe that's right. I will change this when back home.


Sure! I am actually cleaning it up.


We need a single space after `>`, see the option for the commands. Please squash the commits into one after fixing it.


It is a good c++14 feature that encourages safe coding in terms of memory allocation and deallocation. It is well encouraged to use your own version of this function even if you are in c++11 standard (which lacks this feature). For more info see http://en.cppreference.com/w/cpp/memory/unique_ptr/make_unique


or simply

```python
results = []
for ...:
    for ...:
        results.append(...)
```


> WTF

Was that really me...?


what about this one ? is it no more db ?


- has a menu based gui


Ist das nicht eher 'Schriftstil'? Oder 'Variante'? Oder sogar 'Schriftschnitt'?


fixed locally


Fixed


Bedingte Ligaturen heißen wirklich so, glaube ich - ich habe ziemlich lange rumrecherchiert für die korrekten Begriffe. Genauso wie die gewöhnlichen Ligaturen.

Oder hast Du vielleicht bessere Quellen aufgetan? (ich habe das auf Seiten über Illustrator, InDesign, auf typographie.info etc. über Google gefunden).


That has nothing to do with laziness, it would be much more complex to let the standard doctype maintain all cases (if it's possible anyway). And if I would need to implement all doctypes into standard anyway, I would just make an if-case at the beginning and copy the existing code for the doctypes in each case. Not better, it's even worse.
Btw I think my solution with different extraction algorithms is quite good, it's very flexible. I don't know if there are other doctypes for other tools out there, I'm mostly referring to doxygen.


ack


Danke für's Updaten :) - ich wollte schon fragen, als ich gesehn habe, dass Nicolas den Bug bearbeitet hat, aber Du warst einfach viel schneller!


If the settings are incorrect for a certain doctype, it raises directly a ValueError, not when we start iterating over it.


this is not better...


if1live@3ea69436576ba15ea045d4c04b9bb180a29f05f0 Tested as working for me. Thanks


reack


Не вопрос. Только будет, скорее всего, сыроватым.
И разбираюсь с тем, чтобы ещё 2 к своей работе присоединить.


Because Cxx uses it and it has been exported for several years now


this is not a change made here and not really a problem of coala - BUT - I just noticed that **blue** is *very* hard to read on the default settings on my system (Konsole on openSUSE as well as TTY).

This is mostly a shortcoming on openSUSE's side, but if you experience the same on other systems we might want to change to light blue or something.


You should be aware that the access keys are still in the history. You should probably revoke the keys and generate new that you guard closely :)


What you see here is not bad thinking but badly named variables. 1001 is just the default used for ALL servers, and should never be used. So renaming AuthConfig to ServerConfig would be more appropriate. You also can't rely on the name "World", since that can be set as an parameter to the --world like "LUNIServer.exe --world WRLD" would also be a world server but have 1001 as default in your code. So please leave this as it was, just with the little changes I mentioned here.


Добавил.


Дмитрий задал противоположный вопрос и я изменил на ссылочный тип, таким образом избавляясь от флага, указывающего на то, что в int происходило прибавление


Переименовал как temporaryVariableForIntSum


Дополнил javadoc


Дополнил имена:
checkOnNullValueAndPrintIntSumIfNotNull и checkStringCounterAndPrintStringsIfNotZero, соответственно


Что бы не делать проверок на null. Исправил, добавил проверку в метод log для String


В результирующей строке получается {variables,..,variables}, я использую replace по всем не числам, что бы получить строку без скобок.
Сейчас пришла идея как сделать без этого, изменил.


Это было сделано специально для теста, чтобы проверить, что конструктор отрабатывает:)


Что бы Sonar не ругался :) К тому же нам не зачем создавать экземпляр класса, так как все переменные и методы у нас статические.


le cercle horrible removed


Не очень понятен вопрос. Почему element, а не message?


@elgalu Thank you for PR!


ack


I am fairly sure, because I am done this on my repositories and also contributed to other repositories.

Here is notedown's [`setup.cfg`](https://github.com/aaren/notedown/blob/master/setup.cfg) and [notedown](https://pypi.python.org/pypi/notedown) on PyPi.


Does not take into account user-defined aliases yet.
Need a method in logic to read those but shortcut commands(def, all, ...) and operation commands(add, delete, ...) work.


Resolved in #355.


Schriftstil wäre z.B. "serifenlos" / "dicktengleich".
Schriftschnitt wäre z.B. "kursiv" / "fett"

Merkmal ist vielleicht nicht die beste Übersetzung, trifft es aber glaube ich noch am besten. (Wird auch in einschlägiger Literatur verwendet, z.B. http://help.typekit.com/customer/de/portal/articles/1789736).

"Variante" ist meiner Meinung nach völlig nichtssagend und hat seinen Ursprung im zugrunde liegenden CSS attribut `font-variant`.


This should use the Count property, not the method.


Resolved in #366.


Oh, I see the logic now.  And I was thinking along the same lines... having the signer be separate so it can be used (and tested!) independently.  This separating out the signer out makes a lot of sense and has benefits.

I propose, **for now** , that we
1. use the same test keypair that current resides in source control now for both the client and the signer
2. have the test suite configuration be the base config properties in testng.xml
3. allow the environment variables to override the testng.xml file
4. clean up the existing maven pom.xml
5. create two submodules... one for signer and another client which depends on the signer

I think this sequence would allow us to accomplish what we want to do, and while separating into modules makes things a little more complex, I was thinking we'd already want a separate module for the implementing the compute functionality of the manta service.  But for now I'm simply eager to get the outstanding PR merged so that tests run again.


just a though: having the variable `line_number` counting the lines in enumerate-fashion starting with a zero is of course justified, but very error prone.

somebody that doesn't know the code might easily expect this to be a human readable line number.

I think it would be cleaner to start the loop with
```python
for index, line in enumerate(file):
    line_number = index + 1
```

and then use line_number later on


@sils1297 the choice is up to you though, ack besides


Resolved in 436e9e3989ed707745e491e3f5168b31220ad17d as part of #366.


@rhatdan

1) https://github.com/shishir-a412ed/atomic/commit/6c54ac8b8ffb0b3cf8b2f126a534c1ff831db0a6#commitcomment-14074413
Fixed.

2) https://github.com/shishir-a412ed/atomic/commit/6c54ac8b8ffb0b3cf8b2f126a534c1ff831db0a6#commitcomment-14074458
This is done to modularize the code, where Export and Import are in separate files. (This is in sync with atomic mount command). With this: atomic (main script or entrypoint) ----> Atomic/atomic.py -----> Atomic/Export.py || Atomic/Import.py.

3) https://github.com/shishir-a412ed/atomic/commit/6c54ac8b8ffb0b3cf8b2f126a534c1ff831db0a6#commitcomment-14074599
Fixed.

4) https://github.com/shishir-a412ed/atomic/commit/6c54ac8b8ffb0b3cf8b2f126a534c1ff831db0a6#commitcomment-14074618
Fixed.


Оказывается некроопера не обновляет элементы (или обновляет криво) при динамическом изменении `fill`. Плюс не поддерживает `transparent`-ные `fill`ы (слава богу `rgba(0,0,0,0)` заработал). Добавил костылей в скрипт, но вот CSS-писатели с некрооперой обосрутся и не смогут менять цвет фона кнопок например при наведении мыши.

И ещё один доставляющий баг: если у кнопок постов стоит серый градиент, попробуй в треде несколько раз нажать на глаз (т.е. включить/отключить AJAX-обновление). При этом кнопки станут чёрными и будут оставаться такими даже после обновления страницы. Единственный способ исправить - открыть страницу в новой вкладке при этом не использую функцию дублировать вкладку, а вручную скопировав адрес из старой и вставив в новую вкладку. Ну ещё можно перезапустить браузер.


that would belong in another commit at least, I'm not bought on this. I'm
actually thinking of using the index as line numbers, i.e. changing the
result spec

2015-11-01 19:26 GMT+01:00 Fabian Neuschmidt <notifications@github.com>:

> @sils1297 <https://github.com/sils1297> the choice is up to you though,
> ack besides
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/75a318dd0ab9703d740bf611ca597462ccb08d5e#commitcomment-14115637>
> .
>


These are deleted in the `cleanup()` function.


Resolved in #361.


@mcostalba: Sure, but for TT access we don't have a choice... If we had I'd
change it straight away.

On Sun, Nov 1, 2015 at 4:33 PM, Marco Costalba <notifications@github.com>
wrote:

> It is completely undefined as is a TT access... Nevertheless it works.
>
> Even if in the counter ends up to be filled with complete random garbage,
> the worst it can happen is to have a spurious check time call, and counter
> is reset to zero at the next tick.
>
> I will try to write something with atomics, hoping (and testing) it does
> not regress, but in this case it is really overkill and IMO not needed.
>
>
>
>
> Ogn Sunday, November 1, 2015, Joona Kiiski <notifications@github.com>
> wrote:
>
> > @mcostalba:
> >
> > - Until a month ago we used volatile boolean flags to pass some flags
> > between threads. For splitpoint shared variables we always used lock
> > protection to guarantee the expected behaviour.
> >
> > - What we are trying to do here is completely unorthodox... Sure it works
> > well with a spinlock or mutex, but without them the behaviour is
> completely
> > unspecified.
> >
> > On Sun, Nov 1, 2015 at 10:01 AM, Marco Costalba <
> notifications@github.com
> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> > wrote:
> >
> > > @zamar <https://github.com/zamar> timeCounter is (mostly) a local
> thread
> > > object, it is incremented just by one thread and only very occasionally
> > it
> > > is reset to zero by another thread. If we set it atomic, we force a
> sync
> > > after every increment, completely giving up the advantage of a local
> > > counter. Moreover until one month ago, we always used shared variable
> in
> > > this way and it worked well (as it works well in C where there is no
> > > atomic). Said that, we *could* use
> > >
> > > std::atomic_fetch_add(&timeCounter, 1, std::memory_order_relaxed)
> > >
> > > instead of
> > >
> > > ++timeCounter
> > >
> > > But it seems to me an (unnecessary) overkill.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/mcostalba/Stockfish/commit/edac860627b6706e1c0722812c72731dc02d463d#commitcomment-14113328
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/mcostalba/Stockfish/commit/edac860627b6706e1c0722812c72731dc02d463d#commitcomment-14114758
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/edac860627b6706e1c0722812c72731dc02d463d#commitcomment-14115017>
> .
>


Fixes #2


I will see if I can get us a new joyent/java-http-signer repo on Monday.

1. Yes.
2. Yes.
3. Yes, and see this PR: https://github.com/joyent/java-manta/pull/25
4. Do you want to do this?
5. Let's do this once I get the repo created.


Ich bin ebenfalls über "bedingt" gestolpert, aber das ist für den Laien meiner Meinung nach völlig unverständlich. (Ich würde mich sogar aus dem Fenster lehnen und sagen, dass das im professionellen Umfeld ebenfalls nicht sehr gebräuchlich ist)

Die "discretionary ligatures" sind Ligaturen, deren Verwendung "im Ermessen" des Schriftart-Designers liegen. Sie sind für die für die Lesbarkeit nicht notwendig, können jedoch zusätzlich verwendet werden um das Schriftbild in sich stimmiger zu machen.

Ich hänge nicht an "zusätzlich", wenn dir was besseres einfällt gerne ändern, aber "bedingt" versteht kein Mensch weshalb ich mich klar dagegen aussprechen muss.


Resolved in ab526f97c6cd08a7f9eea07dd5b843f9ba849791 as part of #366.


Fixed.


Explained you in person. Fixed.


can't this be still useful if `document_get_current() == doc`?


Resolved in d5c5ae5a523ae4a7b616fbe4ab50cd3e0dd9d78c as part of #365.


Resolved in ff9f22cad75ebc1371911a3127330dc0ae4cf327 as part of #366 and 502f9f39ab34acbbad6ec2cfa8d190f6ac0367d9.


LGTM


> И ещё один доставляющий баг

И, кстати, я не знаю как его исправить, лол. Видимо стоит отключить градиентные кнопки в некроопере, а то после пары-тройки AJAX-ошибок кнопки, ВНЕЗАПНО, станут чёрными.


En réfléchissant, tu as déjà le patient… Pourquoi tu le recherche encore ?


Resolved in 39e9b3592bcdf793c06b4f2c3280cee6a44addde and d5c5ae5a523ae4a7b616fbe4ab50cd3e0dd9d78c as part of #365.


In what sense these areas are "code" and "data"? It's all single SRAM block, right?


Partially resolved in #370.


They are in 'atomic_migrate_export' and 'atomic_migrate_import' functions.


Resolved in #370.


fixed in recent push.


Shouldn't this test allow for equality to the limits? Like so:
```
       if ((intensity >= min_intensity) && (intensity <= max_intensity))
```


These are the results from my MBP with p=0.2
totally different.

I'm just running things with a new MBP from 2015 (graphics card still
Intel), but the results seem to be closer to yours.
This is weird.

On Sun, Nov 1, 2015 at 11:41 AM, François Briatte <notifications@github.com>
wrote:

> I cannot see your results (no image), but my graphics card is an Intel HD
> Graphics 4000 from April 2012
> <https://en.wikipedia.org/wiki/Intel_HD_and_Iris_Graphics#Ivy_Bridge_2>.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sctyner/ggnet-paper/commit/19991a2bc71113c39663b3d95e91ef4394054de8#commitcomment-14115458>
> .
>


Как в воду глядел! :wink:


I think these tests are now unnecessary, right?


Fork and cherry-pick are very easy to use, so the "i'm not that good at git" excuse is not working here. If you sign other users commits as if they were your own, you should at least understand that they will complain. You've been doing the same for a long time now. If you did they same thing in xda, you would be banned. I hope you'll learn before doing the same thing again with the new armani kernel sources.


What 'POS' stands for? My first thought was "Piece Of Sh..", but obviously it is because of my broken sense of humor ;)


I didn't have those symbols.
Github was just outdated.


Looks good! (without actual testing though) Thanks!


@zlot Glad you found it useful! :blush:


oh this stinks we have to change this... I think it might be worth using an exception comment here instead


:wave:


FIxed.


Fixed.


weird change right?


:clap:


look at this kid getting all es6


Are spheres unblocked aswell, @tylerhyperHD ?


`$this->rrmdir`


ни, так не надо.. выбрасывай рантайм, когда не знаешь что делать :)

throw new RuntimeException(e);


I've been thinking the same thing. At some point, yeah.


Maybe name it `MBEDTLS_SSL_DTLS_HS_MAX_QUEUE_LEN` (we might have another queue for something else than handshake some day)?


Hmm. What does that really mean? And why?


у гуавы есть для этого прикольная штука, называется `Preconditions.notNull(je, "source json can't be null");`
Или, аннотация от jetbrains `@NotNull`. Первая кидает NPE если что, а другая не дает компилировать (вроде)


Should RTC be just a tad higher? So that its above PENDSV?


@SanketDG Okay, awesome! I'm going to include this in the 0.3.1 release! Thank you so much!


Rather than fetching this value multiple times via `instance_variable_get`, please stick it into a variable above and us it for both validations


Yep, already working on 4

And for 5, I was not thinking of another repo being created, but I can see the benefits.  But I'd really like to have a parent pom that modules inherited from so that it would be possible to manage properties and dependencies a little easier across modules.  I've worked with a lot of multi-module maven projects, but not sure how it would work across github repos.  Any insights?


Preconditions.checkArgument ...


Can you please put a new-line after this `end` block?


Yep, that's exactly what I was testing :). Was trying to see if I could get a Trusty build on the new infrastructure. Turned out exactly as you said.

Thanks.


One side effect of this change is now `Process.kill` will throw an exception if the process isn't found, whereas `kill` would simple continue.


Excuse me, but why is your mod jar called 'StopModRepsosts'?


Style: save one level of indentation by doing `if( ... == NULL ) return( -1 );` instead?


Seems reasonable, could you please make the two changes I suggested above and then I think this is good to go. Thanks for all the hard work @thenovices


можно заменить одним (Exception e), ну или мультикетч InstantiationException | IllegalAccessException ..


Dap , cu bower, a fost greu la inceput pana am gasit cum trebuia sa fac, ca la tutorial n-am apucat sa notez clar totul :)


если это заменить на Constructor<T> то кастить ниже не нужно будет, и компилятор за тебя проверит корректность.


AciD Part.
http://paste2.org/en1X8LHX


implements #1


лол это что за версия


I'm wondering if it may be saner to treat that in the same way as `total_len > MBEDTLS_SSL_DTLS_MAX_QUEUE_LEN`. After all, this is something we can recover from.


So, Im having a little trouble compiling it. I think this is because I didn't install ctr-wlanbeacontool correctly. I have compiled ctr-wlanbeacontool, and it produces a folder build_nix and ctr-wlanbeacontool file. Do I put these inside /opt/devkitpro/ ? I tried export CTR-WLANBEACONTOOL=path_to_file_it_made but that gives me an error. I realize this is entirely me fault for being stupid, but I would appreciate the help.


I see the value of having it as a separate github repo because that would be the component that you would need if you wanted a SDC SDK for the JVM. Simply, that functionality is related to the API and not to Manta itself.

As for a multi-module project, across repos you can get a pretty efficient workflow when you configure IntelliJ for it, but it can be a pain to get setup correctly.

In this case, I would argue for it being a separate library with no overlap. In fact, I want to figure out a way to shade bouncy castle because bouncy castle doesn't adhere to any notion of semantic versioning and when you get convergent dependencies it is hell to manage.


@karasu dkms should build the modules during your next reboot. Let me know if that's not the case.


не забывай нажимать ctrl+alt+O
зачем мусор в импортах носить с собой


s/flight/queue/


Is that intended?


OK. Unless you've done this already then I'll try again. In future just get in touch, we're a tiny development team.


1.0.15 is out


Perfect! :)


You can use TesSubsriberand assert received onNext/onError/onComplete events using it


@Geesu Could you open an issue so we can revert this change?


I thought private methods don't require javadoc and comments are enough for this


I don't think changing this is necessary, because I will merge the patch into the master branch.


Old year?


Oh.. so yeah we can do that..  although the bigger issue here is on line 42.  That the thing it's complaining mostly about... reusing that var name.  If we could come up with a better var name internally that'd solve this too.. I suppose we could move the iterator as well into it's own function.


What if you serialize the subject?


Maybe it would be clearer to just ignore errors (with a comment)here (see above about not aborting on alloc failures), it makes the logic of the current function more "self-contained". That way `ssl_hs_queue_append()` could return `0` on success or an error code to be printed by `DEBUG_RET`. The `return( MBEDTLS_SSL_ERR_WANT_READ )` is already here a few lines below, to IMO it's better for `ssl_hs_queue_append()` to return `0` on success.


I think I have suggested this material informing time use idea a couple of times.  I still think it is advantageous.  It is probably just a mater of tuning.  I hope you don't give up on this.  I strongly suspect that as time control increases the use of time should shift toward earlier.  The end becomes clear from further away and unchangeable.  Late middlegame early endgame should not be neglected though, that is where the stronger player usually shines...when there are still enough pieces on the board that the end can't be seen.


Symptoms:
 1. open an image, preferably smaller than the screen
 2. zoom in a couple of times, preferably so the image top is now off-screen
 3. click on "view actual size" toolbar button

Note that the image is now displayed such that the top is scrolled off the screen.


That's great. :+1:


This looks good to me. Thanks for this.


Style: `return( ssl_prepare_handshake_record( ssl ) );`


@ahumphreys87 also commented on this one.  The issue here came with an update to chai.  It was a strange test to begin with.  `getEmptyView` is not passed _any_ arguments.  Previously it wasn't complaining about it, but it does now.  https://github.com/marionettejs/backbone.marionette/pull/2801#discussion_r43576436


Already done by the `memset( 0 )` at the top of the function, but why not.


I added an exception for the bin and obj folders in the Source/Content directory. Other Debug and Bin directories contain files generated from building. Why is mac looking inside these build directories?


Absolutely thanks! For now (given we have a capistrano task that iterates through all workers to be killed) I'm just rescuing from `SSHKit::Runner::ExecuteError`.


Hi,
In Osenc::ingestCell( const wxString &FullPath000, const wxString &working_dir )
line 1178
OGRS57DataSource *poS57DS = new OGRS57DataSource;
is never deleted (it was originally in s57chart.cpp).
and it leaks memory and a file handler.

Regards
Didier


Don't you need to `mbedtls_free( cur->p );` first?


Semver requirements apart I think it's better to catch the exception too.


je ne vois pas ton code la


I'm not a big fan of returning `WANT_READ` here because it embeds in this function logic that should be external to it. See comments at the call site.


It looks quite similar to ergoemacs shrink whitespaces function: https://github.com/ergoemacs/ergoemacs-mode/blob/3162f9a039fca97abfef39e02b5ff8aeb8f92649/ergoemacs-functions.el#L1159


Yep, you can skip java doc for private methods.

On Sun, Nov 1, 2015 at 10:03 PM, Paul Caser <notifications@github.com>
wrote:

> I thought private methods don't require javadoc and comments are enough for this
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/desile/java-junior/commit/c671ba252ddd1506456e4ff0299824eabc7d772f#commitcomment-14115775


@alwey: Thank you for all that information. I would like to discuss few more things with you:

* I think proportionality with d is good for perfect engine. For real engine I'm thinking about principle of OverThinking known from grandmasters' play. Even grandmasters can blunder and the main reason is that they were thinking too much. So, I think that dependence on d should first increase and then decrease. Try to draw a function (d-1)*pow(0.96, d-1) and you will understand my idea.

* K[pv][0]=0.5 could be included in rounding system like int (r+0.5),
  and the difference between PV and nonPV (if needed) should be handled in another way.

* Removing dependence on TC is the main task. If someone can do that he will solve the problem completely.


@tylerhyperHD please remove /smite messages & keep the messages like it was.
I don't like the reason part on smites.


вообще это тоже можно в стриме сделать, но читабельнее не станет :)


I'm aware that, text-wise, this file is almost a direct copy of ControllerWin.cs. That was by design. They reference two different GamePad classes, one that is nice and only works on Windows and another that I made that works on OSX and Linux, with fewer supported features. It was simply easier to design a GamePad class to mimic the plugin so that I could copy the code and have it work just as reliably.


http://search.maven.org/#search%7Cgav%7C1%7Cg%3A%22com.google.guava%22%20AND%20a%3A%22guava%22 версии вот тут обычно смотрят


версия уже указана в паренте, тут не надо


ну и да, лучше версию указывать в паренте, в секции депенеденсиМенеджмента. Чтоб было одно единственное место где видно версии всех артефактов проекта.


Name nitpicking: other related functions tend to start with `ssl_hs_queue_`, it might be better to have the same logic here (makes it easier to grep/explore the code for all related functions).


mmm queremos tener estos directorios? me parece se deberian generar... ya que el nombre de la plataforma no siempre es la misma.


没碰过 help2man 以至于看不到全文的只有你啊（


This commit will change HTML and CSS code with proper indentation.


Manchmal wäre es so schön, wenn man jemanden fragen könnte... :(

Ich kenne leider niemanden, der sich damit befasst. Daher würde ich mich auf die Übersetzungen in bekannten (und viel genutzten) Anwendungen verlassen. Ich hatte eigentlich schon den Eindruck, dass es sich dabei um feststehende Begriffe handelt...
Ehrlich gesagt, würde ich meistens eher den Fachbegriff als irgendeinen laienverständlich umschriebenen nehmen, einfach weil sonst gar keiner weiß, was es bedeuten soll - der Laie kann es nicht einmal nachschlagen, und der Fachmensch wundert sich, was sich wohl dahinter verbirgt.


Exactly, and I do want an SDC SDK for the JVM :smiley:

I'm an Intellij user as well, so if you're able to setup workflows across github repos then I'm sure you can point me in the right direction.  But I've never had the luxury of working on a team that was using Intellij across the board, so depending on maven has always been the way I've done things. :wink:


Wow this typo has been here for quite some time before you found it to change it.


should’ve been `.nanos` … I fixed it while doing the manual merge of #18826


What is this?


Why does the event handler need unique_ptr?
It is part of the object tree.


En ignorant les .class (c'est bien pour ceux qui se trouvent dans /bin des projet Eclipse), tu ignore aussi les .class pourni par le prof, pour lequel on a pas les sources


Typo "withn" should be "within".


De plus, chaque projet Eclipse se trouvant dans le dépot peut avoir son propre .gitignore, qui s'occupera d'ignorer uniquement le dossier bin, et non le dossier avec les .class importés ;)


So it does! I've been trucking around various versions of this shrink
whitespace code in my emacs.d for almost 10 years. I tried to find more
about the original that I got off the emacs wiki from my git history before
publishing this package, but couldn't find any information. I bet ergoemacs
got it from the same place! I'll check this out and give credit to the
original author if I can find him/her.
On Nov 1, 2015 2:14 PM, "David Capello" <notifications@github.com> wrote:

It looks quite similar to ergoemacs shrink whitespaces function:
https://github.com/ergoemacs/ergoemacs-mode/blob/3162f9a039fca97abfef39e02b5ff8aeb8f92649/ergoemacs-functions.el#L1159

—
Reply to this email directly or view it on GitHub
<https://github.com/jcpetkovich/shrink-whitespace.el/commit/cead003dffa14736c9cc74f426edbf469ed71741#commitcomment-14115812>
.


@eyalbe4, that works for me.


Yeah, noticed it as well https://github.com/akka/akka/pull/18824#discussion_r43586108
Sorry for merging before the "big one", thanks for merging https://github.com/akka/akka/pull/18826 before more merge trouble


The `<condition>` is unnecessary here because you already have added the `<property>` trigger in the XML sound file.


Ah, jetzt hab ich nochmal genauer geguckt, das ist ja die Beschriftung des Reiters mit den vielen neuen Textmöglichkeiten - alles klar!


@Borovikov it's the way to publish author's week on the site http://backendsecret.ru/borovikov/


Ich glaube, das ist undefiniert, dass man einen ally hat ohne Partei. Also, das sollte quasi eine Invariante sein, dass ally::faction != null. Wenn dem so ist, dann ist ein check wie der hier eher verwirrend, weil es dem Leser den Eindruck vermittelt, dass es Fälle gibt, qwo es nicht so ist.

Mit anderen Worten: assert(sf->faction) hätte es hier evtl. auch getan?


Let's go with @alexfedoseev recommendation


why not returning position * 500


Done


This needs to be a `<product>`. The volume in the XML sound file needs to be a value between 0.0 and 1.0. Note that the volume boost doors give a value between 1.0 (no amplification) and 1.4 (40 % extra amplification). So `dist1` needs to less than or equal to 0.7142857142857143 (1/1.4). (Because 0.7142857142857143 * 1.4 = 1.0)


Good suggestion, done.


Changed to info (did not mean to put this as warning).


Done.


@VisualKei как думаешь, оставить ресницы такими или сделать их тоньше (например 1 пиксель)?


@lots0logs : I'm afraid it is not. I got this:
```
oct 31 19:26:18 testing systemd[1]: systemd-modules-load.service: Main process exited, code=exited, status=1/FAILURE
oct 31 19:26:18 testing systemd[1]: Failed to start Load Kernel Modules.
oct 31 19:26:18 testing systemd[1]: systemd-modules-load.service: Unit entered failed state.
oct 31 19:26:18 testing systemd[1]: systemd-modules-load.service: Failed with result 'exit-code'.
```
and this:
```oct 31 19:26:18 testing systemd-modules-load[140]: Failed to find module 'zfs'```

I had dkms disabled, so I enabled it and reboot. No go.

Finally, I've managed to make it work running this (it also builds spl automagically):
`sudo dkms install zfs/0.6.5.3`

Problem is that I wanted this for Cnchi, so compiling the package is a no go. Could you easily setup our ISO to also have zfs?


Creates a ```$scope.user``` cookie actually.


you forgot to remove "if [ "$logger" -ge "1" ]; then" condition inside post-init.sh script, my friend!


debug:
--> Loot-System: synchronized possible thread collision
--> Chest: no thread collision, at (very fast) double click
--> AttackingScreen: strukture, ...


сиНтезировать


Again, "wthin".


I can see some double logger.Trace and logger.Debug for the same info. Guess its better to have it a one level only. There was a few places where the same info was written both to the console and logger.

I do not think COMPortInfo.cs needs to be changed.


учитываЮщий


Nice !!!


loading process even before PreWindowScreen is now ~15s faster.


1st try to roll the commit into the master


oops! Mistake on my part. StopModReposts is a different mod of mine. I gave the wrong URL.


No problem, I too did one line of sed and it was all right again :)


Wrong package means that i have to use static import?


This wraps up electrostatics for 0.1. DSF and Debye Huckel. Closes #25.


Yes, good point. Biggest problem was that I didn't know what exactly to assert. I have tidied this up now and put a more detailed comment in.


ok, thanks,  i will clean it.


Looks like you misspelled "canola".  Please check.


It's not clear to me what these are for or what 'NF' means.


Bro, spelling mistake!


Those .*s don't exist on our branch, and haven't for a while... just saying.


I know you don't change the variable name from before but in fact `open` doesn't return a filehandle, it's a python builtin `file` object^^


Indeed, this is ugly. It will have to work for now, but will look into it soon. I [asked a question on StackOverflow](http://stackoverflow.com/questions/33458256/expect-that-all-elements-in-a-seq-throws-exception-in-specs2), and got a good answer from the creator (or maintainer) of Specs2. Will look into his suggestion, it seems reasonable.


Testing writing a comment
# Markdown.


Ah, in that case I don't really know where they came from. Might've been from one of us.


Good job!
Here is some advice:
1. It seems that when I have select one piece I am not able to switch to select another piece.
2.
![screen shot 2015-10-31 at 10 02 20 pm](https://cloud.githubusercontent.com/assets/10523180/10866874/4714ce80-801b-11e5-8dad-8bc8617c6201.png)

 In this situation I am not able to jump from place 1 to place 2.
 I'm not sure but maybe we have some different understanding of the rules of the game.


See fixup commit^^


change username


Good job.

I think it will be better if you can do some highlight effect on the piece when I select a piece.


I am now reasonably convinced there is nothing wrong with this patch.


Good Job


I can't integrate the existing one now and a name change later is superfluous imo. Open an issue for integrating the existing class into this one and as soon I come up with more features for this `documentation.DocumentationComment` I'll fix it^^


@lynicidn а в чём проблема? В чём смысл оформлять всё в пр?


I would feel strange putting my name as author and copyrighting it to Microsoft. Is that what you suggest though?


If the end-marker is `#` it would immediately stop at the first occurrence, because that would signal the end^^


O trecho retirado não impede a funcionalidade do programa. Estava ali desnecessariamente.


Yeah maybe since you suggested not to auto left-strip documentation, I'll consider that^^


I didn't mean to add this to my pull request. Oh well, no problem.


:pray:


The inlined function is a generator, the outside not. So it starts checking stuff and raising errors if necessary. The actual extraction part is then performed inside the inlined function which yields^^


Thanks!!


see below


this function is public API, so it doesn't hurt to yield them. Creating a list doesn't make the code simpler btw^^


Will look at it^^


Hi Josh - Thanks for publishing this awesome project! I'm really enjoying using it.  I have a question, though: when I run the branchingDLA_2Dv1.py file, it runs very well.  However, I'm trying to replicate the models that you ended up fabricating, and I can't seem to do it.  None of the other .py files run, except branchingDLA.py.  I've created the 'seed' mesh requested during the run of testGrowRegion.py, but after that I don't know what to do.  Could you point me in the right direction?  Thanks so much again - I'm really enjoying watching these little models "grow" in rhino, and trying to learn Python at the same time!  - Ana


continuous mode and standard mode aren't mergable^^


maybe simple mode is integratable into standard mode (because of your suggestion not to strip lines in simple mode)


На самом деле на SVG переключил все иконки, просто некоторые не перевёл в вектор, а оставил какие были.


:heart: :fire: :fire: :fire: :heart:


not find


guys, please, i want so much that att for the hallowen!!


You should use "Daniel Selsam" as the copyright holder of the files you have created. This is what Jeremy does in his files.


No problem.


What's the point of the while(true) and the if block checking if count % 10 == 0 and then setting count to 0?


This link is broken. (200 OK but empty content.) Some of their own search results on their site do this too; maybe it's temporary...


... except during the month that I was a Microsoft employee, in which case, I assigned the copyright to Microsoft.

If multiple people work on the file, all of them could share the copyright, but one copyright holder is enough.


Saxon,

Sorry, I didn't realize that the changed file wasn't in the archive.
I'll check then pi as soon as I have a chance.  However, I think it's
just a matter of adding the baud rate.
case500000: myBaud = B500000 ; break;

https://github.com/Wyliodrin/wiringPi/blob/master/wiringPi/wiringSerial.c

There's also a forum post here to that effect:

http://www.dexterindustries.com/topic/brickpi_c-and-current-wiringpi-library/

Thanks,

On 10/31/2015 01:42 PM, Saxon wrote:
>
> What modifications did you have to make to pi4j to support a baud rate
> of 500000? I'd like to PR that change against the library.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/13eakers/BrickPiJava/commit/2e7540382c5b89fb1ad978476364e166aa12a5d8#commitcomment-14110770>.
>

--
Jim Redman
(505) 662 5156 x85
http://www.ergotech.com


I won't use this, because as I see it: as minimum Parts are completely optional and as maximum they are only assembly abstraction, so require them unconditionally for primitives is not an option...


Thank you for the kind words, Zeleng

___
Vikram Mullachery

> On Oct 31, 2015, at 10:10 PM, Zeleng Zhuang <notifications@github.com> wrote:
>
> Good Job
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/mullachv/Kalah/commit/7d4449b35a2a83067a4b93a2620f5a3356fee980#commitcomment-14112034>.
>


same as previous


ㅋㅋㅋㅋ
시간나면 좀 하겠습니다


we could also remove those two lines I believe


these lines


Take a look at @LDX-MCPE/VoteReward/plugin.yml and you'll see I already have plans to fix this.


Makes the dog bounce up and down haha


LGTM


The internal compiler error when trying to build flightgear for x86_64-musl is certainly strange.
Building flightgear [worked for me on my box](http://sprunge.us/iCdX)...


Oh hahah, nice touch


Great job.
Very nice UI


Nice job!


Okay I will tomorrow.
On Oct 31, 2015 7:53 PM, "Arthur Pachachura" <notifications@github.com>
wrote:

> Create a pull request with the base being feature-walldetection and the
> compare being dev. Then, wait for unit tests to pass and merge it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lasarobotics/FTCVision/commit/a7c3d6772b0db10947bf638441887eeb9921ce8a#commitcomment-14111845>
> .
>


It's better raise `NoPartitionsForConsumerException` in  https://github.com/Parsely/pykafka/blob/master/pykafka/balancedconsumer.py#L435 ?


Sorry to keep nitpicking, but in current versions of julia this is really slow. I can see the readability aspect, but is `string('%', uppercase(hex(x,2)))` really that bad?


Hey Sean, these commits are amazing. We all love what you did with the gallery, and the logos. Thanks for working so much at home!

BUT......

nah no but this is awesome keep up the good work


I just need more pictures to add to the gallery

On Sat, Oct 31, 2015 at 7:41 PM, Pranav Mathur <notifications@github.com>
wrote:

> Hey Sean, these commits are amazing. We all love what you did with the
> gallery, and the logos. Thanks for working so much at home!
>
> BUT......
>
> nah no but this is awesome keep up the good work
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Thunderbots5604/Thunderbots5604.github.io/commit/db00c8a23505abbe8ab7d1d13fa770c666f54692#commitcomment-14112114>
> .
>


I just did some quick and dirty to get it to work, we can change it later


Should we make it go away in engine again then? Or does it have any potential value somewhere?


Quick Question, if one of us wants to add more photos the Gallery do we just add them to /images or do we have to edit some HTML/JS as well?


You have to edit some HTML
Just leave JS alone

You would add it, rename is to 4.jpg or 5.jpg depending on the number
Then you would copy from
 to
 under wrapper and sb-slider
Give it a description
Then add another  under the

IT adds another dot

On Sat, Oct 31, 2015 at 7:43 PM, Pranav Mathur <notifications@github.com>
wrote:

> Quick Question, if one of us wants to add more photos the Gallery do we
> just add them to /images or do we have to edit some HTML/JS as well?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Thunderbots5604/Thunderbots5604.github.io/commit/db00c8a23505abbe8ab7d1d13fa770c666f54692#commitcomment-14112124>
> .
>


No, you're right of course. The old macro version expands to an unpipelined version so I just got careless.


skipped because most of it doesn't needed.


If you do ```static if(__VERSION__ < 2067L)``` we won't need to import std.compiler. Not sure which one is considered "better" so I'll leave it up to you.


Can @phpbb/native please have a look at this message? Something about it doesn't feel right to me...


I'll only write this here in stead of every time this happens.

There are a couple of times you do this, but you don't do it to every class(Like SoundStream). I have no problem changing it to this, but we should decide on which one is better so we're at least consistent throughout.

I can take care of going through the rest of the library tomorrow and making sure all classes are "the right way" since I am going to upload some code tomorrow anyway, so do we want to have it done with the spaces?


Great job


Euhh I misrebased it, one moment I'll fix


Should be com.acme

On Sun, Nov 1, 2015 at 10:56 PM, Paul Caser <notifications@github.com>
wrote:

> Wrong package means that i have to use static import?
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/desile/java-junior/commit/ca5380c305f030a531e2308834a201461408a6d7#commitcomment-14115948


This is an old commit. I've amended it.


Url_shortner using API


Hi @rellan - how are things going? Thanks for checking out the Study Group repo; do you have any questions? If there's anything I can do to help you get a meetup started in your town, just let me know!


TeamCity LabkeyTrunk :: Git sqlserver 2014 [Build 31](https://teamcity.labkey.org/viewLog.html?buildId=281848&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:22


@insertc0in thanks! :D


@igorid лишнее


@igorid зачем здесь обтекание, если ширины 100%?


Cool - so the remaining 3 lines can now be replaced by

return _civicrm_api3_basic_create(_civicrm_api3_get_BAO(__FUNCTION__), $params);

(I think it preferably takes a couple optional params - I copied that from 4.4)


@tumbili may I ask if this is a fix for some problem you found? If so, I would like to add the same change to my fork.


Number ;)


> ‘zo'n drankjes’ → ‘zulke drankjes’ (since I think ‘zo'n’ sounds strange with a plural noun)

Indeed, this is a very common mistake in Flanders, sorry!

> ‘Gravitatieverschijnselen’ → ‘Gravitatievelden’ (since the original says ‘Gravitational Fields’)

Yeah, fine. I thought Gravitatieverschijnselen had a better ring to it. :)


I don't have a Facebook account...


Why does it need to be a `UUID`? Any particular reason why an incremental integer was not enough?


Small line changes


Awesome!


Does that make it possible for an anonymous user to clear their
cookies/cache and then re-vote?
On Nov 1, 2015 01:43, "Morgan Aubert" <notifications@github.com> wrote:

> Hi Thomas,
> I realize that preventing anonymous users to vote is not be the best
> solution. I will update the forum_conversion.forum_polls app in order to
> properly handle forum polls with anonymous users (the models of this app
> need to be updated to handle the session keys of anonymous voters).
> I keep you in touch.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thomastu/django-machina/commit/8819f79b5feaf7da494a3af35da2abcfc8ab9317#commitcomment-14113257>
> .
>


Never mind, I see why you did this. I'd prefer for this function *not* to be exported.


That is not the most important bit but creating an entity with a valid ID to begin with.


In english, `Singleton` doesn't take a final `e` :wink:


Affiche HelloWorld


Function name doesn't seem to be in line with other API calls. (GetHex()?)


AciD Part:
http://paste2.org/hHJwa44V


so you would prefer that this function is only in Entity. Sure can be done. I thought we may use this somewhere else but now I don't have any other use case. I can make this private.


s/remove/rename/


Hi. Thanks, I have resolved all issues, code is now in flare-engine master.


Is it intentional that everything is in lower case letters?


так делать не надо


Не, так что-то не очень. Тоньше и главное короче, и 4 штуки как было.
Или может ну их нахрен? Ты ведь тот svg-глаз редактировал, или просто за основу брал? Тот выглядит очень гармонично, а вот твой как-то не очень - зрачок слишком маленький и кривизна век другая что-ли. Может ты просто именно тот вставишь, только с цветным зрачком?


what changes got made?


asdfafer


@josiasds I should have caught this. We should move this code somewhere else.

I'm not totally sure on exactly where, but I normally never see some code like this in application.js.

Maybe a file that gets loaded at the end called `rails_startup`.

CC: @samnang, @alexfedoseev


Out of curiosity actually why does it hurt to have this "utility" method available everywhere?


I'm not yet sure why it is or how to stop it, but when you enter say pumpkins/001 it looks for this: /Users/joelmanning/Projects/TheGame/Mac/bin/Debug/Game.app/Contents/Resources/pumpkins/001.xnbAs a temporary fix for that I had added a symlink to content but I would need to remove the gitignore to push it


Какой тот? Кривизна век действительно другая, потому что мне лень играться с кривыми безье и сделал всё довольно топорно, хотя и похоже.


You can also comment on specific lines of code and reference issues.  I'll reference #2.


More complicated than the commit message indicates:
1. moved 2r/de Pen. D.1 d.a.c.1.txt to 1r/C.33 q.3 d.a.c.1.txt ("His breuiter decursis ... qua queritur: ")
2. edited 1r/C.33 q.3 d.a.c.1.txt to remove trailing space character ("His breuiter decursis ... qua queritur:")
3. added C.33 q.3 d.a.c.1 to toc_1r.txt
3. created new 2/de Pen D.1 d.a.c.1.txt ("Ambrosii super Lucam,")
4. re-generated SHA1 checksum for Gratian1.txt


Á ekki að klára þetta? virkar frekar hrátt


same line number thing, but maybe it is better to change the results indeed.

ack


Hi Christien,

This commit looks good – seems like you have the patron list displaying correctly.

However, please keep your commit messages school appropriate and professional. Remember, this is a public portfolio of your work in the class.

Mr. Gordon


eerk. Can we avoid ignoring more compiler warnings?


that smiley face though


@igorid `ul.sidebarmenu` медленнее, чем `.sidebarmenu`


This looks great, Ryan – nice work.

Ryan – could you please fix your name in your GitHub profile?

Please remember that this is a public portfolio of your work in Gr. 12 CS at RSGC – it should be kept professional.

Thanks,

Mr. Gordon


"<" and ">"  should probably be escaped as `<` and `>`


@igorid зачем добавил здесь `li`?


@igorid это нужно сократить


You could use `RSA_padding_add_PKCS1_PSS_mgf1` which is defined in `openssl/rsa.h`


Please look at the warning before complaining. Those two warnings are here to control/disable record disambiguation for people who are wary of this feature. I personally like disambiguation and find it a good feature, and I *will* use it. Note that I did not disabled warning 41 about *ambigous* labels (the case where disambiguation has multiple choice).


Can the ex be put as parameter to enable logging of the exception, too?


@matiwinnetou what he meant was: it's OK to use this for `engi` internally (so `Entity`, but all other `engi` stuff as well), but not have it exported because it's no use to the external programmer. And as such, we could hide it.


Excellent work here. I would not have been able to come up with better work for prototype.


Removing Debug code in production is a good practice.


mmh that's right ^^ I've been a bit hard.


С викимедии который. Ну с кривизной ладно, но сделай зрачок больше. У того просто идеальные пропорции, а у тебя он меньше и получается слишком много пустого пространства, глаз как на спидах какой-то.


@kirbyfan64 ```looking for cl.exe       :
ok c:\LLVM\msbuild-bin\cl.exe
checking cl.exe          :
MainThread: starting 'c:\\LLVM\\msbuild-bin\\cl.exe /nologo /Zi /IC:\\DOCUME~1\\Phase\\LOCALS~1\\Temp\\tmp82ew41dw C:\\DOCUME~1\\Phase\\LOCALS~1\\Temp\\tmp82ew41dw\\temp.c'
 + c:\LLVM\msbuild-bin\cl.exe /nologo /Zi /IC:\DOCUME~1\Phase\LOCALS~1\Temp\tmp82ew41dw C:\DOCUME~1\Phase\LOCALS~1\Temp\tmp82ew41dw\temp.c
clang-cl.exe: error: unable to execute command: program not executable

clang-cl.exe: error: linker command failed with exit code 1 (use -v to see invocation)
 - exit 1, 5.58 sec
failed
cannot find a c static builder for frozenset({'gcc', 'win32', 'clang', 'windows'})
```


Ugh, ugly.


Так что с ресницами делать? Сделать тоньше? Добавить ещё по одной сверху и снизу? Убрать совсем?


LOGGER.debug?


`, ex`


sry :(


ioe as 2nd parameter, not a concatenation?


ufe as 2nd parameter, not a concatenation?


@JuliaBackports


`e` as parameter, not `getMessage()`?


`ex` as parameter, not `getMessage()`?


`ex` as second parameter, not `getMessage()`?


`e1` as second parameter?


`e` as second parameter?


`e` as second parameter instead of `getMessage()`?


Level `debug`?


why is there a "+" in ## comment


alright, I'll word my comment differently then: the current ExecutionEngine only really makes sense when referring to MCJIT, so it is should not be used externally. as we experiment with alternative JIT clients (aka OrcJIT), it may go away. at the same time as I removed the DLLEXPORT, I added an alternative api, `jl_finalize_module` to replace some aspects of this in a more flexible manner. (e.g. the change to `DLLEXPORT JuliaOJIT *jl_ExecutionEngine` is really really awful since `JuliaOJIT` will never be a subclass of `llvm::ExecutionEngine` and is (intentionally) not suitably defined for use outside of libjulia)


Попробуй тоньше и короче, сейчас они слишком выделяются. А вместо 1 по центру 2 по бокам от него, как было - а то сейчас именно точка перегиба сильно акцентируется и ещё увеличивает кривизну век. Ну а если всё равно будет криво, то ну их нахрен. Лучше наверное сначала запости картинку, что получается.


err, aren't approx_invSqrt and fast_invSqrt the same now?


Yes, that's fair enough and at such a time we can figure out what the finalized API will be (I suspect it will just expose adding a function or a module to the execution engine), but for now at least the MCJIT execution engine needs to be exported.


That's because all content resources are copied over to the actual game when you build it. Just set the source content folder as Content and select Copy if newer. That means it will be copied over to the actual game app when you build.


Thanks for clearing that up @EtienneBruines.


xltxtra is loaded by fontspec anyway. xunicode should be needed for
backward compatibility.

But I am able to produce the same result without loading either anyway
when compiling with "xelatex".

--
Hugo Roy – Free Software Foundation Europe https://fsfe.org/about/roy

Please use cryptography for email: see https://emailselfdefense.fsf.org/en/
Merci d’utiliser la cryptographie pour l’email : voir https://emailselfdefense.fsf.org/fr/


I can't find the * in the extended IUPAC alphabet: http://www.bioinformatics.org/sms2/iupac.html.
Maybe we should leave it out?


results with MPB'15:

On Sun, Nov 1, 2015 at 12:39 PM, Heike Hofmann <heike.hofmann@gmail.com>
wrote:

> These are the results from my MBP with p=0.2
> totally different.
>
> I'm just running things with a new MBP from 2015 (graphics card still
> Intel), but the results seem to be closer to yours.
> This is weird.
>
>
>
> On Sun, Nov 1, 2015 at 11:41 AM, François Briatte <
> notifications@github.com> wrote:
>
>> I cannot see your results (no image), but my graphics card is an Intel HD
>> Graphics 4000 from April 2012
>> <https://en.wikipedia.org/wiki/Intel_HD_and_Iris_Graphics#Ivy_Bridge_2>.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/sctyner/ggnet-paper/commit/19991a2bc71113c39663b3d95e91ef4394054de8#commitcomment-14115458>
>> .
>>
>
>


...I added an alternative api, `jl_finalize_module` to replace some aspects of this in a more flexible manner...


Is this ES6 shorthand?


That tool needs to be located in one of your $PATH directories.


white-space error?


@johnda102 What's the story with this?  I'm running with the latest p-tag for EXOT2 derivations (p2432), and PileupWeight is still a float (and therefore this change crashes).  What samples are you running on that has it has a double?


Thanks! I hope it's useful for you.


Commented [here](https://github.com/shakacode/react_on_rails-generator-results-pre-0/pull/6#discussion_r43587108).


@micha could you explain why the `with-eval-in` thing works whereas plain `.require` doesn't?


Nice job


DK_Taulov_1 rlv video..
I was "testing" video near 22-23km (small "hole").. It showed altitude 0..20. I haven't noticed that starting altitude is 0 (and "hole" is at 0 as well..).
It works ok, my mistake :)


Auch wieder wahr. ^^ Ich änder das.


We might get a merge conflict from this


Yes, thanks.


Prioritizes*


Yes, you are right. Will be the same and the code will be more clean.


@cala Yes, because it requires testing which I cant do. Therefore I added this and others to the backport todo list: https://github.com/cmangos/issues/wiki/TBC-Backporting-TODO-WOTLK-Commits


If there's a crash because of an inconsistency in type definitions -- we might need to raise this up to the derivation folks and find out what's going on. I'm surprised that the float/double will crash given that a compiler can implicitly cast it.


@johnda102 @mmilesi  My mistake, actually has nothing to do with the input, but with another call to this in different code.  I see the chain from the last email, sorry for the noise!


uuummmm, did you know the built-in `dir` method?

strings of (public) attributes:
`[attr for attr in dir(obj) if not attr.startswith("_")]`

actual (public) attributes:
`[getattr(obj, attr) for attr in dir(obj) if not attr.startswith("_")]`

example:

```python
>>> class P:
...   def __init__(self,x):
...     self.__x = x
...   def getX(self):
...     return self.__x
...   def setX(self, x):
...     self.__x = x
...   x = property(getX, setX)
...
>>> p = P(2)
>>> dir(p)
['_P__x', '__class__', '__delattr__', '__dict__', '__dir__', '__doc__', '__eq__', '__format__', '__ge__', '__getattribute__', '__gt__', '__hash__', '__init__', '__le__', '__lt__', '__module__', '__ne__', '__new__', '__reduce__', '__reduce_ex__', '__repr__', '__setattr__', '__sizeof__', '__str__', '__subclasshook__', '__weakref__', 'getX', 'setX', 'x']
>>> [attr for attr in dir(p) if not attr.startswith("_")]
['getX', 'setX', 'x']
>>> [getattr(p, attr) for attr in dir(p) if not attr.startswith("_")]
[<bound method P.getX of <__main__.P object at 0x7fd08d21b518>>, <bound method P.setX of <__main__.P object at 0x7fd08d21b518>>, 2]
```


ack


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 769](https://teamcity.labkey.org/viewLog.html?buildId=281850&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 20, passed: 240, muted: 23 Build time: 00:42:26
### Failed tests
```
org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStudyParticipantCounts: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSearch: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilterPane: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

##### there are 10 more failed tests, see build details
```


умный воробушек, жаль что нервный, но он может со всем справиться, имея под боком собственную мышь :)


@martinklepsch I don't know. I suspect that perhaps the pod isn't ready when `.require` is called on it. I'd need to get deeper into ShimDandy to really figure it out.


Fixes second part of #27.


Yes. An anonymous user can vote again if his session key changes. ;) In order to limit possible flood problems, I will soon document how to protect post forms and polls vote forms with re-captcha fields in the documentation.


Chapeau !


generally, you'll want to set a pixel `max-width`. The most common sizes for a `container` `max-width` are probably 960px and 1020px.


Sure! Should that always be the case?


That's not the point though. This change broke Cxx, which people now depend on, so we shouldn't just leave it broken. At the moment I do not have the time to look through how the ExecutionEngine is used in Cxx and Gallium, but I will revisit the issue once we know what the JIT API here looks like.


@VisualKei
![screenshot_20151101234848](https://cloud.githubusercontent.com/assets/1163958/10871261/1e237606-80f3-11e5-9755-228a2386bb0d.png)


"Your account" instead, shouldn't it ?


You dont have latet version. 0.8.6


Thanks a lot :D

On Sun, Nov 1, 2015, 03:56 David Aurelio <notifications@github.com> wrote:

> just accept a Promise as last parameter in your bridged method (instead
> of the usual Callback):
>
> import com.facebook.react.bridge.Promise// …
> @ReactMethodvoid bridgedMethod(int parameter, Promise promise) { /* … */ }
>
> that’s it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/facebook/react-native/commit/b86a6e3b44a63e92cf3a7976d2fa26c4bf412df1#commitcomment-14111390>
> .
>


Could make sense. There are quite a few commented print-debug lines in the code. Some, like the above, which maybe could be "upgraded" to warn or info, some of the more common print-debug style, printing variable values and branch status etc.


It is probably not a good idea to use a pixel `width` here. In order to make sure our sidebar and posts are in two layout, you need to ensure that their combined `width` is 100% of their container (in this case _the_ `.container`). This is most easily achieved with percentage widths for both the `posts` and the `sidebar`


Could we change this to make it Cloud Foundry OSS?


I haven't checked any of the commented out print-debug commands.


Sorry, still not seeing the pictures.

I'm opening an issue to better track the problem(s):

https://github.com/sctyner/ggnet-paper/issues/4


You’ve been very busy, Oliver! Nice!

Can I get a demo sometime in the near future? Looking forward to seeing your progress.


Yes, you are correct. Sometimes the conversion is quite automated. :-)


bugs fixed


I don't quite understand what dir is about, can you file a bug for this? This commit is merely a move out.


Don't forget to write about the complexity of Prim's using complexity.py


@matklad @alexeykudinkin how can I implement this now that #57 is merged? I don't see a way to get to the outer attrs for the `RustFnItem` now.


@dengert do you want to make the PR? @darconeous seems to have lost interest...


Kamila Jambulatova


There's still a double quote in lines 21 and 29.


done!


Да, вот это уже годнота, выглядит отлично.


Having the script build OpenCV in a somewhat portable manner is going to be a pain. Because we're really only going to be using this to generate the cascade for the beacon, and thus other teams are unlikely to actually use this themselves, I just prompted the user to install it if the tools aren't detected. If anyone really wants it to automate the build I can have it do so, but it seems unnecessary.


I totally missed that you updated that wiki entry, sorry.
Great work ! I'll see if I can help with some.


is this like an helper message that shows what kinds of tests will be running but not really running them? If that's the case, can we clarify it here and replace "--show-only" with "-h"?


@justinmk I think this is wrong, see https://www.reddit.com/r/neovim/comments/3r4ciu/neovim_010/


`InjectFunctionProto` is not present on release-0.4. If the bug exists there too, you'll need to find the right place to put it in the 0.4 version of codegen.


OK, überzeugt. Die "bedingte Ligatur" scheint tatsächlich ein feststehender Begriff zu sein, das hatte ich bei meiner ersten Rechercherunde nicht realisiert.

Allgemein scheint in diesem Bereich viel Halbwissen verbreitet und oftmals auch Begriffe durcheinandergebracht und vermischt zu werden. Ich hoffe in 464c54e9846380293589b35eb1e51dce47469cd2 passt es jetzt einigermaßen.


Have you tested this?


the method basically does what you are doing: get the names of attributes of an object. Submitted issue as #1024


mh, I like using tabs. This makes a Dockerfile more clear.


Also, you wouldn't mind if I rewrote Prim's a bit? You mentioned it can look better and it would be neat to include an interface so that Drawer works with your implementation as well + plus get some more hands on experience with Prim for next week.


so this is basically ack then for now.


Oh dont worry about it,its the extra file that android studio makes, if I dont specify the changes its that :)

> On Nov 1, 2015, at 15:12, retwalker <notifications@github.com> wrote:
>
> what changes got made?
>
> —
> Reply to this email directly or view it on GitHub.
>


if we do ctest -R triad, it's only gonna generate the trace?


Why not just use field instead of array?


$rootScope.$broadcast('reloadUser', {});


Hah, you're right. For some reason I thought I had backported this change, but I'm glad I didn't!


Maybe it'd be easier to read it this way?:

while !found && tryAdvance {}
return found


I certainly see the reason why one argument is better than two here - but `geographiclib` may not be installed (and in that case the obspy Vincenty implementation ought to be able to handle any combination of radius and flattening.

It's also a good idea to avoid reimplementing all of `geographiclib`. I think this interface needs some thought.


rootScope


By the way, I just create the java-http-signer repro: https://github.com/joyent/java-http-signer

You've been granted write access.


Thanks, that helped me a bunch. For some reason my $PATH was missing a :


alright


This is a placeholder image for the team logo - passed the test.


do you think it's correct? this code would increase position every time even when limit had been reached already.


не буду коментувати


maybe --skip >= 0 ?


...Waarom wist ik dit nooit eerder lol.


- ~~δομών~~ δομές


I guess we should also push a follow-up commit that re-sets PRERELEASE asap, otherwise multiple commits will report "version 0.1" in --version


Hi Ethan,

What issues are you having with hit detection?  Try to be specific.

On Monday, I'll show you how to use commit messages to auto"magically" create GitHub issues, which can help you keep track of bugs and "to-dos" with a project.

Take care and see you then,

Mr. Gordon


We need to use gwt's InternalPrecoditions for these checks (statically imported)


ні не брєд. мені якось не дуже хочеться запам'ятовувати всі айдішки статусів (а їх багато)


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.56](https://pixelated.design/build/viewLog.html?buildId=185&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 273; exit code 1 (new) Build time: 00:00:10
### Failed tests
```
Lobster::Network::Message: #new: with invalid parameters: with invalid parameters raises an error when given a name that is not a string: <no details avaliable>

Lobster::Network::Message: #new: with invalid parameters: with invalid parameters raises an error when given a nil name: <no details avaliable>

```


Commit messages are more useful to you later when they are specific – what did you accomplish with this commit?

("work done in class") won't mean anything to you a few weeks from now.


συνεπάγεται **με** *σχετικά*(;) σύνθετη λογική


не розумію запитання


i don't think we need PRERELEASE anymore. We can use `git describe` to set the version and commit for non-tagged builds.


left, right or a, b for consistency.
instead of -comparator, you might use Comparators.reverse(comparator)


@jazck is taking credit for this piece of code, although he wrote it with Jacob. We thank @jazck for taking the fall for this terrible piece of code's problems.


But yeah, the reddit comment is right, can't believe I messed this up. That's what happens when multitasking.


Mmm, I might be confused.

Correct (server uses client-generated URI for P3, but own URI for P4):
```
<http://example.org/example?page=3&s=a> hydra:next <http://example.org/example?s=a&page=4
```

Incorrect (server uses client-generated URI for P3, and imitates this for P3):
```
<http://example.org/example?page=3&s=a> hydra:next <http://example.org/example?page=4&s=a>
```


що зміниться?


must have been a typo


Use a reduce better


You did a nice job of making sure the site looks good on mobile. That being said, I have a few pieces of feedback in addition to the ones I left above that I think will help going forward:

1. Your two-column layout doesn't seem to be working on bigger screen sizes. The reason for this is what I mentioned above. Using pixel widths and percentage widths together is possible, but becomes increasingly difficult as we try to make things responsive. In order for two-columns to exist side-by-side, they need to equal 100% of their container element. If you use pixel widths for that, you need to be able to calculate the exact pixel widths that would add up to 100% (say, 960px for example) and then divide that amongst the children. This is a long-winded way of saying...just use percentage widths for columns in a layout. It will save you a lot of headache and time.

2. I noticed about half of your media query styles are adjustments to font-sizes. That right there is the case for `em` or `rem` font-sizing. If you use one of those approaches, you would need to only change one font-size and that would trickle down to everything that is styled in `em`s or `rem`s. If you have any questions about how that works, let me know and I'll explain more.

#FEWD Week #4 Project: Responsive Design

---

###Description

Relaxr's user count has grown, and the number of users accessing the site from a mobile device has surprised everyone, even by today's standards. The product managers at Relaxr have asked you to return to the project and make their site responsive so it renders nicely on mobile devices. They had one of their internal developers make a few tweaks to your source code, and they have given you the new HTML / CSS files to work from. Using these starter files, build out the mobile layouts for the landing page and blog.

Student: Andrea Stafford

|                                                                                |                                |                        |                          |
|-----------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| Technical Requirements                                                                        | Does Not Meet Expectations (0) | Meets Expectations (1) | Exceeds Expectations (2) |
| Use media queries at a 768px breakpoint                                                       |                                |                        |                 X         |
| Content remains in appropriate ```divs```                                                     |                                |                        |               X           |
| Content remains visible to the user                                                           |                                |                        |                X          |
| Fonts change size appropriately                                                               |                                |                  X      |                          |
| Margins and padding change size appropriately                                                 |                                |       X                 |                          |
| Content does not overlap                                                                      |                                |                        | X                         |
| In the blog section, the two-column layout changes to a single-column layout when appropriate |                                |         X               |                          |
| TOTAL: *11*/14                                                                              |                                |                        |                          |


@phase What if you run `vsvarsall.bat`?


go fuck yourself

On 11/01/2015 10:11 PM, Rafa Gómez wrote:
> Use a reduce better
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mariocav/dstat/commit/62bb68fcaac2b0e700b37ec220fa83d0a250a763#commitcomment-14116368>.
>


This looks like a solid addition but please also support iOS specific payload keys - see the list here: https://developers.google.com/cloud-messaging/http-server-ref#notification-payload-support


list.sort(comparator)


В данном случае скрытый конструктор акцентирует внимание пользователя класса на том, что он должен быть использован исключительно как "статический" класс.


But do we want to provide the travel time, and ray parameters we can calculate even if `geographiclib` is not installed?


Από τα τρία loops που παρέχονται από τη Sass, το `@each`loop είναι σίγουρα αυτό που χρησιμοποιείται πιο συχνά. (νομίζω συντακτικά στέκει καλύτερα έτσι στα ελληνικά)


Address your problem by edit your comment :)


Haha good hack though :D