Typo here... not sure why this didn't error. Will investigate


Hello, I don't know whatever you remember me or not. I wonder whatever the DBVM can now run inside KVM with these patches?


Just a minor observation. This shall be more applicable once libnetwork has support for multi-host environment.

In multi-host environment I think filling up `hostname` for the endpoint will need to be delayed until `Join()` as CreateEndpoint() is a global call.

I think we can omit specifying the `hostname` and use the `-host-bindings-cfg` of netdcli at time of `Join()` to achieve endpoint to host binding. Checkout sample json for host-binding configuration here: `examples/late_bindings/multiple_vxlan_nets_host_bindings.json`


@knu ping?


@4aiman: Multi-line setting values should still work...


writing to db here is the next step, and should be relatively easy @mlevans


Shouldn't this take a []byte instead of a file name? The entire file is already loaded into memory, so you could easily hash the file in one go without having to worry about blocks etc.


good concise scope!


Also see https://github.com/matplotlib/matplotlib/issues/4504


Starting from matplotlib 1.4.3, you just need to add a call to draw_if_interactive in set_linewidth etc. in class Line2D to get the interactive behavior. I would be very weary to add yet another hook function.


Personally, I feel like anything which unnecessarily breaks builds, and doesn't do so intentionally (ex, a breaking API change), should not be put in master. I mean is there any reason why this couldn't have remained in a skylands branch?


código comentado = :scissors:


Não vejo pq fazer este terneiro


Привет! Большое спасибо за помощь с переводом! Можно внести некоторые изменения и немного перефразировать? Давай найдем общий вариант?

-----------Waypoint: Удалите HTML Элементы
"Удалите элементы h1 и br,  чтобы упростить вид  нашего приложения.",
"У экрана нашего телефона не такое большое пространство по-вертикали.",
"Давайте уберем ненужные элементы и начнем создавать наш CatPhotoApp.",

----------Waypoint: Измените цвет текста
"Поменяйте стиль элемента h2  так, чтобы текст внутри него стал красным.",
"Чтобы это сделать необходимо изменить у элемента h2  атрибут \"style\".",
"Стилевое свойство, определяющее цвет текста внутри элемента, называется \"color\".",
"Вот пример того, как сделать синим цвет текста внутри элемента h2: <h2 style=\"color: blue\">CatPhotoApp</h2>."

----------Waypoint: Используйте CSS селектор, чтобы добавить элементу стиль
пока только название, сам текст еще не прочитал)


It seems you miss <% } %> before line 21 <div class="archive">  in themes/alpha/layout/_partial/archive.ejs
hope this can help you.


Se você copiou aquele método de extensão que fiz em aula (ToEnum), dá pra usar:

```
reader.GetString(2).ToEnum<Categoria>()
```


Fixed in 36993f6684a23fde38b3a3b91763cc99fee4ef53.


errou a digitação ali: `resutlado`


Teste número 1


detalhe: se você tivesse declarado a variável como `int` não precisaria atribuir 0 à ela. Mas isso é questão de estilo de código, não existe melhor nem pior.


Something meaningful, I will have try.
Thank you ~


@rodrigomolinamunhoz, dá forma como está implementado neste commit ele remove apenas o Restaurante, e não todos os seus relacionamentos (lembre-se que não criamos delete on cascade no banco).


You're checking `hEditMenu` and `hEolFormatMenu`, but not `hLangMenu`?


:clap: :clap: :clap:


C-style cast in new code?


If you're doing any benchmarking, you'll want to configure gcc with
``--enable-checking=release``

I *think* this is enabled by default in the gcc 5 branch, but it's probably best to be explicit about it to be sure.
https://gcc.gnu.org/onlinedocs/jit/internals/index.html#working-on-the-jit-library


não precisa iniciar o valor com 0, por padrão o compilador já faz isso com tipos `int`.


This is now an all-`public` `class`? Huh?


cuida com a convenção de nomes, variáveis são camelCase.


I noticed this. It was the first thing I was going to try when I got back to my development environment. I am glad it works now. :)


