I am in support of this change. I've added an explanation to #4435 explaining some coincidental feedback that I got yesterday before hearing of this change. I am pleasantly surprised.


I am not sure about this test. I can imagine that we run test with `isDebug=false`, but being still in the dev mode, before release to check if everything works file without debugging code. Or something fails with `isDebug=false` and you are looking for the reason. Then you need to remember what this test do and why it is red.


``is_splitter`` could be used as well :)


I think I would remove `$country` from the constructor arguments and pass it instead to `getAddressFormat`.
This way, every object in the constructor is a dependency that could be injected (`Country` is not really a dependency in the sense that it is not a *service*).

To be extra clean, you could also create an `Adapter_AddressFormat` that you inject in the constructor instead of calling `AddressFormat::getOrderedAddressFields` etc.

With the 2 changes, the Adapter_AddressForm could be moved to the new Core and **unit** tests could be written because all the dependencies would be easy to mock.


spelling


I'm not sure if I should open a bug ticket...

There is what is just tried.
  - Removed sysPass's folder
  - Dropped sysPass's database
  - git clone https://github.com/nuxsmin/sysPass.git
  - Set owner apache (www-data) on sysPass's folder and sub-folder
  - Set permission 750 on sysPass's folder and sub-folder
  - Opened chrome  and get to the sysPass's home page (default language: Spanish).
  - Filled the form --> "Instalación finalizada"
  - Logged in, get to setting, set language to English, get to Admin profile, set language to English
  - (Still Spanish) Logged out, logged in.
  - Still Spanish

git log --abbrev-commit --name-status HEAD^..HEAD
commit c3675d7
Author: nuxsmin <nuxsmin@syspass.org>
Date:   Fri Oct 2 14:44:19 2015 +0200
    * [MOD] Update translations

Versión sysPass	1.2.0.03
Base de Datos	SERVER_VERSION : 5.5.5-10.0.20-MariaDB-0+deb8u1
CLIENT_VERSION : mysqlnd 5.0.11-dev - 20120503
SERVER_INFO : Uptime: 84264 Threads: 1 Questions: 6261 Slow queries: 0 Opens: 78 Flush tables: 1 Open tables: 105 Queries per second avg: 0.074
CONNECTION_STATUS : Localhost via UNIX socket
Nombre: syspass@localhost
PHP	Versión: 5.6.13-0+deb8u1
Extensiones: Core, date, ereg, libxml, openssl, pcre, zlib, bcmath, bz2, calendar, ctype, dba, dom, hash, fileinfo, filter, ftp, gettext, SPL, iconv, mbstring, session, posix, Reflection, standard, shmop, SimpleXML, soap, sockets, Phar, exif, sysvmsg, sysvsem, sysvshm, tokenizer, wddx, xml, xmlreader, xmlwriter, zip, apache2handler, mysqlnd, PDO, apcu, curl, gd, imagick, imap, intl, json, ldap, mcrypt, memcache, mysql, mysqli, pdo_mysql, pdo_sqlite, pspell, readline, recode, snmp, sqlite3, tidy, xmlrpc, xsl, mhash, apc, Zend OPcache
Memoria Usada: 768 KB
Usuario: www-data
Servidor	Apache/2.4.10 (Debian)


Well, allow it anyway. Used in a WIP.


I think we mentioned this on IRC, but I think this is supposed to be 3 * the number of indices, so for now it should only be 6 (instead of 9).  Same thing in `DynamicMemoryUsage` which is already in master.  So I'd either change to 6 or add the expire code and the time index.


Good point. I can (or you can) remove this test. Let's not make any relation between these two.


Remove it then.


:+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1: :+1:


Shouldn't it act only on specific actions that are scheduled?


@bjansen this might fix your problem. I have not tested it.


K


Just as reminder, do not forget to reload the correct address format for the form when changing the country in the selector. enjoy :sweat:


Oberer, denke ich. Was machen wir mit dem Unterstrich? (ich glaube, der ist für Tastaturkürzel, kann das sein?)


Yep


Not sure why this changed...I thought our coding standard prefers single quotes in JS?


Merge


See issue #11.


you're right. I will change that.


->condition(a.entity_type, 'node') - just in case we have taxonomies or users linked to treaty ..


Idem ;)


Super :) Alles so schön einheitlich jetzt!


Once again. Broken indentation.


@gitwillsky 这个地方不能删，不然进程回收不了。


Leaving a sample database.yml file is fine - for your colleagues. Original database.yml should be in .gitignore and not shared but sample should be harmless.


Maybe my understanding of Javascript is off, but couldn't you just assign the props to a variable in the scope of the module, and forgo the memoized function?


Why didn't you call it _outputAVStream ?


@debonair -To confirm - is this for #197 or another task? Just making sure if it's for 197, I thought we agreed to leave this out for now?


我是在这里回收的。


@IIvec
2) Yes, 500 is a little too much. :) Some logarithmic formula could work better for sure, but tuning would take an enormous amount of time. Right I think the important thing is to get some long time control tests with the formula that works for short time control.


Ahh, yes, of course, `bl_do_peer_delete` even checks for `case TGL_PEER_ENCR_CHAT:`, so that's probably how it should be, right?


The above fix is only working partial is changing the php formatMoney function but not the javascript one from script.blade.php so the currency in PDF and invoice edit page is not modified.


its always better to use `create!` in `seeds.rb` rather than `create` because the bang version also pops an exception and notifies you in console while non-bang does nothing.


 - __portability__: [include/chaiscript/utility/json_wrap.hpp:15](../blob/6a4647af43040db97b33aa16f4b2dbb5be11566d/include/chaiscript/utility/json_wrap.hpp#L15) The extra qualification 'json_wrap::' is unnecessary and is considered an error by many compilers. (unnecessaryQualification)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/6a4647af43040db97b33aa16f4b2dbb5be11566d/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/6a4647af43040db97b33aa16f4b2dbb5be11566d/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/6a4647af43040db97b33aa16f4b2dbb5be11566d/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/6a4647af43040db97b33aa16f4b2dbb5be11566d/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/6a4647af43040db97b33aa16f4b2dbb5be11566d/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


@omarwanis @MohamedIhab review to merge 3al master.


Mara.


@danfuterman ignore this, I just wanted to push the branch to github for when we want to do the changes


Added a read me file.


@WHPODOO FYI when the `string` of a field is missing, it is set to the (capitalized) field's name.


what


Once you've taken up for reworking logging suggest to make it consistent: somewhere we use prefix _ParallelsDesktopXXX_, somewhere _PDXXX_.


There is a small typo here


ould you call it something like "_wrappedPacketsDuration" ?


This could rather be `%h3= link_to category.name.capitalize, category`- no need to interpolate and you could use shorthand named routes.


@gitwillsky 哦，看到了。

我合并进来，部署到线上跑一下看看，辛苦了！


thanks!


This is not the official unomp repo, The changes being made here are still incomplete and this repo is always changing. please clone https://github.com/UNOMP/unified-node-open-mining-portal instead and if there is a problem open an issue request.


Quick hack here, didn't see need to actually _name_ the class, but normally, I'd always name/namespace as needed.


hey, OK for this time but start also writing everything in english ;)


Are you only wanting to log this information at the error level, or would you prefer it to be info?


You removed the inline style but didn't put it inside the css class.


Hmm.  This doesn't solve the issue universally.  The `Parser::parameterSplit()` function will only work if it is only a function inside a function that accepts comma's - but it won't work if there's anything else inside of it. If we assume the parameters of the function have to be static, ie the `$start` and `$stop` of the `sum` function, a more elegant way would be to `explode` it out and then `array_pop` off the last two elements and `implode` it back together for the function.  However, that may not work for every use-case so I need to think of a solution that will allow it within all parameters. This will probably end up being a function of the `Parser` class, though more similar to how functions are checked within `Parser::solveIF()`


I guess it does not hurt to check ``scanner`` and ``*scanner`` first.


@dmvstar Hey, thanks for the tips!
I started working on it yesterday and I left it unfinished. I will push the full changes today.


Why are you throwing a SocketException? There is no such exception thrown in this method.


Will be fixed shortly.


Hi guys!
use macros GPIO_ACTIVE_HIGH or GPIO_ACTIVE_LOW (from /include/dt-bindings/gpio/gpio.h) to avoid to write the comments /* active low */ or /* active high*/ , like so:
 rst-gpios = <&gpio6 5  GPIO_ACTIVE_HIGH>;
 ;-)


There's a better solution using constant extra space.


Actually I'm just an Arrested Development fan. No innuendo here, I'm sorry to say. I don't think I knew who Tobi was when I wrote this. 😛


rather with a ```s``` as the function will validate the all list


Recommend using a for each loop here.


@bocharsky-bw this is a tricky challenge - the correct answer wasn't quite right the `$crewSize` property must be in `Deathstar`, else `setCrewSize()` doesn't set it. It was passing the grading, but the `crewSize` was null for `Deathstar` classes (the setter was doing nothing because it could not access the property). Unfortunately, setting non-existent / private properties in PHP does not throw an error :)


recommend using StringUtils.isBlank()


Thanks @theshka! By the way: I'm always updating the PR message when changing something, this is what I wrote about these commits:
> Initialization of Pico now works completely different: rather than defining constants (which are probably conflicting with other applications...), Pico now expects its paths to be passed as parameters to the constructor. The constructor doesn't start Picos processing anymore, you now have to call the `Pico::run()` method, which returns the parsed page contents instead of directly `echo`ing them. The `PicoDeprecated` plugin still defines the now deprecated constants `ROOT_DIR`, `LIB_DIR` etc., so old plugins can still use them, but users which are using the constants in their `config.php` (e.g. `$config['content_dir'] = ROOT_DIR . 'content/';`) unfortunately will receive a `PHP Notice: Use of undefined constant ROOT_DIR - assumed 'ROOT_DIR'`. In return, relative paths are now interpreted to be relative to Picos root dir, so the user just has to remove the `ROOT_DIR . ` part.


(269be91) The title of the commit doesn't respect guideline ("type(scope): title (refs #<number>[, ...])")


Jetzt passt alles. Danke!


You can do this in O(nlogn) time.


@lcalvy if we remove this, the audience value never goes to the `getSamlResponse` method


Weee!


When you reverence static string values like the forward slash, I would recommend getting the value from an existing class such as IOUtils.DIR_SEPARATOR


I believe this ignores the excpeiton Message and prints only stack trace.


Awww! It was too good to be true, haha.


Looks much more Erlang‘ish ))

Syntactically, I‘d better write one `json_req_obj_field` definition with `case firstArgument of <<"term1">> -> action1; <<"term2">> -> action2` and so on. For me it will be more readable.

But right now I‘m unable to estimate how different syntactic constructs are optimized by compiler. I just don‘t know yet what is better. Are multimethods in general more effective, then `case of` inside one method?


Nice line, it gives a little more of personality to the project!


Is the build number still 1.9.20 or 1.9.21 ?


