Can this `else` ever be reached if the image is multichannel? I think the if-tree is slightly incorrect, it should be something like this:

```python
if img.ndim == 4 and multichannel:
    # process img as though it were 3D with RGB values
elif img.ndim == 3:
    if multichannel:
        # process img as though it were 2D with RGB values
    else:
        # process img as though it were 3D with intensity values
elif img.ndim == 2 and not multichannel:
    # process img as though it were 2D with intensity values
else:
    # raise ValueError, invalid arguments
```


Approved


`print` 可以考虑改成使用 `log` 输出


jkasfhajkgfh


OK, agrego esas observaciones y actualizo el repo desde casa.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1198](http://teamcity.monogame.net/viewLog.html?buildId=13777&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 561, ignored: 6 Build time: 00:27:59
### Failed tests
```
MonoGameTests.exe: MonoGame.Tests.Visual.RasterizerStateTest.VisualTestCullMode(CullCounterClockwiseFace): <no details avaliable>

```


这一部分可以考虑先注释掉


We are the kings in the university jungle!! ROAR!!!


Ok


Thank you! But we should probabbly also add the version(s) between 1.0.372.2 and 1.0.573.1.


Fixed within 50310f20eea1e966a4724c275044ece3e17fec67


Can you add "32 bit" or "64 bit"
in Device info ? Thanks


I think you could just write `for perp_points in perp_array.T:` here.


fully modified


/make: this also is related to issue: #1251


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


thanks.. that's why i used x11 for now (which means x11 with opengl really).


Missed this.


should this also check for `if self.sql_location`?


We usually try to keep the specificity lower as possible. To avoid issues when overwriting them. Its OK but usually try to use .column_two-thirds


event.header.stop_cells has since 020126065a321353cda02308d6f46a440b857d09
the same interface as event.data, i.e. like this:
```python
a_stop_cell = event.header.stop_cells[pixelindex][gaintype]
a_timeline = event.data[pixelindex][gaintype]
```


Actually the version of EventEmitter bundled with the plugin is still the "old one".
Building the plugin doesn't update it, why is that ?
You and I are running node v4.2.x right ?


I can't remember/don't know entirely how #pragma once works, but I think it's clearer/if the compiler's dumb will save you time if you stick it on the first line before the compiler has had to trundle through <vector> :)


local.rb now uses standard cron resource instead of echo


``other_entry != table.end()``


``remove != table.end()``


No real reason. Could use a cleanup.


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


Gayet güzel olmuş ;

1- Daha az değişken kullanmaya özen gösterebilirsin
2- Yazılan kodun girinti seviyelerine ve parantezlerin bulunacağı yerlere dikkat edebilirsin.
3- Yazdığın kod parçacıklarını static main içerisinde degil de birer metod içerisinde yazmaya çalışabilirsin Ayrıca fonksiyonun tamamını değilde küçük parçalara ayırarak metodların parametre alması , geri dönüş değerinin olması gibi işlere daha çok yer vermeye çalışabilirsin.

İyi günler


Why 8.4? Why not 8.0?


@adrianq LOL


That's a valid workaround as well.

In that same line, I'm curious why you chose `syncAdapter` to be volatile. This would yield the `synchronized` block useless, isn't it? I'd like to hear your thoughts on that if you don't mind.


wow


It seems more logical to do the splitting of the name into body and extension in `_onFileLoaded` rather than `setJobName`, since `setJobName` implies that the function sets the job name, and `_onFileLoaded` does things with files. Splitting an extension off of a file name has to do with files.

As a side effect, when you explicitly set the job name from the GUI to "exam.ple", it would change the job name to "exam", since the text field in JobSpecs.qml calls the `setJobName` function too.


I meant GNU lmao


This is a clumsy way of trimming a string. Can be done in one line: `s.erase(std::find_if_not(s.rbegin(), s.rend(), isspace).base(), s.end());`


A mistake :)
Fixing


Taking ``OrderedCovering`` as an example, I think most of it should be moved into the anonymous namespace of the cpp file so the header just contains ``minimise``. This will:

* Hide implementation
* Prevent the ``OrderedCovering`` namespace being polluted by stuff the user is not going to care about
* The stuff in the anonymous name space can still get inlined (it's the same as marking static) and I think it's safe to say the cost of minimize will outway any savings from inlining


PDB is telling me ``AttributeError`` on ``selector.show_arrow``.


Because


private => protected


Will be changed.


Vous faites un prototype. Vous n'aurez ni utilisateurs, ni contenu à vendre, ni publicités à fournir, ni argent (pour payer vos artistes). C'est un projet scolaire et non une lettre ouverte à Spotify ce projet.

Il vous faut, à mon avis, choisir un camp : est-ce de l'écoute de musique (spotify) ou de la vente (bandcamp). Et mettre l'accent sur ce _sucks_ qui vous tient tant à cœur en apportant vos solutions.


Why the need of merging this into a bugfix-frozen branch?


This may give an error if the file name has no extension.


맙소사... 성인 유-우머는 자제 부탁드립니다 :scream:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11924](https://teamcity.mesosphere.io/viewLog.html?buildId=107521&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:11:52


What modified?


I don't really know why, but I find this commit a bit confusing. Maybe it makes sense to split off "Extract StyleSetProps::checkProperties()"


@RandomAltThingy you changed to MIT but then reverted the change.


This seems a little inconsistent with the rest of your naming convention - ``Merge`` maybe?


Hi Conejo,
I think that this line https://github.com/osclass/Osclass/commit/dba0cc47b1f105c15c22b51e221444f51dd0341c#diff-28e183a36b8e9dc0b0e14948c2629e6bR167 is duplicate (not needed) - can you check this please?


Do we need a spec for this?


prefiero que el parametro no sea obligatorio y que lo seteamos con un default


Why this:

>  ``` @Override
+ protected int getExperiencePoints(EntityPlayer player)
+ {
+ return 1 + this.worldObj.rand.nextInt(3);
+ }
+```

super.getExperiencePoints(player) already has the same result...


Also, line 165


I THINK ``p_entry`` should be a ``const_iterator`` - If it can't, I suspect something is wrong with constness elsewhere


@RazTamir I am ok with the chance now. just please cover these methods with tests.
it is easy, just make sure that it generates expected commands, and correctly parse results.
see tests/test_filesystem.py


@MrPowerGamerBR Yeah this guy on sponge pointed out some downsides and anyway this is GPL it was previous LGPL


Check LocaleHandler.java, InboundCookiesHandler.java, OutboundCookiesHandler.java for private methods.


What is the point of this conversion operator? Why not simply a getter for `str_`?


I don't quite follow this - `makemessages` will coalesce the "Yes" here and "Yes" in `STANDING_CHOICES`, which are both marked as for translation, into a single `msgstr` string (with both locations as a `#:` reference comment above). I don't think they need to have separate translations, though? (If they did, we could use [a contextual marker](https://docs.djangoproject.com/en/1.8/topics/i18n/translation/#contextual-markers).) So I think the code's fine, but I don't follow the comment and commit message.


I'd agree, but it's useful to have access to some of the utility functions for testing purposes. I guess I could rewrite my tests to deal with the limited interface but it would reduce the controllability/observability.


MAGIC NUMBERS :( ``static const uint32_t ROUTE_LINKS = 0xffffffc0;`` (or whatever it relates to) would make what these are much clearer and compile to the same


was trying something . will call u later

On Tue, Jan 19, 2016 at 6:00 PM, Vishal Vijay <notifications@github.com>
wrote:

> What modified?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/VarunRaj101/cleaning-smart/commit/48dd42f225c8a7fcb2ddac04124e398821e4d6fc#commitcomment-15528217>
> .
>


More magic numbers


Lol


Thanks a lot for making this change. I think it's way less confusing now :)


You might not have the serial method on the daemon. Should pull down changes from the master.


I'm certain gtest can work around that - that is basically the purpose of
C++ testing suites :)

On 19 January 2016 at 12:33, Andrew Mundy <notifications@github.com> wrote:

> I'd agree, but it's useful to have access to some of the utility functions
> for testing purposes. I guess I could rewrite my tests to deal with the
> limited interface but it would reduce the controllability/observability.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mundya/rig_routing_tables/commit/4d519023f61ebfa09f55fa96343753c998fe939b#commitcomment-15528280>
> .
>


genial..!!


This


Que font ces signes ⚤, ⚧, ... là?

Je ne suis pas sûr que si vous montriez ces wireframes aux gens de YC movies, ils soient terriblement confiant quant à ce que vous allez leur proposer. Vous devriez tester pour voir.

Dans votre cahier des charges, limitez-vous aux parties récupération de contenu depuis Youtube et contenu statique (petit CMS). paiement, calendrier et https seront certainement hors de portée.


@tripolar this version is not building:

	jow@builder1:~/relman-cc$ (cd .cache/sdk/ar71xx/generic/; make package/tor/compile V=s)
	tmp/.config-package.in:36:warning: ignoring type redefinition of 'PACKAGE_libc' from 'boolean' to 'tristate'
	tmp/.config-package.in:64:warning: ignoring type redefinition of 'PACKAGE_libgcc' from 'boolean' to 'tristate'
	#
	# configuration written to .config
	#
	make[1]: Entering directory `/home/jow/relman-cc/.cache/sdk/ar71xx/generic'
	make[2]: Entering directory `/home/jow/relman-cc/.cache/feeds/base/package/libs/toolchain'
	if [ -f /home/jow/relman-cc/.cache/sdk/ar71xx/generic/staging_dir/target-mips_34kc_uClibc-0.9.33.2/pkginfo/toolchain.default.install.clean ]; then rm -f /home/jow/relman-cc/.cache/sdk/ar71xx/generic/staging_dir/target-mips_34kc_uClibc-0.9.33.2/pkginfo/toolchain.default.install /home/jow/relman-cc/.cache/sdk/ar71xx/generic/staging_dir/target-mips_34kc_uClibc-0.9.33.2/pkginfo/toolchain.default.install.clean; fi; echo "libc" >> /home/jow/relman-cc/.cache/sdk/ar71xx/generic/staging_dir/target-mips_34kc_uClibc-0.9.33.2/pkginfo/toolchain.default.install
	if [ -f /home/jow/relman-cc/.cache/sdk/ar71xx/generic/staging_dir/target-mips_34kc_uClibc-0.9.33.2/pkginfo/toolchain.default.install.clean ]; then rm -f /home/jow/relman-cc/.cache/sdk/ar71xx/generic/staging_dir/target-mips_34kc_uClibc-0.9.33.2/pkginfo/toolchain.default.install /home/jow/relman-cc/.cache/sdk/ar71xx/generic/staging_dir/target-mips_34kc_uClibc-0.9.33.2/pkginfo/toolchain.default.install.clean; fi; echo "libgcc" >> /home/jow/relman-cc/.cache/sdk/ar71xx/generic/staging_dir/target-mips_34kc_uClibc-0.9.33.2/pkginfo/toolchain.default.install
	make[2]: Leaving directory `/home/jow/relman-cc/.cache/feeds/base/package/libs/toolchain'
	make[2]: Entering directory `/home/jow/relman-cc/.cache/feeds/base/package/libs/ocf-crypto-headers'
	make[2]: Leaving directory `/home/jow/relman-cc/.cache/feeds/base/package/libs/ocf-crypto-headers'
	make[2]: Entering directory `/home/jow/relman-cc/.cache/feeds/base/package/libs/zlib'
	make[2]: Leaving directory `/home/jow/relman-cc/.cache/feeds/base/package/libs/zlib'
	make[2]: Entering directory `/home/jow/relman-cc/.cache/feeds/base/package/libs/openssl'
	make[2]: Leaving directory `/home/jow/relman-cc/.cache/feeds/base/package/libs/openssl'
	make[2]: Entering directory `/home/jow/relman-cc/.cache/feeds/base/package/libs/libevent2'
	make[2]: Leaving directory `/home/jow/relman-cc/.cache/feeds/base/package/libs/libevent2'
	make[2]: Entering directory `/home/jow/relman-cc/.cache/feeds/packages/libs/libnatpmp'
	make[2]: Leaving directory `/home/jow/relman-cc/.cache/feeds/packages/libs/libnatpmp'
	make[2]: Entering directory `/home/jow/relman-cc/.cache/feeds/packages/net/miniupnpc'
	make[2]: Leaving directory `/home/jow/relman-cc/.cache/feeds/packages/net/miniupnpc'
	make[2]: Entering directory `/home/jow/relman-cc/.cache/feeds/packages/net/tor'
	mkdir -p /home/jow/relman-cc/.cache/sdk/ar71xx/generic/bin/ar71xx/packages /home/jow/relman-cc/.cache/sdk/ar71xx/generic/build_dir/target-mips_34kc_uClibc-0.9.33.2/tor-0.2.7.6/ipkg-ar71xx/tor-fw-helper/CONTROL /home/jow/relman-cc/.cache/sdk/ar71xx/generic/staging_dir/target-mips_34kc_uClibc-0.9.33.2/pkginfo
	install -d -m0755 /home/jow/relman-cc/.cache/sdk/ar71xx/generic/build_dir/target-mips_34kc_uClibc-0.9.33.2/tor-0.2.7.6/ipkg-ar71xx/tor-fw-helper/usr/bin
	install -m0755 /home/jow/relman-cc/.cache/sdk/ar71xx/generic/build_dir/target-mips_34kc_uClibc-0.9.33.2/tor-0.2.7.6/ipkg-install/usr/bin/tor-fw-helper /home/jow/relman-cc/.cache/sdk/ar71xx/generic/build_dir/target-mips_34kc_uClibc-0.9.33.2/tor-0.2.7.6/ipkg-ar71xx/tor-fw-helper/usr/bin/
	install: cannot stat `/home/jow/relman-cc/.cache/sdk/ar71xx/generic/build_dir/target-mips_34kc_uClibc-0.9.33.2/tor-0.2.7.6/ipkg-install/usr/bin/tor-fw-helper': No such file or directory
	make[2]: *** [/home/jow/relman-cc/.cache/sdk/ar71xx/generic/bin/ar71xx/packages/packages/tor-fw-helper_0.2.7.6-1_ar71xx.ipk] Error 1
	make[2]: Leaving directory `/home/jow/relman-cc/.cache/feeds/packages/net/tor'
	make[1]: *** [package/feeds/packages/tor/compile] Error 2
	make[1]: Leaving directory `/home/jow/relman-cc/.cache/sdk/ar71xx/generic'
	make: *** [package/tor/compile] Fehler 2
	jow@builder1:~/relman-cc$

Upstream replaced "tor-fw-helper" with a Go language implementation which is unsuable for OpenWrt due to its Go runtime requirement (not available) and size (a 6.6 MB binary on x86).


The proper way to do this is to call set_config_option(). That will take care of rolling back the change on error.


I think this should use an OR condition instead of AND. Something like this:
```
lowResolutionDisplay = layoutManager.primaryMonitor &&
                       (layoutManager.primaryMonitor.width < LOW_RESOLUTION_WIDTH ||
                        layoutManager.primaryMonitor.height < LOW_RESOLUTION_HEIGHT);
```


lgtm


Yes, sorry. I was testing your fix and have left some code there.

Correct one should be as before any change : https://github.com/osclass/Osclass/commit/1953723c697a911be91fc9ae891a2b77f91e3207

Please reply the email, as we can not use 's_description_html' since it's nowhere used in the core. I think we could fix it with some filters.


Sorry, I meant SetConfigOption(). set_config_option is more low-level.


The original image doesn't allow to preserve flows in a local mounted volume. This version does but also loose some nodes.
Wait for the next commit to fix this


Extra line. The method args have strange indents. Have you reformatted this file?


Meteor package: [jss:path](https://atmospherejs.com/jss/path)


Just add it and let me know on slack. Lets do that manually for now.

On Tue, Jan 19, 2016 at 12:32 AM, Raghav Sharma <notifications@github.com>
wrote:

> Why remove this?
> Anyway, can you upload the migration file? Need to add a donation column.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AamAadmiParty/cleansweep/commit/eae01d7d57e78a81b0f78d210691e20b41c5bf78#commitcomment-15514654>
> .
>


Hi
Why did You change it ?
Now i see only number (1,2,3,4) under exclamation image

Fantom


Don't listen to him
![motivational-joffrey](https://cloud.githubusercontent.com/assets/7571158/12418833/1d877fe2-beb2-11e5-9c09-ce37c9ad14fd.gif)


Hi Conejo,

I have not yet received any email response from Carlos, I had the original idea with filter (not sure if he forwarded you that email - I can forward it to you, but don't know which email you currently use for this issues), but the problem is filters cannot currently pass any info if they are inside foreach loop. There is one ugly solution that we can use (e.g. if plugin exists & is active), but it is another hard-coded approach, since it is bound to particular name.


Updated the PR with constants as strings @niel.


Again, this seems inconsistent with your naming convention - ``struct === class`` aside from default protection level


alhnsvapnvznspe indeed.


хуйово.)


Good point. Still not playing nice, but we can look at the multi-device GUI thing another time.


seems enum items should all be capitalized.


extra line


Doode, you sure you want to share that super secret token with the rest of the world?


You're absolutely right! Thanks for pointing this out :-D

I just ammended this commit with a fix.


Meteor package: [peerlibrary:fs](https://atmospherejs.com/peerlibrary/fs)


Looks good :+1:


``std::tuple<unsigned int, uint32_t, uint32_t> get_settables(const RoutingTable::KeyMask& a, const RoutingTable::KeyMask& b)`` would seem a nicer signature to me - returning by reference is a semantically a bit gross in C++


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


Right, I replied to carlos only and not to you, sent now.


`connection.close` is both sent and received by clients, when either peer needs to close connection. So this little guy will be back ;)


Now there's more logic in this template, it might be neater to have extract it into a template filter, so you could just have:

```
<dd>{{ extra_field|extra_field_value }}</dd>
```


Meteor package: [patte:mime-npm](https://atmospherejs.com/patte/mime-npm)


BareFunctions::Delete


We haven't been listing other contributors here (there are 6 others I believe), and I don't think we should either. The github contributors page is a better place to look for contributors. I actually think we shouldn't list anyone here at all ("brigade" as the author is enough). I'll probably follow up with that change, so you don't actually have to do anything at this point.


``i != table.end()``


It's not a big deal to me - I kinda feel version numbers are internal anyway - but 3.0 now and 3.1 at pilot makes a bit more sense to me since I see big changes in this release and mostly fixes in 3.1.


not variable, but expression


s/update/upgrade/


Thanks a lot Kim!


LGTM.


s/Updating/Upgrading/


"Class has an extension function" is ambiguous


I think you want to template this:

```
template<typename F>
std::vector<unsigned int> find_removes(
     const Table& table,
     const merge_t& merge,
     F f
 )
```

Then you'll get different versions with your lambda inlined for each call (you may anyway, but ``std::function`` is generally a dynamic construct)


ok


TeamCity Civic Assembly :: Build all [Build 13](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=355&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


Personally, I think this , syntax gains you nothing but confusion :) (it doesn't even save space if you space it out like this)


Should we just unpath it, and rely on path to find coverage?


@julen You left a `PaidTaskFormView` reference behind here.


@pulse00 Could you change the homepage to http://www.ansible.com
See https://github.com/Homebrew/homebrew/commit/ccc3583f30adb0d91f2707fa70d75e1258883dce#diff-5ea2ef50061cead169e37867b860b42a
Thanks!


Oops never mind they have different names.


I'm not sure if "backup wizard" is clear here; I would say that it's part of the guided upgrade process, so no need to do it manually.

That being said, doing a backup the old way can still be useful, to restore Cloud 5 if needed...


s/backup wizard/upgrade wizard/


misleading name here


TeamCity Civic Assembly :: Build all [Build 13](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=355&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **UNKNOWN**
Summary: Canceled (Exit code 2; exit code 1; exit code 137 (new); exit code 123) Build time: 00:02:08


All I see is "vapn" I didn't know you vaped shuudoushi


+#ifdef CONFIG_MACH_hardware


These comments are just noise!


This === ``std::count(merge.begin(), merge.end(), true)``


Is this file still relevant?


Learn to maintain authorship please.


Auntunif


Cut the crap!


You got to bend over on the 80 char limit here!


Gmail.com


Tega nočeš tukaj - to pomeni, naj ti datoteke `analiza.r` v tej mapi git ne sledi. Svetujem, da kar pobrišeš to datoteko.


this is ok, but complex. Please take a look to: http://chriswu.me/blog/a-lru-cache-in-10-lines-of-java/

You dont need to fix or change your implementation.


This == ``std::fill(merge.begin(), merge.end(), false)``


Good morning, Heiko! (though I have been exchanging night for day lately)

I understand the mishap with the positioning, it's not an issue any more. I just wanted to get the textures done so that's why I just manually fixed everything myself. But just notice that while some changes can be explained by rescaling, others have changed when they shouldn't: for instance, the left side of the floor (below the pilot) moved forwards and upwards by quite some margin. Anyway, it's really all fine now, and I really appreciate your comments :smile: :+1:

> I added a dev folder, where also the blender-file is. Would that be the right place?

I think so, it keeps things more organized. In the c172p project, we keep all blender files in the folder (e.g. pitot tube cover, chocks, etc.). But the xcf are too large to keep in this project IMO, only the interior textures are around 200 Mb, and I haven't touched any external textures yet! So what do you say if we make a a side repository as I proposed? See https://github.com/gilbertohasnofb/c172p-detailed-liveries for what I am talking about.

> Good luck for your master thesis! :-)

Thanks a lot, cheers!


Any reason to remove `STATIC`?


see 4010738


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


Thanks.
Will this icon be in the next version of the android client? Or will it only be in the beta-branch to differ the two apps? (Hope it will go to the stable branch soon ;) )

PS.: I provided a [pull request](https://github.com/owncloud/News-Android-App/pull/482) to News-Android-App and i am [using it](https://github.com/stefan-niedermann/OwnCloud-Notes/commit/7460d38403aa631560f70ee8f85ecfd4bd44ea2b) in the ownCloud Notes App now. Would be really cool if all the apps look like one ecosystem in the future!


No - we don't like defaults in this part of the bush :-)


You mean the comment?


put it into .gitignore


I propose to make this a pull request - it seems much cleaner to me.


use a for sentence for this, even a lambda expression. Do not change this.


Cut down on the prose:
```txt:
/* Compiling addDefaultKeyword is heavy, so it's split into multiple files to make it parallelisable. */
```


I think could could ``const auto`` here


don't add unneccesarry whitespace. when reviewing it does not have any meaning.


do not add, commit and push configuration files.


See 1efd68319 for the suggested changes.


Nameless base? :v


This is indeed the new import path, can be confirmed [here](https://github.com/rainycape/vfs/issues/1).


Awesome. Looks good to me!


Since this is a global variable, can we call it something like `ensime--orig-bfn`.

It also means that this will fail if the user starts up two sessions at the same time and moves around.


앙~ 기여어~


I agree. The plan is to make it configurable though. I just haven't had the time to flesh it out (https://github.com/axyr/silverstripe-ideannotator/issues/14)


not working on firefox


Again, could probably use ``cbegin`` and ``cend`` here


actually, can you not do something like

```elisp
`(lambda (reason) (ensime--maybe-update-and-start-noninteractive ,orig-bfn))
```

(note backtick and comma)


This is not needed.. you can use java file class.


```C++
if (write_file)
   update = true;
```
???


Could you please add the command you used for building this?


use 2 spaces for indentation see https://github.com/bbatsov/ruby-style-guide


This is the only change for JDK 1.8 to make the develop branch compile.

The same issue is also in https://github.com/peris/ole/blob/batch-framework-2.1/ole-app/olefs/src/main/java/org/kuali/ole/oleng/service/impl/InvoiceServiceImpl.java


<title>Art World classes</title>
<link rel="stylesheet" href="1.css" type="text/css">

                        Home &nbsp &nbsp &nbsp
                        Gallery&nbsp &nbsp &nbsp &nbsp

                        Courses&nbsp &nbsp &nbsp &nbsp
						Registration&nbsp &nbsp &nbsp

                        About&nbsp &nbsp &nbsp &nbsp

                        Contact&nbsp &nbsp &nbsp &nbsp

Welcome to the World of art!!!!

	&copy art world classes


possible security isssue, one can set arbitrary id


check-out the `Rakefile` if you're interested, it auto-builds the ext or `rake compile` does it


fix the indentation please


Better name:
```C++
const int num_output_files = std::atoi( argv[6] );
```


return is not needed


Huh? It has { }


@merilint  @henrikromanenkov :+1:  loen arvestatuks, koodi vaatasin üle aga endal tööle ei saanud. konfiguratsioonifail ei ole vastavalt nõuetele.


Ok, thanks for the details, Roland!


Je suis assez d'accord, et j'avoue que les mockups ont été faits à l'arrache plus qu'autre chose. Je les retravaillerai dès que j'aurai un peu plus de temps.


SVG Source Code:
```svg
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<svg
   xmlns:svg="http://www.w3.org/2000/svg"
   xmlns="http://www.w3.org/2000/svg"
   enable-background="new 0 0 595.275 311.111"
   xml:space="preserve"
   height="512"
   width="512"
   version="1.1"
   y="0px"
   x="0px"
   id="svg2"
   viewBox="0 0 512 512"><rect
     rx="70"
     ry="70"
     height="448"
     width="448"
     y="32"
     x="32"
     id="rect4"
     style="fill:#1d2d44" /><path
     style="fill:#ffffff"
     d="M 100.92857,89 C 94.320143,89 89,94.320147 89,100.92857 l 0,23.85715 c 0,6.60843 5.320143,11.92856 11.92857,11.92856 l 310.14286,0 c 6.60843,0 11.92857,-5.32013 11.92857,-11.92856 l 0,-23.85715 C 423,94.320147 417.67986,89 411.07143,89 l -310.14286,0 z m 0,95.42857 C 94.320143,184.42857 89,189.74872 89,196.35715 l 0,23.85713 c 0,6.60844 5.320143,11.92857 11.92857,11.92857 l 214.71429,0 c 6.60843,0 11.92857,-5.32013 11.92857,-11.92857 l 0,-23.85713 c 0,-6.60843 -5.32014,-11.92858 -11.92857,-11.92858 l -214.71429,0 z m 0,95.42858 C 94.320143,279.85715 89,285.17729 89,291.78571 l 0,23.85715 c 0,6.60843 5.320143,11.92857 11.92857,11.92857 l 286.28572,0 c 6.60842,0 11.92857,-5.32014 11.92857,-11.92857 l 0,-23.85715 c 0,-6.60842 -5.32015,-11.92856 -11.92857,-11.92856 l -286.28572,0 z m 0,95.42856 C 94.320143,375.28571 89,380.60586 89,387.21429 l 0,23.85714 C 89,417.67986 94.320143,423 100.92857,423 l 143.14286,0 C 250.67986,423 256,417.67986 256,411.07143 l 0,-23.85714 c 0,-6.60843 -5.32014,-11.92858 -11.92857,-11.92858 l -143.14286,0 z"
     id="path4" /></svg>
```


Done.


Prefer colon hash to rocket hash http://stackoverflow.com/questions/10004158/is-hash-rocket-deprecated


If make it configurable, I would allow the user to change the whole format string, so he can decide if he wants a dot or a comma as a separator for example. But this is not really a pressing issue in my opinion. We should rather try to see in what places beancount knows more about the display precision than we do.

Made this change. See #109


Not yet sure whether or not to add a "distinct" here. In case a neighbor cell shares 2 neighbors with the degenerate one, its normal would count twice. But that might be a wanted effect?


If we don't specify a default size, OpenTK crashes......


Can you explain further?


Install ClearLagg :) (Clears stupid entities like fishing things and boats and mine carts!)


I think better would be to use the `fs.FSWatcher`.


we should only call this if we have a `super_message` right? it looks like that's what the previous behavior was


:goat:


Yes that would be nice but it can't be used here. The for cycle here is broken when there is no more items selected so the else branch isn't run here.


@nirvn was the image above generated with layer variables or with editing the expression?


it would be nice if it would render validation errors becase Booking model has validations


retirar as bolas


wat is last news


Can the loops on line 146 and 148 be combined in one loop?


Is this case acceptable? Seems to me no. I propose to add `DHECK(!"Unexpected ApplicationType ");`


Comment by AK


Anyway, I fixed this by using `%p` instead.


I think you can set @listings_near to [] in controller. there is a lot of `if`s in the view


delete


There are other similar cases below. Check them too.


I LOVE THIS TEXT!


@m-kuhn with layer variables which duplicates the variable set by the 2.5D symbology. IMO the symbology should set variables at the layer level so no editing is needed, as per our discussion.


I would suggest to always collect the data in an array and do the `'\n'.join(...)`, you will retain control about where to put extra `'\n'` to separate sections.

But I guess that is OK like it is.


This method have been historically useful for tracking down tests that leave threads lying around, and so tests can use this as a part of thread names, and test utilities did... dunno why they don't anymore. Can we keep it?


This icon is used for the beta only, at the moment... Like I've written in the other issue, I really think we should use the owncloud blue colored icon (for Android and also for iOS) but that decision isn't up to me.


Хм. Не пойму немного, а в чём фикс? Если фрагмент не приаттачен - всё равно идём ниже и там показываем, не? То есть условие Config.getLastWhatsNewVersion() >= BuildConfig.VERSION_CODE вроде не фильтрует на самом деле запуски, не?


This for loop is unnecessary complicated; as long as the iterator and the `i`are running over the same range they do not both need to be updated in the `for`construction - and at the very least the `block` variable should be assigned to in the body of the loop. Readability wins hands down - each time.


The point of that repository is to give people who don't use composer easy access. That's why it's just one file to include.


Remove useless `final` modifier.


@nk0leg I think we have to call `thread_watcher_.cancel()` but not just emit `canceled()` signal


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


Maybe, and I'm fine with them being removed. The elaborate (and somewhat hacky) splitting to get naming right, however, is non-obvious and I added the comment to provide a 'why'.

I can re-submit without the comments if desired.


hi alan :)
dont know where else to write ...
you know gitter? https://gitter.im/
http://www.softwarepreservation.org/projects/LISP - look at the sidebar ;) realy


This was part of the security report, though it is not really a security issue. Feel free to revert all fixes if you find them inappropriate.


safer alternative: `u = urlparse(urljoin(url, iso))`

https://docs.python.org/2/library/urlparse.html#urlparse.urljoin


Comments of this type:

```txt
making development frustrating and inefficient.
```

are of course valid points - but they are not interesting as commit messages.


`OVERRIDE` means that this method is virtual. but virtual key word is missing.


Yes. That file contains everything that *isn't* addDefaultKeyword, so it is still used. The generator outputs n (default 4) files of template instantiations, and 1 file containing "the rest". I found this a reasonable trade-off w.r.t. time spent versus code complexity.


I am not sure about what you suggest:
strncpy do not copy/add zero terminal that is why I add it (http://www.cplusplus.com/reference/cstring/strncpy/)
then, because there is a zero terminal strcat will work and because it add a defined string ("...") final will be with a zero terminal array which is ok
Your code assume strncpy add a zero terminal, which for me it is wrong, so strcat will have an unpredictable behavior, additionally add a zero terminal at the end when strcat is done is actually not necessary because strcat will add it for sure.


Do we really have to create 3 objects to find a partition to write to? It would perhaps be OK to do this once per logical index during a commit, say "give me an IndexWriter that has capacity of adding N items", but not per document...


Wat is de reden hierachter? Het is duidelijk te zien in 24uur accumulaties, en persoonlijk vind ik het jammer omdat deze data juist nog wel representatief is.


```
if (Config.getLastWhatsNewVersion() >= BuildConfig.VERSION_CODE)
```
означает, что мы уже установили флаг "Показать WhatsNew". Если (f != null), значит мы повернули экран, и фрагмент был приаттачен снова. Здесь нужно его удалить и пересоздать заново (из-за проблем с вёрсткой). Если (f == null), значит фрагмент и не создавался до этого, поэтому выходим.


@qaraluch sorry I forgot this, [Maxi](https://dribbble.com/maxialbella) made that


Same goes for me. Not debian, but Ubuntu. But after that the script (tries to) install all required packages, and after that bails out. However, is that when running the script Sickbeard.py from command line (not initscript), it does start. I don't know how to debug :(


Good; I like moving stuff from .hpp to .cpp!


Do you mean the comment (which I sort-of understand, but it's non-obvious why the calculation is so elaborate due to shortcomings in cmake) or the use of a variable?

It took me a while to realise that cmake's foreach was [0,n] which manifested as peculiar compilation errors.


What's the reason to implement all this stuff? Why not to use existing ReverseGeocoder::GetNearbyStreets()?


Look above that... Congig too :P


Ill change it now :P


That is good - these funny typedef constructions should die!


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


It shouldn't.


Has this commit been squashed into another?


You could even have a less garbagy structure like an array, so that you wouldn't need to create an Iterator either... and the `LuceneIndex` could remember the "current" bucket which is most likely to have free space and try that first, otherwise loop through the buckets/create new and remember which one is now the most likely to contain free space, a.s.o.... that should have better performance characteristics.


+ để update cái này thì bỏ dòng
KouTsuneka\Navigation\NavigationServiceProvider::class,
trong config.php

+ dùng composer để cài koutsuneka/navigation

+ cài xong thì thêm lại dòng
KouTsuneka\Navigation\NavigationServiceProvider::class,
vào app.php


Давай добавим коммент зачем вот это добавление повторное из-за траблов с вёрсткой, по коду сложно понять что к чему.


Are you sure, that locking a dictionary is a good way? Why not lock special sync root s_syncLock?


`noti` needs to be passed an argument (text message) sandwiched with double or single quotes.


should be based on verification of expected argument from the methodSymbol associated with the `methodInvocationTree`.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11925](https://teamcity.mesosphere.io/viewLog.html?buildId=107526&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:09:57


They can, but honestly I prefer them not to.

#1: They have completely different responsibilities (146 is wrapping up the file content, 148 is writing to file) and thus makes sense to separate.
#2: It forces 146 to not be a one-line-loop, and to use indexing instead of foreach, which I find far more obvious than jumping to indices.


You didn't use mocks in the exercise. Please modify one of the tests to use a mock.


There are some strange ordering z-index issues which I wonder why they happen


Why is this `using` introduced - I would prefer just `std::shared_ptr< const TimeMap >`?


Class level javadoc


Javadocs


Please allow ardrone2control to be indexed and documented on ros.org!


Javadocs, perhaps explaining the folder layout?


Alyssa and Stephen,

Any ideas on which of the readings we might like to compare our chapter to?

Who will be reading which parts of the presentation in class?


peut on cacher toute la ligne? ce sera plus esthétique


Also possible. I personally like values over conditionals, but if there is a strong project preference then I'm ok with that.


I have removed the line

```
system "noti", "-V", "Agnes", "-t", "''", "-m", "'Thank you for installing noti version 2.'"
```

since Travic-CI was not comfortable with OS X system's VoiceOver being called during the test.


Yeah, you should not `assert_total` here. You really should only use that when interfacing with primitives. I will work on it.


'break' after this line?


I don't disagree, I simply did it this way to not break uniformity in how args were read. Want me to change it?


I don't think this is possible here, notice that we need to pass `{` and `}` as characters, those two will mess with `.format`, not to mention adding more brackets for `.format` messing with the string of dict we want to pass.


这个地方有一点别扭，两个 ： 有点看着别扭，你觉得呢？


OK


@stefan-niedermann nice! Just one thing: The inner note icon radius needs to be adjusted so it matches the outer one. See this post for what I mean: http://usabilitypost.com/2009/01/26/the-proper-way-to-draw-rounded-corners/


This is where i try to fix some configuration widget weights and ajax callbacks. Remove button in table remove last added widget. I'm trying to fix that.


@dkayiwa can you please make the comment you added here be more verbose, and clarify what scenario this is covering?


The comments about cmakes shortcomings! Use of variables is fine.


To be perfectly honest it's a remnant from when I tried to be as little invasive as possible. I personally largely prefer using explicit shared_ptr.


@m-kuhn I can share the full building footprint dataset I'm using here if you want to test things around.


It'd be much better with `Iterable` instead of `List`. Remember that this should an as generic utility as possible.


可是怎么办？或者去掉前面一个冒号我觉得也可以


Voilà.
J'espère que tu voulais cacher tout ce qui était "masqué", et pas seulement le bloc de la création du compte ?


@swatijadhav I've extract the code out of Offer and into the util class as it was needing to refer to Item, Message, Package and Version too. Have updated the code to merge the related objects as needed. Everything is wrapped in a transaction to enable rollback. However, if rollback occurs, push_updates will have already been sent so the client will need to refresh to get actual state. I can't see a way to hold push_updates, can you?


> How are we parallelizing tests these days?

Currently, tests are not being parallelized. This isn't due to any technical limitation. It just happens that I haven't put effort into doing so.


emuNAND 9.6+,sorry,I didn't explain


I have removed the line

```
system "noti", "-V", "Agnes", "-t", "''", "-m", "'Thank you for installing noti version 2.'"
```

since Travic-CI complained that it was not comfortable with OS X system's VoiceOver being called during the test.


Оффф, няма го, сори бях останал с впечатлението, че го имаше в комапата...

On Tue, Jan 19, 2016 at 1:40 PM Miroslav Mironov <notifications@github.com>
wrote:

> В кой клас?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/L3K0V/sofia-swing-guide/commit/2f8c962e9479098045af717a029e8edc72fe8ba9#commitcomment-15527392>
> .
>


The host name of the Linux node will be changed to Enterprise after reboot for SUSE OS if the Enterprise.<hostname> is put before <hostname> in the /etc/hosts file. This issue only happen in SUSE.


You're still using the (invalid) code style of column-style line wrapping


I wouldn't think so. This change should only affect spring-boot apps, where there might be more than one object mapper in play.

This is the ideal behavior in the sense that the message converter that is picked is the first one in the list of converters, consequently the first object mapper. So reversing it prioritizes the object mapper that has first dibs at message conversion.

Do you see a case where that might not be true?


Musimy tutaj zdecydować który z tych trzech wariantów wybieramy:

MPMC::bounded - bezpieczny, ale jak będzie więcej wątków niż rdzeni to może spowalniać
SPMC::bounded_fast - bardzo szybki, ale cały scope z nim musiałby być w unsafe { }
SPMC::unbounded - szybki, ale synchroniczny i może rosnąć w nieskończoność - trzeba samemu obliczać ile requestów zostało w niego wrzuconych

Wg mnie będzie lepiej chyba pozostać przy MPMC i pilnować by nigdy nie było więcej workerów niż rdzeni. Chyba że to się liczy razem z producerami, wtedy 1 producer + 3 workery na 4 rdzeniach. Co o tym myślisz?


`virtual` keyword is redundant in case of `OVERRIDE`.
non-virtual method could not be overridden =)


那样，把第一个冒号改为“-”，如何？


why is the phone number set to 'ewsghana-input-stock'?


Then it should be fixed upstream - i copied the inner note icon from the webapp - https://github.com/owncloud/notes/blob/master/img/notes.svg


That would be nice


Codeception-Tests werden jetzt automatisch von travis ausgeführt :tada:


Hello, in the kernel/core.c
can I change:
-	while (!idle_cpu(cpu))
-		cpu_relax();
to
+	while (!idle_cpu(cpu))
+		cpu_read_relax();
because when use while (!idle_cpu_relaxed(cpu)), there is a error "comparison between pointer and integer" so if I just delete  idle_cpu_relaxed function ,it's ok?


Fair enough.


:+1: sobib


更正安装方法
wget --no-check-certificate https://raw.githubusercontent.com/guochengjie/linux-one-click-install/master/shadowsocks-python.sh
chmod +x shadowsocks-python.sh
./shadowsocks-pyphon.sh 2>&1 | tee shadowsocks.log


Well; I do not about global project preferences but for this particular case I feel the conditional reads much better.


Any good reason to keep it instead of letting it be configured at top level?

The lib is only used to run the code generator, so  its linkage doesn't matter too much. I don't know why it was STATIC in the first place.


If the frontend does not display the captcha challenge, both strings are set to "" or undefined (I think id the former, guess the latter).  If a captcha guess is provided, both are non-empty strings.

If boolsyness of javascript strings works under these circumstances in all browsers, this PR can be merged.


@nitriques, why have you changed this to return the **hash** resp. the **value**? This return value isn't used anywhere in the extension. May I change this?

I am trying to finally fix #28, and I want to:

* remove the `if (empty($data) || !isset($data['value']) || empty($data['value']))` condition (i.e. always run a database update)
* make `compile` return the `$result` of the DB update

My goal is to make the `compile` function do all the work, always. (Just like in the Reflection Field.)


*Update README.md (17th Jan '16)
+16 - 17th January


doh! nice find!


@liewegas why do you add "writes_per_object" here?


oui oui


Remove `final` modifier, rename method to `unstack` and change its type to `ProgramState.Pop` in order to be coherent with other methods from this class.


Je me suis permis de remettre exportations/localisation.php qui n'est pas exploité actuellement mais que je compte utiliser pour des futurs dev chemineur.
Note: on pourrait le réaliser à base de l'API GET/api/polygones si elle réalisait l'intersection des polygone réels et non de leur bounds englobantes.


Unused import


Thanks for the patch. I've been concerned about this warning almost one year.

But this header (backport.h) comes from one of Linux UAPI header. Therefore, it's really hard to change it.
http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/include/uapi/sound/firewire.h

But the idea of using union looks interesting. Let's solve this warning by the other ways.


Yea right! Install ClearLagg for more laa..
A
Aa
Ag.
Sorry i just lagged

> Am 19.01.2016 um 14:06 schrieb Tim (robske_110) <notifications@github.com>:
>
> Install ClearLagg :) (Clears stupid entities like fishing things and boats and mine carts!)
>
> —
> Reply to this email directly or view it on GitHub.
>


Done.


On Squeeze, it is gcc version 4.4.5 (Debian 4.4.5-8).

You *must* avoid doing such code (ie., `for (NSArray *testCase in cases)`). You must also avoid using true/false and use YES/NO.


it's probably a good idea to also include fields for domain and owner_id even if they're not used right now, just so that later on we don't forget that they're missing


I did omit the `std::` for a reason. Use the C function here, since it isn't overloaded. Either plain `isspace` or `::isspace` (if you want to be pedantic) does the job.


Added at https://github.com/openmrs/openmrs-module-uiframework/commit/8050951ceb3650dc94aacd7ff5fba0e9422dbcec


We usually use `x` as parameter name for simple operations like this. Might also cause the `erase`-statement to fit into a single line.


Porque Apagas IIS?


Yes but I can't import it manually for a certain project, the constraint is to use composer to manage all dependencies. What is the problem about that fork would you tell me ?


this should be unauthorized


Forbidden?


requireLoggedIn?


What if there's something that's not win32/linux/darwin?


I prefer that you return int and the user should provide a pointer to size_t * to be filled with the buf size.
in order words,
int sol_http_server_buffer_size(struct sol_http_server *server, size_t *buf_size)


In the original there was no x axis label and I failed to remove the line when I copied the settings block across


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#aUPNYY5JSYusrCjpf2koqQ


111768812_VRFNoLoadOAFlow (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


> requests is used in c2corg_api/security/discourse_sso_provider.py.
>
> Added a parameter to trigger discourse login/logout.

Sorry, my question was not about the requests import. It was more about the whole procedure. When is the `nonce` not available? I thought we would handle the login with the iframe? So why do we need to do this from the server-side?

I would be great if you could add some documentation about the login process.


TeamCity Ppconsul :: Build (OSX) [Build 48](https://78.46.71.115:4433/viewLog.html?buildId=668&buildTypeId=Ppconsul_BuildOsx&guest=1) is now running


I'll change it to ``option_checked``.


哦，不过每次安装到本地，然后遇到问题，改一下再gem build && gem install ? 这样每次改都需要这两部走，是不是很费时间哦？


What for?


I should probably also add something about the colours being dependent on palette settings


TeamCity Ppconsul :: Build (OSX) [Build 48](https://78.46.71.115:4433/viewLog.html?buildId=668&buildTypeId=Ppconsul_BuildOsx&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 96 Build time: 00:03:32
### Failed tests
```
null: global.kv.index: CHECK: 313 > 315
at /private/var/TeamCity_buildAgent/work/b87bef787eb069fe/tests/kv/kv_tests.cpp:697

```


@mimayer - Please fork the repo and make the branches in your fork of the repo.
https://help.github.com/articles/fork-a-repo/

This is the usual workflow on Github, the main repo is just for the `master` branch, as well as stable branches and tags.


No lo he realizado yo de forma intencional..


I'd say yes. Already on it. Cheers!


Changing <%- to <%= would be the simpler way to get it to parse the tags.

We switched to using <%- in most places to prevent malicious code. That's still an issue, so whichever way we inject the content, it needs to go through the HTML sanitizer first, since someone could be using their own client that allows them to put dangerous content in.


@myrsloik says it should be fmUnordered, though


@MagicDroidX
Why it was reverted? What was wrong?


That comment was pre-coffee. You did exactly what I suggested farther down. Good work!


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1199](http://teamcity.monogame.net/viewLog.html?buildId=13781&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:26:11


The passthrough check makes no sense in regard to supports_8ch_pcm, right?


Well yes, the cache caches dependencies we get from Internet, so clearing the cache means every test that requires the same module to be fetched from Internet will have it refetched, resulting in lots of downloads, which slow the tests a lot.


There's also a 'square' image style that could use the same focal point setting


Oops, I did not know the limitation about backport.h. With such a limitation, I have no idea at this time.


@mikkmae @raoulkirsima @arkawa7 veel täiendused puudu. Ootan täna/homme või tagasisidet romil@tlu.ee


fix for #235


Well how do i do an else statement for cmake


It would be more efficient to do this in the model js parse function, so there would only be one change to the DOM instead of two. But that's a bit complicated. This works fine, if I can I'll modify it later.


save+ ?


The c++ version is called #include <cstdio>


Don't worry. So am I :)

Although I believe we can find something better for this issue, I think it better to use our time to the other issues.

Thanks.


Is this seriously needed?


复杂的逻辑用单元测试搞定...


好的


you don't need get "posts/..."
These are covered by resources :posts


bah, copy and paste error - good catch, thanks!


This class should be called `SymbolicValueRelation`.


I believe that it is not necessary.  I originally believed that `onBind` could be called from a Binder thread.  I now understand that, while methods on the returned `Binder` may be called from the Binder thread, `onCreate` and `onBind` are always called from the Main/UI thread.


Latency must be initialized to 0 here


I think we should also check $TERM, and disable colors if it equals "dumb", which is a generally accepted indication to disable all kind of fancy terminal stuff.


What confuses me is, that EAC3 can be output with 2 pcm channels and 192 khz only, so no need for 8 ch pcm: https://github.com/xbmc/xbmc/blob/Helix/xbmc/cores/AudioEngine/Sinks/AESinkALSA.cpp#L113

The 8 ch pcm would only be needed if it's decoded - therefore I don't understand the passthrough settings query.


I change the source code from if (cpu_relaxed_read_long(&rq->curr) != rq->idle) to if (cpu_relaxed_read_long(&rq->curr) != cpu_relaxed_read_long(&rq->idle))
is it correct?


This class should probably be called with something more meaning full regarding its usage, such as `VariableRelation`.


GitHub thinks you were using HTML there...


I don't know Kodi very much so I didn't know under which circumstances this was being called. Since it is something that had sense only if passthrough was enabled I added that check but might be not needed.
Also this has to be modified to target also S805 (initially I thought it was working only on S812 but works good also on S805)


There's quite some other code that also generates color output. Please convert everything at once. In particular grep for OUTPUT_COLOR please, and fix those occurences. Also, please rework the ansi_underline() and similar colors to use colors_enabled().


How does that part work?


If you need this method in editor, please, move it out from ReverseGeocoder. This code confuses me, because GetNearbyStreets() should always return streets ordered by distance from a given point.


BTw, the final if appears very unnecessary. You could write `return parse_boolean(env) != 0;`....


@pestOO, indeed, the `virtual` keyword is required. but it improves the readability. Here is similar discussion http://stackoverflow.com/questions/4895294/c-virtual-keyword-for-functions-in-derived-classes-is-it-necessary .
Regarding `OVERRIDE`. Since we don'r use c++11, this macro should do nothing.


Great, now we can finally delete our forked version. Thanks a lot!


I like the idea of having the doc separate. It is more clear in the matrix what is the cause of the error when a part fail. Ideadlly, I would make this the first part to run (and add test_flake8.py in it), but I don't know if we can order the execution.

 Other then that, this look good.


哦，所有说主要的测试工作还是单元测试做的事吧，跟其他的开发不一样哦，感觉其他的java web, js , ios, android 都很少单元测试，ruby这里就主要还是通过单元测试来run， 是吧


Interesting, thanks.

Also, what's your opinion on making the instance static? That makes the SyncAdapter act as a singleton, which makes sense given the "uniqueness" characteristic of the SyncAdapter


also, note that parse_boolean() actually returns an integer, and parse errors as negative value. It's fine to ignore parse errors here, and consider them the same as the env var unset, but please clarify in the code that this was though off, by not relying on C's automatic-downgrade-to-bool logic, but comparing the result with != 0 explicitly.


accident!!!


touché :)


description....


please, skip this changes


Each class has its respective use. Position is generally used as a point in a level while Location is used by an entity to point out its direction.


Ich glaube das sollte nicht wirklich gelöscht werden, oder?


I will use `::isspace`. Just be sure that the global one is chosen.


This error popped in to trackjs. Easy to reproduce, just ctrl delete a couple of times and you get it


Before, we were using `actions.tap(el).perform()` which was hiding the accessibility error. Now, we get:
```
lementNotAccessibleError: ElementNotAccessibleError: Element is not focusable via the accessibility API: id: option-recents, tagName: A, className: icon icon-recents
  Remote Stack:
  <none>
      at Error.MarionetteError (node_modules/marionette-client/lib/marionette/error.js:105:13)
      at Object.Client._handleCallback (node_modules/marionette-client/lib/marionette/client.js:529:19)
      at Object.<anonymous> (node_modules/marionette-client/lib/marionette/client.js:573:23)
      at TcpSync.sendRaw (node_modules/marionette-client/lib/marionette/drivers/tcp-sync.js:226:10)
      at TcpSync.sendMessage (node_modules/marionette-client/lib/marionette/drivers/tcp-sync.js:209:15)
      at TcpSync.send (node_modules/marionette-client/lib/marionette/drivers/tcp-sync.js:190:17)
      at Object.send (node_modules/marionette-client/lib/marionette/client.js:492:34)
      at Object.Client._sendCommand (node_modules/marionette-client/lib/marionette/client.js:554:21)
      at Object.Element._sendCommand (node_modules/marionette-client/lib/marionette/element.js:51:32)
      at Object.click (node_modules/marionette-client/lib/marionette/element.js:176:19)
      at Object.openCallLogTab (apps/communications/dialer/test/marionette/lib/dialer.js:82:8)
      at Context.<anonymous> (apps/communications/dialer/test/marionette/call_log_test.js:25:22)
      at node_modules/marionette-js-runner/node_modules/promise/lib/core.js:33:15
      at flush (node_modules/marionette-js-runner/node_modules/promise/node_modules/asap/asap.js:27:13)
```


My question basically is: You want to output 8 channel PCM, e.g. decoded dd+ right? If that's the case, you can signal passthrough = false. As the Sink itself only announces AC3 / DTS - it is not called with EAC3 for passthrough anyways, but just with 7.1 decoded audio - so if the pcm speaker layout is 7.1 you will get all dd+ channels.


Are these files good formatted? The indents are just a space


@jmcphers, could you please review this commit?

This is intended to address the fact that when multiple users work actively in a directory they will clobber each other's .RData and .Rhistory files. Note that this possibility doesn't apply exclusively to project sharing in RSP as users sharing projects e.g. over a LAN have the same exposure. We currently have the following KB article that provides guidance on how to handle this:

https://support.rstudio.com/hc/en-us/articles/201751487-Sharing-a-Project-with-multiple-users

This advocates creating a directory named "userdata" and then redirecting the saving and loading of .RData and .Rhistory into that directory.

Now that we are making project sharing much easier (and therefore more common) in RSP I think we need to do more than just hope people find this article. This commit basically promotes the advice in the KB article to built in default behavior when a directory named ".Ruserdata" exists at the root of the project directory (I didn't use "userdata" both to not conflict with installations already using that idiom and because I think using a ".R" prefaced name is more fitting).

There will be a corresponding RSP change to auto-create the .Ruserdata directory when project sharing is active which will trigger this logic to run for all shared projects.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1225.d9f342b85073859d68b6b0b813139affebb86dbb](http://build.palaso.org/viewLog.html?buildId=68043&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 839, ignored: 7 Build time: 00:13:06


不好意思 有点晚了. 之前各个人贡献的时候样式一直不统一. 有的是加粗有的是斜体. 我会抽空把所有的样式统一起来.


Dude, it's 2016. Where did you dig up this old-fashioned function signature?


it really makes it easier to read the tests if you indent them so that pipes alight vertically. textmate lets you do this automatically, i would guess other editors do as well?


It is a very small optimization which, as you have demonstrated, may well not be worth it.

I will, btw, be pulling your code, later today.


makes no sense


cant see any image !!!onene


Das war im alten oauth_keys file so, mit dem aktuellen tuts das nich..
war also eig richtig so oder?


LGTM


Vorschlag:
Vielleicht die Variable umbenennen in sowas wie fnc_extract_objId_from_array.
Dann weiß man beim lesen des Namens schon, was da drin passiert.


Seems the text.js part of the code is quite buggy. No time to looking into fixing it yet, so disabling $ to trigger it and adding some try catches, that seem to get patch you around the errors when deleting yourself out of a text node.


ApplyRFF doesn't seem to modify the shared data, thus fmParallel should be fine. Unless there is some interaction with nfMakeLinear?


Hi @puckett15 the value here can't be hardcoded. It should be dynamic according to each environment.

`aHR0cDovL2xvY2FsaG9zdDo4ODg4L2pvb21sYTEvaW5kZXgucGhw` is `http://localhost:8888/joomla1/index.php`

Despite of the fact it still works in my tests, it would be better to echo the base url dynamically:

```
echo base64_encode(JRoute::_('index.php'));
```


this needs to made an option passed into the page-reducer-factory


:fireworks:


would it be a bit friendlier to say `calendar week` here?


What is the point of that feature? Why should the deck builder let you add more cards than allowed only to remove some at random afterwards?


Done!


111768812_VRFNoLoadOAFlow (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 tests passed, 0 test warnings)

![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-26.85%25-green.svg)


111768812_VRFNoLoadOAFlow (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2088 of 2088 tests passed, 0 test warnings)

![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)


Clang tidy suggest if(&gl...) no clue if it's correct or needed


I am confused about how params be shared. Do params with same name have different Blobs? If that's true, How could the params with the same param name be synchronized? Because in `net.update()`, only the params in `learnable_params_` would be updated. @jeffdonahue


Yes I want to have DD+ 7.1 decoded as 8ch PCM, right now the 8ch track is detected as 6ch so you miss the extra 2 channels like hdmkv reported.
But I'd like to find a way to detect if DD+ is 5.1 or 7.1 for good instead of just setting an 8ch track when Kodi reports a DD+ 6ch track (because it might be a real DD+ 5.1, not seen many of them but it might be a good one, not just a wrong detection).


30 == AE_FMT_FLOATP

which is just the decoded audio format the engine decodes dd+ to. You report back to engine: AE_FMT_16LE later on and audio engine will therefore use sws_resample to transfer the floatp to your 16LE format.

This workaround will also be hit with TrueHD! btw.  and also for DTS-HD when decoded with libdcadec.

So yes, after reading the code it's a general 7.1 pcm hack, which you could incorporate into the channel layout estimation if aml_supports_8ch_pcm is true else use a 5.1 layout so the engine will downmix for you.


Because we need 2 different methods:
1. To get building address close to given coordinates.
2. To get feature's address (it can be empty if nobody specified it), to display and edit in the Editor.


sdfesfesfesfe


@hamidsafdari چرا دیتابیس؟


It's still needed (libtool 2.4.6) if you want shared d2vsource to link static ffmpeg (or any other libs).


I don't think it's a good idea to delete GA.. It's better to let it commented, no?


Two spaces, but doesn't matter. Diff is suppressed anyway. Don't touch autogenerated code!


Ja, hast recht. Hatte ich wohl noch das alte Keyfile. Verrückte Welt.


Can you give me a debuglog of the sink enumeration? If you only enumerate 6 channels - AE won't call you with 8 channels.

You test with 6 speakers, right? Cause when returning 7.1 for given 6 channels just silence for those speakers are added. A debuglog will tell


We can do this with dynamic.   Let me put a script together.  There, we get SIMD, and OMP (but OMP is not really efficient)


This is not required. It is wrong that headers are included in the sources any ways. The depends are determined by gcc -MM. So only .c files are put into the sources.


Thanks again for pushing this out :smile:


A basic version of the BranchMonkey which doesn't utilize dialog.


第一个GitHub   Demo。


To anyone who sees this---
@TheStrix pointed out about this commit. Thanks to him. (From next time, I'll take care of authorship) Although, I never thought that ROM-Devs look into my source too. :P :D

By the way, it is really sad to see that some people on Yu Forums don't even update their source. :( This behaviour of them doesn't promote open-source development. :/


Wecome marshrom 1.9.7 :D


You should change your naming conventions - we don't use 'm_' prefixes for class members


Just a minor tip. Instead of print only the token, you can print all the field using `echo JHtml::_('form.token');`


html-proofer FTW!


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


@flavour Please review


Scheint leider nicht richtig zu funktionieren. Rawgit hat das script zwar übernommen, aber trz immernoch gleich aussehend wie vorher :) @eaglefsd @lostdesign
![fireshot pro screen capture 003 - leitstellenspiel_de - baue deine eigene leitstelle in deiner stadt - www_leitstellenspiel_de_](https://cloud.githubusercontent.com/assets/1109899/12420676/59c99f56-bebe-11e5-802e-79f0c2243ad3.png)


This could already be `subscription = Observable.fromHistory(history)` so maybe it's better to call it `historySubscription` ? Any name suggestion ideas?


At the risk of being annoying or nitpicky, it appears to me that assigning role="menu" to dropdowns is also not right. The menu role seems to be aimed more at a web application widget rather than a normal navigation menu containing a list of links. See https://www.w3.org/WAI/PF/aria-practices/#menu

"A menu type of widget that offers the user a list of choices. It is often a list of links to important sections of a document or a site. The menu role is appropriate when the list of links is presented in a manner similar to a menu on a desktop application. A menubar is also considered a form of menu. A menu should manage the focus of its descendants."

Here is a comment by Steve Faulkner on the subject:

"if the navigation consists of just links on on a web page and does not have the interaction behaviour associated with a menu and menu items then the ARIA roles should not be used"


for good style the trailing "?>" is omitted on php files :)


I recall having to do it, or the build won't work.
Could have been an issue due to something else, too, I'll have a look in some minutes.


Oh, just copy-pasted. Of course it should be `auto fill(...) -> void`.


815，818行多声明了一次api，还有这method名字还蛮长的，哈哈。慢慢打补丁@SuperZDev


no need to assign `start: start`, just doing `start` would expand to the assignment.


These 2 lines under '3D terrain noise'.


The old tricks are the best tricks :smile:


Yeah, absolutely! I broke this link on purpose just to check if it was still running after the changes I made to our travis.yml file. Everything is working fine now, we can deploy to prod using `npm run deploy` :D


Vertical alignment.


چون مرتب کردن اسم شهرها و کشورها آسونتر میشه. فیلتر کردنش هم آسونتره. فیلترش رو هم اضافه میکنم.


Are you sure this move was good? Now you both depend on each other via (deprecated) peerDependencies clause. It is true that the purpose of your two modules overlap, but this one could exist on its own. With this, you are making upgrades, setups etc. a bit of a PITA.

Or, if you are embracing each other, aren't you planning merge into one module? Would make things easier (for you probably as well, and for users surely).


This code confuses you because it was a try to work-around the missing bool "this feature definitely has specific address in the data, not just an approximate one". I can add TODO for @vng or you to fix it when you implement this bool in the new-search branch.

ReverseGeocoder is the best place for this code, because it uses search implementation details.


Помимо изменений в ТЗ, исправлено движение фона и танков


Ping @oney.


It's needed here: https://github.com/dimagi/commcare-hq/blob/master/corehq/apps/commtrack/sms.py#L290. It's current behavior to set ```ewsghana-input-stock``` as a phone number for EWS web submission. It's not possible to leave field phone_number empty.


@wsargent Which discussion?


This comment is related to code below which guarantees it.


Does this actually generate templated results including newline characters?  In my testing Make silently stripped out these characters on variables, by design.


阿萨达


Beautiful tests. I learn a lot by reading your code. :+1:


试试水


I am very interested in this three password thing here.


@Suchty112 das dann bitte hier posten: https://github.com/lostdesign/lss-layout-manager

Hier im Script wurden lediglich zwei IDs angepasst, damit @lostdesign sein Script besser auf meins anpassen kann


Yay thanks! There was a typo (filter_gy, instead of filter_by).

I just pulled in production in Blender.Today and it works fine. Tested updating the favicon (we also had it wrong).


@fichtner good point, I will remove that one too.


@svanoort The templated results are still as broken. IIUC this is not a new issue - just something we found out when we switched to branding.py with more error handling.  (comment in the PR!)


looks ok


Yes, I mean the reference in the XXX comments two lines above. GitHub doesn't allow to comment in the exact line.


@Shnoulle I added this modification


Correct


I think CU_ASSERT_EQUAL_FATAL() usage would be a little cleaner, rather than the CU_ASSERT_EQUAL() + if (ret < 0) return.
Sorry about the late feedback.


unfortunately, this patch disables the correct error-handling for btrfs-receive: if "btrfs send" fails, "btrfs receive" returns 0! so we need to parse the output...

Will try to fix this correctly (catching stderr in perl is pretty painful I'm afraid...)


Clever.


I had done this test back in december when I was trying to figure out why the DD+ track was hanging the box, the system -> audio configuration was set-up as 7.1 with passthrough enabled.
I will post a debug log while playing an 8ch

http://pastebin.com/raw/kmbd6hSw

it says "channels: 6" but should have been 8


You should avoid doing work in a constructor.
it would be better if you have to invoke a new function after the creation of the `User` instance
```java
User usr = new User('./user.txt')
user.load();
```


I'm fine if that was part of the security report.


@hamidsafdari چرا کاستوم دیالوگ اضافه کردی؟


Just checked, running `autoreconf -i -f` or `./autogen.sh` automatically adds `endianness.h` to `Makefile.in`.

If that is not the desired behavior there is something that you have to change.


please add two spaces


libvlc is missing.


will be merged, but without the -e flag (ETA  timer), which does not make sense here.


Which? :)


comecaria a ensinar primeiro sem pacote. depois no fim colocaria jogos. e depois o pacote inteiro. (pode deixar assim, eh soh uma anotacao de dica de como ensinar no livro)


Can we make this arcade drive instead because it's easier to use.


Does `import * as DOMEvent from '../src/dom-event';` work?


boa. comecaria dentro do metodo, depois tiraria


Anything you can do to help with Linux would be greatly appreciated.  We only have two steering committee members right now and neither of us have access to a Linux box.  If this fixes the make file I would gladly merge it into the main code base.  Just don't include the text file with the build errors in the merge request.

Thanks for making things better!


This need to invert the joystick values b the motors are installed backwards


podemos ensinar um extract method. acho que vale a pena. no livro de ruby tambem. pode ficar pra depois no livro tambem


Yo,
What was the need of keeping that comment alive including a rename ?


Math.random() ou new Random() eis a duvida cruel...


I think there shouldn't be a space between `50` and `%` in the warning message. Same for two lines under this one.


@thomasquintana No it was already in there and still is


Ja ensinamos o new por causa do scanner. entao new Random()


vamos ter q ensinar primeiro que o tostring ficou zoado. depois arrays, blz


faltou chaves


Yes! Forgot about that syntax


vo uperguntar no instrutores o que acham de ensinarmos arraylist antes de ensinar array


Only d2vsource has to be fmUnordered.
applyrff can be fmParallell
There's no interaction with nfMakeLinear ever. Apart from the fact that it makes no real sense to use it except on fmUnordered (and fmSerial, but using fmSerial means your code is teh fail).


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Blizzard mistake we will not reproduce their bugs.


Yeah, I will.


test


Looks great for a first version!


It seemed incorrect,, since the allergyui model relies on the allergyapi module as far as I can tell... or am I missing something?


They are not really _immediately_ routed to the actor, are they? They must not be consumed before recovery ends, similar to commands. But this section seems to contrast event handling with command handling during recovery.


V rámci zkvalitňování našich služeb bla, bla, bla.... bez komentu není merge :)

Můžeš ještě opravit formátování v index.html:23?

Díky, Č.


什么鬼，，用例挂了。。你搞两分支干嘛 。。那分支没删 啊。。。


In postgreSQL is not allowing **system identifier as column name** while creating a new table (For example column name : **user** ). So we're using like ** "mytable"."mycolumn" **


comment


Couple of thoughts:

- Perhaps you want to put the steps into a separate file, so that they can be edited more easily by non-developers?
- What about translating the steps at a later point?
- If we keep in mind that we might create such a tour for other pages as well, which might reside in core or modules, how should we make this more generic so that modules can make use of it?
- Not sure, it's good to have a dedicated TWIG template which includes the tour. Perhaps it's better to use the same template, and work with conditional includes/content?


@jmcphers Here is the corresponding RSP commit: https://github.com/rstudio/rstudio-pro/commit/3a1542db14992d86311796ed25d2f0961bdbad08


Great!


Indicate as JavaAPI? Here and `onRecoveryFailure`?


My logic was in an attack, such as a brute force, the hacker gets x number of attempts to crack the password, here the user gets 3 attempts till the system is locked. The hacker also gets three attempts but never at the same password. In my thinking this makes it more difficult to break the users password as you only get one chance at each. Not sure if it is practical or not yet, I have to do more research into how actual password programs are written. Not just what my mind creates.


I should probably make this an:
```perl
else {
    if ($major == 4 and $minor == 0) { ... }
    else { ... }
}
```


Traditional exchanges charge user a percentage based fee when the user's order get filled, and sometimes there is a bottom line for the order fill fee. (need examples here)
Since currently in BitShares platform the order fill fee is 0, the order_creation fee is more or less taking the role of fill order fee.


Maybe it is possible to use named pipes. That way you could run ```btrfs send```, ```pv``` and ```btrfs receive``` independently from each other and correctly check for return values.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11926](https://teamcity.mesosphere.io/viewLog.html?buildId=107569&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 214 Build time: 00:09:00


We moved the allergyapi into platforms 2.0. So it is no longer required.


should be
```perl
if ($major == 4 and $minor == 0) {
    ...
}
```


Is `id` missing quotes?


It looks like the fix works with chef-solo 12.5.1! Many thanks!


This should be modified with the correct user device


Pretty :)


Ah okay, but I remember now... it was *not* working on with "aware of" when running in a 1.10.x environment... was stack tracing when trying to use the Allergy Service... maybe an issue with the way the conditional resources were declared?


I don't remember the actual error, but if you want more info, we can switch it back and I can test.


corrected by commit 61fff80a05b31c1c17edec06223064d2715c80f7


That is a creative solution. It provides an obfuscation of protocol. It makes it more secure unless the attacker knows this protocol is in place.


it is useless commit


see e6f938d


@malirod
It doesn't matter is the `OVERRIDE` working (and it's [unfortunately  working](https://github.com/CustomSDL/sdl_panasonic/blob/7c62f130800404f902f8828ad6df281018d16f24/CMakeLists.txt#L187)).
The 'magic' sense of `virtual` in c++90 is to show that the method is virtual -in case of inheritance, it helps to easier review interface.
In c++11 we get a `override` and it we get a new more powerful tool - it shows as that the method is virtual (non-virtual could not be override) and verify overriding (the main responsibility).


This disables compilation on OS X.


Sure go ahead! :+1:


OK, my bad then. :)


I would like to have at least some explanation regarding this change in the commit message, even for the top part.


Nice to see the shorter syntax! I would probably just merge this commit into the last one, since it does not add any new functionality


Yay!

> On Jan 19, 2016, at 12:31 AM, Sarah Mei <notifications@github.com> wrote:
>
> Add a contributor code of conduct. Thx @coralineada :)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sarahmei/embloggen/commit/608ac983ffc54e138f0268c50fe822bc82604a3e


Will add it to my to-dos for the day, will keep you posted!


Please fix the time stamp properly before commit!
Try following command on Zedboard
date MMDDhhmmYY
ex) $ date 011923302016

In addition, you should configure email address and your name.
$ git config --global user.name "HItomi Kimura"
$ git config --global user.email hitomin@virgo.is.utsunomiya-u.ac.jp


We should check the password is null or not, otherwise we get NullPointerException.
We should create new kind of required  option decorator (e.g.: RequiredBy)
```
options.put(new RequiredBy(new StringOption(owner, SHIELD_KEYSTORE_PWD), SHIELD_KEYSTORE_PATH));
```


Thanks! :)


Right in both cases. Maybe I just turned my brain off when it working on the Java API :stuck_out_tongue_winking_eye:


This commit is a part of master branch - https://github.com/synfig/synfig/commit/044bb928ed0789d328aaad9649e4d183aaa4522a


Thanks for testing.


as we have the if [ ! -d ... ] block its safe to run this command?


Yes, that's right. Sorry for the noise.


They *are* immediately routed to that actor but stashed by the actor until recovery completes. Maybe I should remove *immediately* here to avoid confusion.


@pestOO, I agree that `virtually` is technically redundant. But seems it comes in front of the method, it makes more readability.


@ErwanAliasr1 i didn't get a chance to look at it closely. but you are right, they are not needed. and probably we don't need libedit at all!


@emmenko


Spelling error?


OK, sure.


Or add that they are stashed?!? Just like the next sentence mentions this about commands handling?


به این خاطر که بشه اسم کشورها رو کمی کم رنگ تر کرد. فکر کردم بعدا فیلتر هم بندازم بدون  کاستوم دیالوگ نمیشد. اما اینستنسش رو سیو نمی کنم. چون ساده است چیزی ریستور نمیشه.


111768812_VRFNoLoadOAFlow (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2056 of 2056 tests passed, 0 test warnings)

![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)


`colour` here and `COLOR` two lines below?


Close - if you would have put @wiki = current_user.wikis.new(wiki_params), you could have eliminated lines 21 and 22. You are leveraging the wikis relationship with current_user here, not the @wiki itself, which is why this didn't work.


Yeah, I'll fix it. Wasn't causing any problems though :surfer:


Could you leave a comment that disabled coords mean that this BQ is for synonyms?


Shouldn't it be `treeModel.PositionStickiness`?


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


@morphyno Yes, this is Cisco Proprietary code, which means that Cisco owns it. However, it is published under permissive open source license.


I thought we were going to have a Checklist per Incident Type not per Incident?


This should do the trick to make sure we're always using py2 @dangra


The purpose of ModelNameType is to differentiate the id/name of the model and also re-using "TypeNameExtractor".  I wasn't sure if this was appropriate but guessing form your response it probably isn't.

One thing that might help me troubleshoot this is seeing the end result swagger page.  Is there something in the project I can run and browse, so I can see how my changes effect the end result?  I understand I should be doing this with unit tests but I am not sure if I am even modifying the right stuff.

The way I was thinking this would work is if DefaultModelProvider uses the class name for the "id", it can treat that model as unique even though the modelName is the same.  I didn't make the code change but I think DefaultModelProvider.dependencies would need to use the "id" instead of the name when putting the model into the map.


@djazayeri removed groovy because it is not used, @dkayiwa was asking the mysql mxj connector, if only the standalone uses it i would assume it should be a dependency in the standalone project and not core


ok


Ok, will do.


I suspect you'll want an 'order' field to be able to ensure a specific order


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


FTR, 0.3.8 is published


Why is this a separate field?
Am not sure what exactly you are storing here...but as I imagine a checklist, most entries wouldn't have a numeric quantity attached?


Complete Log, please I need the enumeration.


There already is a format to handle this (dynamic)

```
[List.Generic:dynamic_1029]
Expression=sha256($p) (hash truncated to length 32)
Flag=MGF_FLAT_BUFFERS
MaxInputLen=110
MaxInputLenX86=110
Func=DynamicFunc__clean_input
Func=DynamicFunc__append_keys
Func=DynamicFunc__SHA256_crypt_input1_to_output1_FINAL
Test=$dynamic_1029$e4ad93ca07acb8d908a3aa41e920ea4f:iloveyou
Test=$dynamic_1029$13b1f7ec5beaefc781e43a3b344371cd:freedom
Test=$dynamic_1029$aa97302150fce811425cd84537028a5a:computer
Test=$dynamic_1029$75ff6bea5b0ad25171988e435c24b3ee:12345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890
```

This will crack the hashes without adding the 00000000 to the end of the 'string'.


Graphs might work since we already have some thoughts on it. Should we try and use one of the secondary readings instead or also?

The time constraints make talking tricky since we'd each be able to talk for only a minute or two. I think that's fine, but we'll have to be very specific about what we want to say. Personally, I'd like to do something with the analysis/summary of Maps, but I'd be fine with the comparison too.


@javierjeronimo, esto es un intento de hacer que el contenedor sea más genérico y sirva para (casi) cualquier tipo de aplicación PHP, no sólo para WordPress. Lo vemos en persona.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11927](https://teamcity.mesosphere.io/viewLog.html?buildId=107576&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:04:52


Ok, seems like thread terminating works fine, but I'm not sure about `ExitThread` functionality. ExitThread signal should abort thread but you are still waiting for thread ending with `deleteLater`. I think it's incorrect behavior


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#J_0abgXFSdihOOJh_xsKFw


"Description", perhaps?


It looks like they already are in drive(Joystick)


Cool to see you are working on this already :+1 Thanks!


You should include the original author's name in ```bower.json``` I believe.


@tlaitinen From this commit onward we get an error in Handler.Api.Internal, "No instance for (ToJSON Day)".  I am not really sure why we aren't given access to the instance because we need it.


De reden is dat er dichtbij de radars veel ground clutter is te zien. Deze pixels worden dus van meet af aan als onbetrouwbaar gemarkeerd, net als overigens de extra pixels die in het 'clutterfilter' gemarkeerd zijn. Voor die pixels wordt dan ook altijd de data van de (verticaal gezien) dichtstbijzijnde radar gebruikt. De gemeten waarden daarvan kunnen dus best wel afwijken.


Do we have some analogue of `ExitThread` in Qt?


I will get it later tonight and will post it ;)


