TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 650](https://teamcity.labkey.org/viewLog.html?buildId=272834&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:24:50
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

##### there are 23 more failed tests, see build details
```


it makes sense. I'll pay attention to unit tests!


Presumably there was a point in time where that test was needed and then he introduced a bigger hammer for sym==rvv and rendered this logic dead.


:sheep:


Funzt.


Lost conversation :(


possibly tmi?


"created at" is already in the writeCreate method.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.478.c21701cba8d45097f7d682bb2ec4cbb62af3bd1a](http://build.palaso.org/viewLog.html?buildId=63389&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:16:53


See https://github.com/kilbith/xdecor/commit/007700a6e8b18f731b02f49055308cb301006702. That was a silly mistake.


"is offscreen at" is already in the writeOffscreen method.


Update as V2


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1305


Sorry (esp @pjf ) I meant to make this a PR, but just clicked the Big Green Button on autopilot. :(


Hi Alain, some quick remarks:

a) you're not testing against the latest master, is that on purpose?
b) I sort of remember that last time I checked, calculating "supporting" with `neighbours & pawnAttacksBB[s]` was faster than with `neighbours & rank_bb(s + Up)`
c) perhaps more importantly, I think that the original back code *does* consider b2 as backward in the a3b2c3/a4c4 shape which you describe. What your code does, if I'm not mistaken, is that it *removes* the backward penalty for the a3b2c3/a4c5 and a3b2c3/a5c4 shapes, etc.

Hope this helps,
Stéphane


@mattleff review please!


Agregamos una linea


@carlbrugger Boom.


Can you get the source names to be all the same (and use a constant)? Other than that LGTM


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


You won't be able to compare the two...since this version is stronger than the one in the past.
So you don't know if it's from recent functional changes or by your scale eval...since the masters are not the same.


@PEMapModder License header should go before package declaration.


It's a bit nitpicking, but the space can also be removed.


The trick will be getting the device to specify all this information up front.  It's not calling this API directly either, it's iframing a content-server page that's calling this API on its behalf.  I think we should allow any or all of this information to be missing and fill in sensible defaults where possible (up to and including passing `device: {}` for cases where we know nothing about the device up-front.


@Zephor5 I think you are right, it is a rare dependency only because of Scrapy extending queuelib's testsuite but it is fair to point this dependency for tests requirements. Fix https://github.com/scrapy/scrapy/commit/a3390afc66134e77f98ae3bfae7bc23479ed8566


oneplus one (bacon).

I implement this code in my git. www.github.com/TeamNuclear
Repos: frameworks_base -> branch batery
            packages_apps_Settings -> branch batery

Thank you for this code!!!


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


fast-forwarding master to auto = dcda8bbb


Is "the other info" here the other device info, or the other account info?


PR here: https://github.com/radare/radare2/pull/3312


I would like to see some unit tests for these, please.

See: http://www.django-rest-framework.org/api-guide/testing/


add a comma?


If oauth, how do we know which device is being modified?  Optional id in URL, or POST body?


Oh, I guess this is answered further down with `id` in POST body :-)


Yes. From the start, I had used the same Z value for the camera than the game camera. But the creatures were sometimes displayed behind the game. That's why I used a smaller value (to make sure it is always displayed). But the consequence was that the particle effects or the entities attached to the picked up creature were much too big


The submenu not working properly in mobile devices


It's a code comment that raises an issue with the code.

Nope but this stuff isn't tested outside of my own system. Once it's done I can throw it at my VM army.


Realize that the existing test cases are this way already, but man is that a lot of additional lines, just to test one component of `postal_code`. :confused:


We are inactivating this key.
Thanks for your input.
If you have any idea how to delete history in github, please share.
Thanks again.

From: Adam Ochonicki <notifications@github.com<mailto:notifications@github.com>>
Reply-To: dell-oss/Doradus <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Thursday, September 17, 2015 at 11:07 AM
To: dell-oss/Doradus <Doradus@noreply.github.com<mailto:Doradus@noreply.github.com>>
Cc: Tra Duong <tra.duong@software.dell.com<mailto:tra.duong@software.dell.com>>
Subject: Re: [Doradus] Added eb directory for deployment (d88343b)

@TraDuong1<https://github.com/TraDuong1> this key gave me access to Dell OSS' AWS stack. You probably didn't want this public.

—
Reply to this email directly or view it on GitHub<https://github.com/dell-oss/Doradus/commit/d88343b8c7fa810b027d41cb0287abbdf6134225#commitcomment-13294408>.


metalsmith?


:+1:

I guess it would be safe to include the sessionToken ID, but there's nothing useful that other devices can do with it anyway.


lol it was intended to split oceans and biomes like beaches do :P


Hi,

I've done a rebase in my branch based on the latest merge request in the
develop branch. All conversations in commits were lost :( Can I make the
pull request?

Em sáb, 12 de set de 2015 às 15:02, Maxim <notifications@github.com>
escreveu:

> Good day, we were glad to see some of your changes in the official
> repositories, if you also want this, please make a pull request)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/marcoshoo/propel-laravel/commit/ae77d53599fdd79c883b4d975b4a716a46d5f9d9#commitcomment-13197491>
> .
>


We could include the last-access and UA info from each sessionToken as well.


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.5.56.c21701cba8d45097f7d682bb2ec4cbb62af3bd1a](http://build.palaso.org/viewLog.html?buildId=63388&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 874, ignored: 50 Build time: 00:32:08


FYI: this line throws an EINVAL exception when running node 0.10.38 in the git bash shell on Windows 10. Work-around is to run grunt in a CMD window or to comment out this line.
Reference https://github.com/nodejs/node-v0.x-archive/issues/6459#issuecomment-34620129


I identified three other similar redundancies, two in `RPC::Context` alone, but for the moment the goal is full dependency mapping.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - i386-Windows-7-VisualStudio-12: OK (1873 of 1880 tests passed)

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


We can always do it later with POST `/v1/account/device`


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


Fair enough.  :+1:


Maybe change `git_dir` to `repo_dir`? Sounds semantically more correct.


I merged this, but I'm not a huge fan of all the gray backgrounds


indentation


Python3 Script to convert Fahrenheit to Celsius


themis_components/thDropdown/examples/3 - with th-item/coffee.coffee

I don't think we need an example showing off mixing a controller list and markup list. It might just be confusing. Perhaps take the list out of this one…


yeahhhhhhhhhh!


Right, exactly - specify as much as it can up-front, then fill in the rest as a follow-up call to update its info.

This may also open some interesting UI opportunities, where we can add a page to the content-server that lets you edit the device-name inline in the setup flow.


nit: call it ``cConf`` for consistency with the rest of the code.


thanks! I'll pull this into my rollup actually


I like that this is tested and immutable but I don't think it should be exposed as `ui/metadata`. Previously only `ui/chrome` had access to these and the rest of the application `require`d `ui/chrome` and accessed these values via the getters/setters of the chrome API.


asdasd


Meant to ping @epixa


B I A S E D
I
A
S
E
D


this is my first commit


I think we should prep a way to disconnect/delete devices here as well.  Even if we're not going to expose it to users right away, the device's own state-management logic will need to take the possibility of being booted into account.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-MacOS-10.9-clang: OK (1867 of 1874 tests passed)

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


its ``- 2 * numBatches`` in the code but just ``-2`` in the comment


Thanks, you are right.
I will need to think about this once I get some sleep.


So I checked back on master, this fixes the bug that was preventing you from dragging an section into a different chapter and saving it. I'm not sure if this even comes up often enough to matter, but I guess its a nice option to have.


Nice :+1


absolutely


Could you use makeThirdPartyPath(group) instead of thirdparty.Root here? Because I may change the Root to "apis/" in my PR.


fast-forwarding hotfix/2.0.6-2012.08.25 to borshop-integration-553-bugfix/zl/trycatch-anti-entropy-index-call = cb12c904


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

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


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 651](https://teamcity.labkey.org/viewLog.html?buildId=272837&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 33, passed: 9 Build time: 00:27:15
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

##### there are 23 more failed tests, see build details
```


Either we should use startIndex here or we should remove the variable entirely.


Makes sense..Done


@watjs I've been successfully using `Angular` on `Android 2.3 WebView` for almost 2 years now :smile:


DUDE LEZGO


was a small conflict with one of the other backports on tk/backports3 but easy for me to resolve this time - ref 6457fd3e24fafe65284be7200104a42073c06fa0


:guitar: done


:cool:


test was stopped by fishbench because one bogus worker suddenly had "wrong signature"


I get class cast exceptions on my BSON files because this is DBObject and not BSONObject - is there a necessary reason to use the subclass and not the more-likely-to-cast superclass?  (Or, if you know why my bson dumps are causing this error, I'd enjoy fixing it on my end more than running a custom hadoop connector)


I liked the classic one better as it used for a lot of python style docs with the green and yellow background. Do you prefer the classic better?


Did this patch work for anyone?


See the first lines of [setup.sh](https://github.com/enterprisemediawiki/Meza1/blob/master/setup.sh) for how to make the prompts force the user for input. By putting the while loop around them you force some sort of input.

Watch your indenting, too. Your "then" statements are at some intermediate indenting which is confusing. You have:

```bash
if [ $bitversion == 32]
  then
    if [ -a "$isofilepath/CentOS-6.6-i386-minimal.iso" ]
	  then
	    isofile=defaultisoname #change this
	elif [ ! -a "$isofilepath/CentOS-6.6-i386-minimal.iso" ]
	  then
	    filelist=$(find $isofilepath -maxdepth 1 -name "*.iso" -exec basename '{}' \;)
		select filename in $filelist
		  do
```

When this may be cleaner:

```bash
if [ $bitversion == 32]; then
	if [ -a "$isofilepath/CentOS-6.6-i386-minimal.iso" ]; then

	    isofile=defaultisoname #change this

	elif [ ! -a "$isofilepath/CentOS-6.6-i386-minimal.iso" ]; then
	    filelist=$(find $isofilepath -maxdepth 1 -name "*.iso" -exec basename '{}' \;)

		select filename in $filelist; do
			NEXT INDENT HERE
```

Also, I think you're attempting to loop over the contents of the $isofilepath. You may want to look at [import-wikis.sh](https://github.com/enterprisemediawiki/Meza1/blob/master/client_files/import-wikis.sh), which does the following to loop over all directories in the $imports_dir directory:

```bash
cd $imports_dir
for d in */ ; do
	...do some stuff...
done
```

You of course want to loop over files, not directories. And specifically you want files ending in `.iso`. So yours may look something like:

```bash
cd $isofilepath
for f in *.iso ; do
	...do some stuff...
done
```

This would, I think, look more than one directory deep for iso files, but who cares? What's the likelihood that someone is going to drop their iso file into a directory that has tons of other iso files buried deep in the subdirectories?

I haven't tested your method, and the select method does seem like it may be a good one, so by all means continue using it. Just offering suggestions.


I imagine 'animate me' still isn't working for you? 'filetype' is the wrong parameter name
The correct parameter name is 'as_filetype'

https://developers.google.com/image-search/v1/jsondevguide?hl=en


Why is this needed? Ideally, this optimization should not have to know anything about the column containing the hash (it's a detail only for joins and aggregations)


private


:dart: yes that makes sense. Got rid of it


All conversations were lost in the rebase :( This change is required when the table name differs from class name. Ex: My model is called  "User" but my table is "users".


Oh, as an FYI, you might want to grab (slightly modified) two commits I just pushed to the gesture branch that solves the array pointer problem in a much better way (since that branch adds more string-returning functions, wrote a class that does the hard work for you and using vector)


16-rd toxn avelord er


Alright, I checked it out and published. I took a brief look at observable collections but thought better of it this time around. I included the XML docs as well. Let me know how the packages work. Last but not least be sure to rebase if you care to follow along. Thank you...


thanks ! :)


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

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


Ionic was following the same approach as well so I think this is right.


all tests pass:
success: http://buildbot.bos1/builders/python-client-test/builds/359


@dggrj Can you please file an issue on https://jira.mongodb.org/browse/HADOOP? Please include:
- stack trace and pig logs
- reproduction steps (including Pig script, MongoDB document)


Hahaa


Looks great!


Is there a reason why this needs to be hard-coded and not configurable? Can we add a comment explaining why?


That's what I've done also - went back to version 2.8 for raster displaying. Strange how such a basic function was forgotten in MacOSX version 2.10. I'll be paying more attention in the future builds to try to give my contribution to detect these bugs into the MacOSX compilation


OK fair enough. Thanks for the details.


This could be simply called "Flamenco components".


to vendo um if :-1:


#include <iostream>
using namespace std;
int x, y;
int r=0 ;
void sum (int x, int y)
{
  while (x<=y)
  {
    r=r+x;
    x++;
  }
}
int main ()
{
  cout<<"We will calculate the sum of integers in the range you provide"<< endl;
  cout<<"Please enter lower bound:"; cin>> x;
  cout <<"Please enter upper bound:"; cin>>y;
  if (x<=y)
  {
    sum (x,y);
  }
  else
  {
    cout<<"This is not the way you should do that, we need to change the bounds around, and now it works.";
    sum (y,x);
  }
  cout<< endl;
    cout<<"The sum is:"<< r << endl;
    return 0;
}


this should be a subtype of the cyborg esword.
it has no real reason to be a circular saw (any item can be made to work in surgery), but it DOES have merits being an esword, the fact you've pasta'd it's code here.


@vspinu there it is.

* No blinking when we get back to original state. This gives a clue that we cycled back to start.
* When the original state and the first reformatted state are the same, go on as usual (otherwise nothing would happen during that step)


@rahulbot - i haven't finished setting up oauth2 so there isn't really anything to test out yet. Also I unfortunately won't be able to work on this until after 9/30


Also #41


Sorry for the nitpicking, but in all the documents we try to have only one h1 per doc. This means that this title should use something like --- instead of ===. And no semicolon at the end of any title.


@Spartacus1 thanks, that's exactly what we need! More OSX users testing development builds and reporting bugs prior to releases.


Thanks for the corrections. Besides the two remaining notes, it's good to go!


I wonder if the difference with continuations would be clearer if the whole function call got blinked (i.e., including the function name, not just inside the parens)


Lets use ng-switch here as we discussed.
http://codepen.io/anon/pen/BoKJXd


`ng-click`


`href`


Ah, yes.  Thanks.  I've fixed this now.


@maxbrunsfeld helped on this commit.


Hi Mark, I cannot compile anymore on Windows (QT5.2.1) nor Ubuntu (QT4.8.4) after this commit, errors are:
../httpserver/httplistener.cpp: In destructor ‘virtual HttpListener::~HttpListener()’:
../httpserver/httplistener.cpp:28:5: error: ‘QT_MESSAGELOG_FILE’ was not declared in this scope
../httpserver/httplistener.cpp:28:5: error: ‘QT_MESSAGELOG_LINE’ was not declared in this scope
../httpserver/httplistener.cpp:28:5: error: ‘QT_MESSAGELOG_FUNC’ was not declared in this scope
../httpserver/httplistener.cpp:28:5: error: ‘QMessageLogger’ was not declared in this scope


\>Нан**а**роботы
Кхм.


No I rebased on the same master for exactly that reason.


empd (nmerket) - Win64-Windows-7-VisualStudio-12: OK (1875 of 1880 tests passed)

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


"Нанороботы" сюда не подходят. Предлагаю исправить на "наномашины", как и задумывалось в оригинале.


My question is:
this change: local DIAL_STRING_BASED_ON_USERID = false  mames system exclusive,  red or green, not both.  And in multitenant environments, it is very common to have one tenant in red, and other in green.


Oh I see now...


Gah!  Cruft.


Does scaffold.scss mess with other styles?  I seem to recall I have to remove it when generating.


How hard do you think a `transform` function would be?  so we can do `length(transform(geom, code, code))`


It was a question of timing because 2.10 Pisa was released after the end of my GIS classes :)
I use the MacOSX version and many students also use it and this error would be easily detected.
In version 2.8, for instance, the most annoying bug was the Print Composer inability to plot coordinates outside the frame...
Thanks and keep on the outstanding work on making QGIS the best GIS tool!!


Yes, please destroy it.  I didn't mean to add it.  Sorry. Please kill all the cruft.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Some thoughts: I am going to add a forced landing if GPS is lost for more than some reasonable time (1-2s perhaps). This landing will be handled by NAV code, no need to take any additional action on it here - current code is fine.


Is it responsible for us to ask people to post their address publicly on the web?  Or should we do like for reports and ask them to post contact info?


Gah!  More cruft.


Oops, my example url was a bit of a guess, looks like 'http://imdb.com/name/<id here>/' is a nicer example.


Never mind I see we already have it.  Didn't look hard enough.


Great Idea!

We still need the `index.rst` file which does not get auto-generated here and this is the file we will augment later as it serves as the homepage for the documentation. I have also changed changed the name of the directory from sphinx to api_reference


Lists traditionally have more semantics, significantly starting with index 0 and being contiguous until `size - 1` (assuming size > 0). Also, negative indices are sometimes implemented, but certainly not always.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - Win64-Windows-7-VisualStudio-12: OK (1873 of 1880 tests passed)

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


Will expectedObj.ObjectMeta = list.Items[ix].ObjectMeta work?


Do we even need to use regex here anymore? The config should have already validated we have a nice id.


Since this is in the process of modification: Thoughts on #17? Independently of any GUI-related possibilities, it makes sense given the unweighted signal is part of the fit that it should be included in the output data rather than separate.


Este patrón también se conoce como el patrón de publicación-inscripción o modelo-patrón. Estos nombres sugieren las ideas básicas del patrón, que son: el objeto de datos, que se le puede llamar Sujeto a partir de ahora, contiene atributos mediante los cuales cualquier objeto Observador o vista se puede suscribir a él pasándole una referencia a sí mismo.

El Sujeto mantiene así una lista de las referencias a sus observadores. Los observadores a su vez están obligados a implementar unos métodos determinados mediante los cuales el Sujeto es capaz de notificar a sus observadores suscritos los cambios que sufre para que todos ellos tengan la oportunidad de refrescar el contenido representado.

![estructurapatronobservador](https://cloud.githubusercontent.com/assets/14063724/9948656/551ff3d0-5da5-11e5-9e06-22ed16f637d1.png)

De manera que cuando se produce un cambio en el Sujeto, ejecutado, por ejemplo, por alguno de los observadores, el objeto de datos puede recorrer la lista de observadores avisando a cada uno. Este patrón suele observarse en los frameworks de interfaces gráficas orientados a objetos, en los que la forma de capturar los eventos es suscribir listeners a los objetos que pueden disparar eventos.

El patrón Observer es la clave del patrón de arquitectura Modelo Vista Controlador (MVC).1 De hecho el patrón fue implementado por primera vez en Smalltalk's MVC basado en un framework de interfaz.2 Este patrón está implementado en numerosos librerías y sistemas, incluyendo todos los toolkits de GUI.


Hm, I think that *Strings* is countably infinite.


Use `at(index)` instead of `[index]` when getting values from QVectors.  It is faster because Qt knows that the value will not be modified.


This got removed because this logic is now included in `TypeOf`. There is no need to move the information from `Defs` or `Uses` to `Types` any more.


I guess your regex ran afoul there :P


Ditto


This is temporary? We should be explicit about when we're clearing the resolved status, not as a side-effect of setting status.


Start commit messages with a summary line like this:

    Add virtual to EventQueue destructor.

or

    Make EventQueue destructor virtual.

Then put any extra information necessary after a blank line, including stuff like 'Close issue #22'. Otherwise it's difficult to see what a commit does when just looking at a list of commit summaries.


Issues caused by GSON deserialization and obfuscated field names.


`*types.Basic` is intended to fall though for unsafe pointers. But you're right, I should at least add a comment for this case.


Noooooooooooooo, we should add python2 on `python` instead!


Well this is sane.


I believe you have this a bit backwards.  You will likely want a list of XPSScan pointers (`QList<XPSScan *> scans`) rather than a pointer list of XPSScans (`QList<XPSScan> *scans`).  Could you explain why you switched the list to being a pointer?


Errm, don't delete these two lines? :P


`$SRCDIR/**/*.py`


This seems wrong. There's no "retry on error" remote state, just the resolved mode.


Added :heavy_plus_sign:


Forgot to change ```"V8X_VERSION_MINOR "3"```


It was necessary for the subsequent commits (and it makes more sense).


executeActionInternal is now callAction


Yes. I thought about that and assumed you wanted to handle that as part of RTH code.

One more thing i thought about was when landing is forced by a lost GPS, you may be above water and use a sonar (if present) to burn the rest of your lipo at *very* low altitude. Until either GPS lock recovers or lipo goes down (splash...)


I was starting to get tired of different places in the source being formatted
differently from each other. I arbitrarily choice Linux style because I like
Linux style. I'm going to mention that indent (the tool that did most of the
work) does not understand C++, so I had to go back in and fix it by hand
in some places (references and template types being the main offenders),
so I may have accidentally not fixed something. If so, my bad. At the very
least, it does still compile and run, so I didn't totally break it.

I'm going to finish with this - I merely wanted a consistent style, which
doesn't necessarily need to be Linux style. If you happen to not like
Linux style for this project, that's fine. Suggest a style, or even write
a new one. Again, I only want consistency and Linux style was an arbitrary
choice.


What's up with the huge loss rate?  Is something broken...?


@nedbat this is the essential part to fix the WDK issue


The date of this commit is incorrect.


I think this should just set the resolved mode (in global state) to ResolvedRetryHooks, update localState.RetryOnErrorVersion (via a callback into the remote state op factory), and return ErrNoOperation. It'll eventually be called again with the updated resolved mode.


After reading mstembera above...I see why.
I will stop test.


but why do we need to support older compilers? Won't this work on gcc 4.8?


Yes but the bug is in `TestResponse`, it's not behaving according to the standard Ruby idioms.


Needless addition


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


Hi Gé.  All these recent changes are great, but can I ask what your approach is?  Are you using some analysis tools or just eye-balling the code?  Are you working to a plan or just randomly picking a file to examine?  How much more should we expect.  Some of the changes are not just fixes but re-factoring, how much of that do you intend to do?  Just curious....


This feels like the wrong place to do reset the timer. I think this should be closer (perhaps in the wrapper op factory?) to the running of hook ops.


Hahah tas noob,pero bájate el git y practica pz chichico


*I* is countable because it is a subset of *Strings*


Thanks! It's still a work in progress.

I'd take a look at your stuff, but I have no idea how to use it so it scares me. Haha


El 17 de septiembre de 2015, 16:36, raulcomer <notifications@github.com>
escribió:

> Este patrón también se conoce como el patrón de publicación-inscripción o
> modelo-patrón. Estos nombres sugieren las ideas básicas del patrón, que
> son: el objeto de datos, que se le puede llamar Sujeto a partir de ahora,
> contiene atributos mediante los cuales cualquier objeto Observador o vista
> se puede suscribir a él pasándole una referencia a sí mismo.
>
> El Sujeto mantiene así una lista de las referencias a sus observadores.
> Los observadores a su vez están obligados a implementar unos métodos
> determinados mediante los cuales el Sujeto es capaz de notificar a sus
> observadores suscritos los cambios que sufre para que todos ellos tengan la
> oportunidad de refrescar el contenido representado.
>
> [image: estructurapatronobservador]
> <https://cloud.githubusercontent.com/assets/14063724/9948656/551ff3d0-5da5-11e5-9e06-22ed16f637d1.png>
>
> De manera que cuando se produce un cambio en el Sujeto, ejecutado, por
> ejemplo, por alguno de los observadores, el objeto de datos puede recorrer
> la lista de observadores avisando a cada uno. Este patrón suele observarse
> en los frameworks de interfaces gráficas orientados a objetos, en los que
> la forma de capturar los eventos es suscribir listeners a los objetos que
> pueden disparar eventos.
>
> El patrón Observer es la clave del patrón de arquitectura Modelo Vista
> Controlador (MVC).1 De hecho el patrón fue implementado por primera vez en
> Smalltalk's MVC basado en un framework de interfaz.2 Este patrón está
> implementado en numerosos librerías y sistemas, incluyendo todos los
> toolkits de GUI.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tectijuana/csharpdd/commit/57a6aab4676d1a6dc743f61bce943c8490010fdc#commitcomment-13300253>
> .
>


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


Right! I opened a pull request for gpu-sonar-simulation driver (https://github.com/Brazilian-Institute-of-Robotics/simulation-gpu_sonar_simulation/pull/7). Can you test it?


Done..


Done


@chan @machikoyasuda <https://github.com/machikoyasuda>

Works in Chrome on my Mac, except I don’t see any restaurants. The following is from Safari which is what I screenshotted before, don’t know if it helps.

[Error] SyntaxError: Unexpected token '('. Expected a ':' following the property name 'onEachFeature'.
	(anonymous function) (old-la-restaurants, line 36)
[Error] XMLHttpRequest cannot load safari-extension://com.evernote.safari.clipper-Q79WDW8YH9/54ee313e/_locales/en_us/messages.json. Cross origin requests are only supported for HTTP.
	a (port.js, line 2)
	_getL10nData (port.js, line 2)
	getMessage (port.js, line 2)
	ContentVeil (ContentVeil.js, line 2)
	(anonymous function)
	execCb (require.js, line 6)
	check (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	init (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	execCb (require.js, line 6)
	check (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	init (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	init (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	b
	load
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	s (require.js, line 6)
	callPlugin (require.js, line 6)
	fetch (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	init (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	init (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	runCallbacks
	callReady
	pageLoaded
[Error] XMLHttpRequest cannot load safari-extension://com.evernote.safari.clipper-Q79WDW8YH9/54ee313e/_locales/en/messages.json. Cross origin requests are only supported for HTTP.
	a (port.js, line 2)
	_getL10nData (port.js, line 2)
	getMessage (port.js, line 2)
	ContentVeil (ContentVeil.js, line 2)
	(anonymous function)
	execCb (require.js, line 6)
	check (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	init (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	execCb (require.js, line 6)
	check (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	init (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	init (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	b
	load
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	s (require.js, line 6)
	callPlugin (require.js, line 6)
	fetch (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	init (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	each (require.js, line 6)
	emit (require.js, line 6)
	check (require.js, line 6)
	enable (require.js, line 6)
	init (require.js, line 6)
	(anonymous function) (require.js, line 6)
	(anonymous function) (require.js, line 6)
	runCallbacks
	callReady
	pageLoaded
Greg

> On Sep 17, 2015, at 1:53 PM, Chandler Sterling <notifications@github.com> wrote:
>
> @machikoyasuda <https://github.com/machikoyasuda> @MtnBiker <https://github.com/MtnBiker> what's interesting is that I get those errors in Chrome, but it works fine in Firefox
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/maptimeLA/old-la-restaurants/commit/e57d376b5ea5df55e97f2b37eebc3ab15a4f2b69#commitcomment-13297612>.
>


arguably sensitive


:smile_cat: :six_pointed_star: Agreed


KP's dyslexic typing :joy:


These changes won't be included in the new web docs, as is.


Add constants for all of these `COLUMNS.indexOf()` calls


Yes, I think we can change this behaviour in a separate patch (may be create an issue so that we can track it?) :yellow_heart:


Given the gold, silver bronze, empty approach. Is the idea that it is the backend driver's discretion as to what composes "Gold"?

Also, I assume "EMPTY" is default. So by default drivers choose how they provision volumes today, maybe this should be DEFUALT instead of EMPTY so be clear that there is always some basic confit per volume. I.e thick vs. thin p.


1.9.0-MinorFixes (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2303 of 2349 tests passed)

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


Having these split out makes it harder to read. It would be cleaner to inline these, then add constants:

```java
private static final int SHARD_UUID_COLUMN = getColumnIndex(SHARD_METADATA, "shard_uuid");


This implies that there exists an *r* for every *k* and *i*.


please refer https://help.github.com/articles/closing-issues-via-commit-messages/
to close the two issues, the right commit message must be "fixed #3 , fixed #5"


hasan


still need to fix segfault on model->close(): deallocation of unallocated pointer


The problem is that when using a `SAMFileHeaderWritable`, the actual internal `SAMFileHeader` is marked as `transient` and is reconstructed when needed.  However, during the reconstruction, the `SequenceDictionary` is mutated here:
https://github.com/bigdatagenomics/adam/blob/master/adam-core/src/main/scala/org/bdgenomics/adam/models/SAMFileHeaderWritable.scala#L52
because of the sort.  That means that when a `SAMFileHeaderWritable` goes through serialization/deserialization, the resulting `SAMFileHeader` is different.

Specifically, the resulting `SAMRecords` refer to their reference sequences by an index into the sequence dictionary:
https://github.com/samtools/htsjdk/blob/master/src/java/htsjdk/samtools/SAMRecord.java#L343

This breaks when I test equality of `SAMRecords` because the indices are now different.

We could simply decide that it's my problem to compare the `SAMRecords` correctly.  But it does seem strange that that the `SAMFileHeaderWritable` gives you a different object pre/post serialization.

cc @fnothaft too in case he missed this.


https://help.github.com/articles/closing-issues-via-commit-messages/
commit message must be "fixed #7"


https://help.github.com/articles/closing-issues-via-commit-messages/
commit message must be "fixed #2"


to try and get the page to compile on cdf I needed a non-commit


SO MUCH SALT


PLS NOTE THAT ALL THE COMMITS IN THE PAST 5 MONTHS ARE BY FORMER CSSU MEMBERS @qasim


I find this very odd. Shouldn't it be an `or` operator instead of an `and`?


Apparently this broke Breath of Fire 3.

-[Unknown]


Da pra criar o Elfo José só uma vez aqui


This is wrong. You are changing the semantics of the method. findGaugeDataByTags fetches data points that have the specified tags. You are changing it to fetch metrics that have tags. The method along with the corresponding one for availability should simply be deleted. We already have the findMetricsWithFilters method for querying for metrics by tags.


As per previous comments, I do not think we need this.


Next time, `git commit --allow-empty` is your friend ;p


As per previous comments, I do not think we need this query any more.


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

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


Para visualizar correctamente el programa se debe anexar un archivo de nombre menu.txt con el siguiente contenido:
Menu:
1.Torta cubana - $45
2.Torta jamon - $15
3.Torta especial - $55
4.Refresco - $10
5.Salir


Why not to move the "getForm().checkFieldsValue()" here?


TIL


not anymore @jwebbed


楼主这篇文章不错，但是为了方便大家看 markdown 方便，建议楼主折个行


4967-SZRH-spm (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: OK (1850 of 1879 tests passed)

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


".IDKYET" WILL NOT BE THE SCRIPT NAME.


ireghaliuerhgaliuer


test


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1342 of 1342 tests passed)

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


Why did you make this change?

The CoreCLR PAL is generally trying to match Windows - the name of the argument on Windows is dwBytes, so it was the right name to use in the PAL as well.


@davidfowl Holy stack overflow, Batman!

![dispose-overflow](https://cloud.githubusercontent.com/assets/54385/9924069/b26710dc-5cb0-11e5-8675-c36ef2cd18bc.png)


No luck. We can try taking the old code.


Yeah but I bet it stack-overflowed very performantly!


Photovoltaics (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


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

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


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


이 부분이 원래 없는 것 같습니다.  아무리 찾아봐도 없네요


Yup got it working with env var! Thanks for the guide.


its ok


This a string, the access modifier is the keyword `private` (no quoted strings)


indeed it is


In retrospect, I'm a bit worried that this changes the api for `add_task`. Old luigi clients will communicate something the luigi server won't like.


Test


이거 바꾸니깐 홈페이지가 안뜨네요..


Photovoltaics (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


Photovoltaics (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Another note, the behaviour here is dependent on markdown parser. In original markdown spec, the parser will copy HTML elements verbatim into the resulting page. So the content between `<div id="aboutpage">` and the closing tag will not be parsed.


inline *EDITING*


So, the docs are now interactive. This is awesome.

_That's one small commit for a Schnee, one giant leap for wistia-doc._


me too


Ah crap. Well tests all passed 😀


Did you really just typo your own name?


helo


@laserson I think a simple fix would be to fix the `SAMFileHeader` that we're using during conversion from `AlignmentRecord => SamRecord`, no? If I'm not understanding this correctly, let me know; I won't have time to look until next week though.


Awesome! :+1:


fsdfsdfs


this is my favourite commit


Here too hehe


102497980-Improve-waste-heat-function-handling-for-CoilCoolingDXMultiSpeed-and-CoilHeatingDXMultiSpeed-#5162 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1336 of 1336 tests passed)

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


Thank you for the quick reply!  I was able to change the form class to
"payola-register-form".
I have the trial period set at 30 days.  I confirmed that the trial period
was set using binding.pry from inside of the controller.

I am still receiving a javascript error message stating: "stripeToken
required for new customer subscription ".

Is there something else that I need to do to make sure that the correct
javascript file is referenced?

For reference, here is my registrations controller:
class Users::RegistrationsController < Devise::RegistrationsController
include Payola::StatusBehavior
before_action :cancel_subscription, only: [:destroy]
#skip_before_filter :ensure_eula

  # GET /resource/sign_up
  def new
    build_resource({})
    set_minimum_password_length
    yield resource if block_given?
    respond_with self.resource
  end

  def create

    build_resource(sign_up_params)
    plan = Plan.find_by!(id: params[:user][:plan_id].to_i)
    resource.role = User.roles[plan.stripe_id] unless resource.admin?
    resource.save
    yield resource if block_given?
    if resource.persisted?
      if resource.active_for_authentication?
        set_flash_message :notice, :signed_up if is_flashing_format?
        sign_up(resource_name, resource)
        subscribe
      else
        set_flash_message :notice,
:"signed_up_but_#{resource.inactive_message}" if is_flashing_format?
        expire_data_after_sign_in!
        subscribe
      end
    else
      clean_up_passwords resource
      render json:
        {error: resource.errors.full_messages.to_sentence},
        status: 400
    end
  end

...
def subscribe
   return if resource.admin?
   params[:plan] = current_user.plan
   subscription = Payola::CreateSubscription.call(params, current_user)
   current_user.save
   render_payola_status(subscription)
end
end

And here is the view:

<% content_for :head do %>
  <%= render 'payola/transactions/stripe_header' %>
<% end %>

  <%= form_for(resource, :as => resource_name, :url =>
registration_path(resource_name), :html => { :role => 'form',
        :class => 'payola-register-form',
        'data-payola-base-path' => payola_path,
        'data-payola-plan-type' => resource.plan.plan_class,
        'data-payola-plan-id' => resource.plan.id}) do |f| %>

Sign up

      <%= f.label 'Subscription plan' %>
      <%= f.collection_select(:plan_id, Plan.all, :id, :name) %>

      <%= f.label :email %>
      <%= f.email_field :email, :autofocus => true, class: 'form-control',
data: { payola: 'email' }  %>

      <%= f.label :password %>
      <%= f.password_field :password, class: 'form-control' %>

      <%= f.label :password_confirmation %>
      <%= f.password_field :password_confirmation, class: 'form-control' %>

    <%= f.submit 'Sign up', :class => 'button right' %>
  <% end %>

  <%= render "devise/shared/links" %>

Thanks for your help!!!

Ty Clark, PE
CEO at Bay Efficiency, LLC

LightBids <http://lightbids.bayefficiency.com/> -- Software for
contractors, architects & designers that generates lighting & energy
savings proposals in less than 5 minutes.

www.BayEfficiency.com <http://www.bayefficiency.com>
ty.clark@bayefficiency.com
415-730-2710 <//1-415-730-2710>

On Wed, Sep 16, 2015 at 7:33 PM, Josh <notifications@github.com> wrote:

> There's a little bit of documentation missing on this. The class on the
> form should be payola-register-form instead of
> payola-onestep-subscription-form to omit credit card info. To validate,
> the plan should either have a price of 0.00 or trial_period_days greater
> than 0.
>
> Once the trial period is over, entering credit card should probably happen
> through a subscriptions update action. That's something I haven't done, but
> you would just need to update the customer on stripe with their card info.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/peterkeen/payola/commit/0b0282fb256eba7247999fbf2b33f2ded567c311#commitcomment-13277652>
> .
>


#5167-ExampleFile-VariableRefrigerantFlow_FluidTCtrl_5Zone.idf-Warnings (rongpengzhang) - i386-Windows-7-VisualStudio-12: OK (1879 of 1880 tests passed)

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


test変更


Thanks for moving the battery levels over so that they wouldn't be cut off on the pebble time!


:smile: :tada:


Crashlytics doesn't provide non-ui based tools. If you point me how to do this, i will switch to crashlytics.


this closes #73


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

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


This change should also be applied to: 7_imagenet_classification.ipynb


I'm a bit surprised that no tests wound up failing.

I'll submit a PR to fix this later tonight. I just wanted to take the opportunity to make fun of your bug as quickly as as possible :trollface:


Oopps...yes indeed typo on my own name. xDDD


Maybe requirements.txt should be fixed with version limitation of queuelib


Can you keep this comment with line 61?


check `cooc_tokens_info` for `nullptr`


1.3 是什么呀！


First thing


The `$TMP/$hostname` seems like a recipe for a problem.  If I read the code right, `$TMP` is generic, but don't we want the `$hostname` directory to be created inside a sub-directory of `$TMP` which is specific to the program being run?  E.g. `$TMP/${PROG}.$$/$hostname`


This seems to require a change to the unittests.  At least Travis-CI seems to think so.  :)


Actually, this is not just a ser/de issue.  `header == SAMFileHeaderWritable(header).header` could return false because it reorders the `SequenceRecords`.


Test comment on commit


I see what you mean.
Can we agree that if the verification of FP (or any verify with count) must validate all the priority to avoid defect being introduce silently?
I think that many of your tests already do this.
Other than that

The selected tests that I have modified where not intended to test code that affect priority. Most of them are testing only the signature and the parameter index for injection. That why I taught the priority was unnecessary for the test.

Any test in particular you feel should be change?

Also, I'm really interested in testing so don't hesitate to mention stuff that iretate you.


>``git commit --amend``
>``git push --force``

Those two commands save my typo commits :yum:


Fool, that's his _real_ name!


I tested the following and got a permission denied:

```
$ shutdown -h now
Failed to talk to shutdownd, proceeding with immediate shutdown: Permission denied
Failed to issue method call: Access denied
```

I am seeing similar behavior to shutdown -r, etc. This was done as root and running it as sudo while logged in as toor.


This fixes the loading screen music. Did you manage to find the in-game music?


- One alternative is always to use `symmetric_cooc_values = false`, because the user may generate an input with both pairs present (``a b`` and ``b a``). But I think you are right and this flag gives a useful option, so let's keep it.
- The name of the flag should be `use_symmetric_cooc_values`. Most if not all flags  in `messages.proto` start with s verb (`compact_batches`, `cache_theta`, `opt_for_avx`, `gather_cooc`, `invoke_regularizers`, `clean_cache`, `reset_scores`).  And by far the most common prefix is `use_`, so I suggest to use it here too.
- Maybe by default `use_symmetric_cooc_values` should be `true`... Then BigARTM does less magic, and if users do want BigARTM to do an extra work then they should enable the flag.


Next thing


Não vimos em aula ainda, mas já adiantando, vc pode usar `assertTrue` apenas. Usa o primeiro amigo.


Cuidado com esta implementação. Acertar não está no escopo dos exercícios, apenas apareceu naquele exercício de refactory.


01-js-intro-and-jquery/Topic 1: JavaScript Intro and jQuery/index.html

Ups! Sorry


Why not something like "Adjacent lists and variables are expanded as Cartesian products."?


Estão faltando os principais testes, do método `atirarFlecha`. Em geral faça 3 testes para os métodos mais complexos.


why add this line here?
I met some similar problem that the title position of button is wrong when open a listview.


What an appropriate commit from SH :+1:


Minor issue: those `err()` calls should be `errx()`. `err()` also prints the output of `strerror(errno)`.


Tugas 2 - Membuat Client


Tugas 1 - Membuat Server


GroundDomainUpdate (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1871 of 1880 tests passed)

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


## Hello


Tugas 1 - Membuat Client


yo


Close closes the returned io.ReadCloser and the snapshot got from the underlying storage.


LGTM


We can leave it in phing if we set the checkreturn=true on the phing target, it should error out.


I suggest that here we'd better use const Size &.
const Size& previousSize = ProtectedNode::getContentSize();


The stage_id field is added on top to allow project creation from form view, because the clickable status bar does not allow choosing the stage before the project is first saved (created). It's a temporary solution until I find out a better one (stage_id pulls the status change, so probably a default stage_id and state would do the trick).


although, it doesn't matter, not sure the phing wrapper brings us anything anyway.


Why not use type `RegExp` here?


metadata


but if you work on arrays in C you really just work with pointers.


102497980-Improve-waste-heat-function-handling-for-CoilCoolingDXMultiSpeed-and-CoilHeatingDXMultiSpeed-#5162 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 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)


rc io.ReadCloser // rc is a reader that provides the data of the snapshot.


Why add this line here?
I also met a similar problem, on Android, button title position will be wrong when open a listview.
thanks.


i don't know if it will change anything but you could try changing that to  *(c+r) and so on


reqsnapc -> chan struct{}


// send ... to reqsnapc to notify ... to send back a snapshot to snapc


a chan to receive the requested snapshot from


Photovoltaics (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


this comment is not useful. why do we keep snapshots?


My Git is Perfectly Set now! :V


what do you mean by real time? latest == real time,right? no?


Can we just talk about what this function does instead of mentioning a lot of unrelated stuff?


For example, if v3demo is not enabled, this method will
If v3demo is enabled, this function method will ...


what do you mean by a metrics?


That's true, but kind of unrelated to what I think the root cause of the bug is.

So, what's going on is that the `SequenceDictionary` that is used when converting reads and creating the file header is generated by running an aggregate across all `AlignmentRecord`s. If you don't sort the `SequenceRecord`s, you get a random order in the `SequenceDictionary`, which was what this changeset was fixing. Now, we get "proper" `@SQ` line ordering on conversion to BAM.* I think the bug is caused by us using an unsorted `SequenceDictionary` when converting `AlignmentRecord`s to `SamRecord`s, and then using a sorted `SequenceDictionary` when actually writing out the `SamFileHeader` via Hadoop-BAM. I haven't read through the code recently, so this is just from memory, but I would expect this to be the root cause of the error you are seeing.

`*` Note that this is arguable. As @ryan-williams pointed out earlier, see #794 for a longer discussion. When I say "proper" here, I simply mean "deterministic".


why do we want to append this into rs.snaps?


what if not?


why rs.snap is not a map?!


循环队列


pq `static` ?


não vejo pq ser `static`


this is too ugly.

newRaftStorage

newRaftStorageV3


ACTION: We need to add filter on status. We have to show count of issue book. it should come from book_issue table isn't it?


this is very inconsistent with what you have done for raftStorage


I basically added heaps of comments to the code where it was needed.
I also changed default AI from HARD to MEDIUM.


You know that affects the clouds rendering shadows so it actually does have a benefit of disabling it by reducing some of the rendering the client has to do, ie. performance increase.


var parentUri = this.uri.clone().segment(-1, ''); 를 var parentUri = this.uri.clone(); 로 바꿔야 합니다.
segment(-1, '')가 있으면 아래에서 parentUri.pathname(Path.dirname(path)) 때문에 path의 2단계 상위로 가게 됩니다.


@MingcongBai What's that `build-locale-archive` thing in RH systems?


Why cannot finish the todo now?


Esse teste não faz sentido. Faria mais sentido se fosse algo tipo:

```Java
assertNotNull(elfo3.getNome());
```


Esse cenário foi bom!


this is super strange... why you use db as a prefix and index as a suffix. this is inconsistent with everything we have done.


Photovoltaics (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Do we really need a welcome page?  Seems a bit pointless to me...


Why tingarri.github.io still shows a 404 ?

On Thu, Sep 17, 2015 at 3:38 AM, rishabhnode <notifications@github.com>
wrote:

> initial gh pages branch
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tingarri/tingari.github.io/commit/131f6bd15a4f8a9f04bfdedcdb203a0536812b40#commitcomment-13274435>
> .
>

--
Co-founder Inoeve Solutions (building the internet of everything)
http://www.inoeve.com
My Android apps @ Garian Technologies
https://play.google.com/store/search?q=gariantechnologies

contact: 91-90340-61588
skype: rishabhsharma1492
twitter: @rishabhgarian
blog: mytingarri.hubpages.com


yoyo


I just realized that some or all of those files still need to include `err.h`, too. Should have tested more.  :P


lol
Install lzma and sm libs
Build and see


instead of wrapping these calls (I think there are a a dozen places in CMSSW), you could use `boost/core/demangle.hpp`


Can we do it now? Also this should be renameTempSnapshot


You're welcome.  Sorry it took so long.
Cheers

---- Jessie Pusateri wrote ----

>Thanks for moving the battery levels over so that they wouldn't be cut off on the pebble time!
>
>—
>Reply to this email directly or view it on GitHub.￼
>


SaveTempSnapshot


((newWidth * newHeight) % 8) != 0 ? 1:0


I think there's another react router bug. The index page cannot be Login,
or it just fails.

On Wed, Sep 16, 2015 at 11:47 PM, Branden Lange <notifications@github.com>
wrote:

> Do we really need a welcome page? Seems a bit pointless to me...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CivBase/testing-parse/commit/765685756cee9009920733f610108994c7fd1ddc#commitcomment-13278648>
> .
>

--
Tyler


usesnapshot should include line 495 -> 498


background. it's going to be white or black


Se vc fizer um teste separado só para garantir os valores iniciais de flechas e experiências, vc pode deixar este teste mais simples, tipo:

```Java
Elfo legolas = new Elfo("legolas");
legolas.atirarFlecha(new Dwarf());
assertEquals(41, legolas.getFlechas());
assertEquals(1, legolas.getExp());
```


Result :
```java
{[x=0,y=0]|g:0|r:0|b:f}   {[x=1,y=0]|g:1|r:1|b:f}   {[x=2,y=0]|g:2|r:2|b:f}
{[x=0,y=1]|g:1|r:1|b:f}   {[x=1,y=1]|g:1|r:1|b:f}   {[x=2,y=1]|g:2|r:2|b:f}
{[x=0,y=2]|g:2|r:2|b:f}   {[x=1,y=2]|g:2|r:2|b:f}   {[x=2,y=2]|g:2|r:2|b:f}
```


Cuidar nomes de variáveis que podem ficar difíceis de entender, exemplo: `n2Flechas`, `n2Exp`


is this really needed ?
shouldn't it be handled by the call in `customiseHLTforCMSSW` anyway ?


kkk


return the error


i mean return the error to http.Error. you should not mask it.


can you wrap this in a one-line function called `customiseFor11303(process)` , and call it inside the "CMSSW_7_6" if-block ?


this should not be in this commit.


right. i should just use latest.


a prometheus metrics.


It should be. Will change.


"Supported by the Australian Government, Geoscience Australian and the World Bank-GFDRR"


$input_raw uncommented!!!!


hi @lucernae . I have made a comment for further change to the text. T


just curious, which flashls release is it ?


A reminder to myself to clean this up!


Nice!  The second argument to `forEach` is the context, so you can also do:

```js
agents.forEach(function(agent){
   this.attendingTo[agent] = null
}, this)
```


I learned that from a student the other day :)


develop (nealkruis) - i386-Windows-7-VisualStudio-12: OK (1344 of 1344 tests passed)

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


I think i will use the same way in raftStorage.. the parameter list in rafthttp is out of control. I would clean it up later.


GHX_Vert_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1872 of 1872 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)


Don't do that.  It takes away users' ability to add options to the checker.


Don't do that, either.  It's for broken checkers.


Great.  Now let's keep these variables consistent and match the formatting... A bit of nitpicking here... I hope this is OK....

e.g.

```
SET(LINUX_DESKTOP_EXEC                  "env QT_X11_NO_NATIVE_MENUBAR=1 lmms")
SET(LINUX_DESKTOP_ICON                  "lmms")    # (some comment explaining why this is just "lmms")
SET(LINUX_MENU_ICON                     "/usr/share/pixmaps/lmms.xpm")
```


Looks great!

Why not spray-json `1.3.3`? Is that not the latest version?


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

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


아 그렇군요. 변경해야겠네요.


Good thing you asked about adding arguments.  Oh wait, never mind.


BaseURL  -> CUSTOM


I'm still not entirely sure what `INSTALL_FILE_ICON_LINE` is, but you can probably call this

`DEBIAN_INSTALL_ICON` or something since this variable is of no use to other Linux distros, right?


GHX_Vert_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


ACTION: We have to find out book issued more than 2 times so remove equals.


It will change some API and I think it may need several rounds, and I want to separate them to make it move fast. Let me know if you prefer to finish the TODO here.


make sense. will chagne.


I see your comments below. will do.


didn't read the details but i like the table! nice work :)  I'll merge and dig through later.


现在leetcode上面能直接支持c++ 0x了吗？


@heiko-braun Thank you. Fixed


if () {
     return
}

即使只有一行，最好也别写到一起，通常这个是不符合coding style的


А можно я сделаю на Apache Tiles?


ACTION: Select title name too.


hm.. let me try


Dumb


LGTM - thanks


I'm not seeing anything but the links to the CartoDB versions, no map. Am I right in that there are three versions—two CartoDB and a MapBox one? Or?


LGTM


Not really part of the initial requirements but ok.


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


na hora em que fui compilar deu um erro dizendo que uma variavel não static não podia ser referenciada em um meio static, aconteceu o mesmo com o ''this'' quando tentei utiliza-lo, ia te perguntar em aula sobre isso.


@huangp so what exactly is this a squashed version of?


Oh, because I didn't know that was a thing. I'll do that.


I can add the deprecation notice.

`$compositeFields` is passed by reference to return a result. You don't actually pass a value into it from the caller. I'll update the documentation here.

The principle goal of database_fields() method is that it returns the list of physical fields as a primary return type (used in most cases), but the compositeFields (which are logical only) make sense to treated as a secondary return type, which is where I felt a by-ref parameter would be most appropriate.

Your suggestion to simply cache it on first access is exactly what this method is doing. :) It is the authority from which other methods can rely on, rather than multiple access-points to the underlying schema. As soon as we have more than one cache / authority then we can end up with diverging behaviours, which is what made the api inconsistent to begin with (all the different methods for getting 'the' list of database fields).


segment() 함수를 사용하면 path가 "/"로 끝날 때 parent를 제대로 못 찾는 문제가 있습니다.


thnx awesome


%if 0%{?rhel}
Requires: redhat-access-insights
%endif


Confirming that all tests pass on my machine!


You don't need to declare parameters passed by reference. :)


thanks!


this is out of date now. I will rebase again since you haven't started reviewing this one yet


from the u2f mode the u2f-host utility can be used (https://developers.yubico.com/libu2f-host/Mode_switch_YubiKey_NEO.html) and from ccid the ykneomgr utility can be used (ykneomgr -M 6 for example)


pass使ったことない


最初に `output_path` を定義しているので `return output_path` は最後に１つだけでいいよ


Fixed. Thanks :+1:


i think u added the authtoken.py by mistake or it got added because it was removed from .gitignore


Thank you @Jean85


Okay.  Let me know.

On 17 September 2015 at 15:31, Patrick Huang <notifications@github.com>
wrote:

> this is out of date now. I will rebase again since you haven't started
> reviewing this one yet
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zanata/zanata-server/commit/732de27cdf0b5a40c2a40dee3bd8d59e4e36ced4#commitcomment-13278922>
> .
>

--
Sean Flanigan

Principal Software Engineer
Globalisation Tools Engineering
Red Hat


![1 imagine - 09 14 15](https://cloud.githubusercontent.com/assets/14024852/9925761/0bfffcb8-5cc4-11e5-8ceb-6f5a6fa249dd.png)


Do we also need this?

TARGET_INCLUDE_DIRECTORIES(qlibcext PUBLIC ${qlibc_SOURCE_DIR}/include/qlibc)


:facepalm:


When multiple headers were at the same indent level, projectNames was grabbing any header that was any indent level less than itself, without allowing for the possibility of multiple headers at the same level.
![capture](https://cloud.githubusercontent.com/assets/4820235/9925732/7405122a-5cd4-11e5-928f-95b31e4eb517.PNG)

This commit causes projectNames to ignore any headers that are of the same indent level as a header that has already been used as shown below.
![capture2](https://cloud.githubusercontent.com/assets/4820235/9925803/1a689b78-5cd5-11e5-99d8-647dd08f5171.PNG)


GHX_Vert_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (802 of 802 tests passed)

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


LOL!  Sorry about that.  I am using a virtual machine from jchris and so I accidentally committed under his name, but this was from me.


네 테스트해 봤더니 그렇네요. 그런데 "a/b/c/" 같은 uri의 parent가 "a/b/c"가 아니라 "a/b"가 되는게 맞나요?


For anyone who cares, I apologize that this commit was so massive, took a lot of time, and was not split up into a bunch of smaller commits.


Hi Sebastian,

I wrote a blog post about this here:

http://www.alittlemadness.com/2015/03/18/android-junit-xml-reports-with-gradle/

which hopefully explains the details.


This should be `<code data-x="attr-dim-width">width</code>`, right?

Or at least it seems it should not be `img-dim-width`, because for that Wattsi is emitting a (non-fatal) error:

> Error: missing `<dfn>` for topic "img-dim-width" explicitly from `<code>` element containing "width"; previous heading contents are "4.8.4 The img element"


morph.surface in ["。","、"]とかのほうがスッキリしてて好き


普通，listのinは重い．集合的に扱うならsetの方がいい
2個ぐらいなら元のままでも別にいいかなと個人的には思う


ここ以前に判定条件を書けるはず．
出力を作ってからそれをprintするか判定しているので，データサイズがでかくなると遅くなる．


Thanks, @pavelfatin :cake:


I don't think so... I mean, I haven't tested it, but I believe that because we have `TARGET_LINK_LIBRARIES(qlibcext qlibc)`, we get the same effect by transitivity.


That's ok my bug I'll fix it after I find out why the tests are all happy


Nice, but how can I download the whole project? I don't see the option...


Soon™ ;)


We could monitor the `stdout` stream for the words `downloading` and `remaining`?


Thanks, will give it a try without these when I get around to it.


`sahen`はここ以降変更されないみたいなので，この段階で `sahen == ""` なら以降の処理をスキップできるのでは？


well... can snapshot use another http client? this makes pipeline messy.


すこし見当ちがいだった．
ただ直前のif文がすごく長いので，まず`sahen==""`じゃないかを確認してから，処理を徐々に進めていくほうが個人的には好き（if文を分割する）


Ok, I was wondering because the patch has following but not only for
qlibcext. so thought it might be missed. Are those there for different
reason?

+TARGET_INCLUDE_DIRECTORIES(qlibc-static PUBLIC ${qlibc_SOURCE_DIR}
/include/qlibc) +TARGET_INCLUDE_DIRECTORIES(qlibc PUBLIC ${qlibc_SOURCE_DIR}
/include/qlibc) +TARGET_INCLUDE_DIRECTORIES(qlibcext-static PUBLIC
${qlibc_SOURCE_DIR}/include/qlibc)

On Wed, Sep 16, 2015 at 10:50 PM, Alexandre Lucchesi <
notifications@github.com> wrote:

> I don't think so... I mean, I haven't tested it, but I believe that
> because we have TARGET_LINK_LIBRARIES(qlibcext qlibc), we get the same
> effect by transitivity.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alexandrelucchesi/qlibc/commit/c2afb3b00bce796adb9395088d9b33d321cc33b3#commitcomment-13279057>
> .
>


↑2つのifは1つのifで書ける


unrebranded!


delete it


ifがtrueならcontinueになるので，elseで分けなくても良い（ネストを一つ減らせる）


@Alefas If this is a really hot path, you might even create the array buffer lazily:

```scala
if (res == null) res = new ArrayBuffer[PsiClass]
res += clz
```

We used this trick in a few places in the compiler/library, e.g.: https://github.com/scala/scala/blob/v2.11.7/src/reflect/scala/reflect/internal/util/Collections.scala#L125-L139


Опиши, пожалуйста, зачем сверять пароль и его хеш? Это ведь не имеет смысла. Смысл то в том и был, что хранить хеш и соль. И проверять именно хеш. Поэтому считаю, что это крайне не нужно.


There's a couple of things that bother me here:
1)
killing the refs might not work for all cases, in particular interlaced material would suffer
2)
it does not solve the issue directly. what we really want to know is:
-> does renderer need an additional frame for a successful render?
3)
use of global g_application. I am in the process of kill those for new VideoPlayer

Does WinRenderer know the status whether a render was successful and we have a frame on screen?


This shouldn't  be here.


Thank you fox explanation. Then it should be all ok to keep this commit.


particle.values()


Why do this?


Results:

```java
//case 1 : { 0, 0 } to { 3, 2 }, no blocked
{[x=0,y=0]|g:0|r:0|b:f}		{[x=1,y=0]|g:1|r:1|b:f}		{[x=2,y=0]|g:2|r:2|b:f}		{[x=3,y=0]|g:3|r:3|b:f}+++
{[x=0,y=1]|g:1|r:1|b:f}		{[x=1,y=1]|g:1|r:1|b:f}		{[x=2,y=1]|g:2|r:2|b:f}		{[x=3,y=1]|g:3|r:3|b:f}+++
{[x=0,y=2]|g:2|r:2|b:f}		{[x=1,y=2]|g:2|r:2|b:f}		{[x=2,y=2]|g:2|r:2|b:f}		{[x=3,y=2]|g:3|r:3|b:f}+++
{[x=0,y=3]|g:10000|r:3|b:f}	{[x=1,y=3]|g:10000|r:3|b:f}	{[x=2,y=3]|g:10000|r:3|b:f}	{[x=3,y=3]|g:10000|r:3|b:f}+++
```


Remove duplicated comment?


/// To be removed ASAP ?


Oh my goodness!? Is this for real? I'm so excited! Wow... thank you so much. I will email you at the address given.


// TODO: remove?


```java
{[x=0,y=0]|g:10000|r:10000|b:t}		{[x=1,y=0]|g:10000|r:10000|b:t}		{[x=2,y=0]|g:1|r:1|b:f}			{[x=3,y=0]|g:0|r:0|b:f}
{[x=0,y=1]|g:10000|r:10000|b:t}		{[x=1,y=1]|g:2|r:2|b:f}			{[x=2,y=1]|g:10000|r:10000|b:t}		{[x=3,y=1]|g:1|r:1|b:f}
{[x=0,y=2]|g:10000|r:10000|b:t}		{[x=1,y=2]|g:3|r:3|b:f}			{[x=2,y=2]|g:10000|r:10000|b:t}		{[x=3,y=2]|g:2|r:2|b:f}
{[x=0,y=3]|g:10000|r:10000|b:t}		{[x=1,y=3]|g:4|r:4|b:f}			{[x=2,y=3]|g:10000|r:10000|b:t}		{[x=3,y=3]|g:3|r:3|b:f}
{[x=0,y=4]|g:10000|r:10000|b:t}		{[x=1,y=4]|g:5|r:5|b:f}			{[x=2,y=4]|g:10000|r:10000|b:t}		{[x=3,y=4]|g:4|r:4|b:f}
{[x=0,y=5]|g:6|r:6|b:f}			{[x=1,y=5]|g:10000|r:6|b:f}		{[x=2,y=5]|g:5|r:5|b:f}			{[x=3,y=5]|g:5|r:5|b:f}
```


ここのサ変接続のチェックは，元のChunkのIDが一致するかどうかで見たほうが良さそう


Avoid duplication, introduce a method to initialize the attributes?


`TSLocationManager` provides a `locationManager` property you access instead of creating a new instance of `CLLocationManager`

[bgGeo.locationManager](https://github.com/transistorsoft/cordova-background-geolocation-lt/blob/master/src/ios/TSLocationManager.framework/Headers/TSLocationManager.h#L10)


assert(f.get() != 0);


Introduce a macro to hide the pragma? Define it in debug.hh or something similar?


ditto


change


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


Запиши это себе как пункт 16 а пока сделай по-простому.


That's the objective, but I can add a comment. I was actually wondering: how many releases should be done before breaking backwards compatibility once a method has been flagged as deprecated?


This is implied by the `ROBOPTIM_CORE_DEPRECATED` flag + the doc associated with it, so I don't think a comment is required right here.


The
```javascript
!focusableElementList.length0
```
 are always returning true.


Maybe it should be:
%1 RLM(%2)
RLM%1 RLM(%2)
RLM%1 RLM(%2)RLM

What do you think?


I was thinking about it. I'll add that with the rest.


Perhaps we ought to split up faces-config.xml: http://www.mkyong.com/jsf2/how-to-split-faces-config-xml-into-multiple-files/


Using material-calendarview library
Library Ref: https://github.com/prolificinteractive/material-calendarview


What these lines do is to make the header files in the `${qlibc_SOURCE_DIR}/include/qlibc` directory available to each library and to any other CMakeLists.txt that references the library (hence the `PUBLIC`). Because `qlibcext` is linked against `qlibc`, I believe that if we have in another CMakeLists.txt something as `target_link_libraries(myapp qlibcext)`, those header files would also be available to `myapp` (because they were made `PUBLIC` by `qlibc`, which is a dependency of `qlibcext`). Anyway, adding `TARGET_INCLUDE_DIRECTORIES(qlibcext PUBLIC ${qlibc_SOURCE_DIR}/include/qlibc)` does no harm. :)


We considered the existing message module, but it had limitations. Hence we came up with new feature of secure messaging.


```
key_list, value_list = zip(*particle.iteritems())
print " ".join(map(str, key_list)),
print "\t",
print " ".join(map(str, value_list))
```
こんなふうに書けなくもない．
mapを使うかはkeyとvalueのデータ型に依るけれど


Something like `ROBOPTIM_ALLOW_DEPRECATION`? With the push/pop behavior, I don't know what's the cleanest way. Here this was just to please GCC while building tests involving deprecated features before their removal.


For tests, don't forget assertTrue(EXPECTED, ACTUAL) order

PS. I've commented on this commit like I would on a Gerrit commit - I hope this will not end up in deluge of messages! (Apologies if it does). These are mostly just suggestions, though test order is backwards for jUnit.


OptionalInt may be more optimal here.


Wouldn't it be more conventional for this to throw a parsing exception on error and return plain SaltVersion? With Optional, I think it's not much better than returning null on failure. If there is a true optional SaltVersion that needs to be handled, then that should be handled external to this class.


Optional.of() - this can't be nullable here


I find this construct confusing. It's not easy to parse at first glance. There is really 3 cases here.

    if (this.getReleaseCandiate().isPresent()) {
        if (other.getReleaseCandidate().isPresent()) {
            return Integer.compare...;
        }

        return -1;
    }
    return other.getReleaseCandidate.isPresent() ? 1 : 0;

or something similar.


GHX_Vert_IDD (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1338 of 1338 tests passed)

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


That's nice!


uri 기준으로는 뭐가 맞는지 잘 모르겠습니다. 그런데 createDirectory에서 처리해야 하는 path는 파일시스템 path이므로 "a/b/c/"의 parent가 "a/b"가 되는게 맞을 것 같습니다.


Pacman just looks a bit tired. That's fine.


Muchas gracias divadr, va perfecto. Se echaba de menos :)


@Divran is there still this problem that triggering the same table on the same output twice only yields one TriggerInput call on attached devices?


GroundDomainUpdate (mmatts) - i386-Windows-7-VisualStudio-12: OK (1871 of 1880 tests passed)

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


There is a caveat with the build part of the project (which I believe either is not documented well or not documented at all).

The project has a number of lightweight command-line wrappers that together deliver consistent & reliable CLI experience (Command-Line Interface such as cmd.exe or conemu & derivatives - I find cmder.net to be quite handy & flexible) while building, developing & maintaining the code base.

These wrappers as a whole are called Envy and to use them (well, each & every buildable thing in the project is tailored to rely on its presence) - you execute envy.cmd in the root folder of the code base.

After envy.cmd is executed you might notice new environment variables in your console window have been set, such as NUGETROOT, EXTERNAL, REPOROOT and a few more. This lets you use pre-defined aliases and launch solution/project files in Visual Studio from that command-line window. And the great part is that Visual Studio is happy to use variables in the project files as long as they are defined and have meaningful values.

Each time you execute envy.cmd - the script tries to be as helpful as possible and outputs all the aliases that it defines for your convenience. For example,
* sln - lets you launch the solution file in Visual Studio from any location (in that console window)
* build - lets you build the entire project and restore all the dependencies from Nuget
* runtests - lets you build the entire code base and run all the project's tests right there in the console to validate your latest changes
* jwtdecode - lets you decode any JWT token from the clipboard (comes very handy when you intercept tokens in the browser using DevTools)
* deploy - lets you deploy the latest plugin code to the Fiddler's directory, launch Fiddler in the viewer mode with some sample SAZ file

Replacing these Envy variables in CSPROJ files has an immediate benefit (ability to launch SLN from Windows Explorer) but the original idea was to stay away from the use of relative paths as much as possible.

I encourage you to give that workflow a try and spend some time to get used to it, and all the little neat & convenient things offered by Envy in return should be able to convince you to stick with it.


thanks mr Ray - did you consider running this through ARC conversion?
Edit > Convert  > To Objective-C ARC?
It's pretty straight forward.


Search process starts immidiately after the start


박병훈 조인 추가


This seems to have made the wire support test cases more flaky than before (https://travis-ci.org/cucumber/cucumber-ruby/builds/80127629, https://travis-ci.org/cucumber/cucumber-ruby/builds/80609931), this build is green (https://travis-ci.org/cucumber/cucumber-ruby/builds/79969184) but only because I restarted the Ruby-2.2 job once and the Ruby-1.9.3 job twice.


react-swipe.js

componentDidUpdate: function (prevProps) {
      if(prevProps.children.length > nextProps.children.length) {
        this.swipe.setup();
      }
      ...


It could be improved.
I've left some gaps, marked with the TODO:
I'm wondering if we might want to list multiple components instead of just one.

One of the features here, is when it builds the _physicalItemNames, if it finds two or more items with the exact same name (which may be due to mods), it will add a number to the subsequent name, to provide a unique display name to identify it to the user on screen, so they can pick the exact component they want.


Oops, thanks. Fixed.


подправил


Looks like you've encountered some issues with that snippet while using the inspector on your end. If that's the case - can you also provide an SAZ file that captures this issue, so it can be added to testSamples bundle and executed against the inspector to ensure reliability & stability?


Sorry, I totally blew that, and I have worked that out now.

I’m rather new to GIT, and I suspect I may have bound to the wrong repo when I commited it. Certainly I didn’t intend to push that to you, I had only meant to make that local to my fork.

Apologies if I have pushed anything else to you. I have deleted the copy of your repo now, and only have my fork.

From: ptsurbeleu [mailto:notifications@github.com]
Sent: Thursday, 17 September 2015 4:26 PM
To: dajomo/OInspector <OInspector@noreply.github.com>
Cc: dajomo <dave_moyle@hotmail.com>
Subject: Re: [OInspector] Convert project to use visual studio compatible hint paths (2a2cd6d)

There is a caveat with the build part of the project (which I believe either is not documented well or not documented at all).

The project has a number of lightweight command-line wrappers that together deliver consistent & reliable CLI experience (Command-Line Interface such as cmd.exe or conemu & derivatives - I find cmder.net to be quite handy & flexible) while building, developing & maintaining the code base.

These wrappers as a whole are called Envy and to use them (well, each & every buildable thing in the project is tailored to rely on its presence) - you execute envy.cmd in the root folder of the code base.

After envy.cmd is executed you might notice new environment variables in your console window have been set, such as NUGETROOT, EXTERNAL, REPOROOT and a few more. This lets you use pre-defined aliases and launch solution/project files in Visual Studio from that command-line window. And the great part is that Visual Studio is happy to use variables in the project files as long as they are defined and have meaningful values.

Each time you execute envy.cmd - the script tries to be as helpful as possible and outputs all the aliases that it defines for your convenience. For example,

*	sln - lets you launch the solution file in Visual Studio from any location (in that console window)
*	build - lets you build the entire project and restore all the dependencies from Nuget
*	runtests - lets you build the entire code base and run all the project's tests right there in the console to validate your latest changes
*	jwtdecode - lets you decode any JWT token from the clipboard (comes very handy when you intercept tokens in the browser using DevTools)
*	deploy - lets you deploy the latest plugin code to the Fiddler's directory, launch Fiddler in the viewer mode with some sample SAZ file

Replacing these Envy variables in CSPROJ files has an immediate benefit (ability to launch SLN from Windows Explorer) but the original idea was to stay away from the use of relative paths as much as possible.

I encourage you to give that workflow a try and spend some time to get used to it, and all the little neat & convenient things offered by Envy in return should be able to convince you to stick with it.

—
Reply to this email directly or view it on GitHub <https://github.com/dajomo/OInspector/commit/2a2cd6d23f40f5e7832269f258e7a4774cec38c8#commitcomment-13279351> .


AFAIR the "org.zeroturnaround" dependencies aren't available publicly, so the build will fail.


In this case maybe we should rename it to getParent() like for columns.


Why do this?


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


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@juliabackports


Does `SMTP_NOTLS=false` mean SSL/TLS or STARTTLS?


Hi @maxbrunsfeld when will v1.0.12 be released i'm so amped


Np :+1: That's okay - I'm new to Git and all these things as well :-)

Let me know if I can be of any assistance with that - I will be more than happy to help with any questions.


Fix #18 - This was also done at the same time.


I am ok with this patch since it aligns with how the OSGi does it I guess. However, maybe we should add these version numbers to the EE packages instead?

Could you also add simple test case because OSGi enRoute seems to resolve fine and all the current tests pass.


But go ahead and merge this


pokemon!!!


In previous versions DO_OVERLAY_MOUNTS was (0|1), the code had:
```bash
[ "$DO_OVERLAY_MOUNTS" -eq 1 ];
```


Don't let @mkfifo see this.


Hello @rvalyi
Indeed, you are right. This is fixed in https://github.com/odoo/odoo/commit/4449984c757d506f6875105c2a3a0ddca7bc7490


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1344 of 1344 tests passed)

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


I like api.py so much, it's super convenient for building schema. Hope it'll come back ;)


<?
	//koneksi ke MySQL
	mysql_connect("localhost","root","123");
	//ambil database
	mysql_select_db("db_eks1");
?>


Zkousel jsem, nefungovalo mi :|


asi urcite :) nutno pridat testy


diky!


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


uz me napadlo, jak to asi pujde. hodne dlouho jsem zapasil, ze nema metody, ktere potrebuju, a pak jsem zjistil, ze vsechny property jsou public. Mozna by bylo i rychlejsi udelat si vlastni minimal iterator.


I was unable to reproduce it, but it's still happening. This is the call stack and the object status when the exception is thrown:

![image](https://cloud.githubusercontent.com/assets/1260651/9926663/fb7d6106-5d17-11e5-983e-1b8afc1fc30f.png)

If you need anything, just ask. Good job so far by the way.


Fan still loud.


I sort of expect this to be picked up and posted by some twitter bot that looks for private keys submitted to github to shame people. :P


Looks good.


Both "licence" and "license" are correct spellings, but everywhere else in the website uses "license", so I changed it here for consistency.


Changed case to be consistent with the other list items.


That is odd. Can you provide me with a bit more information about your setup?
1. How do you invoke the script?
2. Do you have a .packages file and a packages directory in your project?
3. Anything else that might not be standard in your environment.

Thanks,
/gustav


I ammended a commit. Got a conflict, fixed that and than had to pull according to the message I got :-D

But think it all went ok, the right file was committed.


oh yeah, and since a long time already ! :P


SF


oh boy, I will have work to do.


Checkmarx scan results 17 Sep 2015 09:51 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

14398c1fc35655a03bab11ce5c8b427e0b4040da	@cxehud


https://developer.apple.com/library/ios/documentation/Miscellaneous/Conceptual/MetalProgrammingGuide/MetalFeatureSetTables/MetalFeatureSetTables.html

It seems like sampler comparion function is A9 only on iOS. I guess on Pre-A9 it would require shader modification to change sampler function.


Mobile version should be cool :+1:


@alysonla Thank you for :star2:!! :grinning:


这里不用改。


Why do this?


This is not ideal from a SEO perspective, as SS uses trailing slashes for all URLs.
This module now produces "To" URLs that do not have trailing slash.
Can we reconsider this, To urls needs to be able to have a trailing slash.
http://googlewebmastercentral.blogspot.fr/2010/04/to-slash-or-not-to-slash.html


#5167-ExampleFile-VariableRefrigerantFlow_FluidTCtrl_5Zone.idf-Warnings (rongpengzhang) - Win64-Windows-7-VisualStudio-12: OK (1879 of 1880 tests passed)

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


to move it strategy's method we would need to pass the httpClient and TT as well - this doesn't suite very well to the interface


Nice!


先简单的学习一下吧


add aeon.nox.5 and make it the default skin


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1851 of 1873 tests passed)

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


Reason: http://forum.unity3d.com/threads/unity-and-git-missing-sprites-and-scripts.224061/


What's this?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.597](http://teamcity.monogame.net/viewLog.html?buildId=10225&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 539, ignored: 6 Build time: 00:19:29


thanks yes is full with src now


Good point, I will add the test case before merging.

--
Neil Bartlett
Sent from a phone

On Thursday, 17 September 2015 at 07:39, Peter Kriens wrote:

> But go ahead and merge this
>
> —
> Reply to this email directly or view it on GitHub (https://github.com/njbartlett/bnd/commit/5f3de920106019dfaaf39d0e4db5f83cd0cfacd9#commitcomment-13279456).
>
>
>
>
>


Interesting, I hadn't thought about the public shaming aspect. Well, we'll
see.

Anyway, I'll squash it all away soon.


Should be removed also from flow documentation


The removal of this note and resolution to keep things as-is for this (that, keep "`Object`" and don’t change it to "`object`")  is per discussion with @heycam http://krijnhoetmer.nl/irc-logs/whatwg/20150917#l-157


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


1: D100 Missing docstring in public module
22:8: E121 continuation line under-indented for hanging indent


Why can't I see this?

also: I have no idea what is going on here.


Checking if the field's ID starts with `"sort"` is perfectly valid in this case (because `getSearchFields` can only add them if they really start with `sort`), but we also have `SearchField.setData(JSONObject)` to store information about search fields that we need for building the search parameters.


no newline at eof. I know how you get about those.


uh that was a thing?


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (803 of 803 tests passed)

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


Dear Sir,
 what should be problem,how can i fix this.


Please remove period. Original msg doesn't have it.


we might aswell rename these "Backports changes from 6.x" to "Broke things that worked just fine till yesterday"


Fixed in 77b3793, thanks. How did you catch them? I don't get warning with PEDANTIC=ON, i.e. -Wall -Wextra -Wno-long-long -Wformat-security -Wno-strict-aliasing.


Looks good.  Will test this tomorrow.


@laruence Is this the proper way to do this compared to 116cc56af9baa9ce7b48538da365fc7b14721035

The zend_symtable_update etc.. is just to match how msgpack_unserialize_map_item works

thanks!


Fixed in 77b3793, thanks.


http-proxy는 reverse proxy 서버에서는 안쓰지만 app server에서 nodejs app으로 request를 routing 할 때 사용하고 있습니다. app server도 http-master를 사용하도록 수정하던지 아니면 원래대로 package.json에 추가를 해야 할 것 같습니다.


I'm not completely certain whether this is the correct way to escape the search field. the db-escape sanitizes the input and if I'm not mistake, it will also add quotes around the search string. We should have a look at how this query looks like and append the % with the search variable as the escape()'s only argument and omit the outer quotes.


if locked, checklock returns null.


You can actually write match/try combos like this more nicely as
```
match f () with
| exception Error.Error (_, s) ->
  if not ... then
    ...
| _ ->
  Error.error at ("expected " ^ err)
```


thank goodness for this revert, I thought i seriously messed something up.


I didn't even notice!


I reviewed the CMake documentation and made some tests here just to make sure and, as I guessed, it's correct the way it is. Anyway, because `qconfig.h` has a `#include "../containers/qlisttbl.h"`, I decided to change `TARGET_LINK_LIBRARIES(qlibcext qlibc)` to `TARGET_LINK_LIBRARIES(qlibcext PUBLIC qlibc)` (it seems that the default is actually `PUBLIC`, but I prefer when things are explicit). To summarize, it's not necessary to add `TARGET_INCLUDE_DIRECTORIES(qlibcext PUBLIC ${qlibc_SOURCE_DIR}/include/qlibc)`.


What does that thing do? Is it neccessary? Does it always what we want? Is it standard javascript?! If yes, wtf?! If no, isn't this worth a comment on what this does and why this does what you want?


Yes, I agree with you 100%.  The `SAMFileHeaderWritable` issue is simply how I stumbled on this.  I submitted a patch that enables you to specify a `SAMFileHeader` rather than compute it by aggregating.  Not sure it works yet bc of a Hadoop-BAM issue, but would be good to check it out.  Especially bc I haven't touched the BAM part of the ADAM code very much.  Let's move any more discussion there.


Nit: one line


Oops. We should probably use normalized addresses everywhere.


eedfda


vfdas


I don't see complete object.js file in the repository.


LGTM


```python
if sync:
    while OCI_is_available(api, oci['id']):  # jakoś ta nazwa mi bardziej leży
         time.sleep(1)  # skoro całość trwa przeważnie kilkaset ms, to może sleep(5) jest za długi?
```

BTW, na założenie OCI jest jakiś timeout, fajnie by go było pewnie też honorować przy usunięciu.


Nit: This local shadowing of `memory` is a bit nasty for my taste.


I guess it's clang (here) vs. gcc (for you)

If you are ok with this PR (and if it succeeds this time) in the future
we will get clang warnings from travis:
https://github.com/qgis/QGIS/pull/2317

On 09/17/2015 09:10 AM, Radim Blazek wrote:
>
> Fixed in 77b3793
> <https://github.com/qgis/QGIS/commit/77b3793640d48183ce21a8a6c07b25ff18f25d3f>,
> thanks. How did you catch them? I don't get warning with PEDANTIC=ON,
> i.e. -Wall -Wextra -Wno-long-long -Wformat-security -Wno-strict-aliasing.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qgis/QGIS/commit/60cce74913c5f6a7c814b68a04d36cbf76b20e1e#commitcomment-13279805>.
>


Nit: same here, I'f prefer to not shadow like that.


:+1: good catch :smile:


same here


I'd call this plain `create`, remove the other, and inline the `ref` in the right places.


Yes, this is standard JavaScript. It performs a double negation, which at first glance does not make sense at all, but its "collateral" effect is that it turns any value into a proper boolean. Google keywords are "truthy", "falsy" and "type coercion". Also, there is a very good video by Gary Bernhard called "WAT".


I fucking swear I couldn't get that switch to work correctly... three years ago.