> s8 is just inconvenient and unclear

![](http://www.ls-rp.net/uploads/profile/photo-1419.jpg?_r=1436277003)

---

While `sizeof(char) == 1`, no one guarantees you that:
1. `std::numeric_limits<char>::is_signed == true`
2. `CHAR_BIT == 8`.

This is stated in the PPU ABI, and the only type that can possibly represent would be something like `s8`, `S8`, `SI8`, `fancy_signed_integer_with_eight_bits_t` or whatever you wanna call it. Then again, I just replied because I got a mention-alert and I love nit-picking, it's not like my opinion matters nor I expect any patches. :-)


it's provided by the `org.apache.hbase:hbase-common` artifact, which is also defined in the root pom.

For future reference, I use a combination of http://search.maven.org/ and http://grepcode.com/ to locate jars containing classes and versions. For instance, http://grepcode.com/file/repo1.maven.org/maven2/org.apache.hbase/hbase-common/1.0.0/org/apache/hadoop/hbase/HBaseConfiguration.java


ha denkt wemer mal en neue account hinzufüegt isches eifacher das nur ah eim ort z'mache (ufem server) anstatt uf zwei orte (server en label adde und front-end en label übersetzig hinzuefüege) ;)


The SPI is the boundary between Presto and plugins, not an SDK for building plugins. DecodableColumnHandle is an implementation detail for plugins that can be modeled that way, so it doesn't really belong in the interface layer.


There are duplicate classes error in the maven audit step I believe without this exclude.


ok


Thanks


What do you think @halfak ?


Need to handle PCRE_ERROR_BADUTF8 as well for bad data ( -> return false)


*seaweed

Thanks for doing this, now I can take this out of my minetweaker scripts :)


change to: throw Regular_Expression_Error(rc);


Ok. Cause that's what broke our ability to remove breakpoints (as far as my initial investigation has showed). :P


Added missing assert.expect() calls


Fixed in f4a4a4db25191e12c5b091afb67500e85a7166e0 (see TheRMainFunctionDebugger)


```
$_POST = array_map(function ($item) {
  return StringUtil::convertEncoding(strtoupper($_POST['charset']), 'UTF-8', $item);
}, $_POST)
```
… or at least `unset($value)` after the loop to avoid accidentally smashing your array by reusing `$value` later for something else.


@dmick, I'll try to clarify this.  The use case here is that ``common_packages``would apply to all versions or possibly just all minor versions of a major version.  For example, ``vars/ubuntu.yml`` lists ``common_packages`` that we need to install across all versions of ubuntu.  And then the version specific files in ``vars/`` define the differences between the distro version with the ``packages`` var.

I know, it's a bit confusing.


This is a very nice feature.
Thank you.


NULL is better. min(L, K) was fine because they are standard stuff in SSA world


Put a comment to explain that the asymmetry here is intentional


done


Why do you need to do this for arrays? In an array literal, you are already taking a union type of all the elements, so the deduplication will happen before you actually create the array type.


I would like to leave this test since placeholder is an optional property. The component could be used with or without passing placeholder.


sdawdwd


where are you use incoming text parameter?


