@flavour so the virtual field just get created but actually not present as a column in the real database (because same as id?), but can be used on other places like list_fields, etc?


Not sure if this is actually correct or in error...  I presumed it was and oversight and took the intention that a little endian access was intended.  a similar case was spotted and noted a few lines below.


NOTE, when I merged, I did not try to look for bugs.  I had to find a couple of creative ways to get the merge done right, but the logic should have stayed the same as it was when in 2 functions.


Agree :( It's more bugfix than enhancement.


Just re-pushed! Tweaked line 85 and added .this in front of makeNoise... Don't know how to make another pull request to a pull request in the queue... yet. :)


As noted above...  big endian fetch where I presume little endian was meant.


Oh man, this slipped through the cracks and now I forgot all the nuances.

I thought through it again just now, and came to the following conclusions. Let me know if you agree:

* Since it seems like you want both `function` and `hybrid` methods to return the caching wrappers, we do in fact need a `CachedHybridOp` class to exist.
* Having `CachedFunctionOp` extend `AbstractFunctionOp` is probably suboptimal, because `AbstractFunctionOp` declares `in` and `out` fields and implements a few things based on those, whereas `CachedFunctionOp` does not need those and in fact should never touch them. Hence, `CachedFunctionOp` should instead extend `AbstractOp` and implement `FunctionOp`. Then `CachedHybridOp` should extend `CachedFunctionOp` and implement `HybridOp`. All methods of those two wrapping ops should simply call `op().foo(...)` for each method `foo(...)`, where `op()` returns the wrapped op. The only exception is the `O compute(I)` method, which has the caching logic you wrote above.

Right? Or am I missing something?


see branch: https://github.com/mrunfeldt/EuclideanDecoders/tree/DNdecodingModel


done


Looking at this commit, I fail to see how me saying `--test-full=0` can end up with benchmark_time not being zero??


s/?/#


Reverting to before this commit, and problem is gone.


thanks @DougFischer


Can you explain, please, why we don't need the maven-settings. although the Settings object is being injected?

Thanks!


I've updated my branch. The new version of this commit is: 8b988665a9f467f5a59a0471db81c5ed8f156baa


Here too...


This is definitely the reason the bot builds are so damn slow now. They end up benchmarking every format for one second...


Elsewhere in the spec it's marked up as `blob:` URLs instead of "`blob`" URLs. Since URL Standard uses the latter, maybe just fix that other part of the spec ("For example, scripts can be external files with the same scheme...").


Is this tarring the file in gzip format?


No need to spend one variable for just one allocation, do like :--
        return {
            'name': _('Customer Invoice'),
            'view_type': 'form',
            'view_mode': 'form',
            'view_id': [self.ref('account.invoice_form')],
            'res_model': 'account.invoice',
            'context': "{'type':'out_invoice'}",
            'type': 'ir.actions.act_window',
            'nodestroy': True,
            'target': 'current',
            'res_id': Invoice and Invoice.ids[0] or False,
        }


another comment on 80


gotcha! What a twisted mind...


Nice! :+1:


``` website.models ``` will not have access to ``` NO_LOGIN_TYPE ```, needs to be changed to ``` website.mails ``` or ``` website.mails.queued_mails ``` or an additional import statement


Hi, Just wondering if this fix, or an alternative fix has been pushed up to the main opentx repo yet, I've been using your script here: https://gist.github.com/xdandys/67612d119be6e0b4841e (modified to work with anysense/ DJI wookong) and the script's crashing, I think when it gets GPS position. Running it with test = true gets no crashes


done


Hmm this actually only happens when OpenCL formats are included. Dafuq? Do we have a stack overwrite somewhere?


Ivan,  Why do you have a colon ':': rather than a semicolon ';' after the declaration of dataWindow?

    DataWindow (T* basePtr = nullptr,
		         unsigned int columns = 0,
                         unsigned int rows = 0
                         ) :     <--- Here

Is this a new convention in C++11 or ... ?

Also GitHub is complaining about a missing newline at the end of

   include/geometry/dataWindow.h

This may be a Windows vs Linux thing, but we might as well add one.


Relates to #1.


@skeptical hehe not yet, there are still a few todos:
- Adding multiple jumpy nodes will probably cause issues as the same references of DLNAResource will be passed to both. Either allow to add a single instance or support multiple.
- Create a DLNAResource to refresh AdditionalFoldersAtRootWrapper children when AdditionalFoldersAtRoot.getChildren has changed.


Pramod, I think the behavior should be consistent with all overloaded method definitions of getResolvedRMWebAppURLWithoutScheme, especially since all of them are public methods.  Now one implementation works with null while the other one does not.


comment on  at the moment display line 84


@mjwitte @sushobhit

I have went back and forth on whether or not to expose the users to those. I am OK either way.

It also looks like we have another issue to track down with this. I ran a little parametric sweep comparing the mesh density parameter and the geometric parameter for the BESTEST GC30b test @BobHenninger sent.
![figure_1](https://cloud.githubusercontent.com/assets/2985979/10289149/8de633e4-6b62-11e5-9951-eda1ee40b46c.png)

It looks like there is still some issue where the heat flux or slab temperature are not being normalized correctly with increasing num cells. We will keep looking into it.


If @api.model use as a decorator then don't browse the order_id its already recordset


LGTM


i have try to connect used "dbname" in users table, password use sha1 but always "Wrong password"


he foo


> RKey.Key() is fishy in general (To go from Key to RKey we use keys.Addr; why is the reverse conversion an identity function?)

`.Key()` is to interpret a `RKey` as a `Key` intentionally (after all, `keys.Addr(rkey.Key(.))` is the identity. It's really only used when we already have a `RKey` but something still needs a key - we could use an explicit cast instead but then there's no function to put a comment on and less support for identifying the places in which it happens.

The situation above is somewhat non-standard but you're probably right in that we should just call `keys.Addr` on the argument. It may look up "unexpected" ranges when you put a range-local key in there, but you shouldn't be doing that anyways (the alternative would be an error).

I'm going to make that change and see if we have any others like it. `.Key()` should really only be called internally (a lot of key constants are `RKey`s but sometimes you want to use them as a `Key`, like below) and not on user data.


Perhaps should be clarified that it is a distinct purpose than the calculated end-time based on durration?


In email, @cperciva said

> foo_bar.h sorts after foo.h.
> This doesn't affect the general rule that if the C file implements functions
> declared in a header, than that header appears by itself at the end.

I assumed that "by itself at the end" meant "there is a blank line before the #include", but now I'm wondering if I misunderstood.


AddDataSetTestSupport (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1935 of 1940 tests passed)

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


Does package-all build binaries, or binaries and source?


嘿嘿，我想做把一个模板移植到这里来。


Hmm, this path was unused, but now I wonder whether the current setup allows mass assignment of the global admin role...


I think we should just go ahead and remove this line entirely rather than just commenting it.


We probably still want.

-  The name specified via the command.  NAME will be replaced with IMAGE if it is not specified.

The way this works is

`atomic stop rhel-tools`

$NAME == rhel-tools

Or

`atomic stop -n myrheltools rhel-tools`

$NAME == myrheltools


First assign variable and then return instead of this method directly return and save some bytes :) like :--
return self.env.cr.fetchone()[0]


done


Any reason why you are grepping for ERROR?  Should we see all output?


Bah, you seem to know my shit better than me :-(


http://lineage.power.plaync.com/exe/ln1/doc.php?doc_no=97&forum_no=41742&page=

I'm having trouble finding the playforum damage tables for either weapon,
live sword of destruction puts it as 3-5 leech, which should be /4 I guess
the random damage comes from the small roll? So it should be rand(dmg)/2,
not rand(dmg). 19+3 /4 = 5.5 (int) = 5, the lower bound is probably capped?

On Mon, Oct 5, 2015 at 12:26 PM, Michael Adams <mikesa83@gmail.com> wrote:

> Standby; have to locate the damage test links for this and exec when
> chao.. I found them on accident before. Of note destruction only affects
> base weapon damage, modifiers like str dmg, zerk, fire etc don't count,
> that's why people thought sword of destruction was broken, it actually
> works fine, it's just so weak no one could tell. Let me find the articles
> and post the numbers.
>
> -Mike
>
> Do we have a reference for the buff?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mikesa83/en/commit/c915bac6cb6b77a4cf9f59359ed6e9543d1f89a2#commitcomment-13585007>
> .
>


I actually don't need these casts at all. There's unfortunately some back and forth in these commits; they should be gone by the last one (a lot of these constants simply turned into `RKey`s).


:abc:


sure, but gearman seems to be the only unit test where this is failing. For some reason, its not a problem for the other unit tests. Right? Its a CricleCI thing; CircleCI fails, but there's no way to see why it fails.  I will try this now ...


AddDataSetTestSupport (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (861 of 866 tests passed)

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


As per new api guidline _order should be upside of fields creation


When we were using ACE for mutexes and guards, this was a return-guard also.  However I'm pretty sure that is wrong, since a failed lock would have (or rather, should have) resulted in a client disconnect.  Therefore I am somewhat skeptical that the ACE return guards were working.


I saw that one, but that file in a completely different format than the ones we use.

But I guess you mean remove the shareiscsi part?


comment on render


test


... nothing like that video happens to me, and it makes no sense at all.  Clear out your Build dir and rebuild, it's probably a result of something not being rebuilt after wolfman's restructuring.


AddDataSetTestSupport (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1931 of 1932 tests passed)

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


Not sure if we should check for `err` here because I guess `err` will always exist at this point.


Ideally, format in a separate function. for instance, to add localization down the road.


I overrode _generateJournalArticleDom() because it calls _generateDOIdataDom() every time https://github.com/pkp/ojs/blob/ojs-stable-2_4_6/plugins/importexport/crossref/classes/CrossRefExportDom.inc.php#L377-L381. Ezid can only have one doi node, so I have it called on the _appendArticleXML() and _appendIssueXML(). Maybe there's a better way to partition this out, but it seems to me that for Ezid appending an article xml and appending an issue cannot be the same process as Crossref has it.

So, the whole method needs to be overridden to remove that one bit--the same with _generateJournalIssueDom().


I'm so *well-cited* in this one... :smile:


Viikon 5 tehtävät olivat ihan ok


Both rawurlencode and decodeURIComponent adhere to RFC 3986; which should allow the values to be initialized unmodified in the JavaScript variables.


可以的，你可以Fork一个版本，做一下看，如果合适的话我会合并过来


Done. In the future, it would be super helpful if you could submit a pull request for such fixes.


No D7 support?


@DaveGillie this makes the yaml unusable for scripture forge


more of a structural question. Are these tabs displayed on all pages? if so you should create a header file and move this html there to prevent redundant code.


done


@Maxthreve The empty stylesheet did not make it into master... this is the final commit:

https://github.com/sophomoric/maddie/commit/67d406ed01a9af2b99a129a9a507a8fe2a39530a


Can we make it say 'Ubuntu' here and 'Fedora' above? I like this better.


Its a lot of forking code for crap like this, not sure how much I want to
write / support for 2-3 more years lifetime on 7

On Mon, Oct 5, 2015 at 11:25 AM Brad Erickson <notifications@github.com>
wrote:

> No D7 support?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mrf/grush/commit/b45bad662e566af93c0688f064bbebd27f45bc61#commitcomment-13598488>
> .
>


I want to test this out on Fedora, so I'm making a note to do so.


Move to a separate method :p


You could define a Set called requiredProperties with the 3 values and then do requiredProperties.removeAll(propertyNames) to get all the unset properties and you could throw a single exception stating all the missing properties.


I like the optional colons in the HO params, I hate the commas.

I think the choice should be which syntax is more regular. The new syntax is more regular because of the always optional colons. I vote for the new syntax


That said, I will still probably take out the jquery part after I finish styling the rest of the pages (assuming I do not use it for other things)


loool


changed


hey this is a comment


@jriesmeier @michaelonken This seems to cause trouble on older MacOSX compiler. See http://massmail.spl.harvard.edu/public-archives/slicer-devel/2015/018824.html reported by @lorensen

I will keep up posted.

Cc:  @pieper


Yes, this should work now. That's the only pending issue right @bozana ?


It'll be more trouble to support both, but shouldn't be crazy. If D7 will not be supported then this needs to go: https://github.com/mrf/grush/blob/b45bad662e566af93c0688f064bbebd27f45bc61/includes/libboot.go#L28


Can you please 7zip and  upload the elf file SLPM_653.24 somewhere? Thanks


How do you do this in Ubuntu? Is there a different set of instructions, or is it not necessary?


I agree, but I have to ask you now: where is that Pull Request to make it so? I mean, if you respect the effort that went into making Git what it is today, you may just as well respect it by contributing this change.


Mengwen, your last pull request looks good on this end!


:+1: for also creating life in the process...


Thanks for patching (again).

Applied fix with b04f278


Convert SQL Query into browse method and deduct 3 line of code like :--
if self.env['pos.order'].browse(obj.id):


bbef2e9 seems to fix the issue.


Hey do you think you can help me to build MAME for android?

On Fri, Oct 2, 2015 at 1:31 PM, not6 <notifications@github.com> wrote:

> when i had merge on my fork , i ran into conflict .
> seems you have the same here in /src/emu/netlist/
> that haven't been removed.
> removing the /src/emu/netlist/ solve then build find again for me.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/libretro/mame/commit/843e59861fe6e13e6a2739236452c9d0d558fc60#commitcomment-13568022>
> .
>


Actually the commit is wrong and should be dropped again. I think it was too late on Friday and I became confused.

The destroy Func should receive plugin->cb_data because it's implemented by the plug-in (perhaps overridden by the proxy). The unload Func is always implemented by the proxy and should receive the proxy's cb_data.

However, a proxy might use the same pointer for plugin->cb_data and plugin->proxy_data, this should be taken care of


This looks like an appropriate change (to update the dict with the cumulative tally dict) out side of the loop, but just wanted to check that it wasn't an accidental whitespace change


Nagios XI Setup config file


```
-self.shortname = 'popcorn';
+self.shortname = 'Popcorn';
```

:rage4:


Thanks for fixing that... but the particular link I was interested is "For more information on using the JCA adaptor, please see Application Server Integration and Java EE" which is broken as well :(


<!DOCTYPE html>

        <title>Soma - Share</title>
        <link href='http://fonts.googleapis.com/css?family=Open+Sans:400,700' rel='stylesheet' type='text/css' />
        <link rel="stylesheet" type="text/css" href="share.css" />
        <link type="text/css" rel="stylesheet" media="only screen and (max-width: 600px)" href="mobile.css" />

        <meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1" />

Support Soma with 3 Clicks

                Hola, rockstar!

We hope you're having an awesome day. You're here because we know you have influence. When you share something, people pay attention. So, please take a few seconds to spread the Soma love.

                Thanks so much,

                Mike, Zach and the entire Soma Team

0%

1. Share on Facebook

Share Soma with your friends

                    <button class="facebook">Share</button>

2. Post on Twitter

Hook your following up

                    <button class="twitter">Tweet</button>

3. Watch our Video

Thank you in advance for backing Soma

                    <button class="play">Play</button>

© Soma 2012


done


I thought we discussed these staying in what is now ipmicommon.h?


We've started to move on to harder material before we go back to reviewing


It's possible that the explicit template instantiation belongs in an implementation file instead of a header file, perhaps with an extern declaration in the header file if necessary.


What I mean to happen here is find the common denominator for an optimum setting that most users would find helpful as the default state.


@bbassett we are using this in places.  It will cause errors to just remove it


You don't need those anymore, you're already opening the files in their constructors.


Thanks for simplifying the if/elses; however, they're not equivalent to what they had been.  For example, previously we'd only set method to TLSv1_2_method if all of the other no's were true, but now we'll do it if at least tls12 is true, even if others are as well.  Is that ok?  And if so, are we checking for these in the preferred order?


Missing `std::` before `ifstream`.


Those could be just one variable, called something like `temporaryLine` and locally declared in `readFile`.


I'm trying to think of ways we can keep the benefits of the usual Crossref code (and XML structure) without violating EZID's requirements.  I know it's a bit terrible form, but what if instead we set a flag in generate() which indicates which DOI we are interested in, and then override _generateDOIdataDom() to only output that DOI?  Since we tend to call _generateDOIdataDom() and then XMLCustomWriter::appendChild(), I think we would need to extend XMLNode to allow XML comments, and then have _generateDOIdataDom() just output a comment for any DOI we weren't interested in.  Thoughts on trying that out if I code out a test?


Seems worth adding a comment that it's only safe (at the moment, at least) to register these predicates once we know that we will return `Ok` (which seems to be the case, here)


I don't see anything wrong with the GET_CAP_BIT enum being here since this library provides that function.  You could argue that maybe the library should be called something like "app_getcap" instead, but I am fine with it being left as is for now, especially since it also provides the wildcard for the app netfun.


All short names are cased like that: http://i.imgur.com/ofvKkbp.png


This is probably the cause of the segfault. You don't initialize those arrays to 0, yet always try to print 50 of them. Just change the storage to `std::vector`.


Hi Chris, it's great that you are trying Prolog again! I have some small comments: Please simply write `clpfd` instead of wrapping it in double quotes. You can omit `flatten/2`, and simply use the library predicate `append/2`, which is a better choice to remove exactly one level of nesting. Finally, all the `!/0` do not help in any way, you can simply remove them from the remainder of your program.

Nice use of CLP(FD) constraints, you are making a lot of progress with Prolog!


In this case it's not a huge deal.  But generally we've brought over entire illumos commits if it makes sense to keep the delta between platforms to a minimum.  In this case that tiny fix appears in the larger patch which is unfortunate, but the larger patch does make sense for Linux so I'd pull the whole thing.


попробуй http://ipinfo.io/json


Only has to be like this as you are using en

https://github.com/RMcGirr83/phpBB-3.1-nationalflags/blob/master/language/en/common.php#L37

Don't need 0, 1, 2, 3, 4 etc.  Just 1 and 2


done


Quick! Someone fix ModBar!


Ah yes. That's true. I was doing it for testing purposes. Let's leave it for now? Then drop it later down the line when we figure out how much work is required?


Yeah, `.Key()` should have a less-innocuous name since it should actually be fairly rare.


Yes, it does. This was something that @cambell-prince  pushed from my machine when he was helping us to get languageforge running (since we don't need scriptureforge)
I'm thinking it might be a good idea for our team to fork this repo and let your team have this repo so that we don't step on each other's toes. Thoughts?


Yes (although not just because same as ID - any Field.Method is like this)


Perhaps **Replace('\u2028', '\n')**


Don't use @api.one


Yeah, all pages if the user is signed in: https://github.com/sophomoric/maddie/blob/master/app/views/layouts/application.html.erb#L15
It's not redundant because it is in the application layout file, which is by default used everywhere... duh

Are you trying to tell me to use a `content_for(:header)` block? Right now it's the same everywhere, so I disrespectfully refuse. :chicken:

You sir are :bread:


Also, comment on the commits that are in master so our intelligent conversation can be viewed by the world.


If it only applies to the clang compiler then it can be guarded with:

#if defined(__clang__)
#endif

On Mon, Oct 5, 2015 at 2:31 PM, Max Smolens <notifications@github.com>
wrote:

> It's possible that the explicit template instantiation belongs in an
> implementation file instead of a header file, perhaps with an extern
> declaration in the header file if necessary.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/commontk/DCMTK/commit/ade7e936e362ba219df0e982d07807c6c7b73881#commitcomment-13598653>
> .
>

--
Unpaid intern in BillsBasement at noware dot com


cc myself to make sure I am watching this


That sounds good. I think you guys have one in the parent of this repo


@api.one deprecated


There is a problem if you apply this to an existing database, as the data there is is encoded as latin1.
Changing the connection string just returns data in the wrong encoding without fixing it.
A function to migrate existing data (or at least a warning) would be very helpful.


## Do some simple math. Add and multiply some numbers.

print(2+56+84+65498+21+22) #Prints the sum of the numbers given

print((1+4)*(9-6)) #Prints the product of the sum / difference given in the parentheses

stdout

s a
pa pam spa

/Users/gs13macpro/Box Sync/Gerhard Pilcher/Georgetown Univ/510 Course 2015 Fall/Python

darwin

import re #Load regular expression pattern matching library

myfile = open('<need to specify a location on your computer>', 'wb')

Something Else

Math 510 Fall 2015
54.0
Something Else

import Class_1_Module #uses Class_1_Module name space

## Show the difference in scope between importing a module and importing a specific variable from the module

Math 510 Fall 2015

#copies into local name space, overwrites same named variable


s a
pa pam spa

/Users/gs13macpro/Box Sync/Gerhard Pilcher/Georgetown Univ/510 Course 2015 Fall/Python

darwin

Something Else

Math 510 Fall 2015
54.0
Something Else

Math 510 Fall 2015


This file looks like it would be valid syntax in a C compiler instead of requiring C++, correct?  (We probably should ensure that)  Better to name this file *.h to indicate C is acceptable for a callback handler library.

Where do we intend the header to be installed?  /usr/include/host-ipmid/ ?  I'm wondering if ipmid-api.h might be a better name.


That URL say 2,999 additions, 3,808 deletions not shown. And looking at the whole file, they don't use the, whatever man/file format we use in our.

Our file:
```
.TH zfs 8 "Nov 19, 2013" "ZFS pool 28, filesystem 5" "System Administration Commands"
.SH NAME
zfs \- configures ZFS file systems
.SH SYNOPSIS
.LP
.nf
\fBzfs\fR [\fB-?\fR]
.fi
[etc]
```

Their file:

```
.Dd June 8, 2015
.Dt ZFS 1M
.Os
.Sh NAME
.Nm zfs
.Nd configures ZFS file systems
.Sh SYNOPSIS
.Nm
.Op Fl \?
```

I'm not even sure our man command understands that format. But if it do, we need to replace the whole file. The commit you link to won't show me the actual changes, but since the commit message say "retabbing", I assume _the whole file_ changed!


No, that is correct.  It was producing correct results before, but was making lots of extra updates.


TeamCity pull requests :: netty-tcnative [Build 26](https://garage.netty.io/teamcity/viewLog.html?buildId=2051&buildTypeId=pulls_netty_tcnative&guest=1) is now running


Actualizacion de repositorio


Here is a comment


TeamCity pull requests :: netty-tcnative [Build 26](https://garage.netty.io/teamcity/viewLog.html?buildId=2051&buildTypeId=pulls_netty_tcnative&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:44


Well then...it's another comment


> If it only applies to the clang compiler then it can be guarded with

@lorensen  Could you confirm that commenting the line allow the build to succeed on your side ?


Yet another comment...enough already


Nice :+1:


Cc: @fedorov


Two indenting styles in the same file? Sweet.


Looks pretty good - but i think we should update the hdfs register_slaves to do a similar -refreshNodes.


oooh... just looked at the whole file.. AND... EEWW!
https://www.youtube.com/watch?v=HOK4aBYNh3s

Kill this case statement por favor.


Two bracing styles in the same project? Sweet.


This include now seems incorrect.  Shouldn't it at least be ipmicommon.H?

Do we want this to use "local" includes or treat it as a callback library using "global" include?  Something like <host-ipmid/ipmicommon.H>.  Maybe we have a bit of chicken/egg problem since we'll be installing this header with the package globally, but also need to compile some libraries as handlers now.


public class TowersOfHanoi {

   public void solve(int n, String start, String auxiliary, String end) {
       if (n == 1) {
           System.out.println(start + " -> " + end);
       } else {
           solve(n - 1, start, end, auxiliary);
           System.out.println(start + " -> " + end);
           solve(n - 1, auxiliary, start, end);
       }
   }

   public static void main(String[] args) {
       TowersOfHanoi towersOfHanoi = new TowersOfHanoi();
       System.out.print("Enter number of discs: ");
       Scanner scanner = new Scanner(System.in);
       int discs = scanner.nextInt();
       towersOfHanoi.solve(discs, "A", "B", "C");
   }
}


Give proper variable name @above 7 lines , you have 3 option for changes
option 1: self.public_pricelist_id = self.ref('product.list0')
option 2: self.public_pricelist = self.env.ref('product.list0')
option 3: self.public_pricelist_id = self.env.ref('product.list0').id


`buildNumber` is a string.

https://github.com/sleroux/google-breakpad-ios/blob/331a581e7caa1443d95a1dd6f05f7063cc3f7a0f/src/client/ios/BreakpadController.mm#L255

expects it to be an integer. What gives?


Yes, it builds with the line commented.

On Mon, Oct 5, 2015 at 2:40 PM, Jean-Christophe Fillion-Robin <
notifications@github.com> wrote:

> If it only applies to the clang compiler then it can be guarded with
>
> @lorensen <https://github.com/lorensen> Could you confirm that commenting
> the line allow the build to succeed on your side ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/commontk/DCMTK/commit/ade7e936e362ba219df0e982d07807c6c7b73881#commitcomment-13598875>
> .
>

--
Unpaid intern in BillsBasement at noware dot com


Thanks Sergey, yes it makes sense, I will refactor it.


Well... I understand for Plush, but Glitter has the nice slider that's a lot more user-friendly than any user input text field.. Don't you think?


netfn_and_cmd.  No need for another make_pair.


Ugh.  I see what you mean now that I look at the full commit.  Well `man` does certainly understand the updated syntax on Linux but I suspect this is going to be conflict hell so let's go with your original tiny patch for now.  It would be nice to drop the iscsi bits tool in a follow up patch until such as thing is supported.


try to disable heads up, enable ticker, change theme and get a notification


I updated the patch to remove the shareiscsi part as a separate commit. Feel free to merge them.


Import should be in alphabetic order like :--
from openerp import api, models


Is this correct?  I presume little endian was meant to be used instead of big endian...  a few other instances noted below...


Consider hide the logic in model.


Is this correct?  I presume little endian was meant to be used instead of big endian...  as noted above.


You just introduced `lastCrashBuildNumber`. Why aren't you using it? And how do you expect it to be both an integer and a string?


To avoid an extra large pumukit2.css.twig file.


all logs should use lowercase


thank you. much appreciated.


I'm on board with that.


hahaha --'


What's the difference between these two properties?


g_ipmid_router_map.emplace(netfn_and_cmd, handler_and_context) is the most efficient.

Otherwise,
g_ipmid_router_map.insert(g_impid_router_map::value_type(netfn_and_cmd, handler_and_context)) is more correct than this make_pair call.  make_pair potentially has a different const-ness than the desired value_type.


why are you getting all records here ? a view parameter would be better to filter values


Let me fix this up


I had it removed in a follow up patch - forgot to remove it before merging this in.


`getDate()`


This is a great paragraph -- nicely put description of the sound we play out


Ya... let me fix this too


Refatoramento do codigo legado (just testing)


All your changes would be nice :-)


I'mm getting this now: https://travis-ci.org/visualfabriq/bquery/jobs/83748778


It works mostly because the string will always be an integer so it converts but it definitely is a bit too implicit. I'll convert it to an integer before adding it to user defaults.


Hi Varinderz,

The error you are getting is likely due to the fact that you can't retrieve your SharePoint context from session state.

The error occurs in the SharePointContext.cs file, on line 752.
protected override SharePointContext LoadSharePointContext(HttpContextBase httpContext)
{
   return httpContext.Session[SPContextKey] as SharePointAcsContext;
}

If I can read the stack trace correctly, I see that your app is called "ITProjectsSharePointAppWeb", so I presume this is your project.
Does the sample solution in PnP work on your machine? (Core.Peoplepickerweb) You can download this solution and run it without any modifications (only fill in a site url in the app file).
If the solution works, you can see if there are any differences between your solution, and the one on PnP.

There are a couple of things you can check in your solution.
1)Check if the "spHostUrl" parameter you are passing in javascript is correct and filled in with your hosturl.
csomPeoplePicker = new CAMControl.CsomPeoplePicker('Default.aspx/GetPeoplePickerData', spHostUrl, $('#spanCsomAdministrators'), $('#inputCsomAdministrators'), $('#divCsomAdministratorsSearch'), $('#hdnCsomAdministrators'));
2)Are you performing a action to SharePoint on page_load? You should at least have the following code in your page_load method:
if (!Page.IsPostBack)
{
  var spContext = SharePointContextProvider.Current.GetSharePointContext(Context);
}
This is needed, because the SharePoint app recieves all the tokens in his first request. It then creates the SharePoint context and saves it in session state, to use in later requests (like the webmethod getting the users). You can also check if you can perform any SharePoint calls from the page_load method, like getting the site title. This way you are sure that you can make calls to SharePoint.
3)Can you check that there aren't 2 SharePointContext.cs files in your project (or by a slightly other name), that contain the same classes (in different folders?). If that is the case, the "as SharePointAcsContext" statement could returns null. There should only be one SharePointAcsContext class in your project.
4)Can you debug you solution and check if there are values in the session? The best position would be a breakpoint on the "return httpContext.Session[SPContextKey] as SharePointAcsContext;" line.

This is all the info I can give you without actually seeing your code.
I hope one of these tips will fix the problem.

Kind regards,
Stijn Neirinckx


I would take out unpretentious, I think friendly is good enough and saying you're unpretentious feels a bit haughty, also missing a 'u' in attitude


remove developer logs


this has indeed fixed #276 , thank you! :)


hello


Ok.  That's a good check.  Ultimately, if you decide on an optimal value to hard-wire rather than leave as a user input, there should be some justification in the comments for one value over another.


             $things = $this.children('.thing:not(.morechildren)')
    -            .sort(function (a, b) {
    -                return (b.dataset.nchildren - a.dataset.nchildren);
    -            }).each(sortMe);
    +        .sort(function(a, b){
    +            return (b.dataset.nchildren - a.dataset.nchildren);
    +        }).each(sortMe);

         $this.prepend($things)
             .prepend($this.children('.thing.tb-drama'))
             .prepend($this.children('.thing.tb-ndrama'));

So are we indenting line continuations or no?


you beat me to it :)


Variable assignment should be in setUp methd becouse at test-case running time take some less time for load that variables like :--
partner_2 = self.env.ref('base.res_partner_2')
CrmLead = self.env.ref('crm.crm_case_21')
Partner_18 = self.env.ref('base.res_partner_18')


Shouldn't this be the other way round?


Scramble
https://scramble.io
Подключение через TOR - разрешают
IP в заголовке - отсутствует
POP3, IMAP - не поддерживают

ScryptMail
https://scryptmail.com
Подключение через TOR - разрешают
IP в заголовке - отсутствует
POP3, IMAP - не поддерживают


Hey, good work! Can you flip these to floating point numbers? Also, getters/setters too. Thanks!


what's the use case for this module ? usually route_resp is targeted , how are you going to get it here ?


Give proper name like crm_lead because its not object its reference to another records and when you assign any reference variable its should be in small case.


Я бы разделитель сделал `.`, а не `/`.


I had to. Couldn't stand the mess any more. Also first moment I had free :p


Favor size_t over unsigned int.


Ehh, there's a technical reason for this name.

I've rephrases this sentence a bit after merging, so it looks a bit less absurd. What do you think?


done


can't you simply wrap the futimens syscall? There are various examples within go-fuse how to do it.


Looks good to me. :shipit:


@Ironholds I haven't gotten any more additions to this gist: https://gist.github.com/rmflight/01d76949952a54568506

if you wanted to add them to this pull request


Feel free to remove that check & error message -- they are not strictly needed and frankly seem like a bad idea. cropOffset can also be used to translate the image bounds outside of the original frame.


Oublie de faire une pull request pour envoyer ton patch vers mon repo :)


any reason for using gradle 1.12 and not 2.6? or what ever the latest is?


This means that there are no public *or pending* versions.


@andresy wanted me to make this change


It is what I have in my storage of clean gradle files lol I should update and move away from the wrappers


use the existing functionality to handle ets handover/recovery


FYI synchronize() only works on the *current* gpu


I think the easiest way to go is to detect when the original center/radius style initialization is used. In that case, create a Transform as before and use it to initialize an AnimatedTransform. Otherwise, load a AnimatedTransform from the plugin parameters.


createTopics*


No idea, sometimes VS2015 has a mind of its own


the wrappers are good, but was wondering why such and old version of gradle. tbh it shouldn't matter.


Already changed them to floats in the commit right after this one. And, okay.


Good idea, but it would need to be in hex like '\u2028'


FYI I've been undertaking some refactoring of date/time widgets in 4.6. Refactoring old forms is a work in progress, but the new functionality is here: https://github.com/civicrm/civicrm-core/blob/4.6.6/js/Common.js#L593
Example usage to transform a single field into a date + time combo:

    $('#my_field').crmDatepicker();

This works on a similar principal as select2, where the original input field (#my_field in this example) will be hidden, and the widget will display the date/time inputs to the user and then update and fire events on the original field.


We all use WebStorm, and generally just roll with what it does by default.  Except where we explicitly need to override it: https://github.com/creesch/reddit-moderator-toolbox/blob/master/.editorconfig


Excsuve me, but didn't we have a talk about the correct git commit message format?
:-1:


Several didn't work; made me reconsider what the point of them was and I came up empty :-)


lol my bad :8ball:


I think there's a better way to put this last paragraph -- maybe something along the lines of our parties having the intimacy of a house party with the sound of a club venue where everyone is a part of the the Zero Fucks family and a room full of smiling strangers dancing with the DJ in a House of Bass late into the night is the standard.


@bwoebi , In my case, the goal is to write "stated class" like write a standard class, to keep reflex of final developper ;) I use my library in some project of my consumer since two years and it's really comfortable to write one methode by state (My lib is an implementation of the State pattern) : Less if, more readable, more understable :)

\ReflecionMethod::getClosure() is usefull here to avoid to write a "sub class" by state (so no errors are throwed by IDE) and not directly real closure, and avoid also to parse method of these sub classes to transform them to real closure and perform an eval... (or any file system operation if we "cache" them).

With bind, bindTo, under PHP5.x, the scope is changed with $newthis and so this behavior is awesome, but, it's forbidden with PHP7.0 for non real closure (because compilation pass, and i suppose it's too late to change its behavior, and too complicated compared to profites ;) ).

