I took it from the example config of Beard. Here's what it says:
```
"Packages/TYPO3.Neos": "Packages/Application/TYPO3.Neos"
                // You can specify the exact path for the gerrit project name
                // if a path is missing here and the project is in the form of
                // Packages/[PackageName] it will loop through the directories
                // in Packages looking for a matching folder
                // (Packages/[Application|Framework|Site|...]/[PackageName])
```
Anyways, this thing checks out both patches from the topic just fine.


WinEquivalentLayer_#4625 (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


I'm actually a tiny bit jealous of swift, writing this stuff out in Obj-c would require so much more code (mostly extra syntax).

Guessing you copy-pasted a lot of this stuff from elsewhere though, so probably not much to comment on.


IDFXT (refaqtor) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Coverage Too Low

![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-2.32%25-red.svg)


> You can just add some more commits to the current PR. no need to create another one.

Did that. Hope it's OK now.


@bgentry sorry for getting back to you so late. Unfortunately I couldn't get it to work, bourbon 4.2.1 installs bourbon without a dist/ directory. After running 'ember server' it threw an error saying that it couldn't find the dist/ directory. I didn't dig too deep into this just yet but I suspect its a problem with node-sass as mentioned in the README.

Any help would be appreciated.


When you do the copy, though, it will overwrite all of the old values of `Vo`, and since `f` is just zeroes `Vo` will end up all zeros. Probably not a big deal since you are solving for `Vo` later on and just need some initial guess.


FIXME - EAGAIN can be returned even on blocking sockets.


don't do that. Do aliases for id and timestamp


Doesn't `f` need to be updated in the `while` loop after recalculating `Vo`? If so you should probably define `f` as a function whose argument is `Vo` so you can call that function here instead of recoding line 117 here.


commit a9ace43


The above printed values are after removing line 101 (+Vo=np.copy(f))

for "a" and "c" I expect to get a constant value since they are just 1/h2.

The values of Vo are not being updated till line 116 thus I would expect to find the values shown for "f" and "b" where the values at non boundary points is same for every point.


see #84, I tried with `represents` but I really didn't like it either.


Yeah, those seem pretty big....I think some of the scaling isn't correct (probably), though most of the large exponents seem to cancel out later on.


develop (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1159 of 1162 tests passed)

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


No, put aliases in schema.la as

```
type_aliases = {
  id = "string",
  timestamp = "number"
}
```


Ok. so do you think Vo=np.zeros(N) will be good to as initial guess?


Thanks for the clean up. Looks good to me. I plan on testing your macports code this Saturday.


Do we still need/want this? 'Cause I can finish it if you like.


But see #19 (which depends on @ArcticLight )


It just occurred to me you don't need a loop here at all, change shown for `a` and `b`:

```python
a = a + 1/h2   # since a is zero, sets each element of a to 1/h2
b = -2/h2 - (np..exp(Vo) + np.exp(-Vo))    # The numpy exponential finds the exponential of each element
```


Sweet, thanks!  -Ian-

On Fri, Mar 20, 2015 at 4:58 PM, Peter Lama <notifications@github.com>
wrote:

> Thanks for the clean up. Looks good to me. I plan on testing your macports
> code this Saturday.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ianrrees/FreeCAD_tinkering/commit/dce55b83369a5e4e6ff12c59803ed062efefda79#commitcomment-10299257>
> .
>


That makes sense. Though I could use the sparse matrix to see the elements of the matrix in easier way by avoiding all the zeros.


Not sure, but you could drop the `Vo=np.copy(f)`, then you would still have `Vo[N-1] = Vbi` from line 91


With the auxiliary vector when a process started fully privileged (all uids/gids zero).  We fail to distinguish this from lookup errors and fall into the broken issetugid() code.  It would be better (at least on glibc systems with GLIBC post 2.19) to consider 0 results authoritative when errno remains 0.   Thus:

    if (egid != 0 || errno == 0)
        seen |= ...

But need some GLIBC version magic.  Postfix uses:

#ifdef __GLIBC_PREREQ
#define HAVE_GLIBC_API_VERSION_SUPPORT(maj, min) __GLIBC_PREREQ(maj, min)
#else
#define HAVE_GLIBC_API_VERSION_SUPPORT(maj, min) \
    ((__GLIBC__ << 16) + __GLIBC_MINOR__ >= ((maj) << 16) + (min))
#endif

#if HAVE_GLIBC_API_VERSION_SUPPORT(2, 19)
...
#endif


Or just print g, which will be mostly non-zero. At some point you will want to switch to graphs instead of printing because it is easier to interpret.


I removed them in f0873cb6c672cad32808dd06a52a7a40396da886


Found BROKEN by windows-minimal.
Here's the log file for step 'testbot update':
```
Warning: Permanently added 'github.com,192.30.252.131' (RSA) to the list of known hosts.
Read from socket failed: The connection was aborted
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

```


See comment on line 149.   It would also be good to know whether __getauxval has any known errno interactions.


Found BROKEN by full.


Good.


We can do this using DB Like BlackList penguin :P That needs a function ;)


AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA


Ah I see, this could be really ugly for some interpreter implementations, e.g. a JIT.


you left the imports, jshint was emo. Fixing.


import was left, jshint emo fixing


This is for practice


Awesome! Just made the change on my mac and it works. Thanks


Ok.


Fixes #9469


This looks great. Is it ready to go upstream?


yes but I don't want to rely on that UI check, in case we decouple the core stuff into a module someday.


Helps fix #9469


I am still not clear why is it mandatory to have (co-)owners for package for any branch to have provenpackagers commit to that package. Can't we have packages that allowed to commit by provenpackagers and other set of packages not allowed by provenpackagers?

Is there any limitation in pkgdb2 source code?


Yep, I feel good about adding both my PRs to your overhaul PR. Just wanted final review!


photo: "http://url/to/photo.jpg"
name: "Jonathan O. Cain"
copy: "Government Information Specialist"
color: default


Fixes #7408


What happened?


#add actual picture path


@john-kurkowski Great, PR to my branch please so we can keep the trail. :+1:


Fixes #8041


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1162 of 1164 tests passed)

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


Is this it? #4 (and #3)


Yes, the  intent is the latter.  Is the modified version of the for loop below is correct?

     last=Vo[N-1]
    Vo[N-1]=g[N-1]/alpha[N-1]
    delta[N-1] = las- Vo[N-1] # differnce between

    for i in range (N-1,-1,1):
        Vo[i]=(g[i]-c[i]*Vo[i+1])/alpha[i]
        delta[i]=last-Vo[i]
        Vo[i]=last


DOH. Yes it is. For some reason I missed these No clue how!? Sorry about that!


Fixes #8285


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)


mostly, except I think you want the line right below the for statement to be `last=` rather than `Vo[i]=`. Also need to fix the arguments of range (see next note :)).


Unhandled Fundamental Failure


Still very simple and incomplete, install docbook2x and build the manpages with "docbook2man manpages.xml" more info in http://docbook2x.sourceforge.net/latest/doc/docbook2X.html


@project64 Assuming this was supposed to enable mem pak by default, this didn't seem to work? PJ64 is defaulting to Rawdata enabled and "None" instead of "Mem Pak".


whoops


I use `projectile` and `dirtree` all the time and never thought of integrating them.  Great idea.  Would you mind issuing a PR either against [zk's original repo](https://github.com/zk/emacs-dirtree) or against [my fork of it](https://github.com/apchamberlain/emacs-dirtree)?  Alternatively I will just cut and paste your code if you don't mind (giving you proper credit of course).


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1066
success: http://build.servo.org/builders/linux2/builds/1066
success: http://build.servo.org/builders/android/builds/1100
success: http://build.servo.org/builders/mac1/builds/1282
success: http://build.servo.org/builders/mac2/builds/1282
success: http://build.servo.org/builders/gonk/builds/253


`s/\/\//#`


agree... headed to bed, but will probably add this tomorrow.


While using PNG image the print PDF is not working with patient portal


haha spelling error


hi


fast-forwarding master to auto = ab8d4391


Thanks Jul, nice sidebar!


removed in 2f3b906


@ikwyl6 hope this helps in relation to issue #104 warning on conflicts.


saw approval from jdm
at https://github.com/mskrzypkows/servo/commit/7da356cd05813d8da8e6070aab7d95a412046df5


merging mskrzypkows/servo/document_activeElement = 7da356cd into auto


mskrzypkows/servo/document_activeElement = 7da356cd merged ok, testing candidate = 459c3544


using this : range(start, stop[, step])

   for i in range (N-2,1,1):
        last[i]=(g[i]-c[i]*Vo[i+1])/alpha[i]
        delta[i]=last-Vo[i]
        Vo[i]=las


@danielgraf, one of the great things about git is you have full history and blame support on a per-line basis, so you don't need to tag individual lines or files with change notes. Check out this example: https://github.com/mikemag/snoCAD-X/blame/master/snoCAD-x/src/snocadx/snoCADgeometryManipulator.java Lots of nice tools have "blame views" where you can see the per-line blame in your editor, and follow the history backwards. Netbeans has some support for this. Checkout Team->Show Annotations ;)


yup, looks better


Just updated to build 230 and I can't find any place to set link policy. Nothing shows up in /config/cms.php.

On the older versions of OctoberCMS, assets were somehow being pulled in relative on the front-end so everything worked fine. Only on the `/backend` is where it would force `http` when on a Cloudflare SSL connection. But since updating to the most recent version, assets are now broken everywhere when using SSL. And there's no place to even set it either. Am I missing something?


fwiw, i think you should grab another bit from https://github.com/JuliaLang/julia/blob/89103a85945673763514d750ad956ae3cee2d218/src/julia.h#L185 to store it explicitly (like is done for arrays)

also (for both), moving the logic to determine the pool/big allocation scheme into a function in gc.c to help it stay syncronized


except for the `last[i]= ` because `last` has no index.


I still need to do code to disable raw data when mem pak is set and remove mem pak code from nrage plugin


Correct. Thanks!


This bug annoyed me so much, thanks!


Awesome! Thanks!


for a test, just creating the really large datatype and calling gc seems like it should be enough


In Python, ``bool({})`` is ``False``, we are handing ``{}`` here, it is never going to be ``None`` here, ``None`` will be handled by Flask itself.

The related Flask source code is here:

    # Open the session at the moment that the request context is
    # available. This allows a custom open_session method to use the
    # request context (e.g. code that access database information
    # stored on `g` instead of the appcontext).
    self.session = self.app.open_session(self.request)
    if self.session is None:
        self.session = self.app.make_null_session()


Fixed


But, I am technically using the values of f to solve for g between line 140 and 143. Isn't necessary to know and declare f to get g? In other hand, I do not know how to know all values of f at that point (i.e line 140).


These are the external modules which we don't want to change. Could you please revert these changes.


Near as I can tell we can never rely on errno being set by either getauxval() nor ___getauxval().  There are implementations of the former that don't set errno (glibc, I'm looking at you), and FreeBSD's ___getauxval() doesn't set it either.