WinRM usage had a small bug in Vagrant (See [#5970](https://github.com/mitchellh/vagrant/issues/5970))
This is present in v1.7.3
Fixed by PR [#5971](https://github.com/mitchellh/vagrant/pull/5971)
Merged into Vagrant [master](https://github.com/mitchellh/vagrant/tree/f540a301b6be6d1ca1d81cba42daf8d40d7a9300)


Now I don't know who's advice to use :(


nice


@jfoug there's `git commit --author=` if you commit changes authored by someone else.


yeah, it's true )


Ahhh.. .that makes sense. Thanks for the explanation


I would like to leave this test since placeholder is an optional property. The component could be used with or without passing placeholder and we are testing different combinations, endDate with placeholder versus startSate with placeholder in the other test.


Предполагается, что список потребуется для вложенных функций.
Пример:

```R
abc <- function() {
    def <- function() {}
}
```

В таком случае для abc - TheRFunction(listOf("abc")), def - TheRFunction(listOf("abc", "def")).


thnks.
my bad.


Really we need it more for tuple types.


Fixed a minor typo.


@MWDelaney I don't think this will work, because `_load_front_end()` actually hooks into the `wp_enqueue_scripts` hook.


Do we really need separate function for "-strcmp()"?))


I think you need to arrange for this field to be written and read in `saveToPath` and `_loadUnibuildsFromPath`. (Trade-off between "having to write this stuff over and over" vs "having the actual data in the tool be implicitly defined by what's on disk instead of explicit".)

(A refactoring I intended to do during this project but failed was to take this "resource" random-JS-object and change it into a classy thing, but failed. That would make this easier.)


You should link to the CL URL, not to one specific patch. Looking at the CL, it's not clear to me if you're just supposed to remove that code or move it somewhere else.


Is this actually still true now that we only remove stuff that can contain null and undefined?


better use arr.partition.each_with_index { |el, i| i.odd?} - more clear. Bad idea use find_index in block which take condition.


Great idea. The tool needs a lot more helper functions and I am often just guilty of making giant functions gianter. Thanks for this improvement.


so looooooooooong line (((( look into #map method.


`if (candidate === type || isTypeDuplicateOf(candidate, type))`


done


I think this is not correct, because "css" is actually handled by a SourceProcessor in the `meteor` package.  (But we are OK with assuming that this is a trivial SourceProcessor and that we can legacy-ify it.)

Maybe it makes more sense to, instead of tagging "hasSourceProcessors", instead tag with something like "builtInJsProcessor" or something?  So then the logic is "unsafe means a source that is not css and that is not builtInJsProcessor"?


I'd call this variable "remainingLines" instead, to go along with "lines"


Same goes for isTypeSubtypeOfSomeType


Actually, never mind.


The default impl should use the container manager


Betty,

Right, I don't expect anything to change either. I just wanted to make sure. If you are certain the changes are isolated to tau then I will go ahead and merge the pull request.

Cheers,
Alexx

Sent from my iPhone

> On Jul 15, 2015, at 12:26 PM, Calpas <notifications@github.com> wrote:
>
> Hi Alexx,
>
> sorry but I did not try making ak4pf jets. Could you please explain me
> what you mean by that?
> I just add the tau information so I would not expect it to affect the
> jet. Is that the reason why
> you commented the tau part?
>
> Cheers,
> Betty
>
> Le 15/07/2015 17:22, Alexx Perloff a écrit :
> >
> > This all looks fine to me. Did you try making ak4pf jets, or something
> > like that, to make sure the changes don't affect the other jet
> > collections?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/Calpas/JetMETAnalysis/commit/a0ba2b025829613fd6575a41d1b2c2dd6e42680b#commitcomment-12180239>.
> >
>
> —
> Reply to this email directly or view it on GitHub.
>


Probably @Nekotekina's advice as he is the de-facto mantainer. I'm just an outsider, you know, like those grandpa's spending their time near construction zones, just criticizing and annoying workers with "*back in my day*" stories. ;-)


Hehe, yeah I want to add support for 3.
But I know there'll be a bit of work involved. I think it'll primarily be around imports since I'm only using built-ins.


Hi Marco,
you removed gui client packages from repo, but problem is gui clients cannot be downloaded now. Packages are referenced from README.md in section Getting GUI Client. What is the reason to removing packages?


The same happens a few more times in the PR.


Coding style:
* Missing empty line after this one.
* "views" and "delegate" are separate words, so the variable must be called `views_delegate`.


We need to consider time ranges also. We hadn't decided how to specify them, do we do
start_week, end_week, by week number?
What is the max range, and how would one request the max range?
The prototype collected 52/53 weeks including the current. Should we be storing previous years (as a year, not weeks)?


I've added some inline comments for you to keep in mind, but the whole commit is not necessary, as I'll explain in 2f43d27b9c2ecd596446f1b271d647368c413cb9.


LGTM - thanks.


why you use flex, it is not crossBrowser solution


This console log is deleted in a later commit


The Exit_point constant now is a Hash which "n" value is the incremental number of the phase.


Not necessarily now, but some time in the future we could setup integration tests using a Geoserver docker image (similar to how it is done in `AbstractDBTest` - though before that we would have to generalize what @sheikhsameer1609 created there for the docker image configuration).
Would these tests require a Geoserver instance with a specific configuration or even a database running?


the whole class is deleted now


I mean, we can add some checks, conditions etc..


Ups :)


@isuftin


@AlexAltea Okey dokey!


done


解決することが出来ました！ありがとうございます！ :smile:


@marineam @crawford can you be more specific about the testing with ovftool that you did? Which version did you use? Great would be the output of the command "ovftool -v", because it will print the build number.


i think hardcoded color is not good, maybe you throw styles from snackbarWrapper from our component ?


Ooh that's nice!


Is the above better @isuftin?


The issue this fix fixed in navClear is now back in ionic 1.0.0 and 1.0.1 when navClear was removed. So if you try to go to the same page you're already on, you will clear the the history and then the next page you open will be the root, which is not what is expected.


bad logic codestyle


This makes it impossible to start the summary with proper names that begin with a lower case letter, like "ls lists files".

Apart from that, I would move mere convention checks to Policy or a new module. Port is already fairly big without those.


The idea behind `SetInstance()` is wrong.

You need to look at the entire upstream commit what is going on here. It is easy to see how there were several places setting `views_delegate` before which have been converted, and upstream did not need `SetInstance()` for that. In fact, if you introduce `SetInstance()` you are reintroducing the very issue that the upstream commit was trying to fix (letting `ViewsDelegate` users freely change the instance, as the commit message says).

Take a look at the changes to, for example, `chrome/browser/ui/views/chrome_browser_main_extra_parts_views.cc` and `ash/shell/content_client/shell_browser_main_parts.cc` to see what you need to do. Additionally, see how `ViewDelegate`'s constructor is now responsible for setting `views_delegate`.


@lukespragg : "Good luck with the project though, you seem to have a talented crew of developers. :)"
Cheers! :+1:


oh i see there all styles hardcoded ;)


Fixed in 4e1e33d.


Trivial patch committed directly.


Steve,
Your suggestions of course worked.  I did not recognize I was overriding an ActiveRecord Model even though the top of the Game class has the following:  Class Game < ActiveRecord::Base.  What I was doing was treating this ActiveRecord instance as just a plain Ruby class, thinking I could initialize as I did with my OldMaid Game class.


remove debugging :)


fair enough


Thinking about this example again (thought I've modified it by switching instances of Foo and Bar):
```ts
class Foo { x }
class Bar extends Foo { y }
var z = [{ a: new Foo, b: new Foo }, { a: undefined, b: new Bar }, { a: new Bar, b: undefined }];
```
The type generated by this deduplication (after widening) will be `{ a: Foo, b: Foo} | { a: any, b: Bar } | { a: Bar, b: any }`. But that means when you retrieve `a` and `b` from the union type, you will get `any` for both. This seems worse than getting Foo, which you would have gotten from subtype reduction. And even when you perform subtype reduction later (in getApparentType), it will give you `{ a: any; b: any }`. This does not seem correct.


look into #inject method, and try rewrite it without taking initial element and block.


here some fix for error you will have https://github.com/Resurrected-Llama-Kernel/android_kernel_lge_msm8974/commit/26db27c644b1398a3c45375694556a6aa456d68b


> This makes it impossible to start the summary with proper names that begin with a lower case letter, like "ls lists files".

`ls` is in the coreutils package. And I thought it was a convention by now to not have the SUMMARY start with the package name (e.g. for ls it would be `A small tool to list files`.)

> Apart from that, I would move mere convention checks to Policy or a new module. Port is already fairly big without those.

My Python isn't good enough to do that, unfortunately.


Ah, yeah..


Oh. Sorry. I've forgotten about it actually.  I can test it again on Gazebo 5 later today to see if it's still needed and then do a PR.


Good job.


done


sqlite_timeseries (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (723 of 723 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-7.0%25-green.svg)


Fixed by 5cf1f26d.


let's postpone


Not sure if it's too bandwidth intensive or otherwise unfeasible but would it be a good idea to have a master "LIVE" list located on say github or elsewhere, where SR can call in hourly or daily to get a updated list, it would stop the need to keep pushing out updates for this if one comes down, and would allow for you to just pop a new one into the list if you have a replacement.

Same could apply for other providers too like Search providers, a way for you to add or remove new providers at will if one goes down, needs updating, or you find new ones to code for.

Just a thought I had, however you tend to think in a lot broader terms then I do anyways since you're waist deep in coding everyday and supporting so many different systems/OS's.


а почему тут elif, а не else?


All the commented issues are fixed (jetty fix need to be done)


@fraserxu Since the `npmignore` file overrides the `gitignore` file, the file inside `dist` will still exist in the published npm repo, and not in the git repo. This way people using the library in their projects would be using the compiled ES5 version in `dist` while people checking out this repo on github would deal with the ES6 version.

Compiled files generally don't belong in a git repo because they're a waste of space. Using `npmignore` allows us to publish these compiled files, and keep our git repos sane.

Read more: https://docs.npmjs.com/misc/developers#keeping-files-out-of-your-package


I still do that sometimes too, and it's always a pain to track down.


потому что тогда он дважды обработает сообщение


We should actually build two `StopTimeEvent`s here, one for arrival and one for departure time (but only if they exist).  So, the whole FOR loop should look something like this:

~~~
for (TimepointPredictionRecord tpr: timepointPredictions) {
           StopTimeUpdate.Builder stopTimeUpdate = StopTimeUpdate.newBuilder();
           stopTimeUpdate.setStopId(AgencyAndId.convertToString(tpr.getTimepointId()));
           stopTimeUpdate.setScheduleRelationship(com.google.transit.realtime.GtfsRealtime.TripUpdate.StopTimeUpdate.ScheduleRelationship.SCHEDULED);

           if (tpr.getTimepointPredictedArrivalTime() != null) {
             StopTimeEvent.Builder arrivalStopTimeEvent = StopTimeEvent.newBuilder();
             arrivalStopTimeEvent.setTime(tpr.getTimepointPredictedArrivalTime());
             stopTimeUpdate.setArrival(arrivalStopTimeEvent);
           }

           if (tpr.getTimepointPredictedDepartureTime() != null) {
             StopTimeEvent.Builder departureStopTimeEvent = StopTimeEvent.newBuilder();
             departureStopTimeEvent.setTime(tpr.getTimepointPredictedDepartureTime());
             stopTimeUpdate.setDeparture(departureStopTimeEvent);
           }

           tripUpdate.addStopTimeUpdate(stopTimeUpdate);
        }
~~~

Please double-check to make sure I got all the arrival/departure references right :).


@nedbat It looks like you added this app to `INSTALLED_APPS` twice, from two separate commits:
- https://github.com/edx/edx-platform/commit/4d237d0d15ce70b5cc355d60f7b1833b46b8b1d8
- https://github.com/edx/edx-platform/commit/95b374d7510884f88342d075b83cf62a8eaed7b0


tsasdasd


Magnificent


test scope


Yup I think that should work. Your variable naming is weird since youre making a link, not a button.


I like the last one :-)


Since 'hs' is a NetNamespace object, won't hs.NetID already be a uint and thus this check is useless?  Or do I misunderstand how the JSON <-> NetNamespace conversion happens?


It's attached to the global window context as `DeepDiff`, not `diff`: https://github.com/flitbit/diff/blob/master/index.js#L17


Did this work?


I'm not sure if getting two selection callbacks with valid IDs without receiving the callback with an invalid id in between would be an error; this code would be fine with that happening, though. So you're right, that DCHECK is unnecessary; I will remove it. What would not be fine would be for the selection callback to occur twice with valid ids without the data offer callback occurring in between. With a valid id, the selection callback _must_ only occur after the data offer callback, so `self->new_offer_` _must_ be set. So I will keep that DCHECK.

An invalid id just means "delete your selection offer." I would not expect the selection callback to be called with invalid id immediately after the data offer callback, which is why I added it, but that does not matter for the correct functioning of the code. So my DCHECK in this case is also unnecessary. I'll remove it.

Note: I only actually run with DCHECKs enforced in DesktopDragDropClientWayland.cc, since I never managed to compile a debug build without errors, unfortunately. In that file I `#undef DCHECK` and `#define DCHECK(n) CHECK(n)`. I know this isn't great, but better than nothing.


sure no problem .. did you change it back? I can do it with next pull request ...


Should the first arg be optional like the rest?


I'd love to do this! but it makes the query a bit tough. I think we'll have to do a separate query for the icons (and eventually batch up all those queries). Its something I've wanted to tackle though. Want to try it here? You'll have to be careful that the views you're changing aren't recycled underneath you (since you'll have to do the query for icons asynchronously).

imageView.sd_setImage() isn't in our code, its in SDWebImage, so we'd either need to override or extend it. I think I'd rather just put a helper in the panel with something like imageview.sd_setImage(findBestFitUrl(icons), placeholder: ...)

Since the top sites and bookmarks and history panel all sorta need to do this, I'd put it in a utility class if you can.


#what hell is this?


Seems reasonable


This method probably isn't private.


@barbeau I was also working on this part. I am testing right now.


missing `#` in link


I understand, but with 'force login' *un-ticked* it is still forcing logins because that if statement will only return true if the database entry for cas_force_login doesn't exist (i.e. it has never been ticked before previously) due to the default from getAppValue.

As it stands if cas_foce_login is in the database, regardless of value, it does not disable the force login (even if the database value is 'false' 'no' '0' etc as 'false' !== false (=== does a type check and 'false' is a string).

If this is intentional then maybe you wish to change the untick function in the admin section to delete the option instead of just setting the database value to 0.


I think the first comma in the last sentence can be removed?


@horkhe depends on a few other (minor) unmerged changed, but this is a rough prototype of not blocking when a particular consumer isn't being drained (or is even just slow).


ejabberd is not translated to those languages. From what I know, the other languages don't need RTL
https://www.ejabberd.im/localization


Restored to the previous commit state.


return is't necessary in ruby.


It did, works on Safari, can't seem to get my local version on IE like I did before so haven't tested on IE


valido


Nuke.


No worry. I am back in a windows environment. I was having trouble with
display drivers in Windows 8.1 which was a large reason to switch. My
brother told me to try out Windows 10 since he had a bit of the same issues
and 10 resolved them so I did and it worked. So now I have a working Git
gui again. This merge was because I was pushing to my own repo so I didn't
clutter or overwrite commits.

Probably need to pull changes back into it, although I won't be using mine
for regular commits.
On Jul 15, 2015 12:59 PM, "Brandon Nozaki Miller" <notifications@github.com>
wrote:

> have you tried developing with brackets? it has a decent git plugin.
> Commandline is pretty easy too ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RIAEvangelist/node-dominos-pizza-api/commit/ea7e7a8753817e80f24939772af5ee00d6ee369e#commitcomment-12185560>
> .
>


I don't see any other way


:) I have tested that today, but it is in a modified copy of the plugin, and thus couldn't do PR. But it is supposed to work on latest gazebo5 release. My problem is: is this just a workaround or it is actually the proper way work with gazebo5?


GRAPHENE_MAX_UNDO_HISTORY isn't actually the max undo size then but rather (the actual max undo size + 1 - GRAPHENE_MIN_UNDO_HISTORY). While it doesn't really make much of a difference for GRAPHENE_MAX_UNDO_HISTORY >> GRAPHENE_MIN_UNDO_HISTORY, I think that either the code should be changed to reflect the name or the name of the constant should be changed to reflect the code.

It would probably be easier to just change the code by setting this line (line 71) to
`_undo_db.set_max_size( _dgp.recently_missed_count );`
AND changing line 54 above to:
`else if( dgp.recently_missed_count > GRAPHENE_MIN_UNDO_HISTORY )`


You explained the motivation for ignore freshness during subtype reduction:
>It was necessary to ignore freshness because otherwise we'd infer the full precise type, which, when widened, would produce a reduced type to which the full type isn't assignable. Basically the opposite problem.

I don't understand how producing the full type makes the original type not assignable anymore. Can you give an example?

Also, please add tests for this part of the change.


:+1: for the quick commit!


@briandoconnor, @niall-byrne brought up a good point that we need to setup a special GitHub account to be used by the workflow to interact with GitHub. Putting personal account credentials out there is something should be avoid.


Не знал, что так можно arr.partition.each_with_index :)


so if i want print 98 from incoming text '98 bottles of beer was on the wall, 1 falls, 97 left.'


Removed in branch `vertical_scrollbars`


Ok.  Unfortunately, I cannot even access one of the machines that I need to test on right now.  Until I can access that machine (which is one of the machines that still uses Python 2.6), I don't want to consider merging this commit.


@oliverkurth our memories are a bit rusty but we'll try to retrace our steps once we have a chance.


nope, `export * from ..` is the intended syntax to show off here.


why are you linking this into your app?


find out #odd? and even? methods .
And be sure, what you need to print elements with even or odd indexes, not elements exactly.


just delete these


I use ext4 on a internal hdd. Both emc-permsort.xml should be identical. Otherwise a reboot would change the emc-permsort.xml without a logical reason. That would be very strange. But I can append a working backup, an actual running version and a emc-permsort.xml after a reboot if it realy differs in the openxta-thread in a tar file. That should save all character encodings.
http://www.xtrend-alliance.com/index.php/topic/1594-emc-vergisst-sortierreihenfolge-nach-neustart/

At the moment some timer are active. So i have to wait to make a reset. I think I could upload it tomorrow, because the running timers today are longer active than me ;-)


Why do we have this empty lop here?
@hernandcb


delete, replace with the reporting an error from mnt_context, check the api docs online.


Thanks @cagryInside!  A few comments inline, include a thing or two that needs to change.

Also, when running `mvn test` I'm seeing an error in what I believe are your added tests:

~~~
Running org.onebusaway.transit_data_federation.impl.blocks.ScheduledBlockLocatio
nLibraryTest
Tests run: 12, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.016 sec
Running org.onebusaway.transit_data_federation.impl.blocks.BlockStopTimeIndicesF
actoryTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.066 sec

Results :

Tests in error:
  testGetArrivalsAndDeparturesForStopInTimeRange03(org.onebusaway.transit_data_f
ederation.impl.ArrivalAndDepartureServiceImplTest)
  testGetArrivalsAndDeparturesForStopInTimeRange02(org.onebusaway.transit_data_f
ederation.impl.ArrivalAndDepartureServiceImplTest)
  testGetArrivalsAndDeparturesForStopInTimeRange01(org.onebusaway.transit_data_f
ederation.impl.ArrivalAndDepartureServiceImplTest)

Tests run: 96, Failures: 0, Errors: 3, Skipped: 0

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] onebusaway-application-modules ..................... SUCCESS [  0.020 s]
[INFO] onebusaway-core .................................... SUCCESS [  1.251 s]
[INFO] onebusaway-geospatial .............................. SUCCESS [  0.955 s]
[INFO] onebusaway-federations ............................. SUCCESS [ 31.859 s]
[INFO] onebusaway-realtime-api ............................ SUCCESS [  0.520 s]
[INFO] onebusaway-transit-data ............................ SUCCESS [  0.340 s]
[INFO] onebusaway-api-core ................................ SUCCESS [  0.313 s]
[INFO] onebusaway-container ............................... SUCCESS [  1.045 s]
[INFO] onebusaway-users ................................... SUCCESS [  3.436 s]
[INFO] onebusaway-geocoder ................................ SUCCESS [  1.981 s]
[INFO] onebusaway-presentation ............................ SUCCESS [  0.906 s]
[INFO] onebusaway-api-webapp .............................. SUCCESS [  1.277 s]
[INFO] onebusaway-phone ................................... SUCCESS [  0.970 s]
[INFO] onebusaway-phone-webapp ............................ SUCCESS [  0.390 s]
[INFO] onebusaway-sms-webapp .............................. SUCCESS [ 20.552 s]
[INFO] onebusaway-webapp .................................. SUCCESS [  0.985 s]
[INFO] onebusaway-transit-data-federation ................. FAILURE [  5.866 s]
[INFO] onebusaway-transit-data-federation-webapp .......... SKIPPED
[INFO] onebusaway-combined-webapp ......................... SKIPPED
[INFO] onebusaway-federations-webapp ...................... SKIPPED
[INFO] onebusaway-gtfs-hibernate-spring ................... SKIPPED
[INFO] onebusaway-gwt-common .............................. SKIPPED
[INFO] onebusaway-quickstart .............................. SKIPPED
[INFO] onebusaway-quickstart-common ....................... SKIPPED
[INFO] onebusaway-quickstart-bootstrap .................... SKIPPED
[INFO] onebusaway-quickstart-mains ........................ SKIPPED
[INFO] onebusaway-quickstart-webapp ....................... SKIPPED
[INFO] onebusaway-transit-data-federation-builder ......... SKIPPED
[INFO] onebusaway-quickstart-assembly ..................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
~~~


how get last element of array without using extra #length, #size methods? look to indexing of array elements.


path = argv[1]
target = arv[2]


just delete this header since it isn't used for anything right now.


@hernandcb @carlosmartinezmoreno
Just so you know, with the new Java version, we can use "switch-case" with strings. Someting to think about it when we upgrade to a recent version.


For example, we can check that a != NULL and b !=NULL or something else... I am not trying to force you do it. It's just my thoughts.


In single mode partscan isn't needed but do we need the autoclear flag?


Reducing the retries from 20 certainly makes sense... but maybe 3 or 5 is fair?


delete the debugging code


Oh, does this actually work? You don't have to put a `:` and a `pass` statement to not cause a syntax error in python?


May you please document the method?


As I wrote, "would", when Arabic, Farsi or Urdu translation be made available.


***R***ake task? :-D


Well done. :+1:


Isn't that a possible DoS on ruby versions before symbols are garbage collected?


Hi.
Here : http://pastebin.com/6TdTJ2KN
Thanks


look  to #map method of array.  Don't use unusual for statement  for ruby. There are a lot of Array#methods.


`size_t` is an unsigned type. However, if you're implying that it were better to use `size_t` instead of `unsigned`, then you're probably correct. I don't think it matters in practice due to both sizeof() operators returning low values.

I'm not sure about the other two cases below thou, I think I chose `unsigned` instead of `size_t` because I found that `certs->count` was of type `unsigned`. I suggest this still be checked once more.


Stop use return at the end of method ))


