I hope this is only a temporal workaroud, without it  for `ant reflection` we get
```
reflection:
     [echo] Cleaning /home/michaeln/mygithub/kotlin/dist/classes/reflection
    [mkdir] Created dir: /home/michaeln/mygithub/kotlin/dist/classes/reflection
     [java] core/reflection.jvm/src/kotlin/reflect/jvm/internal/KProperty0Impl.kt:25:12: error: cannot infer visibility for 'val javaField: java.lang.reflect.Field?'. Please specify it explicitly
     [java] open class KProperty0Impl<out R> : DescriptorBasedProperty<R>, KProperty0<R>, KPropertyImpl<R> {
     [java]            ^
     [java] core/reflection.jvm/src/kotlin/reflect/jvm/internal/KProperty0Impl.kt:25:12: error: cannot infer visibility for 'val container: kotlin.reflect.jvm.internal.KDeclarationContainerImpl'. Please specify it explicitly
     [java] open class KProperty0Impl<out R> : DescriptorBasedProperty<R>, KProperty0<R>, KPropertyImpl<R> {
     [java]            ^
```


I've noticed some lint issues in test files so maybe they aren't getting run through eslint. Hmm.


@chriswmackey thanks for adding this warning. Just a minor comment. Instead of pass you can just write:

```python
if not HBZone.isClosed:
   # give warning
```


test


Anything stopping this from going to `master`?


Is the player volume ignoring handled internally in Bullet?


My bet it's breaking something.


oh wow. That's pretty interesting that the new `Provider` does what we needed before for testing. That's a sign of a good interface.


If I take it out altogether won't it continuously try to look up a mobskill to use every attack round? I figured that was why they zero'd the TP in the 1st place (then it looks it once per every time mobs reaches 100+ TP).


these statements will throw exceptions on pre-existing installs due to the tables already existing. You will need to use a SQL `ALTER` statement to add the new columns onto existing tables.

see: `Contract.Church.SQL_v33_ALTER_PARENT` for an example of how to write an `ALTER` statement to add new columns


It is major release and we have many other API changes :D. They are old finished. So I need only write ChangeLog. I think we will have release tomorrow.


I'll change ... sorry


Odd, not sure how this got deleted. However, the issue is corrected now.


you forgot to include a `break;`


The link is clickable even if the user has not select the checkbox above.


Isn't `t` uninitialised here? Shouldn't you fill it on `sol_rng_engine_generate_bytes` before?


It is very temporal workaround in order to compile sources and run tests. Of course this code should not go into final version.


91958608DOASEffectOnZoneSizing (Myoldmopar) - i386-Windows-7-VisualStudio-12: Tests Failed

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


This bit was changed because before we were passing user as a prop *and* it was being used from state. It's now all coming from state.


besser? @sjmielke


firsname lastname


Not very elegant, but after some reading, I still could not find any better offering


It's all ok! Yes I see, as we had the same test name it is confusing. Thanks for the clarification, I re-activated the test.


Yeah I've seen the same. They aren't. I'll file a bug to check that.


Will update this to move this `.demo` dir to the output dir after I get back to actually coding that up


Fixed in b5124e6

...along with the accidental addition of a string from something else I was working on. I start trying to use the command line more, and get to fail at Git all over again!


Ok, I'll do it.


where does this magic `module.hot` come from? Could you add a comment about it maybe?


`Azure::Armrest::Exception` was my first version :(


I thought about it too.


I'll do it.


Ok in 877b088874c083cfb56a7f64bc47b49aa9a41ad3 I have rolled back the most breaking part of this.


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - i386-Windows-7-VisualStudio-12: OK (1815 of 1816 tests passed)

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


Is this change intentional? Where is `HAVE_GETRANDOM` set?


Here a || instead of &&?


@rkirov what is the motivation for this change?


Fixes #3


meh. Sort of annoying we have to define all exported reducers again but I guess that's how it's done.


Well, don't expect it to get included.  It is precisely because I don;t want this to become a "legacy" issue that I was hesitant to include it at all.

Leave it as an undocumented setting and expect it to change drastically in Marlin 1.2.

(Alternately, I can remove it now)


We wanted to include statically linked CURL. If you have a suggestion how to do it so it will work for Mingw64 too and just running ./mingw64.sh then, let us know.


> and this sort of thing is never notated in ordinary category theory

Well that's what's cool about doing Category Theory in HoTT: it makes things a lot clearer to readers like me. I have spent a lot of time misunderstanding things due to type mistakes.

Is the answer to use `g` instead of `f` to avoid confusion too complicated?


can't this just be `export default const rootReducer = ...` ?


Well, it's not actually output, just a cache, but it should be added to `.gitignore`.


oh I have a vague memory that it doesn't work


test


test 123


The newest release is now online.  It should pass through OPAM within the next half hour or so.


Actually, you did nothing. Just changed container. How this could break UI?! Unbelivable...


the training id will have to be set in a different way if it's not present in the JSON. Don't take any action on this comment at the moment, I'm going to look into if it was an API change or something else before having a recommended solution.


as vrea sa tinem doar un singur argument defualt. Astea sunt aici ca sa poata fi definite de catre utilizatori in index.html de exemplu la noi. Deci nu cred ca ne trebuie widget_domain doar domain. Si domain ar trebuii sa fie default `//epiclogger.com`


Are you sure that we don't need this?


这个是避免流控窗过快增长的


empd (jdwoods21) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1792 of 1797 tests passed)

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


Good point, the sorting is quite an eyesore! Unfortunately the comparators are very lengthy. I'll extract it into a separate utility class.
Thanks!


如果关闭了流控可以不必理会


The order on these should probably be swapped then:
https://github.com/ripple/rippled/blob/develop/src/ripple/rpc/impl/LookupLedger.cpp#L142-L145

It seems there may be a chance of an RPC calling lookupLedger on *this* open ledger and receive an erroneous validated field. https://github.com/ripple/rippled/blob/develop/src/ripple/rpc/impl/LookupLedger.cpp#L223


@BuzzBumbleBee I wish to have a perfect delta. But with your code, at least it will print more accurately. Thank you.

Do you have a "thing" like this one : http://www.thingiverse.com/thing:745523

@boelle @Wackerbarth @thinkyhead Will it be a good idea to create a group on Thingiverse and add some calibration/testing objects?


I like the way this reads but what if we do it like `rootReducer.app(...)` ? My reasoning is that it would test for remembering to export `app` in the root reducer whereas what you have here does not test for that.


@Wackerbarth et al ...  is there any documented goals / changes / ideas of how 1.2 be different to current dev, Id like to help with code changes for this. (for example how rod length trim will become mor generic between printer types)


Great, thank you!


this method should return true/false


the create training event should include the MCC the training is being created for.


I don't think that's anything to do with HoTT; it just means that it takes some effort to learn the standard notations and abuses thereof.  The HoTT book is not intended to serve as an introduction to category theory for people who aren't familiar with it; for that you need a category theory textbook.  The point of chapter 9 is rather to show how category theory is interesting and different in HoTT.


same comment here about doing `rootReducer.transaction()` to cover the integration point


It's a little hacky, but I _think_ this should work for us.

Note that this is not a criticism.  You have to hack around with prev and next  WPs to try to guess if you just aborted a landing.

Something that the autopilot has needed to address this and other hacks, IMHO, is a full fledged LAND mode.  That will probably never happen since it's a BIG change and so far the planes are still landing and doing go arounds OK.


@clefranc that thing would work fine, i literately used a 50mm diameter circle and marked on X Y Z


how about NotFoundElementException?


Awesome! Thanks for the quick fix!!!


empd (jdwoods21) - 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)


No need to use pointers here. Using references would yield the same behavior but will eliminate this "->" and also protect you from taking null pointers.

Several situations when you may prefer pointers instead of references:
1. Objects that are passed can indeed be null.
2. You need to save this pointer inside some object and this is happening not in the object constructor. In this case reference can't be used because it must be bind at the point of initialization.
3. You need to free memory that is pointer by the pointer.
All this cases assume that we need write access to the passed object or we don't want to copy it because it's big.


Commented out code.


I am not sure is not a good default behavior. (1) The build server should run with the default behavior; (2) people should not be building versions that are different from what will get deployed, unless they opt-in.

Willing to be persuaded otherwise.


I am reading "Category Theory" by Awodey and the HoTT book chapter on Category theory to help have two points of views on the same thing. If the HoTT book does not use its capacity to the full then I don't gain anything interesting.

In any case this proof is very concise. Removing information won't make it any clearer: it will just make it more obtuse. I don't see the point if an improvement can be had just by changing `f` to `g` or some such small change.

https://github.com/HoTT/book/issues/862


can we add aliases --update (or --no-update, if we keep the current default) and --verbose?

Also --help?


budur...


Great observation


Are we not going to implement this parameter?


Line 6934 should be
double mb = total / ( 1024.0 * 1024.0 );


We need to make sure removing this branch won't break the [PAR packing feature](http://www.rexify.org/howtos/releases/0.55.0.html). I guess it's OK, but it's best to do tests carefully.


haha


Since we are int32_t in the 32-bit case, why not use int16_t here since we are concerned about the size of the type?


Also removes useless #includes from main.cpp and replaces posix chdir with a multiplatform boost function.


This function is very difficult to read. I suppose that's because asserts are not separated from actual code.
By the way, most of them will be checked by standard library for you. For example
```cpp
assert(!towerFrom->empty());
```
will be checked during
```cpp
towerFrom->pop();
```
And in case of violation the exception will be thrown.


**UserProjectView/core/PluginManager.php**
'fetch_all' consumes a lot of memory
maybe that 'fetch_array' a better choice

**UserProjectView/pages/UserProject.php**
renamed: $i -> $userIndex
fixed: row highlighting


great


So the idea of converting pre-existing hairpins fell through? Oh, well. Thank you very much for all the time you put into this.


could you do the same as
e690caf719aba6ac535b11b26ff20202115b8076


Is the php minimum version still correct ?


Yes, but the skill list is empty so it skips everything that costed time
(shuffling) and just returns immediately


That is the exact reason we used requirements.txt (before we uploaded bottlechest to pypi).

Not everything from requirements.txt belongs to setup.py though, sphinx and other dependences used for building documentation are of no use when orange is (will be) installed as a library from pypi. So they should remain only in requirements.txt (or a separate requirements-dev.txt, if that is a thing).

Concerning the dark magic, I do not se much use in it. It will allow us to skip the python setup.py develop step, but at the same time it will break builds for tomaž (when he builds, he needs to pass --compiler flag, as he is not too fond of visual studio).


