since this a default variable, I think root still makes sense here


This breaks the plurals tests. Run "nosetests" to confirm.


@kiennguyenquoc just remove the url, not remove the author text.


Yeah I'll add that :)


Fixes #2.


:( This typo causes 1.10 to crash w/ Ember 2.0. It should be `Array.prototype`.


Right, so if you're using clang then MINOR is set to 2, even though you have an arbitrary libstdc++ from gcc; meaning the check for 8 will fail. The GCC installs are mostly to build an equivalent libstdc++, then we use clang to compil just a few remaining items all the while keeping the GCC front end. It seems the best fix is to first detect the compiler THEN use either MINOR or has_feature, haha!


code reviewing was not something I had thought of, but will keep in mind for next time if it happens.


The first if block you changed does not ever execute.  That case is already handled in the first if in the function.  The second if block you changed will only execute if the "refresh inv" corrects a synch issue between Mafia & KoL.  Also, I think it would be preferable to move the set_property(..."blueprint") back to where that other changed if block is.  (But with a check for numPirateInsults() >= 7, instead.)  After all, it should be up to, e.g., LX_nastyBooty() to decide whether or not we need the outfit change (I don't think so) or the hair spray buff.  Also, notice that LX_pirateBlueprint() is already responsible for obtaining the blueprints, so the check for insults is the only one needed before we give control to it.


Pretty cool!

Er, what's this TBI and action stuff? Is that something from nucleotid.es?


I agree that I should check the alignment before doing a `strcpy`. I'll update that in a later commit.


I need a bit more time to benchmark JtR on Power or ARM. I'll post the result on john-dev.


Opened #3685.


Thanks!


Using the same name for command and function is a bit tricky for me.


This might still break the bincompat because of `unapply`. Why not add a method named `def localIfFile: Boolean = true`?


This one seems needlessly cryptic to me, I'm afraid.  We have mass_* all over the show.


Separate concerns


any reason you commented this out?


that would be :sunglasses: great assuming they pass (there's several of these in other places as well), thanks


so that human's get me


@adriaPerez I hope this helps your misunderstanding hahah


Ah, so that's why. I worried there was something wrong with the code. :D


good to know, you deserve a spec-tacular :trophy:


That's odd, but maybe our static snapshot in the repo was outdated or modified. Anyways, now it's deleted, less code, less worries :D


typo lol


More typos. What this means is that the structure's length field counts in header and data of the PDU (not just data, as the PDU's length field does).


done


Relates to #13, the original issue.


Thank you!


Make more elaborate


Sweet :D


develop (macumber) - i386-Windows-7-VisualStudio-12: Build Failed

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


You should use the `[]` short array syntax here and throughout the extension. https://github.com/Wikia/guidelines/blob/master/PHP/CodingConventions.md#array-syntax


Actually, we should likely remove all version-specific references, and also likely just point at https://www.percona.com/doc/percona-tokudb/hot-backup.html

@hrvojem  @vadimtk  thoughts?


You could use the `Html::element` helper function here to build the widget.


Didn't check assembly but the instructions generated have to be different otherwise the bench couldn't be different.  I don't believe it's defined by the standard what right shifting a signed integer means.


I imagine this should probably be contained in its own separate module and added to output via `$parser->getOutput()->addModules`.


This alt text should be localized.


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **82/87** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/60) page, and verify those files. If you have a question about this report, please contact @wforums.


You should use `wfMessage()->parse()` here and below :)


This is not what I meant. 😊
I meant that the function below doesn't need to to use this function or this variable. Instead it should call `(filename-directory (locate-library "clj-refactor"))`.


LGTM


Write a readme explaining the steps to get this to work


You could use `__DIR__` here instead of `dirname()`.


haha, right. will make further amends...


Ok, so take a look at what's different about those two pages as far as the background is concern Andy make index same as the map.

On August 21, 2015 12:40:16 AM EDT, Julie Chung <notifications@github.com> wrote:
>Yes. The map's background image shows, but not the main menu's
>background image.
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/j00leee/Livida/commit/50ef688fef64f2e2f1ce5f5b29c26006c5892fc9#commitcomment-12822048

--
Sent from my Android device with K-9 Mail.


That parser sounds like a very useful tool. If you need anything in order to make that happen, let me know.

As for the paste conversation, maybe you're thinking of the Dragonfly Paste action? Or `context.read_selected_without_altering_clipboard()` in conjunction with it?


True, but mostly they are internals. This one get's passed around to other code. In making change I was trying to tidy up things like this from successional diversity

```
  min_bounds <- bounds(
                       lma=c(0.01, 4),
                       mass_seed=10^c(-8, 2),
                       rho=c(10, 1000),
                       hmat=c(1.0, 100))
```

but happy to revert if you'd prefer


希望没事。唉。狗日的XX档。


This permission is required to check whether network is available and connection is present


@Nadahar I changed your forum username to Nadahar, let me know if it broke anything


In the commit you do two things, not only change the ssh key but also change the networking in the bosh release. Shall we split it?


Thanks for your concern. https://github.com/docpad/docpad/commit/2eb322b86f0b1991ed757e4b5e553369ef76e639 looks like a bit of logic for that, so :clap:


Che Khabar?


Structure / Orb / Schematic / Shell are pretty much the same. We need to work on their relationship.
An Orb defines random center and starts the first shell.
A schematic defines a schematic name and rules to interpret it. Those rules should be common, like the FillerSet is. We need another tag to actually place the schematic.
Whether it's an orb or schematic, it can be placed in a chunk or inside another structure.
So this could be something like this:
```
World Generation: in this chunk, we have x% to get structure 'y'
Single asteroid/Star/Moon case: Structure 'y' contains shells and has z% chance to have a schematic
Asteroid field case: Structure 'y' is hardcoded (we're not ready for that)
Derelict/Jumpgate: Structure 'y' is a schematic
```

I did a quick code review and it seems we could switch Star from Entity to TileEntity.

Type/GroupName needs to be hardcoded until we add a Starmap.xml (?) that would define the rules to spawn those structures. For now, it's merely x% chance to get that structure per chunk.

Having a common hashmap for all structures makes it more generic. No need to handle special cases of "I need a cloud, so I go in this table". That structure is named, we don't care of its type or content.
In other words, the starmap doesn't care if it's a schematic, an orb, or whatever. It just want to place that 'named' structure in space.

Yes, we need to fix the checks so it doesn't crash. My point being that we want to improve stability in existing code as we get closer to release instead of creating new crashes, and we can't wait for the perfect stable code because it's not there yet and we wouldn't be in Alpha in that case.


Good idea :)


thanks, I'll fix that.


Sounds like a good idea to me.


the un-commented value of 99999 is to invoke an error.


Che Jelafata


I find the white-on-orange really hard on the eyes.

![screen shot 2015-08-21 at 11 13 29](https://cloud.githubusercontent.com/assets/282788/9406153/b7930c84-47f5-11e5-9b71-7e73d5582065.png)


Add the auto generated files to .gitignore


+1

This is breaking asset pre-compilation with Sprockets 2 which will force me to either use requirejs-rails 0.9.5 or do a careful migration and possibly painful to Sprockets 3.

@agis- solution seems like the way to go, but otherwise at least add an explicit dependency on the new Sprockets version


Async downloading added


eshtebah


Actually, I think I remember now; we were talking about Text and triggering autocomplete, right? In that case, I probably mentioned Dragonfly's Paste, which doesn't modify the clipboard contents.


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


This caused our project to break as it depends on `~> 1.1`. Should this not have been set to version 2.0 as it is a breaking change?


I don't see that as particularly bad.  Also note that we have:

* `lma`: abbreviation of leaf mass per unit leaf area
* `rho`: greek letter, but seems widely used for density
* `hmat`: different style of abbreviation (H MAturation)

I don't really think there's a win here.  `omega` seems pretty arbitrary to me, especially given it's just a mass measurement.  If we used `m_*` everywhere else it might be better, but I _really_ don't want to make that change!

But it doesn't really bother me either, especially as we barely use this anywhere yet.


We have full control of the lua code and unit initialization -- it's a matter of checking the ctor's :)

We should clean things as we go and leave the code in a better state with each change, so we might aswell do this now and be done. Can you rebase your pull request?


And still no. It crashed with new generator, which does not find any structures.


The string "auto cast" is working fine, except when used as an array key, which is a bit weird since you can't use objects as keys...

I'll update this, just to be safe.


That was my original approach, but for some reason I concluded that I needed to `&&` all commands to guarantee that the Travis run would fail if any of the intermediate commands failed. Given your comment, I tested everything again and I can confirm that the `&&` are not necessary... I really don't understand what happened at the time as I could swear I made these tests!


последняя задача


Actually looking at other commits, it looks like semantic versioning isn't exactly considered important to Stripe... :\

Is the main Stripe iOS SDK semver compliant or not?

i.e. should I be attaching to `~> 5.1.2` or `~> 5.1`? I would much prefer the latter to ensure we don't miss fixes.


The change to the subnet from private to public is reverted back in a later commit - we were struggling to get SSH connectivity working at the time of this commit - This commit should probably have been squashed into another later commit.


The three above can be non-pointer things. Qt disables copy ctor for QObjects but we don't need it.


Hey Garret
Did you delete db_connect.php by accident?


You can use QSocket and friends idiomatically if you want here.


Yes just saw it on PnMercantour Android app and it is a good idea indeed !


@vwadekar,
Hi Varun,  You would need to do that in your tegra_get_sys_suspend_power_state() handler. So, going by the current code organization, you may want to define tegra_soc_get_sys_suspend_power_state() and fill the req_state[2] to the required PLAT_SYS_SUSPEND_STATE_ID. Since you would want the lower power levels to power down as well, req_state[1] and req_state[0] should be set to PLAT_MAX_OFF_STATE. As mentioned in my previous comment, PLAT_MAX_OFF_STATE  should be a value > = PLAT_SYS_SUSPEND_STATE_ID.


Habt ihr schon mal darüber nachgedacht, Chef, Puppet oder Ansible zu verwenden?
Wir bauen uns da gerade etwas: https://github.com/FreifunkBremen/ansible


To test page with this server you need first to run `build pages` this is too long to do it while devloping


That leaves a problem if opentrack crashes, but the subprocess stays the way it was.


There are two spaces between `ConnectionFactory` and `connFactory`


You can use QMutexLocker here


I guess `- ./misc/travis-check.sh` may be executed with
`sh -e`.
```
sh -c "help set" | grep -e -e
sh -c "help set" | grep -e -e
      -e  Exit immediately if a command exits with a non-zero status.
              errexit      same as -e
```