Will this work on windows? A path can contain `:` e.g.: `file:/C:/Windows/`


@ChaoKong  remove all these added by Clarence and similar msgs


suggest oai-pmh (since they use that in their own docs)


I am not sure why you are using a while loop here. There is really no need for it.
This section of code is rather complex, and would recommend simplifying it.


It doesn't? That's odd..


The BnP feature, as it works now, was introduced in 1.9.20.

The intent of this message is not to always be changed to the latest version, but rather to point out the minimum version required for BnP HA to work.


You need to check the length of 2 strings.
"aaab" and "aaaah" will fail in this program.


@ChaoKong not a correct way of naming variables: testIntermediate2


Updated file for gradle conversion


Nice, only next time please either mark the test as `@unittest.skip` when you add it before the code, which makes it work. It's harder to bisect problem when different commits break the testing... (I know we don't do this 100% correctly, but adding unittest which always fails hurts my eyes).


@markmhx Okay, I think I figured it out...
Here's a form declaration for the album form. As you can see, not a whole lot goes into it - just the url of the page to go upon form submission, a token for tracking submissions, and an optional display method for the form fields (e.g. "as._p"). Form forms that involve photo/media uploads, the enctype declaration is required.


@ChaoKong  broadcastIntermediateMeasurement -> broadcastIntermediateResults


The same here


This is a nitpick on the old code. Do you want an integer or int ?

Also I believe randomNodeId should have been retrieved by the new Random().nextInt(liveNodes.size());


hostend -> hostent


asd


If you have not already; you may want to add this domain to the list of permissive domains.


you may need to call completePendingCommand to correct set the server response settings.


Making note for Mark to see how album form was constructed in albums.html.


Looks good to me. If this going to be a regular thing, we should figure out how to propagate this information from the LibraryType table.


D'oh. It's not actually needed anymore. I'll clean it up in a new PR


I am curious as to why you are returning a string here.


Hmm, this is a very interesting way to approach the feature. I like it. What do you think about removing the brackets from the ref values? So that you would have `ref = foo` instead of `ref = [foo]`? It seems like they are implied and unnecessary.


ah good :-), I will use what Shiro consisers : classpath: or url: and make check on that


@ChaoKong IM_measurementResult -> intermediateResults
Use lowerCamelCase for methods/variables. UpperCamelCase for classes
https://en.wikipedia.org/wiki/Naming_convention_%28programming%29#Java


Does this actually work now/is required now? The `openvr_driver` file used to contain overlapping definitions because they shared internal headers inside Valve's perforce repo presumably.


I was going to ask if we could remove this. Glad to see it's gone! :+1:


I'd say these macros should go below definition of struct nwrap_vector.


Correct, but greedy is good enough.


Yep, have condensed into `gulpfile.js`.


@ChaoKong remove these unnecessary comments


and here too. It's better to attach the unittest changes in the commit, which changes the behavior, or to temporarily mark the unittest as `skip`.


No, this is a new utility function I added (see below), which returns the full stacktrace (exception's message and stacktrace).


Yes, especially, since this might have to be used after window operations. To assign new timestamps and emit watermarks.


Might want to put in a doxygen style `/// @todo your comment here` here so tools can pull it out easily


hmmmmm, why commented @artemp?


Downside is it's a lot longer than the previous 15 lines, but I think it makes it a bit clearer as to what the tasks are and what they're doing.


openvr_driver.h uses the IVRSettings class, which is declared in openvr.h; if I don't first include openvr.h, the class cannot be found.


becaust -> because


Should I do this to all your TODO comments as well?


dsf


fun fun fun, that would be another change then. A good one, in this case, but still.


I am also curious as to why you replaced all the getter methods that were original in the file. As well why you added the setter methods.


:smile:


Shouldn't you check for NULL here? strdup() may fail.


if you guys are keeping track of module versions in the initial README (or want to leave it as an exercise to the reader, the error message is clear enough):

ceylon run: Version 1.0.0 not found for module com.example.helloworld
Try one of the following versions: 1.1.1

Line 153: https://github.com/ceylon/ceylon-dist/commit/d286a1e0bbda5b0e5d8ce1d44c047576f47b2ea4#diff-04c6e90faac2675aa89e2176d2eec7d8R153

Should be ../../bin/ceylon run com.example.helloworld/1.1.1 John

Line 168: https://github.com/ceylon/ceylon-dist/commit/d286a1e0bbda5b0e5d8ce1d44c047576f47b2ea4#diff-04c6e90faac2675aa89e2176d2eec7d8R169

On my system produces (which may not be intentional):

Warning: Missing documentation for default
Warning: Missing documentation for
Warning: Missing documentation for run(noModule.ceylon:16)

PS. nice job making the compiler distribution simple to build from sources.


There is no need for it as `art_tty_backend()` is already opening the device file in nonblocking mode.


Excellent change!!


А он там нужен?


See issue #12.


`math` -> `meth`


This is the commit that I would want to merge.


same in when using `autopty`


Thanks, Ashkan. I will modify them later.

Best regards,

Chao Kong (Clarence)
PhD Student, Computer Science & Engineering
University of Michigan, Ann Arbor
Email: chaokong@umich.edu

On Fri, Oct 2, 2015 at 10:50 AM, Ashkan Nikravesh <notifications@github.com>
wrote:

> @ChaoKong <https://github.com/ChaoKong> remove these unnecessary comments
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ChaoKong/Mobilyzer/commit/47960a06d6c9f7c474c529a858e00da3165d1d10#commitcomment-13563950>
> .
>


`Requests` library performs location redirection by default.


Basically, no differences on bytecode level, it's more about [syntax](http://learnyousomeerlang.com/syntax-in-functions#which-to-use), especially when you need match by multiple arguments. And one less indention level (:


It is not supported by the backend, but I had it already implemented and I'm keeping it perfectly updated till the day it is enabled by the backend. I know I can remove it, but so far it has been less effort to maintain it than to remove it.


Yep, seems correct.


This code should be on ``nexteuropa_multilingual`` feature, like on the commit https://github.com/ec-europa/platform-dev/commit/d2968dcb95c2c66a44a29e6b21329e8147d8f5f6.
You can keep the second language in this module if you want, but the code for the default language must be on ``nexteuropa_multilingual`` feature.


I actually glossed over that little fact :open_mouth: , I'll keep referring to your original PR message when changes occur in addition to working out the changes manually! Thanks again @PhrozenByte


**no-unused-vars**: locale is defined but never used


**no-unused-vars**: showDropdowns is defined but never used


**no-unused-vars**: startDate is defined but never used


**func-names**: Missing function expression name.


**space-infix-ops**: Infix operators must be spaced.


**require-jsdoc/require-jsdoc**: Missing @jsdoc for ordinal


**yoda**: Expected literal to be on the left side of ===.


**no-var**: Unexpected var, use let or const instead.


**no-unused-vars**: drops is defined but never used


@grobian Is this should be `*blackholed = 0` ?


):

It still gives you a large warning when you try to open a save, telling you how to update. If the OPS format actually changes, it will prevent you from opening a save in that case (but the format never does ...)

Anyway, my mod has let you open saves from newer versions for a long time. It doesn't really have any problems. I think this commit is useful for future proofing reasons.


Yes, it was. But the previous `untyped` was not doing half of the things this `multifaceted` does, so actually both names were correct at the time being. Now it is `facetedSearch`, as it is how we call it in the platform :)


The idea is to try to avoid coherence issues when the cached zpool.cache in the initramfs is out of sync with the zpool.cache on the pool. If you were to add a new top level vdev and reboot with a stale zpool.cache in the initramfs, the pool import would likely fault. This would succeed.


_between_ might be misleading, perhaps `len(pool) -1` instead?


Ok I will change it to your suggested code.


Probably want `'s` after preconnect URL.


In place!


Here you actually want to use https://url.spec.whatwg.org/#concept-url-origin.


-> Remove the object to which this handle refers _to_ from all workers.


@springmeyer - user error sry :)


