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


Sad..lol :D


parsing_reorg (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: Build Failed

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


Easiest way to leave current channel is `self.emit('leaveChannel',self.data.activeChannel.name)`.  `leaveChannel` already checks to see if the channel name is `friends`.


@jenius It's not presumptive to assume that every use-case will involve creating a directory in the user folder?  Moreover, shouldn't distinct instances of Sprout exist in distinct places?


Nice try dude, but I got
```
nginx_1      | 2015/05/01 13:59:30 [emerg] 1#0: unknown directive "more_set_headers" in /etc/nginx/conf.d/registry.conf:34
nginx_1      | nginx: [emerg] unknown directive "more_set_headers" in /etc/nginx/conf.d/registry.conf:34
```


parsing_reorg (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)


s/take/get/ ?


@jenius I think, ideally, the `describe` block would have an after hook for re-installing and defaulting the `roots-base` template.


This signs the summary, not just the deltas, right?


Will this work in case of Exceptions?

hGetContents needs double the memory of the result, maybe there is a better way.


See [this](https://github.com/jenius/roots/pull/632#commitcomment-11001620).


$this->brandUrl === false ?


This and `system` should be refactored, but I can’t find a non-awkward way to do it.


This one should be `_ostree_repo_file_replace_contents` now to ensure the fsync policy is honored, and to make use of `*at()`*.


Dunno....   What are the effects of these changes? I am unable to build a kernel to test now... Could you please tell me?


@myriameayman  perfect


align the divs opennings and closings


parsing_reorg (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (344 of 346 tests passed)

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


Woops my mistake. Never mind about the reference xD


Great! I think write also in README.md about new licence.


Ha! I'm ready to merge this as is, but it's an interesting wrinkle...


We all agreed on changing the password to wodercode, so leave it as wondercode so it wont conflict


Done


Seems to me that trim is the obvious choice here, thanks @dstockto !


A parameter's name 'condition' is confusing me. Can it be described multiple means?


!Fucking :D


2422 ish words at this point


Need to remove the LDLIBS-y here and return back to LDLIBS


The reason was to improve the use of HTML elements and their behavior. The "a" element is an element to navigate to other pages or to another section in the page, so its use in the calendar wasn't correct. In fact, it had the attribute "role" with "button" as value, so the it is clear that the HTML element should be "button".


Good point @fxbenard, silly oversight on my part. Will do another PR soon.


Too many ? here - 12 vs 9 fields


Yes it certainly would :)


I'm not aware of any unit tests failing that are related to it.  I haven't gotten on it yet, unfortunately.  My focus has been on getting Android stuff working right, and that ended up being more complicated than expected.


I'm not going to accept this commit because I use `:`. If you want to make it configurable by metadata that's ok, but `:` is staying as the default.


DMA2_Stream6 is only ever used for this purpose, so if it is disabled when not in use it doesn't matter.
I suggest moving the line:
DMA2_Stream6->CR &= ~DMA_SxCR_EN ;              // Disable DMA
to be always executed in the interrupt just before the call to setUpPulses(). The DMA will have the whole of the execution time of setUpPulses() to actually stop.
This should be better than waiting in an interrupt for something to occur (EN bit going low).


really ? damn lg-devs is worse than i thought jk


add a description here main div


Looks like a duplicate require from line 2


deal with other forms of parsing


See if you can do this entirely within the view - within `ng-click`


the change function in the db/migrate file had nothing in it; you
physically couldn't add a bike with the old copy, so i just generated
migrations

On Fri, May 1, 2015 at 9:58 AM, Franklin Yu <notifications@github.com>
wrote:

> What is the difference between serial_num and bike_serial_num?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/betsy-tse1/bubhub/commit/966b7bb30afe38c6ecc2ca7de9dabaf4405fc458#commitcomment-11001577>
> .
>


done


I mentioned this the other day.
https://www.youtube.com/watch?v=A3s-dFev2TM
Code:
https://github.com/rdegnan/yuki
Very cool stuff.
-Paul

On Wed, Apr 29, 2015 at 11:31 AM, Carmelo Piccione <notifications@github.com
> wrote:

> So, my original implementation still wrapped the string in a protobuf
> encoding (not a big deal but thought it was undesirable for you because of
> visibility or performance reasons).
>
> This new one captures the spirit of your implementation by providing more
> a more abstract form of serialization. See the type Protobuf_capable.Raw_S
> :
>
>
> https://github.com/struktured/riakc_ppx/blob/master/src/lib/protobuf_capable.mli#L13-L18
>
> which allows Core.Std.String to directly implement the key signature
> without protobuf encoding.
>
> The api didn't break however, as I made a functor called
> Protobuf_capable.Conversion which converts a Protobuf_capable.S to a
> Protobuf_capable.Raw_S automatically.
>
> Your desired version with a key as a primitive string is now done with
> Cache.Make_with_strring_key:
>
>
> https://github.com/struktured/riakc_ppx/blob/master/src/lib/protobuf_capable.mli#L13-L18
>
> Hope this clears things up.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/struktured/riakc_ppx/commit/3ad55dc3ea2d8a828f9997b205cfeac9c5d84061#commitcomment-10966248>
> .
>


That's fine, but we still do need this to be added!


This space is actually taken care of with the third parameter to the function, so the forced space really isn't needed here.


serial_num isn't being looked for when you create a bike through the website

On Fri, May 1, 2015 at 10:16 AM, Betsy Tse <btse1@binghamton.edu> wrote:

> the change function in the db/migrate file had nothing in it; you
> physically couldn't add a bike with the old copy, so i just generated
> migrations
>
> On Fri, May 1, 2015 at 9:58 AM, Franklin Yu <notifications@github.com>
> wrote:
>
>> What is the difference between serial_num and bike_serial_num?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/betsy-tse1/bubhub/commit/966b7bb30afe38c6ecc2ca7de9dabaf4405fc458#commitcomment-11001577>
>> .
>>
>
>


you're good to go now :+1:  @ebrahim-elgaml


The time difference for loading the Middle Limitrophe is 11 seconds before implementation, versus 47 seconds on QA where this change is implemented


Wow. This rank idea is pretty clever.

May I suggest something which would be King centric ?
Something similar to...

  m.value = PieceValue[MG][pos.piece_on(to_sq(m))]
        - 200 * distance(pos.king_square(pos.side_to_move(), to_sq(m));


Se o objetivo do cenário é criar grupo de pesquisa, as ações realizadas para realizar a criação devem fazer parte do when e não do then.


`condition` accept any value that can be evaluted by `bool`. so yes, it described various meaning.


Isso é a única coisa que deveria estar no then.


I think a paramete `block=True` makes sense here: I just tried this feature, replying to a message where i don't have the senders key in my chain. The error message pops up, then the editor is started and takes over my terminal. once i exit the term, the notification got removed due to its timeout. It's really difficult to read the error so quickly


something get wrong, i will ask hanan sunday morning


If you want to make it configurable then parse the metadata in `internalreferences.main` and send it as an init argument to `ReferenceManager` (like I've done with `autoref`).


Do we have a use case where it wouldn't? I feel like this is a great place to store config, which is why we chose it. Previously, it wasn't even configurable at all, and nobody has raised an issue with that yet. I feel like making it configurable is a great call, but it would remove a lot of extra logic from a lot of peoples' integrations if we could provide at least some way to hide away and abbreviate this large block of logic. Having it as a default is one option, maybe providing a helper would be another?

I just see this being replicated by every single sprout integration if we don't, which would make me sad.


Rever uso de "I can" em todos os steps


i'm pretty sure someone just edited the schema.rb manually but you're not
really supposed to

On Fri, May 1, 2015 at 10:17 AM, Betsy Tse <btse1@binghamton.edu> wrote:

> serial_num isn't being looked for when you create a bike through the
> website
>
> On Fri, May 1, 2015 at 10:16 AM, Betsy Tse <btse1@binghamton.edu> wrote:
>
>> the change function in the db/migrate file had nothing in it; you
>> physically couldn't add a bike with the old copy, so i just generated
>> migrations
>>
>> On Fri, May 1, 2015 at 9:58 AM, Franklin Yu <notifications@github.com>
>> wrote:
>>
>>> What is the difference between serial_num and bike_serial_num?
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/betsy-tse1/bubhub/commit/966b7bb30afe38c6ecc2ca7de9dabaf4405fc458#commitcomment-11001577>
>>> .
>>>
>>
>>
>


Minor point, but please leave whitespace around methods.


Yep, and this code is super WIP at the moment.


wow absolutly amazing!


And whitespace around spec blocks.


Fazer login é pré-requisito para usar o sistema, certo? Então isso vai se repetir em outros lugares... checar para evitar repetição de código.


Thanks @huoxito


@super132 casting is not required here


Sim, se de fato a opção deve ser parametrizável. Mas antes chegar necessidade de parametrizar isso.


@super132 too much indentation (8 spaces instead of 4)


I hate TextWrangler.


I see, so you would throw the trait onto the request.

I don't think that would work. I see that the filter modifies the `getAttribute` method but the `getValidator` method here is using the `Request` instances `all` method. Request ultimately appears to be accessing `all` from a `ParameterBag` which just passes all the parameters directly through.

So you would still need to modify the getValidator to use something in between instead of `$this->all()`.

Thanks @GrahamCampbell I am sure that would work well. I actually put the snippet from the forum post I linked into my `\App\Http\Requests\Request` class it looks like [this](https://gist.github.com/EHLOVader/43de7cea4bdc65c2f86e) I am probably doing it wrong but am still learning. I would love to see an example of your use of a decorator though.


Trata-se da String "create", tem que ser com aspas mesmo.


Is it something u want disabled all the time or just for debug


Sorry, I didn't intend to make this a part of the pull request. (I'm new to
github.)

Is the proper thing to do to make a separate branch and issue the pull
request from the branch?

On Fri, May 1, 2015 at 10:19 AM, aaren <notifications@github.com> wrote:

> If you want to make it configurable then parse the metadata in
> internalreferences.main and send it as an init argument to
> ReferenceManager (like I've done with autoref).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bwhelm/pandoc-reference-filter/commit/5bc2933f57c86069cdc28d81148cec2c0c1c89d3#commitcomment-11001829>
> .
>


Yeap, Thx. It will be better if there are documents. :)


it also wasn't just for the serial number. it was for the height and
description as well. the only thing that successfully passed from the
website was the bike ID, so i just made it so you can add a bike and see
what the bike_list page looks like with bikes in them

On Fri, May 1, 2015 at 10:20 AM, Betsy Tse <btse1@binghamton.edu> wrote:

> i'm pretty sure someone just edited the schema.rb manually but you're not
> really supposed to
>
> On Fri, May 1, 2015 at 10:17 AM, Betsy Tse <btse1@binghamton.edu> wrote:
>
>> serial_num isn't being looked for when you create a bike through the
>> website
>>
>> On Fri, May 1, 2015 at 10:16 AM, Betsy Tse <btse1@binghamton.edu> wrote:
>>
>>> the change function in the db/migrate file had nothing in it; you
>>> physically couldn't add a bike with the old copy, so i just generated
>>> migrations
>>>
>>> On Fri, May 1, 2015 at 9:58 AM, Franklin Yu <notifications@github.com>
>>> wrote:
>>>
>>>> What is the difference between serial_num and bike_serial_num?
>>>>
>>>> —
>>>> Reply to this email directly or view it on GitHub
>>>> <https://github.com/betsy-tse1/bubhub/commit/966b7bb30afe38c6ecc2ca7de9dabaf4405fc458#commitcomment-11001577>
>>>> .
>>>>
>>>
>>>
>>
>


Rozdział proces jest nadal błędnie formatowany
https://github.com/pwl89/OPW-backend-JavaEE/blob/aeda71e945f9dd9f409a738923926acc650cfda5/README.md#proces


@derpfacedirk this is not supposed to be here...


i have a issue about documenting. see #7


aí é que está... a ação "preencher o campo X com o valor Y" é repetida várias vezes, né? Exatamente por isso o campo e o valor são parametrizáveis... isso é um método! Então esse último comentário não se aplica... O step fica correto fazendo conforme dito no primeiro comentário...
 Then(~'^I can fill the field "([^"]*)" with value "([^"]*)"$') { String field, String name ->


@jenius I think it's worthwhile to consider a default option, but:

- 0.2 was versioned as a breaking change.
- I don't think the code to determining the path is too cumbersome to repeat.
- I don't agree that every single sprout instance _wants_ (important distinction) to live in a user folder
- I think roots' sprout instance should source from a different location than sprout-cli.  Should a roots user even know sprout is the underlying templating mechanism?

I am not wholly opposed to a default option, but I think it's still an opinion I'd like to avoid.  @kylemac: your thoughts are appreciated as well!


So many problems with my patches :-( You're entirely correct -- I had forgotten to update the file.


For the cases here, could you include some comments about exactly what they are checking. I understand this, but other developers may not. For example, you cannot have an unbin option in this case because we are first checking that we have an unfiltered MDEvent workspace as input. Do the same with the other cases.


I use geany - with visible indentation (plus it detects), so makes it easier. it runs on osx too -http://wiki.geany.org/_media/howtos/osx/geany-osx.png :)


I thought this too, but the problem isn't in the addition, but in the -1 literal itself.


@jenius Additionally, in reference to my third point: roots templates are specific to an extent (`app.coffee`).  Templates may exist in a shared sprout folder that aren't roots-friendly.


А теперь расскажи, на каком основании этот код появился на гитхабе да и еще в паблик?


very happy to see the commit.. very sad not to see the description in the tasknote :(

perhaps I'm doing something wrong..


echo?


What about invisible Debuffs like AoE? What about Hakkar self debuffs? And what about debuffs that are lower prio than dots?


we don't need the java options here.


@brandtheisey looks like 'api/services/notifications/manager.js' only has whitespace changes... can you submit a clean PR without that file?

Other than that, this looks good to me!


You could add the post title here:

```
<h1>Sumary for <%= @summary.post.title %> </h1>
```


@lmperry Added this little change in the branch adminModal


not a requirement, i just noticed you were trying to show a title of something and was offering up some advice.


Looks good! Thx


We can remove the entire para now as nothing is installed -- just sent you a simple PR.


I think the only place where I disagree is about the code for determining the path being cumbersome. It really is quite cumbersome, you need to require a number of additional libraries, its many lines and multiple functions to achieve.

I do think splitting roots' location out from sprout's is a good idea though and we should definitely do this. I just would like to see a cleaner implementation where we don't need to manually add all the user folder logic. Honestly even some little utility like `sprout.utils.user_folder()` that runs all that logic I think would save many people and libraries a solid chunk of time and code.


Sorry it took long to push the fixes. Been busy with my current work. :)


@jenius What about a separate package for something like this?


Why is network id specified ?


@shoghicp if you think the ` instanceof ` makes it run slower significantly, don't use type hints anywhere inside the code.


Make sure the number %4 and %5 correspond with your med_gear and med_licenses (i have custom so you might need to switch them around!)


ErrInvalidSandboxConfig


No, this was just due to some plugins overwriting a variable when teleporting players, that caused a crash.


Maybe you add that an option to the ssh object? I'm kind of disabling it
all the time because i need the ansi characters to display color and those
2 lines was stripping them out.

*Joshua Newman **•* Android Engineer

*RedMart*
*M *(65) 9338 0205 | *O *(65) 6261 2284
*A*  3A Jalan Terusan, Level 2M, Singapore 619302

On Fri, May 1, 2015 at 10:24 PM, Andrew McMurtrie <notifications@github.com>
wrote:

> Is it something u want disabled all the time or just for debug
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/joshln87/ssh2shell/commit/fd5ebf7d5738e9aa5efa2e76fbc6135c67518683#commitcomment-11001918>
> .
>


Se o step define 3 parâmetros, deveria haver 3 variáveis após a chave.


Same here.


não faz sentido


Ben - In the config/route.rb file, the code there had these two GET lines. I didn't delete the duplicates because I didn't know they would produce duplicates. Should I delete the duplicates?

```
Rails.application.routes.draw do
  get 'welcome/index'

  get 'welcome/about'

  get 'welcome/index'

  get 'welcome/about'

  root to: 'welcome#index'

end
```


E ter variáveis para acessá-los, tal como feito para authorName


Hm, where is the manifest file?


wow you did great!!!


O step em si está todo errado. Ele não faz sentido e código muito menos. Checar comentários sobre o arquivo .feature e depois refazer implementação aqui.


Should `StatCollector.translateToLocal` be changed to `Lang.localize` for consistency? Or is there a difference between the two?


What the.. commit message..


Instead of `=`, please use `-`


A verificação de que a remoção foi bem sucedida só deve acontecer no then.


Aqui já está sendo solicitada a remoção, mas o texto do step sugere outra coisa... confuso.


Previously, we discussed the idea of wrapping each doc in a `<section>` tag so that it could validly have a `<h1>` tag then just use CSS to format it appropriately.

In other words, treat each description block as its own doc in HTML.


when working with ASP.NET Web.API, data is not serialized as JSON when
contentType is not defined (due to the default value of '
x-www-form-urlencoded').

On Fri, May 1, 2015 at 5:49 AM Jacob Dixon <notifications@github.com> wrote:

> In my tests I was unsuccessful at being able to reference the data through
> $_POST['data'] when the contentType was defined.
>
> Is there a specific benefit to defining the contentType?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ivoviz/feedback/commit/afe41d2e180e9ac1e880fd97ae7f68935f3afbb4#commitcomment-10999554>
> .
>


I liked the `animatable properties` bit.

`To see a full list of animatable properties…`


@nporteschaikin I thought utils was for any sprout utilities :wink: but if you'd feel more comfortable with a micro package, that seems ok to me.


removing direction from "move" and instead passing in the # of spaces as either positive or negative to indicate direction.


But the name of the operating system is _OS X_, not _Mac OS X_.


O texto desse step não está correto (vide comentários no arquivo .feature). Prova disso é que ele diz para fazer a remoção quando esta já foi feita no when, e a ele apenas verifica que a remoção foi bem sucedida (ou seja, que o artigo não existe mais). Portanto, a implementação está correta, mas a escrita do cenário e, por tabela, da expressão regular aqui, está errada.


Ah, there it is. I think you are missing the necessary permission to save files, no? :-1:


93441220-LowTempRadiantWarning-4237 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1631 of 1643 tests passed)

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


@adrielstar with mobile I mean everything you can use on the way for example: tablet, ipad, ipod, celphone. We are not building the website specific for those things but for laptop and pc. If you want to make it compatible for small devices instead use the code that I've send you.


So what exactly is this seamless teleport?


Só se fosse um método static, certo? Ainda assim, esse método existe na classe Periodico?!
Ao que parece aqui, esse é um método auxiliar que faz parte do arquivo de steps (ou pelo menos deveria fazer).


You know what? looking at your code i realized that there is a much simpler solution for the issue i reported above: instead of putting the key lookup stuff into ReplyCommand, it really should go into
ComposeCommand.
There is already some code that tries to find your own key for signing the mail. Here should also be the place where the recipients keys are looked up, and ask the user if he wants to proceed on error. This functionality could be triggered by a boolean parameter (to ComposeCommand.__init__)
and a default should be taken from a (new) config option "encrypt_by_default", similar to the existing "sign_by_default".
Lastly, the lookup of ones own keyid should be validated in the same way the recipients key is looked up.
Currently, only the key hint taken from a users config is copied, not actually looked up using gpgme.
All this is in alot.commands.global.ComposeCommand.

Another alternative to avoid code duplication would be to actually fire a EncryptCommand
after the ComposeCommand (e.g. as post-hook if the encrypt-to-recipient flag is set)..

What do you think?


@adrielstar  okey cool! nice stuff


Плагины надо пересобирать?


@Meticulus @golden-guy mmcblk0p19 which partition is? (System, data, which one? I wanna adapt this for my janice


I'm not quite grokking this -- why all the manual `require`/`resolve` instead of just passing in values or vars directly?


This particular use of skip-binder seems ok. I would add a comment like this:

```
// It is ok to skip past the higher-ranked binders here because the `match` below does not consider regions at all.
```


Looks good.


From what I can tell
- `msg.socket` is the ID value (for example in logging after a run it was the value 3, which is plausible)
- passing it as the third (optional) argument to the constructor should create an object that reuses it

Is the first of these wrong, the second, both, neither?


That may become my favourite comment :)


Thanks for the pointer to the merge function.  Here's where I'm getting confused.  The searchForCompound() (line 1142), searchForAllCompounds() (line 1162), searchForReaction() (line 1182), and searchForAllReactions() (line 1196) methods all "first search by exact alias match".  The methods use the getObjectByAlias() or getObjectsByAlias() methods which search through a hash of all of the aliases for all of the compounds or reactions.  Since the aliases include the name and the various ids the query compound or reaction is found and the search by searchname is skipped.

I think this is the correct behavior for searching for commands like fba-getreactions.  A user wants to find a reaction by any of the names that they know the reaction by.  And the same probably goes for importing models in different formats.

What is the value of the searchname?  And is this all a moot point once the biochemistry is loaded in SOLR?


Looks like you are missing saving files and SQL.


Caused by my snippet XD


I only have it on the develop branch right now. Chances you are on master and so don't see it. I'm planning on fixing docs and some bugs before merging. I should have that done in the next couple days.


thanks, looks good.


does the following imply an unwanted regression with this change? https://github.com/liip/LiipMonitorBundle/issues/103


initial commit


What is a bug bomb? Is it a pesticide bomb? The name seems to suggest that it explodes bugs.


what was GenericTabularInline ever used over admin.ModelAdmin? Was it just something left over from django 1.4 or is it valuable for something?


:tada: 不過你忘了加 .gitignore XD


It will certainly be a change of pace over the summer!  I'm going to be doing an internship at Microsoft Research, working on some medical device or other.  I'm planning on using Julia as much as I can while I'm over there, so we'll see how things go.  :)

Unfortunately, this pretty much means I can't come to JuliaCon, as I have other events taking up all of my vacation time already.  :(

I'm not sure what the problem with the OSX guy is.  I logged in over SSH and ran the `make -C contrib/mac/app` command manually, and everything seemed to work out just fine.  If the next build fails, let's just nuke it and cross our fingers.


Haha, ok, let's see. I think these problems are being caused by the way
lines are mapped onto the earth as a sphere (well, the problems in the
video anyway). I bet your mapping library is taking great-circle paths
rather than coordinate-straight lines, which would cause geohashes to
appear distorted.

Do you still see these problems drawing on a 2D surface?

On Fri, May 1, 2015 at 10:02 AM, Borislav Ivanov <notifications@github.com>
wrote:

> P.S You didn't mention what kind of beer the license applies to :) What
> kind of beer should I send ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Factual/beercode-open/commit/e83a99ca81d4abd163211183d0d40594d7f2ba66#commitcomment-10999668>
> .
>

--

*Factual Confidential Information*
Any non-public information in this email is Factual's confidential
information.  It is only for use by the intended recipient under the terms
of a non-disclosure agreement or other binding confidentiality terms.  If
you've gotten this email in error, please let the sender know of the
mistake and delete the email immediately.


should be iosbuild, not ios_build .


Check out the LIMITS function, that is how you can get mixture triple point.  I'm pretty sure I implemented that?  I needed it for mixture phase envelopes for tabular methods


True. I was just a little annoyed at the moment ;-)


Like I said, I'm old when it comes to this stuff. The name change which came in 10.7 was IMO for the worse. Regardless, the phrase "Mac OS X" does appear elsewhere on the Jekyll site ("Installation", for example).


fix


Oh, and the application is now potentially supported on MacOS!!!


We have the set of aliases named 'searchname' and we use this function to
consistently create the searchnames:

https://github.com/kbase/KBaseFBAModeling/blob/master/lib/Bio/KBase/ObjectAPI/KBaseBiochem/Compound.pm#L314-L345

many compound names can and do vary in little ways, particularly with
capitalization and non-word characters for whatever reason, for example, if
someone were to search for d-glucose instead of D-Glucose, they wouldn't
get an exact match with the name, but the nameToSearchname() function will
consistently return 'dglucose' for both names, and therefore match them.

Users really should not be using names to search for reactions, they can if
they want to, but the room for variation and imagination is far greater,
and some reaction names and EC numbers translate to multiple
stoichiometrically-balanced reactions so users should really search for
reactions by the equation itself.

S

On Fri, May 1, 2015 at 9:51 AM, Mike Mundy <notifications@github.com> wrote:

> Thanks for the pointer to the merge function. Here's where I'm getting
> confused. The searchForCompound() (line 1142), searchForAllCompounds()
> (line 1162), searchForReaction() (line 1182), and searchForAllReactions()
> (line 1196) methods all "first search by exact alias match". The methods
> use the getObjectByAlias() or getObjectsByAlias() methods which search
> through a hash of all of the aliases for all of the compounds or reactions.
> Since the aliases include the name and the various ids the query compound
> or reaction is found and the search by searchname is skipped.
>
> I think this is the correct behavior for searching for commands like
> fba-getreactions. A user wants to find a reaction by any of the names that
> they know the reaction by. And the same probably goes for importing models
> in different formats.
>
> What is the value of the searchname? And is this all a moot point once the
> biochemistry is loaded in SOLR?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ModelSEED/ModelSEEDDatabase/commit/03768bff842f2d0934b241943d42bb05975a075c#commitcomment-11002268>
> .
>

--
Postdoctoral Fellow
Mathematics and Computer Science Division
Argonne National Laboratory
9700 S. Cass Avenue
Argonne, IL 60439

http://www.linkedin.com/pub/sam-seaver/0/412/168
samseaver@gmail.com
(773) 796-7144

"We shall not cease from exploration
And the end of all our exploring
Will be to arrive where we started
And know the place for the first time."
   --T. S. Eliot


Sure ... this is the help text:

The address space of ARM processors is only 4 Gigabytes large and it has to accommodate user address space, kernel address space as well as some memory mapped IO. That means that, if you have a large amount of physical memory and/or IO, not all of the memory can be "permanently mapped" by the kernel. The physical memory that is not permanently mapped is called "high memory".

Depending on the selected kernel/user memory split, minimum vmalloc space and actual amount of RAM, you may not need this option which should result in a slightly faster kernel.


You didn't update ``setup.py`` to match, did you? Pull request coming...


don't you think it would be better to indicate gistr (>= 0.2) instead of no version?


Please send them my way :-)


Note that I also deleted this fail-safe check - when we have window focus implemented, we will have no penalty in adding "press agX to close" functionality to every widget non-closeable in normal way (like "Quit" from menu)


So some of that I believe is caused by QA and Dev running much slower than Prod.  My current no evidence theory is that some index or partitioning somewhere isn't being set up in the weekly push from prod down to dev and qa.

Some of the change is due to us grabbing one whole month of 2 minute data more than we need to.

The everyPeriod parameter describes how long a gap can be in order to interpolate between the points.  The current implementation of that means we grab an extra everyPeriod worth of time in the sql query just in case the last timesteps of the graph are during an interpolatable gap.


Actually I'm not sure this is the right solution.  I think if it is a single-component, you should use INFO, and otherwise, you should use LIMITS.  Otherwise, what is returned for triple point of mixture of methane and ethane?  I suspect with this solution you would get the triple point of methane.

Also, we should do a better job of describing difference between Tmin and Ttriple.


Yes, make a separate branch that only contains the commits that you want as part of the pull request.

From where you are now you could do

```
git branch mod-punctuation    <-- make a new branch containing these commits
git reset --hard HEAD~1             <-- remove the last commit
git push --force origin master   <-- update the pull-request
```

And then you would still have your puncutation change on the `mod-punctuation` branch, which you can use to work with and from which you could make a separate pull request.


I don't remember if there was a reason to not use a var. But we are not using a value because we want that the server will use the new handler if it is redefined.


Does planemo have its own exception classes? Otherwise maybe ``ValueError`` or ``RuntimeError`` might be better here?


jQuery UI libs custom


Hmm, thanks, will try it later. I am fixing WLAN driver of my Z1, and using SGSA less than before...


Yerp, they normally release a huge cloud of poisonous gas that kills insects.


This is brilliant @benhodgson87 . I've been trying to cleanup the modules and this works nicely. Doing a bit of an update based on Dan's original animate.css. Will aim to roll this out across the other modules.

Thanks again! +1 Github!!!


@ace2nutzer
`disabling HIGHMEM without enabling: CONFIG_MMC_BLOCK_BOUNCE will decrease performance ( I/O Speed ) instead of increasing due to disabled CONFIG_BOUNCE`

But how this is connected each other? Having CONFIG_MMC_BLOCK_BOUNCE is probably good option, but is this somehow dependent on CONFIG_BOUNCE?


Could we use `@compat Int(...)` instead of `@compat round(Int, ...)` for these two lines?


I agree, have a look at the latest commit...


should not be this on equipment pbo?


Ok. FWIW in our code vars seem to work fine. Might make for a less magic-seeming example to switch.


Should prepare a ACRE2 version :D


indentation, 'end' is not aligned with its 'def'


Nope, that was the problem before. Limits returns the limits and for water Tmin < Ttriple.


:+1: Thanks!


@dzafman,  i think this change can fix the case where `do_read_entry(bl, 0, corrupt)` returns `MAYBE_CORRUPT`, and leaves `seq` unchanged, which ends up with `assert(0)` or sets `corrupt`. and seems we don't have any test covering this case at this moment. so i suggested add some test or ASSERTs for this case. maybe i missed something. i am not sure.


That was my intention, but I wanted to see what would happen with 0.1....will add a 0.2 requirement when it hits CRAN


thanks @aledsage I forgot to document it


Does *every* license of Office come with Excel? If not, then this needs fixed before we can switch.


Could do with a new line before this


Rubberduck requires the 4.5 version of the .Net framework


You changed ('friends.show', this) to ('friends.show', this.get('model')) in the book already (and that's better).

The thing is that if you edit a friend's firstName, and then hit cancel, the changes are still reflected on the show page. A reload shows that they didn't save it to the api, but it's still reflected in your local version of the model.

What would be the proper way to really cancel the edit?


Nikhil Surana's Solution(https://github.com/NikhilSurana/members-only) | [View in browser]


+1 I was about to submit the same PR :-)

Essentially, the context should be created immediately with `req` and `req` in order to prevent `CookieStateSource` to create a new instance of `ServerCookies` with `req` and `req` equal to `undefined`:

https://github.com/martyjs/marty/blob/3ceb446f95eb4ed8e6d96f52ac5cec1ae1a17089/lib/stateSource/inbuilt/cookie.js#L8

https://github.com/martyjs/marty-express/blob/8edbe91ca0798a638ff3706ea48394ffb017a655/serverCookies.js#L2


Can we leave the `AppData\Roaming\Rubberduck\` directory intact? It's common practice to leave user settings on the file system after an un-install so that if the product is re-installed, the settings are "recovered". It would be even better if we could prompt the user and ask them if they want to have it removed. I have no idea if that's even possible with InnoScript though. I know next to nothing about it.


oh, i like change :)


please fix the indentation here. @HAAKS


@mgp25 :+1:
Thanks a lot.


mmcblk0p19 is j4fs samsung partition


This code quality is hilarious!


![final version](https://cloud.githubusercontent.com/assets/7685263/7432143/7c4cddbc-eff3-11e4-9ab6-bb7b161ee79c.png)


thx


detailed user flow


93441220-LowTempRadiantWarning-4237 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1636 of 1649 tests passed)

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


thank you


`if message is None:` ?


The variable name you have in the Frontmatter is `example`, not `examples`. Please update on or the other. Otherwise looks good to me!


The x86 key should always be installed, because it's possible to have it installed side-by-side with the x64 version. Inother words, always register 32 bit and 64 only if a x64 installation of office is found.


please fix the indentation here. @MohamedIhab


r


Though I agree the NPC Ragged John is not very good at spelling, the removal of the "t" in "the" at the end of the text sounds strange to me. ;)


Sorry it took so long for me to take a look at this. I left some notes. Hope they made sense.


TeamCity tools :: machine-images [Build 17](https://teamcity.gutools.co.uk/viewLog.html?buildId=607273&buildTypeId=Tools_Machi) is now running


That is an interesting discovery - I will certainly take a look. Honestly,
I've always felt that I/O could be better, but since I didn't touch any I/O
related config options, I thought that's just the way it is. How
significant is this decrease? more like 10% or 100%? Have you measured it
(at least more-less) or you just have a feeling?

@cocafe HIGHMEM is used to enable a 32-bit system to access more than 4G of
memory, which is obviously not the case here. Without writing an article on
linux mm split has to be changed to 2/2 so it can access as much memory as
possibly (for "why?" read about linux virtual memory management). And since
we have only 768M of ram, we don't need it. Since it's probably highly
optimized, there won't be any noticeable change in speed or kernel size,
but... the less code the better.
On May 1, 2015 3:41 PM, "ace2nutzer" <notifications@github.com> wrote:

> btw u guys @ChronoMonochrome <https://github.com/ChronoMonochrome>
> @mkaluza <https://github.com/mkaluza>
>
> disabling HIGHMEM without enabling: CONFIG_MMC_BLOCK_BOUNCE will decrease
> performance ( I/O Speed ) instead of increasing due to disabled
> CONFIG_BOUNCE ;-)
>
> @cocafe <https://github.com/cocafe> why u don't disable also HIGHMEM like
> us ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ChronoMonochrome/Chrono_Kernel/commit/42034fd9ee51abee265dc2dbfbe921e602e83f0d#commitcomment-11001356>
> .
>


Am currently testing it out, will let you know when I am happy with it


This may add some memory pressure I think? Clear() is often called in FlowAnalysis.


I wouldn't say 'first' here as it might confuse because installing java and maven should happen before.


呃呃..  对重新push了


hi Steve, do you make changes to header.php because the toggle is not working in iphone, ipad, etc?


the vehicle can be any value currently supported by GraphHopper (see the variables in EncodingManager) and any vehicle that you implement :)


No need for the H1 here; that's the title and will always be rendered to the page


just a suggestion, but since there is only one function exported there is no need for `.byFileName`. Even if you add additional methods you can add them as properties of the exported function. e.g.

```js
var lc = require('language-comments');
var js = lc('js');

// some other method
var foo = lc.bar(baz);
```

 IMHO this is a good pattern to follow when 95% of use cases will only use the main function.  but again, just a suggestion


Too long to write an explanation, if you are into it I can do a quick Q&A :) tomorrow http://doodle.com/m46gt3zbkv5fm7p4#table I'll record it for future generations.

We'll do it over Skype: daniel.meteorspark


I think we cover how t handle this kind of scenarios if you keep going :)

—
Sent from Mailbox

On Fri, May 1, 2015 at 10:13 AM, Thijs van den Anker
<notifications@github.com> wrote:

> You changed ('friends.show', this) to ('friends.show', this.get('model')) in the book already (and that's better).
> The thing is that if you edit a friend's firstName, and then hit cancel, the changes are still reflected on the show page. A reload shows that they didn't save it to the api, but it's still reflected in your local version of the model.
> What would be the proper way to really cancel the edit?
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/abuiles/borrowers/commit/79601014b1567e0ef5c2fda2cd300f3483fa6b22#commitcomment-11002526


https://github.com/graphhopper/graphhopper/blob/master/core/src/main/java/com/graphhopper/routing/util/EncodingManager.java

Peter <notifications@github.com> schrieb am Fr., 1. Mai 2015 um 17:21 Uhr:

> the vehicle can be any value currently supported by GraphHopper (see the
> variables in EncodingManager) and any vehicle that you implement :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/badewannencaptain/map-matching/commit/cdfc2df6ee3b9e619d0cbe0c0e20276135d97b43#commitcomment-11002656>
> .
>


@Turini, The version number is ok now. The changes were made.


I thought I made a comment here last night but github ate it.

Is this needed after the changes in #123? Particularly 7e32ec93eb3.

Also this change seems to be wrong since `(fl>= x -4.0)` returning `#t` does not imply that `x` is `-NonNegFl` since that can hapen if `x` is `-3.0`.


whoopsie, to trigger happy :)


lordofname needs to go in here for the lords sort below to work


TeamCity tools :: machine-images [Build 18](https://teamcity.gutools.co.uk/viewLog.html?buildId=607276&buildTypeId=Tools_Machi) is now running


TeamCity tools :: machine-images [Build 18](https://teamcity.gutools.co.uk/viewLog.html?buildId=607276&buildTypeId=Tools_Machi) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:06


Ok, jestli se ti to hodilo na devu… Na produkci je nginx takže tam se tohle neprojeví, ale tam je to imho nakonfigurovaný dobře.


The `HTTP_ACCEPT_LANGUAGE='de'` kwarg is causing this test to fail.


Could you please also provide the CMake file?


It is absolute path of your server e.g. /var/www/ampache/metadata


Prosím rozděluj commity podle toho čeho se týkají, tzn. v tomhle případě zvlášť adminer a zvlášť apache. :penguin:


Rather than generating an array, and then doing lookups on arrays of extensions, I'm betting it would result in faster lookups if we generate an object, where each key is a file extension and the value is comment patterns. This would obviously be much larger object with duplicate information, but the lookups would be faster, since you would only need to do `lc['js']`. just a thought, might be worth considering


How about:
We have stretched the zoomControl and its "container" (which is
*not* its parent as once thought but rather its DOM element)
into a multi-purpose widget control. This is done by
instantiating a zoom control and then placing additional markup
in its DOM element for other controls.  Subsequently, we refer to it as the
multiControl in order to hide this implementation detail.


Det är lungt, har gjort det själv ett par gånger och det var så jag vist direkt var felet var...


The tests are failing because the name hasn't been changed to the corresponding pycrc name.


yay regressions...


Actually, I always wanted to keep the *.pdf-binaries out of the master branch. It happened here only as an accident. Anyway, I will just keep them there. It's simply easier this way...


This links everything against librt. As discussed, this needs improvement.


When do you have time to help on NumericRanks? (Not rushing you)


/play yeah


Yea, this was brought up internally as well. Dependencies need to be maintained in both the Bower and NPM files.


@ChronoMonochrome CONFIG_BOUNCE and CONFIG_MMC_BLOCK_BOUNCE is probably the same...
but u can use only CONFIG_BOUNCE with HIGHMEM and without HIGHMEM CONFIG_MMC_BLOCK_BOUNCE .. and not viceversa .. at the end speed is same like highmem + bounce .. but i can "feel" a faster kernel since highmem is disabled ...

@mkaluza of course i did benchmarks... as i already said.. u'll recover the "old" speed like bounce + highmem .... but anyway it's really like the help text.. the kernel is then sightly faster .. (if we do it right ;-) )


works all :) thanks but can u help me for fixx the error whit the wantedgrab? :)


This commit seems to break things. I get the following error. Maybe there is a missing configuration/dependency to support this?

```
/home/vagrant/.gem/ruby/2.1.5/bundler/gems/omnibus-software-81bbcbe6fff3/config/software/ncurses.rb:85:in `block in evaluate': undefined method `ppc64le?' for #<Omnibus::Builder (Cleanroom)> (NoMethodError)
	from /home/vagrant/.gem/ruby/2.1.5/gems/cleanroom-1.0.0/lib/cleanroom.rb:70:in `instance_eval'
	from /home/vagrant/.gem/ruby/2.1.5/gems/cleanroom-1.0.0/lib/cleanroom.rb:70:in `evaluate'
	from /home/vagrant/.gem/ruby/2.1.5/gems/cleanroom-1.0.0/lib/cleanroom.rb:184:in `evaluate'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/software.rb:378:in `build'
	from /home/vagrant/.gem/ruby/2.1.5/gems/cleanroom-1.0.0/lib/cleanroom.rb:130:in `public_send'
	from /home/vagrant/.gem/ruby/2.1.5/gems/cleanroom-1.0.0/lib/cleanroom.rb:130:in `block (3 levels) in cleanroom'
	from /home/vagrant/.gem/ruby/2.1.5/bundler/gems/omnibus-software-81bbcbe6fff3/config/software/ncurses.rb:43:in `evaluate'
	from /home/vagrant/.gem/ruby/2.1.5/gems/cleanroom-1.0.0/lib/cleanroom.rb:70:in `instance_eval'
	from /home/vagrant/.gem/ruby/2.1.5/gems/cleanroom-1.0.0/lib/cleanroom.rb:70:in `evaluate'
	from /home/vagrant/.gem/ruby/2.1.5/gems/cleanroom-1.0.0/lib/cleanroom.rb:56:in `evaluate_file'
	from /home/vagrant/.gem/ruby/2.1.5/gems/cleanroom-1.0.0/lib/cleanroom.rb:173:in `evaluate_file'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/software.rb:44:in `load'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/software.rb:569:in `block in load_dependencies'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/software.rb:568:in `each'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/software.rb:568:in `load_dependencies'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/software.rb:45:in `load'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/project.rb:688:in `block in load_dependencies'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/project.rb:687:in `each'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/project.rb:687:in `load_dependencies'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/project.rb:44:in `load'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/cli.rb:66:in `build'
	from /home/vagrant/.gem/ruby/2.1.5/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
	from /home/vagrant/.gem/ruby/2.1.5/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from /home/vagrant/.gem/ruby/2.1.5/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/cli/base.rb:33:in `dispatch'
	from /home/vagrant/.gem/ruby/2.1.5/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/lib/omnibus/cli.rb:41:in `execute!'
	from /home/vagrant/.gem/ruby/2.1.5/gems/omnibus-4.0.0/bin/omnibus:11:in `<top (required)>'
	from bin/omnibus:16:in `load'
	from bin/omnibus:16:in `<main>'
```


I think within the context of just the node's certs these should really be node-client.crt and node-client.key. Will ask David if they're just named improperly. I've added todos for that.


@MohamedIhab  please check.


You do not have to but it would be nice so I understand. :P
I want to remove this because it adds complexity to the script and two
paths have to be maintained instead of one (affects the last commit in the
branch).

On Fri, May 1, 2015 at 11:09 AM, Saunier Thibault <notifications@github.com>
wrote:

> I do not have to justify why I implemented features... even less in detail.
>
> The real question is, why do you want to remove that feature?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aleb/pitivi/commit/2c80bc16231ec224ff8ce61af51505c6438fd0b0#commitcomment-10999256>
> .
>


This `didFail` stuff is gone in master, btw :angel:


Up to you, it should be moving to the API soon where I think it's not too bad to just re-use the node certs for openshift-sdn-node. Perhaps we just wait for that?


Why not just adjust the `-f` above to be `-f /root/rpmbuild/SOURCES/docker-core.tar.gz` and skip the `mv` completely?


[17:18:15 +02:00] [Thread 1620462370] extDB: DB_CUSTOM_V5: Error StatementException: MySQL: [MySQL]: [Comment]: wrong bind parameters count	[statemnt]: UPDATE wanted SET wantedName=? WHERE wantedID=?;


test


test inline comment


If the *FOLDERNAME* provided in [settings.ini](settings.ini) is `None` then the folderID will be an empty string and the URL will simply include a double forward slash.


This is now in master and in v0.0.2


Cool, pero por que?

Mariana Esther

On Fri, May 1, 2015 at 5:53 AM, Ricardo H. Ramírez-Gonzalez <
notifications@github.com> wrote:

> Tal vez quieres agregar .pyc al .gitignore
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mar-esther23/regnet/commit/63dc0e9c0400e2ea7b9f1315f7c169b90d229553#commitcomment-11000041>
> .
>


The same error above happened with me


He-Man reference for sure.


Having detached metadata for the summary file feels a bit weird, given that the idea of the summary file is it gets regenerated whenever something changes - theoretically we could put the signatures in the same file as the content.  Except it'd be a pain to do now because the format is frozen...ugh.

Maybe we could have a "v2 data" in the same summary file.  Or just a `summary-v2?`

I'm not saying change this now, but I feel like we need to consider choices here before just inventing `summary.sig`.


Done.


please fix the indentation. @MohamedIhab


Nice work! The package looks good.


@PEMapModder Since my midterm exams finally done, I think we might be able to finish it until next week at least :D


please fix the indentation here. @MohamedIhab


It's not used anywhere in `kubelet.go` so it is possible. I'll add a TODO. I wonder if we'll need it in the future (maybe for some pre-caching?), but I'm not sure when/if it'll come. We can always add it back.


Roberto;

Obrigado pela dica, mas eu não encontrei nenhum método que retorne o timestamp de expiração da data do certificado. O que eu entendo do padrão "facade" é o seguinte, uma classe ou um metodo esconde a complexidade do que está sendo feito em background no código, não sei se é bem isto, mas é o que eu entendo. Então eu teria uma classe que teria um método retornaDataExpiracaoCertificado() dentro deste método eu faria um $NFeTools = new ToolsNFePHP() e $NFeTools->pLoadCerts(); e usando o objeto $NFeTools eu chamaria algum atributo ou método que retornasse o timestamp da data de expiração do certificado. Mas eu não encontrei um atributo ou método com estas informações. Novamente, muito obrigado pela dica.


It sounds like the issue is a differing binary format between x86 and x86-64. Perhaps a different size for long ints, or the type used to store offsets and strides in the tensors. That would cause the second error, as well.

If you like, I can provide the data in another format, or you can use your own data (see data/ and data_preparation/ for the relevant scripts). Then generate your own vocab.t7 and train.t7. Any large (~50MB or larger) text file will do.


I think we should refactor `ClangExecute` to support diagnotics in aync mode.
You may want to checkout my implement https://github.com/syswow/nvim-clang/tree/add-diagnostics.

After #44 merge, we can do it better. it will both benefit Neovim and Vim.


I think we now have this line in several places... we should probably DRY this away at some point


@ChronoMonochrome i can suggest those changes if u need the max. speed for your laptop :-)
And to have also RCU_BOOST u need to change from PREEMPT_VOLUNTARY to PREEMPT  ;-) that also improve speed... :-)


