I still feel this is very messy, and you're still playing a guessing game, depending on the speed of the computer/headless browser.

IMO, this repo should drop the materialize dependency (markup can remain the same for legacy purposes), but implement the ``sideNav()`` in Ember.

Think this justifies an issue?


I thought about, but that seems worse. What happens if you call it twice? Or never call it? This way avoids having the object in an invalid state.


hpwh_residential (nmerket) - x86_64-MacOS-10.9-clang: OK (1628 of 1643 tests passed)

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


Because this branch has a dependency on nineoldandroids, it's not clean.


fast-forwarding 2.0 to borshop-integration-1132-bugfix/jrd/2.0/epmd = 45d891ea


develop (kbenne) - i386-Windows-7-VisualStudio-12: OK (1152 of 1152 tests passed)

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


I was considering that.


后切勿 format 代码


Would this be ` get /logout => 'sessions#destroy'`?


GG::GUIImpl already keeps track of modal Wnds, so I don't think also registering with the HumanClientApp should be necessary. It also seems likely to not work quite as expected for unexpected reasons. So, if you need access to the list of modal Wnds, I suggest modifying the GG::GUI code to make it available, perhaps with an accessor function.


Two reasons.

 1. Since it's static and non-const it carries history.  if someone were to slip up in the future and modify `boundary` that modification would be carried forward to future calls of the same method.

 2. Since it's a static in a method, it must be initialized prior to its first use.  A typical technique that compilers use for this is to hide a flag somewhere in static memory and initialize that flag `false`.  On the first entry into the method the compiler checks that flag, sees it is `false`, initializes the static, and sets the flag `true`.  But on every following entry into the method the compiler must also check that flag.  If the local is not static then both the flag and the conditionally executed code go away.  The situation is worse with "magic statics", since the compiler must do the moral equivalent of acquiring a lock before it can check the flag.

It is possible that, in this case, the compiler would be able to avoid the flag (since it's not an "interesting" constructor).  But, at least with the old compilers I used to work with, none of them did avoid the flag.


We can maintain that as a state in the class.


If any of the modules contained within these modules have a direct :backward, they will be callled.
For example a direct :backward that can save memory, or one that uses multiple GPU streams to overlap gradInput and accGradParams computation.


Without this PR, even if a module implemented a direct backward, there's no way to use it in the network, because the containers will call updateGradInput and accGradParameters...


you sure?


If you want all inputs then how about
```css
input {
    font-size: 10px;
}
```


    var circleCenter = at.LargeArc ^ !at.SweepClockwise ? c2 : c1;

I'm not sure what you're trying to write here, but this line of code can be written much more clearly as

    var circleCenter = ((bool)at.LargeArc == (bool)at.SweepClockwise ? c2 : c1);

If you meant to use exponentiation instead of xor, use `Math.Pow()`


@rynkwn


Sure! It's better to keep up with the latest version of the little dependencies this has. Plus WS does adhere to the standard with this behavior, which is always nice!


@christopho I mentioned the packager failing a test on the actual 1.4.2 (from website and indicated tag on git).  The following is the relevant output of the failed test:

```
Start  2: mapdata
 2/12 Test  #2: mapdata ..........................***Exception: Other  0.15 sec
Solarus 1.4.2
Opening quest '/home/nathangl/debian/solarus-1.4.2/tests/testing_quest'
Map 'all_entities' differs from the original one after export.
*** Original map file:
properties{
  x = 0,
  y = 0,
  width = 320,
  height = 720,
  tileset = "castle",
}

tile{
  layer = 0,
  x = 0,
  y = 0,
  width = 320,
  height = 720,
  pattern = "3",
}

destination{
  layer = 0,
  x = 24,
  y = 29,
  direction = 3,
}

teletransporter{
  layer = 0,
  x = 16,
  y = 48,
  width = 16,
  height = 16,
  destination_map = "all_entities",
}

pickable{
  layer = 0,
  x = 24,
  y = 93,
  treasure_name = "bomb",
}

destructible{
  layer = 0,
  x = 24,
  y = 125,
  sprite = "entities/pot",
}

chest{
  layer = 0,
  x = 24,
  y = 157,
  sprite = "entities/chest",
}

jumper{
  layer = 0,
  x = 16,
  y = 176,
  width = 32,
  height = 8,
  direction = 6,
  jump_length = 40,
}

enemy{
  layer = 0,
  x = 24,
  y = 221,
  direction = 3,
  breed = "test_enemy",
}

npc{
  layer = 0,
  x = 24,
  y = 253,
  direction = 3,
  subtype = 1,
  sprite = "entities/sign",
}

block{
  layer = 0,
  x = 24,
  y = 285,
  sprite = "entities/block",
  pushable = true,
  pullable = false,
  maximum_moves = 1,
}

switch{
  layer = 0,
  x = 16,
  y = 304,
  subtype = "walkable",
  needs_block = false,
  inactivate_when_leaving = false,
}

wall{
  layer = 0,
  x = 16,
  y = 368,
  width = 16,
  height = 16,
}

sensor{
  layer = 0,
  x = 24,
  y = 413,
  width = 16,
  height = 16,
}

crystal{
  layer = 0,
  x = 24,
  y = 445,
}

crystal_block{
  layer = 0,
  x = 16,
  y = 464,
  width = 16,
  height = 16,
  subtype = 0,
}

shop_treasure{
  layer = 0,
  x = 16,
  y = 496,
  treasure_name = "bomb",
  price = 1,
  dialog = "shop_test_item",
}

stream{
  layer = 0,
  x = 24,
  y = 557,
  direction = 0,
}

door{
  layer = 0,
  x = 16,
  y = 576,
  direction = 1,
  sprite = "entities/door",
}

stairs{
  layer = 0,
  x = 16,
  y = 608,
  direction = 1,
  subtype = 0,
}

separator{
  layer = 0,
  x = 0,
  y = 336,
  width = 320,
  height = 16,
}

custom_entity{
  layer = 0,
  x = 24,
  y = 653,
  width = 16,
  height = 16,
  direction = 3,
}

*** Exported map file:
properties{
  x = 0,
  y = 0,
  width = 320,
  height = 720,
  tileset = "castle",
}

tile{
  layer = 0,
  x = 0,
  y = 0,
  width = 320,
  height = 720,
  pattern = "3",
}

destination{
  layer = 0,
  x = 24,
  y = 29,
  direction = 3,
}

teletransporter{
  layer = 0,
  x = 16,
  y = 48,
  width = 16,
  height = 16,
  destination_map = "all_entities",
}

pickable{
  layer = 0,
  x = 24,
  y = 93,
  treasure_name = "bomb",
}

destructible{
  layer = 0,
  x = 24,
  y = 125,
  sprite = "entities/pot",
}

chest{
  layer = 0,
  x = 24,
  y = 157,
  sprite = "entities/chest",
}

jumper{
  layer = 0,
  x = 16,
  y = 176,
  width = 32,
  height = 8,
  direction = 6,
  jump_length = 40,
}

enemy{
  layer = 0,
  x = 24,
  y = 221,
  direction = 3,
  breed = "test_enemy",
}

npc{
  layer = 0,
  x = 24,
  y = 253,
  direction = 3,
  subtype = 1,
  sprite = "entities/sign",
}

block{
  layer = 0,
  x = 24,
  y = 285,
  sprite = "entities/block",
  pushable = true,
  pullable = false,
  maximum_moves = 1,
}

switch{
  layer = 0,
  x = 16,
  y = 304,
  subtype = "walkable",
  needs_block = false,
  inactivate_when_leaving = false,
}

wall{
  layer = 0,
  x = 16,
  y = 368,
  width = 16,
  height = 16,
}

sensor{
  layer = 0,
  x = 24,
  y = 413,
  width = 16,
  height = 16,
}

crystal{
  layer = 0,
  x = 24,
  y = 445,
}

crystal_block{
  layer = 0,
  x = 16,
  y = 464,
  width = 16,
  height = 16,
  subtype = 0,
}

shop_treasure{
  layer = 0,
  x = 16,
  y = 496,
  treasure_name = "bomb",
  price = 1,
  dialog = "shop_test_item",
}

stream{
  layer = 0,
  x = 24,
  y = 557,
  direction = 0,
}

door{
  layer = 0,
  x = 16,
  y = 576,
  direction = 1,
  sprite = "entities/door",
}

stairs{
  layer = 0,
  x = 16,
  y = 608,
  direction = 1,
  subtype = 0,
}

separator{
  layer = 0,
  x = 0,
  y = 336,
  width = 320,
  height = 16,
}

custom_entity{
  layer = 0,
  x = 24,
  y = 653,
  direction = 3,
}

Fatal: Map 'all_entities': exported file differs from the original one
```

I also know that the original "1.4.2" did not encounter this problem.


Would probably be a good idea to make some minor changes here...


Line Note


The hints look good!

The `StandbySet` was intended to be used so that we could switch to the obvious paint questions for a demo, but we don't use it so it really doesn't matter. We could probably just delete it and the method that swaps the sets.

P.S. It looks like the 2nd and last hints are dupes


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


@me4502: This broke the build.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1152 of 1152 tests passed)

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


I doubt that there's any practical problem here in the short term, since [a 200ms animation](https://github.com/Dogfalo/materialize/blob/master/js/sideNav.js#L79) would have to complete AFTER a `setTimeout(..., 1000)` was called. I don't believe that CPU speed will cause any problems because we only have one thread.

Totally agree that we should use ember-idiomatic JavaScript in place of the materialize JS stuff -- that's the right solution to cleaning this up. Once we're using something like [liquid-fire](https://github.com/ef4/liquid-fire), all of the animations will work properly with `andThen`, and it would be possible to write a test like this in a way that almost looks like synchronous code.

Please go ahead and create issue(s). I'd be more than happy to help out if I have time :)


``positions must be >= 0``


The ```isset()``` would return false if ```$options['option']``` didn't exist.

Thanks a lot for commenting on a commit, I appreciate it a lot!


sure. sure. test it with the account that says "DONT CHANGE DONT EDIT"

we should probably tie the test account to one that wont potentially break all of our demos. lol?


Do we need the same check for start?


`M110` is one of the few commands that is processed immediately on receive. In the old parser, you only saw the "unknown command" error if you sent something like `Q52`. Now it will bark if you send `M429` or `G9999` also. Now there is the question whether to reply with "unknown" if the firmware just happens to be compiled without some feature. For example, `M117` without an LCD controller will just do nothing, but it could bark when there is no LCD if that was desirable.


rm $@ || touch $@ seems wrong to me. If you always want to create an empty file, then you should use:

rm -f $@ && touch $@   (i.e. remove it if it exists and then use touch to create an empty file)

Coded as rm $@ || touch $@, if the file exists it will remove it and NOT touch it. If it doesn't exist it will fail to remove it and then touch it.


@stammen  In my computer, there are no issues.
@ricardoquesada  We do have merged some bad commits which contains a few big binary objects, but we have done `git push --force` to remove the history in the main repository.  We have considered to use `git revert`, but the history is still there. So we use `git reset --hard` to discard the bad commits.

@stammen  Maybe you should clone a new repo? thanks.


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)