@markmhx this is where I use Django's built-in ModelForm to create my form. I save all forms for the site in this forms.py file. ModelForm allows you to simply create a form by modeling it after the models that are defined in models.py. The attributes in the model are converted to fields in the form. I'm now learning that there are these widgets that allow you to fine-tune how ModelForm creates your form, so you can overwrite some of the default form options such as type of field, field label name, etc. I believe that this is also where I should be able to tell it not to use a drop-down list of uploaded photos, like it does by default. What I really want is for the form to ask the user to upload a photo (starting with one, just to get it working, then I'll eventually enable bulk uploads), instead of asking the user to pick from a list of previously uploaded photos. Somehow, by declaring the "photos" field as a M2M table in my models.py, this form is linking the photo upload field to the other photo upload form on the home page, in a way that I don't want.


Hey @MrStaticVoid ,

I can't take credit for the idea since it was ryao's haha. I just implemented it. You are correct in that it is useful for multipath. Personally I don't use multipath but having this implementation benefits the user in a lot of other ways. Primarily the benefit of this is to have one location for the zpool.cache file. Having multiple copies of the cache file (one in initramfs, and one on actual system) is not good and will increase the chances of the cache files being out of sync and causing boot errors or other unforeseeable results.

Another benefit is that any activity you do in the actual system is automatically available in the initramfs without having to remake the initramfs. For example, I have a v28 bootpool so that I can use GRUB 2 zfs features and I keep my / pool in the latest v5000 0.6.5.2 state. So I can upgrade my root pool to the latest zfs on day 1 and not have to worry about my system not booting. So for this I have: 'boot' pool, and 'tank' pool. In the live system, I can export or import the pool either by-id or w/e, and when I reboot, this exact information will be available to the initramfs environment.

Another important feature is the 'refresh' feature that I implemented. Basically this ignores the cachefile in the live system, scans the system for the root pool during early boot, and then imports it, copies the new cachefile to the live system, and boots. This is good if you want to regenerate the root pool (Since you cant do this from the live system since it's being used).

The same exact code above can be used in the situation where the root pool doesn't have a cache file. Let's say you installed Gentoo on ZFS (or another Linux distro) and you forgot to copy the zpool.cache file, this might be a problem in certain situations. However, bliss-initramfs will detect that your rootfs doesn't have a cachefile and will copy the one that the initramfs creates into it. Which again, will mean that the initramfs can now re-use that initramfs during the next boot.

So overall, there are many benefits to having the zpool.cache file available, it's just a matter of having the correct implementation and usage of it.

I hope this explanation helps.


the email should be configurable and potentially the other config parameters too.
We usually do this by passing in a properties instance, e.g. https://github.com/gbif/registry/blob/master/registry-ws/src/main/java/org/gbif/registry/doi/DoiModule.java#L28


Actually, I was super messy (and I forgot we didn't teach abstract functions yet). Here's the final version: sha: 84f1c911bd165d73b664bf5750c392988ecb364e


제보 감사합니다. 하지만 작성하신 코드와는 다른 방식으로 구현하도록 하겠습니다.


Done


감사합니다. 반영하겠습니다.


@ryao Due note that bliss-initramfs already supports cachefile and /dev/disk/by-id at boot :D


:+1:  on listing merge conflict file


I don't think this will work in the view because the element that it needs to bind to doesn't exist until after the dialog is opened (hence why its in `afterOpen`). Is there another way we could handle this?


@rodrigopavezi typo here. Thanks.


Each record has the following structure:

<rdf:RDF>
<rdf:description>
....all DC elements nested here
</rdf:description>
</rdf:RDF>

I didn't include those parent properties in the mapping, not sure if there is a better way to handle it.


Thanks @waynatali !!! Yes, that is a good method to explain what's happening. Better change that.
;)


As that config class does not provide good defaults, maybe offer 2 constructors. One that takes a cfg instance and one that just takes the email adress and uses default values?


@ChaoKong Don't commit all unrelated modifications at the same time. Commit when ever you have fixed a particular issue and write meaningful commit message for that.


AW> if we upload files in GB, some time it fails. So gave option to retry configured number of times.


AW> I think it should be warning.


Fair enough; just wanted to confirm your reasoning.


It looks like the OpenVR headers use include guards around each internal header to prevent overlapping definitions from being a problem.


you forgot the gruntfile


more like 2swad


ok


@MiroCillik tady to bude chtít ten Unit taky vyhodit


@alex-deaconu good catch. cheers


Hmm, this code used to work before. Did something change in a new version of jQuery? Can you create a unit test to validate this fix?


fyi @marc-jan


@CaCTuCaTu4ECKuu, жаль, что тянет за собой много вспомогательных функций.
Хотя на самом деле его нельзя использовать часто: users.search начинает после нескольких десятков запросов отдавать пустой json.


Looks fine to me, but might be worth just using comments a little more aggressively to add to the readability.  It wouldn't be perfect, but it could at least make it a little easier.


Hoppas inte det tog för lång tid att hitta. Som jag skriver i kommentaren, jag kunde inte kompilera om denna rad var med. Får se om det går bättre nu... Sorry.


Maybe keep _Jasmine Centre_ as the first line of this address?


This will be sent to browser. And browser will use this path to send confirmation request which will move the file to permanent location.


@mieliep I restructured the contact pattern slightly. Will go through it with you in person on Monday :)


Wouldn't it be much more natural to do this:

#define nwrap_vector_get_head(vect) ((void *)((vect)->items))
...
create  and fill vector
...
he->h_addr_list = nwrap_vector_get_head(vector);
...


This needs to be in componentWillMount because pendingLayers needs to be setup before render is called.


Oh, sorry, I don't know why I didn't see that. I'll try to fix it today.


I'll add it back in.


ok

On Fri, Oct 2, 2015 at 2:49 AM, Bela Ban <notifications@github.com> wrote:

> I don't want the System.getProperty() call on every invocation of
> handle(). I'll move that into a static initializer.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kedzie/JGroups/commit/a6a4f1d7e569201ccaff71c40e024a9982e24096#commitcomment-13557076>
> .
>


We should remove it then. Earlier code version might need this exception to check. And I forgot to remove its declaration.


I would really prefer not to have that, it makes absolutely no sense for anyone else working with the repository. Please exclude it from pull requests.


As you use `'labels'` several time in this function, you could rely on a variable, to avoid misspelling errors (and, less important, re-instantiation of a new string each time).


Inga problem. Jag kommer faktiskt ihåg att du nämnde det.

Problemet är att man måste köra akka 2.4 för att det skall fungera. Så starta om sbt, eller reload och update. Kanske även clean.

> On 2 okt 2015, at 17:08, Patrik Bergagård <notifications@github.com> wrote:
>
> Hoppas inte det tog för lång tid att hitta. Som jag skriver i kommentaren, jag kunde inte kompilera om denna rad var med. Får se om det går bättre nu... Sorry.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/kristoferB/SP/commit/0bc4064e62d6db5f9ce1664bfdab0b659f454fdc#commitcomment-13564375>.
>


You may use `isinstance` instead of `type() is ...`


Oh what a shame (on me), it would execute pusb_volume_destroy(NULL);

I even haven't thought this would get to this pull request. I was just looking for fixing of "already mounted" error, which breaks auth - sometimes it ends up that that the chosen mountpoint shows to be already busy, which aborts mounting.


Лишняя строка.


Here's a (very messy) user script that adds an "Ignore Whitespace" button next to "Split" and "Unified" when you're viewing a diff:
https://github.com/esoergel/greasemonkey-scripts/blob/master/github-ignore-whitespace.user.js


Really odd.


It had be used because of this statement.
int loopSize = pathParts.length - ((hasFileName)? 1: 0);

It will ignore last path element if path contains filename.


@leouieda Ignore este commit, por favor.


Wouldnt it be better to check if the value is < 0? and still return -1 then? Shouldnt matter anyways, just to make sure negative values give errors (on an overflow maybe or any other corruption that might happen). This change needs to be applied a few lines below as well.


Неиспользуемые переменные часто обозначают идентификатором _. Следует i переименовать в такую.


fine


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


Maybe have _Jubilee Library_ in the address


Нет проверки входных данных!


does npm install phantomjs install phantom 2.0 or do you still require that to be installed from git?


A `typeof unzipResponse === 'function' ? unzipResponse(res) : res` should be enough I think.


Thanks


The pyramid should have 16 vertices, 3 copies of each of the base vertices and 4 of the top vertex. Here you've only pushed 3 copies of each, so you'll need to add another copy of the top vertex.


The base repo fails when pushing the commit. Maybe you should propose this change upstream.


Test it. I think `require` will throw, but not sure.


Agreed.


ah sorry about that, didn't realize this was in the vex modal.


Way too many indices for the pyramid, it contains 6 triangles so it should have 18 indices


Wow! Cool tool, thanks ))

But I‘ll re-write your code using case of. Reasons are more of graphic design nature. There is a common term ‘bullet’ for entities, that open list items. They can be of different nature, but best case is when they give good hint about list item content and get focus first. So compare, from the scope of readability:

_Design_ is a intuitive optimization process, that is restricted by subject scope.
_Implementation_ is a process of turning design into frozen form.

The process, named _design_, is intuitive driven optimization, that s restricted by subject scope.
The proces of _implementation_ turns design into frozen sturdy form.

This is especially important for long lists, and we have long one.


_Chester-le-Street_ with dashes?


ack


(Cosmetic) Since this lambda function doesn't depend on any context information, you could extract it from the current function (maybe to a `staticmethod` or a module reserved one).


Again, too many colours, there should be one colour for each vertex


Dont we to to apply this for the HID Device as well (Keyboard, Mouse?) They are also plugged into the HID which is plugged into USB. This would break the API style and confuse users though. Or does the static problem not apply here? Not sure if that already fixed everything.

Has anyone tried the code so far? (real hardware)


I probably made a mistake here... Sorry, not sure if you want me to make a new Pull request for that.


lel you had an shit done

2015-10-02 9:58 GMT+01:00 Lord Gaben <notifications@github.com>:

> Faken nubcake @xKoda <https://github.com/xKoda>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Lederps/Coding-Lab/commit/89044c2c9e05d7d0519b178f2cd55da941cfc5eb#commitcomment-13558057>
> .
>


@jrchamp We were waiting for the load event because that guarantees that the styles have been downloaded before the HTML gets injected. When we insert the bar HTML early, the unstyled utility bar flashes while the styles are being downloaded. It's unfortunate to have to wait to inject, but we decided there's no reason to rush it and have HTML earlier without the CSS.

Do you have a solution that would get the fully styled bar loaded earlier?


This path is sorta static in config.rb (except the home part but I cannot see where it would be called with home specified)


I'm in two minds on what to do :-) This test is nice and simple, so I'll probably add it, but I note that the other more complex test is also more-or-less OK other than an odd issue with LuaTeX and the Ubuntu system used for Travis. I'll therefore likely to add this one then also add the one by @liamhuang0205 with a bit of editing.


Rodger that


nevermind node install phantomjs is the phantomjs bridge, not phantomjs itself....