Develop
 Em 01/05/2015 12:43, "Jonatan Flores" <notifications@github.com> escreveu:

> Roberto;
>
> Obrigado pela dica, mas eu não encontrei nenhum método que retorne o
> timestamp de expiração da data do certificado. O que eu entendo do padrão
> "facade" é o seguinte, uma classe ou um metodo esconde a complexidade do
> que está sendo feito em background no código, não sei se é bem isto, mas é
> o que eu entendo. Então eu teria uma classe que teria um método
> retornaDataExpiracaoCertificado() dentro deste método eu faria um $NFeTools
> = new ToolsNFePHP() e $NFeTools->pLoadCerts(); e usando o objeto $NFeTools
> eu chamaria algum atributo ou método que retornasse o timestamp da data de
> expiração do certificado. Mas eu não encontrei um atributo ou método com
> estas informações. Novamente, muito obrigado pela dica.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JonatanFlores/nfephp/commit/d5d7c1844d76025ad26027606a124b6867338f57#commitcomment-11002942>
> .
>


I got around this for the scss-worker specs by including the `faraday_middleware` gem and sending the content as json.

https://github.com/thoughtbot/hound-scss-worker/pull/6


@nohaelshourbagy done, please check


I'm linking against the static luajit in the submodule.  This is the normal way luvit is officially built.  I'm not sure how to make this and your shared libraries work.  Help is appreciated.


