So,

I think the skip-binder here is not correct. I think the *correct* thing to do would be to do the whole skolemization protocol, like an impl, but I can't come up with a good use case for when this would be necessary. Therefore, I suggest we do the following instead:

1. Modify `assemble_candidates_for_unsizing` to consider higher-ranked `Unsize` obligations unimplemented. This means calling `no_late_bound_regions` on the obligation -- it will check if any higher-ranked lifetimes exist and return `None` if so. In that case, we can return without creating any candidates. Otherwise, if it returns `Some(o)`, `o` will be the same as the result of `skip_binder`, except for the guarantee that there are in fact no higher-ranked regions (which `skip_binder` doesn't give you).
2. Then in this code we can assert that there exist no late-bound-regions by calling `no_late_bound_regions.unwrap` instead of `skip_binder`.

This should let you remove the TODOs of course and leave the code mostly intact. If we ever DO decide we want builtin impls of `Unsize` for things like `for<'a> &'a T: Unsize<&'a U>` or whatever, we could go ahead and do the correct skolemization stuff here -- but I don't think it's needed, and it'll just make the compiler slower to do it.

Here is a comment trying to explain why I think the higher-ranked considearations are moot here:

```
// We currently never consider higher-ranked obligations e.g. `for<'a> &'a T: Unsize<Trait+'a>` to be implemented. This is not because they are a priori invalid, and we could potentially add support for them later, it's just that there isn't really a strong need for it. A `T: Unsize<U>` obligation is always used as part of a `T: CoerceUnsize<U>` impl, and those are generally applied to concrete types.
//
// That said, one might try to write a fn with a where clause like
//     for<'a> Foo<'a, T>: Unsize<Foo<'a, Trait>>
// where the `'a` is kind of orthogonal to the relevant part of the `Unsize`.
// Still, you'd be more likely to write that where clause as
//     T: Trait
// so it seems ok if we (conservatively) fail to accept that `Unsize` obligation
// above. Should be possible to extend this in the future.
```

That said, the right steps to take would be something like what `collect_predicates_for_types` does, I think:

1. Skolemize the obligation, producing a skolemization map and a version of the obligation with no bound regions.
2. Produce the `nested` vector of obligations below just as we do now.
3. Call `self.infcx().plug_leaks` to convert any skolemized regions that appear in those nested obligations into bound regions.


*Doesn't*, *after update*


Hello


<Russian>Калибр: 6,5x47 мм Creedmor
Патронов: 30
Используются с: MXM</Russian>


Cool, I am fine with it.


The `-Index` case is subsumed by the next case.


couldn't lines 374-378 become:
return !bSomeErrors;


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.34](http://teamcity.monogame.net/viewLog.html?buildId=6704&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 485, ignored: 6 Build time: 00:15:29


OK great. I think most of the changes look solid. I'd prefer not to lock in to a specific logger so maybe I can create a logger interface that we can pass in (similar to what you did with DB).


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

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


blabla


cool, sounds good.


![2015-05-01_19-25-53](https://cloud.githubusercontent.com/assets/6517606/7434105/f06e932c-f037-11e4-9a15-52333f4c7138.png)
i've forgotted that you switched to "franco"kernel :D


O que é controverso aqui?


Should all or most TCHAR buffers be converted to WCHAR buffers since they are being used with UNICODE string functions instead of tchar string functions?


Cool.  I'll put together a PR and send it through with the changes, and let
you work out the logger issue.  Might not be today.

On Fri, May 1, 2015 at 10:25 AM, Harlow Ward <notifications@github.com>
wrote:

> OK great. I think most of the changes look solid. I'd prefer not to lock
> in to a specific logger so maybe I can create a logger interface that we
> can pass in (similar to what you did with DB).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ezoic/kinesis-connectors/commit/2ff748a2d4c1dbcf4ccf727d09e3c1c4dbdedcc0#commitcomment-11004402>
> .
>


What about something like this (warning: untested)?  I think it's clearer and only loops over the array once:

```javascript
function matchToKeyValue (match) {
    var keys = [], values = [],
        splitMatcher = match.replace(/ /g,'').split(/[,=]/);

    $.each(splitMatcher, function (index) {
        if (index % 2 === 0) {
            keys.push(index);
        } else {
            values.push(index);
        }
    });

  return [keys.join(KEY_DELM), values.join(KEY_DELM)];
}```


Vows is the only one I didn't update: The newest vows@0.8.4 waits forever for something after finishing all tests and printing the final OK. Unsure how to debug; reasons not obvious from `strace`, so likely a stray uncalled callback—don't know where.


It's more consistent if we choose to always initialize the error in either the caller or the callee. Because the number of functions in a call stack that take a MonoError should vastly exceed the number of places where they're allocated, I elected to always initialize on the caller side. Making this caller-only init would keep things consistent.


couldn't lines 423-425 become:
return bAtLeastOneDriveOk || !bSomeErrors;


See previous comment about caller-side vs callee-side initialization.


It's already `clean_html`'ed in `_html_search_regex`.


Shouldn't this also include some version flag in use with the middleware version check?


Good catch. I'll think about it and provide a fix later. Feel free to open a new issue for keeping track.


code reviewed


Regardless of how this is implemented it is best practice to explicitly declare these variables using `var` instead of implicitly declaring them.


I don't like the resizing of icons on hover. It causes other elements on the page to move and resize. The jitter isn't useful.

If it's an icon-only link, an inverse box would be better


BOOL bRetVal = false;


Porque supostamente não preciso desta mensagem e só está aqui para testes noutras palavras provavelmente vou apagar


A message here could be useful rather than NULL


@changun It looks like this commit removed all the PAM images from the project, maybe as a side effect of moving to Android Studio.  Was this on purpose?

I get an error at the PAMActivity::loadImages() when running.


Also, as far as I know "splited" isn't a word, it should just be "split".


LGTM


code reviewed .. there is an extra empty line


return bRetVal;


How about...

    Try-expressions have the same semantics as try-statements in JavaScript, though in
    CoffeeScript, the catch and finally parts are both optional.


documentation reviewed for this file


code reviewed .. missing some spacing layout


Would be worth clarifying why the API is so tilted towards performance at the cost of usability.

One would have thought the harness interfaces can do any encoding necessary.

Is it because this API is called repeatedly many times, making performance critical? Is it supposed to be used for coloring in the editor?

Sticking a piece of comment here would help.


Any change anywhere in the tree causes an automatic recompile.

Joseph
Sent via mobile

> On Apr 29, 2015, at 09:43, eddiebrook <notifications@github.com> wrote:
>
> really?
> because android have un MAME core update from few days ago (2015/04/22)
>
> iOS no update since 2015/03/31
>
> —
> Reply to this email directly or view it on GitHub.
>


sure "can contains" is correct?


Code Reviewed


Ran into this problem myself. Curious to know why "javascript://" is the preferred noop, rather than just removing the href altogether?


when the queue:worker in daemon mode attempts to send queued emails.


This is much more secure


Lgtm


I just wonder if the description here is as useful as it was before we removed it at https://github.com/Metaswitch/calico/commit/7e86b112115b260627ad362b3a0cd4f795786495.  Could you please just cross-check against that commit, and reinstate any better descriptions that you think are worthwhile?


1. Cool, seems if a attribute has two word, it is still within one token. But if it is not, then we will discard it.
2. I think we talked about it before. If not, how can we handle those MySql specific attributes. I think we can keep it like this and add more support for other database specific attributes if needed.
3. Sure, I can look into that.
4. Yea, I can add them back.


Sorry for not noticing that. Fixed in 2eb0192155d53d70ae89ab8553ead49feb860ea4


is this a good choice?


Does that mean FELIX_FELIXHOSTNAME?  Or is that one handled specially?


I thought about that too...it was a mistake to have it be public API.  Removing it will show on ABI diff tools like http://upstream-tracker.org/   - but clearly no one will be hurt by it.

It would be better though to change it as you say, I agree.


Same thing here with the `var`, all variables should be declared explicitly, otherwise you get fun things like this:

![screen shot 2015-05-01 at 1 39 28 pm](https://cloud.githubusercontent.com/assets/4116405/7434262/8811b154-f007-11e4-978f-ec1142cb0321.png)

If you assign a value to a variable that has not been declared, it will automatically become a GLOBAL variable.


Really nice, thanks.  Just a few points above to address on Tuesday.


R.I.P. Grooveshark


@jlucerond there's no need to make a `mutableCopy` of `self.array1Strong` since it is defined as a `NSMutableArray` via `@property (nonatomic, strong) NSMutableArray *array1Strong;`. In fact doing this changes the data and that's why the test for this method fails. Now, why does it change the data? Allow yourself 20 minutes to dig into that, but no more.


if i get the system impl by @expez it assumes that clj-refactor and middleware versions are the **same**


LGTM!


"any confirms leaks"

typo


Hva er det her


Ben: Hey you two want to come over for a `beer` and hang for a night?
and that was the last time anyone saw Colin or Matt


This implementation takes a 9-10% speed hit when tested against 9451 putative variants.  Output pipes through var2vcf_valid.pl to produce valid VCFs in both amplicon and non-amplicon mode.

Would you be interested in an update that accepts an "-a" argument for amplicon mode?  I tested it and there is no appreciable speed hit.


It seems modestly fast.
https://gist.github.com/bayashi/6c243af7377820909f36


This should not be needed, I think range handles the nil.


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


Think he means this: https://github.com/clojure-emacs/clj-refactor.el/blob/master/clj-refactor.el#L37 has to be changed to remove the `-SNAPSHOT` prior to release.


@dh10 in the grand scheme of things, including ``vardict.pl``, ``teststrandbias.R`` and ``var2vcf_valid.pl``, how big is the effect? Does it slow down the whole process by 9-10% including all the steps?

At the moment the cool thing is that the scripts pick up amplicon mode automatically so I'd be interested in an implementation that doesn't rely on arguments. Could potentially read just one line from stdin, check the # of columns and then use the old approach.


please change the string  accordingly


Colorization/Classification is usually a hot path and performance-critical. flickers are usually annoying and we try to avoid them. @CyrusNajmabadi did some profiling on the previous API and found it to be spending a considerable amount of time deserializing the JSON output, thus the change.


90946736_WSHP_#4567 (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1644 of 1648 tests passed)

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


Great article, @Manishearth, thanks! Not having the relevant background in strongly typed languages, I often struggle understanding the generics debate in Go. Can you elaborate on what you mean by "we lose out on the ability for advanced type safety here"? If it fits with the spirit of this publication (and hasn't already been exhaustively covered in articles I just happen to have missed), it'd be great to have an article fully describing the advantages/disadvantages of generic types in various languages.


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


Hacky I think


"titre"?
Typo?


FYI, I'll be happy to accept a PR with those extra packages. If you have more R packages that would be useful, just open a PR with them.


bah, names. I get it now. Can't think of anything that makes it more obvious though.


i guess both needs to be 1.0.5 and your check will warn unless middleware version is 1.0.5 too...


Não consigo enviar!!! :'(


In the current spec, "complex attributes" are basically considered new resource objects (this is probably actually a better way of putting it). I *think* we need to retain that, but that wrinkle might make this idea a little less compelling.


no love for checkboxes? How does one add them now, if they are not in the templates variable?


Thanks for catching this Tom. I'll merge this in a bit.


Can you combine this into the loop above, using a second variable for total size?


This worked out nicely.


@michaelkirk this repo is writing out fountain xml


I just mean that I don't get strong static type checking here. Which means that refactoring is harder; and I have to be careful to insert dynamic checks for an externally exposed API; among other things.


Surely you mean ["oasis" "setup"]? My opam doesn't know of any setup command.


@dannynelson why a full-version bump for tests? (1.0.0 was already a major bump)


Oh, there was a fixed revision as well. Never mind then :).


Could we make this private? I'm just speculating, but I suspect an immutable hashmap is pretty slow as far as updates are concerned. Sense we are effectively mutating the Map anyway by replacing it with updated versions, could we just use a mutable map? Additionally, that will give a very logical object to synchronize on instead of the whole ```NonceKeeper``` class. That will remove the extremely unlikely possibility of something else using the instance of ```NonceKeeper``` as a point of synchronization possibly resulting a deadlock.


compile error.please check source


Can't `-PosFl` underflow?


@petrocc This made significant changes to how this worked before. I tried to use %include for the network line but that didn't work (from the kickstart). Unless you have a trick, it seems like we need to add the inline network modifications as we can't but the servers on the network at this time.

Here's what I added back in the code from previously but using %include /ks_scripts/network_include.cfg didn't work

```
        my $net_line = "network --activate --bootproto=static --ip=$ipaddr --netmask=$netmask --gateway=$gateway --nameserver=172.16.15.70 --hostname=$hostname --mtu=$mtu";
        # Fill in a default value for the network device if one isn't specified
        if ($dev =~ m/^\s*$/) {
                $net_line .= " --device=link";
        } else {
                $net_line .= " --device=$dev";
        }

        # IPV6 stuff
        if ($ip6_address =~ m/^\s*$/) {
                $net_line .= " --noipv6";
        } else {
                $net_line .= " --ipv6=$ip6_address";
                $net_line .= " --ipv6gateway=$ip6_gateway";
                # There should probably be some sort of error thrown if there is an ipv6
                # address without a gateway.
        }
        open(NF,">$cfg_dir/network_include.cfg") or die "Could not open network.cfg";
        print NF $net_line;
        close NF;
```


Fixed: https://github.com/goodeggs/mongoose-connection-manager/commit/41826607e345ea528d87b3320152435fdd04306a


Nice. Also (maybe) instead of terminating all these `struct conf_entry` arrays with a `{ NULL, NULL }`, how about telling `set_pointer()` the size of the arrays? Something like:
```
static void set_pointer (struct conf_entry *entry, size_t num, const char *name, const void *ptr)
{
  size_t i;
  for (i = 0; i < num; i++) {
    ...
  }
}
#define SET_POINTER(e,name,ptr) set_pointer (e, sizeof(e) / sizeof((e)[0]), name, ptr)
```
And only use `SET_POINTER()` instead. Looks IMHO a bit clearer and saves some space too.


Need add vim function (or command) to plugin/ directory, which users can bind on PreBufWrite


I like what you did here


What's the point of having keyprints after this change?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.35](http://teamcity.monogame.net/viewLog.html?buildId=6708&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 485, ignored: 6 Build time: 00:15:37


This logic looks a lot like the `generate` code from above. Could be consolidated.


I use this patch to run node js hello.js example and it doesn't work.  Browser complains line 333, and error message is "....the object's state must be opened".   If I use old one, no error and works well.  Please check it. Thank you.


I cleaned this all up in the actual PR, its hopefully clearer, extra variable didn't seem worth the effort:

https://github.com/mozilla/solitude/pull/392/files#diff-1ca48f9ed2d358b145384c66c5f57694R77


Why do this? I'd recommend against this. Is this because of a compliance test from the ASP.Net team?


I don't personally have a use-case for complex attributes but I'd be really curious to hear someone who does weigh in on this.


Can you please remove the nil check from here as well


copy doesn't make sense


update copy


Awesome!!!


should this be done before the site launches?


add the # sign and give it a name in the variables file and then call this name in this file


Longer term I think you want the ability to pass in an existing Container


done


@anselmbradford , why does it defaulting to 'MISSING TITLE'?


On a very huge map (I tested with full global) the used memory has decreased from 2920M to 2870M. Should make room for a few more players :P


This change here broke my app - the inserts don't work because the items dont make it into the crud operation


It looks good to me as far as I can tell, minus a couple comments on individual lines.


should this be done?


1. I had the same thought, at face value it feels gross, but is it really the same test twice? The source functions don't actually use one another, so I feel like they should both be tested.

2. null will fail the first part of the condition `!actual` as null evaluates to a falsy value


This is just wrong. It's my patch and you somehow duplicated and changed the author.


finally, proof that not all of @alloy's commit names are perfect


Sorry, I merged your other pull requests and then there was a conflict. I followed the instructions from github. so this happened.


Hey Damien, I just came up with this, any changes needed? Thanks!


Oh, well. Just commit any stuff after you merge and do git fetch. Otherwise `git rebase -i` and resolve your conflicts properly.


Aaah, OK. Thank you for the clarification!

On Fri, May 1, 2015, 12:55 Manish Goregaokar <notifications@github.com>
wrote:

> I just mean that I don't get strong static type checking here. Which means
> that refactoring is harder; and I have to be careful to insert dynamic
> checks for an externally exposed API; among other things.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/polyglotweekly/polyglotweekly.com/commit/d34507c181253f9ec7d590742772a2053e4ae73d#commitcomment-11004861>
> .
>


@mkaluza, recently I've compiled kernel with CONFIG_MMC_BLOCK_BOUNCE, and was quite surprised how this increased I/O performance. I did few tests in Quadrant, results as follows:
1) without patch: ~9000-9300
2) with patch  ~10300-10600
Both tests were done with following setup:  CPU 1200 MHz (performance), I/O: FIOPS, Fsync - dynamic, software CRCs disabled, file system - F2FS.
Just to ensure that system doesn't lie, I've installed kernel with patch again, and achieved same results as before - ~10600.


I'll remember that for the future. Thanks.


@danielblokus, bardzo dobrze!


relates to #2

still TODO:
  * alias `audio_read`, `video_read`
  * determine fallback if server version `< 2.17`


Use database name variable instead.


Se "New Research Group" é um parâmetro, deveria ser tratado como option no step anterior. Caso contrário, retirar aspas do cenário e daqui.


don't use inline styles


compile error.please check source


Not that this is performance critical, but how about we do strlen once and stuff it in a local to be used in all these places?


Magic of `commit --amend` and `push --force` ;-) (Normally, I don't force, but this was really embarrassing)


Then(~'^I can fill the field "([^"]*)" with value "([^"]*)"$') { String field, String name ->


Se o parâmetro field não é usado, não deveria existir.


ping @hvanoch


This is already included in standard gapps.  I'd done the same thing, but was told that and confirmed it to be true.


@hoytech Oh, thanks! After few works for MS Windows, then I'll send PR!


I think this could go straight into master.


You should add a docstring for the module to clarify that it defines "generic" exceptions.

You left the DTL-specific exceptions in `django.template.base`, which makes sense, but could be misinterpreted as an oversight.


phrasing doesn't make sense


I think you were right before with `serializeQuery`. Could you remove this commit and I'll merge? This is awesome btw - thank you :)


Similar ao caso anterior. O parâmetro field não está correto na expressão regular e parece não fazer sentido existir, pois não é usado de fato. Daí que seria mais adequado eliminar as aspas na expressão regular e corrigir o texto no cenário para ficar de acordo.


Thanks for your comments. I was thinking about the same thing and this is the main reason, why I chose a method as accessor for the lib. The interface is clear. I would do the optimization later, once we know what interface is really needed.
There is one difficulty with your suggestion though. The matcher for the PHP-language is no simple string, but a regex /\.php\d?/, which cannot be used as object key. It could still be optimized by creating an array for the few regex-matchers and an object for the many string matchers. Finally, I guess, the database could be splitted into multiple files which are `require`d only of the corresponding file-type is requested.
But all that can be done later-on once, the other parts are running.


Problema com field de novo


Why is there a translation followed by some hardcoded English text?


On May 1, 2015 8:14 PM, "Shilin Victor" <notifications@github.com> wrote:
>
> @mkaluza, recently I've compiled kernel with CONFIG_MMC_BLOCK_BOUNCE, and
was quite surprised how this increased I/O performance. I did few tests in
Quadrant, results as follows:
> 1) without patch: ~9000-9300
> 2) with patch ~10300-10600
> Both tests were done with following setup: CPU 1200 MHz (performance),
I/O: FIOPS, Fsync - dynamic, software CRCs disabled, file system - F2FS.
> Just to ensure that system doesn't lie, I've installed kernel with patch
again, and achieved same results as before - ~10600.

thanks. ~15% - quite a bit, though from the description here
http://cateee.net/lkddb/web-lkddb/MMC_BLOCK_BOUNCE.html I expected more (at
least I percieve my phone to be more than 15% slower than I wish it to be
;P)

Can you feel any real life difference?

Nevertheless I'll dust off my toolchain and rebuild the kernel with that
option on and get back to you.


Essa chamada não parece alinhada ao objetivo do step.


Where do you set the filenames, I only see SBG?


Existe mesmo esse método findByName?


I use a script which I will publish (somehow, shortly) that creates the
network line from a %pre script.

I'll send that to you under separate cover, and try to get the
documentation writting to better support this.

Regards,
Petro.
:wq

On Fri, May 1, 2015 at 12:01 PM, smalenfant <notifications@github.com>
wrote:

> @petrocc <https://github.com/petrocc> This made significant changes to
> how this worked before. I tried to use %include for the network line but
> that didn't work (from the kickstart). Unless you have a trick, it seems
> like we need to add the inline network modifications as we can't but the
> servers on the network at this time.
>
> Here's what I added back in the code from previously but using %include
> /ks_scripts/network_include.cfg didn't work
>
>         my $net_line = "network --activate --bootproto=static --ip=$ipaddr --netmask=$netmask --gateway=$gateway --nameserver=172.16.15.70 --hostname=$hostname --mtu=$mtu";
>         # Fill in a default value for the network device if one isn't specified
>         if ($dev =~ m/^\s*$/) {
>                 $net_line .= " --device=link";
>         } else {
>                 $net_line .= " --device=$dev";
>         }
>
>         # IPV6 stuff
>         if ($ip6_address =~ m/^\s*$/) {
>                 $net_line .= " --noipv6";
>         } else {
>                 $net_line .= " --ipv6=$ip6_address";
>                 $net_line .= " --ipv6gateway=$ip6_gateway";
>                 # There should probably be some sort of error thrown if there is an ipv6
>                 # address without a gateway.
>         }
>         open(NF,">$cfg_dir/network_include.cfg") or die "Could not open network.cfg";
>         print NF $net_line;
>         close NF;
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/petrocc/traffic_control/commit/7c06fa64767dc76d64e12bc988c049ddba0f5734#commitcomment-11004973>
> .
>


https://github.com/uber/vertica-python/issues/56 opened.


Why?  Nobody should be accessing this hash directly.  It is not public API at all.  If someone is accessing the hash, then it should be through a method that rubygems controls, and at that point we can return the default.  I'd definitely expect a test for this change.


@tomhurd Could you please review these changes?


hey there. the youtube video is broken (it just stops somewhere without any specific end).


remove useless break lines


it's good practice to omit closing tags. Can trigger "headers already sent" in some cases if there is data after the closing tag, could be an enter of space, hard to find


I haven't tested it heavily yet, first what I did is just this test :)


Please look at the PR this was in...


Fazer login é pré-requisito para acesso às funcionalidades do sistema. Daí que os passos necessários para fazer login deveriam estar em um método separado e deveria ser investigado o melhor jeito de chamá-lo, já que todo cenário de GUI irá precisar dele.


indentation


is there any more info on the site about any of these three projects, or about JSI? would be cool for this text to be links instead, even if the best we can do is a link to jsi.com.


It can't be static nor const.

https://github.com/miguelportilla/rippled/blob/ba28d305add440a15a57e8b9aa5452b0547971dc/src/ripple/rpc/handlers/AccountObjects.cpp#L103

https://github.com/miguelportilla/rippled/blob/ba28d305add440a15a57e8b9aa5452b0547971dc/src/ripple/rpc/handlers/AccountObjects.cpp#L108


@segiddins thanks......


Deveria-se saber qual é o field a ser preenchido com valor inválido


Thanks - for the time being I patched my system using the previous method (modified a little bit).


O parâmetro value não é usado


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


Se a ação é sempre clicar em criar, o texto do cenário e, por consequência da expressão regular, não está adequado.


falta o código para checar a mensagem de erro!


sweet thanks for that 411 Pat


I see your point on reaction names but if you provide a way for users to search for reactions by name then they will do it.  In any case, I'll still assert that the names of compounds and reactions should be unique so that when they are displayed when viewing models a user does not get confused by seeing the same name multiple times.

I did a quick analysis of the plantdefault_obs biochemistry object and there are 2,624 compounds with no searchname alias.  There are 193 entries in the compound_alias mapping in which the only key is "obsolete".


This is unecessary as avahi is automatically populated by callPackage. avahi ? null only matters if callPackage cannot provide the dependency in which case it is null. With this setup everything is enabled by default and has to be explicitly disabled.


PHP Catchable fatal error:  Argument 1 passed to Doctrine\DBAL\Migrations\Provider\OrmSchemaProvider::__construct() must be an instance of Doctrine\ORM\EntityManagerInterface, instance of Doctrine\ORM\EntityManager given, called in /vendor/doctrine/migrations/lib/Doctrine/DBAL/Migrations/Tools/Console/Command/DiffCommand.php on line 148 and defined in /vendor/doctrine/migrations/lib/Doctrine/DBAL/Migrations/Provider/OrmSchemaProvider.php on line 37


Complex attributes are just a useful way to group related attributes that don't make sense as their own resource.

For example, I have a complex attribute like this in one of my APIs:

````
"contact": {
  "twitter": "",
  "email": "",
  "github": "",
  //etc
}
```

Complex attributes should **NOT** be framed in the spec's text as resource objects. The key distinction is that complex attributes aren't directly addressable, while resources are. Complex attributes reserve `type` and `id` largely so they can't be made addressable, which prevents complex attributes from blurring into embedded resources and causing confusion about embedding vs. the `include` parameter and the `"included"` key. Meanwhile, `links` is reserved to allow the the possibility of linking out from complex attributes in the future. (This is a question that's come up over and over—see #383—though I'm actually becoming less convinced it's a good idea.)


2 reasons.
1. I don't think links go into translation. And splitting it up into 2 parts makes little sense to me.
2. The form is in English.


@iraj0711: if you have a problem, post an issue

http://www.trinitycore.org/f/topic/37-the-trinitycore-issuetracker-and-you/


would something like `.lead:not(:last-child) { margin-bottom: 10px; }` allow getting rid of this?


done


Thanks for the tip, I always write the redirect with curl and shouldn't.


@drmjo How did you trigger the error ?


LGTM. Interesting that it's not discovered until now, maybe we can expose it in the FileScanTest.


I believe my version is easier to read and understand. Tom, didn't you just mention the opposite in my last PR with the 'complex' condition?


This is what I exactly want to change . The hostname "www.7timer.com" is no longer accessible in China.


I wonder if we'd be better off using an assert?  This way we remove the need for the extra x86_interrupt function.  Does assert play well in MinGW?

```
#include <assert.h>

Change
if ((t1 | t2 | t3 | t5 | t6) & 0x1)
__asm int 3;

to
assert (((t1 | t2 | t3 | t5 | t6) & 0x1) == 0);

```


Which is correct.


I'd like to hear from @bintoro on this, as he had quite a bit to say about complex attributes. I'm personally fine with removing this restriction, but I also don't have a use-case for them (and as such, don't really trust my own judgement on the matter).