I only made the functional change in this commit, the [2nd commit](https://github.com/cjhamil/amazon-sqs-simple/commit/b9ff5111a5eff136cc6b83b938a87682526e55e4) addresses some of the various whitespace issues in this file.  Both are included in the [pull request](https://github.com/fleetfootmike/amazon-sqs-simple/pull/14).


Hi, I uploaded the that user case to my google drive, you can download with
this link if needed,
https://drive.google.com/file/d/0B2b7Zo-aJOgwNlFNM3R2VnpsU1E/view?usp=sharing

2015-10-02 5:03 GMT+08:00 Frédéric Bastien <notifications@github.com>:

> I'll run the time profile again. Do you still have the slow_compile.pkl
> file for this? I don't find it anymore. Otherwise I'll ask @t13m
> <https://github.com/t13m>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nouiz/Theano/commit/85a3b9a1b9a8a4a75f0e5f1e92c5d59d587e5679#commitcomment-13550950>
> .
>


Two spells here have the same random chance, I'm not sure Heat would ever actually be cast.


Please fix the repo url


I assuming it was same file what i had in SVN. So I just replaced it with my updated code. Setters are there to make it configurable as bean.


Shouldn't the accumulator here be `Integer`? This seems vulnerable to overflow as written.


Thank you for this feedback.  I'll make the changes you suggest and push in a minute.

I manually tried some unicode stuff, but didn't put any tests in because I'm not sure what the correct behavior should be.  Also, there might be python2/3 differences at play.  I *think* the following tests would be right, but I'm not sure, so I won't bother with it now:

```python
    def test_unicode(self):
        self.assertEncode(u'€', '\\xe2\\x82\\xac')
        self.assertEncode('\xe2\x82\xac', '\\xe2\\x82\\xac')
        self.assertDecode('\\xe2\\x82\\xac', u'€')
```


I would truncate the string first then removing the file. Then execute the directory creation.


For the present, this may as well be done without the conditionals: simply `\usepackage{xtemplate}\ExplSyntaxOn\cs_set_eq:cc { ver@xtemplate.sty } { ver@expl3.sty } \ExplSyntaxOff` should do it.  (Using the dependency set up means that you'll always be getting the latest `xtemplate` during testing.) I'm going to raise the entire area with the team (again): the SVN Id is long-standing but does lead to a few issues.


We should follow the new npm style. See an example at https://github.com/strongloop/loopback/blob/master/package.json#L105


Fix the url pls.


shouldn't this be a commandline flag instead?


This would be more readable if the indentation of the `then` and `else` clauses were two spaces on the line after the `if` rather than the current situation, which gives very long lines. Consider moving the whole RHS down one line as well.


I'll add them.


返回空数组说明到了php脚本了
1.检查你发送的header信息的Content-Length和X-Content-Range是否正确
2.检查nginx配置和上传的url是否走了upload模块


When merging code you should only replace the lines you changed. Blindly update a file will cause merging / diff issues for your team.


I observed, some time completePendingCommand  got hanged.


Right, that was a blunder. Now fixed in 5910146ea2376d1d1cb13b36268f21f4875690fb

> On 2 Oct 2015, at 15:41, Jonas Have <notifications@github.com> wrote:
>
> The warning you added is on the melphalan function, so perhaps we should change that also
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/oncoclass/hemaClass/commit/b598dddfc5830b241feed4348850fe4806c6ea74#commitcomment-13562510>.
>


ここにtry catch入れてください


got it


Will this break things?


Why wasn't this a "git mv"?


why this?


Should end with a comma. The idea is that you can always add or insert items without touching existing lines.


tried to add sails migration to allow me to drop the whole db at once but ddnt work


That's true but I'm afraid it could affect code readability.

Moreover, that could also apply to MongoDB operators (`$unset`, `$set`, `$project`, etc.) and all the field names, so that's a more general change that would, if that choice was made, require another PR.


Typo: "a different AudioNodes".  I'm guessing you probably want "AudioNode".


Very pedantic comment: since this was changed to use the BngFileInterface, perhaps it would be better to have a variable named "bngfile" (or something) rather than "con"?


What about using enum for the scopes? They are well defined https://maven.apache.org/guides/introduction/introduction-to-dependency-mechanism.html#Dependency_Scope and it would be type safe to use (instead of the strings). Or is there some specific reason to use the String?


lgtm!


Update for  build test javascript


Can we somehow use Boto to build this URL?


There is a [`boto.sqs.jsonmessage`](http://boto.readthedocs.org/en/latest/ref/sqs.html?highlight=get_messages#module-boto.sqs.jsonmessage) class that we should try to make use if we can.


Big thanks.


Regarding `wait_time_seconds`, that is a long polling optimization built-in to SQS. We should probably make use of it and set it to its highest value (`20` seconds).

See also: http://docs.aws.amazon.com/AWSSimpleQueueService/latest/SQSDeveloperGuide/sqs-long-polling.html

Regarding `visibility_timeout`, I think we need to determine a comfortable upper bound for each task type and explicitly set it.


The correct markup is:
```
... to the :class:`ParametericRectifierLayer` class
```
Otherwise it won't be linked to it in the generated documentation.


参考webページ
niconico api関連
-- http://dic.nicovideo.jp/a/%E3%83%8B%E3%82%B3%E3%83%8B%E3%82%B3%E5%8B%95%E7%94%BBapi
-- https://blog.nanoway.net/web/nicovideo-comment-api
RubyでJSON形式の結果が返ってくるURLをParseする
-- http://qiita.com/awakia/items/bd8c1385115df27c15fa


Yes, but I'm working on a commit to fix this.


Peter, the ARTIQ compiler is not working.


I find the little boxes for the AudioParam on the side of the main node kind of odd.

The last time I used a tool like this, the main box had little triangles for the input nodes that were labeled with the names of the input.  But I don't know how you'd use that indicate the gain.gain param defaulting to it's normal usage.


Please compare [your regex](https://regex101.com/r/gV9sV4/1) with [this proposal](https://regex101.com/r/zI6dJ9/1).


Just a heads up. You can combine validators on the same attribute.

```ruby
validates :email, presence: true,
                           format: RFC722::EMAIL
```


This will need to be changed to be based off F24, since F23 won't have support for this.  Then, the pykickstart PR will also need to be changed.  And then I'll need to add F24 support for pykickstart in the first place.  I'd been avoiding doing that as long as possible in order to keep master and F23 the same, but I've probably held out as long as I can.


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


I didn't check that because there is no method doing this is this file. Should we really do it?


thanks.

On Fri, Oct 2, 2015 at 11:24 AM, t13m <notifications@github.com> wrote:

> Hi, I uploaded the that user case to my google drive, you can download with
> this link if needed,
>
> https://drive.google.com/file/d/0B2b7Zo-aJOgwNlFNM3R2VnpsU1E/view?usp=sharing
>
> 2015-10-02 5:03 GMT+08:00 Frédéric Bastien <notifications@github.com>:
>
> > I'll run the time profile again. Do you still have the slow_compile.pkl
> > file for this? I don't find it anymore. Otherwise I'll ask @t13m
> > <https://github.com/t13m>
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/nouiz/Theano/commit/85a3b9a1b9a8a4a75f0e5f1e92c5d59d587e5679#commitcomment-13550950
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nouiz/Theano/commit/85a3b9a1b9a8a4a75f0e5f1e92c5d59d587e5679#commitcomment-13564706>
> .
>


Yep, I'm not satisfied of that part of the code (the way top "arrays" are created).

I'll work on that when I have some time. ;-)


@valib You should try to save which screen/desktop too, but you have to check that it still is valid before that setting is applied upon startup - and just use the standard desktop if it doesn't.


\o/


Was there any reason for going from boto3 back to boto here?


There shouldn't be a need to use `self` here.


@jrchamp Did you encounter a situation where the transition wasn't triggering? Not that I feel strongly whether the timeout is 10 or 30. Just curious what prompted the change.


**semi**: Missing semicolon.


Yep, I have spotted some of these, I'll clean this apart form that.


It is my fault. `filehook` is indeed not required here.


First change file


as I said, it is only a temporary hack.


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


This is a useful attribute; however usedCache might be misinterpreted to mean data cache.  usedMetadataCache is more explicit (although the term 'cache' itself is misleading - here we are referring to file cache on disk, not in-memory).   Another thing:  is a boolean value sufficient ? if there are 2 subdirectories and only one of them has the metadata cache, what would this flag indicate ?  It sounds like if the parent directory has the cache, then it will be true, otherwise false.


@ajithcj
The current idea is only scoring an endgame bonus...
But I like your 2nd formula, or maybe something like.
int attacker_bonus = 7 * pawns - 98 + king_separation > 3 ? 32 :king_separation * 8;
>3 because in rook endings, >=3 or >3 is a big difference.

@snicolet
But your very first try had something quite similar...
int attacker_bonus =  7 * pawns - 98  + 20 * (king_separation >= 2);

So what happened with that test ?
http://tests.stockfishchess.org/tests/view/55fa9d7c0ebc5976a2d6ddb5

There is indeed no point to give a bonus for strongside for moves such Kb1-a1
when opponent King is already on h8. And if you are the weak side,
I don't think you improve if you play Kh8-g8 to reduce the file separation...


I've altered: no problem :-)


I think this, in effect, does not screen any stars.  I think there's a sign error in the MATLAB code.


Instead of using @Nullable, can we use an Optional?


This one should be merged


Either way, we will definitely take the formatting of the markup piece.


It's more or less the same. Although, I like this slightly more than having mutable module level state.

Ideally, controllers would be instantiated objects that we could just assign this state to once in the constructor. I might do this in the future...


Not use in this function as well.


```NULL``` should be used instead of ```mbedtls_net_recv_timeout```, quoting the doc:
> The two most common use cases are:
>  * non-blocking I/O, f_recv != NULL, f_recv_timeout == NULL
>  * blocking I/O, f_recv == NULL, f_recv_timout != NULL


do we need 2 common repo's ?


Have nothing against that (: Readability matters and for mine version it's really not good - too much noise which makes hard to focus on important bits.


+1 This commit looks good. It fiexes #959


The main problem here is that `constant_time.bytes_eq()` raises exceptions if either argument is None. If you have any ideas on how to avoid the branches without introducing more complexity, I'd love to know.


It seems that this enum should have the same values as the OutputRenamingMapFormat, right? If that's the case, should we actually merge the two and just call it "RenamingMapFormat"? What do you think?


See issue #13.


It seems that both your `InputRenamingMapFormat` and the existing `OutputRenamingMapFormat` are missing tests. We should probably add some and make sure that we can round-trip from the output map to the input map and vice-versa.


Sorry I was barely awake: is that path still usable for pet abilities for the avatar pets? if its been refactored out player_whatever.lua seems the way to go. If its still usable though we can just toss whatever changes need to happen in that dir.

Some servers had a working version of player astral flow there based on guesstimated math, since retail is not known. I know you said the mobskill change broke some pet stuff, didn't know if it killed that entire directories use.


author: Andreas Schneider <asn@cryptomilk.org>


I changed the effort from 1 to 0, so it doesn't raise SP up


Nice!


@Markmhx -- here's model that the album creation form is based on. Notice that photos field has M2M relationship with Photo model above. When rendered, the form below presents a multi-choice list of existing photos for end-user to pick from, rather than prompting user to upload a new picture.


@810 just tried this fix, there are some things not okay with it (as far as I can see):
1. the comments that I made on the forum are still in the code :) "<!-- ADDED GUEST ALERT-->" and "<!-- END ADDITION -->"
2. email is not required, but that can also be deliberate: not sure what happens if you configure not to ask for email what will happen if email is required in the code #testcase
3. layout is not correct any more, see attached image (left patched forms.php, right, current implementation)
hope this helps!
![kunenadiscuss](https://cloud.githubusercontent.com/assets/2733197/10251093/d4d09938-692e-11e5-978b-9b48a531e93c.png)


Ok.  Thanks.  I'm still having some kind of issue with my same-origin POST only when CORS is enabled.  I'll spend a little more time digging around and create an issue if I can't figure it out.


The old data was according to WTNG, but I see that's not actually been updated for ages so we should treat anything there as being suspect.


what? any reason to do that


I second Ilvec's suggestion. I just estimated the effort (nodes) per depth for depth n+1: Search at depth n+1 itself requiring 40% and for all of the depths up to n in total requiring 60%: This gives a recursive formula
 S(n+1) = S(0) *(q**(n+1)-1)/(q-1)  (and q is about 1.4).
If the first search thread were working at *index 1* then effort would be distributed best when
th->rootDepth = rootDepth + Depth(log(th->idx)/log(q))

log(1.4) is about 0.336 , so we get
th->rootDepth = rootDepth + Depth (3 * log(th->idx)).
Now, in this formula the additional 1 is missing.  One might add a value of 0.5 in order to round to nearest integer depths or when in doubt round up right away. The latter would result in Ilvec's formula.


> Acts like a dictionary but encodes it’s data as a Base64 encoded JSON payload.

Not sure why they do this by default...


Sorry for the noise.. I wanted to "compact these into one commit".. forgot ooops..


a bit of an awkward name?


Edison I left a comment for hangouts on insight. I can do it tonight if we need to.

I guess in future group assignments we should stick to one form of communication, which should probably be insight since thats were the professor will look for participators.


spelling mistake


@akhilnarang and @furrydaus ***


What is it exactly?
You talk about the fact switch to 6, deleted or made of a 0?


Well, messed that one up a bit, but all in all the file looks clean now.


because people didn't stop him :P . Seriously, I just don't care. But if you want it reverted, I can do that.


after moving to Bootstrap'3' ^^


I write here because Twitter does not work to me, With io_is_busy 1 the system is more responsive but consumes a lot of battery so I always use to 0 and does not lose a lot of fluid. I advise you to put ignore_nice_load to 0 because the gov where this is on 1 (smatmax) , kernel lags


According to C_Heirloom.GetHeirloomInfo this flag is HEIRLOOM_ITEM_FLAG_PVP


@vibhu0009  can I do bacon?


Good point. If the visibility_timeout is lower than the actual time it takes for jobs to perform, another worker could grab the same message which could lead to conflicts.


I'll change BASE_URL with self.application.settings['base_url'] just to stay consistent with others (facebook and google)


Thanks Dan.  Nice one.


@markmhx - my initial photo upload form is based on this model below. The ImageField type is what presents end-user with option to upload a file.


I do not see where is the problem with what I did ...


Still getting odd behaviour on what is shown on the right hand side of the albums list. Undid all my changes back to 40441fb44c2dcd73cd7e81c7f77aa4fb2f74e0a8 and still get artist names missing sometimes but not others. Odd, but not my code doing it. Can anyone explain this?


This change is to fix the build. The API was returning 3368516734 for the balance amount which is too large for an Integer.


I don't completely understand the issue you're encountering. However, if settings are missing from your LMS, why not populate them? I wouldn't expect the LMS to function properly with the configuration it requires missing.


Please rename this field to UpgradeItemID - (ItemUpgradeID would imply entry from ItemUpgrade.db2)


They are only used in this method and this method is only called once.


The 'displayON' is the default mode for the affordance network module ;)


Please rename ItemUpgradeID to UpgradeItemID - (ItemUpgradeID would imply entry from ItemUpgrade.db2)
and BonusID to ItemBonusListID


yeah it was just temp


Merging in master into branch


`dataset.state() == GIVEN_UP` needs to be treated separately, as in the `else if` block.


Edit: Travis Badge


@anselmbradford how about this


This commit, as part of 4.14.0, seems to break my KVM slaves! I've [filed a bug report](http://dev.opennebula.org/issues/4020) which includes the traceback.


Date-stamps on the WTNG pages seem to vary wildly - I've paid more
attention to the newer ones.

On 2 October 2015 at 16:56, David Cantrell <notifications@github.com> wrote:

> The old data was according to WTNG, but I see that's not actually been
> updated for ages so we should treat anything there as being suspect.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mjg17/perl-modules-Number-Phone/commit/ea105fd54ba74c53ff00ebe245d7d244b847f57e#commitcomment-13565339>
> .
>


Nice work. It should be easier to add and remove configuration options, and it is nice to see that it's more complaint with the OpenWrt way of doing things. Thanks again.


Which part?


nice.


Changed to `shouldGiveUpVerification()`. It is needed here because even though `shouldGiveUpVerification()` is checked in `onFailure()`, if a verification job hangs for some reason, `onFailure()` will never be called.


Should this be s['status']  ?


accepting


:+1: !


I would have thought it should be `a`.


we will have to use an analog rc receiver, which does not have a control signal pin as the digital receiver (the digital receiver does not transmit any signals when the controller is OFF and also does not have a control signal, which is on, when the transmitter is on).
Moreover, the sketch was tested with the analog receiver and various bugs were discovered in the previous code.
Finally, variables and constants that are not used any more were removed.


@cscott ping... I reused some of your texvcjs code for this tiny program that prints the parameterized tokens from texvcjs


The Hexchain Commit


And this flag means SHOW_ONLY_IF_KNOWN


@cbrunsdon You might like this one, I got a bit bored :p


To clarify - this fixes train_quotes.py. still need to update mark_quotes.py to finish out the to do


Great stuff thx

If you want submit this as PR with tittle:

```Fix #1548 git version mangled by git describe```

It once its merged it auto closed bug report :)

thanks again.


So kind of minimal - you could load the library here and see if a device is actually connected - that would be the intent of having this separated out.


oye @rodrigoSaladoAnaya  si quieres mandar otro PR mejorando tu página actual de `author` estaría chido


yep, thx


Returning null has the effect of not rendering ProcessingBlock. Do you want to hide it for the sake of performance?


The abilities/pets directory is still used, it's just the table is used by both mobs and pets. Some servers already implemented it? They should have opened a PR.


@sinisa94
proprietary-files.txt doesn't need modifications. Mentioned files are symlinks to files that exist on firmware partition on device, they are not included in built.


This commit provides a revert point with plural parameter name rather than being hamstrung by the singular `screen`.


It did compile.  I missed that change.  I fixed it and updated the branch.


Base64. Just makes it a little harder to inspect the messages in SQS, if needed. Not a big deal though.


this is supposed to implement `transporter`, right? Perhaps use this pattern to make it more explicit:

```
var _ transporter = (*Transport)(nil)
```


@rictic how does one use this event? Are the icon sets going to use it internally or do developers have to hook into it? Also, can we add docs for this new event so they get picked up in the catalog?


All right, i'l work on that.


????
`sounds.Pack.DoesNotExist`


The guy that did it offered it but said its not retail.  He doesn't use IRC so only way to get hold of him is via the forums. https://github.com/starlightknight/darkstar/commits/astral-flow


This is no more working. Yet still lingering in docs.


Note: no longer planning to move to java


Fixed.


Or rather, that he has no idea how retail it is or not. The dmg output seems in the right ballpark from what little I saw it in action though.


It's not a big thing, but you could put this outside the for loop, so it only gets called once.


Ignore.


What's up with this?


`os.path.join`


Specifying $(LDFLAGS) again is not necessary.


looks good :+1:


do we have anything in our deploy scripts to ensure that all of these directories exist?


al proximo commit planeo subir las imagenes reales , si encontrai imagenes choras  me deci nomas xD


Hey Guys,
Is this commit available for play 2.4.3 version ?


I am making a comment about my commit.


this is another comment


Needs EOF newline.

Also, I guess this is an extenal module? Does that mean we don't care about our code style for this, or?


we prefer single quotes, right?


So setting it to 1 would be 16 nodes?


Looks great! :+1:


Isn't MPC-HC's ISR still lacking in performance for heavy typesetting when compared to XySubFilter with madVR? As far as my understanding is anyways.


Works as expected. :+1:


I have no objections to this fix.


@Stasik0
Hi Sten.
first test was **bad** using "server.c" example.
Please try UA-Client and make 2 or 3 connectons with same ip
- use "Current time" and more
 - change subscription settings to:
 + 3000
 + 2000
 + 1
 + 0
- stop 1 connecton
Tested with VS2015
crash in "Subscription_timedUpdateNotificationsJob"
data invalid, but not NULL.

Quickcheck with
https://github.com/joerg555/open62541/tree/simplemsvc/simplemsvc
"jsmserver"


It does warn them when they die, but not when they start to go to far...
I put that there to prevent the "Invalid listener" problem, I don't know if this will work, though.


Glad that worked! Cheers.


I hope you don't mind me just jumping in like this. First PR here!


Think you didn't check in the gray change in the tex file.


I checked, we have a todo note in web.sh (not sure if this is the file) which says "create folder structure" ;). for now I created the folders manually on the nfs (also  in test)


Ah.
I might rename this repo to pixture-dev, and start another without the asset source files or unneeded data.


@mkanoor Tests are failing on this change.  You want `Tenant.find_by!(:id => tenant_id)`


Thanks for fixing `ShareFeatureComponent`, I've ported it to https://github.com/Wikia/mercury/pull/1486


Thanks I'll look into it. I am collecting data from retail.


Well, this is a SOL_API one, I would not leave it like that.


@knowtheory Just ran across this and was very confused by the commenting format. For the record, it's rather confusing to read after the fact. ;)


Merged here, right now.


Ah good point about phony vs fake, I'll just change this to "fake" for now. Also a good point about the main file! I think it'll be harmless to emit (and I don't think it's trivial to filter out in this function), but if it turns out to be a problem we can always change!


This change needs to go to the 2015.4 and 2016.1 branches.

The 2015.3 branch can show this as an upgrade.


needs to be finished


========== Build: 12 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========


What is spawn.py ?


Needs to be started


Ya no es necesario


ja das gut


`inputs.files fileTree("foo"), fileTree("foo")` would break by containing duplicates


You may set "out" to NULL when declaring it and you could add "err_data" label here instead of duplicating:
  free(data); return X just below.


OK, cool. @AlexAntn's planner will need to send the 'displayOFF' command then.


LGTM.


Doesn't work anymore (if it worked)


fix #286


Not related to this patch but you could just loop from i to 0 here. No need for "last" variable.


keehhh


weird.. for me it still works perfectly ! no one issue !!
Maybe it has something to do if someone is undervolting ?
Especially vape .. since it also supplies the LCD ...


looks ok


septemtrio-agerさん、ありがとうございます。


Thanks again Steven.  This has been merged and released in version 2.0.12.


lulz you didn't put username or password


aefthwgfj


before the calculated position was incorrect.


I'm not a big fan of this, since any type errors will be silently dropped with no error. We already have separate channels for input, so I think we may as well have separate channels for the results, too. What do you think?


You should return the shorter one between the original and compressed one.


Remove extra whitespace?


Is it different before changes?


Use too much spaces.


I need to confess: I don't know anything about CSS (HTML/CSS in general), so feel free to modify


Miss follow up question.


Am I right in thinking that we *always* use spawnProcess now?  Do you want to also optionally allow people to run their task in a thread?


Remove /r/n?


      },
      "committer": {
        "login": "octocat",
        "id": 1,
        "avatar_url": "https://github.com/images/error/octocat_happy.gif",
        "gravatar_id": "",
        "url": "https://api.github.com/users/octocat",
        "html_url": "https://github.com/octocat",
        "followers_url": "https://api.github.com/users/octocat/followers",
        "following_url": "https://api.github.com/users/octocat/following{/other_user}",
        "gists_url": "https://api.github.com/users/octocat/gists{/gist_id}",
        "starred_url": "https://api.github.com/users/octocat/starred{/owner}{/repo}",
        "subscriptions_url": "https://api.github.com/users/octocat/subscriptions",
        "organizations_url": "https://api.github.com/users/octocat/orgs",
        "repos_url": "https://api.github.com/users/octocat/repos",
        "events_url": "https://api.github.com/users/octocat/events{/privacy}",
        "received_events_url": "https://api.github.com/users/octocat/received_events",
        "type": "User",
        "site_admin": false
      },
      "parents": [
        {
          "url": "https://api.github.com/repos/octocat/Hello-World/commits/6dcb09b5b57875f334f61aebed695e2e4193db5e",
          "sha": "6dcb09b5b57875f334f61aebed695e2e4193db5e"
        }
      ]
    }
  ],
  "files": [
    {
      "sha": "bbcd538c8e72b8c175046e27cc8f907076331401",
      "filename": "file1.txt",
      "status": "added",
      "additions": 103,
      "deletions": 21,
      "changes": 124,
      "blob_url": "https://github.com/octocat/Hello-World/blob/6dcb09b5b57875f334f61aebed695e2e4193db5e/file1.txt",
      "raw_url": "https://github.com/octocat/Hello-World/raw/6dcb09b5b57875f334f61aebed695e2e4193db5e/file1.txt",
      "contents_url": "https://api.github.com/repos/octocat/Hello-World/contents/file1.txt?ref=6dcb09b5b57875f334f61aebed695e2e4193db5e",
      "patch": "@@ -132,7 +132,7 @@ module Test @@ -1000,7 +1000,7 @@ module Test"
    }
  ]
}


