John,

I see a lot of changes to the xml/py wrappers for Pindel here. What is the include/exclude file you have added support for? Also, does the optimization using -x still hold with this wrapper changes?

Wanted to check with you before I integrated your changes, let me know.

Thanks,
Abi


MIT license created in the root project level.


in kwtags. Also, remove it from it. Otherwise it will end up in the tags.

also, to follow convention rename tags to args and kwtags to kwargs.


This belongs in security - `pkg/security/admission/admission.go`


Turn this into a function, `break Label` is a code smell


@jpadilla that part of the code was updated to use the `tokenExpireName` https://github.com/jpadilla/ember-cli-simple-auth-token/pull/44/files#diff-b3b19e97b14851cfabb1e6558abfc6daR249

I just left msg asking what version is being used because if it's not version that included the change then it probably breaks..


Yes, that is the issue. No real solution suggested other than to rewrite any shader that has the issue...


All errors should always be lower case sentence fragments, not capitalized


Shouldn't this be returning?


You should remove final opt from opts. If a global opt is in the final opt, it make no sence most of the time to also have it registered at the start.


I think you should do: final_opt=True to enable it.


//cc @moollaza  forgot to branch my b


derp


Of course Martti ;)


@derekwaynecarr I view this pattern as a flaw in our admission controllers patterns that every one in the chain loads the same objects over and over again.  And that everyone needs their own reflectors and caches.  Can you open an issue to track ensuring that the OpenShift admission controllers actually perform and don't constantly refetch the same objects over and over again?


idem


Move these constants to `pkg/security`


Huh. Why did you do that? Wondering in which Xcode building is broken? I moved recently all (almost) code base to modern Objective-C.


Thanks!  I have a todo in the code to remove the need for it, and you just saved me a bunch of time. :)
I'll roll this through shortly.


This is actually broken. No longer highlights parameters.


@Loopyloux do you prefer gitter over IRC? If enough people prefer gitter I might consider merging this PR.


Check my earlier comment. I'd try implementing the delete on the repo and then injecting that into the controller.


I'd check the Spring Data documentation. You should be able to add a "deleteby" query on the repository itself: http://docs.spring.io/spring-data/jpa/docs/current/reference/html/ (search it for deleteby). I've never done it so it would be a good exercise!


Rathern than having the caller check a DeepTranslatorOpt property, perhaps PEModuleBuilder could expose a DeepTranslate method overridden in PEDeltaAssemblyBuilder.


... Except `"*n"`. Sorry.


Why do you understand it?


It's already out, check curse :)


Yes.  If something is expensive then it's good to check the version and not do the work.  But if you don't save much it's cleaner just to have the generated code do it.


CMake probably isn't at fault here.  It seems that the act of importing a module in msys python is what causes the conversion of path into unix format:
`c:\rust\foo.py:` ```print(__file__); import bar```
`c:\rust\bar.py:` ```print(__file__)```
```
C:\> python2.7 c:\rust\foo.py
c:\rust\foo.py
/c/rust/bar.py
```
I don't think there is a way to force windows paths in msys, but here's something that sort of works:
```
C:\> mount c:\rust /rust
mount: warning - /rust does not exist.
C:\> python2.7 c:\rust\foo.py
c:\rust\foo.py
/rust/bar.py
```
Apparently the `/rust` mount point wins over `/c/rust` when converting `c:\rust` to unix style.
`/rust/...` can be fed to most native windows programs, including cmake, without problems; they just interpret it as a current drive-relative path.

Thoughts?


:+1: Derp


What generates the data at this URL?


Could otherDeepTranslator be an argument to the MatchSymbols .ctor rather than having a separate SetComparer method?


Works to parse function names and allow parsing of parameters now.


You're right. I added that code after I tested :( Now [fixed](https://github.com/perusio/nginx_ensite/commit/6d042d46284196e901f031688ac4e8c257cf8524).


tudo ok


I realize I may be beating a long dead horse, but in what sense is this not a serious crasher?

Dumpdata is an important part of our system, and crashes when datasets exceed a certain size. I view this as a pretty serious bug, not a new feature.


Please note that there are additional fields in Messages, see http://docs.ably.io/client-lib-development-guide/features/#types


> ` npm install -g gulp && npm install -g bower && npm install && bower install`

`npm install -g gulp bower && npm install && bower install` would have been easier. No big deal though.


I am using `text` for now. Let me know if you want to use any other name.


text sounds good


You're right, these could be one-liners. I think the _really_ idiomatic way to do this in scala would be to just make board (or dacBoard) a publicly-accessible val and not even define a separate getter method. I have changes like that queued up, once all the classes that access this are themselves converted to scala.


cool :)


Context menu on different widgets is more of an issue - we are losing focus (therefore selection) if we open the context menu not on focused label.


What is this demoUrl attribute used for?


Hi, I'm working on a same project in C++ with Qt libraries do u want to have a team work? I'm trying to make a thread for each command passed, like a command to play a kinda of music. So the program won't crash. I'm also trying to make a web interface for what i have written so far besides gui. I think we can u can contribute to my project.

contact me at i.alavifazel@gmail.com,

Best regards,


@pmoulon Removal of manual correction of camera orientation when camera was inclined on men's back.


good catch; will remove. Are you looking at this in intellij? It seems to do a better job detecting unused imports than my eclipse setup... :)


`"*n"` now supported \o/


no, 0.18


I don't think you meant to keep this change.


I have no problem with the package name or the version within the name. I just don't like that the package version within xml is lower in the latest revision than in the one before. In this case I can live with it easily.
Hopefully we will get rid of openssl soon - I also believe we shouldn't handle it through TS.


don't forget to close the url tag on line 1589


the order of these should be reversed


I think this explanation is difficult to follow but I cannot think of a more concise way. There are a few things that can be improved though.

* Here I would be more concrete:
  * "with a variable number of nested loops" -> "with as many nested for loops as ploidy"
* You can show that better adding at least one more loop ($a[2]?) and fixing the indentation, see example bellow.
* Also I think you mean to either reverse the order of the for loop index variables or the println statement, in order to to match the common order AAAA -> AAAB -> AABB. The current code would result in AAAA -> BAAA -> BBAA ...

```
for $a[1] = (0 ... N)
   for $a[2] = (0 ... $a[1])
      ...
          for $a[p] = (0 ... $a[p-1])
              println $a[p] ... $a[2] $a[1]
```


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2129 of 2213 tests passed)

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


Ahhhh right, makes sense. Can you open a PR? If not I can do it.


can't wait


Unless the failure is interesting, we can delete this branch if favor of #40.


emberaddons.com and emberobserver.com (new addition)


Also, is this a real move? Because I'm pretty new to this stuff and actually didn't know, if swap copies other in its argument.


@NOX73 A new version v1.20 appeared so you might have to fix there as well :-).


\o/


Why use an abstract class and not an interface? There's no added functionality in the class (and even if there were I would argue for using composition), so I don't see a reason. An interface would be a perfect fit here imo.


ou voce trocou a ordem dos endereços, ou voce nao deixou explicito que era ordenar do maior para o menor.


Well I believe the issue with `rake test_app` is that we have migrations seeding data which we really shouldn't be doing.  Ideally at some point the migrations would be cleaned up so that seeding data is properly done in the seeds, and migrations only handle schema migrations.  We're so far down the road of it being mixed together I'm not sure if or when that would be fixed at all, but it certainly would be nice to consolidate and cleanup the migrations again like we did with Spree 1.2 https://github.com/spree/spree/blob/master/core/db/migrate/20120831092320_spree_one_two.rb


Duh... thanks. Fixed.


fixed `mask_time` to `mask_ltime` in f16f816d


No, there is also the common.php file to be updated.


this commit :+1:


I tried to find on emberobserver but didn't see anything.


Yes I will.. was just testing here.. dont merge this yet of course.. This needs more work
Basically, when we send a custom header, a preflight OPTIONS request is sent to the server, and I'm having trouble getting that to respond correctly. Will look in to it later


I think maybe this should be broken up into 3 separate tests, though I don't really foresee a scenario where one would fail without the others failing.


achei vago, deveria ter colocado dois exemplos, como no cenario de controlador


Think this might be a mistake...


@wch Curious if you've experienced any problems due to this change?  Getting problems given this change in R 3.2 with duncan's xmlschema package https://github.com/sckott/XMLSchema/issues/1


It is scripts/common/main.sh. I do not think powerline will depend on anything else from there in foreseeable future, but who knows. Problem with github URL is that it is unversioned and file on the other side is sometimes updated, so you cannot use standard source fetching. And direct fetching in src_unpack is not the good idea as I think: this would be rather unexpected (BTW, why git-r3 does fetch on unpack stage and not on fetch which seems more logical (while cloning should still be done on unpack)?).

Why do you think that approach is terribad? bot-ci repository will in any case live in $DISTDIR, I also succeeded to find another example of this approach in main portage tree (though it is only a single one: dragonegg-9999, also in src_unpack and also only with FEATURES=test). Uses more sane variant

    EGIT_REPO_URI="http://llvm.org/git/llvm.git" EGIT_CHECKOUT_DIR="${WORKDIR}/llvm" git-r3_src_unpack

.

I can say why you *should not* do the way you do now: if I happen to update bot-ci repository so that modifications to powerline are needed with separate bot-ci ebuild tests will fail unless user used `emerge powerline` and not `emerge @live-rebuild` (most of time I do the former because I have xonotic live ebuilds which fetches maps and that is rather slow; latter is being done mostly in form `emerge -NuDva @world @live-rebuild @preserved-rebuild`).


Looks good!


Nice!


Take a look at this:

https://github.com/LlamaOpenSourceProject/msm8974_caf_G2/commit/02a6b644703a272ed93ced42360322fc4221333e


@amyxzhang Typo detected, it should be “Dislike,“ not “Dislke.”


Line 5 typo rehearsal
Line 6 Head --> head
line 12 typo "within"
Other than that when we are finished we can add more questions and answers.


Can we assume that it is always gaf version 4.0?  Should probably be a parameter.


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I have tested in april changing the depth limit to 2 and 4, but both failed.
http://tests.stockfishchess.org/tests/view/552d04510ebc596367ee9907
http://tests.stockfishchess.org/tests/view/552d5adf0ebc596367ee9913

But you can do your test as simplification and possibly then it can pass.


Oi.


@JDutil AFAIK the seed is actually NOT a migration, its from that call side: https://github.com/spree/spree/blob/d5c4ed35e0a65d98a8243dd5b641ee6f2c2e4073/core/db/default/spree/stores.rb

Since the `rake test_app` does not invoke migrations to create a schema, AFAIK it reflects from AR models. But I could be mistaken here.


Cillian de Róiste wrote:
> Unless the failure is interesting, we can delete this branch if favor
> of #40 <https://github.com/plone/plone.app.robotframework/pull/40>.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/plone/plone.app.robotframework/commit/48bd0b7f1f1ee34c6e8f77e3181bd503b20d44d3#commitcomment-11481443>.
>

