Why extend RuntimeException rather than Exception? Would we want this bubbling out of a clients code without them realizing it? If not, then make this and the interface extend Exception and throw it from the method.


Ok, that's fine. Thanks.

For some reason I couldn't get Lattice to start by following the instructions on https://docs.docker.com/installation/mac/, but that's not your problem.


You mean the lambdas part in the async guidelines section? Or the general lambdas that I haven't migrated/written yet?

As a meta point, are you asking if we can create links directly within the example code itself?

I suppose I could say as a note above or below the example in https://github.com/hhvm/user-documentation/blob/master/hack/async/extensions.md#streams that this example uses lambdas and link from there.


This condition may not work with the Daily PvP 1, where 2/3 has the progress status as 1 (50%), but it will work with Daily Crafting Equipment 2, where 1/3 and 2/3 return the progress status as 1 (50%).


Лучше назвать `FUTURE_EVENTS_RANGE_IN_DAYS`


OK let me think.


All players are exclusive (mpd, cmus and which one playerctl picks up too), however there are several "exit 0" scattered through the script (including one in the `mpd` subroutine), so this guarantees the exclusivity.

Before this commit, what happened was:

- script enters in the first if (because player_arg equals the empty string)
- there is no music in the playlist of mpd, so `mpc current` returns ''
- the script exits

In this case (player_arq being ''), I believe it is better to pick the first player it finds with a valid output. That's why I had to remove branching (the `else`s) altogether.

Was my explanation clear?


Nice :)


I już działa bez majla!


Add 812: +    SERIAL_ERRORLNPGM(": Spindle switched off. MAXTEMP triggered !");


Is it possible to get content on why you would want to disable packagist?


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (750 of 750 tests passed)

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


Не нужно делать урл руками. Используй org.apache.http.client.utils.URIBuilder


You are right on this. And am the culprit. :)


Хост с картинками должен конфигурироваться из properties файла.


Which cases are failing now?


iterate over List and ArrayList


for the iTypeElement the out-put is below:
interface Collection {key=Ljava/util/Collection<Ljava/lang/Integer;>;} [in Collection.class [in java.util [in /Library/Java/JavaVirtualMachines/jdk1.8.0_31.jdk/Contents/Home/jre/lib/rt.jar]]]


As mentioned in comment. This is just jet... todo. Check docs and make it correct.


The point is that it should be a while true *unless* we were given an empty `plannedNodes` to start with. If we were given an empty one then this whole loop can be skipped. A shortcut for `if (!plannedNodes.isEmpty) while (true) {`

Not a bug


let's name this appropriately too.. and also disallow if the method is anything other than "GET" , also disallow for "GET" on bucket only operations.


Эта логика должна уехать в другой класс


I am working on tests now. createTable and datatypes already done.


Тут было бы удобнее использовать response.getWriter().write


This is dev_test.go, not the main program. This import didn't follow haz-ua-consumer.go so I changed it. Maybe it's not a good idea to add this change in this pr. (But when?)


This directory can be found in ckaestne's GitHub repository.


91977808_LBNL_NewVRFModel (rongpengzhang) - i386-Windows-7-VisualStudio-12: OK (1743 of 1784 tests passed)

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


Почему hypernavis, когда hypermarkets?


Лучше назвать `sortEventsMap`


Сейчас вообще полный отстой происходит - один и тот же код в двух местах


Strange, ```make testall``` works fine, but TravisCI didn't like this fix.


That's the default on Object so there's no point overriding, right?


I suggest we don't every use `let()` and define instance variables in `before(:each)` instead. @instance_vars are visible in code. When I read it, I can instantly tell that this one is not local and affects many unit tests and that I should be looking for the definition in the `before(:each)`. Using `let()` is very confusing because then the variable start to look just like any other local variable.

Please check the rest of the code and change it accordingly.


@thotro Hi !
With 5 bytes for DW1000Time instead of 8 bytes we could go to 10 devices without problem !

I did a lot of optimization regarding to variables which were declared in float whereas it's not necessary. Also for the table of _bias which was in float (so 17*3*4 = 204 bytes and I decrease it to 64bytes in tcheating: passing all the float number in int (in multiplying by 10 or 100) and dividing them by 10 or 100 after. I also removed one DW1000Time in DW1000Device which was not necessary.

For ranging:
Do we have something to do for the tuning of antenna or something like that for the ranging ? The ranging is really stable but I don't know if the real distance between modules fit really the reality. Do you know what we could do about that ?

Cheers !
Léopold


Couldn't think of word, thought I'd get away with it. :- )


Thanks @chrismccord! I moved this over to the site, just fyi.


Надо еще как минимум залогировать эксепшн


```
testLoopIterateOverList(edu.cuny.citytech.foreachlooptolambda.tests.ForeachLoopToLambdaRefactoringTest)  Time elapsed: 0.057 sec  <<< FAILURE!
junit.framework.AssertionFailedError: Precondition was supposed to pass.
```


Well, apparently it didn't have any impact. :stuck_out_tongue:


> ( I hope you don't mind me commenting here.)

Not at all, thanks for chiming in! You're right about the match:

```
require 'benchmark/ips'

string = "foo".freeze

Benchmark.ips do |x|
  x.report("match") {
    /boo/.match(string)
  }
  x.report("=~") {
    /boo/ =~ string
  }
  x.report("[]") {
    string[/boo/]
  }
end
```

results in

```
Calculating -------------------------------------
               match    66.837k i/100ms
                  =~    74.183k i/100ms
                  []    71.748k i/100ms
-------------------------------------------------
               match      1.783M (±12.8%) i/s -      8.822M
                  =~      2.077M (±13.0%) i/s -     10.237M
                  []      1.910M (±13.1%) i/s -      9.399M
```

Looks like switching to `any?` would also give a speed bump:

```
require 'benchmark/ips'

string = "foo".freeze
array = [/zoo/, /boo/, /moo/]

Benchmark.ips do |x|
  x.report("detect") {
    array.detect {|regex| string =~ regex }
  }
  x.report("any?") {
    array.any? {|regex| string =~ regex }
  }
end
```

```
Calculating -------------------------------------
              detect    31.241k i/100ms
                any?    37.831k i/100ms
-------------------------------------------------
              detect    489.087k (±10.2%) i/s -      2.437M
                any?    616.260k (± 9.5%) i/s -      3.064M
```

Could you put that in the change as well? Didn't realize there was a speed difference. Here's the source:

This is used for any?

```c
static VALUE
rb_ary_any_p(VALUE ary)
{
    long i, len = RARRAY_LEN(ary);

    if (!len) return Qfalse;
    if (!rb_block_given_p()) {
    const VALUE *ptr = RARRAY_CONST_PTR(ary);
	for (i = 0; i < len; ++i) if (RTEST(ptr[i])) return Qtrue;
    }
    else {
	for (i = 0; i < RARRAY_LEN(ary); ++i) {
	    if (RTEST(rb_yield(RARRAY_AREF(ary, i)))) return Qtrue;
	}
    }
    return Qfalse;
}
```

This is used for detect:

```c
static VALUE
enum_find(int argc, VALUE *argv, VALUE obj)
{
    struct MEMO *memo;
    VALUE if_none;

    rb_scan_args(argc, argv, "01", &if_none);
    RETURN_ENUMERATOR(obj, argc, argv);
    memo = MEMO_NEW(Qundef, 0, 0);
    rb_block_call(obj, id_each, 0, 0, find_i, (VALUE)memo);
    if (memo->u3.cnt) {
    return memo->v1;
    }
    if (!NIL_P(if_none)) {
    return rb_funcallv(if_none, id_call, 0, 0);
    }
    return Qnil;
}
```

I guess the extra speed comes from not having to allocate an enumerator and the extra layer of indirection? Could you give me a PR and @-mention me to change this to a `str =~ regex` and change the method to `any?` since you found the change?


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-MacOS-10.9-clang: Build Failed

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


I was wondering if someone could finish this project. I would really like to use an updated version on my server.


It would be a good idea to introduce 256 as a constant (e.x. ALPHABET_SIZE).
This way it will be easy to extend algorithm to non-ASCII strings just by changing the constant.


OK, here's the link to the input file: https://github.com/mdarefin/Convert-For-Each-Loop-to-Lambda-Expression-Eclipse-Plugin/blob/master/edu.cuny.citytech.foreachlooptolambda.ui.tests/resources/ForeachLoopToLambda/testLoopIterateOverList/in/A.java.


Metodologia para Constituição do Colegiado da Conscieciologia

Mas e se várias pessoas quiserem alterar ao mesmo tempo?


В общем, ждем https://github.com/amosov-f/hypernavi/issues/9, чтобы количество жести поубавилось


Note that I've tested with MinGW and MSVS2013 so it is pretty safe ^^


This should pass because `List` extends `Collection`, correct?


Which means it has the `stream()` method.=


These arrays will all sit in Eden space and as their ref-count will be zero they can be trivially cleaned up in the "throw out everything in the Eden space" mode, which is faster than C's traditional malloc/free

You cannot have a lock free algorithm without trading CPU/memory for the lock free, but the side-effect is that there is no contention and hence no risk of deadlock...

Is this the final implementation here? Nope this level of lock free algorithm is not designe for massively parallel operation. It will suffice for the max 3 threads that could hit it (one removing nodes, one adding nodes and the third calling getPendingNodes). If we expected more than 15-20 threads hitting this at a time we might need a different algorithm to have lock free, but that level of complexity is unwarranted here for now.


Is that correct?


Id suggest to use
```python
query = self.model.query.filter(self.model.id.in_(ids))
```


Simply using the encoding NSUTF8StringEncoding didn't work in our project. I had to use the approach from the `joinString`Method: Build the String by appending individual UnicodeScalar Characters.


Simple things are complicated. (And complicated things are worse)


"URL" - плохое имя для этого поля. Нужно в имени отобразить, чей это урл


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 275](https://teamcity.labkey.org/viewLog.html?buildId=265038&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:52
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


you misspelled cofhcore


Unimplemented: after close(), current worker should be destructed.


wichtig! weiter so!


Тут можно return; поставить, чтобы следующий else можно было не писать


Du Lauser!

Sent from a mobile device.

Von:"geworfener" <notifications@github.com>
Datum:Do., 6. Aug. 2015 um 22:25
Betreff:Re: [democratizer] reworded tag line (d3c3afa)

wichtig! weiter so!

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


This is probably most famous typo from Colobot 1.9/PL released by Manta. I can't believe we fixed that so late :D . Well, even I forgot about that.


Typo here -- should be config/dpla.yml.example


I meant shareasale.com tracker in the commit >_<


We don't use it, it's used internally by minimongo to deturmine if an operation is valid for a client to perform, for example it's not valid for clients to do this: `collection.update({}, {$set: {hacker: true}})`. Here we modify the helper just slightly so that it checks for an id, but does not reject selectors which also have other properties.


A NSFW venture, obviously.


Ensure


A NSWF venture, obviously.


Hm, wouldn't it be much clearer to make NagVis (or the webserver) answer with CORS HTTP headers to make cross domain ajax requests possible?


this.GetHashCode(), other.GetHashCode()


This commit addresses the ``to_dot(::Pair{UTF8String,Any})`` method error encountered when testing the package on julia 0.4.

The issue is that ``collect(attrs)`` returns an array of ``(UTF8String,Any)`` on julia 0.3 and an array of ``Pair{UTF8String,Any}`` on 0.4.  The ``Pair`` type is not the same thing as a ``Tuple``.  Thus, the associated call to  ``to_dot(attr_tuple::@compat Tuple{UTF8String, Any})`` fails on julia 0.4.  Furthermore, the ``Pair`` type does not exist in julia 0.3.

The solution here, which works on both versions of julia, is to add and call the new, two-argument version ``to_dot(attr::String, value)``.

Note that the old version of ``to_dot`` is no longer called within the package and could be considered for removal or deprecation.


Вообще, стоит какой-нибудь класс HypermarketSerializer написать. Чтобы вынести отсюда эту логику


Have you checked to see how robust this code is at handling all valid ways of calling the relevant methods? For example is it valid to call `collection.remove(function (error) { ... })`?


Ups fix is correct, but master is failing.


add "epel_packages" to this note?


With this change the function seems to ignore the `width` parameter completely.


A nice variety of literature :)