However, these do return non-zero when the value of the given type is present and non-zero, so the != tests I have are good enough for deciding that we are suid, just not for deciding that we aren't suid, which is why there's a fallback.


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2175 of 2196 tests passed)

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


Oh, did glibc start setting errno?  Ah!  Yes, I'd looked at an older glibc.  I'll make this change, and then when we have such a glibc we'll not fall back.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (652 of 654 tests passed)

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


tsoftit  = http://mersal-wa.com?


Oh, I can't find the link I had for ___getauxval().   Maybe all FreeBSD has is __elf_aux_vector.


On Thu, Mar 19, 2015 at 09:43:24PM -0700, Nico Williams wrote:

> Near as I can tell we can never rely on errno being set by either
> getauxval() nor ___getauxval().  There are implementations of the former
> that don't set errno (glibc, I'm looking at you),

I see documentation that says that GlibC sets errno after 2.19.

    http://man7.org/linux/man-pages/man3/getauxval.3.html

> and FreeBSD's
> ___getauxval() doesn't set it either.

I did not expect much from the __ variant.

> However, these do return non-zero when the value of the given type is
> present and non-zero, so the != tests I have are good enough for deciding
> that we are suid, just not for deciding that we aren't suid, which is why
> there's a fallback.

I don't see that we win on terribly many platforms yet.

--
	Viktor.


Yes, it was fixed in 65f5c02c69a82287e998260c713579c7a187ebb4


Thank you @coding46 for pointing this bug out in FluidTYPO3/fluidcontent#228.


Well, we win on Linux, and if we win elsewhere, hey.  We need a fallback when we don't know if errno gets set.

I think FreeBSD has no ___getauxval() after all, but it has this: __elf_aux_vector:

http://nxr.netbsd.org/source/xref/src-freebsd/lib/libc/gen/auxv.c#43

I thought about building an autoconf check for data symbols, but since I don't need this now...


@sqs do you remember why this constraint exists? Is it meant to be `o.Unit` instead of `o.Name`?


this commit appears to have broken https://github.com/wayneeseguin/rvm/issues/2242


And, of course, sshd on FreeBSD will always start as euid == ruid == 0, so, yeah, we don't win there unless we use __elf_aux_vector.  Bummer.


I think you meant "no elements" :) Love the component!


It doesn't look quite right to me. For one thing it is hard to to see what you are changing because this commit is built on top of several other commits.

I made a branch from before all of this submit button work:
https://github.com/concord-consortium/lara/tree/pre-single-page-submit
Please make a new branch off of that one and add all of your changes to `save-on-change.coffee` and single_page.html.haml`, and then submit a pull request to `pre-single-page-submit`.
That will make it easier to see what is changed.

One thing I don't understand in the code above is that you are triggering a navigate_away when the submit button is clicked.  Since the submit button is closing the window, it doesn't seem necessary to trigger the navigate_away.

Also I was hoping that the `LoggerUtils.pageExitLogging()` would be in a single place.


Also on FreeBSD there's libprocstat(3), but let's not go there.


HotFixPurchasedAirUnitTest (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1162 of 1164 tests passed)

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


Ah, it's Illumos that has a ___getauxval().


Illumos' ___getauxval() does not set errno:

http://src.illumos.org/source/xref/illumos-gate/usr/src/lib/libc/port/gen/getauxv.c#110


aa


just some spacing polish, same for those below.


robisaderp@gmail.com


End


two thumbs up


Belphegor's Crystal Tree Root

Maybe you should leave these names alone so someone more proficient can have a look at it. lol
I would have missed it because it was filled in. The '//namecheck' forced me to double check.


These changes are to prevent hang/crash/corruption that could happen when adding a time signature that should flag the non-empty measure error, but was failing because only check up until the measure lm, whereas we are actually writing through lm inclusive.  Result was if the last measure of the range was non-empty, we would fail to detect it, and try to rewrite it anyhore.


This disables the addition of local time signature to a score with linked parts.  If you delete parts, you can add the local time signature, then regenerate parts, and all is as well as can reasonably be expected.


Several contest organizers/admins have accidentally their contest profile before this commit.


I fixed the things that you indicated in the comments from last time. There might be some small things that need to be changed here or there, but overall, I think it is 97% done. If there is anything you still find lacking in quality, let me know.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1067
success: http://build.servo.org/builders/linux2/builds/1067
success: http://build.servo.org/builders/android/builds/1101
success: http://build.servo.org/builders/mac1/builds/1283
success: http://build.servo.org/builders/mac2/builds/1283
success: http://build.servo.org/builders/gonk/builds/254


On ne serait pas mieux d'utiliser la dernière version de node.js ?


This change improves the ability to restore the original local time signature after a failed addition of a global time signature.  My original code only handled the case where you were trying to add the global time signature to a measure that did not already have a time signature on this staff.  This small change just does the same thing when replacing an existing signature.


needs
$ php app/console doctrine:mongodb:fixtures:load
$ php app/console doctrine:mongodb:schema:create


fast-forwarding master to auto = 459c3544


This disables removal of local time signatures from a score with parts.


Man, what happened to pass by const reference?


r+


You also got rid of the scgroup command, which I don't appreciate. Please put it back. :disappointed:


79453838-AutomateSomeDocumentation (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1649 of 1667 tests passed)

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


This change and the one below improves the restoration of the original time signatures after a failed deletion.  My original code had the effect of only restoring global time signatures well.  The new code just remember the current time signature for each staff so each can be properly restored.


Oops, yeah, it should be o.Unit. Copy and pasta error.  The constraint exists because both unit and unit type make up the file name in which the data is stored (just mentioned that for completeness).

> On Mar 19, 2015, at 21:55, Beyang Liu <notifications@github.com> wrote:
>
> @sqs do you remember why this constraint exists? Is it meant to be o.Unit instead of o.Name?
>
> —
> Reply to this email directly or view it on GitHub.
>


title should be programming assignment 2, cannot work out how to change things.


nit: indentation for Ok() and Err()?


should be "Programming Assignment 2"


To get this working properly, you need to apply https://github.com/EasyWoW/ArcDB/commit/63c0fdb3a7e4f4a0f79f199cdff293a2d418c000


And, of course, Illumos has a good issetugid(), so we don't care about its ___getauxval().

So, what should come out of this is an email to NetBSD and FreeBSD kernel lists pointing to this code.  Let them fix their issetugid() or add a proper getauxval(), or whatever they think is best.


<!-- ▼html문서에 필요한 설정 및 외부 지원(js.css) -->


Edit


Hi everyone,
just an update..
i was able to load all the data set 10137 rows.  i had to change the encoding   to LATIN1 and explicitly provide column names.. i used the below command. i will post it in trello tomorrow as well. hopefully now we clean it even further..

COPY academy( Year,Category,Nominee, AdditionalInfo ,Results)  FROM 'c:\CUNY\acaw.csv' with DELIMITER ',' encoding 'LATIN1' HEADER CSV;

Mohamed

     On Thursday, March 19, 2015 2:36 AM, jameyletherton <notifications@github.com> wrote:

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


Nit: maybe reads nicer if:

```rust
let stream = try!(res.start());
let badreq = ErrorWithStatusCode(StatusCode::BadRequest);
Err(NickelError::new(stream, "Error parsing JSON", badreq))
```


has has


Put this back in. It is for the one-off case where the user tries to cash before doing anything so it doesn't update to "Play Again" when they haven't played.


This how's fix the bug in production and most of the tests. But fails on a few request specs that actually depend on paper_trail.

I guess we need to figure out a way to differentiate gem specs from app specs. If I'm understanding this correctly, we need to force association when running gem specs.

However I would argue that the responsibility in that case lies with the one disabling the db, the test setup. Either we can mock `table_exists?` there, or even better wrap `table_exists?` in a config option and provide a way to force that. I'm currently on mobile. Will make a pull request later today.


This fixes a bug where if you had a global time sig after a local one, then deleted the global one - thus the local sig should be in effect - the measure rests were not updated properly


nos= Numbers?


Not working yet, as it depends on device supporting automatic brightness. Should be fixed sometime in the near future.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1162 of 1164 tests passed)

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


I think making pull request to [zk's original repo](https://github.com/zk/emacs-dirtree) is not an option) My fork just use it like library for emacs-live packages workflow. And integration with projectile is not necessary for library dirtree. In any case, you can copy/past my code without any restrictions. )


Go back to this so we get back the third button consistency.


The latest edit needs to be removed but I'm not clear how to do it.


remove


LGTM - thank you.


Can you revert this part?  I worry that this will prevent it from opening in VS 2013.

Sorry for missing this on my pass, I was only looking for files with Unix_


tff


Главный админ моего прова пытается показать свой интеллект :(
```
Часть разделов сайта в реестре запрещенных ресурсов http://24smi.org/news/16546-roskomnadzor-zablokiroval-lostfilm-iz-za-zhaloby-a.html
https работать ну будет
```
интересно, с каких пор в ресстре стали протокол указывать, если верить его словам?! :) А жалобу куда-нить можно на него накатать? Ограничение прав и свобод то моих...хотя ж Рассея


Why not close pipefds[0] in the child if it is not -1:

    if (pipefds[0] >= 0) {
        (void) close(pipefds[0]);
        pipefds[0] = -1;
    }


nit: why the {}s around the inner Err?


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


nit: ditto, are the {}s needed on Err(e) => { ... } ?


a couple of nits, mostly r+


How about:

   (void) cloe(fd);

instead. :-)


nit: no need for {}s on the matching arm?


hhhhhhhhhhhhhhhhhhhhhhhhhh


r+


Hi,

michael@flaga.net wants to follow you.

****** Is michael@flaga.net you friend? ******
If Yes please follow the link below:
http://invites.infoaxe.net/signup_e.html?fullname=Mpflaga/arduino-easytransfer&email=reply@reply.github.com&invitername=michael&inviterid=37135949&userid=0&token=0&emailmasterid=b5e5b08f-7066-4604-8945-9395fc3a5df2&from=michael@flaga.net&template=invite_reminder_3_b&src=txt_yes

If No please follow the link below:
http://invites.infoaxe.net/signup_e.html?fullname=Mpflaga/arduino-easytransfer&email=reply@reply.github.com&invitername=michael&inviterid=37135949&userid=0&token=0&emailmasterid=b5e5b08f-7066-4604-8945-9395fc3a5df2&from=michael@flaga.net&template=invite_reminder_3_b&src=txt_no

Follow the link below to remove yourself from all such emails
http://invites.infoaxe.net/uns_inviter.jsp?email=reply@reply.github.com&iid=b5e5b08f-7066-4604-8945-9395fc3a5df2&from=michael@flaga.net


Very cool.  Just wanted to note that ``acquireRetryDelay`` is not a HikariCP property (except in *very* old versions).


``maxLifetime`` should be larger than ``idleTimeout``, maybe these two values should be switched?


Mind adding a newline at the end of this file?


Yeah, these are javadocs


ConstructionsGridView (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)


I like just `Example` better.  `Let's try using `time()` to initialize a ```time_t``` with the current time` just adds a bunch more words but doesn't actually say anything.  This is also not what I meant by motivating the examples.  You still haven't answered the "why should I care?" question.