Hey @srizzling, I've removed you from the team now :)


Thx for your changes. This saved us a lot of time. I'm not quite familiar with python.


共通テストケースの名前がいけてない


I think this commit needs be split and with a better description in each commit. It is important to keep the commits documenting the changes to help later troubleshooting and allow other people learn from your work.

I would split it in the following commits:

 * Fix typos using the ssh keys (ssh-tunnel using id_rsa, insecure deployer), and in some tags.
 * Adjust firewall rules to restrict connection from the office to a public IP of microbosh to the bosh ports.
 * Adjust public subnet ranges
 * Set the bosh elastic-ip of bosh to be VPC
 * Parametrise security groups and region for the manifest template


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


You forgot to add ```../development/interpreters/ceptre/default.nix```.


Yup, exactly!

In theory, we should actually have two different classes, in two different namespaces (`IServiceCollection` extensions are usually in `Microsoft.Framework.DependencyInjection`), but I find this pattern a bit ugly so... one class is enough :smile:


I think this can now be closed


Oh. You are right! My mistake.


On 5.2?


Бинарные файлики нужно в .gitignore добавлять и не коммитить их :stuck_out_tongue_winking_eye:


but correct folder name is drive_c not .drive_c,  why here with a dot?


It's not crashing now... But after 5 minutes of generating chunks i got nothing but two diamond blocks.


Although, only absolute path, for now.


This should probably be `SSL_PSK` to be somewhat more consistent with other TLS options.