There is a function in standard library called
```cpp
  std::fill(beginIt, endIt, value)
```
In this case the call would look like
```cpp
  std::fill(occurred, occurred + ALPHABET_SIZE, false);
```

Alternatively, you can use
```cpp
std::vector<bool> occurred(ALPHABET_SIZE, false);
```
but be aware, that this will involve allocating memory on the heap, not on the stack, which may be a bit slower.
Though, if stack size is limited (usually it's around 1MB), this is the only option.
So if you are going to allocate an array of >10^5 elements inside of the function you should go for vector.


Thank you! :fire:


If so, I am looking at the code in: https://github.com/mdarefin/Convert-For-Each-Loop-to-Lambda-Expression-Eclipse-Plugin/blob/master/edu.cuny.citytech.foreachlooptolambda.ui/src/edu/cuny/citytech/foreachlooptolambda/ui/refactorings/ForeachLoopToLambdaRefactoring.java#L159-L163.


yes,


It should throw an AmbiguousMatchException.


Regarding the `List` case, you are retrieving all the interfaces from the supertype hierarchy but you are not doing anything with them.


Are you sure this test is deterministic? In other words, are there any cases where the book 'Fiction Book 2' would show up instead?


Hh_single_typecheck is another option that might be better suited to examples than the full server

--
Typed with my thumbs

On Aug 6, 2015, at 08:11, Joel Marcey <notifications@github.com<mailto:notifications@github.com>> wrote:

:)

Let's say

—
Reply to this email directly or view it on GitHub<https://github.com/hhvm/user-documentation/commit/aaeeaf7d3d7d4f9fa419f202e16ab3168921b0cd#commitcomment-12567737>.


Did you want to check them for something?


I find this way of calculating `'0' + digit` a bit creepy. It does work for `digit < 9` because '0' is 0x30... but why not just use `+` operator? :)


Thanks, removed.


i want to check if this interface are instanceof or part of Collections


You could modify your test to match either book: `test.matches(doc.title, /Fiction Book/)`


Nice challenge! This is a markdown file, which means you don't need to (and maybe can't) type HTML in directly. To make something look like a header, you just have to do this:

`# Prime Number Checker`

The pound sign means h1 in markdown.


Thanks, made it explicit.


Тут не должно быть логики про то, откуда мы гипермаркеты берем. Сюда должен @Inject-иться абстрактный HypermarketProvider. Одной из его реализаций будет провайдер, читающий гипермаркеты из ресурсов.


I'd consider making the program not require the user to type anything, but instead take the number as an argument when you run the program. (e.g. you'd run this in the terminal: myprogram.exe 13)


ok


Good job on using the null backed collection, tests should generally use a null backed collection if we're only interested in in testing our functionality (e.g. we're not testing publish functions or security rules)


Same thing here. Just need to type `# BMI Calculator`


> Can someone explain me what the addition of the hard-coded "sketch" folder for output files is good for?

The goal is to prevent conflicts between files in the sketch and files in the core or libraries. IIRC libraries are compiled into a `libraries` subdirectory, so consider a 'Foo' library with a 'foo.cpp', which will create a `build_path/libraries/Foo/foo.o`. Now consider a sketch that contains a `libraries/Foo/foo.cpp` file (so inside subdirectories inside the sketch). Without the `sketch` subdirectory, this file would *also* generate a `build_path/libraries/Foo/foo.o` file. Using the `sketch`, `libraries` and `core` subdirectories, any conflicts between them are prevented.

Using a variable in platform.txt for the sketch path seems reasonable, though I haven't looked closely.

@ffissore, would it perhaps be an option to store the source and .o files inside `sketch`, but put the final linked .elf and .hex files directly in the build folder? This might fix most backward compatibility problems I believe, and it would still make sense (and not cause problems)?


Just want to verify that nothing in this commit affects anything from the Brent's algo stuff. I want to move the clamping and tau adjustment into internal app code so I'm going to take this commit out of master.


Ugh, the sensible way to write this with iterators *isn't* optimized well, not even when you break it down to a single `&[u8]` passed in and returned directly (example: https://play.rust-lang.org/?gist=24bc9155c80c05e08a8d&version=nightly). However, just having a struct at all causes the same problem (even with the ugly imperative loop), so maybe it won't make much of a difference. It will take a while until I can benchmark this.


@mfonville I will be on vacation from tomorrow too, but maybe I can take a look. Since I was not following the development in the latest month, I missed the upload_sources.sh genesis. Have I correctly understood that files removed by the installapk function (line 109) need to be passed to "git rm" (line 34 of upload_sources.sh)?


You can use `test.equal(_.pluck(docs, 'title'), ['Fiction Book 1', 'Fiction Book 2'])`


I wanted to use String[] getSuperInterfaceNames() and then check,


I don't have one handy as my main test case was the AudioKitDemo that ships with the current version of AudioKit. The way we would exhibit the bug was by sending a bunch of notes to an instrument - after a while (10 to 20 notes) we would start getting these PERF errors. Just undoing that one line made this problem go away. I noticed that the crashes/errors where around usage of that `insdhead` member, so that's why I looked closer at recent changes in Csound.

I suspect there may be some kind of race condition or other memory issue as a side effect of this change. I'll try to grab the CSD that AudioKitDemo uses to set up Csound and post it here.


Can you use net.Error instead of defining your own interface?


Again, I'd consider making the app take the users's height and weight as arguments, so you could just run this in the terminal: myapp.exe 6-2 175


In markdown, you don't have to type HTML, do do bold, you can just do this: `*Bonus exercises:*`


the actual first bug was a casting problem of: https://github.com/imglib/imglib2-ij/blob/master/src/main/java/net/imglib2/img/ImagePlusAdapter.java#L75 (it was `UnsignedByteType`  but used as `BitType`). The `Converter` is a workaround (lazy).


Did you forget to bump transporter -v ?


And you don't have to do uls. You can just do this:

    - See if you can ask the user...
    - After this, see if you can round...


as always, yes, thank you


Why extend RuntimeException instead of Exception? Do we want to allow the exception to bubble up out of the client's code without their knowledge? If not, then extend Exception and update the interface to throw GeoEntryQueryException (or create a GeoEntryException base class that GeoEntryQueryException, GeoEntryIndexException, etc. all extend and throw that.)


dangerous, use gson for generating the json


