nit: remove ": " from the end. The stack trace is usually added in a separate section by the logger.


CSS outline not supported by: Opera Mini (5.0-8.0)


Hmm, we can just re-use `linearindexing` and have a 2-argument (or n-argument) version.

`linearindexing` isn't currently exported, but perhaps it should be? It's used only to set a trait for new `AbstractArray` types.


They did work fine, yes. In general, I've kept the dependencies tight because I don't think I need to support a lot of older versions of packages. Maybe I'm being overly cautious, but I usually remove older versions once I upgrade to a new major or minor version.


I'm far from a wiser, more experienced guy. I don't really see anything wrong on my fork though.  No idea whay to do really. Sorry ;-(


Ah, I like that.  And, yes, it's not exported.  I documented it but kept it unexported, with the thinking that you need to explicitly import it anyways and almost all user use-cases will be simply defining the method.


No issue with the code but wondering why we need a metric for rep switch for single bitrate stream.


good!


vsechny testy moji komponenty jsou rozbity a musim procistit ty prototypy, udelam to zitra zatim je tu komponenta ktera uz neco dela. Zkus to pustit a dej vedet jestli se to chova spravne


95652938-jsoncpp (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1676 of 1676 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)


You don't need `-P:minibox:warn` and `-P:minibox:Yrewire-functionX-application`: they are on by default.


Goind through the logs again and while this Dockerfile builds correctly, the audit part is odd:

```sh
Error sending rule list data request (Operation not permitted)
ESC[1;31m[WARN]ESC[0m 1.8  - Audit docker daemon
Error sending rule list data request (Operation not permitted)
ESC[1;31m[WARN]ESC[0m 1.9  - Audit Docker files and directories - /var/lib/docker
Error sending rule list data request (Operation not permitted)
ESC[1;31m[WARN]ESC[0m 1.10 - Audit Docker files and directories - /etc/docker
Error sending rule list data request (Operation not permitted)
ESC[1;31m[WARN]ESC[0m 1.11 - Audit Docker files and directories - docker-registry.service
Error sending rule list data request (Operation not permitted)
ESC[1;31m[WARN]ESC[0m 1.12 - Audit Docker files and directories - docker.service
Error sending rule list data request (Operation not permitted)
ESC[1;31m[WARN]ESC[0m 1.13 - Audit Docker files and directories - /var/run/docker.sock
Error sending rule list data request (Operation not permitted)
ESC[1;31m[WARN]ESC[0m 1.14 - Audit Docker files and directories - /etc/sysconfig/docker
Error sending rule list data request (Operation not permitted)
ESC[1;31m[WARN]ESC[0m 1.15 - Audit Docker files and directories - /etc/sysconfig/docker-network
Error sending rule list data request (Operation not permitted)
ESC[1;31m[WARN]ESC[0m 1.16 - Audit Docker files and directories - /etc/sysconfig/docker-registry
Error sending rule list data request (Operation not permitted)
ESC[1;31m[WARN]ESC[0m 1.17 - Audit Docker files and directories - /etc/sysconfig/docker-storage
Error sending rule list data request (Operation not permitted)
ESC[1;31m[WARN]ESC[0m 1.18 - Audit Docker files and directories - /etc/default/docker
```


And for the miniboxing transformation without functions, you need `-P:minibox:Ykeep-functionx-values`.


Thats nice and clean


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (646 of 646 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)


I believe so, but have only recently woken up. :)


Hi Vincent,

I think only the first additional pair of parentheses is necessary. However, I'll merge this commit first. Thank you so much.


> From my understanding you need a list of
> features (I believe CDS features) to do FBA modeling.
> You do not need the whole genome object.

This is strictly true *right* now for most modeling tools in KBase. But there are many tools outside of KBase that we want to integrate where this isn�t true. It�s not a perfect assumption to make.

On Jun 4, 2015, at 3:17 PM, jkbaumohl <notifications@github.com> wrote:

> Hi Chris,
>
> There are many issues with the existing genome object. Additionally it
> collapses important distinct biological concepts into the 1 level.
> Adam is very frustrated by this. From my understanding you need a list of
> features (I believe CDS features) to do FBA modeling.
> You do not need the whole genome object.
> The current approach is a real issue in terms of the size of the objects.
>
> Initially we were talking about a more collapsed approach in which
> GenomeAnnotation and Taxon would be collapsed, but the there would be
> feature set WS objects.
> However Adam wants it broken out to correctly represent the biological
> concepts.
>
> I would suggest including Adam, Matt and Sam in the discussion.
>
> -Jason
>
>
> On Thu, Jun 4, 2015 at 9:50 AM, cshenry <notifications@github.com> wrote:
>
> > My suggestion: delete it all and go back to the original object and think
> > more about minimal changes from the original object.
> >
> > �
> > Reply to this email directly or view it on GitHub
> > <https://github.com/jkbaumohl/project_guides/commit/0d77051d5ed9c949d323dc1ce42ad73f9022769b#commitcomment-11525506>
> > .
> >
> �
> Reply to this email directly or view it on GitHub.
>


pourquoi sure_flip?


позновательно


@echen The size of the rect is "2" per xmin and xmax, so just need the first day of the weekend.

I adjusted it to be -.5 and 1.5 (instead of weekends and weekends + 2) so that the fill looked more correct, but in reality what should probably happen is I'd want all of the "dates" to be marked to noon and not midnight. :-\ Before what it was filling in was the space between midnight sat AM and midnight mon AM -- which looks weird.


pip install -r requirements.txt


@nirvdrum no, thank you! I'm just cleaning up the truffle.mspec right now. It looks like we can stop excluding a bunch of specs as a result of your Regex updates.


I *think* you're right, I'll confirm this tomorrow.


Typo: `Lets`


cette histoire de 1, 2, 3 n'est pas très rassurante... pk
                            if not isrotatable: et tu teste i==3 apres?


parce que la deuxième image est toujours flippée

2015-06-05 0:10 GMT+02:00 Siqi LIU <notifications@github.com>:

> pourquoi sure_flip?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/liusiqi43/friesnsteaks/commit/daf747dc32a0d58712c109e5d9a8a3c481ae1c53#commitcomment-11531233>
> .
>


Nice! Good example.


@htzh I added the missing length_product theorem


True, but I think leaving the XXX there to reflect the uncertainty about how this stuff should actually work may end up being helpful to future code maintainers.


Of course, this is probably closely related to the issue I mentioned above where it’s basically impossible to get `useDefaultServiceName` to be true.


Hi Guys, I have just bought an account from you guys but my tormail address witch i bought it with has stopped working. Can i please change this address to another one? If so, I will be a very loyal customer!

Best regards


Removed by me in some recent commit.


There was an error with the a hove only showing the first image. So i went and corrected that. I also took out the return to original image. I will probably work on getting that back but not sure if the on hover is the best.


mais si on veut générer plus d'images? plus que 3 par exemple? Que signifie 4, 5, 6, 7 alors?

Si la génération est toujours déterministe, on va forcer le modèle d'apprendre un certain pattern qui conforme à notre contraint prédéfini. Pour moi ça serait plus judicieux de générer chaque image complètement aléatoirement (avec 1/2 de flip, (0.6, 1.4) brightness, (-30, 30) rotation, agitation), on en génère plus mais au moins on sais que n'ajoute pas de contraint sur le modèle


I didn't test velocity setpoint after this change, but I think it shouldn't
change the behavior for velocity setpoint since that was already saturated
as far as I could tell from earlier testing.
On Jun 4, 2015 4:03 PM, "TSC21" <notifications@github.com> wrote:

> So does that mean that velocity setpoint works ok now?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jgoppert/Firmware/commit/1aa05f31336321dab20286aefca2dc9ffc4baeaf#commitcomment-11529047>
> .
>


You fixed this in the MediaView changes.


Ok, I'll take that feature out.

What you can do is put your data fetching in its own wrapper component and use shouldComponentUpdate to avoid extra calls to `trackMeteorData`.  This is one advantage of using a series of wrapper components.


au pire, définie tout en haut FLIPPED = 2, XXXX = 1


looks good! Now the only problem is that the sponsor logo URL for EMF is different than the one from stanford.. can we store this in the database somewhere? we should first find out where the logo is stored and could then add a new column to that table to hold the URL


@echen Looks like they coerce it, but changed nonetheless as both work.


the authors of nocrypto are on the same corridor :-) but mainly we use it in all mirage libraries now as we can compile it to xen if needed.


The recover is based on `request.second()`, which is the list of messages in the original request. `items` is not used in recover.


Don't forget to delete this file from source control


Gracias


(it is not very important, but can we will do the move at one point anyway, so it's better to do it early)


I don't see any difference in the two if blocks, isn't there a problem here?


This function is now small enough that I no longer think it's worth the effort.


I added them because that was the preference of the group of I'm working with right now. Perhaps I'll add it as a parameter later. And I have been toying w/ plot.ly --> which marks clear points where you can hover over.


Please don't leave commented code hanging around. If it's future work, let it be future work.


Comment removed.


Fixed in a previous commit.


See an example of use here: https://github.com/mirage/ocaml-git/blob/master/lib/SHA.ml#L142


PLEASE add a TODO to watch this or poll more frequently - these sleeps add up to e2e tests taking 45 minutes to run.


Please be consistent with your formatting options. I suggest you use Visual Studio defaults: 4 spaces for each indentation level.


What about increasing `arc_c_min` to `2 * SPA_MAXBLOCKSIZE`.  This would prevent this calculation from ever underflowing.  On illumos there is a 64M floor for `arc_c_min` which was decreased to 4M in Linux.  This would explain why they've never seen this.  The original motivation for us decreasing `arc_c_min` was to potentially get the ZFS memory footprint down so running on low memory devices would be possible.


Something that has been taken out of here is causing choppy movement.
I will slowly add each feature one at a time until I can identify what exactly broke the game and caused choppy movement.


Fix a typo with a typo :smile:


@clebergnu, yep, good point.


isn't this the opposite way around? That it will use the default db connection, then the `run` db last?


:)

