Thanks for this! Can you remove some of the extra stuff from it though:
 Don't need the PEER_SEED or KNOWN_PEERS or the comment about testnet servers or the testnet history store


Drop arg to next line.


Drop paren to next line.


uhm, this is prone to be forgotten to update... I recall we had a PR which added a key recently not sure if this is already merged. Could you rebase this one and check for new keys?


I function layer is needed if you want to hide underlying implementation. If this global is kept then you will *only* be able to determine these paths on startup or on some function call. If it is replaced with a function then you may as well redetermine them on each call should it be needed.


@idgserpro porque eles não influem no cálculo e só demora mais


`tax_sign = self.tax_amount/self.amount if self.amount else 1`
should work as well


Congrats on the merge @AdmiralCurtiss!


For all of these, after making these changes, rerun PyCharm's formatting to correct indentation.


Shouldn't the default be in dbGetImageExpiry?


We can not add a checkbox to search in already taken orders during the last month?  Not everyone is that adventurous.


Missing generated declarations include.


Should this also be changed in line 25?

There is a typo in `email_memeber_modal_selector` – up to you whether you care to fix it.


@yohanboniface I should should should have caught this!


Missing generated declarations include. (Should happen after all type declarations.)


standard naming for such functions so far has been doFunctionName. So this should be doDeleteImage.


GroundDomainUpdate (mmatts) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Why did `args` go away? Bring it back. Also, rename to `argv`. Its type is (strs).


Also, `strs` -> `str`.


Gracias!


hey, could you explain the change a bit in the commit message? For more info see http://coala.readthedocs.org/en/latest/Getting_Involved/Writing_Good_Commits/#how-to-write-good-commit-messages

Please explain why this change is needed in the body.


@hadley do you think I should report this as an issue?


Errata: Lines that aren't actually arguments now are 19-22.


$USER may be not defined.


I do not know how work this function from dashboard, need check this. Probably it contains only selected values. I hope Diego knows more about it.


You are right enough. I just found a new version of Audacity on their website. I will see how to update this cask to reflect this change.


@Yamakaky What’s the point of supporting only part of the specification? If we say that we support XDG Base Directory Specification we are expected to default to /etc/xdg.


maybe use [not_binary](https://github.com/cdent/gabbi/blob/master/gabbi/utils.py#L82) here? Otherwise you're also accepting things like `image/png` as printable.


What ?


please nest this under hbevents


Isn't better use `DB_CONNECTION` from Laravel there ?


How about also checking the permissions of the directory? This should probably result in a warning.


Yepp. And/or disable this setting for default. It happened to me many times, that there was offline pages visible and i thinked "wth"?


Are you in contact with the developer of the iOS app? And do you know why it is not available anymore or if it comes back sometime?


that was me forgetting to pull before changing things, it's how you made it, right?


By the way, is this commit in any PR? I feel like we can as well discuss an outdated diff just because we are making comments to a commit which will never change.


why is this added but commented out?


Were these intentionally left in?


I see, that one was only recently added I think. Didn't have it in my project. Will change it.


I'm probably just prematurely optimizing, but I kinda feel like all this work to generate the failure msg should only happen if there is actually is failure. Maybe somebody else has an opinion?


Thinking about it more, I think the current behavior is okay but needs to be documented.


You will have difficulty finding a useable official link for the latest version. We can't use FossHub links for casks because they block hotlinking/command-line downloads. You may be able to make something for homebrew-unofficial though.


Seriously? You added an empty `requirements.txt`?

Take Python .gitignore from [here](https://github.com/github/gitignore/blob/master/Python.gitignore). And name your virtualenv directories as `env`. Otherwise, you'll have to add that also, like you've done here.


LOOKS GOOD :+1:


@kunagpal: I know that. I'm asking why did you have to set it as optional?


Fixed. Thanks for the catch.


The reason why I didn't bother fixing it is because really this entire file is due for a rewrite.  The majority of it should be scrapped imho.


Impressive. I noticed this would not handle nested `source-file`s. But anyone who used nested tmux config files is silly. An edge case I would not support.

Also, I thought your use of `_tmux_conf_contents` and `_sourced_files` was quite clever.

I'll try to get to testing it soon.


How about "I'm feeling *lungry*" ? ;-)


Thanks for looking into that! According to BSD find man there is no -fprint indeed, but there is -print argument. Would it work if "-fprint cscope.files" is simply replaced with "-print > cscope.files"?

Please take a look at the last change in my repo.


I'm sorry, I meant Diogo (auto inserted by my phone).


change manifest.xml + string.xml


Why not merge my pull? The commit have do this job: https://github.com/kongfl888/display-drivers-uninstaller/commit/f387cc9ad92c8274bb5b2836180bc068b632628b


Forgot to commit modified files - ignore this


Once a day would be perfect, if something breaks, it is either obvious which change to blame or `git bisect` will find the first bad commit very fast.
You could even integrate the `git bisect` part into your script.
If a build is successful, remember the value of `git describe --tags` somewhere, e.g.

```
echo -n `git describe --tags` > ../build_success.txt
```
If the next build fails and build_success.txt exists, move to the top level dir, run
```
git bisect start
git bisect bad
git checkout `cat build_success.txt`
git bisect good
```
If you are paranoid, you verify that the old commit still builds (could go wrong due to hardware changes, software upgrades, mood phase, ...)
and then test build until you get the final "first bad commit".

Finally,
```
git bisect reset
git checkout bleeding-jumbo
rm build_success.txt
```
(the rm is to avoid the bisect until you got a new successful build.)


Debug? :)

Neat stuff. Does that mean protochart/prototype can be removed already?


We could change it back to the long-form syntax easily enough. It might be clearer overall to separately specify both `std::swap` with "doesn't participate in overload resolution if" and `ranges_v1::swap` with Concept constraints.


We are missing a multi-version integration test for checkstyle: I'd rather not add a single test case like this, but instead add coverage for several supported versions. Would you be able to add this?

See here for an example: https://github.com/gradle/gradle/blob/master/subprojects/code-quality/src/integTest/groovy/org/gradle/api/plugins/quality/PmdPluginVersionIntegrationTest.groovy


Yes I am seeing that now... Thanks. Lets see if I can come up with something.


I thought I shall add it now, and then I got lazy and I had to clear some shit out. Long story, what is the equivalent of `npm install x --save` ?


Sure


So that people don't have to switch their OS to make one project module work.


~oops~


Good luck :+1:


Good catch, but I already fixed that one ;)


Опять смена имени :disappointed:


hey, thats almost all of them!

did you rewrite those manually?


Very well noticed, I've removed it for him :)


How much wood can a wood chuck chuck if a wood chuck could chuck wood


Why do we inherit service from controller?


Texture size reduction works well, with an annoying side effect.

03500 6 19 tw 9=>9 th 9=>8 uvmax 264,216 wm 1,1 (0,0,0,0)
03500 6 19 tw 9=>9 th 9=>7 uvmax 264,72 wm 1,1 (0,0,0,0)

If the texture is reused with different coverage, it puts multiple entries into the cache with different sizes. In this case we still save 1/4 memory though. But you can imagine if all the permutations of TW/TH are produced what happens. All in all, the gain is much larger than the loss so far.


ありがと！これ、Stringへのキャストって+"" の箇所？


why do we need to slice the array here?


No no, I wrote the `window.navigator.msPointerEnabled` directly in github, I was too lazy.
But I was like "Wtf, haro changed something, but the code is the same" that's the reason of my "What".

I trust your github usage ability :)


Review 311: +#define ULTIPANEL  //the ultipanel as on thingiverse // Geoff old pana


This requires that you also set the definition in the CMake code: https://github.com/ros2/system_tests/blob/cd499c8c06e1d105a0c076d0038170572b04d9bb/test_rclcpp/CMakeLists.txt#L44-L45


This doesn't seem like a good idea for a library.


Please extract a method to make this read a bit better.


