I'm embarrassed I didn't notice this was happening when I created the original patch.

Assuming the patch works as you expect (I haven't tried it), it looks good.


Hello Carlos,

now I have to fix that problem I told you last time. Then, we could think
of other things to do ;) See you soon!

2015-03-02 0:48 GMT+01:00 Carlos J. Rosales <notifications@github.com>:

> Glad to see movement in your repo and that you moved to markdown. Looking
> forward to work on this plugin. Cheers!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/valeria-parnenzini/qbmove_plugin/commit/d4fc574d7c05f8c1f538e0a67c51e1959b2c012d#commitcomment-9977420>
> .
>


@giakki: thanks! You seem to have missed the git tag and GitHub release.


The comment here is the reverse of the meaning of the variable.


This should not be portugal-specific


このファイルはOverlayPluginに含まれているので不要です


again, same here.... this should not be Portugal specific but rather generic


This broke the resource tests on travis.


same


FatalErrorException in FormBuilder.php line 3:
Class 'Collective\Html\Formbuilder' not found                      help me??????/


normally yes,but when am use if $rkhunter_cron_check {} then puppet can not declare variable .i do not know where is issue. there are 2 working versions before commit and newest commit .other ways cron jobs was`t created .


TeamCity Toolbox :: Pull requests [Build 36](http://10.48.34.86:8111/viewLog.html?buildId=4649&buildTypeId=Toolbox_PullRequests) is now running


TeamCity Toolbox :: Pull requests [Build 36](http://10.48.34.86:8111/viewLog.html?buildId=4649&buildTypeId=Toolbox_PullRequests) outcome was **SUCCESS**
Summary: Tests passed: 160 Build time: 00:00:19


Could you add burt@burtbeckwith.com as a recipient for failure emails? I don't see a way to configure notifications as a non-owner


That sounds good to me.  If a need arises, we can add `FindURL` etc methods in future (similar to plek.rb's `find_uri` etc) that do parse and return a URL, and potentially return an error.


@hai-cea I agree. If that's the sole comment, shall I resubmit the PR?


Did this really fix the problem? I don't understand why the path to `node_modules` is `../../node_modules` here for `fa-font-path` while it's `../../..` for `ol.css`, `bootstrap.less` and `font-awesome.less` higher in the file…


I also look over avim firefox, and I found it is very nice structure and have a good test architect. And I hope avim-chrome will be the same architect with avim firefox, that will be more easier for new contributor to join both of us.

I already add you as collaborator with pushing permission.

Thank a lot.


I think better use Uri class for parsing and method `Uri.IsWellFormedUriString`


Implementation on php-reflect is already available, and php-compatinfo will come soon.

Source code:
https://github.com/llaville/php-reflect/commit/fd14dc04c7346c1e319617886b7439610ee41c95

Explains on blog (3.0 RC1 preview) :
http://php5.laurent-laville.org/reflect/blog/


@epaminond Ok, sorry I didn't pay attention to this. We will .gitignore it.


We need Mockito dependency because test code from common module uses it and the module does not have its own ivy.xml. So every shim has Mockito as dependency. For some reason it was missed from hdp21 and hdp22.

As for executeScript method - we didn't test it before, and in current test implementation we need the method just to be able to instantiate the class.


Hi Michael,
I'll be more than happy to accept the pull request, I was just wondering if you would be willing to add the unit test related to the new functionality.


It can be removed unless you use for something else.


@vo-quang-mulodo: Good job! Just go ahead!


Sorry. This one here is not the problem but the one in the Helix repo is because it has version 1.0.3 which is newer than 1.0.0.


This change caused quite a couple of build failures in openSUSE:Factory.
The build logs all have in cocmmon:

[   69s] WARNING: '/usr/lib/rpm/brp-desktop.data/applications-kmenuedit.menu' does not exist
[   69s] WARNING: '/usr/lib/rpm/brp-desktop.data/custom.menu' does not exist
[   69s] Unknown 'DefaultLayout':
[   69s]         'HASH(0x32318f8) 0
[   69s] 		 Merge ARRAY(0x323e630) 0
[   69s] 		 Merge ARRAY(0x323e570) 0
[   69s] 	'
[   70s] ERROR: No sufficient Category definition: /home/abuild/rpmbuild/BUILDROOT/gramps-4.1.1-1.2.x86_64//usr/share/applications/gramps.desktop
[   70s] Errors in installed desktop file detected. Please refer to http://en.opensuse.org/SUSE_Package_Conventions/RPM_Macros
[   70s] error: Bad exit status from /var/tmp/rpm-tmp.Wk8EhI (%install)

All built fine before this update


Paths to less files to include is completely different to paths to other resources (images, fonts, ...).
The first one is to be interpreted by the less compiler while the second one is supposed to be interpreted by the brower.


@carlaschroder Keep in mind, that this is generated out of the config.sample.php. I will create a PR against owncloud/core ;)


@addyosmani: Everything should be ready to release 0.4.1; just a GitHub and npm release are pending for you to make.


Did you change composer.json and do a composer update to load this package ?


CharlieKilian

Nice PR with Tests!!!  One thing you should do here is use CCDrawNode instead of CCDrawPrimitives.


Believe this should be originalTaskFn.call(gulp, name, fn);

Currently aliases are breaking on 4.0 due to this line.


this is pretty gross, it's not you fault, but this should be `p.IsNoSplit()`


the warn can be omitted (because i don't see real debug value here) and both this and next row must be replaced to
return callback(e);


Ok, I didn't know that. So the new problem (#289) is related to the "cache-busted" URLs that we use now for static resources in `geoportailv3/static`. Thanks for the explanation.


hi! how auto create  class file in Entities? or had to create manual.


Erm...maybe I just don't get it but why differentiate between moving and swapping plots? What's the issue if the target plot at a swap is empty? Should be fine?


ReachContObj is incorrect name as it starts with a capital letter. only class names and structure names can start with a capittal letter.


ControlFuncObj is a bad name


@mflorea Shouldn`t we display `superadmin` instead of `Unknown User` when the user does not have a profile page but his name is superadmin? I am assuming that is the reason why we would get `Unknown User`.


Its a bit sad that we can't trust it ^^ I saw the implementation in Super-mining, pretty cool ^^


I don't know how you guys think about security, but usually when you treat it real there is a policy to not output any potentially personal data into logs. The changeset contains charBank that can contain any kind of information such as passwords etc. Putting them in server log opens threat for leaking the data.


I could remove it though it's also a scenario we need to handle. For instance should we put out an update for English in repo and a users upgrades to =>15.0 he could face getting that update at first start.
Come to think of it, that could be the issue i have on Android with no text loaded at boot and only seeing the kaitoast that English update. This could happen for any language.


@kaedroho: Looks like I left a console.log statement in here.


Backup/restore moved from core to client. Though copyFunc still defined in core, TBD: move to client.


please rename isX0inset into isX0InSet


wtf ? i know you didn't add this initially but this is fugly and works by accident.


TODO


TOL is a bad name as it is not meaningful


Hi, the project is quite old. Slicing tool was developed as an Eclipse plugin and was tested on Eclipse Kepler 4.3.2. Not sure if it still works for the newer versions.

We have a section "How to use Map Slicing Tool" in our brief doc. You can find it here http://lemberg.github.io/mappwidget/user_guide.html. You should run map slicing tool as Eclipse application and it should work.

Hope it helps


@endJunction : http://i1182.photobucket.com/albums/x443/Micma99/I%20Like%20Cheats/pebkac-tech-support-cant-help-you-when-youre-the-problemthumbnail.jpg
I still had the variable on the right side added...
All is fine!


For different branches JavaDocs MIGHT be different even for the same
class under the same package and in the same folder (for example
jdk-1.7-parent). There's no guarantee that version for Wicket 6.x has
the same javadocs of version 7.x.
>
> The folder of subprojects are also the same for wicket-6.x / master -
> there is no wicket-core6 for example - the projects are build by there
> parent so why not generate the javadoc in a folder this way?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wicketstuff/core/commit/9ee2764be26b4caab2710c87a783c2f218217be5#commitcomment-9980597>.
>


SPI?


"i" prefix is only for iterators. for just indicees you should use "ind" prefix.


Typo: '...we send to all of of teams...' >  '...we send to all of our teams...'


Please do not leave commented lines.


options is a bad name as it is a structure so options should start with a capital letter.


Wow, just realised how dangerous that is.


You do not need to update the dependency to 5.4.1 to get that. You only need to depend on a newer xar plugin. This way you preserve compatibility with older versions of XWiki, if your code also preserves it, of course.


I need to write the partners section. Will add this to the list for today


Rather than doing this match you can also simply `import Parser.DeliveryScheme.Throw`


Good job. But project structure is messed up. I am going to do a manual merge.


Why do we have to avoid the nested comparison here?


@ReDetection, @novist what about these changes?


https://github.com/owncloud/core/pull/14635
https://github.com/owncloud/core/pull/14636


Where is this method called from?


ok! I just looked at the notes / issue you and sven discussed on, so this doesn't have to go out today :) once the partners section is done, we can change the link!


Shouldn't this sample live in the test?


ok I'm almost convinced :p my only concern is how well "you should use a VC" will work in practice if you download a tarball and want to install it locally.


Maybe add a warning here?