The nice part about ggplot in all of this is you can append styles on top of plotMDTS and they'll overwrite it. E.g. plotMDTS(foobar....) + theme_economist()

https://github.com/jrnold/ggthemes


you're killing me


I find this variable name confusing,  isn't this the sum of mealsPerDay?


I thought the purpose of this interface was to hide the complexity behind all the nested dictionaries? A property with such a long and complex type is just unmaintainable. Please fix this.


That's what did it. Adding locs to game_del exclude contributes to choppy movement!!


Nice use of break to save yourself some unnecessary iterations. return would also work here.


You guys pull on latest changes from this repo as well as pick the latest version of the frameworks patch from gerrit?


Can we extract the token so it can be custom?


use GenerateName


GenerateName


This is a good patch, and is working perfectly for me.  Thanks!


where did index.html come from?


Lol https://github.com/jashkenas/backbone/pull/3661#discussion_r31686602


지친


Awesome, thanks!


Not to be a spoilsport, but what happens if you gzip the .bmp?


Just curious, what pushed you towards this implementation? I would've probably just modified the meerkats array in place but this works too.


Turns out the indention was broken, and fixing that made the problem go away.  That github notices this is pretty surprising, but hey...


I was expecting this method to just return a plain list of word matches (each one with a file path, a line and a column.)

You see, the problem with returning these complex data structures is that all the information is implicit: no one reading your code will know right away what your mapping from ```string``` to ```Dictionary<int, List<int>>``` means, and neither should they. We must make code explicit so it speaks for itself as much as possible.


TODO: test failure to bind (claim too big), and other failure and edge cases


landmaks may not be a good word here. milestones may be better.


		@@ -1,3 +1,4 @@
+ README
 Sex.pm
 매니페스트
 Makefile.PL


Hey Tin,

The removal of imports breaks the example ;)

Can you revert this? Thanks


If it looks wrong, it's spelled wrong.


trackers -> tracking


Already fixed in another commit.


I made a lot of corrections in my branch that aren't reflected here, this script won't work as is.


> I've kept the dependencies tight because I don't think I need to support a lot of older versions of packages.

This policy is responsible for the build errors you keep running into on NixOS.


kim jinny


just picking them now will let you know


should we get rid of the unique lock name for it too?


@andreasohlund is this still required?


no idea why ... but this fix doesnt work for me at all...even tried do fresh compile 2 Times.
anyone can confirn or... just say that this works ?


This can also be accomplished with the trim method. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/Trim


why not pass r up?


This is the only commit that matters.


can this be a var or const?


which generally corresponds to a small number of clusters.


Good call. I'll pull it out like you did with the other regex.


Don't forget var. It may not always be necessary but being consistent will prevent some future headaches.


..model, because tracking methods..


probably need citations when you refer to methods that use Boosting here.


This is not as bad as the previous interfaces, but it still suffers from readability problems. One thing I suggest to make it much more readable is to use type aliases (see http://stackoverflow.com/questions/9257989/defining-type-aliases), so that you can declare your method as ```Dictionary<string, Locations>```` where ```Locations``` is an alias for ```IList<int>```.

That said, use type aliases with care. In this case, it's okay since the data is not too complex and it would be overkill to add a ```Locations``` class just to make things more readable, but in general try to keep your methods and properties types as simple as possible, and most important of all, do not let implementation details of an interface leak, as in the ```IWordDictionary``` interface.


What happens?  The past and the future collide, and the universe implodes.

An .svgz is a valid image format defined in the original SVG specs (you can open it in Inkscape/Illustrator without decompressing it separately).  Gzipped bmp, that's just crazy talk.  16 KB of crazy talk, to be specific.


95652938-jsoncpp (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1676 of 1676 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)


Added.


This already happened in some other commit.


Yeah, this is on the etherpad list of stuff to do at the end.


Also in scope for the etherpad "cleanup debug logging" item.


Please rename ```getWords``` to a noun. In general, objects should be named using nouns, not verbs.


no idea why ..but this doesnt work for me at all... only makes tchem aggro.
I tried to compile fresh core 2 Times.
any one can confirm? or just say that works for him?


Dictation software issue that I didn't catch.  It's already been removed.


I don't think you need a second loop here.


Yes.


Try to avoid using fully qualified names if possible. It clutters the code unnecessarily. In this case, by doing so you can also get rid of the ```lines``` variables which is only used in the ```for``` loop.


Hi Chris,

Do you know what the other tools require?
If there other factors to consider we will try to weigh them in our design
decisions.

Thanks,
Jason

On Thu, Jun 4, 2015 at 3:09 PM, cshenry <notifications@github.com> wrote:

> > From my understanding you need a list of
> > features (I believe CDS features) to do FBA modeling.
> > You do not need the whole genome object.
>
> This is strictly true *right* now for most modeling tools in KBase. But
> there are many tools outside of KBase that we want to integrate where this
> isn�t true. It�s not a perfect assumption to make.
>
> On Jun 4, 2015, at 3:17 PM, jkbaumohl <notifications@github.com> wrote:
>
> > Hi Chris,
> >
> > There are many issues with the existing genome object. Additionally it
> > collapses important distinct biological concepts into the 1 level.
> > Adam is very frustrated by this. From my understanding you need a list of
> > features (I believe CDS features) to do FBA modeling.
> > You do not need the whole genome object.
> > The current approach is a real issue in terms of the size of the objects.
> >
> > Initially we were talking about a more collapsed approach in which
> > GenomeAnnotation and Taxon would be collapsed, but the there would be
> > feature set WS objects.
> > However Adam wants it broken out to correctly represent the biological
> > concepts.
> >
> > I would suggest including Adam, Matt and Sam in the discussion.
> >
> > -Jason
> >
> >
> > On Thu, Jun 4, 2015 at 9:50 AM, cshenry <notifications@github.com>
> wrote:
> >
> > > My suggestion: delete it all and go back to the original object and
> think
> > > more about minimal changes from the original object.
> > >
> > > �
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/jkbaumohl/project_guides/commit/0d77051d5ed9c949d323dc1ce42ad73f9022769b#commitcomment-11525506
> >
>
> > > .
> > >
> > �
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jkbaumohl/project_guides/commit/0d77051d5ed9c949d323dc1ce42ad73f9022769b#commitcomment-11531224>
> .
>


ok


Might be a typo in your file path?

```diff
- "#{Rails.root}/public#{attachment.file}"
+ "#{Rails.root}/public/#{attachment.file}"
```


Thanks!

Le jeu. 4 juin 2015 18:39, Jeffrey Way <notifications@github.com> a écrit :

> Yes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/elixir/commit/54ce8b747470455bc0090026ff5fdbbe12ca7acf#commitcomment-11531691>
> .
>


Firefox also triggers menu on Ctrl+F10. Tell the users that it is a feature not a bug :)


Your tooHungryDay is the number returned in the if statement above. This return statement will only be reached if the other one is not, in which case there won't actually be a tooHungryDay.


Using type aliases, as I suggested in a previous comment, can make this code more readable too. Give it a try.


Please do not capitalize variable names (```LineNumber```)


I'm working on an app which uses RSDayFlow, I want to put a number instead of icon below the dates. I really hope there's something like
```objc
- (UIView *)datePickerView:(RSDFDatePickerView *)view customViewForDate:(NSDate *)date;
```
Then the calendar would be extremely customizable. If you like the idea, I can implement it and open a pull request. :)


probably prefer no spaces ```{steps}```

http://facebook.github.io/react/


listener for listener.data in @listeners


underscore?!


Sticky shift button!


Palsson’s e-matrix needs information about gene sequences, the entire genome, protein sequences, codon usage etc.
Our methods also require information about taxonomy and close genomes.
Costas has methods that use things like codon usage as well.

On Jun 4, 2015, at 5:39 PM, jkbaumohl <notifications@github.com> wrote:

> Hi Chris,
>
> Do you know what the other tools require?
> If there other factors to consider we will try to weigh them in our design
> decisions.
>
> Thanks,
> Jason
>
> On Thu, Jun 4, 2015 at 3:09 PM, cshenry <notifications@github.com> wrote:
>
> > > From my understanding you need a list of
> > > features (I believe CDS features) to do FBA modeling.
> > > You do not need the whole genome object.
> >
> > This is strictly true *right* now for most modeling tools in KBase. But
> > there are many tools outside of KBase that we want to integrate where this
> > isn�t true. It�s not a perfect assumption to make.
> >
> > On Jun 4, 2015, at 3:17 PM, jkbaumohl <notifications@github.com> wrote:
> >
> > > Hi Chris,
> > >
> > > There are many issues with the existing genome object. Additionally it
> > > collapses important distinct biological concepts into the 1 level.
> > > Adam is very frustrated by this. From my understanding you need a list of
> > > features (I believe CDS features) to do FBA modeling.
> > > You do not need the whole genome object.
> > > The current approach is a real issue in terms of the size of the objects.
> > >
> > > Initially we were talking about a more collapsed approach in which
> > > GenomeAnnotation and Taxon would be collapsed, but the there would be
> > > feature set WS objects.
> > > However Adam wants it broken out to correctly represent the biological
> > > concepts.
> > >
> > > I would suggest including Adam, Matt and Sam in the discussion.
> > >
> > > -Jason
> > >
> > >
> > > On Thu, Jun 4, 2015 at 9:50 AM, cshenry <notifications@github.com>
> > wrote:
> > >
> > > > My suggestion: delete it all and go back to the original object and
> > think
> > > > more about minimal changes from the original object.
> > > >
> > > > �
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > https://github.com/jkbaumohl/project_guides/commit/0d77051d5ed9c949d323dc1ce42ad73f9022769b#commitcomment-11525506
> > >
> >
> > > > .
> > > >
> > > �
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/jkbaumohl/project_guides/commit/0d77051d5ed9c949d323dc1ce42ad73f9022769b#commitcomment-11531224>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