I encountered this problem in both my 64 bit true environment and my 32 bit chroot environment.  It's possible to override the testing portion of the package building... if you are OK with me doing that, I will.  I'm not posting an actual bug because this isn't a development version copy and will not effect most users.


This was the magic incantation I was looking for...


`const` goes to the right of the thing it makes const. Also, do we want to edit their sources?


On 5/27/2015 5:11 PM, Allan Stewart wrote:
>
> Hmm... well, |jsdom| still requires |node-gyp| so I end up with the
> python problem again anyway. Are you still using jsdom? I'm trying to
> get straight on which dependencies we're still using.
>
> If so, we may just need to decide that python is a requirement for
> anyone doing this kata. Potentially more complications for getting
> people started, but we can use those tools.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/allan-stewart/react-hogwarts-tdd-kata/commit/5b81a8cfd34407d8f609c3ea58b403a98894a4b9#commitcomment-11396695>.
>
I don't think we are.

--
Zhon Johansen
Facilitating good to great!
zhon@xputah.org
+1.801.901.0146


Are you planning to support localization?


Ok cool, thanks.


since there is no `this.push` here, this pipe does not forward anything sourced from line 6


We only pipe to `gulp.dest` (which, like the above also swallows piped events) if we aren't using closure.


I didn't quite understand the question. Would you elaborate please? Thanks.


Sorry @stammen
Do you meet this issue on Windows system? On my computer, only one file's git status is changed.
And I think we should remove the source files in `cocos2d-x-3rd-bin` repository. And only keep the binary files there.


Full Change log: Changed cored camera mechanics and constructor. Fixed aspect ratio of resizing windows. Added @Legacy annotation to denote legacy code.


If we are using closure, the sourced events from `utilsthemeBuildStream()` make it through (because nothing, like `gulp.dest` is swallowing them - so now we have a floating `default-theme.js`. which will make its way through to the pipes below.


Então enfia esse teu código no cu


This "micro plugin" will concat all of the files contents that come through it (in this case just `default-theme.js`) and then on the stream's `end` event, it will call the flush function which reads the existing file, appends the buffered contents to it and re-emits the original file with the new contents.


See below for the comment on `utils.appendToFile` but since only `core.js` makes it through, we then use `gulp.dest` to write it out.


upon further investigation we should be good. doesn't look like it actually sends data. so thats swell.


We've followed the pattern of capitalizing only the first letter of acronyms so that constructions like HttpFtpAdapter are unambiguous. Let's keep doing that. (Omniglot was an aberration.)


I just wanted to tie it to a real account that isn't likely to be deleted causing this test to fail suddenly. An alternative would be to have a safe word that the api end point recognized as a no-op of sorts.


:tada:


:wolf:


Wow, thank you very much guys :D


This fix is needed for node v10, and was proposed while we were using that version. Agreed that it is not needed for node v12.


Heh, let's hope it'll be accepted now =)


This change-and-change-back is a good example of something I *would* squash out.


Generally we use assertion-style checks for this, which keeps the code more succinct. Is the a reason we're not doing `shouldBe('gl.getParameter(gl.TRANSFORM_FEEDBACK_BINDING)', 'tf0')` or similar?


Indentation looks odd to me.  Usually, when I see the return type on the preceding line, the name of the method is at the same indentation level as the return type.  Like `operator<<` and its returned `scoped_testcase` on line 117.


rax retry tests


@dweeezil you just missed removing the format specifier here.  Nothing major.


The original code assumed that arm and some other architectures would
properly emit assembler that could handle cases where the pointer passed in
was not actually aligned to a jlong. Unfortunately, gcc 4.9 added a new
optimization that emitted a ldrd instruction which does have an alignment
requirement, and caused a fault when trying to execute using one of these
unaligned pointers.

Christopher

On Wed, May 27, 2015 at 9:41 AM, Joe Maples <notifications@github.com>
wrote:

> The message states that there's a bug, but doesn't say what the bug is.
> Mind elaborating @cferris1000 <https://github.com/cferris1000> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/temasek/android_libcore/commit/4d46de25bdb6d3b8150612a49849e251dcfcb035#commitcomment-11389875>
> .
>


While we're removing the `ajax_` prefix, we might also simplify `user_logout` to just `logout` to make it more consistent with this new pattern.


The hledge/Hledger/Cli/Utils.hs:71 one goes away in the next commit. (I like having each commit function properly)

I initially had `add` ensuring that all the files existed, but thought that this behavior made more sense. (]hledger add -f newjournal -f typoedjournal` shouldn't create typoedjournal). So we could change the behavior or split it into primaryJournalFileFromOpts and moreJournalFilesFromOpts.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.183](http://teamcity.monogame.net/viewLog.html?buildId=7706&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:20:18


I really like how all of this turned out. It's so clean, maybe even a better separation of concerns than before. How's the speed?


Don't forget to test that isTransformFeedback(tf) is false after tf is deleted!


This seems like an unresolved merge conflict that was committed accidentally


Thanks for doing this style fix. It looks much nicer. I will follow this pattern as well from now on.


Add in the installation doc and requirement that restriction. I'm fine with that restriction.


Isn't the thing we are deprecating the reverse, not doing the set?


Consider combining these two into a single `set`, as you've done in the other file in this commit: https://github.com/lewfish/model-my-watershed/commit/e7ade5b915d3dfe71ae9a19a07fddf6e119c1226#diff-bbc329caa6aad1d9e2bf745db301390fR172


Same here.


en都改了


Thanks for teaching me about the purpose of the `name` attribute today!


Dammit that was really for #1315!


Which move this to the top since you use it in examples above this section.


In class we moved all this logic into the model and a helper method to render partial.


Not needed. this is always brought in by globals.php


Ah sorry looks like you already fixed it in https://github.com/lewfish/model-my-watershed/commit/26f6eb8a6ccb428a4b04d757e7ffaddd2b26c482. Please ignore the previous comment.


Don't use this commit... should be <Beta> not Beta>


Regarding loader.py - it's a portable way to automatically load anything in the blueprints directory. In the past we did do something like yours, but we kept forgetting to update the configuration we had, and the path building we had before didn't work cross-platform or cross-environment all the time (I use PyCharm, and it doesn't work).


What is this attr() for?


Because the VAO test that this is mostly copied from did it this way. :D
I'll clean it up, thanks!


disregard above comment. realized it's javascript code.


good


The e parameter and other parameters of xl have been dperecated. note that e will bypass the escaping. So, need to always echo it:
``````
<title><?php xlt('Pre-billing Issues Report') ?></title>


Note it is xlt() instead of xla().
text() and xlt() are used for normal html text. While attr() and xla() are used when escaping text within html attributes.


I tried it, it works, it's even the non-jqueried DOM element, so exactly what we need...


For what so many toolchains? It's just increases repos download time and takes a lot space on disk. If i need to use custom toolchain I can download it manually.


above block is correct use of attr()


Fix tabs


Not within an html attribute, so can use text() instead.


people are complaining that voip isn't working under L (and of course they give no logs). Does this help with that?


above 3 rwos can use text()


processCounter.js is live, i comment out the elastic search and google analytic


I suppose that I'm in the static language / compile everything in camp: I think this library should simply require boto as a dependency.  I don't think it would be very useful without it


What do you mean by 'Every other JS gets is very hard to track...'?
Did you mean 'Every other JS is very hard to track...'?


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (636 of 636 tests passed)

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


"Every other JS action" suena mejor?


Faltou o ponto final. Na renderização html ficou tudo junto


This indicates that the `Scope` class inherits from itself.


@RichardLitt it just means it's subclassed. so it's equivalent to something like:

```
class Foo
class Bar extends Foo
```
in some other language

or

```
var Foo = klass(...)
var Bar = Foo.extend(...)
```
in js. but we don't even _need_ to do classes, it just means that the subclass objects should use the parent's fields and functions too.


I lolled :+1:


Seems familiar :+1:


That is so creepy. I have more or less the same commit locally but had to run to catch the train before I could push!


> Why is this an array, and not just a string?

it's an array of different strings. each string is a type of media.

> Also are conversations / annotations / notes themselves a form of "media"?

yeah we can count them as such.

> What is this for, exactly? When is this used?

to help disambiguate how to display things. the UI will change somewhat depending on whether we're looking at a publication or an image or a webpage, ...


grey-extra-light is defined as #e8e8e8. Its nearly identical to #eee, so use grey-extra-light instead of creating a new color.


maybe default to  empty array  , so that we don't need the test "is_array"  above ?


This was previously only returning null


Looks like this is the result of a bad merge?

In 7.x-3.x, $font-heading was renamed to $font-bold so that we didn't have to create new variables like $font-pager.


Changed Snapshots to also use global owner tag key instead of hard coded creator


Use $font-bold


Once again, I think we are on the same page here. We can't be curating and manually fixing things up, but if we can download and attach to what we do to help maintain the user's orientation, that is a clear benefit.

Sent from my iPhone

> On May 26, 2015, at 9:25 PM, cshenry <notifications@github.com> wrote:
>
> I would like us to deal far more thoroughly with pathways, including such data from KEGG, MetaCyc, and BIGG. In playing with BIGG recently, I realize they now offer easy download of virtually all their data, and we should exploit this.
>
> I propose we have an entire primary table dedicated just to pathways, with the following columns:
>
> Name: primary human readable name for the pathways
> ID: we should assign our own consistent IDs to pathways (e.g. path.1, path.2 etc)
> Source: where the pathway came from (e.g. KEGG, MetaCyc, BIGG)
> Source_ID: what is the ID of the pathway in the database it came from
> Aliases: list of aliases for the pathway
> Reactions: list of reactions in the pathway
>
> Pathways are useful in a 1000 ways� and not handling them thoroughly has been a real impediment for us.
>
> I can think of many pathway sources we could integrate: KEGG, MetaCyc, BIGG, Subsystems, Scenarios. And we should do them all�
>
> But a key question? Should we be working towards ultimately reconciling this data to maintain our own pathway ontology??? Something to think about. The issue with integrated pathway data is such data will never be consistently applied across the board.
>
> I generally don�t want to sign on for excessive curation commitments, but if we could come up with a computational rule we could automatically apply to maintain our own pathway ontology, I would favor that...
>
> On May 26, 2015, at 5:18 PM, chian <notifications@github.com> wrote:
>
> > Just to chime in. If it's worth that much we should consider something more explicit than embedding it in a name. I'm not all for tracking every bit of data, but let's add this to our discussion so that we at least cover the pros and cons. Presumably if that data is truly easy to ingest, we can consider it, but otherwise, it will be a project unto itself of which we have enough.
> >
> > Best,
> > Nick
> >
> > Sent from my iPhone
> >
> > > On May 26, 2015, at 12:32 PM, janakagithub <notifications@github.com> wrote:
> > >
> > > May be not directly relevant to what you do, but I would like to point out that having the pathway data from MetaCyc and/or KEGG mapped to modelSEED reactions and integrated to our biochemistry object is extremely important. These traditional pathways that biologists are familiar by their names (e.g reductive TCA cycle) defines the metabolic capability of the organisms for those experts. Not to forget this data could be extremely useful on model/genome analysis, derive organism profiles and on pathway based gapfilling efforts.
> > > Since the data is being reorganized, I would suggest adding these data into the biochemistry object with a pathway identifier and the human readable name for both KEGG and MetaCyc derived pathways. I would imagine, for a given reaction minimally, pathway data could be included as
> > >
> > > pathway_types : MetaCyc or KEGG
> > >
> > > for MetaCyc :
> > >
> > > MetaCyc_pathway_1_name : pathway_1 identifier
> > > MetaCyc_pathway_2_name : pathway_2 identifier
> > >
> > > for KEGG:
> > >
> > > KEGG pathway_1_name : pathway_1 identifier
> > > KEGG pathway_2_name : pathway_2 identifier
> > >
> > > In addition, If you have the data for which step the certain reaction belong in the pathway (e.g MetaCyc_pathway_1_name : pathway_1 identifier : step 3
> > > ) OR the other connecting reactions for the particular pathway would be nice, but one could generate the whole linear pathway if the above information is provided.
> > >
> > > �
> > > Reply to this email directly or view it on GitHub.
> > >
> > �
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub.
>


é possível deixa**r**


Changed Images to also use global owner tag key instead of hard coded creator


應該是空白混 tab 才會對不齊


Scaling should be done w.r.t. nose, currently done w.r.t. origin at 1,1


Nice, works.


asd


Here's a solution. Since this is named "for comprehensions", I'm sure there's a way to reorder it into `for(..) yield { }`, but this is what I came up with:

```scala
  def sentenceAnagrams(sentence: Sentence): List[Sentence] = {
    def f(prefix: Sentence, postfix: Occurrences): List[Sentence] = {
      combinations(postfix)
        .map { subset =>
          dictionaryByOccurrences.get(subset) match {
            case Some(words) =>
              val sentences = words.map { word => prefix :+ word }
              subtract(postfix, subset) match {
                case Nil =>
                  sentences
                case rest =>
                  sentences
                    .map(f(_, rest))
                    .flatten
              }
            case None =>
              List[Sentence]()
          }
        }
        .flatten
    }
    val occurrences = sentenceOccurrences(sentence)

    if(occurrences.isEmpty)
      List[Sentence](Nil)
    else
      f(List(), occurrences)
  }