php app/console doctrine:migrations:diff

Throws the error..


LGTM


super picky, but we'd rather they read the FAQs before contacting us, so shouldn't the FAQs come first?


Will do.


#heeelllllooo


Gotcha. Thanks, Sindre.


should link to FAQs


This `response = self.client.put()` + `self.assertEqual(200, response.status_code, response.data)` seems to be repeated a lot. Test helper method?


Cenários devem ser escritos com exemplos concretos... qual artigo? quais autores?


Should't this consider only the last two characters? What happens if I search for "year:01"?


@pierreinglebert: this is a very first patch to get an idea. I couldn't test it and fix any issues due to that `npm i` error.

When I have something definitely working I'll make a PR :)


bleh -- I just sleuthed & fixed this same bug on Sinclair.


Esse step está redundante com o anterior não?
Lembrar que na hora de executar o teste o And é interpretado como given, when ou then, dependendo de onde ele tenha sido usado no texto do cenário.


Ah, I missed to bump the version in package.json :/


Hi there, great work on this lib! I noticed that it depends on itself though — this package.json line here. It's also weird now that the lib is version 0.6 but the dependency is to 0.4.

AFAICT, this isn't actually needed or used. Was this just an oversight? Cheers.


...and README.md subsequently.


Why not just `ns-path`?


