People didn't really "get" it. I had to explain it to basically everyone in the building, sometimes twice, and people still didn't really seem to like it. If a person had "Always vector" turned on, it made some of the placement of text items wonky, and that was a problem.

In the end, it was more trouble than it was worth. It didn't really help in multi-sheet schematics, for instance.


I've written plenty of inefficient algorithms, so it's not just you.

With the `CanastaConstants`, I was thinking more that you just include the `CanastaConstants` module in both the `RunOfCards` class and `RunOfCards::RunHelper`, that way those same methods will be available in both classes.

Maybe some of the confusion arises from how we were calling `SmallRun`, `LargeRun`, etc, from the `RunOfCards` class. The reason why we can just reference `SmallRun` from the `RunOfCards` class, and not use the fully qualified class name (`RunOfCards::RunHelper`) is because the `SmallRun`, et al, constants are name-spaced inside of `RunOfCards`. If you were to try to just call the `SmallRun` class from a separate class you would get an undefined constant error. Then you would _need_ to use `RunOfCards::RunHelper` to access the class.


она по-хорошему должна быть private, ведь результат зависит от внутренних данных хеш-таблицы и вне неё неприменим. Более того, результат даже хранить нельзя, потому что если размер хеш-таблицы поменяется, результат будет невалиден.


Update task.clj, import backtype.storm.generated.StormTopology


On closer look, `imageAsTikz` already handles flipped axes correctly in a way that is rather different from `imageAsPNG`, so it has no need for `flipImageIfAxesReversed()`.


We have various features of taggit extended (https://github.com/mozilla/kuma/blob/7430f6b957a2df41e44228f75677d3d377815929/kuma/core/managers.py#L78) and backported (https://github.com/mozilla/kuma/blob/7430f6b957a2df41e44228f75677d3d377815929/kuma/core/managers.py#L65). E.g. the latter `PrefetchTaggableManager` is a built-in for a while: https://github.com/alex/django-taggit/commit/4f2e47f833a36ca3bce3bb31ebd6f25f88d4952c


and again :)


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1120 of 1120 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 thought that you had moved it to a configured option in the Automation
menu.

On Tue, Mar 3, 2015 at 5:59 AM, ACG8 <notifications@github.com> wrote:

> I thought we are still asking players whether they want the die roll area
> to the left or at the bottom of the board?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sIKE23/Mage-Wars/commit/a6dc537c1b363dbf73eb6dd779df554743dc07b4#commitcomment-10004780>
> .
>


Also a small thing, but for sake of clarity maybe we could make sure `get_last_uid` returns an int by moving the type check to inside it, and then just increment here. I also don't buy the `TypeError` catch, maybe would change it for a simple if, but feel free to do as you find best.

```python
    def get_next_uid(self, mailbox_uuid):
        check_good_uuid(mailbox_uuid)
        d = self.get_last_uid(mailbox_uuid)
        d.addCallback(lambda uid: uid + 1)
        return d

    def get_last_uid(self, mailbox_uuid):
        check_good_uuid(mailbox_uuid)
        sql = ("SELECT MAX(rowid) FROM {preffix}{name} "
               "LIMIT 1;").format(
            preffix=self.table_preffix, name=sanitize(mailbox_uuid))

        def getit(result):
            item = _maybe_first_query_item(result)
            if not item:
                return 0
            return item

        d = self._query(sql)
        d.addCallback(getit)
        return d
```


@LJeub: Seems even better indeed: that way the flipping happens at the same place and you can just return the `CData` and `AlphaData` from that function.


There should be "var result = NativeMethods.NetUserGetInfo(serverName, username, 1, out userInfoPtr);"


This should be using a read-only git URL.


@jorendorff ClojureScript users tend to upgrade with relative frequency due to the pace of ClojureScript development. When this release goes out we'll be sure to make note of the coming breakage.


Hey Kurt,

Hmm.  Good point about the functionality not actually working at the moment.

I've got some ideas about that at https://docs.google.com/a/ngo.org.uk/document/d/1VtsaJ-ZjGfTHZct3fcvnbuy1WdCizjj92ZJlBFA7eCE/edit?pli=1#heading=h.15ign3cdjhqc -- basically, move the connection management code out of NowPlayingFragment, and in to HomeActivity and DisconnectedActivity, as appropriate.


@barsch Not sure were the link on the webpage is constructed, I guess that has to be adapted as well as this `git_commit_hash()` getter is not stripping away all but the commit hash from the looks of it..


I guess you meant
```
[[ $line =~ "### Additional Information" ]] && out=0
```


Here you need to insert Branches.unsubscribe(codeReview.getBranch());


ResultsSchema (sanyalj) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Coverage Too Low

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


This would be unnecessary because of previous addition of unsub method


sorry Brady, this should be

 switch ($num_parts) {


This is wrong format usage


@optikfluffel Does Travis support 1.0.1 or 1.0.2? What about Erlang 17.1 or 17.3?


Wich extension do i have to use for the scripts?


Ah, yes, will do.


here too

switch ($num_parts) {


Hmm, thinking out loud... do we want these filter/stage GET params to change here as well? That would mean we need to update the endpoints as well. I don't think I have an opinion either way.


can you add explictly the parameter names for posterity to understand :-D


branch is required


@lassewesth You mean in the test?


As far as I can see this is only used by python-openid, so I'm wondering if that's something we actually need at all?


I might be wrong, but it seems like this feature is not used in core. So is this just a convenience function? But aren't we telling people quite often that they shouldn't rely on these helper functions?


I initialize them in multiple places though, either in boehm's environment parser or sgen's. If I mark it as external in the header, then I'll need to define it somewhere. If seemed more consistent to statically allocate it in the header than in two separate places. Using 'extern' does make the header simply declarations though, so that's nice.


Firstly, thanks for the contributions!  If updating the Readme - perhaps you could also update the first paragraph "Currently it defines hooks sufficient to use an alternative CHAP secret provider"  to accurately represent the new hooks you have defined along with these updates?


yeah I noticed this couple of rebases earlier and always failed to fix it...


I'm still testing different values, but 1728 seems not to be that bad


Añadir menu mensajes


It shouldn't have been pushed, my bad


https://github.com/mozilla/django-product-details/pull/35 seems important


I think it is excessive here. It is very exotic chance that user would become autosubbed to branch if he deletes any topic


I really tried to found any reasons, but I just couldn't. Anyway feel free to discard this one...


IOW update to https://github.com/mozilla/django-product-details/commit/d168ab2e689a1b54a30a923386fe7948a74ff334


Since it's used everywhere in this file, maybe extract a helper method, e.g. `require_multi_tenant_options!(options)`?


I think this needs to be on a separate line, or it's not going to work.


again, branch required


looks great!  thanks.


I wasn't sure who manages the CentOS so I wanted to be more easy on them in case they require this...


forgot to logout()


There's only have a couple data types to keep track of at the moment, and I don't really see that changing much.  But I do think it's nice to support versioning within those types so that a change to them (e.g. remove/change a data member) means that old archives can still be read.

Is there specific problems you're thinking would arise that leveraging CAF's type registry would help with?  I had also thought that it might be good to be as CAF-independent as possible for the purpose of persistence so that varying versions of CAF don't cause unexpected breakages in data that someone's stored away for a while.  Or can I actually expect CAF's serialization and type system to have a well-defined backwards compatibility mechanism (e.g. can I expect a fictitious CAF 2.0 to interop with a fictitious CAF 1.0 ?) and also to be able to handle changes in the data types that I need to serialize/archive?  Would be really nice if CAF can do all that for me and I just didn't know!


What happens when this reads in a 64 bit payload to memory? Is there going to be a memory related crash?


I really don't like this! The issue description is shown at the top of each message thread in any case, so repeating it by default in the first message of the thread will lead to more confusion. Best get an explicit OK from Martin for this.


Styling decision @britanyponevelle


nitpicking: code style ;) (also in the other places)


this is the version from this morning. i don't know how much changed since then but the gifs are working are right now for each update method we have so far


My original plan for this assignment had you writing three functions, one per each, instead of writing a general shim for the pattern attribute. I'm happy to see a more general solution here, but it does mean that it's harder to create individual effects for each item.

Instead of manually adding CSS to each input based on validation, it may be easier to add or remove a class from the input, and style it based on that. Classes tend to scale better than `css()` anyway.


這樣看起來會很亂嗎？


[R/model.R:51:9:](https://github.com/paulstaab/coalsimr/blob/801a159cfe643e3b75bc456eb4e0ae72eabb5cef/R/model.R#L51) *style:* **Place a space before left parenthesis, except in a function call.**
```r
    if ((!dm$is_grp_model) && any(get_groups(dm) != 0)) {
        ^
```


uh


TeamCity cat-angular :: build [Build 45](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=15998&buildTypeId=CatAngular_Build) is now running


Maybe `debug` level?


Yep!


If I remove `negative: SyntaxError`, then the test is still useful.

(Source: https://github.com/v8/v8-git-mirror/blob/master/test/mjsunit/harmony/block-let-declaration.js#L83-L84)


@bradking Sure. ENABLE_CURLDEBUG is independent. It can be enabled for any builds. Could you please check patch for added option?
Regarding CMAKE_BUILD_TYPE I also have some doubts.
In autoconfig it related exactly to debug target, so I try to make it similar.

Also I wish to note that CMAKE_BUILD_TYPE does not make sense only unless not provided explicitly. This was my observations on cmake 3.1.2
 And this patch is not targeted to only VS IDE generator. I wish to use it on linux also.


Same response ;)


I've split the tests into separate files as requested, and I'll follow up with those additional tests once this lands.


I think this doc string is wrong. It doesn't convert to UTC, it converts to local time.


The problem is each kickstart defines something else, usually booth of them (!). The latest kickstarts usually use `poweroff` as a check for graceful power off was added to unattended install. In order to work it requires the machine powers off itself after installation. So `poweroff` is currently correct.


great...


@dalgaaf sorry, i mean something like this:
```
        else:
            # tell/<target> is a weird case; the URL includes what
            # would everywhere else be a parameter
            if (flavor == 'tell' and
                (desc.t, desc.name) == (CephOsdName, 'target')):
                url += '/<target>'
            else:
                params.append(desc)
```


(the check was new for me too, so probably it's not that long here, or I haven't check this part of the code for a long time)


Whoops! Bah.


> Node.js를 대체하며

같은 표현은 어떨가요? 대용품은 왠지 어감이 좀 부정적(?)인것 같아서요.


change to
```   logEvent("Login from ".$_SERVER["REMOTE_ADDR"]." for user $user " . ($ret? "success": "failed")); ```

Remote ip required


Source: https://github.com/v8/v8-git-mirror/blob/master/test/mjsunit/harmony/block-let-declaration.js#L158-L159


what is not supported?


YAY :) nice change :)


I believe it has been renamed since then to our old standard, but I will be renaming this as part of #690 which I'm working on next.


