> Why dir-local.el must is here?

It's not indispensably needed. But may provide some basic settings for developers. Just as
> /* vi:set tabstop=4 shiftwidth=4: */

appended to each source file in this project.


Reviewed SoapUi Project changes, and good with updates.


@magcius Sorry, I lied to you yesterday. You should add `, ${shlibs:Depends}` to get the shared library packages added to the dependencies.


This gives me an idea; thanks for helping to point it out by simplifying that.

I think after this pull request is merged I can look into separating this loop into two loops and removing the MES() endian swapping adjustments (or at the very least calling `swap_elements`) thus vectorizing this.


it's a prototype...

![snimek obrazovky porizeny 2015-05-22 18 14 53](https://cloud.githubusercontent.com/assets/158625/7774834/75a3ebba-00ae-11e5-81a6-b255627cfc21.png)


Suggestions are welcome


Missing java docs, for this function and below functions.


why are the redstone lamps in thier?????


yes i know, but my mobile phone uses sailfishos which is not android based


i have cmake and everything but not the android makefiles


Fantastic commit!


Just FYI, if you don't specify `Section` or `Priority` for a binary package, they gets inherited from the source package above.


The `permissions` element for all PIPs is just a comma-separated list of the permission names without single quotes. Not sure if this is something worth considering as it comes done to the question whether permission names should be allowed to contain single quotes.


fantastic commit!


I am having errors when building the project:
[ivy:resolve] 		::::::::::::::::::::::::::::::::::::::::::::::
[ivy:resolve] 		::          UNRESOLVED DEPENDENCIES         ::
[ivy:resolve] 		::::::::::::::::::::::::::::::::::::::::::::::
[ivy:resolve] 		:: org.apache.hadoop#hadoop-yarn-common;1.0.4: not found
[ivy:resolve] 		:: org.apache.hadoop#hadoop-yarn-client;1.0.4: not found
[ivy:resolve] 		:: org.apache.hadoop#hadoop-yarn-api;1.0.4: not found

Can you help me resolve this?

Thanks.
-Peilong


I pushed the change.
I realized there is no reason to fail here. I just output a generic comment and warn in the log.


Another way of setting up this API would be by using Either, but I don't mind using two Options. We should check that they are not both None though.

On a related note, maybe we can abstract the concept of an API key or UID as an "authorization". Ideally I want to extend our security model to have something more like real "capabilities". What do you think? Either way we would pass in an Authorization object and it would have a way of adding


will do


It's not released yet since I haven't had any time to test it.
Here's a snapshot containing the fix if you want to test it: https://dl.dropboxusercontent.com/u/1897231/perforce.hpi


hello


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

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


Честно говоря, если бы не название, я бы не сразу понял что нарисовано.


I suspect this might still not work when some nominally ingroup taxa (e.g. Tarsius) are intermingled with the outgroup. What probably will work is to go over all internal nodes and pick the one that subtends the largest number of tips without any outgroup taxa in it.


from matplotlib?


I thought so! Gotta lay the foundation for a successful project!


Do you get notified if I leave comments here?

If so, what do you think of this high level design?


it's already working (without undist). There is not so much code from my own :). Do you want to integrate it to the upstream repo when it's done ?


I would add indentation here, but it seems there isn't any in the other API methods either, so maybe it's not necessary.


Tra l'altro per cosa sta 'mf'?


When giving a piece of cake you may not eat it.  When given a piece of cake, you may eat it.  There is a difference.  As the author, we are giving a name of Alice to the method hello.  Given the name Alice "It" should greet the world.  Giving the name Bob, we should greet bob.  We should change the noun if we change the word from giving to given, as the view point changes.


Yeah I hope they won't spam users.

> On 22 May 2015, at 17:02, Lloyd Watkin <notifications@github.com> wrote:
>
> Would be nice to say that you can unsubscribe at any time (which is true given they are using mailchimp?)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/project-isizwe/wifi-chat/commit/ca8f00ed07f8bc9ebafbf7e39d7e7cacbfc3c549#commitcomment-11329577>.
>


yeah, wrong import. Hahaha... but it was deleted in the next commit.


The FAB height should be constant (it may even be defined in the 3rd party library), so you could just cache the value the first time it's non-zero.


Add one empty line here.


Looks like I do. I love the high level design, your experience as a software engineer is showing and admirable. I’m checking out C libraries for system profiling now.
On May 22, 2015, at 12:25 PM, ankeesler <notifications@github.com<mailto:notifications@github.com>> wrote:

Do you get notified if I leave comments here?

If so, what do you think of this high level design?

—
Reply to this email directly or view it on GitHub<https://github.com/alhart2015/CPUMonitor/commit/5b18df027c28030cea0daab8ed95f85590db8e11#commitcomment-11331150>.


please avoid commenting code


@DaanHoogland great, master is building on debian again :)

I do see this as an issue that gives extra work to maintainers, and would likely happen again in the future.. i propose this:

https://github.com/apache/cloudstack/pull/291

I took the chance to include a couple more fixes to deb packaging since i was looking into this :)


use " instead of '


typo?


same issue with commenting code


we should go with v1 at the base url


loading isn't changed so I would use const.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1145 of 1145 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)


loadMore isn't changed so I would use const instead of let.


we should avoid this kind of relative imports (not compatible with python 3)


I think this is not related to Fortran.
If you think this should be part of ctags, please submit another PR.
Mixing many ideas/goodies into one commit may make reviewers crasy.

BTW, M-x c-guess also helps you.


I think it's more of a javabeans convention. And I don't really view ErrorRecorder as a javabean; it's more of a service.


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


I would not indent this.


Hi @ORNis
I have just wrote a binary that export undistorted images for the v0.8.1
https://github.com/openMVG/openMVG/issues/305
=> So:
1. just export in the files containing the camera settings (remove todo: undist),
2. call the binary ExportUndistortedImages
3. call MVSTEXTURING and tell it to use the directory with the undistorted images ;-)

I think we can integrate this functionality in the coming v0.8.1.
We just have to show to people that it works (tiny screenshots to share and attract):
openMVG->PMVS->PoissonMeshing->MVSTexturing


we should use capital based naming for constant values


> You may change the tab or whitespace. If you want to do is, could you provide another PR?
These changes are nothing to do with "recognize Fortran generic type bound procedures".

It is the "	{ "non_overridable", KEYWORD_non_overridable }," line in this commit that stucks out and messes up the alignment. I can split this into smaller ones.


The null should be on a separate line.


I can try, but I sort of doubt it.  without putting the rescue in, react seems to catch it internally and just move on without reporting any error, so I imagine if I re-raise the exception the same thing will happen.


please use different lines for each of the imports
eg:
from . import product
from . import country
from . import currency


using sublime text 3, already install the package for powershell via the command B in a mac.  When I run I get this error:
[Errno 2] No such file or directory: 'powershell.exe'
[cmd: ['powershell.exe', '-noprofile', '-file', '/Users/john/Desktop/javascript/powershell/DrSpace.ps1']]
[dir: /Users/john/Desktop/javascript/powershell]
[path: /usr/bin:/bin:/usr/sbin:/sbin]
[Finished]

Wanting to get powershell working on sublime text 3 but not sure what to do now?  Any help is appreciated.


@sagartalla, have you noticed failing tests on TravisCI?
https://travis-ci.org/concord-consortium/rigse/jobs/63660395
Is this expected and you're working on that or it's more like randomly failing tests?
One is `student_requires_password_reset` which seems justified in context of this change, but other is `admin_work_with_interactives`.


Actually I just thought I was being funny by re-using the word 'fault'. I guess I wasn't. :-)


In the upcoming version you have to check that the view as a valid pose & intrinsics:

```
if (!sfm_data.IsPoseAndIntrinsicDefined(view))
        continue;
// Valid view, you can ask a pose & intrinsic data
const Pose3 pose = sfm_data.GetPoseOrDie(view);
Intrinsics::const_iterator iterIntrinsic = sfm_data.GetIntrinsics().find(view->id_intrinsic);
```


const Mat34 P


const int


IRC-Log:

    <holiman> ah, here we go:#echo "6001" | ./disasm
	 6001\00
	 0      PUSH1  => 01\00
	<CF_> ah yes
	 So no way to get it via RPC or javascript console?
	<holiman> CF_: not that I know of, but that does not mean much .. :)
	... 54 minutes later ...
	<ZeroGox> GitHub-go-develop => core/vm, rpc: added disasm to `ext_` RPC => https://github.com/ethereum/go-ethereum/commit/7381be8edb3bec412d31a97977174cf52eed8094 ; https://zerogox.com/bot/aggregator

Thanks, that was quick!


The comments for this note that the verbose_name was chosen to match the backend (although in this case it's defined by social, not by our own backend). Should check to see that this doesn't break anything; if not, fix the comment?


avoid commenting out code


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

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


Could you release this to crates.io? The version there still appears to be 0.0.3.


Use const instead of let.


upcoming version will require new using namespace

```
using namespace openMVG::cameras;
using namespace openMVG::geometry;
using namespace openMVG::sfm;
```


It works in wildfly. I don't know if it's reliable in any java ee environment, though. I'm not sure what the rules are for jax-ws & cdi injection. But since we don't use any other kind of java ee servers at Cru, I think it's ok to let it slide for now.


rgerhrj


We use it all over the tree so why would we stop now?


In line 288, you probably mean "--" instead of "?". This guarantee (i.e., that instantiations do not change the semantics of the predicate, or in other words, that the predicate call and an instantiation of a subterm of one of its arguments can be exchanged without changing the meaning of the conjunction) is called *steadfastness*, and we call such a predicate *steadfast*. You can thus say for example:

> Developers should ensure that predicates are *steadfast* with respect to output arguments. This means
> that instantiation of output arguments at call time ... etc.


Actually I am wondering whether it shoudn't be a bit more complex. Don't redis session expire?


It would be better to use just one style of assertion. So change remove this and change all should asserts to expect.


TeamCity xBIM :: Xbim Geometry :: Release [Build 62](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1669&buildTypeId=XBIM_XbimGeometry_Release) outcome was **FAILURE**
Summary: Compilation error: Xbim.Geometry.Engine\Xbim.Geometry.Engine.vcxproj (new) Build time: 01:48:20


Ah, you're right, the activities use social whereas the other apps are free to have a different `verbose_name`; will see if this does break anything or not (I hadn't re-authed RunKeeper after changing it)


:+1: for testing!


Ah, I think you need to activate a maven profile - either arq-wildfly-managed or arq-wildfly-remote. (I usually do the latter for intellij, and the former for command line builds.)