Only files ending in .matt should be ignored.  Please fix.


You're very welcome :)


Theres a forum post on our forums this morning confirming that the formula
is the same as the merit BPs (not surprising, really), but it only has the
base damage at level 1. Unfortunately, my smn on retail is only level 30,
though


I suspect for coco2/dragon in 64K only mode we'll just have to live with it. The dragon-nx32 port can handle it fine as its using graphics modes (and also does a pile of font effects because I figured if NitrOS9 can do it so can we 8)).


@dkimbell13 is hovering not working for you? Are you on a mobile device? What browser are you using?


oh look, the `?` postfix operator. of course!


@fernandog I think this is a good workaround but per some IRC discussions with @miigotu and @OmgImAlexis, he's going to fix this a different way. I think the BS nav/dropdown code is going to get replaced at some point...


This would probably be total overkill, but it would be great if you stored a history of the player's position each tic while playing the map and then overlaid it on the map view during the intermission to show the route they took.


I don't fully understand this union "?child <fedora-model:hasModel> [<fedora-model:hasModel> <info:fedora/islandora:collectionCModel>] " which case does it cover?


Instead of `'1'` can you use `$entity1->pid` (or whatever the entity key is)?


I'd like to assign this to @dpressel


Use too much space.


There's a slight issue here, and I don't know how critical it is: it's almost certainly not critical right now, but we should at least be aware of it.