Perhaps the solution is to rename the library in the luajit cmake to match what most linux systems name it.


I like the maxTouch idea for the rate, much better than the %


Added image


Alright, I reverted it.


libluajit-[ver] seems to be standard. OSX and most Linux OS's seem to conform


I think that's very similar to what I changed it to. The one question I have is this:
      [myArray addObject:@(smallerNumber)];
the same as
      [myArray addObject: [NSNumber numberWithInteger:smallerNumber]];
is yours shorthand for converting to an NSNumber?


From Cook-my-Catch


93441220-LowTempRadiantWarning-4237 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1636 of 1649 tests passed)

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


Maybe there's a better check here


Thanks, yup that was the suspicion.
When you say "second error" do you mean my last post about Embedding:updateOutput?

Yes please provide it in a different format. The Embedding:updateOutput issue seemed more like a logic issue so another data format would help me resolve it. Thank you!

Here is my email: mszlazak@aol.com


@dzimchuk I'm just curious, what made you decide to use the NuGet packages of Bootstrap and jQuery when Bower is becoming Microsoft's primary package manager for front end frameworks on Visual Studio 2015?


Not sure of the side effects there but that seems reasonable, the subscriber doesn't exist, so don't have to `removeResult`


please fix the indentation here. @MohamedIhab


