version 3.11.92? Thats a prerelease isn't it? Why don't you take 3.16 or so? Haven't they made releases since then?


I'd rather not keep it restrictive to that - it can be for any other commonly used variables


Quite obviously, this file (or at least this block) is tab-indented. Your line is not.


so this will never be executed, right?


TbButton does not have a constant called 'CTX_DEFAULT', so it triggers this error:
Undefined class constant 'TbButton::CTX_DEFAULT'


Yes, I have OpenSSL-Win64 installed. Need it more or less often to create X.509 certs / CSRs.


Help text on these fields seems very clear.


What about webhooks from GitHub etc? That way DevPost would be notified by the git service rather than polling.


I knew shogun, the machine-learning toolbox, works with multiple languages. So, I referred their [.travis.yml](https://github.com/shogun-toolbox/shogun/blob/develop/.travis.yml)


new Date() is being set when the code is first parsed by the browser not when controller is being instantiated. Having an "initializer" method setting displayedTime `.on('init') would make less confusion


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


These tests seem good.  You might also want to add a couple of tests for the end of the time period - e.g., period in the past or period with no end date.


Oh! Thanks anyways!


3.10 is the current stable version according to https://wiki.gnome.org/action/show/Projects/PyGObject?action=show&redirect=PyGObject

It has some issue with python3.4 when compiling - I dont remember the exact bug but I was not able to solve it. Because of that I was forced to use the development release.


Indeed I was!


@isuruf I referred to [.travis.yml](https://github.com/shogun-toolbox/shogun/blob/develop/.travis.yml) of shogun repository. They seem to have chosen multiple languages this way.


test si la variable a bien été implémenté (fonction init_config_tab. ligne 2071)


This isn't specific to your code, but how can a downtime period have no end?  And what happens with the banner in that case?  Is there some behavior we need to document?


Maybe make the `distance` metrics explicit:
`dist[k, m] = distance(covtest[k, :, :], self.covmeans[m], metrics=self.metrics)`


`filteredWeekStats` is consumed only by `weekContent`, you can consider merging those together. Also you could consider creating data transfer object to keep data transformation logic encapsulated.


But, 1) this is just the default table; the code should be querying for available versions, and 2) do we have these versions available for downbursting, for example?...


Might be good to have a brief overview here (or maybe just in the module docs) of what the model is for and that it depends on downtime.


Does the constant table ever change?  If not, and you have access to it during jit-compilation, you might get faster code if you directly emit a reference to the constant, rather than emitting an array lookup (if I'm reading this right).

Then again, premature optimization is the root of all evil, I guess...

Hope this is constructive


Why aren't we checking this anymore for migration templates? It seems wrong to overwrite newer data


Shouldn't that have been there a long time ago?


This seems to be mistakenly included in this commit


Thanks for the review guys. I have incorporated these changes.


Hi,
I am unable to add watermark on image through Qimage component i think problem with $watermarkImage veriable value could you explain more about this?

Thanks


if `polylines` array can have only one object why to keep it as an array? you could have `roads` property on your model (also name roads have much more meaning than polylines)


Can't this be solved in O(n) instead of O(n^2)? Maybe somehow using `SplObjectStorage()`? Or should we assume that `count($listeners)` is usually < 2 and therefore the complexity does not matter?


presentation logic?


Fixed in https://github.com/aspnet/Localization/pull/33 - For the record it's extraordinarily unclear to me which is the official build :P.


why do you first try that and then spawn up the test server? I'd spawn it in the main method below and here a loop trying to connect, waiting .1 seconds if it fails and retrying again and again, if it fails after 10 times or so, fail test.


Ok, so maybe it works. Though then you need to setup the C++ environment properly, probably just defaults will work.


Just try it out on Travis and see.


test si le fichier config_SMTP.php existe. Si c'est le cas, il implémente les variables (si elles sont définies) dans $tab.


Why?

On Tue, Jun 16, 2015 at 1:20 PM Ian Macalinao <notifications@github.com>
wrote:

> why
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TAMSCompSci/tamscompsci.github.io/commit/9c717b5e03ec644530cdcd0122afdf30f70f2664#commitcomment-11709059>
> .
>


Ah: so this is the strategy to change the default account's name, but leave the rest alone? ... i.e., cloud-init overrides the given key without removing the other keys?


sure that is needed? Ember should generate this one on the fly


I'd rather put this method outside and let it return the subprocess, less intermethod coupling


or maybe actually write a `TestServer` object, that sounds sweet doesn't it?


Landed in 1.6.0.


it is being used only in this object so it make little sense to keep it a private interface. also model is not good place for internationalization


FacilityTabGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2179 of 2200 tests passed)

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


wiki pages are often out of date, if you navigate around you'll see that for the current release of GNOME (which is 3.16) there actually is a tarball: http://ftp.gnome.org/pub/GNOME/sources/pygobject/3.16/


:+1: great!


Looks great, but please use the [new string formatting syntax](https://docs.python.org/3.4/library/string.html#format-string-syntax) instead. I'll merge when the changes are made.


then `setup_env_vars.sh` or so, setup only is pretty useless IMO


Hissssssssssss ...svn


if `nameTranslation` (or rather `polishNames`) would be described as a "static" property in scope of this file:
```js
var polishNames: {
//...
};
export default DS.Model.extend({
//...
});
```
then this function could look little simpler:
```
return polishNames[this.get('name')]
```


The job #1595.24, the last one `WITH_RUBY`, [here](https://travis-ci.org/sympy/symengine/builds/67073092) seems ok. I will wait.


We changed the HTTP handler to respond with `ProgramRecord` b/c we didn't want to respond with an unnecessarily nested object. I think we can change `Id.Program` to be more flat instead though. Opened https://issues.cask.co/browse/CDAP-2771.


having points as a model looks a little bit as overkill


Does adding really high values cause an unecessarily large array by auto-filling with a bunch of blanks? Do we care?


do you need to track this relation?


Hi Martin,

I've apply both suggestions in 6c9201fb40c0634b6c3d13a4cad653b810b480c1, and deployed 6.19.3-SNAPSHOT.

Please tell me if it looks ok for you, or if you think the `try/catch` is overkill.
I would like to stage 6.20.0 a few days before releasing, because it's a sensitive release...

Thanks & best regards,
Sebastien


How ti setup hola detect, I dont under stand this.... ?


Could you place imports in alphabetic order?


It was not on purpose.  You can put it back with another pull request and I’ll approve it.

Faisal

> On Jun 15, 2015, at 6:28 AM, mathiear <notifications@github.com> wrote:
>
> Hello,
> Could you explain to me why you removed some of my last commits (among them: max_client_buffer, ssl_sites , passive_monitoring) ? Those we not massive changes but very useful to make the puppet module more complete compared to the GUI options.
>
> Regards,
> Arnaud
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/riverbed/stingray-puppet/commit/0ab22c4ebdf43e0cc79a865737db2029fa470179#commitcomment-11680139>.
>


What does this do, exactly? Is it kind of like true vs truthy?


same as https://github.com/Tuhaj/traffic-watch/commit/ba41648023b976a5a6fc88807d6ef78c69255bd8#commitcomment-11709536 - path could be just `DS.attr('array')`


`NSString` should be used with `copy` attribute in general.


`env = {'SAN': "DNS:localhost, DNS:%s, IP: 127.0.0.1" % common}`


oh and it looks like we're using `.format` here so if you're feeling frisky change it to `"DNS:{}".format(common)`


why no return `weighted_mean` called `value` directly from server. afaik your server have only one consumer


This seems broken for Debian systems (the package name lua-devel only applies to the RHEL/CentOS flavor).

I'm also not sure why the package install action is necessary at all in this recipe?


Could you wrap all `if` statements in brackets, please?

     if (!content) {
       return nil;
     }


This commit added the functions for R Programming Week 3 assignments


I'll have to find the issue number, but at least for openblas there wasn't an API for `get_blas_num_threads` until pretty recently. And I don't think we have the ccall wrapper for it exposed yet, but it easily could be.


Oh.. yeah, this is probably not a great idea.


So is there a reason that this has been moved to here? It does make more sense to me, but I'm wondering if there's a specific reason.


For arrays we usually use `copy` attribute as well.


@rj0495 not currently.


isn't the or here for this purpose?


nonsense, this router could just have:
```
this.route('city', { path: '/:name'});
```
this how this would affect your controllers and code complexity (now you are using 'needs' in city controller - with only one controller this would make little sense)


Why so? Just return `scripts`.


no needed? ember should auto generate this on fly


Seems like this should be a separate commit


Allow supervisor to override MaxIterations without a Stop Work


The module name (.so) is chosen based on the function name, so same
function name -> same module name -> possible conflicts


not checked? none

but sometime, you get a diff, a patch, well, files that should not be
there. I add one and clang format did its best, which resulted in a
memory swap and bringing my machine down ;-)


I'll investigate if there's a better way, yes.


It's becasue if the cmake command fails, it still creates a possibly
faulty CmakeCache.txt

Makefile was a bad choice because it's linked to a given Cmake generator
(e.g. useless for msbuild)


why this is needed?


Right!


indeed


Ack :-)


Not sure `NSLog` is proper way to report errors here. May be you should return error from this method? Though I will have a look later where you call it.


yes


can this file be just removed?


Display JH results


Well it didn't really work :(

Also I get a warning when trying to compress the JS:

(v1.1.4)codeeu@events:~/coding-events$ ./manage.py compress
CommandError: Offline compression is disabled. Set COMPRESS_OFFLINE or use the --force to override.

I did not force this as I wasn't sure it was needed.


The "13" constant would be better off commented: Version 13 of the WebSocket protocol means rfc6455.


Read the documentation: "AudioProcessor which simply forwards the packages to the underlying NetworkWrapper" instead of packing the data into RTP-pacakges, this simply writes them to the NetworkWrapper, like we did before.


This is an odd place to have this comment, but: what does "the-re" mean?  I assume it's defined by Openstack?...but...what is it trying to name?


it looks like this component takes all properties from city model. why not to pass a city as it model?
```
{{city-map model=model}}
```


You have such check here but not on `ActionScripts.m:46`. I think you can remove it here..


Oh shoot thats what I was thinking of, thought that'd been around for a while


Also you use in one case dot notation in another `setSmth:`. It is mostly nit and up to you to fix or not.


Having trouble getting this to work recursively. Any thoughts, @kjbrum? I was able to get it to work with:

```sh
find . -type f -name '*.jpg' -exec mogrify -crop 250x250+0+0 +repage {} \;
```


Everything looks good to me.


I don't like the multi c'tor in ThreadPoolBasedScheduler. I'd make a single c'tor that gets a thread name, and create the thread factory there. Then it's easier to change the tests you didn't want to change.

Also it's time to extract a class for that ThreadFactory.


We use `copy` for `NSDictionary` too.


`environment`


I always assumed that October 25th, 2007 was the date the current chat protocol was invented. So it's weird that they've now changed it to a date in 2008.


Why??


This isn't the same as before though for scheduled tasks I think it's OK.


Hi @xhacker,
To use a custom `RSDFDatePickerDayCell` cell you only need to create a subclass of `RSDFDatePickerView` and override the method `dayCellClass`.


Could `oldEnv[@"PATH"]` be `nil`?


w00t!

:+1:


Fair enough. I figured the 'Start' function might reveal more than it hid, but it sounds like the opposite is true.

I'll test & merge later today.


If yes then use `array` + `pathWithComponents:`.


ack


`https://github.com/owner/project/issues`  -> `https://github.com/jenius/roots/issues`


@kevva @sindresorhus Could you publish v3.0.2 to include this change https://github.com/kevva/bin-check/commit/b1e9a6c4d365d8a97c57273c78ff0b8b5b948b54 to the npm package?


Just return `env`.


p.s. thanks +1


A slightly clearer commit message would help here; as far as I can tell this is used from suite and worker, and are you suggesting that we would not want to actually fetch, but just use something preexisting?  If so, I think I would be confused by the "fetch_teuthology doesn't fetch" aspect, and less so if the calling code changed.  Also, is it usually the case that, if you do this, you want the exact same path on the suite machine and the worker machine?...seems like having them separate would be more preferable, also leaning toward modifying the caller


Just use nemes_s<at>freemail<dot>hu

On Tue, Jun 16, 2015 at 6:08 PM, poxyran <notifications@github.com> wrote:

> Hey Sandor!, do you have an email to drop you some lines?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/snemes/pype32/commit/89bb5a258b5c769d9793beb020b158d397a15337#commitcomment-11706458>
> .
>

--
Sanyi


Era uma aula para o Aniki e o Koush <3


Yup, will fix.


Hmmm.  What about when we're running tests in subprocesses?

I'm working on writing test cases to cover the single-process condition (which I think your code will handle) and the multiple-process condition.

You can use the `g` stub to help test in-place code, by the way.  It's a lot easier than trying to install your custom changes.  `./g -r` is a good way to check your coverage for single-process.  `./g -r -s 0` is a good way to check your coverage for multiple processes.  (You'll see a bunch of unfinished multi-process work not covered -- I'm working on that in another branch right now).


And here you can use `[... copy]`.


shit, forgot to update my copy paste. good catch @hhsnopek


New line here or remove line 220?


This revision is clean.


We already have a file_contents, in dxr.build. So we should either import more shallowly or rename.


Done!


hahahahahhaaha que diver <3


I prefer `_environment` rather then `self.` but it is up to you.


:smile:


Worth noting that the spec's [description of `pre`](http://www.w3.org/html/wg/drafts/html/master/semantics.html#the-pre-element) seems to support this use case:

> The pre element represents a block of preformatted text, in which structure is represented by typographic conventions rather than by elements.
> ...
> Some examples of cases where the pre element could be used:
>
> -  Including an e-mail, with paragraphs indicated by blank lines, lists indicated by lines prefixed with a bullet, and so on.


I suggest you just write "hello fcserver" here. The "\n\x00" is just the "end-of-message" marker, and should be handled by the send() function instead.


Ne használj JUL-t, használj inkább slf4j-t


Also added you to npm @shinnn.


Nice catch!


Ah, certainly. I'm thinking something along lines of `file_contents_at_rev`


Thanks :+1:


I didn't found


And here you want `strong` I think.


I would just change the name of this to reqObj, and include the transmissionBody:

```
var reqObj = {
  transmissionBody: {
    options: {
      clickTracking: true,
      openTracking: true
    },
    content: {
        from: 'you@your-company.com',
        subject: 'First SDK Mailing',
        html: '<html><body><h1>Congratulations, {{name}}!</h1><p>You just sent your very first mailing!</p></body></html>',
        text: 'Congratulations, {{name}}!! You just sent your very first mailing!',
        substitutionData: {
          'name': 'YOUR FIRST NAME'
        }
    },
    recipients: [{ address: { name: 'YOUR FULL NAME', email: 'YOUR EMAIL ADDRESS' } }]
  },
};
```

Then you can just issue the command:
```client.transmissions.send(reqObj, callback...)```

Just reference the object literal as it should be instead of using dot notation, that will clean it up I think.


why don't you do `--prefix=$python_virtualenv` in the first place?


ALONSO


Try it.


I'd do the same for this object and call (to keep it consistent).


And here, also map the object literal and update the name to reqObj.


TeamCity dddlib :: dddlib [Build 69](http://teamcity.cameronfletcher.com/viewLog.html?buildId=434&buildTypeId=dddlib_Dddlib) is now running


Here too.


Appreciate the feedback!

The constant table doesn't change and you are right that a direct reference can be emitted. Or if I could inform the JIT compiler of the size of the array bounds then the optimizer would probably do it anyway.

One of the many things on my todo list.


I'm also forward declaring elsewhere where it isn't necessary.


This is ok but it would be even better to manage files created by an IDE in a user-specific ignore file. That way, a developer's IDE choice doesn't pollute the project-specific .gitignore file.

https://help.github.com/articles/ignoring-files/#create-a-global-gitignore


There. The first comment sums up everything I'm thinking that makes it more understandable.


TeamCity dddlib :: dddlib [Build 69](http://teamcity.cameronfletcher.com/viewLog.html?buildId=434&buildTypeId=dddlib_Dddlib) outcome was **SUCCESS**
Summary: Tests passed: 178, ignored: 7 Build time: 0:0:0


Nice!


rax retry tests


are you tryign to detect an nt2 proper installation or clone ? IIRC we generate some manifest file when installing.
the doc/ folder may not be there if doxygen is not present


@gureckis @jbmartin I made this change in the hopes that Anselm and I would be able to serve experiments from the "burn" server simultaneously as long as we use different ports. As far as I can tell, this should be possible, and safe since one user's processes can't kill another's. However this line was causing psiturk to search for all psiturk server processes across all users and treating them as owned by the current user, causing an error. I changed the flag so it would only find the current user's processes.

This isn't my area of expertise though - do you guys know of any reason we need to be messing with server processes of another user, or why two user's servers couldn't coexist?


:tada:


seen that, you can notice that city-map is just a wrapper around google-map. So maybe CityMapComponent should just extend GoogleMapComponent ? in this way gopt_disableDefaultUI would be described on a class


The comment can be removed.


Needs to remain as `id: polymer-starter-kit`.  This is the unique ID used to identify this specific show, it's not shown to visitors anywhere. For each show, the `id` should be unique, contain no spaces and only be small characters.


Ditto


The index should be superfluous in this case.


I find these insets a little hard to understand (since they control both the left and right insets, which are different in some cases for us). We need to adjust the lock insets a little more (i.e. they still need some padding so that text doesn't run into the lock). Doing so also actually helps offset the reader mode button if its showing.
https://dl.dropboxusercontent.com/u/72157/paddings.png

Unfortunately, the seem to also apply to the editing rect:
https://www.dropbox.com/s/knajrj66qr2djbu/editing.png?dl=0

I played with fixing this a bit, but I actually wonder if we'd just be better off using constraints to do all this.


Let's start the property name with a capital letter and extract it into a constant.


This is the show id, again, not shown to users anywhere but used internally to identify the show, it needs to remain as `showid: polycasts`


Should we also rewrite `(>>=)` to `bind` for the `Eff` instance? For cases where people might have used it explicitly.


Thank goodness. I copied it from its original location in `gjs_register_type`.

It's hard to know what is intentionally formatted and what should be changed when touching old code; cf. also the spaces before function call parentheses. Are there some GJS-specific rules?


Edits made to chapter design notes in response to #21 (but also #2)


Use the constant mentioned earlier instead of a literal string for the property name.


Not sure what this is for, you shouldn't need it as far as I know.


TeamCity dddlib :: dddlib [Build 70](http://teamcity.cameronfletcher.com/viewLog.html?buildId=435&buildTypeId=dddlib_Dddlib) is now running


If a username equals "username", it will also contain "username". You only need to do the contains check.


This is the season id, again, not shown to users anywhere but used internally to identify the show season, it needs to remain as `collection: polycasts-season-2`


TeamCity dddlib :: dddlib [Build 70](http://teamcity.cameronfletcher.com/viewLog.html?buildId=435&buildTypeId=dddlib_Dddlib) outcome was **SUCCESS**
Summary: Tests passed: 178, ignored: 7 Build time: 0:0:0


The description here should be updated, looks like I accidentally left the wrong description in.


Mention in the help string that this argument is implicitly in MB.


Thank you for the explanation. :+1:


This all looks good.


It's also assumed to exist in `gjs_object_class_init`. For me this was a case of "if it ain't broke..."


Done.  I refactored the code to make it more consistent and, hopefully, easier to read/maintain.


If the username contains "username" but is not username then this will incorrectly evaluate to true. Here's a breakdown:

(!(false) or !(true))
(true or false)
(true)

This should be
```
if (!AccountName.ToLower().Contains("username"))
```
taking into account the above, of course.


default event of action is `on="click"` so you can remove that part


ack


Done.


@RockyTV It's `eval`d, so you can GET and execute literally any form of JavaScript, from in-page keyloggers to a short script which redirects you to youareanidiot.org, so it has huge implications.


Haha, the first link on Google for JSAutoRequest is ["Where are the docs for JSAutoRequest?"](http://mozilla.6506.n7.nabble.com/Where-are-the-docs-for-JSAutoRequest-td295740.html)

I can certainly imagine what it does (calls end request when the object goes out of scope) but I'm a bit hesitant to use something that seems to be undocumented API... is it stable?


actually we don't have shebangs in the other tests, if you think they're something we'd want please open a bug for that. For now I'd prefer consistency.


as `time-series-chart` is used in this place only the selectedInterval, horizontalMargin and labelWidth passed here could be just its default values


unnecessary information is the main problem
the [X] is completely out of place
what people liked about each webstyle is only useful for the person writing the website, which you already know, so I would say that it doesn't need to be apart of the notes
websites go down, and there are links to websites that could go down
the people that voted for each style can go either way, but it is once again, more information that doesn't need to be there
the notes that go onto the main website should be short and to the point, I just see this as a huge block of text that could easily be simplified. This kind of extra information should be apart of the offline notes which are the notes that I take on paper


this is a missing feature, i opened an issue #10
Tell me what you think of the proposed api


Thanks for posting, very helpful!


I did not implement it as a package because thought that all of our applications that use this will want authentication and access control.  So, having it in the base saves the steps/time of installing it.

And I will add seeding of the roles and permissions.  Good catch!


Nice!


so you could use `-= 1` as a simplification, I find that better readable but that may well be debatable - so suggestion only


as map is also absolute this probably may be not, just replace left with margin-left and bottom with margin-top


@woffle Did you make sure to add the torch framework to the "copy resources" step in the build phase so that the lua files are copied?


WHY NOT I'll change when done pls chill


I prefer a = a - 1 as It works on all languages and also because it's easy to miss `-=`


I know.  I guess I left it in when I was testing :P


that enum was already defined https://github.com/TrinityCore/TrinityCore/blob/8caea5ec7c6b3187df7033c372aea4ffa5699c3b/src/server/game/Miscellaneous/SharedDefines.h#L4566


n_interfaces is the length of an array passed in to the JS function. I think it's very unlikely that that will turn up garbage at runtime due to something that a JS client does. What do you suggest?


Reason was that we didn't need the Sass files compiled until the CasperJS tests started. Since CasperJS won't run unless all of the non-Casper tests work, it was just eating up time.


As far as I can tell the original version should work fine.

Here is some example code showing that both an array of length 1 and an array of length 2 work fine.

```go
package main

import (
	"fmt"
	"math/rand"
	"strings"
)

func main() {
	for i:= 0; i < 100; i++ {
		var foo = strings.Split("abc", ",")
		fmt.Println(foo[rand.Intn(len(foo))])
		var bar = strings.Split("def,hij", ",")
		fmt.Println(bar[rand.Intn(len(bar))])
	}
}
```
I suggest we try:
```go
server := this.SERVERS[rand.Intn(len(this.SERVERS))]
_D("chose server '%s'", server)
dnsResponse, RTT, err := dnsClient.Exchange(&dnsrequest, server)
```
If it works... great! :smile:

If the code crashes because of `rand.Intn()` for some reason, then we will know for sure.

If the code crashes because of `dnsClient.Exchange()`, then we have to debug that.

Note that we don't really need the `s_len` attribute of our structures, since we can always get that easily by the `len()` built-in function.


ack


Good Idea, please increase major version next time, because this broke my code without I noticed that.


Obviously this is just a workaround, and does not really solve the bug; but it's enough to make the affected system work again.


Arrays are always passed by reference in JavaScript, and so just checking equality here doesn't work, as I needed to check that the *values* of the two arrays were the same, not that the two arrays were identical.


interesting logic, could you document it a bit as I'm not sure what does happen here.


Yes, you're right. So I guess I don't need a variable at all…

The index thing, what is it for then ?


It is meant to do that; otherwise we'd have "username123" and the like.


We started applying/traversing through that visitor class using an aspect node, so there was no point for the if anymore since it was only done for that one aspect


Das implementieren:
https://github.com/AliveDevil/SAE_HAM_P1_Textadventure/blob/master/TextAdventure/FastConsole.cs


constructor function should be called.. `new Date()`


This method is called by the generated code:
```java
invokeStatic(Greatest.class, "checkNotNaN", void.class, double.class)


`interface_type` is a `GType` which is just an int.


Currently? Solely because it's not in the ES7 Observable spec.


Ziemlich langer flagname. Ich würde -dir sagen, oder so


What @ChlorideCull says is correct.


shouldn't it look like this?
```
if(hour === 24) {
  hour = 0;
}
wantedTime.setHours(hour);
```


this is not good


But I think it is good


spurious newline


anyhow, I don't understand this logic, it just need documentation


TeamCity dddlib :: dddlib [Build 71](http://teamcity.cameronfletcher.com/viewLog.html?buildId=436&buildTypeId=dddlib_Dddlib) is now running


Warum erst eine getString Methode definieren und dann nicht benutzen?


On the other hand, Array and String generics require passing the array or string (the `this` object) as the first argument, so this implementation of generics was just wrong. Fixed this in the next iteration.


Right.. Usually it's up to the maintainer to increase the version on breaking changes (they might want to accumulate them in master for a while), but I should have articulated that it's a breaking one better.


TeamCity dddlib :: dddlib [Build 71](http://teamcity.cameronfletcher.com/viewLog.html?buildId=436&buildTypeId=dddlib_Dddlib) outcome was **SUCCESS**
Summary: Tests passed: 179, ignored: 7 Build time: 0:0:0


This commit has UNMERGED changes in GeoTracheWidget! Should not be here!!!


how this is changing anything? it seems that only `getStats` can change weekStats?


filepath.Join (dann muss man sich auch keine Gedanken um trailing slashes oder nicht im pfadnamen machen)


os.Create


TeamCity dddlib :: dddlib [Build 72](http://teamcity.cameronfletcher.com/viewLog.html?buildId=437&buildTypeId=dddlib_Dddlib) is now running


the `displayedTime` property can have `getDayFromName` as it setter implementation, so you wont need separate public function


"Something went wrong" hilft einem nicht wirklich weiter, als Fehlermeldung. "Could not create file %s: %v", name, err schon eher.


Plus, nutze fmt.Fprint{f,ln,} mit os.Stderr


(alternativ: log mit log.SetFlags(0))


ditto


FacilityTabGridView (evanweaver) - i386-Windows-7-VisualStudio-12: Build Failed

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


Hi can fix Armorgrp on Erthiea can open but can't save file to dat


Bei näherer Betrachtung: Nutze log.Fatal{f,ln,} mit log.SetFlags(0)


it looks kinda awkward - maybe it is due to you controllers structure. I would think if those thins could not be decoupled.


Why do you change this?


TeamCity dddlib :: dddlib [Build 72](http://teamcity.cameronfletcher.com/viewLog.html?buildId=437&buildTypeId=dddlib_Dddlib) outcome was **SUCCESS**
Summary: Tests passed: 179, ignored: 7 Build time: 0:0:0


default


thx,昨晚才添加的,不过其实我有一个更大的bug还没找出来,当导入全部的nobench_data.json时,使用find dyn2 = false命令会导致段错误,调试了一下找到的是在第43911~43935里面进行解析的时候读出的attrNum除错,然后就segment default,有兴趣帮我看一下,我更新了版本,使用buffer缓冲,用户使用类似more那样查看find或者showJson的数据(不过不能倒回,那样太麻烦了而且目前没必要);


Statt in einen buffer zu schreiben und den dann in die Datei zu schreiben, einfach datei aufmachen und als writer übergeben.


@aldeed I've been thinking about using `this` helpers in order to keep options immutable. I'll think of a pattern - eg.:
On allow rules the highest limit should win, on deny the lowest, deny overrules allow. Same with fields.

I'll add your suggestions, thanks :)


reek would cry as I'm now. this file is too long and far too complex for such small app


sweet!


again, shouldn't it be a part of city controller?


<t>log?


I'm not sure if this is specific to me, but as soon as I introduce this second unit test, the first one fails.

    ======================================================================
    FAIL: test_saving_and_retrieving_items (lists.tests.test_models.ListAndItemModelTest)
    ----------------------------------------------------------------------
    Traceback (most recent call last):
      File "D:\Code\tdd-tut\superlists\lists\tests\test_models.py", line 25, in      test_saving_and_retrieving_items
        self.assertEqual(saved_list, list_)
    AssertionError: <List: List object> != <List: List object>

I think this is caused by the second test (which is tested first) and the creation of that temporary list in the database. The first test fetches the first list entry (#24) instead of the one earlier generated (#11).

It only fails when both tests are run in that automatic lexical ordering.


128bytes ought to be enough for everybody.


`City` model has stats as async relation, so shouldn't this be replaced with
`city.get('stats').then(//..`  ?


I will revert this !! D;


that's the moment for momentjs


Your workaround doesn´t work for me. I used both codes you´re posted. I´m running a Windows 7bit machine. Same error "avr-g++: command not found" any ideas?


Nice even if the commit message doesn't match the actual change :p
I'm afraid a setInterval of 78 would be too fast


did you consider what's used upstream for pods?
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/pkg/api/validation/validation.go#L994
It could be better than the regular DeepEqual for our purpose:
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/third_party/forked/reflect/deep_equal.go#L225-L233


I think you need to add documentation for this new argument— it looks like our travis builds broke here.


why to wrap it in another day object? just return now. also momentjs can do that for you


Since we are overriding the other writeInternal accepting a Type, would this ever be called? If so then raise UnsupportedOperationException similar to what is done in `boolean supports(Class<?> clazz)`.


Okay, I'm getting real sick of dealing with single-process logic separate from  multiprocess logic.  Now may be a good time to just make the default run actually use the multiprocess architecture I have in place (just with 1 worker).  Then we can just deal with the multiprocess issues as they come up.  Like the fact that a crash in a process that's a worker for a pool gets silently ignored and the worker just gets restarted, unless you do something yourself to handle it.

So that's what I'm experimenting with in conjunction with this catch-unexpected-exceptions idea.


>"notes that go onto the main website should be short and to the point, I just see this as a huge block of text that could easily be simplified. This kind of extra information should be apart of the offline notes which are the notes that I take on paper"

Not at all, the more info you can provide the better, we should be able to go back 6 months from now and be able to reference the notes. Additionally the notes that go on the website are the official ones. As in if we ever need to justify anything that's where we have to reference it from. Any hand written notes are *entirely unofficial* and as far as the organization is concerned, do not exist - that is just an extra step you decided to take to make things easier on yourself.

>"what people liked about each webstyle is only useful for the person writing the website, which you already know, so I would say that it doesn't need to be apart of the notes"

On the other hand why shouldn't it be a part of the notes, you are assuming that no one will ever find this useful except for the one(s) working on the website (very likely me) and that is true, but it is still an ACM matter, thus the more information provided the better. In a year from now if we continue the GameJam it might be helpful to look back and consider some of the design choices that were made and why. Either way the additional info hurts nothing.

> "and there are links to websites that could go down"

All of the websites are hosted on github meaning that it is free hosting, thus vastly decreasing the chance of them ever going down.

>"the [X] is completely out of place"

I will relent that the
- [X] s are unnessisary
I kind of forgot they were there :toilet:


My code isn't liking this too much,

```
>> py('eval', 'print "hi"');
Undefined function or variable 'mfilepath'.

Error in py (line 24)
	mfiledir = fileparts( mfilepath );
```


We really should restrict new major updates at least since now, or we have no chance of releasing this month.


why not return lat and lng from the server? center is not used anywhere in it's original form


This segment of code looks like it should only perform when the text does **not** contain "username", unlike the above. The example I provided contains username executes anyways.


I think you need to update the trainControl man pages.  [R CMD CHECK is returning](https://travis-ci.org/topepo/caret/jobs/65253090):
```{bash}
* checking for missing documentation entries ... WARNING
Undocumented code objects:
  ‘getSamplingInfo’
All user-level objects in a package should have documentation entries.
See chapter ‘Writing R documentation files’ in the ‘Writing R
Extensions’ manual.
* checking for code/documentation mismatches ... WARNING
Codoc mismatches from documentation object 'caret-internal':
createModel
  Code: function(x, y, wts, method, tuneValue, obsLevels, pp = NULL,
                 last = FALSE, sampling = NULL, classProbs, ...)
  Docs: function(x, y, wts, method, tuneValue, obsLevels, pp = NULL,
                 last = FALSE, classProbs, ...)
  Argument names in code not in docs:
    sampling
  Mismatches in argument names:
    Position: 9 Code: sampling Docs: classProbs
    Position: 10 Code: classProbs Docs: ...
Codoc mismatches from documentation object 'trainControl':
trainControl
  Code: function(method = "boot", number = ifelse(grepl("cv", method),
                 10, 25), repeats = ifelse(grepl("cv", method), 1,
                 number), p = 0.75, initialWindow = NULL, horizon = 1,
                 fixedWindow = TRUE, verboseIter = FALSE, returnData =
                 TRUE, returnResamp = "final", savePredictions = FALSE,
                 classProbs = FALSE, summaryFunction = defaultSummary,
                 selectionFunction = "best", preProcOptions =
                 list(thresh = 0.95, ICAcomp = 3, k = 5), sampling =
                 NULL, index = NULL, indexOut = NULL, timingSamps = 0,
                 predictionBounds = rep(FALSE, 2), seeds = NA, adaptive
                 = list(min = 5, alpha = 0.05, method = "gls", complete
                 = TRUE), trim = FALSE, allowParallel = TRUE)
  Docs: function(method = "boot", number = ifelse(grepl("cv", method),
                 10, 25), repeats = ifelse(grepl("cv", method), 1,
                 number), p = 0.75, initialWindow = NULL, horizon = 1,
                 fixedWindow = TRUE, verboseIter = FALSE, returnData =
                 TRUE, returnResamp = "final", savePredictions = FALSE,
                 classProbs = FALSE, summaryFunction = defaultSummary,
                 selectionFunction = "best", preProcOptions =
                 list(thresh = 0.95, ICAcomp = 3, k = 5), index = NULL,
                 indexOut = NULL, timingSamps = 0, predictionBounds =
                 rep(FALSE, 2), seeds = NA, adaptive = list(min = 5,
                 alpha = 0.05, method = "gls", complete = TRUE), trim =
                 FALSE, allowParallel = TRUE)
  Argument names in code not in docs:
    sampling
  Mismatches in argument names (first 3):
    Position: 16 Code: sampling Docs: index
    Position: 17 Code: index Docs: indexOut
    Position: 18 Code: indexOut Docs: timingSamps
```


Same here, can this be called?


sorry, but that is just nonse, and I'm sure you see it


@grunion Touché, you are correct. I've amended the changes and made a pull request to have the code read from the Torch.framework. My apologies. Thanks!


I guess I will have to port it to PCBTARANIS, or rather VIRTUALINPUTS!


Consider
```
void
advance (Env& env, std::shared_ptr<Ledger const>& last);
```
or
```
std::shared_ptr<Ledger const>
advance (Env& env);
```


crap, I corrected that in another commit, guess that didn't get pushed :/


Combine this if with isWormholeRound, so that you can still throw money if it's not wormhole level.


done.


rename this to `START_MINUTES` or something.


Can you make this a sub object of CONTROL?


Curso de Java dia 02, constructor, constructor con herencia


NOTE there were merge conflicts in x86-64.S, I hope I solved them correctly but we need testing on Win64 and X32-ABI. Original commit by Solar is 0dc72f7f1.


This should print the stack trace by passing the exception!


same comment as above


Don't worry :3


Resolves #19


`move` on a `bool` looks odd, @howardhinnant ?


Yeah something like that should work.


-Made it more website-specific friendly
-Fixed missing 'reddit' column


Don't we need to default the move members for non-MSVC?


Is this a recursion issue as in #1045?  CC @dmarcelino @shaunc


I see just over one percent boost for LM


@bmac I like using `service:store` for the injection, nice!


Prefer `/** Return a property if it exists */`


We're seeing IOExceptions for empty attachments which have a Content tag of ```<t:Content/>```. Currently we're only matching against ```:Content>```. This commit modifies the stream matching code to look for that pattern and ```:Content/>```.


Update the version number so we don't have to do any craziness to get the changes.


In general I agree, however there is a complication when it comes to pedigrees. Few people other than us needs the pedigree in the internal format, specifically with al the positioning data (e.g. most pedigree export formats have the pedigree data itself, but no positioning data). So, patient JSON should of course include the pedigree data, but, presumably, without positioning info. But the editor needs the info to display it. So we may still need a dedicated pedigree getter service


Confirmed working OK on non-proxy


Just a normal comment token here.


The original logic is extrapolated to work against a List of patterns. All patterns are checked during the single read of the InputStream.


When a statement introduces a new scope and continues across two lines, the second line shall receive an extra tab stop:
```
            if (auto test = dynamic_cast<
                    detail::prop_type<Prop>*>(prop.get()))
                return &test->t;
```


why?


Fix the empty test.


offline


	open
	/

	click
	id=suggestion-search

	type
	id=suggestion-search
	tv

	clickAndWait
	css=button.search-icon

	clickAndWait
	css=a[title="Smart TV LED 40” Samsung 40H5103 Full HD Função Futebol ConnectShare Movie"] > img.loading

	click
	//button[@type='button']

	click
	id=navegaCarrinho

	clickAndWait
	css=span.cart-icon

	clickAndWait
	//button[@type='button']

	click
	css=i.icon-plus-sign

	type
	id=txtReceiverName
	testescep inválido

	type
	id=txtPostalCode1
	01234

	type
	id=txtPostalCode2
	455

	click
	css=div.modal-footer > #btn-set-delivery


```
    virtual std::unique_ptr<
        basic_prop> clone() const = 0;
```


Could do without braces


Do we need move members?


for rackt/react-router#1317


```
        return std::make_unique<
            prop_type<T>>(t);
```


extra blank line


This might be a nested type of `Env` since that eliminates its visibility and I think, is only used by `Env`


I like the `move` on the `bool` because:
1.  It works, even though does nothing.
2.  If the `move` was missing, I would have to manually analyze whether or not it was accidentally omitted.  The way it is I don't have to do that analysis.
3.  Even the compiler-generated defaulted move constructor will "move the bool".

Yes, we do need explicitly defaulted move members for non-MSVC.


Any particular reason for this rather than keep the name?  It's not ARQ-functionality specific; it just happened to live in ARQ for historical reasons.


http://en.cppreference.com/w/cpp/language/move_constructor
```
     there are no user-declared copy constructors
    there are no user-declared copy assignment operators
    there are no user-declared move assignment operators
    there are no user-declared destructors
    (until C++14) the implicitly-declared move constructor is not defined as deleted due to conditions detailed in the next section
```
Obviously MSVC needs a manual one, but when I included `default`ed members, gcc and/or clang choked because there it didn't auto-generate the copies!


Oh, you commented on the commit, instead of the diff. It's correct here: https://github.com/bgruening/docker-ipython-notebook/pull/49/files


This might need to go in the `jtx` namespace and not a `detail` namespace, or else how could someone write code that calls `Env::set(std::unique_ptr<basic_prop>)` ?


Once you have the source ready, I can test it on my linux dev machine before we do a beta. I know for a fact with plugins that ControlEdit.getcontrol() crashes kodi because I've had it happen with 2 different addons using two completely different window xml's. However, because trakt is a script things might be different since scripts don't get a default UI.


I do not know what the differences are but building with http://www.unicode.org/Public/cldr/21/core.zip seems to give the same result as this fix.


I took it indeed to be ARQ-specific because of the next line: `static { ARQ.init() ; }`. Is that not so?


*g


Do you think it will be helpful or confusing putting elements and components into a single patterns folder? Our sass sections them out.


To reduce confusion, place any needed special members (destructor, default constructor, copy constructor, copy assignment, move constructor, move assignment) prior to any other constructors.


Ok, I don't think that's necessary though; especially since all the other `Id` classes are not flat objects either.


Move it into CmdARQ?


Muy bonito !


Small issue - should be a space between `LocationDecimation` and `{`


How will for this work for dynamically added caches?


Cool! Thanks so much!


`*.iml` files shouldn't be under version control.  Please check the OBA Android `.gitignore` files and replicate them here for the correct files to ignore.  Note that you'll need a `.gitignore` under the root and project directories.


let's taste acid


Don't forget to make a release for bower


I think it's better for the pages structure to have them all in a patterns folder. The front matter then determines which type it is (element or component) and see this is a better route (especially since the naming conventions for types might change).


I think I would prefer to use a non static field. Using a final instance for a complex instance has brought us into trouble before. Basically it was a source for a class loader leak. So unless we do (or you already have done) a full analysis what this parser does, I'd prefer just an instance per bridge. You still get your perf boost.


Yes.


Needs guard against self-copy-assignment.


Sounds good  :+1: especially as things are moving around so much in the beginning.


Thanks! Fixed.


`prop_list` is only used by `JTx`, as opposed to `Env`. I'll move it.


"files.," -> "files,"


But the question is does this actually fix it?


I know... but I want this to work nicely as well. After all, it helped to identify various generated files packaged in the original tarball.


@toshok do you still have your changes that added some exceptions stats?  Even these simple ones that I added are pretty helpful, but I think I remember seeing that you had some more detail available.


Maybe clarify that specifying the preprocessor has to be done by the user, it will not automatically be applied.


Move members wouldn't hurt.


CVE-2015-4619 http://www.openwall.com/lists/oss-security/2015/06/16/11


asdasdas


Confirmed, this causes a recursion issue.


## This is a markdown file


Can we even compile X32 in cygwin64 ???  This was code all in a ```#if defined (_WIN64) || defined (__CYGWIN64__)``` block


Just a small optimization here. You can save `this.getTheme()` into a `var` so that you're not calling this function multiple times.


@juliaelman Just so I know for future naming of files, are we separating words with an underscore (e.g. `layout_system` instead of `layout-system`)? Is this so it's easier to select by double-clicking or is there another reason I should be aware of?


:+1: glad it was simple!


@mfojtik fyi.


why would that change things?


<!-- кружочки делаем сами, и не забываем что там еще 2 слайда в скрытых слоях -->


@thanhdut DWMR. This is all spot on. Well done.


FacilityTabGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

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


This seems like a breaking change if the error which is being exposed has changed. What is `err.message` now?


Please move all the Facebook Login specific functionality to its own function (e.g. ```setupFacebookLogin```) that you can call from here. In general, try to put all your functionality in well-named functions so that it's easier for other developers to read your code.


Hey Bejond,
When I do rebase, it get conflict, could you please solve this conflict and resend it?
lease solve this conflict and resend it?


83811490-Fault_DirtyAirFilter (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1675 of 1678 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)


CONFIG_MSM_QDSP6V2_CODECS=y
Just for clarifying my mind
This defconf entry enable this compilation (audio_utils_aio.c) at least for _shinano-leo_.
But audio_aio_compat_ioctl function is used only if CONFIG_COMPAT is enabled.
Then CONFIG_COMPAT is used for arm64 (I mean it is not present at our arm setup).
So, I am sorry, although it is a fix but we really don't use it on our devices.


`err.message` is `Cannot calculate length of [object Object]`. This is because all the listeners in a state execute.


Added


I'm sorry that I do not quite get your point, would you please explain it in more details?


não entendi isso, esse teu metodo busca o id do projeto não da skill


Kappa


I submitted a PR upstream, as you suggested. We'll see if it gets merged!


Please document where this ID comes from (i.e. what account is associated with it, how it can be modified, etc.)


Please double check your strings for spelling errors


83811490-Fault_DirtyAirFilter (rongpengzhang) - 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)


@diego-d5000 I left you some feedback. Please resolve each comment, push a new commit with the changes and I'll review it again. When it's OK, I'll merge it into the development branch. Thanks


I am generally positive to ssiloti's changes (and I have been gradually merging pieces). The main thing I'm struggling with a little bit is trying to keep the scope of libtorrent minimal (or at least smaller than it is now). I've been deprecating things like rss support (which really is independent from bittorrent) and I've been working towards splitting out portions (not very successfully yet, but with github I imagine that may get easier. I'm thinking of bdecode, the dht, the heterogenuous_queue).


Well, thank you for the advice. However, from the manpage of mkstemp(), it says this function generates a unique temporary filename from template, creates and opens the file, and returns an open file descriptor for the file. For creating a unique filename, it's OK. But the file will then be created(if it does not exist) and opened by the legacy open() system call. It might not be a good choice here as the file needs to be opened by ceph_open(). Please correct me if I'm wrong.


I don't know if we distinguish between holes and outposts, but holes do not exist on the 7th or 8th rank because holes have an opponent's pawn in front of them (they are blocking a backward pawn which is a shield of sorts protecting from a rook sacrifice for the minor piece and the defending pawn and potentially a lever.  The shield against that trade enables that piece in the hole to become more powerful that that trade).  An outpost is any pawn protected location that can't be hit by a pawn and a piece is using that square.  An outpost is only a potential outpost unless occupied, while a hole is still a hole even if not occupied.


Goodnight moon


some of the data in the bio-data is from proprietory sources that is why it is kept in a private repo. Wouldn't you be able to run the script on a sample set of the opendata sources to get the output?


- [] 123
- [] 456


The RAM Disk for Kernel Mode Linux


Grr, I thought I had caught all these. Maybe it was introduced since the last time I checked.


Yes, absolutely, I meant for ProcessMessage to block until the batch has been completely sent, but forgot to have it explicitly wait. I'll push a new commit with a fix, thanks for the catch. :)


This is a pleasingly small file. I'm at 725 lines of code on radial phylo and haven't written the documentation yet lol.

1. One thing I don't see is validation of the function arguments. If someone messes up an argument, the function is going to fail with a cryptic error message that might make the user think the function just isn't written right. It would be much more user-friendly to give some sort of helpful error, and fairly easy with some simple error checking. Some examples from `radial_phylo()`:

    ```R
    > radial_phylo(d)
    Error: Argument 'seqs_col' must be a column index or a column name when 'd' is a
      data.frame, or NULL if 'd' is a vector
    > radial_phylo(d, 'clone')
    Error: Data does not have column named 'clone'
    > radial_phylo(d, 'cdr3')
    Error: Sequences must only contain characters from A-Z and a-z
    > radial_phylo(d, 'cdr3', rings=c("antigen.species"=1))
    Error: Validation of the argument 'rings' failed: the column name 'antigen.species' does
      not exist in input data 'd'
    ```

  * Have you tried passing a single-column data.frame to `data`, or a vector?
  * What if the user does `krange=5`? What about `krange=c(-10, 20)`? `krange=c(5, 2)`? Or `krange=c(4,4)`? In a scientific application where the code is being used for research, I think the worst thing to happen is that your function somehow runs with unexpected inputs and as a result produces a false (but seemingly correct) outcome. Good validation of function arguments can prevent that type of situation. As an example, here's how you might take care of validating `krange` for a decent number of scenarios by simply preventing unexpected inputs (notice how the error checking is cascading, getting more specific with each if statement):

    ```R
    if (is.null(krange)) {
        stop("The argument 'krange' cannot be NULL", call.=FALSE)
    } else if (class(krange) != "numeric" || length(krange) <= 1) {
        stop("The argument 'krange' must be a range of integers", call.=FALSE)
    } else if (krange[1] <= 1) {
        stop("The range 'krange' must begin with a value greater than one", call.=FALSE)
    } else if (krange[length(krange)] < krange[1] || krange[length(krange)] <= 1) {
        stop("The final value of the range 'krange' must be greater than 1 and greater than
              the beginning value of the range", call.=FALSE)
    } else if (any(floor(krange) != krange)) {
        stop("The range `krange` must contain only integers", call.=FALSE)
    }
    # You could wrap all this in a `validate_krange(krange)` function.
    ```

  * What if `item.max=0`?
  * What if `runs=0`?

  I think you'd want to check for:

    1. The right data type (e.g. `data.frame`, `character`, or `numeric`, etc.)
    2. The right length (e.g. if you expect a range, check to make sure a single number isn't passed)
    3. The value conforms (e.g. if you expect an integer, make sure you don't get a float)
    4. `NULL`. Weird things happened when arguments are `NULL`, especially if you use that argument in an if statement.
    5. Anything other unique requirements. In this case, all data in `data` should be digits only. You might also want to issue a warning if the data is only 0s and 1s.

  Since most of your data gets passed into `kmeans` a lot of this might already be taken care of, but it would be worthwhile to check, and add your own validations to supplement when needed. A cursory look reveals `kmeans` validates `centers`, but not `data`:

    ```R
    > kmeans(d)
    Error in kmeans(d) : 'centers' must be a number or a matrix
    > kmeans(d, 3)
    Error in do_one(nmeth) : NA/NaN/Inf in foreign function call (arg 1)
    In addition: Warning message:
    In kmeans(d, 3) : NAs introduced by coercion
    ```

  I see some validation of `krange` in Line 43. Nice. I'd change your error message to "The argument 'krange' must be a range that begins with a value greater than 1" to be as clear as possible, and use `call.=FALSE` in `stop()` to suppress the traceback:

    ```R
    if (1 %in% krange) stop("The argument 'krange' must be a range that begins with a value greater than 1", call.=FALSE)
    ```

    The error message there now sounds like you're going to make something > 1 yourself, and it's a good practice to name the function argument that caused the error.

    The added benefit of doing extra good validation on this function is that you will **know** that the class you return is correct. Then any function that works off this class has to do hardly any error checking and will never break.

2. `data` is already defined in R (`?data`), so I'd steer away from that.
  * I'm using `d` instead (after "data"), other packages have used `x`. We should probably pick something consistent.

3. Your call to `kmeans` should be ``stats::means``, in case the user has some other package loaded that has a `kmeans` function. It shouldn't matter for less common function names, but in my own library I can see three different packages with functions called `kmeans`.

4. Nice...I notice you're using `swss` now as a scoring measure rather than silhouette to increase performance.

5. Have you run `devtools::test()` or `devtools::check()`? I think this will fail the code style check since quotations should be double, not single.

6. Do you have documentation somewhere for you class `cluster_optimal` so we know what data is in it? I can see most things are lists, but I don't know what type `clust_gap` will be, for example.

7. Function name and class name: I still have some confusion based on the existing names since they're saying "optimal" but *everything* is being returned when I look at the code. Doesn't seem like the name matches. What about something like these for the function name:

    * `cluster()` (verb...I can't find any existing functions with that name)
    * `cluster_ranges()` (trying to keep up with the "verb" function names)
    * `ranged_cluster()` (verb)
    * `multi_clust()` (verb)

  And for the class name something that indicates it contains data for a range of clusterings:

    * `clusterRange` (I think it's supposed to be camelCase for classes...don't remember)
    * `multiClust`

  Those seem to make more sense when you think about taking `multiClust` or `clusterRange` and then subsetting `$kbest`. Whereas if you already had `clusterOptimal` why would you need to subset to get `$kbest`?....know what I'm saying?

Coding is hard. This is awesome stuff. I think the final 10% of fine-tuning for production-code takes a bit more time than we're used to compared to writing stuff for labs, etc., but it's worth it and good to take care of now so it's not a worry later (or in case we're unable to change it later because something is already in production). I'm just about at this stage for radial_phylo. Still need to write documentation (added my final error checks/argument validation today). Would you be able to do the same look-over for me in a day or two?


I thought the user of bitseq must do the retry and not the bitseq package itself.
Especially because, if there is a `RetryErrorf` failure, that indicates that the data has been modified and hence PushReservation must be called again with the new data (new ip-address in case of ip-allocator)


Can you specify the last name ?


Ditto


I always build kernels out of woof tree. So "Error" is a bit harsh. I think "Warning" will do. Anyway, doing a bit of work on k-k for 4.x kernels. Or, put kernel skeleton inside k-k? Or at least prompt the user to do that if they are building 'out of tree'.

Also, I'm thinking about adding a routine to grab the git firmware from kernel.org. It will need to be 'out of woof tree' because it is a huge download. The script should check if it exists, offer to download if not, if it does then just 'git pull' to update.

The firmware can be included by way of a config file; just a list of what needs to be included. (Of course that will need to be maintained as newer hardware emerges). Then loop over the list and copy in as needed.


Sure!


@hectorcorrea fits is picking up both, but shows a conflict.
``` xml
    <identity format="Plain text" mimetype="text/plain" toolname="FITS" toolversion="0.6.2">
      <tool toolname="Jhove" toolversion="1.5" />
    </identity>
    <identity format="RTF" mimetype="text/rtf" toolname="FITS" toolversion="0.6.2">
      <tool toolname="Exiftool" toolversion="9.06" />
    </identity>
```
This is with version 0.6.2. See what version we're currently using in AS production.


use `g_clear_object`


i'm definitely guilty of it myself, but in gjs, no space between function name and paren


Will do my man, I'll try tnt

On Monday, June 15, 2015, ttiltonheylin <notifications@github.com> wrote:

> This contains most of the voting sequence necessary to make the sequence
> valid from a political science standpoint.
>
> However I was having trouble manipulating the "countArray" after the first
> counting sequence. An index would return NaN, and I just wanted it to have
> 14 elements instead of 15 with one of them being NaN.
>
> I tried to "proof-read" it but let me know how I did. (spacing, tabbing,
> etc.) Also I will have to show this to someone who isn't good with
> computers at some point, i.e. PoliSci person, so I need the comments to be
> relatively basic.
>
> You have a way about you, so I would love some help!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gummieCrew/demoMath/commit/21b9e255def9efe958c458fe66bb6de42623d4e6#commitcomment-11684418>
> .
>


Yes, I was thinking something along those lines. At least it's more self-evident that the request worked than if you get back the empty string or an empty object like "{}".


please just make the if statement run long rather than doing this awful formatting


Can you explain why this help?


Two nits:
- My laptop volume is on full, and it's still too quiet
- For future videos, maybe edit with something with syntax highlighting, eg Sublime text, so parts of the YAML file are visually distinguished (like in the github view)

Thanks a lot for the documentation work!


rax retry tests


we have used github.com/ngaut/log instead of github.com/ngaut/logging.


I didn't use assert here because I don't want to explicit make this an assertion. I just wanted to avoid a false positive on the following assertion (line 30).

Maybe it's better to remove this entire `if` block as there are other tests from `.has` checking its proper behavior.


Remove the empty line at line 24. This is causing the lint tool to report an error; see https://travis-ci.org/w3c/web-platform-tests/builds/66148603


@joshuap this is exactly the kind of thing I was talking about in #130.  All of this stuff:

* Is it in the YAML?
* Is it in the environment?
* Is it the right type?

which is not the primary focus of the gem, that you all now have to worry about because of all of the problems with storing your configuration in environment variables.  A good configuration manager will [handle all of this stuff for you](https://github.com/thekompanee/chamber/blob/master/lib/chamber/filters/boolean_conversion_filter.rb).


If you can't tell I'm _really_ trying to convince you to switch back to a runtime configuration :grinning:


Remove these two empty lines at line 28 and 29 (they’re not causing lint failures but still there’s no need for them.


make 100 as a const?


Thank you, Mike!


With that commit we now show menu on F10 with ``OnEnterMenuLoop()``. It's weird. Can't decide bad kind of weird or good.

About "unfocus main menu if system menu was entered with alt+space", didn't know windows can cycle between main and system menus with arrow keys. That makes showing hidden menu even more complex. Will look into that in free time.


The method to calculate GC content use (G+C)/total_length might be biased if sequences contain N, or other degenerate nucleotides. Is it better to use (G+C)/(G+C+A+T) to calculate GC content?


HOLA


Is this alert just some debugging code you forgot to remove? Otherwise, I don’t know what the intent is, but please remove it.


Yes, this is redundant with record/history. Now that I think about it, it's probably better to delete the other thing and rename this to "history".


Ah, this one needs `#include <stdint.h>`. I added it in the next commit.


83811490-Fault_DirtyAirFilter (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (646 of 646 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)


new fixes at https://github.com/tc39/test262/commit/6f4c957d83b60a7befc75f85ecbf0eb134b69e91

I'll fix the patches up and open the PR.


> why not use sessionStorage for BrowserHistory as well?

Sure, we could totally do that. I actually had that same thought. In fact, @ryanflorence just suggested that we might be able to use `componentWillUnmount` to save the scroll state instead of hard-coding it into the DOM history implementations. That would definitely be an interesting exercise. You up for making a PR @taurose? :)

> I think we should differentiate between persistent state and state that's only used once.

I think the only state we need to worry about is the persistent kind. If you have state you only use once, you need to figure out some other way to pass that around.


this is one long line, huh? here's a pattern i follow for longer lines:

    @posts = Post.visible_to(current_user)
      .where("posts.created_at > ?", 7.days.ago)
      .paginate(page: params[:page], per_page: 10)


oh, sweet!


Perhaps a comment would be nice at this place to prevent someone doing a bad refactoring here and to help simplifying it again once php min is > 5.4


@bgoertzel Ben, okay to give our GSOC student access to the private gitlab repo with the agreement that he will only work with the public data there?


awesome!


Please remove this heading.


the `new ` should be removed.


Yep. You're here. Now I am too.


sweet!


Remove this div. The test harness doesn’t require it.


The Help roomintro should probably not be limited if it's to be a normal room again. :|


Hm, I wonder if Gem::Platform was Comparable at some point, and I wonder if that wouldn't be the more robust solution.


Good idea.


pretty simple, huh? sweet!


Not particularly. That seems simpler, given that this is for machine parsing.


Jeffry,

So I did some rough draft work on implementing Any and All Operators.  This
implementation just handles the whole query, and doesn't deal with arrays
and such yet.

https://github.com/brantburnett/Linq2Couchbase/commit/b46298047e561559cf37c9ab38495076f3a68270

I'm not sure how well this will perform.  N1QL doesn't seem to support ANY
and ALL keywords wrapping subqueries like more traditional SQL does, so I
had to use COUNT(*) > 0 and COUNT(*) = 0 as my test mechanisms.  I'm
uncertain if N1QL will short-circuit these once the first matching record
is encountered.  Thoughts?

I also had to change the behavior of ExecuteScalar, as it did not appear to
be functioning in its current implementation.  For now I'm assuming that
the first column will always be named "result".  I'm not sure how this will
scale with other usages of the ExecuteScalar function.

Thanks,
Brant

On Fri, Jun 12, 2015 at 3:54 PM, Jeffry Morris <notifications@github.com>
wrote:

> Brant -
>
> I'll take a deeper look this afternoon, but thus far it looks good.
>
> There is definitely room for you to work on the Linq2Couchbase project; I
> am a bit behind on the project management side at the moment, but will open
> some tickets. If you wanted you could always start with Any(), since you
> indicated it wasn't supported yet. Myself and others that have worked on
> the project have found the learning curve a bit steep, so always nice to
> start with something rather simple (seemingly, but you'll find it more
> challenging I suspect once you get past the basic use-cases).
>
> Note that I plan on refactoring the internals and structure of the project
> soon; I am not convinced the current structure will scale to the
> development needs of a full feature Linq provider. There is not whole lot
> of documentation on building a linq provider with ReLinq, so a few steps
> forward and a few back are in order.
>
> Here is a partial list of features and the mapping between Linq and N1QL:
> https://docs.google.com/document/d/1hPNZ-qTKpVzQsFwg_1uUueltzNL1wA75L5F-hYF92Cw/edit
>
> It's very much a WIP and comments and suggestions are encouraged.
>
> Thanks,
>
> Jeff
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brantburnett/Linq2Couchbase/commit/431c6fc0671ac3baab8d79698ff74f6064dc0fd4#commitcomment-11660885>
> .
>

--
Thanks,
Brant Burnett
CenterEdge Software
(336) 598-5934
http://centeredgesoftware.com


@nhpatt @jmnavarro what's the use case for this one? Why do you need to know the os version, model and a app version?

Could we take a different approach and let developers define whatever metadata they want? It could be a JSON object stored in a column called "metadata" or something like that, they could define new properties, like screen size or such.

They wouldn't show up in the search container but we could display them once the user clicks on a device and wants to "view" it.

Another approach would be to use expandos or DDM to store these properties, I think those allow us to sort and group by each property.


trying to under stand how this works


I'd prefer multiple lines.


nice job!


You can safely use std::vector, except that you want to practice your skills in implementation of 'pointers'.


This documentation style is weird, you can refer to:
https://google-styleguide.googlecode.com/svn/trunk/cppguide.html


The thought was that there's not a major reason for the server to round off the value it sends back to an integer, or at least it should send back enough digits (at least several digits after the decimal) so that for all practical purposes the caller has the exact value. I don't care strongly here though. Elo ratings are actually even okay rounded off to an integer - that's already far more precision needed given their accuracy in predicting future game results.


Updated!  http://carltonom.s3-website-us-west-2.amazonaws.com/

On Sat, Jun 13, 2015 at 3:50 PM, Jeff Dean <notifications@github.com> wrote:

> This isn't *quite* done yet. The final requirements for this were to have
> a search bar on the index page so you are not hard-coding "pizza". Please
> add that feature, redeploy and comment here when you are done.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/amandacarlton/omdb-client/commit/97d9ae65fab49df14b7cbd06829aab6b18ded0fa#commitcomment-11668005>
> .
>


use "const Point&" instead.

You should know the reason.


> ここで定義されるべきものはサーバー側から呼び出されたプロシージャの引数のみで十分のはずです。
ここに追加している理由は、dummy-serverで使用するためです。methodそのものの有無は、parse_received_message()の中でチェックしているので、本来不要なものを追加しても、エラーにすべきものが許容されるなどの問題はありません。

> また、仮にプラグイン側でチェックするとしても、この4つだけである理由も分かりません
dummy-serverに必要なものを順次追加しています。たまたま、いまは４つです。


@alfonsojon
This doesn't work, the script just continues with its execution after wget pops up a small error in the command line.

```
Executing winetricks_early_wine regedit C:\windows\Temp\_sound=alsa\set-sound.reg
--2015-06-15 19:50:46--  http://download.cdn.mozilla.net/pub/mozilla.org/firefox/releases/latest-esr/win32/en-US/
Resolving download.cdn.mozilla.net (download.cdn.mozilla.net)... failed: Name or service not known.
wget: unable to resolve host address ‘download.cdn.mozilla.net’
chmod: cannot access ‘/tmp/Firefox-Setup/*.exe’: No such file or directory
wine: cannot find '/tmp/Firefox-Setup/*.exe'
```


Yes. But something that came up that was a bit frustrating when I've tried to write client code that would do smart things in the presence of lag, or in the absence of much lag but simply trying to sync with the server's idea of the time (with the existing gameserver) is that the existing gameserver does only send back the time in seconds.

This makes it incredibly hard to design a client display of the clock that won't occasionally flicker back and forth by a second with the server, or to have the client estimate and adjust its display based on the median round trip time, etc.

The fix is that the server should send back however much precision it has for all timestamps.


Considering that the frog is standing on the i-th ladder, it will try at most O(N) ladders to step onto. That is to say, though there will be at most N ladders in the stack, the total time cost could be as O(N!) since each ladder may be put into the stack several times. For example:

These're independent situations in your algo:
ladder#1 -> ladder#2 -> ladder#4
ladder#1 -> ladder#4 -> ladder#2
ladder#2 -> ladder#4 -> ladder#1
ladder#2 -> ladder#1 -> ladder#4
ladder#4 -> ladder#1 -> ladder#2
ladder#4 -> ladder#2 -> ladder#1


83811490-Fault_DirtyAirFilter (rongpengzhang) - i386-Windows-7-VisualStudio-12: OK (1675 of 1678 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)


83811490-Fault_DirtyAirFilter (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1675 of 1678 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)


should we need specify PUT or POST method?


LInk da ata da primeira Reunião


What you want to do is to use another array to keep the connectivities of all the ladders, right? It'll be perfect if you can skip those ladders which we've already know in-reachable from the target ladder, right? Please think about it carefully ;P


@juliaelman also noticed `pages/layout-system.md` and `pages/visual-style.md` are using dashes while everything in patterns is using underscores.


Hey Tin,

The calculation of the pms needs to happen before `self.params.negotiated.version = p[tls.TLSServerHello].version` otherwise the version number in the pms will be the server one instead of the client hello one.


What is `executeWith`? I've not seen that before.


This feels significantly better than what we had before.


That's what I was trying to do:
If the account name is NOT username/does NOT contain username, go ahead and create the account; otherwise we'd have ONLY "username123", "username124", "usernameee", "username_", etc.
But yeah, the example you provided works fine.


@thanhdut You can begin to fix this code, which is so very very close to what is needed, by dropping this state test. This test is going to cause the pinch movement to not be smooth as it will interrupt the gesture messages.


Paolo,

It looks like we're going to have to revert this commit because the LuaJIT FFI CDEF loader can't parse #if macros.   I was under the impression that the LuaJIT FFI understood "int64_t" and "uint64_t" on all platforms.

Jim


Try doing a `cabal build` before pushing. The euphony rules only support a single transformation, so you need to figure out what is the "real" rule here.


Nice trick defining this structure privately to get the information be need.  Unfortunately this is going to make the code unmaintainable and fragile in the long run because we'll be vulnerable to any upstream change in this structure.  So we can't do this.

What we can do is pass `-1` for the end in this case which `__filemap_fdatawrite_range()` will interpret as end-of-file.  This has the advantage of being safe and maintainable.  It _might_ cost us some latency when for some reason you know you only need to sync a small section of a file but that's definitely an unlikely case.


@thanhdut You can drop this test, and the preceding lines that set it up, as the pinch effects the entire toolbar/view not just one label on it.


haha


Hi, I finished implementing support of python3 in this fork, @pitbulk will merge this to the onelogin back. You can contact to him to get news about merge.


initialize


@thanhdut Pass in the recognizer, not just the scale of it. Adjust the parameter type in the method declaration and implementation.


Yeah, ill fix this, since it is just returning the StringData(s) I dont think we need any other than then return. Thanks for the catch


I've pushed a change to master replacing the gperf hash
lookup with a binary search.  If you have a chance, could
you have a look at what I've done and let me know if it
looks sane?  It shaves off > 100K from the binary, and
there's not a noticeable performance hit.

+++ Nick Wellnhofer [Jun 15 15 16:28 ]:
>   We should consider to use a binary search instead of a gperf hash for
>   entities. This would reduce the size of the binary considerably.
>
>   —
>   Reply to this email directly or [1]view it on GitHub.
>
>References
>
>   1. https://github.com/jgm/cmark/commit/cdcb65c5a209a1a3be4a1b9e804f5a4292127509#commitcomment-11692021


I just realized .. even though LuaJIT understands it, GCC may not on Linux.   It's probably better to simply define "int64_t" and "uint64_t" on the C side using the same ifdefs.  One of the generalized type includes on the C side should work.

I'm going on vacation this weekend, so things are going a little slower than hoped.  I'll try to patch that.


@thanhdut This line is spot on. You'll need to adjust to recognizer.scale


:heart:


Done : https://github.com/Stryzhevskyi/rangeSlider/releases/tag/0.3.4


A few comments about this:

* Let's call this ZFS_AC_KERNEL_VFS_RW_ITER to be consistent with the existing VFS_AC_KERNEL_VFS_ITERATE.
* Defining HAVE_VFS_RW_ITER would be more consistent and descriptive.
* While not strictly necessary checking for both read_iter and write_iter would be prudent.

I was thinking something a little more like this.

```
dnl #
dnl # Linux 4.1.x API
dnl #
AC_DEFUN([ZFS_AC_KERNEL_VFS_RW_ITER],
	[AC_MSG_CHECKING([whether fops->read/write_iter() are available])
	ZFS_LINUX_TRY_COMPILE([
		#include <linux/fs.h>

		ssize_t test_read(struct kiocb *kiocb, struct iov_iter *to)
		    { return 0; }
		ssize_t test_write(struct kiocb *kiocb, struct iov_iter *from)
		    { return 0; }

		static const struct file_operations
		    fops __attribute__ ((unused)) = {
			.read_iter = test_read,
			.write_iter = test_write,
		};
	],[
	],[
		AC_MSG_RESULT(yes)
		AC_DEFINE(HAVE_VFS_RW_ITER, 1,
		    [fops->read/write_iter() are available])
	],[
		AC_MSG_RESULT(no)
	])
])
```


@sofar yay or nay on this one?


@juliaelman I see this has separate pages for visual style, layout system, and patterns. We talked on slack about putting everything on a single index page and having a direct link to each collection individually.


Reigster?


I may have the change laying around in a branch (will look), but if memory serves it was pretty much identical to what you have.  I do recall having a version that just converted the exception to a string and used that for the stat name (so the exception message showed up as well), but that really, really blew up the stats.

Should have added a PR for it at the time :/


@thanhdut This is a test to keep the toolbar from growing too large. Nice. You might want to test for a minimum too.


я бы разделил на левый блок и правый:
```
<div class="left">
  <div class="b32 ">
    <p class="about-main">Мы — маленькая, но гордая дизайн-студия из Краснодара.</p>
    <p>Исповедуем принципы минимализма и чистоты. Ставим математический расчёт выше креатива. Работаем не покладая рук, как роботы.</p>
   </div>
   <p class="pre-ul">Выполняем заказы на разработку:</p>
   <ul class="mdashed">
      <li>Веб-сайтов любой сложности</li>
      <li>Мобильных приложений для iOS и Android</li>
      <li>Слайдшоу и видео для корпоративных презентаций</li>
   </ul>
</div>
<div class="right">
  <div class="b31">
    <img src="#/assets/logo@2x.png" alt="" class="logo">
    <span class="since">Since 2014</span>
  </div>
  <p class="pre-ul">Как мы работаем:</p>
  <ul class="mdashed">
    <li>С самотдачей 146%</li>
    <li>Соблюдая сроки на 100%</li>
    <li>По предоплате 50%</li>
  </ul>
</div>
```
Как то так


@kevva Could you publish a new version? The current latest version v2.1.1 still uses [`download`](https://github.com/kevva/download) v3 that depends on massive [`request`](https://github.com/request/request).


s/no/now/


Axel, any progress? I would also like to get this merged into js-console...


Здесь я бы сделал через списки, хотя можно и так оставить


`prop_type` is intentionally hidden from everything except `prop`. If a caller wants to write some custom code that doesn't involve `prop`, they could access the `detail` namespace. Is that not sufficient?


forgot a word: implemented html **cache** for fw.OptionsModal


Nop, it doesn't since 4k problem is in camerawrapper.  It just set the default 4k bitrate as it should be.


I commented these out in 71d41e2b00a877b2dfc3436525298b67f8cb9354


Is this where modifying mutable runtime configurations would live?


@thanhdut `self.awesomeToolbar.transform = resizeToolbar;`


@kaushik94 Just catching up on this now - it returns nan for me too. I'll try to help investigate.


Я не думаю что здесь нам нужны будут скрипты
Вставим обычный iframe с гугла


@thanhdut You were inches away. Very well done. You should consider this a success. I do!


Не стал бы оборачивать в nav
Здесь нет как таковой навигации, так как это скорее ссылки на сторонние ресурсы
Я бы оставил просто список


I'm trying to guess if the configure succeeded. Which file should I check
the presence of?


The point of the `detail` namespace is that its supposed to be private to the module in which it appears (jtx framework in this case). This prevents someone from writing a unit test that directly instantiates a `prop_type`.


right?


PR #1062.


Why not use the `dps` variable instead of `(damagePerClick * currentClickRate) + s().m_rgPlayerTechTree.dps`? Seems like `dps` and `dpsText` is unused.


@maya first, good catch on the filenames/folders! I've updated those to use hyphens instead of underscores and best for consistency.

Not entirely sure what you mean by `putting everything on a single index page` and what was determined in Slack. AFAIK, and as the wireframes outline, there are separate pages for visual style, layout system and patterns. Each of these pages will have the necessary IA that pertains to that category and be permalinked to as to easily reference each part of the library.

Also, I am not sure it makes much sense to smash everything into one template. We'd definitely have to do some load testing and efficiency to see if this makes the most sense. I honestly think that separating these sections out makes the most sense, especially in that regard.


@vanZeben Can you look at the positioning of the slave buttons & display in this branch? I suck with css. Otherwise the functionality should be working.


s/to in improper URI/to an improper URI/


Здесь нужно либо вынести h1 в header
Или сделать оболочку для container, иначе не получится сделать тянущийся фон


lala


Los ficheros en www/lib/* al ser dependencias `bower` no se suben con el proyecto. Todos estos archivos se pueden borrar


Faltaba declarar esta dependencia `bower`


По функционалу :)


Ack. Will change so that it queries the openstack backend for available versions. Very sensible way to address the problem, what was I thinking.


Hacemos que la dependencia apunte al sitio correcto


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Yes: a rebase gone wrong...


@juliaelman thanks for the file names fix!

Here is part of the conversation from Slack: https://18f.slack.com/archives/govwidepatternlibrary/p1434046716001311 I think @carodew mentioned a goal of having a URL for each individual pattern. When I mentioned we can do both with Jekyll collections it sounded like that was where we wanted to go. Is this something we should clarify with the UX team and @mollieru?


script/DATA CLEANING for network analysis.py

@dariusmehri where is this data? do you create it in another script? I'm reorganizing / cleaning up code and making sure scripts run from start to finish.


rax retry tests


El problema es que la ruta de `app.recipes` acaparaba todo. Había que crear unas rutas diferentes para categorías.


isn't this one no longer necessary?


Uhhh, it's still a github repo. I ported it from svn.

Kayla Iacovino
NSF Post-doctoral Research Fellow
US Geological Survey, Menlo Park, CA
Science Editor, TrekMovie.com <http://www.TrekMovie.com>
[image: Facebook] <http://www.facebook.com/kaylaiacovino>[image: Twitter]
<http://www.twitter.com/kaylai>[image: Blog RSS]
<http://www.kaylaiacovino.com/>

On Tue, Jun 16, 2015 at 11:41 AM, David Rynearson <notifications@github.com>
wrote:

> Hissssssssssss ...svn
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kaylai/ftir-background-subtract/commit/2761452af433a682c4d9c12db85be0953bf9aa14#commitcomment-11709516>
> .
>


Es necesario un nuevo servicio, pues genera otra url diferente con un tipo de respuesta diferente


В каждом блоке есть скрытый блок, который появляется при наведении, ты про него забыл
Посмотреть его можно в styleguide
Соотвественно плашку (с тремя точками) над картинкой нужно сделать отделным блоком
Этот блок будет изменяться при наведении


That's how it does it, yes. The default user name is changed but the default_user still have sudo, from cloud.cfg.


Here we should be using relative imports from the "data" folder so that it will work on everyone's computer.


Este nuevo servicio es el que devuelve recetas por categoría que tiene una url diferente y devuelve la respuesta en `array`. Lo ideal es que todos los endpoints de tu backend devolviera un objeto `JSON` con una estructura de datos predecible, por ejemplo:


Is the update to JUnitRunNotifierResultsListener.java indeed needed? I do not believe I've encountered situations where page results were incorrectly reported as 'successful' with the original code. In other words, what's the situation that `firstFailure == null` but the page/test was not successful?


I have it here because nothing else requires it. It looks like it could be used by something else but in reality it's just used by the OpenStack backend.


I think you still need to call the completion block here, otherwise it would never be called in certain cases.


@iambibhas looks like this throws an error when dependencies isn't defined. Testing with the query `npm colors`.

Something like `if (item.dependencies && item.dependencies.length)` might be better, or `DDG.getProperty`


@maya ah! You mean just for `/patterns/`, right? If so, the idea here to is be able to do something like `/patterns/#accordians`, `patterns/#buttons`, etc. It would still be a long scroll-y page and be able to be reached in this way.

Let me know if that makes sense or not, or you'd like me to walk you through it!


Этот блок будем делать в последнюю очередь
Про него буду рассказывать на вебинаре


actualizar las urls de los menús para que accedan a los nuevos `states`


this the the name of an OpenStack cluster. http://horizon.the.re/ is my personal OpenStack cluster. We'll have redhat: and suse: here as well, once it's tested with them.


It didn't compile without it. I'm not sure why.


Наверное у p так же должен быть класс filter__title


Buen trabajo!
:+1:


What's going on here? Is this right?


I thought this version was using constraints to do all of this....

On 16 June 2015 at 12:04, wes johnston <notifications@github.com> wrote:

> I find these insets a little hard to understand (since they control both
> the left and right insets, which are different in some cases for us). We
> need to adjust the lock insets a little more (i.e. they still need some
> padding so that text doesn't run into the lock). Doing so also actually
> helps offset the reader mode button if its showing.
> https://dl.dropboxusercontent.com/u/72157/paddings.png
>
> Unfortunately, the seem to also apply to the editing rect:
> https://www.dropbox.com/s/knajrj66qr2djbu/editing.png?dl=0
>
> I played with fixing this a bit, but I actually wonder if we'd just be
> better off using constraints to do all this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mozilla/firefox-ios/commit/210d9eb0833a5396fc40a7a05c70e0c287cd6002#commitcomment-11710039>
> .
>


This revision is clean.


I did not helped to write the NES Emulator , but I wanted to write a cool GUI in C ++ .
 I asked you que couse i have little experience with emulation and I want to learn and pratice it.
thanks for replying


Maybe we should flip the conditional around here and use `_.isString(validator)` in order to do the lookup, and otherwise take whatever we're handed and look for the conditional logic?


Fetching the points while the controller is alive..


That's great :) cool theme by the way! I specially like it's logic


Cancelling the interval when the controller / scope will be destroyed...


Should be 'Less than ' + CONTROL.wormholeEndMinutes + ' minutes for game to end.'


LOL


There's nothing a horrible Vimscript one-liner can't fix anyway (0e42cf3).


Hehe, alag hi machane lag gye aap :D
On Jun 16, 2015 11:57 AM, "Prateek Chandan" <notifications@github.com>
wrote:

> much wow */\* such comment */\* [image: :wink:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stabiitb/mainsite/commit/246eb5dcac65fed40d1c8bae4d3e76502c67cd11#commitcomment-11695422>
> .
>


@mitio can someone please try to login via Google who actually has an activated Google Account.

I get an error :( But I also see that I am not in the social_auth_usersocialauth table:
mysql> select * from social_auth_usersocialauth where uid='sclement@gmail.com';
Empty set (0.00 sec)

Meaning I never logged in before, and meaning that I cannot create new accounts.

I find no real log messages but I keep digging.

With a bit of chance, users who were registered previously can now log in… Maybe.


Use scientific notation here: `6.2e6 == 6.2 * pow(10, 6)`. let me know if this is unclear.


Line 137: caused expressionText to be blank, making Goto not work at all.
Line 23: I assumed the thread destructor wasn't being called properly without a parent object.

Overriding start() could've been left out I guess.


Oh dear! Sorry, will fix - thanks for the catch.


Or you know, `return (level % CONTROL.wormholeRounds) === 0;`


averting my eyes here...


Good to go!


When running the integration tests, all runs from the current directory (worker, suite, run etc.). Having the fetch etc. logic with separate directory makes things complicated when developping. With this I do:

* modify code
* tox -e openstack-integration

and wait for the result. I don't want extra steps.


It's possible that that would work, though I don't know enough about the architecture to know how easy it would be to get the views/controllers from a model id. It also assumes that we can re-create an object with the same id, and I don't know if SC lets you do that (if we're talking about SC ids).

Also, some actions work on "child models" of the model that was closed -- e.g. most of the graph gear-menu actions only have a reference to a plot model, which is owned by the graph model. Even if the graph model's id is restored on re-creation, I don't know whether the new plot model that gets created would also get the same id.

This issue of child models could be fixed by requiring undoable actions to only retain references to the closable model's id, so every time an action wanted to act on a plot model, it would look up the graph model by id, and then get the graph's plot model. I don't think there's currently a way to get the parent graph model from a plot model, but that could easily be added.


just a quick but big thank you (!) for the work you put into convincing cxf-3 and grails-3 to play together recently :) your effort here is really _much_ appreciated.


weird, 137 worked for me :)

anyways hopefully this fixes things


got it !


The index thing should take two arguments:

1: Either a slice/array or a map
2: An index (if array) or a key

It returns one element in the given collection.


uhm, do we really want to go this route? patching configure is guaranteed to never be upstreamable


Published and added you as owner.


@JonnyH Alias support is now available


@vcunat Agreed, although I think you should try and get xorgserver 1.17.1
out. I'm actually having issues with Intel's xf86 driver crashing on 1.16.4
and 1.17.1 on staging.

On Tue, Jun 16, 2015, 12:35 Vladimír Čunát <notifications@github.com> wrote:

> We really should restrict new major updates at least since now, or we have
> no chance of releasing this month.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/370fc79b948438694dc4767570afe43c5b1fb7c4#commitcomment-11710659>
> .
>


I would definitely recommend adding a control structure and turning on the progress flags while debugging. See, for example https://github.com/elemental/Elemental/blob/fa23f9a7a6a165db0884cd0e4ae5863c39d8c867/examples/interface/LAV.py


I don't understand when we should give a value to channel and which value we should give it. actually i don't understand what $channel variable do


Really?


No, that stuff is OK then. And according to CircleCI, this merge was OK too.


Indentation for the lines above is wrong


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


You need to doc that this is only valid when reading the value from the kernel - you are not required to specify all fields when creating labels elsewhere


Your errors should be lower cased


By the way, I think you *are* already storing the view in memory, even if it might not be something that can just be opened. Doesn't the reference to `tComponentView` in the undo function create a closure that keeps the object? For example, if you put a debugger statement as the first line in the undo function, you can see that tComponentView has a value.

If we don't want to explicitly save the view, perhaps that should be changed so as not to take up memory?


I was under the impression category and label are distinct sections.


Why define this twice, instead of putting it in theano.compat?


I took @HowardHinnant's suggestion and moved the special members to reduce confusion.


do we need to check that pool name is not empty?


I see how it is rather convenient to not add the patch in our own SCM, but I think it is better to be consistent when possible. If we start have some patches in our SCM and some "out there", it becomes harder to quickly see how much patching we are doing to the stuff we build.


Nope.  Technically the 'label' encompasses the MLS level and MCS category labels, iiuc.


typo?


Have you defined what unsupported means?


It's only used in tests, which is why I did it there.  I can move it though.


There's a bug here.


Good idea, it's cleaner this way.


Not as described here https://github.com/openshift/origin/pull/2544#issue-81635485

> On Jun 16, 2015, at 4:33 PM, Paul Morie <notifications@github.com> wrote:
>
> Nope. Technically the 'label' encompasses the MLS level and MCS category labels, iiuc.
>
> —
> Reply to this email directly or view it on GitHub.
>


With the channel variable you can manage several "channels" or "rooms" at the same time. Users can post messages in one channel or another, but the messages are kept seperate. You could use it for different languages, topics, etc. If you dont want to use it, just leave it to "default".


might want to add a 'self-uninstalling' bit here.  Not sure how TelEx handles uninstall.


why do you wrap the statement in brackets?


Comment updated.


Add newline


indentaion issue here maybe (probably issue in source code copied in)


a method comment would be nice here


Indentation fixed.


It lacks a space before "const" here (it's maybe not mandatory to work, but that seems to be the convention ;)).


Nice cleanup :)


Wrapping fixed.


@vronvali normally i thought that yo angular makes a default .travis.yml for you? I just ran `yo angular` and saw that indeed, there is no travis :(


Excess braces removed. (I think I got all of them.)


do it


Guard added. Braces cleaned up.


do we need this?


You can grab it here right now https://github.com/Razzeee/script.trakt/tree/EditControlTest?files=1


should start at version 1


can you remove spaces around `@`?
the lines seem rather long. how about breaking into several lines, like:

```smalltalk
aCanvas fillRectangle: (...)
    color: Color transparent.
```
but i am not sure about this one.


version 1 here


dont wrap in brackets


this is so freakin kool.


wow! I can comment on certain line. This was mistake that I have fixed later.


I'm guessing this is the cause of the dart failure.


faggot


SAME


A backdrop of any  poet's  colorfully historical  period of  inspiring artistic minds given the opportunity to broaden their knowledge of the githib librato world highly innovative in the field of social coding.


This line that killed 1.3.5... R.I.P ;)


:sweat:


maybe we can have a better short description ? And if i'm right, this is not a 'Luvit' project anymore, but a Luvi's one.
Very nice and clean example: +1


I just added the self for coding standards. refactoring this method would be another task I think.


Profesor, el programa tiene un error en el indice cuando se elimina un elemento. El numero de los elementos no se modifica y mantienen el anterior, lo cual crea confucion a la hora de eliminar un 2do elemento.