@carlosmartinezmoreno Please note that this is not a "General" message.
Please move it to the corresponding section.

@hernandcb The same applies for the partnerSave keys.


Who are TL and QA?


For any classes that implement `Serializable` that we modify, I believe we should also bump the `serialVersionUID`.  So, `serialVersionUID` should be set to `2L` for `TimepointPredictionRecord`.


I think we have a most updated version, don't you?
@hernandcb


See http://stackoverflow.com/questions/285793/what-is-a-serialversionuid-and-why-should-i-use-it.


What would you propose? »Hotkeys« is not German enough for me.


here on my system I see I2C_FUNC_I2C being defined by linux/i2c.h not i2c-dev.h, am I missing something here?


Not sure, but `master` is targeting Ruby 2.2+ so we should be fine?


yay! \o/


@barbeau Yes ```mvn test``` should fail with this patch. Current tests are looking for ```timepointPredictedTime```, I need to change them to test ```timepointPredictedArrivalTime``` and ```timepointPredictedDepartureTime```.


on more return ))


Ah, right, of course.


For has_failsafe to already be set, wouldn't someone have to specify failsafe packages twice? If they do that, it seems like we're in an undefined state and should blow up.


http://ruby-doc.org/core-2.1.2/Enumerable.html#method-i-partition  also included to Array class. There are a plenty of cool methods, and you don't need to organize such logic.


Good job.


@pmeenan it looks like htmlspecialchars fixes the xss here but it does not escape the backslash which could be a problem in some other situations.

For example if you had two variables in a row:
```php
echo "var something=\"{$_GET['foo']}\", \"{$_GET['bar']}\";\n";
```
You can make `foo` equal to `\` and `bar` equal to `,alert(1);//`. Even with `htmlspecialchars` the output would be
```javascript
var something="\", ",alert(1);//";
```
and you're back to a xss.

For a string in a script tag, these characters should be escaped: http://json.org/string.gif


On Wed, Jul 15, 2015 at 11:34:35AM -0700, gus knight wrote:
> > This makes it impossible to start the summary with proper names that begin with a lower case letter, like "ls lists files".
>
> `ls` is in the coreutils package. And I thought it was a convention by now to not have the SUMMARY start with the package name (e.g. for ls it would be `A small tool to list files`.)

+1, otherwise in the package list it looks like:

ls	ls lists files

which is needlessly repetitive.

--
Adrien.


Thanks for catching this! Would you mind adding "(default location is `dist`)" at the end of this line?


Thanks, waiting for more updates so I can push them all at once...


see #15284


lets run coverage report here and flake8 with the standard flake8 config we use elsewhere.


This should be "==" not "=" in the if statement.


Maybe you had a sync problem? The file build/core/clear_vars.mk is available for me.


Actually, this is a merge of the new spawn sys.


Still not working...


Deve desenhar um triângulo retângulo conforme o número passado no argumento, por exemplo:
Eu executo:
metodo(5)

E é mostrado na tela:
*
**
***
****
*****


But that's not all my video recording changes. The biggest thing, the av folder is not pushed.


XFCE4 translates it to something like "Tastenkürzel"


the 2 checks are doing the same, please remove one. as TC standard the ToCreature() result is stored in a local variable and used in the if body


I did repo sync just before adding your patches.
Okay no problem.


This looks good as far as applying the R style recommendations goes. Unfortunately, it looks like you were working from an outdated version of the file. You'll want to checkout commit 9ae8cad by running the following from the working directory in the terminal
```shell
git checkout 9ae8cad -b master .
```
And apply the style changes to that version.


Probably want to 'coverage combine' and 'coverage erase' here since the Procfile uses coverage.


Hah, this is funny, going from result to cause.  Consider perhaps the reverse condition?  Or even:

    auto afterSwitchOver = (env.clock.now () > switchoverTime);
    expect(afterSwitchOver == enableFix):


There is no longer a MediaStop() member in CApplicationMessenger after https://github.com/xbmc/xbmc/pull/7268


@hugoleodev any special reason to do that?
```def up``` / ```def down``` is the old style of migration, and it was replaced by ```change```


972154-NewSimpleChilledCeiling (RKStrand) - i386-Windows-7-VisualStudio-12: OK (1697 of 1700 tests passed)

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


I suppose I should add a lint for that too, then...


hmmm


Oh, it didn't raise any syntax error while testing at my end, and it turns out that there was no call to this function. I'll fix this in the next commit.


oooooohhhh merge conflicts :stuck_out_tongue_winking_eye:


why http?


Use this to improve linking time:
```C
static char const certProperties[][17u] = {"id", "cert", "CN", "issuerCN", "keyUsage", "validFrom", "validTo", "certSerialNumber", "certificateAsPEM", "certificateAsHex"};
```
`17u` is the maximum string length including the zero-terminator. It wastes minimal extra space due to zeroes, but `const` moves these to a read-only ELF section and the runtime linker must not calculate the exact values for the pointers each time the module is loaded. See Ulrich Drepper's excellent paper on [How to write shared libraries](http://www.akkadia.org/drepper/dsohowto.pdf).


@chiaming2000 - What is the timeline for merging the kinetic.proto version 3.0.6 with the master branch?

Are we expecting any more changes to the protocol for batch operations?

Thanks


This should probably be DEBUG for testing.


