Вайтспейсы. Их хорошо видно, если делать `git diff --cached` непосредственно перед тем, как коммитить.


Лучше не коммитить дебажные принты.
Если очень нужны для текущей отладки, используй `git add -p`.


Хорошей практикой считается объявлять переменные через интерфейс, а конкретную реализацию указывать только при создании (если, конечно, на нее не завязан какой-то функционал).
Например,
```java
List<Node> someList = new ArrayList<Node>();
```
В общем, везде в левой части иметь просто `List`.

Мы недавно перевели все в либе на такую схему, теперь ее придерживаемся.


Комментарий уже потерял свой смысл (:


@ExSport https uses same port, i.e. 9001 by default.


+1 for removal. Why would you need a useless action in a tutorial?


This is problematic line. @scytacki, it's present on production, as it was introduced a few days ago (https://github.com/concord-consortium/lara/commit/d0c0b604f2a61445205cbe37f483ee494b13aac9). Probably not very important for this release, but without this fix the global interactive state functionality is broken, so we can't test / demo it on production.


no!!!!


This filter seems to take a sequence of 32 items and move the even-indexed items (0, 2, 4...) to the front, followed by the odd-indexed items.  You can also express this with `new Splitjoin<Float, Float>(new RoundrobinSplitter(1), new RoundrobinJoiner(16), new Identity<Float>(), new Identity<Float>())`.  The splitter sends the even-indexed items to the left branch and the odd-indexed items to the right branch, and the joiner merges 16 items at a time.  StreamJIT can't turn this entirely into an index computation, but at least the resulting copies don't require allocating a temporary array during each execution like this filter does.


if move was legal, in this function you also need to:
1. update enPassantFile variable accordingly if pawn uses its initial move option to advance two squares
2. consider promotion move (as you use move_info as an argument to this function)
3. update kingIsChecked variable if with this move you put oponent's move under check

later when you add king move functionality we also need to update castling allowance in this function (you can add todo here for now)


Is the change in InitializeSystemListener really related to the diff rendering?


develop (axelstudios) - x86_64-MacOS-10.9-clang: Build Failed

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


whoops, that typo slipped off my sight!


Funny story: Spaces are getting removed instead of replaced with dashes.


@davidchappelle Makes sense. I submitted my commit as a PR to your fork. I really think the "single overloaded `provide` method" makes for the most usable API, and would strongly prefer an implementation that works this way.


make sure after this move king is not under check


tried that already...


@orxx I think the "stupidHack" code for the BodySlideApp CharHook is still used. It makes it possible to type the needed characters into the slider readout fields (I think some numbers couldn't be typed because of another event or something).


I filed https://smartos.org/bugview/OS-4205 to track the CIFS problem
separately from the old NFS issue, although right now there is no info in
that bug report. I'll put something together that will help collect some
info and pass it on to you.

Thanks again,
Jerry

On Mon, Apr 20, 2015 at 1:32 PM, Jorge Schrauwen <notifications@github.com>
wrote:

> For now it seems to choke on idmap and smb/client binaries not running.
> But sure, I can run a DTrace script somewhere later this week.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/joyent/illumos-joyent/commit/7652b10523b05b4d89e5deb010e8b0ea590ceb7d#commitcomment-10810362>
> .
>


It seems a bit faster on task list (as guest) with test install of ~25000 tasks which are subtasks of 10 tasks (ids 1-10)
drops from 1.0 sec -> 0.4 sec

Toplevel/Overview page still a bit slow (1.6-1.7 sec here) both versions 20150416 and todays 20150420

And still have a "white page" problem with it here:
/flyspray-beta/index.php?do=details&task_id=9&project=0&taskid=25085
try to dig into it to find reason .. probably again memory limit problem ...

btw: today one patch of me got accepted to adodb. :-) so I hope a new adodb version is coming.. last release is a year old.


C'est vraiment bizarre, c'est comme si le rouge avait du bleu mélangé dedans... Pourtant les lampes sont bien blanches. Le orange est devenu jaune aussi j'ai l'impression. Mais en tout cas ça commence vraiment à avoir l'air bien !


Hm, actually I can't really see why `SaveIndicator` constructor is exported on document.ready. I guess I would just remove all that unnecessary .load / .ready handlers in the future (assuming it's really unnecessary and I haven't missed something).


Why not use a `<pre>`?


I think `y` is only used inside `work`, so you should remove this field and just use a local variable instead.  As is, if StreamJIT decides to parallelize this filter, you have a data race reading and writing `y`.


Hmm... just tried it again and it seems to work this time. Will try and change on all...


Perfect, thanks :D


On Sun, Apr 19, 2015 at 06:50:01PM -0700, daurnimator wrote:
> Why not get rid of `auxL_Unsigned`? it's not used anyway...
>

I wanted to use it for something, but I forgot what. Or maybe I was just
hashing out the API, and I was going to end up using that code in other
modules.

Will need to think about it.

Still a bunch of refactoring work that's needed, anyhow. Like moving to a
more Lua 5.3-ish auxiliary API.


How about a bubbling jetpack sound when in water?


I don't like the fact that getPlan() method needs a boolean resetQuota. The rebalance "plan" should have nothing to do with quota reset or not. It's before/after execution of this plan we want to disable/enable the quota.


._.  So make one that works.


I think this can be `static final`.


I could live with clearing it (or putting in ".*") but surely we can come up with something that allows all letters, all accented letters, and common punctuations (dash and dot).


Good idea, I suppose.


develop (axelstudios) - i386-Windows-7-VisualStudio-12: Build Failed

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


Again, these can be lifted into `private static final` fields.


:+1:  for stock data :)


cinco de choclo


about the WSOD, try to change URL to

/flyspray-beta/index.php?do=details&task_id=9&taskid=25085

if it helps.


Ah, ok, I see why now. It didn't like the ones with http instead of mailto


There was a typo in the commit message. This commit fixes #136.


lgtm


It may be faster to use a lookup table here (by adding 5 to make the -5 case 0), even though you won't use all entries of the table.


me estas cargando?


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Will this be in the release?


How does this sentence sound instead?
Create a disk RAID pool and allocated entire space to new volume.


Maybe we should use different name to avoid confusion because the init function qvector(int options, size_t num, size_t size) takes "num" as max number of elements but here we're using num for number of elements stored and "alloc" is for it. So I suggest to change them to:

num => num
esize => objsize
alloc => max

also change the init func's parameter names to match.

qvector(int options, size_t num, size_t size) => qvector(size_t max, size_t objsize, int options)


ssl_push creates a new userdata object, and pushprotos creates a new table and a bunch of strings. Any of those operations could fail.


Это было сделано автоформатированием?


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


Пропущенные ньюлайны в конце файла тоже хорошо видно на `git diff --cached`. Лучше их не забывать.


Не очень понимаю, как ты тут объявляешь переменную `final`, а инициализируешь ее позднее? Тут нет ошибки или хотя бы ворнинга? Это `Java8` позволяет?


Can we use `-Dnsa.backdoor=false`, or do we have to wait for java9 for that?


Здесь разве не нужно тоже завернуть в локи?


Почему закомменчено?


See comment above for possible reword.


I'm not sure if the word 'connotation' fits here.. 'implications' maybe?


И расставь везде `final`.


On a more serious note, I'll try to track down memory leaks.


Thanks for ruining the formatting. /s


Instead of repeatedly shrinking `value` here, it might be faster to check `value & (1 << i)` to break the data dependency on `value`, allowing the compiler to unroll the loop.  (The JVM might be smart enough to do this on its own, but I wouldn't count on it.)

But maybe even better, you seem to have a small range of possible inputs and outputs from this filter.  Can you turn the whole filter into a lookup table?


ok can we meet and explain this what you guys want it to do. Cause I think we are having a misunderstanding here. Are you free right now?


Guys, the problem is that ecto is not only postgres specific!

at least should be handled param (--database=...)
take a look here: https://github.com/elixir-lang/ecto

now you making decision for other people about database driver, this is not best idea


I am not sure if there is a valid scenario, but at least previously this use case with external parse transformation on erl first files worked (in theory), so I am sending a fix. Basically it's completely new version of this commit :-)


Yes, that word seems to fit better... :+1:


@dkayiwa actually, thinking about this more...

If we want a default regex, we should leverage \p{Lower} and \p{Upper}.

However I don't know if these correctly allow non-latin characters. I'm going to ask the implementers list whether people think that the default out of the box should support only latin (and be changeable by this GP) or if we should just allow anything?


@springraar can you please test if this change fixes your crash?

http://pastebin.com/QJCQgkfk


no difference. WSOD.

project_id=0 means global task list and because we don't store project_id in cookie anymore it should be the same like /flyspray-beta/index.php?do=details&task_id=9&taskid=25085
because we set project id to 0 (global) if no project_id is given by url.


@EmilyRandall what is this? Grunt doesn't like this, and it interferes with the code I've already written for ConfigurePotentialPanel.js . What's the purpose of this?


Why was the deletion of pLineEndBuf in the destructor commented out in this commit?  It is causing a memory leak.


Thanks!


@springraaar can you please test if this change fixes your crash?

http://pastebin.com/QJCQgkfk


Convention has been noun -> verb.  Should this be lsm_plug_volume_raid_create instead?


I think this should be:
if (index < vector->max) ==> this is for NOENT case
otherwise ===> ERANGE case


Replace 'introduction' with post. Introduction sounds like it was chapter 1 of a 12 chapter book :)


Как лучше передать сюда параметр с Url проекта, учитывая что код скрипта будет вынесен в отдельный js-файл?


Convention has been noun -> verb. Should this be lsm_plug_volume_raid_create_get instead?


Yes, I'm curious as well. `neovim.org` conveyed that Neovim is an organization as well as an editor.


lgtm


You have already apologised for not writing idiomatic ruby, I think it's fine to leave this one out


There is still one ```<p />``` here (I didn't check thoroughly if there were others, but Ctrl-F should do the trick :-).


Yes, looks like this has been fixed in #66.


Wouldn't it make sense to drop the inline qualifier entirely here? (and the static qualifier too). Its value is discutable..


@andrzejsliwa precisely why I added this: https://github.com/phoenixframework/phoenix/issues/788


"...although small and _harmless_..."


or "small and harmless looking"


...we will _get_ there...


Yes. you're right. I don't remember what I thought precisely at that moment to use that word. :+1:


Oh look, it highlighted the changes, thats very clever


This pattern will deopt on empty string. Need bounds check:

```js
var first = object.length ? object.charCodeAt(0) : 0;
```


Are we certain that all raid implementation will enforce the same limitations when creating a RAID with regards to number of disks?


Scratch the _a bit_. It feels out of place.


I'm actually a little bit confused here. How is
```ruby
for name in eligible_driver_names
  $stdout.puts(name)
end
```

better than:
```ruby
each.call eligible_driver_names, lambda{|name| $stdout.puts name}
```
It's a little bit unconvincing to me.


:+1: Just what I was searching to find


I see your point. This paragraph was written before that section of the clarification note. And leaving it out that seems reasonable. :+1:


You mean __new, not __call


As above, should this be volume_raid_create instead?


You asked for it: cpt. Obvious here ;-)


