What's the story here?


Don't forget the 500me delay, movement happens in those 500ms. I would bet value is unchanged.


wow, this is quite broad. is it required?


Yeah, that happens if I use auto-completion in Netbeans and accidentally type "Option." instead of "Options."


@rraptorr, I was testings JSJaC with the latest release of eJabberd, which now includes Web Socket support [RFC-6455](https://www.process-one.net/en/ejabberd/protocols/), and noticed that the connection is lost on page change / reload unlike bosh which can resume the session.  I noticed the comment "Resuming connections is not supported by WebSocket.”  Is there a technical limitation, or is this functionality possible?  Any background information would be greatly appreciated.


why this change?


submitted at https://github.com/pantsbuild/pants/commit/f9fce86eb8c98e50a5808d01b6c7b3e392bb304c


wow, > * { .. } is quite broad. is it required?


@alexstoick this has been merged into the v1.0 branch. If this works and there are no further issues I'll likely make a 1.0.2 release next week.


Yeah the dash means something very different than what intuition would lead you to believe, especially when used for bounds in require. If you have suggestions to clarify the docs, please have a go.


try changing this to application-loading.hbs


Not thread-safe, just a (hopefully) temporary implementation


Сделайте плиз такой порядок кнопок: "Правка"/"Сброс"/"Глобальные"/"Файл", это наиболее логчино. Первой должна оставаться "Правка" как самая частоиспользуемая, потом "Сброс" так как это тоже локальная кнопка. А вот после них уже должны идти глобальные (а "Файл" ведь тоже к ним относится).

Алсо, текст "Скачать файл с настройками" лучше заменить на "Сохранить файл с настройками", потому что "Скачать" подразумевает, что откуда-то из интернета, и могут возникать ненужные вопросы..


شركة تسويق الكترونى
شركة تسويق الكترونى
تنظيف بالرياض
شركة عزل خزانات
شركة كشف تسربات بالرياض
مكافحة حشرات بالرياض
شركة تنظيف بالرياض
تنظيف واجهات حجر


It looks like you're trying to make your code look like the code in the test module.  You don't need to have the try and except statements in your code - that's just something that the test module is doing to test your code.  Take out the 'try', 'assert', and 'except' lines and just leave what's currently inside your 'try' section.

Rename your functions in the form 'exhibit_x_error' where x is the error you're trying to produce (see the lines after the 'try' keywords in the test file).

Also watch your indentation.  Everything in a function should be indented four spaces.


Aw thanks!


Wrong comment.


Should we just generalize this to:
```
if (!step) {
   return num;
}
```
That way it will cover for undefined / etc as well?


Oh, you're right


Sounds very legimite. But again, Eagle is rockstable w/o it.
As far as I remember I also tested values up to 1011M but in all cases I had troubles with kswapd0 constantly eating cpu > 50% constantly.

Well I have a week left on a sunny Canary island after that I will be able to test again


You don't need ids (`INTEGER AUTOINCREMENT` at all) in your db?

If so, just change creation script to: `_id PRIMARY KEY TEXT NOT NULL`, don't forget to change `id` to `String` in the code!

See http://stackoverflow.com/questions/7591492/is-it-possible-to-apply-primary-key-on-the-text-fields-in-android-database


When using BOSH you can easily resume session. Stop sending new HTTP connections (server will wait some time  before disconnecting), save connection state in a cookie, reload page, restore it from cookie, start sending new HTTP requests.

In case of a WebSocket you have one TCP connection. As soon as you reload the page that connection is dropped and from the point of view of the sever you are disconnected. There is no way to resume it in general way.

Session resumption can be done using session management, which recent versions of ejabberd supports. However, this probably is not the role of connection class (be it WebSocket or anything else) and the client code should handle it itself.


Per the [docs](http://ffmpeg.org/doxygen/2.8/group__lavc__misc.html#ga66b44aad56fa22a78fb8916eb8cc0516), this should be `if (error_code < 0)`.

Only negative numbers are errors.


Typo ?


Yes I know. :-)


I deleted the code they are testing. Not worth it.


okay, the naming problems are fixed
< naming


Wouldn't be better to do it just once, after merging `data` with `entries_attributes`?


So I can test the bulk of the functionality without paying for loading the framework.


hi @ashar02,

This will be producing and sending a dictionary like this?:
```objc
@{
    @"data": @"true|false",
    @"video": @"true|false",
    @"audio": @"true|false",
    @"attributes": @{  /*... */ }
}
```

My question is if `attributes` key is standard to Licode or your own implementation. I have the understanding that standard Licode doesn't expect that.

My proposal was to leave `ECSignalingChannel:publish:signalingChannelDelegate:` as it was, and solve the custom parameters in `ECRoom:publish:withOptions:customOptions` in a way that if `attributes` or `customAttributes` are not standard Licode, the key used by each implementation of the server might be different.

That is if we have a method like `ECRoom:publish:withOptions:customOptions:` this will be the scenario:

1) Passing root level key/values:

```objc
[room publish:stream withOptions:@{@"data":@TRUE}
                   customOptions:@{@"custom_attr": @"value"}];
```
Should produce and send:
```objc
@{
    @"data": @"true|false",
    @"video": @"true|false",
    @"audio": @"true|false",
    @"custom_attr": @"value"
}
```

2) Passing namespace:
```objc
[room publish:stream withOptions:@{@"data":@TRUE}
                   customOptions:@{@"custom_namespace": @{@"custom_attr": @"value"}}];
```
Should produce and send:
```objc
@{
    @"data": @"true|false",
    @"video": @"true|false",
    @"audio": @"true|false",
    @"custom_namespace": @{
             @"custom_attr": @"value"
    }
}
```

Does it make sense? Waiting for your comments.


/Cloud Storage/Google Cloud Storage/s  for clarity


/Cloud Storage/Google Cloud Storage/s for clarity


Both of these should be initialized to ```TS_NULL_MLOC```


/GCS/Google Cloud Storage/s


This broke username for headercheck when the header check server is different


@Stefano80:
No problem. i see we are neighbors. I'am living in Ulm.


Yeah, I temporarily removed this since I keep track of how many values it encodes elsewhere.  I'm planning to add this back though.

Thanks for the comment (and code!).


oups yes SORRY !


Please optimize `object(new Car.Builder().personId(personId).model(person.cars().get(i).model()).build())`

You are querying same index of list multiple times, it can be done once :)

Otherwise, looks good!


Yeah, that would be better!


public void agrega(String elemento){
    @SupressWarnings("unchecked")T a = (T)(elemento);
    Vertice nv = nuevoVertice(a);
    //Es vacio
    if(raiz == null){
	raiz = nv;
	ultimoAgregado = raiz;
	elementos++;
    }
    if(esNumero(ultimoAgregado)){
	ultimoAgregado = ultimoAgregado.padre;
	agrega(elemento);
    }else{
	if(!ultimoAgregado.hayDerecho()){
	    ultimoAgregado.derecho = nv;
	    ultimoAgregado.derecho.padre = ultimoAgregado;
	    ultimoAgregado = nv;
	    elementos++;
	}
	else{
	    if(!ultimoAgregado.hayIzquierdo()){
		if(operadores.indexOf(ultimoAgregado.padre.toString()) == -1){
		    ultimoAgregado.izquierdo = nv;
		    ultimoAgregado.izquierdo.padre = ultimoAgregado;
		    ultimoAgregado = ultimoAgregado.izquierdo;
		    elementos++;
		}
		if(operadores.indexOf(ultimoAgregado.padre.toString()) > -1){
		    ultimoAgregado = ultimoAgregado.padre;
		    agrega(elemento);
		}
	    }
	}
    }


Ah, nice. I lived there for almost 3 years during my PhD and my wife's family live there. Have fun on the Donau then!


You known, that was friday afternoon :p


This change is causing a build error (if a relatively newer compiler is used - like 4.5.3) due to duplicate declaration of sai_vlan_port_t which was already defined in saivlan.h

/**
 *   @brief Port/vlan membership structure
 */
typedef struct _sai_vlan_port_t
{
    sai_object_id_t port_id;

    sai_vlan_tagging_mode_t tagging_mode;

} sai_vlan_port_t;


Maybe where sink and source buckets are arguments use "name" to be clear it's not the path "gs:// or something else?  So something like "Sink bucket name".


you have to check if the current user have permission to read from & write to serial devices too.


See comments about this idea here:
https://github.com/Rocky640/Stockfish/commit/48ea9d4c8ebd37e797abec8f7d3b552eb5ad4844


But it was earlier :cry:


Might be better as help='The transfer description'.


I think this should be org.mule.extension.api. If not all the APIs that we define will be in org.mule.api and it will be confusing for our users.


modal content has 3 first-level elements it's gonna be applied to :)


This happens in the absence of a gyroscope, right?


Sounds good to me.


looks good


I'll leave up to you because I don't know what the standard or practice is for help strings like this, but I feel like all the help strings need a period at the end of the phrase.


nice, i'll try this out tonight!  no dockerfile?!


java -cp ${ANORMPATCH_CLASSPATH}:/zipkin/zipkin-query.jar com.twitter.zipkin.query.Main -f /query-mysql.scala


Wourdcounts vs countwords.


Suggestions about help strings.


java -cp ${ANORMPATCH_CLASSPATH}:/zipkin/zipkin-collector.jar com.twitter.zipkin.collector.Main -f /collector-mysql.scala


How does this work for CJK languages?


When I ran script on tls connections, sometimes it failed to read the data. The readiness of reading operation was checked by stream_select, consequently when reading fails after stream_select, there is something wrong with stream_select. Looking at [this notice](http://php.net/manual/en/function.stream-socket-recvfrom.php#96459) in [stream_socket_recvfrom](http://php.net/manual/en/function.stream-socket-recvfrom.php) docs there is a chance to verify stream_select return value reading the socket buffer directly. If stream_socket_recvfrom returns everything not equivalent to false the next call to fread will be successful. Since data must not be actually read at this point, MSG_PEEK flag is used.


Thanks! I cant believe I missed that


Internal SD?


@ttfoley, that is duplicate of #65: numpy's openblas is conflicting with Julia's openblas.  See that issue for more information.


That doesn't work. That provides an application-level loading template (which would be hit if route:application returned a promise from it's model hook, for example). I want an application-wide loading template.


removed hard-coded port num here


Did this just close issue #24?


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


yes, because of the gp, vs gpd.  But better be explicit as it's easier to see. Thanks.


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

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


Yes, they should have periods at the end to be consistent with other samples.


Why? What's wrong with it etc? I'm not saying it's wrong, I'm juts interested from the reasoning behind, especially on this "site" being about security :)