thanks! :bow:


Whoa! Big names!


@MennaDarwish Delete all white space trails.


tahawy


Yes, I meant both errors were stemming from garbage data being loaded in,
due to alignment issues from the datatype sizes.

In the meantime, could you please check if vocab.t7 loads correctly? Just
do print(torch.load('vocab.t7')) and see if you get a table mapping roughly
30 characters to ints.

On Fri, May 1, 2015 at 4:52 PM, mszlazak <notifications@github.com> wrote:

> Thanks, yup that was the suspicion.
> When you say "second error" do you mean my last post about
> Embedding:updateOutput?
>
> Yes please provide it in a different format. The Embedding:updateOutput
> issue seemed more like a logic issue so another data format would help me
> resolve it. Thank you!
>
> Here is my email: mszlazak@aol.com
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oxford-cs-ml-2015/practical6/commit/96749c8d9bc93f864c94c048a3c8cd73f59f733b#commitcomment-11003098>
> .
>


@ace2nutzer, thanks for advice :) It's long time since I don't have battery(it's broken), therefore this is an option which I would like to have :)
p.s. how do you think, can be CONFIG_NOHZ disabled (if I'm anyway don't have battery) ? :D


@lsmith77 looks like what is described in #14368


On May 1, 2015 5:30 PM, "ace2nutzer" <notifications@github.com> wrote:
>
> @mkaluza of course i did benchmarks... as i already said.. u'll recover
the "old" speed like bounce + highmem .... but anyway it's really like the
help text.. the kernel is then sightly faster ..

Can you share the numbers? approximately, if you didn't write the down...

> (if we do it right ;-) )