If I understood correctly (i have not written PHP extensions since 8 years...), in your patch, you check scope before perform the binding ? If after this check, we set the scope from $newthis ? I have a doubt about the original issue (on self::)

Else, another comment ;) Rather threw a Warning (and return null), like an RFC about PHP7, can we throw an Exception ? It's more convenient to intercept ;)


test this please


I thought about this as well and did not apply this pattern because you never know which additional information may be required. The lower, upper and upper each only require the word, the title case requires the index and the count as well. But what about when we want to capitalize even small words after a colon? Then we need to add an additional parameter (the previous word) etc.

I would propose to create a data structure that represents a title. In that data structure, the logic of splitting , joining and detecting whether a word is "escaped" with {} or not can be encoded. Then, each algorithm can work on that data structure and use the context they require.

I think with this data structure, we can integrate your solution. It can receive a lambda expression that is only applied on non-escaped words etc.

What do you think, @koppor ?


Shouldn’t there be a statement to reset this flag
``skipRemainingTestsInContextOnFailure <- false``
at the beginning of each context?


That `? extends ArrayDataAccess<?>` will bite us later. Let's not do it that way...


Hi.
ConfiguredTest already default valid flag to true for which groovy automatically creates the isValid method.  Why is this needed?


Bro create new files for El Capitan please T_T


