Is there a reason you didn't link to https://github.com/purpleidea/oh-my-vagrant/issues/78#issuecomment-94166486 in this PR?

I'm :-1: on this change for the reasons above.  Note that Fedora 22 (both Atomic and via `yum install docker` on mainline) do not include the `docker` group, so this would be undoing an intentional change made upstream.

```
[root@localhost ~]# cat /etc/fedora-release
Fedora release 22 (Twenty Two)
[root@localhost ~]# rpm -q docker
docker-1.6.0-0.2.rc6.fc22.x86_64
[root@localhost ~]# getent group docker ; echo $?
2
[root@localhost ~]#
```


91244204-HeatExchangerAirToAirSensibleAndLatent-Supply-Air-Outlet-Temperature-Control-problems-#4794 (mjwitte) - 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)


Thanks for pointing this out. Done.

On 04/21/2015 02:08 PM, Jeremy Mikola wrote:
>
> If this depends on the pymongo 3.0 API, then I believe |pymongo>=2.8|
> needs to be updated in |setup.py| (assuming that's the only place
> where dependencies are tracked).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/10gen/mongo-orchestration/commit/6e9f8e0eac17695cbfb681e78a51f02c27ce9213#commitcomment-10834399>.
>


Adding the docker group is something easily accomplished in provisioning scripts if it's desired by the user, right?

Did you investigate prefixing `docker` invocations with `sudo`?


Any chance the ICancellable would need to be wired up to check if the inner disposables are disposed?  Any cases where something other than the component may have a pointer to the inner disposables?


When this is `40px` then the entered text is exactly over the label.