Is this supposed to be `pjoin(xdg, 'runtime')`? I don't use a desktop Linux machine anymore, but looking over this made me curious about the lack of symmetry with the rest.


@Knorkebrot I don't understand. ExecutableFolder calls Executable, which cleans the path. I don't see this adding anything. Can you be more specific in what it adds?


Unfortunately, the SOAP spec requires a `500` even for client faults.


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


I think it should stay what it was here, but it's just my opinion. Sounds weird that way.


Seems ok to me :)


It's getting there but it's still a little glitchy. Sometimes the output will appear to be doubled for a split second, at the moment it is finished processing and then go back to normal. Also, the energy level data still only syncs when there is a recipe processing. It should be syncing whenever the stored energy value changes.


No, `$XDG_RUNTIME_DIR` is a separate directory (on my machine, `/run/user/1000`), so we need a jupyter folder within it. Adding a 'runtime' subdirectory would be a bit redundant.

(`_HOME` was a mistake that is fixed to `_DIR` in a recent PR)


You may add -Wmissing-prototypes

By the way: My branch compiles without warning with these settings:
https://github.com/daschuer/nemo/blob/nemo_312_merge_nemo_base/configure.ac#L282


I realize that.  I am just telling you there are compilers which do not handle it.


There are contexts where a cert is built but not verified (during signing), so I didn't want to pay the penalty in that case.  Typically you verify only once, so I think it's OK.


It's my mistake.


It's actually not assignable to `Servlet`. But it is a webservice impl, and this is legal (in a java ee environment at least).


The white list is case sensitive, so what happens if you write ['html'], instead of ['HTML']? (I guess it shouldn't make any difference, though.)

I am not too familiar with grunt-ucss, as I am not the author. But in theory it shouldn't make any difference.

Perhaps you could send me the configuration where you have specified the whitelist (I assume that would be your Gruntfile) e.g. using gist.github.com? Also, if you run ucss towards a local/internal site, please include some html and css needed to reproduce the error. If you could also include the final report that you get, I will look into this as soon as I can.


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


The CoreRegistry is currently just forwarding the put and get calls to the available Context instance.

This makes it possible to refactor things step by step till the CoreRegistry gets no longer used.

To give a bit more technical background: The CoreRegistry gets informed about the currently relevant Context via a CoreRegistry#setContext method. During the loading of the engine, the CoreRegistry is forwarding to the engine  private context. During the game selection UI the CoreRegistry is refering to a child context of the engine which lifes until the game starts loading. For the duration of the loading of the game another context gets created that lifes till the end of the game.


what the commit


`getSyncByPath()` worked on my tests, but it would fail if the `localPath` received as parameter ends with a folder separator.


Thanks; copy/paste strikes again.


@hjelmn @ggouaillardet hey guys - shouldn't all these Java fixes come to 1.8.6? If so, can someone submit that for us?


Because SocketFlower is incompatible with PElikePC


This needs to be written back to sunnysideup. My bad!


*RocketShower


This looks good and works.  Two quick questions:

1.  Should we perhaps move migraiton-bean.xml to migration-bean.xml.example and add migration-bean.xml to gitignore so that developers who use this code can leave a good locally configured migration-bean.xml in the project directory?

2.  Once I merge this, do you want to make the referenced release, or should I?


rax retry tests


> doesn't say "Pithos" on the bar anymore :(

I think in GNOME shell the "Pithos" menu is meant to be a window title of sorts.

> is this GNOME-ish? Do / will other apps have dropdowns on the header line that?

I'm not sure what it looks like in GNOME proper but it looks ugly in Ubuntu. But that's probably 99% Ubuntu's fault. I like my mockup, still wondering where the info button went? But it's not really much of a loss as I never really used it anyway.


this is very useful for huge datasets, thanks!

On 05/22/2015 05:59 PM, Martin K. Scherer wrote:
> [coor/data/npreader] added progressbars for len/dim determination (requested by @gph82)
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/markovmodel/PyEMMA/commit/e477de3ed590078f5469931be59d4f152d0aab11
>

--
Dr. Guillermo Pérez-Hernández
Freie Universität Berlin
Institute for Mathematics
Arnimallee 6
D-14195 Berlin
tel       0049 30 838 75776


Awesome. We'll wait on tests and merge. Thanks again!


If you don't mind, I'd like to ask a question about implementation details here. I'm still learning Rust, and I'm trying to learn proper design choices :) Anyway, I was wondering why it is necessary to create another buffer here, which you copy manually. Could you not just implement this by constructing a new owned `CString`, and then passing that to `f`? e.g.,

```rust
    fn with_nix_path<T, F>(&self, f: F) -> Result<T>
            where F: FnOnce(&CStr) -> T {
        let cstring = try!(CString::new(self));
        Ok(f(&cstring))
    }
```
This would take care of all the copying, and avoid using unsafe functions with unclear guarantees.


:+1:  Está muy bien


Is there a typo somewhere? How can a `JetType` instance be `instanceof JetClass`?


<!-- PMABOT:TAB -->
This commit is using tab character for indentation instead of spaces, what is mandated by phpMyAdmin. Please check our [Developer guidelines](http://wiki.phpmyadmin.net/pma/Developer_guidelines#Indentation) for more information.

Offending files: js/messages.php


No problem! We all started somewhere. In any case, see the code for 'submit_link' (and others); if it returns a name or other similar identifier that could be used to construct the url, then go ahead and return that. Otherwise, a well-documented hash of pertinent return values is acceptable.

As to the unit tests, take a look in the t folder. There are plenty of examples for you to use. All network communication is mocked (again, see the other tests in the client test).


I'll add a bulk 'optimize imports' commit


good start to adding all the alt tags, we can add more when we are given guidance


das File bruucht allgemein n'überarbeitig :stuck_out_tongue_closed_eyes:  I hann das prod gnennt willsja denn bim prod target automatisch usgfüert wird - oder? grundsätzlich hanni vor allem z'localdeploy gmacht und nur wella dass dr rest no so isch wie vorchär :stuck_out_tongue_closed_eyes:


Export loop on Macs also fixed


With each Attribute we must be found a unique 'code'

Not value here


Sorry to make you crazy. I don't mean to do it. It's my first time to collaborate with other developers in Github, I've something to learn yet. I'll adjust my work flow to fit these rules.


PortForwarder is only for external use (it only enables mapping and manages port of some clients like webui). libtorrent::session manages own port mappings itself (and, as I remember, it forwards utp port too).
If in the future we will need something to use udp ports with this, we modify this class. I did not do this immediately so my changes are not too huge. The main thing here that I did brought this task (port forwarding) outside of the class Session to improve the design of our code.


For Corporate Town, check out Oversight AI -- it uses a prompt that allows the user to physically click on the target card, instead of the usual multi-choice prompt.  The trick is using a `:choices` that doesn't involve `(req)`, but instead uses a map with a `:req` in it:

`:choices {:req #(and (= (:type %) "ICE") (not (:rezzed %)))}`

triggers a "selection prompt" where the user can click on any card that matches the `:req: filter, in this case, ICE that isn't rezzed. Similary, Snatch and Grab has

`:choices {:req #(has? % :subtype "Connection")}`

to select a Connection. So you could probably use a choices like

`:choices {:req #(= (:type %) "Resource")}`

to get the right effect.


As I wrote above, libtorrent::session manages its port mapping itself. I think that is where it is provided. At least, in the preceding code, we did not process it.


stop_*pnp() also delete all created mappings.


Have you found a solution to do it work with all browser?


Avoid exposing sandbox/test urls (may create some security related issues). Let's try to use env variables instead.


This is used below. We should define it to "" or something, or remove the line that uses it, making sure it doesn't break anything.


The same thing (if you mean bittorrent port) .


No; the `entitySanitizer` is used in `DefaultWebErrorRecorder`. It's possible that I should move the parameter sanitizers there, as well, to be consistent. But I think that would be hard enough to do that I don't think it's worth it.


This revision is clean.


with this new implementation imx implementation will always have sync enabled whereas this call was likely to disable it. I am not sure it is an issue, yet it seems to be a slight change...


It'd be true if the user did not define his own ParameterSanitizer; it'd be false if he did.


So basically uploads are working*. I need to add all the other piece fields to the upload template form and I haven't even touched tags.

But if you want to make a piece that only has a title and composer and upload files to attach to it, this does that!


Again, I would like to say "thank you for submitting the PR."
I need your help to improve Fortran parser because I donn't know Fortran.

o.k. How about providing smaller PR?
I think this PR was too large to submit at once.
I will wait your retyring:-P


forward declare


Since these have default values, consumers that don't care shouldn't notice.


Why inherit QObject? For automatic deletion?


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (632 of 632 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-5.29%25-green.svg)


Indeed. From my memory of our earlier experiments, this is desirable but maybe @smallint could comment on this


thanks Pierre. I will try to finish that this WE. it's already tested (just by copying distorded images) on a dataset. FYI, I wrote a similar program for Apero/MicMac but it's not public. I'm not always satisfy by PMVS.


It occurs to me that this may be needed in many cases where we are using dispatch_sync() also.  E.g. in editString() while the user it editing text, the GC will be blocked.  If the EDT ever gets labeled as an aggressive allocator, it will get locked too.  We should probably wrap calls to dispatch_sync in CN1_YIELD_THREAD/CN1_RESUME_THREAD to prevent this.  What do you think?


one line


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1145 of 1145 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)


let makes dictionaries immutable.


Although, we would need to make sure we only did this when called off the main thread.


Why doesn't `resumed` and `ratioLimit` get copied from `in`?


@cloos, what would specifically you like me to address in the documentation? Given that `--all` doesn't yet exist, I figured the existing "Note that changing this setting does not affect existing attachments, only the new ones that C<sbin/rt-externalize-attachments> hasn't seen yet." describes what happens pretty well.


>PortForwarder(); too.

What do it mean?
I didn't understand it (and other similar comments).


"my first"

On 22 May 2015 at 18:25, Silverweed <notifications@github.com> wrote:

> Tra l'altro per cosa sta 'mf'?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Hamcha/mfemu/commit/3e8d1a1bb3a01c5e5b76e3d39ee5471b8aeca611#commitcomment-11331156>
> .
>


This doesn't need initialization.


お疲れ様です！m(_ _)m
以下について気になりました。理由などはありますか？

* /admin/CalendarのCalendarは小文字でも良いのではないかと思います。
* HTMLのIDにアッパーキャメルケースとチェインケースの混合が見られます。


Not relevant to this PR, but is there any reason we're using a script instead of go generate?

Also, this GOPATH manipulation is unnecessary as godep will run the go tool for you with the correct GOPATH. `godep go run ...`


I'm wondering why this is an object when you can just close over a variable with a lot of (truly) private functions.


Ok. Here I went too far.
Fixed.


If you put this in separate PR, you might be able to convince kimrhh to include this in the coming release :)