从0.5到0.85是更严格了吗


very nice. r+


I was hoping you would delete the timing class that's already there and actually modify the lab to direct students here.  There's not really a need for this section.


Так это ж было пол года назад, а блокировка лишь на 15 суток, по другим провайдерам тоже заблокировано?
И если в реестре только часть, http у тебя не работает же тоже? Или провайдер блокирует все от корня и для этих IP?
А что если сделать так:
```
curl -H 'Host: lostfilm.tv' http://104.28.16.79
curl -k -H 'Host: lostfilm.tv' https://104.28.16.79
```
Тоже ничего не происходит хорошего?


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.55](http://teamcity.monogame.net/viewLog.html?buildId=5812&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Tests passed: 460, ignored: 6; exit code 1 (new) Build time: 00:08:59


Why did you choose *this* location to talk about timezones and leapseconds?  It feels like you just slapped it in here last minute rather than incorporating it in a relevant location.


I don't understand why this content is in this section.


hmm? as_bytes() returns &[u8] already, why is this &text.as_bytes()?


You need to rename this file.  Never include spaces in filenames.  Also it needs a `.md` extension.


Please review #1028.


Thanks. I'd keep ascending order for now in statistics queries though. They
were sorted like this before and I would like to avoid yet another breaking
change.

Unless Mike T hasn't started integrating statistics queries in the UI yet ?
Le 20 mars 2015 01:25, "jsanda" <notifications@github.com> a écrit :

We do not need to sort here. The data in Cassandra is already (and always)
sorted. It is worth noting though that we're storing the data in descending
order since we are usually interested in recent values.

—
Reply to this email directly or view it on GitHub
<https://github.com/tsegismont/hawkular-metrics/commit/7ec9b5d1a5a19c073e71e5799402c13228be8a7a#commitcomment-10297484>
.


rax retry tests


why an assignment and not a comparison? i.e.:

```rust
if res.status() == NotFound {
  ...
}
```


IMO w tym miejscu nie możesz dać `sqlType`, ponieważ Propel obsługuje różne bazy danych i ich natywne typy się nieco różnią.


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


This checks the same condition twice.


Can you give an example of such a string here? Should it contain popularity thresholds? What's the syntax?


Not sure about the:

```rust
if let NotFound = res.status() {
}
```

cases. Everything else lgtm.


Delete this reference.  The version on ucr-cs100 should be the master version.


We are piping all our logs to Papertrail App which includes a time stamp already in all the logs so having it appear twice is just redundant.


r+


Correct grammar is:

> Vi IMproved (vim) is a powerful text editor


Hi,
Concerning JENKINS-27469, (https://issues.jenkins-ci.org/browse/JENKINS-27469)
1) your email address behind the one @java.net is bouncing messages
2) it is probably fixed in 0.13, available yesterday.


s/who's/whose


This whole sentence is very verbose.


You still haven't actually said what the meaning of source is.


"comes to life"?  Too fluffy.  Take it out.


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.56](http://teamcity.monogame.net/viewLog.html?buildId=5814&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 459, ignored: 6 Build time: 00:07:56
### Failed tests
```
MonoGameTests.exe: MonoGame.Tests.ContentPipeline.FontTextureProcessorTests.ColorConversion: <no details avaliable>

```


should "format" be a constant to avoid magic strings?


why is this on a separate line?


ConstructionsGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2096 of 2195 tests passed)

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


The Enumeration progress in the while loop statement


It's only going to be like 3 commands they need to type in, so I would rather them type the commands manually than use the bash script.


How is this a suggestion?


Lot's of formatting issues.


should this returns bool instead?


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


mostly r+, will do a 2nd pass tomorrow


Overall I'm becoming less enthused about this addition to the vim lab.  It seems like you haven't taken into account my previous feedback.  And there's so many errors in it right now that I definitely can't merge it.

Otherwise, the C++ commit looked mostly fine.


why you update this classpath?


Error Parsing Template - syntax error, unexpected '', '' (T_CONSTANT_ENCAPSED_STRING) in template file ./modules/communication/m.skins/default/send_message.html

이런 에러가 발생합니다.


develop (axelstudios) - Win64-Windows-7-VisualStudio-12: OK (2176 of 2196 tests passed)

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


It's better to write comment using english :D


Why is there a zip file?  And where did the tex file go?

I'm mildly against the icons because they don't seem to be related at all to the functions they're near.

In the future, you never want to have to related pull requests open in one project at the same time.  It makes it likely that the maintainer of the repo will have to duplicate work reviewing both of them.


@psvmcc да они его разблокировали буквально на прошлой неделе. Я им письмо написал - говорю - в реестре он уже не бьётся, на что они мне сказали что в их файле выгрузок он ещё есть, потом http заработал, а https - нет. Но тут же явно видно что у них с ssl проблемы, а не просто заглушка. Да и в реестр добавляют по IP, а не по протоколу, там тока http и значится везде.
```
curl -H 'Host: lostfilm.tv' http://104.28.16.79
```
работает правильно
```
curl -k -H 'Host: lostfilm.tv' https://104.28.16.79
``` ошибка от сквида прова


Never use the word `utilize`!!!!


WindowRevealBug (macumber) - 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)


>the files you wish to execute for the `Makefile`

makes no sense


You don't

> compile and run our code with a `Makefile`


You should definitely NOT talk about this first.


ありがとうございます！


是的

2015-03-20 13:45 GMT+08:00 cuihe <notifications@github.com>:

> 从0.5到0.85是更严格了吗
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cuihee/LockScreen/commit/6b32712f472917a116ba1387051807392bef6367#commitcomment-10300006>
> .
>


remove single quotes


The contents of this makefile is short enough you should include it all in one codeblock.


包括显示日历、显示待办事项的月统计数、当前日统计数、选中日期的待办事项列表展示、待办事项添加、删除等功能。


This is way too complicated for a first example.  You're using if statements and variables.  Someone who's never seen a makefile before will be lost.


I found out later this was redundant, as it extends LivingEntity.


Your demo is not as good as it could be.  You haven't shown the reader why you need that if statement.  If we didn't have it, what would happen if we ran `make` twice?  With it, what happens when we run `make` twice?


I find this troubling since pass now requires gpg (version 1) and I already have gpg2 installed and symlinked to /usr/local/bin/gpg. Any ideas how to handle this?


You always only have one default target.


This whole section adds no new material as far as I can see.   So why is it here?  What did I learn from reading this section?


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.57](http://teamcity.monogame.net/viewLog.html?buildId=5816&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 461, ignored: 6 Build time: 00:14:09


Me too, I tried several versions, but since libvirt always sets it and most of our test too, it should be all right.


this part is buggy, correcting it, by the time the inner loop goes to the next iteration, the outer is still on the previous so the match can only be for the first interation


This is not a good example of targets.  There should essentially never be repeated code in two separate targets.  Instead, that repeated code should be refactored out into a third target, and both the `all` and `foo` targets should depend on the factored out target.


Oops, the transient connector should pass an object `{count:count}` instead of a number too. @superkhau can you please fix it in a new PR? Feel free to land it without review if it's only one-line change.


@YJSoft '',act' 에서 ' 하나를 제거하니 발송창이 제대로 나타나는데요. communication.controller.php 파일의 수정부분도 마찬가지로 ' 하나를 제거해야 하나요 ?


Here we unifdef a block of code that we also add above...  Do we really need it twice?


Here we add a block of code that we unifdef below.  Do we really need it twice?


A two-stage write process more closely mimics what happens in a processor, and allows for cycle-level accuracy for memory reads and writes (in the typical naive implementation they would occur a cycle early).
In cycle N, the CPU places address, and value if a write, in the MAR and MDR respectively, and triggers a memory access.
At the start of cycle N+1, the data (for a read) is placed in the MDR and made available. Writes have completed by the start of cycle N+1.

Need to move to this style of memory accesses in instruction implementations.


API Legacy is made up of GET Requests. Check f643c082f5674ef789cd2972b3546fc7aeef8280 for more information.

Make sure that when you do `npm test`, tests for both API and API-Legacy are run.

Remove Karthik and myself from the copyright header, and add youself.


I would put that at the top, actually (see more below).


WindowRevealBug (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2098 of 2195 tests passed)

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


great stuff!


두파일에 ' 하나씩 모두 제거하니 정상적으로 메시지가 발송되네요.


:maple_leaf: https://www.youtube.com/watch?v=jKBLSkN2sRk :maple_leaf:


you're missing a condition similar to the hazelcast one. Is "ehcache.xml" present at the root of the classpath or the location param is set. You need this to avoid enabling ehcache if the user didn't intend to use it even though the library is there for some reason.


The class does not need to be public


Way to drag the PHP-ESAPI library into the 21st century, excellent additions (and deletions). I have code reviewed your changes and don't see any errors or abuses. I am pulling them to my fork which I hope to have one further code review, but am still waiting on someone more representative of OWASP to verify 1 that the google code migration happened without modifications to the source and then 2 get the recent code seen by one more official OWASP member so they can 3 also to merge my fork that will contain all recent work back into the OWASP repo. But, seriously, thank you for your quality contribution to the code base.


Redundant curly braces around a hash parameter.


I wonder, do we need this?


My bad.
How do I check for the captcha image being returned or not?
And the manual login through the API Legacy requires the captcha to be sent, how do I go about that?
Will make those changes


there is a location attribute in the main `CacheProperties`. you should reuse that.


Nice.  See a few in-line comments.


You should do it in more elegant way without magic.


any reason to verify `ModelState.IsValid`? I'm guessing not, but just an idea in case you like it.


Please only include your code in the PR. Having code reformat like this adds unnecessary noise.


Makefile variables are completely unrelated to bash variables.


ehCacheCacheWithCaches()


Bad grammar


You say this, but then don't say how to compile or run my code!!!


LGTM!


Modificación de Rekor para grabar datos en la tabla visitantes junto con el número serial del RFID


Let's move the whole sample thing to a different PR please. It will be easier to merge. Thanks!


develop (macumber) - 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)