![search-text-40px mov](https://cloud.githubusercontent.com/assets/211578/7262761/a8d6992c-e841-11e4-8f93-9a6afb89b0fc.gif)
> 40px

![search-text-45px mov](https://cloud.githubusercontent.com/assets/211578/7262762/a8d700ec-e841-11e4-8d2c-94f60bc41a12.gif)
> 45px


These are `timing functions` - Easing curves translates to a `timing-function` in code.
The explanation of the map is at the bottom of this doc where the "Animation Library Implementation" is explained.


Jag hittade varför det buggar på mobile-safari. Nu försöker jag lista ut varför vi flippar hela tabellen 180 grader :S Har du lust att kommentera Pål?
![mind-blown-2](https://cloud.githubusercontent.com/assets/442870/7262789/a15228b6-e87c-11e4-9af6-a2de1b89455c.gif)


Agreed. Thanks for the catch. I added that in to troubleshoot some alignment issues and forgot to take it out.


should it reject it if the service account is not resolved, or should the kubelet just not run the pod?


there is nothing that enforces that you do not delete the service account at a later point, so I am not sure I see the point in this check at admission.


ok I will remove this.


This seem to have caused the System tests to hang:

https://wrench.mono-project.com/Wrench/ViewWorkTable.aspx?lane_id=10&host_id=8&command_id=42


iOK


Actually not independent, because HasLocals implies isContainer


The answer is: Hope.


@Tux64 Also, I see you forked MySurf... are you planning on helping out? You are always welcome to help with the development! :smiley:


I said iOK


I'm fine with this change, the http://docs.vagrantup.com/v2/boxes/base.html mentions it.


wait, you have changed it to a Switch, so this description is not correct here!


i asked @gokulavasan  about this, right now we emit all user metrics under HANDLER context, do we have a way to include method in the context while emitting ?


This was changed to percentages in a later commit.


I can't believe I set this as a string... fix here ~> 4656358dc37258e86983d271b1d14d045f2caebf


The first option seems really interesting to do and hence I shall go with the second option :P


This is the standard solution I found after searching for how to escape jquery selector strings. I'm not sure whether it would be ok to do the extra escaping that your solutions imply.


Pods should not be able to run without a valid service account - some admission controller has to limit or default it.

----- Original Message -----
> there is nothing that enforces that you do not delete the service account at
> a later point, so I am not sure I see the point in this check at admission.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/liggitt/kubernetes/commit/0b75524e6580eafb3612c0686e8af9d8d4015f9a#commitcomment-10834596


here's a ; too many.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-AddressSanitizer: OK (339 of 340 tests passed)

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


Do you think we need this in the try statement as well? Just like you did above in debug mode.


wtf


Also talk about how block scope produces locals on demand, and often needs to be cleaned up in incremental


you should directly use the updateStatus() methods available on the BaseThingHandler


dito


Looks good.


Should be `<= 0` or `!= 1`.
Negative return value (error) may mean that file is not seekable.


Yes, I'm planning to help with the OS X/Linux side mainly, but I'd be happy to work on the Windows side as well.


That would probably make sense. I was using DeviceRepositroy.SaveDevice() as a template for the debugging


There are no properties for `auto-translate`, unlike all the other `auto-*` operations


Should you have done a compile-time conditional here?


getSrcColor() and getMode() are both marked @hide on the android classes, so they are not available in the API... this shadow should really be preserved as tests depend on this behaviour.

We should probably have the tests depend upon the sdk at compile time rather than the aosp jar as it would prevent this kind of issue.

Is this something you'd consider reinstating? Or should we just kill the tests that used these methods?


@cgwalters I didn't mean to not link to the change. But the more important issue is here:
https://github.com/purpleidea/oh-my-vagrant/issues/78#issuecomment-94188247

I am fine with the docker group not being part of atomic or fedora, etc... But this is for the *devel* *vagrant* version... Which those security arguments don't apply to.


And is there a way to access to the list or array of input parameters to iterate over it?


Eta reduce


This should guarantee that there is always a `_site` folder to serve (without any premature calls to `jekyll_build()`)


when is the release?


I think this should be `/home/vagrant/yournextmp/yournextmp-popit`


@Glenpl Już fajniej to wygląda, skup się teraz na enkapsulacji – w ten sposób wydzielisz nieco logiki z Sapera. W komentarzach podpowiem co mi się rzuca w oczy na pierwszy ogień.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-ThreadSanitizer: OK (340 of 340 tests passed)

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


I don't think you need the `abs` after `sqrt`...


why do you need *three* methods for this? one can just call the other, right?


Oh hey and `sqrt_ . unsafeProb` is slightly more efficient than `unsafeProb . sqrt`


Although I'm not actually sure if this works...note that Kickstart files have an entirely separate verb `rootpw` for this, and for most images we lock the root password.  Did you test it?


@cecilia-donnelly Well this is generating the json that will be passed to the browser.  The browser/javascript doesn't have access to the ORM object, just the json that we pass to it.  So, in our AJAX'y code, we want to give a nice representation of the address object, but all we had are the "bits" of the address before I added this change... the street number, the zip code, etc.  We already produce a "nice" string representation for the python version, which can be seen in dropdowns, so I figured it would be nice to allow our Javascript to have access to that too, so when we serialize the object to send it across the wire, I added that string representation.

This does mean that javascript gets a "field" that isn't actually a row in the database.  I wonder if @xray7224 is okay with that.  I named it "__str__" to make it obvious.  We could do it in Javascript also, but it would be duplicating code, and "Don't Repeat Yourself" is a good mantra.


not needed, utime_t() constructor zeros it


done


(And finally if this does work I'd like to see the change happen in Fedora Vagrant boxes in https://git.fedorahosted.org/git/spin-kickstarts.git as well)


And the docker sudo prefixing has nothing to do with this. The errors are happening because of the code in: https://github.com/mitchellh/vagrant/blob/master/plugins/provisioners/docker/cap/linux/docker_configure_vagrant_user.rb#L11

That's *out of my control*.
Eventually long-term (not short-term, which is what this issue is about) we should have an atomic/ directory here: https://github.com/mitchellh/vagrant/tree/master/plugins/guests similar to how coreos does.

But that's an issue for a different day.


Perhaps a comment is warranted for VisitLambda<T> to establish why we should not be recursing into lambda expressions.


Something broke:

```
nteger@1.0.4, debug@2.1.3, detect-indent@3.0.1, ast-types@0.7.6, repeating@1.1.2, output-file-sync@1.1.0, minimatch@2.0.4, source-map-support@0.2.10, source-map@0.4.2, regexpu@1.1.2, regenerator@0.8.22, core-js@0.8.4)
[14:32:56] Using gulpfile /Volumes/User Data~/Projects/-dev-null/www/gulpfile.js
[14:32:56] Starting 'images'...
[14:32:56] Finished 'images' after 5.69 ms
[14:32:56] Starting 'scripts'...
[14:32:57] Starting 'styles'...
[14:32:57] Finished 'styles' after 46 ms
[14:33:01] Browserify error Cannot find module 'react-inlinesvg' from '/Volumes/User Data/Users/pbhowmick/Projects/-dev-null/www/src/scripts'

/Volumes/User Data/Users/pbhowmick/Projects/-dev-null/www/gulpfile.js:72
  this.end()
       ^
TypeError: Object #<Readable> has no method 'end'
    at Readable.error (/Volumes/User Data/Users/pbhowmick/Projects/-dev-null/www/gulpfile.js:72:8)
    at Readable.browserifyError (/Volumes/User Data/Users/pbhowmick/Projects/-dev-null/www/gulpfile.js:76:9)
    at Readable.emit (events.js:95:17)
    at Labeled.<anonymous> (/Volumes/User Data/Users/pbhowmick/Projects/-dev-null/www/node_modules/browserify/node_modules/read-only-stream/index.js:27:44)
    at Labeled.emit (events.js:95:17)
    at Labeled.<anonymous> (/Volumes/User Data/Users/pbhowmick/Projects/-dev-null/www/node_modules/browserify/node_modules/labeled-stream-splicer/node_modules/stream-splicer/index.js:132:18)
    at Labeled.emit (events.js:117:20)
    at Deps.<anonymous> (/Volumes/User Data/Users/pbhowmick/Projects/-dev-null/www/node_modules/browserify/node_modules/labeled-stream-splicer/node_modules/stream-splicer/index.js:132:18)
    at Deps.emit (events.js:95:17)
    at /Volumes/User Data/Users/pbhowmick/Projects/-dev-null/www/node_modules/browserify/node_modules/module-deps/index.js:345:30
```


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (340 of 340 tests passed)

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


I think this should go anywhere we do clear_object_contexts().  In fact, given that, it should perhaps be named something like object_contexts_start or something, since it may not be related to the peer interval if the primary didn't change..


@Tux64 That's great! We could definitely use the help, especially for the OS X and Linux versions. Where do you think you will be starting?


Ah, I guess that is how it looks natively. I just thought it looked weird touching. We can move it back when we re-center the search bar


Good pickup, this is fixed by #79


TBH, I didn't test this, because I have no way to :(
I'm happy to if someone explains how.


why not just change consumer code to something like
```
insQuery.unbind().bind(pojo)
```


Change isBlockScopeContainer to isBlockScope


Similarly in server:

```shell
> ack.mo@1.0.0 babel-node /Volumes/User Data/Users/pbhowmick/Projects/-dev-null/server
> babel-node index.js

sh: babel-node: command not found

npm ERR! Darwin 14.3.0
npm ERR! argv "node" "/usr/local/bin/npm" "run" "babel-node" "index.js"
npm ERR! node v0.10.33
npm ERR! npm  v2.1.9
npm ERR! code ELIFECYCLE
npm ERR! ack.mo@1.0.0 babel-node: `babel-node index.js`
npm ERR! Exit status 127
npm ERR!
npm ERR! Failed at the ack.mo@1.0.0 babel-node script 'babel-node index.js'.
npm ERR! This is most likely a problem with the ack.mo package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     babel-node index.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls ack.mo
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Volumes/User Data/Users/pbhowmick/Projects/-dev-null/server/npm-debug.log
21 Apr 14:34:46 - [nodemon] app crashed - waiting for file changes before starting...
```


Hey!

I think copy & paste as a start so we don't have to deal with the file/profile management issues.  If the text box can be filled with the default profile/config, that would be ideal.

I'm playing with JSON data for loading the regex data, but ran into issues with non-ascii characters in the PDF text. Not sure why that didn't trip us up before, actually. Anyway, I pushed my changes to a branch - take a look if you want to leverage that.


@mobedi  try my latest commit, if there are still problems send me more information and i will try to solve the problem


I think all these instances of `cd /vagrant` should be `cd $HOME/yournextmp`


You can prob move these validations that are the same for client/location into a concern or module that both these models share.


Ah...the Vagrant Docker provisioner seems unnecessary.  Most of that can be really easily expressed in shell script or Ansible or whatever right?  And of course Kubernetes for scheduling across the cluster.

Can you drop the use of that provisioner?

Note again that the `docker` group is entirely orthogonal to Atomic - there is a goal to have a Vagrant box for Fedora 22 mainline, and this would also apply there.


needs comments explaining why this is being done, yes?


Typo. *Updated IPlayer.DoMove's parameter 'done' *from* Func to Action*


Why `:=` here?(also see l44)


Well now that I know that method exists, your going to have to excuse my abuse of this pattern all over the SqueezeBox binding ;-)


@dawsonlp

FYI,

I've set up AppVeyor jobs that build FreeTDS on Windows. See the http://thread.gmane.org/gmane.comp.db.tds.freetds/15907 mailing list thread for pointers to the relevant work.


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


@dlongley this is a bit rough but would it be worth cleaning up (or porting to optparse)?


fixed [here](https://github.com/pathikrit/-dev-null/commit/0fcb79f43cad62a4a70c073dc690fc9d39ef03cd#diff-30b745a2a488d38df94c35f548f87011L3) and in https://github.com/pathikrit/-dev-null/commit/005219014c720f52a5f34413b98fcffc8a010d09


@bummzack I think the file `code/SortableUploadField.php.orig` was accidentally committed >_<


This code is not indented correctly!  This makes it a lot harder to read :(

Emacs can help you with this.  In general, hitting tab on a line should indent it right.  Use http://web-mode.org/ if standard html-mode is not enough and you also want Django templating support.

Please never leave html unindented!  Please do nested indentation... this helps keep things consistent and especially *readable*.  Code is for humans!


Should this instead be ```if (keyserver == null)```? Getting a crash in the latest development build trying to instantiate a HkpKeyserver with a null host/port when searching for a key on an NFC token.


Are these just the versions we currently test on? Is there a possibility that lower versions could work?


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

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


@MacDada funkcje będące jakby poza samą logiką gry wrzuciłem jako metody planszy np. ustawienie całej planszy na 0, sprawdzenie czy dane pole jest częścią planszy.


BTW, if you find "<RET> <TAB>" tedious, you can simplify things by hitting "C-j", which both inserts a newline *and* indents.


I think I will start on the Linux side and port it over.


Since you are just checking a boolean this could be reduced to
```
if: Proc.new { |a| a.secure_domain? }
```
and
```
unless: Proc.new { |a| a.secure_domain? }
```
on the other one. Apparently (I just read about this) you can also interpolate a string like this
```
 if: "secure_domain?"
```


(Also I know this is an old commit, and this was removed, but I wanted to comment on this before I forgot.)


Hehe, looked like the check for that was temporarily disabled in grunt gate :). Already fixed tho.


Any reason not to use https:// everywhere?


@cgwalters Hehe, no, I can't drop the use of the docker provisioner... If I could, then this wouldn't be an issue :) Rewriting it as shell is _not_ a good idea.

I just want to reiterate... These are changes for vagrant things, not core atomic things. They need to be looked at under a different light.


Great find and solution. There would be an edge case that might make this a little tricky: If the parent element has any nodes that aren't of type `1`, the loop would grab the wrong element each time. That's because elements of other types wouldn't be transferred, and hence we'd always grab the same element at index `0`.

Another, slightly less performant solution would be to loop backwards and use `insertBefore()` instead of `appenChild`. I'd happily take stability over performance here, since this function only gets called once when the content of the wrapper changes. Really, that shouldn't happen too often and I'd rather have it behave as expected than tweak performance for an edge case.

Here's an example of the reverse loop:

```js
shinejs.Splitter.prototype.splitChildren = function(domElement, maskElement, wrapperElement, classPrefix) {
  var childNodes = domElement.childNodes;

  for (var i = childNodes.length - 1; i >= 0; i--) {
    var child = childNodes[i];

    // see https://developer.mozilla.org/en-US/docs/Web/API/Node.nodeType
    if (child.nodeType !== 1) {
      continue;
    }

    child.className += ' ' + classPrefix + 'letter';
    wrapperElement.insertBefore(child, wrapperElement.firstChild);
    this.elements.push(child);
  }

  maskElement.innerHTML = wrapperElement.innerHTML;
  maskElement.className = classPrefix + 'mask';
  wrapperElement.appendChild(maskElement);

  domElement.innerHTML = '';
  domElement.appendChild(wrapperElement);
};
```
If you could use that and make your commit a pull request I'd be happy to merge it into the master branch.


There are a couple of items from before (the interface type block for one) that need to be fixed I've noticed.


.. trying to do

pip install rdflib-jsonld
rdfpipe <url> -o json-ld | pyld --format json-ld --compact

(or faster.)

Please feel free to change any of these opts strings to suit your fancy.


I'll make another issue for needing to re-implement `python setup.py develop`


You might be interested in the very latest addition about status handling in ESH: https://github.com/eclipse/smarthome/blob/master/docs/sources/architecture/thing-status.md
This will actually break the use of getThing()setStatus() - so it might be a good idea to refactor this part!


``` diff
- function saveNewPerson(){
+ function saveNewPerson() {
```

Give those curly braces some breathing room!  It's much better style.


@rozele - Good point; will add.


this script could become a function taking betalist, i_phase, i_subject, i_cond as arguments, than everybody could use it, it could thus be moved to the master branch...


Hi, is there a reason you you decided to leave jquery as a dependency in bower.json?

Unfortunately it's a deal-breaker for my organization -- no jquery allowed.  To verify it's not actually needed at runtime I manually removed the jquery dependency from my local d3pie/bower.json. It worked!

Is there a version out there with jquery removed from bower.json?


When you have something like this (bunch of userdef fields), may be best to place them in a vertical table structure.


@rozele - Relaxing the cref usage a bit for non-public members :-).


The docker provisioning code is fairly close to garbage IMO...I mean, look at https://github.com/mitchellh/vagrant/blob/master/plugins/provisioners/docker/cap/redhat/docker_start_service.rb#L8


Thanks to IntelliJ :notes:


hm i actually think these are minimum versions for everything here with the possible exception of pytables


Ya, but don't you still need gulp. Just tried on a fresh machine:
```
command not found: gulp
```


:+1: looks like a good move  :sushi:


cool


I didn't know you could force it like that.


OK, this looks good.


Basically the Docker provisioner *is* shell script, just wrapped in lots of Ruby block indentation.


develop (lefticus) - x86_64-MacOS-10.9-clang: OK (340 of 340 tests passed)

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


nit: `for current_count in xrange(factor):`


For serialization see RKSerialization, RKMIMETypeSerialization.
Releases prior to iOS7 now reportedly <3% of all devices so probably not worth worrying too much.
Will try and get notes together in next couple of days and maybe we can meet up online to discuss further.

> On 22 Apr 2015, at 4:27 am, Oli Griffiths <notifications@github.com> wrote:
>
> Awesome. With a protocol, if that's required then users can add their own iOS 6 support if needed.
>
>
>
>
>
> > On 21 Apr 2015, at 14:56, Samuel E. Giddins <notifications@github.com> wrote:
> >
> > No, we'll either go iOS 7+ or iOS 8+.
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> —
> Reply to this email directly or view it on GitHub <https://github.com/RestKit/RestKit/commit/5225cb76a788730918e499f227f7079f2c92a66a#commitcomment-10831894>.
>


@Gamecrafter :shit:
Can you please help with pull requests?


@Dunbaratu But it's useful once you know how it works: you can respond to the feedback and make changes without having to close and redo your PR.


This looks reasonable. I'll give it a test here before merging.


now this does not work:

```
assert.Equal(t, 9, 9.0)
```


Anyways as far as treating the Vagrant box as "for developers may be insecure" as a distinct thing from the other deliverables...I understand the point.  But on the other hand, "conveniences" like the `docker` group have a tendency to too easily leak into production environments as developers are used to them locally.


@cgwalters hehe... I won't argue about the merits of the code quality. I assume it is garbage until proven otherwise. Unfortunately it _does_ work, and I use it in oh-my-vagrant, as do many others in various different places. So convincing everyone and omv to rewrite the provisioner as a shell script isn't a good answer.

Either we want atomic+vagrant to work together, or we don't. If we do, then we need to patch the atomic _vagrant_ box. This has nothing to do with upstream vagrant.

In parallel, if we refuse or don't want to patch it here in this kickstart, then with any luck, maybe virt-builder will eventually grow those features, but that's a year away.


This needs to call `Driver::isBlocked` otherwise you'll skip all the memory futures


Naw it got kicked out for mass noncompliance with iOS updates


According to [this](http://pubs.opengroup.org/onlinepubs/007908775/xsh/systypes.h.html)

> The type ssize_t is capable of storing values at least in the range [-1, SSIZE_MAX].


Also I think my dev account expired?


oops


![screenshot 2015-04-21 14 48 05](https://cloud.githubusercontent.com/assets/15744/7263324/753a66ae-e835-11e4-9843-a5af4f98ab78.png)


A devel vagrant box with a docker group causes it to leak into a prod environment with a different machine image... I don't understand this :(


I'd rather not introduce another method to ACRA.java
The functionality can be accomplished via #setLog.
Also #disableLogging implies that you can enable it which you can't without calling #setLog


This looks good to me. If you can add a test for it and submit a pull request, I'll merge it. Obviously the test won't be able to assert an exact expected expiration, but you should be able to assert something within a range.


91244204-HeatExchangerAirToAirSensibleAndLatent-Supply-Air-Outlet-Temperature-Control-problems-#4794 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (623 of 623 tests passed)

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


To explain vertical, means there's an id that ties them all together, but then a column such as:
categoryuser (would hold things like 'drseenuser')
categorytitle (hold the visible title)
categorydatetime (hold the timestamp)
Then can support any category or new category automatically.
(just thinking out loud, will have more thoughts on this after review all the code)


@wluyima are the @since annotations correct?


Please use EqualValues.


As a funny side note about the "garbage docker vagrant code" it's actually something well-known as a docker issue. @rwmjones and I had the same issue with docker, and decided on the unfortunate (but only available) fix: https://github.com/purpleidea/vagrant-builder/blob/master/v7/files/docker.sh#L11


This piece belongs in 4_2_0-to-4_2_1_upgrade.sql (not the patch.sql file).
Also need to add to database.sql file


Como o professor especificou no site #if(nomedoteste) ... #end


Acho que pode estar confundindo GUI com Controller aqui, uma vez que o BibTex file seria um ação chamada pela GUI.


My Vagrant git repo using Atomic works fine without the Docker provisioner...where by "fine" I mean "starts a Kubernetes cluster".  I can certainly imagine using Vagrant may have different needs and might find some of the Vagrant Docker provisioning convenient.

What specifically are you using it for?  Can you link to the code?

The kickstart doesn't actually need to be patched for this right?  Couldn't you run a very simple shell provisioner to add the group *before* the Docker provisioner executes?

As far as the leakage, it's around developers writing things like Ansible/Puppet scripts and expecting that because the `docker` group worked on a devel box that it'll be available in production.  We did have some pain when we took it out of Fedora by default but the decision was made for good reasons.


`my-post` tag

?


I'd probably place the settings in the globals editor.(are the settings temporary or more permanent; any examples?)


Ahh, man, that's embarrassing. Yeah it's exactly as @Dunbaratu described.

I've found an online Git course I'm going to work through on the weekend. No more pull requests from til after then! So embarrassing - all my crappy hacked together code from figuring things out posted for all to see.


spurious change


fixed. when we add ci these problems will become glaringly obvious.


good job


This would be a good idea. Very simple to get that going; let us know if need guidance on doing this.


@austinjames314 dont worry :) this is a safe place. we all do goofy stuff.


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


Problem should go away if we replace all `ssize_t` with `int` in this function.


@AmrEmara 3ashan 7asab mana fahem en el opcode bta3 el jr zay bta3 el add "000000" bas el jr mn awel 11th bit lel a5er feeha "0 0000 0000 0000 0000 1000" w da el beyfara2 el add 3an el jr


You're right, it should. I'll submit a pull request right away. Thank you!


> What specifically are you using it for? Can you link to the code?

Here is one use: https://github.com/purpleidea/oh-my-vagrant/blob/master/vagrant/Vagrantfile#L946 You can ^F to search for more docker related things....

> The kickstart doesn't actually need to be patched for this right? Couldn't you run a very simple shell provisioner to add the group before the Docker provisioner executes?
Although ugly, actually, this is a reasonable workaround. I'll test this tonight and see if that works.

> As far as the leakage, it's around developers writing things like Ansible/Puppet scripts and expecting that because the docker group worked on a devel box that it'll be available in production. We did have some pain when we took it out of Fedora by default but the decision was made for good reasons.
I understand. That's why life isn't perfect, and vagrant isn't sufficient for testing :) Shouldn't be an issue on Atomic though, since docker is supposed to "just work" and shouldn't need configuring. ... Right?


I like that you got this in here first.


You can use `ImmutableSortedSset.copyOf(String.CASE_INSENSITIVE_ORDER, Splitter.on(",").trimResults().omitEmptyStrings().split(getProperty(key)))`.


Ditto.


Shouldn't there be
```
progress.setValue(step)
```
somewhere here?


@abaranau suggested that I add this here even though we do not emit with the method tag yet.


Sim,  e criar duas variáveis que receberiam essas duas citation-keys para checar, já modificando isso no cenário.


@rsbondi what's not clear?


Why did you put here this instead of using the [reference examples](https://github.com/asciidoctor/asciidoctor-doctest/tree/master/data/examples/asciidoc)? o.O


Yeah, I originally had it spaced at 45px since I thought it looked weird when it touched then made it line up to position it properly and figured it made more sense to be touching.


:heart: can't hate the rd


I though this was already clear. In my last comment in main conversation, I said I was going to change to int. Haven't you received email notification?


I agree the value should be set if omitted, but  I am not sure I see the value in resolving it.  I am ok if it's based on a cache.Store, but I don't really want every pod create to require another call.

Sent from my iPhone

> On Apr 21, 2015, at 5:22 PM, Clayton Coleman <notifications@github.com> wrote:
>
> Pods should not be able to run without a valid service account - some admission controller has to limit or default it.
>
> ----- Original Message -----
> > there is nothing that enforces that you do not delete the service account at
> > a later point, so I am not sure I see the point in this check at admission.
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> > https://github.com/liggitt/kubernetes/commit/0b75524e6580eafb3612c0686e8af9d8d4015f9a#commitcomment-10834596
> —
> Reply to this email directly or view it on GitHub.
>


Why is that necessary?


While this constructor attempts to set the `$baseUrl` property of the JavascriptRenderer to a sensible default, it only does so when `$baseUrl` is not specified upon instantiation.  Even when this is the case, there is already a means of replacing this value, as can be seen here: `https://github.com/maximebf/php-debugbar/blob/master/demo/bootstrap.php`


need to html escape output.
use the text() function here:
echo text($appointment['lname'] ... etc


text()


@PyLearner, you explained what you did, not why you did it. You have to explain why this change is necessary in the commit message. I can't accept the patch like this.


ehhh, I'm pretty sure this would break some stuff down the road (especially for iOS builds).

The recommended way is to set the variable CMAKE_OSC_ARCHITECTURES, either on the command line :  `"-DCMAKE_OSX_ARCHITECTURES=x86_64;i386"` or with a call to `set(....)`.

In order to make it easier to specify maybe we should add an `option()` ? (http://stackoverflow.com/questions/5998186/cmake-adding-command-line-options)


Yes!  I usually use emacs to help me indent correctly, but I think I'm missing a js+html mode, or something, so it wasn't indenting properly here and I didn't correct it.  Thanks for the reminder, @cwebber!


Making this unconditional breaks the build on OSX and any other system where "GUNInstallDirs" is not installed.


No `System.out.println` please.


Deveria está escrito "^I am ..." e não I'm, isso está atrapalhando a expressão.


`status.getPath().toString()` is not inexpensive. Better to call it once and store the result into a string.


Can we move this image to our blog instead?


looked like a typo, what is the intention of the dot notation?


agreed


ATOS.pdf deverá ser escrito no cenário, e deverá ser corrigido também a palavra "aquirvo"


This can actually be removed.  I only added it since I thought I needed to define it to use it as a sort field.


Hmm.  Testing is hard.  I guess we do this until we see different problems.

This should choose a random suffix rather than always using the same profile directory, even if we clean it.


Esse cenário está muito grande, deveria ser dividido em mais de um cenário.


When I clicked through to the class, I was probably looking at the decompiled code, so I didn't realize that they weren't part of the SDK :smile: I'll bring the methods back.

> We should probably have the tests depend upon the sdk at compile time rather than the aosp jar as it would prevent this kind of issue.

We could try it. Making the sdk jars available to Maven is such a pain in the ass, though :)


Is it better to make the values configurable and also provide default values?


I wanted to make sure Travis was OK with it first, but yes, I will add
those once the tests pass.
On Apr 21, 2015 4:37 PM, "Seth Tisue" <notifications@github.com> wrote:

> needs comments explaining why this is being done, yes?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NetLogo/Tortoise/commit/681daf7cc3b97b0d3a187601dcbbcfc64c996237#commitcomment-10834914>
> .
>


needs to be escaped, so would wrap entire string with text()


Yeah, I sent this post to preview just to see if an offsite hosted image would work.


text()


Creio que deverá ser escrito a classe que está sendo chamado o metodo fileNoExist(...), como por exemplo Periodico.fileNoExist(fileName)


This is a case where I think the checkstyle rule causes us to write code that is less clean.


@cwebber, oh I see.  Thanks!


Does it deoptimize only V8 or other engines too?


Would be nice to backport the test, too: https://github.com/rails/rails/commit/1555a1800ea550a991eb57ce1ec8236bdba0365a#diff-326db9b657d24c33f43248ff242878ee but I didn't find it in journey…


text and translate minutes


You need to use CLR_CMAKE_PLATFORM_UNIX here so that it works on OSX too.


Yay! The best clearfix.


Still throwing `RuntimeException`? I remembered you already fixed this. Maybe due to the squashing?


Yes it it it make the print_r easier to read I will remove it


Same.  The original goal was that a cluster admin could set a default service account name, and that you wouldn't have to look it up to assign it.  There will be a security context admission controller or initializer that sets the security context off the service account (or combined) which would handle that.  It's possible that we might want a franken-admitter for service account + security context that sets a particular policy (add secret to pod, assign service account, enforce default security context for service account).

----- Original Message -----
> I agree the value should be set if omitted, but  I am not sure I see the
> value in resolving it.  I am ok if it's based on a cache.Store, but I don't
> really want every pod create to require another call.
>
> Sent from my iPhone
>
> > On Apr 21, 2015, at 5:22 PM, Clayton Coleman <notifications@github.com>
> > wrote:
> >
> > Pods should not be able to run without a valid service account - some
> > admission controller has to limit or default it.
> >
> > ----- Original Message -----
> > > there is nothing that enforces that you do not delete the service account
> > > at
> > > a later point, so I am not sure I see the point in this check at
> > > admission.
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > > https://github.com/liggitt/kubernetes/commit/0b75524e6580eafb3612c0686e8af9d8d4015f9a#commitcomment-10834596
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/liggitt/kubernetes/commit/0b75524e6580eafb3612c0686e8af9d8d4015f9a#commitcomment-10835380


Ditto.


Além de faltar fechar a expressão, no final de todas é preciso do simbolo $, Deverá ficar (~'^ TEXTO $') Corrigir o mesmo para todas as expressões abaixo.


Det är för att få scrollbaren längst upp istället för längst ner. Diven
utanför tabellen  flippas också 180 grader vilket gör att totalt blir bara
scrollbaren flippad.
Den 21 apr 2015 23:18 skrev "Daniel Rapp" <notifications@github.com>:

> Jag hittade varför det buggar på mobile-safari. Nu försöker jag lista ut
> varför vi flippar hela tabellen 180 grader :S Har du lust att kommentera
> Pål?
> [image: mind-blown-2]
> <https://cloud.githubusercontent.com/assets/442870/7262789/a15228b6-e87c-11e4-9af6-a2de1b89455c.gif>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jcmandersson/Operation/commit/faedda6a710412c4d71fa9ee02109411f0b4a613#commitcomment-10834578>
> .
>


This isn't going to work on a non-unix environment (Windows, which we are supporting, although it's not great), nor will the test pass if Python is actually `python`, right?


Killdash, I am new to developing and programming but I wanted to know if there is a chance to working on this addon with you.  There is an issue with helix and this addon now that new version is out.  I've seen alot of people wanting a fix for this since it worked so well in the past.  Any chance of working on this with you?


Need "new" security mode flags:
http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan
$fake_register_globals=false;
$sanitize_all_escapes=true;


added view distance of a piece


Needs to be html escaped, so use xlt() instead of xl here.
http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan


"If path path" - word repetition of `path` here.


if piece target piece is next to you. returns true, otherwise piece would never be visible.
not sure if correct.


I only have the refresh setting right now and am not sure if there will be any others I tried to use what the calandar used


use binding (? stuff)
http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan


Yup !


use binding


not sure if correct use of game.terrain.square


Some preliminary front page. To it we will attach separate albums.


binding


xlt()


Terminology: Is _parent instance_ as it relates to broker services well defined? I'm actually not sure, but since _instance_ might be used in rfc8 to indicate a job running Flux services, the _parent instance_ in this case might be mistaken for _parent job_. (Or am I being way too nitpicky?)


\W seems to work ok with my limited testing. I'm willing to go with it.


Corrigir I'm para I am, por que está dando problema no teste, o " ' " do I'm está fechando a expressão.


use addslashes() function around the $info_msg to html escape it:
http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan


"New Research Group" deveria ser trocado por "([^"]*)"


This is a great start, but given that the audience of the paper will be computational biologists it would be good to give a bit more definition.

Also, I don't think that it's the d-fold wreath product, but rather the wreath product of the automorphism group with the symmetric group on d letters. Take a look at https://en.wikipedia.org/wiki/Wreath_product, especially

![image](https://cloud.githubusercontent.com/assets/112708/7263728/14e25dda-e839-11e4-97ea-1f7c639462dd.png)

![image](https://cloud.githubusercontent.com/assets/112708/7263721/102f5d56-e839-11e4-890e-5de780c4f8b1.png)

We wouldn't have to introduce the wreath product in general, but (I think) just the wreath product with a symmetric group.


this is the html branch of the project


attr() around the $record_id:
http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan


I see this was added a whole 24 hours ago!  I better get on it ;-)


Eric, was Reserves removed as well? We just want to Jepson Regions removed;
right now we have no other way to toggle on or off the Reserve footprints
except with the checkboxes.

I am confused where this screenshot came from as Kai's github changes do
not include removal of two checkboxes....
thx

On Tue, Apr 21, 2015 at 2:48 PM, Eric Gelinas <notifications@github.com>
wrote:

> [image: screenshot 2015-04-21 14 48 05]
> <https://cloud.githubusercontent.com/assets/15744/7263324/753a66ae-e835-11e4-9843-a5af4f98ab78.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stamen/ecoengine/commit/c7bb5821362d65896dace9d454925dd5e701e642#commitcomment-10835110>
> .
>


It'll work, but there's always a chance of those images disappearing.


html escape.
Can either place text() around the whole thing or use xlt() on translation and text() on the name


Most likely forgot to launch a build


For constants, never have lead or trailing spaces. should be:
xlt('Change Status for'). ' '


`Iterables.addAll(this.partitions, partitions)`


This comment is lost.


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


This message is somehow wrong since static cos are not deleted


A frase está estranha, pelo o que eu entendi está fazendo uma ação When depedendo da condição de ter dois autores com os nomes citados. Não conseguir entender bem o que o cenário está fazendo. Seria bom especificar melhor o que está ocorrendo nesse passo.


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


Ha, I have learned the hard way multiple times that adding tracked files to .gitignore looks like 100% of the solution but is only 1% ;)


surround $trow with attr() function. Also place ' characters around it.
<input type='text' name='form_note' size='5' maxlength='15' value='<?php echo attr($trow['roomnumber']) ?>' >


Glad you like it. Its a drupal pattern really.

There is a test for them being visible or not, bundled with the other cms
link in the menu:

https://github.com/rafaqz/3000acres/blob/updates/spec/features/cms_spec.rb#L15

https://github.com/rafaqz/3000acres/blob/updates/spec/features/cms_spec.rb#L28
(I
forgot to rename that one and it could be edited a bit...)

But there should be one for the edit/save part. Its not there as it was
pretty messy until this week, with save loading the full cms page with
comfy styling and broken back button etc. I still haven't figured how to
return to the original page after editing, its just being redirected to
root currently, to avoid the cms.

On Wed, Apr 22, 2015 at 3:58 AM, Skud <notifications@github.com> wrote:

> This stuff is awesome! I really love it.
>
> But could we please get a test for it in spec/features? Eg. login as an
> admin, go to one of the relevant pages, and check that there is a cog icon
> there that, when clicked, takes you to edit the relevant snippet.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rafaqz/3000acres/commit/6bd3ebd0a0c99ed8115beaa5eb5f332d97821926#commitcomment-10830782>
> .
>


Firma digital para sistema de formulario de becas colegial


use xla() instead of xlt() here:
http://www.open-emr.org/wiki/index.php/Codebase_Security#Plan


Haha à la base oui, mais de toute manière pour l'instant on n'entre plus dans cette fonction...  en fait ça me faisait mal au coeur d'enlever cette partie de code, pcq elle pourrait toujours servir plus tard :p


The indentation/spacing is a little off.


This would definitely create a NullPointerException at `String[] parts = hostAndPort.split(":");` at Line 153 of HkpKeyserver.java, due to the `keyserver` being passed as null inside the `onCreate()` of `CreateKeyYubiImportFragment`.
I'm afraid my lack of an NFC card prevented me from testing it. It didn't cause errors in other places because `query` was null whenever `keyserver` was null. Sorry.


O cenário estava menor, porém a pedido do professor ele foi expandido.


Agreed, should we change the checkstyle such that it's ok to have 'magic'
numbers for column sizes?

On Tue, Apr 21, 2015 at 3:07 PM, Rob LaRubbio <notifications@github.com>
wrote:

> This is a case where I think the checkstyle rule causes us to write code
> that is less clean.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/koshalt/mim/commit/0d798bdd55850b34dccb42c1fc49cdd3ce7ef24f#commitcomment-10835470>
> .
>

--
Frank Huster
Mobile Health Innovations
Grameen Foundation
+1.206.325.6690 x201


Add `@Override`?


Can't use the attackRange as viewDistance?


I think this is a worthwhile change to PR. I hope you have time to remove `powershell` dependency and PR it.


This diff breaks `:NeomakeSh` since `a:name` is a dictionary.

Logged in #109


Also need to ensure use binding on the queries below.


I'm not ok on this one, the requirements is containing all my packages. So it is too big.


@Tux64 Excellent! Just out of curiosity, where did you hear about us?


"create" não deveria ser colocado entre aspas.


You can also tell Git to have @Joris-van-der-Wel be the author (even if you're the one actually comitting) with `--author="Author Name <email@address.com>"` Based on https://github.com/whatwg/dom/pull/24.patch that would be `Joris van der Wel <joris@jorisvanderwel.com>`.

This can be nice because then you can find author history with `git log` and the [contributors tab](https://github.com/whatwg/dom/graphs/contributors).


One thing I like to do here is `var $this = $(this)`. It started as a micro-optimization to avoid continually instantiating jQuery objects, but now it's just kind of become a convention I like.


WTF?


I believe that parameter is only used to assist with error reporting and isn't relevant to the test, I can modify it to `Paths.get("python")` if you prefer?


Are you familiar with rems, too? I find them a lot easier to work with than ems. [More info](https://css-tricks.com/font-sizing-with-rem/).


OMG


Actually, looking at our code, this is probably be a bit difficult. These data sources are lazily created, so they may not be registered for events coming through the bus. I'm a little hesitant to break that if we don't need to. I could make profile listen, but then profile ends up doing all the work and we basically lose all encapsulation. I'm going to just finish up this way for now...


Do we want check if we have gone over the limit?
```java
        if (outstandingTasks.incrementAndGet() >= maxPartitionBatchSize) {
            outstandingTasks.decrementAndGet();
            return;
        }
```


It's actually not obvious that clear leaves the reserved space there. Did you read somewhere that it would?

But it might and there is no reason not to give it the opportunity. Although I wonder if calling reserve after the clear wouldn't be more guaranteed to accomplish this. Maybe worth measuring? Can do it as an optional followup.


hasil browsing itu kak, begituji caranya klo mau input kah? deh panjangnya -_- ....

acc 2 lagi kak?? di kak gun?
klik mi url ku, langsung ke gdrive laporan pbo-ku. sy belm tulis tujuan praktikum dan dasar teori soalnya blm ada modul


:+1:, was about to submit a PR for that too.


maybe wrap this 310 character line :)


Did you mean to check this in? I'm getting lots of debug output now :-)


You could also just use `"$*"`.


Checked the referenced sources, looks good. Thanks


if options does not have mode, it does not have a default anymore, is that intended ?


If what you have is working, then would keep it. But if you think there will be lots of customization of categories and appt categegories, then may make sense to consider vertical table.


You can achieve the same result via
````cpp
#define RAPIDJSON_VERSION_STRING \
  RAPIDJSON_STRINGIFY(RAPIDJSON_MAJOR_VERSION.RAPIDJSON_MINOR_VERSION.RAPIDJSON_PATCH_VERSION)
````
And for consistency, I would prefer to have a common `RAPIDJSON_VERSION_*` pattern for the macro names. But both of this is just cosmetical.


I heard about you from the DuckDuckGo mailing list.


Nice job getting these shadows.


Testing confirms it auto-disarms in 5 seconds when Throttle Interlock is disabled, even if throttle stick is up.


unreported bug but great you've fixed it :dancers:


For example, if you feel the need to have the userdef1 etc columns below, then a vertical table is likely best.


@ebuchman commenting this line is what's causing failures like: https://circleci.com/gh/eris-ltd/epm-go/279 ... these tests pass on my machine (i.e., not in containers). i have not had time to figure out what this change is getting tripped, but in the meantime perhaps we just want to revert this change and go with the other option presented in #93 namely just to update gendoug to v3.


I didn't :( but it should be resolved on the tip of master now.


Hmmm... I definitely didn't intentionally add that.  I wonder how that got there.


Cool, I'll check into that.

> On 21 Apr 2015, at 17:46, ptclarke <notifications@github.com> wrote:
>
> For serialization see RKSerialization, RKMIMETypeSerialization.
> Releases prior to iOS7 now reportedly <3% of all devices so probably not worth worrying too much.
> Will try and get notes together in next couple of days and maybe we can meet up online to discuss further.
>
>
>
> > On 22 Apr 2015, at 4:27 am, Oli Griffiths <notifications@github.com> wrote:
> >
> > Awesome. With a protocol, if that's required then users can add their own iOS 6 support if needed.
> >
> >
> >
> >
> >
> > > On 21 Apr 2015, at 14:56, Samuel E. Giddins <notifications@github.com> wrote:
> > >
> > > No, we'll either go iOS 7+ or iOS 8+.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> > —
> > Reply to this email directly or view it on GitHub <https://github.com/RestKit/RestKit/commit/5225cb76a788730918e499f227f7079f2c92a66a#commitcomment-10831894>.
> >
>
> —
> Reply to this email directly or view it on GitHub.
>


"recommand" type


- *referred* to as *a* CLI, or *commonly known as a CLI*
- swap: `a node` for `kong`
- change `as no cluster` to `has no cluster`


The only place `file` is used in the method is here. Just replace the parameter with the path string.


- change `needs access` to `requires access`
- change `what is your node's NGiNX working directory` to `where the NGiNX working directory is located`
- choose commas or parenthesis, not both. with parens the sentence should read as though they are not there.
- swap `your Kong` for `the`


Kong comes with a ***Command Line Interface*** ("CLI")


This has no relevance to efficiency. Tagging this as #plebmistake


Note this commit is in a strange developer branch:
https://github.com/teryhill/openemr/commits/patient-flow-board


Curious to hear how you got the colors here.


> I thought this was already clear. In my last comment in main conversation, I said I was going to change to int. Haven't you received email notification?

I read that notification as "`newlen` is changed to `int`", but was not sure all `ssize_t` would also be changed.


maybe name this `files`


@rtaustin Sorry, I don't have it in a seperate css file, because the xs breakpoint must be changed to add this, and as such editing the .less files and rebuilding is the simplest and more maintainable way.

If you want to port this to a newer version, you can ignore the dist folder (css files) as those are built and not edited by hand and if you're using a customized bootstrap, you can do the same thing by editing less/bootstrap.less and re-building bootstrap.css.


and this `file`


Lines are a bit long here.


removed


This info might be still useful.


Yes, it was very confusing.  I don't know if it's explaining how this code might get called, or why something might get deleted, or why it might crash due to double-deletion.


Oh, good point. I should add a default.


> ("CLI")

Doesn't require quotes, but :+1:


> Almost every CLI command needs access to your configuration file in order to be aware of what is your node's NGiNX working directory

Most CLI commands require access to the Kong configuration file in order to know where the nginx working directory is located.


I think you want to use `ConcurrentLinkedDeque` and `addFirst` here so the next worker will start with this. Otherwise you will round-robbin the files.


Nice job adding a hover state here.


#plebmistake


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


Discussed via chat — I will clean this back up.


Vinicius, você usou o construtor de Rede para treinamento. Use o construtor para execução.
Leia os comentários dos construtores para esclarecimentos.


Why apply this at the `p` level instead of the `body` level?


I've only seen evidence of it in v8, but I wouldn't be surprised if it had a similar effect in others too. Since v8 backs 2 browsers and node/iojs, it's enough evidence for me :-)


good call


Sorry it the git stuff we still don't git along. Can you move it to a better place?


libcxx_ubuntu_14_04 (lefticus) - x86_64-MacOS-10.9-clang: OK (337 of 337 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)


No, I think you're right.  The only difference I can see is that output_min does this:

    limit.throttle_upper = false;

Where I set that to true.  I'm not sure if that is strictly necessary, because we also "relax" all the controllers.  But it does seem like a good idea, just to be sure.  I wonder if you use output_min, but change that to true?


How does your local master branch look?


This line may not work as expected in all Rails versions. I believe @shabadahmed pushed a fix to the test helper that can be used to solve that issue, if it surfaces in Travis CI.


This should be placed within the condition above


wfm.

On Tue, Apr 21, 2015 at 3:25 PM, wes johnston <notifications@github.com>
wrote:

> Actually, looking at our code, this is probably be a bit difficult. These
> data sources are lazily created, so they may not be registered for events
> coming through the bus. I'm a little hesitant to break that if we don't
> need to. I could make profile listen, but then profile ends up doing all
> the work and we basically lose all encapsulation. I'm going to just finish
> up this way for now...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wesj/firefox-ios-1/commit/0330e4392e627459a26eaea669ce635412d12afd#commitcomment-10835747>
> .
>


What an ugly line of code.

:+1:


I think here it's better to describe --enable-libless at the end of the list for having a flow how to reduce features more and more.


I misread this.  Since it doesn't matter, what about `Paths.get("non", "important", "path")`?


why do you think that the reserve property is maintained through the assign?

I don't see a reason not to do this, the cost is low, and some implementations might do what you want. but i don't know if they would.

Might be worth measuring it specifically, but it could change by implementation for all we know ;)


Thanks for the input Chris.
I think your idea of moving these validators in to a concern is good for drying up the the shared code, but in my opinion its more readable keeping each on the top of each model.


Ooops, I thought I updated all of those on Bill's advice.  I can update this one and tuning func.  Ah, you know, I bet the update I had done was stuck in with the dropped RC_Input work.


hi this is neha


WOW thanks for catching that. It's back now


I haven't done anything with them as of yet. I do feel it would be beneficial to have the ability to easily increase the number of fields.


This is beautiful! Thank you :+1:


Just wondering here, but wouldn't it make more sense for the Lambda role ( lambda/lambda-roll.template
 ) to be implemented in nubis-meta ?


Couldn't test it since no link to get to the page. My review is complete. Looking forward to next revision.


I actually did look into this,  but went with `if: Proc.new { |a| a.secure_domain == false }` because it thought it was intuitive having both cases use a `if` statement and explicitly show the secure domain value.


Hooray!
thanks

On Tue, Apr 21, 2015 at 3:39 PM, Eric Gelinas <notifications@github.com>
wrote:

> WOW thanks for catching that. It's back now
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stamen/ecoengine/commit/c7bb5821362d65896dace9d454925dd5e701e642#commitcomment-10835976>
> .
>


You're right, it's not necessary. Not sure what you mean about line 44 however


At least log the error?


libcxx_ubuntu_14_04 (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (336 of 337 tests passed)

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


I think in this case, the schema is not well formatted, so the error message should indicate that.


The error message should include the status code.


I suggest the following style of code:

case+ (b, old_b) of
| (HIGH, LOW) => ...
| (HIGH, HIGH) => ...

The reason for the bug you reported not showing up earlier is that it is really unnecessary
to form the tuple @(b, old_b) here.

What is following 'case+' does not have to be a single expression; it can be a list of expressions.


This can be defined in the try clause since it seems not used outside that.


"Nome" deveria ser trocado por "([^"]*)".


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


Taking this line as an example: we used to map names to JuncVars. Now we map names to nums, then look up the nums in a vector. So we still have a map lookup, but also added a vector lookup.

Is this beneficial because we have a single name to num map? or is it a downside that is offset by other upsides?


"Twitter" deveria ser trocado por "([^"]*)".


