It is with the commit I just checked in earlier so no issue number. It's
not affecting anyone - it just means that new functions that I added to
kernel for fast retrieval of a cache of random numbers from GPU/boost
cannot be used from python, for now.

Maybe you have advice? For efficiency reasons, I prefer to use (float *) or
(double *) rather than vector when I create a cache of random numbers (the
cache size may be 10E+6 or more). To make it work in swig, I added:

%include "carrays.i"
                  |

%array_class(double, doubleArray);
              |

%array_class(float, floatArray);

It worked well when I only built kernel, but then made all the other
modules complain that they don't know what doubleArray or floatArray is.
Not sure why.

On Fri, May 1, 2015 at 1:16 PM, Ben Webb <notifications@github.com> wrote:

> Which bug is this? What's the issue number?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/salilab/imp/commit/960740ea248f1b6a2767defeea21012ce4ff58e6#commitcomment-11007155>
> .
>

--
Barak


This commit is pure gold. I love it!
It's a great piece of cleanup, the mechanics are now clear to me.


Awesome, thanks!


@HAAKS please delete the extra empty lines


@HAAKS please fix the indentation


@HAAKS please delete this line


@Glenpl Kompilowałem tylko w głowie :D

Dzięki za skrina ;-)


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)


This is indeed strange and should be fixed...


Can we add the QC license banner to the top? You can find it on top of any of the other .cs files in the solution.


In all cases where they are used in these scripts, throwing is acceptable since the steps are critical and we cannot proceed. Nicer output with error-checking is nice, but since these scripts are not intended for everyday use by humans, I think the value is insignificant.


@anselmbradford Return to the PR for the continuation of the discussion.


Please add the banner


The bug is #44, but it doesn't go into detail for why I decided to switch it. Really there were two main reasons.

1) It's not very secure to ask people to download random scripts from the internet and run them as root. Obviously I trust myself and would never make it malicious, but it makes my web server more of a target, and it trains people to do this, etc. So if it's simple to just change some settings in the GUI, then I think that's better.

2) Each new version of Ubuntu breaks the fixubuntu.sh script, and causes it to get more and more complicated. I think the GUI instructions will continue to work for much longer without needing to modify them, or worry about breaking things in different versions of Ubuntu.


Nice tests!!


Sure, my advice is to use std::vector. ;) Not sure what the "efficiency reasons" you're referring to are, but a std::vector shouldn't use any more memory than a dumb array, stores things contiguously in memory, and can be cast to float* or double* if you need it. Plus it handles memory for you and we already have SWIG support. Plain arrays are evil in C++.


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


Space before `{`


Thanks for the heads up, I was curious re VarDictJava.  We're looking to implement this as the workhorse for our production amplicon pipelines and have ramped up our evaluation/testing as of late.


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


The wording 'evenly distribute' makes me think that each point gets the same weight. Maybe just reuse the term linear? 'This indicator will linearly distribute the weights across the period.'


Put `result` at the bottom of this list, for the sake of locality.


Well, it's short name for count. In this particular case, it's current depth of recursion.


Fixes #3


Like I said: self explanatory indead.


lol


Yeah, I'm thinking about using the following scheme in widgets' update function:
```
set some_var to process[1].
set some_var2 to process[2].
...
// body of function
set process[1] to some_var1.
...
}


Braces


I'm still for explicit returns because in the `update_process` you require `update` function to return something.


Hello.

Build error:

../game/libgame.a(Player.cpp.o): In function `Player::TeleportTo(unsigned int, float, float, float, float, unsigned int)':
Player.cpp:(.text+0x86a15): undefined reference to `WorldPackets::Instance::UpdateLastInstance::Write()'
../game/libgame.a(Player.cpp.o): In function `WorldPackets::Instance::UpdateLastInstance::UpdateLastInstance()':
Player.cpp:(.text._ZN12WorldPackets8Instance18UpdateLastInstanceC2Ev[_ZN12WorldPackets8Instance18UpdateLastInstanceC5Ev]+0x16): undefined reference to `vtable for WorldPackets::Instance::UpdateLastInstance'
collect2: error: ld returned 1 exit status
make[2]: *** [src/server/worldserver/worldserver] Error 1
make[1]: *** [src/server/worldserver/CMakeFiles/worldserver.dir/all] Error 2
make: *** [all] Error 2


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


TeamCity DnnPackager :: Continuos [Build 0.1.0+21](http://dazinator.ddns.net:81/viewLog.html?buildId=785&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:23


re-run cmake?


rax retry tests


tested and working


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


ohh. thx for tipp... :)


Thanks for the review.  I can would be happy to add the banner, change the default Algo, and the other changes you suggest to the project.

How do I update the PullRequest after I have made the changes? That was missing from your writeup.


Good idea. lib_pid is already using this approach.


TeamCity DnnPackager :: Continuos [Build 0.1.0+21](http://dazinator.ddns.net:81/viewLog.html?buildId=787&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:18


This could use more meat but I wanted to get it out and testing. Right now Vne 158 causes wing damage. Left damage if banking right +10 deg, right damage if banking left -10 deg, both wings if > -10 and < 10.
Same conditions of bank with over g of 5.8 does same damage. I don't even know if you can achieve 5.8 g in the sim with the c172p.
Tonight I plan on adding it to master and merging all of master into internal-shadow-control.
Any suggestions, adjustments, additions of other properties are welcomed.


nod


What? `//Change cyclically "x"->"v"->"h"->"x"` made me think that `division` is a a string. Or does empty string evaluates to false?


These look super redundant. Make each of these triggers call a func with a param?


thats what I did @schisamo and it worked for me.

gem 'omnibus', github: 'chef/omnibus'
Rubygems source (https://rubygems.org/gems/omnibus) still depends on chef-sugar ~> 2.2


yayyyy


Can this be added to physiomapNode.js? That way the base class (and all other derivatives) won't need to know about sources, sinks and mediators


`division` -> `next_division_type`?


So I should just cast it as &v[0] - in that case is it safe to write to the
cast array?

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

> Sure, my advice is to use std::vector. ;) Not sure what the "efficiency
> reasons" you're referring to are, but a std::vector shouldn't use any more
> memory than a dumb array, stores things contiguously in memory, and can be
> cast to float* or double* if you need it. Plus it handles memory for you
> and we already have SWIG support. Plain arrays are evil in C++.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/salilab/imp/commit/960740ea248f1b6a2767defeea21012ce4ff58e6#commitcomment-11007322>
> .
>

--
Barak


Are the interfaces you've made here to mock the file system generic enough that we could use them in other contexts? I can see this being useful in other places.


Also, lets talk about adding the text to links so you can get rid of the middle node.


We actually don't need the ```nTx > 0``` check here, because ```HAVE_DATA``` implies ```nTx > 0```.


try parse could be better.


Oh, I should make it more clear. I meant that `division` is a boolean, which, if true, causes the division type change cyclically as shown in comment (no division, vertical one, horizontal)


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


I could see a use case for keeping a per-column annotation like a consensus sequence after removing columns -- say the consensus was calculated separately or emitted from an HMM, and the sequences present in the alignment are being compared to that consensus and iteratively or manually deleted/trimmed by some criteria related to the fixed consensus sequence. In that case, I would not want the annotation line to suddenly disappear while I'm trimming and curating my alignment.

It is also easy to delete the annotation manually, isn't it?:

    del align.column_annotations["secondary_structure"]

Probably easier than adding the annotation back if I did want to keep it.


see my lower comment


