Ok, 0 is now default (and behaves the same way as it does in your GEOS C reduce function, i.e. full floating point precision).

The snapping (and later on, reduced precision mode) would affect any geometry in QGIS which at some stage was processed by GEOS. Various functionalities in QGIS could benefit from the reduced precision mode, since as mentioned above, coordinate transformations etc tend to deteriorate the geometries. But you are right: let's keep the behaviour as before, and have a more general discussion on where to use reduced precision mode once the new GEOS is out.


PersistenceFailure


наконец-то починили тот баг с зависающим браузером


@briatte in case you would know other R users of INSEE data, i would really appreciate if you could direct them to rsdmx. Thanks


Yes you are right. I refactored this to be one method, but forgot to update the message..


How about `Inconsistent parameter name, use ... instead` ?


Thanks @DeenaC! Addressed in 54c40deb46bab7222530c0c86fdfcb805cb418f0.


we should give some thought about what naming pattern we want to use for controller actions. For commands we have so far used the name of the command in the controller action, ie. we used `content_items#put_content` instead of `content_items#update`. This would imply that this action should be `content_items#get_content`.

However, maybe it's preferable to stick to the rails conventions in which case we should consider renaming the commands to follow rails conventions (and this might expose new conceptual resources, controllers etc).

This is not a blocker to merging, but just something which we may want to follow up for the sake of consistency in the codebase.


`paramIsCalled` sounds like it returns a boolean (`isParamCalled`). Also `called` is ambiguous, because "called" can mean the name of the parameter or whether the parameter is called (i.e. when it is a function that needs to be called).

Maybe use *ensureParamName* instead? Also rename `shouldBeCalled` to `expectedParamName`?


Nice, thanks!

I know there are currently no tests for DateConverter at all, but would you mind to create some?

You don't need to write tests for all the cases in this class, just to see that it finds a proper format and actually returns a date.


I used the link here since you used it in the pubsub docs also in the description. I don't think this is to big of a deal, but I can live with just putting the links in the references section ;)


JAR diff at ~keghani/GitHub_diffs/20151008_01/java-build/1.0-SNAPSHOT-jar-with-dependencies_to_1.0-SNAPSHOT-jar-with-dependencies/changes_report.html


Doesn't seem to be used so we can remove it (It was part of the original code)


I thought about this, but opted to avoid #send as I tend to be averse to metaprogramming.

That being said, this is within the test suite, so maybe it's ok?


Yes, you are right,  and always passing "paramNamesFile=paramnames" makes the code looks simpler and cleaner.


package manager.


@fafhrd91 Why? :-(

It logs it with ERROR log level. Maybe at least do it with self.log_debug(..., exc_info=True) ?


You could use `return context.report(...)` as well.


It's a workaround to return poisoning data "bad data" as 0x0808 point to d68000_invalid().
So by poisoning the return of read() we make sur that next read to info->ir and instruction_is_valid() will failed.
And return false to capstone with a skipdata.


Missing \


Agreed. I think we should try to stick to the conventions unless we have a good reason not to.

Because of this, I think we should consider changing `#put_content` to `#update`.


How about `ensureConsistentParamName`? I totally agree with the rename of `shouldBeCalled` to `expectedParamName`. I was thinking about a better name, but nothing better came to my mind ;)


This could also be an `ArrowFunctionExpression`, right? I think there's an `isFunction` utility helper which can be used instead.


good


/cc: @nathansobo @bolinfest for :eyes:


Почему csscomb пустил 0px? и отсутствие запятой после запятой
Что-то с ним вообще не так


Ah good point! So glad you point that out. I was wondering why my test using arrow functions was breaking..


Мою папку не надо было удалять) при принятии вашего пул реквеста моя папка у меня в проекте тоже удалится. Но ничего страшного) Это можно будет обойти. Вам ничего делать по этому поводу не нужно.


Delete this 🚧


Классное название коммита :+1:


je trouve ce patch très bien mais peux tu mettre
//add comment
stp ?


Jon Postel smiles upon you