Oh, good catch.  However, only the English version of the page uses the narrow layout so I'm going to change this patch to,

```diff
commit e0e47a866e4f98f42af91bd1aade1bb3e17b562d
Author: Saivann <saivann@gmail.com>
Date:   Wed Sep 2 11:13:23 2015 -0400

    Prevent contributors' list to overflow outside of the page

diff --git a/_less/screen.less b/_less/screen.less
index c48282d..9ac5c49 100644
--- a/_less/screen.less
+++ b/_less/screen.less
@@ -1358,6 +1358,11 @@ button.active {
 	text-align:left;
 	font-size:80%;
 }
+
+.one-column .contributors {
+        width: 660px;
+}
+
 .contributors img{
 	width:16px;
 	height:16px;
```


В тебе тільки int і стрінг як значення можливі, там можна так само як у векторі зробити - використати тип value (тоді будуть можливі nested хешмапи), але для початку і так гуд.

Ще в тебе якщо вказати ключі однакові в хешмапі, то вони будуть перезаписуватися (там де в циклі `$1[key] = $3[key]`). Я думаю краще викидати помилку.

В цілому ок, якщо працює - то і гуд :-)  З точки зору jison-у ок, я вже сам забув як він працює. З точки зору того, які можливості хочеш дати хешмапам - то вже складне питання, я наслідував clojure щоб довго не думати, твій варіант для хешмапів теж ок, а там далі буде видно :-)


@gubatron @marcelinkaaa We can have a custom app icon (or any resources) per flavor.


```
`mysql:host=localhost;dbname=beans,charaset=utf8`,'root',``,
```

初見で気になった点：

- `mysql:host=localhost;dbname=beans,charaset=utf8`をシングルクォーテーション`'`で囲みましょう。
- dbname=beansとcharsetの間はセミコロン「;」にするといかがでしょうか。

結果として下記のような感じです。試してみてください。

```:php
$pdo = new PDO('mysql:host=localhost;dbname=beans;charaset=utf8','root','',
```


Does this manage to remove all network access? Cf. #5.


Wtf??


101884090-Venting-Availability-Schedule-does-not-work-properly-in-the-AFN-model (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1277 of 1277 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)


PS. Am I the only one that does not see this comments on the pull request's conversation page?


You're doing `out error`. The initialization is useless.


yay
but y u do this as my build is almost over ;-;


Is the [`include`](http://www.rexify.org/api/Rex/Commands.pm.html#include-Module::Name) functionality preserved if we remove `$dont_register_tasks` here?


why rename it?


My patch does not introduce any new field (everything was in existence).
The only change is, that dimension was before stored for all linked images.

The patch now stores the dimension if the URL itself is a image, that are 3 numbers for such documents, while for URLs with images in it before the patch possibly hundrets of numbers were stored (3 for every picture in the URL), this is unchanged.
With this the additional load to keep the same info for a original image URL as for linked images is marginal.
YaCy has tonns of meta data that can be swichted off and this is true for the image-size too.
Actually after this patch .... you have a actual reason to store the size...... because it improves the filtering of small/one pixel images in image search.

cu


cassert


merge ws-update to master


Wow he's a bitch @FDKPIBC


isExist -> userExists


Yes, the column filter is a select drop down with yes/no options.  Matt also mentions possibly adding a nil or blank option.


This is interesting ...
https://www.npmjs.com/package/webpack-hot-middleware


huh?


Are constants not defined for these case values in QNX headers?


We don't embed `APIARY_API_KEY` into tests. This is security issue for me. We have `APIARY_API_KEY` set only in CI. Solution is add encrypted APIARY_API_KEY to travisci but for localtesting you have to use your own key.


I see...
This goes along with the strange smell of `sweep_pool_region`, where the outer loop is *not* over per-thread heaps. This is only per-page, so it's not horrible, but still leaves a bad taste.


Hmmm...


Why?
I did contacts


重新写这个方法


This is part of v4—go to the v4 pull request or blog post to learn more. Badges are now extensions of pills.


Sorry it was late.


`into the browsers’ built-in CSS capability so can hook into it,,`
needs to be fixed.


spring-aspects? For what?


97215920---Fill-Gaps-in-HVAC-Components---Initial-Code (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1815 of 1816 tests passed)

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


Oh yeah, my Gtk days just have me always automatically initialising errors to null...


data services working better.


you should have /usr/local/lib/pkgconfig/libcurl.pc, zlib.pc also (like
linux) its config files to tell how to configure a lib

here is the content of mine :

> # This should most probably benefit from getting a "Requires:" field added
>
> # dynamically by configure.
>
> #
>
> prefix=/usr/local
>
> exec_prefix=${prefix}
>
> libdir=${exec_prefix}/lib
>
> includedir=${prefix}/include
>
> supported_protocols="FILE HTTP HTTPS"
>
> supported_features="SSL libz NTLM TLS-SRP"
>
>
>> Name: libcurl
>
> URL: http://curl.haxx.se/
>
> Description: Library to transfer files with ftp, http, etc.
>
> Version: 7.38.0
>
> Libs: -L${libdir} -lcurl
>
> Libs.private: -lssl -lcrypto -lgdi32 -lz -lws2_32
>
> Cflags: -I${includedir} -DCURL_STATICLIB
>
>
2015-09-03 17:19 GMT+02:00 nicehashdev <notifications@github.com>:

> We wanted to include statically linked CURL. If you have a suggestion how
> to do it so it will work for Mingw64 too and just running ./mingw64.sh
> then, let us know.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nicehash/cpuminer-multi/commit/57bd235705b5e4dce1af20b25ee60703c83cb899#commitcomment-13050178>
> .
>


-__- Make a new Branch! My builds without sm will fail idiot!


you're right. the logic is the opposite. the reason to use "empty" is to avoid a "var does not exist" warning in PHP


This is going wrong


Εδώ γιατί τα έσβησες!?


That's a great idea. I never thought of that


Stable Version with formatting and loading bar


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

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


Thank you for noticing this @ibauersachs and for your input @mondain. I took the less controversial (and less preferable) approach of dropping the Java 1.7+ stuff.


Hi

Now i can not anymore do a construct like this:
.
 tagBuilder.InnerHtml += "";

I had an extension method on beta4 to append inner content to a tag builder, without destroying exsistig code.

    public static TagBuilder Inner(this TagBuilder tagBuilder, string tagName, string[] cssClass, Action<TagBuilder> child = null)
        {
            var builder = new TagBuilder(tagName);

            foreach (var css in cssClass)
                builder.AddCssClass(css);

            if (child != null)
                child(builder);

            tagBuilder.InnerHtml += builder.ToString();

            return tagBuilder;
        }

I stick to find a proper solution on beta8 with the new tagbuilder definition


Εδώ έσβησες την κλάση και έβαλες μια κλάση Authenticator!


Commented out code.


HTTPS and SSL are not really required, only the crypto part of openssl is
used for the SHA

2015-09-03 17:43 GMT+02:00 Tanguy Pruvot <tanguy.pruvot@gmail.com>:

> you should have /usr/local/lib/pkgconfig/libcurl.pc, zlib.pc also (like
> linux) its config files to tell how to configure a lib
>
> here is the content of mine :
>
>> # This should most probably benefit from getting a "Requires:" field added
>>
>> # dynamically by configure.
>>
>> #
>>
>> prefix=/usr/local
>>
>> exec_prefix=${prefix}
>>
>> libdir=${exec_prefix}/lib
>>
>> includedir=${prefix}/include
>>
>> supported_protocols="FILE HTTP HTTPS"
>>
>> supported_features="SSL libz NTLM TLS-SRP"
>>
>>
>>> Name: libcurl
>>
>> URL: http://curl.haxx.se/
>>
>> Description: Library to transfer files with ftp, http, etc.
>>
>> Version: 7.38.0
>>
>> Libs: -L${libdir} -lcurl
>>
>> Libs.private: -lssl -lcrypto -lgdi32 -lz -lws2_32
>>
>> Cflags: -I${includedir} -DCURL_STATICLIB
>>
>>
> 2015-09-03 17:19 GMT+02:00 nicehashdev <notifications@github.com>:
>
>> We wanted to include statically linked CURL. If you have a suggestion how
>> to do it so it will work for Mingw64 too and just running ./mingw64.sh
>> then, let us know.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/nicehash/cpuminer-multi/commit/57bd235705b5e4dce1af20b25ee60703c83cb899#commitcomment-13050178>
>> .
>>
>
>


TODO：
修改chareader，重新确定流程
indegenerator 在完成之后删除，现在是无用的文件
修改test.csv的模版，让type和header形成容易分解的对应关系
把class建全了


@roxtar thanks, fixed it !


The `Card` component takes a `prop`, `value`.  Props are the means in which data is passed from a parent component to its child.  The data could be a value, object, string, function (pretty much anything you can pass around in JS).  Props are accessible from within the component via `this.props`.

Props are immutable data that help configure a React component.  You should not mutate props (that is what `state` is for! which we will get to later)


Referenced link is wrong.


Here we access the prop, `value` that was passed into the `Card` by its parent, `Game`


can one use `f^^-1` for the arrow in the opposite catégory?


Remove backtick after closing hyphen.


Βλέπεις τις ερωτήσεις μου?!


TODO：考虑要不要改成class？


yesh


In this case, in my opinion, it's better to do the next:
```
if 'joined' in raw_member:
    member.joined = epoch_to_datetime(raw_member['joined'])
else:
    member.joined = None
```


@srisurya95
packages/apps/Contacts/res/values-darktheme/styles.xml:62: error: Error: No resource found that matches the given name (at 'android:colorBackground' with value '@*android:drawable/material_bg_blue_grey').


@b-dur the only issue with using this feature is that it will require anyone who uses the module to have `babel` and the polyfill declared.


Sorry for nitpickery but use .empty () or somebody else will require it for the real review:)


independent


good, if we go this route we will need to.


采取的是顺序读下来的方法，重新想下各个情况，可能有点不对


inn -> in

GDocs -> Google Docs


This becomes tricky if the default value of the proto may change: https://android.googlesource.com/platform/external/protobuf/+/master/java/README.txt - "Nano version".

@jias0001 can we assume that this default value will never be anything but the empty string? If so, a comment should be added at phonemetadata.proto on the national_prefix field never to change the default value.