yes you did but nevermind I'll do that


@automata does the ExtactLightness and ExtractSaturation follow some commonly used algorithm?
I was thinking that logic about "is this image bright, dark etc" would be done as correlations of the histogram with an envelope.
I think that would be able to express the logic in Extract* as just a special case: Envelope function would look something like:

    1.0          ________
     0     ____|           |
            0     tr         bins          where tr=threshold, bins=number of bins in histogram (typ 255)

This would be done in general JS helpers on persisted histrogram data.


So silver weapons can be both refined and reforged now?


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

This should go above the compiler statement


It is const.


I see, now JsonOperatorTests.flatteningApplyTest makes it fail.


I also agree that this restriction should be removed. Think about and complex array or object with a large number of levels - it can be difficult to check this on server side. And I don't see a requirement for this.


Cool @rroberge  could you make a PR for this?

(click green button on https://github.com/STIXProject/stixproject.github.io/compare/master...rroberge:rroberge-patch-1-revised-update-to-proposed-homepage-edits)


Não necessariamente... não há problema algum aqui.


the readme will be automatically generated by build-contrub task which is part of the default task


Wait, I'm wrong — the ordered set diff does need the entire library, which is what this whole commit is all about. Whoops. =) I see it was a temporary/quick fix. I suppose it works, but worth revisiting?


un truc cloche


I'm aware of that, I'm simply pointing it out in case you edit package.json via the web interface and you miss that.


Will do.


LGTM


Ok


just out of curiosity, why quote the prefix string with parenthesis?


This should be already defined in common.h.  We should do an #include "common.h" at the top of Mupen64Support.c.

```

/* macro for unused variable warning suppression */
#ifdef __GNUC__
#  define UNUSED(x) UNUSED_ ## x __attribute__((__unused__))
#else
#  define UNUSED(x) UNUSED_ ## x
#endif
```


```
All of the regular expression functions use the `Java pattern`_ syntax,
with a few notable exceptions:


Hi,

I'm wondering is there a reason why we would like to turn off transactions?


"is considered a"


This line can be deleted now.


indentation is off... we should really look into some sort of local vimrc, I do the same all the time. IIRC there's a PR for it, but it's incomplete.


I agree that there should have been a FileScan test that failed under this bug. I can write one that parses all the data types, with various truthy/falsy values in a Boolean column.


Ok.  Not sure why the change from options to option_setters, but presumably there's a good reason.


Technically this is valid under PEP8
```
Be consistent in return statements. Either all return statements in a function should return an expression, or none of them should. If any return statement returns an expression, any return statements where no value is returned should explicitly state this as return None , and an explicit return statement should be present at the end of the function (if reachable).

Yes:

def foo(x):
    if x >= 0:
        return math.sqrt(x)
    else:
        return None

def bar(x):
    if x < 0:
        return None
    return math.sqrt(x)
```


Please don't remove these definitions from config.hpp, they are required for building directly through the VC/CodeBlock project.


This addition seemed to bork our build process, saying:

    TypeError: Install of assemble to ~0.4.42 has no registry property provided.

How do we go about addressing this? I'm confused about this mandate, since you can't reliable depend on third-party libraries to provide the registry key in their package.json, but I'm probably misunderstanding the intent of this.


Deverá existir uma action no controller de Bibtex (similar ao de XMLImport) que recebe no params as publicações selecionadas e usa seus dados para gerar o arquivo bibtex. A implementação do teste seria apenas verificar que o resultado dessa action está correto, ou seja, que o arquivo é gerado e possui os dados corretos. Acho que dar uma olhada em como a coisa é feita em XMLImport pode ajudar, com a ressalva de que são tipos de arquivos distintos.


Ah, I see.

I was trying to get it to resolve the macro definition in MSVC but wasn't able to find it.
I did not know it was in that header.  I will update the pull request to just instead include that.


This will also need to get changed to FieldPane::theme($vars['element']);


link?


Új GUI osztályok felvétele, olyan metódusok implementálása, amelyekhez nem kell gondolkodni, plusz képek (csak mert elfelejtettem commit message-et írni)


Inconsistent with above lines (function at line 57 references createRemoveButton).


After watching Herb Sutter's talk on auto, he convinced me to start using auto for all local variables. I am okay with it and actually like it. But if you want I can change it.


Inconsistent with call from line 62 which is called createMOVEBUTTON.


To reduce merge conflicts and the like, and to make it easier to make a new branch for each minor release without a crap load of changes.


This means that I won't be able to do a number of bootstrapping/security things, like having the scheduler and controller-manager access the master using secrets.  I'm fine deferring that work till post v1, after which we should work on another way to do this.


inconsistent with line 65 which calls createMOVEDOWNBUTTON


Inconsistent with line 56 which calls createADDBUTTON


Addresses #64


I think I will. The problem is that I was using the api key provided for demo purposes. Well they revoked it. So rewrite? I have a private travis ci account.


The no 'searchname' alias was my treatment of BioCyc compound classes.

Very much like KEGG "generic" compounds, BioCyc maintains a hierarchical
order of classes of compounds i.e. sugar -> hexose -> glucose -> d-glucose
-> alpha-d-glucose and I removed the searchnames for classes before merging
because I didn't want them to get merged with any other possibly legit
compounds.

S

On Fri, May 1, 2015 at 1:28 PM, Mike Mundy <notifications@github.com> wrote:

> I see your point on reaction names but if you provide a way for users to
> search for reactions by name then they will do it. In any case, I'll still
> assert that the names of compounds and reactions should be unique so that
> when they are displayed when viewing models a user does not get confused by
> seeing the same name multiple times.
>
> I did a quick analysis of the plantdefault_obs biochemistry object and
> there are 2,624 compounds with no searchname alias. There are 193 entries
> in the compound_alias mapping in which the only key is "obsolete".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ModelSEED/ModelSEEDDatabase/commit/03768bff842f2d0934b241943d42bb05975a075c#commitcomment-11005391>
> .
>

--
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


Removed, invalid string...


@Muddahany : Code Reviewing done


True. It would be nice to get rid of the salt / shell that generates the system secrets for the master components.


You could just use angular.extend here:
```var data = angular.extend({body: '...'}, obj);```

Not a big deal, just a thought.


Plz show how to update d version


OK


Tests


LGTM


This needs to be explained in detail.


Oh, now I see that the documentation for node-postgres mentions that the pooling it has is naive, and that you should try node-any-db.


A lógica é estar em uma tela que possui a listagem de publicações e então selecionar algumas simulando a interação do usuário com a GUI. Isso é feito usando o Geb (e JQuery). Há vários exemplos de uso, basta ver os arquivos Page dos testes (rgms/test/functional/pages).


Did you give your reasoning for the move away from the .sh fixer to the GUI one somewhere?

Not a criticism, I'm just curious.


A lógica nesse step é apenas chamar a action em BibtexController responsável por gerar o arquivo. Acontece que o arquivo é gerado para uma dada publicação e esse step não sabe qual seria ela.


good best practice to move all colors and font sizes into variables, helps to keep things standardized


90946736_WSHP_#4567 (Nigusse) - x86_64-MacOS-10.9-clang: OK (1638 of 1642 tests passed)

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


While here consider making this `auto it = mLedgers.find (hash);`


looks good


Should December 2015 by December 2014, since December 2015 is in the future :confused:


looks like naming convention is usually @like-this ...so these should be @width-large, @height-large, etc.


1640-AddHVACPart1 (kbenne) - x86_64-MacOS-10.9-clang: Build Failed

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


Impressive! Did you use a specific tool to spot all those typos, or did you encounter them while playing the game? I've tried to stay alert to notice typos while playing, but I missed a lot of them it seems :)


The rsync above won't do that?


`findBugs` (with 'B' capital) just like all other entries.


Note: I switched from Komika to Fontastique (see dafont.com for the sources).


Alright. Sounds good then.


tested


a comment on why !important is needed would be helpful


absolutely :) i I tried to make it consistent throughout PR


you're most welcome!


Maybe I should no switch the sourceMap with file.sourceMap


int -> into


merge this first time


tested and working


.editorconfig should probs be in the root of the project, but I'm not sure on the best way to get that in there


Oh right, I meant to change that to `ptr as isize`, which should work properly.


Isso vai se repetir em todos os cenários, pois fazer login é pré-requisito para ter acesso ao sistema. Deveria ser definido um background.


The question is: what does naive mean here?


Ah :+1: thanks!


(We undone and reapplied in a separate commit.)


```
* Case-insensitive matching (enabled via the ``(?i)`` flag) is always
  performed in a Unicode-aware way. However, context-sensitive and
  local-sensitive matching is not supported. Additionally, the
  ``(?u)`` flag is not supported and must not be used.


surprised this isn't `inline-block`


Yeah, but if you're working in the theme or plugin, it'll pick up that one too, right? I figure, better have it here then leave it out of the repo.


Can you add double backticks around `(origin, status)`?


I'm sorry, what do you mean?


```
* ``\uD800\uDC00`` are not treated as ``U+10000``. Instead, use ``\x{10000}``.


Thanks Mr. Shillingford!
Reading in the files as ascii fixed all the problems.
Have to see whether they fixed the cloning issues with shared parameters ... that was a while back.
Fixing the cloning stuff plus your autobw will make things a lot easier.
https://github.com/bshillingford/autobw.torch


And also around `origin` and `status` in the rest of the text?


That actually dont make sense.
Because the output never will be '.as' but '.swf'


"does not handle a"


You can always move it to the root if you need to, but at least this way, at minimum, the theme will have it.


@Glenpl wystarczy `save()` i `load()`


Can you please change the name of the func appropriately?


```
* ``\b`` incorrectly handles a non-spacing mark without base characters.


"are not supported"


Remove


@Glenpl Po co zapisywać `lose`? Jest sens zapisywać ukończoną rozgrywkę? Co będzie po jej wczytaniu?


Yeah sorry about that had to get the packaged build fast. Did not even notice that.


Woop, thanks. Got overzealous with uncomment-region.


I'm new to this so I'm not sure this is the right place to post this.  I installed this update in my seedbox and I'm still having issues with not being able to connect.  I'm wondering if it's something on my end or is this an issue others are having


3 cenários repetidos, o que muda são apenas os valores do bibtex... para evitar tanta repetição de texto poderia-se usar "scenario outline" em vez de "scenario".


This change is certainly okay, but this file has a lot of Java warnings/errors. Some of them are just because of missing java doc, some of them are due to naming pattern, but there are two errors at the definition of "private static class AsList<T>".

I know this file comes from https://gist.github.com/jmkristian/1911614, but I guess we still need to follow our coding style requirement. Meanwhile, I feel this implementation is a little overkill for our need for Cartesian product, and I'm more comfortable with a simple implementation from ourselves, since It may give us better control and help other members understand it easier.


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

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


this should be:

```ts
   getContext(contextId: "2d"): CanvasRenderingContext2D;
   getContext(contextId: "experimental-webgl"): WebGLRenderingContext;
   getContext(contextId: string, ...args: any[]): CanvasRenderingContext2D | WebGLRenderingContext;
```


@Glenpl Przenieś `bombs_amount` do `Board` to będziesz miał jeden parametr mniej.

`hidden_fields_amount` – jest zbędne – `Board` posiada bomby to może je sobie policzyć.


@MacDada zaraz zmienię, btw szykuje się zaraz rewolucja w kodzie, i chyba będzie to rewolucja w stronę enkapsulacji :D niestety nie będą to jeszcze testy jednostkowe


Is there no way to do this with rails/erb?  This is very counter intuitive to me.  Since we are generating all of this HTML server-side it seems to me that we should make use of that capability if possible.


Just to make sure I understand, what is the difference between Differing_Formulas.tsv (which has >5000 entries) and Prioritized_Differing_Formulas.tsv (which has 436 entries)?


There was various bugs related to duplicated payments, shipments etc. so they were turned off to avoid that from occurring. I also believe there was a problem with nested transactions.  If you search the issues for transaction / state machine ones you can likely find more information such as this one https://github.com/spree/spree/pull/4499


shouldn't the logs go in the

'log' folder :o


nice! thx :)


