I deal with this with frequent and aggressive refactoring; I recognise cycles of prototyping/rebuilding that most packages go through.  Recently with [RcppR6](https://github.com/richfitz/RcppR6/commit/6a4d5993437a8ff5cd960f226ede09f9998480b5) (1,085 additions and 1,436 deletions) - I'd reached the point where the idea was fine but everything just needed rewriting from the ground up with the lessons learnt.  I try to hold an *external* interface constant and be willing to entirely change the back end (note that the tests for that changeset are basically unchanged).  And I find that much easier to do with a codebase that is highly functionalised.

I'll have a think about interfaces.  Practically there aren't many arguments that we need to bother with:

```
~ » R CMD build --help
Usage: R CMD build [options] pkgdirs

Build R packages from package sources in the directories specified by
‘pkgdirs’

Options:
  -h, --help		print short help message and exit
  -v, --version		print version info and exit

  --force               force removal of INDEX file
  --keep-empty-dirs     do not remove empty dirs
  --no-build-vignettes  do not (re)build package vignettes
  --no-manual           do not build the PDF manual even if \Sexprs are present
  --resave-data=        re-save data files as compactly as possible:
                        "no", "best", "gzip" (default)
  --resave-data         same as --resave-data=best
  --no-resave-data      same as --resave-data=no
  --compact-vignettes=  try to compact PDF files under inst/doc:
                        "no" (default), "qpdf", "gs", "gs+qpdf", "both"
  --compact-vignettes   same as --compact-vignettes=qpdf
  --md5                 add MD5 sums
```

vignettes and manual are clearly the biggies here.  I can see a benefit of using args (they *are* the canonical way of building packages) and can see the benefit of lists (much more natural to provide via an interface).


Oh my god...  People are still using pie?  Lol.. Sure it makes sense if you
like abnormally high resource usage and bogging down the ui thread.  Do
what you must I suppose.
On May 26, 2015 7:57 PM, "Sira Cuervo" <notifications@github.com> wrote:

> Well, it makes sense if you want to use pie apps or something like that.
> Just for giving user more freedom and no limitations
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MrApocalypse/frameworks_base/commit/f7334333f3a8d1d978de02ef9c9144bc06af20f0#commitcomment-11375517>
> .
>


it overall just protects the hash of corruption, not the viewer from reading unloaded cubes (it’s no access problem because the slots are valid till the loader is killed)


Just pushed what I used. Pretty much what was there before..

I think what matters is the timing of the `createElement` call, which needs to be during the render call for the context to be passed. So passing a factory method to be called during render() works, but creating the element beforehand does not.


of course it's just data structure integrity, but the loader can invalidate the cube while viewer still slice it :-P


develop (kbenne) - Win64-Windows-7-VisualStudio-12: OK (1146 of 1146 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)


...hence the viewer-driven load design we had in mind some months ago


:D


@trevnorris `arguments` is not an array. There is no `arguments.unshift`.


Thanks for your thoughts.  Yeah, I find dealing with the refactoring pretty
natural in the software context, mostly because I expect to rewrite the
code but also because I usually have a clearer idea of what the external
API should look like from the start.  I do sometimes struggle with whether
I should be exposing all options of the underlying functions or not (for
instance I always get annoyed by packages that wrap something like optim
but hardwire all of its options...)

In research I hate refactoring though -- probably because unlike software I
rarely have real test coverage.  It would be easier to write tests if I
knew what the top level api should look like, otherwise refactoring just
breaks all my tests because I've changed the function APIs rather than any
real issue per se.  maybe I just need to bite the bullet more and write
more tests and get used to refactoring both the functions and the tests
though.

On Tue, May 26, 2015 at 5:01 PM Rich FitzJohn <notifications@github.com>
wrote:

> I deal with this with frequent and aggressive refactoring; I recognise
> cycles of prototyping/rebuilding that most packages go through. Recently
> with RcppR6
> <https://github.com/richfitz/RcppR6/commit/6a4d5993437a8ff5cd960f226ede09f9998480b5>
> (1,085 additions and 1,436 deletions) - I'd reached the point where the
> idea was fine but everything just needed rewriting from the ground up with
> the lessons learnt. I try to hold an *external* interface constant and be
> willing to entirely change the back end (note that the tests for that
> changeset are basically unchanged). And I find that much easier to do with
> a codebase that is highly functionalised.
>
> I'll have a think about interfaces. Practically there aren't many
> arguments that we need to bother with:
>
> ~ » R CMD build --help
> Usage: R CMD build [options] pkgdirs
>
> Build R packages from package sources in the directories specified by
> ‘pkgdirs’
>
> Options:
>   -h, --help        print short help message and exit
>   -v, --version     print version info and exit
>
>   --force               force removal of INDEX file
>   --keep-empty-dirs     do not remove empty dirs
>   --no-build-vignettes  do not (re)build package vignettes
>   --no-manual           do not build the PDF manual even if \Sexprs are present
>   --resave-data=        re-save data files as compactly as possible:
>                         "no", "best", "gzip" (default)
>   --resave-data         same as --resave-data=best
>   --no-resave-data      same as --resave-data=no
>   --compact-vignettes=  try to compact PDF files under inst/doc:
>                         "no" (default), "qpdf", "gs", "gs+qpdf", "both"
>   --compact-vignettes   same as --compact-vignettes=qpdf
>   --md5                 add MD5 sums
>
> vignettes and manual are clearly the biggies here. I can see a benefit of
> using args (they *are* the canonical way of building packages) and can
> see the benefit of lists (much more natural to provide via an interface).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cboettig/drat.builder/commit/021a4c68ea19a9be8adaae7dc7baebb81d0fd15d#commitcomment-11375585>
> .
>


Yeah, this is right. React keeps track of the outermost component on the `render` stack, it becomes owner in the `createElement` call. So there's indeed no difference between calling something from `render` or doing it right inside `render`.


Nevermind. I didn't realize that you were the on who had added the files originally a few commits earlier in d4b7eaaaa87bba5cad5c33be547f227e84d0613c and 235397669ad41ae328c78eeb932fd8d10ce48682.


This breaks for me unless I add another group in my inventory file matching this pattern. Not sure I understand the purpose of this change.

Can you clarify?


More to the point, currently the lock only protects the data-structure,
while ownership on the actual slot never leaves the loader, even though
in effect it is the viewer who actually owns it during slicing.


@wva he meant @vanatteveldt, I think :)


Pavel,

I Removed the Firmware common and do a full file crc in the basic_file_server so it is now independent of the "notion firmware"  - we are ignoring the CRC in the bootloader because of the embedded one. The file CRC is of no value. Furthermore the API is ill defined for what the crc of a directory would be. Also for a basic read only file server the type settings of FILE and READABLE make sense. Dong a stat() would get you the type, but if it is readable is answered by opening it. If we want to know the USR GROUP and OWNER rights that would be different.


ugh


Not sure if intentional, but the commit message is `Speling` :stuck_out_tongue_closed_eyes:


may want to update the docs and comments to clarify timetoken.