TeamCity DnnPackager :: Continuos [Build 0.1.0+22](http://dazinator.ddns.net:81/viewLog.html?buildId=789&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:18


"In a world, where an `s` stands alone, one man...."

What is this?


@ctsims does this accurately reflect our current recommendation with respect to triggers/labels? The form builder currently creates labels as triggers with `appearance="minimal"`.


Yes, as long as you don't write past the end of the vector of course.


Can we move this out to a config file?


True. I'm going to change it to be:
```python
bool(orig_bound1.type.signed) != bool(orig_bound2.type.signed)
```
or is there another way that you had in mind?


Agreed.


Looks good!  Thanks!


Tyler, it's all about tooling. ASP.NET 5 projects get great support for Bower task in VS 2015 and it's indeed becoming a new accepted way of managing client side packages. MiniBlog is still a traditional ASP.NET web site project and nuget is still the way to manage external dependencies in these projects.


Thanks. Fixed in https://github.com/xwp/wp-dev-lib/commit/ef300266e5d7073de3dcf25a37029b6501be1caf


I assume we check whether this directory exists somewhere in the Zip method.


Actually it's probably simpler if I let you fix it the way you want!


@obsc Hey Rena, if this line is what you are talking about then I think this is just the way text is wrapped in json haha


clicar -> clico


tirar ponto final


This name makes me feel like there's a `FalseFileInfo` class out there somewhere.


qual disciplina?


rever comentários sobre perfis


Typo


This throws an error:

```
File "whoip.py", line 22
    if unicodedata.category(char) != 'Mn'):
                                         ^
SyntaxError: invalid syntax
```

The somewhat unintuitive ordering is needed in my original syntax due to the way it loops through characters within the join().


Why was the F(str) macro deleted in this (unrelated) commit?


Would the macro have to be a ternary?


@MennaDarwish Delete all white space trails


They are no longer compiler errors, but Python exceptions (`TypeError`).


@taphu so why didn't we specify `monad-control <= 0.3.3` instead of `monad-control == 0.3.3.0`?
cc @glasserc @ababkin


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

It is ok, basically we require that annotators are uimaFIT compatible. So let's add  a@ConfigurationParameter RegExAnnotator


So the similar style as in the reversed file? Should I change all of the docstrings to be that way? I was just keeping it in tune with what was already there.


its defined in WString.cpp
define twist is a bad idea, but seams like i miss to commit it alone.


Haha, yeah noticed that after I posted and was like "what happened???"


Looks like this needs to be updated also actually.


LGTM


@MennaDarwish Use correct chaining of methods e.g "router.route('/signup')
                                                                                .get(func(){...});
                                                                                .post(func(){...});"


We use a space between function names and the open paren of the argument list.  In declarations, too.


also : contrib/debian/control


I'm sorry I offended you, I meant no such thing!

I cited you and added you to the contributors list for fixing the broken test, which is the only code I took from your PR, which you closed before I had a chance to comment on it.

I sincerely wish you the best of luck with your fork.


The name of the file is `tryfinally.pyx`...every time I opened this I had to type that and still didn't realize that this test makes no sense here *facepalm*. I will move this test.


@MennaDarwish No line should be more than 80 characters long.


@nezihyigitbasi should this be credentialsProvider = new STSAssumeRoleSessionCredentialsProvider ?


@MennaDarwish Use correct indentation.


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


Ah, thanks.


output added(unintegrated)


@MennaDarwish Delete all white space trails.


Also this is going to be removed because of https://github.com/SumAll/haskell-eureka-client/commit/d8284d26923e3f984e32bd4c5fafdaa41d74ec19


We should probably have the whole JSON response body here:
```json
{"token": "<authToken>"}
```


Shouldn't these both be compiled as static @wdl1908 to ensure that everything is linked properly?


the heading is different..


same here


They are testing how it handles a nonsensical input (in this case a set). I removed the list and append to save some time since they aren't really used because an exception should be thrown.


except that, lgtm


BTW it's [currently commented out in Wstring.h](https://github.com/esp8266/Arduino/blob/esp8266/hardware/esp8266com/esp8266/cores/esp8266/WString.h#L36) :smiley:


I we don't measure runtime yet, it shouldn't be in the enum.


Bæti við undirsíðu fyrir Dagskrá af viðburðum


Fixed 9b98a0b. Thanks


Let's delete this attribute after the postLink.


I'm confused now.  Why does `Seed` have a get/set and `State` only have a get?

Also it isn't very obvious that `State` can be used to continue a random by feeding it in as the `Seed`.  How do we make that clear in the API?


Never do side-effects that are part of regular program execution in an assert.  The assertions might be disabled and then those side-effects might not happen.


@MohamedIhab  they are highlighted in red, they are already deleted


Same thing here.


Newline between return type and function name.


Just out of curiosity, how does a user defined type work? The only thing I can think of is a struct or object. I try to filter out those in the beginning of this function call.


`sudo` is redundant when installing locally


Líbit ani ne. Rubysti jsou fajn, když jsou. Ale oni nejsou.


@MohamedIhab  can you check now?


@MohamedMarwan Delete all white space trails


May I ask the reason you're asking for a student number, a first name and a last name?


you kidding? I went straight to the dev branch!
(high fault tolerance)


just that message.. lol.
i think it should be `Unrecognized command: /hello`


Why are these bit masks instead of just incremental numbers?  Especially since they index an array this doesn't seem to make sense.


Logically, if the state is changed, than the seed should be as well, The seed basically represents the starting value. So, the user can set the seed, also setting the current state for calculations, run a few calculations, and take the current state. That can be fed in as a new seed, which sets the Random object that is inputted into to the same state as the original. Basically the seed is the master of the two, and if the state is set the seed should be as well.


But why these 2 lines are in all 4 if-s?


groupId usually a root package like:
com.nikita.xo
or something like this


cuz.. `command` isn't the incoming command to have `/` in front.


Yes that's ok, fixed in f406f66


useless comment - better to remove it


Also old silly staff for change size of free space was removed.


Damn. This is the line I couldn't find.


коммент из гита


`set(current_classes.split())` will do exactly same and in a much faster way.


the provider is a step towards customizing the whole thing - the docs are behind as usual, but ill add them as soon as my PR into squire-rte is accepted and the features "officially" work


collapseForm ?


expandForm ?


Why did I delete these spaces?


Yeah, this needs to be atomic, as you said.


public static and non-final field, something is very wrong

+ where is \n after the field


cc: @trilogysci

I'm sorry to bother you here.

To be honest, I am not a big fan of this change. IMHO, it's not a good deal to add a new field to `Scienfitic`, carry it everywhere and introduce extra cost in computation to just support a better `Show` instance. This hurts both performance and maintenance. Most users doesn't care that much about the feature introduced by `DisplayMode` here, or they might want a different behavior. Moreover, `Data.Aeson` depends on `Data.Scientific`, and it's a dependency of many other hackages and projects.

IMHO, the idea of `DisplayMode` should be implemented as a separate `pretty` function, and users can decide which `DiaplyMode` they want. `show` is mainly used in output and exception, which is not that often and cares less about cpu or memory, so it should be ok to decide `DisplayInteger` or `DiaplayFixed` by `base10Exponent` there, with or without normalization.

Sincerely.


Tpyo: woul -> would


What if a thread never compiles anything?  It would never do a state change and therefore its time would never get added to the global counters, right?


why this is public?

could we make it final?

and if this field is final then the name should be upper case: STOP_INPUT


`real` -> `realm`


Hi, @Kowalevskaja !
You seem to have a better understanding how ZotFile works.
Could you please help me with that issue if you have bit time?
#166


I think that comment is directed towards @g2graman - I just did a bit of proofreading


There's no reason to create a defer here.. You can use `Promise.reject()` directly. See `lib/utils.js`:`resolveError`.


This sentence doesn't seem to carry any significant information :)


Use `lastOrUndefined`


Is there a section we can directly link?


this method not only reads something, it is also print something to the Console, maybe better to say something like:

interact(...)

also input argument could be final: final String message


@usfhussam No line should be more than 80 characters long.


OK. This might as well stay explicit.


perhaps better print message somewhere else and not here? it could be awesome if input and output operations are separated


@HAAKS  there are still some extra modifications needed. Please read all the comments above


this could be final

final BufferedReader br ...


I think we can also remove PlaceSpawnPos and SPREAD_POS, since it's not used.


BTW another place to "log": http://ksp-kos.github.io/KOS_DOC/commands/terminalgui.html#global:HUDTEXT
P.S. May be not helpful in this particular situation but sometimes it might be useful.


There are no v_hands, and make sure you do not have any c_hands in any of your addons that would override the default ones.


Factor our into a local variable - `isAppropriateSignature`:

```TypeScript
let isAppropriateSignature = selectConstructors ?
    d.kind === SyntaxKind.Constructor :
    (d.kind === SyntaxKind.FunctionDeclaration || d.kind === SyntaxKind.MethodDeclaration || d.kind === SyntaxKind.MethodSignature);
```

Also maybe factor out

```TypeScript
(d.kind === SyntaxKind.FunctionDeclaration || d.kind === SyntaxKind.MethodDeclaration || d.kind === SyntaxKind.MethodSignature)
```

into a clean helper.


LGTM +1


I've been doing some thinking on this, and maybe it's not necessary to have the design of Iterable<T> on Objects..? The thing is we're trying to use primitive types as much as we can due to boxing/unboxing overhead. Ideally we can have the cartesian product implemented on top of Columns. We would need to extend getJavaExpressionWithAppend() to append a list of primitive values to a Column though.
(we can discuss it 30 mins later :))


@mennaelkashef Delete all white space trails


would you rather build the request URL with the `net/url` package?

```golang
tokenURL, _ := url.Parse(realm)
params := url.Values{}
params.Set("service", service)
tokenURL.RawQuery = params.Encode()
req, err := http.NewRequest("GET", tokenURL.String(), nil)
...
```
This way you can more safely encode the query parameters.


there is method: isEmpty(), but even if you decided to use equals it should be use like this:

!"".equals(consoleInput)

in this case you are avoiding NullPointerExceptio. In your case if consoleInput is null, then you will have NPE


Ah, right, I forgot about it. I kept `tail -f log2.ks` the whole time. But sure, hud is likely better.


why we using state field to pass values in this method?


YOu didnt have to do that. I was going to look at it later. These changes wont affect the ISO until we update the package btw :wink:


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


I'm still against menus that can't fit in their windows.


To me, it seems partial_observation_test.cpp should have been written in a very simple shell script (~2 lines).


It's not quite, though. It's clearing individual bits which are not necessarily on byte boundaries. I'm not sure the added complexity of handling byte boundary alignment is worth it.

(In any case, this expands to the same code that was here before this patch set. It just reads nicer.)


I'm not sure I agree with this


users of this class going to call "readFromConsole" method, and the name of the method ("readFromConsole") do not say anything (even close) about fact that it could KILL the hole APP! This line basically blows entire application but you do not say about it anywhere =)


This `ReadAll` error is being ignored. We should also check the response code before parsing the response body.


else on the next line


what the hell is this =)?


Sure, let's discuss.

On Fri, May 1, 2015 at 2:08 PM, jingjingwang <notifications@github.com>
wrote:

> I've been doing some thinking on this, and maybe it's not necessary to
> have the design of Iterable on Objects..? The thing is we're trying to use
> primitive types as much as we can due to boxing/unboxing overhead. Ideally
> we can have the cartesian product implemented on top of Columns. We would
> need to extend getJavaExpressionWithAppend() to append a list of primitive
> values to a Column though.
> (we can discuss it 30 mins later :))
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uwescience/myria/commit/6cdbd9018ff5457872c7b3af1c1b0d16a5ef1578#commitcomment-11007873>
> .
>


useless comment


This, for example, goes to my concern of using Java objects.


Done.


Make this `pushType := proto.PUSH_TIMESTAMP` to be explicit about the default value.


Dead comment?


not final field,
hardcoded filed size (3)


Can you create a folder in the js folder with the name 'map'? I think it's more overzichtelijk. @adrielstar


why removing log?


if we have getFigureFromPosition and setFigureOnField then why we need this method as well?


But this fork is being updated based on 1.1


input arguments could be (and should be) finals (final int x, final int y)


DITTO


@mennaelkashef Use correct chaining of methods e.g "router.route('/signup')
.get(func(){...});
.post(func(){...});"


Since it's used in several places, I think placing the default arch in a constant is a good idea.


why we need this empty else?


As I said, it's a low-priority issue. But if scrolling as such is a bad idea, what would you say about adding another menu entry:
```
[ ] Next page
```
which would load another set of options? Of course, this would be another widget - it's out of scope of this one.
Eventually, something like this should be done, because without it it is impossible to, for example, select a target if there is too many of them.


I can't comment on the Windows-specific stuff since I don't have the domain knowledge, but other things LGTM.


But most likely also to work on 1.0.


Is this only one method that overrides the method from interface? Or you just forget to annotate other methods?


useless comment


I think the operator will need to apply the cartesian product no matter
what (it can't be done in the evaluators), but maybe we could adapt the
StatefulApply design to pass the same (initially empty) fake relation to
each of the evaluators. Each would append results to its own column, and
then the operator would form the cartesian product of those columns and do
the final append to the output TupleBatch. What do you think?

On Fri, May 1, 2015 at 2:11 PM, Tobin Baker <tdbaker@cs.washington.edu>
wrote:

> Sure, let's discuss.
>
> On Fri, May 1, 2015 at 2:08 PM, jingjingwang <notifications@github.com>
> wrote:
>
>> I've been doing some thinking on this, and maybe it's not necessary to
>> have the design of Iterable on Objects..? The thing is we're trying to use
>> primitive types as much as we can due to boxing/unboxing overhead. Ideally
>> we can have the cartesian product implemented on top of Columns. We would
>> need to extend getJavaExpressionWithAppend() to append a list of primitive
>> values to a Column though.
>> (we can discuss it 30 mins later :))
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/uwescience/myria/commit/6cdbd9018ff5457872c7b3af1c1b0d16a5ef1578#commitcomment-11007873>
>> .
>>
>
>


