Sonatype Nexus' gem repository uses something like `base_url/gems/{first-letter-of-gem_name}/gem_name`.

https://books.sonatype.com/nexus-book/reference/rubygems.html


Our coding style recommends: `if (baseURI == nullptr || relativeURI == nullptr)`


I think it is good enough as it is now (not this commit but the current master): just always return it.


The test will fail if any string 'graceful' is appeared in a request's log. Can you be sure it won't happen, perhaps it would be better to check the log for this string: 'gracefully close the connection'?


@pynicolas I think you should remove `<SyntaxToken>` here and make modifier of type `InternalSyntaxToken` everywhere (except interface).


`min.x` is integer, inaccurate center position.


`max`, `min` are integer vectors. can't calculate proper halfsize?


Gracias! jaja


This dosn't work well on Windows, it returns: `<install_path>\\Schemas/`. This works, though:

    schemas_dir = os.path.join(install_path, "Schemas", "")


That or change the `updated_attributes` to `exiting.links = content_item[:links]` and let the save transaction handle both.

The save which hits last will win, but be internally consistent.


Would it make sense to make query tests fail the build by default in gl-js? They should be as stable as normal tests and don't suffer from gl rendering instability.


Wow, das mit dem GPU sieht ja schon echt nicht schlecht aus!
Was für eine Fehlermeldung kommt denn, wenn du die eine Funktion mit double benutzen willst?
Und ist alles andere soweit getestet und läuft korrekt?


Yeah I see what you mean, but a few of the default atom packages for linux override some of these keybindings. For example, using v0.2.0 (the above code) on Ubuntu, the keybinding resolver shows:

`ctrl-e` is mapped to `find-and-replace:use-selection-as-find-pattern`
`ctrl-d` is mapped to `find-and-replace:select-next`

Since `find-and-replace` comes with the default installation of atom, I'd say that this package should try to work around it. The above code with `.platform-linux` worked for me.


CC @ttuegel


@pynicolas `@Nullable`


Hello, why you deleted `has` method? Since when the use of "@" is a good programming practice?


Hmm.. you think of `replaces="libogre-docs>=0"`? I have no idea what to put into a `provides=`...


This thing has definitely nothing to do with TeamKickHormone at all.


Awesome, thanks for RC support!


The reason I added this is because after editing a profile, the profile would be reloaded too quickly (before it was completely updated) which caused some problems.


Thanks! :+1:


There's another way to fix the dead lock


why remove the checkLoggedIn check ?


I need to know from the SlapperHuman class if the config has "PlayerSkin: true" or not...


dodan link na Learn more btn


I'd argue that you should create a separate package for this. How are you going to maintain it in this repository? It is considered bad practice to specify a branch in your composer json files.


Congrats! You succeeded where I failed.


But I can't access the config from it.


It's improved slightly in commit 2fee4f04ba04548e48d15a73e97d2eaa8cc82efa, and is used sparingly. I like it.


Using the keybinding resolver on Ubuntu with v0.2.0, `ctrl-p` is mapped to `fuzzy-finder:toggle-file-finder`.


Menuda estupidez, no vales para esto


I wonder if this is necessary since directory of $GERRIT_SITE has been created in Dockerfile.


LGTM


in a general way this is not correct when checking permissions, as Anonymous (group Anyone) has its own permissions.
For this specific case, anyone has the permission "provisioning" since 5.2


This rule also has an effect on leaving "optional" function parameters as `undefined` when calling a function.

```js
function foo(bar, goo) {
    if (bar === undefined) {
        bar = 'just a default bar';
    }
    if (goo === undefined) {
        goo = 'just a default goo';
    }
    console.log('My bar is ' + bar + ' and my goo is ' + goo);
}
```
I want this to work:
```js
foo();
```

But I also want to be able to skip `bar` and just provide `goo`:
```js
foo(undefined, 'my special goo');
```

In this case using `undefined` seems like a solution which allows both above invocations to work. That is another reason to drop `no-undefined` from eslint, unless there a better way to do that than by providing `undefined` as value.


Users of this API should catch raised errors. Maybe not all callers but at least the public APIs: you don't want to terminate the program when the users enters an invalid input.


I would prefer a strict comparison here `===`


So uh, I actually didn't mean for this commit to be in this PR, I just apparently didn't know how github's PR system works :)

Agree that this should be in a separate PR! Thanks for taking the time to review it!


Sure :)
On Sep 28, 2015 11:22 AM, "Serhii Komar" <notifications@github.com> wrote:

> I see that different envs produce slightly different results (up to 70
> chars).
>
> @nadavge <https://github.com/nadavge>, could you please set db.string(100)?
> :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nadavge/notejam/commit/f5a41c23014f318d202376b20e2a5e610f111177#commitcomment-13462936>
> .
>


Byl problém v tom, že getattr vytáhlo property object, neaplikoval se dekorátor a tím pádem neměl short_description a když to vytáhneš jako metodu, tak se aplikuje, chápu to dobře?


Thanks, @jkotas. I'll change the check to `dictionary.GetType() == typeof(Dictionary<TKey, TValue>)` to ensure it's not a `Dictionary` subclass (to avoid the subtle behavior change) and post the benchmark numbers to see if this is even worth it.

From a [cursory](http://referencesource.microsoft.com/#mscorlib/system/collections/generic/dictionary.cs,6009e5bfe5f70341,references) [look](http://referencesource.microsoft.com/#mscorlib/system/collections/generic/dictionary.cs,c59275a169cb495a,references) at the full framework reference source, most (perhaps all) callers of these ctor overloads are passing in instances of `Dictionary<TKey,TValue>`.

I didn't mention it in the commit/PR message, but this also avoids the interface dispatch for the `MoveNext`/`Current` enumerator calls, which, in my initial testing, showed improved perf when enumerating larger dictionaries.

I need to verify, but it looks like the new dictionary's private arrays are lazily allocated and not allocated when the passed-in dictionary's count == 0. If the enumerator allocation can be avoided in the empty case, then the only allocation would be the dictionary instance itself.


Use {}


Thanks for the suggestion. How do you deal with subsumption in your type assigner? Do you just return the most precise type?

In any case, I circumvented the problem in a later commit by implementing more efficient automation.
https://github.com/TiarkRompf/minidot/commit/9196a35886487434a7e213706389a1b5d02a8db5


an OSGi service call


Tohle je super. Jinak za sebe žádné chyby nevidím.


The cast to string is implicated by the return type.


How did you build the cm3 toolchain?


Retrieving task data...


Don't use String format with LOGGER


ano přesně tak, potřebuju vytáhnout metodu na class jelikož na instanci už vytáhnu property a metodu nijak nezískám (přesněji vytáhnu výsledek property)


@vinkla Yes I am gonna maintain it in the same repo. It will be way easier to maintain as most of the code is the same.
For me this is not an issue nor a bad practice (laravel/framework is doing the same)


This is beautiful.  I especially like the new implementation for stack/3 in commit ef4c7b2c05fb860dada9e2548c951f2ca7d3ebba


Yeah, I've thought of these things. But I don't want to overcomplicate things by having a new set of automap controls. And I don't want to have it so the automap can appear on 2 displays at once. There is always the console to enable grid, etc., but we'll see... I'm still playing around with things. It's early days yet. :smile: Maybe pressing tab will move the automap to/from the main display.

I came up with this whole idea by playing around with the Nintendo 3DS the other day.


Rozumím, díky moc:)


The wget & curl is installed in buildpack-deps:jessie-curl which is the parent image of java:openjdk-7-jre.


Remove space at the end


don't use String.format, use {}


Using aforementioned xposed module with Google WebView enabled local commute Hotspot to authenticate. Is it feasible to update the cm version?


Yes thanks for the clarification.


I have to admit, I don't understand why there would be a problem here at all... 83886080 is `5<<24` - why on Earth would the compiler decide to do this...? It looks like it's getting the byte order wrong... And the difference between signed and unsigned ints doesn't explain anything: they're both positive, the only difference between the two representations is that the top bit in a signed integer means it's negative - not the case here...

Once you go through all the SFINAE stuff, the function collapses to:

    TypeOUT round_func (TypeIN in) { return in; }

I don't see why this would require an explicit cast. More to the point, this should have created all kinds of problems in other contexts by now - although we don't use `uint32_t` all that much...

Are you sure whatever command is having trouble is running clean in valgrind? There's no other potential reasons for this? Does `mrconvert` to/from `uint32` expose the problem too...? I'm more inclined to think it's a memory access issue in the app than anything wrong with this part of the code. I could be wrong here, but this just doesn't make much sense otherwise...


@dakcarto Coverity flagged this unreachable code - can you check if this solution is OK?


AipoLocalization_it.properties
ではなく
AipoLocalization_ja.properties
を編集するようにしましょう。日本語用です。
こちらのファイルを変更した後、mvn install することによりすべての言語ファイルに同期されるようになりますので、一つのファイルを修正すればOKです。


what task data providers? I meant that this should log something to the effect of: XXX (forgot its name) service registered, starting to register queued providers


Although this is a blocking script, it's likely cached because.. well, you know... it's jQuery. So the performance hit by putting a blocking script in the head is likely negligible.


personally I think it would be good if all this GUI stuff was moved out into a separate project.


**Eslint**: Unexpected token


Update to build #35


why do you need `Product with Serializable` at the trait level?


These 5 lines of code should be moved to their own separate method


Compilation returns an error on this instruction because `QKeySequence::FullScreen` does not exists.
Maybe we can remap this to "F11".


great that you still support the original feature flag!


The ca-certificates is installed in buildpack-deps:jessie-curl which is the parent image of java:openjdk-7-jre.


It means that as soon as any change happens that branch everyone will get that update immediately. If bugs are introduced in new commits everyone will be affected.

