Shouldn't these be linked?


OK


Nice that I removed the whitespace. (test comment)


For future reference: https://github.com/elierotenberg/es6-starterkit/blob/master/.eslintrc


@cmaglie I quickly looked over your commits, and I like where they are going. On thing @NicoHood now has that your commits don't, is making the callback function virtual methods. This adds a bit of overhead, but allows plugging the same code multiple times, without having to duplicate the functions as well. Passing a "userdata" pointer (as discussed elsewhere) might be a bit more flexible, but using virtual methods seems cleaner and less cumbersome and allows the compiler to check more things, so I'm all for going that route.


Fair enough, good catch.


It was a local error that I thought was uploaded here. This commit can be deleted.


I  signed up and still not be able to create a branch


11AC dongle purchased in Japan.


`avocado attempts` instead of `it attempts`?


The `build.bat` script is finally fixed for VS 2015 in 7767e4f5068ab90f6ce9320fcb870cdbd0119052.

If you download the latest source from URL https://github.com/SubtitleEdit/subtitleedit/archive/master.zip, and [run `build.bat` in cmd.exe](https://github.com/SubtitleEdit/subtitleedit/commit/7883b28c264d93c26d7b6b53d95b0c2e422d709f#commitcomment-13460149), you will be able to see the exception message and the stack trace.

Removing those 4 lines from `build_helpers.bat` just ignores the problem, which is fine to force compilation, but not as a long-term solution. :)


`avocado attempts`?


Nice


The commit message associated with the above modification, should read:

```bash
#2141: sample-2.json, remove array structure for 'properties'
```


OK!


I'm guessing you dont want these tests to be run by travis and our CI thingimaboppers, actual network i/o can be funky sometimes, like the time travis broke cos lit.luvit or something was down, but if you did,
https://github.com/virgo-agent-toolkit/rackspace-monitoring-agent/blob/master/tests/run.lua
is a good place to look, basically youd just need to mv this file to  tests/test-remote.lua
Alternatively you can have fixtures like i did with the hostinfo stuff, fixtures go in static/tests/remote perhaps, same place for the test file, but you'd have to make the code unit testable. Lot of rewriting though, i think...


is this change important?  more like, I'm wondering what one versus the other means.


Please read a tutorial on **USING** CMake and how to set options! If you still think you should change any CMakefiles I can't help you.


Unify client and server


This could have stayed in the AI folder, just added a _defines.dm and z_undefs.dm


@dejanpojbic Good work


`try\catch` was removed because there's a `catch` further up the stack?


fixed. if not - provide more info via flowdock


You probably still need to do this here because `visProps` is not available in `Start()` but is currently used. If `g_context` is properly initialized after `ADDON_Create()` the rest can be done in `Start()`. Or is that a problem with the general reason for this change?


We might rope @dscho into this to give you the exact reasons - he was helpful on this and understood more than I do. It was something about the library having to be loaded globally if I remember correctly.


Restrict TLS ciphersuites


Is `g_device` not used at all on OpenGL anymore? If so it's declaration further up could be removed for the OpenGL case.


That's pretty inconsistent with the way we normally handle these, though. And in the future people may want to check whether or not AI's have wireless/radio connections in, for example, computers.


The assets are missing from the repository.


First of all, I love that you've started this club in LA.

IMHO, I would try to wire up the UseCase and declare it as a component elsewhere.  In my attempts at Clean Architecture, I've tried to keep to only using PO(J/R)Os for the UseCases and Entities.  The runnable application which uses the UseCases (Spring here) end up configuring which UseCases they'd like to use.


Disk IO


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


will ucx_assert compiled out? if so - it will compile out --worker... as well


Sorry, I meant to leave that comment as myself. ^


Much nicer!


Maybe name the file so it's clear that it's darwin-specific? I think the .sb file will end up in the Linux share folder as well.


This commit is a total regression of the current pluggable USB.
With this I can only attach a single Modul. The Pluggable USB was supporting 3 before. This IS required for extended HID stuff, so please do not do that.


Yep, sounds better.


Another comment.


Ditto.


Well, the list at the top should be pretty self-explanatory: neighbors, vpn_neighbors, loadavg, rx_bytes, rx_packets, tx_bytes, tx_packets, mgmt_rx_bytes, mgmt_rx_packets, mgmt_tx_bytes, mgmt_tx_packets, forward_bytes, forward_packets.

Yes, everybody could apply this and the data would be added to the RRDs. It wouldn't be displayed anywhere, however. Also, I intend to abandon RRDs altogether in our community and use the recently added Graphite support, which has a command line switch which data is to be saved/graphed.


Hmm. Code analysis would prefer ExpandableQuery<T>  to be sealed...


Ok, agree if this could make things smoother, to me it's just the same destination with different direction.


camel case var name in inconsistent


Oops, there's a typo here also: "have had it running a "while" back..."


Yes I have! I thought the same. I'm glad to help
On 29 Sep 2015 5:40 pm, "Watts Martin" <notifications@github.com> wrote:

> Thanks for catching that -- it looks like a bit of JavaScript syntax style
> that didn't get properly updated to Python's format.
>
> Have you signed the RethinkDB CLA
> <http://www.rethinkdb.com/community/cla/> yet?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bsantanas/docs/commit/3687ea26746a707bc00ddd2173b9bd2332690854#commitcomment-13498066>
> .
>


You are right I'll add it back in, sorry I was trying to refactor the open link to all use randomized UA's.


I changed this up a smidge to get some better (easier to quickly diagnose) column naming.  So at the moment, my model leads to this table:

````
    create table contact_address (
        contact_id integer not null,
        addresses_id integer not null,
        address_type_id integer not null,
        primary key (contact_id, address_type_id)
    )
````

Given this model, a query such as `from Contact c join c.addresses a where key(a) = 'HOME'` leads to the following SQL:

````
     select
        mapfunctio0_.id as id1_2_,
        mapfunctio0_.name as name2_2_
    from
        contact mapfunctio0_ cross
    join
        contact_address addresses1_, address mapfunctio2_
    where
        mapfunctio0_.id=addresses1_.contact_id
        and addresses1_.addresses_id=mapfunctio2_.id
        and addresses1_.address_type_id='HOME'
````

Am I missing something?  Your initial claim was that Hibernate would interpret key() using the value column(s).  But that is clearly not the case here no matter how I try to reproduce it.


oops! sorry..i am on it...function were quite small and obvious so i didn't explain..i am doing it..


English Please!!!!


@mgajre Good Job


Great line.


fast-forwarding master to auto = 11f5d64d


Hello, about to review your file


@SangySantharam  - Make sure you test this . :trollface:


@jasonwilliams200OK Would that even compile? Do you mean something more like:

```
void Foo(const char* file, int line)
{
    UNUSED(file);
    UNUSED(line);
}
```


@christyberman I've got nothing either. Let's ask :8ball:


@zeeshanSD what are you demoing? :ski


@varma-s looks ok


this change change can be delete it after google made the tecla5 verification domain.


test


@Nevcairiel: Is VS2013 still supposed to work? I have an unresolved symbol error `void __cdecl bd_refcnt_dec(void const *)" (?bd_refcnt_dec@@YAXPBX@Z)` when compiling libbluray.


Traducción


I have never felt so in love


Couldn't have done it better myself.  :+1:


Shadows = Shady. Just saying. @ToshForrester where are you?


Comment on header2 :trolleybus:


why did you delete that file?  That was the whole file :-1:


:trollface:


great :+1:


@DAB0mB So you are saying that in TypeScript, for instance, one file can't be dependable on other files?
Ok, what about if I import a class from some file into another file, isn't  it a dependency between files?


Are up aware that all of new leanback apks you added are failing the sig. check?


Changed, thanks.


Is it possible to use a capturing group in `namedParam` and `splatParam` to extract the param names?


Ok, I do see this note about `RTLD_GLOBAL` that he left. It would be good to understand this a little more.

Maybe running python externally would work. However, if loading the library is an issue, we need to be careful to make sure that we are spawning the process. I would think a fork loading this library could cause issues.


Added return statement.  But why did it clobber the comment?


@DeDivers could you move the mp3 cleaning code to the load_raw_mp3 function, that way both that function and this one could use your improvements!


`("|\')` should be simplified as `(["\'])` and exactly one space after `<a` is a bit too strict imho.


@chivygab I've been thinking about this one. I would normally do `a_hash['key'] = 'value'`.

Why did you use `store`? I mean it works, but it looks different enough to make me wonder.

@jaybobo Any comment from any idiomatic Ruby viewpoint?


Any reason not to move this logic into `_routeToRegExp`?


@Fr4nks can you please rephrase the question? I am not sure if you are telling me Sticky.min.js should not include these changes or asking if Sticky.min.js has include the changes.


Is this set anywhere?


Our convention is just for a single underscore at the beginning. `this._routeDetails`. Could we name it something more descriptive than "details"?


Good job :)


BLUE and GREEN services are both using VIRTUAL_HOST=/sort, \*/sort/\*
but other example could be something like VIRTUAL_HOST=service-public-name.com, service-private-name.com
or any other set of VIRTUAL_HOST comma separated.

I guess I more thought comparing the entire VIRTUAL_HOST setting, not the parsed results to tell if it should be same service or else find another way to specify that you want to combine the services by convention like I did in the BLUE GREEN code in my fork?


@mgrauer I made this change to hopefully imporve backward compatibility with older WMS servers.  Here is an example of one I ran into yesterday that had an issue with 1.3 but worked with 1.1.1, which led me to change the version:

Broken with 1.3:
```
http://basemap.nationalmap.gov/arcgis/services/USGSImageryTopo/MapServer/WMSServer?SERVICE=WMS&VERSION=1.3&REQUEST=GetMap&LAYERS=0&STYLES=&BBOX=-8072055.935027753%2C5122198.139446232%2C-8071750.186914613%2C5122503.887559371&WIDTH=256&HEIGHT=256&FORMAT=image%2Fpng&TRANSPARENT=true&SRS=EPSG%3A3857&TILED=true
```

Works with 1.1.1:
```
http://basemap.nationalmap.gov/arcgis/services/USGSImageryTopo/MapServer/WMSServer?SERVICE=WMS&VERSION=1.1.1&REQUEST=GetMap&LAYERS=0&STYLES=&BBOX=-8072055.935027753%2C5122198.139446232%2C-8071750.186914613%2C5122503.887559371&WIDTH=256&HEIGHT=256&FORMAT=image%2Fpng&TRANSPARENT=true&SRS=EPSG%3A3857&TILED=true
```


Good to know that I wasn't the only one lost with pure.css, although you definitely got further with it than I did. Which type of CSS would you say is the best to use? Although coding is a bit difficult for me, I do enjoy it, so I want to familiarize myself with a couple of different ways for future reference.


actually for this use the following should suffice
```c
#ifdef _WIN32
#define snprintf _snprintf
#endif
```