![Jon Postel](http://www.internethalloffame.org/sites/default/files/inductees/Postel_Jon.jpg)


Где ul?


А может таки responseExecutor? :smile_cat:


As far as I can see, this subverts the whole idea of encrypting passwords. If you accept the encrypted password without hashing, it means your stored (encrypted) passwords act exactly like plaintext passwords. Thus you are storing plaintext passwords, which typically considered a big *NO*.


Список не правильный.


Small typo


Oh... icart_mini_navigationにそのプラグインを追加してしまうのか...

そのプラグインの意図を完全に把握しているわけではありませんが、それとは関係なくプラグインの生成から使用までを一つのパッケージで行うのはマズイですね。 皆さん大会が差し迫っていると思いますのでよっぽどの事が無い限りはスルーするんですけど、この設計は良くないのでパッケージを分離してください。 @ShotaroTerato


Отступ отличается от других.


I understand this now


OMG, my system is soooooooooooo fast!

`State 0 | Speed: 1000000% (500000.01) | EE:  43% | GS:   4% | frame-i | GSdx D3D11 HW | 640x512 | None`


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


Как-то боязно вызывать `responser` только с одним аргументов вместо трех. Ты как бы полагаешься на то, что именно при этом вызове `socket` и `request` не будут использованы, но это очень хрупко.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 52](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=265&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:00:54


Any reason the key isn't passed as an InternedString? It is for the other methods, and from what I recall it's stored that way internally as well.


@WHPODOO I believe the issue is using the "ODOO" suffix in your username bringing some confusion if you belong or not to Odoo S.A.


Superfluous comment, because it says the same as the code.

General rule of thumb: Comments -> Why, Code -> What.


Stop lying man...


试下能不能评论


orne_navigationリポジトリにxx_layerのプラグインを作成するパッケージを作成し、icart_mini_navigationはその生成物を使用するように修正してください。 作業の負荷はそれほど無いと思います。 その後masterにマージします。


Maybe just let forEach run anyways? Probably doesn't matter.


Is this necessary? `timoutCop` is only used in the if block. CoffeeScript will hoist the variable declaration, and the arrow function will capture the variable in its scope? Maybe the lack of :coffee: is screwing with my head, but I'm pretty sure we can :fire: this line.


I think it is easier to read if use the positive version of the condition @vbalazs


Nice speedz! with room to spare! :P


given that we [made the contract tests more general using `Pact.like`](https://github.com/alphagov/gds-api-adapters/commit/712987a9903fde3d465ec52cf8ff432b39aa9301), I think that this `#create` should use the defaults from the factory, aside from the `content_id` (which is declared explicitly in the provider state).


All of these (except path and env) are no longer used.


I still can find some occurences of table->find()
e.g. Kwf_Component_Generator_Model_Row::save(), Kwc_Abstract_Admin::_getRow()


好好好


This is `env: NON_METEOR` now.


I have no idea what's the purpose of definition of IMPL...........
and how to use the macro in Makefile @@

2015-10-08 13:40 GMT+08:00 Jim Huang <notifications@github.com>:

> Since we already defined macro IMPL, the access to macro OPT is redundant.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/poa1006/phonebook/commit/51a7c46a7822faf9af9ca228f37683286e4d4b13#commitcomment-13655206>
> .
>


fsdfsafsdsd


you should use Mage::getStoreConfigFlag() instead


Возможно, но например `request` в этом месте вообще нету.


Do we want to be more specific about this? We want descriptive prop types (+ `isRequired` and an explanatory comment if necessary) on child components that use the prop. Do we ever want these on components that _only_ pass a prop through?


Btw. MD5 is widely considered to be not suitable to hash passwords, since it is not very computationally expensive.


That's a tricky question. My gut feeling is no, given that we've gone to a fair amount of trouble to ensure that the outward-facing behaviour is the same with or without `CollectionMember` (up to the point that an admin actually creates some collections), so there's no reason to *not* inherit.

However, given that there's been some suggestion of implementing swappable permission/categorisation models as a future development (#1384, https://groups.google.com/d/msg/wagtail-developers/kE-aDrOIzRI/IC_Tc6btAwAJ last para) I was reluctant to tear out the existing permission logic. Essentially this is my attempt at making collections a pluggable module "in spirit", since we don't currently have the hooks in place to make it a true pluggable module - I've tried to keep it so that Document inheriting `CollectionMember`, or not, is the sole point of integration that determines whether the collection mechanism is active or not.

We also need the "with/without `CollectionMember`" switch on the images side anyhow (to accommodate custom image models in a way that doesn't require developers to create complex migrations at the point of upgrading Wagtail), so eliminating it for documents wouldn't be a massive gain.


:question: What is the 'medium' in date fields for


Can I be sure that properties is an array when `eventMap.type` is an `ObjectExpression`?


It'd be great to see a bugfix release. I'm currently trying to package `actiona-3.9.0` for [Void Linux](https://github.com/voidlinux/void-packages) and have to patch a lot to make it compile with our `qt-5.5.0` packages :)


my bad- sorry


А почему имя аргумента `fetchMeta`? Слишком близко к `fetchMetas`, только одно -- функция, которая возвращает промисы, а другое -- сам промис. Может лучше банально `promise` или хотя бы `metaPromise`?

Еще {} и return лишние.


ajout def


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


I asked on Digilentinc Forum.


What is going on with this line?

Generally it's better to use
```bash
$(sha256sum $HEX_FILE)
``` rather than using the backticks?


All problems resolved with https://github.com/amule-project/amule/commit/4af1ed5b15f2549847f8aff68c9f66dde7dcb860
thanks


hm.. nicht so toll, dass hier noch 2 breaks gemacht werden. Wen man minifizierte Dateien arbeitet, würde das unnötige Bytes brauchen.


I think we can delete these two methods


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


This is some strange behaviour. Why there are now quotation marks around it and that additional space at the end?


Mismatch with `only_distance` parameter name.


Assignment Branch Condition size for diff_teams is too high. [15.07/15]


Assignment Branch Condition size for diff_teams_members is too high. [21.24/15]<br>Method has too many lines. [12/10]


Use a guard clause instead of wrapping the code inside a conditional expression.


Align the parameters of a method call if they span more than one line.


Не подписали.


Expression at 13, 74 should be on its own line.


Avoid using {...} for multi-line blocks.<br>Block body expression is on the same line as the block start.


All problems resolved with https://github.com/amule-project/amule/commit/2cff458bfe1639f6b704ee58584b995f656f43da
thanks


Controls how the date is displayed, else it will come out in the format on
the database, which human beings will not appreciate
On Oct 8, 2015 10:48 AM, "andela-toladele" <notifications@github.com> wrote:

> [image: :question:] What is the 'medium' in date fields for
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/andela/roots/commit/2acde38bb38264929ac95d5affb2d30a677fc503#commitcomment-13658803>
> .
>


Assignment Branch Condition size for log_creating_teams is too high. [16.12/15]<br>Method has too many lines. [13/10]


Use `next` to skip iteration.


Прекрасный пример того, что в JS тоже нужен `fmap`.

И опять {} и return лишние.


shouldn't this be equals.


А вот тут подписали)


Space inside square brackets detected.


This is exactly the improvement I was looking for. Any chance to get out a `1.0.4` release?


shouldn't it be build35?


Definitely some leftover code, thanks @thomasjo!


Yes - delete permission is implied by edit permission, so that's covered by the comment "(It is assumed that the user has edit permission for the document)". In other words: we don't need a specific check for this, because we assume that the views that call `target_collections_for_move` have already confirmed that the document is editable and therefore deletable.

Having said that, I wouldn't be averse to adding an explicit check in, if you think that'd make the logic clearer.


Macro `IMPL` is used for indicating the name of current implementation.


The amount of lives should be part of Player, since if you have more players you can't have more "life counters" in the LevelController.

<organisation


It's still missing a `return devices` after line 228, or you could instead invert the condition to do `avdvice_list_devices` only if `av_opt_set_dict2` doesn't fail and free unconditionally, but otherwise `avformat_free_context` will hapilly double-free if you call it twice.


Please read gcc manual carefully.


Вы сам притянули себе изменения в моём проекте. Не надо удалять папки других студентов если сами их к себе притягиваете. Если я приму ваш пул реквест, папка которую вы удалили у меня тоже удалится.

Вам ничего делать не надо. Просто больше так не делайте.


LGTM


Might be room here to mention / link to [stateless functional components](https://facebook.github.io/react/blog/2015/10/07/react-v0.14.html#stateless-functional-components) from 0.14.

(Web frontend should probably discuss how we'll use these if we haven't yet)


Хм, а может сделать event в зависимости от типа? На exchange это reqEvent. На остальном надо указывать явно как resEvent


C'est moins puissant que gerrit, on peut pas mettre les commentaires en face de la modif


of course, this breaks scripts that are not supposed to be run on NixOS machines


Индентация сьехала


If this is the amount of starting lives, it should be a static value in Settings.

<organisation


Can this check be moved before the search for `key`, to avoid the redundant work? Maybe the `if( !node )`  above could just become `if( !node || !root->isAncestorOf( node ) )`?


I agree that metaprogramming can sometimes obscure the intent behind code or make it difficult to follow, but I think in this case it simplifies and makes it easier to follow. Of course you also have to be careful about using `#send` if the method name is derived from user input, but in this case as you say its only ever invoked from tests so there's no risk of user input being involved.


Instead of this, we could just do:
```ruby
module Presenters
  class ContentItemPresenter
```
below, which I think would be cleaner.


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


you can use username.isEmpty() which is more readable


Yes, according to https://github.com/estree/estree/blob/master/spec.md#objectexpression


ok !

2015-10-08 17:53 GMT+08:00 Jim Huang <notifications@github.com>:

> Please read gcc manual carefully.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/poa1006/phonebook/commit/51a7c46a7822faf9af9ca228f37683286e4d4b13#commitcomment-13658918>
> .
>


Yep, fixed this in https://github.com/gasman/wagtail/commit/ad41e5e88a5bc137292134ec9166da2c2cb76d7c#diff-75ce9a3ce9403576ff07c2e76f1ee3b8R65. (This was needed to fix the tests, as they call get_image_form on custom image models while the global setting is still using wagtailimages.Image.)


OK I will move it to the end

From: Vineet Gupta [mailto:notifications@github.com]
Sent: Thursday, October 08, 2015 10:47 AM
To: EZchip/linux
Cc: Noam Camus
Subject: Re: [linux] ARC: arch_spinlock use cpu_relax() for optimization (91d8a07)

I understand u kept the patch here as part of the 3 buckets (upstream, ARC, eZChip platform). However it seems we need to add this - as an optimization - after EZChip platform introduces it's cpu_relax() - NO ?

Also with your change u r adding a barrier inside the loop - which might not be desirable for other platforms. FWIW, a failed trylock need not be a barrier - I need to check that again and get back to you !

Anyhow this patch is contentious and needs to go at the end !

—
Reply to this email directly or view it on GitHub<https://github.com/EZchip/linux/commit/91d8a07d9135693464d182eb6acda4596b741ebb#commitcomment-13656575>.


All problems resolved cloning from actual HEAD (https://github.com/amule-project/amule/commit/2cff458bfe1639f6b704ee58584b995f656f43da)
thanks


This is an auto-commit from Weblate.
I'll modify it manually before publishing the new release


Ну я говорил, самая большая проблема у меня с неймингами. :(
Может `fetchedMeta` ?
Насчет `{}` и `return` - ты прав, но мне так нагляднее. ок, уберу.


На сервере в этом момент еще аттачментов нет. Он досылает их потом.


hehe


InstanceValues is `ordered_unique`, so unless I'm missing something we shouldn't need a linear search here. See the lookup in `instanceValue()` below where `find()` is used instead.


Okay, let it stay then :)


@manuranga We ignore other queries, only support Delete and Update queries. Please see the message.
and also we have removed this validation.
Thanks.


Sorry, I forgot about **sample** operator, so *$var-variables* can be a continuous ones.
The correct solution is to infer variability of substituted variable  from its binding expression.


lgtm


Съехал отступ.


分区名不一样而已，分别尝试 OS1 和 firmware，能刷进去就可以了。


原来如此，感谢~


formating issue here.


BTW, the fact that the repository field in package.json points to the non-existing "babel-plugins/babel-plugin-react-display-name" repo is quite confusing. What's the story behind this? Also GitHub issues seem to be disabled for this repo. Is this on purpose? Am I even looking at the right project?


Thanks. this was fixed in https://github.com/wso2/carbon-kernel/commit/7600bef19ca3e062454a38bb1ff9283cfbe6482a


As far as this function is concerned it doesn't know about consistency, because it doesn't know the param names of other functions. It ensures that one function has this and that param name at index *x*, so I think `ensureParamName` is better suited.


'a+rwx' does not sound like correct approach from the security POV


Эти красные поля означают, что строки удалены и вместо них стоят другие строки. Всё в порядке.


Всё хорошо.


`fetchedMeta` подразумевает, что он уже зафетчился, а это промис -- он еще не зафетчился.

Я для супер-коротко-живущих имен люблю что-то в духе `{p => p.then(...)}`, но знаю, что кому-то такое не понравится.


Can we use a RecursiveNodeIterator here, to avoid the casting and pruning within the loop?


Ah - it's using the wrong audio description. Fixed.


added myself @CleoQc


Added ```ERR_BadAwaitInStaticVariableInitializer```.

@jaredpar, @gafter


You did a really nice job with the semantic HTML markup in this assignment, nice work! I left some comments above that I think will be helpful in the future and here are some additional thoughts:

1. Organization is important, especially in your CSS files. A general strategy is to break out CSS into related rules and add comments to the file to signify those sections. I.e.
  - utility rules that we've copy pasted (box-sizing, clear fix)
  - base styles (styles on single tags like p, h1, h2, em, etc)
  - then choose an organizational approach for the rest of the file. I usually organize the rest by area of the page (i.e. a nav section, a sidebar section, etc)

2. Formatting and look of the page is also important. Make sure your code looks as pretty as your site!

3. You duplicate selectors in several places in your CSS file (i.e. `header h1` turns up more than once). You should try to include all the styles for a single selector in the same block. It's hard for someone else looking at your code to figure out how an element is supposed to look if you define it in different parts of the file.

4. Make sure you copy _all_ the files and folders your site needs into the "final" folder. As it is, your images aren't in the final folder, so the site is broken.

You asked about centering in your remarks, so here is some general guidance for centering things.

1. Applying `text-align: center` on a parent element will center all the inline elements and text inside of that html element.
```
<style>
.my-centered-content {
  text-align: center;
}
</style>
...
<div class="my-centered-content">
  <span>This will be <strong>centered</strong></span>
</div>
```
2. To center a block element (within it's container - which could be the entire body), use `width` in combination with `margin: auto`.

Let me know if you have any additional questions:

#FEWD Week #2 Project: Styling with CSS

---

###Description

A hot, new NY Tech Startup, Relaxr, has approached you to help them develop a new landing page for their company. They've handed you the design file for the site, along with the copy and assets. You need to take these files and turn it into a landing page.

Student:  Kathryn Connolly

|                                                                                |                                |                        |                          |
|-------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| Technical Requirements                                                                                | Does Not Meet Expectations (0) | Meets Expectations (1) | Exceeds Expectations (2) |
| Use HTML5 structural elements (nav, header, footer)                                                                           |                                |                        |            X              |
| Use a CSS Reset file in addition to your style.css file to style the page                                                     |                                |                        |           X               |
| Use IDs and Classes to to select and style elements on the page                                                               |                                |                        |           X              |
| Style your text with the Google Fonts provided by your style guide                                                            |                                |                        |           X               |
| Follow naming conventions, maintain consistency across .html and .css files and use best practices for naming IDs and Classes |               X               |                        |                          |
| Indent nested elements to increase your code's readability                                                                    |                                |           X             |                          |
| Use width, padding and margin on at least two block elements on your page                                             |                                |                        |             X             |
| TOTAL: *11*/14                                                                                |                                |                        |                          |


Minecraft Modが

> 動作にはMinecraftが必要です

って言ってるみたいで謎。


:+1:  Gracias @san650, sí eso contesta mi pregunta.
La pregunta iba más bien por el lado de cuando se mostraría.


Good, I should learn how to branch, so my pullrequests works properly :-)


Mas se o wait falhou foi porque houve alguma interrupção ou algo do género antes de ele "encontrar" o filho, ou não?
Pensei que o efeito fosse o mesmo que se não tivesses chamado o wait


I'm not sure I totally understand how let! messes up this spec. Could you explain a lil bit


本文よりもコミットメッセージに記載するかWIPブランチを切ったほうが適切。
*ref.:* WIP PR(Work In Progress Pull Request)の運用フロー（作業見える化） | Coffee Breakにプログラミング備忘録 <http://to-developer.com/blog/?p=1379>


ライセンスファイルへのリンク先が不正


Oh- great! I was starting to think through it and I was getting myself nice and confused :smile:


Eliminar ficheros de configuración de eclipse propios de paqo


oh I instantiated the variables because I need a different number of movies in each example.


Hehe me too! I tried reverting it and the sound went really bad on the Mac (still doesn’t work in playthrough mode, by the way, but it seems to record mostly fine, couple of minor glitches - but that’s all a whole other story we won’t get into right now =)).

> On 9 Oct 2015, at 11:07 AM, Steve Rubin <notifications@github.com> wrote:
>
> Oh- great! I was starting to think through it and I was getting myself nice and confused
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/TheAmazingAudioEngine/TheAmazingAudioEngine/commit/7892f550462c51d9f3116fd5aad86182c300e7ca#commitcomment-13676294>.
>


@clanam there's a test in this package that actually tests for trailing/preceding spaces.
Hm how will it break if they're both empty?


Rename to `TypeResponse` to be more Go-y?


s/Ubuntu/Debian/

also:

Gentoo: emerge -auvDN dev-lisp/clisp

Not sure what Arch or Red Hat based distros use.


Nice use of the Google font! You may want to specify at least one "fallback" font in case this one doesn't load:

```
font-family: 'Open Sans', sans-serif;
```


> In context and natural language: "When `bindingsMap` became empty, remove it from `bindingsMaps`".

Maybe I would add that as a comment. It's indeed a bit hard to understand.


Não, isto é no caso em que acontecem mesmo erros (muito improvável).
Mesmo que haja "interrupções" antes ou mesmo que o filho ainda não tenha terminado quando o pai chega ao wait, ele espera à mesma.

Se der erro no wait uma vez, provavelmente não dá para sair daí.


What doesn't work in playthough mode? Playthrough works for me.


To make our class names consistent, they should never start with a capital letter (even though it's a name)


@Chocobo1 can you tell me if there is a better way to do this?
I was really frustrated about indentation. The docs show(in their examples) that the variables declared under `env:` block should be indented. But the validator errored out, until I wrote them without indentation.
Do you have any guidelines for the indentation?
PS: Travis for this commit: https://travis-ci.org/sledgehammer999/qBittorrent/builds/84417253


Since you aren't using any styles after this, you can probably just erase it (or move it up to where it's appropriate). Important to keep our code neat and tidy!


I think this commit broke the project — the `components/qneh.coffee` was removed, and there's no trace of a `components/q-n-eh.coffee` to replace it.


nvm, i just did ''.substring(1) and it's fine. *palm-forehead*

in that case, you could replace the whole method with one line!

```
properties.className = classSet(properties.classSet, properties.className).trim();
```


Nicely done ordering your stylesheets


Estava a falar duma interrupção que fizesse o wait terminar, mas sim, é provável que já não dê. Fica o break ;)
Já agora no makefile o clear serve para quê?
E o "love" está a mais não? ahah


It's stuttering for me - here's a demo: https://www.dropbox.com/s/4r1azez7uxcws8b/demo.mov?dl=0

Probably an easy fix, but I've not looked into it yet.


kjmklkjl


This `h1` is "inside" the `#container` `div`, so we should indent it to make that easily visible:
```
<div id="container">
  <h1>Relaxr</h1>
  ...
```


Closes #37


74564912Issue4347BBRadConvHW (wfbuhl) - 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-71.66%25-red.svg)


Maybe a `<blockquote>` tag would be a good choice here


Thanks for reviewing @klauszhang! The `:hidden` selects only non-visible inputs. Hence, it should be fine I guess and a little bit more flexible regarding form changes – what do you think?


Ah! Mesmo que o filho já tenha acabado, ele espera como zombie que o pai faça wait.
O clean serve para quando fazes "make clean", os ficheiros objecto (.o) são apagados. Estava no makefile do professor.

"make love"... esqueci-me de tirar...vou tirar agora


yeah.. I think so. but actually when I see it on heroku. the javascript actually selected the “name” on dropbox.. this is weird... this is javascript...
> On 9/10/2015, at 1:21 PM, Lukas Pirl <notifications@github.com> wrote:
>
> Thanks for reviewing @klauszhang <https://github.com/klauszhang>! The :hidden selects only non-visible inputs. Hence, it should be fine I guess and a little bit more flexible regarding form changes – what do you think?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/sdm15stream2/serler/commit/02192d5c13f33c91daa6aa4c1d8ca01792934e86#commitcomment-13676437>.
>


Seriously just ignore this. I needed to add the script I had been working on (AdvVRTracking.cs) so I could improve it with the Oculus computer. 100% not in a working state. Next commit will probably be the _good_ one. If everything works out, that is.


Interesting. What's your microphone samplerate set at?

On Thu, Oct 8, 2015 at 5:18 PM Michael Tyson <notifications@github.com>
wrote:

> It's stuttering for me - here's a demo:
> https://www.dropbox.com/s/4r1azez7uxcws8b/demo.mov?dl=0
>
> Probably an easy fix, but I've not looked into it yet.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TheAmazingAudioEngine/TheAmazingAudioEngine/commit/7892f550462c51d9f3116fd5aad86182c300e7ca#commitcomment-13676412>
> .
>


a comment


I'm not sure where all this Parameter -> name replacement came from, I can undo it if it wasn't accidentally pulled in from another branch?


The documentation needs to mention that this only takes effect after the next connect (an explicit disconnect is recommended). Alternatively you could go through a disconnect-reconnect here. I forget whether the SDK supports releasing and re-acquiring the DHCP lease, that would be ideal.


Hum certo, tinha visto lá mas não sabia o que fazia.
Ah makes sense ahah


LGTM


Should throw an exception, right?


Makes sense - I guess the same logic should apply to all the context variables in this section, but I'll fix that up.

Thanks


Thanks Matt, I wasn't sure how to upload images onto here


Atenção


So to address one of your questions as to why these images aren't rendering. There are a couple thing going on here:
1. One is a typo where you have three dots (`.../`) instead of (`../`)
2. The bigger problem is the path is not quite right. From here in index.html we need to go into `images` and find the `twitter.png` file. You have us going "up" one level first (`../`), but we don't actually need to do that: `images/twitter.png`


Atenção que mudaste o makefile da branch errada - a menos que vás mudar nas duas, mas pelo sim pelo não aviso antes que vá o antigo para o fenix x)


Probably we should get rid of `Constants.go` altogether and consolidate the constants with the enum in dispatch.go


Epá obrigado. É o sono. Posso submeter no fénix então (depois de corrigir)?


If there are multiple trees in ExternalSyntaxTrees, should each of those be considered "submission" syntax trees?


will do


Add a `requireNonNull` for `columnReferences` and make callers provide an empty set, if there are no column references


105015426-DefVAVTUMinMax (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1373 of 1373 tests passed)

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


Better, with less copying and allocating:
```java
return ByteBuffer.allocate(2 * Longs.BYTES)
  .putLong(uuid.getMostSignificantBits());
  .putLong(uuid.getLeastSignificantBits());
  .array();
```


looks like this should have `|| die`


Na boa ahah
Se já tiveres alterado de novo para break em vez de continue e se tiveres posto aquele comentário que estava em pt em inglês, siga, submete o zip ;)


