Original is a tab,  **is it ok to replace tab with many spaces ?**

**code indent should use tabs where possible**


Agreed. We're currently storing times the same way as they are output here, which has tripped me up more than once.
When I get time on to work on times this is one of the first things I would like to fix, but the API should just get it right the first time.


The `checkForCompletion` function can be eliminated:

1. Move the `seenErrors.push` to the error part of the `s3.copyObject` callback.
2. Move the `if (seenErrors.length > 0) {` to the end of `exports.handler` after the loop.

Also:

* Rename `itemsToUpload` to `totalObjects`.
* Remove `completedItems` since it's redundant since it is `itemsToUpload - seenErrors.length`.


I have a link to the job modules page in the intro, more subtle though. https://github.com/dturanski/spring-xd/commit/41a894ad60a79ae89f5eaf739dcdcc56832cfd69#diff-6384a60ac9dcc3bb961f426daf772a77R8


Also please rename `seenErrors` to `copyErrors`.


Eu tava com sono.
2015/06/04 11:05 "KleberDarlan" <notifications@github.com>:

> Professora, o commit "up" foi por causa do sono do garoto ...
> kkk
>
> 2015-06-04 10:48 GMT-03:00 fjmartins <notifications@github.com>:
>
> > Já avisei que vou alterar na hora do almoço...
> > 2015/06/04 10:46 "Ivna Valença" <notifications@github.com>:
> >
> >
> > > Que zonaaaa...
> > >
> > > On Thu, Jun 4, 2015 at 10:15 AM, KleberDarlan <
> notifications@github.com>
> > > wrote:
> > >
> > > > Fábinho vei
> > > > eu reclamei, mas ele não alterou ....
> > > >
> > > > On Thu, Jun 4, 2015 at 8:36 AM, Fábio Lopes <
> notifications@github.com>
> > > > wrote:
> > > >
> > > > > UP? QUE P***..
> > > > >
> > > > > —
> > > > > Reply to this email directly or view it on GitHub
> > > > > <
> > > >
> > >
> >
> https://github.com/KleberDarlan/teste-configuracao/commit/8ecba0c66d12aa9f789c4e0403b54c46ece0ac48#commitcomment-11519427
> > > > >
> > > > > .
> > > > >
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/KleberDarlan/teste-configuracao/commit/8ecba0c66d12aa9f789c4e0403b54c46ece0ac48#commitcomment-11520961
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/KleberDarlan/teste-configuracao/commit/8ecba0c66d12aa9f789c4e0403b54c46ece0ac48#commitcomment-11521538
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/KleberDarlan/teste-configuracao/commit/8ecba0c66d12aa9f789c4e0403b54c46ece0ac48#commitcomment-11521566
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KleberDarlan/teste-configuracao/commit/8ecba0c66d12aa9f789c4e0403b54c46ece0ac48#commitcomment-11521954>
> .
>


This is much better on mobile, but it is still a bit off in comparison with my old app demos and with paul's voice memo and even photos.google.com (which has it positioned wrong in vertical direction as the toolbar it too fat).

Also the title now seems too close to the button when in seamed mode


nice.


Feel free to bring the formatting up to the coding style standards while doing this... :)
I don't think we generally put a space between cast and variable.


Yes. That's the idea. Yes, I spent a lot of time wondering if it was a bad idea, or not.  It was not an easy decision. I wanted to eliminate spurious, pointless SetLinks all over the place, with each Set containing just one item.

Somehow I decided that "cutting out the middle-man" would make graphs smaller, easier to read, and to make the API more "intuitive".  Yes, its a few more lines of code, but literally only a few -- just 1 or 2 lines in 1 or 2 places.   So, yes, that was the thinking.  I figured the plusses outweighed the minuses.


I think it is good enough for this PR but we need to decide what's the best user interface for this. Maybe the user will want to have an option to error when the isolators are not enforced. Or maybe it could even be the default. Could you ask on a follow-up issue of #990?


I did that, and then ran git rm --cached *. And everything was removed. So
then I did git add *, and it wouldn't add anything because some files in
the match were blocked by the git ignore. So I added them by file type.

What is the actual way to do this lol?

On Thursday, June 4, 2015, benwaffle <notifications@github.com> wrote:

> just use a .gitignore though
>
> *~
> *.exe
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Musicdude14z/TTT3D/commit/a0e6b7f32ec8a8b5ddb5ee3ca85f6f1659fc88a8#commitcomment-11521801>
> .
>


Nice!!! Although, because it is so short, you could just put it on the same line `thing ? true : false`.  But i like breaking it out on different lines when the line-length is > 80.


Thank you!

We'll address this in the upcoming release `4.1.2` (due next week on 2015-06-12)- I'll create a ticket internally.


looks good.  minor points:

- remove deprecated packages from cabal.config
- open ticket for changing scrypt dep to cryptonite, label low-prio

then please merge this.


Is this fixed?


Reverted this change.


Forgetful. Am working on development now


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.236](http://teamcity.monogame.net/viewLog.html?buildId=7998&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 496, ignored: 6 Build time: 00:23:08


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


I know there is more. I'll keep looking.


It is still stored as a double so there is no actual space savings.  It also makes it non-standard with respect to most of the other time values.  I am betting this change is to improve the presentation.  Ideally we would fix it at the presentation level instead of discarding the precision (i.e., maybe in the future tenths of a second would give you better insight 0.1 vs 0.9 as opposed to both being 0).

If we do alter the raw data I would rather see it encoded as an int in the protobuf and have a representation of "s".


No, this is not indentation, it is alignment.


`confirmactive` ??? Lol


It was not in 1.4 : it's better to keep them IMO.


Wooops!!! Absolutely, previous test paged only 2 users so I had overlooked that. Just pushed a new version of the test adding one user and page size of 2, so first page shows 2 and second, 1. It failed with previous code, verifies @Kartones fix. Thanks!


Do you just ignore the error here?


This is what I mean:

https://goo.gl/photos/zfwyLfzEqkcUbeus8


@frank-dittrich  The problem that astyle formats `while () {` to
```C
while ()
{
```
 maybe because of the `{` before `while ()`.

Here is a small function:

$ cat test.cc
```C
int main()
{
#if C1
        {
                while (condition) {
                        do_something();
                }
        }
#endif
}
```

$ astyle --style=kr -t4 -U -H -p -xC79 -c -k3 -z2 test.cc
$ cat test.cc
```C
int main()
{
#if C1
	{
		while (condition)
		{
			do_something();
		}
	}
#endif
}
```

But if we **remove the '{}' out of while**, astyle does not change the `while() {`

$ cat test.cc
```C
int main()
{
#if C1
		while (condition) {
			do_something();
		}
#endif
}
```
$ astyle --style=kr -t4 -U -H -p -xC79 -c -k3 -z2 test.cc
$ cat test.cc
```C
int main()
{
#if C1
	while (condition) {
		do_something();
	}
#endif
}
```


I guess these are unnecessary?


These as well.


`p-1` --> `p - 1`


spacing, and should be `!==` I think (three character comparison are always better than two, as far as I can see, in javascript)


Yes I didn't spot that one. There are a few very odd things like that. Not sure if they stem from indent or astyle.


I love when we can delete code, or replace reused code with components!


How's it going? Do you need / want help?


This chain is indented different from the next to.  Try to keep it consistent :)


Also, I think the default if you don't pass in something is undefined. The if statement should be
```
if (typeof cancel_function !== 'undefined')
```
Or, you know, check that it's a function, either way.


Those are good ideas. Would the preference be to revert and redevelop and commit again? I wouldn't be able to get a fix for this any time soon.


Cool! :+1:


You should set `MODULESTORE` on the class (not in `setUp`). For example: https://github.com/edx/edx-platform/blob/master/cms/djangoapps/contentstore/tests/test_courseware_index.py#L954


Awesome thanks Fer!


If changed to use build no + timestamp as part of key, theoretically this could mean some items from the same build would be deleted, while others remain.

I'm interpreting the comments to mean that we want to purge all storage items from same *build number* and that my current code ( and old code conditionals) using a key on just maven timestamp is wrong.

Please confirm my interpretation...


Yes, definitely. Actually it did the opposite of what I told it too.


You also don't need to test the XML modulestore, just MongoModuleStore and SplitMongoModulestore


I guess, we can remove this line as well.


@othiym23 Just a quick question, should I continue to use `https://registry.npmjs.org/` or is `https://registry.npmjs.com/` the canonical registry URL? Both seems to work.


Instead of using `neg_poly` here which creates another polynomial, you can do this at the end.


@magnumripper  This one is from indent


Then may be find out which tool did it, create a simplified test case, and report upstream?


Missing that event is rarely a problem.

Moving register_module_notifier earlier requires careful re-thinking all the ordering of the initialization. It is different from what is one in KEDR, so I cannot say yet if that was safe here.


Exactly. That's the whole point of all my rants. Only indentation should be tabs. This second line got lined up with the paren on the line before it. No matter what tab width **you** use will change that as long as tabs were NOT used.


No hard coded


since the component doesn't own/change the user ... this would probably be better accomplished with [defaultProps](https://facebook.github.io/react/docs/reusable-components.html#default-prop-values).  Then you can always reference `this.props.user.someAttribute` safely.


why are we updating the user here?


Sounds good... I can make this starting from 3.0 release, not worth fixing the 2.8 changelog at this point I guess...


Ah, You remove it in this commit.


@chelmuth: please add to staging, it is related to #1327


I would too, but i was unable to come up with a working API...


What the hell?


This was supposed to remove a ArgumentNullException when calling "TryDecodeKeyString", but it seems git decide to change the line endings instead.


I think the slow process is building the query, so this builds one query for all users with tasks and then just loops through them to match, rather than querying for each user for each task. Users that have multiple tasks are only returned once this way too.

It could be even more efficient if we set up an association between User and Task, but this should work well for now.


welcome home


I'm leaning towards the idea of just using the build number as the key (this is all scoped under the same GAV) ... @cstamas can you think of any reason not to do this?


Awesome!


Not optimized for our chip. It's for a chip with other characteristics. We had to change some code which is on my todo list but I didn't have time yet. If you revert you'll notice a temperature average decrease and a performance average increase.


for practical purposes not sure it matters much - at the end of the day the removal decision is based on last requested timestamp - build timestamps off by a second don't matter much when the task granularity is in days.. the other maps use Version as the key, and versionsRemoved list uses maven snap timstamp - given these factors, I am very hesitant to change any more logic...


damn..  1.4 :s
then maybe backport the try .. except in 1.4 too ??


:+1:  reverted, it was out of scope :)