use self.env.context instead of self._context


Nice!

In future we might want to have a way to continously send these messages whenever there is activity on a certain queue/topic.


No I meant comment as in:
```
this.maximumHealth, // The default value
ComparatorUtil.doubleComparator(), // The comparator to be used
0D, // The lower bound
(double) Float.MAX_VALUE, // The upper bound, in this case because Minecraft still deals with floats for health
this.currentHealth); // The current actual value


Should read `The copy namespace`


Your understanding of the code is correct. The idea is we want to be able to hide the tab and show it in the hotdogs menu instead. I agree using auth probably isn't the best. Can you think of a better way? I really didn't want to add another role based configuration option...


Ich habe es jetzt so bei mir eingebaut. Es funktioniert wie gewünscht.


that's a left over from the original donation.. that needs to be written


Hi Francesc,

I was also meaning to ask: currently, when you provide an AnimatedTransform to non-animated shapes, they get wrapped in a shapegroup and referenced with an animated instance. This is a bit slower but should work for any kind of shape (without modifications to the code). Did you run into issues with that? I have a slight tendency to stick with the previous general approach because it is faster in the case when shapes are not animated.

What do you think?

-Wenzel


Can we avoid the unchecked warnings here? Can't we use the `RAIs` utility class?


> What about utils.isPath?

damn, missed that. will fix, thanks.

> And so are we going to drop utils.unixify from here?

I'm thinking I should update both `utils.isPath` and `utils.hasPath` to use `utils.unixify` again, and given that the `unixify` and `normalize-path` libs are both unit tested (and more unit testable with benchmarks), it probably makes more sense to use those in `utils.unixify`. so it would look something like this pseudo-code:

```js
utils.unixify = function unixify(fp, opts) {
  // use `normalize-path` exactly following existing logic
  // if options.unescape, use `unixify` lib, exactly following existing logic
  return fp;
};
```
Honestly it might even make sense to just move some of those utils out and just require one lib back in. In my experience when code is modularized it's more likely to be thoroughly unit tested and optimized.

thoughts?


Wow, "copy" was part of _math_? Weird.


Dang. Ok. Thanks!


Curious to know what it was doing that was so fucked


@guillaume-boutin Thanks! I meant mostly the content lines specified in the last comment: 46, 48, 49 and 50. Almost there~


Why not 3.3, which seems the latest (although also quite old)


It should be a simple thing such as importing the RAR.. if you have issues please let me know.


Why map twice :)?

```js
paths = paths.map(function (p) {
	return trash(path.resolve(String(p)));
});
```


in assertTrue() give proper message, if argument not True


Please consider removing the ```uncaughtException``` catch.  I use throw errors to restart the bot when things malfunction (or occasionally on demand), and this prevents those from working.


please include sip traces so we can recreate them using sipp scenarios to see if this works ?


I didn't saw you already pified the library so I did it as well :). This is how I implemented the `trash` function.

```js
function trash(src) {
	var name = uuid.v4();

	return xdgTrashdir(src)
		.then(function (dir) {
			var dest = path.join(dir, 'files', name);
			var info = path.join(dir, 'info', name + '.trashinfo');
			var msg = [
				'[Trash Info]',
				'Path=' + src.replace(/\s/g, '%20'),
				'DeletionDate=' + new Date().toISOString()
			].join('\n');

			return Promise.all([
				fs.move(src, dest, {mkdirp: true}).then(function () {
					return dest;
				}),
				fs.outputFile(info, msg).then(function () {
					return info;
				})
			]);
		})
		.then(function (result) {
			return {
				path: result[0],
				info: result[1]
			};
		})
		.catch(function (err) {
			if (err.code === 'ENOENT') {
				err.noStack = true;
			}
		});
}
```


I will do that.
I thought of adding them in case they were all set but come to think of it it doesn't make much sense to set center, radius and animated transform at the same time.


AddDataSetTestSupport (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1932 of 1932 tests passed)

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


Ok. Any chance you could fork and take care of it? You'll probably want to change the 'error' event that gets emitted to be called something else, since it seems like emitting something with that name crashes when you aren't listening for it. Maybe something like APIError?


Tests can be done with `async`/`await`

https://github.com/SamVerschueren/xdg-trash/blob/master/test.js


Use read_group instead of search


I know this from ruby scripts that you can get the exact location of a script using `__file__`. With that information, you can resolve a relative path such as `../src/main/resources/l10n` always correctly, regardless from where you called the script. Do you know whether there is such a construct in python as well?


Newline


Inline


It's ok. No problem at all. Maybe it's because AICP use old CM heads up codes. crDroid switches to SlimRoms heads up


was it necessary to remove the `?>` ?


@Ulaize Inline style still there- maybe just forgot to commit the file?


Although this seems like quite useful code when working with FTP server. BUT at the moment there are no fixtures that actually do that, it has no relation with other code in the project, and this does add an extra dependency.
Do you have some fixture(s) that uses this functionality that you can share?

As it stands this might be a worthwhile project/jar on its own as FTP helper, or maybe as part of a bigger library containing networking helpers?


Remove this print stmt


I wasn't aware the original master branch for geoXML3 did NOT support polys. this commit fixes that.


"schema1" -> "other"


Should we add `keysSet :: HashMap k a -> HashSet k` and `fromSet :: (k -> a) -> HashSet k -> HashMap k a` `like in [containers](http://hackage.haskell.org/package/containers-0.5.6.3/docs/Data-Map-Lazy.html#v:keysSet) ?


// Create "orders" table in different schema


For now I think we should just add some unit tests to catch the regression, provide a minimal fix, and ship this release. The changes to `utils.unixify` you described could be part of that, but splitting out to more modules I think is overkill for the moment.

Beyond the 2.3.0 release, no objections to the restructuring ideas you've described.


If I remember correctly, this should simply need to create the query and then call the self.select(query) from the dbdriver base class.


Inline this name as "orders"


Commit text is a lie, no submit for here


Inline the names here too


Fine as a solution to this bug, but I find this whole block could be better designed without reflection:

- reflection means this block isn't type safe.
- the method names and signatures are are now constrained in non-obvious ways in the task service.
- the JavaScript portion and the Java portion are now bound together over method names, which probably shouldn't be visible to JavaScript.

Given the option, I would prefer to have an interface with a single abstract method (SAM with method string invoke(ConanProcesService)) and a Map<String, SAM> that is statically initialised.

That's type-safe and avoid things like the having to know the implementation details of variadic args. If you want to add a TODO and a bug and assign them to me, I can get around to it eventually.


I don't mind reverting this. I started by adding animated transforms to shapes used by the exporter as lamps (disk, rectangle and sphere) before trying to add emitter support to instances.


Add a test for filtering by schema but not table


Looks good.


引號用「」這種的，“”是簡體規範


try to see if you still have the speed up without the check for view_map. Considering view_map as destroy_map disable some inplace case for scan.

If you still have the speed up without the view_map, then just remove it instead of making the check more complicated.


it cause serious problems in compatibility with system :(


Not sure if we want to use javax.xml.bind.DatatypeConverter or another Base64 converter.


Thanks Rich, I did read again the wiki and ofcourse you are right.

I will fix that.


doesn't line up, and you used tabs


So we're making this the highest signed number right?
What happens if things suddenly refer to it? we then have a negative refcount?
(~0)/4 ?


Ok, will try that.


Sorry for the lack of doc. Just started some: http://wiki.civicrm.org/confluence/display/CRMDOC/crmDatepicker


There has disappeared btw a permission config file from /etc/ for camera. Do you think we should add it to the removal list, or just keep it in place for existing installations?


Wrap the args


parace ser internal, en caso afirmativo falta el static.


Is there tests in scan that check we are inplace? Just in case this PR have a small problem that just disable it completely.


ok -- if these changes were mainly intended to support emitters by avoiding to wrap them into instances, then the better and more general way would be to make instances emitter-aware.


@P-Kito, it should to reset.


@idgserpro não esquecer que este é um projeto comunitário; eu também estou com muita coisa atrasada.


Update comment to talk about DebugMode.


(I will take a stab at that when I return from my travels this week)


QUE LINDO :)


Decrease the line of code :
if self.env.context:
    self.env.context = {}
try:


do it


> How do you feel about "Atom will also execute the code you add to your init file on startup" or even just "Atom will also execute the code in your init file on startup"

Yep, much better -- thanks, @thedaniel! Using that as of 61bcdd6.


Make sure that ``` OldSummon ``` shouldn't be hunter pet.


問一下，強盜也是部族的一種嗎？會帶入 <clan> 字串中？


I hardly know her.


@da-an  możesz nadzorować ten kod ? wrzuć coś od siebie w wątku na forum. ;) Nie wiem , zrób magiczne triki w VS z tym.