I get your point ;P mea culpa ;P


Check here 0.7


Yes I get 35 characters. What range of values do i expect. Things I get are arrays with 80's 96's etc. Funny thing is these arrays are filled with just one number.


@nohaelshourbagy check again please


Sounds promising. I will review it this weekend.


+1. Would be an easy addition to TransformSystem.


@dblessing @newx Just update the version of Omnibus your project is using (e.g. `bundle update omnbius`).


It doesn't really have any - sometimes it uses click's ``UsageError`` but that isn't relevant here. I rarely use ``ValueError`` and have never thrown a ``RuntimeError`` - but reading through the desciptions it seems like I should be using at least ``RuntimeError`` a lot more frequently.

Do you have a preference between ``ValueError`` and ``RuntimeError`` in this scenario?


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 20](http://teamcity.otenv.com/viewLog.html?buildId=1285705&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 21](http://teamcity.otenv.com/viewLog.html?buildId=1285706&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


I've found intereting article here http://www.linuxquestions.org/questions/slackware-14/config_no_hz-is-disabled-in-single-cpu-kernels-869617/


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 20](http://teamcity.otenv.com/viewLog.html?buildId=1285705&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


It doesn't work on my phone, i can't add accounts (I loaded it on webIDE)


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 21](http://teamcity.otenv.com/viewLog.html?buildId=1285706&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


Works for me too! When you have a long list of paper-item, this issue is a big problem :)


I tested with a dummy user, and 'normal' users cannot set labels. Thanks @RealRancor


@MizzleMo Fixed in the master of gulp-typescript.


Another side issue while we are at it. If you where not going to flatten everything out like you do then could you not use the clone() function Torch provides instead the reader() ... etc?


The functions to get/set unit loadouts are in main, so it makes sense this is in main too. These functions are made to be used as an API, not only for the equipment module.


Waw, chapeau bas !
De mon coté je suis à du 960/360 px, et il faudrait que la fenetre affichée à l'écrant soit de 140x140 px à peu près ( jte donne un truc sortit du chapeau ( encore lui ! ), c'est une estimation selon la taille de la map et ce que je voudrais qui soit affiché, mais acun interet à ce que ce soit trop grand.


Table名稱預設為Model name的小寫複數（其實習慣就好


What do you mean by "flatten everything out"?

On Fri, May 1, 2015 at 5:04 PM, mszlazak <notifications@github.com> wrote:

> Another side issue while we are at it. If you where not going to flatten
> everything out like you do then could you not use the clone() function
> Torch provides instead the reader() ... etc?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oxford-cs-ml-2015/practical6/commit/96749c8d9bc93f864c94c048a3c8cd73f59f733b#commitcomment-11003301>
> .
>


Derp on my end lol


Eloquent預設會處理`created_at`和`updated_at`兩個時間戳欄位
用於記錄建立和更新時間
http://laravel.tw/docs/5.0/eloquent#timestamps
在migration可以用`$table->timestamps();`快速建立這兩個欄位


Download and extract these two replacement files, now in torch's text-based
format instead of the binary format:

http://www.cs.ox.ac.uk/people/brendan.shillingford/teaching/practical6-data-ascii/index.html

And change all occurrences of torch.load(something) to say
torch.load(something,'ascii').

On Fri, May 1, 2015 at 5:05 PM, Brendan Shillingford <brendan2609@gmail.com>
wrote:

> What do you mean by "flatten everything out"?
>
> On Fri, May 1, 2015 at 5:04 PM, mszlazak <notifications@github.com> wrote:
>
>> Another side issue while we are at it. If you where not going to flatten
>> everything out like you do then could you not use the clone() function
>> Torch provides instead the reader() ... etc?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/oxford-cs-ml-2015/practical6/commit/96749c8d9bc93f864c94c048a3c8cd73f59f733b#commitcomment-11003301>
>> .
>>
>
>


Thanks for this fix.  I hadn't rebased in a while, and was about to re-do this from scratch :)


Great control flow, incredible structure, well written, statements are confusing nonetheless easy to read, thank you for sharing.


See issue #22.