`getEncryptionKey()` method throws Exception in case error occurs.


`instanceof` already checks for null. A null reference is never an instance of any type.


Oh damn, you're right. Let's see if we find a more elegant way to find this field.


bad commit note: actually added team support entirely


I think it might be running at something weird like 16 K. Not 100% sure, not at my desk right now.

> On 9 Oct 2015, at 11:24 AM, Steve Rubin <notifications@github.com> wrote:
>
> Interesting. What's your microphone samplerate set at?
>
> On Thu, Oct 8, 2015 at 5:18 PM Michael Tyson <notifications@github.com>
> wrote:
>
> > It's stuttering for me - here's a demo:
> > https://www.dropbox.com/s/4r1azez7uxcws8b/demo.mov?dl=0
> >
> > Probably an easy fix, but I've not looked into it yet.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/TheAmazingAudioEngine/TheAmazingAudioEngine/commit/7892f550462c51d9f3116fd5aad86182c300e7ca#commitcomment-13676412>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


So `receive` intercepts the request and do not forward it unless you add `and_call_original`?


You're high lol
On Oct 8, 2015 7:58 PM, "Dennis S Lebhardt jr." <notifications@github.com>
wrote:

> Peace, love, and happiness
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CandyDevices/kernel_htc_msm8974/commit/881401a0c00a45b1d7d720118e203e17c85d9e76#commitcomment-13676203>
> .
>


Ah e esta linha no make file está bem? o commandlinereader.h não está a mais?
par_run.o: par_run.c commandlinereader.h par_run.h


well, if you insist... try first two child selector? personally I don’t think there is any difference using the id selector.. If we need to introduce “more advanced (complex)” stuff.  they all need to change anyway.. so …don’t bother.
> On 9/10/2015, at 1:33 PM, Lukas Pirl <notifications@github.com> wrote:
>
> Oh damn, you're right. Let's see if we find a more elegant way to find this field.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/sdm15stream2/serler/commit/02192d5c13f33c91daa6aa4c1d8ca01792934e86#commitcomment-13676569>.
>


Can't do that, because the className itself needs to be trimmed to pass the tests.


Yes, I noticed that the double negation in the comment does more harm than good. Given the non-looping loop is not needed, the code reads perfectly clear to me and doesn't really need a comment, however, I don't mind adding one either.


id selectors broke my neck a couple of times, so consider it a personal thing. ;)


Also do for xml model


* Item 1
* Item 2
  * Item 2a
  * Item 2b


OK… myself have some personal thing with javascript.. I understand..
> On 9/10/2015, at 1:38 PM, Lukas Pirl <notifications@github.com> wrote:
>
> id selectors broke my neck a couple of times, so consider it a personal thing. ;)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/sdm15stream2/serler/commit/02192d5c13f33c91daa6aa4c1d8ca01792934e86#commitcomment-13676636>.
>


Unfortunately `ByteBuffer` doesn't account for endianness


Cool -- I'm pattern matching a bit here, so I'll keep the pattern and we could add an issue for going through and performing a cleanup.


You should use more meaningful variables names


Maybe. Probably better to do it from project.lock.json files -- we actually want the transitive closure of dependencies.


Yes, but I understand that's impossible (per @kevinh_ms).