I think this is an unavoidable regression (as is loops-for-withvec4inductorW) since this is precisely the kind of behaviour we're trying to avoid. Since semantically it's fine to leave it as a while loop with break/increment, I think we should err on the side of caution, as promoting things back to for() loops incorrectly has the chance to change semantics by moving the increment operation.


Thanks for the warning. I'll fix it


Trace the vanilla git code, see:
![committer_date](https://cloud.githubusercontent.com/assets/5831804/7986362/bb5370c6-0b09-11e5-9707-b87e52135fa0.png)


Best news of the day.


If you define "same build" as "one single run of mvn deploy", then all the deployed stuff (across modules, the whole reactor) will have the same timeStamp+buildNumber. So, I don't think your statement "some items (...) would be deleted, while others remain". Do I miss something?


Welcome! :smile:


Please inline that function. This looks like math-style C where the filter and sort functions are separated. But imo this just complicates the code.


@csware
I keep your commit. :-)


una peguita para ochoa :P

2015-06-04 10:31 GMT-03:00 Felipe Hernández G. <notifications@github.com>:

> que está pasando aquí?? :O
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CC5401-MagisterTI-DCC/WEB/commit/9da623f058c5f46d466dc7886fa0fe53a65b345a#commitcomment-11521233>
> .
>


We usually never use _cb suffixes for directly called functions. Suffixes make limited sense if you pass them as real callback, but here it should be totally sufficient to call this "elect_display_compare()" (you don't sort here, you just compare them).


Could you expand a little bit the comment? like:
` Collect robots named 'robot_XXX' into a list called 'robots', allowing to iterate on them, for instance.`


Use Qt::QueuedConnection inside the class too.


Not quite, it also handling things like bytes conversion. It's a more robust version of six.text_type that handles encoding choices.


@mcculls I think that should work too, buildNumber is not unique per-timeStamp, it's unique per GAV deploy as you say.


I'm thinking that backends shouldn't be allowed to create their own bindings anymore, or should only be able to do so when started in some kind of debug mode...


Please always order non-stopping sessions before !SESSION_USER sessions. I mean, there is really no reason to ever choose a stopping session.

I know, this doesn't matter as you already filter beforehand, but lets be safe here.


I would also add:
`self.assertEqual(len(morse.ms), 10)`


不要做奇怪的拓展啊
see http://www.web-tinker.com/article/20967.html
see https://www.v2ex.com/t/190956#r_2061759


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (351 of 351 tests passed)

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


(On the premise that someone who didn't code it may have a different vocabulary, I'll give it a shot): The idea is that when indexing through two arrays at the same time, you want to use the fastest indexing method available to both of them.  This implements that choice — it's like a 'fastest common denominator' or 'common supported functionality.'  Hm.  Those really long.  I don't think I'd call this 'promote,' but maybe 'combine' or 'merge.'


Kill is default, no need to pass true.


@magnumripper
in the john core-1.8.0 loader.c, the second line begins with a tab. So it should be **replaced with spaces**?
```C
static void read_file(struct db_main *db, char *name, int flags,
        void (*process_line)(struct db_main *db, char *line))
{
```


I think the use_pixman switch is going to need a rethink...  as is all the switching between gl and pixman at runtime.  a libweston user might want to provide its own software renderer instead.

I think that can be considered later, after the bulk of this work is integrated though. :)


Ok, I've just pushed up a commit to change that over.


This plan, I like. I'll try to grab a sample and hand-code this evening (50 packages a good number?)


1. Not the right use case for map; you aren't creating transforming an array, you are looking for something so try `array.find` or `array.some`.
1. You aren't actually using the iterator for anything, but just returning a component if the user is a certain role. (see how you don't use `organization` anywhere)
1. Thought for the future: having a `user.role` set to manager is problematic if they stop being the manager of all their orgs (by deleting or transfering to another user) because they will still be set to manager.  This means we'll need extra logic to check if they are still managing an org.  Not a problem now because orgs can't be deleted or transferred, but could be in the future.


I think this text should be configurable, or show the title of the project, what do you think?


I think it's worth changing this to $ticketPriority = null


Rename pathsToPwdFiles to passwordFiles. Let's strive towards simpler and more clearer variable and function naming.


In my opinion, yes definitely. Perhaps not in Solar's opinion though.


Because it's not available in most environments.


This seems like it may be worthy of its own vocabulary – generic array iteration is fairly fresh territory.


GASP!  Are those SPACES?!?!? :)


There's a good chance it will be included anyway and a lot of other libraries, including some MVC ones, depend on it.


Why are you appending the list of files here?


boxed_number_simplifications (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (350 of 351 tests passed)

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


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (351 of 351 tests passed)

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


Yes, you are totally right ! I focused of fixing ``$this->themeOption`` and I didnt pay attention the the title was hardcoded in the theme


Just run an SQL query to replace hercules.ws with herc.ws in the `posts` table..


Convention. Having render return a promise means that async behavior can be added later and the signature doesn't change. Since this implementation is super simple, just using resolve would work for now.


I agree


Shouldn't this be run on main thread? Using `Sync`?

```
Sync (delegate {
  idleFunc ();
  return true;
});
```


egyptian cotton sheets


Hm.. you're using old develop branch.
Xcode Plugin moved here https://github.com/DyCI/dyci-xcode-plugin, so you can add this code there :)


The "||" won't work here. Set the value to -1, and then do another expect for when the value is 1.


Topkek!


good for me.

but the wiki "numbering" is all screwed :s


While executing **terraform plan** or **apply** we're getting error:

There are warnings and/or errors related to your configuration. Please
fix these before continuing.
Errors:
  * module root: module dc2-hosts: flavor_name is not a valid parameter

Fixed by adding **variable flavor_name** into the **terraform/openstack/hosts/main.tf**
Another solution is to delete **flavor_name** from **openstack.sample.tf**. I couldn't find where really we use this parameter.


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


Can I set my role to `superadmin` in user#create or user#update by just calling the API directly?  Plz investigate to guarantee not.


You forgot to update the gemspec file


No problem...
Op 4 jun. 2015 16:34 schreef "Cristiano Matos" <notifications@github.com>:

> Thanks for the warning. I'll fix it
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/crdroidandroid/android_frameworks_base/commit/9394cb67634f114174a86cff9bc5308b0ca836cc#commitcomment-11522545>
> .
>


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.237](http://teamcity.monogame.net/viewLog.html?buildId=8002&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:24:28


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


#blamedrtshock


Actaully, `intersect` seems like it represents the operation fairly well in my minds.  Still a little bit of a pun, but you're intersecting the fast indexing methods?


Don't pass the path to john like that. Set it from outside of this method in preparation for multi-session.


It's really a shame this has to be done to the rpi back-end at all, as it's slated to become entirely obsolete "soon".

I've got a patch against rpi renderer that's been lying in patchwork ignored for a month - if that's any indication of the level of support we're going to provide for the rpi renderer, should we just pre-emptively remove it?

It currently doesn't work well at all (offline mode crashes after 5 windows open, online mode doesn't render properly after 5 windows open)