Sample output:

```
➜ lit git:(master) ✗ luvi . -- ls                                                                                Timothys-MacBook-Pro.local
lit version: 1.3.0
luvi version: v2.1.0
command: ls
creationix/coro-channel    v1.1.0
creationix/coro-fs         v1.2.4
creationix/coro-http       v1.0.7-2
creationix/coro-tcp        v1.0.5-2
creationix/coro-tls        v1.2.0
creationix/coro-wrapper    v1.0.0-1
creationix/git             v2.0.0
creationix/hex-bin         v1.0.0-1
luvit/http-codec           v1.0.0-1
luvit/json                 v2.5.0
luvit/pretty-print         v1.0.2-1
creationix/prompt          v1.0.3-1
luvit/readline             v1.1.0-1
luvit/require              v1.2.0
creationix/semver          v1.0.3
creationix/ssh-rsa         v1.0.0-1
creationix/websocket-codec v1.0.3
done: success
```


nice i'm gonna start using this


what happens if there's an error here? should we ignore it altogether ever?


It's nice, adds warnings if it's violated.


That was supposed to read "Fix $_ for Rubinius in regexp.rb".  I guess I got a bit ahead of myself.


I do not see how citing this figure fits in this sentence?
You probably need a new paragraph explaining that figure. It should be placed in a point where it fits naturally in the flow of the section.


how about ```export {Wizard.<subcomponent> as Wizard.<subcomponent>}``` underneath each component and ```export defaults Wizard```?


the authors of


@chrisseaton Is there a better way to do this?  I didn't see an obvious way to use the proper nodes because I can't figure out the slot until after I have the caller frame.


avoids a costly labeling process.


fun to es6ify it ```step => {```


criteria


This was fixed by some of your recent changes, from what i can see.


@josevalim what do you think of this refactoring? It should work seamlessly because `key/1` is meant to return something that identifies a translation in a set of translations with no duplicates, so we can actually use it to *find* duplicates.


I'm not sure how to create the case where this errors. Because `urlCharactersToEscape` only matches single ASCII characters at a time, `utf8.DecodeLastRuneInString` should only pull out single runes. If it's unable to do that, we have a malformed string and likely other problems.

Let me write a test or two to see what happens with a few "international" Unicode characters.


frames out of 29519


So, none of this is necessary inside of cf-tls. The errors from SayHello can just be ignored in cfssl scan. I'd just revert back to [877a71762490c938f39f5855af4a5e42a38e3c4c](https://github.com/jkroll/cf-tls/commit/877a71762490c938f39f5855af4a5e42a38e3c4c), merge this, and then add one of the checks we talked about in the cfssl PR


Fixed.


```wordColumn``` doesn't really tell what this variable holds. ```wordMatches``` might be more appropriate.


structure is a confusing word here


That would be great, thanks Kelly.


That was my first idea and I do think it's a very good idea.  That said, I do get nervous about all the subtraction operators against unsigned values sprinkled throughout the code.  I certainly didn't audit all of them, but a quick search for similar ones showed there were no others like this one.  I'd suggest we change the floor of ```arc_c_min``` and then add an assert here.


I'm using `IO.iodata_to_binary/1` since a translation could be marked as *different* from another with the same msgid just because the msgid is split over multiple lines in a different way. These two (equal) translations would be considered different:

```po
msgid "my"
  " foo"
msgstr "bar"

msgid "my foo"
msgstr "bar"
```


Probably better to do it all at the end, yeah? Easier to check what the module exports are.

    export default Wizard;
    export {Wizard.ProgressBar as Wizard.ProgressBar}
    export {Wizard.ProgressItem as Wizard.ProgressItem}
    export {Wizard.Step as Wizard.Step}
    export {Wizard.Menu as Wizard.Menu}
    export {Wizard.MenuItem as Wizard.MenuItem}


Can you iterate this the more concise ```foreach``` loop here?


a bit subjective...there's a case for doing it next to the definition so you don't have to hop up and down whenever you add/remove an export


This seems unnecessary.


LGTM!


I do think you need to mention why detection is relevant to tracking, but the sentence you had here was somewhat confusing.


Yeah, i found David and Hannes authored the nocrypto libray.  So I think the deterministic task id has been taken care of, I am moving to persisting the data part : )


Testing this condition is redundant, if ```words.Length``` is zero, the ```for``` loop just won't execute its body.


i literally didn't push this...


`translatedConstants`?


Since this code was added here we need to add test cases for it on aui-layout-col-tests.js.


Alright, so it looks like this does indeed break with UTF-8 characters that span more than one byte. I'll have to loop in the rest of the team to see what we're expecting in that situation. Mind merging this for now and then we'll tackle those characters in a future PR?


I should also mention: it does not fail hard, it just generates an incorrect URL.


It's fine to have this assignment, but I would also augment getTypeFromTypeNode to return booleanType when it encounters a type predicate


I ran into this today as well. Instead of checking which environment I'm checking if `Rack::Timeout` is defined:

```ruby
if defined?(Rack::Timeout)
  Rack::Timeout.timeout = (ENV['RACK_TIMEOUT'] || 10).to_i
end
```


These are still commented on this pr code. I tried just uncommenting these lines, but the test fails in that case. Can you check if those should really be removed or if there's  a case that still needs fixing?


What does this do?


Should be i < res.length not i <= res.length. This causes the loop to run infinitely because res[res.length] will add a chirp to the array after the last element and increase the length by one for the next iteration.


o?


When you've corrected the above, turn the array back into a string and return it.


"The process to" doesn't sound much like anything else, I don't think. "To set the proxy to /proxy/, [a remote end must] run the following steps:" or something, sounds better I think.


If in a new list item


With what status?


LGTM! :D


That sounds like an "otherwise" clause, and requires a status.


s/than/then


nit: an SSH tunnel


The compiler builds maintained by arm on https://launchpad.net/gcc-arm-embedded include newlib nano in them. That will be the only supported distribution mechanism for the GCC tools and related libs.


no idea how the fuck this happened but i did not do this, i just deleted all my ssh keys and changed pass.


Fix spelling here: hasDifferentParameterIndex


It returns true if repartition actually does something useful (vs. maintaining the status quo)


final


You could do this as a switch-type statement (see my PR for Set Timeout)


Wrap the second disjunct to the next line


Accidentally left in too many curly braces


Oh, I added better support for a username-less password here as well.


Don't add spaces here, you don't want leading and trailing spaces before/after the first/last elements.


at the start of the acknowledgments: relief -> relieved


Why not just `Truffle.getRuntime().getCallerFrame()` ?


J’aurais laissé l’ancienne version. `Un ou plusieurs` est considéré comme un adjectif indéfini pluriel.

+ http://fr.wiktionary.org/wiki/un_ou_plusieurs


We need a string back, so join the array before returning


I see your point, but there's not really an easy way to know that this is complete, or we wouldn't be doing this timeout hack. :-) More to the point, if they timeout, they won't ever "complete", for some definition of that.

That said, turning off the frames doesn't look trivial, and it would also mean that the automation wouldn't catch a bunch of errors that it (I think) would today.

I'd be somewhat inclined to just leave things as they are, and bump the timeout up if we start seeing false negatives.


perfect


Yeah, or do the thing where you memoize Tracker results which we were also talking about (might be easy to do with isoquery!)


I uploaded a new version a few mintues ago.


We've got several directives calling parse on their own elements, and $facebook was doing a global parse each time - that was causing delays and flashing of FB components (like comments).  Having $facebook.parse pass parameters along cleared up that issue.


Mind changing this to ``rabbitmq_autocluster``?  It is intentionally set to not mix the service name with normal ``rabbitmq`` entries.


should we call _tunnels_lport something like _used_tunnel_lports ?


Take a look at the two problems described here https://github.com/hablutzel1/cryptoapplet/issues/3.


I didn't know that existed.  I copied the pattern I used for `ProcNodes`.  Thanks for the tip.


You can declare a local further up for node.type so you don't have to keep casting it.


That's fine. There's a node to read out of a thread local though.


rax retry tests


Math.random is a function and needs to be called, Math.random(), here. It is also not inclusive of it's upper bound (1), so you don't need to subtract one.


Fixed.


should we use os.name instead? http://stackoverflow.com/questions/1325581/how-do-i-check-if-im-running-on-windows-in-python


J


Pas d’espace avant les points de suspension. Éventuellement utiliser `…` ou `\dots` si l’environnement `LDA` le permet ?

+ http://fr.wikipedia.org/wiki/Ponctuation#Les_espaces_autour_des_signes_de_ponctuation_.28r.C3.A8gles_actuelles.29


what is adapted? the size of the search space, or the size of the object window?


I think it might have to be `rabbitmq-autocluster` to be DNS-discoverable. But I agree that this should not conflict with a port 5672 broker service.


> Child component **od** EventBox

This is a bit strange. :smirk_cat:


this was better before the change :)


Please update to ``rabbitmq-autocluster``


just return the full string (i.e.: localhost:{port}) from create_ssh_tunnel


random -> particular


relief -> relieved


I think we could do this all with one function


but file IndexExport_p.html / IndexExport_p.java is missing in commit