PS: Taby na spacje jak coś zamieniaj :)


Wrap


Merge from origin.


Not related to loot, no.
But the other crash you get was same crashlog?


yeah, looks better, thx :)


Not to start a big discussion here - just a small comment. In my opinion for parsing galleries we should have a method that takes wikitext and returns structured information about images, captions, alts, etc. (including not existing images). It would make it easy to use, easy to test hence easy to maintain.


ok


I do not understand. I made another commit to double check but everything seems up to date.

Here is a screenshot:

<img width="343" alt="screen shot 2015-10-05 at 3 20 09 pm" src="https://cloud.githubusercontent.com/assets/13457207/10290879/a89c4f4a-6b74-11e5-964e-119397ed669d.png">


This entire case statement should not be needed.  I realize it was there before, but it's not required since the purpose of this section of the test is replacing the opcodes which have a different oneline and full disassembly and OP_NOPX codes are the same for both.


Done.


This should be a `Set`


Give Proper name as per new API guidline like :
IrModel = self.env['ir.model']


The build is slow (about 175% what it used to be), is that the tests now test every index slot (at least I think that is what happens).


nice, was considering even doing this for main settings since i was having a lot of difficultly getting all our machines to install bower on the same path


Make this a `Set`


I like the use of Promise.all() here.


