Are these files all currently tagged with `ignore-license` as we would have to get permission from the contributors to add the current license to them?

A bunch of them may actually just be written by me if it's worth investigating (it probably isn't)


If you are going to bother trying to fix this, get rid of the version and use the permalink
https://github.com/cesanta/mongoose/archive/master.zip


Nit: this feels backward now. It'll read better if we switch sides. Not a perfect yoda condition but pretty much.


@deennoo: Thank you for the added support for the BTHGN129!

Are you sure the BTHGN129 uses the same encoding as the BTHR968? [This code](https://github.com/contactless/domoticz/search?utf8=%E2%9C%93&q=sTypeTHB1) treats it the same as the BTHR918, but I cannot find where the actual data is coming from.

The reason I am asking is that the BTHGN129 has a 5-channel setting that is not captured by your code and that rtl_433 with your changes produces a pressure reading higher than metar for this sensor, which I just purchased. I do not have a display unit, so I cannot know what the decoded value should be, but right now, it is reading 1061 mbar when metar reports 1017 mbar.


Agreed. This was difficult to abstract when the client was introduced.


> Note: This property is not yet part of a W3C CSS working draft or recommendation. It determines the resolution of a device's screen, and describes the aspect ratio of the output device. This value consists of two positive integers separated by a slash ("/") character, and represents the number of horizontal pixels over the number of vertical pixels.

http://www.opera.com/docs/specs/presto28/css/o-vendor/

```
-o-min-device-pixel-ratio: 2/1 /* => 2 */
-o-min-device-pixel-ratio: 13/10 /* => 1.3 */
-o-min-device-pixel-ratio: 3/2 /* => 1.5 */
-o-min-device-pixel-ratio: 5/2 /* => 2.5 */
-o-min-device-pixel-ratio: 3/1 /* => 3 */
```


@MasseGuillaume should this be deleted? any point in having commented code?


Ouh :(.


@MasseGuillaume commented out test?


I will think tomorrow. I plan to support only `n/1` and `n/10`.


You haven't slept?


The request is handled by the Parser and the Service object. While they shouldn't operate on the header array at the same time, I spend a good deal of time last night chasing down a ConcurrentModificationExecption (http://docs.oracle.com/javase/7/docs/api/java/util/ConcurrentModificationException.html) and this fixed it.


`this._super.apply(this, arguments)`


Added more cubes


This doesn't compile right now. Some things are missing. A lot of it is from the SFML example code. Normally I try to get a little creative with things but I'm not 100% sure what's going on here, so I'm going to go by the book until I get it working and can see the complete picture. After that, we'll see what I can do.

On Foreach.hpp... I honestly have no idea what the hell that is doing. But again, by the book.


Still has unused `'tcx`.


I have a sleep :). But now it is my time to go sleep (I have strange sleep time for my timezone, but there is fucking polar night in Saint Petersburg, so anyway sun is only few hours at days :) ).


问题貌似出在这里：

[ `go get github.com/golang/crypto/blowfish` | done: 15.426847011s ]
	package github.com/golang/crypto/blowfish
		imports github.com/golang/crypto/blowfish
		imports github.com/golang/crypto/blowfish: code in directory /Users/elvizlai/Go/src/github.com/golang/crypto/blowfish expects import "golang.org/x/crypto/blowfish"

	exit status 1

但是翻了一下github.com/golang/crypto/blowfish里面，没找到那里需要依赖golang.org/x/crypto/blowfish，
另外，即使将包"golang.org/x/crypto/blowfish"加上也是无效的。


> Why?

That way users wouldn't get th same recovery code?


How do you add a callback from a plugin?


1277852-CommandLineInterface (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1587 of 1592 tests passed)

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


Looks good, I'll test it out.


http://bjango.com/articles/min-device-pixel-ratio/

> Devices with -webkit-min-device-pixel-ratio: 1.5
Google Nexus S
HTC Desire
HTC Desire HD
HTC Incredible S
HTC Velocity
HTC Sensation
Kindle Fire HD
Samsung Galaxy S II
Sony Xperia Tablet Z
Sony Xperia U


Hmm, that's a pretty important one isn't it :+1: for noticing. I'll add that right now.


I will convert `1.5` to `15/10` only for `-o-min-device-pixel-ratio`. Is it OK?


Thanks for this site. Seems like there is no resolution, where wwe need `n/3` or `n/100`.


@eregon you might find this useful to get around the fact that we currently only release the GIL on IO. If you set `-Xtruffle.passalot=10` it will release the GIL on 10% of lines.


找到问题了。。。。
代码"golang/crypto/blowfish"中第6行package blowfish //import "golang.org/x/crypto/blowfish"需要讲注释移到下一行中。
对于golang/crypto/blowfish/cast5也要做同样的处理


rad. thanks.


Uh I meant @soffes


Lol, omg! They put it in!


I look for the NPM module.


1277852-CommandLineInterface (nealkruis) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@eregon @nirvdrum This is `jstack` for JRuby+Truffle. If you run JRuby with `-Xtruffle.passalot=10 -Xtruffle.stack_server_port=8080`, you can then interrupt it to get a dump of all threads' stack traces by running `curl localhost:8080`.

Example output from running Richards with 3 threads:

```
$ curl localhost:8080
../test.rb:111:in `schedule'
	from ../test.rb:39:in `run'
	from ../test.rb:19:in `each'
	from ../test.rb:19:in `run'
	from ../test.rb:395:in `<main>'

../test.rb:191:in `run'
	from ../test.rb:303:in `run'
	from ../test.rb:113:in `schedule'
	from ../test.rb:39:in `run'
	from ../test.rb:19:in `each'
	from ../test.rb:19:in `run'
	from ../test.rb:391:in `<main>'

../test.rb:111:in `schedule'
	from ../test.rb:39:in `run'
	from ../test.rb:19:in `each'
	from ../test.rb:19:in `run'
	from ../test.rb:391:in `<main>'
```

This works via our general safepoint mechanism, but it should also be a useful tool. Running a program and it just hangs somewhere? You can use this tool (as long as it's not hanging on IO - we can't interrupt that yet).


有升级太好了，  不过看这个 扩展接口问题同样是——用了自定义规则的部分后，对于符合自定义规则的内容就完全丧失了 xss 模块中其他默认处理的功能，例如 escapeAttrValue 等。。。这也是为何最终选择直接修改源码的原因。

我们这里理想的扩展接口或扩展配置是：允许增加某些规则的细节范围（例如直接修改正则或扩充字符串），但不影响整个 xss 处理的流程。


qwe


**Show default device icon in tray.** (Handy for the quick switch mode)

Maybe add a separate setting for this, someone may complain that this
makes the icon hard to find...

And also I don't handle nonstandard resolution, just load the icon as
is, there might be some problems.


Why there's no comment?


@rdlowrey I have this fixed. Should a new bug be created so I can create a pull request? This fix should be pulled as soon as possible.


Hmm.. good question. I've been looking at the query code and there's nothing special done there. This code however does double subtract DST from the time stamp on live servers. Looking further into this it appears that Date.localParse() uses strptime() internally which understands daylight savings time depending on locale.

This does hint at possible future bugs depending on weather installing the servers in locations that don't have DST (Singapore?) will screw up DST calculations.


`git push -f` bruh, you could be faking the timestamps for all i know


Big diff.
![image](https://cloud.githubusercontent.com/assets/874898/5912598/28248b80-a62a-11e4-8988-53beca103052.png)

Can you try reverting this commit and making a smaller diff commit so we can view the overall diff here: https://github.com/borisyankov/DefinitelyTyped/pull/3508/files


qq


I would recommend reading (or re-reading) the docs on the WPILibJ's RobotDrive API.  I think this will greatly assist on adding the strafe/rotate/joystick methods.  There is some opportunity to overload some methods further with movement and rotation.  Also, there is a Polar option that may be useful.


Brief explanation: Ed25519 is a deterministic Schnorr-EC signature scheme. It works by using a keyed PRF to generate the challenge for the ZKPK; (most?) security proofs rely on the key for the PRF being independent from the private scalar.

So, representing private keys as the input to a KDF:
- makes proving security of deterministic nonces easier;
- allows the pair (scalar, prf_key) to be compressed;
- and encodes a proof that the scalar is, in fact, the output of a OWF (and so is unlikely to have "special properties").


It's the only SUN that uses macros and the output seems to be identical anyhow. I'd just get rid of them and use the normal routines. Is `sstroutinenolabel` actually needed either?


1277852-CommandLineInterface (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Coverage Too Low

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


Yes, we need `expandPrivateKey`.  The signature algorithm takes the 32-byte private key and expands it into two 32-byte pricate keys: one used as a multiplier and one used as an input to the PRF.

If other sources use the term proto-key, then I'm mostly happy with that.  I'm uncertain if the use of the term proto-key conflicts with the use of this variable in other places in the code.  I'm pretty sure we store the 32-byte private key, not the expanded private key, and we call it "the private key" elsewhere.

I like the explanation of `p`.  Is "key" the right word to use with regard to `k`.  It's an input.  Does "key" have a specific technical meaning with respect to PRFs?

(I don't have a github login using my google email.  This is the author of the file.)


this is a example found on network.


test


Soooo needs updating xD


Saw that you started working on the gh-pages branch. I've come up with a scheme for maintaining gh-pages branches entirely with Rmd files but that still leverages nice theming and navigation via Bootstrap (e.g. http://rstudio.github.io/dygraphs/). I think we could fairly easily take the README.md you already have and create a page with nice internal navigation via a sticky table of contents. Do you want me to take a shot at framing that out?


I think it is needed -- SUN/209 uses \textit{type} in the titles of its sstmacros to indicate a variable that could be either CHARACTER, INTEGER, etc. Using formatting commands within a label command doesn't work. The nolabel variant should be used whenever people have defined their own label instead of letting sstroutine define it automatically from the title of the routine.


Delete it!


Wow u actually read the readme what a nerd


hot


I like structs like this to have a default constructor so that you can have default values of e.g. 0,0 rather than be uninitialized memory. I also like to have an empty() member that tests for the uninitialized state.


I decided to put homework in a private repo


"This file is written by xdg-user-dirs-update"

You should probably just have people run that.


Alternatively, if the semantics are that you can *never* have an uninitialized position then this should have a default constructor with 2 args that allows the compiler to enforce this.


right now you're on an ancient 2.2, because that's the latest version provided by travis' old ubuntu. using 2.6 means you can start adding features like `anyof`, `allow_alias`, `import public`, etc. (see https://github.com/google/protobuf/blob/master/CHANGES.txt)


I aligned the columns in a text editor while we tweak it. There may be other places lacking `.ss_new` formatting too. Was planning to run them through SS at the end.


Evan from webchat.awfulnet.org #lgbteens.


Yes, yes it does, lol


Hi Bernd,

These are great footprints but you had forgotten the courtyard. I added them for you. Next time, if you would like to create a pull request, it will be easier for another librarian to double-check your work. Inversely and if you wish, as a librarian yourself you can give us a hand for approving other's contributions.

Thanks!


Wiggle is a separate data type and would be returned by
different methods.

It's probably clearer to have a separate annotationset object for wiggle,
as it a fundamentally different type of data.

CH Albach <notifications@github.com> writes:
> Possibly a separate PR, but where does Wiggle fit in here? I don't think it's
> referenced by anything else. Is the intent that it will be returned directly
> from a method?
>
> —
> Reply to this email directly or view it on GitHub.*
>


That would reassign all of the macros to $HOME/


Got a new corner case. When used in umbrella projects:

```
> Mix.Project.load_paths
[]
```

Do you want me to send a PR to handle it similarly to the "no Mix" situation?


You'd want to run it with command line flags, not "xdg-user-dirs-update &&
xdg-user-dirs-update && ..."

On Mon, Jan 26, 2015, 6:54 PM Zaq? Wiedmann <notifications@github.com>
wrote:

> That would reassign all of the macros to $HOME/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zaqthefreshman/dotfiles/commit/e3c572c0afb7979a602d1e6ca27b6dfc234936dd#commitcomment-9452120>
> .
>


Why did we nuke inigodockertest? We're still using this image in the Docker lifecycle DATs: https://github.com/cloudfoundry-incubator/diego-acceptance-tests/blob/master/diego/lifecycle_docker_test.go


add_lunch_combo liquid_e980-userdebug


teste


https://dev.change-vision.com/projects/jude/ticket/5969

英語ページは、このリンクでお願いします。
http://astah.net/support/questions/63

日本語サイトのリンクは、下記でお願いします。
(1) [Solution in Detail]ボタン → http://astah.change-vision.com/ja/feature/cpp-reverse-plugin.html#support
(2) [What is Doxygen xml?]ボタン → http://astah.change-vision.com/ja/feature/cpp-reverse-plugin.html#doxygenintro


a series of commands would work, but is hardly nessasary when you can just write the file.  Might be nice if users wanted to keep their directories upper cased though.
```
xdg-user-dirs-update --set VIDEOS ~/videos
```


The django practice to set verbose names is define it as the first positional parameter (except in relationship fields) as described in https://docs.djangoproject.com/en/1.7/topics/db/models/#verbose-field-names. This line should have been written as:
```
name = models.CharField(_('name'), max_length=40)
```


<!doctype html>


Hey, thanks for putting up your scripts, they've been useful for me.

I'm curious about this command. Does it work on your Acme without errors? I use plan9port's acme on OS X, where this command echoes `stdout writing address #13: address out of range; resetting` to +Errors after executing `winclear`. The command does empty the window, though. Do you see different behavior?