We should make it require a user API key (since it's a Mapzen-specific util anyway)


:smile: cuales otros? self ese es un parametro del metodo, pero se refiere al objeto de la misma clase como this


Review 315: +// #define REPRAP_DISCOUNT_SMART_CONTROLLER // Geoff new panel


Thanks very much for this patch!


Sounds like a good idea to pass the parameters of options instead of an `Options` object. It did look odd to have the options being passed and not very pythonic.

I'm not very comfortable with having positional arguments that can be two different things.
Besides not being a common pattern in Python, I think a signature of `AblyRest(key_or_token, key, token...) or AblyRest(key_or_token, token...)` would look rather confuse.


Review 319: +#if defined(ULTIMAKERCONTROLLER) || defined(REPRAP_DISCOUNT_SMART_CONTROLLER)


O hay que comentar cada metodo a ver que hace?
ese es el texto que esta entre '''  '''


I think I could limit this to the cases we really want to reduce, when the difference in TW/TH is at least 3. This game typically goes from 10 to 7 or 6, because it uses 64x64 and 64x128 window sizes to do its pathetic post processing effect :P


Can you add some doc?


The symlink doesn't go into npm, it's just for development convenience.


Why did you bump to protobuf 3.0.0.alpha-3? It doesn't contain the cmake buildsystem which seems to have been added a few days after that tag has been made.


That is true, but this could change if somebody decides to use ReliablePacketBuffer in a different manner (without checks prior to this one).  I'd rather be safe.


You could do `return false;` instead of the previous two lines, with the same effect.


@anhtunguyen đã xóa mất đoạn code của dũng đi rồi

https://github.com/nukeviet/nukeviet/commit/b18fcbe4c47ae88e0179bd37e1a0a4e13fa1c2df

Trong file  themes/default/js/main.js
đoạn: // Load Social script - lasest


seduction nerf №… 7, is it now?


I've never used `include_external_msproject()` myself but I read up on it a while ago and I think it only adds an additional build target but it doesn't do anything with it unless you add it as a dependency to another build target that is actually being built.


@joaomdmoura Why this commit was necessary?


Oh, FFS.  I thought it was the numerical channel ID because it's being compared to as an integer.  Good catch.


To clarify, this concerns the character select menu; while previously it did not prompt you for whether to use A or B, now the menu does.


Isn't this called `attr()` in jQuery?  I can't find any documentation for `setAttribute()`.


+ Add the analog inputs


咦我推荐过这个？以及为啥现在才 push（


change steeringFlag -> steeringChanged, it's more descriptive


Thanks for the link.  :+1:


Сранный стартер контент( Весит как твоя мамаша


Ah, you still haven't fixed your systems? All you need are Visual Studio Command Line Build Tools.

Well, as long as you are using [bcrypt](https://npmjs.org/package/bcrypt) proper in production, it's fine. On your own *Windows* systems, use [bcryptjs](https://www.npmjs.com/package/bcryptjs).


I see. This loop executes more than once only for vararg functions. So what you are saying is that if we had a vararg function with contracts, this would most likely break?


tab in the doBraking and the other functions in the loop


You don't work with this device anymore?


whoa, troop type specializations no longer have maluses to opposite types? that's somewhat significant.


change doBraking to setBrake doSteering to setSteering etc


I know, I just don't understand why they do it that way.


That's right. It's also hard coded to be there, right? Sorry, I seem to not see your point.


Monitor disk status


Make the acceleration function


@greenlikeorange thanks par ... he he မနက် က လောပြီးလုပ်လိုက်တာ မေ့ကျန်ခဲ့တာ


cool story bro! :)


An equivalent command doesn't exist for Python. Check [this](https://stackoverflow.com/questions/20006000/install-a-package-and-write-to-requirements-txt-with-pip) for more information.

Do this: `pip freeze > requirements.txt`. But you need to have the dependencies installed in the Python environment first.


Are these options necessary for us? Specifically, is submitter needed for logging purposes?


No need to have spi_bus, chip_select, frequency, bits_per_word and mode on max31855, you just need this data when opening the spi bus.


Doing `AppTestCase.DISCONNECTED_SIGNALS` means that subclasses won't be able to properly override the variable. I believe changing this to `self.DISCONNECTED_SIGNALS` should fix that.


It's not, but it should be.


Yes, that is the plan :-)

On Wed, Aug 5, 2015 at 8:37 AM, Alex Henning Johannessen <
notifications@github.com> wrote:

> @jedesah <https://github.com/jedesah> It is a bit cumbersome that one
> needs to do all that typing wrt: Field.strict[Foo]("in"),
> Field.strict[Foo]("out"), especially if one is always going to type "in"
> and "out". Could the boilerplate perhaps be reduced by a dsl or some
> default standard values for in and out?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/oncue/remotely/commit/4a90e8390421b9d5d95fcb55f702c17802241ac9#commitcomment-12544647>
> .
>


the node data is calloced, so you don't need initialize anything to 0.


@qsomazzi I wanted to make a PR, wrong button.

Feel free to comment for modification.


This should be placed into a helper method.


xenosaga 2 menu likes it.


optional commit message nitpick:

`Add __copy__ and __deepcopy__ to LazyProxy. `

could be transformed to

`LazyProxy: Add __copy__ and __deepcopy__`


8 was for 8x512 for 4kb  fist 128 ASCII symbols. Avoid using unnecessary flags, it wont' work in MSYS


Also a helper method


Semicilon was for windows compatibility, therwise fs convert is called!


yay, I scanned the code and it looks like src/www/protochart/ can be zapped now


Why everything static?
I think use a macro to this guys is better, less stack memory allocated.


No need for `readlines.join('')`, you can just use `IO.read(input_file)`.  Also I would prefer to invert the logic so the main code path is not indented:

    abort "#{input_file} does not exist." unless File.file?(input_file)
    input_data = IO.read(input_file)


I donno why actually you removed the font_tool. The draw character routine was optimized for 3DS screenbuffer.


Check status before using rx.


Can this be mitigated by using a more specific regular expression instead of simply `indexOf()`?  (I'm not completely sure what's happening here, so it probably can't.)


client cert file


client key file


@petter-fidesmo Ready for merging!


There were serious issues with this entire function for both comment and uncomment. Some of which were fixed here: https://github.com/saltstack/salt/pull/25685 and here: https://github.com/saltstack/salt/pull/26002


@gabest11: Any idea what's causing the lack of colors in HW mode?


Thanks.


Not tried yet.


(I guess), we also have to update the lines(as followings) for using "process.load".
https://github.com/vallot/CATTools/blob/cat75x/CatProducer/python/catGenHFHadronMatching_cff.py#L59
https://github.com/vallot/CATTools/blob/cat75x/CatProducer/python/catGenHFHadronMatching_cff.py#L64
+++++++
process. matchGenCHadron.genJets = genJetCollection
process.GenTtbarCategories.genJets = genJetCollection
++++++++


I think this section can just be:

```if (!VertexLayout.enabled_attribs[location]) {```


/cc @vkarpov15. I don't yet want to file a bug against mongoose until I've consulted with you. Is the side effect intended? This is because trying to reuse the same options after a populate call modifies the argument, in particular the limit.
Thanks.


Done.


It's not just this - I've found a bunch of stuff that didn't get updated in the `-foobar` -> `--foo-bar` transition...


TeamCity xBIM :: XbimScripting :: Build [Build 29](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2208&buildTypeId=XBIM_XbimScripting_Build) is now running


Shouldn't this also show the status of their application? And shouldn't this route have a superadmin menu item link to it?


@erickwilder do we have import policies? I'm usually against importing in functions, rather do it on top of the files, that shows circular dependencies if they exist and avoids import duplications.


I agreed with you. If someone set displayField to something other than a string, there will be an error.
But, if in your code, you don't mess up with displayField, there will always be fine, no errors.
The programmer that changes the default value of displayField should know that it's a string, unless we change its functionality for supporting multiple fields or something else.

You're right in protecting fure programmers from doing something wrong, but in my thoughts, this would just mask the real problem, that is not setting the right value for the property.


looks good besides the pointed out things, waiting for @erickwilder 's comment before merging


On Aug 5, 2015 1:12 AM, "ace2nutzer" <notifications@github.com> wrote:
>
>
> btw: speaking about delays.. why u guys are using such big delay for
LiveOPP ?
>
>
https://github.com/ChronoMonochrome/Chrono_Kernel-1/blob/android-3.0/drivers/cpufreq/dbx500-cpufreq.c#L125
>
>
https://github.com/mkaluza/Samsung_STE_Kernel/blob/master/drivers/cpufreq/dbx500-cpufreq.c#L125
>
> LiveOPP is now quite stable.. (of course with right voltages) so that we
can use lower latency:
>
>
https://github.com/ace2nutzer/Samsung_STE_Kernel/blob/3.0.101/drivers/cpufreq/dbx500-cpufreq.c#L125
>
> That makes the Phone much more faster in response and reduces laggs a lot
.. in other words, u can't "feel" it anymore when the CPU freq is switching
.... just a Tip ;-)

Those are usecs man... Considering that freq switch delay is in order of 10
msec, that is a thousand times less... I can't imagine it being
noticeable...


Ok, Maíra. Done.


TeamCity xBIM :: XbimScripting :: Build [Build 29](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2208&buildTypeId=XBIM_XbimScripting_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:23


Right. In fact, this VARARGS business is creating friction with our original design. Perhaps we should carefully rethink how we translate function declarations and definitions. As soon as we have time to :-)


GroundDomainUpdate (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (747 of 747 tests passed)

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


The cause of the problem is in class Zend\Mvc\Service\ConsoleAdapterFactory, if we're not in a Console environment, the method createService() will return a stdClass object.


Never enter in this condition.


it looks like this isn't an issue since cb-event-forwarder hard codes localhost as the streaming server hostname. So I think we should be good to go there.


Yes, very good point.  We should only call  self._put_it_back() if waiter.done() is true.  Fixing.


I hadn't seen this before merging my changes but this is much stronger.


OK.  Adding comment.


OK.


Ok, lose the shortcut options then, and go for the idiomatic Python route of named args for all.  If someone wants to pass in a token, they can use `AblyRest(token=token)`, but could still use `AblyRest(api_key)`


Is handled by _socket_disconnected.


This commit wiped out `settings.rst`, thus [this page is empty](http://cartridge.jupo.org/configuration.html).


BarcodeCount?


modified version of an example on matplotlib's website that works for pyuniversal library and correctly works with version of matplotlib and python


Let's be a little more explicit... let's say it's the output of running the command `md5sum` on the FASTA file.  Otherwise there are different ways of defining the MD5 hash.


I think that the 1 year grace period I added with https://github.com/HdrHistogram/HdrHistogram/commit/624ebd2274a489b30e02277f962d5c71e3ecf1f0 is a cleaner "fix" for allowing race log starts with clean parsing. My reasoning for avoiding a forceAbsolute option on reader construction as a solution, and trying for the default behavior on the reader to deal with "problem" situations well is that I'd like to have all common processing tools (including e.g. HistogramLogProcessor) able to logs, including ones with expected "problems". Since racy start timestamps for StartTime and the first log line is "possible" (in the sense that we'll likely see it in the wild quite a bit, even if it's not recommended behavior), choosing to deal with that situation well without needing special parameters is preferable.

Do you think that given the 1 year grace time, the problem leading to the forceAbsolute above is taken care of? If so, can you re-submit a PR with just the tests ('cause I really want to take those in)?


What is the scenario where we would have the Barcode info but no `bc` tags present?


5058-fix-readvars-cli (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1795 of 1795 tests passed)

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


I don't know why this would be necessary. The request for the call is made synchronously, so the client.calls.create(...) call should handle everything you'd need.  I wish I had a better answer for you.


notice any speed improvement? Likely not but you could try a channel that has a huge list and then ar-pi1 :)


I debugged, and indeed, that's `clearOptions()` which is the culprit. It calls `clear()` without parameters, while the latter can accept a `silent` parameter. But `clearOptions` doesn't have such parameter, alas. Would need to raise an issue on Selectize, but they accumulate as the project owner is busy elsewhere.
So your workaround is fine, after all... :smile:


다음꺼 커밋할께


Algebra functions removed


More algebra functions removed - looks like a bad merge


I copied the code from the vdr output, but did notice it wasn't working quite right. How can I get the pts for a specific frame, via frames[start]?


wat


Okay, thanks. I'm also looking forward to `:JacVec`!


Did you check that allowing rain/irrigation to null won't cause problems with water register computation


Fixing it. new PR is under preparation


ah, but then we really do need separate entries. One summarising what is in `weave status peers` and one what is in `weave status connections`.


I am sorry, but you are mistaken


Redmond? Cool.


@RevanProdigalKnight god, what have you done?! now our comments will perish in the cloud, unread and stripped of any value and meaning. tree does not fall if noone is there to hear it.


What are your favorite movie genres?


@jacklin213 was there a reason why these abilities (FireBall, Cook, FireBlast, Firejet, etc) could not be updated to use getAbility()? At the moment these are still going to be really server intensive.


TeamCity xBIM :: XbimScripting :: Build [Build 30](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2209&buildTypeId=XBIM_XbimScripting_Build) is now running


Introduced via the rebasing, yeah, see lines below for the actual change.


ha, I missed that one. You're right, in fact, `_module_.[[RegistryEntry]]` is only used in `HostResolveImportedModule` which itself already asserts that `_module_` is a Module Record.


Concerns over API qps, as discussed


Add heading 4


i would use --python-arg- as the prefix...


changes here...


Please use a clearer name for the app like `myFirebaseApp`


All id fields should be long, not int


fixed here 1fb49bee90b123593fdff7848554ed2e611be8f9


In this case it's `!__ctfe`, but there are CT functions in the latest version, where that would make sense.


Please don't use any sort of annotation. Readers should be responsible for knowing about that and it makes the example much easier.


on top of preferring --python-flag-, instead, i'm not sure that it's a good idea to separate args and flags... i'll dig further into the code and check why you chose to do so and if i can see an alternative way...


updateCachedKeys


Note about annotations applies equally here. They're actually entirely unnecessary here since you're never gonna minify tests.


It mostly shouldn't happen, but if you've got long amplicons like HLA Class
2 and low coverage its possible that you won't see any adapters.
On Aug 5, 2015 9:40 AM, "David Alexander" <notifications@github.com> wrote:

> What is the scenario where we would have the Barcode info but no bc tags
> present?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bnbowman/PacBioFileFormats/commit/e8942504a4c24b6a944036b98bd988136ebd1707#commitcomment-12546049>
> .
>


TeamCity xBIM :: XbimScripting :: Build [Build 30](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=2209&buildTypeId=XBIM_XbimScripting_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:03:00


>Unrelated but I saw your firefox issue with mega and I've had the same issue, you need to either disable multi-process in Firefox or uninstall the mega extention

Or people can use non-shitty file hosting services? Dropbox seems to work well for most people.

@Canzah Is that native resolution? Looks pretty sharp for it to be native but then again I've never played these games.


Just these two comments


Needs to be called


Where do you run? Park, riverside, around sports ground?


Since there is only one template instantiation per interface type, I think it should be fine w.r.t. repeated messages, even if the struct gets constructed multiple times.

Regarding the check, I think whatever the rule is, it should be enforced for all interface (client/server) generators the same way, so this was the logical place. But maybe we can add a possibility to disambiguate at the class level (using a UDA). It's already possible to explicitly pass the instance as an interface reference, though...


Mostly, with search and replace etc.
Some were so awfully written with no spaces between vars and operators etc, that was impossible
That was what killed parsing earlier, decided it was going to take 4 times as long to write anything to try to do it automatically as it would to just do them by hand
As these are 'new' components, I have 'rewritten' quite a lot so that it will be a lot easier to do something similar in the future


@CurlyMoo it might be useful indeed! But why do you say that?


Include the object here rather than assigning a var


Hmm, I think you've got the multipler and divisor the wrong way round, so your clock might be running fast!


Sure!


Right, ``waiter.cancelled()`` would be a redundant check here. ``waiter.done()`` is enough.


We don't want to match everything outside /go because, for example, we don't want to reformat Godeps/_workspace or third_party, since that code doesn't belong to us and we want to track the upstream.


Heh, I think you left some debug code in here.


```js
function (argument) {
  // do work
}
```

Space after function keyword and arguments


(That's not relevant to `latest_release_doc_updates` though)


This looks very good. I will merge that into the PR (saying that this is the more lightweight we can do, but still waiting for the public beta to port it to v4). Will do it in the coming days.


add this snippet to your header.inc.file


No underscore/lodash here. `Object.keys`


We can and should have different short/long hints.  The short hints show up next to the input, while the long hints require a user to hover over the 'i' icon for more details.  This way we can add extra information...

I also think we should better explain what 'iterations' means and what happens if the number of rows exceeds the max.


For this particular change I'm not going against importing inside the
function. We could decide about better policies on a separate thread
because this is not the focus of this pull request
Ok to merge if you're good with that

Em qua, 5 de ago de 2015 13:35, Lasse Schuirmann <notifications@github.com>
escreveu:

> looks good besides the pointed out things, waiting for @erickwilder
> <https://github.com/erickwilder> 's comment before merging
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/astaric/babel/commit/a1cc3f1ca8149cddd97bd6823857cc9e3eba544e#commitcomment-12545948>
> .
>


If I was to read it out aloud, the summary of the peer information should say "There are 3 peers with a total of 6 connections between them, 4 of which are established, and 4 are unestablished". And the summary of the connection information should say "The local peer has 4 connections, 2 of which are established, 1 unestablished, and 1 failed".

We need to somehow find a way of distinguishing "connections between peers" and "connections to/from the local peer". I worry that users may get confused (just as I did!)


Also need to remove the constraint at https://github.com/mysociety/alaveteli/blob/develop/config/application.rb#L88


This closes #138


Right...  I just wasn't going for that level of detail!


Yeah, I agree the session should get terminated after an invalid packet, but that's kind of out of scope for this commit.  Simply bailing out shouldn't cause any harm, their bad packet would just be ignored.  The official minetest client does not send invalid packets at this protocol layer so it shouldn't be an issue unless somebody is using a hacked client (which is what I'm mostly concerned about) :-)


@fehguy

It would be great if we could do something like this:

```
{
    "in": "body",
    "name": "abstractPet",
    "schema": {
        "oneOf": [
            {
                "$ref": "#/definitions/ConcreteDog"
            },
            {
                "$ref": "#/definitions/ConcreteCat"
            }
        ]
    }
}
```


@TkTech @Dav1dde fail


:shipit:


The problem found in #193 was that the real goimports command would return exit status 0 even when it failed (we confirmed this at the time from the source code). The only way to know it failed was to look at the message it printed on stderr. Then the problem is that it's not possible in Bash to pipe both stdout and stderr - we need stdout to see which files need formatting.

It's been a while, so maybe someone has fixed goimports to return non-zero exit status. Can you check on that?


Object.keys


そう！正確にはcastしてないけど！文字列を期待するところに数字が来てたからおかしかった


HAHAHAHAHAHAHAHAHAHAHAHAHAHAHAHAHAHA


not sure what you mean by the comment here -- if the references won't be the same even if you use the same url


For the first comment, that looks okay, but I think some macro sugaring is probably best.

As for the second question, `Signatures` is kinda the AST of our interface definitions. Presumably, as we add additional features, Signatures might get additional fields to support those features. We should mostly focus on making the highest level API as nice to work with as possible and the lowest level API as flexible as possible.

I think the lowest handing fruit right now is some macro sugar to help define a Protocol in much the same way as a Scala trait.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 264](https://teamcity.labkey.org/viewLog.html?buildId=264731&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:10:14
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


5058-fix-readvars-cli (Myoldmopar) - i386-Windows-7-VisualStudio-12: Tests Failed

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


I am curious why this change was made in this way instead of modifying the tables where the initial precision is given (specifically lines 59 to 86 of [radio/src/telemetry/frsky_sport.cpp](https://github.com/opentx/opentx/blob/master/radio/src/telemetry/frsky_sport.cpp)?


property name is wrong (studentNO)


> Thanks, I just noticed that on our big screen and was playing. I don't
> understand why our dart stable build seems tri-modal (four if you count
> fast failures).
>

I've added a filter for build results, so now you can filter out the
failures.

> Bigger problem is the graph doesn't go back far enough in history to show
> me when the time increased.
>

Do you have a rough idea how long ago you were still seeing the 18 minute
builds?
I can manually import some older builds, if you like. But let me first
change the code so it save if it is a PR.

BTW : Any chance you could send me a picture of the Buildtime Trend
dashboard on the big screen in your office? ;)


Lance's suggestion.  This allows us to sanity check barcode labels in the
absence of the sequence file, but more usefully it allows us to prepopulate
our list of expected barcode pairs without traversing the whole BAM dataset
an additional time.
On Aug 5, 2015 9:39 AM, "David Alexander" <notifications@github.com> wrote:

> BarcodeCount?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bnbowman/PacBioFileFormats/commit/e8942504a4c24b6a944036b98bd988136ebd1707#commitcomment-12546017>
> .
>


I did not know this.  Agreed.
On Aug 5, 2015 9:40 AM, "David Alexander" <notifications@github.com> wrote:

> Let's be a little more explicit... let's say it's the output of running
> the command md5sum on the FASTA file. Otherwise there are different ways
> of defining the MD5 hash.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bnbowman/PacBioFileFormats/commit/e8942504a4c24b6a944036b98bd988136ebd1707#commitcomment-12546034>
> .
>


I replaced PageMenu.swift in my project with this code, and it started happening. Here is my code that I use for configuring the PageMenu:

        for(var i = 0; i < daveningURLs.count; i++)
        {
            let viewController = self.storyboard?.instantiateViewControllerWithIdentifier("DataViewController") as! DataViewController
            viewController.title = daveningTitles[i]
            viewController.currentUrl = daveningURLs[i]
            viewController.tempFontSize = defaultFontSize
            viewControllers.append(viewController)
        }

        print(daveningURLs)
        var parameters: [CAPSPageMenuOption] = []

        if #available(iOS 8.2, *)
        {
            parameters = [
                .CenterMenuItems(true),
                .MenuItemSeparatorColor(UIColor(netHex:0x0066FF)),
                .ScrollMenuBackgroundColor(UIColor(netHex:0xEDEDED)),
                .ViewBackgroundColor(UIColor(netHex:0xEDEDED)),
                .SelectionIndicatorColor(UIColor(netHex:0x0066FF)),
                .AddBottomMenuHairline(true),
                .MenuHeight(50.0),
                .EnableHorizontalBounce(true),
                .MenuItemFont(UIFont.systemFontOfSize(20, weight: UIFontWeightLight)),
                .BottomMenuHairlineColor(UIColor(netHex:0x0066FF)),
                .SelectionIndicatorHeight(1.0),
                .MenuItemWidthBasedOnTitleTextWidth(true),
                .SelectedMenuItemLabelColor(UIColor(netHex:0x0066FF)),
                .MenuItemSeparatorWidth(4.3),
                .MenuItemSeparatorPercentageHeight(0.5),
                CAPSPageMenuOption.MenuMargin(20.0)
            ]
        }
        else
        {
            // Fallback on earlier versions
            parameters = [
                .CenterMenuItems(true),
                .MenuItemSeparatorColor(UIColor(netHex:0x0066FF)),
                .ScrollMenuBackgroundColor(UIColor(netHex:0xEDEDED)),
                .ViewBackgroundColor(UIColor(netHex:0xEDEDED)),
                .SelectionIndicatorColor(UIColor(netHex:0x0066FF)),
                .AddBottomMenuHairline(true),
                .MenuHeight(50.0),
                .EnableHorizontalBounce(true),
                .BottomMenuHairlineColor(UIColor(netHex:0x0066FF)),
                .SelectionIndicatorHeight(1.0),
                .MenuItemWidthBasedOnTitleTextWidth(true),
                .SelectedMenuItemLabelColor(UIColor(netHex:0x0066FF)),
                .MenuItemSeparatorWidth(4.3),
                .MenuItemSeparatorPercentageHeight(0.5),
                CAPSPageMenuOption.MenuMargin(20.0)
            ]
        }

        var height = self.view.frame.height - (self.navigationController!.navigationBar.frame.size.height) - 20
        let yPos = 0.0 + (self.navigationController!.navigationBar.frame.height / 2) + self.navigationController!.navigationBar.frame.height - 10

        if(self.canDisplayBannerAds == true)
        {
            height -= self.navigationController!.navigationBar.frame.size.height - 10
        }

        pageMenu = CAPSPageMenu(viewControllers: viewControllers, frame: CGRectMake(0.0, yPos, self.view.frame.width, height), pageMenuOptions: parameters)
        pageMenu!.delegate = self
        self.view.addSubview(pageMenu!.view)

Thanks!


@aailpl  e @patriciamadureira87

the code is operational. after you check it out give me some :+1:  so I can merge it.
cheers!


I would create a local variable for the directory, and another for the file itself which reuses that variable.


@gjcarneiro I'd also recommend you to avoid merging master branch in your asyncio PRs. It doesn't really help since we merge 'by hand', instead of using github automerging. But it does make reviewing the PR harder.


Instead of grep | cut, you can run in a single command using (awk '/fabric_core/ {print $1}' $ifc_filename)


@raelgc I like this big button, but it breaks the mobile layout. Need help figuring out how to fix it.


@StormBytePP bring zero to type long.


This is weird behavior. It is working for now, but I am nervous about this :-)


If all you are comparing is just size, what is the logic behind creating a new array list?


Instead of using cat, you can use the syntax $(</sys/class/net/$interface/address). In this way, bash itself opens the file for reading, rather than spawning a separate, and unnecessary, child process.


This test only tests whether the parameters of the opmethods are specified correctly. The other tests however are tested analog to what Martin did with the initial LocalMean implementation.


I would double-quote the echo string.


I changed that to`lft.getLoadBase()` because I changed `calculateBaseFee()` to return fee units.  The old comment at the top of `calculateBaseFee()` was incorrect.  I changed the implementation of `calculateBaseFee()` to match the comment.  Also note that there was a `FIXME` on the declaration of `calculateBaseFee()` in `Transactor.h` which I've removed.

Anyway, previously this calculation took place entirely in units of drops, since `view().fees().units()` was in drops (see https://github.com/ripple/rippled/blob/develop/src/ripple/app/ledger/Ledger.cpp#L1072 and https://github.com/ripple/rippled/blob/develop/src/ripple/core/impl/Config.cpp#L55 ).  It now converts feeUnits (`calculateBaseFee()` and `lft.getLoadBase()`) to drops (`view().fees().base()`).


This also should move up to the top.


While this is technically correct, it's causing RTL language titles to appear on the right, which probably wasn't intended.

![device-2015-08-05-125444](https://cloud.githubusercontent.com/assets/3903757/9092159/561f6220-3b72-11e5-9dae-f700cf68928e.png)


My guess is that a specialized ``JacVec`` evaluation would be about 10x faster (obviously depends on the model) than computing the jacobian explicitly and doing the sparse mat-vec. If this becomes a bottleneck for you, let me know and I can take a look at implementing it.


Add this code in
#sliding_marque {
	height: 38px;
	overflow: hidden;
}


i want talk to you please
i need  to understand sphere decoder


Nice


Same here regarding the creation of an array list.


cool !!


The package should be `net/minecraft/nbt`.

The names for all NBT components are `TagXXX` not `NBTTagXXX`. e.g. `TagCompound` not `NBTTagCompound`


`net/minecraft/entity/boss/`


You could add here `[C <: AnyTypeSet.Of[AnyBlastCommand]](val commands: C, ...) { type Commands = C }`. And then
```scala
case class numThreads(val number: Int) extends BlastOption(
  commands = blastn :~: blastp :~: ∅,
  toSeq = Seq("-num_threads", s"${number}")
)
```


Ability to remove tires without a jack if character have enough strength.


@davispuh All I did was say my opinion. If you ever have (or do have children) and one of them has a special need and comes home to you crying that the other kids call you a retard, you will get this terrible sinking black hole feeling inside you, because it's something you can't fix. You just hug your boy and tell him him, "NO YOU ARE NOT! THEY ARE THE ONES WHO ARE JERKS!".

If you are ever in that situation, I GUARANTEE you will switch sides, unless you were born without a single ounce of love and compassion.

I imagine that is what started this whole thing: one of the owners or moderators of GitHub has, or knows someone they care deeply about, been there. It changes your life forever.  A dividing line. There is a "before", when all you care about is yourself and your problems, and then there is an "after", where you will lay your life on the line for someone you love. You don't care if you live or die, protecting that person who literally NEEDS you becomes your mission in life. That boy (or girl) has special needs.

If making a post on a community I don't even use can make even one individual think to themselves "maybe there is a better, or nicer way of getting my message across, one that doesn't make innocent children miserable" then I feel I've possibly done some good in this life.

Like I said in my OP, I actually don't think that was the intent of the developer, he (or she) was being off the cuff funny (they thought). But that stuff spreads - if a family's "cool uncle hacker" throws around words like retard and nigger, and thinks "intelligent" people are better than "dumb" people, those impressionable youngsters are gonna take that to school, and the bullying begins. I fucking hate bullies. They hurt my son and make him cry.


No, it will be a separate class.


I noticed this too and was about to make a PR for it :)


#### header 4


Hmmm.   I have to look at what the compiler is doing with @synchronized(_date).  My understanding is that it will make some kind of call to a lock function with _date as the parameter, in which case it has the same problem as the stringWithFormat call.

The original problem was:

UI thread calls stringWithFormat, passing original _date value.
Before the _date value gets used, another packet arrives and releases the original _date, freeing its memory.
stringWithFormat tries to use original _date value and crashes.

If you use @synchronized(_date), then all uses of _date need protecting individually.  The broader protection of all member objects by @synchronized(self) seems easier to me.

I will look at the disassembly of @synchronized(_date) and report back.


yup, I just don't want to write that now for all options :)


I guess this does not obey the import statements order anymore.

org
javax
java
com


The issue is that the package won't necessarily give a tight specification for the version of its dependencies. (`"my-dependency": "*"`) Admittedly it's a bit of a theoretical problem.


I see, yeah, that sounds like the best way to do this.


@gmoneda , não faz sentido algum essa classe que vc criou dentro do Card.
Favor removê-la.

valeu!


right, and case class with unset typemember compiles anyway :+1: :trollface:


Just try to move a file.
After the update from Andy it is working now.


@jedesah The compiler and shapeless go wild :/ My types for `specifyx` are almost always sealed traits, e.g:
```
You requested an element of type common.protocol.Result, but there is none in the HList shapeless.::
```
Where `Result` is just this:
```scala
sealed trait Result
case class Accepted(cid: UUID) extends Result
case class Rejected(reason: String, cid: UUID) extends Result

object Result {
  implicit val c1: Codec[Accepted] = sc.uuid.as[Accepted]
  implicit val c2: Codec[Rejected] = (utf8_32 :: sc.uuid).as[Rejected]
  implicit val cp: Codec[Result]   = Codec.coproduct[Result].discriminatedBy(sc.uint8).
    using('Accepted ->> 1 :: 'Rejected ->> 2 :: HNil).as[Result]
}
```
Hopefully I just miss some import or similar. Kind of wish to move to 1.4 ASAP.


dodalem linijke komentarza


GitHub. I am disappoint.


Bugs as features!


I guess I don't *need* any feature, but the flags are directly mapped to RTLD_ definitions, so it would need to be re-written. I think it's a lot of work for no additional portability (because you can't use it on a system that doesn't have dlopen, because bash would disable plugins (it directly uses dlopen).


I do not believe it is an Appium bug; we were able to trace it back to this commit. From my understanding, line 288 has always evaluated to false, and it is line 297 that has consistently returned true for all platforms. Since these lines have been removed and replaced with the equivalent of line 288, all the tests on iOS are now returning false.

We are currently using Appium 1.4.6 and Selenium 2.43. We upgraded to Selenium 2.46 yesterday which is when our tests broke. We are using iOS and Android based devices, but only the iOS devices seem to be affected, which means line 289 returns true for Android and false for iOS.

Thoughts?


Will there always be one state in the list? If yes, why then make it a list?


This looks fine to me.

If Gumby is RIP, we can easily (I suspect) switch to Bootstrap. That stuff is cake in Bootstrap.


I'm ok with removing the virtualenv/python2.7.9, but we still want to have travis cache the pip cache, as much of the time is spent on installing gaia.


:+1: This is correct, but a bit redundant as you are evaluating `true && true`. One alternative would have been to keep just `if (true)` or changing the operator into an `||` like `if(false || true)`. However, all three options are valid.


Typically what you need to do is the following
cd build
cmake ..

I usually have different builds and then put them in build/debug build/profiling build/cluster1 build/clusterX

In each one of those it is then cmake ../..


Sorry, I just removed those to force pip in fetch stuff. I wil force push a change ....


On Wed, Aug 05, 2015 at 07:59:59AM -0700, Stéphane Graber wrote:
> Hmm, making set-default's argument optional seems very wrong to me. Can we just support empty string instead?

Empty string seems too magical and prone to mess up scripts.  How about "none"?


creation class PHT


Выглядит круто :+1:


So http://bower.io/docs/api/#version is not valid any more ?


Because that table defines the format we *receive* from the sensor. If it was changed the received data would be interpreted wrong.

It logically also defines the default displayed precision which explains you why it was in cm.


@smarterclayton updated: https://github.com/mfojtik/origin/blob/image-promotion/docs/proposals/image-promotion.md


Update Eichlerlab fork of ssf_DTS_caller


looks rad


This seems to be copy/paste with the remove/add block above. Probably a common function can be made out of the two of them.


Did you migrate the glyph.cells to store booleans instead of strings?


As the `cache-config` setup is specific to `sw-precache-cache`, should we comment this out as well as include it in the instructions for re-enabling SW support in the README?


$().nestedSortable({relocate = function() {
})};


It's necessary because when the drag:start event is fired the target is hided. Then, when the drag ends, the target is showed again.


you -> your.


perhaps instead:

> however we are informed by @frozenlock that it is still not as fast as Chrome if you are using **deeply** nested flexbox layouts (much more nested than our demo app).


This commit has been reverted by https://github.com/DSpace/DSpace/commit/88398d9caeeed019078f3c5b07ed4bb623b4b45f


A few copy edits:

logined  -> logged in
simpliy -> (remove this word, it is unecessary)
poped -> popup


Oh I see, the `resize_` function calls this, so this used to be done every time, but now `resize_` isn't always called, which is why you need this. It should be enough to add this after line 99 though, since when the else clause runs this will already be called by `resize_` like before.


@gabest11 : I noticed you got "stolen" game. I thought you were getting "horsez". That's the one which suffers from Colorless HW mode.


Done.


UPD


Shouldnt this be abstracted away somewhere else?
We need to reject on improper OS in the same way for places where we read files as well


I would only pass "Server string" here - you don't really need the client.Config - that would be a more minimal change.


Thanks - that makes sense.


please_use_underscores for names instead of CamelCaps


I just had a look in more detail and I understand it better now. To build a packet, three bytes are used for 'ABC', one byte for the command length and one byte for the command, for a total of 5 bytes.

The confusing part was that if `cmdlength` is 0 or not defined, only three bytes would be needed.

I see you've already changed it in your branches, so it should be fixed when you send the next PR.


Got that `Result` issue worked out, however the compilation times are almost exponential per each `.specify`. I probably need to restructure my code into separate compilation units or similar. I'll report back as I work things out.


@oliversong, since we're using node 0.12.x, it seems like these features might be baked into node? see the [release notes](http://blog.nodejs.org/2015/02/06/node-v0-12-0-stable/):

> - HTTP
>   - maxSockets are no longer limited to 5. The default is now set to Infinity with the developer and the operating system given control over how many simultaneous connections an application can keep open to a given host.
>   - Proper KeepAlive support means that sockets will stay open until they timeout at the configured time, are closed by the remote side, or the process exits. Developer's no longer have to make sure requests have been pipelined to keep the socket open, or use an alternative module to get that support.

although it seems [there may be some issues configuring keepAlive](https://github.com/joyent/node/issues/9075), we should consider preferring an instance of [`http.Agent`](http://nodejs.org/docs/v0.12.0/api/http.html#http_new_agent_options)


@daveryder

Here's the problem. People only have some much time/energy/what-have-you for "stuff" outside their wheelhouse.

If whatever the pet issue is isn't relevant to you, more and more it just feels like noise. This is going to sound harsh but it is what it is; other people don't have time for the never ending cavalcade of other people's problems.

This isn't to say people should go around shouting slurs just because. Or that no one should make an effort to sympathize/empathize with others. What I'm saying is making big issues over small things has this tendency to delegitimize everything that's coming after it. Right, wrong, or indifferent.

If you want anyone to care about what you care about you have to 1) tailor it to your audience and 2) be very selective because the time of patience of others is VERY limited.

That said, I feel for you.


Sure, great. Please let me know if you have any issues.


J factors for The random flights model and Rouse model are consistent now.


Am I seeing double?


Poor baby Gators, empathy is hard :(

On Wed, Aug 5, 2015 at 12:26 PM, Chris Cummings <notifications@github.com>
wrote:

> @daveryder <https://github.com/daveryder>
>
> Here's the problem. People only have some much time/energy/what-have-you
> for "stuff" outside their wheelhouse.
>
> If whatever the pet issue is isn't relevant to you, more and more it just
> feels like noise. This is going to sound harsh but it is what it is; other
> people don't have time for the never ending cavalcade of other people's
> problems.
>
> This isn't to say people should go around shouting slurs just because. Or
> that no one should make an effort to sympathize/empathize with others. What
> I'm saying is making big issues over small things has this tendency to
> delegitimize everything that's coming after it. Right, wrong, or
> indifferent.
>
> If you want anyone to care about what you care about you have to 1) tailor
> it to your audience and 2) be very selective because the time of patience
> of others is VERY limited.
>
> That said, I feel for you.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12546918>
> .
>

--
Kurtis Rainbolt-Greene, Hacker
Software Developer
1631 8th St.
New Orleans, LA
70115


Note that for example the paths length limits of most common Unix file systems  can be much longer.
See https://en.wikipedia.org/wiki/Comparison_of_file_systems#Limits


This is wrong
Zip lets you build tuple from lists
Read up what is zip
I was recommeding to zip temp and corresponding pins.


temp is not really a great name for a varaible.


Piss off Ghazi, you're not welcome here.

		 ---- On Wed, 05 Aug 2015 10:29:08 -0700  notifications@github.com  wrote ----Poor baby Gators, empathy is hard :(

On Wed, Aug 5, 2015 at 12:26 PM, Chris Cummings <notifications@github.com>
wrote:

> @daveryder <https://github.com/daveryder>
>
> Here's the problem. People only have some much time/energy/what-have-you
> for "stuff" outside their wheelhouse.
>
> If whatever the pet issue is isn't relevant to you, more and more it just
> feels like noise. This is going to sound harsh but it is what it is; other
> people don't have time for the never ending cavalcade of other people's
> problems.
>
> This isn't to say people should go around shouting slurs just because. Or
> that no one should make an effort to sympathize/empathize with others. What
> I'm saying is making big issues over small things has this tendency to
> delegitimize everything that's coming after it. Right, wrong, or
> indifferent.
>
> If you want anyone to care about what you care about you have to 1) tailor
> it to your audience and 2) be very selective because the time of patience
> of others is VERY limited.
>
> That said, I feel for you.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12546918>
> .
>

--
Kurtis Rainbolt-Greene, Hacker
Software Developer
1631 8th St.
New Orleans, LA
70115
—Reply to this email directly or view it on GitHub.


This looks like a really sweet update. I am glad you fixed my temporary directory stuff.


What about reduciing the if statement?
Could you think of any trick?


I scrolled upwards when I searched for the name of the game, never realized the first post had another one!


Hard-coded "Fetching URL" should be a string resource.


Not sure what you're trying to say but "Must be required by an" seems like it's missing something


Hey @gridaphobe, could you give me a rough idea of why this patch is necessary? I'd like to add the rationale to the patch.


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/dev/4ea5500dcef2f398dff8dae77a10d5c8319782e9/index.html)