See [`ByteBuffer#order(ByteOrder)`](http://docs.oracle.com/javase/7/docs/api/java/nio/ByteBuffer.html#order%28java.nio.ByteOrder%29).


You should follow the java naming convention.. do not use underscore "_" in variables, plz use camel case


Nice work on this assignment, overall it looks really good! A couple of comments in addition to the ones I had above:

1. You did nice work having comments in your CSS file to help organize it. Take that even further! It's so helpful to have ultra-organized code, especially as your projects grow.
2. The reason you are having trouble centering your `<button>` elements is because it is an "inline" element as opposed to a "block" element. You can center "inline" elements in one of two ways:
  - You can put a `text-align: center` on it's parent element (in this case the `header`). `text-align: center` centers text as well as inline elements contained inside of it.
  - You could change it from "inline" to "block" by using the `display: block` property and then give it `margin: auto` to fill up sides with margin.

Note that with both approaches mentioned here, you'd need to get rid of the `float: left` property on the button.

#FEWD Week #2 Project: Styling with CSS

---

###Description

A hot, new NY Tech Startup, Relaxr, has approached you to help them develop a new landing page for their company. They've handed you the design file for the site, along with the copy and assets. You need to take these files and turn it into a landing page.

Student:  Kaylin Fabian

|                                                                                |                                |                        |                          |
|-------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| Technical Requirements                                                                                | Does Not Meet Expectations (0) | Meets Expectations (1) | Exceeds Expectations (2) |
| Use HTML5 structural elements (nav, header, footer)                                                                           |                                |                        |            X              |
| Use a CSS Reset file in addition to your style.css file to style the page                                                     |                                |                        |            X              |
| Use IDs and Classes to to select and style elements on the page                                                               |                                |                        |            X              |
| Style your text with the Google Fonts provided by your style guide                                                            |                                |                        |            X              |
| Follow naming conventions, maintain consistency across .html and .css files and use best practices for naming IDs and Classes |                                |                        |           X               |
| Indent nested elements to increase your code's readability                                                                    |                                |           X             |                          |
| Use width, padding and margin on at least two block elements on your page                                             |                                |                        |            X              |
| TOTAL: *13*/14                                                                                |                                |                        |                          |


oh.

then single line

properties.className = classSet(properties.classSet, (properties.className || '').trim());

:D


done


i'm really not helping, i know. ^_^'


mycat 5台实战


It's impossible to have more than one ```ExternalSyntaxTree``` if ```IsSubmission == true```.  0 and 1 are the only two valid states for submissions.


There we go:

> All `option` elements are considered hidden, regardless of their `selected` state.

From the [docs](http://api.jquery.com/visible-selector/).


saw approval from kuenishi
at https://github.com/laurenrother/riak_cs/commit/d608c2ecda2402f91268e946e916d06460224c1f


merging laurenrother/riak_cs/2.1releasenotes = d608c2ec into borshop-integration-1256-2.1releasenotes


laurenrother/riak_cs/2.1releasenotes = d608c2ec merged ok, testing candidate = d8f8c4aa


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/36bd3d9386556155fb9743fcd38479e420e603bb


merging basho/riak_cs/bugfix/still-rt-is-failing-oh-my-bad = 36bd3d93 into borshop-integration-1262-bugfix/still-rt-is-failing-oh-my-bad


basho/riak_cs/bugfix/still-rt-is-failing-oh-my-bad = 36bd3d93 merged ok, testing candidate = f356aaf0


Aha, that’s why…you can only consider the child selector if you don’t want to use id.
> On 9/10/2015, at 1:47 PM, Lukas Pirl <notifications@github.com> wrote:
>
> There we go:
>
> All option elements are considered hidden, regardless of their selected state.
>
> From the docs <http://api.jquery.com/visible-selector/>.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/sdm15stream2/serler/commit/02192d5c13f33c91daa6aa4c1d8ca01792934e86#commitcomment-13676762>.
>


Hi Jeff, Eric, ccdos, Jeffery, lvxin, oldrev,

This change is related to Chinese COA, your inputs will be highly appreciated.  有关中国科目表的提交，需要各位的审阅。

Thanks,
Oliver

------------------ Original ------------------
From:  "pvy-odoo"<notifications@github.com>;
Date:  Thu, Oct 8, 2015 11:05 PM
To:  "odoo/odoo"<odoo@noreply.github.com>;
Cc:  "Oliver Yuan"<oliverzgy@openstone.cn>;
Subject:  Re: [odoo] [FIX] l10n_cn is back, compliant with the new accounting(2c48bfe)

We've just finished merging the other CoA : odoo-dev@1073e74
 It hasn't been merged yet so any comment is welcome and appreciated :)

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


good job stefan


Nice job having a fallback font


You should use more meaningful variables names


LGTM


thnx  :+1:

Gruss
Peter Pirker-Frühauf

.....................................................
 <<>> Peter Pirker-Fruehauf
 <<>> Systemberatung IT
 <<>> Astallerstr. 19
 <<>> 80339 Muenchen
 <<>> mobil +49 160 156 4340
 <<>> www.ppfnet.de<http://www.ppfnet.de>
.....................................................

Am 09.10.2015 um 01:31 schrieb Welcome Help! <notifications@github.com<mailto:notifications@github.com>>:

Looks very well! [:+1:]

—
Reply to this email directly or view it on GitHub<https://github.com/welcomehelpde/engelsystem/commit/0bd548792461f0a1d0df5d35e4eecdd662857c17#commitcomment-13675949>.


try to be consistent with spacing here. For example, you have a single space between `footer` and `{`, but no space between `p` and `{`. Small detail, but in general, consistency is important!


fix_binary_literal_sizing (lefticus) - x86_64-Linux-Ubuntu-15.04-clang-3.6-AddressSanitizer: OK (376 of 376 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)


Wow, nice use of media queries!! Generally, there are a set of standard "breakpoints" for media query sizes that relate to screen sizes of various popular devices. There is nothing wrong with using ones that you deem important (i.e. the 500px you use here), but it may be more efficient to the standard numbers. This site has a VERY extensive list of common media query "breakpoints": https://responsivedesign.is/develop/browser-feature-support/media-queries-for-common-device-breakpoints


Closes #3


You should put this logic in the controller


update_unit_testing (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1914 of 1914 tests passed)

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


测试


Yeah that's actually precisely the reason that I left it out, but I guess it doesn't hurt to be explicit.


@martinbigio should babelRegisterOnly.js be in packager folder?

Build is still broken on current master.

We either need to tell npm to install ``babelRegisterOnly.js`` from root folder using ``package.json`` where all root files are specified or move ``babelRegisterOnly.js`` into ``packager`` as ``packager`` is already specified in ``package.json``


another comment


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1670


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

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


You could use a `<cite>` tag here


A `<small>` tag may be more semantically appropriate for this section


I've reverted this commit and implemented something something similar but better handling `INCREMENT_DEVEL_VERSIONS`.


@oliverzgy We will have a look


fix_binary_literal_sizing (lefticus) - x86_64-Linux-Ubuntu-15.04-clang-3.6-ThreadSanitizer: OK (376 of 376 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)


an open-source pipeline for Affymetrix microarray data analysis


fix_binary_literal_sizing (lefticus) - x86_64-Linux-Ubuntu-15.04-gcc-4.8: OK (376 of 376 tests passed)

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


@mattpodwysocki FYI tag and the npm version say 4.0.1 not 4.1.0.


Cũng được đấy @KennyTung94  hỗ trợ @phixuandao  tốt đấy @@


Was just thinking, why not just have the Publishers global instead of having a separate global ptr to each?


s/hte/the/


Wow! Looks really great. I don't have too much to add other than my comments above, other than, generally you want to include your font style sheet _before_ your own stylesheet. You had a question in your submission about space between the footer and the article above it, but I saw you fixed it! Here is a small explanation of why setting the `p`s `margin-top` to 0 fixed it:

What you're are encountering is the "collapsing margin" problem. Essentially because the `p` element and it's parent (the `footer`) share the same top border, their margins are collapsed together. So the `p`s top margin is bleeding out the top of the footer. There are two ways to combat this. One is the way that you've implemented (setting the `p`s `margin-top` to 0). The other way is similar to how we fix the "collapsing container" problem for floating elements - set the `footer` property `overflow: auto` in order to force it to contain it's children completely.

#FEWD Week #2 Project: Styling with CSS

---

###Description

A hot, new NY Tech Startup, Relaxr, has approached you to help them develop a new landing page for their company. They've handed you the design file for the site, along with the copy and assets. You need to take these files and turn it into a landing page.

Student: Marne Pike

|                                                                                |                                |                        |                          |
|-------------------------------------------------------------------------------------------------------------------------------|--------------------------------|------------------------|--------------------------|
| Technical Requirements                                                                                | Does Not Meet Expectations (0) | Meets Expectations (1) | Exceeds Expectations (2) |
| Use HTML5 structural elements (nav, header, footer)                                                                           |                                |                        |            X              |
| Use a CSS Reset file in addition to your style.css file to style the page                                                     |                                |                        |            X              |
| Use IDs and Classes to to select and style elements on the page                                                               |                                |                        |            X              |
| Style your text with the Google Fonts provided by your style guide                                                            |                                |                        |            X              |
| Follow naming conventions, maintain consistency across .html and .css files and use best practices for naming IDs and Classes |                                |                        |         X                 |
| Indent nested elements to increase your code's readability                                                                    |                                |                        |           X               |
| Use width, padding and margin on at least two block elements on your page                                             |                                |                        |           X               |
| TOTAL: *14*/14                                                                                |                                |                        |                          |


This should be 0.10.0 since in backwards incompatible. I'll revert tomorrow and rerelease


Do you mean to remove the catch or the entire tcp socket close ?


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1671


Fixed. Thanks


I would rather we include the expected-replace version of this drop method on the PointDataLeaf as with the appendAttribute() method, as this one forces each Leaf Node to have it's own unique descriptor which is wasteful.

Also, I think we should include a wrapper to this method that operates on the PointDataTree as with the appendAttribute() method in PointAttribute.h. This method is also unit tested through the testPointConversion() method so we should include something similar for an equivalent drop method.


:confused:


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


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


fix_binary_literal_sizing (lefticus) - x86_64-Linux-Ubuntu-15.04-gcc-4.6: OK (376 of 376 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)


:heart:


102867486-FixRectangleShapeTest-5173 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1901 of 1910 tests passed)

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


I made the the change regarding the List and also there was a bug in the removePastry method.  I wasn't returning the right value 100% of the time because I was continuing to loop through the rest of the collection even after I found the List that contained the pastry.


should be 中国小企业会计科目表


Hehe, no its not done yet. Very rough still. Need to put more work into it.


should be 中国会计科目表-企业会计准则


No it's not. Because of c4c83af80cc2a2cc4ab400ae3c230b8804906d15 it is still backwards compatible.


fix_binary_literal_sizing (lefticus) - x86_64-Linux-Ubuntu-15.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


should be 其他货币资金


should be False


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


Sure


do you think maybe that commit had a point


can we have a way to put account 1406 as property_valuation in product category ?


This change increases the graph load time by 15ish minutes. Should probably profile to see whether the number of nodes has increased dramatically or whether the remote server bandwidth is too low.


@bsmithgall I don't get it :(

I'm getting all these context errors and I don't really understand why. I can render the template, but I want to *actually* test that an expired opp is posted. Should I do that from the admin test file?


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


TeamCity ChameleonForms :: Continuous Integration [Build 1.2.1-PullRequest113+034](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3668&buildTypeId=bt20) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 3010, ignored: 1 Build time: 0:0:0
### Failed tests
```
ChameleonForms.AcceptanceTests.dll: ChameleonForms.AcceptanceTests.ModelBinding.ModelBindingShould.Function_correctly_with_checkbox_and_radio_lists: Test(s) failed. TestStack.Seleno.Configuration.Interceptors.SelenoReceivedException : {"errorMessage":"Unable to find element with link text 'Model Binding Example with lists'","request":{"headers":{"Accept":"application/json, image/png","Connection":"Close","Content-Length":"64","Content-Type":"application/json;charset=utf-8","Host":"localhost:49365"},"httpVersion":"1.1","method":"POST","post":"{\"using\":\"link text\",\"value\":\"Model Binding Example with lists\"}","url":"/element","urlParsed":{"anchor":"","query":"","file":"element","directory":"/","path":"/element","relative":"/element","port":"","host":"","password":"","user":"","userInfo":"","authority":"","protocol":"","source":"/element","queryKey":{},"chunks":["element"]},"urlOriginal":"/session/3c9f50a0-6e24-11e5-9b53-e391c0c1e7e3/element"}}
  ----> OpenQA.Selenium.NoSuchElementException : {"errorMessage":"Unable to find element with link text 'Model Binding Example with lists'","request":{"headers":{"Accept":"application/json, image/png","Connection":"Close","Content-Length":"64","Content-Type":"application/json;charset=utf-8","Host":"localhost:49365"},"httpVersion":"1.1","method":"POST","post":"{\"using\":\"link text\",\"value\":\"Model Binding Example with lists\"}","url":"/element","urlParsed":{"anchor":"","query":"","file":"element","directory":"/","path":"/element","relative":"/element","port":"","host":"","password":"","user":"","userInfo":"","authority":"","protocol":"","source":"/element","queryKey":{},"chunks":["element"]},"urlOriginal":"/session/3c9f50a0-6e24-11e5-9b53-e391c0c1e7e3/element"}}
   at TestStack.Seleno.Configuration.Interceptors.CameraProxyInterceptor.Intercept(IInvocation invocation) in c:\ConsoleBuildAgent\work\f035b8dd08e41a61\src\TestStack.Seleno\Configuration\Interceptors\CameraProxyInterceptor.cs:line 52

```


//commit


This is awesome, thank you! I've been meaning to find a way to wake up the screen without using that weird passcode thing for a while but I never got a chance. Just tested and it works perfectly. Thanks again.


đéo up được cay


Nope the reason it was done like that was to avoid useless sql queries.


Nice naming conventions :wink:


Hey there, just wanted to pop in and ask what you need this for? If I think there's a good use case for this, I'd like to support it. Cheers!


So this is why it failed?


TeamCity ChameleonForms :: Continuous Integration [Build 1.2.1-PullRequest91+033](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3669&buildTypeId=bt20) outcome was **SUCCESS**
Summary: Tests passed: 3011 Build time: 0:0:0


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


Is there a reason why you wouldn't want it on by default?


fast-forwarding 2.1 to borshop-integration-1256-2.1releasenotes = d8f8c4aa


Merge sha f356aaf0 is stale.


merging basho/riak_cs/bugfix/still-rt-is-failing-oh-my-bad = 36bd3d93 into borshop-integration-1262-bugfix/still-rt-is-failing-oh-my-bad


basho/riak_cs/bugfix/still-rt-is-failing-oh-my-bad = 36bd3d93 merged ok, testing candidate = 8ac14243


update_unit_testing (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (840 of 840 tests passed)

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


You'll probably have to reformat the indentation for this function call.


Don't put trailing comma and closing parentheses together like this.


Again, `,)` is poor style.


Poor indentation.


What's with this blank line?


yea, he was the one that suggested it.


Put spaces around your `-`.


See microvm/microvm-meta#43

(I failed to get the cross-reference syntax correct in the commit message.)


Ah, didn't see that. Ok, my bad.


Never thought of that. Awesome.


thế này a?


I have played around with `autogen`ing helper functions and I'm not really happy with the results. My preference now is to have the helper functions written by hand in a separate module and take advantage of the Pythonic way of doing things, like this [http://stackoverflow.com/a/3394902] and this [http://stackoverflow.com/a/36908].
Thanks to @antonvh and @ddemidov for making me learn more about Python and hopefully making me a better architect/programmer.
Besides, the `DcMotor` devices would need the some of the same helper functions like  `run_timed()` and friends. I think there's a way to create an instance of a `MotorHelper` class that takes one of the base EV3 motor classes in the `__init__` and then figures out which methods apply - but it's too late tonight to figure it out.


Hmmm it's a pity that we need this - https://github.com/sahib/rmlint/blob/develop/lib/shredder.c#L1463-L1464 should be enough.  The fact that it isn't suggests there is a still a race in the shredder logic somewhere (and that the delays associated with disk reads are enough to prevent the race from triggering).  I will look into this some more and hopefully find a solution.


My tutor said everything about this looks correct.  No errors.


ok ok


Not my best work, by any means. I think this is a dead end - too many language specific variants - but I had to try...
I think we'll end up doing helpers by hand, but I'd like to keep this in `spec.json` so we have a concise way of documenting which helpers should be written, OK?


民权经济学草要


khôn phết :+1:


cho thằng tôm với thằng hòa vào nhóm đi


I thought you could disable coerce mode for HNI functions via an attribute. I'm pretty sure that there are even places that actually do this in the standard extensions...


TeamCity ChameleonForms :: Continuous Integration [Build 1.2.1-PullRequest132+037](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3670&buildTypeId=bt20) outcome was **SUCCESS**
Summary: Tests passed: 3011 Build time: 0:0:0


fast-forwarding 2.1 to borshop-integration-1262-bugfix/still-rt-is-failing-oh-my-bad = 8ac14243


This is awesome @sheaivey, would it be possible to share the Eagle files? I'd love to modify this to use an Arduino pro mini and through-hole resistors to make assembly a bit easier.


alo


you mean use
```
// tag busybox busybox:test should have failed,because busybox:test is existed
c.Assert(out, checker.Contains,"Conflict: Tag busybox:test is already set to image")
```

But when the assert failed, the log will not display the error message "tag busybox busybox:test should have failed,because busybox:test is existed"


On 10/08/2015 10:01 AM, Marc Feeley wrote:
> Really? In what way?

I don't remember now, but I do remember struggling to keep the number of
temps down at the time.

Brad


The `kill` here is because the container is under the `d` model and we should stop it when finish the test.


To Be Built Out.


will refactor this to cache the property names, it's much faster: http://jsperf.com/access-keyed


Nope. There's just attributes for if it returns `null` or `false` if coercion fails.


remove


ignore


test


giỏi thế, //bổ sung hj


试试


@nidico no `pool-path` ;) the other one doesn't work, I have tried it before.


Jen nevim, jestli je stastne nahradit "prvni uhradu" za "potvrzeni platby", vetsinou to ceka na platbu a ne na nase zpracovani platby a je dobre vedet, ze jde o prvni uhradu, ze neni neobnoveny.


yes


@lancepioch Good catch! You can submit a PR if you want


this class should be final


nice catch! :)