that's my bad, since got same bootloop on my kernel.


thouggh -> though
)= -> )


I love food


This should be floats, and the formula is (temperature-1000)/3.91


Don't we still need the returns? Otherwise it'll keep trying to do more for each feed, when it's already errored.


hmm, but this prevent multistream tuning.


I meant to put a return in next()
On 27 Jan 2015 20:07, "Oliver Faircliff" <notifications@github.com> wrote:

> Don't we still need the returns? Otherwise it'll keep trying to do more
> for each feed, when it's already errored.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/itsapi/news_feed/commit/44511d63593f8108ea5e53f93865edc77e248abc#commitcomment-9466515>
> .
>


Simply `return` returns undefined, even without `return` statement the result value will be undefined, although the comment is good, I guess that's the reason for the statement below it.


How is the code in line 660 supposed to work? If `mpegts_mux_create0` returns NULL then this is assigned to `mm` and inside the `if` is then freed?
Also I did not see any memory leak as `mpegts_mux_create0` frees mm anyway in its single error path.
However, maybe I misunderstood something...


That wouldn't work...


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


This works around rakudobug RT #123686


Yes, there was a separate PR that added it to the API conversion, this is
to validate that that PR actually fixed the problem.

On Tue, Jan 27, 2015 at 11:14 AM, Alex Mohr <notifications@github.com>
wrote:

> I assume the motivation is to make the test more reliable in some way?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brendandburns/kubernetes/commit/92d65b7c652036bcb834e61e6734019e1608c647#commitcomment-9465616>
> .
>


Unsure if this was the commit, one of the few on server.py, but I can no longer connect to my usenet server :(

2015-01-27 20:03:05 ERROR server: could not connect to news server: The read operation timed out


86956490-SPMMagic (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2072 of 2191 tests passed)

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


dropping it, since we have to remember to remove it... :)


ok, I think I got it now. I somehow expected this, but I did not find any kind of documentation for it - and I guess it was not only due to the fact, that I searched for "once then" in combination ;-)


I know I've done a bit with trying to simplify usage of BroadcastReceivers, some of that being the BroadcastUtils, an abstract BroadcastReceiver that transposes broadcasts into simple callbacks like: ```onMinistryTeamLoaded(String id)``` I just don't remember at the moment where I landed with what worked best with my current knowledge.


Thomas Berger wrote:
>     moviuro@psychoticdelirium ~ % lol=foo/bar
>     moviuro@psychoticdelirium ~ % echo ${lol%/}
>     foo/bar
>     moviuro@psychoticdelirium ~ % lol=/foo/bar/
>     moviuro@psychoticdelirium ~ % echo ${lol%/}
>     /foo/bar
> Nothing funny, works as expected ;-)

Yeah, I know how the expansion works. I was rather wondering, if the
destination file name is supposed to end up in the current working
directory or somewhere down the directory tree.


Thomas Berger wrote:
> @ft How should I do this?

In ~/.zshrc.local:

  abk+=( FOO "| bar baz" )


Lines 238 and 375 make me sad with shame.


that's what I figured too, I got sick of pointing out those `# tweaked` comments in PRs ;)


Testing Github commenting: ehm... "Unsure about font-size."


Thanks @xorrbit for finding, and fixing.


I meant return next()
On 27 Jan 2015 20:10, "Oliver Faircliff" <notifications@github.com> wrote:

> That wouldn't work...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/itsapi/news_feed/commit/44511d63593f8108ea5e53f93865edc77e248abc#commitcomment-9466551>
> .
>


@DNFcode wat?


Yes, Brice is right on spot.


Lol ;)


Typo from web editor.


@SubJunk What I remember Flavor attribute is is read and stored in DB for audio and subs only so how you detect file is WEB-DL? From filename and flavor of audio track as I read the commit? I expect params.aid.getFlavor() returns flavor for audio track but WEB-DL string shoud be read somewhere from video attribs returned by MediaInfo or am I wrong? :)


@chrisseto Thanks for the clarification. Yes, I think raising an error makes sense if the waterbutler API depends on the presence/absence of the slash.


Thanks for the contribution. It is probably better not to depend on Ant here. It works because the test module depends on groovy-ant depending itself on Ant. But if you just grab groovy-core, it would fail.


I think having a `self.schema` is actually a good thing for ForcedMeasurementTask to have as well, even though it's not used here; it's a useful bit of introspection.

But I actually originally had this implemented as a schema argument passed to `initalizePlugins()`, which would then be responsible for setting the instance attribute.  The problem with that is that `SingleFrameMeasurementTask` already takes the schema as its (opaque) keyword argument, so we'd have to pass the same object to that function twice, first as a positional argument and then as a keyword argument.  I felt that was even worse than the clunkiness of having `initializePlugins()`  require an instance argument before it is called.

As for the method name, there actually is a `getOutputSchema()` method I could use instead, but it returns a copy, not a reference, and that *might* defeat `initializePlugins()` attempts to mutate it.  But you're right - `editOutputSchema()` actually *does* return something that should be used and then discarded, not kept around (I think it's pretty safe here, but it wouldn't be everywhere).  I think might be able to use `getOutputSchema()` instead with a modification elsewhere.  I'll give that a try.


86956490-SPMMagic (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: Build Failed

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


yes, better to make it explicit I think.


Doesn't `getCause` does the same? Do you want to use `getMessage`?


Should make these more different...


It is intentional. Cornelius said, they dislike force-pushes, because it screws up github history and commenting, so they do the fixups before actually integrating the commits.


Cool, @tombatchelor and @brianwcook! Would this make more sense as a PR for the xlr-xlr-plugin, though? This is more like a task, rather than a full plugin.

/cc @jdewinne


:+1:  nice code removal


indention seems wrong to me, might be github.vs.Ecplise :)


Yes, thanks for the reminder. I like "Draw Calls" if you don't have a stronger preference... Although I could imagine someone naming one or more of their debug marker groups likewise. How about taking it a bit further to make it more distinctive by prefixing the label with something, e.g. "VOGL", e.g.

VOGL: State changes
VOGL: Draw calls
?

Although if you're comfortable with "Draw calls" by itself I'd be satisfied with that. It can easily be changed if there's any backwash against it.


I had added it during development to fine-tune the offset value and hadn't planned on it being adjusted by the user. In testing, I expanded the timeline to various widths from its minimum up to spanning two monitors (3286 pixels in x). Sometimes I would see a glitch that didn't seem to be affected by any offset. But after slightly moving the vogleditor vertically, the problem seemed to go away and the default offset worked fine. So I have a bit of insecurity about that and left it in as kind of a temporary development backdoor item in case there was feedback that something didn't look right.

I agree it's probably better to have it in the settings just in case even if not, or rarely, used, but at the time I hadn't planned on it needing adjustment.


Could you please replace ```force_unicode``` with ```force_text``` instead of doing ```as something```


Indention again :)


We need a solution that allows both python 2 and python 3 to work, maybe doing a try except... not sure its the right way to do it.


@michaelwehrley Two questions: 1. why define this as an anonymous function, 2. why mix camel and snake case in the variable names?


OK.
Ran into a problem last night that I have yet to solve.  Knowing when the
net drops, is not that easy. At least from the server when the client is
done. At the moment the read fails with a EOF error and I see the stack
trace every time, and the read method returns null. So if I test the read
for null I can make it work. Not the best.

I looked at a  bunch of methods on serversocket and socket and haven't
found one that I can use to know if the socket has been disconnected , so I
can then wait on another accept. I'm not at home; over getting a hair cut.
On Jan 27, 2015 12:49 PM, "Brian Norman" <notifications@github.com> wrote:

> e.printStackTree(); should change to log.warn("", e);.
>
> All the printStackTree does is print it to stderr with println's. It's
> almost the same as System.err.println(ex.toString());. You want all of
> that to go through Logback so you have it in whatever it logs it to.
> Whether that be the console, log file, or both. Just think of the
> difference between stdout and stderr as the difference in levels.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pinorman/wx-mon/commit/2a93438bdc6676646fa17130fbac792b8326edcb#commitcomment-9465157>
> .
>


@vtjnash I am using LLVM 3.7-svm, and the change above in line 5116 breaks my build,
```
codegen.cpp: In function ‘void jl_init_codegen()’:
codegen.cpp:5119:52: error: ‘class llvm::TargetMachine’ has no member named ‘getDataLayout’
     engine_module->setDataLayout(jl_TargetMachine->getDataLayout());
```
I am not sure what should be used instead, but if I remove the ```!defined(LLVM37)``` part it compiles.


Ohh


Thanks!


Empty line, please remove.


Did you mean for instead of form ?


Thank you! This commit fixed the build. It was broken since http://ci.cgeo.org/job/c-geo/3315/


Is the only other possibility NoType? It would be clearer to say `case NoType` here, or at least add a comment saying that only NoType is possible and what NoType means in this context.

Same below in `addOneLowerBound`.


Well, unless you can create a useful represent, it makes indeed more sense to remove (or at least comment) them for now.

Of course it would make sense to have budget information in the list of deployed resources - but these separate list_fields make very little sense with a m2m component  - there can be multiple consecutive or even parallel budget allocations for each resource, so you would get a column with a list of budgets, a column with a list of start-dates, a column with a list of end-dates and a column with a list of daily costs - each in arbitrary order (and not even all columns for all allocations because some could be empty):

| BudgetA, BudgetB, BudgetC | 2014-09-12, 2014-08-13, 2015-01-01 | 2015-01-04, 2015-02-08 | 350, 380, 320 |

And what a nonsense is that?

The budget information should be in /one/ column - it can then still be a list, but just *1* item per allocation, like in:

| BudgetA (2014-09-12 -- 2015-01-04, 320 USD/day), BudgetB (2014-08-13, 350 USD/day), BudgetC (2015-01-01 -- 2015-02-08, 380 USD/day)|

...so that is at least makes sense.

This can be achieved by implementing a represent for allocation.id that renders exactly that - and apply it in the event_* controller, then have allocation.id as list_field for event_*.


размер буфера 1 недостаточный, особенно для кодировок в которых символ может кодироваться несколькими байтами


FYI, these pass statements are unnecessary. You need *any* statement in the body; that docstring is a statement (an expression statement), to be precise.


Same here.


You can fix it to look more like dictionary styling of Ray Wenderlich.


good


On Jan 27, 2015 8:16 PM, "Shilin Victor" <notifications@github.com> wrote:
>
> Will try to provide this soon... from what I've already saw in log -
seems that it's crash of usr/u8500_initramfs/init - at first I'll try to
revert some my changes from that(but what's strange - same script is
working on my other kernels).

That's odd since it's a shell script... and if you can boot into recovery
and not into system, than that's not it - if it's init that crashes, than
it's the binary from ramdisk.cpio

> I thought to do it same way, as you provided patches from XXLQL base -
did you this by some scripts or by maybe there's some manual method?

No, it was done by the script, but my idea was to find minimal diff between
some android commit (which turned out to be bea3738) and XXLQL version and
then apply that diff onto a higher android version (3.0.72). What base and
target versions do you intend to use? That's what I don't understand. Our
TC kernel is 3.0.101, and that's a version that does not exist in android
tree (I think so since android-3.0 ends on 3.0.72...)


avoid unnecessary changes => more difficult review


Ok, tomorrow I will implement it in core Java API

> On 27 янв. 2015 г., at 22:17, Cédric Champeau <notifications@github.com> wrote:
>
> Thanks for the contribution. It is probably better not to depend on Ant here. It works because the test module depends on groovy-ant depending itself on Ant. But if you just grab groovy-core, it would fail.
>
> —
> Reply to this email directly or view it on GitHub.
>


Note that these should probably be links to the underlying budget information (rather than the allocation itself). Well, at least that seems more meaningful to me.

One-man-opinion that, though.


meaningful variable names would help me understand what is happening :-)


Live


Typo at "Disreagard", should be "Disregard".


it feel more natural to me to put arg parsing in a script rather than part of the public library? there isn't really much logic here aside from arg parsing. could you imagine someone else importing this?


Kan gebeuren, misschien moeten wij deze dan maar laten rusten of op termijn verwijderen. Mij maakt het niet zoveel uit ;) Zolang wij maar geen dubbel werk doen..


is xxx really doing sth?


There is an ``assert` for this array comparing with int64 datatype that only worked for 64-bit systems using a dtype of `np.int`. This fixes the test for 32-bit also (all tests pass now on this platform).


@dkavanagh: ms_remove isn't as clear as magicsearch_remove_text, for example


Are you sure, that you want to log this twice? With different spelling? :P


There shouldn't we spaces after { and before }


Your loop fix has the side effect of only displaying a single broken image link when it is valid and multiple nodes request the same invalid image.  I'm not sure if that is intended behavior, but it was unexpected to get  console warning "Could not load brokenImage:" and a single node with the broken image.


Same comment as above: if NoType is the only possibility, make it explicit, or at least add a comment.


`+# this.pickUpFlag(flag);   faq: "FAQ"` - Is this meant to be here?


Fixed, I saw it before your comment


Should we add this as an issue so we remember to get rid of this code?


OK


The middle one is technically a generalized posynomial, not a posynomial, since it is not in the standard posynomial form. Might want to switch that one out.


Is this a debugging println that should be removed or changed to logging?


такое приведение типов не будет работать правильно для кодировок отличающихся от UTF-16, частично правильно будет работать для UTF-8
нужно преобразовывать ByteBuffer с помощью CharsetDecoder в CharBuffer


urls on the left are meant to be human-readable => my/filters/depot would be nicer


That one I did because Boyd uses a similar example.  I'll swap it though.

On Tue, Jan 27, 2015 at 3:29 PM, whoburg <notifications@github.com> wrote:

> The middle one is technically a generalized posynomial, not a posynomial,
> since it is not in the standard posynomial form. Might want to switch that
> one out.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/convexopt/gpkit/commit/c5c31d4f70b1a3378ea7c097432a17760b5f447b#commitcomment-9466897>
> .
>


I have some ideas.  Will take a look.


I believe the class is here to do caching of derivative functions; i.e. avoid instantiations of Matrix<Function>


Sorry, any chance we can synchronize all of this to our own call_process() method on the widget base class?


Spelling error


> to keep fevor up to date

Good idea!


1277852-CommandLineInterface (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1591 of 1596 tests passed)

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