Looks like [json_encode](http://php.net/manual/en/function.json-encode.php) would be the right thing to use here.


A couple of cosmetic comments
* please add a copyright header at the top of the file (can copy from any of the others)
* should have indent levels 4 spaces deep, instead of 2. Alternatively, you can indent with tabs.

Hint: In sublime you can highlight a section of code and use shift tab to unindent, and tab to indent


I'm really sorry. But I had to temporarily revert all the changes regarding SWT. The problem with this is that SWT is system dependent and we would need to either deploy multiple versions of the program on the website or pack all the various system libraries in the program and load them at runtime. In any case it's a huge hassle and not sure if it's worth it at the moment.

If you still want, we can make a branch for SWT and move everything to that, including the various system-dependent versions. Otherwise, I'd recommend checking out Apache Pivot or JavaFX instead. What do you think?


Zu SweepEventCached.java:
Warum nicht `angle = super.calcAngle();` im Konstruktor und dann einfach einen normalen Getter (`distance ` analog)?


Why the thumb down?


@mbadams5 I added the detailed refrigeration case system and checked the results. The major difference came from the Refrigerated Case Restocking Schedule. The setRefrigeratedCaseRestockingSchedule for OpenStudio::Model::RefrigerationCase seems not working. Do you know how to set the Refrigerated Case Restocking Schedule?


I was confused and just included a bunch of things at some point to see what would happen xD Forgot to remove.


@liaxim You're right. This was done by mistake. I didn't intend to set native ptr to null and I agree with you it should never be. I somehow confused second param with shaderType (which was by default 0 - unlit);
I think, in order to fix this, it's suffice to do:
mMaterial = new GVRMaterial(gvrContext);

I'll create a pull request for this when I get home. Right now I don't have access to github.

@jason2kim No. I can't think of a reason to set native ptr for any multipass example.


Obviously `ls` is just an example. And while it might not be necessary to refer to the name of the software itself in the summary, referring to other software has its uses ("ls range extender plug-in"). It is certainly always possible to rephrase to the summary not to start with a proper name, but there's really no reason to prevent that either.


There's a subtle bug in this code because of the way ledger close times are rounded. `env.clock.now()` will return a value different from `env.current()->info().parentCloseTime`.


FWIW:  The Import Result might evaluate to True, so you might as well return it.  But it's best to check whether the places using the result of `gpg_import_keydata` do an `if result is True` or only `if result`.


Yeap, 2.2 is the version which started to collect, thanks! :)

http://www.infoq.com/news/2014/12/ruby-2.2.0-released


nitpick: this is redundant since you're memset'ing above


:lol:


You are right, it is wrong but AFAIK I disabled FM radio support because it wasn't working for me.


For me build/core/clear_vars.mk exist
Tomorrow reset repo to test


I know we were using `NaN` as a sentinel before. Of course, it does not make sense to continue doing so. I would actually be ok to use 0 as a sentinel, since there are many workarounds (even the one you used).

The other option I guess (and much cleaner) is to make the second parameter for `addChild` a boolean that disables `autoDepth` when `=== true`.


Yep, done.


nit: use print() (i.e.: so it can work with Python 3 in the future)


nit: use print() (python 3)


При 14 шрифте и так такого размера получается. На xelatex смотрел


`build/` contains temporary files that should not be tracked. the directory has been added to `.gitignore` but the files it contains will need to be removed as with `git rm`.


hmm, the ZooKeeper documentation says this, but I couldn't find where in the code this check is enforced.


I am fine with this way of doing the merging, but I think this would more explicit/self-documenting if the format of the host yml was changed. Something like:

```yaml
packages:
  production:
    overrides:
      gcc: gcc-2.4_centos6
    additions:
      - some_other_package-1234
```


Updated (although I need to reconfigure so not tested yet...)


This was the previous version of the file, before @hernandcb gave me his development. Then it was updated.


При 12 шрифте он изначально 14,5 пт. На xelatex смотрел


stop tracking this file


FYI you were missing ` spanIncludingReads = extendedLoc;` before the loop :)


stop tracking files in `.gradle/`


I'd prefer to have the important part first ("Extends the range of") -- rather than the least important part first ("ls") -- as the name "ls" is probably already in the package's name.


Not applicable upstream
https://github.com/openstack/api-site/blob/e49094b3d0ad343ed5119793a47526c15e38e9e2/api-ref/src/wadls/orchestration-api/src/v1/samples/stack_show_resp.json


Смысл в этом? Что от этого неправильно смещается?
Я думаю, можно в блоке колонтитулов задать в ex, если действительно есть польза.


Hi,

The 3.0.6 will be merged into the master branch by July 17th.  There will
be no protocol changes to this branch (3.0.6).

Thanks.
chiaming

On Wed, Jul 15, 2015 at 12:12 PM, abhideodhar <notifications@github.com>
wrote:

> @chiaming2000 <https://github.com/chiaming2000> - What is the timeline
> for merging the kinetic.proto version 3.0.6 with the master branch?
>
> Are we expecting any more changes to the protocol for batch operations?
>
> Thanks
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Seagate/kinetic-protocol/commit/a94d756667451dec508491ef1512592849457adf#commitcomment-12187238>
> .
>


meh! What's wrong with:

```javascript
this.pos.x = event.gameX;
this.pos.y = event.gameY;
```


@vinniefalco Yep. Nice find. Fixed (but not yet pushed).


@eapache this is much better, but it still can block if `Config.Consumer.Return.Errors == true` and nobody reads from the errors channel.


NA
https://github.com/openstack/api-site/blob/e49094b3d0ad343ed5119793a47526c15e38e9e2/api-ref/src/wadls/orchestration-api/src/v1/samples/stack_abandon.json


972154-NewSimpleChilledCeiling (RKStrand) - Win64-Windows-7-VisualStudio-12: OK (1697 of 1700 tests passed)

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


If one developer eliminates something, and puts in a comment stating WHY, then you should not just go in there and put it back with no comment why you are doing so.  Please describe WHY you are making changes. In this case, your change creates a double xml tag: <ServiceGroup path="/SAFplusAmf/ServiceGroup"><ServiceGroup>.  You can run testmgtapi to see this.  Also the path attribute does not include "this" node's name (its missing the last item, eg .../ServiceGroup/sg0)


@emilywistia aren't derivatives really just encoded based on widths, and the height will vary depending on the aspect ratio of the original? (I thought that's why it just said the width before).


+1
but don't we need to check the permission if the user is allowed to list the folder contents, as it's done a few lines above?


only the fist case,  sizeof returns size_t and i'm not entirely sure that size_t is unsigned on all platforms and bits


TeamCity Gensokyo Lobby :: Lobster Server [Build 20](http://pixelated.design/build/viewLog.html?buildId=49&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 56 Build time: 00:00:16


I'm not sure what the fix is tbh


Supprimers ? c'est pas français ^^


for has_failsafe you're using a char, here you're using an int. seems like you should be consistent


You are correct, I don't know that the change method could perform rollback correctly.


Little hint regarding Backstreet Boys? can't open it.


I like the idea of flagging the incoming pod.  I initially thought about doing it in an annotation but didn't like the idea of having to manage it.  If it's done on the SC of the pod I think that concern is alleviated.  I'll look at making those changes.


Yeah, good point. I'll do that.


This could be `auto`...not terribly important.


Apropos: https://github.com/spark/firmware/issues/489

On Wed, Jul 15, 2015 at 11:59 AM, Andy Warner <andyw@pobox.com> wrote:

> Apropos:
> http://community.particle.io/t/toolchain-mac-clean-builds-failing-on-arm-4-9-3-linker-cant-find--gettimeofday/13293
>
> On Wed, Jul 15, 2015 at 11:32 AM, Andy Warner <andyw@pobox.com> wrote:
>
>> I stand by my meta-assertion that if you're operating that close to the
>> bleeding edge of the compiler, you're asking for trouble. Been there. Done
>> that. Regretted it.
>>
>> Also expect the community to grumble about it also, so a FAQ entry
>> _might_ save you some heat.
>>
>> On Wed, Jul 15, 2015 at 11:19 AM, Joe Goggins <notifications@github.com>
>> wrote:
>>
>>> I agree with what Mat said, better to encourage consistent version usage
>>> than to hunt bugs due to an out of date compiler.
>>>
>>> Mat: One thing you could do to further reduce friction of gcc-arm
>>> upgrading is linking to https://launchpad.net/gcc-arm-embedded and
>>> noting that the user download something equal to or newer to
>>> "expected_version". Also, from a "user should upgrade compiler" standpoint,
>>> one quirk I noticed is that the version number the binary spits out via
>>> --version is different from what you'd expect from the download
>>> filename, i.e. the compiler in '
>>> https://launchpad.net/gcc-arm-embedded/4.9/4.9-2015-q2-update/+download/gcc-arm-none-eabi-4_9-2015q2-20150609-linux.tar.bz2'
>>> returns '4.9.3 20150529', even though it's download file name is '20150609'
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/spark/firmware/commit/bcd4e51f74f7b2a59bc24b641895b602193d0921#commitcomment-12183297>
>>> .
>>>
>>
>>
>>
>> --
>> Andy
>>
>
>
>
> --
> Andy
>

--
Andy


Why 200%?


ay. Found the problem to my email.
the RC3 ami (`ami-c1ad79aa`) is not public


NA https://github.com/openstack/api-site/blob/e49094b3d0ad343ed5119793a47526c15e38e9e2/api-ref/src/wadls/orchestration-api/src/v1/samples/stack_preview.json


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.21](http://pixelated.design/build/viewLog.html?buildId=50&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 122 Build time: 00:00:23


Hmm... good question, @mrdavidjcole . @jordanmunson what are your thoughts? I suppose in saying the original aspect ratio of the "sample file" the aspect ratios make sense in the chart. Plus, when you right click on the video in HTML5, we've named them by their height, not width. I'd argue width doesn't make a lot of sense to people in a 720/1080p YouTube world.


Indeed; now passing script uuid as well as command cookie.


Removed.


Ok I documented the method, but I have a question: since none of the methods in this file have documentation, should I do it?


I liked the idea and it is indeed looking better. However, the code is a bit messed up here. You have included one row in the other ie. code for row 2 is inside row 1 now.


this is moved up to enable it's use in the `setuptools.dist.Distribution` function


asd


Thanks, it has built now, although got a few more like it in OE patches now... :)


I'm mostly just a pointer wary person :smiley:. If something doesn't need a pointer I tend to avoid them just to avoid weird mistakes where client is modified but the scope of that change is greater than just the function. What are the advantages of not dereferencing?


you can remove this duplicate of `manualjavacall.sh`.


This is a bug: for the RAII lock to work, you need a variable that lasts until the end of the scope.


I see who the code auditor is now! @stinkyfingers you keep the auditor in line now!


@sbma44 If you you want an example that someone can cut and paste, yes, you'd need to stick to urllib/urllib2. But if this example goes on PyPI, it might as well bring in other non standard libs.


Submitted upstream
https://review.openstack.org/#/c/202246/


sqlite_timeseries (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1240 of 1240 tests passed)

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


hummff y'a plus joli comme code ^^
Tu peux utiliser `CollectionUtils.isEmpty` de commons-collection ou `org.springframework.util.CollectionUtils.isEmpty` de spring...
sinon avec guava tu as `MoreObjects.firstNonNull(myList, Collections. emptyList())`


Who's mickelsonm? Max?


oohhhh shots fired