@themightychris good point, I've removed the json dependency in the latest commit.


nice use of computed prop


All these fields can be `final`.


Hi @brianchandotcom!
I think the problem is here: the java2wsdl Ant task looks through the classes, so it needs the full classpath. As you can see [here](https://github.com/Ithildir/liferay-portal/blob/LPS-55068/util-java/src/com/liferay/util/ant/Java2WsddTask.java#L99), I added a `classPath` parameter so it's possible to invoke the tool and specify an additional classpath: this will be useful with the Gradle plugin, where the classpath will be just the Gradle one, and we'll be able to download portal-service as a dependency and pass it to the tool.


@cameronscott137 this is a hack to get the footer to look right... basically, there's 10ish pixels of space between the bottom of the footer and the bottom of the available browsing area. I noticed this with the HTML/CSS you handed off and with how it's implemented in this repo ATM.

thoughts?


Is that going to take care of actually disabling the button though?


can we use another name eg :set_services  instead. it wont merge cause initialize was reserved name,


I acknowledge that using a disabled button here, with a label giving a status, is a total bastardisation btw :-)  But it was easy.


呃是呢，好眼力~


Would `lua_checkstack` be enough here?


太不负责任了


hello


@lembacon 这个小玩意儿有没有什么槽点啊？有的话赶紧吐啊。😆


**hi**


No sweat bro. Nice to see you working on your own kernel! What variants will it be for? Also hit me up on hangouts:

zkennedy87@gmail.com

I always enjoy helping Dev or just talking with like minded ppl.


rename: **transform_pixel**


rename **draw_fractal_color_at**


I just copied the particular test code, but your approach is fine as well.

Hmm, yeah those other failures seem like inconsistencies that should be fixed on the Mono side (though as you said they don't look related to your change, so can be deferred).


Um step pode ser criado para esse And, já que será reutilizado pelo anterior.


Ok, thanks for the insights here. The fact that only the top level outlet has an `OutletView` helps me make sense of what I've seen. I've tried setting the `OutletView` `tagName` to 'div' and I'm running into a few issues (Mostly `Ember.View.views should be empty` errors from `ember-dev`. I'll see if I can work through the test failures.


faltou o "->"


Looks good to me.

Perhaps we should use a regular expression there so that we don't test for two almost equal things (which would, then, be easier to human eyes), but I'm not so sure if we should invest time in this. Should we? If yes, then, please file an issue so that we don't forget to implement this in the future... :)


Thanks, @sueloverso: this lgtm.


And from the looks of it, you could also adhere to using an interface instead of a concrete implementation to preserve SOLID.
```php
public function __construct(string $code, Titon\Io\Bundle $localeBundle, Titon\Io\Bundle $messageBundle) { ... }
```


Yeah that should be changed too. Tech debt.


O controller que deveria ter esse método


h


ping @liris


typo; fourm/forum


Oops thank you very much, I'll fix this :-)


@gep13 guess there were quite a few issues here.


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


needs @params here


Watch the indentation please.


Earlier versions of Rails did not require the explicit declaration of `null: false` for timestamps because they were automatically `null: false`. Rails 4 changed that, so it's now best practice for `null: false` to be added.


good to have a simple java doc comment on all constructors, even the default ones like this one.
/**
  * Default, no-argument constructor.
 */


You're right, I missed that it was never iterated directly so its order would not matter.


@ return a formatted String representation of this instance.

you get the idea ;)


マジックナンバーは避けた方が良い。
7が何を表してるか分からないから、意味の分かる変数の形で定義しておくのが良い。


jaja! merge my pull request pending! I changed it to "respond" instead of "hear". He will only troll when summoned.


Reviewed.  You are also handling the possibility of multiple versions of the same bundle.  Well done.  Did not test.


El estándar que estamos utilizando es crear la consulta query, un string, de forma separada y no al interior del get_records_sql. También recordar que la consulta debe ser escriba con doble comillas, ejemplo $querySessionToClose = "SELECT id FROM ... " ; de esta manera github la mostrar el código reconoce el trozo como código sql.


El estándar que estamos utilizando es crear la consulta query, un string, de forma separada y no al interior del get_records_sql. También recordar que la consulta debe ser escriba con doble comillas para que github lo interprete como código sql.


%0x, %0o or %0b for Extra?


Yay! Thank you. :+1:


a


José Luis Enríquez Flores


after make this modification pfsense stopped completely
i think it has a mistake


I don't know if we need the check `cur_token.value == 'True' or cur_token.value == 'False'`. We want to not split after a `not` for all tokens, right?


Why was this deleted? If it's getting in the way of the `not` operator, then maybe place it after the `prev_token.value == 'not'` check?


Did you mean the name of service file? Of course we can.


test


I don't know if it's necessary to create a whole new file for this check. You can place it in the `reformatter_test.py` file.


ttt dddd


nope but `before_filter :initialize` never did this rather `before_filter :set_services` instead


If the entity is not close. Wurst displays a bug message. Like .invsee when a player is out of range.


I would change 'purchase' to 'get'


Oh ok. We can change it.


thx


I would love to modify this to contine to "hear", but to reply in some random way. Summoning hubot wouldn't have quite the element if surprise that makes me laugh.


Well, I just move this two lines to the end of the functions. 'cause if these lines stay in the top of method, the "if prev_token.value == 'not' and (cur_token.value == 'True' or cur_token.value == 'False'):" verification will never be called.


Yeah, make sense!


How about "0b%b" to make it clear that it is binary?


Inaceptable la manera de crear inputs. Se debe crear formulario correspondiente en form.php, luego ser invocado en código.
$mform->addElement('checkbox', 'sessions', get_string('sessions', 'attendance'));


true true, I'll set something random to it this week :D


falta at <nome da pagina>


Se debe investigar la correcta forma de enviar la información. El input hidden no lo es.


Inaceptable la manera de crear inputs. Se debe crear formulario correspondiente en form.php, luego ser invocado en código.


You've been pushing commits more often than I thought....


はやり問題ですか。
とりあえずはread_battery()を追加するんですかね。

2015年4月19日 19:09 Kei Okada <notifications@github.com>:

> read_battery()みたいに足すならOKとする（iobをいじれない利用者は自分たちでread_battery()とセットする）
> という感じか，コンパイル時に-DIOB_VERSION_2するとbatteryがつかえるようにするとか，
> どちらかというかんじでしょうか．
>
> ところで，travisでチェックしているcprotoというプログラム12.04と14.04で挙動が違いますね．14だと何も出力しません．
> https://launchpad.net/ubuntu/+source/cproto/4.7j-3/+build/2851088
> から，debを撮ってきたら動くから，ソースが違うんですかね．
>
> https://launchpad.net/ubuntu/+source/cproto
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fkanehiro/hrpsys-base/commit/47cf3605ad409585aee23a9e7aa1e290471247fd#commitcomment-10791615>
> .
>


@maximilien, this case is covered in "it 'deletes the service instance with the given guid'", so we won't add new test case for it


I can move the test to the reformatter_test.py file. I just put it in a separated file to show clearly what I did.


I couldn't resist trying it and it works!


Alright, however it still matters for openssl library, those should be with quotes I believe since we're trying to enforce the usage of the version we're providing.


falta implementação


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


The ProgId doesn't need the underscore


Deployed!  Works for me too.


把这行代码注释了就好了o(╯□╰)o


rax retry tests


should cause heart.png to be transparent initially, but not working?


Does it need a clock pulse to start outputting the first sample or will it do so immediately after programming?


I had planned for this to record the actual vales of the tiles in the path, but it didn't work the way I expected it to.  I'll look into it later.


I have a collection:
[{name:'a',description:'this is document a'},{name:'b',description:'this is document b'}]

I want to render it in a list which only need the name field.

So, the url is:
http://api.abc.com/db/collection?fields={"name":1}


It seems that most of the changes are removing the `log4js` module. The original purpose of using it is that there was no good way to print log for both development and product in old express framework. And now express 4.0 use `morgan` and `debug` for product and development environment respectively so `log4js` is indeed not needed anymore.

And I have a question about `morgan`. Did it connect and override the `console.*` methods? If so, under product situation the log can be directly generated by calls of `console.log`, `console.error`, etc.


The test will be skipped if !pxi6733_device so this if/else is dead code.


Is this a better way? or you just don't like it so you change it.


hpwh_residential (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1147 of 1151 tests passed)

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


_A bit_ still doesn't feel right. Maybe try _a part_?


Why are you posting commits as monke_00 but doing everything else as CircuitLord?


r+


I agree on the removing the part that remembers the clarification note. But this one is a bit different in the sense that it tries to fix the confusion with having the `for x in xs` syntax which it really is a xs.each with a lot of sugar...


aahh,

     &task_id=9
     &taskid=25085