what not writen in php ?


Your writeup still needs a lot of work.  As-is, I don't think your writeup improves on the existing writeup.  The main reason is that you cover much less material than the existing writeup.  But a secondary reason is that your writing is not very clear and contains a number of minor technical mistakes.

In its current form, this would be a 30/100.  Please work resubmit by Friday night.


测试一下


LGTM


и зачем я только код писал :C


There is a library called [is-bmp](https://www.npmjs.com/package/is-bmp). Use that to check if it is a BMP buffer or if it is something else.

You can call the Captcha Parser from here. But I suggest you ignore manual submit, just check captcha get and auto login. That covers it.


Rajesh Kajla


my


这里？


Quand je l'ai fait, c'était sans C++11 du coup ca me saoulé d'instancier un boost machin....
Mais maintenant je pourrai faire un chrono::duration.


Okay. Thanks!


Contact Us.
Manish Packers and Movers ™
Contact Person:      Mr. Rajesh Kajla / Mr. Manish Kajla
Mobile No:              09303355424 / 09301905324
Email ID:                kajla_rajesh@yahoo.co.in
Address:                79/2, Panchal Compound, Lasudia Mori
                             Dewas Naka, Indore - 452010 (M. P) India
Website:                http://www.manishpackers.com/
                             http://www.manishpackersmoversindore.in/


Better than checking the BMP, parse the captcha using the [captchaparser.js lib](https://github.com/karthikb351/CaptchaParser) and see if you get a valid captcha. Example usage is here - https://github.com/karthikb351/CaptchaParser/blob/master/examples/online.js


Do you plan on updating the version number and release?


Vast majority of these files are detritus -- not sure this commit will improve much. Need to come through these files with a chainsaw instead ;)


you're not talking about how to *implement* maps


Does the example for [libdefaults] make sense?  Wouldn't as-is always match and thus any later rules never be looked at?  Ditto for quality without a match_domain?  If so the example is confusing, because it is unrealistic.  The as-is rule should be last or should specify a match domain, ...  That is make the example more plausible as an actual set of rules, rather than unrelated alternatives.


Your title is singular, but then your first sentence is plural.  Be consistent.


Let's


This really isn't a great analogy.  The algorithm or finding your room given a hotel key would by O(n) because you have to try every possible room.


There is no "receiving" going on here.


sentence fragment


Looks OK


This needs to be higher up.


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/700dff346a3f4ca8263e5c2bb1a25f33a177caf0


merging basho/riak_cs/feature/update-ceph-tests = 700dff34 into borshop-integration-1096-feature/update-ceph-tests


basho/riak_cs/feature/update-ceph-tests = 700dff34 merged ok, testing candidate = 6ca47b05


`int`s


You need to run a spell checker.


This is not a template


found the change and commit, stay tune.


angle brackets


HotFixPurchasedAirUnitTest (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1671 of 1672 tests passed)

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


`string`s and `int`s


# Matrix inversion is usually a costly computation and there may be some benefit
# to caching the inverse of a matrix rather than compute it repeatedly. The
# following two functions are used to cache the inverse of a matrix.

# makeCacheMatrix creates a list containing a function to
# 1. set the value of the matrix
# 2. get the value of the matrix
# 3. set the value of inverse of the matrix
# 4. get the value of inverse of the matrix
makeCacheMatrix <- function(x = matrix()) {
    inv <- NULL
    set <- function(y) {
        x <<- y
        inv <<- NULL
    }
    get <- function() x
    setinverse <- function(inverse) inv <<- inverse
    getinverse <- function() inv
    list(set=set, get=get, setinverse=setinverse, getinverse=getinverse)
}

# The following function returns the inverse of the matrix. It first checks if
# the inverse has already been computed. If so, it gets the result and skips the
# computation. If not, it computes the inverse, sets the value in the cache via
# setinverse function.

# This function assumes that the matrix is always invertible.
cacheSolve <- function(x, ...) {
    inv <- x$getinverse()
    if(!is.null(inv)) {
        message("getting cached data.")
        return(inv)
    }
    data <- x$get()
    inv <- solve(data)
    x$setinverse(inv)
    inv
}


If I read this correctly, it picks the realm of the first principal in the keytab, which might not have the same principal name (sans realm) as the requested principal.  The code in kinit(1) has a more sophisticated matching function, that finds the best match in the keytab.  Perhaps that should be factored out into the library and used here as well as in kinit(1).


This is all very, very verbose.  Anyone who made it through cs014 should prefer a much briefer introduction.  You haven't got to anything interesting yet


Looks good to me.


Your whole declaration section up to this point could probably be removed, with only a couple sentence addition to your first example.


See inline comment re kinit(1) and matching principals with unspecified realms to keytabs.


No tabs!!!!


You need at least the function prototypes.


decided to remove the `.button` cruft, it was too much of a mess.


>There are many things that you can do with maps.

Then actually show them to me!!!  I don't like being teased :)


The reason why it went weird is because the list isn't fully thread-safe, see https://github.com/kaihu/python-elm-extensions/commit/1b9f12b105a96b32d8fb78451f8e7fbb43a47114 for a simpler solution.


You should have a full working cpp file included in the commit.


Performed this edit after aligning it with clustalw


Had issue with     setVars: Warning: Connection timeout. Retrying...
 Used Vagrant Destroy and then vagrant up again to fix issue.


(it's not a video featuring Rush but) aspnet/Mvc#2192 i.e. the model won't be valid because of reasons


Its not an good Idea to make it incompatible to older java versions but only 30% of the servers which are using WA are on java 1.8 all others are still on java 7. I would recommend to remove this check and compile it with java 7.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 264](http://192.168.30.225:8111/viewLog.html?buildId=990&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


revert back to original, need the type since vlan is not unified object id, however included lag id in the comment for SAI_ROUTER_INTERFACE_ATTR_PORT_ID


javascript disable이 제대로 안 되었던 듯? 지금 html 요청을 날리면 사실 에러가 나게 되어있음 ㅎ_ㅎ
왜 그럴까? @CarsomyrJ


I think you mean `addPost()`?


So cool.


There are 2 separate assertions:
`assertTopicNotificationReceived()`
`assertBranchNotificationNotReceived()`


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/934


CC @teddemunnik @neilogd


Topic Creator is automatically subscribed, why can't we test notifications on him?


more idiomatic:
```ruby
unless ['Staff', 'Coach'].include? current_user.role
```


@sschuberth yep, I agree, but then we really must not make the mistake of confusing a hot bug fix with a redesign.

I agree that we should give the idea of initializing our own version of the environment a really hard stare. I could imagine that we should instead look through the UTF-16 environment we get [here](https://github.com/git-for-windows/git/blob/1bd4e1e642f56da977d2a1db1f9bbb0cfb845a18/compat/mingw.c#L2209) and compare it with the available entries in the *current* `environ` as per Windows, modifying existing entries if they do not match, or adding entries if even the key names should be mangled. Then we could stop overriding `getenv()` and `putenv()` and use the environment just the same as cURL does.

However, the bug that this PR fixes held us up all this time, delaying the net installer unnecessarily, and as a consequence making it harder to work together on much more problematic issues such as the "Ctrl+C does not terminate children of spawned processes" problem. This delay created a problem in and of itself. I would appreciate it if we could avoid similar problems in the future.


You could skip the part `ifSubscribedToBothBranchAndTopic` to simplify the test case name. I think it's implicitly clear what's tested. WDYT?


Isn't the "+ nItems" unnecessary here, since the input and output are of the same size?


79453838-AutomateSomeDocumentation (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1649 of 1667 tests passed)

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


I forgot to track the files for the coulomb plots I made for the documentation.

@chrishanw  the data for the gravitational plots has already been part of the repository on this very branch and if the documentation compiles without errors, they should be in the pdf.
There are no individual png, but the commands of the corresponding tikz pictures can just be copy pasted into the presentation.


In principle the size of cubic and orthorhombicp lattices should be 3D vectors.


Я бы бежал от такого провайдера, хотя даже не знаю куда можно убежать, если по всех РФ блокируют. Ну а ssl, да явно с ним проблема, и они не особо ворочаются что бы его обновить :(


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 264](http://192.168.30.225:8111/viewLog.html?buildId=990&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 61 Build time: 00:11:14


This is just the first patch that made the tests pass for those upgrading to Rails 4.2.1 / 4-2-stable today. It needs a better look.


아 제대로 disable하고 에러를 고친 다음 했음^^ 어떻게 했을까? @CarsomyrJ


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2098 of 2196 tests passed)

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


Because pkgdb2 does not expose in its API packages that have 0 maintainers, and packages not exposed in the API will not be accessible via gitolite.


awesome :)


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 265](http://192.168.30.225:8111/viewLog.html?buildId=991&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Please *do not* commit to master please follow: http://docs.plone.org/about/contributing.html#workflow !!!
For this time I will let is pass, the next time I *will* revert !

Thanks !!


90179052-SetupEMSActuatorMissingDuplicates (jasondegraw) - i386-Windows-7-VisualStudio-12: Tests Failed

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


:heart:


Thanks, but I will HEAD back to safety – I use monex and don't like fatal shutdowns. Anyway, if upgrading log4j requires rebuild and restore, users should be told.


Error here:
UPDATE `creature_template` SET `ScriptName`= 'npc_colonel_jules' WHERE `entry`= 22431;

It must be:
UPDATE `creature_template` SET `ScriptName`= 'npc_colonel_jules' WHERE `entry`= 22432;


Indeed


This has very weird scrolling issues if you have a big header


the missing `throw` in a9c8df0 was the bad one ;)


I'm not sure this is the right way to solve the problem. Need more investigation.


界面没有经过设计师设计，都是个人偏好而就，美观程度有待考量。


90693826-Central-cooling-coils-not-autosizing-to-correct-airflow-in-some-cases-#4810 (rraustad) - x86_64-MacOS-10.9-clang: OK (1663 of 1666 tests passed)

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


嗨


gg


I think you mean recursive version?


你好


I need a "ByteBuilder" like StringBuilder here, but I can't find any. So I just implement it using a LinkedList, and manually process the bytes.


Here I must admit I'm at a complete loss to understand how/why KRB5_TC_DONT_MATCH_REALM is used.  I am not making any claim about this being a problem, but whatever the rationale is, it needs rather detailed comments and possibly even a design document that explains how all the moving parts fit together.

What prevents us from getting the wrong results here?


I didn't know that provided means it will not be bundled into the war!


fast-forwarding develop to borshop-integration-1096-feature/update-ceph-tests = 6ca47b05


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 265](http://192.168.30.225:8111/viewLog.html?buildId=991&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 62 Build time: 00:11:25


This jagged lines should be avoided. I think this caused by tab vs space confusion. Should be solvable in a standard way.


Should we also check that the given index does not exceed the maximum allowed index (size-1)? And just to verify: in update_node the indexes are not checked because we assume that the indexes are not modified between calls to get_node and update_node?


close this patch since another fix


You can install with `--build-from-source` to avoid depending on the `gpg` formula. If there's anything else we can help with, raising new issues is easier for us to track than commenting on commits. Thanks!


Good stuff !!


looks good to me


It doesn't look like this commit exists. I'm getting this error from `git submodule update`:

    fatal: reference is not a tree: 8d8f16e47eec1e0d7c1d907c42d86eb8dedb664b
    Unable to checkout '8d8f16e47eec1e0d7c1d907c42d86eb8dedb664b' in submodule path 'poopshow'

cf. https://github.com/esell/poopshow/tree/8d8f16e47eec1e0d7c1d907c42d86eb8dedb664b (which is currently showing me an error page)


Not really.  MIT license.


@ ybeapps
Reference `SubscriptTests.swift` line 204 `func testErrorNotExist`. The crash should not happen.


@mstahv any reason for not `... extends LazyList<T>`?


No idea.


79453838-AutomateSomeDocumentation (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)


留言测试


Done in the base class.


@psvmcc да было б на что менять...ну вот знакомый дал vpn до своего сервака в нидерландах - в выходные настрою


line note


Since you are the only person I see helping ios, I ask these questions:

this will make posible compiling mame 2010 or better for ios?
whats about psp core for ios?
when will be posible select dynarec/interpreter mode and configure core option?

thx.


方法名


changes


Question: is it ever the case that we end up with creds other than we asked for starting with realms other than the referral realm?  If so, should the test be for the empty realm, or for any difference between the cred's principal and the requested server principal?  Then write out both, and the referral name is just one kind of difference?


Why is this here?  Take it out.


Remove this heading


The two methods operate on copies of the $_FILES array. Can you please give some more details on what exactly is the problem you have? Just telling us that it fails doesn't help fixing it before the imminent Joomla! 3.4.1 release :(


This is the future.


GAAAAA!!! You're STILL not following the markdown formatting instructions of #1028.  Every sentence must be on its own line!


In strict mode, should we maybe throw in both cases?


all credit to #savoca. I dont remember what mistake I made to become the author


You should link to the patch command.


Tabs not working

whenever i switch to "form fields" or "rendered" tabs nothing works


Sorry about that, @mjgold! It's open now.


all credit to #engstk. I just added a few minor tweaks


might be 4


Overall, this is a pretty good explanation.  The one thing you're missing, though, is the motivation.

You should explain that this is what's going on under the hood when you issue a pull request via github.  Also, many larger projects like the Linux kernel don't use pull request on github.  Instead, they require everyone to submit their patch files to the mailing list.  I'm not entirely sure why they follow this workflow, but you should look up the reasons and incorporate them here.


Remove this header


Plz move this to db.


You never actually explained what rebase does or why it's different than merge


@tridge , wouldn't this render the "Flare crash.." string to never execute? If we crashed on approach we would show the flare message every time. In the if above this ti says (height <= g.land_flare_alt) || which means the instant we hit the ground we flare and that's it. is_flying() is not an instant detection, takes a few seconds for it to switch between the states. As is, it takes upwards of 5 seconds so this means the "Flare crash.." string wouldn't happen until after at least 8 seconds of crashing so the "Flare %.1f" would have surely fired by then.


should be backticks, not single quotes


just another example in external_tests, no problem


![2015-03-20 15_27_20](https://cloud.githubusercontent.com/assets/1590679/6747545/e440994e-cf15-11e4-933a-995815811ad4.gif)


As there were no new-style comments in this file, I opted for old-style comments for new function, to match surroundings, and leave changing comments-style for a format-only commit when this file is reformatted. But I'm ok to change them.


The namespace is not defined by our build system nor should it impose any limitations. So it's still up to ourselves to follow the rules of node/npm/component nampespace rules...


Yes, and that's what I'm finding when trying to debug problems on the live server. :smile:


Memory leak?  What happens to the previous value?


We should ping on bad transactions too! (I guess we'll get an exception, but Stripe does it :-P)


@kripken I'll take a look today.


Thank you for trying Nana, I encourage you to contribute this feature by using pull request. If you would like to contribute it, please commit your contribution to the Nana's develop or hotfixes branch.


Todo: simple example code.


You need spaces after all the `$`s


Not sure if I follow this change.


I don't understand your explanation.


Why are there two separate comparisons here?  Doesn't a single compare principal accomplish the same result?


It is used to indicate success or failure, separate from exceptions.


Remove unnecessary classes


You need more explanation about this.


Oops... looks like a copy-paste error


:see_no_evil:

![0fb](https://cloud.githubusercontent.com/assets/280212/6747602/023839be-cedc-11e4-90df-199455515d53.gif)


Add new Dbsets


You description of interactive rebasing is a bit too short and confusing.  You should either remove it or expand it.


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1648 of 1667 tests passed)

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


Put filenames in backticks.


I'll do this myself, see https://github.com/strongloop/loopback-datasource-juggler/pull/518


we should set the bootstrap CSS as dependency, i just checkt the PR and saw that the css file is on top.


#include "stdio.h"

int mian (void)
{
      printf("Hello, word");
}


Why isn't a qualified name used here?


Why so stingy with bits?  Especially for rule ordinals?


ok


For consistency with other methods, let's name the parameter `keyName`


Right. It always catches me by surprise (not in a good sense) when you silently push w/o creating a PR first.


The documentation indicates type is optional. I'm probably wrong.


You need to create a fresh git repo with `git init` so that all of these commands stand by themselves.


![2015-03-20 15_27_20](https://cloud.githubusercontent.com/assets/1590679/6747666/968b33a6-cf17-11e4-92c3-3711c9c2427e.gif)


西暦が奇数の時に-1されてしまう現象を修正


It seemed like a good idea at the time. I'll go back and double check.


yes, its OK, but please remove the line num 99 (...HEAD)


lol


If string strarts with `http://`, it can't start with `https://`.


![2015-03-20 15_27_20](https://cloud.githubusercontent.com/assets/1590679/6747693/f940f3fa-cf17-11e4-9f2b-d15713877b0e.gif)


Added terminal output of the Tor addresses.


oh, I see, I will fix it, and I will buy larger glasses :)


Overall, your content great!  You chose great examples to illustrate your points.  I also like that you added the pictures for the rebasing!  In addition to my comments above, there's one last content issue I have: please divide up this file into three separate files: one file for patches, one file for rebasing, and one file for stashing.  The three files should be completely independent of each other.  Do that and you will get full credit on the content portion of the assignment (50/50).

Your writing stills needs some improvements though.  There's just lots of spelling and grammar mistakes that need fixing.  I recommend asking another cs100 student to go through your writeup with you and correct the mistakes.  If you don't fix the mistakes, I unfortunately can't give you any points on the style section of the writeup, but if you do fix them you'll get full credit on the assignment.

Please let me know if you have any questions about my comments.


Bail if out of range?


As per our short discussion, you are correct, as we wait for Shannon to confirm, it makes sense to assume that without `-f` the CLI should prompt the user to confirm before deleting. Thanks.


Ok let's put it on gitignore  :sweat_smile:


![2015-03-20 15_27_20](https://cloud.githubusercontent.com/assets/1590679/6747713/5960b63a-cf18-11e4-8b1a-8585c46c3a4f.gif)


this is old syntax, please update to the new api


Sorry, I do not understand what you are exactly talking about and what is the issue you focus.
Lets focus here this commit.

Do you hear the difference between keylock and not keylock when using the reverse button in current master?
Without keylock I hear the pitch change. While this is desirable for all scratch actions, it is not desirable for for just reverse. For me reverse should behave just like a seek: not notable.
This commit, does not change the scratching behavior, because it is unlikely that you hit `m_speed_old == -speed during` scratch.
Do you think we have to put in more effort to detect revers change?

An other issue it targeted here:
https://github.com/daschuer/mixxx/commit/487a7575637dffbd027bb9e165f1f7f3defda7d8
Do you agree with this commit? It fixes bypassing the vinylemu in pregain when not hitting speed = 0.07
https://github.com/mixxxdj/mixxx/blob/master/src/engine/enginepregain.cpp#L129
Do you think it is worth to improve it? We may move the whole venylemu stuff to th linear scaler, and apply the 0.07 threshold there.


Is there a better way we could automate this? eg, a compressor filter, or something that triggers it automatically on a deploy?


Of course I think this would work better as a method also.  But waiting until you've had some coffee.


this is old syntax, you should use

```
looker.plugins.visualizations.add(viz)
```

instead of the VisualizationManager stuff


Extra empty line detected at block body beginning.


Only a few comments and please add the `-f` feature to confirm with user.

Finally, please make sure to rebase with CLI master. Seems like you are about 15 commits behind.

Thanks.


The ternary operator makes this a bit hard to read. I think this method and the `print` one could be slightly improved, e.g.

```
    def print(method, options = {})
      return if !Machinery::Config.new.hints

      Machinery::Ui.puts to_string(method, options)
    end

    def to_string(method, options = {})
      return "" if !Machinery::Config.new.hints

      "\nHint: #{send(method, options)}\n"
    end
```


I should fix this copyright.


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (650 of 651 tests passed)

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


related issue: https://github.com/MSOpenTech/opencv/issues/20


better variable name than attempts ?


The sort MUST be stable.  This can't stay unless I am missing something.  In particular you need a 2-level ordinal, first compare the user-specified priority if any, then the rule sequence number.


That should be `returns an empty string if hints are disabled` or something like that.


@xingzhou and @wsxiaozhang, is this the commit I should review?


Good job on the port!!!
This can go inside the `if let` block.  It can also be changed from `var` to a `let` :)


This does not cover the case where the user runs `machinery upgrade-format --all`, because in this case `name` is `nil`.


IDFXT (refaqtor) - i386-Windows-7-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-68.05%25-red.svg)


Arco where did you add the selinux permissions for this maybe is not working due to that ..... Same happens here without selinux permissions it doesn't work


Is removeClass("top") still needed here?


Content:
- What if the long description could also be pulled from a URL; e.g. the repo README. Could avoid some duplicate info.
- Is the author the maintainer?
- What is the default repo url?

Cosmetics:
- Order of the fields (title to the top?)
- Maybe group optional field together, mark them clearly


In-line comment is not necessary.


Made the change based on your comment above. This PR seems to have gone stagnant and I wanted to do something. Do you have an alternative idea on what should be done?


Может попробуем извратиться. Я не с России, у меня пока ничего не банят. Думаю, что таких пользователей у программы полно. Если запилить механизм, который будет закачивать xml с лоста и выкладывать его на некий 3й ресурс. А в ТМ прописать, если с основного ресурса не удалось прочитать, пробовать читать с резервного.


This line class variable seems redundant, why not just use the verticalDirection directly as the class value.


@ayushagarwal95: Check API-v2 Tests and do update the tests for API-Legacy accordingly.


Has approx 7 statements (TooManyStatements)


Has approx 10 statements (TooManyStatements)


I think I'd prefer "top-to-bottom" and "bottom-to-top" as the options, it seems more specific in what it's for.


It can actually be removed entirely since I override it inside the if scope anyway.


"reveted" is not right in your file name.


For what it's worth, I don't think that's an extra newline. It's what EY used to indicate that Harry couldn't even contemplate Hermione's half-eaten body as an object which existed. The same device is used immediately above, in the paragraph which ends without saying _what_ the other hand held. The troll isn't dropping into the red pool; it's dropping Hermione into the red pool.


Generally your code formatting should be tidied up to remove the unnecessary whitespace and match the existing formatting. I'm a bit OCD about code formatting.

"horisontal" should be "horizontal", if that variable is still needed.


That's true. Fixed in 6317bfce5de495b3d581807abd351257d1de017c


I'll list a few


IDFXT (refaqtor) - Win64-Windows-7-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-68.05%25-red.svg)


I am wondering if this optimization can backfire. This for loop will grow the pool by 2 connections every time a checkout is attempted. So effectively it will take the connection count from 0,2,4. If the max connection pool is 60 (for eg. what we have done for lego backend), 5 calls to nonblocking checkout would crete 10 connections. My point is, this optimization could lead to a lot of unused connections especially when the pool size is a large number like 60.


Hmmm, OK, if it's in the source material I guess it's best to leave it then. It seemed really jarring when I read it.

I think it's the troll dropping though, "into the red pool that had spread out beneath **its** feet, raising **its** club high."


Why do we need this?


Why? For virtualbox NFS is not the best choice because of write permission issues. That way you always need a running bindfs installation within the vm.


space vs tabs


I have not enabled it because we need another formatter that displays some kind of timestamp as well.


Awesome +1

Is it possible to post some examples in the Forum with new features?


* https://github.com/nickma82/nut/blob/master/drivers/nutdrv_qx_voltronic.c#L1844
* https://github.com/nickma82/nut/blob/master/drivers/nutdrv_qx_voltronic.c#L1922-L1923
* https://github.com/nickma82/nut/blob/master/drivers/nutdrv_qx_voltronic.c#L1922-L1923
* https://github.com/nickma82/nut/blob/master/drivers/nutdrv_qx_voltronic.c#L2312-L2313
* ....


But we want to enable it again, because of that i just disabled it within the application controller.


I'm totally interested in this kind of runtime errors within development environment.


The default virtualbox synced folder was incredibly slow for me, this is tons faster. No need to grab this one if you don't think it's a good default. :)


can you add some check here to avoid the remount when not necessary please?
or moving the mount to another hook in build-vm-$type


I inexplicably had this section in the POM twice, with a single configuration parameter changed.


@sumghai okay I love you


Doesn't make sense to change that yet. The dashboard is not done yet :)


In this particular case, creating a new err with this scope didn't cause any bugs.  But the change is good in general.

But I've run into issues in the past where I've created a new err variable inadvertently and it did cause problems.  Is there a tool which will scan for these sorts of redefinitions?


The ruby:extensions repo has haml-rails 0.9 now, so ~> 0.8.2 doesn't resolve.


fsa


Das war auch das erste, das mir auffiel


Alright, I just needed some kind of logging to see what was going on.


So you want to create these symlinks only within the rpm?


Feel free to ignore things like this that you are still working on, I'm figuring out what I can do with twitter bootstrap and rails 4 :)


Wait p.hasPermission(null) does not throw an error? thats cool :D
I made it essentials.me because I considered this a temporary replacement
untill the syntax of essentials is fixed.
Everyone has essentials.me though, so it should always send it to everyone.
(Visitor = everyone :P)
Also when you add nodes to visitor, reload PEX before expecting it to
inherit to the other groups. Just saying.

On 19 March 2015 at 21:54, jomo <notifications@github.com> wrote:

> *(Also we shouldn't use Essentials' permission nodes for our plugins.)*
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RedstonerServer/redstoner-utils/commit/40fa9c0692ece6c7230889546122876a4aa36255#commitcomment-10294602>
> .
>


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


After your discussion of the purpose of this feature, I feel like you try to use ```tf``` for something it was not designed to do. Personally, I still do not understand your use-case -- do you want to build a simulation using tf? Anyway, I do not think putting this feature into this repository is a good idea, for the moment.

> > Making the topic latched is not correct. TF2 comes with an extra latched topic over which static transforms shall be published.

> Ok, so how are you going to publish on a tf_static? The point of latched topics, as we all know, is that all new subscribers get the last published package delivered to them. In that sense, it is the publisher who defines if the topic is latched and not the subscriber. In that sense, saying that "TF2 comes with an extra latched topic" is not exactly correct as buffer_server is only subscribing.

The ```tf2``` buffer_server subscribes to two topics: ```/tf``` and ```/tf_static``` where ```/tf_static``` is reserved for latched messages, i.e. static transforms. You can find the rationale on the design page:
http://wiki.ros.org/tf2/Design

As a result, there are two C++ broadcasters:
https://github.com/ros/geometry_experimental/blob/indigo-devel/tf2_ros%2Fsrc%2Fstatic_transform_broadcaster.cpp#L42
https://github.com/ros/geometry_experimental/blob/indigo-devel/tf2_ros%2Fsrc%2Ftransform_broadcaster.cpp#L42

And the C++ listener has to subscribe to both topics:
https://github.com/ros/geometry_experimental/blob/indigo-devel/tf2_ros%2Finclude%2Ftf2_ros%2Ftransform_listener.h#L70


Thanks!


I removed these because I tried using the rsync synced_folder implementation, and it failed if there were dangling symlinks. Either way it's odd to have them here, they are completely SLE specific. It would make more sense to have that as patches in the SLE OBS repository in my opinion.


This will be fetched through AJAX


Great! When will you make a release of this?


Adding clubtv


oops this file was auto created when i reinstalled simple form w bootstrap and i forgot to remove it


But than we have to test it against 0.9 and if everything still works fine we have to increase the version constraint :)


