(0996fdf) The title of the commit have a non authorized type.
 The allowed types are :
 * feat
 * fix
 * docs
 * style
 * refactor
 * test
 * chore
 * version


Découpe avec appels via API


saw approval from Manishearth
at https://github.com/Ms2ger/servo/commit/5eef78c6d85d0984d9516009e44f7b180e704f0d


merging Ms2ger/servo/update-wptrunner = 5eef78c6 into auto


Ms2ger/servo/update-wptrunner = 5eef78c6 merged ok, testing candidate = 6b860056


This profile is a mix-up of manjaroiso and manjaro-tools. Please rebase it.


In our teams we follow http://semver.org/ for bundles/components which we are using in our projects. livingdocs-engine is exactly that for livingdocs-editor or livingdocs-server.

This are the summarized guidelines for versions:

**Version format**
X.Y.Z (Major.Minor.Patch)

**Rules**
Patch version Z (x.y.Z | x > 0) MUST be incremented if
* Only backwards compatible bug fixes are introduced. A bug fix is defined as an internal change that fixes incorrect behavior.

Minor version Y (x.Y.z | x > 0) MUST be incremented if
* new, backwards compatible functionality is introduced to the public API.
* It MUST be incremented if any public API functionality is marked as deprecated.
* It MAY be incremented if substantial new functionality or improvements are introduced within the private code.
* It MAY include patch level changes. Patch version MUST be reset to 0 when minor version is incremented.

Major version X (X.y.z | X > 0) MUST be incremented if
* Any backwards incompatible changes are introduced to the public API.
* It MAY include minor and patch level changes.

Patch and minor version MUST be reset to 0 when major version is incremented.

In my eyes, after every patch you should set a new patch version f.e. 0.4.17 and create a new tag so in the package.json you can just define `livingdocs-engine: 0.4.*` - even better `0.*` which should be backwards compatible.


k thank you philm


I don't know if works or not... I implemented this because I encounter this
how a help for Picasso..
On Jan 14, 2015 9:26 AM, "Tim Bremer" <notifications@github.com> wrote:

> This thing caches the images?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Supremez/Dashboard/commit/7cd85d1d80888101904cd9e042979f2c4041d354#commitcomment-9274173>
> .
>


Supprimer tous les appels directs aux clients OpenStack


@Ramshandilya shouldn't `UIAlertView` be managed by the View Controller instead of View Model?


I think this should be 'net.minecraftforge.event.world.ChunkEvent.Unload.class' ;)


à externaliser


You are welcome!

I think that assignment of `null` to array (`p.data` for example) is not so safe because it could be exist a variable which cache the reference. It could be like [here](https://github.com/tonytomov/jqGrid/blob/v4.7.0/js/grid.base.js#L777-L778), but the variable could be somewhere in outer scope, which will prevent destroying or to destroy it much later.

On the other side calling of `pop` in the loop is not only the most quickly way, but it's the most safe because it clear the array even if a reference to the array exist somewhere. The calling of `pop` in the loop clear the items in the array, but assignment of `[]` will work only if the array have no other references and the garbage collector agrees with the descruction.

Best regards
Oleg


Oops. I was searching for ```onBeforeMove``` but should have searched for ```BeforeMove```.


Also 'net.minecraftforge.event.world.ChunkEvent.Unoad.class' here ;)


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


My bad, already fixed.
Thanks ;)


Thanks. Feel free to make PR if there is a better way to work with this constants. I'm not really want to make them public.


OptionalTypedArgs (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (333 of 333 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)


@rin Der klassische Aus-Der-S-Bahn-Commit. :shipit:


```DEBUG &&``` is a small pattern suggested by Fabrice to use instead of ```if(DEBUG)```. Since ```JSON.stringify``` is an expensive operation in terms of performance it should be only done when were actually in the debug mode. This probably will be raised in the review, I'll add this back in the next iteration.


Yeah, you might want to check the other ones, just in case.

I only knew about this one because I had worked very hard on all of Sky Drop's glitches, and this was one of them.


These are attributes, not constants or classes, so should be camel-cased like now, but with a lower case character at the start.


name turnedOn


what for you have a const?


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


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


Well, part of my resistance to adding lodash is that the end goal isn't to have leaderHost as a clone of host. Something like this will probably be in the next release:

```is
  var hostUrl = clientOptions.url || 'http://127.0.0.1:4001';
  var host = url.parse(hostUrl);

  var leaderUrl = clientOptions.leaderUrl || hostUrl;
  var leaderHost = url.parse(leaderUrl);
```


What's a Smart Router? :)


add details to exception like expected/actual


using AlarmInfo?

lgtm anyways


what for this method? you have pour....


what does this change do?


CouchDB


rc7 has this, thanks for fixing


HTTPS is more secure...


@SBoudrias Why are we `stat: true`'ing on all the glob calls?

It seems totally unnecessary.

> stat Set to true to stat all results. This reduces performance somewhat, and is completely unnecessary, unless readdir is presumed to be an untrustworthy indicator of file existence.


I've made a mistake here. This actually needs to be Apache Fauxton.


Ok. lets optimize it like this:

`$t.p[i] = $.isArray($t.p[i]) ? [] : null;`


Then go ahead and add that instead :)


tests when it is turned off?


This should become `typer.context.initRootContext()`


init?


Why do `crossScalaVersions` include `2.10` here?


кожен комітає свій пароль?


See #7.


What you could do, I think, is to go for the `0.1.0-M0` version of scala.meta that I released on Monday. `SNAPSHOT` is still too volatile. Indeed, Obey will need `SNAPSHOT` for Friday, but let's solve that problem together on Thursday.


TeamCity Ylab :: zmactep/ylab [Build 153](http://192.168.30.225:8111/viewLog.html?buildId=522&buildTypeId=Ylab_ZmactepYlab&guest=1) is now running


niepotrzebne, stringi są automatycznie listami


guava має свої інструменти для toString також, між іншим


Removing the `it` causes the sentence to sound weird.

It erases / Erasing. I'd keep the `it`.


It changes a `notify` to a `before` to prevent `File[/etc/init/heka.conf]` in `Class[heka::config]` from notifying `Class[heka::service]` in addition to `Class[heka::restart]`.


The particular exception that you're facing is caused by the fact that you still have `libraryDependencies += "org.scalameta" % "scalahost_2.11.2" % "0.1.0-SNAPSHOT"` in your dependencies.

Instead of writing `2.11.5` there, I'd suggest changing that line to `libraryDependencies += "org.scalameta" %% "scalahost" % "0.1.0-SNAPSHOT"`


WOW


1: M111 unrecognized component name: authorids


and so have I!


typo on my part


wow


I'll merge parts of this patch one commit per issue. Thanks!


How does it sound weird? `list.erases` erases/removes items from a list, etc. Sounds pretty reasonable.

Actually you were right so I decided to follow conventions from other frameworks.
See here for example: http://jasmine.github.io/2.1/introduction.html


to sprawdzanie niepotrzebnie zwiększa złożoność obliczeniową.
Po pierwsze można by zastosować `if char in numbers` (o ile `numbers` było by listą stringów), ale to tylko ukrywa przeszukiwanie po liście, złożoności nie zmniejsza.
Po drugie można próbować `char` konwertować na inta (`int(char)`) ale wtedy trzeba łapać wyjątek `ValueError` rzucany z funkcji `int()`
Po trzecie można zamieniać char'a na wartość liczbową (`ord(char)` o ile się nie mylę), i robić prostego ifa `if 61 < ord(char) < 71`), ale to trzeba googlnąć "ASCII Table" i odczytać stamtąd jakie wartości wstawić za `61` i `71`


Nevermind, you are correct. I forgot you changed the describe.


It can be cherry-picked, but ipy-2.7-maint and master have diverged too far. I'd recommend splitting IR out like ironpython3.


a po czwarte całościowo można zastosować pythonowego `filter()` podając jako funkcję/lambdę predykat do testowania czy dany znak nie jest liczbą, i wtedy te ponad 10 linii kodu można zamienić w jedną ;)


to jest nieużywane z tego co widzę


@leonerd could you apply this patch on libtermkey? It is also necessary to link correctly against unibilium


tutaj można użyć `enumerate()` pythonowego który upraszcza tą konstrukcję


These are now ```__PIC__```


Maybe it is worth to have `>=2.5.0` instead of `==2.5.0`?


HipChat integrations don't have a `--repository` property


`nose` is not required by `./setup.py test`, and therefore should not be added there. It's only an optional development dependency.


the '@' prefix is not needed because of using decent_exposure gem?


Yeh, looks good to me.

Reviewed-by: James Hogan <james.hogan@imgtec.com>


całą tą drabinkę if/elif/else można po pierwsze wydzielić do osobnej funkcji np. `cardValue(card)`, po drugie można to uprościć poprzez zadeklarowanie słownika z wartościami kart (np. `values = {'W': 10, 'D': 11 ...}`) i stosowanie: `x = values.get(x, x)` (polecam dobre zastanowienie się czemu taki 'trick' zadziała ;))


В общем, идея следующая:

Database вообще ничего не знает про объекты, записи о которых она в себе хранит. Многочисленные клиенты Database, будучи и так довольно самостоятельными товарищами, теперь сами себе формируют хедеры, что позволяет убрать из Database код для всех частных случаев (ресурсы, результаты и проч.) и оставить лаконичное `addEntry(name, sender, object)`. `sensitivityList` становится `sensitivityMap`, и эта мапа хранит списки всех желающих знать все о том или ином объекте. Проблема из bogachev-pa#14 решена. Вот целиком код нового варианта Database:
```java
public class Database2
{
	public static class Entry
	{
		ByteBuffer header;
		ByteBuffer data;

		Entry(ByteBuffer header, ByteBuffer data)
		{
			this.header = header;
			this.data = data;
		}
	}

	ArrayList<Entry> allEntries = new ArrayList<Entry>();

	public static abstract class Index implements Subscriber // привет Tracer
	{
		protected ArrayList<Integer> entries = new ArrayList<Integer>();
	}

	private HashMap<String, LinkedList<Index>> sensitivityMap =
		new HashMap<String, LinkedList<Index>>();

	public void addSensitivity(String objectName, Index index)
	{
		LinkedList<Index> indexList = sensitivityMap.get(objectName);

		if(indexList == null)
		{
			indexList = new LinkedList<Index>();
			sensitivityMap.put(objectName, indexList);
		}

		indexList.add(index);
	}

	public void addEntry
	(
		String objectName,
		Serializable sender,
		Serializable object
	)
	{
		LinkedList<Index> indexList = sensitivityMap.get(objectName);

		if(indexList == null)
			return;

		allEntries.add(new Entry(sender.serialize(), object.serialize()));

		int marker = allEntries.size() - 1;

		for(Index index : indexList)
			index.entries.add(marker);
	}
}
```