this change was made to llvm-svn yesterday. have you done an git pull and recompiled llvm since then? (hopefully they didn't decide to revert this again today)


Did you get to make this work? I have the same requirement to read a CCITT t.6 format tiff. Please let me know.


Could the return type be `bound.type`?


that changes it then, I thought they would be somehow 'stable' in terms of what 3.7 means.
I did a pull a few days ago, didn't know it would change so fast, I am pulling again now and giving it a try, thanks and sorry for the false alarm.


1277852-CommandLineInterface (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1092 of 1099 tests passed)

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


@juliancesar, se estas imagens estão relacionadas a documentação, sugiro que sejam movidas para a pasta /behave/documentation/imagens ou a existente: behave/documentation/reference/pt-BR/images


needs to be indented with 4 spaces


Well, that plan didn't work - but instead I resolved the issue by moving the call to `setupSlots(schema)` out of `initializePlugins()`, which means it doesn't rely on `self.schema` anymore.  So instead we can set `self.schema` in `ForcedMeasurementTask` after the call to `initializePlugins()`, and use `self.mapper.getOutputSchema()` to do it.


Для чего эта строка?


> Good idea!

It builds and passes all tests, so it appears so!


before this we used to have a "template" file for "settings", which would allow for anyone to have their own settings... but it required to make changes and pay attention to template, which isn't included by default on processing editor. so it was too confusing and not worth the trouble.


it's nice to have these new language features implemented. But on the other side I am not convinced, that they really help much. I never considered the verbosity of Eiffel a problem and when we have these syntactic sugar extensions we somehow loose one of Eiffels pros over C++, where there are tons of ways to do the same thing...
Maybe you can add a option to warn if these features are used?


There's a small typo in this line of the man page: Janury instead of January.


IMO WaterButler behaves closer to libcloud than sqlalchemy.


Ah, yeah, sure, I can try that


Hi! How use it on Osx 10.10? I can't find /Library/Keyboard\ Layouts/ folder. I use this hack until last week, but now my keyboard layout is left.
Слава Україні! Героям Слава!


Nice! That is a great solution.


480x800 is more commonly hdpi.


I see, but was irritated because you kept the labels.


Mean this line can be removed


Для чего это строка? И это не значит, что её нужно удалить. Жду ответ здесь.


It would be good to be in the habit of using Journals (this function can just take one as an argument). This will create less work when we go on the "WriteLog witchhunt."


Why?
stack trace will do nothing.


If I don't it clashes with unlink from <unistd.h>.


If we are no longer deserializing is the try block necessary?


Free functions good!


Name is Jötun instead of Jotun. But maybe this will cause problems.


Почему кнопка возврата сделана формой?!


Oh no, sorry. Here is what I thought: or we use different colors for each faction (background+buttons), or we only use the Harkonnen one (which seems to please everyone). So I guess I can really remove the Atreides and Ordors buttons and labels. Sorry about that.


This will fail the first time you try to build after checking out the repository,
as foundation/core/version.h does not exist at that point.


Change Title


Thanks!


Cool, I didn't know coveralls started supporting SVG badges. I bet this url updates faster. Wonder if one has better uptime than the other. In any case, I'll start using this, too.


Удалить. И больше никогда не использовать.


I'm actually not setting all the flags here - I'm setting just the general failure flag for all *radii*.  I'll modify the comment to clarify that.  As you pointed out, having only the general flag set does convey useful information.


Should this be any state (not just initial), just to catch any further errors?


I was wondering about that but didn't take the time to check. I'll revert the change. Thanks for the heads up.


Because time travel rocks :rocket:


According to my book there are no oct or hex escaped sequences, only decimal


ditto for escape sequences


Just a heads up, this commit removed the documentation for the `verbose` setting
https://github.com/outaTiME/grunt-replace/commit/15cb66b05c66b5a0738dcebdbc358ebbced2d6ef


maaf salah menaruh maksudnya ke model,terimakasih


Ack! So, there is another check I need to perform... For the scrypt file format, the salt must be 32 bytes. I definitely need to check this and raise a ValueError if it isn't. That is why it works when you let it choose it for you (because it chooses 32 bytes). I will add that shortly.

There is however a fairly large difference between `if foo is not None:` and `if foo:`... In this case, I could probably omit it, but keep in mind other things return `False` when pushed through `bool()` (i.e. False, 0 and the empty string ""). Imagine this case (I am not saying this is realistic, and people doing this probably deserve to have something buggy happen to them, but might as well be nice to strange people too):

```python
class ImaginaryString(str):
    def __nonzero__(self):
        return False

filename = None
if some_interesting_condition():
    filename = ImaginaryString('I do believe in faeries...')

if filename:
    print("This is never reached, regardless of some_interesting_condition()")

if filename is not None:
    print("Filename exists")
```

In general, you should prefer testing for `is None` or `is not None` to check for existence.


Looks good.


@mbabker Let's make it happen with Joomla! ;-)


Need some clarification on why a separate state was used for escape sequences - it's fine if it's necessary, I just hadn't realized it was necessary.


You can remove this line


I have zero experience with `<progress>` because I never need it.

I'm strongly opposed to replacing elements and/or widget with a bunch of JS just for looks. `<progress>` is [supported pretty well](http://caniuse.com/#feat=progressmeter), so I would at the most think about [polyfilling it[(http://www.useragentman.com/blog/2012/01/03/cross-browser-html5-progress-bars-in-depth/) (haven't read that article myself yet). We could, however, think about [styling it](http://css-tricks.com/html5-progress-element/). :)


I solved reboot problem above (damn, that was faulty installer script which removed boot.cpio).

> What base and
target versions do you intend to use?

Sorry for misfrasing - by base I meant root-commit of TC repo which is 3.0.31.


Mode.TypevarsMissContext should have a comment explaining what it means.


Nice. Better than what I had.


Yes, for us its too obvious to have all kind  stuff so this is reminder for it.


no. 3.7 is the current llvm development version where all the breaking stuff happens. 3.6 and older are their stable releases


лучше использовать allocateDirect()


Better with out the break line


Not sure why we use a separate identifier state instead of just outputting the ID directly


ditto


I'll clarify the comment.  The first call is actually no different from the rest; all this call does is extract a value from `measRecord` and return it, with some error checking (which can include setting flags) first.  It'll be the same error checking every time, but it's not expensive enough enough to merit changing the interfaces to put it outside of the loop over radii.


Никаких шуток и смайлов. Ты не в гугле. При любой ошибке приложение должно возвращать 500 код и сообщение "Внутренняя ошибка сервера"


Do not use non-standard attributes. Change `status` to `data-status`. See https://developer.mozilla.org/en-US/docs/Web/Guide/HTML/Using_data_attributes


Muy bien !!

sigue avanzando :wink:


Sorry to nitpick - one indent here please.


Typo here, regitser.


We don't actually have to do this and I would argue that it's better to allow zero-led integers- ex:
```
var thing = [
  01,
  02,
...
  11,
  12
]
```


Checked it again - and yes, it is possible that a cost item can have multiple sponsors and hence multiple budget allocations both consecutive or parallel.

For instance, on budget could cover the purchase, and another budget the operating costs.

So it makes sense to implement a m2m representation here - and it would even make sense to have some sort of summary information in the cost item super-record (total budget) and make that visible too.

Think that you have some heavy excavator deployed in a landslide disaster, and one organisation covers its purchase and transportation to the site, while another organisation covers operation and maintenance cost (fuel, repair, security...).

Currently we would create two cost items for the same resource (i.e. two different assignments), which is though a but cumbersome and wouldn't allow a total cost analysis per resource - apart from it being unsafe in view of data integrity. And the latter *is* a big deal, because it can easily look dodgy to sponsors who may then withdraw their support.

But as I said before: we really *really* need practitioner input here - I am speculating wildly, I don't really have any kind of expertise in view of budget management (or hardly any understanding of money at all).


With the standard data-attributes you can do `$(event.target).data("status")` to get the value. Or go even without jQuery and use `event.target.dataset.status`.


конвертация в массив байт и затем в строку снижает производительность из-за доп затрат на конвертацию


Extra if :)

just `return this.status==status` will do


the NULL check is not needed (or impossible)?


Should `admin_level` be retrieved from the field `group_definitions` rather than be hardcoded to `admin_level`? cc @nekron


ACK


`TYPE_SIZET` should just be `TYPE_SIZE`.  The `_t` stands for "type".


doorpi.cfg ist angepasst
in dem Script DoorPi.sh sollte es auch so gehen - wenn es nicht vorhanden ist, kann es halt nicht ersetzt werden.


In my mind, it looks a bit clearer to name the variable `n` (this is very often used for the result of `(p|f)read`, or `read_len`). `size` and `len` are similar but we shouldn't needlessly switch between the two for variables in the same scope I think.


вместо конкатенации строк лучше писать обе части последовательно в канал, без конкатенации


Array literal should be used. Ex. `["Alpha","Bravo","Charlie"....]`


@JovicaSusa this will work like this, but consider what happens if you pass a different character_id in the URL. For example, character_id belonging to some other user. You have to do something like: `current_user.characters.find...`


Nice job :8ball:


JSHint will go crazy about this. Use `window.Pinkey =` to explicitly say that you want a global. This looks like a mistake.


@JovicaSusa does it work without the brackets?


other side


I don't think I did. We re-evaluated they way we were handling the data and decided to use DocumentCloud to display our tiff documents.

After that I don't think I ever really touched this code again.


@datahead8888 is correct. Excessive cherry-picking mixes up the commit/branching history. Also, if you change a file in `devel` that has been changed in `release-2.0.0` also at the same point, you’ll get in merge trouble. This should be reserved only for merges from `release-2.0.0` into `devel` as that’s complicated enough. Feel free to revisit [the branching model description](http://nvie.com/posts/a-successful-git-branching-model/) or even [the coding contentions](http://docs.secretchronicles.de/api/devel/md_conventions.html) :-)

Vale,
Quintus


It might be better if you name those enums like
```objc
CRVStubErrorNone,
CRVStubErrorRetriesLimitReached
CRVStubErrorRetriesLimitExceeded
```


fixed in c7bf274


Oh – I thought we were going to do this in addition to the Twitter/Facebook ones, not replace them.


WIP #47


Added salt check:

https://github.com/ricmoo/pyscrypt/commit/8e55f41bd122df9bb1291e540c0a71d04f54196c


Note to self: we should do this on our end too (`util` → `utils`)


Per the discussion on Slack, I made a Github Pages page as a replacement for the About section: http://codefordc.github.io/districthousing/

Everything is the default right now.  You'll notice the site doesn't say anything about our project, but it does say a lot about how to use Github Pages.  Someone needs to make a pull request to the https://github.com/codefordc/districthousing/tree/gh-pages branch that replaces that text with stuff about our project.


@mikeprimm - So could you possibly offload the `org.dynmap.markers.impl.MarkerAPIImpl$DoFileWrites` into another thread? Just run it as an asynchronous task? That small change would improve the performance of your plugin quite a bit.


@brudil that has been removed :sweat_smile: thanks


The STRINGESCAPESEQ doesn't handle the `\f...f\` formatting as far as I can tell. It should handle:
```
"This is a string \
\ that is continued \
\ on multiple lines"
```


teste


Hahaha, nice. A total kludge, but a solid one.


Hey, you need to commit your blhtkd.c file.  I had to comment out your function calls since the program can't compile without your file.


@JovicaSusa did it work?

I know that form_for automatically adds authenticity_token just next to the opening form tag - have you checked if it's already there? Send me an email describing problem/error you are having.


I would export those magic numbers into constants


The only thing the method name here is used for is to get the protocol entry ID when producing the writer functions.  We can remove the method name here and put the protocol entry ID in a local variable in the BEGIN macro.  I don't like the repetition - it just begs for a missed change when copy-and-pasting.


Oe, corregí esto.


ACK (provisionally, I don't know this code well)


This should error out, see Piazza


Oh, right. Good thing you actually run 32-bit and GMP :-)


Yes, that's what it's used for.


I spot a bug - shouldnt it be Prepare Joomla 1.1


Can we just put all these #undefs into a header file (`sgen-protocol-undef.h`?) to avoid repeating it all over?


This looks like a typical scenario for "switch" instead of "if"


The docstring makes me expect this to not always return 0 for the y value


The only allowable control characters are `\n`, `\t`, `\"`, `\\`, and the formattin character sequence, all others should be removed


I've added one more check to `testApertureFlux.py` that should more than cover the tolerances present in the old tests.


lol, yeah ... the price we pay for having to parse their site for the data.


Forgot to mention: Also added fontawesome.


Cool


It would be nicer if there was no space between


plaintext login


Название пакета путанное (и camelCase, и суть). Как насчет
calculator.gui?


@dg do you know this thing called compiler extesnions? They're like super handy and they are meant to solve exactly this kind of problems! You should give it a try :P


this was changed now


:sparkles:


Also you might want to use triple equals (`===`) if you want make sure they are actually the same. http://stackoverflow.com/a/523650/153718

There is no really a use case for the double equals... It's just confusing.


On Jan 27, 2015 9:46 PM, "Shilin Victor" <notifications@github.com> wrote:
>
> I solved reboot problem above (damn, that was faulty installer script
which removed boot.cpio).

So it boots into android?

> Sorry for misfrasing - by base I meant root-commit of TC repo which is
3.0.31.

And isn't the root commit of TC repo one of of samsung releases and not the
android code?


You are right. I took on me to fix it.


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


The mutableCopy is needed else you are trying to initialize a NSMutableArray with a NSArray, isn't it?


@fprochazka have you heard about thing called pull request? ;)


Surely have! Also a great thing :D


Комментарий по читаемости и структуре кода - Длинна метода на грани допустимой. Возможно разбить на логические блоки.