[You should use a specific ( tagged stable at best ) version wherever possible.](http://stackoverflow.com/questions/19009334/when-dev-master-should-be-used-in-composer-json)


Pour mozilla firefox, il faut ajouter le "handler" event pour que le comportement du navigateur soit bloqué.


Test look good, but try to move the many-repeated dates to the constant in this test


as context.getDrawable is deprecated we should use ContextCompat.getDrawable(context,drawable)


ghswdfhgwghw


Ive updated the PR check it out it works fully.


Getestet ist es mit zufallstest.cu . Das setzt natürlich eine korrekte CPU-Rechnung voraus. Der Norm des Defektes ist zwar meist nicht 0 aber Norm(defekt)/Norm(Ergebnis)  ist sehr nah an Maschinengenauigkeit. Ich vermute einfach es sind Rundungsfehler.

Wenn ich double in spmv2 verwende, kommt die Fehlermeldung, dass die Funktion atomicAdd nicht mit (double*, double) überladen wurde.


indentation issues: you are using tabulations.


Just my opinion, but the name is slightly obscure to another person reading the code. Why would you not want to define something that says "do things correctly"? I suppose by correctly you mean a CPython slicing?


Typo: "show [rates] for a *man* [...] who wants to top up *her* pension [...]"


@Maxhy , Does this https://github.com/dokan-dev/dokany/blob/e213110d3bac7b98d70c3a2b607f670ae8417d3d/dokan/volume.c#L54 should not be changed too ?


It's Cool, thanks


use camelcase notation for naming variables


It would be good to explain why this test has been removed. Reading the commit note, It's not immediately obvious to me.


但是我自己机器上面不用添加那些`framework`怎么就能make过了那？


And in a more global way, we've decided some time ago to remove every checks on permission (and logged in) in service and to only do them in WS.
It allows for instance for a service to call another service without being bored with that.


Hi Thomas, I noticed that you left the assert-test macro in the code.  Was this intentional.
It seems to me you can remove it if assert-true does the same thing.  Right?  Perhaps you're just waiting
to verify that it works as expected?


remove this bullshit


By looking at the documentation, it appears that it can only fail if memory allocation fails or if the pointer is NULL... these are unlikely to happen. I'd use an assert() or an ALWAYS_ASSERT here.


Thanks. Don't know if you were aiming at this, but this is a bug as it won't pick up non-final states.
Committed another iteration.
Note that I consider this WIP as its not super critical right now for my experiments. (Albeit a bit annoying)


do you really need all these events????


リファクタの結果を足し算できんかのう…
```
case class RefactorEffect(
  procedureId: Int,
  refactorType: RefactorType,
  changes: Seq[FileEdit], // <-- ここ
  status: scala.Symbol = 'success // redundant field
) extends RpcResponse with RefactorProcedure
```


I actually set this to be the default strategy a few snapshots ago


:red_circle: You may need to improve the commit message. The current header `Refactor` does not provide adequate information on the contents of the commit.


:+1:

just wanted to say that this is very clever, merging objects from all groups and then cleaning them all together instead of having separate setClean() on each group


remove all unused handlers


Hi, why not use the official repositories @ http://packages.icinga.org/ubuntu/ ?


Ok I got your point :smile:, I will create a separate repo when it's ready


:red_circle: The commit message needs to be provide adequate information on the content of the commit.


Defining these svg image sizes in CSS as opposed to inline in the HTML is supposedly more performant. I haven't personally tested this theory. Seems to make sense, considering svg is just a string of characters and not a binary file.


where is onDragggedChange function? find more elegant name


I can see why you want to raise an exception here, but I'm not convinced it should be a `SmartAnswer::InvalidResponse`. Also I think there are a lot of similar places where we are *not* raising an exception at all. I think we should decide on a general approach for this and implement it everywhere.


Nice!


don't use inline templating


这里有多余的逗号


Что-то слишком много лишних скобок.


`除非` 前面的句号 `。` 换成逗号 `，`。


:red_circle: The commit message should provide adequate information on the edits made in the commit.


perfect !


Something like that?If I'm not mistaken we have to do that for every widget

```
{%- block form_widget -%}
    {% set classValue = attr.class|default("") %}
    {%- if not form.vars.valid %}
        {%- set classValue = classValue ~ " formError" -%}
    {%- endif -%}
    {%- set attr = attr|merge({"class": classValue}) -%}
    {{ parent() }}
{%- endblock form_widget -%}
```


Maybe it would be a good idea to use an ellipsis character (`\u{2026}`) here? :wink:


这里跟上面重复了


The failure conditions for `xmlBuildURI` are much more complicated to determine... In doubt, I'd use an assert too.


Nit: nice to have:

```cmake
include_directories(SYSTEM
  /usr/local/include/
)

include_directories(
  dependencies/libcsptr/include/
  dependencies/dyncall/dyncall/
  dependencies/valgrind/include/
)
```

else clang 3.6+ throw warnings from system headers.


:red_circle: The instantiation of the `Index` object and JSON file load should be moved into a `beforeEach` declaration.


Mejor no darle más vueltas, Paloma. Entrégalo.


Very nicely done Andrew! Awesome work.


`self.sendCmd("setValidateSchemasOnStart true")`


'uf' seems ok here and is also much more efficient.


Doesn't this still leave a random Python file (get_war_and_peace.py) in the root directory? Maybe that Python file can be included into the Rust source? It could download W&P into a fixed "temporary" directory perhaps?


:red_circle: The `loadJSON` function should be a method of the `Index` prototype.


Boo for this commit name. "Initial Commit" would be better. Perhaps this is a candidate for interactive rebasing?


We don't expose much non-final state at this point I guess, beyond `RUNNING` (I assume that `SUSPEND/RESUME` are not relevant here?).  Understand it is not final, as its not a PR :)


You forgot to finish your sentence ;)


Give evidence or anecdotes for these issues: it softens your point and provides more context than a general rule. For example, change "Difficulty for testing" to "We found that we had to add a significant amount of code to our test's `beforeEach` methods to inject mocks into Angular. [Code is a liability](http://blogs.msdn.com/b/elee/archive/2009/03/11/source-code-is-a-liability-not-an-asset.aspx), and we found it difficult to write."


Yes the return is wanted.
It loops from right to the left through the array and when the condition hits, it should return the combat strength.


Here lays more info https://bugs.php.net/bug.php?id=69187


**Eslint**: Unexpected token


No, because all released platforms(including Broadwell, Braswell and Skylake) can't support VAProfileHEVCMain10 slice level HW decoding as I know. That's HW limitation. For further discussion, would better send mail to libva@lists.freedesktop.org, otc libva-driver team may give more professional answer.


Oups, sorry for the noise, just see https://github.com/nelmio/NelmioSecurityBundle/commit/b45f81f90d95b7c994db675930e9b1623b66ed03 :p


(最近知ったことなのですが...), globalな名前空間で `_`から始まる識別子は C++が予約していることになっています. 既に多用されてまくっているのであれですが, なるべく使わない方が良いかと思います.

http://ezoeryou.github.io/cpp-book/C++11-Syntax-and-Feature.xhtml#reserved-name

> ひとつのアンダースコアから始まる名前は、グローバル名前空間で、利用を予約されている。ただし、グローバル名前空間との名前の衝突が、時として、意外な結果をもたらすこともあるので、言語のルールの詳細を把握していない限り、利用はおすすめできない。


It feels a bit weird that this defaults to `gender = 'female'`. I think it makes sense in the context of `FEMALE_YOUNGEST_DOB` being later than `MALE_YOUNGEST_DOB `, but it doesn't feel as if it is generally correct.


Commented out lines need killing.


wow, such a monolog :p


Ok
...

Pues hago el gantt o la distribución de meses en forma más o menos gráfica y la entrego :/


As I eluded in my comment, I'd add more anecdotal evidence throughout on how you subjectively found working with these frameworks (this is the valuable stuff here).


Okay, das klingt soweit dann ganz ordentlich.

Hab mich bezüglich des Fehlers mal schlau gemacht und das hier gefunden
http://stackoverflow.com/questions/12626096/why-has-atomicadd-not-been-implemented-for-doubles
Scheinbar ist die Funktion einfach nicht für double implementiert. Auf der Seite ist eine zusätzliche manuelle Implementierung dessen allerdings angegeben. Wenn du die bei dir noch intergrieren möchtest...


Use `let()` statement instead. Using instance variables is not best practise because it makes changing the values harder.


Does the page look sensible with no body?


:yellow_heart: A more robust test would verify the book data of `Index` object instance is empty before the JSON file is loaded and verify that the book data property is not empty after the JSON file is loaded. This means that there should be two assertions.


Just did it.


typo, `nil`


I think creating the empty carrays in-memory is a good idea as it leads to simpler code.


Yep let's keep widget_attributes ;)


I thought `{Node,Store}ID` where signed, so better to check for `<=0` (I doubt we want them negative).


:yellow_heart: This test spec should also verify that the strings are not empty.


No trouble.  Thanks for bringing it to my attention, even if it was by accident :-)


@syohex

初めて知りました。情報提供有難うございます。
グローバル変数はアンダースコアから始めるのはやめて、g_などに切り替えて行
きたいと思います。

太田裕貴
名古屋大学 大学院
情報科学研究科 情報システム学専攻
枝廣・加藤研究室 修士1年
h_ohta@ertl.jp

On 2015年09月28日 18:44, Syohei YOSHIDA wrote:
>
> (最近知ったことなのですが...), globalな名前空間で |_|から始まる識別子
> は C++が予約していることになっています. 既に多用されてまくっているので
> あれですが, なるべく使わない方が良いかと思います.
>
> http://ezoeryou.github.io/cpp-book/C++11-Syntax-and-Feature.xhtml#reserved-name
>
>     ひとつのアンダースコアから始まる名前は、グローバル名前空間で、利用
>     を予約されている。ただし、グローバル名前空間との名前の衝 突が、時
>     として、意外な結果をもたらすこともあるので、言語のルールの詳細を把
>     握していない限り、利用はおすすめできない。
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CPFL/Autoware/commit/168315329c5cb936ca17fd20d88c55d7763a2c34#commitcomment-13464341>.
>


I think it might be useful to calculate the rate for 100 year olds as a local variable in the test to assert against.


Cool, thanks!


s/previous/parent/


any specific reasons why you changed it?
Afaik our coding style guide is based on PEAR Coding Standards which says that you should use "elseif": https://pear.php.net/manual/en/standards.control.php


Typo: The test name talks about an age of `150`, but the call to `#lump_sum_amount` looks as if it's still using `101`.


`reposiotry` typo


Good work, thanks for fixing this issue.


I disabled the texinfo checks a few days ago on the systemd branch: e7631452e95d1ed239c88ede8af3709eb05849bf


rename to changeSucceded


Good catch and I agree.  This should be fixed in v0.2.1.  Let me know if I missed any.


looks like it could be time to change the signature, given that the `req` as such isn't really used.


Thanks again.  This should be fixed in v0.2.1


ditto


@cornae For design issues in ikath, please mention them to @pilz and @cillianderoiste , since they are the only 2 people who have done CSS adjustments in this file. In other words, I have not been involved in those customisations, so I cannot say if they are still needed or not.


Current password is required


:green_apple: This is an excellent use of the string lookup method for accessing attributes of an object.


Then they are herewith mentioned :).


It's worth mentioning that FUSE is single-threaded. In general transactions are not enough for correctness.


Ok, so the return 0.5f should simply be moved behind the loop?


missipelling


BTW, how do you make these nice-looking-very-indispensable tables?


We should ultimately drop this and stop the admin module from saving "null" if thats whats it doing


oh nice


Зачем тут указатель?


s/allocate/allocates/


question is, why where there no detection of the problem in jenkins?


@TheRatG please see the discussion in the PR implementing this change. The previous implementation was subject to race conditions


А тут указатель должен быть константным


fix #11


the second line is technically in correct.


:red_circle: You do not need to execute the `searchIndex` method. Since it is an attribute of the `index` object, you should simply check it only.

Jasmine provides 'spies' to help verify that methods of an object are called. The use of these would improve the robustness of your tests.


Typo: "return a numeric value *for* covered age ranges"


I'm using the [jsperf](http://jsperf.com) to create the performance comparison table.


Embedding those 3 structures in a union would decrease the amount of wasted space.


