Nullability annotations.

Why not add a method for adding functions, thus incapsulating your builder's implementation details?


True. We would have div by zero if width == 0


Sorry. Ignored this file while committing. Will push a change ASAP


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.831.155a88dab5dc384263d97f753c4fd23d9080f575](http://build.palaso.org/viewLog.html?buildId=60060&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:02:31


Can you amend the commit message to describe what effect this has, e.g. which platforms?


I guess these settings could starve resources on the host if many containers were to use that much. In that case, at least increase LimitNPROC to 65536. I just took the default that docker systemd unit ships with. Might be a bit high.


Hah, thanks!


This is strange. It is correct that "subscriptionChangeWeight" does not return a "method" field, but it should return a "seq" field, which pvr.hts has filled in the message before sending it to the server:

https://github.com/kodi-pvr/pvr.hts/blob/master/src/HTSPConnection.cpp#L318

But somehow this "seq" either is not returned by the server (but I doubt this) or pvr.hts cannot find it:

https://github.com/kodi-pvr/pvr.hts/blob/master/src/HTSPConnection.cpp#L245

No clue... but there definitely goes something wrong. And as @Jalle19 said just setting response time to 0 is not an option. I'm afraid you will have to debug this, @Glenn-1990


Sorry... it should actually be `deis-zookeeper`


The election scraper should *not* be in this repo, it should be in the https://github.com/datamade/chicago-openelex repo. That repo should be a namespace package. http://stackoverflow.com/questions/1675734/how-do-i-create-a-namespace-package-in-python


Thanks for catching my typo!


since you're not using lazy then this means this will be evaluated on restart instead on request


Current master values are
                                                          //r2     r3      r4         r5         r6          r7
Value PassedPawnsBonusMg[6] = { V(0), V(0), V(34), V(102), V(204), V(340) };
Value PassedPawnsBonusEg[6] = { V(7), V(14), V(35), V(70), V(119), V(182) };

You can try any starting value you want, but
keep in mind that SPSA tuning will not help at all if your starting values are not close enough from an optimum.

Also, replace the V(0) with V(1) if you want to tune it. (May be you don't)

You are on your own now. Good luck.


Removed.


I'll be in IRC in 10-15 minutes to help out with this.


I think all that's needed is the example and test files. The `.fsproj` file can be omitted. :}


noted here: https://github.com/numenta/numenta-apps/blob/master/unicorn/gui/browser/app.js#L59
thanks


Thanks for the suggestion Josh.

Best,

On Mon, Jul 27, 2015 at 1:03 AM, Josh Goldberg <notifications@github.com>
wrote:

> Wow, this is probably the biggest GitHub diff I've ever seen. :)
>
> You might want to pull again, then rename to FullScreenMario. Development
> on my end is about to pause for a week or two, and a bunch of unit tests
> were just added in, so this is a good time to fork the project.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ajrulez/FullScreenMario-JSON/commit/3661b0d171db27c5e88423bce3a5f971893f7049#commitcomment-12366746>
> .
>


:+1: :-)

On Mon, Jul 27, 2015 at 1:33 PM, Brev Patterson <notifications@github.com>
wrote:

> Third time is a charm? Lock down changes to `docs/htm.java` dir. Verified
> all changes are here. See PRs #219 #220. cc @cogmission
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/brev/numenta.org/commit/2ca85bddd900a927c7344a8012e9209ca9c1b4b7
>

--
*With kind regards,*

David Ray
Java Solutions Architect

*Cortical.io <http://cortical.io/>*
Sponsor of:  HTM.java <https://github.com/numenta/htm.java>

d.ray@cortical.io
http://cortical.io


maybe you should not use 'border' since it expands the element (adds 2px to the width), just fill all divs with different colors.


Done:

- Used [gox](https://github.com/mitchellh/gox) for cross-compile
- Binaries were uploaded at v0.5.0

To accelerate release process by uploading the binaries programatically to github worth checking [github-release](https://github.com/aktau/github-release)


I am sorry for that will do for any future changes!
Am 27.07.2015 20:22 schrieb "David A. Velasco" <notifications@github.com>:@AndyScherzinger , please, don't commit stuff without review. Open a PR even for documentation changes.

Anyway, this commit may stay.

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


just one tiny comment, this won't work on non-english locale, so for me I started to use self.apps.displayed_app.origin but I guess that's not crucial at the moment.


@gjhiggins do you have any objections to these changes?


I think @xfq has it right that a `git clone git@github.com:.../...` is a read/write clone and is therefore only allowed for team members.  Non-team members need to use the old version or `git://github.com/getlantern/www.git`.


Ca y est c'est le putsch, focus appartient à @pierr

![](http://media1.giphy.com/media/2yMKLRp2jwicw/giphy.gif)


Looks like some undesired 2014 info snuck in. My bad for starting from the master branch's file. Changes coming soon


My bad. Damn, rookie mistake :angry:


hello all.


@Flameshot In gossip, it does not appear the names of the teams that are in the arena. Are there any requirements? (TR minimum) or something to that teams are shown?


Nice!


In some cases, if using a font-end proxy, likes Nginx, the nginx listens 80 port,
So `getBaseurl` will return `host:port`, maybe it's wrong.


Not sure about it. First, `codecFor(DataType, Object)` does not accept null values. And also it will prevent the registry from caching the resulting codec. I think we can safely rely on `codecFor(DataType)` and let the codec throw an exception if it cannot serialize the value.


Same for mssql-old, fixed in 5d7a158


@wkennington Are you referring to http://bugs.quassel-irc.org/issues/1323, or something else? It would be nice to have a source so that the maintainers can track the status of this issue.


[25794] TEST2


No. If libsodium is loaded we assume the class is there. Only the class may
be provided by a fallback implementation and may be autoloaded. Should I
make this a comment?


hello


anshul anshul


Also, not sure if it's a good idea to store a `CodecRegistry` instance in a static final field.


No I think it's an issue with the way our qt5 is packaged. Try building the
qt5 version and look at the configuration output and you will see that it
fails to detect qtssl

On Mon, Jul 27, 2015, 14:49 Thomas Tuegel <notifications@github.com> wrote:

> @wkennington <https://github.com/wkennington> Are you referring to
> http://bugs.quassel-irc.org/issues/1323, or something else? It would be
> nice to have a source so that the maintainers can track the status of this
> issue.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/c61d048427719d6e0c131f58825331df0a9d4ccc#commitcomment-12379899>
> .
>


On Mon, 27 Jul 2015, Viktor Szakats wrote:

Hi,

> Not sure I understand. It was using Right(), which changed
> to hb_URight() now, but AFAIU those field have no valid non-ASCII values.

Yes and Right() works like hb_URight() for codepages with character
oriented indexes so hb_BRight() is the fastest solution.

> I also find current prefix a bit strange, maybe this is the reason
> or that I didn't yet grasp when to use that I've avoided them so far.
> 'C' for Character seems to be much clearer.

There is only question about HB_UChar() and HB_UCode()
HB_CChar(), HB_CCode() does not look good.

best regards,
Przemek


Do we ever need this call (i.e. the one that follows symlinks?) if not, then we should not carry the call for now. Only add calls here that we either use or intend to use soon, but not dead code that isn't used nor tested.

Alternatively, merge getxattr_malloc() and lgetxattr_malloc() into one call, and make configurable whether it internally calls getxattr() or lgetxattr via a boolean parameter...


wildfly-clustering-server is an implementation module.
Use wildfly-clustering-singleton instead.


I thought the plan was now to use the value returned by [l]getxattr() now here, instead of *2.


same here.


and here.


I'm not good at model/view thing but it seems that InfoWidget::fillInfo() misses reset() call at line 77.
At least that makes data appear immediately on clicking next/prev, not on hovering.
But data isn't shown on startup anyway. :-(


@dlebauer curious why you want to keep the failed fits? If there is one then `fparm <- lapply(fit,function(x){as.numeric(x$estimate)})` a few lines later throws an error.


Travis just enabled OSX builds for the projects. Yay! Still, thanks for your offer!


Justin: *hums*


The `position` will be the position within the chunk right? Therefore it needs to be converted to an absolute world coord.


Hm. And OJS will actually install without being able to send mail, come to think of it -- so I think this is fine to be left stubbed out. Thanks! We can revisit later if need be.


Or maybe some model signal should be connected to view slot.


How about this:
```scala
if (varUndefined nonEmpty) error(stmt, s"Variable ${varUndefined mkString(", ")} must have bindings")
```


It looks a bit "hacky" but it is possibly as good as it gets with Ansible as YAML is not a programming language and Jinja is not either. At the moment there are only 3 deployment strategies but it is likely that we add more so it will end up being very verbose just skipping the other ones

Plus, it does not introduce any BCs which is great

I would only add some spaces in

```
include: "update-code/{{ ansistrano_deploy_via | default('rsync') }}.yml"
```

in order to keep coding standards

Can you please submit *yet another* PR?


iirc everything is done with world coordinates, even in the chunk. @DDoS had a different thing that provided relative chunk coordinates.


Yes, GWT actually pointed to the exact character and provided an
explanation (that some versions of IE will report the wrong array length
with trailing commas).

On Mon, Jul 27, 2015 at 12:31 PM, Kevin Ushey <notifications@github.com>
wrote:

> Sorry about that -- since I was testing by just recompiling with ant
> acesupport I missed it. Hopefully the GWT compiler made it really obvious
> this was the problem?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rstudio/rstudio/commit/41cc2c37023c226bac5f022612d4d3a3266d6a1a#commitcomment-12376780>
> .
>


What should I be using as an alternative?  Should I just be using CodecRegistry.IMMUTABLE_INSTANCE?  Doesn't seem like we'll be doing anything custom in here.


**TODO** Make sure Typeform projects are not open to the public...


:+1: on codecFor(dataType), I was originally using that but codecFor(dataType, value) seemed nice for getting the exception.   I'll switch it back, but not sure how to check if the inbound value will be compatible with the codec.  Any ideas?  Should I just steal some of the logic from CodecRegistry?   I'd like to surface a better exception than 'ClassCastException'


Правило


Дочерние правила


Some more quick and dirty work fixed in 8ef923e


Any FQDN will do; the way I'm packaging the apps needs a namespace. Alternate domain?


Would 'if (field[0] != ':')' work here?
We don't need to check if the field contains ':', we need to see if the first character of the field is ':' (or not).


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

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


Okay


Dang, you're right. Fixing.


The correct invocation of this was:
` unique-kmers.py --stream-out -k 17 tests/test-data/test-reads.fa | normalize-by-median.py -k 17 -o /dev/stdout /dev/stdin | unique-kmers.py -k 17 -`
(note the `-` in the end, to tell `unique-kmers.py` to read from STDIN, and the `-o /dev/stdout` on `normalize-by-median.py`)


Merges #147.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.422](http://teamcity.monogame.net/viewLog.html?buildId=9183&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 538, ignored: 6 Build time: 00:23:18


Hi There,
when a guest user checks out and then another user logs in from the same browser the order is associated with the logged in user, no way to find the original guests email address anymore.
what is the reason behind this logic?
thanks for help
cheers chris


nit, why not use 'switch case' ?


sdfasdfs


@ryankelly-uiuc sorry - I can't recall exactly what I was thinking when I commented this out, or if the change was intentional ... feel free to uncomment this line (or let me know and I can do so)


sfsdfsad


I'd replace it with `this in JAVA_TYPES_WITH_INLINED_EQUALS` and inline it


This shouldn't be necessary because it's handled by the MVC framework


sdfasfsd


dsfdsfsd


Why not `getOrCreate`?


Can you elaborate?


Hrm.  We broke LLVM.

I ended up doing this instead:
```
rec.int_vector.push_back(ntohl(*(int32_t*)(&bytes[byte_offset])));
```

I ended up trying to use `reinterpret_cast` which also didn't work.   Any of the fancy casting operators trigger this crash in LLVM's frontend:

https://kripken.github.io/emscripten-site/docs/getting_started/FAQ.html#why-do-i-get-error-cannot-compile-this-aggregate-va-arg-expression-yet-and-it-says-compiler-frontend-failed-to-generate-llvm-bitcode-halting-afterwards


There's a blank line in every Language.properties in portal that should be removed.


cxzczxc


sdfsafadss


I guess this one is unnecessary as soon as we have another way to provide option (with space instead of `=`).


@tchaikov I am following what already exists in the script, but I don't mind doing it differently if you think it would be better maintained (more readable) I can certainly change it


Ow yea that was my fault... I tested a bit with enable/disable and read only...

Wil ltest Save & Exit tomorrow... Great that it seems fixed ;)


Not sure I understood here.  Bear with me, please.

Why do you not like merging jars across branches?  Is it the storage space?  They will be stored as one commit anyway.

If we don't commit the sqlite jar, we cannot test it in travis as the test setup is now.  The code you referred to is in the main Gemfile and is used for manual testing only, not travis, right.  We could do the same for the other gem files so we can test different versions of SQLite using the stored gemfiles, just switching using the environment variable.

What is special about SQLite?  Maybe I missed the context.

How do we support older JDBC SQLIte versions?  Should we set up testing using older versions to ensure they work?

Ruboto will use the version installed in the underlying Android version.  Android does upgrade SQLite.  It is now in version 3.8.10.  You are right that if you want to support older Android versions, you need to support older SQLite versions, back to 3.4.0, but only back to 3.7.x if you follow the limitations of Ruboto.  I would love to set up tests targeting the valid Ruboto environments.  Should be 4 in all, I think.

Anyway, lots of thoughts.  I am thankful for your patience, and hope you can share your thoughts and experience in the project with me.  I am trying to learn and be a valuable contributor.


No, I don't want to support "master", only official releases (the last one for now)


I'll do it.


this commit is not backwards compatible with julia v0.3.8:

```
julia> using PyPlot
ERROR: wrong number of arguments
 in include at /usr/bin/../lib64/julia/sys.so
 in include_from_node1 at /usr/bin/../lib64/julia/sys.so
while loading /home/arthurb/.julia/v0.3/PyPlot/src/PyPlot.jl, in expression starting on line 404
```


Please don't do that, meta-rockchip is cross-platform and supported on different SoC on which petitboot might be used (on rk3066 and rk3188 for example which support for now only the limited proprietary bootloader). Could you add new sections in the same README file please ?


Please merge this file with README


You'll want to create the singleton service via a org.wildfly.clustering.singleton.SingletonServiceBuilderFactory


This should be 1, 0 so the upgrade is enabled, otherwise the upgrade won't be executed.


I think that your class file is sharable with all rk3288-based devices right ? so it would be fine to rename it rk3288-emmc-update-img, I think


Cool, thanks. I also had to make a small change so that pda.mcmc's unorthodox use of the function works, i.e. where there's no 'beta.o' column in the samples. It shouldn't affect any case where the column is there.


@efueger saw your work here and tried enabling engines on my repo and pushed the .codeclimate.yml file and now the source file is indexed.

But the coverage view has become somewhat tricky to view, https://codeclimate.com/github/sivakumar-kailasam/broccoli-leasot/src/index.js with the coverage option enabled seems to be showing some wrong stats. Also the file in the coverage page is no longer linked to the file  https://codeclimate.com/github/sivakumar-kailasam/broccoli-leasot/coverage


hello anshul.


https://github.com/wildfly/wildfly/pull/7783 will add a ServiceName factory for the SingletonServiceBuilderFactory, i.e.:
https://github.com/pferraro/wildfly/blob/singleton/clustering/singleton/api/src/main/java/org/wildfly/clustering/singleton/SingletonServiceName.java


Why do you introduce a new machine for that ? Please just propose changes to the original firefly.conf file, in order to use emmc image directly (that's the official way to use the board after all)


my new


build_fixes-20150727 (lefticus) - x86_64-MacOS-10.9-clang: Tests Failed

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


Edited b2gedmntuple_cfg.py
 to add add slimmed PV basic info (with a relative module in the src) in the edmntuple output and MET filters, including HEBE bugfix. Note: you have to take it from the ntuple rather than from trigger result.

Added some additional flags for 50ns data and MC and relative description.


I suggest to merge ifs


No, that's wrong.
As I explained already, it reads the keymap
from tty, and as the result, the X code was
comparing also the alt keymap.
Please undo this part.


My bias is for ```DNSAccessor``` to have a ```Run``` method that takes a stopCh rather than embedding the ```RunUntil``` in this block, but I may be biased based on thinking this piece of the code should be structured more like the controllers.


Are you making these while recording, so it seems like we have an artificial intelligence embedded in `clj-refactor` because all the guesses are spot on? :D


We should remove this from core and use PortletProvider


Personally, I prefer that things in all caps with underscores (like `SF_U8`) are macros.  Which does not mean I want these to be macros :-).  Since we're counting characters, how would you feel about using lowercase `sft` (for Serialized Field Type) as the prefix, instead of `SF_`.  Then the names would look less like macros.


I see, I'll do it.


Having bar height as a config option makes it easier for users to adjust that setting without digging into the code. It's something that will need to change based on what screen ratio this is deployed on.


*no session


Mhhh... I understood what you plan to do here, but perhaps it would be better to introduce a variable to tell to the existing recipe which binary image to use. For example define RK_BOOTLOADER_IMAGE = "my image" in firefly.conf and use this variable in the recipe in order to install the good one. In this way we would have one recipe, and a generic solution


hey @mor7

Ich finds super, dass du lust hast bei rainbow-lollipop mitzuhelfen.
Ich hab mir die changesets gerade durchgelesen und deinen branch kompiliert und ausprobiert. Funktioniert tadellos.
Einen Verbesserungsvorschlag haette ich noch, falls du bock hast, das auch noch zu machen, aber sonst koennen wir das so eigentlich schon reinmergen.


whoa.. this unconditionally returns 0 regardless if the pattern match is found or not.  You probably meant "return $?" instead of 0.


sdsd


same here, this return 0 isn't right. i think return $? is though.


Those are generally accepted shortcuts for POSH. Thoughts?


Implemented. Tests pass.


What about using path.resolve() to be absolutely sure to get an absolute path?
```javascript
staticPath = path.resolve(path.join(this.config.staticFilePath, decodeURIComponent(reqUrl)));
```


Unfamiliar but it's in the template in a section you don't need to touch.


Maybe we shouldn't base this on year.month in case we have slippage like 15.06 -> 15.07?


It may mean that you have multiple things with similar names and your Display Name search is not specific enough. I agree that throwing an error instead (warnings are still treated as successful uninstall) would likely be more beneficial.


@jenslind nice. also I just pushed a `contributing.md` with my release flow in case you were curious https://github.com/maxogden/menubar/blob/master/contributing.md#releases


Remove this recipe, an initrd or initramfs can be directly built by yocto/oe itself. There are no specific modules in it right ? (like on rk3188 with the binary module for NAND)


I've hit this as well. I don't understand what "docker.io" prefix means. I'm good with this patch regardless.


Neat, do you want to file a new issue with this?


What does this do, exactly?  Is there any reason not to always support this?  SingletonServiceBuilderFactory has a non-ha implementation - which should allow the things to work more uniformly in both ha and non-ha environments.


I think we should ideally check for type, so
```
entry, ok := obj.(*dnsEntry)
if !ok {
...
}
```


looks good


Hi Zack,
this sounds awesome. Indeed, the Matrix to matrix is not necessary. I've been testing the changes and I am getting the same results being ~6X faster. Looking forward to the new updates :-)
I can open a pull request, or as the changes are just two lines, you can add them.

Antonio


Hier waers vielleicht cool, ein passendes Icon aus dem Icon-Theme zu beziehen. Es wuerde sich das "audio-volume-high"-icon anbieten, Falls du Spass dran hast, das noch zu machen wuerd ich mich sehr freuen, sonst mach ich das noch irgendwann mal.


Might be too much...


I could be okay with that but we would have `STI_INTEGER`, `STI_ACCOUNT`, etc... so I'm not sure how much we're gaining by renaming.


This method isn't idempotent for an active bomb. Shouldn't ```active_bomb.activate``` be a no-op?


Might not be.


This commit didn't have any effect, and Travis pypy seems to run just fine.


Fixed, thanks!


What's the alternative?


Have a look at other recipes kernel, and please use "linux-yocto.inc" instead (a lot of info can be factorized)


Kinda the same thing I said above.


Did some perf tests and results were not good: MassIndexer is ~2x slower by hammering the index with the ```forEach``` and ```NP-1 threads```, in a 8 CPU machine.


What the.... why don't you use do_compile_append() ?


Good thing, in the future just COMPATIBLE_MACHINE="firefly" will be enough


What made you move from jaccard?


initial webhook commit


Also note that most editors can syntax-color macros:
http://i.imgur.com/oCYVk3x.png


Should this be a constant, or a default which can be overridden on initialization?


nevermind, we'll just add it to #366


Should we be able to initialize a bomb in the active state?


sure a config is good, but hardcoding the require for the config at a component level seems odd
as compared to passing in the config at the root level of the app.

By passing in the config here, you are adding another point of state to be injected, that goes outside the norm react prop interface


FYI - https://github.com/wildfly/wildfly/pull/7739 renames wildfly-clustering-singleton to wildfly-clustering-singleton-api


ok


I'd feel a little better if we just used deepcopy() for this and then just straight-out assigned dict keys in the next line, rather than using merge(). The reasons are that (1) I'm casually trying to get funcy out of the codebase, because its understandability and performance are not very high and (2) sharing refs deep into dicts is just asking for a mutation bug down the line.


Created on Mon Jul 27 22:34:32 2015
@author: Dr. Fayyaz ul Amir Afsar Minhas (afsar <at> pieas dot edu dot pk)

Description:
This module implements the approximate Locality Constrained Linear Coding as described in the 2010 paper by Wang et al. [1]. Given array of datapoints X (N x d) and codebook C (c x d), it returns a vector of coefficients G (N x c) such that G * C approximates X and the mean distortion mean(||G*C - X||). LLC introduces sparsity by forcing those coefficients of a given data point that correspond to codebook vectors which are not that point's k-nearest neighbors. LLC also uses regularization.
This code has been verified to produce the same LLC coefficients as the original matlab implementation by Wang et al. [1] available at: www.ifp.illinois.edu/~jyang29/LLC.htm
Usage: from llc import llc
Running as a script:
When run as a script, this code will plot a toy data and show distortion of each data point (proprtiontal to marker size).
References:
[1] Wang, Jinjun, Jianchao Yang, Kai Yu, Fengjun Lv, T. Huang, and Yihong Gong. “Locality-Constrained Linear Coding for Image Classification.” In 2010 IEEE Conference on Computer Vision and Pattern Recognition (CVPR), 3360–67, 2010. doi:10.1109/CVPR.2010.5540018.


Note, the pending 'Updates Settings' spec, as it is currently failing.


I am not a fan of this variable name. Should it be ```@inactive``` or something else entirely?


Let's say "as a series of inline-level tags" and remove all ambiguity.


that commit :D


Custom build might have different branch. But for non custom, yes it's always master.


actually the default value is not used


These are the 6 functions that are not implemented, but imported from `stats` into `mstats_basic`.


This is an example of how the documentation for some functions is also imported.


1. `match_controller.rb` <- new controller
1. `def create` <- post endpoint

(
honestly we should have:

1. `club_controller.rb` (view (/clubs) / add)
1. `match_controller` (view (/feed) / add)
1. `pong_controller` (leaderboard / about)

)


nameコマンドの修正ありがとうございます。見やすくなりました。

動作確認できましたので、マージお願いします。


what happened here?


You probably don't want to use ext4... extended fs or traditionnal file systems are evil for NAND flash or eMMC (remember that's the control is chineese...). ubifs is probably a better candidate


i've recreated the app from scratch, based on yours : https://github.com/flyingrub/MiBand-Notify.
There is only one service running in the background and no need for your billion of intent/broadcastreceiver :).


So I used the completion block so that we can also return the error if there is one from retrieving the information from the keychain item. I can just have this return a TGRouter if you think that that is sufficient.


As it is, it supports both common argument styles: space or `=` separated. Removing the later will make the invocation less flexible.


coverity is complaining about not check the return of this function and makes sense since you are using conn after this.


What's wrong when pinging openpli?


I wanted to clarify that whether the wire was cut or not.  Inactive implies that it is physically intact, but not used.  Maybe `@cut` instead?


Also remove format_club_name, unused


  Problem 1
    - Installation request for doctrine/doctrine-migrations-bundle 2.0.*@dev -> satisfiable by doctrine/doctrine-migrations-bundle[2.0.x-dev].
    - doctrine/doctrine-migrations-bundle 2.0.x-dev requires symfony/symfony >=2.0,<2.1 -> no matching package found.


Agreed, I'll remove default value.


ok


You're right; thanks for catching me on this silly mistake.

Here's the build that failed: https://travis-ci.org/sampsyo/beets/jobs/72885156
But it looks like it was intermittent.


why did you change data=NULL to data here?

didn't we decide that the user didn't necessarily need to specify a data argument?


It's very unlikely that conn will be NULL or that this will return anything other than CURLE_OK, but I agree.


Overall this is good, but there are some holes in the test coverage. Since it's an exercise, it's hard to know if the immediate intention is for those in training to notice the holes & address them, or if they're in fact just holes. Consider adding simplecov, & please act accordingly with regards to testing for desired behavior.


Minor typo corrections.


This solves #3


Agreed. The `config` object should be passed as a prop from the root component.


probably reference Tungsten views, not Backbone specifically


C'toi le boulay


@obiot I cannot tell by looking at the code alone; what behavior are you seeing that indicates these events do not work together? In my testing last year, it worked great. The subsequent change in 6da62b25d099bddf934280a812aeeba84a8d9233 breaks it for certain. See my comment there for details.

A few thoughts, though:

* You can't use `enter` and `leave` events to choose the hover state for these crazy polygon objects; these events trigger when the pointer enters (or leaves) the *region*, which is the boundary box. In the shapes demo, you only want to activate hover state when the pointer is over one of the shapes.
* `enter` and `leave` are not useful for entities that can be moved by the pointer; The pointer is guaranteed to leave the entity as it moves. It's a function of pointer speed and entity size.
* The move event is being triggered globally (and with an event named `MOUSEMOVE` no less ... shouldn't that be `POINTERMOVE`?) This is the part that is incompatible with `enter` and `leave` events! See bullet point 2 for the reason.


That's how it started, for sure (with `update-in`). And also, the support for `(create-foo)`. The rest is just more stuff that's been plopping into my head. :)


Connecting model's dataChanged() to view's reset() doesn't help.
So I suggest committing your nullptr with my reset.


I don't think it's necessary to deal with the error that's returned from the keychain. If we encounter an error, we should just assume we don't have a router object saved and return nil from the function.


While mostly duplicative, this does serve one useful and one necessary purpose: if rate > maxSafeBasal, dropping it at this stage allows the "No action required" debug output to be displayed if there is already a maxSafeBasal temp running, and it avoids extending the maxSafeBasal temp to another 30 minutes.  (It's safer to let those expire and then re-issue whatever is still needed at that point.)


how about package level privates?


I agree that `SOE_Flags` look more like a "style" then they look like "flags".  But, personally, I'd still be inclined to name this method `flags` until the name of `SOE_Flags` gets fixed some time in the future.  It would be confusing to me to call a `style()` method and have it return an `SOE_Flags`.  But I'll leave it to your discretion.


@flyingfisch

Hi Mark

I had just revert the 3d model of the overhead panel.
In this commit you made 80 additions and 37K deletions to the oh-panel.ac file. This caused the clickable parts of the panel to be non-functional.
See: http://forum.flightgear.org/viewtopic.php?f=14&t=24450&start=15#p250136
See: 7f6fba2ec129f22504e9756a4269301e6498df53

If there is any intent you had to edit the ac file of this overhead, you will need to revise your work, correct respectively and submit your idea again. One that, hopefully, will at least maintaine the current functionality active ;)

thanks
@IAHM-COL


Updated a few minor typos.


not covered by tests


This is an old commit.


how about primary ctor?


Moved in 4b8689469aeb655bbf76243c30889185b94bf4f0.


C# and Entity Framework


did you mean `conv.isPureHangout(i)`  ?


Missing word here, like "… section _on_ updating …"?


Unfortunately, several of these improvements have been made after recording. :grimacing:


> I said that the example in the Polymer kit copies the source to-be-vulcanized into dest and then runs vulcanize with that as the source.

Oops, yeah I misunderstood what you were saying. I'm really not sure how this should make a difference but I will try it.

> wait nevermind I'm just dumb and can't read

Yeah, I'm requiring deps slightly differently than how Polymer does it, but it's still valid gulp and hasn't broken anything else.


Crazy idea: switch to semantic versioning instead of YEAR.MONTH?


In most cases GC will close the file automatically when using open(file).read() or open(file).write(data)

Please review the patch


shouldn't this check be in the line 131 ? and contains a goto out ?

I'm asking this because this value is changing inside the two if's block above.


@Montellese
how do you run it from VS?


Or forget the "semantic" part, just plain integers. Release 1, release 2....


you can always invoke `codec.accepts(value)` to see if the codec is OK (but that only works if `value != null`). IMMUTABLE_INSTANCE is not a good idea neither as it cannot create codecs on the fly, so no codecs for user types and tuples would be available. Maybe we don't need a CodecRegistry but just some of its logic: an Immutable map of codecs for all the basic types, and a method that creates codecs on the fly for the "complex" types: collections (including nested), UDTs and tuples (but without caching them in any way).


Hmm, I'll take a look at it sometime this week. The parts I changed shouldn't have messed anything up.


I'll add tests for secondary contructors. Primary constructrors messages for now should be equal.


Instead of using these cascading `if` statements, I'll make it jump directly to the callback call.


It's used here, it can be removed in the next patch :+1:


I think this is good enough to roll with for the moment.. we can revisit, perhaps with a picture, later this week


filtering part can be extracted too


Need to emit some notification here before attempting to reconnect. We'll use it to show "reconnecting..." message in UI.


we should get something like:
`foo(old.secondaryConstructorList, new.secondaryConstructorList)`


@GrahamCampbell Can you think of any features/functions, or beneficial changes to the public interface that could be made by dropping support?


will change it, thanks :D


Yeah I would be okay with an integer that corresponds with the month.


This doesn't actually query the full dom, only the full subdom of the selectable heirarchy.


internal?


yeah... forgot to take it out  =/


I'm not sure I agree with this as default delivery to the UI thread is useful for populating a UI with the results of a data fetch. Also, tests need to be added for all updates like this.


Maybe use `JvmProtoBufUtil` here to encapsulate the extension registry?


I'll try do it.


Thanks for mentioning fusion in Haskell, which is a done deal, built into GHC so that intermediate "lists" can be used with confidence without their being materialized. Also, people have been writing plugins to do such optimizations in Scala: https://github.com/nativelibs4java/scalaxy-streams

Meanwhile, it does remain the case that newcomers to FP tend to considerably overuse lists, simply because pedagogically they tend to be taught early on. Experienced FPers are mindful of performance and try to use unboxed and packed types where appropriate. Bits, bytes, structs, and arrays will always be an important part of everyone's toolbox.


so maybe add the flag if we're not on windows?


Link to issue #44


It's true that the `SerializedTypeID` enums are all caps with underscores.  To me that seems like it was a mistake.  And my inclination would be to not proliferate the mistake.  But I'm not going to push the issue.  If you feel that `SF_` is the right choice for the prefix, then go for it.  It's just not the choice I would have made.


not sure about this one, I think the exheres to sh filetype mapping would better be in an exherbo specific support file.


does it mean we should unload it first?


This introduced a bug where the images load when the page is initially loaded, but become not found when the page is reloaded.


should be not loaded in admin.. right?
I got following notices in admin area (too) as soon as I upgraded my dev branch.

Notice: Undefined variable: this in customizr/inc/class-fire-placeholders.php on line 234
Warning: array_map() expects parameter 1 to be a valid callback, first array member is not a valid class name or object in customizr/inc/class-fire-placeholders.php on line 234
Warning: array_sum() expects parameter 1 to be array, null given in customizr/inc/class-fire-placeholders.php on line 234
(see comment on the specific line)


But why, really? This is only a problem if you want to make something conditional on a future release. Which seems a hypothetical problem to me.


That's why I want to fix this.


I can either name it something new or deal with all of this:
https://github.com/cms-sw/cmssw/search?utf8=%E2%9C%93&q=rctDigis&type=Code
I'm willing to fix it, but I need to know what of these are old cruft and what (if any) are still important.


Plain integers (incremented whenever we do an incompatible change) are relatively simple/easy, but it isn't obvious to which date was number `X` changed. YY.MM or YY.MM.DD have the additional advantage that they can be added on the moment the change happen, i.e. if we were doing changes on master now, we could compare to 15.07 *regardless* of when the real release happens – just signifying the date of introduction on master (which is linearly ordered).


or `foo(old, new, ProtoBuf.Class::getSecondaryConstructorList)` :)


