> Thrown together

Nice, thanks, that's all great feedback :)

> The threshold actually seems high. There is a really nice website with node stats:

That's a great site - thanks - I'll define the threshold in the header file so it's easy to change - do you think 2K txns with higher fee is an appropriate number?  I don't want to be firing the warning too often, but obviously do want to catch those times when the user is going to suffer long wait times.

> In my opinion the language could be softened ... Maybe a concrete number

Sure, happy to soften the language a bit & add in the count result to the warning :)

> It would be great, if there were a quick note how/where to increase the fee.

I'm in split minds here about whether we should actually move the transaction fee part of the UI into some kind of configuration tab since currently it isn't obvious that this applies to all transactions (it could easily be interpreted as just applying to Bitcoin sends to people unfamiliar with the underlying technology) but yeah I agree a quick note on how to change the fee would be useful here.

> I suggest to extract it, and potentially others, too, and create a file for all "simple" dialogs

That file already exists (see omnicore_qtutils.cpp) and some dialogs are there (such as ```PopulateSimpleDialog``` and ```PopulateTXSentDialog```) - I'll take a look at moving others :)


видимо у тебя остаются пробелы после точки с запятой и фигурных скобок раз код так пляшет в итоге?


97668326-WholeBuildingLoad (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1674 of 1677 tests passed)

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


打发打发


Gostei do rodapé!! hehehehe


cool :+1:


Since self.geom is an np.ndarray, you can just multiply it by the conversion factor.


97647908-DocIssues4615 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1755 of 1755 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)


Kind of immature. I don't care who wrote this either.


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

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


Don't make me regret coming here.


`return` marks the end of a function. Because we're `returning` on line 16, we will never reach line 18.

Meaning: please delete lines 17 and 18, as they're completely unnecessary.


Nm


same comment as above. =)


This doesn't help the emulator at all. It's immature.


Ditto on lines 34-35 being unnecessary.


> I'm in split minds here about whether we should actually move ...

Well, one step at the time, I'd say. And a one-liner, pointing to the fee configuration, already clears up potential confusion, whether the fee configuration has an impact on Omni transactions.

> That file already exists ...

Hehe, yeah, I had `PopulateSimpleDialog()` method in mind, when I mentioned it. :)


hahaahha


Do we know if this `type` property is used at all? It looks to me that we get the `getType()` method for this purpose.


Marc:  Can anyone but you run these compiler tests?  They seem to be tied pretty tightly to your local setup.


The code should cause some exceptions, like ClassCastException. because all IEditorInput(s) are not RepositoryEditorInput at all.
maybe, also need check the NPE for getActiveWorkbenchWindow() and getActivePage().


why `Ember.RSVP.Promise.all`?  It looks like there's only one promise in that array.

could possibly just be
```
this.store.find('friend').then( function (args) {
  _this.set('friends', args[0]);
});
```

The return doesn't even seem terribly important there considering your `then` function isn't returning anything.


I'll check it out.  I did just discover a small bug though - the error message for blocked layers will always show `uri` of "BLOCKED" - probably not that useful.


Also, we can get rid of the `BLOCKED` constant in LayerStore now as we're no longer using it.


#!/usr/bin/env python


so glad to see all this chisel stuff getting added!


Sorry if these comments seem like I'm nitpicking, but as you'll soon learn, programming is allllllll about attention to detail. An ounce of prevention is often worth a metric ton of cure when it comes to writing code, so it's good to establish good habits (such as cleaning up any extraneous code lying around) from the get-go.


I think you should mention UTF-8 here.  The real header is calculated with bytes, not strings.


Oh, hang on. I've just seen that the `getType()` function looks at the `type` property. The function returns a layer class, while the property is a string type of the server.


Getting rid of the duplication between the addUsingLayerLinkDefault and the addUsingLayerLinkNcwms methods is nice.


If someone tries to use this as Java code, the `==` operator could lead to trouble.  On the other hand, it's just pseudocode.  I don't know...


here


you can edit online, there is a pen sign you can click and it will go in edit mode


@gumaestri


Awesome! Thank you, @beckettsean.


Конкатенация ни к чему т.к. это всё одна строка, если по каким то причинам стоит задача разделить, то в echo можно передавать параметры как в функцию через запятую.
echo '

',  'Массив отсортированный в обратном порядке по ключам через ф-ию krsort', '
';


Added Delay() in I2C_Stop()
Added initial value of buf in I2C_Start()
 - compare previous write data and present write data


Thanks :)


Why do this?


Silly americans https://books.google.com/ngrams/graph?content=canceled%2Ccancelled&year_start=1800&year_end=2000&corpus=5&smoothing=3&direct_url=t1%3B%2Ccanceled%3B%2Cc0%3B.t1%3B%2Ccancelled%3B%2Cc0.
By the way, did you read my first comment about redirecting?


Oh wops, I was midway into implementing Chris' comment when he said it didn't need to be in this PR.


I noticed that too, sent a PR fixing this https://github.com/facebook/flux/pull/241


This is redundant if you only want to create 1 user. You could just ditch the loop altogether.


This is a duplicate of line 18.


Given that you only create 1 user, you could put these details on lines 14 and 15.


Things change, dost thou not thinck thusly?


LGTM


@luiges90  对了，为什么我弄的委托军区不新编限制不了AI新编编队？


FYI this code fixes the issue I am seeing:

                var canvas1 = new fabric.Canvas('canvas1');
		canvas1.setWidth(200);
		canvas1.setHeight(200);
		var c1 = new fabric.Circle({
		    left: 100, top: 100,
		    radius: 50, fill: '#ffffff', originX: "center", originY: "center",
		    selectable: false, evented: false, hasControls: false, hasBorders: false
		});
		canvas1.add(c1);
		document.getElementById('canvas1').style.width = 100 + 'px';
		document.getElementById('canvas1').style.height = 100 + 'px';


log.Fatalln("ioutil.Readline:", err)


for #70


We have a bunch of .pem files so it would be good to indicate which one failed.


Everything always changes, yet nothing ever does


They are used as queues, but what's the difference here between `list` and `deque`? Is one slightly faster than the other?


nice to meet you


Helloworld


Same here.


I prefer to keep thing simple and just do 'make([]tls.Certificate)'.


ikl


97668326-WholeBuildingLoad (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (647 of 647 tests passed)

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


Partly selfish. My own ISP's hosting uses 3.2 only, and I imagine many others are in the same boat. Later on I'll shim something up to allow FileExistsError generically.


@sashrika ?????
CC @harsha89, this is epic ;-)


Since the conditional for `instance_cls` is checked first, yes.


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


This is what I meant by the pattern. Linq2Sql does some amazing things. The SQL generated is not executed until it is enumerated. This code will do the same as yours, it is just more readable and cleaner; it is the pattern we prefer. I have not even compiled this code yet, much less tested.

```
public IEnumerable<Person> QueryList(object s, string orderbyparam, bool ascending)
{
    var q = db.PeopleQuery2(s);

    // make age show in opposite direction from birthday
    if (orderbyparam == "age")
        ascending = !ascending;

    if(ascending)
        switch(orderbyparam.ToLower())
        {
            case "age":
            case "birthday":
                q = from p in q
                    orderby p.BirthYear, p.BirthMonth, p.BirthDay
                    select p;
                break;
            case "name":
                q = from p in q
                    orderby p.Name2
                    select p;
                break;
        }
    else
        switch(orderbyparam.ToLower())
        {
            case "age":
            case "birthday":
                q = from p in q
                    orderby p.BirthYear descending, p.BirthMonth descending, p.BirthDay descending
                    select p;
                break;
            case "name":
                q = from p in q
                    orderby p.Name2 descending
                    select p;
                break;
        }
    // nothing has touched the database yet
    // up to this point, the linq2sql has been building a data structure using an expression tree
    // once the query is enumerated, linq2sql will construct SQL code based on the expression tree

    // we want the sort to occur before the take
    return q.Take(1000);
}
```


I considered it, but since I didn't have an output I thought it was best to leave it as is. After the changes required are done I agree it will make sense to test the class on its own.


Its possible to get away with no class on the button. You can select it in css using "header button"


Yeah changing the zoom was a quick hack. I took your advice and implemented this in ConsoleView::CreateFont. I squashed this commit, new one is 76c565e


Identity is hard to know if real or not online


I'm not sure about what the assignment is asking here. I made an if statement to check if there are mediaItems and then to refresh to load new images. When I ran the app, it did load new images without pulling to refresh, but I don't think I'm checking the right thing here


I like what you're doing here, @jcockhren !  Will you be submitting a PR when you're all finished?


The class ".Ass" can be removed by using the selector "#Benefits span" and "#Benefits ul".
There is nothing wrong with using the class, just another way of looking at it.


rax retry tests


Please us "br" tag only for text formatting. CSS should be used for layout. using margins and paddings is recommended here


yeah, `deque` should be slightly faster.  I haven't verified it, but list should have to do a malloc for every element pushed onto it, but deque should be able to do larger allocations (at the cost of not being able to remove elements from the middle of the deque).


This works and does the right thing. Good work!


ooOooOooOoo spooky


Hello, Thank you for your detection framework.
Can explain pleft, dx , sx meaning? I am confused.And in data.c line165: left  = boxes[i].left  * sx - dx; What's the intuitive? Thank you very much!


Please upload the name list to GitHub. Access to google is via phone's mobile data, so expensive.


also DAFUQ IS DIZ SHITZ?


err I guess what I'm saying is, maybe we don'


Ask karel tommorow


Hm that's true. I often had to think about it myself for a while.


don't need a special case for instance_cls?  I'm not sure what about its destructor makes it any more special than any other one (at least, at the implementation level -- to the user they look similar to the tp_del ones) and it seems like it might get handled correctly be the normal logic.


Oh yeah I like this one. I want to make sure at least one list has all N objects.


i'm a little worries about overriding render in this spec


can we just use

template: _.template('')


Why do this?


The FAT FSInfo structure isn't going to be used again in the bootloader.


增加上下文内容，例如询问“北京到上海的飞机”，小艾会询问具体要查询哪天的航班，此时输入日期（例如“明天”），小艾会查询明天北京到上海的航班，而不需要完整输入“明天北京到上海的航班”。


Oh right, yeah, it's redundant. I think I had it when I didn't assign a default `tp_dealloc`.


結果…

沒有加到 player.points_acquired 跟 team.points 啊（遠望