Simply getting rid of components with lifted leads only works while we're ignoring the effect that the other client's multimeters have on the circuits, which is what we're doing right now. However, this is actually a mistake: we *ought* to be including the other multimeters when we do our circuit calculations, just as (under the hood, in Breadboard) we include our own DMM.

The most obvious case where this actually has an effect is when a user lift's a resistor's lead and then bridges the gap with a DMM set to Ammeter. Or when they simply jump over the entire resistor with an ammeter.

In the first case, they've essentially reconnected the lifted lead (by letting the power travel through the low-impedance ammeter), in the second they've essentially short-circuited the resistor. In both cases, the client's own breadboard will reflect this. However, since we're not taking into account external client's DMMs, the other users will get a different result.

While this is a general feature that's missing, it also affects this specific line, because we won't be able to simply skip components with ghost holes. (If an external client has a resistor attached to `c20,c14ghost` and an ammeter connected to `c14ghost,c15`, we'd miss the fact that they had completed the circuit if we just threw away the resistor.)

In any case, as I say this depends on adding the inclusion of the external DMMs, which would be a new feature we'd have to add. On the assumption that the current work is more important than this, I'm OK with this work-around right now.

Pinging @phrwtz, so he can think about the effects of skipping the external client's DMMs, and how critical that is.


Hi, sorry about this again. Back when we were discussing this commit
https://github.com/openwrt/packages/commit/da6d7b57f501eaece621f8f834646aa22b618fa5

we removed this dependency because it breaks the build.
Is there a reason we added it back? thanks a lot.

-gong


ideally we will kill these cycles (or make sure this path is public tested as API in ember-cli)


that's not what I meant >__<


Typically with angular, you want to use
{{ errorMsg }}
 rather than ng-bind. Only time you would absolutely necessarily need to use ng-bind is if there is something that is always displayed (ex.

 since it takes some time for the angular to compile and using {{ welcomeMsg }} before the angular compiles would cause it to display as raw text rather than the variables value. In this case, since we're using ng-hide, it is preferred to use {{ errorMsg }}.


Problem is when. Users can use tablets/phones.
Maybe we should revert that PR


coding standard: all statements end it ;


can remove the panic on `height < 0` below, since that's now impossible


So they re-added leanback as feature to the APK?


You don't need to return the matrix. Java array is passed by reference.


I have used the following example (check for _): https://github.com/JetBrains/kotlin/commit/0a0bfda
Aside adding UnderscoreChecker to TargetPlatform.DEFAULT_DECLARATION_CHECKERS, there is a direct call to  `UnderscoreChecker.INSTANCE$.checkNamed(multiEntry, trace);` from `ModifiersChecker.checkModifiersForMultiDeclaration`, because this method doesn't use list of declaration checkers,
Also there were changes in `LazyImportScope.kt` and `BasicExpressionTypingVisitor`.

I can take a similar approch to in order to add check for "backtick" names in common frontend, but I am not sure how to do it only for JVM without some refactoring of exisiting code.
Should I add check for all platforms or do some refactoring?


Should this be continue?


everywhere you call `manageErr` is already inside a `build.DEBUG` block


`beforeModel`, `model`, and `afterModel` hooks.


The intention is that Game handles the responsibilities of enforcing the rules of the game, so that other classes (Property, Player, etc.) can focus on their own individual responsibilities without worrying about the rules of the game.


Imagine a new property has been added, which is saved as 4 bytes. http://tpt.io/~1863283 contains a line of DMND with this new property set to 0x01000000. Opening it in the old version causes half the DMND particles to turn into DUST (there was no DUST present when saved).

Opening new saves in older versions can also cause problems due to property conversions (the "//Particle specific parsing:" bit). When opening a save from a new version, the old version will simulate it according to the old rules (e.g. LIFE ctype created by CRAY is stored in tmp2). This, or missing elements, may mean that the save does not work correctly in old versions.
However, the brokenness may not be immediately obvious, so someone might then save it from the old version thinking that, despite the warning, it looks fine. It will then open without warning in old versions, but not work correctly. Also, not all property conversions are safe to apply twice, so saving a new save using an old version may break the save in new versions too.

A more robust idea would be to calculate a "minimum compatible version" when saving. This would be the most recent of:
* the version in which the OPS field descriptor flags used by the save were added (or if you want to be less fine grained, the last version in which a flag was added)
* the last version in which any other problematic changes to the OPS format were made (e.g. more field descriptor bytes)
* for each element used in the save, the version when that element last had incompatible changes made to it (property conversions, or major behaviour changes)


This comment should make it clear that this doesn't change the primary GID for people who don't know what supplemental groups are.


There is no need to keep all task/thread data


Move the to the for-loop, like I did


I thought we were creating sM records went sent_to_all was ran? That way it won't break if we change the scheduled time manually.


Split it in multiple tests instead of a unique Test using multiple "and".
So if strings decomposing fails you would know. This way anything may be wrong.

You could have something like
StrValue OUT -> IN[0] StrCmp(string/compare) EQUAL -> RESUILT TestString(test/result)
Splitter STRING_VALUE -> IN[1] StrCmp

The same for other types.


Don't save this information. However I think, like I said in my comments, the running task data should be kept.


I feel like all of the alternatives are the five things that you list below under unresolved questions from my comment on the PR.


Thanks. I've been meaning to change that.


I'm thinking through all the implications of this.

@pweil-


Maybe a static class?


C'est une régression ça. Il faut garder le filtre sur le champ 'deleted'.


You immediately dive into the solution without any preamble as to the basic premise of making `modelFor` return a promise.


dead line


You get syntax highlighting if you use ```javascript


Add an object parameter? See Google/Stackoverlow thread with multiple parameters. (Use the object solution maybe)


Add object as parameter?


Maybe there is no need for this.


Add this to the for loop


For django I made a management command... I haven't done the non-django version yet.
I figure it will live in there.


Let's be consistent and use parentheses for `assert_equal` calls.


coding style!


Where is next_send defined?


test this please


Should return the shorter one.


Yes I think so.
It seems like an obviously superior way to do it with few drawbacks.


I would vote for either merging this or possibly reverting the other one for now. I would like to avoid reverting though.

The issue with this is that it doesnt work for all browsers that support touch, and that the js is in the mako.


Лишний отступ


Этот отступ тоже лишний


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**Eslint**: 'return' outside of function


**no-unused-vars**: showDropdowns is defined but never used


**no-undef**: "expect" is not defined.


**no-unused-vars**: drops is defined but never used


Change i to taskNumber


I'm not sure what you mean. I can't find `OD_UNUSED` anywhere else.


Nope, for s6d cwm + lockscreen is 30ms and for lpm is 5ms.
Also make sure that s6d will work by default @ 44Mhz .. otherwise it'll lagg.

as for lpm works or not.. u can also test this with your own ws2401 ! Just configure for !is_lpm @ ws24 .. and use delay 3000 ... then reboot to lpm .. if it'll take longer till u'll see batt. animation then it works ! otherwise the default delays are used = no fix !


@surajwebworld  Please open it as a new issue under "enhancement" label.
https://github.com/Daemonite/material/labels/enhancement


@VinceVal Also, I think the NAVerrorlogs main click should go to `/errorlogs/viewlog/` since that is the most looked at log. (you only need the error log when there are errors, and we should not be generating hella errors.)


Ah, not having it seems to break some other part of the build.
atm I'm on my phone, so I can't offer many details.
@blogic for more details. I think he requested it.
I hope I'm recalling this right.

But this looks like a good reason to remove the Python host build in favour of just using the host's Python.
OpenWRT's toolchain fails anyway if there's no Python on the host system,  so might as well use it.


I can easily change the diagram, if you feel some other shape or arrangement would be clearer. I think the textual labels are still needed to explain what's going on.  I thought that butting the boxes up against each other made them seem like a single object with multiple parts.


My bad, I wanted non-failed `update`, but ended up disabling everything.


jndcdx


I didn't touch spacing around operators in this PR, it would have added up to quite a few changes.


What about optional named params? This needs to go after the push, no?


Programas con logica en Python!


This isn't in the documentation, what is `Link` meant to represent? I'm assuming I just don't know what link based requests are?


Cool, footer is not looking so good on mobile though.


@etsardou I think it would be better to revert that, unless there is a reason for the default map to change.


aaaasss


kk


Sure we could do that.


i really dont care about php7 i wanna that offstring -1 to get fixed,  i tryed by myself but after i make files back to phar it does not work,  i edit plugins everyday but pocketmine is just..... dont work after i compress tO phar... even if i dont edit... it wont work. Sooooo :|.  Survival servers dont work becouse of that error...  Hunger Games have no problem. :( I really hate 0.12. It crash if y < 1  .... i guess :p
...........


ok
thanks !


Miss follow up.


Ah sure, left it by mistake


For centos7-kilo it's delorean-kilo.repo
Should've been delorean.repo but too late to change Kilo now, fixed in centos7-liberty (reponame=delorean mentioned in https://github.com/redhat-openstack/delorean-instance/issues/12#issuecomment-144838874 )


How about something like this comment instead:
```
# Note that `Plot.save_with_user` does not update the `eco_rev` (`update_map_feature` does)
```


Is this a 401 or a 403?


Same issue:
```
In file included from dldfcn/__osmesa_print__.cc:42:
/opt/X11/include/GL/osmesa.h:119:1: error: unknown type name 'GLAPI'
GLAPI OSMesaContext GLAPIENTRY
^
```


you should assoc nil here instead of dissoc or this will return a map type instead of your record type and break protocol dispatch down the road


Re-read it and now I see.


yeah, i feel the same, but the tests are broken or i didn't know how to run them. I've been working into it here:
https://github.com/MG-Web/Datatable/tree/fixTests
it looks like laravel 5 changed the way to mock Config, and it's breaking on the complex tests...


Ok, but I asked more about will that https://github.com/ChronoMonochrome/Chrono_Kernel-1/commit/49df3c453921bcea4ce427b6c400369ae6adf590#diff-7f92e916b1f11a25727be19b75820fb2R494 work without blackscreen in LPM? If understand correctly, short delay 30 ms causes no animation bug in LPM?


Instead of VCS links, these should just be the package names (e.g., `django-arcutils`).


dat loose indentation

as barrack obama says, "Oh well".


How does this affect the battery life?


@Doca Why was this done? It prevents us from using the fancy new Travis infrastructure. http://docs.travis-ci.com/user/migrating-from-legacy/?utm_source=legacy-notice&utm_medium=banner&utm_campaign=legacy-upgrade


What happened?
On Oct 2, 2015 1:41 AM, "Wilkeningp" <notifications@github.com> wrote:

> Did not realize that this changed because my install in on a different
> drive.
> Might want to fix this...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CSC436-TeamBubbleSort/LunchRoll/commit/1e10a2a46ecc41bb687b8b99f7a0f71dcfa3a804#commitcomment-13557760>
> .
>


was there a particular reason you went with a copy here rather than just an assignment of the string slice?


yay 💛


Dear @mkofarag,

could you explain a bit more the reason for this commit? Removing the cluster seed positions from the pulse encoder breaks some analyses which rely on this.

cc'ing @tbisanz

Regards,
Simon


All of the packages in `install_requires` should have a minimum version like `>=1.2.4`.


ggfgg


> weird.. for me it still works perfectly ! no one issue !!

Maybe here happens that bt404_ts can't be resumed. I removed unneeded early_suspend/late_resume flags protection(see recent commits) just to be sure it can always be resumed.

> Maybe it has something to do if someone is undervolting ?
Especially vape .. since it also supplies the LCD ...

I don't think so.. These freezes users has reported recently - why it could not happen all this time before this dt2w/s2w patchset?


Sounds good to me. Could you update the guidelines to reflect that?


asdf


On Fri, Oct 2, 2015 at 1:10 PM, Michael Bebenita <notifications@github.com>
wrote:

> I'm not sure what you mean. I can't find OD_UNUSED anywhere else.
>
You may need to pull from upstream (it was added recently):
https://github.com/xiph/daala/blob/master/src/odintrin.h#L250


Martín Páez


조사처리를 자동으로 안 하는 고로 수동으로 해줘야 합니다.


Then the x86 should have versioncode 39193
And there might be a 39192 that we don't know of...


@miigotu I'm OK with either way. Personally, the only time I look at logs is when there ARE errors, so this is more efficient for me. :) But like you said, there really aren't that many errors, so I don't really care. Let me know if you'd like me to change it.


Why?
In the case it causes an unimplemented bug!


Keep in mind we get a packetizer failure on EOF too, right?


Use too much space.


Revert the old code.


Defensive coding implemented for a "Task" 's "date", "startingTime" and "endingTime" variables, to make it such that the date is as valid (although not perfectly for the day, but perfect for the month and the year (this does NOT affect the sorting of tasks in the .txt schedule file by date and starting time).

checkDate() is used in checkTask() and also used elsewhere in the code.

checkDate() and checkTask() are the two new methods added.

private static boolean checkDate(String dateString) checks the validity of the date, but not perfectly for the day of the date.

private static boolean checkTask(String taskString) checks the validity of the String, which is forced to be in the format of how the Task looks like in the text file and the output display window (the "half-Graphical User InterFace) "Flex Display".

Changes have been made to
private static void undo(String filename, String previousChangeTerm, String previousAction, Task previousTask) throws IOException, NullPointerException {
so that the program will no longer crash from not having a valid "Task adding"/"Task deleting"/"Task variable change" action to "undo" from.

Team members, please help me test the code before the v0.1 demonstration.

Thank you.


Why is it ok to drop the error?


They are still required in case you would use `2.8.0`, aren't they?


Yeah! :+1:


Again, u need all delays @ 30 for cwm + lockscreen ... but all delays @ 5 ONLY for LPM !

Theoretically u can even use delay 5 for everything (s6d) ... but i'm pretty sure (at least with my s6d) will cause weird tears during screen ON/OFF ... (tested only on cwm) ... but as i said... the problem is then only during ON/OFF .. after that the screen works normally when it's always on ... no tears at all ... but to be honest i don't want to mess it up with the screen ---> cwm + lockscreen .. just to Fix LPM for 3 users around the world !!!


This was really just proof-of-concept. If there is interest in merging this, I can certainly move the JS to the proper location. I'd also like to tighten up the `a.dropdown-toggle` selector so we're scoping to just the navbar in case that class is used elsewhere... I didn't want to step on @OmgImAlexis' toes since it sounds like he had ideas here, but let me know if you'd like me to do anything.


I was thinking of something like this (ascii art)

```
          +-----------------+
          |                 |
          |    Oscillator   |
          |                 |
   0 ---->+ detune          |
   1 ---->+ freq            |
          |                 |
          +-------+---------+
                  |
                  |
                  V
          +-------+---------+
          |      in         |
          |                 |
          |    Gain Node    |
 1 ------>+ gain            |
          |                 |
          |      out        |
          +-------+---------+
                  |
                  |
    +-------------+
    |
    |     +-----------------+
    |     |                 |
    |     |    Oscillator   |
    |     |                 |
    +---->+ detune          |
440 ----->+ freq            |
          |                 |
          |      out        |
          +-------+---------+
                  |
                  |
                  V
          +-------+---------+
          |       in        |
          |                 |
          |    Destination  |
          |                 |
          +-------+---------+
```

Don't know if this is clearer or not.  I'll leave it up to you to choose the appropriate diagram.

Except for the typo, lgtm.


Nice one!


Wow! thats easy!


@dmvstar all should work fine now :)


Yup.. just the domain name in CNAME file


Ingenious!


This commit is for the lack of comments above the code for checkDate(). Thank you.
=============================
Defensive coding implemented for a "Task" 's "date", "startingTime" and "endingTime" variables, to make it such that the date is as valid (although not perfectly for the day, but perfect for the month and the year (this does NOT affect the sorting of tasks in the .txt schedule file by date and starting time).

checkDate() is used in checkTask() and also used elsewhere in the code.

checkDate() and checkTask() are the two new methods added.

private static boolean checkDate(String dateString) checks the validity of the date, but not perfectly for the day of the date.

private static boolean checkTask(String taskString) checks the validity of the String, which is forced to be in the format of how the Task looks like in the text file and the output display window (the "half-Graphical User InterFace) "Flex Display".

Changes have been made to
private static void undo(String filename, String previousChangeTerm, String previousAction, Task previousTask) throws IOException, NullPointerException {
so that the program will no longer crash from not having a valid "Task adding"/"Task deleting"/"Task variable change" action to "undo" from.
Note: It seems that throwing a NullPointerException is not the reason why this works. Please refer to the code.

Team members, please help me test the code before the v0.1 demonstration.

Thank you.


Is there a tab/spaces issue here? :see_no_evil:


How you decided 10px?


> - if the user chose a packsizelimit of, say, 16G, you *still* remove it. That is most distinctly *not* what I want. If the user changed the default, the value is not fair game any more.

But how you can tell that it's not the "2g" the user wants?

> The most important problem here is that I still have no idea whether this patch was tested, and if so, with  > which architectures, whether it was a fresh install or a reinstall, whether the value in the previously
> installed Git for Windows was modified, etc.

I tested the patch with a fresh install and several reinstalls on Win7 Prof 64bit.

> BTW this part of the commit message is wrong:
>
> > (which are not changeable later anymore by git config).

Could you please explain, why this is wrong? It was my understanding, that we
are modifiying <ProgramData>/Git/config here and as this file is different from
the file modified by 'git config --system' (<ProgamFiles>/Git/mingw64/etc/gitconfig),
and also - of course - different from 'git config --global' I cannot see how this could be done.


Чак сега виждам съобщението. Ванка, извинявай дано сега вече всичко да е наред току що тествах камит от направо от проекта. :)


Changement des packages


It's apparent trivial debugging code that has always been commented out in the repo. It first showed up in cc9d5ab5 (as `sys.puts()` and in the `test/simple` dir). Since I was modifying nearby code, I took the opportunity to clean it up by deleting it. Happy to put it back if you or anyone would prefer. I don't feel strongly about it either way.


not part of the change but since I can see it...the method doesn't return errors. Should update this comment and the `types.go`.


Apparently htc's devs are better at removing comment than giving developers proper code.


Принято и засчитано


> if it'll take longer till u'll see batt. animation then it works ! otherwise the default delays are used = no fix !

Ok, here it should work for sure..

> but all delays @ 5 ONLY for LPM !

Well,  I pushed already these values(in LPM) https://github.com/ChronoMonochrome/Chrono_Kernel-1/commit/49df3c453921bcea4ce427b6c400369ae6adf590#diff-7f92e916b1f11a25727be19b75820fb2R494 , will see, how good defaults will work in LPM. If that short delay 5 ms causes some problems (even if it work in LPM without issues), don't think it would be a good idea to set it even in LPM.


Annie?


It's not a good idea to pull in the whole "pkgs" set like that. It makes it more difficult to see what dependencies the expression has, and more cumbersome to override certain inputs.


okay... for now i wouldn't touch anything (on my repo - dt2w) .. never change a running system xD


TeamCity DNN JavaScript Libraries :: CI [Build 101](http://teamcity.engagesoftware.com/viewLog.html?buildId=379&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 101](http://teamcity.engagesoftware.com/viewLog.html?buildId=379&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:8


@VinceVal its not stepping on toes, we just need to agree on a split of duties between you two. aka who is going to work on what and when. I think @OmgImAlexis is just trying to save you from doing extra work that will probably get changed soon.

@OmgImAlexis has had pretty much free reign on mako and js for awhile now so it just takes some grease to find a plan that works for everyone.


In what scenario would this.muleRegistryHelper.lookupByType(ExceptionContextProvider.class) return null?


s/dictionraies/dictionaries./ (typo and dot at the end - not sure about the dot, but I like that better in general.


I used the existing eslint core `restricted-modules` rule as a starting point and adhered to that their style with the idea of seeing if this might be appropriate to PR to eslint. If they have a `restricted-modules` lint rule in core, it stands to reason that a `required-modules` lint rule might be appropriate too. I can change it to be single quotes if we feel strongly, but I don't know that we want to worry too much about imposing style rules on things in the `tools` dir since all of `tools/eslint` is external code, for example.


We'd need to put in an alias for the old function, add a deprecation warning, and list the upcoming API deprecation in the TODO file.


s/dictionraies/dictionaries/


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


Да, без него часы не останавливаются по нажатию на кнопку.


Its the base size for by html page after which i can use rem metric in other sections and 1rem would be 10px, which will be an easy calculation


I'm not so keen on modifying passed arguments. Let's bikeshed this decision a bit more (you may have been right in the first place).


@VinceVal lets just wait and see if a solution comes up soon. Things tend to get resolved pretty quick and we arent racing here.

maybe for now you could take a look at the (css?) problem in Episode Status Management when you select a status to view, and then expand and collapse a show the layout breaks??? Not sure how into css you are =P

There are a few small broken things like that here and there that can take priority over this for the next few days.


Wow, does this mean you're deprecating `webpack:hot-reload`?


where did u set delay 5 for lpm ? i don't see it.
But as i said, delays @ 5 works on s6d for lpm ..... tested by manthes ..... (no one custom roms or kernel worked for him @ LPM ! )


Nopes, at this point, this `replace` handles all named parameters, including optional ones, as previous `replace` already appends `:` to the name:

```js
'named/optional/(y:z)'
  .replace(escapeRegExp, '\\$&')                    // 'named/optional/(y:z)''
  .replace(optionalParam, '(?:$1)?')                // 'named/optional/(?:y:z)?'
  .replace(namedParam, function(match, optional) {
    // this function gets called twice
    // 1st
    //   match: '(?:y'
    //   optional: '(?'
    // 2nd
    //   match: ':z'
    //   optional: undefined
  });
```


https://www.dropbox.com/s/keapwm5mzk5dfj6/draft%20kerwin.docx?dl=0

draft do ensaio


ASI DEBE DE SER


@Fishrock123 newline added, thanks. The style in the file adheres to the style used by the rules that are in `tools/eslint/lib/rules`. It's based on `tools/eslint/lib/rules/no-restricted-modules`. I can rewrite it to adhere to Node style, but that could mildly complicate merging any future bugfixes or changes from that file in new versions of eslint.


Thanks! This makes sense. I am not even sure why I added ```self``` here. I think I was debugging something and before finding the cause of the bug tried several dumb changes like this and did not remove them later.


I'll do this later @dscho as I must go out now.


**Code Quality - Nitpick**

You have a couple of redundant new lines in your code base that should remove, please make these adjustments.


I think andriod studio changed the file path because my install is on a different drive other than C:. I fixed the repository earlier so now i'm going to re-install android studio so that this will not be a huge issue going forward


Got it.


> But as i said, delays @ 5 works on s6d for lpm ..... tested by manthes ..... (no one custom roms or kernel worked for him @ LPM ! )

Ok, then it makes sense.


this shouldn't be here, take it out to some other file. and buttons and calendar stuff must be in different files.


be careful with the indentation


Feel free to open this as a proper pull against master. Easier to keep track of things that way.

How about `_routeInfos` => `_routeParams`. Still not great, but a little clearer perhaps.


I don't know about the routeData object though. How about `(params, queryString, name)` as the arguments to the handler? Or `(params, routeData)` where `routeData` is `queryString` and `name`.


you should use either 'start_time' and 'end_time' or 'start_at' and 'end_at', but don't mix them


fantastic.


I think this issue is fixed now, and you can remove this workaround.


DOM Example from Jeffery's Lesson


@joeman4ever hey check this out!


why is this comment here? doesn't add any useful information


Yep, working great now, thanks!


s/Non/Not i believe.


alerts and messages are awesome, why aren't u sending one here as you do in the else statement?


<img width="747" alt="cho_han_gam" src="https://cloud.githubusercontent.com/assets/558565/10253784/29b2c43e-68f4-11e5-8996-c92b3cfc66cc.png">


@jon-a-nygaard Can you review my changes? All visual tests pass, but I'm not sure how good the coverage is.


Seems I have a lot more to learn about readability haha. I'm also used to working with components so I never considered that. Thanks for taking the time to explain.

My colleague rederived the equation in terms of a vector equation as [such (Google Drive Shared Image)](https://drive.google.com/file/d/0B6XahvAYOZCNRFVxWWRudWlBTHpSUm80amlvalZjLVVlUE9r/view?usp=sharing). Any comments?

Yeah I changed it right after I posted that to a const reference. I just wanted to show the idea.

Also, is this related to the overriding of gravity issue described [here](https://github.com/woodem/woo/pull/17#issuecomment-143195175)? It doesn't appear so but I'm not sure.


avoid empty helpers


Shouldn't you check `strdup` result here?


Check comments in recently rejected pull requests regarding to `info_app.log`


@taconaut found it, ```while ((child = l.getChildren().next() ...)``` is using brand new iterator in each loop:
```diff
+++ b/core/src/main/java/net/pms/medialibrary/dlna/RootFolder.java
@@ -19,6 +19,7 @@
 package net.pms.medialibrary.dlna;

 import java.util.ArrayList;
+import java.util.Iterator;

 import net.pms.dlna.DLNAResource;
 import net.pms.external.AdditionalFolderAtRoot;
@@ -110,9 +111,9 @@ public class RootFolder extends MediaLibraryFolder {
 			addChild(l.getChild());
 		}
 		for (AdditionalFoldersAtRoot l : PluginsFactory.getAdditionalFoldersAtRootList()) {
-			DLNAResource child;
-			while ((child = l.getChildren().next()) != null) {
-				addChild(child);
+			Iterator i = l.getChildren();
+			while (i.hasNext()) {
+				addChild((DLNAResource)i.next());
 			}
 		}
 	}
```
and of course the first child always has a previous id after being added once :).


This is really buggy. when  you enter an instance, sometimes even if there is only like 1 thing (for example, 1 strongbox) the sound gets played like 6 times.


@rco-odoo but its migration commit and when u migration code that change the meaning of code migrate into new api. In old code in field have string so it must to gives string parameter in migrated code.


Can you understand this location thing and tell me what is it?

http://emberjs.com/api/classes/Ember.Location.html#toc_autolocation


My bad, http://docs.oracle.com/javase/7/docs/api/java/lang/Throwable.html#printStackTrace() it prints the error message as the first line, which I missed. this is fine.


It's actually a collection of route params for each route, `_routeParamsCollection`? Or I can settle for `_routeParams`.

Created PR for easier tracking.

Thanks!


empty test! don't delete it, you should actually fill it.
you can check how to test controller here https://www.relishapp.com/rspec/rspec-rails/docs/controller-specs


@gnscott this can be removed, right?


all good now


yes. thanks


That is sad :(

What do you mean by saying "stop one connection" ?

> Am 02.10.2015 um 18:25 schrieb Jörg Schüler-Maroldt <notifications@github.com>:
>
> @Stasik0
> Hi Sten.
> first test was bad using "server.c" example.
> Please try UA-Client and make 2 or 3 connectons with same ip
>
> use "Current time" and more
> change subscription settings to:
> 3000
> 2000
> 1
> 0
> stop 1 connecton Tested with VS2015 crash in "Subscription_timedUpdateNotificationsJob" data invalid, but not NULL.
> Quickcheck with
> https://github.com/joerg555/open62541/tree/simplemsvc/simplemsvc
> "jsmserver"
>
> —
> Reply to this email directly or view it on GitHub.
>


Would you mind licensing the `FS_BlackListedZipFile` stuff under GPL-2+ instead of GPL-2? I'd like to put together some possible patches for ioquake/ioq3#130, and this looks like a really nice option.

No need to change anything in the files, just tell me "yes, you may treat those parts as GPL-2+" if you agree.


Are you deliberately making the license of these files more restrictive? You don't need to change individual source files from GPL-2+ to GPL-2 unless your modifications have made them a derivative work of JK2/JA, which I don't think is true here.

If you do want to make the license more restrictive, that's entirely valid for you to do; but it will mean that improvements you make here can't be copied into ioquake3, which would be a shame.

I realise you want to say it's part of OpenJK without implying that OpenJK as a whole is GPL-2+, but perhaps for files that are still GPL-2+, something like this would work?

```
This file is part of the OpenJK source code.

This source file is free software; you can redistribute it and/or
(etc.)
```


Main points:

-  Too much logic in the views. Try to extract it to the controller or, even better, to some service models.
-  be careful with the indentations. REALLY.


Yes! we had this conversation I forgot. You need to again tell me all those things.


you should use rails path helpers here (like root_path)


GroundDomainUpdate (sushobhit) - x86_64-MacOS-10.9-clang: Tests Failed

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


@talesgalvao @asagalo lembrem-se do arquivo CHANGELOG também


Give proper name of variable. :) what is f ?


Hi, Arhchi!
Whether it is possible somehow improve the performance of RAM?
Help solve the problem backporting (vmalloc, madvise, page-writeback, pagealloc, page_cgroup and etc.) with newer kernel versions(for instance 3.10)?


This project is not using it anymore. It makes more sense to run the webpack-dev-server outside Meteor because otherwise, when Meteor restart, it would need to recompile webpack with no cache. This means the restart is taking an addition 10 secs for no reason. I think it is better to keep it out of Meteor.


Can you formulate a bit better what this does?


it is a test for Github


Fixed in `b1e75a1`


That's probably not the circle you are looking for.


Didn't intend to drop the error, nice catch :)


Or create a fork, I can update it. BTW, missing a few versions doesn't hurt:)


why isn't this just

    resources :appointments, only: [:create, :edit, :update, :delete]

and get rid of the **/appointments/destroy/:id** route (it's not RESTful at all)
?


Yes i know...just use HvC master version by now...it's already the new version with design included. He said he will fix the sound glitch.


Like


Well there can't be any type errors because of the synchronization. The reader loop is the only producer to this channel, and it can only produce the corresponding type for the function that it's read. That said, I can add in a separate result channel for this too. Or think up a slightly cleaner way to do it.


If you don't have for any test for testing then why create test method (test_00_requisition_demo) just put all records in the setUp


What does that do?


And please combine the multiple commits into one commit or use a more
descriptive commit message, please. Thanks!


Wouldn't it be better to 1. first copy zpool cache to a temporary file within ramfs, then 2. export as usual (without -F flag) and then 3. move that temporary to `${_cache_file}` ? This would avoid the use of undocumented flag. Alternatively it would seem prudent to raise an issue at https://github.com/zfsonlinux/zfs to document the -F flag, as otherwise there is possibility that it might be removed in the future