$mkdir -p svc/*


Intentionally non-const?


$ cd - p svc/(


It should be Exercise


@gdi2290 nope. This was throwing an error, something about `Cannot read property 0 of null`, not exactly sure though if that was the exact message. But it might have been for a different reason, I was using the pipe on a `router-link="{{ route.component.name }}"`, where `component` was is a constructor. Once I removed the caching mechanism, it worked just fine.


se você copiou o método de extensão `ToEnum` que fiz em aula basta você fazer:

```
reader.GetString(2).ToEnum<Categoria>()
```


cuidado com a convenção de nomes, variáveis são em camelCase


just wondering but where do the 30x+ faster comes from?


This was caught by trying to make a regex pattern that worked with the provided output.


Can You take this down, Please?


Esse método tu implementou sozinho?


$ mkdir-local/brosif6guns@gmail.com/0/emulator


Whoops :D


The `draw_if_interactive` functions currently live in the backends which all of the artist related modules do not currently know about.  We can use `draw_if_interactive` in `pyplot` because it manages selecting and loading a specific backend and import a number of functions including `draw_if_interactive`.  One thing that might be possible is to hook `draw_if_interactive`  up to the parameter change call backs that live in the `Artist` class and hope that the `draw_idle` methods on all of the backends work as promised.


@mdehoon, that was the alternative approach in #4082, but with some machinery to prevent useless intermediate draw operations.  @tacaswell's approach is more elegant.


relevant: issue #30


Oh god no.


Alles vorbei.


@tacaswell, your idea is that in a complex plotting operation, there would be many calls to draw_if_interactive, but because they would be calling draw_idle, the draw operation would be executed only once? Because the idle state would only be encountered at the end?


PullLogicFromPlantComponents (Myoldmopar) - 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-81.41%25-red.svg)


Yup, it never ends. Don't tell Maria.

Works surprisingly well though!


Please change this line to some other meaningfull text.


This line has to be deleted.


Again wrong text. Please see to it.


You can leave this at `RCT_EXPORT_VIEW_PROPERTY(mapType, MKMapType)` as the implementation of `RCT_EXPORT_VIEW_PROPERTY` is anyways like this:

```objective-c
#define RCT_EXPORT_VIEW_PROPERTY(name, type) RCT_REMAP_VIEW_PROPERTY(name, name, type)
```


CBECC (kbenne) - 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)


what


Please fix typo in GIT commit messages.


测试类提交


Oh, nevermind, github got it right. It had everything highlighted green on the desktop one, so I kinda assumed, haha.


size_t is architecture dependent, suggest to use __u32, and do not make type conversion


The message is a bit misleading. It should be part of CuRT x86 port instead of standalone work


Now it seems fine.


Yeah, at least some of the backends maintain enough internal state (ex https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/backends/backend_qt5.py#L425 ) so that multiple calls to `draw_idle` before the actual draw are more-or-less no-ops.

Also, if we tie this to the state change of `stale` on the `Figure`, it will only get flipped to `True` once so there should only be one call to `draw_if_interactive`/`draw_idle` anyway.

As much as I have been cursing the GIL in my day job, it is making our lives a lot easier here.

My current inclination is to change the repl registration to use the IPython hooks if they are available and the Figure-based callback otherwise.

@mdehoon Another advantage of using the `stale` flags instead of scattering `draw_if_interactive` calls everywhere is that third party libraries that want nothing to do with pyplot or our notion of 'interactive' can use it for their own purposes.


PullLogicFromPlantComponents (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Put this method `$result->fetch_array()` inside the `database::SQL()` wrapper so that it directly returns the associative array, so you wont have to write this line everywhere


why this?


All configurations are stored in `libs/globals.php` don't make a separate file again for that. All dynamic files are supposed to include the globals lib and thus get properties from there


I just added an IIFE here. If I break it out to a normal function — I would need to make the variable `hasReportedError` to a global flag, which I think is overkill for a reporting case. I can't also reference outer scope variables such as `typePredicate` and `typePredicateNode`.


PullLogicFromPlantComponents (Myoldmopar) - 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)


Just use `(funcall helm-elisp-help-function candidate)` is enough here.


Does this class really need public ```attr_writer```s?
Could you use ```attr_reader``` in place of ```attr_accessor```?
In general, the public interface of a class should not include any more methods than needed.


Nitpick: "who" is for persons in English. :)


To be deleted


You can `use cl-type-case` here.


`intern-soft` is better here.


Instead of calling it always with the same argument of `DEFAULT_CACHE_FILENAME`, consider making it a default parameter like in `filename=DEFAULT_CACHE_FILENAME`, after all positional parameters. This way, calling the function becomes more programmer friendly. :)


Same comment regarding where to save the file.


I prefer having a one line doc in mode-line for this, but I am ok to merge your changes.
I have added small suggestions, have a look.

Thanks.


Let me see if I get the point of this function here: the idea here is to implement incremental downloads (say, you're weekly following a course, and you only want the new stuff), right?


Why are you committing directly to master?  You shouldn't be...


I will surely add some tests and support for dot notation (e.g. `Hello {worlds.pluto}!`). Will ping you after the changes are pushed.


You might consider putting this in the model.  vote = IdeaVote.voted_by({idea => @idea, :project => @project, :user => current_user})

also since you are returning the vote you can do:

     if @idea.has_been_voted_for_by? current_user
         current_user.idea_votes(project: @project).first
       else
         current_user.idea_votes.new(project: @project)
       end
You can do then same thing in the project/votes controller.


こちらこそPRありがとうございました！


@TechReborn/coders , I'm not sure where these files should go. Feel free to move them around / change what package they're in.


gcc also supports `#pragma pack`, so you might want to just use that on all platforms: https://gcc.gnu.org/onlinedocs/gcc/Structure-Packing-Pragmas.html


Looks OK, other than the reason for the function itself.


path could be shortened down to, path = params[:return_to] || idea_path(@idea)


I've used something like this in my local copy. I have not inspected this change in particular, but do you still print what it would do? (e.g., the videos that would be downloaded)?

That being said, I love programs that have `--dry-run` options, as I always use them before doing anything for good.


NOTE: Engine will screw up if you try to remove static objects. That's why I converted everything into dynamics.


There's no need to have so many functions "private" at first, IMVHO.


Or "0" | "1", as DNT is defined?


OK.


OK, better.


compat_print instead? Or should it be part of the dictionary to be returned?


PullLogicFromPlantComponents (Myoldmopar) - 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-81.48%25-red.svg)


yay! thanks!! the lack of PARENT_SCOPE here was exactly what was making the Android build fail!


Oh, is this print just for logging purposes? The intent is not very clear and I'm guessing here...


Good job. I am so glad to do it. And i an do the review work.


First of all, *Chief.

Second of all, I think we can all agree that a life term is a bit hazy.


2nd comment


We enable __builtin_expect on MSVC? Is it a typo?


Yes, that should be an #ifndef. Thanks for noticing, I'll go fix that.


Thanks. What I ended up doing was:
`if("@MSVC@")`
`  hunter_fatal_error("Autotools scheme not supported with Visual Studio")`
`endif()`

Since WIN32 was 0 for Cygwin, I was not sure how to handle it best, so just failing on MSVC seemed sensible.


PullLogicFromPlantComponents (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (644 of 644 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-5.32%25-green.svg)


vamvammmmm


Good point, done.


Why ignore the exception and not rethrow it?


no strong feelings.


This is the most painful part of the script, right? That's one of the advantages that coursera-dl has regarding this program...

Do units have name in edX? If so, we could use them and be done with that in the same fashion that coursera-dl does.


Yeah I was thinking about that. I don't think dyn4j supports dynamically moving fixtures, so we might just have to make do with one for the moment


Looks OK in general, but I have not paid so much attention. Perhaps you should have split this into more commits?


Se debiese haber arreglado issue https://github.com/CamilaAlvarez/IndicadoresDCC/issues/28


OK.


@ruslo how is it looking now?
I moved the autotools functions to modules, added the `hunter_` prefix.

Thanks


Verified! Can you summarize the changes in the first line?


Please refer to http://chris.beams.io/posts/git-commit/


I think you should keep the same name everywhere for consistency.


That test fails for me. It doesn't seem to pass unless the `~^` is on the line directly beneath `pub`. Am I missing something about how compile-fail tests work?


You should mention the considerations of enabling UAO in GIT commit messages.


Importen todos esta base de datos.


Ah, interesting, I guess it checks for the start of the span, not any place within the span, that is a little annoying. I guess we don't need any more for the test


Please improve your GIT commit messages.
Reference: http://chris.beams.io/posts/git-commit/


iyahh . . . ganti lg


soft keypad doesn't fire events for backspace key. So added a temporary button for BACKSPACE.


It looks a bit redundant since the string "telnet://" appears twice.


Does this mean ladders will be added back?


Did you have to validate the availability of existing tabs?


Actually no, it was a comment change. Forgot to commit this when I did it so I had to guess.


Can you avoid duplicated string "telnet://"?


Any update?


people love edited texts


Please improve the description of GIT commit messages.


I'm going for trebuchet after trying out customizable launcher3 and slim launcher. Found Trebuchet the best. Can you please tell us what's wrong with trebuchet


Why did you reduce the targetSDK and compileSDK levels?


I wondered why you decided to support it. :-)


PullLogicFromPlantComponents (Myoldmopar) - Win64-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-81.48%25-red.svg)


+1
Seems legit.


`city.stats.where(created_at: (7.days.ago.midnight)..(Time.new.midnight))` should probably be a scope on `City` instead.


This should be handled in `city_params`.


"dependant" is a totally kosher spelling for this.  So sayeth the OSX dictionary, anyway.


`stack<T>::stack() : this(DEFAULT_INIT_SIZE)`


`std::copy`


`marker.samples.where(created_at: (time - margin)..(time + margin)).last.traffic_load` should be a scope (with the `margin` being defined there).

Also, rather than iterating over `markers` to construct `traffic_load`, use `map` or `group_by`.


`delete storage`


I think the word "formula" you replaced is supposed to be in "<>" brackets to refer back to the parameter, not changed to "formulae".


Hey Elco were you able to test these changes with arduino (specifically pin sensing on DS2413)? I'm keen to update my arduino with these changes if they work.

Cheers,

Dan


I ran into difficulties on my travel/blog page in terms of adding text underneath each country's photo while still keeping the same 2 x 2 style. I wanted each country (img + p tag) as it's own entity separate from each other without using a table. I tried styling the p tags for that section to have a certain width but the text made each country section turn into a block style.

I'm thinking that it has to do with `float` in which we haven't encountered yet in class. In the following days, I will be searching for ways to fix this using the information we learn in class and additional research.


It would be beneficial to have a consistent approach to error handling – either by handling exceptions (then use `City.find_by_name!` here and in `CitiesController#stats`) or by checking whether `city` is `nil` or `traffic_load` is empty.


这里不能添加 `project.exclude` 这个问题有过讨论，因为你虽然可以实现 `project.exclude` 不 watch，但是不能实现 `project.include` watch，就会导致watch与编译的 `include` 与 `exclude` 表现不一致。


This should be indented two spaces more.


The 0.7 release notes mention [issues with `toTask`](https://github.com/scalaz/scalaz-stream/wiki/0.7#compatibility) and recommend using `step` instead. I didn't look into details yet but what was the reason why you didn't follow the recommendation?


The file rename was kind of a breaking change so maybe it should not have been released as a patch version right?


Привет, да не за что.  Да конечно можно!  Любые улучшения это только плюс.  Мне вот этот перевод нравится.

Evgeny Klimenchenko

> On Jun 7, 2015, at 8:12 PM, alnero <notifications@github.com> :
>
> Привет! Большое спасибо за помощь с переводом! Можно внести некоторые изменения и немного перефразировать? Давай найдем общий вариант?
>
> -----------Waypoint: Удалите HTML Элементы
> "Удалите элементы h1 и br, чтобы упростить вид нашего приложения.",
> "У экрана нашего телефона не такое большое пространство по-вертикали.",
> "Давайте уберем ненужные элементы и начнем создавать наш CatPhotoApp.",
>
> ----------Waypoint: Измените цвет текста
> "Поменяйте стиль элемента h2 так, чтобы текст внутри него стал красным.",
> "Чтобы это сделать необходимо изменить у элемента h2 атрибут \"style\".",
> "Стилевое свойство, определяющее цвет текста внутри элемента, называется \"color\".",
> "Вот пример того, как сделать синим цвет текста внутри элемента h2:
CatPhotoApp
."
>
> ----------Waypoint: Используйте CSS селектор, чтобы добавить элементу стиль
> пока только название, сам текст еще не прочитал)
>
> —
> Reply to this email directly or view it on GitHub.
>


так краще
```
if () {
 return;
}


Given that `City` `has_many :samples`, it should be possible to just query `City#samples` (and `group_by` them), rather than iterating over each `City#markers` element.


Коментарі в коді мають бути англійською


This method could perfectly fine have a one-line body. :)


经测试发现已经修复


Is keeping this (commented out) necessary?


пробілу не має бути після імені перед скобочкою
function name() {


@tractorcow This breaks any protector that uses a Text Field for the value as the value will be saved into the database. IsSpam should be internal and not presented to the user right? For example Math Spam Protection now saves the value of the answer into the database (i.e 10) which evaluates to true and therefore makes it all as spam.

My feeling is to rename this name away from IsSpam, protectors can always write back to the form data using `getForm()`. If you really want to keep IsSpam then we need to update all the protectors we support.


ім'я класу має бути маленькими літерами
атрібути HTML регістронезалежні!


I would rather remove the code for verifying the validity of the IPv6 address than have this pull request held in limbo indefinitely, because the main purpose of the change is to allow non-IPv4 reports to be closed. I was trying to get a head start on desired future functionality, but it's not essential.


Rather than fetching the data in the constructor it could be beneficial to lazy-fetch them on the first access to `#data_base` (and just cache the result).


завжди === замість ==, аби не було неявного приведення


These days I think it might be worth mentioning that issues related to the actual templates should be opened in the right repos. Many people will end up opening them here.


I can see the use-case for `forEachElement` even for sparse input. Do you want to have a specialized version for sparse vector?

Since the default implementation in `AbstrastVector` will do binary search to find the data when ` consumer.consume(i, this.get(i));` is called.

Maybe it worths to document that
```scala
  /**
   * Applies the given function to each element in this vector. This is a
   * dense operation that includes all values even for SparseVector.
   */
```


`data_base` is not a very good name (and ‘data’, in general, should be avoided). :) Maybe `api_results`?


但理论上来说，设置了 include 的，这以外的文件是不做考虑的；exclude 在 include 的文件里面再进行排除。那么不管哪种情况下 exclude 的文件都应该被排除掉；

另外你上面所说的 case 有实际案例吗？


This should be a `Module` with three `module_function`s, rather than a stateless class.


Думаю було б краще якби чужий код був в окремих файлах, аби я його не передивлявся


Nice workaround for Java. :+1:


It would be better to just pass a `city` here and not do an extra trip to the database for every city.


Ah, I see. Thank you @plujon . That makes some sense. In this case I don't see either as a failure. It is a design intention that one of two options are possible. And both are acceptable. The assumption is that there is more than one match to `*wiki*.bz2` and that the user might run the script more than once. If that is the case then the time it takes to process the data in `/Wiki-Data/` should not be expected to be re encountered. with some of the larger wikipedia files, the processing time is 3.5+ hours.


fdgdfg


I think it would be better to make sure `Marker#time_without_traffic` returns an `Integer` and `BingMapDataApi#time` returns a `Float` than to do the casting here.

`unless time_without_traffic.zero?` would be a bit more readable.


I _really_ dislike the notion of capitalising only the first letter of an acronym. `BingMapDataAPI` is much more readable.


Do we want to use the hack first? We can remove the hack once MTJ changes the behavior.


@ShadowNinja How should I set those via setting_set?


` _meshCommand.setTransparent(isTransparent);`missed here, which is fixed in pr
#12212


noticed that you have dropped the confirmAll code you added to dialogs in 43dc5e2089feaae805ee91f7e5644b3ece5d65d6. intended?


`BingMapAPI` or `BingMapsAPI` or `BingAPI` would be a better name.


The English word is spelt _report_. :)


Also, naming variables `*_hash` smells of [Hungarian notation](http://en.wikipedia.org/wiki/Hungarian_notation).


It depends on what is setting the wallpaper. Windows 8 doesn't in send anything for wallpaper transitions that go through IDesktopWallpaper. It also doesn't change the wallpaper if you just modify the registry key though.

The problem you're having is due to the very last change I made to LiteStep, https://github.com/lsdev/LiteStep/commit/ca01731f1a3504ebcde33a7eba79f223c5231f04

I removed the WM_SETTINGCHANGE message because it doesn't match what explorer does and may confuse other programs that listen for it. I'd suggest just handling the LM_WALLPAPERCHANGE message instead of listening for changes to the registry value.


A lot of typecasting. Don’t / can’t these variables have the right types already?


It looks like this method does a lot of math that is related to a `Marker` – I think this should be moved to either a `Marker#load(time)` method (that computes the `load`) or `Marker#create_sample(time)` method (that also creates the `Sample`).


Extra space before the closing paren. A RuboCop plugged into the default Rake task would catch / fix this. ;)