@Sidious1 On ne peut pas importer Color 2 fois, sinon le compilateur ne sais pas lequel prendre.


Ah now I see what you meant: Yes you are right, the 0.5f must be moved to the end.


s/operatin/operating/


jenkins detected it http://jenkins.plone.org/job/plone-4.3-python-2.7/845/console
I think it's just Eric that released 4.3.7-pending without waiting the tests results. ;)


I made showModels module with optional parameter - "show full information" and created showModelsFull module for nice looking callback. Please, is any mistakes I made here?


:question: This test spec is unclear. What is being verified here?


a slight aside, should further pages still list the markers from 1 to 10 (as they do now) or should they continue counting up (which might make a little more sense and then this `count` becomes more useful too)?


Could you take out the changes in ext_typoscript_setup.txt? I'd like to see it as a separated feature.


Thanks, I'll cherry-pick this to release branch


Why are you not using environmental variables?

- Find documents folder name in HKEY_CURRENT_USER\Software\Microsoft\Windows\CurrentVersion\Explorer\Shell Folders "Personal"
- %APPDATA% for "%%x\AppData\Roaming"


RE: camelCase

Man that's embarrassing.  Too much Rust programming has gotten to my head :stuck_out_tongue_closed_eyes:

RE: features flag.

I didn't want to break anyone that was using the features flag, but if you think no one is depending on it, I'll remove it.


I append this here because the rest of the code that you wrote assumes that it's present.


@HipsterBrown  how does this work in local development?


Having started university I don't have the free time I normally would. I might come back to this and provide documentation, and will probably in the process optimize the code.

It would help if I knew what specs I was dealing with, when the browser freezes and what browser you are using :)


这个测试主要是为了以后的直接打开XML和XML转bin准备，其实我感觉这里也可以贴上一个解析好的txt文本。。。


It will work for sure 👍🏻


/*На экран выводится информация о
создании нового процесса + от каждого
родительского, что его ребёнок
завершился.*/
//
while (level) {
		pid[level] = fork();
		if (pid[level] == 0) {
		    	level--;
			std::cout << "New son has been born\n";
		    	continue;
		}
		if (pid[level] > 0) {
			int* status;
			wait(status);
			// doing some job
			std::cout << "My son has died\n";
			// some job has been done
			break;
		}
	}


Looks like... someone was a little confused... :)


lmao


Thanks.


Looks like this is causing the build to fail - descripion instead of description.


All fine if you set the lively-loader version to >= 0.2.5 instead of "roughly" 0.2.5 :-D


@asofold Jenkins hasn't compiled a build for the last 6 commits.


Valid point ;)
Anyway solving #24 will fix that as well. I see you self-assigned it. Are you planning to fix it in the near future, or can I get started on it ? I opened the issue mostly because I have no clue how much impact it has on harbored or other apps that could depend on libddoc.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 734](https://teamcity.labkey.org/viewLog.html?buildId=274801&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 9, muted: 34 Build time: 00:26:15
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