I'm a little surprised at freezing the substitution pattern string. Any particular reason why?


yes; package-all only builds the binaries


This fixes #42


Yes


this is only used by internal users of the client (think of `db.Inc(keys.AllocatorSomethingKey)`). I initially tried to remove it, but it lead to a bunch of awkward call sites (`.Key()`) so this looked more elegant. But if we're worried that we might have client users that legitimately use `RKey` we should probably just pay that price and remove this.


What is " f " ? Give proper name of the instance like : fetchmail_field or field


Not sure it will cause runtime issues or not but the version of mothur we have out has only been tested with Boost version 1.58.0.  Do you have to have to specify a version here? Can you leave this as a generic search and warn if its not 1.58?


Yeah good to have example code in there for the split because will likely
need that later for SOMETHING anyway

On Mon, Oct 5, 2015 at 11:35 AM Brad Erickson <notifications@github.com>
wrote:

> Ah yes. That's true. I was doing it for testing purposes. Let's leave it
> for now? Then drop it later down the line when we figure out how much work
> is required?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mrf/grush/commit/b45bad662e566af93c0688f064bbebd27f45bc61#commitcomment-13598739>
> .
>


For our next release we are removing USE_MPI and USE_COMPRESSION options, can you remove these options?


eliminating the page long repo downloads


I like this, done. I also added a docstring to explain why it sometimes returns None and other times raises an Exception.


Why? I can't see a substantial difference in the order we terminate client & server during emergency situations.

The client is supposed to have finished by this point during normal operation (see line 108).
If the client has not terminated by this point, something must have gone wrong meaning that a) the test run was a failure and b) we'll have to kill both the client & the server either way.

Am I missing something?


Pretty good so far team! Keep up the great work :)


Addresses issue noted in PR #43. This approach is more complete so will close the PR.


If its not contain anything then must remove it.


For convenience, you can use the variable `SUPPORT_LINK` defined at the top of this file. (slightly more concise).


yeah. Suggestions? `AsKey` comes to mind, but maybe not innocuous enough?


What is the significance of this function returning 6 spaces?


[I literally copy-pasted from active_support](https://github.com/rails/rails/blob/32424b6bfbb8bde796d417a907e3e1e9a5d22b76/activesupport/lib/active_support/inflector/methods.rb#L95). I think it's a performance enhancer, since ruby will only ever allocate that string once if it's frozen. Scheems explains it better: http://www.schneems.com/blogs/2015-08-06-patching-rails-performance/ and https://github.com/rails/rails/pull/21057/files


Possibly stray console.log.


I've actually seen this written about at length by many web developers; the consensus seemed to be that changing the color scheme of a website is relatively rare unless you are providing a "skinnable" website (which we aren't), and that working directly with color names is much easier than working with semantic names (i.e. elements should directly reference 'main-green' rather than creating 'header-highlight-color' or something like that.).

In the event that a rare color scheme change does occur, grep is more than adequate.


I think that we should remove it. I'm currently looking for app changes and I noticed that Wallet got moved from /system/priv-app to /system/app (again), so I'm updating the logic, but you should move it from the sources repos, since I'm not able to discard my local changes, and committing would create a mess.


This is not needed for the increment part, do you need it for something else ?


This has been removed at some point after this commit.


Yes, boto3 didn't have an easy way to make an anonymous request to S3.  In order for the CI tests to work without AWS credentials on the testing server, we needed that functionality.


Good call. Fixed.


Looking good. I'm happy to pull this


Could this be done with something like our line from the make file?OBJECTS=$(patsubst %.cpp,%.o,$(wildcard $(addsuffix *.cpp,$(subdirs))))


you've wonderED


Yes, happy to do so.

Nick

> On 5 Oct 2015, at 21:24, Sarah Westcott <notifications@github.com> wrote:
>
> For our next release we are removing USE_MPI and USE_COMPRESSION options, can you remove these options?
>
> —
> Reply to this email directly or view it on GitHub.
>


@cjcoats the declaration of facD was missing from your previous commit.  Please confirm that this commit is correct.


That should be fine. It's pretty flexible.

Nick

> On 5 Oct 2015, at 21:24, Sarah Westcott <notifications@github.com> wrote:
>
> Not sure it will cause runtime issues or not but the version of mothur we have out has only been tested with Boost version 1.58.0. Do you have to have to specify a version here? Can you leave this as a generic search and warn if its not 1.58?
>
> —
> Reply to this email directly or view it on GitHub.
>


by design, this case should already be filtered out. the correct signature is ``may_overlap(ndarray const&) ``


of one of the one of


Хорошие правки, мне нравятся :)


Done. Hopefully it is more readable now.


^^! but that was correct :-)


imagine I instead introduced `UnresolvedKey` and changed the semantics of `Key` - would you still want to change them all to `UnresolvedKey`?

I think **really** they should just be `[]byte` and unexported because they're just used as building blocks of a key (which may be `Key` or `RKey`). Does that sound convincing?


I don't mind using it as long as it's easy to use.
On Mon, Oct 5, 2015 at 12:14 PM DavidBC <notifications@github.com> wrote:

> Not sure if we want to use javax.xml.bind.DatatypeConverter or another
> Base64 converter.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jkdv/bg-scalaris/commit/c9ab2bdcb2d1a50294006f788e1bcd1c540d3587#commitcomment-13599575>
> .
>


@ace2nutzer, you also may want to try this trick :) works on my ws2401 like charm :)


That would not be translatable


Presence of operation here is very misleading. It makes you feel like this code will work:

```scala
def average(s: Iterator[Int]) = s.sum/s.size
```


I am less sure if this is a good idea.  It will force us to prove lots of those iso lemmas "by hand".  I think that making the underlying Equivalence machinery abstract is worth it though.


no need to write @new " from openerp.tools.translate import _ " .
" _ " includes into openerp so you can import from openerp like : --
from openerp import api, fields, models, _


Cool. That's what I thought.


:exclamation: This operator pairing is incorrect. It should be:

| Old | New |
| --- | --- |
| `==` | `==` |
| `!=` | `!=` |
| `>=` | `<` :exclamation:  |
| `<=` | `>` :exclamation:  |
| `>` | `<=` :exclamation:  |
| `<` | `>=` :exclamation:  |


Archive not created!
http://i.hizliresim.com/52RONM.png

Can not find a suitable archive manager.
http://i.hizliresim.com/V6yJQq.png

I think required engrampa-thunar package.


Clarified.


Yes, it can. The CMake developers advise against this way of doing things because it means that during development the build system doesn't identify if files have been added to the tree and thus trigger a rebuild of Makefiles etc. However, all it means is that the developers will have to manually call cmake after adding or removing any headers or source files from the build tree. I've seen this style of CMakeLists.txt done with other projects (e.g. g3log)

Nick

> On 5 Oct 2015, at 21:30, Sarah Westcott <notifications@github.com> wrote:
>
> Could this be done with something like our line from the make file?OBJECTS=$(patsubst %.cpp,%.o,$(wildcard $(addsuffix *.cpp,$(subdirs))))
>
> —
> Reply to this email directly or view it on GitHub.
>


Yes, because ``broadcast`` only applies to scalar.


Not sure if related but after fixing the disconnect immediately issue this crashlog was created soon after.

[code]
Revision: * * *
Date 5:10:2015. Time 15:13
//=====================================================
*** Hardware ***
Processor: AMD A8-5500 APU with Radeon(tm) HD Graphics
Number Of Processors: 4
Physical Memory: 4194303 KB (Available: 4194303 KB)
Commit Charge Limit: 4194303 KB

*** Operation System ***
Windows 8 (Version 6.2, Build 9200)

//=====================================================
Exception code: C0000005 ACCESS_VIOLATION
Fault address:  00756E97 01:00355E97 C:\Users\andrew\Desktop\Playerbots\mangosd.exe

Registers:
EAX:00000000
EBX:F227EB00
ECX:4275E166
EDX:0000052E
ESI:EFBBEA60
EDI:EFBCB380
CS:EIP:0023:00756E97
SS:ESP:002B:07CBECF8  EBP:07CBED04
DS:002B  ES:002B  FS:0053  GS:002B
Flags:00010246

