I think you need some parenthesis around the multiline string here:

    SUBTITLE = ("Please choose an application, a type of bisection"
                "and the number of bits for the application.")


@dirvine, do you like the nested match statement?  Alternative is to return message_received() after handling pre-sentinel message.


"trazadora" no es un modelo ni una marca, y como en castellano no se capitaliza el texto, debe ir en minúsculas.


Nice one, was looking to implement it ! Thx


Delete it from Android.mk, probably it is already included somewhere else but you have to find where and overwrite the old file with the new one.


This was a typo for toxic.


when is this message triggered?

is there a *.log file or CMake*.log to analyze the issue?


Darn, that's one of the things I had to rebase against later on. Will fix.


This was already in last issue...


Can we do some commenting through these tests? Ideally we'll only need to write the tests once and they'll work for... well... forever. But, if we're only going to write it once we might as well do the due diligence and explain each of the tests.


Will we also try and change this on AWS?


just a test


Oh okay I see


+1 on OK/Cancel.

Yes this is why you can't ESC any more.


I thought we had decided to do that, but I'm happy to remove it. Not that `normalize` is only called on a few of the builtin filters that either expect a string or a number. If they want to actually accept one of these values that can just normalize their arguments manually. What do you think?


Is this for ruby, js, or coffeescript?


@phoet can we open a PR for this, so we can go back to the mainline in time?


The issue is that `PreviewKeyDown` is emitted in the opposite order to `KeyDown`. `PreviewKeyDown` emits on the parent widget then the child widget. Xwt assume the events will be emitted on the child widget first and then propagate up to the Parent if the event is unhandled. This is what the other backends do and is what the Wpf backend needs to do in order to be consistent.

As such we will have to revert the change to use `PreviewKeyDown` instead of `KeyDown`. What was the exact case which was breaking for you?


You know it!


True...


Fixed :+1:


Actually, that's not true, they were always in an ASG. Removed in 711da6e120076710a86a35ecbc8e5d1ec4f6e318


Shoot, no, it didn't :/


@liggitt do you know what the second parameter (name) should be here?


(Define guards don't work cross-compilation-unit, after all.)


1: M100 needs more reviewers
1: M101 signature is missing
3: M102 unrecognized bullet/signature
1: M110 missing component name


Describe the interconnect and how it was measured.  Need a bit more specificity on what we mean by the interconnect.  Everything that you need to operate the network that is not part of the compute subsystem.  This may include infrastructure that is shared, but excludes that part that is not servicing the current cluster.


#4833-Documentation-for-HX-controlling-to-SP (rraustad) - x86_64-MacOS-10.9-clang: OK (1641 of 1641 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)


Please work on a branch for new features.


Do we really care about all the error messages in the data? Or could we just assert the `status_code`?


we should catch only `ValueError` here (provided that's the exception you are looking for)


come -> some


Nothing against Ubuntu, but once we'll make the NAT solution Highly Available (multiple instances monitoring each other), we will need to divert the two scripts anyway because they will need to call different APIs. The semi-official HA NAT AWS solution involves using Amazon Linux: https://aws.amazon.com/articles/2781451301784570


AWS is using an AMI that's specifically for NAT instances:

https://github.com/alphagov/tsuru-terraform/blob/8be049ddd54607d12ae42938136375b2b68bd068/aws/variables.tf#L58-L64

I'm unsure whether we'd want to change this to Ubuntu.


Needs indentation here.


@ardasgup The styling has been updated, but the logic needs to disallow the click instead of the css. 'pointer-events' disables all element interaction meaning no hover events either, so javascript must control the interaction.


OK, cool. Thanks both.


좋네요!


can also skip if !hebrewTypeAtt.isHebrew()


:memo:


It's `{@inheritdoc}`, see https://github.com/phpDocumentor/fig-standards/blob/master/proposed/phpdoc.md#6-inheritance.


This one should fix everything.


How are you???


Have we raised/is there an existing issue on Terraform to address this missing resource type?


That's all for the moment. I don't have to change more the repository, I add methods to set the attributes of the recyclerview manager.


I can implement it, would it be fine if I added a new function to the
library though - say a custom add function. The PCB bits are pretty
extraneous for what I need.

On another note, are you in the GDC?

On Tue, Apr 28, 2015 at 10:03 AM, Lane Kolbly <notifications@github.com>
wrote:

> Yeah, you can't guarantee the order in which they will be added.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rellermeyer/course_os/commit/2816bed09fdbe6fe528ddab72550bad8745e461e#commitcomment-10943321>
> .
>


It does not help a much. I was able to build statically linked ncurses version (using 32-bit msys2 shell). But no luck with tile one.

Still working to find the solution :(


Yup: https://github.com/hashicorp/terraform/issues/1148


Was testing if someone was seeing this :o


deleted it from android.mk. but don't know what now
it's not needed to add files here: https://github.com/dadi11/android_device_lge_e975/blob/aicp/device.mk
?


Cool, thanks @mikkoc.

Also, :+1: for very clear shell script here.


Because I didn't copy the example correctly from the book?  :P  It's fixed now.


I was thinking this would also go on our tech debt list.

Unsure if it makes sense to add a story for implementing the provider, whether we'd ever get round to it..


just today google for work changed the url to https://docs.google.com/presentation/d ... at leas for me


I know, bad excuse.  (I was trying to make a joke!!)


Ok. In the meantime I managed to build your source code (with some minor fixes to the code ;-)). For now I changed the OGLVideo::start() code in order to draw a triangle and everything goes well during the GLES3 context creation + drawing. So you're good to go from here ;-)


IMHO a BinaryNotFoundException (not related to plugin) would be sufficient, as the CommandExecutor can be used out of plugin context.


I misunderstood. It is a bug. I'll correct.


Are we updating the website/docs every release as well?


ValueError?


Maybe here :

PRODUCT_PACKAGES += \
	hostapd_default.conf \
	wpa_supplicant_overlay.conf \
	p2p_supplicant_overlay.conf

We should edit it to :

PRODUCT_PACKAGES += \
	hostapd_default.conf \
	wpa_supplicant.conf \
	wpa_supplicant_overlay.conf \
	p2p_supplicant.conf \
	p2p_supplicant_overlay.conf \
        FTM_PowerTable.XML


spelling: `spurious`


I will remove from this pr but i did it because the spacing so darn annoying. I also figured it was fine at the time because I often see large clean ups in some prs that (which often times sends me to rebase hell) but they are indeed mostly separate prs.


`/sys/class/infiniband/usnic_X`, not `usnic_verbs`


#bug
it wasn't said in issue, but it good practice to use UPPER_CASE for constants, in this case them are good visible in code.


Should we allow a type here as well, so that you can map constructors efficiently?


i found another extension with a dynamic slider approach but it is not working for me and i am yet lagging debugging styles to check why, see here https://github.com/jpotterm/chrome-resizable-speaker-notes, he is making a name.match(/^punch_speaker_notes/) check to fire his functions only when the speaker notes are opened but somehow his extension is not working for me, i think it is a better universal approach then checking for the language attribute.


I am guilty of combining PRs sometimes, too, but I'll try to stop as well :)


I was thinking exactly about it. Done


Actually I used this for testing this fixer. Pretty sure it's gone from a more recent commit though.


Thank you. Will look little bit later.


Small typo: should be "we need to verify"


There's only one thing i don't understand, wpa_supplicant.conf is written down here : https://github.com/dadi11/android_device_lge_gproj-common/blob/cm-12.1/gproj.mk

But from where is it taking that file?


Would it make sense to recommend https here?


1つずつの商品の値段も表示したんですね:+1:


#2, #4, #5
Ahora se ve mucho mejor la web :)


Thanks!


Change specification to requirement.  Add requirement for 1.5% accuracy of meter.


Looks like this edit is just in the general Formatting section. What would you do different per language?


ez itt például egy pager service


\n


PR submitted on ESH


I wonder if including the day element might get in the way of the real example? We could illustrate the TIL with:

```ruby
month = Time.now.strftime('%B') # in both examples
puts 'The month does not start with M.' if !(month =~ /\AM/) # in first example
puts 'The month does not start with M.' if month !~ /\AM/ # in second example
```


db に関する情報が、アップロードされないようにする。


I don't see why create takes a name, actually... it's only used to pass to `urlfortype`, which should probably take an empty name since creates run against the collection url.


Is there a reason why line 2192 is removed, or user error ?


#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int main() {
	srand(time(0));
	int i,N,*v;
    N=10;
	v=(int *)malloc(N*sizeof(int));
	printf("慢速排序法...\n製造隨機的十個1-100的整數\n以下 : ");
	for(int i=0;i<N;i++){
		v[i]=rand()%99+1;
		printf("%d ",v[i]);
	}

	for(int j=0;j<N-1;j++){
		for(int i=0;i<(N-1-j);i++){
			if(v[i]>v[i+1]){
				int f=v[i+1];
				v[i+1]=v[i];
				v[i]=f;
			}
		}
	}
	printf("\n排列之後~\n");
	for(int i=0;i<N;i++){
		printf("%d ",v[i]);
	}
	if(N%2==1){
		printf("\n中位數為%d",v[(N+1)/2]);
	}else{
		float mid=(v[N/2]+v[(N-2)/2])/2;
		printf("\n中位數為%f",mid);
	}
	return 0;
}


Nevermind... it's used for some subresources. For regular resources, it should be empty


Can you please add this language  to example file also? Thanks


Will do that. The only reason why I had it, was so that it was a bit more clear that it wasn't just to avoid using unless... But I guess that's also defeated by the fact that I could change the regex [^M] instead, if we follow that... I will remove the day from the check.


I have no idea. It seems from nowhere :P maybe we should add these files or delete these lines