Usando el TextView invisible se borran los datos cuando cambias de estado? O es sólo por hacer el paripé?


You should consider using a dedicated service to send the emails, handle the errors etc...

http://alexandre-salome.fr/blog/Generate-Mails-With-Twig


Probably should add a is_callable check here since method_exists doesn't check visibility


Thanks for the hint about the nopanic PDF. I reuploaded that one too, now. Hopefully this is everything now...


Would we be better naming these digests functions with a dasync name so it is clear these functions are implementing something specific for the dasync engine (like the rsa function names), same with dummy pause job. I know they are not exported so I don't feel strongly about it, more from a consistency point of view.


:+1: nice job


Changed already. I'll push it on the next commits


I didn't know this either until I tried to run some tests on this.


It occurs to me reading again that the last sentence "There are no other external dependencies (...)." is pointless & argumentative, I'd get rid of it.


with dedicated controllers you could get ride of this test and use template inheritance instead


You need an absolute url here


ok


hope this will be a great success as more capacity will nuetraI


Oh a really good catch!


closes https://github.com/freifunk-gluon/gluon/pull/504


It is in silicon with no way to configure.
The architect was not aware to PMD align requirement.

Sent by MailWise<http://www.mail-wise.com/installation/4> – Your emails, with style.:)

-------- Original Message --------
From: Vineet Gupta <notifications@github.com>
Sent: Friday, October 9, 2015 12:16 PM
To: EZchip/linux <linux@noreply.github.com>
Subject: Re: [linux] ARC: [plat-eznps] Use dedicated user stack top (2f3380e)
CC: Noam Camus <noamc@ezchip.com>

so between STACK_TOP and CMEM there is unused gutter of 1MB.
It would been better if CMEM start was also aligned to 2M (PGD) so in future we could provide some mechanism to map those to user space just like normal pages !
I think this is not programmable - etched in silicon ?

—
Reply to this email directly or view it on GitHub<https://github.com/EZchip/linux/commit/2f3380e0c44de7b4fe0f3d9eb3ccdd2e0e38760e#commitcomment-13682172>.


@micheljuillard I am not really happy with this commit. The naming with the attached ```_1.m``` is not really informative. Even worse, it breaks all codes and pull requests affecting the likelihood functions. For example, the prefiltering bugfixes no apply to a different function. I would have preferred something like ```dsge_likelihood_wrapper``` for the outer function and leaving the names of the functions unaltered


Do we want to force the validation on the component users? I would leave it up to the user to decide if he wants to have the validation running and disable it by default (by leaving it as `null`)


Thanks for input.

I will probably move this to an option so I will be able to validate the given callback is a callable when loading the Admin class.  It would also allow use i.e. a an anonymous function as a callable