Call stack:
Address   Frame     Function      SourceFile
00756E97  00000000  Loot::GetLootItemsListFor+17
0059635E  00000000  PlayerbotAI::findNearbyCorpse+11E
0057D1B3  00000000  PlayerbotAI::UpdateAI+813
005E27A6  00000000  Player::Update+5D6
0051D105  00000000  Map::Update+175
006C2C19  00000000  MapManager::Update+69
004930E4  00000000  World::Update+2B4
0046770F  00000000  WorldRunnable::run+4F
0047A101  00000000  std::_LaunchPad<std::_Bind<1,void,void (__cdecl*const)(void *),void *> >::_Go+11
6980F33C  00000000  ?_Release@_Pad@std@@QAEXXZ+6C
6909C01D  00000000  __get_tlsindex+6F
6909C001  00000000  __get_tlsindex+53
75737C04  00000000  BaseThreadInitThunk+24
770BB5AF  00000000  RtlInitializeExceptionChain+8F
770BB57A  00000000  RtlInitializeExceptionChain+5A
========================
Local Variables And Parameters

Call stack:
Address   Frame     Function      SourceFile
00756E97  00000000  Loot::GetLootItemsListFor+17
	Local  <user defined> 'player'
	Local  <user defined> 'lootList'
	Local  <user defined> 'lootItem'

0059635E  00000000  PlayerbotAI::findNearbyCorpse+11E
	Local  <user defined> 'corpseList'
	Local  <user defined> 'reaper'
	Local  <user defined> 'corpse_check'
punting on symbol radius
	Local  <user defined> 'lootList'

0057D1B3  00000000  PlayerbotAI::UpdateAI+813
	Local  <user defined> '__formal'
	Local  <user defined> 'loc'
	Local  <user defined> 'ch'
	Local  <user defined> 'targets'
	Local  <user defined> 'targets'
	Local  <user defined> 'emptyPacket'

005E27A6  00000000  Player::Update+5D6
punting on symbol update_diff
punting on symbol p_time
	Local  <user defined> 'iter'
punting on symbol newzone
punting on symbol newarea

0051D105  00000000  Map::Update+175
	Local  <user defined> 't_diff'
	Local  <user defined> 'updater'
	Local  <user defined> 'updater'
	Local  <user defined> 'area'
	Local  <user defined> 'cell'
	Local  <user defined> 'area'
	Local  <user defined> 'cell'
	Local  <user defined> 'st'

006C2C19  00000000  MapManager::Update+69
punting on symbol diff

004930E4  00000000  World::Update+2B4
punting on symbol diff

0046770F  00000000  WorldRunnable::run+4F

0047A101  00000000  std::_LaunchPad<std::_Bind<1,void,void (__cdecl*const)(void *),void *> >::_Go+11

6980F33C  00000000  ?_Release@_Pad@std@@QAEXXZ+6C

6909C01D  00000000  __get_tlsindex+6F

6909C001  00000000  __get_tlsindex+53

75737C04  00000000  BaseThreadInitThunk+24

770BB5AF  00000000  RtlInitializeExceptionChain+8F

770BB57A  00000000  RtlInitializeExceptionChain+5A

========================
Global Variables

[/code]

Posted here because it seems to obviously have to do with Playerbot and looting.


Nope. These two number are from the source code. We have to take them to get the SvIMMORTAL() logic


I need it for unit testing.


В России принято разделять элементы даты косой чертой ([http://www.ifap.ru/library/gost/86012001.pdf](см. пункт 3.5.2)). Тема с точками — европейская.
Но на самом деле, это вкусовщина. Но главное — может заюзать либу для форматирования даты?


Basically it doesn't return unless it is a string. I am overriding the functionality of the hideous error box that pushes around the DOM to display really large letters for a growl notification. The erratic spaces could be made to be only one.


BOOM!! The idea


Fixed.


Give proper name of variable , search not gives you id its gives list of recordset so name should be like : project or projects


couldn't get in. The only thing


you're right, it doesn't really matter.


baymax said blalalala


Anyone care if I move all the artwork, blend's and xcf's to their own "artwork" folder?
If so, my next question would be should it reside at c172p/artwork or c172p/Models/artwork?

@onox, I need to add rotation to the skies on the back edge touchdown and liftoff. I guess I'll have to add a "ground contact" and a rotation animation to make this happen, yes?


Hi Ralf,
Es geht nicht darum was mir gefällt. Ich versuche nur, wenn bereits Standard Attribute einen Anwendungsfall abdecken diese zu verwenden.

ich bin noch immer an dem Punkt, was der gewünschte Effekt ist.
Ich fasse noch mal zusammen:

Ich hatte minsecs und equalmsg als define Parameter eingebaut. So ist das vor Jahren mal im TX Modul implementiert wurden und wurde dann auch bei Fhemduino Modulen übernommen. Zwischenzeitlich wurden dazu vier Standardattribute in FHEM implementiert, mit denen gesteuert werden kann ob und wann Events ausgelöst werden.
Dann kamen wir an den Punkt, dass es im define irgendwie niemand vermuten würde. Ich habe mich auf die Suche nach einer Lösung mit Attributen gemacht und die seit ca. 1 Jahr verfügbaren gefunden.
Die werden bei jedem Dispatch geprüft und berücksichtigt.

Da seit Fhemduino nun Standard Attribute verfügbar sind, habe ich diese eingebaut.

Bislang habe ich noch nicht verstanden, wozu ein Attribut "minsecs" notwendig ist, wenn es doch bereits Attribute gibt, welche das gleiche und auch noch wesentlich flexibler realisieren.
Ich hab kein Problem damit, ein neues Attribut zu verwenden, aber wenn bereits welche für diesen Zweck gibt, dann würde ich doch diese verwenden.

Ein Modul, dass ein Attribut minseqs verwendet habe ich nicht gefunden. Ich schau mir das aber gerne noch mal an, wenn Du mir mitteilst, welches das ist.

Grüße Sidey


Смысл этих двух строк непонятен. Нужно просто написать `var today = new Date();`


I won't be able to get to a machine with Julia v0.3.11 until tomorrow. I'm on 64-bit Ubuntu 14.04. versioninfo() on my current machine:
```
Julia Version 0.3.8
Commit 79599ad (2015-04-30 23:40 UTC)
Platform Info:
  System: Linux (x86_64-linux-gnu)
  CPU: Intel(R) Core(TM) i7 CPU         920  @ 2.67GHz
  WORD_SIZE: 64
  BLAS: libopenblas (NO_LAPACK NO_LAPACKE DYNAMIC_ARCH NO_AFFINITY Nehalem)
  LAPACK: liblapack.so.3
  LIBM: libopenlibm
  LLVM: libLLVM-3.3
```

The relevant lines from Pkg.status():
```
 - PyCall                        1.1.2
 - PyPlot                        2.1.1
```

I have Python 3.4.1 :: Anaconda 2.1.0 (64-bit).

The following works for me. I saved it in a file, and used "include()" on the Julia REPL.

```
using PyPlot

#################
#  Create Data  #
#################
x = randn(1000) # Values
nbins = 50 # Number of bins

##########
#  Plot  #
##########
fig = figure("pyplot_histogram",figsize=(10,10)) # Not strictly required
ax = axes() # Not strictly required
PyPlot.plt[:hist](x,nbins) # Histogram, PyPlot.plt[:hist] required for PyPlot versions v2.1.0 and above, otherwise replace Pyplot.plt[:hist] with Pyplot.plt.hist()

grid("on")
xlabel("X")
ylabel("Y")
title("Histogram")
```

Without the proposed change, I get:
```
ERROR: type PyObject has no field hist
 in include at ./boot.jl:245
 in include_from_node1 at ./loading.jl:128
```

I will try again when I get to the machine with Julia v0.3.11.


AddDataSetTestSupport (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1399 of 1404 tests passed)

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


Объедини инициализацию объекта с записью данных в него. После этого поставь перенос строки.


Decrease line of code : --
from openerp import api , fields, models, tools,  _
from openerp.exceptions import UserError


Тут то же самое.


Ok. I will do that when I will be home again.
But since the wallet is still filled with Android pay, I think we will drop it coming days in from the packages, which will also mean it'll have to be added to the removal lists...


Dont use @api.one, its depreciated


Nice!


don't use @api.one


Yeah, the constants for key parts would make more sense as `[]byte` and we should unexport everything we can.


TeamCity LabkeyTrunk :: Git sqlserver 2014 [Build 5](https://teamcity.labkey.org/viewLog.html?buildId=275832&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:01:16


You seem to be mixing conventions here; above you are referencing by color names, but these refer to specific colors of specific elements.


for consistency, use `-= sizeof("rt=") - 1`


I know the code is generally kept to 80, but I think for consistency, given every other entry, even those that are longer, are on a single line, this one should be too.


I guess so. I'll try without it and see the result.


@wlbragg that's not a good idea because the xcf files are huge and we have a repository just for that: https://github.com/gilbertohasnofb/c172p-detailed-liveries


don't use " id " as variable name


I arrived in San Francisco


remove print statement before push :)


Ok if there no other way. I just noticed it and thought it was strange.


Hello,
Good catch :)
Can you try with this?: (line 565)

parser.add_argument('--ignore', type=str,
                    default="",
                    help="a list of files to be ignored separated by ','\n\

The original default value is wrong and should be set to "".
On my computer it corrects the wrong behavior of setting the default value to list...


wwdc = WWDC


This button does not change on click. Need help here.


Perhaps rewrite this as:
return SellIn <= 0


Yes, I know about that. It was just to keep the logic in sync with Google's. Hangouts has been moved from priv-app to app again, and Earth and Street have been completely removed. Should I completely remove them from the logic?


I'm not sure how to best implement:
1) Follow/Unfollow for a single story
2) Follow/Unfollow for following a user

Should I use the FEED constant? Or a different variable?


That's not the case. Added and spec'd working set admin functionality in b38fb78011fc058716fc38835db0ecd71c6116dc


This didn't actually fix the problem. It would almost be easier to build in a Perl Travis-CI worker and then install the node modules, rather than the other way around


This constructor was public, and e.g. `ae.utils.funopt` was using it to throw `GetOptException`. As such this change broke code. An overload must be added.


Is there any way to test that the id of the first "host" is no longer registered? Or that the registration of `host` is now ID number 2?

I see that you have the second test below to make sure that the state is the opposite if the timeout is > 0 but this part of this test feels a little shaky since you can only test that `host` is registered but not if it was the first `host` or the second `host`.


Since this will never change, there's no need to convert it to virtual DOM elements wholesale - Max's suggestion of converting the entire thing to a single string using m.trust() is probably the best solution.  The entire string will be a single element of the virtual DOM, and when rendered will become real DOM elements.  This should only happen on the first render, at which point the DOM elements are being re-used for subsequent re-renders.


cockroachIcon, not Icons


Decrease line of code import in one line alphabetically like :--

from openerp import fields, models, tools


Strange.  Perhaps you need to update pip?