TeamCity cat-angular :: build [Build 45](http://catbuild06.internal.catalysts.cc/viewLog.html?buildId=15998&buildTypeId=CatAngular_Build) outcome was **SUCCESS**
Summary: Tests passed: 29 Build time: 00:03:09


We set it as "primary" color so that the designer can apply it their own color palette and change it as they like.


All thanks to you Thyrus...Appreciated


This parameter does not exist (any more).


耕哥非常喜欢在 commit log 里加上各种图标:P


You have this a lot. Wouldn't it be better to use `@override_settings(TIME_ZONE='US/Pacific')` on the test class, so it is constant in the tests? Even better might be to use a different time zone, which might catch subtle errors.


OK. All done now! Modified the examples and tests too.


Perhaps an IllegalArgumentException if `cacheSeconds < 0` ?


Yes, It is my fault. I was writing for...


> 우리는 어떤 언어보다 큰 패키지 생태계를 가지고 있으므로 이를 망쳐서는 안된다.

처럼 이어붙히는 건 어떨까요?


I think this is one of the reasons this is still failing: because you are now never deleting anything, you are shipping all schema elements that have ever been deleted ever to the client... so if I create a database with thousands of indicators  (through the API for example), share the database with you, and then delete it, you still have to synchronize the whole motherload.


Этот момент не понял. fmod стал использовать, т.к. % выдавал ошибку по-моему из-за разных типов данных.


No Comment


AFAIK, we don't use closed `</p>` tags in Spring Javadoc. Just the opening `<p>` tag.


ok thanks for the feedback. you are right, I need to do the copy


@davglass Did you ever get to the bottom of your data loss case? I'm concerned about upgrading now.


@snikulov , the code snippet I provided will work both for VS and for Makefile and Ninja generators, the latter when `CMAKE_BUILD_TYPE` is set to `Debug`.  It is the documented way to pass per-config preprocessor definitions on all platforms.  The code would be simpler if we required CMake 3.0, but I do not think curl is ready to do that.


Mostly just thinking about readability/understandability.  This is all internal implementation details and didn't want to write a lot of documentation so separate save/load seemed easy to understand.   And I'm not that consistent on pointer versus reference arguments (you can see I'm not even consistent within the same function here...), but sometimes I just like a pointer because you can read the calling code and more easily infer that an argument is intended to be modified.  E.g. without knowing the definition of `foo`, the later of these suggests the input may be modified:

```c++
int i = 42;
foo(i);
```

```c++
int i = 42;
foo(&i);
```


"nan이" 보다는 "nan에서"가 더 자연스럽지 않을가요?


@benbc Yes. Actually that's something you taught me


What does this mean/do?


I think we should remove Grappelli as it's a moving target and overrides styles and core templates.


Pivo ID: [#85113772]


We should add a check in `getPaymentProviderPlugin` that the payment method belongs to the right account, I don't think we verify it today.


Why commented out?


@lassewesth You want me to rename `expected`?


We don't use DOSGI anymore. Should be remove this comment, or did you set it just to know why we used this version?


such an improvement :3


> This is not specific enough i suspect. There may also be dracut systemd units in there (/usr/lib/dracut/modules.d/.*.service)

Updated to account for these as well.


As discussed in #10507, this was originally supposed to be a minor and harmless enhancement of `extend()`, but evaolved into a separator function :smiley:

@Narretz, has a point here.
(Plus, the names aren't super-intuitive imo.)


This is very likely to violate notification settings set in "My Account"


Yeah, I like the versioning support you added. It will certainly come in handy in the future and allow for backwards-compatible upgrading.

The problem I was trying to identify is portability: `typeid` is not guaranteed to return the same name on different platforms. Upon looking closer though, it does not seem to be issue, because you're not relying on the type *name*, only on the order of registering type *versions*. Sorry for barking up the wrong tree.

There exist thoughts to change CAF's serialization library so that it's fully exchangeable with Boost Serialization. At that point it could be a drop-in replacement.


.send({
  grant_type


Why not `$plugin->$getter()`?


i like the scoping and usage of the Dispose()-Functionality. As discussed, the converter might not be needed


I don't know why I'm here as author, but thank you :smiley:


Shouldn't this be class User?


1: M100 needs more reviewers
1: M101 signature is missing


Published to http://ui.edev.d2l.com/46_c8f09baec419ebabb6056f5f4feef33cceee416f


Ha-ha! They are back!!! :+1:


Stuff happens, all's well now and that's the important part :smile:


If you make your condition `(array_key_exists($cfgBlock, $configuration) && is_array($configuration[$cfgBlock]))` you only need one `else`.


`otrp_message_receiving` is a wrapper around `otrl_message_receiving` (see https://otr.cypherpunks.ca/README-libotr-4.1.0.txt), whose return value is a boolean. If it's true, this means that message received is OTR protocol's internal chatter and it shouldn't be presented to the user. The debug description indicates just that: an internal OTR message has been received, it is shown for debugging, but otherwise not presented to user. If you prefer other words for that, please let me know; I assume that if somebody is interested in these, they know enough about OTR to understand the context.


Looks like you used the word Model instead of User in your devise generator.


Should be users


@JordanCheney Two things:

1. This breaks existing code.
2. Such a change should go through a pull request first.

We just finished making the changes needed to support the new naming (HistBin), so not sure what the motivation was for this change.


I like to keep explicit, but it's ok without this. I think we shall have some in house standards... We really need an in-person appointment.


Just a thought - maybe `ConfigurationJSONParser#parseJSONConfig` should throw an exception if it couldn't parse the configuration. They way we would get rid of that nasty `if` one line below.


Shouldn't it be simply `return SimpleHistory::get_instance();` ?


Changed.


This violates the setting "Attach the diff file to commit email." that you did not remove consequently, doesn't it? At least it significantly changes it's semantic to "Additionally attach the diff file to commit email."


Oops! Looks like I misspelled "canola" wrong!


Jackson is no longer used and it is removed in a later commit (merge one, sorry)


Exceptions..?


break without random_token
-----------------------------------

`def generate_token
      token = loop do
        puts "Inside loop"
        random_token = SecureRandom.urlsafe_base64(nil, false)
        puts "#{random_token}"
        break unless AuthenticationToken.exists?(token: random_token)
      end
      puts "final token - #{token}"
      return token
    end`

Output
------------

[1] pry(main)> AuthenticationToken.generate_token
Inside loop
Fnt3umwOPeJiUf2dyxxZ3w
  AuthenticationToken Exists (0.2ms)  SELECT  1 AS one FROM `authentication_tokens` WHERE `authentication_tokens`.`token` = 'Fnt3umwOPeJiUf2dyxxZ3w' LIMIT 1
final token -
=> nil

break with random_token
-----------------------------------

`def generate_token
      token = loop do
        puts "Inside loop"
        random_token = SecureRandom.urlsafe_base64(nil, false)
        puts "#{random_token}"
        break random_token unless AuthenticationToken.exists?(token: random_token)
      end
      puts "final token - #{token}"
      return token
    end`

Output
-----------

[1] pry(main)> AuthenticationToken.generate_token
Inside loop
8v2aqx2aVY7f_8lid36euw
  AuthenticationToken Exists (13.3ms)  SELECT  1 AS one FROM `authentication_tokens` WHERE `authentication_tokens`.`token` = '8v2aqx2aVY7f_8lid36euw' LIMIT 1
final token - 8v2aqx2aVY7f_8lid36euw
=> "8v2aqx2aVY7f_8lid36euw"


nice idea!


What happens if a link is strong? Or if strong text is used outside of a paragraph text? Or in a sidebar? I generally recommend that we shouldn't be setting colors for strong and emphasis unless that's how we're offsetting them; they should flow from the content.


Your whitespace is a little bit off. I think you've used tabs instead of spaces.


`System.out.println(jen_config)` might be removed.


This requires updating the setting `STATICI18N_ROOT` to use `STATIC_ROOT` and may be dependent on John's work to update our static file handling to today's Django locations.

Also, the new version has my `i18ninline`, so our helper in `core/helpers.py` could be updated to use that instead of our own implementation.


Hahaha. This class was taken from opennaas, where you (and me) already appeared as authors ;)


ICU가 International Components for Unicode  가 아닐까 싶은데(검색후 추정입니다.) 그러면 "impact of including icu"를 icu를 사용할 때의 영향 혹은 icu를 포함할 때의 영향으로 번역하는게 좋지 않을까요?


The call_user_func is doing a static call in the background, so if i try the dynamic call maybe it will :bomb:


Sorry about that.  I accidentally compiled on the wrong platform.  Is there anything I need to do to fix it?


May be replace by diamond notation: `Configuration<MavenRepoDriverModuleConfig> config = new Configuration<>();`


right, i will change that.


As mentioned before, the return value is not "success", but a flag indicating that the message was internal communication of the OTR protocol that shouldn't be presented to user. There are three options:

 - return value true, `newmessage` irrelevant → don't process the message further
 - return value false, `newmessage == NULL` → non-OTR message, process as plaintext
 - return value false, `newmessage != NULL` → decrypted OTR message returned in `newmessage`, to be freed with `otrl_message_free()`. Replace message with `newmessage`.


Oops, I see, sorry :blush:


1540218-SlinkyGroundHX (mmatts) - i386-Windows-7-VisualStudio-12: OK (1629 of 1629 tests passed)

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


BTW there is some documentation how to use the js catalogs with require.js in case that helps with @openjck's work: http://django-statici18n.readthedocs.org/en/latest/faq.html#can-i-use-the-generated-catalog-with-requirejs


No, I didn't do that. Though it could certainly be done.


Problem E: Permutation Recovery


Yes @mgencur , we calculate average read/write per cache across the cluster. Were you expecting something else?


dcdv


SEPARATE LISTS WITH COMMAS!!!


That's good, what about:
"Especially if someone else modifies your component, he can be sure with your tests that he doesn't introduce an error."


:+1: Nice!


That's fine. Will update the rest of the readme file and push later today.


StyleCop will hit you... http://stylecop.soyuz5.com/SA1516.html


:+1: Filed https://bugzilla.mozilla.org/show_bug.cgi?id=1138971


Ah good to know, will add that.


Good point, Sam. Let's leave the weight change and get rid of the color


gotcha.


Take a look at the skeleton repo, comment on that if you have any objections. Since library is the default, it does not have to be explicit.


我自己當時是這麼覺得啦 因為我就只有做那三件事小事，可是上面寫得我自己都看不懂了

// 不過現在來看好像又還好了…


Hello...


@thlorenz 3.8.3 not found :)


Seems as if Django 1.6 support is only after 0.4. This would need a PyPI release https://github.com/erikrose/django-tidings/compare/0.4...master


Shouldn't we use `System.lineSeparator()`? http://docs.oracle.com/javase/8/docs/api/java/lang/System.html#lineSeparator--


… `forcing **eq.js** to rescan`


not sure if this converter is really needed


... version major mismatch here... will open an issue ticket later if this isn't addressed yet.


Not sure why this was renamed either.


okay so system imports before `sys.path.insert(0, ".")` and user imports after that statement?


it cuts off the last part of the path. As you did before, but I hope/guess a bit more efficient.


I am fine with doing it that way. I was still seeing the new location get
replaced by the old location when testing yesterday.

Do you think it should be the host of the game, Player(1), that decides
or should both(all) players have it configured at the bottom of the map? I
am inclined to go with the first but a, fine with the second, this was
Schwenkgotts request...

//FC

On Tue, Mar 3, 2015 at 9:51 AM, ACG8 <notifications@github.com> wrote:

> No, I didn't do that. Though it could certainly be done.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sIKE23/Mage-Wars/commit/a6dc537c1b363dbf73eb6dd779df554743dc07b4#commitcomment-10009797>
> .
>


Aha, but why sum? I am not an admin, and do not have the admin background but average read/write per node (across all cache on that node) has a lot more meaning to me. It gives admin perspective if a certain Infinispan node is under/over performing. I don't see how usedful sum would be. Can you elaborate @mgencur ?


Just one example: Large octocopters with ample power reserve to allow continued flight after motor failure, but which employ low-C batteries not capable of sustaining continuous full-throttle operation of all 8 motors.


cool!
i got a working INI parser in javascript!
(but It only supports reading INI at present)

http://jsfiddle.net/em4egypr/1/

There are also lotsa C/C++ based libs available for that, though...
But, this is the basic gist of if you tried to implement one yourself.
I have read (and actually agree) that despite all the many options, lots are bloated, or have various flaws, etc... and it's maybe better to re-invent the wheel for your specific case... (I wouldn't normally agree with "re-invent the wheel", but seriously... INI isn't a complex data format, how hard is it to write a complete parser?! I managed the reading and storing in memory part with one relatively straightforward function and an object with two methods, not too bad, imho... idk your thoughts on it, though)


The unreleased 0.10.2 would have tools for testing like we have as well https://github.com/jsocol/django-waffle/commit/1ba73be293108084413de7fe2e347c2bd2b5fb3f


The errors will show what's good and what's totally wrong :stuck_out_tongue:


@mythmon Does this read correctly?


Will change.
->
"You should derive your test suite from `unittest.Test` to have access to the `setUp()` and `tearDown()` functions (covered in section **`setUp()` and `tearDown()`** below) and also to the assertion functions."


convenient theme change。 ：）


author is diana, commited by me by mistake, was fixed


Will add.


As said, this is just for my taste, and I don't adhere to its presence. I'll take a look.


Pour en finir avec les tags, cette PR corrige les oublis de la PR précédente. Du coup, le code est un chouilla moins lourd.

Avant :

    $tags = array(
        array('tag' => 'PHP', 'nb' => 2),
        array('tag' => 'sciences' => 24),
    );

Après :

    $tags = array(
        'PHP' => 2,
        'sciences' => 24,
    );


Sorry, guess this didn't get to you fast enough. Please remove this whole paragraph.


"피하기위해" -> 피하고자

> 1.4.0 버전은 태그 되고 빌드되긴 했지만 릴리스하지는 않았다. 도중 libuv 버그가 발견되어 릴리스가 취소되었기 때문이다.

-> 1.4.0버전은 태깅하고 빌드는 했지만 릴리스하지는 않았다. 작업도중 libuv 버그를 발견해서 릴리즈를 취소했다.

수동태 표현이 어색하게 느껴지는 것 같아요.


yes. Oh and you probably want to mention coverage somewhere.


thanks. I meant add the documentation here but its Okay, I/we can add that later.

/*
  * @extends geo.pointFeature
  * @returns {geo.gl.pointFeature}
  */


okay so what does `assertIn()` concretely? you can check for an item in a list?


No problem.  Yeah, `typeid` is used at runtime only, nothing from that gets stored.  It's just used to find out if a version number has been stored already for the given type so that it's not necessary to store duplicate version tags for the same type.


@erikrose ? I know that guy! @erikrose - got some time to make a PyPI release of django-tidings? @jezdez - is your super-awesome PyPI release script online somewhere we can try it out? :smile:


right, and mentioning the cause so people will know when they get that error due to wrongly ordered imports


とりあえず練習っていうのはわかるけどHTMLとCSSの基礎はもう抑えてあるっぽいから、jQuery使って適当なプラグインでカレンダーを生成したほうがいいかも。


How is the case expression intended to work? Shouldn't be a single expression bound to a list of cases and their resulting expressions?


oooohhhh


Your patch does modify the expected output. It does REMOVE the leading 0.
That's why i didn't use the %32s in the first place.

Example:
0baad88c3c6cdda8e4ea45fa27f0b4f0 master
 baad88c3c6cdda8e4ea45fa27f0b4f0 with this patch


Okay, I moved in my own style to make *paths* italic and only code and shell commands with `code highlighting`.
Will change that.


showRaw: I want a string representation to show the user
toSeq: noted


Completely forgot about this :P


> Would it be better to avoid creating the RubyString?

It probably would be faster yes, but unless we have a benchmark to exercise the code and check how fast it is, we're working in the dark. Our approach is to write everything we can in Ruby. If for some reason that's more effort than it's worth, or if we know it's hot and the Ruby code isn't good enough, and we have a benchmark to prove that, we'll write it in Java.

You've implemented this method to get more specs passing, not because we have a benchmark, so we should default to using Ruby.

I think we already have `Symbol#to_proc` in Java, and I think it's exercised by benchmarks.


I hope you manually wrote all getters and setters :)


Sorry for the noise. I was apparently using an older version of `Hal.php`.
I totally missed that it was already there in the repo.


python has the `in` operator so you can do `if elem in list`, assertIn asserts that `elem in list` evaluates to true


Are you sure it should be if(activity != null) instead of if(activity == null) ?


The motivation for this change was that the original change from `Bin` to `HistBin` broke 6 algorithms in `Globals->abbreviations`.  I asked him to do this in a pull request of yours @bklare.


nice repo.


I h4c|<3d y0u h3r3.


can't we just add `bluebird` as a dep?


@mmostajab Ok, I haven't succeeded with SDK rendering in this integration. And since the SDK rendering relies on driver injection it is not possible (at least as I understand it) to use gDebugger, since gDebugger uses its own driver injection.


We're using very little from that package (`commonware.middleware.NoVarySessionMiddleware`, https://github.com/jsocol/commonware/blob/master/commonware/session/middleware.py) and some logger helper that we can easily replace with something of our own. We should remove the dependency and ship the middleware ourselves.


response.should make a cup of coffee


hey, hey, just a light jab :)


ResultsSchema (sanyalj) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Even if we don't end up supporting Boost-like archives: versioning is still a good idea. I'll make a new issue.


Let's hope to get rid of it soon


I suggest using FeatureForbidden function here


:+1: thanks


now you can hack the whole test env :feelsgood:


Hi,

Could you tell me if you have an index "product_manufacturer" on the table ps_product ?

Thanks and regards,


Pls don't use "rpm" to install packages

Use yum to install "local" packages, automatically checking/satisfying dependencies.  Maybe use --nogpgcheck, too.
yum --nogpgcheck localinstall packagename.arch.rpm


Unfortunately, `Promise` is required deeper in the dependency chain (I think it's `co`) so it needed to be loaded as a global/polyfill. :(


I meant sum across all caches on that single node. If you have 100 caches defined on that node and you only write/read to a single one, you still divide the number by 100 (existingCacheCounter) in the end which gives a totally different number. I did not mean sum of cache across different nodes.


Already fixed on https://github.com/ekho/capistrano-git-submodule-strategy/commit/c633a7af6b29a8cab0be487ca8e68f6bb6c28e16


LGTM


Looks pretty good, however I found some minor things to fix. Nevertheless - great job! Hopefully we'll have all the integration tests up and running after merging it.


Yes the expression in `case expression of [clauses]` I've totally missed.

Question: How should lifting pattern matching functions with multiple arguments? Implement tuples?


1. emit된다.는 "발생한다"로 표현해도 될것 같습니다. 이해를 돕기 위해서 "unhandledRejection 이벤트는"을 뒤로 붙혀서 "않았을 때 process에서 unhandledRejection 이벤트가 발생한다" 정도도 괜찮아 보입니다.

2. "이벤트 루프의 턴"이 번역해 놓으면 이해하기가 좀 애매해 보이는데 "이벤트 루프에서 차례가 돌아왔을 때"는 어떨까요?

3. "이벤트 루프의 턴에"가 "promise가 거부되었지만" 앞에 붙어야 할것 같습니다.

4. "에러 핸들러가 Promise에 부착되지 않았을 때" -> "Promise에 연결된 에러 핸들러가 없을 때"는 어떨가요?


Why did this change when it doesn't look like any test assertions changed?


OK. Thanks


Preporuka: prva linija commita bi trebala biti u imperativu. Bolja commit poruka bi bila "Style text".

Taakođer bi bilo super da si dodao u opis commita ili u opis pull request-a na koji se issue ovaj pull request odnosi (npr. "Closes #7").


Oops.  Thanks!


r+


:smile:


@groovecoder no, please don't crash the surprise party ;)


This won't work, @Aevin1387. Check my build pack for the new binary :)


Is this variable used anywhere?


saw approval from SimonSapin
at https://github.com/servo/servo/commit/a82fc00806edf63672a660827117c743ef03c63f


merging servo/servo/overflow-xy = a82fc008 into auto


servo/servo/overflow-xy = a82fc008 merged ok, testing candidate = 6fcc02e9


@kalashnikovisme  можно, я в trash контроллере так хуйнул оо


"tls.connect()의 기본 소켓"이라기 보다는 "tls.connect()를 사용할 때 기반 소켓"으로 일반 스트림을 사용할 수 있다는 의미가 아닐가요?
underlying socket은  의존환경이나 기반환경(여기서는 OS의 소켓)을 의미한다고 생각되서(이게 맞다면)

"tls.connect()에서 의존환경의 소켓으로 일반 스트림을 사용할 수 있다"는 어떨까요?


What prompted moving the client.EndPipe(); from the finally to this area?


Is it worth including an error message here? We might be able to search the logs even if we're getting different exceptions.


@benbc I would imagine `int defaultClusterNameHash=123456;`


클라이언트가 http.ClientRequest를 중단했을 때 새로운 abort 이벤트가 발생한다.

로 주어를 바꾸는게 자연스러워 보입니다.


Should error count be incremented here?


This same style appears on [line 262](https://github.com/brianjmiller/TinCanPHP/blob/e583e8fed91f40354ecd936d30c2a7ff3fb74cd1/tests/RemoteLRSTest.php#L262) so should be edited there too to match.

Should have made that clearer in my original comment.


I'm not a big fan of this method, and also the resulting implementation below. The whole point of the `ResponseEntity` and its builder is to be opinionated and convenient about adding headers; this method is neither as it accepts a String array for additional Cache-Control headers. The fact that I have to read the javadoc just to understand what will happen with the `cacheSeconds` parameter does not help either.

Can't we split this single method into multiple methods, each focussing on one particular typical `Cache-Control` value? For instance, we could have:
```java
B cacheControlMaxAge(long maxAge)
B cacheControlNoCache()
B cacheControlPublic()
...
```
So that we can do something like:
```java
ResponseEntity.status(HttpStatus.OK)
    .cacheControlMaxAge(360)
    .cacheControlPublic()
    .eTag("deadb33f8badf00d")
    .body(entity);
```

I don't think we need to support all possible Cache-Control header values, just the common ones, though I have no idea which are common.


I like the margin-top value. Good work!


Gotcha - and yes, understood on the multi-sheet if the information can differ per-sheet :)


I can see the where you're coming from, IIRC Google listed this coding style ("no non-const references as function arguments") as part of their C++ guide once, but I cannot find it anymore. The standard argument against that style is that such an API makes often only sense for reasonable semantics for the `nullptr` case, otherwise you'd always add something along the lines of `assert(x != nullptr)` unnecessarily.

I find symmetric and intuitive APIs trump non-const reference arguments, but since this is an implementation detail in your case, it's probably not much of a concern. If you let `save` also take a pointer, you get both under one hat.


FYI Seems as if the requirements/prod.txt file isn't accurate, we're not using 0.17.0 right now but 0.18.0


Git is weird.


ResultsSchema (sanyalj) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


I don't understand. The energy required to hover is about the same regardless of 7 or 8 motors.
does anyone want a forced land in sea/people/city in case of higher-than-normal current ? - a downwind or strong turbulence can easily increase total current more than a motor failure, it does not sound safe to reduce altitude to save a under-rated battery pack.
Can we have a word from the author of the idea/code?


Difference https://github.com/pennersr/django-allauth/compare/0.18.0...0.19.1


maybe install both packages in one yum call... yum -y install puppet epel-release...?


Just before somebody is going to ask: If you have objects inside here it would lead to a raw fatal error (at least it did for me here) and even if not, it will cause a HUGE response in some cases.


다른데에서도 몇번 본 단어인데, underlying이 그냥 로직뒤 정도로도 많이 쓰이더라구요. 첫번째 해석 "tls.connect()를 사용할 때 기반 소켓" 으로 하겠습니다.


Those are all consistent with the other places that we send statement so I applied principle of least change there. In other words the only time the characters are grouped is when we are calling new on a new object as part of passing an array of statement parts.


@ruipenso It's definitely on npm. Please open an issue rather than commenting on commits if you're still having trouble.


На мой взгляд, все эти категории стоило бы поместить в отдельный заголовочный файл (или вообще отказаться от этой идеи).


Вот это включение мне кажется совсем лишним.


Why do we need to check on length of input ?  Forget that we have done it for the past 15 years, why do we need to do it from this day forward ?


> You are right about the possible incorrect behaviour but I don't see any
> nice way to fix this - we could pass the list of files we get when parsing
> the commandline to osx.c (the callback happens always after our parameter
> processing) and then check in the callback if we have already opened them.
>
> Well, maybe then resolve relative paths and open everything (twice)?
>
Yeah, good idea. This however means that Geany mustn't change its working
directory which is _almost_ true:

1. There's one chdir() call in some Windows-specific code - not sure if
it's needed but won't be used in the osx code anyway.
2. But there are 2 rather wild-looking chdir() calls in
vte_get_working_directory() - I think the code is supposed to just find the
shell's working directory but not change Geany's working directory
(probably caused by copy/pasting without thinking what it causes in Geany).

The signal will probably be fired before anyway but I think (2) should be
fixed.


It is unclear what parts in this 4.2 XSD are actually new;  it probably would have been better to make the 4.1->4.2 migration/copy part of a separate pull request, and base this PR on that. Too late now, I guess :).


30411/Danciu P. Diana Lacramioara/Assignment 1/main.c

What happens if in the first if statement, you insert "tail = head" as the last statement. Would this branch (tail == NULL) be necessary?


breaking ABI change -> 이거 API 의 오타 아닐까요? 원문 오타같은데요.. ㅎ

"hold back" 이 막다, 감추다 등의 의미이니 "호환성이 깨진 API 변경사항은 이번 업드레이드에는 포함되지 않았지만 io.js에 V8 4.2를 합칠 때 포함할 수도 있다" 가 되지 않을가 합니다.

금수조치된은 그냥 금지된으로 번역하면 의미가 와전되려나요?


same as above


Why don't you use caret version like ^1.0.0 ?


https://github.com/praekelt/django-recaptcha/compare/0.0.9...1.0.3?w=0#diff-169615d05819ca2858906e2eb9d809f3R66 leads to the recaptcha widget to show up localized now FTR


Thanks for that


Я бы предпочел увидеть еще и категорию над NSError, а не просто enum.


develop (jasondegraw) - 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)


If the function returns here, then the CifsScanButton remains disabled.


@staticfloat I changed the logic in 34a78d4640ea29abf34f based on this discussion, I think we should try to test this if possible as our coverage for error code paths is really bad.


Lib's using `otrl_`, I have used `otrp_` for "OTR Plugin". Other options were namespacing with `ekg_` or `ekg2_` (which seems more appropriate to EKG2's core functions rather than plugins), or using `ekg2_otr_plugin_` prefix (which would be impractical). `otrp_` seemed like a good compromise, but replacing it is just a sed call away if you think a different prefix would be better style.


Would you be opposed to naming the option 'estimatedSize' instead to match [the specification](http://www.w3.org/TR/webdatabase/#databases)?

Sorry so slow to respond.  I happened to run into a need for this with an iOS Chrome bug about 2 weeks ago.


"새로운 레지스트리와 `npm@3`의 준비를 위한 기능이 포함된다." -> "새로운 저장소와 npm@3를 준비하는 기능이 포함된다.


LGTM !


На мой взгляд, логичнее было бы сделать этот метод методом класса.


Dependencies are pinned to mitigate the "$cool-library bumped feature flag while breaking everything" case


Very well, nice and clean, however I would never say "no" to a more modular solution


Typo disabled


Nice work @FrankRuns


А кто заполнит ошибку?


Once again, no `</p>` tags in Spring Javadoc.


30411/Birladeanu N. Adrian Vasile/.changeMe!

if you were to assign tail = head in the first branch of the if statement, would this branch (tail==NULL) be necessary?


In a perfect world. The name would be "BooleanToVisibilityConverter" ;)


아 저도 그래서 찾아봤는데 아니었습니다.
Application Binary Interface인가봐요.
금지된도 괜찮을거 같아요.


I know we get the source and line number, but it's a lot clearer if we say *something* in those log messages (especially if it's a warning).