Looks like this layout change affected the visibility of the menu, try hitting the green hamburger icon on the NTA screen.


Moving this to after line 164?


hmm, it's not a map - why call it one?


May want to discuss this on the forums. One possibility is the main table have all the elements up to `facility_id` since these don't change over the appointment. Then you could have another table patient_tracker_element which simple has 4 columns (appt_id,status,value,timestamp). If you'd like to track room numbers could even migrate the room number to the patient_tracker_elements. Then you place an entry whenever change the status.


Ditto. Error message should include the status code.


oh, I see the next commit addresses that


Ditto. Error message should indicate the schema is not well formatted.


Reviewed.


I think @(b, old_b) is unboxed tuple that doesn't use allocation, and (b, old_b) is boxed tuple that use allocation.

Isn't it?


You should just change the source thread pool to name the threads.


This looks great. Want me to close my pull?


Ah, we need to do this for all source classes in another pull request.


Really cool stuff. It is quite exciting :). Perhaps a pixie website can be hosted in pixie soon.


> Passing doctest

It does **not** pass the DocTest reference examples. Read output of the `rake doctest:latex` carefully: 240 runs, 5 assertions, 1 failures, 0 errors, **235 skips**. It passes only 5 examples, 1 fails and 235 are skipped (i.e. no output example is defined).