I think it would be a good idea to add a `defaults` section to the `nconf` config in `Server.js` so that `APP_HEAD_URL` uses the `http://localhost:3000` value if nothing else is supplied. See the [nconf documentation](https://github.com/indexzero/nconf#hierarchical-configuration) for more details. This makes it so that this code "just works" in the development environment.


Professor, fiz as correções, eu mesmo posso colocar a tarefa como DONE?

2015-04-28 12:06 GMT-03:00 Altieres de Matos <notifications@github.com>:

>
>    - Altera de "long" para "Long"
>    - O que seria esse iso?
>    - O merge pode deixar que eu faço, pode fazer até o pull request
>    apenas ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/altitdb/ondetemjogo/commit/fc9a0a68fa9396d91278f84bdd538ea4dd9a57c5#commitcomment-10943398>
> .
>


```
Priority of the config is (1) runserver command arguments, (2) SERVER_BIND_* configuration, (3) data from CFG_SITE_URL, (4) Flask-Script defaults.
```


`s/inveniomanaga/inveniomanage/`


I think this last point makes sense (`absEval` should return proper values only, no lvals). Looking at the code just now, I guess I'd expect we'd have an abstract domain for the program state; interpreting an lval, as in `go (EVar x) = lValM (varLVal x)` should look up the value of the freevar `x` in the current abstract state, and likewise for array lookups, etc., in the current abstract heap?


Any revenue grade meter, any meter accepted by spec power, any meter documented to have accuracy of x% or better.  Need to specify by level.  Need to determine L1 =5%, L3 = 1.0%, L2 = 2.0%
#8


I get a syntax error here (on the backtick)


Cool, thanks - will try it out asap!


we had to revert the change because this broke many working playbooks, when you have an 'implied' template there is no way to distinguish between a bare string (no quotes) and a undefined variable (no {{ }}).


Yeah, that's be splendiferous. I'm not in the GDC now but will be around 1ish.


難しかったですか？
でも花子さんが夕飯に作りたかったものは分かりましたよね？


benn van a session, mégis a request-ről veszed le


El menu si tiene material design (waves) pero los botones aun no.
El boton en el header no funciona.


delete branch


I'm not 100% sure this is the right approach; maybe objRelease() is not the right mechanism here, maybe the module was left loaded for a purpose. There is no comment, though.


Did it work?


Incredible commit description!


Delete wpa_supplicant.conf from here : https://github.com/dadi11/android_device_lge_gproj-common/blob/cm-12.1/gproj.mk

Then add it to Android.mk :

include $(CLEAR_VARS)
LOCAL_MODULE := wpa_supplicant.conf
LOCAL_MODULE_TAGS := optional
LOCAL_MODULE_CLASS := ETC
LOCAL_SRC_FILES := $(LOCAL_MODULE)
LOCAL_MODULE_PATH := $(TARGET_OUT_ETC)/wifi
include $(BUILD_PREBUILT)

And finally edit device.mk to look like this :

PRODUCT_PACKAGES += \
hostapd_default.conf \
wpa_supplicant.conf \
wpa_supplicant_overlay.conf \
p2p_supplicant.conf \
p2p_supplicant_overlay.conf \
FTM_PowerTable.XML


Impressive!  It wasn't intentional. Thanks!


sorry for delay
my net connection is very slow
cannot access


Yep, but not get to that part yet


Good catch.  I think rxn12184 should have name "(2E,6E)-farnesyl-diphosphate diphosphate-lyase (adding water;acetone-geosmin-forming)" and abbreviation "R08543" to match the KEGG reaction ID.  Although the KEGG reaction is defined as "trans,trans-Farnesyl diphosphate + 2 H2O <=> Geosmin + Diphosphate + Acetone" while rxn12184 is missing (2) H2O on the left hand side.


@brazabr se for sobre o plugin é bom usar as issues


Newlines are meaningless when _displaying_ source code in readable form (printing the hex value or operator character for each byte).

In actual source bytes, newlines are no different from other bytes; they just push the numbers `9` or `13` onto the stack.


sorry for delay
my net connection is very slow
yes


Can you elaborate a bit on the real world problem behind it? That would help me think about the approach.


This must be
if(bead1->getMolecule() != bead2->getMolecule()) return false;


from gproj.mk delete wpa_supplicant too? or only .conf
done.


No to jiste muzu, ale porad to nemuzu volat z twigu (to muzu jen gettery, ne?). Musel bych si tam uz z controlleru posilat nejakou mapu a podle ni volat makra.
(Z controlleru zase nemuzu volat twig makra asi, ne?)


Yes, I have added a message box to inform the user of this manual step.  Fixed.


Yup it is. :)


What is it that's not currently supported by Terraform? Health checks for load balancers? Is there an issue on the repo?


This is really helpful but I am having issues with it. First, on a windows platform I had to change the encoding to "WinAnsiEncoding" - how can this be made cross platform? Secondly, when the PDF opens in Acrobat Reader I get an error "The font 'ComicSansMS' contains a bad /BBox". This doesn't appear to affect the PDF - the new fonts work fine. However it is obviously not good for the end user. Interstingly the Chrome plugin to preview PDFs (via an iframe) doesn't have the error. Does anyone have any suggestions?


Don't delete wpa_supplicant, it is the binary file inside /system/bin


Yup


ilyen nincs


Link  build 8


The check is there to be able to handle pre-2.6 kernels, that is handled differently.

Feel free to create a patch for setting the kernel version through configure.  For now, you would gain much with it though.


\n


whoa, thanks for catching that! I'll fix it


muzes si napsat vlastni fci pro twig {% mojeFce(item) %} ktera zavola javu.
Komentoval jsem to nekde u jineho vlakna. To muze vracet string jmeno macra
a jine macro s ifem vlozi spravne macro.

2015-04-28 17:46 GMT+02:00 smolijar <notifications@github.com>:

> No to jiste muzu, ale porad to nemuzu volat z twigu (to muzu jen gettery,
> ne?). Musel bych si tam uz z controlleru posilat nejakou mapu a podle ni
> volat makra.
> (Z controlleru zase nemuzu volat twig makra asi, ne?)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fit-fms/fms/commit/83706e58a1bc75fa7b1df876ed55d5a6350939ad#commitcomment-10944482>
> .
>

--
Jindřich Pilař


this frequency does not exist on opo devices


I wrote a variation on this program, called "20150425-6 copy" which combines the meat of getHeight and getWidth.  However, part of me wonders where the balance is in terms of adding additional functions for simplification of each function vs. having too many layers of functions which makes the logic (overall) harder to trace?

(Another way to have done this would have been to get rid of getHeight and getWidth, and basically define width and height variables in main() and use the specific printf() statements combined with capturePositiveValue() there, but I thought one of the goals was in minimize what's in main()? ).

Is there a different thing you had in mind though?  :)


I don't see how. We're never leaking this object anywhere, so why would it be stringi- or valueified?


Done


so it's time to compile? :P


The problem with this test is that is non-deterministic. I.e. it could pass just out of pure luck, since the thread scheduling is a dice roll (unlikely I know).

I think (although I may be wrong) that this could be re-written to use two threads and something like a `ManualResetEventSlim` to get thread 1 into the state where it would interfere with thread 2, block thread 1 on the event until the thread 2 has asserted that its behaviour is correct, thread 2 releases the event and thread 1 unblocks. If so, that would be deterministic.


no..no


> One large comment explaining how the double map resolution works would be nice.

Done: https://github.com/tschneidereit/shumway/commit/5f0832f83797700acb01104490c01edd3d65c2e2#diff-16fdce0a9f66d244030e9b52d4696463R212


Yes, now everything should be fine (for real) :P


agree, will change to generateFactories in all appropriate places


and hardware cm12 trees? for u9200


Unhandled Fundamental Failure


@as-cii so this supposed to archive http://catchup.io/boards/3/cards/57?


I'm aware that this perhaps isn't the general solution (judging by the discussion on the 'make threadsafe' issue) but it is a step in the right direction at least.


ano, napriklad


Used factoryDestination since it seems more grammatically correct but have no issue with factoriesDestination and will change it.


Pay attention to special cases. The linkedList does not need to be rotated


Cool. Thanks for letting me know. I hope someone does very soon


we're -> we're using?


https://github.com/ShevT дерзай :)
--
Отправлено из Mail.Ru для Android
вторник, 28 апреля 2015г., 18:50 +03:00 от "Andrea Turchet (aka rap_turkez)" <notifications@github.com>:
>and hardware cm12 trees? for u9200
>—
>Reply to this email directly or  view it on GitHub .


How about ```generateCompiledFile```? I'll change it to that and see if it sticks..


Google tells me that the error is due to a recent change in binutils, it no longer attempts tto resolve all symbols when creating a shared lib. The work-around is to add this to the link line:  `-Wl,--copy-dt-needed-entries`  I added this to moses/CMakefile and it seemed to fix  the previous incarnation of this problem.  May need to add it to cogutil as well..


this is my mistake, the offending line was the `Object.prototype.hasOwnProperty`. sorry about the confusion.

in this case, the change is still incorrect though; this should be `obj != null` (that is, a loose comparsion which checks for both undefined and null)


做此题是发现你你改变了头文件里面的内容，结果你没有重新编译头文件，那么你编译.cpp源文件，这时他并没太有编译头文件，以后这个问题要注意。


my god.. same error again :s


```javascript
var bar = null; for (var foo in bar) {}
var bar = undefined; for (var foo in bar) {}
```

Seem to run fine for me.


TeamCity MonoGame :: Develop (Win) [Build ???](http://teamcity.monogame.net/viewLog.html?buildId=6446&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:00


Изменил Вы и Вам, маленькие изменения в некоторых словах.
А в целом все ОК!


Looks like the `id` field isn't used anymore with notes (https://github.com/gitlabhq/gitlabhq/blob/master/lib/api/entities.rb#L186). Only seems to be the standalone `Note` entity that exposes it as the other note types do not extend it.

I'll look more into this ready for 7.10.


Nope, it's "the value we're in the middle of constructing" - it's `this` passed from the constructor (hence the unusual parameter name). Clearly the comment isn't obvious enough at the moment though - any updated suggestions?


```
> Object.prototype.hasOwnProperty.call(null, "x")
TypeError: Cannot convert undefined or null to object
    at hasOwnProperty (native)
```


@as-cii I hardcoded URL here, I know, but I promise I will get rid of it when doing http://catchup.io/boards/3/cards/148. :pray:


Ah right, good catch.


"weird" is very kind.  This is a bug, and has to be fixed.  Somewhere in the type inference engine, something goes wrong.  It's likely in the lattice join code for dealing with subtyping of number classes.


leave a comment


@as-cii N.B.: removed `FORWARDER_URL` config from heroku, since only `FORWARDER_ROOT_URL` is used.


leave a comment and commit


... and checked if `FORWARDER_ROOT_URL` is already configured (which is).


Hey @databus23
This was intended to fix https://github.com/bflad/chef-docker/issues/278 which I was experiencing as well.

The behavior seemed really strange to me. Why would you specify a socket with `fd://` in front of it if you don't intend to use it that way?


@astrofrog Yes! I suggest https exclusively at http://guides.dataverse.org/en/latest/api/sword.html

Please feel free to make this into an issue or even a pull request.

In practice, we force https but it's inefficient to follow at 302 redirect when you can just hit https directly.


Ok :+1:


@yDelouis @dodgex this commit and 9de0be80 was a bad idea. It broke AA usage with JRebel and the debugger hot swap. Actually this entry is needed:
```xml
<factorypathentry kind="PLUGIN" id="org.androidannotations" enabled="true" runInBatchMode="false"/>
```

So the debugger and JRebel can figure out the code is used from the first eclipse instance. *.factorypath* generated by `m2e-apt`just adds the extenal jar from maven repo.

So i think we have two options:
* readding this file and *.settings/org.eclipse.jdt.apt.core.prefs*
* add a note in the wiki, that the generated .factorypath should be changed if one wants to use hot swap debugging

@WDYT? Maybe the second option is better, because just generating the file is more convenient instead of updating it always.


Double checked and there aren't any different health check options for "load balancers": https://cloud.google.com/compute/docs/load-balancing/health-checks

> Network load balancing configurations use HTTP health checks, as there is no TCP-specific health check type.

The thing that isn't currently supported by Terraform is [HTTP/HTTPS load balancing](https://cloud.google.com/compute/docs/load-balancing/http/) (as opposed to [network load balancing](https://cloud.google.com/compute/docs/load-balancing/network/) that we're using). I don't think there is an upstream issue for that, but based on the above it's not a pressing issue at the moment.