October 9th 2015 Jari Marjakangas student number 1660180
Sudoku Game is made by the following rules: Rule #1 - Rule #4
Note!
Programs are: SudokuVerifier.java and TestSudokuVerifier.java
SudokuVerifier.java program is not working quite perfectly, but the principle is correct (there are two failures).
TestSudokuVerifier.java program is not working quite perfectly, but the principle is correct (public void test1() is ok, (public void test2()should be ok but (public void test3() and (public void test4() are not yet implemented ok, nut
The test has been run public void test1 and it worked ok.

Sudoku is a game with few simple rules, where the goal is to place nine sets of positive digits (1...9) into the cells of a fixed grid structure (i.e. board).

The Sudoku board (or global grid) consists of a 3x3 arrangement of sub-grids, and each sub-grid is a 3x3 arrangement of cells. This yields a 9x9 arrangement of cells on the Sudoku board.

A valid Sudoku solution should conform to the following rules:

R1: A cell in a Sudoku game can only store positive digits, i.e. 1...9.
R2: All digits appear only once in a sub-grid, i.e. they cannot repeat.
R3: A digit can appear only once in the rows of the global grid.
R4: A digit can appear only once in the columns of the global grid.
Your task is to check the validity of a given solution for a Sudoku game

You should read the candidate solution from a string variable, which should be exactly 81 characters long, i.e. first 9 are the first row, second 9 are the second row etc.
You shall check whether the provided string follows the correct format (i.e. 9 rows with 9 entries in each row).
You shall check the validity of the candidate solution against the rules listed above.
You program shall provide the functionality to return an integer code on the validity of the solution:
Return 0: means it is a valid Sudoku solution
Return -1: means it is violating Rule #1
Return -2: means it is violating Rule #2
Return -3: means it is violating Rule #3
Return -4: means it is violating Rule #4
You should implement the functionality with a single API method with the following signature:
public int verify(String candidateSolution)

Here are the Sudoku code:

//SudokuVerifier.java
I'm sorry that I did not get my Sudoku program code to GitHub.

Here is my Sudoku program code:

package org.univoulu.tol.sqatlab.sudoku;
import java.util.Arrays;

public class SudokuVerifier {

public int verify(String candidateSolution) {
int[]   counts = new int[9];
        int ok_count = 0;
}
// returns -1 if the candidate solution is correct
if (candidateSolution.length() != 81)
return -1;

int i = 0, j = 0;

for(; i < 9, i++)
{
for(; j < 9; j++)
{
candidateSolution.charAt(i * 9 + j);
}
}
return 0;
// returns -2 : means it is violating Rule #2 : All digits appear only once in a sub-grid, i.e. they cannot repeat
// 3*3 grid blocks
        for (int i = 0; i < 3; i++)
        {
            for (int j = 0; j < 3; j++)
            {
            ok_count++;
                for (int r = i * 3; r < i * 3 + 3; r++)
                {
                    for (int c = j * 3; c < j * 3 + 3; c++)
                    {
                        counts[arr[r]1 - 1]++;
                        if (counts[arr[r]1 - 1] != ok_count)
                            return -2;
                    }
                }
            }
        }

// returns -3 : means it is violating Rule #3 :  A digit can appear only once in the rows of the global grid.
        for (int i = 0; i < 9; i++)
        {
            ok_count++;
            for (int j = 0; j < 9; j++)
            {
                counts[arr[i][j] - 1]++;
                if (counts[arr[i][j] - 1] != ok_count)
                    return -3;
            }
        }

        //returns -4 : means it is violating Rule #4: A digit can appear only once in the columns of the global grid.
        for (int j = 0; j < 9; j++)
        {
            ok_count++;
            for (int i = 0; i < 9; i++)
            {
                counts[arr[i][j] - 1]++;
                if (counts[arr[i][j] - 1] != ok_count)
                    return -4;
            }
        }

}

} //END

//TestSudokuVerifier.java
package org.univoulu.tol.sqatlab.sudoku;

import static org.junit.Assert.*;

import org.junit.Test;

public class TestSudokuVerifier {

@Test
public void test() {
SudokuVerifier sv = new SudokuVerifier();
assertEquals(0, sv.verify("417369825632158947958724316825437169791586432346912758289643571573291684164875293"));
}

*@Test
public void test2() {
SudokuVerifier sv1 = new SudokuVerifier();
assertEquals(0, sv1.verify("123456789912345678891234567789123456678912345567891234456789123345678912234567891"));
//fail("Not yet implemented");
}

@Test
public void test3() {
fail("Not yet implemented");
}*/

@Test
public void test4() {
fail("Not yet implemented");
  }*/
}

A correct Sudoku string: 417369825632158947958724316825437169791586432346912758289643571573291684164875293

An incorrect Sudoku string: 123456789912345678891234567789123456678912345567891234456789123345678912234567891


I love it. Very impressed.

I was wondering, would it be possible to annotate the list of candidates (above and/or below) with the +Light gain from the infusion (e.g above) and a -Light sacrifice (i.e. sacrifice item level - new_infused level)?

So you could see at a glance which items would not be used to their full potential?


Thanks Dan. IIUC, even though the case that you mentioned might be problematic for other scenarios - checking pointer for NULL and accessing it after &&), the if statement above would not fail/crash if the "right" side executed before the "left". Although I like to fix all review comments, I would like to keep the code as is., if it is OK with you


You’re not receiving notifications from this thread.


Yepp, I was thinking of make a button, 'Transfer all to this character' so you have all the items ready for infusion...


LGTM


@h-lame @mikejustdoit What do you think about this approach for stubbing out the known_manual_slugs?


DAFUQ?


Should we have a check at the start of this function whether ctx is NULL, as ctx is passed to the data macro which is then accessing ctx->md_data. This comment is relevent for digest_sha1_init, digest_sha1_update and digest_sha1_final.


It should pull from `DATABASE_URL`, get rid of this man


Remove the note.


необходимо посчитать сумму цифр в числе. например вводим 81, программа считает 8 + 1, получаем 9.


`set`


`discard` and you won't need to check if it's in there or not


return (queryId != null ? queryId.equals(that.queryId) : that.queryId == null);


I think I need some more information to understand what is going on here, care to elaborate? :)


Actually, you can just `return set(items) - {'reg_date'}`


good point, thx


Yes, it is okay. Nothing to add. Thanks!

On Fri, 9 Oct 2015 11:30 Ben Cipollini <notifications@github.com> wrote:

> @alexsavio <https://github.com/alexsavio> [image: :+1:]
>
> I rebased and pushed to bcipolli/enh/py3k. If it's OK with you, I will
> force-push here, and this will become mergeable. Anything you'd like me to
> do before force-pushing?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alexsavio/nipype/commit/3b9463141e2cc7929aa10df125654813ae1778f8#commitcomment-13682463>
> .
>
--

Sent from my phone, sorry for brevity or typos.


typo, `move`


Hm... since `move` and `rename` boil down to the same operation, I'd prefer to only have `move` and document to use `move` to rename stuff accordingly in the API docs instead.


looks good


Testing comment from github


Add param to change context colors (default, primary, info, success, etc) with a default context.


after iOS9.0 ，like '0.4' , system change it become NSTaggedPointString, change '12345678' become CFString, NSTaggedPointString string will crash in the JSONKit, ‘- (NSString *)JSONString’ method.
Do you understand.


There is `find_by` for pattern `where(...).first` (http://api.rubyonrails.org/classes/ActiveRecord/FinderMethods.html#method-i-find_by). For next time ;-)


hahahahaha


I'm not sure there's any more elegant way to do this, having thought about it for a few minutes. On my own server there's a mod_rewrite that appends trailing slashes when they're missing.


Unfortunately, this name is still unclear. Like I said, you don't have to enter the full path to the texture, but just the name after "texture_". I think this could better be renamed to `textureName` and add JavaDoc saying `this will load a texture in the gui package with name "texture_[textureName].png"`.

<review, naming


Sudoku Game is made by the following rules: Rule #1 - Rule #4
Note!
Programs are: SudokuVerifier.java and TestSudokuVerifier.java
SudokuVerifier.java program is not working quite perfectly, but the principle is correct (there are two failures).
TestSudokuVerifier.java program is not working quite perfectly, but the principle is correct (public void test1() is ok, (public void test2()should be ok but (public void test3() and (public void test4() are not yet implemented ok, nut
The test has been run public void test1 and it worked ok.

Sudoku is a game with few simple rules, where the goal is to place nine sets of positive digits (1...9) into the cells of a fixed grid structure (i.e. board).

The Sudoku board (or global grid) consists of a 3x3 arrangement of sub-grids, and each sub-grid is a 3x3 arrangement of cells. This yields a 9x9 arrangement of cells on the Sudoku board.

A valid Sudoku solution should conform to the following rules:

R1: A cell in a Sudoku game can only store positive digits, i.e. 1...9.
R2: All digits appear only once in a sub-grid, i.e. they cannot repeat.
R3: A digit can appear only once in the rows of the global grid.
R4: A digit can appear only once in the columns of the global grid.
Your task is to check the validity of a given solution for a Sudoku game

You should read the candidate solution from a string variable, which should be exactly 81 characters long, i.e. first 9 are the first row, second 9 are the second row etc.
You shall check whether the provided string follows the correct format (i.e. 9 rows with 9 entries in each row).
You shall check the validity of the candidate solution against the rules listed above.
You program shall provide the functionality to return an integer code on the validity of the solution:
Return 0: means it is a valid Sudoku solution
Return -1: means it is violating Rule #1
Return -2: means it is violating Rule #2
Return -3: means it is violating Rule #3
Return -4: means it is violating Rule #4
You should implement the functionality with a single API method with the following signature:
public int verify(String candidateSolution)

A correct Sudoku string: 417369825632158947958724316825437169791586432346912758289643571573291684164875293

An incorrect Sudoku string: 123456789912345678891234567789123456678912345567891234456789123345678912234567891

Here is my Sudoku program code:

package org.univoulu.tol.sqatlab.sudoku;
import java.util.Arrays;

public class SudokuVerifier {

public int verify(String candidateSolution) {
int[]   counts = new int[9];
        int ok_count = 0;
}
// returns -1 if the candidate solution is correct
if (candidateSolution.length() != 81)
return -1;

int i = 0, j = 0;

for(; i < 9, i++)
{
for(; j < 9; j++)
{
candidateSolution.charAt(i * 9 + j);
}
}
return 0;
// returns -2 : means it is violating Rule #2 : All digits appear only once in a sub-grid, i.e. they cannot repeat
// 3*3 grid blocks
        for (int i = 0; i < 3; i++)
        {
            for (int j = 0; j < 3; j++)
            {
            ok_count++;
                for (int r = i * 3; r < i * 3 + 3; r++)
                {
                    for (int c = j * 3; c < j * 3 + 3; c++)
                    {
                        counts[arr[r]1 - 1]++;
                        if (counts[arr[r]1 - 1] != ok_count)
                            return -2;
                    }
                }
            }
        }

// returns -3 : means it is violating Rule #3 :  A digit can appear only once in the rows of the global grid.
        for (int i = 0; i < 9; i++)
        {
            ok_count++;
            for (int j = 0; j < 9; j++)
            {
                counts[arr[i][j] - 1]++;
                if (counts[arr[i][j] - 1] != ok_count)
                    return -3;
            }
        }

        //returns -4 : means it is violating Rule #4: A digit can appear only once in the columns of the global grid.
        for (int j = 0; j < 9; j++)
        {
            ok_count++;
            for (int i = 0; i < 9; i++)
            {
                counts[arr[i][j] - 1]++;
                if (counts[arr[i][j] - 1] != ok_count)
                    return -4;
            }
        }

}

} //END

//TestSudokuVerifier.java
package org.univoulu.tol.sqatlab.sudoku;

import static org.junit.Assert.*;

import org.junit.Test;

public class TestSudokuVerifier {

@Test
public void test() {
SudokuVerifier sv = new SudokuVerifier();
assertEquals(0, sv.verify("417369825632158947958724316825437169791586432346912758289643571573291684164875293"));
}

*@Test
public void test2() {
SudokuVerifier sv1 = new SudokuVerifier();
assertEquals(0, sv1.verify("123456789912345678891234567789123456678912345567891234456789123345678912234567891"));
//fail("Not yet implemented");
}

@Test
public void test3() {
fail("Not yet implemented");
}*/

@Test
public void test4() {
fail("Not yet implemented");
}*/

}


I like this way, how to enforce implementation in subclasses. I often forget it.


There's a lot of code repetition going on between this and `move_folder` -- any chance to refactor that so that we have some shared `_move` method that both utilize?


Same goes here, `textureName` seems better.

<review, naming


You should mention the effect or benefit form your modifications.


closes #83


Suggestion: how about a version update in both index.js and module.json?


It feels a bit strange to me as the list of acceptable slugs is class-level information, whereas we're setting it as instance-level information.  That said, it's following the precedent set by topics and consistency is good.  It's also clearly better than it was; it's easy to stub things out as demonstrated by the specs - good job!  So I reckon we're good.

What do you think @mikejustdoit?


Quite honestly, I changed that to 1, just for the sake of sense. If we have a selectable box, you will mandatorily want to select one item. In any case, I dug a bit into the code, and realised the ``ams`` property is being used to do several checks, plus, minSelectableItems is not used anywhere, neither on select or dropdown.

Is there any functional difference to it being 1 or null? Because no check/property is using minSelectableItems at all.


Mentioned fs_visitor in a comment. To be changed for vec4_visitor, but first it is necessary to examine if this comment makes sense for vec4.


Leave a comment coffee ~~this~~ commit ~/gitconfig  -- global


Hm... doesn't that also need some kind of overwrite check and hence a `allow_overwrite` parameter? Or do we deny copying folders to a place that already exists altogether since merging might be a PITA?


Are you testing anything here? I would try to test whether the skin is actually being equipped on clicking the list, instead of just clicking something. This is because Travis will not be able to see in the game whether equipping a skin is successful. Of course this is enough for the local runs, but not for Travis.

<review, testing


Not playing sound from local at the beginning supports m3u8 format?


That's a clever solution, but I am not sure if it fits into our presenter concept. Up to now, they have been one-way data reader. Wouldn't it be better, if additional meta data were created here: https://github.com/Madek/madek-webapp/blob/madek-v3/app/controllers/modules/media_entries/meta_data_update.rb ? I am not sure. Let's take a look at it next time together.


Thank! Search_tests didnt get this error


Given that @janhancic is no longer at Caplin, the following feedback can be made **after** we merge the issue...


first comment


Good so far, see my comments above.


obtain -> obtained


не понял, что за parent?

не хватает еще url-а


it's -> its


Since you have implemented `findName`, you can safely remove the comment "TODO".


куда ссылка делась? Если url-а нет, то по умолчанию ставить '#' но урл бывает


`actionCb` -> `actionCallback` (on the basis that it fits better with the naming we've used elsewhere)


parent, это селектор того элемента на странице, внутри которого баннер будет высчитывать ширину


`removeAtion` -> `removeAction`


@pwz3n0 thanks a lot! - can you create a PR? :grin:


Названия событий выноси в константы в заголовок компонента.


Minor typo here: extra parenthesis at the end of the line.

After this update, these two warnings remain:
```
cl.exe   -I. -I../../../../../include -nologo -TP -W4 -wd4127 -O2  -DUNICODE -DHB_HAS_PCRE -IC:/hb/src/3rd/pcre -DPCRE_STATIC -DHB_HAS_ZLIB -IC:/hb/src/3rd/zlib  -Fohbsocket.obj  -c ../../../hbsocket.c
hbsocket.c
../../../hbsocket.c(3250): warning C4996: 'gethostbyaddr': Use getnameinfo() or GetNameInfoW() instead or define _WINSOCK_DEPRECATED_NO_WARNINGS to disable deprecated API warnings
C:\Program Files (x86)\Windows Kits\10\include\10.0.10240.0\um\winsock2.h(2192): note: see declaration of 'gethostbyaddr'
cl.exe   -I. -I../../../../../include -nologo -TP -W4 -wd4127 -O2  -DUNICODE -DHB_HAS_PCRE -IC:/hb/src/3rd/pcre -DPCRE_STATIC -DHB_HAS_ZLIB -IC:/hb/src/3rd/zlib  -Fohbsocket_dyn.obj -DHB_DYNLIB  -c ../../../hbsocket.c
hbsocket.c
../../../hbsocket.c(3250): warning C4996: 'gethostbyaddr': Use getnameinfo() or GetNameInfoW() instead or define _WINSOCK_DEPRECATED_NO_WARNINGS to disable deprecated API warnings
C:\Program Files (x86)\Windows Kits\10\include\10.0.10240.0\um\winsock2.h(2192): note: see declaration of 'gethostbyaddr'
```


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


Somehow `data` is empty. Could you (@thiefmaster) please check before merging?


E.g.
![screenclip](https://cloud.githubusercontent.com/assets/76093/10391623/09540c3c-6e78-11e5-8b87-b06db4ad6566.png)


Iam approving ra nayeem


I want this module for the HDMI2USB stuff!

What is the best way to copy it?


It's used [here](https://github.com/ember-vcl/select/blob/master/main.js#L133) to set the validation but nothing more then that. It's a bit confusing because `maxSelectableItems` isn't validating it's limiting how many items you can select. I agree that it would be good to clean up the API of the component a bit but this is how we've decided to do it for now after a quick chat with @vanthome


Would this be better called ssl_start_async_job as it is specific to ssl, i.e. it raises SSLerr's when things fail.


Yes, or at least, we don't support it anymore because it was unreliable in term of performances.
We generally use a raspberry pi as host (but any other similar host can do), connected with the reference board through their native SPI connectors.


@brettwooldridge
side question: as close() should be no-op for already closed connection, class ClosedConnection should have ?

if ("close".equals(methodName)) {
   return Void.TYPE;
}

calling closedconnection.close() should not throw exception.


nice :+1:


While technically correct, I personally would rephrase this comment. :-)
This isn't the Linux kernel development team! :-D

Also; this seems automated; I assume IDE or formatter related issue?


You are right. Thanks for the Note.

I think I used the global variable while I was doing some tests and
forgot that it is only used by the disable function.


This is an example.


Sounds more like the current tutorial should be broken up into smaller chunks of information per page @devknoll rather than adding the H1 tags back? Once the tutorial is in smaller pages, having H1 tags are no issue.

@devknoll what is styling the checkboxes to appear after H1 tags in the OA? Can we just change that to be H2?


at this time we are not ready to meet your standards ask 1 day 24 hours to get ready you for your attention


The error comes from the function toepoch() which does assumes time must always be after epoch 1970..

Namely the following lines:
diff_yr = year-1970;
for i = 1:diff_yr

It does not handle times when diff_yr becomes negative. I did one quick change to the toepoch() function but it got the date wrong by one day somehow, so I didn't commit it here yet...


There are lots of places where 'binded' has been used for the past participle of bind, instead of 'bound'.


Ok. Is more information available on in what way it is unreliable? We have seen a few cases were packets get lost or are never transmitted. However our packet load is not particularly heavy, there is 5 seconds or more between packets.


test commit


nuvvu approve cheste nakenti be !!


Why not allow them?


Damn it, good catch. It probably crossed my eyes. Yeah, it makes sense what ``maxSelectableItems`` is doing there. The only problem I faced was trying to understand the approach - nothing too hard.

Well, in this case, I'll change it back to null. This needs some refactoring indeed. After I'm done messing with some stuff on rhea, I'll jump in on jss-form and here, and do a general cleanup.


In the database level set transaction isolation level to READ COMMITTED.

SHOW VARIABLES LIKE 'tx_isolation';
If this is not READ COMMITTED, then set it so because the update/delete queries work fine when done so and doesn't show "Lock variable error" in the Apache Tomcat console.


These two fixed the above:
https://github.com/vszakats/harbour-core/commit/184fa9650efc0a54f0544f24aff638d7f1f618eb
https://github.com/vszakats/harbour-core/commit/6112c589c81763bd471eab1155ce19cc2505e953


May want a check on vargs for NULL. I know you said that functions that are named with what they access (so it should be obvious you can't pass NULL) don't have the check so you may class this as one of those. It just makes me nervous when we are passing void * pointers and casting them and accessing them without checking them for NULL.


Test run: https://ci.appveyor.com/project/vsz/harbour-core-evxk0


Feel free to change it then. Let is supposed to work in WK too.


Пишем в этом проекте интерфейс пользователя на литовском


`src/main/java/org/joda/time/tz/src/readme.txt` still says it uses 2015**e** data!


Хорошая практика - ставить перенос строки в конце каждого файла


The original folder


No... please provide the JSON string and code you're using.


Thanks! the fix is in master and will be included in the next (after 1.3.2) release.


i can write stuff here


We also have a lot of code shared between `copy_folder`, `move_folder`, `copy_file` and `move_file`. That will bite us later. The more reuse the better. We should look into refactoring this.


good opening


hmmm… dodgy


I barely remember what I did yesterday, let alone what I did 5 years ago :)
Yes it would probably be better to convert single newlines to <br>.  Send a pull request!


I will remove the commits

On October 9, 2015 12:06:05 PM CEST, Johannes Pfeifer <notifications@github.com> wrote:
>@micheljuillard I am not really happy with this commit. The naming with
>the attached ```_1.m``` is not really informative. Even worse, it
>breaks all codes and pull requests affecting the likelihood functions.
>For example, the prefiltering bugfixes no apply to a different
>function. I would have preferred something like
>```dsge_likelihood_wrapper``` for the outer function and leaving the
>names of the functions unaltered
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/DynareTeam/dynare/commit/1ad8df4635d99ba775e22f5693bf18c2cbb6c0aa#commitcomment-13683189

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


404 mean "Not Found". I think a 409 ("Conflict") would be more appropriate here.


I think you should better call combineInfillLayers every [amount] layers, instead of calling it on every layer and opting out on the wrong ones.

You should introduce a for-loop somewhere for which the update step is "+= n_skip_layers" instead of "++".

Since combineInfillLayers is principally a multi-layer process, I think you should move the for-loop at FffPolygonGenerator::187 inside the combineInfillLayers function.


dewdwedew


Ok, now the difference between `move` and `rename` is clear, forget my comment on the earlier commit about that ;)


А для чего они нужны?


Ouch! Field default relying on request args?? Better to compute this onaccept from the alert_id in the record!


Will you try to cover these corner cases?


just in case you had not noticed yet...

03:39:54 T:9896  NOTICE: skin helper service version 0.0.29 started
03:39:54 T:7664  NOTICE: Skin Helper Service --> ERROR in HomeMonitor doBackgroundWork ! --> global name 'pluginContent'
 is not defined
03:39:54 T:7664   ERROR: Traceback (most recent call last):
03:39:54 T:7664   ERROR:   File "C:\Users\Mario\AppData\Roaming\Kodi\addons\script.skin.helper.service\resources\lib\Lis
tItemMonitor.py", line 212, in doBackgroundWork
03:39:54 T:7664   ERROR:     pluginContent.buildWidgetsListing()
03:39:54 T:7664   ERROR: NameError: global name 'pluginContent' is not defined


Maybe change this to
```js
let errorMessage = `The test suite has timed out because it has been running` +
  ` for more than ${timeoutInMinutes} minutes.`
```

or something equivalent?


And if it needs to be pre-populated in the CRUD form, then do that in the prep! Should /never/ assume an interactive request through a particular controller - there are a million reasons why this could go completely wrong.


`var`


TESTING


Ha, thanks for digging up so much! I wasn't aware that go-bindata would embed full paths in the binary test file, that's a nice find. I'll rebuild the test without `debug` mode so it should embed the binary file in the go source instead of making an absolute-path reference.

I'll just go investigate the more immediate decryption bug right away, BRB. :)


Cosmetics but: `done` indented on the next line, so it doesn't get lost when reading the code :)


cdscdsc


Nice implementation of the Singleton design pattern for both factories, but Clinton you probably need to upgrade your diagrams again now, and do we need change our requirements to put these two factories also as Singleton design patterns?
< review, organisation


Thank you for being meticulous and please discard these changes.  I was trying to figure out how it worked - how pop up's linked back to the main page and to a lesser extent how fancybox works.  Remember I do not know what I am doing - for me this is all trial and error and this time I was not successful.

If you open a patient and then an Eye Exam, the top right "+" icon will blow out the page to fullscreen and render a bootstrap fix-header menu.  I wanted to open OpenEMR things like calendar under this structure - things that the clinician might want to while seeing patients.  That was the direction I was investigating.


@seanwarren I had to make a change in this area in master recently so merging this commit could be tricky  Do you have a test file I can check with?


`var` \o/ :dancer: :fireworks:


This checks the same role over and over. But since it's not callable anyway, you can just call it once and pass in the result.


Testing now after fetching all comments


...and if it is about user roles, you could also use "restrict" instead of "check".


dwdwe


@anaconda


Zouden deze files niet uit git moeten gehaald worden? wachtwoorden/logins etc


idem


@nemobis: I've added you as collaborator, feel free to implement any changes.


K, I think I've fixed the test case issue here: https://github.com/cathalgarvey/go-minilock/commit/b75d6bda6d2c98f0cfd292cdf336ab628c499c5b

Thanks for that! Onto the decryption bug..


@TomaszKolek Hi, do you remember your reason for joining tags with comma? ``', '.join(tag_list[1:])``


მოყვანილი ალგორითმი, კერძოდ პროცედურა dfs ასრულებს სრულ გადარჩევას გრაფის შემოვლის გზით და შესაბამისად მუშაობს მხოლოდ პატარა ზომის დაფებზე...


Controlevraag: Wat ging er fout in user_permissions.count()?


git pull origin master


My instinct would have been to use something related to the instance name as the username, but I can't really come up with a good reason why, so I guess your approach is just as valid. :)


Is this always deterministic?


If the transfer rate between the SX1301 board and the host is too slow, the SX1301 FIFO can get full, and from there packet loss happens. If a gateway receives packets from many nodes, the traffic may be too heavy.

Also we are planning to perform some scheduling of downlink transfers, handled by the host, and this need a low latency connection between the host and the SX1301 board which cannot be expected through a USB connexion.


This is a standard rails pattern (responding with a new, but not persisted instance of a model on the "new" action.
We don't really create (and save!) it, it's just an empty thing to fill out.
Creation and persistence happens only if something is actually submitted.


@guykogus ， resolve it just now, https://github.com/johnezang/JSONKit/issues/185


Such a simple but excellent change.


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


Would the type be better as a hash define rather than a magic number or is it too difficult to describe what the value represents?


Der Contentbrowser unterstütz ja die Angabe mehrerer Filter. So das an einer Stelle nur das Auswählbar ist was auch Sinn macht. Das könnte man hier gleich angeben.


Quote?


@datakurre, Sorry but I don't remember. Is some problem with that? Maybe if you will describe your situation I will remind something.


@tordans Merci!


Hm... maybe skip all kinds of hidden files (`.startswith(".")`)?


Whoa nice structure man!


:heart:


:+1:  :+1: :+1:


tady ma byt misto $now toto $this->time->now();


Ich glaube, nested omp macht keinen Sinn bzw wird ignoriert.


dfd


``` javascript
return data.type && (data.type != "folder" || data.children.length > 0)
```

(and if not that, then curly braces please ;))


@erylee Are you sure we want this? The note at erlang docs on process priority flags](http://www.erlang.org/doc/man/erlang.html#process_flag_priority) is scary, and discourages use of process priority high.

Was a benchmark performed with this and without?


This works on GCC 4.6:
```c++
boost::join(endpoints | boost::adaptors::transformed(boost::lexical_cast<std::string, tcp::endpoint>), ",");
```


dal bych stejne parametry jako ma microtime:

mixed microtime ([ bool $get_as_float = false ] )


You should always `sanitize`. Never trust your input.


Identified the problem: I'd accidentally left the -o and -p flags as flags of the sub-command "encrypt" instead of moving them up to the base parser. They appear to work well now.

In the process, I've discovered another little bug: the default mode of encrypting-to-self isn't working as expected/intended, either. :) Thanks for finding these dodgy cases!


SD cards actually seem to support folders, at least on some firmwares. So I think just `False` is a bit too much.


ne, je to vyfaktorováno o 3 řádky výše :-)


@langfr: Please try again. I should fix the related issues. Our CI builds are successful.


That the one wonderful commi! ;)


Hm... any objections to renaming that to `_isFolderBusy` and inverting the return values? Functions that contain a "not" have a tendency to cause issues when you are tired and overlook the "not"...


```R
var2 <- var
```
?


ns + combo_input_label + ".foo", alle andre steder mange tak. =P
ns + combo_list_label


to jo, ale zmenil jsi chovani a predtim to bylo spravnejsi :)


Taky jsem nad tím přemýšlel, ale nakonec jsem došel k tomu, že je to zbytečné. Stejně se ten parametr nikde používat nebude. A když jo, tak se udělá rychlý refactoring. Ale jinak jsem si všiml, že by se tomu microtime mělo předávat `true`, takže díky, nevšiml jsem si, že jsem to tam zapomněl :-)