LGTM


At 3:10 you mention that git-town can be configured to merge the main development branch instead of rebasing it. I know this was something you envisioned early on, but this isn't something we currently support.


Commit also thins the body's div's down. They seemed too wide, but that is a subjective observation. Feel free to discuss.


What is `f.value &&` guarding against? Looking into why I can't have `"0"` as a choice key for a `multipleRadio` widget on a boolean field. My database, sqlite, returns the number `0` for false boolean values. I'm currently working around by casting the database value to string before binding to my form, which this code also does.


Maybe we should include this in OE-lite/core instead.  Although it might not be the most actively used component, it is a rather core toolchain thingy.


merged


Do we need to worry about `options.query` being set _and_ having a `?` in the url?


`incompletedSize()`


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1145 of 1145 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)


These literals are for 80-bit x87 reals, that's why I guarded this block with a `static if`. It's not related to VS 2015.


@kardianos Of course.
If you split the file part off, the directory part [still contains the trailing slash][1], that can be a problem when you're trying to get the parent directory of `ExecutableFolder()`'s result, see this [play][2]. And since [`filepath.Dir()`][3] is essentially the same as [`filepath.Split()`][4] + `Clean()`, so I thought "why take the long path?" :)

[1]: https://godoc.org/path/filepath#Split
[2]: https://play.golang.org/p/bliU5WKYTn
[3]: http://golang.org/src/path/filepath/path.go?s=12741:12769#L438
[4]: http://golang.org/src/path/filepath/path.go?s=5143:5185#L177


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


Both outcomes are wrong.
max_ts_signal_y needs a look
max_ts_signal_time gives the correct times, but meshed in a vector of 0s


>libtorrent::session manages own port mappings itself

I am not sure about that. I'll leave as a last thing after I finish the review.


OK, I'll provide smaller PR.


We need to write to a stream instead of a buffer, so that we can enforce back-pressure.. basically, so we don't use a lot of RAM if the sending server is faster than the receiving service.


All the calls to this are on the EDT so I don't see the case where this will happen. Edit string uses a sleep loop in invokeAndBlock


I think you have an extra `:`


Same here


Hi Jason

For DSS as primary case,  why dont we brk() down to lower max address just as we do chunk_dalloc_mmap()?

thanks
--sk


I was using readdir incorrectly, this was a workaround. You fixed it.


I'm curious why you need to queryProp again when you already have the password?


Thanks, I'll remove them from the enum then.


Please delete this, security fixes shouldn't be sent publically.


Double check it's working from the command line  "sftp" -- the SSH2 module talking to itself is more forgiving than the command line SFTP program.


Oh. In fact, these operators are not used. I'll delete them.


This is a Singleton (as are the others I commented).
If you don't define the `PortForwarder();` as private a user can do:

```c++
//somewhere outside PortForwarder class

PortForwarder forwarder;
forwarder.appPort(36674);
```


Ok. Done.


File that gets written


why sleep?


Nice one @chochos!


sure, feel free to change this


Inside `void Java_com_codename1_impl_ios_IOSImplementation_editStringAtImpl` we have

~~~~
dispatch_sync(dispatch_get_main_queue(), ^{
//... do all of the editing stuff

});
~~~~

Whatever thread this is running on, whether it is the EDT or not, will be blocked for an extended period of time, but its `threadActive` flag will remain `true`.  In the mark phase of the GC, it will set the `blockedByGC` flag on this thread, and then wait for the thread to respond by setting its `threadActive` flag to false.  This means that the GC won't be able to proceed until the dispatch_async() call is finished - and this could be an extended period of time.

Is my logic wrong?


I agree it makes sense. However, I wanted to have something working asap because we're still not meeting our 1.0 goal and this should have significant impact.
And we can get back to refactoring you're proposing once we will meet our goals.
Does it sound ok?


System time is about 60x faster, but I think something is still off. The two graphs look different.


I believe you had a bug here...you should have used Piece and not PieceType.  Since PieceType doesn't distinguish color and you will only clear half of the array.


```java
SslException ex = new SslException(t.getMessage());
ex.initCause(t.getCause());
```
?


@thatch45 @jacksontj im unsure about this one


Great tool!
Do you know a way to change the Apple logo during boot screen als well?


```b``` is of type **bool** - ```m_value``` is of type TriStateBool::ValueType.
Theoretically, the type **bool** can be implemented as you want (not necessarily true == 1 and false == 0).


Fixes #70


I think this is the best approach. Another argument for it is that you do not want to compile in extra code when targeting Windows w/o Cygwin. And guessing based on environment variables at run time would perhaps be too complicated and violate the Least Surprise Principle.


sorry it's a typo.. it was fixed in later commits.


@andrefyg the asset was added on this commit.


@domoritz  Quick reminder: when you push this kind of changes directly to master, please also update https://github.com/uwdata/voyager accordingly.  :)


Wir sollten eine Klasse items machen und von der wearables, useable, usw erben lassen und dann zB ausrüstung bzw so Rüstungen, Helme usw davon erben lassen


I disagree that printing the strack trace is worse than not reporting anything.

Sure, in the perfect world the message portion of the exception is accurate, succinct, and enough to get the user unblocked. But all too often it isn't, especially when you are calling extensions.

It's far better to give information that helps users and us understand why something has failed. As I had in the commit message, the error handling as it is today does not let the user understand what the problem is.

This one actually came from one of the users, and he spent a couple of hours trying this and that in the absence of diagnostic information. That's far more frustrating than the annoyance someone get from seeing a stack trace.