Далее код из текущей версии Database расползается по клиентам. Паттерны сами заполняют статусы `Keep`/`Create`, и время с типом операции тоже сами. Все номера (`enum.ordinal()`) вместе с оффсетами и размерами переезжают в какой нибудь `DatabaseHelper`. Единственное, надо придумать какой нибудь механизм для добавления в `sensitivityMap` вновь созданных ресурсов. Раньше был костыль с `Имя_паттерна.Имя_релевантного_ресурса` в `sensitivityList`, и полем `sender` в `addResourceEntry(...)`. Теперь не все так просто, надо придумать этому альтернативу. Паттерны должны у кого-то спросить, надо ли добавлять зависимость на созданный ресурс, или нет. Причем этот кто-то должен быть проинициализирован `TraceConfig`'ом. Такие вот дела. Но такой вариант гораздо гибче, на самом деле, и Database не распухает.


w pythonie `&` jest sumą bitową i nie o to Ci raczej chodziło. Używamy `and` oraz `or` zamiast pokracznych `&&` i `||` z innych języków


TeamCity Ylab :: zmactep/ylab [Build 153](http://192.168.30.225:8111/viewLog.html?buildId=522&buildTypeId=Ylab_ZmactepYlab&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 122 Build time: 00:10:39


Does this need to be here? Is there any explicit reason?


Set this as arm64_getdev may not in all failure cases.


@alex-ab It would be very good to document the patch a little so that we (not just you but others as well) will be able to reconstruct the rationale behind it later. I.e., what symptom does it fix, and how does to fix it? With symptom, I do not refer to the TinyCore scenario mentioned in the issue but to the underlying issue (a misbehaving device model?). It is really not obvious to me.


tu jest wogóle ciekawa sprawa z całą tą pętlą `while` i warunkiem jej wyjścia - psama pętla while reaguje tylko na duże `N`, podczas kiedy tutaj reagujesz na małe `n`


Related to #476, #391, #13.


Comments from Ed in 9e9b1d8ef8060a4c64461a3d78fa7c00ce8e6ae6 on if this comment is needed


all tests pass:
success: http://build.servo.org/builders/linux1/builds/530
success: http://build.servo.org/builders/linux2/builds/528
success: http://build.servo.org/builders/android/builds/566
success: http://build.servo.org/builders/mac1/builds/747
success: http://build.servo.org/builders/mac2/builds/747


develop (macumber) - i386-Windows-7-VisualStudio-12: OK (2178 of 2190 tests passed)

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


I missed these magic numbers.


Good


whitespace - "ua-parser" needs indenting one more space


StuyPulse*


You're right, I'll change that and create a PR


czy mi się wydaje, czy to pytanie nie ma tutaj sensu? Ponieważ jeśli bedę chciał grać ponownie, to rezultat będzie taki sam, bo karty nie zostaną przetasowane. Czy może czegoś nie widzę?


emaste:
```
FWIW GCC 5 uses 67 as FIRST_PSEUDO_REGISTER
```


Oops, my fail. You can't use %% here. Do `"org.scalameta" % "scalahost" % "0.1.0-SNAPSHOT" cross CrossVersion.full`.


Y creo que he comprobado todas las posibilidades, y ahora si


Rename to UaParserEnrichmentConfig


tutaj widzę analogie i dodatkowo to, że kolory jakby nie są uwzględniane w żaden sposób w rozgrywce


fast-forwarding master to auto = 6b860056


Comment is out of of date


emaste:
```
I think we had ARM64TODO elsewhere.
```


We don't need a case class - delete this


UaParserEnrichment


Add newline at end


There was further clarification on the history of the arm64 headers in 2de5c30b6d10dd234a22c0005e5bf25808c6fdbe


OptionalTypedArgs (lefticus) - x86_64-MacOS-10.9-clang-Debug: OK (333 of 333 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)


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


Type should be Validation[String, JObject] - in other words, we don't stringify the JSON. We don't stringify the JSON because we are going to compose multiple JObjects into a single JValue in the EnrichmentManager (out of scope for now).


No need to stringify, see above


NICE WORK ^^


modelManager->setExtraDiagnostics() does not exist (or has been deprecated, not sure exactly) in 3.3; but there is now an equivalent in TextEditor, as TextEditorWidget::setExtraSelections() can now take a Core::Id argument.

I did not test or look any further, but I am surprised calling addDiagnosticMessage() is working: as far as I understand it, this does not cause the editor to refresh the 'selections', and they will thus likely be lost. Or maybe it works to some extent due to timing of some other analysis (e.g. code highlighting, clang diags...)

I would recommend using TextEditorWidget::setExtraSelections() instead, as it seems to be safer. It also allows full control over the highlighting (e.g. color, ...)


Also looking forward to seeing the wiring into the EnrichmentManager.scala (remember, just assign it to a nonsense var), and the tests...


tutaj te ify można znacząco uprościć. Wydaje mi się że nie ma sensu sprawdzać dwóch wariantów kiedy `smallest` jest albo nie jest ustawione. Pierwsza liczba zawsze bedzie jednocześnie największą i najmniejszą, a każda następna wystarczy że będzie porównana po kolei z najmniejszą i największą


I'm curious, why build tools instead of checkstyle?


Another comment


No boot ? :dart:


Add a space before each ~, so `data.useragentFamily) ~`


proponuję jakie rzeczy można wykręcać z zakresami w indeksach tablicowych w pythonie - np. tricki takie jak `text[5:]`, `text[:]`, `text[2:-2]` ... i wtedy można prościej to zapisać.


not accept change, I used the wrong file to copy and paste the changes :(


Outdated comment


może jakiś split + strip?


Space missed


zauważyłem że masz do czegoś takiego tendencję, a zupełnie nie jest potrzebne


Well, not sure what I can actually review. I trust you if you say it compiles and works locally (with Hawkular Metrics for example)


Please don't check for strings. I propose just `done` on fail


przeciekają Ci otwarte pliki, a powinno się zawsze o tym pamiętać - `fh.close()`. Ale oczywiscie w pythonie jest prościej - jeśli używa się `with open(...) as fh`, ale sugeruję poczytać jak to działa


Fuck off leave me alone and tell me how to delete these bullshit hidden apps please


I've found this.
http://stackoverflow.com/questions/11848705/undefined-symbols-for-architecture-i386-on-unit-tests
But the decision with linker flags doesn't help me :(
But that seems to be a real reason of the issue, cause I tried to move thos constans to any other .m file from the .mm and simply insert extern declaration, where it's used. That solced a linker error.


no proszę, to już prawie split + strip ;)


:+1: looks good and works fine, so I think it's time for a PR :)


Thanks!


No assertions in all tests. better to expect something than just end test by timeout


closed issue #8


to jest bardzo ciekawy kwiatek ;) Takie ciągłe usuwanie. może bawiąc się operatorami tablicowymi w pythonie dało by radę? Bo szczerze mówiąc to teraz nawet cięzko stwierdzić co tu zostaje a co nie.


Could it be done with more Jasmine 2.0 style. I.e. with `done()`


Rename to // ua-parser


I think it would be less error-prone to use `' '.join` here:
```python
    error_message = ' '.join((
        "TEMPLATE_CONTEXT_PROCESSORS does not contain add_page_if_missing.",
        "FeinCMSLoginRequiredMiddleware requires the FeinCMS page middleware",
        "to be installed. Ensure your TEMPLATE_CONTEXT_PROCESSORS setting",
        "includes 'feincms.context_processors.add_page_if_missing'.",
    ))
```


I saw that setExtraDiagnostics() did not work and looked at other code in other parts of Creator on how they handled it, I think I looked at the part for missing include files...

I will look into setExtraSelections(), thanks.


lgtm


Please use one space between function parameters


nie prościej poprostu `word[1]` wykorzystać? Może do tego celu przydać się znajomość wszystkich parametrów do funkcji `split`, w szczególności `maxsplit` bodaj.


The most obvious solution is to replace
foreach (var window in list.ToArray()) window.ClearWindowsInContext();

to remove all children


Thanks for your review!


@ulferts I can't figure out what `password.html.erb` is doing here.


też dobre, ale bardziej po pythonowemu było by użycie `defaultdict` z modułu `collections`


We are *always* passing an empty password, so if at any point this code block is executed, it will always fail. What is the purpose of this `if` statement? Under which circumstances will it be executed?
Consider removing this specific treatment or properly propagating a password value.


That seems sensible.  Those double parentheses look bizarre though :P


jeśli zależy Ci na samodzielnym napisaniu algorytmu na min i max to spoko, ale są funkcje `min` in `max` w pythonie które robią dokładnie to


Wait, does that work in Python 2.7?


Yes, I think so.


:p


I would add
```csharp
readonly
```


`hours = counts.items()`? :) (zamiast tej pętli pod spodem)


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


Não entendi, podia mudar não?...


Watched the video, it looks great.  Few questions, is there any way players can exploit this?  Can the commander move away from a base area with that view?  Can he go into the view away from the mhq?  Can factories be built inside buildings?  How far from mhq can a commander use this view?


Not quite what I intended, that's where you would have used `canPutTile` IMHO


hello word


It's a bit over-complex I think. I'm happy with it as it is, but I understand @Ian-Foote's concern.

I think this is a nice middle-ground:

```
error_message = (
    "TEMPLATE_CONTEXT_PROCESSORS does not contain add_page_if_missing." +
    "FeinCMSLoginRequiredMiddleware requires the FeinCMS page middleware" +
    "to be installed. Ensure your TEMPLATE_CONTEXT_PROCESSORS setting" +
    "includes 'feincms.context_processors.add_page_if_missing'."
)
```


ストイック！ :runner: LGTM!


> Ah, I get it. Reading your comment again, you're calling shouldApplyMixin when evaluating the list of mixins to apply, and not at the just-before-patching stage.

Exactly, which is why, until that phase has run (and plugins have vetoed (or not) their own mixins) it's not really possible to batch them up for other plugins to look at.

Mixins themselves actually already have a priority, but that's only applicable to mixins in the same config, I think what I should do is basically agreggate mixins from all configs when applying them, and sort them all together to allow priorities to cross the config boundary.

Like I said, leave it with me and I'll have a think about how best to adjust it.


funkcja pythonowa `max` w połaczeniu z parametrem `key` załatwiła by ten problem w 1-2 linijkach


I was able to apply Spring Boot version from one variable, but I didn't know that Spring Boot plugin registers its [own resolution strategy](http://docs.spring.io/spring-boot/docs/current/reference/html/build-tool-plugins-gradle-plugin.html#build-tool-plugins-gradle-dependencies-without-versions) to set Spring Boot, Spring and other known dependencies to "blessed" versions which is [even better](45a70b8ddcd4f843ededa2c556b55caf60871e5f). Thanks Mariusz.


Both the current version and your suggestion have missing spaces where the lines are joined.


What if it's night time? Need a night vision button in that view.


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


No es lo mejor tener una función que modifique un elemento, y a su vez retorne un booleano indicando si hizo una modificación o no. En realidad tendrías que tener dos funciones, una indicando si hay que hacer el reemplazo y otra para hacerlo


Thanks :)


So, one possible decision is to create new .m file and put constants there.


Good spot!


@kevva, does this require neatness? :)


Podría cambiarse a utilizar únicamente el objeto promise que retorna:
```
$.getJSON(...)
  .done(successFunction)
  .fail(failFunction);
//o
$.getJSON(...)
  .then(sucessFunction, failFunction);
```


Thanks!