Well, then...


Note: this commit is broken since my changes to BlockOre aren't in yet. Next commit will fix that.


The `raport` variable is useless.

Rather than constructing a `raport_hash` with some keys being `Markers` and one being a `String`, how about passing the `weighted_mean` as a separate argument to `RaportLogger.new`? (With three arguments it could also benefit from them being kwargs.)


This seems to have a lot in common with `BingMapDataApi` – maybe a common `MapsAPI` ancestor can be factored out?


PullLogicFromPlantComponents (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1084 of 1160 tests passed)

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


Log lines usually start with the current time anyway. :)


The change looks reasonable to me. Leoboiko, could you address the issues raised by ospalh so we can merge it?


#!/bin/bash

read -p "请输入一些数字：" key
echo $key


439201356f715d9223bc9512ba0fb79000955733 should fix that, please let met know if you see other issues.


Consider logging a JSON representation: first, it will be easier to grep when there will be one line (rather than three lines) for each entry; second, it will be possible to parse and analyse it programatically if needed.


Can’t `@weighted_mean` be a `Float` in the first place?


Why this change? Now it might not return a boolean anymore, right?


This looks like it _calculates_ the percentage (rather than _counting_ it).

This method does not take a denominator, it takes a `Marker` and calls `#time_without_traffic` on it to get the denominator (it should definitely be called with a number, not a marker, though).

If you do `(100.0 * numerator / denominator).round` then you don’t have to typecast either of the numbers and they can even be `Integers` (`100.0` is a `Float`).


> I had a glance over the API the other day and it seems like it should be possible to render to an intermediate texture with "nearest", then render to the screen with "linear" (ie. what gl-branch does). So I think it's doable.

In fact, we actually do this already, since we need to blit the paletted 8-bit surface into an intermediate 32-bit RGBA surface which we can push into the texture. This rgbabuffer (that's its name in the code) can be of any arbitrary size.


Also it's strange that there was no test complaining about this.


Tu as pas oublié le fichier index.jpg ?
Pour info: https://github.com/putaindecode/putaindecode.fr/issues/436


lolwut ;)


Well ... I sort of did. As you can see below this `toTask` implementation uses `step`. This is actually not an identical copy from the former `toTask` code, but slightly modified and I hoped you could check, if this is correct. TBH I am not sure, if this suffers from the same problems. It just made the tests run :-)


It should be fine for now. If you have compacted vectors they're the same. Just when they are non-compacted it copies them.


But now, he did not load the hosters.


thanks Alex. Why double n = std::max(1.,dDiffMax/dTol); ?


I think I would just inline all of the `subject` calls – each is used exactly once, but being defined separately from the callsite means you have to jump up and down to read every single scenario.


Spelling police - "Height"


That is true. I wouldn't recommend using dense operations on sparse vectors, so wouldn't worry about optimizing it. But if it is useful, I can implement it.


This will clash with The Silver Searcher, a grep/ack-like tool, which uses `ag` as its command. How about `ansg` instead?


Yay :D


Per issue #99.


Ok, did check it explicilty but I guess that should be'width' of course.


I just added an IIFE here. If I break it out to a normal function — I would need to make the variable `hasReportedError` to a global flag, which I think is overkill for a reporting case. I can't also reference outer scope variables such as `typePredicate` and `typePredicateNode`.


*Note*: Cleaned to remove RPM history from repo.


lgtm


markChangedTagJudgmentsAsNotIndexedStep


In all of your specs you create three cities and one marker – maybe bringing in Factory Girl for this is an overkill? (I’m not sure; I never use it in my projects, so I’m biased against it.)


Yeahh, that can now be deleted. I added that in a time in which it was not that way, that it was directly overwritten to be sure it has a defined value. But now it can be deleted.


It says
```
fatal: No annotated tags can describe '97c6ae2719faac74687fe409b5a7258eab06b22e'.
However, there were unannotated tags: try --tags.
fatal: No annotated tags can describe '64f96a7d88b0e22321a2c64100980d6f962b2aa6'.
However, there were unannotated tags: try --tags.
fatal: No names found, cannot describe anything.
fatal: No annotated tags can describe 'a2a51904b0811510e90d8e6cf554de237b9205ab'.
However, there were unannotated tags: try --tags.
fatal: No annotated tags can describe 'b757b4f1ca57c6dcabfe4ad696a2b887544d1a71'.
However, there were unannotated tags: try --tags.
```
But versions contains
```
vid.stab 97c6ae2719faac74687fe409b5a7258eab06b22e
opus 3bd0231be14b77ab83f3afe2785f82338a1418b8 draft-ietf-codec-oggopus-06-26-g3bd0231
libvpx 4feae6791c0da9535a1c4c347e473ebc996c9d39 v1.4.0-200-g4feae67
x265 64f96a7d88b0e22321a2c64100980d6f962b2aa6
x264 121396c71b4907ca82301d1a529795d98daab5f8
libepoxy a2a51904b0811510e90d8e6cf554de237b9205ab
movit b757b4f1ca57c6dcabfe4ad696a2b887544d1a71
frei0r baa08d27193d758b5139dfa503dbedc881298f1c v1.2.1-242-gbaa08d2
FFmpeg db27f50e0658e91758e8a17fdcf390e6bc93c1d2 n2.3.6
mlt 2f25c83014f3be2d536fa47533cc520b69c376a2 v0.9.2-250-g2f25c83
shotcut 0d4531f4f75f3f580cf99036606304979ba87e50 v14.12-249-g0d4531f
webvfx c533cfd57e317f9cbaa1f6f2ad887b1ba3595d3d 0.4.4-45-gc533cfd
```


I think `DataDepositor` does more than just creating a single stat and a single sample. :)


nooooooooooooooooo, newline is missing at the end of the file :cry:

C++ is sad :D


Maybe we could also just return `null`. But I need to check the calling code first.


I also saw you other changes. I now have to go to work. So I can only correct the things you pointed out this evening.

Thanks for checking my code, again. Just a question: are you willing to work on the ODT plugin? What are your plans?


exceptionのraise / reraise の使い分け整理
① コマンドが正常に終了してその結果、標準出力からドライバ内で条件判定する場合⇒　raise
② コマンドが失敗し、その原因をログ出力したり、解析のヒント情報をログに残す場合⇒ re-raise


how filter whatsapp number ?


dog コマンドと collieコマンドの混在回避
⇒ 最終的にOpenstackコミュニティに投稿する場合にはパッチを分割できるように、コミット粒度に気を付ける


VDIサイズの下限値は、Sheepdogの実装上0が許容されるので、現状の判定条件でOK
ただし、定数は直書きを避け、ローカル変数として定義することが望ましい


L86 / L90はログ内容が全く同じで見分けがつかないため、ログ出力内容を変える。
特にL86はSheepdogのステータス情報が'running'以外になっていることを確認できているので、
そのステータスを表示できると好ましい


ItemInfo is currently a globally shared class, it is /not/ per-item. Store state variables on the player for now. Preferably in reusable ways.


@wkennington: I was going to ask "why", but then I noticed this in the diff

```
-  # There will not likely be an icedtea8 so users should move to openjdk
```

So I guess icedtea is dead/obsolete? Please say so in the commit message (explain the "why") :-)


There are parts of the code that is commented, so that we can reuse it in case of alternate design. Please ignore the commented code. Once design review is done, I will update and give composite pull request.


@garethgreenaway Where is ``salt.utils.http.query`` being imported?


@Tntrololol, you are truly a god among men. :P


Iced tea is just a build helper for openjdk, if you can package openjdk you
have no need for iced tea. There are no notable features that iced tea
adds, it was mostly to make openjdk easier to build. We still keep
icedtea-web for applet support and jnlp.

On Sun, Jun 7, 2015, 22:50 Bjørn Forsman <notifications@github.com> wrote:

> @wkennington <https://github.com/wkennington>: I was going to ask "why",
> but then I noticed this in the diff
>
> -  # There will not likely be an icedtea8 so users should move to openjdk
>
> So I guess icedtea is dead/obsolete? Please say so in the commit message
> (explain the "why") :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/a8d10100279636146b8d66da582e1f23d9d91742#commitcomment-11560655>
> .
>


Yes, and `.classpath` shouldn’t also be in this directory (and it’s in fact already missing in my local repository).


Wow! 600 lines changed. You deserve a cookie! :+1:


>Iced tea is just a build helper for openjdk, if you can package openjdk you
>have no need for iced tea. There are no notable features that iced tea
>adds, it was mostly to make openjdk easier to build.

It is an oversimplification for version 7.

For example, VUE had a few weird bugs with OpenJDK 7, including wanting
to write into /etc/.java/ even when launched by a non-root user, but it
worked fine with IcedTea 7 or Oracle JDK 7.


Hi, a little puzzle
If the innodb_lra_size changed to a bigger size.  how to promise the memory alloced is enough?
since I didnt find trx_lra_reset to 0 when query statement finished.


This is brilliant! Way to go Tess!!


Please add check that ``perun != null`` to prevent NullPointerException. Thanks.


PocketMine managed to do it so I'm gonna give it a shot at some point.


Yep I agree. If you're interested in doing a PR for it I'm happy to review and merge, otherwise I've got it on my todo list to try to get to this week.


So in that case would we prefer to support icedtea over openjdk for version 7? We should really settle on a good implementation for v7 in that case. Ideally, we should only need to support version 8 as it is supposed to be completely backward compatible.


Yea I was about to do that but committed this just as I was about to leave, so I just tested the functionality, but I'm doing the refactor now.