Strange, is it happening even with a make clean?


if  `dead_aircraft_time_ms` is negative the test is always false and the A/C is never seen as alive. Shouldn't it be the opposite ?


Its possible that this bug is still in the code if the string is a bunch of white space though I cant test it right now to see if it is.


The only tricky bit with absEVal returning ```v``` and not ```AVal v```` (which accommodates either a location value or an actual value) has to do with how do we implement ```EArrRead {}``` and ```EProj {}```.

If we *first* evaluate the whole array or struct to a value and then we purely project out or take a slice, then we have no way of recording the fact that we performed a side-effect on *only a slice* of the array or *only a field* of the struct. We will just say "oh I read the whole array" or "oh I read the whole struct". It has to do with our notion of what constitutes a side-effect. For the LUT table generation I want the analysis to be precise, e.g. if we are only reading 1 small field of a large struct it is very important to know this, NOT that we are reading from the whole struct. It's quite a nuisance but this is why I've done it like this.


This is identical to `contains`.


```
active = (dev for dev in devices
   if not dev.isDisk and
  ((dev.status and not isinstance(dev, PartitionDevice)) or dev.format.status))
```

has fewer redundant parts and seems a bit clearer.


Include language for intent - heroics


There is no error-handling caller. Nothing injects a MonoError* into this function. It should init and assert about the error it declares. In fact, this is only called in one place(a MonoAPI function), and only to pass a null as an extra argument. I'm going to move this body into the MonoAPI function and remove this function.


Sure the javascript code of the snippet could be enhanced a bit with a bit more digging in openstreetmap api btw.


yes, i've done repo sync and make clean then brunch


The notice:

``` javascript
Services.cpmm...
Services.mm...
Services.ppmm...
```
Not compatible for Firefox 38- (39+ OK)


Why do you have 3 times the same gradient here?

In second and third positions you should have the gradients with respect to the positive and negative terms respectively, i.e something like `... * (p - a) * 2/N` and `... * (a - n) * 2/N`. Am I missing something?


Why are the double quotes even being added? If the point is to make it appear in the format you'd use in C# code, then it needs to be escaped as well (although I would not use ProcessArgumentBuilder to shell-quote the string).


yeah eric is the bad guy...


Agree about renaming ```exportedElements```.

Regarding ```map```, I believe ```filter``` is more appropriate here as the resulting set is a subset. Anyway I agree and originally I tried to use array comprehensions, which appears to be the way es6 does ```filter```, but it seems the option to include support for them is not turned on. I saw that Object.keys was used elsewhere in the project so I used it instead. Would you guys rather I turn this on?


This is a nifty trick. Do you have a little script to do this? Perhaps you could commit that to tools and use it for the rest of our stdlib that just redirects.


EDIT: Actually these files do not have to be updated, only if a new annotation processing option is needed or we are switching to a newer version of codemodel (or add a totally new dependency).


Would it make more sense to put that messagebox here (just prior to where you open Pro?) (or am I just confused about what this button does - opens Pro?)


As a side note: do not use `and` as a boolean operator: http://devblog.avdi.org/2010/08/02/using-and-and-or-in-ruby/

For the scope. The best we can do is to use a subquery. There is support for this is Rails 4.2, however there is still no support for `or` clauses and to flip your conditional we would need one.

You could drop down to Arel which has `or` support but that's quite messy. Or you could just write a string where clause in this case.

Basically you could do:

~~~ruby
sql = <<-SQL
is_master = false
or product_id in (
  select p.id
  from spree_products p
  join spree_variants v on v.product_id = p.id
  group by p.id
  having count(v.id) <= 1
)
SQL

Spree::Variant.includes(product: [:taxons, master: :images]).where(sql)
~~~

This would be really messy in a scope so you might want to use a class method instead of the scope (they are basically equivalent). Or you could even extract this into a query object and write a class method on `Variant` for convenient access.


Sorry Benji but it doesn't make sense for me. Maybe I'm wrong but, should "Citation" always be above Suggested Citation?. As I see in other infowindows, they always work like this.

If you take a look at line 2003, Citation is just above Suggested citation but in translate text you have changed the order. I've made it manually following my own judgment, you can check it out on PRO.


Augh!  Thanks for the warning.  This almost definitely needs to be reverted.


yeah, should be fixed with 25d975e59cfaed96a13a9557ff929dfb03c8bfc3


This means there needs to be a connection on each module, since they don't recognize the parent `mongoose` dependency. Will most likely need an alternative to this, something that allows us to pass the parent mongoose connection through to the modules.


Updated


And to get back to your point about "reading" a variable, it is fine that ```go (EVar x) = lValM (varLVal x)``` because we are going to record the side-effect when we *use* that location. E.g. in  ```y := x``` or in ```y := x[3]``` (assignment RHS evaluate all the way to values)


I was unsure about this since the other fields didn't need to specify where in the form the data was coming from.. but without this I was getting null values. Would like feedback on this.


Well done, this definitely gives the right answer!

Few things to consider:

1. You're doing 2x more operations than you need to, meaning your iterations are O(n) + O(n), which is still linear but not good at infinity. Take advantage of the fact that you only care about one condition **OR** the other, perhaps with something like this:

```ruby
results = (1..999).to_a.select { |n| (n % 3 == 0) || (n % 5 == 0) }
```

2. Really awesome job on using `inject` to sum across elements! One really cool thing you should know is that you can send operators as symbols to that method. This was done because doing things like summations are really common operations done in Ruby. Also worth knowing - when you don't pass an initial value to `inject`, the accumulator value, by default, is set to `0`.

This means you can replace:

```ruby
puts results.inject { |s, n| s + n }
# with the following short-hand
puts results.inject(:+)
```

Much cleaner! Great job overall, glad to see you're doing this.

-- Carlos


In addition I've noticed that you don't have any sql statements which results in the input in the field when submitting a job isn't showing when you preview the job.


you can find me on hangouts - dadi11


Fixed via cbedbe6dc37e8018f513cd97737c4af0bd4c5506 :-)


Nice one @TYRONEMICHAEL


Hi @phuna,

Thank you.  I've inserted the new change into the plist.info.  I no longer
get the error, but the script doesn't open the email -- actually it doesn't
seem to do much of anything.  I know the script runs because upon clicking
the link in Things, I briefly see a gear icon in the status bar, but it
doesn't switch the focus to Outlook and does not open to the pertinent
email.  I'll continue playing with it later, but am swamped on other
matters today.

On Mon, Apr 27, 2015 at 10:29 PM, phuna <notifications@github.com> wrote:

> Hi @dylanor <https://github.com/dylanor> ,
> Sorry, my bad, the code to paste into Info.plist should be like this:
>
> <key>CFBundleURLTypes</key>
> <array>
>     <dict>
>         <key>CFBundleURLName</key>
>         <string>Outlook To Things Helper</string>
>         <key>CFBundleURLSchemes</key>
>         <array>
>             <string>outlooktothings</string>
>         </array>
>     </dict>
> </array>
>
> Please try if it's OK, then I'll update my guide
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/phuna/EmailsToThings/commit/1c759ec4e4991ca488bd46caab595ea9aebc590b#commitcomment-10932389>
> .
>


Did you mean `list` here?


And yes, there IS a one line version:

```ruby
puts (1..999).to_a.select { |n| (n % 3 == 0) || (n % 5 == 0) }.inject(:+)
```

Because why not. :yum:


In rsyslogd_InitGlobalClasses(), there's a call to objUse() on a number of classes, including net (with a TODO that it shall go away), parser, rsconf, etc.
In deinitAll(), other classes are deinitialized, but not via objRelease(), mostly, and net not at all.
Hence I'm not sure whether there's some kind of logic in it besides "faster" or "historical reasons".


What about inventions?


Done


Why `CMAKE_PREFIX_PATH` needs to point to `lib/cmake` folder? Usually it's used by `find_library()` to search in non-standard locations.


/* TODO
* printPerm 순서 이상해
* printComb 문제의도?대로 recursive도 구현
*/


This is because the cmake modules for Qt 5 are actually provided by Qt rather than included in cmake.


After I make this change and then the SolrServer node changes the main.uri
http://*.compute-1.amazonaws.com:8984/solr
Basically it is a problem in SolrServerImpl.connectSensors()
Svet explained that this is because getting port is based on location not on entity and since 8983 is already preserved by  RiakNode.SEARCH_SOLR_PORT the SolrServer.getSolrPort() returns 8984


I don't see an option as you describe here: http://babeljs.io/docs/usage/experimental/. Do you have a link to it?


I didn't think of that, if that is the case, I should be checking for
whitespace, or just trim the string before checking. I think I will go with
the latter.

On Tue 28 Apr, 2015 9:30 pm ShadowLordAlpha <notifications@github.com>
wrote:

> Its possible that this bug is still in the code if the string is a bunch
> of white space though I cant test it right now to see if it is.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sriharshachilakapati/SilenceEngine/commit/76a08793fb893a8806c1ea6a3342955ef2b3a5ff#commitcomment-10944815>
> .
>


Is there a reason why the $ssh_service_name resource was moved in front of the $mail_application resource? The order in the manifest doesn't change execution order.


Nope, you're right! I just added a few additional changes to the Spanish version. The order is perfect!


Why not relay the exit code?


I think a ```templates``` folder makes sense.

I think also naming ```factory.index.js.template``` to use plural```factories``` makes sense. I'll leave ```factory.js.template``` as is since it's a template for a single factory.

Will change the file name to ```generateFactories```.


Decoda


aca todos los r_vir = 1 entonces esas son las concentraciones?


The following changes are made in preparation for implementing the convolution networking described in [#14]:

* In the layer-specific parameters (Layer::params), the convolveMatrix member is now a [possibly empty] vector of convolveMatrix objects. Normal layers don't use any convolve matrix, filter layers use exactly one, and convolution networking layers will use more than one.

* Separated most of the topology config lexical parser into its own file named parseTopologyConfig.cpp and cleaned up its interface. The parser now recognizes the new syntax described in [#14] for convolution networking (the config syntax is recognized, but convolution networking is not yet fully implemented).

* Added unit testing in a new file named unitTest.cpp. Added a Makefile target "unitTest."

* Messages and comments were made more precise to distinguish between "convolution filtering" and "convolution networking."

* Removed the remaining "using namespace std" directives.

* Added class Logger to make it easier to catch, filter, or redirect the console messages from neural2d.

* Made the error and exception handling more consistent. The convention now is to print an informative error message using the Logger object, then throw one of a few custom exception types derived from std::exception.


Couldn't `CMAKE_MODULE_PATH` be used instead? I'm using `CMAKE_PREFIX_PATH` to specify R location and this line would break it.


This is a pretty big change, which distro's have you tried it with? Ubuntu / Centos


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


Burde egentlig vært miljøbestemt om man vil migrere eller oppdatere tabeller


There is a reson they call me Eric "unreadable PR" Larson


I did that at first, but the exit status in the case I tried was 256 and doing exit 256 gives back 0 for $?. I think I need to do some bit mask there, but I'm not sure what exactly.


Unfortunatelly I'm getting while trying to run ```Pkg.test("MLBase")````:
```
signal (11): Segmentation fault
unknown function (ip: -452283368)
unknown function (ip: -452075899)
unknown function (ip: -452073124)
unknown function (ip: -452072836)
unknown function (ip: -452302490)
unknown function (ip: -452104931)
unknown function (ip: -452152404)
unknown function (ip: -452074811)
unknown function (ip: -452073124)
unknown function (ip: -452072836)
jl_trampoline at /julia/usr/bin/../lib/libjulia.so (unknown line)
jl_apply_generic at /julia/usr/bin/../lib/libjulia.so (unknown line)
unknown function (ip: -451972013)
unknown function (ip: -451974847)
unknown function (ip: -451975336)
unknown function (ip: -451970447)
unknown function (ip: -451978131)
unknown function (ip: -451898129)
unknown function (ip: -451896367)
jl_load at /julia/usr/bin/../lib/libjulia.so (unknown line)
include at ./boot.jl:250
jl_apply_generic at ../julia/usr/bin/../lib/libjulia.so (unknown line)
include_from_node1 at ./loading.jl:129
jl_apply_generic at /lib/libjulia.so (unknown line)
anonymous at no file:11
unknown function (ip: -451899061)
unknown function (ip: -451896367)
jl_load at /lib/libjulia.so (unknown line)
include at ./boot.jl:250
jl_apply_generic at../julia/usr/bin/../lib/libjulia.so (unknown line)
include_from_node1 at loading.jl:129
jl_apply_generic at ../julia/usr/bin/../lib/libjulia.so (unknown line)
process_options at ./client.jl:308
_start at ./client.jl:407
unknown function (ip: -583508887)
jl_apply_generic at ../julia/usr/bin/../lib/libjulia.so (unknown line)
unknown function (ip: 4200475)
unknown function (ip: 4199375)
__libc_start_main at /lib64/libc.so.6 (unknown line)
unknown function (ip: 4199453)
unknown function (ip: 0)
```