The failure is not related to this branch, so branch can be deleted.


Team names can only have up to 16 characters, I think. I'm pretty sure it threw errors when I tried otherwise


The only way to not have to specify all the import is to use dynamic lazy loading using require() with a var (which will break static build with browserify/webpack which will make us unable to make a playground - and people love playground ^^)


The real test running in OpenResty with LuaJIT seems to be working: https://travis-ci.org/Mashape/kong#L1489

`stub_coverage_spec.lua` is failing because it's probably using Lua instead of LuaJIT (by default OpenResty uses LuaJIT): https://travis-ci.org/Mashape/kong#L2095

I guess that can be fixed by compiling Luarocks with LuaJIT support.


This is repeated from main method.  Move main code into function and call from both places.


I think we should drop all the "is DraconianTheme" and "as DraconianTheme" special cases. These should be a part of the interface, in there the Draconian theme will have the code and the Vanilla will do nothing.
For example, this piece of code could be part of a "ProvidePropertiesComboBox" method, or something similar.
It's very reasonable to assume that if Draconian needs to do special things here, other themes will also need it, we shouldn't have special cases, it should be wrapped in the interface.


OK, thanks @Galixte ;)


ich denke, das war das https://github.com/openatv/enigma2/commit/4f254fcfb7f3b54bd7cb617eb1cf65fa4603741b
bin mir aber nicht sicher, ob es noch wo verwendet wird, deshalb hab ich es drin gelassen.


rax retry tests


Why are we adding an unused virtual `getJob`?


It's actually building from migrations not a schema w/db:test:prepare:
https://github.com/spree/spree/blob/d5c4ed35e0a65d98a8243dd5b641ee6f2c2e4073/core/lib/spree/testing_support/common_rake.rb#L19


Sorry, I don't think I can be of much help - I haven't encountered any problems, but then again, I haven't tested this stuff any more than other R users. Maybe ask on R-devel?


It has not been moved still - maybe some features revert is needed? https://elevforeningen.vih.dk/groups/jubil-ums-og-gensynsweekend-2015-0

@mglaman Maybe remove the field entirely instead of just setting access?


A simpler and more efficient way to spell `boost::optional <Job&>` is `Job*`, no?


>Is this a special case here?

No, of course.
Fixed.

P.S. Old "obscure" logic doesn't want to give up :)


Already did. I'm at the metro, so I can't send you the link.
Le 2 juin 2015 7:39 PM, "Eric Larson" <notifications@github.com> a écrit :

> Ahhhh right, makes sense. Can you open a PR? If not I can do it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mne-tools/mne-python/commit/1956dbb8f7ac3efe0248b6393f0ea06c5bd93ee4#commitcomment-11481440>
> .
>


Interesting. Thx for pointer. I'll double check my assumptions here. Nevertheless lots of spree development semantics depend on the seed. We should start adding the `before(:suite)` everywhere and than look into fixing the seeding. I assume you seed for manual verification?


The Qt repo shouldn't know about this environment variable. Maybe add a separate QT_BUILD_SWDEV env variable.


How about `boost_cart:detectorrail`?
It does not exist in PilzAdam's version, thus, we can register this as boost_cart node.


"PR" ready:
https://github.com/marttiphpbb/phpbb-ext-customcode/pull/29


Yeah that would be better.


yeah, we should probably do that...


I think it's a much better place to put them default chroot in `/var/manjaro-tools/buildiso`


Thanks, Just thought I'd ask just in case you had.


:shipit:


nao foi necessario


`mesecons_rules` is defined on top, why not use that one?


I build sandbox apps quite frequently for troubleshooting issues or testing new features.  The sample modules specs also run them.


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


I suppose the comment `//escape all backslashes` can be removed?


awesome - thanks so much!


The link is broken, should be `CONTRIBUTING.md` (the uppercased filename is [intentional](https://help.github.com/articles/setting-guidelines-for-repository-contributors/)).

~~on the wiki~~ -> in the docs


Hum indeed. I don't even know why I did an array of pointer of nodes.


Please use `minetest.log("action", msg)`


tenho pessimas lembranças com uso de acentos no intellij


Ah, nope :) That fixed it.


@koistya , why not extending the passed class `ComposedComponent` itself?


I know for openresty but by simply installing luajit we would have ffi everywhere, so why not. I can also ignore some files for the stub coverage but I don't think it's the solution here


Yes, you're right.
Fixed.


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


@zachwise @emilywithrow @richafur this is the final version of our story for the soft launch! We're having a few issues with the images that we'll ask you about in class.


@JDutil Thx for clarification. Expect some upstreamed commits that add `--order random` to your specs. We are close to having fixed all indeterminism in the specs that used hidden sequential properties.


pd: no lo subo directamente al resto del codigo para no hacer quilombo


ja foi corrigido, (inclusive antes deste comentario ter sido feito)


Some tool chains defined IP_MTU_DISCOVER, but do not define IP_PMTUDISC_PROBE.  As a result, this commit breaks the build for these tool chains.  The Android NDK is one example.


The motivation for this code is to give a warning for old config settings, so it needs a different change. Something like:
```
if self.get_config("drop_upload", "enabled", False, boolean=True):
    raise OldConfigOptionError("The [drop_upload] section must be renamed to [magic_folder].\n"
                      "See docs/frontends/magic-folder.rst for more information.")
```


This is inconsistent with all other packages, the change should be made in peel-analyzer/pom.xml.


Take a look here: https://medium.com/@dan_abramov/mixins-are-dead-long-live-higher-order-components-94a0d2f9e750


Sounds good @stubailo. That matches the current implementation on my local branch.


Not sure it was needed, but I took the same version of firebase and angularfire as in the doc.
I might do the same for the other scripts as well.


__API Blueprint syntax error on line 339__: base type 'CUBE' is not defined in the document (Autogenerated message from apiary.io)


Define the node`meta`a line above `local speed_mod = tonumber(...)` and check for the detector function there.


Thanks @Blubberboy333, I've fixed it now.


So good so far! Your diction is amazing sir. I definitely want to experiment with creating things as object literals. Some sticklers may say you have a syntax error on line 14, since you never close the function body, but I'll let that slide. :+1: :+1:


Yes, 90% of my comments are because intellij highlights something. ;-)

On Tue, Jun 2, 2015 at 10:24 AM, Matthew Neeley <notifications@github.com>
wrote:

> good catch; will remove. Are you looking at this in intellij? It seems to
> do a better job detecting unused imports than my eclipse setup... :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/martinisgroup/servers/commit/63364fcc6b15ac7a46c0f5712562cabd8da891c2#commitcomment-11481372>
> .
>


Radziłbym usunąć hasło z publicznych plików


Thats a really good question.

The behavior on web is that it doesn't take any space until the query resolves, then it displays and bumps the layout. We could do this on iOS by making the button have a height of 0, then resize as soon as the query resolves. The layout jump on native may be jarring though.


@gracjan surely, I've planned to write tests already! I also saw some interesting way to writing test something like "If I insert blah-blah-blah and run whatever then I should see blah-blah-blah". IIRC correctly it was called features. Have you seen it?


Can you briefly explain the use of `case` here? I understand what it's doing but I couldn't find the right words to google to find an explanation of this use of case.


@DonOregano yes that would be it. Sorry I cannot give you much more help, I have no experience with cmake.


Fix indentation of this line.


These are all great changes. However, I'd still like to go through a pull request for all changes in the future.


This script should work on Mac (and FreeBSD, etc) as well, so linking to the Linux instructions isn't ideal. Just a minor thing though, can be fixed in a follow-up PR ;)


Otherwise LGTM. Note that docs/frontends/drop-upload.rst also needs to be moved.


Why `return true`? We don't support all actions ever.


No it hasn't - this is on a branch (wanted to get initial work up)


Não sei como agradecer pelo arquivo de root vlw obrigado gente


So, I'm going to go with a very simple hidden/not hidden for now.


This looks reasonable. I can merge this.


@oblador Got it. I updated to the latest version of `react-native-vector-icons` and everything seems to work fine (except for some renaming stuff I had to sort out). I might close this repo end delete it, in which case this whole thread will go away. It was nice talking to you, I am sure our paths will cross again.

Cheers,

- Bernd


This is much better really. The only thing I'd add is an `except requests.exceptions.RequestException:` to make sure that all unexpected exceptions are caught as well (so basically any of the other exceptions that aren't a connection or timeout error).


## This is a markdown file


Cadê vc!!!


Hi Abi,
Yes, you can still use the same parameters I suggested.
The previous wrapper was not properly utilizing the breakdancer input, or exclude list.  Also, I found an option for an include list, which I did not think existed.  This will allow us to pass an interval list so that pindel only works on the specified exome regions.  You will need to pass the agilent_probes.bed file as the 'include' list.  I'm not sure if you already have this file?
John

     On Tuesday, June 2, 2015 10:04 AM, Abirami Prabhakaran <notifications@github.com> wrote:

 John, I see a lot of changes to the xml/py wrappers for Pindel here. What is the include/exclude file you have added support for? Also, does the optimization using -x still hold with this wrapper changes? Wanted to check with you before I integrated your changes, let me know.Thanks,
Abi—
Reply to this email directly or view it on GitHub.


>"muncipianity"


Do we really need 1.9.1? Can we switch up the main jquery so that everything uses 1.9.1? Is there a reasonable way to test all/most of the pages to ensure minimal breakage? In general, I'm not a big fan of having a site design with a bunch of conflicting versions of the same library.


@cruppstahl  @weltling  Wouldn't something like the following code work well?

```C
#if _MSC_VER
typedef unsigned int uint32_t;
typedef unsigned char uint8_t;
#else
#include <stdint.h>
#endif
```


We're working on adding it to emberobserver right now.


Good job sir. You should include the img tag for sure. Also, line 5, `the` is used in place of `that`. :+1:


somehow a new installation used `/var/lib/manjaro-tools/` on my end without doing anything ...


flags are uint32


I see that you bundled with this commit the move to changing flags on trustlines.

I would rather change "mAllowTrust.authorize" to "(mAllowTrust.flags & AUTHORIZED_FLAG)" here as it's what was tested before so that we're explicit.


one quick comment: DB doesn't like the terminology "marginal likelihood", he prefers to just call it "likelihood" (for what it's worth)


We probably want to make the same change to the SUMO LocalizingClient if it doesn't already have something similar. The theory here is that the LocalizingClient should have the same world view as the LocalizingMiddleware and Prefixer. When the two world views are different, then really silly things happen.


that is a weird configuration, I think :-)

should we comment what that really mean?


Why do we need a third size? Wasn't there already some kind of small and large (I think I'd previously called it zoomed)?

Can the code version be applied on top of the non-code version (ie., inherits) such that edits don't need to be done to both?


This feels like something to do in a cleanup commit (probably in a separate pull request at this point) since there are a fair number of `g_new` calls pre-existing in the metrics codebase.