> - @jirutka and @mojavelinux, I plan to release in two days if
>  I encounter no problems in testing over that period, or unless
>  you believe I should hold back.

IMO it shouldn’t be released until it passes all the [reference examples](https://github.com/asciidoctor/asciidoctor-doctest/tree/master/data/examples/asciidoc). Then you can be quite sure that it handle all the AsciiDoc elements, i.e. any valid AsciiDoc document (well, almost, I’m not sure if it really cover everything). This is the main purpose of DocTest.

This means:

1. Generate output for the reference examples.
2. Check if the generated output is correct!
3. a. If not, edit it manually to be correct, then fix the converter until the test passes.
    b. Fix the converter, then regenerate output examples and check if it’s okay now.

Look at [HTML5 backend](https://github.com/asciidoctor/asciidoctor-backends/tree/master/test/examples) for example.


'(b, old_b) is a boxed tuple. Sometimes, (b, old_b) is the same as @(b, old_b). In this case, (b, old_b) stands for a list of expressions; it is not considered a tuple.


There are a few approaches to this API. My first instinct when seeing this code is to use pattern matching functions rather than creating a new data constructor to mirror the existing one.

``` haskell
inHandle InHandle{} = True
inHandle _ = False
```

However, I think best is to provide a record.

``` haskell
data HandleInitializer
data StdInit = StdInit { inInit :: HandleInitializer, outInit :: HandleInitializer, errInit :: HandleInitializer }

defInit = StdInit (const $ return ()) (const $ return ()) (const $ return ())
outErrInit f = StdInit (const $ return ()) f f
initStdHandles f = StdInit f f f
```

The advantage being that if desired there can be different behavior for different handles (I could see a potential case for stdin being different from stdout, I think the current API is designed for this but it just restricts one of the 2 to be `return ()`). And we can define shortcuts such as `defInit`.


Why remove the play and pause functions? They might be useful too.


These threads must have the connector class loader set in the thread to because of the call to `getInputFormat` below which loads the input format class.  This can be done automatically in the thread pool.  The `io.airlift.concurrent.Threads` pools do this.


... which I can't reproduce anymore. Changed.


> I always need to use the commit hash for links to code in master branch because the master branch is not static; it changes with each commit.

Ah yes yes. So maybe the option should be to use the commit on master only? Or, whatever the user designates as *"master"*.

Something like:
```elisp
(setq git-link-use-commit-on '("master" "some-thang" "some-other-thang"))
```

But that may get cumbersome...


INSERT INTO warden_checks VALUES ('782', '243', '', '', '8146241', '6', 'D9818C', 'Movement Related - Run Speed?');


Make sense. Thank's for your advice.


Why that?


erwer


Please extract csv file before running the program


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2104 of 2196 tests passed)

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


You're right, this is non-obvious. It's definitely a case of "I read it somewhere", but I can't remember where and I doubt it was authoritative.

However, the implementation for _Hashtable on my linux PC (which, if I'm able to grep /usr/include correctly, is what unordered_map uses behind the scenes) appears to have the following implementation:
```
  template<typename _Key, typename _Value,
       typename _Alloc, typename _ExtractKey, typename _Equal,
       typename _H1, typename _H2, typename _Hash, typename _RehashPolicy,
       typename _Traits>
    void
    _Hashtable<_Key, _Value, _Alloc, _ExtractKey, _Equal,
           _H1, _H2, _Hash, _RehashPolicy, _Traits>::
    clear() noexcept
    {
      _M_deallocate_nodes(_M_begin());
      __builtin_memset(_M_buckets, 0, _M_bucket_count * sizeof(__bucket_type));
      _M_element_count = 0;
      _M_before_begin()._M_nxt = nullptr;
    }
```
Note no call to `_M_deallocate_buckets`.

So a measurement by me won't show up anything interesting. If you think it's worth doing I'll add a 'code review comments' commit containing this (and any other changes from your other comments).


I'd make this explicitly `addLast` since we would want to process open partitions first and new partitions last


Why it catch error? The case covers all of cases...

```
$ git diff | cat
diff --git a/demo/pwm_button_ats/DATS/main.dats b/demo/pwm_button_ats/DATS/main.dats
index b9d40a8..2602fb6 100644
--- a/demo/pwm_button_ats/DATS/main.dats
+++ b/demo/pwm_button_ats/DATS/main.dats
@@ -30,9 +30,10 @@ fun state_helddown (sd:state_dat):state_dat = @{ state= state, brightness= br, s
 }

 fun do_state (b: natLt(2), old_b:natLt(2), sd:state_dat): state_dat =
-  if b = HIGH && old_b = LOW then state_transition sd
-  else if b = HIGH && old_b = HIGH then state_helddown sd
-       else sd
+  case+ (b, old_b) of
+  | (HIGH, LOW)  => state_transition sd
+  | (HIGH, HIGH) => state_helddown sd
+  | _ => sd

 fun do_action (sd:state_dat): void =
   case+ sd.state of
$ make
patsopt -o DATS/main_dats.c.tmp -d DATS/main.dats
/home/kiwamu/src/arduino-ats/demo/pwm_button_ats/DATS/main.dats: 1159(line=36, offs=5) -- 1166(line=36, offs=12): error(2): this clause should contain more patterns.
patsopt(TRANS2): there are [1] errors in total.
exit(ATS): uncaught exception: _2home_2kiwamu_2src_2ATS_2dPostiats_2src_2pats_error_2esats__FatalErrorExn(1025)
../../Makefile.common:61: recipe for target 'DATS/main_dats.c' failed
make: *** [DATS/main_dats.c] Error 1
```


The following patch is compilable.

```
$ git diff | cat
diff --git a/demo/pwm_button_ats/DATS/main.dats b/demo/pwm_button_ats/DATS/main.dats
index b9d40a8..dbdaf72 100644
--- a/demo/pwm_button_ats/DATS/main.dats
+++ b/demo/pwm_button_ats/DATS/main.dats
@@ -30,9 +30,10 @@ fun state_helddown (sd:state_dat):state_dat = @{ state= state, brightness= br, s
 }

 fun do_state (b: natLt(2), old_b:natLt(2), sd:state_dat): state_dat =
-  if b = HIGH && old_b = LOW then state_transition sd
-  else if b = HIGH && old_b = HIGH then state_helddown sd
-       else sd
+  case+ @(b, old_b) of
+  | @(HIGH, LOW)  => state_transition sd
+  | @(HIGH, HIGH) => state_helddown sd
+  | _ => sd

 fun do_action (sd:state_dat): void =
   case+ sd.state of
```


You need (_, _) for (_)


(on the subject of vector reservation - http://stackoverflow.com/a/6882816/2352259)


A different option is to create a new type of `RuntimeException`, say, `KafkaExtractorException`, which is aware of the Kafka topic on which the exception is thrown and automatically includes the topic name in the exception message.


Thank's. Following has no problem for compiling.

```
$ git diff | cat
diff --git a/demo/pwm_button_ats/DATS/main.dats b/demo/pwm_button_ats/DATS/main.dats
index b9d40a8..41584ea 100644
--- a/demo/pwm_button_ats/DATS/main.dats
+++ b/demo/pwm_button_ats/DATS/main.dats
@@ -30,9 +30,10 @@ fun state_helddown (sd:state_dat):state_dat = @{ state= state, brightness= br, s
 }

 fun do_state (b: natLt(2), old_b:natLt(2), sd:state_dat): state_dat =
-  if b = HIGH && old_b = LOW then state_transition sd
-  else if b = HIGH && old_b = HIGH then state_helddown sd
-       else sd
+  case+ (b, old_b) of
+  | (HIGH, LOW)  => state_transition sd
+  | (HIGH, HIGH) => state_helddown sd
+  | (_, _) => sd

 fun do_action (sd:state_dat): void =
   case+ sd.state of
```


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (340 of 340 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)


"Descrição" deveria ser trocado por "([^"]*)";


"Sigla" deveria ser trocado por "([^"]*)".


