TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=2936e2071adb44759b4d32e7908c60be
 plone-5.0-python-2.7 [SUCCESS]


Indeed it is not safe and what you suggest would be much better. (I think at that time feature requests did not support expressions - and during MTR I haven't touched the joins code much...)


Timothy, thank you! I will check it right now.
Could you, please, provide me a debug output if possible?

I am sorry for that :(


Please make the name of the part before ".pub" equal to your shell username in the future, as otherwise you break ansible - which prevents the servers from running ansible. Fixed this in 59d64b8a8887b354873e6563339c17a7bc7f8477 and 4fedc26bcb780b76a4ad180a85c2cfedc9a40e7e.


This was a mistake. I am currently working on a ticket to update this page. I'll roll this into that commit and reference this story.


@dreadatour using Package Control https://packagecontrol.io/ It was auto-updated.

Regards


@n0ha please check if this change is also reflected in migration scripts for mysql, we need to be able to update existing installation.


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 39](http://192.168.44.2:8111/viewLog.html?buildId=102&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 12 Build time: 00:06:39
### Failed tests
```
Project Test Suite: Test: testDaemonStatus: <no details avaliable>

```


I'm pretty sure that's what's being done outside of guessing at a
mathematical operator.  There was a bit of exploratory work done by others
to learn how to do things like print to the screen and converting between
number formats, but nothing that really delved into future sections.  At
least from my point of view.

On Thu, Jan 22, 2015 at 9:38 AM, tezyn <notifications@github.com> wrote:

> @vht-hfenton <https://github.com/vht-hfenton> @jdantonio
> <https://github.com/jdantonio> I don't know if this would be well
> received by the group, but I was considering posing the challenge to see if
> anyone wanted to up the learning stakes, and keep track of what is
> introduced in each section of the book, and try to complete the exercises
> with only the building blocks that they have presented up to that point or
> things you explicitly define yourself.
>
> The benefits I would see are two fold, one the user needs to pay very
> close attention to the material and two it may force people to think
> outside of there typical box and have to better consider the little things
> we take for granted.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AkronCodeClub/sicp-study-group/commit/e761fca9440b98cc10ea36826fc52dcd287bd18f#commitcomment-9397791>
> .
>


Yay cheers. Are you gonna sort out the makefile some point, too?


@dreadatour: How do I do that?


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


    ------------------------------------------------------------
    Client connecting to localhost, TCP port 5001
    TCP window size: 2.50 MByte (default)
    ------------------------------------------------------------
    [  3] local 127.0.0.1 port 45420 connected with 127.0.0.1 port 5001
    [ ID] Interval       Transfer     Bandwidth
    [  3]  0.0-10.0 sec  7.92 GBytes  6.80 Gbits/sec


Nah, none of it was showing. I see that you had to modify base.py to get it to work, and it is working now.

Why did this end up in bsc_website/templates/wiki rather than website/templates/wiki?


@dtynn@dtynn


コンストラクタがデータメンバを初期化する順番はclass定義で記述される順番である事が規格上指定されています。
g++ではコンストラクタの初期化子の順序がclass定義上のデータメンバ順と異なる場合にwarningを発するようになっています。このコミットはコンストラクタあるいはデータメンバの順番を入れ替える事でwarningの発生を抑制する為の物です。


Yeah exactly. This makes it possible to set any parameter's class default from the command line by specifying --TaskA-parameter 123

So there's a hierarchy of where it pulls the value. If you instantiate TaskA it will get the value from
1. Direct argument in Python
2. Class value set on the command line (with this PR)
3. Configuration option
4. The default value for the parameter


done


there's the reason to implent this packets in wpp ? just 1 people from TC team can implent PetBatlle - Shauren and i think he'll not do this in neares year


@eea02 / @mkl-eea-sk please comment classes and methods in helpers/dpu/localization


same issue with folders getting the 341 chmod but patch didnt work. Using Revo 232. Whats the path from the root to: /processors/system/filesys/folder/create.php please?
I cant see it.


Ok, I'll make those changes.  As I explained in the chatroom yesteday, I did the various sizes so I could manually manipulate each one and see which ones affected which screens.  I was planning on cleaning that up, but I pushed what I had before all the changes we discussed yesterday so you had something to show the students (in case you had time for it, which you indicated later on that you didn't).


I have another question, why do you use the median to calculate a z-score? In this script the z-score is calculated as  (value - **median**)/standard deviation. Why not use the mean? The standard deviation is calculated based on the mean, so this seems inconsistent.


Do we really want to recommend that as oppose to `opam install --deps-only ocamlorg`?


Well, I must say I'd rather check for the entity and bones existences and early exit instead.
Also, the indentation is broken. Seems more like some quick try to me, right? :)


umm, i think it was correct before... I realize the grammar is confusing though.


Of course there is, it makes implementing new features in TC a lot easier when you can see real values with their meaning as examples or test cases


This commit also redirects to the index when the user, for whatever reason (perhaps to back out of a forbidden URL), goes to the login view despite being active.


and and? :)


The only thing is that with filters it will be evaluated locally while
with a subset-string it could be evaulated on the server.
This can be a major performance impact.
(...Still trying to sell my expression to sql compiler project...)


Planning on it, yes. I have even more interviews to do now, though, so I'm still slightly busy. But I haven't forgotten about it.


Please use `util::make_socket_closeonexec(fd)` here.


@eglelaume what is this extra statement for?


Read proxy from environment variables.


(we will support this this year)
we can start soon, so feel free to go for it :)


Thanks. Reading the docs using RouteState is the way to go on this one. Tests created and fix in play. Should have a new release out this evening. Thanks for sending the feedback, let me know if any other issues or enhancements.


@tdhopper set "debug": true in plugin config, restart ST and open ST console (with Cmd+` on OS X or "View" -> "Show Console" from menu). I need any exceptions or errors.


扩展方法是C#很帅气的功能！如果有
```C#
public class MyClass{}
public static class ExtMed{
   public static MyClass M1(this MyClass c){return c;}
   public static MyClass M2(this MyClass c){return c;}
   public static MyClass M3(this MyClass c){return c;}
}
```
使用时可以有
```C#
var c=new MyClass();
c.M1().M2().M3().M1();
```
这种连点形式在LINQ使用中很常见。例如：
```C#
Enumerable.Range(1,10).Select(x=>x*x).Where(x=>x%3==0).Sum();
```


@eglelaume you should have your own extensions in main.css or other file. Only modify core skeleton when you need to change essential values like breakpoints etc.


Hi Bill,

I think that we have a misunderstanding regarding the proper handling of unknown codes :-)
The way that you have implemented it now resembles the way it is (still) implemented in CipherSuite. In both cases I think it should actually be done as it was originally done in CompressionMethod, i.e. if an unknown code is fed into the getByCode() method it should return null.
In this case, however, returning the NULL compression method leads to the correct behavior during the handshake, i.e. the server will indicate the NULL compression method in its ServerHello message. However, I think that the selection of the NULL compression method in this case (code provided by client is unknown) should better be made explicitly in the ServerHandshaker.receivedClientHello() method because it is reasonable in that context but may be wrong in another context.
I would also decrease the log level in this context since it is a totally expected (and easily handled) case that is not worth pointing out explicitly to operations personnel, is it? I'd rather log at Level.FINE or Level.FINER, i.e. a developer might be interested in this information when debugging the code ...


Maybe this?

```
Reloading plugin /Users/tdhopper/Library/Application Support/Sublime Text 2/Packages/Python Flake8 Lint/Flake8Lint.py
Traceback (most recent call last):
  File "./sublime_plugin.py", line 62, in reload_plugin
  File "./Flake8Lint.py", line 20, in <module>
    from helpers import (
ImportError: cannot import name filename_match
```


    ------------------------------------------------------------
    Client connecting to localhost, TCP port 5001
    TCP window size: 2.50 MByte (default)
    ------------------------------------------------------------
    [  3] local 127.0.0.1 port 39468 connected with 127.0.0.1 port 5001
    [ ID] Interval       Transfer     Bandwidth
    [  3]  0.0-10.0 sec  7.95 GBytes  6.83 Gbits/sec


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=c6c8c0df93044ee1a2a5ba225ea546b2
 plone-5.0-python-2.7 [SUCCESS]


I will fix the indentation. Why is it a dirty try?


Fixes #1


Great :) good luck on those.


We haven't exported any of the XPC work. But you can grab the piece that fixes a bug if necessary.

> On Jan 22, 2015, at 9:42 AM, Scott Haynes <notifications@github.com> wrote:
>
> @allisoncorey @davideaster
> I'll take a look at this issue. Maybe there is something else going on.
>
> PS. - there is a update from an internal branch dealing with the way transforms are being stored in the model driver. David did we ever cherry pick the commits from XPC (contribute or something like that was used for the branch names) ?
> Scott
>
> —
> Reply to this email directly or view it on GitHub.
>


@JanTvrdik Your second version should be compatible across all version. Removing a typehint during inheritance is not an LSP violation.


done


We need a special case in the css for the ".annotationsclass" fill, so that the marker series is always a consistent colour


I want to fix commit comment before we merge.


Well, I did not encounter a problem, but git.apache.org is a slow dumb server.


I've cherry-picked the removal of the defunct function into a separate pull request and updated the documentation.

I think loading the namespace is necessary, since we do not plan to refer to `RANN1` in `DESCRIPTION` for now. Do you know a more elegant way?

I'll get back to you with a benchmark of the costs of `get_nn2_func`.


Can we take this puts out please?


Ok, I'm going to also add an apr-util mirror. Btw. is there any problem with your mail account?


@Binipro: one thing I see (which might just be a copy-paste error) is you didn't quote `pause` or `play` in your second overlay definition. If it turns out that's not the problem, it would be helpful if you could provide a link to your player. Also, I'd [file an issue](https://github.com/brightcove/videojs-overlay/issues) so it doesn't get lost.


r=SimonSapin


No problem.


It is my understanding that the action is always a proxy, so probably ok to take in an array and assign that to callbacks under the hood like Reflux does. I would prefer to have the callback name and action name the same, to avoid doing any mapping in my head. With the assumption that the action will never do any more thinking and the callback name is the same as the action type, the interface could be an array of strings like in Reflux.


This needs to get turned back to 577 again...


Frank and I sent you several mails and go no answers. :-/


Yes, this is the reason, but I removed this imports from file: https://github.com/dreadatour/Flake8Lint/blob/master/Flake8Lint.py

I think the reason is on update Package Control removed helpers.py first and then update Flake8Lint.py, but because of error ST decide to do not work with this plugin anymore. Does ST restart helps?


I think the reason this makes me uncomfortable is that I don't think we should be putting specific APIs for third-party packages in the core files of Daguerre. It'd be like if Django had AWS-storage explicitly coded into django core with some conditionals to check whether you were using AWS or not.

IMO this should be moved to a separate Jinja-specific either as a utility that has to be explicitly imported by projects using Jinja or as a Jinja extension to be added to someone's Jinja environment (c.f., http://django-pipeline.readthedocs.org/en/latest/usage.html#jinja).


You should use m_list for this because otherwise it will crash on not Note objects and you will not get notes that are child of something else (e.g., app/rdg). If you sort the Notes by oct/pitch in filter list, you can also directly get the highest and lowest one by looking at the beginning and end of the list


I would simplify this with an `else if (value instanceof Float) { valid = false; }`


TeamCity Ylab :: synteny/ylab [Build 37](http://192.168.30.225:8111/viewLog.html?buildId=629&buildTypeId=Ylab_SyntenyYlab&guest=1) is now running


Unhandled Fundamental Failure


Justin, Add a line that says what reference is updated (after line 59):

  say "updating title : $_->{title},  uri : $_->{uri}";


saw approval from SimonSapin
at https://github.com/Ms2ger/servo/commit/13c7cf928a5817de315a58a4fa15dc9b7fdc3d7f


merging Ms2ger/servo/rustup-prepare = 13c7cf92 into auto


Ms2ger/servo/rustup-prepare = 13c7cf92 merged ok, testing candidate = 55fbf1f2


Is quite complex that to happen, I've seen no header has the value "null"


Correctly:

say "updating title : $_->{title}, uri : $_->{uri}";


I'm going to have to put this pull request on hold until the weekend. I've been pushing a small set of problems around in what is sometimes called "whack a mole". This work has been complicated by early confidence when the 'refresh' event almost worked so quickly and the belief that the changes can be confined to the localStorage functions. Its time for me to step back and find a more through approach.


You are right. I did initialization from habit. I'll remove this lines.


To be clear, I don't mind having the helper function there. That seems genuinely useful. It's the jingo line and the awkward jingo conditional import that squicks me.


Thanks Alex


Restarting ST doesn't help, nor does reinstalling the package.


lower case 'barlines' please


Ah, good point. I can change that second sed command to grab from the last `#VAGRANT-END` to build the post-vagrant snippet. Of course the funny part of that is that I could have just done that to begin with and that would have fixed the issue.


I copied and pasted https://raw.githubusercontent.com/dreadatour/Flake8Lint/master/Flake8Lint.py and it works fine.


This is backwards


Created pull request laravel/framework#7088


Mostly for teaching @kenwheeler but prefer to break this into a named function to make it easier to keep track that these are arguments to the createStore method above.


Thanks


I will try to release new version now (I think it is ready now after I fixed few errors I want to fix before release) and check it out.


In this particular instance yes, but returning an object allows for a couple of different scenarios:

- If you want to add a property that differentiates between server and client actions
- If you want multiple actionTypes to be possible, such as optimistic updates, or different types based upon success or failure.


    ------------------------------------------------------------
    Client connecting to localhost, TCP port 5001
    TCP window size: 2.50 MByte (default)
    ------------------------------------------------------------
    [  3] local 127.0.0.1 port 56860 connected with 127.0.0.1 port 5001
    [ ID] Interval       Transfer     Bandwidth
    [  3]  0.0-10.0 sec  7.67 GBytes  6.59 Gbits/sec


@SubJunk Thanks! So similar to deselect other not used Pan/Samsung renderers :-)


This is a *much* clearer word than trigger a la reflux.


Really appreciate it. This is such a great tool.


Just a note, in this given scenario, you really could just put the emit change at the bottom, but I typically put it inside the action handlers in case there is a condition in which one would not emit a change.


all included

On Thu, Jan 22, 2015 at 11:46 AM, Krzycho666 <notifications@github.com>
wrote:

> Lighting system for 2D - that's awesome =] How about shadows in 2D space?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/okamstudio/godot/commit/67d357191ff74b2cfc80015941363a97e7ee19fd#commitcomment-9397949>
> .
>


done


Great class!


ah, okay :)