I think this should be `sampled.equals("0") || Boolean.valueOf(sampled) == "false"`

Would break existing services if they are used to passing "False"/"FALSE" in the header.

What do you think?


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.22](http://pixelated.design/build/viewLog.html?buildId=51&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 56 Build time: 00:00:15


I hope you do. I wish I had your free time. Man, it must be nice for this to be your life...


Why you are still using slim while CZS is not here?


select hosts.* from hosts inner join experiences on hosts.id = experiences.host_id where dob >= '1815-01-01' and dob <= '2015-12-31' and state = 'Zeeland' and hosts.id in(1,2,3,4) and experiences.max_group_size >= 2

host_id = " AND hosts.id IN(#{free_hosts.join(',')})"

group = " AND experiences.max_group_size >= #{group}"


Thank you **so** much for adding this, future forkers will thank you!


Yes, It worked
We can pass port number -Dphoenix.traceserver.http.port=8897. (if not default port number)


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/31844c0276759c876ed7c4ecfc6143adee3fc17b


merging graydon/stellar-core/bug-615-flush-tombstones = 31844c02 into auto


graydon/stellar-core/bug-615-flush-tombstones = 31844c02 merged ok, testing candidate = b383ff7d


Nice!

Can you also add the call `self:backCompatibility()` to `type()`? That's required if you reload the model and cast it right away.


`to can publish`?


No :(


I dont think that's needed. It's always called in updateOutput and accGradParameters at every iteration.


:p


@mattkirby Can you please respond to @xaque208 suggestion.  His suggestion is sound.


Not wrong ^^


I don't think we should use ids for selectors, even though this is pretty specific. If a user had a class they wanted to apply to every field, this would be impossible for them to override unless they had to up the specificity.


I'm also wondering why this error is a type while `ErrRepositoryNotExist` is a value


seems good


Oh yes, they're re-created there, not cast. Cool!


We could look at using `[name="password"]`, I think it will be a *little* better. Also, do we need the input[type="text"] selector?


use `os.Open` instead of `ReadFile`


Github supports Issues to handle inquiries like this. Please create one and we can discuss this there.

https://github.com/austynmahoney/mobile-export-scripts-illustrator/issues


Good to know. I was fearig it was some compiler specific stuff that would make less standard compilers (like the ones I use) fail ^^


You might want to init a static final boolean IS_ANDROID with the value of this method so that it can be evaluated only once. The value is unlikely to change at runtime.


I submitted a PR #6601 to remove that functionality completely, since it doesn't seem to really be needed? Also not sure why its named ```assign_default_credit_card``` when its actually assigning a default payment and the default card is already assigned in another method.


:) spam?


I agree. Updated in 12049c4.


@sindresorhus this needs to get published. Somehow in the PR we missed the fact `no-color` flags were taken out. The tests were faulty (`no-color` tests were already under an environment without color to begin with).


I think that `npm` still tries to install optional dependencies - and most likely it succeeds. So `optional` in this context merely means `don't fail if you cannot install it`. It that's the case, it would make sense to make it optional if we suspect `chockidar` might not available in all environments/operating systems.
I might be totally wrong on this.


Your version of http://www.xtrend-alliance.com/index.php/topic/1594-emc-vergisst-sortierreihenfolge-nach-neustart/page-2#entry21052

fixes the problem. Thank you.


I'm concerned that this usage of "satisfy" is inconsistent with the Concepts TS. In Concepts, satisfaction is a property of *constraints*, not of concepts: "In order for a constrained template to be instantiated (14.8), its associated constraints shall be *satisfied* (14.10.2). Determining if a constraint is satisfied
entails the substitution of template arguments into that constraint." The intuitive meaning of the phrase "`C<Args...>` is satisfied" would then be something like "The constraints defined by `C` are satisfied after substitution of `Args...` into those constraints", i.e., "satisfies" is equivalent to "meets the syntactic requirements of."

For `Same`, `Derived`, and `ConvertibleTo`, that's perfectly sensible. We don't specify the constraints introduced by the concept definitions, we simply say that those constraints are satisfied if and only if `trait<Args..>::value` is `true`.

For concepts with non-syntactic requirements, the usage of "satisfy" starts to make less sense. `Common`, e.g., defines a concept with a few syntactic constraints and then says "`Common<T, U>()` is satisfied if and only if [additional semantic requirements]" which could be interpreted as a requirement that implementations must verify [additional semantic requirements] and ensure that `Common<T, U>()` is unsatisfied when they are not met.

We could clear this up by explicitly defining satisfaction for concepts as "A set of parameters `Args...` satisfies a concept `C` if and only if the constraints defined by `C` are satisfied after substitution of `Args...` and all additional semantic requirements for `C` are met.", but I am concerned that different meanings for satisfaction when applied to concepts and constraints would cause more confusion. It would be clearer to use a distinct term like "models" for this notion:

> * The set of parameters `Args..` *satisfies* concept `C` if and only if the constraints defined by `C` are satisfied after substitution of `Args`.
> * The set of parameters `Args..` *models* concept `C` if and only if `Args..` satisfies `C` and meets all additional semantic requirements of `C` as defined herein.

This would make it easier to discuss the difference between meeting the syntactic requirements of a concept and meeting *all* the requirements of a concept, e.g. the counter-example for implicit expression variants:
> Since \tcode{T} fails to satisfy the implicit requirements of \tcode{C}, \tcode{C<T>()} is not satisfied in principle. Whether it is satisfied in practice is undefined.

Could be worded as "Since \tcode{T} fails to meet the implicit requirements of \tcode{C}, \tcode{T} does not model \tcode{C}. Whether \tcode{C<T>()} is satisfied is unspecified." Using different verbiage for "meets the syntactic requirements" - which the compiler can verify - and "meets all the requirements" - which the compiler cannot - is more comprehensible.


LOL


What exactly is weird? Described method is just for voltage **approximating** on selected capacitance interval. If at starting point, you'll choose, say, 3.72V(not 3.829V) for C=50%, F(50%) will be == 3.72.


what if roundTrip is nil? you should probably set it to `http.DefaultTransport`, otherwise things will panic down the road.


@Olion17 - Because in our code we already had those lines, the change of title was an oversight


What you propose would remove all assert I think. We don't want to do that by default.

I think that if node.inputs[0] is an assert, you must set in the merge_candidate:

node.inputs[0].owner.inputs[0].clients like on line 524-525.

Then around line 546, you must modify the input_match condition to check for assert.

Be sure to keep the assert in the graph.

Also, think that there is 2 cases:

1) the assert is on the new node introduced.
2) the assert is on the node already seen.


I saw you first used `apt` plugin and then decided to go with `provided`.
What was the reason to go back to `apt` again?


Probably yes, but having a descriptive error message helped us to know what was missing much more than the stacktrace with null pointer exceptions.


`set -e` does nothing for commands copy-n-pasted into an interactive terminal


`$dbc->groupSalePrice` should be `$item->groupSalePrice`, otherwise, this is great!


The "concept promotion" requirement I added to Range:
> Either \tcode{T} models \tcode{SizedRange}, \tcode{T} does not meet the syntactic requirements of \tcode{SizedRange}, or \tcode{is_sized_range<T>::value} is \tcode{false}

Could be simplified to "Either \tcode{T} models \tcode{SizedRange}, or \tcode{SizedRange<T>()} is not satisfied."


is this never used?


This change broke removal of tags (at least) for photos. If the last tag is removed we update with no terms - but the delete of the term will be ignored if it is inside if(is_array($terms)) { }
So we end up with a not deletable tag...
I just reverted this bit. Hope that's fine for you...


what is `gun`?


Why did you originally abandon grunt-aws? We're evaluating it. Thanks


@josketres Yes, it is not needed anymore because Travis now has Android support and installs the right SDK before build.


Does it work though? I did not see the notification in game even when getting targeted by the portal wave.


There's http://www.eventbrite.co.uk/o/flossuk-2723584564, but that lists events UK-wide, so `fetchICSFromEventBrite` would need to be extended to filter-out non-Birmingham events.  See #5.


Oh boy, we can use scripts in DECORATE now! FANCYPANTS!


I wanted something more focused on S3 only that was lighter. However I decided to switch back thinking maybe the issue I was having with the cache was because of the S3 only. I am now unsure of that though.


I don't think we want Android knowledge in LoggerFactory. A system property (say, "slf4j.detectMultipleBindings") turning off / on the ambiguity search would be more useful on other environments beyond Android. Your app could set this system property and get the same benefits. By default it would be true.


It works only when you set a range and your dungeon temple is not in range at game start. It will trigger when you claim a tile (ground or wall) on the border.


No need for `{{}}` I meant `[IsBaby]` By using {} you create a link to a class or value.


please remove printf


Is there a reason this change is necessary? This breaks pulling in Winstrap through NPM.


Sent an email to the PowerShell product team to see if this was a known issue with Nested Modules. Seems like this is a recurring issue across several DSC resources.


Ah right I should have read the context. It's even cooler this way then :-D


You should check your git hub settings to prevent updating always the complete file.

https://help.github.com/articles/dealing-with-line-endings/


could be helpful to return `name` in a `fmt.Errorf`


oooops :blush:


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/71314294


r+


`TUFChange`

@endophage


`Action` @endophage


this is necessary in order to build the project properly when it's included in another project via bower.  Winstrap isn't available via npm.  I believe there is a request for that, but when/if we add that, we'll address the path issue then.


Uhhh, we were hoping to test this across devices first too?


@reidmv @cyberious Darn - This actually was an issue - [MODULES-1671]https://tickets.puppetlabs.com/browse/MODULES-1671)


Type def this: `type Action int`

@endophage


Example of this script actually working: https://github.com/sils1297/coala/commit/832d660ec0c411a69e1ce1a0f27b5d49f5c18d29
https://github.com/sils1297/coala/releases/tag/0.2.0

was created with rultor release, tag=`0.2.0`.

Next step on the road: make rultor upload the stuff to pypi, on release as well as on merge.


Yeah that's what I was thinking too


Did you mean:
`sles_iter_impl& operator= (sles_iter_impl const&) = delete;`
?


Nice catch. That's what I get for not testing.


Yes!


yeah, not sure what makes more sense.  Right now we are limited because the patch exposes `<env>-<user>`


I guess Ben's cube was updated meanwhile.
Should we also update Ben's Cube in xmage?


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


@erickt That was fast  )