Why do you need onClick for this?


TODO language by user, if system language mismatches?
what about custom skins (light / dark)
example https://github.com/alexpana/qt-creator-wombat-theme


this should go in maintenance


check should be
```
if (mAllowTrust.flags & ~AUTHORIZED_FLAGS)
```


That file is in the Documentation folder, the link is broken.


Btw. just noticed this is actually linking to itself now?


Added Files


Sorry, I didn't know that the message could happen after the redirection.  I've learned something new today!


I'm not a fan of using subprojects in this case. Let's merge in a reasonable version once and replace it as needed. Point to the repo commit/tag in the commit comments. Also, ACE is already there.


Awesome update! Thanks for your work on this! :D


Probably don't want to commit this pdb stuff, correct?


Right, I was going to try displaying the json code inline, but didn't have time to finish. Commit that my by mistake -- new code should be up! Sorry


Indeed I do.  My main reason for calling it that is because it is the likelihood (and there is only one). :smile:


It's not grammar nazi if it's a spelling mistake... :unamused:


That postinstall is problematic for us, too. Please fix this.


actually /var/lib/manjaro-tools/... is where I store all the manjara-tools chroots, isos, etc, forgot the lib...


check should be
```
if(*mSetOptions.setFlags &
~(AUTH_REQUIRED_FLAG |
AUTH_REVOCABLE_FLAG))
{
```


+1 This should go in upstream