The crash happened to me and that change solved it!
My Mac is not working now so I can't check what is the differences between the cases...
I think my commit is safer anyway... Do what you think...


Mark the rule as broken?  We should not use partially typo'd rules.


If we decide to install RSAT-Hyper-V-Tools-Feature, then this should be removed, because is part of the RSAT-Hyper-V-Tools-Feature. Also the reboot should happen after installing RSAT-Hyper-V-Tools-Feature.
The dependencies can be seen here http://vniklas.djungeln.se/wp-content/uploads/2013/01/Screen-Shot-2013-01-29-at-21.09.51.png


But as i said, than you need to enabled the bindfs stuff for virtualbox as well, if you are not doing that the processes within the vm are not allowed to write to /vagrant (including tmp and log folder).


![if not perm](https://i.imgur.com/kyxrxRB.png)


Sure, the first step was just to get it to install :) It seems to work so far for me with 0.9. But why increase the constraint if we are also OK with 0.8?


The logging is enabled... If you don't get the logsa it's related to the NFS virtualbox mounts :)


Also set type to bogus explicitly instead of relying on calloc() and the bogus enum being zero.


@Marti This commit has stopped the bleeding, but I'm not sure if I didn't break some authentication related things due to moving passport initialize before the session middleware. The reason I did this is because passport modifies the session data and which forces that change to be saved, which means every unique visitor creates a session record.


