mkdir_p is a no-op on an existing directory, no need for the extra check


Good to know I'll change this back.


Consider isolate scope `=`.


@jmazzitelli ATM, this is implemented as `GraphListener`. Do you find this good enough? Or we maybe want our own dedicated `ResourceEnhancer` listener interface that could be registered on {{ResourceManager}}s? - Something like the following?
```
public interface ResourceEnhancer {
    void enhance(Resource r);
}
```


`width` attribute doesn't take a pixel value (just a number, in this case it would be 60). Consider using actual CSS for this, however.


Thanks, @eric-wieser! The course catalog's HTML is pretty ugly, so this seemed like the best way to get it in better shape for Nokogiri to parse.


> USE DESCRIPTIVE COMMIT MESSAGES!


You may consider adding some links: [Veewee](https://github.com/jedi4ever/veewee), [Vagrant](http://vagrantup.com/) and [Pennyworth](https://github.com/SUSE/pennyworth).


I am free, will test it, no prob.


This is OK, but we might want to think about reducing the scope of these tests in the future. If all we're testing is that forms are processed properly, we could just do a form test instead of having to spin up the entire client. However, if there are additional things in the view, it would be more work to abstract these. We could talk about it after the break.


This includes non-standard whitespace (4-spaces indentation is standard), also in other locations


This doesn't work because `SlackNotConnected` is not actually thrown by the `SlackClient` because of a bug with its sockets.


93469234-DemandResponse2 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1683 of 1791 tests passed)

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


Hi @jk, believe me, we are aware that we did not precisely follow semver with this change. We discussed this quite heavily in #593 and decided it was best for the community overall to make the change allowing better support for response serializer generics. We also documented the change in the release [notes](https://github.com/Alamofire/Alamofire/releases/tag/1.3.0).

We do everything we can to consider all the best options for everyone using Alamofire. There are always going to be tradeoffs, and 99.9% of the time we will ALWAYS follow semver.


`getChildren`


This is because we need corresponding get/set methods working, we can't override them via property so we have to implement it in old manner


French speakers will be pleased to loose their favourite app that informs them about late ("en retard") trains https://github.com/GeminiCode/EncoreEnRetard

they will be even more pleased that they can't even use the word for being late in french.


When the `kodo_trace_callback_t` function is invoked by the library, it will receive the user provided `context` pointer. This allows the user to pass custom information to the callback function. If no context is needed the pointer can be set to NULL.


Please extract "See Also" to a constant.


Why is there a user ID attached as a parameter to this state? Would it not be reasonable to assume that the cart you are displaying belongs to the logged in user?


The stuff about background matching really doesn't belong in `AssembleCoaddTask`, but in some background matching Task.  But maybe leave that for a future reorganisation?


Authenticated state?


This should honor the platformStatic annotation.


Okay, that will come. I modified this comment to ```# TODO convert me to grid - Update when PyArt updates this interface```


Try using `.reduce`.


I am adding the address only as a property. Should I perhaps rather use `addConfigurationProperty()`?


Just chiming in now that I'm back from hols. First: thanks all for
getting to an agreeable conclusion!

On 31/07/2015 11:25 , Andreas Tolfsen wrote:
> The fundamental issue here appears to be the difference in view on
> whether Respec is a preprocessor that transforms a non-HTML,
> Respec-flavoured language into a valid HTML document, or a tool that
> extends the browser processor to enhance the presentation of an HTML
> document.
>
> Bikeshed, with its |.bs| file extensions certainly falls in the first
> category. I was always under the impression that Respec fell into the
> other. Whether an explicitly announced feature or not, this change to
> Respec breaks this assumption.

I just wanted to note that this is interesting. My expectation was
always that the ReSpec source could be as invalid as it wanted to be
since it was a pre-processor; and in fact the initial Perl version did
contain a fair bit of invalid markup (like Bikeshed, only even more).

As a result, I was very surprised to learn that people were actually
validating the ReSpec source. That the ReSpec source be valid HTML was
*never* a design consideration (and in that @halindrome's initial
response was completely in line with the thinking to date).

But hey, it does indeed turn out that ReSpec sources became HTML5 valid
as of 2009 and have somehow miraculously remained so to this day. It's
an unintentional feature, people are relying on it (with good
arguments), so I say: let's enshrine it as a principle :)

--
Robin Berjon - http://berjon.com/ - @robinberjon


Fix formatting.


LSST may prefer the use of "coaddTempExp" instead of "warp" (though I very much disagree).


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

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


why `window.swal` and not `swal` ? :confused:


Could also use lodash.pluck here.


This type mapping should be performed in JavaLanguageService.


I'm not sure what you are referring to. Could you send a PR to fix it against beta? That will help to understand the required change.


Code coverage is handled by [`test-transformer`](https://github.com/jstransformers/test-jstransformer):
https://github.com/jstransformers/test-jstransformer#usage


Ok, I see the issue - fixing it in a sec.


First off, this is experimental to see if we and others like it.
The ```_update_infolabel``` will update when upon tilt, field or file change. I don't think these are all strong changes though right (e.g. tilt)? This was added to address @tjlang comment.

I originally was going to do status bar, but I wanted to try a small info box at top for quick reference. We could then get rid of the print out to terminal window.


WAT


@pilhuhn so ATM I am just getting a set of IP addresses for the hostname which works, but does not guarantee that the given AS is really bound to the given interface. This could be checked using DMR and I am going to do that unless you stop me :)


One ! is enough.


is there a particular reason to do 3 separate loops here? just wondering


в этом коде ошибка:

procedure TgdEmailMessage.ResponseMsg;
var
  Msg: String;
  LP: Integer;
begin
  if ErrorMsg > '' then
  begin
    Msg := ErrorMsg;
    LP := 0
  end
  else
  begin
    Msg := 'Done';
    LP := 1
  end;

  if AutoTaskKey > 0 then
    PostThreadMessage(ThreadID,
      WM_GD_FINISH_SEND_EMAIL,
      AutoTaskKey,
      Integer(Pointer(PChar(Msg)))
      )
  else if Handle > 0 then
    PostMessage(Handle,
      WM_GD_FINISH_SEND_EMAIL,
      LP,
      Integer(Pointer(PChar(Msg)))
      );
end;

объяснение тут:

http://stackoverflow.com/questions/1325270/how-to-send-a-string-via-postmessage


So TickUntil jumpts to a location n in the animation and gives me the info if the audio was triggered while doing so. Additionally it gives me the starting time. (== 0). Once i have that i can determine to what audio location i need to jump - thats the call mentioned above.

So i.e. for an animation with: Rampup: 2s, Animation Duration: 4s
The valid range for JumpTo would be 0-6 seconds then.
JumpTo(1) would not trigger any audio
JumpTo(2) would trigger audio at audio playing position 0
JumpTo(3) would trigger audio at audio playing position 1

But i think there was a missing consideration of the ramp up time...it should have been:
		const float AudioPosition = Position + CurrentAnimStart;
		checkf(AudioPosition >= 0.F, TEXT("Invalid audio playback range."));
		PlayAudio(AudioPosition);

Whenever audio was started. CurrentAnimStart would be the negative ramp up time and Position the time to jump to.


Just noticed that your default settings deviate form the hard coded values you chose earlier...

You probably wanted `60 minutes` or more...


This line would have cause a fatal error so OK to move without deprecation message.


Explain what's going on here.


Does this need to be here? If so, can we use logging instead?


There is a `currency` filter in Angular.


You and your blood magic see everything.. https://github.com/opf/openproject/pull/3277/files#diff-646ad6b10917e4385cbcc8c8524e24a2R271 Set them down for testing + pushed too fast..


Mostly stylistic. It has minimal perf hit (pretty much none), but it's easier to reason about when reading the code.


surely it should not be there, I thought I removed all of these but apparently I did not, thanks @triceo will remove it immediately


`ng-src`


It's like context matters or something.

On Tue, Aug 4, 2015 at 10:37 AM, Seegras <notifications@github.com> wrote:

> French speakers will be pleased to loose their favourite app that informs
> them about late ("en retard") trains
> https://github.com/GeminiCode/EncoreEnRetard
>
> they will be even more pleased that they can't even use the word for being
> late in french.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12519851>
> .
>

--
Kurtis Rainbolt-Greene, Hacker
Software Developer
1631 8th St.
New Orleans, LA
70115


Fixed in 4cf51be.


Pourquoi t'as viré vendor du gitignore ?


Using these star characters will certainly not be cross-browser -- but you're not worried about that so it's cool.


fair enough :-)

I just wondered if it *had to* be ran in bulks like that


@Tobschinski ，nice！


Where is `deleteUser`?


ack


This is sad since Chef is still releasing minor version bumps for 11. Isn't Rightscale a partner? They are only as far as Chef 11.6.


I got ahead of myself. In all doc-strings I changed this to ```radius of influence```. I think being explicit in documentation will be important since we shorthanded ROI elsewhere for "region of interest" also.


Credit where credit is due:
http://unix.stackexchange.com/questions/5229/add-arguments-from-previous-command-to-zsh-completion
I'm not positive leaving it here in my zshrc is sourcing it correctly -- in today's tests, when I run this line manually the functionality is added correctly, but it doesn't work when the line is sourced from configs automatically. Is yours working smoothly by default?


Very good. But the package is still in the latest master branch along with
other packages that should be removed as well:

                 com.google.android.apps.maps|    arm| 16|         213-240|
                9.12.1|  912100102

                 com.google.android.apps.plus|    arm| 15|             240|
        6.0.0.99065591|  414118881

      com.google.android.googlequicksearchbox|    arm| 16|           nodpi|
         4.9.22.16.arm|  300413916

2015-08-04 16:45 GMT+02:00 mfonville <notifications@github.com>:

> I now added a quick check to add_sourceapp.sh that should prevent issues
> like this in the future:
> opengapps/opengapps@094c4c0
> <https://github.com/opengapps/opengapps/commit/094c4c0bb3abe9437130ec9a113c4b32dc79542b>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opengapps/arm/commit/f20648dee99f7b2fa832d91979dc2db06ea101a9#commitcomment-12518472>
> .
>


Currency filter.


Ah, I'd have to check the logic, but if I remember right, I don't think that they are dependent on any particular order.


Not necessary, but a helpful tool: https://docs.angularjs.org/api/ng/directive/ngPluralize


Is the last `.filter()` needed? I.e., is it possible to have a non-purged HDA associated to a purged dataset?


REVERT THAT AND TRY


Reverte esse commit, está apontando pra suas badges não do devmatogrosso.


In the end of conversation, is possible to happen an error to generate this data msg?

Errors happen when:
- Need to generate an instance tag if we don't have one
- TheirKeyId/OurKeyId == 0 or not found


Erro ao carregaro registro -> Erro ao carregar "o" registro


Looks good (shouldn't it be MockComparator**And**CopierTest though ;-) hehe.


@yole do you think it should be moved to the service?


This is still WIP :-)


Why did we comment this out ? We do need to update the user cheese count


invode -> invoke