Комбинация клавиш Alt + Ctrl + I (https://www.jetbrains.com/idea/help/code-style-and-formatting.html) тебе в помощь


Hey you around? Trying to get in touch with you on bitcoin/litecointalk


```java
getListAddress(eventId, email, name, language).ifPresent(listAddress -> {
    getApiKey(eventId, email, name, language).ifPresent(apiKey -> {
        send(eventId, listAddress, apiKey, email, name, language);
    });
});
```
is better I think :D


Your tests would be more robust, and less likely to interfear with eachother if you created a new null-backed collection for each test. You could create a generator function to make this simpler:
```javascript
makeCollection = function () {
  var Books = new Mongo.Collection(null);
  Books.insert({ title: 'Fiction Book 1', kind: 'book', genre: 'fiction', catalogId: 0 });
  Books.insert({ title: 'Fiction Book 2', kind: 'book', genre: 'fiction', catalogId: 1 });
  Books.insert({ title: 'Science Book 1', kind: 'book', genre: 'science', catalogId: 2 });
  Books.insert({ title: 'Science Magazine 1', kind: 'magazine', genre: 'science', catalogId: 3 });
  Books.insert({ title: 'Fiction Magazine 1', kind: 'magazine', genre: 'fiction', catalogId: 4 });
  Books.insert({ title: 'Removable 1', kind: 'kind1', genre: 'removable', catalogId: 5 });
  Books.insert({ title: 'Removable 2', kind: 'kind2', genre: 'removable', catalogId: 6 });
  Books.insert({ title: 'Removable 3', kind: 'kind2', genre: 'removable', catalogId: 6 });
  Books.insert({ title: 'Non-Removable 1', kind: 'kind2', genre: 'nonremovable', catalogId: 6 });
  Books.insert({ title: 'Removable 4', kind: 'kind3', genre: 'removable', catalogId: 6 });
  return Books;
};
```


It seems that usage of iterators is redundant here. Usually you want to use them if you're doing inserts and erases in original array because they take iterators as arguments.
Also you would use iterators to traverse containers that has no convenient random access methods (e.x. set, map, unordered_map).
In other cases using integer indexes can be much more readable.
You can also easily convert back and forth between indexes and iterators in constant time for random access iterators (that's true for std::vector, std::string):
```cpp
int middle = s.size() / 2;
auto middleIterator = s.begin() + middle;
int indexOfIterator = middleIterator - s.begin();
```


лишний файл


ОК!


"a schema version with which to compare"


No, these bounds are by me. They are bounds for the inputs that Bellerophon can handle without running into trouble with overflow or underflow, which is not considered in detail in the paper.


Не самое лучшее название. Positioned смотрелось бы лучше


у меня пишет что не влезает в лонг :(


logic is quite similar to onTicketAssignment ? Could be refactored ?


тут и далее во всех случаях, когда встречаются файлы CSS, образованные из LESS-компонентов: лишний файл.


несоблюдение кодгайда


Зачем тут айди?


Just to explain (even if I will remove the `FeatureSet` for now):  All other `Op`s which are called from this `FeatureSet` should also be using the cached version of `OpService` (= `FeatureOpService`), such that everything becomes trillion times faster ;-)


+ не подумал, спс


int не бывает @NotNull. Идея со мной согласна


размер — в переменную!


this stuff should not be in a custom service and should be done directly in the listener


It's not obvious to me from the wording that the "literal block" doesn't include the preceding `::`. It has a certain consistency if you think of the `::` as a blank paragraph, but that's not at all intuitive. Either way, it's fixed in #12435.


я сортировал массив(во 1 мне было интересно во вторых я не придумал как по другому находить большие значения)


все подобные правила: размер — в переменную!
или получать мат. операцией от какого-то базового размера, записанного в переменную.


Внутри модуля `project-config.js` используется `dir-glob`, поэтому теперь эти предупреждения выводятся всегда.

@blond Можно ли выпустить патч версию без этого вывода?

Советую в будущем не использовать `console.log` (и почистить такую же проблему в других местах) для вывода депрекейтов (в случае использования ansi-последовательности), так как заранее неизвестно какой `tty` будет на пользовательской машине.

Ко всему прочему, это можем испортить формат для не `console.log` вывода, если например используемый логгер что выводит сообщение в xml'е.

--
Ну и три восклицательных знака?


Should use conditional variable instead of thread sleep.


In terms of each `iType2`, what do you want to check?


We should not cache the result, instead we should progressively go up the parent chain, resolving each query as we go, and only use the query for the current invocation.


Nit: this should probably say "A subset of the codes used in the RFC standard of SOCKS version 5."


This logic won't cascade correctly.  Instead, you should use:
if (this.schema.since(InfinispanSchema.VERSION_3_0)) {
    throw ParseUtils.unexpectedAttribute(reader, i);
}
ROOT_LOGGER.executorIgnored(CacheContainerResourceDefinition.ExecutorAttribute.LISTENER.getDefinition().getName());
break;


Yep, done.


Эта штука должна быть типизирована
MapStructure<T extends GeoPointed>
И выдавать в методах T, а не GeoPointed


We are green on travis, but not my machine. Which is weird.


*nix vs osx?


What remains? The text area editing? Thats the only remaining thing i have on my list.

> On Aug 6, 2015, at 13:01, Jaka Kranjc <notifications@github.com> wrote:
>
> On Thursday 06 of August 2015 10:04:27 Brad Allred wrote:
> > Finally! looked at that like years ago :p
> >
> > makes me want to go though the code even more to privatize members... just a
> > ton of tedious work that i never seem to find time for :)
> First, please find some for the hopefully last few regressions from the text
> rewrite.
>
> This is what works for me with the borderline useless reports on Coverity
> (importer stuff). Each time it catches a new valid error, I fix more than that
> one report. I don't what would be a good cue in this commit's case though.
> —
> Reply to this email directly or view it on GitHub <https://github.com/gemrb/gemrb/commit/c238682afa9754d57fabf616229eddec9a96e358#commitcomment-12573156>.
>


Streams!!! :+1:


This is for demonstration only, ignore contents otherwise.


Sounds like a clean solution. Would need to provide an example json and add a comment in the header.


Same for other executors.


Remove this -- we only need to log once to show how it works.


revert


Опять тут проглатываемые эксепшены. Такие эксепшены - баги. Их нужно оборачивать в RuntimeException


Ok. If it is generalized into file asset I guess it don't matter either way
to me!
D

Le Thu, Aug 6, 2015 à 15:26, divmazie <notifications@github.com> a écrit :

> I know we talked about not doing it this way, but WP's query functions are
> a total mess, and I know this way works. At least I've generalized it this
> time so that it is all being done in WordpressFileAsset, and will work fine
> with both encodes and zips.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/divmazie/jonathancoulton-theme/commit/db8eee9e0dd3ddbd46a064cb179eade102539f97#commitcomment-12573602>
> .
>


you don't need to store the tracewriter -- just leave it config.EnableSystemDiagnosticsTracing();


I've published a new v0.1.0 release


the for loop was for the print out to check what kind of data I was getting, Since we are getting the array of iTypeHeirchay, we could iterate over it and get the superInterfaceName using the method, and then check with if it's part of Collections,


why you comment this line?


Because we want to lazily resolve the query, we can't extend the query here, instead write your code like this:
```javascript
collectionView._narrowingQuery = {query: query};
collectionView._narrowingParent = parent;
```

then you can write your `_mutateSelector` function like this:
```javascript
var collection = this;
// query is defined as an argument
while (collection) {
  var narrowingQuery = collection.__narrowingQuery.query;
  if (_.isFunction(narrowingQuery))
    narrowingQuery = narrowingQuery();
  _.extend(query, narrowingQuery)
  collection = collection.__narrowingParent
}
```


Presumably these tests are here for generic mode versus loading on LaTeX2e? If so, couldn't we just have two versions (perhaps with an auxiliary) to determine if this is necessary at all?


Small typo: "isntead"


цвет — в переменную!
и так — со всеми цветами, которые не черный и не белый.


This is fine - so long as we remember that any since(...) conditions are at the end of the switch statement, by ascending versions.


а, я вспомни почему я так делал, я же прикрутил каунт к созданию нового екземпляра класа, и по этому счётчику ограничивал массив, мол сколько создано такого обьёма и массив, если создавать сразу то только массивы известной длины


Exactly... :)


довольно необычная для LESS конструкция... ))


@rapperskull that is correct. So I thought maybe upload_sources.sh should also do a seq iteration to track down those lower API versions...


Some more explanation on the changes made...
1) If loaded with traffic, with iperf for example, the recv() call would return -1 and run() would exit. Restructured to look for ENOBUFS error and continue in that case. Still exits on any other error, including keyboard interrupts.
2) Packet drops in the receive socket is not desirable, better to drop earlier in the netfilter queue. So would like to raise the limits on the receive socket as requested by the user. Added an argument to bind() to allow setting the receive socket buffer size. Better yet, default to a receive buffer size that will fit the max number of packets of the copysize specified. Found by experimentation the overhead per packet and added code to set the appropriate default receive buffer size.


Sure thing.


страшные табы!


А что делает этот метод? Отдает всех в отсортированном порядке? Тогда его можно сделать дефолтным (default) и реализовать через find(GeoPointed possition, int number) и size()


Here, I'd be tempted to have a definition for the kernel then alter it later. We do that for a few bootstrap-like items. Or we note that up to some point we have to stick to primitives: again, we see that during bootstrapping.


Не забываем про аннотации @NotNull/@Nullable у аргументов


Should pass CI now


Maybe, just maybe, I'll overlook the lack of structural parentheses.


I like tests, but there is a problem with them.
They all assume *stable* behaviour from RemoveDuplicates function, meaning that no characters will be reordered during duplicates removal.
If someone else writes another implementation of RemoveDuplicates that will give
```cpp
  RemoveDuplicates("aaabbbb") == "ba"
```
your testcases will fail for valid solution.

Sometimes writing tests and checkers can be trickier then writing the program itself ;)
But in this case you don't care about performance, so you can implement almost any procedure to check correctness of result (e.x. sort results before comparison).


вангую возможное отсутствие класса .main-nav__link в коде...


First part of fix for #11


Well, then bundle exec cap deploy should work for you.


тут и далее — все брейкпоинты д.б. в переменных.


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/dev/ddc0487d748720e56f2fa71b51f7c1018c73b803/index.html)


Thats awesome!


следующие несколько десятков строк — несоблюдение кодгайда.
плохо.


The link from yesterday was no good? I guess my only fear is that it will
get created and not linked somehow and then lost. As long as creation links
it, then I am fine with it set up however. And as long as we don't fuck
with the links. And deleting stuff is easy. As long as this way isn't
kludgy I guess I am saying. Can we basically not actually look up an encode
then, we have to go through track somehow?
Le Thu, Aug 6, 2015 à 16:45, Drew Westphal <drew.e.westphal@gmail.com> a
écrit :

> Ok. If it is generalized into file asset I guess it don't matter either
> way to me!
> D
>
> Le Thu, Aug 6, 2015 à 15:26, divmazie <notifications@github.com> a écrit :
>
>> I know we talked about not doing it this way, but WP's query functions
>> are a total mess, and I know this way works. At least I've generalized it
>> this time so that it is all being done in WordpressFileAsset, and will work
>> fine with both encodes and zips.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/divmazie/jonathancoulton-theme/commit/db8eee9e0dd3ddbd46a064cb179eade102539f97#commitcomment-12573602>
>> .
>>
>


I added the String#=~ to fast-ruby btw https://github.com/JuanitoFatas/fast-ruby/pull/59. It would be good to add the any? versus detect benchmark too if you get a bit more time after your rails patch.

Thanks again for the help :+1: :heart:


Боже, какая древность. Им уже давно никто не пользуется. Ну ладно, фиг с ним.


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/0.2.0/index.html)


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/dev/9e158ae895e8ca678835a3e545e993a88b36f3e3/index.html)


This change of `%%` => `%` breaks the script. Double percent sign `%%` is required, as Python interprets single percent `%` as an escape character for string formatting.


indent...


Nice.


Ye looks neat. Can overhaul the whole ui


cc @sbrossie for review (especially for payment states).


Deja el combineStocks, eso sirve aun para las versiones anteriores. #noPichoneen.


expect**f** is fine for the test. But for the reader it's impractical.


Why not reference the Attribute?
e.g. `KEEPALIVE_TIME(ThreadPoolResourceDefinition.values()[0].getKeepAliveTime())`


nit: add a trailing comma?


скрипты д.б. подключены перед закрывающим body


Really nice to cut this dependency.


for the test Case
https://github.com/mdarefin/Convert-For-Each-Loop-to-Lambda-Expression-Eclipse-Plugin/commit/f9cdebc5d974e6bd0640855364ec2c77658fd06a
Below is the ourPut, if I use getsuperInterfaceName():
-------------------------------Out------Put------------------of-----Above-----------TestCase----------
This is type name collectionTestCase.C
this is iTypeElement class C {key=LcollectionTestCase/A~C;} [in [Working copy] A.java [in collectionTestCase [in src [in TestCase]]]]
  int size()
  boolean isEmpty()
  boolean contains(Object)
  Iterator iterator()
  Object[] toArray()
  Object[] toArray(Object[])
  boolean add(Object)
  boolean remove(Object)
  boolean containsAll(Collection)
  boolean addAll(Collection)
  boolean removeAll(Collection)
  boolean retainAll(Collection)
  void clear()
--------------------------------------------------
this is ITypeHeirchay Focus: C [in [Working copy] A.java [in collectionTestCase [in src [in TestCase]]]]
Super types:
  Collection [in Collection.class [in java.util [in /Library/Java/JavaVirtualMachines/jdk1.8.0_31.jdk/Contents/Home/jre/lib/rt.jar]]]
    Iterable [in Iterable.class [in java.lang [in /Library/Java/JavaVirtualMachines/jdk1.8.0_31.jdk/Contents/Home/jre/lib/rt.jar]]]
  Object [in Object.class [in java.lang [in /Library/Java/JavaVirtualMachines/jdk1.8.0_31.jdk/Contents/Home/jre/lib/rt.jar]]]