Available on php-compatinfo now (see commit https://github.com/llaville/php-compat-info/commit/ba461c81357488bd27bf0538399c77e90fd97d61)


Does this grammar somehow correspond to an official kind of specification?
Or is it something completely custom?


rewrote the comment to make it more intuitive


Really now, Glynn. American format!?


Oh that. That should probably removed. I assume it was added as a hack around repo checksumming being broken (which I hopefully fixed in #6343). You should *never* have to forcefully delete a repository from the db.


I love the fact that Prog has a usable zero value.


I got confused by how far into the year we were and the above one being 2015-02-03


Is this supposed to have `@Overwrite` here?


I also improve layout of https://github.com/1ec5/avim/wiki/Architecture. Please check it.


cookbooksディレクトリは、berkshelfで依存解決したものが勝手にはいるものなので、バージョン管理から外しています。


Sniffs should conform to PHPCS coding standard.

Underscore is required as first character in private properties.


Haha yeah, it's a pretty misleading opcode. Now just to sort out why the graphics are all screwy. I think it's likely something to do with VBLANK interrupt timing.


Hahaha sorry!
I hope it'll all make even more sense now and in the future. Things have gotten a lot simpler.

Your main input is now js/app.routes.js !


No doubt on this commit.


 @lauraluiz Can you please check the release notes?

http://sphereio.github.io/sphere-jvm-sdk/javadoc/master/io/sphere/sdk/meta/ReleaseNotes.html

Also there have been search related changes since M10, can you please add the release notes to id?


Here sdlog2 does work!


@Enygma2002 Using a newer parent doesn't break compatibility. The parent is used for build purpose. It brings recent build tools. It doesn't necessarily mean you have to depend on XWiki 5.4.1 at run time. You can use a different version for your dependencies, for instance by declaring a ``${xwiki.version}`` property with an older version.


Разрешение: 1600x900, Размер HUD: Большой
![](https://raw.githubusercontent.com/TEHGAM/Docs/master/images/adr/900-big.png)


I agree with oliver, the general timestanp should be an overall reference and not a meassured value. It should be the Time that the Linux OS delivers. Refardless how wrong that is. In Addition the hardware should set the Linux-time and date as soon as possible. For example via PVT. The times that belong to Measurements should be there and have a different meaning. There it always belong to the GNSS System to whome the Measurement or PVT belongs.


Yes, that's true.
Added --skip-networking while doing init to prevent this.


We might have to simple the explanation, as it's a little tricky to grasp from the outset.
Perhaps something like:

The way it works: Companies have two ways they can find a team to coach for the summer.

1) the company should reach out to possible students in their area before they apply for the program, through blog posts on their company blog, meetups and announcements on social media.

2) register with us on our teams app, and hope that a team finds you during their application process.

We also encourage students who want to apply to look for local coaching companies before applying, and list said company and coaches on their application. This is important, because it allows the teams to mention they have a coaching company backing them. After we go through the selection process, we will notify the teams, and their companies, if they've been accepted.


@Noart537, у тебя 1920x1080 и Нормальный HUD?


watch this : http://toki78.github.io/?approx


this is first commit


s/return/raise/


AWESOME SAUCESS!!!


a wild space after `.` appeared


that xhtml slash :(


2nd march


Agree with @danielmarbach , don't think we can do much until we have another default transport (filebased?) or have a core without a default transport


thank you for the great feedback @sareg0. I will implement it later today! (:


It's the http side that is broken.


[![Qin (Zither)](http://api.artsmia.org/images/4973/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=4973)


[![The Flower Gardener, from Des Manouvriers habilles](http://api.artsmia.org/images/61310/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=61310)


Those can stay in serializers.py thanks!


Each test is run inside its own 'trash' directory, so the `actual` files from one test wouldnt interfere with any other tests (only within the same test)


I have a free Certificate also from Github. so can you help me?


@bbodenmiller I actually prefer ```UIAlertController```-- it's much better (see [some of the arguments why here](http://nshipster.com/uialertcontroller/). From our analytics, we definitely have enough users who have upgraded to iOS 8. I'd also rather not use a deprecated class.


文字化けしているのはShift-JISだからですね。
Web業界で一般的な文字コードはUTF-8です。
できたら、今後はUTF-8でチャレンジしてみてください。


~ 76% of our users in the past month have been on iOS 8. I'm cool with this, and was just curious.


Sorry, I've been way busier with other stuff than I expected to so I
haven't done anything on the code generator yet.

As for the `Expression a` vs `[Expression a]`, using a single expression is
how we did it initially (with case expressions in mind) so I think that is
a good idea.

On Tue, Mar 3, 2015 at 12:54 AM, SoamJaws <notifications@github.com> wrote:

> Sorry for the delayed answer.
>
> It is kind of both ways in core erlang. All functions are wrapped with
> lambdas, but different pattern matched cases will be a case clause inside
> said lambda, just as you state. I think the best way is to try to make the
> new AST as close to this case as possible, like you say, Fun Identifier a
> (Expression a), where Expression a may be a case clause.
>
> As for the fourth thing, I would say that is a single pattern, but it
> would require that tuples are implemented.
>
> I like the idea of a utility module, and I will evaluate the code gen and
> see if anything there belongs in the utility module. I will branch from the
> new-ast branch now and begin making the changes / additions in the code
> generator, Jakob are you tinkering with the code gen?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DavidLindbom/DATX02-15-29/commit/d422003525304b34976569687f0e77017ba64874#commitcomment-9996632>
> .
>


please refactory by puell npe


si, las cambie porque usaba el master de los repo en vez de una versión especifica.


I have updated npm and node. I get

Assertion failed: (0), function uv_err_name, file
../deps/uv/src/uv-common.c, line 75.
Abort trap: 6
ᐧ

Thanks
David

Shaping Connected Experiences

<http://davidmermelstein.dunked.com/>  Portfolio
<http://davidmermelstein.com/>     <http://www.linkedin.com/in/merms>
Linkedin <http://www.linkedin.com/in/merms>    Twitter
<http://www.twitter.com/dmerms>      773.744.0760

On Mon, Mar 2, 2015 at 1:16 AM, Martin N. <notifications@github.com> wrote:

> Sorry, my bad for using that notation. Run npm install -g grunt-cli - if
> you're seeing errors with that, your node/npm installation might be
> somewhat broken. I've never seen any errors like that before.
>
> Are you on OSX, Windows or Linux?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AVGP/susi/commit/fa59094a26641840530c9fef5110fa1f75d7b7e0#commitcomment-9979975>
> .
>


@GhoulPoP , you should have just made these into issues.


All ChildAction actions MUST be marked with [ChildActionOnly] otherwise these actions are publicly routable!


@GhoulPoP , you should have just made these as issues.


Yeah, also failing-to-fail for julia-debug on windows where sys.dll does not get loaded https://github.com/JuliaLang/julia/pull/9482#discussion_r25147428

Should we just comment out these lines, with a link to this discussion about why we can't test this flag reliably?


84692518-Markdown-Documentation (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (622 of 622 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-3.65%25-green.svg)


Ideally we would default to the most recent date range selection, but the combo box portion of that widget is a private member and the selected index is not currently exposed. That should be changed so that people can choose a new default without needing to modify this file.


Doesn't this break gensoc's ``with_memtest``?


// Пользователь должен указать с клавиатуры чётное положительное число, а программа должна создать массив указанного размера
// из случайных целых чисел из [-5;5] и вывести его на экран в строку. После этого программа должна определить и сообщить
// пользователю о том, сумма модулей какой половины массива больше: левой или правой, либо сообщить, что эти суммы модулей равны.
// Если пользователь введёт неподходящее число, то программа должна требовать повторного ввода до тех пор, пока не будет указано
// корректное значение.


Slicer commit 0d6d4872 (r24010) should allow this patch to work as expected.

@gregsharp  Could you remind me which extension requires OpenMP ?


Alright, I will work with the gode gen and the report/half time presentation material all day tomorrow, so if you want to pitch in just let me know!


teste


This approach looks nice: it would be better if all those loops were inside ```models/character.rb```.

Then, this would look like

```ruby
  @stories = @character.eligible_stories
```


Hey Francois,

I have a question on this, the minimum Chrome version was upped to 42, which is the currently Canary channel. Our Chromeboxes here are all set by the admin to be on the stable release (currently 40) and so we're no longer compatible with the App.

The window frame color was released in Chrome 36, could we move the minimum version back to 36?
https://developer.chrome.com/apps/app_window#type-FrameOptions

Thanks!


Sort usings.


Sorry, I really just experimented with a bunch of things and eventually it worked. I am not sure exactly what I did. One thing you **need** is the certificate trust chain, which takes a little bit of research, but is fairly easy to find. You can look at the following tutorials:
* http://www.steveneppler.com/blog/2011/02/09/signing-ios-mobileconfig-files-with-your-certificate
* https://wiki.terena.org/display/tcs/Sign+Apple+mobileconfig+files
* http://www.rootmanager.com/iphone-ota-configuration/iphone-ota-setup-with-signed-mobileconfig.html


@aaronbrethorst Yup! Thanks. Will make the changes and then submit.


execute effects should really live inside a model :)


IIRC `.finally


Thanks for your comments @hannesj, I have added them to the Javadoc and will take them into account when working on this.


Have you verified that this can fail?


Apparently it was more broken before, and I tried to unbreak it.


Consider merging these statements into a single ternary operator.


1540218-SlinkyGroundHX (mmatts) - x86_64-MacOS-10.9-clang: OK (1617 of 1623 tests passed)

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


Some changes for the new branch


all tests pass:
success: http://build.servo.org/builders/linux1/builds/900
success: http://build.servo.org/builders/linux2/builds/898
success: http://build.servo.org/builders/android/builds/936
success: http://build.servo.org/builders/mac1/builds/1115
success: http://build.servo.org/builders/mac2/builds/1115
success: http://build.servo.org/builders/gonk/builds/86


Sample


This is not related to the commit message. Please create two commits next time.


The comment is somewhat confusing because the method returns Task<BuildResponse> rather than BuildResponse.


Thanks @lclementi !


All fine.


fast-forwarding master to auto = 65454e51


75127368_DCFSEC (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1120 of 1126 tests passed)

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


Perhaps you might consider to increase the worker version to 56?
Regards, Alfred


@gianarb it should be "ind**e**pendent" (I always do the same mistake, Italian language influence fucks us up!) :stuck_out_tongue:


from build repo... dont think its needed though... about to test 2morrow hopefully, got other problems atm


saw approval from mbrubeck
at https://github.com/glennw/servo/commit/2517bb561a09fc16adfd348d80cee81bc61151c4


merging glennw/servo/fix-title = 2517bb56 into auto


glennw/servo/fix-title = 2517bb56 merged ok, testing candidate = 6927bf6f


Are you going to implement soft shadows? It can be done almost no cost in a couple of ways from this point.


This function was expecting its caller to pass scalar values for all parameters. Why change it to expect scalars for ``a`` and ``ba`` and a phy object for ``d`` and ``nphases``?

Also, using a phy object here reduces the independence between DFII and the PHY. Please keep the scalar parameters unless there is a good reason not to do so.


Async I/O latency to a ssd greatly increased, 34% gain (Creds: kleikamp)

Authored-by: shskyinfo


merged.


there are soft shadows, go to the project settings and set shadow mode to
PCF5 or PCF13, then change the shadow resolution to adjust softness

On Mon, Mar 2, 2015 at 9:25 PM, Krzycho666 <notifications@github.com> wrote:

> Are you going to implement soft shadows? It can be done almost no cost in
> a couple of ways from this point.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/okamstudio/godot/commit/a1f715a4da71fbc2b7d6fad68624bf8b22c6da17#commitcomment-9996986>
> .
>


Perhaps you might consider increasing the worker version to 56?
Regards, Alfred


I just submitted [issue #3656](https://github.com/jekyll/jekyll/issues/3536) in the Jekyll repository then I noticed the link for this repository.  Basically I'm trying to find out what page.meta means in this line.  Is it a valid variable?


actually, the restartable attribute applies only to non-listening controllers.
the main listening controller must not be restartable; otherwise it will be restarted when the controller is removed via indigo_controller_remove().
see the comment block preceding listen_socket_ready() for more details.


Hello Simon,The sketches I was referring to are the ones used in your book, "Programming Arduino". Page 23, chapter 2, you say "All of the skeches used in this book can be downloaded as a zip file (Programming_Arduino.zip".  I found the sketches listed individually at "www.arduinobook.com" under the tab "Download code".  I do not find any listing or reference to a zip file.Thank your for writing this book on the Arduino Uno board.  I've been working with the PIC 16f684 for a couple of years now, but reached a point where I want to send data to my PC.  Arduino has made this task possible.
Sorry for the lack of clarity in my request.Thank you again!Regards,Ralph Tarrant (tarrant39@yahoo.com)

     On Sunday, March 1, 2015 1:31 PM, whinge <notifications@github.com> wrote:

 What are you talking about? Which sketches?

On Sunday, 1 March 2015, tarrant39 <notifications@github.com> wrote:

> I can't locate the zip file containing these sketches. I can see the
> individual modules, but no zip with all modules.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/simonmonk/programming_arduino/commit/ee775946dcbce61b1470d9cd8c4b10ee42d45438#commitcomment-9976627>
> .
>
—
Reply to this email directly or view it on GitHub.


Ooh a Perl highlight. Excellent. Hello there!


Keep ``phy_settings`` here 1) to be consistent 2) to keep some independence with the PHY.


Same as above.


Yes, two reasons:

* it appears in text as literally **RFC 4462#section-5**, which is pretty ugly - I'd prefer **RFC 4462 (section 5)** or **RFC 4462 sec. 5** or some such. Or even just as **RFC 4462** (i.e. only link target differing).
* Having to type out `section-` every single time is pretty silly; should be possible to just say e.g.:

  ```
  :rfc:`4462#5.5`
  ```

  or similar.


We will do a short session about git, committing and stuff, just to get a little bit more familiar with the workflow, because you've pushed a few commits where the title is unrelated with the changes and/or several concerns are mixed in the same commit.

How does it sound?


Scales better :)
![screen shot 2015-03-03 at 01 35 37](https://cloud.githubusercontent.com/assets/125676/6454061/71d6cfdc-c145-11e4-8878-5d6386082e9e.png)


Don't think it should, look at point 4 on that page.


The ``get_sdram_phy_header`` function should take ``phy_settings`` as well instead of a PHY object.


this isn't smart


I was trying my best


Shouldn't it be the molten enderpearls not redstone?


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2091 of 2212 tests passed)

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


:point_down: :point_down: :point_right: :point_right: :point_right: :point_right: :point_right: :point_right: :point_right: :point_right: :point_right: :point_down: :point_down: :+1: :point_left: :point_down: :point_left: :point_down: :point_left: :point_left:
:point_down: :point_right: :point_right: :point_right: :point_right: :point_up: :point_up: :point_up: :point_right: :point_right: :point_right: :point_right: :point_right: :point_up: :point_left: :point_left: :point_left: :point_left: :point_up: :point_left:
:point_right: :point_right: :point_down: :point_down: :point_down: :point_down: :point_left: :point_right: :point_right: :point_up: :point_up: :point_right: :point_down: :point_right: :point_up: :point_up: :point_left: :point_up: :point_down: :point_left:
:point_up: :point_down: :point_down: :point_down: :point_right: :point_down: :point_left: :point_down: :point_right: :point_down: :point_up: :point_right: :point_right: :point_up: :point_up: :point_up: :point_left: :point_left: :point_left: :point_left:
:point_down: :point_right: :point_right: :point_down: :point_left: :point_down: :point_left: :point_right: :point_right: :point_right: :point_down: :point_right: :point_right: :point_up: :point_left: :point_up: :point_left: :point_up: :point_up: :point_left:
:point_down: :point_down: :point_up: :point_down: :point_down: :point_down: :point_right: :point_down: :point_up: :point_right: :point_right: :point_right: :point_up: :point_up: :point_left: :point_left: :point_left: :point_up: :point_up: :point_up:
:point_right: :point_right: :point_right: :point_right: :point_right: :point_right: :point_right: :point_right: :point_right: :point_up: :point_up: :point_left: :point_right: :point_up: :point_up: :point_left: :point_left: :point_down: :point_right: :point_up:
:point_up: :point_right: :point_up: :point_up: :point_left: :point_up: :point_left: :point_up: :point_up: :point_up: :point_left: :point_up: :point_up: :point_up: :point_left: :point_right: :point_up: :point_left: :point_up: :point_up:
:point_up: :point_right: :point_up: :point_right: :point_up: :point_left: :point_left: :point_up: :point_up: :point_up: :point_up: :point_up: :point_right: :point_up: :point_up: :point_up: :point_left: :point_up: :point_left: :point_up:
:point_right: :point_right: :point_up: :point_left: :point_up: :point_left: :point_right: :point_up: :point_up: :point_up: :point_up: :point_right: :point_up: :point_up: :point_up: :point_left: :point_up: :point_left: :point_left: :point_up:


let's back-burner this local vs. listening issue for now.  i'll file a JIRA for it to track.


If the returned Task<bool> completes, it always have true Result, right? Maybe we can just return a Task?


cc @nevern02


I am not sure how I feel about requiring the (gdb). On intended purpose of the tool would be to format a single line of output that you copied from a session transcript. Requiring the (gdb) makes it less convenient. For that reason, we could be a bit more permissive than the format syntax. WDYT?


``HBaseTableNameConverter``? ``HTableNameConverter``?


Thanks, @aantonop!
Just to confirm, should we wait until https://github.com/aantonop/bitcoinbook/blob/569be354b8491240e421a4e04cd9d53a465afac4/LICENSE reflects the update?


Yeah, I already got quite close to 6 seconds with regular CSL styles...


The LASMI crossbar is a fairly generic component that does not depend on a particular SDRAM controller implementation. It should stay in lasmibus, and probably in the same file as the interface definition (as the arbitrers/decoders for wishbone, csr, etc. do this). I think you should revert this commit.


hmmmm... not working or we miss something. How can it be soft if light source is point anyways? Do we speak about the same thing? :P


Either the parser tries to consume it, but does not fail if it's not found (less formal), or when the parser's user does not find `(gdb)\n` at the end, it appends it before giving it to the parser (more formal).

What's suits you best?


Awesome!  Thanks for the quick response.


Super awesome to see this. Great resource you're using.


actually this is a fix for #1359, not #1386.


Also made slight improvement on scheduling algorithm.


This finishes up the "I don't want to store things in /iocage cause I'm a snob like that" changes made. I haven't fully tested as of 02/03/2015.


An important fix to properly work the the Gravity Forms + Custom Post Types plugin


Some other spelling errors I noticed:

- ravnika package
- ConditionalContinousEffect class
- WhileConditionContiniousEffect class
- ContinuousRuleModifiyingEffect[Impl] classes

And probably others I can't remember right now.


Small dictionary for T9 usage.


  I'm finished for the night.  I have lots of questions.  Tomorrow.

@mrcwinn


Thanks O'Reilly! I have Andreas' book for Kindle but will also purchase the print version just to honor your important demonstration of  support to an open culture.
@aantonop I'm contacting info@bitcoinbook.info in the name of the Brazilian translators (https://github.com/btcbrdev/dominandobitcoin-livro/graphs/contributors) for coordination.


You made a very clever favicon. Totally cool representation of the content and very crisp, readable image.


Very nice enforcement of poster size.


It also might be the real fix for #1114 (which is already closed).  It also fixes #1264.


I really appreciate these comments in your CSS. Thanks!


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

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


r+


Yay!


``getattr(platform, "is_sim", False)``. Guess you can remove the function after this.


For the pretty printing use-case, I think the first option would be
preferable, since we are not looking to make a validating parser. Do you
think it could be made as an option? e.g.

    parser = pygdbmi.parser.Parser(require_gdb=True)  <--- needs a better
parameter name
    tree = parser.parse(mi_text)

This way, it could be adapted to the situation.

On 2 March 2015 at 19:44, Philippe Proulx <notifications@github.com> wrote:

> Either the parser tries to consume it, but does not fail if it's not found
> (less formal), or when the parser's user does not find (gdb)\n at the
> end, it appends it before giving it to the parser (more formal).
>
> What's suits you best?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/eepp/pygdbmi/commit/fd1589c160ec5d41574109246afd678846662b87#commitcomment-9997192>
> .
>


Have you thought about using CAF's type registry for your archive? I'm not sure how easy it is to export the type map (and if it's possible at all, @neverlord should know), but it would help with portability.


Ok, replying by email sucks.

    parser = pygdbmi.parser.Parser(require_gdb=True) <--- needs a better parameter name
    tree = parser.parse(mi_text)


Yes, this is a great alternative, I'll do this.


This is what I got for distance 10000
![scrn1](https://cloud.githubusercontent.com/assets/6615905/6454260/cd173a10-c147-11e4-8095-bb39e6b9446c.png),
and for distance 0
![scrn2](https://cloud.githubusercontent.com/assets/6615905/6454267/e0c74e6a-c147-11e4-93c3-c7cadfe260a9.png)

It doesn't work as it should to me, or I don't understand how you designed it.


The condition "fullFileName != null" looks redundant.


Looks good to me.


The HEAD of the develop branch now shows the CC-BY-SA in the LICENSE file.
The old history of the LICENSE file will not change, but if you go to the
latest version, you will see it

On Mon, Mar 2, 2015 at 7:40 PM, Fernando Bitti Loureiro <
notifications@github.com> wrote:

> Thanks, @aantonop <https://github.com/aantonop>!
> Just to confirm, should we wait until
> https://github.com/aantonop/bitcoinbook/blob/569be354b8491240e421a4e04cd9d53a465afac4/LICENSE
> reflects the update?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/aantonop/bitcoinbook/commit/569be354b8491240e421a4e04cd9d53a465afac4#commitcomment-9997132>
> .
>

--
Andreas M. Antonopoulos
andreas@antonopoulos.com
Tel: +1 347 744 9404


Congrats on getting the background video to work!


all tests pass:
success: http://build.servo.org/builders/linux1/builds/901
success: http://build.servo.org/builders/linux2/builds/899
success: http://build.servo.org/builders/android/builds/937
success: http://build.servo.org/builders/mac1/builds/1116
success: http://build.servo.org/builders/mac2/builds/1116
success: http://build.servo.org/builders/gonk/builds/87


one thing I thought would be cool would be to introduce each others tools and then divy up the other stuff. that way it doesn't sound like we're talking about how cool we are. But I can be like scott built this totally awesome thing that I love and you should too. It will come off differently.


hello


I like how you used the monkey image in the background -- it lines up nicely on the page and creates a cool aesthetic.


ff


I would not have thought to do this, since the password gets hashed on save, but I see it could be useful in some edge cases.


@goatslacker Shouldn't this be a `peerDependency` @ `0.12.x`?


Just wondering, what's your rational for choosing a asymmetric interface of `save` and `load` where one takes a const-reference and the other a pointer?


75127368_DCFSEC (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1613 of 1619 tests passed)

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


GasHumidifier (Myoldmopar) - i386-Windows-7-VisualStudio-12: Tests Failed

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


What is this?


fast-forwarding master to auto = 6927bf6f


Ha. iocage fetch.


Proces -> Process


1/20 = 5%, ops !


saw approval from jdm
at https://github.com/brson/servo/commit/1995723e5cc07aa5a9867f3ccd57fe9dfc0c8c8d


merging brson/servo/cleanup2 = 1995723e into auto


brson/servo/cleanup2 = 1995723e merged ok, testing candidate = 6ed5111a


Yep: https://github.com/aantonop/bitcoinbook/commit/a2031ea7c9a9776f7caa2ce8a21d0116a7d4c168


HVACSizingSim_CoincidentPlant (EnergyArchmage) - i386-Windows-7-VisualStudio-12: Tests Failed

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


> I'm in "monkey see-monkey do" mode. I cleaned up because I saw other test cases doing it.

No worries -- we've a lot of conventions :). Good anyway, cause raises the question of whether cleaning up may actually be useful.

> (lots of tests use the "actual" and "expected" files)

Yeah, i also dislike this. Numbering them (which we've started doing too) is an ok solution, though also has downsides. *shrug*


@wallflower1 Don’t add the javascript link in every page translation groups may be displayed. Instead, add it only once, within the function responsible of creating collapsed groups. Giving proper parameters, you can call ->link() anytime and as many times as you want, and it will only add a single link to the Javascript file in the `<head>` section.


Why?


This email looks like it is from 2001? "looking at it" seems a little optimistic here...


Not sure of what other high level goals to add. Please add content if you can think of more things.


The reason your style isn't working here is because `headshot` is the class on the image. Your selector here is not targeting anything on the page. However, you've written the proper style for getting the images to line up. The selector you should use should be: `.weybec-people li`.


r+


Ok, this sucks. We need to find a better solution than the ``Platform`` factory functions in Mibuild, or just use ``add_extension`` outside of the platform like ARTIQ and other designs do.


The change here is very subtle and easily missed.  I think we need to make it explicit either through a comment or reworking the code that we depend on the side effects of `spa_features_check()`.


I would prefer to see you control this sizing using CSS. Although the `height` attribute on an `img` tag is valid, it's not the preferred way to control image size. A better way would be to write something like this:

```
.headshot {
    height: 200px;
    width: auto;
}
```


It's not really a good idea to write to params, could you do it as:
````
  if params[:i18npath]
    redirect_to params[:i18npath]
  else
    redirect_to params[:return_to].to_s
  end
````


@egh I see, so you'll run this Rake task as part of your deploy?


Oh, that's way better, thanks!


@Lukas-W I assume this works on Linux regardless?

Minor, but the environmental variable `%ProgramFiles(x86)` is the determining factor between 32-bit and 64-bit on Windows if that helps here.

Also, if ANT supports variable expansion, `%ProgramFiles%` and `%ProgramFiles(x86)%` are generally favored over `C:\Progra...`.


Shouldn't this be ``sata_host``? You are doing ``self.lookup_request("sata_host")`` further down this file.


it's pretty easy to add tests for this, see test/integrations/redirects_test.rb for examples


Is MSBuild open sourced with the same license as Roslyn?


A dependency is fine. I had it as a peer dependency without specifying it. Thanks!


Could you add semi-colons? Otherwise LGTM


To reference the new favicon you used, you should use the full filename: `favicon_m.ico`. But, really, you should just overwrite the existing `favicon.ico` and use the standard filename so that any browser that doesn't properly recognize your `link` tag will still find the proper favicon.


I'll do as you suggest, thanks.


I'm just looking at this and wondering whether `removeTemporaryMetrics` should not always be executed? Doesn't this mean if someone sets `hideColumns` or `showColumns` the temporary ones never get removed causing a completely different result?


I included this as a dependency because I use react's internal escaper https://github.com/goatslacker/iso/blob/master/src/iso.js#L1


oh, and where do you want to redirect to if params[:return_to] is not set? the homepage?


/*
 a) encapsular a animla
public class animal {
private String nombre;
private int edad;
public void setNombre (String nombre){
this.Nombre=nombre;
}
public void setEdad (int edad){
this.edade=edad;
}
public string getNombre(){
return nombre;
}
public int getEdad(){
return edad;
}
}
b) implementar el cosntructor q inicialise un atributo

public class zologico {
private Animal animal
public Zoologico (Animal animal){
this.animal=animal;}
}
}
c)crear una excepcion que se llama EdadNoNegativaException

correcto
public class EdadNoNegativoException extends Exception{

      public EdadNoNegativoException(){

      super("Lo siento pero no acepto edades negativas");

      }

d)crear una clase q se llame validar edad
public class ValidarEdad
public static void validar (int  edad)throws EdadNoNegativoException {
        if(valor<0)throw new EdadNoNegativoException();

      }
e)crear una clase donde creas 3 animales en un metodo que genara arreglo generico a lso animales
import.java.util.*
public class GenerarAnimales {
    public  static ArrayList<Animales>  ObtenerAnimalitos () {

    ArrayList<Animales>animalitos=new ArrayList<Animales>();
    Animal a1=new Animal(12,"mono");
            Animal a2=new Animal(3,"leon");
Animal a3=new Animal(4,"oso");
return animalitos;
}
}
(consturctor poir defecto que inicializa public Animal(int edad,String nombre){this.edad=edad; this.nombre=nombre;}
f)crea un main que se llame test animal q inicialis a los animas
public static void main(String arg[]) {
for(animales a:Generaaranimales.Obteneranimales() ){
System.out.println(a.getnombre())

});

*/


I'd extend the comment above `feature_get_refcount()` to make it clear that while well designed features themselves shouldn't need to use this the `spa_get_stats()` interface depends on it.


The initializer looks unnecessary, as it's always overwritten by passing as an out parmeter.


@jngli could you please explain me this again ? I think I misunderstood your point.


Maybe be better to add the reason why that is the minimum required version...


Nice one!


I don’t understand why this is needed. Can’t
```php
'text' => $hiddenLinkText,
```
do the job?


Maybe there is some way to DRY this code up so that the config-check logic isn't repeated between gridTable.js, griddle.js, among other places row _.uniqueId was previously being used.

Also, I think that this should produce a warning that `_.uniqueId` is being used if no `key` is defined in the `rowMetadata` config


Is this file supposed to have a copyright/license comment at the top?


This looks pretty chunky non retina. Intended?


Can you add the filename to the error message? That would make it really useful, especially when I'm not sure what the value of `base` is.


Update the threashold, higher sensitivity


ESM gives me some results, but in the way that I don't get it (it looks quite not right), PCF does not change anything.
Maybe I just should wait for more stable version instead of finding new problems. Sorry ;)


Aside from those minor nits, this LGTM.  I'll be interested to see if the buildbot turns up anything.


test


The reason why you had to duplicate the three checking in `RunOfCards::RunHelper` class is because that class didn't have the `CardRunMethods` module included into it. When you define a class within a class, the nested class doesn't inherit anything from the class it resides in. Think of it more as a location, and not anything to do with inheritance.

For the methods involving checking of threes, it might make more sense to delegate that work to the `Card` class, since checking whether or not a card is a red three or a black three seems like something to do with cards in general, not just Canasta cards. That would allow you to write something like `Card.red_three?(card)` for any given card. Or better yet would be something like `card.three?(:red)` (where `card = Card.new(...)`), that way you only have to make one method, and it could also be used to check on a card being a three in general by omitting the color argument to the method.

Most of methods in `CanastaConstants` seem like things that the `Card` class should be responsible for knowing (twos, jokers, threes). If those moved to the `Card` class then you could move the `THREES` constant with them. The `is_wild?` and `is_natural?` methods are Canasta specific, so those feel like they belong. And obviously the scoring constants belong with Canasta.

When I look through things like this, I usually just ask myself "Where does the responsibility for `x` lie?" A lot of times, like with the different card-related methods, it's pretty obvious (to me, at least :-) Other times, like with the scoring-related stuff, it becomes murkier because right now it makes sense to have scoring shared between `RunOfCards` and the `RunOfCards::RunHelper` classes. Maybe later, as the code evolves, it will become clearer where the scoring code should live.


Perf: Consider moving the array creation to a static field initializer.


Yes, homepage.


AdvancedAFN (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1621 of 1621 tests passed)

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


I don't think so. This is the case were a query is cancelled while it's still queued, so it's corrected not to count it as started


Awesome!

If you drop the "self" in ```def self.execute_effects``, this becomes even more simple:

```ruby
  @branch.execute_effects(@character)
```

A good principle of object oriented programming is that you should mostly be calling methods on instances. If you're passing an instance as an argument to a method, it's a really good sign that the method belongs on that instance.

If you're passing two instances, then the method probably belongs on one of them.

That said, having this method be on Branch is a great step forward either way - most logic should live in model classes.


Hi Avinash, thank you for your interest in search idea. I think official opening for student's applications starts on March 16th, so let's follow Google's rules, good luck!


I don't think so since this isn't a user error. If this happens it's a bug in the queueing system, since you're suppose to acquire a queue permit first


Are min and max both supposed to be `defaultMin`? Won't this make the segment max always `Infinity`?


See the doc comment for this method (GitHub does not allow me to place a comment there). It says "Note that CSharp does support assigning a value to the constants", but probably means "Note that CSharp does NOT support assigning a value to the constants".


I don't see this value getting set anywhere and I don't see it used in the logs on push or pull. How is this supposed to be set and is it the same value as "vars.digest"?


84692518-Markdown-Documentation (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1122 of 1122 tests passed)

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


This change should be in cloudrouter-dockerfiles repo.


I really don't care myself but I saw you wanting correct authorship not long ago so maybe could have just mentioned where you got the theme fix. Now resume with your awesome rom building please LOL and thanks as well. I used to run aicp all the time until I starting building


`array` is not a very descriptive argument name. `children`? `childNodes`?


Dejalo de 6, el de ecuentros :cry:


I think you should use an
`<addr>` element here instead.


some tests failed:
failure: http://build.servo.org/builders/android/builds/938
failure: http://build.servo.org/builders/mac2/builds/1117


AdvancedAFN (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


oops, sorry. i keep breaking this as i experiment with different configurations of this file.


This should be #127


We have permission to open source this.


I notice that when your page first loads, the poster image on the video is much larger than the video itself. You can control the size of the video container to make sure it stays the same size all throughout page load. Something like this would work:

```
video {
    width: 1920px;
    height: 1080px;
}
```


Does that happen? Want to write a test for that? :)

On Mon, Mar 2, 2015 at 5:18 PM, Tanya Parker <notifications@github.com>
wrote:

> Yes, homepage.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/code-dot-org/code-dot-org/commit/c026511df937307775d65b3eda7d81f60f1451df#commitcomment-9997579>
> .
>


Nice work adding the background images!


Fair point. I can see why this wouldn't be useful in the main project.


@briannesbitt Any chance of bumping this up to a release? Laravel requires ~1.0, but we have tests failing constantly without this commit. For the moment, I'm using a composer alias (`dev-master as 1.15`), but I want to get rid of that as soon as possible.


saw approval from jdm
at https://github.com/psdh/servo/commit/dbc654ffa8960649ad9a74f964e8e44192717306


merging psdh/servo/foldsum = dbc654ff into auto


psdh/servo/foldsum = dbc654ff merged ok, testing candidate = 290b025f


What is `NULL`?


r+


btw, there's also a test for set_locale in
test/controllers/home_controller_test.rb (also a good example of how to
check that the cookie is set) -- you can put yours there too. It might be
easier for you to test the redirecting if you specify paths as strings in
the integration test than in the controller test.

On Mon, Mar 2, 2015 at 5:27 PM, Laurel Fan <laurel@code.org> wrote:

> Does that happen? Want to write a test for that? :)
>
> On Mon, Mar 2, 2015 at 5:18 PM, Tanya Parker <notifications@github.com>
> wrote:
>
>> Yes, homepage.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/code-dot-org/code-dot-org/commit/c026511df937307775d65b3eda7d81f60f1451df#commitcomment-9997579>
>> .
>>
>
>


Good idea, here you go: https://github.com/brooklynDev/airborne/commit/6eedb2a6089a7f52d3455ea19042fb14b96a3e4f


Unfortunately, `background-clip` does not refer to "video clip". It is used to apply a "clipping group", which is a digital imaging concept close to "masks". So it allows you to use an image to "cut out" a shape from a background image.


http://github.com - automatic!
[GitHub](http://github.com)


I don't think the ternary operator is necessary here.


This gets set automatically by `gorilla/mux`. It will have either a digest or a tag, as declared in the specification.


> What about backward compatibility, i.e. joining settings as before? Actually utilizing the already included knockout_prefix would make more sense for me.

It is possible by changing the rails_config-configuration. There is an option to use array overwriting.

> I could be wrong, but AFAICS your "Only define Engine if in Rails" patch to the fork is redundant (because already handled in rails_config.rb).

I did not see that. Thanks for the hint. Then the second commit in the fork is redundant. I will change this.

> people which know RailsConfig wouldn't expect this behavior

That's why we explicitly use a reference to a commit/branch in the Gemfile and comment it.
Maintaining it is not such a big problem here.


It is a bit, isn't it.


![GitHub Logo](/images/logo.png)
Format: ![Alt Text](url)


`null` :) Author was having a C++ day I expect.


Good!


```javascript
function fancyAlert(arg) {
  if(arg) {
    $.facebox({div:'#foo'})
  }
}
```


To the ending brackets of the if statements?


> Actually utilizing the already included knockout_prefix would make more sense for me.

The knockout prefix is not working in this case because you need a intermediate settings file that knocks the array out and then the third settings file adds the new value.


- [x] @mentions, #refs, [links](), **formatting**, and <del>tags</del> supported
- [x] list syntax required (any unordered or ordered list supported)
- [x] this is a complete item
- [ ] this is an incomplete item


Specifying the routes individually here (instead of using `resources`) is what caused you to need to use a more convoluted link syntax (e.g. `<%= link_to(object.name, {:controller => "classifications", :action => "show", :id => object.id}) %>`)


It seems a bit odd to duck type suites here. I'm a fan of Suites and Specs (Runnables) sharing a common interface whenever possible, so it seems weird that Specs know how to run themselves, but Suites don't.


I hardcoded the computer's code and went through player guesses the exact same way and in the same order as when I noticed the bug. This is how I found the index issue in the Mastermind::setResults() function


Either fill in some comments, or remove the empty XML element.


If `QueryFullProcessImageName` returns `false` then fullFileName is `null`.


El cambio no fue mio fue de deyby, Conectate a face, i need you


You got this element on the page, and I'm guessing you wanted it to go in the background of something. Unfortunately, there's a small typo in your `video` tag. It should be:

`<video autoplay loop poster="media/video/loop_poster.png" id="bgvid">`

Notice how the `id` attribute was included in the `poster` attribute before? (You just forgot an end quote.)

Once you have that `id` issue resolved, you can grab that element by styling `#bgvid` and put it behind something (using `z-index` and absolute positioning most likely).


First Header | Second Header
------------ | -------------
Content from cell 1 | Content from cell 2
Content in the first column | Content in the second column


16c999e8c71134401a78d4d46435517b2271d6ac
mojombo@16c999e8c71134401a78d4d46435517b2271d6ac
mojombo/github-flavored-markdown@16c999e8c71134401a78d4d46435517b2271d6ac


Woah -- tests! :+1:


Love this scheme... would it be possible to get a version of it that works in VIM (console)?  This scheme currently only works in GUI-VIM.


@jsgorman look at this.


Are these used anywhere?


86046200_CondenserToZone (Nigusse) - x86_64-MacOS-10.9-clang: OK (1455 of 1598 tests passed)

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


Temporary metrics can be requested through show/hideColumns (I have no opinion on whether this is useful or not). If showColumns is used, the result should remain the same, since all other columns will be removed, including temporary ones if they are not requested. If hideColumns is used, perhaps temporary columns should be removed too...


~~this~~


I can't find details about this radar number. I'm currently using Xcode 6.3 beta with the "swift12" branch and I'm getting a compilation error because of this change. Can you help me understand this better?


Yes, that is the idea. Thanks!


I might call this `root` or `rootNode`.


Keep the SPI flash core even if ``with_rom`` is enabled, so that flash booting in the BIOS still works.


89445170CoilVsSpacePeak_ExampleFiles (wfbuhl) - i386-Windows-7-VisualStudio-12: OK (1620 of 1620 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 is almost right! You forgot about the `dd` for the "definition". Remember, `dt` is the "term" and `dd` is the "definition". So each section of this list should actually go like this:

```
<dl>
    <dt>Born:</dt>
    <dd>January 19, 1809</dd>
    ...
</dl>
```


Did this work for you?  `@animal.common_name` should have been an array-like thing, since animal can have _many_ common names.


works for me!
thank you.


Same as above.


This explains #95.


It was more about getting people up and running quickly. Some defaults for these values would also work. This commit was more than 4 months ago, but I believe things failed silently on the commit I was using when  these variables were not set. This was most likely just an issue with GEM_SERVER_LOG being null, but I can't really recall now.


Need empty line after this line for rendering v1.4.2 heading correctly.


Thanks!


Is it supposed to call `stream.ToArray()` rather than `stream.GetBuffer()`?


Shame on you Davor LOL


This all looks great, @evelynbernardo.  The only thing to watch out for is naming variables with Capital letters.  In Ruby it's only constants, classes and modules that start with a capital, so we'd want to rename your `@Classifications` to `@classifications`.


``"".format()``


OSS Header


Explicit visibility modifiers


ok


No -- GetBuffer() avoids an allocation and we know it works because we created and hold the MemoryStream.


Pass a string comparer to `StartsWith`


bu OK


Why didn't we make this an async method that used await here?


no further questions...r+


are you trying to do a giant light for the whole level?

On Mon, Mar 2, 2015 at 10:16 PM, Krzycho666 <notifications@github.com>
wrote:

> ESM gives me some results, but in the way that I don't get it (it looks
> quite not right), PCF does not change anything.
> Maybe I just should wait for more stable version instead of finding new
> problems. Sorry ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/okamstudio/godot/commit/a1f715a4da71fbc2b7d6fad68624bf8b22c6da17#commitcomment-9997560>
> .
>


A todo item for later.. figure out how to DRY this `120` and the one in the CSS. Maybe we need a `spotlight/variables.scss.erb` file or something, so we can use engine configuration?


The exception thrown by setting `OutputEncoding` is treated as non-fatal here but above results in an unhandled exception.  That seems odd to me.


In some cases 1 is used as an exit code and others -1.  Lets make it a constant and use that uniformly across the code.


Hey.
I'm sure it works in console Vim as that's where I used to use it.

Could you detail exactly what's broken for you in an [issue](https://github.com/adlawson/vim-sorcerer/issues/new) and I'll try to help in the morning (I realise issues were turned off before but they're back now :+1:)


Explicit modifiers and OSS header


Thanks.


Hey Marcos- Not sure this is right. I made some edits in some of the html text. Let me know if it's not right or you have trouble retrieving. I will check github during work tomorrow to check in.

Thank you!
V


Why does this need to be `public` now?


Kill it with :fire:


https://github.com/hx/private-gem-server/pull/2 :)


Use a `StringComparer` when calling `StartsWith`


lgtm


thanks @hail2u

pushed as 6433ad1 without review, I hope I don't get in trouble!


I meant string.Equals will return false anyway.


这个没必要改吧?


@Brock17 uhh i also just tried deleting atmosphere.wav.meta and got the same result as last time the old sounds still play.. hmm.


There's already `buildbot.util.flatten` for this purpose


:+1: I had thought about that for a bit, but knowing we may end up making this configurable (and different sizes) I figured we could look into it once we knew more about what we're going to do there.

Also, these CarrierWave versions are evaluated at class loading time (I think) which makes dynamically changing these numbers....difficult.


@davebx Can you sanity check this?  It absolutely blew up before, but I just want to make sure verifying tool_dependencies_dict has contents works with what the intent is here.


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


It'd be great if the list itself could be a renderable, too.


Shouldn't a slash only be added if req.url is not empty as was the case when the blueprint used `path.join` (incorrectly, as that created invalid urls)?  In my use req.url is often equal to `"/"` so I get 404s by default on my backend because of the trailing slash.

For example a proxy of `localhost:4200/posts` becomes `backendserver.com/posts/` by default and 404s because of the slash on my python flask backend.

In my proxies I'm using the following for correct behavior:
```
    if (req.url.length > 1) {
      req.url = proxyPath + '/' + req.url;
    } else {
      req.url = proxyPath;
    }
```


self.__sortquery = bson_encode {['$query'] = self.__query, ['$orderby'] = key_list}

This would be better ?


kkkk


In general, I think this looks good. I still don't fully grok how the tree segmentation algorithm works, but I wasn't about to find any cases where it broke.

I like the more explicit tree structure and pulling some logic out of Env.


hmm you're right thanks!


Sure, I have no issue with that. Just wanted to know about the implications :)
It's probably best to document this somewhere, though, lest we forget about it somewhere down the line.


Without ```id``` explicit field, does this
https://github.com/pedrobaeza/partner-contact/blob/c4ac5ed58a84299dad0c95a254fbbdda8da6a2e4/base_partner_merge/base_partner_merge_view.xml#L104
still work?


Hmm. is this really a "Redis Error" or an error reading Redis instance name from some different data store?


@geigerb shouldn't you have ```counter = counter/vel_vec_red_lev``` here?


The `block` variable name is terribly ambiguous here, let's just name it sequence or so.


Tulilmshar -> Tulimshar ;-)


Correct, I meant BSS, thanks! :-)


Can we be more explicit about logger error message -> "Error"


Приветствую,  Zorgzerg!
Присоединяюсь к выше сказанному комментарию большое спасибо за minimap особенно за последние обновления это то что нужно супер.
Есть правда ещё одна просьба смотри, раньше я использовал один плагин ( Overscroll ) но сейчас он после обновления Brackets перестал работать и я могу сказать что он теперь даже и не нужен мне так как появилась возможность просто в настройках файла brackets.jcon просто прописать ("scrollPastEnd": true ) и всё и работает даже лучше чем сам ( Overscroll ).
Но за то потом появляется другая проблема с Minimap то есть скрол minimapa начинает работать не совсем правильно
Если можно исправить этот баг было бы хорошо.
![2015-3-3_14-55-19 835](https://cloud.githubusercontent.com/assets/9529516/6460047/c34b0322-c1b5-11e4-8fe0-249252bb0e1d.jpg)


Indeed, that's an option which is easy to setup and will run everywhere, much better.


Did you make sure that the JavaScript translation file was generated when you performed your test?


tits ftw? :dancer:


@psvmcc да, для того что бы сохранить lost надо обновление ставить руками и не выполнять запросы относящиеся к удалению данных lost`а
добавил в TODO


logger.error("Redis Error", e);

Should we also change from + to , in logger.error?
so we invoke this:  void error(java.lang.String s, java.lang.Throwable throwable);


Should we also change from + to , in logger.error?
so we invoke this:  void error(java.lang.String s, java.lang.Throwable throwable);


@MAXakaWIZARD Yes, you do not need to do nothing with that, it will be mapped automatically.


Je me demande s'il faudrait mieux pas passer par un [ElementVisitor](http://docs.oracle.com/javase/7/docs/api/javax/lang/model/element/ElementVisitor.html).

Exemple existant dans Linker: https://github.com/softwarevidal/jax-rs-linker/blob/master/jax-rs-linker/src/main/java/fr/vidal/oss/jax_rs_linker/visitor/ApplicationNameVisitor.java.


Good point, I'll remove


Well, I have to try it. Thanks for pointing it.


swiTch :)


Should we change from + to , in logger.error?
so we invoke this:  void error(java.lang.String s, java.lang.Throwable throwable);


please add brackets.


I will cancel it out.


Probably this comes from the DNB: http://www.dnb.de/SharedDocs/Downloads/DE/DNB/standardisierung/marc21FieldsPnd2009En.pdf?__blob=publicationFile
May we should transfer this later to 500? The GND uses this field for relationships now (e.g. http://d-nb.info/11850553X/about/marcxml) and we are more in the standard.


this get(key)  or get(returnSanitizedKey(key))


this get(key) or get(returnSanitizedKey(key))


Should we change from + to , in logger.error?
so we invoke this: void error(java.lang.String s, java.lang.Throwable throwable);


you might well be correct there - yes, it would be better


This is incorrect english


+ return type == 'void' non ?


yes!


I've already added a header, you can see this in action here:

https://github.com/Particular/NServiceBus/pull/2665/files#diff-85995efae31974cc95e2a375d7f7e8a6R23


separado em 4 arquivos (1.9.1, 1.9.2, 1.9.3 e 1.9.4)


Ok I think i get it now. I was confused by what you meant by saving every 2,4,... index. I thought it was taking every 2,4,...index in the full velocity vector and shoving all of them into the front of the array.  But what you meant is taking every 2,4,... velocity vector and then storing it. I had it backwards where you save every vector and then throw some out.

I will need to think on it some more.


Great!


Reviewed and approved.


Very cool :+1: Yeah, the synchronisation step sounds like a pain, but it'd be awesome to automate creation of node modules (I would love to add this as a publish step to https://github.com/blakeembrey/node-module-template).


log the exception as well.


e.URL.User, e.User, e.BaseURL.User, user = bzrOwner, bzrOwner, bzrOwner, bzrOwner


@psvmcc, а у Вас лост еще не отвалился? Или он опять начал работать?


Please exclude the Lexer.hs change for now. Those types look a little gross to me but frankly I don't know how to do that properly. I'm not even sure if the lexer should be restricting the monad to `ReaderT env IO`. I need some time to look at that but I don't want to hold your PR for this.


oki


Do we need that, its obsoleted with error so no one is able to call it?


Because it is incompatible with preload_app which is enabled by default: https://github.com/puma/puma/blob/master/examples/config.rb#L132


I like it that you found this bug! Great!


My understanding, from the spec, is that non-browsable virtual folders are still used to prioritise (i.e. sort)


Created #171


Хм... а у меня нет такого brackets.json. Его руками создавать нужно? И где?
Мне б пронаблюдать своими глазами

3 марта 2015 г., 16:03 пользователь Aleksandr <notifications@github.com>
написал:

> Приветствую, Zorgzerg!
> Присоединяюсь к выше сказанному комментарию большое спасибо за minimap
> особенно за последние обновления это то что нужно супер.
> Есть правда ещё одна просьба смотри, раньше я использовал один плагин (
> Overscroll ) но сейчас он после обновления Brackets перестал работать и я
> могу сказать что он теперь даже и не нужен мне так как появилась
> возможность просто в настройках файла brackets.jcon просто прописать
> ("scrollPastEnd": true ) и всё и работает даже лучше чем сам ( Overscroll ).
> Но за то потом появляется другая проблема с Minimap то есть скрол minimapa
> начинает работать не совсем правильно
> Если можно исправить этот баг было бы хорошо.
> [image: 2015-3-3_14-55-19 835]
> <https://cloud.githubusercontent.com/assets/9529516/6460047/c34b0322-c1b5-11e4-8fe0-249252bb0e1d.jpg>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zorgzerg/brackets-minimap/commit/0506231a672e9dabe9e272a2aeb23a2cf4be2fd3#commitcomment-10002751>
> .
>


oh noez :(


That is a huge Goblin.


ansible-1.8.2-1.fc21.noarch and python-jinja2-2.7.3-2.fc21.noarch
I will update ansible.


It might be easier to put this all in a table and actually use the name of the JSON variable.  The descriptions below can then just be a column


any in-the-middle lib without strict nuget ranges could still call it


I mean something like :
```
std operator+(char const (&self)[N], std const& other) {
  return other + self;
}
```


Why do you want to mention maven-scr-plugin version here ? ideally this should come from Carbon parent POM if not right place is <pluginmanagement> section of parent POM of this repo.


This should come from Carbon parent POM if not right place is section of parent POM of this repo.


Already fixed (do U look at individual commits or the entire change set)


Indead, the default alike (line 270) will capture it and set it to false.


Lets rather avoid writing docs that infer some future way.  We might ever do that future way, so write the docs for the present.

*To add or modify the properties of virtual folders use the add_vfolder command.*


I think it's not. the role of ``alike`` is to detect std::tuple patterns that could be converted to array of string.


On windows 8.1 professional line 55 throws an exception:

Traceback (most recent call last):
  File "SickRage\sickbeard.py", line 55, in <module>
    ssl._create_default_https_context = ssl._create_unverified_context
AttributeError: 'module' object has no attribute '_create_unverified_context'


This command...

I'd link to description of said JSON file.


Agreed, @SimonCropp has been saying that as well!

Lets introduce that on the handler context first? (I'd say out of scope in this pull)


Пардон, нашел ) Буду разбираться, ок


Interesting!

This adds another level of wrapping, which might be undesirable in the future.

My main concern is how to handle different classes of return values.
Like, `3.taint` gives a `RuntimeError: can't modify frozen Fixnum` in some versions of MRI (seems fixed on recent Ruby though). Same could be for `nil` since it is frozen as well in some versions.
Ruby 2.2 seems to fix this by having tainting have no effect on frozen objects, so this might become less important (we just have useless calls to `nil.taint` but that is fine).


My understanding is that if they are not browsable, then they are not displayed, and thus it is stupid to have them in account for sorting. In other words, for me browsable means enabled. And yes, the specs say that.

The way this is implemented, when adding a vfolder that is no browsable, no relationships with matching units are saved. But if you alter browsability and the vfolder is saved (necessary to persist the change) then the relationships are calculated and stored, and thus they can be used for sorting.


You're right. I'll remove this commit and push a correct one.


Sorry, I don't understand :-/


What should we use instead?


Thanks for review, I am not sure I will be able to continue today's evening. I'll show the more code Thursday morning


vérifier cette ligne


Can you explain why do you need DynamicImport here  ?


Please configure your IDE to avoid wild card imports like this ?


Documentation


Later we have to the version to the URL.


Better do to it on initialization step (like `var myTpl = ...` in global namespace) or in `init` function.


Try not to write from a technical implementation perspective, rather write from a functional experience.

*If a virtual folder applies in the current location, then clicking on the links on the overview page will provide the units in priority order when editing in the editor* (Sorry that isn't perfect either)


Let's not box when we can avoid it ;)


This fix now resolves the "lower is better" score flagship.


Um, doesn't this override the active status for every single vertex primitive variable, not just Cs?


@dwaynebailey They are written for the present. This doc is added on the third commit in the PR, being adding the management command the second commit.


test


Remove empty line


Is this code simply redundant now, or has removing this fixed a bug?


This look not nice to me, can't we use proper data structure here instead of 3 inner Maps ?


This comment has fixed LANG-1077, but it produce another bug. such as
StringUtils.ordinalIndexOf("abaabaa", "ab", 1) != 0


since when "a" + b == b + "a" ?


Corrected commit is 034c923


Where do these `@SuppressWarnings` come from?
It makes it very hard to read method parameters with 2+ parameters under a non warning-collapsing editor such as Eclipse.


Short explanation of none - even its documented in dev portal


I'd suggest moving all this JSON specific documentation out of here and into the actual JSON file.  So that the JSON file essentially is self documenting with good examples.


Its redundant now. the default cast is used as the casting constructor (if one isn't explicitly set) so an object is always created


Short docu for exception


If ssize == 0, It means we have something like : "".join("lalalalalal"). In this case, you can return "lalalalalalal" instead of doing an explicit copy and then move it.


I've run this test: https://gist.github.com/mgencur/a1b662355da4bb013e26 and the ReadWrite ratio is 0.25. I would expect it to be 1.0.
As I understand it, the current implementation returns average ReadWrite ratio per cache. Is this expected? I'd expect a sum of all ReadWrite ratios in all caches. Thanks


I agree with @julen that this clutters things.  The command page is massive as it is.  Rather put the spec in a none included JSON file or on its own page.  That allows scope to give complex and simple examples for anyone wanting to understand how this could be done for their situation.


Why not to move this piece of code to separate function like `getCurrentDayEntries()` which will be like this:

```go
for _, entry := range journal.Entry {
  if entry.Day == getCurrentDay() {
    return Journal{
      Entry: []JournalEntry{entry},
    }
  }
}
```


* Results:
 * Height estimation
I believe we follow too much the sonar, leading to jumps in the height estimation:
![kalman3_baro_sonar_estheight](https://cloud.githubusercontent.com/assets/6390059/6460419/6a641654-c197-11e4-8a47-155b913e1e5f.png)
The estimated height goes below zero
![kalman3_baro_sonar_estheight_negative](https://cloud.githubusercontent.com/assets/6390059/6460470/c2567ce4-c197-11e4-8901-1ab8385bbaf7.png)
Anyway, we perform already better than the complementary filter previously used
![kalman3_baro_sonar_estheight_posestrel](https://cloud.githubusercontent.com/assets/6390059/6460495/efbff07a-c197-11e4-83ac-4d157f81d175.png)

 * Altitude estimation
Here the GPS seems not to drift too much compared to the pressure sensor
![kalman3_baro_gps_estalt_drift](https://cloud.githubusercontent.com/assets/6390059/6460438/92936ea4-c197-11e4-8a27-4b785b662017.png)


Calling this amount of API calls here is not nice, introduce a variable, process the value and pass it into error() method.


Note: ViewableData::cachedCall(), ViewableData::hasValue(), and ViewableData::XML_val() pass "false" to forceObject and that's where the trouble starts. If they didn't do that, this additional line would have been unnecessary.

In just adding this line, we've got default_cast overrides scattered at a few places and we can probably remove the up-stream ones (escapeTypeForField & obj)


I think we have missed something here about not-browseable. My reading of the spec is simply that they are not displayed on the overview, but that they still influence the priority ordering.  Perhaps the issue is that those two things should be separate.


... sorry :-)


1. Error logging is not consistent .
2.   e.getMessage() will result into drop the stack trace.


@KrishnaGumpu correct.


This might be a good time to raise the idea of annotations again ;-)


You need a variable for this (executionInfo.getNextConnectionTime() - currentTime)


Given the changes you're making here, we need to assume that this will be an upgrade challenge. I think that justifies embargoing this until 4.0. Also, we should put some instructions in the upgrade docs before merging the PR - even just a simple link to "look what we did in Form.php to get this working" ;-)


How are you doing the cross-compilation? I haven't managed to get anything to work (on my mac).


test


I think it might be better to have tests for this change. I have a scenario where
```
instance 1  |  instance 2  |  instance 3
[PENDING]     [SLAVE]        [SLAVE]
-----------masterIsElected(1)------------
[TO_MASTER]   [SLAVE]        [SLAVE]
-----------masterIsAvailable(1)----------
[MASTER]      [SLAVE]        [SLAVE]
```
In that scenario,
question 1: what would happen if we pull updates when instance 1 is in `TO_MASTER` state?
question 2: if we pull updates when instance 1 is in `MASTER` state, what would happen? Hint: how should update puller response to `InvalidEpochException`? At the end of the scenario, instance 1 have a new epoch, but instance 2 & 3 are still using a stale epoch. From the update puller's point of view, the invalidEpochException will be handled https://github.com/neo4j/neo4j/blob/master/enterprise/com/src/main/java/org/neo4j/com/Client.java#L251

So it would be great helpful if we have some tests address the above questions.


* Results
 * Height estimation
Same issue than with the 4D filter with the jump is the estimation, otherwise quite good
![kalman4_baro_sonar_posest_estheight](https://cloud.githubusercontent.com/assets/6390059/6460569/82d5326c-c198-11e4-95f8-2bd0e05416d0.png)
 * Altitude estimation
Here the barometer offset has probably something to do with the difference between the estimation and the GPS raw value
![kalman4_baro_gps_posest_estalt](https://cloud.githubusercontent.com/assets/6390059/6460575/8e59102c-c198-11e4-8a1c-6d7f07c4cf46.png)

 * baro biais estimation
The biais of the baro is quite constant during the flight
![kalman4_baro_biais](https://cloud.githubusercontent.com/assets/6390059/6460565/79582280-c198-11e4-9a2b-7a856752432a.png)


A few more infos on what still needs to be done would be more productive..


do you recover now the total time for integration ? or are there still lost micro/nano seconds ?


Move String literals into a separate constant class.


`policy_scope` can be omitted here, as `@posts` is the result of using `policy_scope(Post)` in `PostsController#index`, there's no advantage in applying `policy_scope` a 2nd time.


Again you are not logging exception.


also geopc is common to all thread and used only to read.
Would it be better to initialize it once and use it in all theads ?


Can you change this line to:

+audio_not_allowed=Your browser does not have permission to use the microphone. Please adjust the permissions in your browser to use it and refresh the page.


I'd split this into two files, one for object literals and another one for number/boolean literals.

For applicable primitives types (Boolean, Number, String, Symbol), I'd probably even like to cover all eight cases (Boolean, Number, String, Symbol) x (Symbol.iterator present on primitive prototype, Symbol.iterator not present). Maybe in a follow-up issue?


So are you saying we should put a throw new NotImplemented on the getter and setters?


OK but you wrote "cashed" instead of "cached".


Yep, I will look into it as soon I have a little more time on my hands.


add to clone


You can also use 'return len(stk) == 0'


yes. otherwise it is a hidden bug


stop clobbering our newlines at the end of our files D:


Yes you're right. Much better. I should have also used a different function name, `exit` is too common and there's risk of collision.


I can't, because what I care is user's relation to the *current* project. I will make this more clear by extracting this logic into a separate function.


Heh


I'm curious to know why these changes were needed. Are we pushing much more content into HTMLValue now? If so, I'm worried what the performance impact is, because HTMLValue wasn't really designed to be used lots of times on a pageview: I expect the DOMDocument instantiation is a bit slow.


use === (don't forget space)


use ===


@ThinkFlipp This destroys the loading of ActiveInspector as a data source in apps. What are you using it for?


also moduleFns is unused


I can now build without this patch. Arch must have fixed something in the latest jdk update.
I will remove this patch when I push my next batch of updates.


Avoid such identifier as 'dict', since it's a built-in identifier in Python.
After setting 'dict' like this, a single line as 'a = dict()' in this function will raise unexpected error.


Don't sweat it. I fixed the issue with ShaderKey that was causing the ShaderNodes based material to crash, and appart from the inversion of tessellation shader we talked about it seems to work fine.
It's just that it's been merged a bit hastily, that's all, and that's not on your end.
I'll comment in the PR if I find other issues.
Gonna try to make it fit with ShaderNodes


Hi,

It's still working, just remove if ($this->page_name == 'index') in function hookHeader

best regards.


This breaks the public API for limitExceeded which is now always set to NO unless we bind slk_limitExceeded to update limitExceeded. (and therefore breaks the enabled state on the send button). I think this should maybe just be considered a public API and not have the prefix.


75127368_DCFSEC (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Indeed, i was expecting at least a little discussion and a few required changes before the merge ;)
No problem on my side, just want to say that if something still needs a refactoring/fix i am willing to help.


u can suppress the "required doco" warning at the file level


dict.get(c) is enough. None is a default argument.
Get familiar with the argument format used in python document, for example, "get(key[, default])" in https://docs.python.org/2/library/stdtypes.html#dict.get


I'm not sure what the top two files are changed for but my guess is that it's changes which AS or Gradle does automatically after making the changes that I've done.


Hello Thomas,

Yes, I did meet this problem in one of my JUnits in production failing, and
took me a while to debug hehe. For some reason, I did not have this problem
with UISpec4j 2.4 and jdk6.

Regards,
Ruben

On 2 March 2015 at 01:45, Thomas Santana <notifications@github.com> wrote:

> This is an interesting addition. Have you hit this problem on real
> production code?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lqbweb/UISpec4J/commit/dfd4cd77ef4a2f89c284cb7a4daf892bb3e79fa4#commitcomment-9977654>
> .
>


I have started translating the code gen for the new AST and I've got some issues.

* Are we agreed on the single expression instead of a list of expressions?
* When will the AST annotated with (Maybe Signature) be translated into an AST annotated with Signature Will that be a guaranteed case for the code gen?
* Do you think the beamwriter module should be scrapped and the function within just moved to utils?
* Some minor build errors: some modules are still importing Parser.Errm instead of Utils.Errm, I needed to regenerate the BNFC generated stuff cause it failed to build, and now the big pile of warnings are back (did I mess up my local build?), Main.hs seems to have some compilation issues.

I guess some of the problems might be changes that wasn't added to commits.


我这样回复你能看到咩[星星眼]


75127368_DCFSEC (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1619 of 1625 tests passed)

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


Do not use yum on Debian/Ubuntu/Mint.
@cs-shadow Please rectify this and resend a pull request.


Radius of 105 = protectionRange of 210... not within the borders of most islands.

I think we need to limit the onIsland calculation to be within the island at hand... not currently checked.


:+1: thx for your work on lower-side code. in some other areas also helpful.


@jpfr not really, because nodeid e.g. for ```UA_UInt32``` does not have ```UA_UInt32_binaryEncoding``` counterpart, if i increase everything by 2 it breaks.... I think only strucutred types (i.e. those with additional *_binaryEncoding IDs) are to inrease.... jsut guessing, though...


Shouldn't this follow the same rules as any addon when it comes to versioning? If you have the same addon (but different content) in multiple repositories the version number of the newer repository must be significantly higher than the one in the older repository. So if we have version `1.0.0` of `resource.language.en_gb` in Isengard we need to have version `2.0.0` of the same addon in Jonathan ;-)
The current situation where we have a newer version of an addon in an older repository is IMO not something we should ever do.


It's a chance to learn more.


good boy :)


There is a much simpler way:
key in d
Return True if d has a key key, else False.
See https://docs.python.org/2/library/stdtypes.html#typesmapping.
Be familiar with available methods of common data structures, you can try them with simple code snippet, or ipython.


changes to readme, further changes will follow.


This commit broke ./build.sh as $clang could be a relative path and that does not work if you are in subdirs of the project. If you run build.sh you will get some errors about ../llvm/Release/bin/clang not being found.

You can fix it by using something like ```clang=$(readlink -f "$llvmdir/bin/clang")``` to convert it to an absolute path


OK, that's indeed what I saw.


> Expose trace();

This is great. always copy pasted that part from base class into my own error handlers to get the highlighted trace on my dev website. this will be handy.


* Yes, but I havn't pushed anything with that fix
* Yes I think so. The function type of the type checker is right now `Module (Maybe Signature) -> Err (Module Signature)`.
* Or just move the beamwriter module to utils?
* It was just a proposal to use Utils.ErrM, I havn't changed anything yet. The rest I'm working on right now (got a lot of errors in LexHopper.x now :/ )


@jdesboeufs tu valides ?


What if i will provide PR with this change ?


Ok good! I'll move the beamwriter in my upcoming commit then. I'll make sure to just commit the code gen changes so to not interfere with your work


Please allow $filename to be null by default. This will keep retro compatibility.


Note that `foldl1` is a partial function and may throw exceptions. :)


`if (is_null($filemame))` looks better


Mispelled *canola*.


`if (is_null($filename))` looks better


yes.

well, I didn't just remove it.

I also made an attempt to fix this.. setting the proper varid near line 6567 had unwanted side effects. that didn't feel very solid.

this is code that we do want to remove. The entire Tokenizer::simplifyKnownVariables() should be refactored away.

I figured that making a hacky fix in this code can lead to further bugs and can delay the refactoring.


Okay, no problem http://hastebin.com/qirepevivi.bash
edit and copy/past to terminal. In this script variable BUILD points to /media/Kernels/build. Installer should be in same level as BUILD folder, ie, in /media/Kernels.


Google 'python range xrange'.
Answer why xrange is better than range here.
Then read: [For Loops](https://wiki.python.org/moin/ForLoop),
[Generators](https://wiki.python.org/moin/Generators).  These would be the first thing you learn in python that significantly differs from other languages you've known.


a contextual note would be nice, so something like:
#. name of an addon category


Yes, I know. But after type checking, we have at least one data constructor in `Data` expr and one pattern in `Case` expr, so I think we can use it safely.


are we really limited to WAV files here or could this be extended to use all supported sound extensions?


That's what I meant. ```UA_UInt32``` is a builtin type.
It never reaches that code branch.


I see.


Apart from that, I am not satisfied with the naming --ones of the hardest problems, for various reasons (it doesn't reflect the method properly, it sounds funny, etc..), ideas are more than welcome.

Also I will update comment to reflect the const-correctnes which I sneaked in this commit, maybe split it instead of 'and' ing in the comment?


Cool


Just curious what was the problem with popd/pushd?


Not sure what direction your comment is about.
yes Jonathan should have a higher version number. Could be 1.1.x as well. We could also follow 1.15.x version (at least for the language add-ons to make it more clear).
    X = feature version (say we head to plural addition)
    Y = Kodi version
    Z = language update

It doesn't matter if Isengard has en_gb 1.0.24 in repo and nothing in J** repo but does have en_gb 1.1.0 as default core addon again. Because the default shipped is higher than available in combined repo it won't update. For any other language we should indeed push a higher version to repo (if needed).


Hi there!  Could you explain what the removal of the >DESIGNER and >VERSION fields fixes?  I know there's pros and cons with both local and global attributes, I'm just curious about the rationale specifically for this symbol :)


@cristianomatos https://github.com/crdroidandroid/android_frameworks_base/commit/75d78fdb9d6999ac361def19faba43ee5f9e9634


Use 'enumerate' to simplify the code. See https://docs.python.org/2.3/whatsnew/section-enumerate.html


Does this change the button colour from red to blue? The red colouring is there to ensure the user is left in no doubt which is the "dangerous" operation.

It may be that setting it to default in this manner, while pragmatic, may need an alternative implementation on this screen.


I tested in my local machine and it wasnt working. Maybe I can put back that code and commit again


still if you remove 'if(src->type->isStructure == UA_TRUE){', it will, imho break


I usually look at individual commits. Good point. Need to change that


hello


Ok.


You should *NOT* persist this ID in the user defaults, as this would also be used when another device will be restored from backup, which will then end up being used for multiple devices and potentially different users. Instead the keychain provides a proper place for also ensuring the ID is not synced on any other device.


Some formatting here, pls. Either indent the cascade or put it in single line.


When we used to be part of the Graal repository we used CheckStyle, which warned about unused parameters, hence the annotation to suppress that warning. They should go. I thought I did I big search-replace a while ago but maybe I didn't catch them all.


This is invalid


This would read a lot more naturally if you supported
validating_params_has(:single_required_param)


This should not be there.


Может, это логичнее вынести для всех форматов, а не только json?


I'm not entirely sure about this, but I think the CGUIAudioManager only loads wav files. Will have to look into that.


Indeed, it should be `{@link Difficulty Difficulties}` in order to preserve both the language syntax and the link itself.


Т.е., получить события для всех устройств сразу невозможно?


Hmm. Dunno.
How can we test this?


Ara em dóna un error al destructor ja que estava provant alguna forma de fer el Assert que em funcionés. Llavors no ho he pogut deixar com abans ja que m'ha donat un error de LINK amb el .dll del UnitTest i he preferit no pujar-lo


Думается, статус должен быть другим.


I pulled the taint and is-tainted nodes into nodes separate from Kernel, so they don't need some of the ceremony of a more method node (RubyNode[] etc). This could therefore be IsTaintedNode and TaintNode.


@cristianomatos needs this too https://github.com/crdroidandroid/android_packages_apps_Settings/commit/f7a555944ef7230b6966fa313b66e4ba28d9f1bd


Всё тот же вопрос — резонно это вынести не только для json.


Comment end on its own line and aligned please.

```javascript
/*
 * This is a comment
 */
```


What is the recomended way to do this? I want to reuse the Pat grammar for arguments but they differ with that arguments can be a empty list while [Pat] can't.


Why's it called `taintSource` - it taints the result doesn't it? Apart from that, good idea. More declarative is good.


Я предлагаю перевести то, что у нас уже есть ([на основе справочника](http://apps.workflower.fi/vocabs/css/en)) и как только мы закончим — начать улучшать. По шагу за раз.


Возможно, лучше добавить в index через доп. параметр (?folded=true или что-то вроде), чтобы не дублировать код.


Looks like you removed one half of your emphasis here and not the other :P


1 Letter was approriate, it was to make Inventory Window Smaller(and have sensible width) , since many people become mad with long Inventory,
me don't care with 3 letters, since that's also appropriate and having sensible meaning, rather than predications.


@SelenIT сделайте ПР с вашим вариантом, пожалуйста, а там Вадик разберется


Ещё надо делать проверку на таймаут, потому что так прощёлкаются клювом периоды, когда события не регистрировались.


@vbuterin : what is the encryption AES , ECIES ?


I'm not sure why this change is needed. It makes the diff much more complicated than it needs to be, as far as I can see there are only real changes from line 426, but I had to spend a bunch of time verifying that. Please keep commits focused on the task at hand.


Тот ли статус?


This is very nice. Extremely succinct, replacing 20 lines with 3 more readable ones. Demonstrates the value of your approach here.


Тот же вопрос: для всех устройств сразу события получить невозможно.


Intéressant...


Since a list of Pat can't be empty I don't think it is possible. Maybe if we make the list of Pat able to be empty, and solve that special case instead?


_hm_

The general handling of 'constness' in typelib looks rather dodgy (and using _const_cast_ where needed). The [mutable](http://en.cppreference.com/w/cpp/language/cv) keyword is what should've been used? Well, just mentioning -- the code is as it is, and trying to unravel it is not trivial... We just have to keep in mind that there _might_ be undefined behavior ;-)


The problem is "inpArray" is a bad name in inpArray=ones(3,10); I suggest you do the following:
timeVec=1:10;
inpMat=ones(3,10);
inpArgList={inpMat,timeVec};
check();
%
timeVec=1:10;
inpArray=ones(3,1,10);
inpArgList={inpArray,timeVec};
check();
----------
function check()
intObj=gras.mat.interp.MatrixNearestInterp(inpArgList{:});
 resArray=intObj.evaluate(interpTimeVec);
isOk=max(abs(expArray(:)-resArray(:)))<=CMP_TOL;
mlunitext.assert(isOk);
end


Hmm, so you're slightly giving up a little validation here, but it doesn't matter because you actually ignore params[:card_index] in the resolve block? If someone hacks the response to point at a different card_index, it would previously have errored, whereas now it just treats it as a set aside instruction. Fair enough. Might be worth adding a comment though so that if you copy this code for a slightly-similar card in future you don't forget what's going on.


Change wb_amr into nb_amr for all, look at my repos! already fixed these commits to work for lollipop a few weeks ago.

Cheers
BlackDragon


Re https://forums.zotero.org/discussion/46078/2/zotero-integration-error/ I'm able to reproduce it on a Windows XP machine. Using [Dependency Walker](http://www.dependencywalker.com/) I can see that there's some checksum mismatch between linked DLLs and available DLLs on the machine.
![dep-walker](https://cloud.githubusercontent.com/assets/1381759/6461458/bd48de9a-c165-11e4-8c50-2ffee15a4d70.PNG)

This is not the case for Fx35 component. However, I see the same mismatches on a Windows 7 machine, which works fine, so I'm not really sure that this is the culprit.


I think you need to define somewhere, perhaps here, what "shorthand" and "longhand" mean in this context. Just something like `$ne is a query shorthand for the longhand { $not: { $eq: .. } }`.


er, NOT known by ...?


75127368_DCFSEC (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (625 of 625 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-3.65%25-green.svg)


This comment doesn't make sense -- pass back to where (e.g., I see no return in this `if`).


javadoc what this method does.


Should probalby also be a new commit given you're adding functionality in a commit that claims to be a refactoring.


@pepelsbey у справочника уже есть [русская версия](http://apps.workflower.fi/vocabs/css/ru) авторства @meecrobe, она неправильная?

@tachisis, понял, сделаю.


don't you have the problem issued by #19 ?


test add


Delayed until I have a useful Gitter client for Windows Phone...


I am also implementing something similar to your code. The previous version was using 0.4.2 api but now, I changed it to 0.4.4 and I did the same changes as you did but my output image is not as it was:

![before](https://cloud.githubusercontent.com/assets/4034259/6461553/15401394-c1a1-11e4-8900-b10ed762b8a5.png)
Before Changes

![after](https://cloud.githubusercontent.com/assets/4034259/6461554/1abcdcee-c1a1-11e4-9ae5-ccbb1bd32ef4.png)
After Changes


Maybe I should've written here... But I think that it might be better to keep them separated, in case more differences show up. I think it's better to have a somewhat bloated grammar then a compiler with more special cases than necessary.


shh


I guess the confusing thing here is that your comment refers to the else branch but appears to refer to the non-else branch.


No, change your defaults in config/deploy.rb to:
```
set :puma_preload_app, false
set :puma_prune_bundler, true
```


FWIW I find the code more useful if you leave in this comment that was on lines 73-75. The match block on the new line 76 does imply it, but the code is more approachable (and thus more able for newcomers to contribute to) if it's explained better.


this should match the mesos master url for the vagrant instance?


The name of this variable needs to reference the fact it's for shorthands, right now it's not what it says.


i like the new table name