are they supposed to differ, or is this something we should worry about? (I thought there was a related issue about refit/original-fit discrepancies, but can't find it now ...)


6: M102 unrecognized bullet/signature
8: M102 unrecognized bullet/signature
9: M102 unrecognized bullet/signature
6: M190 line is too long (74 > 72)


You can't use angular directives in a CSS file. You'll have to define different CSS classes here, then apply them in the actual .html.

In this case, I'd probably do something like:

    .graph-legend {
       .. properties which are applicable if series is active or inactive
       .. properties which are applicable to the active series
    }
    .graph-legend .series-inactive {
        ... overrides for the active property
    }

Then, inside the HTML use the ng-class to dynamically set series-inactive if the series is inactive.


What's the point of not adding to manager?


Don't use underscores anymore on object, method, action names.


This is very good news. I have been testing Idhor’s work as submitted with multiple users in Beta and have experienced no problems. To use this properly to support stored procedures requires some explanation.

1.       Only ‘in’ parameters are supported. ‘out’ parameters are not supported.

2.       Only the first row set returned by the stored procedure will be seen.

3.       A stored procedure works as expected with either database.Query() or database.QueryRow().

4.       database.Exec() does  not set sql.result properly with stored procedures. Therefore, your stored procedure should return something even if the intent is just to insert or update.

5.       To determine the last key added or rows affected, have the stored procedure return a useful value. E.G.  ‘select last_insert_id() as ID’ or  ‘select row_count() as RowCount.’

With the above conditions observed, we have a robust solution to stored procedure support.

Richard Knouse

From: Julien Schmidt [mailto:notifications@github.com]
Sent: Wednesday, June 03, 2015 3:55 PM
To: Idhor/mysql
Cc: Richard Knouse
Subject: Re: [mysql] Enable Multi Results support and discard additional results (9542bdd)

Hi,
I'd like to use this commit in the main repository. Can you please make an entry into the AUTHORS file for copyright purposes?
Thank you!

—
Reply to this email directly or view it on GitHub <https://github.com/Idhor/mysql/commit/9542bdd9491b36ade6b4248c62a0d9a63c621d2f#commitcomment-11510198> .  <https://github.com/notifications/beacon/ALGXRvrDlNwvhFi0Wfw9PKh2nRbHGgrPks5oP29GgaJpZM4EURcv.gif>


I don't know if we should use `super` for now, see #534.
@sils1297: Can we keep it here or should we enforce using old-style "super"?


No exclamation marks please.


Just did some more testing.  Everything works just as it did before :+1: Very beneficial change!


Mancano delle parti da cancellare


awesome :clap:


This will probably break a few things.

If this if for distributive, we just need to install the rpm during common and then each role can push down their test files.


@edolstra is this you taking a position against that style of expression or just a practical concern with its implementation in `pulseaudio`?


boxed_number_simplifications (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (350 of 351 tests passed)

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


once u've run rake db:migrate link me to that commit and i'll say that this assignment is done.


Surprisingly, 居然有人看挨行看咱的commit...
.


Welcome to the new credit_pseudocode.txt... now family friendly!


Brian Cady here. Sorry if I didn't send this correctly. It's my first pull request. Anyway, the script is in Python 3. It's a little primitive but I think it fulfills the requirements.


after reading latest comments and discussing with Rich, it seems like a good idea to roll up the most recently requested item time per build number, and not timestamp.

Will change the cache key to use build number.


This seems like an odd one to revert


@sdt Hey, I was watching perl6 with great anticipation but things seem to have stalled.  Any plan to finish it up? I think that would be a great addition to have if you find time to finish it up and send a pull request. :-)


Update to indicate that this was `OK`, but that there is bug in the test() script that required testing from a checked-out version of xija.


I think that we should use the `Array.isArray(val)` because there are two bellow reason.

- Why doesn’t Vue.js support IE8? (Common FAQs)
  http://vuejs.org/guide/faq.html
- ECMAScript compatibility table
  http://kangax.github.io/compat-table/es5/


why don't you just do
`self.path = path`
without the if?
I think the if is useless here^^


You don't need a list here; `add_cmd_output()` accepts either a single string or a list of strings.


6: M102 unrecognized bullet/signature
8: M102 unrecognized bullet/signature
9: M102 unrecognized bullet/signature
6: M190 line is too long (74 > 72)


I'm not sure about using `@property` attributes.
@sils1297 What's your opinion about them?


http://www.bootstrapcdn.com/
Utilisation des fichiers css de bootstrap


What's authenticate?  It's removed and not replaced anywhere, but as far as I can tell it's vestigial?  Perhaps this should be removed in another patch.  #excessivepedantry


There were many commits that depended on each other so I had to revert them all in reverse order.


Ah, okay, makes sense.


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (351 of 351 tests passed)

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


Can you add a `TODO` comment so we remember to remove them?


If you don't pass something in, it's ```undefined```, not ```null```, so I think you want

```if (cancel_function !== undefined)```


general import structure:
I think we do import the module `unittest` after `sys.path.insert(0, ".")`, but I'm not sure if this is really needed.

But definitely you should place a return before this import^^


This is actually for https://bugzilla.mozilla.org/show_bug.cgi?id=1170897 not 1169231 as it is in the commit message.


http://getbootstrap.com/examples/starter-template/
Template disponible sur le site de Bootstrap dans les exemples
faire apparaître le code source du site.


"Your passphrase guess:"


well are you sure about this


Why the separate logic for `conf_glob`? Any member of a list passed to `add_copy_spec()` can contain a glob, e.g.:

```python
    self.add_copy_spec([
        "/etc/monit.conf",
        "/var/log/monit.log",
        "/etc/monitrc",
        "/etc/monit.d/*"
    ])
```

Although to be honest, if the directory contains only config files, you might as well just add the directory path and let the directory recursion in `do_copy_paths()` take care of it:

```python
    self.add_copy_spec([
        "/etc/monit.conf",
        "/var/log/monit.log",
        "/etc/monitrc",
        "/etc/monit.d"
    ])
```


@alexkalderimis looks like you got really close to finishing. Any plan to finish this up and send a pull request for adding to the upstream? LiveScript would be a great addition.

BTW, this describes what I like to see before merging: https://github.com/kanaka/mal/blob/master/docs/FAQ.md#add_implementation


The `if` is needed because it tells if the path has ever been set.
If I do `self.path` ... then, `_path` will have the current directory - because `self.path`'s getter will have done an `abspath` ... and abspath will change "" to the current directory.

So, this has more information that just doing `self.path` Seemed like a nicer things to do :D because this could be useful to the client - if it sees "" implies it hasn't done something right


The former. See e.g. my comments at https://github.com/NixOS/nixpkgs/commit/25a148fa196b944b3f134527da87e43d88c066f9#commitcomment-11464463 and https://github.com/NixOS/nixpkgs/commit/a2a35082ccd64b4d00d88778d9633fe18c0bb215#commitcomment-11157014.


I have fixed this, apperantly it did not register me going to the location, now it did


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (351 of 351 tests passed)

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


You should squash this commit into the parent (it's just a fixup) using `git rebase -i` and then update the branch in the pull request by running `git push --force`.


You don't use `self.session_bus` and `self.dbus_name` in the test.
Do you need them?


It's fixed in master, may be improved but it's better (experience will tell us if it's ok). I'll add it to the documention too asap (I have begun to write the api generation documentation)


I've merged these in with standard sorting.


Hey Greg, sorry I haven't gotten back to you on this. I am a little wary of actually trying to walk the process tree since I suspect it'd lead to cases of runaway processes when child processes do weird things. Propagating signals is one of the reasons that process groups were implemented in POSIX in the first place. The `setpgrp(2)` man page has this to say:

```
       A child created via fork(2) inherits its parent's process group ID.  The PGID is preserved across an execve(2).

       Each process group is a member of a session and each process is a member of the session of which its process group is a member.

       A session can have a controlling terminal.  At any time, one (and only one) of the process groups in the session can be the foreground process group for the terminal; the remaining process groups are
       in the background.  If a signal is generated from the terminal (e.g., typing the interrupt key to generate SIGINT), that signal is sent to  the  foreground  process  group.   (See  termios(3)  for  a
       description  of  the  characters  that  generate signals.)  Only the foreground process group may read(2) from the terminal; if a background process group tries to read(2) from the terminal, then the
       group is sent a SIGTTIN signal, which suspends it.  The tcgetpgrp(3) and tcsetpgrp(3) functions are used to get/set the foreground process group of the controlling terminal.

       The setpgid() and getpgrp() calls are used by programs such as bash(1) to create process groups in order to implement shell job control.

       If a session has a controlling terminal, and the CLOCAL flag for that terminal is not set, and a terminal hangup occurs, then the session leader is sent a SIGHUP.  If the session leader exits, then a
       SIGHUP signal will also be sent to each process in the foreground process group of the controlling terminal.

       If  the exit of the process causes a process group to become orphaned, and if any member of the newly orphaned process group is stopped, then a SIGHUP signal followed by a SIGCONT signal will be sent
       to each process in the newly orphaned process group.  An orphaned process group is one in which the parent of every member of process group is either itself also a member of the process group or is a
       member of a process group in a different session (see also credentials(7)).
```

Could you check to see whether your process is in the `T` (stopped) state? If so, it sounds like a background process group could be trying to access the terminal. I'd also be curious to know if any of the child processes in your job are in a different pgid than the job parent (not Galaxy)'s pgid - this could happen if one of the subsequent processes calls `setpgrp()` or `setpgid()` themselves. The `pgid` format specifier for `ps -o` will include the pgid in `ps` output.


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (351 of 351 tests passed)

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


I don't think you can do the check this way, you need to use `typeof`. Trying this in the javascript console:

```
if (a === undefined) {console.log('cool')}
VM382:2 Uncaught ReferenceError: a is not defined
    at <anonymous>:2:5
    at Object.InjectedScript._evaluateOn (<anonymous>:904:140)
    at Object.InjectedScript._evaluateAndWrap (<anonymous>:837:34)
    at Object.InjectedScript.evaluate (<anonymous>:693:21)
```
versus
```
if (typeof a === 'undefined') {console.log('cool')}
VM383:2 cool
```


Another version location. Any way to get this to read __version__.py?


Ну ты и разогнался :) :+1:


@jgdovin are you making the assumption here that messages will arrive or render in order? I'm not sure this will work in all cases.


" \$DIST" -> " $DIST" in commit message? otherwise looks good to me.


This can be fixed if add '()' around gpu_amd
```C
if (gpu_amd(device_info[sequential_id])) {
```


Then I would catch the `""` case inside the setter, so:
```python
@path.setter
def path(self, new_path):
  if new_path:
    new_path = os.path.abspath(os.path.expanduser(new_path))
  self._path = new_path
```


What the…? I see a horrible code duplication. Please extract it to a function.


Or do you want to set the current directory when invoking the setter with an empty string?


Agreed, done @kazupon


boxed_number_simplifications (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (349 of 351 tests passed)

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


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (351 of 351 tests passed)

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


Dans le mail, j'avais dis que c'était pas utile de refaire ces calculs, il fallait juste utiliser les rectangles déjà calculés par Florian.
Comme ça si un jour on veut modifier la distance entre deux tile et tout ces détails, on aura juste à modifier son code.


Indentation^^


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


==================================
VitaUpdateBlocker v1.2a
http://iamghost.kr
==================================
[Thu Jun  4 09:59:57 2015] Listening on 192.168.2.103:8080
[Thu Jun  4 09:59:57 2015] Blocking non-related traffic is DISABLED.
Traceback (most recent call last):
  File "/Users/S0N/Downloads/VitaUpdateBlocker-master/vita_update_blocker.py", line 131, in <module>
    main()
  File "/Users/S0N/Downloads/VitaUpdateBlocker-master/vita_update_blocker.py", line 125, in main
    config = proxy.ProxyConfig(port=port)
  File "/Library/Python/2.7/site-packages/libmproxy/proxy/config.py", line 86, in __init__
    CONF_BASENAME)
  File "/Library/Python/2.7/site-packages/netlib/certutils.py", line 207, in from_store
    dh = cls.load_dhparam(dh_path)
  File "/Library/Python/2.7/site-packages/netlib/certutils.py", line 181, in load_dhparam
    bio = OpenSSL.SSL._lib.BIO_new_file(path, b"r")
AttributeError: 'module' object has no attribute '_lib'

Using Python 2.7.6 & mitmproxy 0.12.1
This error shows.


Next time, please split up the commit.  After scanning dozens of pages of new and modified tests, I can't tell where the fix part of this commit is for the python versus your new STACIE implementation is.


This will get over-written by the playbook: https://code.library.ualberta.ca/hg/ansible/file/769e0b981db3/hydra/roles/hydra/templates/production.rb.j2.  We should probably fix this there.


That seems good to me.

I'll probably tweak this, I'm thinking that slightly better tack for these override calls is to call the original call with unchanged parameters as a safer way to ensure we get the original results. I've done a couple of calls this way, and will probably do a full sweep with this in mind.

I'll also add this sequence to EnumAdapters1, as presently it doesn't wrap the adapter at all and is legal for our Win7 no-platform update target.


Just for interest: Since you have a server class, don't you need to derive from something like `dbus.service.server` or something like that?


Nice this is helpful!


copy form ic2 ItemTreetap


Verified


To avoid warnings we need to have differences in parameter classes ( not rely on generics). Consequently ```def apply[In, [#A1#]](unzipper: In ⇒ Tuple1[[#A1#]])``` after removing generics we don't know what kind of tuple function will return. So we need to substitute unzipper with classes with index in name 2..20, or introduce some additional parameter.
Per my understanding there can be two ways:
* Having class like UnzipWithCreator1 that implicitly wraps function with tuple. Signature in this case will be ```def apply[In, [#A1#]](UnzipWithCreator1(unzipper: In ⇒ Tuple1[[#A1#]]))```. Plus is that there is no additional implicit defs. Minus is that API is not clear
* Having additional implicit attribute ```def apply[In, [#A1#]](unzipper: In ⇒ Tuple1[[#A1#]])(implicit creator: UnzipWithCreator1[In, [#A1#]])```. Plus is it influencing API inless. Minus is that it introduce 20 defs.

@ktoso Does it make sense?


You should change parameter name, it doesn't describe when the callback is invoked, which is the main information when passing callbacks :)
Maybe rename to `on_unconnected` or `on_unconnected_callback`, don't know :P


@jlucerond DWMR. Checkpoint complete. Did you try any of the assignments?


No, the given key is used as a PartitionKey (and this should be probably tweaked to remove any invalid chars).


closes #16


Hi Kathy, is there something that we need to do to see this in action? I tried cloning the repository and browsing to the src/index.html on my local webserver. Was there something that needs to be built?


OK for me


Since I already found something, you can place dots at both params `bus` and `path` :)


Okay, this is fine.


We are setting initialState in the js file.  When did we get this sugar?


no idea.


closes #14


boxed_number_simplifications (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (349 of 351 tests passed)

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


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


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


All three methods seem to be public, you should document them^^


Aqui foi só a reversão da alteração! :P


Ok for me


We still need to inform the user via the UI if something bad happened.


See also my comments on https://github.com/NixOS/nixpkgs/issues/7013#issuecomment-86776800. Agree that it would be nice to have a ticket or mailing list discussion about this new style.


Yeah, I was hoping to keep current directory with empty string ... as it'd be easier for a client - as "a.py" would automatically assume current path, I felt "" should also automatically assume current path (That's is the way most things function right ?)


I didn't have enough knowledge for judge of this word. so I reverted.


Yep, this one is mine.

Chris

> On Jun 3, 2015, at 9:39 PM, Mike Sinkula <notifications@github.com> wrote:
>
> Yours?
>
> --
>
> Mike Sinkula
> Professional Twelve Year Old
> http://www.mikesinkula.com
>
>
> > On Jun 3, 2015, at 9:03 PM, cwlock <notifications@github.com> wrote:
> >
> > Meet Myah :)
> >
> >
> > Chris
> >
> > > On Jun 3, 2015, at 6:15 PM, Mike Sinkula <notifications@github.com> wrote:
> > >
> > > That dog looked pretty sad. :-(
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> —
> Reply to this email directly or view it on GitHub.
>


Ah, this fixes the problem I spent two hours scratching my head on <3

I have a `link_to "foo", @hash` in a cell template (so that it is generic and works wherever it is included from), with that, it works again, if I add:

```
@hash[:controller] ||= controller.controller_name
@hash[:action] ||= controller.action_name
@hash[:only_path] ||= true
```

in my cell code, otherwise, I get a `No route matches {the hash I gave it}`


Maybe for aesthetic: define a private method that manages id counting, so you just need to implement `_next_id()` that servers the right id?


The -1 entry in s.conns and s.fds is the exec control socket.


@joao-manywho This needs to bracketed with an if as current custom players won't have callbacks on the options


This change brokes my iptables rules:
```puppet
    firewall { "phpdaemon_tcp_${port}":
      protocol    => 'tcp',
      destination => $::ipaddress,
      port        => $port,
    }
    ensure_resource( 'firewall', "nginx-vhost-${port}", { protocol => 'tcp', port => $port, destination => $::ipaddress } )

```

```diff
--A INPUT   -p tcp --dport 8047 -j ACCEPT -m comment --comment "phpdaemon_tcp_8047"
+-A INPUT   -p tcp --dport 8047 -s  -d 192.168.21.15 -j ACCEPT -m comment --comment "phpdaemon_tcp_8047"
--A INPUT   -p tcp --dport 80 -j ACCEPT -m comment --comment "nginx-vhost-80"
+-A INPUT   -p tcp --dport 80 -s  -d 192.168.21.15 -j ACCEPT -m comment --comment "nginx-vhost-80"
```


Imo you should call this class more "testful", so i.e. `TestDbusDocument`, since it is not a real component and only used for tests^^


The code is optimized before execution by KLEE (Assuming KLEE is given `--optimize` flag)

It uses LLVM's opt passes anyway and doing it like this allows KLEE in theory to skip over problematic passes.


Hi there - yes. Thanks for the message. I just started a new job so have
been out of things but plan to take this up again.

Next week or so?

On 4 June 2015 at 15:56, Joel Martin <notifications@github.com> wrote:

> @alexkalderimis <https://github.com/alexkalderimis> looks like you got
> really close to finishing. Any plan to finish this up and send a pull
> request for adding to the upstream? LiveScript would be a great addition.
>
> BTW, this describes what I like to see before merging:
> https://github.com/kanaka/mal/blob/master/docs/FAQ.md#add_implementation
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alexkalderimis/mal/commit/61f47e206f5c84acfb79668988aefaae3f9bd6e8#commitcomment-11523090>
> .
>


if (style == ChatColor.RESET) return this.removeStyles();


Great!


This one is OK for now, but we'll want to revisit it once we learn a little more about how the games use the API.  The check for "typeid(*pDevice)" will crash if the pDevice isn't a wrapped version.  Gives a RTTI exception because normal objects have no RTTI info.  Pretty much notification that we have an unknown type though. :->

Changing the HackerDevice to IUnknown in the header declaration is exactly right.  I had modified a handful of those to get it to compile, but the right answer is to leave the declaration identical to the original, and do the static_casts.  If you see more of those, please go ahead and fix them back.


Hi @jgok,

the `A.Lang.String.contains` is not enough, the hash example:
http://evil.com/?someParam=https://basesrc.com


Out of curiosity : What's the issue with `@property` attributes ?
I found them being used in [LineDiff](https://github.com/coala-analyzer/coala/blob/master/coalib/results/LineDiff.py), [StringConverter](https://github.com/coala-analyzer/coala/blob/master/coalib/misc/StringConverter.py) and [Setting](https://github.com/coala-analyzer/coala/blob/master/coalib/settings/Setting.py)


Indentation!
```python
        self.session_bus = dbus.SessionBus(
            mainloop=dbus.mainloop.NULL_MAIN_LOOP)
        self.dbus_name = dbus.service.BusName("org.coala.v1.test",
                                              self.session_bus)
```


If you want to keep it the existing Ubuntu 12.04 instructions use an ubuntu ppa with newer gcc.
You'd just need to add gcc-4.8 and g++-4.8 to the list.


Could we have the preload.png image in 1920x1080 exactly?


Is it the nature of the fix that you don't use resolveToDescriptor()?


There isn't a particular style guide as far as I could gather.

For example :

[ConfigGatheringTest](https://github.com/coala-analyzer/coala/blob/master/coalib/tests/settings/ConfigurationGatheringTest.py) has return after `sys.path` and unittest above

[SectionFillingTest](https://github.com/coala-analyzer/coala/blob/master/coalib/tests/settings/SectionFillingTest.py) has return after `sys.path` and unittest above


PHPCS: Line exceeds 120 characters; contains 128 characters


this is changed in the most recent version to a typeof check...


LGTM


I’d rather not. I think it simplifies the code, and centralises the decision making about which sessions should be filtered. I could easily see this function needing to be reused in other bits of the code for the same filtering decision.


Good point, done.


Fair enough, done.


rax retry tests


Hi, just tracing the commits to see which broke color on cygwin (rxvt and xterm) on windows. Looks like the detection here doesn't catch these.

The ENV['TERM'] for rxvt and xterm (both which support colour is):
- rxvt-cygwin-native
- xterm

Neither return true for STDOUT.tty? so perhaps the tty detection logic if off for cygwin too.

What would you suggest? I've locked our version of minitest-reporters to 1.0.8 till I figure out what to do.

Thanks,
Michael


That's a pretty astute question.


@edolstra can you make a ticket for something like "revert new style" so people can be aware of this happening outside of those (like me) who spy on individual commits?


Trivial performance thing, but its better to test "wildi > 0" before making a call to keyed,length.  Also, can you get the keyed.length once and save that value, and then just test that value?


I'm not sure if this is really useful to have a function, that creates the app if not existent...
On first sight I would prefer that `get_app` would raise, and then you create an app via `create_app`...
But maybe you have reason to do it like this^^


That was a very useful thing to do as it uncovered a problem with the code. see https://github.com/alphagov/smart-answers/commit/6047bdd9dfcafcf8d672ae7984585ab450a9a143

Thanks!


Yes.


I did not actually think about that :p


Thanks. Can you do a pull request again with this change?

On Wed, Jun 3, 2015 at 4:10 PM, Josh LaBounty <notifications@github.com>
wrote:

> Yes. Fixed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jlabounty/coresoftware/commit/4f96251752ca6df476826d97ff458cfa017dee86#commitcomment-11508134>
> .
>

--
*Dr. Nils Feege*
Research Associate

SUNY at Stony Brook
Department of Physics & Astronomy
Stony Brook, NY 11794-3800

e-mail nils.feege@stonybrook.edu
skype nils1920
phone +1-631-632-8112

www.linkedin.com/in/nilsfeege


We can ditch the `_has_readme` variable completely, right? We're not pattern matching on it so it seems it doesn't have a purpose. Wdyt?


are these real keys?


We probably want a little more explanation on the docstring, like:

    Returns statistics from the last 10 jobs executed in the server

    * id
    * name
    * start time
    * status


I would inline as a lambda since this is a really short statement:
`lambda: assert(1 == 2)`
Also because python3 wraps print and others to functions, I would use assert also like that^^


So here you would just place:
`callback=lambda: assert(1 == 2)`


The same change needs to be made to the subclasses.


PHPCS: Expected 1 newline at end of file; 0 found


what's different about this fork?


In fact, I realize the function returns from all jobs, then the 10 last ones, then the last executed job. Interesting, that's another catch that would be well served to go to the docstring.


As a part of the algorithm, I want leading coefficients positive before proceeding. That's why this is done.


@starkos Sorry! This was meant to be a PR, but I accidentally pushed to the wrong remote (premake-core instead of my personal fork >_<)
It should be benign.


Shouldn't you keep ```Email;```?


get from config?


generate from config


no vpn, can't download the image.


The commit description looks like a good starting point on a docstring for the function.


OK


Right!


`intersect` sounds logical. But this might deserve more thought, if you consider it more broadly in terms of what could become traits in the future. If traits could inherit from one another (just like types currently do), this would be more like finding the common ancestor to two traits. I.e., `LinearFast` would be a trait that some types implement, and `LinearSlow` would just be a fallback used for all other types, or for combinations where `LinearFast` is not a common ancestor to both types.


This still leaves the @flusher_lock to synchronize the flusher thread executing and the flusher_thread kill.
However, I'm not convinced that there's a need to protect the thread's "queue.push FlushEvent" against interruptions since the queue should be threadsafe


No, GLib.Idle.Add runs it on the main thread, so Sync wouldn't do anything extra


PS: Documentation! You need to describe `path`!


fixes #1


OK, I remember you mentioning dj-static is not packaged on Fedora (which I think it's not that big of a deal since we can package it in our COPR repos for now).


fine with me.


I don't have a mouse with me at the moment, and two finger scrolling on my scratchpad does not zoom correctly.


This is a great improvement and covers a lot more VDB scenarios. Nice.


close #1


Please mention GPLv2+ (GPL v2 or any later version), instead of only GPLv2.


But if someone wants to set `path` after, there's no possibility to have this automatic current path.
I'm not sure also if it's really necessary to have a relative path^^


computing a unique id based on package name, version, and dependencies' ids


Hello, I don't think this change is appropriate. It is altering the name of my instances.

Say that I want to deploy my code in:

```
- hosts: tag_Name_MDI-image-gallery-QA
```

I'd get `tag_Name_MDI_image_gallery_QA` instead and obviously they are not going to be found in the inventory. I think we shouldn't be altering the information retrieved from AWS.

Regards


Fix for #14


Why is this refactor strictly necessary?  We already have a struct weston_compositor that's extended by the backends, why do they need to be further separated?

If this becomes obvious later in the series, perhaps a comment in the commit log will help...


fixed!


release for #14


This kinda works


Yep, if I can't parse the limit I just ignore it and return the original opts


Indeed, I never encountered them during my participation of this project, so I better asked @sils1297 :)
But yes, then this is definitely okay :)


`Unless explicitly otherwise stated, all files in Avocado Dashboard
are covered by the GPLv2+ (i.e: v2, or any later version).`

(or something like that)


Sure


nooice.


Awesome.  Unfortunately, there are probably more custom properties from Teiid, but we'll have to look through their Docs to find them.


@einatsc
Hi Reuven,

we updated the presentation as you wanted.

https://github.com/havivha/sw-modeling-class/blob/master/project/Open_Whisper_Systems-TextSecure_v2.pptx


:heart_eyes: Thanks!


I'm getting a test failure after the update:
```
...
Failures:

  1) Git solves rebase level
   Message:
     timeout: timed out after 400 msec waiting for trees should be equal
   Stacktrace:
     undefined

Finished in 29.989 seconds
126 tests, 113 assertions, 1 failure, 0 skipped
```


@benbieler could you please review the file src/Sententiaregum/AppBundle/Resources/public/js/application/app.js?
That file is a wrapper for backbone and react


> I don't think you can do the check this way

It works fine for me inside a function.  Try doing ```function test(a) { console.log(a===undefined);}; test();``` at the console.


You're right
I even wrote in the "writing-tests" guide that `sys.path.insert(0, ".")` comes after the `unittest `import.


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 48](https://teamcity.labkey.org/viewLog.html?buildId=253478&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 2, passed: 24 Build time: 00:34:35
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyScatterPlot: <no details avaliable>

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

```


@racerx What about having it be something about `/dev/tty.Bluetooth-Modem` or something, for increased authenticity? :bowtie: :thought_balloon:


- svn checkout svn://scm.r-forge.r-project.org/svnroot/rgdal/trunk
+ svn checkout svn://scm.r-forge.r-project.org/svnroot/rgdal/pkg
- etc. TODO


Non ça ne sert à rien, les dimensions de la grid sont calculés de façon dynamique, donc il faut mettre la taille des tile en dynamique, j'ai commencé à le faire.


haha, apparently your editor added it back in? oh well


20150604
重写了 登入，验证，注册，验证....


Perfectly agree of course.
But the idea of not knowing how long a character is makes the software much slower processing text.
Cant we adopt internally unicode? We deal with unicode with no problems after all these years.
Beside, the UTF8 stuff we are using is weak and can crash any time if a file is corrupted.

From: Bruno de Lacheisserie
Sent: Thursday, June 04, 2015 4:03 PM
To: LK8000/LK8000
Cc: Paolo Ventafridda
Subject: Re: [LK8000] Overlay: calculate correctly max possible len of waypoint name, dinamically (ced462f)

to different subject :

  a.. input charset, depends of what users do, so all input need to be checked and converted to internal charset.

  b.. internal software charset ( utf8 for linux, unicode for windows ), so all code need to be writed compatible with internal charset, for utf8, a characters are not always one byte !

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


Syntax error, self is undefined. Fix:

self.Button.OnChange = function( self, _, val ) self:OnChange( val ) end


Right on! Back to having `LUA=1` again! :-)


No, there's no special thing called a Server in dbus ... It's a name I made up as it's the first point of contact to the client from where it can control other objects.


ORCID has the Auth and redeem code on different domains. Simple-OAuth2 doesn't support that yet. I've already submitted a pull request: https://github.com/andreareginato/simple-oauth2/pull/38.


maybe `\z`? @rafaelfranca


ben c'est ce que j'ai mis en commentaire non ? ...
J'ai modifié leur taille pour ce ca corresponde avec les rectangles pour le moment


comment not clear?


Ah okay, makes sense to do like this :)


If we keep email it will not allow it to be empty.
But we should and change the Email requirement to allow empty string.


this was a place holder. I will add dmesg disconnect style message, but
bigger issue is that the message shows up on aplite but not on basalt.

On Thu, Jun 4, 2015 at 11:34 AM, Ish Ot Jr. <notifications@github.com>
wrote:

> @racerx <https://github.com/racerx> What about having it be something
> about /dev/tty.Bluetooth-Modem or something, for increased authenticity? [image:
> :bowtie:] [image: :thought_balloon:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/racerx/pw-dos/commit/cc8a8d103f081223f36af8998162846aea9c758c#commitcomment-11523896>
> .
>


I'm not sure I understand. As far as I can tell, `#begin` and `#end` return the beginning and end of the range irrespective of whether that range is ascending or descending. By sorting them, you can find your `min` and `max` values.

```
start = Date.parse('2015-06-05')
finish = Date.parse('2015-06-03')
range = start..finish
[range.begin, range.end].sort.map(&:to_s)
=> ["2015-06-03", "2015-06-05"]
```


Sorry, I don't understand this. Dots at `bus` and `path` ?


:+1:
Good for you.


Borra este import


ah oui, pour le moment.
J'avais pas compris le commentaire pardon.


If you are trying to suggest something else, let me know.


Vas bien Camilo


I originally meant that in get_bam_lock of AR.pm:
my ($self, $package) = @_;
my $class_name = $package || __PACKAGE__;
$package is optional so the change would be minimal and __PACKAGE__ is not needed for get_bam_lock at create_bam_flagstat and create_bam_header.  But your way works too.


actually, it does show up on basalt. Just needed to wait a bit longer to
see it.

On Thu, Jun 4, 2015 at 11:36 AM, Bryan Thomas <bryan.r.thomas@gmail.com>
wrote:

> this was a place holder. I will add dmesg disconnect style message, but
> bigger issue is that the message shows up on aplite but not on basalt.
>
>
> On Thu, Jun 4, 2015 at 11:34 AM, Ish Ot Jr. <notifications@github.com>
> wrote:
>
>> @racerx <https://github.com/racerx> What about having it be something
>> about /dev/tty.Bluetooth-Modem or something, for increased authenticity? [image:
>> :bowtie:] [image: :thought_balloon:]
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/racerx/pw-dos/commit/cc8a8d103f081223f36af8998162846aea9c758c#commitcomment-11523896>
>> .
>>
>
>


Why not just use autoprefixer?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.239](http://teamcity.monogame.net/viewLog.html?buildId=8010&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:28:10


Thanks, I just added it back.


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


Oh, sorry, I misunderstood and somehow did not see that `.sort` in `[range.begin, range.end].sort`. Will update


ok, good to know about the limits of typeid. I suspect the "correct" way to do this might be to use QueryInterface instead, but at the moment I don't think our DXGIDevice wrapper handles that (or I missed it), and I didn't want to touch more code then necessary until I get a better understanding of how that is supposed to work.


"If needed" is pretty much always a redundant phrase. I intentionally kept this passage terse because it doesn't deserve much space, but it is worth documenting. The new version is too chatty.


Perfect correction in modification.php.


Ehh sorry, I mean at the end of line in the documentation sentence, so:
"[...] connect this object path to**.**"
"[...] can communicate**.**"


YAY :+1:


It's a very frequently found function in Django where it creates a table in the database if it doesn't exist :D

Basically, when a new client is created, I need an app for it.
So, it needs to create an application for the new client if it doesnt exist, but if it does exist, it should use the already existing app for it ... so, created a function for it


Please look at https://docs.docker.com/articles/networking/#container-networking
I think it should be --net instead of --hostname


That sounds like a good idea to me :)


looks reasonable. Question, if _DEBUG is not defined do all DUMPPTREE calls get processed as typical function call? Is there a way to force an actual noop?


Missing translation


```
 π          π
--- * s +  ---
 2         s+1
```

Why is that? How does it make sure that [practically] every angle is tried out?


Just not have any led.


Won't this always be true, seems to be `(!blah) OR YES`


What happens after 1000000th iteration?


change it properly


Maybe it was already there. I only found the problem today.
Op 4 jun. 2015 16:51 schreef "Martin Thoma" <notifications@github.com>:

> Did this problem not occur before, but was introduced by my changes?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BVLC/caffe/commit/adf31899fde826b98d64e4c344be1924bfa52b7e#commitcomment-11521622>
> .
>


The function names are very self explanatory ... I don't really have much more to say other than "Creates a dbus application" ... or "Disposes of a dbus application" ...

Is there something else that you would like me to mention ?


Okay. I'll create another issue to remind someone to switch back to simple-oauth package on npm once that PR is merged.


You convinced me :)


Just confirming they do show up properly now.


-5 could be the 60 second sell RPI students do for PD3 ?  That takes place early in the school year and it's a great for people to do and receive feedback on.


No that would be pretty fine.
You can mention some extra behaviours like for `dispose_app`, if the app is not existent, nothing happens (so the user knows nothing is thrown).


Forgot to remove this?


It strikes me as most similar to type promotion, though maybe not identical.


I don't think fopen with "a" sets CLOEXEC, so it's possible the log file fd could be available in compositor child processes?


Ah, sorry the way they ran the projects is that they copy the source and host it from their own web server to test it.  I doubt it will work just browsing to the index.html on your local webserver due to cross-origin requests issues.  The way I always ran it was hosting it on the local webserver on a VM on my laptop and then browsing to that port on the host machine (my laptop).

Sorry it is a hassle.  Here it is hosted on my laptop if that helps.
https://22c9048f.ngrok.com/


Verified I receive the rc variable in sessionstate now as well. Thats new (ish). [Here is a comparison](https://dl.dropboxusercontent.com/u/9659390/MFC/roomcount_comparison.json) of a snapshot of the two counts. For the most part very close but not exact, which is to be expected.

I was looking at the most popular rooms list yesterday at some point and I think it was a dead on match for what I was getting for room count based off the roomdata messages. Not sure which is preferred.


@sapk Wouldn't you need to use a regex to check an email is supplied (if there is text in the field)? To prevent an exception.


Guess you get isDir set in `fileExistsAtPath:isDirectory:` but it's a bit confusing.


These probably belong in a separate commit, don't seem to be necessary for moving file creation?


As Ivan pointed out, I would like to follow that up. Are PostgreSQL and
SQLite deprecated on Otserv?

On Thu, Jun 4, 2015 at 7:24 AM, Oso <notifications@github.com> wrote:

> Those are good ideas. Would the preference be to revert and redevelop and
> commit again? I wouldn't be able to get a fix for this any time soon.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opentibia/server/commit/b06bf832a0d36c2c91169553d79afd8c7e0fb1f3#commitcomment-11522350>
> .
>

--
*Alexandre A. Severino*
*Systems Developer and Analyst*
*+1 (941) 445 1824*

*Pacta sunt servanda*


I did it because in some projects I was working on, I was creating all the needed gestures in the 'ready' function, but I wasn't using all of them in that moment, I didn't want to loop over the 'unused' gestures.

Looking at it now, it hasn't much sense though. It won't affect to performance to loop over seven gestures at most.


I think you should revert the findIndex code. The while loop was a part of a refactor to improve speed.

http://jsperf.com/while-vs-map-findindex


Jenkins test this please


>:I


comentario en la linea


This looks like noise.


Xamarin.Insights!! version update to 1.10.3.111


Comentarios en la Linea Nro 2


Thanks for replying.
I tried as suggested, but still i can see below error message.

Any expert comments!!!

0 info it worked if it ends with ok
1 verbose cli [ 'c:\\Program Files\\nodejs\\\\node.exe',
1 verbose cli   'c:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js',
1 verbose cli   '--proxy',
1 verbose cli   'http://proxy.XXXXXXXXX.com:80',
1 verbose cli   'install',
1 verbose cli   '-g',
1 verbose cli   'openfin-cli' ]
2 info using npm@2.10.1
3 info using node@v0.12.4
4 verbose install initial load of C:\Users\XXXXXXX\AppData\Roaming\npm\package.json
5 verbose readDependencies loading dependencies from C:\Users\xxxxx\AppData\Roaming\npm\package.json
6 silly cache add args [ 'openfin-cli', null ]
7 verbose cache add spec openfin-cli
8 silly cache add parsed spec { raw: 'openfin-cli',
8 silly cache add   scope: null,
8 silly cache add   name: 'openfin-cli',
8 silly cache add   rawSpec: '',
8 silly cache add   spec: '*',
8 silly cache add   type: 'range' }
9 silly addNamed openfin-cli@*
10 verbose addNamed "*" is a valid semver range for openfin-cli
11 silly addNameRange { name: 'openfin-cli', range: '*', hasData: false }
12 silly mapToRegistry name openfin-cli
13 silly mapToRegistry using default registry
14 silly mapToRegistry registry http://registry.npmjs.org/
15 silly mapToRegistry uri http://registry.npmjs.org/openfin-cli
16 verbose addNameRange registry:http://registry.npmjs.org/openfin-cli not in flight; fetching
17 verbose request uri http://registry.npmjs.org/openfin-cli
18 verbose request no auth needed
19 info attempt registry request try #1 at 11:43:33 AM
20 verbose request id bac2ec20b20a6c15
21 http request GET http://registry.npmjs.org/openfin-cli
22 info retry will retry, error on last attempt: Error: connect ETIMEDOUT
23 info attempt registry request try #2 at 11:44:04 AM
24 http request GET http://registry.npmjs.org/openfin-cli
25 info retry will retry, error on last attempt: Error: connect ETIMEDOUT
26 info attempt registry request try #3 at 11:45:25 AM
27 http request GET http://registry.npmjs.org/openfin-cli
28 verbose stack Error: connect ETIMEDOUT
28 verbose stack     at exports._errnoException (util.js:746:11)
28 verbose stack     at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1010:19)
29 verbose cwd c:\Working\Openfin
30 error Windows_NT 6.1.7601
31 error argv "c:\\Program Files\\nodejs\\\\node.exe" "c:\\Program Files\\nodejs\\node_modules\\npm\\bin\\npm-cli.js" "--proxy" "http://proxy.ml.com:80" "install" "-g" "openfin-cli"
32 error node v0.12.4
33 error npm  v2.10.1
34 error code ETIMEDOUT
35 error errno ETIMEDOUT
36 error syscall connect
37 error network connect ETIMEDOUT
37 error network This is most likely not a problem with npm itself
37 error network and is related to network connectivity.
37 error network In most cases you are behind a proxy or have bad network settings.
37 error network
37 error network If you are behind a proxy, please make sure that the
37 error network 'proxy' config is set properly.  See: 'npm help config'
38 verbose exit [ 1, true ]


Comentarios generales


RPCD_AIO is neither alive nor dead.


I should add a condition to check that the e-mail is part of the domain. This means that I should probably have the domain inputted as a data member of the class.


test


One minor comments, otherwise looks good


initializeForView should be initialized in onCreate, otherwise it will be run before onCreate and setContentView which could do weird things


Sure

On Wed, Jun 3, 2015, 18:22 Drew DeVault <notifications@github.com> wrote:

> I filed a bug with mono
> <https://bugzilla.xamarin.com/show_bug.cgi?id=30613> that has been
> triaged and will hopefully be fixed soon. Until then, I'm just going to
> keep breaking it and leaving it up to others to fix if they want it to
> build on Windows, at least until we get the runtime dependency resolution
> working.
>
> @robinkanters <https://github.com/robinkanters> if I give you write
> access to the repo, would you volunteer to maintain Windows support?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SirCmpwn/TrueCraft/commit/c8497b887ad90f346bc31708552569807f107f2a#commitcomment-11502944>
> .
>


Whats this change for?


Do you mind adding `.sass-cache` to `.gitignore`? Also, we should probably keep Foundation out of `bower.json` as an update would destroy your modifications.


one more :)