This issue breaks the auto_install. It reports the error as below:
......
Uploading yum-epel     [0.6.0]
FATAL: Cookbook file attributes/default.rb has a ruby syntax error:
FATAL: /home/vagrant/chef-bcpc/cookbooks/bcpc-hadoop/attributes/default.rb:44: syntax error, unexpected '=', expecting ']'
FATAL: default["bcpc"]["hadoop"]["balancer"]["bandwidth]" = 1048576
FATAL:                                                                               ^
......


thanks, man... that was some tricky shit but very very useful.


Do you have thoughts on how the nsync-listener should be behaving smarter? Should it be the one to retry on conflict?

cc @luan @leochu


Why are these special cases? Maps, slices, and structs that contain them can't be keys in a map. Everything else should be value assignable. Pointers (and structs that contain them) are the only tricky part, but the key is memory address and not the value of the memory address. So shallow copying those pointers is actually the right thing to do.


```c++
TriStateBool b = TriStateBool::Undefined;
if (b) {} // we use TriStateBool::operator bool() here.
if (b != TriStateBool::Undefined) {} // we use TriStateBool::operator ValueType() here
// because TriStateBool::Undefined is of type TriStateBool::ValueType
// we cast TriStateBool variable to TriStateBool::ValueType
```


In my testing it didn't break anything but sure I can make it the empty string.


```
recipe['cdo-mysql::server']
...
recipe['cdo-mysql::client']
```

You can simplify this to just:

```
...
recipe['cdo-mysql']
```

Because that rule installs both client and server.


Will do. Btw, Travis was failing because of this.
On May 22, 2015 10:32 AM, "Kanit Wongsuphasawat" <notifications@github.com>
wrote:

> @domoritz <https://github.com/domoritz> Quick reminder: when you push
> this kind of changes directly to master, please also update
> https://github.com/uwdata/voyager accordingly. :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uwdata/polestar/commit/75cfa36c74efec415412fc4792729b3150732974#commitcomment-11332195>
> .
>


Great screencast! Love how everything gets shown off.

One interesting tidbit that comes to my mind is git ship is built on the idea of squash-merging, which isn't the standard (but I think should be) for merging in feature branches. Any thoughts if we should explain that shipping is squash merging, before using that command instead of in the middle as it is currently.

Along that line would be interesting to check if teams that are against squash-merging would use git-ship if it could be configured to merge instead of squash-merge. I don't want to build that until someone wants it but just an idea for the future if we run into users against squash-merging.


It's probably best not to install this on VMs. This installs a monitoring tool (cpu, memory, disk usage, etc.) that costs money per instance deployed. Developer instances don't need to be monitored.


Oh... we're still building the graph here. This'll be compiled. Nevermind... *facepalm*


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.160](http://teamcity.monogame.net/viewLog.html?buildId=7544&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:19:30


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


![image](https://cloud.githubusercontent.com/assets/9274905/7776115/5d63771e-006d-11e5-8e4a-d8993f32b4a6.png)

My method above

![image](https://cloud.githubusercontent.com/assets/9274905/7776147/93de5520-006d-11e5-9a46-de9a6c878590.png)

Original Method looks more sparse

I looked at some output, the idea is right, but something is off. It's recording sequential values both as max, sometimes.

![image](https://cloud.githubusercontent.com/assets/9274905/7776255/5f377ef4-006e-11e5-823d-319ad052d2d1.png)

"a" is my lazy encoding for the new method's output, the "max_ts_signal_time" is the original method's output.


What is `context.login = login` for in this context?


maybe run gofmt on it as well?


Just noticed that there is a missing changelog entry...


Are you sure you can safely cast `xmlChar*` to `const char *`. maybe use a static_cast.


Equivalent to `callback(false)`, right?


This commit is related to manifoldjs/ManifoldJS#60


According to this SO article: https://stackoverflow.com/questions/5369770/bool-to-int-conversion
the C++ Standard says that false->0 and true->1 when converting to int.
(enum are int)


@vilinfield  please can you format this page a bit, thanks!


將 build 的目錄給 static 出去不會有安全性問題啊，還是你還有要把什麼不能給別人放在 build ?


You'd just have to negate it for it to return the correct response. Of course, that wouldn't be atomic, but I don't believe it matters here
```scala
override def cancel(): Boolean = !wasCancelled.getAndSet(true)
```


I'd like to see a comment/code that links the event_type to the incoming JSON data.


Yes, I'll use it in other places (or will use in the future, if we consider the current commit as of now).
Scoped enums (enum classes in c++11), in fact, a very handy thing unlike regular enums, which are an anachronism. But while we will not use the updated c++, it is necessary so to be perverted.


Understood. Thanks.


ok


Probably because I screwed up - this is fixed in trunk - but there's also something odd going on with macros containing newlines. Needs fixing properly, but I would love a set of tests first.


I just tested this commit and the leaks seem to be gone. I sign off on this.


Nitpicker!


Please use https://help.github.com/articles/using-git-rebase/ instead.


I guess importance was weight of each instenace? maybe change to weight instead? ot inst_weight Sometimes importance was mistaken it for feature importance


But with a ``use`` method as an abstract method of the ``Model``, we could hide away everything that's related to the building of the final graph from taking the model's graph and applying the loss function.

An idea would be to separate ``train_output`` and ``use`` (dropout being an example). ``train_output`` would be used to build the ``learn`` function whilst the ``use`` would be used for the tests.

In any cases, these function could be overridden for more funky models.


See my previous comment. The point is to be able to directly access the model output. without having to know the model. At least as a default behavior.


There is probably no reason to install Postfix on the VM at all.


classic!


Looks alright but not my expertise. Wido can you look at this?

Op vr 22 mei 2015 om 18:28 schreef Rafael da Fonseca <
notifications@github.com>:

> @DaanHoogland <https://github.com/DaanHoogland> great, master is building
> on debian again :)
>
> I do see this as an issue that gives extra work to maintainers, and would
> likely happen again in the future.. i propose this:
>
> #291 <https://github.com/apache/cloudstack/pull/291>
>
> I took the chance to include a couple more fixes to deb packaging since i
> was looking into this :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/cloudstack/commit/1c408dec37a278c97c1b40da0bdbca437d7a564c#commitcomment-11331207>
> .
>


Looks good to me, for the second argument of RegistryNewSession you can probably just do &registry.HTTPRequestFactory{} inline (no need to define a variable for this). To test you could modify the [stub of RegistryNewSession here]( https://github.com/cloudfoundry-incubator/garden-linux/blob/a4a8bdcdfb45ae47efa3b49c42da2af003b26df1/old/repository_fetcher/repository_provider_test.go#L40) to record what was passed as its arguments (just like it currently records its third argument) and add an assertion or two underneath [line 121](https://github.com/cloudfoundry-incubator/garden-linux/blob/a4a8bdcdfb45ae47efa3b49c42da2af003b26df1/old/repository_fetcher/repository_provider_test.go#L121) along the lines of Expect(receivedAuthConfig).NotTo(BeNil())


Can be optimized further by removing assignments, but I'm going to leave it as is for readability.

```
local_max<-function(ts_signal1_test,time_vector=0){
  n <- length(ts_signal1_test)
  max_ts_signal_time <- vector(length=n)
  max_ts_signal_y<- vector(length=n)
  CURRENT<-(ts_signal1_test[2:n] - ts_signal1_test[1:(n-1)])
  PAST<- (c(0,CURRENT[-(n-1)]) >= 0)
  CURRENT <- (CURRENT < 0)
  return(list((((1:n) * (time_vector==0)) + time_vector)[which(CURRENT & PAST)],ts_signal1_test[max_ts_signal_time]))
}
```


@westurner You're welcome! Glad this helps.


Bad Practice, Anakod deserves some big respect from you guys, respect the creator and common decency..


I've been tracking down a similar issue today where flows on a deleted slice never seem to get purged.  I haven't gotten very far in investigating it, but at least in principle, it seems like this fix applies to that.  Let me know if you want me to build a snapshot of master and run it through my tests.


We'll need to update the links to the API documentation and overview page.


"INVALID_STATE_ERR" -> InvalidStateError.  Same change below.


I think there should be $filename ;)


What is "it" here?  currentTime?


Grammar: Should this say "is in the <running> state" instead of just "is <running> state".


>  But that's probably 99% Ubuntu's fault.

To be fair one of the features of headerbars is that they are completely controlled by the application and its toolkit. So the theme changes how it looks for better or worse. =\

>  I like my mockup, still wondering where the info button went?

It was useless so I just added it as a right click item in the station list.


Welp: this change [made all Travis jobs run on Python 2.7](https://github.com/zopefoundation/ZEO/pull/12#issuecomment-104660810).


@Mumfrey It looks like this with redirecting enabled now: https://gist.github.com/Minecrell/0c8ec135683b43fe52ae (no more line breaks inbetween)
And with redirecting disabled it looks as usual: https://gist.github.com/Minecrell/8722fe48ab86e544820a


On account QString I agree. Fixed.
But std::string is a
```c++
typedef basic_string<char, char_traits<char>, allocator<char> >
string;
```
Can for fun to try to forward declare it.


Fix #1


Fix #2


Ok. Fixed.


akprind.looveya@gmail.com

081916004743


:thumbsup:


`strings.h` != `cstring`
`strings.h` is mostly deprecated (except for `str(n)casecmp` and `ffs`).


IIRC the original reason for pinning 4.0.0a1 here and elsewhere was to get a version that supports Python 3 (buildout defaults to final releases only, so it'd get something like 3.x.y).  IIRC we didn't know about the `zdaemon = >= 4.0.0a1` syntax back then.

I think the exact pin should be removed altogether.


Expand default blacklist with .bdmv and .mpls as they are parts of a bluray file structure.


Whoops, that should just be a return statement


You're right. Don't bother for std::string: https://stackoverflow.com/questions/4801411/forward-declaration-of-stdwstring


Fixed.


`return resp;`


1.1 stable where?!


*retification : suppression du fichier modgrille et non modcases !


Not going to happen. RetroArch 1.2 will get released instead.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.161](http://teamcity.monogame.net/viewLog.html?buildId=7548&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:22:15


@Generation0 Sau nais!


Fixed. Thank you.


I would prefer if we stopped using too many short (single dash) option on the command line, if you're not really opposed to that.


as far as I can see, this only removes the warnings, but does not changed the text


nit: ``scopes is not None``. Yoda would be proud, though.


@thomasgohard - I love the filtering to check for the ID, I am not sure we want to mix-up both the CSS ID with what this ID is suppose to provide. I know for sake of brevity it works, but I assume many sites and developers sometimes leverage the weight of CSS ID, so there is a need a separate ID to reduce any chances of  developers having to pick and choose or duplicate CSS since it know contains 2 potential uses. Maybe something like "data-id"?


No, this should have been `cmd.extend(youtube_extra_option)` to append one list (instead of a single element) to another and eliminate the preceding `for`.


You've gone so far with default values - why not give `mTotCoins`, `mParentCloseTime`, and `mCloseFlags` default values of 0?  That's nearly always what they're initialized with...


With this change, this has become essentially just `unsafe_getindex`. We might as well remove the `bit` prefix then, and the redundant definition in multidimensional.jl.


What do you want me to do? Fix this personally or will you send in another commit?


you can make `mImmutable` a const member, since the immutability status of a `Ledger` never changes after its construction...


Yet, apparently, only for this. Although I can't remember exactly.


This is also the same as `unsafe_setindex!` except that it doesn't return `B`.


We should never lock the code, only the data. Can you rework the code to only lock the code? If you still want to serialize the code path use empty channels (`chan struct{}`) idiom to synchronize.


Same here.


Use the old method. It is faster/cleaner. No need to wrap it in QLatin1String().


Fixed.


Maybe put a comment here about the mutability being a consequence of those deprecated methods above?


yes, that would be quicker and simpler then


The `make` command will look for a file called "Makefile" so this can be just `make html`.


yes, please explain. :)


Pode tirar esses 4 espaços no início do parágrafo, também? Eles aparecem na renderização final (que está tratando o texto como pre-formatted)


got it .. will fix it ..thanks .


This is wrong, but the current keyword is also not completely right.


vfuzz controls the warnings -- that much is ok
What does the rest do ?


This second commit is just a move, right?

If that's so, there's very little to complain about here...


Why you do this john
Works for single case but trying to make it work for multiple options then display similar to decision so I can finish testing a real battle


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1145 of 1145 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-69.81%25-green.svg)


Does it make more sense to change the function definition to: "func parseNumber(cmd []rune) (n, i int, err error)" ?


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1650 of 1650 tests passed)

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


Right, just moving


Much API, such intentional.


Yes, or maybe
`func parseNumber(cmd []rune) (int, []rune, error)` and just have it return the rest of cmd. Then the caller doesn't have to deal with reslicing. Does the caller ever care about `i` besides to reslice `cmd`?


Had you ever add a block? Your code is 200% wrong.


AddTorrentParams used for adding torrent to session (not for resuming) so it does not have such fields.


we'll see if ...400 can be achieved ;)


The `Symbol()` object can still have `Symbol.prototype` and not be `instanceof Symbol` - I'm not sure why that would be a problem?


Fixed.


Should revert this line; xsd:dateTime should there.


This looks like random changes by 1 (2 in one case) for all parameters ...


Should revert this line; xsd:dateTime should stay.


Where is the 99 restriction coming from?


To we really have to wait for the unbind to proceed with the callback?


saw approval from cmeiklejohn
at https://github.com/basho/riak_kv/commit/8eb9c1a32d573cbd403ee2236fc6083ff1f3ff69


merging basho/riak_kv/bugfix/jrd/epmd = 8eb9c1a3 into borshop-integration-1119-bugfix/jrd/epmd


basho/riak_kv/bugfix/jrd/epmd = 8eb9c1a3 merged ok, testing candidate = 0311c548


Yes you're right. This was basically a proof of concept. As for `strncpy` it does not append a null character if I understand the doc correctly.

Anyway I just pushed a version with the string constructor taking a char pointer and a size.


@kohsuke some real errors was missed after some commits, and now instead of investigating everybody just adding stacktraces. Some commits before lost stderr was restored. What git plugin and core version was? What was the root of error for your particular case? Also why not just print message 'error fetching remote repo' and re-throw existed `ex` that stacktrace will be printed by core because it not `AbortException`?

Btw @jglick commented very well https://github.com/jenkinsci/git-plugin/pull/303#discussion_r29375743


On May 22 2015, Ismael Mejia wrote:
> I prefer to keep the full url since it gives more information to someone
> who is debugging in the other functions, even if youtube-dl can download
> just with the ids. Additionally since the argument is called url I expect
> to have a full url there, so I think it is good to keep it.

OK, I don't have that strong opinion on having the links being just the
video id's. It is only that the concatenation of strings make my eyes
automatically remind me of an expensive computation.

Yes, I know that the program is in Python and that we should strive for
readability. :)  It is just a reflex and hard to avoid. :)

Regards,

--
Rogério Brito : rbrito@{ime.usp.br,gmail.com} : GPG key 4096R/BCFCAAAA
http://cynic.cc/blog/ : github.com/rbrito : profiles.google.com/rbrito
DebianQA: http://qa.debian.org/developer.php?login=rbrito%40ime.usp.br