Okay.


Shouldn't have this one and few others been `STANDARD_ERROR_CODE [:card_declined]`?

The meaning is quite different when we try to give feedback to the user.


I think that we are going to need to have this be in another table.
i.e. for each Incident Type there is a Checklist with multiple Entries.
When an Incident occurs we need multiple people to fill this in - probably 1 per affected location.
We have the usual pair of options:
(A) An is_template field in the event_checklist / event_checklist_entry
(B) Separate tables for templates & instances


I am okay with the merge of these two modules.


Yeah, need to add some physics to them


I'm sorry to insist, but this method is available only with JDK 1.7. I know it's a bit of a shame to be restricted to such old Java versions as 1.6, but that's our constraint in this project and we currently don't have the capacity to change that, and maintain two parallel versions...


"Entries" but I think that a CustomForm with InlineComponents will make for better UX...


You got me wrong. ^.^
There isn't any problem, I am asking the composer.json file.

```json
{
    "target-dir": "Jrk/LiveCodingAuth",
    "extra": {
        "branch-alias": {
            "dev-master": "1.0-dev"
        }
    }
}
```

These seemed a bit strange, never seen them before. For what did you add those configurations and why is the `"symfony/framework-bundle": "2.*"` dependency.


I already thought of this, but in my opinion this would make it even more complicated (creating temp files, more signal handlers, etc.). I tried to do it "correctly" using open3 in the (run_cmd_open3 branch)[https://github.com/digint/btrbk/tree/run_cmd_open3], but this opened other issues (and unreadable/untraceable code) as well so I discontinued this.

I'm solving this with additional file descriptors in the shell now, this should also work. Something like this should work:
```
{ btrfs send <src> 2>&3 | pv -trab | btrfs receive <dst> 2>&3 ; } 3>&1
```


Oh oh.. so the thing existed there..
There is a checkbox "featured"
If its checked featured maps are shown (which is default and makes sense to use order)
If not its latest first...
Hmm perhaps we should only change title to reflect that instead of this?


@mayya-sharipova @ykye could you please have a look at my changes to the README? Are you both ok with the details provided here? Thanks


Change name


Looks good.


The program stops at the password_verify(). But in the normal login (not ACP) it works.


Hi Feng
i am ready with my changes
but i have problem
after i put traffic to IMQ mashine stop work after remove iptables rull:

#-A POSTROUTING -o bond+ -j IMQ --todev 0
#-A POSTROUTING -o br0+ -j IMQ --todev 1

Mashine start and run fine and transfer traffic

may be need to change iptables patch for new iptables and new kernel 4.4

Please check this


delete


Alternate tables would be: event_checklist_template & event_checklist_template_entry


Sorry, it was the not-sure, yes, no bit that's stored in the database that I was referring to.


Lets set |ret| properly, and fall through to stop_publishing() below. Otherwise this will just fail in a loop.


Oups it's already merged. I'll commit directly in master.


new database


Typo, `does not exist`


Thanks.


Hehe cool :)