fantastic!!


Ooops, this isn't your fault at all, it's in weston now.  sorry.


Not sold on the name Regular here... The only way to understand what it means it to compare it with the UniqueIndex.


This reverts this part of the commit, please let me know
https://github.com/example42/puppet-iptables/commit/59f722c9ffa9ac531b0d397c73793fc30e38318e


:ok:


This should be after game.startGame, since you only need to set the listener once.

Then this method will do the UI updates when the state changes


we'll probably have to go in later and switch the quote types in this file. so no worries on these!


Hmm. `privateData` perhaps -- this name doesn't really suggest it's usage.

Also, do you need a new property rather than a custom setter/getter?


:+1: every bit of this!


minor nitpick, shouldn't this be content_type unless it's a function?


:beers:


I think generating redirect_uri from config makes sense


De rien mais demande moi si tu as besoin d'aide ;) Baye :P

2015-06-04 16:15 GMT+02:00 Mathieu HIREL <notifications@github.com>:

> Fix link (thanks @kweebix)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ralmn/DashBoard/commit/0089db668f241ae3dee60b5a8dbf1bfa82b48ff5
>


Nikhil, I'm interested in connecting. Could you shoot me an email? kyler.vandeberg@godynamo.co

Thanks.


No unit tests supporting these changes. You should add some to AssumeIndependenceQueryGraphCardinalityModelTest at least