You should not check for OpenSSL specific defines here. Instead you should define something like `have_curlssl_psk` in the `lib/vtls/openssl.h` file and then check for that (see [this](https://github.com/bagder/curl/blob/master/lib/vtls/openssl.h#L85) for example). This way PSK support can be later added for other backends as well.


และจะมีฟังชั่นมากที่สุดเมื่อ n(n-1)


* Thu Aug 20 2015Erwan Georget <egeorget@openmailbox.org> - 3.15.0-0.1.newui.afdc2a1c.20150820gitbc1b3fda
- Added "Requires: python-pyquery"


101682488_EvapCoolPrimaryAirSizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1793 of 1799 tests passed)

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


#24 Fixed wrong chart shift


Так как в member_tokens создаётся foreign key (member_tokens_memberid_foreign), связанный с таблицей members, то id этих двух таблиц (memberId и id) должны быть идентичны. Без инкремента это выглядит так:
таблица members - `id` int(10) unsigned NOT NULL AUTO_INCREMENT, таблица member_tokens - `memberId` int(11) NOT NULL.


Оукай :hamster:


je pense que tu n'est pas asse complet. peux-tu rajouter "tres" genial


franchement .....


Because of a fault in my original PR, sorry, will be fixed soon


Да, но до методов down я не добрался, так как ни на postgres, ни на mysql откат (php artisan migration:reset) не работает (в основном из-за того, что не дропаются индексы). Поэтому забыл привести поле в lowercase.


I think this should be removed. If the certificate verification is not needed the `CURLOPT_SSL_VERIFYPEER` option should be disabled instead (not sure if it actually works though...).


101682488_EvapCoolPrimaryAirSizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I generally favour explicit over implicit (variable/function names, etc)


Cool, I will follow the same pattern


但愿有海外大神接过这个火种，感谢C大一直为我们的付出。


Data_sources happens to have `~gen_events:false` and `persist:PersistNothing`. I don't see any special mentions of it in the bindings. Not if `~in_db` actually has any downsides though. Many of the fields in datamodel_types are a little mysterious to me :)


Why both user and trainer routes directed to users function only:
Route rule for users is
$route['people/(.)'] = "main/users/$1";
For trainers you can use rule like:
$route['trainer/(.)'] = "main/users/$1";


It really does. Was truly explanatory. Thank you very much for helping with my understanding of things in life.


 > Will the suspend hook be only ever invoked for the system suspend API ?

Yes. That's why the check for PLAT_SYS_SUSPEND_STATE_ID in validate_power_state().

> It seems that at line number 124, there is a hard coded dependency for CPU 1. So I infer that the CPU 0 will always be the last CPU to enter system suspend. Please confirm if that is the case.

Correct.

> If the target state is encoded correctly as mentioned in comments for the validate_power_state() handler, the target_state[2] will have the required state-ID

I'm a bit confused. If validate_power_state() does not get called during system_suspend, how would the platform code get the state-id?

> thus removing the need for the platform layer to keep track using tegra_pm_system_suspend_entry() and tegra_pm_system_suspend_exit() functions

These functions allow the platforms to understand if we are exiting system_suspend in the suspend_finish handler. What would be the alternative then to understand system_suspend exit in that case?


I'm sorry, but still dont understand how we would handle the system_suspend scenario.


convert_new_coils_modelobjectlist (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (2304 of 2341 tests passed)

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


?
pas compris...

*Cécile DUBOSCLARD*
Responsable Logistique
<https://sites.google.com/a/zenika.com/wiki/communication-interne/signature-de-mail/simple.png?attredirects=0>

*ZENIKA PARIS*
10 rue de Milan - 75009 PARIS

Mobile : +33 (0)6 62 33 61 57
Ligne Directe : +33 (0)1 75 43 08 19

Le 21 août 2015 12:38, zenikatraining2 <notifications@github.com> a écrit :

> je pense que tu n'est pas asse complet. peux-tu rajouter "tres" genial
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zenikatraining1/git-training2/commit/17527e919b1b5af3c9140180fc8129e7b4d8f1c7#commitcomment-12826060>
> .
>


answer question 1-4


only encrypt. decrypt will be soon


Yeah, we probably should to to 5.4+ only.
@alecpl - what do you think?


Sort from larger to smaller


Answer question5 & Delete flag.


saw approval from andytill
at https://github.com/basho/riak_api/commit/9f3390c929a9cc622e79a158a80106c0ece4d2ce


merging basho/riak_api/fix/az/correct-riak_pb-dep-is-branch-not-tag = 9f3390c9 into borshop-integration-96-fix/az/correct-riak_pb-dep-is-branch-not-tag


basho/riak_api/fix/az/correct-riak_pb-dep-is-branch-not-tag = 9f3390c9 merged ok, testing candidate = bafdc01d


Roundcube is 5.3+ and I'd prefer that, but we should consider 5.4+ for next version, yeah.


saw approval from andytill
at https://github.com/basho/riak_core/commit/e80d9932a3af12f6a0dd03e0586812a1bbe557dc


merging basho/riak_core/fix/az/pin-eleveldv-to-branch-prototype/timeseries-range-scan = e80d9932 into borshop-integration-772-fix/az/pin-eleveldv-to-branch-prototype/timeseries-range-scan


basho/riak_core/fix/az/pin-eleveldv-to-branch-prototype/timeseries-range-scan = e80d9932 merged ok, testing candidate = 8c50b3c1


Please build one function and us it in both functions.


But not adding the event_type?


Ok. I did not see your commit comment. I took for granted the bench is the same.


Ok, I've removed PHP 5.2 from `.travis.yml` in master.


This commit contains extra hunk of changes to `DiagnosticDialog` that are unrelated.


I confirm that if the shebang is removed from the script file Travis is still able to run it and correctly detects the errors. So I would say the script does not need to have the execute permission and instead is run using `sh -e` (or equivalent).
If you agree, I'll update my implementation with the simplified version and will include the `-e` in the shebang to mimic the behavior inside Travis environment.


ElementAt(0) -> [0]


Welke afmeting had de oude foto?


BTW, I was unaware of the `-e` option. Thanks for that tip!


Somehow I suspected that /source/_patchinfo doesn't make sense. So it should always be "/source/#{self.init_options[:project]}/patchinfo" ?

If it's patchinfo instead of _patchinfo why did this ever work? In other words do we really need this?


Thanks for the tip! This trick just helped me to simplify things! ")


Hi Varun,
   For PSCI SYSTEM SUSPEND API,  the `get_sys_suspend_power_state()` handler will be invoked. Once the req_state[] is populated by the handler, this will get stored in the PSCI Node information. On resuming from system suspend,  the `target_state[]` parameter in the suspend_finish handler will have the state being woken up from.

So in your case, the `target_state[2]` of suspend_finish handler will be PLAT_SYS_SUSPEND_STATE_ID thus allowing the platform to identify that it is a wakeup from system suspend.  Hence you dont need tegra_pm_system_suspend_entry() and tegra_pm_system_suspend_exit() functions to identify the same.

Hope it is clear now.


Doh, just noticed this is wrong as it needs to be I386 not x86 for the 32 bit build.


а че так решил?


It seems that there isn't any simple way to declare the padding for the title or the icon in the styles. The Stack Overflow question on this only has solutions for setting the padding dynamically: https://stackoverflow.com/questions/10706975/padding-the-actionbar-up-indicator-image

Even the dynamic solutions don't rely on an API, and thus are subject to potential breakage: https://stackoverflow.com/questions/27723266/why-does-findviewbyidr-android-id-home-always-return-null

It seems that Hanning used the solution from this answer: http://stackoverflow.com/a/21578740

It should be easy enough to fully customize this after we switch to the `Toolbar` from the material design.


@philboeselager All tests OK, so I assume this is fine. Still, please take a
look. Like we discussed, there is not really a necessity for a
merge-logic here, incoming data is always considered a complete
representation.


If we can manage the fields from my side that would be great :)


if you override generateWgsMetrics and the metrics class you will be able to add more fields that might be interesting in this case, for example TOTAL_TARGET_BASES,  TOTAL_BASES, TOTAL_USABLE_BASES, as you did in CollectWgsMetricsFromQuerySorted. assuming that you find them interesting.


"starting slash"


evaluate strlen() only once


Oops. fixed.


Looks good.


@vramirez122000 CloudFlare es el sponsor de cdnjs.com, el go to CDN para muchas librerias ya que sirven todos los assets, no solo css/js y lo más importante es que tienen support para HTTPS. El CDN de leaflet no tiene un certificado válido. Puedes ver más info en https://cdnjs.com/about


Is 'calendar_weeks' option available? I really need this feature, can I help you with something?


@aliskov This has not been merged into the master branch of the axitkhurana/buster repo. You can clone https://github.com/thinkamabob/buster/commits/master (it has this feature, different commit though) and use the modified buster script directly:

    python buster.py generate


This seem to fail on current cygwin (where vasprintf needs to be compiled in)


Is enum34 excluded from install_requires after this PR merged if `sys.argv` has `bdist_wheel` option? like  [iso4217](https://github.com/spoqa/iso4217/blob/master/setup.py#L55) did


:+1:
zdjąłeś po response disable?


You misspelled "canola"


forgot # in #297


This comment seems to be truncated.


大家去了美利坚 要记得还在水生火热的gcd领导下的码农啊


Should this include the "autorebuild is disabled" message in the exception's message?


I suspect this test was not doing was it is supposed to. Only checking that the result of `_batch_mean_variance_update` is OK is not sufficient. We also need to be sure that the chosen data *does* present issues when we run a naive benchmark algorithm. The following test does that.


Because for reporting to dashboard one needs the crab_retry, which in this case is not available.


@magic96 tutaj ;)


strtok is not thread-safe. use strtok_r. Maybe use Qt instead?


Их не должно быть, я гиту не указал, чтобы он их не втягивал.


Made changes as requested.


Bonjour, qui êtes-vous ?


all tests pass:
success: http://buildbot.bos1/builders/test-riak_api/builds/141


all tests pass:
success: http://buildbot.bos1/builders/test-riak_core/builds/1717


I install jsdom 3.1.2 but jsdom required contextify and contextify required C++ compilor and python. Is am right? I am new to node.js


Would this be better as a comment inside the try clause explaining why we don't catch that exception? (Not sure)


Get rid of the spaces.


Thanks for the advice we will work on it.


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

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


When building comp_cnt and path_components like this, there's a chance that path_components.size() != comp_cnt


This is not working properly ... See this commit ( https://github.com/rooque/android_kernel_xiaomi_cancro/commit/23c253d10d7b59767121d86fc919eebf50445d02 ) , looks it's fix AD on mi3 , but still some work to be done.


Fix formatting -- should be `if (...) {`


develop (kbenne) - i386-Windows-7-VisualStudio-12: Build Failed

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


originalbilder sind bestellt ansonsten kann man die vorhanden schonmal als Platzhalter verwenden und später "einfach" ersetzen.


@djstrong I don't think this is correct, it should be rather
```ruby
cyc.cor{|cyc| cyc.isa?(c,t); @cyc.genls?(c,t)}
```
(mind the closing par. `}`)


It's possible to do the computation without building the path-component string list. Why not run the equivalent of strtok_r on demand?


@xordoquy When will this be available via PyPI?


I just wonder - why?


Use the `C<'foo'>` format for values (here and elsewhere).


this is how it's looking on 2 times smaller display:
![2015-08-21-131451_1920x1080_scrot](https://cloud.githubusercontent.com/assets/1655669/9407148/f5a4fcb0-4806-11e5-8622-2b9aa4654899.png)

i think that can be an issue only on some laptops, where is still common 1400x900 resolution, so, it will make sense to make widget smaller if it not fits available workarea

but it will require some work on pagination and actually i think that should be somehow generalized, because `awful.menu` is missing some pagination/scrolling as well (i think only `menubar` is aware of it atm)


470 x 705

(testen of deze manier van beantwoorden ook werkt?) xx

Date: Fri, 21 Aug 2015 03:57:26 -0700
From: notifications@github.com
To: LeonieThijssen.github.io@noreply.github.com
CC: leonie_thijssen@hotmail.com
Subject: Re: [LeonieThijssen.github.io] profielfoto veranderd (d91c54e)

Welke afmeting had de oude foto?

—
Reply to this email directly or view it on GitHub.


I tried much but I couldn't run your code


That was ahead of the assignment that's for sure :)

Well done!


Тогда давайте разбираться, `memberId`  это поле которое заполняется при добавлении токена и оно заполняется не само по себе а мы указываем ID пользователя для которого добавляется токен
и `memberId` могут идти не последовательно (как в случае с автоинкрементом) а в произвольном порядке к примеру 23, 44, 12 и т.д. Как по мне то автоинкремент для `memberId` неуместен.  Если бы это был порядковый номер токена то да туда можно инкремент, а так  не стоит. А за идентичность  (memberId и id) будет отвечать не автоинкремент, а разработчик который будет писать логику добавления токена.


Nice.


OK, thanks @madmatt! I'll try unwatching the groups, hopefully that fix this.


`.to eq(expected)`


It won't have worked for normal users for sure. We could also require the parameter in the function and break if it isn't there.


Qt isn't linked from there, the realsense process relies only on realsense and system libraries to keep it simple and independant.


I'll create an issue about this and make the update since the pull request
has been merged.

On Friday, August 21, 2015, Mike Krüger <notifications@github.com> wrote:

> ElementAt(0) -> [0]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Kavignon/RefactoringEssentials/commit/1450a8f52c24668a65f80e98ac4d705e85162a80#commitcomment-12826267>
> .
>

--
Kevin Avignon
Étudiant en génie des technologies de l'information (ETS)


Rename the method to `recaptcha_verify` to be consistent with reCAPTCHA's nomenclature.


Could use `set(existing) - set(labels)`.


I'd write a view spec to confirm that these tool tips are available


This regexp could do with a comment explaining how it works. (So I understand it ;-) )


this could also test for WONTNEED ?


Don't need the opts arg


@albertosouza da um help na parte de pegar a config pelo locals.js


Yes, that is necessary. I missed it.


Will fix


So do we still need to remove (double-)quotes here, even though we're calling `_split()` with `preserve_quotes=False` above?


Hi.

This module implements the page /ekurser, from where a user can browse all e-courses in the well, optionally limited to popular subjects.

Technical considerations:

We want to reuse the functionality from the ding searchresult page, but with a few adjustments, so it makes sense in regards to only listing e-courses.

The e-courses are queried from the well with the string "ekurser.nu", as we currently can't uniquely identify them all by source or publisher.

We want to offer the user a way to limit the results via the facet "subject". But we're not interested in other facets, and since there's only one facet, we want to display it as a submenu, instead of the usual folding facetbrowser. So we remove all facets from the facetbrowser and build our own list of terms.

The subject-terms are currently defined by dbc, and may include terms that cover e-courses in general, like "Vejledninger". To get rid of these from our submenu, we remove terms that cover more than 80% of the total resultset.

Some of the fields in the searchresults posts seem redundant when only listing e-courses. Like availability and year (year is misleading, as many e-courses are updated regularly, which the wells posts currently don't reflect). So we hide these fields via css.

We add our own sort-options to the searchresult, as ordering by something like author doesn't make sense, while ordering by acquisitiondate does.

Note:
Morten, please correct me if I'm wrong. Also, I think we need to find a way not to remove facets explicitly, as the siteowner may have added or removed facets.


@dAnjou it should have been uploaded a few days ago. No idea why it wasn't on pypi. This is fixed.


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


T210 is a quad core, dual cluster SoC. Even though there are 8 CPU cores available, the OS only views 4 at any given time. Only one of the clusters is available for the OS at any given time.


I think this should come above the class definition, and not `private`. I don't know the exact mechanics of how private this actually is in a module, but semantically I'd read this as an internal method not part of the public API of the module, when in fact it's the core functionality.

```ruby
module TranslatableParams
  extend ActiveSupport::Concern

  # Some usage doc
  def translatable_params(keys, params)
    WhitelistedParams.new(keys).whitelist(params)
  end

  class WhitelistedParams
    # …
  end
end
```


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


@str1fe  Wouldn't it be possible to add Appsignal.exception in our own app? It renders the exception as json, so shouldn't it be possible to send to that to Appsignal without this fork?


Ja die werkt vaak sneller bij mij ;)


change `nil?` to `blank?`


@soby-mathew I have some follow-on questions that would help clarify my doubts. Thanks.


Nicely done. You could somehow simplify the code by using early `return` statements instead of a unique final one, so that you don't have to use `break` multiple times.


Good point, I'll add some code to handle that.


Cool, thanks :)


![Uploading 20150424_214027.jp
![Uploading UB Reader User's Guide_80700_cover.jpg…]()

g…]()
![Uploading UB Reader User's Guide_80700_cover.jpg…]()


no need to use white spaces in this case, they should be together


Może od razu warto przejść na <TD class="bold"> ?


probably worth one spec like:

```ruby
  describe TranslatableParams
    include TranslatableParams

    it '…' do
      expect(translatable_params(keys, params)).
        to eq(WhitelistedParams.new(keys).whitelist(params))
    end
end
```

Not sure of the exact way to test concerns


gff


fix #38


:+1:
Also internal functions can be double underscored as private ones, eg. `__do_dp()`


+1, really nice.

Minor note: quite a few missing `\n`


I have a case where it's easier to use it as a plain injectable observable. Basically just a proxy for the Observable which comes later. Anyway it's just an additional feature.


The "yes" string in the updateMedicationEntry can be declared as a variable once and used multiple times when needed.


@soby-mathew thanks for the info. The scenario is much clear now.


@soby-mathew it's clear now. Thanks.


It seems the _patchinfo thing is something similar to _product. See [here](https://build.opensuse.org/package/show/home:bgeuken/patchinfo).


In fact, I probably wouldn't use the class in the `eq` – just give it a Hash so that you're not using internal implementation details.


Why the multiple declaration of these variables


Here's a comment


you can deleted these comments


delete


:) Tak, http://js13kgames.com/


@hiuryanderson a correção já está disponível em: https://github.com/eneiasramos/pagarme-magento/commit/d051f6639c64a1e95bd1aef43213c50b7023e1ce


Hey Matt...I may have...I've placed a .git ignore on my local server so I won't do it again.
BTW I'm currently implementing the home page to display latest posted jobs

Sent from my Samsung device

-------- Original message --------
From: tomble21 <notifications@github.com>
Date: 21/08/2015  11:26  (GMT+00:00)
To: Mattw46/CPT375-A2 <CPT375-A2@noreply.github.com>
Cc: Garret Bourke <bourkg@hotmail.com>
Subject: Re: [CPT375-A2] implemented job posted ok page and error page for posting (8f16387)

Hey Garret
Did you delete db_connect.php by accident?

---
Reply to this email directly or view it on GitHub:
https://github.com/Mattw46/CPT375-A2/commit/8f163871448d19478bd7cc2af9da95ca5d7c5fa0#commitcomment-12825897


This one gives complication with having the same key on multiple values on a meta_query:

For example:
``
	$args['wp_query'] = array(
		'post_type' => 'POST_TYPE',
		'posts_per_page' => 12,
		'type' => 'THE_TYPE',
		'meta_query' => 	array(
								'relation' => 'AND',
								array( 'key' => 'SINGLEKEY', 'value' => 'VALUE01', 'compare' => '!='),
								array( 'key' => 'SINGLEKEY', 'value' => 'VALUE02', 'compare' => '!='),
							)
	);
``
With use of this row, the second key (VALUE02) is used on all array's.
Without use of this row they load seperated values.


Looks like you corrected your mistake here from the previous commit and put the nav inside the header and wrapped them all in the body element, good job!


remove commented testing codes not used for it should not be seen in this final code


希望没事儿啊


Hm, I get it. My only concern is that it can make simple mistakes harder to find. If you just forgot to pass argument to constructor, no error is thrown, but application does not work as you expect. It's just a trade off, I think, maybe it's worth it.


I'd question the use of a h5 here.  You could use a h2 as it seems to be of higher importance than a level 5 heading.  Then use css to set the font size accordingly.


Good use of he alt tag.


101280704_lowTempRad (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1785 of 1800 tests passed)

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


Thanks, I had problems running MNIST without it, perhaps I did something wrong when downloading data


Thank you for your work. And sorry for what happened.


Yes, thanks, fixed.


Good use of header and footer semantic tags


I think this option is only available in eonasdan v4. But as mentioned in https://github.com/sonata-project/SonataCoreBundle/pull/149 the eonasdan upgrade is only planned for Sonata v3.

If you do want to use it in Sonata v2, you probably could create your own date type, but off coarse you would break the existing date type when upgrading eonasdan as mentioned.


I don't think that's possible. If `AutoRebuildCanceledException` isn't caught and reraised explicitly, then the below `except Exception as ex` will catch this, transform it to a message and then raise `PluginFailedException` with that message. This would mean that we would need to parse the exception message in workflow to determine whether the rebuild should be skipped. I'll add a comment explaining this, if it seems ok to you.


Put these (4) variables inside the `describe` block.


This causes a PaymentKit itself to break, making it impossible to use the latest Stripe iOS SDK along with features from PaymentKit which haven't made it to the new SDK.

The error it causes is:

```
duplicate symbol _OBJC_CLASS_$_PTKView in:
    /Users/james/Library/Developer/Xcode/DerivedData/CuvvaHourly-hcvwpyiqnxxugbbvprcyamjerpnn/Build/Products/Debug-iphoneos/libPaymentKit.a(PTKView.o)
    /Users/james/Library/Developer/Xcode/DerivedData/CuvvaHourly-hcvwpyiqnxxugbbvprcyamjerpnn/Build/Products/Debug-iphoneos/libStripe.a(STPPaymentCardTextField.o)
duplicate symbol _OBJC_METACLASS_$_PTKView in:
    /Users/james/Library/Developer/Xcode/DerivedData/CuvvaHourly-hcvwpyiqnxxugbbvprcyamjerpnn/Build/Products/Debug-iphoneos/libPaymentKit.a(PTKView.o)
    /Users/james/Library/Developer/Xcode/DerivedData/CuvvaHourly-hcvwpyiqnxxugbbvprcyamjerpnn/Build/Products/Debug-iphoneos/libStripe.a(STPPaymentCardTextField.o)
duplicate symbol _OBJC_CLASS_$_PTKCard in:
    /Users/james/Library/Developer/Xcode/DerivedData/CuvvaHourly-hcvwpyiqnxxugbbvprcyamjerpnn/Build/Products/Debug-iphoneos/libPaymentKit.a(PTKCard.o)
    /Users/james/Library/Developer/Xcode/DerivedData/CuvvaHourly-hcvwpyiqnxxugbbvprcyamjerpnn/Build/Products/Debug-iphoneos/libStripe.a(STPPaymentCardTextField.o)
duplicate symbol _OBJC_METACLASS_$_PTKCard in:
    /Users/james/Library/Developer/Xcode/DerivedData/CuvvaHourly-hcvwpyiqnxxugbbvprcyamjerpnn/Build/Products/Debug-iphoneos/libPaymentKit.a(PTKCard.o)
    /Users/james/Library/Developer/Xcode/DerivedData/CuvvaHourly-hcvwpyiqnxxugbbvprcyamjerpnn/Build/Products/Debug-iphoneos/libStripe.a(STPPaymentCardTextField.o)
ld: 4 duplicate symbols for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
```


You could periodically send a single byte with an irrelevant value to the subprocess. Then quit the subprocess if a single-byte datagram hasn't been received in, say, 200 frames.


_product is always a package container, while _patchinfo is the file name of the source file.

A _patchinfo can be in a container with any name. It's existens make the container then a patchinfo build. (Like an _aggregate makes it an aggregate or a .spec file makes it a rpm build).

_product container will always be expanded into multiple _product:$something containers based on the .product files inside.


So thank you for information. now that code is not giving me right value as well.. but I'll search more information.

Thank you.


Thanks! I was looking for an example.


сделайте это параметром аннотатора.
Проще всего это сделать через @ConfigurationParameter от UIMAFit (см. документацию).


use trim() on text you get from edittext.


Remove. + Each test case creates its own instance and there is no need to store it in the upper-level scope.


Use arrow function so you will not need this closure.


Hi Many thanks goy your reply.

Keyboard modification while programming the arduino has no effect, I tried different settings.
Default setting on the MBA is French numerique. When manually modified to American, it works. But as soon as the computer reboots, setting is back to default.
You're right, the right thing would be to program to move the mouse to kb setting on the top of the screen, then to press first line, iE American. But I notice that your routine to put the wifi off does not seem to have an effect (visually speaking). And I ami not able to do so.

I also tried to modify the code with with what i read in the arduino Usb kb wiki (with Keylock) but I have no knowledge of coding, i guess sometimes but that's all.

Thanks anyway for taking time to reply. Your code (very quick compared to other) and idea of reboot MBA is pure Genius.

> Le 21 août 2015 à 11:08, knoy <notifications@github.com> a écrit :
>
> Thanks, but I'm unsure why pressing the Caps lock button should matter?
>
> If you use a different keyboard the buttons may be in different places, you will probably want to adjust some moveMouse() coordinates to change that using trial and error. Let me know if that solves your issue, and if you have any code to contribute back please feel free to post it here.
>
> —
> Reply to this email directly or view it on GitHub.
>


mě se obecně nelíbilo, že jsi tam na jednom místě dělal i del z kwargs, spíš by se měli zkopírovat a pak odstraňovat, vem si, že tam pošleš data a vrátí se ti něco jiného není to moc programátorsky příjemné..


Only a minor thing really but I usually like to have the html and body styles at the top of the file as they always logically come first in the html.   Good use of the Web Font Trebuchet MS, can be used fairly safely but I'd have a fall back for the 2% case that don't have em e.g.

`font-family: Trebuchet MS, sans-serif;`


According to the [composer docs](https://getcomposer.org/doc/01-basic-usage.md#composer-lock-the-lock-file), the lock file should be committed to the repository. Am I reading that right, does it apply to libraries too?


Should it really? It makes more sense to me if this returns `null` instead of throwing.


use requestFocus on the edittext to assign focus to the view.


I think it should be named `isCurrentWindowActive`


можно эти циклы проще написать, используйте for-each. Почитайте про него и интерфейс Iterable.


на постгре все откатывается
какая у тебя версия?


Is this already fixed in v1.4.4?


诶，不太清楚究竟发生了什么。还是对 clowwindy 和各位开发者表达一下谢意吧，真的是很棒的 socks 代理工具，自己也受益颇多。

作为一个优秀的开源软件，ss 也一定能在开源社区延续生命的吧。

希望大家都能得到自己的幸福 :)


v send_html_mail_from_email_template to je


A releases footer should be added..

Releases: master, 3.0? might also be present in previous versions?


The hardcoded path will only work with WoW64 installations in the default path. You can use VS120COMNTOOLS to ease the pain.

Please use -MT unless the SDK uses -MD. In the latter case, copy MSVC CRT into the directory where subprocess lives. In the latter case, the subprocess needs its own subdirectory to avoid cluttering too much.


Note that bundling MSVC CRT dll's as-is without installer is legal.


Since it's an object and not a ctor, it should be lowercased: `activeWindowTracker`


@blickly This is now out of whack with the method's javadoc. And in my custom pass unit test, the actual/expected messages are the wrong way around now. See CompilerTestCase:1219.


English: `when add is called with no arguments` + enclose "add" in backticks (`)


\r-


а ты не на винде случайно?


The TreeHouse learning is paying off you're certainly ahead of the curve with grouping style blocks with commas ;-)


We can test it and see how it goes, and revert if its not good.


No way. You may not access private member variables. This is an implementation detail. If there is no `getKeymap()` method, then this test should not exist at all.


Oh, right, I see that now. Somehow I didn't see that it would have been caught anyway. It's best as it is then.


When you find yourself having to use both `margin-top` and `margin-bottom` in the same block it's best to refactor into the shorthand property `margin`


First off, no you didn't.  This code was submitted as a PR, but I never pushed it through.  I wanted the default icons from this library because I liked them for my use.  That said, I will make this better when I have time.  It is not a priority at all right now.

If you don't like the implementation, override it or submit a PR.


Weird expectation notation ("to exist").

```
// Using expect.js, this would read better as…
shortcuts = manager.getAllShortcuts()
expect(shortcuts).to.be.an('object')
expect(shortcuts[someKey]).to.be.ok() // for example
```


In that case you should have both, since I expect that reacting to an exception in the success handler would need different behaviour to reacting to a failed HTTP request:

```js
$http.
  get(...).
  then(successCallback, failedCallback).
  catch(errorCallback);
```


I think it can be refactored. Something like this :

    key = data.kind_of?(Array) ? "#{key}_ids" : "#{key_id}"
    data = [data] unless data.kind_of? Array
    value = data.map { |ri| ri['id'] }

If the json is well formatted, data must have a value.


`.toBe(true)`


感谢C大一直为我们的付出


`it 'should have a displayName'`


Then like a _link? In any case it seems that it's valid as it is in line 243. I am not so sure about "/source/_patchinfo". But that's nothing that changes with this PR.


Should have used String.valueOf()


it 'should default the eventType property to null'


В новой версии этот чендж порождает другую проблему, по этому мерджить не надо :)


Thanks, I fixed it also for mailed field


`it 'should create a <shortcuts> DOM element'`


non of your business :D
actually, i'm working on a small project to visualize the structure of a repo (so class, methods, properties for the beginng)..and first i planned to analyse source files by "parse" tokenizers output. But instead of this reinvention, i use doxygen xml output now (the plus: works for big bunch of languages :D)...so...this repo will be removed soon ^^ ...new one will be added soon ... ^^


Done 9f4f955b71552080b6a0be10f66a2342cbd26927.


ок =)


Thanks for the comments.
`data` can have a `nil` value in case of an to-one relationship (http://jsonapi.org/format/#document-resource-object-linkage).
The problem of packing data into an array is that we have to unpack it later for to-one relationships, since ActiveRecord expects `:author_id` to be a value, and not an array.


`it` refers to the tested component, not the resulting rendering. Better: `it 'should add a tabindex attribute to the <shortcuts> element'`


`@_cursor` has been removed due to change in api. 9f4f955b71552080b6a0be10f66a2342cbd26927.


`it 'should accept a custom tabindex attribute value'`


Sure!

2015-08-21 13:49 GMT+02:00 André Staltz <notifications@github.com>:

> We can test it and see how it goes, and revert if its not good.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cyclejs/rx-injectable-observable/commit/822d7df22e85f75e7bda4747da65a382ccd5c4b7#commitcomment-12827068>
> .
>


Nonsense. Removed. 9f4f955b71552080b6a0be10f66a2342cbd26927.


I have no time next week so I'll finish with this PR a week later.


A _link is special again :)

Yes, this PR is okay, the current code is broken. But we could also fix this while being over it :)

Either raising exception when package is not defined or using the default-by-convention name "patchinfo" should be fine.


is there a developer guide we can read? i'm a little nervous about these tokens in the open.


`it 'should render its children as the children of the <shortcuts> element'` + test that it's actually rendered within the `<shortcuts>` element.


The workaround to model the template method works. It eliminates 263 false positives in one swoop. However, we get another new 73 defects from SQLite library. Not sure how many are false positives again. Sigh.


`it 'should expose the change event type as a constant'`


I thought it is an eventemitter method but isn't. Changed to `@removeListener`. 9f4f955b71552080b6a0be10f66a2342cbd26927


Thanks!


Yep 9f4f955b71552080b6a0be10f66a2342cbd26927.


ok so

Youre right. And do only one if instead two is not correct too because there is too many conditions for the key... I think you have a good solution.


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


81211464-LinkInfiltrationAndReturnFlows (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1274 of 1274 tests passed)

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


LOL. Done. 9f4f955b71552080b6a0be10f66a2342cbd26927.


Done 9f4f955b71552080b6a0be10f66a2342cbd26927.


Agreed. Now it's `findShortcutName: (keyName, componentName)`.


Done.


I think you broke the tests with this one.


The `it` message talks about an *arbitrary* value (it should just be a defined value, we don't care what the actual value is) but the test case checks for a *concrete* value.

This test case can probably be safely merged with the case above while keeping the `it` message of the above case ("should expose the change event type as a static constant").


Is this how you could go about using `prototype` in this assignment?


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


Thank you for the great patch! Ideally, we would leave the top-level README.md file alone, because changing it means that I can no longer use the automatic merging tools for GitHub/git, and that I'll need to redo each patch manually.


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


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Here is where I call `.destroy()`


This patch includes files like `phpBB/ext/vse/.DS_Store`, which are actually artifacts of the Macintosh file system, and which we want to keep out of the source tree. I'll add `.DS_Store` to the top-level `.gitignore` file so that this doesn't happen again.

Also, for the sake of future maintainers, when add an extension or a language pack, I'd like us to include two things in the commit message:

- The version of the extension.
- The URL from which the extension was downloaded.


Would be cool if the test dir structure copied the lib dir structure. (`lib/components/shortcuts.coffee` + `test/components/shortcuts.spec.coffee`).


You should use double quotes here instead.


I don't check the code but, what change the player to 'claimed hero'? If nothing change it, you could claim the noblesse passes a fex times (maybe).


This file should be in the `_fixtures` subdirectory of the `test` dir.


Thank for your patch, I've checked and submitted.


Are these whitelines necessary?


Use the `*.spec.coffee` extension for spec/test files.


This change is probably not what was intended.
Selecting a class _value_ here transforms the problem into binary classification. So the default should probably stay None, to consider multiple class values.
But the gui element needs to be clarified...


The newlines are required here before and after `depends_on`. It should look like this:
```ruby
license :apache

depends_on :macos => '>= :lion'

app 'Inbox by Google.app'
```


While it doesn't affect the validity of the code indentation is important for other developers picking up your code.  You can even use Brackets to beautify with a single click ;-)


Terminology was technically not quite correct:
content directory != Medienserver: technically media server and content directory are two different things
renderer != Player: rather stick to the technical upnp term Renderer
share != teilen: teilen means to pass on sth., e.g. a hyperllink, freigeben means to make accessible
quit != Schließen: make it clear that we are going to stop services and free resources, not just close the current activity


When adding both extensions and language packs, please put every extension in its own branch based off `custom`. This allows me to consider each extension independently, and merge some right away, while leaving others for more discussion and testing. If you mix together multiple extensions and language packs in one branch, I'm forced to decide between taking all, taking none, or manually separating them out.

As for language packs, if you add several at a time, please feel free to put them all on one branch, unless some of them are likely to require further discussion, in which case you can put them on a separate branch. All the current language packs are fine.


I have a litte problem:

I am not able to create line breaks:

Code:

	...


This HTML is not valid you have put the forward slash in the closing anchor at the end instead of at the front i.e. it should be

`<a href="portfolio.html"> here </a>`

what you have tried to do is create a self-closing tag.  Get in the habit of using a HTML validator https://validator.w3.org/ to make sure your code is valid it will save you heaps of pain when styling later.


The python client has also been removed.


@jwgoh Yeah I feared that might become a problem with the README. I think from here on I'll just do development on a separate branch and keep master for the stable release. But in the meantime we'll all just suffer a bit.

BTW you should get on the beta :P (see #78)


I propose omit all OptionType/OptionValues stuff in prefer of typed properties, that could be variable by sku.
ERD: https://www.dropbox.com/s/hvfy3ghke9ofr18/Catalog.html?dl=0
Presence of different entities for option types & product properties is a bug of Spree architecture.


catch Exception and then remove "Throwables.propagateIfInstanceOf(t, Error.class);"


My mistake. @hhkbp2 also pointed this out to me in another thread.


Zanimljivo da switch radi i sada sasvim normalno, sve opcije poredane normalnim redoslijedom.. Ne razumijem.. :)
	switch (option) {
		case 1:
			Product monitor = createProduct();
			break;
		case 2:
			User first = createUser();
			break;
		case 3:
			// TODO implementiranje proizvodas
			// Purchase firstUserPurchasedMonitor = CreatePurchase(first, monitor);
			break;
		case 4:
			getAllUsers();
			break;
		}


ArrayAdapter


BaseAdapter


I will transition the four example files that have "EvaporativeCooler:Direct:ResearchSpecial" and re-run.  The only difference that I am expecting is due to additional reporting variable and sizing variable renaming.


Code looks fine. Still pondering the logic...


You are really limiting your site to Mac users here.  Less than 8% of windows users have the font installed by default.  I suggest you have a fall back such as

`font-family: 'Helvetica Neue', Helvetica, Arial, sans-serif;`

to support use cases where the font you have chosen is not installed.  There are other techniques such as the `@font-face` declaration that can be used to host your own fonts if you have purchased a licence.


Please move in to the Navigator.goToSettingsScreen.


In fact, option types were trying to compensate for the lack of choice_from_set_of_values type. Along the way, ignoring the fact that the options may be of other types, such as integer.


Normally we try to control the space around the elements with CSS as inserting `<br>` tags for controlling line spacing is considered bad practice.


@str1fe Why the need for `*args` here? It's not used in the method


@hildevestol I'm not sure if that is possible. I can do some research to see if it is


you dont need to compile es6 to [run tests](https://github.com/iamstarkov/generator-tiny-es-nm/blob/master/app/templates/_package.json#L10)


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


test comment


@teo No problem renaming all to license1Url, license2Url
I do wish to stay with buttons though, do not like how it is handled in the Welcome page with just text.
Is it possible to make the text on the buttons configurable?


test


Nope, it still looks out of place in that particular area though. Need to look at that.


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


I really doubt this horrible thing is platform independent:
http://stackoverflow.com/questions/50605/signed-to-unsigned-conversion-in-c-is-it-always-safe

In particular:
"Now we need to refer to (2) above. Your i will be converted to an unsigned value by adding UINT_MAX + 1. So the result will depend on how UINT_MAX is defined on your implementation."


Yep, not a problem. Still learning Git - I’ll try to them as seperately. It seems my branch was off my fork and not off custom.

> On 21 Aug 2015, at 14:13, Eric Kidd <notifications@github.com> wrote:
>
> When adding both extensions and language packs, please put every extension in its own branch based off custom. This allows me to consider each extension independently, and merge some right away, while leaving others for more discussion and testing. If you mix together multiple extensions and language packs in one branch, I'm forced to decide between taking all, taking none, or manually separating them out.
>
> As for language packs, if you add several at a time, please feel free to put them all on one branch, unless some of them are likely to require further discussion, in which case you can put them on a separate branch. All the current language packs are fine.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/egrapa/phpbb/commit/4a39becddb2b8c3a33937a5914c695125614d5cd#commitcomment-12827404>.
>


NPE if defaultThemeZip is null


is it safe to store this token like this?


This appears to remove the filters from 'My reports' pages?


Olha só, o texto ta bom, mas achei que falta um pouco mais de informações, mas ta beleza!
Outra coisa, suponho que Tu esta explicando a instalação no elementary por linha de comando certo?!  Bom talvez achei que seria melhor ou Tu especificar no blog que tu é um usuário linux que tudo que Tu escrever de instalação e tal será dessa maneira, ou então Eu acho que seria legal Tu explicar a instalação tanto Pra linux quanto Pra windows, o que deixaria mais interessante!
Só umas dicas! ;)


Hey perusio,

I'm experiencing the behaviour where this is just overwriting any of the values set before, basically rendering the map before obsolete.. Are you certain this behaviour works like you expect it to do?

Cheers!


why this change?


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


Moui... Je vais surement me faire traiter de macho, mais je n'affectionne pas particulièrement ces ajouts de forme. Ça alourdit la lecture, sans vraiment faire avancer la cause de l'égalité entre les sexes.


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


@samnazarko in my tests 10seconds is the best safe value. sometimes hdd needs 5, 7, 3. So with 10 I never had error with transmission and hdd


Thanks. I'll investigate and fix right away.


Nice!

I'm very curious...who are you?


OK -- this will delay startup by some time though.


They allow a short breather on the end of the file.


Why was this removed but not `_s_wp_title` as well?


I tried with both CURLOPT_SSL_VERIFYPEER and CURLOPT_SSL_VERIFYHOST disabled but it fails since we return CURLE_PEER_FAILED_VERIFICATION anyway.

Perhaps it is saner to get the cipher that was used with SSL_get_cipher_name() and check if it is a cipher that does not have certificates in which case no certificate verification will be done (for now only "PSK" as "DHE_PSK" isn't seem to be supported and "RSA_PSK" has a server certificate).


There should be a newline at the end of the file.


please change the url `s/ondrej-hosak/AVGTechnologies/`


Also needs a newline.


I used it in my code, but didn't check if it worked. But anyways, i can take a look and see if i can fix this.


It could be better to allow getTheme to return null and check it instead of catching a not found exception.


At the moment every module other than the filter/transformer ones is using the same value as both artifactId and name. I agree we can remove it, but we should do something like that as a conscious global refactoring in a single commit across all modules rather than buried in one commit at a time. I do think "http-source" is human readable FWIW. Also, regarding the linked PR, we are no longer expecting a module developer to configure the binding names, so the problem we had earlier with a space in the name when used for a broker queue name is no longer relevant. That said, +1 for removing all "name"s for now.


It would helps with https://github.com/tinyclues/saltpad/issues/84 but not for all possibilities. I really hope salt will fix their ACL system. I'd love to merge a PR for this commit.


import is not being used


Only who uses tvheadend and/or transmission


Your present implementation eagerly evaluates each operation. I mean that when I call `first(3)` you loop and select first 3 elements. But in lazy evaluation you don't do the calculation until required. So when a termination call such as `foreach` or 'max' or 'toList', is made we actually calculate the result of whole _chain_ of operations. So when user is calling non-termination methods you just create a chain in background by just creating a new instance of `FluentIterable` and wrapping present `Iterable` inside it and user calls other methods on it. Java `Stream` API and Guava `FluentIterable` works on same fundamental. Having said that it is not necessary that you create a lazy implementation.


Just a random visitor googled for "yes" translations ;-)


@matthew-andrews: Not sure whether this is what you're looking for but we have a pre-commit hook for running `make verify`.


Here's the usage of the isNaN-function stolen from Underscore.
This check means 'not a number'.
We could rename this function to avoid confusion.


优化：Messenger.prototype.listen 增加区分回调函数是否被重复被监听的功能。


test


wtf?


i thought there are no code change only version change
:beer


OK. 10 seconds it is :+1:


I don't like the repetitions I see in methods like boundMethodsExtraParameters() but I see no easy way around it, lgtm


It's not a good idea to use sync API here, use this instead:
```js
gulp.task('clean', function (cb) {
    del('lib', cb);
});
```


I wanted to test this out. But now have to find how to delete, (or uncommit?) things.


优化：所有Messenger实例的generalCallback都绑定在 window.addEventListener('message',generalCallback); 当页面发生postMessage时，会把所有实例的generalCallback都执行一次，这里用projectName+messengerName 作为key并添加到要发送的msg前面作为区分回调函数。


优化：因为prefix是公用变量，如果同一个页面创建多个Messenger实例并且projectName不同时，会导致msg = msg.slice(prefix.length); 得到非预期结果。


Clockskew; we wanted to tolerate some clock skew between nodes (15mins I think), and we made tombstone timeout 2x that.  Will file a bug to document and tidy up.


Yep, removed.


@enykeev can you take a look? That's supposed to work: outputs a base64 string and a valid (as far as I can tell) data-url. The image ends up broken though.


## A pair of functions that cache the inverse of a matrix
+
+
+## Creates a special matrix object that can cache its inverse
+makeCacheMatrix <- function( m = matrix() ) {
+
+	## Initialize the inverse property
+    i <- NULL
+
+    ## Method to set the matrix
+    set <- function( matrix ) {
+            m <<- matrix
+            i <<- NULL
+    }
+
+    ## Method the get the matrix
+    get <- function() {
+    	## Return the matrix
+    	m
+    }
+
+    ## Method to set the inverse of the matrix
+    setInverse <- function(inverse) {
+        i <<- inverse
+    }
+
+    ## Method to get the inverse of the matrix
+    getInverse <- function() {
+        ## Return the inverse property
+        i
+    }
+
+    ## Return a list of the methods
+    list(set = set, get = get,
+         setInverse = setInverse,
+         getInverse = getInverse)
+}
+
+
+## Compute the inverse of the special matrix returned by "makeCacheMatrix"
+## above. If the inverse has already been calculated (and the matrix has not
+## changed), then the "cachesolve" should retrieve the inverse from the cache.
+cacheSolve <- function(x, ...) {
+
+    ## Return a matrix that is the inverse of 'x'
+    m <- x$getInverse()
+
+    ## Just return the inverse if its already set
+    if( !is.null(m) ) {
+            message("getting cached data")
+            return(m)
+    }
+
+    ## Get the matrix from our object
+    data <- x$get()
+
+    ## Calculate the inverse using matrix multiplication
+    m <- solve(data) %*% data
+
+    ## Set the inverse to the object
+    x$setInverse(m)
+
+    ## Return the matrix
+    m
+}


:) I meant please explain this here, not in a comment. What other bug? Has this happened to you? In any case, this should probably be a separate PR.


保重保重保重，感谢你过去的付出，现在开始看我们的了


That's great - I just wanted to get an idea of whether it was a feasible thing to wait for.  Enjoy your vacation!


develop (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2302 of 2341 tests passed)

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


@sicarul because I missed that, good catch!


保重，感谢你的付出！


This should be set in the `constructor` as `@_platformName = …`.


extra space between times and "countDown"


Hi yyr,

Don't you get any error when updating WRF version beside of my netcdf error issued by me?
I have resolved netcdf issue with updating netcdf version, but there is no executable after compiling WRF 3.7. I will make pull request for this later after I try to compile v3.7 with different compiler version to see the result.


Is this the end of error #114 on linux?


This is not testable as the platform resolution logic is a static dependency. How can we test this without sacrificing API cleanness?


good catch


+1 for showing the type of the stream


makes sense as section, we should restructure them soon I think btw.


And the list of reports pages also, unless I've misread it.


awesome, thanks @dunn


Yup, thanks


Makes sense :ok_hand:


101682488_EvapCoolPrimaryAirSizing (Nigusse) - x86_64-MacOS-10.9-clang: OK (1787 of 1793 tests passed)

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


合ってる！eslintの設定をどうするかによるけど、コマンドの実行結果をエラーとみなす（bashの実行結果がエラー）ならTravisのビルドそのものがエラーになるね！

（今はエラーなしの状況なので検証しづらいね。とりあえずこのままにしておいて必要あれば後で対応で良いと思います！）


Or maybe...
`Events deleted using deleteMessages(toSequenceNr) are not deleted from the "tagged stream".` To explain that they *are deleted* from the normal collection, just not from the tagged one.


What is a reason ?


@AMDIvailo yes


It's really uncool that we have `build` and `rebuild` tasks. Can we do it the way it's done in testcafe? (https://github.com/superroma/testcafe-phoenix/blob/master/Gulpfile.js#L13)


\r-


Would you mind I pull this great feature to my project?


https://github.com/neo-shadowsocks/shadowsocks/wiki

Open source never die.


`implements WriteEventAdapter` then you don't need to implement `fromJournal / manifest`, they automatically get the "identity" behaviour.


`extends WriteEventAdapter`, which allows not implementing the `fromJournal` and `manifest`;
 behaviour remains the same as currently


Would make more sense as `isCancelled`.


I DERP EVEN THE NAME OF THE COMMIT


101682488_EvapCoolPrimaryAirSizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (751 of 751 tests passed)

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


@Syphurith Those functions should work in a way similar to ninjhax.
My final goal is to make a complete debugger (including both of ARM9 and ARM11) . The scene should be independent from other powers like Gateway.


@miltontony I have problem figuring out how to add a section or an article nothing worked out can you give me more instruction on this


git push


typo "CountdonwLatch"


Yeap, it's so windows-ish :mushroom:


is this trailing whitespace intentional?


Is there a reason why you don't just write this into the src="" attribute of your script tag?


I tried spying but it didn't work. Seems like these are failing randomly as well.

Would you like to jump on this branch for to fix this one?


trailing spaces


is the return type an array or a list?


The lazy evaluation is harder to tell apart from the builder pattern, that's why I chose to not implement it this way. Thank you for your explanation :) I did a pull request.


对不对比较


I think that's a bug, see https://github.com/bagder/curl/pull/392


Correction, version 1.8 actually


Don't think that Bokken Win32 makes a lot of sense, just Bokken is more logical. Due internal error the changes didn't come through (firewall)


Removed the version from the sourcefiles's Bokken directory.


Just for the record:
http://php.net/supported-versions.php


Yup, that was the only one. Removing it has no impact besides removing the WARN.


I need because I'm testing transpiled sources, not ES6 itself. Already discussed this in gitter.


Yes.  Before this it somehow was encoded in UTF-8 with the 3 leading UTF-8 BOM bytes which prevented it from compiling.

From: junwuma [mailto:notifications@github.com]
Sent: Friday, August 21, 2015 4:23 AM
To: SAFSDEV/Core <Core@noreply.github.com>
Subject: Re: [Core] Fixed encoding formate to Cp1252. (1512f94)

Hi Dharmesh, was there problem without changing its encoding? Thank you!

—
Reply to this email directly or view it on GitHub<https://github.com/SAFSDEV/Core/commit/1512f94384613f55eb27082ed6aa20a814471641#commitcomment-12824130>.


:+1: nice


How about doing
```
#ifndef BTN_DPAD_UP
#define BTN_DPAD_UP 0xMAGIC.
#endif
```
?


nu imi place executia asta aici. As prefera daca ai apela metoda in loc de `@data_params`

Adica sa faci metoda cum zic mai jos si sa folosesti `error_params['email']`


redenumeste in `error_params`


do you think if we put this as a dep of nbt it'd work?


schimba asta cu `@data_params ||= JSON.parse(params[:error])`


schimba asta cu `@data_params ||= params[:error]`


Forgotten print


Two more changes are yet be done,on what you have told.


Ezt az xbmc/vdr párost majd kipróbálom amint hozzájutok a gyakorlatban, hogy miképp működik.
Ha szerencse, akkor nem lesz gond, ha igen, várhatunk pipelka -ra.
https://github.com/pipelka/xbmc-addon-xvdr/commits/master


If s3_credentials[:s3_region] is a Proc, we'll end up returning the Proc.  We can probably just remove this line.


Any Mapped DST name must be set with all caps. If we are going to allow creds_dst to be set with lowercase we should probably $uc the rest to be consistent.


This has an interesting effect on the work packages table:
![image](https://cloud.githubusercontent.com/assets/617519/9408876/71fcf484-4816-11e5-9fbb-719aa25a9a0f.png)


_layer = 85125;
_exitWaitUntil = false;
if (isServer) exitwith {};
waitUntil {
if (currentVisionMode player == 2) then																		// check for TI Mode
{
	if (daytime <= 19 && daytime > 5)then															// check if player has launcher as current weapon
		{
            _layer    cutText ["Thermal Imaging OFFLINE until night time!.","BLACK",-1];										// Blackout screen with Text
			playSound "FD_CP_Not_Clear_F";
            waituntil {currentVisionMode player != 2};
            _layer cutText ["", "PLAIN"];
		}
	else
		{
	if (alive (getConnectedUAV player)) then																// check if player is connected to a UAV
	{
            _layer cutText ["", "PLAIN"];
	};
};
sleep 1;
_exitWaitUntil
};
};

I've been running this one for quite a while to allow TI at night. I also moved the init launch from the main init to the client init quite a while back. Main thing slowing my server is Australia dedicated server pricing and lack of headless client.

Cheers,

My teamspeak if you even wanna chat.

108.61.227.20:9357


are the outer most parenthesis needed?


Greg,
  What are you specifically asking me to do?  Just for your information, Steve and I do not do any pull requests.  Since Steve faithfully looks at all of my commits, I just address his comments by changing my code and then I make another commit and eventually merge back onto master.  I then create a new branch and start writing the appropriate tests.  I am asking this question, because I do not want to hold you up in your development cycle.
  By the way, the West End Ruby meetup went very well.  As you might have expected Steve blew everybody away with his "Mario" themed Sokoban game.  I even got a few compliments because Steve and I were the only ones who used a gui tool kit to render the game.  I was some what surprised because the vast majority of the programmers there are way better than I am.  One other note, other than Sandi Metz's game, Steve's implementation really illustrated a much deeper understanding of OOP than the way the other people approached the problem.  It could be that Steve just spent more time on the program (I am sure he did) but nonetheless the way he organized his code was dramatically different compared to the other programmers.
  Anyway, how did the meeting with John go?  Were you encouraged?  Did John give any indication that he might be willing to meet with you on a regular basis?

Eric


I'm in the process of adding in a fade ihnto TI if activated at night to the guy popping out from behind a rock and turning thermal on doesn't have an advantage, say 5-6 seconds for thermal to fade in while yours eyes adjust.


Hm, is fixed in a later commit


I will remove it, thanks


There's an extractor for types:

    case (s: Select) :@ (_: ScalaBaseType[_]]) =>


Couple of these threadId parameter docs have the words "id of thread" in the end which seem like leftovers. Might also help to split the lines so that they're not so long.


Hej Dan,

Good question, not sure how or why it works for you. I've been using ubuntu and debian in recent years, basically just out of the box, with plain old xterm (without any explicit configrations) as a terminal -- and these keys never worked. I must note that all other console applications I use in the same xterm work perfectly fine, so the terminal setup is probably fine, too.

According to the original code, Esc-O-H and Esc-O-F are interpreted as HOME and END. However, my xterm sends Esc-[-H and Esc-[-F instead of those, so my patch adds support for these escape sequences. I found that these are sent by other terminals (eg. gnome-terminal) too, so this is probably some kind of standard for vt100-family emulators. But I'm not an expert on that.

Neither has DEL ever worked for me in the Erlang shell. It results in inserting a ~ (tilde) instead of deleting the char. Even more annoying than HOME and END not working properly.

FYI:
$ echo $TERM
xterm

Can you tell me if your setting is different?


good idea


Extending the application form save code for accommodate file type
Including the field for resume and cover letter
The maximum  size of the file to uploaded have limited to 5MB
The format allowed are restricted to doc, docx, rtf, xls, ppt, odt, ods
Displaying the uploaded resume an cover letter for the back-end


you could also use `dbClient.closeSession` in the finally


@dragotin
DEB: we have /etc/owncloud for branded clients with shortname=owncloud.
RPM: we have /etc/ownCloud for branded clients with shortname=owncloud.

Setting [% sysconfdir %] to etc/ownCloud fixes things for RPM, but breaks things for DEB and vice versa.
I am using an inconsistent setting for now to build both.


This was called out as a deprecation in 4.6.0 (in the 'Feature' section no less), but was a breaking change. Ugh.


Separating the application form with next button and required JS code are added
The next page show the personal questions for the specified job
Creating the post type for questions
Adding the taxonomy for link it with the job will display in the second page
Creating new meta-box to add category of the questions.
These bunch of questions will be displayed with that job.
Displaying the question and answers in the admin back-end
So that the administrator can view all the question  answered by the candidate.


I don't think so. The package adds the hooks to the git repo it's installed within.


Teste


hello gives me the dsd can listen to digital voice.


What's the rationale for this change? ADL rules prevent the use of nonintrusive tile interface in tile_interface.h unless one of the arguments to functions lives in TiledArray ... this breaks TCC for example.


is interesting that finally is called before return function(xxx)


for consistency with above jq call, this should also be `jq <pattern> <file>` instead of `cat file | jq <pattern>`


I'd prefer a throwExceptionIfOverflow() style method name over the Guava style checkOverflow, but I'll not fight for it.


This was the problem.


Do you think it would be possible to do this using `If` with a `Value` (returning a constant) in the else clause? I think that would be clearer.

BTW, why ctx['_']?

In any case, a comment here would be nice - explaining this conditional


I would inline initialization with declaration, makes the fact that these properties are mock explicit


@matthew-andrews you could probably steal from https://github.com/bahmutov/pre-git/blob/master/install.js and do something similar in `nbt` directly.


Yes. And it might be good to add a comment saying why this is done otherwise people will wonder.


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


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


why is TEN and ONE a const and 5 and 0 are not?


Can any one please help me to import this kali linux image to GCE . I'm trying so hard with qemu to create a kali image for gce .


Bluetooth Share has been intermittently hanging on my Samsung Galaxy S4 running Cyanogenmod 12. Is your fix at all applicable to this issue?


didnt know about gitter. anyway, why not test original sources?


this sounds more like `PHP_VERSION_NUMBER`


$options is not used?


I don't see testing for this new feature - do you have a simple object file with DWARF v4 line programs to test this on (using the comprehensive readelf tests)?


`--silent` should be combined with `--show-error`


lgtm % comment


I don't know. I followed the style of the hilighters. Will change to name, seems more reasonable. Will change the hilighters aswell.


Was this change tested in a real app? I don't believe the only change required was bumping the version


no need to be public at class level if all methods are package protected


measurement_windows are tuples of time and length, right? If so, they are defined in the reference frame of the individual subtemplate and we need to add the total time of all previous subtemplates.


Maybe put this into a constant and use it in all relevant places instead of the plain string, which is prone to typos.


I think this should be `...:\$PATH"`, otherwise it will throw away the PATH config in the container which could be different from the build container


Except this contains the built `dist` directory, this is completely the same version as in here https://github.com/hawkular/hawkular-ui-services/pull/28


Yes, this should build an array of addons and not just a space separated list


I know this just fixes Dain's patch, but I hope he'll decide what to do with the roundBracketDepth == 1 meaning inLiteralParameters == true. Because the latter is more informative.


Looks like a bad merge. This feature is absent from `6.26.0`. Probably should go under `Next Version`.


Because I want to be sure that code that I have delivered to user will works. That's why I'm testing transpiled code.


Making the tocitem different for each toolbox may be more confusing. The features appear merged into raster, optical, radar, etc in the menu and the help should follow this. If each toolbox has it own highlevel item then the menu structure will need to be repeated for each toolbox assuming they put something at various levels.
We should make more use of the icons we have for each toolbox. In the help, the S3TBX modules could have the snap header with an added s3tbx icon on the left. Likewise operators should use the toolbox icons and have the toolbox name and funding in the about for that operator.


Minor nitpick: Use [`MoreObjects#firstNonNull(T, T)`](http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/base/MoreObjects.html#firstNonNull(T,%20T))


Ok, HOME and END works in gnome-terminal for me, but xterm requires your patch.

Thanks.


> This also means that we can catch errors in the failCallback too!

If one has errors in their `failCallback`, I think they are beyond salvation :stuck_out_tongue:

--
You could do it with separate `failCallback` and `errorCallback`. In practise I usually use the `errorCacllback` only, where I log the error and display a notification to the user (using either the failed response error info or a generic error message in case of an unexpected error in the `successCallback`).

But (theoretically speaking :wink:) your version is more "thorough".


Ansibleコード化おつかれさまです！
こちら、AmazonLinux等OS環境によってDockerは各種バージョンが入る、とお話されていたましたが、Weaveは(git.ioを使っているので)、何にせよ(今は)v1.0.2になるかなと思います。
これって、(Weaveが新しい分にはDockerが古くても)良い感じなんでしょうか。
なんか、開発が激しすぎてこのあたりの依存がよくわかりませんが、もしそのあたり何かトラブった、などあれば是非情報入れておいてください！


shouldn't "X" be a const? here and in other patches? you're referring to it in multiple places.


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


but, transpiling is the babel’s work, dont you trust babel and want test it itself?


Yes, I really am going to write this information in a TSV file. Right up until the time I get a chance to get it entered into a RDB table, or into the xml metadata files.


I would create a method `Group checkNotNull(Group group, String msg)` would make the method much shorter and avoid this mutable group variable


As it turns out, I got this mixed up when you messaged me about it.

This is a broadcast of our own player_info object to everyone else online. https://github.com/FAForever/server/commit/b78a644f47a2a1da89565bcc15c6ab37aefe8b38 addresses the issue.


k, thx


this method is not consistent, if any parameter is set, it throws a NotFoundException, but if none is set, it throws a NPE. Is that the desired behavior?


thanks. appreciated


You don't need to add `$loaded`, check for `self::$metadata === NULL` should be enough


Right, the other _dsts need to be $uc'd too. Also, I'm thinking about
handling the gaf dst so it wont be the static dst.

On Fri, Aug 21, 2015 at 8:08 AM, tim <notifications@github.com> wrote:

> Any Mapped DST name must be set with all caps. If we are going to allow
> creds_dst to be set with lowercase we should probably $uc the rest to be
> consistent.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/txace/icd10-sql/commit/5f130044832abffec4a51e684e0fb05b0c2ffb66#commitcomment-12828264>
> .
>


will throw a NPE, not a NotFoundException, is that the desired behavior? it's inconsistent with getGroup


should probably mention that the time is in miliseconds


Thanks, resolved these in http://git.io/vs0f7


What was wrong with having an accession number of the form YYYYmmddHHMMSS as originally suggested? Using a date-time can be automated, trying to remember which is the next accession number to assign is not so easily done.


why is the installer still interesting on the legacy path?


como hago el merge?


I think we should put here `set-aws bosh-delete destroy`. Or otherwise make a `terraform-destroy` task for the terraform part and make `destroy-aws` and `destroy-gce` call both `bosh-delete` and `terraform-destroy`

I say that because if you call directly `make destroy-aws` (which is the intuitive step) without calling first bosh-delete, the bastion host will be destroyed and you won't be able to run `bosh-delete` after... you need to delete the microbosh machine manually.

It just happened to me :)


Oh yeah it's a bit confusing... Maybe we can open another PR to rename it as`Util.isValidNumber` or similar.


The reason is that useradd makes it difficult to deal with the mkmailspool option. But maybe it doesn't matter.

Useradd, for whatever reason, has two types of settings: ones in /etc/login.defs, and "defaults" in /etc/default/useradd. We can change login.defs settings from the command-line, but not defaults. CREATE_MAIL_SPOOL is only in the defaults. So what I did to support mkmailspool was to change the setting we can, the MAIL_DIR path, to a temp directory that is immediately removed, and then if mkmailspool was True, create the spool file manually. That way the same end result is correct regardless of what is in /etc/default/useradd.

On the other hand, who cares. The only reason mkmailspool exists is for the kickstart sshpw command, so that the ssh user added to the stage2 environment doesn't end up with a mail spool. That, along with a couple other things, was causing issues with sshpw on RHEL 6. https://bugzilla.redhat.com/show_bug.cgi?id=602308

If /etc/default/useradd is missing, CREATE_MAIL_SPOOL defaults to "no". The lorax change to bring in shadow-utils does not install /etc/default/useradd, so the user created by sshpw in / will not create a mail spool, and the once the useradd chroot bug is fixed so that it uses configuration files within the chroot, users created in /mnt/sysroot will use the settings installed by Fedora, which has CREATE_MAIL_SPOOL=yes. So maybe the better thing to do is to just remove mkmailspool.


inconsistencies I pointed out below would not have been created if the methods contract had been written in a javadoc,  I think.

"Ce qui est simple s'énonce clairement"


src2[3] should be 1


This line can (and should be removed).

> There is no way to prevent the exiting of the event loop at this point, and once all exit listeners have finished running the process will exit.

from https://nodejs.org/api/process.html#process_event_exit


Вот так вообще отлично будет
телефон:+7 (495) 666-02-66


Thanks to @mikeNG


Whoops! Thanks, that was leftover from libuser running in a child process, and obviously I missed that one.


No, not 'might be stored' but in this set of scripts 'will be stored' unless you can tell me a reason to change the architecture. This is not a policy document but an explanation of how the software works.


They're not really secrets because all they do is enable the bearer to start a chat as a client of their system... They're not related to accessing the back end. The integration docs are minimal, but they're here: https://support.happyfoxchat.com/kb/article/324-install-happyfox-chat-widget-in-website


nvm, the non-intrusive interface is simply missing using declarations ...


I am not quite sure why I used exit status 1 here in the first place. If we receive a SIGTERM (like on Ctrl-C), we should exit gracefully (as we do), so this should probably just be `exit(0)`


101682488_EvapCoolPrimaryAirSizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1272 of 1276 tests passed)

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


Will do, though as noted in the mail spool comment I think I'll just leave out the mention of mkmailspool.


is this used anywhere?


Change the structure of this project


Thank you for adding this. Not sure how to use it though. What would be the html to use noUiSlider?
Thanks again.


This sentence is very unclear; it says neither where the problem occured, how to resolve it and how the BP is handling this problem automatically. Also PHP_VERSION seems to be undefined here


please send him this patch again so that he can incorporate that into his change


Hi, this changes it from heading to bold-face.
With all the intended subheadings intact, makes the post looks unorganized. Can you please change it back to heading?


я про эти переносы


this should be `$PHP_CONFIGS_PATH/010_php.ini` for consistency


is it replaced by anything?


It allows users create non-intusive interface function that lives outside the TA namespace. So the non-intrusive interface functions in TCC should be in the same namespace as the function arguments.

The following example shows how the interface function, `g`, should be defined.

```
namespace X {

    class A { };

    void g(const A& a) { }

    namespace Y {

        template <typename T>
        void f(const T& t) {
            g(t);
        }
    }
}

namespace Z {
    class B { };

    void g(const B& b) { }
}

int main() {
    X::A a;
    Z::B b;

    X::Y::f(a);
    X::Y::f(b);

   return 0;
}
```


## This is a markdown file


If root + homedir is /, so for the sshpw user. I'll add a comment noting that.


In Permission WS


Linija viška.


Minor aside: `startsWith` rocks :ice_cream: , but is only implemented in Chrome and FF atm. That said, until IE, Opera or Safari land Web Bluetooth this really only will work in Chrome. Deferring to Jeff's review for whether to keep the ES6 code.


Tab desno.


can we call MethodsGroup -> MethodsWithSameExtraParameters, this doesn't sound good either, but MethodsGroup is so generic that it's doesn't convey any information, why are they grouped, on what basis, etc ...


This appears to be wrongly indented and should be inside the `{% if object_list %}`


As @jimconner mention to me, that will be an issue later anyway because when we deploy CF a lot of VMs will be deployed later...

Probably the solution is out of the scope of this PR, but I would suggest that we do the creation of the microbosh outside of the bastion host and handle the deployment fails in git or similar.


101682488_EvapCoolPrimaryAirSizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1793 of 1799 tests passed)

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


I would document this method


I didn't want to change the behavior of the legacy build. This will be anyway removed as soon as we change the default build to customphp versions.


You could probably create an input mapping here which creates the variable 'childProcessVariable' in the sub process scope. This way it won't get porpagated further:

```xml
<camunda:inputOutput>
  <camunda:inputParameter name="childProcessVariable"></camunda:inputParameter>
</camunda:inputOutput>
```


感谢！！


Could you @merlin83 or @rubik please explain why you are closing the DB explicitly here? You did this already on the old background task project...


Yes, because we don't use the `shlex_splits` here - we can't because shlex.split() strips whitespaces we need to preserve. I'll make a comment in code.


This is improved in a later commit, should I still change it here?


@tosovic Mnogo čudno izgledaju naše slike na dnu, bolje da ih skroz uklonimo.


this class could really use a method `<T> T checkNotNull(T value, String message)` which would throw a `NotFoundException` with the specified message if first arg is null or return it


well, I can add javadoc in the PermissionRequest class. Moreover, I'll add a hasGroup check in the PermissionRequest which already checks there's at least the group id or the group name and not both.
I'll remove the checkForNull in the end which cannot happen


It's now in v 1.0.5 release


@tvsonar this class is getting huge, and it seems to re-implement a lot of what the analyzer already contains


TeamCity pull requests :: netty [Build 91](https://garage.netty.io/teamcity/viewLog.html?buildId=635&buildTypeId=pulls_netty&guest=1) is now running


k, thx


101682488_EvapCoolPrimaryAirSizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


well it's not possible to have a NPE normally, but I'll make the code more explicit. I was thinking about SQ analysis. Nevermind


I see, but I find mine expression more understandable :-)


OK, so we still execute the call here, so the callee will do a OP_RETURN which will pop the frame stack, no? Isn't this a pop without a push, or am I missing something?

In general, why not implement this in the compiler rather than in the VM? Then both VMs could enjoy it and it would also be more efficient - because when the VM has to do this peeking business it also slows down, so it's really a question of what's more expensive - peeking and this condition or just executing an extra stack push/pop.


`-z "${PHP_VERSION}"`


you mean the legacy gaf dst in lib_DX10:getPreviousGaf?

I'm not sure how to handle that one. We'll have to either

a. map in the main script and pass the mapped variable into the function as a variant
b. pass the mapping into the function and have the function map prior to the read
c. have some kind of initializing function for the library to gets and sets the mapping.

c. is what I did for the sql credential settings, but I'm not sure that it makes sense to do that for a couple of dsts in one function.

I think I'm leaning towards option a at the moment...


TeamCity pull requests :: netty [Build 91](https://garage.netty.io/teamcity/viewLog.html?buildId=635&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:02


closes #67


Some of these numerical instability issues only show up on older numpy - I think version 1.7 or 1.8


Fair enough. :-)


ComponentFinder already checks and throws a NotFoundException if the project is not found.


I'm setting the value of "Otros" when the `id_document` is not present in the record. This is required because the hash need this value.


final keyword seems useless here


I asked here, still no reply: http://news.gmane.org/gmane.linux.kernel.initramfs


I actually googled it out after I had decided that I'm giving up and to be honest, neither I understand it :-( but we have test_split() test case which proves that it does what we need.


when does it get published in the maven repository usually?


Addressed all of @psigen's comments in https://github.com/personalrobotics/prpy/commit/3b296a26c00fa4303a02ae91a0257a1a4194514a.


 The doc section update

- Logo
- Favicon
- The CSS update
- Some markup update to make it look similar to the doc. site


include a copy in the app


when I click da magic button.


which I just clicked :-P give it 10 mins


Ping @r-a-y. This changeset is utterly awful, but it made me think of how you couldn't use the script due to its dependence on php-svn. That dependence has been removed :)


yes, I understand. Without using TiledArray::xxx the default non-intrusive interface implementation is not considered when using custom tiles living outside TiledArray. See the first batch of changes (it only fixes enough to make TCC hf_l_inv example work) in 794a843e8982079846a79f002e5c68b0503d3599 ...


this retrieves all users to the java and nothing is done from it (just iterating on it with a simple test).
This could be replaced by a SQL query with lots of performance benefit


We decided that the last parameter was not needed anymore, hence removed it.


This should be named TORTLS_PRIVATE to follow the pattern from the other files.


Deleted the line.


I'm not technical, so this is my understanding. It seems that Django's i18n features are one part of the internalization process if you need to translate field content, which Django i18n doesn't do without adding code like the one you included in this documentation.

Maybe it would help non technical like me to mention this somehow.


`--show-error` again


I think you forgot a `tag` here.


is this already not the response object? Why does it need to return a value? Also, address is missing a description


1774_add_misc_objects (mbadams5) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


except for new-lines....I can't imagine a tinier commit!  Although the effect of this commit is more than a newline!


Corrected.


corrected.


k, thx


I filed bug 1197197 to split this schema into separate files since it's not possible to understand which action requires which parameters. I guess we can live with that until that's solved.


Protocol only uses the term Array for all collections and it identifies how it is encoded/decoded.


Nice to have user love this.


so mb then just expose LABELS as `hot.widget.labels` (yeah, i remember what we need to come up with the better name) so user can override key labels then?


What does KB means in this context, kill bugs?


It skips the push, but also skips the corresponding pop (since control never returns, having not pushed a return address).  So it's correct.

Good point about the efficiency, though..