lgtm


Pourquoi use_listener: false ?
Pour le menu, si tu veux gérer ça comme un pro de symfony, il y a le https://github.com/KnpLabs/KnpMenuBundle qui fait référence. Dans tous les cas, renvoyer un string depuis le controller est une mauvaise pratique qui viole le contrat du MVC. Je suis dispo pour toute demande de renseignement complémentaire.


if you are going to put properties in here - it's good to make sure they match what the plugin will use so you don;t confuse people.  `findbugs.failOnError`


Looks like we can remove a lot of duplication by creating a single function that maps a DocumentationNode to a Doc, and using that together with Kotlin's `as` operator in all the asWhatever() functions.


This is a strange thing to do. The `$scope.cart` being referred to isn't the one attached to this ConfirmCtrl scope, but the one from your CartCtrl ... I would maybe attach the function to the cart object itself or repeat the logic (which isn't anything more than summing).


Default boilerplate for CFN templates


Protected should be visible enough and I don't understand at first glance why the `$codes` parameter is optional.


Move this to a parameter since it gets used in a lot of resources


`blockBelow instanceof BlockLog`?


typo: s/interator/iterator/


I don't think this is a good place for an `autoreleasepool` since no temporary variables are being created here that we need to release immediately. From what I've seen in Apple's guides, `autoreleasepool` is only really going to be useful if you're pushing onto another thread, or creating lots of temporary variables, like in a for loop.

I'm basing my thoughts on it from this article: https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/MemoryMgmt/Articles/mmAutoreleasePools.html


Will you still be working on this project? It is really great and promising.


Use for tagging resources


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


Need to make sure this name is unique (i.e. the user can define a parameter explicitly named "receiver", and then this parameter needs to get a different autogenerated name).


No, it is parameter for form (show ip select or not)


We will very soon have the ability to annotate the receiver parameter as well.


I put these in nearly all templates and use lookups to avoid typos when doing it manually.


reduce


J'ai pas eu le temps de tester, mais il se dégage un douce impression de travail à la lecture des diffs de tes commits.


@flavour please review
Some issues are:
1. When I click the 'Publish Alert' for a particular alert_id and run the run_scheduler_tasks (https://github.com/flavour/eden/blob/master/static/scripts/tools/run_scheduler_tasks.py) all the alert_id's issued_on field is modified...Should it not only affect that particular alert_id as it is called for?

2. When I add new info to existing alert, it doesn't notify me when I run the scheduler_task (python web2py.py -K eden). How can I notify if the component(cap/info; cap/area) of a resource (cap/alert) is modified/added. Should not resource options in the filter do this?


This would generate an error, because you can't refer to a resource in its own properties.


Different from the other instantiation.


why do you suggest $? in other words, I understand why $, but my system has ~ as shell welcome message, so dollar sign make it harder to copy paste suggested command


thanks!


This `kind` check doesn't seem to be correct.


@yole do you think we can implement all interfaces by one class? It would be dangerous btw


Since this is inside a VPC, all the security groups need to reference it, as they are VPC-specific.


Please remove this line.


On one hand, I don't know why we would ever want that behavior, on the other hand, someone may be relying on it.


I added quotes around all the port declarations, as suggested in CFN documentation. I recall getting errors at one point when they weren't quoted.


Adding name and project code tags to all taggable resources.


Confirmed - on PROD, log file at '/subscriptions/logs/subscriptions-frontend.log' is now populated.


@nursix please review


Please use DokkaLogger here.


The JS bundle builds fine but throws a runtime error in production. The output is difficult to debug as it's minified but there is a method being called with string containing `raven`, if that helps.


yes but everyone uses SoomlaProfile to login. I think it's just adding unnecessary code...


@Zarel pls


Just a hinting that it's a shell command.


Yea in fact that's the only effect it has.

Sent from my iPhone

> On Aug 4, 2015, at 11:26 AM, Andrew Mao <notifications@github.com> wrote:
>
> Does this also update the "When about X players have joined..." in the lobby?
>
> —
> Reply to this email directly or view it on GitHub.
>


Solid color tiles will optionally be hashed slightly differently (based on tile location in VRAM) so that they can have multiple replacements if a game permits it. Still TODO at the moment, but that should solve the problem GBE faced in the past without being too complicated :)


Why are name() and kind() different here?


Ping @jkuchar, I tried to use the test<method name>_given<input>[_when<condition>]_then<output> naming, as you've agreed on with @trebi. Please look if this is okay.


How come this service knows anything about sie `krakensten`?


Why are we using Kotlin syntax in JavaDoc documentation?


This is probably done more easily as a checkbox, but I see what you did here.


It's the default prompt symbol I believe. Read more here http://superuser.com/questions/57575/what-is-the-origin-of-the-unix-dollar-prompt.


Hey, MegaApuTurkUltra, I would like to talk to you, please email me @ arinesau@gmail.com. By the way, I am Arinerron on Scratch and OpenSprites. :) Thanks!


are you able to recreate it locally by running `NODE_ENV=production webpack --bail; make run`?


I think your example should work.
The only potential problem is locating the file specified in the stylesheet string. The base URI would be where this file is run from.

You can however set the current working directory in Saxon/C either on the SaxonProcessor using the method setcwd(const char *) or using the constructor method of XsltProcessor(proc, false, cwd).

> On 4 Aug 2015, at 16:56, Sean McIlvenna <notifications@github.com <mailto:notifications@github.com>> wrote:
>
> I'm not sure that this captures the intent of my question from #2 <https://github.com/rimmartin/saxon-node/issues/2>
> This demonstrates compiling the xslt directly from memory, but what I'm wondering about is when the xslt references other files from the file system. A unit test to better describe what I'm talking about would be:
>
>     it("should use direct xml file and an in memory xslt", function*() {
>         try
>         {
>             // baz.xsl references another transform file, intercept the I/O request for it and return it from memory
>             saxonProcessor.resolveFile = function(fileName) {
>                 if (fileName == "myReferencedFile.xsl") { return new Buffer("XSLT_CONTENT_HERE"); }
>             };
>
>             var xsltProcessor = saxonProcessor.newTransformer();
>             var xslt = yield fs.readFile("./test/examples/xml/baz.xsl", "utf8");
>             xsltProcessor.compileString(xslt);
>             var pdbContent = xsltProcessor.xsltApplyStylesheet("./test/examples/xml/foo.xml");
>             pdbContent.toString().should.equal("<?xml version=\"1.0\" encoding=\"UTF-8\"?><out><?xml-stylesheet type=\"text/xsl\" href=\"../xsl/foo.xsl\" title=\"default-stylesheet\"?><foo:document xmlns:foo=\"http://apache.org/foo\ <http://apache.org/foo/>" xmlns:bar=\"http://apache.org/bar\ <http://apache.org/bar/>" file-name=\"test\" file-path=\"work\" creation-date=\"971255692078\">\n<bar:element>MyBar</bar:element>\n</foo:document></out>");
>         }
>         catch (err) {
>         console.dir(err.message);
>         }
>     });
> —
> Reply to this email directly or view it on GitHub <https://github.com/rimmartin/saxon-node/commit/bb28041a9c4d77c51db2d920b027a7ec132cbb11#commitcomment-12520311>.
>


chandni, can you refactor this properly (once added to the parent pom, it will be inherited by child poms) and create a pull request on `build-changes` branch?


Please move this function inside `buildInlineTags` and get rid of all the parameters that could be taken from the closure.


<img width="682" alt="screen shot 2015-08-04 at 12 00 49 pm" src="https://cloud.githubusercontent.com/assets/40650/9065640/e81e4452-3aa0-11e5-8c23-eddc60c3e074.png">


Can we prefix .credits with .StoryTiles


`user`, not `data`.


I agree.  It was just for testing to see if it made any difference/improvement in iAP reconnection behavior, which it doesn't.


Can we preplace <div /> with <aside /> please?


Oh, yeah, because I started with adding the config to krakenstein, then copied that over here. I'm just going to remove the values.


Yes


I've backed this stanza out in a later commit - it is unnecessary once all the other fixes are in place.


Haha, should have looked into git, I used this code as a template for sending an SNMP trap from EOS  - I've just spent half an afternoon with our monitoring tools engineer trying to trace why his tools wouldn't swallow the traps, before I decided to truncate it to int.

What's funny is that this works fine with snmptrap if you use v1, only causes problems with v2c (possibly 3)


There are 3 possible spawn-points for Dishu, maybe I'll get them later ... or maybe not.


@mc1100, I think if you do a "git status" on the source/arm directory, you will find that those files are not tracked.  If you remove them from your filesystem, then ./report_sources.sh will stop reporting them.  I made the same mistake...


No, that's not what I'm saying.
`fun DocumentationNode.asDoc() : Doc = when(kind) {
    DocumentationNode.Kind.Class -> ClassDocumentationNodeAdapter(module, this)
   // and so on
}
`


This menu could be generated in a more automated way.


Consider CableSwig.  That may have been what you meant by Swig.  It is what Kitware uses to wrap VTK and its other libraries.