Well, Core Erlang handles this quite simply. Consider the following erlang function:

one([],[]) -> empty;
one(A,B) -> A++B.

This is what it compiles down to:

'one'/2 =
    fun (_cor1,_cor0) ->
        case <_cor1,_cor0> of
          <[],[]> when 'true' ->
              'empty'
          <A,B> when 'true' ->
              call 'erlang':'++'
                  (A, B)
        end

The <> embraces lists are not tuples according to the core erlang specification, but just a list of variables. So I guess the pattern matching should just be translated into these kinds of case clauses.


This seems not used anymore, I checked the prod settings, it's not referenced there as well


새로운 logout 명령어가 추가되어서 bearer-based와 basic-based에서 인증되는 클라이언트 모두에서 편리해졌다.

는 어떨까요?(logout은 명령어라서 번역보다는 그냥 써주는게 어떻까요?)


Published to http://ui.edev.d2l.com/47_c8f09baec419ebabb6056f5f4feef33cceee416f


oh lame :/


same as above


That's because I built the Firefox 35 component against the final beta, but I build the Firefox 36 component against the developer edition. I don't think this is the issue.

More likely, this has something to do with the Visual Studio upgrade (which was necessary because Mozilla did it). I can investigate, but our policy is generally that we don't support software that is no longer supported by its maker, and Windows XP falls into that category.


