Cocoon helps with using nested forms.
My understanding is that it makes it easier to have one form that sends off
the information to multiple tables.

On Mon, Aug 31, 2015 at 10:26 AM Kurt Nelson <notifications@github.com>
wrote:

> What does this do?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CheeseFry/TeachingTool/commit/d2a7746501d1121b975758fc913ea8b3188148a0#commitcomment-12978646>
> .
>


fixed


Call the file `exceptions`.


we use the node package pre-commit, so actually everything should get setup when you run `npm install`


I just want to apologize to @gero3 my feedback is just my opinions, regardless of what you incorporate from my feedback into this PR, it is still great and a needed improvement.


Leave this commit out for now.


For simulating the occurrence of a taxon in an alignment, the taxon has to be present in the replicated tree. Some taxa however are not present in the final and therefore in the replicated tree. When replicating an alignment, we therefore can miss sequences for taxa that are not in the tree. In this commit, we replace these missing taxa with taxa that are in the tree.
This way, the replicated alignments have approximately the same number of taxa as the original ones.
We have to keep in mind though, that this might add a bias towards the taxa that are present in the final (and replicated) tree.


It's fine to fix it with a followup pull request.


You can also add a blanket ignore for the bin and obj directories as well as *.suo files. These are all generated


Sure, will clarify.


Forget, this is the right commit, I'm going crazy.


And as an alternate, it could be published under a diff name like `tinyconjs`


Note: can be also activated with F12


"Currently available.." ? Do you mean the driver provided with openPOWERLINK? If so, please make that clear!


Does the bow shooting and food eating bug still exist?


Yes, rcu_barrier() does not interact with rcu_read_lock/unlock() directly, but rather with the worker thread(s) that run call_rcu callbacks. Actually, if rcu_barrier() is executed from an online QSBR reader thread, it will put it offline. If rcu_barrier() is executed within a RCU read-side critical section, it will trigger a warning on stderr. The idea is that we cannot call rcu_barrier() from a context that prevents call_rcu callbacks to execute, or we would cause a hang.


Yeah, I'll look into that later.


This enables the verbose debugging of events and other things, make sure you have -loglevel trace to see the effect. See also -debug commandline parameter


Remove the man page, it's completely incorrect.


Because my brain has a stricter type-system than C :P But sure, I see your point, will change.


Does it make sense to provide a download link (or is it clear, except for me :) )?


Good point. Will change.


done


user_record extension uses only path with domain name. in fact without this changes i do not see record in cdr


Thanks its really annoying


legacy, I remove


it might be good to check the permissions of these folders


Yes, rcu_barrier() has, by definition, exactly the intended effect detailed in the comment. It only returns once all prior call_rcu have been executed. What different effect do you think it has ?


Nice Work!

How do you do? xD


Well, it can be worked-around to always yield valid loginuid or None:

```pid, uid, gid = ... peercred()
loginuid = getloginuid()
pid2, _, _ = peercred()
if pid != pid2:
   loginuid = None```


"... build the *stack*..." ---> but below you describe how to build the driver!?


Suggest:
>To preserve promised information throughout the docs, search the repository for links to the revised section and update related articles accordingly.


-s returns 0 for almost any file in /proc. So "read $file, my $content, -s $file, 0" reads no data. But actually those files are normally readable if we read them until EOF.


I think this method is wrong.
I'd guess it was intended to add retrocompatibility for the older calls with the new method, but it's not working the way it used to. By passing a void array as the layers affected by constant we completely disable the constant, instead of applying it w/o any filter, its former behaviour. It should pass an array with the names of all the layers (both view and overview).


"Default is narrow" may be a bit confusing. The factory default is narrow, but the default on a project can be changed (and there are good reasons to change it). Maybe we should be returning (and mentioning in the error) the project default, rather than the factory default?


win


@dsabrinarg el campo booleano "module_purchase_third_validation", donde lo puedo agregar, http://awesomescreenshot.com/0a4573xvc0, en este momento lo tengo en este modulo, es algo recursivo.


Just so it doesn't get lost, I've added an issue for this bug at #5


produce() is also a Level Controller command, I wouldn't split it like that


:beers:


This change won't work, because it allows numbers meant for `location` or `columns` to fill in that space. In order to avoid that, we have to give `span` a failing value as soon as we see these keywords.


To me a url with a trailing slash denotes a _collection of stuff here_ and without it _the resource at this location_. I have an uncommitted description of how the top level urls should work to enable discovery, and in this case it is a listing of resources available under this prefix.

Having said that it is imho bad practice to only serve the trailing slash version, as people will get stupid 404 errors. Often the non-trailing version redirects to the trailing one. I guess people will enter the non-trailing one more often than the trailing one.

I am not really fixed on this, so if you feel strongly about it I can revert the change, no problem.


why the double indent?


We typically do styling with CSS rather than in the JavaScript


@alan-guzman yo lo veo bien allí o no entiendo tu duda, a que te refieres con "algo recursivo"?.


@enoter Huh?


We don't commit the JS generated from the TypeScript files


Can you clean up the formatting of this code


incidentally, i would be happy to convert this all to RST