gng


So let's integrate it into the RPM only.


logically tabs you should be here else {} block but it is not present, or am I wrong?
if (m_SeekBarActive)
	m_SeekBarActive = false;
else
{
	m_SeekBarActive = true,
	m_SeekBarActivatedTick = CurrentTick;
}


I will ignore this as well as i want to fix it differently :)


I didn't touch any bindfs stuff and the vms are writing to /vagrant just fine


No reason. I wasn't sure how AST nodes are compared and to be sure used their tokens.


thanks


So maybe just remove the last digit from all dependencies? It should work most of the time quite well.


No, no need for bindfs. It's not even installed in the VM, and the VM is writing to the log folder without issues.


And for our RPM's we are anyway bound to the versions provided through other RPMs


Don' forget to free also match_domain and match_realm.


Yeah, I am fine with that. Exactly, we don't really have any choice..


If at the end of the previous loop k < n, there are (n-k) bogus rules at the end of the list.  They need to be freed.  All the rules at the front of the list (0..k-1) are valid.  Discard the bad rules *before* the qsort().  Otherwise you have a mess.  The zeroth rule after qsort might be bogus, and proper linkage might never happen.  The invalid rules are  then leaked memory


Than you are just in the luck that your host user got the same id like the vagrant user within the vm.


imagine how many other "return"s in file and how many other "__init__"s. If function is not __init__ we quickly stop process.
And we should raise issue on __init__. It would be difficult to check whether issue is already raised on this __init__ before or not.