Fairly sure it was Gator repos that got banned.

On Wed, Aug 5, 2015 at 12:30 PM, Arthur Kay <notifications@github.com>
wrote:

> Piss off Ghazi, you're not welcome here.
>
>
>
> ---- On Wed, 05 Aug 2015 10:29:08 -0700 notifications@github.com wrote
> ----Poor baby Gators, empathy is hard :(
>
> On Wed, Aug 5, 2015 at 12:26 PM, Chris Cummings <notifications@github.com>
> wrote:
>
> > @daveryder <https://github.com/daveryder>
> >
> > Here's the problem. People only have some much time/energy/what-have-you
> > for "stuff" outside their wheelhouse.
> >
> > If whatever the pet issue is isn't relevant to you, more and more it just
> > feels like noise. This is going to sound harsh but it is what it is;
> other
> > people don't have time for the never ending cavalcade of other people's
> > problems.
> >
> > This isn't to say people should go around shouting slurs just because. Or
> > that no one should make an effort to sympathize/empathize with others.
> What
> > I'm saying is making big issues over small things has this tendency to
> > delegitimize everything that's coming after it. Right, wrong, or
> > indifferent.
> >
> > If you want anyone to care about what you care about you have to 1)
> tailor
> > it to your audience and 2) be very selective because the time of patience
> > of others is VERY limited.
> >
> > That said, I feel for you.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12546918
> >
> > .
> >
>
>
>
> --
> Kurtis Rainbolt-Greene, Hacker
> Software Developer
> 1631 8th St.
> New Orleans, LA
> 70115
> —Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12546993>
> .
>