```


evitar o empilhamento de expressões ternárias


how'd you get rid of the jsoncpp dependency?


Hmm, should use event delegation and add `rel="external"` to the links instead.


dx, dy should only be found from nose, eyes and mouth haven't been scaled/rotated yet so there is error. This translation is for setting the reference point as the nose @alanxwl


missing new line.


(...), ela **ainda** será chamada...


Test


I see what this is doing but it's a bit confusing (even though the documentation recommends).  You can be a bit verbose by using `Scope < ApplicationPolicy::Scope`


Rather than use `reject` you should be able to use `.where.not(id: excluded_reservation.id)` so that it's part of the query instead of a separate Array operation. You could also therefore pass in an array of reservations to exclude and call `.where.not(id: excluded_reservatons.map(&:id)`.


I'll be back Monday or Tuesday


For semantics let's use either `excluded_reservation` or `excluded_reservations` (see my [other comment](https://github.com/YaleSTC/reservations/commit/8e6efc3da704a3a5a663a372e23e6f0fab5171ac#commitcomment-11377303)).


This is a great commit message.


Extra line break.


Fixes #5


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: 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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-69.81%25-green.svg)


Fixes #3


eu diria "caracteres ASCII planos (a-z e A-Z)". Mas sugiro buscar uma segunda opinião sobre isto


se**r**ão ignorados


Just a suggestion, include the activity name in the dialog

"Wanna save changes made to X first?"


1836664-ReturnWaterTempReset (mjwitte) - x86_64-MacOS-10.9-clang: OK (1659 of 1659 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)


src/modulo-03-web/Extra/exercício adicional AJAX e JSON/exercicios/exercicio1.html

oi, esta lógica funciona, mas fazer o seletor **PARA CADA** resultado dentro do array items pode ficar pesado se o array for grande. Invés de fazer 50 consultas (no HTML, através do seletor) fica melhor fazer apenas uma, não? Para isso basta jogar `$('#imagens')` para uma variável antes do `forEach`.


Tipos escalares


做个标记


"May be worth mentioning that it’s linux? dunno if that matters for this, I know we transitioned the original original server from windows to linux without issue, but maybe it matters for mods?"

Copied and pasted from chat when I inquired about any other possible changes.  Only thing he could think of, not sure if it makes enough of a difference.

Thank you so much for looking in to this for me. D: I really appreciate the help.


vai ficar difícil de dar manutenção nesse estilo. sugiro criar uma classe `quadrado` que tem `width: 300px; height: 300px;` e aí tu adiciona no append `'<img class="quadrado" src=...>'`


Why touch first?


nice passwords


outra coisa, pode até funcionar esta lógica.. mas aí tu vai gerar um HTML não semântico (cada capa do album vai ser uma ul), quando o certo seria:

```html
<ul>
  <li><img src="album1.png"></li>
  <li><img src="album2.png"></li>
  <li><img src="album15.png"></li>
</ul>
```


Em alguns trechos deste arquivo foi citado como Integer, em outros como inteiro...
Julgo correto buscar utilizar somenta uma forma de representar este tipo de dado, ou indicar por parênteses seu significado:

inteiro (integer)
integer (inteiro).

O mesmo se aplica ao boolean e float.


This comment is not necessary, and in fact somewhat misleading. We can't assume that whomever called the function was smart enough to verify their work is done. It's also relevant to note that the "process" as it is is not in fact done, because PHP is still running if you can call this =) So, it's really the "work" that is presumably done ;-)


@mrunalp i updated and amended this commit.


Verdade, fiz isso mais vezes nesse exercício, vou atualizar.
Boa!


Nice. I need to learn more about what these are. Today I encountered `StandardGeneric` and didn't really know what it was.


Just straight `return true;` and `return false;` I'm not a fan of extra variables like these.


This does not work completly. Sometimes I get the dialog when i didn't edit an activity.

1. Start with a new database ( delete the database.db )
2. Edit "The cool activity" by putting Earliest Start to 1.
4. Save change made to "The cool activity".
5. Select "The ugly activity".

Then the dialog will appear, even though I just saved "The cool activity".


what about signal raised again in the second read call?

the classic codes for this, is use while..

thanks


`Returns true if $checkPid is found in the process list; otherwise false`


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

Yeah, this was before I saw your second message. I'll fix it when I get home.


Hah - sorry it took me a while to push this, too bad we duplicated work.


"the all caps representation..." não seria dizer que "a representação em caixa alta..." ?

Segunda opinião, por favor.


Don't eat the exception, at least throw a RuntimeException


It was displaying the room, now how many tiles away it was.
`the nearest pit is Room[id = 30...`


src/modulo-03-web/Extra/exercício adicional AJAX e JSON/exercicios/exercicio2.html

tenta sempre carregar os scripts no final do <body>, para otimizar a renderização da página.


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

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


estrutura coesa


Ahhhh I see.


@moollaza did I get this accurate?


#include <iostream>

using namespace std;

int bsort_min(int *a,int n);
int bsort_max(int **a,int i,int temp_y,int m);

int main()
{
    int **a,i,j,m,n,temp_y,temp_x,k=0;
    cin>>m>>n;

    a= new int*[m];
    for(i=0; i<n; i++)
    {
        a[i]=new int[n];
    }

    for(i=0; i<m; i++)
        for(j=0; j<m; j++)
            cin>>a[i][j];

    for(i=0; i<m; i++)
    {
        temp_y= bsort_min(a[i],n);
        temp_x=bsort_max(a,i,temp_y,m);

        if(temp_x==i)
        cout<<i<<' '<<temp_y<<' '<<a[i][temp_y]<<endl;
        else
        k++;
    }

    if(k==i)
    cout<<"Not found"<<endl;

    return 0;
}

int bsort_min(int *a,int n)
{
    int i,min=a[0],min_y;
    for(i=0;i<n;i++)
    {
       if(a[i]<min)
       {
           min=a[i];
           min_y=i;
       }
    }

    return min_y;
}

int bsort_max(int **a,int i,int temp_y,int m)
{
    int j,max_x,max=a[i][0];
    for(j=0;j<m;j++)
    {
       if(a[i][j])
       {
           max=a[i][j];
           max_x=j;
       }
    }
       return max_x;
}


It can also be `process.env['no_proxy']`.


这是什么


Agreed, will remove.


Done.


OBS: Os cenários foram equivocadamente enviados em outro commit: https://github.com/tiagonog/rgms/commit/1fae15b16023b745e28a2f6b68606c201446cddb


好像有点问题


Would you mind using `$command->getResult()` so that you can check the exit status instead of letting bad command raise exceptions. Sorry, this was a bad design decision I made long ago and I need to mark `run()` as deprecated.


this is too specific, what happens if I add another page? the filter would need to be modified with a new counter and this violates the open/close principle use a map instead


Please name this variable something closer to its purpose, e.g. `$psCmd`


close #2


Also could be written `process.env.NO_PROXY || process.env.no_proxy || ''`


Can you add a few more tests that try locking on the same pid twice? And another that does the full lock and cleanup?


I'd recommend looking at the implementation in request: https://github.com/request/request/blob/5d4c83db53accf44328862e75ed56424bd88e1d1/lib/getProxyFromURI.js#L45


Variable Variables é meio complicado... Não são variáveis de variável.
A tradução literal "variáveis variáveis" é correta, porém confusa.

A tradução mais próxima que já encontrei foi "variáveis variantes".

Fica a seu critério


don't eat the exception, at least throw a RuntimeException


catching RuntimeException is considered to be a bad practice


same thing


Did you intend for this to go in?


@mhugent just to confirm - is the intention here to alter the feature's stored geometry?


@alexandru7 you should make a pull request to the main cryptoglance app for this change. It can be useful


I think you want to use getAttribute instead, since you are setting that value on the servlet


If the file doesn't exist, `file_get_contents` throws an exception.


No, I'll undo that.


fica mais fácil vc colocar um input submit no formulário e, via jQuery, interceptar o evento de submissão

```javascript
$('form').submit(function(e) {
  // código do Giovani
  return e.preventDefault();
});
```


C**o**nstrutores da linguagem

Talvez "construtores" não seja a palavra mais adequada... pois estamos lidando com estruturas da linguagem.


```FunctorDepthAverageSinkingVelocity``` uses ```in.position[q]``` to look up gravity. Position is not initialized if you return ```false here```. :-)


seria melhor "limpar" a tela se vier resultados na próxima busca, não acha?


True. The !== undefined works well. But I wasn't sure why hasOwnProperty was used. So this is just a safer alternative.


:+1: por quebrar o problema em "funções"


Indeed. See the commit message: "local_assemble_advection_system calls compute_viscosity which calls
compute_advection_system_residual which uses
scratch.explicit_material_model_outputs.reaction_terms which was not
initialized for compositional fields."


(...) internas do PHP ou classes/funções/constantes de terceiros.


Another bug where we used to read invalid data. :-)


weird, that definitely looks like it *should* work.


Can you raise an issue with the errors?


The Gpermute has been tested on AVX too (Grid_cshifts). OK.


``` go
"main: setting maximum number of CPUs to %d, total number of CPUs is %d, GoMaxProcs, runtime.NumCPU()"
```


LHS should probably just be Set.


So that's why it wasn't working. Nice job.


GoMaxProcs. Golang should always be `Go`


95077770-HVACTemplateLoadManage-4493 (mjwitte) - Win64-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)


com uma exceção: a palavra chave [declare](http://php.net/manual/pt_BR/control-structures.declare.php)


Done.


Nice job, Michelle!


just stop.


impricará -> implicará


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

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


nome de clásse pré-fixado


Canola not spelled right dummy just like in the udacity lesson LoL!


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2117 of 2202 tests passed)

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


I guess the gist of both branches of this is that they produce 1..N VisualAct objects. So you could try using flatMap() instead of forEach(), which would return a Stream<VisualAct> which is then collectible into the final collection using Collectors.toCollection. If you extracted the lambda into a proper method it might look neater.


nice seeds, indeeds.


I'm fairly sure `wv;` here just indicates webview, but not necessarily "Twitter Webview". i see some that have wv + other app suffixes, and some that have wv + `TwitterAndroid`.

see: https://developer.chrome.com/multidevice/user-agent


Ah. Maybe add a comment?


Fixes #10


I don't understand your comment. I am looping over all values i and if one is zero I set all to zero?!


Aw, darn. That's awkward.

You left the block that followed the conditional in braces. That may or may not make sense. Use best judgment.


**são** aproximadamente 2500 funções


第12章少了一节，难道是因数我看是旧版的PDF?
12.1. QUIC


Fix blinking issue on some web browsers


Utilizar link para versão pt-BR do manual PHP


Please return sources HDPI.


cuidar indentação


终于拆分了。。。= =


mesma questão, cache de seletor!


:+1: ok will try this


I'll try again tomorrow, but this workaround didn't appear to work for me.  Perhaps a better approach would be to go about parsing the exceptions list without having to set IFS.


For readability, Let's change the way dependencies are declared.
Based on docs http://hapijs.com/api#plugins
```
register.attributes = {
    name: 'Auth',
    dependencies:'haps-auth-basic'
};
```

> dependencies is an optional string or array of string indicating a plugin dependency.
> Same as setting dependencies via server.dependency().

Earlier gitter conversation with hapijs gitter channel, led us to think setting in attributes would not work.
But, according to the documentation, this should be fine. Let's change it to make the code more readable.


Maybe it would be better to move include $(CLEAR_VARS) after line LOCAL_PATH: $(call my-dir).


setFormData
loadFormData
で定義することになると思うので、setterは不要になると思います。


why you do this @Alx31
you kicked xiaomi :(


Arreglado problema en creación de areas y unidades mencionado en issue https://github.com/CamilaAlvarez/IndicadoresDCC/issues/11 .


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


valueの値がデータベースに入ることになるので、ある程度他の人にもわかりやすいものにしておいてください。


This file can now be removed, as well as its reference in the widget HTML


Should use bootstrap helper class `hide` instead


src/modulo-03-web/Extra/exercício adicional AJAX e JSON/exercicios/exercicio3.html

duas coisas:

1. Pode retornar direto, sem o terneiro.
2. Pode usar direto o `Math.random()`. Ele gera um semi-randômico entre 100 números (0 a 99). if (Math.random() < 0.2) { console.log('deu azar!'); }


nice work, David!


cool :)


cache no seletor!


We usually put the template result in a `-container` element


Giovani vai tomar um :coffee:  e depois arruma estes itens observados!


Can you apply `modal-body` to this div and get rid of one layer of divs?


:+1:  pro String.format!


Тут тоже явно нужно?


I think this is the bug in the sinking velocity depth average (where the gravity used to be wrong).


Hey @weboAp thanks, and thanks for creating this port to Laravel! We're using an environment variable for the cache-busting string, though it's conceivable could we use a random number instead as you've done. The more important change in this fork for us was keeping the function signatures the same as the original Carabiner library. I didn't want to manually edit all the function calls in our existing codebase. Is that in your plans for v3.0?


あｓｓｓ


`btn-primary` can be removed here


Was the previous behavior to prepend an updated user to the beginning of the list?


yeah. I like the block to indicate logical grouping.


1836664-ReturnWaterTempReset (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1665 of 1665 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)


@graemerocher Thanks for the feedback.  I pushed up a better fix (https://github.com/grails/grails-core/commit/ea16781da049c0b1236eef945dba4cdf8f918888).  Please let me know.


I could have sworn that when I looked at this, it said `if (values_out[0] == 0.0)` with a fixed index `[0]`. Apparently I'm wrong -- forget my comment then.


这怎么是乱码


awesome job, Andrew – nice work on Carrierwave!


Ah, that makes sense. The velocity is 1e16, so the average sinking velocity of 1e-9 clearly made no sense.

Are all the big differences below for the sinking velocity?


1836664-ReturnWaterTempReset (mjwitte) - 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)


このメソッドが怪しい気がします。NSLogで0の時の計算を試してみては？


I'd love to merge this, but doesn't this change break everything before 11.2? Or is this a change we should have done earlier?


The location is also used further down when doing the daily videos - couldn't use a relative path.


りょうかいです！


Missed that. Please ignore my ramblings.


LabelTextのLは大文字じゃなくて小文字の方がいいよ〜〜


細かいけどここも、numberBtnみたいな感じの方が読みやすいかも~~


woooooooooooooooowwwwwwwwwww


I'm trying to get a link in the Post show view to redirect to the summary.  I keep getting different errors, but I'm pretty sure they all come back to this line.  I'm trying to create an instance variable of summary to use in the the post show view here, but it isn't working.  In the Rails console, I tried `Summary.column_names`, and post_id is not listed, even though I have already created a migration `AddPostToSummaries`.  I tried the same test with Post to see if it included topic_id, and it does.  Did I associate these backwards? Or did something just go wrong on my migration?


This is an easy solution and fixes a bug that has affected me as well. Thanks!


@jsho32 Yes you did :smiley:


You don't need to pass browser as parameter.
You are getting the value line #3


+2. Files


Did you mean to remove this line?


Why 0755 instead of +x?


comment


comment 2


@mattdrayer  I wasn't able to take a look until late this evening, and I'm running into the devstack issue others were seeing. Do you have an issue building this locally? If this is rebased to the latest master, I think now with 8172 merged (https://github.com/edx/edx-platform/pull/8172) devstack should work again. I didn't want to rebase myself in case there was something else going on. I can work on this tomorrow morning to focus on unblocking this work. Thanks for the note.


一覧の際にはgetResultDataの方でURLをセットしてあげる必要があります。


Yeah, it's a tab for code formatting, the yml bit doesn't do anything.


Because that's how all init.d scripts should be. +x just fixes whatever is already set. All of them *should* be 0755.


It does syntax highlighting for some stuff (though I guess YML highlighting doesn't matter too much *shrug*)


That seems like the choice of the administrator of the server. Is there a reason just +x wouldn't work? I have a slight preference for keeping it +x, since that's the convention of what people write to say "make it executable"


It does, provided you use a markdown to HTML formatter that supports it, but we only use Github for deploynaut docs.


Se le añadió funcionalidad a botón de logout. Esto para saldar el issue https://github.com/CamilaAlvarez/IndicadoresDCC/issues/10 .


Hi @eldy,

Sorry, I don't view this modification. I will propose an another PR because a lot of link still linked to this page.


That's not really a preference. On Debian all of them put there by packages are `0755`. It's not that major, am willing to leave it at `+x` if it makes you happy :)


Yeah, makes me happy*er* :D


ok


Squashed and pushed :)


With this, there is a little less confusion with running the examples.


hard-coding 형태라면 값을 변경할 때마다 재컴파일 및 재배포를 해야할 것 같습니다.

timeout에 해당하는 값을 외부 프로퍼티 파일로 관리하는 방법이 어떨까 하는 생각이 듭니다.


テストコミットは製品にマージされてしまうと問題があるので差し戻しておいてください。


This `code` binding will have a value that is not used anywhere, which means this commit probably broke source mapping?


Progresses towards #112


+=item B<-h, --help>


+feed it "foo.MSG" and "foo.msg", you'll end up with one "foo.eml",


Thanks!

Sent from my iPhone

> On May 26, 2015, at 8:28 PM, Daniel, Dao Quang Minh <notifications@github.com> wrote:
>
> @mrunalp i updated and amended this commit.
>
> —
> Reply to this email directly or view it on GitHub.
>


is the trailing underscore to indicate that it's private?


@idokleinman,

is there a need for 2 separate copies of **iOS SDK** content for core and photon?

Are there differences we should know with regards to the SDK?


Please return HDPI sources.. . There are many HDPI devices still and we want to build again..


Please use a variable to keep the result of `listStatus` and reuse it below. `listStatus` results in a RPC call to the namenode, so it is pretty costly.


LGTM other than the comment on `listStatus`.


1836664-ReturnWaterTempReset (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (635 of 635 tests passed)

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


Sorry, I should have merged the manpage instead of rebasing before the pull request... this messed up all the git history of last month :(


try the dev-master now, set the browserCacheClear in the config to
env(build_number)  then activate the cachebust => true for individual
assets.
let me know.

On Wed, May 27, 2015 at 12:10 AM, Richard K Miller <notifications@github.com
> wrote:

> Hey @weboAp <https://github.com/weboAp> thanks, and thanks for creating
> this port to Laravel! We're using an environment variable for the
> cache-busting string, though it's conceivable could we use a random number
> instead as you've done. The more important change in this fork for us was
> keeping the function signatures the same as the original Carabiner library.
> I didn't want to manually edit all the function calls in our existing
> codebase. Is that in your plans for v3.0?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mocavo/lar-carabiner/commit/7c500ae3d209994f32d7f63a4d6f7af618ed77fd#commitcomment-11377780>
> .
>


It took me a few reads to understand the difference between ```samePositionAs``` and ```sameSituationAs```.
Can we change this to "sameBoardStateAs" or "sameGameStateAs"?


Gotta reverse these suckers


Haven't been able to work out what lock(?) is getting corrupted so far. I tried commenting out things little by little, but haven't been able to chase into a corner yet. Only the offline context. I tried forcing the various threads to run at least once (like the graph update, and HRTF loader), but so far no dice. The demos all seems fine with the default context...


Yep! Conceptually, this is trying to create an erased type (expressed in Scala's pre-erasure type system rather than the JVM's) for use with isInstanceOf (which is why it also breaks with value classes).


readme


def correr


maybe assign `u"\x1e"` to a variable that tells people what it is? also because it appears in several spots -- sort of "magic number"-ish...


so this is initializing the vector to size `block_size * num_channels` with each element set to `amp_t{0}`? why the trailing `{}`?


Bug fixed here:

https://github.com/samseaver/KBaseFBAModeling/commit/4b064ace8dc6618929f4864a053ad855991de7e5


1836664-ReturnWaterTempReset (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1665 of 1665 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)


Can we change to ``boardStateOccursTwice```?


is this an inline constructor definition?


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

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


commite comment


we should declare each parameter, for example the right docstring would be:
"""A method to compute displacement rate. <A one line description of what this function does in general>

The displacement rate will be 1 if mmi >= displacement_threshold. Otherwise, it will be set to 0. <A more comprehensive description on how this function works. This can be more than one line>

:param mmi: The mmi value.
:type mmi: int

:returns: The displacement rate given an MMI value.
:rtype: int
"""


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


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


Now there are  two `main` properties.


Nothing came up in intellij >.<


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


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


We're planning on implementing a "plan" window at some point with undo/redo?
Do we know whether that functionality is going to live on the client or the server?
If it lives on the server, we'll need a way to unmake a move and remove it from the game history.


Changed indentations as requested. fixes #4


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


TeamCity xBIM :: XbimScripting :: Build [Build 18](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1752&buildTypeId=XBIM_XbimScripting_Build) is now running


TeamCity xBIM :: XbimScripting :: Build [Build 18](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1752&buildTypeId=XBIM_XbimScripting_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:37


TeamCity xBIM :: Xbim Essentials :: Release [Build 50](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1753&buildTypeId=XBIM_XbimEssentials_Release) is now running


TeamCity xBIM :: Xbim Essentials :: Release [Build 51](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1754&buildTypeId=XBIM_XbimEssentials_Release) is now running


TeamCity xBIM :: Xbim Essentials :: Release [Build 50](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1753&buildTypeId=XBIM_XbimEssentials_Release) outcome was **SUCCESS**
Summary: Running Build time: 00:00:44


Note that mb_strtolower() is very SLOW


aasd


TeamCity xBIM :: Xbim Essentials :: Release [Build 51](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1754&buildTypeId=XBIM_XbimEssentials_Release) outcome was **SUCCESS**
Summary: Running Build time: 00:00:41


Chris,

I tried this code on a mixed language design.
And I'm see some errors.

If I iterate over a top level verilog module with vhdl instances, I will eventually get a vhpiCompInstStmtK type object being returned by next_handle.

Which then causes an error when we call  vpi_get_str(vpiFullName, obj) , since the vhpiCompInstStmtK object has no name.
I get this error in ius:
```
ERROR:          VPI       NONAME
         Object of type:  vhpiCompInstStmtK does not have a name.
ncsim: *F,INTERR: INTERNAL EXCEPTION
```

Seems like next_handle would need to figure out whether the object it gets from the iterator is verilog or vhdl and act accordingly.

Luke


Does calling `reverse` work here?  scopes are database queries that should be able to be chained together.  In order for that to happen, they need to return an `ActiveRecord::Relation` result.  But I think the `reverse` method will just return a standard `Array` which can't be chained with another scope.


I am trying to learn JavaScript among other languages


https://shuge.org/ebook/yanben/

@klamtlne  请叫我雷锋


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I was thinking about this more, and do you think that this bookkeeping being something the client does? Eg we send back `Success` and then it waits until it's seen that log committed (since in order to have Qourum the log must be commited) before responding to the caller? This would prevent the replica from needing to remember or keep books regarding such a thing. I'm concerned there might be a subtle flaw in this approach though.

Otherwise I could see this as being a state that the leader keeps akin to `match_index` and `next_index` perhaps?


remove the 'per day' here


Place a period after year


remove 'main'


comma after 'Checked'


looks good!


that needs to be The Chest Has Been Broken :)


1836664-ReturnWaterTempReset (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1665 of 1665 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)


Fixes #26


what does this line do?


@kmike Like `@command(lua_property=True)`?


@georg90 Should be fixed in v0.8.70 (and its `resetTime`).


I also disabled autoupdate in order to make this stick on the server itself; it would get immediately overwritten with the release version if I had kept it as-is.


@snowtimeglass
This commit broke the links... I fixed it in 06d8bfb.
Could you please also add a translation for the following text which I added in 06d8bfb?

AnkiDroid (the Android version of Anki) is created and managed independently from other Anki versions. For the PC / Web / iOS versions, and general Anki issues that are not limited to Android, please visit the main https://anki.tenderapp.com/[Anki support site].


Ingreso nuevamente el codigo hoy dia el git hub me lo subio con otra cuenta github mia la cual es chelo1988 me olvide la configuracion de git bash y me hiso push con mi otra cuenta por llo que en esta lo subi nuevamente para verificar si los archivos se subieron correctamente


OWTF wants to help penetration testers use their time most effectively, even if they don’t use OWTF directly. An unfortunate reality of penetration testing is the amount of time that is gone via reporting. Explaining vulnerabilities to non technical customers is difficult. Conveying the urgency of fixing XSS, CSRF and many other issues tends to be non-trivial. Especially when the overall security background from the customer is poor (which is common).

This project aims to:

Provide boilerplate vulnerability explanations which can easily be copy-pasted into real-world reports
Provide boilerplate vulnerability fixing recommendations which can easily be copy-pasted into real-world reports


`doc/html/assets/config.json` is required?


What kind of build issues do you have with this commit?


let me see if i'm read this right: `kernel_t` is an anonymous function that returns a `block_v` and takes an optional `args_v`?/


When we implement the database we're going to store games in a single canonical notation, right?


Maybe not, but since a teacher complained that old classes are show, I just wanted a simple way of verifying. Anything else takes a lot more time :=)


what does the `const` between the parameter list and method body do?


Hi


This is going to be used for Kindergarten classes, since they have a locaction, an address and are located on a map. Kindergarten classes have 3 pages: home, calendar, location.


delete "some"


just merge


y esto? e,e


good job! looks like the class 'DtoMappings' is obsoleted.


I just did this as a placeholder. Since you told me that this method needs to be implemented with 'file' caching on and thinking I'd implement it when I know what to do or with you together


Good ridance. Our static analyzer was going nuts because of how improvised stb_image was working.


@ace2nutzer how do you compile the kernel and flash to boot? I'm using ubuntu 15.04 64bit, I have downloaded lk 4.1 source but I don t know how to proceed, please give me some hints


Related to skalfa/workflow#67


I would prefer this call to be also in the wicket-debugbar.js

http://stackoverflow.com/questions/799981/document-ready-equivalent-without-jquery


done!!!


This should be done without jQuery, because it is not required to load the jQuery library at all.


very good comment!
간단하게는
Torpedo.properties 에
... property를 추가하고
idle_timeout_default = 33
hard_timeout_default = 33
priority_default = 33

아래와 같이 값을 초기화부분에서 세팅해서 사용하시면 됩니다.
물론 final 키워드는 제거하시고요.
public OFMLearningMac() {

	TorpedoProperties sysconf = TorpedoProperties.loadConfiguration();
	IDLE_TIMEOUT_DEFAULT = (short) sysconf.getInt("idle_timeout_default");
	HARD_TIMEOUT_DEFAULT = (short) sysconf.getInt("hard_timeout_default");
	PRIORITY_DEFAULT = (short) sysconf.getInt("priority_default");
}

하드코딩으로 당장 불편하시면 제가 알려드린 방법으로 사용하시면 되고요.
MacLearning뿐만아니라 staticEntryPusher와 Forwarding까지 통합해서 관리하도록 수정된 버전은 IRIS 멤버들과 상의해서 적용하도록 하겠습니다.


Adds progress to #1, #2


Cool. Thank you.


I have another problem，the volume of speaker on calls is less than that on official rom，Thks。


Not to be picky, but that code really looks as if coming from the Flow codebase (variable names, comments), so the copyright and the full header as such are wrong. Leave out the copyright notice, stay LGPL. Feel free to prove me wrong.


There is one for this.


I love how this function is identical to `_logWarning` except that one of the keywords isn't actually in the string.


The issue with keeping the api as it's is the flexibility of adding more flags... Per assets , processing them... Redundant code... .
It's doable ... Will give it a shot sometime. Or if you think it's important create a pull and I can incorporate it
Thanks for your interest.


Presudo?


I am not sure why i just realized this now, but this script and the popup should be moved to interface/patient_tracker/ directory


Only for the information: Currently there is a discussion about to drop the debug bar at all:

http://apache-wicket.1842946.n4.nabble.com/Wicket-Ajax-debug-window-usefulness-td4670364.html

All in all this change looks good to me, but there should be another reviewer except me.

@martin-g : Would you be so kind and have a look at it?


This is a placeholder for myself to get encounter working and to mimick behavior of patient lists gui.


This should be two steps - enterConfirmDeleteText and clickConfirmDelete.  That way the other case (incorrect project name entered) can be tested.


:+1:  Don't stop! :)


How would the client know when the entry was committed?


1836664-ReturnWaterTempReset (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1151 of 1151 tests passed)

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


@klopfdreh In this thread I meant https://github.com/apache/wicket/blob/549827f9fe41be3611ba9ac84e0db507c7e1e149/wicket-core/src/main/java/org/apache/wicket/ajax/res/js/wicket-ajax-jquery-debug.js, not wicket-devutils.


Also should have a toggle at the bottom of the tracker to choose this if user wants (same behavior as Patient/client->Patients with the Open in New Window Toggle(note since it is a self-refreshing script, need to ensure bring over the selected setting when the script refreshes which can do with a GET parameter)).


bradymiller TODO
This is a placeholder for myself to get encounter working and to mimick behavior of patient lists gui.


* I like the namespace imports, we are doing that throughout Flow as well by now
* any reason to rename ``Tests`` and ``Classes`` - other than "it's another way to do it"?
* Even if automatic DI might not be possible (as easily), it would be nice to keep the flexibility and use "manual" injection instead of ``new``. Eases testing and swapping out of components.

Didn't try it out, but from the looks of it, it probably works!? :+1:


Gilles,

I'm afraid I don't have access to a machine where I can test your patch.
But I am not sure about it's correctness in the case where the use of the
optimized description allows for predefined datatypes to be cut in the
middle during the pack. I do remember that I added a lot of code to handle
this case (opal_unpack_partial_datatype), but I am not sure that it still
works as expected. The current assumption is that it is safer in the
heterogeneous case to just fall back to a more conservative approach and
make sure that no predefined types are split.

And this brings me to a possible solution as your problem reminds me of an
old issue where the PML code did not take in account the amount returned by
pack and set_position. As a result, the upper level assumed more data than
expected has been packed, while the convertor was unable to operate on the
expected length because it is not allowed to cut a predefined datatype in
the middle.

  George.

On Mon, May 25, 2015 at 4:36 AM, Gilles Gouaillardet <
notifications@github.com> wrote:

> so i clearly misunderstood your prior email, sorry about that.
>
> the issue i am trying to solve is an assertion error, that might follow a
> ddt error
>
> [1,1]<stddiag>:[kusnode45:02263] ../../../../src/ompi-master/opal/datatype/opal_datatype_pack.c:236
> [1,1]<stddiag>: Pointer 0xfffff80100807d0c size 4 is outside [0xfffff80100800010,0xfffff80100807d0c] for
> [1,1]<stddiag>: base ptr 0xfffff80100800010 count 2000 and data
> [1,1]<stddiag>:[kusnode45:02263] Datatype 0x20001710[MPI_LONG_INT] size 12 align 8 id 0 length 3 used 2
> [1,1]<stddiag>:true_lb 0 true_ub 12 (true_extent 12) lb 0 ub 16 (extent 16)
> [1,1]<stddiag>:nbElems 2 loops 0 flags 5314 (committed contiguous )-cC----GD--[---][---]
> [1,1]<stddiag>:   contain OPAL_INT4 OPAL_INT8
> [1,1]<stddiag>:--C---P-D--[---][---]      OPAL_INT8 count 1 disp 0x0 (0) extent 8 (size 8)
> [1,1]<stddiag>:--C---P-D--[---][---]      OPAL_INT4 count 1 disp 0x8 (8) extent 4 (size 4)
> [1,1]<stddiag>:-------G---[---][---]  OPAL_END_LOOP prev 2 elements first elem displacement 0 size of data 12
> [1,1]<stddiag>:Optimized description
> [1,1]<stddiag>:-cC---P-DB-[---][---]     OPAL_UINT1 count 12 disp 0x0 (0) extent 1 (size 12)
> [1,1]<stddiag>:-------G---[---][---]  OPAL_END_LOOP prev 1 elements first elem displacement 0 size of data 12
> [1,1]<stddiag>:
> [1,1]<stderr>:reduce_loc: ../../../../src/ompi-master/opal/datatype/opal_datatype_pack.c:139: opal_pack_homogeneous_contig_with_gaps: Assertion `i == ((uint32_t)(pConv->bConverted / pData->size))' failed.
>
> this only occurs between two nodes with different arch.
>
> a trivial way to reproduce it is to MPI_Send(buf, 1000000, MPI_LONG_INT)
> the collective/reduce_loc test from the mpi test suite can also be used to
> trigger the issue.
>
> what happens is opal_pack_homogeneous_contig_with_gaps is using the
> vanilla description and not the optimized description. the vanilla
> description is made of more that one kind of type and it seems
> opal_pack_homogeneous_contig_with_gaps was not designed to handle this.
>
> could the following patch fix this ?
> (when sending in hetero mode, use the opt_desc if the type is contiguous)
>
> diff --git a/opal/datatype/opal_convertor.c b/opal/datatype/opal_convertor.c
> index 9a211ba..68aeb27 100644--- a/opal/datatype/opal_convertor.c+++ b/opal/datatype/opal_convertor.c@@ -12,7 +12,7 @@
>   *                         All rights reserved.
>   * Copyright (c) 2009      Oak Ridge National Labs.  All rights reserved.
>   * Copyright (c) 2011      NVIDIA Corporation.  All rights reserved.- * Copyright (c) 2013      Research Organization for Information Science + * Copyright (c) 2013-2015 Research Organization for Information Science
>   *                         and Technology (RIST). All rights reserved.
>   * $COPYRIGHT$
>   *@@ -455,7 +455,7 @@ int32_t opal_convertor_set_position_nocheck( opal_convertor_t* convertor,
>   * Compute the remote size.
>   */
>  #if OPAL_ENABLE_HETEROGENEOUS_SUPPORT-#define OPAL_CONVERTOR_COMPUTE_REMOTE_SIZE(convertor, datatype, bdt_mask) \+#define OPAL_CONVERTOR_COMPUTE_REMOTE_SIZE(convertor, datatype, bdt_mask, send) \
>  {                                                                         \
>      if( OPAL_UNLIKELY(0 != (bdt_mask)) ) {                                \
>          opal_convertor_master_t* master;                                  \@@ -472,11 +472,12 @@ int32_t opal_convertor_set_position_nocheck( opal_convertor_t* convertor,
>              }                                                             \
>          }                                                                 \
>          convertor->remote_size *= convertor->count;                       \-        convertor->use_desc = &(datatype->desc);                          \+        if (!(send && convertor->flags & OPAL_DATATYPE_FLAG_CONTIGUOUS))  \+            convertor->use_desc = &(datatype->desc);                      \
>      }                                                                     \
>  }
>  #else-#define OPAL_CONVERTOR_COMPUTE_REMOTE_SIZE(convertor, datatype, bdt_mask) \+#define OPAL_CONVERTOR_COMPUTE_REMOTE_SIZE(convertor, datatype, bdt_mask, send) \
>      assert(0 == (bdt_mask))
>  #endif  /* OPAL_ENABLE_HETEROGENEOUS_SUPPORT */
> @@ -487,7 +488,7 @@ int32_t opal_convertor_set_position_nocheck( opal_convertor_t* convertor,
>   * here that the convertor is clean, either never initialized or already
>   * cleaned.
>   */-#define OPAL_CONVERTOR_PREPARE( convertor, datatype, count, pUserBuf )  \+#define OPAL_CONVERTOR_PREPARE( convertor, datatype, count, pUserBuf, send )  \
>      {                                                                   \
>          uint32_t bdt_mask;                                              \
>                                                                          \@@ -527,7 +528,7 @@ int32_t opal_convertor_set_position_nocheck( opal_convertor_t* convertor,
>                                                                          \
>          bdt_mask = datatype->bdt_used & convertor->master->hetero_mask; \
>          OPAL_CONVERTOR_COMPUTE_REMOTE_SIZE( convertor, datatype,        \-                                            bdt_mask );                 \+                                            bdt_mask, send );           \
>          assert( NULL != convertor->use_desc->desc );                    \
>          /* For predefined datatypes (contiguous) do nothing more */     \
>          /* if checksum is enabled then always continue */               \@@ -563,7 +564,7 @@ int32_t opal_convertor_prepare_for_recv( opal_convertor_t* convertor,
>      mca_cuda_convertor_init(convertor, pUserBuf);
>  #endif
> -    OPAL_CONVERTOR_PREPARE( convertor, datatype, count, pUserBuf );+    OPAL_CONVERTOR_PREPARE( convertor, datatype, count, pUserBuf, 0 );
>
>      if( convertor->flags & CONVERTOR_WITH_CHECKSUM ) {
>  #if OPAL_ENABLE_HETEROGENEOUS_SUPPORT@@ -602,7 +603,7 @@ int32_t opal_convertor_prepare_for_send( opal_convertor_t* convertor,
>      mca_cuda_convertor_init(convertor, pUserBuf);
>  #endif
> -    OPAL_CONVERTOR_PREPARE( convertor, datatype, count, pUserBuf );+    OPAL_CONVERTOR_PREPARE( convertor, datatype, count, pUserBuf, 1 );
>
>      if( convertor->flags & CONVERTOR_WITH_CHECKSUM ) {
>          if( datatype->flags & OPAL_DATATYPE_FLAG_CONTIGUOUS ) {
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/open-mpi/ompi/commit/e47cb9636d9fc12a95279abb699ddfd1a6ca644e#commitcomment-11347319>
> .
>


ohhh i can put a comment in my pull request on this line


and one on this line


@emacarron Yeah, it is very good ! Thank you !


@martin-g : I think this branch is ready to be integrated. All issues are fixed and streams are closed - what do you think?


New to udidio-example happy awesome A++++


Ok Cool!!


With this comment gone there's no indication that the relevant fields are being ignored because we just haven't bothered to do it yet, as opposed to there being some significant reason.


Change to(so make it clear and drop unneeded parameters):
fetch_patient_tracker_events($from_date, $to_date)


Do we need to ignore this? Rect<> will always have a heap size of 0, but should we define that so Rect<> field don't need ignoring?


Same story for Rect<> here.


Oops, forgot to re-add the comment it. Will do.


Similar story for BorderRadii<> here.


As mentioned in the PR, I have yet to add zero-heapsize impls for all the geom stuff.


salaaaaaaaaaaaam


This is bad!


leave this!


all credit goes to @dtognazzini for creating this sweet list :pray:


r=me.


Add --update?


typo: distingues -> distinguish


Can we merge this?


The comment isn't gone, it was moved to where the #derive is


Isn't the reason that the mapper service doesn't learn about new types all at once, so the order may be backwards, which could create a false error?


This caught me out; makes sense; but double change as you didn't need to specify any of the default identifiers in RawShader before, if you used them e.g. ```position```, ```uv```. Worth a highlight in release notes.


I believe this should be handled in `frontend/index.js`. Would prefer keeping `render` file pure with no side effects so it's less magical and does not require any modifications! In real life, there's usually a middleware attaching `req.user` (e.g. `jwt auth`) and that's why I would prefer exposing `loadUserState` as a middleware, to show how to use it with other middlewares :)


i think i understand what's going on here, but line 254 is a little unclear. does `[this]` on that line just set the return type to the instance of `ugen_graph_t`?


I would rather be explicit here than rely on the default. So instead of passing null, pass the value of version.onOrAfter(2.0)?


Isn't this token supposed to be:
pt_tracker_id


I think fixing this will break things. Can this if go away?


On a seulement besoin du flux ostream


/////////helllllo//////


afarin be in code


Do we need both #[heapsize] and #[derive(HeapSizeOf)]? I think just the latter would suffice.


Why create branches for different versions? Why not have only "1.8" and "1.7.10"?


is this allocating a new vector to hold the ugens every time this method is called?


Nah -- I was using `heapsize` first and it occurred to me that we could hook directly into `derive()` but I didn't remove the first one.


implicit type coercion?


This code should not have been changed in this commit?


Because 0.1.1 is just a hotfix, all the new features from the master and 1.8 branch will release in 0.2.0


I assume the plug-in handles enums like this appropriately...


95077770-HVACTemplateLoadManage-4493 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: 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)


Yep.


bradymiller TODO
This is a placeholder for myself to get encounter working and to mimick behavior of patient lists gui.


Why put all the routes in the helper block?


For 7.x  I'd recommend renaming the file (path) to: src/main/resources/META-INF/wicket/com.googlecode.wicket.kendo.ui.properties
See https://issues.apache.org/jira/browse/WICKET-5713


avarin


I included these changes in the next commit, so will unroll this one.


Ok.


LGTM


I am not sure why i just realized this now, but this script and the above script should be moved to interface/patient_tracker/ directory


I squash them to one commit and push them to master this evening.


Thanks for the fix, even if I expected the M(O) :(
I have no major problem with your decission. Is is just a documentation problem.


95077770-HVACTemplateLoadManage-4493 (mjwitte) - 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)


Not necessarily. Tuning the unusued values should not affect the used ones because each value is individually tuned. It just looks wrong.


I like this, its applicable eg. in a chat module


when bring in script via a library best to use absolute paths:
require_once(dirname(__FILE__) . '/appointments.inc.php');


Great thanks!


I haven't tested on windows, but I suspect that as of 6d3dbb83 this is no longer needed, since the noParse path regex is now platform-appropriate (backslashes on windows)


Did you forget to remove the TODO before pushing or you will do it before the release?


We are trying to do another release this year. All that is missing is a start and an end sceen and some maps.


You forgot to fix this also in `test-package.sh` :-)


Why did you add the composer.lock file?


a comment ehrer


@tamaspap,
maybe you need to see this pen http://codepen.io/qivhou/pen/oXYymM

the issue is caused that you bind the eventHandlers on prototype, but the prpperty on prototype couldn't be deleted by instance.

It's better to read following https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/delete

`
If the object inherits a property from a prototype, and doesn't have the property itself, the property can't be deleted by referencing the object. You can, however, delete it directly on the prototype.
`


I will add a comment. I added it as `Bool?` instead of `Bool` purely for convenience – as `delegate` in both `BrowserLocationView` and `URLBarView` is optional, calling `delegate?.browserLocationViewDidPressReaderMode` will return `Bool?` even when the method itself returns `Bool`. So if it all those methods returned `Bool`, I would need to add `?? false` at the end to handle case where `delegate == nil`.

But now I realize that is not clean as it opens the question "and when do I return `nil` instead of `false`?", so I will fix these methods to return `Bool`.


To know what is will builded into phar.


What do you mean ?
Hydro is the last version of ROS that we can use on the PR2, we need it.


Ok. I had forgotten that I can find out the year of creation of the file from the history.


Blegh, thanks :D


Great Job man, looks awesome :+1:


@wva Oh haha, yes, I meant @vanatteveldt. Sorry!


在英语表述中省略了`已知大小“的返回值”`这样的表述，但是翻译为中文之后，`的返回值`是必须的补充。不需要加括号。
翻译的原意还是让读者读懂，并不需要完全按照原文。


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


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


Iirc we decided against dev/Sound.cpp. Can we remove it?


Just curious why the conditional for Jetpack was removed?


lgtm - sorry for delay


>Lol. git thinks that you renamed torrentcreatorthread.h. This will be bad if anyone tries to find the history of each file... Anyway I don't think you can do something in this case.

git often does so. Therefore, I have divided these changes into two commits (I delete some files in the following). Otherwise such things would be more.


Please check how it works now.
As I remember 'Exec' field stores not only program name with command line but also may be complex shell command (I not remember now, but I think that I sow this in some .desktop files). It may contain pipeline, STDI/STDERR redirection, etc.
To handle that I used 'system()' function with this strange (at first look) fork/system mechanism.
We can use ::StartDetached() but for 100% compatibility we should call comething like /bin/sh -c "%command%". But in this case we should make some magic with quotes in command.


Like the shadows, I still need to add amphibious model damage. Right now I am using pontoon damage for the amphibious damage.

Also, the damaged pontoons looked darker than the regular pontoons until I added bumpspec for them. Using fuselage-normal.png for the pontoons.

I haven't checked any of this on multi-player.


Added names to the complexity and descriptions with examples


use formTrim() here

(this is to avoid confusion when convert this script in future to "new" security model)


This introduces an XSS vulnerability, unfortunately. If code.label contains an `<script>` tag, it will be executed.


remove formTrim use here


As a workaround, you *could* rename your attribute to have a prefix, like `activityType`. That will solve your issues faster than demanding a fix.


ReadBits = 10


Oh I just realised you removed the ignore-rule, not added it. :)
My mistake. :)


в VecTools обычно метод возвращает исправленную копию вектора, а тут сам вектор правится.
Поэтому не уверен, что лучше в VecTools


All good


“Thanks! Could you add a line about how this could be achieved?”


I copy what I told Stefan (adding that I looked only for 10 minutes and am in no way expert in using that Sqlite Swift wrapper library, and that I was trying to compare the error code with `SQLITE_CONSTRAINT`): I can't seem to find a way to get at the numeric sqlite error code: `Database` has public `String` `lastError`, but no sqlite error code. `Statement` has private `status`, which is exactly what I would need, but it is private.


You can use `hide()` instead of `css("display", "none")`, and in reverse `show`. (I you copied it, just a general tip.)


remove this comment


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


Again, I know you copied it so general tip: 'd' is never initialized with the 'var' keyword, so it ends up in the global scope.


only do below if tracker_id is set


fine with that


set it to false be default:
``````
function fetchEvents( $from_date, $to_date, $where_param = null, $orderby_param = null, $tracker_board = false )


place the $tracker_fields = ''; and $tracker_joins = ''; here to make it more clear.


nice!


You got to love nested zsh expansions — readability was never a goal,
just short sequences to type at a terminal's prompt. ☺

Anyway, I don't think that (z) is the right thing to use here, since it uses
zsh's undestanding of what a word is, not really just split at whitespace.
I think the right thing™ is to do something along the lines of what is
done with the /etc/host handling; i.e. split with spaces and tabs.


Bulk updates in a single commit are very difficult to browse and provide feedback on. Below are problems noticed in a quick overview
1- Elco and HH entities: All field names and getters, setters are Uppercase. This is not standard java naming practice
2- RegisterControllerTest and FormDownloadTest are commented out. Can we try resolving if there is any issue instead of commenting out. Or if not possible without massive effort please make a note of it in release notes.
3- No release notes file found which can describe about changes made into this commit
4- Infact I can see alot of tests commented out. These should be either resolved or mentioned somewhere with reason about the commented out thing
5- FakeReportController has been commented out... !!??
6- Raihan has sent new form files so plz make sure to get latest files and test with those


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


TeamCity YaccConstructor :: YC.Core General Build [Build 389](http://teamcity.codebetter.com/viewLog.html?buildId=192072&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Building ..\src\YC.SDK.sln failed with exitcode 1. Build time: 00:10:33


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


Doesn't it make more sense to just display the map on the portal?

> On 27.05.2015, at 07:37, Jürg Messmer <notifications@github.com> wrote:
>
> This is going to be used for Kindergarten classes, since they have a locaction, an address and are located on a map. Kindergarten classes have 3 pages: home, calendar, location.
>
> —
> Reply to this email directly or view it on GitHub.
>


Set it to false by default:
``````
function fetchAppointments( $from_date, $to_date, $patient_id = null, $provider_id = null, $facility_id = null, $pc_appstatus = null, $with_out_provider = null, $with_out_facility = null, $pc_catid = null, $tracker_board = false )


window.startEventLoop 不可以拿掉喔，那是刻意要模擬 GUI toolkit event loop 行為的


TeamCity YaccConstructor :: YC.Core General Build [Build 388](http://teamcity.codebetter.com/viewLog.html?buildId=192071&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: NUnit test failed (1).   at Fake.NUnitSequential.NUnit(FSharpFunc`2 setParams, IEnumerable`1 assemblies)   at FSI_0002.Core.commonConfig@178-12.Invoke(Unit arg10@) in Z:\Builds\work\4138ce7c6332a78c\tools\Build.Tools\Core.fsx:line 1... Build time: 00:11:14
### Failed tests
```
RNGLRAbstractParserTests+RNGLR abstract parser tests.temp: <no details avaliable>

```


Without some sort of TTL setup you're gonna be leaking leak memory pretty badly after a few deploys.


Ok, maybe not


I have updated this for you ddtdan


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


You may want to consider using a more robust approach here. Just in case :)
https://github.com/Modernizr/Modernizr/blob/master/feature-detects/storage/localstorage.js


TeamCity YaccConstructor :: YC.Core General Build [Build 390](http://teamcity.codebetter.com/viewLog.html?buildId=192073&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: Error during Nuget update. ..\tools\Build.Tools\NuGet\NuGet.exe install "Z:\Builds\work\4138ce7c6332a78c\src\FsYARD\packages.config" -OutputDirectory "..\src\packages" -ConfigFile "..\src\NuGet.config"   at FSI_0002.Packaging.restor... Build time: 00:03:29


I'm not really liking the explode on the smg. Not only is not what you'd expect from that weapon, but because of the way smg particles taper out, it causes explosions in midair with no associated particle making it look out of place. A change this dramatic should maybe be polled/discussed first.


Ok. I moved ```torrentcreatorthread.*``` deletion in next commit.
It will not save its history, but the history of bandwidthsheduler only.


So I was taking a look at this.

`export class DisplayObject extends EventEmitter implements interaction.InteractiveTarget, extras.cacheAsBitmap, extras.getChildByName, extras.getGlobalPosition`

This treats the extras as interfaces even though they are classes as described in the handbook.

`Error class DisplayObject incorrectly implements extras.cacheAsBitmap. Property _cacheAsBitmap is missing`

So it seems I need to inline the signature anyway.

I added:

`private _cacheAsBitmap: boolean;`

To satisfy the compiler, but then I get:

`Types have separate declarations of a private property _cacheAsBitmap`

I struggle with the private variables. I asked on SO and Ryan from the TS Team told me that private members should be added.  To prevent you extending DisplayObject and adding your own "_cacheAsBitmap" which is a nightmare waiting to happen behind the scenes.


Still not in PPA archive https://launchpad.net/~ubuntu-lxc/+archive/ubuntu/lxd-git-master
When are those rebuilt? I thought it is automatic process.


No worries @leafac :smile:


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


I think this fails when two or more jobs are waiting here and they'd all run parallel once the previous job finished


As discussed above, change to(so make it clear and drop unneeded parameters):
fetch_patient_tracker_events($from_date, $to_date)

Or use caps to separate words, whatever you prefer.


set to null instead:
``````
$provider_id = null;


hard-code $tracker_board parameter to true:
``````
$events = fetchAppointments( $from_date, $to_date, null, $provider_id, null, null, null, null, null, true );


Shouldn't this be `el.readOnly = !!value`?


I understand that it's more work, but I think it would be better for clients if they could safely pass around JObjects (for example).


@daton Hola profesor tengo  una pequeña duda sobre lo que dejo en la parte de modelo deberia ir el arraylist o en otra clase java y tambn debe autenticar o ya es diferente una variable tomara el valor escogido en el combo box y se comparara con el tipo de bebida, otra cosa profesor para cada tipo de bebida solo se necesita una clase o una por cada tipo


Better to place this comment above function.


Oui oui t'inquiète ;) c'était juste pour rigoler


Unfortunately, this commit breaks all of our existing internal cookbooks with the windows_task resource using this new pr as well as chef-client::task...

https://github.com/opscode-cookbooks/chef-client/issues/313


bytes_at_next_dot is defined twice here. The variable `r` might be uninitialized here. So better only put the variable declaration here and define a value below when r is defined as well.


The index multiplier as in `a[i] = vload((vtype*)&reload_state[i*16*VS64+0*VS64]);' is SHA_BUF_SIZ, yes. Actually I changed it - it was 32 before for SHA512. That was a bug (unless I'm very confused). Since reload_state is a long_long in this case, it will end up right as 16. Had reload_state been an int (like for SHA256) we'd be using 32.