much cleaner.  good to merge.


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 20](https://teamcity.labkey.org/viewLog.html?buildId=252006&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 25 Build time: 00:33:34
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

```


I was lurking around and kind of shuddering at the thought of context behaving differently with a difference of one stack frame :wink: Glad to know that's not the case though :+1:


What is the reason for this?


intentinal


@pavel-kirienko  Code fails to compile without  these scope changes.


For system stability and better diagnostic, consider watching for children actors with `context.watch(replicator)`.
Even this is not a requirement, a long lasting interactions between actors may benefit from proactive cleanup in parent actor, in case a different supervision strategy is implemented later by the child.


Está marcado como lucazartu porquê foi feito pelo bash dele, professor.


Should the label be set to an unusual name to make it unlikely that it will share the same label as a real object when setting options by label? Having the group be set to empty presumably makes such problems already unlikely at the group level? I'm just imagining setting some option that makes sense for most objects but would mistakenly not be ignored by Empty.Maybe that's not possible but it seems like it could be...


Just `sender`. It has no side-effects.


It's getting some merge entries "<<<<<<< HEAD" and "========" in the docs.


Good solution!


When I run `npm run pocket`, I get the following error:
```
  ypeError: Object #<Object> has no method 'waitForElement'
      at Context.<anonymous> (tests/pocket_signup.js:45:8)
```


LGTM.


It took me a little bit longer to find out why it didn't work, but I was able to get it up and running locally. I pushed all changes, it should work for Jenkins too. See https://github.com/scala-ide/uber-build/commit/73d13b2c22831125a6f742bee89effc1001ba477 and https://github.com/scala-ide/scala-ide/pull/957


Thanks a lot, merged here 33b508110e95a31cbe3eee385488356f8a3a8c2d.


селектор `*` и глобальные ресеты не очень-то приветствуются в БЭМ, т.к. блоки должны быть достаточно универсальны для любого окружения и содержать сброс своих отступов сами в себе


достаточно просто `margin: auto`


логичнее сделать универсальный блок `input`, и миксовать его с `login__input`


Would you mind adding a line of docs for each parameter as well?


I think this should be `Deletes a category ...` or `Clicks the delete button ...`


Awesome. You have notes for the whole conference. Definitely a good reference.

:+1:


[#94122560] [#94127390] Starting to implement these stories


[#94122560] [#94127390] Implementing these stories


Do you mean that this will add all users to the course with the specified role?


@davidfowl , what if I handle "lock file not found" in this way?


saw approval from cmeiklejohn
at https://github.com/basho/riak_core/commit/727fa776334e262b79eb38f2cd9ddca494b91565


merging basho/riak_core/bugfix/sdc/remove-legacy-gossip-rebased = 727fa776 into borshop-integration-737-bugfix/sdc/remove-legacy-gossip-rebased


basho/riak_core/bugfix/sdc/remove-legacy-gossip-rebased = 727fa776 merged ok, testing candidate = 9251173d


Empty is not an Element so won't ever show up in the OptsMagic. It can only be used as part of a Layout and cannot be nested inside any of the UniformNdMapping types, so it bypasses any issues like this.


Is course id a string or an integer?


[Completed #94122560] [Completed #94127390] Implementation of these stories completed


fast-forwarding 2.0 to borshop-integration-1125-bugfix-2.0/jrd/drop-cover = 93cf82fb


Dammit, that's what I get for testing in production.  Fixed.


DIRENT_ISFILE is defined in nuttx. The size of the d_type is uint8 and it is a set of bits discriminated by the macros.
```
                    // TODO: This is not POSIX compliant
                    if (DIRENT_ISFILE(pfile->d_type))
```
In linux this looks like a set of enumerations.

How would you like this dealt with?


this would mean we can never create files with more than 2B rows. Seems like an implementation specific detail is driving a constraint on how data can be stored.


Isn't 1 minute interval too long? you may end up wasting up to 1 minute until launching attack. Being online all the time already makes you non-human.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_core/builds/1573


Ok, got it working now. Compile the binary using PIE


Add the following test:

```
diff --git a/test/model.rb b/test/model.rb
index f6a7d66..61a5ec1 100644
--- a/test/model.rb
+++ b/test/model.rb
@@ -104,6 +104,12 @@ test "set" do
   assert_equal 0, Meetup.redis.call("HEXISTS", m.key, :name)
 end

+test "128 bytes string" do
+  name = "a" * 128
+  m = Meetup.create(:name => name)
+  assert_equal name, m.name
+end
+
 test "assign attributes from the hash" do
   event = Event.new(:name => "Ruby Tuesday")
   assert event.name == "Ruby Tuesday"
```

And you'll get

```
  test: 128 bytes string
  line: m = Meetup.create(:name => name)
  file: /Users/seppo/Projects/free-software/ohm/test/model.rb +109
  line: new(atts).save
  file: /Users/seppo/Projects/free-software/ohm/lib/ohm.rb +1080
  line: raise response
  file: /Users/seppo/Projects/free-software/ohm/lib/ohm.rb +1313

RuntimeError: ERR Error running script (call to f_d84093efbb10df1c5995c6b0e7340520973b306d): @user_script:34: user_script:34: Bad data format in input.
```


On May 26 2015, Ismael Mejia wrote:
> This one I won't take since the new default that creates directories per
> section guarantees that this number will be way lower, and I still have
> not seen a single session with more than 99 videos.

I'm OK with that. Thank you for merging the rest of the changes.

--
Rogério Brito : rbrito@{ime.usp.br,gmail.com} : GPG key 4096R/BCFCAAAA
http://cynic.cc/blog/ : github.com/rbrito : profiles.google.com/rbrito
DebianQA: http://qa.debian.org/developer.php?login=rbrito%40ime.usp.br


![drop the faith](http://pbs.twimg.com/media/BUKXRIVIgAIWlMk.jpg)


lulz...it looks like all those g2's bricked by your crap kernel WAS caused by you...smh


Strangely, I think this also fixed the strange order modules where outputted sometimes (like 2 before 1 in simple networks).


thanks


I feel that the the bottom percentage label needs a bit more margin from the legend to be more aesthetically pleasant. Maybe try `legendDotRatius*3` (and modify all related metrics accordingly)?


Fixed in 1216caf2ef1f02597b8b0bac29e542123126c77d.


課題のアップお疲れ様です。

プログラムはまだ動かせていないけど、コーディングで思ったことを書きます。

まずはC#の変数の命名規則(ガイドラインなので、絶対ではないですが...)を覚えて下さい。といっても難しくはなくて、小文字から始まるパターンは以下の二つだけです。

* ローカル変数
* プライベートなフィールド

それ以外はすべて大文字から始まります。(名前空間も！)

http://komaken.me/blog/2014/02/08/クラス-ライブラリ開発のデザイン-ガイドラインま/

次にコーディングスタイルについてですが、インデントがタブとスペースが混在しているせいか、ずれてます。まずはどちらかに揃えて下さい。インデントがずれてるととても読みにくいです。
コードの見た目が汚いといつかきっと損します。

最後に課題5ですが、課題じゃなければ書かないとは思いますが、私の中では絶対に書いてはいけないコードです。
簡単な処理は簡潔に書きましょうね。

最後と言いつつもう一つ、コメントを(特に課題自体をコードの先頭に)書いてもらえると嬉しいです。
この調子で頑張っていきましょうね！

※私的に案外楽しいので、基本情報より優先度上げて欲しいなんて思ったり思わなかったり。


Sure.  Just a quick fix because I like green lights on travis =).


Lots of great work here!

Looks like about 7 tests are failing in Travis in:

gramps/gramps/gen/lib/test/merge_test.py

https://travis-ci.org/gramps-project/gramps/builds/64153108


fast-forwarding develop to borshop-integration-737-bugfix/sdc/remove-legacy-gossip-rebased = 9251173d


"Tag Padrão", no texto original **Tag** está no plural

Na versão original, a compatibilidade é específica: "retrocompatibilidade" (backwards compatibility)


Hey gosh


@khk624  in iOS we set the web app session listener (delegate) when webAppSession_connect is called . I think we can do the same thing here instead of having a separate Command


Esta frase se refere às Tags PHP, portanto o trecho "(...) será válido e (...)" deverá concordar com o plural "tags". Assim como "desabilitado"


<img src="forkit.gif" id="octocat" alt="" />


as a usecase, it is better to let user to insert the phone number , then read it out,
rather than hardcode in the case,


> ... could be replaced by calling at the very beginning (or anywhere):

That's quite interesting actually - if done that way the ```CMPTransaction``` object would be fully populated already and could be passed into functions like ```populateRPCTypeSimpleSend() ```as ```const```...

I'll take a look - thanks bud


課題5はなるほどですが、やっぱり微妙です...。100まで数えるだけなので、for文は1つにすべきと思います。


Hi, how I can use this plugin in my AngularJS project? I appreciate if you help me how to call the card swiper from the controller.
Thanks


Nah, just making sure you knew.


"Short Tags" apresentada no plural no texto original.


Yes, it's possible.


TeamCity xBIM :: Xbim Geometry :: Release [Build 72](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1748&buildTypeId=XBIM_XbimGeometry_Release) outcome was **SUCCESS**
Summary: Tests passed: 70 Build time: 03:50:29


There are some hardcoded value `16` used in SSESHA1body and SSESHA256body. Is it the same thing as `SHA_BUF_SIZ`?


TeamCity xBIM :: Xbim Geometry :: Release [Build 73](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1749&buildTypeId=XBIM_XbimGeometry_Release) is now running


#7 #8 #9 #10 #11 #12 #13 #14 #15 #16 #17 #18 #19 #20 #21 #22 #23


Should this be here?


comment tooltip placement is not bottom, that's why it's not fixed.


can write `return stickyHeader ?: [super hitTest..];`


"(...) que o código HTML depois..." creio que seria melhor descrito **o código** como **um código**


it's more a convenience, generally tooltips are bottom (in UX), just override when needed


Likewise; Should this be here?


"(...) estiver habilitado", a palavra "habilitado" deve concordar em gênero com "a diretiva"


If an untakable item is crafted, it is added to the game world instead of the player's inventory. As before, crafting a takable item adds the item to the player's inventory.


looks good


Whoops...I guess I simplified the instructions too much. Can you take a look at https://phabricator.paperg.com/D83 please?


Session stopped working, this.app.use(session(...)); is needed.


thanks for your hardworking，I have a bug that the Hardware decoding is not working well，when i start playing the MP4 video file，if i fast forward ，the player contine playing after a few seconds instead instantly，because i am a chinese，i do not know how to feedback the bug，please help me。


Note that these are useless to anyone. While they should not have been pushed the to repo (oops), they don't do anyone any good & have been deprecated by 7digital.


I install many roms based on cm12.1 such as RR、nameless，they have the same bug。


Test branch


rax retry tests


This is true. Currently, the client doesn't expose any options that you can specify but max_retries would be a good one to add. If you want to submit a PR I'll merge it.


hello world 1


hello world 1.1


Sweet new desktop bg! By free-write do you mean this picture is my prompt?


@dannyroberts did I do something here that I forgot? (I'm not sure why you tagged me)


テーブルの関係としては、AttendanceMonth　1:n　AttendanceDay
です。


新規登録の場合に、画面を開いた時点の当月の1~末日までのテキストボックスを表示させて、入力保存したいです。


Nope

On Tuesday, May 26, 2015, jxcache <notifications@github.com> wrote:

> Sweet new desktop bg! By free-write do you mean this picture is my prompt?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/botbotbotbot/spaceII/commit/ed2419c72320d022cc7516f023fe7210398be1c0#commitcomment-11376381>
> .
>

--
latest Vidcode:
*Vidcode in Intel
<http://iq.intel.com/inside-programming-tool-designed-get-girls-coding/>*
*Vidcode in Fast Company
<http://www.fastcompany.com/3036097/strong-female-lead/a-learn-to-code-startup-for-instagram-loving-teenage-girls>*

*Leandra Tejedor, *Designer and Front-end Developer

*vidcode.io <http://www.vidcode.io/> / @vidcode
<https://twitter.com/vidcode>* / *Insta*
<https://instagram.com/vidcode>* /** FB
<https://www.facebook.com/myvidcode>*


Gotta find it

On Tuesday, May 26, 2015, Leandra Tejedor <leandra@vidcode.io> wrote:

> Nope
>
> On Tuesday, May 26, 2015, jxcache <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
>> Sweet new desktop bg! By free-write do you mean this picture is my prompt?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/botbotbotbot/spaceII/commit/ed2419c72320d022cc7516f023fe7210398be1c0#commitcomment-11376381>
>> .
>>
>
>
> --
> latest Vidcode:
> *Vidcode in Intel
> <http://iq.intel.com/inside-programming-tool-designed-get-girls-coding/>*
> *Vidcode in Fast Company
> <http://www.fastcompany.com/3036097/strong-female-lead/a-learn-to-code-startup-for-instagram-loving-teenage-girls>*
>
>
> *Leandra Tejedor, *Designer and Front-end Developer
>
> *vidcode.io <http://www.vidcode.io/> / @vidcode
> <https://twitter.com/vidcode>* / *Insta* <https://instagram.com/vidcode>
> * /** FB <https://www.facebook.com/myvidcode>*
>
>

--
latest Vidcode:
*Vidcode in Intel
<http://iq.intel.com/inside-programming-tool-designed-get-girls-coding/>*
*Vidcode in Fast Company
<http://www.fastcompany.com/3036097/strong-female-lead/a-learn-to-code-startup-for-instagram-loving-teenage-girls>*

*Leandra Tejedor, *Designer and Front-end Developer

*vidcode.io <http://www.vidcode.io/> / @vidcode
<https://twitter.com/vidcode>* / *Insta*
<https://instagram.com/vidcode>* /** FB
<https://www.facebook.com/myvidcode>*


This broke including the .app file in archives. `filelib` does not ignore the whitespace. This commit is only on master and has not been released, correct? /cc @josevalim


tested this and it's working


Why is there a space after the bracket


Let's leave vbox alone. No idea what it supports...


These have been deprecated by 7digital.  Bad dev for pushing these to master.


Esse aqui "private Long idEvent;" você pode deixar.


親（AttendanceMonth）のレコード作成は、Kintai/Views/Kintai/Create.cshtmlの一番先頭に
@model Kintai.Models.AttendanceMonth
と記述して、@Html.xxxForつかって、model => model.xxxで保存できるのは理解できたんですが、

子(AttendanceDay)のレコード作成の場合に、うまいことmodel.AttendanceDaysみたいな感じで、値をつめて、
controllerのcreateアクションで

db.AttendanceMonths.Add(attendancemonth);
db.SaveChanges();

の、attendancemonthで、AttendanceDaysも一緒に保存したいです。


Should the label be set to an unusual name to make it unlikely that it will share the same label as a real object when setting options by label? Having the group be set to empty presumably makes such problems already unlikely at the group level? I'm just imagining setting some option that makes sense for most objects but would mistakenly not be ignored by Empty.Maybe that's not possible but it seems like it could be...


This looks good. Descriptive variable name too. Good work


@saraloupot  can you clean up this commit a little please. you don't need to commit the .log .aux files ? what is the .synctex.gz file ? why are there two pdf's in this directory ?


@saraloupot does N/A/m = tesla ?


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 21](https://teamcity.labkey.org/viewLog.html?buildId=252025&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **SUCCESS**
Summary: Tests passed: 26 Build time: 00:34:44


The arguments for WriteCommand is []string, but we pass [][]byte here, compile failed.


@saraloupot what were the magnetic field units before the bias correction/ normalization ?


Updated


Brandon I think you screwed up here.


init code


You have to be root to run it, as it needs access stop /var/lib/docker.
It's meant to be used by the kubelet / OpenShift node to run commands in
containers.

On Tue, May 26, 2015 at 1:39 PM, Scott Dodson <notifications@github.com>
wrote:

> Thus far I don't think we put any v3 components in sbin but it seems fine
> in either location. Is it only useful to privileged users / processes?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ncdc/dockerexec/commit/2dea3e377235780454b962a0c19a824dc980bb80#commitcomment-11369492>
> .
>


See #356. Documentation coming tomorrow. :)


@necolas What do you mean?


Cero drama!


ooh i found it!! you're sneaky and i like the way you write

On Tue, May 26, 2015 at 9:28 PM, botbotbotbot <notifications@github.com>
wrote:

> Gotta find it
>
> On Tuesday, May 26, 2015, Leandra Tejedor <leandra@vidcode.io> wrote:
>
> > Nope
> >
> > On Tuesday, May 26, 2015, jxcache <notifications@github.com
> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
> >
> >> Sweet new desktop bg! By free-write do you mean this picture is my
> prompt?
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/botbotbotbot/spaceII/commit/ed2419c72320d022cc7516f023fe7210398be1c0#commitcomment-11376381
> >
> >> .
> >>
> >
> >
> > --
> > latest Vidcode:
> > *Vidcode in Intel
> > <http://iq.intel.com/inside-programming-tool-designed-get-girls-coding/
> >*
> > *Vidcode in Fast Company
> > <
> http://www.fastcompany.com/3036097/strong-female-lead/a-learn-to-code-startup-for-instagram-loving-teenage-girls
> >*
> >
> >
> > *Leandra Tejedor, *Designer and Front-end Developer
> >
> > *vidcode.io <http://www.vidcode.io/> / @vidcode
> > <https://twitter.com/vidcode>* / *Insta* <https://instagram.com/vidcode>
> > * /** FB <https://www.facebook.com/myvidcode>*
> >
> >
>
> --
> latest Vidcode:
> *Vidcode in Intel
> <http://iq.intel.com/inside-programming-tool-designed-get-girls-coding/>*
> *Vidcode in Fast Company
> <
> http://www.fastcompany.com/3036097/strong-female-lead/a-learn-to-code-startup-for-instagram-loving-teenage-girls
> >*
>
>
> *Leandra Tejedor, *Designer and Front-end Developer
>
> *vidcode.io <http://www.vidcode.io/> / @vidcode
> <https://twitter.com/vidcode>* / *Insta*
> <https://instagram.com/vidcode>* /** FB
> <https://www.facebook.com/myvidcode>*
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/botbotbotbot/spaceII/commit/ed2419c72320d022cc7516f023fe7210398be1c0#commitcomment-11376405>
> .
>


"Comentário de diversas linhas" -> "Um comentário" ou "Comentários de diversas (...) terminam na primeira"

A frase original não foi mantida, porém o sentido continua semelhante.
É interessante, porém, deixar claro aqui a ideia de "não aninhar" comentários de mais de uma linha.


もし、↑ができない場合には、AttendanceDaysの外部キーは月(AttendanceMonthId)なので、それを一緒にhiddenでセットすれば、紐付きは作れるのですが、わからない部分が、
1画面で複数のテーブルの設定がわからないです。

@model Kintai.Models.AttendanceMonth
@model Kintai.Models.AttendanceDay

てきなことをしたいんですが、modelのクラスを宣言する場合には、@modelと宣言しないとerrorになってしまい、うまく動かせてないです。（ここは僕がなにか勘違いしてるかもしれません・・・）


init


Te las re rebuscaste! pero funciona (creo)


입력된 파일이 없을 경우, jjal 값은 None 이 됩니다.


input_f 가 문자열일 경우, 파일경로일 것이라고 가정하고 처리합니다.


any chance you can update this to work with centos 6 as well and make a pull request into hw-cookbooks?


This commit also introduce a pretty bad memory leak as the new implementation of FreeLabelDict was only freeing the  content of the node, forgetting the node itself. It was fixed after a valgrind inspection in 5855d13ea34.


input_f 에 name 속성이 있을 경우, file 인터페이스를 지원한다고 가정하고 처리합니다.


Please discard this Ill try something different


Po, tiro toda emocao do readme =(

:broken_heart:


Acá el *return* va después de mostrarle la carta *self.pedidos.mostrar_carta(jugadores_en_partida, carta)* para terminar la ronda.
Y en caso de que termine el for y ningún jugador tiene una carta, le mostras el *self.pedidos.mostrar_no_hay_cartas()*


PIL 에서 quality 옵션은 이미지가 jpeg 일 경우에만 유효합니다. 다른 포맷 (png, gif 등) 에서는 무시됩니다.


Exemplo de documentação de API


output_f 에 내용을 모두 기록(write) 하고 나서, 현재 파일의 offset 위치를 처음으로 위치시킵니다.

offset 을 처음으로 두지 않으면, offset 이 파일의 마지막에 위치해있으며, output_f.read() 시에 빈 데이터가 읽혀집니다.


Se alguém compartilhar o link, essa era a imagem padrão e ficava muito estranho uma vaga com esse emotion

:)

Enviado pelo meu Windows Phone
________________________________
From: Luiz Freneda<mailto:notifications@github.com>
Sent: 5/26/2015 22:42
To: minutoseguros/oportunidade<mailto:oportunidade@noreply.github.com>
Cc: Eduardo Santos<mailto:eduardoluizsantos@hotmail.com>
Subject: Re: [oportunidade] Update README.md (7a8f2df)

Po, tiro toda emocao do readme =(

:broken_heart:

---
Reply to this email directly or view it on GitHub:
https://github.com/minutoseguros/oportunidade/commit/7a8f2df8e25c05148c1d24e69c397a5a20c9ddaa#commitcomment-11376555


@taurose Can I ask you to look into re-implementing this now?

>* scroll position can be saved by history entry, not just by path (already implemented)

I believe it shouldn't be hard to reapply your changes from #843, but I'd prefer you to do it if possible, so I don't introduce bugs there :-). Should I just use `this.state.location.key` instead of `renderedFullPath`? Can `key` be `null`?


jajajaja


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1146 of 1146 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)


PIL (Python Image Library) 대신에 Pillow 를 씁니다.

PIL 는 2009년 이후로 관리되고 있지 않는 프로젝트입니다. Pillow 가 이를 관리하기 위해, 새로운 프로젝트를 시작했으며, PIL 의 모든 인터페이스를 그대로 지원하고 있습니다.


How about "numeric_histogram bucket count must be greater than one"


"So that the construction itself becomes an expression"
"De forma que a própria construção se torna uma expressão."


Hey man, I went ahead and made these changes but it still doesnt fix it. Do I need to do anything with the css?


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


Looks good! I just added the GUI for secrets, so that should work now.

I think that, because we didn't directly follow the scheme that the spec set out, we shouldn't give the player the room IDs and instead give them distance and direction -- use the cardinal directions and tell them that the Wumpus is 2 tiles SE, for example. But this should be fine for now (and probably will have to stay this way... I don't think we have time to work on making it better, sadly).

Can you see if you can change it so that you only get the hints when entering a new room, and the game ends when you are out of gold/arrows? It should be a petty simple set of additions, but I'm currently tied up working on other finalization tasks.


develop (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-passing-green.svg)


wtf is ur css!!! 😝


wtf is ur css?!?!?!?! 😝


Alright, will do.


Why?


"chamado de (...)" utilizar "o operador ternário", dando ênfase na ideia de ser o único operador ternário existente.


I didnt brick shit. wtf are you talking about?


Yep


Thanks for contributing some of your time -- I'm doing what I can to get this thing in a reasonable condition for Thursday, and the extra code flow is definitely helping.


# plz


nice images, Michelle!


this is bad code


haters gonna hate


I'd prefer to ticket that as follow-on work, due to the size of this work so far. Also, the mcollective setup is based off https://github.com/richardc/mcollective-aio-beaker, which @johnduarte may be adding to. I'd like to piggy-back on any work done there.


We never use our gold. Is that supposed to decrement during trivia?


Uhhhh... I think so (?) I think it's every time you are asked a question.


Just curious: What was the bug here?


Do you not think we need to pass in the router? We could alias it instead I suppose.


95077770-HVACTemplateLoadManage-4493 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1657 of 1657 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)


Final Game/Shared Code/Game Control/GameController.cs

I think that "turn off formatting on autosave" plan seems to have derailed somewhere :wink: It's fine since you're adding actual code here as well, but it's best to catch this stuff before it starts creating merge conflicts.


Can you open an issue for this specifically so that I can better follow what is going on?

-Sam, sent on his iPod. You should be proud of him; typing on that thing is hard.

> On May 26, 2015, at 7:13 PM, shy401 <notifications@github.com> wrote:
>
> That would require us to detect collisions. My idea for detecting in and out is based off of something called the jordan curve theorem, which actually seems pretty obvious once you look up what it states.
>
> —
> Reply to this email directly or view it on GitHub.
>


scanf not sscanf


I assume we could use a similar route more than once, so here is another idea:
```c++
bool GetTransaction(const uint256& txid, CMPTransaction& mptx)
{
    CTransaction tx;
    uint256 hashBlock = 0;
    if (!GetTransaction(txid, tx, hashBlock, true)) {
        return false;
    }
    int nHeight = 0;
    int64_t nTime = 0;
    CBlockIndex* pindex = GetBlockIndex(hashBlock);
    if (pindex != NULL) {
        nHeight = pindex->nHeight;
        nTime = pindex->nTime;
    }
    if (ParseTransaction(tx, nHeight, 1, mptx, nTime) != 0) {
        return false;
    }
    return mptx.interpret_Transaction();
}
```
... to wrap it up.

I'm still not sure, how to get away from using `CMPTransaction` for basically everything though.


No, I mean why is the Router being added as a parameter? The `Injector` is automatically injected when the `Router` is created in #2.


Select `File > Export > FAB image script`, third column from the left. It's a check box.


Try harder.


@jamiei thank **you** for taking the time and making this package :)


I think it's worth extracting the loop content into a method of its own and giving it a name, just to simplify the code.
e.g.
```
@contacts.each_with_index do |key, i|
    puts_contact_line key, i
end
```


RealUrl auto config enabled? Added RealUrl TS on your blogsystem plugin page? All PIDs in TS configured?


Don't check in commented out code--comments are terrible. Just delete the code. It's saved in your Git history if you need to get it back (we can go through how to do this on Friday if you like).


I couldn't make it work with a custom icon from ../Ensime/icons repository. I found one solution that required to put an icon folder in the user package of the sublime installation, but I prefer the solution to have the icons in the plugin project.


Add a call to `chomp`


Use hasDesc


Yes, I copy pasted from some places and my editor didn't clean it...sorry!


添加注释


We should add comment to explain each constant with comment (Following same practice as per xMatters existing code)


Won't this Line wrap after "#"?


If we will reuse it the If component should be moved to commons


Shouldn't this go in `gitfixtures.py`?  Or does every SCM really support `--date`?


styling: now all the classes can be declared in a single call to add()


You can use the toggler component


This should be outputting `@contacts`, not the hardcoded `ContactsDisplay::DUMMY_CONTACTS`.
Also remember that you don't need to use string interpolation if you're just printing out a variable.
`@output.puts "#{ContactsDisplay::DUMMY_CONTACTS}"`
is exactly the same as
`@output.puts ContactsDisplay::DUMMY_CONTACTS`

String interpolation is useful when you're outputting a mix of text of variables.


i'm wondering if Maven does not provide utility functions to do these basic things (like zipping).

But did this actually help on memory usage ?


Auth_URL and Event_URL constants map to localhost instance while Webservice URL maps to xMatters server.

Should constants of one file not be on same domain?


Why are you removing Whitespace Here and so many other places?


Rename this class (and the file too)


instead of calling getLegends event per each layer the legend component should listen to the layerStore and renders the legends when the state of the store changes


I initially thought I'd need one, because if there are multiple requests to expand different macros - in different or the same file, I'd like to know which response corresponds to which request. Finally I wont need it since I have the position in the handler for the response.


lygaj.ydzf.haapn


Can this hardcoded data be moved to a separate "Constants" file?


10.68.35.122/5000
10.68.35.122/2005


> What are the other DLS beamlines running?  The Model 2 PMAC driver?

Yes.

> A related question, what is the status of the  TPMAC support module?  Is the most recent version hosted at DLS, at GMCA at APS?

I think (and Nick agrees) that we are the official host at http://controls.diamond.ac.uk/downloads/support/tpmac/

> Could it be moved to epics-modules on github?

In our opinion, yes. Nick (@NicholasRees) is just conferring with the original development partners first. If he gives the go ahead then I will convert to git and upload it to epics-modules.
I need to get invited to the epics-modules organization. Can you do that?

> I have made changes to get it to run on Windows and sent them to GMCA, but I don't know if they have been put back into the distribution.

Sorry, I have no idea. I'm not really involved with motion much - and subversion merges only retain the author name who committed them... Can you take a look at the sources from our website - or wait until I put it on github?


OK stupid question - obviously `bzr` doesn't, based on your overridden method above.  I'm wondering if it would be better if the base method raises `NotImplementedError`, to ensure that each subclass correctly overrides it.  But I guess that would result in more code duplication.  Hmm.


Regarding API, I wonder if it wouldn't be better to change `Place.set_name()` to `Place.set_placename()` and `PlaceName.set_value()` to `PlaceName.set_name()`? That would help us find instances of the old API and update, and also help us to remember that a name is not a string.

Also, perhaps we need some validation on the `set_name()` to make sure it gets a `PlaceName`.

Also, what about some kind of shortcut for:

```python
# currently:
place.set_name("Jonestown")
```

```python
# now:
pn = PlaceName()
pn.set_name("Jonestown")
place.set_name(pn)
```
Maybe:
```python
place.set_name(name="Jonestown")
```


Because total_size will NOT be equal to prepare_size.
As described in comments and qdoc, prepare size reserves space on disk (on some backends), it has nothing with object size (or data_size in eblob terms) and thus total_size


28:1: I100 Imports statements are in the wrong order. from invenio.ext.cache should be before from werkzeug.datastructures


1: D210 No whitespaces allowed surrounding docstring text
27:1: I101 Imported names are in the wrong order. Should be CFG_SITE_ADMIN_EMAIL, CFG_SITE_NAME, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL
27:1: E402 module level import not at top of file
28:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.config
28:1: E402 module level import not at top of file
29:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.base.i18n
29:1: F401 'config' imported but unused
29:1: E402 module level import not at top of file
147:16: E126 continuation line over-indented for hanging indent
153:58: E203 whitespace before ','
213:15: E126 continuation line over-indented for hanging indent
216:1: E402 module level import not at top of file
250:17: E203 whitespace before ':'
251:22: E203 whitespace before ':'
252:20: E203 whitespace before ':'
253:18: E203 whitespace before ':'
254:24: E203 whitespace before ':'
255:22: E203 whitespace before ':'
256:19: E203 whitespace before ':'
257:20: E203 whitespace before ':'
258:23: E203 whitespace before ':'
259:18: E203 whitespace before ':'
260:17: E203 whitespace before ':'
261:19: E203 whitespace before ':'
262:20: E203 whitespace before ':'
263:21: E203 whitespace before ':'
264:19: E203 whitespace before ':'
265:19: E203 whitespace before ':'
266:20: E203 whitespace before ':'
267:17: E203 whitespace before ':'
268:24: E203 whitespace before ':'
269:23: E203 whitespace before ':'
270:21: E203 whitespace before ':'
271:37: E203 whitespace before ':'
285:4: E131 continuation line unaligned for hanging indent
314:1: W391 blank line at end of file


37:1: I201 Missing newline before sections or imports.
145: D102 Missing docstring in public method
154: D400 First line should end with a period (not 'n')
154: D205 1 blank line required between summary line and description (found 0)
243: D102 Missing docstring in public method
283: D400 First line should end with a period (not 'w')
283: D205 1 blank line required between summary line and description (found 0)
296: D400 First line should end with a period (not 'e')
296: D205 1 blank line required between summary line and description (found 0)
311: D400 First line should end with a period (not 'e')
311: D205 1 blank line required between summary line and description (found 0)
319: D400 First line should end with a period (not 'e')
319: D205 1 blank line required between summary line and description (found 0)
330: D400 First line should end with a period (not 'd')
330: D205 1 blank line required between summary line and description (found 0)
352: D102 Missing docstring in public method
356: D102 Missing docstring in public method
368: D400 First line should end with a period (not 'd')
391: D400 First line should end with a period (not 'n')
391: D200 One-line docstring should fit on one line with quotes (found 3)
398: D400 First line should end with a period (not 't')
398: D200 One-line docstring should fit on one line with quotes (found 3)
405: D400 First line should end with a period (not 'e')
405: D200 One-line docstring should fit on one line with quotes (found 3)
412: D102 Missing docstring in public method
434: D102 Missing docstring in public method
442: D102 Missing docstring in public method
446: D210 No whitespaces allowed surrounding docstring text
446: D400 First line should end with a period (not ')')
451: D210 No whitespaces allowed surrounding docstring text
451: D400 First line should end with a period (not 'd')
458: D210 No whitespaces allowed surrounding docstring text
458: D400 First line should end with a period (not 't')
474:17: W503 line break before binary operator
488: D102 Missing docstring in public method
502: D102 Missing docstring in public method
523: D210 No whitespaces allowed surrounding docstring text
523: D400 First line should end with a period (not 's')
529: D102 Missing docstring in public method
536: D210 No whitespaces allowed surrounding docstring text
536: D400 First line should end with a period (not 's')
541: D210 No whitespaces allowed surrounding docstring text
541: D400 First line should end with a period (not 's')
546: D210 No whitespaces allowed surrounding docstring text
546: D400 First line should end with a period (not 's')
551: D210 No whitespaces allowed surrounding docstring text
551: D400 First line should end with a period (not 's')
556: D400 First line should end with a period (not 'r')
556: D205 1 blank line required between summary line and description (found 0)
569: D400 First line should end with a period (not 'e')
614: D102 Missing docstring in public method
619: D102 Missing docstring in public method
630: D102 Missing docstring in public method
637: D102 Missing docstring in public method
643: D102 Missing docstring in public method
649: D102 Missing docstring in public method
654: D102 Missing docstring in public method
661: D210 No whitespaces allowed surrounding docstring text
661: D400 First line should end with a period (not 'e')
665: D210 No whitespaces allowed surrounding docstring text
665: D400 First line should end with a period (not 'e')
669: D210 No whitespaces allowed surrounding docstring text
669: D400 First line should end with a period (not 'e')
673: D210 No whitespaces allowed surrounding docstring text
673: D400 First line should end with a period (not 'e')
678: D203 1 blank line required before class docstring (found 0)
678: D204 1 blank line required after class docstring (found 0)
678: D400 First line should end with a period (not 'r')
678: D205 1 blank line required between summary line and description (found 0)
684: D102 Missing docstring in public method
689: D200 One-line docstring should fit on one line with quotes (found 3)
705: D102 Missing docstring in public method
710: D210 No whitespaces allowed surrounding docstring text
710: D400 First line should end with a period (not 'n')
718: D210 No whitespaces allowed surrounding docstring text
718: D400 First line should end with a period (not 'n')
722: D210 No whitespaces allowed surrounding docstring text
722: D400 First line should end with a period (not 'n')
728: D200 One-line docstring should fit on one line with quotes (found 3)
734: D400 First line should end with a period (not 's')
734: D200 One-line docstring should fit on one line with quotes (found 3)
747: D203 1 blank line required before class docstring (found 0)
747: D204 1 blank line required after class docstring (found 0)
747: D400 First line should end with a period (not 'm')
747: D200 One-line docstring should fit on one line with quotes (found 3)
751: D102 Missing docstring in public method
762: D102 Missing docstring in public method
770: D102 Missing docstring in public method
781: D102 Missing docstring in public method
784: D102 Missing docstring in public method
787: D102 Missing docstring in public method
794: D200 One-line docstring should fit on one line with quotes (found 3)
800: D200 One-line docstring should fit on one line with quotes (found 3)
810: D200 One-line docstring should fit on one line with quotes (found 3)
873:13: W503 line break before binary operator
943: D400 First line should end with a period (not 's')
951:9: E731 do not assign a lambda expression, use a def
966: D203 1 blank line required before class docstring (found 0)
966: D204 1 blank line required after class docstring (found 0)
966: D400 First line should end with a period (not 't')
973: D102 Missing docstring in public method
995: D102 Missing docstring in public method
999: D102 Missing docstring in public method
1003: D102 Missing docstring in public method
1008: D102 Missing docstring in public method
1012: D102 Missing docstring in public method
1016: D102 Missing docstring in public method
1023: D400 First line should end with a period (not 'd')
1043: D400 First line should end with a period (not 't')
1043: D200 One-line docstring should fit on one line with quotes (found 3)
1066: D400 First line should end with a period (not 't')
1066: D200 One-line docstring should fit on one line with quotes (found 3)
1093: D200 One-line docstring should fit on one line with quotes (found 3)
1107: D400 First line should end with a period (not 't')
1107: D200 One-line docstring should fit on one line with quotes (found 3)
1124: D400 First line should end with a period (not 'n')
1124: D200 One-line docstring should fit on one line with quotes (found 3)
1147: D400 First line should end with a period (not 'w')
1175: D400 First line should end with a period (not ')')
1175: D200 One-line docstring should fit on one line with quotes (found 3)
1188: D200 One-line docstring should fit on one line with quotes (found 3)
1201: D400 First line should end with a period (not 'e')
1201: D205 1 blank line required between summary line and description (found 0)
1208: D400 First line should end with a period (not 'p')
1208: D200 One-line docstring should fit on one line with quotes (found 3)
1215: D400 First line should end with a period (not 'r')
1215: D200 One-line docstring should fit on one line with quotes (found 3)
1232: D400 First line should end with a period (not 't')
1232: D200 One-line docstring should fit on one line with quotes (found 3)
1240: D400 First line should end with a period (not 'd')
1240: D200 One-line docstring should fit on one line with quotes (found 3)
1258: D200 One-line docstring should fit on one line with quotes (found 3)
1267: D400 First line should end with a period (not 'e')
1284: D400 First line should end with a period (not 'm')
1284: D205 1 blank line required between summary line and description (found 0)
1300: D200 One-line docstring should fit on one line with quotes (found 3)
1311: D102 Missing docstring in public method
1317: D102 Missing docstring in public method
1323: D102 Missing docstring in public method
1337: D102 Missing docstring in public method
1350: D200 One-line docstring should fit on one line with quotes (found 3)
1377: D102 Missing docstring in public method
1436: D102 Missing docstring in public method
1481: D102 Missing docstring in public method
1489: D102 Missing docstring in public method
1499: D102 Missing docstring in public method
1508: D102 Missing docstring in public method
1511: D102 Missing docstring in public method
1515: D102 Missing docstring in public method
1519: D102 Missing docstring in public method
1539: D102 Missing docstring in public method
1547: D102 Missing docstring in public method
1555: D102 Missing docstring in public method
1561: D102 Missing docstring in public method


We need to add this to include other fields. The method is getting quite long. Any ideas how you can break it up?


My understanding is that we are mapping events data from two APIs?

Can we create Enums in separate files (xMattersEvent.js and groundworkEvent.js)?

Can use enum after referring to that file? e.g. xMattersEvent.service

Code maintainability may increase by moving constants or hardcoded texts to separate files


My opinion is, _where we can_, we should use JS templates (on client and server) with common JSON representations - rather than HTML blobs. That said, it's more of a reckon than a super strong conviction on my part. There are some practical advantages to pjax-style updates.

@tijmenb i'm curious what your reason for the change was?


this is hardcoded response structure and should be removed from library


Maybe the necessary features can be added to ffmpeg? Using `ffmpeg-full` seems like a bad idea because it is so bloated. The regular ffmpeg should have optional features so you could say:
```
ffmpeg = if youtubeSupport then ffmpeg.override { enableBla = true; } else ffmpeg;
```


Use Iterator#hasNext


Typo - "bazaar"


Add the comment to explain the function's usage


Your current code has different logic for both first name and last name. That should be equivalent, though. We need to think of a better way of factoring this code.


Wat heb je nou gedaan?
![bi-logo](https://cloud.githubusercontent.com/assets/10028768/7833751/7ecf88fc-0469-11e5-9334-66378a4278f2.png)


Früher hat er 3 API calls für jede Nachricht die zum Slack gesendet wurde gemacht ;)


An Mson is mutable. So now you have child and arg, linking to the same object on the heap.


@andyjmaclean could you delete from the repo the placeholder template `app/ui-components/channels/show.mustache` if it is no longer needed.


You are creating two `ContactChooser` objects. Just creating one, save it in a variable, and then call its methods twice.


chyba teraz działa :)


Is this mapping correct? This seems confusing.

servicename --> hostname
statusinformation --> service


You should do it in the implementation, not in the test. If a product doesn't have a slug, you generate it out of the name.


Co jeśli wyjdzie liczba ujemna?


Agreed. In https://github.com/joshmoore/openmicroscopy/commit/7d6f1830645679ea8ca91ef025434c2491a13702 I also listed the IJ changes as well as "magnification"/"interpolation"-stuff.


See my update: https://github.com/hflynn/openmicroscopy/pull/3


All this over a single bit. .


For loop iterator, baby.


I just get back on my feet but not for too long unfortunately...


Ja, auf meinem System gibt es sonst eine exception


Confusing code. Severity value is pushed to "recipients"?


@tadatuta ну это типа easy reset/normalize.css


Is there any other benefit to the method?

The current approach is quite widespread and we wouldn't like to confuse the users with the new syntax.


Franco kernel?


We can move its inner functions to separate helper functions to make code cleaner.


To będzie oznaczało, że gra działa gorzej niż 1FPS.
Sleep się nie wykona.


5FPS jest ustawione, później się ustawi 24-30.


Yeah, that would be the best of course. But since it's now a bourne shell, and not a bourne again shell, we can't use arrays but we still need to support spaces in pool names. Since IFS=" " is the default, that clashes with the space of/in the pool name.

So I have no idea how to solve this properly.

I tested this version on CentOS 6.0 and it worked for me..


@tadatuta а как в яндекс делают выравнивание по центру, слышал что таких способов не один. и еще слышал существуют инструменты которые показывают переопределяемые правила в css, не подскажешь название?


Please fix typo here :)


Add proper linting to make code more readable.


Wäre evtl von vorteil wenn man gleich die user-Daten vom slackclient.server nimmt.
Aber das sollte ohnehin nicht oft vorkommen


Python ^^


Every time you use this class, you'll end up with code like this:
```
chooser = ContactChooser.new(...)
chooser.show_contacts_list
chooser.contact_chosen
```
That's 3 lines of repeated code, when it could just be two:
```
chooser = ContactChooser.new(...)
contact = chooser.show_list_and_choose_contact
```


(der Thread _ist_ ein SlackRTMThread, also gibt es auch die Methode)


It is :)


Added AI difficulties


Fixes #58.


@ichrispa @jpfr imho, this commit and the following 3 must go into master and-or another branch


reqStr is currently initialized to hardcoded text. It can be moved to separate helper file and stringbuilder be used to create the url here.

Goal: Hardcoded text should not be present in coded files


The reason I am reluctant to do that is twofold:

- for short indices this is really superfluous (and in line with dataframe repr -> also there only the dimensions are shown when truncated)
- we wanted to go for an eval-able repr, and this `length=` does not fit that


This is not a test about offset, this is a test about simultaneously setting prepare, plain_write and commit flags, which is described in comment section.
Various offsets are tested in test_prepare_commit() tests, please take a more precise look at our tests.

Design of prepare/commit calls is already described in qdoc. This pull request is a bug fix for the ONLY case of simultaneous setting of 3 flags (PREPARE,PLAIN_WRITE,COMMIT) which was previously broken since it committed zeroes. Zeroes at the end of the data are not supposed to be there by design.

I hope to see this level of thoroughness in other pull requests too including yours.
If there are no more related comments, I will merge this pull request.


sorry, but no cake.

before, we copied the string to the node. and then cleaned up the old string without freeing the pointer to the string's data.

now, you copy the string and clean the old string up without deleting the content. that's a memleak!


Now that the download and build happens each time, I fear we may be stressing your FTP site. Yesterday some downloads were arriving here at about 10 Kb/s.


Add method for move player 2 as AI


Why do you check fields when the issue is about classes? You should check Class.Namespace.Classes.


The message is incorrect:
- it's an argument, not a parameter
- the projection is not unsupported, it just makes no sense

I think we should simply report ILLEGAL_MODIFIER here


yes. let's put non-subscription stuff into master when it has matured enough.
this commit could also be cherry-picked to 0.1


This function is used only once


@as-cii How would you feel about limiting this to a property on `TextEditorElement`, like `::hasTiledRendering` just to limit its scope?


here we go =(


Most of your changes seem reasonable, but I want to check with Ulf and he's not in the office this week.

Let me come back to you next week when we had time to discuss things.

> On 27 May 2015, at 11:06, Alexei Sholik <notifications@github.com> wrote:
>
> Perhaps a different name would better convey the fact that the call one happen until later. call_to_ast or call_to_symbolic come to mind.
>
> But if you're comfortable with having this macro in eqc_ex, no problem. I can remove it from the PR and we'll define it in our own project.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/PSPDFKit-labs/eqc_ex/commit/4c20c2a21af2050f81918c05c5212613e054d509#commitcomment-11381153>.
>


Fixed GUI merge in 91ac3c6, but please check to make sure that I didn't miss something.


can you then remove it from hera and apply it no master and 0.1 @jpfr? and then merge master into again?


I realized I did not modify the author of this theme: Akkarinage
Version? no idea
description: I forgot to modify it from
Website: I think there is none

Sorry about that


@hflynn : Not sure about the formatting: why is the OMERO.web section intercalated in between of other (non-paragraphed) points ?


Ich habe es selber noch nicht gesehen. :/


What happens then if the name is not unique and generated slug can not be used?


You could probably make this a bit more concise by saying "if '--all' is specified"


Shouldn't this be a tab, or is there something I'm missing?


`isNil` takes `string` or `code`.
Reference: https://community.bistudio.com/wiki/isNil


don't we have something like that somewhere else in the code? LineParser? StringConverter? I recall it was somewhere there...


@shahata , @alonn24 , why was this added? It makes long tests suits crawl to halt by injecting hundred modules.


essentially this function is not limited to strings but can also handle lists so maybe rename it to `trim_empty`


Mostly we are returning null in case of errors. Any specific reason why its undefined in this?


why double underscore and not single?


Hint: links which lead you to pages which are not hood.ie should have `target="_blank"`.


Not sure if we can automatically determine where rpc-extras is, but probably better to use /opt.


Can some of these playbooks be specified conditionally based on what user variables exist, ie. remote_check, dell_check, hp_check, etc.


In the MC I use this implementation to generate a slug
https://github.com/sphereio/sphere-mc/blob/master/app/assets/javascripts/models/Product.coffee#L149-L151

```coffeescript
generateSlug: (name, timestamp)->
  timestamp = new Date().getTime() unless timestamp
  _.slugify(name).concat("-#{timestamp}").substring(0, 256)
```


about time too lol


Can we use the rackspace-monitoring library instead?  I don't think there are long-term plans to keep pyrax around.  We're also already using rackspace-monitoring elsewhere.


We should be able to use the following: `var el = this.element;` instead of `rootElem = this.$()[0];` @vanthome @st-agnes


The file should be opened in binary mode (on line 267)


Add newline at EOF?


works for me..
```
rosdep check -i --from-paths src/
```
returns
```
System dependencies have not been satisified:
apt     libmuparser-dev
```


I wonder if you use the `git mv <oldlocation> <newlocation> on the bandwidthscheduler and then `git rm torrentcreatorthread.h` will override the automatic detection of git...


you never test search in between when start and end are the same do you?


@oritwas where is RGWFCGXRequest::fcgx getting initialized?


What is the reason for this pause?


Yes, that would be better.


TeamCity xBIM :: Xbim Geometry :: Release [Build 74](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1755&buildTypeId=XBIM_XbimGeometry_Release) is now running


yup


```ruby
<li<%= current_page?(search_path) ? " class='active'" : '' %>>
```
tambien puedes usar ```link_to_if``` o ```link_to_unless_current``` y cambiar los estilos entre ```li``` y ```li>a``` en vez de con la clase _active_


I think this debug message is now misplaced and should be moved to before the corresponding `ifdim` statement.


I don't want to permanently revert this commit, but I was getting strange run-time errors when I tried putting GLideN64 in its own makefile and I just wanted to get something posted before I went to bed.

I noticed two strange things.
 - When I created a separate makefile for GLideN64, I had to change the order of the cpu_features inclusion macro, otherwise the build failed.  I had to move it to the end of the GLideN64 makefile.
 - After doing that, I was able to build and run the front-end, but I got a segfault as soon as GLideN64 loaded.  I didn't have the segfault with your branch, and the only difference I could see was that your branch still had the monolithic makefile.

I probably just made an error somewhere when I cherry-picked your stuff onto a clean new branch and transcribed the changes into a separate makefile.  The android makefile syntax is very confusing sometimes.  I'll definitely give this another try next time I have some time, so that we can keep the split makefiles.


And who will be doing big endian and uClibc support?


it's always fun when it comes to debian packaging.

i don't want to waste more of your precious time on this before sage or sam acks on your pull request. just for the fun of reading.

- the Copyright fields do not confirm to the Debian Policy, see https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
- this is the MIT License. see http://spdx.org/licenses/MIT.html . and per https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/ we can simply reference it using its short name.


Not a criticism, but for my own knowledge, why is `\relax` necessary here?


Missing PHPDoc block


There should be a check on the value of $categoryId, even if it is set to int later on. What happens if it's "-1"?


Some issues with Tools, "Extract Place Data from Place Title", and "PlaceCompletion".


You can use any kind of brackets in filenames on Windows ([source](https://support.microsoft.com/en-us/kb/177506)), please add them to the list of allowed characters.


@oritwas I think this doesn't work, in master it uses the QueueRing structure to pre-init the FCGX_Request, in firefly we should retain the old behavior. So this commit shouldn't change the way we handle RGWFCGXRequest::fcgx.


esto pertenece a secrets.yml y no suele ir en github


id = '$lp_id' should go without the quotes, as it is supposed to be an integer.
Same thing for category_id (it is an int, no need for quotes).
It is particularly important for other DBMS (like Postgres) that do not accept setting an int *field* with a string


Again editing the README haha. Well done!


Couldn't this just say `if (this._observeResult)`?


The current command line did always implicitly download the best available format. Now the best available format is the DASH one which has to be merged using ffmpeg. I don't see why this deviates from ones expectation. If you want a specific format don't rely on default values.


Acabo de mandar unas correciones.


Added to the beforeEach because disable animation works only until you call browser.clearMockModules.
I think an browser.clearMockModules is missing inside an afterEach statement.


If this method is protected then it shouldn't have an underscore in the name.


corregido


Do you need the `=== true`?


а пустой тег точно необходим ?


Add this loader

```
      {
        test: /\.json$/,
        loaders: [
          "json",
        ],
      },
```

and json-loader as a dep


There should be an intval($_REQUEST['lp_id']) here, otherwise you are not filtering it and potentially using it in an HTTP redirect (header() call) without filtering, which is an XSS vulnerability


Still looking over the actual code itself, but tested this and it works really nicely.  This will be a great addition to our jenkins testing!

:+1:


что-то с кодировкой случилось


```ruby
<%= render partial: 'sources/search_item', object: resource %>
```
y dentro puedes user la variable **search_item** que toma su nombre del partial en vez de **resource** que lo toma de la clave del hash de la llamada y es mas dificil de rastrear


A bunch of places in this file which would more typically be `if (!Array.isArray(this.store))` rather than `if (Array.isArray(this.store) === false)`


I would suggest not to mix Task, Qualification and Currency information with workers model.  The serializers instead should be added to the corresponding serializer files.


что значит APA ?


Not sure what this event is needed for?   Also, the naming seems non-standard.


почему в случае "0" заполняется именно таким образом ?


These are unused imports, should be removed.


Is the `side` property relevant here?


возможно стоит эти самые случаи как-то назвать


magic numbers


What I did in the old version: https://github.com/sphereio/sphere-node-product-csv-sync/blob/master/src/coffee/mapping.coffee#L62


The calibration pipeline


1 2 0 1 0 1 0 1 0 1 0 2 0 1 0 2  ??


Interesting that this needs to be an O(n) operation.  Is there a performance issue?   I'm not sure how often this is called.


These characters shouldn't be here?


что именно объясняет этот коммент ?


Weird...maybe I missed a ```rosdep update```...now it works...sorry for troubling...


Do we have a stance on using the typed features of TypeScript?


Please don't print stuff to stdout in the tests


статику обычно называют аппер-кейсом


Google ip had blocked, :(


Ok, that makes sense.


Il est préférable je pense que cette fonciton renvoi un bool, qui vaut true si la future case est vide et false sinon. C'est préférable car cela permet de stopper le processus dans une direction impossible ce qui détermine directement si nous rajouterons la structure dans notre pile


тут творится что-то похлеще, чем в предыдущем switch


TaskViewSet, QualificationViewSet, CurrencyViewSet should not be merged with that of workers view sets. Its always helpful to develop each component as a stand alone module. I would suggest adding them to corresponding viewsets.

Also after making the changes run loaddata fixtures/meghaWorkerData.json  and other .json files in fixtures to see the effect of your code. (Whether it breaks anything its dependent on and stuff like that?)


This SimEngine code is currently untested.


не очень ок, что параметр функции изменяется во время её выполнения


Typo.


The exception is thrown on ln110


Will update the `clean` logic - good catch.

The FTP site has a throttled window... I'll poke at some things.


если ты для всех случаев делаешь фрейм невидимым, может стоит вынести эту операцию за пределы цикла ?


Bra projekt! Glad att se hur multikulturell du är! Lycka till med projektet :+1:  :+1:  :+1:  :+1: :+1:  :+1:


The naive idea is that if the piece must move, it can't move on a piece of same colour...

There are 4 problems here
a) must be really sure that the piece will have to move (the pawn attack is "solid")
b) some of this logic is probably already in theThreatedByPawn bonus
c) one of the piece of same colour can move with a strong treat (for example a safe check),
    leaving way for our piece.
d) let's say it must move. On the new square, the new mobility is mob2
    It is quite possible that the original "mobility + our pieces" was a better estimator of this new mob2
    than "mobility without our pieces".

