Hi @emichael - how does this new licensing portion for my README sound?

```
This work is distributed under the MIT license (`LICENSE-bamos.mit`)
with portions copyright Ellis Michael (GitHub: emichael).
Ellis' portions are also distributed under the MIT license
(`LICENSE-emichael.mit`) and include:
+ TODO
```


the slide anim can be removed.


This shouldn't work with OS X version of find, where argument is not optional.

A couple more points:
- I do mind `./` prefix. Appending `cut -b3-` seems like a valid solution.
- You forgot to update fish version

My suggestion:
- `.` instead of the current `*`
- `cut -b3-`


Maintaing this fork does take some effort, but thats because we are making changes to llvm code, not adding new code, so llvm changes will break our code.


Hello @christophwille ,
did you dowloaded the dll of System.Net.Mail or import it to sharpDevelop ?
We are working on university project and we need to send mail throw smtp protocol.

Any help ? thanks


Or the readme.


Ok , like using container in configs to set anything ?


Yup, will do.


oops, looks like i accidentally undid my previous readme edit


What do you mean? `file-name-directory` does indeed return nil.

```lisp
(file-name-firectory "foo") ; => nil
```


hdmi_group is in the config twice, see line 87


Yea something like that will work.


@Shani-08  filmon high stoped after  a couple  min, please try mbc 1


The steps I took to provide a little more security to `items_controller`. I'm not sure how to get the create method to be more secure? Line 13 is where I thought this could happen but am not sure how I would get the current user's list id I tried `current_user.list.id` but kept coming up against an `noMethodError` and figured I'd not taken steps in the list controller.


Because it does different things, see the comments.


The patch worked on OS X using the preinstalled ruby 2.0 and python 2.7.1 and a fontforge build based on sources from 10th January 2015.

The patch is the result of me trying to get rid of the python errors which showed up when executing your fontcustom fork. I have to admit I have little to no understanding of font definitions or lookup tables. So the patch itself is likely to be buggy.


I'm talking about for when players are actually giving us code. There are a lot of things to consider with allowing arbitrary execution of code.


@littleguy77 when frameskipping is used for low powered devices, frames that aren't drawn will idle on SDL_Delay() however frames that are drawn may take too long to process and therefore emulation is slower than real-time.

My original implementation only used a 6 point moving average filter to moderate VI timing. I figured people would only do 1:3, 1:2 or 2:3 frame skipping. I'm not in a position to test the current implementation with a smaller array but it might be worth a try.

I have noticed emulation seems to be a little fast when it first starts, as if the array for the moving average filter was not initialized correctly but from a quick scan through the code I haven't determined it this is the case given it works on differentials rather than absolute times.


Should `TrackingConstraint` be deleted now that this exist?


Could be done but now it gets overwritten in `mockApplication($config = null)`. So doing this does not work in test env. But as we said before, tests should be isolated.


That was one hell of a typo :-)


To be honest, I don't know either. I've done this using local reasoning only.
Note, though, that this just leaves things as they were, but in a type-checked way.
I mean, if a user hits an assert because he entered a too big number, it means that before, a truncating cast was being done, completely silently, and execution was allowed to continue that way.
Now, we will get a new tracker issue that will point us exactly to the place where error-control must be added.
I think that's better than we had. I mean, not as a definitive state for the code, but as a step towards a better thing.


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

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


We were using 4 spaces to match the standard in our php files.


Looks good, incorporated in latest GM file.


Should this comment be updated to refer to `OrderingConstraint`?


I need to push an update for our test server so guns work again. Want me to try my fix out? The worst case for it is to delay almost one round of ticks before firing. Right now you have to wait up to an hour and a half when it fails.


Ah, your right. So please drop that commit. (It might however be a good idea to replace `cl-loop` + `mapcar` with just `cl-mapcan`. And you might even consider using `while` and avoiding duplicates upfront instead of relying on `cl-remove-duplicates`. But it's only cosmetic, performance isn't really gonna matter here.)


`new Random()` would probably suffice, as this is the same as calling `new Random(Environment.TickCount)` (see [here](http://referencesource.microsoft.com/#mscorlib/system/random.cs,5d22f8880fc9f8d9))


> Now, we will get a new tracker issue that will point us exactly to the place where error-control must be added.
I think that's better than we had. I mean, not as a definitive state for the code, but as a step towards a better thing.

Sure, that seems like something we could do.


I'll fix it using your method


:-) glad you like it


Merge with master


Good catch. BTW, I wasn't sure if the commented line is really needed. I didn't notice any issues without it.


This fixes the issue #8875


I'm using this build version of fontforge

```bash
fontforge --version
Copyright (c) 2000-2012 by George Williams.
 Executable based on sources from 14:57 GMT 31-Jul-2012-D.
 Library based on sources from 14:57 GMT 31-Jul-2012.
fontforge 20120731
libfontforge 20120731
```
Could be this the difference?


Commit message uses a lowercase starting letter after the prefix, please use a versal (example: "[TASK] This subject is valid")


Instead of the `mapcar` you should call `magit-decode-git-paths` in the call to `file-name-directory`.


Tested on my system (Linux) and it's working there... I'll have to set up a Windows environment and build system (hopefully the free version of Studio will build it) and test there...


But in the past zoneminder could run without installing the sys v init files - you could start it via "zmpkg.pl start"

With the current zoneminder systemd intergration, if you don't install the zoneminder.service file, zoneminder can't start. In addition, there's nothing about this requirement in the INSTALL file.

It really should be installed even for source builds, or at least the INSTALL documentation should be updated.

Similar for the /etc/tmpfiles.d/zoneminder.conf - also, that file should be installed to /usr/lib/tmpfiles.d, at least on Fedora 21, per man tmpfiles.d and http://fedoraproject.org/wiki/Packaging:Tmpfiles.d

PS: I'm running zoneminder 1.28 on Fedora 21, via the tar file download. Keep up the good work :-)


I think you could do this in the .pot check. When you create the new input file from the pot file, do it like this:
`hash:plain` -> `plain:hash`. Then while cracking that one, all found should show as "password (password)" where the word in parens is the login and should always be the same.


This is looking good: added authentication in a `before_filter` and the `new` action scopes the db query on the `current_user`, nice one. That's what was needed.

For the `create` action, fetch `@list` from the db in the same way you've correctly doing it in the `new` action:

```ruby
  # Change line 11 to:
  @list = current_user.lists.find(params[:list_id])
```


@johnotander The todo in the readme has to be ticked too.


This seems to be the cause of:

https://github.com/benoitc/couchdbkit/issues/147


Also on line 15, if there's only one model object passed to a `redirect_to`, there's no need to put it in an array.


I feel like the lighting on the cores is coming from the wrong direction. I am not sure what direction the light comes from in JS but right now it is looking like the sun is coming from the bottom right. I usually do light from the top left. Can @Greenhawk837 help out here?


Debe funcionar también con IEnumerable


Both ToString() invocations would appear to be unnecessary (Core.StringExtensions).


This should be removed entirely at some point, or at least locked down tight.


both the left and right tags can be removed.
it's an incorrect c/p from an other control (which i just fixed in https://github.com/xbmc/xbmc/commit/138060d335c2b09db49e6e37d997ef3fe3faf5bd)


HVACSizingSim_CoincidentPlant (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Coverage Too Low

![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-2.32%25-red.svg)


Single digit hour is prepended with a space with the %l - changed to %I which is zero padded, and then dropped the leading zero.


This was deliberate
if row[xx] is None then it is "-" which is True


Non so se può servire, ma bisogna scaricare questo file se si vuole utilizzare visual studio https://visualstudiogallery.msdn.microsoft.com/20b80b8c-659b-45ef-96c1-437828fe7cf2


Build and Unit Testing
----------------------
Use the Maven command after extracting the file to a specified folder.
mvn clean install
Note : For JUnit test, the application uses default number range from 1 to 100.

Setup
-----
Modify the config.properties file under ..\cfg to set the lower and higher number for the game
Note : The Application uses default number range from 1 to 20.

Run
---
Rename play.txt to play.bat
Then type "play" and press enter key


For backwards compatibility, this might be best to make readable/writable=False until someone actually wants to make use of this.
Also I don't see any point in having this flag unless it actually works!
The PR should be accompanied by a task which goes through & expires posts based on their expiry date...wouldn't be enabled by default...but can be enabled if sites wish to make use of this feature...


I would just make the field name 'expiry_date' not expiry_datetime' too & probably make the default widget a date one (but having this be a datetime field means that it would be possible to use a datetime widget for those sites that really want this granular a level of control.)


Maybe comment the widget out so that it's just a suggestion since by default the field is disabled anyway


I don't think we want a menu for evr in the default s3menus.py since this is a very template-specific module


Making this change may need more consideration...are there any live sites to migrate? (None that I'm aware of actually)
Have you tested that the timeline feature works properly with this?


Commit de remoção de package e servlet de teste. Criação de servlet de upload está em commit posterior!


This makes the feed validator pass with zero warnings.


CHANGELOG.md hasn't been updated.


This could very well be the reason. Here is my fontforge version info:

```shell
fontforge --version
# Copyright (c) 2000-2014 by George Williams.
# Based on sources from 10-Jan-2015-ML-D.
# Based on source from git with hash:f6ffb25d7732adcc8d32dae75bd90bda90250304
# fontforge 10-Jan-2015
# libfontforge 20150110
```

That’s quite an age difference between our fontforge builds.


Testando a passagem de dados do servlet para a interface.


I don't think checking the property is necessary.

03e7297300846811cfef01f8294b7b7b8b623a93 sets the default value if NULL


Setando jsp para servlet de upload.


General rule: "If it's this easy to action a ToDo, why was it left as a ToDo rather than just actioning?"


Should you be closing this connection, or do you expect the client to close the connection? It was the client who opened the connection in the first place, right ?

In case of the ChromeApp, I am getting an exception when trying to read. I am able to read the entire thing, but the connection closes after that and I get an error.


Ya I'm working making the project in that tutorial, and this core data thing is awesome!! It's pretty simple too, well so far


Hi @r-type, does the MAME libretro core work again or are there still 'untracked memory' errors occurring?

I had to change some stuff after the revamp to the OSD system and I suspect I might have done a few things wrong. Would love to hear what the current state of things are.


Good catch @arthurvr :), will do.


Ok, issue fixed see,
FB: https://github.com/FusionSP/android_frameworks_base/commit/cf8ea9858c273cab9971d8ce0983d3ab74f90a41
Settings: https://github.com/FusionSP/android_packages_apps_Settings/commit/eb821e044c0102809e989a0739197da2ad3cc4ea

Cheers
Dragon


Pourquoi tu ne travailles pas dans un package hexGame ?


@tkon99  als in hoe vaak een cijfer 'mee telt'. Een proefwerk heeft vaak een valentie van 4, en een schriftelijke overhoring bijvoorbeeld 2.


Is used to Know that a from addon itself a Dialog is opened. I start tomorrow evening to do a cleanup on all places. Thanks for the notes.


this codes make all breadcrumbs to only one url to HOME
Home > 1 > 2 > 3 > LAST

GOOGLE
    Breadcrumb 1
    url:    http://www.mabnait.ir/
    title:    HOME


Just as a notice: Currently, if available, elevation data in the route table is used.


Def Project version


@skeptical it doesn't seem to have caused problems in past releases, but I've done your suggestion now :)


ATTN guys - use _WIN32 from now on for Windows.

If you need to specifically target Mingw, there is a way to do that too.

#if defined(_WIN32) && defined(__MINGW32__)


Paging someone smarter.  Can this be done a better way?

You will waste a bunch of time tracing this code.  I wrote it and I wasted a bunch of time trying to trace it.


MkdirAll checks for the existence of the directory first, and is a no-op if that directory already exists.


hah, that was part of the fix. The other one was line 495.


Just merging the files from server.


Nuuuu


nie zamieniaj tego na HeightScale. to specjalnie zostalo zmienione.


![screen shot 2015-01-24 at 13 40 40](https://cloud.githubusercontent.com/assets/1155163/5888497/9dfe7ef0-a3ce-11e4-81e8-6e62a38bf4cd.png)


![screen shot 2015-01-24 at 13 40 44](https://cloud.githubusercontent.com/assets/1155163/5888498/a2d0154c-a3ce-11e4-9cd5-e2f2142be2c2.png)


Fascinating, I learned quite a lot from that entire article.  Thanks for the link.  Anyway, yeah, `std::move()` is indeed necessary there, and I now know a lot more about rvalue references.


Why change this into datetime?


Good idea. Seems to be more generic. I'll change it.


Usun od razu angle, bo jest nieuzywany


Good jop !


It's much better to use the `sh` module than `os.system`.


```python
from sh import psql
psql('-c', 'drop database ooiuidev')
```


ok


Editado


Sound seems to heavy (almost 1mb) and too long to play at instant. If you'd manage to cut it and leave only one frame of it (to repeat it later) it can stay in changed form, otherwise i recommend using different sound file.


Hi,

Not at all.  I was planning on doing a pull request for you this week end anyway.

Thanks,
Dennis

> On Jan 23, 2015, at 1:16 PM, dak180 <notifications@github.com> wrote:
>
> @dbiringer <https://github.com/dbiringer> these commits look interesting, mind if I fold them into the main repo?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/dbiringer/autorevision/commit/2b3407b216e8d5f3c254191113dc363b6638a839#commitcomment-9425122>.
>


Could we revert this? This patch costs build time and it isn't even fixing the problem, only hiding it.


@SubJunk "seem to" is the operative phrase, I think :-).  Excellent that you've taken the trouble to highlight all the java6 fixes here btw :+1:


I was recently learning File I/O and looking for one done in C. I still not ok with that at this time :D


@Optimusprimeums @skeptical I'm going to delete the 5.0.0 tags soon and redo them when the issues have been fixed. I'll be working on them today.


:) Yeah I guess - should be slightly faster too


Thanks for reporting the missing file. Committed and pushed now.

I don't follow you with the first point. The continuation in test_10964.cpp returns already a future<void>. Could clarify?

I will check the second point asap.


@dmyersturnbull As of this patch, currently end gaps are not getting penalized. (at least at the beginning, there still seems to be an issue in the end). I see your point, but as you can see from this discussions there are really two different approaches for scoring end gaps. We won't be able to fully resolve this for this release, but we should add configuration for different approaches for scoring the end gaps for the next release.

@josemduarte Could the bug with the end-gap be related to the traceback?


I would think that this issue should be reported to the WebRTC folks directly. I don't believe it's a problem with the build but with the underlying code.

--sent from the iPhone

> On Jan 24, 2015, at 12:14 PM, pu <notifications@github.com> wrote:
>
> I tested individually srtp srtp_init 10000times is ok。I don’t knowwant iswrong。
>
>
> 原始邮件
> 发件人:puyidepuyide@gmail.com
> 收件人:pristineio/webrtc-build-scriptswebrtc-build-scripts@noreply.github.com; pristineio/webrtc-build-scriptsreply@reply.github.com
> 发送时间:2015年1月24日(周六) 20:57
> 主题:Re: [webrtc-build-scripts] New libWebRTC_objc target gets injectedinto the libjingle_examples.gyp file after doing a gclient sync after callingan update2Revision. Don't ask for code_signing unless you are buildingAppRTCDemo (ipa needs to be code signed) (8d863e4)
>
>
> Imade in r8136. r8135Alwayscrash inSRTP
> Error(srtpfilter.cc:706): Failed to init SRTP, err=2
> Warning(channel.cc:852): DTLS-SRTP key installation failed
> Error(common.cc:59): ../../webrtc/base/thread.cc(347): ASSERT FAILED: !current || current-blocking_calls_allowed_ @ AssertBlockingIsAllowedOnCurrentThread
>
>
> my environmentXcode 6.1 sdk 8.1
> run on iphone6
>
> 原始邮件
> 发件人:Ryan C. Paynenotifications@github.com
> 收件人:pristineio/webrtc-build-scriptswebrtc-build-scripts@noreply.github.com
> 抄送:pupuyide@gmail.com
> 发送时间:2015年1月24日(周六) 02:38
> 主题:Re: [webrtc-build-scripts] New libWebRTC_objc target gets injectedinto the libjingle_examples.gyp file after doing a gclient sync after callingan update2Revision. Don't ask for code_signing unless you are buildingAppRTCDemo (ipa needs to be code signed) (8d863e4)
>
>
> The crash I am seeing is related to the changes they made in r8136. r8135 works...
> From what I can tell this approach is working fine for building just the library.
> —
> Reply to this email directly or view it on GitHub.
> —
> Reply to this email directly or view it on GitHub.
>


Cygwinでビルドしたので


Nice!!


@trmfreitas Would you mind if I didn't expose this property? Does it need to be exposed?


olala


This code gives the output I want, but I feel like there's got to be a way to get this job done without splitting the multi-line string. But everything I tried ended up printing my image tags before the HTML code, and then just printing the array of file names (without img tags) in the HTML code. Like this: http://cl.ly/image/1H3b0M2U292f


these changes are not needed :-)


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

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