I didn't understand here. I'm locking `p.listener`.


@timothysc can you elaborate on how this is breaking in your automation?

`hack/ginkgo-e2e.sh` (hopefully) sets --testContext to a reasonable value, so this test should be able to access `cluster/kubectl.sh` without issue. Is that having problems? Or is `cluster/kubectl.sh` unable to find the kubectl binary?


It was just a shot in the dark to see if I could get the Heroku deployment to work.  At the time I didn't know that it was a Java7 issue.


I hope there will be tests soon ;)


That's a shame.


This is quite hard to read.


I didn't understand a word, sorry :) Is it okay to have `setListener` method with lock there?


Also, it's not mentioned in the comment here, but by using kubectl.sh, you can set `KUBECTL_PATH` in the environment and it will use that path rather than trying to guess which kubectl to use.


wait, why is uri still here? Is it actually being used - if so I don't see it.


Need to make the form hidden and add it to the DOM in order for firefox to submit it.


What happens when the divisor is 0?

Also, perhaps a test where the remainder is 0?

Also, what happens if the remainder or divisor are < 0?


mmmmmm


yggu


llşl


You still need this piece.


le doh


>Use the old method. It is faster/cleaner. No need to wrap it in QLatin1String().

Explain, please.
As far as I know, the constructor QString(const char *) makes a deep copy of char array data, but QLatin1String just uses it.


looks good