As an historical note, originally, the mobilityArea was not containing our Minors or Majors. This was changed thanks to a patch by John Hellis 2 years ago.

Somehow, it is good to reward the defensive capabilities of a piece...but
e) should this be done in the mobility measure
f) a dark square Knight once it moves looses all the light squares defenses it was doing
g) a Rook or Bishop will keep defenses along the same rank, file or diagonal
h) a Queen might keep more

From there
I) I can retry this idea with a more solid "pawn threat" calculation, possibly retuning the ThreatedByPawn too
m) might try to allow at most 2 non-pawn defenses in the mobility count (3 for the Queen)
     (and might have to tune a little bit the MobilityBonus)
n) might consider a distinct Overloading penalty based on a naïve count of defenses done by each piece.

Mainly, we would like to catch situations where a piece has nowhere to go.


Потому что теперь DetailMessagesFragment.notifyDataSetChanged не вызывается, который раньше вызывался в parseSendMessageResponse


Will put back the bounce method once the boundaries are better defined.


ничего происходить не должно ?
может как-то логгировать это ?


I tested the patch without this hunk and the crash is still there. This validation seems to be required.


No. That was the file create when I made the custom bootstrap distribution.
If we use bootstrap from a CDN, it's irrelevant.
On May 27, 2015 12:31 AM, "Maks3w" <notifications@github.com> wrote:

> doc/html/assets/config.json is required?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/weierophinney/zend-diactoros/commit/a713c059b22a8d3b366d5f04996996119d86f7be#commitcomment-11378328>
> .
>


This looks wrong.

```#define DEGREES(r) ((r)*180.0/M_PI)``` looks better :)


Te weinig JQuery -1


Pk tu n'itères pas positivement ? currentAminoAcid est sensé croître non ?


если этот код не нужен - удали.


For testing purposes, search for the most generic properties from which you can logically deduce all desired properties. In case of recursive data structures, you may want to follow the recursion step. In case of sum:List[Int] → Int the following 2 properties suffice like 42 (to give an answer to everything):

Here n:Int , ℓ:List[Int]

1) sum(List.Nil) = 0
2) sum(List.Cons(n,ℓ)) = n + sum(ℓ)

From (1) and (2) you can show

* sum(List(n)) = n
* sum(List(n,m)) = n+m
* sum(ℓ without 0's) = sum(ℓ)
* sum(ℓ.reverse) = sum(ℓ)

For the last property you need to show first some properties for reverse:List[A] → List[A]


This patch has been merged as i into gstreamer-vaapi:

https://github.com/01org/gstreamer-vaapi/commit/f1a60ec6a2d9523c10713a43871df22ddc68d720

Thanks Jan!


Destructor should immediately follow constructors.


In primul rand ar fi ok daca nu ai mai da commituri pe github cu foldere bin si obj . Adauga te rog fisierul gitignore  https://github.com/github/gitignore (cauta pt visual studio)


One step for a man = 17 for octocat.


лучше ошибку тоже выводить.
а еще лучше - в лог


nu e ok string, DateTime


si contacts > bestProteinContacts alors ne faut-il pas supprimer tout le tableau bestProtein qui n'est alors plus à jour ?
et ne faut-il pas gérer le cas d'&galité ?


magic


Tempting to add real vector operators but doubt know how slow it will be.


Hi, If found some problems using your fork trying to upload the app with Application loader about an invalid signature in all .o files


side-note: we should be careful with the length of test file names ... this damn 99 char limit breaks "make distcheck". But this here seems to be within limit. Just wanted to make you aware of the problem.


Corregido.


Don't think this should be here, it's too specific to your use-case. I can imagine providing some generally helpful preds here (e.g. ignore these status codes, ignore 4xx).


Il est itéré ligne 30, puis on passe aux rangs au dessus (avec run()), puis une fois qu'on a fini les rangs aux dessus on décrément currentAminoAcid.


Perfect. Thanks.


чтобы бинарники не отправлялись на сервер существует .gitignore


Pretty much a rewrite :+1:


`nil` is a bit odd here, not self-explanatory imo. Maybe a note below about `:hystrix/bad-request-pred` is, and what the effect of not specifying `:hystrix/bad-request-pred` is, would help?


что-то с форматом случилось


à ce niveau là je pense qu'après un try positioning il faut rappeler run ? ou recreer une proteine...


C'est le cas, bestProtein est remplacé par currentConfiguration ligne 49.
Et oui il faut gérer ces cas-là dans la version "pile" qu'on va implémenter


LeaveType ar trebui sa fie enum


Then find out why, this is not expected. :)