Note that this is an nginx keepalive connection pool cache. It only makes sense to remove this if the governor of php-fpm is **on-demand**. It makes sense also for dynamic, not only static because the dynamic governor imposes a minimum number of servers.


saw approval from graydon
at https://github.com/MonsieurNicolas/stellar-core/commit/30eefa79e41b6400f23eacb767162d041753ebff


merging MonsieurNicolas/stellar-core/windowsBuildfix = 30eefa79 into auto


MonsieurNicolas/stellar-core/windowsBuildfix = 30eefa79 merged ok, testing candidate = 5f94fa02


`return remote.SetMeta(...`


This is going to throw a `std::bad_cast` when the `dynamic_cast` fails.  Is that intended?


Could you please update the modules pom's versions to 1.4.1
(libreplan-business/pom.xml, libreplan-webapp/pom.xml, libreplan/ganttzk/pom.xml)
Currently their pom files state that their version as 1.4.0, which breaks the build.

Thanks!


This is a terrible commit that broke everything.


Tony, idk if we can change this on the fly in system.prop or not...


I'll look but maybe via adb?


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

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

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

```


@RobertGiff You should never commit/upload autogened/compiled code. This is what's causing a good deal of problems on @lab176's compilation.


Salut,
Merci pour le commentaire mais malheureusement je l'avais déjà testé ... et cela ne change rien.
Tu es sous quelle distribution ? 32bits ou 64bits ?


![2014 - 1](https://cloud.githubusercontent.com/assets/5148896/8734352/8053ffc4-2bd7-11e5-8f22-c9f44974e875.jpg)


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

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


Whoops, this can cause nasty surprise as c_str() is called on temporary :(


Uhh that doesn't make much sense, are you sure it's the only change?


funny that I wrote “mistage” :D


the "oops" was actually because I sent the PR with the incorrect documentation :stuck_out_tongue_winking_eye: :relaxed:


then keep alive upstream must be equal or less than the minimum number of children in dynamic configuration?


why ignore error?


There is a method called `find` that you could call on `entries` that does something similar, and using that you could write this method like this:

```
def iterative_search(name)
  entries.find {|entry| name == entry.name }
```

`find` will return the first entry that causes the block to evaluate to true and if there are no matches, it returns `nil`.


*Always* `nullptr` for `begin` *and* `end`?


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


This is https://tickets.puppetlabs.com/browse/MODULES-1671


LMAO
I know you can change LMK values, I don't know where this one is stored @NoSpamDan


Do we need a local copy here?


You know this won't work if you define any headers via `add_header` in a location? The reason why it's on a location (including the microcache_fcgi.conf` file) is because it circumvents that. See: http://nginx.org/en/docs/http/ngx_http_headers_module.html#add_header


Bug in commit: contains too much sass


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


Won't it stay valid until the end of strncat call? It actually copies the string to the buffer.
It should according to http://stackoverflow.com/questions/10006891/stdstringc-str-and-temporaries


I'm guessing this is where the axis call broke...but I have no idea what's going on anymore


Looks good (poptart)


Stressing:

> These directives are inherited from the previous level **if and only if** there are no add_header directives defined on the current level.


Problem. Too many lines!! (JK.)


TODO: clean up these files in the weavAnalyst app


That's ok. Some messages may be empty. I'll provide an example during skype call.


:sparkles:


Code for testing support shouldn't be commented out. We'll discuss it in skype. Move this to bottom of file and uncomment.


why ignore error?


Maybe we should rename the other `Option` class to `Optional` in a separate commit. The current name is not good i think.


lgtm


Using LA(1) would match wire "\blk00000001/sig0000017" in "wire \blk00000001/sig0000017a ;" where the 'a' was cutoff (and caused other parse errors). Actually, the look ahead isn't even needed? I think

    '\\' ~[ \r\t\n]*

should be sufficient? I.e. match starting from a \ and match everything up to white-space.


All this fields look like candidates for 'protected' visibility. They are not needed outside of DynStr class (quick look shows me this).


I have an issue to go through and fix these capitalizations. Just been working on more critical things so far.


Je suis sous Debian jessie 64 bits, python 3.4.
La lib fonctionne bien en supprimant ce patch (compilé et installé depuis Arch, la dernière version Repo ayant des problèmes d'import)


They have to be exported for the json tags to work.


i was about to say something, but was afraid of an endless waste of time :)

--
Kind regards / Saludos / Mit freundlichen Grüßen
Wolfram Kriesing, uxebu Co-Founder

kriesing@uxebu.com, mobile: +49 174 300 4595

Learn ES6, one kata a day http://es6katas.org

uxebu GmbH
Goethestraße 28
86161 Augsburg

Amtsgericht Augsburg, Handelsregister HRB 28613
Directors: W. Kriesing, T. v. Klipstein

On Thu, Jul 16, 2015 at 10:29 PM, David Aurelio <notifications@github.com>
wrote:

> funny that I wrote “mistage” :D
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uxebu/react-components-asserts/commit/072f729d51418137b1b2e260047c4d3fe34474dc#commitcomment-12212233>
> .
>


Ok, I see, it will stay valid in this case. But I still cringe a bit whenever I see c_str() used with C functions. We should try to refactor as much as possible to std::string.


@stevvooe can't have a method and field with the same name.


Yes, I think that's okay. Someone who have to intentionally write code to mismatch the implementation with its enclosing type to get that to fail.


LA(1) is a character in the lexer though, right, not a token?  yeah, we could probably get rid of that predicate completely.


There were two options to fix failing builds: add babel as a first level dependency for this one script or just make this much simpler. Guess which one I chose.


Collections.emptyList() ?


Please run code before submitting pull requests.


Heh well....errr....this could probably be improved


argh good point


It would be nice to make the fields final.


Good question - I am not sure


|| charPresence[nextChar] looks more like a syntax error recovery code then actual 'controlSequenceEnded' condition.


This commit probably caused a conflict in your local git repo. Not sure if you're familiar with conflicts but they occur when git can't figure out which branch of code to keep because both sides of the merge contain changes.

It represents these changes by wrapping the 2 sets of changes in `<<<< HEAD`, `====` and `>>>> some_commit_hash`.

Currently you've left these conflict markers in the file and haven't resolved the conflict yet.

You need to review the 2 sets of changes and decide which ones you want to keep (usually it's enough to pick one side of the `===` and delete the other).

Ensure the completed file no longer contains the conflict artifacts (`<<<`, etc) and then commit it again.


please put `5<<20` in a const


I am using Chrome. Since the "0 means don't sort" is non-standard, we cannot rely on its behavior. At least with using infinities, we guarantee that the non-positioned entities get pushed to the ends of the array (randomly). Which is better than having them divide the sort space, which is what I experienced in Chrome.


We've seen a few issues with cache where we can't get it cleared.  In one case the structure of a plugin block changed (added a folder to the path), and the javascript would not refresh even after restarting IIS, clicking Clear Cache in Rock, and clicking Restart Rock.  @azturner Is there anything I can look for next time that would be helpful to troubleshoot?


I'm more inclined to just delete this file entirely. The AppendChangelist is effectively deprecated in favour of the FilesChangelist


console.warn(LOG_TAG, '...', curStr, '....');


Reviewed and good with PR changes.


delete


The uglier one. :smirk_cat: Okay, I admit that run it through Babel just for string interpolation is quite an overkill.


just fyi, you can gain a line by doing `if err := ...; err != nil {`, same goes for the other lines below


I think we should init all `Option`s in the constructor, instead of always using `setValue`.


I think we can remove TODOs from this function now.


k


`err` is always nil


I want public interface functions near the top of the file so developers would find them easily :)


Yay!


I tested this code and from what I can see it does not work properly. Maybe I'm doing something wrong but I can't see what is going on here and why this should work and so on. Let me poke at it tomorrow. Sorry I took so long to get to this. I'm dealing with a lot on my plate but appreciate your input here. I hope to add a few things too. Sorry for the delay!


And bonus: you don't need to have whole friggin babel installed to run just one friggin utility script.


:+1: ^5


What is the value in doing that? I'll refer you to the net/http status codes. They do not define a custom type. If you're using the consts, you're safe regardless of whether a type is declared or not. If you're not using the consts, a type does not guarantee value safety, and therefore provides little to no benefit.


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

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

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

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

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

```


delete and replace with suggestion below


This is bullshit, because you need Babel to compile fittable itself.


Got this problem after updating the source.


delete and replace my change


It's used to initialize the HTTPStore when calling getRemoteStore


Rest in pepperoni. -_-

Soon the rest will be gone too. I'm redoing basically everything. Wooo


no newline


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

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

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

```


j'ai pushé une nouvelle version sur le repo qui corrige le problème d'import. J'ai également mise en place la directive :
#cython: c_string_type=unicode, c_string_encoding=ascii

Pourrais-tu confirmer que tout fonctionne dans le ticket : https://github.com/OpenZWave/python-openzwave/issues/81

Pourrais tu également y attaché la version complète de python :