I don't quite like this one. We should add a windows.skip or something like this


I get some warning notices here.
Inside static methods we should refer to self::$instance instead of $this
Can you reproduce the issue?


Instead of the shifts, do the comparison first and then a pshufb.


It seems we need to add `JvmProtoBufUtil.readClassDataFrom(ByteArray)`. I'll do it.


MASTER: Version without the capture which just passed simplification tests
TEST: Same, but no restriction on rank 3

Using a "parameter tweak" bounds [0, 4]


Will add thanks :)


This should be an `enum class`, and then remove the `kind` prefix from each enum value.


Happy Late Bday. lol. Sorry was just trying to absorb your use of github, came across bday, so had to say something... Carry on!


Awesome! :smile:


Follow Google code guideline.


I think it's good to leave the provenance of these files in the code itself (similar to examples/data/getSEDs.py and examples/data/getLSSTBandpass.py).  Personally I think it's okay to keep this as is (somewhat hidden), but I could probably be convinced otherwise if someone else has a strong opinion.


What about machines with 2G+ internal flash?

I think imternal flash should be detect using another technique and not depend on contants


I'd say `Callable.isPrivate`


I've looked into replacing ownCloud with the underlying DAV server, but then we'd need new web frontends for contacts and calendar.


Agreed, I'll change it.