@dukedorje: @aquabu suggested that we should store all ratings, immutably timestamped, and in the analysis phase consider what to make of someone changing their rating.


It looks like a message that happens to contain a double-quote will break this.


а що не так було?


use "Claim = self.env['crm.claim']" instead of "Claim = self.pool['crm.claim']"


started the following Sunday (started on Sunday seria como que arranca el domingo que viene actual)


Since this is in all likelihood temporary, and since this is a static resource file, I'm not sure why this is split across multiple files in the first place; it seems this would all work nicely in a single file.


Also, it would be nice to have some sort of module-level comment describing what SvgIcons contains.


Oh, good idea.


The place was awesome. There were


Great. I will then add the clang guard.


Line 13 is now valid. I don't know why my previous commit didn't go through...


and go beyond


:+1:  Denn muesma nur bim deploy z'richtiga profil neh und alles passt scho - sehr schön :wink:


The initial idea from Solar was to do that, but in one (or very few) call to crypt_all(). That shouldn't take longer. I'm not sure Solar looked much at the resulting code though. And I haven't looked at it either, in fact not at all. I think that GSoC task was insufficiently planned (by Solar or "us", this is *not* Kai's fault). We have discussed reworking self-tests for some time, but now we merely got a second variant of "not quite it".


Oh, what have I done?


Oh yes you're right, it seems to be happy with the default case :)


Why does this need to change from
#if defined (__APPLE__) || (__MACH__) || (linux) || (__linux) || (__linux__) || (__unix__) || (__unix)


i will look tomorrow; it worked on my osx laptop, just travis seems to be failing. on the positive side: pip install bquery now works :) I have to buy you (and @esc) a fair amount of beers when I meet you!