Sub types:

--------------------------------------------------
this is superInterface interface Iterable {key=Ljava/lang/Iterable<TT;>;} [in Iterable.class [in java.lang [in /Library/Java/JavaVirtualMachines/jdk1.8.0_31.jdk/Contents/Home/jre/lib/rt.jar]]]
--------------------------------------------------
this is superInterface interface Collection {key=Ljava/util/Collection<TE;>;} [in Collection.class [in java.util [in /Library/Java/JavaVirtualMachines/jdk1.8.0_31.jdk/Contents/Home/jre/lib/rt.jar]]]
--------------------------------------------------
this is interfaceName java.lang.Iterable


Перенеси его в в корневой pom в dependencyManagement. И подключи его тут как dependency без версии


can simply write ```menu = create(:menu)```


Does it free or copy a message?


Alternatively, set the asset manifest prefix in you cap file.
````
set :asset_manifest_prefix, "manifest"
````


тут и далее — некрасивый код, закрытие тегов не проверяю.
(тупо не видно).


You sir can have my :heart:


Big oops here, should be:
```javascript
CollectionView = function (collection, options) {
  if (collection instanceof CollectionView) {
    this._mongoCollection = collection._mongoCollection
    this._parentCollection = collection;
  } else if (collection instanceof Mongo.Collection) {
    this._mongoCollection = collection;
  } else
    // we don't need to support collection views which auto-create a backing collection
    throw new Error('CollectionView must have a backing mongo collection');
}
```


If you look at this particular `expectf` file, the output is readable even with the regex items. The overall portrayal of what is being conveyed about ResultOrExceptionWrapper is the important point. The only thing that is missing are filenames, object numbers which are not really part of the main point.


эта строка отлично демонстрирует почему двойной пробел лучше табуляции.


Is this maybe too much detail for the end user?  (i.e. talking about implementation when all that really matters is usage)?  What about:

```
If an explicit axis is not provided, ``plt.gca()`` will be used.

The function can be called as any of::
...
```


БЭМ-модификатор без класса, который он модифицирует.


For this file you need to add #include "util/gvr_gl.h" as one of headers.

Otherwise the checkglError will trigger a compiling error.


At this point, we should also check if the status code is in [400, 500) and, if so, display the complaint that Marathon sends back. I'm not sure of the whole structure, but at minimum we see the following:
```
{
  message: <string>,
  details: [
    {
      path: <string>,
      errors: [
        <string>,
        ...
    }]
  ]
}
```
That is, the object has top-level "message" and "details" fields, where "details" is an array of objects each of which has a "path" and some number of "errors".

I see that there's [a `Message` struct in file _json.go_](https://github.com/shoenig/marathonctl/blob/e7ad96df44cdbc5fe17dbb3f7f12d8d5e2bb0fbb/json.go#L145), but it would only aid in decoding one of the fields here.


Forgot this change on last commit


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


Why "+1"?


The commit message is a little bit misleading I think.


I think this should be `mongoose.disconnect();`


when you write "which is bad" above, you mean "bad" because it is ... slow?  or bad style, because the input may in fact not be utf-8 formatted, and thus we'd get UB?


because of implicit return you could write this as:
```js
let funcs = m.map(val => fs.readdirSync(path.join(lodashPath, val)).map(name => name.slice(0, -3)));
```


I would like to have a link to general lambda. Lambas are a concept introduced in hack, and it's super hard to search for it, if you don't know it's called lambda (i.e. searching `==>` probably leads to no good result). So people might not know what's happening here instantly.

However lambdas are a concept that will be often used in the docs I think. I don't know if linking to them everytime is the perfect solution though.

A note above or below is adequate. Throwing the name "lambda" in a comment in code would also work, although I don't really like it.


To get the span after the prompt (as ```GetClosestPrecedingInputSpan``` used to do).


(Having said that, your item (2.) below this serves as a *fine* motivation for the function. :)


не уверен, что это действительно нужное подключение...


No need for `includes` use since it's in es6 or you could use `_.find` since it works on objects.


это делается изменением бутстраповской переменной


haha, just do it then :smile:


аналогияно, ищите BS_переменную.


Even though I don't know yet how to get rid of this ugly static_casts, we can already decrease the amount of ugliness by using it only once:
```cpp
auto c = static_cast<unsigned char>(*it);
```
and then using this variable in other places.


Yeah, you're probably right. I think nobody will be disturbed in this case.


Looks good... to make `main` a bit more readable you might try: `var command = process.argv[2];` and referencing that. (Could do the same with `key` and `value` too.)

Also, just for sanity I always use `===` rather than `==` ([fun comparison](http://dorey.github.io/JavaScript-Equality-Table/))


trailing commas


:angry:


comma


Do realize too that I am just getting out an initial version of the async docs so that we can use them as our test bed for the user reference. Once I get something workable done, there will be some iteration on how things are represented, etc. This may not be the final form of some things.


It will by problem on windows !!! So, later we must try


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1268 of 1268 tests passed)

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


This line has to be changed. The link should be saved to the database, not inserted on the view.


@emord curious what the story is here


> the for loop was for the print out to check what kind of data I was getting, Since we are getting the array of iTypeHeirchay, we could iterate over it and get the superInterfaceName using the method, and then check with if it's part of Collections,

You are already iterating. You just need to check `iType2`. I suggest using a debugger to examine that variable upon each iteration.


I'm not a fan of obvious comments. I believe that in this case appropriate names of variables are sufficient and say the same thing, so why repeating yourself twice?


It doesn't work this way, could you please just revert and release 2.1.0.alpha for this @elucash?


feel free to PR =]


typo: s/is done/if done/


thanks for that! ;-)


Perfect, this will work until we can come up with something better


Can you take a look at my branch [rhino-autodetec-singleton](https://github.com/potyl/stetho/tree/rhino-autodetect-singleton)? It is based on your branch.

What I did is to make the class a singleton, this way the auto detection can do it's magic and applications that wish to customize the JavaScript environment can access the builder easily.

I think that everyone wins: stetho detects js-rhino and loads it, apps can customize data easily.

Sample:
```java
        // Javascript configuration
        JsRuntimeReplFactoryBuilder jsRuntimeFactory = JsRuntimeReplFactoryBuilder.getInstance(context);
        jsRuntimeFactory.addVariable("foo", "bar");

        // Standard stetho configuration (unchanged)
        Stetho.initialize(
            Stetho.newInitializerBuilder(context)
                .enableDumpapp(new SampleDumperPluginsProvider(context))
                .enableWebKitInspector(Stetho.defaultInspectorModulesProvider(context))
                .build());
```
https://github.com/potyl/stetho/tree/rhino-autodetect-singleton


really? return NULL here?


The management command we were using is deprecated (and stopped working with django 1.7)
https://github.com/benoitc/gunicorn/issues/705#issuecomment-37805232

There's some docs (not the greatest) from django https://docs.djangoproject.com/en/1.7/howto/deployment/wsgi/gunicorn/

_set_source_root came from what the management command did before running gunicorn


@Dominique7 You realize that this is a private key, right?  Having this public is bad.


commit lgtm


also here...


Please give specific version. In the future, may indicate a problem with the changes under the hands.


master


This looks like a debug left over?


sry for that


I spent like two hours today playing with the parameters of wordpress
functions, so this is a bit of a "fuck it, do it live" move for me, but
also one that I'm not really worried about. It really should be possible to
just search wordpress attachments, but I don't think I'm going to spend any
more effort on it right now.

On Thu, Aug 6, 2015 at 4:50 PM, Drew Westphal <notifications@github.com>
wrote:

> The link from yesterday was no good? I guess my only fear is that it will
> get created and not linked somehow and then lost. As long as creation links
> it, then I am fine with it set up however. And as long as we don't fuck
> with the links. And deleting stuff is easy. As long as this way isn't
> kludgy I guess I am saying. Can we basically not actually look up an encode
> then, we have to go through track somehow?
> Le Thu, Aug 6, 2015 à 16:45, Drew Westphal <drew.e.westphal@gmail.com> a
> écrit :
>
>
> > Ok. If it is generalized into file asset I guess it don't matter either
> > way to me!
> > D
> >
> > Le Thu, Aug 6, 2015 à 15:26, divmazie <notifications@github.com> a
> écrit :
> >
> >> I know we talked about not doing it this way, but WP's query functions
> >> are a total mess, and I know this way works. At least I've generalized
> it
> >> this time so that it is all being done in WordpressFileAsset, and will
> work
> >> fine with both encodes and zips.
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/divmazie/jonathancoulton-theme/commit/db8eee9e0dd3ddbd46a064cb179eade102539f97#commitcomment-12573602
> >
> >> .
> >>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/divmazie/jonathancoulton-theme/commit/db8eee9e0dd3ddbd46a064cb179eade102539f97#commitcomment-12575293>
> .
>


91977808_LBNL_NewVRFModel (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1777 of 1784 tests passed)

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


Yes, I know :) I'm just posting my thoughts on these things. Please don't feel obliged to do anything with it. I am just thinking about fixing things instantly, before those little things are forgotten.
Most of these comments were the first thing that popped in my mind. Please tell me if this is too bold. ;)


Could you explain why this is bad?
Update rate is ignored on the first call but is then back to normal after.


EINVAL doesn't seem to be a proper error code in this situation.... maybe make it consistent with mosquito_reconnect_async() return code, and simply return r, maybe?


Got it
Le Thu, Aug 6, 2015 à 17:11, divmazie <notifications@github.com> a écrit :

> I spent like two hours today playing with the parameters of wordpress
> functions, so this is a bit of a "fuck it, do it live" move for me, but
> also one that I'm not really worried about. It really should be possible to
> just search wordpress attachments, but I don't think I'm going to spend any
> more effort on it right now.
>
> On Thu, Aug 6, 2015 at 4:50 PM, Drew Westphal <notifications@github.com>
> wrote:
>
> > The link from yesterday was no good? I guess my only fear is that it will
> > get created and not linked somehow and then lost. As long as creation
> links
> > it, then I am fine with it set up however. And as long as we don't fuck
> > with the links. And deleting stuff is easy. As long as this way isn't
> > kludgy I guess I am saying. Can we basically not actually look up an
> encode
> > then, we have to go through track somehow?
> > Le Thu, Aug 6, 2015 à 16:45, Drew Westphal <drew.e.westphal@gmail.com> a
> > écrit :
> >
> >
> > > Ok. If it is generalized into file asset I guess it don't matter either
> > > way to me!
> > > D
> > >
> > > Le Thu, Aug 6, 2015 à 15:26, divmazie <notifications@github.com> a
> > écrit :
> > >
> > >> I know we talked about not doing it this way, but WP's query functions
> > >> are a total mess, and I know this way works. At least I've generalized
> > it
> > >> this time so that it is all being done in WordpressFileAsset, and will
> > work
> > >> fine with both encodes and zips.
> > >>
> > >> —
> > >> Reply to this email directly or view it on GitHub
> > >> <
> >
> https://github.com/divmazie/jonathancoulton-theme/commit/db8eee9e0dd3ddbd46a064cb179eade102539f97#commitcomment-12573602
> > >
> > >> .
> > >>
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/divmazie/jonathancoulton-theme/commit/db8eee9e0dd3ddbd46a064cb179eade102539f97#commitcomment-12575293
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/divmazie/jonathancoulton-theme/commit/db8eee9e0dd3ddbd46a064cb179eade102539f97#commitcomment-12575691>
> .
>