Grate


please dont review last commit, it is really work in progress and not proved to be working, all but last commit would be in working condition.


Good code.


SharePoint page markup


I normally don't think about `case`-`when`. I need to keep that in mind.


This seemed to 'fix' the problem.

Now, I am not sure why, but shouldn't a double quote be ok with square brackets and a period ?

Putting single quotes seemed to solve problem, but I just do not understand it.   It must have something to do with YAML, or something.  Well, this fix gives the 'results' I was hoping for, so I guess it does not matter the 'why', we just need to know to do it this way.

This change we added, so that I could 'remove' all of the disabled dynamic formats, and instead just use any 'base' formats that are listed as being disabled for the CI testing.


Thanks - it was actually `conda install --use-local pymongo` that worked for me here. Pardon my ignorance/n00bism!


Maybe put this on the prototype? Or at least convert to a boolean: `this.enableRouteData = !!options.enableRouteData`.

A better name for this would imply that it is a switch between arrayParams or objectParams, how about `useObjectParams` or something?

Finally, what's the reason for this to be an option? When would you want to mix-and-match routers like this?


I think this was a smidge early.  After a package update, I get

```
WARNING: julia is fixed at 0.4.0-rc3 conflicting with requirement for Lint: [0.4.0,∞)
```

Perhaps `0.4-`


WOW good explain :+1:


Remove trailing whitespace.


ok good code


@jgkamat So move where I strip the metadata to the raw from mp3 function?


I like this


Hmmm... you can still call PluggableUSB.plug(...) three time, maybe I'm missing something?
Can you link your use case?


As far as I remember we actually have a rule to not use unnecessary apostrophes, but for z-index it's needed because it contains dash.


did not work as is, made some changes to make it less verbose.


looks okay


line 46 _in is not initialised before calling nextLine() and causes null pointer exception.


OK, makes sense.


you cannot spell you big stupid


OK.


Why the sanityCheck is not necessary anymore?


@jmatracia  :8ball:


You guessed the main reason.  `expanded` doesn't have enough context, and I was worried that other mixins might also define `expanded`.  I was thinking about making a change in `simulation-store.coffee`.  Ultimately I decided to do this for now.  I think the long term solution would be to change that state variable name in the store though.


Is this also an issue here:
https://github.com/mmanos/laravel-billing/blob/master/src/Mmanos/Billing/CustomerBillableTrait/Subscriptions.php#L3


@chivygab From our Ruby Style Guide, `when` should be indented to the level as the `case`.


Thanks for the explanation.  Yes, I had the comparison backwards.  Returning `tesSUCCESS` on an expired operation is a bit surprising, but I've seen it done elsewhere in the code.


:+1: :ship: :shipit: test comment


test


My preference is to use `path.charAt(path.length - 1)`, since it's generally "safer" (doesn't fail on zero-length strings) and more declarative.


the device has never been used on opengl. it is a dx11 (now, used to be dx9) context. just stupid code.


I think you should add a comment on where this pattern came from.


that is the intention - since this variable is only present in debug mode, to catch recursive progress


Tried denying the perms for delete and I'm not getting the perms right or something because players can still delete anyones plots. I've type the permission node fe.economy.plot.command.delete=false and also tried fe.economy.plot.command.perms.delete=false and still nothing


Okay so I got a few more things done:
https://github.com/NicoHood/Arduino/commit/a1d4fe2cbadfae09da0ee2503300f511c9b6ad52
(with this fixed HID-Project)
https://github.com/NicoHood/HID/commit/f444a00f2d6df070951fa0a66dc79dfc391a159e

I currently have the problem that the usb device takes a 3sec timeout after uploading. I switched back to the master branch and it still occurs. I think I messed up the linux drivers, but I do not want to reboot for now.

endpointType is giving me headache. A pointer is used, to create multiple interfaces. Thats good and makes sense. Then the USBDevice defines the array with the needed size. But the problem is that this array needs to be initialized before the USBDevice (inherited) constructor is called. But since you cannot init an array via constructor this seems impossible. However I was not able to test this behvor since my device goes crazy right now.

Maybe we should limit the pluggable USB to one interface per plug, but with multiple endpoints. A CDC serial might not be possible to create with this. Or we need a static array size variable. This would not hurt in this case since if you want to create multiple (HID( devices the number of interfaces should be always the same and this also saves the duplicated data anyways.

Not sure if you can follow, but work in progress, I promise ;)

@cmaglie We really need a class for each USB Device. Otherwise the code will be duplicated too much.
If this works you can make your Keyboard and Mouse API even more relyable and also boot compatible. Multireports are still possible though. How nice is this?


@Parth-Brahmbhatt I was missing a `.keys` above:

```resourcesToAcls.keys.map(ResourceToCommand).foldLeft(Array[String]())(_ ++ _)```


:tada:


:shipit:


@rejectedOrchid :laughing:


done


**Documentation - Nitpick**

The indentation used to highlight an example of your library method seems to have bad indentation.


Agreed!


falta index


I did. You're looking at an old commit. The correct/new one is 4cb74e3b9f96f17ea26daf3a817f72c0733a561e.


D'oh! Thanks for the catch!


`.[]` is definitely more common but I think its all a matter of preference and keystrokes really.


Yep, I like this.


@ewertonorg aqui faltou acento na palavrá m**ó**dulo, e acho que pode escrever JavaScript :smile:


Good to know.
I would still try to keep all the timezone-related code co-located


OK!


A commitment


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


Rename to avoid conflicts


good


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1360 of 1360 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 doesn't seem to work for me :-1: I need this desperately :(


This is missing an update to the man page.


Wow, good catch.


@jgillis FTFY


do you mean "arial"? (somebody seems to love cartoons :) )


but the function  endpoint() (and interface) will always return the same value. Then all plugged devices are configured on the same EP.

I get when you comment to the upstream or not. but I suggest to create a PR first. I guess you do this, but github doesnt display this clear enough that this is on your fork. I hope so.


Yup, because I had to manually deodex them. I still have to add an extra resultclass that I want to call 'warn' when there is no classes.dex specified in the Manifest. Because in those cases the APK has been deodexed.
Haven't done that because of lack of time, but I am thinking of maybe adding an extra personal PGP signature to those, that people can check that e.g. I did do the deodexing.


This is a sample of the json output and it is lower case in json.


I would lose the order of param names if I used `namedParam` and `splatParam` separately. And I need the order to create the object when the route is matched.

I could update `paramNamesRegExp` to use capturing groups instead, something like `/[:*](\w+)/g`. But then I'd need to do `paramNamesRegExp.exec` multiple times on each route till no matches left, pushing matches to resulting array Which does not sound faster.


True, but they are compiled into separate files, am I wrong? `.sass` files on the other hand takes multiple files and import them into a single file, from what I understood, that's the purpose of MultiFileCachingCompiler, which is not necessary in this case.


I'd put `[^]` instead of `[\s\S]`. Can you amend? Then I'll merge.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (822 of 822 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-24.31%25-green.svg)


OK, I ran the simulation by changing the epoch by subtracting 30 seconds (TAI-UTC conversion) and also with the color conversion that matches S-type asteroids.

The astrometric solution improved in RA a bit but it is still off by -0.49" in RA and by 0.26" in DEC on average.