It seems the goal of `clean_params` here is to render the parameters included in the HTTP request, excluding things added by Rails like `controller` and `action`. It's been in Rails since this commit https://github.com/rails/rails/commit/21187c0fb4fc559927a2f947f909dedd85969ffb#diff-9985c296caac36ee75b7e0df6c373796R29.

For this reason, I'm also excluding `format` from parameters, keeping consistency with the rest of the `clean_params` stuff that shows only request params in the debug session. The changes in this PR forces to have a format already set when this code is executed, so I decided to filter `format` here.

I didn't include tests for this line because I think it isn't worth to have it tested (it's a very minor display detail for a error page for developers)


I'd prefer `$key === null` over that..


I'm not sure what you mean here, but those modules are all supposed to be
'rctDigis' because they take the rctDigis as input. For MC there's a
customization fragment that replaces 'rctDigis' with 'simRctDigis' for some
of them, so I'm not sure what the issue is.

If 'rctDigis' is already in the 'process' object you might get an error,
but that means that some other version of the unpacker has incorrectly been
included in some sequence and that needs a fix.

On Mon, Jul 27, 2015 at 3:37 PM, Nicholas Smith <notifications@github.com>
wrote:

> I can either name it something new or deal with all of this:
> https://github.com/cms-sw/cmssw/search?utf8=%E2%9C%93&q=rctDigis&type=Code
> I'm willing to fix it, but I need to know what of these are old cruft and
> what (if any) are still important.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cms-l1t-offline/cmssw/commit/d4fbecb612c15c6524a63b8fcebb5038fe8026f3#commitcomment-12380913>
> .
>


great


should be fixed now


haha, good catch! :)