-90 works for vertical. -45 works.
  But yes, +90 is whacked.
I discovered that also after the fact. Its just a matter of adjusting the
pivot point on positive numbers I believe.

Rotating text in d3 and or html5 can be confusing.

I have to build a circuit card generator so I am having to make some small
adjustments.
Another adjust is the ability for one shape node to butt up against the
other without a visual line connector. Basically a zero length edge. Like
gluing shapes together.

[image: Inline image 1]

On Mon, Apr 27, 2015 at 10:08 PM, Chris Pettitt <notifications@github.com>
wrote:

> I tried this out, but it looks it rotates around its top left corner
> instead of its center. Are you seeing the same behavior? Easy to fix?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/randallto/dagre-d3/commit/75be7da653c47c7ac657e9ecd5e760d6213ec123#commitcomment-10931224>
> .
>


Rather than writing `#` at the beginning of each line, you can use a Ruby `heredoc`, defined by `<<-symbol` and terminating with the `symbol` you choose.

For example, my `symbol` is **PROBLEM**:

```ruby
# This
# Gets
# Really
# Long
# And
# Ugly

<<-PROBLEM

But
This
Is
Much
Cleaner
Right?

PROBLEM
```


Burde finne en lur måte å samle alle mock-data på. Nå det en egen for seed, egen for test og en egen for FE. Kan skille ut alle mock-data i json-filer på BE. FE kan da oppdatere mock-data som brukes, slik at FE alltid har oppdaterte mock-data


Hi @irvingpop

I can't really follow, my point is that you basically filter out every type of socket in that loop. In its current form its a no-op, so the only thing that remains is the default socket you added below that: `/var/run/docker.sock`.
The result is a hardcoded `/var/run/docker.sock` when using systemd.

I think the real problem is this pull request which didn't do what the author intended: [https://github.com/bflad/chef-docker/pull/296](https://github.com/bflad/chef-docker/pull/296#commitcomment-10712133).

IMO setting the node['docker']['host'] to `fd://` for `init_type=systemd` is wrong. It should contain the actual sockets, the user wants docker to be available on. Its an implementation details of the systemd unit files that the docker daemon needs to be started with `--host fd://` while the actual locations go in the `docker.socket` as `ListenStream=` stanzas.


Estaria bien si tenemos tambien la possib de voxel-wise


Denne trenger en refaktorering.


darn ... but probably let-bindings too should return *values*, in the same way that control flow returns values? And maybe Sequence nodes? Maybe only things that are syntactically LValues should be allowed to return lvalues.


Burde også dekorere "app" med en logge-funksjon? A la som det er i gulp $log eller $custom.log ellenro


Just curious, why are you doing this check twice in a row?


Mh, right I think we would need to differentiate how high the exit code is

> The value of status may be 0, EXIT_SUCCESS, EXIT_FAILURE, [CX] [Option Start]  or any other value, though only the least significant 8 bits (that is, status & 0377) shall be available to a waiting parent process.

Since higher exit codes are used by the system to signal for example SIGSEGV


Burde også ha DB_SEED_ENABLED istedenfor DB_SEED_DISABLED. bedre at seed er disabled som standard. :dancers:


why wpa_supplicant cannot be in android.mk "already defined by device/lge/e975" but where?


rebase instead of merging, please?


Corrijido em um commit a frente


@MrTheodor, can you test that?


Corrigido em um commit a frente


SDL2 comes with PKGConfig if you want to try running that and using the output.


No probs.
TO DO list for next week.


Yes, thanks. Found out also by myself in between.


Coucou....


Meed a period at the end.


Agreed


Yes, thanks


Hello,

Is the systemui part working for you? or does, same as me trds also only works for Settings?
Also other apps like contacts can't be trds themed, they arent triggered to use the darktheme.


:+1: :+1: :+1: :+1: :+1:


These names cannot be deprecated; nixpkgs can be used with just Nix (no NixOS with its modules).


No, the aim is to remove duplicate names.  For example, a compound with the name "Benzyl alcohol" was listed 3 times with the compound IDs cpd00435, cpd02199, and cpd11784.  The last two compounds had the name changed to "Benzyl alcohol (dup2)" and "Benzyl alcohol (dup3)" respectively.  The abbreviation was also updated with the " (dupN)" suffix.  For many compounds, the name and abbreviation are the same.


Oh yikes. A lot of changes at once! I much prefer the NZ Pelican theme though. Not least because it has a 'flat' design like the old <TD> markup :^)

Link breakage?
Can we rewrite pelican slightly to mask the links? I think all you'd need is a .php >> .html or whatever symbolic link in the current dirn.

I also seem to recall that the NZ site was rendering to full URLs, which isn't great for backup + will break the website potential if you approach it from a weird angle

Photo archive plans?  I managed to blow away the NZ one, and it wasn't even obvious what I should have done / could have done.

Timed + sensible roll over period before replacing the /root domain?


6 menit cuy...


We're doing it elsewhere as well (see: optparse).  It makes copy & pasting a lot easier.  But adding something that automates the process to `jt` would be fantastic.  `jt unmask stdlib/json` or something.


It might be cleaner and more maintainable to have a submakefile in the test directory instead of having all the tests directly in the main makefile.


That's quite frightening; why is it necessary?


@runcom oh i was thinking of passing the timeout duration as `d.Shutdown(timeout)` that way you can put this whole logic in there (where i think it belongs).


One "expect" is *toThrow()* and another is **not.toThrow()**.  When I resolve promise on Line 1361, there should not be pending request.

The second case has one request to throw.


Works well for me for me locally.  I found a bug that I have confirmed is unrelated:  https://github.com/18F/midas/issues/748

I would be inclined to only do this for 403 errors.  I think it might make it hard to debug coding errors which might result in 404s or 500s.

Also, would be good to apply to add the admin views.  I must admit that this is the first time I've actually looked deeply at our administration UI code.  I wonder if the views could be refactored so we could put this logic in one place for all of the admin views?


RowMajorWithVectorization (amirroth) - x86_64-MacOS-10.9-clang: Build Failed

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


This is very useful, but if we're adding fields to the YAML, it should probably be thought through carefully exactly what we need + the best / most succinct way to express everything.


Had the same thought yesterday but ended up wrongly understanding what you said. Fixing..


I'm not sure what exactly do you mean.

Currently, `libraries` is called while computing ocamlfind libraries to link to. It calls out to `ocamlfind`, detects transitive deps on `nocrypto`, returns the extra (ocamlfind) libs that `nocrypto` wants, and **remembers** that.

The `configure` step is called later, takes the remembered information, and verifies that all of the extra libraries needed ended up being installed.

Lastly, `preamble` looks at that same information, and emits a preamble if the set of extra libs is not empty (and `nocrypto` is therefore in use).

The invariant is that `libraries` needs to be called before either `configure` or `preamble`, and the code checks that this is the case.

This ordering dependency is not super-nice, but the code that detects `nocrypto` has to a) run after all other things have declared their libraries, and b) before the call to `Mirage.libraries` finishes, to add its own entries to the set. It is therefore called as part of that function. Later in the configuration sequence, a full list of libraries is no longer available and would have to be recomputed, including the call to `ocamlfind`, so this code instead caches its answer.

Thus, if you skip opam, either:

* You transitively depend on `nocrypto` and `nocrypto` is not installed, in which case ocamlfind does not find `nocrypto` in the deps, entropy is not added to the linking set, preamble is not emitted, but the build process breaks early in whichever library you use that in turn needs `nocrypto`; or
* You transitively depend on `nocrypto` and `nocrypto` is already installed, entropy is detected, added to the linking set and activated through the preamble, and if the entropy is installed all goes well and otherwise `configure` warns you about this; or
* You do not depend on `nocrypto`.

Does this sounds reasonable?


Thanks @tmichel for the super suggestions.


So we can set SITEURL to "/caving/" ? does that still work?
Then all the links should be sensible local ones I would hope.


r=mbrubeck


I think @nirvdrum did this nifty trick first. I don't have any scripts or tool to add these.


dead code