doesn't make sense. Clicked from subtask 25085 view on the parent task 9 and
/index.php?do=details&task_id=9&taskid=25085 was the link. (a bit wrong! which one to choose..)
![flyspray_master-beta](https://cloud.githubusercontent.com/assets/1839154/7239242/91dfaff4-e7a8-11e4-8001-27663a82035f.png)

/index.php?do=details&task_id=9 brings WSOD, because task 9 has thousands of subtasks in my test installation.


Use lowercase-b `byte` whenever possible (which should be all the time).  StreamJIT will remove the implicit unboxing of `peek` and `pop` return values and the implicit boxing of `push` arguments, but it will not make global changes to the body of `work` functions.


Oh! yes. :+1:


This change will break our production URLs.  I initially tried to use `urljoin`, but it behaves very strangely with paths that begin or end with `/`, which is why I wrote this helper function instead.

The static url is defined as `"/static/"`, and the `gulp-rev-all` plugin outputs locations with a leading `/`.

When I run that through `urljoin` with a sample file, I get this for output:
```python
>>> from urlparse import urljoin
>>> urljoin("/static/", "/font/fontello/css/entypo.42e8ea1e.css")
'/font/fontello/css/entypo.42e8ea1e.css'
```
which is missing the leading `/static/`


Nice!


So if vsync completion the only event that can happen?


Lowercase-b `byte`, again.


Я не знаю, это было у тебя, я смотрел диффом. Я решил, что надо либо перенести `STRING` вниз, либо `"string")` наверх. Остановился на 2м варианте


In this case, two symbols got created. Is it significant that you chose to return the local, instead of the export?


Thanks!


@scrogson thanks!


Try using Android studio, I will try to look into it tonight, or tomorrow.
David
On Apr 20, 2015 11:36 AM, "LegoYoda112" <notifications@github.com> wrote:

> The Toolkit files do not work on linux, they seam to be windows specific.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/archerD/FTC-autoWriter/commit/049ff2fcfa2d502d1ca7a2c767d3a1b7662c09db#commitcomment-10809304>
> .
>


May be worth to unroll for cases like this one: https://github.com/nodeca/js-yaml/issues/132#issuecomment-91338837?


:+1: well explained.


I would explicitly return undefined here


You realize that things like this are quite toxic for teamwork? It's nice that you have an opinion on how things should work, but someone actually spent quite some time consulting multiple other people on how to implement this the best way.

My advice: If you don't like how something is implemented, ask the submitter and give them a chance to convince you. If they can't, then it's your decision that counts in the end---but they will feel respected.

You do not want people to get the impression that you don't care about the time and energy they put into a PR.


What do you mean, grunt doesn’t like it? I just ran grunt, and it worked fine (except for a warning that step is defined but never used in WaveFunctionPlotNode).  Passing in options like that makes the sim compliant with PhET's standards.


How about adding some options for the default resize actions like
QVECTOR_RESIZE_DOUBLE <= always do (2 * current size) which would be a default action.
QVECTOR_RESIZE_LINEAR  <= always add up (current size + initial size)
QVECTOR_RESIZE_EXACT <= add up only as much as needed when it's full.


I want to denote here the idea that generalisation is not a black or white switch. It is more like a grayscale that you can push further if you want.
Not sure how to express that this refactor make that code a bit more generic without using `a bit`...


Sorry, fix is live in a minute


You're modifying a field from a `work` method.  Make the field a local variable to avoid data races.  (Have you checked that StreamJIT with Compiler2StreamCompiler gives the same results as your other implementations?  Data races like this will lead to unpredictable results if StreamJIT chooses to parallelize this filter.)


This change should probably be with the other commit.


Ни того, ни другого. По ходу позволяет. Уберу.


'...or several clusters. Or..'  is a little clunky


I like these much better than those implicit fallthroughs that we had before


I assume this will be dealt with later? I haven't been keeping track of this PR, so sorry if I'm missing something.


Disregard that; I had a merge conflict elsewhere when pulling your changes, and a stray - was left in the code nearby, causing grunt to gag. It does compile, I'm just a dumbass.


What do you think about: `But that will solve a part of the problem and only for this concrete scenario.`


Yeah, also curious as to what a *.io domain in general connotes and brings to neovim? Seems like ever since the Google IO conferences from a few years back that a ton of startups are going with *.io domains.


Везде это где? Не совсем понимаю где их надо, а где не надо использовать.


This one is a node, not a symbol, right?


Are Decoder1 and Decoder2 different?


This needs to be QueueNames.retrieveProvider. the queue variable in line 42 needs to be the destination queue of the producer.


These arrays are entirely unused.


Coursera Project Part3


Yes, pretty much.
http://manpages.ubuntu.com/manpages/saucy/man3/drmHandleEvent.3.html
the other event that could happen would be *vblank*. But we don't support it here, and according to robin, it's okay.


`final`


Потому что добавил `cancel()` в таймертаске этого таймера. Надо удалить вообще.


## This is a markdown file to the document.


into


'In object oriented languages...'


Do you notice something? The readline.h include is completely superfluous here.


Sorry for asking. But could you explain why we need this overrides? What is wrong with the defaults?


You can make this a 5-to-1 filter instead and give StreamJIT a bit more flexibility in scheduling.


Да, согласен, надо бы.


They actually **are** in scope in their containing enum


Yep Heroku just let me know that they pushed a fix


Very impressive work. I left some of my comments above.


Same here. The readline.h include is superfluous and should be replaced by ctype.h for all cases.


Also, that is what Android studio shows as the code behind the image
creation for a image icon, so I do not know why it would not work in Linux.
On Apr 20, 2015 11:36 AM, "LegoYoda112" <notifications@github.com> wrote:

> The Toolkit files do not work on linux, they seam to be windows specific.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/archerD/FTC-autoWriter/commit/049ff2fcfa2d502d1ca7a2c767d3a1b7662c09db#commitcomment-10809304>
> .
>


Everything you removed I added xD


Hopefully `Printer` is just for debugging and removed during actual benchmarking?


No worries - that was a speedy fix, thanks!

On Mon, Apr 20, 2015 at 4:05 PM, Felix Krause <notifications@github.com>
wrote:

> Sorry, fix is live in a minute
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/KrauseFx/fastlane/commit/b522ed292fa65def2680cc0eed5a6a71ff52e992#commitcomment-10811069


That rule is not implemented by putting them into the locals. It is handled by resolveName in the checker


Ahh, I see. I opened your mail after writing this comment...


@emgarten @davidfowl, extract the extra compatibility checking to a utility method


Sweet! Thanks, I pinged them too :-D


I have no idea why it shows up as monke_000. That is my computer name.


remove "more or less"


Mh, and at http://lov.okfn.org/dataset/lov/vocabs/omn they only like the ones with http ;)
Btw: could you add a link (e.g. rdfs:seeAlso ) to http://lov.okfn.org/dataset/lov/vocabs/omn and to http://w3id.org/omn (e.g. via owl:sameAs)?


Well `service_name` refers to the meaning of the content of the container, not the container. We're saying "this container is an instance of `foo_service`", for example.  There is a random hash appended to the end to let you run more than one instance of `foo_service` on the same host.

Internally we have services that run in multiple containers on the same host, with different ports to let that work. The problem we'd like to solve is making that first class without requiring two different deployments. I think that's also what you're working toward, right?


If there are more than 1,000 pages, we want to run a checkpoint. And we want to keep checkpointing at every opportunity until there are fewer than 1,000 pages.


Yeah, after looking at all the code, I feel like it would be easier for both user and us if we just take new max as 'size_t max'.


Добавил список компа, два однострочных редактора с названиями списков. Изменил позиционирование окон. Начал делать движок игры. Сделал защиту от дурака.


One improvement is that the first one:
```ruby
each.call eligible_driver_names, lambda{|name| $stdout.puts name}
```
can have the `lambda{|name| $stdout.puts name}` switched at runtime.

While in the second one
```ruby
for name in eligible_driver_names
  $stdout.puts(name)
end
```
the code to do with each `name` is "hardcoded" at "compile time" (I know, ruby doesn't compile. I mean that you cannot change that once you decide to run the code).

Part of the confusion comes with the idea clarified in line 103: That the `for x in xs` is really syntactic sugar using the xs.each method call.


Yep. :)


I haven't much experience with the C++11, 14 handling in CMake but I know there is a [CXX_STANDARD](http://www.cmake.org/cmake/help/git-master/prop_tgt/CXX_STANDARD.html?highlight=cxx#prop_tgt:CXX_STANDARD) property. I think we should try to use as much of the standard CMake machinery as we can.


Bug!
U forgot to commit this folder.


In theory it could do some internal processing, even if none of the defined functions are invoked.


Please! Change the message from the alert. ;)


If there is real uint16, it will get broken. Can you add an assert that there is no uint16 in the composite? You could raise an error in that case and keep the hack.

Another option (but probably longer) is to change the inner composite to add a cast op on inputs and outputs to have computation in float32, but inputs/outputs in f16.


Just a nit, but I might add a ternary for checking development and set err based of that.  We can just cut some duped code.


The old doc here don't seem good. Could you at least add that float16 support is experimental?


Posts have more than one vote, so it's an array not a single value.


Right, but we already have people deploying a service with the same name multiple times on a host (with different ports). We need to support that. So this test is verifying that there aren't allowed to be two with the same name which is something we explicitly need to allow, right?


It's a hack. Can't be the same port.


I don't believe that is the case. It would be super weird.


Yes, it's online already, please update and let me know if it works as expected :+1:


Can it be pulled to 3.3.5 too?


It might make sense to move this to a platform specific include file. Similar to how it is done in [Apache Thrift](https://github.com/apache/thrift/blob/master/build/cmake/DefinePlatformSpecifc.cmake).


I thought the most significant bit is used by the runtime. We might need a second enum.


This causes test failures in the System tests:
https://wrench.mono-project.com/Wrench/ViewWorkTable.aspx?lane_id=10&host_id=8&command_id=42


Yes! that sounds better :+1:


It has been merged a while agi, so yes ;-)


I was just looking at this, and you throw and catch..but then I figured you wanted to print the full stock trace..


I don't remember any tutorial, can you use domainname instead of IP ?


Do you know how to fix it anybody?


rax retry tests


Я же потом пишу в них. Зачем их делать `final`?


All the meta and scripts should be inside <head>. Also add please an indent for the sub-elements according to the depth to make the nesting obvious.


indent


should this be `r.expires_at.date()` or something, because `expires_at` is a datetime, and these things are grouped by date?


Btw, is it showing any of the changes I have made on you guy's computers? (blaze block, carrot block)


Почему так?


You should declare variable as a new inside the functions to avoid using one from the global scope.


not working


Thank you for submitting your changes. Most thinks look reasonable and should be included in Poco.
However some things are not clear to me (see comments above) maybe you can give some explanation.


Please wrap it with <body>


Confirmed it's not super weird: https://github.com/pathscale/libdrm_pscnv/blob/master/xf86drmMode.c#L745-L794


I cant find this directory. Did u commit it?


Always add an empty new line in the of the file.


CR done, waiting CR fix.


Yes, I see the changes. But it says monke_00 did them, not circuitlord. In
eclipse, above the commit and push button, what does it say for author and
commiter?

    ____       __
   / __ \___  / /____  _____
  / /_/ / _ \/ __/ _ \/ ___/
 / ____/  __/ /_/  __/ /
/_/    \___/\__/\___/_/

Not sent from my iPhone because I don't have one

On Mon, Apr 20, 2015 at 3:19 PM, CircuitLord <notifications@github.com>
wrote:

> Btw, is it showing any of the changes I have made on you guy's computers?
> (blaze block, carrot block)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Kalman98/KalStuff/commit/ff8ea4498b2d0ac96473bfbfa1d9fc5eb1659c70#commitcomment-10811398>
> .
>


I am not a lawyer but you cant change the license of a project without written consent of all contributors at that time. Especially while moving from a copyleft license such as the GPLv3 to a permissive license such as the MIT this is creating a derivative work from copyleft code which is infringing on the license.


yes, please


for my education, does returning a string here produce a log message for the celery task?