But should I leave the `setAuthToken` as well? Because that will be called on the server on initial render + on client in main.js


Okay relax. We won't get hacked. There's nothing sensitive exposed. I have fixed it anyway.
I didn't realize `git commit --amend` would change the commit id. Is this one a ghost commit now?


这不是紫色么


try it


don't know if R uses pointers so i made a deep copy


Yes, it does need to be prepopulated so user can change...prep would do


Nice :D


How could I have missed these ;)


i think you don't want to call method now() on null.. Replace that to only Time type.


No, not `request.json`. We either have a form data request (which that API endpoint is defined as) or JSON, but mixing both screams trouble. And we need to have a form data request here for file uploads to work, so `request.values` it is.

Btw, in case you now want to raise a hand because you saw me adding a couple of these to some API endpoints recently:

``` python
data = request.values
if hasattr(request, "json") and request.json:
    data = request.json
```

those were only to keep backwards compatibility after I ironed out an actual bug in those endpoints which were supposed to speak JSON in the first place but spoke form data instead (which I only noticed once the client lib was in place). But supporting both should not be the norm and I'll remove those migration stuff in the future and enforce JSON content types on those endpoints, after some time for people to migrate their clients if necessary.


Maybe we should attach a `.date2num(self)` (or `.to_mpl_num(self)`, ..) method to `UTCDateTime` to make this easier in our codebase (and also having to import `date2num` in only one central place..).