```


Amazing how much of a difference such a small change makes!


Je devrais vraiment faire gaffe à plus refaire ce genre de commit aussi gros que ... ouais


Renamed repo from WesData to ReproData in connection with dropping Wesleyan data Coursera and continuing repro of Seattle rent-vs-buy on my own.


Can't do that as it's a source breaking change.  Anyone who recompiles against our code will get different output behavior and possibly build errors.


TeamCity pull requests :: netty [Build 330](https://garage.netty.io/teamcity/viewLog.html?buildId=1811&buildTypeId=pulls_netty&guest=1) is now running


Err… these are not meant to be "inline ProcessingJS scripts", but examples that may be copied. I am just using <script> tags (with a custom MIME type) to make sure the browser leaves them alone.

I am afraid that if ProcessingJS is ever loaded in this page, it will start running those scripts…


I think we can remove tihs comment.  It's very unlikely we'd break compat by changing the original ctor to have deterministic behavior.


Shades of lisp here :smile:


Bad feels


In the PDF, you've made a copy-paste error: `addApplication` instead of `addRoom`.


The previous version of the test file was big-endian (think FreeSurfer uses this by default). dce962b?


Right, well then we should keep it non-deterministic by default forever.


oh!  i didn't local_deploy again, so my test passed.  sorry about that.  fixing now.


I unassigned severity from a property that refines something in the parent class, and the generated OCL statement for this property changed to "true". So I guess it is no longer validated against?


LGTM


The plan is to remove `s.Roster` unless there's a good reason for the session to keep track of its current roster.


It's getting really hard to tell who's responsible for which checks (it was difficult enough before).  So I'm taking your word that you know when an `assert` is appropriate.  I don't know how how I can audit it as I'm doing a `[FOLD]` by `[FOLD]` review.


okay, now which variable do you want to use for problem 6 when it says "the GC content (which should be equal to (G+C)/(A+C+G+T))" ?


I believe there must be a mapping config -> session somewhere. I'm happy with any alternative to this. We felt like adding support to multiple accounts would help to expose the problems sooner.


Yes!


Aha that makes sense


Ty


Let's fold in to your ' Fixing some (minor) issues' patch and keep this check_boolean refactoring separate.  We'll merge both to master and then decide if both should be cherry picked to the release branch.


I'm not sure it's a cut and paste error.   You did preserve the previous behavior of the script...  I'm just not sure that's the behavior we want.  :)


Remove the newline. The attribute values need to be on a single line.


Ahh I see.  OK.  Then that looks good.


Was wondering about this... much :+1:


I'm just afraid `guiSessionEventHandler` will need too much information from UI to respond to the events.

> and when some of the events in interface "SessionEventHandler" are not requiring a gtkUI instance

I can't think of any event not needing the UI since the purpose of the event handler seems to be making the UI react to events.

For me, `guiSessionEventHandler` is simply `type guiSessionEventHandler struct { *gtkUI }` which will delegate everything to the wrapped object.


Wrapping the text here will make for more clear diffs and comment threads. Will work as it stands, though.


你有兴趣保持不断的更新  其他项目都停了好久了。。所以我选择你


This revision is clean.


There might have been a reason for this... oh yes it was because the `default` so to speak was an `''` at one point way-way back.

:+1: to this change, just note that it might be breaking in the event that there is a `baragon-agent` service acting as the default


So then this will soon impact `zfs-import` so it doesn't make sense to change the name.  OK, I can see that argument, then I'm OK with it as is.


@binky123
The executable is 1.2MB larger which is the size of the 4men tablebases.
It won't help in this TCEC season but some seasons don't use on disk tablebases and it would help in those.

@VoyagerOne
Thanks.  I think even more importantly than being separated it's not actually code but just data minimally wrapped in some braces for the compiler to be able to read and not intended for human/programmer consumption.  Since tb's are already perfect this data will not change and have to be modified/maintained in the future.  Still it is 1.2MB.   I also wish I knew if tablebases gain more or less elo at longer TC because by searching deeper you will access them from earlier phases of the game?  Maybe you are right and we should run sprt so at the very least we get the maintainers opinion on this which is always valuable and informative.


huh, interesting... did not know about that


In the text above, "Campuses" would be grammatically incorrect in prose (when a noun is used as an adjective, it normally takes singular form). This is a proper noun, so anything goes, but if would be cleaner with "Campus."


> "... from select Asian countries who have already completed ..."
That's saying that the countries themselves have completed a Masters, which I don't think is what you mean.


This is much clearer to me.  Let's go with it.


This was probably the root of our symbols to strings issues.

I don't think chef converts the hash specified here into it's own chef attributes that are symbol/string agnostic, instead it keeps the hash here as is.

Might be wrong but this is what I remember


:+1:  Yes, you definitely can go quite compact with Eigen I started noticing recently.
The one-liner surely is the best/most efficient one in this case.

Just a side node this context: if for some reason, you need a "y = T * x" or similar type of expression, where there's no risk of aliasing, you can specify that explicitely by, e.g., "y.noalias() = ...." to avoid an unnecessary copy in such cases (Eigen assumes risk of aliasing by default for matrix multiplication; as a safe guard I guess).  In your final one-liner, of course, you wouldn't want to do that (there it would effectively give rise to aliasing issues).


> "... provides enrolled students an opportunity ..."

Should that be "provides ... with ..."? I'm not sure.


I think it's reasonable to pull in even if it doesn't strictly address a bug.  I'm surprised we overlooked it the first time around actually.


And now we are back to using strings in this hash :+1:


:+1: :clap: this is a win


:+1: this was needed


It has worked thanks man .


This should be safe_delete_array(tmp);


Heh, was wondering whether you were going to revert this change


Validação de senha deve ser feita em tempo real, ou seja, não é um atributo que vai para banco de dados, logo, não é um atributo para uma classe e sim um atributo para um metodo de validação de senha. Por gentileza, verificar junto ao Product Owner, pois é um erro no escopo não na programação.


Why we still want to skip GKE here?


space between your print keyword and parens.  Watch those little flake8 errors!


If you'll notice the existing code is actually looking for each user folder. The method you are proposing only works for the current user (environment variables that point to the users' folders don't refer to each  individual user, also hkey_current_user only applies to the current user which the name implies). In many cases my use for the script actually involves logging out of whatever unprivileged user screwed up the machine and logging in with a hidden admin account and running Tron. So generally when I submitted those changes I tried to make them all apply to every user folder, rather than just the current user.


Embed the link. The site uses markdown (same as the GitHub comment engine), so the syntax is like this:
``` markdown
   [the project website](http://ascii.nagoya-u.ac.jp)
```
Also, the link should be HTTPS if that's supported by the site. If it's not, I can help with the application process - there is a university consortium for issuing SSL certificates, and the certs are free.


我摔，竟然是因为大写。。。妈妈我脑子被僵尸吃了。。。


In these cases, it's better to use `this.$el.find('.movie')` instead `$(.movie)`. The first one finds in the childs of the current rendered element instead of try to search in the DOM. It's faster, and isolates the module behavior. I'm mean that in a view you can change directly the elementes rendered, and indirectly trough child views if you have one.


The link gives me a 404. If the site is not yet available, when will it go up?


I think there is a difference between what's the default on the command line, and what is the default in the compiler.  Agreed the compiler is probably non-deterministic by default forever.  Command line though I think we can change.


Nice work! Everything works. If you have a chance to contact me by chat, I can explain another aproach to resolve the user experience. ;)


Cool. I didn't realise you could use [noalias()](http://eigen.tuxfamily.org/dox/group__TopicAliasing.html). I guess we should definitely start using this where possible.


Não utilizar caracteres especiais na criação do nome de variaveis, ou seja, no lugar de "endereço" utilize "endereco".


Se subió el commit al sitio en vivo pero el logo no aparece centrado, y debajo del menú hace falta un margen.
![cecac_live](https://cloud.githubusercontent.com/assets/10886131/10152557/fe28f488-6618-11e5-94eb-d8baca9f7d36.jpg)
Se usará la versión de http://mvp.oakandsand.com/felipe/cecac/


LowerCaseSymLink (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1896 of 1896 tests passed)

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


This gets undone by running `doc/genstdlib.jl`, needs to be fixed there too.


I think this would be jumbled into a single paragraph. Something like this will get you formatting (with a space above "Key information"):
``` markdown
**Key information**

- Successful applicants will be expected to commence the program in April 2016.
- Application documentation is available at the top right hand corner of this page
- The deadline for submitting the application is 18 December 2015.
- Please submit your application directly to your local Satellite Campus office.
- Your local representative will liaise directly with Nagoya University main campus on your behalf and will be able to respond to questions associated with admissions.
- Contact details of your local Satellite Campus Office are provided on the last page of the admissions documentation.
```


Good plan ;)


rax retry tests


I felt it was unnecessary. I may have been wrong


A better implementation of this change has been merged to upsteam livestreamer (latest version) you don't need my mod, it's already on the latest livestreamer.


TeamCity pull requests :: netty [Build 330](https://garage.netty.io/teamcity/viewLog.html?buildId=1811&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5619, ignored: 21 Build time: 00:38:04


Good use of SQL instead of iterating with Ruby


Well it did/does for 0.9.9.8. What version are you testing on?


Yup.


The addressed NullPointerException was fixed by other changes (a while ago).

@dosoft, can you close this pull request?


Merged.


Good use of strong params!


This conditional used to be (the equivalent of) `(*sle)[sfSequence] - 1 <= offerSequence`.  Did you lose the `-1` intentionally?


Excellent!


<img width="873" alt="screenshot 2015-09-28 20 56 29" src="https://cloud.githubusercontent.com/assets/2177366/10152749/775bdbb8-6623-11e5-889c-550bb3a5f280.png">


Why make this a non-native package? Will this be useful outside of Ubuntu?


:blush:


有什么想法都可以提。我觉得语言本身的差别并不会太影响使用，搞个python的又会有人希望来个ruby的，或者javascript的，或者java的，还是类似docker的概念只需要使用 不需要关心实现 会更好一些


Still needs to be set on the previous line, like so:
``` markdown
pagetitle: Transnational Doctoral Program for Leading Professionals in Asian Countries
```


Looks good to me.


more_doc_changes (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1360 of 1360 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-70.36%25-green.svg)


実は自分も2行書いてました！
glob形式でファイル指定しているので、
https://github.com/isaacs/node-glob
が参考になります。


Ahora se busca por CI


判断


Hi @StefanoRaggi, I was testing the StopLimit Orders with the bound set as the stop price. However, by setting the bounds as the same price as the stop price, often it resulted in order cancellations.


using `push.o` instead of `push.cpp` here seems to break the build for me, as it causes make to generate a generic c++ compile step, which fails with the following error:

    c++    -c -o push.o push.cpp
    push.cpp:16:10: fatal error: 'znc/znc.h' file not found

If I change that dependency back to `push.cpp`, then it works correctly again, and I get the appropriate version from `/msg *push version`.


That's good embedded-link syntax, but I now see that we were talking at cross purposes. It looks like you want the link itself to display in a block-indent. I'll take care of that with a separate commit.


The `name` attribute for `A` element is obsolete since HTML5. The `id` attribute should be used instead to create anchors. For example, instead of using an empty `A` element with a `name` attribute, add `id` attribute to a heading of the corresponding section:

FAQ.1: What do these guidelines aim to achieve?

Also, it is a good practice to use a prefix like `h-` for all anchor `id`'s to prevent collisions with `id`'s used for different (non-anchor) purposes.


Looks good.


these are cute!


good job


Proper fix for devastate looks like that:

--- a/src/game/SpellEffects.cpp
+++ b/src/game/SpellEffects.cpp
@@ -4318,6 +4318,28 @@ void Spell::EffectWeaponDmg(SpellEffectIndex eff_idx)
                     // 25 * stack
                     unitTarget->AddThreat(m_caster, 25.0f * sunder->GetStackAmount(), false, GetSpellSchoolMask(m_spellInfo), m_spellInfo);
                 }
+                if( !sunder || sunder->GetStackAmount() < sunder->GetSpellProto()->StackAmount ){
+
+                    // get highest rank of the Sunder Armor spell
+                    const PlayerSpellMap& sp_list = ((Player*)m_caster)->GetSpellMap();
+                    for (PlayerSpellMap::const_iterator itr = sp_list.begin(); itr != sp_list.end(); ++itr)
+                    {
+                        // only highest rank is shown in spell book, so simply check if shown in spell book
+                        if (!itr->second.active || itr->second.disabled || itr->second.state == PLAYERSPELL_REMOVED)
+                            continue;
+
+                        SpellEntry const *spellInfo = sSpellStore.LookupEntry(itr->first);
+                        if (!spellInfo)
+                            continue;
+
+                        if( spellInfo->IsFitToFamily(SPELLFAMILY_WARRIOR, UI64LIT(0x00004000))
+                            && spellInfo->Id != m_spellInfo->Id )
+                        {
+                            m_caster->CastSpell(unitTarget, spellInfo, true);
+                            break;
+                        }
+                    }
+                }
             }

Plus add one more fix, that triggered spells shouldn't eat power/rage/mana.


С этим исправлением если поменять 2 соседних элемента местами, то получится пересечение диапазонов lft rgt.
Если оставить как было, то подобный обмен пройдет при первом дейтвии (диапазоны станут друг за другом), но вернуть назад уже нод не получится (будет дубль rgt)


Ah, it seems my commit message has expanded $HOME to my (then) current home variable...


Não haviam commitado isso, por isso o app do Github do pessoal de Windows parou de funcionar


What is this for? The pre-written code above this already inserts a "0.5" smoothing.


I set it to `carmel` as a reasonable compromise between all of us. Please don't push this change +_+.


latest sync with fork master


Having a closer read of our OrderType implementation, I do tend to agree this to be the case.

I'll do some testing on this change.


Sorry. Bad title.
Take 1= S(0, 10)


getDay() already returns a String, so toString() is redundant


Will we want to put this in `fancy` instead?  This part is supposed to closely match the `baseline` one described in the project.


lol.


Consider constructing `expiration` closer to its first usage on line 185.


Yes you are right. This is not needed.


Sorry. I'll correct this.


It would probably be easier to move `initializeScreenshotSecurity()` that is currently in `PassphraseRequiredActionBarActivity` and move it to `BaseActionBarActivity`. Then we can just call that from `PassphraseActivity`.


@frenchy64 nice catch, would you mind committing this bugfix to master?


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


No. I just wanted to try what will be the result of this code on bengali.test file. The code shows a P/F/R of [1.0,0.0,0.0] for this since we do not have the correct values of the segmentation for the bengali.test file. We can only find out the f-score for that by submitting it on the handin portal.


Why is M3L trying to standardise power systems? Not all mods agree on this.


I think your discussion of content is really interesting. I know this isn't exactly the same thing that you are talking about, but it made me think about how the "content" of our CSS codes is behind-the-scenes, not even something the viewer would see or know was there. But it is still playing a big role in what they do see. I was thinking about the question we talked about at the beginning of this course: something like, "What is technical communication?" In terms of CSS and html, communication can be using a language readers/viewers probably don't know to depict an image or message via webpages.


I think this needs to go in customize_machine.rb


Why?


I think this needs to go in its own *.rb to avoid it needing to be duplicated here and customize_machine.rb.  Suggestions?


CW is a known bogus machine.


my bad!  this should be a whiel(true) with a break at the end, then.  i'll fix separately.. can drop it from here


Is there a reason to leave this as "TODO" rather than just plug that new line in now?


cool


What if someone raced us to create the partition? Since all we care about is files, probably best to leave empty directories, and update writing code to ignore empty directories


Do we really need to declare an extra pointer *bop here?Instead use bo directly by type casting to (drm_intel_bo *) ?
bo = (drm_intel_bo *) pOsInterface->pfnAllocUserptr(pOsInterface->pOsContext,"CM Buffer UP",(void *)(pParam->pData), tileformat,ROUND_UP_TO(iSize, GENOS_PAGE_SIZE),ROUND_UP_TO(iSize, GENOS_PAGE_SIZE),0);


Same comment about bop as mentioned above.


I think a better solution is in BaseActionBarActivity the initializeScreenshotSecurity always be called by the onResume or onCreate. Because i think there is a lot of sensible information in text secure.

I agree with this all project. And i will try to make a lot of contributions. =D


We don't even use this anymore anyway :(


Thx, I'll test it


LGTM other than the above inline comments.

Verified VPx use cases.


Should also be "in layman's terms" instead of "on layman's terms"


what is this change?


These need to be in decimal format to give you the right answers (0.6, 0.4).


Hmmm.  This looks funny.  If there's an `expiration` and we haven't passed it then we return `tesSUCCESS`?  And skip the call to `checkAcceptAsset()`?.  The only other call to `checkAcceptAsset()` I found in this file is on line 714 inside of an assert.  Something feels out-of-balance here...


@athanatos this probably shouldn't be a dynamic cast?


I would appreciate a short description of the change.

From what I can understand from reading the code: the claim is that we don't remove the clean-up callbacks and they close over the whole query object?


That's correct.


@jcsp yeah?


bad cells! bad cells!


JLOG?


A better way to do this might be:
```
#!/bin/bash
rcsoccersim
python main.py &

case $1
0) # run opponent 0 here
1) # run opponent 1 here
*) # run default opponent none?
```

On my machine running both `rcssserver` and `rcssmonitor` is not necessary, instead there is a script `rcsoccersim`. I haven't verified it works yet on the computes, but I check it out and get back to you.

I also think we need to find some opponents other than our own. A *REALLY* dumb one would be nice, maybe a static one and an intelligent one as well.

Let me know what you think


empd (nmerket) - Win64-Windows-7-VisualStudio-12: OK (1893 of 1897 tests passed)

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


Man this is a lot of replaces aye


:-1: :kiss: god damn! :snowflake: as :ice_cream:


This wasn't supposed to be in this commit. It's fixed in 26b43500.


haha


Fixed in the new Meteor 1.2.0.2.
Just run `meteor update` in the command line


@jdurgin ?


Coucou, tu veux voir mon commit ?


There are 3 cases on the GBC that should trigger an update for BG hashes:

1) The Tile Data VRAM is changed (0x8000 - 0x97FF in VRAM Bank 0).
2) The Tile Map/Map Attributes is changed (0x9800 - 0x9FFF, both banks).
3) The BG Color Palettes are changed.

For now, this case only deals with (1). Some graphics do not update the hash as a result.


Then, is it better to increment `total_count` only when the sequence character is G, C, A, or T?
And `not_gcat` can be removed.


Done.


Thanks. This fixes the issue jspm/systemjs. import now works as expected


Thanks for verifying.


Mmh, doesn't seem to work for embedding images.


I agree that a pre-determined format could be helpful, but I couldn't figure out from their website what the format was or which stylesheet went with that format.


First Commit


Excellent Job...


~Thank you


I will run a few more tests before this is confirmed.


This should be called `members` and should be an inherited property. There's no real value of having this property specific to this class.


Already translated


bug:player is going into the floor and giving index error.


Hi,

just some quick notes / suggestions ..

- Could the constants eg MAX_BUFFERS_BY_THREAD be configurable parameters ?

Apologies if I'm missing reading code (thinking aloud) but ...

- I haven't worked through the use case but  the usual gotcha with this type of caching (seen this a few times)  is a scenario where if you could exceed the cache size (eg > 16 in this case) but the 17th or greater entry gets back into the cache (17th does free first before one of the 16 not returned yet), releasing the old one. The issue is that garbage collectors (particularly the standard) expect most objects to die young and are optimized as such. So we generally  try to get objects to die quickly or not at all in edge cases I've seen similar algorithms make medium lived objects from short lived ones  (in extreme though very reproducible cases) i.e. helps to create objects that can't be collected by a minor GC and drive you towards a full GC (where they will get collected) which is normally a no no for a trading app as you get a long pause. The standard  java thread pools are good at this problem if you aren't careful with the parameters you pick.
(I'm not convinced my explanation :-) is very clear but this is a real world observed problem with some similar looking caches I've seen)

If you can configure the size of the buffers this shouldn't be much of an issue i.e. you can always try to pick a number bigger than you expect to use., ie alternative would be to track who the cached ones are.


This method as well as `current_size` method should have been inherited member functions. There should also be a method for adding new members to the `members` list as an inherited property as this should be done within the class.


this.IFRAME_ADDED_EVENT value contains '@' symbol.
Another contants doesn't contain it.


This logic should be moved to the constructor.


bug:game is giving jerks.it may irritate user.


Missing Module docstring


I think the notice deserves an aria label because the message itself is not explicit enough and, you are right, from this perspective the icon is important. So I will move the icon to its own DOM element and put the label on it it will make it simpler to translate :)


Based on fertobar@8874829


I think you followed the steps in https://github.com/fguillot/kanboard/blob/master/doc/ubuntu-installation.markdown

Then you have installed the latest version and this fix is already included.


I think there should be notification on http://framework.zend.com/manual/current/en/user-guide/skeleton-application.html page that skeleton application required php at least 5.5, because zend framework 2 requires php 5.3+


Avoid qualifying class selectors with an element.


Have you tested this?
For example, I believe that `sys.argv[i]` will always return a `string` which would break lines `17` and `30`


Selector should have depth of applicability no greater than 3, but was 4<br>Each selector in a comma sequence should be on its own line


Selector should have depth of applicability no greater than 3, but was 4


Each selector in a comma sequence should be on its own line


Self attribute defined outside __init__, bad practice.


Not quite an ideal variable name ("sync_model_ids"). Isn't it rather "properties"?

Moreover, why do you return the properties at all - if you just want to check whether the function succeeded? True/False would well do the trick?

And yet - _update_sync has only one exit, so it will always return True-ish here. It can not fail.


TItle:Rouge
Inspired by same name game on UNIX over 20 years ago.
Move rouge character by cursor in 5 min.
２０年以上前に有ったUNIXのローグをイメージして作りました。
カーソルキーで動かします。制限時間５分。
Sep/27/'15 Last Update


commit


可以增加


Merging for now, although I'm a bit torn whether or not to release this, as some people might have added this fix themselves.


Original script gives this error:
```
build: 4: build: [[: not found
```


bug: if player falls into a gap on the floor he is landing at wrong position i.e somewards backwards than required.


errhhm how do i use this?


Indeed; check.
But still the empty page (error 500)


From the sysadmin point of view, I think is better to use the package provided by the OS unless you need the capabilities of a newer version


Although if you need to update the comments field, of course you need to return the properties. Just, why call it sync_model_ids?

Also note that you're JSON-parsing/dumping forth and back all the time. It would be more effective to JSON-dump only once on write (at the point of write), and JSON-parse only once on read (at the point of read), and else pass the dict into subfunctions.


FYI This commit message could probably use a [ci skip] to avoid running tests on Travis CI (I often forget myself)


give atleast one white space after '#'.not following pep8 standards.


And please follow the CAF coding style. For example, `if`, `for`, etc. are followed by a whitespace:

`if (something)`, `for (auto& a: as)`, ...


Yes, you're right, thanks for the reminder.


Should JSON-parse right here, and only once during the entire process.


@julienw Given that I am going to add a DOM element for the icon I would stick to having a template, plus it follows the message status logic. But maybe you had something in mind... why would you get rid of the template ?
BTW Your explanation highlight the reason to use `setAttributes` rather than `formatValue`. I am going to make this change.


@sushanpoudel Like I said, please follow the build instructions in the README.md file.


is provided?


Should certainly not JSON-parse the same thing twice!


bug:jump is not implemented correctly.


needs?


@anrishitko


@jasnow I have the same issue, did you figure out how to apply the changes to jbuilder?


bad whitespace


@AnriShitko


Please use absolute imports, see https://github.com/c2corg/v6_api/pull/20


Single ` should be used instrad of three. Three are for code blocks. Single is for inline code.


Certainly not dumps here, and then loads again later in the function. This function should in fact not have any dumps/loads - but receive the properties as dict, and return them as dict.


Yeah, I'd question that too.


You readded this later, don't you?


Ah, makes sense. Those accidental committing buttons where indeed a problem!

How did you change this default, is it in a javascript?


mixed indentation.


Title: practice for javaScript by solving Eigen value problem.
This script is made by the Jacobi eigenvalue algorithm.

Sep/18/'15


This looks a lot better now :)


only thing is that there was already a PHP version active on this server.
Which is: PHP 5.5.9-ubuntu4.11 (cli) (built; Jul 2 2015 15:23:08)


Try to enable debugging and check the logs: https://github.com/fguillot/kanboard/blob/master/doc/config.markdown#enabledisable-debug-mode

Could please create a new issue, instead commenting this commit ;)