2015-05-27 12:55 GMT+02:00 Víctor Manuel Jáquez Leal <
notifications@github.com>:

> I tested the patch without this hunk and the crash is still there. This
> validation seems to be required.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thaytan/gstreamer-vaapi/commit/b84e65ada15f9396b8c9ab507c1cb5f10c3f3914#commitcomment-11382936>
> .
>

--
Gwenole Beauchesne
Intel Corporation SAS / 2 rue de Paris, 92196 Meudon Cedex, France
Registration Number (RCS): Nanterre B 302 456 199


As we discussed, check on the localhost the paths appended work fine :)


зачем пустые строки в перед скобкой ?


Yeah, char received unknow pakcet.


> EACCESS has been is my favorite typo the past days :wink:

Yeah, I can see that. :smiley_cat:


Why do you have to add this partial? Any thing different with the advanced search?


Good point. I like your alternative solution using `MSYS`.


de ce sunt metodele astea statice?


Imprint ?


Ca change pas grand chose au niveau complexité, mais je suis d'accord ça sera plus pratique


Why not simply set nSubsidy = 1 ?

The rounding of int64_t will make the calculation equal to 1 during the next ~45 years.. then it will turn 2 for the next ~100 years.

#include <stdint.h>
#include <stdio.h>
#include <inttypes.h>
int main(void)
{
    int64_t money = 75000000;
    int64_t numerator = 20180075228;
    int64_t denominator = 1000000000000000000;
    int64_t quotient = money * numerator / denominator;
    printf("%" PRId64 " / %" PRId64 " = %" PRId64 "\n",
           money * numerator, denominator, quotient);
    return 0;
}