th> print(torch.load('vocab.t7'))
{
  q : 26
  . : 6
  c : 12
  b : 11
  s : 28
  d : 13
  ( : 3
  e : 14
  t : 29
  n : 23
  g : 16
  w : 32
  ; : 8
  h : 17
  , : 5
  i : 18
  f : 15
  y : 34
  v : 31
  k : 20
  j : 19
    : 1
  ? : 9
  l : 21
  m : 22
  ! : 2
  u : 30
  r : 27
  o : 24
  : : 7
  z : 35
  x : 33
  ) : 4
  a : 10
  p : 25
}


It isn't necessary to create a new ListViewItem here. This line is basically creating a ListViewItem, setting the "Text" property of the new item, then indexing the text property and only passing the string and not giving lstLogs the new ListViewItem we just created. lstLogs must create its own ListViewItem now. This causes each iteration to be roughly 35%-40% slower every iteration on both DEBUG and RELEASE configurations (is not optimized out!).
Instead, consider using this instead:
    client.Value.FrmKl.lstLogs.Items.Add(new ListViewItem() { Text = file1.Name });
This way, the new ListViewItem is the parameter being used, and not just the string.


Looks fine, must be just the tensor then. The one I sent in the link should
work fine. It's the exact same dataset, just converted format.

On Fri, May 1, 2015 at 5:09 PM, mszlazak <notifications@github.com> wrote:

> th> print(torch.load('vocab.t7'))
> {
> q : 26
> . : 6
> c : 12
> b : 11
> s : 28
> d : 13
> ( : 3
> e : 14
> t : 29
> n : 23
> g : 16
> w : 32
> ; : 8
> h : 17
> , : 5
> i : 18
> f : 15
> y : 34
> v : 31
> k : 20
> j : 19
> : 1
> ? : 9
> l : 21
> m : 22
> ! : 2
> u : 30
> r : 27
> o : 24
> : : 7
> z : 35
> x : 33
> ) : 4
> a : 10
> p : 25
> }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oxford-cs-ml-2015/practical6/commit/96749c8d9bc93f864c94c048a3c8cd73f59f733b#commitcomment-11003375>
> .
>


What's this about?


I just want to keep things as simple as possible at first and do not gather things up into one long storage. It just for teaching/self-learning purposes.

More simply, why not just use clone() provided by Torch instead of reader()?

BTW, seem that auto differentiation script you just made. Thanks ... again. LOL


I tried installing x64 alongside x86, Office setup didn't let me.


"NOOOOOOOO" ~ Darth Vader
:cry:
I was a JEDI, BOOOOO.


Sure, could you point me to which line/file you're referring to, though?

On Fri, May 1, 2015 at 5:13 PM, mszlazak <notifications@github.com> wrote:

> I just want to keep things as simple as possible at first and do not
> gather things up into one long storage. It just for teaching/self-learning
> purposes.
>
> More simply, why not just use clone() provided by Torch instead of
> reader()?
>
> BTW, seem that auto differentiation script you just made. Thanks ...
> again. LOL
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oxford-cs-ml-2015/practical6/commit/96749c8d9bc93f864c94c048a3c8cd73f59f733b#commitcomment-11003429>
> .
>


Actually, I'm over it.


I'm dumb...


En el caso del radioButton, habria que ver como poner que elija solo una de las opciones en los radioButton. :question:


`GVariantDict` is a bit more convenient API than `GVariantBuilder` for working with `"a{sv}"` variants, and it guards against duplicate keys which `GVariantBuilder` does not.  You might consider using it here and for `deltas_builder` below.

I already bumped OSTree's GLib req so I could use `GVariantDict` elsewhere.


Why not use something like the following instead of model_utils.clone_many_times,
      function makeClones(model, numClones)
         local clones = {}
         for name, module in pairs(model) do
            if module.getParameters then
               clones[name] = {}
               for i = 1, numClones do
                  clones[name][i] = module:clone()
               end
               print('Cloning\t'..#clones[name]..' '..name..' modules.')
            end
         end
         return clones
      end


Привет!


Привет


@WebReflection https://rawgit.com/es-shims/es5-shim/master/tests/index.html and https://rawgit.com/es-shims/es5-shim/master/tests/native.html are usually what I use. If we need to add tests for Opera Mini then let's do that :-)


This is a bit dangerous as it means it will happen during introspection scanning too.  I tend to use `g_once_init_enter()` for these things inside `_init()`.


@davidrleonard just added some of the logos - on this branch, can you mockup what they would look like in the 2015 Companies section of the Supporters page? Thanks!


Add the possibility of a bottle of wine, color ain't matter.


When this not performed on the main thread I was presented with the following error:
``
CoreAnimation: warning, deleted thread with uncommitted CATransaction; set CA_DEBUG_TRANSACTIONS=1 in environment to log backtraces.
``

I was under the impression that was the purpose of the waitUntilDone functionality. As for the last parameter we should be waiting for completion (by bad). My python knowledge is quite limited and multi-threading even more so. I'll update and test later today. Thank you for feedback!


@zer0n I think this change has a bug. The side effect of this is, it causes the connection to reset for every call to execute causing every execute to be a separate transaction. So, code that used to work for "insert/update" in a execute call and "commit" in the subsequent call won't work anymore. This is related to discussion in issue https://github.com/uber/vertica-python/issues/55.


Hi @katierawson — This looks fine to me!


(Y)


This is Apple's Cocoa frameworks and not really anything to do with Python per se.


Ok, maybe it's a major case.


Todo en orden


Nice one


I agree with your comments. I think @grahamgilbert might have solution in place for this when this functionality is moved to an optional tab view in Xcode. My thought process was simply avoiding the need to add a  custom script to each workflow for naming purposes.


thanks @bitti09 :)


I don't get why this is the Ancestor Facet that points to a descendant
(and the Descendant facet points to the Ancestor).

i.e. Why doesn't the descendant Facet point to the descendant image and the Ancestor facet point to the Ancestor?


Defined in upload_test.go actually,  running integration tests caused an "already defined error". This change just causes integration_test.go to use the var defined by upload_test.go.

Alternatively Could just rename the integration_test.go buf12MB to something else like integBuf12MB. What do you think?


Nice enhancement, thank you.


You also need to make sure the parameter and gradients share the same
tensors, for the purposes of weight sharing. In the example you give, you
won't share the parameters/gradients.

The code is a modified version of this, see the comment:
https://github.com/wojciechz/learning_to_execute/blob/master/utils/utils.lua#L24

For an few details about parameter/gradient sharing in torch, this old
issue has a few details:
https://github.com/torch/DEPRECEATED-torch7-distro/issues/156

On Fri, May 1, 2015 at 5:17 PM, mszlazak <notifications@github.com> wrote:

> Why not use something like the following instead of
> model_utils.clone_many_times,
> function makeClones(model, numClones)
> local clones = {}
> for name, module in pairs(model) do
> if module.getParameters then
> clones[name] = {}
> for i = 1, numClones do
> clones[name][i] = module:clone()
> end
> print('Cloning\t'..#clones[name]..' '..name..' modules.')
> end
> end
> return clones
> end
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oxford-cs-ml-2015/practical6/commit/96749c8d9bc93f864c94c048a3c8cd73f59f733b#commitcomment-11003495>
> .
>


what to check?


@schisamo I literally installed `omnibus` for the first time today so it's completely up to date. Fresh install has this problem.


Github already has `position:sticky` on the issue sidebar, so I thought it'd be nice on the repo too.

Firefox only right now, but Chrome [is working on it](https://code.google.com/p/chromium/issues/detail?id=231752#c32).


What about a new FfixedSizeLevel flag for more flexibility ?


nit: isn't this just imageId


Here's a spot check of 3 more reactions that have MI in the status.

rxn00115 has status "MI:H:-3|O:-2|CI:-1" and definition CO2 + H4N <=> CHN and there are 3 extra H and 2 extra O on the left hand side.  The name of the reaction is R00152 and on KEGG the definition is CH202 + NH3 <=> CHN + 2 H2O which is balanced.

rxn01120 has status "MI:C11/N2/O1/P-2|CI:2|HI:-22" and definition C17H25N3O17P2 <=> (glycoprotein cpd00273 which has noformula).  It has an alias of R01536 but that reaction is not found in the current KEGG database.

rxn01400 has status "MI:H:8|O:5|C:1" and definition C8H11NO3 => C4H9NO4 + C5H10O4 and there is an extra C, 8 extra H, and 5 extra O on the right hand side.  The name of the reaction is R01913 and on KEGG the definition is the same with a comment of "incomplete reaction".

BTW, there are 71 reactions with a status value of "OK|HB" although 13 of the 71 still had a charge imbalance indicated in the status value.

What is the value to keeping reactions with a known problems in the biochemistry?  There are thousands of them.


@r3nya а два раза — это два раза.


So this is a hard and dirty exit, any reason why? This will kill our ability to profile, right?


Such plot, many twist, wow.


This API makes me nervous -- trusting output buffers to be large enough.

I'd prefer the returned strings be allocated.  Something like:
````
void
_ostree_parse_delta_name (const char  *delta_name,
                          char       **out_from,
                          char       **out_to)
{
  g_autoptr(GStrv) **parts;

  parts = g_strsplit (delta_name, "-", 2);
  /* Transfer the parts to `out_from` and `out_to` accordingly. */
}
````


It should, yes.


Fixes the green lasers in Mega Man Zero's intro and mostly fixes the intro to Pokemon FR/LG. Also code cleanup.


The problem here is that this function will be called too much, as the parent function (https://github.com/nearform/process-container/blob/master/lib/container.js#L50) is called for each topology.
You should put this into a 'service', like the docker-registry and the aws-autoscaling-container.


Hey @bruinkid, looking at this now.  Out of curiosity, be sure to have the fastclick.js file enqueued in your theme (it's enqueued by default in functions.php).  We've noticed that the Foundation hamburger doesn't work when that library isn't enqueued.


Thank you, I will take a look.

BTW, here is a shorter version of Wojciechz code:

https://github.com/wojzaremba/lstm


The state machine is broken for a clean exit, and so it fails... frequently.
To wit, most of our Travis failures in the past 18 hours have shown some
evidence of this.

We need to stop the bleeding on Travis first, and then figure out how to
fix up things longer term.

You're right on our ability to profile.

On Fri, May 1, 2015 at 9:30 AM Vijay Pai <notifications@github.com> wrote:

> So this is a hard and dirty exit, any reason why? This will kill our
> ability to profile, right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ctiller/grpc/commit/ff1ad1308edd7f790ea3c6644b7254af79f58787#commitcomment-11003669>
> .
>


uml is great :dancer:


I confirmed with xemacs21-nomule (sid), APEL (https://github.com/wanderlust/apel), FLIM-1.14.9 and SEMI-1.14.6.  `(require 'mime-edit)` raises error because `mime-edit.el` contains below code
```lisp
(mime-encode-field-body
 (concat " 1.0 (split by " mime-edit-version ")\n")
 "MIME-Version")
```
Above code raises error `Symbol's function definition is void: char-charset` on non-mule environment.


I added into comments:
 * $s - string to test
 * $n - needle to search


Stray comment here - from debugging perhaps?


To make things simple: let's suppose there is a single element (`N=1`) which gives a positive loss (`Li[1] > 0`) then we have:

* `gradInput1` (`gradInput` with respect to `a`) = `2 * (n - p)`
* `gradInput2` (`gradInput` with respect to `p`) = `2 * (p - a)`
* `gradInput3` (`gradInput` with respect to `n`) = `2 * (a - n)`

And `self.gradInput = {gradInput1, gradInput2, gradInput3}`. Are you OK?


Yup, you may prefer to try that one if you want to try fbcunn.

On Fri, May 1, 2015 at 5:32 PM, mszlazak <notifications@github.com> wrote:

> Thank you, I will take a look.
>
> BTW, here is a shorter version of Wojciechz code:
>
> https://github.com/wojzaremba/lstm
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oxford-cs-ml-2015/practical6/commit/96749c8d9bc93f864c94c048a3c8cd73f59f733b#commitcomment-11003703>
> .
>


>Плагины надо пересобирать?

Нет, не нужно.


Tengo entendido que eso se hace con el swing. Creo que ya el radio button por definicion es para eso, para elegir una opcion nada mas.

El may. 1, 2015 1:16 PM, "Cristhian R. Cantero" <notifications@github.com> escribió:
En el caso del radioButton, habria que ver como poner que elija solo una de las opciones en los radioButton. :question:

---
Reply to this email directly or view it on GitHub:
https://github.com/ccantero/algo2GenericApp/commit/3351bd2e9267b6e4ef4ba4a3edbd906097824021#commitcomment-11003483


Cool stuff! Thanks to @dylanmei for making `winrmtest` :grinning:


Are we okay with just up and breaking the public API like this?  Shouldn't this function be deprecated and just throw a "not supported" error if called?


Hey @bruinkid , I can't seem to reproduce the error you're experiencing.  Let me know whether fastclick.js is enqueued so we can continue to troubleshoot.

Thanks!


Awesome! Thanks! I've been struggling on the socket unit tests, glad to see you were able to get it!


Let's keep it, what if a commit passes a travis run but breaks one of the unit tests?


@MennaDarwish No line should exceed 80 characters


Am I right in thinking that you're only using `..` in pattern matching to designate the fact that the position should match multiple entities?


You misspelled an oil.


Why the isFinite? If an item is `Infinity` then I just want the result to be `Infinity` as well.


Indeed... Will remove that one while rebasing...


Abhi, I think you can open a ticket with ASF infra to proxy asking Travis to increase build timeout to say 120mins instead of 50mins?


Thanks how do I use this stuff ????


It's hard to know which sign is correct without knowing the exact nature of the atmospheric winds, but this is implemented with the reverse sign in gfdl ice model to rotate ATM stress to OCN  tripolar grid. I.e.,
```
             Ice % flux_u_top(i,j,k) = u*cos_rot(i,j)-v*sin_rot(i,j) ! rotate stress from lat/lon
             Ice % flux_v_top(i,j,k) = v*cos_rot(i,j)+u*sin_rot(i,j) ! to ocean coordinates
```

And also there is the business of putting the stress from A grid to B grid (cell center to cell corner) if your ATM stresses are on cell centers.
Look at the gfdl ice_model.F90 in any public release code and search for sin_rot


@MennaDarwish Delete white space trails


I didn't thought about it.

Do I create as many of flags than prefixes ?  (FshortFile|FlongFile|FshortFunction|FlongFunction)

And what about the size ? On flogLvel, no problem, we already know the max size, but not on the others.


And this one needs to be removed as well! Also doing that while rebasing...


Donc techniquement la carte double de taille.


Should this be "false" to be consistent


Well played


*spits out coffee* WTF, this lets Mario & Luigi: Superstar Saga boot.


Should this be bRetVal = true; for consistency


:bow:


The summary format has a so-far-unused `extensions` dictionary.  Couldn't we just stuff signatures there without breaking the format?


Can you tell what is the reason about this change?


@brentp or @tanghaibao could you push this to PyPI? The latest version there is v0.1.4.


Seamless teleport is e.g. used for the garrison system. If you are entering your garrison the character will change the map. An example for this is the garrison in shadowmoon valley. Draenor's map id is 1116 and the garrison map id for level 2 is 1158. If you are entering the garrison area, the server will teleport you to map 1158. If you are using a seamless teleport, like on retail, you won't get a load screen on map change.


The three reactions you spot-checked are fine for me, regardless of the
variations in the status message itself, I can see that the calculations
were done correctly (and the reactions no longer compare with KEGG, as they
are out-of-date).

The reactions that are listed as OK|HB were proton-balanced, but still
charge-imbalanced as you say. This group of reactions belong to a single
AraCyc pathway for fatty-acid elongation that I needed in my plant models,
but I couldn't figure out how to consistently balance the charge (changing
the charge of a cofactor in these reactions had an impact on the
charge-balance in other reactions).

In my mind there's two main reasons for keeping unbalanced reactions in the
database, 1) We anticipate being able to increase and fix the number of
balanced reactions we use in the database, for example, rxn00115 may now be
balanced according to the update in KEGG; 2) Record-keeping: we're
integrating public databases, and users will want to find individual
compounds and reactions in our database regardless of their utility.

On Fri, May 1, 2015 at 11:30 AM, Mike Mundy <notifications@github.com>
wrote:

> Here's a spot check of 3 more reactions that have MI in the status.
>
> rxn00115 has status "MI:H:-3|O:-2|CI:-1" and definition CO2 + H4N <=> CHN
> and there are 3 extra H and 2 extra O on the left hand side. The name of
> the reaction is R00152 and on KEGG the definition is CH202 + NH3 <=> CHN +
> 2 H2O which is balanced.
>
> rxn01120 has status "MI:C11/N2/O1/P-2|CI:2|HI:-22" and definition
> C17H25N3O17P2 <=> (glycoprotein cpd00273 which has noformula). It has an
> alias of R01536 but that reaction is not found in the current KEGG database.
>
> rxn01400 has status "MI:H:8|O:5|C:1" and definition C8H11NO3 => C4H9NO4 +
> C5H10O4 and there is an extra C, 8 extra H, and 5 extra O on the right hand
> side. The name of the reaction is R01913 and on KEGG the definition is the
> same with a comment of "incomplete reaction".
>
> BTW, there are 71 reactions with a status value of "OK|HB" although 13 of
> the 71 still had a charge imbalance indicated in the status value.
>
> What is the value to keeping reactions with a known problems in the
> biochemistry? There are thousands of them.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ModelSEED/ModelSEEDDatabase/commit/c2e4824b9ca05a6230cc68a101b2ce0a4b2395e2#commitcomment-11003663>
> .
>

--
Postdoctoral Fellow
Mathematics and Computer Science Division
Argonne National Laboratory
9700 S. Cass Avenue
Argonne, IL 60439

http://www.linkedin.com/pub/sam-seaver/0/412/168
samseaver@gmail.com
(773) 796-7144

"We shall not cease from exploration
And the end of all our exploring
Will be to arrive where we started
And know the place for the first time."
   --T. S. Eliot


base code


@MennaDarwish Delete all white space trails.


For safety, can you empty out $LIBS here? it might contain other libraries.


@dlusi is this still tentative?


DocsReadyForMerge (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1641 of 1641 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 would have moved this too root-gg/utils for reuse in future projects rather than making it plikcontext dependent


Again I think this is implemented with the reverse sign in gfdl ice model:
```
Ice % u_surf(i,j,k) =  u*cos_rot(i,j)+v*sin_rot(i,j) ! rotate velocity from ocean
Ice % v_surf(i,j,k) =  v*cos_rot(i,j)-u*sin_rot(i,j) ! coord. to lat/lon coord.
```


return bRetVal;


Shouldn't this better become an issue entry?


It's hard to know which sign is correct without knowing the exact nature of the atmospheric winds, but this is implemented with the reverse sign in gfdl ice model to rotate ATM stress to OCN tripolar grid. I.e.,

             Ice % flux_u_top(i,j,k) = u*cos_rot(i,j)-v*sin_rot(i,j) ! rotate stress from lat/lon
             Ice % flux_v_top(i,j,k) = v*cos_rot(i,j)+u*sin_rot(i,j) ! to ocean coordinates
And also there is the business of putting the stress from A grid to B grid (cell center to cell corner) if your ATM stresses are on cell centers.
Look at the gfdl ice_model.F90 in any public release code and search for sin_rot


BOOLEAN bRetVal = false;


this change broke some stuff and fixed others. I wasn't able to understand why / what to do about it.


just nitpicking, but you can probably skip the test -n since if $1 is "mandatory" it's also a non-empty string.


I'd prefer these to go into PyInstaller/utils/windows/..., because they are not only related to hooks. We could re-export them in hookutils, though.


no need for the quotes


@renaud I'd like your input on this TODO. Thanks.


sandbox/c++/cppshell/CMakeLists.txt ?


Hi Steve, After I update my HAL to this version, somehow I got grey image (CV_8U) instead of Depth image (CV_16U) for the Depth Channel. Would you please have a look at this?


@samoht Thanks! I will obviously use it.


First off all, I've been adding folders ad-hoc, so if you want to re-arrange the repo, I've no problem with that.

Here, I only looked at compounds that are in the ModelTemplates (listed in Biochemistry/Workspaces/KBaseTemplateModels.cpd), and printed the list of compounds for which formulas differ between the two databases (Priortized_Differing_Formulas.tsv)

I invented a "code" for delineating the different ways in which the formulas differ:
EMPTY: both compounds have no formula but they differ in the string used (should we go with simply an empty formula?
MISSING: one compound does not have a formula
MATCH: after parsing and re-sorting the formula (if it has multiple components) there is no difference after all
ELEMENTS: either formula has missing elements, in some cases, the "element" is a parenthesis or lower-case 'n' indicating a polymer
NUMBERS: both formulas have the same number of elements, but differ in their number. By far the most common difference in the formulas is simply a change in the number of protons.


Nem kellett volna array-re alakítani. list.get(i)-vel az i.-edik elemet veszed ki.

Sőt: [API docs on AbstractList](http://docs.oracle.com/javase/1.5.0/docs/api/java/util/AbstractList.html#equals%28java.lang.Object%29)


I didn't realise. I wouldn't have done it this way if I knew. I thought that the parent function was only called when the process container module was initially required (when the kernel boots.) Can you show me how this is called so I know? :)


I named it to look like lua51 because that's what the official luajit makefile does for windows `lua51.dll`.  I know windows is picky about renaming things.  Even if this is required for windows (which I'm not sure about), we could use some other name for the rest.


Fair enough!


Awesome !


this is unrelated and does not impact anything


@samaasallam


This commit fixes issue with `ExtractSameCoocurrences`. The problem was that Sherlok was not looking for fields in the superclass (here: `ExtractCoocurrences`).

However, it now makes sure that using `RegExAnnotator`, or any other class that doesn't use `@ConfigurationParameter` annotations, result in an error.

`sbt test` currently fails because of that (`RegExAnnotator` is used in bluima.regions pipeline).

The question is, how should it be fixed? Should annotation be added in bluima (strong requirement) or should sherlok be clever enough to find default value even if no annotation are used (weak requirement but I have no idea how to do that easily...)


Here it is: https://github.com/nearform/nscale-kernel/blob/master/lib/containers.js#L143

That code is a hack to support per-system config on the current APIs.


Estrutura da classe aluno


should there be an else statement? If by some chance the file's size is less than 0x177 then line 111 will cause the masterKey buffer to be outside the file's contents. If the file is less than 0x126 bytes then line 103 will read beyond the buffer.


You might instead take an array of key ID strings here to reduce overhead, since you're just going to loop over them at the CLI layer.  I wish we had done that for signing commits.

This could also allow the function to normalize the key strings and catch duplicates, if we care to do so.
(see https://bugzilla.gnome.org/748257)


@imikushin With the exposing of the ParseRestartPolicy from docker, we can use their function to parse restart.

https://github.com/docker/docker/pull/12762


try try try to separate them
it's an illusion
try try try and you will only come to this conclusion
love and marriage love and marriage go together like a horse and carriage
dad was told by mother
you can't have one
you can't have none
you can't have one without the ooottther


I know, but that's sadly a limitation of Sublime.


check that yearOffset + sizeof(SYSTEMTIME) is less than dwFileSize


yes because i moved all the tests of ant inside maven, now all the tests run with maven, and none with ant.


Here i get a cast issue between BaseSignatureChecker and DummySignatureChecker.


Zrobiłem mały refaktoring ;) Trzymajmy się języka angielskiego w kodzie, a polskiego przy wyświetlaniu informacji. Zwracajcie też uwagę na ostrzeżenia z IDE. Jak używacie Netbeansa to po zmianach używajcie Ctrl+Shift+i (optymalizacja importów) i Alt+Shift+f (optymalizacja kodu)


Because it is like a parent token. I flatten it out.


I believe with the restructuring of the Godeps in https://github.com/rancherio/os/pull/270. We can use the exposed policy now. Unless this is a totally unrelated issue (which it might be. ;))


That's a great idea!


check that masterKeyOffset + 32 < dwFileSize


It's usually better to do:
```
while True:
    if condition:
        break
    # do stuff
```
rather than
```
flag = False:
while not flag:
    if condition:
        flag = True
    # do stuff
```


columns_definition_children_tokens, this is the children tokens. And I fill it with all children tokens.


this show the change of number of tests in the maven suite : http://helios.orientechnologies.com/job/orient-maven-nightly/


:metal:


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 22](http://teamcity.otenv.com/viewLog.html?buildId=1285755&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 23](http://teamcity.otenv.com/viewLog.html?buildId=1285756&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


Muy bien.


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 22](http://teamcity.otenv.com/viewLog.html?buildId=1285755&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 23](http://teamcity.otenv.com/viewLog.html?buildId=1285756&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


// for consistency
if (pbMasterKeyStripped) *pbMasterKeyStripped = false;


this probably needs a explicit initialization with `... dummyChecker()`?


Can we use [`.hide()`](http://api.jquery.com/hide/) and [`.show()`](http://api.jquery.com/show/) instead of adding and removing the hide class ?


@pcasaretto: I checked our current production webhook payloads for deployment events, and I'm not seeing this attribute. Can you share a recent example that shows that this attribute is included in the payloads you're seeing?


return bRetVal;


ну `cell.textLabel.text = self.candidatesForDeletion[indexPath.row][@"title"]` же


yes, its in the queue to make this a configurable warning/error and deprecate the bad syntax to allow this eventually always be an error


Thanks Cedric, you're absolutely right. I have replaced it with `commerce_paypal_price_amount()` (even though I don't like it 100%), and, using the opportunity, added better currency handling.


Regardless, are we sure we want to be signing the `extensions` part of the summary?  Not sure of its original intent, but it strikes me as analogous to detached metadata.


It is a token called columns_definition and it groups all single column definitions.


@rawan-abdelhalim Good Job :dancer:


Why not use:
bRetVal = GetSha256(key, sizeof(key), sha256);


What was the issue with connection pooling you had? I think we can leverage the built-in connection pooling in `pg` without much changes. [Here's](https://github.com/anttih/purescript-node-postgres/commit/ba234b277b9df919940f97c2a71514e49228a434) one crack at it. Needs a bit more testing but I think that's about it.

As for node-any-db, I'm not a fan of abstracting over the different db layers. I'd also personally prefer to keep this module as small as possible with the minimum amount of dependencies on the JS side. Like you said, maybe another PS project would be the place for that sort of stuff.


I don't like this, shipping modules that already have pep8 exceptions... Bleh!


In CESM, we have a convention that all vectors passed though the coupler
have
a specific reference.  that is east and north.  each model has to rotate
to these
reference directions before sending vectors and rotate into their model
reference
after receiving.  i think we have to do something similar in NEMS. if
each model
has to know what the relative rotation angle is of every field in every
other model
in order to rotate onto their coordinate system, it will never work.
the only way
to do it is to pass around the rotation angle for each grid in each
model to each
other.

you cannot know how to rotate unless you know both the source and
destination
angle, reference point, and direction of rotation.  it's much easier to
pick a standard
for the mediator and force each model to send/recv data on that
coordinate system.
east and north are arbitrary and "break down" for pole points, so maybe
there is
something better.   but i think we need to specify a standard, just like
with units and
what positive values mean in fluxes.

thanks,

tony..........

On 5/1/15 9:46 AM, Niki Zadeh wrote:
>
> It's hard to know which sign is correct without knowing the exact
> nature of the atmospheric winds, but this is implemented with the
> reverse sign in gfdl ice model to rotate ATM stress to OCN tripolar
> grid. I.e.,
>
> |          Ice % flux_u_top(i,j,k) = u*cos_rot(i,j)-v*sin_rot(i,j) ! rotate stress from lat/lon
>           Ice % flux_v_top(i,j,k) = v*cos_rot(i,j)+u*sin_rot(i,j) ! to ocean coordinates
> |
>
> And also there is the business of putting the stress from A grid to B
> grid (cell center to cell corner) if your ATM stresses are on cell
> centers.
> Look at the gfdl ice_model.F90 in any public release code and search
> for sin_rot
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/feiliuesmf/nems_mom_cap/commit/be985d8821d83efbeed86cc9b26ad7d533cf14d9#commitcomment-11003892>.
>


Ilyenkor akár lehetne final is a list. Nem fogunk ennek a refenrenciának új értéket adni, s a konstruktorban töltjük fel.


You should use `const EQ_DELM = "=";` here.  As this is written currently these are global variables because they don't have a `var` in front of them.


code needs to be aligned


@deniseschannon I'll be happy to reuse docker's function once it's available. I'm a big fan of deleting code, especially, my own.


I'm I started be involved  in being monitored


BOOL bRetVal = false;


Itt csinálhattad volna azt is, hogy handCards.addAll(entry.getValue()). Igen van olyan metódusa a listának amely egy másik listát kap paraméterül, s annak összes elemét beteszi az aktuális listában (valszeg az implementációja pont az amit itt leírtál)


What scenario(s) is this fix targetting, and is it actually possible for self.sgs to be updated in that scenario, between iterations of the above loop?

(self.sgs is updated by resync_security_groups, and by security_group_updated.  profile_rules is used from the resync thread, endpoint_created and security_group_updated.  Therefore I guess the relevant context is endpoint_created followed immediately by security group creation - but does the scheduling allow self.sgs to be updated in that scenario?)


code reviewed


I'm hoping it simply Will be over soon   how  video and all  that works


urgency=low ? (I'd say major personally)


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


@rkirov – Now you can do `bootstrap(Compt, [routerInjectables])`. Should be a bit cleaner. @vsavkin recommended this to me.


Someone Got to really teaching me


a list-en van olyan metódus, hogy sublist(from, to), ami egy metszetét adja vissza a listának. Ezt felhasználva lehetett volna handCards.addAll(cards.sublist(0, 6));-t hívni.


return value from VirtualAlloc should be checked


Looks good.


@rafaelfranca @matthewd did we say we should just use Exception#cause here?


thanks, will look at it shortly!


develop (kbenne) - x86_64-MacOS-10.9-clang: OK (1133 of 1133 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)


This assignment is to cache the inverse of a matrix, rather than computing it manually.


完成部分:
增加轉向方法1. 直接改變方向 2. 利用向量相加方式計算心的方向
增加維度限制1. 不限制 2. 只有X軸 3. 只有Y軸 4. 全部限制
增加最大速度、加速度及減速度屬性
增加移動: 依照上述速度屬性以及方向及現在速度控制移動，並有轉向速度控制(只有在向量相加模式)
將角色方向由角度改為弧度，只在顯示時轉換為角度
增強觸控判斷


I've added in quite a few changes, some dependent on upstream libraries
which I forked.  The upstream changes were necessary to get more info on
AWS errors so that I could add recoverability for transient errors as
well.  Still some in there killing my process occasionally, haven't kicked
them all yet.  I've also linked it to https://github.com/ezoic/log4go, so
that we can see what is going on with this in a production environment.

Are you interested in getting all the changes or just the ones the commit
you referred to?

Dan

On Thu, Apr 30, 2015 at 9:58 PM, Harlow Ward <notifications@github.com>
wrote:

> Nice catch. Any interest in creating a PR w/ these changes back to
> original repo?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ezoic/kinesis-connectors/commit/2ff748a2d4c1dbcf4ccf727d09e3c1c4dbdedcc0#commitcomment-10997755>
> .
>


It would appear that imposing the no shorting restriction on the ask quantity significantly reduces market liquidity: in every simulation without shorting the model reaches a state where the order books are for all assets are uncrossed (i.e., no further trading is possible).


I guess it makes the parse easier and also after parse columns, I keep a cleaner token_list without any whitespace token. Also makes it easier to get the column definition, otherwise you have to filter out all whitespaces.


There should be a default value defined for this option because old instances will break w/o updating the config, like so: https://gist.github.com/anonymous/b4b419fbd537db1f423e


@SimonGoring This line throws an error if you uncomment the block and run the tests. Essentially you have:

```r
expect_that('site' %in% class(get_site(get_download(1))), # <- this line is incomplete
expect_is(get_site(gpid='Canada'), 'site')
            shows_message('API call was')) # <- this is stand alone
```


:grinning:


it's .... beautiful


BOOL bRetVal = false;


Ah ok, I haven't played WoD so I don't know its features.


seems not, I will delete it.


This breaks the build atm.


nice commit message :)


sure, agree.


LGTM


@lipsitzm What do you think about having the model be an optional binding?


Thanks @ruffrey!


Somehow the NEI handlers use StatCollector directly - I just kept the existing usage.


Why is this needed? I think `attributes` do not have to reserve anything, isn't it one of the main goals of this PR?