Hmm, I guess there's no good way of getting at the documentation without the code right now. Though it does say quite explicitly in the docstring of `deriving_monoid<in_terms_of_plus<M>>` what the intent is:

	 * This particular strategy of implementing `monoid` requires the existance
	 * of an `operator+ (M, M) -> M`, and that the default constructor of `M`
	 * is the identity element with regards to this operation.

Is this not clear enough? If so, how can I improve it?

Also, I'm curious to know why you expected `M(0)`? I picked the default constructor to make sure e.g. `std::string` could be used, just like @splinterofchaos guessed.


@zoe-1 Me of course :) It's the same as using dependency **without** callback.


Reverted because not sure if 127.0.0.1 in /etc/resolv.conf is actually a security concern...


ここにはPG機能を簡単に書いてください


In order to keep this item contained recommend placing all pertinent code for it here:
``````
$yearly_limit = $GLOBALS['maximum_drug_test_yearly'];
$percentage = $GLOBALS['drug_testing_percentage'];


@osynge sorry Owen. i don't quite understand the change here.

- before this change:
  - for suse: we add "libexpat-devel, FastCGI-devel" to BuildRequires, and apache2-mod_fcgid to Requires
  - and for all other systems, we simply add "expat-devel" and "fcgi-devel" to BuildRequires