Well u can say that...!shouldn't have messed anything up.!

I was with my GF in  bed last night and I trainer her up and she was flyiing approach she was a bit miffed to say the least..

I spent all night explaining to her that she was not at fault.. and it was a sim problem..

So guys please sort this out, I want my joystick back.. and be Captain again...instead of FO

ta ;))))


@ch3cooli Thanks for pointing this out!


Emacs auto-complete, I didn't have a second look at this, which I do before sending patches :)


@Adaptivity Any updates?


Yea will add something on this


I would have written if windows : ... else: ....


is this safe? don't you need it to happen in a subshell?


Is the cast really needed ?


why do you still need the '/' test?


Yah - a comment would be good. Thanks!


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.832.157cd552897d022b2ee4d47cd9d0687332d05aac](http://build.palaso.org/viewLog.html?buildId=60061&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:03:17


Are you sure that -Incursesw will search for a ncursesw dir in the standard include paths ? I was under the impression it will use the path relative to the current directory.


How about
offset = ((keycode - min) * width) + group * 2 + index;
and then maybe simply inline this?


These (void *) 0's should be changed to NULL I think?


But having two options for the same task is not really something required, so let's keep code as simple as possible.


Have you checked the perf? What was the portability issue?


cool!


Laurynas, I was planning to pick up your proposed change, but it doesn't look to me like you've signed our CLA, https://code.facebook.com/cla. Would you be willing to do so?

Cheers,

Justin


lgtm!


YOU ARE FANTASTICALLY STUPID


Maybe we could merge that with #340 as a gentoo-exherbo support script, that provides glep42 highlighting and ebuilds/exheres shell filetypes.


Mostly to eliminate the need for the cb.ret variable!


Merge remote-tracking branch 'OSU/master'


@pedromorgan

;))))

I really don't know where you are going with this, but I can  tell you

no need to console your lady. She is totally right to be contrite. She just killed 150 simulated souls.

@IAHM-COL


I think we can support it later when private at toplevel will mean private in file


ACK


Combinations!


seems the same - but last commit is from 3 hours ago?


How would you feel about a solution that used `java.nio.file.Path.getFileName(worldName)` to retrieve the last segment of the path?


We don't expose the enablement pref to users; it's a side-effect of having things to sync. (That we don't consider the clients engine to be a 'thing' that needs syncing in order to support a feature is why this is a problem. Send Tab is an awful hack.)

The 'right' fix for this is for send tab to be an engine or a service of its own, so it's a box to check. That's not really a viable short-term solution.

*The lowest-effort, safest thing to do here is to tell the user to choose to sync something, probably tabs*. I think the number of users who want to use Sync only to send commands to other clients is pretty low, and having nothing checked is likely to be due to a bug, not an intentional configuration choice.

Flipping the pref is both good and bad. Bad because we'll now be syncing every few minutes, even if there's nothing to sync. Good because we'll pick up remote engine election changes. Bad because this is also how admins are supposed to turn Sync off. And so on. If you are determined to go that route, please get review from mhammond.


I like your suggestion, let's file an issue so we don't forget.


Avec tite bière au Sacrement, ça se fait bien ;)

Date: Mon, 27 Jul 2015 12:39:03 -0700
From: notifications@github.com
To: CarbonCopy@noreply.github.com
CC: julien.methot@zincinformatique.ca
Subject: Re: [CarbonCopy] Fait marcher c# (960f640)

Awesome!

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


The idea behind it was so that if a user placed an order but later created an account it would associate their guest orders.  We should probably make this configurable and turned off by default so that people can opt in  to use it instead if you want to submit a PR


Como consigo obter uma app key?


I believe it could also be changed to pass the original email so that the email doesn't get changed also. Or perhaps add a conditional check that will only associate the order if the email address on the order matches the user email.


Fixed; see #147


@ch3cooli Thanks for pointing this out!

I see two ways for fixing this:

1. in GitStatusListCtrl.cpp (https://github.com/TortoiseGit/TortoiseGit/blob/265973697a1f68cf4cf2ac979a7a5b4445e39885/src/Git/GitStatusListCtrl.cpp#L3943) change to `if (((gitpatch->m_Action & CTGitPath::LOGACTIONS_ADDED) || (gitpatch->m_Action & CTGitPath::LOGACTIONS_REPLACED)) && !gitpatch->Exists())` (maybe `!gitpatch->Exists()` could also be replaced with `(gitpatch->m_Action & CTGitPath::LOGACTIONS_MISSING)` as we already know that it is missing)
2. Change Git.cpp (https://github.com/TortoiseGit/TortoiseGit/commit/265973697a1f68cf4cf2ac979a7a5b4445e39885#diff-585d0f5c4e733a311b2ed12de6b772aaR3302) so that Missing can olnly occur with DELETED (that would also require the test cases to be updated)

Unsure which fix is better: As we already state in the messagebox that tortoisegit cannot cope with those files, both fixes are ok - otherwise i'd vote for the second possible fix as we might need to adjust all `==` in `gitstatuslistctrl` checks).


Shouldn't this be 4 now?


Why is this commented out? Regtest used to generate static rewards which makes the tests cleaner.


That sounds fine, but I won't be able to help with making NEI going into a quasi-disabled state.  There's far too much work involved to ensure the safe usage of the `world` variable everywhere.  Personally, I'd vote for a fail fast policy, rather than a fail-but-limp-along-in-a-broken-state.  That latter policy cost me several hours debugging this issue.

The current solution simply fails in the same way as 1.0.5.111, but adds an error message instructing the user about what's going on.   If you disagree with this, I can revert the `try/catch` around the world setup, but I'd recommend against that.


Yes that's ok as long as group is always 0 or 1. I had it in mind to ensure if group was anything else it wouldn't index incorrectly into the map.

Sent from Samsung tablet

-------- Original message --------
From Stas Sergeev <notifications@github.com>
Date: 27/07/2015  20:44  (GMT+00:00)
To andrewbird/dosemu2 <dosemu2@noreply.github.com>
Cc Andrew Bird <ajb@spheresystems.co.uk>
Subject Re: [dosemu2] Keymap: Rework X11 detection (5a5d7b4)

How about
offset = ((keycode - min) * width) + group * 2 + index;
and then maybe simply inline this?

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


I have to verify that it has value grater that 0 rather than default int value 0. Not it has value 0 that is incorrect.


lemonparty meta refresh


This test is testing [`Request#path_parameters`](https://github.com/jmbejar/rails/blob/c906764e32f31cdf55ddf4c50f7c9523bcdc399b/actionpack/lib/action_dispatch/http/parameters.rb#L31) that now includes the format deduced from the URL by default, even if the router does not define a format by itself.


After clicking the cheatButton and than followed by showAnswer button, the checkAnswer method is not working.


develop (asparke2) - x86_64-MacOS-10.9-clang: Tests Failed

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


why -4 ? size -1 ?


Officiellement jaloux!  :smile:

Si tu remets ça un autre après-midi tu me fera signe!


fannccy


Thanks for the Fix!


Fixes #18


Can you give me an example for this?


Fixed in later commit.


Spec and Talents seems working when i enabled them? you can try test them?


... I think.

4 was enough to clear the title bar. Now it's handled by correctly placing the frame in the window.


I'm now realizing that routes defined without the `:format` part will end up having a `format` value in the `Request#path_parameters` (deduced from the URL) instead of having a `nil` value that would result from the router calculation.

Although I like this behavior, I see it can affect existent apps that are relying on the absence of `format` for routes that do not include the `:format` part.

Should I make sure router always override the `format` calculated in a early way based on the URL extension? Even when the router says the format is nil? I don't know if that would be worth.


add_fixtures_to_VS_solution (mbadams5) - 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)


damn, and I just wanted to start using it... isn't it usefull to keep it there somehow to be able to keep track of durations? are you sure whmcs isn't using part of that info?

Sandro
________________________________
Van: Alexander Trofimov [notifications@github.com]
Verzonden: maandag 27 juli 2015 5:26
Aan: Virtuworks/Websitepanel
CC: Sandro Mastronardi - Mastronardi Software
Onderwerp: Re: [Websitepanel] Remove eCommerce in WebPortal and Enterprise Server (c705b9e)

Completely gone, because there was no people who use it a lot and could maintenance it...

—
Reply to this email directly or view it on GitHub<https://github.com/Virtuworks/Websitepanel/commit/c705b9ee4633528e6ef590e14a8fb2b3cd2de962#commitcomment-12364518>.


you suck at orgormag


Ok, as far as I could see we really don't need this patch so far. We may revisit this use case when we get there, but for this task we don't need this.


kk


Published to http://ui.edev.d2l.com/92_e6dd8826d624e391f1251041d068f34a969a0672


Thank you @mwjames! Fixed in 4f5bd45.


This is the forum post: http://emby.media/community/index.php?/topic/23065-how-i-solved-the-emby-and-kodi-master-lock-issue/


Not at the moment, sorry.

On Fri, Jul 24, 2015 at 1:37 PM, Arthur Pachachura <notifications@github.com
> wrote:

> @russellsayshi <https://github.com/russellsayshi> Any updates?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/epitome3d/Epitome3D/commit/b4296bf6bbf6c86823168181009c88cc99ae3816#commitcomment-12349459>
> .
>


I don't understand why you need one more indirection to render().


LGTM.


Installed via [pip install autopep8] on python34. Will it automatically pep8 my files if I add this line?
$ autopep8 --in-place --aggressive --aggressive <filename>


https://travis-ci.org/jimlind/tivo-php/jobs/72766734
Atoum needs some kind of additional configuration.
Booo.


This... greatly upset things. Check out https://travis-ci.org/chef-rbenv/chef-rbenv/jobs/72766918


Ethan's patch seems to work on d2 for this


My music library is fine over here....no missing artists, no missing content, no missing fanart... I have the dummy sources in place and everything is working fine for me. Except that video OSD thing (which I now believe I'm the only one to see)? You think it's because of the sources?


this will become un-maintainable... Please define constant close to where you use them (ex.) class level).


No. I want to compare apples to apples. If it is indeed a speedup, the same speedup trick can be applied to marco's patch.

Also, fishtest results are driven by windows machines using non PGO compiles based on Gary's whatever version of mingw (Linux to Windows cross compiler). Whether fishtest results match yours or mine when measuring speedups is far from clear. But fishtest is wrong in that area so it doesn't matter. What matters is how fast the fastest PGO build is.


Next time please ask before adding such important stuff :-/


按时打发士大夫


Yes, you are right. I'll correct it and re submit.

My values are based on the "algoritym", those were the good values : http://tests.stockfishchess.org/tests/view/55b036390ebc590abbe1bc3c
and those were the wrong ones, I mistyped them :
http://tests.stockfishchess.org/tests/view/55b2ac820ebc590abbe1bcc2

As you can see the huge difference in W/L between the two.


design is not right, if you have to pass the whole context to the "view"...


looks like this should be declared as @required. Similar to other views.


This code uses the right direction: what you pass in command line has the absolute priority, then we consider the configuration file, and then the default options.

The problem is that we merge objects in a non-smart way: every fields defined in the `cliOptions` simply replaces the fields defined in `argOptions`… and because the `ignore` is always set in the cli, the `argOptions.ignore` field is never used.

I think the right resolution is to fix this object merge by merging the two `ignore` arrays as well :-)


Why the actual fuck did you make this change? lol


you need a better function name.


@firejun Thanks for taking the benchmark. I'll ask you to take another benchmark today's afternoon.


You really believe that changing `>=` into `>` will have a measurable effect ? Why ?

I suggest you measure the frequency of those conditions with `dbg_mean_of`.


rename


Inicio do Projeto: 26/07/2015.


访问权限


Probably - I've got a list of packaging tasks I'm wanting to knock off
shortly - will add this. Don't expect it to be difficult.

Do you know if there is a minimum version I need to do?

Thanks,

Joe

On Mon, Jul 27, 2015 at 9:37 AM, Pedro Algarvio <notifications@github.com>
wrote:

> @joehealy <https://github.com/joehealy> is there a chance, in your tight
> schedule, to backport a more recent python-msgpack to precise?
> We could always install it from pypi but users tend to prefer distribution
> packages.
>
> As always, thank you for your hard work!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/saltstack/salt/commit/a192ecfd7448365f396c6fa36d0ae0d938c7dc8c#commitcomment-12363443>
> .
>


assert 옵션이 enabled 되어있지 않을 경우 assert 문을 확인하지 않기 때문에 0~99 이외의 값이 할당될 가능성이 있습니다. ( -ea 옵션)
바로 위의 setTimeoutExceptionThreshold method 와 같이 처리를 해주거나 method 의 argument 검사는 assert 보다 IllegalArgumentException 으로 하는 것이 의미적으로 더 명확하기 때문에 exception 처리를 하는 것이 좋을 것 같습니다.


Why you need these fine margins @Acidhub ?


Nvm, this is the right thing that I wanted to do, as :
Relative rank :    value mid game:  Value endgame :
2 (Rank 3).    28.      35
3.                    74.      70
4.                     ...        ...
5
6
7 (Rank 8)

If this pass I'll keep my const array at 8, keeping the 0 and 1 relative rank.


part_material now can capture material quantity and parts per material. I still need to update the front end for part so we can add material and counts to a part just like po can have parts added to it in expandable sections


tsina 没有问题吧。这个修改的意义是 ？


This seems to cause a problem when sending a POST request with curl.  It throws the error `Unable to decode JSON body` with this code, but changing it back to what it was originally makes it work again.


额，用的周剑斌的电脑，这个是我提交的


asdsaasdfsda


The openemr code base security implementation examples has some scripts
that have being changed from htmlspecialchars. like this one:

https://github.com/openemr/openemr/commit/8a8a4607ba5ae2b9eb6b6a3b1b8ed7c6ea7e03b1

Roberto

On Sun, Jul 26, 2015 at 5:52 PM, Brady Miller <notifications@github.com>
wrote:

> one thing to consider is placing single quotes around the value here:
>
> <input type=hidden name=noteid id=noteid value='" . attr($noteid) . "'>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robertogagliotta/openemr/commit/19258dc3786be6a174cb31c202755730b7cf94c0#commitcomment-12363709>
> .
>


This is so cool!


## Usage: ./get_hdu.sh 题目编号


そのようですねぇ :eyes:


Soon... full expression parsing


Also may be worth it to add a type synonym for `ReadyPackage ConfiguredPackage InstalledPackageInfo`.


I would like to know more about when to use ' and/or " and/or .
Could you please give me a brief explanation ?
Roberto


Again, being polymorphic in result/failure types here seems unnecessary.


This looks like something we should have a type synonym for.


rax retry tests


@koistya this keeps giving a 400 error when going to one of the route above.


@jserv the latest commit of jit-arm.c work really nice on Freescale i.MX6DL, it looks like we don't need to add `__clear_cache()` if you use DyNASM (why?).


I was wondering about this for the longest time.  I noticed that when setting programmable pipeline, the display was blank unless I called:

```c
al_use_shader(NULL);
```

I thought it was by design, actually.


I think it does not really solve the problem, Tox user needs a more stable file transfer mechanism.


In general, I think we could make the code less verbose by renaming `InstallPlan InstalledPackageInfo ConfiguredPackage BuildSuccess BuildFailure` to `InstallPlan` and `InstallPlan` to `GenericInstallPlan`. Similarly for `ReadyPackage ConfiguredPackage InstalledPackageInfo`.


Look at PassedPawnsBonusMg[relative_rank(Us, s) - 2]

If relative_rank(Us, s) = 1 , this will produce a negative index.

PassedPawnsBonusMg[relative_rank(Us, s) - RANK_2] is the way to go.


Also, many instances of unnecessary (IMO) polymorphism. Most type signatures with `isuccess` and `ifailure` in them could use `BuildSuccess` and `BuildFailure` instead.


why did I get tagged here, grow up dentmaged


When read tracking is disabled, your sql is counting the number of discussion threads viewed - whereas the forum_read was counting the number of posts viewed. It will also count the same discussion repeatedly when viewed multiple times.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.833.09924e8f20ff5791252333b070c0874379a5e02b](http://build.palaso.org/viewLog.html?buildId=60062&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:02:49


:+1: This is a pretty severe problem. Maybe remove dog/kennel altogether. I am not sure if the AI uses dogs efficiently at all. They may end in canon fodder.


Not sure I completely agree with your edits to `txplib_publish.php` here. But I might have missed something, so please explain if I have.

Since you're testing `$production_status` in `trace_add()` I can see the appeal of deferring the test there, but from a performance perspective my Spider Sense thinks that the simple act of testing the global in `processTags()` is far better than pushing `trace_add()` onto the call stack, testing the global, determining no action is required and popping `trace_add()` off the call stack again. By all means leave the test in the function, but if we don't have to call the trace function at all in something as fundamental and time-sensitive as `processTags()` then my guess is that's better given the number of times it's called on a typical page.

As I say, I may be wrong here, so please feel free to educate me if my thinking is outdated. I'm just of the mindset that every nanosecond counts. [See the optimisation parts of this discussion](http://forum.textpattern.com/viewtopic.php?id=43518).


Fair enough. I generally agree. Fixed in e58194acf7a935680e590d4d96196ec44944f249.


how does gdb model work with functoin,?  Many regard, ben


Shouldn't it be reflected in README?


Any idea what's happening? Looks like something also broke decorators this evening.


What's the thought behind having the diff commands here?


Yep, good catch. Cleaning that up now


Filed as https://github.com/mozilla/pluotsorbet/issues/1769.


should this have included a CRAN link? or does the `<pkg>googlesheets</pkg>` syntax accomplish this automagically? from the surroundings, I'm suspect the latter is true but am just checking and learning


Conditionally changing the *datatype* of a variable is troublesome for JIT compilers. Just like calling a function with different datatypes is a no-no (called "polymorphic" in language-geek parlance.) I guess we're kind of doing both, here...


It breaks compilation - directories var is defined after usage.


generally, the "default" library for a package should match the package name.

so 'angular2_test.dart'


Ok :) Anyway, pas testé à moitié c'te patente ;)