IDFXT (refaqtor) - x86_64-MacOS-10.9-clang: Build Failed

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


Oops! Sorry I missed this.


I tried what you've said and it is necessary.


this gives you a very bad execption when the cast fails somewhen.... better do this in the ctor


IDFXT (refaqtor) - i386-Windows-7-VisualStudio-12: Build Failed

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


Option four seems best. privatebindir?


peerDependency 동작이 변경되었다는 경고가 나오고 문서에도 추가된다.

라는 의미가 아닐까요?


This should already be resolved on the main tree by moving Reader.m into +types.


why you did this?


https://github.com/lmorchard/django-teamwork/commit/1c9577f674de5571e0fd00ef07f5735973f17a93 points at a non-master branch and should not be used. This should be easy to get a release up on PyPI if needed, @lmorchard?


Do you want to provide some signal that the currentBatchCount has topped out?


again, why?
You should import specific models only.


shouln't be needed. See https://github.com/michaelschnyder/dotnetbay/blob/wpf/source/DotNetBay.WPF/MainWindow.xaml.cs


Missing update


If you don't want to use the feature, don't use it.

I have an Octocopter with 8 40A ESC's, capable of drawing. 320A.  It only requires 6A each to fly, or about 48A total.  The battery is only able to supply 200A total before it becomes damaged.  I do not need more than 200A to climb or fly in a wind.

Yes, I could have undersized the ESC's.  But then in the case of a motor failure, the two adjacent motors need the headroom.

People build large Octocopters all the time that cannot sustain flight after a motor failure because they have undersized the power system.  If your hover throttle is over 60%, there's no guarantee it will fly properly with a motor failure.  And then what's the point making an Octocopter in the first place, which is more expensive, less efficient, and has two as many failure points as a quadcopter?


Why is this method protected and final? Either make it private, or make it non-final.


This feels like something worth having some detailed unit tests for.


Sorry for that. og is right.


IDFXT (refaqtor) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Can the user supply the MaxBatchCount?


I can't really confirm that this only affects Windows XP (though that seems to be most common). On that thread I see Windows 7 and Mac, but it's possible that those are unrelated.


setTrim の役割は、入力された文字列の前後の空白を除去するかどうかです。
必要に応じて true/false を切り替えるとよいとおもいます。


Don't be reluctant into giving full names for your functions. No one will know what del_f means, but everyone would if it were to be delete_first() !


1cfdddfcc1c96ca2552d0afdeb9eaff63dfe1647 is not the latest HEAD https://github.com/jsocol/bleach/compare/1cfdddfcc1c96ca2552d0afdeb9eaff63dfe1647...master


Looking at GitHub's inline code as an example, here are some differences that I'd like to see addressed:

* Margins, especially for elements that are meant to be read inline, often don't go over too well and may throw off legibility by having too much spacing on the sides
* Padding has similar problems. GitHub doesn't include left and right padding, and they only include `.2em` top and bottom padding after having reduced the font size
* Are you sure you wanted [outline](https://developer.mozilla.org/en-US/docs/Web/CSS/outline) and not [border](https://developer.mozilla.org/en-US/docs/Web/CSS/border)? In either case, are you concerned that the border/outline will have a negative effect on the legibility of the surrounding text? GitHub uses a semi-transparent background to resolve this potential issue (`rgba(0, 0, 0, .04)` to be exact)
* Same comments I've had before about setting colors apply
* Code/Pre formatted content should use a monospace font


Actually, is this really across cluster or is it again an average (something like readWrite ratio / number of nodes) ? Just curious. My test which does one put, one get and one remove in sequence, returns 0.0 for HitRatio. There are two nodes in the cluster. Is this expected?


This is already infinitely more helpful!
Ok, I've bloated the binary with some more static text :)


seems easiest to try https://github.com/jsocol/bleach/tree/v1.4.1 out?


Err, Mac is unrelated of course.


how are we storing passwords?  One way hash?


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1123 of 1123 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)


Similarly, I have a mid-size long-duration quadcopter, with very large props and small motors and ESC's rated at only 20A.  It is also capable of damaging the power system if full power is applied in a manual mode.  Otherwise though, it is very efficient.  I currently use Thr_Max of 1800 to try and prevent this, but the power limit is a better method.