Hey @peterneubauer, [this looks really cool](http://blog.mapillary.com/update/2015/06/02/id-signs.html). My time to devote to iD is pretty limited at the moment, so I don't think I can debug this for you. The most active iD developer right now is @bhousel. If this code is in a good state for review, please open a pull request and he can take a look at it.


if(*mSetOptions.clearFlags &
~(AUTH_REQUIRED_FLAG |
AUTH_REVOCABLE_FLAG))
{


Got a gif of this in action? (Add to Readme...)


Ja, das meinte ich.
Müßte man wohl mal im Quellcode suchen und ggf. aufdröseln.

Args, mir fällt grad eine grauenhafte Formulierung auf ...


semicolon


Hmm, curious. What behavior are you seeing that necessitates this fix? Would you consider a pull request with some more details?


This doesn't really seem to map anything to anything except for static values... what is the point of this?


This should be `const`, and that will require `JobQueueImp::m_mutex` to be made `mutable`, which is fine.  We should not be doing non-const operations under an `assert`:
`assert (getApp().getJobQueue().isRunningOnJobQueue());`


Код неважный...

Переделать на модуль.
И начать с малого.
Вначале сделать просто кнопу вверх, как мы и делали на занятии.
Позже добавить события
- сколл
- клик
- ресайз

Позже добавить функционал кнопки вниз


Now that I think of it, I'd rather not make this change now.
If we're going with this change, we should follow the set/clear flag semantics that we use for the account flags to make it less error prone


Interesting, we use similar schema, but not under the name 'features', just
plain ERT.


гуд


GJ


short answer: the `case` makes this a partial function literal.

scala has a match statement, which is like switch on steroids. The cases inside the match can contain patterns that can decompose more complicated structures. For example:
```scala
val foo = (1.0, 2.0)
foo match {
  case (i, q) => i + q // this case will always match
}
```
If you just take that bracketed block after the match by itself, you have a partial function literal:
```scala
val pf: PartialFunction[(Double, Double), Double] = {
  case (i, q) => i + q
}
pf(foo)  // = 3.0
```
Because `PartialFunction` extends `Function1`, we can pass a partial function as an argument to `map` just like a regular function, and get the pattern matching to destructure the tuple for free. If we wanted to write this with a regular function literal instead, then we'd have to refer to the tuple members by name, which is less intuitive:
```scala
(Is zip Qs).map { iq =>
  Math.atan2(iq._1 + this.offsetQ, iq._2 + this.offsetI)
}
```
Note that if we left off the `case` but kept the parens, then scala would interpret it as a `Function2`, that is a function that takes 2 parameters, instead of a `Function1` that takes a tuple:
```scala
(Is zip Qs).map { (i, q) => ... } // error! map does not accept Function2
```


@levicole DWMR. Not much to say about this checkpoint... Press on!


`asScala`... well I'm certainly not ever going to mispronounce _that_...


It will be cool to have something like this `" 🌙"` (crescent moon unicode) as lighter.


It's really nice theme. First we need to ask how this there is licensed, because there is no any information about it on alexpana page. Could you ask him about it?
In general it should be not so difficult to implement.


Yes, it did feel a bit naughty to be putting it in a file intended for sprockets as that's not my use case. However it doesn't look like you've added `octicons.scss` to this commit?


```java
// when management complains about speed, remove a few zeros
Thread.sleep(1000000000);
```


Well, this one for bootstrap, when I used jquery.js it through an error as bootstrap requires jquery 1.9.1 or higher.


Remove it


@gracjan here it is https://youtu.be/5axK-VUKJnk?t=6m6s


Inherit bonecard-code from bonecard-zoomed. Rename to bonecard-large if you think it is a lot clearer than 'zoomed'.

I should be careful using the term 'inherit'. What I actually mean is to label code cards something like:

Code here

And get the base size/outline from bonecard-zoomed but overwrite whatever values needed in the definition of bonecard-code.

You probably already know this, but precedence is somewhat reasonably covered in https://css-tricks.com/specifics-on-css-specificity/.

Also, be careful that #editor isn't defined elsewhere. You might want a less generic name.


I completely agree @dmbates.  I was certainly being too quick and dirty.

However, I still like the idea of a two-by-two table for explaining the various kinds of deviance that one might want to get.  Does this look better?

| relative to saturated model | based on density for y given u     | based on density for y          |
| -------------------------- | -------------------------- | -------------------------- |
| yes                           | `deviance(object)`   | not currently available in `lme4`                        |
| no                            | `object@resp$aic()` | `-2 * logLik(object)` |


@perploug  Please please please do not use singleton access when it is not required. You already have access to the ApplicationContext instance - this now means that you cannot unit test this nicely. Just change to use the applicationContext instance passed in to the ContentTypeModelMapper like all other app ctx usages in the model mappers


Ah. So because we're guaranteed that `(i, q)` will match we're basically just using it for the destructuring? I assume that the compiler knows it will match because of the types of everything.


Sweet, BTW if you put "closes #issue_number" in your commit message it automatically closes the issue. Mega satisfying.


Re issue #72


@lemire yeah, as for me just with a small correction

`#if defined(_MSC_VER) && _MSC_VER < 1600`

1600 is VS2010, so this way the custom typedefs are only done when stdint.h is unusable.

Thanks.


Kaushal Modi <notifications@github.com> writes:

> It will be cool to have something like this " 🌙" (crescent moon
> unicode) as lighter.

Good idea.  Though this looks really bad, if the font doesn't have a
glyph? for it.


I think this should be `&&`?


Hallo, habe mal mit
#sudo xstart
den Desktop gestartet und dann im LxTerminal
#./run.sh
probiert. Passiert leider auch nichts. Der rappelt aber im Hintergrund,
das System ist tierisch langsam. Zugriff per FTP dauert >10s

Hier mal das LOG von gestern:
http://pastebin.com/SEnjnzMR
und hier das von gerade:
http://pastebin.com/BZAx9yAX

Das 2te Log ist realitv kurz, der schreibt anscheinend noch.......


I believe @TheDash is referring to the fact that the Hydro build farm will be turned off now that the Jade release is complete.

I say, all the more reason to push to get this fixed. @aclodic this is more likely to move quickly if you can submit a pull request that fixes it.


Ok, can I use this version (with a reasonable merge) instead of the existing one coz I think it is outdated.


Why is the command repeating twice?? Does that somehow solve the problem of it not running??
It looks like it should throw an error.


The `clone()` is actually a slight change in behavior, right? Though it shouldn't make a difference because the data are only passed to the deconvolution server, not modified. I suppose this would make an extra copy of the data, not that it really matters.


Shouldn't it allow `https` as well? :neckbeard:


This reminds me there are a couple of ampersand issues floating around in the HTML. I'll want to remember to run everything though the w3c validator.

You seem to be adding the script stuff. I thought it was there somewhere, but I don't see it elsewhere in this commit where you'd be moving it from. Shouldn't the script stuff be in the header before the body?!?

Is there somewhere you are planning on reusing the footer?


Just gave it a shot, its as simple as:

QFile File("/home/sebastian/mmm/mandelbulber2/qt/skins/dark.css");
      File.open(QFile::ReadOnly);
      QString StyleSheet = QLatin1String(File.readAll());
     gApplication->setStyleSheet(StyleSheet);

but the theme has some issues (see red marked).
Maybe we are better off doing it programatically. But with stylesheet
files users can write their own styles...

Am 02.06.2015 um 20:08 schrieb Krzysztof Marczak:
>
> It's really nice theme. First we need to ask how this there is
> licensed, because there is no any information about it on alexpana
> page. Could you ask him about it?
> In general it should be not so difficult to implement.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/buddhi1980/mandelbulber2/commit/b3fce91e3373e5ccff42af9f0b272b63bf9f504e#commitcomment-11482417>.
>


I wouldn't mind setting the package to private. Dropping the version and name fields also seem fine. Related discussion in case we go down this route #102


Vlw


We don't use tab. Please change them to spaces.


I think that you'll also need to tweak `autogen_config.py` to construct the path relative to its own location, as I did in jupyter/qtconsole#15.


Confirmed, the horse is dead :-)
Seriously, your only options currently are either patching your Django instance or upgrade to a newer release.


unused: experiment, dacfiberid, dcrackfiberid, fastbias


@phgeller FYI this does the same, but loads in the email from the _config.yml file. Prevents typos and allows easy changes.


Yes. This works. Was scratching my head over this for past few days.


гуд.


There are tailing spaces in the following 22 lines. Please remove them.


sounds good. See you soon.


This will raise a 500 which isn't ideal. I think we should do away with auth_error and respond with a generic not Authorized response.


I see that earlier you have a TODO for type parameters occurring in upper bounds, which is the source of the ambiguity I mentioned, and the cause of failure in those situations. So besides that, this seems fine.


Vlw man


Odd indentation is... odd.  Like the method and placement though.


Tailing spaces. Please remove this line.


Looks like you are using different IDE.


Does this mean we can Sell & Buy Spawners on signs again?


La clase Edificio deberia ser abstracta. El modificar eso va a hacer fallar dos test. Debemos implementar algun tipo de edficio para realizar testeos de eso. Por ahora esto queda asi.


Yeah something is wrong with this code.


I have this in my init that takes care of that:

- [Check if unicode font is available (Symbola in my case)](https://github.com/kaushalmodi/.emacs.d/blob/ab7f55f91a4c57075d3440f1af53540771f45a7c/setup-files/setup-symbola.el#L8-L12)

- [Set backup lighter if symbola not available](https://github.com/kaushalmodi/.emacs.d/blob/ab7f55f91a4c57075d3440f1af53540771f45a7c/setup-files/setup-shackle.el#L25-L27)


This looks like all RedHat-specific stuff, so I'd prefer to separate it out from this PR. The policy install for file_contexts.subs_dist doesn't obey build options right now, so we can't put in an ifdef distro_redhat block either.


Misread directive to place entries.cv file in the project root and originally placed it in them models folder.
Added a second commit to the checkpoint-23-csv branch correcting this mistake and references in address_book_spec.rb.


Thx

Pozdrawiam
Kevin Ćwiek

Dnia 2 cze 2015 o godz. 19:52 Patryk Wychowaniec <notifications@github.com> napisał(a):

> Radziłbym usunąć hasło z publicznych plików
>
> —
> Reply to this email directly or view it on GitHub.
>


Yes apparently \o/


Vlw


Also the < > make it a link


LOL.

Note, this is just temporary until we convert everything to use scala collections instead of java collections. Until then, I'm importing `scala.collection.JavaConverters._` which effectively adds these 'extension methods' onto java collections to convert to scala equivalents. This works with so-called implicit wrapper classes, which allow you to add extra methods onto existing classes. There's another way to do this, by instead importing `scala.collection.JavaConversions._` which brings in implicit conversions between scala and java collections; in that case the `.asScala` would not be needed here at all but rather the compiler would just insert the needed conversion automatically. However, implicit conversions are generally a bad idea. Implicit wrappers which add new methods are pretty magical themselves, but at least they must be explicitly invoked so you have some hope of knowing what's going on.


J


I see that you bundled with this commit the move to changing flags on trustlines.

I would rather change "mAllowTrust.authorize" to "(mAllowTrust.flags & AUTHORIZED_FLAG)" here as it's what was tested before so that we're explicit.


Oh, such an adorable line number 51!


Now that I think of it, I'd rather not make this change now.
If we're going with this change, we should follow the set/clear flag semantics that we use for the account flags to make it less error prone


Would you move all this rearranging/style cleanup out of the PR?


There is never better feeling than committing of errors <3


mkdir -p Wiki-Data # will not delete any files already in Wiki-Data

The purpose of '|| exit 1' is to say, "Do not proceed if this fails."

Imagine mkdir -p fails.  Why might it fail?  It will fail if the user lacks permission to create a directory.  It will also fail if Wiki-Data already exists and is a regular file.  Under those conditions,

mv *wiki*.bz2 Wiki-Data

will not do what is expected.  If there are multiple files matching the pattern *wiki*.bz2, the mv will fail.  If there is a single file matching *wiki*.bz2, the mv will succeed, but will not move the file into the Wiki-Data directory.  Instead it will overwrite the Wiki-Data file with itself.  And so on.  Bugs.

In general, do not ignore failures unless you are very sure you do not care if the command fails.


I likw our own super simple no-special-syntax of the same:

```el
(ert-deftest haskell-indentation-check-17a ()
  "A type for a function"
  (haskell-indentation-check
   "fun :: Int"
   "    -> Int"
   "    ^"))
```

The above means that the only valid indentation point for line `-> Int` is 3rd column.

That is about visible stuff, invisible stuff is also super simple:

```el
(ert-deftest haskell-syntactic-test-quasiquoter-1 ()
  "Basic syntax for QuasiQuote"
  (check-properties
   '("v = [quoter| string |] Cons")
   '(("[" t nil)
     ("|" t font-lock-string-face)
     ("string" t font-lock-string-face)
     ("|" t font-lock-string-face)
     ("]" t nil)
     ("Cons" "w" haskell-constructor-face))))
```

The above means that all chars of `Cons` should have `w` (word) syntax and have `haskell-constructor-face`.

All of these is pretty easy to define, easy to debug, they explain what went wrong should something fail, and, as a bonus, no special syntax!


This section looks like almost all whitespace changes, which I'd like to keep out of the PR


lol:
'-A' was made to make sure that correct "id" from ACID will be set during commiting proces...
and I thought .. when u see new text with '-A' .. u'll change it...

And BTW...
what happened to text that suposed to be under that '-A'?
```code
DELETE FROM `creature_ai_texts` WHERE `entry` = '-A';
INSERT into `creature_ai_texts` (`entry`,`content_default`,`sound`,`type`,`language`,`comment`,`emote`) VALUES
('-A','%s appears very grateful to be free of the koi-koi spirit\'s influence.','0','2','0','21326','34');
```


Probably want files_mountpoint(sysctl_fs_t) instead.


unused


correct


I actually blew away this commit because I accidentally included code I was
playing around with and it won't build as is.

Kevin

On Tue, Jun 2, 2015 at 10:46 AM, Leymarie Valmon <notifications@github.com>
wrote:

> Hum indeed. I don't even know why I did an array of pointer of nodes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/klueska/cputopology/commit/c5f60ede893272c4bf4349ba17abd685c5367345#commitcomment-11481878>
> .
>

--
~Kevin


It looks to me like this should be `run mkdir -p "$directory"` to support dry runs and verbose runs.


what is this?


`git format-patch SHA1`

This will output all patches after SHA1.


Acceptable for immediate needs. Eventually this should prob throw an error, or find a way to report this scalar field to Progress


This is probably /proc/pid for a kernel thread, right? If so, it's kernel_search_state()


`SoupSession` automatically unrefs the SoupMessage once we're done with it: https://developer.gnome.org/libsoup/stable/SoupSession.html#soup-session-queue-message.


Why the __?


гуд


flags are uint32


Most of the added comments seem pretty unnecessary. The code is mostly self-explanatory.

Also, the comment style isn't PEP8 (single # per comment line)


This will be for small scripts only, I put in the footer coz want it to be loaded after the page complete loading, when I tried to put it in the head it through error for not finding the selected tag.


but we should follow the set/get pattern for this one


> I think that ultimately we'll want rules/index.js to use the ES6 module syntax like everywhere else.

Agreed.

>  in the future, adding rules one at a time, it will not be much of a pain

Agreed. I've pushed up the `.default` syntax for now, but I've created a `0.1.0` milestone issue to convert it to ES6 imports. I'll hopefully do that in the next couple of days if you're both happy with the approach (which is sounds like you are as a. it's consistent b. it will allow a playground).


Agora sim


Hey, these changes should have been made to rdlist.yaml. And I shouldn't have the rdlist.json file checked in, as it's built from YAML. I'll make the change.


Ok. I will try this with my workflow, and i don't have the agilent_probes.bed file. Can you email or put it in /mnt/lustre1/CompBio ?


Nonexistent parameters described here.


Sure.  Install gnu make (via cygwin is the easiest).  However, you probably just want to run the ant version of the command line, rather than the make version.  Make will download and install IDEA in a local directory, and then call the ant build.  Presumably, you already have the several versions of IDEA installed.


not a fan of adding things that we don't use.
Also the contract is not very clean: are those flags for the offer or for the createOffer operation?


Good stuff.
One last nitpick: that should say gli.texture_filter_anisotropic to keep with the gli.<some GL name> convention.


testing github comment plugin


IF you want, hit my up on hangouts:

zkennedy87@gmail.com


Is this conflicting with something in systemd? If so, then it needs to be put in the init_systemd else block, otherwise the other init systems might be broken.


Breaks encapsulation.


I just reverted it


Agrupa propiedades


testing better line number


Yes we should not use an xml file here but populate the items programatically.


clone and Arrays.copyOf do exactly the same thing in this case; both create a new copy of the data. You're right that this is not really necessary here; this was probably just defensive coding because arrays are mutable, but we could drop the copying.


unused


Markdown swallowed a part of `gli.<some GL name>` in my previous comment.


Why is this? Is it depreciated in Laravel 5.1?


@vruano - there is a relevant discussion at https://github.com/samtools/hts-specs/pull/83


Añade sólo los pesos de tipografía que vayas a necesitar. Cada uno incrementa considerablemente el tamaño de archivo a descargar.


> Why is this? Is it depreciated in Laravel 5.1?

No, just not documented yet. It'll be coming back, don't worry. :)


@bmorris3 - looking more closely, can you clarify why you aren't using the astropy machinery to do the conversion to AltAz?  Just taking the `get_sun` outputs and using the ra/dec from there is definitely *not* the way to do it (I think that's what you're doing here): those RA/Decs do not necessarily have all the effects that need to be accounted for (I'm not sure if that matters at this level, but it is possible).

Also, you might try using some ``scipy.optimize`` optimizers to get the zero-altitude crossing with that machinery.  It's not going to be terribly efficient because coordinates works much faster if used as arrays, but it's nominally a good test to see if your grids have converged.


El tag <header> es más semántico.


I'll submit a PR.


In your test it doesn't look good. I found in Issues section short discussion about license. There was no final answer.
I don't know how it's difficult, but we can try to create own StyleSheet. It could define look only for widgets which we use in Mandelbulber (it will reduce scope of work)


Por norma general: Evita estilos inline


Same here


Yes, the file is in /mnt/lustre1/CompBio/genomic_resources/ohsu_hg19_resource_bundle.

     On Tuesday, June 2, 2015 11:26 AM, Abirami Prabhakaran <notifications@github.com> wrote:

 Ok. I will try this with my workflow, and i don't have the agilent_probes.bed file. Can you email or put it in /mnt/lustre1/CompBio ? —
Reply to this email directly or view it on GitHub.


This can be replaced by @Getter of Lombok and the return type added to the field.


IMO, making power optimal freq less than input_boost_freq makes no sense... have you read wiki before tuning? https://github.com/mkaluza/project-zen/wiki/Dynamic-governor

> power_optimal_freq
Frequency above which power usage increase is significantly bigger than speed increase. A good hint might be the frequency OEM call powersaving mode (if they have such a thing). Use when enforcing frequency limits.


Everything seems to be working in this commit.

* Installed smack to local --prefix directory using build.sh
* Installed SMACKBench using setupSMACKBench.sh
* Set up environment vars by running `. smack.environment`
* Executed SMACKBench using installed runSMACKBench.sh
* Added virtual directory to apache2 to serve ${SMACKBENCHINSTALLDIR}/data

No other modifications were needed to make things work.

Benchmark results seem reasonable (that is, seems like things are working for smack.py's svcomp mode).

Tables generated all contain correct links to files related to each benchmark execution.


Why did you remove those ?


When will be new intreasting update? :dancer:


Oh, wow, somehow I misread the original text and thought it was just `return settlingAmplitudes`. Carry on then.


Fixes issue #2.


I.e. Something like, although maybe not exactly the same as this:
```
t=Time(time)
def altat(toffhour, loc, t):
    aa = AltAz(location=location, obstime=t+toffhour*u.hour)
    return get_sun(t).transform_to(aa).alt.value

toff0 = optimize.leastsq(lambda to:altat(to,location, t)**2)[0]
tnearest = toff0*u.hour + t
```


Ok, got it, but I need a time to learn this stuff (:


Ok got it.


Okay, done.


Las mayúsculas te limitan. Mejor transforma con 'uppercase'


No I mean if is possible to make a patch of a file edited locally not a patches of commits..
for example I have edited a file in frameworks/av and I wanna make patch using diff between the file in upstream and the local file..is this possible? If not how should I proceed?


Ok, thanks for the heads up, let's see how this one goes.


This looks fine (if it hasn't made it to master yet). ...i'm a _little_ worried about storing so many URIs in the code; it might be best if there were an attribute on the KnownUri model (in the table, I mean)... but I suppose for now we don't need one.


Un <form> ha de tener siempre un action


is java 8 mandatory ?


Should probably detect if the user attempts to use the `GC_*_LIST` variables and
direct them toward the new variables and meanings.


ok great! will download it. Also, i noticed there are two 1000g vcf files. Which one should i use for GATK? And what's the use for the hapmap_3.3.vcf file?


you probably should had left configure.ac in this branch, Makefile.am useless without it

not that this ever matters.


When I run I get the error `dustc: command not found`


We won't need my edits here to gnupg building for this patch, but I did have to delete USE_SHM_PROCESSING to get it to build completely.


Nuke your node_modules and install again. Something about how npm2 handles installing deps over the previous in this case goes ugly.


@SDkie This patch would pull the source from Debian (it may need to be updated to a different version at a later date, but it's currently 0.6.4) and includes the necessary headers to allow a full self-compiled app from source.


Yes :)


This could be solved by replacing `authenticate_with_http_basic` with `authenticate_or_request_with_http_basic` above.  The only consideration is that it removes the option of supplying a custom auth message.


Looking good so far.  A few more things to make it complete:

- Emit a warning if `MAKEFLOW_INPUTS/OUPUTS` are not defined.
- Double check that makeflow still does reasonable things if the variables are not defined.
- Set up a benchmark workflow and use `strace` to count file operations as you run, stop, recover, clean, etc, and make sure that the system is doing what you expect.
- Update manual and man page, keeping in mind that we are changing our default advice on how to write a makeflow.


24 vs 20 was a fail. Or well, it might take a very large number of games.  The test is +/-3 but they are too close.

   Program                          Elo    +   -   Games   Score   Av.Op.  Draws

  1 SF mbp 20                      : 3401    3   3 21024    50.3 %   3399   45.6 %
  2 SF mbp 24                      : 3399    3   3 21024    49.7 %   3401   45.6 %

Maybe my approach is wrong.  Maybe they should run against the master they are based on.  I don't know if 24 is better or not.  It was leading well into 10,000 games but it has moved the other direction.  You are probably right.
You probably just have to try some different values in the Framework.


Why are these tests removed ?


I think it may have just been the fact I was opening a diff of `fugitive.vim` which is going to be slow anyway because it's a huge file. For smaller files, this change has had no noticeable effect on the speed of opening or closing the Gdiff. My bad.


Recuerda que los inputs han de llevar una etiqueta <lavel> asociada por accesibilidad


und hier vom PI2
http://pastebin.com/QdZasekL


Add a new line at the end of the file (Git likes it).


No longer linking to our bug tracker on forums. Slowly moving to Issues tracker VIA github.


page not found


This seems strange. What happens in this dir?


> I have edited a file in frameworks/av and I wanna make patch using diff between the file in upstream and the local file

Yes, first commit your changes:

`git add edited.file`
`git commit -am "commit"`

Then you can do as I said before, e.g. git format-patch SHA1

or, if you want, do it following way:
`git diff HEAD~1 HEAD > 1.patch`

You also can don't commit any changes and just do following:
`git diff HEAD edited.file > 1.patch`
There are many ways to do so :)


I believe this will break older systems.


Can you clarify why this is a security/maintenance release? I know it's bad practice to insert a data string that's too long, but this sanity check will now return false in some cases, which has the possibility of breaking 3rd-party code. Shouldn't security/maintenance releases not change functionality like this?


A la hora de nombrar clases procura extraer patrones: Si tienes un contenedor de cajas puedes llamarlo: cajas-container y luego cada caja puede llamarse class="caja price", class="caja wetravel" etc... Teniendo "caja" las propiedades de layout comunes.


delete extra space


Les quedo exelente me apoyare para estudiar


Should be in corecommands.fc


> Doesn't go well with 3d borders of the seekbar, in my opinion.

I can agree on that. I will do another run on that. I wanted to do minor lifting for the time being, before we get more complex redesign.

> I'm really puzzled by the lack of thumb. I'm not so sure it should be completely off by default.

The idea is to have flat progress bar. You can change "tip" color to have sort of position indicator.

> Also I think you made it bigger than it used to be, I'm not sure it's useful.

This wasn't really intentional. (fixed)


I've never understood the obsession by some projects with making sure blank lines have no whitespace. The argument that the saved bytes are important is BS to me, at least in the case of this project.

That said, if you look closely at what your IDE tried to auto-remove, you'll notice that the indentation on every blank line matches the indentation of the lines around it. I'm pretty OCD about that, for a reason.

Personally, I do all my development work in editors that show whitespace characters -- e.g., I see little dots in place of all spaces. Whether I'm scripting in python or bash, the whitespace provides context clues as to the scope of a particular set of code.

That said, it makes no sense to me to remove whitespace from blank lines. (I can see there's some odd trailing whitespace at the end of line 3099, but that's another story.)


Is this a kernel module config? Is it systemd-specific? If so it should be something more obvious like systemd_kmod_conf_t. If not, it shouldn't be here.


solo que cual base de datos fue la definitiva? jaja


Not sure if Error is the best thing to raise?


What if there would be a way for emacs, via command line flag, presumabaly, to specify which directory to use as a temporary one? This way emacs will also be responsible for deleting them.

It's just that my original concert was lots of temporary directories that remain in /tmp after ghc-mod processes get terminated due to memory constraints or emacs issuing `kill-process`. When Haskell program is terminated in such a fashion, it's hard for it to clean up it's temp dirs. But if emacs would manage the temporary directory, then it could possibly reuse it when necessary, remove after issuing kill-process and generally prevent race conditions.


For this and the search interface, the "dirs" is redundant, so you can drop that. Search/list imply directories.


Why do we need that. Wouldn't it be better to edit all branches instead. Doing it just for [core] might create some issues ...


Estas 4 clases: .price, .wetravel, .gastronomic y .book repiten todas sus propiedades menos el color de fondo. Deberías extraer una clase comun "caja" y añadir un modificador que sólo aplique el color de fondo.


Contrasted here, the dirs is appropriate, since create doesn't imply any specific object class.


it is to initialize the repository!


@eteq Only because I didn't know that `AltAz` does that until just now. I'll get on that. :+1:


lnk_file -> symlinks


Si añades la tipografía al Body el resto de elementos la **heredan**, de forma que NO has de ir añadiendo font-family: open-sans a cada uno de ellos.


I just realized out of sync pac db
Will change it to set all repos


This memory_mapping addresses should probably come in through raspberrypi.h?


nice idea but no more loops thanks add it into the current building loop


Agrupa propiedades


Depending on the answer to my prior kmod_conf_t question, these probably will need to be revised.


Is it enough to only check that the keys are valid params and not check that their values are also valid? For instance, would it be a good idea to check that alm_method is either https, http, ssh?


Great, thanks!


> I'd like to propose that we make improvements to this implementation in order to support all types of binary.

Yes, for sure. It doesn't have to happen all at the same time, though.

> I don't think there's a reason for specifically using image reading methods?

There is. The `CameraRoll` API is so far the only API we ship with React Native that lets you get a URI handle to a native resource. Supporting those URIs as an XHR payload gets us one-step closer to actually being able to write a useful demo app that makes use of that `CameraRoll` API, not to mention writing real apps.

> Also wondering if it makes more sense to keep this outside of core.

Oh neat, didn't know about react-native-file-transfer!

So, we're actually going back to the drawing board on this one a bit. The goal is still to make this work via the XHR polyfill; and I'd like to support the webby `FormData` API to make `multipart/form-data` requests work -- this will allow us to run code written for the web in React Native. But the idea is to make it pluggable so that (a) we untangle dependencies better and (b) 3rd parties can provide their own ways of resolving URIs.


Por norma general siempre dan mejores resultados los anchos relativos ya que se adaptan a múltiples anchos de pantalla


A comment to remove itself? :)


I don't think we should be embedding a lock within the BSP code. Probably the framebuffer layer above should ensure mutual exclusion/synchronization if it is needed...


Nevermind I guess this is how it is done in the other framebuffers. I suppose they account for the mutex in the confdefs.h already then, so this should be safe to do.


@shama lmk if anything should be changed !


@magnumripper thanks! hopefully, I can use this example to write SIMD-enabled formats myself.


Please make the interface plural: udev_create_kobject_uevent_socket**s**


Because

c-stackcalc/00-static $ cat compile.sh
#!/usr/bin/env bash

cd "$1"

make main --silent || exit 2

On 02/06/15 03:26, Oleksandr Shturmov wrote:
> Why? ;-)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/onlineta/workbench/commit/cbb5348e9f23b7b2e54f860ce3277f110de0f4cb#commitcomment-11472665


put variable declarations at the start of the block scope/function.


Oh, now I see why we need to mark the server-uri paramater G_PARAM_CONSTRUCT. Otherwise we only end up setting priv->server_uri in the test case, not in production. Yikes!


avoid multiple blank lines in a row, see https://devel.rtems.org/wiki/Developer/Coding/Conventions


I am investigating running Luarocks with LuaJIT


Is this new element needs to be set on AuditRepository Unsecured WebServices?


Thank you, found out that I just need a simple git diff > patch.patch


in groovy it can be simplified to if(formatter)


Briefly, this is not as bad as it looks, but I'm not sure if I or most of chrono kernel users will use it. Most of them uses app written on java; I still prefer old but working shell scripts.


do you see a value in using the with{} rather than file.write? (since we do only one operation on the file)


Be careful about claiming copyright on code you didn't write. Reproduce the appropriate copyrights and acknowledge other licenses if they are used.


to me it would be ok to just log the exception, then no need for this operation variable


There is a configure flag it's what we use for the homebrew install

> On Jun 2, 2015, at 8:52 PM, Marco Palladino <notifications@github.com> wrote:
>
> I am investigating running Luarocks with LuaJIT
>
> —
> Reply to this email directly or view it on GitHub.
>


Was having errors caused by redirect-from frontmatter syntax error in notes.html, 0-byte file size for private-tutoring.md
![screenshot 2015-06-02 14 49 22](https://cloud.githubusercontent.com/assets/509564/7944211/5678bf40-0937-11e5-8893-c17541885c0f.png)


perhaps a gradle exception?
and then maybe we need to set the interrupt flag back ? (http://stackoverflow.com/questions/3976344/handling-interruptedexception-in-java)


is it ok to do the put concurrently?


I prefer something along the lines of:

    while True:
        time.sleep(1)


Adjusted test to account for add_entry inserting entries in alphabetical order by name.


Hi, sorry for not getting back to you sooner. Thanks for the PR
I added a few comments.


@geertschuring @gautric Hi guys, I created the changelog. Please keep it updated from now on :) .


hello i'm learning github


i am hungry


This is not a state, unlike all other events—it might make more sense as a public function.


we discussed throwing an error when grunt isn't passed on the options. I can't remember the conclusion, so consider this a placeholder.


Don't forget to close the url tag on line 1589.

Then, line 1590 doesn't work: it seems $shop_uri isn't right.


Thanks for the report, I’ll check it tomorrow.

--
Simon

Le 2 juin 2015 à 21:00:41, romaric17 (notifications@github.com) a écrit:

Don't forget to close the url tag on line 1589.

Then, line 1590 doesn't work: it seems $shop_uri isn't right.

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


Oh gosh, these signatures are too long :(


Yes, managing different translation is confusing...
I still have to ask for the others (if they read this comment) about which translation should I use...
:banana:


rax retry tests


How can NPM INSTALL be used without a working package.json file?


Thanks for the heads up, Xu.


back ported into branch-1.4


Thanks @arthurvr!


78bc8f4 better?


aaaaaa


Was your intention here to just remove the file or to totally stop supporting server configs?

I think we should do the latter, TBH. [The according line in the gulpfile](https://github.com/PolymerElements/polymer-starter-kit/blob/master/gulpfile.js#L88) would need to be removed then. We made the decision to remove server configs at some Yeoman generators too.

Happy to submit a PR if you agree.


Is there any other (may be "dirty" ) way to add support of SplitView somehow in a project what use Caliburn.Micro 2.0.2 today? I mean without downloading the full source and recompiling it manually just to get this small change with it?


Ah, actually `attemptDidFinish` needs to be called outside the if, otherwise the failure animation doesn't show.


I'm not sure if Error is the best thing to raise here?


This is throwing 500 errors when `ra` is a group:

  Error Rendering View[/permissions-manage-files.xhtml]
javax.el.PropertyNotFoundException: /permissions-manage-files.xhtml @94,92 value="#{ra.key.email}": The class 'edu.harvard.iq.dataverse.authorization.groups.impl.ipaddress.IpGroup' does not have the property 'email'.
	at com.sun.faces.facelets.el.TagValueExpression.getValue(TagValueExpression.java:111)


Or if you have this repository cloned, >git pull


@lancehalvorsen do you think we should expand on what digesting and compressing means here?


Can you please provide guidance for using npm install command indicated in readme file without having package.json file any longer? Thank you very much for your help.


I agree on the latter per your comment and would happily accept a PR to fix up that one-liner in the gulpfile.

I feel bad for not catching 'node_modules/apache-server-configs/dist/.htaccess' needing to be removed there. I'm going to blame jetlag :snowman:


This test always end up false.  I had to replace it with `eval { Net::SSLeay::X509_V_FLAG_TRUSTED_FIRST() }` to make this patch do useful things for me.


Could also make a constant with a list of the valid params like I did with frequency and connector_type...would that be better?


>.< you are correct. Hold please…


I did that and got the same issue. I wonder if you've got dustc in your node_modules/.bin directory from before removing dustjs-linkedin as a direct dependency? I had to manually install dustjs-linkedin before the command would finally run.


So what do you suggest to keep the API simple als it is now and to let the user choose where the event is handled?

I want the user to handle as less javascript as possible.
Mhhh


Rather than catching Exception you can use the fact that getSelectedIndex returns -1 when nothing is selected.

https://docs.oracle.com/javase/8/docs/api/javax/swing/JComboBox.html#getSelectedIndex--


There we go: https://github.com/PolymerElements/polymer-starter-kit/pull/108 :)


If we're going to use the system taskq, don't we need to grab the taskq ID here and then use ```taskq_wait_id()``` in order to keep from waiting for all the other tasks on the system taskq?


I renamed self.job_return_msgs to self.job_return_logs since it seems more fitting now that we arn't just appending the .msg part.


I don't know does it launch properly now?
On 2 Jun 2015 11:13, "timtrendlier" <notifications@github.com> wrote:

> Why is the command repeating twice?? Does that somehow solve the problem
> of it not running??
> It looks like it should throw an error.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Trendlier/restx_api_porting/commit/ab0e9e213a1ad033d4f7fb11c31f323acc7ba299#commitcomment-11482542>
> .
>


Hey @tomchristie I cant seem to find out in the documentation what to do here.  But while writing this out, I solved my problem, but decided to keep my full message for you to review.

Excellent work BTW on this project!  I might throw a pull request eventually where I will fix an annoying thing in the HTML output where the HTML forms lose information when you post and I have to copy and paste stuff again.

Have a great day!

DETAILS OF HOW I SOLVED MY OWN PROBLEM POSSIBLY NEEDING HELP FROM YOU:

I basically solved my problem by this where PaginationAPI is similar to what that guy in the stack overflow called "PaginatedUserSerializer".  Why do you not give a full example like this in the docs to support pagination?

```
    @handle_anonymous
    def list(self, request, format=None):
        queryset = applogic.list_account_users(request.user.account, request.user)
        serialized_page = PaginationAPI()
        serializer = UserSerializer(serialized_page.paginate_queryset(queryset, request) , many=True, context={'request': request})
        return serialized_page.get_paginated_response(serializer.data)
```

ORIGINAL PROBLEM I WROTE OUT ASKING FOR HELP:

I have a custom viewset where I have defined a list def:

```
class UserViewSet(viewsets.ViewSet):
    """
    This endpoint presents Account Users under your account.
    """
    queryset = AccountUser.objects.all()
    serializer_class = UserSerializer

    @handle_anonymous
    def list(self, request, format=None):
        queryset = applogic.list_account_users(request.user.account, request.user)
        serializer = UserSerializer(queryset, many=True, context={'request': request})
        return Response(serializer.data)
```

And it seems when you do that pagination doesnt work.  So in order to put it back, I found an old stack overflow where it paginates the resultset:

http://stackoverflow.com/questions/23819193/django-rest-framework-pagination-how-to-fix-name-pagination-is-not-defined

But the problem here is that the old Pagination class in this file prior to 3.1 you would pass some parameters to the PaginatedUserSerializer.

How might you rewrite what this dude did now in the new way of paginating in 3.1?

Are we lacking some documentation for how to set up custom pagination?  or am i just misconfigured for pagination.  I feel like I tried everything I could to configure and get default pagination to show up and this is the only way I can seem to get data to filter down is by this stack overflow.

I just have no clue why pagination is not working in my example and dont see any reason why it shouldnt be showing up.

FYI I am using a
```
class UserSerializer(serializers.HyperlinkedModelSerializer):
```

It is pretty confusing as to why we have examples on this page:

http://www.django-rest-framework.org/api-guide/viewsets/

Where we have empty functions for list def if you ever have a pass there django would throw:

```
Expected a `Response`, `HttpResponse` or `HttpStreamingResponse` to be returned from the view, but received a `<type 'NoneType'>`
```

So if a person is returning a Response with a resultset, how does one integrate pagination properly?  This is definitely confusing once someone defines a class inheriting from a viewsets.ViewSet if all the documentation we have is "/api-guide/viewsets/" to go off of.  You guys really should put an "advanced example" like this in your rest-framework-tutorial project.

If you can see I am using applogic to perform all my crud and I cant use all the magic of simple model management that django rest framework provides for my use case so I was thinking I needed to write it out like this to hook into things like I needed:

```
class UserViewSet(viewsets.ViewSet):
    """
    This endpoint presents Account Users under your account.
    """
    queryset = AccountUser.objects.all()
    serializer_class = UserSerializer

    @handle_anonymous
    def list(self, request, format=None):
        queryset = applogic.list_account_users(request.user.account, request.user)
        serialized_page = PaginationAPI()
        serializer = UserSerializer(serialized_page.paginate_queryset(queryset, request) , many=True, context={'request': request})
        return serialized_page.get_paginated_response(serializer.data)

    @handle_anonymous
    def retrieve(self, request, pk=None, format=None):
        queryset = applogic.list_account_users(request.user.account, request.user)
        user = get_object_or_404(queryset, pk=pk)
        serializer = UserSerializer(user, context={'request': request})
        return Response(serializer.data)

    @handle_anonymous
    def create(self, request, format=None):
        return self.update(request, pk=None, format=format, partial=False, new=True)

    @handle_anonymous
    def update(self, request, pk=None, format=None, partial=None, new=None):
        if new is None:
            queryset = applogic.list_account_users(request.user.account, request.user)
            get_object_or_404(queryset, pk=pk)

        serializer = UserSerializer(data=request.data, context={'request': request}, partial=partial)

        if serializer.is_valid():
            try:
                instance = serializer.save(pk=pk, request=request)
            except ValidationError as e:
                return Response(e.message_dict,
                                status=status.HTTP_400_BAD_REQUEST)
            response = {}
            if new:
                response['detail'] = OBJECT_DESC + ' created'
            else:
                response['detail'] = OBJECT_DESC + ' updated'
            response['url'] = reverse('accountuser-detail', kwargs={'pk': instance.id}, request=request)
            for k, v in enumerate(UserSerializer.Meta.fields):
                if v not in SKIP_FIELDS:
                    response[v] = getattr(instance, v)
            return Response(response)
        else:
            return Response(serializer.errors,
                            status=status.HTTP_400_BAD_REQUEST)

    @handle_anonymous
    def partial_update(self, request, pk=None, format=None):
        return self.update(request, pk=pk, format=format, partial=True)

    @handle_anonymous
    def destroy(self, request, pk=None, format=None):
        response = {}
        if pk is not None:
            try:
                UserSerializer.destroy(pk=pk, request=request)
            except ValidationError as e:
                return Response(e.message_dict,
                                status=status.HTTP_400_BAD_REQUEST)
            response['detail'] = OBJECT_DESC + ' deleted'
            return Response(response)
        else:
            return Response({'detail': 'No pk passed'},
                            status=status.HTTP_400_BAD_REQUEST)
```


Oh, doh.  So I did.


```int(seq_len/2)``` can be ```seq_learn // 2```


Fixed!


Cont'd... which presumably means we'd need to store the taskq ID in the ```arc_prune_t```.


Hi Simon,

just checked out this version, and noticing strange behaviour. *The version I was using previously is commit 63c828e042030e6382af3f073 from May 27.*
In summary:
  - all my subscreens are mapped (**squeezed**) into a small area in the top-right sector of the screen. (this
    behaviour is consistent in all the examples I tried from https://github.com/AlainLich/Romeo.jl/tree/XML ). They seem to function. At the same time, **mouse inputs** are sent to subscreens based
on the correct (not **squeezed**) mouse position. **This looks** like a subscreen location was stored into   `GLVisualize.ROOT_SCREEN` (may be due to aliasing).

  - several of the tests in GLVisualize/test fail.

For now, I can just remain with the previous (functionning) commit 63c828e042030e6382af3f073. For the longer term, I would like to know if this behaviour is related to an issue with my use of the API. Also, I do not know if the version I loaded from `master` was usable enough or in flux.

Best
Alain


Hello @danielsdeleo,

The "result_count" is not an error, I think is better to keep this as standard output. What do you think?
Please, see: https://github.com/chef/chef/issues/3229

Thanks!


Updated for 2.4.1 Sorry about that.


Within init()  mem_calloc() is used.  Is that going to assure us proper alignment??


>Why do you understand it?

I am not sure what you mean here. Anyway, I think you did the right thing here. I was under the impression that if the user had unchecked the "start seeding after creation" checkbox that it added the torrent in the session and rechecked it. That wasn't the case. It just created the torrent without adding it to the session.


Hm... Did you also check out GLAbstraction!?


@weltling Checked in as you suggested.


Review plz @Nelrohd


What is the actual string from the XML document? Convert the ascii back to alpha characters.


Thanks a lot. I don't know if you do wish to work more on this code but I think it's not working properly under PS1.6.0.14: in the module configuration, I don't have any category showing so I can't choose the Google categories for my PS categories and related options.


Good job @Altaf-Mahdi


Thanks, I will fix this.
Best wishes,
Andreas


Could this be simplified to the following?

```coffeescript
return 'values must not be blank' unless vars.values
```


Thats weird. I know @dim00z is working on it, contact it :)
Try to uninstall and reinstall your module, maybe It can solve your issue.


People wanted to use this in scripts with unix pipelines, and the result count was a headache.


Thank you so much for this! It's really, really amazing what you've done! I only have one question. I can open and view every model except for Minotaur A. I get a very quick error message that quickly closes, giving me very little time to read it, but it says something about not being able to locate Minotaur A. I have the shadow of the colossus PS2 game disc that I'm using to reference the files. Do you know what might be happening? Like I said, I can open all the other models, just not Minotaur A, which was the one I really wanted. Any ideas?


REAL CODEZ!!! 💁


We could probably avoid this `for... in` loop.


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 42](https://teamcity.labkey.org/viewLog.html?buildId=253149&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 0 Build time: 00:08:02
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

```


```ruby
require 'redcarpet'
markdown = Redcarpet.new("Hello World!")
puts markdown.to_html
```


```
asd
````


```diff
diff --git a/src/server/game/Spells/Spell.cpp b/src/server/game/Spells/Spell.cpp
index 0a25b2c..54a17ea 100755
--- a/src/server/game/Spells/Spell.cpp
+++ b/src/server/game/Spells/Spell.cpp
@@ -4770,7 +4770,10 @@ SpellCastResult Spell::CheckCast(bool strict)
             if (m_triggeredByAuraSpell)
                 return SPELL_FAILED_DONT_REPORT;
             else
-                return SPELL_FAILED_NOT_READY;
+                return return (m_spellInfo->Id == 15473 || m_spellInfo->Id == 46584) ? SPELL_FAILED_FIZZLE : SPELL_FAILED_NOT_READY;
         }
     }

@@ -4782,7 +4785,10 @@ SpellCastResult Spell::CheckCast(bool strict)

     // Check global cooldown
     if (strict && !(_triggeredCastFlags & TRIGGERED_IGNORE_GCD) && HasGlobalCooldown())
-        return SPELL_FAILED_NOT_READY;
+        return (m_spellInfo->Id == 15473 || m_spellInfo->Id == 46584) ? SPELL_FAILED_FIZZLE : SPELL_FAILED_NOT_READY;
return (m_spellInfo->Id == 15473 || m_spellInfo->Id == 46584) ? SPELL_FAILED_FIZZLE : SPELL_FAILED_NOT_READY;

     // only triggered spells can be processed an ended battleground
     if (!IsTriggered() && m_caster->GetTypeId() == TYPEID_PLAYER)

```


@cosimoc, I realized that when we retry an upload we usually retry against the same server even if the environment has changed in `/etc/metrics/eos-metrics-permissions.conf`. However, if someone runs `eos-upload-metrics` after we send a request but before we retry, then we will pick up any changes to the environment and potentially switch servers if we retry. I am not sure whether it is worth ensuring that all retries go to the same server, but curious what you think.


The tag is allow listed in the content. Change this to remove the tag here and show that all the bits are part of the node tree content and we can see that these bits represent a document node based on the first byte of the tag value pointable in the node tree content.


@mwjames Thanks as always for this release! I have updated the wiki and uploaded the files to SF. @JeroenDeDauw The wiki is ready for `composer update` I guess. :)


SET @MAXID := (SELECT MAX(`entry`) FROM `creature_ai_texts`);
DELETE FROM `creature_ai_texts` WHERE `entry` = '@MAXID';
INSERT into `creature_ai_texts` (`entry`,`content_default`,`sound`,`type`,`language`,`comment`,`emote`) VALUES
('MAXID','%s appears very grateful to be free of the koi-koi spirit\'s influence.','0','2','0','21326','34');

You can try this way.


I like this; proviging the illumos behavior as was discussed earlier in the issue.


trip-eq


I want to build myself a kernel for Android M especially for activating this in the config. Did you test this and had problems, or do you want to be in sync with the "mainline"?

I merged and enabled partialresume in my CAF kernel with no problems (most probably because I don't think it's working), but Android M might be different.


@23Skidoo This breaks `GET /genomeRelease` without species specified


I'm unclear on what this does?


this rule busts sums in firefox.

before:

![screen shot 2015-06-02 at 12 18 45 pm](https://cloud.githubusercontent.com/assets/478331/7944762/93c6c7c2-0921-11e5-9c5a-58f489945175.png)

after:

![screen shot 2015-06-02 at 12 20 07 pm](https://cloud.githubusercontent.com/assets/478331/7944787/b91db1de-0921-11e5-8f07-e64b3d02da84.png)


Try to keep the lines with in 80 characters to help reading on screens that can not scroll. The rows with bytes can be left in a single line. Fix the text in the rest of the document.


EC2 user is associated securely with my account. I am doing this to set up automatic server updates to reflect git changes, such that any pushes to github will automatically be replicated on the server with no farther effort needed.


Please address these comment.


Confirm, sometime it fails


After further consideration, I think this ought to have its own taskq.  Among other things (see previous comment), it complicates troubleshooting should the task ever run away for some reason.


I prefer adding messages where I have multiple assertions per test, since it helps me quickly recognize which assertion is failing. Even with different assertions that could be deciphered from the default error message, I think this makes it easier. If it bothers you, I'll tone them down.


Default profile time is 100ms.  Our website says 20ms but it sets just the profiling level and reuses whatever profiling threshold is currently set.
Account monitor check, notice the single value being passed into the set_profiling_level method:
https://github.com/objectrocket/core/blob/develop/viper/checks/instance_checks.py#L380-L394

Calls this in CORE:
https://github.com/objectrocket/core/blob/develop/viper/mongo_instance.py#L965-L983


That `sys.module['lib']`, because of this I guess. Installer is shared in `lib.__init__` `__all__`.


This change breaks a lot of packages that depend (non-optional) on node-syslog when installing on Windows platform.


Neighbors is a good concept, but we're not quite sure whose responsibility it should be.


Probably it is best to use a variable for this:
- hosts: all
  remote_user: '{{ user }}'
...

Then callout:
$ ansible-playbook playbook.yml --extra-vars "user=root"


It seems to work fine. Mongo docs don't discuss $in arguments needing to be unique, so I don't believe it's an issue.


I didn't see `getPromiseError`, which is a decent alternative. `assert.isRejected` seems cleaner, but you're right that it doesn't help much with testing `e.name`, so it's not a replacement (but I also don't see us switching to `e.name` testing for a lot of these errors). It's not a very huge dependency, so if you don't mind it too much, I would prefer to keep it.


I think this is the best way to track turn count.


Would you like me to set this to say the default is 100ms? Or do you want a larger note about how it's set to whatever it was last, and that we can change it, etc.


Balancer Window is PDT


nei das passt :) wenn du scho witergleitet wirsch bevor du all date hesch isch das ergendwie ned so en ideali usgangslag fürs dashboard, das brucht ja denn die date und müssti denn druf warte. isch also suber so. wenns en fehler git, wetsch doch ned ufs dashboard cho? :wink: denn bruchts en fehler. chasch ja mit mim coole neue globalMessagesService realisiere :stuck_out_tongue_winking_eye:

isch de rootscope i dem fall the best way to go? mit dem deferred bootstrap ding chöntisch so globali sache au hole, denn hettisch das scho vor em lade vo angular als injectbari variable :)


@stfx Don't get me started! I've done a fair amount of backporting lately, and keeping things in sync was always a challenge apparently! To give a perfect example: this had to be resolved in several places because master uses CREATURE_FLAG_EXTRA as prefix while the other cores use CREATURE_EXTRA_FLAG as prefix (notice the swapped FLAG and EXTRA)! I've made quite a few notes about things like this already, and I will change things one day to make things easier.

That said, you certainly have a point. In this case I caused it myself because I didn't backport the commit that changed the include to a forward declaration - I just remembered the comment about it and figured I could do it in one go. Not my best move of the weekend, I agree. I'll make another commit soon to sync the rest (and more).

And that typo... well, it's either that or making a separate commit to fix it, which I'm sure you agree is a bit silly. The voices in my head make me do it! I'll make up for it by making a commit in master that fixes this typo and more, then backport that to sync things :stuck_out_tongue_winking_eye:


> is another way to know when a element has finished being upgraded into a custom element ?

I guess by using the `created` property.

Regardless, the main problem with the name is the word "no".  It's unclear if you mean "number" or "non".   Maybe "_numNonUpgradedChildren" would make more sense.


+1 for this commit. It is a pretty thing, and I would have it. Is it precious?


It won't conflict if it would be declared in this way and if it would work the same as it is now if second parameter is missing:

function addField(\Ip\Field $field, $position = null) { .. }

I don't like expanding parameters either. But in this case, I don't see any additional parameters coming. This one is waiting for implementation for a year :)

Your suggested way of doing sounds interesting. But in that way I'm really not sure how to manage the backward compatibility. Till now all getFields methods have returned fields in the same order as render method. If the sorting is done only in the rendering phase, getFields and render will give different results and some plugins that now are doing field removal according to the position will likely fail.

Regarding fieldsets. Forms can't contain fields. Forms can contain only fieldsets. The addField method of the form actually add a field to the last fieldset. It is a shortcut for $form->getFielsets[last]->addField()


Yay!


Ah, environment variables. Gotcha!

Just one other thing I've noticed its to do with your update code. It works as intended but when I manually supply the -s switch the updater runs anyway but when the update process completes and script restarts it doesn't go into an endless loop (the script uses the skip that time)

Am I mis-understanding something here?


PDT here as well.


I haven't tested it, my goal was to be in sync with googles repo, yes :) I don't know the reason google disabled it, and since I don't own the device anymore, I can't do tests by myself.


Can you give me a Homebrew command that I can use that will test this function?


I'm still waiting for an explanation...


duh.


I'm trying
```ruby
  receipt_file_path = Utils.popen_read("tar", "-tzf", bottle_file,
    "--wildcards" if OS.linux?,
    "*/*/INSTALL_RECEIPT.json").chomp
```


Yeah, I'm rewriting a longer test with a bunch of isFalse() assertions, and I was about to amend this just to reiterate that I'm fine with them in tests with similar ambiguous assertions (and I see that I actually misread this one, which wouldn't show default message wouldn't show -1, so this was a bad example). I do think we can avoid them where there the assertions in a test are pretty distinct or where the default message would be perfectly clear.


I would link to Jeffs blog page first then mongos.


"existant" -> "existent"


@shintasmith Looks good to me. Thanks for making the updates to the URLs, Shinta!


Hi,
I now did, and nothing changed (squeezed graphics, mouse located normally based on unsqueezed positions)


This actually enables or disables the ability to click the `compact` option in the actions drop down.


+1 for the rename! ( and actually using a local var :dizzy_face: )

I don't think that the `hashchange` handler needs to use `preventOnTransition` though.

It behaves normally w/o it since `preventOnTransition` was only needed to stop us from pushing new history after calling `onpopstate` . We shouldn't need to account for that behavior when using `hashchange` :)

Your thoughts on removing `preventOnTransition` usage from `hashchange` callback?


But better it's maybe revert it, because it causing high bug


Hey @pei-jung,

Is there anyway we can not pass a variable into Liferay.Language.get()?  It is bad practice to pass a variable as the language key.

Please let me know if you have any questions.  Thanks!


Yes I am in the process of incorporating all the changes. Thanks for the review!


Xcode 5.1 on Mountain Lion. More details @ https://github.com/Homebrew/homebrew/pull/40289


// dibuja en pantalla las figuras


Made sure that ctr wil be initialized


We can keep it (unless @simonster has objections), but I don't think we should use it for much more than `isRejected`. `isFulfilled()` and the rest seem pointless when we have `yield`.


(Never mind, this is not in the latest revision.)


Should not run for loop on runchecks


Did we drop support for Xcode 5.1?


this is hw1


maybe that happens because of wasted time on loading screen or smth else


I took this out because I figured that if a user filters based on a connection name that isn't in the list of connections but is still technically a valid connection name, then an Error shouldn't be raised, rather the filtering would just have no effect. Same with valid_connection_ids. However I'm not so sure about taking it out because maybe we should warn them at least that their filtering has no effect...


this is new class


Can you try updating every Package? Probably by running the script... I'll see if I can reproduce this with the XML tree


Style violation:
*Prefer a possibly null pointer over boost::optional with a reference type.*
https://github.com/vinniefalco/rippled/blob/style/doc/CodingStyle.md


gradlew javadoc :-)

In something like IntelliJ you can also use CTRL-Q with a method, variable, etc selected and a little popup window will appear with the javadoc


Hmmm not sure this change is right. We need to use real and imag functions (not the fields) so that we can pick out the real and imaginary components on non-complex arrays.

I think we could do with some tests that verify the desired behaviours, including but noit limited to:
- adding a complex array to a complex array
- adding a real array to a complex array
- adding a real scalar to a complex array
- adding a complex scalar to a complex array
- etc.


@bmcfee is there any reason why confidence is constrained to [0,1] here but not in other namespaces?


@taconaut it was your idea and I agree to remove it. :)


> as it will not stepdown if a secondary isn't within a small enough difference from the current primary.

Double negative


Is the using declaration necessary?  You don't seem to be actually using it.  That also goes for the next file.


Might as well change Handle to Local while you're here.


If the argument isn't passed, I'd like to assume that it's in ~/.ssh/id_rsa-acceptance.


Both.  Say it is 100ms but that we can change it upon request.


FURST !!1


need to pass MainFrame.this here as well, doen't compile! :fearful:


@kghbln Do you think we want to give `mediawiki/semantic-breadcrumb-links "~1.1"` and `mediawiki/semantic-interlanguage-links "~1.1"` a try on smw@org?


Ah crap. Thanks for catching it. Added this when I was doing implementation experimentation.


Already did that, I'll contact @dim00z


> notifcations

notifications


> There is. The CameraRoll API is so far the only API we ship with React Native that lets you get a URI handle to a native resource. Supporting those URIs as an XHR payload gets us one-step closer to actually being able to write a useful demo app that makes use of that CameraRoll API, not to mention writing real apps.

@philikon Ahh, sorry - I wasn't aware that the `assets-library` protocol could only be managed by the assets library.

I agree, this should be part of the polyfill. I think this just needs a second code path for the `file` protocol then - and we'd just use the method above for getting the data for that protocol. Currently the camera module uses the `file` protocol, as well as pretty much any app that's storing data in the filesystem.


I believe most/all of the beaker methods can take an array of hosts as the first argument, which means you don't need to iterate yourself.

Not a big deal obviously, but useful to be able to shorten the one-liners.
```ruby
if puppet_build_version
  install_puppetlabs_dev_repo(hosts, 'puppet-agent', puppet_build_version, '/tmp/repo_configs', opts)
else
  abort(...)
```


Is there a general rule for not repeating strings? (So that text wouldn't have to repeat itself 6 times like in the old code)


I don't know if this is the time or not....
but I'd like to get away from hacking on the $SYSTEST_CONFIG file and consider generating it with genconfg2, and passing in the name of the thing we want to test as an argument on the command line.


Did you document anywhere how to start romeo with an xml file? I always forget the command and spend several minutes looking for the command^^


I see, will fix in a minute.


Same thing about passing in the whole `hosts` array


@mwjames Sure, why not! @JeroenDeDauw May you do the honours?!


Is there any harm to have the extra check on `hashchange`?


Yes, I agree


@gitter-badger
Please remove this commit. In reference to the following issue/bug.
https://github.com/gitterHQ/gitter/issues/709


Probably `const` it at the top and use it here?


Why not expose ```componentNameMap``` as a global config to enable custom inputs using the same convenient ```{{input prop type="custom-comp"}}``` ?


I like this interface :+1:

To stay with the concepts of "decoupling", we should move the string constants from MainFrame to here:
LOGIN_PANEL, PROJECTS_PANEL

OR

For each view we could have a method in SwapInterface. For example: showLoginView, showProjectsView, showActivityView.


done


I'm sorry. I did not know that clicking on the badge would lead to such information. Thank you.


The shared PBKDF2 code will gather keys into its own aligned buffers, doesn't it? Same for the output.


@gabriel-bezerra no problem :)


Thank you! :D


Why is this change necessary? The function still takes a `token` argument which is now completely ignored.


I think the logic for a login should be as follows:

If username/password is bad, show text in red "Invalid username/password combination"

If login succeeds and user is a manager, goto projects view.

If login succeeds and user is a project member, the goto member view. But member view is not implemented yet, so display a Dialog, "Project member login not supported yet."


noticed! looks cool. versatile node!


Technically: "For one liner docstrings, please keep the closing """ on the same line." :) https://www.python.org/dev/peps/pep-0008/#documentation-strings
https://www.python.org/dev/peps/pep-0257/#one-line-docstrings


Can't test this at the moment, but you should be able to use

``` csharp
BindingScope.AddChildResolver(t => t == typeof(SplitView), d =>
{
    var splitView = d as SplitView;

    if (splitView == null)
        return new DependencyObject[0];

    return new []
    {
        splitView.Pane as DependencyObject,
        splitView.Content as DependencyObject
    }.Where(e => e != null);
});
```

in your App.xaml.cs


Punt this to the main thread if we're not already there. Be careful not to capture an ivar in the block, of course.


> I agree, this should be part of the polyfill. I think this just needs a second code path for the file protocol then

Basically, yes. We want to make the resolution of those URIs pluggable, so that the networking library is independent of the image library (that implements the `asset-library` and `ph` URI scheme resolution). Adding handling for `file` will be simply and almost certainly something we'll support, too.

*cough* @nicklockwood (hai!)


Worth a comment about when these two sets might not be disjoint (e.g. the rotation example I gave).


不是 `\AtEndPreamble` 的问题，旧版本也是在 `\AtEndPreamble` 阶段才引入 `hyperref` 的。而是由于我们在随后的提交https://github.com/CTeX-org/ctex-kit/commit/fba00d3cd071130ba9bc8bed2f1eab6c04c28e08 中将 `\hypersetup` 改成了内部名称 ` \ctex_hypersetup:n`。忘了还有这一茬了，现在给它补回来就可以。


Actually, the line it puts there is not the line we are expected to use in Scala. It shows:
`libraryDependencies += "com.github.t3hnar" % "scala-bcrypt_2.11" % "2.4"`
instead of
`libraryDependencies += "com.github.t3hnar" %% "scala-bcrypt" % "2.4"`.

Anyway, problem solved. Thanks.


Nit: { on new line


normalerwiis tuet mer zerst d dependencies vo angular (mit $) agä und denn di eigete ohni $ :)


Reviewed and good with pr merge


How do we use it?


Já aprendi :+1:


Looking good @kahmali :+1:


w/SSL? :)


Yes, that's right. If I remember correctly simple_task.py does nothing.

On Tue, Jun 2, 2015 at 11:29 AM, Jasper Snoek <notifications@github.com>
wrote:

> Delete simple_task.py
>
> This file is deprecated and looks like it can be safely removed.  I'm not
> sure we'll need any layers of abstraction here if we just have one Task
> (@mgelbart?)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/HIPS/Spearmint/commit/c97959ae0449a6594226fe7d18d1384d9463d215
>


Looks good to me.


Fixed in https://github.com/PR2/pr2_gripper_sensor/compare/fix-catkin-install?expand=1


Cool. All yours.


@ceckert @thescubageek this provides the rake task for fresh deploying singledomain clients, and sets multidomain clients to always flush and go fresh


he he he


Remove dead code.


Yes: confidence here is inherited from the [mirex task specification](http://www.music-ir.org/mirex/wiki/2014:Audio_Tempo_Estimation#Submission_Format).


what's the reason for the string limits?


RadianceRefactor (rpg777) - x86_64-MacOS-10.9-clang: Tests Failed

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


God this is awful


Add parens


"edited file" is not really a descriptive commit message. You could have left this bit out so that your commit message read: "Fixed: removed "mei:" in element tags" which would make your commit subjects more skimmable for review purposes.


Hi Haeawebteam,

The correct package.json is located in the 'webapp' subdirectory.

Cheers


That sounds like a good start - or even putting `throw new TypeError("Expected options to be an object or a string but got " + typeof options + "instead")` as a function on top (or with your text it's still a big improvement).


This should be `startDay`. Update the tests to catch this.