I think you should reference something (my serious suggestion is https://web.archive.org/web/20070602031929/http://wp.netscape.com/eng/mozilla/2.0/relnotes/demo/proxy-live.html) that defines what a proxy autoconfig is like:

If the implementation supports proxy autoconfiguration [link], set the implementation's autoproxy configuration url to /proxy auto configuration url/. (can this fail?)

Otherwise return an error with status /invalid argument/.


This also logs an event with initial device orientation. Do we want this @scsouthw?


Ok so if I am understanding this correctly when we in the future
incorporate other model methods we will need to be able to get to all of
these details.
Currently codon usage is not covered in the current or new designed genome
approach.
So we will need to think about how to incorporate that.
Beyond that I think all of this should be possible, although you are
correct it would require bringing in multiple objects and traversing
relationships.

-Jason

On Thu, Jun 4, 2015 at 3:43 PM, cshenry <notifications@github.com> wrote:

> Palsson’s e-matrix needs information about gene sequences, the entire
> genome, protein sequences, codon usage etc.
> Our methods also require information about taxonomy and close genomes.
> Costas has methods that use things like codon usage as well.
>
>
> On Jun 4, 2015, at 5:39 PM, jkbaumohl <notifications@github.com> wrote:
>
> > Hi Chris,
> >
> > Do you know what the other tools require?
> > If there other factors to consider we will try to weigh them in our
> design
> > decisions.
> >
> > Thanks,
> > Jason
> >
> > On Thu, Jun 4, 2015 at 3:09 PM, cshenry <notifications@github.com>
> wrote:
> >
> > > > From my understanding you need a list of
> > > > features (I believe CDS features) to do FBA modeling.
> > > > You do not need the whole genome object.
> > >
> > > This is strictly true *right* now for most modeling tools in KBase. But
> > > there are many tools outside of KBase that we want to integrate where
> this
> > > isn�t true. It�s not a perfect assumption to make.
> > >
> > > On Jun 4, 2015, at 3:17 PM, jkbaumohl <notifications@github.com>
> wrote:
> > >
> > > > Hi Chris,
> > > >
> > > > There are many issues with the existing genome object. Additionally
> it
> > > > collapses important distinct biological concepts into the 1 level.
> > > > Adam is very frustrated by this. From my understanding you need a
> list of
> > > > features (I believe CDS features) to do FBA modeling.
> > > > You do not need the whole genome object.
> > > > The current approach is a real issue in terms of the size of the
> objects.
> > > >
> > > > Initially we were talking about a more collapsed approach in which
> > > > GenomeAnnotation and Taxon would be collapsed, but the there would be
> > > > feature set WS objects.
> > > > However Adam wants it broken out to correctly represent the
> biological
> > > > concepts.
> > > >
> > > > I would suggest including Adam, Matt and Sam in the discussion.
> > > >
> > > > -Jason
> > > >
> > > >
> > > > On Thu, Jun 4, 2015 at 9:50 AM, cshenry <notifications@github.com>
> > > wrote:
> > > >
> > > > > My suggestion: delete it all and go back to the original object and
> > > think
> > > > > more about minimal changes from the original object.
> > > > >
> > > > > �
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > >
> https://github.com/jkbaumohl/project_guides/commit/0d77051d5ed9c949d323dc1ce42ad73f9022769b#commitcomment-11525506
> > > >
> > >
> > > > > .
> > > > >
> > > > �
> > > > Reply to this email directly or view it on GitHub.
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/jkbaumohl/project_guides/commit/0d77051d5ed9c949d323dc1ce42ad73f9022769b#commitcomment-11531224
> >
>
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jkbaumohl/project_guides/commit/0d77051d5ed9c949d323dc1ce42ad73f9022769b#commitcomment-11531758>
> .
>


These three properties are being updated in all the JARs (including third party dependencies like Log4j).

The other properties are only included in the main JAR (`uji-ui-applet`), see https://github.com/hablutzel1/cryptoapplet/blob/559085d0ecc0593ca2fbca0b9d8523c5abe49873/uji-ui-applet/pom.xml#L83.


i think you are putting two different definitions into a single sentence. It is better to have the two definitions in separate sentences.


Is it really "nulls" or empty strings?


this would fail with IPv6, use something like: https://github.com/twitter/zktraffic/blob/master/zktraffic/base/client_message.py#L142


Além disso, foi corrigido um erro nessa linha =)


Falla la búsqueda por apellido. #Arreglar.


handles camera rotation sequences better,


You've made this extend from `A.Widget`, but I haven't seen anywhere calling `render()` on it. Widgets should only do dom operations after being rendered. The append call that is happening on the `initializer` function should only happen on `renderUI`, and the same goes for the `_uiSetFields` call.

Having this be a widget can cause complications though. We don't really need a separate render phase for this component and widgets are more expensive. Making this extend just from `A.Base` and have a `content` attribute like field has will make it more consistent with the other form builder classes, and you also won't need to change you had to make on aui-layout-col on the other commit to support widgets.

So, to sum it up, this needs to either follow the pattern of widgets, separating dom operations into `renderUI` and calling render on it, or it can stop being a widget, like FromBuilderField.


Why is this not manual? What do direct, autodetect and system do?


I presume you intended to do this in the /fr/rules/README.md file ? You've now been changing the main one - including a revert to an older version (previous commit)


Errors have to have a code, and if you can return an error you also have to return success when there is no error.


not hard to do recursive `ls` ? (thinking of a dependency tree view would be nice)


Please rename ```column``` to ```columns```


What’s the purpose of this code? I don’t understand these dayNum 18..22.


What do you think about this implementation, @anlinde?


This needs to check the return value.


Also, to deal correctly with widgets this should e appending `boundingBox`, not the `contentBox`, otherwise the connection between the 2 widget boxes will be lost.


This commit breaks mono/tests/marshal.exe.

My theory is that since not all signature copying functions (we got many :sob: ) have been changed to copy the return value, we got signatures now picking a a pointer from inside the dup'd string that gets eventually freed.

All sig dupe functions should be fixed first.


@ugocupcic  I just noticed this patch **is required** to run the ur + shadow hand
From what I see, this might happen with other arms using the new transmissions from ros_control as well as the new gazebo_ros_control.

It is quite bad to require the arm that carry your hand to remove the ros_control and use your transmission model. Things should be modified at the Shadow side to handle that.

as long as the shadow hand is not using the gazebo_ros_control, sr_gazebo_plugin should not see the ur_description but only the sr_description. This is due to a bad robot_description separation between incompatible robots/plugins.
In our setup we used namespaces for the sr_gazebo_plugin to avoid this problem, but of course the plugin produces a joint_state in its own namespace. (rh/joint_states)
The namespace can be specified in the urdf with
 `<robotNamespace>${namespace}</robotNamespace>`
even the robot description can be specified  avoid the problem

`<robotParam>sh_description</robotParam>`

loading a second (or more) robot_description for the hand is not a big problem. We run 7 different robot_description
la_arm+ra_arm+lh_hand+rh_hand (on robot_description for full bimanual robot)
ra_arm (for plugins, on ra/robot_description)
la_arm
rh_hand (for plugin, on rh/robot_description)
lh_hand
ra_arm+rh_hand for spawning
la_arm+lh_hand

and this is probably always necessary as not every controller / plugin in the world can consider all the possible special cases of other robots living in the same world.
Except if all the controller us the same simulation plugin and conform to it.

Please consider fixing or removing this patch on transmissions/plugin when ever possible.


I suggest leaving this one as it was as the typical use-case is on one line embedded within HTML.


Script de l'upload d'une image: Terminé


Sorted


Sorry, meant to say add check for default language. (Which is English)


These are transitive dependencies exclusions, and `com.lowagie:itext:2.1.5` depends on the 'bouncycastle' groupId version of Bouncy Castle libraries, take a look at http://central.maven.org/maven2/com/lowagie/itext/2.1.5/itext-2.1.5.pom. The previous exclusion declarations were not having any effect.


Thanks for this ;)

Other than bloating the database, what are the side-effects of adding "duplicate" Programs on each sync?


o.O Damn. Glad you've noticed, that was just for creating template, now it have to be dynamic. It's day of month, used for table headers in vertical layout.


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1162 of 1162 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)


This commit broke config-pin for me. The kernel 4.1 check is not right: /sys/devices/platform/ exists even on 3.8.13, so it breaks everything.