Title: Rotate a wire flame box by Euler  angle/オイラー角によるボックスの回転
Be able to change a view point by slider object on excel.
スライダーにより、視点の調整ができます。

Sep/17/'15 Upload


If this bug is fixed we can remove the xfail and leave the test


The following test will also pass, is it intended?
```
    fun testRange2() {
        val n = create {
            string("abc", operation = INTERNAL_TO_CLASS_ID, range = 2)
            string("d\$e\$f", operation = NONE, substringIndex = listOf(2, 5))
        }

        assertEquals("e.f", n.getString(1))
    }
```


Looks a bit odd to me in title case, it's not actually a title.


That's ok.


bug:player is not able to move down through broken ladders.


そうだった


Good!


Currently this tree is a work in progress. This include is for a deprecated api which isn't on the normal include path. I'd be happy to accept an improved "hack".

If you are interested if you build this kernel, it appears functional on 2709 for a while, but tends to panic in memcpy eventually. 2708 appears reliable but needs the usb fiq disabled on command line.


Und wie wär es mit Pages? evl mit filter collapsible set?


Tried with an even older ubuntu , with gnome 3.6.2 and still works
![screenshot from 2015-09-29 13-13-25](https://cloud.githubusercontent.com/assets/1508457/10161338/0f26df1e-66ac-11e5-87c6-8cf0d4362fdc.png)


Avoid using id selectors


K, PR with other small readme fixes coming


too many branches.


@julienw OK, Do you have an answer yet ?


Why it is needed to create empty constructor.
Why in actionbar u are adding view, directly we can share option icon.


I see.


We apply substring and replace char operations using record from types.getRecord(index), but here we are using records[index].operation, which could be from the record for some index1<index. Is is intended?


This produces the following warnings on Ubuntu LTS:

```
Error: COMPILER_WARNING:
src/global.c: scope_hint: In function ‘libssh2_init’
src/global.c:48:9: warning: implicit declaration of function ‘ENGINE_load_builtin_engines’ [-Wimplicit-function-declaration]
#   46|   {
#   47|       if (_libssh2_initialized == 0 && !(flags & LIBSSH2_INIT_NO_CRYPTO)) {
#   48|->         libssh2_crypto_init();
#   49|           _libssh2_init_aes_ctr();
#   50|       }

Error: COMPILER_WARNING:
src/global.c:48:9: warning: implicit declaration of function ‘ENGINE_register_all_complete’ [-Wimplicit-function-declaration]
#   46|   {
#   47|       if (_libssh2_initialized == 0 && !(flags & LIBSSH2_INIT_NO_CRYPTO)) {
#   48|->         libssh2_crypto_init();
#   49|           _libssh2_init_aes_ctr();
#   50|       }
```

Please also use a do { ... } while (0) wrapper for a compound statement.  Otherwise the macro definition is very fragile when used in the future!


player is just moving in one floor left and right. this does not meet even basic requirements.


Done.


Closes #5


It is the default “type” property. From “submit” to “button”.

I think we should mention it in the release notes.


from this line upto line 208 all are unused existing code that i missed to remove


I could imagine that Fran will complain - too much whitespace, and maybe even too much code to inject.

Just a tip.


I found out that --v=2 prints way to much data to be able to see whats happening inside. This is a test tool, so I believe we should have increased log verbosity.


bug:pixel arrangement is also bad.player cannot coincide with ladder.


global variable z undefined at module level.


# Makefile

main: commandlinereader.o par_shell.o main.o
	gcc -o main commandlinereader.o par_shell.o main.o

commandlinereader.o: commandlinereader.c commandlinereader.h
	gcc -Wall -g -c commandlinereader.c

parshell.o: par_shell.c commandlinereader.h par_shell.h
	gcc -Wall -g -c par_shell.c

main.o: main.c commandlinereader.h par_shell.h
	gcc -Wall -g -c main.c


So it was changed in the button scomp?  Makes sense.

We need to add this to the next release notes, so that people can easily hunt down problems.


You totally right. I just found weird (personal feeling) when we have both message status and late arrival notice that one icon is smaller than the other one.... but writing this, I am wondering if it is even possible to have message status and late arrival notice at the same time apart from within our tests ? Maybe not :)
BTW I will correct this.

![message-status lateness-notice](https://cloud.githubusercontent.com/assets/169818/10161407/3204e466-66a4-11e5-9d07-08063d7a5388.png)


Line too long . PEP8 standard not followed


Btw - if the id appears as s3-groupedopts-*, then you forgot to specify an id for the widget ;) It's the fallback, and way too generic to have events triggered on it.


quite a number of docstrings missing.


Thought I was losing my mind or had somehow corrupted my git repo! https://github.com/mxe/mxe/commit/c7cfb30513f8eef66bba0c7a26bf2c4bff296a04


There is release notes 0.13.4. Which one for the next release?


It's just one line, but I'd still like to put it in e.g., $(RIOTBASE)/cpu/Makefile.periph.common, and include that in the periph directories. What do you think?


This is not even  used in the game at all.


я имею ввиду List[CustomAttr]


I'm not sure this is the world's most useful `assert` :)


unused import.message module is not used in the game.


+1
I just tried cherry picking this commit into my branch.  Worked perfectly.


looks neat


Imanta izmaiņas