--
Kurtis Rainbolt-Greene, Hacker
Software Developer
1631 8th St.
New Orleans, LA
70115


Typo: should be "expected" rather than "exprected"


@MrUpsideDown , this is an interesting and powerful change. We are a bit concerned, however, that this might make what's already a somewhat complex UI even more complicated for the average user (i.e. double-negatives, etc). Do you have a use-case that you could share that would require this type of advanced grouping and which can't be accomplished through creative grouping of the current all/any filter groups?


>All I did was say my opinion. If you ever have (or do have children) and one of them has a special need and comes home to you crying that the other kids call you a retard, you will get this terrible sinking black hole feeling inside you, because it's something you can't fix. You just hug your boy and tell him him, "NO YOU ARE NOT! THEY ARE THE ONES WHO ARE JERKS!".

You're right. I would be offended by that.

>If you are ever in that situation, I GUARANTEE you will switch sides, unless you were born without a single ounce of love and compassion.

However, this has absolutely nothing to do with GitHUB decision. Using the word jokingly (like the "for dummies" series) is vastly different than using the word directly to refer to a mentally challenged person.

>those impressionable youngsters are gonna take that to school, and the bullying begins. I fucking hate bullies. They hurt my son and make him cry.

You are stretching that extremely far


@jnimety said he will deploy tomorrow.