what is this file


Fixed


Well, I wasn't getting the logs before I tried NFS.. so that can't be it. :)


But it's some kind of duplication


:+1:  Perfeito!


@mherger Difficult question, because I can't upgrade to jessie with 5.20 as that would break my squeezeboxserver :-/ I unfortunately don't have a spare device to play around with.

IIRC armv6 doesn't have an FPU though, thus your installed architecture definitely isn't hf. perl -v on my wheezy install says

This is perl 5, version 14, subversion 2 (v5.14.2) built for arm-linux-gnueabihf-thread-multi-64int

I just noticed that the Pi indeed is ARMv6 only; it's only the Pi v2 which has ARMv7 and thus can use armhf: http://sjoerd.luon.net/posts/2015/02/debian-jessie-on-rpi2/

Sorry for leading you in the wrong direction :( I assume you can't build the armhf libs in your setup then? Would you accept a pull request for those binaries if I manage to compile them?


Ah right yes, that's probably true.


The "safe_name_canon" feature seems to be undocumented.


The label says underflow, but the message says over. Is this a typo?


Hey Vicki,

Those are good points and I really like the example from Atlassian that you
linked too.

My concern if we do the same ourselves is that many customers may assume
that we are supported all recent Sharepoint versions. That is true in the
case of Atlassian, but it is not true (as of now) in our case. When this
happens I think it is important to explicitly state what are the versions
that we do support. Something like:

The Sharepoint connector supports Sharepoint 2007 and 2010.

We can even say that support for 2013 will be coming soon if you think that
would help.

If we don't say that, then customers assume it will work with 2013, only to
find out later (when they are stuck) that is not the case. By then we have
an angry customer which requires a lot of time from support and creates an
urgent escalation in Engineering. This is a very high price to pay and even
if we do a perfect job the customer won't be as happy as before.

A recent example where something similar has happened has been from a
customer where Minhchau was working. They had made a large investment
assuming that Site Templates worked in a certain way, the documentation
very explicitly says that it does not work that way but they had not read
the documentation. When they found out (with trial and error) they were
very upset. Even Minhchau was very upset saying that we *hide* this very
important information in the documentation. For me the lessons from this
case was that information that is very relevant for the customer to
determine whether they will be able to use an App or not at all should be
very clearly visible. If we just put it in the documentation many of them
will miss it.

The documentation can then expand on it if necessary. But I don't think the
documentation for one specific App should define terms such as "Officially
Supported" which are not unique to them. They should be documented in a
more general place.

Thoughts?

On Thu, Mar 19, 2015 at 1:51 AM, Vicki Tsang <notifications@github.com>
wrote:

> Hi Jorge,
>
> We've tended to do the opposite, since the MP page is more of a marketing
> page to "sell" the benefits of the plugin (and we've turned down support
> requests to add known issues to this page"). I was checking out the
> Marketplace and I noticed that they also keep this type of information in
> their documentation:
> https://confluence.atlassian.com/display/SPCON/Comparing+SharePoint+Versions+and+Editions
> and not on their marketplace page:
>
> https://marketplace.atlassian.com/plugins/com.atlassian.confluence.extra.sharepoint
> .
>
> I don't know of any other places this information is kept - there has been
> an ongoing conversation about this with support and QA, since internally,
> we haven't quite been clear on what "versions" we support. Additionally, we
> still need to explain the general CMIS integration.
>
> I'm not 100% against having this info on the MP if it can be made to be
> unobtrusive, however, I think keeping it in the documentation might allow
> us more flexibility to provide more in-depth information.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/liferay/liferay-docs/commit/69440e9253c2417bde0f5554e488a72ffabe46ef#commitcomment-10276949>
> .
>

--
Jorge Ferrer
Vice President, Engineering
@jorgeferrer <https://twitter.com/jorgeferrer>

*Liferay Inc*
Web <http://www.liferay.com> | Facebook <http://facebook.com/liferay> |
Twitter <http://twitter.com/liferay>

*Aviso de Confidencialidad: *Este mensaje de correo electrónico, incluidos
los archivos adjuntos, enviados en nombre de Liferay SL es para el uso
exclusivo del destinatario (s) previsto y puede contener información
confidencial y privilegiada. Se prohíbe cualquier revisión no autorizada,
uso, copia, divulgación o distribución. Si usted no es el destinatario, por
favor póngase en contacto con el remitente por correo electrónico de
respuesta y destruir todas las copias del mensaje original. Gracias.

*Confidentiality Notice: *This e-mail message, including any attachments,
sent on behalf of Liferay SL is for the sole use of the intended
recipient(s) and may contain confidential and privileged information. Any
unauthorized review, use, copying, disclosure or distribution is
prohibited. If you are not the intended recipient, please contact the
sender by reply e-mail and destroy all copies of the original message. Thank
you.


const Common::String &setup


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1162 of 1165 tests passed)

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


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.58](http://teamcity.monogame.net/viewLog.html?buildId=5819&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 460, ignored: 6 Build time: 00:14:03


&-alignment


Has this been tested? I think you may need to load string_underflow into a register before accessing the address (e.g. sw $a0, 0($t0))


Да там же не только в rss проблема, его можно и так скачать, но и с последующей загрузкой файла, где нужна авторизация на сайте :)


It's uncatchable John!


There isn't anything stopping you from creating tightly coupled interfaces in a strongly typed language. The fact that you are guaranteed the methods exist just happens to be a convenience. What it means to decouple interfaces is that the two classes can be instantiated and used independently, which is important for BDD.

Any class to expects to call a method on another class reference creates a loose coupling. In some cases this is tolerable, e.g. I can test `me.Vector2d` without a `me.Entity`, but I cannot test a `me.Entity` without a `me.Vector2d`. The coupling only goes one direction.

On the other hand, I cannot test a `me.Entity` without a `me.Body`, and vice versa, because of the mutual dependency. This is a tight coupling. The behavior of Entity relies on the behavior of Body which relies on the behavior of Entity. It's utter madness.

Callbacks are by no means an ideal solution, but they can be used to break the "formal contract" of the mutual dependency. I'm open to other ideas; message passing (pubsub), DOM-style onWhatever callback properties, ...


> `p.hasPermission(None)` does not throw an error

it doesn't get there.


Just posted!

On Fri, Mar 20, 2015 at 9:03 PM, 7sevenx7 <notifications@github.com> wrote:

> Awesome +1
>
> Is it possible to post some examples in the Forum with new features?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/blitz-research/monkey/commit/4e42bae58122397697e0256060560bdeee34cb02#commitcomment-10301295>
> .
>


Is the use of the Assembler temporary intentional, or should this be ori $t0, $t4, 1 ?


This was when building for Debian Jessie using qubes-builder in a Fedora 20 VM.  If you want to see all the errors, try building with qubuntu; they are the most strict.  Going to be submitting another one in a few minutes :)


I agree, this deactivate KVO for properties set to the AFHTTPRequestSerializer instance meant to be transfer to the actual NSMutableURLRequest. I had to patch this file for timeoutInterval was not taken into consideration.


develop (jasondegraw) - 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)


Thanks mark!

An other question, can you not merge this pull from me? its small but has a big effect..
https://github.com/blitz-research/monkey/pull/67


Did you manage to convert "SURF" Akaze descriptor to uchar values ?


Yes, it will - it will also crash anywhere in mono that tries to access a similar object.
I'm not too sure about this fix:
 * It just hides the problem (why are we getting an invalid MonoObject?)
 * It only avoids the problem in one place, when there are many places where we dereference MonoObjects
 * We shouldn't add a magic number check (0x08040000)
 * guint isn't large enough to hold a pointer value

If you have a use case that reliably reproduces this issue, we'll be happy to take a look at it.


If we rip the /s out we need to add them back. "/" is a perfectly valid character for branch names.


Good thinking and consideration.

The call to calc_begin is currently handled by the interrupt routine. If this is replacing the old code, let's start a GitHub discussion on the change, and validate the design. Otherwise, this line becomes as simple as jr $ra


bjjj


develop (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1163 of 1165 tests passed)

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


Here's the another compile error I received with trusty.  I have VERBOSE set to 1.

```
gcc -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Werror=format-security -I. -g -Wall -Wextra -Werror -pie -fPIC `pkg-config --cflags vchan-xen` -D_FORTIFY_SOURCE=2  -c -o qrexec-client-vm.o qrexec-client-vm.c
qrexec-client-vm.c: In function 'main':
qrexec-client-vm.c:172:12: error: 'child_pid' may be used uninitialized in this function [-Werror=maybe-uninitialized]
     waitpid(child_pid, &i, 0);
            ^
cc1: all warnings being treated as errors
make[3]: *** [qrexec-client-vm.o] Error 1
```


This is wrong.  Don't use strstr().  Compute the tail offset, and compare tails making sure that hostname is longer and has a "." just before the tail matching the rule's domain.


You are awesome! Keep up the good work! This will be a very nice addition and of great help/use to the htm.java and nupic community!  :+1:


develop (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1163 of 1165 tests passed)

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


Should this be a `Name` rather than an `Ident`? Crate names shouldn't be affected by hygiene.


Здесь нужно проверить как себя еще будет вести reversion с шаблонами. Как будет работать восстановление истории и тд и тп. Так же поведение на всех поддерживаемых версиях Django.


The phrase `is not supported` means that someday it may be supported. I prefer `is not allowed` here.


Thank you so much! So *this* is what that error message means. I got very confused about why it would not be able to find the library in some cases.

Anyway, these libraries come straight from homebrew. I really don't want to recompile all of that every time I package PySoundFile. I think we'll therefore have to live with the dynamic libraries.