Approved


transaction is misspelled as `transacion` in a few places in the concurrency section


Thanks!


Apologies for this.


I would switch this to txn_busy as well - to make it uniform.


My mistake; will remove.


This has been done just so we can keep our headers clean.


IIRC - this was used by shared cache for allocation logic - no? We should probably remove it everywhere if we are not making the app threads sleep hard anymore.

I would also see if we want a WT_PAUSE() at least instead of spinning hard.


I added it in mine and at the very least it makes the game brighter by not having the clouds up there darkening things.


what is the g function doing?  Also shouldn't the second get use transaction2?   It might make the code more readable to use tx1 & tx2 instead of transaction1 & transaction2


As I was sipping a little coffee this afternoon, I had a thought about the Reservations table (I had a reservation about it?).  My concern was with the field that represented a comma separated list of IDs, the Guests field.  My thinking here is that you create an additional table:  ReservationGuestXRef.  It might look something like this:
XRefkey -- flightID -- guestID -- employeeID: just four fields.

This would allow you to select all guests for a given flightID.  It would also allow you to remove the guests field from the reservation table altogether.  By using the flightID you can link to the XRef table and pick up guests, and perhaps employees as well.  It's just a ruff idea, probably needs a little polish, but let me know what you think.


this is not entirely true for all modern browsers:
in Safari CustomEvent is an object that identifies itself as "CustomEventConstructor"


what are dockerfiles?


The kernel just crashes at boot with this commit.


fyi @theengineear @etpinard