![comp4](https://cloud.githubusercontent.com/assets/13923689/10172821/4186edfc-6696-11e5-90cf-5293816d19e8.jpg)

Magnitude wise, we have numbers that are very close. On average by -0.001 plus/minus 0.035 mag.

![comp5_utc_tai_mag](https://cloud.githubusercontent.com/assets/13923689/10172842/551dc0e8-6696-11e5-8815-d32fdaa2ce39.jpg)


Yeah I wasn't sure if SignIn needed to be chained.  The question is whether we allow all the HandleError call to be invoked though, I felt it would be more consistent if we enabled that everywhere.  So SignIn also supports calling HandleError(which results in a redirect to the error page by default)


This changes the current behavior which is <PlatformToolset>v110</PlatformToolset>.
It would be better to add one more case like in vcxproj files.


fixes #393


parseFloat shouldn't be wrapped in [[]] for line protocol


I want to keep the `name` as part of the route details.

`_routeToRegExp` doesn't receive the `name`


Pull   1.  Revert  2. Revert 2015.09.29


**Code Quality - Nitpick**

`String.prototype.length` will always give `0`, as it never gets set as a prototype property. This check is negligible/redundant.


Who knows, I switched everything to 2015.


To me this is a blurry place.  The `JLOG` is becoming idiomatic, so when it's missing I pause to think about why.  But you're right that invoking `JLOG` on 3 lines will give 3 tests, when one will do.

I assume that `j_.trace` is invoked 3 times to get newlines?  If that's the case, could there be one invocation of `JLOG` and a couple of extra `<< std::endl` uses?  Would that put the equivalent text into the log?  I dunno.  It's probably not worth thinking about too hard.  What you've got is fine.


Yeah I wasn't sure here, the old behavior relied on an exception, and we were just catching it and returning an empty ticket...  Feels like we should raise an error here instead.  But I didn't want to drill into the specifics of each provider yet.  I just want to make sure we get the shape of the error handling down first before doing the case by case review


Totally! I am terrible at naming, I'd need help on what else to call it.

`routeDetails` isn't really good here, since it's really `array routeDetails by route regexp`


blah


Can we move this to an html file?


The default base implementation of HandleError logs all errors.  That's why I nuked all of the log messages surrounding the new error path


Yes. https://github.com/ximinez/rippled/blob/preclaim/src/ripple/app/tx/impl/apply.cpp#L237-L259


It should be rather `self.cp.getboolean()` than that. Take a look at `keep_sorted()` below.


here is a comment for this line of code :100:


> but the function endpoint() (and interface) will always return the same value. Then all plugged devices are configured on the same EP.

No, they returns the value of the member of the PUSBListNode class (that now is the equivalent of your CUSBDevice!). I think that, besides naming, our branches are "compatible" at 90%.

> I get when you comment to the upstream or not. but I suggest to create a PR first. I guess you do this, but github doesnt display this clear enough that this is on your fork. I hope so.

Ok, let me make a PR, so we can follow this much better.


I'm just a big fan of 80's New Wave bands.

...Fixed.


Haha nice catch! Though I think it was probably Ariel Pink rather than the cartoon character.


Then I guess we will wait after we release a stable version to update LAV.


Так всё-таки писать или не писать flush? Учитывая что BufferedWriter делает его автоматически при переполнении себя и при закрытии файла.


## Commit 3


id lft rgt depth
1 0 2147483647 0
2 920350135 1022611260 1
3 1124872387 1227133512 1
---------------------------
$node2 = Document::findOne(2);
$node3 = Document::findOne(3);
$node2->insertAfter($node3)->save();
----------------------------
1 0 2147483647 0
3 1022611261 1124872386 1
2 1124872387 1227133512 1
-----------------------------
$node2->insertBefore($node3)->save();
-----------------------------
SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '1124872387-1227133512' for key 'lft'
The SQL being executed was: UPDATE `document` SET `rgt`=`rgt`+102261126 WHERE (`rgt` BETWEEN 1022611261 AND 1124872386)


@DAB0mB Ok, I didn't work with this stuff yet, I am trying to clear it up.
So, lets say above mentioned class has changed, file that has this class will be re-compiled, that's ok, but should another file, that uses this class, be re-compiled too?


I would tend to agree. At this point this is a patch. The real solution is
to rewrite it...

Dave Cramer

On 29 September 2015 at 13:34, Vladimir Sitnikov <notifications@github.com>
wrote:

> Good to know.
> I would still try to keep all the timezone-related code co-located
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/davecramer/pgjdbc/commit/19699cdc10cbd02abb396ddbebed5eaed425bd62#commitcomment-13499257>
> .
>


A good idea!  I've updated the PR.  (I modelled my `_resolve_param_value_from_text_by_name_or_id` on the one in `Genome::FeatureList`.)


lol, wow, didn't even notice until now.

At least it doesn't do anything bad.


- I am not quite understanding how putting on prototype is better - honest question!
- I should totally convert it to boolean, thanks for the catch.
- `useObjectParams` is a great name, I struggled quite a bit with the naming of this option. Although the object being passed to route handlers is not just `params`. It has `name`, `params`, `queryString`
- Having the option to make it opt in only is mostly for backward compatibility. But if this feature could replace the current array, sure. It can be removed.

Also I noticed that this logic does not work well when the route is not a string, but a regular expression itself. We won't be able to create object params.


Yes, I've fixed this up in my squashed version.


Trying again to commit.


**Test Coverage - Nitpick**

You should use the correct operator for your tests so that the test result actually gives the correct context and failure message.


Strange. In fact, no quotes should be needed at all.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 736](https://teamcity.labkey.org/viewLog.html?buildId=274908&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **SUCCESS**
Summary: Tests passed: 9, muted: 34 Build time: 00:28:02


Doesn't appear to.  Nice catch.


No, the other file which depends on the class stays the same, if you change the class's api it's up to you to update the other files, it's not the processor's job.


Shouldn't "1000" be stored in a constant?


???
OK. I'll fix on merge for your `filter` commit and after rebase.
BTW: just was going to merge to `master`... :smile:


I decided to modify `simulation-store.coffee` and use the mixin.

It might seem like it isn't dry, because I added a new state variable `simulating` which for now is identical to `expanded`.

Your thoughts on this welcome.


Are we sure we are cool with dropping 3.2 and 3.4? F22/23 for e.g. are still on 3.4 - 3.5 will land in F24.

This just looks like upstream breakage in the nose coverage package for those versions so I am fine with this as a workaround until that is fixed but I'd like to get at least 3.4 back on the matrix if possible.


is this correct?


This is just a placeholder. @James0x57 is looking into common styling for those pages, once it is up I will finalize this one.


I just dropped 3.2 and 3.3, did you want 3.3 back?


k


Shoudn't if include test for ctrlDown?


Ugh. Nm. I am blind. And dumb. And unable to read. It's all those '-' :-) :+1:


That is one possibility. autogen.sh does this for you (submodule init and update) as well as autoreconf.


haha np, I think though I may look at adding 3.5 if travis supports it yet just to be prepared


:watch means you want to pick up updates that have been published to the local mvn repo.  So there would be no surprises.  This issue applies to any distro that does not use a docker based assembly.  This includes karaf distros.  In general custom distros will run into this issue.  I personally don't think the docker plugin should be creating application assemblies since that's agnostic to the Docker.  This is true for RPM based apps and it's true for java based apps too.  Docker and the docker maven plugin should just focus on install app assemblies into the container.


You need to restore this block when `private static void ensureReadable(final ChannelBuffer buf, final int nbytes)` doesn't have enough bytes in the buffer it throws an exception and the request will need to be replayed when there is more data in the buffer.   If you remove the `rpcs_inflight.remove(rpcid)` you won't be able to decode the replayed request.


There's a duplicate test assertion. Lines _89_ and _90_.


Hmmmm - I added it to .gitignore


@jisikoff

I have talked to @fermayo about it. And we decide to do it only based on a string match of VIRTUAL_HOST, which means `abc.abc, xyz.xyz` and `xyz.xyz, abc.abc` will not be merged together.


Nevermind,  it  reorders correctly on responsive  http://codepen.io/anon/pen/qORbgY
but in my app it does not since 11.1, trying to figure whats wrong..


Doh; sorry


I pushed a fix. libbluray is build as a C project in VS2015, so it works much better there, and C++ in 2013 because the C compiler is too dumb before 2015 - so linkage errors won't immediately show up.

But nevertheless, I don't test with 2013 anymore.

Regardless of the fix, I would still prefer to stick to the LAV release for a stable mpc-hc release, unless you don't plan it for like the next half year :p


You should probably use window.localstorage in if


merge


What about dropping `virttest.data_dir.get_tmp_dir()` and using `avocado.get_tmp_dir()` directly?


@timbeardshear what pages would use this new template?


I assume you've looked into what Gecko does and this matches?


@tvaught , I'll take a look at it tonight. The general approach has been to build charts around table-like data. For a table, the categories are specified by inspecting all values in one of the columns. What I tried to do is support more simple input types by detecting them and quickly converting them to a table-like format. Instead of specifying what the categories are, everything is derived through specification of the one to many columns that contain what previously was told to the chart.

Instead of a df, a dict of lists is similar and should be compatible. `{'a': [1, 2, 3, 4], 'b': ['some', 'list', 'of', 'items']}`.

This would be input at data into `Bar(data, label='b', values='a')`

If you want to take a look before I get back to this, [here is the current examples script](https://github.com/bokeh/bokeh/blob/master/examples/charts/file/bar.py). It uses dataframes, but a dict should work as well. I recommend running the example at this point to see how it is being used. The output is below:

![image](https://cloud.githubusercontent.com/assets/452052/10173356/67b21462-66a9-11e5-894d-b0e5595907ee.png)

If you do see an issue or regression, definitely open one up so we can track it.


ha, github doesn't show that I posted the entire hotgym dataset inline here.
`[[21.2 false 0] [16.4 false 1] [4.7 false 2] [4.7 false 3] ...]`

Apparently it doesn't like lines with 66000 columns.

Really it should be in a separate file, otherwise every demo downloads this data as part of comportexviz.js


Fixed


Looks fine. Let me support that by running a couple of functional (manual) tests.


shared code with ``fs?``


What the crap? The fact that it rolled all the way down without regard to item count was what made this item awesome. Talk about downgrade!


Thanks for the fix.

> Regardless of the fix, I would still prefer to stick to the LAV release for a stable mpc-hc release

That's the plan in theory, I just doing a quick update to test a fix in ffmpeg.


Well, I think it's not an argument. Type-checking is really powerful and useful thing in TypeScript and you just want to get rid of it.


@jesilz I'm planning on using it for the /about/location page.


You could also just set the order on the entities you already created. That'd remove the weird ambiguity around why the first two entities are first in order.


Those static members needs to be moved inside the class. (maybe add this in the next commit)


C:\Program Files\SF-Express\sf_input_client_new\data


This ``editors.komodo`` would be a good separate commit so everyone can use it!


`os.path.join(data_dir.get_tmp_dir(), "libvirt-autotest-vm-create.lock")` is a lot more Pythonic and "feels" better IMHO.


@rejectedOrchid now *explain* the precess


Now that Osmond is using the airhorn.ogg, how do you shut it off. I love the announcement of my waypoints but the airhorn is startling and unnecessary. Can the horn be shutoff and the announcements be on?


giggle


could use some tests for this.


These seem like pretty small residuals, although still bigger than the expected error.
Can you comment on the single object in particular (https://github.com/testMOPS/mops/issues/10), because I don't understand what's going on there.
I can match python + command line oorb, for orbits with epochs of 49353, 50100, and the original PSMOPS epoch (54466), but none of these seem to match the MOPS calculated position for the same object and also don't match my previously calculated position (which came from an old "propagated to 49353" version of the orbit .. I'm guessing my old propagated elements have some difference compared to how OpenOrb runs now, perhaps in the number of perturbers or how the code was configured when I generated the old version of the elements).
I'm thinking because I seem to have found some problem with the old propagated version of the elements, I will have to re-generate all of these Maf n747 detections using a new version of the elements. That may fix the problem, but I'd like to understand what's going on with that single object before I rerun everything. (especially since I still don't match the MOPS position!).


this file duplicates ``work/allegro5/version/version.factor``


Hey Jake, I been getting this error every time I test my code. Did you run into this?

======================================================================
ERROR: Failure: ImportError (No module named task_02)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/nose/loader.py", line 418, in loadTestsFromName
    addr.filename, addr.module)
  File "/usr/local/lib/python2.7/dist-packages/nose/importer.py", line 47, in importFromPath
    return self.importFromDir(dir_path, fqname)
  File "/usr/local/lib/python2.7/dist-packages/nose/importer.py", line 79, in importFromDir
    fh, filename, desc = find_module(part, path)
ImportError: No module named task_02


Doesn't always exist for me. Could use `touch` to make sure the file exists:

```
sudo touch /tmp/xdebug-remote.log
sudo chmod 666 /tmp/xdebug-remote.log
```


Wicked cool.


webView is a weak reference which may be nil in runtime, so it should not check with assert.
However, typeInfo should always be nil before calling the function, otherwise it's a logic error which should check with assert.
So I prefer:
```
assert(typeInfo == nil)
guard let webView = webView else { return nil }
```


We need to check the return value. since the limit can be reached fast (with different HID reports via endpoint or with a u2 device). I'd add a TODO mark here. This requires of course some further changes/checks inside the API. (shouldnt be the problem right now, just for the TODO list)


OK.


I took a full day looking rasonador compatible with OWLAPI 4.0.2. Are you sure it is compatible ?.
"Snorocket" say it is compatible and when I tried, everything was fine until I got to "NodeSet <OWLNamedIndividual> reasoner.getObjectPropertyValues individualValues = (i, p);" "Jajajaj" ... right there launched a execpt


Sorry for digging it up, but this is beautiful :100:


lachete teo =))


Xperia KING :) :) @Alx31