/me slaps past @lucaswerkmeister :(


I understand what you mean, @vmassol , but it`s not about that. I actually spent more time than needed to come up with that message, even consulted @evalica about it.

The problem is that, at the location where that exception is thrown we don't have the reason why the permission check failed, just a boolean with a `false` value. Any detailed checking is done in each `MacroPermissionPolicy` implementation individually.
So we have:
* VelocityMacroPermissionPolicy:
```
!context.getTransformationContext().isRestricted() && getAuthorizationManager().hasAccess(Right.SCRIPT)
```
* GroovyMacroPermissionPolicy
```
        boolean hasPermission;
        if (this.configuration.getCompilationCustomizerNames().contains("secure")) {
            // If we are not running in a restricted context and we have the script right, the macro may run, but
            // security will be delegated to the Groovy Secure Customizer.
            hasPermission =
                !context.getTransformationContext().isRestricted() && getAuthorizationManager().hasAccess(Right.SCRIPT);
        } else {
            // Defaults to checking PR
            hasPermission = super.hasPermission(parameters, context);
        }
        return hasPermission;
```
* AbstractScriptMacroPermissionPolicy (i.e. any other script like `python` or whatever else):
```
return authorizationManager.hasAccess(Right.PROGRAM);
```

The conclusion is that the reason varies a lot, but whatever happens there are 2 big groups of cases:
1. the rights of the last author are not correct (missing PR or SR)
2. the rendering process is configured explicitly to block execution (`context.getTransformationContext().isRestricted()`)

Given these 2 general cases, I considered that it's better to express these 2 cases instead of listing all the 3 (PR, SR and restricted context) specific cases and risking to send the user on a wild goose chase or even being completely wrong since there might be some custom MacroPermissionPolicy active which might not like the color of the current user's eyes.

Does this make sense?


signal 和 slot必须保持参数一致，但是button的signal又无法重写.. 只能重复写多个不带参的SLOT，蛋疼。。


@dsabrinarg si le das en el check para desactivar esa característica, no aparece mas ese botón.


what xd


I'm going to revert it for now just to keep them all consistent, but I think a general review of URLs is warranted. I agree we need to serve both, no one types training slashes and it's generally confusing to most people that `/foo/` and `/foo` are different URLs.


this lines are the parts of the changes that must be logged? ( issue #2228 )


The pid doesn't change, the race is about pid-reuse


new line


Please use `e.originalEvent.dataTransfer`


or i can parse all of this to see what are changed? (issue #2228 )


return updated, nil


this test should not have a redirect_uri param instead run it as it was before...backwards compatibility


This didn't actually work.

I can reproduce this locally, though. With the current configuration, output capturing is disabled if I run `py.test`, but not if I run `py.test tests/functional`. I'm still looking into why.


defer d.Close() on previous line.


please remove this stuff it is useless, getpeercred is race free because it stores the pid/uid/gid variables at socket creation time, re-reading them from the scoket will never yield a different answer.


remove this line


93651760-RowPerObjectReport (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1807 of 1807 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)


can we add failures here too?


Sure, but that would happen on a different socket connection.

By peercredding after reading '/proc' you get the guarantee  that it's still the same process, on the same socket.


"And the quality bar keeps getting lower."


Closes issue #2


ioutil.TempDir("", "cheat")


ideal is good, make a option to detect chinese and japanese string, detect a lof of crap can filter by something or someone who know about it.


На мен дължина на ред 100 и дължина на метод 10 ми вършат идеална работа. :)


use filepath instead of path


I'm a bit worried 1s will prove too little for CI. I think 5s is good too , no?


Interesting, oauthlib itself is pretty effusive in debug mode https://github.com/idan/oauthlib/blob/master/oauthlib/oauth1/rfc5849/__init__.py#L139 ... prints out all the signatures and tokens.

I'd rather just remove these log messages, I tried sanitizing and it was more code than I want to add for little benefit.


a different socket == a completely different request
we'll never have a socket changing under us as we process a request


ditto


API说明文档:
/reg /login
(mail,password)其中密码以明文发送

/logout
(无参数)

/addtask
(library,taskname,taskinfo,score,[limit])
其中limit为可选,不填则为-1(即不限次数)

/updatetask
(library,taskname,taskinfo,score,[limit])
其中limit为可选,不填则为-1(即不限次数)

/deltask
(tid)

/compeletetask
(tid)

/addproject
(projectname,info)

/updateproject
(pid,projectname,info)

/delproject
(pid)
注意:只能删除当前登陆账号所拥有的project,否则返回错误

/shareproject
(pid)
注意:只能分享当前登陆账号所拥有的project,否则返回错误

/unshareproject
(pid)
注意:只能分享当前登陆账号所拥有的project,否则返回错误


Аз май ще го сложа 100 / 20 :)


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.460.772ca7bca26be3e9fadb9889136a0ea7ec944800](http://build.palaso.org/viewLog.html?buildId=62311&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests passed: 3356, ignored: 134; exit code 163 (new) Build time: 00:22:22


So what happens if I am not logged in and hack the url? Feels like there is a security hole here. Take another look and see what you think.


/reg /login
(mail,password)其中密码以明文发送

/logout
(无参数)

/addtask
(library,taskname,taskinfo,score,[limit])
其中limit为可选,不填则为-1(即不限次数)

/updatetask
(library,taskname,taskinfo,score,[limit])
其中limit为可选,不填则为-1(即不限次数)

/deltask
(tid)

/compeletetask
(tid)

/addproject
(projectname,info)

/updateproject
(pid,projectname,info)

/delproject
(pid)
注意:只能删除当前登陆账号所拥有的project,否则返回错误

/shareproject
(pid)
注意:只能分享当前登陆账号所拥有的project,否则返回错误

/unshareproject
(pid)
注意:只能分享当前登陆账号所拥有的project,否则返回错误


if you call "defer d.Close()" in above, you don't need to call here. Or if you want to return err of d.Close(), remove above "defer d.Close()".


We don't use synchronous http requests. https://www.youtube.com/watch?v=SfKTDMEoC88
Why? They freeze the user interface while they run.


looks good


test


I think this exposes the security hole. Even in the last iteration - if there wasn't a current user, but I made the right request to delete the post, I think I could.


does the build.sh script run on Windows ?


No point in using the any type, more keystrokes and no benefit.


There is a much easier way to accomplish this

[.with(securityContext(marissaContext))](https://github.com/cloudfoundry/uaa/blob/feature/invitations_flow_by_email_domain/uaa/src/test/java/org/cloudfoundry/identity/uaa/login/InvitationsServiceMockMvcTests.java#L110-110)


Ohja, vervelend; helemaal niet btw?


changing an existing test to make it pass means you break backwards compatibility - if you change the test to be "" why not make the default value of the parameter "" and not require it at all?


Oh, the issues we had involved it not compiling at all, not "secretly doing something wrong".  :-)


Gcloud-golang-compute-metadata inherits from gcloud-golang which breaks
evaluating vault and a significant number of things. I've been looking at
how to fix gcloud-golang but it is still overwriting oauth2/google for some
reason.

On Mon, Aug 31, 2015, 02:22 lethalman <notifications@github.com> wrote:

> Also please cc people next time you revert something.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/2d16959bb838797a60a10ae48ad35c4d818ab462#commitcomment-12971976>
> .
>


I see why that is. It's because of the `io.open` which only accepts `unicode`. I'll fix up `test_source` to accept either `str` or `unicode` on Python 2.7.


Thinking about it on the weekend, I believe that this was an issue with an earlier version of VS that I don't think appears any more.


first I'm not considering you responsible for anyhing :) I know you spent time. I'm just stating the fact that what we have now is not good enough for our users IMO.

You nailed the problem when you said "The problem is that, at the location where that exception is thrown we don't have the reason why the permission check failed," This is what needs to be fixed. We could imagine several solutions, one of them would be to put the error message in the execution context so that it could be printed from the generic place. Other solution would involve changing the returned boolean by a more complete data structure including the message (harder to do).

So my preference would go for the message in the context. WDYT?


Right, that boils down to that checking if is_alive(socket) *after* reading '/proc' will give the same guarantee.


Why we treat decimal type special here. There are no partitioning tests for other types here.


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


@ernfrid As author of the original AdspQcSummary you may want to review this commit.  I removed insert size since that seemed specific to the ADSP project needs.


/reg /login
(mail,password)其中密码以明文发送

/logout
(无参数)

/addtask
(library,taskname,taskinfo,score,[limit])
其中limit为可选,不填则为-1(即不限次数)

/updatetask
(library,taskname,taskinfo,score,[limit])
其中limit为可选,不填则为-1(即不限次数)

/deltask
(tid)

/compeletetask
(tid)

/addproject
(projectname,info)

/updateproject
(pid,projectname,info)

/delproject
(pid)
注意:只能删除当前登陆账号所拥有的project,否则返回错误

/shareproject
(pid)
注意:只能分享当前登陆账号所拥有的project,否则返回错误

/unshareproject
(pid)
注意:只能分享当前登陆账号所拥有的project,否则返回错误


the problem with that is #lead_unit_radio takes 2 parameters, a name and a unit. I'm not looking for a specific unit. Just need to know if the person has unit radio buttons or not because this affects the indexing for finding the units in the table.


Looks like the file wasn't formatted before it was committed. You can run Ctrl K-D to do it. It might seem like a small deal, but it can cause very hairy merge conflicts if at a later stage someone edits this file and someone else formats it on another branch.


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1813 of 1813 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)


Oh...


Ser bra ut!


Then we should return the project default.


oh, ok. that makes sense. :)


Pull requests are appreciated ;)


Thanks for the stats, that's interesting.

Another piece of info to consider is [Oracle Java SE support roadmap](http://www.oracle.com/technetwork/java/eol-135779.html), where we can see Java 6 is indeed quite old, and even Java 7 support ended in April 2015.

I had mixed feelings too, but the [TwelveMonkeys library](https://github.com/haraldk/TwelveMonkeys) page said that versions 3.0.x –which do support Java 6–, are unsupported on Java 8. I did try with those, and it could build and run fine, but then I thought that it w/b OK to drop support for Java 6 and be 100% safe everything will work fine in Java 8 and higher.

Basically, we have 2 options:
- keep it as is, i.e. drop build and runtime support for Java 6 in the default dist (people can always make it work by replacing the twelvemonkeys lib by an older version).
- use twelvemonkeys 3.0.5, at the risk of having potential runtime issues on Java 8.

@mgylling any thoughts?


Seems to cause mounting issues.
sda1 is not always hdd, same applies for sdb1 as usb.


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


SHould you break here?


@tofi86 about Apple's approach: do you know what's planned for El Capitan (which is now around the corner)? Will they keep proposing Java 6? or just delegate systematically to Oracle Java?


No it won't, because a process can fork and let a child keep using the ticket, then die itself na dhave the pid reused. We simply cannot trust info read this way from proc for access control, as the race condition can be easily exploited by a clever attacker.


Ah. Okay. I'll accept it for now, then. Though at some point I'll probably tweak it and the related code. :)


done in next patch


93651760-RowPerObjectReport (JasonGlazer) - 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)


Gottem, nice.


sgtm! bridgy/logs.py tries to be smart about sanitizing creds, so as long as it sanitizes oauthlib's log messages so full creds aren't in the publicly visible logs, I'm happy.


Even like this:
```haxe
@:build(CFFIMacro.build("lime"))
class System {
    static function lime_system_get_display(n:Int):Dynamic;
}
```
that would generate:
```haxe
class System {
    static var _lime_system_get_display = loadPrime("lime", "lime_system_get_display", "io");
    public static inline function lime_system_get_display(n:Int):Dynamic
        return _lime_system_get_display(n);
}
```


Pretty intense


Can `member` be anything, or is there some structure expected?


good point, I removed it


Why not `sig:signature-clause`?


Do we need SyntaxAndDeclarationManager to be nested?


Of course.


It's not wrong, but we don't use exceptions in our www codebase - it tends to hide errors. We rather let the app die


I should have noticed this sooner.  I do think the registration should mirror the DNS-SRV, as we've been doing up until the IESG reviews.

In particular, I think xmpp-client can get messy (RFC6120 vs websocket vs non-yet-existent "xmpp-over-spud" vs ...).


Поля класса лучше сделать private.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.32.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=62314&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:48:01


Нужно ещё 2 два get-метода для date и title


Yes, although I do find it's current meaning quite humorous.


Rename paramter to syntaxAndDeclarationManager?


Unhandled Fundamental Failure


We have a `{{fa-icon 'user-secret'}}` helper which is not only slightly less to type, but protects us incase font awesome changes everything again (which they've done at least once)


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.461.bf113795a83ff39bb34d1cfb9711ff0b45c5d22a](http://build.palaso.org/viewLog.html?buildId=62320&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3356, ignored: 134 Build time: 00:16:06


I'd use an `is-string`-type package here. Just a thought, since this fails for `String` objects.


TypeScript has support for [template strings](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/template_strings). They are awesome!


humm, mine is same. I guess I don't like the big empty window that shows when going from windowed to fullscreen. A little icky looking.


det här verkar lite kul


"reuseSyntaxManager"?


How should we do this?
I propose that I run some tests and if everything is ok merge this branch with the feature/yarn branch and delete it? Otherwise, should I make changes here and move them later?


What's the point of wrapping this in a function?


We have a `{{fa-icon "bookmark"}}` helper which is not only less typing, but is future proof against a breaking refactor of icons (this happened at least once and ruined @nlalonde's week)


syntaxManager?


Absent the comment above, I'm fine with all these changes.  Thanks again, stpeter!


@ashwinr64 look at this


Somehow you added this comment right as I had made a change, but, either way:

It was working, but, went ahead and moved it inside as that makes more sense
thanks


Since TypeScript 1.5 we started using "let" instead of "var". We should write all new code using let.


If you are putting `{{handlebars}}` inside an attribute you don't need to surround it with quotes anymore unless you are doing string interpolation.  `title={{i18n 'thing'}}`


It's to show that you can supply the filename even if you don't have access to the parseJson function.


This change removed the closing quote at the end of line 5 - cask is now broken


For me, this throws compiler exception:

>/away3d/core/managers/Stage3DProxy.hx:542: characters 35-58 : String should be Null<flash.display3D.Context3DRenderMode>
>/away3d/core/managers/Stage3DProxy.hx:542: characters 35-58 : String should be flash.display3D.Context3DRenderMode
>/away3d/core/managers/Stage3DProxy.hx:542: characters 35-58 : For optional function argument 'context3DRenderMode'


please name "int" parameter, so we know what it is.

either int *bleh* or int bleh is fine


Not really. I don't want to support String objects.


Ah okay. I think the fact it's an exception makes it clear enough, though. To me it's not clear that the function does anything special.


maybe {hook h='displayHeader'}?


You may want to make these boolean arguments keyword arguments, to make it clearer what they are.


lgtm, lets squash to the previous commit.


Created a pull request for the broken quote in #1236


Ok, removed.


Thanks, I'll leave it as block instead.


It's fine to use JavaScript's built in JSON.parse here because all browsers we support have support for it. A very useful resource to check browser support is [caniuse.com](http://caniuse.com/#search=json)


Unrelated, but could you replace String.Format(CSharpResources.Trees0, i) with $"trees[{i}]" and remove the resource? The parameter name should certainly not be localized.


Ouch, true. Thanks.


I believe the current style doesn't require "this.".


:+1: No opposition to that.


je ne crois pas. C'est une élision 《qu'il est nécessaire 》.


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


Could we do this in a batch for all the trees being added?


You're not using the input for this problem. This won't work for other test cases. Please fix.


FWIW `systemctl --global enable dbus.socket` is only needed if the distribution (e.g. Arch as of dbus 1.10.0-2) removes the static enable.


Write a comment explaining this magic  number please.


This chunk should not be here, just ignore it :)


Could we do this as a batch for all the trees being added? Perhaps as a second pass over the tree list.


Smells like redemption


This file should not be here, just ignore it :)


thank you


$"{nameof(trees)[{i}]"


*keyword*. :wink:


Don't use namespaces like this please, see https://google-styleguide.googlecode.com/svn/trunk/cppguide.html.


nameof(trees)


TeamCity mobile-music-automatization :: PR_BUILD_CONFIGURATION [Build 42](http://teamcity.mobile.dev.yandex.net/viewLog.html?buildId=1190279&buildTypeId=MobileMusicAutomatization_TestBuildConfiguration) is now running


Looks good mi wise


TeamCity mobile-music-automatization :: PR_BUILD_CONFIGURATION [Build 42](http://teamcity.mobile.dev.yandex.net/viewLog.html?buildId=1190279&buildTypeId=MobileMusicAutomatization_TestBuildConfiguration) outcome was **SUCCESS**
Summary: Running Build time: 00:00:08


You should make it async, so that if IndexedDB ever matures and is usable, we can switch to that.


so accoring to https://github.com/owncloud/contacts/issues/581 and what I said in https://github.com/bitfireAT/davdroid/issues/578#issuecomment-136418262 : doesn't this "revert change" expose invalid URLs and isn't the urldecode() in line 209 (https://github.com/owncloud/contacts/commit/484fce0941e367cb984298bc4bc7a74f1505a5aa#diff-b1ec55d2ffd61f9d014f3bde20b82546R209) irrelevant then?


"syntaxManager"?


This should be in the library.


"this." not needed.


That is a great point, I had no idea that non `IConvertable` types could be coerced with `Convert.ToDouble()`

One method that might have more success would be something like

    double doubleValue;
    if (double.TryParse(value.ToString(), out doubleValue))
    {
        c.wheelThrottle = (float)Safe.Utilities.Math.Clamp(doubleValue, -1, 1);
    }

try parse is stricter and i cant think of any types that would successfully parse that wont convert to a string. We could have an `else` that logs an error.


1786_fix_python_bindings (elainethale) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2287 of 2343 tests passed)

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


I'm hitting crashes within `LLVMGetElementType` when I do `v.is_function()` where `v` is a value of type `double`.

Perhaps the `ty.is_element()` call above should be guarded in some way?


It would be better if this was a method "GetSyntaxManager()" to avoid expensive evaluation while debugging.


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

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

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

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

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

```


@AdamChlan - I think we should be able to move these lines completely.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.936.75851c14f6a6058798e741d9b343a7b07b0b678c](http://build.palaso.org/viewLog.html?buildId=62322&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:16:07


From El Capitan [release notes](https://developer.apple.com/library/prerelease/mac/releasenotes/General/rn-osx-10.11/):

> OS X El Capitan is the last major release of OS X that will support the previously deprecated Java 6 runtime and tools provided by Apple. Applications or features that depend upon Java 6 may not function properly or will not launch when Java 6 is removed. Developers should move to a newer version of Java as provided by Oracle.

That said, I'm not sure what's the UX when user launch the CLI for the first time (i.e. what dialog shows up, redirecting to which version of Java).
For other distros (e.g. Pagina's EPUB-Checker), devs are free bundle whichever version of TwelveMonkeys (or none at all) they want to provide.


Does the user name depend on how hive/hdfs are configured?


yes, it runs for generating the projects (but obviously the "make" doesn't execute)


Yes I tough it might be dangerous to get rid of it since this page is meant to be included.


Fixed, thanks.


Why did you remove the test for whitelisted webadmin? As long as we have it I can see value in testing it, no?


Oui tu as raison, je m'en rends compte maintenant que tu le dis.


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1811 of 1815 tests passed)

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


I have activated it and corrected travis configuration file location. It starts to execute script, but it cannot find some of packages. I think the problem is that this system works under Ubuntu 12 which probably doesn't have Qt5 yet.

I got following log:

Using worker: worker-linux-9eb6c2a4-2.bb.travis-ci.org:travis-linux-14
system_info
Build system information
Build language: ruby
Build image provisioning date and time
Wed Feb  4 18:22:50 UTC 2015
Operating System Details
Distributor ID:	Ubuntu
Description:	Ubuntu 12.04 LTS

............

0.59s$ sudo apt-get install libqt5gui5 qt5-default libpng12-0 libpng12-dev qttools5-dev qttools5-dev-tools libgsl0-dev libgsl0ldbl
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package libqt5gui5
E: Unable to locate package qt5-default
E: Unable to locate package qttools5-dev
E: Unable to locate package qttools5-dev-tools
The command "sudo apt-get install libqt5gui5 qt5-default libpng12-0 libpng12-dev qttools5-dev qttools5-dev-tools libgsl0-dev libgsl0ldbl" failed and exited with 100 during .
Your build has been stopped.


Unhandled Fundamental Failure


TeamCity mobile-music-automatization :: PR_BUILD_CONFIGURATION [Build 43](http://teamcity.mobile.dev.yandex.net/viewLog.html?buildId=1190292&buildTypeId=MobileMusicAutomatization_TestBuildConfiguration) is now running


TeamCity mobile-music-automatization :: PR_BUILD_CONFIGURATION [Build 43](http://teamcity.mobile.dev.yandex.net/viewLog.html?buildId=1190292&buildTypeId=MobileMusicAutomatization_TestBuildConfiguration) outcome was **SUCCESS**
Summary: Running Build time: 00:00:04


I am still not 100% sure on the title


hmm as I said on the issue I was fixing them... I've been doing that all day actually :) But I wanted to finish moving the annotation functional tests and possibly add some new tests to verify it won't happen again before fixing them... Anyway, I'll just not pull your change for now... :)


This is a good idea, but unfortunately this is not so simple to detect.


We don't store references to DOM elements, but rather references to their IDs. We sometimes have utility methods that give you back a DOM element, but it is never stored. The reason we do this is because we think of controls as existing in memory rather than in the DOM. The DOM is just the rendered representation of the state of the control. We often re-render controls from scratch which would invalidate the reference to the DOM element.


Should be mi_write_event_exclusions?


Tried the desktop with emacs on fc14 and there was a space between each letter.


Esta version funciona con la libs 6.0.0.  Canbia la conf con la version 6.2


Usually I think we name these enable/disable options ending in `enabled`. See other such options in [GraphDatabaseSettings.java](https://github.com/neo4j/neo4j/blob/2.3/community/kernel/src/main/java/org/neo4j/graphdb/factory/GraphDatabaseSettings.java)


You can use the JQuery shortcut to get the DOM element : `$("#next") `


We typically use CSS for styling


1786_fix_python_bindings (elainethale) - i386-Windows-7-VisualStudio-12: OK (2303 of 2343 tests passed)

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


@geemus Ooof, good catch Wes :$


Guau!!


just a tiny bit :stuck_out_tongue_winking_eye:


LGTM, but can you explain how you've tested that this works as you expect it to?


If you have comments in first person singular (as makes sense here), it's a good idea to sign them.


@aleph7 Take a look; this may effect us.


Something like

```perl
use Foo;
Foo::Bar->{xyz};
```

wrongly identifies the PPI::Token::Word 'Foo::Bar' as a class name when 'Foo::Bar' is actually a constant hashref (created with the constant pragma.)

this should fix that issue.


@nle-odoo  No more excuses :droplet:


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (759 of 759 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-9.35%25-green.svg)


Sounds reasonable. Easy to implement, does not break compatibility and we can probably have a fallback in case no reason is provided. We also need to clean the message in the execution before each permission check, but that's expected.

The only thing we can not achieve with this solution is to programatically get the reason (since it will be a string). Not sure if this would be that much of a problem, since a good idea for each policy would be to return a translation key instead of returning the english message string.

Anyway, it should be good enough.


<!DOCTYPE html>

<meta charset="UTF-8">
<title>CV 1103120019</title>

 Story about me

  	Nama saya muhammad afif, saya lahir di subang 9 agustus 1994. saya kini tinggal di cimahi selatan yakni di perumahan melong. hobby saya yakni bermain game online. sekarang saya sedang menjalani perkuliahan di telkom university yang dahulu namanya adalah institut teknologi telkom. banyak sekali perubahan dalam hidup saya, banyak juga teman-teman dan pengalaman baru yang saya dapatkan yang dimulai sejak saya lahir hingga saat ini.

    <input type="button" value="back" onClick="history.go(-1);return true;">


Peut-être une reformulation éclaircirait la phrase.


1786_fix_python_bindings (elainethale) - 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)


Thanks @pm0733464 !


Well, I don't know the conventions, but we're in `mi-lttng.c`, and the function is static, so it's pretty obvious it's related to MI.


Looks good except for naming and missing test case.  @jgalar xsd validation for exclusions string is already present.


nope.


updating iso right now


What's the status here? That doesn't look ready to push.
If it's only used for error reporting above, you may want to add a comment to explain that, and that the attributes are dummy values.


@tracker1 none of the above variables are intended to be overridden, even though they are not constants. But if you accidentally add a line of code which overrides such a variable, the project just won't compile and you will be able to spot the bug right a way, at compile / build time.


It's been a long day, sorry for my harsh comment. Thanks for taking the time during your day off to actually fix the issue :)


TeamCity YaccConstructor :: YC.Core General Build [Build 489](http://teamcity.codebetter.com/viewLog.html?buildId=198328&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 236 Build time: 00:18:23


> and missing test case

Where is the test checking for single XML event fields?


Created http://jira.xwiki.org/browse/XWIKI-12461


Why dont you use the Push icons we already have?
They are already cached, so setting the size to 16x16 would work correct?
Could you make a patch that works with the default icons ?


:+1: This change alone makes updates much more efficient. Thanks.

Any ideas when the next release will be?


Is this repo building on the new CI?


Is there a reason to keep the second return value? Especially since it complicates the code.


We need to move the laziness into the syntax manager, not here. Otherwise a sequence of compilations created using With* methods won't share the manager unless they are created after the first one's manager is initialized. See ReferenceManager creation pattern.


1771-SeriesPIURemove (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


I like the patch, but.....
I think this has to be handled in the Scheduler.cpp class correct?
So this will happen automatically
could you make a patch for this ?


Should the second value be `aliases-raw` instead? This throws away the values accumulated so far. If so, you may want a comment explaining why.


A common convention to use use the prefix `rev-` for accumulators that are built in reverse order, then reversed.


@greg80303 - did you try this? If you seek towards the end, lots of 404 not found errors are thrown.


Maybe we can consider adding an interface to the pages that can't have the burgermenu instead?
IVeganMenu :D


What is it?


`define-type` as a signature-form errors when parsed. I can just remove the attributes for now since they shouldn't be used anywhere. It would be nice to support type definitions in signatures, but I'm not exactly sure how to implement them right now.


"aussi complexes que vous le souhaitez" me paraît plus clair et plus proche de la phrase d'origine "as complex as you wish"


Thank god...


cc @eelcovisser (it looks like this thing is purely internal to nixland, right?)


I did think about doing that, but figured in reality, there will almost certainly be way more pages that can have it, than those that can't.


Thank you, merged to master!


Ok, then let's clean up the code, and make it clear what the status of that feature is. And maybe add support later.


Because Java 8 is becoming the main java version? Or minecraft is bumping?


Could you add comments to explain what each part of the expansion is for?


That's a good idea :)

Perhaps it should work with defaults:

```haxe
@:build(lime.system.CFFI.build())

...

@:cffi("lime", "lime_system_get_display") private static function lime_system_get_display (n:Int):Dynamic;
```

```haxe
@:build(lime.system.CFFI.build("lime"))

...

@:cffi(null, "lime_system_get_display") private static function lime_system_get_display (n:Int):Dynamic;
```

```haxe
@:build(lime.system.CFFI.build("lime"))

...

@:cffi() private static function lime_system_get_display (n:Int):Dynamic;
```

With defaults, it may be possible to ensure you can link multiple NDLL names, and not match the function name, if desired, while making the default case more terse. I already have the first case (with no defaults) working now, thanks to your help :smile:


d is circular, so does not readily work.  However, there is a solution.  I'll try to implement, so all non-circular top-level information is included.


Por mi parte ok, he arreglado pequeñas erratillas y he cambiado un poco las
frases alrededor de lo de Ingeniería del Software, que me sonaban un opco
raras.

También he separado en párrafos lo del final.

Finalmente, el titulo lo dejaría en: "Control de versiones y su ecosistema"
 porque no se si lo del alrededor realmente aporta algo, pero lo dejo a tu
criterio.

On Sun, Aug 30, 2015 at 6:17 PM, Juan Julián Merelo Guervós <
notifications@github.com> wrote:

> Completando y terminando @fernand0 échale un vistazo
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ReVision-Docencia-20/Columnas/commit/5968d6f000c326ae7ecd1b5fd339d31a13963934

--
Fernando Tricas
http://elmundoesimperfecto.com/


Ok good :+1: But why does this LiveCD get so much intention on the index?


Tyler Henderson, tyler.henderson07, [thenderson37](https://github.com/thenderson37)


That does sound right. So not an issue.


the windows installer could also be interesting


Include what's only necessary:

//#cgo pkg-config: libavcodec
//#include <libavcodec/avcodec.h>


Me parece bien ese título. Envíalo cuando quieras.


Forgive me, but where's that?


http://pastebin.com/PemaiWPQ getting dis error .. please help while compiling opti


Is there also a new version of 64tass? So I may provide new OS X binaries?


This will break if `$IP1` and `$IP2` aren't set.


This was just an optimization for when the replace isn't needed on mono. The real fix is the other commit where the slash replace was missing.


I'm not sure if FEL's teaching week after holiday is even or odd; @jirutka will look into this.


@pdshetley This looks great!


Thanks. I didn't think of the rebuild check. I will merge your changes, so that this bug gets corrected, but I'm not happy with the solution.

I think the small data set needs some optimizations. In my test it's much slower than the large dataset.

Ideas that need some testing:
- Reduces the extension length for this data set type - perhaps also to one character.
- Do not use the digit replacement for the small data set type.


I have changed default enum to point to zero. I don't like to use maps here, too much unnecessary overhead.


I know you removed these lines but comments should be before the line like below -
# Import socket module
import socket


@dequis @radare currently test is failing


it should have been put in BROKEN=1 state


@rodfersou I think you should use [plone.i18n](http://docs.plone.org/develop/plone/misc/normalizing_ids.html) instead


Sorry not the recordings for IVR Menu


use `NFKD` instead


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.33.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=62326&buildTypeId=bt2) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 873, ignored: 50 Build time: 00:35:50
### Failed tests
```
LibChorus.Tests.dll: LibChorus.Tests.VcsDrivers.Mercurial.HgWrappingTests.Commit_WLockExists_GetTimeoutException: <no details avaliable>

```


But here, in this switch we have the `coup de grâce`. Unfortunately it is partially my fault because I was a bit negligent when posting this part of the assignment.

Question: what happens after a statement is executed in a switch block and there is ***no break; ***?


.DS_Store files should not be in the commit, add it to .gitignore. Also, remove from the project-
git rm .DS_Store
stage this change.
git commit -m "Removed .DS_Store files. Added this to .gitignore"


I think there's no need for a function; just normalize the character here


One last thing. In Eclipse, you can right click your project from the Package Explorer tab and go to `Source` and `Format`. This way, all the files in your project will be nicely formatted and squeaky clean.


Sounds nice!


It works in this usage. You can have a datadir or a datasource that is interpolated, but you cannot dynamically load a different hiera.yaml, so you cannot have different hierarchies based on the environment.


congrats! Now I can update all my repos.  Looking forward to the training session in Austin!


You're returning the same thing in all three paths - why not use an `else` statement and have one return?


GameOver is not an InvalidAction.


There should only be exactly 1 new-line at the end of file.


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

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

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

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

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

```


I think you lost a carriage return here?


Don't we reserve all upper case variables for macros?


Even more important, pick a name that says what this does.  I get to line 314 and I really don't understand what's going on at all.


I would implicitly add our files:

Either:

    file(GLOB ext_target_files ${CMAKE_CURRENT_LIST_DIR}/path/to/include/*.<extension>})

or

    file(GLOB_RECURSIVE ext_target_files ${CMAKE_CURRENT_LIST_DIR}/path/to/include/*.<extension>})

You can then just space delineate in the add command:

    ADD_LIBRARY(Engine ALL SOURCES ${ext1_target_files} ${ext2_target_files})


nice :)


Unhandled Fundamental Failure


> All the dbus libraries have already been fixed long ago to use the user-bus as default.

Your definition of "long ago" does not seem to match mine. sd-bus has tried this path for a while, but the other major implementations, libdbus and GDBus, have only done it since March and June respectively if you count development releases, or less than a week ago and not yet if you count stable releases (which is what most people should be running).

I would recommend continuing to set `DBUS_SESSION_BUS_ADDRESS` for quite a long time - as @heftig says, more niche implementations such as txdbus, dbus-java, dbus-sharp are unlikely to pick up the `XDG_RUNTIME_DIR/bus` default for a while.

`dbus-launch --autolaunch` from dbus 1.10 will also try returning that address, which helps out some other implementations; but that only works in X11 (I want dbus-launch to go away entirely in Wayland/Mir/other non-X11 environments). Also, it involves running `dbus-launch`, which IMO is always undesirable.

> systemctl --global enable dbus.socket is only needed if the distribution (e.g. Arch as of dbus 1.10.0-2) removes the static enable

If you have influence over the maintainers of dbus in non-Debian distributions, and they want to support both the traditional "bus per login session" and the newer "bus per user-session" models, please suggest that they ship a `dbus-user-session` subpackage that enables the user bus, as I have done in Debian - it's currently stuck in the experimental suite to avoid disrupting the worst C++ ABI transition we've ever done, but will be uploaded to unstable/testing as soon as I can. That way, other packages that benefit from the user bus can suggest, recommend or depend on it.


Hi Iannis,

thanks for the ping -- I wanted to update you anyway, but got distracted :P

I am not exactly sure I did the right thing everywhere -- so yes, some tests would be great.  It seems to me that the non-yarn case still works ok, which is a good sign :)

You'll see that the agent structure changed somewhat:  before, we had a `run()` method which iterated over all CUs it could find in the queue -- now we have a `work()` method which only gets one CU to work on at any specific time.  So that `work()` routine represents the inner loop of the former `run()`, and all the queue pulling is done in the background.

You'll also see (or realize) that the ownership of the LRMS and scheduler classes has changed, and for that reason I had to move the yarn cluster shutdown to a different place -- please check if that is still functional!

I'll be offline for some more hours, but very happy to sync afterwards, or whenever convenient...

Cheers, Andre :)


That is also why I changed the name. I think you are right to keep this out of the WebglCapabilities class. It is more valuable anyways as a  `supportsFloatVertexTextures` property


Please don't commit so many empty *.py files. If you do explain in multi-line comments in that particular file, what file is this for and why this is empty and why this is in the commit.


So `env` is both callable, and has methods?  What's the advantage of having `operator()` over a named method that would explain what's happening here?


@binford2k thanks for clarification


@Maijin where does it fail?

I got a message this morning about http://ci.rada.re/job/valgrind-radare2-regressions/147/ but the console output has completely unrelated failures.


ok


Lol


good!, now, can you add rotation?


@blacktrash I did test the content, but I did not see the 404 errors until you pointed them out because they did not seem to affect the playback.  I'll take a closer look...


I've implemented this (plus the rest) on the subsequent commit. I think while it cleans up the questionable Demeter violations, it also adds a substantial amount of complication to the code base on the basis of checking that we're querying against a `Configuration`-like object, and not a `HashWithIndifferentAccess`.

I think we might be at the point where we should subclass `HashWithIndifferentAccess`. WDYT?


@RedstoneGunMade He could do it in his own time


There is no test for single XML of event_field. But the addition of a nested array of elements justify the need to test for it.

Event_field array is tested here tests/regression/tools/mi/test_mi:543


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

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

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

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

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

```


@aharley40 please test ALL your commits with "sencha app build"


Every one of your python files should be like following -
#!usr/bin/env python
#-*-coding:utf8-*-
__doc__ = """
Remember every python file you write is actually a python module, not just a script.
Explain clearly what this module does and what it has.
"""
# All the code you write
# One new line in the end

Reference:
https://github.com/nishworks/academic/blob/master/Index-creation-for-search-Engines/filesystem.py


Can these errors explicitly mention the regexes we're using? Something like `"invalid access_key_id; %s does not match %s" (access_key_id, ACCESS_KEY_ID_RE)`


> I would recommend continuing to set DBUS_SESSION_BUS_ADDRESS for quite a long time.

Yes of course. But I'm not exactly sure whether upstream systemd is the place to carry it. It's much more likely to be moved into a legacy-compat package someday, similar to the bus-proxy (maybe involving pam_legacy.so or something like this).

I just want to make clear that this is legacy and should someday be removed.


I think all have been renamed now to PascalCase


Works perfectly! Thank you!


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-MacOS-10.9-clang: Tests Failed

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


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.454.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=62333&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:34:41


I think, that if compiler doesn't output an error and freezes, it's a bug :smile:


No. https://github.com/aspnet/BasicMiddleware/issues/11


Hey @cgand , thank you ! it's loading via FTP now
( note : https://plus.google.com/+DavidRevoy/posts/fj5Q96rdTk2 , read the comment, a happy customer ; thank you! )


Fixes #33


Hm... Would this match look better if you replace it with `item.map(/* ... */).getOrElse(/* ... */)` pattern?


Ref https://github.com/graphistry/graph-viz/issues/69 for follow-on work. Thanks!


Hey Chris - Cab we add a check for the existence of `.data`

Ideally the payload that is resolved is already the `series` options object.


Isn't this so much nicer that one-off params? Imagine if you had a really big form...


You may merge Security Frameworks into Authentication and entitle it "Frameworks". What do you think?


Hi Helmut, is a TER release planned as well? Cheers for upload


For business logic that performs validations, my style is to use a separate statement for each individual check. For both clarity of exposition and also so that you can have fine-grained control over breakpoints.


wrong `@` I guess


Hi Carlos,

I'm working on a conda recipe ( on osx ) for VTK 6.2 as well, so this is great...
Would the patch `dlls.patch` extend to unix as well?

I'm curious what
```bash
+  has_prefix_files:                           [win]
+    - Lib/site-packages/vtk/__init__.py       [win]
```
accomplishes...

Terrific to see support for PCL coming about, awesome...


@motivatedHamster The parameter name needs to be "teacher_id" since that's what the controller expects.


we need if-else for checking number lower than 1 is must be equal 1


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.937.d14dd83052c2f9894fe95f2ca0a0f41cb1530c80](http://build.palaso.org/viewLog.html?buildId=62338&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:07:44


`T(S{1})` is shorthand for "one second from now"


马丹！我错了23333 顺手就按了版本号[0.0.0]


Also added the company name of the packing owner


@blacktrash I wonder if this is a bug in the player.  The 404 errors come when the player tries to download the "98" segment which is past the end of the content, so therefore, it doesn't exist on the server.  The segment lengths shown in the manifest all add up to the mediaPresentationDuration, so the manifest seems correct.  Unless the media segments themselves are not the exact duration listed in the segment template, I'm not sure why the player is requesting this last segment.


Un... I'm not sure that moving Authomatic here is appropriate.


@dequis For the test result, look at the TravisCI https://travis-ci.org/radare/radare2/jobs/78063152#L397-L406, not at the jenkins, and if you look at the jenkins, don't look at the valgrind.


This is a bit of a no no, it would require the restore_command to have access to the config file. (which it in most user setups would probably not have)


@AkamaiDASH @KozhinM If you have a few minutes, could you take a look at this updated Envivio content and see if you think the player is incorrectly choosing to download an additional segment at the end of the presentation?


Header Image


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

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

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

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

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

```


The design of `jtx` values brevity over clarity. You have to first understand the framework. A goal of `jtx` is to allow expressing an entire test in a single line. So a lot of design decisions were made to enable tests that previously spanned many lines to fit on one.


Will change in another PR.


It doesn't *need* to be, but it's not used outside the Compilation, so I just made it nested.  It's ```internal``` right now, but I think I should make it ```private```.


You also need to sync inside xg.


Helemaal geen semicolons; einde verhaal; :D

*Met vriendelijke groet,*
NullByte.nl <http://nullbyte.nl/>
Merijn Testroote | Freelance Developer
T 06 52717960
KVK 59978678

On Mon, Aug 31, 2015 at 6:01 PM, Ninalys <notifications@github.com> wrote:

> Ohja, vervelend; helemaal niet btw?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ninalys/HelloOcto/commit/70f923537a375375569ed245e2c206841382e077#commitcomment-12979406>
> .
>


Jeff's updates, cumulative distributions and n=2 lineages


shouldn't it be {{ value }} because I get an error about mapping in

```
local:
    Data failed to compile:
----------
    Rendering SLS 'base:salt.formulas' failed: mapping values are not allowed here; line 18

---
[...]

# Setup the directory hosting the Git repository

/srv/formulas
...:    <======================
  file.directory:
    - makedirs: True
    - group: root
    - user: root
    - mode: 755
[...]
```

And if I remove the ```|yaml``` part it works.??


This is changed in a new PR
https://github.com/ripple/rippled/pull/1255


Appropriate comment location.


Yeup... what a mess. What do you think about biicode?


@mefistotelis , do you have a way to reliably test multiplayer on a single pc?


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1287 of 1287 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-69.86%25-yellow.svg)


Bad idea. At least add a check if it actually comes from the server not some random guy lol.

A remote kill switch is a very malicious thing to add. I would recommend letting the server kick/ban/mute or whatever people.


I generally prefer complex objects like this to be separate entities. It's easier to avoid dependencies on private members of the compilation that way.


I'm not sure this is a good change. I feel like it is better to stick with Java 6 for maximum ease of use.


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

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

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

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

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

```


There's a space at the start of the line that doesn't belong there.


Formula (2.24) was a mistake which got corrected in 2nd edition of the book. Unfortunately 2nd edition is not freely available, but one can look at [Matlab sources](http://www2.hsu-hh.de/ant/dafx2002/DAFX_Book_Page_2nd_edition/M_files_chap02.zip), file apbandpass.m.

Actually 1st edition has correct formula (2.19) on page 41 as well.

There is the same error in src/algorithms/filters/bandreject.cpp.


`WithMap`, `WithTransform`?


thanks :)


1. Consider extracting the common term `(*slep)[~sfDigest]` with a name to document it.
2. Why is `ctx_.tx[~sfMethod]` being examined?


RoomAirflowNetwork (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1810 of 1810 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)


not sure that it simpler, but ok


why you add this ?
this can make some bugs (ex: random reboot).


# Introduction
### Manuel De La Rosa
##### My majors are:
 * Computer Engineer
 * Electricl Engineer


Well, fuck. I changed the binary right before submitting the PR to a smaller version, forgot to update the offset.

```
$ r2 ../../bins/elf/hello-linux-i386-pie
 -- Well this is embarrasing...
```

Indeed, r2, it is.

https://github.com/radare/radare2-regressions/pull/220


The presence of `sfMethod` indicates that there is a condition that must be fulfilled. If `sfDigest` is present, then `sfMethod` must be specified, and the `sfDigest` must match what's in the SLE.


Error: El sistema permite hacer mas cantidad que la del lote, ya que no toma en cuenta los packing list sin confirmar el recibo.
Solución: El sistema debe de sumar la cantiad total en packing list abiertos y cerrados.
function deliveredQty2($id)


:ship:


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


Ok, I believe that we need a new top section named "Services" and put CIs under it.

@makrusak Would you do that for me?


hi.. are you shure to delete them all??
http://www.wowhead.com/npc=50376/angered-earth
its part of heroic..
normal we should do it with creature.spawnMask there you can set a mask to spawn creatures in specific mode..
http://collab.kpsn.org/display/tc/creature#creature-spawnMask (this wiki is surely outdated)
the mode now is 3 also spawn in normal 10/25 man mode: what I think is wrong..
have you/someone better info/wiki about spawnmask in cata 434


issue mistake, this commit is related with issue #5


Unhandled Fundamental Failure


We need these to work in order to merge this pull


Thanks for this.  Upon inspection, I'm finding that a few things don't work - probably has to do with the latest changes in staticdocs.  My errors happen in `get_rd_data()`.

First, it appears that `staticdocs::parse_rd` now prints the result instead of returning the object, which throws things off here.  I tried just replacing the call to `parse_rd` with the necessary code inside the function (without the print), but that still doesn't work - need to look into what else has changed in staticdocs.

Also, `to_html.Rd_doc` is not exported from staticdocs.


Thank you so much for the PR.


View jsp should show a list of Foo objects that can edit the foo entities.


:cry:


@dequis awesome :+1:


See here for the jsps source:
https://github.com/liferay/liferay-plugins/tree/master/portlets/sample-service-builder-portlet/docroot


I suspect nobody else uses gaia decision, so this probably isn't a problem (ie. no conflicts).
But in general it's smart to choose cache names that are fairly unique, and not likely to be reused by another project using the same `workerType` (and ideally we should have fewer workerTypes).

Anyways, `gaia-tc-vcs` and `gaia-linux-cache` might be safer :)

----
This is a nit, no reason to fix it... I don't think there is any conflicts on `gaia-decision` workerType, that's probably only used for this anyways.


91958608DOASEffectOnZoneSizing (wfbuhl) - 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-81.84%25-red.svg)


1810-osversionfail (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Following the tutorial, there is a comment that suggests that the pagination should be displayed after this step, however the count is missing and therefore the pagination wont be displayed.

"Notice that the buttons of the directive doesn't look very nice now because we haven't added any design and CSS to our application, we will do it later on."

It would be good to move this comment further down the tutorial to not confuse novices as myself.


The `clearRequire.all()` destroyed `babel-eslint`'s monkey patching which gave me quite random errors until I understood the cause :P


Why were these methods removed?


Well, my bad. This doesn't necessarily signify a successful upload. Just tells us that the upload operation has finished. Could very well be a failure, too.


minor: unnecessary space before second listOf


Usually whatever is in `class << self` is indented.


91958608DOASEffectOnZoneSizing (wfbuhl) - 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)


Why is this needed?


I think this causes giant topic titles in topic lists. see
![image](https://cloud.githubusercontent.com/assets/2583394/9585877/71818e7e-4fe7-11e5-8dcb-188ba225cee1.png)


managed to read up to this point and I like the changes (don't answer this comment - it's just a mark for me to know where to resume reviewing)


@thenickcox Why did you remove Raven here?


you're not storing the `Insurance` instance


A brief note: this and the following commit (045632a and ab7e59e) were pushed by mistake and meant to be one commit with a clearer commit message.


Not sure we need to raise error in reg_delete_value?  Maybe simpler to just call RegDeleteValueW directly from delete_value, and check result (== 0).


The latest version should be fine, see this: https://github.com/hadley/staticdocs/pull/90/files.

staticdocs::parse_rd should return a `Rd_doc` object.

Sorry about that, to_html.Rd_doc should be to_html instead.


Ik denk dat ik je snap, en het lijkt zo inderdaad alsof het niet klopt.
Ik stem de tool op het moment af op de functionaliteit die Judith nodig heeft (maar ik neem aan andere ondernemers dus ook).
Als ik mij niet vergis, heeft zij bepaalde dingen gekocht, voor de zaak ,en over 21% hiervan kan ze aftrek krijgen, en die 89% niet. Dus van de bedragen wil ze twee nieuwe bedragen, de BTW ervan, en het overblijfsel.
De eerdere versie was mijn gedachte van wat nodig was, dit is de versie van wat daadwerkelijk berekend moet worden.
Of klopt er toch nog iets niet ?


Hey @tsromana . I modified the TigerVPN config files to specify which tun device to create, ie tun0 instead of dynamic allocation. This is part of the scheme to avoid IP leaks. transmission/start.sh checks tun0 for its ip address and makes transmission bind to that. It will default to tun0, but now it's set explicitly just to be sure.

The other providers already specify this and I don't see a reason why this should cause trouble for TigerVPN, but now you know :)


Great


Also indent is off in the comment for LONG WINAPI RegDeleteValue(


Isn't such a check required to prevent explots?


@philips can you elaborate on how and when this "name" field would be used?
For example, if I already have the `config.json` downloaded would I ever need to use this URL at all?
If I'm downloading the `config.json` then I clearly got the URL via some other mechanism since I can't get to 'name' w/o first downloading it.
If this is just a way to say "where this config.json/image came from", then if I copy an image from someone else am I expected to crack it open and modify this field (so it points to my copy) prior to using it?

I'm not against a 'name' field, but we need a clearer definition of its purpose in life.

w.r.t using URLs for a discovery mechanism, I totally agree with using a URL scheme but my gut is telling me that the URL needs to live outside of the image since you can't look inside it until you downloaded it.


101789044_gridviewFilters (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)


Note that the string `'false'` is truthy:

```js
!!'false' === true
```


E agora está bacana?


Thanks


The lines in the kFunctionPointers array can/should be made shorter by 8 spaces (in the third field, the one for the pointer, before the }). You only need to fit ```&Functions::nameOfTheLongestFunctionName ``` in there. For example

```
{  454, "ZY_EFFECT_EffectForceResistanceDecrease", &Functions::ZY_EFFECT_EffectForceResistanceDecrease },
```


Hmmm. I will have to check how I will be able to change this variable inside of IntelliJ. I am not sure where "misc" settings are configured.

I will look into this and get back to you.


Thanks.  I just got it to work and manually merged your PR but worked around this by putting in the contents of `parse_rd` without the print (will change back once your PR to staticdocs is merged).

One other thing I changed - I put the manual "usage" extraction back in.  I hadn't documented it, but I ended up doing this manually because staticdocs puts all the usage onto one line, which doesn't look nice in the output.  But the manual handling of usage in packagedocs preserves the newlines.  If you have a better idea for doing this without doing it manually, or if something is incorrect with the way we are doing it manually, please let me know.

Thanks again for the updates.


I don't think this is a good solution. I think we should move the template to a header and include it instead. This introduces a lot of mistakes. I think most of the files include singleton.h, maybe change it to util.h or something and add this aswell?


+ işaretlerinin vigül olması gerekiyordu .


If the public interface is `configure` and `validate_user`, you should consider making all of the other methods private to enforce the public interface.


Can we dynamically output the current year instead of updating it every year?

For example: <?php echo date("Y"); ?>


I see about the heap operation causing iterator instability, but I think we work around that by incrementing *rangeIt->first IN PLACE BEFORE caching the tuple_range BEFORE pop_heap. So:

            // There are more than one partition left. Remove the current top partition from the heap
            tuple_range nextPartition = *rangeIt;
            std::pop_heap(partitions.begin(), partitions.end(), reverseTupleRangeComp);
            partitions.pop_back();
            // The rangeIt is invalidated at this point and can't be used to access the top partition
            if (nextPartition.first != nextPartition.second) {
                // The partition is not empty yet. Reinsert it back to the heap.
                partitions.push_back(nextPartition);
                std::push_heap(partitions.begin(), partitions.end(), reverseTupleRangeComp);
            }

Won't this work?


It only works in the .cc files, and we can keep it like this untill Ubuntu 14 goes away:)


For some reason the manual regex your have in packagedocs was not working well for S3 class, nor was staticdocs. I have a PR submitted there as well.

Please see this: https://github.com/hadley/staticdocs/pull/90.

now the usage sections look pretty:
http://docs.flowr.space/en/latest/rd/topics/complete-help.html#id151


How come this isn't in the `Errors` class?


That's right, Thx @eregon .  Tests within modules go inside folders. The test class names are the names of the exclude files.  To exclude a whole test, you can comment it out in mri_truffle.index


yeah, annoying. btw i'm totally fine with your multiple commits per PR, i was just hoping to get rid of the merge commits specifically. (but you probably already knew that! :P)


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.938.1478ecd22f30f1f09ef846514ac0788ce97484d4](http://build.palaso.org/viewLog.html?buildId=62341&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:04:26


Not really, I think. I took it from here (line 82) https://github.com/joestump/framework/blob/master/Framework/Template/Smarty.php

Should $cache_id and $compile_id have `null` or `' '` but not both?


There's a [mailing list thread][1] discussing naming, discovery, and distribution in general, which has a few references to this commit.  It may be able to shed some light on intended use cases.

[1]: https://groups.google.com/a/opencontainers.org/forum/#!topic/dev/OqnUp4jOacs


on the respective architecture.

As I wrote above: I tested the WORDS_BIGENDIAN case on a *little endian* architecture using the very same data as for the !WORDS_BIGENDIAN case.

What outcome would you expect in that [i.e. testing a big-endian conversion algorithm on a little endian architecture] case? Again "87654321"? Really?


Are you still getting this Problem? A little odd, that travis uses that outdated ubuntu.
There is a ppa which should work with newer qt, http://askubuntu.com/questions/378397/how-to-install-libqtwebkit5
 But it would be better if we could setup a vm with ubuntu 14 or 15


I think it's more clear to rephrase this as `size() != 1 || !typeParameters.isEmpty()`


`NumberOfArguments`


hva er dette?


I see.  It would be nice if long lines could still be broken up intelligently though (break at a new parameter), for example, in your usage example above, the `## method for class 'flowmat'` line extends beyond the available space.  I have packages with functions that have a lot of arguments.

So we could stick with the staticdocs usage - I'd just need to add something after that deals with lines that are too long.


import matplotlib


Later versions of sigc++ will also support c++11 lambdas out of the box, without the need of this ugly code. But we have to wait for Ubuntu (slowest distro atm sadly...) to catch up.


you are right. Can you plz try to write `for (var i = 0; i < video.buffered.length; i += 1) {`
I know it is not 100% correct but I did not find any safer way on a quick look. As it is only for debugging purpose and not part of the real playback engine, I think we can live with this, if it works for you?


Given the `AuthorizationFilter` no longer hard-codes the whitelist, then this test is no longer needed - the test above covers ensuring that the filter correctly applies the injected whitelist (you'll note that it does use `/browser.*` in the test config, and ensures it whitelists correctly).


`?: return null`


Очень странно.


Should have solved simple case first : "When there are no coins inserted, the machine displays INSERT COIN."


I think it looks pretty cool. I would rather it backended into something cool and hash-y like submodules do. There is a facility for convincing it that a dependency for a block should really be filled by using your own personal forked block, so it would work OK with all the overriding that has been going on.


Здесь должно быть не 2, а sizeof


There definitely should be a way to not build two copies of protobuf in the project (in xg and in vg).


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/f3dbb7ed8738b592e64f8c4d64fd49a3305e450f/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/f3dbb7ed8738b592e64f8c4d64fd49a3305e450f/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/f3dbb7ed8738b592e64f8c4d64fd49a3305e450f/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/f3dbb7ed8738b592e64f8c4d64fd49a3305e450f/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/f3dbb7ed8738b592e64f8c4d64fd49a3305e450f/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


How is Java 6 easier to use than Java 7 or 8?


我还能说什么呢？对于这样的兲朝上国，我还能说什么呢？！


Зачем толпа пробелов?
И || надо перенести.


Após essa modificação o comando default do TexStudio de build deve falhar.
Será preciso rodar o script para geração dos arquivos das figuras. :ghost:


1786_fix_python_bindings (elainethale) - Win64-Windows-7-VisualStudio-12: OK (2305 of 2343 tests passed)

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


Client will close it automatically and as for exploits it gets trade distance gets checked on trad open and since https://github.com/cmangos/mangos-wotlk/commit/c0fea6e1d596276bdf8a23de1c64a09784b08336 on trade close (should have moved that specific check probably into this commit)


yep - know about that thread and I'm trying to leave the discovery side of it aside for a moment, I'm more interested in what this field is for once this file has been downloaded and is sitting on the FS.


I was just browsing GitHub for mentions of driverpracticaltest (to see if anyone else had already made a scraper), and came across this. Thought I'd just mention it's a bad idea to check keys into repos. Also see https://www.google.co.uk/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=0259d827c6fd3539c6e350bd0571ad95


Ah, I see !

Wish there was a way to detect code window size and wrap this.


;D
Maybe this is useful for you too: https://github.com/ezio84/packages_apps_Settings/commit/2b69fa834dac711dc8f5dabf43326c30c8f49b28


Don't forget about validations!


Ooops! :P No worries here, but that's one of the reasons we need easy and user friendly ways to generate test data / have tests at our fingertips.


I've removed the code - there are ways to subvert it.

Loginuid is by no means suitable for access control/authentication; it's just for audit / logging.


Awesome.


I'd like @seelabs to review this change to make absolutely certain it does the right thing. Should be trivial.


The Runtime Android doc said this wasn't the recommended method - so I just wanted to check: https://developers.arcgis.com/android/guide/install-and-set-up.htm#ESRI_SECTION1_4108D3B809C54DD4A0BD37E6397EBFA8


Minor: the version on the [download page](https://developers.arcgis.com/en/downloads/) was listed as "Version 10.2.6-2" so not sure if that mattered


Absolutely, I will submit a pull request once my soon, thanks!


Indeed it is! I like how we discover better ways of writing the same code as we progress. Soon I'll be generating entire websites with the click of a button :-)


It is valuable to have for logging, all we need to do is simply to return loginuid in a different attribute, my comment was simply that we cannot have in in "creds" because those are suitable for access control while loginuid is not.

Instead of creds, have it set in its own attribute on the request, or maybe as a member of a dictionary called "audit".

So I would maintain the code and just add the info in a different place.


Is there a reason for this, or will it even be merged with the main branch or is this just for testing?


+<!-- Estos son cambios de Montelongo -->
+<!-- Estos son cambios de Montelongo -->
+<!-- Estos son cambios de Montelongo -->
+<!-- Estos son cambios de Montelongo -->


> I'm not sure this is a good change. I feel like it is better to stick with Java 6 for maximum ease of use.

Yeah no.... There's absolutely nothing about Java 6 that is easier to use than that of Java 8. First hand experience of the failings of the various versions of jdk6 can be the biggest headache and require the messiest of workarounds you can imagine.


Looks good to me.


util.HandleError


sasasasasasassa


(I can't really comment on the right line) The block starting from `**The default callback:**` should be indented, currently the docs for the constructor breaks apart.

On a related note, I do not think there's any reason to put emphasis on `The default callback:`


11:18 <@dakta> The reason changing that class breaks it is because both TBui methods popup() and overlay() leave handling of the close button to the caller, since we usually want to do cleanup on close.
11:18 <@dakta> So calling it tb-settings just inherits some close call from somewhere else.
11:19 <@dakta> I’ll try and figure out a clean way to provide default close handling and allow custom handling. Maybe a “beforeClose” method that you pass on initialization...


@rtibbles I think the error here is that I tried to consolidate conditional logic but I misinterpreted part here. I moved the `node["format"]` update and `shutil.copy` stuff so that it's only executed for node kinds in `["Video", "Audio", "Image"]` in  the `construct_media_node_bundle` function.


Do you need to empty the table each time you update the contact list? You could build the table's first row statically in your index.html, and only remove the rows that hold contact information in them. For example, your table could look like this:

```
<table id="contacts">
  <thead><tr class="contacts-headings"><td>Contact ID</td><td>Contact Name</td><td>Contact Email</td></tr></thhead>
  <tbody><tbody>
</table>
```

and then you can empty out just the contents of the `<tbody>` in jQuery:
`var tableContent = $('#contacts tbody').empty();`

This will be more efficient so you don't have to rebuild the same table headings every time (and using the `thead`/`tbody` tags makes it a bit more semantic, too).


Why do we need to propagate?


Esta es mi nota.


Oups, need to fix that :/


Keep an eye on the spacing! It'll make it much easier to read/debug your code if you kept a consistent code style :)


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.462.3f7d3ceb38edf2eda7175eb90dcb7ed9cd442435](http://build.palaso.org/viewLog.html?buildId=62344&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3356, ignored: 134 Build time: 00:10:49


Since you document it here, I do not think you really need to describe actual implementations for each child class.


@KozhinM I also prepared a PR #737


:cry:


Ah-ha! Thanks for figuring this out for me. I never would have guessed to try a URL with spaces at the beginning.


I don't believe you want the question mark here.


This block needs to be indented.  And same comment about emphasis: I do not think it helps here, it rather distracts.


The patch is okay.  It is a bit slower than the previous code but I think `bn_mod` is never called in an inner loop so this doesn't matter.  At least it is now much easier to read.

There are some (maybe only one) places related to bip32 where `bn_fastmod` and then `bn_mod` is called on a number that is guaranteed to be < 2*p.  The `bn_fastmod` was only there, because `bn_mod` wasn't constant time.  We can remove this in a later commit, though.


I think this is wrong -- `on_` should correspond to `did`, not `will`.


Is it possible for pod.Status.PodIP at this point to already be set?


ditto for this block


Yeah, I don't want to deal with the issues. Apparently people copy paste examples a lot. Gonna go through my repos and change it when Node 5.0.0 lands with arrow functions.


This might be a bad sample name.


Still not keen on this as it means more places now have to know about the singleton (i.e. any place that uses the save button controller) and it introduces a fragility in that if we ever pass it a saved pages list from anywhere other than the singleton these lists will be able to go out of sync.


Good catch. I clarified the SDK download requirement in 1dc81c4a929c16d12ff9087726eb850c6271b816 . You have to download the SDK to get the visual analysis libraries required for viewshed analysis. That's the only way to get those libraries for now. But you don't have to install the whole SDK. The project uses the online Gradle-based SDK.


can I get a space inserted ?  it's a game breaker for me.
Here's a sample that I need to deal with `BXVU Account#`


Should there be a default returned here if the class can't be loaded? If not, should we just raise this exception instead of warning with it?


fuck Hub and all other repos


I just saw that sanity check for this parameter is missing in the code.


101884090--NewVenting-Availability-Schedule (lgu1234) - x86_64-MacOS-10.9-clang: OK (1808 of 1808 tests passed)

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


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.463.0e67a8d3c3d4c57555cdee60ace0c38bf8defcba](http://build.palaso.org/viewLog.html?buildId=62345&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3356, ignored: 134 Build time: 00:05:54


Why not use memset or calloc?


I installed and started using a week ago and it was working fine. I have updated the firefox and tried to import events then I saw the "Extension type not supported" error. I believe it is the update from FF. I have added another caase and now it is working fine.
Version 40.0.3, first offered to Release channel users on August 27, 2015


You've already got most of this functionality in your `show_list` - you could turn this into a function, for example one with a signature a bit like this: `buildContacts(contacts, search)`. You could pass your AJAX data along to this function as well (optionally) a search parameter: `buildContacts(data, search);`


to get the stack back to the entry point for the request handling... the stack logged here stops where the goroutine was spawned


ooh good call. Thanks Dustin!


oh man...you stole my commit and didn't push a fix for https://github.com/opengapps/opengapps/issues/104 :wink:


These methods return a bool, I thought it could be "visible" to display it with the question mark


yes, that makes sense


Clarified in 84501cd53b50c7dd6978317470cc288c2b302941


No reason really, was just kind of in self-contained mindset when I wrote it. It'd probably generate the same code.


in the case of a panic, I think we want more stack depth than in other places... this was just a move of the function into the apiserver package so we could write the errorJSON on the next line


Note that this is not a valid Java file (missing enclosing class).


Ja, meestal wil je inderdaad rekentools hebben voor het incl. bedrag. Dus als je iets koopt van € 121 wil je weten welk deel BTW is, en welk deel niet. (Trouwens, welk deel niet heb je voor de belastingen meestal niet nodig bij mijn weten.)

Je kunt het dan dus testen met € 121:

- Dit is het bedrag: € 121
- De btw (21%) van het bedrag: € 25,41 (`BTW = bedrag * 0.21`)
- Het bedrag min de BTW (21%): € 95,59 (`bedrag - BTW`)

Dan is de eerste dus niet correct. De tweede berekening is wel ok, alleen omdat de eerste niet goed is, is het antwoord van de tweede ook niet ok.


Can you say why we want REST framework ?


> On Aug 30, 2015, at 11:33 PM, ifrotz <notifications@github.com> wrote:
>
> Do you have any interest in working on glk sound support? That’s something I’ve been putting off for a while that really needs doing (although I admit there’s a lot of code cleanup to be done and the kinds of things you’ve been doing already are very helpful.)
I’ll look into sound support.
>
> Could you explain the common crypto changes? That falls in the tads interpreter part of the source, so that should probably be contributed back to Michael Roberts if you think it’s worthwhile.
CommonCrypto is a system library (literally. libSystem.dylib links to it) that can do some cryptography functions. In fact, Apple recommends using Common Crypto instead of openssl. Using CommonCrypto instead of building tads’, should decrease the size of the executable as the code would be in a system library instead of in the program.
As for porting it back? CommonCrypto is, to my knowledge, exclusive to Apple’s OSes. If the maintainer of tads wants to add the functionality
>
> Craig
>
>
>
> > On Aug 30, 2015, at 11:51 PM, MaddTheSane <notifications@github.com> wrote:
> >
> > Mainly learning. I will admit that my platform of choice is OS X.
> >
> > Sent from my iPad
> >
> > > On Aug 30, 2015, at 10:42 PM, ifrotz <notifications@github.com> wrote:
> > >
> > > Uhm, you’re porting to OS X?
> > >
> > > Would you mind sharing/communicating a bit more about what you’re doing? Is this really just a learning experience or do you have other end goals?
> > > ‘Cause maybe we could collaborate a bit more if our visions align. Although it seems like you have a lore more time than I do right now for hacking.
> > >
> > > Thanks
> > > Craig
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> > —
> > Reply to this email directly or view it on GitHub <https://github.com/MaddTheSane/iosfrotz/commit/c57e97a88f784d7af2e95ffbb9ba2337a3b48005#commitcomment-12969142>.
> >
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/MaddTheSane/iosfrotz/commit/c57e97a88f784d7af2e95ffbb9ba2337a3b48005#commitcomment-12969340>.
>


a função self::clearDefaultScripts( $tmpl ); causa erro quando usuário logado, ele limpa as entradas do script do mootools e qualquer modulo que use essa função para de funcionar, por exemplo um slide show


This assumes that the dictionary is unsorted, but in that case as you noticed there is really not much you can do apart from checking each word in the dictionary until you find what you are looking for. Perhaps a more interesting case is if the dictionary is sorted? Or you can sort it the first time, and then take advantage of that?


@winstonf88 @idgserpro @caduvieira FYI


We can add organization url.


It does not seem to work:
route: 2474197
node: 1055406177
Should show Hl 8.


I was trying to reproduce your issue ;-)


@gabizou I'm not speaking from a development perspective, I'm saying that all users will have Java 6, and while many, fewer users will have Java 8. Keeping Java 6 would remove a possible step in the setup process for users.


In general `Model` names are singular.


91958608DOASEffectOnZoneSizing (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (761 of 761 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-9.45%25-green.svg)


latest pull request


Note: class name is normally Uppercase (`Two` in your case).


Nice!


It's not in the Errors module because it's specific to the Sipowicz class, whereas those errors are used in a more general sense.


@twizmwazin
https://twitter.com/dinnerbone/status/540535986003140608
https://twitter.com/dinnerbone/status/561096484981334016
https://twitter.com/dinnerbone/status/561879446924574720
https://twitter.com/dinnerbone/status/574866831229517824
https://twitter.com/dinnerbone/status/592967371226226688


s/INTERGRATION/INTEGRATION


Yes.  We can probably just have a default of 80 characters wide - this seems to be standard and even if your window size is much larger, it becomes unreadable beyond this.  Once your PR at staticdocs is merged we can change back to using staticdocs usage (perhaps adding code to break up lines) and change parse_rd back as well.


It was just temporary.


In original TS, deployed actors who received a move order would undeploy and then move there, not reject the move order.
This is even more important for tick tanks and artillery, so it would be better to reproduce that behavior.


s/must/may


I would suggest having these two into two different tables since they are two different views of same event. There are can be different set of questions later.


I would claim the build-in facilities are faster. Anyway new _Bucket[nc]() will also work.


Your implementation is not actually tied to the input list being `ArrayList`. You could accept a generic `List` here and everything would still work.


Note that this modifies the list passed as input, which is normally not desirable (or should be clearly documented).


1762-removeSupplyBranchWithComponent-Doc (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


Great work, Swapnil!

>>If a NM is flexed down, it takes time for the RM to realize the NM is no longer up.
     However the Myriad Scheduler has already forgotten about this NM. This difference
     in view between RM and Myriad scheduler was causing less than desired number of nodes
     to be flexed down

I think we should reduce the yarn.nm.liveness-monitor.expiry-interval-ms to something in the range of a few seconds (5 seconds, perhaps). Not for the current change, though.


cpp_fun_call_performance (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (354 of 357 tests passed)

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


Bug fixed:
1. Smooth bot aiming, no remarkable steps/jitter.
2. More fair bot thinktime for every bot, regardless of loading number.

Bug introduced:
1. Less accurate bot movement prediction (obstacles).
2. Broken 'Air control' when travelling 'oppositing' jumppads (q3dm19, q3dm12).

FIX:
Increase ms->thinktime for some movement prediction cases.
New air control, or even better, no air control for all players (-> 'Smokin Guns' movement).


Newbies need automagic more than anyone -- we shouldn't be worrying about teaching vagrant here, but about getting people up and running quickly w/ atomic. I think a page about understanding vagrant in an atomic context would be great, but that's a separate piece of documentation.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.482.0e67a8d3c3d4c57555cdee60ace0c38bf8defcba](http://build.palaso.org/viewLog.html?buildId=62343&buildTypeId=bt223) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3317, ignored: 140 Build time: 00:27:48
### Failed tests
```
Palaso.Tests.dll: Palaso.Tests.Progress.ConsoleProgressTests.LongRunningMethodUsingConsoleHandler_ProducesLog: <no details avaliable>

```


Yes, I think that would be a useful addition.


I still remember when NSKeyValueObservingOptionInitial wasn't available - I just need to remember that it *is* :)


"_hc(b->k) % nc" can be replaced by the much faster "(_hc(b->k) & (_bc - 1))" if _bc is a power of two.
But the speed of iterating over the linked list is proably dominant.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 12](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=135&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


It does work now. It wasn't uploaded to the server, yet.


10 minutes


@CallumMain
Great job on these command line exercises!


That's buggy, you can't do it. ;)

It finds all hl1* states, even hl10, hl11 etc.


Yikes - 5 GBs required for this install...let me find another machine to try this on...


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.539](http://teamcity.monogame.net/viewLog.html?buildId=9883&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Win10) Build time: 00:00:02


If you are curious, the more efficient way of doing this involves using a `PriorityQueue`... :)


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 12](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=135&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **SUCCESS**
Summary: Running Build time: 00:01:08


And is the signal announcing the Hl 8 correct? ;)


Je ne vois pas l'intérêt de dupliquer la définition d'un Repository...autant en objet, avec l'argument de la multiplication des findByXXX, à la limite, mais là, je ne vois pas...il n'y a aucun intérêt à avoir autre chose que Add et Remove.
Sinon ok.


Are you sure? http://developer.android.com/sdk/index.html#Requirements

![image](https://cloud.githubusercontent.com/assets/3091666/9587089/99dc6fa4-4fee-11e5-8765-3e6235a4d8fd.png)


Final grade: B-. It would have been nice to see alternative approaches to some of the issues raised by those problems, and also your assumptions on Ex1 made its implementation a little too straighforward...


I think it is better to not do a hard fail because simulations can still run fine. But there should probably be a default returned.


This PR is to get the column mapping working - I think I will do another PR after this one for things like spaces and special characters in the field name.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.455.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=62348&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:16:46


Done.

It needed `avformat.h`.


Hi hora,

Thank you so much for your comments, I actually haven't finished the
contact app v4, still have some functions to add, I will modify my codes as
your commented.

Brian Lu

On Mon, Aug 31, 2015 at 11:20 AM, hora <notifications@github.com> wrote:

> Keep an eye on the spacing! It'll make it much easier to read/debug your
> code if you kept a consistent code style :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LuBrian/contact-app-v4/commit/1f5d6c1e43e6336941c1173ff05d2e67ff8ebe8c#commitcomment-12982054>
> .
>


Great, thanks for documenting all your assumptions, in real life this is one of the most important things you can do when writing code! :)


`.sav`?


Not very interesting, is it? :)


Looks good to me, you might want to consider making NV12 the default (the one at index 0 or change the default index in try_fmt()).