NVM - I see bcrypt in the next commit.


But if we choose option four above re systembindir rename, it needs to be added here as well, since the whole purpose is to provide a platform agnostic path to the internal puppet packaged binaries.


Wait, we know the developer of that app, can we ask him to do a PyPI release?


What would we have to do to get this driver (hlcdc) working on at91sam9x5 like it does on v3.15?


"발전적인 서버로" -> "발전적인 서버 플랫폼으로"


This interface should probably be renamed, as its name currently does not fully reflect what it does. Going off the name alone, I would guess it has something to do with HTTP caching in general, but the javadoc tells me that it only deals with caching-related HTTP headers.

What about `CacheHeaderStrategy`?


Get rid of those extra spaces!

if (!$check['db']) {


at some point, we will want to refactor the route naming to match our terminology.  This will be fine for now.


is it possible that this enhancement for data-* attributes only affects <table> + <td>, but not <tr>?
your plugin is really great and i hope you can also add <tr> for this.

Thank you very much for your efforts


https://github.com/mozilla/basket-client/commit/4bc1251a3b6b3cb7ec9dfa772cbfaf677305a4e5 is v0.3.10 even though the Git tags are missing. I filed https://github.com/mozilla/basket-client/issues/28


this var is freed using other functions a few lines above, see e.g.
https://github.com/php/php-src/commit/cddde315c7e915ec9b428cadc799d9705b4d08c9#diff-3291092b024fc91e40bd9b6948adb819R282


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


Please open a new PR.  Please also include an explanation for the change in the commit message.


sorry, my tags were filtert.
i was asking for *tr* support. *table* and *td* works fine with this


Vladimir, maybe it's best if you look at my test: https://gist.github.com/mgencur/dd205d0a372858d69076 and tell whether my expectations are correct or should be different. The "mc" in the test means MemcachedClient instance. The test runs in two-node cluster, DIST, 2 owners.


this entire file will be split out into controllers soon enough.


Ok, so if you prefer having a return statement instead of using if/else if/else ... you can just use a void function and simply have a return; statement (notice, it returns nothing)


Each one of these will represent a controller in my game.


Calling our own, pre 4.2 cache header behavior 'legacy' is probably not a good idea. Perhaps `Http10CacheHeaderStrategy`? Not sure if that covers it well enough.


"채용되었기 때문에, V8의 새로운 버전은 이미 지원되고 있다." -> "도입되었으므로 새로운 버전의 V8도 이미 지원하고 있다."

"중추적인 역할을 하기 때문에," -> 중추적인 역할을 하고 있으므로

때문에가 많아서 어색한것 같아서요.


You could also do a simple face-split / Ctrl+r if it's just symbolic.
None of those patches seem to affect me.

if someone set up a proxy repo on github which pushed and pulled commits into and from the official git.blender.org / addons_contrib repo I would be totally OK with that. The github issue tracker is also a joy to work with and would get my vote any time.


Надо бы отступ.


Hopefully that will make our integration tests more reliable. :+1:


TeamCity Toolbox :: Pull requests [Build 38](http://10.48.34.86:8111/viewLog.html?buildId=4944&buildTypeId=Toolbox_PullRequests) is now running


@graemerocher fixed by 947a1bcd . EclipsePlugin in Gradle uses "testRuntime" by default too. The Nebula Extra-Configuration implementation of "provided" makes "compile" extend from "provided" so "testRuntime" includes "provided" configuration.


lacking a space in a lot of places


TeamCity Toolbox :: Pull requests [Build 38](http://10.48.34.86:8111/viewLog.html?buildId=4944&buildTypeId=Toolbox_PullRequests) outcome was **SUCCESS**
Summary: Tests passed: 161 Build time: 00:00:17


It may make sense to ask for a 1.8 compat release, https://github.com/django-nose/django-nose/compare/v1.3...master


r+


"Node.js 재단" 여기에 공백이 2개 들어갔습니다. ^^;;


extIndex = 0
Да и объявление можно вверх


Yes, that was intended; I also had `Http10CacheHeaderStrategy` in mind, but I wasn't sure it was the right one since this strategy also acts as way to reproduce old behavior. I'll probably go with `Http10CacheHeaderStrategy` anyway.


I know, right? It's sad. Did you see what I had to do?

Here's what I had to do just to get the I.D. of a block:
```java
net.minecraft.block.Block.getIdFromBlock(location.getWorld().MC().getBlockState(location.toBlockPos()).getBlock())
```
...and to get the data value of a block:
```java
location.getWorld().MC().getBlockState(location.toBlockPos()).getBlock().getMetaFromState(location.getWorld().MC().getBlockState(location.toBlockPos()))
```

That's all *after* I made and/or revised some utilities to help. Frustratingly, the data value code segment could have been about half that length if they had made `getMetaFromState` `static`, but they didn't, even though they could have and should have. Eclipse even marked it as potentially static.

I can't wait for Corundum to be working so modders can finally have a more future-proof, rational A.P.I. (at least for server-side mods).


Название не говорящее, надо бы чего по интереснее


why not just have something like
while(aux!=null && i<=nod){
   ...
}
?


@devinus yepp, they support these too, I wasn't sure about how much different versions you want to check so I thought about the most recent and the `.0` releases should do it. Just add what you want to test ^^


"것같으면" -> "것 같으면"


`sc` is already a socket right here.. does this leak the previously allocated socceted when allocating using the same var?


Probably easier to require that all players have it configured in order for it to be displayed at the bottom.


We might as well include a link to the getting started page too.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/910
success: http://build.servo.org/builders/linux2/builds/908
success: http://build.servo.org/builders/android/builds/946
success: http://build.servo.org/builders/mac1/builds/1125
success: http://build.servo.org/builders/mac2/builds/1125
success: http://build.servo.org/builders/gonk/builds/96


Please check https://github.com/binoc-network/reactos/blob/master/cmake/gcc.cmake#L386 and our PCH support in general.

The PCH is part of the source files of the target, it gets compiled thanks to HEADER_FILE_ONLY FALSE, which we can replace by your idea of having the pch language property set to CXXPCH...etc, and it doesn't get linked thanks to EXTERNAL_SOURCE TRUE

This way we don't need 900+ extra targets, and we make the source files list ( _sources in https://github.com/binoc-network/reactos/blob/master/cmake/gcc.cmake#L373 ) OBJECT_DEPENDS on the compiled PCH

Here's an example of how the use is simplified with natural exclusions of source files from using the PCH: https://github.com/binoc-network/reactos/blob/master/dll/shellext/netshell/CMakeLists.txt#L36 and https://github.com/binoc-network/reactos/blob/master/dll/shellext/netshell/CMakeLists.txt#L17


Added for training purpose


Even though you may not consider it as being terribly important, variable/function naming is terribly important so work on it a bit harder!
Good job otherwise


"튜토리얼"이나 "튜토리얼을 출간"에 링크거는 게 더 자연스럽지 않을까요?


No 0x in devicetree reg address  (after the @ ).


Can we please get rid of these obsolete comments?


thats a good commit :) great


This should not be a javadoc comment, `/*` suffices.


Wait I pasted wrong, the Core Erlang should be

'one'/2 =
fun (_cor1,_cor0) ->
case <_cor1,_cor0> of
<[],[]> when 'true' ->
'empty'
<A,B> when 'true' ->
call 'erlang':'++'
(A, B)
end


I find null pointer derefs are one of the easiest things to debug and fix, so crashing is usually "reasonable semantics" in my book :)


Use lowercase for hex values, so:  0000c000


These need to be replaced with the CIFS specific errors. But for this we need to update the model with the new api bindings.


Надо бы всю секцию в отдельный метод ибо длинновато и не соответствует принципу единственной ответственности


The second example, limit is for total current, not per motor, so undersized ESC will still be overloaded - or am I missing the point ?

In the first example,  I think you are saying you would set it to max 200A , which should give plenty of power reserves, and the setting is there only to prevent very high manual throttle input ?   (as an alternative to limit by PWM)


Is this really better than `auto const& section = config.section ("sqdb");`?


Ooooh, that's a good catch. That reminds me that we should use the `memcached_hashring.backend.MemcachedHashRingCache` class for this instead to use consistent hashing when using python-memcached.


Itt a buzzwizz kellet volna, de nem gond.
(Igen ez nem volt explicite meghatározva)


@50Wliu looks like the account of the author of that PR has been marked as spam which is why it is no longer visible, I'm looking into it now.


Then I'd argue that Genome::Sys::Lock shouldn't die there.

I could see throwing an exception if asked to lock in a scope that has no backends (though silently doing nothing also has merit).  But during release_all(), there's no sense throwing an exception when a scope has no backends and no locks.

It would also have the benefit that the custom code would all be in the SGMS site-specific module, and Genome/Sys/Lock.pm could stay more generic.


I don't see the difference?


If `get(section, keyName, "sqlite")` doesn't compile perhaps we can improve it?


Markdown does not like the <> brackets...

'one'/2 =
fun (_cor1,_cor0) ->
case <_cor1,_cor0> of
<[],[]> when 'true' ->
'empty'
&ltA,B&gt when 'true' ->
call 'erlang':'++'
(A, B)
end


void addFirst(int code)
{
    NodeT *elem;
    if (head == NULL)
    {
        head = (NodeT *)malloc(sizeof(NodeT));
        head->data = code;
        head->next = NULL;
		tail = head;
    }
    else
    {
        elem = (NodeT *)(malloc (sizeof(NodeT)));
        elem->data = code;
        elem->next = head;
        head = elem;        //now this becomes the head of the list
    }
}

Try this out


fast-forwarding master to auto = 6fcc02e9


Az execute-ből elmaradt az érték :)


В константу можно или вообще в конфиг для гибкости, вдруг юзер точно знает что он хочет


Tab stops are at 4 spaces please


@kruppel Thanks for trying to fix the tests! Could you try to do this on another branch off this one, in case I have uncommited changes or unpushed commits...?


Liquid lets you add any type of front matter you want, and name your front-matter variables however you wish. So that was just a convention for adding any type of meta information to the front matter of a page, and if it exists it will print there.

e.g.:

```
---
layout: default
author: jglovier
meta: "Notes On Design Series"
---
```


Use triple backticks for code

```
'one'/2 =
fun (_cor1,_cor0) ->
  case <_cor1,_cor0> of
    <[],[]> when 'true' ->
      'empty'
    <A,B> when 'true' ->
      call 'erlang':'++'
        (A, B)
  end
```


I will try to merge this today.


Просится в отдельный метод, в основном из-за наличия вложенности внутри секции


remove blank line


saw approval from SimonSapin
at https://github.com/servo/servo/commit/50800940dc5c8cf424d5786c447f7ff1d0432a97


`^` is the worst thing ever (not an overstatement).


merging servo/servo/counters = 50800940 into auto


servo/servo/counters = 50800940 merged ok, testing candidate = 4fa72166


geiler punkt!


1540218-SlinkyGroundHX (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1629 of 1629 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)


Unnecessary spacing detected.


Here yes. But I tried the first time on addLast and it didn't work.


hex numbers are usually lowercase


Good work.