TeamCity pull requests :: netty-tcnative [Build 10](https://garage.netty.io/teamcity/viewLog.html?buildId=1835&buildTypeId=pulls_netty_tcnative&guest=1) is now running


This does not work with splat params, this does though:

```
/(\(\?)?[:*]\w+/g
```


added comment to commit


It's certainly not an invalid flag.  Perhaps just plain-old `temINVALID` is a better fit, since you probably don't want to create a new code?


TeamCity pull requests :: netty-tcnative [Build 10](https://garage.netty.io/teamcity/viewLog.html?buildId=1835&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:51


`os.path.join()` should probably also be used here.


Commit message was intended to be:

Bug solved: October (10th month) tasks not shown

- A date construction problem: 010 instead of 10


Agreed.


Another good candidate for `os.path.join()`.


double reader1


PRs welcome!


Because than I'd have to change indexes and stuff on all other strings. ;)


Presumably this is what Gecko does too?


Okey dokey, please feel free to use the plumbing in this project and configure an additional nginx Docker container, plus command line option(s) for build.sh.

-- ab1


Precede this line with `eor %1, %1, #0x40` so that you don't need to invert Y in C++ later.


typeInfo should never be nil while calling unbind, otherwise it's a logic error. Assertion is proper.


So here we're using `data_dir` straight from Avocado. Maybe my previous suggestion is a good idea also for uniformity, though I'm aware a lot of code uses `virttest.data_dir` for things that `avocado.core.data_dir` doesn't have.


We should be good here as the ``ensureReadable(buf, size);`` at L1374 will force replays until we have the entire response in the buffer. (well, for 0.96+ at least). Then we'll advance the index to either the end of the buffer or the start of the next RPC.


Minor formatting issue, can you align ``{USB_DEVICE...`` with its below lines?


Hi, yes, that's where I got the initial code from :) I would have used your plugin but it didn't work with LiveScript classes and replaces the method completely (AFAIR). So since I patched core for LiveScript to work, that was the only way to go. As soon as Nate has something better, I'll switch.


No worries. I spent an annoying amount of time head-scratching, trying to figure that one out.


lol no ;)


Hmmm, but `simulating` doesn't really describe what it's doing either -- it's not simulating, you just have the panel opening. `simulating` might be useful in the future when we're either in live mode or running a simulation with slow steps.

In any case, if the nodes need to know specifically that the panel is open (since the feature is to show the slider when the panel is open, not when we're currently "simulating"), it seems that this really is the exact same thing as `expanded`. The node need to know if we're expanded.

To me, changing it to one variable, `simulationPanelExpanded` or something, makes the most sense. I don't really get the redundancy of having `simulating`, when what we actually want to know is whether the panel is expanded.

But I'm perfectly happy with this if you think otherwise :)


If we are start checking the control tint, we should also react to control tint changes


first  pull


This is very helpful.  Thanks Nick.
> On Sep 29, 2015, at 12:57 PM, Nick Roth <notifications@github.com> wrote:
>
> @tvaught <https://github.com/tvaught> , I'll take a look at it tonight. The general approach has been to build charts around table-like data. For a table, the categories are specified by inspecting all values in one of the columns. What I tried to do is support more simple input types by detecting them and quickly converting them to a table-like format. Instead of specifying what the categories are, everything is derived through specification of the one to many columns that contain what previously was told to the chart.
>
> Instead of a df, a dict of lists is similar and should be compatible. {'a': [1, 2, 3, 4], 'b': ['some', 'list', 'of', 'items']}.
>
> This would be input at data into Bar(data, label='b', values='a')
>
> If you want to take a look before I get back to this, here is the current examples script <https://github.com/bokeh/bokeh/blob/master/examples/charts/file/bar.py>. It uses dataframes, but a dict should work as well. I recommend running the example at this point to see how it is being used. The output is below:
>
>  <https://cloud.githubusercontent.com/assets/452052/10173356/67b21462-66a9-11e5-894d-b0e5595907ee.png>
> If you do see an issue or regression, definitely open one up so we can track it.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bokeh/bokeh/commit/e7ce2a48f3c8794e6a2985b21bef75b357f77355#commitcomment-13499725>.
>


Perhaps file an issue on this?


Hi Rich,

I've fixed those issues up and re-committed. Let me know if there's anything else.

Cheers,
Bill


Iḿ thinking something like this:
```
getSceneName: function(tvdbID, name) {
    tvdbID = parseInt(tvdbID);
    var exception = (tvdbID in exceptions) ? exceptions[tvdbID] : name;
    return exception.replace(/[^0-9a-zA-Z- |[12][09][0-9]{2}]/g, '').replace(/[12][09][0-9]{2}/, "");
}
```
This will strip out anything non-ascii and full years.
My tests have shown that leaving 'and' in is not really causing any problems. Please give it a shot an see if this will do :)


I assume that `j_.trace` is invoked 3 times to get newlines _and_ the log line headers, so the columns will line up. I don't know of an equivalent way to do that inline.


s/boolean/getboolean :)


Thanks, I don't have menu icons at all. Pushed a fix, see if that does it


I agree to remove this assertion because javascript side may send a wrong instance id. but It better to raise a warning message.


Lachetele nu vazu


Sorry my bad


Supposedly yes. If you know how to fix please open a PR.


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


No longer need this String buffer.


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


I thought about using a function instead of a variable "need_update" to
minimize the quantity of code, but there are templates here so it's a
little messy and the test is not done so many times (3 if I remember
correctly).
Yeah for the naming convention, I thought about it after commiting, but
it's extremely easy to fix.

2015-09-29 16:33 GMT+02:00 Marc Zinnschlag <notifications@github.com>:

> This looks correct, but I don't like it. Too verbose. Not sure about a
> better solution though. Maybe changing updatePtr to take two arguments
> (from, to) and only change the ptr to to, if it was from originally?
>
> btw. need_update is not consistent with our naming style. The right name
> would be needUpdate.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zelurker/openmw/commit/1d806bc40ed157f979c3c1a7765a8933d2c49336#commitcomment-13494893>
> .
>


This will also help Beignet's direct calls into for GPU plugin based builtins.

I see now that QA have verified.  The patches look sound.

lgtm +2


Good point. I'll remove the `--user` from the existing lines, then.


Doing nothing ? It seemed a little too much for me, but if you prefer this
solution, go for it !

2015-09-29 16:29 GMT+02:00 Marc Zinnschlag <notifications@github.com>:

> That does not belong into the parser. The right way to implement fixme
> (even a no-op one) would be to add a new script instruction.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zelurker/openmw/commit/c0d7a6b9812255f4354467affae79017beda77e3#commitcomment-13494767>
> .
>


@rdvincent @andrewjanke  what do you think? This speeds up file access to MINC2 files created with MINC2  API dramatically, but uses about 10 times more memory:

(old code)
\time mincstats -sum library_stx_sym/local_model.mnc
Sum:               2048604543
54.20user 0.39system 0:54.59elapsed 100%CPU (0avgtext+0avgdata 10540maxresident)k

(new code)
\time mincstats -sum library_stx_sym/local_model.mnc
Sum:               2048604543
1.32user 0.05system 0:01.40elapsed 98%CPU (0avgtext+0avgdata 109812maxresident)k
0inputs+0outputs (0major+27689minor)pagefaults 0swaps

$ mincinfo library_stx_sym/local_model.mnc
file: library_stx_sym/local_model.mnc
image: unsigned short 0 to 65535
image dimensions: zspace yspace xspace
    dimension name         length         step        start
    --------------         ------         ----        -----
    zspace                    311          0.3          -78
    yspace                    313          0.3         -110
    xspace                    475          0.3        -72.8

Should it be configurable with environment variable perhaps?


A little better maybe...


I think this is a good example of "the medium is the message"—the way computers think, and the strict way they interpret instructions (like in CSS), has shaped how we need to communicate with and through computers.

Think about how much more difficult it would be to tell computers to interpret a message written way A, way B, and way C as the same thing. By giving code writers a limited language structure, we reduce the steps a computer has to go through to identify the command. In some ways, this makes communication using computers easier (one way to write things) and in some ways it makes it more difficult (one missing character breaks your code).


(y)


Собственно вот это было изначально плохо.. А мы не заметили. 75422cf6b95cc46c36c43b83965b135809c961df — имеено это исправляет :smile:


So close... fixed in 057c0cd


@nataliejeremy this last edit seems to confuse what the blockers are.

Blockers for d8 sites to enable apache solr are:
1. Noted contrib modules have not released a stable version, only an alpha release.
2. Development finalized for pantheon_apachesolr module

They are not dependent on one another, unless I'm mistaken. ping @bmackinney


Good call. Fixed in 057c0cd


Put everything inside a function.

'use strict' should be done in a function by function basis: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Strict_mode#Invoking_strict_mode


RIP in Pieces :(


不建议将 CSS 写在 HTML 文件中，应该分开来，便于后期维护


Well, it works for me. No issues so far


Ouch! Good catch. I guess I'll need to find a spec that makes this fail.


change into raw string literal..


NoArgsCommand appears to be deprecated starting with Django1.8 https://docs.djangoproject.com/en/1.8/howto/custom-management-commands/


HTML 是树形结构，所以此处请注意对齐，这样方便版本控制


It works, but unfortunately we are no closer to fixing the root cause, which is the premature return after 100ms in the ARCH condVarWait()....


No need to apologize. And yeah, @megawac is a wizard with Underscore/Lodash :)


TAB 你们可以协商好，是4还是8，确定后就不要变了


I had the same issue with saving + opening a new file, until I tried refreshing the HTML page in my browser after saving. Maybe try that and see how it works for you?

I started without using a framework, too, and after applying pure.css to my site, I really regret it. It looks like the naming conventions pure.css uses are different than what I came up with by looking at the book and internet tutorials, which made applying pure.css styling that much harder.


@lukasepple Genau das wollte ich auch vorher schreiben, hab es dann aber gelassen.


agreed.


La till översättningar för popupfönster, dock är "Default Status & Type" fortfarande på engelska.


Filename is one word though not two: http://dictionary.reference.com/browse/filename


good catch


You are right. The graph panel is interested in if we are in 'simulation-mode'  (not `simulating` per-se)

For now I will concede that `simulationPanelExpanded ` might be a good proxy for this.  Seems likely that some other state variables will be required in the future.


This is another case of copied code. The original is at: https://github.com/ximinez/rippled/blob/bdbbe30cd4a8e9f5a11f1464a9685e270c101efa/src/ripple/app/tx/impl/Transactor.cpp#L600-L609. It also uses `temINVALID_FLAG`. Also note that I didn't remove that corresponding return blocks in `operator()` because I didn't want to entirely rely on `assert`s yet for such mission-critical (and cheap) checks.


I think this check duplicates the check in `preflight0()` which is now unconditionally called at the top of this method.  Remove the check here?


Filed #7788


Variables must be lower camel case. All caps is reserved for constant names. This var would be called js.


Huh. Maybe I was missing that boardconfig flag, but I had to switched to by num partitioning first


Yes, this Pellet release has been tested in the ORE 2015 competition and has worked well - did not win any prizes but it succeeded in many reasoning tasks, so I'm confident it is compatible and usable.


Hey, Juliet! I took a look at your links, and I have an idea why they aren't working. With HTML, spacing and how you write the code is really important. This is what is in your file `<a href = "http://julietnakasi.weebly.com/"> </a>`. If you want, you could try writing it more like this, and see if it works for you: `<a href="http://julietnakasi.weebly.com/">My website</a>`. You can replace the words "My website" with whatever you want, but that's where you put the text for your link.


This is really really minor but can we do `len(args) > 0` to explicitly coerce to a boolean?  I much prefer this behavior from more strict static languages rather than trying to remember what values are true in which languages.


LGTM


It seems that the patch doesn't fix the problem. Just a workaround to avoid assertion?
According to the [TN2277](https://developer.apple.com/library/ios/technotes/tn2277/_index.html), the httpserver should be stopped when app running in background and restarted when resume.
This means, the port number should be allocated from registered range (1024 ... 49151), app should remember the port number for restarting. So, `start` method should have a parameter for port number.
What do you think?


Yes. The corresponding Gecko code is in [gfxScriptItemizer](https://dxr.mozilla.org/mozilla-central/source/gfx/thebes/gfxScriptItemizer.cpp). It more complicated because it handles paired bracket characters (e.g. parentheses) specially.  My code has a TODO comment for this, and I can file a follow-up issue for it.


theres a cool solution to the consecutive subarray one

you sum up all the elements to that point, so a[i] = a[i-1] + c[i]
c = 1, 5, 4, -2, -2, 7, 5
a = 1, 6, 10, 8, 6, 15, 20
and you see that there are two spots that have 6. If we take the index of the first one to the second one,
we have an array that sums up to 0.

this one finds all the number of subarrays that sum up to 0, so instead of returning true or false, it gives the number.
https://github.com/bradyz/sandbox/blob/master/challenges/ut-acm-9-18/zero.py


Could you please file an issue ticket for this bug?


Same applies here. Method name starts lower case.


Weird! According to the warning message,
```
  No valid AssemblyVersion in template file '...\src\Properties\AssemblyInfo.cs.template'
```
it would appear that something is wrong with the downloaded ZIP-archive?!

[Line 34 in `src\Properties\AssemblyInfo.cs.template`](https://github.com/SubtitleEdit/subtitleedit/blob/master/src/Properties/AssemblyInfo.cs.template#L34) seems to be incorrect in your source?


We already check for color tint in New Solution Dialog, but I don't recall if we detect changes. @mrward, do you know?


I already added tips in my plugin; check it out :P


bogdan.... mai taci dracu!


@Phantom66

listo, problema solucionado!!

hice un script para ayudar a la instalacion del sistemita

mira [nueva parte de la documentacion](https://github.com/slayerfat/sistemaPCI/blob/0667b3dfcaffc827bb88d0f322801311c1aa8d7b/README.md#instalacion-rapida)

dime si te funciona!


...However, given the descriptions for the two errors (https://github.com/ximinez/rippled/blob/preclaim/src/ripple/protocol/impl/TER.cpp#L123-L124), and where each one is used in the transactor, I agree that `temINVALID` is better in both cases. I'll make that change.


It's fully intended


@jmchilton


@barbatus Your'e right, I'm just new to TypeScript, is there an easy way to get the dependencies?


i proposed a fix in https://github.com/NixOS/nixpkgs/pull/10137


Do you really need this map? See below.


Please put the UUIDs in constants:
```js
const CANDLE_SERVICE_UUID = 0xFF02;

```


Why not get the characteristics here as well? Once you discover the services, getting the characteristics has no cost.


Could you use the service name instead?
```js
servier.getPrimaryService('battery_service');
```


If you cache the characteristics by uuid this could be changed to:
```js
return this._characteristics[CANDLE_DEVICE_NAME]
  .this(this._decodeString)
```


You can get rid of this code if you cache the characteristic when you discover the services.


I don't think you need to cache the services. The characteristics should be enough.


Needs the role API before I can finish #32


`dyn.torque+=node->ori*T`


I know you were saying that these comments felt like they were getting verbose.  But I like having this comment that justifies why an `assert` is valid.  Yeah, like all comments it may get stale.  But it gives someone a starting place for where to look if the `assert` fires.  That's a long-winded way of saying, "Nice comment."


All these 3 assertion are checking for logic errors. Are you sure they can occur reasonably?


The link says "Not found" :/


@CheeseWiz, @philkr, @akosiorek

I want to add a target to build the tests without running the tests. How do I do that? I tried just changing `add_executable(${the_target},...)` to `add_custom_target(...)`, but that didn't work.

I want `make test` to generate `test.testbin` without actually executing it - matching the Make build.


Typo: should be `$transparent-wash`.


ss


debug/ in whole is hugely outdated, I removed it from the main branch, at least for now


is this the best way to do this?


And I'd like to raise an exception in place if really necessary.


llenandonos de urls


nit: space before `=`


nit: one space after `return`


It's an option, but it implies changing a lot of calls throughout virt_test.


You can also do some mild Sass magic and reduce this to:
```sass
$transparent-white: rgba($white, 0.6);
$transparent-black: rgba($black, 0.3);
$transparent-wash: rgba($wash, 0.96);
$transparent-stick: rgba($white, 0.96);
```


I /think/ we're already returning much of this information from the driver already.  Check _sessionCapabilities in tvdriver.js


Hello,

@a2800276: Your help would be very much appreciated, though your will come handy reviewing the pull request. By the end of the week I'll be pushing back the changes.


You can change it back to one minute


Sorry about that, I used a temporary hotlink URL. I linked to the google drive shared image now and it should be fine.


Ok, so it only happens when the key is an entity and when the key() expression is in the select clause.


So CSS is CSS. Pure.CSS is a framework that works without javascript -> It's "Just purely" CSS, no Javascript.

How you put the CSS on the pages is the difference.

Inline CSS looks like below and will only affect that element:

External CSS is kept in it's own file, say style.css.  Pure.CSS uses external CSS, because it's made by a third party and that's the easiest way to pass the code to anyone who wants to use it. Your page was being affected by a file they made and you linked to. The inside of that style.css, there may be the following code:

    img{
             height: 90px;
         }

And the webpage using it will link to that file like so:

          <link rel="stylesheet" href="style.css">

Internal CSS is kept on the webpage is it being used on in between style tags. This may be easier if the code won't be used anywhere else, ever.

      ....

Now the 'problem' with pure.css is it came with it's own font. The people that made pure.css made that external CSS file, and you would not be able to edit it; you would have to work with them. What you could do in the future is:

1) Completely copy pure.css and edit your own copy. Be careful doing this, or you might violate the software's license (https://github.com/yahoo/pure-site/blob/master/LICENSE.md).

2) Try to override the style using inline styles (which usually 'win' if multiple settings are made) or using the !important flag in your own CSS.

3) Try a different framework. If the framework is being too strong, sometimes you just have to use something else.


That fixed it


done


This looks good to me <3


We need a real check against `null` here, AFACT.


@soffes this clearly is making the execute call to never be called more than once. I had to revert to a9ed140 to have it working again.


Aha! this must be why QuartzImageIO isn't working for me.  I'm going to guess that this version of FileIO isn't tagged, so I don't have it.  (...let me go check...)


I'm guessing react changed how it outputs standalone tags like this. Do we have any output tests that test <br/>s?


Message needs to be updated.


I'm ripping the band-aid off and removing these redundant checks.


[2H] Invite Contact and Share items #84


damn, I just realised I didn't set up a `copyright-owner` value in my ensime `.dir-locals.el` so my name has come out here instead of "The ENSIME authors". I'll update that for my next PR.


@lukeyeager

There is the `test.testbin` target. You can just type `make test.testbin` and it will build `test.testbin` for you. Isn't it enough?


Alright, but how is it derived, on what basis? Reference to a book or something like that? I would like to see the formula in vectors, not per-component. Absolute values are not C1-continuous, and that is highly suspicious in anything to do with continuum...


Fair point. I'll add a few to the missing places.


I tried to investigate:

1) I propagated orbit to the epoch of observation by using JPL tools and used oorb to generate ephemeris. The residuals are 0.12", 0.33" (JPL - oorb).

2) I propagated orbit to the epoch of observation by using oorb first and then calculated ephemeris by oorb.
Residuals are -76", 25" !!!

oorb.conf says I am using all planets, including Pluto as a perturber (n-body). I even tried to change the epoch by 30 seconds by the difference is only -1.47", 0.11" (comparing JPL ephemeris shifted by 30 seconds). So, is the propagation and oorb the problem? Perhaps, we can give you orbits propagated by us to the epoch of night 747?


I prefer a dedicated xcode project for the example instead as a target in XWebView.xcodeproj. I'd like to keep the XWebView.xcodeproj simple and readable to easy review and track.


I wrote a fairly length response to your other post about the different types. External CSS is usually the best to use because it keeps things organized, and you can switch out the file on different device types. In-line is best if there's just one, dumb little thing that needs to change, although it should be avoided. Internal is best if your CSS serves absolutely no use elsewhere and will only be used on that one page.


@tenderlove I think this breaks Sinatra (https://github.com/sinatra/sinatra/blob/master/lib/sinatra/base.rb#L367) and another gem I use. Could this change be included in the HISTORY file?


I thought it is good practice to close the file although that did not cause a problem.


How'd you decide on the use of DataTypes? I guess you split the file to have more shorter files?
I kind of feel term Services would be more appropriate than DataTypes?


Yup, its perfect now!


(Yes, we tried to be more XHTML-compatible.)


A real check is needed here as well.


This is the exception that was being thrown own commodity creation prior to this modification:

```
Error inserting bean [class io.pathfinder.models.Commodity] with unidirectional relationship.
For inserts you must use cascade save on the master bean [class io.pathfinder.models.Cluster]
```


Ah.  Excellent point.  Thanks for the explanation.  Carry on.


Yep, that was it.  Works (mostly) on master.  Can we tag this?


I think in most cases you will need to either use invariant culture or use another culture for all calls so I don't think it makes much sense to add the Culture parameter to all methods of the class.


On Tue, Sep 29, 2015 at 2:18 PM, Dmitriy Chukhin <notifications@github.com>
wrote:

> NoArgsCommand appears to be deprecated starting with Django1.8
> https://docs.djangoproject.com/en/1.8/howto/custom-management-commands/
>

Thanks, I hadn't been aware of that.


@djacobsmeyer I don't see anything relating to `.navbar-link:hover` here. Is some code missing in this commit?


Yeah, supposedly the way to fix it is make the one to many relationships many to many.


A real check against `null` is needed here as well.


Nice


@DAB0mB It works through declaration files, I think if a declaration file has been changed, all dependent files should be recompiled. TypeScript compile seems be able to give you a list of declaration files current file use but I think it's not a trivial task to handle this dependency as now. It'll need some advanced transpiler wrapper with API in first place to make it right.

So, lets commit current processor for time being. I have plans to improve how TypeScript is used now and then we'll see how to change processor to handle type-checking.


Source files in text should end with one and only one newline.


ditto


Already ahead of you, see https://github.com/JuliaLang/METADATA.jl/pull/3579


I had to use the `:elixir_quote` module to escape the stack because `Macro.escape` isn't compiled at this point during the bootstrap phase.


Glad to hear it (mostly) works! Hope we didn't screw too much up.


this.recurring suffices the condition


Can you please paste in your RA/Dec/time values that you calculated with
oorb?
Also, here's my oorb config file -- do we have the same setup?

On Tue, Sep 29, 2015 at 11:45 AM pveres <notifications@github.com> wrote:

> I tried to investigate:
>
> 1) I propagated orbit to the epoch of observation by using JPL tools and
> used oorb to generate ephemeris. The residuals are 0.12", 0.33" (JPL -
> oorb).
>
> 2) I propagated orbit to the epoch of observation by using oorb first and
> then calculated ephemeris by oorb.
> Residuals are -76", 25" !!!
>
> oorb.conf says I am using all planets, including Pluto as a perturber
> (n-body). I even tried to change the epoch by 30 seconds by the difference
> is only -1.47", 0.11" (comparing JPL ephemeris shifted by 30 seconds). So,
> is the propagation and oorb the problem? Perhaps, we can give you orbits
> propagated by us to the epoch of night 747?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/testMOPS/mops/commit/f995cb0253c75dcac003e68a62c813c8c5fb22b7#commitcomment-13500814>
> .
>