Close, we want bar || foo (for TARGET_ARCH), which `#.*` will do.


Issue #336


you scared me when you said a global var... this is just a class constant, which is totally fine.


Great place to extract a helper method I would say :) thanks


Absolutely sure.


`minutes{1}` in two places.  Eschew manual conversions.


These new methods need a test case to demonstrate their usage.


It allows users to not the status buffer while still being able to look at the changes before initiating a commit. Whether that's a good idea to avoid the status buffer, is a different question. And yet another question is whether we should encourage this by offering these commands here.


> In fact, version is not an attribute per se, but it makes the expression easier to update and read.

Hence not "exporting" it *as* an attribute (except under `meta` where it's documented) strikes me as the correct thing to do. Like `public` and `private` variables in less civilised languages.

> Maybe monitor thinks the same way, and then updates version accordingly.

Oh, I completely agree. I just wonder why it doesn't use/fall back to`meta.version` instead.

I don't entirely agree that it's a merely cosmetic style issue, but it certainly is a minor one. Thanks!


You're not setting `LOOPEND` before using, looks like a bug.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 214](https://teamcity.labkey.org/viewLog.html?buildId=262857&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 6 (1 new), passed: 19 Build time: 00:35:06
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

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

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

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

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

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

```


add_fixtures_to_VS_solution (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


@jennybc  When we run `make` on this (I'm on a windows laptop at the moment at the moment that doesn't `make` very nicely otherwise I would have run it), it will compile the WebTechnologies.ctv file using pandoc and then build the README from that (via an html intermediary). A bit convoluted, but WebTechnologies.ctv is what CRAN needs and the README is what Github needs, so it makes sense to start from one original file. [This vignette](https://cran.r-project.org/web/packages/ctv/vignettes/ctv-howto.pdf) provides an overview of the .ctv format. It's not the most user friendly, so I've found starting from markdown is quite a bit easier.


RDI? LOOPCOUNT is ESI


Ah looks like you guys finally touched on objects with your last lesson in java. Make sure you become solid with that concept of an object, it is something that will never go away whenever you code in Java (or any other object-oriented language)


whitespaces .. there's a typo ;)


You really should use uuids if possible :)


Don't think you need `force5bytes` here since you're not going to back-patch it.


VBOs have been introduced in OpenGL 1.4. That's 12 years old. It might not be perfectly supported on some very very old graphic cards, but I would personally not bother until we have a real problem.


No objection here.


Hallo Steffen (@stundzig ), ich habe cssName und minWidth aus dem ctor rausgenommen, weil sie an der Stelle nicht notwendig sind, sondern eher verwirren. Man würde den ctor damit praktisch immer mit (null, null) aufrufen. Beides sind Optionen für die es Default-Werte gibt. Genau um sowas gut modellieren zu können, ist das Config-System da. Wo ich das eventuell machen würde, wäre bei zwingend erforderlichen Einstellungen - aber genau das sind die beiden nicht.


@danielweinmann tu já alterou né?


dd


this route should be "/:category" to catch gets to specific categories, then you can use req.params.category in the mongo query


I think sorting the original list is fine. Are these lists used anywhere else after being passed to printers?


Ah, I'd missed the @Deprecated. Ok. Makes sense.

I still think 100 is completely arbitrary value (even more so than MAX_VALUE!), and so high it might as well be 'not specified'.

Even Integer (null = not specified) would be preferable. I don't know what support @Databoundconstructor and the ui framework has for that.


s/ofter/often


s/there/their


this is 3th line in this file


You can just initialize `LOOPCOUNT` to the desired number of iters and then use `SUB`, which will also automatically set the flags if the count is 0.


@stundzig : ich war mir nicht sicher wo du das JUnit hergenommen hast. Bei mir haben die Tests nicht kompiliert. Ich hab JUnit als Library im Plugin classpath reingenommen, guck dir das bitte nochmal an.

außerdem: die jars für die tests sind als lib im Eclipse-Projekt eingetragen aber nicht im Plugin. Das ist richtig, weil die nur beim Testen gebraucht werden, richtig?


@rsivasundaram Yes, this is the more recent one!


Why not take a third arg to append/prepend which is the content?


Oh no I actually meant to comment out the comment. It sounds weird, but it is what it is.


test


That is interesting, do you have a link to the test to run this?


`private`!


[String]


This should be in some other location. Registering one random gun to be the event listener is a very odd and untidy way to do things


don't need this line.


required


all this is useless. You should be using the application.html.erb template. This is all code duplication.


Move upgroup code together.

Update bash/atomic for command completion code.


When I ran the old-version parser, I just ran 1 iteration on test section 2 ~ 6 and the accuracy is 0.868442014165
Now I've run through the whole data set on this patch,
07/27/2015 07:30:27 AM INFO: Feature count: 11844715
07/27/2015 07:30:27 AM DEBUG: Data finishing 0.00% ...
07/27/2015 07:34:46 AM DEBUG: Data finishing 52.06% ...
07/27/2015 07:39:28 AM INFO: Unlabeled accuracy: 0.885664463628 (41140, 46451)
It turns out that the accuracy is still below 0.9


You should use requestAnimationFrame instead.


22:07:19   pvr.hts - sending message [subscriptionChangeWeight : 16]
22:07:19   pvr.hts - write done, socket socket error = 0
22:07:34   pvr.hts - demux read nothing
22:07:34   pvr.hts - Command subscriptionChangeWeight failed: No response received: 16
22:07:35   pvr.hts - bytes read = -22, socket error = Invalid argument

Something really strange is happening here


Yep.


no need to be adding line spaces.


I agree about leaving this code in here somewhere.  It might make sense to move it to a standalone program in devel/external.  But I also don't mind if it stays here.


I noticed two PIN definitions got left out of this file.  They are needed for the INTERCOM feature.  I suppose this feature is still supported, but I do not know if anyone is using it.

    #define TX_ENABLE_PIN            DIO12
    #define RX_ENABLE_PIN            DIO13

I have added them back in another commit (116238d) , but I have not looked to see how this affects the template code.


Yeah, I do think it's nice to be able to bypass the status buffer, but that is already pretty easy to do if the dispatch popup is bound to a global key, like you suggest in the manual.  But perhaps it's still worth binding these here in case the user doesn't want to bind the dispatch popup to a global key.


As an extension, we could define this as something like

`<!-- BEGIN MUNGE: EXAMPLE(filename) -->`

Then you could offer a forEachMacroBlock("EXAMPLE", func(mlines, lineno, filename) { ... }) which updated the file contents.  Just thinking as I read it.


Is it needed to add the log-bin option in the slave as well?


This is to expensive as it adds an additional call to git for most magit commands. We should only do this when the current buffer isn't a magit buffer. When we the current buffer is a magit buffer, then we already know that `default-directory` == top-level.


Fixed


I think it's rather strange and don't really see why this is a good style.


don't use a preview version of build tools, set it to the same as the library.


Please implement comments above.


If you're updating this here, it should also be updated in the library. Else this should be set to 21


I don't remember exactly. I often test various snippets I find on the Internet and some of them has triggered this change. I then thought it might be a good idea for a few languages so I went ahead and added those without any explicit mis-detection problems.


This returns a dangling pointer, as `items`' lifetime is the body of this function.

A hacky solution could be to leak the vector, making it live forever.
```rust
let ptr = items.as_ptr();
mem::forget(items);
ptr
```

A clean solution is probably to store the vector somewhere and clean up in `zbx_module_uninit`.


Yes, I figured. Good idea.


change the applicationId to com.colintmiller.simplenosql.sampleapp


Returning `actor#ID` would require to keep a `string` member around. This approach allows flexibility while not imposing any additional members to all actors. For logging purposes, I think this is easy enough to on the application end. Plus, `stateful_actor` makes it really easy to add a custom name to your actor.


@tkruse @wjwwood
I made some changes to install each completion scripts to the default locations of platforms.
Please review this. If you prefer without this change, I will revert.


I agree this was right before: the parenthesize here is to prevent the directory change in the main script. Also, why does this lose the `local` declaration?


I think the framework can handle this kind of situation too. Without data-activates or id attribute:

```
<a class='dropdown-button' href='#'>Drop Me!</a>
<ul class='dropdown-content'>
    <li><a href="#!">one</a></li>
    <li><a href="#!">two</a></li>
    <li class="divider"></li>
    <li><a href="#!">three</a></li>
</ul>
```

Seems useful to apply in grids, tables or dynamic lists that have dropdowns in each line. Can you add this in the next release?


package should be com.colintmiller.simplenosql.sampleapp


Haha thanks! <3


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1260 of 1260 tests passed)

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


This unfortunately isn't an overly useful sample app. Something that actually takes in input, stores it, and retrieves it would be better than a profiling app. For code samples the test code and documentation already provide what you're doing here.


`\S`? What it the stop name contains whitespace? Like "Zellescher Weg"?


This is not a useful comment. Also, JavaDocs have no creators or creation date in them. JavaDocs should explain what the class is used for.


Will try it in a view minutes. I made also an implementation, but I use `helm-comp-read` for it. The only problem right now is that `ivy` stores the input from the minibuffer rather than the chosen target, as `helm` and `ido` do.


I think this is the same as prefix + 'etc/bash_completion.d'


On second thought, if we add such bindings here then we will get requests for even more bindings, until finally this is a copy of `magit-dispatch-popup`. So it's probably best to remove everything that does not act on the current file and only the current file.


Pretty certain it had to do with pika 0.9.13 being buggy. So we bumped to 0.9.14 (which wasn't officially released yet). Pretty far behind now that 0.10.x is out...


I don't think the kubectl updates them, just screams if they are wrong


@wjwwood I'll let you judge, I am not a MacOS user.


Weird, how the hell did that compile here...


Overall I don't see this as a particularly useful sample application. Perhaps trying to make one that allows the user to input data and later retrieve it, utilizing different buckets and applying their own filters and ability to delete entries.


Usually doesn't make a difference, since "Zellescher" is enough for the API to figure it out. Otherwise one can also use "ZellescherWeg", but theoretically you're right, this was rather lazy.


Replace magic number 50 by MAX_POEM_AMOUNT constant. (define it before replacing =) )


web development - google-homepage


Thank you! :smiley:


just saying, `dvb\s+(.*)` would be better


I found a gotcha that whenever you call @mentions, it returns a invalid id instead of a valid username.


@jquery/core For a moment I thought we'll be able to finally rip out our custom `mouseenter`/`mouseleave` implementation... but no, Safari 7 needs it. :(


If you are accessing a locally installed company file e.g.  http://localhost:8080/accountright then you do not need a dev key.