- after this change,
  - for suse: we add "libexpat-devel, FastCGI-devel" to BuildRequires, and apache2-mod_fcgid to Requires
  - for rhel, centos or fedora: we add add "expat-devel" and "fcgi-devel" to BuildRequires
  - otherwise, we do **nothing**.

so i think this commit changes nothing for ceph rpm packaging in suse, rhel, centos or fedora. but it removes the expat-devel, cgi-devel from BuildRequires when ceph is built on other systems where non of these macros are defined. and my question is why we need to remove them when building on those systems?


95077770-HVACTemplateLoadManage-4493 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (633 of 633 tests passed)

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


Bon dia @msanroman! Que tal?!
Estic barallant-me amb les pestanyes... Voldria aconseguir que des d'una altra pàgina de la web, poder posar un link a una pestanya concreta d'aquesta pàgina. Aquesta pàgina té 6 pestanyes i per defecte està oberta la de Sobre mi, però en alguns casos voldria que l'usuari entrés amb la pestanya Valoraciones desplegada en comptes de la de Sobre mi. I no sé com posar una url per indicar amb quina pestanya s'ha d'obrir.
He fet proves amb ancles (tot i que necessitaria que fos des d'una altra pàgina i no des de la mateixa) linkant-ho amb IDs, amb classes però no funciona.

Gràcies!


hai


I say this because there is a chance this may get migrated outside the manage function if more patient/group specific features are built in.


In case of .Append(<somthing that results in string.Empty>).AppendLine()
I think Append(object) should look like

public virtual IndentedStringBuilder Append([NotNull] object o)
{
    Check.NotNull(o, nameof(o));

    var value = o.ToString();
    if (value != string.Emtpy)
    {
        DoIndent();
        _stringBuilder.Append(value);
    }

    return this;
}


Hi ypu,
didn't fine ovmf file in the cmd

/bin/qemu-kvm \
    -S  \
    -name 'virt-tests-vm1'  \
    -sandbox off  \
    -M pc  \
    -nodefaults  \
    -vga std  \
    -chardev socket,id=qmp_id_qmpmonitor1,path=/tmp/monitor-qmpmonitor1-20150527-142208-YcoEoLfA,server,nowait \
    -mon chardev=qmp_id_qmpmonitor1,mode=control  \
    -chardev socket,id=serial_id_serial0,path=/tmp/serial-serial0-20150527-142208-YcoEoLfA,server,nowait \
    -device isa-serial,chardev=serial_id_serial0  \
    -chardev socket,id=seabioslog_id_20150527-142208-YcoEoLfA,path=/tmp/seabios-20150527-142208-YcoEoLfA,server,nowait \
    -device isa-debugcon,chardev=seabioslog_id_20150527-142208-YcoEoLfA,iobase=0x402 \
    -device ich9-usb-uhci1,id=usb1,bus=pci.0,addr=03 \
    -drive id=drive_image1,if=none,cache=none,snapshot=off,aio=native,file=/root/patch_review/autotest-devel/client/tests/virt/shared/data/images/win7-64-sp1-virtio.qcow2 \
    -device virtio-blk-pci,id=image1,drive=drive_image1,bootindex=0,bus=pci.0,addr=04 \
    -device virtio-net-pci,mac=9a:37:38:39:3a:3b,id=idYrHZz8,vectors=4,netdev=idzC2C8B,bus=pci.0,addr=05  \
    -netdev tap,id=idzC2C8B,vhost=on,vhostfd=23,fd=22  \
    -m 4096  \
    -smp 1,maxcpus=1,cores=1,threads=1,sockets=1  \
    -cpu 'Opteron_G2',hv_relaxed,+kvm_pv_unhalt,hv_relaxed,hv_spinlocks=0x1fff,hv_vapic,hv_time \
    -drive id=drive_cd1,if=none,snapshot=off,aio=native,media=cdrom,file=/root/patch_review/autotest-devel/client/tests/virt/shared/data/isos/ISO/Windows7/en_windows_7_ultimate_with_sp1_x64_dvd_u_677332.iso \
    -device ide-cd,id=cd1,drive=drive_cd1,bootindex=1,bus=ide.0,unit=0 \
    -drive id=drive_winutils,if=none,snapshot=off,aio=native,media=cdrom,file=/root/patch_review/autotest-devel/client/tests/virt/shared/data/isos/windows/winutils.iso \
    -device ide-cd,id=winutils,drive=drive_winutils,bootindex=2,bus=ide.0,unit=1 \
    -drive id=drive_unattended,if=none,snapshot=off,aio=native,media=cdrom,file=/root/patch_review/autotest-devel/client/tests/virt/shared/data/images/win7-64-sp1/autounattend.iso \
    -device ide-cd,id=unattended,drive=drive_unattended,bootindex=3,bus=ide.1,unit=0 \
    -device usb-tablet,id=usb-tablet1,bus=usb1.0,port=1  \
    -vnc :0  \
    -rtc base=localtime,clock=host,driftfix=slew  \
    -boot order=cdn,once=d,menu=off,strict=off \
    -enable-kvm


Ja, es gibt zu einer destination nur einen einzigen gültigen next-hop (FIB) Eintrag.
Gäbe es mehrere passende Einträge, würde das im Umkehrschluss bedeuten, daß verschiedene RPs Überschneidungen in ihren zuständigen Präfixen hätten.
Das wär doof, da dann kein RP eine gesunde Topologie aufbauen könnte (im überlappenden Bereich) :)