A real check against `null` is needed here as well, I guess.


Practicing by copying others code and playing with it is one of the best ways to learn, in my opinion. Figuring out what the computer does when I do this or when I do that is how you learn.

I find programming documentation to usually be a terrible starting point because they tend to miss a lot of 'step zero' details. They more interested in explaining specific functions than how to start. Looking at other's code or actual tutorials tends to be a better start.


A real check against null is needed here as well, I guess.


@ragesoss Make sure you only check in the changes to `schema.rb` your migration changes. These look to be from different changes, and that could mess up anyone else's app that pulls this in.


you should cache all messages per this page


dito


Nice comment.  Thanks.


Hi there, thanks for restoring the credits but could you change "Kudomiku" to "kurozael (Conna)" -- thanks!


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

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


Yeah, it was pretty smooth sailing.  Looks like good work, gang!

There's one weird test failure, where it looks like ImageMagick is being used to read the file instead of Quartz.  I don't know why that's happening yet.  I'll go ahead and push the tests, and then continue from there.


akkoord


We should not even be shipping this, but the changes look fine.


if maximaal aantal liters bereikt


Any idea when this might be released?


No additions, only deletions, I love it :)


@eoinoconghaile :+1:


would be good to put each item on one line. I.e.:
```
[
   'wikia.window',
   require.optional('mw')
   ...
]
```