This is still a big jump from 2.38. to 2.4.4 (https://github.com/tomchristie/django-rest-framework/compare/2.3.8...2.4.4). I fear this requires going through all releases notes up until 2.3.8: http://www.django-rest-framework.org/topics/release-notes/#24x-series


I  think mean "needs to be added to `puppetbindir`" below?


Hmmm... I think this qualifies as trolling on Saturday morning personal projects. :smiley:

I know that the grammar is ambiguous. It'll be corrected when I actually implement these expressions.


Ilyen nem kéne.

Lehetséges kimenetek:
Fizz - osztható 3-al vagy tartalmaz 3-ast
Buzz - osztható 5-el vagy tartalmaz 5-öst
Wizz - osztható 7-el vagy tartalmaz 7-est
FizzBuzz - igaz a Fizz és a Buzz szabály
FizzWizz - igaz a Fizz és a Wizz szabály
BuzzWizz - igaz a Buzz és a Wizz szabály
FizzBuzzWizz - igaz a Fizz, Buzz és a Wizz szabály


sure can do that.


ok, we should fight it with:
1. common region (data shared between all dbs) - admin level, country
2. data specific to db
2.a) remove everything related to this db (not all data in tables but only specific to this db)
2.b) insert data specific to db
other ideas?


Doesn't need to be a javadoc comment, `/*` suffices


The contents of this file could simply be added to SociDB.cpp


Yeah, still rocking and rolling - starting to teach individual workshops and after school things soon. Let's have a `chaps` reunion sooner rather than later!


test


@vysheng I've done a lot of testing and this is the last commit that sends documents/video/audio correctly.

Reference:
https://github.com/vysheng/tg/issues/401
https://github.com/vysheng/tg/issues/428


What the...? We're allowing callers to modify the vector? This looks suspicious


Are these free functions? Why isn't the function scope definition on the same line? Why is there a semicolon at the end? This is confusing


Please remove this obsolete comment


Want to help out in other ways


Ok, the bug was due to a couple of things.
- Mainly the YARP_PROTOCOL_VERSION was introduced and implemented in the remoteControlBoard & controlBoardWrapper2 by @lornat75  but not in the old controlboard (the one used in the test). It was off by default and in this commit I turn it on because the change introduced by this commit needed un upgrade of the protocol version.
- The open of the controlboard was failing because it has no protocol version, but the test was going on anyway and therefore crashed.

We add the parameter ignoreProtocolCheck in the configuration of the test so the remoteControlBoard is fine without the check... just prints the warning message.


You only need beast's memory.h if you're calling `std::make_unique`, otherwise the regular `<memory>` is fine.


Canola is misspelled.


Здесь будет логично сделать зависимость ProcessMonitor от Settings, передавать в конструктор Settings, а не Timespan и long


`detail` namespaces are only useful in header files.


I don't have a legal opinion on this as I don't fully understand the consequences, so I'll just share a few thoughts with no strong view on it.

Taking that into account, and as said on IRC, the ideal would be having a license declaration in a single place and have it effect over the entire codebase. But that depends on the license requirements, and GPL advises to use the copyright statement on every single source file. Is that the same as requiring? Probably not, but I cannot say if it has any legal effects or not.

Having said that, you rarely see in real world templates and CSS with license declarations — I'm not saying they should or shouldn't, though.

In practical terms, the biggest caveat of having licenses in CSS is that you need to ensure to remove it when creating bundles that will be served to client, or you'd be wasting precious bandwidth. Generally speaking, it's also more difficult to switch to a different license because you need to modify every single file as in this PR.


Myaccount should be MyAccount


r+


Nemuzeme pouzivat dvoje konce radku, to bude rozbijet diffy.

-----Original Message-----
From: Lukas Jelinek
Sent: 3.3.2015 16:06
To: hrnr/robo-rescue-simulation-vrep
Cc: Jiri Horner
Subject: Re: [robo-rescue-simulation-vrep] Change of output topic format forAccelerometer and Gyroscope (8fb400d)

neexistuju zle konce riadkov jednoducho su len konce riadkov :D Navise toto bude ten najmensi problem s ktorym sa v mojom kode stretnes :D

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


"to callback only on"