Pourquoi ça ? Le run est inclus dans le try positioning


You're changing the actual value that will get stored in the property, so if I set it to "foobar" and then to "a", if you read the value back again it will be "a     " which breaks the property.


Minor:

AUTH_URL and EVENT_URL refer to localhost instance, while WEB_SERVICE_URL map to xMatters server.

Should these not be on same instance?


Fix the linting error on this


> Also, it's easy to just keep a reference of this particular element in here, so there is no need for an id anways.

How do I do that?


Add comment for usage


Do I do that in size-rope.js then? And then I can just use it like that?


clasa Person nu are de ce sa aiba o instanta a lui Program, de fapt Program nu are trebui instantata niciodata.


This partial is to display a “condensed” view of results in the listing. I could not make it like this only modifying field accessors.


These hardcoded values (XML_HEADER, CT_JSON, CT_FORM) should be present in some other helper file.


How so? All output for the CLI should go through stdout not stderr


My understanding is that this maps the event data of groundwork and xMatters.

Can we keep two separate files e.g. xMattersEvent.js and groundworkEvent.js

These files can contain these strings as enums and those enums can be referred anywhere.


Add comment to explain the function's usage


Fortunately lua accepts \r characters.
39869aaa25ca577faa0da2b6cb83db6db7234342


xMatters seem to write comment for each constant being used.