The ```.navbar-link:hover ``` is just a sample to model after. It happens to be just out of view in the main.css file on line 59.


Beter bedrag of saldo mag niet negatief worden


`current_user.update(user_params)` or `current_user.update(status: params[:user][:status])`


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


Postconditie: Wat denk je van saldo verlaagd met bedrag


Right, I just had to add it so that future `make all` invocations wouldn't crash on that.


Merge away.


bedrag is goed


This no longer matches the above require (Pack is now capitalized):
```javascript
Pack = require('./package');
```

Suggest changing this line to:
```javascript
apiVersion: Pack.version
```


This should be the name of the tender entry template {{>tenderEntry}}


Bedrag mag niet hoger zijn dan maximum bedrag (lijkt op kredietlimiet) Kan eventueel met een constructor ingesteld worden


Hi

I using latest version now.

My problem is hls file download slow and some time in speed.

Example

When i play hls stream its play ok Its getting segment from http and play
well till to 10 minutes.

But segment downloading slow .....example ( we have segment 1 to 50 ) when
segment download 1 to 10 then its playing but 11 is downloading if 11
downloading slow then livestreamer timeout occur and it cant fetch segment
11 ).Its wait 5 minute then read.

Example : when download done 11 to 20 livestreamer cant read it .

In this situation video pause 3 to 5 minute.

1.Why its wait 5 minute its fill ring buffer .? if yes which size for
ringbuffer its not wait much.
2.How i can control this timeout with hls segment timeout,hls attempt etc..?

or how i can fix this.?

Thanks
soon reply

On Tue, Sep 29, 2015 at 2:48 AM, Javier Villavicencio <
notifications@github.com> wrote:

> A better implementation of this change has been merged to upsteam
> livestreamer (latest version) you don't need my mod, it's already on the
> latest livestreamer.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/villavic/livestreamer/commit/f3d5359faddffa8ef159fec87b5c68a431611e94#commitcomment-13482100>
> .
>

--
Cordialement,
Khuram


should this still be nullable?


Message needs update.


Please install rebracer extension


If we execute `ensureReadable` doesn't block it throws an exception in `org.jboss.netty.handler.codec.replay.ReplayingDecoderBuffer.checkReadableBytes(int)` which throws an exception.

    private void checkReadableBytes(int readableBytes) {
        if (buf().readableBytes() < readableBytes) {
            throw REPLAY;
        }
    }

When this exception is thrown you need to leave the `HBaseRpc` in `rpcs_inflight` since the Channel is still replaying