Not sure which one is better: this.get('language.content'); or Em.get(Mercury, 'wiki.language.content');


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 15](https://teamcity.labkey.org/viewLog.html?buildId=251485&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 24 Build time: 00:33:35
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyColorAxisSelector: <no details avaliable>

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

```


We have a work-around atm, but the path over-ride will work.

@rrati ^


Ah, Sync, you sad bastard.


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


sould -> should


User can't do this.
We already have private constructor, so compiler doesn't create default one.


I'm not in favor of switching to PEP-8 naming conventions for methods, actually.  I think there are enough big Python modules that use CamelCase for methods that I don't think it's a terribly strong expectation in the Python community.  And I think we do gain from having similarity with the C++ layer where it's not unpythonic to do so (if nothing else, it will make it much easier to trace a shim interface to its implementation when it happens in C++, and I think that's important).


This one LGTM.


OK, I am sorry for breaking your automation. Let's fix this and understand what broke so we can avoid having it happen again.


Comment.


Plase don't mix tab and spaces.


_to_ resize


I suggest you use the git client to avoid too much commits


Why GPL?  Can you make it Apache2 license instead?  That would make it easier to allow this project accepted by the saltstack community.


It is quite funny that it includes definition for Inkscape window dimensions...


Agree. But we can still change ValueType definition for some reason...
So, safer.


What I am saying is you should not be holding the lock when you are calling `net.ListenTCP` which might be invoking a blocking system call. So all you need to do is lock only when you are setting `p.listener` and if you are solving only a data race that will solve the problem.


I'm not convinced that shims are the right way to go in the long run (I think there's much more agreement on how we want our Python interfaces to look than how to get there); even if we do stay with Swig, I think it may be simpler and more maintainable to write a lot of `%extend` and `%feature("shadow")` blocks than to write a completely separate Python interface.  But the argument that this is a good way to transition slowly is compelling; assuming we don't burn all the effort we've allocated towards this in writing incredibly deep shims, we could then change the implementation without users noticing.


one line


Here too you are holding the lock while you are invoking `p.listener.Close`


`has_many :myclasses` should work bruh. believe in the magic


Writing distinct shims does not stop us removing them later and replacing them with improved C++ interface generation. Think of the shim as a public API. You can replace them any time when a better interface turns up lower down. Also if the shims are requiring astropy and pandas and the like then it may be cleaner to keep that out of the deep interface stuff.


Let's say that the other macros are justified somewhat. This is not. Use the functions directly for better readability.


Remove these from send/receive calls.


Great analogy. I definitely think I got beat in the first quarter. Now I'm trying to pick up points in garbage time to soften the blow. :+1:


I'd actually say it's not necessary for C++ code to be able to access pure-Python objects we put in `Exposure`; I still believe that if C++ code needs to make use of a class, that is almost always a sufficient reason to implement that class in C++.  My goal here is to make sure nothing is implemented in C++ only because it needs to be held and persisted by `Exposure`, and to meet requirement that it's only necessary for the C++ `Exposure` to hold them as opaque pointers with some way to persist and unpersist them.


The `it` part of `digit` was too long?


I'm not convinced that it's hard to go from `coadd_inputs` to `coaddInputs`. I do think we need to start running flake8 on our python code as part of the build. Once we've cleaned it up (including disabling items that we disagree with) we can in theory trigger a CI failure if some new code adds a warning.


This should not be changed here because it breaks the `dashboard_use_pegasus` setting used by developers. This setting should be overridden in your role file instead, and only if needed.


Hey there, the main reason for doing it this way is to avoid an allocation. There are lots of reasons that this is preferable, for example calling functions after a fork()


thank you. It works !


Plus these macros "disable" QtCreator's ability to "Follow Symbol Under Cursor" and show the function signature in the tooltip.


You could use `not ...is_dig` here.


close #16


And it looks from its use that `is_dig` should be on `Char` instead.


The MainActivuty will not start
Pretty confused


camera geom has been mentioned to me as something that might be an easier sell in Python (for people wanting to use data from their instruments) rather than insisting on C++. I have not looked into this though.


Fixed.


default=2. need to fix.


Doesn't the entering native flag solve that issue?
We never call dispatch_sync for anything that will take a lot of time as far as I can tell.


When we do not specify its version to 1.0.16, it break the auto_install as below:

.....
Uploading cobblerd     [0.3.0]
ERROR: Cookbook cobblerd depends on cookbooks which are not currently
ERROR: being uploaded and cannot be found on the server.
ERROR: The missing cookbook(s) are: 'poise' version '~> 1.0'
......


@gronnbeck hvorfor fikk jeg med alt du allerede har gjort? kanskje det var en pull fra et annet repo for jeg pullet før jeg endre source.


See libtorrent code for example:
```c++
        upnp* session_impl::start_upnp()
	{
		INVARIANT_CHECK;

		if (m_upnp) return m_upnp.get();

		// the upnp constructor may fail and call the callbacks
		upnp* u = new (std::nothrow) upnp(m_io_service
			, m_half_open
			, m_listen_interface.address()
			, m_settings.user_agent
			, boost::bind(&session_impl::on_port_mapping
				, this, _1, _2, _3, _4, 1)
			, boost::bind(&session_impl::on_port_map_log
				, this, _1, 1)
			, m_settings.upnp_ignore_nonrouters);

		if (u == 0) return 0;

		m_upnp = u;

		m_upnp->discover_device();
		if (m_listen_interface.port() > 0 || ssl_listen_port() > 0)
		{
			remap_tcp_ports(2, m_listen_interface.port(), ssl_listen_port());
		}
		if (m_udp_socket.is_open())
		{
			m_udp_mapping[1] = m_upnp->add_mapping(upnp::udp
				, m_listen_interface.port(), m_listen_interface.port());
		}
		return u;
	}
```


Sé que es un config que raramente vamos a cambiar... pero Confiig plz! :)


Ok... modifico


First one was a small test method to only check the getter methods from the KeyBoardMovement class, which succeeds. In the tests for the drawablepixmap however, I keep getting errors when I try to work with a pixmap. Pushed this so Jorai could take a look at it.


Currently the e2e.test binary that is produced has hard coded relative paths in it.. so if you were to run it from a different location other then kubernetes/test/e2e it will fail on a number of tests.


It would probably be better forward port 80 from the guest as the primary port because this is the port of the Varnish server. Varnish is an caching HTTP server that improves performance on our production servers (a LOT) but also acts to combine the two applications (Dashboard and Pegasus) into a single site (this behavior is what the `dashboard_use_pegasus` setting is meant to emulate on developer's boxes where Varnish isn't installed). Ideally all developers would access the site through the Varnish server because (in addition to caching and unification) it contains rules that control language selection, cookie setting, etc. so accessing the app servers directly can result in writing code that won't actually work when deployed.

So, I think you probably want something like this...

```
config.vm.network "forwarded_port", guest: 80, host: 3000
config.vm.network "forwarded_port", guest: 8080, host: 8080
config.vm.network "forwarded_port", guest: 8081, host: 8081
```

... because this will forward Varnish as port 3000 as well as the private ports for the app instances (which can be useful in specific debugging scenarios).

For vagrant instances, instead of connecting and running `bin/dashboard-server` and `bin/pegasus-server` you should probably use something like:

```
rerun -p '**/*' -d '/home/ubuntu/development' "sudo service dashboard restart && sudo service pegasus restart && sudo service varnish restart"
```


This feels like one abstraction too far.  Haskell's syndrome, noun: when the type signature support lets you abstract everything.


I won't hold up the merge for it, but these tests should be renamed (and might actually overlap with other tests now, I'm not sure).


This probably should be GREEN too


Show me how :) I see only ugly ways.


Oi Douglas! Dá uma olhada nesse comentário: https://github.com/carolmartinussi/guia-de-servicos/commit/e6185b7ebb54630b4e135c24d8b1d390553943dc#commitcomment-11332749

A formatação do campo descricao respeita espaços em branco, então esses espaços no início do parágrafo renderizam.

Pode tirar e atualizar o PR?


I think you should pass all the query_* flags here.


I should add that I'd be perfectly happy using lowercase and underscores for Python attributes and properties; most of these will be new (as we transform getters and setters to properties), so there's less old code to transition, and I do find it weirder that they're camelCase in our current codebase (even for C++ struct data members).  Maybe I'm more okay with `verbNoun` than `nounNoun`?

I think camelCase methods with lower_case properties and attributes is what GalSim does.


This reads strangely; you think there's trickery, but there isn't.  Why not make `doDeleted` take `lastTimestamp` as a parameter and bind through?  Also, s/do/upload or post/.


Consider s/places/guids/.


Yeah, CameraGeom was the major thing I was thinking of where we had a hard decision to make about whether to make it pure-Python or not, and needing to put it in `Exposure` required us to put at least some parts of it in C++.


Couldn't you include the date check into the query?

    query = DBSession.query(Task).filter(
        and_(
            Task.lock_date!=None,
            Task.lock_date < datetime.datetime.utcnow() - EXPIRATION_DELTA))


GuiIconProvider inherits IconProvider.
So, you have to use multiple inheritance. I prefer not to do that unless absolutely necessary. (If all ancestors, except one is just completely abstract classes, then it is safe).


some tests failed:
failure: http://buildbot.bos1/builders/test-riak_kv/builds/2039


remove_std_function (lefticus) - i386-Windows-8.1-VisualStudio-14: Build Failed

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


Extract to utilities.


Why change this behavior? The previous behavior ensured that the bundle was always updated whenever the chef-client ran, while the new behavior only installs the bundle the first time chef is invoked. This is likely to break other parts of the continuous integration system.


I agree. Need to have better tests that show understanding of the $mod operator for various numerical types and that it correctly handles floating point precision issues.

Will you add any examples and documentation?


J'ai modifié les fichiers et utilisé le module **set** mais cela ne fonctionne pas comme attendu.
J'ai placé le tag set juste après le extend+load mais je n'arrive pas à utiliser la variable affectée dans un block…

Par contre si je place le tag set dans un block ça fonctionne pour tout le bloc.

Du coup je fais quoi ?


@Mingcongbai I save it as Inkscape SVG, and it certainly contains some rubbish.


remove_std_function (lefticus) - Win64-Windows-8.1-VisualStudio-14: Build Failed

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


Also change the identifier here to correct English...


remove_std_function (lefticus) - i386-Windows-8.1-VisualStudio-12: Build Failed

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


Although it executes faster, `rake db:schema:load` has been problematic in practice. Other than performance was there a reason to change this?


Presumably this code can be deleted.


That didn't work for me, however this did:

https://gist.github.com/DavidNielsen/33a5a1078e7e21757bf7


remove_std_function (lefticus) - Win64-Windows-8.1-VisualStudio-12: Build Failed

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


I see that temp8 was split into 2 products, but they have the exact same description.


LGTM


Did you test this? Using HTTP.Request on a brightcove url called the URLService, hence urllib.


Can we talk about our failure modes here?

We're using flags rather than timestamps, so if we upload a batch that sticks, we'll never upload them again.  If all the posts succeed, we set the upload timestamp below.  It's strange to me that we do that for both deletions and modifications.  Is there an edge case that we're missing here?  I can't think of one, but we should say that setting our timestamp multiple times will interact with I-U-S.


Also, there is an import for set_trace left in.


Je ne sais pas si j'ai bien compris. Si tu places le `{% set %}` dans le `{% block content %}`, ça fonctionne ? Si oui, c'est ça qu'il faut faire !


> It was useless so I just added it as a right click item in the station list.

I agree it's pretty much useless. Tucked away in a right click menu sounds good to me.


if we are exiting on obs_test being unavailable wouldn't the default exception message be good enough on its own?


OK, we’re moving kind of fast at the moment. I’ll fix that.

> On May 22, 2015, at 2:40 PM, Matthew Hanson <notifications@github.com> wrote:
>
> I see that temp8 was split into 2 products, but they have the exact same description.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/gipit/gips/commit/bbb5af91cd62c164c4430f484eb56fffaf70158f#commitcomment-11333329>.
>


If you are referring to the use of getPackageDir then please do not delete it. This is what loads the camera-specific config override files.


and the product names in the product groups don't match the actual products.

Best thing to do in these cases is to create a branch, or use your own repository, then issue a pull request.


Oui mais du coup pour les block breacrumb et title par exemple c'est inutile, autant utiliser with…


I agree


I apologize. I will take care of it.

> On May 22, 2015, at 2:42 PM, Matthew Hanson <notifications@github.com> wrote:
>
> and the product names in the product groups don't match the actual products.
>
> Best thing to do in these cases is to create a branch, or use your own repository, then issue a pull request.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/gipit/gips/commit/bbb5af91cd62c164c4430f484eb56fffaf70158f#commitcomment-11333361>.
>


I'm talking about lines 479 and 480. The RunTimeError raising. `getPackageDir` will have already raised an exception.


is ```MultiPoint``` used anywhere?


I guess `tensor.isnan(previous_step).sum() + tensor.isinf(previous_step).sum()` works too, maybe that's even faster?


This is by default.


Ah, je viens de comprendre ! :D Ok, bah laisse comme ça du coup :)


Doesn't seem like I wrote the code to read it.


I only just noticed that you removed these notices.  I'm not sure whether that's in compliance with the GPL—not that I care—but it's wrong imho to just leave “Magit Project Developers” in place, since neither Florian nor me have ever been part of the “Magit Project”.

I'd appreciate if you added these notices back again.


Du coup là aussi tu peux laisser comme ça ;)


we should call this `BackupConfig`.


Hi!  First off, thank you so much for making this an option, I really appreciate it.  This mod is honestly the only one I've been able to find where I really like the way vampires are handled.

Unfortunately, now that we've actually gone and tried adding it to the server, we've found a conflict with one of the other mods that's making the server crash (it works on single player, however).  The problem seems to be world gen in general with the custommobspawner, which we need for mo creatures to work.  The village alterations were causing a crash, so we disabled them.  Now the blood altar is causing the crash.  I would ask if it was possible to disable all of that generation completely, but I don't know if mo creatures might be conflicting with the way the mobs were coded in this mod?

Essentially, I would like to know if there is any way to get this working with mo creatures.  As long as the mobs, vampirism effect, and crafting for things such as the blood bottles still works, I'm fine with whatever has to be disabled.  Future versions that utilize what was disabled more fully can be implemented on future servers.

Thanks again, and really, no worries if what I'm asking is too difficult, I fully understand and can just use your awesome mod on the next server I play. ^__^ I'm really sorry if I've at all been a pain. ;__;


Ok. Fixed.


Technically this part of the patch is in the wrong place as you are also adding an import that does not look to be used.


why moved files to up level directory?


@borshop: retry


- __error__: [include/chaiscript/dispatchkit/callable_traits.hpp:70](../blob/df724b5c33561ca1bb975c09d9d7f794c3389442/include/chaiscript/dispatchkit/callable_traits.hpp#L70) syntax error (syntaxError)


I forgot that node 0.8 doesn't support caret notation..


Do we want to change the prefix for this config? Since this is a storage related config, this might be fine, but simply looking at the config name, I'd expect it to be in the StorageMangerConfig.


Par contre ici tu peux utiliser `{% set %}` au lieu des `with site_name=app.site.litteral_name` :)


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


I do not understand the value of 16 here. Why stop at 16 instead of using 32, 64 or even higher number. At longer time controls it may be useful to allow movecount-based pruning at higher depths.
This "magic number" of 16 is actually also one of the dimensions of the FutilityMoveCounts array, so it is also a code style issue.


Can you rename to AssertAllDefinitionsHaveTokens?


@KunalKene1797 update repos :v


so can we view the images even from github, like this: https://github.com/valleyrain-org/www/blob/master/content/2015-03-22.md


Sure, cc5106df59f64b78745ec3204c101c92fb4090c3


While you're at it, server.js needs to be updated too.


saw approval from cmeiklejohn
at https://github.com/basho/riak_kv/commit/8eb9c1a32d573cbd403ee2236fc6083ff1f3ff69


merging basho/riak_kv/bugfix/jrd/epmd = 8eb9c1a3 into borshop-integration-1119-bugfix/jrd/epmd


basho/riak_kv/bugfix/jrd/epmd = 8eb9c1a3 merged ok, testing candidate = 40ce5a76


Please format the code correctly in the future.


"We never call dispatch_sync for anything that will take a lot of time as far as I can tell."

Oh. My mistake.  I thought the code inside dispatch_sync() blocked until editing was done.  In fact it just sets things up but doesn't wait for editing to complete.   Not a big issue I guess.

"Doesn't the entering native flag solve that issue?"

I think it does the opposite.  It actually prevents the thread from going "inactive", so as long as the native allocation mode is on, the GC will be waiting for that thread during its mark phase.

Just things to be conscious of.  If native code is in "native allocation" mode or is blocked on something, the GC will be unable to complete a cycle.   If in addition another thread is marked as an "aggressive" allocator during this time, that thread would be blocked until the GC cycle completes.   If the "native allocation" mode is very short lived or the thread is only blocked (e.g. on dispatch_sync) for a very short period, this probably isn't an issue.


I'm not sure what happened there.  I'll remove it.


Bumped into this commit from Bing when I was looking for a way to get node-gyp to run with VS2015. This is godsend. Can someone port this to the main TooTallNate/node-gyp repo?


okay, cool!


Should be "binded", not "bounded".


Did that also build gstreamer correctly? Your patch builds fine but it's then still building for x86_64 which will then fail when gstreamer tries to consume the lib.


Are you try to set `participant_id` to current_participant.id, is that why you can't use `params.permit(:id, :participant_id, :action_type)`?  Thanks!


Hmm weird I just did a build through gstreamer without an issue..

Oddness to investigate.


I think this can cause future issues. Is better use the verbose way `organisms/header` etc.


You mean the commit message?
Forgot about that edit...


No, you changed `Februari` to `February` but i would like:
`februari|February` to `february|February`


Maybe `@proposal.approved?` would be cleaner?


Ah ok, didn't see that. Will do :)


rax retry tests


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1656 of 1656 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)


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@gsouvik

I've merged feature_devsupport branch together with feature_mobile together into dev and then back to our respective branches. So do a pull before pushing new code. Thanks!


Why is tf being set to nil?


I will send you the commit in some time, I didn't know that extend function (shame on  me).


After this commit, entering something like `https://www.reddit.com/user/deimorz` into a username entry (such as approved submitter) will not do anything, but also not give an error.

It will just show `submitting...` for a second, then return to normal


I think there should be Exception thrown just to make test die silently


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I'm OK with this, but lets avoid wildcard imports. This makes static analyzers mad. :)


also it should be less then uid.max


OK. Better naming.


* Bug number - fixes Watson crash 1167564
* Customer scenario - winmdobj project contains generated code (WCF or the like) with unimplemented partial methods. Compiler tries to emit additional winmdobj specific data to PDB for all member symbols and crashes over missing method defs
* Affects both C# and VB
* Fix description - skip over unimplemented partial methods when gathering data to report to the extra PDB payload. Added a Debug.Assert that should be triggered if such data contains other cases of unemittable definitions, even in non-winmdobj case.
* Testing done - new unit-tests scenarios. Existing unit tests do extra validation because of the assert.
@jaredpar FYI


I didn't know how to make the other imports be included, I tried with only import _print but it didn't get some of the six imports.


```
Have you heard of this new mod?
  Which one?
It's a strange mod...
  Just tell me already!
People call it OpenComptuers, I have no idea what it does.
  Strange...
```


I see. Good point. I agree that the code does nothing. If the existing exception in getPackageDir suffices (and I hope it does) then those lines can be deleted. If not, then try/catch is necessary (or improving the message from getPackageDir).


Thanks, for PR, please update marked lines


Ok


If it works, I'm fine with removing it. :) One fewer complaint from static analyzers, then. :)


How do I add myself to the list of members?


The docs about that QString constructor don't say that it performs a deep copy. They say it about other constructors.


Yeah, that's how it "worked" before this commit as well. Those boxes don't display errors correctly for anything except valid usernames that don't exist.


What is the type of this? Could this be...something?


If you're on ES 1.2+ you should do this for a couple reasons: 1. It's faster. 2. You may prefer to disable ES scripts for security reasons and `field_value_factor` doesn't use scripting.


We can probably change that back - Travis and I modified it because the prints were all skipping, but it ended up not fixing that, and we just forgot to put it back-Mike P.

Date: Wed, 20 May 2015 12:33:01 -0700
From: notifications@github.com
To: Marlin@noreply.github.com
Subject: Re: [Marlin] accelerations and bed level locations changed (2297ce8)

What is the logic behind defining X_MAX_POS and Y_MAX_POS as larger than they actually are? This can cause the machine to crash, even motors don't skip any steps.

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


@bjornbm FYI, don't break the haddocks, travis is watching them too now.


11/10 would randomly enter keys again.


Yes, you're right. I forgot about that detail.


Oops :flushed:

Sorry for making the assumption there, had never really noticed before.

That said, I'd love if those boxes supported reddit URLs :wink:


@kbenne

I am starting to rough out the code for my ground temperatures project and I'd like to get your input on it.

The current thought is to have a vector of shared pointers to each instance of the ground temp models within the ground temp manager. This way (hopefully) the objects that use the ground temp models will be able use a generalized call that the manager can polymorphically call on the appropriate model and return the temperature to the caller.

In the caller object, that might look something like this:

```C++
// In simulation model - data structure
class groundHX {
  std::shared_ptr< GroundTemperatureModel > groundTempModel;
}

// In simulation model - get input
// read ground temp model name
myGTMname = alphas[3]
this->groundTempModel = GroundTemperatures.getGroundTempInstance(myGTMname);

// In simulation model - calculation
this->groundTempModel.getCurrentGroundTemp(time, depth);
```

I feel like I'm still a OO newb, so if you have any thoughts, I'd love to here them.

@myoldmopar FYI


Could you make a pull request out of this? It's not as hacky as you call it, it's actually quite good. It might need some commenting though ;)