Also literally committed [this](https://github.com/learning-layers/AchSo/commit/025761718a17cc98881fca915d3f210971b3b48a#diff-a1c1db093f15ca101e70df51647cb6d2R67) for the second point, so I think we are on the same wavelength on this one :smile:


More details on the bugs:

+ Hope that your research on job is going well. Attached you find a file *.par. In this file I dfined theta as:
theta = {t1+pi/2,t2-pi/2,t3,th4,th5,th6}. When reading this file by the programme it transforms al theta as: th1,th2, and so on. It does not take into account the offsets, and transforms t into th. What we said is just when the user defines a frame with sigmaj = 0, the programme by default put thetaj = thej, but the user has the possibility to modify it as he likes, similar thing when sigmaj=1. Thus when reading a  *.par file, the programme should leave the parameters as given, without any modification. The only thing to modify is Pi/2 into pi/2, in order to be able to read *.par done for symoro-Mathematica.

+ The program reads good the saved robot. Many thanks but there is still one bug. If for a robot the user modifies sigma of any joint (from 1 to 0 or 0 to one) the interface puts the correct default value for all the joints and not only the one which has been modified by the user.
Hope that this will not be difficult to solve.


Oh dear, I think you're right.

Can we add a new API that separates the "Sets IsSpam flag" from the actual name of the field? (which probably should be left up to the protector).

As long as we can still save spam if using the akismet field (https://github.com/silverstripe/silverstripe-akismet#comments) I don't care what the API is.


Whoops, the commit message of course should say `pytest-qt`.


yes


I suggest a `FlagField` to save the boolean value into, although I'd imagine that making this a core api in spam protector would require each of the other modules to be updated. Can it be done in a non-breaking way? (old spamprotector / new field or new spamprotector / old field?)


Add an extra newline before this function.


Can you prefix this method with an _?  Also add a space between the `)` and the `{`

e.g., `function _arraysAreDifferent(array1, array2) {`


Yeah, this application makes sense.  Currently the `selected` state only gets updated as part of  direct user input and the initial state from defaultSelected.

If you need to programmatically update the selection, we'll need something better, like this!

Only question I have (ironically, I am a bit of a react noob) is: why do we need to diff nextProps' with this.props values?  This seems like a bigger perf hit than it might save to just always copy.  Could we just get away with the object reference comparison? (e.g., `self.props.defaultSelected != nextProps.defaultSelected`), or will this *never* return `true`, due to some internal copying in react?

I don't see any problems taking this as-is if you address the minor tweaks though.


dfsdf


ffp_packet_queue_get_or_buffering() needs to be replaced with ffp_packet_queue_get()


Looks good; thanks for your work on this


LGTM


UserInfoTest.java
line 11, 19 - remove
turn comments into javadocs
make getText() method generic


Whoa, never got a message on here so I didn't know to check them. Uhhh, I am not sure why it was crashing, I just know I tried to have it use the getSelectedUnit and forgot to fallback to the adventurer and it went splurch. I'll poke at it later and see if I can reproduce it and grab a log, don't think I have any from it after checking my folders and terminal.


>So in that case would we prefer to support icedtea over openjdk for version 7? We should really settle on a good implementation for v7 in that case. Ideally, we should only need to support version 8 as it is supposed to be completely backward compatible.

Apparently OpenJDK 8 has less quirks than OpenJDK 7 (tried to override
jre in VUE).


Ok, thanks for the feedback. I've configured Jenkins to build with maven 3.2.2 and java 8 now.


Add prefix for avoid possible conflicts `<tr id="prefix_{{ resource.id }}">`?
And main question: will grig support query with aggegation i.e.: `select('o order, COUNT(item) count_items')`?


Yeah, I put that in and then the uhhh, adventurer army selection because I just keep gm-editor on shift+d since it's so damn useful.


Right, so with openjdk8 as the default is there any reason to have icedtea or even openjdk7 for that matter? I'd like to clean out the cruft for java builds if we don't need these things.


Also oracle jdk7 is EOL:ed..


yeah, this is probably the better solution


Maybe setting/unsetting target would be more of a clear way in that case?


cpp doesn't suck, you're just bad at using it xD


This should be addressed by https://github.com/reschke/xml2rfc/commit/9bbbdb1409b02843c87bb552e0ad29ec207860d9


this is where we might want to do the explicit pairing between colors and labels. THe separate lists seem very error prone, in particular when we do nested color schemes like for ebola regions


Thanks for the pull request. Can you line up the indentation and I'll merge it.


変数に入れなくてもいいですね


10 tiles away, 60 Rads.

This was why I proposed a multiple of 10 for the multiplier, so "40" not "4"


変数にいれなくてもいいですね


can you replace ```new /image(icon, "lights-off")``` with ```image(icon,"lights-off")``` since the image() proc already handles the ```new``` aspect.

Same for all other instances.


Ok thanks David
________________________________
From: David Malcolm<mailto:notifications@github.com>
Sent: 08/06/2015 04:17
To: dibyendumajumdar/ravi<mailto:ravi@noreply.github.com>
Subject: Re: [ravi] start work on port to libgccjit (09943b5)

If you're doing any benchmarking, you'll want to configure gcc with
``--enable-checking=release``

I *think* this is enabled by default in the gcc 5 branch, but it's probably best to be explicit about it to be sure.
https://gcc.gnu.org/onlinedocs/jit/internals/index.html#working-on-the-jit-library

---
Reply to this email directly or view it on GitHub:
https://github.com/dibyendumajumdar/ravi/commit/09943b576b686760d96a801fddbf437132c63136#commitcomment-11559922


I don't really see the good use of that... I think the old method was way more convenient.


Nice! Maybe you should get mustache.js' tests and `skip` those in which you are not interested for now.


@benconnito, @cboden

If I understand http://wamp.ws/spec/wamp1/#uris_and_curies correctly, curie is to be used only if a prefix has been defined and agreed upon. In other words in the absence of a prefix, there is no curie, it's just a URI. However it may be a bad URI, but it will still not be a CURIE.

In addition it is also a breaking change (and should be mentioned in the release notes as such):
Consider the case where a user is subscribed to a topic "room:3" (without any prefix). The code linked to this comment will instead subscribe the user to "room#3", which is not what was expected by enduser. As I mentioned earlier, argument can be made against the URI to be inaccurate, but without any prefix this is not a curie either.

Perhaps an alternative approach be something like this:

```php
if(strpos($uri, $seperator) !== false && isset($this->WAMP->prefixes[$prefix])){
...
}
```


I've raised an issue at https://github.com/silverstripe/silverstripe-spamprotection/issues/22

I'm really not sure if there's a clear way to approach this.


@jserv, thanks for the suggestion. Will rewrite the code and send the PR again.


This merge was not needed, I'm learning to use git. I was used to use svn.


It's amazing what removing a bit of whitespace can do. :D


@nsh87 I have not tried to generate the documentation from the *roxygen* yet, and I am sure that some more will need to be added, but I wanted to get a decent version up on Github for you to see and review.


Returns an iterator that repeats the primitive-directions for a N times.

:param str moves: string with a sequence of primitive directions

:param str times: N of repetitions. If None it means infinite repetitions.

:return:
    An iterator.
:rtype: iterator


do we want the || true here? I'm wondering..


I think the test for Windows in common.js is not going to work on 64-bit windows:

`exports.isWindows = process.platform === 'win32';`

This should be used instead:

`exports.isWindows = process.platform === 'win32' || process.platform === 'win64';`


> do we want the || true here? I'm wondering..

If the repo is already there, zypper fails to add it again(and `-e` is set). And we don't cleanup all availalable repos before we start. That's why it's there.


另外，这些序列化顺序可以满足外部一致性（等同于线性一致性（ linearizability））的需求  --->
另外，这些序列化顺序可以满足外部一致性（即线性一致性（ linearizability） [20]）


@andreapernici quindi il file js nella root di js non serve più?


    def _parse_rect_expansions(rect_exp):
        """
        Parse rect-expansion into a list of dir-letters iterable.
        :param rect_exp:
            A string with a sequence of primitive moves:
            es. L1U1R1D1
        :type rect_exp: str
        :return:
             A list of primitive-dir chains.
        :rtype: list


Reviewed!


如果一个事务T1的提交早于另外一个事务T2的开始   -->
如果一个事务T1的提交时间早于另外一个事务T2的开始时间


It will fail if the temporary directory has been cleaned by ``shutil.rmtree``.


Example::


真实时间 ---> TrueTime
TrueTime并不等价真实时间，是带有误差的时间，或者说和真实时间有一个bound，不知道中文该怎么翻译，感觉直接用TrueTime更好点


:param int, str coord_margins:    the coord to translate


We don't need to set up an isolate scope here at the moment, so I took it out.


Also good lord I'm kinda nervous now having lethosor poking around at my code. >.<


I guess unannotated tags is better than nothing... Would you like to make
the "add --tags" change or should I and submit a pull request?

On Mon, Jun 8, 2015 at 7:39 AM Dan Dennedy <notifications@github.com> wrote:

> It says
>
> fatal: No annotated tags can describe '97c6ae2719faac74687fe409b5a7258eab06b22e'.
> However, there were unannotated tags: try --tags.
> fatal: No annotated tags can describe '64f96a7d88b0e22321a2c64100980d6f962b2aa6'.
> However, there were unannotated tags: try --tags.
> fatal: No names found, cannot describe anything.
> fatal: No annotated tags can describe 'a2a51904b0811510e90d8e6cf554de237b9205ab'.
> However, there were unannotated tags: try --tags.
> fatal: No annotated tags can describe 'b757b4f1ca57c6dcabfe4ad696a2b887544d1a71'.
> However, there were unannotated tags: try --tags.
>
> But versions contains
>
> vid.stab 97c6ae2719faac74687fe409b5a7258eab06b22e
> opus 3bd0231be14b77ab83f3afe2785f82338a1418b8 draft-ietf-codec-oggopus-06-26-g3bd0231
> libvpx 4feae6791c0da9535a1c4c347e473ebc996c9d39 v1.4.0-200-g4feae67
> x265 64f96a7d88b0e22321a2c64100980d6f962b2aa6
> x264 121396c71b4907ca82301d1a529795d98daab5f8
> libepoxy a2a51904b0811510e90d8e6cf554de237b9205ab
> movit b757b4f1ca57c6dcabfe4ad696a2b887544d1a71
> frei0r baa08d27193d758b5139dfa503dbedc881298f1c v1.2.1-242-gbaa08d2
> FFmpeg db27f50e0658e91758e8a17fdcf390e6bc93c1d2 n2.3.6
> mlt 2f25c83014f3be2d536fa47533cc520b69c376a2 v0.9.2-250-g2f25c83
> shotcut 0d4531f4f75f3f580cf99036606304979ba87e50 v14.12-249-g0d4531f
> webvfx c533cfd57e317f9cbaa1f6f2ad887b1ba3595d3d 0.4.4-45-gc533cfd
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/metellius/shotcut/commit/92a3a3663a0485fc16950ae74c225c7dfdd408fb#commitcomment-11560583>
> .
>


    def _target_opposite_state(state, cell, full_cells, dn, moves, last=False):
        """
        Returns the FIRST `cell` with OPPOSITE `state` of the starting-state.
        :param bool state: the starting-state
        :param Cell cell: landing-cell
        :param ndarray full_cells:  A boolean ndarray with `False` wherever cell are
                                    blank or empty. Use :func:`get_full_cells()`.
        :param tuple dn: bottom sheet limits
        :param str moves: string with a sequence of primitive directions
        :param bool last: If True `_target_opposite_state` returns the last cell
        that has the same state of the `starting-state`.
        :rtype: Cell


I didn't look into Netty internal, just judge from the API. What if an eventLoop is shared among multiple channels? Is this runnable task associated per-channel?


> The naive approach would be to remove the class as key in
> |$objectsByPath|, but there might be a reason for that ;-)

i think the only reason is historical and comes from the very early time
when we started this. i can't think of a valid use case of ever having
different NodeInterface instances for the same path (e.g. a Node and a
Version).

so indeed step one seems to me to flatten this. however, with
jackalope-jackrabbit, we do not want to try to store Version nodes or
other such, only Node. we need a way to keep that part of the behaviour.

> Another
> possibility would be to map certain node types to specific classes, so
> that the current structure of |$objectsByPath| stay the same and the
> node can be instantiated with the right class.

i think this is what we want to do for 2.0, yes. the class is
determined by the primary type, so we could use that information rather
than put that job onto the caller.


    :param bool state: The starting-state.
    :param tuple xl_rect: A ``(Cell, Cell)`` with the 1st and 2nd bouding-cells
    of the rect.
    :param ndarray full_cells:  A boolean ndarray with `False` wherever cell are
                                blank or empty. Use :func:`get_full_cells()`.
    :param list rect_exp: A list of primitive-dir chains.
    :rtype: tuple


    Captures the bouding-cells of the rect.
    :param ndarray full_cells:
            A boolean ndarray with `False` wherever cell are
            blank or empty. Use :func:`get_full_cells()`.
    :param TargetRef st_ref: `1st-target-ref`.
    :param TargetRef nd_ref: `2nd-target-ref`.
    :param rect_exp: A list of primitive-dir chains.
    :return: A ``(Cell, Cell)`` with the 1st and 2nd bouding-cells of the rect.
    :rtype: tuple


Hi Gavin, I've been wracking my brain trying to understand how this "reentrancy" situation may be possible.
 > If a particular consumer is still active when delivery appears for it, queue the deliveries up until it finally exits.

Judging by the code in `basic_consume`, the tag is always added to `self._consumers` **before** sending the Basic.Consume frame. And not removed until Cancel or Cancel-ok from the server.

Is it possible that these two code sections are never going to execute?

```
+        if consumer_tag not in self._consumers:
+            return self._add_pending_msg(consumer_tag, method_frame,
+                                         header_frame, body)
+        while self._pending[consumer_tag]:
+            self._consumers[consumer_tag](self,
+                                          *self._get_pending_msg(consumer_tag))
```

@gmr


Tem que ter apenas um ui:define para a tag content.


    Reads and parses excel sheet values of the `rect`.
    :param sheet: xlrd sheet.
    :type sheet: xlrd.sheet.Sheet
    :param tuple xl_rect: A ``(Cell, Cell)`` with the 1st and 2nd bouding-cells
    of the rect.
    :param indices: Indices for full-cells.
    :param epoch1904:
        Which date system was in force when this file was last saved.
        False => 1900 system (the Excel for Windows default).
        True => 1904 system (the Excel for Macintosh default).
    :type epoch1904: bool
    :return: A table with excel sheet values of `xl_rect`.
    :rtype: list


``````
"$days ". xl(days)


why the end if line is so important anyway:)