HVACSizingSim_CoincidentPlant (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Looks good!  thank you


It should be yellow canola oil


I think it's safe to assume that, for a single plot, the theme remains constants across panels. It would make things a bit simpler if one xanchor/xangle for each plot (as opposed to each axis/panel)


this line should be yellow canola


while I appreciate the reach out, every line of code change or the class="img-responsive" isn't the problem. it's the footer images, or icons. pull up your favorite browser, and have a look. landing page footer...adjust browser to reflect..be well


I mean - I can read the ToDo, but I don't understand it. Do you?


Created a merge conflict to test nano resolution


Nice
Glad to see you getting your git on


Thanks for adding these in too. We usually have to ask for them after-the-fact :)


Why are you changing the distance sign ?


I believe you meant for this to be ```&CreatePrivateHostedZoneResponse```


Thanks!


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

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


The [`package.json`'s description](https://github.com/npm/npm/blob/9d61e96fb1f48687a85c211e4e0cd44c7f95a38e/package.json#L4) still reads "https://www.npmjs.com/package/npm still reads "A package manager for node".


That could be miss leading because surely you need the .js files that contain the tests as well. so shouldn't that read.
 files: [
'test/fixture*.js',
'test/fixture/*.json',
'*.html'
}
I have yet to test this to see if it solve the problem I am having.


Just testing out the commit function


j


Connor's way seems fine to me, but we probably should keep the bitfields consistent as all `unsigned` anyway.


I remember either you or someone else already asked me to add tests to other pull requests few times)


@devonCream True. Is that better? https://github.com/EE/karma-ng-json2js-preprocessor/commit/29e18a7cf9202c08c3035005023e8bcafafd80ee


@andreasprlic Isn't penalizing end gaps the definition of global alignment? If end gaps aren't penalized, the DP finds only the highest local similarity.


Includes fix for #191


Update.  You are right, it is hard/impossible to remove `Indx` from the RSP recompiler easily.

I did a quick fix attempt at removing Indx from the recompiler core, but my shuffle method doesn't seem to blend in with this self-modifying code trickery this recompiler seems to be full of.  I do not have the knowledge nor understanding of the RSP recompiler to remove this hacky array, but I still think it can be done by vectorizing the shuffling with a little MMX/SSE2 for someone who knows how to use Jabo's functions more closely.  I think I'll nag RPGMaster about that sometime.

In the meantime, I don't really see why it matters.  I think it's more important that the interpreter removes it for accuracy and readability than it is that the recompiler removes it for speed.


At least tell me what you want it to do so I can fix it?


@dmyersturnbull Please see my earlier comment on this, further up in the thread...


? there is no node_modules in root


You're right, that's a better fix. This is pretty annoying, since it's actually a valid deprecation notice, but anyone using EJS under the current version of Express will want these turned completely off.


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

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


Careful with iphone4 : maybe not all buttons fit together in screen


Fixed checkbox logic according the changed label text.


error


@arkadyp :+1: I second that !


Yes ofc, blink work for that case without problem :)
This patch is only to ensure the z of liquid level is not too low from the ground one, if it is, then we are probably in edge case.


I guess we just add a class to the button and display none in responsive.styl


ols wur hace7


There must be a base class too.


We don't really need to name this go import, because of the package name for this helper. 'error' is just a directory name, so we're safe.


In fact, the scenario you're describing is apparently what was happening in Issue #36. I wasn't following the commentary closely, but now I see this:

@dgofman-equinix wrote:
> I am not using renderFile (only ejs.render) and still getting the same waning using v2.2.2

I this case, it might have been appropriate to say, "sorry, you're using the new API."


Ok. Review commit added.


@bbroerman30 thanks for taking a look. To just test, you can simply get a master snapshot http://windows.php.net/downloads/snaps/master/ . Some latest after your commit should be sufficient. For compiling, yeah - the express edition will do the job, and you can use the 5.6 dependencies from http://windows.php.net/downloads/php-sdk/ .

Thanks.


There must be the plugin name.


@patmans Exactly. In the past and in a sys v init environment.  Systemd requires a service file. This is just one example where systemd is not flexible like the old sys v init.  Systemd only looks similar on the surface.  Under the hood it is a completely different animal and there is no way to accomplish what you are asking w/o breaking other things.

Zoneminder handles the systemd service file exactly the same way it has always handled the old sys v init script.  This functionality has not changed.  When building from source, it is up to you to choose the correct startup script, verify it is correct for your system, copy it from the source tree, and into the correct place.  It is not possible to write a script that gets this right for every single distro out there.

Now if you install from a package then you can expect the package maintainer to have done this work for you.  The correct script should get auto-installed.  If it isn't then it should be reported as a bug to the package maintainer.


This is already fixed on the master branch.


Crap, the more devices that are added to the network the more out of sync the show will get. This is crap if there's more than 1 or 2 devices in the network.


oops! right? @mereskin


Pls, remove changes on the dist folder from the commit.


@gpn39f hey, did you experienced any performance issues with smartai+conditions? Asking, cuz you r the second person (mb same) from arkania whos using cpp scripts vs smartai.


You could conditionally copy the service file depending on whether or not systemd is in installed.

But in any case the INSTALL file should be updated.


Supah fast! Thanks.


@andreasprlic I re-read your comment.

I should have clarified in my above comment that I can see the argument for not penalizing end gaps at all.