```
IDACSharp...
Loading IDACSharp...
Done.
ОґХТµЅ·Ѕ·ЁInitЈЎЗлИ·ИПТС°СIDACSharp.dll·ЕµЅIDAёщДїВјПВЈЎ
---------------------------------------------------------------------------
Removing plugin...
ОґХТµЅ·Ѕ·ЁTermЈЎЗлИ·ИПТС°СIDACSharp.dll·ЕµЅIDAёщДїВјПВЈЎ
```
 :C


Actually, we should error out so the user notice this problem.


@alexkalderimis no hurry at all. Just thought I would check to see if it is still likely to happen. I certainly understand being too busy to spend much time on personal projects.


I'm not in favor of hardcoding file locations inside the client.


@simon3z fyi


It would be nicer to move this to a function in mono-threads.


Do you mean that the F* compiler emits platform-native line endings when using `--ocaml-output`?

That seems like a good trick, thanks!


I'm not sure what this really means for either us or an external developer?


The view puts this in the serializer context once and saves repeated fetches


Weirdly named constant?


:ok:


What for? Is this checking it's at least 1 CBC block + header?


Is this lookup guaranteed to succeed ?


I think this will break some ZFS functionality.
Specifically, it will probably break the auto-mounting / auto-unmounting of the `.zfs` snapshot directory tree.
During the build, we are doing this:
```
substituteInPlace ./module/zfs/zfs_ctldir.c   --replace "umount -t zfs"           "${utillinux}/bin/umount -t zfs"
substituteInPlace ./module/zfs/zfs_ctldir.c   --replace "mount -t zfs"            "${utillinux}/bin/mount -t zfs"
```