$ python
Python 2.7.6 (default, Jun 22 2015, 17:58:13)
[GCC 4.8.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.

Merci d'avance


No no no. LOCALES enum is part of liblocalize interface. Use it here (as you did before).


@diogomonica


"Globally Unique Name" - It's documented further in the notary repository (or will be by release)


Very nice implementation! :+1:


You could also use `click_link 'Old Maid'` here, which would be more robust if you ever decided to rearrange the UI structure.


Je teste de suite...


I totally missed this too. Good catch. Just published 3.0.1.


Hey, I've got a refund for my ability in the onConstructionCancelled callback; after adding "builder.work = nil" its still refunded once if you right click during construction (and the building still completes) which is obviously not ideal! Any ideas how to stop this?


This is kind of a soft error. It's up to the caller to decide what they want to do and the caller already knows the name.


> How on earth did you find that?

Basically I'm rewriting this library in Erlang/OTP :) thus I have to look over how things are implemented :)


Damn you got me. @diogomonica just so you can see I make this mistake too.


ok agree, will keep it for the time being


Is it worth the fork?


        items_t::mapped_type value(Action::cache, std::make_shared<SLE>(sle));
        iter = items_.insert(iter, {sle->key(), std::move(value)});

is just as efficient and expresses it in two clear lines instead of six obscure ones...


:D


What about ldapurl, ldif and dsml? Should they be moved under pyldap module (in case of going this way)?


Config typo corrected


ok


Obviously correct!


Found it when testing the chalk/supports-color-cli#2 PR.


You don't need to install it globally. And to have it in your `$PATH`. And don't need to remember to use `babel-node` instead of `node`.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/71318868


`hardened_hash_set<uint256>` from ripple/basics/UnorderedContainers.h is an equivalent name for this.


Sorry, for nitpicking here, but these two changes should be in a separate commit.


In other words, KISS. kthxbie


nice catch.


It’s not needed to install it globally nor have it on your `$PATH`. npm automatically adds `node_modules/.bin` to PATH when running scripts.


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


We should comment the `finalizeToJSONRowsObject` method that order is important here to fetch the state and then clear.  Which should be obvious, but apparently we screwed it up.


1712-AvailabilityManagers (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


What is needed to move this back to `'>= 1.20.1'` or maybe `'>= 1.20.3'`


Why don't you then 1. build PCL 2. build VES (with PCL) and 3. build Apps with a separate External Project? That's what I will do. Greath work btw, very helpful. I brought it to VTK 6.2, PCL 1.8 and Boost 1.55.0. Now I will add CUDA for mobile also.


`boost::optional` does this dance for you.  This can be simplified to:
`return digest_ == other.digest_;`


2 lines up should be `err := applyTargetsChange(repo, c)` and no need to declare `var err error` at the start.


Oh wow that's awesome! Please don't hesitate to ask if you have any questions. If you use slack, I've been hanging out on `reactiflux.slack.com` (it's a team related to react though, maybe if enough people want it we could create an irc channel/gitter channel/slack team...)


So this might be an issue, cloudformation docs say

> Update requires: Some interruptions

And we probably don't want to cause interruptions if this stack gets pushed out. Maybe the better way to do this is by adding a parameter

```json
"Parameters": {
  "RDSEngineVersion": {
      "Description": "RDS Engine version",
      "Type": "String",
      "Default": "5.6.22"
  }
}
```

And then doing

```json
"EngineVersion": { "Ref": "RDSEngineVersion" }
```


I don't see how using the standard C++ library could in any way be considered obscure


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.768.feb6720fbcdc5c127335b5c3aad5445c36bdc319](http://build.palaso.org/viewLog.html?buildId=59813&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 379, ignored: 3 Build time: 00:06:40


:birthday: Yahoo thanks :smile:


Yes, it is ok to get rid of the `#if 0` branch.


hahaha, I actually LOLed ;)


Nice!


fast-forwarding master to auto = 5f94fa02


> when running scripts


With the more narrow type, readers have a better idea of what they are getting. They understand that only the behavior of the hash function is customize (versus having to look up `hardened_hash_set` in order to realize its just a `std::unordered_set`).


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.356](http://teamcity.monogame.net/viewLog.html?buildId=8795&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 522, ignored: 6 Build time: 00:25:05


Oops, this should call super.update(), and not repeat the code from the base


Yeah, that’s our case! The `script/build-static` script is intended to be run via `npm run build:static`.


Oh rad, I have this same issue and didn't know how to fix it properly. I've been working around it by just temporarily changing an upstream dependency. It looks like gulp-less-dependents is the right solution though, I'll play with that and see what works.


1 -body {  delete and replace with suggestion below

2 -    background-color: #00ffff;   delete and replace with suggestion below

3 -}  delete and replace with suggestion below

 7 -<p>  delete and replace with suggestion below

12 -  color: limegreen;   delete and replace with suggestion below

13 -}  delete and replace with suggestion below

17 +     delete and replace with suggestion below


1712-AvailabilityManagers (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Fine with me if we get rid of the aliases in ripple/basics/UnorderedContainers.h and follow this policy consistently (not that it should be done in this P/R).


@bors: retry


I spent a day and a half changing "models" to "satisfies" in all places in this document at @asutton behest. I have no motivation to change it back.


beware of end-of-line encodings, latest master removed .gitattributes


Yes, that's true except for STATUS changing from ONLINE
 to COMMLOST when USB disconnected.
I was told it is a soft/hard float compiling error. Oleg 2.6.22.19 with
wl-500gp is a soft float platform. Right?
Adam Kropelin states:
"It's pretty clear that the apcupsd package you are using for OpenWRT
(mipselsf) has been mis-compiled. Apcupsd needs (very basic) floating point
support. It appears that the package for OpenWRT was compiled either
without soft fp support or with hard fp and you are lacking hard fp in your
kernel (and/or hw). It's really an issue for whomever made that package."

Alexander would you please confirm and then initiate new compiling?
Attached please find apcaccess log of NaN values with USB modbus connected
and disconnected. NaN value readouts happen also with plain USB.
I wonder that nobody realized that NaN issue so far?

On Thu, Jul 16, 2015 at 1:40 PM, Alexander Ryzhov <notifications@github.com>
wrote:

> I have no idea. AFAIK, apcaccess gives NaN values too, when UPS
> connection was lost.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Entware/entware/commit/f2e17c85164756e656d43c56a776b0de77afdb0d#commitcomment-12201130>
> .
>

 apcaccess status
APC      : 001,036,0835
DATE     : 2015-07-16 22:30:00 +0200
HOSTNAME : Router4
VERSION  : 3.14.13 (02 February 2015) unknown
UPSNAME  : APCUPS
CABLE    : USB Cable
DRIVER   : MODBUS UPS Driver
UPSMODE  : Stand Alone
STARTTIME: 2015-07-16 22:28:47 +0200
MODEL    : Smart-UPS 1500
STATUS   : ONLINE
LINEV    : nan Volts
LOADPCT  : nan Percent
BCHARGE  : nan Percent
TIMELEFT : nan Minutes
MBATTCHG : 10 Percent
MINTIMEL : 5 Minutes
MAXTIME  : 0 Seconds
OUTPUTV  : nan Volts
DWAKE    : 0 Seconds
DSHUTD   : 0 Seconds
ITEMP    : nan C
BATTV    : nan Volts
LINEFREQ : nan Hz
NUMXFERS : 0
TONBATT  : 0 Seconds
CUMONBATT: 0 Seconds
XOFFBATT : N/A
SELFTEST : NO
STATFLAG : 0x05000008
MANDATE  : 2011-02-18
SERIALNO : AS1108212712
BATTDATE : 2011-02-18
NOMOUTV  : 230 Volts
NOMPOWER : 1000 Watts
FIRMWARE : UPS 09.3 / 00.4
END APC  : 2015-07-16 22:30:27 +0200

[C0nf4@Router4 root]$ DISCONNECT OF USB

[C0nf4@Router4 root]$ apcaccess status
APC      : 001,036,0837
DATE     : 2015-07-16 22:30:00 +0200
HOSTNAME : Router4
VERSION  : 3.14.13 (02 February 2015) unknown
UPSNAME  : APCUPS
CABLE    : USB Cable
DRIVER   : MODBUS UPS Driver
UPSMODE  : Stand Alone
STARTTIME: 2015-07-16 22:28:47 +0200
MODEL    : Smart-UPS 1500
STATUS   : COMMLOST
LINEV    : nan Volts
LOADPCT  : nan Percent
BCHARGE  : nan Percent
TIMELEFT : nan Minutes
MBATTCHG : 10 Percent
MINTIMEL : 5 Minutes
MAXTIME  : 0 Seconds
OUTPUTV  : nan Volts
DWAKE    : 0 Seconds
DSHUTD   : 0 Seconds
ITEMP    : nan C
BATTV    : nan Volts
LINEFREQ : nan Hz
NUMXFERS : 0
TONBATT  : 0 Seconds
CUMONBATT: 0 Seconds
XOFFBATT : N/A
SELFTEST : NO
STATFLAG : 0x05000108
MANDATE  : 2011-02-18
SERIALNO : AS1108212712
BATTDATE : 2011-02-18
NOMOUTV  : 230 Volts
NOMPOWER : 1000 Watts
FIRMWARE : UPS 09.3 / 00.4
END APC  : 2015-07-16 22:30:59 +0200


I don't understand this comment.


Wouldn't the variable values for this template be translated in the js, and not marked translatable here?


hmm - without the `clientMu` lock perhaps this races. Let's see if the race detector eventually complains.


1712-AvailabilityManagers (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


now nexus all the way xD


Je teste de suite...

Ma version complète de python3:

Python 3.4.2 (default, Oct 8 2014, 10:45:20)
[GCC 4.9.1] on linux
Type "help", "copyright", "credits" or "license" for more information.

J'ai toujours un problème d'import en lançant : examples/test_lib.py:

Traceback (most recent call last):
File "test_lib.py", line 28, in
import libopenzwave
File "libopenzwave/libopenzwave.pyx", line 55, in init libopenzwave.libopenzwave (src-lib/libopenzwave/libopenzwave.cpp:31689)
SystemError: Parent module '' not loaded, cannot perform relative import


I noticed that there's an overlap between the reporting guidelines and the enforcement manual in regards to what happens after a report is filed. Perhaps this is because the enforcement manual would normally be an internal document for the committee, whereas the guidelines are for people to use. It seems that we should either make sure each one is complete, or, where there's overlap and divergence, it should be purposeful and systematized. For example, I'd suggest these lists be phrased, punctuated, and capitalized identically:

```
The committee will then review the incident and determine, to the best of their ability:

*   what happened
*   whether this event constitutes a code of conduct violation
*   who, if anyone, was the bad actor
*   whether this is an ongoing situation, and there is a threat to anyone's physical safety
```
vs.
```
Members of the committee will immediately meet to review the incident and determine:

*   What happened.
*   Whether this event constitutes a code of conduct violation.
*   Who the bad actor was.
*   Whether this is an ongoing situation, or if there is a threat to anyone's physical safety.
```


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (725 of 725 tests passed)

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


Are we really using docker here?


test


switched to `util.Error`.


This commit should be squashed into commit https://github.com/jquery/jquery-mobile/commit/91ac2c2375124fa496bea5298343fbd0b5d7b5b2 before we merge 1.5-dev in master.


Gseht ja sehr hübsch us, aber lueg mal was ich in rust gmacht han ;)
https://github.com/rnestler/rpsrtsrs


What if !isOk === true here? preprocessDict?


> Since it was added to support outstanding packets to non 0, 0 chips on SC&MP 1.4 (which isn't going to see the light of day), this shouldn't be required.

Really?!


Add more general var names.


indentation?


Commit https://github.com/jquery/jquery-mobile/commit/93b06ed45f7712f2f4c3dbd2d6ed8bc022427179 should be squashed into this commit before we merge 1.5-dev into master.


added a comment indicating that it breaks the `select`, not the `for`.


Change to more general var names.


Ah kk good to know. And ah yes you're right that slipped me, thanks ^_^


Game text


I ordered git rm --cached file located in .idea folder.

Welcome to Git (version 1.9.5-preview20150319)

Run 'git help git' to display the help index.
Run 'git help <command>' to display help for specific commands.

Anton@SERRAMESTRE ~ (master)
$ cd SymfonyTraining
sh.exe": cd: SymfonyTraining: No such file or directory

Anton@SERRAMESTRE ~ (master)
$ cd PhpstormProject
sh.exe": cd: PhpstormProject: No such file or directory

Anton@SERRAMESTRE ~ (master)
$ cd SymfonyTraining
sh.exe": cd: SymfonyTraining: No such file or directory

Anton@SERRAMESTRE ~ (master)
$ cd PhpstormProjects

Anton@SERRAMESTRE ~/PhpstormProjects (master)
$ cd SymfonyTraining

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining (devel)
$ git status
On branch devel
Your branch is up-to-date with 'origin/devel'.

Changes to be committed:
  (use "git reset HEAD <file>..." to unstage)

        new file:   .gitignore
        new file:   composer.lock

Changes not staged for commit:
  (use "git add <file>..." to update what will be committed)
  (use "git checkout -- <file>..." to discard changes in working directory)

        modified:   .gitignore
        modified:   .idea/workspace.xml

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining (devel)
$ git rm --cached .name
fatal: pathspec '.name' did not match any files

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining (devel)
$ cd .idea

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining/.idea (devel)
$ git rm --cached .name
rm '.idea/.name'

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining/.idea (devel)
$ git rm --cached Symfony Training.iml
fatal: pathspec 'Symfony' did not match any files

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining/.idea (devel)
$ git rm --cached SymfonyTraining.iml
fatal: pathspec 'SymfonyTraining.iml' did not match any files

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining/.idea (devel)
$ git rm --cached "SymfonyTraining.iml"
fatal: pathspec 'SymfonyTraining.iml' did not match any files

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining/.idea (devel)
$ git rm --cached misc.xml
rm '.idea/misc.xml'

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining/.idea (devel)
$ git rm --cached modules.xml
rm '.idea/modules.xml'

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining/.idea (devel)
$ git rm --cached vcs.xml
rm '.idea/vcs.xml'

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining/.idea (devel)
$ git rm --cached workspace.xml
rm '.idea/workspace.xml'

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining/.idea (devel)
$ git rm --cached "Symfony Training.iml"
fatal: pathspec 'Symfony Training.iml' did not match any files

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining/.idea (devel)
$ git rm --cached "Symfony Training".iml
fatal: pathspec 'Symfony Training.iml' did not match any files

Anton@SERRAMESTRE ~/PhpstormProjects/SymfonyTraining/.idea (devel)
$


corrections to swp, retrying sync


Initializing game text.


plutôt un dossier helpers dans le dossier app/, étant donné que nous utiliserons principalement des classes ;)


sorry, that's not the race i meant: the first branch of the select is not guaranteed to be taken when the channel is closed (i believe i ran into this, but if you can find literature to the contrary, i'd like to see it).