Again strstr() is wrong here.  I actually avoiding duplicate tails is wrong.  If a suffix works for some hosts, it should work for all hosts, even those already ending in that suffix (and certainly merely containing it is not right).


All this should go to `Select` method, not `Where` method :)


It seems to be necessary for building the wheel. Most of the time anyway. `python setup.py bdist_wheel` has been acting very inconsistently for me.


@legaultpierre:
- Steve Rayner, Columbia University
- Ravi Sharma, United Nations Environment Program (There is a mistake in how this person is classed for Country; India is the country, UNEP is the institution
- John Robinson, University of British Columbia
- Kirit Parikh, Indira Gandhi Institute for Development Research


Why? I always was sure, that we firstly include non-detailed headers.


lol, you're right. I didn't test it on Windows yet.


Appropriate cannot build a chain.
http://yugioh-wiki.net/index.php?%A1%D4%CA%D8%BE%E8%A1%D5#t1d7cc30
Ｑ：２枚ドローする効果はチェーンブロックを作りますか？
Ａ：チェーンブロックを作りません。(14/08/30)


Working on Magento CE 1.9.1.0 .


@legaultpierre:
- John Robinson, University of British Columbia
- Bert de Vries, National Institute for Public Health and Environment (RIVM)
- Jyoti Parikh, Indira Gandhi Institute for Development Research


Typo in the comment in `03-urls.cpp` ; *must me freed* -> *must be freed*

You really should add a comment about the `owner` boolean flag. What about multi-threaded application? C++11 seems to have a more precise ownership concept (with its several smart pointers template classes).


As mentioned before also match_realm and match_domain.


LOL


@legaultpierre
- Ken Gregory, Centre for Business and the Environment
- Lynn Price, Lawrence Berkeley National Laboratory
- Peter Zhou, Energy, Environment, Computer, and Geophysical Applications Group
- Frank Ackerman, Tufts University


Should this be a `Name` rather than an `Ident`? I don't think crate names are affected by hygiene.


90693826-Central-cooling-coils-not-autosizing-to-correct-airflow-in-some-cases-#4810 (rraustad) - i386-Windows-7-VisualStudio-12: OK (1669 of 1672 tests passed)

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


@legaultpierre:
- David Hall, King's College London


LOL!!


@legaultpierre:
- Tianzhu Zhang, Tsinghua University
- Tim Taylor, University of Bath


The following files were missed:
release/Magarena/scripts_missing/Arena.txt
release/Magarena/scripts_missing/Sewers_of_Estark.txt
release/Magarena/scripts_missing/Ugin's_Nexus.txt
release/Magarena/scripts_missing/Kin_Tree_Incovation.txt
release/Magarena/scripts_missing/Nalathni_Dragon.txt
release/Magarena/scripts_missing/Shambling_Attendantsr.txt


@vlmaksime так а что у тебя за проблема с lostfilm? может тебе просто нужно обновить openssl как поступил @Upitep выше?


Haha, thanks. :smiley:


Lots of work here...  Do we REALLY need name-canon rules?  Lots of complex code. Who is going to use this?


@legaultpierre:
- Jae Edmonds, Pacific Northwest Laboratory
- Knut Einar Rosendahl, Statistics Norway
- ZhongXiang Zhang, University of Groningen


wwwwwwwwwwwwwwaaaaaaaaaaaaaaaaaaaaaaaaaaaaahhhhhhhhhhhhhhhhhhhhhhh


Does the test really check what it should if its expectations are defined in the tested code?


:+1: for orange color :)


@legaultpierre:
- Lee Ann Kozak, Southern Company Services
- Sudhir Sharma, United Nations Environment Programme (UNEP)


Let's not leave any comments. Remove if not needed. Can always add later.


I believe this change is causing the errors in being reported in #3.


@legaultpierre:
- Jae Edmonds, Pacific Northwest Laboratory


Hi, With this modification, I have a path error (php 5.4) :
 Warning: include(****\filemanager\include/lang/en_EN.php): failed to open stream: No such file or directory in ****\filemanager\include\utils.php on line 70
But if I replace par__DIR__ no error


Всем доброе утро)) Продолжаем мучать lostfilm..


``kwargs.pop('parent_field', None)``


Good point)


``parent_field and parent_field.widget.attrs.get('id')``


I know this is for future story, but remove in final commit. You can recreate later by looking at other controllers.


Extra line. Please remove.


Wont this break functionallity in the listener?

```java
                case Sensor.TYPE_ROTATION_VECTOR:
                    System.arraycopy(sensorEvent.values, 0, rotationVector, 0, 3);
                    break;
```


So here you are only creating these `Errors` object and not raising them. Is that the intent?


@legaultpierre:
- Jae Edmonds, Pacific Northwest Laboratory, USA
- Ken Gregory, Centre for Business and the Environment, UK
- Jose Roberto Moreira, Biomass Users Network, Brazil
- Tsuneyuki Morita, National Institute for Environmental Studies, Japan
- Lynn Price, Lawrence Berkeley National Laboratory, USA
- Richard Richels, Electric Power Research Institute, USA
- John Robinson, University of British Columbia, Canada


Weird I cannot recreate on the latest in development.

I've enabled/disabled the DEBUG define and it compiles fine.


please add the namespace std::


same here


msg_count is better than msg_ct


`parent_field` is required
id of parent_field widget is not, so it's ok or i don't get you concept :D


That's interesting - mainly the promo cards from the book releases (Arena, Sewers of Estark, Nalathni Dragon - Although Arena was reprinted) Will have a look at the builder

The other two look like typos

Shambling_Attendantsr.txt -> Shambling_Attendants.txt
Same for Kin_Tree_Incovation.txt -> Kin_Tree_Invocation.txt

Those files should be removed, as the correct ones are already there.


and here


thanks for the update!


@legaultpierre:
- Lynn Price, Lawrence Berkeley National Laboratory
- John Robinson, University of British Columbia
- Tim Taylor, University of Bath


std::


Extra line. Please remove. RuboCop will complain also when running all unit tests.


I don't think so. Issue should be raised on a function, and it's easier to analyse each function instead of each statement.


Same here. Extra line.


Doesn't this create a problem if sheetName starts with ' already?
I haven't had the chance to test it, btw


Extra line.


Original ARMv6 (bcm2708) Pi can run hard-float and has a fpu. Binaries compiled with '-march=armv6 -mfloat-abi=hard -mfpu=vfp' will run on both original Pi and (bcm2709) ARMv7 Pi. Binaries compiled optimised for bcm2709 with flags, '-march=armv7-a -mfpu=neon  -mfloat-abi=hard', would not be backward compatible.


Should this be `:service_instance` or `:service_instance_id`?


@salimhb you are right, I didn't think about that at all!
Wouldn't the following be more "bulletproof"?
```
var thumbnailLink = dom_widget.find('.img-polaroid, .img-thumbnail').first().parent();
```


79453838-AutomateSomeDocumentation (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1159 of 1160 tests passed)

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


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (653 of 655 tests passed)

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


Although I don't think that ```mCursorAdaper``` can ever get into a null state, I would like to see this in a null check.
```java
if(mCursorAdapter != null && mCursorAdapter.getCursor() != null){
    mCursorAdapter.getCursor().close();
}
```


:ghost:ghost:


Wrong indent?


Thanks :)


@legaultpierre yeah, let's discuss. I think maybe for these ones we should just say IPCC Technical Support Unit (TSU) w/out the WG, this way the category stays broader than just WGI. This "institution" changes location, but there is one for each WG each year...otherwise we get stuck in trying to track down their original institution, which becomes rather difficult...


remove comment before submitting PR


Extra lines...


Remove and also new line


Extra lines


Extra line


Fixed in 70f121e5b4b5a21a96eee6ff2e51a0596d22cfe7


Extra lines == EL


@Dispersia
From joduska.me :

https://img.joduska.me/?q=https://i.imgur.com/SyXPnvP.jpg

latest build(.98) i can't right click on friend or anything. it did likes left click but .96 is work fine.

and friendlist look weird when i click someone it normally highlight blue then click on whatever it still highlight at white and didn't disappear. not a big trouble but made me feel "WTF it hard to see that name QQ"


EL


Dzięki za podpowiedź, zasugerowałem się tym, że "u nas działa" :P Będę musiał w takim razie zanurkować głębiej, i chyba w ogóle napisać testy dla tego typu enumów, bo nigdzie nie widzę testów dla `sqlType="ENUM(//...)"`.


aaa


90693826-Central-cooling-coils-not-autosizing-to-correct-airflow-in-some-cases-#4810 (rraustad) - 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)


@Dispersia I think this commit causes the problem.


@wiktor256 Since it was a significant contribution, added you as a contributor in the package.json, assuming you'd like this.


I'll take a look at your updated commit. Also, don't forget to rebase code.


Now hte image is completly gone :(


@shakusa This file is now better synced with the internal version. Note that the internal and external CapUtil.java are not identical because the apache library is slightly different.


should this commit not be squashed with one of the earlier commits?


This is what makes collection mode fail. You only need to run one iteration :-)


why do you use double underscores at the beginning and end of this global variable?


Возможно была проблема с curl и ssl. У меня из-за отсутствия сертификата не хотел работать PushBullet.
Полечил следующим образом:
1. Скачал файл http://curl.haxx.se/ca/cacert.pem
2. В php.ini в параметре "curl.cainfo" указал путь к файлу сертификата.
3. Перезапустил службу сервера.

После этих манипуляций у меня заработал PushBullet через https протокол. Может кому-то будет полезно.


what does this change have to do with storing the default dpi in a global variable?


or this one


dpi is not an integer but a float, so you should not use `%d`


Xenotix v4.5


yeah, that would work too. I'll try it out.


in this commit you are moving around the location of `pagesize_options` - could you make the pull request such that unnecessary changes of this kind are not included?


I think this should be `self.eat_keyword(..)` instead, since we *are* expecting it now.


Same with the refactoring of the arguments to `get_ndpi` - this should've been done right in the commit that introduces `get_ndpi`


Yes this should be a separate PR, sorry.

This is necessary for PHP 5.6, because noTrimWrap is not a static method from the cObj. Until PHP 5.6 this only caused a WARNING: "static method call on a non-static method". Since PHP 5.6 this is forbidden and throws a RuntimeException.

The TSFE always contains an instance of a cObj, so we can use this to call the non-static method.


this line is becoming way too long - please wrap it at 80 columns