TeamCity LabkeyRelease15.2 :: CDS postgres 9.4 [Build 10](https://teamcity.labkey.org/viewLog.html?buildId=261132&buildTypeId=LabkeyRelease152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 22 Build time: 00:34:42
### 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>

```


Oups...


This is the line that downloads and makes available `cython` and `numpy` before the rest of the script runs.

See [this stackoverflow answer](http://stackoverflow.com/a/12061891/74291) for more info.


Did you mean `guide-key` here or `which-key`?


Yes, very concise. Well done.


ouhh notFoundException de chaine vide ^^  berk!


I had no idea on how to go about dynamically creating form fields, I had assumed it required JS though.  I found these 2 functions online and I was wondering if you could explain how all of this is working to me tomorrow?


`r` très joli nom de variable !


@rec Both you and vinnie had similar comments. When two reviewers say similar things I need to accept the change.


This is a hack to workaround a bug in `numpy`.

See [this stackoverflow answer](http://stackoverflow.com/a/21621493/74291) for more info.


TeamCity LabkeyModules15.2 :: CDS postgres 9.4 [Build 2](https://teamcity.labkey.org/viewLog.html?buildId=261130&buildTypeId=LabkeyModules152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 22 Build time: 00:36:19
### 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>

```


t'es obligé de faire le catch ?


I am a Javascript Jedi sent back in time to ensure the survival of the Aries Automotive website codebase... :smirk:


as i mentioned to brandon, i prefer to start with project imports (e.g., `edu.oregonstate.*`) followed by dropwizard imports (e.g., `io.dropwizard.*`) followed by external dependencies. this is not a rule, just a suggestion to make code more descriptive, since it is easier at a glace to understand class dependencies.


Needs proper formatting.


Agreed.


Maybe it's worth noting that we'll make derivative based on the _width_ and if you're using 16:9 ratio (the most common here), they'll result in these sizes.

It seems to be that the decision is do we want to make it clear for most people or do we want to make sure it's 100% accurate for everyone (but potentially making it less clear cut for some).

For me, I'd think we could list the widths, then explain that for 16:9 videos, those widths result in the 720p/1080p sort of things people are used to thinking in.

From a practical perspective, the knowledge probably doesn't help many people at all.


:smile: yes


j'espère que le `adminUserServiceCustom.getCurrentUsert().getEntityId() est gratos. tu l'appelles 3x en 3 lignes...


Same as comment above.  I found this method online and I am not really sure how it works.


Done.


tu stockes "adminViewedSession" ?? tu peux stocker ça localement = front ?? attention ça sent le defect ça !


:+1: I think we'll want to adapt the code a bit for PyPI anyway. In particular, the batch and simple geocoding examples should be combined, and we should add flags for reverse geocoding, helicopter mode and maybe autocomplete (though tbh I'd be fine keeping that last one solely in Mapbox.js).


What's all this clientside JS?
Put(it(on(the(server(){}){}){}){}){};


I think most of this method you could drop if you instead of loading the project manually use the `eu.esdihumboldt.hale.common.headless.impl.ProjectTransformationEnvironment` class that is used to load projects headless for transformation. You can access project, schemas and alignment from the object after creating it with the project location.


@xiaolu i tried to use that, with latest CM 12.1, and this TWRP repository, but
'mka recoveryimage' creates for me an image that is not booting, and also has much less files in it then the TWRP recovery i've found here: https://translate.google.com/translate?hl=en&sl=zh-CN&tl=en&u=http%3A%2F%2Fbbs.anzhuo.cn%2Fforum-278-1.html
Can you go into details?
So what I did:
1. clone this repo and copy it too /bootable/recovery-twrp
2. clone the Device repo to the device section of CM device tree part
3. lunch, select ml tuhl
4. mka recoveryimage

But it doesn't boot. Can you detail me what you do to compile this? Thanks a lot!!


c'est de l'anglais ça ^^ 'supprimers'


This actually installs `numpy` and `cython`. The install will work without the `cython` portion (since it's actually only needed to compile bcolz), but I left it in to make extension installation easier.


Move to `thread.js`
```js
if (!document.location.hash)
```


Looks like :hankey: to me lol


There are some irreversible migrations that the ```change``` method can't perform rollback (see [the docs](http://api.rubyonrails.org/classes/ActiveRecord/Migration.html#class-ActiveRecord::Migration-label-Reversible+Migrations)), like ```remove_column``` or ```change_column```. But in most cases it should just work fine.


Better change the command alliance and set public chat clear to /cc


Yes, thanks, there was a rebase mistake in there somewhere :)  I'll fix it.


EvansHVACObjects (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)


I don't know about C89, but according to C99 it should be an unsigned integer type regardless of platform and width.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.765.85e82aa4327a58d98dfebaf996045cdd212ef919](http://build.palaso.org/viewLog.html?buildId=59751&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 377, ignored: 3 Build time: 00:02:47


What Developers thing means?


Sorry about the AMI - a bit rushed to wrap this up so didn't go through the checklist yet... It's public now.

Yes, the new path is under `/opt/cloudman` (vs. `/tmp/cm`) - this move is part of a larger security hardening effort so a fairly big set of changes for this late in the release process but needed to be done.


This file shouldn't be here, my bad


utilise lodash mec ! c'est la solution à tes méthodes toutes moches :+1:
https://lodash.com/docs


This form_params method caused some serious confusion until I finally figured out I needed to include the `form_field_attributes`  as well as the `form attributes`.


you can cast the obj directly to creature


Totally inspired by this commit
https://github.com/SpeedProg/PveLauncher/commit/b4b86a2ab96d5aa4a971eeb46b3291acb133e0a2


This file isn't used anywhere, I'l lremove it.


tu laisses le alert ??


Saw that lol


since we're using groovy, use implicit getters and setters:
http://groovy-lang.org/style-guide.html#_getters_and_setters

tight access control is best practice in java, but i've found that trying to do so in groovy is like cutting against the grain.


What it does?


Well, I don't know why in fact, it should work without it, you're right ! Gonna try tomorrow.


I'm not sure if this line will still be relevant in later commits - and I'm not sure if you're doing this in purpose, but to delete an item from a dictionary do

```
del self.marked_containers[key]
```


where is the spellmod taken before?


There is typo "canolaoil"!


I like it! @mrdavidjcole do you have any final thoughts before I change the copy?


Those bugs were totally my bad, I should have left plenty of TODOs, the system was, and probably still is, far from perfect.


this line does not make sense after setfacing


After this change, the number of failed requests fell from 360 to 261.

[OVERALL], RunTime(ms), 56514.0
[OVERALL], Throughput(ops/sec), 17.694730509254345
[UPDATE], Operations, 494
[UPDATE], AverageLatency(us), 56373.838056680164
[UPDATE], MinLatency(us), 52758
[UPDATE], MaxLatency(us), 251339
[UPDATE], 95thPercentileLatency(ms), 60
[UPDATE], 99thPercentileLatency(ms), 63
[UPDATE], Return=0, 233
**[UPDATE], Return=-1, 261**


For me this is going in the wrong direction. Typescript is created to add types to a dynamic language why are you going to remove the types again? You could simply stick to JavaScript.


All good by me!


This is similar to how pandas outputs large data frames: http://pandas.pydata.org/pandas-docs/version/0.15.1/whatsnew.html#display-changes


This seems a bit more readable:
```python
self.loggable_content_types = loggable_content_types or []
```


I don't think this dictionary needs to exist if the keys are always identical to the values...


we could possibly just add a capacity to this guy and then a `default:` to bail, might not really be a problem in practice. I suppose if we wanted to get super extreme we could do the offloading to disk stuff as well but seems overkill unless someone wants that


No you can't really see it :) It is actually a credit more than anything else, Nick pointed me to the right way on how he generated his spectra images. The key point was to perform the negative values offset before applying the OECF, in my tests as I was using the convenient `colour.XYZ_to_sRGB` definition which applies it by default, I still had issues.

Here is an extract from our conversation:

```
[...]
I just converted the color matching functions in CIE space to sRGB, and allowed negative numbers, then I added a constant until I didn't have any more negative numbers

Make sure you do it in linear space

That way you get accurate color data, but just projected on a grey background, instead of black.
[...]
```

The *Retrieved from* is just coming from Mendeley standard APA citation export as I entered Nick's website url (well a Facebook page) into our citation database. It is a bit confusing when read this way though :)

```
@misc{Spiker2015,
author = {Spiker, Nick},
mendeley-groups = {colour-science/colour},
title = {{Private Discussion with Mansencal, T.}},
url = {http://www.invisiblelightimages.com/},
year = {2015}
}
```


84691154_AutomateBuildingOfDocs (lefticus) - i386-Windows-7-VisualStudio-12: OK (1681 of 1684 tests passed)

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


LOL


HAHA


I am not entirely sure what you mean. In the manual section of a list the order is completely controlled by the user (drag&drop). In the automatic section of a list the order is controlled by the `sort` part of the elastic query that is configured with this list. Was that your question?


To people reading this thread later, you may have previously had a Podfile like
```
pod "Fabric/Crashlytics"
```
To migrate, change to:

```
pod ‘Fabric’
pod ‘Crashlytics’
pod ‘Digits’
pod ‘TwitterKit’
pod ‘MoPub’
```