It's probably safe to say that we're all ready to move away from Gumby. I think @caseywatts had some ideas there, too.


> Those are usecs man... Considering that freq switch delay is in order of 10 msec, that is a thousand times less... I can't imagine it being noticeable..

Hmm i've build the kernel at least 3 times .. because i didn't also believe what i see on UI .. but it's true.
If u like to test.. use exactly my LiveOPP Version.. we all three are using different LiveOPP Version/Code ... and set one time in the linked File 50 for latency .. and one time with 10.
U'll feel the difference after some seconds when playing with the UI.
When i had asked cocafe why LiveOPP needs transition latency 30 - 50 and without, only 20, he said because LiveOPP needs more time in order to set new pll, new voltage and so on ..
And reducing this latency should make it more unstable .. but it's stable since months with even 10. :-)


@jonleung you say "set variable 'y' to be equal to 2" but you say to type var y = 4 into the console.


This patch no longer applies, so I'm trying to figure out what needs to be done to fix the build. Removing the patch entirely allows this to build successfully, but I don't know if the build is then silently bugged.


,,!,,     ,,!,,Went too far when you shitbirds came after this repo just because it's redchannit's favorite webm conversion tool. You're not even honestly offended by the name, you just want to kick over them dirty gamergate losers' sand castle. You're not even happy with the name change, you still want the repo banned. So once again, piss off.

		 ---- On Wed, 05 Aug 2015 10:32:45 -0700  notifications@github.com  wrote ----Fairly sure it was Gator repos that got banned.

On Wed, Aug 5, 2015 at 12:30 PM, Arthur Kay <notifications@github.com>
wrote:

> Piss off Ghazi, you're not welcome here.
>
>
>
> ---- On Wed, 05 Aug 2015 10:29:08 -0700 notifications@github.com wrote
> ----Poor baby Gators, empathy is hard :(
>
> On Wed, Aug 5, 2015 at 12:26 PM, Chris Cummings <notifications@github.com>
> wrote:
>
> > @daveryder <https://github.com/daveryder>
> >
> > Here's the problem. People only have some much time/energy/what-have-you
> > for "stuff" outside their wheelhouse.
> >
> > If whatever the pet issue is isn't relevant to you, more and more it just
> > feels like noise. This is going to sound harsh but it is what it is;
> other
> > people don't have time for the never ending cavalcade of other people's
> > problems.
> >
> > This isn't to say people should go around shouting slurs just because. Or
> > that no one should make an effort to sympathize/empathize with others.
> What
> > I'm saying is making big issues over small things has this tendency to
> > delegitimize everything that's coming after it. Right, wrong, or
> > indifferent.
> >
> > If you want anyone to care about what you care about you have to 1)
> tailor
> > it to your audience and 2) be very selective because the time of patience
> > of others is VERY limited.
> >
> > That said, I feel for you.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12546918
> >
> > .
> >
>
>
>
> --
> Kurtis Rainbolt-Greene, Hacker
> Software Developer
> 1631 8th St.
> New Orleans, LA
> 70115
> —Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nixxquality/WebMConverter/commit/c1ac0baac06fa7175677a4a1bf65860a84708d67#commitcomment-12546993>
> .
>