If it may change, @AnastasiaSulyagina can opt in to generate a hazzer method has<fieldname>() by generating the java file using "optional_field_style=accessors" as an option to javanano_out (https://android.googlesource.com/platform/external/protobuf/+/master/java/README.txt - "Nano Generator options"). In this case, you should use the generated getters and setters too.


also how it
- This reduces space occupied by a normal application by 11.8%.

you mean in memory ?


@muhammad-ammar we should make our forks under edx rather than personal accounts. DevOps can help make that happen.


Also, whitespace is now stripped from url paths


91958608DOASEffectOnZoneSizing (Myoldmopar) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


surprisingly the two files are created with exactly the same (full resolution) mtime on my machine. I changed the sleep to 0.1 and added a comment explaining it.


Why you touchy touchy?
Tested?
What does it fix?


I thought that was too minor a cleanup to be worth mentioning, but feel free to add!


Cool to make the search filter better :)

Thanks again for the work Bubu!


There's a typo in the docs at [line 44](https://github.com/silverstripe/silverstripe-framework/commit/782c4cbf6f5cde2fa4d45cdbd17552773a67f88f?w=1#diff-40e78f71726c28f3fc75b0733005ca72R44) — it has 'FulltextFiler' for 'FulltextFilter' in the api link.


Hi Gaël,
I'm interesting about this v2 of your binding. I have questions to test it.
In code, i see that the Discovery Service (to discover the devices) is registered only when API Bridge is created. How this bridge is created ? And where did you set the API client ID, secret and token in this V2 (it was in a openhab.cfg in V1. Is there an equivalent in V2) ?
Regards,
Enric


Ok fair enough. I've created a failing test case that shows the issue that caused me to raise this pr in the first place: https://github.com/getsentry/freight/pull/47 Any help would be appreciated!


No need to change this file. Please revert.


`mail.outbox` is cleared before each test, so we should be able to assume its state at the start of the test.  (e.g. 0 messages unless we've done something that we'd expect to send one during setUp).


We discuss about this at meeting... Should be a function field on forum no ?
idem for next... ?


you should not commit binaries into repositories.
It might be best to remove all *.pyc files and add an exception to .gitignore


One of the things I really dont like about Python is that we need to write so many tests.

Julia is better than this. It allows declaring type so we dont need to add checking if 'not isinstance(x, Structure)'.

Hopefully type hint in python will does this

https://www.python.org/dev/peps/pep-0484/


LDLIBS value needs quotes; I'm rather amazed this doesn't fail for lack of libgssapi_krb5 or for too much Make load average!


Just setup a new thing of the Netatmo type. You'll then have the opportunity to configure it (entering your API client ID, tokens...). Then the discovering will go on by itself.


Rod length is particular to delta printers. However, I do expect a move toward a standardized way to express  any characteristic of this type. For example, the offset of an axis alignment switch is very similar.

My proposed format is to express each such distance as the sum of components.
First, we have the "design length" (perhaps 160mm in your example above)
To that, we add an "adjustment" ( 3mm ?) which is a correction that gets compiled into the firmware.
And then we add a configurable "trim" which would be stored in EEPROM and changed during operation.

The idea is that, unless you know better, zero is the appropriate default for any adjustment or trim.

Given a uniform description, we can do away with all of the various M-codes for the setting of each particular parameter and replace them with a general purpose "set parameter" M-code that provides access to any parameter of this type. This allows a table-driven extensibility that automatically provides the support for the setting, retrieval and use of any new parameter.


In case heartbeats will fail, there will be no logs. Is idea to rely on SP rather than logs?


Me parece que você vai ter que fazer uma análise de tempo e complexidade desse algorítmo :stuck_out_tongue_closed_eyes:. Porque como o manoel disse 2h é demência! hahaha


@Aveline67

Yes, we mean memory. The goal of this change is to get rid of the intermediate string allocations that happen when building HTML imperatively like this.

Can you please take your feedback about this here https://github.com/aspnet/Mvc/issues/3087 - or open a new issue if you don't think is applies. It's unlikely we'll keep track of your feedback in a comment on an cold commit :laughing:


Nice solution. Thanks for the patch


작동만 가능하지 기능은 01에 비해 상당히 떨어짐... 문제점 찾기


@davidfestal this is the second place I have had to add this workaround, and there are a bunch of other places where it might be needed. Did something change here?


Oops "no type checking happens at runtime ."


Mejor from **addons.l10n_ec_authorisation import authorisation **


"While these annotations are available at runtime through the usual __annotations__ attribute, no type checking happens at runtime . Instead, the proposal assumes the existence of a separate off-line type checker which users can run over their source code voluntarily. Essentially, such a type checker acts as a very powerful linter. (While it would of course be possible for individual users to employ a similar checker at run time for Design By Contract enforcement or JIT optimization, those tools are not yet as mature.)"


Your rationale makes sense to me. Unless anybody else chimes in to object to me flipping it back to update-by-default, I'll push a fixup that does that.


Yes to all. Fixup to follow.


Other option : you create things and items files, here is an example based on my configuration.
Things file :
```
Bridge netatmo:netatmoapi:home [ clientId="52f3....", clientSecret="tXl...", refreshToken = "51c..." ] {

    Thing NAMain interieur [ deviceId="70:ee:50:00:c1:72" ]
    Thing NAModule1 exterieur [ deviceId="70:ee:50:00:c1:72", moduleId="02:00:00:00:bc:dc" ]
}
```

Items file :
```
Group netatmo_in "Module Intérieur" [ "home-group" ]
Number netatmo_in_aq (netatmo_in) {channel = "netatmo:NAMain:home:interieur:co2" }
Number netatmo_in_temp "Température Intérieure [%.1f °C]" <temperature> (netatmo_in) {channel = "netatmo:NAMain:home:interieur:temperature" }
Number netatmo_in_humidity (netatmo_in) {channel = "netatmo:NAMain:home:interieur:humidity" }
Number netatmo_in_noise (netatmo_in) {channel = "netatmo:NAMain:home:interieur:noise" }
Number netatmo_in_pressure (netatmo_in) {channel = "netatmo:NAMain:home:interieur:pressure" }
Number netatmo_in_wifi (netatmo_in) {channel = "netatmo:NAMain:home:interieur:wifistatus" }
Number netatmo_in_humidex (netatmo_in) {channel = "netatmo:NAMain:home:interieur:humidex" }
Number netatmo_in_heatindex (netatmo_in) {channel = "netatmo:NAMain:home:interieur:heatindex" }
Number netatmo_in_dewpoint (netatmo_in) {channel = "netatmo:NAMain:home:interieur:dewpoint" }
Number netatmo_in_dewpointdep (netatmo_in) {channel = "netatmo:NAMain:home:interieur:dewpointdepression" }
DateTime netatmo_in_ts (netatmo_in) {channel = "netatmo:NAMain:home:interieur:timestamp" }
Location netatmo_in_loc (netatmo_in) {channel = "netatmo:NAMain:home:interieur:location" }

Group netatmo_out "Module extérieur" [ "home-group" ]
Number netatmo_out_temp (netatmo_out) {channel = "netatmo:NAModule1:home:exterieur:temperature" }
Number netatmo_out_hygro (netatmo_out) {channel = "netatmo:NAModule1:home:exterieur:humidity" }
Number netatmo_out_rf (netatmo_out) {channel="netatmo:NAModule1:home:exterieur:rfstatus" }
Number netatmo_out_batt (netatmo_out) {channel = "netatmo:NAModule1:home:exterieur:batteryvp" }
Number netatmo_out_humidex (netatmo_out) {channel = "netatmo:NAModule1:home:exterieur:humidex" }
Number netatmo_out_heatindex (netatmo_out) {channel = "netatmo:NAModule1:home:exterieur:heatindex" }
Number netatmo_out_dewpoint (netatmo_out) {channel = "netatmo:NAModule1:home:exterieur:dewpoint" }
Number netatmo_out_dewpointdep (netatmo_out) {channel = "netatmo:NAModule1:home:exterieur:dewpointdepression" }
```


Looks wonderful


useless ?
can be done with t-esc="len(posts)"


@clytwynec Is this change needed for travis?  Unfortunately, I'm unable pass environment variables in when running acceptance tests on my local machine.


Useless ?
Can be done with <t-esc='len(xx)" />


please add ```override``` to this expression and the next to allow the compiler to double check everything is proper.


fixed shortly, thanks


This is a good enhancement but the puppet module from the forge is still with the old
```puppet
file { "${elasticsearch::params::defaults_location}/elasticsearch":
-      ensure => 'absent',
```
And that produces the error below ("Can't open //etc/default/elasticsearch") while installing a plugin:

```puppet
Notice: /Stage[main]/G10b::Elk/Elasticsearch::Plugin[mobz/elasticsearch-head]/Exec[install_plugin_mobz/elasticsearch-head]/returns: /usr/share/elasticsearch/bin/plugin: 55: .: Can't open //etc/default/elasticsearch
Error: /usr/share/elasticsearch/bin/plugin -install mobz/elasticsearch-head returned 2 instead of one of [0]
Error: /Stage[main]/G10b::Elk/Elasticsearch::Plugin[mobz/elasticsearch-head]/Exec[install_plugin_mobz/elasticsearch-head]/returns: change from notrun to 0 failed: /usr/share/elasticsearch/bin/plugin -install mobz/elasticsearch-head returned 2 instead of one of [0]
```

Could you please be so kind to rebuild the puppet module and upload it in the forge?


this is always the return of the if statement - and If we sell in a unit other than the one defined on the product then the sale journal items on the 1) stock out account, 2) cogs account, 3) the product expense account are total mess.
Plz have a look at this thread on the community to get more info about the bug and its solution that I found:
https://www.odoo.com/groups/community-59/community-13743387

@antonylesuisse @fpodoo @mart-e


@FedeMPouzols I'm not quite sure why its not finding the `hrp39191.raw` file


an invoice.line object have no move_id in version 7 and v. 8.
So this if statement never work.


BTW, still the flavors are better even if we decide to keep the same application id.


No, they never contain group


it's also safer, if someone adds fields later on.  fixed.


Hey @campgurus , why is your commit message title "enter the commit message for your changes. Lines starting" ?


dehumidifier-fixup (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2310 of 2332 tests passed)

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


thanks!


fixed, thanks.


I copied this name from RustHighlighter, but I don't really undestand what it means. I suggest rename it in both places to `tokenMap`


This should allow a className through as well:
```javascript
this.contentComponent = React.render(React.DOM.div({children, className: this.props.className}), this.leafletElement._contentNode)
```


This is meant to be final at this point, so we shouldn't set the international prefix in the constructor.

@hagbard @jias0001 can we do:

private static class EmptyMetadataFactory {
  private static PhoneMetadata metadata = null;

  static PhoneMetadata getEmptyMetadata() {
    if (metadata == null) {
      metadata = new PhoneMetadata();
      metadata.internationalPrefix = "NA";
    }
    return metadata;
  }
}