See use case in lib_lazcalc


It was never meant to be committed. It was just a small bit of debug code while I was developing some feature and it kept breaking.


My hands are quite full at the moment, but if later you need some testing to be done on this instance, feel to ask, I'll gladly to it, like for CoS.

Nice job :+1:


Might as well also print out the definition list at that point since this is the first thing I'd be wondering about.


what about getFigureFromPosition method?


Do we really want to do that? We're talking about

    mono_bitset_foreach_fast (set, idx, {
        do_stuff (idx);
    });

versus

    MONO_BITSET_FOREACH (set, idx) {
        do_stuff (idx);
    } END_MONO_BITSET_FOREACH

I'm not at all convinced that the latter is better or that it's a style we should keep using. It quite simply looks ugly. There's also less of a chance that the user screws up usage of the macro when you pass the statement/block as an argument.


missing newLine


\n


@MohamedIhab check again?


Mi stavo dimenticando: ho sistemato anche i messaggi di alert e i colori


Understood, and I think I appreciate the column-oriented design now better than I did when I wrote this code :)


On 2015-05-01 4:59 PM, Kyle Warwick-Mathieu wrote:
>
> May I ask the reason you're asking for a student number, a first name
> and a last name?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zonzorp/COMP1071/commit/eceb6e1b6fb8811e594fd58d6ffee1f1ac609978#commitcomment-11007746>.
>
We will be naming the hosts piNNNNNNNN where NNNNNNNN is the student
number. Since we are using the pi, I will be having everyone join a
private VPN so that we do not have addressing conflicts and they can run
them anywhere they have internet without preventing them from having
access to the lab network. To make it work better for evaluation, I want
every host having a predictable, unique hostname.

I did have a look at the feedback on that shellcheck website. I didn't
know about the site, thanks for bringing it to my attention. Like most
automated checkers, it did correctly identify a variable not used in the
current version of the script, an inline execution syntax I personally
prefer to use, and a potential issue to do with literal versus regular
expression use. The variable is there because I may or may not go back
to using it and can't be bothered to remove it. Along the way, I will
either use it or remove it. The execution syntax observation is
informational only, both forms are valid. In the case of the literal
versus regular expression, I meant to use a literal expression there,
the checker just doesn't have the smarts to divine my intent. Having
said that, I may add it to the scripting class as a resource people can
use to help them become aware of syntax choices. We'll see. I don't like
spell checkers because they have no idea what you mean, only which words
show up in a list. This checker seems more useful, but until I run a
number of scripts through it, I will remain a bit wary of it.

Thanks,
dennis


is this really need to be package local?

why it's not final?


>`^^`
What does it mean?


Good catch for the error.

I'll also parse Token only when 200 and Details only when 401, not parse otherwise


Haven't tested it on a Mac, but I suppose it should work. Setup should probably look like this:

1. Install Wine and libav using a package manager such as [Homebrew](http://brew.sh)
1. Install mingw32 cross-compiler (possibly from [here](http://crossgcc.rts-software.org/doku.php?id=compiling_for_win32))
1. Install the Windows version of [AudibleManager](https://www.audible.com/sw)
1. Run AudibleManager and set it up with your Audible account
1. Follow the Setup steps in the README


cant this just be if(done) since canceling sets done?


Sure. I'm not sure where it should be placed, though, given that it's used all over the place.


I hope I didn't break anything!


Does `getDefintionFromSymbol` ever fail and return `undefined`?


`side_str` -> `middle_str`?


partial work on #191


What this class is representing?

From the name of the class and Constructor I can't tell the main idea of it. Move in XO could be represented by 2 value:
- figure that takes pat in the move
- coordinates where you going to put the figure (x, y)

Could we think about better naming?


When does a type not have a symbol?


fixes #191


Split this condition up instead of adding another clause: add a block like `else if args.PushType == proto.CONFIRM_NOT_PENDING { pusherWins = false}`

Is CONFIRM_NOT_PENDING equivalent to PUSH_TRANSACTION with the lowest possible priority?


method move in class Move, looks odd and do not have any logic behind. If the class represents the move and the method triggers it, then it could be something like: execute/apply/etc


yup


where is "tab"?


\o/

:dancer: :package:


:heart: @nthykier


What about accessors?


pushing fixes into garbage lead to nothing. those files, here, are useless. delete with no mercy, no sane OS will use them


@a-kamel Use correct indexing


this is SKEL. should i explain again what skel means?


@a-kamel Use correct indentation


this doesn't look right


Just an FYI, you'll have players still be able to access these safes on occasion.  I'm using the same inventoryLock variable to try and lock people out of "secured" bases/crates, and occasionally they can still see the inventory of the object.  Furthermore, the "Take" action sometimes still comes up, and completely bypasses the inventoryOpen EH.


@a-kamel Delete all white space trails


Can it just be
```
function draw_empty_window{
  parameter rect.
  draw_empty_background(rect). // not sure about using `rect` without shifting it
  draw_empty_background(rect).
}
```


starting from this line it's becomes more clear what you are doing in the ReadCoordinate class. And it looks like you storing the state in the states variables. There should not be status fields for storing temp values like X or Y. Also you should come up with generic solution that can be used for Setting/Reading both X and/or Y


Yeah, Thanks a lot! Really loved the suggestion. I feel a lot better handing this off to someone to use :)


same here


Although this fix solves the overflow problem, it doesn't seem to be the perfect fix to me, since the fault is not on H_SIZE :) What if, for example, ngas * G_SIZE gets larger than INT_MAX in another simulation..?

I think depends on your expectation of the data, you may need to declare ngas/ndark/nstar as long.


sequence diagrams reviewed


this line makes clear understanding of ReadCoordinate usage, and the ReadCoordinate should be re-thinked and re-implemented


and here


Isn't it a code duplication? Can it be abstracted?


I can change it back.


useless comment


@3adel94 Delete all white space trails


well, you get the idea ;)


Nice, didn't see that before.


@3adel94 Use correct indentation


How about “Same as `a + 1`”? There’s no mutation here.


Do they teach this in AP Comp Sci? If not, it is an additional requirement for everyone instead of just non-APCS students.


UML reviewed


Modified `string` to consider empty space as most minimal. If you want the control characters, do it yourself.


@feiranchen I actually mean this line is wrapped by an extra set of quotes. The problem with this is that this is a raw request. For example, if the user wants to render two bullet points, they would send `* Hello\n* World\n`
and receive a response of
```
<ul>
    <li>hello</li>
    <li>world</li>
</ul>
```

However, if they render with the json encoder, they are incapable of sending `* Hello\n* World\n` and can only send `"* Hello\n* World\n"`. This results in the response of broken html
```
<p>"* hello</p>
<ul>
    <li>world"</li>
</ul>
```

There needs to be a way to raw encode strings. However, with the way sawyer is setup right now, we cannot do anything without adding a new encoder/decoder pair or manually writing a bitarray into the body. :(


Hmm, did you look at maybe just getting rid of that? The old format is 4.5 years old at this point and we can probably put something in to block it politely.


This is what _actually_ fixes the issue. The access was wrong, plus it was not a class method, so...the docs weren't correct.

ad0c35f974b51de0dd4b1e8e2c91b02d12830b1d is the spec that was missing that would have caught this :grimacing:


Thanks for clearing that up Dennis. I am defiantly looking forward to the scripting course. If you wish you can view my progress on that "server install" script I've been writing for the past few months. It's defiantly come a long way since you've looked at it last in class https://github.com/kyle95wm/dwc_network_installer


because you using static variable you have to create variable for each coordinate (like X, Y), and what if our field will be in 3D, will you add Z variable?

This class should be changed to something more generic that could be used with any variable. It's better to have generic class for each variable, something like:

final CoordintaeGetter xCoordintaeGetter = CoordintaeGetter("X");
final CoordintaeGetter yCoordintaeGetter = CoordintaeGetter("Y");
xCoordintaeGetter.get();
// ...

this example also far away from perfect, however it will be a good start point


this could (and should) be final


why we need this extra \n?


VERY USEFUL!


capital first letter 0_0 ?!?!?! it's not a C#


please resize the boxes and make the back arrow dashed @markmorcos


I think we should keep the function in there in any case, just to reduce the number of `#ifdef`s we have, especially outside of SGen.


useless comment


variable that is private and you never READS and NEVER writes to it


You're right. I had some false memories about `split()` choking on consecutive spaces or tabs but actually it doesn't. Want to submit a pull request?


> In case idx == 0 for the first and only output,
> you try reallocated zero bytes that must fail,
> resulting to an immediate shutdown of ympd without any log-message :/

I'll take a closer look.


Only the last two functions: `draw_window_outline` and `draw_focused_window_outline` can really be abstracted. The rest, while they look the same, and do pretty much the same, have one thing different: inside the printing loop, they print just one thing (not two). This might not seem like a big deal, but this drawing code already takes a relatively long time to do its work - and I'm not going to slow it down further.


why it's not final?


> This is changing the behavior of the mono_threads lock, which is scary/

Changing it? Scary in what way? I don't quite follow. All that's changed is that we also take the context lock.

> Why do we need it both locks here?

Because we can't tell if the static data we're going to free is thread or context static. See `do_free_special_slot` which is called on every element in the `special_static_fields` table. So we need to acquire both locks to be safe.

Two alternatives:

1. Taking the relevant lock in `do_free_special_slot`. This potentially turns into a lot of locking/unlocking for large apps.
2. Split the `special_static_fields` table into two tables. This will slow some users of it down because they'll have to look up fields in both. We'll also be exposing the format of special static offsets to the rest of the runtime then (which may or may not be a bad thing).

This just seemed like the easier and nicer way to do it.

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

Added it to the lock tracer in 98bad1ebb821ac2ec7342ba70e15ccf47cec5bde.

That page doesn't seem to document the threads lock at all.


each time when you going to instantiate the object this constructor will override static field? This is very wrong


SGen can't include `gc-internal.h` because it'll screw up SGen independence.  There has to be a client function `sgen_client_get_provenance()` that's implemented in `sgen-mono.c`.


why so many newlines?


> Use of unitialized stack-variable :(

Where exactly?


comment should be removed


nice usage of enum


getDefin***i***tionFromSymbol


final


Is there any thought into moving these "Condition='$(OsEnvironment)'=='Unix'" into an optionally included dir.unix.props file to ease readability and clearly separate OS specific properties?


DITTO


println не случайно называется print_ln_ :smile:


> `[ ] Next page`
Nice idea. I'll use it in my lib_list_dialog.
esp. if "next page" is the first option and "previous page" is the second.
P.S. I actually had a similar idea but it was about "cancel" option in the dialog, not about scrolling.


fixes #4


I renamed this to CLEANUP_TXN


if it does not have a declaration for instance intrinsic type, union type, tuple types, types from signatures,  anonymous type..


do not think that we need to store the player somewhere, we can alway determine the player fro the field state, aren't we?


On further consideration I think it's better (and less code) to not replicate the binary protocol entry types and instead just always add the provenance.  When provenance logging is disabled the client function would just return `NULL`.


use printf for String formatting, or String.format(...)


You guys need to agree on whether it is possible to install x64 **and** x32 on the same system.  You seem to be giving me conflicting answers.  I'm pretty sure that I had to uninstall Project x64 2013 when I installed Office x32 2013.


remove the comment


useless comment


Never mind, I have overseen the static declaration of *soutputs*.


whuups, ska vara password, ikke email 2 ggr.


final, \n


argument's could be final


I believe there are multiple simple ways to do a test. I feel it's just nicer and cleaner to organize everything into a single test framework. Otherwise, we can choose to do all tests with simple scripts/c++ programs instead of using a testing framework, which I feel less organized.  Also, this test alone may seem simple enough, but there are/will be other similar tests that could involve processing the inference results, which are cleaner in C++.


Missing period.


@Dogild not sure what you intend with this change, but why shouldn't a `CPTextField` be able to resign in a non-key window? And what do you mean about blurring? Can't it blur as much as it likes given that only a text field in the key window can be non-blurred (aka focused)?


Gah! I had an autocommand, but didn't trigger for .inl file...


> I'll use it in my lib_list_dialog.

Go ahead.

> esp. if "next page" is the first option and "previous page" is the second.

Why not the other way around?

> I actually had a similar idea but it was about "cancel" option in the dialog, not about scrolling.

Yeah, I already have cancels/quits in main menu and am going to add it to the future process manager.


done.


I like the idea of this class, nice!


done


I think that you are wasting your time.  You can only register (regasm) types against the libraries for x32 or x64 bit not both.  So having the addin keys present pointing to types that aren't registered would be pointless.


No else after return.


The message says to "check the new order amounts and try again" but nothing has been saved, so if they try again it will just keep failing unless they manage to trigger `order.update!` somehow.  Also, if it has changed amounts we need to trigger tax calculations & setup pending remote tax docs & etc. For us this happens when the order transitions to confirm.

How about we move this order back to a pre-confirm state and run `order.update!`?  And perhaps even try to transition it back to `confirm` automatically?

I'm thinking we should just


looks nice, however it's questionable that we should put here any UI interaction


What does it mean even? a file that should be included in exactly one .c file?


final


Yeah there are, in the HL2 .vpk. That's what you converted it from, right?


> Why not the other way around?

Because most of the time the user will scroll forward, not backward.


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


You need a BN_CTX_start here.


TeamCity DnnPackager :: Continuos [Build 0.1.0+22](http://dazinator.ddns.net:81/viewLog.html?buildId=791&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:29


this method could break the hall game, so it should NOT be public. Despite we always can calculate answer to the question: "who is current player" from the board state, so this method should not be here


remove the comment


It worked! :)


But that is only valid for the `filter` function **and only for the first argument**


I think they all have Excel as an option but its up to the user to decide whether to install Excel (most people would and its the default).  So yes, we would need to check each Office application until we find one that is installed.  I haven't yet added the necessary code to do that but it should be straight-forward since all the office keys are in the same parent key.

If the other issues are resolved, I'll add the extra checks.  What do you want to happen if the user doesn't have **any** of the apps installed?  Install 32 bit?  Don't install?


My goal (probably not quite yet achieved in this PR) is for the node not to contact the cloud provider AT ALL. It should be allowed to hit a local metadata server, but it shouldn't need any special privileges (e.g. compute-read) to join the cluster.


> Because most of the time the user will scroll forward, not backward.

ok, that's valid point.


> Curious whether these other approaches inspire you other name solutions though.

I know you prefer to stick to your favorite pattern: If we have `THREE.Line`, then the extension should be `THREE.xxxxxxLine`.

That is why I suggested `SegmentedLine` -- in spite of the fact that I do not particularly like it.

I really think that if you want to use `THREE.Line` to mean a polyline -- and I think that is OK -- then, in spite of your pattern preference, the other one should be `THREE.LineSegments` (or `THREE.Segments`).

I prefer `THREE.LineSegments` for clarity.


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


No, I removed the mesh from models/weapons/c_arms_citizen.mdl, the animations are from the view model arms MaxOfS2D made.


According to bitcoin this should be built w/ db 4.8 (a compiled version of this is provided by the bitcoin launchpad ppa), I wonder if it wouldn't be better to switch to that wallet portability.


TeamCity DnnPackager :: Continuos [Build 0.1.0+22](http://dazinator.ddns.net:81/viewLog.html?buildId=793&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:19


@smarterclayton already asked for such a flag and I'm planning to add it.


Would be nice to have a parameter option to either sort by the run order or not before outputting them.  Also would be nice to support all the parameters that affect the categories in the run command, to be able to just swap the command out and see the exact run order.

One thing i keep intending to work on is having a functionality to be able to output a script file, that could then either be modified externally then passed back into concord and modify the exact behavior.  Which this could be a subset of that functionality very easily.


Link to changelog for 3.4.2: https://www.kernel.org/pub/linux/kernel/v3.0/ChangeLog-3.4.2


I thought about it. I didn't because we'd probably want to do the same for Windows and it gets awkward as some of the related defines are shared. (E.g. we probably don't want to separate related defines or duplicate them) I didn't break out for Roslyn here because I want the config to be set before I import the Roslyn targets (don't want two extra files) and these will hopefully go away soon (once the bugs are fixed).


OK, no problem.  For my reference: http://kynosarges.org/DotNetVersion.html


conclusion:
you have made dramatic improvements in your programming skills and you are able to solve tasks on your own. Looks like you can deliver any project already, however stye and architecture decisions require attentions and additional practice. I would recommend to do the refactoring of this code before moving on with next steps.


Also, when we add or change binary protocol entries we also need to add/change the client binary protocol callback functions.

Yeah, that kinda semi-defeats the purpose of having this def file - we're back to changing in more than one place.  If you have a good idea for how to fix that, bring it on!


@rossknudsen  @ckuhn203

> **Running 32-bit and 64-bit versions of Office on the same computer isn’t supported**

> *Office 2013 doesn’t support running side-by-side installations of 64-bit and 32-bit versions of Office. For example, you can’t install Office 2010 32-bit side-by-side with Office 2013 64-bit. This applies to both Windows Installer (MSI) and Click-to-Run installations of Office 2013. If you try to do this, you'll receive an error message and be prevented from continuing.*

https://technet.microsoft.com/en-us/library/ee681792.aspx


@ndeloof I thought you were given the repo history. Sorry!


Aight!


I agree with @rossknudsen here, we should only create the registry keys that are required.


Done.

You mean PUSH_TIMESTAMP? If you squint perhaps, but since you really do have to squint, it's better to break out the use case explicitly. We certainly do not want to be pushing timestamps.


Maybe "the following sets of terms:" is enough. Or otherwise, "your choice of the the Apache 2.0 license and/or the BSD license below:"


Done.


Thanks for the info. How about when adding an attachment? I see a <b>Reassignment:</b> field that allows me to take that bug when uploading -- do you have that option?


Hey @evejweinberg
maybe the changes i introduced with pull request [#139](https://github.com/openframeworks/ofBook/pull/139) answer your questions.


Traditionally, if it's to be read from STDIN, the file name is a dash, -.  If I'm reading this correctly, you want to read from standard input when no banner file is specified?  I'd suggest supporting the dash character instead.


`--vtable` takes an argument, namely a vtable pointer.  i.e.

    sgen-grep-binprot 0x123 </tmp/binprot

searches for an object at `0x123`, and

    sgen-grep-binprot 0x123 --vtable 0x456

searches for an object at `0x123` and a vtable `0x456`.


mmmh hmm


Yes, this is pretty easy it seems.
    Name: {app}\*.dll; Type: filesandordirs


Line is too long.
Usually I see it as "static const" instead of "const static."


@tunnckoCore pull requests welcome :)


Line is too long.


You can't possibly install Office without selecting at least one application.... ...right?


This should be updated to the style that the rest of the nistz256 code was updated to, to no use bn_wexpand+...+bn_correct_top, but instead use bn_set_words.


Remove the following part:
```
Lambdas, Closures, Blocks and Methods are also fancy names to say: *functions* (That's an oversimplification but will be enough for this post).
```
Because the post no longer states the object oriented analogy, nor uses Ruby. The post now only uses JavaScript functions.


:ghost:


Boris may not have editbugs.  But he should!

On Fri, May 1, 2015 at 2:44 PM, Brian Nicholson <notifications@github.com>
wrote:

> Thanks for the info. How about when adding an attachment? I see a
> *Reassignment:* field that allows me to take that bug when uploading --
> do you have that option?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mozilla/firefox-ios/commit/81f1cf76197c8253e03f3d423fad75c2533b2230#commitcomment-11008409>
> .
>


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


TeamCity DnnPackager :: Continuos [Build 1.0.5+0](http://dazinator.ddns.net:81/viewLog.html?buildId=795&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:20


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


TeamCity DnnPackager :: Continuos [Build 1.0.5+0](http://dazinator.ddns.net:81/viewLog.html?buildId=797&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:17


Nick,

I've gone and added more to the contributor's guideline.  In short you'll
need to make the changes on the same branch you've been working on and then
re-push the changes.

https://github.com/QuantConnect/Lean/blob/master/CONTRIBUTING.md

-Michael

On Fri, May 1, 2015 at 4:35 PM, Nicholas Stein <notifications@github.com>
wrote:

> Thanks for the review. I can would be happy to add the banner, change the
> default Algo, and the other changes you suggest to the project.
>
> How do I update the PullRequest after I have made the changes? That was
> missing from your writeup.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bizcad/Lean/commit/26abca6ec62b552ccceb86016c0e957822432cfd#commitcomment-11007389>
> .
>


I'll add the state revert from current RenderState fb, reverting that when binding/unbinding later


I think we should go back to committing `plugin.jar` directly with the source code, as this hard-coding of Program Files/Java/jdk1.x.x_xx changes all too often.


There needs to be a comment that bn_set_words (or bn_wexpand and bn_correct_top) is not constant time. Maybe it is good to discuss this by email. Unlike the other nistz code that works like this, this code actually is operating on secret data.


Was this what you had in mind for the `measurements` dictionary?

```python
{'2015-04-26': 123.4,
 '2015-04-27': 123.2,
 '2015-04-24': 121.4,
 '2015-04-25': 120.8,
 '2015-04-22': 122.4,
 '2015-04-30': 123.0,
 '2015-05-01': 123.0,
 '2015-04-28': 123.4,
 '2015-04-29': 122.8,
 '2015-04-23': 121.4}
```


check out the following link for the tutorial. I just tested it and it works.

http://lafro.github.io/lafro/


This was a bug.


This seems a keywords cloud just to show off. It can be intimidating. Maybe simplify the description by keeping only the generic/specific keywords.


The other code in the file uses a test like this:

# if !(defined(__x86_64) || defined(__x86_64__) || \
       defined(_M_AMD64) || defined(_MX64)) || \

I suggest finding a way to make this consistent with the existing code, and also I suggest trying to find a simpler test. e.g. Use the existing complicated test to #define or not #define a single macro that can then be tested in both places.


I can agree with that @hhware.  I've updated the PR to include a restriction on top level `id` within the attributes object.


Oh, I was talking about v_hands.mdl. Could you change the name to c_arms to prevent confusion?


@keedi - Great. If you can get it to work on windows that would make me quite happy, since I tried to do it a while ago and couldn't get it working.


Indeed, I republished my package as I mentioned in the prev email -
official version doesn't work properly from packages.

I submitted a pull request (
https://github.com/epeli/underscore.string/pull/420), so if they merge,
I'll unpublish my package

On Fri, 1 May 2015 at 09:08 Luca Mussi <notifications@github.com> wrote:

> it seems still visible in the atmosphere search: see here
> <https://atmospherejs.com/?q=underscore%20string>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/taskputty/meteor-underscore-string/commit/ad14b3f53dac56f66f53b6d41298b75f5aa049ed#commitcomment-10998499>
> .
>


I thought about adding them, but tbh they are quite long, and since they weren't translated to the other languages yet, I figured it wouldn't be a big loss if I didn't add them in this PR. Would you mind if I submit a separate PR when I get the time to translate them properly?


This comment should be moved along with the other code.


oh yea, duh!


#EXTINF:-1,ABC NEWS LIVE STREAM (USA)
http://abclive.abcnews.com/i/abc_live1@136327/master.m3u8


actually - slotNumber is only valid for slotDef when the Slot is in the machine inventory and not the player inventory


Уффф... Поправил баг. Однако я озадачен количеством логических багов в коде((((


do you think this is a bug ... or its something in my app???


aren't you missing an "otherwise, add the entry to the set?


Change `concrete` for `specific` to reduce the amount keywords used.


Thanks for pointing out the commas, that detail bites me often enough that I should have learned to look out for it already! I'll add a commit to fix these.


Change `concrete` for `specific` to simplify keywords.


whoops, just saw the committer and failed to notice the repository, sorry


This is here on purpose to test parsing when there are additional statements in the method.


should getEntries() and getEntries({}) return the same set?


> Thanks for the info. How about when adding an attachment? I see a Reassignment: field that allows me to take that bug when uploading -- do you have that option?

Nope:
![screen shot 2015-05-02 at 12 00 11 am](https://cloud.githubusercontent.com/assets/41975/7438041/47812244-f05e-11e4-87f8-5115eecd6a61.png)


donc on doit regarder le referrer et non le retour json ?


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

`string.Format` would make this much easier for you


As written, this would regress performance of ECDSA verification, at least when the custom inversion isn't being used.

I think that instead, EC_GROUP_do_inverse_ord should take a flag parameter indicating whether or not the inversion needs to be constant time, and then choose which inversion algorithm to use based on that flag.

BTW, is the new custom inversion faster than the non-constant-time BN_mod_inverse?


This has been included in the primary trunk


I thought about adding them, but tbh they are quite long, and since they weren't translated to the other languages yet, I figured it wouldn't be a big loss if I didn't add them in this PR. Would you mind if I submit a separate PR when I get the time to translate them properly?


Is `otherPeerNicknames` just so the admin can tombstone by nickname?  If so, rather than having the ipam code track nicknames->peernames indirectly, wouldn't it make more sense to enhance the router core with the ability to lookup peers by name (whether it is the string representation of some peer's PeerName or a nickname)?


Isn't BN_mod_exp_mont_consttime only going to work when group->field_encode is not NULL? Otherwise, why was the old code doing testing group->field_encode? (IIRC, the existing code assumes that if group->field_encode is non-NULL, then the encoding is montgomery and not something else.)


I'd suggest "fails validation" instead of "don't work".


this has been committed to the core trunk


don't think so, that's a field for humans to use to read more about the project, I believe.


rax retry tests


le cas avec referrer est pour les vues pur-backend, comme register. On espère que ça va disparaître.
Le cas qui t'intéresse est la ligne return HTTPBadRequest (regardes la version plus récente). Comme c'est une erreur, il n'y a pas de payload json... mais en fait je me demande si ça serait possible de renvoyer une payload json en cas d'erreur? À réfléchir.


What about 'I update this content setting "2" as the Field Value' ?

Overall, I'd use "set" rather than "use" when applicable (field values for instance).


Thanks, I'll remove this part from the README then. But I agree you should get [editbug privileges](https://bugzilla.mozilla.org/page.cgi?id=get_permissions.html) since you're such an active contributor!


@azamzamy  test is great :+1: , i couldn't run rails test on my pc for some error with the pc, however the android test is fine


I see the `:: Python :: 2.7` field...does it matter that this doesn't actually support 2.7? It's 3.3 or better, I think, as I use some of the new features.


This function should be marked __owur like the rest of the functions in the file were recently.


Code is fine.


Hmm the variable might not get pushed public correctly then?


Thanks for sending in this cleanup of our sample! This line does look like an unnecessary holdover from earlier debugging. I'll check this independently to make sure.


Will change that to 3.3 to be accurate, but, FYI, the parts used by
DetectorMorse work for me on simple tests.

On Fri, May 1, 2015 at 3:07 PM, Kyle Gorman <notifications@github.com>
wrote:

> I see the :: Python :: 2.7 field...does it matter that this doesn't
> actually support 2.7? It's 3.3 or better, I think, as I use some of the new
> features.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/hobson/nlup/commit/ed5214e38ca76386fb86c2da563703c80621bbb6#commitcomment-11008710>
> .
>


thanks. yea, you're right. I tested.

On Fri, May 1, 2015 at 3:05 PM, Kyle Gorman <notifications@github.com>
wrote:

> don't think so, that's a field for humans to use to read more about the
> project, I believe.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/hobson/nlup/commit/ac95eb9b62fdc61e123158d09cc0a619fcd123f3#commitcomment-11008692>
> .
>


Empty merge


For unit tests I have no objection at all using gtest.  However, for end-to-end tests like this, I think it's a bad practice to keep the input/output (which are the most important test artifacts in this case) in an obscure binary format, and leave only the boring command-line arguments visible in the test code.  Rather, the factor graph should've been clearly constructed in C++, then passed to the sampler to check the final result.  Or, we could write the factor graph in plain text, use the [format-converter in DeepDive](https://github.com/HazyResearch/deepdive/blob/master/src/main/c/binarize.cpp) to produce the binary files, and run the sampler to check its output.  Latter sounds much easier to do in shell script than in C++, right?


Gotcha. I used your script in every 'system setup' script that I've made whenever a customer requires Ubuntu- thanks for keeping up with it for so long!

I definitely appreciate the security concerns and not wanting to encourage people to copy/paste random scripts from the internet.


LGTM.


There is a trailing space here, and also in at least one other place in the new code.


combination **of** synchronous


great, do that and i'll merge. i can submit it to pypi sometime weekend
unless you want to.

On Fri, May 1, 2015 at 6:09 PM, Hobson Lane <notifications@github.com>
wrote:

> Will change that to 3.3 to be accurate, but, FYI, the parts used by
> DetectorMorse work for me on simple tests.
>
> On Fri, May 1, 2015 at 3:07 PM, Kyle Gorman <notifications@github.com>
> wrote:
>
> > I see the :: Python :: 2.7 field...does it matter that this doesn't
> > actually support 2.7? It's 3.3 or better, I think, as I use some of the
> new
> > features.
> >
> > --
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/hobson/nlup/commit/ed5214e38ca76386fb86c2da563703c80621bbb6#commitcomment-11008710
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hobson/nlup/commit/ed5214e38ca76386fb86c2da563703c80621bbb6#commitcomment-11008747>
> .
>


The other part of this is remembering all the peers which have disconnected, because that's the only time you should 'rmpeer' one. The router core presently discards all knowledge of a peer when it cannot be reached. We can debate whether this enhanced function should be in the core, or if there are other ideas on how to resolve such a situation.


Really?


Version 1.0.0 out now.


actually I was just considering to remove the lines
```
 } else {
gtk_widget_hide(popup_window);
}
```

Does anything speak against that?


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)


Hi, I was trying to do some tests from a VM and I needed to make this work to be able to do those tests... just this. Don't get mad at me... xD


the if-else statement needs to be surrounded with brackets {}

https://github.com/JackpotClavin/android_kernel_lge_msm8974/commit/87697a318abcca8ec95d61845d410185dc346d9d#diff-07244491ccb9f3397e5761ffe63b6b31R2340


This solved the Loading "connect_proxy.js" tasks...ERROR problem. Thank you @snide


Credits for this fix goes to cnotethegr8 (http://stackoverflow.com/users/278629/cnotethegr8), as it is posted here by him: http://stackoverflow.com/a/25862878/2189481


Ugg I hope you guys werent going to go through with the ones, the part that got merged last night...and I even tested it when it was on the 5.0.2 branch makes PLMN return null, so no carrier display shows.  It may interfere with the keyguard carrier text display code review that is currently on here. prior to this all you had to do was add ERI file overlays to the device tree to make carrier text display correctly.   This just messed it all up.  PLMN returns null or nothing valid


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


TeamCity DnnPackager :: Continuos [Build 1.0.5+0](http://dazinator.ddns.net:81/viewLog.html?buildId=823&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:20


@kennycason Indeed! I think with a more strongly typed API you'll have a more robust library and less bug reports.


Also, we should use Guava's overflow checks.


Great!


Thanks guys, I just applied.


not sure how this differs from the example above


disabled instead of diabled?


Ok.  But it looks like `otherPeerNicknames` is simply held locally, i.e. not communicated to other peers via gossip or otherwise.  So the admin can only tombstone a peer by finding another peer that hasn't been restarted and so remembers it.

Wouldn't it be more robust to simply to track the nicknames in/alongside the ring, so that any peer recorded in the ring can be tombstoned by nickname from any peer?


replace for-in with indexOf for checking values


So another option would be to make a script which dumps the env after a recognisable token

    echo MARKER && env

Then look for the marker and parse everything after it?


I am drawing surface polygons , there shouldn't be a problem on a 3d surface. But anyway, here is a 2d mercator projection :

![image](https://cloud.githubusercontent.com/assets/927575/7438279/4890e6fe-f06a-11e4-900b-310d12bf1a52.png)


Can you add `$tick_callable` to this?


Well, there's also the GDK_GRAB_BROKEN event that is linked to this callback, it's explicit in the XML file. It's not handled explicitly in the callback, it is handled in the 'else' you want to remove.

I have no idea about that, I don't even know what is the purpose of it. But I think that if you remove it from here, you should also remove it from the XML as well.


But..this looks really strange .. This really looks a problem of N.A.S.A's  WorldWind


Ah good catch


Допустимые значения:


This doesn't match the interface


Can we pass $connection into this


I'd say in the commit message that the problem is `docker-selinux` packaging (it is right?)  And can we also link to a bug/mailing list thread about this?

Regardless :+1: from me if we need to do this now...but we're going to need to figure out how to fix virt-sig docker packaging with selinux.


In your latest commit you already handle GDK_GRAB_BROKEN explicitly. So I think we can safely make the default case do nothing and remove the double/triple-click case.


Is there a build bot that can be used to download development builds to test with?  I'd be interested in testing out some of these changes if it's ok.


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


didn't notice these until now. yup, sysfs_remove_link() would be more appropriate.


TeamCity DnnPackager :: Continuos [Build 1.0.6+1](http://dazinator.ddns.net:81/viewLog.html?buildId=845&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:24


I added a handleTick() function. Or should I just pass null to handleMessage?


Agree with that


Huh, that might be needed to do what we want.


we do not support ECM please remove


Right, look at the interface declaration that you added


>Left damage if banking right +10 deg, right damage if banking left -10 deg, both wings if > -10 and < 10.

I don't understand. Damage for +/- 10 deg banking?

Will you add your structure contacts to the c172p-fdm-dany93.xml? If not, it will give nasal errors.


I agree, let's remove the size change


We should just implement an `AbstractAction` that does nothing for each of these methods.


Baixa todos os plugins


в сравнении с


Why change from file.path to file.base?  Could this be causing the issue #17?


Permite usar passar params da classe de teste para o controlador, e da uma configuração no broswer.


I believe you meant this line to be:
PATH="/opt/mono-sil/bin:${PATH}"


Permite o uso de metodos do geb na classe de implementação dos steps.


Configura o geb e diz onde fica o chrome driver no computador.


По умолчанию:


adiciona tag de ignore configura as pastas do cucumber.


You can probably drop support for Plotme 0.13g at this point. My stats show that only 34 servers are using it.


Just finished up the last of the search parameters and added extra endpoints.
What are your thoughts about the response json? It seems tedious to map it all to Aeson. I'm debating just returning the strings as is and letting the client decide how they want to use the response. Given that http-conduit is pretty popular this seems to be fair.


looks like this line got duplicated twice.


Por exemplo: não marcar a pasta steps porém marcar cucumber


comando do console para testar


On May 1, 2015 3:41 PM, "ace2nutzer" <notifications@github.com> wrote:
>
> btw u guys @ChronoMonochrome @mkaluza
>
> disabling HIGHMEM without enabling: CONFIG_MMC_BLOCK_BOUNCE will decrease
performance ( I/O Speed ) instead of increasing due to disabled
CONFIG_BOUNCE ;-)

Ok, I've rebuilt the kernel with this option on and it feels a bit better
now ;-) I also managed to make zcache work - so far only for page cache
since I don't use swap, but will play with it as well later. It's
efficiency is about 13%, which also helps a bit with speed at the expense
of cpu time used. Need to use it for a while now...

Thanks again for this tip ;-)


Forgot to write the line ## This is a markdown file as per the request of the assignment. If you are grading me you can give me a 5 as I did not have the line 'This is a markdown file' when I submitted the assignment.


Название


Prefixing a line with `>` is not very common in Elixir documentation. You may want to use a common "pattern" found in Elixir docs when you can't use the `iex>` prompt: don't prefix the code with anything and prefix the result with `#=>`.

```elixir
File.stream!("./test/test.data", [], 2048)
#=> %File.Stream{line_or_bytes: 2048,
#=>              modes: [:raw, :read_ahead, :binary],
#=>              path: "./test/test.data", raw: true}
```

Also, don't forget the spaces after the commas between the function arguments :).


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


This just move the template to a presenter class so you can use config cache command without any issue


same here


TeamCity DnnPackager :: Continuos [Build 1.0.6+1](http://dazinator.ddns.net:81/viewLog.html?buildId=857&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:18


blank line :smiley:


What about line 6?


blank line


Blank line :smiley:


Blank Line


You aren't doing anything with the return value here and this call doesn't have any side-effects.


Metoda niepotrzebnie publiczna, do poprawy w następnym commicie


This should _probably_ be a `std::uint32_t` instead of `auto`.


Blank line


The names of these functions are a little long, which isn't that bad, but from the name I can't tell what they do.  They are always paired with the check_size_ok calls.  Why not merge them or call check_size_ok from these check calls?

nit: you can replace if(boolean_check) return 1 else return 0 with just return boolean_check


:v


Okay so I've worked out how it works. OpenVR is a runtime, which is linked to with libovenvr_api.so (or the equivelent on Windows). Basically the only current implementation connects to the SteamVR runtime, but assuming Oculus implements their own OpenVR runtime, it would have the software connect to the Oculus OpenVR server (probably located within ovrd.exe). So essentially it's up to the hardware manufacturer to implement a way for their headset to talk to OpenVR, but if they do, then any OpenVR software should work without issue.

The only issue currently is that while the SteamVR runtime supports the Rift, the current implementation doesn't use Timewarp (probably to help downplay the fact that the Vive will lack timewarp). It's quite noticable in TF2 and any demos I've tried.

If Oculus (or an interested third party with access to the oculus runtime, ie anyone) makes an OpenVR runtime implementation that successfully supplies timewarp, then implementing it for Dolphin VR would let it work with any headset with an OpenVR runtime implementation.

Also it looks like 2EyeGuy is going to look at implementing it in the next week regardless.


indentation :D


Stałe prawdopodobnie niepotrzebnie publiczne, obadam w następnym commicie i jeśli będzie to możliwe to przerzucę je pod klauzulę private.


Shouldn't systemd be a "Requires" and not a "BuildRequires"?  We start docker through systemd, so it seems like it should be a requirement.


@Glenpl Jak używasz `this` to chyba możesz dać normalną nazwę argumentu, `bombs_amount`?


I'm starting to like `THREE.Segments`. It is true that `THREE.xxxLine` is helpful to understand what extends what, but we don't have `THREE.xxxLineObject3D` either.


Please move this into a function


This is not tested. Waiting for results from Jenkins.


@MacDada mogę, zmiana czysto kosmetyczna


@Glenpl Nie rozumiem idei `update_board` – co ona „aktualizuje”? Czemu przekazuje wynik gry?

Nie zaglądam co robi `field.update()` ani kto korzysta z tej metody – po samej nazwie i zawartości nie wiem o co chodzi.


am I forking now?


Actually... Maybe `THREE.LineSegments` is better.


This logic change introduces a bug into the code since input $args can include unrelated content for api parameters (See line 113 in the commit). Basically your assumption to pass proper content for $args is not correct.


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


_The_ support ...


I don't believe its implied that mutation occurs by saying crossover. These are different operators. Same with replacement. Different evolutionary approaches do these things in very different ways and although the order is usually crossover - mutation - replacement I don't believe we should make the reader make that assumption. I agree that it sucks to have long function names though. Sentence would probably be better. Since these functions actually do selection - crossover - mutation - and replacement they should be named and described as such. Maybe test suite crossover could be "cross-breed organisms" or something and testCaseCrossover could be "breed new test cases within an organism" to better describe at a high level whats happening. Then when we describe these we'll say exactly how the selection, crossover, mutation, and replacement is done.


TeamCity DnnPackager :: Continuos [Build 1.0.6+1](http://dazinator.ddns.net:81/viewLog.html?buildId=859&buildTypeId=DnnPackager_Continuos&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:19


I have merged the request using rebsae, the new commit is bad1b867ec91e2b57dd2a13ce8a82c9a6f10c39a


Can you squash this commit?


@Glenpl Może utwórz strukturę reprezentującą koordynat – jest sporo miejsc gdzie używasz właśnie tych dwóch argumentów razem.


>> -Notice- The ... message means ...


Huh. When that happens to me, I'm either on the wrong branch or commit. You sure you pulled and all that? I just tried it myself and it works. And is it the same error?


Typo here. Too lazy to send a pull request.


ya i meant to ask @vbatts if it is a requires "systemd-units" bc that seems like the norm


TypeScript 1.5 is now available by installing [TypeScript 1.5-beta](https://github.com/Microsoft/TypeScript/releases/tag/v1.5.0-beta)


I'd prefer you keep the "this." for all of these. It helps (at least to me) knowing what is instance vs. parameter or local. And the rest of the code is using this.


I think you also need libdevmapper-event.so.1.02 otherwise device-mapper-event-libs doesn't get pulled in.


All this is quite interesting.
May I ask what time control and number of games you used to gather your statistics ?


I think we want to keep this property (and set it to 1)?


Could you write the loop instead of using Linq? I use this in projects where Linq is not allowed.


Could you write the loop instead of using Linq? I use this in projects where Linq is not allowed


Instead of manually changing registry entries, i would use [this script](https://github.com/Microsoft/TypeScript/blob/master/scripts/VSDevMode.ps1)

See the [Dev Mode wiki page](https://github.com/Microsoft/TypeScript/wiki/Dev-Mode-in-Visual-Studio) for more information.


Not sure how this interacts with the above comment, but we might want to bring it back


I think we want to keep this, the culture on the team seems to be to remove these things.


Wouldn't cpu migrations be problematic?


What is the use case for this api call? All MCP api calls must be done for admin user in the plugin. And, there is a search() method where api calls are initiated.


This should be fixed now in master now.


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)


I think you probably also need libselinux, libsepol and xz.


Actually, this doesn't account for the situation where the vg is marked as partial because a previous owner marked it as such, but we're not actually missing any physical volumes (such as if the resource moved from a node with a missing device to a node without).  I have an alternate approach incoming.


d


[![Naga-enthroned Buddha](http://api.artsmia.org/images/8146/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=8146)


[![Fully-articulated human skeleton: #Z81:21:1](http://api.artsmia.org/images/122944/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122944)


[![Human skull: EDU:Skull 1](http://api.artsmia.org/images/122945/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122945)


[![Human skull: EDU:Skull 2](http://api.artsmia.org/images/122946/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122946)


[![Hand of Ernest Schelling](http://api.artsmia.org/images/122960/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122960)


[![Mary Magdalene Meditating on the Bible, with Skull](http://api.artsmia.org/images/122961/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122961)


@mhegazy sorry for the distraction.  i believe this can all go away now with the VS 2013 and VS 2015 TS 1.5 Beta installs on http://typescriptlang.org


Is there any evidence that this helps resolve the problem, or is this just magical thinking?

I think the docs you want to clarify NetworkManager's load order are `/usr/share/doc/network-manager/README` and/or `/usr/share/doc/network-manager/README.Debian`.


It seems that "canola oil" has been misspelled as "canoli oil". Need to make the proper correction.


Hey @emanlove, we need to discuss moving our testing over to sauselabs as well.


Don't forget I added an extra variable :)
https://github.com/LouDnl/ArmA3_Wasteland.Altis/commit/4506d585c3cab9588014ea0d72830d1fb5d52367#diff-947e8e22f5baac85e91ba6d70da120d2


Here are some useful docs from the [Arch Wiki](https://wiki.archlinux.org/index.php/NetworkManager#Network_services_with_NetworkManager_dispatcher).


it did not work for my environment, need to use http:// instead of svn://


@Glenpl Po co? Ktoś chce nowe pole, niech sobie nowe zrobi.


Oh wow this was forever ago. Do you need me to test it?


Make Tick stable, and make HyperSpeed better


@Glenpl W innych metodach nie dokładasz angielskich `a` czy `an` ;)


CONFIG_NO_HZ is deprecated.. new configurations uses: CONFIG_NO_HZ_IDLE and is good for powersaving.

But CONFIG_NO_HZ is the fastest ... see here:

If CONFIG_NO_HZ is set, the limit is 10ms fixed.
If CONFIG_NO_HZ is not set or no_hz=off boot parameter is used, the
limits depend on the CONFIG_HZ option:
HZ=1000: min=20000us  (20ms)
HZ=250:  min=80000us  (80ms)
HZ=100:  min=200000us (200ms)

in our codina config HZ is 100 .. so very slow.. but if we can trust this.. we don't need to care about it because we use no_hz ... but anyway i'll try to change this @ 1000 xD .. the Problem is that menuconfig always reset this @ 100 again .. i need to hack this ;-)

But gut idea i'll use this config also for laptop here...

the Value: CONFIG_RCU_FANOUT_LEAF=32 is causing too much overhead .. u can hear that because the fan is going loud for every single small application loading ... of course I/O is faster ... but on the other hand it cause too much CPU Load .. and i think at the same time the rest of system have less CPU bandwidth = less system performance.

btw: i'll do new commit for RCU (for ace2 and laptop)... i've found here some great optimizations:
https://forums.gentoo.org/viewtopic-t-793263-start-125.html

As u can see that Value for LEAF is at minimum = 2 ! instead of Stock = 16 ! - That will reduce the Power consumption and load ...

RCU_NOCB_CPU and RCU_BOOST both is to improve I/O .. it reduces the OS Jitter ... and will make user application running more "smooth" and at the same time powersaving... that's now a great config for speed + powersaving .. usually good enough also for our codina device .. but i don't now if it's possible to implement the "missing" options ...


This is looks like bad design, to return an internal mutable field from an accessor and have something else modify it directly. Even more, e.register just adds e to the list. Why isn't it the other way around, like kem.register(e)?


Esse repo me permitiu baixar os plugins


Does this actually save with a pointer to the source?


@Glenpl Dorobiłeś helperki typu `is_bomb()` – więc możesz uprościć IFy.


The Initial WP BuyRite WP project to work on


you're right


Done in bcd5463


Or is this what I want?


The point I was trying to make is that although both crossovers use essentially the same code. They serve very different tasks in the algorithm. I wasn't sure about the work perturbation but the idea is that this test suite crossover and adapt ranges only occur during a very specific time in the algorithm and they're done for a very specific purpose so they should be grouped together and described as such. Perhaps local optimization should be included in this section as well, or in its own separate section since it too is a separate workflow from the typical generation. The typical generation is just test case crossover and mutation and as such I think these should be described on their own as the main, most typical, most basic, etc operations of our GA.

The goal of my rearranging was to try and break the algorithm down piece by piece not by similar pieces of code but by the logical reasoning behind the operations, what they're supposed to do and the order and situations in which they occur. I don't think we should provide algorithms for test suite crossover, or selection or replacement because they are very basic. But they do need an explicit sentence or two saying what they are and a reason why they are used.


Maybe if I comment on the new version instead of the old?


@Glenpl Trochę masło maślane, proponuję `TYPE_BOMB` i analogicznie reszta.


Did you want to include the fixes for docker.service/docker.socket which up the file/proc limits? Also, did you want to include the sysconfig files?


@Glenpl Nadal wszystkie bebechy na wierzchu, zamiast enkapsulacji.


@MacDada no już Ci mówiłem, że w ten sposób czyszczę śmieci z RAMu :D przy każdej kolejnej grze jest tworzona nowa plansza, inaczej nie dałoby się zmieniać rozmiaru planszy itd. Jak chcesz wiedzieć jak to dokładnie przebiega to poszukaj w bebechach klasy Gameplay.


Could you change all the tabs to 4 spaces please?


@Glenpl Argument `lose` jest tu zbędny (nieużywany).


@MacDada no wiem właśnie, tylko "bomb" może być też czasownikiem i to a jest dla podkreślenia, że to rzeczownik :D zmienię w następnym commicie


83811490-Fault_DirtyAirFilter (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: 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)


@Glenpl Co to są te „flagi” i co one interesują pole planszy?


btw, only when you are sure it can't  make document down, you can use 100,000x.


@MacDada tak, zapomnialem pozmieniac


Can this have a better name?


@Glenpl Liczba bomb także nie interesuje pola planszy.

Hint: interesuje to Boarda – Board trzyma pola, więc on wie ile z nich jest bombami. Logika należy więc do Boarda, a nie BoardFielda.


@Glenpl Jedyna linia, która ma prawo być w tej funkcji.


Назови criterias =)


Does this do the right thing WRT DST transitions?


Renamed! Thanks @WestLangley!


I believe I got the details right here, but you will want to double check. I was just trying to get things installable ASAP to start playing around.

Did you write your cabal file from hand or use cabal init?


@lavalamp PTAL at this addition


@Glenpl Co BoardFielda interesuje jakiś wynik gry? Czemu on miałby rozumieć jakieś skomplikowane pojęcia jak „zasady gry”? On rozumie tylko siebie! Nie każ szeregowemu decydować o taktyce bitwy! :)


83811490-Fault_DirtyAirFilter (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


`(boost::none)` can be omitted, as optional default constructs to the disengaged state.


if that's what you use I've problem already without Opera ...
![screenshot from 2015-05-02 00-12-13](https://cloud.githubusercontent.com/assets/85749/7438680/092444c0-f060-11e4-8453-1260aa4318bf.png)


@MacDada dobrze ze wspomimasz, przemyslalem to i chyba zmienną flaga uda się całkowicie usunąć z programu


mad? me? at you? ..psht never! :wink:


Can Bubbles still exist? If so, is just ignoring them the right response?


looks good :+1:


@Glenpl Po co chcesz zmieniać rozmiar planszy? W którym momencie rozgrywki jest Ci to potrzebne?

Oh wait, nie jest.

Jest Ci to potrzebne do nowej rozgrywki. Nowa rozgrywka => nowa plansza.

Zamiast modyfikować pola i plansze przy tworzeniu nowej rozgrywki – twórz po prostu nowe. I tak się bawisz w czyszczenie – czyść pamięć po starych obiektach, a nie obiekty jako takie.


this commit breaks jackd compilation on debian wheezy 3.8.13-bone70 armv7l GNU/Linux with Python 2.7.3 and gcc version 4.7.2

```
HEAD is now at 7f36813... fix configuration when svnversion.h exists in source directory
debian@pollyws14bbb:~/jack2$ ./waf configure --alsa
Setting top to                           : /home/debian/jack2
Setting out to                           : /home/debian/jack2/build
Checking for 'g++' (c++ compiler)        : /usr/bin/g++
Checking for 'gcc' (c compiler)          : /usr/bin/gcc
Linux detected
Checking for program doxygen             : not found
Checking for program pkg-config          : /usr/bin/pkg-config
Checking for 'alsa' >= 1.0.18            : yes
Checking for 'libffado' >= 1.999.17      : not found
Checking for 'libfreebob' >= 1.0.0       : not found
Checking for 'gtkIOStream' >= 1.4.0      : not found
Checking for 'eigen3' >= 3.1.2           : not found
Checking for header windows.h            : not found
Checking for 'portaudio-2.0' >= 19       : not found
Checking for header mmsystem.h           : no
Checking for 'celt' >= 0.11.0            : not found
Checking for 'celt' >= 0.8.0             : not found
Checking for 'celt' >= 0.7.0             : not found
Checking for 'celt' >= 0.5.0             : not found
Checking for header opus/opus_custom.h   : not found
Checking for 'opus' >= 0.9.0             : not found
Checking for 'samplerate' >= 0           : yes
Checking for 'sndfile' >= 0              : yes
Checking for library readline            : yes
Checking for header readline/readline.h  : yes
Traceback (most recent call last):
  File "/home/debian/jack2/.waf-1.6.11-30618c54883417962c38f5d395f83584/waflib/Scripting.py", line 97, in waf_entry_point
    run_commands()
  File "/home/debian/jack2/.waf-1.6.11-30618c54883417962c38f5d395f83584/waflib/Scripting.py", line 151, in run_commands
    run_command(cmd_name)
  File "/home/debian/jack2/.waf-1.6.11-30618c54883417962c38f5d395f83584/waflib/Scripting.py", line 143, in run_command
    ctx.execute()
  File "/home/debian/jack2/.waf-1.6.11-30618c54883417962c38f5d395f83584/waflib/Configure.py", line 128, in execute
    super(ConfigurationContext,self).execute()
  File "/home/debian/jack2/.waf-1.6.11-30618c54883417962c38f5d395f83584/waflib/Context.py", line 87, in execute
    self.recurse([os.path.dirname(g_module.root_path)])
  File "/home/debian/jack2/.waf-1.6.11-30618c54883417962c38f5d395f83584/waflib/Context.py", line 128, in recurse
    user_function(self)
  File "/home/debian/jack2/wscript", line 631, in configure
    except FileNotFoundError:
NameError: global name 'FileNotFoundError' is not defined
```


I don't really get the m_ prefix.  I would probably just call it GetFileTime or something...


@Glenpl Metody to funkcje, funkcje go działanie, działania opisujemy czasownikami.

`is_bomb` – czy jest bombą – `is` to już czasownik, `bomb` musi być rzeczownikiem ;)


@MacDada no dokładnie to robię! :D trochę się nie zrozumielismy, mogę Ci dokladnie opowiedzieć jak to wygląda, ale chyba nie jest to potrzebne :p

Zmiana rozmiaru planszy -> miałem na myśli to, że po wygranej/przegranej, po wybraniu opcji zagraj jeszcze raz user może wybrać inny rozmiar planszy. Rozmiar planszy jest argumentem konstruktora, nie da się go później zmienic - TRZEBA tworzyć nowy obiekt.


Gg


@MacDada a funkcja clear() jest wywoływana zaraz po utworzeniu obiektu, żeby usunąć śmieć-wartosci które ma w sobie - wartości zapisane wczesniej przez inny program w danej komórce pamięci RAM. Takie uroki dynamicznej alokacji :p


looks good.


I saw your note in the glimmer PR. I pulled this and ran it, think we're hitting something similar. If you haven't already, perhaps you should open another issue on the ember repo, since this probably isn't directly related to glimmer (but definitely a place where improvements would be awesome!)


looks good


What response json? Map it all to Aeson? Oh, you mean make all the responses into ADT's and define To/FromJSON instances right?

Well to begin with you could try this:

http://stackoverflow.com/questions/17278472/arbitrary-json-keys-with-aeson-haskell

Then if you have issues or want to statically define the responses later you can do it in true Haskell style, lazily evaluated :P


ეს არის რელატიური font-ები, ანუ იცი ზუსტად ეს ზომა რას ნიშნავს, რომ თუ არა დაგილინკო და დავაკომენტარო?


You're right, the m_ is inappropriate (left over from when this was a virtual property that returned a delegate)

I'll do GetTime (since it's not specific to files or directories)


It's a bug, I bet EntityManagerInterface is from a newer version of the orm than you're using.

> On May 1, 2015, at 2:59 PM, Coyote <notifications@github.com> wrote:
>
> do you think this is a bug ... or its something in my app???
>
> —
> Reply to this email directly or view it on GitHub.
>


ეს მაგალითად შეგიძლია ესე მოკლედაც დაწერო ხოლმე, იგივეა:
margin: 0 auto; - ზედა და ქვედა mergin 0 არის მარჯვენა და მარცხენა auto.
margin: 0 auto 0 auto; ესეც იგივეა ზედა 0 მარჯვენა auto დაბლა 0 მარცხენა 0


How about `veryveryrecognisabletokenthatnoonehasintheirloginscript`?


Ack!


My 733t git skillz apparently deleted the patch that added these.  Here they come back!


И тут то я понял что это за фабрика, ))
 из книги   -- Автор: «Банда четырёх»: Эрих Гамма, Ричард Хелм, Ральф Джонсон, Джон Влиссидс ... Паттерны проектирования)  >>:|)- То DeleteModal не есть подходящим названием функции внутри этой Фабрики, из-за двузначия в значениях слова Delete :
( это может быть как существительое, так и сказуемое (глагол) , { Підмет або Присудок, дієслівний складений } ). По крайней мере, для меня название любого метода начинающегося c Delete, Remove, Reboot, KillAll и FormatC вызывает лишний нервный импульс. Не так ли, о мудрейший АлексДжи ?!


This was a revert of 066d5db, an accidentally merged pull request that lacks an association with a signed contributors agreement.


Just familiarizing myself with Github so that I can start using it for my re-entry into programming world!


Haha hm.. I'm not sure, about the failure message.

Is there a way to detect, whether it really failed or not? Maybe by returning a different status code from `load_most_relevant_state()`, such as `-1`, if it failed, but `0` (or so), if there was nothing to restore?


I could not find a contributor's agreement for htgoebel and thus reverted this commit. Sorry!
We may wish to ask the Plone Foundation if certain repositories that lack a strong relation to plone the product may be handled more loosely.


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


აქ
-moz-transition: all 0.2s;  /* mozila browser */
-ms-transition: all 0.2s; /* internet explorer */
-o-transition: all 0.2s;  /* opera */

ძველი ბროუზერის ვერსიებისთვის არის ახლებისთვის -webkit საკმარისია


Would it be useful to make pod.Spec.ServiceAccount be a *string so that in the case where it is explicitly "" we can not add one, and only automatically add one if the field is unset?


```c++
uint64_t calculate_and_update_devmsc(unsigned int nTime)
```
... should ideally be updated as well, given that `devmsc` is now signed, and it also returns a negative status code `-9919`.


It's cool to see that u like my tip Guys ;-)

@ mk as for zcache .. good idea.. i'll take a look when i've more time for this. :-)
ATM i'm impressing from RCU Tuning ....


იმისთვის რომ ამ შენმა ანიმაციამ იმუშაოს hover გჭირდება ზუსტად ამ tag-ზე.
ანუ შენ ეხა გაუწერე, რომ ანიმირებული იყოს მაგრამ უნდა დაამატო რა შემთხვევაში, როდესაც რა იცველბა, მაგალითად:
nav ul li:hover {
padding: 50px;
}


In response to https://github.com/noinnion/newsplus/issues/97


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


Oh, bummer-- I have an unpublished WIP that would have made this easier. If you beat me in I'll just refactor the call sites.


@Youmna-Salah  fix the indentation please


Hay que agregar al PostDTO id publicación además de id publicante, avatar y nombre de publicante


@Youmna-Salah  indentation again


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.37](http://teamcity.monogame.net/viewLog.html?buildId=6716&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:01


I suspect not, since a DST transition is like changing timezones, and DateTime doesn't store time zone information, only that it is "local time."

I'm not sure if there's a better way to test APIs that return a DateTime.  I could capture DateTime.Now before and after the call to GetTime() and pass the test if either comparison succeeds... what do you think?


Variables should be in camel case


> copied from 0.9

In 0.10 it was extended to:
```c++
/** Abort with a message */
bool AbortNode(const std::string &strMessage, const std::string &userMessage) {
    strMiscWarning = strMessage;
    LogPrintf("*** %s\n", strMessage);
    uiInterface.ThreadSafeMessageBox(
        userMessage.empty() ? _("Error: A fatal internal error occured, see debug.log for details") : userMessage,
        "", CClientUIInterface::MSG_ERROR);
    StartShutdown();
    return false;
}
```

And the following would functionally be the same:
```c++
std::string strMessage;
mastercore::AbortOmniNode(strMessage);
AbortNode(strMessage, strMessage);
```

I think the general idea is fine, but it may also (or instead?) write to mastercore.log. What do you think?


Yeah looks good.


აქ რამდენადაც მივხვდი <p> ტექსტი არ გინდა, რომ მიწეპებული იყოს დივზე, აი ნახე ეს ლინკი, და თან თუ სხვაგანაც მაგალითად margin ინტუიციურად გაქვს გამოყენებული ზუსტად მიხვდები მნიშვნელობას:
http://www.w3schools.com/css/css_boxmodel.asp


Please add a comment that PKT is only allowed for back compat reasons and the value is actually not used.


yii2-dev repo has the `"type": "yii2-extension",` in composer.json to trigger the symlink for Yii class to be created in yii2 directory. So it has to be added here. Also when only yii2-dev is installed, yii2-composer needs to be required to allow installation of extensions.

should all be resolved by https://github.com/yiisoft/yii2/commit/f6c72d25bfc2d112e0c50dd3e5592615aea92a96 now.


Nope deb packages will always link against the installed libraries. And the build system provides all the needed libraries and dependencies.


this is...


Ok


For the input files, I agree it's obscure to read directly... but there's a parser for this if a developer wants to look inside it. Also, each is well documented with what's in the factor graph, and what's to be tested. I think for end-to-end tests, the goal is that given input, the output should match the expected. I don't think it's very necessary to add format converter here. If we write the factor graph in plain text, when test fails, we would have no idea if the factor graph is not valid or the sampler is failing...


a


83811490-Fault_DirtyAirFilter (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (623 of 623 tests passed)

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


f


Ограничения:


показать версию:


Параметры


Не нашел, чтобы где-то использовалось


Название


rax retry tests


Сгенерировано с помощью


Общая информация


When I tested the sample with this change, it doesn't work correctly. Before, the canvas is initialized with all the random states. After, it's empty (white) and cell states do not propagate correctly to their neighbor.

Just looking at the code, it appears that the DiscreteTransferEffect 'liveOrDieEffect' is depending on its input being in the 0-1 range.

Did you test this some other way? Or, could there be something missing here?


сравнить с:


LGTM


Any explanation why was this reverted? It fixed Issue #49 as well as other subtitle bugs.


Why another class? VehicleParams.cs has the same logic / constants

(https://github.com/ikkentim/SampSharp/blob/fedf4545aecb9f1c106490644d3909f99170d7a9/src/SampSharp.GameMode/Definitions/VehicleParams.cs)


Indentation!!


აქ სატესტოთ შენთვის ცადე float-ის მაგივრად display: inline-block;-ის გამოყენება, დაინახავ ნათლად რისთვის იყენებენ ხოლმე, გამოგადგება ვიცი :).