If it's not implemented then add a TODO on the function signature


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.397](http://teamcity.monogame.net/viewLog.html?buildId=6449&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 484, ignored: 6 Build time: 00:14:56


Should this be:
```
if (root && [root isPlayer] && ![PLAYER weaponsOnline])
```
just to be sure that we will never have to access a nil pointer (even though it will be very unlikely)?


To catch typos, IRRC. You should use selectors in your recipe (like `# [win]` and so on).


I wanted to use the newzealand folder but andys trip was in there and the rsyncing gets rid of files so this just made it so they were "part" of the new site.


Oh I'm sure it works with KRAD lookups. Just wondering about the permissions check. What I expect would happen is it would bypass all perm checks here when it wasn't for the KNS. This might be ok for us, but is a little bit scary as I'm not sure if any customers are using these type of permission checks.


Ukladani zatim probiha tak, ze ke klici=zdrojoveUrl se prida cely feed. Ze zdrojoveho url se zparsuje RssFeed, ktery obsahuje hlavicku a jednotlive polozky. Do budoucna je potreba rozsirit toto ukladani o ukladani prirustku tech polozek feedu (jednotlive zpravy), abychon neztraceli stara data.


May I just underline how incredible dangerous 'rsync --delete' is?

Also, I think it's probably better / longer term more portable to have one rsync script, and have it switch $USER for you.

Personally I have a ssh-config, which knows who I want to be when I talk to @union , in spite of what user I am on the local system.
https://github.com/jarvist/filthy-dotfiles/blob/master/ssh-config


This section of the readme needs updating: https://github.com/yeoman/update-notifier#user-settings. Also, no tests for this?


Items jsou vlastne jednotlive zpravy feedu.


My bad, that makes perfect sense. :+1:


Here we need to take StdFds from the config
stdFds = c.Config.StdFds


Awesome @r4space ! Can you open up an issue in jira?


Great comments!
Small thing: line 120 (/1): Not texture file found... --> No texture file found...


Yep, I noticed too. Regarding the tests, see https://github.com/yeoman/update-notifier/issues/5.


Awesome work!


ReferenceError: placeholder is not defined xD


It has been a while since I dealt with this code, but I recall there being other types that derived from LanguageChain other than Expect (now called ExpectShould). Is this break of those types intentional?


Will do!

From: Chris Mattmann <notifications@github.com<mailto:notifications@github.com>>
Reply-To: apache/oodt <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Tuesday, 28 April 2015 09:30
To: apache/oodt <oodt@noreply.github.com<mailto:oodt@noreply.github.com>>
Cc: Jane Wyngaard <jane.r.wyngaard@jpl.nasa.gov<mailto:jane.r.wyngaard@jpl.nasa.gov>>
Subject: Re: [oodt] Moved scala plugin from core pom to streaming pom only (01afbe8)

Awesome @r4space<https://github.com/r4space> ! Can you open up an issue in jira?

—
Reply to this email directly or view it on GitHub<https://github.com/apache/oodt/commit/01afbe89953ef621379f906bb4e186a5eccf1e8e#commitcomment-10945509>.


So, are these enabled by default or disabled ?


hello !


That sounds quite fragile, and I'm pretty impressed that the original files survived a few rsync --delete rotations. All it would have required would be a permissions failure on the sh copying, or this code generally failing on a windows machine ( you get weird interactions with the ACL permissions when using the Python pseudo-POSIX emulation).

But anyway, the code is gone now :)


Yes, I agree. I'm in support of a fix that would do that for 2015.


@roji Each EF provider can choose whether to use GetValues or multiple calls to GetFieldValue<T> by configuring a different factory. Currently SQL Server is using the GetValues mechanism, while SQLite is using GetFieldValue<T>--see the differences in SqlServerEntityServicesBuilderExtensions and SqliteEntityServicesBuilderExtensions. The GetFieldValue<T> version is the most likely to work correctly so certainly you should use this for Npgsql. Using GetValues was at one point faster when using SQL Server, although lots of things have changed and so this may not be the case anymore. Also currently GetFieldValue<T> doesn't work for SQL Server on Mono.

Previous to this change it was not possible to use the GetFieldValue<T> mechanism. The original IValueReader implementation worked sometimes, but it could cause query to fail because it didn't buffer results. This change is specifically allow GetFieldValue<T> to be used in a way that is compatible with query, which is required for SQLite and it sounds like also for Npgsql.

With regard to boxing/upboxing. EF currently always creates an object array buffer, and hence always boxes the read value. This may mean that when using GetFieldValue<T> a value is unboxed in the data reader and then boxed again by EF, and hence GetFieldValue<T> may result in more boxing that GetValues--it all depends on how the data reader is implemented. However, we have a bug to potentially change the query buffering such that EF would not box the value coming back from GetFieldValue<T>.


add your favorite color


**[phpUnit_trunkIntegration run failed](http://qa-s2.wikia-prod:8080/job/phpUnit_trunkIntegration/7154/testReport/) for this commit**


* 3
* 4


Scheduler, kterym resim spousteni naplanovaneho stahovani feedu. Prozatim se stahuji jen ty feedy, ktere jsou predany metodou schedule. Jeste musim vymyslet logiku pro dynamicke pridavani/odebirani tasku, teda uzivatel si v gui prida odkaz a v implementaci to bude tak, ze puvodni ponecham a jen pridam tento novy task. Pri odebirani by to melo byt obdobne, tedy odeberu pouze ten, ktery byl smazan a ostatni ponecham bezet. Chci dosahnout toho, abych nemusel pokazde vypinat scheduler, znovu ho inicializovat, znovu mu predat vsechny zdroje, ktere ma sbirat a znovu ho spustit.


I'm glad that almost the only change that I coded so long ago, and was rejected, is again on track :)


this guy is good


Détrompe toi !


@asmeurer this was a backwards incompatible change and chance for review since there wasn't a PR.  When I found that this had happened there was also no explanation as to why it happened.  The commit message explains clearly what changed, but so does the code.

---

My original intent with this code was to provide a means of saying "if this file is there, always include it."  This changes that semantic to "this file is always required."  Unless there is an explicit use-case that this fixed that wasn't referenced in the commit log, I want to either:

* revert this
* change it to print a warning to stderr

Once that's done, we need to release 1.12.1 so builds that were working prior to 1.12.0 are working again.


I think this call of MASTER_LOCALE is not coorect -- it leads to a problem -- the publication format grid on the production page cannot be loaded (just loading...)
Maybe just MASTER_LOCALE instead of AppLocale::MASTER_LOCALE?


This definition is kind of a bug. It's only here because some code in inference expects `map` to behave this way. That code should be changed to request `Any` arrays, and then this definition can be removed.


Did you run this? It used the same theme.

Links: My plan was to include the old link in the metadata and then have the site generate a htaccess file to redirect them to the new look versions.

Yes it is possible to use just /caving/. That is a good point.

Photoarchive: Oh I dont think the photos thing was your fault. It was just the page with the links and I broke it just before we left for NZ without realising I think. Im not sure what plans are for future. I do think curator should be replaced and I will continue to work on the thing I made. If you have any suggestions...

This is just a test at the moment. Nowhere near being finished.


Is the Yak shaved now?


U ukladani feedu jeste musi udelat prirustkove ukladani FeedItemu do RssFeedu. Aktualne kazdym nove stazenym RssFeedem prepisu ten puvodni.


This will throw an error if you do `System.newModule(null)`

You can just do a `&& obj != null` here to fix it. Should I create an issue?


Yes, I forgot to cherry-pick a subsequent fix across. Done, thanks.


hello iKarith, i noticed that MAME 2015 not update since 1 month ago, any idea why?


@j-white I'm getting a ClassCastException (can't convert ClassPathXmlApplicationContext to NodeDao) in BSFNotificationStrategy at this point:
```
NodeDao nodeDao = BeanUtils.getFactory("notifdContext", NodeDao.class);
```
Changing it to
```
BeanFactoryReference bf = BeanUtils.getBeanFactory("daoContext");
NodeDao nodeDao = BeanUtils.getBean(bf, "nodeDao", NodeDao.class);
```
resolves it. However, this is on a fork of 15.0.2 with a couple BSF patches applied, so maybe the former would actually work in develop? Not a Spring expert, so thought I'd ask before submitting a PR with the change.


Sure, I this is something that will have to be changed. Also I originally made the script for myself (as its not hard to without) and Jack and you adapted it.


>Frankly, I was talking last time with my roommate about what you said on having model for other people. He was basically saying that you can easely predict the other people answer and that being so, the other people didn't matter anymore. I answered that, since you are the one interacting with them, if you can predict their answer, it's mainly your fault. You are the one that don't explore their equally interresting live.
Moreover, it's taking other people behavior for granted, and I don't like that.

So, the point I would like to explore and write about is that we should'nt model people. We do it automatically and it's a good short-cut but it can mess our relation.


unneccessary


@srikanthchitipotu  you didn't add any information!


I haven't looked into adding other repos yet, just got it working and then updated some colors. I don't use it, use a theme build myself but no I don't believe systemui was working correctly. The setup is different this time, I think you need a java call now for it to work in new repos


This is the exceptions that happens after this commit http://pastebin.com/3wGWq6Cz


@karenbruzzo

Glad you were able to figure this out.  Got a little lost when writing the pull request.

I love The Sound of Music!


ok but do you love NYC music?


Cool, thanks! Nice looking and simple: you can't have them both all the
time :(
On Apr 28, 2015 6:02 PM, "Tamás Michelberger" <notifications@github.com>
wrote:

> As a side note: do not use and as a boolean operator:
> http://devblog.avdi.org/2010/08/02/using-and-and-or-in-ruby/
>
> For the scope. The best we can do is to use a subquery. There is support
> for this is Rails 4.2, however there is still no support for or clauses
> and to flip your conditional we would need one.
>
> You could drop down to Arel which has or support but that's quite messy.
> Or you could just write a string where clause in this case.
>
> Basically you could do:
>
> sql = <<-SQLis_master = falseor product_id in (  select p.id   from spree_products p   join spree_variants v on v.product_id = p.id   group by p.id  having count(v.id) <= 1)SQL
> Spree::Variant.includes(product: [:taxons, master: :images]).where(sql)
>
> This would be really messy in a scope so you might want to use a class
> method instead of the scope (they are basically equivalent). Or you could
> even extract this into a query object and write a class method on Variant
> for convenient access.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sspinc/spree-google-base/commit/eb283352febe7d1dc63b34615e3703d8dcbacd16#commitcomment-10944882>
> .
>


.12 is an unlucky number? :smile:


Where is the depends Mako?


The old stuff is in the NZ git repo, it is copied over each time as well so there shouldn't be any way for it to be deleted (permanently anyway).


This is a line comment.


Yup!


Don't need an exclamation point


Little bit clear code.

var _ = require('underscore');
var $ = $ || requir('jquery');

And add jquery to deps.


Check this code.


don't need a period


Could you be more specific?


the trailing comma is not needed


Third time is a charm


This is my comment!


can you please rename `seconds` to `timeout` ?


should add an exclamation point.


I don't know .. go ahaed


es el inverso de las concentraciones


just checking if comments are shown


I'm following the suggestions of the Qt5 documentation: http://doc.qt.io/qt-5/cmake-manual.html


I agree ARBY'S is best


We need to merge this with the DataClient somehow so that everything can use this caching.


That's awesome! I have the current code in for review in a PR. Perhaps we can submit this in a subsequent update? (I'm thinking that there should be another update anyway at least once Clojure 1.7.0 ships and ClojureScript is revised to depend on it.)