private static final PhoneMetadata EMPTY_METADATA = EmptyMetadataFactory.getEmptyMetadata();


Anyway, the current patch is good enough. Making the text more readable by tying in the previous paragrpahs' f and this one is another issue.


this doesn't seem to be reflected in the gem distribution
/Users/maysam/.rvm/gems/ruby-2.2.2/gems/farsifu-0.5.0/lib/farsifu/constants.rb:73: warning: duplicated key at line 79 ignored: "دسیلیون"


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1291 of 1291 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.0%25-green.svg)


thanks


We use the same anonymous object factory in Rust language. However, there is an unused RustHighlighterFactory class. I guess we should remove it.


Not really sure about correct value for boolean


No need for new empty line, please remove


empd (jdwoods21) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (749 of 749 tests passed)

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


Do we really want to make all of these ado_debug calls every time this is called?  Or was this mostly here for debugging purposes?  It seems like logging in an exception case or at the very beginning of a function was fairly standard in other drivers.


Hi,

Where do you see that? the defaults file is replaced by a augeas source and all my acceptance tests are passing.
Are you sure you are using an up2date version?


This must be:
```
#  define UAVCAN_STM32_IRQ_PRIORITY_MASK  CORTEX_MAX_KERNEL_PRIORITY
```
see [Porting code from ChibiOS 2.x to 3.x](http://www.chibios.org/dokuwiki/doku.php?id=chibios:articles:porting_from_2_to_3):
> Now the NVIC minidriver always take the priority as parameter, the use of the CORTEX_PRIORITY_MASK() macro is no more required.


:+1: Certainly can. Very nice, thank you sir.


what is this?


The CMake parts look fine except for this bit.

I'm not sure it is legal to put a value after the variable name in a `#cmakedefine` because CMake is meant to [replace the line](http://www.cmake.org/cmake/help/v3.0/command/configure_file.html) with a `#define` based on its cache value.

If the cache value is meant to be a user-set option then it should also be exposed to the user and documented via [a call to `OPTION`](http://www.cmake.org/cmake/help/v3.0/command/option.html).  If that option is only relevant on Windows, then it should be guarded by `if(WIN32)`.

It should also be documented in `docs/INSTALL_CMAKE`.  What would be really helpful is if that documentation could explain why you would ever want to disable the option.


I was talking about throwing `snd_pcm_hw_params_get_period_size` away. Since buffer size is set independently from period size, that call have no sense. `period_size` is just number of frames, so it can be calculated from `period_time` [1].

[1] `period_time/1000000.0` is period duration in seconds, so `rate * period_time / 1000000` is number of frames per period.


Yes, You are right, it's small change, but we don't have the time to test the compatibility against hard RoCE. That's why I think that it's should be added to the TODO file for now.

About the incoming packets ICRC check, We don't check ICRC for incoming packets and I'll add this also to the TODO file.


This is a test file....


That's actually wrong... it will put the result in a list, while the if evaluates to what 'prepend..' is without a wrapping list.


Just forgot about it.  Adding it now.  Will only affect scores from an actual released version version.


@takikawa, @btlachance: What's the ETA on that? I'm hoping to get the unit stuff pushed next week.


There's no need to give me a Signed-off-by in this commit's message.  I don't think LK4D4/specs#1 was large enough to be copyrightable (and if it was, I'll assign the copyright to you ;).  I consider it on-par with other in-line suggestions for improving commits, and those come with no strings attached :).

On the other hand, I'm fine with you leaving it in if that's easier.


It's the Typescript that is pulled down when running `npm install` in the DefinitelyTyped directory.

```
=============================================================================
                    DefinitelyTyped Test Runner 0.5.0
=============================================================================
 Typescript version: 1.5.3
 Typings           : 1405
 Tests             : 1224
 TypeScript files  : 2682
 Total Memory      : 15998 mb
 Free Memory       : 3039 mb
 Cores             : 8
 Concurrent        : 6
```

I noticed that your merge request passed on those same files, so not sure what is going on.


You can change this to ```NumberFormat[] formats ...```
Then you don't have to convert the arrays to lists, and in the for loop, you can use
```
for (int i = 0; i < formats.length; i++)
  NumberFormat format = formats[i];
  ...
```
Easy way to make it more efficient :)


Please change this to inherit from ```one::OutputModule<one::outputmodule::RunWatcher>```
Then you just keep the regular ```beginRun``` and ```endRun``` function signatures.


* Updating tag indexes at the end of parsing an entire file in the src.
* Pushing the tag indexes into `.json` files in the `indexes/` directory at the end of parsing all files.


This was changed because the a11y tests need the same setup as the regular bok choy tests (the stuff in runTests.sh).  Why can't you use environment variables?  Can you explain a bit more?


Some of these commits (c3778fb, 18bdea1, 2930d6c, 60e638e, 528e17c) generate errors using the checkstyle plugin or unchecked conversions. I can take care of them if you are ok.


Where did you get this?


Perhaps this should go outside the lock?


+jias0001 the above question on default value forever being "" also applies to national_prefix_for_parsing


I know ;) But for me, the loop looks more straight-forward and I'm sure that it will use the shell and other commands from the BusyBox version I have compiled.


When you come in, I'd like you to explain why you think this escape analysis may be necessary.


The core now is unaware of the fact that the UI will process the network keys (equal to the keys in the NETWORK_OPTIONS dict) to get "displayNames"


Thanks for reporting this Maysam, I'll release a new version of the gem ASAP. Should happen in the next 24 hours.  :)


101884090-Venting-Availability-Schedule-does-not-work-properly-in-the-AFN-model (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1803 of 1804 tests passed)

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


This should be .fit-content


> One of the things I really dont like about Python is that we need to write so many tests.

I think that's more a reflection of the package than the language.  The point of Python's dynamic typing system is that duck typing means you don't *have* to do `isinstance` checks.  I myself don't do too many of them, even in my tests.

I personally don't see how writing ParmEd in Julia (or any static typed language) would eliminate any of the tests I have right now...


Please leave the individual imports


Oh yeah, get this party rolling.


Small typo, but looks good otherwise!


I disagree that removing information can't make something clearer.  There is no ambiguity in removing the subscripts, because, as has become clear in this discussion, it is possible to deduce what they must be from the type of `f`.  Too much notation can obscure the essential points of a calculation.

A superscript -1, if that's what you meant, can _not_ be used for an arrow in the opposite category because it means an inverse.


Perhaps an ado_debug call would be appropriate here to indicate we are just going to write zeroes?  It seems like an exceptional case.


Awesome. :+1:

Thanks for all the fast fixing! I am really looking forward to the new release!


> maybe it's purpouse is to servfail for DNSSEC capable clients).

```
% 1 dig prefetch.validatorsearch.verisignlabs.com @208.67.222.222

; <<>> DiG 9.10.2-P4 <<>> prefetch.validatorsearch.verisignlabs.com @208.67.222.222
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 60030
;; flags: qr rd ra; QUERY: 1, ANSWER: 2, AUTHORITY: 0, ADDITIONAL: 1

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 4096
;; QUESTION SECTION:
;prefetch.validatorsearch.verisignlabs.com. IN A

;; ANSWER SECTION:
prefetch.validatorsearch.verisignlabs.com. 7200 IN CNAME cn1441296819903004.validatorsearch.verisignlabs.com.
cn1441296819903004.validatorsearch.verisignlabs.com. 7200 IN A 127.0.0.1

;; Query time: 582 msec
;; SERVER: 208.67.222.222#53(208.67.222.222)
;; WHEN: to syys 03 19:13:40 EEST 2015
;; MSG SIZE  rcvd: 119
```


In fact I use the puppet module install instruction. And when I check the code in my module path I can see the "old fashion" (ensure => absent).

I tried a puppet module upgrade but the result is "that is already the last version".


This should be `eps(max(abs(limits, [], 1))` with possibly some safeguards for when those limits are `Inf`: `eps(range)` would give the precision of the difference, e.g. for an axis form `2 to 10` that would be the precision of `8` instead of the precision of `10`, i.e. slightly less conservative.


Please remove added indentation


Can I inquire what the note is?


Commented out code.


Would not match our coding style


Please remove increased indentation


FourPipeBeamAirTerminal (EnergyArchmage) - x86_64-MacOS-10.9-clang: OK (1816 of 1816 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)


Good one :+1:


Unfortunately there is not type for bytes, uint8_t seamed a good choice. why would char be any better ?


Sorry, I was having some issues earlier.  Didn’t mean to leave anything.
> On Sep 3, 2015, at 12:15 PM, Sean Campbell <notifications@github.com> wrote:
>
> Can I inquire what the note is?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/natac13/freecodecamp/commit/fa0160b639cca66d187be6a6ef9af464ac287708#commitcomment-13051658>.
>


Ok, thanks !


I can push it today. It likely needs some small feedback but it's otherwise a pretty trivial change.


Right, I think I remember that. Ok then.


Indent, not align


As a beginner who is reading the book, I can tell you that the extra information - had it been correct - would have helped me read the proof. Of course mistaken information, which the current patch fixes, does not help. So its not surprising that I had trouble reading this passage of the book.  I don't think less information would have made it easier.

So please push the current patch. I'll be satisfied with this.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2300 of 2336 tests passed)

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


Yes, I'll remove the goto.


At dependencies.json


It's probably better to compute that target string in the constructor. This way you are not doing this every time getTarget() is called. And if constituents are needed at some later time, then that can be addressed later.


Why the change here?


Thanks for the hint.

@fabiencastan @simogasp Maybe I will also factorize the loading of features in this function by using `readDescFromFiles` to avoid useless code repetition.


Damn, t should go away, thanks.


枚举法


case snd v of ..   would be more symmetric to the "fst" above.
Redundant parens in case.
You can replace this map/filter with a list comprehension:

`[ path | (path, Db.FileDescExisting _) <- M.toList elInputDescs ]`


That's very weird.
As you can see here: https://github.com/elastic/puppet-elasticsearch/blob/master/manifests/config.pp#L121-L133  it doesn't remove the defaults file but modifies it.


@jrwalker3 Oh no that good to hear I just wanted to make sure is was nothing on my end! Cheers


Could it be you have the module installed in 2 different locations?


Glad you agree.

Regarding how to include the RedBlackTree<T> class into your code, you will have to include the namespace DataStructures.Trees after your System.* imports.

On a side note, I was thinking of implementing the SortedSet data structure, and since we have a SortedList (your implementation), and all of these will be based on the trees data structures, I am thinking of creating a separate namespace for them, something like TreeBasedADT (ADT is short for Abstract Data Types). I am calling them ADTs because they are not novel or original data structures, they are either balanced trees of unique elements or duplicate elements. This is just a passing thought, what do you think?