Results: (Note that for this flight, the MAV is standing still after 380 seconds)
* Altitude: the estimation follows again the baro and not the GPS
![kalman_5d_alt](https://cloud.githubusercontent.com/assets/6390059/6467064/265971d6-c1cc-11e4-9bcf-0c8df05b8904.png)
* Height: the estimation is going below zero pushed by the baro.
![kalman_5d_height](https://cloud.githubusercontent.com/assets/6390059/6467092/60e9acf8-c1cc-11e4-8b44-7e2b1ca9f4e8.png)
* Baro offset: drifts! Doesn't converge in this flight, especially when standing still
![kalman_5d_baro_biais_drift_time](https://cloud.githubusercontent.com/assets/6390059/6467199/f921675e-c1cc-11e4-9c81-e8080cb22f04.png)
* Accelero offset: One order of magnitude below the acceleration, converges to zero when standing still
![kalman_5d_acc_biais](https://cloud.githubusercontent.com/assets/6390059/6467204/03c58816-c1cd-11e4-978a-0a04c0dc832f.png)
* Covariance: linked to the healthiness of sonar
![kalman_5d_covariance](https://cloud.githubusercontent.com/assets/6390059/6467229/1f04fd00-c1cd-11e4-98f1-a3970921df3a.png)
* GPS uncertainties: values smaller than thought
![kalman_5d_gps_uncertainty](https://cloud.githubusercontent.com/assets/6390059/6467272/70c4203a-c1cd-11e4-85d5-1837929de07a.png)


Indeed.


board[-1][-1] would cause uncaught error, put into try-catch block should solve the problem here


I meant that if we add a 'privatebindir' or similar to replace systembindir in host/unix.rb and provide for how we are currently using puppetbindir in some of the tests, then we need to add a privatebindir in host/windows.rb, otherwise the test is still having to inspect the host type to pick a path.


I will do it now unless somebody stops me. :-)


saw approval from SimonSapin
at https://github.com/servo/servo/commit/e209f5e0da5a3dd3b188d11e884b2859daf053d3


merging servo/servo/counters = e209f5e0 into auto


servo/servo/counters = e209f5e0 merged ok, testing candidate = 56a21e5f


"Instead of 19 callbacks"


"you get 3 callbacks".


Создавать Settings через контейнер в этом случае нет никакого смысла. Здесь ты должен просто создать один экземпляр Settings и указать контейнеру всегда использовать его (ToConstant)


good point, I don't need the param here anymore


I'm going to call it 1.0 as well, because…come on.


"when the Edit Type change,"


right, all_server_resources will be probably the best


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (623 of 623 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-3.67%25-green.svg)


I did a PR #2897


@erikrose Don't fear the 1.0, just the first step to 42.0.


I have moved the try to the first line of the method and removed #154


I'm liking this a lot. This might be really really really useful in PRISM developments.

#openSOURCEisNOTstealing


Это тот самый знак что нужно сделать еще 1 файл :)


Please send upstream for review.


I want to make the timeline of the Nam June Paik (July 20, 1932 – January 29, 2006) who was a Korean American artist. He worked with a variety of media and is considered to be the founder of video art. He did television projects, installations, performances, collaborations but He is very famous for his vedio art. so, I thought his work can be represent well on the web because he enjoyed to use videos and lights in his works. also, he is one of my favorite artists. I am not sure that I can do this or not but I want to try his experiment of media on my website.


Follow the same logic, it should work there as well


This won't handle unicode in Python 2.x because str==bytes there - is it intentional?

Also, there is `isinstance(foo, (type1, type2, ..))` form which is a shorter version of `isinstance(foo, type1) or isinstance(foo, type2) or ...`


As already intimated I find both the summary name and the doc for this function not very understandable. I have used a similar function a lot in my programs, and it is nice to see it built in. Basically I understand it to give the ratio to apply to the frequency generated by a midi note to change it to sound n semitones higher. Thus pitch_ratio 12 => 2, This means that a relative midi shift of 12 gives a frequency  ratio of 2. i.e. the frequency is doubled or up one octave.
The function is great to use with the rate of a sample to let you change it's sounding frequency.
Thus to sound a sample 7 semitones (a musical 5th) higher use
 sample <samplename> , rate:  pitch_ratio 7

I'm happy to put in a pull request, but I'm not sure how to express the above succinctly and briefly.

It is a great function and lets you define a "play" function for a sample to play any note you like.
eg

inst=:ambi_glass_rub
samplepitch=:fs5 #find by experiment listening to the sample and compare to synth playing this note
define :pl do |nv,dv|
  shift=note(nv)-note(samplepitch)
  sample inst,rate: (pitch_ratio shift),sustain: 0.8*dv,release: dv
end

pl(:c5,0.2)

plays the sample at pitch :c2 for 0.2 seconds

Comments?


Results: (Note that after 180 seconds, the MAV is standing still)
* Altitude: Here I had an issue with the offset, still have to figure out what happened
![kalman_4d_alt_with_gps](https://cloud.githubusercontent.com/assets/6390059/6467327/b3dc199a-c1cd-11e4-85e4-b6c068425221.png)
* Height: goes below zero when hard landing
![kalman_4d_height_flight](https://cloud.githubusercontent.com/assets/6390059/6467343/c6d0d5d6-c1cd-11e4-9cd3-e09efa66dcf9.png)
Large offset when standing
![kalman_4d_height_long_run](https://cloud.githubusercontent.com/assets/6390059/6467355/e6b5e9e0-c1cd-11e4-8a8c-ba5643cc0241.png)
* Accelero biais: one order of magnitude below the accelerometer values
![kalman_4d_acc_biais](https://cloud.githubusercontent.com/assets/6390059/6467297/92854e9c-c1cd-11e4-8d65-df879ad8aba1.png)
* Covariance: Almost equal for altitude, height and speed.
![kalman_4d_covariance](https://cloud.githubusercontent.com/assets/6390059/6467376/00909ae0-c1ce-11e4-9521-7c29aa2e9d5b.png)
* GPS uncertainties: Values could be used in Matrix R
![kalman_4d_gps_uncertainty](https://cloud.githubusercontent.com/assets/6390059/6467396/29b2e900-c1ce-11e4-8f5c-aaaf3b6d5bb1.png)


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1126 of 1126 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)


Also added to Transifex for future improvements
https://www.transifex.com/projects/p/cops


@winhamwr excuse me for typing these in the commit. Do they make sense to you?


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


No, commands don't return exit code but Job*


Wow. I thought they'd made some things harder, but **getting a block at a certain coordinate** takes two method calls, compared to the old single method we had back in 1.7.10. I mean, come on, Mojang. At least have a helper method that get's the block from the block state and returns that u.u. Holy Arceus' golden ring. I'm going


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


Maybe `if not isinstance(values, (list, tuple))  # set, frozenset?` check is enough? What do you think?


The same problems here, solved by using QT_NO_KEYWORDS and some sed magic ;-)

The culprit is "namespace foreach" in foreach.cpp. This will be clobbered by the preprocessor if the Qt "foreach" macro is active. The same happens in Boost.Signals with the "signals" macro.

Blame Qt for introducing "pseudo keywords".


Nagyon vad ez a sok if. Egyszerűbben is megfogalmazhattad volna.
Ezen még javítunk. :)


The y coordinate of the right bar was calculated incorrectly, I guess a screen positioned at a y!=0 with a right-side gap has never been tested. I've repushed my commits with an added test that should demonstrate all this.


Gambaso gordo. Thanks for fixing it :) And adding the settings to the `test_settings.py`


But what if we run out of integers?!


[v0.0.3 is on PyPi now](https://pypi.python.org/pypi/django-teamwork/0.0.3) - you probably want to use that and not track master. I'm actually in the middle of a pretty big overhaul of that project, and it will cause breaking changes.


Note, `GroupBy_Sum_min_Max_Avg` is failing on CI, assigned to you.


Возможно стоит поменять имя


then we're doooooooomed


I know there are too much lines, but those functions will go into a librabry, for the next assignments.


Actually i 'd rather keep it info. We don't expect to see so many of those and being able to check that trace was printed will/could become very valuable in multi-node when people start to make config change...


Can we please abandon this obsolete comment?


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


Идеально было бы код от сюда до низа упаковать в метод


Is this a typo? _get_reponse? Not _get_response?


r+


I think my wording is weird, I meant CIDA, the OWI, and the USGS as the three groups.  I'll adjust.


I'm not 100% sure this belongs here. What if someone includes core/soci.h directly? You might consider putting it in src/BeastConfig.h


That's a very good point. I'll add the check.


how can I 'like' this comment? :+1:


These are constant expressions appearing in a conditional they will likely produce warnings. Even so, its bad form to write `if (true`...


érdemes:
if (kifejezés) {
   // do
}
módon if-eket képezni, még akkor is ha egy soros. Apple-nél volt néhány hónapja, vagy talán egy éve, hogy ennek hiányából származott egy bug.


Also handled in my webtests_cleanup branch :)


That's how they were. Just wanted to have a quick way to enable those properties if needed. Now we can discuss about changing the current defaults, i don't have string feelings about it.


Why we need a separate method to garbage collect untagged images here? Why not handle all image removing by FreeSpace based on reference? Are you concerning there are a bunch of image intermediate layers without tag and won't show when list images?


Hey thanks man! Actually this repository is an output from https://github.com/ripsawridge/tech-site I'll go make the change you suggest over there.
Much appreciated,
--Michael


I'd like @joelkatz to weigh in on removing this


Whoops, wrong issue link, should be #277


`final` is pure noise here


Низкоуровневая вещь сверху, хорошо было бы спустить её в самый низ


But why leave it out completely? I was missing this info and there was no note. Maybe if you add a clear note to the instructions on the sparkfun page, people would understand it better. I am really missing this feature. Or simply provide 2 versions.


Yeah, I do think something like this is a good idea.  Should be something to add to the list of things to try!


You are totally correct! touchStatusView was not in my subviews, so it only contained UILabels, and it does crash if I add it. Interesting, the code lies! :P Type checking is a good solution.


If the fb doesn't have a kernel mapping, it can't be used with fbcon, so probably shouldn't be used for fbdev either.  Instead, I recommend implementing two gem buffer allocation routines.  One for use by fbdev, that can give kernel mappings, and the other used by drm proper, that does not use kernel mappings.
See: https://patchwork.kernel.org/patch/5608311/


https://imgur.com/a/JJhTS#0  Example of the difference made with this commit.


Damn you're right, it's zend_string *. It'll not crash but leak the actual string otherwise. Thanks.


We do naturally build up untagged images (when we download newer versions of an old image for example). We don't get much value keeping those around so this aims to regularly clean those up. The FreeSpace() we can call only when we are getting close to the disk limit and want to free some space from the useful images.


Just me being silly :) Have fun!

![](http://i.imgur.com/S1Wccs4.gif)


82420068_GridViewMultiEdit (evanweaver) - 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)


Fair enough


The handling of `.geany`s here is different than on the CLI: only the first filename is considered for opening a project, effectively opening the project's file itself when a .geany appears as later arguments.

I however don't see how this could be changed, and it doesn't look important.


The commit is in there. Feel free to remove the binary, but I adjusted the
settings so it shouldn't happen again. I still haven't looked at all your
code, but try to merge some of the branches when you get a chance.

Also, FYI. I'll be back for this weekend.

Jake
On Mar 3, 2015 10:50 AM, "nerdonator" <notifications@github.com> wrote:

> Sorry about that. I accidentally compiled on the wrong platform. Is there
> anything I need to do to fix it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/team-2449/robot-code-2015/commit/91cd670e97fe522d3b581aae85c16df9f3c846f3#commitcomment-10009778>
> .
>


Small thing... "In this case `1A`" is not a sentence.


If the order is indeed important, might be good to add a obvious comment so the line does not get shifted back up at some point. Now, i am ready to believe you but that sounds crazy!


We usually avoid the .* for import, but not such a big deal, especially for tests.


I think the cleanest way to handler this is to allocate a singleton instance of the ExceptionEntry class, and then reuse it each time we access entry:

```
getEntryByIndex(i: number): ExceptionEntry {
  ExceptionEntry.instance.seek(exception_entry_table + i * 8);
  return ExceptionEntry.instance;
}
```


just curious what is this Array to be full of?


Thanks for the review @ianturton.


tasks 2.4, 2.5


Does this Imply that we still can use @NotTransactional?


ok, so we have the same error here: https://github.com/php/php-src/commit/cddde315c7e915ec9b428cadc799d9705b4d08c9#diff-3291092b024fc91e40bd9b6948adb819R295


Looks good, Would be interesting to understand why you came up with the new 'minIdle' setting. Next time we chat. :+1:


Done! https://pypi.python.org/pypi/django-tidings/ Still fighting with uploading the new docs.


Odd, I distinctly remember inserting src   = null somewhere around here to fix an issue with fuel not spreading.

Either way this is a way better solution.


Чтобы избавиться от постоянной проверки "!muted &&" мы можем сделать метод getLogger(mute) который вернет пустую функцию если muted === true и console.log если иначе. Тогда мы можем 1 раз вызвать геттер, получить логгер и просто отправлять туда сообщение, если muted то просто ничего и не выведется, а код очистит и упростит в разы.


This commit removes the existing find widget.  The next commit renames the AdvancedFind widget to the Find widget.


This commit introduced http://jira.xwiki.org/browse/XWIKI-11876 and because of the xar formatting is very hard to read. XWiki/AdminSheet.xml changes are not displayed by github because of the big number of changes.


This commit renames the AdvancedFind widget.


Didn't notice that.  That works!


How about: https://github.com/popcornmix/xbmc/commit/759b6da3b437ce641743f443a022378971a79c28


1: M111 unrecognized component name: I18N


This is a little consistent with the changes below, e.g. `dimension_1`


@NCNecros first of all `synchronized` is a very big change, it required a solid justification. Second - neither from commit message, nor from comments in the code it's not clear why it was introduced and how it should help.


right, you do not have to believe me - that is why I added a test instead that fails without this change ...


QwtUpdate (axelstudios) - 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)


Looks good.


Don't think these changes were really required but if this style is better for everyone then sounds good, we should try and keep this consistent across the whole code base however.


some tests failed:
failure: http://build.servo.org/builders/mac2/builds/1127


Sure


Run starting. Expected test count is: 1
DataTest:
[2015-03-03 CET 18:11:46.582] [WARN] (org.apache.spark.Logging$class.java:71):logWarning - Your hostname, eric resolves to a loopback address: 127.0.0.1; using 192.168.1.5 instead (on interface wlan0)
[2015-03-03 CET 18:11:46.585] [WARN] (org.apache.spark.Logging$class.java:71):logWarning - Set SPARK_LOCAL_IP if you need to bind to another address
[2015-03-03 CET 18:11:47.046] [WARN] (org.apache.hadoop.util.NativeCodeLoader.java:62):<clinit> - Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
- Test that both Seq and RDD works
  + Some serious stuff going on...
Run completed in 3 seconds, 94 milliseconds.
Total number of tests run: 1
Suites: completed 1, aborted 0
Tests: succeeded 1, failed 0, canceled 0, ignored 0, pending 0
All tests passed.
@Supermanu Pretty cool man!


Можно параметризировать логгинг - вместо конкатенаций мы можем указывать в строке %s и просто дальше следующими аргументами указать то что хотим туда подставить


done, thanks )


If this no longer exists we should probably remove this method and we'll update the major version.


mostly experimenting - wanted to see if it helps as it seemed HikariCP's initializing connections rather slowly, since than I found that upgrading to latest HikariCP 2.3.x makes it just fine ...


Typo, this should be `Unparsable object`


1540218-SlinkyGroundHX (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


saw approval from SimonSapin
at https://github.com/servo/servo/commit/09c53f461dadf0c1f2896754c329ee6ad3e9359e


merging servo/servo/background-size = 09c53f46 into auto


servo/servo/background-size = 09c53f46 merged ok, testing candidate = aa22f777


Not sure on this change, I prefer we are explicit when naming are variables. Is this just to reduce line length?


As above we should probably remove this method


Nagyon jó hogy összefűzted a stringeket!


This patch needs a better commit message explaining why you need to do this.  It is not obvious why you should have to clear each IRQ bit individually rather than just write 0xFF to AFE_IRQ_CLR.


I'd like to find a better way to fix this...


Final proposal


i had an empty string isRemote. So i added the following code:
   `<cfif isBoolean(variables.$.siteConfig('isRemote')) and variables.$.siteConfig('isRemote')>`


As I reread this, I'm not sure you need this last sentence anyway. You basically said what the value was before the previous sentence.


I am sorry but it seems you refer to some custom properties that are NOT in mainstream CMake and has been reverted. You may be used some patched version of CMake then.

http://www.cmake.org/pipermail/cmake-commits/2013-November/017289.html
> Revert "add source file properties EXTERNAL_SOURCE and OBJECT_LOCATION"
> This reverts commit d207f1bf8fd697ae6521a1336dd27222168f548e.


Можно вынести вверх т.к. в обеих блоках одно и тоже


cool thanks.

another newbie question:
do we also need to free the zend_string* somehere created in https://github.com/php/php-src/commit/2f79fd8d43967aa3fab7bbbc8a75e2ae13d88445#diff-3291092b024fc91e40bd9b6948adb819R243 ?


Why not just make it abstract ?


I think this endpoint has been badly named, in reality I think it should be `sso/users`, we'll stick with `sso_user` however to stay consistent.


Yes, I went off the standard approach rails used. In the DB there are two column hash and salt. Let me know if you want me to change it.

_____________________________
Jason R Alexander
  Sent as binary to your brain!!!

> On Mar 3, 2015, at 8:26 AM, James Vastbinder <notifications@github.com> wrote:
>
> NVM - I see bcrypt in the next commit.
>
> —
> Reply to this email directly or view it on GitHub.
>


Можно без непонятный сокращений)


That's an easy fix. Just didn't know what to name it.

Jason

_____________________________
Jason R Alexander
  Sent as binary to your brain!!!

> On Mar 3, 2015, at 8:29 AM, James Vastbinder <notifications@github.com> wrote:
>
> at some point, we will want to refactor the route naming to match our terminology. This will be fine for now.
>
> —
> Reply to this email directly or view it on GitHub.
>


Можно без статики?


Что-то мне подсказывает что это можно вынести в самый верх и упростить


OBSERVED A BUG after the real test at the lake:
the function is_tack_completed() didn't work as expected: both the LQR and MPC tack were completed after opc_data.min_time_in_band_us micro seconds were elapsed since the FIRST entering in the tube near the origin! So, even if the system went out the tube after the first entrance, after opc_data.min_time_in_band_us micro seconds from the first entrance, the tack was considered rightly completed! By the way, this could be a "good" bug since every tack was more or less good ...


Ok, minor thing: Commit message, how about "Do not require default_doafile" as shortlog message?


wondering whether this should be named: `client_reseller_usage_summary` to read better, @benSlaughter - what do you think?


Was planning to jump on this today but you beat me to it!  Looks good.


I still think that the plugin should not be instanciated if it is used as a static object (An instanciated object may have its static method used, but on a domain point of view, it is wrong...) and even if it is nothing, it avoids to instantiate something for no reasons at all


what are you doing


What now?


Cheers for the info. Ok. I'll try to migrate it with some tests added of course. It just wasn't clear to me how to go about that. Thanks for pointing that out to me!


I think we should rename this to: `reseller_mention_usage_report`


@jezdez, can you see if you can run ``fab updoc``? I don't have a working Django setup atm, so Sphinx can't import what it needs, and my attempts to get a setup set up are not going well.


1540218-SlinkyGroundHX (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)


You mentioned that you're considering modifying CMake itself to handle PCH naturally, so I am suggesting that you consider this option in the EXTERNAL_SOURCE concept to avoid linking, and the PCH language concept as a replacement for the HEADER_FILE_ONLY concept. I'm not suggesting that you apply this work, I'm suggesting that you make use of these ideas, as they allow a nice PCH implementation, compatible with GCC, Clang and MSVC, without tons of extra targets in significant projects like ReactOS.


Yes, the getCountFromHTML is a revamped version of countIt()


beautiful


If I understand your question right, that array contains all the keys
belonging to the user. (hrm, might need to return id-key pairs instead of
just keys)

On Tue, Mar 3, 2015 at 10:06 AM, Chris <notifications@github.com> wrote:

> just curious what is this Array to be full of?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MRN-Code/nodeapi/commit/f44ffde8ea838bf8493f8416afd7e66fadfcee3d#commitcomment-10011560>
> .
>


I don't think this method needs to exist in its own module, we could just put this in the parent class.


It doesn't really make sense to chain the managers, I think you should use just the one and be done with it. Moreover, as it is called statically, you cannot have different managers (hence different plugins) anyway... so your `$configuration` parameter should go, and the `getConfiguration` should just return an array as it actually does


Indeed it's a pain to read. The issue was that I exported the XAR from the wiki and committed it after running mvn XAR format. I should have exported before making any change and run XAR format and then only start changing things (i.e. 2 round trips).


Повторяющийся оператор, можно сделать abstract ScalarOperator


Hacking with the Fujitsu guys. I moved it on a separate branch.


Ezt azért tudtad volna egyszerűbben is.


пожалуйста избавьтесь везде от статики


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


deps[i].name === name должно хватить


82420068_GridViewMultiEdit (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Tests Failed

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


r+


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


Adding code to repository


Drat; this actually won't work. :\

Sorry, but: this isn't the section where the visibility is actually applied. The "preview" visibility that it's using here is actually useful; it allows harvests to be processed but not published, visible only to the ContentPartner (and admins) before finally being published.

There's a section later in the code of ```app/models/Resources.php``` where auto_publish is checked (see line 239, where it "fixes" preview mode), and I suspect _that_ is where this code needs to live... or some combination of here and there.

More complicated than expected; sorry!


повторение тела метода


Is this line indented one too many levels?


is this line indented one too many levels?


Did you actually mean this, or did you mean
.hb h1, .hb h2, .hb h3, ...
?


зачем нужна строка инициализации раз уж класс сам по себе уникален?


should this line be indented one more level?


Even better, it could loop over self.indexLabels instead of subviews. Sorry I didn't realize it didn't loop over only labels.


no need for it - tests will be run 3x3 = 9 times after this change :) This option sets Python version used to run tox; tox runs tests under different Python interpreters, selecting Python version based on TOXENV variable.


Ezen if-re nincs is szükséged.


конструктор всегда - первый метод класса


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


Ye better be fixed, for reference: it's easier if you comment on the prq :)


please wrap by { and }


As I said, more trouble than it was worth. I didn't want to be answering the same question, over and over, forever.

SparkFun's EAGLE library is primarily intended for our internal use. We share it because we like sharing, but the internal usage comes first, and it wasn't working for us, internally.


@thinkyhead , I'm trying to get rid of those CHECKENDSTOPS.. I will deploy a mod in Z check endstops soon and will change it as well. Changing this IRS is dangerous  ;-) we need caution.

About the logical analysis and those 2 IFs, I always prefer a Human Readable code!
Probably the compiler will optimize it anyway..

I *hate* when I have to do maintenance on other's code and it is too much complicate than it suppose to be.

Cheers.

Alex.


Would you mind looking into your database tsettings table and see what the value of the isRemote column is?


$authenticatedToken = new AutoLoginToken($this->providerKey, $token->getKey(), $user->getRoles());

This works too, then you don't have to check if it's a UserInterface


Other than "it's shorter" there's no real reason. Let's just leave it as is.


Whats the status on Umbraco 7 supporting MySQL? I've tried a fresh install on a mysql db and it failed with exceptions regarding the DB and here I'm reading that the changes have been incorporated into 7.2.2 which I've been using.


not sure if it's worth trying to put the page insert and this insert into a transaction


I needed to an additional `unique` into the string to prevent a `type /base/Vector2d already
    defined in registry` error from happening during opaque-creation. Not sure if this is the right thing to do to remove the actual problem (someone is adding too much into `options[:opaques]`... So:

            options[:opaques].uniq.each do |opaque_t|
                 opaque_registry.create_opaque(opaque_t, 0)
            end


this could be a little safer


Calendarización
Base de datos
Análisis de base de datos: 7 días hábiles. 7 de marzo de 2015 se entrega.
Normalización de tablas: 3 días. Entrega 11 de marzo.
Diseño de base de datos: 4 días hábiles Entrega  16 de marzo
Creación del código: 2 días. Entrega 18 de marzo
Pruebas y corrección de código: 5 días. Entrega 24 marzo
 Aplicación de escritorio
Sistema da Inicio sesión: 4 días. Entrega 28 de marzo
Vendedor Consulta datos del cliente: 6 días hábiles. Entrega 4 de abril
Vendedor Registra clientes: 7 días hábiles. Entrega 13 de abril
Vendedor Da de baja clientes: 7 días hábiles. Entrega 21 de abril
Vendedor Genera pedido de cliente: 8 días hábiles. Entrega 30 de abril
Vendedor Cancela pedidos de clientes: 7 días hábiles. Entrega 8 de mayo
Vendedor Modifica clientes: 6 días hábiles. Entrega 15 de mayo
Vendedor Recibe pedidos de distribuidor: 9 días hábiles. Entrega 26 de mayo
Vendedor Consulta pedidos de cliente: 8 días hábiles. Entrega 4 de junio
Gerente Registra empleados: 6 días hábiles. Entrega 11 de junio
Gerente Da de alta a empleados: 6 días hábiles. Entrega 18 de junio
Gerente Da de baja a empleados: 6 días hábiles. Entrega 25 de junio
Gerente Registra sueldo de personal: 5 días hábiles. Entrega 1 julio
Gerente Registra proveedores: 8 días hábiles. Entrega 8 de julio
Gerente Recibe reportes (ventas, quejas, sugerencias de todos): 15 días hábiles. Entrega 25 de julio
Gerente Genera nomina: 10 días hábiles. Entrega 6 de agosto
Gerente Cancela pedidos de distribuidor: 6 días hábiles. 13 de agosto
Gerente Hace pedidos al distribuidor: 12 días hábiles. 27 de agosto
Gerente Recibe pago con tarjeta de crédito: 10 días hábiles. Entrega 8 de septiembre
Gerente Recibe pago con tarjeta debito: 8 días hábiles. Entrega 16 de septiembre
Gerente Recibe pago en efectivo: 8 días hábiles. Entrega 24 de septiembre
Gerente Genera facturas: 10 días hábiles. Entrega 6 de octubre
Gerente Genera reporte de gastos: 13 días hábiles. Entrega 21 de octubre.

Aplicación web
Cliente Inicia sesión: 6 días hábiles. Entrega 26 de marzo
Cliente Consulta catalogo/galería: 17 días hábiles. 15 de abril
Cliente Agrega productos al carrito: 12 días hábiles. 29 de abril
Cliente Consulta carrito: 9 días hábiles. 2 de mayo
Cliente Genera pedido: 9 días hábiles. 13 de mayo
Cliente Recibe número de pedido: 9 días hábiles. 24 de mayo
Cliente Imprime factura: 10 días hábiles. 5 de junio
Cliente Consulta factura: 10 días hábiles. 17 de junio
Cliente Recupera contraseña: 10 días hábiles. 29 de junio
Cliente Se registra en sistema: 14 días hábiles. 15 de julio

Nota: La aplicación web se comenzará a desarrollar al mismo tiempo que la aplicación de escritorio.


The `result = "";`, is it your design decision or is it defined by standard. I am not questioning the decision, I am just wondering.


Не сразу заметил что это несколько алгоритмов сразу и как тест :D
Так или иначе есть огрехи в написании, однако, к сожалению, не могу проверить на тестовых данных, увы.


Hey Remi,

You've linked to /2015/02/R-at-command-line/ as your permalink - overwriting my submission on the blog! Can you please make a pull request with a new title (alternatively I can change mine, but I think yours doesn't match the concept map)


I'd like to refactor this whole chain, probably move more of it to the pages service


Is it necessary to bind this to $parent?


Interesting approach! Be aware that you are missing the rolling back functionality with this commit!
Cheers


Please remember to quote all paths as this all breaks if they have spaces.


Thanks for contributing. I can't find a signed Contributor License Agreement for your username. I can't accept contributions unless we have one on file. It should be pretty quick to do. Go to the link listed in this file: https://github.com/youtube/js_mse_eme/blob/master/CONTRIBUTING


That looks like you had spaces between the dashes.
Copy: ```node --version`` - what does that give you?


also includes .gitignore


82420068_GridViewMultiEdit (evanweaver) - 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)


please drop `|| ret=$?` as well.

Otherwise we are not really testing anything.


Please quote paths here too.


Just "node", a space, "--version". That should print out the node.js version you're using.


maybe add a line like:
``` config.vm.network :forwarded_port, guest: 9869, host: 9869, auto_correct: true ```

so one can get the gui. but to use this one has to set se variable:
```  $listen_ip      = $::one::params::sunstone_listen_ip,```
in manifests/oned/sunstone/config.pp ...
if not set, the gui binds to 127.0.0.1 and one cannot access it.


`NSLocalizedString`


Are there any other (non instance) elements that could be used here for testing?

I mean, by this change, we are basically dropping test-case for unresolved elements. The test case, that might be important one (you don't want oscap to execute random xml).