void promise strikes again :( Good catch!


Mauvais libellé: Ajout de la gestion du mot de passe vide et du Login déja existant


I hope you realize that these tests are hard to read (e.g. what is ```dict.dictionary.Count```?), and hopefully once you address my other comments you'll be able to write clearer tests.

On the other hand, you might want to start by writing the tests first so that your API is guided by principles of readability and usability, and not by the implementation details. Google "Test Driven Development" for more details on this.


.domain([data.metrics.slice(-1).pop(), data.metrics[0])


Looks good, but untested. Thanks!


You are totally right, will send a new PR for this.


No Prob.


Why return this if it's psedo classcial?


Please check the integrity of included DLLs by comparing them with the official ones, http://www.openoces.org/opensign/releases/opensign-1.8.0-bin.tar.gz (note that OpenOCES website is soon to be closed so it is a good idea to download the required packages soon).


aha, i see ... above change is wrong.  the pkg_version should be prefixed with ~ instead.  so sugar-speak-activity_50~0godiard1 should have been sugar-speak-activity_50-0~godiard1


same here.


Originally, yes. In a5007705, I pulled it out 1. Due to erroneous math that has now been fixed and 2. I thought it would be more flexible to let the user either project the initial acceleration or compute it. So I factored those out into stand alone functions. That said, I have not guarded against the case the user does not initialize using one of the two methods above.


Consider:

    info.synonyms.join(", ")


How about using the good old negation operator ``!`` rather than "not"?


And thanks a lot for catching this :+1:


Don't forget to be consistent with your spaces. I suspect this problem may be due to your mixing tabs and spaces. Use only spaces to avoid problems (VS has a configuration option in "Settings" where you can set this)


I haven't tested it, but it might also differ from driver to driver.
If it turns out that it's different between our official drivers, it might be best to not mention the `db` optarg to `run` until we've made the driver behaviors consistent.


@marcobiedermann

Was not working because, it's not exposed by the API publicly.


This should be in the configuration.


The db specified in `connect` sets that connections' default, so it's the one that's used if you don't specify one in `r.db()` and don't pass an argument to `run`. The argument to `run` has to be higher priority, doesn't it? If the default connection took higher priority then the argument to `run` would always be ignored. Unless I'm not following the logic correctly.


Please separate the three essential parts of your test (i.e. Preparation, Operation and Assertions) with a blank line, it's much easier to read a test at a glance that way.


Instead of this mess? https://github.com/monarch-initiative/monarch-app/commit/1759203104e39a4a68413f451a278d811a63b0b8#diff-ec9b722530bc335568e7c40caf837487R3280


@viomac There's quite a few things to fix in these last changes. I left you plenty of feedback for you to chew on...


You're welcome.

I'd like to start contributing to an open source project. I had heard a
Talk Python to me" podcast about EVE and thought I'd drop by to check it
out. Your commit just happened to be the first that I looked at. =)  A
fortuitous coincidence.
 On Jun 4, 2015 7:17 PM, "Magdas Adrian" <notifications@github.com> wrote:

> And thanks a lot for catching this [image: :+1:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nicolaiarocci/eve/commit/64a633a2a2b54fdccd86df20873630f3444d63e8#commitcomment-11532204>
> .
>


An API 'service' may consist of several 'rpc services', these services may be defined in multiple protos.

- This script should not depend on services being in a single proto that as the same name as the package like this.
- Rather, it should package all the protos it finds of the specified <package>/<version> subdir of the repo.


Does async get used ?


r+


Awww, that dog is too cute to turn into a zombie!!! Lol! :laughing:


What's `linux.dev`?
I could only find one reference to that string in nixpkgs but it doesn't give me any clue...


If line 10 is removed, validation will fail. Now I prefer to used event + ordinal format time, however, when will ISO format time function?


Oh OK, then ignore my comment.


@ruslo Can you please review?

Likely the autotools_* support files shouldn't be living in schemes directory.

This scheme used to work with android as well (for the example library), but something changed in the toolchain on these last changes that we did while reverting to upstream, so I will investigate why is it not working anymore.

Can add hunter_ prefix to the functions after it matures.

Thanks!


saw approval from graydon
at https://github.com/MonsieurNicolas/stellar-core/commit/75cc7e139e628bccbb2d09cc13e6badf8773b3e5


merging MonsieurNicolas/stellar-core/bugfixChangeTrust = 75cc7e13 into auto


MonsieurNicolas/stellar-core/bugfixChangeTrust = 75cc7e13 merged ok, testing candidate = 6d956bd0


Crash in settings/miscellaneous.


test


Hello This is test file


Test File


code = singular, pattern = plural


pattern**s**


I think we may want to support Android API levels as low as 10 (GingerBread) to try and reach as many users as possible.


``` ValveBiped.Bip01_R_Hand ```
Why did you not use this by default?


I dont think we need this instance variable


день и месяц рождения нарочно пропущены?


1 на конце не нужна


Why do we sometimes use `count(self.text!)` and sometimes without the `!`? I assume there's no shadowing `text` in scope.


It's the same behavior as the previous HashTypeChecker PR, I just removed the variable as it wasn't used anywhere except there. You can see the old commit 2f242f5 and the reason why " Make HashTypeChecker more robust. In the rare case where john finished by himself and we are doing the parsing on another thread, a user can change file at the same time of the parsing(before the signal typeChanged is emitted/or handled by MainWindow). By verifying the file, we make sure the right information will be put in the tableview, or ignored"
In a multi-session environment where we can open multiple time the same file, we could also append the tab number.


i think this should be called name


Назвать Contact


@tomgidden Was it [`realpath`](http://php.net/realpath) instead of `readlink` you were originally looking to use?


create_contact - методы называем начиная с глагола


remove :blank line


If `endingString` is the substring of `suggestion` from `suggestion.startIndex` + the length of `text`, and the suggestion always starts with `text`, then isn't `text + endingString` always going to be identical to `suggestion`?


Seems unsafe; I could see "user forgets to set acceleration" being a tricky source of bugs.  Add a guard in NDEBUG mode?  Regardless, I just wanted to make sure this was a design decision and not a redundancy.


So can't we just do

```
let markedString = NSMutableAttributedString(string: suggestion)
markedString.addAttribute(…, range: NSMakeRange(enteredTextLength, count(endingString))
```
?


What is this?


What is this "noinspection" thing here?


I think it's worth adding a little comment here that this is the magic: by checking that `text` is shorter than `suggestion`, we're ensuring that `completionActive` is only true if there are additional characters to suggest, and that this suppresses the caret.


whoop, accidentally removed the imports while playing with your nice example :)
fixed on master 5398e710345764341d6006dca3f1ee4189725846


Please use camel case for your variable names, as is the usual Java convention (i.e. ```loginButton``` instead of ```button_login```)


yep


Oops... I'll fix this asap!

On Thu, 4 Jun 2015 03:58 Mattias Wadman <notifications@github.com> wrote:

> Hmm my extension seems to have reset my options to default and opens the
> options page on every start. Im guessing this happens because
> config_version is gone. Im guessing this happens because localStorage.verConfig
> = chrome.app.getDetails().config_version in options.js does not really
> work that well when undefined as it seems the string "undefined" get stored [image:
> :disappointed:] So at startup chrome.app.getDetails().config_version !==
> localStorage.verConfig is always true and the condition for options reset
> is localStorage.verConfig === 'undefined' (looks familiar hmm)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/YodaDaCoda/chrome-transmission-remote-plus/commit/c3cc9f38b5f51911eb8ec0dd343f2c45b04f4772#commitcomment-11504864>
> .
>


"StandardVMType/JDK 8"
This looks like Java 8 will be now required.. Is that the case? I think it should be possible to set a minimum version of Java and anything higher would work without problem..


The logic for the button and the image is the same. Please refactor it to remove the duplication.


r+


```UtilActivity``` doesn't really tell much about the purpose of this activity. Think of a better name.


Using the issue tracking system is awesome!


Some of this code was taken from a third-party lib from earlier Swift days, so my guess is that there are some remnants still hanging around. I'll remove.


Shall we restore it to the illumos default `MAX(allmem / 32, 64 << 20)` or to the bare minimum `2 * SPA_MAXBLOCKSIZE`?  And I definitely agree about adding the assert.  This kind of code makes me nervous too.


Yeah, sorry @kanaka, that's been left on the back burner a little too long.

Everything's working, all the native and self-hosting tests pass, bar one.

The fibonacci tests in step 4 work correctly, but time out. I haven't run fib(10) to completion, but I'd estimate about 2 hours for this case. IIRC, this is in native mode, not self-hosting (I could be wrong there...)

Can I get a special pass for a too-slow implementation? :)

ps. This has been my first go at Perl 6. There's a lot to like. There's also a touch of:
![perl6](http://www.wired.com/wp-content/uploads/2014/06/the-homer-inline4.jpg)


Is the first file the DetailActivity or the DetailFragment?


Comments that explain something about how a framework works are okay for tutorials, but for a real-world app let's try to make the code speak for itself as much as we can and limit our use of comments to explain things about our app or the decisions we make in the code.


> "If needed" is pretty much always a redundant phrase

:+1: I'll remove it in a later PR.

> The new version is too chatty

In general, or just because of "if needed"? Since `--serverlist` was removed, I prefer to mention it explicitly here.


Will do. /cc #2.


These comments are not very useful, as they don't tell me anything that can't be easily gathered from the code. Please avoid them.


Line 400 was edited by github online editor


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


This is still there in latest master. https://github.com/senchalabs/jsduck/blob/d3ef83dac/template/app/Comments.js#L209-L224

Probably shouldn't hardcode "http". There is `location.protocol`.


Use Ima_ADPCMStream::_imaTable instead.


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


这个是因为只要涉及到装备更新都得刷一遍装备数量，每次都回母港更新一次的话保证正确。
加的话其他地方也要加，getship，creatitem等等。


Use ADPCMStream::_stepAdjustTable instead.


Great to see that we can get the photo to be in Image View. But I noticed that everytime I click on a photo from the Gallery, I get the image view that somehow gets rotated 90 degrees clockwise. I am not sure why.


Makes sense to me -- I'll test and land.


Isn't this just DVI_ADPCMStream?


Formatting of all these rows.


Hi! I'll be away for a few days startring this morning. Here the line,
using spring 98.0.1-665-g088db4b develop & ZK from wget/GIT  (ghetto mode,
I know ^^)

[cawidgets.lua] Error: Error in GameFrame(): [string
"LuaUI/Widgets/unit_global_build_command.lua"]:1291: attempt to index local
'cachedJob' (a nil value)

[cawidgets.lua] Error: Removed widget: Global Build Command

spring user config:

3DTrees = 0
CamFreeScrollSpeed = 0
CamMode = 5
FPSEnabled = 0
FPSScrollSpeed = 0
FixAltTab = 1
GrassDetail = 0
GroundDetail = 40
HardwareCursor = 1
InputTextGeo = 0.26 0.73 0.02 0.028
MaxNanoParticles = 10000
MaxParticles = 10000
ReflectiveWater = 2
RotOverheadEnabled = 0
RotOverheadScrollSpeed = 0
ScreenshotCounter = 1
Shadows = 0
ShowFPS = 1
ShowPlayerInfo = 0
SmoothLines = 1
SmoothPoints = 0
UnitIconDist = 10000
UnitLodDist = 484

Hope that help ! See you !

2015-06-04 14:21 GMT+02:00 mtroyka <notifications@github.com>:

> Hmm, well it's good that the gfx bug is fixed. From what you posted it
> seems you are using the latest version, could you post the relevant line
> from infolog? It's probably crashing for some other reason. I don't think
> spring version should break anything, it's more likely an actual bug.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mtroyka/Zero-K/commit/1cf8399446fb56d70c06c5994949614f73f65e87#commitcomment-11520024>
> .
>


I assumed that the one in run would be higher priority, but in my testing with the JS driver it didn't seem to be. Run seemed to only be used when there was no default db specified on the connection.


This should inherit from AudioStream instead of decoding the entire thing to a buffer first.


Re https://github.com/angular/material/issues/2618#issuecomment-109097066: this label still throws a warning. I'm not actually sure what the interaction should be for this example, if any. It's not reachable by a keyboard or a screen reader, so I'm not sure it even needs a label. It definitely should not throw a warning, in any case.


Violeta and Frida are currently working on the Android UI for Hood, and one of the things they're doing is to ensure that the sizes of all elements are in relative terms so that they look good in different screen sizes. Feel free to talk to them and/or ask them for advice on that to ensure that Omni also looks good on all sorts of devices.


Awesome. I can actually see the share stuff from and it lets me share to the main and other place. Good Job and keep it up!


Also Jatinder and Arvind stepped down


revert test


Hmm the JavaScript driver seems to overwrite the 'db' optarg if a default db is explicitly defined on the connection. It doesn't check whether the user has set the 'db' optarg in run or not.


Please check this. Summary:

(a) Even with overlap_size = 0, the edges will overlap.
(b) Number of local DOFs got reduced. There was a bug previously which included them at upper boundaries of the subdomain that were in the patch interior.


This is great! My photos that I took where stored, but then I pressed the other buttons and it wasn't working. Don't know why...


The Python driver appears to use the one passed to `run` rather than the one set on the connection.
So there the priority is
1. r.db
2. run optarg
3. connection.use


This causes two problems:
* this won't compile if build with python and testing support as there is no CMakeLists.txt in `test/py/`
* the Debian package generated by cpack cannot be installed due to the new white space in the package name (`"Apache Thrift"`)


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


1700 is a lot more plausible than 1997, but the [text](http://w3.jcommops.org/FTPRoot/Argo/Doc/argo-quality-control-manual.pdf) definitely says 1997 - will leave this as 1700 if we're sure.


this one was really odd. createAnswer should never have accepted this.


Yeah sorry I didn't do a code review but I figured it was mission-critical


OK Thanks Sultan! I'll have a try :)
However it still download the full package.....
Could you tell me how to make a delta OTA package? Every time I compile CM11, the "cm_villec2-ota-xxx.zip" is the same size as the full package.


Ok, I simply noticed that viewing sessions by collection in the UI was missing project information.  This meant the project column in the sessions list was blank and so was the project detail accordion.  Perhaps we need to conditionally expose this information?  If you have access to the project already, I would think you should be able to see it.  If you don't, then we should maybe adjust the UI?  I guess it's probably fine for the demo.


Awesome plots!! Ouch about no one testing. (Which is just reminding me to feel guilty that broom has only like 20-25% test coverage).

> Would also appreciate your interpretation of the fact that essentially every single packages has been downloaded in 1 month. Does that seem right to you?

Sure- after all, we personally downloaded every single package multiple times in April. That was >30 days ago, but it's entirely possible others (perhaps a CRAN maintainer) have done similar tests.

If we're serious about analyzing downloads, I think we need to process the [CRAN download logs](http://cran-logs.rstudio.com/) instead of using the API, as it also provides the IP addresses of each download. That would let us distinguish between # of downloads and distinct IP downloads (and also give information on each ). To get a sense of why this helps, take a look at ggplot2's downloads over the last year with [this app](https://dgrtwo.shinyapps.io/cranview/). On June 26, someone installed ggplot2 about 25,000 times (a testing script run amok or something), but by checking the log we can see it's just the same person.

Sorry, random travel drama getting in the way of working right now but keep trying to make extended time for this!!


Are you saying we wont be needing to check for providers this way in case we decided to drop support for pre 1.9.x versions?


stole it fair and square appointment_report.php


Ordering here looks good too.


Interesting. Translating a space :)
Try to do this instead:
<th> </th>


Fixed, this kind of formatting style reminds me that uncrustify config file you mentioned before...


Well, That is what I thought until I got an error complaining about constraint violation exception, provider not found. It is  hacky way of getting around this exception. Other than that I agree we don't need to save the providers at all.


done


The white space around `-` right? Fixed.


Fixed.


remove text(). only used for when sending variable to html


need add Requirement section:
specify the requirement for webCL


Sorry for the off-topic comment, but this is a function that we should kill. Notice that here we are changing the "constants" and maintaining global variables.

Here is the point were we could create an object (and proper constructor to do the job that we do here) to contain the state that we need.

My rough sketch would be something like this (it doesn't work, it's just a sketch of the idea):

https://github.com/rbrito/edx-downloader/commit/3ce26490a7c6fe1b9982df8fe63b3e8c3eb37ac0

OK, enough for the off-topic comment.


put text() around this also:
``````
echo text(xl_appt_category($appointment['pc_catname']))


text()


Excellent. Keeping the code tidy is lovely. ;)


remove text()
use xlt() for the Yes and No translation


Could you please undo the change in all other files under src/inc/... ? It should not be needed in any of them.


text() around $j


Removes the errors from the previous save attempt.


attr()


Nuttx build, gcc  4.8.4 20140725 (release) [ARM/embedded-4_8-branch revision 213147]

Why the addition of the long long x?


Why `$0` is before `endsnippet`?


Could you please delete this method in pal.h as well?


Ensure keep the SQL comments less than 60 characters.


typo


Also, don't forget database.sql stuff


What target is this for? NuttX? What compiler are you using clang or gcc?

On x86_64 gcc uses 1 and 4 byte alignment and clang uses 2 and 3 (clang
produced better data packing). I described the issue in
systemlib/param/param.h:
```
// GCC 4.8 and higher don't implement proper alignment of static data on
// 64-bit. This means that the 24-byte param_info_s variables are
// 16 byte aligned by GCC and that messes up the assumption that
// sequential items in the __param segment can be addressed as an array.
// The assumption is that the address of the second parameter is at
// &param[0]+sizeof(param[0]). When compiled with clang it is
// true, with gcc is is not true.
// See https://llvm.org/bugs/show_bug.cgi?format=multiple&id=18006
// The following hack is for GCC >=4.8 only. Clang works fine without
// this.
#ifdef __PX4_POSIX
                                __attribute__((aligned(16)));
#else
                                ;
#endif
```

I figured out what was happening using objdump -h on the final binary
(mainapp for posix build):

```
$ objdump -h ./Build/posix_default.build/mainapp

Sections:
Idx Name          Size      VMA               LMA               File off Algn
...
 14 __param       00002760  000000000048bdf0  000000000048bdf0  0008bdf0  2**4
                  CONTENTS, ALLOC, LOAD, READONLY, DATA
```
This shows that the __param segment is 16byte aligned, so a pointer will be
properly aligned.

```
$ objdump -x ./Build/posix_default.build/mainapp | grep param_start
000000000048bdf0 g       __param 0000000000000000              __param_start

$ objdump -x ./Build/posix_default.build/mainapp | grep param_end
000000000048e550 g       __param 0000000000000000              __param_end

$ objdump -s --start-address=0x48bdf0 --stop-address=0x48e550
./Build/posix_default.build/mainapp

./Build/posix_default.build/mainapp:     file format elf64-x86-64

Contents of section __param:
 48bdf0 33c14700 00000000 01000000 00000000  3.G.............
 48be00 17b7d138 00000000 00000000 00000000  ...8............
 48be10 eac34700 00000000 01000000 00000000  ..G.............
 48be20 0ad7a33d 00000000 00000000 00000000  ...=............
 48be30 0cc74700 00000000 01000000 00000000  ..G.............
 48be40 bc74133c 00000000 00000000 00000000  .t.<............
```
This should give you some idea if your data looks sane.


This is a comment on my test


Could not create DockerContainerizer: Failed to execute 'docker version': exited with status 1


che, se jodio la musica :S


Script portion of #109


Is there a way to include an en-dash? do you need the html entity? the unicode #?
In proper typographic style, and en-dash with no spaces should be used to represent duration from one time to another...


Agregado requerimientos de issue https://github.com/CamilaAlvarez/IndicadoresDCC/issues/28


done


+ #  younge


Sounds plausible.


@robbiecarlton note that i've skipped User here and gone straight to Device via user_id; fewer queries


README.md 에 35fba0d


i removed the `post.relations.communities` check because it just shouldn't ever happen -- if it does, it may be a sign of more mixing of transaction-wrapped and non-transaction-wrapped DB queries, e.g. the two `save` calls in `Device#sendPushNotification`


I'm checking if the long long is needed. I don't even recall adding that.


you never know ascii 32 could be different


Shouldn't we use a new doc in that case?


The most proper way would require changing the order in which you process the cone. Notice that in your case screaming from left to right and from right to left will bash different tiles.

At the moment sounds aren't stopped by walls, so I'd say that it's OK for both bashing and stun to get through.


`vim.current.window.cursor = (line, column)`.


It is not used in nuttx and is the case of the issue. But I have to run a few builds to see why.


@d3agle Hey, catch! :D


We can whip add_edit_event into shape, though. (The reason I changed this refresh method is it was hard to understand and was going through the loop every second.)


it seems not used - Seeing there is no corresponding PARAM_DEFINE_XXX  to define one.


still nice :)


Seeing this commit here makes it look like you didn't rebase properly. https://github.com/OpenRA/OpenRA/wiki/Contributing#updating-a-pull-request has some instructions.


Testing now :)