Search form value reusability


Le problème c'était pas le fait que c'était mal indenté, mais que la balise PHP ne commençait pas le document : du coup quand on voulait faire des redirections par exemple, ça plantait. Tu connais ce bug ?


BTW this also had be scratch my head for a while
```
    memcpy(w, data, 16*sizeof(vtype)*SIMD_PARA_SHA512);
```
This is 16 for MD4, MD5, SHA1 and SHA256. And it should still be 16 for SHA512 even though the buffer is twice as big. Right? Because we're not copying a single SHA512 input buffer, but all buffers that fit in vtype.


Its an api issue with GCE, you need the project name in lowercase, Create a new project with name satisfying this regex pattern (?:(?:[-a-z0-9]{1,63}.)*(?:a-z?):)?(?:[0-9]{1,19}|(?:a-z?)).


TeamCity YaccConstructor :: YC.Core General Build [Build 391](http://teamcity.codebetter.com/viewLog.html?buildId=192074&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 116 Build time: 00:12:01


Hey, I think there are typo in line 10 canols --> canola


註解必須是 /** ... */ 的形式才會被正確的產生成 JavaDoc，像這樣：

/**
 * JavaDoc here.
 */


If the TS team says you should add private members, I see this as a slight TS error. I say 'slight' because I myself wouldn't expect an interface to define anything but public or protected members either. It makes sense that privates should stay in the implementation. This sounds like it's more difficult to solve than it should be, my recommendation would be to just define the private in DisplayObject and leave it out of extras.cacheAsBitmap (but with a comment). Any majors issues with this?


glad you figured this out! :+1:


You are right, I will change this as part of one of my next pull requests.


> There are some hardcoded value 16 used in SSESHA1body and SSESHA256body

Hmm you mean `	SHA1_PARA_DO(i) tmp[i] = vxor( data[i*16+t-3], data[i*16+t-8] ); \`

Yes, this is probably it! We have this in SHA512
```
#define SHA512_STEP(a,b,c,d,e,f,g,h,x,K)                        \
{                                                               \
    SHA512_PARA_DO(i)                                           \
    {                                                           \
        tmp1[i] = vadd_epi64(h[i],    w[i][x]);                 \
        tmp2[i] = vadd_epi64(S1(e[i]),vset1_epi64x(K));         \
        tmp1[i] = vadd_epi64(tmp1[i], Ch(e[i],f[i],g[i]));      \
        tmp1[i] = vadd_epi64(tmp1[i], tmp2[i]);                 \
        tmp2[i] = vadd_epi64(S0(a[i]),Maj(a[i],b[i],c[i]));     \
        d[i]    = vadd_epi64(tmp1[i], d[i]);                    \
        h[i]    = vadd_epi64(tmp1[i], tmp2[i]);                 \
    }                                                           \
}
```
I guess `w[i][x]` should be `w[16*i][x]` or something like that.


Any particular reason why this hasn't been merged into mainline?


capitalizing If just doesn't feel right :)


I opened an issue where we can talk about this: https://github.com/non/jawn/issues/32.


What kind of Qt5 problems?
Anyway, I think the editor should be in a separate package.
How can I add you packages to the website? Do I copy the .deb files directly or is it possible to make a ppa?


You're welcome :smile:


Sahada çalışan personele iş atamak, işleri organize etmek, raporlarını almak amacıyla geliştirilmiş uygulamadır.
İstek,önerileriniz için muyilmaz@gmail.com adresine e-mail gönderebilirsiniz.


Hi @carlosfrodriguez, can you please add the new CPAN modules in COPYING-ThirdParty?


window.startEventLoop() 不可以拿掉喔，那是刻意要模擬 GUI toolkit event loop 行為的


We could either change to a normalized coordinate system, or continue using 1280x720, but saved in floating points, to preserve accuracy.


@natarajaya you can use `when 'debian', 'ubuntu'`


It's strange that a few SHA512 formats is working at all. Maybe the self-tests doesn't catch it because we have only one single test vector.


nice.


bu ney?


I vote to copy the value rather then bend the stack-pointers (error prone)
``` C
*lifetime = entry[0]->lifetime;
```


Why bluebird is not used here?


I don't think there is really a need to get the actual resolution most of the time though. Considering it's all floating points. 0 - 1 that is.


don't use a compound key here. pid should be it's own key:
``````
KEY (`eid`),
KEY (`pid`)