Fair enough, that's a good enough reason. I just wanted to make sure there wasn't, like, some difference in behaviour I wasn't aware of.


@peti The `base-orphans` update in this upgrade fails for me with:

```
Building base-orphans-0.4.2...
Preprocessing library base-orphans-0.4.2...
[1 of 1] Compiling Data.Orphans     ( src/Data/Orphans.hs, dist/build/Data/Orphans.o )
In-place registering base-orphans-0.4.2...
Preprocessing test suite 'spec' for base-orphans-0.4.2...
[ 1 of 11] Compiling System.Posix.Types.OrphansSpec ( test/System/Posix/Types/OrphansSpec.hs, dist/build/spec/spec-tmp/System/Posix/Types/OrphansSpec.dyn_o )

test/System/Posix/Types/OrphansSpec.hs:19:13:
    Not in scope: type constructor or class ‘Int32’
builder for ‘/nix/store/8s0vn49w3x25f7xp0g06vi4fr3fa9lw0-haskell-base-orphans-0.4.2.drv’ failed with exit code 1
```

Do you see this?  It is a dependency of the `lens` library.


Check


Your solution is correct :+1: However, there is one alternative which might be simpler:

```java
mWeight += WEIGHT_GAIN;
mFeedCounter ++;
int growingCounter = mFeedCounter / 3;
if (growingCounter < 1) {
    size = "tiny";
} else if (growingCounter < 2) {
    size = "small";
} else if (growingCounter < 3) {
    size = "average";
} else {
    size = "large";
}
```

Notice how `growingCounter` represents how many times the dog has been fed three times (our cap for increasing to the next size, from `tiny` -> `small`, etc).
The same can be applied for `play()` as well.

As I said, your solution is correct, this is just an alternative.


Good stuff :+1:


I will report upstream.


Yes, it's bad because it *would* go horribly wrong (UB) if other code screwed up and passed in something that's not ASCII.


Szia

Kicsit szünetelt a fejlesztés, de hamarosan folytatom...
Köszi!

Üdv. Krisztián

2015-08-06 18:23 GMT+02:00 borazslo <notifications@github.com>:

> Drukkolok!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Sektorka/szentiras.hu-android/commit/f0be42676d1065269fb95d17c14bfd4bb7110af9#commitcomment-12569537>
> .
>


master


Awesome, exactly what I was looking for.
@afgane how do you handle this one: https://github.com/galaxyproject/galaxy/issues/549. I know that the script we are using asking for the latest available version before installation and providing the `--revision` in every installation.


nesmysl nejde to ignorovat :D


admin jsem tu nechal, ale asi je zbytečný


This variable is not used :tongue: I suppose you added it and then rethink the solution and forgot to delete it.


trošku mě vadí řešení těch odkazů přes absolute_url, určitě by to šlo lépe, ale nechal jsem to tak jak je to v is-core


I love your comments! Keep them coming....


nechci zobrazovat json, když se použije jiný typ např yaml nebo nějaký binární tak by to nebylo čitelné


These annotations can't be removed as they're involved in handling the configuration.yaml file.

For example if I remove the database JsonProperty annotation I get this error

configuration.yaml has an error:
  * Unrecognized field at: database
    Did you mean?:
      - metrics
      - template
      - defaultName
      - server
      - logging
        [2 more]


přeložili jsem jen češtinu :(


huh tohle ještě smažu blbý gedit


Upsi... fixed in dc4e9a5cfc9d390cfb32130c9b9107b34e008958
Thanks for noticing.


nemůžeme si tu hrát na nějakou session, jelikož v is-coru se session nepoužívá na přihlašování. Popravdě moc nechápu proč to tu měli, nějak nevidím smysl. Možná kdyby někdo do requestu dal usera ručně, ale to je blbost ne?


this is weird to see you cleaning up the read/write sockets and [re]adding only the read one, I'm sure there's a reason  for that but at first glance I would expect the same operations to happen in the pair (wondering: maybe it's the case of socket_write to be optional)...


tohle mě vytáčelo i to formátování a nikde nebylo verbose_name (u verze)


přišlo mi nesmyslné tam dávat všechny verze


je podle mě rozumné použít defaultní ordering


toto jsem přidal


ok, after debuging, if I check with
 String interfaceName = iType2.getFullyQualifiedParameterizedName();
method, I am getting the java.util.Collection or any name that is implemented, so at this point should we just check if Collection is implemented or not


Yeah, that should do it! :sparkles: If I need more, I'll let you know, but I think I can solve it with this.


I was changing the very same thing as part of the wipe effort and I need to be convinced that forgetting the old GX is equivalent to wiping `ake.ourPublicValue`.

I'm leaning toward wiping the `conversation.keys` (as we do before sending a DH-commit message). What do you think?


potřebuji to při zobrazování změn nechci data z parenta


Thank you


If the revision is not specified, it checks for the `latest_installable_revision ` and uses it: https://github.com/galaxyproject/ansible-tools/blob/master/files/install_tool_shed_tools.py#L155

BTW, the command line options allow only installation of this `latest_installable_revision` while the yaml file allows a specific revision to be set. If you think you'll be defining the revisions, let me know and I'll see how to integrate that as well.


tohle je z KNJ automaticky to vrátí hodnoty které jsou v jsonu (nebo v parentech)


zmíněný operátor


If the `is_final` test really is an edge case, I'd rather not include it in the bundle (the less I have to maintain, the better :D)

P.S: English isn't my native language either ;)


I would prefer the otherway around something like #ifdef __unused && _GLIBC_SOMETHING
 #undef __unused
#define __unsued blabla
#endif

is that possible?


You added these lines
```
+  .option '-g, --grep <pattern>', 'run only tests matching pattern'
+  .option '--url', 'url that should be loaded at the start of test'
+  .option '--no-color', 'do not use colors in output'
+  .option '--no-exit', "don't close browser window after failure"
```

but in commit message isn't description, just like in README
I think it would be best for each option have their own revision describing the behavior of + public description in the README


Updated time to a real timestamp, and added lpvex as a possible campaign.


teoreticky nesmyslný stav, ale kdyby se stalo

C D C a to deleted nebylo verzováno tak je to jen C


TeamCity ChameleonForms :: Continuous Integration [Build 247](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3643&buildTypeId=bt20) is now running


vytvořený a změněný je primárně vytvořený


is this correct?! Or shouldn't it be
```php
$query->select(array('p.*','CASE WHEN CHAR_LENGTH(p.alias) THEN CONCAT_WS(":", p.id, p.alias) ELSE p.id END AS value', 'p.name AS text'));
```


tohle je dost nesmyslný stav v jednom requestu vytvořit a smazat by se teoreticky nemělo verzovat, ale radši bych to dělal


We don't need this yet. If at any later point we can add it.
Very nice! I will try to get this in as soon as I branch the next dev release. So after 15.07/07 is released.


done


Copy/Paste error, ty


Ah, I see. Maybe a comment?


follow prakticky znamená, že nevím co se s objektem stalo a když to vím z předchozího přidání do verze tak si typ vemu (dále nekomentuji)


Having `AdtDef<'tcx>` be `&'tcx AdtDef<'tcx,'tcx>` is certainly nice. `AdtDef_` is used basically nowhere (once in the tcx, once in decoder, once in collect), I'm not sure it should be behind a type alias. Also, this isn't really `Mut` - you can still fulfill an `AdtDef<'tcx>` with a `Ty<'static>`.


opět stát se může když

CH  D C to vlastně znamená jen změnu objektu


In your commit message is `initialize winston logger and try to search for config` so, this is two topic => two revision .

Your revision:
![Your revision](https://v.cdn.vine.co/r/avatars/28CBF18B89980850922890899456_1c0b898f62e.3.1_tfT3KfNwp9XWMMHXFxFhi0twSk0NFJPvVkTs9Kpq1d6ozyMCc9vGwjUXLO48yuqL.jpg?versionId=ogTed84F1XvVJ2QSFN13birsCvi92jT_)


když se něco smaže a zas se to vytvoří v jednom requestu (bloku) tak je to vlastně jen změna


merge damage I imagine


to by se také nemělo stát ale opět

D C CH to je stajný případ jako výše jen je vynecháno to C z verze


jedině když

D C D -> D


ADTDef/ADTDef_ was the variance distinction. I guess I could go AdtDefData/AdtDef.


where do you save stuff to the jail file?
or is it just for manual adding banend ips?


mám to schválně v listu, ale pro rychlejší zpracování z toho udělám dict dictů


ok


"set index.coffee as main file" okay, but why? :-)


potřebuji to jinak by smazané objekty se vynechávaly (nezmění se jen se smaže), stejně to nepoužíváme


přidá typ do dat nové verze


As far as I can tell no facet of this solution manages affinity state at all - that should be entirely in-kernel, right?  That should simplify this commit a bunch.  Or am I misunderstanding?  What code path would ever insert something into this map?  I see no such insert anywhere here.  At that point this ceases to be a "load balancer" at all and becomes a simple endpoint manager, and can be renamed to reflect that.


ack


These are IMO unnecessary - this is not perf sensitive code. I wouldn't worry about accessing properties multiple times..


@niel  - Dam, I missed, this needs to be in the latest patch too:

AND (TABLE_NAME LIKE "binaries\_%" OR TABLE_NAME="binaries");


automaticky registruju parenty objektů, pokud není řečeno jinak v registraci.


I copied somewhere that did `.0` (you can't believe how many places do this). I match on the binder in several places - maybe I'll add a `tuple_ctor_to_enum` function.


Could we make it not confused?


oops


Additional fix for #11


There are two possible returns from mosquito_reconnect_async: EINVAL and ENOMEM, but they kindly redefined all error codes. As I don't want the return of the API to be dependent on mosquitto, I went for the easy way out, but it may be worth it to have a mosquitto-to-errno converter or something.


když změníš potomka tak typ verze parenta není follow ale respektuje typ potomka. Změní se potomek tak se změní i přede, smaže se potomek smaže se předek, vytvoří se potomek vytvoří se předek. JE to tedy zvlášní typ follow


That's what I meant by "hand-variancing". I guess I will split the methods anyway.


to co jsem popisoval v předchozím komentáři je vyřešeno zde


congrats on the big stable update - lots of new features for the mainstream folks.
also - I do like the new Netflix menu / interface style btw
sorry - Ive been busy - just yell if you need any help - I will be back to sanity soon :)


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0-PullRequest124+016](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3643&buildTypeId=bt20) outcome was **SUCCESS**
Summary: Tests passed: 3003 Build time: 0:0:0


NOTE: this is commented to prevent duplicate error messages, and is uncommented in a later commit.


zde nastavuji typ podle toho jaký signál se zavolá


duplicate/copy everything for message_copy() scenario seems ok to me, but to message_new() it doesn't seem to be optimal to me, can't we just keep the pointers the user provided? and in the case of message_copy() the user frees the duplicated stuffs?


smazal jsem south migrace jedeme na nové django


> ok, after debuging, if I check with
String interfaceName = iType2.getFullyQualifiedParameterizedName();
method, I am getting the java.util.Collection

:+1:

> or any name that is implemented, so at this point should we just check if Collection is implemented or not

Yes, just compare the strings.


Makes sense, although I never got to the point were mosquitto_want_write() is true here. But I guess its a valid concern. I'll rework it.


reapplied changes and released as 2.1.0.alpha, but might take some time to propagate to central


that's for the entire patchset up to this point (I wanted to separate it from `Ty::is_simd`).


Change this to: ``3 to 28``


lol since was forked during those days... i will fix tmo too


Right. This PR is already somewhat too big to my taste.


no, mosquitto-to-errno seems to be the worst solution :-)