seems a little fragile checking against a translated string, can we an id or a non-translated constant


Player health/damage system!


Looks good! I really like the idea to validate the file size before we upload it so we save all that time.


Hm, why not have a package-local var that we reuse, though?


BTW. README.debugging (which has the same content) was replaced with doc/debugging.html in gc6_0 (commit 9f27ae7). debugging.html has been updated several times later.


It looks like lines 57-66 in this file and the base class are the same.  Can we get those merged into a protected method in the base class?  Only the code to modify mFilesOnDiskBytes and the subsequent loop to calculate mWifiCount, mReportCount, mCellCount are different between base and subclass.


I planned to remove the other class `Option` and to throw exceptions when the files are not found.


Fixes #1


are we using the docker package here?


I agree with you in case `missingTestCase` and `missingConfigCson`, but they certainly added to other methods that will call `wrongConfiguration`

I fix methods `missingTestCase` and `missingCondigCson` on basic function, oki? :-)


Yeah, we could get rid of that class completely.


nopppeee


I' ll get right on it.


@gero3 do you really need 3 vectors here


Add our client library version to the user agent?


Your'e right `https://nodejs.org/api/process.html#process_process_exit_code` thanks


merci pour les infos.
J'ai remis les import comme dans la précédente release étant donné que cela 'a rien changé pour le segfault.
Un nouvel essai ?


*claps claps claps* :D


Should be using i18n methods here.  Add default translation to en.json


This fixes it: https://github.com/sequelize/sequelize/pull/4113/commits


Great fix. I wonder where you got this genius idea.


lol ^


It's now software listed above. Apart from that this location works well.


good point


Also broke my build. gulp-mocha-phantomjs needs to either fix phantomjs's version to 1.9.15 or stick mocha-phantomjs to 3.5.6 with the more liberal range?


Good functionality.  Does it gain much being by-name, with `Task.delay`?


This seems to cause some problems with firefox. In a 1-1 call with ff and chrome, chrome plays firefox's audio and video, but firefox plays nothing. One firefox + jigasi doesn't work either (ff doesn't play audio). I don't see any new errors in the console.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.769.bd2b467650eee4db79ce9c75ed1c3098e737b7c6](http://build.palaso.org/viewLog.html?buildId=59814&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 379, ignored: 3 Build time: 00:04:47


Yes, that's the direction I took.

I find it weird when a package installs files "outside" its namespace.


Would be possible to tell how to deploy these extension into firebug? or firebug lite?


You can use `c.newAKE()` to do this ;) I forgot to tell you


LGTM


Not _completely_ different class. A subclass. Casting to an completely different class would not be allowed with static_cast. :P

That CMessageProxy approach would make CMessage and CJoinMessage completely different, unrelated classes. Users would not be able to pass CJoinMessage to a method that takes CMessage...


I mean the hacks to make 133 support outstanding packets; 140 hopefully will appear one day...


@mattray This is the change that broke Ohai for us. There are two possible ways that ohai can be running:

1) Guestshell environment is a CentOS container. This has both /etc/redhat-release and /etc/os-release files.
2) Native environment is wind river linux. It has /etc/os-release but no /etc/redhat-release.

I'll open a pull request tomorrow with what I believe to be the correct fix.


Alors,
Les imports refonctionnent, toutefois, `Fatal Python error: take_gil: NULL tstate`
juste après que la lib c++ appèle une fonction python (le callback)

...
...
2015-07-16 23:16:02.231 Detail, Node001, Queuing (Query) Query Stage Complete (ProtocolInfo)
2015-07-16 23:16:02.231 Info, Node001, Initilizing Node. New Node: false (false)
2015-07-16 23:16:02.231 Detail,   Expected reply was received
2015-07-16 23:16:02.231 Detail,   Message transaction complete
2015-07-16 23:16:02.231 Detail,
2015-07-16 23:16:02.231 Detail, contrlr, Removing current message
2015-07-16 23:16:02.231 Detail, Node001, Notification: DriverReady
Fatal Python error: take_gil: NULL tstate
Abandon


@rpearce
Would you mind taking a peek and letting me know if it feels right to you


add newline


Tisy


@taldcroft Is there any value in searching all of the possible rolls if we've already found one that can be as warm as WARM_T_CCD?  I put code here to work away from nominal roll  in small steps.  What other kind of search through the rolls would make sense (equal chunks then smaller steps or some such)?


on remet ça ?


Is something like WARM_T_CCD = -5 reasonable for our search?  Or should this go warmer to further optimize "best" roll/temp candidate?


Bra jobba kompis! (Y)


And 140 will support outstanding packets, right?

(NB: I'm happy for us/you to revert this line until we need it, just worried that speed ups may not appear later :s)


I think the highlight is more obvious than a nick, especially if you change your name frequently.


Hi! Sorry. This is unexpected behavior. I'm to work to solve this behavior.


Du treng ikkje å ha krokodillemunn på denna linja, nok me bare "="


Change the switch key to q


You should try to reproduce it on the clean emulator. I can't receive crash.


It'll get added by the python client library.


Honestly, that's only true for you.  :-)
`
For example, when I rewrote it, I realized that this part: `make_shared<SLE>(*sle)` was actually the key thing that's going on - we're making a mutable copy.  I simply _did not see that_ in the welter of STL symbols.

If you think of it from the bigger pictures, neither `std::piecewise_construct` nor `std::forward_as_tuple` is there for any reason other than to fix b0rk3d syntax in C++.  No one would ever come up with those ideas on their own if C++ just let you easily insert items to a table

Even better, you could write it this way:

        if (auto sle = base.read(k))
        {
            auto copy = std::make_shared<SLE>(*sle);
            items_.insert(iter, {sle->key(), {Action::cache, copy}});
            return copy;
        }
        return nullptr;

It exposes the fact that you're making a mutable `shared_ptr`, inserting it, and then returning _that very pointer_ - something that's completely lost by the `return iter->second.second;` -  and it does it in fewer, simpler lines.


Probably? Or a service, since this is another of those cases where a bunch of app logic is in a controller (the root controller, no less).


Hey @bruno-c I though you might like this update


You lost some comment. The old comment explained that this must be done after processing the type annotation and the initializer, because those win over the implied type.


extra line


allez...

Cette fois-ci, Erreur de segmentation !

2015-07-16 23:27:10.816 Detail, Node001, Queuing (Query) Query Stage Complete (ProtocolInfo)
2015-07-16 23:27:10.816 Info, Node001, Initilizing Node. New Node: false (false)
2015-07-16 23:27:10.816 Detail,   Expected reply was received
2015-07-16 23:27:10.816 Detail,   Message transaction complete
2015-07-16 23:27:10.816 Detail,
2015-07-16 23:27:10.816 Detail, contrlr, Removing current message
2015-07-16 23:27:10.816 Detail, Node001, Notification: DriverReady
Erreur de segmentation


I actually think getRootDeclaration should be changed to accept a binding pattern, and just do the right thing internally.


Why did you remove this rule?


@Paxxi
I know you did a complete clean up in ApplicationMessenger (https://github.com/xbmc/xbmc/pull/7268), but at the moment I'm not sure where is the correct place for this method. Can you help or send a PR?


убрать статик, и подпихивать всего 1 другой мотоцикл. Вместо первого использовать this


Aside from the style complexity, I checked this code and it's fine.


I was just wondering!


lol


при реализации нестатического метода, сделал бы

```
kavasaki.race(honda);
honda.race(java);
java.race(kawasaki);
```


@jmtuley Yep, we had installed the hook but turns out we didn't have gosub installed as we were working from a new machine.


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


Your'e right, thanks. I'm vemeno!


L'erreur renvoyé par GDB

Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7ffff4733700 (LWP 5983)]
0x00007ffff65aa363 in __Pyx_PyObject_Call (func=0x7ffff6a4bac8, arg=0x7ffff6945d30, kw=0x0)
    at src-lib/libopenzwave/libopenzwave.cpp:35594
35594       if (unlikely(Py_EnterRecursiveCall((char*)" while calling a Python object")))


Looks good!  Handles any number of customers and does so in 4 lines fewer than the original, that's a good thing.


@cep21 Note this change. Removing it so that its more accurate when doing the send_flush calculations.


Initial


старайся в циклах и ifах ставить брейсы:

```
if(condition) {
    body;
}

for(conditions) {
    body;
}
```


I'm not sure how to parse that sentence.... Base: A subclass of...?


Just out of curiosity, should this method call `super nextPutAll:` like this?

```Smalltalk
nextPutAll: aString
	"#encodeUsing: is a primitive in GemStone, so it does the mapping very quickly..."
	| encodedString |
	encodedString := aString encodeUsing: table.
	encodedString ~~ nil
		ifTrue: [ stream nextPutAll: encodedString ]
		ifFalse: [ super nextPutAll: aString ]].