![image](https://cloud.githubusercontent.com/assets/1280389/9967552/4740063a-5e11-11e5-956d-6c8adc588c23.png)

![image](https://cloud.githubusercontent.com/assets/1280389/9967555/499a2d34-5e11-11e5-81ca-80562bf10b13.png)

![image](https://cloud.githubusercontent.com/assets/1280389/9967557/4e80f4d6-5e11-11e5-84c4-e8268c677bf4.png)

![image](https://cloud.githubusercontent.com/assets/1280389/9967559/514e8eb2-5e11-11e5-8a0e-800803724e35.png)

![image](https://cloud.githubusercontent.com/assets/1280389/9967561/546ac2c8-5e11-11e5-8aef-a521d947682c.png)

![image](https://cloud.githubusercontent.com/assets/1280389/9967566/5796ee9a-5e11-11e5-8310-597d9aa5c9b0.png)

![image](https://cloud.githubusercontent.com/assets/1280389/9967569/5a2ccbc0-5e11-11e5-8199-d044e7602711.png)

![image](https://cloud.githubusercontent.com/assets/1280389/9967574/5ce7432c-5e11-11e5-8a4f-ddb2c0607ea3.png)

![image](https://cloud.githubusercontent.com/assets/1280389/9967579/5fa3d56c-5e11-11e5-876f-f9b3584dc4a6.png)

![image](https://cloud.githubusercontent.com/assets/1280389/9967588/6c94e900-5e11-11e5-99a9-2e58282fbb7f.png)


A real human bean


`Annotation`, *not* `Annotations`

![image](https://cloud.githubusercontent.com/assets/1280389/9967598/7dcd2d68-5e11-11e5-83ce-ae8a6013bb30.png)

Conditional on that objects that are `_isLinkedToArray` end in an `s`


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

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


But uuid are generated in constructor - now what should I do with this:

```java
return person.id() != null
                    ? PutResult.newUpdateResult(1, affectedTables)
                    : PutResult.newInsertResult(personId, affectedTables);
```

Now I couldn't decide whether the person was inserted or updated...


LAMA


Ingen förväntning i form av `expect`? Alla tester måste ha det.


Just worning on the Up Next side then going to start to break it all up and drop it into some markup :)


Hi all


Your functions look great.

Were you using these try and except blocks to test your own code?  They don't really need to be here.


This is just a basic layout to give some html to work with PHP


print ("hello")


Okay, vad händer med `bike`? På vilket sätt kan den bli en del av `subject.docked_bikes`?


GetFields is recursive, so this is taken care of.


I didn't think of that. Will do.


Alahu akbar


dobre kazano


Är denna metod definierad? Vad används den till?


opa

2015-09-18 21:33 GMT+03:00 Ivan Siderov <notifications@github.com>:

> Alahu akbar
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SoftUni/test/commit/0143bc6edeb9cd15005933d22bd1a283d7701383#commitcomment-13317144>
> .
>


No; the director thing is only in try-out phase. I'm block by derivative/custom_forward/custom_reverse/jacobiansparsity refactoring before I finis the director approach and remove the older one.


printed out so that everybody can read it.


@vanmh I have made this change here: https://github.com/kiva/vertex/commit/67b3ff948d697ccb278eab9a16a8ecdab81aa92b

Can you please merge the pull request if the rest looks good? Thanks!


the if(localTemps) is redundant. ST won't print anything unless localTemps is not empty


dobre procheteno

2015-09-18 21:34 GMT+03:00 MMQuants <notifications@github.com>:

> dobre kazano
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SoftUni/test/commit/0143bc6edeb9cd15005933d22bd1a283d7701383#commitcomment-13317160>
> .
>


Thanks. I'll take a look at it.


this smacks of logic that should go into the code generator. there should be a model object for each kind of print maybe or a single print model that changes the vector or string or whatever that is suffix of print_.


We should use `Ints.saturatedCast(results.getUpdateCount().longValue())` to better handle huge result counts.


xcode7 (mbadams5) - 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)


The previous code had the effect that current thread will at least free a page and ensure there is space for itself. IIUC - Now we are getting out if we see net eviction happen across all threads. Overall I like this change (You don't want to hold up and spin just because you keep getting EBUSY) - but is there a case where we want to ensure this app thread has evicted at least one page? This might bump up our memory usage above allotted size - no?


is there a proper way to reload knxd to reparse the conf or anything?


yeah I know :( But since the re-design I made on the train it was not anymore
My weekend will be half dedicated to the chat to have the MVP asap.

But once again, I insist, we DO HAVE to find a good workflow


If you are not going to use them before initialization then you would set them as soon as you create the object, meaning that they all should be set in constructor. Otherwise there is no way for you to distinguish object with uninitialized values from correct objects.


Should not it be sprt  [0, 4]?


seems like the code generator should specialize the func call model to be multiple: heap object with and w/o Wich assignment and then nonheap object.


wazzup


i would also specialize these model objects. the basic goal is to avoid IFs in templates when possible/reasonable.


note there's a typo in the commit message.  It should say: `rails generate controller static_pages home about test`  (pluralize controller)


there is no precedence in Wich so we need to (...) each operator I think so C interps correctly.


Tiny style note. I prefer to use line wrapping to break up the lines and let them go on for as long as they would without a paragraph break. This is so:

1. They don't wrap awkwardly if your window is small.
2. You don't have to deal with people wrapping at different character counts
3. It makes altering the text easier, you don't need to break more if it gets longer or try to figure out it getting shorter.

I would make the text something like:
```
// HAX: {{Link}}
// If this method fails, free the buffer and fall back to the less performant method.
```


Not returning anything?


why is Typ not covariant?


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


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


Sorry for overlooking the "one-item" problem, where $resources is not an array and thus implode creating a warning.


what happens to the result of visit(s)? Don't we need to add to statements in Script object?


Would it be more slightly more maintainable to zero the one field that we allow updating, and then check that  the rest is DeepEqual?


No.


Yes. I wrote the test and then had to figure out why it was failing. The test is this:
DependencyExtension_Polymorphism+Capture
    Capture_PrivateDependencyInBase_IsCaptured


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


cc/ @oliversong, @dgouldin - needs investigation as to why the `preinstall` fails here:

```
➜  temp  npm i git+ssh://git@github.com:clara-labs/react-popover.git#npm-preinstall
\
> react-popover@0.3.4 preinstall /Users/stephen/clara/temp/node_modules/react-popover
> npm run build

> react-popover@0.3.4 build /Users/stephen/clara/temp/node_modules/react-popover
> rm -rf build && babel --out-dir build lib

sh: babel: command not found

npm ERR! Darwin 14.4.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "build"
npm ERR! node v0.12.7
npm ERR! npm  v2.11.3
npm ERR! file sh
npm ERR! code ELIFECYCLE
npm ERR! errno ENOENT
npm ERR! syscall spawn
npm ERR! react-popover@0.3.4 build: `rm -rf build && babel --out-dir build lib`
npm ERR! spawn ENOENT
npm ERR!
npm ERR! Failed at the react-popover@0.3.4 build script 'rm -rf build && babel --out-dir build lib'.
npm ERR! This is most likely a problem with the react-popover package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     rm -rf build && babel --out-dir build lib
npm ERR! You can get their info via:
npm ERR!     npm owner ls react-popover
npm ERR! There is likely additional logging output above.
```


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


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


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


Hah! Oh man, that's embarrassing.


we are walking visit(s) again here.


I'll do that


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


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


missing "()"
self._get_mandatory_shipping_fields => self._get_mandatory_shipping_fields()

It's why the runbot is red ...
Take a look in log, it's explains ...
```
    all_fields = self._get_mandatory_billing_fields + self._get_optional_billing_fields
TypeError: unsupported operand type(s) for +: 'instancemethod' and 'instancemethod'
```


I considered it, but without per field check you can't return the exact field that caused this error to occur in the reason so there is a trade off


I am not sure if you have any questions or concerns please feel free to contact me with any questions or concerns.


What is this commit


According to the exif spec, the subsec fields have variable length, and contain decimal fractional digits, not microseconds. So this field may have "5", which means 500000 microseconds.
The easiest would be = float("0."+sub_sec) for seconds, as I did in https://github.com/blaueente/mapillary_tools/commit/7817da9943e92e84f1040a9084a9eef25b05383f, and using that directly in the timedelta constructor. Thinking more about it, this may exhibit the usual problems of binary float representation. An alternative completely avoiding float: datetime.timedelta(microseconds=int(sub_sec[:6].ljust(6,"0")))


Sorry, had a mess by rebasing and changing...the PR is already updated...runbot does not trigger the correct commit anymore...can you trigger it from the PR?


Fixes [OnioNS-server/76](https://github.com/Jesse-V/OnioNS-server/issues/76).


should this be listed under enhancements instead of bug fixes?


All templates are instantiated (and checked) during compilation time.
If user passes objects that can't act as iterators, she will get compilation error.
Indeed it doesn't eliminate the possibility to pass wrong type here that can act as iterator.
If you want to be really thorough, in C++11 there is such thing as **static_assert** which allows you to add some additional typechecks that will be executed during compilation.
But no one uses it to check iterator type.
More details:
http://en.cppreference.com/w/cpp/language/static_assert


Now that I think of it, does this form have some benefits? It runs the command in a separate ruby process - could that somehow/sometime affect the results?


Why clear out file pointer here as opposed to in `saveAsync`?


can we check ctx.type()==TypeHelper._vector instead?


nit: weird spacing before `ParseFileUtils` and just use `return`


If I understand the hdf5 doc correctly, it should not happen,
except if your hdf5 has been compiled in a hdf1.6 non default compatibility mode.
Or it is a bug in the hdf5. After all, 1.8.4 is quite old, it was released in 2009.
Can you simply upgrade ? We have no machine with such an old hdf5, so I can not fix and check anyway.
@mferrero : shall I raise the version requirement to 1.8.11, which is default Ubuntu 14.04LTS ?


More descriptive attributes, unnecessary aliases removed.


maybe new CType(ctx.type()) is better?


You can remove these two lines by using a `defaultdict` here
```
self.process_cache = defaultdict(dict)
```


103353562-Issue-when-using-different-heating-and-cooling-airflow-rates-with-Airflow-Network-#5134 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1878 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.89%25-yellow.svg)


Same reasons why `Manifest[T]` is not. Representation of a given type B which is subtype of type A does not have to be a subtype of the representation of a type A. Or in other words: `B <:< A` does not imply `Typ[B] <:< Typ[A]`


we are repeatedly visiting s again. seems like we want to save the result anyway, which lets us factor it out naturally.


I doubt I'd understand this code anyway, but these two integers are pretty indistinguishable from magic in this context.


@rokas456 Isnt this going to breakt the sign in ?


@sirhanshafahath This line needs to check ``Array.isArray(include)`` before treating ``include`` as an array. It throws an error in the unit tests.


let's make a function that tells us if the type is heap-based like str, vec


r+ for TEST_CURVES="True" - after argument processing, cipherscan automatically disables TEST_CURVES if the chosen openssl doesn't support it.

r+ for TEST_TOLERANCE assuming it behaves similarly.


I don't believe there is yet, but you have the right idea. You want to file an issue with this?


i'd like to revisit this function once you've made the other changes; i can look at it in the IDE.


Pushed up a `NaN` sentinel change.


Thanks for your work @krazyeom!


Un commit qui diminue le nombre de lignes sans altérer la lisibilité est un bon commit.


The "getInstance()" is redundant, When you do: var game = new Game(); ... game should already be the instance, for this you can add an init method in your Game clase.


we should change all type tests that use strings to use Type objects


Also file an issue on making the new command an interactive shell.


Tutaj nie rozumiem co mam zrobić


we already have result in ifStat.stat; why not use that instead of visiting again?


Manda
`git pull https://github.com/12345ieee/cmg-cmssw.git master`
e sei a posto


We don't want to add anymore ``throw`` statements so this can be safely used as a module. I haven't reviewed the code enough, but if we're having a complete network failure, wouldn't both connections be down when you try to call this?


repeated visit of else stat


103353562-Issue-when-using-different-heating-and-cooling-airflow-rates-with-Airflow-Network-#5134 (lgu1234) - 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)


@lp-- thanks, I will resubmit!


Where is the instance of the Board (new)?, also you have the same problem that Game class (getInstance is too redundant).


repeated visit of statement


This should be removed prior to merging.


This is a good explanation :+1: I think it should get moved to the ``README.md`` if merged.


Maybe use `missingCasesSingle`?


With months of hard work, I'm so glad that we are finally to the point of achieve this! Rejoice for anyone who ever contributed to this project and worked hard to make it happen!

We are awesome.


In general terms is not a good practice to set JavaScript code in the index or in any HTML file, I suggest you to create an init.js file.


If $subdir is not 'htdocs', we don't need to do the set_include_path,
so i will move te line 172 after the 170 into the second test


I'm wondering if var def shouldnt have just one type field and then we set it to nonheap or heap var type.


I think a call to `transpose` is needed here, followed by a `foldl`, or something like that.


Do we need multiple types here any more?


I knew you would like lambda syntax :)
It's just a way to declare a function object without name and then assign it to some variable or pass it to another function.
Somehow equivalent to lambda in python:
```python
   f = lambda x: x * x
   v = f(10)
```
```cpp
   auto f = [] (int x) { return x * x; };
   int v = f(10);
```


woot, worked


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

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


So my thinking on this has changed a little bit. I think it's right to move back to single argument `case`, and this PR looks great, but I've changed my mind about monomorphizing case arguments.

It depends on the binder whether or not we should monomorphize. For `NullBinder` and `VarBinder`, it doesn't make sense, and the code change should be in the type checker. For everything else, I think we do need to instantiate the argument.

I'll work on the typechecker bit after this PR is merged, unless you want to tackle it?


So you were not able to find text 8122


How about using `std::ostringstream` for `buf`? Then you should be able to do:

```
buf << timestamp_ << std::endl;
buf << sequence_number_ << std::endl;
...
```


I think this makes for a much nicer core AST.


typo 'Controller'


can our var statement have more than one declaration?


**Code Quality - Nitpick**
Every key/value pair for the associative array should be on it's own line, thus:
```php
[
    "slang" => "hiya",
    "description" => "corrupt way of saying hi",
    "sample-sentence" => "Hiya Jen!"
]

``


We don't need it.


oh for calls to functions in init expr?


Can you drop the 'media' remove calls in lieu of #284 ?


Now that this is in, can we remove the `Data.Functor.Coproduct.Nested`, or is it still needed?


Se a coordenada for um numero negativo diga andando -%passos


Second too


Do you mean the actual command is visible in the build log or where is it visible? I still don't understand :)


Can all versions of openssl can perform these tests?

Why does this patch replace iteration over all available tolerance tests with only 'big-TLSv1.2' at line 776->777?


@karimnaaji was there any other reason of using `initUniformSampler`.
It seems to me we can rely on `contextLoss` to determine if uniforms need to be set.


What do you mean?


Merge inicial


coś to powinno być


You do realise it should be static class right? You can't have a public class inside a public class :P


os.system is pretty old; this functionality now exists in subprocess (which we now use in Pydpiper).


IntelliJ and scala compiler warn that these `Long`s can never `== null`


I've been seeing that as well.


I think the apostrophe has to be doubled in order to show up :-/


Nice! Typo: "learn by hard" --> "learn by *heart*"


This gives me a checkstyle error:
Wrong lexicographical order for 'java.net.URL' import.


As per `require()` it doesn't really matter where it will be located, because it will be installed in `node_modules` anyway, and this is where `require()` looks by default. You could even put it in both and then you would get a nice warning. However, there is semantic difference between both locations. Perhaps the issue originated when you still used custom version in `vendor`. That's a completely different story. Anyway, I moved it back in PR #2800, so no action on your side is required.


**Code Quality - Nitpick**
Please be consistent with the coding style, the opening braces come on the following line, as recommended in `PSR-2`.


"Forward" Page


it would been simpler to remove the version from the logmanager. .there's a method on maven to do this....

But this is good enough already.. I will merge this.


Wrong lexicographical order for 'java.net.URL' import.
Here as well
< layout


Special Content Page


comment


Could you add phpdoc?


> The InputStream that reads an incoming file

sounds too generic.  `in` here is almost certainly supposed to be stream from which to read the `fetch.txt`, so I think this line can be more explicit about that.  The methods below should probably be modified in the same way so that they're more clear.


@aharley40 use null to represent "unset" things, and there should be spaces around this `=`


fixed, thanks.

On Fri, Sep 18, 2015 at 10:34 AM, Peter Salvatore <notifications@github.com>
wrote:

> I think you mean to drop 0.6.6 here. The tag would still exist on
> Dockerhub for repeatability, but it wouldn't get prominently displayed in
> the tag list at the top of the documentation users see.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nats-io/official-images/commit/aab4a118c1c5a169489c7da98d85cb3921c73cda#commitcomment-13316015>
> .
>


In the specification for function H5Tarray_create parameter 'perm' is defined as unused but it is still there in the declaration. It seems that the HDF5 developers understand this definition differently, depending on the version.
I can try to upgrade HDF5, although this would require searching for some upstream package for my Ubuntu 12.xx and reinstalling h5py.
In fact, I see it as a problem when the requirements for the same version of TRIQS in the master branch evolve with time. Ideally, I would expect no major change in the compilation procedure (as well as API) within the same minor version.


Please write a comment on the protocol followed. When a client asks for tokens, he is returned the last index at which the sequencer was with a lease for the number of tokens it has asked.


Maybe want to exec rather than subprocess:

    os.execv((python, pip_faster) + reqs))  # never returns


"interactive shell" as in editing the script or nuspec inside of the shell like with nano in linux?


Yes it should.


Not the command, for example the clientSecret for the sphere project should be a secret.


@ckunkel can you look into this report? I don't think this logic is sound

@aharley40 Demonstrations should never be filtered by DemonstratedLevel, if the goal here is to include a line for each level the student has completed, TargetLevel is what you want to be looking at, which in this query was already controlled by the `$currentLevel` variable. I think that's what you would want to replace with the passed in level to get their completion of a given level. TargetLevel is what separates each "pass" students take through the standards at each grade level, and then their average level for that grade is the average of DemonstratedLevel


lol, hater.


how come?


**Code Quality - Nitpick**
Often for a long `param` like this for a function, try and separate it for better readability. I will recommend something like this:
```php
$data = [
    "slang" => "Net Forget",
    "description" => "Someone who forgets the information they are
            seeking in the internet due to the overwhelming amount of information in the internet.",
    "sample-sentence" => "Prosper is a Net Forget person!"
];
$testy->addData($data);
```


already pushed a fix.


@aharley40 this `9` should be replaced with the TargetLevel, it shouldn't be removed from the export


I think there's a boo-boo?
I don't see that a pip-faster.py got added.


oh i think i kow what you mean:

type in the package name
> packagename
type in the version
> 1.0.0
...


**Comments - Nitpick**
Use proper `docBlock` syntax, refer to the link I provided in my earlier comment.


ah, here's the moved file


ditto


Slice already returns a pointer, so *[] returns **.


I believe the fixtured-pypi branch is the Right Solution for this, but we can work on that after we merge this.


i'm beginning to wonder if the visitor function return type shouldn't be a pair of things: a model object and a list of any temps needed to satisfy the model.  I think ANTLR has a Pair<...> built in if you want to try it out and see how the code looks.


Also please comment that the Max is used to handle multithreading issues. Although there seems to be an inconsistency here. There could be multiple threads updating the same stream sequence and at that point what is store in the map is not necessarily the value the stream is at. Could this could affect the get as we fetch the tail everytime ?


I still can't seem to get the same warning as you do. If I fix it with ctrl+shift+o, I get even more warnings. <layout


*mce button


i think here is where we generate specialized print model objects.


Doubling the Isp seems too much, what I can do is increase to RCS Arcjet to 1000s for Hydrogen, which is about the upper limited possible with this technology. Any higher any it starts to melt the nozzle.


The point of using `*[]` was that `omitempty` would hide the empty `<nodes>` element. The pointer to a slice was a pain and wrong as far as Go goes, so I reverted to not using a pointer and leaving the empty XML elements present.


This one overwrites the commit f5d8acc, which will results confused printing of messages in the console.


Some comments on heartbeat as to what they are used for and what is the frequency would be useful.


a switch would look nicer i think.


Esta solução é baseada no tutorial http://useyourloaf.com/blog/updating-to-the-ios-8-search-controller.html


should we also check that `this.size === image.size` ?


sounds good to me.

On Fri, Sep 18, 2015 at 3:14 PM, sswelm <notifications@github.com> wrote:

> Doubling the Isp seems too much, what I can do is increase to RCS Arcjet
> to 1000s for Hydrogen, which is about the upper limited possible with this
> technology. Any higher any it starts to melt the nozzle.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ABZB/KSPInterstellar/commit/585ede712c03a35cc13071979be8149a7d581985#commitcomment-13318054>
> .
>


maybe we can use the grammar to break this into multiple visitor methods.


We could rely on that, if `Tangram::contextLost` was called when the app initializes itself, which I think was the case until #269 . I think that now it's unsafe to rely on this, it might just work, but probably not for all platforms. But anyway this should probably go away for something more reliable in the shader to more easily init uniforms that need to be set once until any any context loss; maybe a `setConstantUniform()`.


Now i see your commit.


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

This is a first step towards discovery. In second iteration, i hope to have ``discover_profiles`` in the storage driver that will discover existing profiles defined by storage backend. That way, we could support power users' custom profiles usecase that you described yesterday. Does this sound reasonable?

> 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.

Thanks for suggesting DEFAULT! Yes, it sounds better than EMPTY. Will change. Thanks!


Filed https://github.com/bigdatagenomics/adam/issues/826


Unexisting


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

org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridColumnSelector: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

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


TeamCity DNN JavaScript Libraries :: CI [Build 91](http://teamcity.engagesoftware.com/viewLog.html?buildId=363&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 91](http://teamcity.engagesoftware.com/viewLog.html?buildId=363&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:7


We want to catch any thrown errors / non-zero exit codes though so the logic to roll back the venv will execute. Wont using `execv` cause that not to happen? Or am I totally misunderstanding how that works?


Awesome, we had this on the backlog already!


Thanks. I will continue to try and get better at this.


Now I am confused.  Does desired value of BISON_PKGDATADIR depend on recipe which uses native:bison, or the provider (recipe/package or host system) of native:bison?  I assume it is one of them, not both.

In case it is recipe using native:bison, I don't see why BISON_PKGDATADIR could not just be set in recipe.

If it depends on the provider, and needs to be set to something non-standard in the native:bison recipe provider, I would think it could be implemented with a wrapper that derives the value of BISON_PKGDATADIR from path of the wrapper, and is simply not set or wrapped or anything in when bison is provided by host system.

Am I right, or where am I wrong?


if my those imports are at the old places, there are no checkstyle warnings for me, anyway, I'm going to work on the fixing the logger
< layout, feature


This sounds like it may be a clue as to what's going on. I'll get back to you.


Actually, would you mind changing these variables to use the `var` keyword instead of `string`?


**Functionality - Awesome**
Good application of _Regular Expressions_


Looks good.


We could move to loadParams to make faster.


This will break old forge version - there is no getFluidID in old versions of forge.


This may actually finally fix problems with netty native, but who knows? ----> I know for you :)

When do you fix the Time cycle problem?? I cant wait for many bug fix it


We could make a class out of this.


This actually stopped being an issue when i fell back to the single vendored file approach, I'll put this back and deal with it in the future if it ever pops up again.


why are u doing stdbool now? i really like R_TRUE, R_FALSE and R_TRUFAE


See #24.


**Functionality - Critical**
This should be a static property, according to the requirements.


I am sure arithmetic capability in rc has been discussed before. Maybe it is time to open it for discussion again?
(Creating a subprocess just to increment a number or compare two numbers seems wasteful.)


This doesn't sound right.  The "algorithm" in this case is the hash function used to compute the checksums for each file in the manifest.  From what I remember, a manifest that used MD5 to compute hashes would be named `manifest-md5.txt`.  A bag can contain multiple manifests with different algorithms, so the same bag could also ahve `manifest-sha1.txt`.  Check the implementation, but I'd guess that this method is used to locate a manifest that used the given algorithm to compute hashes.


**Code Quality - Critical**
File name mismatch with the name of the class.


There are more uses of `strftime (3)` in the manual, can you make it consistent for all uses?


Please check spec.  A payload manifest is a file that lists all of the file part of the bag's payload.


Same comment here as for `getTagManifest` below


Nit:  you can probably remove these `&**` thanks to deref coercions, no?


Why do we need this leniency? That would mean having "gLaCIeR" would also work. Let's just support the lower case and that's it? We are documenting this on our own, no need to support enums from the library itself.


**Functionality - Nitpick**
I see you have _PHPSpec_ here but I don't see where you are using it in your project, did you need it? Please clarify.


This is incorrect.  A "tag" in a bag *is* a `BagFile`, so this method is really just returning the list of tag files found in the bag.


#69 Just adding the imports you need might be more 'clean' (at least according to Checkstyle :P). Smashing 'ctrl+shift+o' will help you out here, if you agree ;)

<organisation, review


Hi snicolet, how does this makes a big difference, only though you tuned it very slightly?

And can I know how did you get the idea around fixing this? :)


103353562-Issue-when-using-different-heating-and-cooling-airflow-rates-with-Airflow-Network-#5134 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (806 of 806 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.93%25-green.svg)


The concorde was broken
this fixes it

See:
http://www.thejabberwocky.net/viewtopic.php?f=6&t=40


I'd disagree in this case. glacier seems like a special case, and if aws releases yet another useful storage class tomorrow, we wouldn't actually need to change anything but the docs here. The enum values just gives us a nice way of verifying our sdk version supports the input given... I don't think leniency is bad in this case.


**Functionality - Critical**
The new words should be added to the `static property` in the `Urban` class (or Slang class — whatever you choose to call it). As it is right now, you are creating a new object of the `Crud` class and adding this new entry, not according to the requirements.


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

org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridColumnSelector: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

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


About Git : be careful when you commit to master. You should create a branch by `git checkout -b banana` (choose the name you want) and, after commit, `git push origin banana`.


@stmcginnis : moving email discussion over here for consolidation. Thanks!


But why you want to generate UUID in the constructor? It's a perfect work
for the PutResolver!

пт, 18 Сен 2015, 21:29, Rainer-Lang <notifications@github.com>:

> But uuid are generated in constructor - now what should I do with this:
>
> return person.id() != null
>                     ? PutResult.newUpdateResult(1, affectedTables)
>                     : PutResult.newInsertResult(personId, affectedTables);
>
> Now I couldn't decide whether the person was inserted or updated...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pushtorefresh/storio/commit/def44d62cc36823215009f20e2a8921cc2817109#commitcomment-13317069>
> .
>
--

@artem_zin


**Functionality - Nitpick**
What if you are updating something that's not there, how do you tell? I suggest that this method should give some response, either way.


although this doesn't pull from a separate site, the functionality is the same.


@ke-yu You caught me. I set this flag to true in order to force `UpdateRenderPackageAsync` to render. I think we should change the signature of `RequestVisualUpdateAsync` to
```c#
RequestVisualUpdateAsync(IScheduler scheduler, EngineController engineController, IRenderPackageFactory factory, bool forceUpdate = false)
```


**Code Quality - Nitpick**
As I've noted in my summary comments, you don't need all these code in your commit, the `composer.json` file is enough, then the end-user can do `composer install`.


TeamCity LabkeyModules15.2 :: CDS postgres 9.4 [Build 16](https://teamcity.labkey.org/viewLog.html?buildId=272972&buildTypeId=LabkeyModules152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 22 Build time: 00:35:20
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

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

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

```


[App.jsx] ----- ERROR -----> Error: Invariant Violation: Missing "account_name" parameter for path "/account/:account_name/overview"


Might it be better/easier to use [Objects.hash(...)](http://docs.oracle.com/javase/7/docs/api/java/util/Objects.html#hash(java.lang.Object...)) or at least [Objects.hashCode(.)](http://docs.oracle.com/javase/7/docs/api/java/util/Objects.html#hashCode(java.lang.Object)) as to not need the null checking?


Agreed!


It is being used on line 7, after all :)


**Code Quality - Nitpick**
This blank line is not necessary!


I think maybe just > than size we still want to check the last element and in my test build it leaves out if I implement it as >=


Yeah, all these `TODO` points I was intending on coming back to after I got the desugaring part working. `missingCasesSingle` definitely seems like the answer here!


Nope! Another one of the things I was going to come back to. This change makes for quite a few nice simplifications :smile:


@dsanders11 despite the label being updated, was this host change necessary internal to Java for the websocket server to listen on a non-localhost?

According to [this article](http://www.programcreek.com/java-api-examples/index.php?api=org.eclipse.jetty.server.Connector), we may want to use `0.0.0.0` generically so that it doesn't fuss about incoming connections.


That actually brings up a good point. Getting the same word again if you switch back. If it didn't, I could see some people cheating by switching weapons until they get an easier word or something.


rollbacking change from previous commit. it is no longer necessary as request url is stored in the log


:D


this patch was reverted. the defaults remain.


TeamCity DNN JavaScript Libraries :: CI [Build 92](http://teamcity.engagesoftware.com/viewLog.html?buildId=364&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


I'm happy to leave the typechecker changes to you :wink:

Hopefully it won't take me long to figure out the desugaring thing now that I'm not half asleep! The `transpose` thing sounds about right.


TeamCity DNN JavaScript Libraries :: CI [Build 92](http://teamcity.engagesoftware.com/viewLog.html?buildId=364&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:7


Why not Promise?


**Code Quality - Nitpick**
Wrong casing for the 3 methods, should be `testDeleteData`, `testUpdateData`, `testRetriveData`.

And I even suggest that other than using `*data` in your project, let the methods be more self-explanatory, you could use `word` instead.


Yep, we can ditch that now. I have already done so in the set of changes I'm working on.


Can we avoid doing it, e.g. by passing `XMLHttpRequest` ctor to methods (with appropriate comment which clarifies why we do this)? I don't like hammerhead dep here as well.


Would adding a more complex test case for equality true help? (i think it will.)


**Code Quality - Nitpick**
Fix this indentation.


103404908-AddWarningWhenScheduleTimestepsDoNotAlighn (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1874 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-100.0%25-green.svg)


Nice! Thanks.


This should be pos == 0 since the else is now returning player 1


@Alymosul  @omarwanis  please review this commit


Here and in all other places: why it's called `hammerheadAPI` why not just `hammerhead`?


I patched the command parser so that it should no longer need this sleep hack. However, I have no tested the patch as I don't know how to run the adventure sample app.


Excellent! I moved out all legacy code and now only use the NIO APIs: https://github.com/pathikrit/better-files/commit/9d6e92af44f32e35a24d7c2e69aaf1ba8d47fc7c

the more I think about it is this is all we need: `scala.nio.Path`

```scala
package scala.nio

import java.nio.files.{Path => JPath, Files}

case class Path(path: JPath) {
   def toJava: JPath = path
   def name: String = Files.getName(path)

   ...
}

implicit class PathToScala(path: JPath) {
   def toScala: Path = Path(path)
}
```


The behaviour of 1.6.x compatibility mode is not simple. Maybe hdf5 in 12.04 was compiled with some compatibility mode option, I do not know.
You can try to compile the code with -DH5Tarray_create_vers=2
to enforce the right version of the function


ship


ok


I don't understand why (I don't know a lot about the Go compiler and was copying almost verbatim from [here](http://blog.paracode.com/2015/08/28/ruby-and-go-sitting-in-a-tree)), but that doesn't work. The compilation works, but the result can't be loaded, and in fact doesn't even seem to be a shared library.

With `package main` and dummy `func main()`:

```
$ make libmylib.so
go build -buildmode=c-shared -o libmylib.so mylib.go

$ file libmylib.so
libmylib.so: Mach-O 64-bit dynamically linked shared library x86_64

$ ruby mylib.rb
42
```
With `package main`, removing the dummy main:

```
$ make libmylib.so
go build -buildmode=c-shared -o libmylib.so mylib.go
# command-line-arguments
runtime.main: call to external function main.main
runtime.main: main.main: not defined
runtime.main: undefined: main.main
make: *** [libmylib.so] Error 2
```
(no `libmylib.so` generated)

With `package mylib`, independently of leaving or removing the dummy main:

```
$ make libmylib.so
go build -buildmode=c-shared -o libmylib.so mylib.go

$ file libmylib.so
libmylib.so: current ar archive

$ ruby mylib.rb
/usr/local/var/rbenv/versions/2.2.2/lib/ruby/2.2.0/fiddle/import.rb:85:in `rescue in block in dlload': can't load libmylib.so (Fiddle::DLError)
  from /usr/local/var/rbenv/versions/2.2.2/lib/ruby/2.2.0/fiddle/import.rb:82:in `block in dlload'
  from /usr/local/var/rbenv/versions/2.2.2/lib/ruby/2.2.0/fiddle/import.rb:73:in `collect'
  from /usr/local/var/rbenv/versions/2.2.2/lib/ruby/2.2.0/fiddle/import.rb:73:in `dlload'
  from mylib.rb:6:in `<module:MyLib>'
  from mylib.rb:4:in `<main>'
```

Interestingly, the example in the blog post I linked, does **not** have a dummy main...


ack.


ack


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

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>

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


I was mapping a legacy database today that uses multi-dimensional Postgres arrays. The definition did not look like anything special, it was `character varying[]`, but inside were actually 2-tuples and the plugin threw an exception.

So I looked at the code a bit and did the minimal required to support such a mapping. The domain class looks like

	import net.kaleidos.hibernate.usertype.ArrayType

	class ArrayTable {
		String[][] arrayColumn

		static mapping = {
			arrayColumn type:ArrayType, params: [type: String, dim: 2]
		}
	}

I'm not actually expecting this pull request to be merged (maybe there's a better way for getting comments on Github), it doesn't even have unit tests, I'm sure additional criteria support would be required etc, but I wanted to get the author's opinion on supporting multi-dimensional arrays via this plugin.


selector is `undefined` here


Looks good!


TeamCity pull requests :: netty [Build 223](https://garage.netty.io/teamcity/viewLog.html?buildId=1480&buildTypeId=pulls_netty&guest=1) is now running


I think we want to do our own block level encryption on a loopback device (preferably backed by RAM) as a stopgap.


develop (DavidGoldwasser) - 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)


Will help doing what?


systen -> system


I "think" this revert is causing my build not to boot.  Thoughts?
Thanks.

--------- beginning of crash
E/AndroidRuntime(  807): *** FATAL EXCEPTION IN SYSTEM PROCESS: main
E/AndroidRuntime(  807): java.lang.UnsatisfiedLinkError: dlopen failed: cannot locate symbol "__htclog_init_mask" referenced by "libtime_genoff.so"...
E/AndroidRuntime(  807): 	at java.lang.Runtime.loadLibrary(Runtime.java:371)
E/AndroidRuntime(  807): 	at java.lang.System.loadLibrary(System.java:988)
E/AndroidRuntime(  807): 	at com.android.server.SystemServer.run(SystemServer.java:266)
E/AndroidRuntime(  807): 	at com.android.server.SystemServer.main(SystemServer.java:185)
E/AndroidRuntime(  807): 	at java.lang.reflect.Method.invoke(Native Method)
E/AndroidRuntime(  807): 	at java.lang.reflect.Method.invoke(Method.java:372)
E/AndroidRuntime(  807): 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:911)
E/AndroidRuntime(  807): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:698)


Yes!

On Sep 18, 2015, at 11:18 AM, "Keith T. Star" <notifications@github.com> wrote:

Did this just close issue #24?
—
Reply to this email directly or view it on GitHub.


Loosk alright, we should maybe think about editing the logger to make it more 'test-friendly'. Other than that, The tests work for me. <testing


Congrats!


Except that we still need @kmonson to update the PDB2PQR page.

On Sep 18, 2015, at 12:51 PM, Nathan Baker <nathanandrewbaker@icloud.com> wrote:

Yes!

On Sep 18, 2015, at 11:18 AM, "Keith T. Star" <notifications@github.com> wrote:

Did this just close issue #24?
—
Reply to this email directly or view it on GitHub.


Please review/fix these changes after reviewing the terminology in the BagIt spec.  Many of them don't seem correct.


The little train that could...


O continue pode ser usado em laços de repetição for, while, e do-while para interromper prematuramente o fluxo de exercução e avançando para a proxima interação.


ack.


Grammar needs to be fixed here.


see tests/regression/tools/wildcard
"modprobe lttng-test" and its use.

e.g.
function run_kernel
{
        # Trigger the event for 100 iterations
        echo -n "100" > /proc/lttng-test-filter-event
}


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


@dmfenton so this was removed else where? IdFilter is used in selecting pages of data from the DB based on the ID column where relying on `where` wont work.

Just checking, you may have solved this...


It will be more solid than expecting the scheduler to emit events during a sleep.


We don't know yet if this makes a "big difference", we only have a hint that the gradient may not be completely flat in this direction at STC :-)

In general, when we change something in the evaluation function, we try to make a correction in the order of 10-20 eval points (that's about one twentieth or one tenth of a pawn). Bigger eval change may have the desired driving effect on the search (Stockfish obeys the new rule which you have coded and changes its style of play) but often at the cost of "search rigidity" (it obeys the new rule *blindly* and deteriorates other parts of its position).

Here I changed the multiplicator from 201/256 to 206/256, which is about 2.5%, but this is applied to the pawn structure evaluation in pawns.cpp, which can easily reach several hundred points (see the Scores and the Seed[] over there). So we're probably not too far from the 10-20 points range.

As to why I tried this, it is because of the patch "Scales the endgame score by the number of pawns" by Stefan. Its effect is probably that Stockfish enters in endgames with a little bit more pawns on average, so I thought that it would maybe help to raise the importance of the pawn structure quality score in endgames.

Or it was pure luck, of course :-)


Some javadocs like these sound overly verbose.  It should be fine (and probably easier to read) if it were just something like "Remove a tag directory from the bag."


Merged


new line here


we missed this one


for completeness


is possible to disable this cop from rubucop? It would be more readable.


flake8 says: E241 multiple spaces after ','


flake8 says: E128 continuation line under-indented for visual indent


:shipit:


Use `utf8Slice()`


That's great! I have been trying to get this to work recently with @2x icons, but without success. Hope it works for you.


Never call `getBytes()` on `String`, it uses the default charset


Nice！


Name this `schemaName`


así está:

https://github.com/datal-org/datal/blob/develop/core/lifecycle/datasets.py#L335

no sé bien por qué hace el resto de las cosas, pero deja la nueva revisión como DRAFT


is this not a bug?

e.g. woudl you not need to expand the filename with some get_file functions where argument is argument filename. I do not think that the check yaml file expands the file


Correcciones a la estructura básica.


#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)


Seems the use of the old ConfigDict from base is a bug


@maddox if your bridged device support event and you can forward that event to Homebridge then you can use that to setup event trigger


Sérieux j'ai pogné dekoi quand j'ai vu à quel point ça torche côté temps d'exécution et mémoire utilisée !

Selon les stats de CodeEval, les challenges que j'ai fait en C# prenaient autour de 100 ms à s'éxécuter, au mieux. Ceux en C++ à date en ont tous pris 3 ms... Côté mémoire, ceux en C++ prenaient autour de 10k. Ceux en C#, 4.5 mb au mieux...

J'avais jamais codé en C++ avant ce mois-ci, seulement un peu de C à l'école, mais je pense que j'vas continuer à pousser un peu plus de ce côté là... C'est trop puissant.

Ça donne des bons points aussi pour le classement, d'ailleurs... Je m'en viens te chercher, Steve ! : P


Need to include the name in the function call as well?


Also removed auth filed


@pkallos is there a way to access commission data or reporting via an API?


The context of this method being used for SSC backups isn't present within the method. I propose moving this log message to the outer context or altering the message to be more context agnostic.


Add some more description, as well as say that if a timeout of the same name already exists, the old one will be removed.


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

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


I think it should be `==` operator.

http://stackoverflow.com/a/4528453/2047418


a) why do we use bson and not just json fro the dump function
b) we must not import pymongo as we do not want to cretae a dependency on mongo for windows.


since `false` and `true` are frozen objects in Ruby this and test below fail.

https://github.com/svenfuchs/i18n/issues/324

I can't really figure out how to fix it


please use inspect code as you do not follow pep8 coding standards


Add a helper to materialize the cursor into a `List<MaterializedRow>`


I think something like: [.TouchDown, .TouchDragInside, .TouchDragEnter] is cleaner


@Spartacus1 @dakcarto's builds are broken. Look at the file date, they haven't been updated in months. I think you need to use homebrew and build from master yourself, but @dakcarto can probably confirm. (I'm not knowledgeable about OSX)


Inline all these `value` variables


Thanks! :V


Make the stats non-zero (and all different) so we verify they are being selected properly


Looks like it should be all good, will pull.

The only slight issue might be with the changes made at @@ -533,7 +533,7 @@ protected function render_graph_init() for shifting from 1 to 0. IIRC, they had to be at 1 otherwise the stroke/border around the graph wouldn't show. Will test and see if it was just a browser compatibility problem of the past.


empd (nmerket) - 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)


Your namespace is polluted, you goof.


Here books table should also be used as purchase date is in that table.


@Marinofull tem gente commitando na master...


我们这里只使用了kDenseRowDtypeID


这样会有一个空格啊。要把，换成+才行。


Would rather see optional parameters be added to an array, rather than as a separate parameter, to allow for further extensibility.


Las instrucciones indican que hables sobre ti e incluyas una forma, esto falta.


try to use join.


Se vida é igual a 0, será que precisa atribuir 0 para a vida?


camera woorks?


sobre el dataset se habia  hecho mejoras ue debieron replicarse en datastreams y visualizaciones @danielpadula pti


This is a cute hack for hints. Nice!


Asi es. HAce un tiempo que no tiene mas sentido hacerlo de la vieja forma.


try putting it inside the adapter definition so the class/modules are already defined.


Unfortunately, yes :) Proper kernel driver needed.


shouldn't this be `instance_options`? whoops on me if I missed that


Corrected the code to not use ConfigDict from base:
https://github.com/cloudmesh/client/commit/d8eed0cae8e21accd2a1a397338f0bd6952fb0b8


It's always good to chain the other exception so it appears in the log file. Like --
throw new RuntimeException("blah blah blah", e)


isso compila?


Las instrucciones indican usar cada uno de los selectores (clases, ids, universales, etc) y sólo usas la etiqueta.


Hahaha ! Fuck !

Va falloir j'écrive une version C++ de ce que j'ai solutionné à date.

Je savais que le temps d'exécution a plus d'allure avec du code natif mais pas à ce point-là ! 100ms versus 3ms c'est immense.

L'autre petit downside de solutionner en C# est qu'on peut tricher les algorithmes (avec LINQ) qu'ils veulent nous faire implémenter. Sur CodeAbbey, ils sont + conscients de ça et ils disent exemple "utilisez pas LINQ" ou "utilisez pas telle méthode si vous voulez apprendre telle technique".

Bon fak j'me retape les 40 problèmes que j'ai solutionnés en C++ ! Ça risque de prendre + de temps pour ceux là !


I'm all about the simplest possible thing :)


Already fixed in the PR I just merged.


@bf4 nope I think that was me! 😀 I'll fix it.


precisa do return aqui?


The filename is expanded in CloudRegister.check_yaml_for_completeness(filename)


por que não retornar o próprio objeto de data de nascimento?


102966118-HeatRecoveryBypass-4568 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: 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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.22%25-green.svg)


Big thanks.


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

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


Why do this?


i think i saw you assign cloudmesh.yaml if none.

however would it not be soem function such as

???_get_config_file_location(“cloudmesh.yaml”)

maybe thats done internally? I just want to raise this issue.

> On Sep 17, 2015, at 10:30 PM, ehdsouza <notifications@github.com> wrote:
>
> The filename is expanded in CloudRegister.check_yaml_for_completeness(filename)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/cloudmesh/client/commit/210c2825981813651991b9441fa366d82b5cc64e#commitcomment-13301859>.
>


Course Project for Practical Machine Learning on Coursera


Good job dude!

For default event detail setting in `InteractApp/EventInfoPage.xaml.cs` you could do something like:

```
evtName.Text = "Event Name 12";
if (evt.Name != null) {
		evtName.Text = evt.Name;
}
```

as opposed to what you're doing right now:
```
string eventName = "Event Name 12";
if (evt.Name != null) {
		evtName.Text = evt.Name;
}  else {
		evtName.Text = eventName;
}
```

Or if you want to centralize this stuff:

Event.cs
```
public static readonly String DEFAULT_NAME = "Event Name 12";
```

InteractApp/EventInfoPage.xaml.cs
```
if (evt.Name != null) {
		evtName.Text = evt.Name;
}  else {
		evtName.Text = Event.DEFAULT_NAME;
}
```

It'll save lots of lines of code. We might also want to sit down one day as a team and decide some standards/conventions to keep our code consistent.

Keep up the good work!


改行


Fiz assim pra que ele retornasse no formato dd/mm/aaaa


#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)


it's not just the .icon that needs box-sizing: content-box. I found that bootstrap was causing horizontal misalignment of the entire alert because the negative margin offset calculation takes into account padding, which is not added to the alert's width when box-sizing: border-box is in affect.


すごく長ったらしいので、`scopeメソッド`をつかって分かりやすい言葉に変えてしまいましょう！

where(prototype_id: @prototype.id).exists?あたりがいいかと思います！


+1 / LGTM / Confirming this update works well!


Let's also try to avoid the huge chunks of commented code before we merge this stuff eventually. Whenever you think you're ready just submit a PR into develop. Has this been tested?


Ok, e você usa ele para algum teste ou para exibição?


@mkodekar unable to fix authorship via smartgit original commit is https://github.com/CyanogenMod/android_bionic/commit/70533720eac28474835925b5cd26a2ca08438dee


but then i'd have to config inside homekit i suppose. I'm sort of bridging Home Assistant to HomeKit 100% for siri as a frontend. I don't want to mix up automation configs.

@KhaosT Thanks for clearing that up!


Almost there! Just got the user "float" left :P


Do you think I should add it back in? Is it something you were thinking of using for the islands?

I'm happy to put it back if we're going to use it - I was just trying to clean up the code to make it easier to make sense of it all.


There is no reason to do this, without Authorization header all requests are treated as anonymous requests and are handled up the chain.


oi cara, tenta remover estas duplicações de código nos construtores!


I am a nice comment


não é necessário os `()` entre as variáveis.


Realmente me equivoquei, corrigindo aqui.


##### This commit will be rejected for:

Conventionally, we do not need to set up a dev env on Travis virtual environment. Instead, the only tools we need to compile & dist a Scala app are SBT and JRE(most of our projects are using Java 8). The lines here below are the demo .travis.yml which is used in cloud_shennong:

```scala
language: scala
jdk:
- oraclejdk8
scala:
- 2.11.6
env:
  global:
  - secure: [here is access key encrypted by Travis]
  - secure: [here is secret key encrypted by Travis]
before_script:
- mkdir -p $HOME/.sbt/launchers/0.13.8/
- curl -L -o $HOME/.sbt/launchers/0.13.8/sbt-launch.jar http://dl.bintray.com/typesafe/ivy-releases/org.scala-sbt/sbt-launch/0.13.8/sbt-launch.jar
- curl http://$ARES_SERVER/downloads/ares-linux-386.tar.gz | tar xzv
script:
- sbt ++$TRAVIS_SCALA_VERSION -J-XX:ReservedCodeCacheSize=256M clean coverage test
  && ./ares
branches:
  only:
  - develop
cache:
  directories:
  - $HOME/.ivy2/cache
  - $HOME/.sbt/
```


Looks reasonable. A couple points:

1. Can you rebase off develop so we get a nice green Travis check mark next to the PR
2. Since we want to go for the full table like IBM eventually it doesn't seem too messy to generalize to take an iterable of qubits and iterable of sequences and use ``reduce`` and ``operator.mul``.  See [this unreadable line](https://github.com/BBN-Q/PyQLab/blob/develop/QGL/BasicSequences/helpers.py#L19) for an e.g.  Then we can just call it ``SimultaneousRB_AC``


Não precisa de tanta formalidade, podes colocar direto

```Java
return this.ano % 400 == 0 ||  (this.ano % 4 == 0 && this.ano % 100 != 0);
```


I need to fix this somehow :dancers:


What's the point? What's the profit? What's the purpose?


Hey guys, please suggest me what are the changes you want in this application?


Hey neobuddy! Does this require any updates scheduler side?


test commit for no reason :)


I used a while loop and a for loop, assuming you're talking about the bisection search that you were asking about on the forums.

I believe the issue with your code is that you also need to change the upper_bound if rem_balance < 0, you need to remove rem_balance = balance from your if statements and move it so it looks like below, and finally you need to compare the abs(rem_balance) to some epsilon. There is no guarantee that rem_balance will ever go negative or equal 0.

while abs(rem_balance) > epsilon: # epsilon = 0.01
    rem_balance = balance

You need rem_balance there because once you add your if rem_balance < 0 statement as well, you'd be setting rem_balance to balance before you ever check that while statement and you'd loop forever.


I want to play some Space Invaders, so hurry up


Thanks for correcting this.


글자가 폰 기종에 따라 잘려서 내려가는거 막으려고 글자 크기를 줄였습니다.


this should be a ForeignField


for (int i=0; i<5; i++){
  if (i ==3){
     continue;
}
 System.out.println(i);
}


Not quite sure how some of the error codes should be handled.


Very nice, congrats :)

Finally now we can see SF giving more value to outposts reflected in its playstyle like other top engines.

This test proved its worthiness. :)


Hierdie is fine lyk nes die voorbeelde wat hulle vir ons gegee het.


Should suffice just to write "Gets the bagPartFactory".


> @param encoding Checks for the contents of the manifest file or a file name.

Doesn't seem right; most likely specifies the encoding of the data in the `InputStream`.


We should probably have a STRINGIZE macro that simplifies this.


please do not remove that comma.


o return é usado para parar a execução do metodo corrente e transferir o controle para o metodo que o chamou.


@vpodzime  lookup added with 1293bad86362dc988731fc6110f4cb11ba6cc8e6


For testing purposes. If we like this approach, we'll move these into a more permanent place.


> Creates a ManifestWriter that writes data to other files.
> @param out The OutputStream that writes to a file.

More correct to say it writes the manifest out to the input stream.


I'm jealous of the default elixir printer, it actually uses new lines and indentation :).

I'm working on pretty-printer next because it's difficult to see where things go wrong when tests fail. It's shouldn't be that different from JSON pretty printer, which there are many.


Repositories for work done at TCNJ and SBU.

Thanks!


TeamCity pull requests :: netty [Build 223](https://garage.netty.io/teamcity/viewLog.html?buildId=1480&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5197, ignored: 21 Build time: 00:15:41


I don't think this is correct.


Ok... I just modified `deployQZ` as follows:

```diff
function deployQZ(host) {
+    if (host) {
+        qzConfig.uri = host;
+    }
+
    console.log(WebSocket);
...
```

With no other changes, and unsecure connections come through without any issues.

Next, I'll test my luck with HTTPS.


I've added the missing dependencies to the Maven WOCommunity repository. Everyone should be able to build WOJRebel module without problems again. :)


I think I commented on this elsewhere, but this method more likely creates a `Manifest` object from the given `BagFile`.


I know that's not of your code, but what do you think, the use of `strval` looks fishy here, doesn't it?


Do we really need this optional parameter? Does not look sexy tbh :-)


Nice to hear that, thnx, yes true that it will enter endgames with more pawns, so its better to give higher value to the pawn structure in endgames, great job.


This method doesn't change the behavior of getting game rules; it only replaces the calls in this method. I assume there would be only one call.


It is like gcc and include paths, the path(ex: package/stage/machine/usr/includes) is set from from the recipe which include it(native:bison/cross:gcc) . gcc use class for that. But in our case we can't because in the case where native:bison is provided by the system we should not override the default value.


Typo - ```createLeauge```.


+1

I think thats a good idea, the details of establishing a connection (ie. handshake, protocols, etc) can be moved down into the remoting package and callers can have a simple interface to work with.


Congrats ! It was a good idea to retry your first idea with a higher bonus !


Yes, we will figure out a way to unify these shaders into some useful mechanism so that in future we might be able to do customer shaders if possible at all.


More correct to say "reads *a* bagit.txt" instead of from.


@smischke I'd like to get rid of this `bool` as well. I replaced the `#if def` with a bool, so I could control it without having to have a custom compiled version of Helix.

Can we just add the deferred render techniques to `DefaultRenderTechniquesManager` and `DefaultEffectsManager`? If we don't do this, and we don't have some sort of flag here, the third-party's `RenderTechniquesManager` is always going to have to include defferred techniques.


Awesome job! One thing we happened to notice, you're using jQuery.  We actually have support for bundling jQuery into the build of the project, to prevent the copy you install from interfering with the user's page.  You can see an example of how it works here: https://github.com/EagerApps/rrssb/blob/master/install.json#L76-L85.  Please feel free to ping us with any questions.


Why did you force set SpongeAPI version ?


@GokhanTurunc it would be better if you move this if check and do that stuff in line [42](https://github.com/GokhanTurunc/kd/commit/5c0a9a433df9b169267173768ea5c873947ae86d?diff=unified#diff-70bebb45f5ec0301b2348baa293ec680R42) in a `click` callback.


xcode7 (mbadams5) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2313 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.47%25-yellow.svg)


Doc for this doesn't look right.  Please verify with the implementation, but sounds like this creates a `BagItTxt` _from_ the given `bagFile`


If it's an HRESULT, you have the SUCCEEDED and FAILED macros to see if it worked correctly. I put a macro in to assert it with decent information. Do something like ,"if (FAILED(retCode)) AssertWindowsHRESULT(retCode);" If its a GetLastError error code, then check for NULL. If the object is NULL, just call the macro AssertWindowsError(). That will automatically lookup the error string for you.


Hi,

nice commit, but what exactly does it bring compared to doing

    bm -t | grep TRE

It is shorter but is length the only issue?


Same typo here.


Oh, and you might want to tune this at the same time
https://github.com/jcalovski/Stockfish/commit/bdc64ab01c3872b03556ad08175f02d0ad625329
the new outpost values are interesting, (with the new <=RANK_6)
but I'm quite sure it is too much for Bishops.


Let us ignore unknown properties.
@JsonIgnoreProperties(ignoreUnknown = true)
so that the json deserialization doesn't brake for every new property that gets added?
I see that this property is not used internally by camel.


To force my IDE to update the SpongeAPI dependency, because it wouldn't do it itself... :/


This is still incorrect...can't say "Gets a `BagFile`" since its return type is clearly `File` and not `BagFile`.  Like always, verify with the implementation, but this probably returns a `File` **points** to the bag itself.


Seems like this broke qemu: https://hydra.nixos.org/build/26072978


Reviewed


Related to above.  This is probably to set the `File` object that points to the bag.


Not a tag, but all tags.


Why is this `&=` based on `branchPCSize`? It seems it should be outside the `if` on line 331.


Merged here right now.


Ah thanks! I missed this one.


TeamCity pull requests :: netty [Build 224](https://garage.netty.io/teamcity/viewLog.html?buildId=1481&buildTypeId=pulls_netty&guest=1) is now running


Hi :) it's interesting what happens when you set step = 211 here as well and
unused = i * step % arrayToSum.length
sum += arrayToSum[i]


this is great!! maybe use:
```
ipfs://ipfs/QmcdXYJp6e4zNuimuGeWPwNMHQdxuqWmKx7NhZofQ1nw2V
```
for now, but may switch to `fs://ipfs/QmcdXYJp6e4zNuimuGeWPwNMHQdxuqWmKx7NhZofQ1nw2V` in the future. see the discussion on https://github.com/ipfs/go-ipfs/issues/1678


I would like some help on setting the install.json file. I'm trying the preview installation and I can't make it select anything, also typing CSS selectors does not work. It is probably a problem with my install.json.


What is the point of this change?
It's AS3 where Types are welcome.


Seems to fixed the problem. Nice fix! <bug, feature


I'm not a huge fan of singeltons, but I agree that this should just work "out of the box" . See my other comment regarding putting deferred techniques into the `DefaultRenderTechniqueManager`.


Hey buddy, sorry about that, it was a bit of a mess w/ creating and destroying some repos, but all should be well. Resolved with commit 063914f4d81a7b7817f694c60280220f8eef1292

Miss you @bsouthga Let's grab a beer soon?


Also, me description, tagline and logo are not great. So, if you think of something better, please feel free to replace mine.


I think this is complicating things for a very small subset of users (people using external gyros, and then those wanting a different rate in Acro).  But OK.

Is it worth renaming _ext_gyro_gain to _ext_gyro_gain_std or *_normal so that it's more clear which is which in the code?  Probably don't need to change the param name, but just in the code.


This really is a belt and suspenders test: `scaling` is an output of  `match(scaling, tab) - 1` which cannot be negative -- you could drop `scaling > 0` part of `if()`.


not exactly, if `click` handler on the close icon's view is fired that means that we don't need any other check there.


I find a bit strange to 'test' and continue on something that should never happen
```
len(mocked_processes) == 0 and mname != system.processes.number
```


In my head, ScoreKeeper primarily manages the high score list rather than handling the current game score. Technically, the round is the that is being tracked in the SimonController class is the same as the player's score.

My plan for LoadScores() (that I failed to leave a comment on...) was that it would load in the scores from an outside source that stores them when the application isn't running. It can just be a text file since you guys are doing that in Advanced Java right now. Ideally, the scores would be ordered before they were written out so that they don't need to be rearranged when loaded in. But yes, we should have a method for ordering the scores.


there are no autorelease scripts


xcode7 (mbadams5) - 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)


I can't find that method on the trait?


пересохрани в UTF-8, пожалуйста


lear to code


Ah cool. I'll use this in the future.


On the other hand, the result of `match()` can be `NA` so that the test perhaps should be `if (!is.na(scaling) && correlation || hill)`.


+ ugrade to etcd 2.2 (forgot that it wasn't pushed)


oops spelling error!


oh sorry. but i think the `!== ''` test is better than the `strlen()`.
To fix this i think it's better to add `isset() && ...`


Will this work with the imports at [server/config.go](https://github.com/hpcloud/stackato-go/blob/16d83d929df8c685adaa6e3d43641ee1306c3ab2/server/config.go#L4)?


Leave as pointer. The return value should be *lib.Config


I think it would make more sense to do the motors.set_acro_tail(true) in the heli_acro_init function so that it only runs once, rather than running all the time in heli_acro_run?  I realize it barely takes any time at all.  But historically we have tried to make the code efficient.


Leave as pointer. The return value should be *gcp.GoogleCloudPrint


обычно, не стесняются и пишут прямо в теле класса

    virtual ~IBaseExprModel() {}

а не

    virtual ~IBaseExprModel() = 0;


Cool :)
I hope it works now even for a blank clone (was apparently more difficult than I thought)


74564912Issue4347BBRadConvHW (wfbuhl) - x86_64-MacOS-10.9-clang: OK (1871 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.84%25-yellow.svg)


Argument should be pointer *lib.Config


@straleyb I was able to test the audio derivatives and they are created successfully and rendered properly in the webpage as well.

I was also looking at the test coverage and I think that we can improve the overall coverage of the project if we find a way to test the invalid part of the update action https://github.com/OregonDigital/oregondigital_2/blob/master/app/controllers/admin/facets_controller.rb#L20-L22 It seems like it coveralls doesn't hit this part.


Awesome! Thanks! Definitely down for :beers:, maybe next week?


Reviewed


The following two statements should be equal:
```
scaling <- match.arg(scaling)
scaling <- match(scaling, tab) - 1
```
and
```
scaling <- pmatch(scaling, tab) - 1
```


You could convert it to `xx.yy.zz+rr` with a regex.


@arjunguha: Just found this. This doesn't make sense to me. Could you explain?


Good


утечка


@olmari See #39.


Looks good. I would think about adding in an initialize method where you can store state in instance variables. You can store different type of inputs from the user this way.


@Mook-as for some reason it still works with that import statement, but updated it here anyways https://github.com/hpcloud/stackato-go/commit/f7a3ecb69ad6cd638bc64c901dc6a5c69b083adf#diff-a8a8090b66912f265b621fd23193869cR10


You can use ALTER TABLE if the DB contain user generated content to Ad such columns


the installer finished with no errors, but still when i perform roundcube/installer check 1 module would still fail.  Spent 2 days figuring this out, best choice is to download the complete package of RoundCube. .  Then the folders from Vendors just copy the missing files/folders. and just fix the broken paths in vendor/composer/autoload_classmap.php

	'Net_SMTP' => $vendorDir . '/pear-pear.php.net/Net_SMTP/Net/SMTP.php',
	'Net_LDAP3' => $vendorDir . '/kolab/Net_LDAP3/lib/Net/LDAP3.php',
    'Net_LDAP3_Result' => $vendorDir . '/kolab/Net_LDAP3/lib/Net/LDAP3/Result.php',

I was trying to install it on windows and centos, 1 worked just fine, but the other OS didn't.
So this is how i solved it. hope it helps others.


Ah, great, that's what I hoped. Does it also work without the bogus buffered stream instance in the picture? Ie. just `res.pipe(res);`?


nice thanks


I do not understand this part. What is running used for ? Why not just close the channel ?


FYI, the insecure `ws://` comes up as `null` for me when I try this.


TeamCity pull requests :: netty [Build 224](https://garage.netty.io/teamcity/viewLog.html?buildId=1481&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5197, ignored: 21 Build time: 00:14:51


good catch! :-)


This commit breaks the compatibility of pycoin with most of the cloud infrastructures such as Appengine because of the usage of ctypes library: https://cloud.google.com/appengine/kb/libraries


Wow. I don't understand it either. My intuition is that this is completely wrong and any application that reads packets at the controller would break (e.g., mac-learning). I think our experiments use `opt_to_table`. Let me try to test this.


103404908-AddWarningWhenScheduleTimestepsDoNotAlighn (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1880 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-100.0%25-green.svg)


General level comments as to what it does ?


Hard to tell what is changed here. It seems like indenting got updated and caused everything to appear as modified. Are there specific changes in here we should be aware of?


I cannot replicate this, do you have a stack trace by a chance?


@neitsab Indeed, thank you for clarification!


Cool. I agree these bindings are better up one level.


Not all of them though. I think the Prince of Bel Air examples are fine. But sorting out odd numbers is pretty synthetic.


This bit me as well in Canvas Spaces. When testing with kipling, it kept adding an old deleted account.


you didn't do what darin said here... `return Object.keys(this).length === 0`


This was noticed and reverted already on both master and release-15.09. 5cd5fe376


added a test result document


I added here these lines to ignore the .envrc file and a local folder for packages so those using direnv could setup their local environment as they prefer.


Unused import of meta.


Let's document the return (see os_keypair for an example)


There is no config file yet. It's just command line arguments.
knxd.cnf is parsed by systemd. So you'd need to systemctl "daemon-reload" plus "restart knxd".

You want to implement .ini file parsing, be my guest … there's already an open issue for it.


Make sure that your `/etc/ldap/ldap.conf` really belongs to nss_ldap package, not OpenLDAP or something else.

btw What do you mean by “modern distributions?!” Debian-based distos?


Same company? :)


an SEH is correct if you pronounce it as ess-eee-ach :)


yes sorry.


saw approval from fadushin
at https://github.com/basho/yokozuna/commit/5871aa49793070d64b6ce944a83d01646fedd9d7


merging basho/yokozuna/hotfix/2.0.6-2012.08.25-hash-needs-to-match-kv = 5871aa49 into borshop-integration-555-hotfix/2.0.6-2012.08.25-hash-needs-to-match-kv


basho/yokozuna/hotfix/2.0.6-2012.08.25-hash-needs-to-match-kv = 5871aa49 merged ok, testing candidate = 815288df


розбиває текст на окремі слова, кожне слово записується в новому рядку за алфавітом


X 2WWza Z,Z,


Ugh, @florapdx fixme plz


@BenWhitehead Let me know what you think of this version.


https://github.com/choroba/syntactic-construct/issues/8 fixed.


103353562-Issue-when-using-different-heating-and-cooling-airflow-rates-with-Airflow-Network-#5134 (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: 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) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.24%25-green.svg)


Yea, it comes up null for me as well. But since I only use WSS I didn't bother to look into why.


Cool tool. Whoever showed that to you must be pretty cool as well.


I definitely agree, and my workflow is basically the same as what you want it I think, but what I was trying to say is that also the layout needs to be done semantically on dedicated branches without touching on other things if you get what I mean?


This is very nice.


I think this is just a typo, but this comment could make it clearer that the radial separation is the *only* thing the statistics are calculated on.  The function name doesn't really help with this either, but I'm not asking that it be renamed.


:beers:


Oops. My bad.


Looks good.


Is it possible to assert on `psutil.AccessDenied` exception only ?


I'd suggest using an actual ellipsis character to save room: "\u2026"
* If you wanted, you could put it in strings.xml as …
* you could increase the display length to 5 or 6 characters if it saves enough room


You can't have (branchPC & 1 && branchPCSize == 0)


I was just really excited about filtering :sunglasses:


I'd suggest using an actual ellipsis character to save room: "\u2026"
* If you wanted, you could put it in strings.xml as I'd suggest using an actual ellipsis character to save room: "\u2026"

* If you wanted, you could put it in strings.xml as & # 8230;
* you could increase the display length to 5 or 6 characters if it saves enough room


this line is not needed...


belongs to #48


Reviewed

I wish these strings could be a little more descriptive, but I guess we're limited to string length.


Why not `make deps` ?


Reviewed


not sure. I'm currently in the process of sanitizing code, but certainly eieio is becoming a compatibility issue.
I guess you're welcome to try :)


Hey!
Thanks for the pull request.
Can you provide a configuration that will cause the error?
I have tried with
{minValue:-10, maxValue:9, minLength:2}
but with that, I had no problem typing "-10" in the field.
How can I recreate the error?

Thanks in advance.


Change this line to these two lines:
for(var i = 0; i < TAVERN_POSSIBLE_RANDOM_ENCOUNTERS.length; i++) {
  var encounter = TAVERN_POSSIBLE_RANDOM_ENCOUNTERS[i];


Opps this commit message should have said "Don't ***hide*** recent search terms if only spaces entered as search term."


Link some to commits


Which is actually part of the default make target.


@tylerstauss not currently


I take that back, apparently there's an android:ellipsize="end" that you can just throw on the view and then you wouldn't have to do any truncating.  Could you try that?


fixes #82


74564912Issue4347BBRadConvHW (wfbuhl) - i386-Windows-7-VisualStudio-12: OK (1877 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.84%25-yellow.svg)