Why has this method been removed? Looks like we can't get a collection for a specific class anymore which was pretty neat to use together with the codecs


En este caso que sentido tiene retornar un booleano, si en la función tenés una acción que es asincrónica?


What happens if you're in combat with Kromcrush? You can no longer fight him back.


哈哈，果然厉害，看代码应该是解决了这个问题。有个小问题：第126行的单词拼写错误：* don't excute window.onpopstate when page load

这里面的excute 应该是拼写错了，应该修改为：execute


hahahahahahah I forgot the commas and got this:

![screenshot from 2015-01-14 11 22 43](https://cloud.githubusercontent.com/assets/6973481/5738062/b5d9a4ba-9bdf-11e4-80bc-85bb2942c4ba.png)


assert failure if exception happens


move in try{}
and again wrappers!


the same


add tests for exception cases


This is going to affect other search info bars which expect to be right aligned.


URL은 YAML에 통으로 넣는 것보다 라우터를 참고해서 코드에서 넣어주는 게 좋지 않을까요?
config/routes.rb 쪽도 같이 고쳐야겠지만요 ㅠㅠ

```erb
<%= t('views.login.index.instructions_2_html', url: build_invitation_path) %>
```

http://guides.rubyonrails.org/i18n.html#passing-variables-to-translations


mm ,i just wanted to have the port be the single entry point of the server to networking stuff.
i also think that recovery wise its better to have the port hold this configuration. this way after recovery we just need to attach the port to the server, as opposed to attach the port and the security group


fixing it. thanks


@sergeyromanov you done your PR for January yet? :-p


This markup is needed only if microdata is enabled regardless of choice. If microdata:0, the old markaup for playlist should be used


Single quotes per conventions.


Lo mejor sería sacar el append de adentro del DOM. Armar un string con todo el markup, e hacer un único append al final, fuera del loop. Cada acceso al DOM es costoso, se deben minimizar. El DOM es uno de los principales cuellos de botella l trabajar en el browser


Hi,

Firstly, I'm sorry it's taken me so long to take a look.

Manifold sounds interesting- thanks for doing some experiments to see how it would be to integrate. As you say, I think I'd prefer the core of the lib to remain with its current dependencies and introduce a separate lib that could expose the messages through a manifold stream (rather than consuming from the `messages` fn).

From the changes you've submitted- the changes to `src/clj_kafka/consumer/zk.clj` could remain (albeit refactoring `messages` to use `stream` to get its data)- doing so should set you up to be able to open a stream and then adapt that with manifold?

If you're up for doing it I'd definitely be interested in taking a pull request for the `src/clj_kafka/consumer/zk.clj` (and dependent changes). It'd be really cool to see the other stuff as a supporting lib.


Eliminar variables innecesaria.


That's a good improvement.


seems like you need to modify resolve_lifetimes to resolve the path region parameters? or perhaps not, I have to double check the code. Maybe it happens automatically through the visitor.


Ok cool, thats what i have and seems to be working fine just wanted to confirm.


Some parts of the code which were necessary for successful pci+ahci compilation were copied from sys/arm/arm/busdma_machdep.c, other functions were stubbed. We will definitely have to review this code and find arch-dependent parts which would need to be modified for arm64 (if there are any).


Siempre agregar ```{``` y ```}``` a ``ìf```. Es necesario agregar un boolean?? En el medio hay una operación asíncrona, cualquier valor que se retorne va a depender de lo que pase con esa operación...


Mismo comentario que antes, utilizar promises


`:D`


Siempre agregar ```{``` y ```}``` a ```if```


@ggouaillardet Can you explain what this commit does?


```csharp
Assert.Throws(typeof(ObjectDisposedException), () => b2.Navigate("http://localhost/"));
```


Remove useless elements


There was a short discussion in https://docs.google.com/a/redhat.com/document/d/1rXxu1is-dCXe6CV6eQI9GKnF9OAsZHQQUaQrxzCIhYg/edit?pli=1 (that, apparently, somebody must have deleted in between). The argument was that the git repo can contain many other things except for Checkstyle config, such as our own maven plugins, build scripts and even the text of the license header which I am just about to add.


My backups were failing when i was adding protected(using the condition), there was some archive types differences, and the archive types must be the same for all the data files. Hence the system wouldn't restore the backup


Will be fixed in the next commit with other changes


Need to think, because _allContexts contains two objects, although the first closed.
Memory leak will be less, but it will.


writeStringToParcel(p, p_cur[i].addresses); and (char*)p_cur[i].addresses); appears twice, is that right?


Damn. I totally forgot about this. Thanks for spotting it!


When I looked at the patch, I realized that something is totally odd here. We definitely have tests with rectangles and rounded corners. They do have radii in the OM as well as in the generated SVG documents.

I compared the Generator raw data in the test suite with the latest version of Generator. Apparently the JSON changed which had affect on our test coverage.

With version 1.1 we did not have `shapeListKey` and `shapeOperation` in the JSON. Therefore we never entered `inferTransformForShape` and still got our rounded corners. This is not the case with version 1.2 anymore.

While I think the patch looks good to me, we need to see that reflecting in the OM and SVG tests.

r- because the test data needs to be updated before we can be sure that everything behaves correctly. We might find more regressions then.


Thx. It's an acceptable solution.


I think that DashManifestExtensions must not store the preferences like a preferred language or have any information about the lang that has been set. I think all these properties (as well as new added methods) has to be moved into other class (maybe StreamController, Stream or even more likely into a new class responsible for adaptation switch). MediaPlayer class also has to have API to get the available langs, the current one and the preferred.


The variables.less file does not exist in master, this causes core and plugin builds to fail. @mattab What is supposed to be in this file? Please advise.


![image](https://cloud.githubusercontent.com/assets/10122049/5738229/197bb814-9bea-11e4-9362-b2ed110f29a9.jpg)
The plugin doesn't work... :/


Wait.. self is required, `accepted = true` would only creating a local variable, not set the attribute?


Fyi, this broke compilation with GHCs prior to GHC 7.6 as they didn't tolerate yet hiding non-existent symbols


@ggouaillardet Was there a case that the previous case did not catch?

I ask because if the test wasn't good, I need to also update the test upstream in libfabric proper, too.


Commit message uses a lowercase starting letter after the prefix, please use a versal (example: "[TASK] This subject is valid")


Sorry, the file was added by mistake, its related to another ticket and is not finished yet


De hecho es preferible el estilo que usaste antes (**function declarations**). Ahora estás usando *function expressions*. Este tipo de declaraciones están sujetas a un mecanismo propio de JavaScript que se denomina **variable hoisting**. Esto puede ocasionar problemas: investigar! :)


Perhaps at least the URLs in the `pom.xml` should be checked.
I must say I have not tried with Hawkular Metrics yet. Going to try it now.


Just verified: My idea from yesterday was not too bad, bot I forgot to set encoding. Correctly it has to be
mb_strtoupper(ossn_print('friends'), 'UTF-8')

The problem with normal stroupper is: It doesn't handle utf-8 at all, thus small umlaut chars won't be converted. (see http://ossn.z-mans.net/u/zetman?ref=ossntb)

So I think, we can't circumvent relying on the PHP mb extension
(also see http://stackoverflow.com/questions/15051286/php-mb-strtoupper-not-working  ANSWER #2 )

But perhaps all this is too much overhead anyway, and we simply should add 2 new strings with CAPITAL chars directly to the language file?!


fyi, this requires `base >= 4.4` as `Control.Monad.Zip` was only added with `base-4.4`


@Jailander why is the default the lincoln settings :-) ?I


I added the above code only to find that it would work some times and not others, leaving protector nodes untouched...  will look into it...


When Zend OPcache is enabled, if php exits with zero status, it returns also messages like

    (1 of 1): P Fri Jan  9 16:52:12 2015 (140345699161856): Message Cached script '/path/to/file.php'

and vim complains with

    E885: Not possible to change sign GenericError


Extract functions do not work tho, the data gets damaged in the process (almost all the md5sum fails). Using the dump function will extract all the contents the way they should be (identical md5sums).

I guess this is because of the Protected Archive Types.

I might be fixing this soon too...Just kinda a bit out of time.


This is what I experience in the browser. My scope variable does just contain the text I typed, no html-tags.


Indentar código con 2 espacios


Agregar espacio entre declaración y regla: ```header {```


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


Ok, what about 00000-00000, I prefer this instead of nothing to avoid confusion with other values, like app name


Nunca subir código comentado


Have you tried to create the reward on the after(:build) callback?


En realidad tendrías que agregar por cada field del form un elemento ```label```que describa lo que ese field representa. También podés agrupar fields mediante un ```fieldset```. Fijate acá: https://developer.mozilla.org/en-US/docs/Web/Guide/HTML/Forms/How_to_structure_an_HTML_form


Ok, I will remove the class and really it is not used because it had not been finished


Arreglar la indentación


Separación debe hacerse con una sóla línea en blanco. Eliminar espacios de más


Acomodar indentación. Agregar espacio entre nombre de la regla y la regla


Ack


Agregar labels asociadas a los inputs


Cambiar comillas dobles por comillas simples al escribir JavaScript


@OlegKi Are you sure? You have written "jgrid.msiever() < **8**". I don't see how IE8's version can be smaller than 8


Agregar espacio entre declaración de la función y la llave


Implementation is strictly the same than NativeView's one, except i've just slighly tweaked it to match the construct of  `beget`, `indexOf` etc. Let me know if you prefer to keep it like the original.


This might solve your problem but now what if my app needs the `I18n.enforce_available_locales` to be `false`?

This is actually the case here. We added faker and now we have some tests failing that rely on enforce_available_locales to be false to pass.


Yes, you are right. Unless there will be no method to create local variable. :no_mouth:


J'ai oublié d'ajouter des fichiers lors de mon commit, non fonctionnel pour l'instant, je corrige ce soir.


No crear código en una única línea


`@lang` attribute is optional, so if it is absent the method will return `undefined`. I think this could be confusing and instead it should return a string (e.g. 'unknown').


@bilderbuchi i just uploaded the scripts to the ofSite repo, it's really just a bash script that runs the needed tasks to build the site.

btw, we usually rebuild the docs every new version not every new commit. the docs in the web should be updated to the most recent release not to master


Usar promises


Looks good. So actually we had this information and didn't require different fetching from MGMT?
BTW - training portal is on http right? We encountered interesting issue with Chen that if the "go" button is triggered from https application (from UI2.0 for example) we get an error.
I opened a bug for it:
https://ravellosystems.atlassian.net/browse/RAV-5603?filter=15902

Just FYI, IIRC we tested the "go" from the portal and no issues there.


- Arreglar indentación
- Usar promises
- Subdividir en funciones más pequeñas.
- Eliminar console.logs y alerts


Exactly!


canola misspelled?


It is better to use `getLanguage` here instead of `adaptation.Representation_asArray[0].lang`


How can i use it to delete stopwords from a large dataset (wikipedia long abstract for exemple)?


What happens if you just return null?


Speler aanpassen doet goed jongens Hoeraa!!!!


These mixins should have the correct super type.


Hi kinjo,

Cool! I'm looking forward for these changes and for Pull Request as well.

Thank you!


yeah looks ok


What is interesting - if you call just `accept` here it'd read model attribute while `accept = 'smth'` creates local variable.


The resulting URLs will be different in [Symfony 2.6](https://github.com/symfony/symfony/blob/2.6/src/Symfony/Bundle/TwigBundle/Resources/config/routing/errors.xml), but as they take the same parameters I consider it :+1:.


```
<br>
<br>
キーワードを入力せずに検索すると、最新のごちそう写真（なう飯）が表示されます。
```


pip does not have a public API, everything is internal and this means that there is no backwards compatibility promises. The reason we're requiring a session and not just creating one is because in our internal uses of this API it was too easy to forget to pass in the session kwarg and lose settings like proxy and timeout settings.


It will never go. :(


$config must be an empty?


@kenhkan `browserstack-runner@0.2.2` has a shasum checking failure, fixed by using `0.2.1`.


Comment should indicate we've gone the other way.


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


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


We may have to fix all the routines if we do this here. I guess we never set strict INFO code guidelines. New issue?


`self.noop` was undefined and `karma.loaded` was replaced anyway on `./main.js:1`


Agregar por cada field del form un label. Los labels proveen una descripción de cada field. Es una cuestión de accesibilidad, en case que el form sea "leído". Más práticas para escribir mejor los forms: https://developer.mozilla.org/en-US/docs/Web/Guide/HTML/Forms/How_to_structure_an_HTML_form


I will double check this tomorrow with upstream libfabric and intel c compiler


Maybe simply remove the redirect (and re-instate the `flash.now`)? The tenant screen will be displayed (with no tenant to choose from) but with the error message.


What is that in reference to?


Pablo, porqué agregas este reset, si ya tenés incluido normalize.css? Ambos tienen el mismo fin, y normalize es más actual :)


"Specifies" - typo
"triggered with proceed" - not sure this is proper english


Yes, it's a new issue. Most of the time you were following this convention, but I, indeed, didn't check if it's everywhere like this.


Simpler, i'll replace it


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3139
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3132
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3133
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3124
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3124
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3127
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3120
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3115
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2792
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2790
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1628
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1621


fast-forwarding master to auto = d52398ef


It'd make sense to put images in the repo, eg. doc/ folder


You're right. I will update.


why not using markdown?


[The bug](https://github.com/s9tpepper/karma-cucumberjs/issues/9) was that `require(...)` internally executes `setTimeout(..., 4);` this sometimes is faster than HTML evaluation and sometimes is slower. That's the race condition why the tests work randomly.

I don't understand why the `require(...)` sentence is outside `__karma__.loaded` neither why `__karma__.loaded` was set to an empty function. `__karma__.loaded` is the way Karma has to tell us the tests are ready to be executed so the most rational thing is to place the starting code inside `__karma__.loaded`.


sorry ,for this I will be more careful next time @westcripp


`simplifies`


this is mandatory for construction, mark this as final.


@tsegismont , I renamed a couple of more leftovers in the following commit a985a280388a379b7b89bfa0ff4250052aa50584 which I could also sucessfully test with RHQ Metrics.


This does not ensure that the persist partition is mounted before modules are loaded? should the persitent partition mount have Before= modules-load.service (or what ever it is called)


Remove breaklines. Use CSS to layout elements (set margins, paddings, etc)


This shouldn't be there.


:thumbsdown:


`and proceed button as your log in view` - I don't think that's proper english


this method is too long and repeat code (appliances construction). Not mandatory in this topic.


And thanks to @ernie for #pluck ;)


The function expected its return type not to return a null value .


Usar el patrón promise:
```
$.ajax(...)
  .done(successFunction)
  .error(errorFunction);
//o
$.ajax(...)
  .then(successFunction, errorFunction);
```
el uso de promises permite reducir el nivel de anidamiento de las funciones, creando un código más legible y mantenible.


Missing changes here:
```
--- a/imp/lib/Ajax/Application.php
+++ b/imp/lib/Ajax/Application.php
@@ -106,7 +106,13 @@ class IMP_Ajax_Application extends Horde_Core_Ajax_Application

         /* Check for global msgload task. */
         if (isset($this->_vars->msgload)) {
-            $this->queue->message($this->indices->mailbox->fromBuids(array($this->_vars->msgload)), true, true);
+            $this->queue->message(
+                $this->indices->mailbox->fromBuids(array($this->_vars->msgload)),
+                array(
+                    'preview' => true,
+                    'peek' => true
+                )
+            );
         }

         /* Check for global poll task. */
```


`below combination` I think you meant `following combination`.
also `a password reminder`


Good afternoon
Sorry to bother you, do you think this update could be related to the problem reported in http://forum.cyanogenmod.org/topic/103688-downloading-nightlies/ ?
Thanks and Best Regards


this should be needed too https://github.com/sirmordred/android_device_samsung_ancora/commit/11e99849d61a7e4d82b1afea30c11cc5c75c124a
because ı looked to /system/etc and gps.conf wasn't same as in gps repo


Siempre usar ```{``` y ```}```


My bad, two distinct commits at once ...
1. Unpatch romio e.g. no more disabled weak symbols by default, and pass the --disable-weak-symbols option from ompi
2. Pass the --enable-strict to romio configure and the -D__EXTENSIONS__ CFLAGS
This is the only way i was able to compile romio on solaris 10 since ompi uses c99 standard (unlike mpich)
That fixes the build issue reported by Siegmar on the users ml.


300 is a magic number. Consider to add a constant that describe the value, like DEFAULT_SQUARE_FEET=300.


Just out of curiosity: Why did you choose `new RegExp search.term, "i"` instead of `///#{search.term}///i`?


@bors: retry


Ver lo de activate() aca: https://github.com/johnpapa/generator-hottowel/blob/master/app/templates/src/client/app/dashboard/dashboard.controller.js


Fair enough, go ahead


saw approval from alexcrichton
at https://github.com/brson/rust/commit/f0fe4bb11404b2834bc3160683c220861142f955


merging brson/rust/finally = f0fe4bb1 into auto


status: {"merge_sha": "896cb36ecab3eaeb7f101087e030e43771eca5ca"}


brson/rust/finally = f0fe4bb1 merged ok, testing candidate = 896cb36e


你好，我在一台机器的Rstudio上已经安装了大量的包，我现在要将整个工作环境迁移到另外一台机器，请问是否有什么方法可以不用再一个个重复安装那些包?


Mejor:
```
$('.message')
  .html('Error!!!')
  .css('color','red');
```
Una función encadenada por línea. Permite el trabajo colaborativo, imaginate un dev cambiando el mensaje de error, y otro el color. Si trabajan en líneas diferentes es más fácil mergear. Además que el codigo es más simple de entender de un simple vistazo


What happens when you return null?


Use List<Room> rooms; instead ArrayList. Usually we use the interface in the field definition and implementation in the construction (polymorphism).


@chelmuth done.


Return the interface List instead the ArrayList


Interface List instead ArrayList


Eliminar espacios de más, con una línea en blanco alcanza


saw approval from nikomatsakis
at https://github.com/pnkfelix/rust/commit/7cae921fccc659eb65c509c14400902b07ca8923


merging pnkfelix/rust/kill-ref-cycle-in-librustc_trans = 7cae921f into auto


status: {"merge_sha": "d03a874736658bbb20d545e180dc6bd1ee43d046"}


pnkfelix/rust/kill-ref-cycle-in-librustc_trans = 7cae921f merged ok, testing candidate = d03a8747


Does this still allow Umlauts and other language specific characters?


`getMediaInfoForType` can be called any time and anywhere in the code. Setting audio lang and initiating languages must not be here, otherwise there will be issues. For example, you can see that `getMediaInfoForType` is called after each manifest update. If the mpd has two langs (e.g. 'en' and 'fr') and 'en' is main, and during the playback current lang has been change to 'fr', after manifest update has been completed the current lang resets to main ('en'). It happens because `this.manifestExt.getAdaptationForType(manifest, streamInfo.index, type)` always returns main adaptation and when it is passed into `setCurrentAudioLanguage`


Maybe blacklisting would be less complicated in this case, since JS doesn't seem to support unicode character classes.


Usually we use StringBuilder to construct a string with several parts.


还是不要存 buffer 了，最多给个可配置的 stream。buffer 在显示的时候很奇怪，等很久到完成才显示。

```
var exeq = require('exeq')({
  stdout: through(),
  stderr: through(),
})
```

默认可以输出到终端


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


Я бы скорее убрал из группы "продакшен" этот гем, а оставил вне групп. Он у тебя везде используется.


如果CPU体系架构一样（比如都是x86_64），应该是可以直接把包的目录拷贝过去用的。


Две пустых строки в конце опять?


Пустой хелпер вообще удалить можно, сам файл


@popomore

这里是直接 pipe 到终端的，buffer 只是为了最后的 results 。


@stevehodgkiss I think we should also cover the scenario access via string too


In the first place this project is hardware independent. But I used USRPs. N series and B210.
Only 3.7. is supported but there is a label marking the last 3.6.5 supported commit.
Did you try open the properties of a block and close it? That should update GRC evaluations. Besides if blocks are invalid they might not be installed correctly.


As i said earlier on the forum. C already supports the modulus operator:
[code]
	sprintf(*ret, "%f", (double)((int)a % (int)b));
[/code]


I assume the docker pricing scheme is applicable for collecting standard metrics through cAdvisor.

How a user be billed, if one have the dd-agent running on each containers to track metrics other than cAdvisor standard (like  dd-integrations,  custom metrics)?


好，我先试试


@fredericBregier
Are you sure that variable 'pastDelay' not equals zero all the time?
If I understood correct lastTimeCheck > lastReadingTime so (lastReadingTime - lastTimeCheck) < 0


It's the output of contexts_to_features_categorical from experiment_utils.py.
See line 124 of this file.


The function doesn't expect a null to be returned, if null is returned , Form object returned by formBackingObject() must not be null error appears


如果是默认的安装路径，包目录是哪个，找了几个都不确定，


r+


Builder pattern is explained in next topic, but this is a good approach. Builder have methods like:
public Builder addRoom(...);
and
public House build();

So you can do something like:

House house = new HouseBuilder().addRoom(x).addRoom(y).build();

This is not mandatory in this topic.


Lo mejor es evitar el acceso al DOM, o en todo caso minimizarlo. Crea tu markup en una variable, y hace un append al final, fuera del loop. De esa forma se hace un sólo acceso al DOM. En web apps el DOM es el principal cuello de botella de performance


f.input :data, as: :string


> btw, we usually rebuild the docs every new version not every new commit. the docs in the web should be updated to the most recent release not to master

at the same time, it's very useful, especially because of the recent doc drive, to have a master-current version of the docs available, to check the docs and see where they are lacking/need work. and an automated solution is perfect for this. optimally, we'd have docs for different versions/releases, like you can do with Sphinx/readthedocs.


Eliminar múltiples accesos al DOM!


@nfeske: ok - added a longish comment into the patch file


Add javadoc in the interfaces.


change Albun to Album


thanks, I will correct this.


Use promises to simplify code!


Use promises!


Confusing: getWalls() return just a Wall?. Maybe you should change it like Collection<Wall> getWalls();


I like the approach of returning a delegator object from the `#parallel` call. I'm not sure I agree with `Parallel` extending `SimpleDelegator`, though. I'm not sure what behavior is [least astonishing](http://c2.com/cgi/wiki?PrincipleOfLeastAstonishment) to the user: returning a delegate that doesn't implement the entire `Enumerable` API or having a "parallel" delegator that doesn't perform all operations in parallel.

I would love to see a (mostly) full parallel `Enumerable` implementation in v0.9. I'll happily help with the implementation if you want to take the lead on this.

Thank you for suggesting it and looking at other implementations!


thanx for that one!


Tudo loko haha :P


The simplest solution for you would be to have, say:

``` haskell
type Get' a = Get [JSON] a
type Post' a = Post [JSON] a
-- ...
```

It would clutter the API a bit with more types, so I think whether we add this or not depends on how much people want it?


crawler ?? and what about walker ?
https://en.wikipedia.org/wiki/Tree_%28data_structure%29#Traversal_methods


Sorry, I misread it as 'from mux to Martini'.

Ignore.


Just based on this diff I don't see the need to include <vector>


this method should be added to the prototype


No es suficiente, además hay que setear el prototype. Acá solo estás invocando al constructor de Movie.
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Function/call


bug fixes


`installed.packages()[, "LibPath"]`


I'm not sure why prn is used here. :message is just a string right? Unless I'm missing something, I think you can use https://github.com/puppetlabs/puppetdb/blob/master/src/puppetlabs/puppetdb/utils.clj#L31


Really Appreciate your help


También podría agregarse un check para ver si el attr existe. Cómo harías eso?


On Wednesday, 14. January 2015 04:35:11 amaigil2 wrote:
> Just based on this diff I don't see the need to include <vector>

good point, that's a leftover when the previous version of the
implementation had to keep an extra list of the checked filenames.

I'll remove that one before the final commit. Thanks.


I want to ask if there are other flags to be set beside prune to work properly as i tried to run this version but prune isn't work with any of the ngram orders!!!


Esto no es un módulo. Envolver en una función auto-invocada


r+


De que forma se mixean estos métodos en otros objetos?


Needs to be a .md link


Typo "Nedd". And missing a full stop


I meant which code (which feature extractor(s)) generates categorical features like: [u'petit', [u'le'], [u'garcon'], ...]? Now I understand that it's the alignment feature extractor.
Do we need to make some distinction between feature extractors which extract categorical features and feature extractors which extract numerical features? The reasoning would be that categorical features may require some further post-processing (i.e. binarization), and we need to know whether this post-processing is required or not.


I've tried some variations. We have a problem for the timestamp returned by expires_at is slightly different from the one returned by the ruby code. Converting both explicitly without the milliseconds should do the trick.
This should work:
```
it{ expect(subject.to_s(:simple)).to eq(Time.zone.tomorrow.end_of_day.to_s(:simple)) }
```


@ronkorving actually it's the opposite. `fs.exists()` simply did an `fs.stat()` and returned whether the file existed or not. `access(2)` is a much cheaper way of finding out if a file exists or not.


@chrisseaton What do you think?


fixed :-|

On 14 January 2015 at 12:41, Simon Cropp <notifications@github.com> wrote:

> Needs to be a .md link
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Particular/docs.particular.net/commit/ac2a69f75b52f1063e2dbfd3ebdada40b1330c7d#commitcomment-9277726>
> .
>

--
Regards,

Sean Farmar

+353 87 2149289

www.seanfarmar.com


I don't understand what you mean by "extract the script to a separate file and use the correct helpers to call it".
Is to create a view helper to generate the json, or create a js file and pass the parameters as a json to it. You can explain better the meaning?

And I think the scriptsForLayout may continue be needed because all the scripts are loaded at the page bottom


请代理访问


fixed

On 14 January 2015 at 12:42, Simon Cropp <notifications@github.com> wrote:

> Typo "Nedd". And missing a full stop
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Particular/docs.particular.net/commit/4db9853598f4a30865e1b0c840efe51befcae0cf#commitcomment-9277737>
> .
>

--
Regards,

Sean Farmar

+353 87 2149289

www.seanfarmar.com


I thought about using an exclusion filter by having a file with a list of all these slow specs, but there seems to be no support for that (the profile filter has nothing to do with that, although it does load from a file), contrary to tag which should be reasonably well supported (but clutter a bit our tag files).


no problem


for displaying? to display?


@willianjusten Claro que podia, haha. Eu que coloquei o Twitter ao invés do Github por algum motivo desconhecido :p


Pruning does have a bug on which I will be working this week or weekend. However, it works _most_ of the time. What error exactly are you getting?


Wait... that locks out the possibility to morph humans into any other valid humanoid, doesn't it?


I fixed this today, once @adulau merges the branches, pull the changes and then you can push yours


@scytacki Thanks,

Please refer this commit for the changes https://github.com/concord-consortium/rigse/commit/01965c6fa3f5ff775f1955c03ededef7e8625117


Looks great!


Better indeed.


What if the `block.asset` actually is an URL? My html processing breaks with this change as now something like `https://somedomain.com/something/` gets replaced with `https:/somedomain.com/something`...


Why don't you just remove the tajaran species definition..?


Bien, voy a investigar un poco. En uno de los documentos recomendaban este estilo :S


ok ... maybe 00bc333 helps


There is no difference between feature extractors for categorical and numerical features.
The difference is in functions that call them: contexts_to_features and contexts_to_features_categorical. The latter doesn't use numpy hstack and vstack functions (because categorical features are of different types and can't be stored in np.array). If we get rid of them, the two functions can be merged.


Do we want a `jt test fast` command? The whole point of `jt` is that you don't have to remember any extra command or options.


I think this test will fail if you add select field option to the populate, e.g. .populate('users', {username:1, name:1});


cool, but need lot of stuff to appear


static const float ao_gamma =
				rangelim(g_settings->getFloat("ambient_occlusion_gamma"), 0.25, 4);


Alright, just wanted to make sure.


Valid values are in the range 0.25 to 4 inclusive.


Remove commented code. :P


About post-processing of categorical features: they need to be binarized, but we don't need to make any distinction between categorical and numerical features here. If we feed numerical features to experiment_utils.binarize() it will just leave them unchanged. I hope so.


FOR SHAME :>


Gracias!


@timholy this is very cool and elegant but it seems to require a corresponding generic function for each functor, no? So we'd need to have names for thing like `POrNotQ`, i.e. `x | ~y`, etc.

Not that it would be dramatic. But I think we can also decide to do that after the PR is merged.


Got it. Dint notice "set(USE_FSAL_GLUSTER OFF)" before. I am fine with your approach.


ACK


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


btw, a shorter way of doing this is `[thread isGroupThread]`


remove logging?


Typo


Just another cluttering echo ...


what happens when the user specifies a feature extractor which outputs
categorical features, then tries to use those features with a learning
method that doesn't support them? We should at least think about a warning
to the user to let them know that they are getting non-real valued features.

On Wed, Jan 14, 2015 at 12:56 PM, varvara-l <notifications@github.com>
wrote:

> About post-processing of categorical features: they need to be binarized,
> but we don't need to make any distinction between categorical and numerical
> features here. If we feed numerical features to experiment_utils.binarize()
> it will just leave them unchanged. I hope so.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qe-team/marmot/commit/92e9f0a857c95259304172f0028bfe4aed80761a#commitcomment-9277868>
> .
>


scheinbar ist hier die Page "Offene Anfragen" kaputt gegangen. refers to #243


The stars should be aligned - the spaces should go in again.


Like this?

`#define IS_GE_IOS_8 kCFCoreFoundationVersionNumber >= 1140.10`


OMG.
Thanks for reporting.


@gamwang I think we need to set return value for addView, getViews, addUpVote, and getUpVotes methods. Right now, only addView and addUpVote return new Model while getViews and getUpVotes return only the Number. I don't think we need endpoints for getViews and getUpVotes since the Comment or Content users see will have the fields, but we definitely should keep class methods for computing priority later.


Owen, these changes are so old that they are no longer useful. It isn't even the latest version of the chapter.
I'll look them over as I'm finalizing chapter 6 now.


then why not just keep IR in master?


When grailsResourceLocator is used in development mode (resourceLocatorEnabled=true by default in dev.mode) , the check failed and therefore I disabled it.
Example of debugging grails-website project:
```
urlAsString: file:/Users/lari/workspace-grails-projects/grails-website/target/plugins/jquery-ui-1.8.24/web-app/jquery-ui/themes/ui-lightness/jquery-ui-1.8.24.custom.css
uri: /plugins/jquery-ui-1.8.24/jquery-ui/themes/ui-lightness/jquery-ui-1.8.24.custom.css
```
urlAsString never ends with uri in this case.


Just to clarify, the commit message should be "blocked any user that *did not have* 'user' or 'manager' roles".

I can confirm that I am able to successfully clone with the broken build after adding 'user' role to my user that already has 'admin' role. This change seems to fix it so that 'user' or 'manager' roles are no longer required. Thanks.


if ('mob_summoned == 2) end;


Yeah, that does remove any humanoid species transformation. I would suggest creating a new list out of all_species, remove tajaran, and use that


'mob_summoned = 2;


Those spaces *should* be there


Please drop this and add it to your repo specific ignore file ```.git/info/exclude``` or global even your global ignore file. See http://git-scm.com/docs/gitignore for details.


You can delete this OnInstanceInit, 'mob_summoned = 0; need to be load once


You'll be a hero, if you will make it, thanks for efforts! I wanted to do it as well, but did not have time.

Pls do not hesitate to ask questions or to raise bugs in the case of issues.


Probably we should make features extraction and binarization a single procedure, so that it's impossible to pass any categorical features to learning?


Not sure if there's a way to avoid this duplicate wording. Only want the `sectionTitle` to be different for Masterclasses. Might be OK as it is as Will has a PR up which lets us move away from all the case/match stuff anyway (PR https://github.com/guardian/membership-frontend/pull/66)


Your wish is our command

http://wiki.cyanogenmod.org/w/Build_for_tomato


Rebased & merged, thanks!


Here it is :smiley:

http://wiki.cyanogenmod.org/w/Build_for_tomato


yeah :), let me finish core specs first!


Hold your horses, this is WIP ;)


Not so sure that red insulated wire and blue insulated wire should be ore dictionaried identically. Changing colour requires dye and making them the same in the ore dictionary circumvents this.


私も作ってみました．比較してみてください．
[simplex_grid](https://github.com/oyamad/simplex_grid)


I 'm unable to find 'f_traceApp.R file and it is used in command "source('f_traceApp.R')".

can you please upload in github.

Thanks
Haneesh


@LFeh, ah tá ahuahuahu
O @felquis falou ali, eu num tinha entendido nada ahuahau


Thanks! :+1:


Te falta un intro


Con uno bastaba :dancer:


The removal of the brackets changed the evaluation order. Now it is

```
 record.is_a?( Array && (search = record.detect { |o| o.is_a? Ransack::Search }) )
```
which results in a type error because it returns a `Ransack::Search` instance instead of the `Array` class. (i.e. `&&` is evaluated before the `is_a?` method call)

To fix this the code should either be changed back or the `&&` changed to the less strong `and`.

```
 record.is_a?(Array) && (search = record.detect { |o| o.is_a? Ransack::Search })
 record.is_a? Array and (search = record.detect { |o| o.is_a? Ransack::Search })
```


Solo un enter de diferencia


We use a custom build of Modernizr, and want to enable people to do so. That's why we don't use a cdn for this. Also, you might want to read this blogpost: http://modernizr.com/news/modernizr-and-cdns/.


(niko and i discussed offline: he wants `BlockScope` to be specialized to only take the node-id for a `DestructionScope`, if possible.  He and I both agreed that a new-typed node-id was most appropriate for this role.)


If ?


Fine with me.


Sorry I wasn't clear. Though it's no longer a magic number it's still not possible to verify the expected outcome from the test code without making a jump into testcommon.js. If you used ANIMATION_DURATION / 2 instead of 50 (or TIMEOUT/DELAY) then it would be easier to understand what the test is doing.


Para proxima, las carpetas siempre en minuscula eh xD


This change has a similar problem as the one above: The ternary operator is evaluated before the `is_a?` method call, which results in `o.klass` being called on every item of the array. (Which fails for symbols for example)


The refcount is increased with NineVertexDeclaration9_ctor, but it is decreased, and the dtor is called if it returns a failure.

No code should be in NineVertexDeclaration9_new.

Please also put the !This->usage_map in another patch


well some learning applications might be able to use them -- i.e. a CRF
implementation might know how to handle the categorical features. Some
scikit learn method try to handle them as well. Let's just keep this as a
note and deal with it later.

On Wed, Jan 14, 2015 at 1:10 PM, varvara-l <notifications@github.com> wrote:

> Probably we should make features extraction and binarization a single
> procedure, so that it's impossible to pass any categorical features to
> learning?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qe-team/marmot/commit/92e9f0a857c95259304172f0028bfe4aed80761a#commitcomment-9278021>
> .
>


La id de las spell en el nombre del commit no es necesario


Not bad, but now librt is no longer linked:
```
gmake[2]: Entering directory '/home/dam/work/rrdtool-1.x/src'
/bin/bash ../libtool  --tag=CC   --mode=link gcc -g -O2 -D_GNU_SOURCE -fno-strict-aliasing -Wall -std=gnu99 -pedantic -Wundef -Wshadow -Wpointer-arith -Wcast-align -Wmissing-prototypes -Wmissing-declarations -Wnested-externs -Winline -Wold-style-definition -W  -fPIC -DPIC -I.. -g -O2 -D_GNU_SOURCE -fno-strict-aliasing -Wall -std=gnu99 -pedantic -Wundef -Wshadow -Wpointer-arith -Wcast-align -Wmissing-prototypes -Wmissing-declarations -Wnested-externs -Winline -Wold-style-definition -W  -fPIC -DPIC  -L/opt/csw/lib     -L/opt/csw/lib     -L/opt/csw/lib     -o rrdcached rrdcached-rrd_daemon.o librrd_th.la -lm  -lsocket -ldbi -ldl -lwrap -lglib-2.0 -lintl   -lpangocairo-1.0 -lpango-1.0 -lcairo -lgobject-2.0 -lglib-2.0 -lintl   -lxml2
libtool: link: gcc -g -O2 -D_GNU_SOURCE -fno-strict-aliasing -Wall -std=gnu99 -pedantic -Wundef -Wshadow -Wpointer-arith -Wcast-align -Wmissing-prototypes -Wmissing-declarations -Wnested-externs -Winline -Wold-style-definition -W -fPIC -DPIC -I.. -g -O2 -D_GNU_SOURCE -fno-strict-aliasing -Wall -std=gnu99 -pedantic -Wundef -Wshadow -Wpointer-arith -Wcast-align -Wmissing-prototypes -Wmissing-declarations -Wnested-externs -Winline -Wold-style-definition -W -fPIC -DPIC -o .libs/rrdcached rrdcached-rrd_daemon.o  -L/opt/csw/lib ./.libs/librrd_th.so -lm -lsocket -ldbi -ldl -lwrap -lpangocairo-1.0 -lpango-1.0 -lcairo -lgobject-2.0 -lglib-2.0 -lintl -lxml2 -pthread -R/opt/rrdtool-1.5.0-rc1/lib
Undefined                       first referenced
 symbol                             in file
sigwaitinfo                         rrdcached-rrd_daemon.o  (symbol belongs to implicit dependency /lib/librt.so.1)
ld: fatal: symbol referencing errors. No output written to .libs/rrdcached
collect2: error: ld returned 1 exit status
Makefile:747: recipe for target 'rrdcached' failed
```
When I manually add -lrt it works.

BTW, I have set up builedbot for rrdtool on Solaris with Sun Studio 12.3:
https://buildfarm.opencsw.org/buildbot_admin/waterfall?builder=rrdtool-solaris10-i386&builder=rrdtool-solaris10-sparc&reload=60


OptionalTypedArgs (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (332 of 333 tests passed)

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


thank  you .....
starting with TWRP then will move on to Slimkat :smiley:

would have been lot nicer if you could add this to readme ...... anyways MUCHAS GRCIAS


oki no problemo.......... :smiley:


git clone https://github.com/hgarc014/git-game.git


Ah, that comes from me newly emptying the argv array. Maybe I got a different version of something so that it does not show up here.


:smiley: Thanks


fixed https://github.com/Rudolph-Miller/react-sortable-table/commit/ad71e1cc3899887bf2b48a4ee4e2a311fc5800d4#diff-00383b752c9832fdcb5eb47b3ce84b76L69


Looks spot on.


I think the old code with This->caps.MaxVertexShaderConst seems fine,
however the part for ps isn't.
instead of checking agains NINE_MAX_CONST_F, it should compare to the number of ps constants allowed (32 for hw supporting <= sm2, and 224 for sm3)


Sorry for the imports like that....
I haven't found the way to change it....
confuserr may change it after :)


ok, then we this patch is ok


Sort imports.


Just out of curiousity, why aren't we just looking at the sourceID to determine what the output should be regardless of the sourceName?


actuatordrv --> actuatordrv0
lens_actuator --> lens_actuator0


Very nice. Should the image expand on mouse over?  I also like how you moved the file size too.


Worked like a charm.


Cool.

I'm happy to commit this to libfabric upstream (or accept your PR -- either is fine) if the existing test wasn't sufficient for some reason (which, although in different places in OMPI vs. libfabric, is the same).

Thanks!


arf oui :/


Thse changes broke wifi in moto g 2014

E/WifiHW  (  826): Supplicant not running, cannot connect

I/wpa_supplicant( 3418): Successfully initialized wpa_supplicant
E/wpa_supplicant( 3418): Failed to open config file '/data/misc/wifi/p2p_supplicant.conf', error: Permission denied
E/wpa_supplicant( 3418): Failed to read or parse configuration '/data/misc/wifi/p2p_supplicant.conf'.


在Mic面前没人敢自称处女座。


also, we can't know the proper binarization until we see the test data,
because the test data may contain categorical features that we didn't
observe in training. You handled this correctly in your example
implementation here [1]

[1]
https://github.com/qe-team/marmot/blob/master/examples/word_level_quality_estimation/wmt_word_level_experiment.py#L82-L84

On Wed, Jan 14, 2015 at 1:24 PM, Chris Hokamp <chris.hokamp@gmail.com>
wrote:

> well some learning applications might be able to use them -- i.e. a CRF
> implementation might know how to handle the categorical features. Some
> scikit learn method try to handle them as well. Let's just keep this as a
> note and deal with it later.
>
> On Wed, Jan 14, 2015 at 1:10 PM, varvara-l <notifications@github.com>
> wrote:
>
>> Probably we should make features extraction and binarization a single
>> procedure, so that it's impossible to pass any categorical features to
>> learning?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/qe-team/marmot/commit/92e9f0a857c95259304172f0028bfe4aed80761a#commitcomment-9278021>
>> .
>>
>
>


This was originally created by the previous Project owners but of course we can change it


Doaa e-mailed me directly.  Once is sufficient.


great, these are dead?


You will need to update playstore description also :-)


Esto no me termina de convencer, podría ocasionar crash si no encuentra caster.

Sería mejor:

```c++
                if (Unit* caster = GetCaster())
                    caster->RemoveAura(SPELL_DK_GLYPH_PATH_OF_FROST_EFF);
```


Aquí lo mismo y creo que sería mejor con un CastSpell
```C++
                if (Unit* caster = GetCaster())
                    if (caster->HasAura(SPELL_DK_GLYPH_PATH_OF_FROST))
                        caster->CastSpell(caster, SPELL_DK_GLYPH_PATH_OF_FROST_EFF, true);
```


I think the answer lies in your question! :-P


Looks ok to me.


Thanks for fixing. Hadn't time to test.


Is there a reason for these single quotes surrounding the object?
These are the reason the paths of jumpscale7 get messed up into jumpscale7'


:D My version would be a few chars shorter. But I guess you're gonna delete this now anyway.


Carlos he made me realize that to make users benefit from PCT-1025 changes I need to introduce a default value for PidFile, as opposed to accept to execute percona-agent without a pidfile. This is to overcome the need of reinstalling the agent to add a PidFile option in agent.conf.


who benefits from making this shorter? its ugly as hell :-D


I think it's beautiful as heaven.


Literals and interpolation ftw!


Hat's nen Grund warum das hier unten steht? Müsste das nicht nach oben unter "Allgemeine Dokumentation"?

Ansonsten sind das gute Links, würde sagen die Liste kann man so lassen. Oder?


the more complicated and big-dataish it looks the better, true? :-P


@DefinitlyEvil dp.getPid() is null :/ also, how do I get the decoded fields
and actually do something with the session or player class? See, it's
messed up.

On Wednesday, January 14, 2015, The Evil Man <notifications@github.com>
wrote:

> @jython234 <https://github.com/jython234> You should use
> switch(dp.getPid()){} here. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BlockServerProject/BlockServer/commit/e2eb31bf95e90a4dfe4ae457fec9c0073956ef7b#commitcomment-9274189>
> .
>

--


Looks good to me but I don't have write access to merge this.


This commit caused the following error during composer install:

  [ErrorException]
  Invalid row on line 1 has 1 parts, expected 2

It is thrown by modman as a second parameter is missing


Naja, alle oberen Links sind in meiner Liste auch drin. Oben sind die spziellen Links, unten ist die umfangreiche Liste. Ich denke, das ist okay so.


Siempre que haya que pensar mensajes de este tipo, es interesante pensar en qué contexto aparecen y qué mensaje es útil o informativo para al usuario. Por ejemplo, project.not.found aparece cuando abres la aplicación después de cerrada y se trata de abrir un proyecto que no existe. Ahí lo más útil es decir lo que sabemos. 'He tratado de abrir el último proyecto que editaste ($path_al_proyecto), pero parece que ya no existe. ¿Quizá se borró manualmente?'.

Bien pensado, quizá estos dos mensajes en concreto tiene más sentido mostrarlos en un diálogo en vez de en un toast, porque no queremos que los usuarios lo pasen por alto.


update to green


This will fetch more than necessary.
Missing equal sign?
```python
... if len(documents) >= limit else ...
```


should say warn


In my eyes it looks less complicated.


perfect!


variable keysign is not needed


does ril works after this????


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


if ```$key``` is already cached, but ```$results``` has returned empty, this will continue and query again.

![lock-role-key](https://cloud.githubusercontent.com/assets/1854511/5739672/20c44aca-9bca-11e4-8eb5-6100f665c050.gif)


Whats the reason for hiding the buttons for that referer?

It's not obvious (to me) from the commit itself, or the message. It's always good to include the "why" of a change in the commit message. It's usually more important than the "what", as someone can always work out the "what" from the diff of the commit itself :)


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


Can you abstract this a little bit with no inner classes?


According to [package.json](https://docs.npmjs.com/files/package.json) docs, there should be only one person here. But maybe you can drop @Swiip :p


Given that the target is now 6.4, these should all be changed..


URL should be lowercase


its my code, I win :-D


Is it? I like the way these things are communicated ;)

On 14 January 2015 at 13:53, Marco Rietveld <notifications@github.com>
wrote:

> Given that the target is now 6.4, these should all be changed..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/droolsjbpm/droolsjbpm-knowledge/commit/bbd7f9dc6a8dcefa2e2d6c2e7369869868052b3e#commitcomment-9278581>
> .
>


@diogob yep, tested all callbacks from factory girl


Yeah!


IMHO, design of `self:serial({ json = xx, msgpack = xxx })`  API is wrong at least due to  this code:
```
+ if opts.json == true then
+ slib = json
+ elseif opts.msgpack == true then
+ slib = msgpack
+ else
+ error("Unsupported serialization format")
+ return nil
+ end
```

It would be nice if http handler check `content-type` header and decode body automatically.


Sure, if it makes you happy :tongue:


Ah. I added an explanation to the PR description but not the commit message. If I amend it we'll lose these comments, but before we both forget, here's my reasoning (that might be misguided). "Depending on where the user comes from, display either the confirm/reject/back buttons or nothing at all in the case of coming from /list_all_redirects. The list all page should not, IMO, be a place to deal with the requests - that should go through 'manage requests'."


Move the code in this method to the toString() method

That allows us to eliminate this method all together

If you want a pound sign pyramid, just say "new PoundSignPyramid" and it will have the correct symbol


Are you sure you need to extend this to use inputStream? Can't you obtain a valid url from an Uri? This dualization uri - inputStream feels weird.


He did this with my agreement. I know it's not really by the book but...


Looks cool for me in a glance.

I have no much time to take a look but I will do it as soon I have time.

Just asking, why didn't you just create a listener for that instead of change and such?


Make this method print out the string of the pyramid.  Does not need to be abstract since every object inherits from Object's toString method


Hat mein Editor ganz automatisch gemacht.


Fix https://github.com/BeatSwitch/lock-laravel/commit/ae98f7b77a1b87ad96c0c286a1991df427cccc80#commitcomment-9278548


move this code to pyramidCharacter, make this return just a "%"


make this return a pound sign and move this to pyramid character


this should just return a string


Oops sorry, I somehow missed this pull request. I fixed it meanwhile, the logo did not mean to be so large.


Just thinking. If the kie-drools-wb uses the same features for asset management, maybe we should move this to the shared docs so that we can also include it in the Drools documentation?


What do you think about this https://github.com/studiofrenetic/lock-laravel/commit/48d55919a5e4af6438bc858964b4b69b36649445 ?


And I suppose now we're restricting the list page to only list "accepted" requests, there's (definitely?) no point in showing the buttons from that referer. Or should we hide them based on state?


This comment will have lived it's entire life in vain.  Let's take a moment to honour it for it's service!


The first thing you should do is remove every line which is not required for this test.  For example, a lot of these modules are not used in the test.  They don't need to be here.  Just peel everything back to just what is absolutely necessary.


Typo has been fixed https://github.com/studiofrenetic/lock-laravel/commit/7b5c5c0d36a4dda9e99f727c47ce38bbe60c4129


왜 이따구로 했냐


Falta el código


Yes, I know you said that, but it gives bad results if a is negative.
E.g.  -42 mod 60 returns -42 with your method, where it should be +18.


I do not fully understand why those were changed. Was is an esthetic choice or does it achieve something specific?


@MathieuDoyon this is indeed probably better. Send it in as a PR :)


thanks!


use the `$testfile` variable


Great :+1:


Spacing with "ua-parser" - add another space


Change to // ua-parser


works now?


ss


Yea
Keep in mind I moved them to msm8226 repo


Whitespace and please add a docstring comment for this object. Like /** */


Something is weird with my terminal, garbling the commit message. This should read "when a value represented by a runtime object is used as a function, the error message needs to toString() the object. AFAIK, this only happens for bignums."


Change this to a /** */ comment, which introduces the UaParserEnrichment. Explain what it does so a future you (somebody looking at this in 6 months) knows what this enrichment does.


Filtering the dataset with maxlen=100 as in this script drops the number of training cases from 25000 to less than 2000. Are we 100% sure it what's intended? I know a lot of people remove undesired training cases from their datasets, but dropping 90% of the dataset is not something I have seen people intentionally do before.


Ok.


Add spaces before the ~s, so `c.userAgent.family) ~`


If you agree to have one contributor more, i'll be happy to help as much as possible. I've 3 PR that will conflict on merge, and maybe it would be easier if I merge them myself. You can send me an email if you want (even in french), on PM on Twitter.

I find this generator really efficient now, and it definitively makes the official angular generator deprecated ! Great job anyway @Swiip and @zckrs.


TeamCity Bij12 Voorgangsrapportage Natuur - LCV :: CDS BIJ12 :: Build [Build 2.3-SNAPSHOT-74-eaab172ffbe675060a561d0843fc00468cf8e124](http://buildserver.geodan.nl/teamcity/viewLog.html?buildId=73969&buildTypeId=vrn_CdsBij12_Build) outcome was **SUCCESS**
Summary: Tests passed: 61 Build time: 00:02:15


It's best not provide markup (colors etc) until the presentation/view layer.

You can do this once in a log
```ruby
begin
[ .. do stuff.. ]
rescue Exception => ex
  puts ex.message.red
end
```


Noch nicht ganz. Die zweite exception fliegt noch. Hab das jetzt nur
gepusht, damit dave sich das anschauen kann.

--
Sent from my Google Nexus 5.
On Jan 14, 2015 3:03 PM, "Christian Dietz" <notifications@github.com> wrote:

> works now?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Treiblesschorle/seminar-data-challenge/commit/7332344624569749dfa762ceb10e31e8eb307e44#commitcomment-9278718>
> .
>


I think it would more useful if you added a transparent attribute to FileldCommand.


don't the conventions usually say "x ? y : z" rather than x?y:z


You can submit issue ;)

BTW I think that was part of what's new (changelog), so there's no other artifacts. Changelog for new version - removes the changelog for old ;)