```


yeah just forget this ;-) .. i did several tests.. with different Values and different percentages .. but no luck.. u're right .. works only for 100-50% great .....

at the end i used this:

for i in [100,99,98,95,90,87,84,80,79,77,75,72,69,66,63,60,58,55,52,50]: print("\t{%d, %d},"%(cap2v(i,1.701967910238427, 3.7935)*1000,i))

and i get this:

	{4150, 100},
	{4140, 99},
	{4131, 98},
	{4104, 95},
	{4061, 90},
	{4037, 87},
	{4013, 84},
	{3984, 80},
	{3977, 79},
	{3963, 77},
	{3950, 75},
	{3931, 72},
	{3913, 69},
	{3896, 66},
	{3881, 63},
	{3866, 60},
	{3857, 58},
	{3844, 55},
	{3832, 52},
	{3824, 50},

so i think this is enough :D thank again for your amazing help !


peut me donner la sortie complète du make build :
J'ai ouvert une discussion sur cython-users, ça me permettrait de donner plus d'info. j'ai en particulier cette ligne qui n'apparait que sur la version 64 bits :

src-lib/libopenzwave/libopenzwave.cpp:441:36: note: in definition of macro ‘__Pyx_PyBool_FromLong’
 #define __Pyx_PyBool_FromLong(b) ((b) ? (Py_INCREF(Py_True), Py_True) : (Py_INCREF(Py_False), Py_False))

Merci pour ta collaboration et bonne nuit ;)


more comments


Credits to whom has the work! Always!!!


I will clean this, thanks!


@diogomonica This was a joke.


Why do you need to do this? The old code just relied on getTypeForVariableLikeDeclaration to do the right thing. What was wrong with that?


AFAICT this should be [type.typeBuilderImpl.simple], and it works correctly only by coincidence that when useFactoryMethod is true, type.typeBuilderImpl == type.typeBuilder
the same goes for factory builder, but there it less error-prone I guess


The call to \SimpleSAML_Session::registerLogoutHandler is missing the first parameter:
@param string $authority The authority for which register the handler.
This was changed recently:
https://github.com/simplesamlphp/simplesamlphp/commit/166345c3541e35a4a6b8492d8736c60530ea6a13


Well, to be clear — this has an actual performance and complexity impact, in that in this case it will bother to create and publish backwards compatibility pieces that will never be used.  I'm kind of hoping that as people start using new compilers more and more, the back compat stuff gets run less and less.


I meant that this update contains many older mistakes, that where documented around YAT, but never implemented, am I right?


Non-static method Tiqr_Response_Abstract::createResponse() should not be called statically


[INFO] -------------------------------------------------------------
[ERROR] COMPILATION ERROR :
[INFO] -------------------------------------------------------------
[ERROR] /home/opentree/repo/oti/src/main/java/org/opentree/oti/plugins/studies.java:[286,9] error: unreported exception NexsonParseException; must be caught or declared to be thrown
[INFO] 1 error
[INFO] -------------------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 3.942s
[INFO] Finished at: Thu Jul 16 21:41:24 UTC 2015
[INFO] Final Memory: 20M/277M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.5:compile (default-compile) on project oti-neo4j-plugins: Compilation failure
[ERROR] /home/opentree/repo/oti/src/main/java/org/opentree/oti/plugins/studies.java:[286,9] error: unreported exception NexsonParseException; must be caught or declared to be thrown


When USB is disconnected and apcupsd process restarted I get this much
shorter log with *no *NaN values:

STARTTIME: 2015-07-16 23:36:49 +0200
STATUS   : COMMLOST
MBATTCHG : 10 Percent
MINTIMEL : 5 Minutes
MAXTIME  : 0 Seconds
NUMXFERS : 0
TONBATT  : 0 Seconds
CUMONBATT: 0 Seconds
XOFFBATT : N/A
STATFLAG : 0x05000100
END APC  : 2015-07-16 23:38:06 +0200

On Thu, Jul 16, 2015 at 10:59 PM, Walter Bauer <bauer1walter@gmail.com>
wrote:

> Yes, that's true except for STATUS changing from ONLINE
>  to COMMLOST when USB disconnected.
> I was told it is a soft/hard float compiling error. Oleg 2.6.22.19 with
> wl-500gp is a soft float platform. Right?
> Adam Kropelin states:
> "It's pretty clear that the apcupsd package you are using for OpenWRT
> (mipselsf) has been mis-compiled. Apcupsd needs (very basic) floating point
> support. It appears that the package for OpenWRT was compiled either
> without soft fp support or with hard fp and you are lacking hard fp in your
> kernel (and/or hw). It's really an issue for whomever made that package."
>
> Alexander would you please confirm and then initiate new compiling?
> Attached please find apcaccess log of NaN values with USB modbus connected
> and disconnected. NaN value readouts happen also with plain USB.
> I wonder that nobody realized that NaN issue so far?
>
>
> On Thu, Jul 16, 2015 at 1:40 PM, Alexander Ryzhov <
> notifications@github.com> wrote:
>
>> I have no idea. AFAIK, apcaccess gives NaN values too, when UPS
>> connection was lost.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/Entware/entware/commit/f2e17c85164756e656d43c56a776b0de77afdb0d#commitcomment-12201130>
>> .
>>
>
>


uhm not sure if we can test this outside master - fwiw locally this command worked at least.


Gosh, I could really use that "hierarchy of View classes" document about now.

Why do rules about protocol behavior need `DigestAwareReadView` and none other?


Two small issues that make this module hard to use with recent simplesamlphp installs


d


unintended import


This is such a wrong way to fill your activity feed. FAGGGGGOOOOOOOOOOOTTTTTTTTTT!


What is this humor you speak of?


better to rename file as _UnderNaming also, least surprise


commit is for issue scenarioo/scenarioo#149


J'ai cette ligne également,
Ci-dessous, la sortie de la compilation de la lib (uniquement la lib)
Merci également, si besoin d'autres tests, n'hésite pas !

python3 setup-lib.py build :

running build
running build_ext
cythoning src-lib/libopenzwave/libopenzwave.pyx to src-lib/libopenzwave/libopenzwave.cpp
building 'libopenzwave' extension
creating build
creating build/temp.linux-x86_64-3.4
creating build/temp.linux-x86_64-3.4/src-lib
creating build/temp.linux-x86_64-3.4/src-lib/libopenzwave
x86_64-linux-gnu-gcc -pthread -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -g -fstack-protector-strong -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 -fPIC -Iopenzwave/cpp/src/ -Iopenzwave/cpp/src/value_classes/ -Iopenzwave/cpp/src/platform/ -Iopenzwave/cpp/build/linux/ -I/usr/include/python3.4m -c src-lib/libopenzwave/libopenzwave.cpp -o build/temp.linux-x86_64-3.4/src-lib/libopenzwave/libopenzwave.o
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++
In file included from openzwave/cpp/src/aes/aescpp.h:26:0,
                 from openzwave/cpp/src/Driver.h:42,
                 from src-lib/libopenzwave/libopenzwave.cpp:248:
openzwave/cpp/src/aes/aes.h:75:0: warning: ignoring #pragma warning  [-Wunknown-pragmas]
 #pragma warning( disable : 4324 )
 ^
openzwave/cpp/src/aes/aes.h:93:0: warning: ignoring #pragma warning  [-Wunknown-pragmas]
 #pragma warning( default : 4324 )
 ^
src-lib/libopenzwave/libopenzwave.cpp: In function ‘PyObject* __pyx_pf_12libopenzwave_9PyManager_250beginControllerCommand(__pyx_obj_12libopenzwave_PyManager*, PyObject*, PyObject*, PyObject*, PyObject*, PyObject*, PyObject*)’:
src-lib/libopenzwave/libopenzwave.cpp:24292:204: warning: ‘bool OpenZWave::Manager::BeginControllerCommand(uint32, OpenZWave::Driver::ControllerCommand, OpenZWave::Driver::pfnControllerCallback_t, void*, bool, uint8, uint8)’ is deprecated (declared at openzwave/cpp/src/Manager.h:1715) [-Wdeprecated-declarations]
   __pyx_t_6 = __Pyx_PyBool_FromLong(__pyx_v_self->manager->BeginControllerCommand(__pyx_t_1, __pyx_t_2, __pyx_f_12libopenzwave_ctrl_callback, ((void *)__pyx_v_pythonfunc), __pyx_t_3, __pyx_t_4, __pyx_t_5)); if (unlikely(!__pyx_t_6)) {__pyx_filename = __pyx_f[0]; __pyx_lineno = 3833; __pyx_clineno = __LINE__; goto __pyx_L1_error;}
                                                                                ^
src-lib/libopenzwave/libopenzwave.cpp:328:36: note: in definition of macro ‘__Pyx_PyBool_FromLong’
 #define __Pyx_PyBool_FromLong(b) ((b) ? (Py_INCREF(Py_True), Py_True) : (Py_INCREF(Py_False), Py_False))
                                    ^
creating build/lib.linux-x86_64-3.4
x86_64-linux-gnu-g++ -pthread -shared -Wl,-O1 -Wl,-Bsymbolic-functions -Wl,-z,relro -Wl,-z,relro -g -fstack-protector-strong -Wformat -Werror=format-security -D_FORTIFY_SOURCE=2 build/temp.linux-x86_64-3.4/src-lib/libopenzwave/libopenzwave.o openzwave/libopenzwave.a -ludev -lstdc++ -o build/lib.linux-x86_64-3.4/libopenzwave.cpython-34m.so


it is questionable if it made code any simpler for serialization tracking, but let's leave it as is


Two choices:

1. `operator==()` means identical - "have the same shared pointer".
2. `operator==()` means equal - "contents are the same".

Why did you pick choice 1 over choice 2?


Should I try the current HEAD? @jwpeterson I see that you made some commits too?


By the way, we have a `HERMIT.Debug` module which has a `debug` variable for this kind of thing. You could replace the above lines with `when debug $ putStrLn "[Start Glyphs]"` and change the value of `debug` when it's convenient.


Oh, I think somewhere you comment it out, do not declare the variable on the top, which is invalid to this rule, so I removed it. Should I add it back?


This commit also simplifies FFNNLayer construction, making `bias` a keyword argument


Sorry, I see what I got wrong.


Are we going to rename these files from .md to .html in this PR?


Type stability


Thanks!


Yes, 9b643e8 is working for me now.  Haven't tried 'make dist' yet.


No Alt: My edition
Alt: Original, edited but no top long shadow
Alt 2: Orginal, edited


BTW, there is a nice shortcut for that: `clink.is_match()` (see https://github.com/mridgers/clink/blob/master/docs/clink.md#the-clink-lua-api). No need to update this PR once more, this is just FYI


crash_fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (352 of 352 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)


LGTM, will merge tomorrow.


Good Job Horla!


crash_fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (352 of 352 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)


@NilayKapadia Please try removing the `i.Tools.Subha ... `  tag if possible..


Here it is also (partially) solving #5


Since modules depends on the application which itself depends on QTCLI. I anticipate that doing a clean build will fail.

Since the updates to QTCLI seems to be using base classes (vtkMRMLDisplayableNode, ... )  it seems introducing that compile dependency is not needed.


I don't like this API


See above.


Here it is also (again partially) solving #5


Here it is also (partially again) solving #5


Since node type is a string and not a reference to the class ... I think we can avoid the compile time dependency.


@adam-kinsley


Humm ... this one may be problematic if we remove the compile time dependency


Not sure why this big ol' image wasnt optimzed. Now it is. Let's upload that, as it's the biggest thing the site is pulling down.


No. Jekyl is going to render these as .html for us I think. #11344 for what this PR is trying to accomplish.


clearer IMHO: `return impl_ && impl_->enabled(feature);` (why is Mr. `false` entering the picture at all?)


1. Reviews assign to first team will be randomly selected from all available participants;
2. Reviews assign to other teams will be randomly selected from a list of participants whose review number is less than mean value of total review number;
3. I also consider the edge cases, such as empty list and prohibit one student to review his/her own artifact.


Alright I got over my desperation to know the position of the radix point before thinking I was able to calculate the value, ha. Thanks for the feedback :+1:


I said that wrong, I meant I optimized it by checking if it was already merged.


please separate semantic CLs from whitespace refactors.  also, whitespace refactors should have a good reason.


crash_fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (352 of 352 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)


`@tparam <name> <description>`


puts "Text please: "
text = gets.chomp

words = text.split(" ")
frequencies = Hash.new(0)
words.each { |i| puts frequencies[i] +=1 }
frequencies = frequencies.sort_by do |word, count|
    count
end
frequencies.reverse!
frequencies.each do |word, count|
    puts word + " " + count.to_s
end


just a little ruby practice


Arg- thanks for fixing.


Same here.


Thanks, fixed in a90316a16f639aaa3531208fc0451a1b79a35a7d.

> On 17 Jul 2015, at 12:09 am, Yves Lafon <notifications@github.com> wrote:
>
> The reference to #idl-array points to nothing, so this specific change looks suspicious.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/heycam/webidl/commit/416ebfb2c538363c1f2a763d1b2921f386b09e2e#commitcomment-12204223>.
>


Addressed in https://github.com/10gen-labs/mongo-hhvm-driver-prototype/pull/21


crash_fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (352 of 352 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)


Nit: newline missing


Hey, want to issue a PR to merge this into master?


Typo in `Destrcuture`. Looking awesome BTW!


crash_fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (352 of 352 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, this was where I was having problems managing the dependencies. I'll leave a clean build going over night to double check.


The plan is to go as far as I'd need for my project so I'd only do content upload, content download, and checking whether a particular resource has new updates.


Comparing digests is a fast way of determining if the rules changed, without having to retrieve the entire ledger item.


@oohira Shouldnt this be `String.valueOf(body).length()` ? You're essentially putting all of the content in the content-length header


It might, I'll take a look.


Yes, FileIDs are ObjectIDs. Although we use the language FileID in the API documentation, so I'm going to be consistent with the API.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1242 of 1242 tests passed)

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


crash_fixes (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (352 of 352 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)


According ORDERS_FOR_MISSION https://github.com/freeorion/freeorion/blob/master/default/AI/AIFleetMission.py#L18  I don't expect this mission types  have ORDER_MOVE except FLEET_MISSION_EXPLORATION.

This code really conflict with my order refactoring.


I'm really sad to see this.  Can you add a `TODO - catch ReflectiveOperationException once sbt is migrated to JDK7`.  That way we can grep TODO JDK7 and find things we need to fix.


Sure! give me a min to do it.


dsadadasdasda


crash_fixes (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)


This type of field has no locking and limited concurrency support, so some notifications will get lost in courses with many students.


Here is an idea: to remove the compile dependency to the markups module and simplify the code. Would it make sense to store the "CoordinateSystem" as a node attribute instead of class ivar.

Since the the SetAttribute/GetAttribute interface is available within vtkMRMLNode, it would also simplify the code. You could then do:
```
out->SetAttribute("CoordinateSystem", nd->GetAttribute("CoordinateSystem"))
```


I think this can be simplified.  Rather than using the internal `_find_home`, if you *unset* `XDG_CACHE_HOME` and then use `astropy.config.paths.get_cache_dir()` you can temporarily set `XDG_CACHE_HOME` to its default location:

```python
from astropy.config.paths import get_cache_dir
xch = os.environ.get('XDG_CACHE_HOME')
if xch is not None:
    del os.environ['XDG_CACHE_HOME']