Should we follow same or aggregated comments for 2-3 constants is OK?


ti-am zis, eu nu as baga console.writeline aici, nu e treaba acestei clase. Poti trimite un obiect de tip leave si sa-l adaugi la lista de concedii.


Try to avoid hardcoding strings in code. We can refer to constants through helper files


@mhugent Coverity has flagged this function as a possible error, as the loop will never execute more than once. Is this intentional?


Hello, the cc indicator is in library. I just add the configuration file for parameters and associations. I will check tonight if the indicator cc also works


This function is quite big.

Can we split this by using helper functions?


The option was just added a week ago, wouldn't it be better to remove it? And maybe add a make unoptimized target.


Добавил


Great idea! I just picked the first name that came to my mind because I was hungry. :D Will change accordingly.


MAJOR:

This is quite confusing and seems to be wrong mapping.

servicename is mapped to hostname
statusinformation is mapped to service

This needs to be corrected


bad rebase.


Jup, dacht ich mir auch.. ich hab mich deshalb auch dazu entschieden, nur exact matches zurückzugeben


completely true. will change.


@mattdrayer I have removed the 'singleEntity' attribute from certificate model and define the 'maxAllowed' attribute in certificate collection level because i think it make sense to add limit on collection level not in the model as every model can have its own maxAllowed value.


I'll try to use **kw also for calling the batcher, bypassing the function that takes "options" and passing directly the arguments, which may be beneficial for the user, e.g.:
batcher.run_in_batches (source, end_batch, **kw)


@andyjmaclean: with the plan to install Spotlight into our application, it became necessary to make the use of the styleguide optional per-controller, not automatically used application-wide, so that Spotlight can still use its own templates.

This has resulted in an adjustment to the europeana-styleguide gem, and any controller needing to use the styleguide now needs one of these `include Europeana::Styleguide` lines. Merge the develop branch into any of your feature branches to pick up these changes.


I will try to do it automatically.

On Wed, May 27, 2015 at 11:15 AM, Ian Mayo <notifications@github.com> wrote:

> @pecko <https://github.com/pecko> - this typo doesn't get caught by the
> compiler. I only found it because the BuoyPatternWrapper class has a
> "testMe" class that tests the properties.
>
> We were lucky that this class had a testMe class. I am pretty sure there
> are classes that don't have a similar "testMe" that creates a fake class
> object then tries to extract the editable properties.
>
> It would be bad if we had to wait for users to find places where the
> properties are incorrectly named. So, unfortunately I think we need to
> check that each class that has a Editable.EditorType member class also
> includes a testMe class.
>
> It would be acceptable to do this by hand. It would be better if we could
> automate this test - probably via introspection.
>
> Would you be able to do this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pecko/debrief/commit/68cab6442b523699ff186b644e3d4c3273fa8c26#commitcomment-11381318>
> .
>


Please remove this comment line.


No, that's a typo thanks.


This should be replaced by:

recipients = [];

....
Reason:

Using the new operator causes the interpreter to take all sorts of extra steps to go to the global scope, look for the constructor, call the constructor and assign the result... which in the majority case is going to be a a runtime array. You can avoid the overhead of looking for the global constructor by just using []


Yeah, but I didn't think there were any other cases where this would be useful. Are you thinking of Time?


by the way, ignore the information for now :P i'll add more detail to it later


It is currently hardcoded here.

Can be moved to separate file as constants/enums. These values can be used here to build the URL through string builder.


In C++ land we'd indent by 4 here.


And here.


Mostly, we are returning null in case of errors.

Any specific reason why it is undefined in this case?


not sure, might aswell add it back :P


Apparently, the problem is that I moved the files, and then changed them in the same commit.


Yes it was. Probably an oversight due to the fact that the create/update events shared the same action (I think we would want to prepend a new user but not update one?).

I'll just get rid of this function and handle the events specially.


Was expecting this to fail, but no so quickly.

The idea is that if my attacker is not attacked,
a) the positional pressure will last for at least one move
b) the threat, if any, cannot be ignored or exchanged, it is more "serious"
    (but opponent might simply interpose)
Problems are
a) more Bishop attacking Knight.
b) Values are not good. Local tuning suggest S(7,5) S(5,5) instead
c) Might have to reduce the Threat a little bit to avoid some overscoring.


Hard coded string to be replaced with Constants/variables


why should it? I cannot see why


"[...] where a character." - I feel like there's something missing at the end of that sentence :-)


I am not sure I understand the idea behind this fix.

Reading https://maven.apache.org/plugins/maven-assembly-plugin/assembly.html#fileSet

> outputDirectory - Sets the output directory relative to the root of the root directory
> of the assembly. For example, "log" will put the specified files in the log directory.

should the proper solution not be just to remove the initial slash characters from `<outputDirectory>` elements?
```
<outputDirectory>/</outputDirectory> -> <outputDirectory></outputDirectory>
<outputDirectory>/standalone/configuration/</outputDirectory> -> <outputDirectory>standalone/configuration/</outputDirectory>
```


These seem to be important queries to fetch the data. These can be placed in xMattersServiceConfig.js


Follow proper linting. Can use a tool like beautifier for this


Remove hardcoded strings


This is very nice indeed. As for the extension matter, I would choose something like `.hist` or even `.history` as modern extensions don't mind too much about the number of characters used nowadays. My two cents. Cheers :+1: :wine_glass:


Sorry for the abrupt commit. But it would be much better to have special cases like unoptimized as make targets than having them as default in the cabal file.


Só vi depois que você já tinha consertado, @Bouckaert. :P


Typo -> "developer"


typo: "create"


undefined vs null?

When are we using what?


shouldn't be there a line with `sign = -1;` here ? It is never set to -1 and it is a parameter of the `insertDof` function.


If it comes to else, response will contain nothing? Both tail and response are declared only.


Considered using a cleaning util like [rimraf](https://github.com/isaacs/rimraf)? I'm using  it as a precursor to the build step in react-stampit.

https://github.com/troutowicz/react-stampit/blob/master/package.json#L10

Also, I usually put linting steps before tests, reduces chance of errors..


Are you sure you want to silently *ignore* some columns if their name doesn't match the method name _[1-n] ?


I have the SolarInfoBank API from Sungrow if you want it.


removed and added new attribute in certifcates.py. one upper level.


I think it's too presumptuous to call `clearMockModules`, we will be removing stuff we didn't add. Let's introduce `afterEach` here, but remove only `disableNgAnimate` and `disableCssAnimate`.


Function name and function's actual work seems to be different.

Can consider renaming the function to a more appropriate name


Again test the generic properties along the recursive build-up of List[Int] using the corresponding function of Int

1. List.max(List.Nil) = Int.MinValue
2. List.max(List.Cons(n,ℓ) = max(n,List.max(ℓ))

and you are done, using a bit of maths.

Note, that it is common to define the maximum of emptiness to be the minimal element (Int, Double, String), and analogue min(Empty) = maximal Element. To be precise that is the definition of the supremum (resp. infimum) but this little impreciseness makes min/max complete functions and fits well

* **supremum**: least upper bound, i.e. the minimal element of the set of all inhabitants of underlying type which are larger or equal than all list elements
* **infimum**: greatest lower bound, i.e. the maximal element of the set of all inhabitants of underlying type which are smaller or equal than all list elements

The supremum is the max if the supremum is an element of the list. This is the case for all finite list except Nil. However an infinite stream might have a supremum but not a max (if the underlying type is not well-founded). Likewise for the infimum being the min.

Certainly the empty list of strings has as maximum the empty string for the lexicographical order but the minimum is not defined:

* List[String].max(List.Nil) = ""
* List[String].min(List.Nil) *undefined*

Using the real (unbounded) integers we could not define min and max of the empty integer list either.


We can easily get the tail by using RegEx rather than writing this logic.


if($tracker_id != 0) added


done


Function name: parse_events is completely different from returning the first event.

Can consider renaming the function


Spelling


Linting error


this could be
```javascript
return this._element || 'element'
```


Should not use variable names like i,j, k.... please make use of descriptive variable names


Why should the default be without -O? We can still add more -O settings in Makefile targets.


Does this want to be curly braces? rather than a functions normal parenthesis? As is in the same example below.


variable names like i,j, k should be avoided


Nice :)


If I understand correctly, steward has done exactely the same changes (without the cmake reorganisation) than I made, so I guess that the last issue was finally on my developer's environment.

I'll check when I get my MacBook back, and push onto master though 2 commits (one for the fix and the other for the reorganisation of the bundle build) :)

Thanks to him by the way :)


Need to understand eventMapping contents and validate the logic.


This is a spot where we might want to support a logger.


Fixed locally, will push when you're done.


Here I'd write:
```python
with pytest.raises(ValidationError) as excinfo:
    with data.validation as d:
        d['enumed_field'] = 3
assert 'is not in enum' in str(excinfo.value)
```


use generic defconfig from /boot and use this script:

#!/bin/bash

	export ARCH=x64

	export SUBARCH=x64

	make defconfig (from /boot)

	make

	make modules_install

	make install

u'll use default compiler.. don't use the compiler for Android xD.. so if u'Re building the kernel local for your own machine then an cross-compiler is not needed.


skompilowałem zły program - kolejny commit będzie właściwego, jak przekleję kod


form.properties is declared as object {}

form.recipients is declared as array []

Any specific reason? Need to validate the logic.

May also help to put helpful comments for this


+1 on what @verpixelt says :)
sorry, didn't see that before!


And here:
```python
with pytest.raises(ValidationError) as excinfo:
    with data.validation as d:
        d['enumed_field'] = 'aaa'
assert 'is not in enum' in str(excinfo.value)  # Enum error takes precedence.
```


Complex logic! Needs a refactoring.


Break this into helper functions as per Single Responsibility Principle


This is a horrendous patch, the original flirty_pawn patch was about 0.81 elo better than dev while this patch of yours is quite worse.


Empty `WEAVE_CIDR: ""` is due to weaveworks/weave#744


I will back it out and look at the change again to see why it breaks Boost.Parameter and MSVC.