Awesome :) Really huge improvement for any who want to ass client library for gobgp :)


понятности это не добавит. почему бы не назвать ее "runPluginSymbolAttributes".


@julienw It is related to having both message status and late arrival notice side by side. If it can't happen then we can get rid of it. If it can, then the message status having a min-width of 3rem, it gives it spaces on both side of the message icon. So I remove the margin when they are side by side not to double the spaces between message and lateness icons.


также неясно почему blahSymbolAttribute а не blahCustomAttr, раз уж мы работаем с CustomAttr


: 21, 0: Found indentation with tabs instead of spaces (mixed-indentation)
C: 21, 0: Exactly one space required around assignment
					z[j][i]=' '
            ^ (bad-whitespace)
C: 22, 0: Trailing whitespace (trailing-whitespace)
W: 23, 0: Found indentation with tabs instead of spaces (mixed-indentation)
C: 23, 0: Unnecessary parens after 'if' keyword (superfluous-parens)
W: 24, 0: Found indentation with tabs instead of spaces (mixed-indentation)
C: 24, 0: Unnecessary parens after 'if' keyword (superfluous-parens)


Not made yet.  Though maybe we could add notes to the 0.13.5 milestone?  (can't think of another place at the moment)

https://github.com/zotonic/zotonic/milestones


@julienw  Love it :) Much more readable !


ladders are just printed on the output but they are useless in the game.


unable to provide conflict


Unused import random.


github edit


Thanks man!


Or name it “next”, or “upcoming”?


done


cool script


I like to name it by its numbers, then we can plan for the next N releases :)


there is a green block in the game.we cannot understand what it is.it is better use some symbol or picture that any user can easily understand.


why do we need to omit testutils?


Bad code smell : Reimported random.


constructive criticism


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 44](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=234&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 44](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=234&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **FAILURE**
Summary: Compilation error: Xbim.Exchanger\Xbim.Exchanger.csproj (new) Build time: 00:00:25


game seems to be stuck when player is just besides the donkey sometimes.donkey is unable to kill player.


that's my question with this PR, whether that counts as including test-only code in the production binary or whether we just don't care.


not used in the game at all and no white spaces around the operator.


Then: "0.13.5 (next)"


:skull: why :godmode: don't :hurtrealbad: you :rage2: use :rage4: zsh :goberserk: ?!?! :bomb:


The reason we blacklist "testing" is not because it's test-only code, but because it registers flags which pollute the global flags list. Including (some) test-only code is fine.


Administrator Guide/Export Netgroup Authentication.md

Typo -
This information is stored
based on those files.


Made an new issue; thanks for the comment so far !

Frans


You're right, this is a mistake, I forgot to use `records[index]` everywhere. Thanks!


Invalid name 'a'. Uncommunicative.


See #36


@RJP43 I see you were going to post this in markdown as .md, but your computer added a .txt extension anyway! There's a way to stop your system from adding hidden file extensions--check (I think) in your file explorer...You can also set it so you always see hidden extensions. Within the software you're using to generate the file, try Save as and select type as "any" instead of text.


Not intended and I think it's a consequence of the bug you mentioned. Will add this test


It may be useful to provide entire command


All this code ( 160 - 252 ) belongs to movement of player . so it must be implemented as a method in Players class . So that it can be checked and maintained  better.


Give a white space after '#' in comments.not following pep8 standards.


Previously there was `options.data` as the fifth argument, now it's requestOptions. Is this correct?


I hope not :smile: Unless the thread performing the require is killed it should always progress to success or a failure. What did you have in mind?


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 45](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=235&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


@kboyarshinov Original u2020 has the same comment in code. I think switch from imgur to github API should be done within a nother problem.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 45](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=235&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:43


I am not sure if T.S.P is well known abbreviation. It may be good to expand it, at least during it first occurrence.


First of all I did not understand what are you sharing from Contact detail activity.
Please explain if you both have any understanding on this.

I don't see this changes related to task description.["Invite contact" and "share items"]

I do not understand why seprate xml file is needed to attach one image view in any layout.

What empty constructor is doing here?? any idea??

In my life first time I have seen constructor for an activity, so I would love to know the purpose.


because zsh is not installed by default on Mac OS X :)


You forgot to update your N, silly brits


Actually, they need to be added to the *upgrade notes*.

http://zotonic.com/docs/0.13/manuals/upgrading.html


Namespaces es noteikti atstāšu lowercase, lai ir case-insensitive. Js - mani iniciāļi.


"withan" -> "with an"

What option are valid? Really these too "sec", "anonuid"?


ОК, поменял на runPluginCustomAttributes


ОК, поправил, спасибо!


Not the way to fix it. Proper fix is

~~~
if [[ $(/sbin/service ${SVC} start)$? -ne 0 ]]
~~~


closes #13


Don't forget the dot. Put it in.


謝謝老師, 我剛剛才看到, 會好好的study一下


There are \n listings here, that is not correct.

A real example of a netgroup with hostnames and IP addresses would help.