Module version numbers appear in double quotes. The single quotes are stripped out by the error message parser.

Sent from my iPhone

> On 21 Apr 2015, at 11:13 pm, Stéphane Épardaud <notifications@github.com> wrote:
>
> Because we don't want single+double quote on some error messages. Most have single quotes and that's enough. version '"1"' is crazy. We should not pollute error messages for the IDE and I can't understand why only this one then? The typechecker is full of error messages using just single quotes.
>
> —
> Reply to this email directly or view it on GitHub.
>


Ill start work on chopping models into many objs. That and make the skeleton look more like a skeleton


Don't really need extra line break.


I'm interested in difference between (A, B) and @(A, B) on pattern matching.


Don't need extra line break.


i dont know what that does but it looks cool :)


Even though this code was not working as intended (new_branch was the actual branch created). It allowed to create new branch based on other branch. I guess this could be added.


Hi @maupanz,

This would have undesirable consequences for non-grid forms. We could do something like this instead:
```
return /theme-.*grid.*/.test( $('form.or').attr('class').test());
```
This would then work for all themes that have the word "grid" in them.

Would that work for you?


replace these if statements with:
```
if (!hasNext()) {
    throw new NoSuchElementException();
}
```


(A, B) is a list of patterns; @(A, B) is a single pattern.