--
Kurtis Rainbolt-Greene, Hacker
Software Developer
1631 8th St.
New Orleans, LA
70115
—Reply to this email directly or view it on GitHub.


Throwback or what... Haha.


You might as well do `data = json.loads(data)` and insert it into the model as a dict.


Typo speculatitve -> speculative :)


beautiful!!! you rock @sebix  :D


Change these two lines to:
```
except ValueError as exception:
    return http.HttpResponseBadRequest(exception)
```


Hey @dhcole, just wondering what the value is of *not* having source maps in production? It seems like it would make any debugging that much easier, with almost no cost.


I only changed it because of one file I had in mind: test.go ;-)

I'll revert this commit.


I don't like us having to reserve a remote name for this :(


Having it empty match with having it not set seems fine to me. You can't have a remote without a name so it doesn't need any special handling there.


^ +1

Can you please release this as stable version? When installing using Composer it still got the v0.7.1 (the latest stable version released on April) and still require to Phly/Http (which is deprecated, so the Composer will show a warning)

Thanks!


I was here first.


Hrm, I don't actually remember adding this patch.. But it looks like one of those situations where clang is stricter in reporting errors vs warnings than gcc.

So if `libxkbfile` compiles patch-free with clang now, I'd just remove the patch :)


@hatzopoulos, you are right...

* I was trying to fix the issue (on Windows) where `crlf` in `stdin` gets silently translated to `lf` (:-1: :-1:) and found that using two arguments with `env` (e.g. `#!/usr/bin/env python2 -u`) doesn't work, due to the issue you mentioned.
* It seems like the correct approach to un-derping the EOL translation on windows is to use [`msvcrt.setmode`](https://docs.python.org/2/library/msvcrt.html). Lemme code something up...


Is this horsez 1 or 2?


@daveryder

> If you are ever in that situation, I GUARANTEE you will switch sides, unless you were born without a single ounce of love and compassion.

I understand you, I've been in a similar position. But it just takes a bit of maturity to understand that the word "retard" is being used with the same sense as i.e. "dummy". They are *not* trying to make fun of actual retarded people, so why should they get offended by that?


I forgot to answer your question about the WebView, @lukeis. It's actually reversed; the findElement calls in the native portion are all still successful, but all findElement calls in the WebView fail.


+ Change pos and other abbreviations to their full name (position).
+ Add the void for `setup` and `loop`
+ Make your if statement simpler like such:
Instead of
```
    if (steeringPos < goal - threshold) { // always move toward the value
            digitalWrite(STEERINGDPIN, HIGH);
            analogWrite(STEERINGVPIN, power);
    } else if (steeringPos > goal + threshold) {
            digitalWrite(STEERINGDPIN, LOW);
            analogWrite(STEERINGVPIN, power);
    } else {
            analogWrite(STEERINGVPIN, 0);
            steeringFlag = false;
    }
```
Do something like:
```

```


kindly fix the values in the circularSquare
goodjob buddy :+1:


Darmowa lista to do - lista zadań i zarządzanie zadaniami


No sure what you mean by waiter.cancelled().  I only added CancelledError
handling (which refers to cancellation of the outermost Queue.get()
coroutine).

On 5 August 2015 at 17:51, Yury Selivanov <notifications@github.com> wrote:

> Right, waiter.cancelled() would be a redundant check here. waiter.done()
> is enough.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gjcarneiro/asyncio/commit/73a8d559a0739e9c747791f4babf5cba8ff7ceec#commitcomment-12546282>
> .
>

--
Gustavo J. A. M. Carneiro
Gambit Research
"The universe is always one step beyond logic." -- Frank Herbert


This is great!


@krainboltgreene

You sound like a small child crying and stomping his feet.


I don't think that zero-length input is defined at least for CBC because the underlying algorithm does not work here:

![](http://i.stack.imgur.com/ET57e.png)

from [6.2 The Cipher Block Chaining Mode](http://csrc.nist.gov/publications/nistpubs/800-38a/sp800-38a.pdf)

When `P_1` is undefined, the whole algorithm does not work.

As far as I understand this, the input padding which does ensure that zero-length plaintext is expanded to block size must be applied before modes come into play.


On Wed, Aug 5, 2015 at 9:57 AM Dieter Adriaenssens <notifications@github.com>
wrote:

> > Thanks, I just noticed that on our big screen and was playing. I don't
> > understand why our dart stable build seems tri-modal (four if you count
> > fast failures).
> >
>
> I've added a filter for build results, so now you can filter out the
> failures.
>
>
> > Bigger problem is the graph doesn't go back far enough in history to show
> > me when the time increased.
> >
>
> Do you have a rough idea how long ago you were still seeing the 18 minute
> builds?
> I can manually import some older builds, if you like. But let me first
> change the code so it save if it is a PR.
>

It was 15 days ago when I'm sure it was 18m
https://github.com/angular/angular/pull/3174
but we only started sending you the data 9 days ago:
https://github.com/angular/angular/pull/3302

Actually I think it would be a really useful feature if you could use the
travis API to backfill data for the time period before the webhook was
added. Most users of this thing are probably adding it because they already
have a problem, and you won't be able to solve that problem with the
current model, only subsequent ones.

> BTW : Any chance you could send me a picture of the Buildtime Trend
> dashboard on the big screen in your office? ;)
>
Photo of Victor Savkin with the screen hovering over his head :)
https://goo.gl/photos/qqUAXLixGnxWb5gU8

> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/angular/commit/30bd2a3fc72f8f1cdb0ffc4b9d11b4ca42e9ece7#commitcomment-12546410>
> .
>


>I had already thought about moving some items (back) from the panel to the widget in order to avoid these widget() calls that tend to become circular. This would for example be slotShowToolbar and the session slots.

I think this is a good idea! That way we don't need to call the widget from the outside...


Yes, absolutely amazing that git keeps project history, now can you go away so the rest of us don't get emails for pointless comments?


It's related to my own comment (first comment for 208 line).


that is correct... platforms == 'linux, 'win32', 'darwin', etc


Ah, I did it only to make sure the patch was still applying to master.

I'll keep that in mind, thanks.

On 5 August 2015 at 17:59, Yury Selivanov <notifications@github.com> wrote:

> @gjcarneiro <https://github.com/gjcarneiro> I'd also recommend you to
> avoid merging master branch in your asyncio PRs. It doesn't really help
> since we merge 'by hand', instead of using github automerging. But it does
> make reviewing the PR harder.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gjcarneiro/asyncio/commit/20d5df7d3bb4ba696bfdca422bf1bc60841b7c19#commitcomment-12546439>
> .
>

--
Gustavo J. A. M. Carneiro
Gambit Research
"The universe is always one step beyond logic." -- Frank Herbert


I think `many` should be capitalized, since what follows the colon is a complete sentence.  After that tiny change I'll merge!


> @daveryder: If you are ever in that situation, I GUARANTEE you will switch sides, unless you were born without a single ounce of love and compassion.

Context matters. If such an event drives you to take issue with every use of the word, regardless of context, then that speaks of your own personal bias - not the inappropriateness of others.

The situation you sketched really has nothing to do with this incident, and there's no plausible evidence that insulting use of a word is encouraged by it being used in a facetious manner - quite the opposite, in fact. The facetious use is derived *from* the insulting use, and really only serves to weaken the effectiveness of it as an insult (as the word is not taken seriously anymore).


Slice means clone. I don't want to shuffle the array of VALUE_LIST, so I clone a new one.


Shall we leave readhits = false by default so people would run simulation by default?


goimports always returns a non-zero exit status now: https://go.googlesource.com/tools/+/master/cmd/goimports/goimports.go

See line 44.

For simplicity, I suggest to drop this pull request and the other one as well. The only situation where we cannot catch an error with the current approach is when goimports returned neither stdout nor stderr output and it's exit code is != 0. That's probably not worth the extra complexity?


Would be great to show an XCode native toast view once that's copied to the pasteboard, don't you think?


sorry - that was from testing it - I forgot to revert this - will do
this just now

Chris

On 08/05/2015 01:47 PM, Jin Huang wrote:
> Shall we leave readhits = false by default so people would run simulation by default?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sPHENIX-Collaboration/macros/commit/a0b203cb21be80d9f63839d8110807bb0ae0c943#commitcomment-12547285

--

*********************************************************

Christopher H. Pinkenburg      ; pinkenburg@bnl.gov

Brookhaven National Laboratory ; phone: (631) 344-5692
Physics Department Bldg 510 C  ; fax:   (631) 344-3253
Upton, NY 11973-5000

http://www.phenix.bnl.gov/~pinkenbu/

*********************************************************


Is this meant to be uncommented?


Why did u add spaces?


saw approval from zeeshanlakhani
at https://github.com/basho/yokozuna/commit/eeb90d057781463606e378bda4c4373610d1fe2f


merging basho/yokozuna/bugfix/fd/RIAK-1937 = eeb90d05 into borshop-integration-529-bugfix/fd/RIAK-1937


basho/yokozuna/bugfix/fd/RIAK-1937 = eeb90d05 merged ok, testing candidate = c102f504


LGTM


Before assigning the string, I would first verify that you do have a valid string
```
if (isString(self.snippet.url.absoluteString)) {
    // Implement pasteboard logic
}
```


@vcunat Well, it's not an incompatible change.


Is this still in here on purpose?


nit - I think we usually keep these at the top of the file (even when the provider is confined) - see https://github.com/puppetlabs/puppet/blob/master/lib/puppet/provider/package/windows.rb for example


My joy can experience no OutOfBounds exception...


Package should be `net/minecraft/nbt/`


having had a look at the way ARG_PREFS is managed, i think that it should be possible to check for i+1 and see if left(1) is not (-) and in that case use it as the argument of the current arg and i++.
otherwise just continue.
what do you think about it?


为什么把 eclass 版本往回退？有特殊理由吗？


Не совсем понял как проверить данные, которые отсылаются в канал. Точнее как передать актуальный хеш. Предполагаю, что нужно делать через заглушку при помощи allow, но как тогда вызвать expect верно?


Okay, I think I fixed the unwanted `stdin`/`stdout` EOL translation on Windows, without killing the shebang in the process: b3b0675b71b8120d6560c6c5187415ea5334a35f


Let's make these log at Info level


you suck as black wizard .... just like the Git


`fullResolutionParsedResult()...` ?


Isn't better to leave this `unsigned` before the `uint8_t` ones?


I am best at splitting something bad into something worse ... nothing comes from nothing except something


It's probably safe then if you want to edit the grep pattern to also include go files in the repo root. Maybe something like: `grep -E '^(go/.+|[^/]+)\.go$'`


I wonder what [XCHUDView](https://github.com/luisobo/Xcode-RuntimeHeaders/blob/fd2e90b6414a20fb83c57ca52fadd97df736ae59/DevToolsInterface/XCHUDView.h) does...


Could we add a test that asserts what happens when we mix matrix and querystring params?


Thank you.


I only have horsez 1 but I know horsez 2 is same engine and most probably suffers from same error.


Perhaps "There are also" is clearer here? Though I do have a tendency to overuse the word "also" so feel free to disagree!


ရပါတယ် တွေ့မှ လက်ဖက်ရေတိုက်တော့ :joy:


this will throw if `config.packageName() === null` & `config.getPreference('WindowsStoreIdentityName') = "some val"`


:metal:


Nothing. New version will be packed into ZIP and released to public when new application features are stable and complete.


Good work. This fixes Horsez and stolen and I guess all other games which suffered out of memory issue. :+1:


done! thanks for the heads up, hope we're getting ready to do some coding together


process is a global, not needed


Done.


process is built in


Yes, well it's more of a "new feature", no big deal.


Expanded the description.


Hmm,  this should actually be used here:

https://github.com/radical-cybertools/saga-python/blob/devel/src/saga/task.py#L658

I would need to look more careful at the code to check if that setting is still needed, I am not sure if/what fallback kicks in.  I assume you don't have the setting in the condor adaptor, and thus assumed that it is not needed?  Is the bulking working ok for you (in condor), in the sense that all jobs arrive in a single container_run call in the adaptor?


the function is called where it is defined, so it's already a value


whitespace :trollface:


86139842-AddAutoSizing (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1740 of 1795 tests passed)

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


@aki EscapeFun is not necessary here, required job is done by QuoteFun.


You're going to have to describe what each of these states means.


:) Ah, that's not worth it. It's already unreadable enough :)