and no, c++ rulez;)

ce mai zici mestere?

> On 08 Jun 2015, at 08:40, Timotei Dolean <notifications@github.com> wrote:
>
> nooooooooooooooooo, newline is missing at the end of the file
>
> C++ is sad :D
>
> —
> Reply to this email directly or view it on GitHub.
>


El ´´´<button>´´´debería estar dentro de la  ´´´<form>´´´ y fala un ´´´<label>´´´


xl(hours)


    """
    Returns the table dimension.
    :param table: Matrix or vector or value.
    :type table: list of lists, list, value
    :rtype: int


Thanks :-)

$(@D) is in fact Hyperstake-Qt.app/Contents/MacOS, so that directory should already be there but I don't think it harms either, on the contrary.


These minor improvements are mainly because I saw minor fixes, which I
could improve simply :)

I'm now trying to create the export action component for ODT, one like
dw2pdf/bookcreator combination.

I don' t expect to contribute to real ODT (xml like stuff) on short term.
(In that part of the export action I hope you can help( or advice) with
building the real ODT file.)
But maintaining and keeping the code of this export action component in
sync with the dw2pdf and bookcreator plugins is fine for me to perform.

I will publish tomorrow my first piece of the export component.

2015-06-08 7:44 GMT+02:00 LarsGit223 <notifications@github.com>:

> I also saw you other changes. I now have to go to work. So I can only
> correct the things you pointed out this evening.
>
> Thanks for checking my code, again. Just a question: are you willing to
> work on the ODT plugin? What are your plans?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LarsGit223/dokuwiki-plugin-odt/commit/a5e88a9b831154008453cdc2de5e7d9f354971a2#commitcomment-11560615>
> .
>


xl('minutes')


remove the and (this will be impossible for translators) and just use a comma


Is it configurable somewhere, apart from changing source code?


xl('seconds')


you do it, and I will merge it. Thx


    def _redim_table(table, n):
        """
        Returns the reshaped table.
        :param table: Matrix or vector or value.
        :type table: list of lists, list, value
        :param n: Delta re-dimension.
        :rtype: list of lists, list, value
        """


react作为单项数据流，是不应该在子组建中更改props的，然后这个赋值的操作应该会有问题。


No


Would also separate into whether singular xl('day') or plural xl('days')


2 раза у вас выполняется getFirstContact(); Здесь и в следующей строке


Esta imagen iría mejor como un background.


Also should to singular vs plural in items below.


Perhaps you should get your local repository in synch with the github repository (or the other way around).


Выход можно вынести в @AfterTest


How about adding it to config, so user can override the default if she wants?


Результат здесь возвращайте типа boolean


"Мусор" не заливаем в репозиторий


I think it *is* an improvement.   Later in the text you still have "marginal" and "conditional".
Maybe you can keep "conditional" (as it is "y given u") and call the deviance / likelihood "unconditional" in the cases you need to distinguish it from the conditional one.


Don't do that. It will mutate the hidden class.


¿Este texto es realmente un subtitulo del ```<h1>``` o debería ser un párrafo?


¿Y el ```<h3```>?


It was a security issue. It is not recommended in general to allow query string params into a flash movie.


no, upgrading maven plugin is a very risky thing, we should consider updates carefully and do them with a motivation (fixing a bug, getting a new feature).


Если выше (в createContact()) написали rebuildCache();, то здесь обнулять кеш уже не нужно


@thisiserik you should also update the version value defined in `lib/react/router/rails/version.rb`


Hmm, didn't catch this initially, since pkg-config works correctly, but I think the original version was correct? On my machine, the include path is `${DEPENDS_DIR}/libusb/include/` without the additional libusb...


вместо всего if можно сразу написать:
return driver.getCurrentUrl().contains("/groups.php") && driver.findElements(By.name("new")).size()>0


demo实现了： 1.Android客户端调用js方法
 2.js调用Android客户端中提供的方法
WebView API：http://developer.android.com/reference/android/webkit/WebView.html#addJavascriptInterface(java.lang.Object, java.lang.String)


AssertJUnit - не писать


static забыли


sync with the original jquery


That is because sp_get_i reads a 64 bit integer, and MVMExceptionBody.category is a MVMuint32


Please, follow the project naming "rules". All source code files *.h *.cpp use LOWER-case naming without any delimiters. So use listdelegate instead of listDelegate. Thank you.

I would like to avoid having a mess in our source code after merging it into master.


what's your reasoning for not overloading track?


Same as before. All comments in source code use SPACE after //. Also, it is good habit to end comments with a DOT since comments are just normal sentences.

So  // Color: #AAAAAA. is better than //Color: #AAAAAA


From enet


Use consistent whitespace indentation. In our project, we use TWO spaces per one indentation level.


Mis-aligned comment.


Hiii one more help..i cannot able to get the resource suggested by you[CloudFerry/devlab/]...Finally,i decided to simply understanding the architecture involving in this migration process by simply go through the entire code...i found one interesting code in this section CloudFerry/tests/cloudferrylib/os/[actions  compute  identity  image  __init__.py  network  storage]  ... is this ur test case...it contains lots and lots of functions under dummy name is created..if I read all the code in this folder,am i able to understand the workflow of this tool and will it be helpful for me to make an architechture.??....Last but not least..thanks in advance...

---
Radhakrishnan.R


On line 165 `prior_draw_gsa` needs to be initialized somewhere else. I think it should be moved in block at lines 101-104.
Also, in block 106-109 [ML estimate], `opt_gsa.prior_range` should be forced to be 1, if not done elsewhere.
thanks a lot


Not sure how that passed by the spell checker - its meant to be unfortunat**e**ly, thanks for pointing it out :+1:


Follow project variable naming scheme. We use LOWER-case names WITH underscore delimiters for LOCAL variables - so use first_list_topic instead of firstListTopic. Class-wide member variables use camel notation with m_ prefix. So typical class member variable could have name m_classVariable.


Use QString() instead of "". It is faster.


Ah so sticking another one into `RaiseException` constructor would be redundant.
`CompilerAsserts.neverPartOfCompilation()` will fire only when it's on hot path, right? Do we have some way to enforce that all our Ruby nodes are always compiled e.g. in tests so we could enforce getting these errors?


Shouldn't you wrap the text with tr("") here? So that it is translatable.


Please avoid magic numbers. Create MACROS for them in definitions.h.in.


Make sure all contructors your explicit keyword and all destructors are marked as virtual.


Test


Hi @mkbond777,

This is normal that in your database the vouchers have cart_rule_restriction set to 1 when you disable "combine vouchers" into your back-office. This means the voucher created can't be cumulate with another one.

Best regards.


Always use TWO blank lines after last #include and first actual source code.


Wrong import. :wink:


Dot.


Shouldn't this be `ImmutableList`?


Fix indentation.


Wrong import.


Using `Collections#emptyList()` will get rid of the unchecked cast warning.


Use spaces around mathematical operators. " > ".


"else" branch is redundant here.


For example during tests. I think since this is a disk based thingy, we should recommend the with statement always. Or perhaps any flush needs to check that the thing it will try to flush is still there.


Indentations....


开新坑啦，相机姿态估计？


Line endings is not the only problem. There are also lots of automatically generated names that change all the time; see screenshot:
![screenshot from 2015-06-08 09-24-27](https://cloud.githubusercontent.com/assets/5850655/8029672/027866a6-0dc0-11e5-96a9-546e870f312d.png)


W README maz numery zadan, powinny byc katalogi z klasami, zeb wygladalo troche lepiej.


mac북과 함께하는 즐거운 코딩.....


How'd that get in there? Wasn't .gitignore supposed to prevent that from being pushed/merged?


Aby wygladalo bardizej elegancko, warto stworzyc public class zadanie1 a w niej klase FizzBuzz. Nie powinno sie nigdy robic za wiele w main()


Почему закомментил, а не удалил?


brakuje docblockow - wszedzie


:cake: :sparkles:


w Javie przyjeta koncencja nazewnicza jest camelCase a nie under_score


to powinien byc wyjatek a nie system out


magic number -> powinien to byc public static Int


Maybe is a problem of my implementation. But with this fix my code doesn't work anymore

SQLSTATE[42S22]: Column not found: 1054 Unknown column


Если не ошибаюсь, генерить вьюхи девайса не обязательно. Ты это специально сделал, чтобы кастомизировать?


Wszystkie metody sa absolutnie BEZ POWODU statycznie, a nie powinny byc.


Gracza powinno reprezentowac sie chociazby prosta klasa Player -> nigdy Stringiem


magic number


looking at this closer, I think it is right the way it is;

for example, assuming vnodes, we own local ranges `(a, b], (c, d], (e, f], (g, h]`. And we have 2 data directories, we flush/compact like this, `[min(minToken), e), [e, max(maxToken)]`

and, for non-vnodes and we have range `(a, b]` - we would flush `[min(minToken), midpoint(a, b)), [midpoint(a,b), max(maxToken)]`

does this make sense?


Без этого никак? Не хорошо такие запросы делать в application_controller


aa


To samo co przy 1szym zadaniu, w main jest zasadniczo za duzo. Powinno byc GraWKosci gra = new GraWKosci(new Player());
gra.start();


В руби методы так не именуются. И скобочки тоже не ставятся. Посмотри в стайлгайде, тебе должны были скинуть ссылку


Else jestr zbedny, powoduje jedyne wieksza indentacje.


to powinno byc w docblocku, a nie komentarzu inline - wystrzegaj sie komentarzy inline


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


wszystkie magic number do likwidacji


@rattoma I fixed the initialization in 27622a29da9163d707d750f03bef951f4ae4f6af
The forcing of prior_range is now done in 6e24c5f37eac7c3a81f6c232dd7a07360590eb34


mmm I cannot see what has changed... for some reason (probably formatting) all lines are displayed as changed. What did you fix?


ta cala klase mozna napisac w paru linijkach http://stackoverflow.com/questions/226910/how-to-sanity-check-a-date-in-java


Actually, that article seems to prefer the "qxl" driver, which we do have. I might get to try to run the failing tests with it.


Why do we need both the db_type and get_internal_type ?


Unless по-хорошему не стоит использовать, если у тебя планируется else.
В этом случае лучше юзай if.


We have typically, just for fun, used the current date when creating tests that involve dates. The actual fun part is looking at those tests later and see how time flies. Definitely no real need to change the date and time here but of course you can... =)


Should imo be extracted into a local var - otherwise it will be (re-)evaluated in each loop. Which is bad in case of this function pointer, because there is not even a chance to inline the call.


Эти лишние файлы запихни в gitignor


$data['product_recurrings'] => $this->getRecurrings($product_id)));
too much ));
and last one =>


Hmm, I don't get it... where else should the state be stored?


LGTM


Have you had much success with this?


やべぇ...作っちまった...


应该也修复了 #605  问题。close #605


VDIサイズの上限設定はやらない方針とする


it should be viceversa. Uniform sampling is triggered with
`opt_gsa.prior_range=1;`
thank you


Good shout! Thank you for the contribution :+1:


these next few chunks are the method bodies for the missing array methods.


OK, I see thank you.


using haxe.ds.ArraySort for now


IPアドレス、Port番号はほかのドライバに準じて標準的なライブラリを用いてチェックする方針とする


#605 的问题应该也就是 【`<select>` 选项为空时 Selected 报错问题】，只是用户描述得不清楚罢了。

Fixes #605.


Indentation here is a bit off, can you amend it?


@lgarciaes Igual es un detalle y no tiene que ver con la finalidad de la kata ... Aún así, si no se hace lazy-load el puntero al archivo permanece abierto a lo largo de todo el tiempo de vida de todas las instancias de la clase! Y sin un ```__destruct``` que lo cierre, hasta que el GC no se ejecute no se cierran los punteros al archivo!


I don't feel good about the (absolute) symlink, it may break subtly later on (e.g. can't cross file systems, may subvert DESTDIR installation and may cause errors for pkgng as well). I'd suggest a portable shell script launcher.

Also, naming conventions for UNIX would make this "configctl". ;)


Just put it in util


Please strip period from the description for consistency (no period for single sentence)


Looks okay, except for the class name `ResourceController`. It's a bit vague considering that it's used for a specific goal: authorization. Maybe `AuthorizedResourceController` is a 'better' name. Bit long, but it's much more telling than `ResourceController`.


This is how the form and the previewer are displayed!

![screenshot from 2015-06-08 10 44 25](https://cloud.githubusercontent.com/assets/11241058/8029958/5c821fec-0dcb-11e5-996f-db7609d73983.png)


so
`
if opt_gsa.prior_range==0`
`        warning('GSA:: When using ML, sampling from the prior is not possible. Setting prior_range=0')`
`        opt_gsa.prior_range=1;`
`end`


@rattoma Ok. I corrected it.


Now I do some changes


If its an API it should go in the API package

Sent from my iPhone

> On 8 Jun 2015, at 8:42 am, Gigabit101 <notifications@github.com> wrote:
>
> Just put it in util
>
> —
> Reply to this email directly or view it on GitHub.
>


There's a typo in this file name that may cause problems if referenced without the typo.


Very descriptive naming there Rob. Great work.


thank you.


Also, can this be deployed? :angel:


@brianharvey The time in MS is the time since 1/1/1970


much simpler.


super instead of __construct.


no more _hx_continues!  Thanks @Simn


rev


roll back


Thanks David, my mistake.


Exemple de fonction ?


> I didn't rebase between the two merges

Then this must be the cause.


If symlinks don't function correctly with pkgng we should certainly refactor this to something else.
Do you have a proper example of the shell script launcher you suggested? I'm not totally sure about the parameter passing between those two.


Also, I noticed a typo in `help/doStopOthers.png` "except" is misspelled in the description of the second block.


With your change, this variable is no longer used and can be deleted.


use "vertx" because it can be resolved in all languages and getVertx() does not make sense in JS, etc...


Ah bah je l'avais mis au début, et comme ca fonctionnait sans je me suis dis que ca ne devait pas être utile. Je vais le remettre


This doesn't look quite right. Should the parameter be ignored?


because toString() does not make sense in other langs and "" + visits is properly handled


Etrange, surement une histoire de cache.


Iterable<Map.Entry> iteration is a java api that cannot be codetranslated


map.values() does not exist in other languages, forEach on map does.


i'm not funding them now, but in general every single page website minified won't go a minified website is a website that deliver html+css+js all embedded inline in a index.html.
e.g. we are working since time in delivering globaleaks as a single html file (this is needed to reduce the overhead of round trips in Tor).

a part from a fully minified website other simple examples that wont work are:

    <head>
    <style>
    body {
        background-image: url("http://onion/url");
    }
    </style>
    </head>

or:

    <head>
    <script>
        ajax_requiest = "http://onion/url");
    </script>
    </head>


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.247](http://teamcity.monogame.net/viewLog.html?buildId=8112&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:24:08


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


I'd add the general case: if you use any Morepath directive using a decorator, or something like that.


Oops, I think I messed up the tabs here


whish -> wish.


エラー内容に応じたExceptionのバリエーションを追加したほうが、
エラーログやエラーチェック機能を充実するよりも好ましい。
（他製品Ceph等を参照してみる）


make it "Python package" the first time you talk about package.


Perhaps a note here that Morepath applies overrides here and may report conflicts.


Morepath is a micro framework. Let's pretend we have succeeded at our aim. :)


Is there some way to avoid wildcard? Maybe add new constant for algorithms, that apple uses fo APN sandbox now?


"If you prefer everything to be explicit, you can use manual scanning." Avoid the word 'magic', I think. Just call it explicit.


collieコマンドはメソッドを共通化を図りたい（見栄え向上、コード量削減）


From the JavaDocs
> * A callback is called upon completing the dispatch of an event. Callbacks
  * are executed in the order opposite of that in which they were registered

So when are the callbacks executed? After the event resolves and the action that the event represents has been done? Or can you now specify a callback occurs before the vanilla callback happening?


Looking at the code, it seems fine :) Know we need a form to configure elts_per_page from the webui :)


"Manual scanning gets tedious and error prone"


whoops sorry, my bug


"which scans all Python packages that depend on Morepath." No will and no try.


I still don't like this bit. Just eliminate this paragraph.


I think these three lines can be made more succinct: "Do not run if CCM updates are disabled globally via the /etc/noquattor file. Unless this option is  set, ncm-ncd will ignore the /etc/noquattor file and run unconditionally." ?


Also remove this paragraph. Just say: "We can use even less code with autosetup".


@jwocher this one looks real


"autoscan combines automatic scan with an automatic commit." Make it an active sentence.


Get rid of this. Instead:

For this to work, we do need to place our own code in its own Python project with a `setup.py`. Let's look at what this looks like.


Get rid of "this should go without saying".


maybe we can add a private field to OSDUtilizationDumper, something like <int32_t, int> osd_pgs, for the first time to call get_bucket_utilization() for the first osd, we scan pg_stat for once, and collect the PG number of all osds into osd_pgs, for the sbsequent calls to get_bucket_utilization(), we retrieve the information from osd_pgs directly, this avoids to search pg_stat many times


I assume the command line processing framework doesn't support negating options, e.g. --no-check-quattor  to unset it again? That would be a bit nicer than having two different options to control/override the same thing.


Replace this: without a `setup.py` Morepath cannot find your Python package.

(Morepath can after all deal with any Python package, including those without setup.py, as long as you scan explicitly.)


Not all the resources on acom.azurecomcdn.net support HTTPS, eg. https://azure.microsoft.com/blog/2014/08/18/enabling-command-line-or-continuous-delivery-of-azure-webjobs/ references *http*://acom.azurecomcdn.net/80C57D/blogmedia/blogmedia/2014/08/publish-stack_thumb.png but this image is not available via HTTPS


no.
Se vuoi lo puoi lasciare come fallback, ma non serve.


There is something a bit wrong here, as when I set the weighting for both lists to the same value, I don't get the same behaviour I did before. You'll probably need to walk me through this.


Maybe this? https://github.com/opnsense/core/commit/01870edad79902925fb4e60f1317d62455a1a634


"This is typically a virtualenv or a buildout during development, but it can also be your global site-packages during deployment."

Does that make sense? I don't think global site-packages can make sense outside of deployment, right?


This wasn't enough, need to call the supervisor after the call fails to ensure it has sent the log message. Exit signals sent before monitors so the  `:EXIT` be received before a `which_children`.


"When Morepath's autoconfig/autoscan machinery looks for packages to scan..."


loving this btw :D


Nice! Looking good. I had to read some of the documentation twice before understanding it (the description, in particular). I think if that was clarified a bit more my confusion on the parameters wouldn't be there. For someone who hasn't looked at the code yet, it kind of runs together and it might help to break up the description into more than two sentences, where each sentence explains the "multiple times", "chooses the best one", "best number of clusters", "silhouette".

Don't use the the `@import cluster` notation. Add the cluster package to the `Imports:` section of the `DESCRIPTION` file instead, so it gets installed automatically.

What's the second `m` in the variable `kmm`? Just curious. Also, there may be other packages that use a function called `kmeans`, so I would namespace it and do `stats::kmeans(data, k, iter.max = iter.max)`.

I think using `$` in scripts is a bad practice (so I read somewhere, can't remember why now). Supposedly better to do `sil_sum['avg. width']` or `sil_sum[['avg. width']]` for a list.

I'm not sure how much performance gains there would be, but you could preallocate the size of the vectors in `km` since you know how many runs you're going to do. They're not huge vectors now, so gains would be minimal, but pre-allocation would be more R-like, I suppose.

Question: why return *all* the clusters objects? Why not just the best one?


I really dislike functions for this simple kind of oneliner code.


Isn't it spelled "install_requires" with an underscore?


This line doesn't make sense, it should be either `Following scopes are identical in both descriptions: ...` or `The scopes ... are identical in both descriptions.`


So if your package depends on Morepath, you need to make it explicit in ``setup.py``:


Duplicate point 3?


Thanks! It was still on my list..... the lack of logging triggered the action :-)


Please provide a correct summary or no summary at all. The usage of an empty summary tag is prohibited.


The Python package name must be identical the same as the name in `setup.py`, or if not, the package or module to scan must be listed in `entry_points`.

Then a definition list:

```
setup.py name the same as package name:
    info here

entry_points:
    info here
```
This allows you to get rid of "alternative A" and "alternative B"


It breaks the karen trunk...


it's not that far


Biiiiiig exception when there is a malformed log-file name (e.g from other projects) :stuck_out_tongue_winking_eye:


Yeah, we should probably do that. I didn't realise quite how long those where! Personally I like a strict limit for comments, code I'd be more flexible on. I'm tempted to enforce 119, that seems reasonable.


Nice!


could this be based somehow on what I am doing here: https://github.com/GNOME/ostree/pull/117  ?


你都闲到改博客了哈哈，看来是真的闲~


Should use maven for it. You can query version online. Example: http://mvnrepository.com/artifact/org.springframework/spring-jdbc/4.1.6.RELEASE


Why put this in comments instead of deleting this.


Thank you for suggestion!
It's awesome. I've added .editorconfig.


OK！
![LGTM](http://puu.sh/bHaY4/57045edeba.gif)


No need to checkin .idea directory.


@ned21 to answer your question: `--no-check-quattor` should work too
but `check-noquattor` is more a boolean to distingsuish between legacy behaviour (i.e. ignore `/etc/noqauttor`) and new behaviour (do as `ccm-fetch`). i do not assume people to set `check-noquattor` on commandline, but rather in the config file.
`NOQUATTOR_FORCE` is there to support same functionality as `ccm-fetch` with same name.


Чтоб помнить что я тут до этого писал. Старая привычка. Перед окончанием все равно пробегаюсь и подчищаю.


:question:  Does HSQL even support JDBC?


I would stick these at the bottom of files, just otherwise somebody will add something in to it here and all the cascadeingness will fall off!


Tested and merged, cheers. Take another look if everything keeps working. I'm going to fiddle with firmware upgrades for a while now. Don't trust the code, hehe.


:cool: This file is regarded as resource file and need put directly under resource directory (and import as src in intellj).


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.248](http://teamcity.monogame.net/viewLog.html?buildId=8116&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:26:40


Thanks, fixed in v0.3


添加新的依赖，所以需要重新安装
maven install


I just followed the general style of the surrounding code where each feature has an init function. I do not mind much either way.

Personally if the function name helps making the code self-documenting without adding comments and hides implementation details (in this case the bindCopy library) I like one line functions


sorry for answering a question with a question, i heard that is inpolite ;)

* why do we need `databag["deployment"][@bc_name]["elements"]["to_remove"]` in the databag after the `_remove` role is done and is no longer in the runlist? Isn't it better to have a clean databag (in a state like it was before) in the end?

I think I have addressed this in one of the last comments on an old commit, but i don't receive my own comments as notification mails, so i can't find it anymore


Please also remove modules/core-api/src/test/resources/com/enonic/xp/xml/parser/XmlRelationshipTypeParserTest.xml which is not longer used.


Will this not throw if element is a string? Suggest

```javascript
if (! (element && ( (element instanceof Element) || typeof element === 'string') ) )
```


Thanks, I'll have a look


あざます！


Ajouter un commentaire pour expliquer le rôle du mixin (reuse label from instance)


Pourquoi le return est modofié


first draft


Why ?


Thanks,
I'll give it a whirl on that version

On 5 June 2015 at 23:18, Geoff Winsor <notifications@github.com> wrote:

> That seems to do the job! I originally put in the --gnu switch when I
> first encountered the bug but there seems to be an issue with parallel v.
> 20141022. I dropped it and went with 20130922 and all works fine. Might be
> worth testing it using the more recent version 20141022.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sanger-pathogens/Roary/commit/e38e8b2b39c1407793f6b3159caa69390f213f17#commitcomment-11550247>
> .
>


I actually meant this line down here: "will cause cider to print extensive information on the `nrepl` buffer".  I'm not sure many people know of this buffer, or what it's for.  On the other hand, most people will be able to find the repl buffer, I think :)


Provide more info in the commit message


This is the VEVENT im trying to fix.

BEGIN:VEVENT
UID:DE-IF-1225
DTSTART;VALUE=DATE:20001225
DTEND;VALUE=DATE:20001226
RRULE:FREQ=YEARLY;INTERVAL=1
SUMMARY:1. Weihnachtsfeiertag
DTSTAMP:20000101T000000Z
END:VEVENT


No need for a temporary storage in the "log" variable. It can be "streamed" directly.


ditto


modify tsung xmpp chat and groupchat data format, fit with easemob msg format


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


Since for jdbc the underlying calls are not on the handle lambda I fetch now a connection as a preroute handler and set it on the context for later use.

Otherwise I would need to have methods to handle inserts, selects and select with args...


Goed om gedacht zal het ook de databasenaam aanpassen


Django select2 rocks impose l'utilisation d'un widget mais la signature de la fonction est celle de Django, il faut donc la changer au risque de ne plus respecter l'ordre des arguments (widget avant empty_label) ou sinon assert. Quand penses tu @bgelineau ?


Using `seq.el`, awesome :)


Yes, there are issues in this area; this is ticket #19705.


@jpuri entering a value in the field "5. [Optional] Specify the maximum funding (for plotting only)" does not change the plot display. It should change the display so that with the value entered here becomes the upper limit of the x axis. It seems to work from the back-end so perhaps the value is not being passed correctly?


See the recent discussion in erl-q@, Ingela proposed more valid patch:
http://erlang.org/pipermail/erlang-questions/2015-June/084868.html


At this point it can be removed from AM_CXXFLAGS, right ?


Obviously I meant why the first Saturday :)


Your MinGW fix caused this:
```
$ ./testparas.pl
gcc version 5.1.1 20150422 (Red Hat 5.1.1-1) (GCC)
This will take a while.
Initial configure...
./configure: line 6952: Extract: command not found
```


NEWLINE AT END OF FILE D:<


You gotta record this :-)


@Ovilia 哈哈哈~闲下来就开始学习才是好徒弟嘛~


What is this one?


Sure, done


what's the semantics of the single pipe?  The Composer docs have only the double pipe as "OR":  https://getcomposer.org/doc/01-basic-usage.md#package-versions


Опечатка `переопределния` здесь и ниже в тексте.


Do we really need a unique iteminfo class for every ammo type? Why not just a 'bullet' item that extends the basic ammo class, and re-use that with different display names / etc.?


@andalusi there was no need for that, the HTML pages are also copied to output, to the same place where the generated HTML is, so it is also using everything in assets.


ok, se c'è già quello di default la fallback non serve più, e poi è comunque doppio, lo elimino direttamente.


Any sample usage example will be appreciated a lot.


It... It wasn't there before D:


it totally was.


@frank-dittrich hi! I don't really know anything about this stuff ;(

Maybe I shouldn't have touched it at all.


Should be in the user's preference page ?


Oh! Sorry, that was just a typo of mine. :-) The information is printed on the repl buffer not the nrepl.


``queryset, widget, **kwargs`` ? Ça fait bizarre, mais c'est sans doute le mieux.


<tt>Exception</tt> or <tt>tvb.basic.readers.ReaderException</tt> would have been good in this case.


I'm not sure that the additional `CAMLlocal`s are needed (and they have a small cost). They seem to never be live across an allocation point.

(The move of `sp = caml_extern_sp` is definitely correct and I assume that is what fixes #4. Is that the case?)


thank you - huge changeset! i will merge and we will test 8-)


La méthode `to_python` de Django retourne une liste d'instance dans le cas d'un choix multiple, triées par `pk`. On ne peut plus reposer sur la valeur de l'attribut `value` en cas d'erreur dans le formulaire pour réafficher les choix de l'utilisateur. Je me retrouvais avec des labels ne correspondant pas aux `pk`s. Je rajoute des commentaires.


Please can you make sure it is better than now


https://github.com/h5bp/Front-end-Developer-Interview-Questions
https://github.com/paddingme/Front-end-Web-Development-Interview-Question


Placeholders for Claudio to fill up


@robynstuart ,

I actually debugged entire flow while reverting the changes. And as I mentioned in the PR comment also. All the parameters all passed from frontend to backend, but params (3,4,5,6,7 ) are utilized by BE only if parameters 1,2 are present:

![screen shot 2015-06-08 at 2 13 45 pm](https://cloud.githubusercontent.com/assets/2182307/8030953/a5e3f990-0de8-11e5-8b12-6bd9ac09d21d.png)

You can see value xupperlim being passed here.

I guess some processing of params passed over from FE is happening in makeccocs.py function plotcurves.


Changed not deleted


@mossroy
Interesting. Does the plugin work without these dependencies?


The [documentation](http://thrift.apache.org/docs/types) says that a string is UTF8 encoded:

> string: A text string encoded using UTF-8 encoding

So I would treat it as a far-end error; if the user wants to send raw binary data, a Thrift byte array should work.


Should be ``.min`` and ``.max``


>Если Не ЗначениеЗаполнено(ЭтотОбъект.Мир) Или Не ЭтотОбъект.Мир.Свойство(ИдентификаторКонтекста, Контекст)

@pumbaEO Женя, а почему ты поменял это условие?
Мы с @ValeraS специально заложили такое поведение - контекст сначала нужно явно установить. если не было установки, это ошибка теста!


I added a fix for this in https://github.com/Frede1012/scripthookvdotnet/commit/2bb78cb328fcd932860953579f6307e5f6d0c55c


I believe this won't work as expected for multi-configuration toolsets like VS. You will need to use a generator expression for this purpose instead. Somethind like:
COMMAND lib.exe $<$<OR:$<CONFIG:Release>,$<CONFIG:Relwithdebinfo>>:/LTCG>


It could be a good idea to add it but I would put it in Rnext (or even Rnextnext). Will you open an issue describing this idea?


2.3-Функциональная система
управление продажами , управление бухгалтерского учета, управления закупками . Для этого будет добавлен модуль управления обслуживания.
В этом разделе мы сначала описать все особенности , характерные для каждого модуля системы в целом . Следующая диаграмма показывает различные связи между
функциональные процедуры.
В этой части мы сначала описать все особенности , характерные для каждого модуля системы в целом . Следующая диаграмма показывает различные связи между
функциональные процедуры:


github test


Hi maximebiloe,

If you think that setting cart_rule_restriction to 1 is expected behavior when "combine vouchers" option is disabled, then that's OK for me. But the vouchers created in this fashion are getting cumulated with other vouchers by the customers.

For your information I installed v1.6.2 of this module and vouchers created using that can't get cumulated with another one. When I checked Database for those vouchers "cart_rule_restriction" was set to 0.

Thanks
Manish Kumar


but why is a single output line better than two? I liked the old output


ease of extendibility reader, a little better readability, automatic close of file?


I don't think this is needed. For VS, cmake generates multi-config solution, that means that it contains all the build types. I guess this change only compensates for the other issue I've mentioned.


@Guria спасибо, поправлю.


J'avais tenté le `**kwargs`, je vais voir mais ça posait problème car Django n'appelle pas son `super` pareil dans `ModelMultipleChoiceField` et `ModelChoiceField`, j'avais des doublons en passant les kwargs bêtement. Sinon, on peut juste reorganiser les paramètres au moment de l'appel


I beg to differ, as we are executing parallel tasks, the logging happens latter grouped together by error type  (it's essential for grouping this extra step)


What is the purpose of this? IMHO it changes nothing. The call to `register()` will not fire `registered` event until a while, so from the user point of view nothing changes. Do I miss something?


optimize (US) rather than optimise (UK)


How about `finally { Transaction = null; }` instead of catch?


Tests should not depend on a working installation in hard coded directory!
To fix tests that couldn't handle several VERSION files, create_test_database.php creates expected symlinks since 4cf8fc45b68f99fa56675152b0e1c14854bdfeb3.


None of this is a "fix", this is just to see if we can get any more information about what is going wrong. But to be honest, I think this is an issue with the bot, rather than anything wrong with the code


1. Ошибка не понятна, да это ошибка но ее имхо можно было-бы оформить в тесте по другому.
2. У меня в bdd процедура может быть случайно вызвана не в том порядке, как контекст был установлен, поэтому надо правильное сообщение выводить.
3. *Сломано обратная функциональность*, т.е. код который раньше работал полностью сломался и нет альтрнативы. Я согласен на метод "КонтекстУстановлен" возвращающий истину или ложь, но не исключение.


Ohne save(...) wird die Änderung nicht gespeichert


Pls update the description.


I think so, yes. In fact, user's preferences could be a form.


Checking all dependencies


리뷰 중 잘못된 변수명이 있어서 수정 하였습니다.


we only use upper case for defines, please tolower() it.


I ve made a mistake on my code, sorry


Be aware that there is no guard for svalue2 being 0 so the behaviour here will differ from Java I think.


对


you're right, changing it to concurrent hash map


Sure! What's your twitter handle?


@m4r0v3r You have to initialize m_favouritesmodel with nullptr in the constructor. Otherwise it might be that you have random garbage and obviously disconnect will crash.


No sorry, what does "E hanno questi attributi" mean? Where is it used? I was translating the french


@iglesias, this issue has been resolved already: https://github.com/Itseez/opencv_contrib/commit/9d38de38b2d26734cfc948108c33c3c2cdc22dd5#diff-75a5692889f7938ba22ff1574cf9f7c0
Please, update contrib repository to the latest revision.


dasdsadsdsdsadsadsadsadassadasd


At this stage `this.dynConfiguration.register` determines whether the UA was registered before disconnection and thus whether it should re-register or not.

If 'connected' is emitted as before, and the user registers in the callback, then such parameter will be `true`and we would re-register again, which is redundant.


added :) thx


wena


Gaaaaanz schlecht! Muss überarbeitet werden!


Is there a difference between SchematicSIP and GeneralSIP?  Can these be merged?


You are making the assumption that PidTagDeliveryTime ordering always guarantee to retrieve the highest mid. Do you think of any use case that wouldn't match this behavior?


THAT was the only one i had asked for.
Still not received an answer!
Check the old signature with `up`/`dn` args.

Why there might be a difference between the `st_cell`/nd_cell`??


I'm going to drop this commit until we can do https://github.com/mysociety/alaveteli/issues/2383, as the shorter title doesn't align with the existing left column:

![screen shot 2015-06-08 at 10 07 45](https://cloud.githubusercontent.com/assets/282788/8031453/3aeb2b30-0dc6-11e5-9838-3a69d1afac1f.png)


You need to be careful when making cosmetic changes to whitespace, as they screw with all other users' merges, and make it immensely difficult to track useful changes. They should also be considered carefully in their own topic branches - this commit has exactly 0 to do with "JENKINS-28575 include GitHub teams as groups", but is adding 1000 lines of changes to it.

Also, as a general stylistic point, never change an existing project from tabs to spaces (or vice versa)


OOOH, but it's recorded from one page to another! I did'nt understood it was working like this, this is really nice! You can forget my lasts comments.


I have filed bugs with patches for all this against the doomsday package. Guess you have seen them already?


@dasilvacontin I added tests and dot notation support in #3. Run `npm test` to run the tests (probably after doing `npm i`). Looking forward for your improvements. :smile:


There is just one problem, you have to display the form all the time. For example, currently if I select "100 per page", and I'm not satisfied with the result, if I have only 95 entries, I can't reset to 20 per page.


I have removed the `--disallow-test-skipping` argument from our global scrutinizer config, as we agreed to remove the build from scrutinizer anyway I'd rather reject this commit. Analyzers can be more easily configured at the scrutinizer web UI.


Indeed, which I think is probably the most common use (at least mine) for this script.


So, if you look at the output, Xamarin Studio starts correctly and then crashes, which usually suggests to me that an assembly has had an issue when being loaded.


> Can you explain what is "order independent"? :)

The warning message is a bit clearer, I hope. It's about abstract code that will not necessarily lead to the same result when the iteration is done in a different order. For example, if you write a synaptic model with `pre='v_post = target_v'`, where `target_v` is a synaptic variable, then this is not order-dependent. If several synapses are targetting the same post-synaptic neuron and those synapses receive spikes at the same time step, then the final value of `v_post` depends on which synapse is evaluated last. Especially in the context of parallel execution, this is not well-defined.


Szia,

Úgy tűnik a logolás valahogy helyreállt, a logging modul végül jól csinálja
amit kell.
Ami gondolm van az annyi, hogy ha több szálat indítok el mint ahány fájl
van, akkor egyszerűen leáll amikor a nem használt szál kilép - a többi is
elkezd dolgozni, de megáll amikor a munkához nem jutó szál kilép. Ez
szerinted mitől lehet? Vagy írjam át "pool"-os megoldásra?

Üdvözlettel / Best Regards:

Kálmán (KAMI) Szalai

2015. június 1. 22:07 Daniel Darabos írta, <notifications@github.com>:

> Teljesen jó szerintem! Működik is? :) Mi a gond a loggolással? Simán nem
> megy?
>
> Ja, lehet, hogy mivel külön processzek vannak, külön loggereket is kapnak.
> Ha ez nagyon kritikus, megpróbálhatod sima threadekkel. Akkor a csodálatos
> Poolnak lőttek, de egyébként ugyanez. (Csak multiprocessing helyett a
> thread/threading/Queue modulokkal előadva.)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KAMI911/wgslas2eovlas/commit/60b2fb5cdc2f7294b66da65124679ec30a65221c#commitcomment-11462019>
> .
>


oculos iacta est :+1:


We plan to start running the specs in a loop, with Graal and a low compilation threshold, and any compiler exceptions being fatal to test this, but aren't doing it yet.

I think the hash specs work like this. You could try BigDecimal.


Увидел переписку в основном репо.
Считаю правильным добавление спец.метода для проверки наличия контекста.
Вопрос закрыт


the way


> Further, one can explicitly

Elsewhere in the docs we use "you" to mean the library user  ([example](https://github.com/ably/docs/commit/5ef1212d709c91e3f36128f9af1dea6c53563040#diff-ae0452c0d98bd5a09eba3a880d849aceR10)), so "one" and "if there is a desire to" (rather than "if you desire to") read oddly


Nice! Thanks a lot,
I will try it.


presenton


Yes, I will add Doctrine SQL driver, which will operate on array results from query, not objects. :)


Does the user will see the warnings of its C-code here ?


Tiptop, merci!
Wiso hesch d lernkontrolle usegno?


Why do we need these as submodules?


Ik zou dit zo verwoorden om de zinnen gemakkelijker te maken:
 However, the parser component has never been carefully analyzed for improvements.
 In particular, the quality of the error messages (hence the user friendliness) is an attention point.


I agree with the overindentation.


ah jo ich mach alli froge ine seperats file und dues den ufelade


We should not decide here on exception level, since the check is only done in debug mode. Thus, in release mode, the code will behave differently.


This one always gets me! Who makes a class that starts with lowercase???


Should this be the root of the server?


Контекст схема
Проектирование и производство
Поле внутренней деятельности :
• Управление продажами
• Закупка и управление запасами
• управление взаимоотношениями с клиентами
• Управленческий учет
• Управление обслуживания