Which means that the kernel module will contain direct references to the location of the `mount`/`unmount` binaries in the Nix store. But if we indiscriminately nuke all references to the Nix store, then the ZFS kernel module will point to invalid paths.

Or am I misunderstanding something?


Thanks Andre,

Appreciate the quick resolution.

Regards,
Liam


They aren't, it's relatively easy to change their schema even without knowing much about it. Anyway I would suggest to revert the changes and make main Logger class for such purpose (not only for conversations of players but actually a logger for everything). The logger should have it's own thread.


It's exactly the rules of boolean `&` with `LinearFast=true` so I don't think `&` is that weird. You could say the same with `({0,1},*)` and `LinearFast=1` so I don't think `*` is that weird either.


Remove these?


95652938-jsoncpp (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1670 of 1670 tests passed)

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


cancel the wait, not the thread.


This could get the comments which were at wapi_interrupt_thread ().


Hey Tim, I think there's a string import that can be dropped in this file.


i agree


Do I even want to know why


@abonas It's a hard coded file location in kubernetes. This is the location that the service account token is mounted inside a pod if a service account is specified in the pod spec. It's a convenience more than anything as of course end users could just read the file & pass it in as bearer_token, but this makes it transparent.


This might need some asserts.


we deal with unicode on windows because only unicode version of Win32 API exist in winCE !
in linux, native charset are utf8.