Thanks fixed d7d9afe9aee4e3e2d11945fab1c5d764d4625fed


He did! I like his jekyll-based idea. It's now just a question of storing
local info somewhere.

On Wed, Aug 5, 2015 at 1:34 PM, Lou Rinaldi <notifications@github.com>
wrote:

> @jnimety <https://github.com/jnimety> said he will deploy tomorrow.
>
> It's probably safe to say that we're all ready to move away from Gumby. I
> think @caseywatts <https://github.com/caseywatts> had some ideas there,
> too.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/newhavenio/newhavenio-website/commit/b8f7a583a4146f427ad2d9a9c3fe5325a312eb06#commitcomment-12547056>
> .
>

--
twitter @danbernier
http://wordcram.org | http://newhaven.io


Test


fixes #2


2015-08-05 18:07:46 Event 184901 Action 1 uses nonexistent SpellID 12552.
2015-08-05 18:07:46 Event 1071702 Action 1 uses nonexistent SpellID 16619.
2015-08-05 18:07:46 Event 890602 Action1 uses incorrect Target type 1 for event-type 1 (cannot be used OOC)


lgtm


in table `db_script_string` has Emote 337 but emote does not exist.


Looks good. I think the fluent API is better then the traditonal getter/setter approach.


Add a comment


Ooops
`"Session property value must not be null"`


What is the point of this change? Is time not enough?


@pepelsbey внезапно! До сегодняшнего дня был уверен, что «бекенд». Полез за пруфами, и [оказалось, что «э»](https://ru.wikipedia.org/wiki/%D0%90%D0%BD%D0%B3%D0%BB%D0%BE-%D1%80%D1%83%D1%81%D1%81%D0%BA%D0%B0%D1%8F_%D0%BF%D1%80%D0%B0%D0%BA%D1%82%D0%B8%D1%87%D0%B5%D1%81%D0%BA%D0%B0%D1%8F_%D1%82%D1%80%D0%B0%D0%BD%D1%81%D0%BA%D1%80%D0%B8%D0%BF%D1%86%D0%B8%D1%8F)


should we fail build at this point? seems like something is really wrong


That's not why I said. just $file = $items->getEntity(). And the empty check below isn't needed.


This change is also not related to the Check_MK compatibility change. I would prefer to have individual commits/patches for each feature. This makes it a lot easier to track and understand.


Strips out the group from the version argument. I have renamed the variable for clarity. If you're asking about the syntax, see the substring removal section [here](http://www.tldp.org/LDP/abs/html/string-manipulation.html).


Just to verify ... Instructions only have param names yes? And this stringification function is only for the tests no?


Must have pressed backspace by mistake, whoops. :P


Unfortunately, it doesn't look like this fixed it completely. Everything can compile now that the header file is there, but I'm getting some undefined symbols during runtime (these errors are not present earlier than e573850a8dd3d3de6bd8423f92b93c3fa53ff2dc).

I'll update this if I can figure out a fix.


OK so we finally have support for mixing both together?


I think this is the type of functionality you were looking for in the pystamps methods


I missed the discussion on this. Can you point me to it before I revert, since this is still a contentious point.


Link to issue #52


Fixed


This fn calls console.log/dir at least 4 times. In high verbosity mode it is ok but as default it's a bit overkill imo


Good catch


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1205


releaseFocus should be called before teleportTo, otherwise the NPC will already have released focus since the teleport destination is out of range from the NPC.


Nope >.>


yes, provided the matrix and query params are owned by distinct components


The other parts seem to look fine. I haven't tested them but they seem to be more robust. I would like to apply them. Could you explain the both parts in question?


So a question about this that maybe I should have raised before in the old version... we're making a little bit of an assumption here about location (especially since we're not even looking at `PSModulePath`)

* Do we care about which module path we use since there are a few?  I would think that we want to prefer `Machine` level settings to `User`?
* Are we sure that that module path is always injected for all users (i.e. for `LocalSystem`)?
* Do we want to insulate ourselves from env vars by using the registry (or calling `GetEnvironmentVariable`)?

I did a quick test with my local Administrator:

```
> $env:PSModulePath -split ';'

C:\Users\Administrator\Documents\WindowsPowerShell\Modules
C:\Program Files\WindowsPowerShell\Modules
C:\Windows\system32\WindowsPowerShell\v1.0\Modules\
C:\Program Files (x86)\Microsoft SDKs\Windows Azure\PowerShell\Azure
```

And also with `SYSTEM` (that shows we still get the `Program Files`):

```
PS C:\Windows\system32> $env:PSModulePath -split ';'

WindowsPowerShell\Modules
C:\Program Files\WindowsPowerShell\Modules
C:\Windows\system32\WindowsPowerShell\v1.0\Modules\
C:\Program Files (x86)\Microsoft SDKs\Windows Azure\PowerShell\Azure
```

But note that the `C:\Program Files\WindowsPowerShell\Modules` isn't a configurable `PSModulePath`, so must otherwise be injected.

```ruby
[6] pry(main)> value = ''
[7] pry(main)> Win32::Registry::HKEY_LOCAL_MACHINE.open('SYSTEM\CurrentControlSet\Control\Session Manager\Environment') do |reg|t\Control\Session Manager\Environment') do |reg|

[7] pry(main)*   value = reg['PSModulePath']
[7] pry(main)* end
=> "C:\\Windows\\system32\\WindowsPowerShell\\v1.0\\Modules\\;C:\\Program Files (x86)\\Microsoft SDKs\\Windows Azure\\PowerShell\\Azure"
```

Also, that there are no config values under `HKCU\Environment` for `PSModulePath` either.

So I think that what you're doing here is right (but I might change how we get the path to `ProgramFiles`).

Thoughts?


Ok, that makes sense.  Thanks.


It should fail elsewhere.  This code is based on other code that does something similar in prepare; given that this only occurs sometimes (specifically if publisherId is specified by build.json or as a parameter), I don't think this should be the build gate.  And what is the outcome?  No manifests found = failure from msbuild.


Relations can't be mutable, just return a new relation instance with updated dataset


86139842-AddAutoSizing (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1770 of 1795 tests passed)

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


@rzarzynski why only 4 digits padding for the instance_id and unique_num?


Hi,

I believe its not, because when in() < 5 then int[5] is invalid. So from
that perspective the
clause would be:

if (in() < 5 || in() < (8 + (in[5] & 0x0F))

But then in() < 8 is more efficient because the second part is evaluated
only
when the telegram has a chance to be complete.

/// Jürgen

On 08/05/2015 05:45 PM, Matthias Urlichs wrote:
>
>   * if (in() < 8 || in() < (8 + (in[5] & 0x0F))) // Telegram complete
>
> The first clause of this if() is redundant.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jsauermann/knxd/commit/dcbbbd334342ae6279569260a2e68c4034a4ad05#commitcomment-12544862>.
>


Oh nice! Yeah, the changes didn't seem to make a difference.

—
Sent from Mailbox

On Wed, Aug 5, 2015 at 10:26 AM, Carlos Correa <notifications@github.com>
wrote:

> @oliversong, since we're using node 0.12.x, it seems like these features might be baked into node? see the [release notes](http://blog.nodejs.org/2015/02/06/node-v0-12-0-stable/):
>> - HTTP
>>   - maxSockets are no longer limited to 5. The default is now set to Infinity with the developer and the operating system given control over how many simultaneous connections an application can keep open to a given host.
>>   - Proper KeepAlive support means that sockets will stay open until they timeout at the configured time, are closed by the remote side, or the process exits. Developer's no longer have to make sure requests have been pipelined to keep the socket open, or use an alternative module to get that support.
> although it seems [there may be some issues configuring keepAlive](https://github.com/joyent/node/issues/9075), we should consider preferring an instance of [`http.Agent`](http://nodejs.org/docs/v0.12.0/api/http.html#http_new_agent_options)
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/clara-labs/nylas-nodejs/commit/18be350eef728ed0fda0f75a8fd6fa6c8ee46fec#commitcomment-12546917


Here we aren't just checking the size. we are checking also the end date and uuid of states. So List is needed here.


[Deployment available online](https://s.brightspace.com/apps/hello-world-fra-iframe/dev/144ed0531a943ec0e818449f2959762454002c14/index.html)


Fixed! I knew we had something that did this! :)


Porque você colocou essa consulta no evento?


If all you wanted to do was add the kernel config, why are you removing network stack source?


You're right, I missed that Will merge.


Here, we can keep it as set instead of list. And yeah there will be only one state as per the data setup


Should line 30 of the README actually say:
`meteor run --settings config/settingsDev.json`
without the leading slash?


I would suggest to echo it before trying to generate it with dot (2 lines above). (so if any error shows up later you see about each svg was the problem).

If you want to echo it after generation is concluded, show after thumbnails are created.

The point it is now is in the middle of processing.


Кажется второе после || здесь надо завернуть в скобки


device repo looks good to go, if this is in fact done.


Actually, is adding a kernel config the only thing necessary to support v2?


You should probably only resolve this later on, like you do with the debugbar instance.


This is actually a duplicate, it is already created and registered in pkg/api/v1/deep_copy_generated.go. It looks like runtime.RawExtension doesn't get a deep copy though. I'm not sure what the best way to solve this is. Instead of having genunits (i.e. a whitelist), maybe we should blacklist packages instead. We will likely still have duplicates, but I think it would take care of most of them in kubernetes, I'm not sure how well it would work for openshift. Thoughts?

@smarterclayton @wojtek-t


terbonium - are you just looking at it and saying it is duplicate or are you checking the index statistics to see that it isn't being used?


Ao invés de sempre executar o seeds eu sempre executo rake db:reset (ele executa drop, migrate e seed) é por isso que você colocou as consultas?


These tests are excellent.


Yes please - I'll take a look this afternoon.


Should probably only wrap the command that would fail if VMD is not launchable.


upsertDoc method renamed to be private.


Thanks. Can you test this as I'm on my phone and haven't tested it

Sent from my iPhone

> On Aug 4, 2015, at 11:32 PM, Adaptified <notifications@github.com> wrote:
>
> Well done.
>
> —
> Reply to this email directly or view it on GitHub.
>


_upsertDoc moved upper to be closer to save method where it's used.


Error con la línea 8 de mrp_subcontracting/wizard/wizard_addition.py
class WizProductionProductLine(models.TransientModel):
+    _inherit = 'wiz.production.product.line'

Ver si pueden duplicarlo


NSURL would throw error way before that, but we can add double-check.


So all changes on git will automatically result in a new deb in the apt repository.


removeDoc renamed to be private.


It would be a good idea to create a module wrapper for vmd that would raise
an exception on import if vmd is not installed/cannot be found.

On Wed, Aug 5, 2015 at 1:25 PM Christoph Klein <notifications@github.com>
wrote:

> Should probably only wrap the command that would fail if VMD is not
> launchable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iModels/mbuild/commit/4e9a827d30bc27c74d3bd26e01efbeb253d55d33#commitcomment-12548057>
> .
>


_removeDoc moved to be closer to remove method.


I noticed I did that last night was going to fix tonight lol


Added a "if tableExists check before trying to create the Amazon Login table" this mainly addresses errors when the merchants try to reinstall the plugin on their dev environment but the login table still exist for some reason.


The comment was referring to the logging output. It misses action before the actual action.
Currently: Calling match with an invalid 'SET_ZAMMIT_SANDWICH'
What I asked: Calling match with an invalid action 'SET_ZAMMIT_SANDWICH'

First case, we may understand that the SET_ZAMMIT_SANDWICH value is not valid... not that SET_ZAMMIT_SANDWICH is not a valid action...


Why, just for performance? Or other implications?


@shuvalov-anton, предлагаю лазить за пруфами в [web-standards-ru/dictionary](https://github.com/web-standards-ru/dictionary), а если что-то не так или не хватает — участвовать ;)


Why? You shouldn't need to modify anything in /build that is not in the repository, I think?


That would be lovely to show the native toast. I'll look into, maybe there is a way :)


Oh did not think about that, fixed.


setAutoBind moved from $meteorCollection factory definition to be a AngularMeteorCollection's method.


@zhouyuan will has_existing_subuser() check for the existence of the specific subuser, or just any subuser?


How are you integrating this forked version into your project? I am relatively new to the ember eco-system but want to use eslint with ember 1.13.6 and the npm version throwing Illegal import declaration errors which I believe is addressed by this version bump. Thanks for any help!


Double copying removed here.


@kirkshoop  [blogged](http://ledentsov.de/2015/08/05/deterministic-testing-concurrency-rxcpp/)


Fixed.


oops, I meant, donation groups for this commit msg ..


Yeh, and consistency. Seems silly to pass through an application instance otherwise.


Calls setAutoBind method if arg autoBind is true.


Permissive, Woo! :bomb:


> How about "you can enable it in MPC-HC's Options"?

Are you sure about the capital "O"?


We delete here to make sure we are altering a clone not passed document.


337? i se there 377 -
EMOTE_ONESHOT_MOUNTSPECIAL          = 377,
;)


Async test is not really needed here.


Good points!!!


@Aran85 I don't understand the m_stack.empty() check. Does that mean that it will only add eol if it's the last section to be closed? Doesn't sound right to me.


@ngoldman dividing in half so you can set a single global value in the config


So the period is now 6.5ms? The comment above should be corrected then.


@Urigo please see comments in the diff base.


Thanks @mattdowle this is actually as issue with devtools::install_github - see hadley/devtools#383 and hadley/devtools#394. From what I understand a training blank line doesn't cause a problem but a blank line between fields does. What packrat does is append some lines to the description file so if there is a trailing blank line in the description this will become a trailing blank line. In any case, I've opened an issue with rstudio/packrat#237


it puts static libraries in lib and dll's in bin, `-print-search-dirs` just isn't quite nuanced enough in its output to reflect that


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


@joseph-ross-work Who passes an object with `lftoken` key? I don't see it anywhere else in this codebase. This commit broke other auth flows where `credentials` is an object, but not like the one you're expecting here.


@dwj192 missing a space after the curly bracket


Porque esse contador é um atributo da classe? Dá uma olhada na sua classe e tenta ver se não tem como eliminar esse contador. Outro detalhe é o nome desse atributo: ele conta o que?


missing a space before the round bracket and before the curly bracket


@dwj192 overall ok, see my comments. Also, there's a conflict, so it needs to be rebased on top of master so that I could merge it.


Cute pattern.


Did you want to commit this private key? :wink:


Joseph say: https://github.com/Livefyre/designer/blob/cca6b843cbdba45b657b10c5df218ab1a4bdc622/src/templates/outside-embed.lodash#L31


Generally, I think using daemon threads for this type of work is a best practice.  User threads are fine when you want the application to wait for the threads to finish before exiting.  In this case, I see no reason why a scan should prevent my app from exiting — if there was cleanup or some other action that had to happen to exit gracefully then user threads are appropriate, but in this instance I shouldn't have to wait for the parallel scanner to finish before my application exits.


Perhaps worth a pull request back to the original repo? @drjimmie1976 / @AshleyRudland to confirm if it's in line with company guidelines.


Heh, it looks like this was already fixed thanks to https://github.com/hadley/ggplot2/commit/f46b206e3aa962302ad3bfb525f4ad108e8a1e09 ?


Double-export doesn't work so well here. Duplicating just the `libraries` line with the extra `-e` at the end works for the stdlibs, but the llvm patches aren't applying correctly in cygwin.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1269 of 1269 tests passed)

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


This one should go into the collection solution pack.


In https://livefyre.atlassian.net/browse/LF-18710, `credentials` is like:

```json
{
  "bpChannel": "https://backplane1.janrainbackplane.com/v1.2/bus/blahblah,
  "network": "coxnews-0.fyre.co"
}
```

But this line rewrites the object to be... worse, which causes the downstream failure here https://github.com/Livefyre/livefyre-auth/blob/967f8e62359d500067c090b42c9cfbe09c3bef73/src/auth-api.js#L42


@alfhen skal du bygge noget spændende?


Alternatively, we could just wipe away all deep copies and conversions, and, assuming import dependencies are traversed in the correct order, we could simply check if a conversion has already been generated (by just adding a HasGeneratedConversionFunc method to conversion.Scheme), and if not, generate one. If you traverse the import dependencies in the wrong order, I think you'll just have duplicates, not any bad behavior.

This would require fixing hand-written conversions to use Scheme.Convert instead of calling the convert_x_y_... functions directly, but I think that's a win anyway so we can avoid having circular dependencies (broke conversions, can't generate new ones because hand-written ones depend on autogenerated ones so there's an import failure). If we do this, we could also just use build tags to disable building autogenerated files instead of explicitly removing them.


will coverity complain about filename even if you're checking for asprintf() return?


No problem!


@garrett-schlesinger I don't believe that status will be completed necessarily.  See: https://www.twilio.com/docs/api/rest/call#call-status-values

Calls are not made synchronous to the request. In fact the api supports modifying live calls and looking at the status of an ongoing call. The only synchronous part of Twilio are the url (twimlets) and status callback which we aren't using in this case.


Porque você escolheu salvar os livros nesse momento? Aqui ainda não existe possibilidade nenhuma de que a sua Activity seja destruída. Lembre-se da existência do método onSaveInstanceState. Qual era a função dele?


@dwj192 also can you prefix the commit message with 'rgw:', and sign off the commit?


Novamente não é necessário salvar o conteúdo da lista nesse momento.


Is this being removed because it was no longer being used?


Consider noting the in commit message why this is being changed.


O nome desse método tá estranho. O que é uma lista de retorno?


Nice!!


What I meant is that the rest call to create the call resource is made synchronously. And yes, the call will be queued initially.


I didn't see it used in the shell adaptor either, removed it there too, and was still able to do bulk with it.
But now I "see" that it is used indeed ... don't really understand how and what though ...


This wasn't really questioning your methods or anything, I'm really just trying to ask whether or not you are a DBA in real life.  We could really use some help day to day with optimization.


Wonderful. Can you do the same for harfbuzz? @behdad


@mahendra if you are ok introducing sleeps the right thing to do is to poll twilio until the status is not in one of the terminal states: {queued, ringing, in-progress}


i don't see why it should care, but it also isn't necessary any more (it's a really old hack)


Por que o nome do método é getNomeLivro()? Você já está na classe Livro, não tem necessidade de colocar o sufixo Livro em todos os métodos.


Should we assert the result of the Task is true here? Then remove the ```System.Diagnostics.ProcessTests.ProcessTests.TestEnableRaiseEvents``` test, since it's testing the same thing, and this also fixes #2562?


It was already optional if `antialias` was `false`. The real problem is in GLSL 1.50 which is what shaders version currently is. OpenGL 2.1 requires GLSL 1.20 and that mean reverting 48098e32fe30e8b4bede50e339161c4757a60539 and fixing `color` attribute after that.


Future improvement, for people who have data in multiple regions, perhaps this could be part of selecting the metric rather than the datasource. For example, my EC2 instances are in us-west-2, but global (ie: billing) data appears in us-east-1, for this use case, I would need to add a "datasource" for each region.


could you please explain how to run this script? do you run bridge.js through phantomjs ? Thanks a lot


Aqui não é muito interessante tornar a sua própria classe o Listener para o NavigationView. E se você precisar reaproveitar o código desse listener em outra Activity?