A list of patterns is used to match a list of expressions of the same length.


list' deveria ser substituído por: list$'


Are they same structure over the memory?


This seems weird, should this be referencing a string directly? Where is it being allocated?


Were any of the files updated to Pi V2?
Since we have an all new processor, does the code no longer work for pi V2?

Thanks.


`getDefaultWriterFilePath` is already including the extract table name. Why you need the new method? Also the new method does not consider the case of multiple forked branches?


ah, i see `$test_timestamp` now that i look a little closer. thx, will fix.


Note that this won't do anything, because it's the class that's observing the notifications (not the instances themselves). The class is only unloaded when the process terminates, so there's no need/reason to remove it as an observer.


where does this comment go


Thank you, Nicola! :heart:


You can simplify this by extending `com.google.common.collect.AbstractIterator`


Then change this to `@Override protected LocatedFileStatus computeNext()`


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

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


and `return status;`


and `return endOfData();`


and then drop these two methods


A list of patterns itself is not considered a pattern.

A list of expressions itself is not considered a valid expression; it has not representation in memory.
Say you write (e1, e2). The values of e1 and e2 can be stored into two (unrelated) variables.


Well, reserve requests the vector be able to hold at least N elements, and assign populates the vector with N elements. I can't imagine why the vector would need to change size for the second part!
The assign is necessary so I can use it as a contiguous array at will. The reserve probably isn't necessary - I'd expect it does a reserve+multiple memcpy (or similar) as part of an assign call.

Anyway, this bit of code is gone now :)