would be better to use [nocrypto](https://github.com/mirleft/ocaml-nocrypto) instead.


thanks for your submition @rmccune8
I'm wondering if this change is backward compatible.
Meaning, people using current version of the library, can upgrade to this version without code changes and it won't break what they already had?
Specifically the change from Tab to DateSignTab, would that still work for people using Tab? I am thinking it's ok since it's inheriting from Tab, but not 100% sure there's no scenario where it would break, did you think of that?

Also, just in general, I assume this was tested and works fine and you had some unit tests. etc? right?


possible division by 0, should this be checked?


Nevermind, I just committed those changes.


why change what already works?
But really, we could, but we already have everything setup and it works fine for us.
We'll probably move to use autoprefixer down the road, but I wouldn't expect it any time soon, unless you wouldn't mind putting together a PR for it.


Evite les vector c'est deprecated => List / ArrayList


@hvpareja  the things is - in the group names breaks things, but i think there is a feature request to make this configurable in ec2.ini


Et pour les behaviour on estp lutot sur des ticker à 1 sec


Before 320 MB ram usage
After this patch ram usage 50 MB

Good job iryont


Creo que el precio de la refineria deberia ser un atributo (o constante) dentro de refineria y asi, en el constructor de refineria preguntas si el jugador tiene suficiente mineral y/o gas para hacer la operacion


doesnt look pretty
My toolchain went missing


You could just as easily use a single table and split in partition. Bear in mind that if you send every row to a different partition (as it sounds like you're currently doing) you won't get too much in the way of scale out performance.


@renemarcelo, can you take a look at the translations in this commit?  They were done with google translate, so likely not great.


C'est pas moi qui ait codé les vetors etc... Faudrait vraiment une réunion tout le monde est parti sur sa petite idée la


Check https://github.com/kern/minitest-reporters/blob/master/lib/minitest/reporters/ansi.rb#L6-L9
I think you should add some logic there.


change this to res.bootstrap('user', account);


Et


aha, this might have been it. (issues setting up the album art)

CC: @kad3mlia


Maybe I changed that... I think it just makes sense to just do

1. system imports
2. set path
3. coalib imports


@renemarcelo, can you also look at these translations please?


@changetip send $1 to @aldenml


Hi @aldenml, @gubatron sent you a Bitcoin tip worth 4,635 bits ($1.00), and I'm here to deliver it ➔ **[collect your tip](https://www.changetip.com/collect/853792)**.

**[Learn more about ChangeTip](https://www.changetip.com/tip-online/github)**


Et pour les ticker à 1 sec ca va créer des problèmes de désynchro j'ai l'impression, il faudrait un truc qui cadence l'ensemble


@MichaelDiBernardo I think this is ready to merge - it's been sitting here for a while but I think that's just because we all got busy with other stuff. :)


Please, do not make me beg you :) Can you update your IDE settings for imports?


Ruby coding standard is to use underscores for variable names, ie to_include. I know it doesn't match our JavaScript code, but I'd prefer to keep ruby code idiomatic ruby and JS code idiomatic JS even if there's a bit of cognitive dissonance there.


Latin -> dicorebus

English -> I Say Things


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1676 of 1676 tests passed)

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


I wrote you privately to continue this discussion.
From: Bruno de Lacheisserie
Sent: Thursday, June 04, 2015 6:01 PM
To: LK8000/LK8000
Cc: Paolo Ventafridda
Subject: Re: [LK8000] Overlay: calculate correctly max possible len of waypoint name, dinamically (ced462f)

we deal with unicode on windows because only unicode version of Win32 API exist in winCE !
in linux, native charset are utf8.

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


looks good. thanks


hmm, maybe use player or engineer instead of astronaut?
because we are not much astronauts. more like construction workers .... ;o)

or icecream, since they are in cryo ... jk ...


@bennojoy https://github.com/ansible/ansible/pull/11165 ;)


no, just remove the "_" of the _resource_get declaration which currently indicates that is a private method. But here you are using it like a public one.


I think that's what we were planning on doing for that. I'll work on organizing that with professors/ the Archer center


@Bouckaert