Why not just jump to 1.6 (#472) at once? I'm asking because I got one compilation error on Core due to `MBeanServerFactory.findMBeanServer` (JDK 1.5), called at [this line](https://github.com/psi-probe/psi-probe/blob/master/core/src/main/java/com/googlecode/psiprobe/beans/JBossResourceResolverBean.java#L47), returns Object instead of `MBeanServer`.


Yeah, that sounds better


I just remember it from math or physics classes, like v = dx/dt. No one would write velocity = distanceTraveled / timeElapsed on the black board.


> I am starting to think that this is a CMake generator bug

After looking at the source code I think that it will be quite tricky to use such approach. CMake set "Header Search Path" internally (without using `CMAKE_XCODE_ATTRIBUTE_HEADER_SEARCH_PATH`). You can build an empty project and see that it's already set to `/.../Contents/Developer/Toolchains/.../usr/include` which is part of `CMAKE_CXX_IMPLICIT_INCLUDE_DIRECTORIES`. Just one more benefit of merging headers idea.

> If you think they can be of any benefit

No I don't think they have any since frameworks are OSX specific and we are building cross-platform tool :)


:red_circle: There are a few modules that should be in the `dependencies` list. They are: `bower`, `mongoose`, `method-override`, `http-server`, `body-parser`, `passport` and `passport-local`. These are required for the application to run.

Packages in the 'dependencies` list are required packages. Packages in the `devDependencies' are not required for the application to run. You can read more about it [dependencies vs devDependencies](http://stackoverflow.com/questions/18875674/whats-the-difference-between-dependencies-devdependencies-and-peerdependencies)


I show some errors in the MMQ version of this as well.


Closes #3


:yellow_heart: You do not need to specify the file type (`.js`).


what are these keys? if someone clones this repository, do these stay the same?


I have done the changes


@yole because kind value is old-fashioned javadoc tool enum that not always equal to name


ako registrovaný uzivatel s tým ze som tam aj prihlas udaje zadal a konci to chybou ...

> ** Tue, 04 Aug 2015 00:39:18 -0700 - "feixm1/fastshare.host" <reply@reply.github.com>, "feixm1/fastshare.host" <fastshare.host@noreply.github.com> **
>
> Zkoušíte to jako registrovaný nebo neregistrovaný uživatel Fastsharu?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/feixm1/fastshare.host/commit/a2f38f103bf5d0e2c84253c471c3b32bd86a5c2a#commitcomment-12509688


ths commit is useless. This branch is not the master branch, so the ``dev-master`` alias will not be used at all here


:yellow_heart: The structure for the `test` directory should be defined like this:

`test`
 - `unit`
  - `user` This will contain tests for the user model.
 - `e2e`
  - `core` This will contain tests for the core of the application. An example would be the `server-spec.js` file.


@GantMan is it really though?  I still have an open issue for it, and I can see this crashing bug using a 4s
http://hipbyte.myjetbrains.com/youtrack/issue/RM-904


:yellow_heart: You may need to indent all the code in the `it` block by one tab to keep the code neat.


The cert isn't, no.  The UpgradeRequest is a shallow copy of the original request used when setting up the websockets session, so we can ask things like `isSecure` but not 'what was the peer certificate'.  http://download.eclipse.org/jetty/9.2.12.v20150709/apidocs/org/eclipse/jetty/websocket/api/UpgradeRequest.html


@nonyibe: Can you add .idea to the gitignore file?


Oh sure, nice!


they said it would, and the tests passed, so I assume so.


xfail still runs the test, I think it should be skip


:green_heart: Good work defining a configuration file for `jasmine`.


Hum... in fact this still occurs if JDK is changed to 1.6. The problem is with jmxri. Does anyone change that recently?


@nonyibe: Extraneous echo.


interesting, yeah I have a production app that crashes using that exact line with latest RM - so I changed up how i built the color for now *shrug*


Yea, in that context it works.
I'm one of those that can't begin to read any of GSdx's code though.
As an example: `GSVector4 dt = dv.t / dv.p.xyxy();`
That's just random letters at this point :p


granted my crash was a 4s (also 32 bit) but like you said, they pass - so my concern level is lower :)


I really like this part. I think it's very clear, coherent and it gives a good idea of the subjects to come.


:yellow_heart: This config file will have to be improved to handle configuration for such things as database connections and API keys.


@ttuegel: I'm on yesterday's master + `hplipWithPlugin` and can print just fine.

What breaking changes are we talking about here?


:yellow_heart: The name of the application is `hairven`.


Looks good.


and mention moon


I believe these are the public keys for these servers? I pulled them straight from node's docker. Keep them.


The ce3f5fdfa4510a24365a1da5a0b09849276ef4c9 commit changed the for loop at `JBossResourceResolverBean`. Is It compiling for you?


ES6 WIN


Bad docs


@flavour @nursix please review...corrected previous issue...
One question that I have is does the subscription also looks in the component addition/edit of the resource?


Lumance?


Do you want me to change inverse_power, inverse_squared to InversePower ... etc? I was going off of the previously existing links for naming convention.


PCL-based printers (the plugin). I'm not sure exactly what's wrong, and I don't have time to debug this problem, that's why I removed myself.

I had previously tried updating the HPLIP version of a *working* configuration and found that the plugin didn't work with 3.15.7, so part of the problem is the version bump. But, I also tried reverting the version and hashes of the current expression only, and I still couldn't print, so it's not *just* the version bump.

Binary blobs can be problematic, but this one is a statically linked PS-to-PCL translator so it should be easy. I agree there is little way to test most of this package, short of acquiring a farm of HP printers.


This needs to go on a branch of its own. Will like to see that this gets done in backward compatible fashion. There may be a hope since it looks like the change is not in API but in engine.

To be discussed.


Here, you are trusting the order of the items in the array to achieve a mapping between a name and its matching `open` function. It could be turned into something both more abstract and robust using a hash:

``` js
var buttons = {
  character: {
    name: "Character", // as displayed on the UI
    open: characterMenu
  },
  inventory: {
    name: "Items",
    open: inventoryMenu
  },
  // …
};
```

You may either pass only the name along:

``` js
// The following piece of code could be wrapped inside a createMenuItems function.
Object.keys(buttons).forEach(function(buttonName) {
  $scope.menuItems.push(createMenuItem(buttonName);
});

function createMenuItem(name) {
  // get the button's details
  var buttonRef = buttons[name];
};
```

Or you may pass the full object along:

``` js
Object.keys(buttons).forEach(function(buttonName, buttonRef) {
  $scope.menuItems.push(createMenuItem(buttonName, buttonRef);
});

function createMenuItem(name, details) {
  // all data already accessible
};
```

Both ways, adding or removing a button is just a matter of editing the `buttons` hash, with no specific order required (easier maintenance, less prone to bugs).

---

Once again, because it's starting to look like the buttons have some internal state (a name, a "constructor" function…), they could also be implemented as their own constructor functions and used like this:

``` js
var buttons = {
  character: new CharacterBtn(),
  inventory: new InventoryBtn(),
  // …
};
```

Each constructor would define a `this.name` value, and may return a public API (for instance a getter for the name and some `create`/`init` function).

It's not necessary at this stage, I guess, though.


Не забывайте const ставить параметрам методов, которые не меняются в теле метода. Сижу чищу код, как будто делать больше нечего.


It is far more effective to base your template logic on things attached to your scope, and not attach to your scope things that represent template logic. This should be something like `error`.


Hey, that's cool. Should I update gp store entry?


Si a eso me referia, que el contenido nuevo que se incluya en el archivo es_ES tambien sea añadido en el archivo es_MX y yo luego corrijo los pequeños "errores" y asi tu das soporte al archivo es_ES y yo al es_MX basandome en tu archivo. No sé mucho de inglés por eso no me atrevo a traducir todo desde el archivo en_US


@darobin Thanks for the history lesson, this was useful! (-:


LGTM


This patch is really usefull. It would be nice if the maintainer would release a new version, so that one can use 0.2.4 on fedora systems instead of github master


I had tried it that way, but couldn't get it working as `proxy` seems to want a real java interface to consume.  https://clojuredocs.org/clojure.core/proxy


Use KClass here?


No point (and generally problematic) to include the same controller for two states, especially parent/children relationship states.


I didn't realize that. I'll update it to dynamically use the sessionN value instead. Thanks.


[random Stephen Fry, ladies and gentlemen](https://www.youtube.com/watch?v=zqPcjm-X5GQ)


This change breaks find on OS X ...

```
$ find . \! -perm /444 -prune -o \( -type f -print0 \)
find: -perm: /444: illegal mode string
```


AA should be lowercase


Are these good names? They will be used like this:

```sql
create table foo (abc bigint, xyz bigint, ts timestamp)
with (sort_order = 'abc,xyz', temporal_column = 'ts')


`dishId[property]` is weird.


I feel like this needs a better description but I can't think of one


> No I don't think they have any since frameworks are OSX specific and we are building cross-platform tool :)

Yeah, but in this case we are giving autotools support for iOS specifically. But I understand your point, if we are to support other platforms in the same style, it would be easier to refactor if we leave it more generic as it is.


This whole thing makes me sad. Far too much infrastructure for a project as simple as Dokka.


should use lowercase here; capitals are meant for class names, not abbreviations


argh. Right you are. Dangers of copying the file structure with `tar`. Thanks, will fix for 2.6.1.


Not sure if this is ideal (as its tied to `ng:model`), but it means we can have css like:

```css
.radio-list--selected .radio-list__label-value {
  /*rules*/
}
```

rather than:

```css
.radio-list__circle:checked + .radio-list__label .radio-list__label-value {
  /*rules*/
}
```


Shows and works despite the new password not matching confirm.


:+1:  For the fixing of my mass renaming.

In france we use the Greek's letter delta (the triangle one) for variation. And a curved d for the derivative. Hum, UTF8 could be nice for math stuff.


Let's leave a comment then.


aha, I missed run=False


This is not cache friendly. I think we should just remove `*.la` files.


FYI, not in the scope of this project, but these Unicode symbols would crash other browsers that weren't modern.


It would definitely help having the derivative d available for variable names ;p


All of the above belongs in a factory.


case


fail indentation?


cupuuuu....


It certainly looks better to me; I just didn't know any better.


map makes this slightly less complex


needs more tests here: test combinations of modifications, etc.


For some reason, the commits got squashed :)

Here's the original text of the pull request:

**Fix guards following through closed gates**
Guards may "follow" the kid to the room on the left, even though there is a closed gate in between.
The fix attempts to check first that the guard can actually move through the gate.

**Freeze time during the end level music**
Normally, time passes while the level ending music plays; however, the music can be skipped by disabling sound.
This optional change disables time passing while the ending music plays, so you can leave sound on.

**Add option for debug cheats**
This adds a few cheats that are useful for testing purposes:
- The square brackets ] and [ nudge the X position of the kid, to help align him perfectly
- After pressing T, the exact remaining time in minutes, seconds and ticks is displayed

**Improve user-friendliness of quicksaving**
- Also quicksave/quickload when shift is pressed
- Do not even attempt to quicksave after the kid has died
- Deny quicksave during the feather fall effect
- Allow quickloading from the title screen

**Fix seqtbl not compiling correctly in release mode**
Disabled the LABEL macro because apparently the sections do get moved around...


Should be temporary, right? I believe scalac will also change behavior to unsorted.


yes ;) i was waiting AVX source :+1:


Better with reduce.


default settings:
t_pre=200*1e-3;
t_post=1000*1e-3;
delay=0*1e-3;
f_res=1;
f_max=70;


I added some more cleaning calls where I knew they should be. I didn't touch anything I was unsure about. I'm not sure I fully understand all of the links well enough to know exactly how this should be but I can look into it if we feel it need to be done.


Use `!==`, always.


Why using String function?


On the whole, looks good.

Some minor points:
- made some minor comments about convention
- the most important change is to add some unit tests that encompass sequences with nonstandard amino acids, so that the use of `get_next_amino_acid` can be tested within the context of complicated residues in a sequence, not just on their own.


Here, you can see you have kind of *duplication*: despite being inside a `characterMenu` function, you had to hard-code the string `"Character"`. It would be nice if the function could be "self-aware" somehow. A straightforward way to do this is to simply pass the name as an argument, but that leads to the weird-looking piece of code:

``` js
obj.open(obj.name);
```

That's the kind of semi-issues that may lead you to eventually use constructor functions, holding state:

``` js
function CharacterMenu() { // I'm using an uppercased C to denote the fact the function
                           // is to be used as a constructor function later on.
  this.name = "character";
  this.text = "Character"; // Could be different, like "inventory" vs "Items".

  // The prototype, as a public API:
  return {
    init: function() {
      // …
    },
    open: function() {
      // …
    },
    close: function() {
      // …
    }
  }
};
```


I highly recommend to use logstasher gem to get rails log in json at source (https://github.com/shadabahmed/logstasher)


> The base URI would be where this file is run from.

My goal is to not have any I/O though, and keep it all in memory... The point of my example's resolveFile method is to intercept any I/O requests for files and return the content directly (perhaps from a DB). For example, when Saxon is compiling the XSLT, and encounters a file reference *within* the XSLT, instead of getting the file from the hard drive, the resolveFile() would intercept the request and provide the XSLT's content directly to Saxon.


i believe we can wrap an io.Reader and split chunks from the reader without copying anything.


So when pocketmine will start supporting 0.12 version?


where do the buffers we pass into the reader come from?


The https url requires user authentication while the git url doesn't. I used uart in a yocto build environment which made it hard to use authenticated https urls, so I figured this would be more reasonable generally.

Regarding fnotify, correct. What would you fix?


@ayhankuru  changing license, and including you as author.


I usually forget what my code does after a few years, but I can remember these, can't be bad :P


I re-created it and (ressuringly) it still explodes - https://github.com/richardc/trapperkeeper-webserver-jetty9/commit/8bbb2ccd3ca53dc8f289fed5ed45c0cda62799cf - if you can point out my mistake that'd be cool


Please protect this with quotes


ack


You probably get these selected items a lot, maybe it should be its own function.


Thanks Nathan. My only comment to this would be that it requires volume mapping (so that data does not get list) and we should be very clear about that to the users of this container. I will need to stick to the instructions that I posted before as we have started collecting prod data with the previous version.


Actually the correct here would be **its**, not **it's**. **it's** means **it is**, which is not what we want here. The possessive pronoun for **it** is **its**, so that part was correct before and is wrong now.


Why `padding` `!important`?


Can you send another pr fixing this, @thiago-rocha?


Fix this formatting.


Very clever :D


Never use this error message.


I like your suggestions. However, this way you wouldn't know whether the character was intentional or replaced by the system.


Yes, unfortunately I didn't save my test data.


consider a spitter like this
```go
type Splitter struct {
  R    io.Reader
  Size int64
}

// this version allocates a single buffer and reuses it
func (s *Splitter) NextBytes() ([]byte, error) {
  buf := make([]byte, s.Size)
  n, err := s.R.Read(buf)
  return buf[:n], err // pass io.EOF and any other error.
}

// this version does not copy.
func (s *Splitter) NextReader() io.Reader {
  return io.LimitReader(s.R, s.Size)
}
```


ah - using the virtualenv for python was something I was afraid of.
I had major issues with that and python-gi and python-dbus - I dont remember the problems right now - but things were getting messed up because I had to use `deadsnakes` ppa to get the `-dev` versions (to compile dbus/gi) - and that implied I couldn't use container builds which meant they were super slow - and also had some compilation errors in general because autoconf found python version clashes.

I should actually give it a try once again and see if it does have issues. pyenv was nice on circleci :)
There are some minor things like the fact that you need to use $VIRTUALENV for the path and so on, so, I'll need to work on the small details.


whoa, looks like I got added twice?


I'm stated as maintainer in the README, so put it here too?


right


Yes, you can update it. But some features as savestates still not finished
in new GUI.
I also want to publish it via my googleplay account.

2015-08-04 19:20 GMT+03:00 Gerhard Stein <notifications@github.com>:

> Hey, that's cool. Should I update gp store entry?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pelya/commandergenius/commit/25c047059c62d90dbd811901f435611136c867c0#commitcomment-12520898>
> .
>


what does `cmdclass` do?


Ah, okay.


conflict*


Let's talk about this file.


it adds the `./setup.py test` command to execute tests


You can variablize to save typing.


What about
```cmake
set(merged_file_contents "${merged_file_contents}\n#elif ${arch_define}\n")
```
?

Also why we need `#elif`? Maybe use pattern:
```cmake
#ifdef ARCH_N
#endif

#ifdef ARCH_M
#endif
```
?

Also I guess it would be nice to add a note at the top that this file generated automatically?


As you do nothijng here you can remove this inherit


I don't see any commit with cleaning calls.
What I meant was adding `clean' like
https://github.com/statsmodels/statsmodels/blob/master/statsmodels/genmod/families/links.py#L425
to the InversePower links, that keeps the values away from zero, so we don't get inf and nans


why not use `tempfile`? https://docs.python.org/3.4/library/tempfile.html


No point in putting chef in the `=`


Does it actually make sense? I mean only we know that this file will be moved to `*-${arch}` so any other tool will not find it.


ey ey ey ey


and how gets `run_tests()` called? you never reference it


Use UPPERCASE for constante


"circumstance"


Why .rating? You can just return a number.


> On 4 Aug 2015, at 17:33, Sean McIlvenna <notifications@github.com <mailto:notifications@github.com>> wrote:
>
> My goal is to not have any I/O though, and keep it all in memory... The point of my example's resolveFile method is to intercept any I/O requests for files and return the content directly (perhaps from a DB). For example, when Saxon is compiling the XSLT, and encounters a file reference within the XSLT, instead of getting the file from the hard drive, the resolveFile() would intercept the request and provide the XSLT's content directly to Saxon.

Oh I see. Call-backs for this sort of feature will be tricky to implement but is something which is on our todo list for a while. I have created a bug entry to keep track of this in our development. Please see: https://saxonica.plan.io/issues/2437 <https://saxonica.plan.io/issues/2437> which you can subscribe to.

kind regards,

-------------------------------
O'Neil Delpratt
Software Developer, Saxonica Limited
Email: oneil@saxonica.com <mailto:oneil@saxonica.com>
Twitter: https://twitter.com/ond1 <https://twitter.com/ond1>
Tel: +44 118 946 5894
Web: http://www.saxonica.com <http://www.saxonica.com/>
Saxonica Community site: http://dev.saxonica.com <http://dev.saxonica.com/>
Bug tracking site: https://saxonica.plan.io/ <https://saxonica.plan.io/>


直接 replace 掉, setting 頁面的 logout 會沒有東西可以 pop 啊.


Credits ?


"configuration"


Hey Chetan,
In the parent pom, I added checkstyle configuration to plugin management so that it can be shared.
But as described here, I still need to refer it in child poms:
http://stackoverflow.com/questions/10483180/maven-what-is-pluginmanagement

Let me know if I am missing something.

Thanks,
Chandni


color var


"inconsistent"


ibid. "inconsistent"


Lot of stuff you mean sir


missing newline :)


Wonder whether the locale switcher should have `assets/stylesheets/responsive/_locale_switcher.css.scss` and get included in `custom.css.scss`?


Can you please update the `onload` handler in this function? PR #476


This is a test! Please don't use it!


can you make an issue for the TODO? since we don't use coala here we likely forget to fix this^^
ack


"check out"?

Maybe "read the :ref:`clustering-framework` tutorial first: "?


Somebody copy and pasted navbar.


still missing newline :P


the user cheese count is already set on the listener inside the TheftActivity (that checks for count changes coming for server side), so we've been setting it twice.


Nice code!

Just for future reference, numpy.array(zip(array1, array2)) is not required for pca.fit and zip(array1, array2) is sufficient.


why do you ask such questions? I dont know!


Makasih!!!


Kr\xfcger ...


Sorry. Should be pushed now.


One might want to use the sqlite3-armv7 on their iPhone :-)

I want to push the GEOS project to compile spatialite, and in the `./configure` line I have to add something along the lines of `--with-geos=@HUNTER_PACKAGE_INSTALL_DIR@/bin/geos-arm7`. Geos installs a binary (or script, I must check) that knows where `GEOS` is installed and gives some meta information about it. hence I was making an effort to not change the build result so much, because other projects down the road may need them.

This `geos` binary might not be cache friendly either, I must check. I must test before to be sure, I'll do this as my next task.


why does ascii not contain umlauts T.T


Oh right I did not see the refactor branch! I am testing it right now.. Thank you for the great work


Well damn. How comes Nvidia tout full GL 4.5 support for Fermi and yet it doesn't support this feature, eh?

Hopefully it'll work on my 290X if I can coax that back into life. Otherwise 970 it is.


it's a `TestCommand` class which defines this `run_tests` hook


Do you really need to import fvtk again?


all commands have a `run` hook and the `TestCOmmand` class has this specialized hook on top


I think it's better to have individual SVG files rather than one sketch file. See `CONTRIBUTING.md`.

Also - for consistency I think we should use lower case for all text. I have added another Cukeup logo: 385e88466f5c1972977ccc37fc0b2e4a4c48f34d


Same for these imports from `clusterin` and `metric`. Some of them repeat imports that happened above


ilo cupuuuu


fixed


currency filter


you can add the plugin to the parent pom (as plugins and not plugin
management) and it will be inherited by all the child modules. There are
other examples such as maven-dependency-plugin.

--
Chetan

On Tue, Aug 4, 2015 at 9:45 AM, Chandni Singh <notifications@github.com>
wrote:

> Hey Chetan,
> In the parent pom, I added checkstyle configuration to plugin management
> so that it can be shared.
> But as described here, I still need to refer it in child poms:
> http://stackoverflow.com/questions/10483180/maven-what-is-pluginmanagement
>
> Let me know if I am missing something.
>
> Thanks,
> Chandni
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DataTorrent/Apex/commit/126f3a90297b2c2473bff173c5158b1ce6467ede#commitcomment-12521484>
> .
>


we will of course use coala here soon.


hanny cupuuuuu


filed https://github.com/coala-analyzer/PyPrint/issues/5


english :P


maybe this can help http://web.archive.org/web/20110803232818/http://www.wowhead.com/npc=5865


there is an easiest way, see my solution for example


So this addition of :election/types to the initial schema will fail to get applied in production, because it has already run this migration. All new additions must be in new migration files that have not yet been deployed.


```go
type Splitter interface {
  NextBytes(buf []byte) (n int, err error)
  NextReader() io.Reader
}
```


fine, but that 101 is strange, I prefer a <= 100


Typo Error: i9500 to wave


This all `REMOVE_RECURSE` and `GLOB` code make me nervous a little bit :) For example if `multi_arch_install_root` will be empty (if somebody make a mistake) we will try to remove empty directories (I hope so) in the whole system. Please check that `dir` argument is not an empty string in `remove_empty_dirs_recursive` function. Also please protect every path with quotes. It will not change anything in most cases but it's a good practice I think.


> set(merged_file_contents "${merged_file_contents}\n#elif ${arch_define}\n")

It was like this before, but it got difficult to read at the else part

```
#else
# error Architecture not supported
#endif\n"
```

> Also why we need #elif? Maybe use pattern:

I thought about that, but I thought it would be nice to fail faster with an error of an unsupported architecture, then the have a more obscure error down the compilation stage. Maybe we need just a more descriptive error message :-)

> Also I guess it would be nice to add a note at the top that this file generated automatically?

Agreed


didn't work, missing a return. And see the solution when negative numbers.


Yes, but this should not affect the local user on the device. This was more for increasing the performance, so that you see the count change as soon as you steal, instead of waiting for server to return. And even if server returns it will update the same count again, if no other transaction takes place. Will explain further..


Vale, perfecto. La verdad yo tampoco soy traductor y hay veces que cuesta sobretodo el nuevo contenido que van añadiendo, ya que lo tengo que hacer a ciegas y a veces tengo que arreglarlo para la siguiente versión.


i assume this is a faked one


So you won't be able to tell at how to change the color, at least? ;c


hehe, good solution :)


:P


uhmm... don't mix camel case and _, use only one :)


the refactor branch is not in a usable state! But have fun working with the master version :)


Does this actually work? One of the technical requirements of all SciJava plugins (including `Op` plugins) is that they have a noargs constructor.


Ever heard of the "tabs vs. spaces" thing? It can get quite serious.


Fold Tree can be implemented using the same code from simple_docking, lines 48 to 50. (Side Note to Kim/Kevin/Tessa/James/Matt/whoever- I would *not* make a separate function for setting the fold tree without thoroughly testing that it works properly - I'd worry the pose wouldn't be modified properly outside the function. foldtree.hash_value() is useful for comparing fold trees automatically, use it to test if someone does create a new function to set fold trees.


The movemap ranges should be an argument of the function as well, we'll want to test a variety. Also, this would be another good spot for the find_pyrosetta_res_nums function (see mutate_pdb.py) to be used, so we can just use the actual protein sequence number regardless of trimmed/mutated/etc pdb being used.


I can't verify those, but it sounds wrong to receive syncItemDiscovered for an item that won't be synced (i.e. dropped before the propagate step). Can you think of something?


These lines and several others that are on develop have been removed. I think your client might have been out of date. Can you merge into your branch from develop so that these lines don't get clobbered?


ditto first comment about merging from develop


In this commit I add a simple example of plotting a simple png (lena grayscale image). The file required to be downloaded as I didn't find any image in bokeh repository without alpha channel. I don't know if this example should be integrated in the pull request.

As expected we get the image up-side down, but it's coherent to what we got with pylab.imshow(img, origin='top').

![lena_bokeh](https://cloud.githubusercontent.com/assets/2479539/9066992/0e2665b4-3adb-11e5-8b6b-2c944fe32674.png)


ditto clobbered line


ditto clobbered lines


probably want to add dock_partners and foldtree to the function arguments in addition to the pose, they can use the same script args as simple docking. Additionally, we'll want to be able to pass ranges for the movemap, so there should be arguments for that.


Would it be good to put at least a `assert.pass('parse ' + filename);` after this line so the test output is consumable TAP. I know L36 would produce a TAP comment, but I think it's moar TAP-like to get the `1 ok msg` than relying on comments that won't be parsed and won't count towards the total test count.

Perhaps we can move line 36 outside the loop.


I think these were test methods that got left over and should be removed. Or are they serving some other purpose?


Are you sure you are `pre-incrementing` or you rather `pre-decrementing`? :smiley_cat:


This should be able to delegate to `NewCustomFramework`, right?


Isn't this try still doing the same thing more or less? Why did the comment go away?


please format according to guidelines. This might be using tabs? Anyway, we use 2-char indent, all spaces.


Open the upstream pull so you get a PR number to include in your commit for rebasing.


Output coming wrong for n>=6.


Ok, thanks for the tip for quoting the paths, I will do it.

> Please check that dir argument is not an empty string in remove_empty_dirs_recursive function

That is a *very* good idea :-)


Are the updates to `inc/install/update-v2.1-to-v2.2.sql` correct? It looks like a conflict wasn't resolved correctly, could that be?


Probably a more automated way of doing this.


Inverted return logic for button command for easier scripting.


Why are you using array notation?


@E3V3A Problem reported & diagnosed in #596, & enhancement/fix suggested in
#597.

On Tue, Aug 4, 2015, 5:35 AM E:V:A <notifications@github.com> wrote:

Logcat please!

--

https://zhjf9h10ji3.ting.com/


This default is not really necessary since `UserMailer` inherits from `ApplicationMailer` and a default `from` is already defined there. Looking at your `contact_form` method, the default `from` doesn't seem necessary at all since you set it in the method. :smile:


Nice :+1:


Not sure if `filteredOrders =` is necessary here.


You can use `contact_path` insteaf of `/contact` here.


Fixed that a minute ago


ln6 needs a comma at the end


will try to fix tomorrow


And? I could let the user specify their package manager or whatever, but it's easier and guarantees an up-to-date pull.


> Headings could have a color

They should even have the same color and thickness as the headings in other magit buffers.

And I also think that the keys should a color different from the descriptions and the --arguments. And green for "on" is also a good idea.

I think you should revert this.


thank you


Probably could have used nested states here.


Can't we just get rid of this method entirely? Why is it still here if it's not doing anything?


data !== song


The comparison above is a `string in other_string` match so `application/json` will match `application/json-home` but for sake of completeness I'll add a test as that seems a worth thing to do.

This would also be a good time to make this section not case sensitive.


Huh, I didn't know about `IDebugControl::Disassemble`, I'll take a look.

You're right, it would be a much easier way to do the disassembly, thanks for pointing it out.


OK, thanks! That was bad timing on my part, sorry.


Dummy data.


I think for now using a GraphListener is good. In the future, we can look into designing a listener interface that can interact with resources coming and going from the agent inventory.


Worked for me on PrestaShop 1.5.3.1 too


This file shouldn't be in Git at all. Can you try to remove it using `git rm -rf spring/` and commit that?


Use manage_daemon here.


I don't care what color is being used, but the names should be in a different color than the commit messages. This is especially important in narrow windows where the name-date margin covers the summary of some or even most commit messages.


Fixes #1


why ascii and not utf8?


I kinda like this one.


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 37](http://teamcity.otenv.com/viewLog.html?buildId=1437905&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 38](http://teamcity.otenv.com/viewLog.html?buildId=1437906&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) is now running


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 38](http://teamcity.otenv.com/viewLog.html?buildId=1437906&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 34 Build time: 0:0:0


I stopped doing anything with "resource configuration" until inventory designs in resource config as a first class object. Putting it in as a property is good for now as that is how the UI is pulling other information about a resource. There is a JIRA on inventory to implement "resource configuration" - but first they need to implement operations as a first class concept. Heiko and I agreed that we should first get operations to work and once that workflow design is implemented and we understand better how things can work, we'll then move to resource configuration (because resource configuration will involve not only reading config properties but also changing them).


Can you add fav food?


ack


TeamCity Architecture Team :: Java Platform :: otj-components :: otj-data-structures :: build [Build 37](http://teamcity.otenv.com/viewLog.html?buildId=1437905&buildTypeId=ArchitectureTeam_JavaPlatform_OtjComponents_OtjDataStructures_Build) outcome was **SUCCESS**
Summary: Tests passed: 34 Build time: 0:0:0


haha  ;) No problem


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 46](http://teamcity.otenv.com/viewLog.html?buildId=1437916&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


May want to add ``pause`` as well - less useful currently than ``data_collection`` though.


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 46](http://teamcity.otenv.com/viewLog.html?buildId=1437916&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 47](http://teamcity.otenv.com/viewLog.html?buildId=1437919&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


on_failure.  I like the sound of that.


On OS X it works smoothly. I haven't tried at NV. I just fucked all my configs by pushing from a machine where I didn't filter-branch out my emacs configs, so give me a day to fix that.


We should probably change the == 2 and ==3 to constants. It'd make this far more readable.


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 47](http://teamcity.otenv.com/viewLog.html?buildId=1437919&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


I believe the only heuristic it uses is "needs a full stop".  But your sentence is better.


Why isn't this just `Resource`?


@vadimcn Not right now. That would add a lot of complexity and probably break stability in some way.


And he should probably reset his git to HEAD.
Best would probably to run in each architecture's directory: git reset HEAD --hard


> You should design around the people your thing is important to

The notion of "the people your thing is important to" could use a half-sentence introducing it, since the subtitle phrases it as "the ones who will geek out over it", and that's not as immediately apparently the same thing as "the ones who will eat, breathe, and live it".


The reason for separate app is because current app on Google Play got 2.9 stars, and it's unlikely that you'll fix the rating by updating it, because most people who rated it badly already uninstalled it. Anyway, would be nice to push the update to the remaining, more devoted users.


Oh wow.  Yeah, that seems valuable.


Does this apply to the save hex changes button? This doesn't appear to.


There are a LOT of changes that can be done for readability, but it has to be done all over in that case. For now I suggest we focus on getting things running again.


UpdateReleaseNoteScript (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1778 of 1778 tests passed)

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


Why did you revert it? It would be nice if you could upload the newest version :+1:


Hi, what‘s case for use disown api.


It applies to the menu item "Save Hex Changes", is there any other button you had in mind?


Corrected ultra low sampling speed is corrected (see https://github.com/bvernoux/hydrafw/pull/18).

Regarding triggers, I don't know how to do it properly :
Let's say you configure a trigger with 1000/3000 before/after samples. if the trigger is matching at sample #200, what should be done ? MAybe I'll add a capture count, and only send captured data.


that's problematic. Should copy zone.name.length() bytes, then pad with zeros.


> it would be nice to fail faster with an error of an unsupported architecture

I see. What about:

```cpp
#if !defined(ARCH_A) && !defined(ARCH_B) && !defined(ARCH_C) && ...
# error "Architecture not supported"
#endif

#ifdef ARCH_A
#endif

#ifdef ARCH_B
#endif
```


> Why isn't this just Resource?

This answers questions like: "who can list pods" or "who can get pods named foo", etc.  The determination is about who can perform a given action.


@bbcrddave Why not fix getIsTextTrack to check for type === 'fragmentedText'?


Yep, that makes sense.


@jmchilton :open_mouth: Didn't know about `pause`, is a new step type or is simply not available through the UI?


getIsTextTrack expects a mimeType which is not known at this point. fragmentedText is not a mimeType.


PTAL @josephsnyder.  I noticed that I was getting slightly different coverage numbers when I ran your test on a different system.  This turned out to be due to the fact that ``CoverageFileLog::Insert()`` overwrites any previously existing information for the file, so I'd get different results depending on the order that the JSON files were parsed.

My reworked version of your parser gathers up all the data and then saves it to the database as a subsequent step.


ack


There is a typo in this commit message.
Hedhat should be Redhat


Excuse me. Could you help me with extracting the art of both servants and essence cards? Already extracted everything from the apk, but still no clue how to unpack. Thanks for your attention anyway!


Better to leave the review to Fran as he has been working with you before - I don't currently have the bandwidth nor actually enough insight into your project.


Can you move this to Sugarable or some similar module.  We'll need this in project.rb as well.


May fix issue #75


Thanks man, this cool ;)


Great commit! I think after that this looks ready to merge.


This is some of the work that was originally done in the `uslm-export` branch, but that was hopelessly out of date, so I'm starting over and trashing that branch.


yep, sure...


Duh


Doesn't this operation fail outright for any BaseModel defined in OSGi modules?

If you look at the method implementation, the portal classloader will fail to find anything that is not defined in it's or it's parents classloaders, which any OSGi service is not.


That made the trick.

2015-08-04 19:17 GMT+02:00 mfonville <notifications@github.com>:

> And he should probably reset his git to HEAD.
> Best would probably to run in each architecture's directory: git reset
> HEAD --hard
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/opengapps/arm/commit/f20648dee99f7b2fa832d91979dc2db06ea101a9#commitcomment-12522186>
> .
>


Mind adding title here. So users can see this as top level functionality!


@ttuegel:

> I had previously tried updating the HPLIP version of a working configuration and found that the plugin didn't work with 3.15.7

Do you remember if this working version was 3.15.4 or 3.15.6? Might as well roll back if it breaks certain models.

I did *need* 3.15.7 to support a newish DeskJet 1112 (don't buy one). I'll hack around it for a while until [?] the PCL bug is fixed. It shall be my penance for breaking the plugin.


I'm on github mobile, so I can't see the whole code. I wasn't aware of an
extract button in PCCEditor2.
On Aug 4, 2015 11:19 AM, "Rasmus Egerö" <notifications@github.com> wrote:

> It applies to the menu item "Save Hex Changes", is there any other button
> you had in mind?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ME3Explorer/ME3Explorer/commit/ad837bbf64514092709716145550701ba3f1625a#commitcomment-12522236>
> .
>


For what purpose?


Are there any specific reasons to use the original order?


Yes that's better, I'll change it.


this was intentional, since `visit` "should" iterate over an array, then visit each object. were you having an issue?


@nckx The working version was 3.15.6, but rolling back HPLIP alone won't help, as I indicated. I tried rolling back HPLIP to 9f5b9de5595534d52d278de9d3d2781c63ea0a31, but that didn't help. The only solution was to roll back my entire Nixpkgs to that revision.


This is much nicer.


fix


fix, why did this change


Yeah, I could do that, but what do you see as not being positive with the `#elif`? The main reasoning is that they are mutually exclusive blocks, because the contents of the files should be mutually exclusive by architecture. I do agree that the resulting `config.h` was easier to read with an IDE where there is folding for the preprocessor blocks, but apart from that, they reflect the intentions about wanting mutually exclusive blocks quite clearly.


I wanted the examples to be almost self-contained. That way, you don't have to go back at the top of the tutorial to see required imports. What do you think?


Just made a PR for it!


Not available through UI unless you enable some magic option - it would be useful but there is not GUI for continuing the workflow after a pause step - can be done via the API though.


@anareyna fix identation


Yeah... when i did `template.layouts(['path/to/layouts/*.hbs'])` it was iterating over the array of glob patterns.


@josefdlange awesome! :ship:


ditto


Probably need to use `is-glob`


I see what you mean. if it's working now no need to change it.


We're trying to upload this to an internal supermarket and it fails because it's looking for a metadata.rb file.


This is actually subtly broken. ```std::string``` doesn't have a ```(const char*,size_t,size_t)``` constructor:
http://en.cppreference.com/w/cpp/string/basic_string/basic_string
What seems to happen instead is it converts the char array to a temporary ```std::string``` using the ```(const char*)``` constructor, which cuts the string off at the first null byte, and then passing the temporary to the ```(const std::string&,size_t,size_t)``` constructor to create another copy.

The problem is, the ```(const char*)``` constructor assumes a null-terminated string, which isn't guaranteed here (the BSA footer stores names in 12 bytes, and an 8.3 filename is exactly 12 bytes including the dot; there's no room for a null byte). It's just luck that the next byte on the stack after the array is 0. And apparently, the ```(const char*,size_t)``` constructor copies the whole size, including null bytes.

A proper fix would be to increase the array size to 13, while still only reading 12 bytes, and ensure the last byte is 0. Then it can be safely passed to the ```(const char*)``` constructor. I've committed a fix myself, and it's properly extracting all files from midi.bsa now.


I have added the codes for the 4 plots. For, Plot 4, the sub-plot in the left bottom has a real line box. I need to remove it.


Thank you Caleb, noted!


:clap:


Version 1.0


PS_AUTOUP_CHANGE_TO_DEFAULT_THEME is too long for old versions of Prestashop (1.4 and 1.5): only 32 characters allowed for Configuration->name property.


Thank you Caleb for reviewing my code. All comments were helpful, I'll pay more attention next time to the accuracy of the syntax and indentation, thanks!


What we are passing to the function getIsTextTrack is chunk.mediaType, which is from streamProcessor:getType() which is in turn fragmentedText. So that should work?


Nice!


"blue too!" is not a color :(
try again


Can you please change this to be grammatically correct? Either `options` or `anOption` or something like that.


Honestly, I'm a bit iffy about this code, and perhaps the driver status manager should take the full list of IDs instead of just the number of previous containers. This was the previous logic though. Should I open a bug for this?


Then why not `Action`?


"blue/red" is not a color!
try again


Ah, the Peview() is added to saveHexChangesToolStripMenuItem_Click, and seems to work as expected in my tests.


good job!


Great!


this seems like it reads better as review.Action.Verb


> Then why not Action?

This aligns with the roughly corresponding interface here https://github.com/openshift/origin/blob/master/pkg/authorization/authorizer/interfaces.go#L20


I see. I will fix it.
Thanks,
Chandni


The following 3 operations seem like they could go into a resolve.


On XFSHoster.py add:
from module.plugins.internal.Plugin import set_cookie


Currency filter!


@MisterHighQ let me know if this commit fixes Issue #1 . Thanks.


Not `response`.


Well said.


The 'a' stands for 'argument'. It's not meant to be grammatical.


I tested locally and g-ir-compiler is complaining, trying to get help from IRC.


That wouldn't work as intended if the exception thrown were not to contain a 'message' property. (e.g: throw 'testError' )


Seems strange that you would create an object on the fly for submission.


Now you're just making things up...


Dank memes


Done in [697ba8b](https://github.com/pantheon-systems/documentation/commit/697ba8bc966d6dcecc29d0610c2a84fcfe77664c) @bmackinney


No, I'm not.


Also, no validation. :disappointed:


"argument options"?


Dahora! "Revert "Revert "Revert "Revert "Revert "Revert "Revert "XTPO""""""""""""

aseuhasiuehaisueh


LGTM


It's an argument to the function/method.


Look at my other methods.


I know. It's pervasive, and it looks silly.


All changes are in. Please see an example of rendering the same formula on the test server and on mathquill website. There is a slight difference, but I am not sure why.


@carloshs92 corregir aqui la indentación.


Added in commit 077a3ffdd5e6b5cbe83ed6e22877b4c52be24247. Thanks!


Yes, being explicit about things is helpful. Also, this is probably not a `List` but a `Set`, right?


Agreed, but getIsTextTrack currently compares mimeTypes, not mediaTypes. There is no method currently available to compare a mediaType and this wouldn't belong in manifestExt if there was anyway. Could you propose the method you would like to see?

I understand that this isn't a great solution, but it's really hacking round a nasty existing solution.


The intented is understood, of course. But if we would have Bring sanhw1 up to date (IEG), that would read as the batch changes where made in IEG and not all dictionaries at an equal level.


That wouldn't work as intended if what is thrown were not to contain a 'message' property. (e.g: throw 'testError', throw 3, etc...).
And we're not sure what could be thrown from the 'prepare' function.


`ng-src`


That's a personal opinion.


HISS


Groups


Thanks for all the feedback, @alexanderkjeldaas! I've pushed a commit that attempts to address all of them. Let me know what you think so I can upload it to hackage when you think the changes are good enough.


Missed a spot:

STR_CIVILIANS_KILLED_BY_XCOM_OPERATIVES (us)
STR_XCOM_CRAFT_ARMAMENT (us & gb)
STR_XCOM_PROJECT_MONTHLY_REPORT (us & gb)


I meant not using getIsTextTrack from manifest extensions but having a similar function in SourceBufferExtensions, (a private function)
var getIsTextTrack = function(type) {
        return (type=== "text/vtt" || type=== "application/ttml+xml" || type === "fragmentedText");
    };


> Groups

Believe me, I would have except that makes the default conversion barf on mismatched types, because internally groups are `util.StringSet`


Test


You forgot to update `README.md` and `examples/events.py` for this change. :innocent:


It is unlikely I will be able to do all that for you as I know nothing about OpenEdge and how it works but I can create an assembly that will help you get started and from there you can start to learn about how to integrate with the MYOB API. I think the best thing as well is to take this conversation to another project where we can collaborate on this for while. What version of Visual Studio are you using by the way - I can use either 2013 or 2015?


@Noryoko, why are you removing the usage of LINQ?


This should be two separate lines. Try again.


yeah looks like in newer jasmine versions you have fail(msg) method. which will do what I want here.


This is a very sloppy commit, don't pull this change, unless you rewrite before you commit.


I'm assuming you mean `git fetch release`


Good job!


Wow this is really awesome! I can't wait to try incorporating this.

I am actually trying to figure out how to use babel-compiler to compile user supplied js from an ace editor into es5, and I got that part working even with async function in the signature, but it bombed on "await", so I'm curious about getting it all working. Although, I would imagine your sample is bring compiled by meteor on the server natively to es5?


ok, spelling correction too.


Yes


Changing the module name doesn't completely work, since some needed files are not tracked by git...


good job!


Glad you added the sync up check.


"turqoise?" is not a color
try again


Well, since github.io only hosts static, we'll have to do it by loading the
whole list and then filtering with js or perhaps making an ajax call.

On Tue, Aug 4, 2015 at 10:09 AM, AnEmortalKid <notifications@github.com>
wrote:

> I guess we'd need an anchor with random in the name?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AnEmortalKid/anemortalkid.github.io/commit/f2f1916ef92324c9886cd3a3c30b989ee397e54f#commitcomment-12517508>
> .
>


Андрей, привет)
Тебе нужно доделать свой макет:
1)Оформить заголовки по стилям + создать див с оранжевым подчеркиванием;
2) Заменить lorem на текст из макета и сделать классу .about_me-text вертикальные и горизонтальные отступы согласно макету
3) Доставить иконки в круги
4) Загрузить bg в класс .map и картинку с историей в ней