Yes indeed, and I'm pretty sure scripting is disabled.  Will do!


Hi,

just tried   (Just did a Pkg.update and a Pkg.clone to get Packing);
shows this issue:
```
alain@alain-linux:~/src/Julia/GLVisualize.jl/test$ Julia4 test_text.jl
ERROR: LoadError: LoadError: LoadError: UndefVarError: resize_nocopy! not defined
 in anonymous at /home/alain/.julia/v0.4/GLVisualize/src/display/renderloop.jl:64
 in include at ./boot.jl:252
 in include_from_node1 at loading.jl:134
 in include_all at /home/alain/.julia/v0.4/GLVisualize/src/utils.jl:4
 in include at ./boot.jl:252
 in include_from_node1 at ./loading.jl:134
 in reload_path at ./loading.jl:158
 in _require at ./loading.jl:70
 in require at ./loading.jl:53
 in require at /home/alain/.julia/v0.4/Requires/src/require.jl:8
 in include at ./boot.jl:252
 in include_from_node1 at loading.jl:134
 in process_options at ./client.jl:310
 in _start at ./client.jl:409
while loading /home/alain/.julia/v0.4/GLVisualize/src/display/renderloop.jl, in expression starting on line 62
while loading /home/alain/.julia/v0.4/GLVisualize/src/GLVisualize.jl, in expression starting on line 38
while loading /home/alain/src/Julia/GLVisualize.jl/test/test_text.jl, in expression starting on line 1

```


@OrKoN  Lolz, just trying out Visual Studio code and obviously it has no formatter :)


If we change the definition and nobody comes to check this it still won't be safer.
Anyway, I don't really care here. I doubt it will have a performance penalty. I wrote about this for compact code.


Indeed. Thanks.


Sorry, glabstraction and abstractgpuarray needs to be updated as well... If
abstractgpuarray doesnt update, try deleting and cloning it... I renamed it
an put an .jl in the end

On Fri, 22 May 2015 21:08 AlainLich <notifications@github.com> wrote:

> Hi,
>
> just tried (Just did a Pkg.update and a Pkg.clone to get Packing);
> shows this issue:
>
> alain@alain-linux:~/src/Julia/GLVisualize.jl/test$ Julia4 test_text.jl
> ERROR: LoadError: LoadError: LoadError: UndefVarError: resize_nocopy! not defined
>  in anonymous at /home/alain/.julia/v0.4/GLVisualize/src/display/renderloop.jl:64
>  in include at ./boot.jl:252
>  in include_from_node1 at loading.jl:134
>  in include_all at /home/alain/.julia/v0.4/GLVisualize/src/utils.jl:4
>  in include at ./boot.jl:252
>  in include_from_node1 at ./loading.jl:134
>  in reload_path at ./loading.jl:158
>  in _require at ./loading.jl:70
>  in require at ./loading.jl:53
>  in require at /home/alain/.julia/v0.4/Requires/src/require.jl:8
>  in include at ./boot.jl:252
>  in include_from_node1 at loading.jl:134
>  in process_options at ./client.jl:310
>  in _start at ./client.jl:409
> while loading /home/alain/.julia/v0.4/GLVisualize/src/display/renderloop.jl, in expression starting on line 62
> while loading /home/alain/.julia/v0.4/GLVisualize/src/GLVisualize.jl, in expression starting on line 38
> while loading /home/alain/src/Julia/GLVisualize.jl/test/test_text.jl, in expression starting on line 1
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JuliaGL/GLVisualize.jl/commit/379ba22d13c654d216663ea5819c0b844df1b927#commitcomment-11333750>
> .
>


Thanks Tom!


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/2040


I was unsure whether to do a static import here. For the test cases, it makes sense since it documents recommend usage. Inside the implementation, I felt citing the source of the method made sense to clearly denote where the method is located.


IconProvider is also usable in the nox version by the gui. It doesn't need QObject.
However, you decide if you'll remove it or not. I am just stating that it isn't really needed to add extra code here.


I want that.


@arthurlogilab @jpic that one is going to please you :)


please keep same coding style - opening brace on the separate line


## This is a markdown file


Add tests for file backup store (at least the shardSize call)


I think we should raise `Not_available` here


Add javadoc explaining the contract.


hasOwnProperty could probably be removed here


Yes, but some functions will return incorrect data until TorrentHandle::handleStateUpdate() is called with the full status.


Maybe we should create a separate backup recovery config, and move all the backup recovery related configs there


The fixes look good.


To destroy it. It decrements refcount and it will get garbage collected.
(At least that was my intent, it's the first time I ever wrote any go ;)
Afaict it's an idiom with destructor-y things.
22 maj 2015 20:59, "Jon Mason" <notifications@github.com> napisał(a):

> Why is tf being set to nil?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/conjurinc/cauldron/commit/3c25bbfa1d31c5ca9248d638d2b29cacb52e33b4#commitcomment-11333656>
> .
>


What, we're not doing `System.exit()` anymore? That's confusing...


``requireNonNull``


Keep in mind that including the Enumerable module is for allowing instances of *your* Board class to have enumerable methods called on them and that you'd need to implement `each`. You can use the Enumerable methods on the Array in your `@board` instance variable already without doing `include Enumerable` in your class definition.


Actually, we still need these because Google's MariaDB has carried over super_to_set_timestamp. We override this function internally with a plugin.


Is this one locating itself?


c


Huh, that should be 999. Same as `bulkInsert`: `SQLITE_MAX_VARIABLE_NUMBER`.


Super duper!

On Fri, May 22, 2015 at 9:07 PM, Douglas McClean <notifications@github.com>
wrote:

> @bjornbm FYI, don't break the haddocks, travis is watching them too now.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/bjornbm/dimensional-dk/commit/b9b3dfb85ec6b37aca527d84b8b89de011aabdb4#commitcomment-11333733


Not that I really like using `IN` with a thousand items.


Looks like this commit broke build.
Please see http://ci.cgeo.org/job/c-geo/3759/console to remedy the problem.


constant should be upper case: "ARRAY_LENGTH"


Yeah. Very deep chains are tough without better syntactic support.


really loads the whole tree? or just loads the next parent until dom with selector is found... not sure whether i want to introduce another behavioral CSS selector.


Thanks! I will reply tomorrow.


why this field is not final and is not private?


CBECC (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


we have 3 methods:
- setPosition
- getPosition
- isEmpty

do we need this getter at all?


why is not final?


finals?


Done: https://github.com/digitalbazaar/credentials-polyfill/commit/27d540ba021ef781811e9327ac01a145ef6fdb15


\n?


Fuck you!!!!


Similarly, `attr_accessor`s are for defining reader and writer methods for instance variables *in* the class the `attr_accessor` is used on. Thus the `:designation` option makes sense here but not `board` or `name` as those aren't instance variables on Human.


Wow thats a pretty heavy duty question for a Friday afternoon!  I think the issues you are raising here are shared among all of the developers.  Perhaps a phone call (say Monday) with you me and @myoldmopar would be good? Most pointedly here, how do we construct a new instances from idf.  It appears you are proposing do that from getGroundTempInstance is that right?  Inside getGroundTempInstance you probably leverage input processor to load up the data.

A virtual getCurrentGroundTemp seems like a good idea to me.

I can't exactly distinguish the roles of the various classes, including GroundTemperatureManager, GroundTemperatureModel, and groundHX.


Yeah, I just noticed that bug too. :/ The funny part is that the unit test tests for the wrong thing, too.


first commit


Issue #4 addressed in getConditionalProbs


Hi,

I get:
```
ERROR: LoadError: UndefVarError: xwidth not defined
 in push! at /home/alain/.julia/v0.4/Packing/src/rectangle.jl:33
 in render at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:98
 in render at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:95
 in anonymous at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:68
 in get! at dict.jl:630
 in get! at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:67
 in anonymous at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:84
 in map at abstractarray.jl:1330
 in get! at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:83
 in visualize at /home/alain/.julia/v0.4/GLVisualize/src/visualize/text.jl:31
 in visualize at /home/alain/.julia/v0.4/GLVisualize/src/visualize_interface.jl:18
 in include at ./boot.jl:252
 in include_from_node1 at loading.jl:134
 in process_options at ./client.jl:310
 in _start at ./client.jl:409
while loading /home/alain/src/Julia/GLVisualize.jl/test/test_text.jl, in expression starting on line 6
```

with `GLAbstraction`:
```
commit 3ce6c5617b5819ead659f95839959f834d3577dd
Author: Simon Danisch <sdanisch@gmail.com>
Date:   Thu May 21 21:39:54 2015 +0200

    added some additional texture functions
```

and `AbstractGPUArray`

```
commit 56c06488ad6f849ac489232a0fcae9eff0546c3d
Author: Simon Danisch <sdanisch@gmail.com>
Date:   Thu May 21 21:40:15 2015 +0200

    some more functionality

```


71693606-CompilerWarnings1 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1656 of 1656 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)


Are you planning on sending a PR with this?


:+1: good stuff


Sublime buildsystem:
```json
{
	"cmd": ["bash", "-c", "runobjc $file"],
  	"selector": "source.objc",
}
```


Hi there.

On May 22 2015, Ismael Mejia wrote:
> I will send you the commit in some time,

Great.

> I didn't know that extend function (shame on me).

Hey, we are all learning stuff here. :) There's no shame or whatever. :) If
I don't know something, I will ask you and I also ask you that you criticize
my code, since learning design is like an art and we have to "distill things
from experience".