Oh, read about it, but decided to ignore when someone said use posix format.

So the problem is still real?

On Wed, May 27, 2015 at 4:28 PM, Rainer Gerhards <notifications@github.com>
wrote:

> side-note: we should be careful with the length of test file names ...
> this damn 99 char limit breaks "make distcheck". But this here seems to be
> within limit. Just wanted to make you aware of the problem.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/janmejay/rsyslog/commit/9441bbd67f000625b8b4b9e7c697f1d7a70cf3d7#commitcomment-11383000>
> .
>

--
Regards,
Janmejay
http://codehunk.wordpress.com


```python
schema['$ref'][2:]
```
should probably clarify the intent: why are we ignoring the first two characters?


Can add internal comments for help in logic


This wouldn't catch an error if some of the columns had correct aliases and some columns didn''t have aliases at all (but their names would be different than _1, _2, _3...). If any column has an alias, we're binding *all* columns by name, so we should check *all* columns, not just these with different aliases.


This HAS to be logged somewhere: if we're failing for a network error we are accepting everything without the user knowing.


I think you missed the case in pollfd.c remove_wsi_socket_from_fds() where there is another

	if (wsi->sock > context->max_fds) {
		lwsl_err("Socket fd %d too high (%d)\n",
						   wsi->sock, context->max_fds);
		return 1;
	}

Note it use > here not >= which seems incorrect too.


I'll use std::unordered_map.


Linting error


I'm not a fan of this. Maybe there's a nicer solution.


TeamCity YaccConstructor :: YC.Core General Build [Build 392](http://teamcity.codebetter.com/viewLog.html?buildId=192089&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Whitespace at the end of the line.


@sagartalla, starting from this commit, your cucumber tests related to interactives are failing: https://travis-ci.org/concord-consortium/rigse/jobs/64085146
There were some build before it that errored, so it could have been one of them too.

This time it doesn't look like random fail, as it's always:
```
cucumber features/admin_work_with_interactives.feature:126 # Scenario: User can search Interactives using Subject Areas
cucumber features/admin_work_with_interactives.feature:157
```
Could you take a look at it?


Maybe this should be some namespace we own?


I added the RealUrl Ts on the blogsystem plugin page , also the PIDs in TS are configured , but how i can enable the RealUrl auto config ?


@mhugent this check has broken the "delete ring" tool with multipolygons


Нафига ты в ответы пихаешь вопросы? string[,] answers = new string[20,3] { {Tests.Answer1_1, Tests.Answer1_2, Tests.Answer1_3}, {Tests.Answer2_1, ..., ...}, ...}


Here as well.


TeamCity YaccConstructor :: YC.Core General Build [Build 392](http://teamcity.codebetter.com/viewLog.html?buildId=192089&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:50


I don't know which implementation is more efficient, but rather than convert to a Java String, you could use a node and call one of the Ruby String methods.


This can be changed as:

var ack = {}, event = {};


Can consider this function as a good reference for other javascript functions


Hardcoded strings


Why this function definition does not have "@property"?


Try to avoid i,j, k etc.


mm.gg


I guess we are supposed to use CRM_Mailchimp_Utils::checkDebug instead of CRM_Core_Error::debug_var


Consider adding comment for describing the functionality of this javascript.


i guess we are supposed to use CRM_Mailchimp_Utils::checkDebug instead of CRM_Core_Error::debug_var


Add comment


а здесь вместо s нужно подставить твоё рандомное число (фейспалм)


Because the main problem is that there is one internal field created per
anonymous struct in the class where it is contained. And the incompilation
error was only due to the generation of those fields. This code solves the
issue.
On 27-May-2015 3:52 PM, "Dimitar Dobrev" <notifications@github.com> wrote:

> Why do you check fields when the issue is about classes? You should check
> Class.Namespace.Classes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/genuinelucifer/CppSharp/commit/19fbecbb7032b3f48a89fefeec749dd4c58148f1#commitcomment-11382490>
> .
>


i guess we are supposed to use CRM_Mailchimp_Utils::checkDebug instead of CRM_Core_Error::debug_var

CRM_Core_Error::debug_var has been used in many places. Please change it to use CRM_Mailchimp_Utils::checkDebug instead .


if max_batches is None and is not specified... we have a problem. I'll check why it did not happen before


If this means to includ in the TS Template the "Additional RealUrl TS Config" , I included it...


Because then the user gets what they have configured. Like they has always done.

-O1 can be a serious performance degrade. Setting that as standard because of reported edge cases is unfair to the mainstream users.

I will try to measure what -O1 means in compilation performance when I get home.

There might be reason to use -O1 on Travis unless releasing as building is a major part of the run.

----- Ursprungligt meddelande -----
Från: "David Christiansen" <notifications@github.com>
Skickat: 2015-05-27 13:29
Till: "idris-lang/Idris-dev" <Idris-dev@noreply.github.com>
Kopia: "Niklas Larsson" <metaniklas@gmail.com>
Ämne: Re: [Idris-dev] Obey the user's optimization setting (0dd0d3b)

Why should the default be without -O? We can still add more -O settings in Makefile targets.
—
Reply to this email directly or view it on GitHub.


The below files are not supposed to be in the commit?

*CRM/Mailchimp/Form/.htaccess
*CRM/Mailchimp/Form/Config.IDS.ini
*CRM/Mailchimp/Form/fefe.txt
*CRM/Mailchimp/Form/httpd-app.conf
*CRM/Mailchimp/Form/httpd-vhosts.conf
*CRM/Mailchimp/Form/last error log moring.txt


Make initial-scale=10


Good spotting. I think this fix is simple enough to be sure it will work.


Description should be something like: Path to the system properties file used by the flow or operation. ${project_path} can be used for specifying a path relative to the project path. e.g. systemPropertiesFile: ${project_path}\content\base\properties.yaml


done


:cake:


`!important`?


The commit name is wrong. It fixes #26


TeamCity YaccConstructor :: YC.Core General Build [Build 393](http://teamcity.codebetter.com/viewLog.html?buildId=192090&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


But this period is used just for assertion. If anything I'll say "then-and"


Are these $binaries due to the use of mongodict? I hate this. In any case I think we should generate the fixture (instead of having this pre-generated JSON) on the fly, so that it would always conform to the current data-structures generated by the backend.


Right


TeamCity YaccConstructor :: YC.Core General Build [Build 393](http://teamcity.codebetter.com/viewLog.html?buildId=192090&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:01:33


Is this a typo? Shouldn't it be CONFIG and not CONTIG?


Grouping by date and bubble up is make data file normal size enough: https://tbicr.cartodb.com/viz/e6891cce-045f-11e5-8ed4-0e9d821ea90d/public_map


changed to fetch_Trkr_Events


Can we rename the table to 'mailchimp_civicrm_sync_errors' to make it more obvious.


This line doesn't work from here for some reason. But it works if applied manually afterwards.


The next line doesn't work from here for some reason. But it works if applied manually afterwards.


@papandreou: I did as you suggested and added a specific test for this in https://github.com/gustavnikolaj/express-interceptor/commit/30a15ba44c2f268412d4f9a31cc890ec269f04e7


Super działa ;) obecnie student ma liste przedmiotow i liste ocen, natomiast nie mamy wiedzy z jakiego przedmiotu pochodzi ocena... a fajnie byłoby dodać opcję wyświetl świadectwo studenta, wtedy dla danego studenta moglibyśmy drukować na ekran coś w stylu:
```
Małgorzata Dąbrowska 9911
Lista ocen:

Matematyka 5
Historia 4
Informatyka 6
```

ten [artykuł](http://javablog.pl/) opdsumowuje nasza wiedzę o kolekcjach i rozszerza ją o możliwość powyższego modelowania, z koleji w tym [pdfie](http://www-med.eti.pg.gda.pl/~jwr/Kursy/JAVA/Wpisy/2009/10/26_Java_4_3:_Modelowanie_obiektowe:_adaptery,_klasy_anonimowe,_tablice_i_kolekcje_files/java_jwr_4_3.pdf) na stronach 9-25 jest bardzo dokładny opis implementacji i użycia potrzebnych nam kolekcji :)


That's a good idea, will change it to the action!


Bonjour Julien,

Je n'ai pas de problème pour la mise en ligne des fichiers sur Google
Drive, tout marche bien (au moins il me semble). Par contre, en ce qui
concerne l'IHM je ne sais pas par où commencer. J'ai cherché sur Google
pour comprendre un peu mais c'est tout... Avez-vous des exemples ou des
indices?

Cordialement,

On 19 May 2015 at 18:28, Julien Holtzer <notifications@github.com> wrote:

> J'adooooore !
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JulienPobot/LeCubeMedia/commit/fb1b0b75bc4e00af622c48a66212a35d532d89e6#commitcomment-11265715>
> .
>


Lo hizo @iblancasa yo lo puse mal.


Remember to update the version in the README as well.


It might be better to move this check into `Serialize()`


Consider replacing these raw status codes with DRF constants which aid readability

http://www.django-rest-framework.org/api-guide/status-codes/


Don' really understand what the rest of the code does... need some explanations before I can review this.
What is the `with mock.patch...` is trying to accomplish? you define a context variable `mocked_method` which is never used...


Eagle eyes! Not so much 'broke', as 'didn't fix'... on it


What does -100 mean?


You're not checking that `config` contains key `PropertiesKey`. That will blow up at runtime. Also, it would better to move the check into `Deserialize` (to be on-par with `Serialize`)


I couldn't make up what is this supposed to do (how could this happen?) - could you explain? (short answer is fine). Is it related to this: https://github.com/refractproject/minim/pull/27/files#r31122438 ?


Removed in the latest commit. Was added for debugging purposes.


Test mention @ticketbis/backoffice


done


I'll be happy to do any improvement you think should be done here: it would teach me how providing more quality commits/PR in the future. :smile:


should now be fixed...


Can be shortened with `||`


Check out [the test](https://github.com/refractproject/minim/commit/f35b32abc40b24fe6c841deef0287d20fa1bffe3#diff-e17175f47337f0bf3b729243c76e62e1R45) to see a little example.

If you don't want to subclass, but just make an element that is pretty much a StringType for example, with the changes I introduced, you couldn't do that. It would break roundtrip, which means that if you convert from Refract then back to it, it would say that special element is a string rather than "foo."


I have done so in the final commit :) Thanks for that.


:) Im working on KLEIMDL1 now


我把你这一章节挪到了使用手册里面了 见文件夹 guide


@ktakagaki

I do not know yet how to respond to this error message. But the build.sbt file should be updated to resolve all dependency issues I had inmy latest commit.


Rather than doing DOM selection 3 times, consider caching this element in `$userModal`.


tiiiildeeeesss!
A python no lo gusta ni ahí las tildes! D:


@ktakagaki
I have added the dependencies to the sbt. I will read a bit about sbt and Scala now.


we could but it is not used if I remember correctly. this is the local table (element-wise) that holds the information on the sign and not the dof itself. The sign associated to the dof is used by modal basis in fact.


Been having fun? :p


stdenv.lib.optionalString (stdenv.isCygwin) ''echo ...'' is a bit nicer to express this IMO


Cool, I just had _aha_ moment.


It's exactly the same patch as the original flirty pawn, it even has the same bench. The only difference is I chose correct parameters for the run. Any variance between the two runs is just due to chance


@sajuptpm : Done.