If you are accessing a cloud file at what point of the process are you failing?
- Getting OAuth token
- Getting a lit of company files via https://api.myob.com/accountright


scitran.io?


@dysinger here's the rebased version of your changes.  I think `withBuildConfigExt` should be able to provide what you need rather than doing your own `lcLoadBuildConfig` and `Docker.reexecWithOptionalContainer`.


Let me suggest to adopt PRs for your own contributions. By this way we can assist you.

Take a look to this travis.yml you may see not everything is executed in every php version. https://github.com/Maks3w/Psr7Assertions/blob/master/.travis.yml


Broken in Ruby code or the JS?


this change causes the issue mentioned in #13


Need a default (otherwise it will error)


Put CategoryDocument in an array


you prefer without `os.path.join()`? ok I will change.


You're probably right. If this script gets a few other functions though we'll have to be careful to match those first or use another keyword besides `dvb`.


I've already removed this section in another commit.


Oh, that makes sense. Sorry about that.


Parameters a & b could be better as "base, override"


I would suggest calling this `parent-url` or `directory-url` or something like that.


I think it is some babel bug. I updated modules today and it broke my tests


Why don't these errors show up in Test/baseResults/preprocessor.cpp_style_line_directive.vert.out?


This has been changed.


looks like whitespace/indentation got messed up here.


@jaswsinc Thanks for explaining. I'm still a little fuzzy on how this fixes the issue (an issue which I originally reproduced in an environment where there were no issues writing to `wp-config.php`), but I just tried reproducing the original double-definition issue with WP Super Cache → ZenCache using this updated code, and I was not able to reproduce it. So, the issue appears to be fixed.


That's probably a good idea. I'll do that and close this.


looks good. +1


This is the main class https://github.com/gustavonalle/ispn-client-test/blob/master/src/main/java/com/gustavonalle/infinispan/perf/Main.java

I don't think the issue is the distributed stream itself, but rather high  contention in the Lucene index and the associated back pressure


SGTM; change made.

https://github.com/scitran/apps/commit/ae9177467dadb5f4e908c7a0ade40f2610ccaff9


I think its in Ruby code and primary key relies on a User ID, not @mentions like @rodrigoargumedo.

```ruby
ActiveRecord::RecordNotFound in UsersController#show

Couldn't find User with 'id'=rodrigoargumedo

Rails.root: c:/Users/Rodrigo Argumedo/code/gorails-episode-78

app/controllers/users_controller.rb:67:in `set_user'

Parameters:

{"id"=>"rodrigoargumedo"}


This one propably https://github.com/babel/babel/issues/2094


Uh-oh! It doesn't make sense that this file is almost empty. Some mistake must have happened before checking in the code.


Burda container'i mi versek?


@brandonly101 Please use sentence casing: https://docs.moodle.org/dev/Coding_style#Language_strings


You should explain this in the commit message. Also, I understand that this simplifies the code, but wasn't the check there for a reason?

IOW, the commit message should say 1) why this makes the code better and 2) why it is acceptable to remove the check.


do we need to keep it on `self`?


ah, yes, I see ;)


@rsivasundaram This file was entirely created by @brandonly101, so it isn't a core edit.


join() is for joining actual path elements. I am not even sure what happens under windows if you you call join('/', 'xyz'). But if that was for windows, then you should also use join('etc', 'bash_completion.d')


@brandizzi Is Nate ok with this? I don't think we're supposed to use Liferay.Lamnguage.get with variables because it'd make a sync ajax request. We should only call it using constant keys (literal strings).


do we still need the prefixed versions?


@mawww However, I'm not really certain about the Gentoo stuff; I've never used Gentoo's ebuilds.


This might be a good place for `JLOG`, since `toBase58` is not cheap.


You're rotating the config, not the logs. Maybe there should be log backups as well in a different commit.


Thinking just need a new local var?


I just committed a fix for the misspelling, but, either you'll have to
figure out a way to use babel es6 modules to `import` this library:
https://github.com/DevinRhode2/sendUncaughtException or simply copy-paste
it in. It may be of interest to modify the module export code at the bottom
or something, idk. I never got to figuring out how to import the
sendUncaughtException library a few years back, but now we have es6 modules
with babel and should use that imo

-Devin <http://devinrhode2.github.io>
<http://zerply.com/devinrhode2>

On Mon, Jul 27, 2015 at 1:32 PM, Tadas Giniotis <notifications@github.com>
wrote:

> this change causes the issue mentioned in #13
> <https://github.com/devinrhode2/extendFunction.js/issues/13>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/devinrhode2/extendFunction.js/commit/32c2a0db168f88b2fa3ba734e5cadcbba00a33a0#commitcomment-12381977>
> .
>


Remove Core Edit comments


@rodrigoargumedo Pretty sure this is the culprit. This should probably be a `find_by(username: params[:id])`. You'll also have to update the `to_param` method on the User model so that it uses the username in the URL instead of the ID number. Also would be good to index on this field so that it's fast.


I shall give it a try again. And I'll come back.
My email is aleixo@muchanga.net
rgds.


You don't need to do anything in the destructor for the `_forceReporter` now. Its destructor will be called automatically, which will cause the deletion.


This shouldn't be necessary because I've declared support for our post type here:
https://github.com/cuny-academic-commons/social-paper/blob/master/includes/hooks-wp-fee.php#L19

Is this working for you, Dan?

We also shouldn't be removing post type support for FEE since other plugins might be using FEE.


@ldoktor I now see your point. I'm updating this one now.


No need for Core Edits, since you created this file.


This commit is empty...


This code technically isn't incorrect but apparently the related compiler bug was fixed in VS2015. Should have checked in a workaround now.


I found the solution, will open a pull request shorty.


I plan to do an integration with our ecommerce system using the API.

Sandro would you be interested in pooling resources a bit ?


There are apparently new test files above, but not added here?  I'm worried it seems an inconsistent testing result state is created.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1260 of 1260 tests passed)

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


@ldoktor I decided to just fix this and add the docstrings. Coming up in a v3.


This has been changed in a later commit.


Yes, I will try to add only the target flags but I am not sure it will be enough.


In fact, it would be need if the emitter had support for 16-bit offsets. Then we could save 2 bytes on every jump!


You don't have to use the `-r`recursive flag here since it's not a directory. It doesn't really matter in this case, but try not to use a dangerous flag like that at all if you can help it.


I don't expect under windows, so just using `prefix +`


I see you are also using a cache loader [1].  This could be affecting it as well.  It would be nice to see what the profiler says.

[1] https://github.com/gustavonalle/ispn-client-test/blob/master/src/main/java/com/gustavonalle/infinispan/perf/config/CacheManager.java#L54


I meant to point out that the change in the line above this comment caused it to break. If you add `var` it no longer works for some reason. I tried debugging it, but I don't know JS to such a degree that I could fix it proper.


I've seen that you are really lock on class field and define variable with same name only after it - but I've understood it only when I've read it second time very careful.
Thanks for quick fix!


Also the chunk size of 2 million seems a bit high [1].  It may be allocating additional memory based on that.

[1] https://github.com/gustavonalle/ispn-client-test/blob/master/src/main/java/com/gustavonalle/infinispan/perf/config/CacheManager.java#L38


TSourceLoc has been passed around by value, everywhere, as what used to be small object.  But between the column and name* change, it's size has nearly tripled.  Should it now be passed by reference?  Either that, or 'string' and 'column' could be 16-bits instead of 32, and, more generally, string and name should be a union, since they are mutually exclusive.


you use shortened urls here, and full urls below. Is it safe to use shortened urls?


looting is very important too


ah sorry, didn't realized that push failed, should be ok now


Wait...was oxli being tested via the command line in previous releases? In any case I'm not sure using load_entry_point in this script is the right move; does it still work with the other install mechanisms!?


You should do a `mkdir -p` for these directories to ensure they exist before copying or it will all fail.


Not sure how I managed to put this in master... supposed to be in Dev.


Certainly but Python2.7 (at least) doesn't support full unloading : http://bugs.python.org/issue9072

I don't know if removing it from sys.modules and itself will be enough.
If it is not enough, we can move generated file in a dedicated folder.


Blasted autocorrect!


Micro*


You should explicitly specify the databases to dump as we don't want the test database, among other things.


Does this make sense on its own?  Or, will it migrate to a fuller feature?  Maybe should only be one extension here when it is done, which includes this.


Improve this part


Even if I fully agree that the previous measures were overkill, by issuing a Close/Create cycle they ensured some cleanup that is now missing. Indeed, this change seems to introduce a memory leak (m_pageCrops is reallocated without being freed) and it maps again the fb without unmapping the previous one (m_fbVirtAddr)
I think they are the only two missing required action to safely push these changes...


That would be the safest thing, I believe; try not to shadow variables. Not that this is perf-critical code or anything. Just pointing it out for conversational reasons. :smiley:


note why, mention the error.


Do we really need it? I mean, we will remove gmp support in a few month.


I build and run the `Debug Testsuite` configuration. But the cmd window disappears immediately after the last test has finished so I add a breakpoint to the end of `main()` in `xbmc/test/xbmc-test.cpp`.


Not too happy with this one. Did you check if backward jumps don't work?


1679-DualDuct (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2172 of 2216 tests passed)

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


:) sadly, it is not supported by windows for now. But at least, it is C++ standard and not posix only.


Inlining JMPs sounds like it'll lead to a clusterfuck.


ok, my fault, didn't know how to deal with git submodules and end-up with this workaround, I close my repo :-1:


Essa função não funciona.


Why are you doing this instead of simply passing "new_offset_ptr" to the inner routine?


I don't see where else you do this, so I'll just add the note here:

When you select an alternative boot enviornment, you also need to set 'currdev', as this is where the loader loads the kernel from. If it is not switched you will boot with the work from the selected BE, but the kernel from the 'active' BE.


I almost wonder if we can re-use some of the code in the SQL's serializer. No great ideas right now, but something to consider.


add_fixtures_to_VS_solution (mbadams5) - x86_64-MacOS-10.9-clang: OK (1770 of 1770 tests passed)

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


I believe this is not working:

```
root@EU1:~# facter --version
3.0.2 (commit 5dc120fa9db4c19150466b1bbd1d0cf42c87c6bd)
root@EU1:~# facter networking
{
  fqdn => "EU1",
  hostname => "EU1",
  interfaces => {
    eth0 => {
      ip => "178.62.178.58",
      ip6 => "fe80::601:5fff:fe2f:c301",
      mac => "04:01:5f:2f:c3:01",
      mtu => 1500,
      netmask => "255.255.192.0",
      netmask6 => "ffff:ffff:ffff:ffff::",
      network => "178.62.128.0",
      network6 => "2a03:b0c0:0:1010::"
    },
    eth1 => {
      mac => "04:01:5f:2f:c3:02",
      mtu => 1500
    },
    lo => {
      ip => "127.0.0.1",
      ip6 => "::1",
      mtu => 65536,
      netmask => "255.0.0.0",
      netmask6 => "ffff:ffff:ffff:ffff:ffff:ffff:ffff:ffff",
      network => "127.0.0.0",
      network6 => "::1"
    }
  },
  ip => "178.62.178.58",
  ip6 => "fe80::601:5fff:fe2f:c301",
  mac => "04:01:5f:2f:c3:01",
  mtu => 1500,
  netmask => "255.255.192.0",
  netmask6 => "ffff:ffff:ffff:ffff::",
  network => "178.62.128.0",
  network6 => "2a03:b0c0:0:1010::"
}
```