Yeah -- that's just because we don't support up-to-down languages at all at the moment (and thus don't track y-advance).  Should I update the docstring or just change this method to only return `x`?  Either way, it's a major project to be more general here, so maybe I'm overengineering this.


Working now on checking if everything works okay on my Linux machine.


dumped from client and verified with patter


this one is a particular case in my opinion... I wouldn't merge.


pathContents need not be mutable since all you are doing with it is iteration.


It is quite possible if you did a clean install that the directory just doesn’t exist by default. So you should create it:

sudo mkdir /Library/Keyboard\ Layouts/

cameron.

> On 27. jan. 2015, at 21.38, tarasknyazev <notifications@github.com> wrote:
>
> Hi! How use it on Osx 10.10? I can't find /Library/Keyboard\ Layouts/ folder. I use this hack until last week, but now my keyboard layout is left.
> Слава Україні! Героям Слава!
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/palmerc/Ukrainian-Russian/commit/a83224b7011952240572d338dbcb900871b51425#commitcomment-9467050>.
>


Avoid extra whitespaces


This is awesome!  If you fix the few small things we'll merge.


Может как-то объединить методы классов Read/Write в один класс с двумя этими методами?


Ack


Agree :-) I fixed it.


We can also replace this with `Char.fromString`


User schema created twice


I think I misunderstand something - isn't XXLQL is samsung release too?


Make sure to properly indent :

etc.


@diegorojas qual o problema que apresentou? O que vale é o que está em verde?


Thank you, Cameron! Its work!


I believe this line is not necessary here


Why check every variable for nullness when they are modified at the same time? One test is enough, if `blurBoxShadow` is not null for example, then others aren't either, and if it is, others are as well.


There's a couple newline differences, and two files which have mild differences. Other than that everything seems good to go! I'll do some more testing tonight. I should be able to squeeze my changes in later.


Spelling error with 'mark'


Currently, the other cards with special characters use the closest representation (AEther Vial, Seance, Juzam Djinn, ...).

While I would prefer to have the actual names, I guess it would be a lot of work to make sure everything still works for no real payoff.


I think you accidentally the wrong direction.


Great! Now to see if it all still works on my Windows box...

On 27 January 2015 at 21:10, Zach Jaggi <notifications@github.com> wrote:

> There's a couple newline differences, and two files which have mild
> differences. Other than that everything seems good to go! I'll do some more
> testing tonight. I should be able to squeeze my changes in later.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mabrowning/minecrift/commit/baa8a53f9d0fa14904304165a034722be33dcca1#commitcomment-9467609>
> .
>


Wouldn't it be more correct if the Treefolk would get "Islandwalk","Swampwalk" etc. as separate abilities?


Can you make `wait`  an AtomicBoolean, and have the drain task set the boolean? Then you shouldn't need a custom executor service.


Do you need this separation line between imports of similar kind


If this controller is not being used please remove the file and reference to it.


overridable method? when for such quest it is different, you override the method instead of hardcoding


this is my initial commit.  This is for the new BucketBall website


add some code comments to describe what you've got going on


Let me know! I'll get working on re-implementing my changes in an hour or so (I lost my place)


Along with the atomic boolean thing, constantly submitting a "AtomicBoolean::set(true)" task shouldn't be an issue.


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=4421e17c9be644f8be0d0bedf81d1fba
 plone-5.0-python-2.7 [SUCCESS]


Okay, if you don't think it will often need to be adjusted, then I think using the env var could be fine, or even just leaving it as a member variable with a defautl value. If someone is up for the task, they can figure out a programmatic way to set it.

From: Lawrence Love [mailto:notifications@github.com]
Sent: Tuesday, January 27, 2015 12:21 PM
To: lawlove/vogl
Cc: Peter Lohrmann
Subject: Re: [vogl] UI: Color code timeline for groups (6b3d159)

I had added it during development to fine-tune the offset value and hadn't planned on it being adjusted by the user. In testing, I expanded the timeline to various widths from its minimum up to spanning two monitors (3286 pixels in x). Sometimes I would see a glitch that didn't seem to be affected by any offset. But after slightly moving the vogleditor vertically, the problem seemed to go away and the default offset worked fine. So I have a bit of insecurity about that and left it in as kind of a temporary development backdoor item in case there was feedback that something didn't look right.

I agree it's probably better to have it in the settings just in case even if not, or rarely, used, but at the time I hadn't planned on it needing adjustment.

—
Reply to this email directly or view it on GitHub<https://github.com/lawlove/vogl/commit/6b3d1595ac4f30625919e0f02db67f92d5403a1b#commitcomment-9466746>.


Excellent, thank you very much!


Lowercased lodash as that's how it's branded now.


volume monitor doesn't normally see the system mounts, but ok.


change variables are local, so I don't think the `re-run` test will fail.