Sure.


In your revision missing explain upgrade. What's news in new version?


GetKindOrdinal?


Frankly, I was thinking more about mutations between accesses.  I can revert them though, if you prefer.


$"Unexpected value: '{span.Kind}'" to avoid inconsistency when renamed.


what's the use case for this message_copy() API? I didn't see anything using this API in your sample code...


:) will take a look


My original idea was to return a list of `rpycheck_fail` for the *two* `rpy` alternatives, something like: `[[True, False, True], [False, True, False]]`. However, it makes sense for the the success check to only return one 3x1 array, something like: `[True, True, True]`. As a consequence, were left with a mismatch in return types [`numpy.array` vs. `list`] which requires much extra book-keeping on the recipient's side. Making them both lists by returning `[rpycheck_succ`]` is one possibility, but somewhat annoying no?
It actually makes a *lot* of sense to return *both* `rpycheck_fail` I think. That's why I put the code in place to do it, hoping that you Python gurus could help me out of this predicament.
If we decide that we just want to return one of them, then, I'll definitely make the code a lot more succinct.


meh, but I'll be rebasing it anyway


dropck was somewhat an hotspot on my profile. We really should split up `ty` someday, through.


I talked to @basoundr, who suggested that the issue may not be with how we're using the service.  If you like, I can file a bug to investigate further.


tabs... aren't we getting rid of them?


Awesome. Any thoughts of adding [`#pragma strict_gs_check(on)`](https://msdn.microsoft.com/en-us/library/bb507721.aspx) on core functions?


yep, it's now working, I guess we could move into the next one


98318570_RemoveRadianceCheckbox (macumber) - Win64-Windows-7-VisualStudio-12: OK (2200 of 2215 tests passed)

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


update SG


:hankey:


Adding IT Test tovalidate we can write to a MT view. I've added a new table to this to separate this from existing tests so as not to get false positives/negatives.


hmm, there seems to be something wrong with it so it needs to change a bit more.


You don't need to worry about updating these to be double quotes. I think single quotes are fine. Hound is a little too aggressive right now.


why not. It depends on the impact on performance.


Added tenantId attribute to the Pherf Scenario.


Please catch by reference? (i.e. `catch (std::exception& ex)`)


For views that use double-quotes for lower-case names we need to do some cleanup here.


```
// JSONProgress describes a Progress. terminalFd is the fd of the current terminal,
// Start is the initial value for the operation. Current is the current status and
// value of the progress made towards Total. Total is the end value describing when
// we made 100% progress for an operation
```


Did you mean to `git rebase` this?


```
// Display displays the JSONMessage to `out`. `isTerminal` describes if `out`
// is a terminal. If this is the case, it will erase the entire current line
// when dislaying the progressbar.
```


Pass in the Scenario so that we can supply the tenant Id when we get a connection. If the attribute wasn't specified this getTenantId() returns null and we just get a standard connection.


```
// DisplayJSONMessagesStream displays a json message stream from `in` to `out`, `isTerminal`
// describes if `out` is a terminal. If this is the case, it will print `\n` at the end of
// each line and move the cursor while displaying
```


HI!
My Phone NX404H
no roms?


Underscore filter http://underscorejs.org/#filter
... returning an array of all the values

So `"#{testFiles.length} test files were found."` is right for one or more test files and `if` is unnecessary, right?


I'm sorry - I can't track exactly what's changed here - is this partly in response to my comments in 660951bdf673d57ed5ea3263e96422d17d766b60?


Change to the WriteWorkload to use the tenantId on the scenario when getting a connection so that we can write to a tenant-specific view.


apostrophes


Adding this for the new test table and view.


ah makes sense. Thanks!


`logger.debug "#{testFiles.length} test files were found."` ? it's possible?


je to vysvětleno níže už z reversion, pokud uložím změnu zpožděně jako to dělají ostatní signály (pre_save, post_save), tedy vytáhnou data z modelu až při ukládání změny, tak  v db už model není a verze se nevytvoří. Proto je nutné vytáhnout data ještě když je objekt v db, z tohoto důvodu to musím přidat do eager_signals


oops


It seems like `invalidUpdatedUserPreferences` can just be a `List<String>` to begin with, to avoid having to create it from the `List<Preference>`


Nice adding different markers on your map.


Structure to be all CSS at top.
JS at bottom.


I'm a little torn about this. I think it's probably the right thing to do.


use ul > li when you have list of items to display.


May be worth stating in the doc that data ids for bad data must be fully qualified.


That is what the test above is showing. If we start the client with the first server down its normally removed. The new code guarantees that it remains in the list.


Great use of variables! Excellent naming and use of HTML and Javascript here!


Maybe make this log.info since it's not a warning (you asked it to do that).  Same for isBadId


I see. That seems legit. Then perhaps a comment would clarify.


yep!


@madongfly LGTM


1). Why was removed `async` and `test-stack-helpers` ?
2). Why was removed `bin/webdriver` it was important for `./node_modules/.bin/webdriver someTestCase ...` It was the same mechanism just like at MochaJs


TeamCity ChameleonForms :: Continuous Integration [Build 248](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3644&buildTypeId=bt20) is now running


not really :)


SF IMHO


Do not wait for the DOM to be loaded to perform calculations.
Execute things which are dependent on DOM here like fetching a DOM elem, appending etc...


Start from 2008; we already have lower numbers used for LOB types.


Excellent.  Looking forward to.  Don't forget tests!


Actually I wonder if all 3 iterations can be combined:

```java
final List<Preference> preferencesToUpdate = new ArrayList<>();
final List<String> invalidPreferenceNames = new ArrayList<>();

updatedUserPreferences
  .stream()
  .map(p -> {
    if (preferenceWhiteList.isEmpty() || preferenceWhiteList.allows(p.getName())) {
      preferencesToUpdate.add(p);
    } else {
      invalidPreferenceNames.add(p.getName());
    }
  });
```


@krainboltgreene
shoo


It will work if the user does not have rights to dir create?


not -> now*


Nice. You're clearly going to streamline all the communication layer. :)


you must try it on windows :-/


I like this! Liked that you set up a limit of results. Nice


Please remove from code and move to a new issue


Seems like this snippet can be extracted to a method ("GetInputProjectionSpan(lineNumber)" or something like that) and reused.


Note: this required the create of a PGSQL [function](Modify subscription service for new user_aoi structure #14)


I was told in the discussion about my customization point design that, in order to prevent ODR violations, this actually needs to be a class static and *not* just a variable template. Can I suggest:

```c++
template <class T>
struct __static_const_ {
  static constexpr T value {};
};
template <class T>
constexpr T __static_const_<T>::value;

template <class T>
constexpr const T& __static_const = __static_const_<T>::value;
```


In my opinion it should be printed info after the action is done, not before.


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0+016](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3644&buildTypeId=bt20) outcome was **SUCCESS**
Summary: Tests passed: 3003 Build time: 0:0:0


SQL Error (1064)
It should be like $search = '%' . $db->escape( $search, true ) . '%' );

source: https://docs.joomla.org/Secure_coding_guidelines


@tremainebuchanan remember reuse... don't make the calls listing specific to a user's id.  It can accept as query parameters the `us_user_id`.  Try and adjust this before merging.


That's correct. It takes the place of the old `LoadBalancer` but does no such load balancing. It's a pretty terrible name, I hadn't renamed it yet as I wasn't sure what the best alternative would be: https://github.com/GoogleCloudPlatform/kubernetes/pull/9210#issuecomment-128461824

I'd forgotten that the affinity stuff was in here, as this has 'just worked' for about two months now.

I'll push a rewrite removing the unnecessary bits and renaming it shortly.


@tremainebuchanan same as before....don't make the api specific to a user... we might want to look on all calls for all users eventually.  That would require an additional change.  Write this so it accepts query parameters.


@alphan102 I am not sure how to check file_exists and to get the file date for the nsDumpUtils temp file.


I think it will be more clear if we add the sleep in the Process method.  It's possible other tests could fail now or that we could add other tests later and forget about this and lose hours troubleshooting.  10ms will not kill us for these tests.


Great job. Extracting the information using the getJSON method


Hmm, not without moving the copy somewhere else. When the broker sends a subscribed client a message, the pointeer provided to message_new() is controlled by mosquitto and it is freed once the moquitto on_subscribe callback is done. In this case, we would have to copy it and keep the reference somewhere to be able to free them once our on_subscribe callback is done.


I have it in my source too and it works perfectly here


Keep the data from a message arriving on a on_message() callback. It's just an utility function, I'm not sure if you would really want the data that way.


Logged #4394 to generalize with MetadataReferenceResolver as the common base class.


change your ellipsis to …


Que tal deletar o código?


This needs a license that's MPL2.0-compatible; as it stands we cannot include this code. Open an issue here:

https://github.com/NikolaiRuhe/NRFoundation

to ask about the license, and make sure you amend our licenses list accordingly.


MetadataReferenceResolver is a combined resolver and provider currently. See #4394.


First line of commit message is too long.