Fixed #120


You're right. Do you think we should also remove the `build` field?


@dmlap : thankyou for your replay ,
the problem is not with the quote, and i am using the new brightcove player.which you can find it in the link
http://docs.brightcove.com/en/video-cloud/brightcove-player/index.html
i created one sample player with my account to test the player.


Really?


i don't use this file anymore. If you don't use it too we can delete it (or stop updating it)


you need also to update component-remotes to the new version: 1.1.11 https://www.npmjs.com/package/component-remotes


You might be interested in TcpServerEventGroupTest.
On my machine I get latencies around 5 micro-seconds for 64 bytes
writes/reads and 2.7 GB/s for 32 KiB writes/reads over loop back, ultra low
garbage.

On 22 January 2015 at 10:56, Richard Thompson <notifications@github.com>
wrote:

> yup - make sense, thanks. I'm spending a lot of time in this area at the
> moment so always on the look out for new approaches - appreciate that this
> work in progress
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OpenHFT/Chronicle-Network/commit/acf2caf8a58b12c980c6c67ef0d44d57ba51a1fa#commitcomment-9394605>
> .
>


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 40](http://192.168.44.2:8111/viewLog.html?buildId=103&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) is now running


TeamCity Ylab :: synteny/ylab [Build 37](http://192.168.30.225:8111/viewLog.html?buildId=629&buildTypeId=Ylab_SyntenyYlab&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 127 Build time: 00:11:43


@kenwheeler why not flatten that out and make each its own action callback? They seem free enough.

```
[ "recipeCreated", "recipeCreatedAlternate1WithLongNameToClarify", "recipeCreatedOptimistic" ]

```

The assumption there is that this is the best place to capture that logic and I'm not sure that's true - shouldn't it live in the component?


```C#
interface
abstract class
class
```
从抽象到具体的阅读顺序可取。


not comfortable with the first patch, if you want to have custom format support i can add an overload that lets you specify the origin encoding, but as it's written here i wouldn't be comfortable merging.


Right. I shouldn't make any change for FIND_PATH. Originally I only made the change for BOOST_REGEX since it uses "set" instead of "FIND_PATH". I will update it.


In one of the next commits i will programmatically override the designer settings with a widget->resize(...) call to get rid of this...


Aha.  I knew I was missing a move there.


Does this commit introduce the "pds->serving?"  What does serving mean?  If it means the same as "related," i'd prefer we stick with that.


@stve It looks like your editor (or possibly `rubocop`) made some changes that were causing the build to break. Reverted in bf6a90d2eb70c29b61de112a308fd0abba8e1484.


My bad. We were trying to figure out why I couldn't login XD. Thanks, I don't use ansible.


Doing that raises a compiler warning that the else if statement can be replaced with what was their prior.  Are you OK with that?


Is there test in there to make sure a json string `"Key": "None"` will be correctly interpreted as Python string `'None'`


Great cleanup, thanks!


Because I felt some speed behind that change, that's all, really. :)
But fixing the indentation isn't enough IMO. Please return instead and let's avoid exceptions when they can be avoided. I'd like the code to act as we read it, and exceptions don't permit actually this.
It's simply a question of good practice IMHO.


can add some comment indicating what is a, b, and c so its easier to understand


@H0neySingh
This file is required to display project/repository information on github site.
Is not required for the project, but may contain the information to build, use, access, permissions, etc., is so useful to maintain the standard of development.

So in my opinion, this file should be maintained.


we need to move the skip test section - https://github.com/owncloud/core/blob/d7d2b68ae145d98ff5293b44dedeaa6b802eabd1/tests/lib/mail.php#L15

... or introduce a second unit test file


How about just defining this in the vcproj itself?


The first step could be to implement support for setFilterNativeExpression() - so we would not need to alter subsetString in the provider.

Then of course expression to sql conversion will be handy!


I feel like the component should worry about itself. Multiple components can act upon an action as long as they are subscribed to it. Doing actions like this sets them up as a sort of schema. It gives you a 10k ft view of what your app can do, the data each method is passing, and because each action returns a promise object, allows you to call an action that hits an API and then dispatches on resolution.


Duh. Of course! DONE in 614cea5b3b195956d34cdd45036e711dc53ab6ba


已处理，这个分支现在不存在了


i don understand why you are resubmitting the form because mine defaults to fetch data from as far as 2000 already


Curious why these needed to take advantage of the getActiveEditor call (I'm sure it was required, just am unclear why). Let's discuss when you get in today.


@dcoffey3296 I appreciate the feedback. I'll be submitting a new pull request here shortly with some updated code that allows the end user to download videos in 4k and also makes sure the week 0 and 6 Friday lectures are downloaded!


Why do you need this comment?


Hi Manolis.  Not sure I'm following.  Did I mess something up?  Are you
suggesting something different/better?

Sorry if I'm being thick..

On Thu, Jan 22, 2015 at 9:23 AM, Manolis Stamatogiannakis <
notifications@github.com> wrote:

> Not a big deal, but short-circuiting configure and make was better.
>
> Now if you try to rebuild PANDA and configure fails, the previously
> generated configuration will be used and chances are you won't even notice.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/moyix/panda/commit/9f60a29311ffcf62833c0aae628d6efa4da2916c#commitcomment-9397465>
> .
>

--
When  I see an adult on a bicycle, I do not despair for the future of the
human race. - H. G.Wells


Oh, maybe because they are only definable on a global level within Ace?


Not sure what you mean...?


Thank you! Glad to hear it :)
And sorry again for this situation.

I am released plugin right now - could you, please, reinstall it from Package Control after a while? (we need to give some time to Package Control to update plugin: https://packagecontrol.io/packages/Python%20Flake8%20Lint should be version 2.0.0)


@stve Out of curiosity, what versions of `ruby`, `rubocop`, and `parser` were you running when (I’m assuming) you ran `bundle exec rake rubocop:auto_correct` (or `bundle exec rubocop --auto-correct`)?


Ship it!


Yeah, I think I want something less obtrusive here.

I agree that it's nice to be able to just build from source without warnings, but I don't want to gum up the source with a bunch of Windows-specific shenanigans either. I feel like Windows is being particularly annoying here: they yell at you for using certain functions but then don't implement enough of C99 to give you a standards-compliant alternative. :angry:

Given that, it might be best just to make it a project setting.


    ------------------------------------------------------------
    Client connecting to localhost, TCP port 5001
    TCP window size: 2.50 MByte (default)
    ------------------------------------------------------------
    [  3] local 127.0.0.1 port 37060 connected with 127.0.0.1 port 5001
    [ ID] Interval       Transfer     Bandwidth
    [  3]  0.0-10.0 sec  7.68 GBytes  6.59 Gbits/sec


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 40](http://192.168.44.2:8111/viewLog.html?buildId=103&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 12 Build time: 00:07:53
### Failed tests
```
Project Test Suite: Test: testDaemonStatus: <no details avaliable>

```


update 1/22/15/10:20


It's more separation of concerns than anything. The action determines whether or not to dispatch the payload, and what to dispatch, whereas the component decides how to act upon that payload and its data.


done


See b162c18 — maybe you want to recommend that in the README (continuing to point to `opam` for info about the packages)?


usually i would keep this method outside instead of nesting it inside another method... do they teach it like this at tealeaf?


We created `['monitoring_hostname']` which we default to site_name. site_name is the attribute name spacing and might be ANY id, and there is no convention or whatsoever do specify it as a FQDN.

However server_name is a clear Vhost attribute which should be a FQDN :
![screen shot 2015-01-22 at 15 19 24](https://cloud.githubusercontent.com/assets/3965231/5858361/80ead38e-a24a-11e4-8700-82abf26f2166.png)

One of the goal of this PR is to prevent the definition of 20 attributes, and now we have a new one.


Very similar method already exists in System.Linq
https://msdn.microsoft.com/en-us/library/vstudio/bb534336.aspx

Your extension would be equivalent to calling SelectMany(i => i)


@kenwheeler "...as long as they are subscribed to it." But components don't subscribe to actions they emit them right?


TeamCity OsmSharp :: GitHub - OsmSharp - PullRequests [Build 91](http://build.osmsharp.com:8080/viewLog.html?buildId=4156&buildTypeId=OsmSharp_GitHubOsmSharpPullrequests) is now running


Justin, This needs to go before line 59 so that that it is also printed when it isn't a dry run. -Robert


You have a semicolon at the end of this line.

This means that the `z = 0` on the next line is actually declaring a global variable, not a local `var`.

This is why I don't like the comma-separated-var-on-each-line paradigm. I like to just put a `var` on every line, with the variable I'm declaring. You avoid this easy mistake.

I would do:

```js
var x = 0;
var y = 0;
var z = 0;
```


Any idea when the Java8 grammar will be functional and not slow?


looks ok, so far http://3v4l.org/CjbSK


That's a big sausage craig!


Why are we skipping the values that are equal to the values of an empty `svg` element?

Shouldn't we just skip the values that are empty?


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=0816e1379c1c44ac93ef513358235075
 plone-5.0-python-2.7 [SUCCESS]


", used by Wikipedia." although important, looks out of place to me and awkward here.  I don't have better suggestion, other than just removing it here and mentioning it elsewhere.


Yeah you're right, the Store acts upon the action, and the components listen to the store.


This is not necessary because `setKeyTapEnabled:` already guards against this. Also, this introduces a bit of tight coupling with the implementation of `setKeyTapEnabled:`. I'm being pedantic, I know :)


much better :)


As I wrote earlier, I'm not very fond of project-wise configurations for libraries. It should be possible to include their source files and without any configuration.

Another possible solution is to differentiate the code as in the following example
```
#if _MSC_VER >= 1400
sprintf_s(message, sizeof(message),
    "Class '%s' may not have more than %d fields, including inherited "
    "ones.", name->value, MAX_FIELDS);
#else
sprintf(message,
    "Class '%s' may not have more than %d fields, including inherited "
    "ones.", name->value, MAX_FIELDS);
#endif
```

Would this be acceptable?


But the assumption is that actions are always coming from a component. I've had scenarios where I have an API module that fires actions as well.


@galenandrew Just FYI, `httpstatuscod.es` doesn't appear to be a thing anymore, but [httpstatus.es](http://httpstatus.es/) seems to be a solid replacement for it.


Why should this be an extra condition?


It's pretty clear how the code acts.


if i return stock_data in download_data_from_yahoo, then I can do
stocks_data_collection += download_data_from_yahoo(page)
without needing to use an instance variable @


Looks good.


maybe use ```case 'package':```


maybe use ```case 'module':```


Btw. extending .gitignore after some files are added to git will not remove them. You will still have undo the changes to `.idea/gradle.xml` added in https://github.com/psycholold/TapToUnlock/commit/dde2b9e21dea91fdbf7f66fac0ca8b01017884ae manually.


See comment for line 69.


done


Yes, that'd be much better than having to fix this time and time again...


Thanks @rex! They must have secured a new domain.


Hmm, placing it on new Line 59 (right after "# say " generic pub :\n"...)
produces the following output if two new titles are updated:
updating title : 0>{title}, uri : 0>{uri}
updating title : 0>{title}, uri : 0>{uri}

for 35 new titles, the same text repeats 35 times.  I'm not sure this is very helpful.


Unzipped with finder / archive util


Unzipped with zip-decompress


can do 1-liner:
no_chinese_data = collection_raw_data.map{|element| element.text.strip.gsub(/[年月日]/, "-")}


all tests pass:
success: http://build.servo.org/builders/linux1/builds/590
success: http://build.servo.org/builders/linux2/builds/589
success: http://build.servo.org/builders/android/builds/626
success: http://build.servo.org/builders/mac1/builds/807
success: http://build.servo.org/builders/mac2/builds/807


+1 on setting up the schema, I like that as well. But wouldn't the promise return in the component? like:

```
recipeCreated: function() {
    recipeAction.recipeCreated(this.refs.somethingGettingSentToApi).then(dispatchActionThatHitsStore)
}
...
onClick="this.recipeCreated"
```

It feels wrong, but where else would the promise return?


I don't think we need to pass `hideThumbnailLink` into the `listItem` macro. This macro already takes care of rendering the title link next to the thumbnail, so this can default to automatically hiding the link for screenreaders


Yeah, I think It’s a programmer error to call startMonitoring twice, but it could be less strict about that.


This might tell you things you already know, but here goes.

This commit failed to build with errors (below).  Note my configure includes `Pism_USE_TR1` set to `ON`.  (Changing that to `OFF` generated different errors.)

```
~/pism/buildnewerror[new-error-handling*]$ make
Scanning dependencies of target pismcalcalcs
Scanning dependencies of target pism_config
Scanning dependencies of target touch_pism_revision
Scanning dependencies of target install_site_packages_in_the_build_directory
[  0%] [  0%] Generating pism_config.nc
Building C object src/calcalcs/CMakeFiles/pismcalcalcs.dir/calcalcs.c.o
[  0%] Built target touch_pism_revision
[  0%] Scanning dependencies of target pismrevision
Built target install_site_packages_in_the_build_directory
[  1%] [  1%] Built target pism_config
Building C object src/calcalcs/CMakeFiles/pismcalcalcs.dir/utCalendar2_cal.c.o
[  2%] Building CXX object src/CMakeFiles/pismrevision.dir/base/pism_revision.cc.o
Linking C shared library libpismcalcalcs.so
Linking CXX static library libpismrevision.a
[  2%] Built target pismcalcalcs
[  2%] Built target pismrevision
Scanning dependencies of target pismutil
[  3%] [  4%] [  4%] [  5%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/options.cc.o
Building CXX object src/CMakeFiles/pismutil.dir/base/util/PetscInitializer.cc.o
Building CXX object src/CMakeFiles/pismutil.dir/base/util/error_handling.cc.o
Building CXX object src/CMakeFiles/pismutil.dir/base/util/Context.cc.o
[  5%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PISMDiagnostic.cc.o
[  6%] Building CXX object src/CMakeFiles/pismutil.dir/base/enthalpyConverter.cc.o
[  6%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/IceGrid.cc.o
[  7%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/ColumnInterpolation.cc.o
[  7%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/DM.cc.o
[  8%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/Viewer.cc.o
[  9%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/Mask.cc.o
[  9%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/NCVariable.cc.o
[ 10%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PISMConfig.cc.o
[ 10%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PISMComponent.cc.o
[ 11%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PISMTime.cc.o
[ 11%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PISMTime_Calendar.cc.o
[ 12%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/PISMVars.cc.o
[ 13%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/Timeseries.cc.o
[ 13%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/iceModelVec.cc.o
[ 14%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/iceModelVec2.cc.o
[ 14%] Building CXX object src/CMakeFiles/pismutil.dir/base/util/iceModelVec2T.cc.o
/home/ed/pism/src/base/util/iceModelVec2.cc: In static member function ‘static pism::IceModelVec2S::Ptr pism::IceModelVec2S::To2DScalar(pism::IceModelVec::Ptr)’:
/home/ed/pism/src/base/util/iceModelVec2.cc:50:31: error: ‘dynamic_pointer_cast’ is not a member of ‘std’
   IceModelVec2S::Ptr result = std::dynamic_pointer_cast<IceModelVec2S,IceModelVec>(input);
                               ^
/home/ed/pism/src/base/util/iceModelVec2.cc:50:31: note: suggested alternative:
In file included from /usr/include/c++/4.8/tr1/memory:50:0,
                 from /home/ed/pism/src/base/util/PISMUnits.hh:27,
                 from /home/ed/pism/src/base/util/io/PIO.hh:22,
                 from /home/ed/pism/src/base/util/iceModelVec2.cc:26:
/usr/include/c++/4.8/tr1/shared_ptr.h:1066:5: note:   ‘std::tr1::dynamic_pointer_cast’
     dynamic_pointer_cast(const shared_ptr<_Tp1>& __r)
     ^
/home/ed/pism/src/base/util/iceModelVec2.cc:50:70: error: expected primary-expression before ‘,’ token
   IceModelVec2S::Ptr result = std::dynamic_pointer_cast<IceModelVec2S,IceModelVec>(input);
                                                                      ^
/home/ed/pism/src/base/util/iceModelVec2.cc: In static member function ‘static pism::IceModelVec2Stag::Ptr pism::IceModelVec2Stag::ToStaggered(pism::IceModelVec::Ptr)’:
/home/ed/pism/src/base/util/iceModelVec2.cc:661:34: error: ‘dynamic_pointer_cast’ is not a member of ‘std’
   IceModelVec2Stag::Ptr result = std::dynamic_pointer_cast<IceModelVec2Stag,IceModelVec>(input);
                                  ^
/home/ed/pism/src/base/util/iceModelVec2.cc:661:34: note: suggested alternative:
In file included from /usr/include/c++/4.8/tr1/memory:50:0,
                 from /home/ed/pism/src/base/util/PISMUnits.hh:27,
                 from /home/ed/pism/src/base/util/io/PIO.hh:22,
                 from /home/ed/pism/src/base/util/iceModelVec2.cc:26:
/usr/include/c++/4.8/tr1/shared_ptr.h:1066:5: note:   ‘std::tr1::dynamic_pointer_cast’
     dynamic_pointer_cast(const shared_ptr<_Tp1>& __r)
     ^
/home/ed/pism/src/base/util/iceModelVec2.cc:661:76: error: expected primary-expression before ‘,’ token
   IceModelVec2Stag::Ptr result = std::dynamic_pointer_cast<IceModelVec2Stag,IceModelVec>(input);
                                                                            ^
make[2]: *** [src/CMakeFiles/pismutil.dir/base/util/iceModelVec2.cc.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [src/CMakeFiles/pismutil.dir/all] Error 2
make: *** [all] Error 2
~/pism/buildnewerror[new-error-handling*]$
```


@KozhinM, Thanks -- this was a good idea.  I have implemented the new solution in 628c564.


Yeah - that makes sense


Woops, change $-> to $_->


Does it support arrays inside arrays inside arrays? (example above)


@KozhinM, I'm pretty confident that we won't need this elsewhere, so I'm going to leave it here.


Message describing changes


Indentation on all of these


Because this forces all apps that use this library to have that key set in the bundle, making it possible to detect those apps (see line 156.)


TeamCity OsmSharp :: GitHub - OsmSharp - PullRequests [Build 4.3.0.91](http://build.osmsharp.com:8080/viewLog.html?buildId=4156&buildTypeId=OsmSharp_GitHubOsmSharpPullrequests) outcome was **FAILURE**
Summary: Tests failed: 17, passed: 447 Build time: 0:0:0
### Failed tests
```
OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceAgainstReference: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceAgainstReferenceBig: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  21.38385316718788d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceAgainstReferenceRealNetwork: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  37.315286549428841d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceBig: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  21.38385316718788d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceOneWayAgainstReference: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  16.332267149909338d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression1: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression2: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  21.38385316718788d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression2Regression1: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression2Regression2: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.Serialization.Sorted.CHRoutingSerializationComparizonTests.RoutingSerializationV2CHRoutingV2ComparisonTestNetwork: Test(s) failed. System.NotImplementedException : The method or operation is not implemented.
   at OsmSharp.Routing.CH.Serialization.Sorted.CHEdgeDataDataSource.GetEdges(UInt32 vertex1, UInt32 vertex2) in c:\14K-SERVER-ROOT\TeamCity\buildAgent\work\927bffbe08a03474\OsmSharp.Routing\CH\Serialization\Sorted\CHEdgeDataDataSource.cs:line 187
   at OsmSharp.Routing.CH.CHRouter.GetEdgeShape(IGraphReadOnly`1 graph, UInt32 from, UInt32 to, ICoordinateCollection& data) in c:\14K-SERVER-ROOT\TeamCity\buildAgent\work\927bffbe08a03474\OsmSharp.Routing\CH\CHRouter.cs:line 1477

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.Serialization.Sorted.CHRoutingSerializationComparizonTests.RoutingSerializationV2CHRoutingV2ComparisonTestNetworkBig: Test(s) failed. System.NotImplementedException : The method or operation is not implemented.
   at OsmSharp.Routing.CH.Serialization.Sorted.CHEdgeDataDataSource.GetEdges(UInt32 vertex1, UInt32 vertex2) in c:\14K-SERVER-ROOT\TeamCity\buildAgent\work\927bffbe08a03474\OsmSharp.Routing\CH\Serialization\Sorted\CHEdgeDataDataSource.cs:line 187
   at OsmSharp.Routing.CH.CHRouter.GetEdgeShape(IGraphReadOnly`1 graph, UInt32 from, UInt32 to, ICoordinateCollection& data) in c:\14K-SERVER-ROOT\TeamCity\buildAgent\work\927bffbe08a03474\OsmSharp.Routing\CH\CHRouter.cs:line 1477

##### there are 7 more failed tests, see build details
```


As the `listItem` will be setting this itself, it's no longer necessary to pass these in.


Makes sense, thanks for the explanation.


An assert is a bit harsh here no? It's not a programmer error if this fails.


@jujugrrr when I looked over it, I couldn't come up with a better solution (they feel very much like different settings - I often want a monitoring Host header to be different). And we are defaulting it to another attribute, so it isn't another required attribute (optional vs. required should be a factor here, I think).


If you don’t mind, that would be great.


See comment for line 255.


saw approval from nickelization
at https://github.com/basho/riak_core/commit/a1cca68e855c3285e0358c4a77464d2a2af36a4d


merging basho/riak_core/bump-clique-tag = a1cca68e into borshop-integration-696-bump-clique-tag


basho/riak_core/bump-clique-tag = a1cca68e merged ok, testing candidate = 235ca173


Let's add a 'needs feedback from support' label and I'll go over this with them at their weekly tech forum.


fast-forwarding master to auto = 55fbf1f2


Yeah, good point. Trying to think what should be done if the client IS null. Throw an runtime exception? Just return as if nothing happened and output a log message? Almost wondering if an NPE is the correct answer, but might be better to declare that in the javadoc (for example, a lot of the IOUtils and StringUtils methods throw an NPE but they explain that in the javadoc). The subject check for null was in there from the previous code where this was called almost every time a request was sent, so having a null subject was a valid use-case.


We need to be extra careful here. On Windows the default encoding is *not* UTF-8 (except when you are in an RStudio project). You can call uiPrefs_.defaultEncoding().getValue() to determine the global (or project specific) default encoding, however you have to sort out whether the current file is actually in a project or not.


With the way you've ordered the strings on the equality check, you don't need the null check on rtype.


Et sinon bonjour et bonne année ^^.


Did you manage to solve the issue by this? I am still getting this exception with this change.


These symlinks are hard? because I know I moved the app out of `wallpaper/wallpaper-app` and when I rand this is was on the desktop, so they never moved when I moved the app.


How do I do that?  [?]

There are two branches with 'contrib' in the ssc branch.

contrib/update-transform-on-animation
contrib/rest-api

Scott

On Thu, Jan 22, 2015 at 10:03 AM, davideaster <notifications@github.com>
wrote:

> We haven't exported any of the XPC work. But you can grab the piece that
> fixes a bug if necessary.
>
> > On Jan 22, 2015, at 9:42 AM, Scott Haynes <notifications@github.com>
> wrote:
> >
> > @allisoncorey @davideaster
> > I'll take a look at this issue. Maybe there is something else going on.
> >
> > PS. - there is a update from an internal branch dealing with the way
> transforms are being stored in the model driver. David did we ever cherry
> pick the commits from XPC (contribute or something like that was used for
> the branch names) ?
> > Scott
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/virtual-world-framework/vwf/commit/e6fc48e6af9d59affbaaba67b6d731cd338c6f42#commitcomment-9398327>
> .
>


Thanks I just made the changes

On Thursday, January 22, 2015, Andrew Joslin <notifications@github.com>
wrote:
> You have a semicolon at the end of this line.
>
> This means that the z = 0 on the next line is actually declaring a global
variable, not a local var.
>
> This is why I don't like the comma-separated-var-on-each-line paradigm. I
like to just put a var on every line, with the variable I'm declaring. You
avoid this easy mistake.
>
> I would do:
>
> var x = 0;
> var y = 0;
> var z = 0;
>
> —
> Reply to this email directly or view it on GitHub.<
https://ci4.googleusercontent.com/proxy/W_0cfsdUa9Rcbe2yx_NZMuZN7JK63mw-M6yb0gC_fvuS6D_g_-e3KP3ParhyjgWouFNCuEw336Vuy1nNa4r_nk9z3Mbi2cppzur9KFWBYVgGO-Oi2ydQA81z9lJ9RVrqQ1SnQ0TOwBISyLNi5gu6CaCcD19JMQ=s0-d-e1-ft#https://github.com/notifications/beacon/AHg6E7nKb6TOoj80fqYcL5EkPsSHnvmFks5nkQ6wgaJpZM4DV1-T.gif
>

--
Lydia E. Joslin
Master of Science Student, Technical Communication
North Carolina State University
lejoslin@ncsu.edu


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=1291f301e0b641f581f8cab9460d1977
 plone-5.0-python-2.7 [SUCCESS]


Let's discuss this whole set of commits when you get in (just to make sure we are calling through the right codepaths and have handled all of the encoding issues properly)


more about the @ symbol here http://apple.stackexchange.com/questions/42177/what-does-signify-in-unix-file-permissions


my comment on a java-buildpack issue about overriding java-buildpack config without forking it: https://github.com/cloudfoundry/java-buildpack/issues/122#issuecomment-71039290


Done in 5f5890e6e3968080fd26e4dbd420f6400bdacd78.


done


Fair enough, I won't uselessly argue for small bits. At least, you have my point of view.


Yeah, I was originally raising a fatal Chef exception, but that didn't give a chance to handle it, so I went back to just using a string. This seems to give the right behavior (if no handler, chef run bombs).


I doubt this is valid: you have a merge conflict from the stash that hasn't been resolved.


https://github.com/MassiveCraft/Factions/commit/c81d7d8794d80846cfc8d11cfe1df98367c868d5#commitcomment-9391609
> Sorry. I tried using Maven but it was just slow, buggy and hard to work with in comparison to just exporting the jar using the built in feature in Eclipse.

Can you please explain what part of the process was slow, what bugs you experienced, and what was harder?


Reviewed PR and looks good.


    ------------------------------------------------------------
    Client connecting to localhost, TCP port 5001
    TCP window size: 2.50 MByte (default)
    ------------------------------------------------------------
    [  3] local 127.0.0.1 port 55977 connected with 127.0.0.1 port 5001
    [ ID] Interval       Transfer     Bandwidth
    [  3]  0.0-10.0 sec  8.01 GBytes  6.88 Gbits/sec


# Based on my understanding of Gem Mechanize, a, b, c are selector of gem Mechanize; Using command of gem Machanize "pp page",  Gem Mechanize will give view to select links, forms and other components. Each form will have name a, b,c in mechanize gem.
Please point out my misunderstanding if any. OrZ


Давно уже хотел добавить подобное колдунство. Теперь все нинужные и ёмкие данные постов будут удаляться после использования.


I am not sure, the first branch is taken for CoreMethodSourceSection (builtin methods), which is likely to not be well separated on core classes, but be advantageous for pure Ruby classes.
We might want to support reporting the Java file and line of builtins in the future.
Also sourceSection is already _compilation final_ in respect to `method` here.
Anyway I see this as a debugging tool rather than anything fast-path.


Good question, but either way, there's a single value used for the whole cluster, so I'm not sure about the load distribution part. Also, I'm not sure if the original intention was to actually have this run at the same time on all headnodes... Perhaps adding a check to no-op unless you're the VIP holder would be better? I'll defer to @caiush


code below gets a little messy, if i were to do this, i would try to parse each row (tr) instead of each yfnc_tabledata1

each tr will have an array of td's, and you already know td[0] corresponds to the date, td[1] corresponds to the open, etc., so you can build an array of hashes right away instead of looping through it a few times deleting elements and then building the hashes from the array elements


Thanks! :smile:


can combine with above: /股利|股票分拆/


Agree strongly, I've already wanted to get rid of indirect and scale_sep. I will get rid of the options and use a better logic for deciding when to use indirect and when not to.

To be honest, the only thing scale_sep is really used for in the current code is as a crude “is it supported?” check, because it gets disabled if FBOs are not supported. But that should/could be refactored.


hm... not sure what you're trying to do here


sorry for that.
overload is fine. thank you.
i just want to fix a bug for me.
try this:
//D
//my computer default encoding is GB2312
private static Encoding defaultEncoding = Encoding.GetEncoding("GB2312");
private static void Main(string[] args)
{
    string msg = "{msg:\"汉字\"}";
    string msg1 = ConvertFromAscii(ConvertToAscii(msg));
    Console.WriteLine("msg: {0}; msg1: {1}; msg == msg1: {2}", msg, msg1, msg==msg1);
    Console.ReadLine();
}
private static Byte[] ConvertToAscii(String unicode){
    byte[] bytes = defaultEncoding.GetBytes(unicode);
    return Encoding.Convert(defaultEncoding, Encoding.ASCII, bytes);
}
private static String ConvertFromAscii(Byte[] bytes)
{
    return Encoding.Default.GetString(Encoding.Convert(Encoding.ASCII, defaultEncoding, bytes));
}


> I often want a monitoring Host header to be different

Good feedback, usually I need my host header to be the one I use for my App, as I want to test my App not another Vhost.
Do you have a more specific example?


Good catch!


I'd consider using `Date.now()` rather than `new Date()` - otherwise we're at the mercy of how Date.toString() chooses to encode the string.


Also in flatMap.js?


Which exception? Not sure what you mean, but in this commit, the intention was to just move the get_config calls to a template where they will always get late resolution


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=1e9de37c1883488d9e718400b980bdf9
 plone-5.0-python-2.7 [SUCCESS]


Agreed, this is a bit of an SPMediaKeyTap leftover. It is targeted at the library maintainer though, not the library user. (It can happen if CGEventTapCreate gets passed wrong arguments.)


This was already removed in a later commit.


Yes, i should not resubmit the form. This is my error.
I am concerned the default is universal. That's why i would like to submit once to confirm it  to fetch data from 2001


A very common example would be for monitoring a webapp. I might want to put a php script into the document root that dumps out diagnostic data, but I would always put it on a separate vhost than the production app, so that it wasn't exposed to the public, and so I could restrict it to specific users or subnets. This is very common to do in Tomcat with an extra JSP, or even for load balancers to probe a health check file.


Hola que tal


I'm not terribly fond of having two execute methods (two different return types feels like it violates some design principals), but the DTOs themselves look fine.

Perhaps a minor optimization we could make there is by explicitly setting the bucket sizes on the maps since we know how big they're going to be.


as per previous comment - `Date.now()` rather than `new Date()`


Nice use of regroup and the parliamentary session :)


bro @hariawan define variable sebisa mungin cameCase ya


Tea-leaf never teach method nesting inside another method. This is my mistake too.


Me pica el ano


I would love to, but I don't think there's a way to drop comments in a json file... I can lob one to another file somewhere or perhaps just open an Issue here on GH to track it?


I don't see any reason to use a raw string literal here (backquotes).  Can't this just be `"[\n"`?


@andibraeu Die Community-Datei "brilon" ist nicht mehr verfügbar (404). Bitte behebt das Problem möglichst zeitnah. Thx!


83442720-ERVZoneMult-Issue4338 (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1591 of 1592 tests passed)

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


During the development, i was confused about scoping. Even later methods, i do not an instance variable again.
However, i did not refactor it properly. That's it still have .


When consensus sequence isn't found, consensus_seq_lookup should have been defaultdict. It was saying it was NonType instead. That lead me to discover this dreadful mistake. Fixing it has made the NoConsensusSeqsError unnecessary, as it no longer fails even if the defaultdict is empty.


Nothing serious, just a pinch of user-friendliness.

`make` should be executed only when `configure` runs successfully. The short-circuit operator (`&&`) used to enforce this.
Now, an attempt to run `make` will be made anyway. If `config-host.mak` has not been properly removed (i.e. run `make distclean` prior to `build.sh`) the old configuration will be reused without the user noticing.

I have re-added the short-circuit in #40, so you can ignore the issue until it is reviewed.


I think I'd probably have made this `pombola/kenya/management/commands/kenya_scrape_bills.py` so it doesn't even appear in the `manage.py` help if you're using a different COUNTRY_APP setting.


    ------------------------------------------------------------
    Client connecting to localhost, TCP port 5001
    TCP window size: 2.50 MByte (default)
    ------------------------------------------------------------
    [  3] local 127.0.0.1 port 46733 connected with 127.0.0.1 port 5001
    [ ID] Interval       Transfer     Bandwidth
    [  3]  0.0-10.0 sec  7.76 GBytes  6.67 Gbits/sec


Ref https://bugs.php.net/bug.php?id=52906
Ref http://www.matyasdanter.com/2010/12/elliptic-curve-php-oop-dsa-and-diffie-hellman/ ¤ 5.1.2


Thx.


chupala

Nicolas Matias del Río – IT

Tel: +54 11 5236-7007                     <http://www.facebook.com/VKsurGroup>   |  <http://www.linkedin.com/company/424938?trk=tyah>   |  <http://www.twitter.com/VKSur>

 <mailto:nicolas.delrio@vksur.com> nicolas.delrio@vksur.com |   <http://www.vksur.com/> http://www.vksur.com

De: Systemas [mailto:notifications@github.com]
Enviado el: jueves, 22 de enero de 2015 12:50 p.m.
Para: VKsurGroup/SL90010
Asunto: Re: [SL90010] 12:42 (cb123b0)

Hditditxigxitclbc4agdjgi

—
Reply to this email directly or view it on GitHub <https://github.com/VKsurGroup/SL90010/commit/cb123b0d3ea222802dddb4f4d7af034c7ce97cd5#commitcomment-9399379> .  <https://github.com/notifications/beacon/AKHKwN-on6BFSRI0GpDgoLucPuhBV5X3ks5nkROxgaJpZM4DV2m7.gif>


Comemos?


Oh that's what the heck that was about.  I'll put it back in!  Sorry.

On Thu, Jan 22, 2015 at 10:50 AM, Manolis Stamatogiannakis <
notifications@github.com> wrote:

> Nothing serious, just a pinch of user-friendliness.
>
> make should be executed only when configure runs successfully. The
> short-circuit operator (&&) used to enforce this.
> Now, an attempt to run make will be made anyway. If config-host.mak has
> not been properly removed (i.e. run make distclean prior to build.sh) the
> old configuration will be reused without the user noticing.
>
> I have re-added the short-circuit in #40
> <https://github.com/moyix/panda/pull/40>, so you can ignore the issue
> until it is reviewed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/moyix/panda/commit/9f60a29311ffcf62833c0aae628d6efa4da2916c#commitcomment-9399380>
> .
>

--
When  I see an adult on a bicycle, I do not despair for the future of the
human race. - H. G.Wells


done


Can't the getParentProject-method be deleted alltogether? Didn't https://github.com/jenkinsci/git-parameter-plugin/commit/9e7483c7d2f58aa27f1c2d5c62759c169ae5b5d1 deprecate it?


No worries. I only noticed because I got a conflict for #40 :)


I do like the GLOBALs here, thanks.

We'll see if the Hash[] solution makes it fast enough. :D  Looks good enough for now, though, thanks. Could you commit this to master?


I didn't actually know about `bundle exec rake rubocop:auto_correct` so these were all changed manually. :sob:

As for what happened, I think I found the culprit - https://github.com/daryltucker/MagiclessQuotes I use it to convert magic quotes, but it looks like it also converts ellipsis. Sorry about that!


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=c86cced9c5d44b9391f2f6479bb79718
 plone-5.0-python-2.7 [SUCCESS]


@sferik thanks for fixing the build! I uninstalled the package so that won't happen again.


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 41](http://192.168.44.2:8111/viewLog.html?buildId=104&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) is now running


Do you use emcee 2.1.0? We'll need to add it to requirements.txt


Ah, right. I mean the exception that might be thrown by `get_config()`


Correct, because by the end of the test, there are thousands of files in `dir`.


@stve I have another theory for what may have caused this issue. Have you set the `LANG` or `LC_ALL` environment variables? If set to `C`, this causes Ruby to use `US-ASCII` as the default external encoding, which would cause this type of error.


Use proper spaces between string literals. Refer the segment >> if(device!=null){


what is triangle? Is it this: https://pypi.python.org/pypi/triangle/ ?


Because it's discardable. When you import this file, you want to refresh what you have with new, fresh data, don't you?


TeamCity OsmSharp :: GitHub - OsmSharp - PullRequests [Build 92](http://build.osmsharp.com:8080/viewLog.html?buildId=4157&buildTypeId=OsmSharp_GitHubOsmSharpPullrequests) is now running


@Ceiu Couldn't agree more on the two exec and return types. :(

That's the primary reason I didn't like this code.

I'll look to see if I can do this a little better.


fd


Formatting issues need to be fixed. Refer the segment >> }else{


It's code golf, really, but I guess this could be date_string = DATE_FIXES.get(date_string, date_string)


Jordan, Vallmer                                          vallmer


Darnit, I made that change, then started playing around with Lazy.js and apparently reverted too many files :/.  I'm generating new results now, so I'll re-make the change in flatMap.js along with the updated results.


Since `0xA0` is just newline, just `.strip()` would work as well, and might catch a whole lot of other random leading / trailing whitespace problems in the future


You can also use the && here to have just one select, like this: `select{ |x| x.is_a? String && x == x.downcase}`


maybe we should use a deque<> here.. should will avoid a memmove, or whatever the vector<> is doing when erasing the front element


TeamCity OsmSharp :: GitHub - OsmSharp - PullRequests [Build 93](http://build.osmsharp.com:8080/viewLog.html?buildId=4158&buildTypeId=OsmSharp_GitHubOsmSharpPullrequests) is now running


Should be VPCS instance deleted ;)


Woot!


"request"  or "op" (an item, not a log)


I am not sure whether there has to be check for instanceof since I set that message in a converter and the worst scenario which can happen is that it will set message to null for AssumptionViolatedException


@andibraeu Die Community-Datei "esslingen" ist nicht mehr verfügbar (404). Bitte behebt das Problem möglichst zeitnah. Thx!


TeamCity OsmSharp :: GitHub - OsmSharp - PullRequests [Build 94](http://build.osmsharp.com:8080/viewLog.html?buildId=4159&buildTypeId=OsmSharp_GitHubOsmSharpPullrequests) is now running


2014年09月1日	140.400	142.600	140.200	141.600	2,351,400	141.600
2014年9月1日	0.638 股利

take 0001 as a example, scrapping from yahoo stock website, will scrapping two td accidentally. which are undesired.
Therefore, i have to remove td 2014-9-1 and td 0.638 股利 which are undesired my database.


hi there! is there anything I can do for you?


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_core/builds/1476


    ------------------------------------------------------------
    Client connecting to localhost, TCP port 5001
    TCP window size: 2.50 MByte (default)
    ------------------------------------------------------------
    [  3] local 127.0.0.1 port 34769 connected with 127.0.0.1 port 5001
    [ ID] Interval       Transfer     Bandwidth
    [  3]  0.0-10.0 sec  8.00 GBytes  6.87 Gbits/sec


i think we need either is_tier() || is_base(), or we do this unconditionally...


@andibraeu Die Community-Datei "hemer" ist nicht mehr verfügbar (404). Bitte behebt das Problem möglichst zeitnah. Thx!


@Pugio ah, ok. Well, it's a pretty interesting edge-case I hadn't considered, so thanks for finding it!


Why not use is_numeric() here?  Way more readable


done


Thanks! I think it should build now (I just pushed some changes).


This works, but there's a much simpler solution using symbol-to-proc: `people.map(&:age).inject(:+)`


【参考】
http://blog.h13i32maru.jp/entry/2014/12/29/163209


@nikic Removing typehint does result in `E_STRICT` error, see http://3v4l.org/6ARJm

@staabm Yes, but only because 3v4l does not have the latest snapshot. This commit is less than 2 days old.


Nice job writing your own test here


Looks cool. Feel free to send a PR so I can test the code and give feedback!


TeamCity CI Experimen :: PHP Symfony2 project :: Symfony Project Test [Build 41](http://192.168.44.2:8111/viewLog.html?buildId=104&buildTypeId=CiExperimen_PhpSymfony2project_SymfonyProjectBuild) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 12 Build time: 00:08:18
### Failed tests
```
Project Test Suite: Test: testDaemonStatus: <no details avaliable>

```


+1 on removing the TODO statements


I might be missing something, but if we run this subsequently, won't we end up with duplicate bills in the database? I guess not appending to `bills` if there already is a Bill with that `source_url` (or maybe `title`?) is already present would be sufficient.


TeamCity OsmSharp :: GitHub - OsmSharp - PullRequests [Build 4.3.0.92](http://build.osmsharp.com:8080/viewLog.html?buildId=4157&buildTypeId=OsmSharp_GitHubOsmSharpPullrequests) outcome was **FAILURE**
Summary: Tests failed: 15, passed: 449 Build time: 0:0:0
### Failed tests
```
OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceAgainstReference: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceAgainstReferenceBig: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  21.38385316718788d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceAgainstReferenceRealNetwork: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  37.315286549428841d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceBig: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  21.38385316718788d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceOneWayAgainstReference: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  16.332267149909338d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression1: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression2: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  21.38385316718788d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression2Regression1: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression2Regression2: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.Serialization.Sorted.CHRoutingSerializationComparizonTests.RoutingSerializationV2CHRoutingV2ComparisonTestNetwork: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  20.535391644169415d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.Serialization.Sorted.CHRoutingSerializationComparizonTests.RoutingSerializationV2CHRoutingV2ComparisonTestNetworkBig: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  46.883092369547114d

##### there are 5 more failed tests, see build details
```


Using util::make_socket_closeonexec(fd) as sugested


ok. If you prefer, I will do that.


@vaicine can you just change it so the optional params are a hash instead? now there is more than one param it makes more sense to just pass a hash and have them passed through. you should slice the hash though so we only pass known params. cheers


Mit den Änderungen an der Datei Model.php geht bei mir die Dateiverwaltung wieder.
Ich kann Ordner erstellen und Dateien bearbeiten UND ich sehe bei Bildern die Bild-/Billdschwerpunktsansicht. Das war davor nicht der Fall.

VG


@hwoarangmy Here, I must say @oyvindln has a point in this specific use of exception. (Yeah, I know I can contradict myself sometimes but only stupidos never change their minds, right? ;]).
Why?
Because it is making simpler to know whether a command argument is anyhow wrong.
So in that case, I agree about the boost::lexical_cast use.

Now, Helper:: functions should be enforced when a default value should be returned in any case, to make sure of the code behaviour when possible.
In the mana word older client, we even used to add a default return value as parameter when we wanted to make sure of the return values bound, for instance, btw.


TeamCity OsmSharp :: GitHub - OsmSharp - PullRequests [Build 4.3.0.94](http://build.osmsharp.com:8080/viewLog.html?buildId=4159&buildTypeId=OsmSharp_GitHubOsmSharpPullrequests) outcome was **FAILURE**
Summary: Tests failed: 15, passed: 449 Build time: 0:0:0
### Failed tests
```
OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceAgainstReference: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceAgainstReferenceBig: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  21.38385316718788d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceAgainstReferenceRealNetwork: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  37.315286549428841d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceBig: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  21.38385316718788d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceOneWayAgainstReference: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  16.332267149909338d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression1: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression2: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  21.38385316718788d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression2Regression1: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.CHEdgeDifferenceComparisonTests.TestCHEdgeDifferenceRegression2Regression2: Test(s) failed.   Expected: not null
  But was:  null
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.Serialization.Sorted.CHRoutingSerializationComparizonTests.RoutingSerializationV2CHRoutingV2ComparisonTestNetwork: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  20.535391644169415d

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.CH.Serialization.Sorted.CHRoutingSerializationComparizonTests.RoutingSerializationV2CHRoutingV2ComparisonTestNetworkBig: Test(s) failed.   Reference and resolved are more than 10cm apart.
  Expected: 0.0d +/- 0.10000000000000001d
  But was:  46.883092369547114d

##### there are 5 more failed tests, see build details
```


    options.slice(*ALLOWED_TRACKER_OPTIONS)

will remove one `if`


Cool, thanks :)


It looks like it's just that one commit: 276a789. I would just use that info as a reference and redo it.

The one just before it about visible might also be needed: 77b4d80.

> On Jan 22, 2015, at 10:39 AM, Scott Haynes <notifications@github.com> wrote:
>
> How do I do that? [?]
>
> There are two branches with 'contrib' in the ssc branch.
>
> contrib/update-transform-on-animation
> contrib/rest-api
>
> Scott
>
> On Thu, Jan 22, 2015 at 10:03 AM, davideaster <notifications@github.com>
> wrote:
>
> > We haven't exported any of the XPC work. But you can grab the piece that
> > fixes a bug if necessary.
> >
> > > On Jan 22, 2015, at 9:42 AM, Scott Haynes <notifications@github.com>
> > wrote:
> > >
> > > @allisoncorey @davideaster
> > > I'll take a look at this issue. Maybe there is something else going on.
> > >
> > > PS. - there is a update from an internal branch dealing with the way
> > transforms are being stored in the model driver. David did we ever cherry
> > pick the commits from XPC (contribute or something like that was used for
> > the branch names) ?
> > > Scott
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/virtual-world-framework/vwf/commit/e6fc48e6af9d59affbaaba67b6d731cd338c6f42#commitcomment-9398327>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


Higher!


Is this the commit that fixes the 1px white border on the full screen videos?


Is it safe to rely on a symbol that may not be there? The usual approach in Qt is to look up the symbol dynamically using `QLibraryLoader`? That would be a bit safer.


thinking of it, why did we ever put this in core anyway? If a module requires OPER they should do this themselves.


thanks.  fixing confirmal references elsewhere is also on our list.


* You can use `sed "…" < $file` instead, which removes the need for `cat`.
* You should quote `$file`, it might contain spaces: `"$file"`
* You can use a different delimiter for sed, for example `%`. This is useful when matching URLs, e.g.: `s%http://%%` instead of `s/http:\/\///`


1: D200 One-line docstring should not occupy 3 lines
4: L102 copyright year is outdated, expected 2015 but got 2014
29:1: I101 Imported names are in the wrong order. Should be CFG_BIBFORMAT_FORMAT_TEMPLATE_EXTENSION, CFG_BIBFORMAT_TEMPLATES_PATH
30:6: E126 continuation line over-indented for hanging indent
32:1: I100 Imports statements are in the wrong order. from invenio.modules.formatter should be before from invenio.modules.formatter.config
35: D401 First line should be imperative: 'Return', not 'Returns'
35: D205 Blank line missing between one-line summary and description
60:52: E231 missing whitespace after ':'
61:49: E231 missing whitespace after ':'
62:48: E231 missing whitespace after ':'
68:43: E261 at least two spaces before inline comment
68:44: E262 inline comment should start with '# '
73:52: E231 missing whitespace after ':'
74:49: E231 missing whitespace after ':'
75:48: E231 missing whitespace after ':'
79:5: E303 too many blank lines (3)
83:1: E302 expected 2 blank lines, found 1
83: D401 First line should be imperative: 'Return', not 'Returns'
83: D205 Blank line missing between one-line summary and description
113:16: E127 continuation line over-indented for visual indent
117:57: E231 missing whitespace after ':'
118:61: E231 missing whitespace after ':'
119:57: E231 missing whitespace after ':'
125:57: E231 missing whitespace after ':'
126:61: E231 missing whitespace after ':'
127:57: E231 missing whitespace after ':'
135:24: E713 test for membership should be 'not in'
147: D401 First line should be imperative: 'Return', not 'Returns'
147: D400 First line should end with '.', not 't'
165:32: E225 missing whitespace around operator
190:1: E302 expected 2 blank lines, found 1
190: D401 First line should be imperative: 'Return', not 'Returns'
190: D205 Blank line missing between one-line summary and description
206:5: F841 local variable 'tags' is assigned to but never used
207:53: E261 at least two spaces before inline comment
207:54: E262 inline comment should start with '# '
210:79: E261 at least two spaces before inline comment
210:80: E262 inline comment should start with '# '
212:17: E261 at least two spaces before inline comment
212:18: E262 inline comment should start with '# '
213:52: E261 at least two spaces before inline comment
213:53: E262 inline comment should start with '# '
217:58: E231 missing whitespace after ':'
218:62: E231 missing whitespace after ':'
219:13: B901 blind except: statement
229:1: E302 expected 2 blank lines, found 1
229: D401 First line should be imperative: 'Return', not 'Returns'
229: D400 First line should end with '.', not 'e'
229: D205 Blank line missing between one-line summary and description
256:41: E231 missing whitespace after ':'
257:45: E231 missing whitespace after ':'
258:45: E231 missing whitespace after ':'
261:5: E303 too many blank lines (2)


1: D400 First line should end with '.', not 'd'
1: D205 Blank line missing between one-line summary and description
4: L102 copyright year is outdated, expected 2015 but got 2012
28:1: I100 Imports statements are in the wrong order. import time should be before import zlib
34: D401 First line should be imperative: 'Return', not 'Returns'
49:1: E302 expected 2 blank lines, found 1
49: D401 First line should be imperative: 'Return', not 'Returns'
65:1: E302 expected 2 blank lines, found 1
65: D401 First line should be imperative: 'Return', not 'Returns'
65: D400 First line should end with '.', not 'e'
73:16: E225 missing whitespace around operator
78:1: E302 expected 2 blank lines, found 1
78: D401 First line should be imperative: 'Return', not 'Returns'
78: D400 First line should end with '.', not ','
78: D205 Blank line missing between one-line summary and description
87:16: E225 missing whitespace around operator
92:1: E302 expected 2 blank lines, found 1
92: D401 First line should be imperative: 'Return', not 'Returns'
104:1: E302 expected 2 blank lines, found 1
104: D401 First line should be imperative: 'Return', not 'Returns'
104: D400 First line should end with '.', not 'e'
112:16: E225 missing whitespace around operator
117:1: E302 expected 2 blank lines, found 1
117: D401 First line should be imperative: 'Return', not 'Returns'
129:1: E302 expected 2 blank lines, found 1
149: D401 First line should be imperative: 'Return', not 'Returns'
159:17: E225 missing whitespace around operator
162:16: E225 missing whitespace around operator
167:1: E302 expected 2 blank lines, found 1
187:1: E302 expected 2 blank lines, found 1
187: D401 First line should be imperative: 'Remove', not 'Removes'
187: D400 First line should end with '.', not "'"
206:1: E302 expected 2 blank lines, found 1
206: D401 First line should be imperative: 'Return', not 'Returns'
206: D202 No blank lines allowed *after* function docstring, found 1
206: D400 First line should end with '.', not 'e'
223:1: E302 expected 2 blank lines, found 1
223: D401 First line should be imperative: 'Set', not 'Sets'
223: D400 First line should end with '.', not 'e'
241:1: E302 expected 2 blank lines, found 1
241: D401 First line should be imperative: 'Return', not 'Returns'
241: D400 First line should end with '.', not 'e'
257:1: E302 expected 2 blank lines, found 1
257: D401 First line should be imperative: 'Set', not 'Sets'
257: D400 First line should end with '.', not 'e'
276: D401 First line should be imperative: 'Set', not 'Sets'
276: D400 First line should end with '.', not 'e'
296: D401 First line should be imperative: 'Set', not 'Sets'
296: D202 No blank lines allowed *after* function docstring, found 1
322:54: E261 at least two spaces before inline comment
324:15: E225 missing whitespace around operator
324:41: E225 missing whitespace around operator
334:1: E302 expected 2 blank lines, found 1
334: D400 First line should end with '.', not 't'
350:1: E302 expected 2 blank lines, found 1
350: D401 First line should be imperative: 'Return', not 'Returns'
350: D400 First line should end with '.', not "'"
350: D205 Blank line missing between one-line summary and description
365:29: E261 at least two spaces before inline comment
367:28: E261 at least two spaces before inline comment
382: D103 Docstring missing


1: D400 First line should end with '.', not 'n'
23: D203 Expected 1 blank line *before* class docstring, found 0
23: D400 First line should end with '.', not 'l'
26: D401 First line should be imperative: 'Print', not 'Prints'
26: D202 No blank lines allowed *after* method docstring, found 1
26: D400 First line should end with '.', not 's'


4: L102 copyright year is outdated, expected 2015 but got 2014
101:41: E126 continuation line over-indented for hanging indent
113:37: E126 continuation line over-indented for hanging indent
299: D103 Docstring missing


26:1: I100 Imports statements are in the wrong order. from invenio.ext.sqlalchemy should be before from sqlalchemy.exc
31:1: I101 Imported names are in the wrong order. Should be Bibfmt, Format, Formatname
34: D401 First line should be imperative: 'Return', not 'Returns'
51: D401 First line should be imperative: 'Return', not 'Returns'
69: D401 First line should be imperative: 'Return', not 'Returns'
69: D400 First line should end with '.', not 'e'
83: D401 First line should be imperative: 'Return', not 'Returns'
83: D400 First line should end with '.', not ','
83: D205 Blank line missing between one-line summary and description
100: D401 First line should be imperative: 'Return', not 'Returns'
110: D401 First line should be imperative: 'Return', not 'Returns'
110: D400 First line should end with '.', not 'e'
124: D401 First line should be imperative: 'Return', not 'Returns'
153: D401 First line should be imperative: 'Return', not 'Returns'
173: D401 First line should be imperative: 'Return', not 'Returns'
187: D401 First line should be imperative: 'Set', not 'Sets'
187: D400 First line should end with '.', not 'e'
210: D401 First line should be imperative: 'Return', not 'Returns'
251: D401 First line should be imperative: 'Remove', not 'Removes'
251: D400 First line should end with '.', not "'"
270: D401 First line should be imperative: 'Return', not 'Returns'
270: D400 First line should end with '.', not 'e'
282: D401 First line should be imperative: 'Set', not 'Sets'
282: D400 First line should end with '.', not 'e'
295: D401 First line should be imperative: 'Return', not 'Returns'
295: D400 First line should end with '.', not 'e'
312: D401 First line should be imperative: 'Set', not 'Sets'
312: D400 First line should end with '.', not 'e'
326:1: E303 too many blank lines (3)
326: D401 First line should be imperative: 'Return', not 'Returns'
326: D400 First line should end with '.', not 'e'
338: D401 First line should be imperative: 'Set', not 'Sets'
338: D400 First line should end with '.', not 'e'
352:1: E303 too many blank lines (3)
352: D401 First line should be imperative: 'Set', not 'Sets'
382: D400 First line should end with '.', not 't'
393: D401 First line should be imperative: 'Return', not 'Returns'
393: D400 First line should end with '.', not "'"
423: D401 First line should be imperative: 'Return', not 'Returns'
423: D400 First line should end with '.', not 'd'
423: D205 Blank line missing between one-line summary and description


1: D205 Blank line missing between one-line summary and description
4: L102 copyright year is outdated, expected 2015 but got 2014
45:1: I100 Imports statements are in the wrong order. import os should be before import sys
46:1: I100 Imports statements are in the wrong order. import inspect should be before import os
48:1: I100 Imports statements are in the wrong order. import cgi should be before import traceback
50:1: I101 Imported names are in the wrong order. Should be current_app, has_app_context
50:1: I201 Missing newline before sections or imports.
51:1: I100 Imports statements are in the wrong order. from operator should be before from flask
51:1: I201 Missing newline before sections or imports.
52:1: I201 Missing newline before sections or imports.
53:1: I201 Missing newline before sections or imports.
55:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.utils
56:1: I101 Imported names are in the wrong order. Should be CFG_BIBFORMAT_CACHED_FORMATS, CFG_BIBFORMAT_DISABLE_I18N_FOR_CACHED_FORMATS, CFG_BIBFORMAT_HIDDEN_TAGS, CFG_SITE_LANG
56:1: F401 'CFG_BIBFORMAT_CACHED_FORMATS' imported but unused
56:1: F401 'CFG_BIBFORMAT_HIDDEN_TAGS' imported but unused
57:6: E126 continuation line over-indented for hanging indent
62:6: E126 continuation line over-indented for hanging indent
63:1: I101 Imported names are in the wrong order. Should be create_record, record_empty, record_get_field_instances, record_get_field_value, record_get_field_values, record_xml_output
64:6: E126 continuation line over-indented for hanging indent
73:1: I101 Imported names are in the wrong order. Should be gettext_set_language, language_list_long, wash_language
73:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from .models
73:1: I201 Missing newline before sections or imports.
74:6: E126 continuation line over-indented for hanging indent
78:1: I101 Imported names are in the wrong order. Should be CFG_BIBFORMAT_FORMAT_JINJA_TEMPLATE_EXTENSION, CFG_BIBFORMAT_FORMAT_OUTPUT_EXTENSION, CFG_BIBFORMAT_FORMAT_TEMPLATE_EXTENSION, CFG_BIBFORMAT_OUTPUTS_PATH, CFG_BIBFORMAT_TEMPLATES_DIR, InvenioBibFormatError
78:1: I201 Missing newline before sections or imports.
78:1: F401 'CFG_BIBFORMAT_TEMPLATES_DIR' imported but unused
78:1: F401 'CFG_BIBFORMAT_OUTPUTS_PATH' imported but unused
79:6: E126 continuation line over-indented for hanging indent
85:1: I101 Imported names are in the wrong order. Should be parse_tag, record_get_xml
85:1: I100 Imports statements are in the wrong order. from invenio.modules.formatter.utils should be before from .config
86:6: E126 continuation line over-indented for hanging indent
88:1: I101 Imported names are in the wrong order. Should be CFG_HTML_BUFFER_ALLOWED_ATTRIBUTE_WHITELIST, CFG_HTML_BUFFER_ALLOWED_TAG_WHITELIST, HTMLWasher
89:6: E126 continuation line over-indented for hanging indent
92:1: I100 Imports statements are in the wrong order. from invenio.modules.knowledge.api should be before from invenio.utils.html
93:1: I100 Imports statements are in the wrong order. from invenio.ext.template should be before from invenio.modules.knowledge.api
94:1: I100 Imports statements are in the wrong order. from HTMLParser should be before from invenio.ext.template
94:1: I201 Missing newline before sections or imports.
95:1: I201 Missing newline before sections or imports.
103:27: E261 at least two spaces before inline comment
104:28: E111 indentation is not a multiple of four
104:28: E113 unexpected indentation
105:28: E111 indentation is not a multiple of four
105:28: E113 unexpected indentation
106:28: E111 indentation is not a multiple of four
106:28: E113 unexpected indentation
109:28: E111 indentation is not a multiple of four
109:28: E113 unexpected indentation
123:33: E225 missing whitespace around operator
214: D203 Expected 1 blank line *before* class docstring, found 0
218: D401 First line should be imperative: 'Return', not 'Returns'
218: D400 First line should end with '.', not 's'
230: D401 First line should be imperative: 'Return', not 'Returns'
255: D102 Docstring missing
294: D103 Docstring missing
300: D401 First line should be imperative: 'Format', not 'Formats'
300: D400 First line should end with '.', not 'f'
300: D205 Blank line missing between one-line summary and description
347:45: E127 continuation line over-indented for visual indent
490:27: E225 missing whitespace around operator
493:5: E303 too many blank lines (2)
511:82: E225 missing whitespace around operator
553: D103 Docstring missing
569: D401 First line should be imperative: 'Return', not 'Returns'
569: D202 No blank lines allowed *after* function docstring, found 1
569: D400 First line should end with '.', not 'g'
569: D205 Blank line missing between one-line summary and description
590:64: E261 at least two spaces before inline comment
590:65: E262 inline comment should start with '# '
597:38: E261 at least two spaces before inline comment
597:39: E262 inline comment should start with '# '
598:48: E261 at least two spaces before inline comment
598:49: E262 inline comment should start with '# '
601:24: E127 continuation line over-indented for visual indent
610: D103 Docstring missing
613: D400 First line should end with '.', not 's'
613: D200 One-line docstring should not occupy 3 lines
626: D290 White spaces around docstring should be trimmed.
626: D400 First line should end with '.', not 'a'
626: D205 Blank line missing between one-line summary and description
658:57: E126 continuation line over-indented for hanging indent
665:5: E303 too many blank lines (2)
724: D401 First line should be imperative: 'Evalute', not 'Evalutes'
744: D401 First line should be imperative: 'Analyse', not 'Analyses'
744: D202 No blank lines allowed *after* function docstring, found 1
813: D401 First line should be imperative: 'Return', not 'Returns'
813: D400 First line should end with '.', not 't'
813: D205 Blank line missing between one-line summary and description
912:13: B901 blind except: statement
943:78: E225 missing whitespace around operator
944:54: E225 missing whitespace around operator
976:47: E261 at least two spaces before inline comment
991:27: E225 missing whitespace around operator
1051: D401 First line should be imperative: 'Filter', not 'Filters'
1061: D401 First line should be imperative: 'Searche', not 'Searches'
1061: D400 First line should end with '.', not 's'
1061: D205 Blank line missing between one-line summary and description
1087:9: E303 too many blank lines (2)
1101:5: E303 too many blank lines (2)
1104:1: E302 expected 2 blank lines, found 1
1104: D401 First line should be imperative: 'Return', not 'Returns'
1118:5: F841 local variable '_' is assigned to but never used
1121:12: E127 continuation line over-indented for visual indent
1130:16: E126 continuation line over-indented for hanging indent
1152:5: B901 blind except: statement
1164: D401 First line should be imperative: 'Return', not 'Returns'
1164: D400 First line should end with '.', not 's'
1184:16: E126 continuation line over-indented for hanging indent
1191:1: E302 expected 2 blank lines, found 1
1191: D401 First line should be imperative: 'Return', not 'Returns'
1191: D400 First line should end with '.', not 'e'
1223:13: E303 too many blank lines (2)
1237: D401 First line should be imperative: 'Return', not 'Returns'
1271:16: E127 continuation line over-indented for visual indent
1281:31: E126 continuation line over-indented for hanging indent
1308:9: B901 blind except: statement
1320:17: E126 continuation line over-indented for hanging indent
1327:1: E302 expected 2 blank lines, found 1
1327: D401 First line should be imperative: 'Return', not 'Returns'
1327: D400 First line should end with '.', not 'e'
1371:1: E302 expected 2 blank lines, found 1
1371: D401 First line should be imperative: 'Return', not 'Returns'
1371: D202 No blank lines allowed *after* function docstring, found 1
1498:1: E302 expected 2 blank lines, found 1
1498: D401 First line should be imperative: 'Return', not 'Returns'
1498: D202 No blank lines allowed *after* function docstring, found 1
1525:17: E225 missing whitespace around operator
1593:1: E302 expected 2 blank lines, found 1
1593: D401 First line should be imperative: 'Return', not 'Returns'
1593: D400 First line should end with '.', not 't'
1637:28: E261 at least two spaces before inline comment
1637:29: E262 inline comment should start with '# '
1685:5: B901 blind except: statement
1749: D401 First line should be imperative: 'Return', not 'Returns'
1749: D400 First line should end with '.', not 'y'
1790:1: E302 expected 2 blank lines, found 1
1790: D401 First line should be imperative: 'Return', not 'Returns'
1790: D202 No blank lines allowed *after* function docstring, found 1
1790: D400 First line should end with '.', not '}'
1807:56: E225 missing whitespace around operator
1831:1: E302 expected 2 blank lines, found 1
1831: D401 First line should be imperative: 'Return', not 'Returns'
1831: D400 First line should end with '.', not 'e'
1860:26: F821 undefined name 'exc'
1862:22: F821 undefined name 'exc'
1865:1: E302 expected 2 blank lines, found 1
1865: D401 First line should be imperative: 'Return', not 'Returns'
1898:1: E302 expected 2 blank lines, found 1
1898: D401 First line should be imperative: 'Return', not 'Returns'
1942:1: E302 expected 2 blank lines, found 1
1953:1: E302 expected 2 blank lines, found 1
1953: D203 Expected 1 blank line *before* class docstring, found 0
1953: D204 Expected 1 blank line *after* class docstring, found 0
1953: D400 First line should end with '.', not 'n'
1953: D205 Blank line missing between one-line summary and description
1984:15: E261 at least two spaces before inline comment
1986: D401 First line should be imperative: 'Create', not 'Creates'
2018:31: E261 at least two spaces before inline comment
2042: D401 First line should be imperative: 'Return', not 'Returns'
2042: D400 First line should end with '.', not 'e'
2057: D401 First line should be imperative: 'Return', not 'Returns'
2057: D400 First line should end with '.', not 'd'
2080: D401 First line should be imperative: 'Return', not 'Returns'
2080: D400 First line should end with '.', not 'e'
2080: D205 Blank line missing between one-line summary and description
2118: D401 First line should be imperative: 'Return', not 'Returns'
2118: D202 No blank lines allowed *after* method docstring, found 1
2227:36: E127 continuation line over-indented for visual indent
2230: D401 First line should be imperative: 'Return', not 'Returns'
2287:33: E127 continuation line over-indented for visual indent
2298:60: E251 unexpected spaces around keyword / parameter equals
2300:54: E251 unexpected spaces around keyword / parameter equals
2310:37: E127 continuation line over-indented for visual indent
2321:64: E251 unexpected spaces around keyword / parameter equals
2323:58: E251 unexpected spaces around keyword / parameter equals
2416: D401 First line should be imperative: 'Run', not 'Runs'
2416: D400 First line should end with '.', not 'k'


Ok, cool.  I'll get to that now, and check out the other commit as well.

Scott

On Thu, Jan 22, 2015 at 11:07 AM, davideaster <notifications@github.com>
wrote:

> It looks like it's just that one commit: 276a789. I would just use that
> info as a reference and redo it.
>
> The one just before it about visible might also be needed: 77b4d80.
>
>
> > On Jan 22, 2015, at 10:39 AM, Scott Haynes <notifications@github.com>
> wrote:
> >
> > How do I do that? [?]
> >
> > There are two branches with 'contrib' in the ssc branch.
> >
> > contrib/update-transform-on-animation
> > contrib/rest-api
> >
> > Scott
> >
> > On Thu, Jan 22, 2015 at 10:03 AM, davideaster <notifications@github.com>
>
> > wrote:
> >
> > > We haven't exported any of the XPC work. But you can grab the piece
> that
> > > fixes a bug if necessary.
> > >
> > > > On Jan 22, 2015, at 9:42 AM, Scott Haynes <notifications@github.com>
>
> > > wrote:
> > > >
> > > > @allisoncorey @davideaster
> > > > I'll take a look at this issue. Maybe there is something else going
> on.
> > > >
> > > > PS. - there is a update from an internal branch dealing with the way
> > > transforms are being stored in the model driver. David did we ever
> cherry
> > > pick the commits from XPC (contribute or something like that was used
> for
> > > the branch names) ?
> > > > Scott
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub.
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> https://github.com/virtual-world-framework/vwf/commit/e6fc48e6af9d59affbaaba67b6d731cd338c6f42#commitcomment-9398327>
>
> > > .
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/virtual-world-framework/vwf/commit/e6fc48e6af9d59affbaaba67b6d731cd338c6f42#commitcomment-9399769>
> .
>


You can add `local` before a variable assignment to declare it as a *local* variable:

    local CAT=$(which cat)


Are these originally wrong indentations from the indentation script?


Actually you'd get a DuplicateKeyError and the command would crash - arguably better than bad data. The command is intended to be a one-off that's used to get the existing bills imported as opposed to something that's run on a regular basis. Perhaps I should make this clearer in the comments? Or would a non-management command script be more appropriate?


Thank for that. Will help own-tests. :)


If you can make the emoji works that'd be even better!


...What about languages other than English?


The comment wrapper, that is? Might be worthwhile fixing this in the script as well.


Hi @bar, this was an unintentional mention of your username, sorry.


Ah, great. Thanks.


Ok, see 0f9a0d07a0a9e4a75e3482ca950571cc9dd34532


We can create a new constant `Translations::MERGE_PLURALS` or something like that


It is weird it is not working because I pulled down a fresh copy of the master branch and it is running just like usual, no errors in the console nor in the browser.


I think you need an if options.has_key(:margin) or similar here otherwise all urls will get '&margin=' on the end I think.


@SoniEx2, I'm a bit confused. I thought the idea was to disable *any* function calls as Serpent doesn't generate them. If the user for some reason includes functions, they can provide their own loader or disable "safe" mode.


Btw, I stumbled over this the other day as well. Is this a new model since 1.1? If so, should we rename the model "Morency Doin" (uppercase) since we also have "Steinberger"? Or "Morency and Doin"? This might be our opportunity to do it.


I like the use of booleans here to make the code a little easier to read and debug. Nice work.


It is ok, but can you make it a bit optimized, no need to check .is and attribute twice:
```
if (isLink) {
    if (clicked.is(app.params.extenalLinks) {
        if(clicked.attr('target') === '_system') {
            e.preventDefault();
            window.open(url, '_system');
        }
        return;
    }
}


Why not ```Application.Current.Shutdown()```?

There may be some clean up code to run but ```Enviroment.Exit(0)``` is a brutal way to exit and the cleanup won't run.


The variable ```$unarchive_if_necessary``` isn't needed


you need to ```local($@);``` with eval, or use Try::Tiny


Although glancing over the code I see we also have some explicit AWS storage stuff in this file too. So IDK, maybe I'm being puritanical.


Sweet ship it


The commented-out lines should be removed if they're not needed


Yes, I think the ```client.read_feed``` method returns a Bundle since you are querying the entire feed for that particular ```resource_class```. It would be an instance of ```resource_class``` if you were using just the ```client.read``` method, as ```client1_reply``` is defined below.