@sreechandran hi!


The tests show the effect of setting extra_newlines to True, how does that get set for AMO since it defaults to False?


this line is good,
revert this change.
Now the functions are actually named _sticky_refresh() and _sticky_header_scrolling_actions()


Should have added more.


It is a new feature that was just introduced, enhancement by iu. I am pretty sure no one is using it.


this is not a bio


r+wc


Or... since this is in repl.cljs, it might be suitable as a separate PR against Himera, that doesn't depend on the updates I just made?


this is better than is replacement I think, revert also this


Should "sudo rsync" really be one string and not two?


If we want to add it, it will be an enhancement as well.


Hah I left a comment on a line.  Still 3 or 4 steps behind.  Ive missed new steps trying to figure this one out.  Arghhh1!!!!


same thing here, revert also these two lines


See https://github.com/jashkenas/backbone/pull/3421#issue-52623115 and my [response](https://github.com/jashkenas/backbone/pull/3421#issuecomment-67872505).


Great comments--as usual !

Fairly important: spell out conjunctions
(People may not know conjunctions and it emphasizes the NO aspect.)

Do not worry about it here (such as Couldn't --> Could and Don't --> Do not), but avoid it in the future.

Possessives are okay (such as The node's parent....)


Looks great @franks999 can you change the color of the "save" button?


Here is a line note.


Hey @mrericslone can you add another line to your file?


This will not catch an exception on linux. Maybe try WindowsError, IOError, OSError ?


I also have a Blue Crown conure parrot. he is 21 years old.


Dont need and an jquery to deps do we can use zepto or others libs


How to avoid empty comments?, I need to skip them


It should never leak out, but naively clipping isn't a good way to accomplish that. I'd preferably rather see the number shortened to less precision if necessary. (I don't like automatically changing the font size because it can mess with laying things out - essentially, if the user sets a font size, then we should render at that font size.)


I didn't realize it would be so controversial :). At any rate, I had considered this, but my feeling at the time was that if you need to handle different files for different platforms, use selectors.

I wrote this as I was building [this](https://github.com/conda/conda-recipes/blob/a52f1e04de66474347f278bccc82b3dee4e1055a/libgcc/meta.yaml#L19). As you can see, the potential for typos is high, and I wanted to avoid them.


Why nil instead of the default "root"?


We need you to fix this


I added a PR: https://github.com/ModuleLoader/es6-module-loader/pull/362


Additional comment on pull request


Ok, I will work on it


@donaldltrott  Please add more lines of content for me to edit.


I think it uses spaces as tabs. You might want to check the settings if there is an option to change it.


how awesome is it ? Why are we using pull requests ?


oh yeah, would be better as a separate PR to mainline.  I'll do that.


why using pull requests ? @githubteacher


But there they mention only setting "the CMAKE_PREFIX_PATH environment variable to the *install* prefix of Qt 5", not to the location of Qt5 cmake modules. `CMAKE_MODULE_PATH` is already used by rstudio, see e.g. `CMakeGlobals.txt`.


why using pull requests @dkehn


Why "=" at the end?


I don't have a strong preference for the standard, but we need to double-check that the behavior is consistent in different languages.


Doh! Sorry this issue may stem from using an old httr version in developing the PR.

Will require modifying tests too. I will do so.

But, when running as-is I get an error `Error: is.config(config) is not TRUE` from line 110. This disappears if I pass `config = list()` explicitly, which doesn't make sense since the function signature looks like

``` r
httr::HEAD
function (url = NULL, config = list(), ..., handle = NULL)
{
    hu <- handle_url(handle, url, ...)
    config <- make_config(config, ..., list(nobody = TRUE))
    make_request("head", hu$handle, hu$url, config)
}
<environment: namespace:httr>
```


what about a 3rd item?  haha


If I'm not mistaken, you don't specify that this (and `wineWow`) cannot be built on i686 systems which would cause Hydra failures.


Okay, let me experiment with this and see if I can get it working with the
CMAKE_MODULE_PATH.

On Tue, Apr 28, 2015 at 12:48 PM, Alexey Stukalov <notifications@github.com>
wrote:

> But there they mention only setting "the CMAKE_PREFIX_PATH environment
> variable to the *install* prefix of Qt 5", not to the location of Qt5
> cmake modules. CMAKE_MODULE_PATH is already used by rstudio, see e.g.
> CMakeGlobals.txt.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rstudio/rstudio/commit/6c7b139c2f8e167a1a2ff0df9ad3e15a3b5acf15#commitcomment-10945979>
> .
>


Rocco, you seem to have a much better picture than me of those changes.
Would it be possible to make a quick PR with those reverts ?


I guess this is where there's all the potential to go beyond a fully static site - and have tags and cross-linked lists.

I've had to Google search the site before looking for all the trip reports on e.g. Penyghent, in order to remember something that happened. So a set of 'caves visited' index might be really nice. If you automate the back end to do it via tags, then you can just leave suggestions commented in the template to avoid mispellings (i.e. standardise on 'OFD' or 'Ogof Fynnon Ddu' etc.)
Similarly with the hut, though thats perhaps less useful.

Similarly (though perhaps rather excessively), being able to cross-link + aggregate based on Name of caver could be really nice, but perhaps too much faff. You could then have a #TripTart leader list.


Concesionario con interfaz gráfica


we have `crash` for this now, right?  (i wonder why was it not used in the code that you are changing here?)  can you fix that?


Thanks!


Yes, string had to be escaped, fixed in new commit, logic for that commit was taken from https://github.com/mono/debugger-libs/blob/master/Mono.Debugging/Mono.Debugging.Evaluation/ExpressionEvaluator.cs#L108


I think you forget to call islandora_libraries.sh from the Vagrantfile after you split it out.


Typical Nikeo's sentence :P


Maybe we can add one line were root was explicitly specified just to make it clear that the default is root.


傻比辉


This may cause problem, since highlight_line is no longer a part of window.onload. Could this be rewritten to combine this entire script tag with the one in the head and redeclare onload function there to call both functions inside it?


otherwise it looks indeed better this way.  please fix and pass back to me for merge.


Just pushed another approach in PR #776 that fixes the underlying issue. If you prefer using `httr::status_code` as here I can figure out the issue mentioned above and fix the tests for this. LMK


...I mean it !!
;)


The description here needs updating too.


Would a single call of the function "fprintf" be sufficient here?


please indent all lines in one comment equally.


Manque un retour à la ligne


test


この箇所 ALTER TABL の記述、書き損じかもしれないですね。


Thank you for the review Pedro. Like this?


@vtjnash We're having some trouble here.  Specifically, if I let Julia call `__init__()` when she wants to (rather than calling `__init__()` manually as I'm doing here) the [names that I dynamically export](https://github.com/staticfloat/Nettle.jl/blob/8458e0f53cd501a2146bd5aa31dbe75873766af3/src/hash.jl#L94-L97) don't show up properly.  I can REPL-autocomplete to them, but attempting to access them causes issues.


Was the option `--rsync-path` tested on all supported platforms?


Les 3 lignes en dessous sont t-elle utile ?


C'est la que le code degeu commence


Will this give the exact endElementPosition?  Because already you have assumed null as 1 ( or you have any way incremented it).


rax retry tests


Utile ou pas ?


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


Här kom det en bibba filer!


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


Le bon refacto ! ^^


please at least use `Safe.fromJustNote`.


"I want the analysis to be precise, e.g. if we are only reading 1 small field of a large struct it is very important to know this, NOT that we are reading from the whole struct. It's quite a nuisance but this is why I've done it like this."

Ah, this makes sense now...I wasn't aware of the subtlety regarding read/write effect precision.


Manque un retour à la ligne


Is setlocale per-thread or per-process? If it is the later, this code has race.


Hehe, japp, sorry! ☺
/H

From: Johan Asplund [mailto:notifications@github.com]
Sent: den 28 april 2015 18:58
To: organismen/4fun
Cc: Håkan Wikemar
Subject: Re: [4fun] funkare? (cb25b56)

Här kom det en bibba filer!

—
Reply to this email directly or view it on GitHub<https://github.com/organismen/4fun/commit/cb25b56e0bf951ea63b433e20f16da660b8c82aa#commitcomment-10946216>.


Is there a way to drop "any" table that the worker you want to delete has, without prior knowledge of which table you need to delete on which worker? Maybe that isn't a safe way to do it though.


LGTM, but it may need work to fix the race if there is one.


Pas forcément trés clair le commentaire, enfait c'est pour tester si on n'envois pas un email si il a pas encore lu la derniére réponse


Shouldn't we break at the bracket and have one line for the cmd instead of two?


favorite color orange


Leaving a comment for you.  Hope to know more about your favorites.


In CommonJS style that is the right way.

```javascript
var _ = require('underscore'); // if you want too
var $ = require('jquery');
```

All deps capulate. Nothing is global. Backbone require what it needs in it own scope in the required version.

Make a decision to one framework like jquery.


This doesn't appear to actually return a ModuleItem, it is rendering out JSON messages instead.  This document line should be updated to reflect that.


Yes. They shouldn't have been LanguageChains in the first place. See [the documentation](http://chaijs.com/api/bdd/), where language chains are defined as:
- to
- be
- been
- is
- that
- which
- and
- has
- have
- with
- at
- of
- same

See also that the tests still pass with these changes, so nothing has been broken.


I think you're missing 'have been imported' or similar from the commit message here.


OK


fast-forwarding 2.0 to borshop-integration-1108-bugfix/dr/backport-hashtree_pid_fix_to_2.0 = 72b8bae6


favorite colr is orange


I wonder what the label should be for this transaction.  At least the way you implement it here, it doesn't become appearent that you are looking into the services dictionary, which could be a problem.

This is a though -- we may be changing the code soon and get rid of labels in transactions altogether; until the general concept has stabilized further, we don't have to be too exact with the correct permissions.


You can use `.where(user_id: user.id)` to avoid instantiating all the progressions.


@jt000 maybe `eventually` is supposed to be a language chain?


変数名、`n1`とか`n2`とかは何なのかよくわからないのでやめましょう
コメントで「最新の値」と書くならそれが変数名でいいですよね


Change this to RED


Change your favorite color to Red.


The name of this function is misleading; we're not deleting the requirement. maybe "uncomplete_requirement" would be better?


This is a string literal, we do not really want spaces inside our data.


ここで急に上で宣言してない`n3`が出てきますがスコープ違うので適切に命名しましょう


use `crash`.


Great comments !

The stuff below is for your future writing.  Do not worry about any of the stuff below now.

Advanced tutorial (breaking up sentences with commas and WHICH / THAT):

Main message COMMA (,) WHICH elaborates the noun right before the comma but is not essential to the noun.
line 3: ...the functionality provided by the assimp integration of GVRF which enables it....
...the functionality provided by the assimp integration of GVRF, which enables it....

Noun NO-COMMA THAT part is essential to describing the noun...
line 11:  All the 3D models which are being used to load....
All the 3D models THAT are being used to load....

Even more important:
Never separate noun and verb with a comma.
line 11: All the 3D models that are... loadScene(...) method, are having textures included....
All the 3D models that are... loadScene(...) method are having textures included....


handle left.  or at least add FIXME.


Yes, it's inlined ;-)


Why do you use a namespace in this file?
You could call the Frogsystem class by calling new Frogsystem\Frogsystem\Frogsystem(). Or you could use an use-as-statement.
But to import this file into the namespace seems quite unnecessary to me…


Oh! Good point. It should be 'c' for timecode - not 't'. Hm...


素数かどうかチェックしましょう


@HeikkiDev No compila , da un error en un espacio de nombre llamado PowerPacks de visual basic.....


looks interesting...you are adding some more new features..im  eagerly waiting...


We'd prefer not to instantiate every module and module item.
 `@context.module_items_visible_to(@user).where(content_id: @content.id, content_type: @content.class_name)`
but keep in mind that the same content item may be in multiple modules, or multiple times in the same module. this code doesn't appear to deal with this.

We should probably actually be searching based on the module item (content tag) id anyways. The item might have different requirements in different modules.


ループの終了条件は√nではないでしょうか.


lol


We do not have that API, and indeed I'm a bit worried about safety of this operation. Tables in CSS should be dropped as a result of table management operations (like actually deleting tables from all worker databases) and not through the node management API.


nil should only be possible if there's a subentity turret which is not a subentity of a ShipEntity. I doubt it would be possible to get as far as calling this method in that case, and I'm not sure how that would occur at all in current code.

I believe `[nil anyMethod]` will itself return `nil` anyway, though, so it shouldn't need the double check. They're supposed to be safer than C NULL pointers.


:shipit:


What is this?

> On Apr 28, 2015, at 1:06 PM, Sam Shleifer <notifications@github.com> wrote:
>
> lol
>
> —
> Reply to this email directly or view it on GitHub.
>


My commit? Pretty good

> On Apr 28, 2015, at 1:06 PM, Sam Shleifer <notifications@github.com> wrote:
>
> lol
>
> —
> Reply to this email directly or view it on GitHub.
>


"description" -- I know it is a nit, but this one is almost perfect now


The user can't set font sizes, though, only "big font when using single value". With this option unchecked, or when using two values, the font size is always "small":


This needs to be removed.


It's not a matter of controversial, I'm just a little miffed that my builds stopped working with no warning. :-/

Do you have a preference on the two options above?  I'm leaning toward the warning so there is feedback.


exactly .


Great comments !

The stuff below is for the future.  Do not worry about it in the above.

"Cannot" is always one word.
(Actually, there MAY be some two-word use that is justified, but I have not been able to come up with, read, or heard of it.)

line 319:  ... can not....
... cannot....


Bolon de Verde

El bolón de verde está considerado plato nacional de la República del Ecuador y posee como ingrediente estrella el plátano verde, del que Ecuador es uno de los mayores productores a nivel mundial. Tal y como su propio nombre indica, el bolón de verde consiste en una bola del tamaño aproximado de un puño que se prepara a base de una masa hecha con plátano verde, que puede bien asarse o bien freírse previamente. Dicha masa se prepara manualmente hasta que la mezcla queda homogénea y se le añade sal y algún ingrediente adicional, como queso o carne frita. Una vez amasado, se puede comer directamente o freírse en aceite hasta que la parte exterior queda dorada y crujiente.


It would be best if we could pull this onclick handling out to a separate javascript file, and either just refer to the function here or link it up with jquery or event handling logic outside here.

Also, you don't appear to be checking the result of the AJAX call.


Er, I meant more something like
```javascript
  if($("#facets").length == 0) {

  }
```


Yes, but I figured I'd be a better JVM citizen if I did what I needed and put it back the same way as before.


This function needs a rewrite. Its code is not so good and you'd better use `switch`.


@ecdsa What do you mean it doesn't bring security? It prevents people from spying on your traffic that should count for something right? Since all the data is encrypted anyway I didn't care much for https to begin with.

If you have a wildcard certificate feel free to use it and set it up yourself.


From this commit I'm not able to run the Doctrine unit tests. Doctrine is intalled as a vendor lib of my project.
Before this commit I was able to run:
``phpunit tests/Doctrine/Tests/ORM/Functional/Ticket/DDC3330Test.php`` from ``myproject/vendor/doctrine/orm``

But Now if I do the same, I got this error:
```
PHP Fatal error:  Class 'Doctrine\Tests\OrmFunctionalTestCase' not found in myproject/vendor/doctrine/orm/tests/Doctrine/Tests/ORM/Functional/Ticket/DD...Test.php on line 14
```

It seems that there is a problem of autoloading. Do you see what goes wrong?


clean


Thanks for catching!


I need to make this into an npm package github-webhook-emitter


this is neat javascript. keep it up


cool @Evryjazz :)


These are extensions to chai (not defined in chai itself) via [utils.addProperty and utils.addMethod](https://github.com/domenic/chai-as-promised/blob/master/lib/chai-as-promised.js#L64). I'm not certain if this adds them to the LanguageChain or to something that is only used for ExpectShould, but I think this should follow whatever chai does with them. I'll look into chai & chai-as-promised to confirm...


You are supposed to clone the repo to run the tests (rather than running them in `vendor`.

Indeed, the tests won't be autoloaded unless they are also in your root `composer.json`


@jmoody look at this line for example


The problem was use of ```!==``` operator instead of ```!=``` for comparison of junctionId (String) with element.id (number).


The copy's not strictly necessary, but it's good practice. Sure!


I prefer less corporate!


Just adding to the CMAKE_MODULE_PATH doesn't seem to do it (you still get
lots of configure errors about Qt5).

I hesitate to change this because our current configuration works in all of
our build flavors (development builds on all platforms, production builds,
etc.) and I don't want to expend a lot of time change this, re-testing on
all platforms, running down problems, writing wrapper scripts for
developers to add the extra -D whenever the configure, etc.

If you want to force a certain version of R on Linux or Windows you can use
the PATH. On the Mac it's hard-coded to the Framework directory. If this is
for the Mac I recommend a local patch or if you want a PR that to our
FindLibR.cmake that preserves our existing behavior but does what you need
in your environment.

On Tue, Apr 28, 2015 at 12:48 PM, Alexey Stukalov <notifications@github.com>
wrote:

> But there they mention only setting "the CMAKE_PREFIX_PATH environment
> variable to the *install* prefix of Qt 5", not to the location of Qt5
> cmake modules. CMAKE_MODULE_PATH is already used by rstudio, see e.g.
> CMakeGlobals.txt.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rstudio/rstudio/commit/6c7b139c2f8e167a1a2ff0df9ad3e15a3b5acf15#commitcomment-10945979>
> .
>


"Stop" --> "Terminating"
"an another" --> "a different"


@rsanchezsaez Does that constitute a review? ;-)


Yes, good point.


Me neither.


This needs to be `%config(noreplace)` so we don't blow away modified config files on update.

(OSTree has better default semantics here than RPM)


@cgwalters

I thought you wanted to use "rename" instead of "mv"?


Less typing!


I just noticed, it should be "debug", not "info" (everywhere)


@kjellm I've added inline comments that I received from one of the engineers who is an expert with Modules and the progressions functionality.

The biggest thing is in the MarkDonePresenter where you are searching based on content_id / content_type where they aren't guaranteed to be unique.  You should probably switch everything over to using the `module_item_id` which is passed as a query parameter to the wiki page view.


Minor, but can we call this file `docker-storage-setup.conf.sample` or something in the sources just so it's clearer that it's different from `docker-storage-setup.sh`?


This currently only works by invoking `ln -fs TARGET DESTINATION`, this is incorrect behavior


as you see, this controller (main.js) is getting messy. A better approach would be to define a controller for each user story. On the main page, use tabs for different user stories, clicking on any of those shows the corresponding chart which uses its own controller.
This way the maintenance is easy and debugging won't be a hell :)


i think we prefer lazy formatting


I like the "*-ing" type words better (but I don't feel strongly about it)


Something


Yay!


The default is whatever the chromium sources says. Look at build/common.gypi. Now, we can use CHROMIUM_EXTRA_OEGYP in local.conf to change the default.


If CSS is initialized correctly but there are no databases defined yet, what is returned?


extra space after "in"


I'm a little curious - to my untrained eyes this looks like a noop change? just moving out and naming the password bind? And () instead of {}.


What do you think about putting stuff like this in the wiki?


(好みだけど)配列を使ったほうがいいんじゃないかな
Arrayを継承してもいいけど

```ruby

fibonacci_sequence = [0,1]

fibonacci_sequence.instance_eval {|instance|
  def add_next
    self.push self[self.size-1] + self.last
  end

}

```

みたいなメソッド定義してやれば、

```ruby

while(fibonacci_sequnce.last < 4000000) do
  last = fibonacci_sequence.add_next()
  sum += last if last%2=0
end

```
といった感じで、外のスコープから見て気にするべき変数名は`fibonacci_sequence`だけになるし。


Should this be `pointerCursor: !options.crosshairs || true`?


That is actually different handling. Current analyzers do add a "lemma" (duplicate) for each non-hebrew word.
This was simply done to fit the current Analyzers.


sure. So is docker-storage-setup.conf sufficient? I wanted to avoid .sample as we could install this this file and ship our own defaults.


awww


Well we do have the text layouting algorithm I wrote for text screen back then, right?
in theory, that could be used.
I don't remember if it split long whitespace-less lines, though...


https://github.com/lift/framework/pull/1703#issuecomment-97091981

See that comment for the explanation.


こういうことですか！
![image](https://cloud.githubusercontent.com/assets/34204/7375852/4e953088-ee16-11e4-9047-777925d88523.png)


Recorda de canviar el camp de la Base de dades foto a varchar2(500)


[to, be, been, etc. are all added to Assertion](https://github.com/chaijs/chai/blob/3de55026458ace296df354757361953ec1949859/lib/chai/core/assertions.js#L45), which also where [chai-as-promised sets become, eventually, etc](https://github.com/domenic/chai-as-promised/blob/master/lib/chai-as-promised.js#L64). I think LanguageChain accurately represents the Assertion class (used under the covers in expect\should\assert). I'd suggest leaving it as is for reuse in case any other utils use the Assertion object & need an interface to represent it.

(that said, I'm not certain why they return Expect instead of LanguageChain...)


Done.


There are two password fields. The SHtml.password_* call was being made twice... once for each password field and thus the List[String] passed to setPassswordFromListString was 1 element. This means the password and "retype password" fields are not being compared.

Moving the call to a single field means there's 1 GUID generates and the two form elements are submitted and you get a 2 element List[String] and that makes the "are the two passwords the same" routine happy.


no sms ??


For now thats what we expect, I was trying to figure out what cURL does but it didn't say too specifically.


я тебя правильно понял, но у меня раньше почему то не работало часто когда я убирал лишние ожидания, сейчас из 7 раз один раз не сработало, может просто интернет тупит приодически


extra space after "Create"


Ok, will do.


technically, "SSSE3" :)


@Ocramius Thanks for your quick answer! Endeed I will clone the repo, it would be more clean.


@CrankNPlank it was later discovered that this PR breaks things like the `ng-include` directive. Discussed [here](https://github.com/jakemmarsh/angularjs-gulp-browserify-boilerplate/issues/32#issuecomment-91393900)


This is missing a `$`:

```
if test "x${KDU_SSE3}" ...
```


Ya, I am familiar with build/common.gypi, they are disabled by default


でもフィボナッチ数列だと2つとっておく数値の変数名、n1とかn2とかに俺もしちゃうなあ。というか最新の値じゃなくて先頭の値へのポインタだよね


LGTM.


:koko::end:


I'd rename this function. It is not only "checking", it is actually "creating" symlinks. How about update_root_symlinks?


Updated release with notes.


Ok, thanks for trying. I'm not on Mac, it's more weird. I'm trying to set up a working Rstudio environment within Gentoo Prefix on top of a rather old SLES11, while still using R installation that is outside of Gentoo Prefix. So, while in Gentoo Prefix environment, I need to expose as little "outer" paths as possible and things like CMAKE_PREFIX_PATH become more sensitive. I'll try to figure it out myself and file PR, if successful.


Are we trying to support python 2 and 3 or only 3?


Why do we get an error here? I know you're busy with fs so take your time


I wonder what will happen if one of these operations fail. My reaction is: wrap it in try/except and handle it, but if feels like that is a major surgery that would have to be done in many places. Maybe a new story?


My preference is to leave it as is. This was done intentionally after all.


@yjun2 given that all apps that provide browser functionality must use `UIWebView` per Apple requirements, and given the brevity of the methods and properties provided by it, you can see how much work has to go into creating a full-featured app like Chrome.


@yjun2 done with my review. checkpoint complete.


Don't depend on GC to close the NamedTemporaryFile object.


I see too many edge cases where this loops forever.


Hi @simod, yes this makes sense as an optimization. It might make sense to try to reduce the number of failed requests - this could be done if the caller knows which type of store they want (feature, coverage, wms). But also allow just looking by name.

The main difference is that the existing code allows for searching the entire catalog for a store matching the name.


↑ちょっとあれだったので忘れて下さい

```ruby

fibonacci_sequence = [0,1]

def fibonacci_sequnece.add_next

  ~~

end

```
でよかった
```


test comment


#4833-Documentation-for-HX-controlling-to-SP (rraustad) - i386-Windows-7-VisualStudio-12: OK (1647 of 1647 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)


@Niisp  can you speak english please? :) anyway i can't find hardware trees for u9200 -b cm12/12.1


Indeed, good find, thanks!


LGTM.


i do not know english. you need to fix the tree from D1 to P1
--
Отправлено из Mail.Ru для Android
вторник, 28 апреля 2015г., 20:29 +03:00 от "Andrea Turchet (aka rap_turkez)" <notifications@github.com>:
>@Niisp can you speak english please? :) anyway i can't find hardware trees for u9200 -b cm12/12.1
>—
>Reply to this email directly or  view it on GitHub .


@asmeurer yes -- but the original design was intention as well.  This is breaking builds where an error message would help.


@yjun2 well done here.


Test comment


One important test case that would be needed other than testArrayPrependFunction23 is
Create an existing array that has less than 255 nulls in the beginning and next a case where we already have around 255+ nulls in the beginning.  To that we will add another null.

Also a case where we have 254 nulls and then add a new null taking it to 255 so that all that logic related to nulls are evaluated and tested in the test case.


Hi resmo,

Why did you add the validate for configs? this validate runs [before to copy files](http://docs.ansible.com/copy_module.html) and if this files doesn't exist before the copy the task will fail


there is a '>' missing at the end of /canvas tag.
No big deal as html has a high tolerance for these kinda errors but sometimes it gets confused and then it's extremely hard to debug what's going wrong.
using an IDE like WebStorm would help catch these


Any reason you didn't remove the parent interface?


Ya something like

```java
if(text == null || text.trim().equals(""))
{
    return;
}
```

Will probably fix any case that it should be handling though remember that you don't want to store and use that string as it could mess up text formatting for anyone doing it with white space. Though if you do store it there is the possibility of saving small amounts of time rendering and that time adds up.


imo un-needed


Style these sections for better UX


Much much better to do Optional instead of throwing an exception.


Here too, I would remove this import logic and just use the python 3 import.


You forgot that strings are immutable in java. Using text.trim() will not effect the text in rendering.


Rename to Property


great idea!


I don't think this is a good idea. The only reason to have a static period is to indicate that script can go off and do things that might take time (e.g. load an image). If you need anything to be happening during this period like checking of key presses then you should remove the static period


@jeffreyway  Can you please merge this fix when you have a chance ;-)


This is a temporary name before I (possibly) remove Property


I didn't add that method yet


I'm not seeing the point of this method. Could you provide an example?


Looks good to me. Thx @sampritipanda


test comment


If you want to have `IMAGEKIT_CACHE_BACKEND` this is the only way.

Other approach (which actually can be better) is to hove something like `IMAGEKIT_CACHE_ALIAS` with default value `"default"`. This will ensure that there will be a configured cache by default instead of `DummyCache`.


This replaces the function of Data manipulators by restricting the type. I will provide more examples soon.


:cake: :tada: :balloon:


What's up with the formatting?


another way could be
```
for size in xs sm md lg
  for display in block inline-block inline
    responsive-invisibility()
```
but i cant figure out how to make them output as a list instead of individual declarations


Fixes jakirkham/nanshe#229.


what was the reason you changed the implementation of this factory? Isn't promise a better/safer approach?


No need for this restriction. Much better to unbound this interface.


Keeping it for now before I overhaul DataHolder


Go to Eclipse marketplace and add the Groovy dependency.


imo needed as a sanity check to keep data within one game, as well as some sort of base type for these marker interfaces


I did not forget it was done that way so as not to affect whitespace based text formatting if you want to do it the other way it would be `(text = text.trim()).equals("")`


No, the entire point is to keep the data bounded and typesafe.


what happens if the server is down? whenever you deal with a remote server, you should have a back-up plan in case you didn't get the response back.
The least you could do is to show the user an appropriate message on what went wrong (as opposed to not showing anything)


This method shouldn't even exist period. No assumptions should ever be made about a property being present.


В инструментах используем только точные зависимости.


PR is open #386.


Probably could have worded my other comment better though


Yeah, I borrowed most of this from Bootstrap’s stylus port and it seems super WET (anti-DRY).

I think individual declarations is fine. The way I think you could make them join up is have the extend a placeholder selector.


目前進度: 將原本依照觸控相對位置移動改為依照觸控相對位置決定方向及速度
未完成: 速度設置優化，使物件依方向速度移動


There was a typo for this commit reason. A new commit was made on top of this with the right one, but this is where the bulk of the input was initiated.


thanks, I'm on vacation away from my laptop, but I'll look into that when I'm back. :)


Haha damn Murphy. It's alright, no harm in keeping this rule in, it doesn't break anything


wrt to property: `(key in obj)` works for me. I have also seen `(obj.hasOwnProperty(key))` used in our current source code.

wrt stdio: I would not go with customStreams (at least for now). The definition of stdio explicitly mentions it is a dictionary of streams, with a few predefined ones (https://github.com/chromium/axiom/blob/master/lib/axiom/fs/stdio.js#L22). This is similar to what node.js does, except we use a dictionary (vs an array with predefined entries in node.js).


There are some cases where it is easier to deal with an exception than working with optionals, like working with massive amounts of implied data. I agree that for the majority of use cases an optional is abetter idea


Oops, now fixed!


GameData should be the parent interface then.


Because it is not a predefined entry. Think of it as an environment variable.


yay, another top-level directory!


Looks good for merge.


Great! Seems to compile fine now on Kakadu 7.6. Thank you!


@rwaldron As per https://bugs.ecmascript.org/show_bug.cgi?id=4334 , this statement should be enclosed in a `assert.throws` block. It probably makes sense to throw


rax retry tests


short-id is a generic directive for shortening ids, the id passed to it should just be the alphanumeric part, handle the replace elsewhere.  Also is sha256 the only algorithm we are going to hit, shouldn't we be generically splitting on the : character instead?


todo


...an instance of a custom constructor to avoid false positives with erroneously-generated native errors.


As long as we use the GZ C Interface it's not possible to use something else then file.
Currently not.

To boost path, I don't like the overload operator "/=", usually "/=" is a division.
So my personal opinion is to think about another solution. (No specific proposal at the moment)


Please make a separate PR for this patch.  The patch looks fine, but it has nothing to do with the DIAG GRUB menu entries.


The /survey/offline endpoint only exists in /api/v2, so you probably want to change that.

-----------
The below is maybe not relevant for this PR but fwiw, I would use something like the following names to make it easier to refactor later:

actual settings people should configure when installing:
ENKETO_API_URL => 'http://something.something/api/v2'
ENKETO_API_KEY => 'enketoisthebestest'

not settings, but API endpoint constants that should not be touched:
ENKETO_API_ENDPOINT_OFFLINE_SURVEYS = '/survey/offline'
ENKETO_API_ENDPOINT_ONLINE_SURVEYS = '/survey'
ENKETO_API_ENDPOINT_PREVIEW = '/survey/preview' (requires changes in code to use API for previews instead of hardcoded url)
ENKETO_API_ENDPOINT_INSTANCE = '/instance'
ENKETO_API_ENDPOINT_INSTANCE_IFRAME = '/instance/iframe'
and many more...


Whoops


Maybe we should just put only his links on goodies every week and leave the rest out.


a to odwołanie ma sens w bibliotece?


should this have both `WC_API_Sever::CREATABLE` and `WC_API_SERVER::EDITABLE`?


  >>next a case where we already have around 255+ nulls in the beginning. To that we will add another null.
I just saw that testArrayPrependFunction23  is doing the above.  We may need the case where we are changing from 254 to something above.


I believe it would be enabling the Stage 0 transform, you can see Comprehensions listed there. It's noted in the docs that they are disabled by default: http://babeljs.io/docs/learn-es6/#comprehensions.


Тоже самое и для dev-зависимостей.


Looks good to me.


reversing the conditional here would be more readable IMO


Зачем понадобился `chai`?


@madamski, dodaj jeszcze jakieś składniki, bo po usunięciu przycisku dół okna jest pusty.


nice usage of correct HTTP response code :heart_eyes:


In the description, please read:

`Adds required javascript to transform any Drupal (Omega 4) *menu* into a small touch screen specific button.`


it doesn't work with whitespace-less lines. It just makes those invisible.


Thanks, great feadback!


I don't like this. I think we need to make this configurable with a pattern or something similar. I don't want to see the assignee name as long as the bug/ticket is not in progress.