In this subject, I can't help but point you to my blog post on a very
condensed list of points that I learned from developing these small tools in
the past years:

    http://cynic.cc/blog/posts/2015-05-16_coursera-dl_activities/

Any comments (here or there) will be more than welcome as we're all on the
same boat!

Cheers!

--
Rogério Brito : rbrito@{ime.usp.br,gmail.com} : GPG key 4096R/BCFCAAAA
http://cynic.cc/blog/ : github.com/rbrito : profiles.google.com/rbrito
DebianQA: http://qa.debian.org/developer.php?login=rbrito%40ime.usp.br


rax retry tests


Back


Oh, damn. GeometryTypes got an update as well! I should keep better track
of this!

2015-05-22 21:31 GMT+02:00 AlainLich <notifications@github.com>:

> Hi,
>
> I get:
>
> ERROR: LoadError: UndefVarError: xwidth not defined
>  in push! at /home/alain/.julia/v0.4/Packing/src/rectangle.jl:33
>  in render at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:98
>  in render at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:95
>  in anonymous at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:68
>  in get! at dict.jl:630
>  in get! at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:67
>  in anonymous at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:84
>  in map at abstractarray.jl:1330
>  in get! at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:83
>  in visualize at /home/alain/.julia/v0.4/GLVisualize/src/visualize/text.jl:31
>  in visualize at /home/alain/.julia/v0.4/GLVisualize/src/visualize_interface.jl:18
>  in include at ./boot.jl:252
>  in include_from_node1 at loading.jl:134
>  in process_options at ./client.jl:310
>  in _start at ./client.jl:409
> while loading /home/alain/src/Julia/GLVisualize.jl/test/test_text.jl, in expression starting on line 6
>
> with GLAbstraction:
>
> commit 3ce6c5617b5819ead659f95839959f834d3577dd
> Author: Simon Danisch <sdanisch@gmail.com>
> Date:   Thu May 21 21:39:54 2015 +0200
>
>     added some additional texture functions
>
> and AbstractGPUArray
>
> commit 56c06488ad6f849ac489232a0fcae9eff0546c3d
> Author: Simon Danisch <sdanisch@gmail.com>
> Date:   Thu May 21 21:40:15 2015 +0200
>
>     some more functionality
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JuliaGL/GLVisualize.jl/commit/379ba22d13c654d216663ea5819c0b844df1b927#commitcomment-11334035>
> .
>


You should use the plugin's logger here with INFO level.


Hey bud, you seemed to have misspelled canola, let me know when you fix it.


yay


ship it ;)


I mean, I *could* make the signature specific, but I might as well let the compiler do that, and make it easy to lift this out when I come to write password sync…


What motivated you to do this change? The problem is you cannot assume long is larger than size_t or vice versa. The original code would cause a promotion but new code can cause undefined behavior.


71693606-CompilerWarnings1 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (632 of 632 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-5.29%25-green.svg)


generally, clean the commented code out


In one case it is not really a conflict but a change already applied. Given that it is this case we hit, we should make sure that we just return a 200 rather than a 409.
In the other case where it is a conflict, the question is really about the data. We would/could send back a 409 with the current etag and have the CC determine if a retry is necessary at all. If it matches, nothing is needed although we should be able to detect that on the nsync side. If it doesn't match, then we should retry since we are not in sync at least on the fast path.

Much of the behavior/processing comes down to how the data is formulated and applied, and what it means for nsync.

If I am desiring an app which is already desired, it's a no-op, returning a 409 is fine. The log line is wrong and you cannot write header twice, first one wins.

UpdateApp is similar to desireApp. We know that something changed, and if its already done, we are done, otherwise...


My brain doesn't want to chime in right now, but I'll say Monday won't work (and nor will Tuesday).  I'll try to add to @mmatts 's description over the weekend and we can re-convene Wednesday as needed?


fast-forwarding develop to borshop-integration-1119-bugfix/jrd/epmd = 40ce5a76


IGameViewHelper, helper is not a very good idea here because this is Cordinate Reader (part of UI)


Thanks you for your feedback, and help. This change will be accepted.


same, it's not a Helper or Controller


Unsure why you change the engine parameter passed into the class here from the event. Possibly a mistake as you wouldn't change the default search engine passed from the plugin.


\n


it seems if you want the public link, then we can't use the permalink under
https://console.developers.google.com/storage/browser/appengine-sdks/deprecated :(


Ugghh... Yeah I guess Monday wasn't such a good suggestion.


Please review @wurstchristoph @jancborchardt

Fixes: #712


* Please document all of the function you added
* Remove the example

```
j = Jenkins('http://developers.compbio.cs.cmu.edu:8080', 'yileic', 'cyl19890629')
job_name = 'cellorganizer-demo2D00-glnx64'
j.set_branch(job_name, 'hello1')
print 'updated branch is', j.get_branch(job_name)
j.set_node_name(job_name, 'hello2')
print 'updated node is', j.get_node_name(job_name)
```


* Make to never push .DS_Store files. Add them to the global .gitignore file


I should have caught that.  I'm starting to go cross eyed as I mechanically crew threw these tests.


Most likely you don't want dupes here. I'd encourage a Set instead and override equals/hashcode to verify based on their name.


I don't think anyone has looked at this in a while, but I believe you are getting very close to the issues being thought about in [this](https://docs.google.com/document/d/1GgVj215OANT58XiQXLTd8wzPHLAmetTK9V8SuUTHbIE/edit#heading=h.lb8jaytzd4g5) document.


(actually, I can blame Russel ;)


Hey @martinheidegger ,

Thanks for merging this! Any idea on when the next patch release will make it to npm?


Just updated GeometryTypes,
```
commit a348c8d3ebd9e652355c3bb3ad424a82292a69d0
Author: Simon Danisch <sdanisch@gmail.com>
Date:   Mon May 18 18:14:36 2015 +0200

    added comments
```

error remains:
```
 Julia4 test_text.jl
Executing ~/.juliarc.jl -- this version does not fix PATH
Warning: using ColorTypes.Color in module Main conflicts with an existing identifier.
ERROR: LoadError: UndefVarError: xwidth not defined
 in push! at /home/alain/.julia/v0.4/Packing/src/rectangle.jl:33
 in render at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:98
 in render at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:95
 in anonymous at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:68
 in get! at dict.jl:630
 in get! at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:67
 in anonymous at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:84
 in map at abstractarray.jl:1330
 in get! at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:83
 in visualize at /home/alain/.julia/v0.4/GLVisualize/src/visualize/text.jl:31
 in visualize at /home/alain/.julia/v0.4/GLVisualize/src/visualize_interface.jl:18
 in include at ./boot.jl:252
 in include_from_node1 at loading.jl:134
 in process_options at ./client.jl:310
 in _start at ./client.jl:409
while loading /home/alain/src/Julia/GLVisualize.jl/test/test_text.jl, in expression starting on line 6
```


This one can go as well.


`absoluteFilePathsUnwanted()` for better search results.


Ugh... Sorry, I didn't commit the change in GeometryTypes -.-

2015-05-22 21:42 GMT+02:00 AlainLich <notifications@github.com>:

> Just updated GeometryTypes,
>
> commit a348c8d3ebd9e652355c3bb3ad424a82292a69d0
> Author: Simon Danisch <sdanisch@gmail.com>
> Date:   Mon May 18 18:14:36 2015 +0200
>
>     added comments
>
> error remains:
>
>  Julia4 test_text.jl
> Executing ~/.juliarc.jl -- this version does not fix PATH
> Warning: using ColorTypes.Color in module Main conflicts with an existing identifier.
> ERROR: LoadError: UndefVarError: xwidth not defined
>  in push! at /home/alain/.julia/v0.4/Packing/src/rectangle.jl:33
>  in render at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:98
>  in render at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:95
>  in anonymous at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:68
>  in get! at dict.jl:630
>  in get! at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:67
>  in anonymous at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:84
>  in map at abstractarray.jl:1330
>  in get! at /home/alain/.julia/v0.4/GLVisualize/src/texture_atlas/texture_atlas.jl:83
>  in visualize at /home/alain/.julia/v0.4/GLVisualize/src/visualize/text.jl:31
>  in visualize at /home/alain/.julia/v0.4/GLVisualize/src/visualize_interface.jl:18
>  in include at ./boot.jl:252
>  in include_from_node1 at loading.jl:134
>  in process_options at ./client.jl:310
>  in _start at ./client.jl:409
> while loading /home/alain/src/Julia/GLVisualize.jl/test/test_text.jl, in expression starting on line 6
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JuliaGL/GLVisualize.jl/commit/379ba22d13c654d216663ea5819c0b844df1b927#commitcomment-11334185>
> .
>


* Rename the file from `test_yilei.py` to something more descriptive. Use the command `git rename test_yilei.py <useful_name>`
* Document example


This commit is related to #60


final


remove all the flags you removed. As we really don't need these? or at least not as many as they have?


#obj.GetObjectAt(id).featurecode is another option
this line can be added where line 69 is to use featureCode instead of resultID, and combine it with the name.