ACCIDENTAL ACCIDENTAL I didn't mean to commit, but I reverted so it should be fine.


This seems like a bug.


Why is this necessary over what's in sbt 0.13.8 by default: https://github.com/sbt/sbt/blob/0.13.8/main/src/main/scala/sbt/Defaults.scala#L184 ?


Oh ... I get it.


> Yeah, I could do that, but what do you see as not being positive with the #elif

I just think that this may simplify CMake logic a little bit.

> The main reasoning is that they are mutually exclusive blocks

Also is it true for now? For example if think that `armv7s` architecture has quite the same variables defined as `armv7`. Is it working correctly?


Personal use only. I collect the arts of TM. But if it causes problems, I will understand, and sorry for the inconvenience.

I saw that someone had already posted on the imgur and on tumblr, but imgur album got deleted, and tumblr does not have all the images. I was able to extract some basic images that were visible for programs such as Unity Assets Explorer and Unity Studio (which can be found on the xentax forum).

Would try to use the libwebp to see if it worked, but I can't run the program. So I ended up resorting to ask for your help. But as I mentioned, if it causes problems, I will understand, and sorry for the inconvenience.


This looks good.


Read through this again from the start.  This looks super now, and I really do understand what it does.  :-)

:+1:


This file contains all the study materials to learn web development.