My {{ip6}} entries still return addresses with {{fe80}} in them.

https://tickets.puppetlabs.com/browse/FACT-1150


Note: Looks like this earlier pull request is more complete: https://github.com/PolymerLabs/more-routing/commit/de6eeb3c4d278fda8836350cc55c342c7a704a15#diff-380a78efdd09029da1a4c73869147596L167


Yes because OS_SEP is a char and there are no overload for this. We could add one but I didn't want to much work on this support.


Yeah it's the one, thanks!


I don't see any advantage in doing this over just complaining once during shader compilation. This will only generate extra spam.


Straightforward.


Because on windows, both separator may be kept. (I don't know why, it looks like a bug but it is what is done by os.path.join)


Hi @queeno

The puppetlabs-apt API changed between 1.5 and 2.0. So my idea was starting with the new API for this functionality / future use and not the use the "legacy" module version.

Maybe there could be added a case handling in the puppet module for the old and for the new api ... never seen that till now.

Cheers


Comment inside the block. The one above the if too.


snprintf is available in the bootcode, would it be easier to just do:

snprintf(mountfrom, sizeof(mountfrom) - 1, "zfs:%s/%s", spa->spa_name, rootname);


This would have been helpful today, as I try to relate a set of annotations in the sidebar to their highlights (or lack thereof) in a long PDF.


Ha. Well, that would do it. For some reason I had assumed the idea here was to have something else perform the exit step via the event.


Sorry, I have really no idea.


I still don't like this. If we change it to "position", can we not add offset to the context?


1679-DualDuct (kbenne) - 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)


getline doesn't exists on windows. (At least on my windows). It looks to be a posix think and I didn't find any other function to do the job.
I didn't perform perf measure on it.


Decided to just use `!class_exists('\Sodium')` instead.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.834.6e7d343143841419dabd153d9ae4c69f1a6638ae](http://build.palaso.org/viewLog.html?buildId=60063&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 395, ignored: 3 Build time: 00:02:45


ah sorry, forgot to say: my test was with only one node, and I changed the config to not use a cache loader and to store index on RAM, infinispan or File. I also configured the heap to avoid excessive GC  :)


I'd recommend putting this in utils/StatUtils rather than LuceneUtils (since there's no Lucene dependency).


The other (perhaps more desirable) way to write this is

```
if (token == ppAtomConstString) {
    requireExtensions(loc,...E_GL_GOOGLE_cpp_style_line_directive...);
    ...
```

That way, the user gets an error about what extension is not turned on that they might be using.


Does a `testpack` target exist?


BTW @kares :  The reason for the upgrade on 1-3-stable is to test SQLITE3 + PREPARED_STATEMENTS="true" with version 3.8.10.1 in travis-ci since that worked for me locally.


should be lower case


See other comment about requiredExtensions();


Soon @mzgol soon :-)


don't use Russian characters


Looks good, modulo a few line notes I made.  Can you address those?


Which branch is this?


Since there are no comments to say on the code, I'll allow myself to report a small typo here ;) s/do/does/
No need to fix it before merging the PR though.


lgtm


After a lot of trial and error and some help, we figured out we needed to use cpanm to install the modules instead of yum or anything else. And also the scripts require a custom Encode module, i believe it's 2.49.


Thanks


move "else" to next line


I'd suggest using `Zotero.Utilities.xpath` (or `ZU.xpath`) instead, which makes the code cleaner. You also don't need an `nsResolver` in that case. (take a look at some more recent translators if you need more guidance)


fro->for


> When the machine is cold from start the bed slightly changes so the measurements are never the same.

Check! That's something we need to mention on our Auto Bed Leveling page on the wiki for sure.

> People say that the machine and correction planes are one

It's pretty straightforward. Normally when you set up coordinates (`current_position`, `destination`, `target`, etc.) and send them to the planner, the planner will faithfully move to the given positions. So, moving in the X plane will only change X, etc.

When ABL is turned on the __planner__ takes the given coordinates and runs them through the __rotation matrix__ before it sends them to the steppers, so the __planner coordinates__ and the __idealized coordinates__ will differ. As you move in the X and Y planes, you will get movement in the Z plane, and as you move in the Z plane, you will get movement in the X and Y planes.

(Note that I use the term __model coordinates__ and __idealized coordinates__ to mean the same thing – the ideal model-oriented coordinates that the slicer generates.)

The interesting thing is, the __Marlin code__ can always ask the __stepper code__ for the stepper coordinates (the __machine coordinates__), and put those back into the `current_position` so that the `current_position` now has the true machine coordinates. Marlin should do this when turning off ABL (I'm not sure if it does at this time) so you don't have to re-home to obtain the true machine position.


That code is about picking what is considered to be the primary interface, which in this case is `eth0` because either it is the interface that `0.0.0.0` is routed to (i.e. the interface considered default for IPv4 traffic) or because it has an IPv4 address that doesn't start with `127.`  If it is not the interface for the default route and had an IPv4 address that started with `127.`, then this code should preclude considering `eth0` as the primary interface.  It isn't intended to hide the link-local IPv6 address.


Discussion moved to issue #179.


perfekt!


The way the mass indexer is on master, there are too much threads busy already: lucene uses np/2 to do merge, hibernate search uses another to apply changesets, index deletions are handled asynchronously by another executor, adding more is not doing any good :)


Makes sense.....

On Mon, Jul 27, 2015 at 3:59 PM, Dominic Widdows <notifications@github.com>
wrote:

> I'd recommend putting this in utils/StatUtils rather than LuceneUtils
> (since there's no Lucene dependency).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/treversec/semanticvectors/commit/3edf56d393ed1c4a07ce8979b883cf51f20704fb#commitcomment-12382514>
> .
>


This seems to get called multiple times...


Looks like FreeBSD 11-CURRENT to me


this should be an asynchronous call. Pass a callback function as the second parameter to `Zotero.selectItems` that takes one argument, which is an object containing selected items (same format as `items` you pass in). Return true if returned object is "falsy" (I think it's `null`, but it may be `false`).


Are you sure that you want to recommend `done` and `fail` versus `then` which is becoming standards compliant in 3.0?


The design pattern I've used for service communication is quite common and
allows for adaptability.

However, feel free to contribute upstream - I'll be more than happy to pull
changes!
On 27 Jul 2015 8:21 p.m., "Ronan Nello" <notifications@github.com> wrote:

> i've recreated the app from scratch, based on yours :
> https://github.com/flyingrub/MiBand-Notify.
> There is only one service running in the background and no need for your
> billion of intent/broadcastreceiver :).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lwis/miband-notifier/commit/9192266545352003aae880f196beb90aadc1cb0a#commitcomment-12380573>
> .
>


yes the syntax has changed, in main:

```
ofGLWindowSettings settings;
settings.setGLVersion(3,2);
ofCreateWindow(settings);
ofRunApp(new ofApp);
```


Do we still want to keep this line or the 'WUStateAborting' should not be set here?


you don't need the `Zotero.done()` callback and you don't need `Zotero.wait()` below. Those are outdated constructs that are almost never necessary.


With the above you can run - `jspm install semantic-ui` and should just need `import 'semantic-ui'` for everything to work. If it's not working correctly please do post back though.


Also, regarding the binary log name, what about renaming it to include the -log on it. Right now It might be confusing with the binary executable name.


Well, that appears to be what's currently happening though, it's seeing it as the primary. Or at least, I would expect the `ip6` facts to return the "primary" IP.


I'm a bit puzzled as to why in some commits the `;` is added, and others c078ab1a81e9cac62f58213b2d9ffb40cf269fe8 its taken away?


Looks like a sane change.


hi @ilganeli sorry I should have made this more clear, you gotta delete these printlns before submitting to the PR builder, they will fail the style test.  I just find it way easier to work with than log statements when I am really focusing on one tests.  You could replace with logging, but really I think we probably want to just delete before merging.


Why did you choose to set up the binary log format to ROW and no Mixed for example?


Would be swell to have a checkbox in the Qt UI like we have for the hwrenderer.


So that this:

Becomes this:

Once the screen is under 70 characters long.


Note: Removed the check on what vessel this is and checking the external boil controller. Assumed the boil controller has cleared the setpoint. This hasn't happened but the PID Mode is being toggled. I will add a check to PID mode.


Correct.


this is actually the same as `.issueDate` See http://aurimasv.github.io/z2csl/typeMap.xml for all item type-specific fields and their base fields (in parentheses).


Needed so saved pages work!


I'm very reluctant to make any changes to the license, it's a pretty standard `BSD 3-Clause` license

http://opensource.org/licenses/BSD-3-Clause

What is the reason for this change?


@adelevie Shouldn't it be updated to 1.0.0 on rubygems?


`HADDPS` is SSSE3 only. Just implement this with shufps. It'll save you from loading the constant too.


Nice


Same.


It's actually a `3-Clause` license.


Can you run the most recent commit. Check the UAS which is typically a but
higher.

On Tuesday, July 28, 2015, Yizhou Wang <notifications@github.com> wrote:

> When I ran the old-version parser, I just ran 1 iteration on test section
> 2 ~ 6 and the accuracy is 0.868442014165
> Now I've run through the whole data set on this patch,
>
> 07/27/2015 07:30:27 AM INFO: Feature count: 11844715
>
> 07/27/2015 07:30:27 AM DEBUG: Data finishing 0.00% ...
>
> 07/27/2015 07:34:46 AM DEBUG: Data finishing 52.06% ...
>
> 07/27/2015 07:39:28 AM INFO: Unlabeled accuracy: 0.885664463628 (41140,
> 46451)
> It turns out that the accuracy is still below 0.9
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sfu-natlang/glm-parser/commit/641428b53d8c15056e1cf2ac74c2624f6d7c2677#commitcomment-12381739>
> .
>


Результат у них разный.

не факт, что именно с 14 шрифтом результат setspace, заточенного под 10,11 и 12 (остальные по остаточному принципу там), будет лучше.

linespread ближе к "топорной" реализации ворда.

Предлагаю оставить как закомментированный вариант.


In case that we pass the MARIADB_DATABASE to the master container, shouldn't we restrict the replication access to only that database?


I think it is enough to just write
``` emacs-lisp
:preselect (format "^%s" (car helm-make-target-history))
```