For more details see [http://docs.fabric.io/ios/fabric/cocoapods.html](docs.fabric.io)


Faut virer toute la fonction.


Yet another version change.


Now you can add the same form more than once — make sure the selected item is not disabled


@htobon I checked the messages regarding project locations but they are not in the general session. On the other hand, those "partnerSave" messages are not in the correct location, but I didn't make any change in that part.


Why are we not using statuses.new here?


I couldn't test the way I needed to without being able to mock the ```ExchangeService```. Brought in mockito as a test dependency.


why did you decide to call it versionNum() instead of protocolVersion() like in context? I much prefer protocolVersion, and I absolutely hate shortenings of words like "num"


It appears that you are effectively cloning the client in cases where you don't need to. And the Go idiom seems to be that structs should generally be passed by pointer. There's also a question of whether the pass-by-value will keep the HTTP keep-alive logic from re-using the same connection.

In general, though, I'd suggest just going with the way the Go standard libraries go, which is using New() functions to return pointers to structs, and hang the functions of the *struct.


ok


Since the constructor ensures that `self.loggable_content_types` is a list, this can be simplified to:
```python
return content_type in self.loggable_content_types
```


Ya même des trucs en français alors que c'est normalement en.


This is crazy but it worked. I use the EWS parsing code to surface the actual error.


such code should never be in spellscripts


If there's an easier way to copy data from an ```InputStream``` to an ```OutputStream``` with plan old Java, let me know.


Ui, la première fois la ligne existe pas.


i don't believe it is necessary to add `@JsonProperty` annotations to configuration class properties. the configuration class is fed into the main class but is not used elsewhere.


Those frustrating times :sweat_smile:


It's not needed anymore now that we've dropped IE8.


s/and DTLS 1.2/and the corresponding versions of DTLS/


Can you explain that RegExp for me?


I'm not sure single letter attributes are ideal. I've no idea what this variable is for based on its name. What does it do?


Matt - great job on the design, BTW!


that's why it's `false` by default... is there a use case for turning it on, and  then *not* reading from it?


un commentaire serait alors de bon aloi :palm_tree:


It's not a matter of personal preference. It's against the convention to name local variables in uppercase. Simply calling the var `u` would be closer to the Go idiom, and isn't confusing.


@barbeau I added fixes based on the commit (https://github.com/CUTR-at-USF/onebusaway-application-modules/commit/e183964c1fce18da3281078da574eb9d338e19b8).


On 7/15/15 2:17 AM, Ola Bini wrote:
> Disagree. I'm saying we shouldn't decide whether to panic or return>
error instances at all. We should leave that choice to the user of the>
library.

I don't believe I understand the motivation for this. Why would we leave
the choice of how to be informed of an error to the user of the library
when some of these errors are internal to the otr protocol and are
outside of the client domain (an xmpp-client client)?

What is the reason for allowing the user of the library choosing how
he/she wants to be informed about an I/O failure that can't be recovered
(errors when reading from the rand reader, for example)? Is this the
same reason for choosing how to be informed about a broken AKE/SMP
message that simply can't be parsed?

In my mind, the analogy is something like if I expected a TCP socket
library to allow me to define how I want to be informed about problems
happening during the TCP congestion control.

The protocol needs to handle its internal errors rather than delegating
this to the user of the library, otherwise how would it be able to keep
the state machines running without intervention of the user of the library?


The issue reference should looks like this - `clojure-emacs/cider#1203`; otherwise this won't be linked to the actual ticket.


что-то


S'il y a aucun props et que \__str__ return just le nom de la classe, c'est exactement le défaut. Donc remote \__str__ dans ce cas la.


Выясняется, что при xelatex, \curtextsize выдает шрифт 10 (будто extsizes не сработал, но он сработал)


Nice.


The indentation looks off here.


inplace est une props.


idem


EvansHVACObjects (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


whatever it takes.


mode is a props.


Whatever you prefer, when you look through the summaries of the existing recipes, you'll find a wide variety of different styles. If you want to standardize that somehow, great, but you might want to discuss it on some mailing list first. Ideally before introducing checks that disallow certain styles. That's really all I wanted to point out.


Design? Who designs? Don't we hire graphic designers to do that?


useless default implementation of \__init__ remove it.


After testing, I confirm the merging of cultures and on_actions is working properly, yay !


remove


remoe


I knew this one, should have done that ages ago, sorry >.<


I get the following warning window when a game loads :
![0](https://cloud.githubusercontent.com/assets/3978690/8708331/92e88dcc-2b3b-11e5-857a-9c7ac53c624b.png)
The corresponding line in the debug console :
```
31:14:244 user_main    E[*]: MsgHandler.cpp:54 Warning: WriteRest: op out of range (0x13ed1beb1 uses 0x7127d48)
```

These values are not always the same.
It seems that this warning happens on x64 builds only.
This also doesn't happen with all games. For example, it happens in MHFU, MHP2ndG, GTA LCS and GTA VCS but not in the WipEout games...


Fixed in https://github.com/CUTR-at-USF/onebusaway-application-modules/commit/84f7a6eb92b92ac0e817349729ff192c5e820735#commitcomment-12188073.


You could also just rescue a failed Etc.getpwnam instead of shelling out to id.  I think I'd prefer that also, or specifying the full path to id, which might be different on different platforms.


These are all still execute, not invoke.


Well, I disagree. Returning an error instance might be right in some cases, but causing a panic is _never_ the right choice for a library to do, just as calling System.exit() is always the wrong thing to do in a Java library.

I'm not saying the client has to know the details, but you will notice that a typical TCP will will give you levers to tweak congestion parameters, and incidentally linux 2.4 support explicit congestion notifications as well.

Fundamentally this is part of feature parity with libotr that does offer error handling customization.


Eh, whatever. This will tide people over until I can make a cool looking gui with javafx and css :D


Ah will check it


EvansHVACObjects (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


J'en déduis que je dois également retirer tous les  :

    def __str__(self):
        return self.__class__.__name__

Non ?

Le 15 juillet 2015 15:54, Frédéric Bastien <notifications@github.com> a
écrit :

> S'il y a aucun props et que __str__ return just le nom de la classe, c'est
> exactement le défaut. Donc remote __str__ dans ce cas la.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/harlouci/Theano/commit/9188880f67dee6b9c3ba842321bf3b74885d7fdf#commitcomment-12188097>
> .
>


Why not only @Path("{name}")? I think it is more intuitive and analog to other rest APIs (e.g github --> GET /repos/:owner/:repo)


May not make any difference speed-wise, but this can also be `const`


The expected behaviour is to log IFF the list is empty OR the content type is in the list.


portal_actions already checks permissions.


gem này lạ. k biết có dc sử dung j k @@


@levicole can you look at this? I keep getting an error about undefined method 'bookmarks_path'

```bash
Started GET "/topics/8/bookmarks/new" for ::1 at 2015-07-14 20:56:08 -0500
Processing by BookmarksController#new as HTML
  Parameters: {"topic_id"=>"8"}
  Rendered bookmarks/new.html.erb within layouts/application (9.9ms)
Completed 500 Internal Server Error in 14ms

ActionView::Template::Error (undefined method `bookmarks_path' for #<#<Class:0x007fe219672f70>:0x007fe21f7b35c8>):
    1: <div class="row">
    2:   <div class="col-md-8">
    3:     <h2>New Bookmark</h2>
    4:     <%= form_for [@bookmark.topic, @bookmark] do |f| %>
    5:       <div class="form-group">
    6:         <%= f.text_field :url, class: 'form-control' %>
    7:       </div>
  app/views/bookmarks/new.html.erb:4:in `_app_views_bookmarks_new_html_erb__150230780522125823_70304600889640'

  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/templates/rescues/_source.erb (6.9ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/templates/rescues/_trace.html.erb (2.4ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/templates/rescues/_request_and_response.html.erb (0.6ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/actionpack-4.2.0/lib/action_dispatch/middleware/templates/rescues/template_error.html.erb within rescues/layout (50.4ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.3/lib/web_console/templates/_markup.html.erb (0.2ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.3/lib/web_console/templates/style.css.erb within layouts/inlined_string (0.2ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.3/lib/web_console/templates/_inner_console_markup.html.erb within layouts/inlined_string (0.2ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.3/lib/web_console/templates/_prompt_box_markup.html.erb within layouts/inlined_string (0.2ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.3/lib/web_console/templates/console.js.erb within layouts/javascript (38.4ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.3/lib/web_console/templates/main.js.erb within layouts/javascript (0.2ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.3/lib/web_console/templates/error_page.js.erb within layouts/javascript (0.3ms)
  Rendered /Users/sfiller/.rvm/gems/ruby-2.2.0/gems/web-console-2.1.3/lib/web_console/templates/index.html.erb (85.6ms)
```
I'm not sure what I'm missing.


I am wondering is it possible to improve it so that it can keep O(1) but also handle invalid input such as kill one solder twice


2 move to Setting


if this is generalizable, would accept as PR


@TekkitCommando I already added @mastercoms , let's go on skype.


Por qué agregar el rol de 'Perito Valuador' y sus permisos con un Seeder y no con el Admin? Si es una tarea administrativa y no técnico-operativa.


greate! we are developing/fixing for KDE5:

https://projects.kde.org/projects/kde/workspace/plasma-desktop/repository/show?rev=user-accounts

https://projects.kde.org/projects/kde/workspace/plasma-workspace/repository/show?rev=accounts-service

and we must work hard try to merge into upstream master ;-)

On 2015年07月15日 09:52, Jeff Bai wrote:
> I've been enjoying KF5 recently (wow)...
> 2015年7月15日 上午9:51于 "sirtoozee Is 小顺毛の爹"
> <notifications@github.com>写道：
>
> > Thank @MingcongBai <https://github.com/MingcongBai> `s advice ;P
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/AOSC-Dev/qtaccountsservice/commit/622f623632e549b5bb98a4b46ed17b1fd1803379#commitcomment-12169746>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AOSC-Dev/qtaccountsservice/commit/622f623632e549b5bb98a4b46ed17b1fd1803379#commitcomment-12169756>.
>


Consider replacing lambda with `std::bind(&Operator::Forward, op, _1, std::move(in), std::move(out)))` as the vectors are copied here when captured by lambda. In C++14 this is better supported https://en.wikipedia.org/wiki/C%2B%2B14#Lambda_capture_expressions Then you can do `[this, opt, ctx, in{std::move(in)}, out{std::move(out)}](RunContext ctx) {`


+ code cleanup
+ stop storing user venmo friends in db
+ store auth token with user until expire
+ update versions to improve Heroku performance


Thanks ! this helpful for me !


@DefinitlyEvil Could you go on our gitter? We are chatting there now :)


Okay. ;)


xem thêm thông tin bằng scolling down. đưa vào chuẩn bị cho show all user


I think we'll need to do the heavier lifting to actually resolve HEAD, rather than assuming that the right hand side of the refspec can be assigned to the left hand side.

I was trying to push HEAD to a new remote branch named feature/feature-test and this caused the JGit implementation to assume I was pushing refs/heads/feature/push-test to refs/heads/feature/push-test.  I didn't have a local branch named feature/push-test, so the push attempt failed.

I believe there is already a "rev-parse" implementation reference in the JGitAPIImpl, and it seems like it would be worth the extra effort to reduce the risk that HEAD will be misinterpreted and will push the wrong source branch.


@parasyte squashed another bug thanks to the test units ! Jasmine is definitely one of the best addition in melonJS :P


i'm surprised though, this was not reported by jslint !


@claudiosmweb, é uma pena que as pessoas pensem assim; melhor ficar sem link mesmo, em aberto. Vida que segue.


Steve,
I am having problems adding players to the game queue.  I tried the following in the game.rb:

def initialize
  game_players = []
end

In my game model tests I get the following error:

Failure/Error: @game = FactoryGirl.build(:game)
     NoMethodError:
       undefined method `title=' for #<Game not initialized>

it the start_game_spec.rb I get the following error:

Failure/Error: click_link "Old Maid"
     NoMethodError:
       undefined method `title=' for #<Game not initialized>

Actually, in this commit everything works, I am going to push up the next commit as that one fails.


se borro "listo"


why are we not supporting sorting anymore?


Borro el echo


que bien!!


Steve,
Here is the offending code.  My thinking is the following, I have to fill up the game_players array with a certain amount of players before I can start the game.  Also, I am not sure I need the title validations, as the game title is going to come from the links once the player is signed in.


Hmm, I'm not sure. If I map the task_labels to the Jira description field it shows up like this for example:
['SD-Elements', u'?afi[]', u'sdf asdf s-af', u'tagtest', u'yeahtag']
All those tags are exactly the same as the way I inputted them in SDE besides for the u'...
What Jira field are tags supposed to be mapped to by default though? Labels? I can't seem to get that to work even on master...


olak ase


Hola!


I think we're just missing an entry in the `build_args` to properly handle labels, no worries


Que hace esta linea de codigo?


try to use #  instead of +


teat


Oh!


Hi, the result file was supposed to be unlinked by `xl2tpd-control` command.  But in my test running, I found there were a lot of result file left in the `/var/run/xl2tpd-control-NNN.out` and they were of type regular (not pipe file).

The problem seems to be that
 1. `xl2tpd` opened the result file
 2. `xl2tpd-control` encountered error case, then close the result file and unlink it
 3. later `xl2tpd` `fclose()` it which will leave a regular file there with the same name as the pipe result file.

The regular-file-leftover can be reproduced by the following commands sequence.

		mkfifo h

		cat >h <<EOF
		hello
		EOF

		# Open another prompt
		rm h

		# Then
		# 1. ctrl-z the cat command
		# 2. continue it with fg command
		# 3. cat command will come up and quit immediately
		# 4. a regular file h will be present


Estoy usando las versiones más recientes de PostgreSQL y Sequelize, 4.4.0 y 3.3.2 respectivamente, pero al consultar por las preguntas se me presenta el siguiente error:
```bash
The argument passed to findOne must be an options object,
use findById if you wish to pass a single primary key value
```

Así que he reemplazado la siguiente línea de código:
```javascript
models.Quiz.find(req.params.quizId)
```
Por esta:
```javascript
models.Quiz.findById(req.params.quizId)
```


legit?


likewise


Ok, I'll try looking into that more after the Hardware Hacking course tomorrow.


@parasyte have to do this now, to force the z value for the debugPanel. Any other suggestion on how to properly manage this ? maybe the `z` parameter in the `addChild` function should force the value whatever is `autoDepth` set to ?


Cool, have fun with that


@jpodwys Awesome. That's very helpful! Thanks so much!


That picture.... haha, +1


My .bashrc isn't located there.  I think it might be easier to look at different places where the file might be located.  For starters, probably here:

```
Path.Combine(Environment.GetEnvironmentVariable("home"), BASH_HISTORY_FILE_NAME);
Path.Combine(Environment.GetEnvironmentVariable("userprofile"), BASH_HISTORY_FILE_NAME)
```


trying to close this out


Indeed, I moved the text for add-lns wrongly to add-lac.

Thank you for the careful catch.  I will correct it.


I confused. Did we all agree that the statistical error of the Asymmetry (A) should "Astat" but not "A*Astat"?


Curious as to why you removed this..


Not sure what's going on there :confused:

Wish I had time to really dig into the codebase and build out a full test suite :frowning:


associate #1


+1 thanks for this!


The ramdisk implementation seemed kludgy, and Minecraft isn't well designed to run off RAM.


呵呵我就试试


lgtm


Interesting, been using it coming upon four years now and haven't had a problem with it.


It wasn't "broken" per-se, I just feel as though it could be much better implemented.


Looking at your implementation, which uses buf.len(), it seems that the semantics are slightly different than in the original proposal.  The original version doesn't adjust len at all.  I believe that len should not be changed, because read() doesn't change len.  So I think you can't save a register that way.


This has actually caught a failing test in the 'new-styles' branch. Thanks!


Fair enough, any plans to implement it again?


Foi alterado para facilitar os testes


Was this a typo?  You have     t = - t  in the code.. your negative sign comes *after* the equal sign instead of before it.


Yes, but it isn't going to be soon; I need to research how to better implement it first. Many of the issues with Ahtenus' script was due to the multi-world implementation.


Deleting htcaccess


Alrighty, good luck with it.


Agree with the added text....easy to read and simple.


Javascript doesn't use `pass`. It supports empty {} block.


Thank you for your understanding, sorry if this change caused any inconvenience!


Nifty's ChannelHandlerContext is per-handler per-channel, so multiple channels won't conflict. To get the ChannelHandlerContext, someone would have to get the Channel's ChannelPipeline and explicitly ask for the ChannelHandlerContext for the ConnectionContextHandler.


No worries, I'll just try Ahtenus one since I need it and my physical server was built around ramdisk being the main host and has worked for two years now. I just recently did a fresh install so was look at all the options out there now.


list 是为了 conj 之后新的在前面。这个是个手误，不应该放进来的。。。


> Didn't look to closely at this test, I suppose it repros and the NPE and verifies that the fix indeed fixes it?

Yup. One `@Test` method for the usual case (connection opened, serve request, close connection) and one `@Test` for the case with the NPE.


Explain in comment what 2 means here. Or alternatively, implement different versions of get_file_from_server (get_city_file_from_server, get_zipcode_file_from_server, etc) in the helper code.


Instead of "given file", it's better to use this file as an example http://sensibilityclearinghouse.poly.edu/data/states/US.txt I don't know if the file has the country/state combination of all countries in the world.


I wonder why not use magit?


Line 52-55: Can we further refactor openfile, readat, and close into the helper code?


By default the members are private in the class. So the naming convention should be according to that.


@ayyash again, sorry for the delay. this looks fine, but I worry that isDefaultPrevented() is ported to all jQuery derived libraries. Would you mind sending along an example on how you use this?

Thanks!


Follow the naming convention.


I had a problem with my code not applying the crop properly  Changing this line to the following resolved the issue:
```const CGSize size = CGSizeMake(CGImageGetWidth(self.CGImage), CGImageGetHeight(self.CGImage));```


@tqchen what about the Symbol->bind returns an operator that directly push to dagengine rather than keeping a graph of narray operator? This is still dynamic scheduling, but in this way we can save indirection/overhead.


Follow the naming conventions.


You shouldn't have to run any external commands - that line of code should create a collection. Did you have Meteor running?


Explain in comment why address = get_geolocation(lat, lon, 1)[0]? Because get_geolocation can possibly return more than one address. We take the first one.


You need to add some text in Swahili


Fixed in https://github.com/CUTR-at-USF/onebusaway-application-modules/commit/84f7a6eb92b92ac0e817349729ff192c5e820735#commitcomment-12188073.


@aggelosQQ
In any type of playermsg or bcast msg you put playerMsg(TFM_Messages.Method and it shows the method, so like
playerMsg(TFM_Messages.MOD_TAG + ChatColor.GOLD + " You cleared your own chat");
[ImmaFreedomMod] You cleared your own chat


Isn't the `0xff` mask unnecessary here (at least in the non-WCHAR case)?


```js
//TODO: Only the Peak Performer can work on this code...
```

Look guys... I am designing!!!


@thet see ^


Why would you log if the the list is empty? Isn't that a case where the content type is not in the list?


Only players that are inside the developers list on the util or the whocanusesys on the util can use this command.


Probably worth a similar function comment here, or at least reference the string function's comment above.


yes

2015-07-15 15:59 GMT-04:00 harlouci <notifications@github.com>:

> J'en déduis que je dois également retirer tous les :
>
> def __str__(self):
> return self.__class__.__name__
>
> Non ?
>
> Le 15 juillet 2015 15:54, Frédéric Bastien <notifications@github.com> a
> écrit :
>
> > S'il y a aucun props et que __str__ return just le nom de la classe,
> c'est
> > exactement le défaut. Donc remote __str__ dans ce cas la.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/harlouci/Theano/commit/9188880f67dee6b9c3ba842321bf3b74885d7fdf#commitcomment-12188097
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/harlouci/Theano/commit/9188880f67dee6b9c3ba842321bf3b74885d7fdf#commitcomment-12188195>
> .
>


"Defect" c'est quoi ?
Je peux aussi, c'est comme tu veux, là ça marche, mais je peux tout passer côté front.


I already fixed this.


Nit: I normally use '0' instead of 0x30 when converting from ASCII to an int  as I find it slightly more explicit what the conversion is doing.


Fixes headers centering in Moodle 2.6


No longer needed, uses padding at the bottom of the tree instead


Oh, right. It uses `isFinite`, which coerces the argument to number.

Fixed in https://github.com/reindexio/youmightnotneedunderscore/pull/8.


Hide this to stop dataTables from turning into an AMD Module and becoming unsupported


develop (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

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


It was only ever done by an admin before, when their profile was created


Fixes the Tree area being too small and placing a vertical scroll bar, also replaces gap to compare button


it is conventional to start shell scripts with a "crunchbang" like `#!/bin/bash`


Here is a crude attempt at doing it with js...

http://jsfiddle.net/Mottie/una3ecnn/

One issue with it is that sometimes it breaks up selector groups on to separate lines.


Sooo coool!!! :D


I might be tempted to make this explicitly compare against `FASTPARSE_EXHAUSTED` simply because it makes it more obvious to the reader that's why we're returning.


Dans mon cas il y a un __props__, donc je le laisse.

Le 15 juillet 2015 16:01, Frédéric Bastien <notifications@github.com> a
écrit :

> yes
>
> 2015-07-15 15:59 GMT-04:00 harlouci <notifications@github.com>:
>
> > J'en déduis que je dois également retirer tous les :
> >
> > def __str__(self):
> > return self.__class__.__name__
> >
> > Non ?
> >
> > Le 15 juillet 2015 15:54, Frédéric Bastien <notifications@github.com> a
> > écrit :
> >
> > > S'il y a aucun props et que __str__ return just le nom de la classe,
> > c'est
> > > exactement le défaut. Donc remote __str__ dans ce cas la.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/harlouci/Theano/commit/9188880f67dee6b9c3ba842321bf3b74885d7fdf#commitcomment-12188097
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/harlouci/Theano/commit/9188880f67dee6b9c3ba842321bf3b74885d7fdf#commitcomment-12188195
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/harlouci/Theano/commit/9188880f67dee6b9c3ba842321bf3b74885d7fdf#commitcomment-12188247>
> .
>


I might be tempted to make this explicitly compare against `FASTPARSE_EXHAUSTED` simply because it makes it more obvious to the reader that's why we're returning


Hoi Rik,

Dat van INTERNAL_IPS .. wat met ik daar precies voor doen? Nou ja, dat ga
ik uitzoeken zodra k weer op school ben :-)

Hele fijne vakantie toegewenst en dank maar weer voor je inzet :-)

Groet van Kees

2015-07-15 8:47 GMT+02:00 rsnel <notifications@github.com>:

> Beste Kees,
>
> On Tue, Jul 14, 2015 at 11:40:58PM -0700, ko3st wrote:
> > Hoi Rik, Kun je er automatisch voor zorgen dat de gebruiker
> > afhankelijk van het IP adres, wel of niet de leerlingnamen te zien
> > krijgt? Het idee is dat als je dat doet vanuit het intranet, dat je
> > dan wel de namen ziet, maar vanuit de rest van de wereld, dan zijn de
> > leerling namen niet zichtbaar (en zie je slechts de nummers). Idem
> > voor de (volledige) docentnaam VS de docentafkorting... Groet van Kees
>
> Voor leerlingennamen zit dat er al in. Zie de optie INTERNAL_IPS. Alle
> netwerken in het lijstje krijgen de namen te zien en alle andere niet.
> Dit is ook handig voor de zoekbox (autocomplete), hiermee kan je snel
> het rooster an een leerling vinden als je het leerlingnummer niet weet.
>
> Docentnamen zijn niet fatsoenlijk beschikbaar in de udmz (niet voornaam,
> tussenvoegsel, achternaam, maar alleen de volledige naam; dus niet
> sorteerbaar), daar doe ik niks mee.
>
> Groeten,
>
> Rik.
>
> --
> Nothing is ever a total loss; it can always serve as a bad example.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rsnel/ez-roosterbord/commit/827dff6cb1c5c8c53030bd8cff348dc2f494d376#commitcomment-12172160>
> .
>

--
================
Kees Podt, Mondriaanlaan 33, 2172 DG, Sassenheim, 0252 - 70 70 24
kees.podt@gmail.com
06 - 51 77 89 30


+1

Nice work! Couple of minor comments, nothing signficant. Have you got any before/after numbers?


This doesn't sound too right.
Imho it should be more like
> The property player.*Race* is deprecated! Use player.*Faction* instead!

Anyway:
> 21:00 (abcdefg30) also, the Race property is only in since one playtest (i think), and i'm pretty sure it is (almost) unused
21:00 (abcdefg30) so what against removing it? we'll want to remove it anyway somewhen


I submitted a non-RS-specific version to replace this
https://github.com/openstack/api-site/blob/e49094b3d0ad343ed5119793a47526c15e38e9e2/api-ref/src/wadls/orchestration-api/src/v1/samples/stack_adopt.json


This is not going to work with 32-bit platforms (Tegra, ARM). Need `uintptr_t` instead of `unsigned long long` for pointer arithmetic and expressions instead of magic numbers.

Also, it's probably better to use reinterpret_cast for casting here.

```cpp
const size_t alignment = 64;
size_t space = width * height * bytes_per_pixel + alignment;
rawdata = new unsigned char[space];
uintptr_t ptr = reinterpret_cast<uintptr_t>(rawdata);
uintptr_t aligned = (ptr - 1u + alignment) & -alignment;
data = reinterpret_cast<unsigned char *>(aligned);
```


And by the way, I moved the selectors to the end in https://github.com/StylishThemes/GitHub-Dark/commit/71f87dd4abfc89db5b114f835ec5c8dbdee7b02a :wink:.


Hi,

Is there any reason why the old commands (those with no “Merlin” prefix) are still named in the help file, although they do not exist anymore? That’s confusing.


Removed the return for both , Ran the test and it still passes.


This gave an error on mac?


Here, I'd return instead.


NA
https://github.com/openstack/api-site/blob/e49094b3d0ad343ed5119793a47526c15e38e9e2/api-ref/src/wadls/orchestration-api/src/v1/samples/stack_create.json


Oh. Is this how backward compatibility is maintained. If so, it would be cleaner to have the constructor do:
```python
self.loggable_content_types = loggable_content_types or ['application.json']
```
That way the loggable_content_types list is used consistently: if a content type is in the list it gets logged, else not logged.


NA
https://github.com/openstack/api-site/blob/e49094b3d0ad343ed5119793a47526c15e38e9e2/api-ref/src/wadls/orchestration-api/src/v1/samples/stack_update.json


Given the previous behavior of OBA, I think filling both arrival and departure fields, if one is null, is ok.  So, I think this is ok as implemented in this patch.  Would be nice to get comments from others, though.


develop (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Fixed in https://github.com/CUTR-at-USF/onebusaway-application-modules/commit/84f7a6eb92b92ac0e817349729ff192c5e820735.


is this annotation necessary?


must be Qt version. didn't test, but should be qt4 trouble

when there's no auto-conversion from QColor to rgb().


свушь?


I just thought that maybe http://www.repairfaq.org/sam/repspec/ could have been a better reference, as it is accessible (even google cached) and the process is actually explained there. :) But I'm probably being too...I am going to say 'thorough'. :P Sorry.


I'm not sure that they are the best


Need to bump `serialVersionUID` to `2L` in `TimepointPredictionBean`.


Mark: Where will the VM get the guest additions if we don't provide the ISO?  When I run the setup and subsequent vagrant packaging I get a VM that does not have guest additions installed.


Nit: private functions should use `/*` not `/**`


NO!


Kent, @kshefchek  I added rm -rf `node_modules` about 30 minutes ago, then I saw this merge message 20 minutes ago. I'm very confused. Did you have to merge my change? I didn't send a pull request though.


@vinniefalco: in `Env::close` you seem to call `setAccepted` with `correctCloseTime = true` so the code should treat the time that you pass in (`tp`) as the exact time and not round it.


Nice tsunamical, I'll have to take a look at this now.


Interesting. I was always under the impression that hosts could update any part of the profile! They should def be able to edit any part they want, including topics, photos, etc


Oops. By default it appears to log everything. I was reading the old behaviour incorrectly.


I think this is going to fail on enforce-format. Locally, I'm getting this after rebase, for a file I haven't touched:

```
WARNING: Files are not properly formatted. Please run
./node_modules/clang-format/index.js -i -style="file" angular/modules/angular2/src/web-workers/shared/serializer.ts
```


Removing the order check is not OK. While we do want to skip testing the reordering of the modules based on their priority, it is still important to make sure that once a module has given an answer, the following ones aren't asked.


Put more information


That sounds plausible but @seelabs claims that calling
```
env.close(t);
env.close(t+seconds(4))
```
leaves `env.current()->info().parentCloseTime == t`


They have an `_.assign` alias to align closer to `Object.assign`.


This does appear to be correct and maintaining the old behaviour (which I had forgotten). Sorry for the confusion.


My pleasure, thanks for the solid library!


parameters


and


This is always writing `true` for the field in the catalog commands sent to the remote cluster, regardless of the value set in the local catalog. It will not work for active/passive DR.


its not designing, its implementing a design :+1:
lol you guys are awesome.


I think you misunderstand the code—each package_spec_t has a group. That is, the YAML structure:

    packages:
      failsafe:
      - foo-1.2.3
      - baz-4.5.6

Will result in two `package_spec_t` lines, both with `failsafe` in their `group` field. So, it is entirely legal and expected for this line to run after `has_failsafe` has already been set; the check for existence is there to save the trouble of the `strcmp()`.


'generates' and 'copies the'. Also replace 'in AxiSEM' with 'to AxiSEM'.


Good catch


Agreed this is redundant, but the pattern followed in the rest of the code is:

* `malloc()` to allocate memory
* `memset()` to clear out region
* Application level initialization of each field

I like this pattern and think it's worth not hiding the explicit `next` initialization.


Huh, alright.


Is this that 'bug' what I told you?


Done in 425c7cd0ffec89dd70e8d4c6280dc902ffee7653 :)


Hm, indeed. I'll take a look.


Excellent suggestion, thanks. Will amend the PR tomorrow. Brief question: what's the difference of `reinterpret_cast<uintptr_t>` vs. `(uintptr_t)` here?


TL - ArtyomChibiryov, QA - Lymsik.


Actually, the primary advantage of the syntax here is that it matches hostclass files. This is so DWIM that some Groupon Roller users have already (blindly) put `packages.production` blocks into host files, expecting that this override would work. I think adding an alternate syntax would make things really complicated.

I get how your variant is more explicit, but how is the new syntax more flexible? Is it just because you don't need the package name to match what you are overriding?


Isn't this a bit contraintuitive?  The package of the file is google.protobuf, so first place where user would be looking for the wrapper is under Google.Protobuf namespace.

So I think the files should be under WellKnownTypes folder in the project, but should still have the Google.Protobuf namespace. (and that's hard to to currently without a hack in the generate script, right?).


That is pretty crazy.


You should probably just include:

```
pod 'Fabric'
pod 'Crashlytics'
```

The others are unrelated and unnecessary for using Craslytics.


I hate that I think this is awesome.


I think using apache IOUtils would do it, but that's another test dependency to add.


Alas, it is the `hostclass_config_t` and `host_config_t` that are inconsistent. We fairly consistently use `int` everywhere else for booleans inside the code.


What he said.


:yep.


no problemo!


Hi Mikkel,
I've tried using your Kwik IO in OpenElectrophy, but - besides the Neo version compatibility issue - I can't seem to be able to import Kwik data in an OE database. Any advice on this?
(see post here https://groups.google.com/d/msg/openelectrophy/IY_bBaPuRh8/NxTXqFOlATQJ)
Thanks
Vincent


removed options observer, added space in test function.


@yuanzhou I would definitely look for another way of doing things besides "rm -rf", which is a wildly dangerous command that traverses links in an unknown environment. Maybe mode all ./node_modules/ subdirectories to a self-destructing tmp dir or the like.


This award goes to the man with the @stinkyfingers :trophy:


`s/fluxapp/fluxApp` to match above code segment (`import`)


It is only minor PEP8 but can you fix order of imports please.


didn't sound right to me...wanted to change it. :)


Ok, got it, fix coming up


Cool, thank you :)


some new bugs


I think this is the result of merging my local branch with the remote master branch.


Cris é só gerente de projeto aqui é?


Wouldn't that result in an a non-uniform distribution as well?


Does this work if baseDir is given as relative?


look to #with_index method


BTW, for maximum perfomance the expressions could be re-ordered.


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-MacOS-10.9-clang: OK (1691 of 1694 tests passed)

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


Let's be sure that we don't end with duplicated entries in the calavera host file.


I've done this for two reasons:

- To keep the root namespace clearer, as *most* code won't even need the well-known types, I suspect
- To avoid a common conflict with `System.Type`

Without the change, any code with:

```csharp
using System;
using Google.Protobuf;
```

... which also wants to use `System.Type` is going to have to fully-qualify it or have a using type alias. I learned this the hard way, when a lot of code broke after generating the code *without* the change. Far more users are going to need to import `ByteString`, `IMessage` or `CodedInputStream` than the well-known types, I suspect - so let's not make their lives hard :) (In retrospect, I suspect that the name `Type` wasn't a great choice...)


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


Reading List App/Reading List App/AppDelegate.swift

I think this took a step back, you should still do an `if let dict = dict {` check rather than force unwrapping these.
If this is a 'must succeed' section of the app it would be nicer to throw/assert with a descriptive reason rather than confusing people with 'Unexpectedly found nil while unwrapping an optional value'


Comment on why we need genFileName for one and newnamePath for the other


Yes. At first, I was planning on doing something about the strategy but I've moved it after all. I will change that


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-MacOS-10.9-clang: OK (0 of 0 tests passed)

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


This line determines _if the check returns zero_, used by [`wtf -w`](https://github.com/zalora/microgram/blob/eb77584abe2c55988564a4de5c9e621e843f3f2e/wtf/wtf#L43).


It should, but I'll add unit test coverage for this.


@baseigneurie needs a :star: for making this awkward moment possible...