Yes, the changes in Apex and DTX are not visible to outside so there won't be any compatibility issue.  However, I have two other pull requests for upgrading jetty library for Malhar and Megh that will have backward compatibility issues.  Let's discuss them.


Shit, thanks :)


Why doesn't this use access review with namespace set instead of local?


os.walk() returns a 3-tuple, subdirs is a required part of iterator.


Good code should be clear. "a" is not a suitable abbreviation for "arguments"; at the very least, it should be "args", but that's somewhat beside the point. This name leads to confusion whether you pass in *one* option, or *many* options. More importantly, (like me) it's more likely to get misinterpreted as bad grammar, which makes the code look bad.

I would appreciate it if you chose *either* `arguments` or `options` in this case.


Good job!


> Why doesn't this use access review with namespace set instead of local?

No reason.  That's your preference?


Yes, lots of stuff. None of it is useable atm, the rest of the team just wanted me to push all my changes to GH.


So what were the possible consequences of not having this call?


This is also obsolete, should be replaced by
    rm -rf {test-,}requirements.txt


Yes


:D


> rolling back HPLIP alone won't help, as I indicated.

No, but [this](https://github.com/NixOS/nixpkgs/commit/b62debe0423d72062c60f93dab895de532325e38) might. Plus, if .7 can't ever work for some current users there's not much point in keeping it.


I guess it's test code so it doesn't matter


> Also is it true for now? For example if think that armv7s architecture has quite the same variables defined as armv7. Is it working correctly?

I believe so, the merged `config.h` had all contents of all architectures the same except for `arm64`. It is just that like this I guarantee that the file generated by architecture X will only pick up the part of architecture X and no other.

> I just think that this may simplify CMake logic a little bit.

I can refactor from this big loop into functions to make it more readable :-)