Error fixed, browser caching even after checking "no cache when dev tools open plz" _and_ explicitly clearing the browser cache.  Changing server's port made it stop.


You probably can't use forward slashes for ANY parameter passed to DSC, especially the Package resource. I'm fine leaving it to just package resource for now though. :)


I mean return r, it must have been removed on some rebase.


Not sure what u mean here. You can define a range 1 to 512 and in the code we shift is by 20 to get full value ?


@ryleykimmel @lare96 it's all buffered so in practice it's probably reading the whole file into memory in one call, and then just reading from the buffer each time. The real reason I don't really care about this is that when apollo eventually gets postgres support I want everyone to use the SQL saver instead of the binary.


```
$ host freifunk.net
freifunk.net has address 176.28.11.93
```


Why not partitionM instead? The code below would be prettier that way, wouldn't it?


Release version 1.0.1


@ichrispa one here


Não seria "melhor" se ele fosse p/ a branch 'master', da mesma forma que a url do footer?


Wow, I'm officially an idiot for not including this. /me jumps off a cliff.


Yeah, that's a bad error, but no worse than those we get with generative structs. And I think forall types can also give that kind of bad error, but I forget.
Adding the escape check is bad for two reasons. First, it's overly restrictive (IMO). If some unit exports a lexically-bound signature, that's not a problem. It's a bit pointless, as no-one will be able to access those exports, but it shouldn't be an error (which the escape check makes it into). Second, it adds some pretty unpleasant coupling in the implementation (between `letrec` and signatures).


Correction:  4448 is actually 6437.


Vimism. I will try to fix it.


makefileTargetPath**s**


May be this is already answered but wanted to know that what does size 'in bytes' mean. Whats the size of one entry ?
Should these (FDB, L3 etc.) not be just in terms of number of entries ?


Does this look OK?


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

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


I was wondering if we would prefer to use ``.get()`` here instead of taking the risk to throwing an exception.


List comprehension: `[ "touch " <> path | isDirectory tgt, path <- makefileTargetPaths tgt ]`


typo: "initial"


I didn't know about the defgroup thing D:

On Thu, Sep 3, 2015 at 5:45 AM, Artur Malabarba <notifications@github.com>
wrote:

> Why don't you namespace your functions then? malabarba/le-func
>
> Just because org/insert-checkbox is shorter than
> malabarba/org-insert-checkbox.
>
> Also, with the release of your nameless package will the names package be
> actually of no use? Are there still some benefits one gains from the names
> packages that the nameless pacakge doesn't provide? (practically they are
> the same right?)
>
> For the most important functionality (hiding symbol prefixes) they are
> pretty equivalent.
> Names is a bit more versatile in that it allows absolutely any prefix,
> while Nameless requires that the prefix ends in a - (so your current
> usage wouldn't work).
>
> Names also has some secondary functionality like generating defgroup and
> a couple other tricks.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CestDiego/nand2tetris.el/commit/3be93c78a704dc4c562b3876bdad3dff7b34837f#commitcomment-13044158>
> .
>


I reverted back. Does that help with compiling on linux?


Same as above, ``.get()``?

(Especially since the exception could occur in the self.left (not sure how but ``.left`` and ``.right`` aren't private) in which case we would end up with another discrepancy between left and right).


Cool, note that we need to escape two dollar signs, else it will render inline math. We also need to escape the closing dollars otherwise it will start a new (empty) mathblock.


Why are you using nul_byte, instead of null_byte?


I missed the fact that the class which inherits from this one wants the luminosity block. So the full inheritance should be
```one::OutputModule<one::outputmodule::RunWatcher, one::outputmodule::LuminosityBlockWatcher>```.
Once again this allows you to preserve the ```beginLuminosityBlock``` and ```endLuminosityBlock``` call names.


Perhaps the call to `kill` should be in a `finally` block.


Thanks. I've fixed that!


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Because it is a NUL byte and not a NULL byte. See `ascii(7)`.


Also, look into https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html


Yeah, makes sense. Didn't think about it :dancer:


`ft` is not a monad.


Adding initial tests


ああああああ


Can we have a `fastbuild.sh` or some such that does the reverse? Kinda annoying having to add flags after tab-autocompletion. Or will I now be told to use aliases?


I deleted my VM and automatically rebuilded from scratch, all is scripted with "puppet module install".

Only one version on the system.

It seems that there is a difference between GitHub and the puppet forge. How can I verify this point?


Of course it's not, thanks for reading carefully :). I meant `t`.


yeah that would be better i'm not sure if this class will stay the way it is there is also talk about providing typed keys similar to how the config works now for getting grains out of the result of the `grains.items` function. There will probably be potential there to reuse that for the Grain target.


Why adding tests for PHP5.5 if you require the PHP5.6 version?


I think it would be a better idea if you allow the commit I finished to be PR'ed when I finish the 0.614 dev branch to avoid re-using the code, unless that was what you meant.


うううううう


I wish the Spine license didn't have this clause, they require a purchased license for anything other than "personal or internal usage"

https://esotericsoftware.com/spine-purchase


When I created this code I didn't realized about the headers of the files (usually hidden in the editor), so surely it is needed to rewrite them saying that they are for the Image Map questions and not for the matching or any other.


Right, but this means that *any* id will match, which is most likely not what you want.
Have a look at what `base-special-env.rkt` does to give types to other modules' private identifiers. That trick should work here too.


There is already very similar approach implemented in https://github.com/marmarek/qubes-gui-daemon/blob/master/screen-layout-handler/watch-screen-layout-changes.c
If there is no really good reason in having this in the same process that handles **untrusted input** from a VM, I'd not keep it as a separate process.


Inline comments and logging would be helpful.
In terms of logging, we will want to make it easy for admins to determine what ACLs and Authorizations are being used and what decisions are being made based on those.


stratified_tank_docfix (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1818 of 1818 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)


I am very uncomfortable with this changeset!

Our namespace is our permanent public API for our front end components.  It is essentially permanent.  We should make very sure that if we're changing our namespaces that we're doing so in a way that is backwards compatible, and will be as future proof as we can make it.


Finally embraced with loving arms.


Just released 0.5.1


Apparently, python-script is the right one. My bad. I'll fix this.


@awoods, yes working on that (and some legitimate testing routines) now


In general we prefer to avoid `#if` in our code whenever possible favoring instead a runtime check and avoiding the troublesome code path.  Is there a way we could detect we are on ProjectN here and avoid this code?


Then, I think in order to not break anything in the update process, regardless of the flavor we should keep the same application id, and the provider issue is also gone.


thanks! overall looks ok. maybe wrap the json decode in a `try`? also probably worth documenting this behavior (of course), and maybe comment in `ihe` that this logic is here. otherwise lgtm!


You're a good man, Charlie Brown.


Can you elaborate how it works? How focus stealing is affected by setting `_NET_WM_USER_TIME` .
Also keep in mind we don't want to break confirmation dialogs - it this makes them hidden under an active window, the user would not notice it and would think that something hangs.


Ok, well remove the comment then.


Instead of complicating this code we should adjust the calling code to make `clientDirectory == sdkDirectory` in the case of ProjectN.  This is how we intend to handle the differences between CoreCLR and Desktop


@paulfalgout Shouldn't we be checking for the view comparator regardless of whether BB gives us an index?


Are the failures here mostly the result of not handling variables in shorthands, or are they something else?


wow looky here what is this


stratified_tank_docfix (nmerket) - 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)


The `else` should be on a separate line


it gets checked again inside of `_filteredSortedModels`


Is it possible for us to just use `object` on Windows as well?  If so we could remove this `#if`


Ce test et ceux qui suivent auraient pu être mis sur un autre controller. Vu que c'est implémenté avec le before_filter check_membership dans AppController (et quasi tous les controllers héritent de AppController)


This is where the dynamic aspect of TO_OS_STR was lost in changing it to only be OS_STRNCPY


If `opts` contains `at` but not `index`, then `_filteredSortedModels()` is never invoked, unless I'm missing something.


change


Maybe you could re-use `CONTEXT_BUTTON_REFRESH`? But it's not a big deal.


A noter que j'ai fait cette PR pour que tu regardes le code, mais elle n'est pas mergeable en l'état. Beaucoup d'erreurs au niveau des tests sur les controllers. Pour l'instant, seuls les controllers HomeController et WelcomeController passent au vert)


Maybe an internal error is not the right thing, but it looks like, unless the expected type is a unit type, it's dropped on the floor. So I could annotate a unit expression with type `Integer`, and the unit wouldn't be checked against its expected type. Am I misunderstanding something?


OTOH, the `RemoteException` / `CapturedException` has the remote call stack shich will be lost if we only throw the underlying exception.


A noter qu'on créé un user de la forme :admin. Je ne comprends pas pourquoi le test ne passe pas lorsqu'on fait FactoryGirl.create(:user)


Siempre había creído que nosotros necesitábamos B2, pero es verdad que no encuentro que se diga en ningún lado que deba ser así. Por otro lado, con CEFR me refiero al marco de lenguas. PET se refiere al título de Cambridge reconociendo ese B1 (CEFR).


Thank you!


I was going to say, I wish this manifest supported `${variables}`, it does?


My understanding is that check happens in another part of the type system, the program:
```
(: u Integer)
(define u (unit (import) (export)))
```
Gives:
Type Checker: type mismatch
  expected: Integer
  given: (Unit (import) (export) (init-depend) Void) in: (unit (import) (export))


where is the `${applicationId}` set?


Sounds good.


No keywords?


I think these need to be reversed. `DomEventsPlugin` is to be first as it is end of the chain. Otherwise if someone adds new binding it will sandwich the `DomEventsPlugin` which is not correct.


From which test are you talking?

Note: this branch is very very experimental (it's the future v2).


Agree with all the concerns, still think it might be worth it (even if `dcloud` isn't the one). Here's my thinking:

1. `dc` is short and [non-unique](https://github.com/documentcloud/documentcloud/blob/0e2eb1641615dae09794ed0e0f3efc9f5ffe75cf/public/javascripts/embed/search_embed.js#L37-L40)
2. `dcloud` is more specific and thus less likely to stomp or get stomped
3. If we're going to introduce a namespace change, this is a sensible time/place, unless we expect this new embed loader to be API-compatible with the old. (My understanding was we'd keep serving the old loader indefinitely, but that this new one could be backwards-incompatible, e.g. not keying off the object ID alone.)

I can be easily convinced to go back to `dc`.


Why is `@Injecto(NgZove)` needed here? It was not needed before.


Can I haz a readme sort of document or any kind of documentation actually. I currently have no idea of how anything works. In fact, I currently have no idea of what's going on. (Sorry for the disturb. Delete this at wish)


@saschadube Yes, thanks!


So with this, GBE+ can dump and load graphics for DMG games, and it can replace them at scales of 1:1 all the way to 10:1 (I wouldn't try 10x HD though... the limit is arbitrary however).

Next up, loading GBC BGs and OBJs the same way. After that, making the UI human friendly ;)


I can't find the thread again on a quick glance, but I'm almost sure this no_nul feature was already planned. Looking good to me.


@TobiasRoikjer preferred otherwise


Will yield an extra empty directory path: `plugins\foo\\blah` in the common case (no targetDir).  Alternative:

    var paths = ['plugins', plugin_id, path.basename(src);
    if (obj.targetDir) paths.splice(2, 0, obj.targetDir);
    dest = path.join(paths);


some cleanup, LGTM otherwise


rax retry tests


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Maybe something went wrong, looks like gaia-fast-list.js is not a symlink but contains the change


Puoi velocizzare leggermente il tutto mettendo un check più furbo sull'`if` appena sotto, visto che per gli eventi che correggi con `0, 0, 0` non hai bisogno anche di calcolare la correzione centrale, ce l'hai già.


The `-1` was a WebJar bug fix on the same upstream version.  So that is correct.


It was a factory before and NgZone was passed explicitely.

However the class should probably have `@Injectable()` now for Dart ?


Is SNAPSHOT on maven central?


Avaleht, pealkiri, taust


@gpolitis same. Please install a JDK6 and build against it or it will burn you again and again.


`return getInput().numDimensions()==2;`


@vsavkin What is the behavior wrt to child Injector ? Would adding a multi binding to a child injector add them to the parent bindings or replace them ? (May be there should be a test for this use case ?).


My bad I misread you *removed* the PHP5.5 for Travis test


This would be slow. You should use either:
- erlang:function_exported(Module, module_info, 0)
- erlang:module_loaded(Module)


Man, getters are so much easier. :disappointed:


Much better way: eliminate this function entirely and replace `to_string(colors[count])` on line 602 with `colors[count]._cssString`.


One more thing -

IF there is a problem with the number of characters in specific words while transferring those into Souliss (which might appear changing from one language to another and having limited spacing for words) just let me know.

I'll think about a work-around then (e.g. abbreviations, etc.).


This is a welcome improvement.


whoops, I'll fix later.


this should be in the file defaultSettings.json
at the same level as the `core` key you can have `presentation`, `exercise`, `presentationShared` keys


I should've left a comment.  See the commit message of dde75eb.


> We don't embed APIARY_API_KEY into tests. This is security issue for me. We have APIARY_API_KEY set only in CI. Solution is add encrypted APIARY_API_KEY to travisci but for localtesting you have to use your own key.

But my own key does not have access to the apiaryclienttest domain. What can we do about that?


can we also add a mixed version, i.e. each "safe" pixel is calculated like this and all "border-pixels" are calculated in the "normal" way? like this we have same behaviour for as with `MapNeighborhood`


`api` should have a `settings` objects which exposes `lib/settings/index.js` and augments it with your `defaultSettings`


LGTM


:heart:


I don't follow...?


Can I use the Spine Runtimes in my open source project?
While we do contribute to a number of OSS projects (libgdx, Kryo, KryoNet, and many more), Spine is a commercial product. The Spine Runtimes and any derivative works or improvements must be released under Spine Runtimes license. They cannot be released under the GPL or any other incompatible license.
To avoid having your software project become encumbered by the Spine Runtimes license, one solution is to make the Spine Runtimes portion of your software an optional module which is licensed under the Spine Runtimes license. This gives your users the choice to opt in to the Spine Runtimes license if they want, while still allowing you to license the rest of your project as you like.


#5067-CoilCoolingWaterToAirHeatPumpVariableSpeedEquationFit-inside-AirLoopHVACUnitarySystem (rraustad) - x86_64-MacOS-10.9-clang: OK (1810 of 1811 tests passed)

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


hello oblonski...could you help me understand the Java client api by providing a detailed documentation rather than that on GITHUB on how it works and how to implement it on Android Studio and how to make it running on a mapview. It's really confusing probably because i am new to android programming.
Help me out brother..

Regards from Nepal


Yes, I think this is a pattern used pre Ruby 1.9 because of ordering issues, and I personally don't think it worth continuing.


after you do the refactoring suggest above, maybe you don't need this file since you have the defaultSettings sorted out in `model/setting`


But when I put the default settings as the some level of core key, there is an error at Line 84 in /models/settings.js.


The output in the comment is wrong, doesn't match the input.


Is this lambda necessary?


Any reason for removing this line in the example here?


`queue_counts` only gets set to the themes queue counts on the themes homepage so the `tab` was never in `queue_counts`. `queue_counts` is gone now so this isn't needed.


@mtholder, sorry for any confusion by my comment with this commit. I had assumed you were trying to clean up unused API methods, but I don't see evidence of this now. :confused:


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - 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-89.07%25-red.svg)


Please help me out.I am really confused on how to implement this Java Client API on my Android Studio and view it in a map.Could anyone share detailed documentation on how to implement the API on android studio and view the result in Map View in android.

Thank you in advance.


a nicer, shorthand way to write this would be

```php
<?= single_post_title() ?>
```

which is the equivalent of

```php
<?php echo single_post_title(); ?>
```

Also, looks like you removed the `<h1>` tag but probably didn't mean to?

```php
<h1><?= single_post_title() ?></h1>
```


Thank you Rob - I'll double check this, but I think it should be just ignored and this should work correct


In this commit, the change to `gaia-fast-list.js` is not about what the commit says it is.


yes... but it's temporary.. will update to the latest fabric8 release when
it makes it out of the build...

On Thu, Sep 3, 2015 at 9:52 AM, Arun Gupta <notifications@github.com> wrote:

> Is SNAPSHOT on maven central?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/christian-posta/kubernetes-maven/commit/4d3e06a795c926c61c47774357acdf325267e94d#commitcomment-13052406>
> .
>

--
*Christian Posta*
twitter: @christianposta
http://www.christianposta.com/blog
http://fabric8.io


Thanks, that was incredibly quick service :-)


Keep track


@Th3Mouk  and everyone else wondering http://v4-alpha.getbootstrap.com/components/label/


Indeed. In my initial change tests were failing because of not disposed models. Will revert the change.


On Thu, Sep 3, 2015 at 3:38 AM, daviesrob <notifications@github.com> wrote:

> It this related to there apparently being two ways to specify positions in
> .bed files?
> See the comment in samtool's test.pl here:
> https://github.com/samtools/samtools/blob/develop/test/test.pl#L749
>
>
# The other format has only two elements (not legal in a normal BED file).

FWIW, it's a misnomer to call the second format "BED".  It's more like the
first two columns of VCF.


Everything you wrote makes sense.  I'm pretty new to cmake, but I have a feeling the way forward is to remove this from here and leave it the stuff I added to libssh2.h to figure out USE_WIN32_LARGE_FILES.  That's what I did in https://github.com/dbyron0/libssh2/commit/8780ff5491247854d2fb695b91d7888f2b480c33.  How does that look?


Did it actually fix your problem? cause I didn't change anything in the code logic, just released a new version with the latest changes.


101884090-Venting-Availability-Schedule-does-not-work-properly-in-the-AFN-model (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1803 of 1804 tests passed)

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


th


htdh


hdh


ff


I assume there is supposed to be an ALLOW here?


The key is exposing the "onAuthorized" event, I could have used a global event emitter too (using ``microevents``), or initialized GAPI in ``index.js``, or like here expose a promise resolved once API is ready.

None of those 3 methods is satisfying but it works :)

ping @Delapouite @t8g


Dude, you still use get_document_or_404 and it is undefined. WTF?


Tooltje


I haven't actually changed anything at all!  Heroku keeps referencing old files so I forcibly repushed every file in the repo.  This commit is identical to the previous one, except I deleted some whitespace (so that i could make a new commit)


(new for me) I think you can inject all kind of variables in the manifest from build.grade.


It does prevent focus stealing, but also on KWin it seems to cause windows to appear behind others even in the same VM.

Needs some more work to figure out something that works exactly right.

So yes this is probably not quite the right solution, which is why I kept it separate.


well there was no serious problem, but the annoying warning is removed now


Tryed this fix, it seems to work =D Cheers!


In fact I had pull request, which is a same damn code. And my pull request was waiting for this for so long time. Dude, why? It is so sad that you prefer to drop pull requests but still use code. Even... you forgot to remove old line so now this will throw exception about undefined function. I have no words. Just a shame.


Ok, thanks.


This should be solvedby escaping table name:

static mapping = {
  table '`condition`'
  ...
 }

http://blog.adamcreeger.com/2010/05/random-grails-tip-using-db-reserved.html


nice catch!


These are great suggestions!

I implemented your suggestion of removing the v_getUrl() function in a recent release; that makes perfect sense.

Would you be willing to checkout the latest version and submit a pull request with your `stop_handle` filter?


Should I remove finals from all local variables or just here?


Hmm my symlinks didn't seem to be setup correctly. Pushed. But we have some test failures now, likely due to reordering things.


This might break P31XX and P51XX builds, you might want to check


test...


why duplicates of e2fsck,make_ext4fs,resite2fs,setup_fs


Looks good although I think you've swapped a grammar error for a spelling error ;-)


OMG !
![3vgswq](https://cloud.githubusercontent.com/assets/2074571/9665417/902fdd10-5246-11e5-8b30-50cd60b00b1b.jpg)


<img src="cinqueterre.jpg" class="img-circle" alt="Cinque Terre" width="236" height="236">


empd (jdwoods21) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1272 of 1274 tests passed)

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


En verdad "trillocientos" tampoco está en la RAE ni nada. #GrammarMatters


I think this should be `exp(uwts - max(uwts))` if you are using the elpd rather than the ic.


Without this change, we get
```
mnt/agent/work/34ebaeb8809bf071/compiler/frontend/src/org/jetbrains/kotlin/types/expressions/LocalClassifierAnalyzer.kt:134: [Internal Error] org.jetbrains.kotlin.codegen.CompilationException: Back-end (JVM) Internal error: Error type encountered: [ERROR : LookupTracker$Companion$DO_NOTHING$1] (DeserializedType). One of the possible reasons may be that this type is not directly accessible from this module. To workaround this error, try adding an explicit dependency on the module or library which contains this type to the classpath

Cause: Error type encountered: [ERROR : LookupTracker$Companion$DO_NOTHING$1] (DeserializedType). One of the possible reasons may be that this type is not directly accessible from this module. To workaround this error, try adding an explicit dependency on the module or library which contains this type to the classpath

File being compiled and position: (134,83) in /mnt/agent/work/34ebaeb8809bf071/compiler/frontend/src/org/jetbrains/kotlin/types/expressions/LocalClassifierAnalyzer.kt
```


Cool!


For the button DOM IDs, do not suggest using generic names like "okay" and "cancel" since it can also be a possible DOM ID for the Web Application the client is using.

Suggested terms: "adapterjs_okay", "adapterjs_cancel"


I removed the FIXME comment based on https://lists.w3.org/Archives/Public/www-style/2015Jul/0360.html


stratified_tank_docfix (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (762 of 762 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-10.35%25-green.svg)


Mind changing this to `const`? I forgot to add `prefer-const` to the linter, sorry about that.


I now know what the fuss was about.


Yes, I saw it as a temporary measure, I wanted to move everything out of the File_system namespace. I think that if the VFS server works well enough the ram_fs will be redundant.


TeamCity ReOpenLDAP :: devel-2.4 [Build 186](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=312&buildTypeId=ReOpenLDAP_Devel24&guest=1) is now running


That’s some of them. Tests with "supports" in the name fail because we don’t support `@supports`. I’ll look into this next.


Amazing!


Agreed with Alec's last comment here. What would be great would be to split the plugin entirely so that there's a usage stats plugin (possibly shipped with OJS, but easily enough just added as a submodule), and an ALM plugin that *isn't* included as a submodule or included natively. For now, if we include this as a submodule in its entirety, we'll still be providing some basis for new OJS users to think that the ALM service is ready to be used. I think we should avoid that.


Hi @temasek  i hate that kind of messages but  . this is wrong authorship mate .https://github.com/ResurrectionRemix/android_frameworks_base/commit/6345783cea184a9f5b18914d7fe4b233656642b1


Good fix because the Bitcoiners are being mypoic


Hmm, in retrospect, I think this should probably just be `visualize(self)`. If other visualization methods appear (could also do `asciitree`), we can distinguish between them using a new argument (maybe `mode='nltk'`). Also, this needs a docstring.


Seems like there was some endline issue at this line, since I don't introduce any changes here.


Ah, OK. That's fine then.


before
```css
.col-xs-1 {
  -webkit-box-flex: 0; /* no width breaks down a markup */
  -webkit-flex: 0 0 8.33333%;
  -moz-box-flex: 0;
  -ms-flex: 0 0 8.33333%;
  flex: 0 0 8.33333%;
}
```

after
```css
.col-xs-1 {
  -webkit-box-flex: 0;
  -webkit-flex: 0 0 8.33333%;
  -moz-box-flex: 0;
  -ms-flex: 0 0 8.33333%;
  flex: 0 0 8.33333%;
  width: 8.33333%; /* fallback for flexbox 2011 */
}
```


#5067-CoilCoolingWaterToAirHeatPumpVariableSpeedEquationFit-inside-AirLoopHVACUnitarySystem (rraustad) - i386-Windows-7-VisualStudio-12: OK (1816 of 1817 tests passed)

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


That comment was already there since the first implementation, what I meant with it was to see if we can get a specific error in the case that it failed because it tried to set a polarity value that's not supported, as opposed to failing because it couldn't write at all, for example. I was considering the case of someone failing to open the PWM because they wanted an inversed polarity, but never considered that a driver that exposes the value would fail to write at all, even the same value it already has, because it doesn't support changing it, as is the case with Galileo and Edison, apparently.


It is not random: this is the function where the bg cube gets its bg rate units, and is called each time a cube is created.
If we put it in read or write, the cube might still be using other units in the meanwhile. I think `read` should read the cube as it is in the file, and `write` should only write, not interpret the results.

But now that you mention it, this is not the right place, since this function is inside the generic `Cube` container class, and we don't want to force `MeV` for all cubes, right? I'll move the units forcing to the `make_bg_cube_model` function.


Ok you insist on join :smile:


Not sure, I think @CestDiego explicitly meant to add `˜/.emacs.d/ini.el` to the Emacs perspective which makes sense. We can also add the dotfile which is indeed discovered with the function @robbyoconnor mentioned.


return move || getInput();


Why are you rescuing `LoadError` again?

This is just suppressing errors which is bad. Also, sites probably won't function correctly.


Done.


@BuzzBumbleBee How do you calibrate `DELTA_RADIUS_TRIM_TOWER`, I think it's a little more complicated than `DELTA_DIAGONAL_ROD_TRIM_TOWER`. Traditionnaly, moving the nozzle back and forth will gives a concave/convexe trajectionary. Is there a more "precise/clever" way?


please add a

version_added: "2.0"


@syl20bnr I already made the change...this perspective comes from my old setup where *MY* init.el was the ~/.emacs.d/init.el, but now that spacemacs is in use I guess it doesn't make sense now to open that..and instead the user's dotfile is more intuitive


No change in infiltration object, maybe something with line ending?


Yup, you're right. I had first coded it in terms of the ic and then switch
to elpd and forgot to swap the min and max. Thanks.

On Thu, Sep 3, 2015 at 1:20 PM, bgoodri <notifications@github.com> wrote:

> I think this should be exp(uwts - max(uwts)) if you are using the elpd
> rather than the ic.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jgabry/loo/commit/5090904c6dfeff08bc761995703f8de2dd355696#commitcomment-13052999>
> .
>


nice catch here. lgtm


The other commit wasnt needed if you guys know what your doing instead of copy/paste everything.
You guys copy/paste so many commits that you even don't know what you even pull and also don't even know your own source. The original buttons made by hyde would have worked without any issues on bliss source as the "sync status bar signal cluster code with AOSP" commit was already present in the bliss source . So instead of stealing this commit and the main qsbar commit out of our (Cyanide and Fusion) private repo's thru Jacob McSwain. And just pulled it from hyde then you only had to modify a few little things to get it to work on cm based roms. (Only 5 minutes work) But you need to know what your doing!! You guys proved multiple times that you all don't know what your doing. So to get a nice feature that you guys want and cant port/make yourselves then steal or copy/paste it from all REAL devs that do the actual work and spend many hours of work on such features, ports or fixes.

Also the comment in the main qsbar commit is total bullshit. like i said just had to pull it from hyde, then
you could have written own comment to it, if you got it to work by yourselves on cm based roms.

Happy copy/pasting in the future and be happy with this.


@chrisdev  :smiley:


This looks good - I need to compare my current dev version to ensure it's not going to conflict with changes, but otherwise I will look to get this in as part of the next release.


Thanks! Looks great. Should I populate method in updater to match changes?


guid needs to be able call SetMonitorLayout itself anyway for private mode, so keeping the existing system means that the existing system also has to know about whether private mode is set and do nothing in that case.

It's possible to do that though.

It's also possible to move the private mode SetMonitorLayout code to the GUI agents and then completely remove all monitor layout code from guid.


I am facing same concern.... Can you please confirm in which files these changed need to be incorporated.... as unable to find any such files
install-dev/data/db_structure.sql
install-dev/upgrade/sql/1.6.1.0.sql
install-dev/upgrade/sql/1.6.1.1.sql


It seems some hard dependencies to your setup got added to the gradle.xml. Please remove those from your PR. I would suggest not to add any changes to gradle.xml.


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

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


Oh, right. Thanks!


Spine is awesome and it looks like a great addition.  This is just something we wouldn't be able to compile in without a separate agreement with Esoteric.  I wish they would release the runtimes under a permissive license, which would help adoption of their tooling.


You also need to update the argument of `cancelAnimationFrame()`.


@boodri Should be fixed now on the master branch

On Thu, Sep 3, 2015 at 1:30 PM, Jonah Sol Gabry <jgabry@gmail.com> wrote:

> Yup, you're right. I had first coded it in terms of the ic and then switch
> to elpd and forgot to swap the min and max. Thanks.
>
> On Thu, Sep 3, 2015 at 1:20 PM, bgoodri <notifications@github.com> wrote:
>
>> I think this should be exp(uwts - max(uwts)) if you are using the elpd
>> rather than the ic.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/jgabry/loo/commit/5090904c6dfeff08bc761995703f8de2dd355696#commitcomment-13052999>
>> .
>>
>
>


It got patched in the next commit, please look at HEAD...

On Thursday, September 3, 2015, Ryan Leach <notifications@github.com> wrote:

> I assume there is supposed to be an ALLOW here?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SpongePowered/SpongeAPI/commit/3c4187b900e7bdcc30e8eda5ac933830e2450b0e#commitcomment-13052769>
> .
>


The changes to the vcxproj generator need to be in its own commit with a beast-specific commit message.


good idea


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.489.e8faa8feeae2d545dcf99cf9c5b81db4ab07fac4](http://build.palaso.org/viewLog.html?buildId=62641&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3318, ignored: 140 Build time: 00:09:46


I wasnt able to build MetaEmoji on my Linux system until I've commented this error macro, which seems to be wrong.


Why it relies on str_shuffle? Bias is spreaded randomly between all chars.


Hi @mdinsmore
I removed the requirement for `wizard_fields`.


@alimaredia could you squash this commit into bf82c65e330e36971ce182c7eefaa2a5e14f9d9e ?


For now it's OK. Later, here will be custom color scheme.


Why 30 and not 29? It seems like they included 31 (below this) to probably indicate to people that 31 is the upper limit but I'm not sure why you decided to skip 29...


Good use of the nav classes. And specifying different IDs


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.470.e8faa8feeae2d545dcf99cf9c5b81db4ab07fac4](http://build.palaso.org/viewLog.html?buildId=62640&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3357, ignored: 134 Build time: 00:12:33


Do you need this additional block { and } here?


why?


Mas já melhorei pra 38 segundos :smile_cat:


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2295 of 2336 tests passed)

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


Thanks.


Cover should be in the root, not the images folder.


thank you


I think would be more urho's style to have it's own runtime and convert the spine files to urho's.  That is much more work naturally :(


Why?


Hi,

If you are in 1.6.1.0 or 1.6.1.1 you need to run thses queries against your database directly

See http://forge.prestashop.com/browse/PSCSX-6430

Regards


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

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

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

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

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

```


Maybe. Or maybe I should implement pickaxe support soon. Though probably blame would have been enough here too.  :-)


why would conn be `nil` here?


it looks like it was pulled out of the API in v5. I would include the training id as parameter to the `Completion.listFromJson` and `Completion.fromJson` methods.

i.e.
`Completion.listFromJson(long trainingId, JSONArray json)` and `Completion.fromJson(long trainingId, JSONObject json)`


In JUnit4 the exception (actually Error) thrown by a JUnit assert is the same as the error thrown by the java assert keyword (AssertionError), so it is exactly the same as assertTrue and other than the stack trace you couldn't tell the difference.

That being said, asserts have to run with a special flag in the JVM, causing many tests to appear to pass just because someone forgot to configure the system with that flag when the JUnit tests were run - not good.

In general, because of this, I would argue that using the JUnit assertTrue is the better practice, because it guarantees the test is run, ensures consistency (you sometimes use assertThat or other asserts that are not a java keyword) and if the behavior of JUnit asserts should change in the future (such as hooking into some kind of filter or other future JUnit feature) your code will be able to leverage that.

The real purpose of the assert keyword in java is to be able to turn it off without runtime penalty. That doesn't apply to unit tests.


Span is not really needed here. Inside of <li> element you can have multiple elements but you are only interested in text on the <li> element. So you create <a> element, sat .text on that element to be whatever it was on <li> element and at the end you insert <a> at the first position.

```python
        for li in chapter_content.xpath('//li[starts-with(@id, "endnote-")]'):
            a = etree.Element("a")
            a.set('name', li.get('id'))
            a.text = li.text
            li.text = ''

            li.insert(0, a)
```

So you head something like
```html
<li id="endnote-1441301648776">This is my footnote. <b>This is bold</b>. Here we have <a href="http://www.google.com/">link</a>.</li>
```

and it will become

```
<li id="endnote-1441301648776"><a name="endnote-1441301648776">This is my footnote. </a><b>This is
bold</b>. Here we have <a href="http://www.google.com/">link</a>.</li>
```


Queries Only with +


OK


It might be useful to print out some error message here


http://developer.android.com/intl/zh-cn/reference/android/bluetooth/BluetoothAdapter.html

> To get a BluetoothAdapter representing the local Bluetooth adapter, when running on JELLY_BEAN_MR1 and below, call the static getDefaultAdapter() method; when running on JELLY_BEAN_MR2 and higher, retrieve it through getSystemService(Class) with BLUETOOTH_SERVICE.

Let's keep using getSystemService.


i dont understand how to instal updates for wa??? Where is download option? Please help!


It might be useful to describe the error


yep


Ele melhorou muito. Esse problema é desgraçado. Eu sofri demais para conseguir esse desempenho dos 10s. Busca perfomática de números primos é maracutaia das brabas. 38s já tá na one-minute-rule. A outra solução fica lá como exemplo "não devemos fazer isso". Eu deveria ter pego as minhas primeiras soluções pra esse problema, demorava pra caramba também. Seria interessante conseguir em menos de 1s com C. HAHAHAH


don't we want the canonical address before we get the proxied url? or do we want to let the proxy choose a default port for us if req.URL doesn't include one?


If this dies here, doesn't the `send_data` thread run forever listening for messages that won't ever occur?


TeamCity YaccConstructor :: YC.Core General Build [Build 491](http://teamcity.codebetter.com/viewLog.html?buildId=198488&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


You need to look at authproc rules and do something like:
        'authproc' => array(
            85 => array('class' => 'authorize:Authorize',
                        'mail' => '/^my\.special\.domain$/',),
        ),


You could also use [string1 isEqualToString:string2];


ALTER TABLE `PREFIX_specific_price` DROP KEY `id_product_2`;
RESULT >>  #1146 - Table 'esadar_nmkz1.PREFIX_specific_price' doesn't exist


Ok, I understood my mistake. Thanks!


А как, кстати, лучше располагать js виджетов - в корне (/widgets/nooui-blabla.js), или выделить для плагина подкаталог (/widgets/nooui/blabla.js)?


Moved, thanks.


You can check at www.esadar.com


В текст ссылки можем добавлять сам текст))  Должно получится вроде этого: `<a class="main-logo" href="/">Nerds</a>`


understood, thanks for the explanation ;)


//return /* Your Expression */;


```
❯ g mybranches
3 months ago  origin/ruby-2-and-beyond
3 months ago  origin/update-client-cassettes
3 months ago  origin/add-ruby-head
```


От br'ок избавляемся!


Good catch. This has been fixed, and now says `>=5.4`


True, but until you realize the need for `std_string_from_jstring()` they're just as maddening.


I didn't know that.


Why doesn't the elemental part have successful map_addblock check?


может, поменяем span на header?


You could do it without span element, like this:

```python
        for li in chapter_content.xpath('//li[starts-with(@id, "endnote-")]'):
            a = etree.Element("a")
            a.set('name', li.get('id'))
            a.tail = li.text
            li.text = ''
            li.insert(0, a)
```

Original html:
```html
<li id="endnote-1441301648776">This is my footnote. <b>This is bold</b>. Here we have <a href="http://www.google.com/">link</a>.</li>
```

After html:
```python
<li id="endnote-1441301648776"><a name="endnote-1441301648776"></a>This is my footnote. <b>This is
bold</b>. Here we have <a href="http://www.google.com/">link</a>.</li>
```


OK. I'll try to do that as soon as possible.

2015年9月3日星期四，apuder <notifications@github.com> 写道：

> It seems some hard dependencies to your setup got added to the gradle.xml.
> Please remove those from your PR. I would suggest not to add any changes to
> gradle.xml.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ZTGeng/android-tutorials/commit/e069beab9a58a55a999f90740c6428168ea612a3#commitcomment-13053284>
> .
>


Ah yes... the trick is in this a.tail part.


Clean code issue:
The 'test' prefix is simply a holdover from the pre annotation days when that was required. I would suggest you simply use meaningful names for your test cases (and that may mean with or without 'test').
Consider the following name: validateFirstNameSize() or shouldValidateFirstNameSize()


I would just include the FLAG_PREVENT_DOWNLOAD_EXECUTION check before the current DEBUG_POLICY_INSTALL print statement and dispense with all the other logging about updating the seinfo labels. I think one logging statement is just fine to cover all the cases. Something like the following:

 if ((pkg.applicationInfo.flags & ApplicationInfo.FLAG_PREVENT_DOWNLOAD_EXECUTION) != 0) {
    pkg.applicationInfo.seinfo += NOEXECUTE_STR;
 }

 if (DEBUG_POLICY_INSTALL) {
            Slog.i(TAG, "package (" + pkg.packageName + ") labeled with " +
                    "seinfo=" + pkg.applicationInfo.seinfo);
 }

Also, there can never be an empty seinfo tag so your check concerning that will never hit.


I think I added this for future use. However, I've reverted back to the original.


спан — это строчный элемент. А заголовок — это блочный. Давай заменим тег тут тоже


develop (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Hi,

PREFIX_ by your prefix, like ps_ ?

I invite you to contact our support team to get custome help wether than getting support ghere.

email: support@prestashop.com
Phone (French): +33 1 83 64 16 54
Phone (English): +1 (888) 947-6543

Best regards.


Just curious. Was this validation failing for you?


needs to be changed in the interface as well. Also requires a note in the UPGRADING file.


1579840-GroundTemps (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1808 of 1824 tests passed)

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


Really? :D


This looks great 👍🏻


@erichelgeson can you please cut a new release with this change?


back to class


Sorry but the pdffiller is not the correct module. node-pdffiller is the update npm package. The old pdffiller npm module hasn't been updated in quite a while and the person who owns said module is no longer involved with development of this library.

Before changing this, could you please make an issue or comment?


small world, you're working for REA and I'm scrapping data from REA websites ;-)


Don't use wildcards. They unnecessarily increase buildtime.


Можно попробовать сделать Editable.Factory.newEditable(text) и работать с ним.


Closes #47


ALTER TABLE PREFIX_specific_price DROP KEY id_product_2;
RESULT >> #1091 - Can't DROP 'id_product_2'; check that column/key exists

ALTER IGNORE TABLE `PREFIX_specific_price` ADD UNIQUE KEY `id_product_2` (`id_cart`, `id_product`,`id_shop`,`id_shop_group`,`id_currency`,`id_country`,`id_group`,`id_customer`,`id_product_attribute`,`from_quantity`,`id_specific_price_rule`,`from`,`to`);
RESULT >>  #1062 - Duplicate entry '0-3921-1-0-0-0-0-0-0-1-0-0000-00-00 00:00:00-0000-00-00 00:00:00' for key 'id_product_2'


yes. that is what i was saying.


A closer analysis with valgrind revealed that the `ConnInputEndPoint` instances indeed own they own copy of the `ConnID` and that it needs to be deleted in the destructor. This is different than for the `ConnOutputEndPoint` class, that only has a pointer to the `ConnID` stored in the `ConnectionManager`.

I will revert this patch.


Very impress with this project. Very nice, that you took this game and completed in a short amount of time. Great job.


Why do you have to do this? Does another test depend on the application being in its original state?


Hi,
ok i will check your patch.
BTW in my test i make 2 calls in the same tab, not in another tab.
Process: Load Plugin/Install Plugin -> Call -> Close Call -> Redirection
on page without the plugin -> Go back to first page

I don't know how IE handle the plugin behind the scene, but
rtc::ThreadManager lose the reference to the parent pointer after the
first call and __Utils::Initialize() is not triggered a second time when
the plugin is loaded, and i end up with a segfault when webrtc try to
use the currentThread.

Are activexctrl loaded per Tab, or for everytab ?

ref:
VideoCapturer use rtc::Thread::CurrentThread in the constructor
https://chromium.googlesource.com/external/webrtc/+/master/talk/media/base/videocapturer.cc#106
VideoCapturer::SetCaptureState doesn't check if thread_ is not NULL
https://chromium.googlesource.com/external/webrtc/+/master/talk/media/base/videocapturer.cc#555

Le 03/09/2015 16:15, sarandogou a écrit :
> I don't have such issue. I can close a tab and make another call without
> crash but I had to patch WebRTC code
> (https://github.com/sarandogou/webrtc-everywhere/blob/master/webrtc_win.patch).
> I have to check your patch.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MikeTangoEcho/webrtc-everywhere/commit/4f9c66e924aacb67498ecbca0d13f3a550dcc903#commitcomment-13048182>.
>


I really like how you separated the solution into a Fraction class and an addition/reduction class.


1579840-GroundTemps (mmatts) - 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)


Some test would be great, at least for new classes


gtgbemxj


@jalperin, do you mean removing the ALM plugin from the distro, or leaving it in, or splitting it in two?


This may be left over from the previous design.


Consider statically importing the assertEquals method. Having "Assert." everywhere doesn't add any value and is code bloat.