The code the consumes the RPC isn;t correct
if this code is executed then the `size == 0` and the pre 95 RPC isn't fully consumed.

     } else {  // HBase 0.94 and before.
      size = 0;

because you don't know the size until `org.hbase.async.RegionClient.deserialize(ChannelBuffer, HBaseRpc)`

What's worse is that you you consume some bytes from the request so those bytes will now replay as if they are rpc responses

Here is a simple log that shows a decode, replay then failure then state  "corruption"

     2015-09-29 17:24:56,520 async.RegionClient DEBUG: handleUpstream [id: 0xeed4b515, /192.168.201.198:50050 => /10.148.196.98:60020] RECEIVED: BigEndianHeapChannelBuffer(ridx=0, widx=1024, cap=1024)
     2015-09-29 17:24:56,521 async.RegionClient DEBUG: ------------------>> ENTERING DECODE >>------------------
     2015-09-29 17:24:56,522 async.RegionClient DEBUG: handleUpstream [id: 0xeed4b515, /192.168.201.198:50050 => /10.148.196.98:60020] RECEIVED: BigEndianHeapChannelBuffer(ridx=0, widx=266, cap=266)
     2015-09-29 17:24:56,523 async.RegionClient DEBUG: ------------------>> ENTERING DECODE >>------------------
     2015-09-29 17:24:56,523 async.RegionClient DEBUG: Received a response for rpcid: 1 that is no longer in our inflight map on region client RegionClient@398818873(chan=[id: 0xeed4b515, /192.168.201.198:50050 => /10.148.196.98:60020], #pending_rpcs=0, #batched=0, #rpcs_inflight=0). It may have been evicted. buf=(failed to extract content of buffer of type org.jboss.netty.handler.codec.replay.ReplayingDecoderBuffer)
     2015-09-29 17:24:56,523 async.RegionClient DEBUG: Skipped timed out RPC ID 1 of 0 bytes on RegionClient@398818873(chan=[id: 0xeed4b515, /192.168.201.198:50050 => /10.148.196.98:60020], #pending_rpcs=0, #batched=0, #rpcs_inflight=0)
     2015-09-29 17:24:56,523 async.RegionClient DEBUG: ------------------>> ENTERING DECODE >>------------------
     2015-09-29 17:24:56,523 async.RegionClient DEBUG: Received rpcid: 33554437 that doesn't seem to be a valid ID on region client RegionClient@398818873(chan=[id: 0xeed4b515, /192.168.201.198:50050 => /10.148.196.98:60020], #pending_rpcs=0, #batched=0, #rpcs_inflight=0). This packet may have been corrupted. buf=(failed to extract content of buffer of type org.jboss.netty.handler.codec.replay.ReplayingDecoderBuffer)
     2015-09-29 17:24:56,523 async.RegionClient DEBUG: Skipped timed out RPC ID 33554437 of 0 bytes on RegionClient@398818873(chan=[id: 0xeed4b515, /192.168.201.198:50050 => /10.148.196.98:60020], #pending_rpcs=0, #batched=0, #rpcs_inflight=0)


Vamos la ver se da certo.


Playbook for testing this -- not the imprecise rounding for some values (ie, 98%FREE of 100MiB resulting in "96.00 MiB"
https://gist.github.com/chrrrles/696e9bba8f13ee90007d


@ejsmith The log message will now output the cache value instead of the existing value. We may want to override CacheValue.ToString().


I changed it, but are you sure? Don't I want the tenderEntries template?


This is disable. Why?
Unit test (for example, [this](https://github.com/XWebView/XWebView/blob/master/XWebViewTests/XWVMetaObjectTest.swift#L18)) relies on the flag.


Actually now that the rebase has passed and I can think more clearly this should be an OR right? We want both the password and the username to be fully populated.


Message needs to be updated.


JPL (MOPS) = RA DEC EPOCH (mjd_UTC):
141.712224022272 23.7960664658642 50100.367067221996
141.7118153071 23.7960976998 50100.36672

OORB (propagated by JPL)
141.7121884745 23.7959724684 50100.367067222
141.7117797625 23.7960037043 50100.36672

OORB (propagated by oorb)
141.7335240091 23.7891842477 50100.367067222
141.7331145152 23.7892157784 50100.36672

(Lynne, where is your oorb.conf?)

By the way, when running oorb, I use --start-mjd-utc option for ephemeris and for propagation --epoch-mjd-tt. But, even when times (UTC, TT, TAI) are somehow confused, I should not have a such a huge difference in RA and DEC.


I think there's are two different problems during setting the root password and removing the anonymous users:
- the elif basically matches everything because the condition is incorrect (noticed it while using arch linux. I also added support for arch linux)
- the endif is incorrectly places and thus anonymous users do not get removed on debian
(still a salt newbie thouth. These changes might considerably change how the module behave!


Message wrong


`gc_content` = `gc_count`/`total_count` where `total_count` is strictly G, C, A, and T count?


nichts von wert ward verloren.


Same here, unmatching message.


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


because the protobuf version hasn't removed the inflight rpc  when `ensureReadable(buf, size)` is thrown the replay works correctly for  `server_version >= SERVER_VERSION_095_OR_ABOVE`


Make a real README.md PLEASE PLEASE PLEASE


Since these are the only plan verification tests for Union, I suggest you keep the plan verification tests (with the modified plan) and file a JIRA for re-enabling the rules.


add checkbox views here


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


Pls file a JIRA for re-enabling the rule in a subsequent release.


Why did you change the default config? This should be trace since its used for debugging when developing.
The logconfig.xml when running the system is places in /etc/stampzilla/nodes/
Thats where you change/add it for production.


Actually, after this [commit](https://github.com/XWebView/XWebView/commit/449e82f87535aa064d95bc8ccc11c9ab7766d21c), plugin can be AnyClass. However, methods must have @objc attribute if the class is not inherit from NSObject.


Haha not sure how I missed that :relaxed:
Thanks @akiomik!


Oh, for bower, yes. npm prohibits uppercase


This should go in the Util package. And please add a docstring ;)


Hmm, if you think you have a solution that satisfies these then please prep a PR and link the other one so we can discuss it!


The doc comment is misleading, will update this.


It's probably clearner to do instead do:
os.path.join(os.path.dirname(__file__), os.pardir, os.pardir)


Message…


Yea you were right about that. You want a tenderEntries template with a tenderEntry template nested in it. You have both of the templates correct but what I was just talking about is the call to the tenderEntry template on 18. {{> Tender Entry}} should be {{>tenderEntry}}


do you really need the committedOffset here?


OMG this feels like another file now ! :smiley:


Wat hip!


Change works if there's only one player. Original message supports multiple players.


拖延症犯了……选择省份学校那个明天再写orz


I would prefer to merge the various OnClickListener/TextWatcherAdapter by having DeviceFragment implement those interfaces, and then do a switch.

Saves a lot of boilerplate code.


Based on what I have seen, I surmise that selecting the map entry() will suffer from a similar problem.


I had the same problem with using the PureCss program after downloading the .zip file, but then I was able to navigate the commands by watching some tutorial videos. However I am not sure if I want to keep exploring PureCSS or if I might want to change over to Bootstrap since that is a more widely supported CSS markup.


Confusing, better use !== null


good catch, removed


This seems like a weird way to test for this. And it'll be broken for anyone using an ES6 polyfill, right?


might want to use `date_effective` rather than `time`


Better `!this.lastCoordinate_`


no comment


nice!


Ah. I see. I only read the email and didn't see what line you were talking about. My bad.


Better `!this.lastFeature`


The idea here was to always consider this as `bytes` even for Py3: the this commit was to use a "escaped-bytes" encoding when creating ids for parametrized tests which received `bytes` objects, as discussed [in this comment](https://github.com/pytest-dev/pytest/pull/1031#issuecomment-141899503):

* In Python 2, we use this "escaped-bytes" representation for any `str` object which is non `ascii` (as `bytes` and `str` are the same type).
* In Python 3, we use this "escaped-bytes" only for `bytes` objects, as the distinction is clearer,

After this logic was implemented, this test started to obtain two separate ids depending if ran in Python 2 or 3: for the former, it would use the "escaped-bytes" representation as the string does not decode from ascii. For Python3 this was a valid unicode character, and the id produced would be `"À"` if I remember correctly.


It's all good


You can just do:

```js
const b = chalk.blue;
``


TeamCity pull requests :: netty-tcnative [Build 11](https://garage.netty.io/teamcity/viewLog.html?buildId=1837&buildTypeId=pulls_netty_tcnative&guest=1) is now running


What's the point of these two functions? If you think mIsCreate is badly named, it would be better to rename it. Or replace it with an enum.


TeamCity pull requests :: netty-tcnative [Build 11](https://garage.netty.io/teamcity/viewLog.html?buildId=1837&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:48


Indeed, looks pretty good! :+1:


My colleague followed the method [here](http://www.orcina.com/SoftwareProducts/OrcaFlex/Documentation/Help/Content/html/6DBuoys,HydrodynamicPropertiesofaRectangularBox.htm) to derive the moment. In the link, the moment is described for a cuboid. My colleague derived it for spheres.


You'll need to import `../../dist/cjs/Rx.KitchenSink` for this tests now. As the operators shouldn't exist on the Observable in Rx.


Better negating?


Is this related?

<code>
D/QSEECOMAPI: (17872): QSEECom_get_handle sb_length = 0x2000
D/QSEECOMAPI: (17872): App is not loaded in QSEE
E/QSEECOMAPI: (17872): Error::Load image request failed ret = -1, errno = 22
E/QSEECOMAPI: (17872): Error::Loading image failed with ret = -1
[...]
E/QCOMKeyMaster(17872): Loading keymaster app failed
E/keystore(17872): could not open keymaster device in keystore (Operation not permitted)
E/keystore(17872): keystore keymaster could not be initialized; exiting
</code>

This started popping up only recently. Still happens on newest revision.


As I said above, I would just have the Fragments implement TextWatcher, and remove this wrapper.


Was this change intended as part of this PR?


isn't elseif more appropriate here to test for future block sessions


No, seems remaining pieces while I tried to introduce inheritance subscriber. Will revert it and update PR.


Hey, can someone confirm about @UltCombo's note about the missing changed line?  Also, I noticed an anomaly:

![image](https://cloud.githubusercontent.com/assets/130580/10175643/26768afe-66be-11e5-96fd-778ca70d1bc5.png)

The line seems to write-over the next line when it wraps.  Odd...


I don't have any statistics, but I do think bootstrap is more popular.

Simpler isn't always better; simpler usually means it doesn't do as many things. But I think exploring both bootstrap and pure.css is a good way to understand the differences between different frameworks and maybe learn more CSS.


It looks like component->prec and component->bpp are always equal in the code. Is this always true?


duplicate code (check line 270 for first instance)


Good point on the polyfills, it's a weak inference for sure. I could do a WinRT weak inference for another API or make the inference a bit stronger looking for the decompiled source.


How about a template in `image.artlabel` mode to generate the inline content for the label, and then call that mode in both places?


asas


@thenickcox We should talk about how this is supposed to be done, then. I recently reset my database, and that extra index was there. I guess I don't under how schema.rb is intended to be maintained. It gets automatically overwritten each time there's a migration, but we're supposed to isolate changes for each new migration?


nts, still buggy but i gues from now everything will be going to cannary

fixes will be coming soon for everything


We transitioned from using usernames to UIDs in the runtime dir once, and this was useful then, as it allowed live upgrades for ongoing sessions... We also made other changes to the runtime dir every now and then, such as giving each user his own tmpfs instance. While the latter didn't cause us to rearrange the directory, the runtime directory has changed every now and then, and having this as part of the saved state certainly did provide benefits for us there...

That said, I am not opposed to just scrapping all that, and declare /run/user/$UID/ API... If so, we should clarify that in the sources however, so that the next one changing the precise location will see this. i.e. place a comment in user_new() where we determine the directory.

Also, we should change the wording on this in our file-hierarchy(7) man page, and indicate that the XDG_RUNTIME_DIR stuff is recommended for compat, but that /run/user/$UID is actually API, too.


Ca c'est moi qui avait zappé un truc, je le corrige mtnt


Ca marche bien sans


`key` variable here is redundant, the for loop could just use `key` variable name.


you only need one getter/setter method for recurring variable where you pass the boolean in the setRecurring method


@torkelo just to be sure I was explicit enough with that exec part. When no global `phantomjs` is installed, current released version of phantomjs npm module default on linux ia32 phantomjs binary on non ia64 linux. So, It can install wrong binary. This `exec` was there to detect such case and skip the copy if needed (exec throw, so end up in catch with warn). This happen for exemple on raspberry pi.
By the way, if you think this should be handled somewhere else (and it should be in the next release of the npm module) I think you can remove the line 18 and the try/catch to just leave the copy.phantom_bin grunt task.


I didn't wanna perform unnecessary concatenation. So I use ',' instead of '+'.  Also, C and Matlab messages don't use concatenation so I'm not used to using that in my print messages. Any particular benefit to using that?


Petite imprécision: $course étant local à la fonction, les groupes étaient ajoutés deux fois pour les cours apparaissant plusieurs fois dans la semaine


sorry i can merge your pull request, i added


Use 3/4 cup instead of 1/2 cup since you need more canola oil than vegetable.


LGTM
Thanks!


Added cursor as an attr by accident


Message…


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.985.bc7200f289265947a05150244ab9b0accb642367](http://build.palaso.org/viewLog.html?buildId=63869&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 541, ignored: 6 Build time: 00:03:32


Well, I always though that maybe we should actually rename `user@.service` to `systemd-user@.service` or so, in order to clarify that that service is not going to contain anything else but a systemd instance... Making this stored has the benefit of still allowing this...

But then again, I am also open to just declare this stable and drop the code to save/restor things.


(moving your calculated positions to the compare individual object issue - #10)
Apparently my oorb.conf doesn't attach to a commit here. I will see if I can put it in the issue thread, otherwise will email.


Whoops, indent got funky here.


"no" not "now", right?


dito


needs a test.


First question: are the clipping distances applicable to all camera types? Is there a valid camera type without clipping distance?


LGTM


Now not sure if here the move constructor will be used... just for sure shouldn't we be using `return std::move(array);`?


Why don't you use sprt?
This is what it's for.


Did we want to make use of the Penalty on the ErrorHandlingStrategy here before transferring the FlowFile? Doesn't look like it is being used anywhere right now


does this not mean we are simply checking `self["id"].isString` twice? What exactly are we overriding to add here? If it is to make the function public, then `super.isValid()` should provide the same result.


This looks much better.  Thank you for updating.


So is the clipping distance truly per camera or more per camera rig? Based on the repetitive code for the setters and the fallback to leftCamera for getters, it seems like it should be per camera rig.


It's only Javascript :laughing:


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


It's better to make try blocks cover as little code as possible.  Style wise, it's better to use two lines instead of one for a condtional.  So this is better written as:
``` python
if not '%' in size:
    try:
        float(size)
        if not size[0].isdigit():
            raise ValueError()
    except ValueError:
```


@jbecla I don't think this is right. I realise that on Unix it won't make a difference but I think the `text="True"` should be `text=True`. I also think we should probably drop the `dir` argument and just let it use the default (which is a temporary directory not the current directory).


I'' change this to a type check for `"symbol"` which is unshimmable.


TeamCity LabkeyRelease15.2 :: CDS postgres 9.4 [Build 22](https://teamcity.labkey.org/viewLog.html?buildId=274946&buildTypeId=LabkeyRelease152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4 (1 new), passed: 21 Build time: 00:35:24
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

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>

```


Looks like RETRY is only added to the Get processor, did we want it on Put too?


@jbecla These should all be `assertEqual`


Yep, my brain didn't work last night. I'll fix those tonight.

On Sep 29, 2015 12:58 PM, "Morshu32" <notifications@github.com> wrote:
>
> Same here Receive not recieve
>
> —
> Reply to this email directly or view it on GitHub.


@naveenoid take a look on line 38: why did you use fB{i} instead of f{model.parent(i)} in the backward propagation?


OCD me doesn't like.

I'll add it to my next "Miscellaneous fixes"


This is a good idea, I'm surprised it didn't happen when the change to rolling releases happened.


@valtersboze - please retry with the latest changes in master:

To expedite resolution, please always test with the latest versions:
*  Angular Material HEAD
  *  Bower Install - https://github.com/angular/bower-material/blob/master/angular-material.js
  *  Bower Install - https://github.com/angular/bower-material/blob/master/angular-material.css
  *  RAWGIT - https://rawgit.com/angular/bower-material/master/angular-material.js
  *  RAWGIT - https://rawgit.com/angular/bower-material/master/angular-material.css
* AngularJS Latest Release versions:
  *  Angular 1.3.19 (or greater)
  *  Angular 1.4.6 (or greater).


With a dash?


Yeah I revisited this. It's a bit verbose. I'm changing the structure around a bit so that each BlockSession object has an empty list of days when constructed, and the boolean for if its recurring simply checks if that list is larger than 0.


Use `!`


understand your logic but why not write as below:
public boolean recursToday() {
    boolean isPartOfTodaysTasks = false;
    if (this.isRecurring) {
        //add lines 94-99
    }
return isPartOfTodaysTasks;


I think we can leave off this try/catch, the framework will catch the runtime exceptions coming out of onTrigger and have the same result as us throwing a ProcessException


What's the reason you're using `tools` instead of `android` here? Also, I originally added the placeholder id because the drawer took a second to load and the layout jumped. With your improvement there, it shouldn't be necessary any more.


Should be `Matt McMurry`, not `Matt McMurrym`. Has an extra `m` at the end.


Same comment as above about leaving out this try/catch


dash? do you mean "slash" and are referring to the trailing slash?

access() has a special feature built-in, that allows checking if something is not only existing but also a directory rather than a regular file or other node: suffix the path with a slash... Since we are replacing a stat() + S_ISDIR() check that's what we want to take benefit from.


true, will fix


The failures were instant (1s one time and no time at all the next).  Listed that it was not valid YAML format.  But the single quotes 'fixed' the problem.  Like you said, for shell requirements, it does not make sense, BUT it under control of the CI, it simply did not work the other way AFAICT.


To deal with camera types that have no clipping distance, you could define an interface (e.g. CameraWithClippingDistance) that is implemented only by these who support the concept. Then in the camera rig you could internally check whether the cameras the rig has support CameraWithClippingDistance (instanceof) and call into the implementation from the rig. If the cameras do not support it you could either throw or no-op. For example in camera rig:

public void setNearClippingDistance(float near) {
  if (leftCamera instanceof CameraWithClippingDistance) {
    ((CWCD)leftCamera).setNCD(near);
  } else {
    throw new UnsupportedOperation();
  }
  ...
}

This way you can keep the old CustomCamera.


Yes, that does look like a derp.  As it stands, this would require a pair of Handles and an int, and I'm pretty sure that's wrong.


Probably want to remove this. :+1:


You should also update the usage message for the `sign_for` command in src/ripple/app/main/Main.cpp


collapse lines


Ive added all your suggestions, tested them, messed up while coding, cleaned up code, tested them again, and its working properly with 2/3 of the code, also coincidentally, trigger spells no longer consume resource


That is a development leftover right?


Closing this to make a clean pull request


Good question...I just want a 20M game, and some ELO measure to check if my tuning is going anywhere. And if I have my "answer", I can finish at 10M.
If I would run a [0,4], it might take many more games.

The following patch may or may not be approved
https://github.com/official-stockfish/Stockfish/pull/435
See my comments there. I'm just preparing to see if idea to use only scores,
and not a mix of scores and attackunits, might work and also if  my params are OK.

I'm planning to run full SPRT simplification (-3,1) against 435 once pulled into new master.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 737](https://teamcity.labkey.org/viewLog.html?buildId=274950&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **SUCCESS**
Summary: Tests passed: 9, muted: 34 Build time: 00:25:44


Can we use that ArgumentCaptor approach in this test case to prove that the doc id was "doc-p"? or maybe check the flow file attribute for "couchbase.doc.id" at the end? Just to prove that evaluation of the expression is happening, and not falling back to the uuid.


hah really?

that

User.create(req.params.all(), function userCreated(err,user){

```
+
+         if(err){
+             return next(err);
+         }
+         else{
+           return res.jsonp(201,users:{
                     //the json boject containg the user that has been made
              } )
+         }
+
+     });

```


Not sure if this actually right. Seems to be taking the long way across the street. But it does work and gets rid of those pesky depreciation warnings


can you post  a sql file of your project ?


There is a glitch left in the manifest. With the code re-organisation, org.telegram.android.GcmBroadcastReceiver does not resolve anymore:  https://github.com/DrKLO/Telegram/blob/master/TMessagesProj/config/release/AndroidManifest.xml#L35


I think those should be my last comments :) Nice work on the exception handling approach.


We dont have a custom camera wrapper ;)


Only the absolute minimum was done to get started - this is just another step in the process.


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

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

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

```


I slightly changed this code and added:
       ext='x64'
       if os.path.isdir(pi+'\\lib\\Win64'): ext='Win64'


Últimas líneas de errorlog

[Mon Sep 28 14:50:44.932692 2015] [core:alert] [pid 1288] [client
127.0.0.1:38048]
/var/www/html/GitHub/ForkSistemaPCI/sistemaPCI/public/.htaccess: Invalid
command ']', perhaps misspelled or defined by a module not included in the
server configuration
[Mon Sep 28 14:56:17.454265 2015] [core:alert] [pid 1286] [client
127.0.0.1:38049]
/var/www/html/GitHub/ForkSistemaPCI/sistemaPCI/public/.htaccess: Invalid
command ']', perhaps misspelled or defined by a module not included in the
server configuration
[Mon Sep 28 14:56:22.789975 2015] [core:alert] [pid 1285] [client
127.0.0.1:38050]
/var/www/html/GitHub/ForkSistemaPCI/sistemaPCI/public/.htaccess: Invalid
command ']', perhaps misspelled or defined by a module not included in the
server configuration, referer: http://apachesistemapci.dev/
[Mon Sep 28 15:13:36.041483 2015] [core:alert] [pid 1289] [client
127.0.0.1:38195]
/var/www/html/GitHub/ForkSistemaPCI/sistemaPCI/public/.htaccess: Invalid
command ']', perhaps misspelled or defined by a module not included in the
server configuration
[Mon Sep 28 15:13:37.290436 2015] [core:alert] [pid 1288] [client
127.0.0.1:38196]
/var/www/html/GitHub/ForkSistemaPCI/sistemaPCI/public/.htaccess: Invalid
command ']', perhaps misspelled or defined by a module not included in the
server configuration, referer: http://apachesistemapci.dev/

2015-09-29 13:58 GMT-04:30 Alejandro Granadillo <notifications@github.com>:

> @Phantom66 <https://github.com/Phantom66>
>
> listo, problema solucionado!!
>
> hice un script para ayudar a la instalacion del sistemita
>
> mira nueva parte de la documentacion
> <https://github.com/slayerfat/sistemaPCI/blob/0667b3dfcaffc827bb88d0f322801311c1aa8d7b/README.md#instalacion-rapida>
>
> dime si te funciona!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/slayerfat/sistemaPCI/commit/0667b3dfcaffc827bb88d0f322801311c1aa8d7b#commitcomment-13500424>
> .
>


Tarkistapa vielä millä tavalla tekstieditorisi päättää rivit - nyt rivin loppu on muuttunut Unix-tyylistä Windows-tyyliin, minkä takia tiedostojen kaikki rivit näyttävät muuttuneen.

Tästä aiheutuu seuraavia ongelma:
- Tämän pull requestin läpikäynti on mahdotonta, kun ei pysty näkemään mikä oikeasti on muuttunut
- Projektin eri tiedostoihin tulee eri tavoin päätettyjä rivejä, mikä aiheuttaa omat ongelmansa
- Näiden muutosten mergeäminen masteriin pitää tehdä käsin joka riville erikseen

Eli käytännössä pitäisi ratketa ihan sillä että käyt vaihtamassa line endingit oikein, seivaat tiedoston uudestaan ja teet uuden tiedoston tähän bränchiin :)


Jesli dasz w catch wylacznie `echo` wtedy nie przerwiesz dzialania skryptu - jesli to jest obsluzone pozniej (czyli kod, ktory wywoluje ta funkcje wie, ze moze sie zdarzyc sytuacja braku polaczenia) to jest w miare ok, jesli nie jest to zamierzone to wyswietlisz tekst gdzies na stronie a cala reszta bedzie udawala, ze wszystko jest ok.
Sugeruje dwie opcje (najlatwiejsze od gory):
1. die('Polaczenie nie moglo zostac...')
2. przekierowac uzytkownika na strone z bledem


TeamCity pull requests :: netty-tcnative [Build 12](https://garage.netty.io/teamcity/viewLog.html?buildId=1838&buildTypeId=pulls_netty_tcnative&guest=1) is now running


This reverts https://github.com/crawl/crawl/commit/354d47989254


TeamCity pull requests :: netty-tcnative [Build 12](https://garage.netty.io/teamcity/viewLog.html?buildId=1838&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:46


Co w sytuacji kiedy nie znaleziono rekordu o okreslonym id? ;)


Seems promising, would you mind to also make a PR to node-serial-obd?
I have plans to merge them together, but I need to find time for this.


Just delete this rather than comment it out, since we now know it is wrong.


TeamCity pull requests :: netty-tcnative [Build 13](https://garage.netty.io/teamcity/viewLog.html?buildId=1839&buildTypeId=pulls_netty_tcnative&guest=1) is now running


Ok...I just don't want users to get in the habit of running fixed games.

SPRT run has the ultimate say.
Your test may fail at 9s TC, but do well at 15s.


I'm not sure my prior comment was clear. I meant that there should be a space between `if` and the `(` following it. So `(if(shortString...` becomes `(if (shortString...`.


TeamCity pull requests :: netty-tcnative [Build 13](https://garage.netty.io/teamcity/viewLog.html?buildId=1839&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:39


@cala Isn't that NPC 9398 part of the _alliance_ quest chain?


K. Tried to see if there was an "API version" API for UWP since that would seem ideal, but this should be fine.


I'm not sure how to do that. IMHO the `:or` map cannot refer to `:keys`


working like that for the bootani?


Have you checked if the type has the right registry entry in FileIO?
Also, we currently silently move to the next loader/saver library if anythin fails while loading. Pretty bad solution.
This will be replaced by some better exception flow in the future, so that we can give more elaborate information like `library not installed`, `binary not build`, `file corruption`, so that we can better decide what to do. Putting some logging with different levels would also make sense.


I have to check whether the override is correctly taken into account.

- If so, I would be in favor of removing the symlink from etckeeper and adding it to `.gitignore`, and keeping all the other stuff as is.
- If not, I would be in favor of either removing the alias (prefered) or adding a symlink to have both invocations use the same override.


Many thansk borti4938, it nows compile :)


If we hardcode this, I think it is best done inside the AROCHIME reader rather than here. Alternatively, one could add another flag in `observations.conf` that would allow one generically to take `conj()` for complex data. For ARO, this might then default to `True`.


I agree but currently have no time to do that. For my purpose the fix enough. Hope to get back to this soon.


the word "max" is already used multiple times, for example in uct api and ud..