As far as I know not yet (see the [note in the index.html](https://github.com/Chadderz121/bakingpi-www/blob/master/index.html#L17-L19)):

> This course has not yet been updated to work with the Raspberry Pi models B+
> and A+. Some elements may not work, in particular the first few lessons about the
> LED. It has also not been updated for Raspberry Pi v2.

But I hope people will help adding support for the current version soon.


Umm... I should learn more about it. However thank's.


Clever.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (340 of 340 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)


Neo, you are good ppl.


you might want to rename this to `ignored` so IntelliJ (and David) doesn't complain


Well I thought that `'ws2_32'` is basically convered to 64-bit int. It's not interpreted as a string at all. Seems to work! It was taken from the original shellcode over in `external/source/shellcode`.


@karasu I think I remember reading that there were improvements made to Network Managers dbus interface. It was around the time GNOME 3.16 was released. I will try to remember to look into it :grimacing:


I also *added* Constants for the width and height and the table element and *removed* get the "i" parameter from JavaScript


FWIW, it was being used as a map (lookup table) rather than a vector (access via iteration).
I just needed a different name to convert this lookup table into the thing I'd access via iteration.


It's my pleasure :-)


Corrected in a subsequent commit


Yeah, I fixed this locally and was about to commit this same thing, but I just saw that you took care of it.


/*Это надо переделать:
   * dropdown list
 		<select name="bday">
		<option selected="selected" value="0">-</option>
		<option value="1">1</option>
		<option value="2">2</option>
   *или добавить метод fillEmptyContact, где делать не selectByVisibleText("text"),
   *а selectByValue(Value) ещё может быть что-то есть, пока я не придумала, мало знаний :)
   */


Instead of manually closing, you can use a named return and then a deferred function that mutates the return value. sorta looks like this:

```go
func foo() (err error) {
    foo := open(foo)
    defer func() {
        newErr := foo.Close()
        // "err" refers to the named return value
        // if it already contains a meaningful error, avoid clobbering it
        if newErr != nil && err == nil {
            err = newErr
        }
    }
    return nil
```


it works now :), but please use the logging.debug('... %s', yourstring) format.


static import


and so on...


Hi Dominic,

Yes I had already found the issue of duplicate entries and fixed it my
development version, as I said my development version has not as yet
been copied back to master yet.

Rgds

tobiz

On 20/04/15 23:58, Dominic Spreitz wrote:
> Tobiz,
>
> I finished an initial version of the OGN_db.py code. It's in my github.
> Problem with the code is, that if you execute it several times, you get
> several identical entries in the db. Same is true for the flarm_db.py
> code. At the end of each script some smart sql statment should delete
> all duplicate entries and only keep the latest (max(id)) one.
>
> Cheers
>
> Dominic
>
> Am 20.04.15 um 09:20 schrieb tobiz:
> > Hi Dominic,
> >
> > Access to Flarm_db has been ok I think all through the recent flarm
> > update.
> >
> > I didn't know how to (or that you could) pull the OGN_db so worked on
> > flarm_db really as an exercise since code in perl already exists to do
> > this, I just don't like perl.
> >
> > What I suggest is a value in Settings to say which of OGN_db or
> > Flarm_db is to be used. Then add in code to pull OGN_db or Flarm_db as
> > required, modify the function that does the mapping to Registration
> > depend on the new setting and everything after that remains the same.
> >
> > Rgds
> >
> > tobiz
> >
> >
> > Sent from Samsung tablet
> >
> >
> > -------- Original message --------
> > From: Dominic Spreitz <notifications@github.com>
> > Date:20/04/2015 06:26 (GMT+00:00)
> > To: tobiz/OGN-Flight-Logger <OGN-Flight-Logger@noreply.github.com>
> > Cc: tobiz <pjrobinson@metronet.co.uk>
> > Subject: Re: [OGN-Flight-Logger] Created INSTALL.txt file (f8ab361)
> >
> > Hi Robiz,
> >
> > Thanks for checking my pull request. Unfortunately I mixed several
> > commits in one pull request. The work I did on flarm_db.py should be
> > disregared, since I created a new file OGN_db.py which contains all the
> > mods for the OGN DDB. To my big suprise, flarm_db.py was still able to
> > pull data from Flarmnet. They re-enabled the download of Flarmnet-data.
> >
> > Rgds,
> >
> > Dominic
> >
> >
> >
> > Am 20.04.15 um 00:28 schrieb tobiz:
> > > Hi Dominic,
> > >
> > > Thanks for your useful modifications, I'll look into including them,
> > > they seem useful additions.
> > >
> > > I had noted some one running the code from the Glidern2 server log.
> > > I've made a number of enhancements in my development version that I've
> > > not uploaded to the master version yet. When I do next sync these back
> > > to the master I'll include your enhancements.
> > >
> > > Rgds
> > >
> > >
> > > -------- Original message --------
> > > From: Dominic Spreitz <notifications@github.com>
> > > Date:19/04/2015 20:28 (GMT+00:00)
> > > To: tobiz/OGN-Flight-Logger <OGN-Flight-Logger@noreply.github.com>
> > > Subject: Re: [OGN-Flight-Logger] Created INSTALL.txt file (f8ab361)
> > >
> > > Created INSTALL.txt file to make it easier for new users to install
> > > the software
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > >
> >
> <https://github.com/tobiz/OGN-Flight-Logger/commit/f8ab3614d3162a2212b7711223f4495732e6cd05#commitcomment-10794346>.
> > >
> >
> > --
> > Dominic Spreitz
> > Kothauer Str. 123a
> > 85053 Ingolstadt
> > Germany
> >
> > Mobil: +49.179.9016643
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/tobiz/OGN-Flight-Logger/commit/f8ab3614d3162a2212b7711223f4495732e6cd05#commitcomment-10797223>.
> >
>
> --
> Dominic Spreitz
> Kothauer Str. 123a
> 85053 Ingolstadt
> Germany
>
> Mobil: +49.179.9016643
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tobiz/OGN-Flight-Logger/commit/f8ab3614d3162a2212b7711223f4495732e6cd05#commitcomment-10813952>.
>


`getDefaultWriterFilePath` returns `this.getTable() + "/" + this.getExtractId() + "_" + (this.getIsFull() ? "full" : "append")`, while I only need the table name, since I need to write the data to e.g., `/data/tracking/PageViewEvent/2015/...`.

Table name is not fork dependent - it's defined in the workunit, so all forks of the same workunit should have the same table name.


I'm surprised there was no warning for the lack of space before the `{`. cc @JDGrimes


FacilityTabGridView (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2174 of 2196 tests passed)

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


91244204-HeatExchangerAirToAirSensibleAndLatent-Supply-Air-Outlet-Temperature-Control-problems-#4794 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1136 of 1136 tests passed)

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


Good timing – I was just pulling my hair out earlier today over this.

In a dbjoin element I tried moving a Where condition down from the ‘Joins where’  to the ‘Filters where’.

It is already working as expected if in the ‘Joins where’. It was only when I moved it to the ‘Filters where’ (an yes, without including the ‘WHERE’ part) when I had a problem. It seems like the ‘Filters where’ will not parse the || (OR condition) in a placeholder – and always returns blank if the || condition is included – so triggers an error in the query.

So the problem wasn’t because I was using the placeholder - it was because I was using the || condition in the placeholder.  But without the || condition, the result could very well be blank - so at times, would also give an error.

I gave up on trying to figure that placeholder code. But what was happening with the dbjoins, from what I could tell, was that initially the data being used to search for the placeholder value was not using the form model data – but rather the list model data (at least the first time the form was opened from the list). So it was never finding the element and returning an empty string.

From: Hugh Messenger [mailto:notifications@github.com]
Sent: Tuesday, April 21, 2015 1:53 PM
To: Fabrik/fabrik
Cc: bauer-git
Subject: Re: [fabrik] Added parseMessageForRepeats() to main helper, which handles replacing (b22878d)

Yup, that's exactly what the new function does. Of course, it only does it
for elements in the same repeat group. I'll try and find some time today
or tomorrow to apply it to the database WHERE clause.

-- hugh

On Sat, Apr 18, 2015 at 8:20 PM, bauer-git <notifications@github.com> wrote:

> What I need is a placeholder that can be used in the databasejoin element
> WHERE condition that will return the value from another element in the same
> current row of the repeat group - but NOT a comma delimited string of all
> the repeat groups values - just the value for that element in the same row
> the database join element is in.
>
> If there is a way you can figure to do that - it would be a godsend to me.
>
> For example I have a dbj element in a repeat group (e.g.
> fb_organizations___dbj_element )
> If I try to use this...
> WHERE {thistable}.id IN ( SELECT parent_id FROM
> membership_form_setup_repeat_include_with WHERE parent_id={thistable}.id
> AND include_with={fb_organizations___facility_type_id_raw} )
>
> There will be a 500 and 404 error generated because
> {fb_organizations___facility_type_id_raw} is being returned as a comma
> delimited string of values rather than just the single value of
> {fb_organizations___facility_type_id_raw} in that same row (repeat group)
> only.
>
> Why does a repeat group placeholder work correctly (as I expect it anyhow)
> when used in a calc element but not in a dbjoin?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Fabrik/fabrik/commit/b22878dbd04938433506d1dbda3aa12defcb8495#commitcomment-10790423>
> .
>

—
Reply to this email directly or view it on GitHub <https://github.com/Fabrik/fabrik/commit/b22878dbd04938433506d1dbda3aa12defcb8495#commitcomment-10830680> .  <https://github.com/notifications/beacon/AB7s2uJWGi9Pp5nqjP8npCbTbeAbw30yks5oBoYUgaJpZM4EC7rc.gif>


I like what you've wrote, but most of the time I prefer `where` clauses rather than `let` bindings:
```Haskell
separation self neighbors = sumV (map (^-^ p) (positions neighbors))
  where p = position self
```


Trying to get it to build properly in Carthage. It doesn't seem to want to build the sqlite3 framework though.


Also, you can avoid the unsightly parens by using the `$` operator:
``` Haskell
separation self neighbors = sumV $ map (^-^ p) $ positions neighbors
  where p = position self
```


there is still an exception being thrown. it happens al the time when you click on the first tab ...
System.ComponentModel.Win32Exception was unhandled
  IsTransient=false
  Message=Error creating window handle.
  Source=System.Windows.Forms
  ErrorCode=-2147467259
  NativeErrorCode=1400
  StackTrace:
       at System.Windows.Forms.NativeWindow.CreateHandle(CreateParams cp)
       at System.Windows.Forms.Control.CreateHandle()
       at System.Windows.Forms.TextBoxBase.CreateHandle()
       at System.Windows.Forms.Control.CreateControl(Boolean fIgnoreVisible)
       at System.Windows.Forms.Control.CreateControl(Boolean fIgnoreVisible)
       at System.Windows.Forms.Control.CreateControl()
       at System.Windows.Forms.Control.OnVisibleChanged(EventArgs e)
       at System.Windows.Forms.Label.OnVisibleChanged(EventArgs e)
       at System.Windows.Forms.Control.OnVisibleChanged(EventArgs e)
       at System.Windows.Forms.Control.SetVisibleCore(Boolean value)
       at System.Windows.Forms.TabControl.UpdateTabSelection(Boolean updateFocus)
       at System.Windows.Forms.TabControl.OnSelectedIndexChanged(EventArgs e)
       at MetroFramework.Controls.MetroTabControl.OnSelectedIndexChanged(EventArgs e) in d:\CodeLib\frameworks\winforms-modernui\MetroFramework\Controls\MetroTabControl.cs:line 571
       at System.Windows.Forms.TabControl.WmSelChange()
       at System.Windows.Forms.TabControl.WndProc(Message& m)
       at MetroFramework.Controls.MetroTabControl.WndProc(Message& m) in d:\CodeLib\frameworks\winforms-modernui\MetroFramework\Controls\MetroTabControl.cs:line 488
       at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
       at System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
       at System.Windows.Forms.UnsafeNativeMethods.SendMessage(HandleRef hWnd, Int32 msg, IntPtr wParam, IntPtr lParam)
       at System.Windows.Forms.Control.SendMessage(Int32 msg, IntPtr wparam, IntPtr lparam)
       at System.Windows.Forms.Control.ReflectMessageInternal(IntPtr hWnd, Message& m)
       at System.Windows.Forms.Control.WmNotify(Message& m)
       at System.Windows.Forms.Control.WndProc(Message& m)
       at MetroFramework.Forms.MetroForm.WndProc(Message& m) in d:\CodeLib\frameworks\winforms-modernui\MetroFramework\Forms\MetroForm.cs:line 581
       at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
       at System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
       at System.Windows.Forms.UnsafeNativeMethods.CallWindowProc(IntPtr wndProc, IntPtr hWnd, Int32 msg, IntPtr wParam, IntPtr lParam)
       at System.Windows.Forms.NativeWindow.DefWndProc(Message& m)
       at System.Windows.Forms.Control.WmMouseDown(Message& m, MouseButtons button, Int32 clicks)
       at System.Windows.Forms.Control.WndProc(Message& m)
       at System.Windows.Forms.TabControl.WndProc(Message& m)
       at MetroFramework.Controls.MetroTabControl.WndProc(Message& m) in d:\CodeLib\frameworks\winforms-modernui\MetroFramework\Controls\MetroTabControl.cs:line 488
       at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
       at System.Windows.Forms.NativeWindow.DebuggableCallback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)
       at System.Windows.Forms.UnsafeNativeMethods.DispatchMessageW(MSG& msg)
       at System.Windows.Forms.Application.ComponentManager.System.Windows.Forms.UnsafeNativeMethods.IMsoComponentManager.FPushMessageLoop(Int32 dwComponentID, Int32 reason, Int32 pvLoopData)
       at System.Windows.Forms.Application.ThreadContext.RunMessageLoopInner(Int32 reason, ApplicationContext context)
       at System.Windows.Forms.Application.ThreadContext.RunMessageLoop(Int32 reason, ApplicationContext context)
       at MetroFramework.Demo.Program.Main() in d:\CodeLib\frameworks\winforms-modernui\MetroFramework.Demo\Program.cs:line 16
       at System.AppDomain._nExecuteAssembly(Assembly assembly, String[] args)
       at Microsoft.VisualStudio.HostingProcess.HostProc.RunUsersAssembly()
       at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
       at System.Threading.ThreadHelper.ThreadStart()
  InnerException:


`$` is equivalent to function application, but "groups" the right side before applying the function. So it's equivalent to having parens that go from the point of the `$` to the end of the line:
``` Haskell
map (^-^ p) (positions neighbors)
==
map (^-^ p) $ positions neighbors
```


This `return` can go as well.


Yeah, I'm still figuring out the dot and dollar operators and wanted to write it with parens first and then rewrite it prettier once I'm sure it compiles (I'm basically testing using CI since I'm still struggling with making Cabal build the project).


Yeah, this won't catch ALL typos but it'll catch when you make a typo in the styler name.


I thought that running `rake generate:html` and `rake generate:latex` generated the reference examples.  What am I missing?

I put some of my own examples in to better understand what was happening and to make changes and additions.  Below is what I see when I run the tests.  I will try to understand what is happening re the reference examples.

In (1), `rake dockets:latex` I get `240 runs, 199 assertions, 0 failures, 0 errors, 41 skips`
In (2), `rake dockets:html`, I get `19 runs, 19 assertions, 0 failures, 0 errors, 0 skips`

'==================================`

The details of (1) and (2):

(1) run `rake doctest:latex`:
```
$ rake doctest:latex
Run options: --seed 5560

# Running:

......SSSSPlease implement me! (inline_callout_process)
Please implement me! (inline_callout_process)
***
Node to implement: colist, class = Asciidoctor::List
.......................................................................................................SPlease implement me! (toc_process)
.Please implement me! (toc_process)
***
.............SSSSSSSS...SSSSSSS...S....................SPlease implement me! (inline_callout_process)
....SSSSSSSSSSSSSSSSSSS........................................../Users/carlson/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/asciidoctor-doctest-1.5.2.dev/lib/asciidoctor/doctest/test.rb:75: warning: instance variable @test_methods not initialized

Finished in 0.280923s, 854.3266 runs/s, 708.3792 assertions/s.

240 runs, 199 assertions, 0 failures, 0 errors, 41 skips

You have skipped tests. Run with --verbose for details.
```

(2)

```
rake doctest:html
Run options: --seed 41384

# Running:

....node.attributes (EQUATION): {1=>"env.equation", "role"=>"equation", "original_title"=>nil, "options"=>"no_number", "title"=>"equation"}
.node.attributes (EQUATION): {1=>"env.equation%no_number", "role"=>"equation", "no_number-option"=>"", "options"=>"no_number", "original_title"=>nil, "title"=>"equation"}
.node.attributes (EQUATION ALIGN): {1=>"env.equationalign", "role"=>"equationalign", "original_title"=>nil, "options"=>"no_number", "title"=>"equationalign"}
............./Users/carlson/.rbenv/versions/2.1.2/lib/ruby/gems/2.1.0/gems/asciidoctor-doctest-1.5.2.dev/lib/asciidoctor/doctest/test.rb:75: warning: instance variable @test_methods not initialized

Finished in 0.092294s, 205.8639 runs/s, 205.8639 assertions/s.

19 runs, 19 assertions, 0 failures, 0 errors, 0 skips
```


If you're not using `event` you don't need to define it: `$('.parallax').scroll ->`


Yeah, don't worry about style too much yet. Also, we can get a little pointfree with `separation`:
``` Haskell
separation self = sumV . map (^-^ p) . positions
  where p = position self
```


This revision is clean.


Also, I did a bit of googling on when one should use `where` and when one should use `let`, and I wasn't really sure which was more appropriate here. I went with `let` since I figured it was more readable; I feel like the code makes more sense if you know what `p` is _before_ it's referenced, but if you think using `where` is better code style, I'm happy to rewrite it. I'm not really quite sure what the difference is 'under the hood' yet.


Extra white line here.


I like how I said "don't worry about style" and then proceeded to worry about style. Haha.


Extra line here.


This is a downside that's offset by other upsides.
This section of code is pretty quick. The main time sinks from my experiments were previously live variable analysis (addressed in pr 1 and 2) and conflict analysis (addressed in pr 2 and 3).
What I the 'block processing' and 'reconstruction' stages (i.e. the stages affected by the extra vector lookup you mention) are proportionally larger after my three PRs, but are still (even when summed) generally swamped by live variable analysis time.
My fourth and final PR in this set will probably be to clean up my high level `clock`-based profiling and submit that for potential future use.


I'd say zap the `console.log` at this point.


@smannlymann I like it. The best rendition yet. I think we can go with this until somebody complains about something. :+1:


I won't release.  It may take me some time then, unless I get help, since there is only partial coverage of Asciidoctor.  How do I unrelease what I released to ruby gems.org?


this test fails if the cwd includes a symlink:
```julia
exception on 1: ERROR: LoadError: test failed:
("/Volumes/Lion/Users/jameson/Documents/julia\n" == "/Users/jameson/Documents/no-backup/julia\n")
 in expression: readall(setenv(@cmd "sh -c \"pwd\""; dir="..")) == readall(setenv(@cmd "sh -c \"cd .. && pwd\""))
```
even though they are the same location:
```julia
julia> Base.samefile("/Volumes/Lion/Users/jameson/Documents/julia\n","/Users/jameson/Documents/no-backup/julia\n")
true
```
but in the test, the location of the left has been normalized to the actual location, whereas the "no-backup" folder on the right is a soft link

is the test wrong or is the code wrong?


I removed this.  How did this get back here?


@pwidry15 @t20rex

Here is a 1st draft of my key takeaway. Can you please provide feedback and suggest at least one way to improve my contribution.

Thanks!


I think this context should follow the pattern of the other contexts, where they hold a reference to the child objects (TaskContext in this case), and construction of child objects happen by calling a method on the parent.  This will allow us to add query level stats and such to this object.


I removed this earlier.  Why is it back?


Hey Mitch, use
~~~
git pull
~~~
to pull these changes, and if that doesn't work and it's asking you to "stash" changes, just do
~~~
git fetch origin
git reset --hard origin/master
~~~
to completely overwrite everything with the latest version of the project


this too


Is that exactly equivalent? If not, how? They mean slightly different things.


ping


Am adaugat new.md


You lost the require_in (which may not be necessary, I haven't tested).


-currently


@dboehm3
The process.waitFor() waits for the process to EXIT/terminate, which is not desired behaviour. We want to wait for it to be ready.


Change "I" to "you"? We're pretty consistently using the second person throughout the docs I think.


複数項目のテストを1つに結合するという改悪をしてまで変換をした理由が知りたいです。


@Dletz @Quexer69 Note: removed private package, code may needs refactoring


It might be interesting to look through the commit history of this, it was contributed to by students over some period of time.


what happens if you use more than one?


On my system python was in /usr/local/bin/python


OK, make sense.


we can use [GetTickCount()](https://msdn.microsoft.com/en-us/library/windows/desktop/ms724408(v=vs.85).aspx) here to seed srand().  It is defined when calling windows.h and spills CPU cycles for the last 49 days or something.


*temporary


Removing the social icons was requested by legal counsel for the material design icons. They were not intended to be public.


We don't want to use that version, it's built by Homebrew and Will Not Work


*creates, *checks


I want your name in here somewhere, or your getHub id.  It's your function.


For a string `+=` is not the same as `<<` in that new objects are created for the `+=` whereas the `<<` modifies the receiver instead of creating a new string.


む、`sum()`は分離した方がよかったかな?


The `modulo` method is defined on each of these classes:

* Bignum
* Fixnum
* Float
* Numeric


Cool way of implementing tags.


Não achei "def selectMember(String memberId){}" na página "ResearchGroupCreatePage".


can we blacklist regenerator on newer platforms? or at least include the runtime


I don't think that this resolves, the problem, but comparing what is on GitHub with what I have locally, I noticed that the directory `test/examples/html` was missing on GitHub -- my bad, my `.gitignore` file. It is there now.