let's try to use the Icon component for all of our icons


LGTM with license issue addressed. Please make sure you test this, even if only by adding some janky code to the AppDelegate that creates a ton of files.


let's use the Icon component


And this is only triggered with max_reconnect_attempts as -1, correct?


OK.


we should be able to use the actual character rather than an entity, right? option-semicolon on a mac …


I like EndpointTracker or EndpointManager


Why do you need to autoload tests? Do we need to instantiate test classes anywhere?


I have `EndpointsTracker` right now since it tracks all of the endpoints.
Removing all of the cludge, will have something much simpler in soon.


unit test this.


`…` for ellipsis here also


Good to know xD


MSVC is fine with this way of doing it.


I don't see how to use the nsDumpUtils temp file to check for file exists and to check the file date


`...` is not the same as a correct ellipsis `…` in typography itself. The spacing makes a big difference.


`*Future` generally maps well to `Task`. Implementing `DefaultChannelGroupFuture` as a `TaskCompletionSource` results in dropping all the value on a floor when you do `.Task` on it here. I would say, let's do Task.WhenAll(...) for now and think about how we can port it completely going forward.


This is now done as part of "receiving a DH-Commit message in AUTHSTATE_NONE"


ok


Please review @stubailo


this might be more obvious as something like:

```
if major != minMajor {
    return major > minMajor
}
if minor != minMinor {
    return minor > minMinor
}
return patch >= minPatch
```


Referencing #6


and this will match the repos built by ice-tools, which already have Origin: Red Hat, Inc. in them


`let`


This will basically go into a busy loop, hammering the OSD with requests (and getting eagain replies) until the pg peers.  I think we need an exponential backoff of some sort...


use `as` instead of `is` + cast


@pekohl DWMR. Good. Press on!


I don't think we want to add an arg to every method.  Insetad, add a single method to teh IoCtx that sets a timeout that is used for subsequent requests.  (This is the pattern that we already use for all other unusual options/args.)


This is Fei's editing


This should probably be done in VersionedSaveDialog() but I can't wrap my grey matter around it. As an example I give you this instead. It works and the odds that you ever come across it is tiny. You need to first want to save a template file and also want to write over a present file or accidentally type the name of a present file... all this without typing in the suffix manually, then this message box will execute and since it's done after the versionedSaveDialog has is executed it won't take you back to the save dialog but to the project window.


@faloi podemos extender esto?

Explicar que ahora vamos a enseñarle a la máquina a hacer varias veces lo mismo, capaz incluso con un mini ejemplito que haga tres veces la misma operacion, y luego digas: "te resulta molesto escribir tres veces lo mismo? Gobstones puede hacernos la vida más facil


I don't think this should be a template string because we don't want the newlines?


This seems to be indeed the correct approach, according to Python (2.6) docs:

"Some care must be taken if both signals and threads are used in the same program. The fundamental thing to remember in using signals and threads simultaneously is: always perform signal() operations in the main thread of execution. Any thread can perform an alarm(), getsignal(), pause(), setitimer() or getitimer(); only the main thread can set a new signal handler, and the main thread will be the only one to receive signals (this is enforced by the Python signal module, even if the underlying thread implementation supports sending signals to individual threads). This means that signals can’t be used as a means of inter-thread communication. Use locks instead."

Still, I'm a bit confused at how/why a thread that is not a main thread would reach this code... a spawned thread running processes feels awkward... But I may be missing context. Maybe it is in the mailing list? I will look for it.


1. Only admins can add/edit categories;
2. Setting overall apearence of view template of categories.


this is all extra indentation because of the testkit fixture


Yeah that is on the todo list :)


Looks like you didn't have an upstream set for your branch. You could use `git push -u origin bleeding-jumbo` once and it would be set for future pushes.


Increased hard limit for levels, this fixes  https://github.com/dmwm/DMWMMON/issues/21.


Commit message for GameWindow.java should read "Debugged layout change. Now the app recognizes both portrait and landscape. And does not reset everything on layout change"


oh you're right. but it's still much more convenient to write long strings this way. how about i remove `\n`s from this string. is that too weird?


Ah yeah, that makes sense and more cleaner. Thanks!


Looks good to me otherwise.


I like how you made this work. With the empty method!


can we get away with doing `Contains(item.Id)`? Or if we want complete behavioral analog: `.TryGetValue(item.Id, out channel) && channel == item`


`this.` is missing. Please do code formatting


feel free to modify the whole way the breadcrumbs works if you felt like it - I tossed this together quickly - but I was thinking about maybe modifying this to include icons for repo/branch/module etc.


Actually, would it be better to do:

```
const savedSetState = component.setState;
try { ...
```


00200_La computadora repite por nosotros/description.md

@faloi en todos los lenguajes _imperativos_ :stuck_out_tongue:

Ahora, en serio, evitaría este tipo de aclaraciones, porque en realidad podríamos decirlo para un montón de cosas que vamos a ir mostrando.


It would let us use const, and reduce the lines of code


tru


@siracuervo well :P to you.....That dont help me none..lmao


nit but I like underscores in file names :)


if we're not doing any set method impl, let's go with `ICollection<T>`


Does the color match after rebooting the phone?


:beer:


:you were right:


changed. will come in with the EndpointsTracker stuff.


@kwm4385, what type of trouble are you having with the Link component? We should be using it the majority of the time - otherwise we're going to get full page refreshes if we use a standard a link, which aren't super smooth.


That does look cleaner, I'll switch when I write the tests.


@HerrLukas Can you open a pull requrest with the change? See: https://docs.joomla.org/Using_the_Github_UI_to_Make_Pull_Requests if you need help. Else this gets lost.


It makes me a little nervous to impose the same defaults in two places because you have to keep both up to date.  Seems like not defaulting the config parameter and just using the context default if it's not set may be the way to go.


It looks like it's possible at this point that `events` is empty?


Yeah, I would suggest using the concatenation of regular strings here... sorry! https://github.com/meteor/javascript#6.2


cc @Cogito :bow:


Alas, it's possible to read these docs somewhere other than the docs.puppetlabs.com website. So we should use the full https://docs.puppetlabs.com/ prefix on the URLs. I realize this is inconsistent with some other sources, but the embedded stuff is kind of a special case. :/


would it be hard to impl these using Enumerable.XXX methods? For completeness.


Heh, never noticed this commit before. You've gone mad!


If you are looking for a marker, but you do not know exactly which one, typing in MARKER_ and using auto-completion in your IDE will give you more precise results that if it can start with any letter.


for later, as this is 99% approved: I think we're going to need to test kernel version or existence of the route_localnet sysctl as well as the hairpin brctl.

Th real decision will be whether we support the user-space proxy for old kernels at all, and if not, do we support userspace proxy for dnat-to-self when route_localnet is not supported.

Let's discuss this all once we have the other wrinkles ironed out.


Thanks for this fix!


we could also rename proxier.go to proxier_userspace.go


Please change this to:

    #!/usr/bin/env python


@tursics @k-nut Please review the article before I send a pull request. An image would be nice to have - any ideas?


This test takes much more time than any other one in Roboconf. :P
We really need to activate it manually when necessary.


¿Está bien dicho "secuencia de _comandos_"? ¿Le dicen así?

Yo hubiera pensado que le decían "operaciones" o "sentencias"


Could we actually find multiple transactions (e.g. several failures due to retries)?


I think the pattern is sufficiently common.


I like it.


Thanks once again Suresh.


I'm going to keep it, but use ```nameof``` (as I should have to begin with).


Idea - maybe makes things cleaner, maybe not.  let's discuss.

What if we moved the userspace proxier and LB logic into a sub-pkg here (userspace) and moved this all into a sub-pkg (iptables).  That way you don't have to put the word iptables in every type...

?


I love comments. But I think these ones were useless.
More generally, I prefer skipping comments on fields, except for very special ones. :)


cmd2 is not in the standard library - or at least it wasn't when I tried to use it last.  Please add this requirement to the top of the file so others know what is required to run this.


Yeah re: '==' it was the code from the course itself. Agreed it should be '==='. Add it to the list! ;)


@faloi Probá como queda el layout editor_bottom en este ejercicio; tiene bastante código y es largo.


Do you know why the old batch fixer used waitall here?


I guess it was **NOT** supposed, right?
Anyway, we can easily rename the method.


this is nice, but it makes it a little hard to click/know you can click on the row to expand the modules when the branch name is a little long...I suppose user can always click where the arrow is the the right. just thinking out loud here, I don't have a particular solution right now, but let's think a little bit about it.


That seems reasonable.

On Thu, Aug 6, 2015 at 6:26 PM, Tim Hockin <notifications@github.com> wrote:

> Idea - maybe makes things cleaner, maybe not. let's discuss.
>
> What if we moved the userspace proxier and LB logic into a sub-pkg here
> (userspace) and moved this all into a sub-pkg (iptables). That way you
> don't have to put the word iptables in every type...
>
> ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12576932>
> .
>


nice refactoring!  This is some real OOP stuff


@schneems is this really supposd to be `VVERBOSE`? That seems really... verbose


Ok. The masquerade rule i added is also currently untested and will need
discussing when we come back to this.

On Thu, Aug 6, 2015 at 6:23 PM, Tim Hockin <notifications@github.com> wrote:

> for later, as this is 99% approved: I think we're going to need to test
> kernel version or existence of the route_localnet sysctl as well as the
> hairpin brctl.
>
> Th real decision will be whether we support the user-space proxy for old
> kernels at all, and if not, do we support userspace proxy for dnat-to-self
> when route_localnet is not supported.
>
> Let's discuss this all once we have the other wrinkles ironed out.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BenTheElder/kubernetes/commit/ab81a4a89ef21444a303eabcdfcd4d27e3a881fb#commitcomment-12576891>
> .
>


Comments like this are frivolous.  If we can't tell what the if statement is doing then it's too complicated.  In this case the if statement is extremely clear so documenting it is not really something that is required nor would I recommend it.


Turning this into an issue so I can track this. #26


I think this should be `string.toUtf8().constData()`. Also, the `.toUtf8()` part would need to be done earlier in order to get the right value for `stringLength`. In the Tales client, actually this function takes a `QByteArray` to begin with.


la solución a este problema esta en slack
----------------------------------------------------------


I think that was for debugging a problem that resque was swallowing. We should really switch to sidekiq, just haven't gotten around to it


[nit] while (true)


The single space here after the comma is the only whitespace covered in this class. Reading this and the functional tests, readers would think the patterns must be separated by ", " and nothing else.

Need coverage in this class of `"      ~/..."`, `"~/*.css,~/*.txt"`, and other variants.

Separately, as @NTaylorMullen asked in an earlier iteration, why `$"...`?