The simple solution will be to detect 'random' in the url and trigger the
button you've already made.

The long term solution would, I think, involve a flatfile database of
entries and calling a single line from it at random with javascript. So
easy with a server side language, kinda crazy in js.

On Tue, Aug 4, 2015 at 1:57 PM, Chris Knowles <cknowles117@gmail.com> wrote:

> Well, since github.io only hosts static, we'll have to do it by loading
> the whole list and then filtering with js or perhaps making an ajax call.
>
> On Tue, Aug 4, 2015 at 10:09 AM, AnEmortalKid <notifications@github.com>
> wrote:
>
>> I guess we'd need an anchor with random in the name?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/AnEmortalKid/anemortalkid.github.io/commit/f2f1916ef92324c9886cd3a3c30b989ee397e54f#commitcomment-12517508>
>> .
>>
>
>


Would expect this to use SubjectAccessReview


This is logic that should be in the client code, not here.


great job!


setTimeout, 0 could also work here.


Looks like I misspelled "canola"!!


мои опасения подтвердились


FWIW - I think an earlier commit in this PR removed this line, just wanted to note that in case it fixes something but breaks another feature area. (cc'ed @dan-f )


I was under the impression that it is safe to shutdown something, even if it hasn't been started.  (ServerImpl, for example).  Would it be better to make transport.Server and its implementations always safe to shut down, rather than checking for it here?


yah! good job


No problem. I will do as such and make changes as I see them. However, from
my experience code clarity is subjective - I've worked in environments
where such abbreviations are clearly understood (and not considered silly).

On Tue, Aug 4, 2015 at 2:01 PM, Andrew Hankinson <notifications@github.com>
wrote:

> Good code should be clear. "a" is not a suitable abbreviation for
> "arguments"; at the very least, it should be "args", but that's somewhat
> beside the point. This name leads to confusion whether you pass in *one*
> option, or *many* options. More importantly, (like me) it's more likely
> to get misinterpreted as bad grammar, which makes the code look bad.
>
> I would appreciate it if you chose *either* arguments or options in this
> case.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ELVIS-Project/vis-client/commit/32b5ee60092e7e0679ce583b7c87c45c22499081#commitcomment-12523164>
> .
>


yah!


@trek mind cutting another one for this?


@LZorro added time delay for the escape key worked just like you suggested! Actually, I really like how this works and you might want to look at adding this for some of the other actions like plant and shoot seeds as those may be being triggered multiple times.


yes!


good job!


With `CrossType.Pure` it adds nothing. A "normal" `CrossProject` puts shared files in other directories so it will not work.


@abma help!
If all tests pass, why does make return 2 (making travis declare failure)?


Drop this one


Great work @raffi!


Can we please call this HasTrait?  ActorInfo is (slightly) more than just a collection of TraitInfos, so it's a bit weird not to qualify this further.


actually, I think you have something like this already after testing.


what is your favorite color?
try again


I suggest to split the string over multiple lines and add new line (See `// XXX`)

```
QString labelText = QString(
  "Data is present in the current scene but not in the scene view being restored.\n"
  "\n"  // XXX
  "Continue restoring and Discard data, Save missing data to this scene view, or Cancel?\n"
  "\n" // XXX
  "Saved Volumes may not be shown in slice views after restore.\n"
  "Default if don't show again: Save.");
```


good!


1: M111 unrecognized component name: i18n


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: i18n


not idiomatic, should be `isEqualToArticleTitle:`, which I would potentially argue should be a convenience method on `MWKArticle` itself


unnecessary `if`, just return the result directly


Ugh. Sorry. :-/ But these improvements are purty, anyway. :)