@Taluu could you please create a new issue? Otherwise your comment will be forgotten.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 735](https://teamcity.labkey.org/viewLog.html?buildId=274862&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **SUCCESS**
Summary: Tests passed: 9, muted: 34 Build time: 00:25:22


same issue at my custom board (AT91SAM9M10G45 + Micron MT29F2G16ABBEAHC-IT)-> after fix from Justin On-Die ECC working!


do we need to use [https://github.com/jaredhanson/passport-oauth2-client-password/issues/8](https://github.com/jaredhanson/passport-oauth2-client-password/issues/8)
 passport-anonymous


Great!


Upvote this! (see https://github.com/cargomedia/puppet-packages/issues/937)

We run into issues for using stock Debian Wheezy ruby (v1.9.3)

May be `~>2.8` would do the job? For us, it solves the problem


I mean line 20, of course!


Hmm.. tad sanāk, ka tu izmanto PSR-4, bet tajā pašā laikā ej pret tā [specifikāciju](http://www.php-fig.org/psr/psr-4/).

    6. All class names MUST be referenced in a case-sensitive fashion.


Please spell out T.S.P. You an mark `filenames` and `commands` with backticks for giving them better formatting.


Should be canola


Tas ir tikai par class names...


The MarkDown format for numbered list only have "1." as prefix, adding the ")" breaks the formatting.


Should we instead define a macro in some common header:

```c
#define UNUSED(p) (void)p;
```

and use it as:

```c
LockingCallback(int mode, int n, UNUSED const char* file, UNUSED int line)
{
```


Ack, although we should have noted (in the previous commit e88a085a932b16718d879c30db480203c755aad9) that the flipping of the option back to False was just a merge regression introduced by the sysroot backport in commit cb3d265849771f7e53b0587196930328005414e0:

```
commit cb3d265849771f7e53b0587196930328005414e0
Author: Bryn M. Reeves <bmr@redhat.com>
Date:   Mon Jan 19 18:54:09 2015 +0000

    [sosreport] add --sysroot option

    Add a --sysroot=SYSROOT option to specify that the root file system to
    be inspected is mounted at SYSROOT.

    This allows basic support for container environments where sos is
    running in a container and inspecting the containing host and its
    environment ('superspection').

    For this to work currently the following conditions must be met:

    - sos is sufficiently privileged to read and search relevant file
      system paths within SYSROOT

    - sos must share the PID and network namespace of the target host

    - binaries called by sos must be present and executable in the
      SYSROOT inherited by sos. If PATH includes paths inside SYSROOT
      appropriate values must be set for LD_LIBRARY_PATH to allow
      shared executables to be linked.

    Signed-off-by: Bryn M. Reeves <bmr@redhat.com>

[...]
-                          help="Disable HTML/XML reporting", default=True)
+                          help="Disable HTML/XML reporting", default=False)
+        parser.add_option("-s", "--sysroot", action="store", dest="sysroot",
+                          help="system root directory path (default='/')",
+                          default="/")
```

The cleanup is obviously good-to-have but we should take care to document regressions when we fix them (it helps people trying to track down already fixed bugs in old releases).


You are right. But we should also include changes to minor/bug releases, not just "when upgrading to a new major Zotonic version"


mbed?


No tās pašas specifikācijas:
2.2. The contiguous sub-namespace names after the "namespace prefix" correspond to a subdirectory within a "base directory", in which the namespace separators represent directory separators. **The subdirectory name MUST match the case of the sub-namespace names.**


well, nothing done right now but I'll try to do it :)

There's also a riscv-lecture based on your arm-lecture, you can find it here:
https://github.com/coldnew/riscv-lecture


Hi Shankar.
Can you open a new issue with your question.
Thanks.


Reviewed by Dietmar Zabel.


True.  But a breaking change like this should have not been committed in the 0.13 branch in the first place, in my opinion.


[Another good reason](http://stackoverflow.com/a/24496138).

Bet, jebkurā gadījumā, it all boils down to you and if you want this. Es savam kodam pieeju ļoti sīkumaini un izmantoju vai nu visur lowercase vai camelcase. Abus divus nemiksēju kopā. Šajā gadījumā class names - CamelCase, bet namespaces - lowercase.


Proper declarative format to set default values on positional-argvs (as in lines 25-26):

~~~
VARNAME="${<POSITIONAL>:-<DEFAULT>}"
~~~

In general, the recommended method for signifying vars is to use curly-braces. Some shell interpreters can be more or less pedantic about this.


Missing a slash here? `Cannot find module '.responses/spotify'`


This is not used in the game.so this is not required.


Could you please change it back if it is possible? AWS RedHat instances doesn't have preinstalled syslog user. And there is an error during chef execution.


We need try to use by putParcelableArrayListExtra method.


Tas ir tas pats "class names", neskaitās. Namespace != class name.

Es vienmēr visus folderus saucu lowercase only. Tā, kā šie namespaces potenciāli var būt folderi projektiem, kuri neizmanto composer, tad namespaces = lowercase.


I think it would make more sense to store the number of days worked per week (i.e. `response.split(',').size`) rather than the raw response here. If we did actually want just to store the response, I think we could use a `save_input_as` call instead of `next_node_calculation`.


I'd probably delegate to the instance variable directly and avoid exposing the flow as an accessor.


Please note I've slighty improved the Work Table nodes registration since then (you don't have to specify the mod name for each node).


Shouldn't just
addLocale(locale.trim());
be used here?


1次元配列を渡さないと動かないっぽい・・・。


The query of the bitmaps activated twice. We need pass object with bitmap between activities


Also true. I am used to make changes in 0.13 and then cherry-pick for master. In this case it should have been the other way around.
I am happy to revert the changes.


Yeap. I started from another LPC board (https://github.com/RIOT-OS/RIOT/tree/master/boards/mbed_lpc1768). I didn't write a new script to flash the MCU.
It's possible that other refs to this boards or to lpc1768 are visible elsewhere. I need to cleanup the code


Ultimately it would be better if the `OutcomeDecision` class didn't do any calculations itself, but just selected the appropriate outcome based on "predicate" methods on the calculator. However, that's probably a job for another day. This ought to mean that its only dependency would be the calculator and not any of the other "state variables".


Too many parameters. Can use an object instead of 8 parameters.


PHPCS: Opening parenthesis of a multi-line function call must be the last content on the line


PHPCS: Only one argument is allowed per line in a multi-line function call


I think I'd be tempted to add unit tests around the `OutcomeDecision` class at this point.


PHPCS: Multi-line function call not indented correctly; expected 12 spaces but found 16


PHPCS: Closing parenthesis of a multi-line function call must be on a line by itself


those CSS are not DRY @markphi2013, can you make it DRY?


TeamCity pull requests :: netty [Build 331](https://garage.netty.io/teamcity/viewLog.html?buildId=1818&buildTypeId=pulls_netty&guest=1) is now running


Фиксы тут: https://github.com/scala/scala/compare/2.11.x...VladimirNik:backendPlugin


Whats this?


This is a leftover...


extra blank lines..


Do we really need to conditionalise this? What harm is there in just trying the command?


wrong?


I've updated the patch according to your suggestions.


(!) There is a numerical error with TSE factor values above approx. 120! Flip angle trains and MTFs are computed incorrectly for these cases...


> An organization is a good idea if someone are able and willing to dedicate the time necessary. We will
> continue to support and contribute to any official repository that will keep this project a general
> implementation of ISO8583. On request, we will also take on the role as maintainers.

So I think we all agree on the issue of having an organisation. I feel that especially because any potential user would have a strong commercial interest, the library shouldn't be seen to be controlled by any one party.

## Testing

Concerning testing, this has been a major deal breaker for me in the past. NO ONE has submitted any significant PR concerning tests. For the use cases of this library, I feel providing robustly tested code should be the number one concern and should be high on the road map going forward.

## Merging

I'm looking forward to the new features and doc from @MaG21. Let me know if you need any help I have a bit of spare time on my hands this week. The encoding issues should be fairly simple to sort out, basically they entail some index acess to strings `[x,len]` needing to be replaced by `byteslice` and  judicious use of `force_encoding`. Once we're merged it would be great if @ta could evaluate to check that it's backwards compatible.

Once we're through, I'll set up an organisation, transfer ownership and add everyone involved with full access. I think the group of interested parties is small enough to reach consensus informally. IMO future membership should be open to anyone who makes a substantial PR.


If the [checkstyle maven plugin's execution configuration](https://github.com/MarginallyClever/Makelangelo/blob/cd7bb4aa8d302c934ed008db5587d13a5954efaa/java/pom.xml#L86-L99) causes an error on your system, it most likely means the code is not conforming to the [checkstyle configuration](https://github.com/MarginallyClever/Makelangelo/blob/cd7bb4aa8d302c934ed008db5587d13a5954efaa/java/src/checkstyle/checkstyle.xml).

If you are talking about Eclipse complaining about something like "no plugin for lifecycle" or something like that, there is a Checkstyle [m2e](http://www.eclipse.org/m2e/) adapter for Eclipse. If you do a "Quick Fix" in Eclipse it will look for m2e adapters and download the Checkstyle one.

Rather than comment it out:

1. Adapt the Checkstyle configuration to adhere to your coding style
2. or get it to work in Eclipse if that was the issue, and see what coding styles are configured. Repeat step 1.

If you are not going to use Checkstyle, then I will remove it. Having it forces a coding style, which in turn makes for more clean, readable code; barring the code style you use is sensible...


@heilage-nsk Bad things happens )))


Nope, don't have it.


You could to the `progress()` right here?


Yes, `options` is undefined here.


This doesn't need to be a list - the `add_copy_spec()` call that you pass it to accepts either a single string, or a list of strings.


Uncommunicative names.


On the other hand v=4 produces logs that are completely unreadable and there is way too much of them.
I would suggest using 2 (but I'm not going to block this PR on it).


Why does this need to be a plugin option? We don't generally provide a mechanism for disabling scrubbing (yet: it has been discussed and the consensus was that we should have a global mechanism for it).


I don't get your comment: in both the old code and the new one, nexthop is a NextHop instance.


We don't generally hide usernames (as they are considered less sensitive). Is there a particular need to for monit?


1. Files rearranged to make them easier to navigate through.
2. DUK transitions made smoother.
3. Sign up integrated
4. DUK responsive


hi shiva


Too many branching.


Glad to help


Shouldn't this be `%LOCALAPPDATA%`?


I think it would be clearer to do the conversion from `calculate` to `next_node_calculation` blocks in a separate earlier commit.


I think it would be clearer to make any change to the "layout" of the parameters in a separate commit.


'Dom' -> 'dom'


You are not funny


Redefined 'x' and 'y'.


Typically, put a ~ (tie) before citation commands to avoid awkward spacing problems:
http://tex.stackexchange.com/questions/29840/consistent-typography


Reviewed by Dietmar Zabel.


Oversight from me.
I wrongly though there was already a proper repr in the INET superclass.


The support of multiple FTS servers is still not implemented (https://github.com/dmwm/AsyncStageout/issues/4371). Usually the transfers that are facing channel overload are less than 1-2% of the whole transfers. So I would consider it as fatal now and once the support of multiple FTS3 servers is implemented
we can submit to another server on the next retry.

About the second, FTS3 does not do any wait/retry. The default number of ASO retries is 3 (the default cooloff is 2 hours). I would retry for this specific error only 2 times and then give up. How sounds?


Oh yeah, probably should have mentioned to only use the last bit of the path :cold_sweat:, looks like you caught it though


Забыл убрать название конкретной таблицы


Thanks!  I will definitely update to that.


Unused variable 'k'.


There's no mention of why we need to add this `precalculate` block in the commit note, but it looks as if it's because it was being calculated in the final question which has now been moved earlier in the flow and so this block wouldn't have access to the variables it needs. In any case it makes sense for values used in an outcome to be in `precalculate` blocks. However, I think we could make this change as a separate commit.


You are right, I wrongly though this was bagpipe specific and that in the exabgp case the superclass eq was good.
I'll submit a patch without this change.


We are never suffixing interfaces with the "Interface" suffix. So it would be strange to name this interface different than the other interfaces. Rename the project class to something different could be a solution for the alias issue.

The mean reason why we have an interface for projects this that we want to be able to mock the project. But we wanted to have all classes final. I disagree with you that an interface with only getName isn't very usefull from that point of view. It could even have been empty.


Why don't you use a String(Buffer/Builder) for `retval` ?


I don't know it and probably won't use it.  If it was working automatic in the background I'd probably leave it on.


No need for this, just use PacketReader.ReadString() straight, it will handle errors and stuff :]


Shouldn't this be AllianceSearchResponsePacket? Just keep the pattern going.


Attributes should be defined inside  __init__. 'wall', 'dt' , 'height', 'size' are defined outside.


This one I'm not sure. Unlike bagpipe, ExaBGP does not make use of hash on NLRI objects, and the implementation of hash is not consistent in the various subclasses of NLRI.  Wouldn't it be more consistent I think to not define it anywhere until needed...?


Same here use PacketReader.ReadString().


I saw that working earlier in the code, so of course I tried it first. It didn't work for this case. Didn't feel like messing with it further. Redirecting output inside the test to `/dev/null` also worked, i.e. `if [[ $(/sbin/service ${SVC} start > /dev/null) -ne 0 ]]`, but I wanted the output so cfn-init could potentially catch it.


In our case, in addition to subsumption, there's also var unpacking that can change the type.


You don't need that with the xdecor's API. Just add :

`infotext = "Barrel",`

Example : https://github.com/kilbith/xdecor/blob/master/nodes.lua#L31


This is the code generated by `javacc`. All I did was to simplify it by
removing the section that generates all possible choices at the error
point.


Does it make sense to do a local copy of command->attitude->quat ?


Avoiding those random newline would great too xD


Easy enough. I figured user auth is the most likely bit of this to change drastically from here, too.

How can we better escape the password for the printf statement? I'd like it to not get hung up on special characters...


@CodeClubRik: I think this rolls back an important change.


should this not also app_name now to be consistent?


saw approval from hmmr
at https://github.com/basho/riak_kv/commit/46ee4834ef6b52373a5021787f5d7059a1386059


merging basho/riak_kv/rts348_console_errors_3 = 46ee4834 into borshop-integration-1195-rts348_console_errors_3


basho/riak_kv/rts348_console_errors_3 = 46ee4834 merged ok, testing candidate = 27301643


Такие комментарии либо уберают, либо пишут реального автора, поковыряй ide


We have a similar method in golo/compiler/utils/StringUnescaping.java (which does not seems to be used...)
Maybe we could move this code there.


Probably we can have a look at the QX11Info later on. Right now it would be great to get a new version ready.


The `UIP_CONF_IPV6` is deprecated and should be replaced by `NETSTACK_CONF_WITH_IPV6`


If you don't actually plan to use the `header-style` class and are just using it as an internal class to be extended, then you should make it a _placeholder_. Placeholders use a `%` instead of a `.` to denote the class, but unlike a regular class that is extended, placeholders are not outputted.

Also, you have to be careful with extend because it changes the source order of the output, which in turn affects your cascade.

References:
http://thesassway.com/intermediate/understanding-placeholder-selectors
https://css-tricks.com/the-extend-concept/


Переименуй в `settings`


Такой путь крайне странный... лучше его передавать.


Why did you remove const here ?


Я не вижу чтобы он кидал такую ошибку...


OK that was stupid from my side I just need a 32-bit hash. re-adding pcgr

On Tue, Sep 29, 2015 at 12:26 AM Damian Gryski <notifications@github.com>
wrote:

> Hrm.. all the strong pcgr with 64-bit ouputs require 128-bit math, which
> will slow them down since it has to be emulated in Go. Second, calling the
> 32-bit generator twice brings the speed in line with the xorshift +
> interface call. Oh well.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/seiflotfy/count-min-log/commit/2e7f2fb42df58557ba8fea561d5ce1860496c335#commitcomment-13480414>
> .
>


@pablius @aschelstraete Avisen si necesitan algo más


Thanks for pointing it out, filed #1286.


Euw... A dump to `/dev/null` is seriously twitch-inducing if all you're seeking to collect is an exit status. I mean, that's twelve typed-characters versus two!


@prudhomm in markedfaces, use dir.meshMarkers() instead of dir.marker()


Works like a charm.
http://code.dlang.org/packages/scod


Все пути должны передаваться


I made my own port for personal use, is includes the changes required to ommit `gui-private`: https://github.com/MOZGIII/QxtGlobalShortcut
I have tested in with linux and windows.


Šeit ir interesanta diskusija par tēmu: https://groups.google.com/forum/#!topic/php-fig/VA0ZVUjCH-Q


Мне кажется, что Web Analizer не должен заниматься сохранением данных куда-либо. Это не его функция.
Этим должен заниматься какой-то другой класс. который будет передан к примеру с помощью интерфейса в конструктор.


I would not require such a new version of Catmandu. Better: `">=0.9, !=0.9402"`


Heh! I had just opened that page a few seconds before the GitHub notification came in. Scod looks really nice! Unfortunately I didn't have the time to make the default theme pretty, yet.


No Just use the same retry policy as for any other error.
Simplicity is worth more then subtle optimizations

On 09/29/2015 01:19 PM, HassenRiahi wrote:
> About the second, FTS3 does not do any wait/retry. The default number of ASO retries is 3 (the
> default cooloff is 2 hours). I would retry for this specific error only 2 times and then give up.


Любые повторяющиеся текстовые конструкции должны быть константами.


@ttuegel, please do not add warning messages such as this one. That message is displayed every time I run `nix-env -qa` or `nix-env -u` regardless of whether I actually use the `networkmanagement` attribute or not. This is very annoying.


Перенеси `AutofacConfig` в `Devmon.Server.Collector.App`


`certian` => `certain`


License should be updated properly.


Учитывая что они повторяются не в одном методе, а в нескольких - константами класса.


PHPCS: Opening parenthesis of a multi-line function call must be the last content on the line


PHPCS: Only one argument is allowed per line in a multi-line function call


PHPCS: Multi-line function call not indented correctly; expected 12 spaces but found 16


PHPCS: Closing parenthesis of a multi-line function call must be on a line by itself


Great :smile:


kunt ge output-format niet default op yaml zetten? Dien xliff is echt nie leesbaar.


Tack för stödet i kampen mot de stora företagen kamrat ☭☭☭


`this.util` -> `this.utils`


Hi,

I have one more question, if I disconnect one of the devices i.e remove the power supply
then the PLINK_CLOSE frame obviously is not transmitted, in that case how will the other
device detect this situaion.
the "iw mesh station dump" of the other device still shows plink_state as established.

On Mon, 28 Sep 2015 23:55:01 -0700, Chun-Yeow wrote
> You need this when building the wpa_supplicant.
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/cozybit/wpa_supplicant/commit/02bf5235c07168053e357d356f2c363bb7f4e2d
f#commitcomment-13485131

Disclaimer:
----------
This email and any files transmitted with it are confidential and intended solely for
the use of the individual or entity to whom they are addressed.  If you are not the
intended recipient you are notified that disclosing, copying, distributing or taking any
action in reliance on the contents of this information is strictly prohibited. The
sender does not accept liability for any errors or omissions in the contents of this
message, which arise as a result of e-mail transmission.

--
Open WebMail Project (http://openwebmail.org)


Agro works like it should.
About triggered spells i don't find commit where i fix it to show patch, but you can do it yourself. File spell.cpp function void Spell::prepare(SpellCastTargets const* targets, Aura* triggeredByAura)
This place:
// Fill cost data
m_powerCost = CalculatePowerCost(m_spellInfo, m_caster, this, m_CastItem);
Should be changed to:
// Fill cost data
// Triggered spells not eat power ;)
if( !m_IsTriggeredSpell ){
  m_powerCost = CalculatePowerCost(m_spellInfo, m_caster, this, m_CastItem);
}
Thats it.


sgamato lol


Method has too many lines. [16/10]


nice for loop :-)


Либо должна возвращаться структура данных...которая будет представлять из себя описание этих страниц и какой-то другой класс должен мочь их записывать в файловую систему. (Этот вариант правильней)


License should be updated


@vignesh-s  Remove it then!


We are using hex for packet ids.
Maybe you could add the packet id in decimal at the end like this
"LoginRequestPacket - 0x2775 (10101)"
instead.


just for test


Hi Patrick! How about merging these changes to [official pywinauto repo](https://github.com/pywinauto/pywinauto)? I thought about manual merge, but you may create a pull request to show your contribution.


`Hammerhead` should be lowercased since it's not a ctor. But this issue will be auto-resolved of we'll fix #18 in context of this PR.


no - as the default in the switchcase and all our others default to a return


iuff


**Functionality - Nitpick**
You changed the structure of the _static property_, from the requirements, `$data` was to be as follows:

```php
[
  "slang" => "init",
  "description" => "Lazy way of saying isn't it",
  "sample-sentence" => "That movie was awesome, init?!"
]
```


Переименуй класс и реализацию в `(I)SettingsReader`. Класс с настройками переименуй в `Settings`


Please write down meaningful and useful GIT commit messages: https://robots.thoughtbot.com/5-useful-tips-for-a-better-commit-message


Why do you commit this? Issue+fix should be the way to handle it. RC_1.4.0 should be treated like master.


#include "stdafx.h"
#include <iostream>
#include <string>
#include <ctime>
using namespace std;

void chuvan()
{
	string a, d;
	cout << "Rock Paper Scissors" << endl;
	cin >> a;
	srand(time(0));
	int b = 1 + rand() % 3;
	if ((b == 1) && (a == "Rock"))
	{
		cout << "Rock" << endl;
		cout << "\tDraw!" << endl;
	}
	if ((b == 1) && (a == "Paper"))
	{
		cout << "Rock" << endl;
		cout << "\tYou win!" << endl;
	}
	if ((b == 1) && (a == "Scissors"))
	{
		cout << "Rock" << endl;
		cout << "\tYou lose!" << endl;
	}
	if ((b == 2) && (a == "Rock"))
	{
		cout << "Paper" << endl;
		cout << "\tYou lose!" << endl;
	}
	if ((b == 2) && (a == "Paper"))
	{
		cout << "Paper" << endl;
		cout << "\tDraw!" << endl;
	}
	if ((b == 2) && (a == "Scissors"))
	{
		cout << "Paper" << endl;
		cout << "\tYou win!" << endl;
	}
	if ((b == 3) && (a == "Rock"))
	{
		cout << "Scissors" << endl;
		cout << "\tYou win!" << endl;
	}
	if ((b == 3) && (a == "Paper"))
	{
		cout << "Scissors" << endl;
		cout << "\tYou lose!" << endl;
	}
	if ((b == 3) && (a == "Scissors"))
	{
		cout << "Scissors" << endl;
		cout << "\tDraw!" << endl;
	}
}

int main()
{
	while (5 > 4)
	{
		chuvan();
		cout << endl;
	}
	system("Pause");
	return 0;
}


Ah ok, didn't thought it would be that easy. So you think you can make a pull request for launchy?


Переименуй в `Read`


I'm glad that you separated this out, but this should go before the `style.css` file; this is the equivalent of putting it at the top of your CSS file. Make sense?


TeamCity pull requests :: netty [Build 331](https://garage.netty.io/teamcity/viewLog.html?buildId=1818&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5604, ignored: 21 Build time: 00:37:37


Reverted on 0.13


What's the reasoning behind this removal? Is there a replacement?


">=0.9, !=0.9402, !=0.9401"


Well, I think that the code of EmissionVetoHook1 will be removed soon, so I did not do all these changes, just used "using ..." statements from the pythia8 headers (they list the needed std classes explicitly starting from version 212).


**Functionality / Testing - Critical**
How does this work when class `UrbanDictionary` does not have a _constructor_?


Подобные if можно инвертировать. Тем самым уменьшить вложенность.
К примеру, есть такой код
if ( a == 1) {
   if ( a == 12 ) {
       if ( getSomething(a) = 20) {
.......

Их можно записывать наоборот
if ( a != 1) {
   return ...
}
if ( a != 12) {
  return ...
}
.....

Пример утрированный.


этого if'a тоже касается.


Pretty sure we dont need a -1 here.


Method BlogControllerTest::testAdministratorUsersCanAccessToTheBackend does not have a Docblock comment.


We can also rename `gtkUI` to `gtkController` if that feels better. `gtkUI` is managing the configuration and session models (which can also be renamed it that feels better).


@garaemon `jsk-ros-pkg/.*`という名前のレポジトリに関してだけマージを求める通知が行くようにしました．


never use root, even in testing, if someone gets root access through vulnerability, you're screwed!


It seems in all usages of this method we have bytes = serializer.serialize(<some MessageLite instance>).
For example
```
AsmUtil.writeAnnotationData(av, serializer, serializer.serialize(packageProto))
AsmUtil.writeAnnotationData(av, serializer, serializer.serialize(callableProto))
writeAnnotationData(av, serializer, serializer.serialize(classProto))
```
I suggest using something like the following:
```
    public static void writeAnnotationData(@NotNull AnnotationVisitor av, @NotNull DescriptorSerializer serializer, @NotNull MessageLite messageLite) {
        byte[] bytes = serializer.serialize(messageLite);
        ....
```


@peti I removed it. What is the correct way to deprecate packages?


!$(tr).closest('table').has('td, th')


Always set a password, try to get it from a secure source (not human, not predictable).


start-jsk系もあったらいいかな。


@a2800276: I agree that we've never looked at testing as a whole. However, every commit with functional changes has been accompanied by corresponding changes or additions in test library. I feel it is important to clarify that :)

> Once we're through, I'll set up an organisation, transfer ownership and add everyone involved with full access. I think the group of interested parties is small enough to reach consensus informally. IMO future membership should be open to anyone who makes a substantial PR.

:+1:


ip


Ok I will change that


regenerat session id on every request, helps prevent session hijacking, maybe also always store some server vars in the session and verify, or destroy session.

http://php.net/manual/en/function.session-regenerate-id.php


This isn't very dry; take a look at your final output and you can see what I mean.

```css
...

.info-01:last-child {
  margin-right: 0;
}

...

.info-02:last-child {
  margin-right: 0;
}

...

.info-03:last-child {
  margin-right: 0;
}
```

I think you intended on the last child of the parent `.info` element to have that styling, ie. just the last child. Update your SCSS to the following:

**line 117**
```scss
.info {
  @include cf;
  height: auto;
  background-color: #E6D4B0;
  &:last-child {
    margin-right: 0;
  }
}
```
and remove this from your mixin, its unnecessary.


What about 2.1.4?


here too I guess ?


On Sep 29, 2015 1:21 PM, "Jaap van Otterdijk" <notifications@github.com>
wrote:
>
> We are never suffixing interfaces with the "Interface" suffix. So it
would be strange to name this interface different than the other interfaces.

Ok if it works for you :-)

>
> The mean reason why we have an interface for projects this that we want
to be able to mock the project. But we wanted to have all classes final. I
disagree with you that an interface with only getName isn't very usefull
from that point of view. It could even have been empty.

The point of an interface is to force every implementation (incl the mock)
to implement all the interface methods. So dependent code can depend on the
interface instead of a class, and still be sure all the methods are there.
Liskov substitution, and depending on abstractions.

And it is good for the IDE.

And I like final classes :-)

>
> —
> Reply to this email directly or view it on GitHub.


يبلريل


heh. it was general advice)


you're trying to call an ERB tag within an ERB tag. try this:

`<%= image_tag("http://img.pokemondb.net/sprites/black-white/normal/" + pokemon.name.tr('[]', '').gsub(/[""]/, '') + ".png") %>


maybe relocate session logic into another file so that it remains consistent throughout project. Eventually separate this from the project so you can re-use, but for now, even having session logic in separate file is smarter work-flow. Also do it before you output any content, and try not to have it in presentation / view files.


Ah, nested labeled lists! I didn't about that.


this looks to be done already?