wenn ich später dazu komme schau ich auch nochmal drauf


Optionally, you could rename this method to be more explicit (no big deal though)


Can we get the body instead, and compare its name to "East Sussex County Council" instead of to 2224 ?


Please add space before { everywhere for consistency with the other files.


Minor comments


Not convinced this is better: I'm assuming Array.newInstance() is slower than new X[]. The code could be changed to be String specific.


There is an error in the link (hreft).


yeah, $row is a DB::Problem, so $row->body=~/East Sussex County Council/ (as that returns a string like "Body X and Body Y") should do it?


"Date de commande"


If you add this line, considering exitPan definition, you can remove stateMachine.exitFullScreen(), right?


some tests failed:
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3126
exception: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3126
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3122
exception: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3117
exception: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2794
exception: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2792
exception: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1630
exception: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1623


@bms8197 Thanks


## Refactor Suppression

**.... (not like this..) this is duplication of the Veto Model**

Use the Veto Model, then leverage the `.errors.full_messages` method

## Veto Model

If your entities and validators have a strict 1-to-1 relationship (an entity will only ever be validated by a single validator, and a validator will only ever validate a single entity), it might make sense to embed the validator directly inside the entity.

The Veto Model extension does exactly this, associating your entity with a specified validator, and adding 3 additional methods to your entity: valid?, validate!, and errors
```ruby
class Person
    include Veto.model(PersonValidator.new)
end

person.new
person.valid? # => false
person.errors.full_messages # => ["first name is not present", "last name is not present"]
person.validate! # => Veto::InvalidEntity, ["first name is not present", "..."]
```


why is json lower case here and all caps everywhere else?


why don't you add this to SceneListener instead of creating a new GroupListener?


> Maybe a better way could be to throw the right exception and force a failed when the configuration is checked if the macro is not defined

yep, that is the right thing to do imho.
I've not more read the code but what happens in the cases where you 'return failed_output' if you instead raise an exception (say ValueError (with a nice message)) at the place of the return ?


you will need to familiarize yourself with JSON


How to make a plugin


This require-package function comes from the example-config configuration, not Emacs core.


On else code segment you have to set "loaded" property with "oProgress.loaded" value not "loaded" as "var loaded" is defined on "if" scope.


Excuse me, I'm going to pay attention better.
You must also learn to GitHub, I'm still an amateur :)


Can you `delete baseOptions.updateWhileAnimating;` before calling the base constructor? (see #3104)


done


Short array syntax would require PHP 5.4+.

The [download page](https://fluxbb.org/downloads/) says:

> FluxBB v1.5.7 requires PHP 4.4.0 or later.


up and running.
Use the following command to run it:


Yeah, sorry about that.


To not rely on a protected attribute and to hide the parent class inner behavior, I think it'd be more useful add a new method to GroupEditor, like panTo(x, y), that does this, and then invoke it from here.


:+1: thanks


Ping @SebastianMarzjan @adamkarminski


Today, we're doing something like:

```
<script>
Morris.Donut({
  element: '...',
  data: { /* this is the only thing we should care about here /* }
})
</script>
```

The thing is, we're coupling the data with the behavior. The view should only output the data, and the behavior should be inside a separate javascript file, like so:

```
<script>window.visitorGraphData = { /* the json that we care about */ };</script>
```

And in a separate js file, you could do something like:

```
Object.keys(window.visitorGraphData).forEach(function(key) {
    Morris.Donut({ element: key, data: window.visitorGraphData[key] });
});
```


To write the code that defines the Turtle blocks defined in your plugin, you will need to know how to program in Javascript. The blocks are defined in a dictionary element. An example is found here:


I don't know if you are aware and if it matters but Visual Studio told me the following :
> DSoundStream.cpp(408): warning C4018: '<' : signed/unsigned incompatibility


A heads-up:


Basically, it would show from 0px to 960px and hide at >960px.


This needs quotation marks around the `CHTML-preview`, since the dash would otherwise be illegal as a hash key.


shouldn't this also have `bower`?


This handles the SceneEditState, so maybe SceneEditListener is a better name.


Basically, it would hide from 600px to 1200px.


Whoops, thanks, good catch! Have been coding too much PHP 5.4+ stuff lately. =)


Yeah I can live with it. I was more or less wondering if there is a "release" script that would take care of compiling all of the assets before you distributed it. That was more in line with what I was thinking


Is it because of absolute path or because of drive letter ?


Thanks, now I understand


此处有一笔误：“封开”


I was thinking about that. what should it do though, or what should it match?


What is strange is the use of scriptsOfLayout, should we remove it to include a tag script on html ?
This way to optimize performance isn't obsolet with defer attribute ?

Why the controller return an array ($report['subscribed'] and $report['unsubscribed']) instead a json ({datas:[{value: {$report['subscribed']}, label: 's("SUBSCRIBED")'},{value: {$report['unsubscribed']}, label: 's("Unsubscribed")'}]}), where labels should be token to be translated ?

Same thing for AppVersions, I think that the json should be created on controller, there's no datas that need to be processed on view.


all tests pass:
success: http://buildbot.rust-lang.org/builders/auto-mac-32-opt/builds/3140
success: http://buildbot.rust-lang.org/builders/auto-mac-64-opt/builds/3133
success: http://buildbot.rust-lang.org/builders/auto-mac-64-nopt-t/builds/3134
success: http://buildbot.rust-lang.org/builders/auto-linux-32-opt/builds/3125
success: http://buildbot.rust-lang.org/builders/auto-linux-32-nopt-t/builds/3125
success: http://buildbot.rust-lang.org/builders/auto-linux-64-opt/builds/3128
success: http://buildbot.rust-lang.org/builders/auto-linux-64-nopt-t/builds/3121
success: http://buildbot.rust-lang.org/builders/auto-linux-64-x-android-t/builds/3116
success: http://buildbot.rust-lang.org/builders/auto-win-32-opt/builds/2793
success: http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/2791
success: http://buildbot.rust-lang.org/builders/auto-win-64-opt/builds/1629
success: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/1622


fast-forwarding master to auto = 896cb36e


I "think" it only matters for windows style absolute paths.


globs. With a replace you could do multiple things, exact a glob, detect a glob, remove a glob, etc. This could depend on the regex. Just a suggestion though. I don't really care :)


# @ woocommerce-delivery-notes
#: ../templates/print-order/print-content.php:71
msgid "Price"
msgstr "Prix"


"disables" rather than "disable".


hahaha, foi mal, eu quis dizer que eu estava satisfeito com a alteração.

Em 14 de janeiro de 2015 11:13, Willian Justen <notifications@github.com>
escreveu:

> @LFeh <https://github.com/LFeh>, ah tá ahuahuahu
> O @felquis <https://github.com/felquis> falou ali, eu num tinha entendido
> nada ahuahau
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LFeh/1-post-por-dia/commit/7f9a979aa8ce283c4681d2ecf1c83098d9b3ff95#commitcomment-9278066>
> .
>

--
Felquis Gimenes - Front-end
felquis.com +55 011 95980 2233


`if $d(aliases(command),command)`
Is this fun way to write `set command = $Get(aliases(command),command)`  ?


To do this, you could use the method panTo (see comment below) with animate to false.


Merged windows test fix.


"*a* two-pass rendering mode" (missing "a")?


asdf


here is trailing space


This should be private, once commentes of next commit are addressed.


I augmented the valid proportion. I set them to about 3 epochs. save is 3x bigger the validFreq.


Comments for update of french translation


I am not sure this is feasible because of the recursive call to `loadHandler` here https://github.com/ibm-js/requirejs-dplugins/commit/ddc707c3e2f65fb8238f2519457def9004a4cec1#diff-2b85e238ffb3010183809ed616d824bfR69


fix such pep8 errors


This is to make it fast enough to train for a tutorial. We can revisit this next week. I make an issue about this.


Is line 535 necessary?


"allows" rather than "allow".


This breaks the capitalisation on lang pack statuses.
Is it worth adding another string for this, or is life too short?

![screen shot 2015-01-14 at 14 24 47](https://cloud.githubusercontent.com/assets/226578/5740226/590d1090-9bf9-11e4-8df5-224613c4bf25.png)


It looks like this commit broke 1.7's j2ee build on Travis: https://travis-ci.org/jruby/jruby/jobs/46985622#L853

Looks like some dependency is compiled to Java 7 bytecode.


This is actually for the **second** pass that goes on behind the scenes.


yes :-)


Yes, it is. If you ommit it __slots__ disappear and the object gets a __dict__ again.

See [the Python data model](https://docs.python.org/2/reference/datamodel.html#slots):

>The action of a __slots__ declaration is limited to the class where it is defined. As a result, subclasses will have a __dict__ unless they also define __slots__ (which must only contain names of any additional slots).


I'd add:
```
if (sceneEntity != null){
```

It's kind of the same, but adding the `if` makes clearer that you are deleting the listener from the previous scene, if any.


Hello World


You might want to note that this is for the messages during the **second** pass.


You might want to note that the user can still enable fast previews via the contextual menu.


Should probably be fixed for consistency but it's not an issue. Since `i` is always greater or equal to 0 and `pNumAvFrames` is never above `2147483647`


If I read correctly, this change also alters how `DECL_INTERFACE` is parsed for C++, is that expected?


saw approval from nick29581
at https://github.com/michaelwoerister/rust/commit/e087c53540ec35c80cefe487264e918f21d74033


merging michaelwoerister/rust/sane-source-locations-pt1 = e087c535 into auto


status: {"merge_sha": "1a1f09111fe458ae1be4323c29c22b0d7a60f66a"}


michaelwoerister/rust/sane-source-locations-pt1 = e087c535 merged ok, testing candidate = 1a1f0911


Schaut gut aus.
Nico möchte weitere Gütezahlen q.

Ich denke, er hat erklärt, wie man diese errechnet. Schicke mir bitte den Scan der Abgabe hierfür zu!

:)


i want more crap back.


Commented via CURL


ParseFile returns %Status if there was error during parsing


that definitely helps, but the top-right and bottom right of the pill is squared off while the left top/bottom is rounded.


can you remove the final in filled?


Ok, good.  Just making sure.


`http://guides.rubyonrails.org/association_basics.html#options-for-has-many-class-name`를 보면, `contributors`를 이렇게 인스턴스 메소드로 쓰지 않고 `association`으로 정의할 수 있음


I'm not sure that clearArray should be applied in every array in grid. The most arrays like selarrrow , savedRow and others are build internally and do not contain any reference. Be a carefully with this. This can slow down the building of the grid.

Regards


Respekt


fill is probably a better name, and javadoc should be revise (I know it's not from this PR, but since it's there)


Regarding the technical point, currently you need one subtype per functor (even for ones that you already have generic functions for), so it's not obvious that's any better.


hehe, yep could be possible bug in qt, or even in the opengl/driver implementation handling incorrect formats? should we file a bug in qt tracker (if thats possible)?

I added a commit, I left the format to SingleBuffer, but also removed the setAutoBufferSwap completely. Since this is true in default, even if the machine cannot accept SingleBuffer video should work.

For me it seems the video is more responsive in SingleBuffer mode, but also more sensitive to the "rolling shutter" of my webcam. So i'm undecided which mode is better.


``gittio info com.alcoapps.actionbarextras``
Tells me that 1.5.0 is the latest version :(


You need quotation marks around the `content-mathml` here.


adding comment from api


Currently browserify can end the pipeline before remapify calls `b.transform()`, resulting in the error. This code is significantly different in that it adds the transform before the pipeline is ended, avoiding that error.

As you said, that leaves the issue of whether the transform function can be invoked before aliaser becomes defined. I'm looking for feedback on that. In my testing things work even if I introduce a long (say, 10 second) delay into the completion of the globbing (and therefore availability of aliaser). I'm fully prepared to accept otherwise if someone can createa  reproducible test case that shows it or clearly elucidate the control flow in a way that demonstrates it.


+1 on play it by ear, readability is the key.  I've added in additional brackets as requested and a statement around brackets in it's own section


@jython234 Get the pid and force convert variables to the packet class, like this:
```
int pid = 0x13; //Just for test
Packet packet = receivePacket(); //Example method
switch(pid){
  case 0x13:
    SomeSpecificPacket pkSpecific = (SomeSpecificPacket)packet;
    pkSpecific.someField();
    break;
}
```


Yeah I spotted that the other day and forgot to mention it, thanks for the nudge.

Presumably Phil changed this string because it's used elsewhere in the system and it looks odd there. We'll have to decide which instance requires the fewest changes and add a new string for that use case.

So where else is it used other than the languages panel, Phil? Should be simple enough to introduce a new string where needed, but shout if you'd like me to do it.


r+ rollup


I would prefer:
```
var options = {
  data: el.attr('id') + '=' + encodeURIComponent(el.prop('checked') ? 1 : null),
  no_encoding: true
};
if (el.attr('data-miq_sparkle_on')){
  options['compete'] = ....
 ...
}
miqJqueryRequest(url, options);
```
Also: have you researched, that when there's on, than there's also off? I mean, don't those two to be tested and set separately?


This is a weird way to do a link...


Fixed


우끼!


other.to_h instead of exposing response!


To make this more general, this should take as parameters originX and originY, and forget about width and height, since we only have access to them in some elements.


Because I don't think anyone else is using it, but I can revert the pull request and fix, or maybe we should deprecate this modules? I'm not sure :/ @marc-hanheide any thoughts?


or just $p(tFileName, "/", *)


Couldn't this be sent more than once if there are multiple EP queries in one page load?


Acked-by: Niels de Vos <ndevos@redhat.com>


Replace class var @@days_in_month with a class instance var.<br>Surrounding space missing for operator '*'.


Extra empty line detected at body end.


Missing space after #.


Width and height being originX and originY doubled is not a valid assumption, that's why it's better if createCenteredEntity takes the originX and originY as parameters.


그리고 이렇게만 하면 한 유저가 같은 해시태그를 여러 번 쓴 경우에 중복으로 계속 나오겠지? https://github.com/fools-gold/crossroads/blob/master/app/models/team.rb#L4 를 참고


thanks @hooverlunch I fixed it in development branch. https://github.com/TID-Lab/aggie/commit/f7c44cd3182b2103e8352ff65e76acd70a38b8ed


Que te falta un intro, van 2  ¬¬


Vainglory


sure


replace :response with :type


Simple, yes - but necessary? Am I being to pernickety that one word is caps and one is lower case?


Eigen sollte eigentlich jeder selbst installieren (können), daher wäre ich eher dafür das lokal zu halten und es aus dem git repo zu entfernen...


Can this set of values get changed by change in the supported values in DB?


Yes, I've updated 30 mins ago. Gittio needs up to 24h to trigger the update in its registry.


r+ rollup


@JacobH1992 Add `.DS_Store` to `.gitignore`


Yes


@thinkindeep, the p2p operations, including nesting have been fully validated against HW and our CI tests run p2p op tests against multiple simulators for regression testing. Let us know once you integrate and test if you run into any issues.

- Greg


Done in https://github.com/jruby/jruby/commit/e8ca19d36de46320a03f0e380888ab64aac94bd3.


Make sense!
I'll wait no worries.
Thanks for your good work and your reply.
I'll let you know if I'm experiencing any problem with that version ;)