1. Add weather client feature

2. Add weather server feature

3. Dimiss Sync view when setting/editevent page show up

4. Disable selection of title textfield

5. Create NoEvent for tomorrow


It's my own fault for not merging sooner.  :)


@cmarcelo, could you please take a look?


1: M110 missing component name


> I tried rolling back HPLIP to 9f5b9de, but that didn't help. The only solution was to roll back my entire Nixpkgs to that revision.

@nckx To reiterate, I tried reverting individual commits, such as the 3.15.7 update, without success. I think there is some dependency problem, because I even tried reverting all the HPLIP patches back to the commit indicated above (which was commit that updated to 3.15.6), and it still didn't work. What finally did work was `git reset --hard 9f5b9de` on my whole Nixpkgs tree.


I just readded all the type signatures (thanks to haskell-mode's `C-u C-c C-t`!). Thanks for point this out @yaxu!


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.884.528e03d02feeccfa291cbe2e0af73e7e4f83d5ac](http://build.palaso.org/viewLog.html?buildId=60588&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 397, ignored: 4 Build time: 00:03:02


"

Hello @stevenovakov <https://github.com/stevenovakov>,

  * Can you send me a Minimal (non-)Working Example, so I can run the
    example on my computer.
  * Can you create a issue in the LTPV project for this problem, it will
    be simplier to discuss

"

Hello,

Simon Denel and I have already had a conversation about this, see
forwarded. Cheers,

*Steve *
On 8/1/2015 3:45 PM, Simon DENEL wrote:
> Thank you for you documented help. I will look deeper your promising
> solution in 10 days. I am currently stuck with a 20th century Internet
> connexion, and cannot even download "ocl-icd-opencl-dev" that seems to
> be a requirement (cf my screenshot)
>
> Have a nice week,
> Simon
>
> On Sat, Aug 1, 2015 at 1:40 AM, Steve Novakov <stevenovakov@gmail.com
> <mailto:stevenovakov@gmail.com>> wrote:
>
>     Yep, no problem, I'll try LTPV when I get a chance. RE: the nvvp
>     stuff, pull my latest repository and check out the README,
>
>     $ git clone https://github.com/stevenovakov/learnOpenCL.git
>
>      I've fully scripted its use as per this article:
>
>     http://uob-hpc.github.io/2015/05/27/nvvp-import-opencl/
>
>     so just compile and run like so for a fast simple test:
>
>     $git checkout simple_events
>     $make clean program
>     $./profiling.sh -datasize=50 -chunksize=25
>     $nvvp cuda_profile.log
>
>     Be careful if you're using multiple contexts, I haven't fixed my
>     sed scripts to toss out unused devices in contexts and nvvp will
>     spit out errors.  Maybe, if you're interested, you can even help
>     me with my issues I'm having with NVIDIA's OpenCL implementation,
>     see here:
>
>     https://www.khronos.org/message_boards/showthread.php/11619-NVIDIA-Multi-Device-Command-Queue-Concurrency-Issue
>
>     Cheers,
>
>     *Steve *
>     On 7/31/2015 5:03 PM, Simon DENEL wrote:
>>     * Do you have a link about using nvvp while programming in OpenCL
>>     ? I am quite interested!
>>     * Included is an "old" copy of LTPV: could you try it and tell me
>>     if it works better?
>>
>>     Have a nice week-end,
>>     Simon DENEL
>>
>>     On Fri, Jul 31, 2015 at 11:36 AM, Steve Novakov
>>     <stevenovakov@gmail.com <mailto:stevenovakov@gmail.com>> wrote:
>>
>>         Yes sir, thanks for replying.
>>
>>         I actually found another solution using nvidia's driver
>>         logging and nvvp (it's a kluge but it works *great*). That
>>         being said I'm open to helping debug your tool, the more
>>         tools the merrier.  I like that it exports to HTML, I may use
>>         it to put up logs on my website in a series of OpenCL articles.
>>
>>         *Steve *
>>         On 7/30/2015 3:01 PM, Simon DENEL wrote:
>>>         Hello Steve,
>>>         Can you confirm me I am using the right email address?
>>>         Thanks,
>>>         Simon DENEL
>>>
>>>         On Thu, Jul 30, 2015 at 7:49 AM, stevenovakov
>>>         <notifications@github.com <mailto:notifications@github.com>>
>>>         wrote:
>>>
>>>             Hello, I'm not sure how else to contact you, so I'm
>>>             leaving this here on the latest commit...
>>>
>>>             I recently pulled and installed LTPV to help me do some
>>>             analysis on some multi-gpu software I'm writing. Here is
>>>             the result of the following sequence of commands, ran
>>>             from the root directory of the LTPV repo (I put it in
>>>             ~/LTPV):
>>>
>>>             $ ./waf configure
>>>             Setting top to : /home/tiger/LTPV
>>>             Setting out to : /home/tiger/LTPV/build
>>>             Checking for 'gcc' (C compiler) : /usr/bin/gcc
>>>             Checking for 'g++' (C++ compiler) : /usr/bin/g++
>>>             Checking for header dlfcn.h : yes
>>>             ('Mode: ', 'debug')
>>>             ('Flags: ', ['-O0', '-g', '-ggdb', '-w', '-std=c99'])
>>>             'configure' finished successfully (0.128s)
>>>
>>>             $ ./waf
>>>             Waf: Entering directory |/home/tiger/LTPV/build'
>>>             ('Mode: ', 'debug')
>>>             ('Flags: ', ['-O0', '-g', '-ggdb', '-w', '-std=c99'])
>>>             [3/4] cxx: src/opencl-hook.cc ->
>>>             build/src/opencl-hook.cc.1.o
>>>             [4/4] cxxshlib: build/src/common.cc.1.o
>>>             build/src/init.cc.1.o build/src/opencl-hook.cc.1.o ->
>>>             build/libltpv.so
>>>             Waf: Leaving directory|/home/tiger/LTPV/build'
>>>             'build' finished successfully (0.816s)
>>>
>>>             $ sudo ./waf install
>>>             [sudo] password for xxxxxx:
>>>             Waf: Entering directory `/home/xxxxxx/LTPV/build'
>>>             ('Mode: ', 'debug')
>>>             ('Flags: ', ['-O0', '-g', '-ggdb', '-w', '-std=c99'])
>>>
>>>               * install /usr/local/bin/ltpv (from bin/ltpv)
>>>               * install /usr/local/include/ltpv.h (from src/ltpv.h)
>>>               * install /usr/local/lib/libltpv.so (from
>>>                 build/libltpv.so)
>>>               * install /usr/local/share/ltpv/inc/favicon.png (from
>>>                 share/ltpv/inc/favicon.png)
>>>               * install /usr/local/share/ltpv/inc/functions.js (from
>>>                 share/ltpv/inc/functions.js)
>>>               * install /usr/local/share/ltpv/inc/getQueryString.js
>>>                 (from share/ltpv/inc/getQueryString.js)
>>>               * install /usr/local/share/ltpv/inc/jquery.min.js
>>>                 (from share/ltpv/inc/jquery.min.js)
>>>               * install /usr/local/share/ltpv/inc/jquery.xml2json.js
>>>                 (from share/ltpv/inc/jquery.xml2json.js)
>>>               * install /usr/local/share/ltpv/inc/main.js (from
>>>                 share/ltpv/inc/main.js)
>>>               * install /usr/local/share/ltpv/inc/minus.png (from
>>>                 share/ltpv/inc/minus.png)
>>>               * install /usr/local/share/ltpv/inc/minus2.png (from
>>>                 share/ltpv/inc/minus2.png)
>>>               * install /usr/local/share/ltpv/inc/plus.png (from
>>>                 share/ltpv/inc/plus.png)
>>>               * install /usr/local/share/ltpv/inc/plus2.png (from
>>>                 share/ltpv/inc/plus2.png)
>>>               * install
>>>                 /usr/local/share/ltpv/inc/profiling_model.xml (from
>>>                 share/ltpv/inc/profiling_model.xml)
>>>               * install /usr/local/share/ltpv/inc/shortcut.js (from
>>>                 share/ltpv/inc/shortcut.js)
>>>               * install /usr/local/share/ltpv/inc/sorttable.js (from
>>>                 share/ltpv/inc/sorttable.js)
>>>               * install /usr/local/share/ltpv/inc/style.css (from
>>>                 share/ltpv/inc/style.css)
>>>               * install /usr/local/share/ltpv/index.html (from
>>>                 share/ltpv/index.html) Waf: Leaving directory
>>>                 `/home/xxxxxx/LTPV/build' 'install' finished
>>>                 successfully (0.181s)
>>>
>>>             Great. Now when I run my program with $lptv "./program",
>>>             it segfaults somewhere in this function.
>>>
>>>             void OclEnv::NewCLCommandQueues()
>>>             {
>>>             this->ocl_device_queues.clear();
>>>
>>>             for (uint32_t k = 0; k < this->ocl_devices.size(); k++)
>>>             {
>>>             std::cout<<"Create CommQueue, Device: "<<k<<"\n";
>>>
>>>             |this->ocl_device_queues.push_back(
>>>             cl::CommandQueue(this->ocl_context,
>>>             this->ocl_devices[k])); |
>>>
>>>             }
>>>             }
>>>
>>>             I guessed correctly that it crashes when it tries to use
>>>             your overloaded "clCreateCommandQueue" because the
>>>             normal output looks like
>>>
>>>             ...
>>>             Create CommQueue, Device: 0
>>>             Create CommQueue, Device: 1
>>>             ...
>>>
>>>             and when running w/ ltpv I see
>>>
>>>             ...
>>>             Create CommQueue, Device: 0
>>>             Segmentation fault
>>>             ...
>>>
>>>             Here's the gdb dump:
>>>
>>>             Create CommQueue, Device: 0
>>>
>>>             Program received signal SIGSEGV, Segmentation fault.
>>>             0x00007ffff7bb9f2c in std::_Rb_tree, std::_Select1st >,
>>>             std::less, std::allocator > >::_M_begin (this=0x20)
>>>             at /usr/include/c++/4.8/bits/stl_tree.h:502
>>>             502 { return
>>>             static_cast<_Link_type>(this->_M_impl._M_header._M_parent);
>>>             }
>>>             (gdb) bt
>>>             #0 0x00007ffff7bb9f2c in std::_Rb_tree, std::_Select1st
>>>             >, std::less, std::allocator > >::_M_begin (this=0x20)
>>>             at /usr/include/c++/4.8/bits/stl_tree.h:502
>>>             #1 <https://github.com/LTPV/LTPV/issues/1>
>>>             0x00007ffff7bbc382 in std::_Rb_tree, std::_Select1st >,
>>>             std::less, std::allocator > >::lower_bound (
>>>             this=0x20, __k=@0x7fffffffd980: 15337424) at
>>>             /usr/include/c++/4.8/bits/stl_tree.h:879
>>>             #2 0x00007ffff7bba2dd in std::map, std::allocator >
>>>             >::lower_bound (this=0x20, __x=@0x7fffffffd980:
>>>             15337424) at /usr/include/c++/4.8/bits/stl_map.h:864
>>>             #3 0x00007ffff7bb8d22 in std::map, std::allocator >
>>>             >::operator (this=0x20, __k=)
>>>             at /usr/include/c++/4.8/bits/stl_map.h:481
>>>             #4 0x00007ffff7bb6c1d in ltpv_addStream
>>>             (idStream=15337424, idDevice=6601488,
>>>             name=0x7fffffffd9f0 "Queue 1")
>>>             at ../src/common.cc:121
>>>             #5 0x00007ffff7bc1392 in clCreateCommandQueue
>>>             (context=0x64bcb0, device=0x64bb10, properties=0,
>>>             errcode_ret=0x7fffffffda6c) at ../src/opencl-hook.cc:342
>>>             #6 0x00000000004076b1 in
>>>             cl::CommandQueue::CommandQueue(cl::Context const&,
>>>             cl::Device const&, unsigned long, int*)
>>>             ()
>>>             #7 0x000000000040511d in OclEnv::NewCLCommandQueues() ()
>>>             #8 0x00000000004021e2 in main ()
>>>
>>>             So it fails in your code starting at opencl-hook.cc:342
>>>             which is:
>>>
>>>             |ltpv_addStream((size_t) queue, idDevice, queueName); |
>>>
>>>             then it goes into ltpv_addStream until line 121 which is:
>>>
>>>             |ltpv_devices[idDevice]->streams[idStream] = newStream; |
>>>
>>>             and after that point it fails in stl stuff. Please help!
>>>             I want to use your code!
>>>
>>>             —
>>>             Reply to this email directly or view it on GitHub
>>>             <https://github.com/LTPV/LTPV/commit/c7fa54d6d278d3675c96a999d91566ef5e91a80e#commitcomment-12436590>.
>>>
>>>
>>
>>
>
>


Thank you!


It is important for me to put it out there: This is purely a learning exercise! I do not intend to commercialize this or distribute this in any way. I only intend to learn how to make an android app in a systematic way (given that the first app I made was chaotic madness).


There's no player reference to know when the session ends. Also, what happens if this is not sent to a player?


not sure if it's worth it since this will be moving into nav transition soon, but might want to consider factoring all this into a category on `UIViewController`


Yeah, that's changing in the impl since I realized that didn't work -- probably going to have it be like last 500 or something callbacks are kept? Or callbacks are kept until executed maybe, with a cap of like 1000?


`ng-src`


This was fixed in #17!
Sorry about that guys.


What's the behavior of this equation? `(d / 30) < rng`

Feels like it's not linear in respect to the number of days in the month and instead, it increases in likelihood over time. I'm so far removed from graphing things that I can't figure out how to graph this in a way that makes it clear whether that feeling is right or not.


`ng-model` inside `<p>`?


It should be user now


camelCase AND hyphenated!


What is the purpose of this event?


But why?


rollback to last stable version because new jquery messed up the smart menus and fonts


prompt={'Enter Value of a :','Enter Value of b:','Enter Value of c'};
title='Shri_Dharacharya_Formula';
answer=inputdlg(prompt,title);
a=answer{1};
b=answer{2};
c= answer{3};
prompt={'do you want to find roots'};
title='action to perform';
answer=inputdlg(prompt,title);
action=answer{1};
B=str2num(b);
B=[B];
C=str2num(c);
C=[C];
A=str2num(a);
A=[A];
disp(A);
disp(B);
disp(C);
D=((B.^2)-(4*C*A));
x1=((-B)+ (sqrt(D)))/(2*A);
x2=((-B)- (sqrt(D)))/(2*A);
disp('DISCREMINENT IS ');
disp(D);
disp('your roots are ');
disp('x1=');
disp(x1);
disp('x2=');
disp(x2);


I think the rename is unnecessary: RunUntilSignaled is fine; reconecting is an implementation detail.

Also, the new comment doesn't say anything.


Adição dos pontos por comentário.


Hmm I sort-of see. Perhaps it's something that should exist in `sbt-scalajs`.


Thanks @lennart, + that's a handy key combo!


Redirecionando para o evento na qual o comentário em questão pertence.


Luckily the digest loop makes this work but the async is all over the place here.


Might be a better, more automated way to generate this menu.


1: M110 missing component name


Instanciando um comentário para o usuário que estiver logado (current_user).


As in: return resolve(users) instead of return resolve(users.id)?


Cool IIFE, but is it necessary? :thought_balloon:


Remoção do campo user_id não editado diretamente.


There is a `date` filter built-in to Angular ... could be useful but not sure.


Adicionando o contador de comentários.


... there is no pb with this configuration (I assume it). But it's without the condition and with the condition... I'm stuck ( ":" is a reserved character :'( ). If you have an idea, I'll be very very very happy :p


@jonathancadepowers why not ^^. For me, the test reporter is required on the fly in .travis in the build process for loosely dependence.


@gamaanderson @tjlang @pfhein Here's the next round of commits to clean a few things up.
The documentation was updated, though there are some issues that need to worked out as more warnings were thrown as mentioned in #29 .


LGTM


1: M111 unrecognized component name: version


Iniciando o formulário para comentários.


no, user instead of users. since first returns an object, it shouldn't be plural.

My apologies for commenting in a commit instead of the PR.


Great idea!  Thanks, william


No contributions since April?  Slacker..


Always add deleted copy constructor and assign operator if you want to make this class non copyable (which is mostly what you want)