Your suggestion for xkcd above looks fine. It's simple and efficient. I'd choose normal font weight though (bold font weight was only intended for white text on blue background; with black text on white background it's not needed anymore).

For what if: Why not simply choose a white background, too, with an `opacity` of say 0.4?
![whatif](https://cloud.githubusercontent.com/assets/904570/10291514/7fd75b8c-6baa-11e5-87c6-0dfea4d4cf18.png)
I think it works nicely. Your suggestion of light blue is too "noisy" in my opinion (catches the eye).
For `font-variant` I'd stick with `normal` since allmost all text o what if uses `normal` and caps are not used anywhere, therefore look somehow out of place when used in the title text box (for me it fails to resemble the uppercase font used in the comic itself).


* And add MIT License


<h:panelGrid id="gridRespostas" columns="2">


Genau, meinsch passt das mi de security settings vom uzh ldap? Mit dn patterns und so..? Mer chönds ja ned teste..


update="@this, gridRespostas"


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 743](https://teamcity.labkey.org/viewLog.html?buildId=275834&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 8 (7 new), passed: 0, muted: 1 Build time: 00:12:11
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.Runner$ErrorTest.org.labkey.test.tests.JUnitTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.util.JUnitFooter: <no details avaliable>

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

```


@gilbertohasnofb , I didn't even know that existed. So for sure the the xcf's should go there, but what about the blends. I'm trying to get them out of the main repo/Models area. should they eventially be moved to c172p-detailed-liveries?

Also as a side note, beware, there is a bit of cleanup (duplicate definitions) needed to be done on the material definitions in some of the blend files, especially on my creations. Chalk it up to my learning curve. It finally all clicked and I now know what I was doing wrong. Eventually I hope to eventually get it all cleaned up. Any new additions shouldn't have this issue.


Try to convert SQL Query into serach mehod


We had the most inspiring talks. One of them


Esse ```address``` que você carregou aqui não é um endereço válido. O endereço que você montou na variável, lá em cima, é : ```http://0.0.0.0:8080/v2/search/:platformId/```. Esse endereço não é válido pra fazer requisições. O que você tinha que ter feito ali na variável ```address``` era preencher um valor de platformId e, após a última barra, pôr uma lista para ser buscada.


about `Technology`, `Liberal arts` and how Apple as a company


Adding (~0)/2 causes additional tests to start failing.


LGTM.


TeamCity LabkeyRelease15.2 :: CDS postgres 9.4 [Build 23](https://teamcity.labkey.org/viewLog.html?buildId=275828&buildTypeId=LabkeyRelease152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 22 Build time: 00:35:17
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

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

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

```


I know it is perfectly valid but this realy hurts my eyes :P


i like the git status command :)


A Lighthouse slipped by ;)


[This is the thread](https://groups.google.com/forum/#!msg/elm-discuss/gOU1yxAgWA8/raqfieuLCQAJ) that raised the idea, and it builds off of the [design guidelines](http://package.elm-lang.org/help/design-guidelines#avoid-infix-operators) about infix operators. I thought these operators were cool when I added them, but came to regret them as time went on. This fits into the broader trend of removing syntax in 0.16 that's not necessarily *bad*, but not really necessary either (like record extension/deletion, multi-way if, etc.)


Good catches. Yes, you're right. For the first point we should always keep the most recent checkpoint-worth of history around before LCL, in case it's still accumulating a checkpoint (hasn't queued it yet). For the second point, we need to keep L-64 ledgers for any minimum queued checkpoint number L.

I don't think manually-triggered checkpoints will be a problem though. They're always a subset of a full checkpoint (i.e. <= 64 ledgers) and so if we're keeping a safe 64-ledger-long trailing edge, a manual checkpoint _need_ less than the full checkpoint (take a subset of the trailing edge) and will _cause_ only an extension of the trailing edge a little further back from where the automatic checkpoints leave it.

I think! Feel free to correct my misunderstanding or suggest ways it might be worse. Meantime I'll fix the first two points.


@aw-bib beside the line 1869 above everything else could be applied to `maint-1.1` Even the regression test should work :-)


Always use 'time.Now().UTC()' for all API requests.


Ja i glaube in dem auth.php wo au ir drive isch hemer zimli z'gliicha drin kah und mim seba hets dr thomas doch due zum laufe kriegt... wennt döt wo das ldap züüg gmacht wird n aigeta logger machsch unds denn deploysch söttemers nachr inda log files gseh öbs tued oder nid :wink:


### General comments

I think the approach that you're taking (noting when the number of rows of columns is modified and using that information to recalculate the tool dimensions) is correct and definitely leads to the correct behaviour in the UI.  Good work!

However, I do think that you could make the code clearer.  There's actually no need for the `refs.flags.minus_col`, etc. variables because in every case that you use one of these variables, you immediately call `_updatePagesModifier`, which immediately calls `_updateToolDimensionsModifier` which makes use of the `minus_col`, etc. values, then resets those variables to zero.

What I would do in this case is simply to change (as an example) `_onAddRow` to look like this:

```
    _onAddRow: function(evt) {
      evt.stopPropagation();
      this.refs.rows++;
      this._updatePagesModifier('plus_row');
    },
```

then change `_updatePagesModifier` and `_updateToolDimensionsModifier` like this:

```
    _updatePagesModifier: function(grid_mod) {
      this._setDimensions();
      this._updateToolDimensionsModifier(grid_mod);
      this._createPages();
      this.fire("change");
    },

    ...

    _updateToolDimensionsModifier: function(grid_mod) {
      if (grid_mod === 'minus_col') {
        this.dimensions.width=this.dimensions.width-(this.dimensions.width/(this.refs.cols+1));
      }
      if (grid_mod === 'minus_row') {
        this.dimensions.height=this.dimensions.height-(this.dimensions.height/(this.refs.rows+1));
      }
      if (grid_mod === 'plus_row') {
        this.dimensions.height=this.dimensions.height+(this.dimensions.height/(this.refs.rows-1));
      }
      if (grid_mod === 'plus_col') {
        this.dimensions.width=this.dimensions.width+(this.dimensions.width/(this.refs.cols-1));
      }
      // re-calc bounds
      this.bounds = this._getBoundsPinToNorthWest();
      this._render();
    },
```

This way, you don't need any extra variables in the UI state and you pass the "grid modificaton information" through exactly to where it's needed in the simplest possible way.

I think that if you make those minor changes and sort out the little things listed below, you'll be ready to submit your first pull request to the main Field Papers repo!  That's good and quick work.  Well done.

### Small comments

##### Lines 37-42

Try to respect the indentation style of the surrounding code: the code in this file uses an indent of two and no tabs (never use tabs in source code!).  If you don't use a syntax-aware editor that will keep the indentation straight for you, you should find one that you like and use it (vi, Emacs, WebStorms and other IDEs all do this).

##### Lines 416-428

You introduced a new `_updatePagesModifier` function to deal with your new way of managing the tool dimensions.  However, the old `_updatePages` function is now no longer called anywhere, so you could just replace `_updatePages` with your `_updatePagesModifier`.  How do I know that the old `_updatePages` isn't called anywhere else?  The convention in this code is that function names starting with an underscore are *private*, and don't get called from anything outside the module in which they're defined.  So it would be safe to completely replace `_updatePages` with your `_updatePagesModifier`.  (To be on the safe side, you can just do `grep -r _updatePages` in the `app/assets/javascripts` directory to see where there are any references to `_updatePages` in the other JS source files in the project.)

##### Several places

In a few places, you've added extra whitespace at the end of lines.  This can really confuse Git, since it will show the whitespace changes as differences in the code, even though they obviously don't make any real difference at all!  This is really another editor setup thing, like the indentation issue I mentioned above.


Ha! :smile:


@gilbertohasnofb, thank you so much for the "stainless" texture. I wanted to let you know I used it on the DaSH also. It worked really well. I'll get your credit in the DaSH repo soon. Thanks again


Unfortunately, this doesn't work.
```
Traceback (most recent call last):
  File "/home/piter/software/pyload/module/PluginThread.py", line 187, in run
    pyfile.plugin.preprocessing(self)
  File "/home/piter/.pyload/userplugins/internal/Base.py", line 201, in preprocessing
    return self._process(*args, **kwargs)
  File "/home/piter/.pyload/userplugins/internal/Hoster.py", line 80, in _process
    self.process(self.pyfile)
  File "/home/piter/software/pyload/module/plugins/internal/SimpleHoster.py", line 262, in process
    self.check_info()  #@TODO: Remove in 0.4.10
  File "/home/piter/software/pyload/module/plugins/internal/SimpleHoster.py", line 481, in check_info
    self.check_name_size()
  File "/home/piter/software/pyload/module/plugins/internal/SimpleHoster.py", line 450, in check_name_size
    self.info.update(self.get_info(self.pyfile.url, self.html))
  File "/home/piter/.pyload/userplugins/hoster/OpenloadIo.py", line 40, in get_info
    file_id   = self.info['pattern']['ID']
NameError: global name 'self' is not defined
```


BTW also **/system/framework/com.google.android.camera2.jar** has disappeared.


LGTM. I was about to add support for aarch64 and facing the issue where the high precision timer (cntvct_el0) is not required to be implemented by the platform, which in turn may lead to a SIGILL in some systems. This makes the ports much more straighforward.


It is better to take explicitly 'SecretKey and Region' rather than passing down 'config pointer'


Same as above...


For me as a tech person and also a musician hearing this was really moving, on how we were born with a fundamental life skill, that is programming, but what really should move us is our passion, our heart, for me, my music.

Esta la re-escribiria.


Any idea when this might be fixed for Chrome? I can install with TamperMonkey but continue to get the box that tells me to visit pinboard, click RSS and return to search. I suspect its the user name and key which you are attempting to auto strip and store from Pinboard? Can I hard code these in somewhere?


Este archivo no debería estar en la aplicación para que el usuario configure el bus en función de lo que tiene en la placa?
En las CIAA-NXP y FSL no hay necesidad de cambiarlo, ya que el hardware esta perfectamente definido, pero en una placa como la EDU-CIAA-NXP cada usuario puede conectar lo que desee en los buses I2C.


salah url. jangan dari situ. pakai dari homepage


@Xfurry, what do you say :rofl? Just please do not ask why I added, added - means it is necessary!


Remover a transição do estado S3 -> S2 corrige  o problema da seguinte expressão:

AG( error -> AF !error )

Remover a propriedade START dos estados S2 e S3 resolve o problema da expressão:

AG( start -> AF heat)

Estas alterações fazem com que a maquina se torne consistente para as 3 propriedades que foram propostas.


`const`?  We don't want to be able to rip out/change the api component

Also, this part could *possibly* be under the `/stocks` route (if that is the only place it is needed)


loop removido


The Variate and Tune types could be omitted since this is a Model-based only sampler.  Those types are typically defined for the user interactions that occur with stand-alone samplers.


not sure why this isn't a diff, but what about `/stocks?symbol=GOOG` (query param for symbol)?


yo pondría de ejemplo un chip concreto:
"24c256"
32767
64
6
0xA0
2


s2 e s3 agora tem ¬ start


To get a vector of values from multiple nodes, use unlist:

```julia
d = unlist(model, keys(model, :output))
```


Added open cleveland meetup url


Thanks gilbertohasnofb.
Thanks onox.


Is this extract necessary?


Could alternatively generate normal draws with the ``randn`` function:

```julia
y = v + v.tune.sigma * randn(length(v))
```


A couple notes:

* Use ``relist!(model, y, params)`` to put values into the model to ensure that all dependent nodes/distributions are updated.
* ``y`` being a normal draw assumes the parameters have unconstrained support.  Will eventually need to code the sampler to accommodate parameters with constrained spaces.


Good point. This is why there is a `dataset.noteId` in the following quizzes and examples.


* ``rand(model[d])`` will generate a random draw for a node.
* The random draws will have the same structure as the nodes.  Since the nodes may be a collection of scalars, vectors, arrays, ect.; the ``vcat`` call may not work.
* Use ``unlist`` on each random draw to get a set of vectors to concatenate together that will be comparable to the data vector ``d``, such as

```julia
vcat(map(d -> unlist(model[d], rand(model[d])), keys(model, :output)...)
```


No need to change the model here, could just save the re-listed dictionary of values as a separate variable, say, ``values``.


Fixed in 40c35b1


You got it. We will use `forEach` to replace the loop, and to avoid create new function every iteration by `handleEvent`.


Could use the more recently version of the ``logpdf`` method that allows evaluation at specified values:

```julia
logpdf(model[key], values[key])
```


Would not be needed if making the changes suggested above.


You got it. That's why these elements and the listeners need to be registered as members of a manager in the following examples.


For all your extracts, would it make sense to use the EXTR_SKIP flag in the extract command, so don't overwrite current variables.


A few of the comments added to this comment require more recent versions of the ``rand`` and ``logpdf`` methods.  To get those, merge in the release-0.6 branch at [brian-j-smith/Mamba](https://github.com/brian-j-smith/Mamba.jl).


I'd prefer two functions: `createMetaGlobalWithEngineConfiguration(engineConfiguration:)` and `createNewMetaGlobal`. It pushes a little complexity into the caller(s), but also makes them a little more explicit.


Whyh aren't you using the original function in document.php?


AddDataSetTestSupport (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1404 of 1404 tests passed)

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


Of course one can call into the other.


device's


Oh, I understand.
Thank you.

How about:
```
# PRETTY_ARG_VAR: AC_ARG_VAR with default value
# PRETTY_ARG_VAR(VARIABLE, DESCRIPTION, DEFAULT)
```
?

(If namespace allows I would like to use `AC_ARG_VAR_WITH_DEFAULT_VALUE` as name.)


One of the decisions that needs to be made is if the files you are storing need to be tracked by the documents module.

If so, then should be using the original addNewDocument() function. In this case, all the path work is not necessary; the document module will sort that out when you store it (note that some users use couchdb database instead, which the document module sorts out). And when you collect in other parts of your code, all you need to know is the document id.

If not, then will need to do all the path work, but not enter it into the documents.

Are these files you creating stand alone files or just bits and pieces of stuff that is used by the eye module?


GroundDomainUpdate (mmatts) - i386-Windows-7-VisualStudio-12: OK (1887 of 1896 tests passed)

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


This defeats the goal of test/REQUIRE of having this test run on travis


Interestingly, what I'm seeing is not latency compensation - I don't see the "(client)" version come up at all. Instead, The page suspends for five seconds before going to the postsList page. The Meteor version is 1.2.0.2.


Which of the above categories are actually being used in your module?


Is this addition to list_options used anywhere in your code?


This is an interesting idea, which we have discussed previously. Would be nice to have a mechanism to provide categorized medical problems, meds, etc. This will be a TODO to keep in mind.


GroundDomainUpdate (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1887 of 1896 tests passed)

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


Breakpad handles the writing itself, and the documentation promises that it won't work if the directory doesn't exist.


consider use vector instead, otherwise memory was not freed.


Lets add all your database work to sql/database.sql and register your module by default (see examples there for forms that are already registered), since your module is going to turned on by default(it is simply too cool to not do that).


Add a function MX_CALL to all MX C API calls, that checks the return value equals -1 and  the error message is given my MXGetLastError


use defined constants instead of 19?


PLACEMARKER FOR ONGOING REVIEW.


You got it. That's why to put closure as the strictly pattern everyone should only follow is not a good idea. You need some different mechanism to put the shareable functions and data to reduce duplicated code.


This is used in the PMH subsection, separating medical problems with subtype "eye" out of PMH into POH.  Same will be done for Surgeries specific to eye (as it can be for any surg specialty) and medicines.  For now just medical problems though.


Well it's a issue, but not a collision since it will not collide with others in the runtime.


Good point. With a shareable structure we can do that. Like in the following examples.


Having some cell highlighting in that screenshot would be nice!


test


You got it. The same issue as the comment above.


Stack the dots, please.


@dmoagx Probably we should use [NSThread isMainThread] instead of dispatch_get_current_queue() since it is deprecated since OSX 10.9.


Yes, you got it.


Can this be closed now if the docker image was updated?  Or ?


Yeah debug print, I already got rid of it in another one of the commits.


Hmm.  I'm not convinced - are you saying that processes are always better than threads, no matter what?

At large scale, they have way higher RAM overhead.  They also have a longer startup time.  Finally, it's impossible to pass non-pickleable objects to them.

I think threads are still the main go-to; processes are good at the specific areas where they excel (cpu-bound tasks and tasks that, for any number of reasons, are best run on multiple compute resources).


It's more like `constants`. But yes, it's still an issue.


File a follow-up; I don't think this is quite correct. If an engine is disabled, we're going to stop syncing it… and that means we need to do the same things as if we disconnected the account (e.g., clearing a mirror), but for that engine alone.


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


These are to plug into PRESTO, I'm guessing?


Wait, why do we have an onClick handler for labels?


Maybe we need a way to disable the type checker without modifying any
checked-in files...

On Sun, Oct 4, 2015 at 4:16 PM Tobias Bosch <notifications@github.com>
wrote:

> Sorry! This must have slipped through.... Disabled it during development...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/angular/commit/5b5d31fa9a8871cfdba468fb70bbda0cbb9ce4ff#commitcomment-13582953>
> .
>


`mysql -u root -p < commands.txt`


API symmetry, please! `add()` -> `remove()`, or `addContent()` -> `removeContent()`


Why did this one need to go?


Nice!.


//Remove exited conversion server with ip and port


Guess he figures it won't matter after that?  his call, Rich is the original author


Variable naming again (don't use `m` prefix for local vars)


Please don't keep these kinds of comments, they're useless


Very interesting Jasmine. I like your take on this assignment. Reminds me of an essay assignment I had in English that I submitted with the title :" Why This Essay Assignment Was Unnecessary "


Was joking: Usually the copyright just goes to the current year.


Description about workspace repository


What purpose does `.svg-content-responsive` serve? If I remove this class, the effect appears to be the same. Am I missing something?


I have asked Rich to login and comment on this one.


Basically to enable `if (boolean) return;`. JSCS is working on a more fine-tuned configuration, but until they got it, that's the way to go. See https://github.com/jscs-dev/node-jscs/issues/244


Good you were right.


I couldn't find where the renderable geometry was even used... I noticed the font from the platformer HUD was updating the bounds on every frame, and I thought that was really weird and unnecessary. That's why I removed these lines.


It's auto generated. I forgot to remove it.


These two timestamps exist for two reasons.

The server timestamp is for I-M-S. We want to avoid fetching if it's not changed. If we do that somewhere else, great. (Elsewhere in the state machine, I assume.) If not, we should.

The record timestamp is to ignore unchanged records if i/c indicates a change that might affect m/g. We don't need to do that at all if we're happy re-processing if another write occurs to a different record in meta/.


May be some one want to click on the Label...


@JJHedges are you planning on using pympler or is it just for testing. Ill just have to make a note for in the installation file.


i was actually in the process of fixing bounds update for both me.Font and me.BitmapFont ... you don't think it's necessary ?


You probably wanted to call didDoubleTapImageView: instead of didTapImageView:


This should me mm_button. The [button] shortcode is actually defined in Trestle. The params might be a bit different for mm_button.


first comment


Looks like this whole file is indented with spaces instead of tabs.


trash file


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


The object you pointed out is used by `renderer.drawFont`. It can be optimized, but I feel a better optimization would be memoizing the whole function.


Please change the name of this to be less confusing, and to not have naming conflicts when we use both classes later. I suggest adding a `Facade` to the end of all names, like `Table` -> `TableFacade` and `Label` -> `LabelFacade`. Here's a reference: https://en.wikipedia.org/wiki/Facade_pattern


Typo: boint :-)


Yeah, we need to remove the `z` argument from `me.ColorLayer`. That screwed me up in #756