boxed_number_simplifications (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (351 of 351 tests passed)

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


ah, https://github.com/w3c/webrtc-pc/pull/171 brings back answer options so no action required. Thanks @jan-ivar


I am pretty certain the change won't be that great initially. Reconnection of the clients should have a noticeable improvement, though, because the buffers should be held in the buffer manager's pool of extra buffers that are freely available for clients that need one.


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/be159759ea03c102893cf45119a11e364f728feb/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/be159759ea03c102893cf45119a11e364f728feb/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/be159759ea03c102893cf45119a11e364f728feb/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/be159759ea03c102893cf45119a11e364f728feb/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/be159759ea03c102893cf45119a11e364f728feb/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (351 of 351 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)


添加注释？


Why does this always happen to me.....


I dropped this to Fred on IRC but in case he doesn't see it: this doesn't work when running in batch mode. The `hhvm` arg is handled elsewhere in batch-run.php and not passed into `$argv` or available in `getopt()` at this point. So proxygen and precompile end up un-overrideably `FALSE` with this change. Oops.


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (351 of 351 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)


under 60


Menyesuaikan perubahan di beberapa kalimat.


boxed_number_simplifications (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (351 of 351 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 did a posix build without it as well and it built fine. Seems like that line can be removed.


done


fixed


It was probably just a holdover from debugging the alignment issue and I had tried to increase the size of the struct. I clearly forgot that I added it. Apologies.


1.git
下载代码
git clone https://github.com/zhufengnodejs/js2th.git
修改代码
1. 把代码添加到暂存区
git add -A
2. 把代码添加到历史区
git commit -m'init'
3. 把代码提交到github上
git push
4.从git上修改代码
git pull
#nodejs介绍
1. 可以让js运行在服务器端的平台
2. 实现了一些文件系统，操作系统API等核心模块。
#node.js特性
1.


added onsubmit='return top.restoreSession()'


Oops it fixes #41, not #40.


super cool!


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (351 of 351 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)


Noticed an immediate performance increase! :D


It needs to be hires input as well.


boxed_number_simplifications (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

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


With reconnection of clients, that is.  Initially, (Starting 100 clients with server not listening) The CPU usage is still quite high.  However, when reconnecting them all at once, the CPU usage is seemingly halved.


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (351 of 351 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)


yes, but currently I use python to generate from python lib path


Indentação


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (351 of 351 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)


:clap: :clap: :clap:


CHANGELOG:

- Enabled completion for `/events/EVENT/ register_alert` command -- tab-completes possible email addresses from the Ravello account
- In doing above, I fixed a bug in the `register_alert` command (it wasn't working with other emails at all)
- Enabled completion for `print_event_names` and `print_registered_alerts`
- Random doc cleanup
- All of the commands in `/billing` take an optional `outputFile` argument ... changed that to be the LAST arg for each command, instead of the first. More intuitive now, e.g., the first arg of the `select_month_summary` command now is `month`.


\"Bate-papo\"


Good! I don't really think that the CPU usage can be reduced by much initially. I still hope to try to find that out. How is the memory consumption after full client disconnection on the server?


CBECC (kbenne) - 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)


the code name is tblte


it don't like
echo "    <option value='attr($provid)'"; or echo "    <option value=attr('$provid')";


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (351 of 351 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)


Go team! :D

On Wed, Jun 3, 2015 at 8:55 PM, Andrea Coravos <notifications@github.com>
wrote:

> Host new blog using Jekyll via @dannyshafer tutorial
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/acoravos/coravos/commit/8b94ff1f939827aa4b97ea9112449af6fa94a9d4
>


done
 echo ">" . text($urow['lname']) . ", " . text($urow['fname']) . "\n";


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


CHANGELOG:

* `/blueprints/BP/ delete_bp` command no longer backs up the blueprint if you cancel the deletion
* Bumped TTL for userCache and alertCache up to `120 sec` from `60 sec`


Because I'm dumb :) Fixed.


@shaunc: I think I have the library dealing well with the copy protocol, thanks for the tip on those two async messages.

I've added support and testing for queries where a single result row outstrips a single call to _transform. It works great in both 0.10 and 0.12, but in 0.12 98% of the time it throws a "connection terminated" error. Anything jump out at you that would be causing it?


boxed_number_simplifications (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

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


After disconnecting all the clients from the server, the memory isn't freed.  When reconnecting the clients, memory goes down a little bit while the clients are connecting and goes back up to where it initially was when all the clients are connected.  Exiting the server application, the memory is freed.


You should add ```description``` here, where the ```description``` is ```keyword.doc.split('\n')[0]```. It's standard robot framework practice for the first line in a docstring to represent a brief summary of the keyword.


Lol who wrote this? I really hope it was me.


Meet Myah :)

Chris

> On Jun 3, 2015, at 6:15 PM, Mike Sinkula <notifications@github.com> wrote:
>
> That dog looked pretty sad. :-(
>
> —
> Reply to this email directly or view it on GitHub.
>


Why did you commit this?


update issue #19


@KradekMFC Interesting change. What prompted the need for it? I've never come across anything longer than a single websocket message.

* The 4 digits of msgLength can carry a number up to 9999, so I assumed they would send anything up to that length as a single websocket message. But your buffering change suggests that MFC actually splits messages across buffers long before that limit?
* Do you have any way to trigger the receipt of a mfcMessage split across multiple websocket messages, so that I can verify this change? I suppose maybe asking for METRICS via the chat method will send a massive blob of JSON data over the wire and may need this change.
* That JSON blob from chat-METRICS would probably easily exceed 9999 bytes. I wonder what MFC does with messages >9999...

I tried to investigate how the official site does it...

In the official top.html, they instantiate a message queue:
```
var g_oCmdQ = new MfcQueue();
```

Its implementation is in mfccore.js, and it's just an array, where each entry holds a single COMPLETE mfc message WITHOUT the "0000-9999" integer at the start of the message (i.e. a ping message in that queue would be "0 0 0 0 0", with no length property at the start). The queue is being processed on a 10 millisecond timer:
```
function MfcQueue() {
    this.aCmdBuf = new Array();
    this.nInterval = null;
    this.nSuspend = null;
};
MfcQueue.prototype.Queue = function(sCommand) {
    if (sCommand) this.aCmdBuf.push(sCommand);
};
MfcQueue.prototype.Process = function() {
    while (this.aCmdBuf.length > 0) {
        ParseLine(this.aCmdBuf.shift());
    }
};
```

The ParseLine() function that is run on each queue message is in top.html and it instantly assumes that a single array entry in the queue is a single, complete, unbroken MFC Message, without any length property at the start..

So to know how MFC officially buffers/reconstructs messages/deals with lengths >9999, we need to know how they add a single, complete MFCMessage to the queue. They add to the queue in top.html via this:
```
function ParseCommand(sCommand) {
    g_oCmdQ.Queue(sCommand);
}
```

Strangely, I cannot find anywhere that calls ParseCommand, so I don't know where they add to the queue. I cannot find it in mfccore, top.html or player.html... any ideas? I even looked for pure ".Queue(" calls to see if any place directly adds to the queue and they don't. So the best lead is ParseCommand()... If we can follow this trail to where ParseCommand() is adding to the queue, we can emulate the *exact* method that they're using for dealing with incoming socket messages.


No worries - I had suspected that.

Without the long long size looks to be 12 bytes Aligned on 4 byte boundaries. gcc 4.8 arm
```
_param         0x080d734c     0x180c
                0x080d734c                __param_start = ABSOLUTE (.)
 *(__param*)
 __param        0x080d734c        0xc /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/drivers/mkblctrl/module.pre.o
 __param        0x080d7358       0x30 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/examples/flow_position_estimator/module.pre.o
 __param        0x080d7388        0xc /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/examples/rover_steering_control/module.pre.o
 __param        0x080d7394       0x48 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/lib/launchdetection/module.pre.o
 __param        0x080d73dc       0x9c /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/attitude_estimator_ekf/module.pre.o
 __param        0x080d7478       0x48 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/bottle_drop/module.pre.o
 __param        0x080d74c0       0xb4 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/commander/module.pre.o
 __param        0x080d7574       0x90 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/controllib/module.pre.o
 __param        0x080d7604       0xe4 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/ekf_att_pos_estimator/module.pre.o
 __param        0x080d76e8      0x144 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/fw_att_control/module.pre.o
 __param        0x080d782c      0x390 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/fw_pos_control_l1/module.pre.o
 __param        0x080d7bbc       0x54 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/land_detector/module.pre.o
 __param        0x080d7c10       0x3c /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/mavlink/module.pre.o
 __param        0x080d7c4c       0xf0 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/mc_att_control/module.pre.o
 __param        0x080d7d3c       0xf0 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/mc_pos_control/module.pre.o
 __param        0x080d7e2c      0x180 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/navigator/module.pre.o
 __param        0x080d7fac      0x114 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/position_estimator_inav/module.pre.o
 __param        0x080d80c0       0x18 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/sdlog2/module.pre.o
 __param        0x080d80d8      0x954 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/sensors/module.pre.o
 __param        0x080d8a2c       0x90 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/systemlib/module.pre.o
 __param        0x080d8abc       0x24 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/uavcan/module.pre.o
 __param        0x080d8ae0       0x78 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/vtol_att_control/module.pre.o
                0x080d8b58                __param_end = ABSOLUTE (.)
```

With the long long there was a fill emitted and the size of looks to be 16 on 16. SO I would say that is the issue.

```
__param         0x080d733c     0x2014
                0x080d733c                __param_start = ABSOLUTE (.)
 *(__param*)
 *fill*         0x080d733c        0x4
 __param        0x080d7340       0x10 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/drivers/mkblctrl/module.pre.o
 __param        0x080d7350       0x40 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/examples/flow_position_estimator/module.pre.o
 __param        0x080d7390       0x10 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/examples/rover_steering_control/module.pre.o
 __param        0x080d73a0       0x60 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/lib/launchdetection/module.pre.o
 __param        0x080d7400       0xd0 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/attitude_estimator_ekf/module.pre.o
 __param        0x080d74d0       0x60 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/bottle_drop/module.pre.o
 __param        0x080d7530       0xf0 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/commander/module.pre.o
 __param        0x080d7620       0xc0 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/controllib/module.pre.o
 __param        0x080d76e0      0x130 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/ekf_att_pos_estimator/module.pre.o
 __param        0x080d7810      0x1b0 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/fw_att_control/module.pre.o
 __param        0x080d79c0      0x4c0 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/fw_pos_control_l1/module.pre.o
 __param        0x080d7e80       0x70 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/land_detector/module.pre.o
 __param        0x080d7ef0       0x50 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/mavlink/module.pre.o
 __param        0x080d7f40      0x140 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/mc_att_control/module.pre.o
 __param        0x080d8080      0x140 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/mc_pos_control/module.pre.o
 __param        0x080d81c0      0x200 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/navigator/module.pre.o
 __param        0x080d83c0      0x170 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/position_estimator_inav/module.pre.o
 __param        0x080d8530       0x20 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/sdlog2/module.pre.o
 __param        0x080d8550      0xc70 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/sensors/module.pre.o
 __param        0x080d91c0       0xc0 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/systemlib/module.pre.o
 __param        0x080d9280       0x30 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/uavcan/module.pre.o
 __param        0x080d92b0       0xa0 /home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/Build/px4fmu-v2_default.build//home/david_s5/Desktop/dev/PX4/repos/mainline/Firmware/src/modules/vtol_att_control/module.pre.o
                0x080d9350                __param_end = ABSOLUTE (.)

```


Yeah, I know it will be freed on application exit. The frustrating problem is that, even if GC.Collect() is called as it was in the implementation in MaxXor/master and in different spots I just tested, the garbage collector still doesn't give love to the LOH. It does not do much with it unless it really needs the memory, then it will finally completely collect... Not really much of a problem, but it is still rather irritating.


Note- these paths need to be generalized for basespace for each respective transcriptome , fastq samples, transcriptomes fasta files desired for merging, and output.


yeah, good point :+1: will fix


天青色等烟雨，而我在等你


this commit is not needed, the indexKallisto merges on the fly.


test


Perhaps we could use a pack, aligned on 4 and ccasert on the size mod 4  to let the compiler tell us if we mess it up.


@kanitw why are we going backwards? Can you just send pr without version bumps?


test~~~~~~


Good


这是是？


updated.


Just to be complete... after I wrote my last note I actually read the doc in 49.2.6. [Asynchronous Operations](http://www.postgresql.org/docs/9.4/static/protocol-flow.html#PROTOCOL-ASYNC) ... in theory besides "noticeResponse" and "parameterStatus" there is also "notificationResponse" (For notification from "notify" command -- code "A") with proviso:

> Note: At present, NotificationResponse can only be sent outside a transaction, and thus it will not occur in the middle of a command-response series, though it might occur just before ReadyForQuery. It is unwise to design frontend logic that assumes that, however. Good practice is to be able to accept NotificationResponse at any point in the protocol.

:) ...


@KradekMFC Oh shit... it just hit me that they probably call ParseCommand() directly from the main "server communication" flash object... so they're calling it from inside a .swf, and that one probably doesn't connect to a websocket server, so it'd be pointless to decompile it to see if it buffers.

That brought me to the idea of checking the mobile client, and yeah I see loads of calls to ParseCommand() in their websocket message listener. The code is quite heavily obfuscated/compressed though, and it's being called all over the place and will require untangling, so I'll read through it later.

Have you already investigated this and know if the mobile/websocket client is doing message buffering, and how it deals with messages >9999 length? I am super curious what they do with "mobj.php" metrics received via the alternative chat system, since those probably exceed even 9999 bytes.


Erm. This is for my bio?


嗷嗷


Perhaps it's not too much of a problem if the memory isn't freed upon client disconnection.  If, for any reason, the server needs to stay active after disconnection of clients, any client that connects after that would still re-use already allocated buffers, right?


1) This commit changed schema. I wonder, is this breaking any compatibility with tools like Gesior?
2) I've always avoided to change anything at the database because of compatibility. But if we are doing this, wouldn't better to implement as well the support to string values at getPlayerStorageValue? And adding storage values for accounts (so we could save information at players accounts, instead of only being able to store it at specific characters).


If we use pack, won't that slow access when types (like pointers) are not properly byte aligned? If we always use 2**4 alignment, won't that be sufficient?


Yes, that is right. The buffers are freely-accessible to the clients whenever they need them and the BufferManager has an allocated buffer that the client has given back.


I'm not actually an expert -- at least not yet :) ... I'll try to spend some time with the debugger and see what I can root out. In general it would seem that the streams implementation has changed in 0.12 and it may follow the doc still but perhaps because the implementors are trying to squeeze out better performance some "undocumented features" found by users through experience are no longer present.

I won't have any time to do anything until at least Monday though as I have a deadline. Good luck!


Definitely fine for the demo. Beyond that, we first need to figure out the exact semantics of sharing, which aren't always obvious at the margins.


The relevant source from them for parsing is this:

        this.RxData = function(e) {
            if (e && e.data && e.data.length)
                for (this.receiveBuffer += e.data; ; ) {
                    if (this.receiveBuffer.length < 4)
                        return;
                    var t = 0 | this.receiveBuffer.substr(0, 4);
                    if (0 == t)
                        return Log("Desynced from server: " + this.receiveBuffer.length + " / " + this.receiveBuffer.substr(0, 4)), void this.Disconnect();
                    if (this.receiveBuffer.length < t + 4)
                        return;
                    this.receiveBuffer = this.receiveBuffer.substr(4);
                    var n = this.receiveBuffer.substr(0, t);
                    ServerSocket.eventCache.push(n), this.receiveBuffer = this.receiveBuffer.substr(t)
                }
        },

What triggered it for me is I have some code that wants to get the ROOMDATA messages so that you can get room counts. You don't receive those messages unless you join a particular chat channel. You can join that channel by sending this message:

        socket.send(new MFCMessage({ Type: MessageType.FCTYPE_ROOMDATA, From: socket.getSessionId(), To: 0, Arg1: MFCChatOpt.FCCHAN_JOIN, Arg2: 0 }));

Room channel messages are basically long arrays of alternating values, first the broadcasterId then the room count.

Joining this channel also gives you access to countdown update messages.

As you can see from above, it doesn't look like they do anything client side to check for lengths greater than 9999.


refs #1821 입니다. -_-;


One thing I was wondering about was whether the "end" event as a query and the "end" event as a transform stream aren't colliding. One thing to check is whether this is called after flush? (always?)  -- eg debug by putting console.log() in _flush and here...


What's the status of this PR? Good to merge?


@Temptin Regarding the METRICS message, they don't actually send it through the chat server. They send you a pointer to a temporary file on the server, and then you grab it separately.


Why `-DMY_PTHREAD_FASTMUTEX` option is also removed? (I couldn't see `Bug#20590904`.)
The option is needed for [Mroonga](http://mroonga.org/) that is a storage engine.


Why not?


Thanks, Jack. Would you mind to change the line 4 "count" to "length"?


You must also change the grouping character. You can't have the same separators for the grouping and the decimals: `#.###,## €` OR `# ###,## €`. The thin-space character used between the first and second `#` is known to be a universally accepted grouping character. so it is acceptable, but I believe the full-stop `.` character is Germany's preferred grouping character for thousands.


See https://github.com/tractorcow/silverstripe-cms/commit/703d50ec406aa01169e41aa9f4c3a60a7cafd68f for the reason this used to be backwards.

This is so bad in many ways. =(


See https://github.com/tractorcow/silverstripe-framework/commit/fa35d1d0f201e8990139839d218f68297e2c3c27 for the reason we are setting $newClassName to the old class. So backwards....


@kirillseva any thoughts on answer to exercise 5?


Lol true that


I didn't know about the code name. ..illl change it later today


Awesome.  Tomorrow, I'll do some more tests with clients and multi-tasking like file-transfers, etc...  but I'm quite tired so I'll be passing out.  Good job on this!  I can't see this implementation causing any problems, let's cross our fingers :+1:


You're absolutely right. That's what's happening when I code in hurry. Thanks Jirko!


Wonderful. Have a good sleep. Once you give me the okay tomorrow I will send out a pull request.


As a little note, it doesn't matter that much if the server is consuming more memory than it should when it will fix its own memory problem if necessary. Most of the memory will be needed in buffers, but even then this change will reduce the memory consumption a good bit too, since these buffers are not going to be wasted on client disconnection and disposal. :)


Yours?

--

Mike Sinkula
Professional Twelve Year Old
http://www.mikesinkula.com

> On Jun 3, 2015, at 9:03 PM, cwlock <notifications@github.com> wrote:
>
> Meet Myah :)
>
>
> Chris
>
> > On Jun 3, 2015, at 6:15 PM, Mike Sinkula <notifications@github.com> wrote:
> >
> > That dog looked pretty sad. :-(
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> —
> Reply to this email directly or view it on GitHub.
>


CBECC (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2042 of 2158 tests passed)

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


@indutny
offset += [output_ length] - offset - stream->avail_out;

it may be simpler to write it as
offset = [output_ length] - stream->avail_out;

That way it is easier to understand.


`stringsAsFactors=FALSE`  what's the use of this?


yes, but the whole dialect.zep contains this declaration "string escapeChar = null" in all getSqlExpressionXXX methods , this change is only to be coherent.


Moment.js --> already exists in base template --> possible duplicate import to page


I was just suggesting some portuguese translation small changes, because sometimes the syntax looks a bit too english-like and it's harder for understanding, I also added some articles, wich are very important for portuguese, but were missing, along with other small changes. There was only one actual mistake I believe, at line 91, it says "Java" when I think it's supposed to say "JavaScript".

Eu tava sugerindo umas pequenas mudanças na tradução pro portugues, porque às vezes a sintaxe tava muito parecida com a do inglês e assim fica mais difícil de entender, eu também acrescentei uns artigos, que são bastante importantes pro português, mas tavam faltando, e também fiz umas outras mudanças pequenas. Acho que só tinha um erro mesmo, na linha 91, que diz "Java", mas eu acho que deveria dizer "JavaScript".


Great @YanMagale  :+1:


Formatting note: need a space after the opening paren `(` in the `for` condition.