Why not keep the order of the parameters consistent with other ```expr()``` methods, with the property first, and let the method handle the logical ordering?  This would also allow programmatic use of these expressions, as in:
```
foreach ($filters as $filter) {
    $expr = $filter['expression'];
    $query->andWhere(
        $queryBuilder->expr()->$expr('x.' . $filter['property'], $filter['value'])
    );
}
```


somente na repetição da parte A


But `VVERBOSE` with two `V` characters vs `VERBOSE`?


Don't know if you want to fix this name?


This delegate callback in the recorder is not intended to modify the view, but rather just to retrieve the view that will be presented.. It seems like it would be best to just delete the step that uses it as such from `ORKTest`.


this is redundant


Aº fica mais com o jeito do arranjo do "Nosso"


Changed ```GetPromptIndexForLine``` to ```GetPromptIndexForPoint```.  I'm not sure how much else can be reused.


@mjdesa This will cause telegraf to fail and exit if no config file was specified


Looks good

Lets add unit tests for `getRank` and then this will be good to merge


just return this.Current


@jze Do you know why the gradle version is being changed here?


I would suggest moving comments like this to below the `def functionname(args):` so that they then can become self documenting strings.  They should probably also be inside a triple double quote, etc.  Same comment applies to all other functions.


rename and some cleanup have been pushed.


Typically the `'''` would be on it's own line.  It makes it easier to see that the comment has ended.


@XXMrHyde nah man...I must say theat i had reverted alot of battery code before i added this, but again, everything works, all options, even color inside the icon, but not next to it..stays white no matter what...It doesnt bug me, as i never do next to icon, but apparently some users do..lol


What is the benefit of using the call method vs exports?


That's the good point of polymorphism ^^


Some extra information:
I tried to prove SetEqual, but still couldn't.
drop_last_label is done, and it wasn't necessary to prove sigma_downInternalLabel


Starting here the spacing seems inconsistent throughout the rest of the file.  Sometimes 4 spaces sometimes 2.  For readability purposes please stick with one and preferably 4 spaces.


yup, missed this one


develop (mbadams5) - x86_64-MacOS-10.9-clang: Tests Failed

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


Trim trailing spaces in this repo. \*shrug\*


remove the last ;? some compilers may warning for double ;; if `DISALLOW_COPY_AND_ASSIGN(my_class);`


Please use a context manager for open, it will ensure that the file is closed:

    with open(self.debugfilename, "a") as configfile
        configfile.write(newmsg)


also add `void operator=(const T&)` and `void operator=(T&&)` ?


use Contract.Requires(e1 != null)


FourPipeBeamAirTerminal (EnergyArchmage) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Starting here you have inconsistent spacing on things where prior to this you did not have spaces before the ( but after this you seem to have a space then (.  Ideally you would stick to one consistent style and preferably that style would be no space.


Do we need the `chmod` on line 207 if we are always going to overwrite it here? (and if you did the `chmod` before the file was closed initially you could use `fchmod` which might be more efficient).


They should do under known apps in `deviceSpecs.js` since that's what the file is create for.


Iterator in Java has different behavior from IEnumerator in .NET. Just proxying `this.currentEnumerator.Current` should be enough here.


Can you please explain more what it means to properly configure ACF in this context? Provide an example config.


La descripción de la guía quedó en TODO, escribí eso y me agarró el síndrome de la hoja en blanco. Tomo tu sugerencia para cuando efectivamente la escriba :smile:


Please use spaces around comparison operators, for example:

    if myswitch == 'leaf' or myswitch == 'spine':

You aren't really consistent with this throughout the code some do have spaces and some don't.  Preferably you should have spaces.


This list tells the data type of the arguments, but it should also say what purpose those arguments serve. Like:

1. An array or hash to iterate over.
2. A lambda, which gets called for each element in the array or hash. It can request either one or two parameters; see below.


Please use spaces between concatenation operators as well.

target_url = "http://" + switch[1] + "/ins"


@preethi29 i have made a fix at https://github.com/openmrs/openmrs-module-webservices.rest/commit/7dd54027f4a97c63147a4c209e5e3a73736c3be7
Please try again.


Nitpick: I think this can be just "Settings for notifications that appear on your device" rather than "Settings for **app** notifications that appear on your device"


What does pooling exactly do?
Will that reset it every 10 connections?


Recién me fijé en el libro y siempre se hace mención a "bloque". Y el bloque está definido así:
> Un bloque es un comando formado por un grupo de comandos delimitados por llaves (caracteres { y }).


I'd prefer not to call out curl but use python-requests with python-gssapi.
This is an examplke of client code using python-requests with python-gssapi:
https://fedorapeople.org/cgit/simo/public_git/freeipa.git/tree/daemons/ipa-custodia/ipakeys/client.py?h=custodia&id=6c377243a548a01e2c63b09f5ef862749141777f


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 276](https://teamcity.labkey.org/viewLog.html?buildId=265089&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:00
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


It would be better if this explained what the "notifications tab" is.


Please inherit from object.


Convention in the docs is to not capitalize data types like "array" unless we're quoting a piece of code that uses data type objects (like `Array`) to test or restrict values. Henrik always capitalizes them, but that's because he lives in implementation-land, not user-land.


Remove YARD tags from the puppet/parser/functions version of these docs, because the thing we use to extract those docs can't process them usefully. Instead, you can make this a H4 header or a **bold** thing.


97204612_MultiSpeedFan (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1697 of 1795 tests passed)

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


The /./ means any "one character". Was it what you wanted?
and if we do not use it should we comment it out? The documentation is not clear on whether it's mandatory to have it.
https://theintern.github.io/intern/#common-config


this pins dor-services to the 4.21.x releases. if you need to pin it to a specific version, omit the "~>":

```
gem 'dor-services', '4.21.4' # comment on why we pin it
```


Looks good. Thanks for the update to use common code.


It'd be more illustrative to use a variable to capture the value of the expression: `$transformed_data = $data.map .....`


@clebergnu `avocado-vt` has the screendump thread, which calls vm.screendump(). In qemu, this is made through QEMU monitor calls, in libvirt this is made calling a subprocess (`process.run('virsh screendump')`).


[17:44:14]  <@drtshock>	because why not
[17:44:28]  <@drtshock>	if anyone that is good at concurrency wants to tell me how bad that is, let me know.


@fiatjaf Please add trailing `;` to these lines to prevent potential minification issues.


"passes each key/value pair to..."


This didn't come up until I decided to spend more time in running the `libvirt` `avocado-vt` test backend. So any thread that calls `process.run` or `process.system` or `process.SubProcess` would have this problem.


Is this a typo?


97204612_MultiSpeedFan (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


You can't really call a value. Instead, mimic what you said immediately below about hashes.


"...and a one-parameter lambda"


one-parameter lambda


Possibly so it cancels immediately in the event a fixer task does not propagate cancellation tokens properly?


So I set up i setup bower.json and grunt, but what do you mean by the next commit should have bootstrap coming from bower? I thought it was just a package manager.


For all of these, we should be adding a comment showing the resulting value; otherwise it's too opaque. In this case, it'd be the array `[a, b, c]`.


result: `[0, 1, 2]`


Very much


according to @jtlan, ```displayText == null``` is the standard way for nullity check


Added gitignore

review: @gregllong, @jayich


Also, the sidereal year is 365.256363004 days long, not exactly 365, though leap days make some years longer than others.


That's not as useful as it could be. We want to know what the order of the parameters is. The first parameter is the index (starting from 0), and the second parameter is the value.


:question: Why use `WaitAll` instead of the following?

```csharp
await Task.WhenAll(fixerTasks).ConfigureAwait(false);
```


can we just do `foreach / yield return` over both dictionaries instead? There's inherently no guarantee of a perfect snapshot so I don't think it will be a problem to extend this behavior with lazy iteration.


Hm now that I think about it, this wouldn't work since an empty whitelist means that all preferences should pass. But here, it's saying "an empty whitelist means everything is invalid". oops. sorry for the bad advice :P


Same as I said for the other function: let's lowercase the data type names.


Thank you!


I [left a comment](https://github.com/dotnet/roslyn/commit/56e17171887333afb2573f09effa6a8af15da0cd#commitcomment-12577331) on the original commit where this was introduced.


I was concerned about shrinking caches that were already under pressure, but you're right that this could also prevent growing a busy cache (which is exactly what we want).  I'll revert this part.


This breaks IE10, which has no concept of a const.


DRY: just return `this.GetEnumerator()`


Ahh I see whats going on here. So we can add whatever we want to the form as long as it is permitted and we will then have access to those in the models attributes?


Yes, it should be fixed. Let me do so right away. Thanks.


make it static


We can change it to `/^.*/`, which will mean "anything". The one character regex will always return true for any file--which is the intent (and the functional equivalent for `/./`). It's to disable this report, which is useless for functional testing:

![](https://cldup.com/73lhfUU4rG-2000x2000.png)

(it looks like if we upgrade to 3.0 we can just use `true`).


Should be able to simplify the code to:
```
Rcpp::stop(Rcpp::sprintf<100>(
                             "Recieved %d names and %d types but worksheet contains %d columns.",
                             names.size(), types.size(),  ncol_).c_str()
                             )
           );
```


This is a potential solve for https://github.com/fiatjaf/eager-hypothesis/issues/1


These examples are actually untrue!!!

The value of this function call is actually `["abc123", "abc"]`. If you think about it in terms of the regex match variables, the value is always something like `[$0, $1, $2...]` — whole match, first capture, second capture...


So do we have access to the methods included in these decorators at all times for all records?


develop (mbadams5) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Value is actually `["abc123", "abc", "123"]`


In this controller, yeah.


是不是发送消息没有设置至少2～3s的间隔 导致机器人回复速度太快 ，这样很容易被腾讯认为异常的


This is adding `CMAKE_CXX_FLAGS_DEBUG` flags to `CMAKE_CXX_FLAGS`


Sorry never got around to building roms.


I think this is fine. I have been ported telepathy-hanging locally to use setMembersIdentifiers() and it works as expected. My only concern is that the method name isn't as intuitive as addMembers()/removeMembers(), but I am fine to drop these as long as we make it clear to developers this is the official way to add/remove members in the documentation.


All right, it must works.


looks good to me. I haven't tested with telepathy-hanging yet, but will do soon.


RoomAirflowNetwork (lgu1234) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Did you pick only this commit?


ah ok, didn't realize that, will fix the tests.


The "next to" battery level text is using it`s own class (BatteryLevelTextView),
so first compare your BatteryLevelTextView with the BatteryLevelTextView version of this commit, (use the "View" button next to the BatteryLevelTextView code). Also take a look at all methods related to the BatteryLevelTextView, (KeyguardStatusBarView -, PhoneStatusBar - and StatusBarHeaderView class, also compare with versions of this commit)


I think this is backwards. If the checkbox is checked, the form passes a 1 through the form, which means you want it to be re-reviewed. If I'm understanding what is going on here at least.