Already is - at least all the file I/O is dished off (that was the nature of the change that this commit was part of - see https://github.com/webbukkit/DynmapCore/commit/8829d86ef889e315cc4d1538e129c805669b6b6f for the details


I'll merge this, but can you try the is_native() call as mentioned in #257?


HVACSizingSim_CoincidentPlant (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


in our code, we use

final String id = UUID.randomUUID().toString();

you can also use

new RandomValueStringGenerator().generate()

to generate GUIDs


we should leave this in so that no matter what the matricom launcher is always set as default


> So it boots into android?

unfortunately , here's another problem with broken shrm. I always was suffered from that when tried to build kernel from something except TC's sources. I uploaded log above.


This should use the [`{% static %}`](https://docs.djangoproject.com/en/1.5/ref/contrib/staticfiles/#std:templatetag-staticfiles-static) template tag syntax.


I'm fine with using "Draw calls".

From: Lawrence Love [mailto:notifications@github.com]
Sent: Tuesday, January 27, 2015 12:21 PM
To: lawlove/vogl
Cc: Peter Lohrmann
Subject: Re: [vogl] UI: Color code timeline for groups (6b3d159)

Yes, thanks for the reminder. I like "Draw Calls" if you don't have a stronger preference... Although I could imagine someone naming one or more of their debug marker groups likewise. How about taking it a bit further to make it more distinctive by prefixing the label with something, e.g. "VOGL", e.g.

VOGL: State changes
VOGL: Draw calls
?

Although if you're comfortable with "Draw calls" by itself I'd be satisfied with that. It can easily be changed if there's any backwash against it.

—
Reply to this email directly or view it on GitHub<https://github.com/lawlove/vogl/commit/6b3d1595ac4f30625919e0f02db67f92d5403a1b#commitcomment-9466745>.


Some release groups (one very popular one in particular) put the filename as the title for the audio tracks in the file so this detects them :)


1. In the INITIAL state is enough
2. There is no way for a rule to apply to all states. If we get rid of <INITIAL>, ML-Lex defaults it to <INITIAL>


You're right! Slipped this in accidentally. Fixed now.


Looks good!


Yes. You are right. I was following this: https://www.lrde.epita.fr/~akim/ccmp/tiger.html#Lexical-Specifications
and thought it's the same as the book. But let's do it according to the book.


This line is missing a closing brace: `}`
Was this template tested?


http://review.cyanogenmod.org/#/c/86325/

ahem


There are so many unnecessary white changes in this commit. I had to look twice to find the actual change.


I was following this: https://www.lrde.epita.fr/~akim/ccmp/tiger.html#Lexical-Specifications. But if the book says only decimal, let's do decimal


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


Yes, I asked that question on Piazza this morning.


What did you use to do the "git push" part? At the moment when I use Emacs with Git it commits changes locally fine, but I've never actually pushed those changes directly from Emacs (admittedly I have never tried very hard).

I tried `M-x magit-push` at one point but it seems the subprocess is waiting for my github username and password but `magit-push` isn't prompting for them.


Merge Balltree implementation


Okay, cool.  The test doesn't belong on the JTS specific class, but that's minor.  I'm glad you didn't have this normalization occur by default... people screw-up X/Y with Lat/Lon all the time (by far the most common error) and the current behavior will throw an exception if latitude is out of bounds which generally occurs when you mix these up (but not always, of course).


Tried a lot of variations of both, but looks like no matter what it still tries launching the zygote process.


Hahaha, nice kang retard! Pretty obvious that you took this from Dirty Unicorns.


I would update the docstring to be a bit more clear.

On Tue Jan 27 2015 at 4:04:15 PM Michael Droettboom <
notifications@github.com> wrote:

> Yeah -- that's just because we don't support up-to-down languages at all
> at the moment (and thus don't track y-advance). Should I update the
> docstring or just change this method to only return x? Either way, it's a
> major project to be more general here, so maybe I'm overengineering this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mdboom/matplotlib/commit/c6d01835816ace3204212d072da1ac3af2b54997#commitcomment-9467512>
> .
>


http://review.cyanogenmod.org/#/c/86598/

Hmmm.


Yea, we can't get here with NULL, Coverity saw this check and then assumed possibility of NULL pointer prior to this in the code, do removing this unnecessary test will prevent Coverity from finding the preceding references suspicious


HVACSizingSim_CoincidentPlant (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


can't see the log... can you give me the link?
On Jan 27, 2015 10:17 PM, "Shilin Victor" <notifications@github.com> wrote:

> So it boots into android?
>
> unfortunately , here's another problem with broken shrm. I always was
> suffered from that when tried to build kernel from something except TC's
> sources. I uploaded log above.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ChronoMonochrome/kernel_common/commit/cb0b2af4f5cabd075cb9d2c027c1df1649e18eb6#commitcomment-9467714>
> .
>


I am an idiot and cannot believe I missed these.


@manny-fp Just ran into this but I don't think we've ever received a pull request. Did you intend to send one?


Should probably be a bool


Adding this state makes the program's logic easier to understand, because we are treating escape sequences within strings as completely different cases. Also, it makes the program easier to change. For example, if we need to add more escape sequence control, we can quickly target the place in the program where we need to change, and change is simple. We don't have to go back and read through what we wrote and try to understand the logic.


I see. Thank you.


Sure. I'll ask this on Piazza to make sure


Indeed. I'll fix it up.


@chrisseto The "extension" pattern is appropriate here because you use multiple providers (entry points) at runtime. The "driver" pattern is for when a single entry point (e.g. a db driver in SQLA, or a client in libcloud), is used at runtime.

Notice with the `ExtensionManager`, you don't need to instantiate a new manager every time you create a provider object.


This will break building on windows since windows does not have bash or make.


I was following this: https://www.lrde.epita.fr/~akim/ccmp/tiger.html#Lexical-Specifications
But the use of a separate state makes it easier to change our program's policy in term of handling escape sequences within strings. See a comment below for rationale of adding a state for escape sequences within strings.


Would it be worth extracting this to a method on `PolyParam` similar to `underlying`, but which considers the current typer state? Similar logic appears, for example, in `PlainPrinter.toText`


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Should be "Two arguments". And you might want to include the fact that the arguments can be files or directories. Or keep it general without mentioning what they are.


saw approval from macintux
at https://github.com/basho/clique/commit/6a1fe808cd67fa54cd286a54fc477c7d463b4968


merging basho/clique/bugfix/global-flags-regression = 6a1fe808 into borshop-integration-50-bugfix/global-flags-regression


basho/clique/bugfix/global-flags-regression = 6a1fe808 merged ok, testing candidate = 3af4db8e


http://review.cyanogenmod.org/#/c/85035/

Ooopppss, I guess.


yes. perhaps we should have a separate `.bat` file for building Windows library?


Refazer o post delayed. Está atrasando o scroll


http://review.cyanogenmod.org/#/c/84966/

Again, man - this is too easy.


Happy to update the homebrew formula once you pushed the binaries.


Now this is interesting arrangement of assertions but I doubt Ray Wenderlich would be happy with that.


Should this be timezoned?


http://review.cyanogenmod.org/#/c/84108/

you even name it the same.


Those files are excluded:
Files generated during the .Net build like obj, bin, NuGet packages.
Avro and Proto buf auto generated files.
Binary data files used in testing

Header are added to other files like .csproj, packages.config, etc.

Review comments are addressed in this push.


Thanks! But now I think it was a bad patch :(

It appears to need either use a more highlevel interface to figure out the default value,
or have a quite complex check for architectures, for example, for the fedora supported
architectures (some secondary ones), the values are not translated, but need to pass
the proper bit pattern:

https://sourceware.org/git/?p=glibc.git;a=blob;f=sysdeps/s390/fpu/fpu_control.h;h=098f0d06f34cc9e6c28342ebb56ff056deb1ebe5;hb=HEAD

https://sourceware.org/git/?p=glibc.git;a=blob;f=sysdeps/arm/fpu_control.h;h=cd750add979f2ff4c574202fcb6310f95516b810;hb=HEAD

https://sourceware.org/git/?p=glibc.git;a=blob;f=sysdeps/powerpc/fpu_control.h;h=4e1847ae53a31807195f68fe9e3456b41b5e1166;hb=HEAD

I think a more highlevel interface would be better, and the proper solution
should be to use the interfaces from fenv.h, that abstract the architecture.


Hey Guys...I just received  last minute request for job interview presentation  in the morning.....Need to practice...So I will be see you next Tuesday...Please keep me inform and I will keep an eye on the  Github..along with more input.


You need to remove this source from within the callback or it will keep firing every 3 seconds!


why does this need to be exported if opt_trace_rc is not set?


also, why is this change necessary?


Mostly seems ok apart from the above comment, however can you split out refactoring and timeout changes into seperate commits?


Most :dog:  TIL ever. Hi @mehulkar !


ack


I think we should follow the book specs (Appendix starting on page 512) because that's what he talked about in class. That resource definitely conflicts with the book


Änderungen von Katharina und Sarah


Personally, I think the documentation would flow better if the example were on the same page as the explanations of each of the features, or merely left in smaller chunks under the relevant descriptions.  Right now, it feels like a lot of foreign concepts with no clear path to find their definitions (even though it really is the next page).


I have a Kendo ComboBox that's configured to use `filter: contains`. When I type a value that doesn't exist in the list then click the dropdown button, the request contains a `logic` property set to `'and'`, but it does not include a `field` property. What do you think about using the following to replace line 43?

`if (filter != null && (filter.Filters.Any() || filter.Field != null))`


Published to http://vui-docs-internal.s3-website-us-east-1.amazonaws.com/34_a06f8a8200700d29277dddbc6ffd31fc6b88875f


Shouldn't this replace [mozSetMessageHandler('notification', ...)](https://github.com/lissyx/gaia/blob/a6877a1dc05b8f538a79b4d5c1beb23493669727/apps/system/js/findmydevice_launcher.js#L163)? It looks like you still kept it.


I will update version.sh next time I update LAV Filters.

For `GCCInfo.o`, note that it was sort of made like this on purpose. I know it's far from perfect but at least it was an easy way to ensure compiler changes wouldn't pass unnoticed. I guess we can decide that it requires a clean to have some detect though.


ack - although this is a possible candidate to be reworked when we get to diff results since it could propose some fancy language dependent line break positions etc.


@jdalton, @mathiasbynens Travis CI now [has support](http://docs.travis-ci.com/user/build-environment-updates/2015-02-03/#iojs_support) for `io.js`.


Package jq and socat do not exist, which leads to errors.


NVM, I thought you had to upload something, but all you need is a tag.


ngModel added to the search-box isolate scope (optional)


Why not acquire this lock in cache_inode_dec_pin_ref() itself  if the flag is true? This is the only place where cache_inode_dec_pin_ref() is called with closefile set to true!


This looks quite similar to [FMDCloseNotifications](https://github.com/lissyx/gaia/blob/a6877a1dc05b8f538a79b4d5c1beb23493669727/apps/system/js/findmydevice_launcher.js#L34), do we really need both?


Confusing, this is.


No because we would need to override the created_at instance when saving also. If we take the stored date like a central time and convert it at a later point it's better than having to change all store dates. I hope it makes sense :)


I see a lot of such changes in pester PRs (white spaces / line ending).
It's better to reduce the amount of such noise in PRs, so reviewers will concentrate on the meaningful changes.
For white-spaces, there is a practice to fail the build, if any source file have trailing spaces. It can be done as a pester test :)


I like those buttons and did not find anything terrible about them when they were faction colored. It was confusing though that Harkonnen was not red, but using a pretty neutral tone that matches the rest of the palette. Then just remove the faction labels from the PNG file I guess and I have to change the UI yaml again. Probably at the weekend.


Not a fan of multiple flags to cache_inode_close  (it confuses me a lot!), but the patch looks correct.


block comment -> line comments


all tests pass:
success: http://buildbot.bos1/builders/test-clique/builds/38


The last sentence would flow better if it was written "It will then be designated as such in various locations."


Nit: can we use `FMD_ENABLE_FAILURE_NOTIFICATION_TAG` here?


You do not default entiy_id to nil; is that ok?


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


Why do we need the pry gem?  If the gem is this one (http://pryrepl.org), why do we need it for running VCR tests?


makes sense


ng-model passed on ui-gmap-search-box


'findmydevice'?


typo: "setls"


add ng-model="ngModel" to template HTML


:+1:  +1


nitpicking: add line new line (git best practice). It also can be added as a test.


"This is helpful in situations where documentation for *a module's contents* is spread over ..." - perhaps?


OK just realized that I wasn't supposed to "close" this pull request...I thought I was closing my comment.


I think you can just do:

    var appId = this.appId;

and then use `appId` in the closure.


Why did this have to be loaded here instead of using `require`?


This could probably automated btw ;)


Whoops, assigning to global variable.


The full dependency graph looks something like this:

```make
$(DEPS): Godeps
build: $(TARGET)
%.pb.go: %.proto $(DEPS)
%.capnp.go: %.capnp $(DEPS)
$(TARGET): $(DEPS) $(GEN_PATHS)
test: $(MOCKS) $(GEN_PATHS)
test-server: $(MOCKS) $(GEN_PATHS)
test-gomc: $(MOCKS) $(GEN_PATHS)
test-gomemcache: $(MOCKS) $(GEN_PATHS)
$(COVER_PATH)/%.out: $(CURDIR)/src/%/*.go $(MOCKS) $(GEN_PATHS)
$(COVER_PATH)/%.server.out: $(CURDIR)/src/%/*.go $(MOCKS) $(GEN_PATHS)
bench: $(MOCKS) $(GEN_PATHS)
bench-server: $(MOCKS) $(GEN_PATHS)
```

That would make it possible to run commands in parallel, too (e.g., running  `make simplepush html-cov -j2` would fetch dependencies, generate Protobuf, Cap'n Proto, and mock source files in parallel, then build the executable and generate a coverage report), but we're already checking the generated files in—and we definitely don't want Travis regenerating those.


Fixes #12 :+1:


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


typo: mariontette


FValue1 and FValue2 seems unused.


mochitest-browser-chrome-[0-9] is in here twice


s/a field/The Kubelet will have an extension/plugin/interface/


This is very good but if you would later need to access some files from other class I would certainly make those methods in a separate class for reuse


why the -1?


Can you supply a sample signal also ? Look at the rtl_433_tests repository to know how.


Unfortunately our actual code style don't cover how we should group `imports`. According to this guides http://iosstyle.us/#class-headers a good approach is to order `imports`like this: `superclass, frameworks, local classes`. I always add empty lines for better separations of these groups and to make code more clear. As you can see in here `CRVImageAsset.h` is superclass and `CRVAssertTypeUtils.h` is local dependency.

Also as far as I can remember, in our previous project, Szymon and Emil also separated `import` groups separated with empty lines :wink:.


* Make pricing policy mixins subclass `PricingPolicyMixin` to make code more DRY.
* Duck typing instead of explicit type checking


Secure as a verb is a bit weird, may want look for others.


Looks good!


Can we always use int?


This empty line is not necessary here


Do we want to publish this in demo mode only? We probably don't want it to be possible to probe someone's personal server (where you don't have install rights) to discover what apps are installed. (It may be that some of our existing publishes are already revealing this info; if so we should fix it.)


Privileged ~ uid=0 or no?  If I specify uid=0 and AllowPlivileged what happens?


I would either change this to  "Failed to read prlude.... : "+e or now wrap it ith a try


indentation


@haneensa How would I put an icon for the email us button? I can't seem to find the class or the link to refer to the image I'd like to place: http://alexlidell.com/wp-content/uploads/2014/11/email-envelope-icon.png


I'll probably leave the example in the README, since that shows up on github and the pypi package description. I agree that it is not so useful on the main page of the package documentation, though (i.e. in docs/index.rst). Thanks!


Typo: `_clear` -> `_reset`.


Note that for good security, the default for the cluster should be that every namespace gets its own security account even without administrator intervention.  The administrator should have to choose to alter the security context.


lol


Hey!

It got so far ! superb !

As for this shrm stuff - disable this in config
CONFIG_U8500_SHRM_ENABLE_FEATURE_SIGNATURE_VERIFICATION

I have it disabled and never touched it.

If that doesn't help, try building this kernel without CONFIG_U8500_SHRM
and/or MODEM altogether.

But anyway it's just a matter of deebugging why it fails, but as long as it
shows a handled error instead of an oops, it's good.
On Jan 27, 2015 10:28 PM, "Shilin Victor" <notifications@github.com> wrote:

> http://pastebin.ru/m2MI3mmd
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ChronoMonochrome/kernel_common/commit/cb0b2af4f5cabd075cb9d2c027c1df1649e18eb6#commitcomment-9467799>
> .
>


According to the Ray Wenderlich code style there should be no spaces after { and before }


true buuut our last implementation is not good enough.


yo


@Optimusprimeums that's a good question! We should probably make an automated test for it. I'll do that after I finish fixing bugs (so never :p)
I think the issue with it on OS X was just that it didn't work at all, not that it was giving inaccurate results, so maybe just replace the mediainfo files and then see if the library is loaded on the Logs tab. For me it says:

Loading MediaInfo library
Loaded MediaInfoLib - v0.7.71


But thats a thing to work on and thats a cool start for providing language dependent/independent things via bearlib. Another thing is a language independent CommentExtractor object or so so we can e.g. make the KeywordBear only search comments.


I think this should be `path.join(__dirname, 'package.json').version`, otherwise it will get the package json relative to `process.cwd()`, right?


@mdavy86 you might find this configuration helpful


did you mean to leave a rebase commit in there?


escribiendo algo adicional


I removed the `GOLDFLAGS` trick because it's never worked in any of our environments—[`dev`](http://push.dev.mozaws.net/realstatus/), [`stage`](http://push.stage.mozaws.net/realstatus/), or [`prod`](https://updates.push.services.mozilla.com/realstatus/) (note the `"version":""` field). :cry: We could do something like this:

```make
GITREF:
	([ -e .git ] && git describe --tags --always HEAD 2>/dev/null || echo $(notdir $(CURDIR))) > $@

$(TARGET): GITREF
    $(GO) build \
        -ldflags "-X $(PACKAGE)/simplepush.VERSION $(shell tr -cd "[:alnum:]" <GITREF) $(GOLDFLAGS)" \
        -tags libmemcached -o $(TARGET) $(PACKAGE)
```

:scream: ...Or have a wrapper shell script that calls the Go binary with `-ldflags`—since we'll need it for every package build. That seems better, but still too clever.


fast-forwarding develop to borshop-integration-50-bugfix/global-flags-regression = 3af4db8e


Creating these ContentValues can probably be refactored out to a helper method since they are pretty identical


Reference the original service account proposal.


.kubeconfig


I'd leave the description as merely "signature", "function signature" or something along those lines.  Calling out parameters gives them a special precedence over other aspects of the signature, when really we just want the user to read the signature section and put in what feels right to them.  Since parameters can also have intents associated with them (and that is very relevant to anyone intending to use that function), those details would be glossed over by calling it out so early and with so little else surrounding it.


for this line a else (without if) would be sufficient.
Why: either it's >0 or it's <0 , if non of both it must be 0, there is no other option


Woo!


May be better as a Timestamp since synced will be true many times in a lifecycle.


Add: The design does not proscribe a method for rotating secrets in use in pods


Thanks for pointing this out.


But I feel the example is fine as is


Will these changes kick in on Feb 2?


bring comment back?


i wanna cry


HVACSizingSim_CoincidentPlant (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (599 of 599 tests passed)

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


Spelling error inspired instead of inspirated


heh, @jdalton beat me to it.  that patch isn't scheduled until feb 3 2015


Link to the upstream Docker "vaults" proposal, the "secrets" patch that red hat tried to get in, how they fit in (doesn't have to be this section).  Link to stp-ip's proposal to Kube and discuss how this proposal modifies it.


JSLint will tell you if you've left out a closing paren. I use a command line version:
````
jslint4java --white --maxerr 10000 --nomen --plusplus --predef on,_,state,sendChat,findObjs,filterObjs,getObj,createObj,Campaign,toBack,toFront,randomInteger,isGM,getAttrByName SCRIPT.js
````


The Biggest of 3 Numbers/The_Biggest_of_3_Numbers.cs

You should never come to this line, right? Otherwise non of the 3 numbers is bigger then 2 others ;-)
So the third test is superfluous, if non of the other two test found a or b to be the biggest number, then it must be implicitly number c.


createPrelude is a bit of a weird name. Unfortunately, I can't think of something better of the top-of myhead. Maybe readPrelude?


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


Closes #140 Show the current game mode during a game


Removal due to rerun needed as input data was lacking needed quality. 2015-01-23 result should be recommitted within a few hours.


Okay


Hrm - this seems really hard to get right.  What happens if i don't specify this?


Need a test using worker.perform with entity_type instead of entity_id


By "the description", you mean the section immediately following the directive, correct?  I don't think that's clear from this paragraph or the example - I'd add it to the example to show placement and more fully describe the connection.  From the perspective of someone with little exposure to the domain syntax (as many members of our team will be), it will make things easier.

I'm envisioning:
.. chpl:function:: signature
    description


Is there a use case for this not being the security context?


What is worker.send?


should this be a promise?


Does it help to use `::unlink()` in line 201? Or do you encounter a real name clash?


Maybe importPrelude instead of addPrelude? Or usePrelude?


Reference out to service accounts and the section on secrets, point out that the key goal is to restrict access to view / edit / retrieve secrets to "admin users" and "kubelets".


Nit: I don't think this class actually implement that interface (or need/should do so), see: https://github.com/mozilla/pdf.js/blob/master/web/interfaces.js#L56-L74.


I'm super excited about this change, thank you!


typo


You loose the value of a!
swapping values of two variables requires a temporary variable to hold the value of one while swapping.
double temp = a;
a = b;
b = temp;


ack - does not conflict with referenced import


let's keep this <0.1 for now please


According to Ray: Conditional bodies should always use braces even when a conditional body could be written without braces (e.g., it is one line only) to prevent errors.


I don't see what you mean when you say you might have to change some UI yaml lines.
I have just removed the Faction labels from the PNG file.


The very first entry from the loop will be same as the one with glist_first_entry() call, right? The code just doesn't seem to process anything, right? Am I missing something?

Looks like you want to record the first state that you move to tail, rather than any first state. That would work.


yes indeed got nothing to hide. most commits are from cyanogenmod dude look into framework!! there you will find the right commits. dont you tell me how i should work or commit.
now fuck off you piece of shit. i told you the last time to dont bother me again with this nonsense.


we shouldn't need this anymore. I removed it in previous commit. Running with gc enabled gives us `gc` API


Dividing method code into small sections separated by empty lines is good approach IMO. In here I've treated `return` as that kind of separte section. In my opinion it makes code more clear and easier to read. This is widely used approach. You can see some examples in [AFNetworking] (https://github.com/AFNetworking/AFNetworking/blob/master/AFNetworking/AFHTTPRequestOperationManager.m) or in [RestKit](https://github.com/RestKit/RestKit/blob/development/Code/CoreData/RKManagedObjectStore.m) (they are more restrictive in here).


:shipit:


ack, merging


You should be able to simply cd into capstone/msvc/ and type msbuild to
build using msvc. Providing the solution files are set up correctly it
should work from the command line.

On 27 January 2015 at 22:27, Nguyen Anh Quynh <notifications@github.com>
wrote:

> yes. perhaps we should have a separate .bat file for building Windows
> library?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aquynh/capstone/commit/39fd1541392d356ec0c96667d375bbb6d84a84fd#commitcomment-9467899>
> .
>


Good luck on the interview!

P.s my trip got pushed a week so I will be attending tonight!

On Tue, Jan 27, 2015 at 1:42 PM, Rena Cummings <notifications@github.com>
wrote:

> Hey Guys...I just received  last minute request for job interview presentation  in the morning.....Need to practice...So I will be see you next Tuesday...Please keep me inform and I will keep an eye on the  Github..along with more input.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/CodeCarrots-Jan-Feb-2015/MedicalCrowdsourcing/commit/8f837ab671b645c84cdd6a638a1e544c796f6d57#commitcomment-9467924


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


good find


I assume this (and the case above) are here because we sometimes access these dimensions? Is it possible to omit these cases? I agree that there is some sense in which these outputs are correct, but if we don't use them, I'd prefer they fail, since in many cases they may indicate bugs...


your the man!!


Ditto on the "signature" note


... and receive the decryption key via another channel (stored in the image, as ENV, as a token retrieved from a URL)


Good catch. Will fix.


Good point. Does this also fix pulling back of the incidents under the right day too? We had an issue where it was added on the 25th (Australia) but was listed under 24th.


I vaguely remember trying that, but don't have any clear evidence that I did. I'l (re?)test it your proposed way.


Hola


whoops indeed.


Maybe not that  a prelude file can be read with `CWorker.readPrelude`  when passed as argument?


replace by

"Proxy mode allows to serve pages from another webserver (like PHP) while continuing to edit JS and CCS into Cakepan source dir. If you want to enable proxy mode, set *proxy.url* and *proxy.watch_dir* in your `app.config.json`"


+1 to only publishing in demo mode


I see what you're doing it here, and it could be useful, but perhaps it deserves a comment? And what if `index + shape_.size() < 0`? (Yes, the method as written succeeds returning `1`, but that probably hides a bug in the caller.)


You're awesome! Just saw https://github.com/Homebrew/homebrew/pull/36294 !

Right now the building & pushing of the binaries is semi-automatic...I still need to run a script after I pushed the new version. I am planning on making it more automatic though :)


Is the original way not working for you in some circumstance?

`File.extname()` should return a file extension from a filename (if there is one), so if what is returned (minus the dot/period doesn't match the filename, it adds it.

    2.0.0-p576 :001 > File.extname('foo')
     => ""
    2.0.0-p576 :002 > File.extname('foo.zip')
     => ".zip"
    2.0.0-p576 :003 > File.extname('foo.zip')[1..-1]
     => "zip"

This also works for filenames with dots in them:

    2.0.0-p576 :004 > File.extname('foo.tar.gz')[1..-1]
     => "gz"

The way you propose doing this here would false positive in some circumstances, where the extension is anywhere else in the filename, like this:

    2.0.0-p576 :005 > "foo.tar.gz".split(".").include?('tar')
     => true


Thanks!


1277852-CommandLineInterface (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1591 of 1596 tests passed)

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


Looks like this duplicates or shadows the type portion of chpl:data::


Responsive data tables CSS was removed from the style-specific custom.css and added to the main theme's custom.css instead, to make it universal.


 glist_for_each(state) {
       if (state == first_moved) {
             we are re-examining states, all must be non-layout type;
              break
       )
      if (this state needs to go to tail) {
           move to tail;
           if not first_moved:
                  first_moved = state
      }
  }


@UniversalMediaServer/developers there seems to be issues with detecting Xbox One and Xbox 360 recently but I don't have either of those devices to test, can one of you with them please check this?
According to logs submitted at http://www.universalmediaserver.com/forum/viewtopic.php?f=9&t=2920 cling is receiving the information from the Xbox One but we aren't recognizing it or connecting.


@rgpass done.


your even blind!! look who signed this off.. hmmm now fuck off.

Btw did you find what you want to steal... oh no all other devs steal and call it there own work
you dont do that... your really pathetic dude


Yeah it should :) can you test it?


вот тут используется :sku, которого еще нету


I'd reorganize things so that data and type come before function and iterfunction.  Mostly because the structures would then be described in ascending complexity and detail.


not with referenced import but maybe with the work you'll do when reworking
CLI args

2015-01-27 22:56 GMT+01:00 Fabian Neuschmidt <notifications@github.com>:

> ack - does not conflict with referenced import
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/ba7f9396133d2c44d5e2ac0c36f429fdeb05d466#commitcomment-9468342>
> .
>


split(a\nb\nc\n).length = 4, but it's 3 lines. Should check to make sure there's a \n as last character before doing this...


If can't contribute by coding (lack of ocaml knowledge) (yet!) I can at least do this :)


reack


Uuups, didn't think about that. Sorry.


Really nice.


I'm baffled by the ternary here; what's the use case?


Yes. I had to do a couple of merges and I was hoping you could double check those as well.


The reference in AzureResourceManager.ps1 is probably incorrect here. It is still  Insights\Microsoft.Azure.Insights.dll instead of Insights\Microsoft.Azure.Commands.Insights.dll as it should be.


Actually, I might add it to a more general example, since all the directives will have the potential for a description.


it doesn't have to be (and thus shouldn't)


`std::accumulate` is a thing, but this way is fine with me.


давай пока без этого, если тут проблемы, уберем эту фичу. Не критично на данный момент


mod 4 is HP > MP    mod 5 is MP


In our previous big project we had non written rule that if there is only one line in `if` statement we can place it in one line. It was exception from guidelines (similar to use 4 spaces instead of 2 :wink:).


thanks for advice, will try tomorrow, for now gonna go to sleep :)


I dislike that these end up on the same line.  I also would consider the constructor closer to a method on the class.  If the class designer has made multiple constructors, how would the second one be specified?


1277852-CommandLineInterface (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1585 of 1590 tests passed)

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


Nit: comments should be sentences - capitals and punctuation


r+


This is a *small* optimization, but would it make sense to only do `getFileName` once?

E.g. you could either let `_bindLink` return the `fileName`, or perhaps pass `item.content` and `getFileName(item.filename)` as parameters to `_bindLink`.

Not sure if this matters much, and I can't decide which solution I prefer :-)


This probably shouldn't be exposed in the kernel API.


Don't worry about it now, but just to make a note, we should think about how to generalize this.


(using inline docs implies that it will be)


Great commit! will need to discuss it the day after tomorrow! good job!


saw approval from glennw
at https://github.com/servo/servo/commit/134b36b47a2885848d628d971a0db03a0e88e3bd


merging servo/servo/rustup_20150109 = 134b36b4 into auto


servo/servo/rustup_20150109 = 134b36b4 merged ok, testing candidate = 8c7fe847


LGTM!


Ditto on "signature"


Says the guy who kanged his entire repo


And here.


I don't understand what happened here. Why is my merge here??


Thanks for the tip!


That's aggressive, I like it.


As stated in the commit message, gcc -E showed that these were just 3 printf() calls, although there is a funtion of the same name declared somewhere else in the project...


I'm not completely sure as to why these are marked as no-ops to gettext (N_(...)), given that they are the parts I'd think would be translated. Or will they be translated when use[i] is referenced on line 2090?


Ref #2726, #3005, #2970. I'd like to use field tags to clear user-provided values in readonly fields prior to invoking initialization, defaulting, and validation.

One option for controllers to set status is a "custom verb", akin to resize, stop, or clone. We could support a /methods/updatestatus "verb" (aka synthetic control resource) at a subpath under the object.


@eschlange this should also have `null: false`


...sorry, got it; Object#send - this bypasses 'private'?


/cc @davidopp


"can be specified similar to how they would be written in Chapel"


see above


Don't know either, we can see tomorrow!


Flags (or other information from the calling environment, at any rate), is the appropriate way to resolve this type of issue, which arises merely from the conflicting assumption of the calling and called modules.


Constructors can also be put in `.. chpl:method::` directives. That's where I would expect `chpldoc` to put them.


I've tried to do this where I can, but haven't for strings that are basically consts. Are you talkinga bout:
"CREATE TABLE IF NOT EXISTS \(name) (\(self.rows))"
or is there a particular place where you think we should bind instead (everywhere, even for TableName?)? I'm happy to fix up any.


I think I have a better fix than this, because `cl` is deprecated: https://github.com/luismbo/slime/commit/70f24959dda953e7dbb26cb97939bab47698133a Do you agree?


If you decide to consider constructors similar to other methods, the reference to classes and records in this sentence can be dropped


See the signatures section.


Good catch, I didn't intend to expose it, but I copied the documentation from monthLength and forgot to change it. I'll change it.


that is, the recursively called module(s)


agreed. removed


delayed retry looks good to me. In Taskmanager are there any other places where we'll encounter synchronization issues related to the queues?


For now, I was going with what you had. I agree there are improvements that would be nice to make, but for now I wanted to fix the immediately broken parts of the app. (The app was completely broken if you selected any theme other than “None”, presumably because the CSS file couldn’t be loaded.)

Our current implementation supports full-width-solid-stripe-top-bar-style headers, and nothing else. So something like the bar on the top of the [HubSpot-style docs](http://github.hubspot.com/vex/) works. The GitHub nav might also work.

To support anything else we’ll probably need to be more sophisticated about the left, right, and top positioning (possibly using JS), and the `offset` we pass to Headroom’s `init` (and possibly `tolerance`). A good example you might want to look at is the `#nav-wrap` element on [http://mitadmissions.org/](mitadmissions.org). If we can support something like that in a generic way, I think we win.

Cloning the element might definitely pose a problem for people who have initialized JS (dropdowns, etc.) in their header. So we may want to have an option for that. Although not-cloning will make it tricky to seamlessly transition between the `unpinned` and `pinned` states.


@thockin We absolutely want to support the general pattern of GET, POST, GET. We want to make it easy to both create objects from other objects (e.g., clone) and to create clean config files from existing objects:
https://github.com/bgrant0607/kubernetes/blob/config/docs/cli-roadmap.md

This is critical to facilitate the transition from imperative to declarative management, but also for import/export across namespaces and across clusters.

One example where we'd want to do this for objects with generated names is pods spawned by a replication controller, which should switch to this mechanism.


Might be nice to document that this can also take an IP address:port combo, and that the default port is 2003?


Is it intended that you floor the kept probabilities and not just the outputted scores?


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=d99e533fa3ab4477a4893c777fa7d324
 plone-5.0-python-2.7 [FAILURE]


Oh, I see! Outline cache invalidation already used to trigger a bitmap cache invalidation, so it isn’t broken!


The which command will return an empty string when presented with an absolute path.  So when I run
  --php5=/usr/bin/php-cgi
this invariant fails.


It is not a good practice to manipulate the DOM directly in the controller. Can you move this functionality to a directive? Or if that is too much effort, you can even do `ng-class` based on a scope variable.


is there a special reason why you want to avoid closures? If you care about performance, you probably don't want to use a single-threaded FS anyway.


I'm not sure I know what you mean. Could you make a jsbin showing this?


Love it.


Test comment. Dbj checkbox mode, additional commit just to make it work correctly


Dear all, I tried to upload some java files. I hope that I succeded.


Well, i'm more familiar with "normal" gobject, where if method returns false the timeout isn't firing anymore (i.e. stops); on my system, it does stop when if finds a network interface that it up (which was the goal).

Also, i'm afraid that the refactoring cannot be really separated from the timeout changes because the function didn't exist per se, as such i couldn't call it with a timeout. I can try to split the other changes if you insist (logging changes, _nm suffix method naming addition).


some tests failed:
failure: http://build.servo.org/builders/linux2/builds/597


If I wanted to 'steal' I would make sure I maintained authorship. If you knew how to dev properly you'd have done the same.

Also, because I am pathetic lets amend some erroneous spelling, or should I say use of words.

there - their
your - you're x 2


Any chance of infinite loop?


Please dont remove my copyright!

I invested many many hours to create this theme, so please keep it!

Thank you very much :)


Yes.  In the interest of keeping constructors together, and of keeping methods together, I would drop this option here.


It is not a good practice to manipulate the DOM directly in the controller. Can you move this functionality to a directive? Or if that is too much effort, you can even do ng-class based on a scope variable.


Luís Oliveira <notifications@github.com> writes:

> I think I have a better fix than this, because `cl` is deprecated:
> https://github.com/luismbo/slime/commit/70f24959dda953e7dbb26cb97939bab47698133a
> Do you agree?

Yes.

I didn't figure out what to instead of lexical-let and didn't want to
research whether old emacs versions would work with the replacement I
found.


Hmm.  This isn't valid Chapel code, type statements don't support a : TYPE.  To avoid confusion, the ":" should be an "="


Hmm.  I was going to complain about exists_key rather than :exists.  But on consideration maybe (%default__port{$scheme} // Int).Int ?


Remove this


>  that patch isn't scheduled until feb 3 2015

@jdalton, @phated Yes. Sorry for my poor wording and the confusion.


At least locally, this change generated more changes in the Wxi file too. In the section for Resources.


rax retry tests


"Type of the yield value"?


`GrammarDefinition` is the abstract and empty grammar template of the PetitParser library. It doesn't contain any grammar. `XmlGrammarDefinition` is the abstract XML grammar. It doesn't specify any parse actions but provides the template methods.  `XmlParserDefinition` adds these abstract parse actions that build the default parse nodes. If I now want to build a different kind of tree I need to extend the XML grammar defined in `XmlGrammarDefinition`; however, if `XmlGrammarDefinition` hardcodes the types generated in the sub-class `XmlParserDefinition` I cannot do that.

I agree that this is confusing. Probably PetitParser should add support for defining parse actions separately from the grammar (delegation instead of subclassing seems a good idea in most cases). This is just a bit hard to do now without breaking all users ...


That might be a good option


if you pass T*& p you could also set p = NULL and avoid any issues with dup finish calls.


Waiting for this to get updated as per discussion in the issue.


The `vector` constructor parameters are unfortunately a bit confusing, but I think we'll just have to accept that.


Order of precedence


Looks good! Ship her!


It was deliberate to put function first. It is not as complex as classes/method, but more useful for someone familiar with sphinx and domain. It's a quick tutorial, with links to other sections with info on the info fields and the signatures.

I'm open to putting data and type first, but I think the current form better accommodates both the expert and beginner users.


player.getGroup().getMembers()


I assume this is meant to say `:type keys:` and ditto for `:param keys:` below?


Oops, was supposed to set first to NULL here...