os.environ['XDG_CACHE_HOME'] = get_cache_dir()
try:
    simobj = ProcessedSnapshot()
finally:
    os.environ['XDG_CACHE_HOME] = xch
```

See if that works. (Also might want to fix the tabs :)


Ah sorry I mislead you on the correct invocation of pkg-config here...

I suggest using --exact-version instead of --exists, because: it isn't supported by PkgConfig.pm (yet), which is used when the system doesn't provide its own pkg-config.  Also it doesn't require any quotation marks which aren't portable (though if you are only going to support unix that is probably not a problem).

I opened an issue with PkgConfig here, because I think that would be useful capability:

https://github.com/mnunberg/perl-PkgConfig/issues/22


@theengineear  hey I've changed load to dump, to run my tests.  It seemed like a typo


What about `CoordinateSystem` as attribute name ?


Do we ever have a time when you do **not** use a path for a route? Maybe we could have the syntax be something like:

```js
new Route(path, { component: ... });
```


If no node to flag is found .. should a warning or error be logged ?


1712-AvailabilityManagers (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

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


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (352 of 352 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)


It's a flying circus. Maybe `supported{Image|Audio}Types`?


Assuming ... `CoordinateSystem` is store as an attribute, you could then do:

```
mrmlStorageNode->SetAttribute("CoordinateSystem", coordinateSystemStr.c_str())
```


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (352 of 352 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)


ok :)


Since module can depend on other modules. This dependency is ok.


Ok, I've been playing around with things and gulp-watch causes my build to hang on the less task. I'm not really sure what is up with that. If you can come up with a good task for me that works well, I'd love to integrate it.


added. at the moment I can just list all of the exceptions it can throw.

But as we fall back to the current behavior which will succeed regardless environment I decided on Throwable.


I need at least one specific example.


as `the` original image.


Where is batch_max_timestamp used?


ok


What is this?


1712-AvailabilityManagers (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


this text is inherited. I'll see what I can do


Sounds good.


Just a heads up, the latest version of the TS redefines "associated
constraints" in terms of expressions, so writing "E is satisfied"... will
make sense. Constraints are now totally internal processing.

I think it's reasonable to have, in the introduction, a statement along the
lines of:

"Concepts defined in the library impose semantic constraints in addition to
those required by the language. Within the library, a concept is satisfied
if and only if both the language constraints and semantic constraints are
satisfied.

[Example -- ]"

Or something like that. We overload terms all the time in the spec. As long
as its clear what's meant than we should be okay.

Andrew Sutton

On Thu, Jul 16, 2015 at 4:06 PM, Casey Carter <notifications@github.com>
wrote:

> I'm concerned that this usage of "satisfy" is inconsistent with the
> Concepts TS. In Concepts, satisfaction is a property of *constraints*,
> not of concepts: "In order for a constrained template to be instantiated
> (14.8), its associated constraints shall be *satisfied* (14.10.2).
> Determining if a constraint is satisfied
> entails the substitution of template arguments into that constraint." The
> intuitive meaning of the phrase "C<Args...> is satisfied" would then be
> something like "The constraints defined by C are satisfied after
> substitution of Args... into those constraints", i.e., "satisfies" is
> equivalent to "meets the syntactic requirements of."
>
> For Same, Derived, and ConvertibleTo, that's perfectly sensible. We don't
> specify the constraints introduced by the concept definitions, we simply
> say that those constraints are satisfied if and only if
> trait<Args..>::value is true.
>
> For concepts with non-syntactic requirements, the usage of "satisfy"
> starts to make less sense. Common, e.g., defines a concept with a few
> syntactic constraints and then says "Common<T, U>() is satisfied if and
> only if [additional semantic requirements]" which could be interpreted as a
> requirement that implementations must verify [additional semantic
> requirements] and ensure that Common<T, U>() is unsatisfied when they are
> not met.
>
> We could clear this up by explicitly defining satisfaction for concepts as
> "A set of parameters Args... satisfies a concept C if and only if the
> constraints defined by C are satisfied after substitution of Args... and
> all additional semantic requirements for C are met.", but I am concerned
> that different meanings for satisfaction when applied to concepts and
> constraints would cause more confusion. It would be clearer to use a
> distinct term like "models" for this notion:
>
>
>    - The set of parameters Args.. *satisfies* concept C if and only if
>    the constraints defined by C are satisfied after substitution of Args.
>    - The set of parameters Args.. *models* concept C if and only if Args..
>    satisfies C and meets all additional semantic requirements of C as
>    defined herein.
>
>  This would make it easier to discuss the difference between meeting the
> syntactic requirements of a concept and meeting *all* the requirements of
> a concept, e.g. the counter-example for implicit expression variants:
>
> Since \tcode{T} fails to satisfy the implicit requirements of \tcode{C},
> \tcode{C()} is not satisfied in principle. Whether it is satisfied in
> practice is undefined.
>
> Could be worded as "Since \tcode{T} fails to meet the implicit
> requirements of \tcode{C}, \tcode{T} does not model \tcode{C}. Whether
> \tcode{C()} is satisfied is unspecified." Using different verbiage for
> "meets the syntactic requirements" - which the compiler can verify - and
> "meets all the requirements" - which the compiler cannot - is more
> comprehensible.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ericniebler/stl2/commit/697351dfd9bd0c2b5f4b1b6d4956c022541afa29#commitcomment-12211720>
> .
>


What is this command?


Thanks for the feedback. Committed:

https://github.com/betonme/e2openplugin-EnhancedMovieCenter/commit/3e048e145ada5e5bad5d76b6f0c20ad6c4f59e20


Ugh.  Why is your git not detecting these as moves :(


Sure, I'll do that when I get back from dinner tonight.


este cambio a que se debe ?

la frase en ingles es incorrecta ?


As long as you didn't change the logic in CompletionRules, then I :+1: this.


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (352 of 352 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)


```Result<bool>```?


That's not the PLN book, although it does uses a fair bit of PLN.


Is the idea here that we never send enter through for this use case regardless of the user setting?  If so, can you doc this?


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.4](http://pixelated.design/build/viewLog.html?buildId=73&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 123; exit code 1 (new) Build time: 00:00:17
### Failed tests
```
Class: Class can create an event: <no details avaliable>

Class: Class creates a method that returns an Event instance: <no details avaliable>

```


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (352 of 352 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)


This is Darts way of making an abstract constructor right?


Looks like an "Option" type.  Do we not have that already in Roslyn anywhere?


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

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

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

```


This is all effectively ```Nullable<t>``` :)