Why are `left` and `right` left behind, shouldn't the part of the `dispatch` dictionary in `_column_action_handler`?


Should be "x-oauth2" ;)


Hi @whatyouhide, I saw this test failing few times within travis.ci - What was the reason why you changed this message?


@cyberhobo - that looks fine, you can go on and release as far as WPML is concerned. If you want you can add me to this repo so we can chat through here :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 11928](https://teamcity.mesosphere.io/viewLog.html?buildId=107585&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 213 Build time: 00:04:02


Good catch. But return(foo) in C++ evaluates to return foo. So, it works
either way.

On Tue, Jan 19, 2016 at 12:00 AM, Jonah Gabry <notifications@github.com>
wrote:

> fix return statements
>
> @bgoodri how have these been working?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/stan-dev/rstanarm/commit/adfac562296f83f347016e3cd9d5d8bab0235cba
>


ok
نمیدونم الان دقیقا چه جوری هست
ولی اگر تعداد شهر‌ها و کشور‌ها زیاد شده شاید بهتر باشه انتخاب کشور رو جدا گانه بزاریم

مورد بعد منم سعی میکنم از این هفته شروع کنم کارای باقی مونده رو تموم کنیم که تا قبل از اسفند برنامه تموم شه و برای سال جدید بتونیم آپدیت بدیم
با هم هماهنگ باشیم و زود به زود پوش کنیم که به کانفیلیکت نخوریم


You could instead use destroy_all from those associations. This would generally be more efficient and still fire callbacks and lifecycle hooks.


funktion gehört noch implementiert


I'm really sorry for intrusion and commenting in your private repo, but do you think you could add a compiled `.jar` version with this fix to the repo? Thank you very much in advance.


I removed this. It's not used.


Wow!


Fixed


Fixed.


حرف خوبیه


This helps, but doesn't fully solve the problem.  If for example any of the input arguments contain whitespace, at best it could break.

The only really correct way to do subprocesses is to ensure all of the input is built into an argument array from the start.  Unfortunately, we have stringified representation in the labels, but we could split those on whitespace first, then append.


ouch, sry, will fix after your review (or in v2)


Tak jak Pan mówił na zajęciach, przeniosłam wszystko do modelu. Wydaje mi się, że wszystko działa, tzn. nie wyskakują błędy, a recenzja dodaje się tylko jeśli osoba jest z innego instytutu. Ale problem pojawia się, kiedy osoby są z tej samej instytucji. Recenzja co prawda się nie dodaje, ale zamiast tego formularz wyświetla się ponownie, wypełniony wybranym przez nas recenzentem (tym złym) bez możliwości zmiany.

Zastanawiam się, jak rozwiązac ten problem. Może powinnam zrobić coś jeszcze z

  if !shared_institutions.empty?
  + errors.add(:person,"'#{person.full_name}' has the same affiliation as authors.")
  + end

w pliku app/models/review.rb (podczas walidacji)?


Oh sorry. I'm copy-pasting from my skeleton sf2 bundle (https://github.com/jreziga/JRKFooBundle).

Target-dir is the the directory where all the file from my repository will be inserted in that case it will be 'vendors/jrk/livecoding-auth\JRK\LiveCodingAuth' useful for namespace understanding.

I'm using the extra option for naming my branch through aliases.

Indeed the dependency for symfony is useless but I didn't finished the integration, there will be more classes, I have no time for the moment !


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#Ff1XTN7fQUmyBowh31EWlQ


# THIS IS A COMMENT


- We can put the steps into a separate .js or .json file, but with twig we can later use translation features
- The twig template only inherits the original index template and adds the necessary JS/CSS in the right places. There is no duplicate content.
We could use conditionals but then they will be parsed on every request even though we only need all this stuff a single time on the users first login.
- I'm not sure how make it more generic, as any tour is individual. What is happening in index_with_intro.html.twig are the minimum changes required to implement a tour:
 - add JS/CSS references
 - define step order, position and content
 - define extra JS to handle special features like opening a dropdown
Do you have any idea in mind?


No, `key` is meant to be a unique value that includes the ID of the selected candidate or committee, like "C00431445-checkbox".


Great job @modsmoss :+1:


آقا گیتر رو برای همین حرفا درست کردن
بعضی حرفا به این کامیت ربط نداره بهتره اونجا زده بشن
من ریدمی رو تغییر دادم که لینک گیتر توش باشه
فاصله بین لیست ها رو هم حذف کردم شما ببینید

https://gist.github.com/hamidsafdari/1ed6532d02a78abcc1f6


@nyalldawson @nirvn
Does this also fix [Redmine #13972](http://hub.qgis.org/issues/13972) _Categorize symbols uses wrong field after table join_?


Unless someone plans to make a list of at least the top 10-20 maps by quality, I think there probably isn't any real purpose to use the featured order (I can't discern any particular pattern to it atm).

Always sorting (effectively) by date has the bonus of keeping the list "fresh" as new good maps are added. We don't have to worry about bad maps appearing there any more since the featured/supported distinction.


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (915 of 915 tests passed, 0 test warnings)

![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-26.1%25-green.svg)


TeamCity Jarvis Framework :: CI-GitFlow [Build 82](http://build.prxm.it:8811/viewLog.html?buildId=5587&buildTypeId=CqrsFramework_CiGitFlow) is now running


Just to avoid rounding errors incompletely writing out the final object.


@Josh-G @CHAZICLE  Can one of you check if this app works properly for you?


sarahs changes


I think @LinusU's proposition makes sense. Otherwise we will have to copy the arguments validation logic from V8's implementation of subarray if we use the Uint8Array constructor: https://github.com/targos/node/commit/a04e46352f040d13b03f2d1319b05622da27800d


Hey @jreback is this warning OK?


Woaw, nice! I missed a $ there... Sorry!


![kepernyokep 2016-01-19 15 58 42](https://cloud.githubusercontent.com/assets/5840038/12422115/93d6ae80-bec5-11e5-918f-5981c0ffcae5.png)


Correct@AcadGild


Ohh, that `target-dir` option is actually pretty cool. :+1: Thanks!
I will try to implement this to my packages because currently everything is in `vendor/ozankurt/package/` but I use `Kurt\Package\` as the namespace. Do you think it's fine?

Btw. If you want you can also check my [LiveCoding-API](https://github.com/OzanKurt/LiveCoding-API) which is a bit more object oriented. ♥


finish README


whole file suppressions is first point to avoid.

http://checkstyle.sourceforge.net/config_misc.html#AvoidEscapedUnicodeCharacters
this Check have bunch of options it is better to use them. Default config was used only because we added it when we force all Checks to exists in config file.


__API Description file syntax error on line 3155__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


there seems to be an extra "WU" here...


 login " : " telegramshaman " ,
     " id " : 16778864 ,
     " url " : " https://api.github.com/orgs/telegramshaman " ,
     " repos_url " : " https://api.github.com/orgs/telegramshaman/repos " ,
     " events_url " : " https://api.github.com/orgs/telegramshaman/events " ,
     " members_url " : " https://api.github.com/orgs/telegramshaman/members{/member} " ,
     " public_members_url " : " https://api.github.com/orgs/telegramshaman/public_members{/member} " ,
     " avatar_url " : " https://avatars.githubusercontent.com/u/16778864?v=3 " ,
     " description " : null


### UPDATE: this is a non-working commit. See https://github.com/geddski/csstyle/pull/64 instead


![kepernyokep 2016-01-19 16 00 21](https://cloud.githubusercontent.com/assets/5840038/12422160/cb2775fe-bec5-11e5-9f49-2acc10a26f05.png)


commit in Russian? :)


Est-ce qu'on n'avait pas pris l'habitude d'écrire "unsigned int" partout ailleurs ?


With `delete` having a different meaning in the terminology for an instance, and given how simple this solution is (despite not being just the default), I feel we should stick with `terminate`.


TeamCity Jarvis Framework :: CI-GitFlow [Build 1.1.13-beta.1+1](http://build.prxm.it:8811/viewLog.html?buildId=5587&buildTypeId=CqrsFramework_CiGitFlow) outcome was **SUCCESS**
Summary: Tests passed: 138 Build time: 00:04:00


If you need a sorted list of streets and a matching street (if any), just return instead of a vector of streets pair<vector of streets, index of mathing street>.


Maybe update "Ci si aspetta che tutti i membri della comunità faranno rispettare questo codice" to "Ci si aspetta che tutti i membri della comunità *facciano* rispettare questo codice" on line 68


Same here; when starting trying to start SiCKRAGE by running `python2 sickrage.py` manually, I get this error:
```
Patching SiCKRAGE SSL Context

Usage:
  pip <command> [options]

no such option: --no-cache-dir```


А які є default-ні?


Those names are misleading. I think that _inletX name suggests value to be of type Inlet.


Please, see my comment above. This hack with swap is ugly. Why not to return streetIndex as is?


Now injecting pugins works as it should. If we look at `cli`, we'll get:

```
CLI {
  …
  injectedPlugins: [ [ [Function: plugin], undefined ] ],
  …
}
```


Consider adding a description of what the class does at the top of the class file. States functionality of code section


for devellopement just check out in your node_modules and nw will pick it up, until we have api stability this makes it useable for people as it will pick up the published versions (and not the latest)


If you're aiming to develop a symfony bundle, it must end with "Bundle" in camel case.

Just look at the JrkFooBundle repository this is a good starting point. You'll find a file "bundle-skel.txt" which shows you what you need to "search and replace" within the entire repository (start from top to bottom).

Yes I'll definitely check this ;) Thank you.


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (1675 of 1993 tests passed, 0 test warnings)

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


@Ichimonji10 That is fine, I just wanted to make sure we wouldn't break anything since these aren't safe to run in parallel. All tests that modify services aren't safe to run in parallel so they should be easy to identify when we do get around to parallelization.


line 71
"I nostri contatti sono nel nostro sito web e nella email che abbiamo inviato."
"I nostri contatti sono *disponibili sul* nostro sito web e *in tutte le* email *inviate*."


Agreed. :+1:


Cette condition devrait >= et non >


__API Description file syntax error on line 3159__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


Ici aussi il faut un >= et non un >


Ahaaa. Thanks for pointing this out. Looks like I designed this test incorrectly.


Revisando lo que hice ayer con Dio_ToggleSync(channel) además de los remarks que solicitaste encontré una potencial fuente de problemas:
   * En el archivo DioArch.h no se incluye a "IODriver_Base.h"
      + agregar:
         #include "IODriver_Base.h"

   * En el archivo DioArch.h no incluí el prototipo de "Dio_ToggleSync_Arch(channel)"
      + agregar:
         extern void Dio_ToggleSync_Arch(IO_ChannelType channel);

   * En el archivo DioArch.c  asigné un parámetro de más a la función "Dio_ToggleSync_Arch(channel)"
      + eliminar parámetro para que quede:
         extern void Dio_ToggleSync_Arch(IO_ChannelType channel)

Ya corregí mis archivos, actualizo el repo desde casa.


Поясни причину заміни `Get Current Date` на `Get Date`. Звідки то `Get Date` приходить?


to free the linked tabs while iterating this should to use `gputop_list_for_each_safe()` so iteration gets a pointer to the next tab before it's freed.


@elisenharper so does the user have to type in the language by name instead of choosing a number? It seems this might be more prone to error. I suggest changing it so you accept the numerical response (1, 2, or 3) before submitting for grading.