DIOS, NO ME ENTERO DE NADA!


thanks, I missed that one :) autogenerated ...


Podle me to tak udelat bylo stejne rychle jako to premysleni ne? :D


Look at line 70:

 $this->time = $time ?: new Time();


You think RegisterMachine will sidetrack the patch?


hi


@brizer esl.js是一个模块加载器，同require.js等效


isDraft is a boolean and because with Doctrine you work with DQL you should use false instead of à imo


jasně, ale já přemýšlel nad tím, jestli chci ten kód zbytečně komplikvat :-) všude bych pak musel předávat argument `true`, ikdyž tam není případ, kdy by se nepředával. Tak proč to nezjednodušit a ten parametr nezahodit? :) Nebylo to zjednodušení práce, ale zlepšení čitelnosti kódu. Navíc chápu, že implementace té metody je fakt simple (prostě volání `microtime(true)`), ale řekněme, že "to je jedno" - účel té metody je vracet float s aktuálním microtimem. To, že je uvnitř nějaká metoda s nějakým parametrem, už tomu ostatnímu kódu může být jedno.


@TomaszKolek Only received a user report about it. It changed double space separated tags (valid robot syntax) into comma separated tags (invalid robot syntax) without clear documentation benefit.

But no worries. I released 0.8.0 with double spaces instead of comma. If you are still using it, you can keep with the latest 0.7.x version.


Your build is failing because there is no s_sort_post_dir variable defined in that file.  Not sure what this line does `'S_POST_SORT_DIR'		=> $s_sort_post_dir)` on line 388 if anything though