i'd call it "Value" and "HasValue" to match that type.


Which timer deletion handling fixes?


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (352 of 352 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)


why .Default here?


Can we make it easier to get to the fun page?


Any way this could be made final?


El texto anterior era correcto, pero demasiado largo.
La ventana de configuración no lo lograba mostrar.

Lo refraseé para que se muestre completamente.


To answer my own question, it looks like that issue has been fixed in Meteor. This works. I made a PR to the original repo: https://github.com/AdmitHub/meteor-buildpack-horse/pull/35

Thanks!


let's add the snapchat day photos http://techcrunch.com/2015/07/11/a-new-users-guide-to-understanding-snapchat/


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (352 of 352 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)


As far as I could tell there is no server-side validation for code-injection settings so we should only expect a single all-out failure here if something has gone wrong. Is that correct? I haven't had a chance to deep-dive into the server side yet and I couldn't spot any specific tests for the code-injection area unless I missed them.


1712-AvailabilityManagers (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


I'm starting to see why you have ```Result<Bool>```... but it's so close to ```Nullable<Bool>``` that I would just use the latter instead.  I would then doc things as "if null is returned, then the default completion list behavior will be used."  But that's just me.


Cool. I didn't know you could have an expression that could accept two different types of types for an input param.


multithreaded_performance (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)


@sanderborgman Review ? :sun_with_face:


easier how? @lsemel


When diamond calls process() -> should_flush() which uses self.batch_max_timestamp. That part of the code which u wrote has not been changed here.


Do you plan a unit test to test Rsa::generateKey?


Don't listen to me.  Can we actually hard code this in as the default for CompletionRules?


Az eduke32 forrásban valamiért nem fut le a gdk-pixbuf-csource fordításkor, így "undefined reference to startbanner_pixdata" hibával leáll.
Így inkább legeneráltam külön a .c fájlokat, és lett belőlük egy patch.


Is this the exact same list as C#?  If so, it's also the same list for TS I believe.  Let's just move it into the root and consider it the default set.  If people don't want it, they just override (i.e. how you're doing things already).


I'm not sure. This one seems like it is within the threshold for git to call it a rename.


:smile: Yeah, I want to clean some of that up, but I resisted this time.


en ese caso se deber reportar el bug, para que redimensionen la ventana, ya que es un bug que puede afectar a mas traducciones, pero cambiar una frase como solucion temporal, ya que bug pasara al olvido y no sera corregido.


@jef-n You're probably already aware of this, but this change has caused "Access Violation" errors in some of the unit tests for the nightlies. Eg, see http://dash.orfeo-toolbox.org/testDetails.php?test=31608774&build=191179


jpeg.js is a generated file, so patching it without the source files is kinda pointless. :-/


This file needs the license header to run on Jenkins.


Case 3057 is not a few years old?


It seems like that's the way it was. I just copied the body of this method from AbstractMemberInsertingCompletionProvider above to avoid changing behavior. I'm assuming the existing behavior was correct.


here '.' is the PATH argument to build.


Should I capitalize this? This is the PATH argument to build.


In the future I think it would be generally valuable to move InsertionText up into CompletionItem.  If a user doesn't provide it, then it just gets set to "DisplayText". But I think it would be generally valuable for the core architecture to support the concept of having an item that inserts something different.


That's essentially what it. I don't believe we have one in Roslyn any longer so I created this one purposed to this particular API.


I am okay with this change going in, but this part still feels like a band-aid and something that can be fixed in a more rigorous way.  What we want is to order classes the same way we're ordering finalizable objects, but without adding finalizers to them.  In fact if you squint just right the loops here look a lot like what the pypy algorithm is doing.  marking from them, and then only actually finalizing a subset of them.

We can address this either by duplicating the ordering algorithm and specializing it for types, or somehow generalize the pypy algorithm to support objects that don't have finalizers (or by doing a hack like stuffing the types onto the list of objects the pypy algorithm walks, then removing them after.)

definitely an area for further work.


Why was this removed?  The official documentation still references this method when selecting a retrieving a single model.  This breaks findOrNew unless find is implemented by an extended class.

 public static function findOrNew($id, $columns = ['*'])
    {
        if (!is_null($model = static::find($id, $columns))) {
            return $model;
        }

        return new static;
    }


What is a "CompletionListTag"?


This line seems repetitive in the context of some of what follows.

The fact that `dockerfile == steps to construct image` seems to be a key concept that is possibly worth repeating, as long as we say it the same way every time.


Oops! We do have one: [```Optional<T>```](http://source.roslyn.io/#Microsoft.CodeAnalysis/Optional.cs). I'll use that instead. Good catch!


Looks good to me.


You can still use the find method.


Another place with InsertionText... I think we should move this up int he future.


I'll just use ```Optional<T>``` :smile:


It;s implemented through `__call`.


Worth discussing! Moving to a https://github.com/diydrones/dronekit-python/issues/209 issue.


Agreed, and thanks. :+1:


Thanks @nblumhardt :)


got it.  thank you.


please follow Google's Java style guide (http://google-styleguide.googlecode.com/svn/trunk/javaguide.html) which mandates indents of 2 spaces (not 4).


You could probably get away with something nastier than Black Hawks and Ghosts for guardspirit. There's only something between a 10-20% chance per priest (although priest level might factor into that) for a guardian spirit to show up for a given priest, and the only ones currently in game are Fravashi - 1A1S, Awe 3, sacred, ethereal, and 28hp - and they don't seem unbalanced. Admittedly, they also mostly hang back and cast A1/S1 spells, which can be pretty underwhelming.

Hmm, the more I think about this, though, the harder it is to come up with good examples that won't be OP, especially in moderately massed (I saw very unpleasant things happen in my last NG game when it was possible for T2 mages to have batsummon air elementals). It's hard to find low-level divine-ish casters who can plausibly show up from nowhere, which pretty much leaves fighting guardians. Hmm. Ancestral Spirits actually might make a pretty decent fit both flavor-wise and balance-wise. Some of the weaker angels *almost* seem like they'd work, but would probably be a bit too good against undead-heavy nations (who'd already be encouraging you to spam priests). Fairies *might* work. Hmm, heh, lesser othernesses or other low-end void beings might also be viable.


I thought I'd try rewriting that whole paragraph, but I'm not sure it reads any better than the 'simple' version.


Default means that the default behavior should be used.


K,

Things are coming along nicely

Sent from my Windows Phone
________________________________
From: nugycal<mailto:notifications@github.com>
Sent: 7/17/2015 7:25 AM
To: nugyflex/sifigame<mailto:sifigame@noreply.github.com>
Cc: Jack Ransome<mailto:jransome@creativesoftware.com.au>
Subject: Re: [sifigame] try gui for gun switching (92ee1f7)

Change the switch key to q

—
Reply to this email directly or view it on GitHub<https://github.com/nugyflex/sifigame/commit/92ee1f7dcf31867164b26a782944f1683d416718#commitcomment-12213527>.


yay


s/in another format or the same with a different value//


No, I just wanted to understand what's going on.


@mitchellh So I guess we need to update the vault demo server?


Yeah, that feels somehow strange to me. It's inverted in my mind. I'll try ```Optional<T>``` since we have the type.


Was this the only modification you had to make?


Estoy de acuerdo.

Aunque siendo realistas, un bug de esa naturaleza tardará mucho tiempo en resolverse (hay mucho más interés en resolver bugs de emulación).


out of curiosity, if you add `if (class_objects.count(cls->cls) == 0) printf ("re-adding\n");` do you see anything when running the test added as part of this PR?  it seems like you should, given the number of iterations in the test.


what is the intended purpose of this variable?


Edited the wording to remove any reference to 'source repository' which has no occurrences for this particular concept anywhere else in the source or docs.


You are correct on both counts. I was careless here. Thank you for catching it.