@MacDada nic, ale wartość tej zmiennej i tak musi zostać gdzieś ustawiona na false - jest ustawiana jako parametr pobrany z pliku (takie brzydkie ułatwienie :p).


@Glenpl Uwględnij takie rzeczy jak:

- [ ] plik może już istnieć
- [ ] da się zapisać więcej niż jedną grę?
- [ ] możesz nie mieć praw do zapisu w tym miejscu (folder tylko do odczytu)


O parâmetro não é identificado e nem tampouco usado. Logo, deveria não existir.... daí que a expressão regular seria assim: And(~'I am at the Bibtex Main Menu$') {->
O texto do cenário também precisaria ser ajustado para casar com a expressão regular.


I am seeing the "matchers" being displayed twice in the select:

![screen shot 2015-05-01 at 3 11 11 pm](https://cloud.githubusercontent.com/assets/4116405/7435740/553f77ae-f014-11e4-9dc6-acb0355dcd18.png)


The build team has reviewed. Please let us know if this will need to be back-ported.


referings a folder named 'profiles' seems mainly to make sense when you run osrm in the original repo folder. maybe better to follow symlinks, to make them work?

we could also choose to remove the 'profiles.lua' symlink. you can just pass profiles/car.lua to osrm-prepare, which seems cleaner.


sounds good


fix "Dissect Structure" and others which use ParseStringAndWriteToAddress for CustomTypeFloat.


The list of compounds in Prioritized_Differing_Formulas.tsv is taken from
Biochemistry/Workspaces/KBaseTemplateModels.cpd

On Fri, May 1, 2015 at 2:09 PM, Mike Mundy <notifications@github.com> wrote:

> Just to make sure I understand, what is the difference between
> Differing_Formulas.tsv (which has >5000 entries) and
> Prioritized_Differing_Formulas.tsv (which has 436 entries)?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ModelSEED/ModelSEEDDatabase/commit/77e1f301ce3e6599aae7858eb7d231101f21a359#commitcomment-11006056>
> .
>

--
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


This is Editor code, and it toggles outline on vtkMRMLSliceNode https://github.com/Slicer/Slicer/blob/master/Modules/Scripted/EditorLib/EditUtil.py#L207-L210. I would think that should be more robust and correct than setting it on slice widget.


rax retry tests


Oh, cool! Just now pulling down the library but this is pretty much what I had in mind IIRC.


To napisz gdzieś plan działania – jak coś zupełnie nowego to rób issue. Dokumentacja ;)


sounds good.   I would say as standard practice we should probably just keep it in the root.  So keep it in repo here, but move it to the project root when you start making all the things.


`assert` also works.  I made this change, think I got them all this time:
https://github.com/cxd4/AziAudio/commit/921f424cfd6d8b54615f03ce83c4276892301f1a

plus looked over some comments that were raising the interrupt...I figure if it was #ifdef _DEBUG (Debug build in project settings) it would be easier to toggle on or off.

Both the MinGW and MSVC builds still play audio in Mario64 as a result of this change, so I think I did it right.


`raw_ptr` is more general as it applies to other pointer-like types not supported by `boost::get_pointer`, such as, for instance, [`offset_ptr`](http://www.boost.org/doc/html/interprocess/offset_ptr.html), which allows for [interoperatiblity with Boost.Interprocess allocators](http://www.boost.org/libs/multi_index/doc/examples.html#example12).


@MacDada hidden_fields_amount nie trzyma ilości zakrytych bomb, jeśli przy każdym ruchu będzie trzeba będzie jechać po wszystkich polach i liczyć to ucierpi na tym wydajność, ale mogę zmienić


Thanks


@Glenpl Kosz :P


s/cluster master/apiserver/


You can use this which makes the intent a little clearer in my opinion:

```javascript
$('.matchers').each(function (index, element) {
    match = $(element).find('.match').val();
});
```


Nevermind. I'm wrong. I like these.


@Glenpl Ta klasa służy tylko i wyłącznie do wczytania danych – nie kombinuj z optymalizacją dopóki nie masz problemu z wydajnością.

Potrzebne? Zostaje. Niepotrzebne? Kosz.

Z tego co rozumiem to niepotrzebne, bo można to sobie jednorazowo policzyć.


Ah, crap.  Git GUI diffs highlighting wasn't showing me that I forgot a stray `*/` comment ending.

Missed it since it's in the _DEBUG macro condition anyway; I'll do another quick commit for that too.


chamar o método de select usando "o" como parâmetro de forma direta assim provavelmente não vai dar certo. O método select precisa saber o elemento html que vai sofrer o clique e é muito improvável que ele seja identificado de forma idêntica ao texto usado no cenário.


Okay. I've removed the concept of complex attributes entirely. I still want to hear from @bintoro on this, though.


Ideally there'll be a wp install script that installs wp with the boilerplates and puts an .editorconfig in place, but I need to learn how to write the install script 😜


I have tried using os.arch, and it seems to be returning amd64 string for
me. Since the step is only for convenience loading anyway, I will leave it
with the one I pushed last time, since you can manually include if you wish
so anyway.

On Fri, May 1, 2015 at 12:07 PM, Sri Harsha Chilakapati <
notifications@github.com> wrote:

> Unfortunately, I'm no longer using a mac. My new GPU is incompatible with
> it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gamingchris3/jfreetype/commit/bbfe78e4a5d913c0ff18450621bcf75929b57111#commitcomment-10999243>
> .
>


ainda falta o código para verificar que foi exibida uma mensagem de erro


yo man


falta implementar


We want to make sure that we don't eat up someones GCE API call quota if there is, say, a problem with the cloudproviders requests.  Should the retry interval be less frequent for contacting the cloud provider, than for the apiserver?


90946736_WSHP_#4567 (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1644 of 1648 tests passed)

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


Как эскиз - вполне. Жду комментариев @FaceLess1.


@Glenpl Ogólnie w pliku ma być **minimalna** ilość informacji, żeby dało się odtworzyć grę.

Wszelkie nadmiarowości możesz dorzucić jeśli faktycznie gdzieś zaczyna mulić i faktycznie wymaga to optymalizacji :)


Parameters are not shared by default in clone(), but there's an optional
args for the names of the parameters to clone. AFAIK this doesn't work with
nngraph modules though, since you need to specify the names of the
parameters, but I could be wrong. If you look at the clone() implementation
in nn.Module it's very similar to clonemanytimes anyway.

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

> Thanks Mr. Shillingford!
> Reading in the files as ascii fixed all the problems.
> Have to see whether they fixed the cloning issues with shared parameters
> ... that was a while back.
> Fixing the cloning stuff plus your autobw will make things a lot easier.
> https://github.com/bshillingford/autobw.torch
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oxford-cs-ml-2015/practical6/commit/96749c8d9bc93f864c94c048a3c8cd73f59f733b#commitcomment-11005993>
> .
>


I load this property file with `Properties.load` method and value was `val34567890`.
So the test seems good to me.


It does allow wrong keyprints in hub connection as untrusted so why disconnect clients, there seem to be unkown errors with the verification so better pass the connection in those cases instead of disconnect, if allowuntrusted is disabled it should disconnect them.


The `<legend>` tag is generally only used within `<fieldset>`s.


@dblessing Just update your `Gemfile` to look like:

```ruby
gem 'omnibus', github: 'chef/omnibus'
```

Next perform a `bundle update omnbius` and you should be good to go.


Why 18? Isn't a bit high? What's not working with 15?


> oops, missed a file in my commit. Should be good now.

@e85fb0a6b6fa700447ab4b558769a1ae5dcace3e


fixed! :+1:


Sweet errors :tada: .  Anybody think we should not add the compiled css to sassyplate?  Not necessarily excluding via .gitignore, but just not adding them.


This implementation of Math.round() has problems for negative numbers.  Consider the following example run on Hotspot:

~~~~
System.out.println("Math.round(-1.9)="+Math.round(-1.9));
System.out.println("(int)(-1.9+0.5)="+((int)(-1.9+0.5)));
~~~~

Output:

~~~~
Math.round(-1.9)=-2
(int)(-1.9+0.5)=-1
~~~~


Yeah removing the symbolic link would be best. Also printing a warning if someone tries to point OSRM to a symbolic link.


@sethvargo can you bump the version and push this out? I'm running into a version issue running the currently published 0.1.1 and I think this is the cause.


oh man, even my beloved "#'*'" to disable all repos works! Thanks man!


You have checked in your AWS secret access key to a public github repo.

This is bad!  I have disabled your account for now and will re-provision you later.


or follow/expand a symlink before passing it to lua?


It seems like there should be a slightly better way to do this than to simply copy the definition. Is there common code to get defined system properties?


full gapps solved it.
thanks.


No, in the grand scheme, I think we're good to go with this as it is ... no overall slowdown.


This was the typo.


You may have success with this, since it accurately cover the horizontal and vertical pin case.
With your change was SF playing the quoted game better ?
http://talkchess.com/forum/viewtopic.php?topic_view=threads&p=622443&t=56109

You may find more ideas about pinned pawns here.
https://groups.google.com/forum/?fromgroups=#!searchin/fishcooking_results/pinned$20pawn

Among these G Lindscott idea was not the most accurate, but quite simple.
But will have an impact on all the pawn attack threats in the code.
https://github.com/glinscott/Stockfish/commit/fcb4c0d9e9c0a473f46ead42322579b3c8616c2f


Tab


Should we start with 0, so that we drop indexes once?


This is incorrect usage of `.form-group` which is meant to be used on a form group, not a table.


Don't forget `use strict; use warnings;`! :)


Logically it would make more sense to do this in validateAndNormalizeSchema and makeSchemaInfo, imho.


Hi @richey-v, thanks for submitting this PR. But the only difference I see is this has an entry menu.


OK, I can factor out a simple interface/impl for iterable-valued cartesian product of many iterables.


@westonruter This command results in an error:

```
fatal: pathspec 'phpunit.xml.dist' did not match any files
```


Oh, that's actually correct!
When I first wrote this criterion, I had in mind to recycle the embeddings for the positive and negative examples from the previous epoch. Then I thought I could recompute the embedding for all the images (anchor, positive and negative).
Now, I should have an option on the criterion which should let me choose between *recycling* and *fresh-estimating* the embeddings for the *positive* and *negative* classes.


seemed like a hack, but I just checked: Python also first follows symbolic links.


This shouldn't be needed -- and when running via `make test` or `prove -b`, it is not a good idea as we want `blib` to be used for the module source, not `lib`.


Hi @JDutil Thank you so much for your prompt response. I'm very new to Spree so the pointer is very helpful!

I can definitely see how turning off Order's transaction help resolve duplicated payment submissions since the Payment's `processing` state is updated within transactions and hence by definition isolated from each other. Yet I think turning off Order's transaction sacrifices a lot in terms of extensibility of the checkout flow because we lose any atomicity guarantee in various before and after transition callbacks and also there are no automatic rollbacks. IMHO, a better way to approach this is to use pessimistic locking in case of interfacing with something outside of our DB, i.e. locking Order or Payment when we need to process them with gateways


Why are you defining these prototype methods and creating these new recipes inside of your constructor function?


Generally it's better to test the exception that is thrown, rather than just "it died" -- it might be dying for a different reason than you think it is, masking other issues.


These `is()` tests can all be collapsed down to one with `is_deeply`, or `Test::Deep::cmp_deeply` (you're already loading Test::Deep).


@goern My pleasure! It was always supposed to, it was just hidden behind the bug you found!

Thanks!


corrected merge


Não vai ser publicado um novo artigo, vai ser cadastrado um novo artigo no sistema, né?!


I'm getting sloppy. Will fix.

I'm for putting compiled CSS in .gitignore


@deltheil, would you like to send a PL or would you rather prefer me to fix it?


Não usar verbo no gerúndio. Além disso, given deve expressar pré-condição para a ação no when. Se o usuário está adicionando um novo artigo, considerando que este é um cenário de GUI, ele está em alguma tela, certo? Então aqui tem que ser dito que tela é essa.


done.


Os steps devem sempre ser descritos com foco no sistema... ações que o sistema realiza ou que o usuário pode realizar no sistema, mas sempre ações de sistema. Esse step está muito vago e nada significativo... ele poderia sair.


you can use the `id`/`meta` argument for this


Blank Line


@dh10 thanks for checking, I figured `vardict.pl` would be several orders of magnitude slower that any of the downstream scripts so we have a little bit of flexibility here. There is a Java implementation to replace `vardict.pl` by the way that is about 10x faster than the Perl script. It's at https://github.com/AstraZeneca-NGS/VarDictJava/releases and works as a drop-in replacement for `vardict.pl`. It's also integrated in bcbio-nextgen https://github.com/chapmanb/bcbio-nextgen and will replace the Perl version as the default soon hopefully.


Hi @sherzberg

I am waiting on the release of Vault 0.1.1 which includes a fix we need to run the integration tests on Travis CI, then I'll release a new version :smile:


Cenários devem ser escritos com exemplos concretos... qual a letra inserida?


Quais são nomes sugeridos?


Blank Lines


No it isn't ;) SWIG gets defined by, er, SWIG.


ART stands for Astrological Research Toolkit.
The program is aimed at researchers in astrology. It will provide tools for statistical analysis, research of cycles but also single radix calcuations, including some less common features.
ART is written in Java 8.


> Complex attributes reserve type and id largely so they can't be made addressable

I now think `attributes.id` could really generate conflicts and should be probably disallowed, but unavailability of attribute `type` was a gripe of many, I think it should be allowed.

> which prevents complex attributes from blurring into embedded resources and causing confusion about embedding vs. the include parameter and the "included" key.

Will there be any confusion with embedding? Embedded resources will obviously have to have the same structure, i.e. hold attributes under key `attributes`.

Another consideration: possibility to directly address JSON-API-reserved keys at resource level: `meta` (discussion: #470), `links` (example: https://github.com/hhware/json-api/commit/51bb65abd695e1c9d618a6147db1fe3ff33823ad#diff-26e38355b69056b0db1c2b1c612241b2R932). This addressing is of no use now, but could be useful in the future, or could be useful for custom implementations. But if we look at the question from this perspective, we should reserve all such keys at attribute level too, which defeats the purpose of this PR.

So my updated suggestion is to prohibit `attributes.id` but do not restrict content of complex attributes at all, even `id` should be allowed in there.


@sethvargo cool, thanks. I'd like Vault 0.1.1 as well!


Initial readme


Se a sugestão depende de dados previamente existentes, esses precisam ser mencionados no given.


This long list of node traversals looks fragile.


omg ....  So sorry about that. We are trying to configure carrierwave -____-


Sure.  We could do it with ServiceAccounts instead.  But as long as the kubelet doesn't have an --api_servers argument, then it can't pull the secret.

If you implemented  a flag (separate from the --apiservers flag) that skips the register node function but lets the kubelet still try to post node status updates, and other things it is allowed to do, that would enable it.


My favorite commit of the day!


much better


The only thing is, then people have to delete that line when they copy it over.  Maybe?


deveria usar exemplos concretos... quais artigos? quais conferências?


What do you mean by this`? If you're talking about the mutiplex param for `createStream` children, we're doing the socket creation at the other end of the connection, and we're generating an `id` at that point.

This is just a (poorly named) method that sets up the arguments so we can call the socket body. I'm actually in the midst of cleaning this up right now as I'm writing tests.


wxTED can now import and export pages made in the edit mode of teletext40.com


@xray7224, I think the repo is missing this migration?  It was throwing errors for me and I think this is the source.


"signed" can have three values: 0 for unsigned, 1 for implicitly signed, 2 for explicitly signed. Comparing for inequality would also distinguish between 1 and 2.


The typo was the absence of "totalTimeAtStart".


This line needs a comma at the end.


Line needs a comma at the end.


Comma.


Comma


The shortcut key (ctrl-W) didn't work on my Xubuntu in the released version.

On 05/01/2015 03:28 PM, Rael Gugelmin Cunha wrote:
>
> Hi @richey-v <https://github.com/richey-v>, thanks for submitting this
> PR. But the only difference I see is this has an entry menu.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/richey-v/scudcloud/commit/35cae87caf8b2e1a210c1e8423a93a55bec1bffa#commitcomment-11006325>.
>


This particular HandleFunc is left out because the testing didn't use a fixture. If this refactoring is adopted I will be sure to go through every test and ensure consistent format, including this test case.


sure alright. should probably get combined eventually


typo?


Não necessariamente....


It would be great if you could also translate the 'help-xx' and 'unpin' messages. Just add a 'pt-PT': '.....' section after the 'en' one in the same way as the other messages.


nvrmind I see it is fixed later!


@wkennington Oh I didn't know that. Thank you very much! Fixed it


This looks like it should go into a utility function in C instead.


use: utils.isArray() instead.


Just move below snippet
if (xreq.setRequestHeader) {
            xreq.setRequestHeader('Accept', 'application/vnd.apache.thrift.json; charset=utf-8');
            xreq.setRequestHeader('Content-Type', 'application/vnd.apache.thrift.json; charset=utf-8');
 }
"RIGHT AFTER"
xreq.open('POST', this.url, !!async);
and works perfectly.


This doesn't look right.  We need to remove these lines completely.  I believe I did this when I was initially testing whether I could save my HLE progress in the N64 memory without having them become corrupted.


yeah I meant that the rng is :) will take care of it


There's some lines missing commas (causes a SyntaxError when loading) and a couple of untranslated messages. I have pt-BR colleagues but I don't know anyone confident in pt-PT so it would be great if you could add those. Thanks for this!


Hey - this fix is not actually pushed to the supermarket.  This change would prevent some of the Couchbase processes from restarting at every Chef run.  Could we get a new version of the Cookbook published to the Supermarket with this fix?


this is a weird location for this function.

How about metadata/debug-helpers.c?

You can use G_GNUC_UNUSED macro instead of explicitly checking for gcc.


I assume this is trying to find out if 'arg' has coerced? There is an `unwrap_coerced_node()` helper function.


We've also added pessimistic locking.  We're open to any improvements on this behavior if you'd like to work on it, but others have tried as well and run into other issues. That's not to say it can't be done, but so far this has been the least buggy solution developed.


Not sure why this used to function.


check


typo "there is more information"


Além da chamada do método deveria ser feita alguma verificação usando assert


this line and above one indicated same link and same meaning.


Thank you!


lgtm


yeah!


I like these block quotes.


mesmo problema de cenário anterior... no then deve-se verificar o resultado da ação do when... logo, é esperado o uso de assert.


Too many expects() one it(). You should use another describe to define what kind of test you are performing.


I can fill -> I fill


I made `details` an alias for `symbolic`, but that may not be the right thing to do. I could make it return the same type as `details` in v2.0.0. I did not do that initially because it seemed backwards incompatible, but now I realize that `details` is a new method. I think I'll switch it over to match the types.


@reubenmoes what's the thinking behind leaving the compiled css out?


I just laugh a lot with this repo name! Priceless! :D


Scenario: create a new Member web


This is just code re-indentation to handle rangeLookup being true or false. Don't be fooled.


Scenario: missing field error when creating a new Member web


@matthewdaniel and it would always fail before this commit?


It wasn't working it was super weird but I will try again it actually was
driving me nuts

On Friday, May 1, 2015, Tianon Gravi <notifications@github.com> wrote:

> Why not just adjust the -f above to be -f
> /root/rpmbuild/SOURCES/docker-core.tar.gz and skip the mv completely?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jfrazelle/docker/commit/7c1bd72b910d53d115acd588741ab9b896463fdb#commitcomment-11002830>
> .
>

--

Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu <http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3>


Noticed I had "equipped with an ordering ordering instance", changed to "Ord instance", minded the indentation and capitalized Map


Ficaria mais simples dizer que é gerado um arquivo html


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


Só faz sentido colocar ente aspas aqui se o dado for usado como parâmetro na implementação do step


thnx!


Isso deveria fazer parte do given


`loopvar_type` might be a user defined type, in which case parsing won't work. In fact, there might not be a way to find a `unsigned` variant of the given integer type.


deveria fazer parte do given


ok!


mais simples dizer que é gerado um arquivo pdf


@andmiriam  Adding the compiled CSS to the git repo isn't really useful at all, and it only just makes the git log a bit noisy.  If it's excluded from the repo, we can look through the git log and see the changes to just the src.


What is the output of CONFIRM_NOT_PENDING?


o sistema gera um arquivo pdf


The return value of StartTask must be checked.


o sistema gera um arquivo xml


yii2-composer is needed by composer packages that are of type `yii2-extension`. Since `yii2` no longer has extensions, this is no longer required. Did you meet any problem?


ToDo: Added locales


So mean.


@MacDada plansza jest kwadratem o boku board_size - czyli plansza składa się z board_size^2 pól. Ciężko, aby bomb było więcej niż pól w planszy :D


shouldn't these tests be testing something?


I would prefer a private `ops` with an accessor to avoid protected fields..


só precisa usar aspas se o dado for usado como parâmetro na implementação do step... isso vale para diversos outros textos de cenário


I'm 100 percent for leaving compiled css out entirely. My only question is staging.

For .gitignore, you're probably using a project's .gitignore, not sassyplate's, right? You'll remove the .git dir once you clone.


I think we would have to change the doctype to match the format it ends up with when golang marshals it as json. Probably not urgent to do.


VvnLeave a line note


outputting the title minus the symbol. So it looks a little cleaner on the screen.


Hmm, strange it was working fine on my test server. Not sure why! I'll go
check that stuff out.

*Neal Wright *
*Hendeca Design LLC*

*http://www.hendeca.com <http://www.hendeca.com>*
-----------------------
*Web/Graphic Design, Front-end Web Development*

On Thu, Apr 30, 2015 at 6:44 PM, Bryant Syme <notifications@github.com>
wrote:

> missing Chat and Login require
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bryzaguy/transient/commit/933f436306034ffb8de937b54d7ba262eb3e6f74#commitcomment-10996658>
> .
>


`describe "POST` :)


The log is a blank page, exactly like every other Travis page.

That looks like a clang warning bug.  As I mentioned above, there is no function body, so there is no place to use the parameter, so every parameter passed to an explicitly deleted function will always be unused.  Unfortunately, clang is currently broken here, so I cannot test that.  For me, it fails one of the early sanity checks, before it even tries to compile any Rebirth code.


This function creates a special matrix object that can cache its inverse.


Nice, thanks for fixing that stuff. Wasn't getting any errors on my machine, kind of weird that I wasn't!


This just broke my application.  The error here is with the second preg_replace.  Should be:
```
$json = preg_replace('/\s*$/m', '', $json);
```

The result of this commit removes whitespace from everything in a JSON object.


Nightly build failed last night again, this time due to:

https://github.com/playframework/play-ebean/pull/24

I'll kick one off manually now.  We do have some docs, but they are probably out of date.


Probably shouldn't do that. will make the change in the next commit. Just finished and sent one


That seems like a reasonable guess.


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


Looks good bro


+1 on the ++


not sure how just this file got separated into this commit... my mistake


Is the comment above this function still relevant, or can it be removed?  I find it kind of scary and cryptic. :)


rax retry tests


After looking at the commands in their entirety, I am uncomfortable with the approach of this commit.  I do not like downloading code unauthenticated over http and running it, even in the theoretical confines of the continuous integration platform.  I do not like building a C++ standard library with every build of Rebirth.  I do not like scribbling over /usr/include and /usr/lib, especially since some distributions do crazy things with lib/lib32/lib64.

Why do we need to use a private libc++ for clang at all?  I thought clang was supposed to work with libstdc++ when run on Linux.


cyclic namespace dependency error is somewhere in this commit.... hrmmmm....


requires a check for if completed data length is equal to partition data length


The space after the formatting should still go away. See original patch.


JvfLeave a line note


@moguideBY 有冲突，请解决冲突~ 谢啦


Uh


This changes the code generated for EDITOR && !NDEBUG.  There is no need to change this just to eliminate an overzealous clang whine.


Fixed, thanks Freddy!


Still looking into this one.


It's sanitizing values in comma separated tables, stripping whitespace around commas.


Can't we call sock_cq_check_size_ok() from _sock_cq_write()?


alright, I axed the memory bounds checks; after looking back I'm pretty sure I added those after adding the cart RAM view to the hex editor, but forgot to remove them after deciding to handle the same case in the editor itself instead.

on that note, is there any real purpose to giving unallocated MappedRAM a size of -1U instead of 0? I don't see any reason why a properly mapped instance of one would have a size of 0 (or why you couldn't just check for a null data pointer instead of a size of -1U if you needed to, which seems a lot more intuitive...)


@Selaginella99 Thanks so much for pointing this out! I found this comment very interesting too. In addition to the bottle neck, it also emphasized the importance of commmunity acknowlegement:

> Though systematic methods evaluation remains the exception rather than the rule, community-approved benchmarks can be tremendously useful even outside the confines of such a study.

which is one of the biggest issues whie doing methods comparison. This one would be very helpful for the DDD project. :smiley: :heart:


fetchManifestUrlFromSite ?


MULTI and EXEC are not forbiden command in cluster, as any other potential multi key commands you must just make sure that all key are on same slot...
Or if you consider that pattern as invalid, you should reject all multi key command as like mset mget or eval etc...


createFirefoxOSApp ? (notice "create" not "crete")


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.16](http://teamcity.monogame.net/viewLog.html?buildId=6607&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Tests passed: 485, ignored: 6; exit code 1 Build time: 00:15:24


Clang handles some trivial test programs, and seems not to exhibit this problem for me:
```
void a(int i) = delete;
int main()
{
	return 0;
}
```
```
$ clang -Wall -Wextra -O2 c.cpp -o /dev/null -std=gnu++11
$ clang --version
clang version 3.5.2 (tags/RELEASE_352/final)
Target: x86_64-pc-linux-gnu
Thread model: posix
$
```
Does this test program fail for you?


This should be merged into an existing helper class.  Normally, OData Web API is a server side library.


I considered pstypes.h, but since it compiles fine without `<cstdint>`, I decided to add the include in `"maths.h"`, which requires those types to be defined.  That should fix the problem you saw.


@EasyAsABC123 : when I try to that.. I get following error
hdesai-ltm:chef hdesai$ knife role from file roles/voldemort.rb
ERROR: knife encountered an unexpected error
This may be a bug in the 'role from file' knife command or plugin
Please collect the output of this command with the `-VV` option before filing a bug report.
Exception: NameError: undefined local variable or method `node' for #<Chef::Role:0x007fa935a9ba10>


No, that will not be useful to application.
If CQ size is low/empty, we should return -FI_EAGAIN while initiating tx/rx operation.


Create an issue to mention that currently the change support untyped scenario only.  Though, the ODataDeltaFeedSerializer support both typed and un-typed.


@chrismccord I think you may want to tag this commit as v0.12.0? The new version isn't up at https://github.com/phoenixframework/phoenix/releases yet.


I think we should leave the "f" at the end of the floating point constant or we will get a new compiler warning.


In fact I hesitate actually ..


可以在同一文件夹执行：只须各自有自己的conf文件即可（其中需要设置不同的 dir）


About single quotes...For consistency between languages, I suggest keeping double quotes for strings and using single quotes for chars.


also just for kicks on my machine i use ,n as waiting for nurse and nn as in with nurse , and , d as waiting for doctor and dd as in with doctor. Did not blow up... That's good


Doesn't twisted use this still?  Have you removed the abbrevmapping option?


You successfully ruined my world


All it does is eliminate a check against constant zero, right?


Oh, I'll make sure to remove it.

Glyph and several others have also requested we stop using it, so, there's little use to it otherwise :)


Prueba 1


As most names in the W3C Manifest spec are lower case what about using something like: "mjs_access_whitelist"

For me it is also not clear what _urlAccess stood for... like it wasn't clear that it was a whitelisting of URLs who could access extended APIs.


Gah, something every time -_-


@keedi - I have given you co-maint CPAN permissions on Thrust and Alien::Thrust. Cheers. :)


In fact I hesitate actually ..because then we could or should have both.. With a kind of cache of inherited attributes on each object.. Which could be invalidated when a used template has an attr updated.. Hmm


Lol


If you do not locally flip that constant to 1 before building, yes.


第一次玩插件哈。it is cool.


There's a major performance issue with dcadec that I've not seen before when building it as a shared object library.

Now, using this latest patch, any DTS audio stream will max out one core on the ARM processor when DTS HD support is enabled, resulting in significant stutter.

Something is wrong with how dcadec is now being built, as before in my test builds the CPU load would stay relatively low (about 30-40% on a RPi2 when playing a 1080/DTS-HD MA stream over nfs) and free of stutter.

I've debugged kodi and it is using the "libdcadec" codec, but I can't say why there is now such a significant performance issue.

I've done a completely clean build to confirm this behaviour (cleared ccache, removed all build directories etc.). Nightly builds from [http://openelec.thestateofme.com/dev_builds/](Chimney), ie. r20742-g9e684fd, also show the same DTS-HD performance issue so this doesn't appear to be the result of any other newclock4 patches that I'm including/excluding/etc.

Obviously there's something subtle I'm overlooking, but can't see what it is right now. The only obvious difference is shared works well, static works badly.


  ================================================================================
  Recipe Compile Error in /var/chef/cache/cookbooks/datadog/recipes/jmx.rb
  ================================================================================

  NoMethodError
  -------------
  undefined method `[]' for nil:NilClass

  Cookbook Trace:
  ---------------
    /var/chef/cache/cookbooks/datadog/recipes/jmx.rb:4:in `block in from_file'
    /var/chef/cache/cookbooks/datadog/recipes/jmx.rb:3:in `from_file'

  Relevant File Content:
  ----------------------
  /var/chef/cache/cookbooks/datadog/recipes/jmx.rb:

    1:  include_recipe 'datadog::dd-agent'
    2:
    3:  datadog_monitor 'jmx' do
    4>>   instances node['datadog']['jmx']['instances']
    5:  end
    6:

  Running handlers:
[2015-05-01T02:42:37+00:00] ERROR: Running exception handlers
  Running handlers complete
[2015-05-01T02:42:37+00:00] ERROR: Exception handlers complete
[2015-05-01T02:42:37+00:00] FATAL: Stacktrace dumped to /var/chef/cache/chef-stacktrace.out
  Chef Client failed. 1 resources updated in 5.812183239 seconds
[2015-05-01T02:42:37+00:00] ERROR: undefined method `[]' for nil:NilClass
[2015-05-01T02:42:37+00:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

@EasyAsABC123 : When I put the node.override then it gives the above error


@vovkasm  I'm trying to do `xcopy "$(ProjectDir)Resources" "$(TargetDir)Resources" /E` to get around this issue, but its exiting with code 4. What's the proper command?


(fixes issue #15)


Current can either mean "actual", or "how many amperes", so I think a better term would be "measured", or "actual".


这样也好。说明原来暴露出来的接口不够。


question, how can i find this type of fixes like you did? thanks


Recipe Compile Error in /var/chef/cache/cookbooks/datadog/recipes/jmx.rb
  ================================================================================

  Chef::Exceptions::ValidationFailed
  ----------------------------------
  Option instances must be a kind of Array!  You passed {"host"=>"localhost", "port"=>"7201", "conf"=>[{"include"=>{"attributes"=>["numberOfNodes"], "bean_name"=>"voldemort.cluster:type:Cluster", "domain"=>"voldemort.cluster"}}]}.

  Cookbook Trace:
  ---------------
    /var/chef/cache/cookbooks/datadog/recipes/jmx.rb:4:in `block in from_file'
    /var/chef/cache/cookbooks/datadog/recipes/jmx.rb:3:in `from_file'

  Relevant File Content:
  ----------------------
  /var/chef/cache/cookbooks/datadog/recipes/jmx.rb:

    1:  include_recipe 'datadog::dd-agent'
    2:
    3:  datadog_monitor 'jmx' do
    4>>   instances node['datadog']['jmx']['instances']
    5:  end
    6:


The next step for this change (and the other enum changes) is to actually use them to specify that the control_mode variable is an "autopilot_modes".  Similarly the set_mode function should only accept "autopilot_modes" instead of a uint8_t.


OUTPUT_ENABLED had some meaning. It's possible to enable or disable a channel, the output of a channel, or the whole device. I'm not sure if losing the distinction is the ideal choice. What did Uwe and Bert think?


My rule of thumb is "if it's interface-driven, don't use protected fields." But this is not interface driven... See also e.g. AbstractOpTest for a similar pattern in the test scope type hierarchies.


Como a implementação do step não usa parâmetro algum, deveriam ser retiradas as aspas em New Member, sendo necessário ajustar o texto do cenário também para casar com a expressão regular aqui.


This should probably check that `node_inps.owner.op == node.op`


if resolveWriteIntentError returns an error it should at least be logged.


c/v_hands doesn't have the punching animation, it only has admire


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)


Fix for #167.


It would definitely need to be removed once you start a project.  I'm fine adding a .gitignore.  You would clone, remove .git, remove .gitignore.


huh


Slight error here :P


Shouldn't sort and direction be outside the `if`?


Yes this might be a better idea since there is very little difference otherwise.


code conventions broken


deveria ser tela de show, não?


var naming should be different here too


This change needs to be made by anyone who has forked DemoSite and is affected by BroadleafCommerce/BroadleafCommerce#1337.


OK fine, I'm making it interface driven! :-)


Can you configure your editor not to use tabs?


This file is the wrong place to add this test. It deals with try-finally as a language construct, not with using it to test other constructs. The range optimisation test file seems more appropriate.


@cnjinhao DARN! I didn't mean to commit that! That's what I get for using `git commit -a`!


uhm. bit arithmetic is unexpected here. it hides the intent, right? also it's not quite middle, middle is left + right / 2, it is only correct while left is 0


so modern. such fun.


:+1: :eyes: good to me!


code conventions. if 'if' construction is single element only, it still should be on next line


I'm all for adding it to .gitignore as that should be removed anyway when using sassyplate.


I have one interesting idea for optimization. Are you going to support setting `process_update_function` while process runs? If not than it may be possible to increase performance by using lib_exec only when a new process is created or even only when a new widget type is registred.


brackets missing for if block


Dat byates-va


formatting


If possible, please provide a brief description on how to set up the dependencies on a Mac if it's possible. Not a heavy user and things can get a little complicated when in the beginning stages. Thank you.


A action show não é static, precisa de um objeto de controller para chamar. Além disso, essa action faz com que seja exibida a tela de show, exibindo os dados do grupo de pesquisa... o objetivo do step seria gerar um arquivo html com tais dados, como um formato alternativo de relatório (que poderia ser pdf, xml, etc...)...


test coverage is not sufficient


No, I think I'm okay with static update functions.


> (require 'mime-edit) raises error
> Symbol's function definition is void: char-charset on non-mule environment.

Ah, I see.  `(require 'poem)` may prevent this error.

I didn't meet this error because I used `M-x wl RET`.  And then I can create
a message and save it to +draft without errors on xemacs21-nomule.


why do you add these?


typo


O step deve informar algo que é feito apenas... se não é feito, não caracteriza ação no sistema, então não faz sentido aparecer aqui!


use `this.` here too for consistency


why not make this final?


@cnjinhao I rebased the commit and removed the change.


In general, writeFileSync and readFileSync will throw if something goes bad. Use try-catch around them.


I can select -> I select
Falar sempre o que é feito, não o que se deseja ou se pode fazer.


@carlsmith Send a PR.


same here if not intended to be changed, let's make it final


to MemberListPage
at MemberListPage


Esse step não faz sentido... ver comentário no arquivo de feature


attribute values in a tag should be quoted too. it generates not strictly valid html


These tests should go into the function docstrings, not the global docstring.


я слегка не уяснил 2 моментов:
1. какие именно и по какому конкретно вопросу от меня нужны комментарии
2. чем морфология орка отличается от морфологии, скажем, Фила Хита (прилагаю фото)
![image](https://cloud.githubusercontent.com/assets/4042329/7436507/fb3628f0-f056-11e4-90f9-1e749dec8f92.png)


to PublicationsPage
at PublicationsPage


as they're read only, how about initializing them in init?


This should follow the style of the other foreach macros.

See FOREACH_THREAD and the one for sgen hashtable.


why are these no longer errors?


esse step nao faz sentido... ver comentários anteriores


formatting


I know I was arguing for removal of `return`s. But after thinking about this I remembered that "explicit is better when implicit". So if your architecture requires each process to return something it's better to user explicit returns.


gj kappa


extract local variable here


see lists (i.e. li, ul, ol tags) in html


wow


@pierreinglebert: any chance you add the missing releases on GitHub?


This should be '//', also below.


Solo divide un XML en dos archivos (values e ids)

/* Por algo hay que empezar */


Actually, this is just a workaround for a kOS bug (which will be fixed in a future release): https://github.com/KSP-KOS/KOS/issues/934


ver


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


It isn't obvious to me what the advantage is to passing these in separately rather than in instrument_data_properties - it seems like it takes more code to accomplish the same thing?


this whole method can be substituted by `categories.values()`, right? why it's not static?


@Glenpl Racja ;)

To dorzuć do Boarda metodę pomocniczą: `get_fields_count()` – operacje boardowe, niech będą maksymalnie w Boardzie.


Is there a reason not to move `get_window_list(os_data)` inside `make_process_system_struct`? eg: make_process_system_struct(os_data, "update_window_manager",0)


Comment from web


TeamCity DnnPackager :: Continuos [Build 1](http://dazinator.ddns.net:81/viewLog.html?buildId=782&buildTypeId=DnnPackager_Continuos&guest=1) is now running


@MacDada w sumie to nie wiem gdzie Ci to napisać, więc zrobię to tu. Nie wiem czy kompilowałeś ten program, a nawet jeśli to musiałeś pewnie wywalic bibliotekę windows.h w i ogóle ble. Tutaj masz screen jak wygląda plansza w trakcie rozgrywki:

http://i.imgur.com/WfLyv9i.png

tak żebyś chociaż wiedział jak to wszystko wygląda :D

Jak widzisz, można podawać wiele poleceń na raz (patrz najniższa linijka) i dopiero wtedy klikać enter. Jest to wygodne szczególnie na początku rozgrywki, kiedy można wiele pól bez zastanawiania oznaczyć jako bombę albo pokazać.

Jak chcesz to mogę Ci podesłać jeszcze screeny jak wygląda ustawianie wielkości planszy itd i jak wygląda plansza zeskalowana na inne rozmiary niż 10 na 10 pól.


TeamCity DnnPackager :: Continuos [Build 0.1.0+20](http://dazinator.ddns.net:81/viewLog.html?buildId=782&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:23


what's the O(n) estimate for the computational complexity here? can you suggest what should we do to be able to list projects per category faster?


Forgot to sub-shell the nvm installation!


DocsReadyForMerge (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1647 of 1647 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 is a memset in disguise.

Can't we add a mono_bitset_clear_range function that does the trick?


Texto confuso. Sugestão: So that eu possa gerenciar melhor meus estudos.


Indeed. Fixed with https://github.com/facebook/jest/commit/68f3cbd941f848931fa65e428fc06740c8486c49 (and https://github.com/facebook/jest/commit/e84b782ef708bbc45aef0b73f18a3a0606e0b93c for actual docs)


Which bug is this? What's the issue number?


Se existem diferentes perfis de usuário, deveria existir um arquivo de feature para cada um deles.


In a future commit I've gone even further - now the system struct includes the os_data reference itself.


Thanks, fixed.


Done, sorry about that


Na descrição da feature é mencionado o perfil do usuário, certo? Aquilo é feito exatamente para se delimitar o escopo das ações/funcionalidades que podem ser realizadas/usadas, não devendo ser necessário especificar no texto do cenário o perfil do usuário.


You should probably document all fields in the comments. In other languages we have variable names but still managing to forget what information is stored in each field.


I wonder if this couldn't be done in a C macro instead of spelling it out in syntax nodes.


Deveria existir um step de when para expressar a ação principal do cenário e então o then deveria apenas checar o resultado dessa ação.


Looking at Collect_Biochemistry_from_Workspaces.pl, KBaseTemplateModels.rxn and  KBaseTemplateModels.cpd are generated by finding the conditional and universal reactions from all ModelTemplate objects in the KBaseTemplateModels workspace.  Do we also need to include gapfilling reactions in the analysis?  There are only 2,740 reactions and 1,813 compounds when only conditional and universal reactions are included.


To import the certificate manually as @purejava suggest works fine. But why is this necessary? Why is the certificate verification failing in the first place?


@sebworks Got point, probably not. @jameshupp What should our default title be?


اريد فتح هككست بليززز


@gitars
The distinction between using define and using define-syntax has nothing to do with whether the blocks are top-level or not.  Consequently you can use a define-syntax for your generic events and model it on define-event.  In fact, I have a sneaking suspicion that you can just use define-event itself.


instrument_data_properties are $key=$value and converted to a hash. These would need to be converted to description=$value etc, and immediately converted back to a hash. I would rather these get passed from the command line via @instrument_data_properties since they go into the %instrument_data_properties in the end.


Steps devem expressar ações, não habilidades, desejos ou intenções. Portanto, usar termos do tipo "I should", "I can", "I wanna" não é apropriado.
Além disso, then deve checar resultado de ação realizada no when, que erroneamente foi omitido desse cenário.


Nested locking.
This is changing the behavior of the mono_threads lock, which is scary/

Why do we need it both locks here?

How could we avoid this?

If it's not possible, we'll have to change those locks to be part of the lock tracer, update the checker
and update the docs http://www.mono-project.com/docs/advanced/runtime/docs/thread-safety/

locking


"but not impactful for the DNX" -> "but is ignored by DNX" ?


> `cnt`
pretty self explanatory.


Given -> And


Esse step não faz sentido... vide comentários anteriores.


deveria ter step de when antes... e aqui deveria checar resultado da ação do when...
esse problema se repete muito


The conditional and universal reactions are by definition the reactions
used to build the models directly based on annotation, and it is therefore
a good start to focus our curation efforts.

The gap-filling reactions, well, in my case in the Plant template, consists
of all other reactions in the database, so I'm ignoring those for now, and
I believe that some of the coarse rules we could apply during the focused
effort can be applied to the rest of the reactions too.

On Fri, May 1, 2015 at 3:18 PM, Mike Mundy <notifications@github.com> wrote:

> Looking at Collect_Biochemistry_from_Workspaces.pl,
> KBaseTemplateModels.rxn and KBaseTemplateModels.cpd are generated by
> finding the conditional and universal reactions from all ModelTemplate
> objects in the KBaseTemplateModels workspace. Do we also need to include
> gapfilling reactions in the analysis? There are only 2,740 reactions and
> 1,813 compounds when only conditional and universal reactions are included.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ModelSEED/ModelSEEDDatabase/commit/77e1f301ce3e6599aae7858eb7d231101f21a359#commitcomment-11007200>
> .
>

--
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


Can we use a define here instead of a magic number?


nice @ivogabe :+1: is this related? https://github.com/Microsoft/TypeScript/issues/2953#issuecomment-98183519


Somente professor pode ser administrador da disciplina né? Nesse caso, considerando que existiria um arquivo de feature para o perfil de professor e que nem todo professor é administrador de disciplina, faria sentido existir esse step.


Please revert this change, we want the default to be the BasicTemplateAlgorithm


The same happens 4 more times below.


Cenários devem usar exemplos concretos... qual disciplina?


Awesome doc! Thanks!!


I think we want to use "LWMA" + period here for the name