fiz essas modificações por que as specs estavam falhando mas o ideal é usarmos o [Pundit](https://github.com/elabs/pundit) pra lidar com questões de autorização do usuário.


Какво мислите за включване graph flow algorithms и може би some advanced data structures?


Pour la réunion une conv skype c'est cool !
Sinon je vois pas trop le problème de désynchro et surtout au final si c'est réellement un problème sinon suffit de crèer un agent supp qui lui est un ticker qui envoie à tous les agents hors env un message.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.240](http://teamcity.monogame.net/viewLog.html?buildId=8014&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:29:57


YAY!


Hey Team,

I think we merged the one from my master branch instead, right? Amy, it was
easier to do it there, and I had made your changes and mine directly on it.
I think that means we can remove this PR. Correct me if wrong!

D

On Thu, Jun 4, 2015 at 12:06 PM, Amy Brown <notifications@github.com> wrote:

> @MichaelDiBernardo <https://github.com/MichaelDiBernardo> I think this is
> ready to merge - it's been sitting here for a while but I think that's just
> because we all got busy with other stuff. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aosabook/500lines/commit/ec0d12acd42ed92a92e94f0ebcd5b73a72201744#commitcomment-11524610>
> .
>

--
Dessy Daskalov
Co-Founder & CTO, Nudge Rewards <http://nudgerewards.com/>
425 Adelaide St West, Suite 300, Toronto
@dess_e <http://twitter.com/dess_e> | (416) 670-8945


sorry，这里应该是Update .idea/workspace.xml，斜杠写反了，所以Revert了


See also #26


Unfortunately a lot of automatic whitespace changes, but thats an issue of code generation.

Image of change:

![Uploading VisiCut1.png…]()


Just a query on this one, $this->replaceIn is replacing text in a file, whereas the path which is passed in of "{database_path}/factories" points to a folder, not a file (https://github.com/laravel/laravel/tree/develop/database/factories).

When I run app:name, It's throwing a "FileNotFoundException" from https://github.com/illuminate/filesystem/blob/master/Filesystem.php#L40 as its expecting a file.

All the other calls into replaceIn are passing individual .php files. I'm guessing this should be looping through any files within the factories folder like in https://github.com/laravel/framework/blob/5.1/src/Illuminate/Foundation/Console/AppNameCommand.php#L96


希望保持与之前的描述一致


Criei um [card](https://trello.com/c/rG8MwNFx/60-utilizar-pundit-para-autorizacao).


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


On Thu, Jun 4, 2015 at 5:56 PM, Jonathan Protzenko <notifications@github.com
> wrote:

> Do you mean that the F* compiler emits platform-native line endings when
> using --ocaml-output?
>
Yes, it seems so. At least the line endings I got were different from the
ones in the repo.


Ca va nous amener à la semaine pro ca....
Bah si ca peut creer des probleme de désynch si le capitaine attend plusieurs tour avant de prendre une décision ou ce genre de choses...


Are these changes stable?

Latest version of the plugin in the October market is 1.0.12 from Apr 23. Are you planning on releasing a new version soon?

Thanks


@blankenberg do you know if we can write tests for such tools?


Yeah, I'm also not yet sure what the "correct" way is for these as well.  I don't quite have a handle on how people use QueryInterface, and the documentation is terrible.  As we fix bugs we'll get a better sense for how this works.

That is also why I added so much logging around QueryInterface, including converting UUIDs to types.  There's a bunch of stuff I added to the DXGI wrappers just to log so that we can see how they are using the API.  I use these logs to find object leaks, where we haven't wrapped something we should.


I dont know if we want the auto logout set


![thickfiction desktop wireframe](https://cloud.githubusercontent.com/assets/12265114/7988731/f3e1b080-0ab2-11e5-8833-a18da6f98173.png)
![thickfiction mobile wireframe](https://cloud.githubusercontent.com/assets/12265114/7988730/f3df1c76-0ab2-11e5-99bb-17ae995ced09.png)
![thickfiction mobile wireframe 2](https://cloud.githubusercontent.com/assets/12265114/7988732/f3e45614-0ab2-11e5-8d05-308c3a11eb1a.png)
Created wireframe for mobile and desktop layouts.


`**kwargs` is always a fresh dictionary, so it can never be shared.


This feels a little weird due to the map. I think I'd write it as `self.pop().and(Forth::ok()).


okay. thanks


Actually I think "reenter" is fine, and we generally write documentation in American-English. Similar to the [Rails Guideline](http://guides.rubyonrails.org/api_documentation_guidelines.html#english), though I'm not sure we have documented this anywhere..


Problem link :
http://lightoj.com/volume_showproblem.php?problem=1083
Lets say min value(from 1 to n)'s index is mnindx
so if we take this positions value we find a area of arr[mnindx]*(n-1+1),
again we can split the array to arr[begin-->mnindex-1],arr[minindex+1-->end] and find the solution recursively..

lets say, you have an array of rectangle example [4 4 3 2 4], you can get a area of length(5) * minimum in the array(2) that it is 10. In order of use a bigger height you have to delete the 2 so split the array [4 4 3], [4] and make the same problem recursive.


PHPCS: Whitespace found at end of line


not needed, in neutral Spanish you can use them at the end too.


Ah, yes! Good advice, thanks. :)


_std::vector_, _std::list_, _std::deque_ だけでなく、自作のコンテナでも自由にVMArrayに型変換できるようにしました。
（ただしSTLコンテナの要件を満たしている必要があります。）
同様にVMArrayからの逆変換もできます。
配列からの変換もできます。

（例）
> // OK
> std::vector<SInt32> a_vec(5);
> VMArray<SInt32> arr = a_vec;

> // OK
> SInt32 a_arr[] = {1, 2, 3, 4, 5};
> VMArray<SInt32> arr = a_arr;


This is to insall ssh


Python 3.4 fails tests, we may add a loop similar to this:
         try:
            mask = set(random.sample(xrange(batch_size), number))
        except NameError:
            mask = set(random.sample(range(batch_size), number))


Pour une réu skype je suis op quand tu veux pas besoin dêtre la semaine prochaine.
Sinon même si le capitaine mets plusieurs tick à décider ca le désynchro par rapport a qui ?
J'avouerais avori un peu de mal à comprnedre les problèmes de désynchro mais bon à voir !


* TODO: refactor these 2 functions


Hi Igor,
Thanks, I fixed this.

Ciao,
Pri


Does uwdb really require superuser authorization?  I don't believe I'm doing so on the EC2 side.


Hm....


What about version zero?


Weird uncrustify moved this over. Will fix.


added `export ORCID_REDIRECT_URI=http://localhost:5000/orcid_auth_callback` it's easy to read that way. However we may want to instead have a config for domain name and concat a derived redirect url?


Yep.  At least, I think so.  Not sure whether the Python runtime guarantees this.  Felt like it was worth changing regardless, though, to set a good example.


It would be nice if there was a way to ensure that the tests keep passing on windows. It doesn't sound like travis will support windows in the near future: https://github.com/travis-ci/travis-ci/issues/2104


This is a userscript that you install using an extension or plugin for your browser. Such as tamper monkey for chrome or greasemonkey for firefox, After installing the plugin you can press the "raw" button on github and the userscript will install. Or do you want to know how it finds the first post? :p


Parens can be part of the filename. Maybe use something like

````js
/^at\s.+\s\(/.test(location)
````


Merged as part of https://github.com/paulmillr/es6-shim/pull/345


close #5


close #6


merged to master because unsure how to specify omnibus-software to point at a branch


This change breaks usage like EpochTT([...]).tlim(lim)


This is now the 3rd (or 4th) CRC32 code we have in JtR.  We really should unify them into a single source/header pair, and change any code to use this single type.


LGTM


Hmm I don't think this behaves the same, now the caller has to explicitely set a method param.


This is checked and can be taken


scroll down about 30 lines?


me di cuenta que el límite inferior es donde detectan que esta automatizado, ahora lo tengo funcionando sin problemas en el rango 30-100 y creo que funcionará en 30-60


they act appendable


On Thu, Jun 04, 2015 at 09:14:48AM -0700, kiarasky wrote:
> Python 3.4 fails tests, we may add a loop similar to this:
>          try:
>             mask = set(random.sample(xrange(batch_size), number))
>         except NameError:
>             mask = set(random.sample(range(batch_size), number)

xrange also exists on python 2, why not just use xrange all the time?

--
Brian Sutherland


โอเคแล้วนะ CRUD ง่ายๆ


Also added objects for templates and password policy, and clean up of study.


_EMBED_MEMBER_FN_ と _EMBED_MEMBER_FN_const_ マクロを追加しました。
使い方は _DEFINE_MEMBER_FN_ マクロと同じですが、なんちゃってメンバ関数ではく、完全な形でメンバ関数を定義できる点が違います。

EMBED_MEMBER_FN_constは、constメンバ関数を定義するときに使ってください。


Is it worth getting the bytes themselves here? Someone could always create a subclass of CDTEncryptionKey provider which returns `nil`.


Actually, no, it's not equivalent! When driverInfo is an empty string, I want this method to return false. If I leave that part out, it would return true in that case.

And the reason I explicitely wrote "!= null" is that some IDEs treat that automatic conversion into a boolean value with a warning.


I see this merge pull that was from two days ago, it's now irrelevant, hehe.


yes... range() in python3 behaves like xrange() in python2, except it works with values of arbitrary size. And xrange no longer exists, so i cannot call xrange because python3 would raise an error...


What @andreasnoack said. But I suggested the analogy to promotion because it's also similar (`Float64+Float64=Float64`, `Float64+Int=Float64`, `Int+Int=Int`, so `Int=true` and `Float64=false`).


There are workarounds that allow using RQ under Windows, see https://github.com/nvie/rq/pull/426
Otherwise, Pootle runs under Windows with no major issues. Having said that, I would prefer Pootle to continue supporting Windows for _development_. When it comes to _production_, using Windows as a platform should be strongly discouraged.


Well I guess that's one way of working around the line length limit. I'm fine with it if the linter is.


did you have to delete and re-create this activity?


Waiting input from ktoso/sbt-jmh#45.


Could try reading with an encrypted blob reader.


great! i see you have moved these methods to vanilla, now it's just missing in demotivational.


Sounds correct. thanks.
i will check, and i will check effect on existing users...


Hope @vkurchatkin is alright with this :wink:


Couldn't you test these against known-correct SHA1 for the plainText?


Using a single table could be an option (or we could provide both implementations), although many unique partitions increase table scalability.

Anyway, I don't think Don's example is affected by the partitioning scheme. It would be interesting to have a similar example using only Azure tables, to have a baseline to compare to.


okay. thanks


What is the purpose of adding django.contrib.staticfiles and debug_toolbar?


@johanmvingt cf https://github.com/processing-lille/bombsight-video/commit/464eb43d9f97172c536a6acd3e11e0ea5c51dbf7 pour une solution plus puissante et plus générique


`IntelliSense: 定数式では、この演算子を使用できません。`
これがうざくて我慢できなかったので、ReleaseビルドではSTATIC_ASSERTマクロを評価しないようにしました。


@johanmvingt indentation


You should make an attempt to verify the input.  It looks like any form of:
tcp://host:2375
unix://path/to/socket

is valid.


That style of indentation is not the norm for us. Maybe you can align to the opening parens.


Do you mean the existing content is deleted as soon as openForWriting is called? "clean its content" doesn't work in English.


:heart:


Lets go ahead and pull of the band-aid here.  Are you up to define self.conn in applicationconfiguration and then switch each instance that uses it to that?


"even if no data", otherwise it implies the file _is_ removed in all cases but where one writes no data.


Hard coded location in kubernetes is https://github.com/GoogleCloudPlatform/kubernetes/blob/a15c69798b1f01067ac07101c1d92c4a6d540d54/plugin/pkg/admission/serviceaccount/admission.go#L43


Good stuff.


I tried `\z` and it breaks the behavior.


Align the strings to the same indent here.


Just as in the discussion about Int^N, and @prcastro 's suggestion, could you have both?  A nice short form, and a spelled out function, to make it easy to find documentation, etc.?


Yeah, I messed up here. I didn't feel like any particular part was a good stopping point so I didn't end up committing until I found a minor bug in my test case.


"clean its content" again.


Yes. I'll rebase one after the other is merged.  This one should go first IMHO.


But I'm still not getting why `ninja=1`?  Why not just a target `ninja` like `vcxproj` or `count`?


Why remove the advice?


Let's definitely have that in here.  I would think you could hit it with a tcp socket to see if it is alive or see if the api returns an error and then catch it/ raise it.


@cenavarro thanks for your contribution but why do we need to specify a port? Port 80 is specified as the default later on in the code and works for most use cases.


Not default value, but the one it should be, right?


I think this xml can be cleaner before committing. I did clean up this xml a bit in my commit. I recommend merging from master, fixing any merge conflicts, opening up the program and making sure there is almost no commented out code such as this,  refactoring the code, and then double checking to see if the xml is working.


Also indent to same level.


Removed extra character that was breaking the ability to set the Nav_Next param


You are correct. Shall I change it to `location.test(/\(.*:\d+:\d+\)/)`?


Also of interest to @vkurchatkin possibly.


(y)


@digitalextremist I separate the example classes & specs.


Also indent.


I'm using it here to determine if I can resolve the input list into container or image ids.


Thanks. I've given you write access.


hh


Good start!  Happy to discuss my comments.


@silverwind Yup, I ran `make test` and linter didn't complain :-)


In test method check app works fine or not when using this API in code.


That could work too, yeah.


It is possible that we'll get a response that is not valid JSON, and so `res.json()` will raise `ValueError`.

I think we should catch that and raise a KeenApiError instead.

And was it intentional to do this `raise` regardless of the response status code?


This isn't the mysterious part.  The comment simply repeats the name of the function, which was perfectly good to start with!  :-)

The part I commented on was the complex logic after line 755.  I'm going to have to port the logic to the new system (which is by default data driven so for things like this you wouldn't write code) and I need to understand it.


@silverwind It is a setter, so I thought `getConnections` would not be right here.


$routeParams also shouldn't be needed anymore right?  It's also exposed by ngRoute


@ximinez Nit: Update the comment on line 62.


Our linter rules are just too tolerant right now :weary:


Initializing a `std::unique_ptr` with NULL is a no-no in C++11 mode, the previous version of the code ommitted it for a reason.  You can initialize them with std::nullptr, the issue is that `NULL` is ambiguous with integral types.  While we're at it, can we also initialize one member per line?  Just makes it easier to read.


:+1: We're about to do the same.


On Thu, Jun 04, 2015 at 09:27:17AM -0700, kiarasky wrote:
> yes... range() in python3 behaves like xrange() in python2, except it works with values of arbitrary size. And xrange no longer exists, so i cannot call xrange because python3 would raise an error...

Ah, I always forget the details of the python2/3 transition, you can do
this kind of code at the top of the file:

_PY2 = sys.version_info[0] == 2

if _PY2:
    range = xrange

>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/vanguardistas/mp.importer/commit/188b8b0cb2ad727d209185f378339c11960ca186#commitcomment-11525007

--
Brian Sutherland


FWIW On Windows 8 it does call `evr!CMFVideoSample::Release`.


ok cool, thanks! :)


it should be 'schedules' not 'scehdules'.


Tim komm mal in Skype online!!!


@seven1m You caught me committing straight to master from the GH editor :)


Again, if a message is supplied, then we should be consistent and do the same thing `assertEquals(String, Object, Object)` does. unless we have a very strong reason not to. Yes, people may disagree on whether that behavior was the right thing to do, but being consistent makes the API more predictable and avoids unnecessary back-and-forth in the code reviews.


OK.


> @abonas It's a hard coded file location in kubernetes. This is the location that the service account token is mounted inside a pod if a service account is specified in the pod spec. It's a convenience more than anything as of course end users could just read the file & pass it in as bearer_token, but this makes it transparent.

I don't think it's correct to hardcode this in the client. Any change in this string in the server code (and file locations like this tend to change) will break the client and will require immediate client code change, instead of just passing it to client.
Moreover, please keep in mind that the client is not always used inside k8s, it is often used to access the REST api from outside of k8s. So k8s internals like this really shouldn't be part of it.


@danielmurphy nothing wrong with that. Progress!


2. Yep. In cnx the URL lines up nicely so it is `<a href="[UUID]#optional-link-to-element">`


95652938-jsoncpp (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1676 of 1676 tests passed)

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


thanks.
two fast questions:
1. what about the "-O3" in case enable_debug is off?
2. I understand that on the way we can also remove _DEBUG that doesn't seem to be used. Do you agree?


Oh, that's right! Thanks, @dessy. I am juggling too many things, obviously! :)


Juntar todas las llamadas a Google Fonts en una sola.
Recuerda la clase de performance, cada llamada al servidor es tiempo que le hacemos perder al usuario.
Además, hacen falta tantas tipografias? Descarga solo los pesos y estilos que vayas a utilizar.


 Las rutas de los archivos no estan bien configuradas para funcionar en dist/
El workflow de Grunt está roto.


That's right, so I'm packing static (python-static) and dj-static (python-dj-static) and I will put these on my COPR repo.


95652938-jsoncpp (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1676 of 1676 tests passed)

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


There was a bug when zooming into a page, the status overlay view would go ahead, leaving the web view to display it's content above the URLBarView. Turns out the topLayoutGuide changes height when zooming in, so I set this to the statusBarFrame height instead (which is probably what it should have been in the first place).