How does `... take advantage of multi-core or several clusters. We could also change the implementation ...` sound? Is it a good alternative?


I actually wanted to remove CacheMap in other commit. I didn't notice it before.


sounds good to me


Hmm... Suboptimal. Solution?


Hello,

Looks like handleHasContent() should 'addClass' instead of 'hasClass'.

Thanks!


Jup.


-Added more JPEG
-Shrouded based Chooper's true form


Agree with Uku.


Google doesn't get credit for I/O, one of the oldest concepts in computing :)


'''java
for (int i = 0; i < 10; i++)


@Flet I removed the output from the transform tests but I'm thinking about adding it back let me know what you think.


Thanks :two_hearts:


How come http://godoc.org/github.com/go-check/check is empty? :S


name = Timothy Dea


:P


I noticed the LogTrigger doesn't clean up after itself when bot.clearTriggers() is called, so the bot would crash with an EADDRINUSE error when the trigger starts the webserver up a second time.

I'd also prefer not having any callbacks in the trigger def so it can be serialized, and instead put that functionality in the bot itself, e.g.:

```
callback: ['reloadTriggers', './example-config-triggers']

Chatbot.prototype.reloadTriggers = function(requirePath) {
  var triggers = require(requirePath);
  this.clearTriggers();
  this.addTriggers(triggers);
}
```


As I understand that SX answer, "staged function" is really a wider term than this - you could also call e.g. currying a form of staged functions. The breadcrumb idea is good though, so we might as well leave it be.


Doesn't `gopkg.in/check.v1` work?


lodash is not a dependency of this project.  Please add it to the package.json and bump up the version string.


@Flet Trying to use the eslint rule examples provided on their site for consistency.


I was probably doing that to force the StackTrace to be reconstructed. But now I've just removed the behavior. NullTracer does what its name implies: nothing.


I'll check.


@drexin k. Guess I don't get the release paradigm. I was assuming releases are made off of master and was looking at when this was merged into master (seems like 7 days ago?). I am just learning how to contribute on github so forgive my ignorance.


Looks it worth moving into `fold()` completely.


good point. addressed in 5cb7105


@Flet Should `ObjectExpressionClosingBrace` be included in this?


OHHHHHHHH......
Epic fail me....


You are setting the initial condition to 2* the node id?  You could say that, as it does not seem to be documented elsewhere what DataStructIC actually does...


Вот и Все!


@kbenne it looks like there might be a mismatch between the environment variable used by EnergyPlus 'ENERGYPLUS_DIR' and the one used by OpenStudio 'ENERGYPLUSDIR', https://github.com/NREL/OpenStudio/blob/16c6dd1b52eb78d15e9efb91d947dd4a496af21e/openstudiocore/ruby/openstudio/energyplus/find_energyplus.rb#L92  Any thoughts?

@nllong FYI


la texture vérifiés le format du bitmap


@eliotjordan what error are you getting with this again?


si c'est fait avec gimp y'a un truc qu'il faut décocher quand on export en bitmap


I would not like to have two flags whose names differ only by the "is" prefix. I think the distinction you want to make is that this aggregate flag means the node introduces a block scope. The single bit above signifies that it introduces a block scope, **and is not a declaration**


:+1: for the reference to multiple dispatch.  The more I work with ~~staged~~ generated functions the more I think of them as a way to programmatically populate the method table.


This can be squashed into https://github.com/morcos/bitcoin/commit/ff34d782517f9cabed87772db96a7e7a76ac5db0 after review


Uhmm, I don't think so. I vendored it from github to that path. And goimports imported it for me.


I don't know RST very well, but I do know it's pretty picky at times.  Does it need this underline to be the same length as the header?


Nope, no reason. Thanks for pointing out. Fixed in version in 0.0.2.


I had a requirement where part of my transcluded treeitem content had a floated right item but I wanted my label row to have a hover effect. I needed the label parts in a block element that I could target for my hover effect instead of the entire li.


In the comment above, one of the lets should be a var (both in the code and the explanation). They both became lets when you bulk changed all vars to lets


Did you forget?


Thanks.  Is `__OpeneBSD__` correct ?


Yes.


I think the usage of `is the` here (and in other places) isn't needed for the sake of brevity, although not sure about that. What do you think?


`@generated function`


К сожалению, мы все делаем автоформатированием и оно иногда делает все убого, но ничего не поделать. Нажимай `Ctrl+A`, `Ctrl+Shift+F` перед коммитом.


vous pouvez vérifier là, j'ai push les bmp


What is the expected output if someone calls selectColor(array, PieceColor.Black)?
The code seems like it returns the white pieces (based on line 153 it looks like you <i>remove</i> the black pieces leaving only the white ones).


seems so.  A third wrinkle is the CLI.  If you are digging into this I would verify CLI behavior too.  Does it respond to an environment variable when responding to arguments like -w?  Not sure.


also [#2632]


Если переменная не будет изменяться, то ставь `final`. Если это объект, и его поля будут изменяться, но самому ему пристаиваться нового объекта не будет, `final` нужно ставить все равно.


y'a encore du violet mais bon lol


It may be an outdated tip, but yes, this sets the type of the 3 fields.

Setting checkpoint and result in the ctor is unnecessary, I'm realizing now.  Since they are scalar values, there's no risk of them being leaked between instances.


There, I fixed it.


i have found a few bugs that come up when your network isnt a 192.168.17.* .

i am working on it but thats all i know.

you must be connected to the internet for this code to run!

to find your IP, open a terminal window and type ifconfig. its in there somewhere.


I think is a false positive, i.e., the detected lint isn't an issue. The range of dc:contributor is a dc:Agent which is a class and "mailto:" is a valid URI scheme. Therefore both types are fine and we ignore the validator and the LOV policies and leave the contribution sector as it is (either with multiple dc:contributor properties or with one - which is semantically the same)


Shouldn't that be `[generic](./generics.md)`?

Also, it would sound better if it was:

> That `T` represents a generic type, meaning that we can have vectors of any type.

Or, even:

> That `T` means that we can have vectors of any type.


Noted, will change.  Thanks.


Order matters - you have to iterate in order but skip ones already set.  StringSet is unordered


makeClockWithCurrentTime will be broken, if we shrink the zones. If it is something you already done and if you are just copying it, it is fine. If it is something you are introducing newly, just go with nodeId 0 and with current time to make it work.


Сам объект мапы не меняется, меняется только его содержание. Поэтому `final`.


dont hate me :) but this doesn't need the call to list


No ignored warnings anymore, otherwise :+1: assuming config is remaining flat.


Если захочешь изменить имплементацию (LinkedList вместо ArrayList, например), нужно будет изменить одну строчку, а не все, где они этот лист используется.


Ну, это больше не авто-женерэйтед метод стаб.


Il faudrait une fonction pour réinitialiser la position des petits cubes quand on fait une nouvelle partie


Sure, I can do that.


Nope, here is the PR: https://github.com/bitcoin/bips/pull/149 please review :)


Consider removing the (newly introduced?) whitespace between the colon and bullet point. Also, the additional period after p.m. looks a little funny.
```
Date: Mon, 20 Apr 2015 20:34:34 -0000
Message-ID: <20150420203434.2076.28123@app>

--===============5447823000655561093==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit

Thank you for signing up to map on your own during TreesCount! 2015.

The
following block edges have been confirmed:

* 32276

The block edges will be reserved for you to map until May 4, 2015, 4:34 p.m..

You can view and change your other block edge reservations at
http://localhost:8000/blockedge/reserve/.

Please contact us at Treescount.Help@parks.nyc.gov for technical assistance or any questions related to the TreesCount! 2015 website.
```


Error on line 26 Code::make('helper/validation/error') should be Core::make('helper/validation/error')


I can accept it, because it will not be called often in most of cases (probably). But implementation is slow (string compare instead of charcodes, regexp-s and so on).

Since none of us has time to polish code infinitely, we can leave it as is for next interested person.


just noticed this could be even fancier. since its for fanc, its of course non blocking:

```python
for item in izip(iter_a, iter_b):
    filename = "p1_%d-a1_%d_p2_%d-a2_%d" % flatten(item)
```


If you look at the README it says to use gopkg.in :S


Hey the package Nick gave me didn't come with easyGui. Could someone send me the module please?,The link in the book for it doesn't work.


Learn the Github


...sorry, supposed to be working on a presentation, which means im not working on the presentation


How to get the effect id from Config.yml ?


@Crocus303 nice eye


Esta se quedó en español


The idea of rewriting the examples in JS seems to suit best to avoid this confusion...


Many thanks :+1:


asdfasdf


Typo - Can't


`proper version control`

`and changes` está de más, hay que verificar lo que se quiere decir aquí


How would dashes in telephone numbers work for this?


Hello. It's better to use QDesktopServices::openUrl("/usr/share/applications/harbour-meecast.desktop");


Try installing the one from the readme alongside portable python. The Hello
World Python comes with everything used in the book.

    ____       __
   / __ \___  / /____  _____
  / /_/ / _ \/ __/ _ \/ ___/
 / ____/  __/ /_/  __/ /
/_/    \___/\__/\___/_/

Not sent from my iPhone because I don't have one

On Mon, Apr 20, 2015 at 3:39 PM, ploquel <notifications@github.com> wrote:

> Hey the package Nick gave me didn't come with easyGui. Could someone send
> me the module please?,The link in the book for it doesn't work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/PieCrafter314/python-browser-bar/commit/632314b3abbec6e8f83c14a5c78e6b1f758d778a#commitcomment-10811780>
> .
>


ok


I'd leave current constructor as is, but dropped those fields from prototype.


Um.. i think you merged an old copy of your branch with your current branch. I you look around you'll see that there are old files floating around that we had previously moved to packages.


Ok.

> On Apr 20, 2015, at 3:41 PM, Peter Zerangue <notifications@github.com> wrote:
>
> Try installing the one from the readme alongside portable python. The Hello
> World Python comes with everything used in the book.
>
> ____ __
> / __ \___ / /____ _____
> / /_/ / _ \/ __/ _ \/ ___/
> / ____/ __/ /_/ __/ /
> /_/ \___/\__/\___/_/
>
>
> Not sent from my iPhone because I don't have one
>
>
> On Mon, Apr 20, 2015 at 3:39 PM, ploquel <notifications@github.com> wrote:
>
> > Hey the package Nick gave me didn't come with easyGui. Could someone send
> > me the module please?,The link in the book for it doesn't work.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/PieCrafter314/python-browser-bar/commit/632314b3abbec6e8f83c14a5c78e6b1f758d778a#commitcomment-10811780>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


С чего ты это взял?


Yeah, but it's impossible with vendoring.


Ahh excellent thank you, put in a PR please and I'll merge.


Verificar toda la semántica de este párrafo, esta raro


Потому что ты наполнил его собственными данными и никакого `TODO` тут уже нет.


* automake


Should be `Gets` to follow style guide, but maybe a rewrite would be better as `Gets ... into buffer ...` sounds odd IMO. Maybe `Saves ... into buffer ...`.


Error received while indexing against load balancer:
<title>413 Request Entity Too Large</title>

413 Request Entity Too Large

nginx/1.4.6 (Ubuntu)


This could be a poorly named variable problem. These are really supposed to be grouped by "datetime".


I'll fix that with a later commit that replaces 'npy_float16'


As a start, can you please:
1. submit new pictures without the debug info and the greek arrows.
2. are you adding the sample code you added to the programmers-guide_samples?


This should really go into isolatedclient/. isolate/ is really just about parsing a .isolate into a tree that can then be hashed and uploaded.


The formula does not really matter here. He's trying to test something "more important"...
If we wanted `2*node_id` as na IC we would not bother with this whole `DataStructIC` business anyway...


Can we simultaneously remove the one from:
https://github.com/google/closure-compiler/blob/master/externs/gecko_dom.js#L109
?


You'd have it to return an error (that you could ignore afterward) instead of panic.


I wrote a shorthand function for that in common/isolated/


Adding:
ability to view top User Agents and IP addresses on 404 URLS
ability to blacklist User Agents and IP addresses from the 404 URLS (this action purges the blacklisted item from logs as well)
ability to remove User Agents and IP addresses from the backlog


Why did this fail?


The commit message states:
* src/obconversion.cpp: Comment out line end buffer deletion, so cml conversion works but still have memory leak.

I know nothing beyond this...


test revert 1


Esta línea ya no es necesaria?


The cookie timeout went from 24 seconds to 2 years? Was it a bug before, or meant to be 24 hours before, or...?


Ouch... This change should be in another commit :confused:


That's correct.


Whatever was there before was incomplete, so maybe.


you have been verified = tu hayas sido verificado


Yup, I'll add a comment.


Por supuesto. Voy a cenar y ahora después le hecho un rato ;)


Toda esta sección ya no es cierta... ahora se tiene el comando `backup` checa como funciona y documentalo por fa


@cameronscott137 this line will allow you to access the most recently uploaded sponsor image. To upload a new sponsor image, visit /advertisements/new as an admin user.


Uh, didn't mean to do that last one....


Why not use a binary search?


Still need a `function` in there. :)


Am I right in reading this as meaning that the domain name is no longer used to ascertain which agency a site belongs to?

I think this change may fix #16. Will update that issue if so.


The only disadvantage is that you over-allocate permanently.


Feature/SIM-166 update opsim interface
added opsimblitz1_1133_sqlite.db for unit tests involving an opsim output


I added a jira ticket that we should eventually migrate this to a git repo that doesn't track history.  Obviously, we have plenty of other data sets that need the same thing.


Does it matter that there's no SConstruct file?  I'm ok with it as long as the auto builder doesn't care.


@tyleraland - I think we have to put this back in, blast coverage (qcovs), it appears Blast is calculating it's coverage based on alignment gaps.  See https://bitbucket.org/uwlabmed/markergene_pipeline/issue/42/calculate-coverage-explicitly


Sure. I found some of their examples are not great though.


your call. It could be said that that is requisite celery knowledge.


check it out, this is what saves settings that were changed on the options page


For reference, it looks like #15 and #13 were not addressed in this update.


Hah, it's getting time to sleep over here, I think :)


@afiune Thanks for this! You're always one step ahead of me :-)


Quand j'ai fait le diagramme de classe j'ai pensé que les fonctions gagner et perdre serviraient à enregistrer le résultat de la partie dans la base de données, mais on peut changer si ça ne vous conviens pas.


the issue, #1052 does not mention datetime, is that spec somewhere else?
```

The following block edge reservations are expiring on [DATE]:
```


previously listed by month


we don't need to do this anymore since .filter gets made by default with subscriptions


why?


It's very difficult to understand logic you do with `trailing` var. Could you add comments, what should happen with tail?


:+1: Too much coding for me in these days :wink:


That's interesting, I didn't know that either. Is that a reasonable protocol? Seems like it could be a point of confusion for years to come.

I wonder if you'd be better served just having "errors", and always returning it in the form of an array (even if it's only a single error).


Please add flags to parse at line 90 instead of new functions.

[duh at github not permitting commenting outside the new code]


The problem was that the if/else block [here](https://github.com/pulibrary/pul-worthwhile/commit/e032204f82bf1a4165cf70297da0a93517cc46ff#diff-a7d33cba60e009c0f15d14f571c81b53R14) was in the method `after_update_reponse`, and for some reason Ruby could find the method. We tried renaming to confirm that this was the controller instance that was firing (and it was), but the only solution that ultimately worked was to paste the content of that method directly in to the calling `update` method.

Could it have something to do with the visibility of the override? Grasping at straws here, but in the `Worthwhile::CurationConcernController` module the [`after_update_response`](https://github.com/projecthydra-labs/worthwhile/blob/master/app/controllers/concerns/worthwhile/curation_concern_controller.rb#L108-L115) method is protected. I don't really see how that could be an issue...just throwing it out there.


I see `.replace('www.','')` in a few places, regarding the cookie domain. It doesn't look like it guarantees that `www.` begins at the start of the string, so a site whose subdomain had a level which ended in `www`, e.g. `site-www.example.gov` would be broken and the cookie would not stick.


Good idea, please send a CL for that with git-cl.


(Scott Daniel) Feature/SIM-166 update opsim interface
added opsimblitz1_1133_sqlite.db for unit tests involving an opsim output


(Peter Yoachim) I added a jira ticket that we should eventually migrate this to a git repo that doesn't track history.  Obviously, we have plenty of other data sets that need the same thing.


(Peter Yoachim) Does it matter that there's no SConstruct file?  I'm ok with it as long as the auto builder doesn't care.


Error messages always start by convention with a lower case letter.
Please send a CL with git-cl for this.


$this->effect = $c["EffectID"];


I don't think this really needs to be post. It can be done during binding the parameter


rax retry tests


this is cheesy


A parameter doesn't have children


I don't think this needs to be post. I think it can be done during. It doesn't do anything with the children


why are you specifying base 65 for the normal annual hdd and cdd? base 65 is only used for the simplified calibration path.


Same... If possible we could fix those through eslint.


I imagine the reason this needs to be done post bind is that it makes use of getModuleInstanceState, and the moduleInstanceState is set as a result of binding the children of the module.

I bet it can be done during bind of the children, instead of as a post pass. Perhaps in declareModuleMember.


Yeah I went back and fourth a bit on it too... I like it for completeness, but if we are running through esformatter anyways it could be considered redundant.


good work Amy!


I do not like the whole post thing, but I see why it's necessary here. Not sure if there is a way to make this clearer. To explain what I mean, when I see a function name of the form "preX" or "postX", I think to myself,

"Well I already know that this function happens before/after X. I can tell that from seeing where it's called. I wish the name actually told me what the function did."

I am skeptical that any of these actually need to be post passes.


Just bindSourceFile is fine


I can meet right now. Are you in the 3rd floor lab?


What does it mean to return SymbolFlags.None?


The API is very big, a small team working on many modules, some of the standard errors have slipped through the gap by the looks of it.

It's on the list for version 1 to tie up these loose ends. :smile:


Note in future @kltm recommends that we push the whole engine.config object into js via pup_tent, rather than variaby by variable


This will be workshoped once i get my pc working again.


Please don't just assume that I made the license change without the consent of contributors. I contacted everyone in the commit log prior to the license change.


indeed


(Scott Daniel) Feature/SIM-943 move utility functions
This is the base pull request.  It creates a minimal package (should not depend on anything else LSST) in which we can store utility functions

right now, it contains coordinate transformations, spherical geometry functions, and the Site class (which is where CatSim stores information about the telescope site)

* added boiler-plate code
* added observationMetadataUtils.py
* renamed to coordinateTransformations.py
* added Site.py to this package


Gracias por tu aporte, agregaré estas funciones a Xubucleaner.


Лучше явно проинициализировать нулем.


Можно просто ```new XYSeries(node.getName())```, это ведь и так строка.


No specific reason for the switch except it looks cooler!

If `async.io` wasn't taken, I would have bought it to make `neovim.async.io` another alias :smirk:


Not sure why the `indent.before.CatchKeyword` doesn't work... I've submitted a bug in [esformatter/#312](https://github.com/millermedeiros/esformatter/issues/312)


What was the problem with floatformat:0? I've added it due to this issue: https://github.com/diefenbach/django-lfs/issues/168

seems it is now broken again?


Totally understandable... I can only imagine the scope of the API "behind the scenes". :)

You guys are doing a great job though, keep up the great work!


Eh?  You're assigning a const char * value to a char.


Deleted readme.md as I am uploading another version


We'll revisit it if we feel the need to add it again.


Actually - we talked about that :) The change is ok.


I'm wondering this too... Maybe it was unintentional? It even broke the footer in the polymer docs.


Наверное, должен быть `private`.


Oh, I see you've removed this later


I'm going to punt on refactoring `VersionEncoding` as part of this branch.  We can revisit it in a separate maintenance task.


Of course, good catch. Fixed.


Generally, "public" methods help mark what the object's API is to the other objects in your project.  Occasionally unit tests become a lot cleaner if they have access to non-public methods, like in this case, and for that I like to use "protected" rather than "public" since that will allow unit tests to access them (since unit tests by convention have the same package as the class they are testing) and it still keeps the method out of the view as part of the main API.


replicating


Nice


Sure.

In a nutshell, the `trailing` var is a regexp match of any trailing `\n` characters.

There are three cases we care about:

1. One trailing `\n` on the string.  Just use `|` or `>`.  This is the assumed default.
2. No trailing `\n` on the string.  Use `|-` or `>-` to "chomp" the end.
3. More than one trailing `\n` on the string.  Use `|+` or `>+`.  In the case of `>+`, these line breaks are *not* doubled (like the line breaks within the string), so it's important to only end with the exact same number as we started.

In order to handle all these cases, I stop parsing once I get to the trailing line feed part, and just handle that separately.


Nvm, I see it removed later


[Building community google doc](https://docs.google.com/document/d/1LuAWlQznxCJ7rxVdwssFTHhxjMHDADO13dlWNNYo0Tw/edit#heading=h.1iuhf1ab9ang)

[MCMC google doc](https://docs.google.com/document/d/1C35PzVxsip3S4oMy9Ht1ppjlDfScUdz_nBi9_gjRViY/edit)

[observatory planning and scheduling link google doc](https://docs.google.com/document/d/1Vv1AVOHw0XkpsAICb70b7YSkTKur3Rf5V4_QiEZRn4w/edit)


I believe it should be only:
article = TestDataAndOperationsPublication.containsUser(author)


And this one is removed too


When testing loading or searching methods, we want more than one item in the data store so we know we are correctly filtering those items and only pulling back the specific ones we want.


Типа надо удалять?


It's not clear to me what downshi means, and the declaration comment ("hi-#( downsutary-list ) for incidence matrix") does not help to understand.  Could you update for clarity?


Ahhhh yisss!!!!


No - `lookup` is an array of 128 `const char*` - each element is `char*` (=null terminated string), and this assigns the string `\\0" (backslash, the digit zero, and an implicit NUL) to index 0 of the array `lookup`.


Mudar título.


(Scott Daniel) Feature/SIM-943 move utility functions
This is the base pull request.  It creates a minimal package (should not depend on anything else LSST) in which we can store utility functions

right now, it contains coordinate transformations, spherical geometry functions, and the Site class (which is where CatSim stores information about the telescope site)

* added boiler-plate code
* added observationMetadataUtils.py
* renamed to coordinateTransformations.py
* added Site.py to this package


@justinmk, lol, for sure. It just seemed like those domains got extremely popular a few years ago around the time of several Google IO conferences.

@tarruda, cool cool. Yeah, the site/domain name does look cooler. Nice fresh look and feel to it.  Lol, as to the other domain alias.

Great work both of you and the whole neovim team! It's so great to use neovim, and to follow the excitement and buzz for the whole project and community! Thanks once again!


@jtwiest the second argument to the logger functions in `$context` which is an optional associative array of any relevant data you want to log with the error. I think you're taking the exception(Class|Message|Code) properties names too literally from where you copied the usage from. This context object should maybe have two properties responseStatus and responseBody


ok looks good to me.


When a method returns a Collection or array of objects, I much prefer that method to return an empty collection rather than "null".  Always returning a collection makes life easier on the person calling this method since they don't have to worry about that null case and won't have to always add the "if (results != null) {" block.


My reading of section 3.2.2 of the [standard](http://www.bpi.org/Web%20Download/BPI%20Standards/BPI-2400-S-2012_Standard_Practice_for_Standardized_Qualification_of_Whole-House%20Energy%20Savings_9-28-12_sg.pdf) specifies base 65 for a calculation that presumably determines whether or not the billing data covers periods of a typical year that have a high enough variation and coverage of potential HDD/CDD values

In case this is your concern, the CVRMSE metric is calculated for a regression that does not restrict the HDD/CDD base to 65. I don't believe there are any other places where HDD/CDD are used in this particular section.


@blueyed works for it.
https://github.com/neovim/neovim/issues/2433


It just makes it alot easier for someone to change up the compiler settings imo.
Also it might be nice to have the different methods of compiling Poco Library to use the same settings by default.

Anyway if you don't go this route you will need to change method of adding /MD /MT flag to similar way Apache Thrift uses https://github.com/apache/thrift/blob/master/build/cmake/DefinePlatformSpecifc.cmake#L53

Like i said it is prob worth someone looking at the flag settings i put there.


Super boulot !


Thanks. The documentation page looks better now. It is not as much of a problem that the items are smaller on the people page.


Thanks.  I will fix it.


We want to test the case where no query was provided, too.


Fail :)
APP_KEY is supposed to be set in .env


Oh, I'm blind.  Please excuse me.


Not work


Added comment explaining this, and used explicit character code comparison in the other case.


Sure, I'll do it tomorrow
On Apr 20, 2015 5:12 PM, "Víctor Terrón" <notifications@github.com> wrote:

> @fred3m <https://github.com/fred3m>, would you like to finish this
> function?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vterron/astrometry-wrapper/commit/a91c3e5e7f8ebd5c22c8e10085faee12ae603ce7#commitcomment-10805268>
> .
>


> It is not as much of a problem that the items are smaller on the people page.

OK. I'll close the #31.


requirement (spelling)


(Peter Yoachim) I guess it would be good to fill in the doc strings for these first few functions.


(Peter Yoachim) Looks like it works--maybe add a ticket to add some unit tests.


(Scott Daniel) There is a separate issue to make all of these depend on PALPY, rather than our own home-brewed algorithms.

when that happens, unit tests and docstrings will be updated.


I kind of feel like it made sense here because it has to do with binding children. But I see why you moved it, and there's an argument for having all the symbol tables initialized together


Only use the unit-test-only methods when you have to.  In this case we can fully test the remove functionality with only the ID of the test MovieData object.


Hooray for improved COFC cleanliness.


Fixed. :-)


Habría alguna forma de evitar el uso de tantas constantes hard-coded? http://es.wikipedia.org/wiki/Hard_code


MarkdownDocs (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1633 of 1633 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)


I see it. I'll fix it tomorrow.


Casting the Integer to long technically will work, but it's an unnecessarily roundabout way of doing it.


I can't think of a reason *not* to include the date so I'll leave it for now.


cmake_find_energyplus_or_EnergyPlus (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


I was just going by what I saw elsewhere, so I'm happy to change it if there's a preferred style.


Might aswell do the same for "OS Detection".


We'll be able to clear out the release year on the Movie Edit page, so we'll want to test that case here, too.


:D


cmake_find_energyplus_or_EnergyPlus (kbenne) - x86_64-MacOS-10.9-clang: Build Failed

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


Yes. There are a few other ones in the same category as well.


Test to ensure the date is being saved correctly.


s/buildbox-artifact/buildkite-agent/ ?


s/missing/lacking/ IMO


s/luring/lurking/ too :)


Today I updated the URL included in line 17 of the current "gpac.rb" with the one from line 22 above in the patch file above, then "brew install gpac --HEAD" worked fine.


This check should probably also just be `if hash buildkite-agent 2>/dev/null` because we can't get it from there if we don't have that. We can certainly put in a sensible error if both the .service file and buildkite-agent are missing (and should probably run the check for the .service file after attempting or skipping the artifact download anyway).


Угу.


s/an useless/a useless/


We should probably add a hash table at some point ...


this method should use an ImmutableMap.Builder to construct the outer map. For the entries in this map (the inner maps), you can use ImmutableMap.copyOf(entry.getValue()) each time. Then you also don't need ImmutableMap.copyOf in setMapReduceCounters.


Ah, missed that in the changelog.  Putting the delete back in results in a segfault, but when, say, you are processing all of pubchem, even small memory leaks add up.. I'm looking into it..


I see you fixed this later


I think we should avoid mentioning particular companies.


"with the"


I wasn't able to get much further with error. I think we should follow Matt's advice here and come back to this, unless it causes other issues or is symptomatic of a broader issue.


  # => a Mail::Message object
   #


you need console and console_libs too.

You should also provide init scripts and a config.


I could have written the comment right here...


So when do you pass actual flags to createSymbol?


[call_command](https://docs.djangoproject.com/en/1.6/ref/django-admin/#running-management-commands-from-your-code) could do what you want and I believe it picks up the defaults properly.


I would use lowercase "gods" and "dungeons" here, the current emphasize makes it harder to understand the phrase IMO.


Replace GMT with UTC or Universal please.


dito


cmake_find_energyplus_or_EnergyPlus (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This was not my work. Thanks @Defman21 !!!


Ctrl+Alt+R is the keybinding for 'reload atom', so we may need to change that.


merged with server here :
- need write news
- need activate connected_chest
- try mesecons and mobs...
- with the latest update of homedecor_modpack, many signs are "strange moved", it's normal, we nee to explain it to ingame players
- etc...


Aí sim!


This won't work for, e.g, ``fqdn == 'alumni.m4x.org'``.

We need to adjust the SQL query to use ``email_virtual_domains.aliasing`` instead of ``email_virtual_domains.id``.

This means that:
* We select all source ``mbox``
* For each ``mbox``, we select each possible suffix (=alias)
* We then select the account for each such email address
* And choose the forlife mbox/domain
* Then we restrict everything per source mbox / source fqdn

```sql
SELECT  CONCAT(esa_forlife.email, '@', evd_forlife.name), a.full_name
                               FROM  email_source_account AS esa_source
                          LEFT JOIN  email_virtual_domains AS evd_source ON (evd_source.aliasing = esa_source.domain)
                          LEFT JOIN  accounts AS a ON (esa_source.uid = a.uid)
                          LEFT JOIN  email_source_account AS esa_forlife ON (a.uid = esa_forlife.uid AND esa_forlife.type = 'forlife')
                          LEFT JOIN  email_virtual_domains AS evd_forlife ON (evd_forlife.id = esa_forlife.domain)
                              WHERE  esa_source.email = '%s' AND evd_source.name = '%s' AND a.state = 'active'
                              LIMIT  1;


> Also your own code has undefined in one case and null in others, I presume because express treats them differently than handlebars used to.

The `undefined` is for the default String **value**, since there's no value by default, I set it to `undefined`. The two config props I default to `null` are for props that could have objects, so I used to no-object object, `null`.

> doesn't require us to support a vestigial undocumented "feature"

My worry it that is happen of using default arguments and removing type-check code will lead to other places where this issue will arise.


luring/lurking. ok
IMO, missing/lacking might be seen as more vulgar, even if it's quite a detail.


I would rather do:
```javascript
fs: (this.player_.options()['ytFullScreenControls'] || typeof this.player_.options()['ytFullScreenControls'] === 'undefined') ? 1 : 0,
```

otherwise it will work the opposite of what it says it does


True! The phonetic rule, I forgot. thanks.


k. :)


(See below)


Are you guys meeting? Can you please comment here what your API will look like.

—
Sent from Mailbox

On Mon, Apr 20, 2015 at 4:02 PM, Lane Kolbly <notifications@github.com>
wrote:

> I can meet right now. Are you in the 3rd floor lab?
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/rellermeyer/course_os/commit/39f6a31008dd664fd3fcbe9b5dbc62d998312e3f#commitcomment-10812250


ehsan added ok


sdf


Since this can only run within one region, maybe try to get the region via metadata service? I know there's not a field there for that but seems like parsing `placement/availability-zone` is pretty safe. Either chop off last char or regex to `^\w{2}-\w+-\d+`. I'm worried about people screwing this up and changing routes of a remote region.


@ndobson Not really.. But thanks! :smile:


Coolt! Detta visste jag inte om :)


GA's default cookie expire is 24 months


This doesn't work. msg prints repr() instead of printing text. Result:
[----------------------------------------------------registers----------------------------------------------------]
b'\x1b[;32mEAX\x1b[0m: 0x0 \n\x1b[;32mEBX\x1b[0m: \x1b[;34m0xb7fd8000\x1b[0m --> 0x1ef8 \n\x1b[;32mECX
[...]


- [ ] @shanem CR


If no user with the given id is found you need to send to status code 404 ("Not found") to the client otherwise your route will send {user: undefined} and ember will crash.

```javascript
var user = users[req.params.id]
if (!user) {
  return res.sendStatus(404)
}
res.send({user: user})
```


What's the point of this? Does it have some effect on permissions?


Granskad!
Bonus-katt!
![Uploading Funny-Cats-Dogs-19.jpg . . .]()


No need to use quotes for the key unless it contains characters that are not allowed like _, + etc.

```{user: users[req.params.id]}```


Shouldn't generated files generally *not* be added?


Before sending the user you need to store it in the users array, otherwise if you open a new tab and try to login with the newly created user it will fail.

```javascript
users[user.id] = user
```


Quite oddly, I rolled back the Lottery plugin to the official repository, but the jackpot still seems to be winning. I even verified by checking the jar size. :/


We aren't meeting today, but we'll absolutely let you know ASAP. Are there
any feature requests in terms of the API?

On Mon, Apr 20, 2015 at 4:34 PM, Mathew Kurian <notifications@github.com>
wrote:

> Are you guys meeting? Can you please comment here what your API will look
> like.
>
>
>
> —
> Sent from Mailbox
>
> On Mon, Apr 20, 2015 at 4:02 PM, Lane Kolbly <notifications@github.com>
> wrote:
>
> > I can meet right now. Are you in the 3rd floor lab?
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/rellermeyer/course_os/commit/39f6a31008dd664fd3fcbe9b5dbc62d998312e3f#commitcomment-10812250
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rellermeyer/course_os/commit/39f6a31008dd664fd3fcbe9b5dbc62d998312e3f#commitcomment-10812783>
> .
>


Vill bli granskad! @palka285
![xwtf-hilarious-funny-nicholas-cage-cats1 jpg pagespeed ic 5qe-rzjaqjo_qy7s10y3](https://cloud.githubusercontent.com/assets/442870/7241111/cd62a190-e7b6-11e4-8e24-12566ef3be16.png)


It loos like the added max-width:62% is causing some trouble with dynamic dropdowns.
https://formidablepro.com/help-desk/data-from-entries-field-not-big-enough-to-display-entry/


Before sending the user you need to to check if the password matches as well. If it doesn't then you need to respond with status code 404 to the client.


Fixes #4


From my reading of [the docs](https://docs.djangoproject.com/en/1.6/ref/request-response/#passing-iterators), this might still create a streaming response as you're still passing an iterator. I could be wrong though.

Either way, putting `'\n'.join()` around this list and removing the `\n` on the inside might be a good idea.


This case will eventually require more care. It's doubtful that anyone would want to have URLs looked up and open on the individual points of a patch glyph, e.g., they only want to open up a single URL if the patch is hit at all. But this does not quite fit with how URLs are referenced/looked up in the other cases.


it works in buildbot_travis:
I remember I had similar issues, but I dont remember how I actually solved it. You can see how it is configured here:
https://github.com/tardyp/buildbot_travis/blob/master/buildbot_travis/vcs/gerrit.py


Esto me parece que hay que hacerlo de forma diferente. Porque en una clase va a haber varios partial_def y si haces un class.define_method siempre con el mismo nombre (:concat por ejemplo) te va a dejar solo la última definición (por el tema que Ruby te pisa con la ultima definicion del metodo que le pongas).
Por lo que me parece que habria que almacenar en un mapa/hash (la clave del mapa general seria el nombre del metodo ) y el valor de este mapa serian un array u otro mapa con todos los PartialBlocks.

Ej:
```ruby
mapa_multi_methods = { :concat => { [String, Integer] => partial_block1_1,  [Object, Object] => partial_block1_2 },
                       :mostrar => { [String] => partial_block2_1,  [Integer] => partial_block2_2}
                     }
```

PD: el self aca seria el receptor del mensaje o el Multimethod?? me confunden mucho los self


I don't understand this comment:
# Gtk 3.12 doesn't contain Gtk.TextDirection.RTL

As I know, this is come long time ago, from added pango support into GTK+, since GTK+ 2 for my mind.


Can we keep it in a namespace that is different from Differencing? The code in Differencing is not dependent on Roslyn.


Can this be deleted?


Interesting, I can't find any docs on kwargs to `write`. What does it do?


What is this for?


Might be safer to check that the error type matches before incrementing.


Will `unicode` not work instead?


Hooray!


Brian Miles wrote:
> It's not clear to me what downshi means, and the declaration comment
> ("hi-#( downsutary-list ) for incidence matrix") does not help to
> understand. Could you update for clarity?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RHESSys/RHESSys/commit/b40a7209385c70301ed81bb8ddd2a79b452d18bc#commitcomment-10812408>.

How to phrase this:  To enumerate the distributaries for reach i,
look at { stream[k]: k = downsdex[j] for j = downslo[i]...downshi[i] }

Code that processes these distributaries will have the form

     for ( j = downslo[i] ; j < downshi[i] ; j++ )
	{
	k = downsdex[j] ;
	...

(the "downsutary-list" is an "editor-oops" ;-( -- it should be
"distributary-list".)

-- Carlie


[] is a list(), not an iterator.

Also, I like the trailing newline.


This route we'll get back to it once we have sessions because we can't really implement it without knowing who is the authenticated user.


I don't think your ember app is ever making a delete request for users. So we can delete this route.


We need to store the newly created post in the posts array. If I login again I want to see this post as well. The posts array will be your temporary in-memory database. So as long as you don't restart the server the data will be there.


https://docs.djangoproject.com/en/1.6/howto/custom-management-commands/#management-commands-output is the docs, https://github.com/django/django/blob/master/django/core/management/base.py#L105 is the code.


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1639 of 1639 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)


falta o metodo fillData com 3 parametros


I don't think your ember app is ever making a request to get a specific post by id. So, we can delete this route. If ember  made a request like this you'd have to respond with a json in the format ```{post: post}``` not ```{posts: post}```


hmmm


We'll get back to this route as well in the future.


Nickel !


Why is this not calling `[_projectile] call ace_frag_fnc_addManualTrack`? There is no need to spawn off a whole fired handler, the `addManualTrack` function is designed specifically for script spawned ammo objects.


I would add that we are building these libraries in CoreCLR repo for now because of corefx repo does not have a native code build system (yet).


The post must be deleted from the posts array and only then respond with status code 204.


ادرسهم قبل أن تقدم لهم .. من الممكن أنهم لا لا يستحقون.


LOL - it's fixed!  But there is apparently no limitation on size of image (I'm not suggesting there should be - I just found this funny when I added a link to an image :) )

![image](https://cloud.githubusercontent.com/assets/10480656/7241333/30e5158a-e786-11e4-982a-6326d7341aab.png)


Bash completion, add or I can add after committed.


Okay, switched in the most recent commit.  I'm new to these, so you may want to have a look and make sure I did it right.  No seg faults at least. :-)


 private bool _pendingResetRequest;
         #endif


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-MacOS-10.9-clang: Tests Failed

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


                 var args = new object[] { position, value, scrollPos, style };
-                var newValue = scrollableTextArea.Invoke(null, args) as string;
+                var newValue = _scrollableTextArea.Invoke(null, args) as string;
                 scrollPos = (Vector2)args[2];
                 return newValue;


من أنهم??? من الممكن ... لا !!!!!


Wouldn't this work without `sudo`, too?


Namespace


It passes through force_str in write(). I suspect I could use repr, or something of six. instead of str()


This line isn't quite accurate, per my response in #11.


rax retry tests


There's no need to use a custom view here.


@themightychris Ok, I'm thinking it would also be useful to include the the email data incase that's what's causing the error, ie a missing or incomplete email field etc. The only disadvantage would be the HtmlBody param in data might muck up the log file.

```
 \Emergence\Logger::general_error('PostmarkMailer Delivery Error', [
   'responseStatus' => $httpStatus,
   'responseMessage' => $result,
   'requestBody' => $data
]);
```


RAIRD?


MarkdownDocs (Myoldmopar) - 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)


Commit message "errors"


No need for comment


I think the threshold with or without _use_prob should be set once in `__init__` .


Else is redundant


Hi, I really love cakephp-upload and it has been working wonders for me. I just have 1 question regarding the original file re-naming method that was added some time ago. How exactly do I enable this in my model, specifically if i want the hash to involve the current time stamp? This is what my model currently looks like for cakephp-upload which supports two types of file uploads:

public $actsAs = array(
        'Upload.Upload' => array(
            'banner_image' => array(
                'fields' => array(
                    'dir' => 'banner_dir',
                )
            ),
             'background_image' => array(
                'fields' => array(
                    'dir' => 'background_dir',
                )
            )
        )
    );

Any tips in the right direction would be greatly appreciated, thanks!


@b4n I used your comparison function, just reordered the comparisons a bit:

1. line - for the symbol tree this one is probably the fastest (in)equality test (unless it's some minified javascript on single line)
2. file - will be the same for the symbol tree, I kept it second in case we use this function somewhere else
3. name - even though it's rather expensive strcmp() it makes sense to make the test early for things like 'int a, b, c, d;' because all other fields will be identical.


There's no need to use a custom view here. The font is the same as [ORKTapCountLabel defaultFont] provided the text size is normal. I verified it in the debugger: the size will be 56.0


It seems best practice (because clearer at first glance) to just `import numpy as np`, then use `np.where` .


good point, thanks!


for #834


Makes sense. I was just curious as to why that was better.

Making some progress - though I have a question about the foreign keys. should they be populated in clocktimes table, from jobs? Clock ins are being written to both jobs and clocktimes (only clock_in is written to clocktimes during project_start(), the rest are left null) but the job_id field is not being populated.

The code for writing to jobs and clocktimes follows.

    new_task_job = [Job(abbr=abbrev, name=project_name, rate=p_rate), Clocktime(time_in=datetime.datetime.now())]
        for i in new_task_job:
            session.add(i)
        session.commit()

Check the screenshot to see what I mean.
![screenshot from 2015-04-20 17 58 47](https://cloud.githubusercontent.com/assets/2933789/7241441/f17bb092-e786-11e4-9b13-d6013ab40944.png)


Got it! Thank you for the clarification.


Commented out example code is nice to have when you've just started a new project, but once you get a little deeper into it, it's safe to remove it since it just clutters your files and makes them harder to read. If you need it, you can find it later in the docs or in your github history.


I can.


someone else should make a nicer one!


How about using a hash table instead?


@b4n To your comment to the previous patch, I think update_parents_table has to be called always because parents_table is then also used in the second pass.


Now we cannot declare static size constants, because the progress indicator sizes depend on user-selected text size. It's for the best; using a magic number 104 doesn't seem like the best idea anyway.


I think it's more natural to compute a radius first, and the the diameter is simply double.


Some excellent film choices in here :P


Now we don't divide anything by 2.


Ah, I didn't know you were doing this. But I'd still like to avoid ports. I'll try to use the full container id here. And yes, multiple services on the same host is my goal.


How about hash table?


Let's try to reuse.


1. Hah, the use case for a constant surface was just prototyping on my part before attempting a larger "real" grid. I have removed that code block.

2. Good point, I think my newest commit will do the trick but I am not sure. I cannot check it against `vgmdl.dmat` because it is missing from `geoRcb`. Also, this causes `example("krige.conv")` to fail.

3. Another good point. I have dropped `accCost()` in favor of `costDistance()`.


You're right, this must be allowed.


I'm also wondering what this change could break.


Similar a https://github.com/nopr/sassdown/issues/77, por lo que se ve está relacionado con la configuración que se le da a `grunt-shell`. Voy a probar sólo a hacer algo el `grunt-shell` y te digo


I've confirmed this with Brian on IRC. The feature branch now includes a `fixup!` commit that factors these assertions into distinct files.


The only subtle difference is that the new label's size can change when user selects new dynamic text size. So we shouldn't use the magic constant 104. Frankly, perhaps this constant shouldn't have been here in the first place.


This is nitpicky, but since these validates are short, I would put them on one line since I think that's easier to read:
`validates :email, presence: true`

Also, since you're only validating `presence: true` for all your attributes, you could combine them:
`validates :email, :firstname, :lastname, presence: true`

The downside of that is if you wanted to add more validations to each individual field (that differ), you'll have to then break them up into multiple lines again. Ultimately it's a matter of preference, both ways are fine, I'm just pointing out an option :)


Examples in jsx now...


test_benchmark.rb passed.
```
Finished tests in 4.215000s, 3.0842 tests/s, 14.2349 assertions/s.

13 tests, 60 assertions, 0 failures, 0 errors, 0 skips
```


Sorry, my fault... Issue in your previous merge was on Gtk.IconLookupFlags.DIR_RTL...

Revert commits


i actually like the previous docstring better - this field doesn't summarize the whole application, only shows screening statuses, gps assigned and awards, right?


## This is a markdown file


You could combine all the `numericality` validations in one `validates` call, I think that would be a bit faster to read:
`validates: rating_out_of_ten, numericality: {only_integer: true, :greater_than_or_equal_to: 1, less_than_or_equal_to: 10}`


Just to close out this thread, I took a quick look at our lx src and we
have no support whatsoever for mounting CIFS/SMB in the emulation, so that
explains why it doesn't work. We'll continue to use OS-4205 to track the
work to add the missing support.

Sorry that this won't work yet,
Jerry

On Mon, Apr 20, 2015 at 1:40 PM, Jerry Jelinek <jerry.jelinek@joyent.com>
wrote:

> I filed https://smartos.org/bugview/OS-4205 to track the CIFS problem
> separately from the old NFS issue, although right now there is no info in
> that bug report. I'll put something together that will help collect some
> info and pass it on to you.
>
> Thanks again,
> Jerry
>
>
> On Mon, Apr 20, 2015 at 1:32 PM, Jorge Schrauwen <notifications@github.com
> > wrote:
>
>> For now it seems to choke on idmap and smb/client binaries not running.
>> But sure, I can run a DTrace script somewhere later this week.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/joyent/illumos-joyent/commit/7652b10523b05b4d89e5deb010e8b0ea590ceb7d#commitcomment-10810362>
>> .
>>
>
>


This is nitpicky, but since these validates are short, I would put them on one line since I think that's easier to read:
`validates :user, presence: true`


don't need the `if` check - if `projectapp` is empty, the `for` loop just won't be entered. also might be good to call it `projectapps` just to be clear that it's a list, not an individual object.


`unicode()` instead of `str()` - otherwise any non-ASCII characters will break this. (Not super likely to happen with GP titles, but it is possible, and also good habit to use throughout the code where it may be more likely)


also, you could combine that to one line unless you use the `gp` variable again.
```
summary += unicode(papp.giving_project)
```


You can do a pull request :D can't fix it because i'm not at home.


Don't need the `is not None`, can just do `if papp.get_screening_status_display():` and it will rule out `None`


That’s more like it.


Since github keeps track of all your history, it's safe to just remove old code instead of commenting it out. It keeps the file cleaner and easier to read. Further, by removing old code, it's less likely to introduce bugs. A new developer might not know why the code is commented out and might uncomment it, bringing back old and possibly broken functionality.

If you need some old code back, you could just browse a file's history in git to bring it back.


pretty sure this only returns strings so you should be ok to drop this line


Add 'EmitDefaultValue' attributes to be consistent with other properties (i.e. Payment -> PaymentType)


DELETED does not exist as a OP status


Changing Deleted to Voided is technically a breaking change, but the OverpaymentStatus would not have been working in its current form anyway.


it's due to the `/archive/` vs `/tarball/` link difference that I kept complaining about in the PR


Ok, probably better to be explicit since a summary of the whole application would be more comprehensive.


Will this be published to npm?


when a chip is selected, clicking another chip causes a slight flicker of the bottom border

One way to remove this flickering is to call the $mdChipsCtrl.selectChip($index) on ng-focus instead of ng-click

```
var MD_CHIPS_TEMPLATE = '\
      <md-chips-wrap\
          ng-if="!$mdChipsCtrl.readonly || $mdChipsCtrl.items.length > 0"\
          ng-keydown="$mdChipsCtrl.chipKeydown($event)"\
          ng-class="{ \'md-focused\': $mdChipsCtrl.hasFocus() }"\
          class="md-chips">\
        <md-chip ng-repeat="$chip in $mdChipsCtrl.items"\
            index="{{$index}}"\
            ng-class="{\'md-focused\': $mdChipsCtrl.selectedChip == $index}">\
          <div class="md-chip-content"\
              tabindex="-1"\
              aria-hidden="true"\
              ng-focus="!$mdChipsCtrl.readonly && $mdChipsCtrl.selectChip($index)"\
              md-chip-transclude="$mdChipsCtrl.chipContentsTemplate"></div>\
          <div class="md-chip-remove-container"\
              md-chip-transclude="$mdChipsCtrl.chipRemoveTemplate"></div>\
        </md-chip>\
        <div ng-if="!$mdChipsCtrl.readonly && $mdChipsCtrl.ngModelCtrl"\
            class="md-chip-input-container"\
            md-chip-transclude="$mdChipsCtrl.chipInputTemplate"></div>\
        </div>\
      </md-chips-wrap>';

```

I have changed **~~ng-click~~ng-focus="!$mdChipsCtrl.readonly && $mdChipsCtrl.selectChip($index)"\**


Need one of the npm owners to do so https://www.npmjs.com/package/canvas


MarkdownDocs (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1639 of 1639 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)


As more and more reviews are added to a movie, this calculation may become very expensive. Imagine you had 10,000 reviews - this calculation would have to read all 10,000 reviews from the DB each time the average is needed. As an advanced feature, it may be worth considering adding a new column to the movies table that stores the running `review_average`; then, the average would be calculated whenever a new review is added or an existing one is removed or modified (which you could do using callbacks). When displaying the average on a page, you would only have to look it up once in the movie row instead of calculating it each time using all the reviews.


I tried doing `if papp.get_screening_status_display():` and the inline would disappear when `screening_status = None`. How do you view the admin errors? I couldn't figure that out so I just used `./manage.py shell` to test out code.


Moving files to allow for better organization.  Previously everything was just dumped in a single folder haphazardly.


Why did you add this line?


Legal... nao parece dificil d automatizar. O maior trampo eh fazer o bixo rodar como um servico em vez d só executar o bin... mas isso já foi feito p outras coisas (ex uwsgi) na receita do radar, então acho q  damos conta :)


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


@domenic I've added you as an npm owner


@rhvgoyal

I was under the impression that providing an actual block device (not loop-lvm) to `dm.datadev` and `dm.metadadev` was recommended, but are you saying that this is not recommended?


Assuming you had a lookup hash between strings and Volume.RAID* values like shown above you could reduce the above code to:

```Python
supported_raid_types = []

for k, v in c.items():
    if v in mega_raid_types:
        supported_raid_types.append(k)

```


Hm, we can check that out tonight. I didn't find any way to see the errors being swallowed up by the admin site. I just put a bunch of `print` statements in and noticed where they cut off. In this case since you're calling the function twice, you could do this instead, and put in a print statement
```python
screening = papp.get_screening_status_display()
print(screening)
if screening:
  summary += '. ' + screening
```
Then you should be able to see what it returned, even if the line adding it to the summary is throwing an error.

Let me know what that does if you end up trying it out, if not we can do it tonight.


Materialsammlung:

 * Open-Transit-Diplomarbeit: <http://dbis.eprints.uni-ulm.de/1054/>

 * GTFS Spec: <https://developers.google.com/transit/gtfs/reference>

 * Kurze Uebersicht des Datenmodells:
<http://blog.openplans.org/2012/08/the-openplans-guide-to-gtfs-data/>

 * Beispielanwendung in Perl auf Basis von EFA:
<http://finalrewind.org/projects/Travel-Routing-DE-VRR/>

 * Transitdaten, GTFS und Anwendungen, mit Links zu OpenTripPlanner und Co:
<https://prezi.com/p3s0p6tkg0uf/the-many-uses-of-gtfs-data-apta-transitech-march-2013/>

 * Mapnificent gibt es jetzt endlich auch als Freie/Open-Source-Variante: <https://stefanw.github.io/mapnificent/>


`return deferResult((token: …))`


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2089 of 2201 tests passed)

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


\o/ thanks!


@vbatts

That option is better than loop devices but even providing actual devices using dm.datadev and dm.metadeve has issues. And biggest issue is pool management. docker can create pool using those devices but after that docker does not know how to do pool maintenance.

For example, if pool gets full, pool will go in read only mode and docker commands to create or delete containers will start failing. docker does not know how to grow pool. lvm commands do know.

it is advisable to run thin_check on pool before usage to make sure pool is in good state and docker does none of that and lvm tools do.

If pool gets into bad shape and needs-check flag is set, it might require repair and docker has no idea how to do that repair. lvm tools provide some commands to repair pool metadata.

So passing an lvm thin pool is recommended because then get lvm's capability to manage pool as a benefit. Passing in data and metadta devices will work fine as long as nothing goes wrong with pool.


88737258-EPW-Ruby-Access (jasondegraw) - 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)