However, the commit only references the open penalty. If no penalty is applied but the extension is, I find that counterintuitive.


@dmyersturnbull ok now I understand. I agree.


The last version was 0.2.0 but apparently I never committed it?


Nice.


After thinking things over, I ended up leaving this in. I actually think it's less prone to edge case errors and ultimately more understandable (despite being a very long regex) than doing scattered validation on substrings throughout the parsing code. However, if you'd like me to remove it and implement non-regex validation, I can certainly do so.


and that's why I remove it from this file.


that would work.


Hi, I've fixed the same issue on #236 but instead of using reflection I just cast the object and do `setId()`. I'm probably overlooking some use case, can you explain me which is the advantage of using reflection to set the field?


Hello


I moved the isKeyword logic to the KeywordOption and now CharacterHelper is only slightly changed from its state prior to these changes.


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


That explains a lot of things o.o


No, we absolutely cannot! That would require us to make assumptions that won't be true in every case.
Take Ubuntu 14.10 for example.  It has *both* sys v init and systemd installed on it at the same time.  As soon as we make the assumption that we should go with sys v init then that would break any install that chooses to go with systemd.

Go look at other open source softwares and you will see that this task has always been handled by the package maintainer.

I will update the INSTALL file to make it more clear what you need to do (step 10 I presume)


Concatenation is fine for now.


OK. This does have a little bit of overhead but I don't think there is a case where we don't want to create these.


This shouldn't be `dependencies`. `dev-dependencies` is more like t.


*Never* change the directory just because it's different on your local machine.


The standard convention isn't `.on` and `.once`, it's `:on` and `:once`. It's weird. I know.


If you are going to be using single ticks, you might as well wrap the string in double ticks, so you won't have to escape them.


What does "error protection" mean?


##This is a markdown file


Indentation.


Consistency. Below, you have `to.be.ok()`, while here you have `to.ok()`


Penis


Spelling.


why this test was removed ?


You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'IN(5188,5049,5191)' at line 1 */


Yeah, that was my fuck up. Fixed in 36b7486d0ca80f406c2b319d79422ea53580da41


You misunderstand, the question is not if this commit changed a warning to an error but rather if the files that where generated with this warning where valid. If they are not valid, they may fail to play with some players, which would be bad, the intend of the error is to prevent the user from generating such invalid files. Do you have any evidence that the files where valid before ? did you check the mkv spec or test o a wide range of players ?


Looks like sublime's indentation has been weird for me. Gotta bring it down to tabsize: 1.


There's no need for this test, really. You can just run a multiple `on`/`emit` test, because that's the purpose behind the UID.


Yeah, fixed in 36b7486d0ca80f406c2b319d79422ea53580da41


Noted!


Yeah. I realise that now. Won't happen again.


Weird, but, noted!


I thought I was pushing and pulling to my own fork.
Sorry about the confusion.


My setup is so it *always* (!!!) uses spaces for tabs, and that the tab size is 4.


You already have distro specific cmake files - so only install it for Fedora.


Let's say you have a function that doesn't check the argument types and then tries to call a string. That would throw an error. In the program eventing.js there is protection against something like that happening and therefore the 'Error protection' tests are designed to ensure those protections function correctly.


The double ticks are ugly and it can make code look inconsistent however, I do agree with you.


The time remaining is shown below the progress bar:

![time remaining](https://cloud.githubusercontent.com/assets/1324685/5888683/0e7d7d30-a40a-11e4-9474-d99244fe6937.png)

It's not possible to translate this parametrized string (and a lot of others already in the source, too) without #175.


More indentation.


Exatamente. Até se voce for ver, é melhor o formato explícito ```array << item``` do que ser críptico demais.


Generates a random key using uppercase/lowercase letters, numbers, and symbols (!@#$%^&*()) at your bidding.

(Version 1.3, Python 3.4)


Why not do it manually?


Alright.


Better yet:

```
is(typeof((ref B b) { ++b; })) &&
```

i.e. don't assume copyability


what do you mean by manually?


Then wrap *all* the strings in double ticks for consistency's sake.


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1966](http://teamcity.monogame.net/viewLog.html?buildId=5071&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 366, ignored: 6 Build time: 00:15:44


This seems like a lot more work than https://github.com/genome/genome/blob/master/lib/perl/Genome/VariantReporting/Command/CreateReport.pm#L74 for getting a process to run.  Are we doing this because we wanted to preserve the behavior of the command (i.e. running to completion before returning vs. bsubing and telling you the process id)?


Using the opposite of `Q_WCharToUtf8`.


I see. Was just trying everything I could think of. Even if it was obvious that it would work.


This is awesome - thanks!


Changed.


put these files in a lib folder


Hello!


I updated these finder functions and added some refactoring as far as the storedCds and storedArtists variables in the data service layer - I hope the updates make sense to you that I pushed up.


But I don's see anything like `Q_Utf8ToWChar' in qufsion codebase


    return blockRecord.getStart() <= variant.getStart()
        && blockRecord.getEnd() >= variant.getStart() + 1;


TeamCity vs :: Monitor pull requests on ed [Build 6](http://localhost:8111/viewLog.html?buildId=22&buildTypeId=Vs_MonitorPullRequests) is now running


TeamCity vs :: Monitor pull requests on ed [Build 6](http://localhost:8111/viewLog.html?buildId=22&buildTypeId=Vs_MonitorPullRequests) outcome was **SUCCESS**
Summary: Running Build time: 00:00:01


The way this was formatted before was better (breaking before ?, and indenting on ? and : )


This patch makes no sense. It violates white/black symmetry:

    sf = ScaleFactor(sf * (eg_value(score) > 0 ? EndgameScaleAhead : EndgameScaleBehind) / 128);

score is from white's pov, so your patch is based on treating "white is ahead" differently from "black is ahead".

and even if you try to "fix" this by using the root color instead of white, it's still gonna wrong, because we skip evaluate() after null (speedup trick).


Awesome!  I'm excited to see your progress:)
On Jan 23, 2015 6:01 PM, "Joel Christensen" <notifications@github.com>
wrote:

> We need to make sure that the script tags that this function is echoing
> aren't gonna mess with the browsers creation of the html page from the
> $html variable.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/torgrimsonja/mathStats/commit/a70098213e37c5247679654d14d0193f780d1d87#commitcomment-9426056>
> .
>


Preconditions.checkState(0 < options.getBinSize(), "binSize must be greater than zero");


No need to rewrite every historical state information here. It each historical state will write its own data when exiting. So here you just need back trace parent states if any parent state is defined as deep history type and then just write current state status. e.g.

```java
if (getParentState() != null) {
            // update historical state
            ImmutableState<T, S, E, C> parent = getParentState();
            boolean shouldUpdateHistoricalState = parent.getHistoryType()!=HistoryType.NONE;
            if(!shouldUpdateHistoricalState) {
                ImmutableState<T, S, E, C> iter = parent.getParentState();
                while (iter != null) {
                    if(iter.getHistoryType()==HistoryType.DEEP) {
                        shouldUpdateHistoricalState = true;
                        break;
                    }
                    iter = iter.getParentState();
                }
            }
            if(shouldUpdateHistoricalState) {
                stateContext.getStateMachineData().write().lastActiveChildStateFor(getParentState().getStateId(), getStateId());
            }

            if(getParentState().isRegion()) {
                S grandParentId = getParentState().getParentState().getStateId();
                stateContext.getStateMachineData().write().removeSubState(grandParentId, getStateId());
            }
        }
        logger.debug("State \""+getStateId()+"\" exit.");
```


@enartey that's the one awesome thing with pull requests, they allow you to have discussions on parts of the code.


Radio Janjot
Android App


Thanks, @oazmon. Would you mind pushing that change to the PR? One less remote call during our unit tests.

Otherwise LGTM.


So this test should fail if you comment out the lines from `yourCode.js` that declare a `makeStack` function.


url added to link
-mm


The original behaviour, which this purports to maintain, was to (explicitly or not) set the `_value` to NaN internally for amounts which overflowed.

It would be nice to see a commit (not buried in a rebase) which introduced tests, clearly defining all this weird behaviour. It would be useful in the next phase you've been alluding to, and would show due care has been taken.

Pull requests can be composed of [many commits] (http://jakegoulding.com/blog/2012/11/04/refactor-and-make-changes-in-different-commits/). With regards to making review easier, can I point out that at one point I had to deal with a bunch of changes mixed in with stuff I'd already reviewed, all squashed down into one commit. There's a lot of changes here so that's something to look out for as well.


Maybe we should use question marks instead.


Use De Morgan's Laws + Iterables.all to simplify

    private static final Predicate<String> LENGTH_IS_1 = Predicates.compose(
        Predicates.equalTo(1),
        new Function<String, Integer>() {
          @Override public Integer apply(String string) {
            return string.length();
          }
        });
    ...
    return isVariant(variant)
        && LENGTH_IS_1.apply(variant.getReferenceBases())
        && Iterables.all(variant.getAlternateBases(), LENGTH_IS_1);


Yes, I should do that. Thanks for the heads up


whwyghew


bgeqw


@chrishaid take a look at [student_scaffold](https://github.com/almartin82/mapvizieR/blob/c9d8bb7ed1ec9313fef2da6dd6e350be549ab8f6/R/growth_df_prep.R#L65)
not all the way there, but it is an abstracted method for figuring out which students have matching data given any generic start_season and end_season.  the tests should give you a sense of what it is doing:

f2s_scaffold <- student_scaffold(
  processed_cdf = processed_cdf
 ,start_season = 'Fall'
 ,end_season = 'Spring'
 ,year_offset = 0
)

expect_equal(nrow(f2s_scaffold), 4179)

f2s_match_counts <- table(f2s_scaffold$match_status)
expect_equal(f2s_match_counts[['only end']], 1993)
expect_equal(f2s_match_counts[['start and end']], 2186)

once we have a scaffold, it's just simply a matter of finding the matching data in the long cdf, and adding it to the growth term.


    List<String> alternateBases = variant.getAlternateBases();
    return !(null == alternateBases() || alternateBases().isEmpty());


help


Not sure if you want aar or jar here. Aar will allow you to add
resources and such later.


Break after the open-curly:
    static final Call[] variantCalls = new Call[] {
        DataUtils.makeCall("het-alt sample", 1, 0),
        DataUtils.makeCall("hom-alt sample", 1, 1)
    };


first commit


Won't this fail if the console is enabled?


Yes will u fix or u want me to do it?  :-)


Go ahead and statically import all these asserThats


Ah, now I see why this guy isn't an anonymous inner class


i'd suggest inlining all those intermediate PCollections to make the pipeline a single statement.


Why are you wrapping the call to Arrays.asList with newArrayList? Seems redundant.


All of those setters return this, so you can method-chain them together into a single expression if it pleases you.


Ok Nicole, hear you go....

Let me know if you want me to do another pass.


Replacing `List.fold (fun acc str -> acc+(if acc="" then "" else ",")+str) "" ` with `String.concat ","` will make this more readable and, last but not the least, much more efficient by avoiding O(N^2) string reallocation. I understand that this is an unimportant function, but it is still part of an example.

Ditto at line 28 (if the leading comma is required, which I think is a tiny bug, you can add it to `sprintf` below.


I do not think we should check in generated files. The other test project does not. They should still be referred from the `.fsproj` of course.


Isn't this same as `['0'-'9']+` ?
Does JSON spec allow for the leading `+` in numbers?


While I should clearly use the function you made aka getCurrentModeExceptConsole(), this is fine already as you can click on the map when in the console.
I'll change that anyway. Thanks for pointing out. :)


@favonia, does it clear the concerns that you had in the function with exceptions? I couldn't really understand what you explained after the class, it would be great if you can do it one more time somewhere (maybe in a new issue?).


I think rather than subclass VKInputAction, it might be better to make a new base class that defines an interface to receive key up and down events and register objects derived from that or one of it's subclasses with the input subsystem instead of the callbacks. That keeps the details of the input backend separate from the input receivers, but still gives us a nice event based API. I might rename the InputAction class to InputBackend while I'm at it, since InputAction sounds more like a receiver.

With that model we can also implement a generic callback subclass for use with global keys that don't have their own associated object, then hide that details inside the existing RegisterIniKeyBinding function.


The original canonicalize function never sets anything to NaN. It doesn't even check for overflow of native amounts. I moved the native overflow check from to_text to canonicalize because it belongs there, but you and Tom were right that it is a behavior change so I moved it back.

I'm applying the same concept to pull requests as the post author applies to commits. The same reasoning stands. Sorry about squashing prematurely earlier, I thought it was ready to be merged. I'll be more cautious with that.

> On Jan 23, 2015, at 6:13 PM, sublimator <notifications@github.com> wrote:
>
> The original behaviour, which this purports to maintain, was to (explicitly or not) set the _value to NaN internally for amounts which overflowed.
>
> It would be nice to see a commit (not buried in a rebase) which introduced tests, clearly defining all this weird behaviour. It would be useful in the next phase you've been alluding to, and would show due care has been taken.
>
> Pull requests can be composed of many commits. With regards to making review easier, can I point out that at one point I had to deal with a bunch of changes mixed in with stuff I'd already reviewed, all squashed down into one commit. There's a lot of changes here so that's something to look out for as well.
>
> \
> Reply to this email directly or view it on GitHub.
>


I certainly didn't want documentation types "inside" the type. I don't mind having Intro "some stuff" :> API, though.

Up to you. I'd be happy to make a docsForWith if you'd prefer that.

markdown seems like the wrong layer to add functionality, if we want to remain modular in terms of output formats.


Using just `int(lexeme lexbuf)` in this line would be consistent with the next line's use of `float()` -- unless there is a good reason for this usage that I am missing.


thos padding additions causing errors

frameworks/base/packages/SystemUI/res/values/cm_dimens.xml:22: error: Error parsing XML: junk after document element

frameworks/base/packages/SystemUI/res/layout/location.xml:45: error: Error: No resource found that matches the given name (at 'paddingTop' with value '@dimen/detail_radio_group_padding').

frameworks/base/packages/SystemUI/res/layout/location.xml:45: error: Error: No resource found that matches the given name (at 'paddingLeft' with value '@dimen/detail_radio_group_padding_left').

frameworks/base/packages/SystemUI/res/layout/location.xml:45: error: Error: No resource found that matches the given name (at 'paddingRight' with value '@dimen/detail_radio_group_padding').

frameworks/base/packages/SystemUI/res/layout/location.xml:45: error: Error: No resource found that matches the given name (at 'paddingBottom' with value '@dimen/detail_radio_group_padding').

frameworks/base/packages/SystemUI/res/layout/location.xml:54: error: Error: No resource found that matches the given name (at 'paddingTop' with value '@dimen/detail_radio_group_padding').

frameworks/base/packages/SystemUI/res/layout/location.xml:54: error: Error: No resource found that matches the given name (at 'paddingLeft' with value '@dimen/detail_radio_group_padding_left').

frameworks/base/packages/SystemUI/res/layout/location.xml:54: error: Error: No resource found that matches the given name (at 'paddingRight' with value '@dimen/detail_radio_group_padding').

frameworks/base/packages/SystemUI/res/layout/location.xml:54: error: Error: No resource found that matches the given name (at 'paddingBottom' with value '@dimen/detail_radio_group_padding').

frameworks/base/packages/SystemUI/res/layout/location.xml:63: error: Error: No resource found that matches the given name (at 'paddingTop' with value '@dimen/detail_radio_group_padding').

frameworks/base/packages/SystemUI/res/layout/location.xml:63: error: Error: No resource found that matches the given name (at 'paddingLeft' with value '@dimen/detail_radio_group_padding_left').

frameworks/base/packages/SystemUI/res/layout/location.xml:63: error: Error: No resource found that matches the given name (at 'paddingRight' with value '@dimen/detail_radio_group_padding').

frameworks/base/packages/SystemUI/res/layout/location.xml:63: error: Error: No resource found that matches the given name (at 'paddingBottom' with value '@dimen/detail_radio_group_padding').

make: *** [/home/joshndroid/teamsource/out/target/common/obj/APPS/SystemUI_intermediates/src/R.stamp] Error 1
make: *** Waiting for unfinished jobs....


Missing space before '}', to be consistent.


I think the default message will be to this same effect, but also add line and character number, so this catch-all clause might not be necessary. I may be wrong though.


probably just an oversight


Nice!


Fixed.


Ahh, thanks, makes sense now.


Generated file.


To be consisted on spacing, I think this whole rule missing spaces before each `{`.


Typo: s/prase/parse/ (and 4 below).


This fixes #9


Why? KISS


LGTM

Thanks! Works on my set of projects. appengineRun reports as UP-TO-DATE on projects that are not executing appengineFunctionalTest and appengineStop is SKIPPED.


Get ripple-lib prior to this patch and construct a native amount with a
huge number and then check the _value. I never said where/how this
happened, just that it did. Did you check?


Are you sure the mkv files generated before this commit where correct ?
If so, its probably best to submit a bug report to trac or patch to the ffmpeg-devel mailing list. Few developers follow github.
If OTOH it did actually not produce correct files before then something has to be changed/added to the AVParser or a bitstream filter to format the AAC in a way that is compatible with mkv.
It would also be possible to allow the user to override this check by checking strict_std_compliance but if the resulting files arent valid that would be a rather half baked solution


LOL @ COMMENT


Fixes #122. The final line of the commit message was cut off because it began with a # symbol.


please don't include script tags in head! put them below bower components


What does this do?


Works for me on .local and remote server


One thing I find counterintuitive: if I comment out the whole body of this function, the latency is still there, no difference whatsoever.  Perhaps the old implementation had some undocumented side-effect that no longer exists in the new implementation?  That is the best hypothesis I have right now.


We need PROG to be overridable for things like `make bench PROG=sundown`.
But I'll merge your whole PR and revert this part.


nope!


Please don't add editor modelines to source files. Seeing that you are using kate, i would recommend to try kdevelop and set the tab-width as project specific settings.


Looking at the code again, I did miss something obvious (line 731 where the index changes).  It subtracts the value that was stored 64 frames ago, so the total delta doesn't grow forever.  Anyway, sorry for not reading more closely before commenting.

I do still question if CurrentFPSTime should be updated after the SDL_Delay, though.


-viz looks good from here, didn't break anything from here.


WTF! no globals. put this inside the app namespace!


nope! fix whitespace.


Some comment


That is already done in the Fedora specfile.  If you don't want to have to put on your developer hat on then you should not be building from source.  Use zmrepo to install the rpm package instead.


:shipit:


no replication for now. remove these comments.


r? give it a meaningful name


if this gobalTestString is temporary and only for testing, declare it here. don't pollute the global namespace


There aren't any repos for F21, and the RPM I'd been using was for F19, so I chose to use the source.

I thought of just trying to build via the specfile, but decided to build from source first.

I wouldn't be posting here if I hadn't put my developer hat on ...


Agreed. I've changed adjusted the PR (new commit).

btw: when testing bash I've seen that it does something odd on `C-t` sometimes, where is looks like this after selecting `test.c` - but that happens also without my changes.

    $ test.c _
    $ $(__fsel)


Please put opening brackets for all blocks other than functions on the same line:
```
if (...) {
```


No, I dont. As @flavour said small todos are not always for action shall I mark it a question mark.
@nursix : I really didnt investigated for a use case for this


Found BROKEN by full.
.py
11. sdp.py
12. implicitfunction.py
13. integration.py
14. simulator.py
15. typemaps_futurediv.py
16. matrix.py
17. ocp.py
18. mx.py
19. nlp.py
20. vectortools.py
21. function.py
22. lpsolver.py
23. ad.py
24. socpsolver.py
25. sx.py
26. mx_jacobian_sparsity.py
Ran 26 tests, 0 fails.
python internal/test_py.py python -skipfiles="alltests.py helpers.py complexity.py speed.py" -memcheck -passoptions="--ignore_memory_heavy --run_slow"
Running test in python
01. tools.py
02. qcqpsolver.py
03. linearsolver.py
04. control.py
05. misc.py
06. homotopynlp.py
07. qpsolver.py
08. typemaps.py
09. sparsity.py
10. symbolic.py
11. sdp.py
12. implicitfunction.py
13. integration.py
14. simulator.py
15. typemaps_futurediv.py
16. matrix.py
17. ocp.py
18. mx.py
19. nlp.py
20. vectortools.py
21. function.py
22. lpsolver.py
23. ad.py
24. socpsolver.py
25. sx.py
26. mx_jacobian_sparsity.py
Ran 26 tests, 0 fails.
python internal/test_py.py '../docs/api/examples'
Running test in ../docs/api/examples
01. ssym.py
02. jacSparsity.py
03. generateCode.py
04. SXFunction_constr_py.py
05. symbolicsubstitution.py
06. countNodes.py
07. tolerance.py
08. simulator.py
09. controlsimulator.py
10. monodromy.py
11. idas.py
12. jacobian.py
13. dulmageMendelsohn.py
14. coloring.py
In ../docs/api/examples/matrix, coloring.py failed:
==============================
returncode:  1
================================================================================
Original:
IMatrix(
[[1, 00, 00, 00, 00],
 [00, 1, 00, 00, 00],
 [00, 00, 1, 00, 00],
 [00, 00, 00, 1, 00],
 [00, 00, 00, 00, 1]])
Colored:
IMatrix([1, 1, 1, 1, 1])
================================================================================
Original:
IMatrix(
[[1, 1, 1, 1, 1, 1, 1, 1, 1, 1],
 [1, 1, 1, 1, 1, 1, 1, 1, 1, 1],
 [1, 1, 1, 1, 1, 1, 1, 1, 1, 1],
 [1, 1, 1, 1, 1, 1, 1, 1, 1, 1],
 [1, 1, 1, 1, 1, 1, 1, 1, 1, 1]])
Colored:
IMatrix(
[[1, 00, 00, 00, 00, 00, 00, 00, 00, 00],
 [00, 1, 00, 00, 00, 00, 00, 00, 00, 00],
 [00, 00, 1, 00, 00, 00, 00, 00, 00, 00],
 [00, 00, 00, 1, 00, 00, 00, 00, 00, 00],
 [00, 00, 00, 00, 1, 00, 00, 00, 00, 00],
 [00, 00, 00, 00, 00, 1, 00, 00, 00, 00],
 [00, 00, 00, 00, 00, 00, 1, 00, 00, 00],
 [00, 00, 00, 00, 00, 00, 00, 1, 00, 00],
 [00, 00, 00, 00, 00, 00, 00, 00, 1, 00],
 [00, 00, 00, 00, 00, 00, 00, 00, 00, 1]])
================================================================================
Original:
IMatrix(
[[1, 00, 00, 00, 1],
 [00, 1, 00, 00, 00],
 [00, 00, 1, 00, 00],
 [00, 00, 00, 1, 00],
 [00, 00, 00, 00, 1]])
Colored:
IMatrix(
[[1, 00],
 [1, 00],
 [1, 00],
 [1, 00],
 [00, 1]])
================================================================================
Original:
IMatrix(
[[1, 00, 00, 00, 00],
 [00, 1, 00, 00, 00],
 [00, 00, 1, 00, 00],
 [00, 00, 00, 1, 00],
 [1, 00, 00, 00, 1]])
Colored:
IMatrix(
[[1, 00],
 [1, 00],
 [1, 00],
 [1, 00],
 [00, 1]])
================================================================================
Original:
IMatrix(
[[1, 1, 1, 1, 1],
 [00, 1, 00, 00, 00],
 [00, 00, 1, 00, 00],
 [00, 00, 00, 1, 00],
 [00, 00, 00, 00, 1]])
Colored:
IMatrix(
[[1, 00, 00, 00, 00],
 [00, 1, 00, 00, 00],
 [00, 00, 1, 00, 00],
 [00, 00, 00, 1, 00],
 [00, 00, 00, 00, 1]])
================================================================================
Original:
IMatrix(
[[1, 00, 00, 00, 00],
 [1, 1, 00, 00, 00],
 [1, 00, 1, 00, 00],
 [1, 00, 00, 1, 00],
 [1, 00, 00, 00, 1]])
Colored:
IMatrix(
[[1, 00],
 [00, 1],
 [00, 1],
 [00, 1],
 [00, 1]])
================================================================================
Original:
IMatrix(
[[1, 00, 00, 00, 00],
 [00, 1, 00, 00, 00],
 [00, 00, 1, 00, 00],
 [00, 00, 00, 1, 00],
 [00, 00, 00, 00, 1]])
Star colored:
IMatrix([1, 1, 1, 1, 1])
================================================================================
Original:
IMatrix(
[[1, 1, 1, 1, 1],
 [1, 1, 1, 1, 1],
 [1, 1, 1, 1, 1],
 [1, 1, 1, 1, 1],
 [1, 1, 1, 1, 1]])
Star colored:
IMatrix(
[[1, 00, 00, 00, 00],
 [00, 1, 00, 00, 00],
 [00, 00, 1, 00, 00],
 [00, 00, 00, 1, 00],
 [00, 00, 00, 00, 1]])
================================================================================
Original:
IMatrix(
[[1, 1, 1, 1, 1],
 [1, 1, 00, 00, 00],
 [1, 00, 1, 00, 00],
 [1, 00, 00, 1, 00],
 [1, 00, 00, 00, 1]])
Star colored:
IMatrix(
[[1, 00],
 [00, 1],
 [00, 1],
 [00, 1],
 [00, 1]])

Traceback (most recent call last):
  File "coloring.py", line 76, in <module>
    A = sparse(A)
NameError: name 'sparse' is not defined

==============================
15. stronglyConnectedComponents.py
16. createParent.py
17. matrix_expand.py
18. veccat.py
19. expand.py
20. customfunction.py
21. parallelizer.py
22. limitPrinting.py
23. assertion.py
24. printme.py
25. dotdraw.py
26. callback.py
27. KinsolSolver.py
28. NewtonImplicitSolver.py
29. NLPImplicitSolver.py
30. exacthessian.py
31. ipopt.py
32. LinearSolver.py
Ran 32 tests, 1 fails.
make: *** [examples_indoc_py] Error 1

```


No need for brackets here


Yes, of course - pushing up fix - ty


I'm planning to update the chat example in the same way -- remove the handleViewAction() wrapper and call dispatch() directly.  Also a pull request to do that would be most welcome.


Not really mandatory, but it's nice to align the above statements


So I'm working on making some unit tests for the search plugins. I'll look into it in a bit. It does seem a bit odd, but I'm still only like 20% sure how it all works yet.


Don't mess up the whitespace!


Remove these comments


Why? remove this comment


This update for Laravel 5 breaks Cashier in 4.2 as the loadViewFrom method doesn't exist in 4.2. To rectify that you need to lock in to the commit before this one in composer.

        "laravel/cashier": "dev-master#a08d376a380a713c613d6bf09082e2c18a8a0031",


Great find! How about adding:
```java
Class<?> fieldType = field.getType();
if (fieldType.isAnnotationPresent(Table.class) || SugarRecord.class.isAssignableFrom(fieldType)) {
```
To also contemplate the case where the entity is extending SugarRecord instead of using `@Table`. You may want to take a look at d46ff5d44d3819296bfd0c67ed50def60f2973ad


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


Where's `type="text/javascript"`?


Unnecessary whitespace added. :P


There should be a new line after this.


should be `termish`


You're missing the summary here like what the others have.


Master branch Cashier is intended for L5 only. You should be using ~2.0 if
you want to use it with L4.

On Sat, Jan 24, 2015 at 2:15 PM, Stephen Neander <notifications@github.com>
wrote:

> This update for Laravel 5 breaks Cashier in 4.2 as the loadViewFrom method
> doesn't exist in 4.2. To rectify that you need to lock in to the commit
> before this one in composer.
>
>     "laravel/cashier": "dev-master#a08d376a380a713c613d6bf09082e2c18a8a0031",
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/cashier/commit/67e9c423221021103ed50f7d60ad46f769e5d7c5#commitcomment-9429344>
> .
>


Hondu el node_modules no se comitea xD

Tienes que crear un [http://browsenpm.org/package.json](http://browsenpm.org/package.json)


@NCNecros what about unit tests?


Didn't know fftpack doesn't support np.float128.

No, no, let's do not complicate things with dropping back to np.fft --- let's tackle this if someone will ever complain.


@Optimusprimeums can you please test the latest OSX-Java6 branch before I tag up?


whitespace!


If `new()` and `update()` are the same, wouldn't it be better to just override `sync()` instead?


sweet!


can't figure out what these functions do. convert them to named functions and call it from here.


test


done


1. 255.255.255.255 is indeed an invalid IP
2. [256-999]-[256-999]-[256-999]-[256-999] is easily to be treated as an fake IP pattern because it is not even a valid IPv4 address.


Almost aligned, but not quite there yet :) Maybe not align at all here, since that might actually look really dumb and not increase readability that much ...


A few spaces too much here


should be:
public void TeleportToPos(BasePlayer player, Vector3 destination)


Then I think `package.json` is wrong. All JSON matches now though. I can revert?


not needed. Compiled in kernel not as module


You can simply use ```#pragma once``` for consistency and simplicity


"## This is a markdown file"


I think we could use an exception in case `new()` or `update()` is not redefined. More specifically, require subclasses to either redefine `sync()`, or both `new()` and `update()`.


What instead?


Really very super neat toy, inspired an [idea](https://github.com/rthbound/WhatEarthIsIt) :) Thanks @MarvinMenzerath


What is the purpose of this class?


@razzeee sorry for going ot, is the trakt forum down or is it just me? i left a post there about this commit last night/this morning and haven't been able to get back in since, i realize the post is almost certainly obe (or obc, overcome by commits, lol), wondering about the other forum's status


Tsk tsk... we might let you slide on this one...


@eliotsykes my user_id field is NULL for each list at the moment, I'm not sure how to assign this value to a pre-existing set of users (currently 2) that's random each time? Should I be doing it this way or creating users too and then creating a permanent user that i know the login credentials, I vaguely remember this for Bloccit, just need a steer? Ta.


@darkessence87 - Server and Client should be used only with "try-with-resouces" statements, lets try to use full richness off language.

see https://github.com/warloc69/JGame/blob/master/Server/src/test/java/org/jgame/protocol/ServerImplTest.java#L36


I think you misunderstood me. I meant having one class with empty `sync()`, and other one having pre-defined `sync()` that calls `new()` and `update()`. Then you could also make them ABCs, to get the closest to compile-time error checking Python can have.


## This is a markdown file


touch Test4.md
## This is a markdown file


Persistence rules


I think the reason of the error is explained here:

https://docs.djangoproject.com/en/dev/releases/1.7/#standalone-scripts


"We're sorry, but something went wrong."

Message I'm getting so probably down


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1967](http://teamcity.monogame.net/viewLog.html?buildId=5074&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 366, ignored: 6 Build time: 00:15:41


Thank for your update!


`bool|bool[]` perhaps ?


1. It is a broadcast IP, and in my opinion we should not use it, we do not know how Linkbucks reacts to this kind of request (might provoke bad consequences on their end.)
2. Yes indeed, however I suspect them to match this field in a stupid way (not checking if each component is lower than 256), which is why it stills works. Anyway, if they decide to drop requests from incorrect IP in the future, 255.255.255.255 should be forbidden as well.

Nevertheless, there is still the issue of getting this IP banned if Linkbucks blindly counts the number of requests per unit of time, and ban the IP if they are too many. With the great number of AdsBypasser users, it would not be suprising for it to happen.


Yeh, freel free to revert this.


@Sk00bz should add a line here too, some distros are adamant about that.


@Xunil1337 that has been fixed 8 days ago https://github.com/TrinityCore/TrinityCore/commit/32a728b33162d5f2a0f0f4b648e1a25b886544a2


Then it would probably be cleaner to use a different variable like `BENCHPROG` that defaults to `PROG`.


and Breeze Dark, please)


d3838b56e5bb7a3896d04056288977e20ac12089 0


I suggest we delete  lines 1-4


Ah, you already did that.


These are probably not "big", but IMHO worth mentioning:
- Partial Vertical Stretch option
- Screen Scaling Filter option
- Depth range hack (phantasy star Portable 2)
- Invert Axes button mapping
- Option to test analog sticks
- Logbook in dev tools


created file


Got help to get started on the vector stuff from http://stackoverflow.com/a/4694555


Creating `build` inside `src` won't work, I think. At least one will need to change `..` to `../..` in the cmake command below. It's better to create `build` at the same level as `src`, i.e. at the source root.


for webs i suggest
URLField¶
class URLField([max_length=200, **options])¶


@s7eele
yeah, happens from time to time. I think its self written and has some huge problems, you can't post in some topics etc


I tried running the cmake command from the root and it didn't work. I
never created the build directory myself but when I ran cmake from src
it compiled just fine.
On 01/24/2015 01:42 PM, Rinat Ibragimov wrote:
>
> Creating |build| inside |src| won't work, I think. At least one will
> need to change |..| to |../..| in the cmake command below. It's better
> to create |build| at the same level as |src|, i.e. at the source root.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ericfowler303/freshplayerplugin/commit/001891353a49da81d0f022cf7d4564522d9badda#commitcomment-9429447>.
>


Well, in this case I need to defer to @flavour

Most likely the idea of the ToDo is to be able to allocate to the exact time if the resource has an hourly rate, but I don't think that start/end time is appropriate for that. It seems more plausible to have a start date, an end date - and a total number of hours (actually: units).

If you for instance allocate a human resource, then start may be on Monday, end on Friday and total number of hours is 20 (...which could be auto-filled from individual hours logged in tasks). At least that is how it's handled in most PM tools that I know.

Although, the actual "units" is not always "hours" - it should be flexible as it varies with the resource type, it could be "days" (if you have a daily rate), "iterations" (if you have an effort-based rate), "kWh", "litres", "megabytes", "minutes", "km" or "miles", "tonne" etc...really depending on what the resource is charged by.

IMHO, we should stick with dates here - and additionally have accounting unit + total fields.


Anyway, we are still in 0.x versions. So I think breaking compatibility would be OK


Maybe it's better to depend on `build-essential` then? It strongly depends on `make`, `gcc`, and `g++` while `cmake` have only recommendation for `make`.


Perfect ;)


Hi Mary Rose! You do not need to especify the path of npm modules in npm scripts, you can write as well:
```json
"scripts": {
  "test": "jasmine-node spec"
},
```

Great work, I hope to learn a lot using this repository :)


Will do! Thanks :) I have to say, this assignment was a lot of fun.

K

On Fri, Jan 23, 2015 at 8:42 PM, Thomas Wilburn <notifications@github.com>
wrote:

> Just as a quick note, don't forget to put this file into a sub-folder. It
> helps me be a little more organized if they're all sorted that way.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/klavin/web150-w15/commit/ba826739409096c48969946a99a2d0b54dcdc33c#commitcomment-9426817>
> .
>


It's at the *Raw level at the moment. I'm going to add it back in some shape or form soon.


for finding end of `def` in python, you can grab the white space before the `def`, and find the next line that starts with the same amount of white space, and that line is the first line after the end of the function.

though, not sure of a great way to express that in the template.


You can compile from source root, but you need to run `cmake .`, not `cmake ..`. Note two dots and single dot. Those are relative paths to where source is located. When you create `build` directory, and go there, `..` points to source root.
Actually you don't have to use name `build`. It can be anything, one can even not to create any directory at all.


yes. Good catch. Reverted.


annotate with @after? (thats how overrides are being done in other methods in client tests - annotating with the junit annotation in super and overriding methods)


Hey! I was about to fork and possibly create a patch for a Skip Confirmation option on ng-token-auth. Did you ever use this code or push a patch for consideration?


cool, thanks. will save thinking of a simple way to express that in the template for later :)


![Thumbs up](http://i.imgur.com/FW2EnlC.gif)


I see, I'm using 75a20b9 for now


@jareware yes.


this should probably not be gettexted, as it result in translated text in Library info - not very useful


That's not necessary, CroppingLabel inherits publicly from QLabel, so anyone can call myCroppingLabel.setOpenExternalLinks() directly anyways :)


@backinside ez mit csinál? Nálam `Assets/Scripts/Door.cs(13,26): error CS0115: droptarget.Drop(UnityEngine.Collider)' is marked as an override but no suitable method found to override
`


when you ask for plane of normal (0, 1, 0) and distance 1. Logically you ask for Y = 1. But if distance (0, 0, 0) to plan = 1 => a0+b0+c*0+d = 1 => d = 1, you receive the plane Y = -1 ! https://bitbucket.org/sinbad/ogre/src/5ee010d450fd064007b61442124222bfb3d57a58/OgreMain/src/OgrePlane.cpp?at=default#cl-50 That's why in Ogre distance becomes -distance (line 50).


I made the changes to use the bootstrap 3 alert classes.


Oh right, didn't know about that !


ups - just wanted to do a commit and can't as you were first. got to merge the stuff together... :-)
Sidenote: Why is the "AVI" thumb so big?


i ment this lines, from django.db import models is already repeated in line 5


@jdavis Thanks much for open sourcing this template, it's awesome!


Copy-paste derping, shouldn't `$skype` be `$python`?


Hi Peter.

Do you feel that this branch should go into the repository, or is it dormant?

Best,
Søren


This gem doesn't work for Ruby 2.2+, so I want to avoid including it on the Gemfile.


@jcrubino I was about to give you a serious lecture about pushing big hacks like this to `master`.  Glad to see you caught this quickly, and I do not need to rip you a new asshole.


Updated .gitignore file to use Unity


@bamos
Right, I still haven't actually uploaded the HTML version to my site. This README is now out of date, anyways. When I update it, I'll probably update my website, too. Thanks!


Lol... you've got to be quick :D AVI thumbs the same size as the rest for me....


The R code for for project is in run_analysis.R


@SubJunk building and testing the branch now


Right... just noticed. But here in the github show it looks really big: https://github.com/iBaa/PlexConnect/commit/2f651ee0df20a673228c93c748de00486805894e
Maybe zoomed in as it's less wide? Strange.


Hi @bamos, that is totally fine by me!


@weltling Does it error in the test case itself above as it was written, or did you do something else?


Nope... looks ok at your link as well :)


@tobyli Could you push this change against the "develop" branch? Master now reflects the latest mavenized release. Develop is the active development version.


Also, thanks for the catch!


Since the accelerometer returns values in g-forces, should we multiply by 9.81 (g) to get acceleration in standard units? That might explain the wrong numbers.


@yakky I think this check does not work with https://github.com/aldryn/aldryn-apphooks-config/commit/d836402e7fcf61d0f5dcc1a47c8b486eb2fb95ca#diff-4812ab4220a7365cc82d1b9eafe936ebR18
Maybe it should check also if `app_config.cmsapp is not None`?


return name != null && name.startsWIth(packageName);


Okay ... I added a retry mechanism deal with it.


global.getHandlers().forEach(h -> h.setFilter(filter));


Looks like you're using font-awesome!  May I suggest using svg icons?

Some resources:

* http://css-tricks.com/svg-sprites-use-better-icon-fonts/
* http://caniuse.com/#feat=svg
* http://evil-icons.io/


Really strange. Did a reload and it's back in the line.
Question - since from the code I don't really understand what you did... :-)
Why didn't you prepare those badges in XMLTemplate and fill/cut them if needed, instead of adding them hardcoded from XMLConverter? It doesn't seem reasonable, compared to the rest of the process.


No...just...no.


you can drop the system tag. it's not used anymore.


this group has no control with id="301"


`replicateA` is in `Applicative`.


Alright, this is better.
Although I prefer giving a false IP address, because we never obtain a survey this way, while using a random IP might need many requests before it is associated with a country for which Linkbucks has no survey.

Besides, I think that this change might fix some weird issues in which users were always obtaining an invalid token, maybe were they in a country where Linkbucks is disabled.


indenting goes wrong here


getHandlers returns an array, so this is going to be: `Arrays.stream(global.getHandlers()).forEach(...);`


just saw that we have another and better pull request for this part, so will try to drop this pull request.


please use an RDS prefix for builtins you add.
just 'Action()' is way to generic.
RDS.Action(foo) would be much better


@bbroerman30 that's a general behaviour, like also with bug65538_031.phpt . I've debugged the test only till now, and there

- the server part executes and terminates
- the client part blocks on file_get_contents() call

I'm testing the builds in daily tact manually, and that was not the case before. Also there are the automated test reports available at qa.php.net. Note that those tests didn't change since quite long, and much less by your commit. I first thought that file_get_contents makes a blocking read, but that should be actually not true as it terminates as expected when passing a timeout manually. If the connection is closed by the counterpart, the timeout should be abandoned by the underlying libc API. At least that's the behavior expected.

Thanks.


> It doesn't seem reasonable

Oh here we go again.

Because it's a hell of a lot of "fill/cutting" that needs to be done in multiple xml files..... better to just writ one simple function that can then be used in any xml file.

The index ordering of the badges is extremely important, if the ordering is wrong then the aTV just crashes :( Making sure each badge has the right index would be very difficult if not impossible in xml.


Wow I'm embarrassed ;/ . Can't believe it was that small of a change. There are other flaws that I ended up fixing in Recompiler Analysis.c. I'll send a pull request later today.


Commits during the Week End are not allowed !


Agree with @ftomassetti on the enum.


test


this line is removed


I'm not keen on hard-coding the format here - `guide` isn't the only format that's multi-part (`programme` is also a multi-part format, see [an example here](https://www.preview.alphagov.co.uk/1619-bursary-fund)) and there could be other new formats introduced. I'd prefer a duck-typing approach (anything that's multi-part).


?That's going to be a problem next weekend... Saturday is the last day of the project, and I'm probably not going to be done Friday...

Ben M. Angel
Documentation Writer

?OnePoint

m: +375 (29) 234-27-28
e: angel@1pt.com
w: 1pt.com?

________________________________
From: Jérôme Bogaerts <notifications@github.com>
Sent: Sunday, January 25, 2015 00:25
To: oat-sa/doc-converter
Subject: Re: [doc-converter] finished navi (e498d6f)

Commits during the Week End are not allowed !

-
Reply to this email directly or view it on GitHub<https://github.com/oat-sa/doc-converter/commit/e498d6fdb7546c90822fd9872f69cf1b67993f55#commitcomment-9429595>.


This could be made more generic with `<h2>Metrics for entire <%= format%></h2>` or `Metrics across all parts`?


Works great .thanks for this


This is fixed over on my repo now btw. :)


как всегда шикарная работа, спасибо :+1:

как я понял это парсер для кэша клиента?


Is `hostname` in this context guaranteed to be stripped of eventual credentials (which means two `:` characters in total)?


small change yes, still took me a few hours to find that small change


Ok. I asked for a reason, didn't I?
...


I might be missing something... how does `/<!-- START doctoc/` break backwards compatibility? It will match extant TOC `<!-- START doctoc generated TOC -->` headers.


What about https://github.com/hrydgard/ppsspp/pull/7369 ? :)


Wouldn't it be better to change SUN/209 so that it doesn't use these? Now that I know what they do I can't see any reason that we should use them.


Actually... I just cherry-picked this change into develop. Thanks again!


@Subjunk just a few this time
net/pms/newgui/GuiUtil.java:[28,73] type javax.swing.DefaultComboBoxModel does not take parameters
net/pms/newgui/StatusTab.java:[116,39] cannot find symbol
symbol  : method revalidate()
location: class java.awt.Container
net/pms/newgui/GuiUtil.java:[41,45] non-static variable super cannot be referenced from a static context
net/pms/newgui/GuiUtil.java:[41,50] cannot find symbol
symbol  : method getElementAt(int)
location: class java.lang.Object
net/pms/newgui/GuiUtil.java:[39,17] method does not override or implement a method from a supertype
net/pms/newgui/TranscodingTab.java:[547,22] cannot find symbol
symbol  : constructor JComboBox(net.pms.newgui.GuiUtil.MyComboBoxModel)
location: class javax.swing.JComboBox
net/pms/newgui/TranscodingTab.java:[579,31] cannot find symbol
symbol  : constructor JComboBox(net.pms.newgui.GuiUtil.MyComboBoxModel)
location: class javax.swing.JComboBox
net/pms/newgui/TranscodingTab.java:[845,36] cannot find symbol
symbol  : constructor JComboBox(net.pms.newgui.GuiUtil.MyComboBoxModel)
location: class javax.swing.JComboBox
8 errors


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


Thanks. I see where the problem is, but I dont know how fix it :-(


Pretty sure it was an accident to put this file here lol


Not sure what really needs to be in the list, but these are some of the bigger ones since v0.9.9.1 I think (it's been a while):

2dd9b10 Take screenshots with savestates.
d9e543f Reduce PowerVR blockiness problems on older GPU revisions. #7153 #7150
34fe830 Support flat shading. (weird graphics in some games, Dante?  Brave Story)
7409263 Make CPUDetect.cpp not blow up on Zenfone devices.
f3f7fb2 Show an on screen message when disk full is hit.
49aff7c Correct unload of imported vars. (God Eater 2, Tales of Radiant Mythology)
c8e6a80 Use a more frequent mini hash in the vertex cache. (less vertex cache glitches)
afefac8 Update to SDL2
361b9a2 Use 8888, always, for Danganronpa's FBOs. (Danganronpa on Adreno, #6807)
dc91dc1 x86jit: Support fpu rounding modes for mul, etc. (Gods Eater Burst, Peace Walker, NOTE PREVIOUS SAVES IN GEB)
385df1c Force positions to scale by 128/32768 as psp does. (Final Fantasy 4, Tekken, etc.)
62a1023 Add "Screen Scaling Filter" option

-[Unknown]


Solid work everyone.


Да, проблема есть, но парсить топик каждый раз при выводе на предмет нового логина тоже не хорошо. Текст топика формируется по правилам jevix-а только при его сохранении. Т.е. при пересохранении топика имя сменится (неполное решение, но всё же). К тому же ссылка на профиль постоянна.


seems like maybe this is missing?


Wrong commit message because.. derp


Надо в правила джевикса добавить class к тегу a, иначе из комментариев этот значок при редактировании выпиливается.


LDCDS


And no, I am not deleting everything and than pasting. Netbeans is being funky again.


I think this would be a bit more readable if it were:

```
<h1>Why is this <%= format=='guide' ? 'guide' : 'page' %> on GOV.UK?</h1>
```


This slows it down to about 45 seconds, but is still much faster than the make build while being just as safe because I make a copy of the NuttX source directory for each board so that they don't have to wait on each other.


Disney's gm icons make it look too much like TTR ; ;


The point was here to use a system font in case the bundled font is missing.


I guess you have edited this from Eclipse, indentation is a bit broken. :)


I updated this jd to be
```
/**
 * Represents a {@link Selector selector} argument. An Argument is a
 * type-to-value mapping used only for building a selector, as Map provides a
 * better way of getting this information than iterating over a list of
 * Arguments.
 *
 * @param <T> The type of the value
 */
```


This doesn't do anything and should be removed... (it's `<%` instead of `<%=`)


Looks good!  Be careful about extraneous whitespace and blank lines.  I realize it might seem trivial, but good discipline around these things makes it easier to read the code at a glance.  It also gives the reader — whether it's another programmer or your future self — a clearer idea about the intended divisions in the code.

I like that you have support for inputting multiple files.  From a user-experience perspective, I might consider adding something to the output so that you can tell which histogram corresponds to which file.

See my comment to @sags1083 about creating a `Hash` with a default value other than `nil` (https://github.com/codeunion/feedback-requests-web-fundamentals/issues/684).  This will simplify your `item_counts` method.


`{@link Argument.Invertible}`?


AFAIK the credentials are not put into the `Host` HTTP header so that shouldn't be a problem.


The idiom in Ruby is to put a space around operators like `+`, `<`, `%`, etc.


Looks good! :+1:


@Optimusprimeums can you try pulling? Since it's showing errors that I've already fixed


And I've given you two.... Are you happy now?


See this commit here: plumber-madeline-52027.bitballoon.com


FYI: I pushed this change in maven release 0.5.2.


merged into master because, well, whatever, it's too hard to maintain otherwise... probably the only way to do it later will be to create a special fake username?


This methods does not seem to be very idiomatic for Fluent Assertions.  The string parameter would typically be the `because` reason not an additional assertion.  As reported in #219.

Users could get the same functionality by:
`collection.Should().Contain(strArray2).And.Contain(strArray2)` or,
`collection.Should().Contain(strArray.Concat(strArray2))`


Carbon plugin required default timezone settings.


Same concern as method below.

ref #219


You must not edit config directly. Check out other functions that edit this variable, *they are not using global*.


Oh my! Touche.


See this commit here: http://waiter-initials-37564.bitballoon.com/


`get_powerline_raw` does *not* receive module name. In my diff it received `Powerline` class.


`.kill TODO`


In idiomatic Ruby, you'd surround comparison operators like `==` with spaces (for readability).


Tabbing here is indented too much.


`x` is not a great name; I'd at least name it something like `item` if not `str` or `string`.


Will look into this in more detail on Monday.


Lowercase `true`, this isn't Python.


Note that although the above pass, the `__repr__`'s are different:

```python
In [6]: c1
Out[6]: gpkit.Constraint(1 >= 10*x)

In [7]: c2
Out[7]: gpkit.Constraint(1 >= 0.5 + 5*x)
```

I assume this is the expected behavior, and I'm certainly fine with it -- it seems consistent with how we store `left` and `right` for constraints. I only mention it because https://github.com/convexopt/gpkit/issues/123 suggested that these should be "simplified automatically" and "equivalent".


if python3 input else raw_input right?


To remove all $$hashkey


We definitely should start adding meaningful tests


Rupee Ore only spawns between 150 to 256 Y now?


IIRC don't have a comprehensive page for the differences between Nvim and Vim yet, outside of the wiki of course. That'd be great if there was one available in nvim.


Thanks for adding the feature flag.


You need to use the 0.8 syntax:

socket.join("topic:play", {}, function(channel) {
 // ..
});


Ok tested it and worked.


`Represents an immutable selector of targets as used in commands.`?


`raw_input()` is not available in Python 3.


your topic is `"topic: play"`, so here you are handling an event of the same name. Likely, you want to use a different event name here.


Thanks for fixing it :+1:


A better name might be something like `item`, `str`, or `string`.


https://github.com/DORSIGUER/vagrant-shack/blob/a9e5e390e1c2536b896d5daea5f936f355e58e7c/shack#L79

Can you please remove the try catch as u did on the options?


Could be shortened to `If you can't find what you are looking for in the help system,`, although it doesn't make much of a difference.


Surround comparison operators like `<` with spaces.


Surround comparison operators like `==` with spaces.


Ай-яй. А ускорить никак нельзя? Очень уж медленный селектор получается.


There's no need for these extra blank lines! :)


Maybe `Welcome to Neovim!`?


I like these test cases!

I'd also include a few more basic checks, e.g.,

```ruby
p word_count("word") == 1
```


This is a fine name for the variable, although a more idiomatic name in Ruby would just be `words`.


testing thing out


Likewise, I'd name the loop variable `word`.  The loop would wind up looking like...

```ruby
words.each do |word|
  # ...
end
```


this can be condensed to one line.  redirect_to home_path if signed_in?


No need for this extra blank line! :smile:


this can also be one line


This should be `@Nullable`, as player head skulls don't have to have an associated profile.


Good test cases!  I'd include some more "basic" sanity checks, too, like...

```ruby
p count_in_list([], "a") == 0
p count_in_list(["a"], "a") == 1
p count_in_list(["a"], "b") == 1
p count_in_list(["a", "a"], "a") == 2
```


I actually had a question.  I noticed in the solutions Kevin wrote:
<code> redirect_to home_path, notice: "You have successfully signed in."

I'm not sure if that is doing something different than the flash?


you can use ILIKE to make the search not case sensitive


Moet het niet 'smalle spoorweg' zijn? Als smal op het spoor slaat is 'smalspoorweg' wel goed.


This works, but there's no need to assign the return value to a separate variable before returning it.  You could just do

```ruby
def count_max(list)
  largest_in_list = max(list)
  return count_in_list(list, largest_in_list)
end
```

Or even more directly

```ruby
def count_max(list)
  return count_in_list(list, max(list))
end
```

Or using Ruby's "implicit return" feature...

```ruby
def count_max(list)
  count_in_list(list, max(list))
end
```


Idiomatically, you put spaces after commas (`,`) when calling a method in Ruby.


I left a few line notes for you.  Everything looked good!


I'd use a name like `item` or `num` here.


Ignore this commit. Some weirdness when testing my directory structure on my local development box.


This 100% works.  You can also do this, which is a slightly more idiomatic short-hand for what you wrote:

```ruby
total += x
```


No need for the extra blank line here!


Just `name++` here, and then have a single `imcb_chat_name_hint(gc, name)` below.


/me feels very dump
Thanks! :)


for it again : https://github.com/unified-db/Database/issues/6#issuecomment-69845480


@samdark this file could be safely removed, because the topic is mentioned in "design decision" and "translation workflow".


I commented out some tests that gave me problems:

-grocery_list_spec: How would you test that method? It gave me a weird error message that I couldn't decipher.

-author_spec: I tried to test if a new instance of author inherited from Person, the same way I did for user_spec, but it gave me a loading error. What am I doing wrong here?

Thanks.


Good test!

I'd also want to make a decision about what should happen when we call `sum` with an empty `Array` as input, e.g., `sum([])` and write a test case for that.  If I'm reading it correctly, your code right now will return `0`.  Is that the most sensible behavior?  What are some possible alternatives?

Don't take that as a leading question, BTW.  It's very possible that returning `0` is the most sensible thing to do!  The important thing is to consider the landscape of possible behavior, decide which makes the most sense, and then encode that decision as an explicit test case.

Other developers will thank you — including your future self!


@SubJunk i'll give it a shot, hopefully that fixes it


Idiomatically, you'd include spaces around arithmetical operators like `/`.


-grocery_list_spec: How would you test that method? It gave me a weird error message that I couldn't decipher.

-author_spec: I tried to test if a new instance of author inherited from Person, the same way I did for user_spec, but it gave me a loading error. What am I doing wrong here?

Thanks.


I commented out some tests that gave me errors:

-grocery_list_spec: How would you test that method? It gave me a weird error message that I couldn't decipher.

-author_spec: I tried to test if a new instance of author inherited from Person, the same way I did for user_spec, but it gave me a loading error. What am I doing wrong here?

Thanks.


This is good!  However, there's no need to assign this value to the `average` variable before returning it.  You could just write

```ruby
def mean(list)
  total = sum(list)
  return total.to_f / list.length
end
```

Or even shorter

```ruby
def mean(list)
  return sum(list).to_f / list.length
end
```

Or using Ruby's "implicit return" feature

```ruby
def mean(list)
  sum(list).to_f / list.length
end
```


@softark thanks, removed it: ea15c15


Could have removed Desolate Land and Primordial Sea :P


thanks, update in local, in the next update fixed.


Good test cases!  Make sure to indent.

I'd include some more essential test cases, too.  What should happen when the input is an empty `Array`?  When it's a single-element `Array`?


If you send me a pull request update instant :)


Looks like you misspelled "canola"


Looks good!


The irony is strong, but yea...


Yeah, that works too.


Use `strchr(CTYPES, name[0]) != NULL`.  The `CTYPES` macro is defined in `irc.h` as `"&#"`.  This is how bitlbee does similar checks internally, might as well keep it consistent.  While group chats always use the `#` symbol, it will correct invalid attempts at usage of the `&` symbol.


Let's get rid of the p's before committing


Thanks for the patch!  This certainly needs to be addressed, as this could be very misleading otherwise.

Kudos on the first patch to the project!


there are a few trailing whitespaces as well across all the commits:
```
$ git diff --check `git merge-base HEAD origin/master`..HEAD
cpp/audio/dynamic_resource.cpp:107: trailing whitespace.
+
cpp/audio/dynamic_resource.cpp:108: trailing whitespace.
+	return {nullptr, 1};
cpp/job/job_group.h:44: trailing whitespace.
+	 *        job has finished
cpp/job/job_group.h:65: trailing whitespace.
+	 *        job has finished
cpp/job/job_group.h:80: trailing whitespace.
+	/**
cpp/job/job_manager.cpp:60: trailing whitespace.
+		std::swap(jobs, it->second);
cpp/job/job_manager.cpp:106: trailing whitespace.
+	// otherwise, we append the job to the existing entryy
cpp/job/job_manager.h:34: trailing whitespace.
+
cpp/job/job_manager.h:53: trailing whitespace.
+	/**
cpp/job/job_manager.h:91: trailing whitespace.
+	 *        job has finished
cpp/job/job_manager.h:111: trailing whitespace.
+	 *        job has finished
cpp/job/job_state_base.h:25: trailing whitespace.
+	/**
cpp/job/tests.cpp:26: trailing whitespace.
+
cpp/job/typed_job_state_base.h:30: trailing whitespace.
+
cpp/job/types.h:30: trailing whitespace.
+ * exception, it is rethrown by this function.
cpp/job/worker.cpp:64: trailing whitespace.
+
cpp/job/worker.cpp:66: trailing whitespace.
+			// stopped
```


trailing space


Do you mean line 18?  In this commit, there are only 21 lines in the file.  I'll assume yes and reply accordingly.  Let me know if you're talking about a different version of the file. :smile:

There's one main reason people use the `if __FILE__ == $PROGRAM_NAME` idiom: sometimes you want a file that you can both run directly and that you can `require` inside other files.  In the `require`-inside-another-file case, we almost _never_ want the code to do something automatically simply because it was required.  The file doing the requiring can decide whether and how it wants to call the methods defined in the required file.  Conversely, in the run-the-file-directly case, we _by definition_ want it do something.

This idiom allows us to have our cake and eat it too, so to speak.

Given that, even if you put the the code on lines 17-20 into their own method, you'd still have to call that method inside the `if __FILE__ == $PROGRAM_NAME` block.  Remember: we want the ability to require this file elsewhere, which is why we have that conditional in the first place.

So, there will always be "some" code that isn't wrapped in a method.  It's not uncommon to leave code related specifically to running the program from the command-line outside of a method, e.g., parsing command line arguments, reading from input files or other input streams, and so on.

Sometimes that logic winds up being complex enough that it merits its own method.


I was going to, but then I saw Morfent's earlier PR and left them in. (Hail doesn't exist in Gen 2 either!)


Das wusste ich nicht zu übersetzen, was ist hier mit macro gemeint?
Original: "While `layout/` is *macro* (defining the global wireframe), `components/` is more focused on widgets"


Thats a nice copy and paste job you "implemented" here :D


oops I was doing that for testing and forgot to fix it. I need to get my brain checked.


This little "error" means there isn't a newline character (`\n`) at the end of the file.  Ensuring your files always end with a newline is a "good housekeeping" habit.  For example, the line count of this file will be off by one due to the missing newline character.

There are a few ways to make sure this doesn't happen.

1. Configure your editor to automatically add a newline to the end of the file if one isn't present.  There's a configuration setting in Sublime Text 3 you can turn on to enable this behavior.  See [this Google search](https://www.google.com/#q=sublime+text+automatically+insert+new+line) for a bunch of pages describing how to do it.
2.  Run `git diff` both before you add files to commit and actually commit them.  `git diff` output the same line-by-line "diff" or "changeset" you see here on GitHub, highlighting the lines you've added, edited, or removed.  It will also indicate whether there's a newline at the end of the file or not.  Hit `q` to exit `git diff`.

    I run `git diff` pathologically.  It's burned into my muscles. :smile:


compile error:
In file included from src/layer/layer_impl.cpp:5:0:
src/layer/layer_impl-inl.hpp:24:36: fatal error: ./insanity_layer-inl.hpp: No such file or directory
 #include "./insanity_layer-inl.hpp"


my first commit


committed. Try again plz

On Sun, Jan 25, 2015 at 6:37 AM, Mu Li <notifications@github.com> wrote:

> compile error:
> In file included from src/layer/layer_impl.cpp:5:0:
>
> src/layer/layer_impl-inl.hpp:24:36: fatal error: ./insanity_layer-inl.hpp:
> No such file or directory
>
> #include "./insanity_layer-inl.hpp"
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/antinucleon/cxxnet/commit/7a3c419f8526ecea6807b620eeacbf251f01c947#commitcomment-9429849>
> .
>


@Subjunk, not sure if i'm doing something wrong but it won't build at all with Java 6 (Using JDK 6 in netbeans) same issues as above. Pulled into a new clone of UMS


of course the master writes to etcd, that's what it uses as a database....    that isn't what the kubelet should use.  what's the real bug?


Daily events/dungeons? Doesn't sound like G1.


Oh joy, more spaces.


Fixed.


lol yes i didn't notice that bad past add a + before #  !

and yes, it  still have "untracked memory ", even if game works.
so must have something wrong in osd part .

i'll looking at when i'll have time, if no one else does it before.


FavorExpression.


I will investigate the legal/licensing aspect of using this


"Unimplemented" is a clear message, "Come back later" sounds like a bug if the time is not wrong.


ok will use _WIN32 & __MINGW32 now


thx for the response and given its still not available im gonna ask here:

@nn-afly - if u dont mind, what platforms are you running on and w/ which versions of kodi? sorry if i missed it somewhere, ive noticed some differences between windows and android so looking at the need to compare notes

@Razzeee - i haven't had time to go through them yet but i was able to gather some good logs capturing whats going on with the whole virtual library item thing, im definitely still seeing it on both android and windows, i only mention it because up until last night/this morning things havent been stable enough to make gathering the logs possible, dont know if it was just a matter of api stability but it was the first time the add-on has delivered consistent results in quite a while, just another fyi


@wayneyu Here we should run through ALL tags, and set `selected="selected"` only on those tags that the question currently belongs to. Could you please pass the list of all topics to each question page?


thanks:)

On Sat, Jan 24, 2015 at 5:38 PM, winstywang <notifications@github.com>
wrote:

> committed. Try again plz
>
> On Sun, Jan 25, 2015 at 6:37 AM, Mu Li <notifications@github.com> wrote:
>
> > compile error:
> > In file included from src/layer/layer_impl.cpp:5:0:
> >
> > src/layer/layer_impl-inl.hpp:24:36: fatal error:
> ./insanity_layer-inl.hpp:
> > No such file or directory
> >
> > #include "./insanity_layer-inl.hpp"
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/antinucleon/cxxnet/commit/7a3c419f8526ecea6807b620eeacbf251f01c947#commitcomment-9429849>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/antinucleon/cxxnet/commit/7a3c419f8526ecea6807b620eeacbf251f01c947#commitcomment-9429854>
> .
>


If you don't have this issue on openrc we can add a check for systemd also and exclude it on openrc systems. Would be then:

if [[ -e "/usr/bin/lightdm" ]]  && [[ -e "/usr/bin/systemctl" ]]; then


Там в /b встречаются посты с comic sans шрифтами. То ли шутка админов, то ли фиг знает

```html
<div id="p594433442" class="post op"><div class="postInfoM mobile" id="pim594433442"><font face="comic sans ms"><span class="nameBlock"><span class="name">Anonymous</span><br><span class="subject"></span> </span><span class="dateTime postNum" data-utc="1422132305">01/24/15(Sat)15:45:05 <a href="thread/594433442#p594433442" title="Link to this post">No.</a><a href="thread/594433442#q594433442" title="Reply to this post">594433442</a></span></font></div><div class="file" id="f594433442"><div class="fileText" id="fT594433442"><font face="comic sans ms">File: <a href="//i.4cdn.org/b/1422132305961.png" target="_blank">deejay.png</a> (128 KB, 700x600)</font></div><font face="comic sans ms"><a class="fileThumb" href="//i.4cdn.org/b/1422132305961.png" target="_blank"><img src="//1.t.4cdn.org/b/1422132305961s.jpg" alt="128 KB" data-md5="8hpVG6VFcefrm+2ectBFDw==" style="height: 214px; width: 250px;"><div data-tip="" data-tip-cb="mShowFull" class="mFileInfo mobile">128 KB PNG</div></a></font></div><div class="postInfo desktop" id="pi594433442"><font face="comic sans ms"><input name="594433442" value="delete" type="checkbox"> <span class="subject"></span> <span class="nameBlock"><span class="name">Anonymous</span> </span> <span class="dateTime" data-utc="1422132305">01/24/15(Sat)15:45:05</span> <span class="postNum desktop"><a href="thread/594433442#p594433442" title="Link to this post">No.</a><a href="thread/594433442#q594433442" title="Reply to this post">594433442</a>   <span>[<a href="thread/594433442/draw-thread" class="replylink">Reply</a>]</span></span></font></div><blockquote class="postMessage" id="m594433442"><font face="comic sans ms">Draw Thread<br>Pin's Body Pillow Edition<br><span class="quote">>Try not to bump requests too often</span><br><span class="quote">>Don't take the bait</span><br><span class="quote">>Don't talk to it</span><br><span class="quote">>Don't be afraid to try a request</span><br><span class="quote">>Hide and Report Spammers</span><br><span class="quote">>Keep Reposting to a minimum</span><br><span class="quote">>New Drawfriends are ALWAYS welcomed </span><br><br><br><span class="abbr">Comment too long. <a href="thread/594433442#p594433442">Click here</a> to view the full text.</span></font></blockquote></div>
```


Untracked memory errors have been happening since we were on 0.155
depending on the compiler used. It has never caused any real issues it's
explained somewhere in the Mame source code

On Sat, Jan 24, 2015, 5:40 PM not6 <notifications@github.com> wrote:

> lol yes i didn't notice that bad past add a + before #  !
>
> and yes, it still have "untracked memory ", even if game works.
> so must have something wrong in osd part .
>
> i'll looking at when i'll have time, if no one else does it before.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/libretro/mame/commit/d96423e62451afb539049c88ec74d4be53d5e5ed#commitcomment-9429864>
> .
>


So basically this should be fine.


Вот так?
`.postInfo > .postNum > a:nth-child(2), .postInfo > font > .postNum > a:nth-child(2)`


Nice


I was mainly referring to the fact you've put www.cyclestreets.net, rather than a cyclescape.org address.

But hard-coding this is problematic anyway, as there is another installation on a .cz address, and could be others in future.

Can you clarify what the problem trying to be solved is? I can't say I've ever spotted the link not working from an e-mail.


Thanks, this is going to be fixed in ``feature/hotfixes`` branch


Сейчас там таких постов уже нет... Хм.