WARNING! Makes offset work in opposite direction! Danger of chrashing nozzle into bed! See code.

However, fixes #2357


My refactoring of [this](https://github.com/edx/edx-platform/commit/4ba5e104e9825f04daa1418de79595c4a61b419e#diff-be0481503e8fd3f1d02e60ef92bc39f4R508) function causes `_has_staff_access_to_descriptor` to be called once more than before in certain scenarios.


why do we need to use pointer events? is this to support window 8 tablets?

just wondering why these changes are in this commit as it seems better to have them separated from the cleanup stuff.


Why isn't the body a factory like the $rootElement? Or vice versa, why isn't the $rootElement obtained like the body?


This is roughly what `Datastore.Query` is for -- not listed here, but this is: https://github.com/jbenet/go-datastore/


How about a logo kit for promotional material etc.?


``ensuretheminoftherequestedroomandthemaximumsizeofthebufferinthecasethathisbufferhasamaximum``


@akbertram would you please check? it seems conflicts are not resolved correctly. fillForm is also in ApplicationDriver.


LGTM, except for the question above.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.698.a46c0a67c3afa99748ae76925488c03179c31086](http://build.palaso.org/viewLog.html?buildId=59051&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 355, ignored: 3 Build time: 00:03:11


@ma2moto this should be `ブロックを無効にする` (without the `"`), no?

There's a missing `=` sign. I'll add it in the next version.


Let's name this server-dependent or something like that. Would be clearer.
Strictly speaking, every cap can be part of CAP NEW. But this caps is for strings which should not be offered to anonymous user, or when IRC server is not connected.


? It's incremented in the `if (...)`.

We now `getAndIncrement()` and then mod the old value to see if it's time to capture the time. It's incremented regardless of whether we do so.


const auto&


Just auto makes a copy


I've replaced all the headers with the proper Licence and Copyright now. (commit : 40ef4bff94f2630044d4a4fd0d395fdf2a5d73cf)
Since the Apache licence is going to be used in this project as well, I've taken the OBA android example and re-purposed it for this project.


Will do,  did that shit has asleep 😀


Commit message^^


Regarding 32bit starcheck: PGPLOT is broken again in the last build on the 32bit virtualbox, and I didn't notice because of all of the warnings I was already getting in 32bit starcheck on HEAD when I tested (more the sybase failures).  PGPLOT was actually working in the dev build of these same packages.


should include 'user_staff' here since staff users bypass the mobile_available flag.


@nyalldawson Could you try adding

    QgsOgrConnPool::instance()->invalidateConnections( filePath() );

at the end of `QgsOgrProvider::changeAttributeValues`, before the `return true`? If that solves the issue, it appears that the internal state of the ogr connection handle also depends on the actual attribute values...

@gioman Yes that sounds like the same issue...


i want the Key and Block objects for the added type safety and the methods:
- https://github.com/ipfs/go-blocks/blob/master/key/key.go
- https://github.com/ipfs/go-blocks/blob/master/blocks.go
casting is not a huge deal because they are the same underlying type, so it's not expensive to go. only annoying to humans (but maybe also helpful with type safety)


Looks like this is breaking everything :)


`OpenRepository` signature returns `(*Repository, error)` and it has been updated to use `initRepositoryObject`.

But `Clone` func also has signature that returns `(*Repository, error)`, yet it doesn't call `initRepositoryObject`. So now if you do `repo, err := Clone(...)`, then `repo.Remotes.Lookup` will result in a nil pointer panic since `repo.Remotes` will be `nil`.

Before this commit, doing `repo.LookupRemote` worked okay. Is this intended, or a bug? If it's expected behavior, what's the expected usage of `Clone`, are you supposed to close/ignore the returned `*Repository` and use `OpenRepository` afterwards?


Rather than include this in the dracut package I think it makes more sense to create a dedicated __zfs-initramfs__ package.


See #35


I'll do a pull request!
--


This is all dist stuff - its whatever was made from when I ran the build script. I assume this is stuff that hasn't been built and committed yet in the main repository. If you like I can separate the built artifacts into a different commit.


Nit: consider renaming this to `assertCanAccessCourse` so it's clear when reading the callers that it only applies to courses.


I think @JeanFred forget this one when doing 15ea5c87ac6e2646097b2d17adb655ba14d81ebe


You might want a sentinel like None instead to avoid the [mutable default arg](http://docs.python-guide.org/en/latest/writing/gotchas/#mutable-default-arguments) :grin:


[It seems to be](http://docs.openstack.org/admin-guide-cloud/content/section_users-and-projects.html) part of essential openstack glossary. I am open to improving the help further, though I do not think Jenkins plugins should explain what that is. Suggestions welcome.


@dharaburda d'oh. I thought I had run the test suite. Shows me to check in things in a hurry. :)

As to 2.6.0 -- was hoping by end of June, but not happening now. Not 100% sure if there will be rc4 before 2.6.0 final, but regardless next release will happen at earlier on week of july 6th (I'll be on vacation until then). But at the same time, planning to get the release out by mid-July now.


Turns out they're both correct spellings, but I'll change it to 2 Ls for consistency


+1

should we go for the standard go marshal/unmarshal?
```go
func Marshal(DAG.Node) ([]byte, error)
func Unmarshal([]byte, DAG.Node) (error)
```


Thanks


Good catch. It looks like @adampalay added this feature with this story:

https://openedx.atlassian.net/browse/PLAT-122

I don't exactly remember why I changed the default to including contracts, but there were some code paths that wanted it. I'll switch the default back to 'no contracts' and fix up what breaks.


Let me know if I correctly understood: take for example Google Play Services, is this the structure that will be generated?
Core
|-------- gms
            |-------- 160
            |-------- 240
            |-------- 320
            |-------- 480
            |-------- common
            |-------- nodpi
With 160 being the -432 variant that's not available and nodpi being the universal -430 one?
Is it right?


消してください


It looks more like an unintentional omission and that `Clone` should also have its `runtime.SetFinalizer(repo, (*Repository).Free)` line replaced with `initRepositoryObject(repo)`.


Is there any way we can leverage the existing `zfs-functions.sh` script to provide this functionality.  It's a shame to duplicate this code if we don't have too.


We don't need to mention whether it's blocking or not - user's won't care. Can be renamed to `artifactQueueDumpFile`


I think right now the behavior actually reflects what you've written in comment 2.

Stylus doesn't actually lookup files on file-system if a file is importing using relative path. It expects the user to set the `include` option to the location of the compiled file on disk to do this. If you don't set and include paths, effectively, you always get to use your custom importer.


I think capability shouldn't be offered to client until it got negotiated with server.
Capability should be offered if:
* ZNC supports the capability
* Server replied ACK when ZNC enabled it

Otherwise, client can request cap which is not supported, or not enabled


sry, pressed enter too soon. We probably want both:

```go
func Marshal(DAG.Node) ([]byte, error) { ... }
func Unmarshal([]byte, DAG.Node) (error) { ... }

func (DAG.Node) Block() error { ... }
func Parse(blocks.Block, DAG.Node) error { ... }
```


"will produce"


Oh yeah that's bitten me before, thanks


That's a good shout; how would you suggest doing this without re-calling lxd.NewClient()?


what about:
```
return allowNonRefPaths ? edges.stream().filter(e -> !blackListedEdges.contains(e)).findFirst().getOrElse(null) : null
```


This is not good and I don't understand what does code in lines 161-167. Please fix, migration completion isn't indicated because of this.


It's funny because I've definitely relied on that fact while using CourseFactory before, but for some reason thought it was necessary to generate my own while writing this

#9amCode


We shouldn't need a `.gitignore` file here since the script is tracked by git.


I would revert this code change so it is optimized to not compute `is_masquerading_as_student` when `start_dates_disabled`.


Sure thing :+1: Thanks for the library!


Fixed


[style] I'd add initramfs after dracut since they provide the same kind of functionality.


I think this can be done without needing to calculate size. see http://blog.golang.org/go-slices-usage-and-internals


This is a raw [string literal](http://en.cppreference.com/w/cpp/language/string_literal) from C++11
@github


Looks good! :rocket:


Then use iface. You never start arguments with a capital letter.


Do you mean that why don't we have a `$body` service? That would clean up many lines of code across ngAnimate.


missing EurekaRegistryConfig binding?


Not pulling, but some related changes in following commit: https://github.com/Pricetx/jonathanprice.uk/commit/880aa9a109310bdd20f0cbebadbd9843f2a0ce6c


missing a few bindings?


Maybe use Guava Objects.equal() for null-safeness?


Aaaaaaaaaaaaaaaaallright, man får lære så lenge man koder ;)
Jeg har rydda opp i koden nå og kjører ny commit.


edges can be final if you refactor the if at the end of the method.


get a reference/shortcut to the vrparameters and the dash tree


It works...only a building problem. Now all screens are OK for s8 thank you
very much

Inviato con AquaMail per Android
http://www.aqua-mail.com

Il 29 giugno 2015 22:17:37 AdrianLxM <notifications@github.com> ha scritto:

> Sorry, via Githup I don't see the picture.
> I think you have to upload it.
>
> I don't build apk's I directly start them from Android Studio.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/StephenBlackWasAlreadyTaken/xDrip-Experimental/commit/0640e6c02812a6735e23d35aa063510f382d0bd9#commitcomment-11918362


Yep, upgrading bundler was enough, the documentation for v4.2.3 and v4.1.12 has been now generated.


All english comments should have a space between the // and the first word. Please go through and fix.


done


LGTM, except for one inline comment.


Did I? Thanks :)

(was it crashing the webapp?)


I  would rather fail with a IAE if passed a null; but I guess there is a risk some code actually depends on this behaviour... what do you think?


Please add note that the 'detached' tag of the descriptor is to be checked by the caller.


The problem is that the default for the ``devstack`` command is that contracts are on. I suppose it makes sense that this is only an option for ``devstack`` and that all other commands don't support contracts at all. I think my attempts to standardize this was probably a bad idea, so I'll remove the setting from the other commands.


Same as above I would like it to fail with IAE when passed a null.


continued roxygenization of package, documentation (such as it is) for all data objects joined in one file. Added file 'geo.R' documenting the package with import tags. Fixed incorrect import from splines, removed last.warning.rda, added @useDynLib tags to more files (might be better placed in the pacage doc file 'geo.R')


Just 'Set' or no verb at all. Most of the options have no verb ;)


Requests=>Request? It should only be Requests if you can send many.


You seem to prefer using ADDR rather than `size_t` directly. Is there a reason for that ?


@jglick Please, remove {{{import hudson.model.Node.Mode;}}} never used import


I would revert the code so it checks the `ENABLE_PREREQUISITE_COURSES` setting before checking the rest of this method.  This may account for the extra SQL call in your unit test.


:+1: Polymorphism to the max!


typo 'Calcualtes'


I got to update GitTest, I'm afraid.

Welcome to Git (version 1.9.5-preview20150319)

Run 'git help git' to display the help index.
Run 'git help <command>' to display help for specific commands.

Anton@SERRAMESTRE ~ (master)
$ cd ProvaGid

Anton@SERRAMESTRE ~/ProvaGid (master|MERGING)
$ git status
On branch master
All conflicts fixed but you are still merging.
  (use "git commit" to conclude merge)

Changes to be committed:

        modified:   index.html

Untracked files:
  (use "git add <file>..." to include in what will be committed)

        TryitEditor_files/
        index_files/

Anton@SERRAMESTRE ~/ProvaGid (master|MERGING)
$ git commit -m "modified: index.html"
[master ed0773c] modified: index.html

Anton@SERRAMESTRE ~/ProvaGid (master)
$ git pull nouremot
You asked to pull from the remote 'nouremot', but did not specify
a branch. Because this is not the default configured remote
for your current branch, you must specify a branch on the command line.

Anton@SERRAMESTRE ~/ProvaGid (master)
$ git pull nouremot master
From https://github.com/aserratorta/GitTest
 * branch            master     -> FETCH_HEAD
Already up-to-date.

Anton@SERRAMESTRE ~/ProvaGid (master)
$


That makes sense to me to ensure a common behavior, and eventually expose the native one if it differs in a significant manner.


@horak!!


Doh! Your correct.


We could define type aliases `type ComponentF s f g = Component s (Free f) g)`, `type ComponentFC s f g = Component s (FreeC f) g` to make common cases easier.


Except for ``toId`` in the sense of getting ``obj.id`` / ``obj.userid``, ``get`` doesn't look like the right prefix.
Imo conversion functions must start with ``To``, or have any of ``to`` / ``To`` somewhere in their name.

``Tools.String`` / ``Tools.Id`` look a lot like a constructor, Adopting those names would severely hurt readability imo.


No it was flagged as Error in landscape :)


LGTM


Yeah definitely, I like that.


Elasticsearch is misspelled.


@ivogabe This check is cause the compilation to fail because the file globs are not matched

tsconfig.json

```json
{
    "version" : "1.5.0-beta",
    "compilerOptions": {
        "target" : "es5",
        "module": "commonjs",
        "noImplicitAny": false,
        "removeComments": false,
        "preserveConstEnums": true,
        "outDir": "dist",
        "sourceMap": true
    },
    "files": [
        "src/**/*.ts",
        "typings/**/*.ts",
        "gulp/**/*.ts"
    ]
}
```


You can also do `for section in ConfigManager.get_sections()[:]:`


This is misspelled.


Acknowledged, but I pretty much don't want to waste precious life time on DVD code, or working around DVD code. Life is too short for that. So unless someone rewrites it in a way that doesn't get in my way, it'll most likely go.


Cache the daemons (in a hash).  This might end up taking way too much memory, otoh if you're acting on 20 containers on the same remote you could end up only calling NewClient once that way.


be checked -> be checked by callers


Typo in `untrackCalbacksPayload`, should be `untrackCallbacksPayload`.


Er, but what about the block at line 898?  Doesn't util.find/lookupIndex involve actually looking on disk with `glob.sync`?


@manisandro yep, adding that line fixes it


@mogadanez please see #78 for your suggestions.


Please add the docblock.


This generates "Offset" instead of "buffer.Offset", since `nameof()` uses the name of the last component in a qualified name. There's no parameter named "Offset". (And I think the idea with `Argument...Exceptions` is that they should just refer to the parameter and not to a property path.)


Typo: dependant -> dependent


were these added on purpose?


You can add this item serializer in the configuration file `resources/config/repository.php` :)

```php
'fractal'=>[
        'serializer'=>'League\\Fractal\\Serializer\\JsonApiSerializer',
        'params'=>[
            'include'=>'include'
        ]
    ],
```


and these?


Aren't you already in this directory? I don't think `cd`-ing here makes sense.


you need to specify an appropriate precision and scale. http://stackoverflow.com/questions/1019939/ruby-on-rails-best-method-of-handling-currency-money


90946736_SHRgt1_4567 (Nigusse) - x86_64-MacOS-10.9-clang: Tests Failed

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


@jzoldak I've pushed up a commit that only supports contracts for ``devstack``, and fixed all the tests to match.


Agreed; There seems to be something in RSpec that's swallowing exceptions thrown during the setup steps, including the exceptions I added to the Connect gem to warn the user.  I'm tempted to `rescue` them during the setup step and `exit 1` after printing them to stderr.  Seems kind of nuclear though.


Fixed


We don't need this anymore, just use the default "invalid' value.


Yes. And I was referring to that fork as well. It builds fine with VC14. See the 'excel' header in
https://phpdev.toolsforresearch.com/php-7.0.0alpha2-nts-Win32-VC14-x86.htm


Thanks!


@nyalldawson Ok... Can you commit that change?


Why did you create this global self?


If only there were an elegant way to prevent extra maintenance of this list.  Like returning `_has_access_course_overview.checkers.keys()`.  Hmm..


Great, thanks!


Fixed!


@glasser yes, but look at the arguments: it performs the search only relative to paths given in `this.paths` which is the list of `include`s you set when a file is compiled.


good call, removed the redundancies


it is not a search relative to the original file, unless you set it explicitly as `include`


It is possible in other languages to do urlAddress[:-1] or ssomething close to this but not in GO. Every example I have searched you need to find the length to find the last element of the slice


Instead of `cd`-ing into libtiff-ios and eventually `cd`-ing back out, how about just calling the scripts from the current directory? i.e. `./libtiff-ios/build-png.sh`, etc.


What


this is somewhat funny since i struggle to build it for hours now... :-/


This can be squashed into the previous commit


I'm not versed into these things, but maybe we could use one of those packaging libraries to generate a standalone `.exe` with a proper built-in stripped down version of a Perl interpreter. So, for users without Perl, download the batteries-included `makeglossaries.exe` and have fun. :smile:


...what was wrong with using `auto` here?


From its fields it looks like it represents artifacts queue rather than configuration. Moreover it's not clear why it's in `api` package..


Just to add that I agree that this was spurious in the end. Sorry for the noise!


Our deepest apologies @skalnik


No need to name it after `Blocking` Queue.


@manisandro will do- what about the other methods, like QgsOgrProvider::changeGeometryValues? Will they also need the fix?


1679-DualDuct (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2181 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.42%25-yellow.svg)


RIP `ero`. My favorite characters in .@tec27.yml


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

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


sorry @skalnik


It's hard to tell but I don't think this `cd ..` is correct. Wouldn't this remove you from the "TesseractOCR" directory? The next line is to `cd` into the leptonica folder, which I believe is in the "TesseractOCR" directory.

My main assumption here is that the user is running "build-dependencies.sh" from the "TesseractOCR" directory (where the "README_howto_compile_libaries.md" file exists). Is that what you're assuming as well? If so, does running this script from the TesseractOCR directory work for you?


Signed versus unsigned comparison.  auto picks signed int for i.


RSpec weirdness would be easier to isolate if we didn't support all RSpec versions going back for years. It'd be nice to reduce some of this technical debt then improvements like this would be easier.


I'm sorry, I really was sure that the implementations all agreed on what was whitespace. Okay, in that sense I guess we should indeed roll our own, specified, one. But IMHO it should handle more than `\n`,`\r` and `\t`


1) App Key is the *old* way of identifying an app, we're trying to move to exclusively using Branch keys now.

2) Registering a key isn't associated with sign up, it's associated with creating an app. One key per app.


Why do you think the collection is called a *Blocking* Queue? :)


No dash.


Practising


gotcha.


ah. ok.  It wasn't immediately evident given the code-style change.


For reference: https://github.com/rakudo/rakudo


really?  good to know.


I know that the way this was displayed previously wasn't perfect, but no there isn't really clear indication what this SDK actually does. Also, the "invite" name is a misnomer, as this includes three different feature sets now.


Standard me donava un error raro usant side i ho he pasat a n i ha desaparegut l'error, why? :rage4:


See line #177


This is not working!


On every save/delete we want to copy the whole file?


Board Game Tool/Collection Game Tool/Board Game Tool.csproj

We should have a more descriptive name than logo.ico


:joy:


sorry forgot to include the config file


You can consider using PyContracts (search for `@contract` in the codebase).  However, it does have a performance cost and I wouldn't use it unless we think it will be valuable in catching type issues.


I don't think you should be catching this error.  Something really wrong is happening if `ip addr` is throwing us invalid data, so we shouldn't just mask it.


Thanks @tomsilver, good catch. Fixed in #147.


You could add a method `addAll()` into the `ArtifactMetaInfoBlockingQueueDump` to abide Law of Demeter.


:joy:
This brings back some childhood memories, for sure.
https://en.wikipedia.org/wiki/Mouse_Trap_(game)


I was thinking more along the lines of fail-safe, but I see your point


v4 and v6 functions look very similar.  Can you combine them?


@nyalldawson At this point I'd say yes, anything which changes the dataset. Situation should be:
- `add/deleteFeatures` are taken care of via `recalculateFeatureCount`
- `add/deleteAttributes` are taken care of via `loadFields`
- `changeAttributeValues` and `changeGeometryValues` need the fix

I think that should cover it?


not `routes`; they're addresses.


Is that all? No logging, no re-throwing, just printing stack trace?


Maybe add:
```
assert(
    all(k in COURSE_OVERVIEW_SUPPORTED_ACTIONS for k in checkers.keys()) and
    len(COURSE_OVERVIEW_SUPPORTED_ACTIONS) == len(checkers.keys())
)
```
to the body of the function? It wouldn't prevent future maintenance, but it would make tests fail if they ever fell out of sync.


LOL yeah this was when I was pumping out plugins everyday just copying and pasting the first plugin I made as a skeleton and then "replace all" keywords for documentation and plugin configs. I'm fairly certain there are a few more like this


1679-DualDuct (kbenne) - i386-Windows-7-VisualStudio-12: OK (2192 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.92%25-yellow.svg)


The log in Freenect2::log is a noun. The log in Log::log is a verb.

Can we use Freenect::logger, and Logger::log?


What would really be nice would be refactoring this whole file to not depend on these strings and dictionaries being right :P


s/notifiable/server-dependent/


No syntax highlighting anymore? :-(


What do you think? I've never used PyContracts before


Do you want to log the exception here ?


ditto


Did you try PHP 7 Alpha 2? The sources are at http://windows.php.net/qa
And the dependencies are now available at http://windows.php.net/downloads/php-sdk/
You will have to install VS 2015 aka VC14. The Professional RC is free:
http://go.microsoft.com/?linkid=9877557


Oh, you get a notification for forks, too? Didn't know, that;s why I moved it to your repo. :frowning:


Correct. More changes are  queued btw, I am almost done, just getting rid of the very last bugs (unzip on android/twrp seems to be broken/different from the ubuntu one)


yeah, but won't it still do the wrong thing for `@import '/foo/bar'` which we intend to mean... um I'm actually not sure what we intend it to mean...  but presumably not '/foo/bar on your hard drive'?  In `less` I made that mean `/foo/bar` inside the current package/app.

Also btw do we need to reimplement the stuff in `lookupIndex` where you look for `index.styl` and `{basename}.styl`?


One nitpick is I see multiple levels of nesting in code. Though this is probably fine, it can be avoided with some method and some shifting of the logic. But it is not a big deal.


Too much logic for a constructor, WDYT?


I was ready about to submit the exact same thing...but my numbers were:
{0,10,35,75,75,35,10,0}


@jesilz Are these really all undergrad student levels?


@ariccio  I have accepted your patch once and it brought 2 important regressions. I don't blame you because it is my responsibility to check your request. Afterward you have done 2 pull requests for "fixing" 2 bugs. However, the both pr do everything-else but fix the bugs. After checking and getting these bugs fixed, I was surprised you didn't even producing the bug, but just shooting in the dark instead.

The rules we are talking about will avoid such problem: it makes contributors focus on the real problem in order to provide pr more pertinent, and save project team time to accept more useful pr. For people who want to contribute whichever modifications, they could still use issue. In fack, the new workflow is even for enhancing the communication between contributors and project team (as @vgturtle127 suggested):
Create an issue before coding, and contributors code and link their pr to the issue untile the issue is categorized (bug or enhancement). That avoids unecessary coding efforts and ensures project team reviews your pr.

Believe me, I would accept all pr to make everyone happy. Unfortunately in the real world, we cannot manage a project without any strictness and consistency.


Well still, there is another way to do it using strings.TrimRight()


Instead of fixing this method, we should have said, all stores will decide the version based on the timestamp from epoch. This would have made the code simpler


uh idk merging something


@manisandro this ok? https://github.com/nyalldawson/QGIS/commit/ca1670357c0c28d12d115ad4285bc2a39075dd08


Since this assumes that vertices are an ordered collection (extrema), should this function take abstractgeneralgraph or simplegraph?


let me know what You think!


oh, I didn't know that's a feature of stylus. I reimplemented relative paths in the same package/app, but not globbing and stuff like that. At this point I am not sure what is the best way to about it:

- for Meteor, the batch plugins API doesn't get access to the actual FS
- stylus uses all these features by calling other libs such as `glob` and they just use `fs`

So either:

- we need to reimplement stylus features on top of our in-memory fs
- or somehow construct things on cache fs and let stylus handle all the imports

Either way, this looks unnecessary time-consuming


The `fill_...` function? Could be `fill_class_relationships` I guess.


You read my mind.  The obvious next step is to make this a lot more extensible than it currently is.


See here for data http://www.ons.gov.uk/ons/rel/census/2011-census/population-and-household-estimates-for-wards-and-output-areas-in-england-and-wales/stb-population-and-household-estimates-for-small-areas-in-england-and-wales.html


Ktos tu chyba przeoczyl komentarze wieloliniowe podczas nauki :)


@nyalldawson Yes, thanks a lot.


Not sure, didn't really know what to do with this.


CLEM DOWN


I think you can simplify this and move the declaration of `sorted_tables` and the `std::sort` call here, I don’t think you need the sorted tables anywhere else.


I work my way backwards through the linked list, so I should never try to remove an item that doesn't exist. Depending on how many items we have and the complexity of the object, I would rather stay way from list duplication.

I do remember that spawning a lot of balls would cause an out of bounds error, but I think that was because of the mouseClicked event and not specifically this section.


I mean can't we just change your patch so that if options.importer is specified, we skip the couple lines under `// Lookup`?


You don't need a webserver to actually run this. You can use the file open dialog in your web browser to run this, since it's all static JavaScript and HTML5. There's no server-side scripts that'd require a web server service.


Literally opening index.html in firefox's file open dialog is enough to load the emulator. You just need to provide the page after opening the BIOS and game ROMs, and then click play.


All the addVertices and edges operation here are used by Test code and a single production class with rather simple inputs like a couple of vertices only... I would move this method to some test utils class (perhaps a new one) in order to avoid pollution the class with redundant methods.

I think I am the author of this sorry but in my defense perhaps was being used by the "deseased" GraphBasedLikelihoods.


The max-width: 100% in .dropdown-menu makes the item dropdown menu to small. All lines are cut on the right in my config (a bit larger font size and Dutch as language). In my next commit I will revert the max-width.


done


Is this comment about CLI behavior relevant to the constructor?


Yay GW!


I'm looking into inc.updatebinary.sh and I found a way to select the correct version based on the device density and on the available packages. Tomorrow I'll refine some things and push it, but if you're done with it, go ahead and push it when it's done.


What if I call it like ```addEdges(someV)```? this won't do anything but does it make sense at all? Check whether remaining length is 1 or larger.


hpwh_residential (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1672 of 1679 tests passed)

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


you need `sorted_tables` inside `FixChecksums` as well, where it is looping through each `tables` and storing the checksums in the table directory entries.


How did you determine your values? :)


do when need to check wether next is null?


could probably just use QubitPdo::fetchAll() in this case rather than prepareAndExecute?


WHAT.


We should issue a deprecation warning in the XWiki logs when using the old deprecated "space" parameter IMO. WDYT?


This one took the longest time to find because the little fucker was hiding in a default argument, ungreppable :D


Using iterators prohibits us from both looping backwards and removing items mid-iteration. What do you mean by "the complexity of the object"? Also we aren't duplicating lists in this case. We could rewrite that method to use Java 8's ArrayList#removeIf but we aren't using Java 8.


Still need to force large y-coordinates down to a smaller canvas.


in favor OF.


90946736_SHRgt1_4567 (Nigusse) - i386-Windows-7-VisualStudio-12: Tests Failed

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


I think you can resolve this TODO by using a ```Supplier<E>``` instead of a E template.


Duplicate of existing meta sorta... delete the first two lines of meta (the ones you didn't touch)


> 9. Patch version Z (x.y.Z | x > 0) MUST be incremented if only backwards compatible bug fixes are introduced. A bug fix is defined as an internal change that fixes incorrect behavior.

An enhancement is NOT a bug fix...


No, not anymore.


@glasser what is the change exactly? I don't understand how is it going to work?


This design seems convoluted. It means the log is output only when an instance of LogMessage goes out of scope and the destructor gets called. That may be much later than intended.


This seems like an odd place to put this - maybe in the inputs section?


Check ```start``` and ```next``` for nulls using ```Utils.nonNull```.


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

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


@waneck yes let's add actual spaces too :)


Fixed.


TeamCity NEventStore :: Raven :: Build [Build 35](http://build.prxm.it:8811/viewLog.html?buildId=3240&buildTypeId=NEventStore_Raven_Build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:00:57


```Utils.nonNull(v)```


This has been changed by #308.


```
Utils.nonNull(v)
```


Use ```String.format```


Considering these files:

a.php
```php
<?php

echo "Hello from a.php\n";

$filename = 'a.php';

return "returned from a\n";
```

b.php
```php
<?php

echo "Hello from b.php\n";

$str = include "a.php";

echo $str;
echo $filename;

```

It looks like executing `php b.php` prints:
```
Hello from b.php
Hello from a.php
returned from a
a.php
```

In this situation, since `parse_analyze.inc.php` is never executed directly and only included, it won't stop the execution and will return that value.

I found it more elegant to return the value than simply including the file, because it behaves like a function and you'd expect it to return something. This way, you also have more control over it (the name).


Hello, I was wondering why this method has been removed instead of deprecated?


thanks!  I'll try it out later today.


AddCalcCoilSurfTempSource (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1677 of 1677 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 just tried to do what you suggested, and indeed `woff2_decompress` then manages to decode the unsorted WOFF2 font. However, when I load the resulting ttf in fontTools with checkChecksums=1, I get this:
```
>>> font = TTFont('/Users/cosimolupo/Documents/Github/roboto/out/RobotoTTF/Roboto-Regular.ttf', checkChecksums=1)
>>> [font[tag] for tag in font.keys()]
bad checksum for 'head' table
bad checksum for 'hhea' table
bad checksum for 'maxp' table
bad checksum for 'OS/2' table
bad checksum for 'hmtx' table
bad checksum for 'post' table
bad checksum for 'cmap' table
bad checksum for 'loca' table
bad checksum for 'glyf' table
bad checksum for 'name' table
bad checksum for 'gasp' table
bad checksum for 'GDEF' table
bad checksum for 'GPOS' table
bad checksum for 'GSUB' table
```


Fixed. This is a callback that is being overridden. The original appearance is in `Fragment`.


I am just wondering, if this commit could not be formally released. Looking at the commit history I do not see any other relevant changes, which would cause problems. This would help to get the pep8 vs. flake8 incompatibility resolved. Thanks for considering this.


Hmmh. This does actually lead to a problem however. If expected target is `String`, then coercion should not occur from Empty String into `null` as that is quite unexpected, and Jackson does not have a way to prevent this.
But at streaming parser level there is typically no way to know whether expectation is to get a `null` token, or Empty String.
Conversely, however, empty String is easy to coerce at databind layer.

So what I think I will do is to actually default to `null`, which means "no coercion from anything to Java null"; but allow specifying of empty String, either as default for schema, or on per column basis.

For generation both values should still produce empty String as expected; to output String "null", explicit null value must be spelled out.


Note: this will be refactored to GetClusterMetrics(metric_name, start) ([]store.TimePoint, time.Time, error)


I create a list of all possible values of how we score the captures now (victim - 200*relRank). I then sort the values from highest to lowest....I see what the differences are between the values in ordered.  I figured too high of a value may mess things up too much.  So I settled on using smaller numbers...I didn't make a linear relationship as the far files are far more under protected than the middle files.


Same.


Same, and remove comment.


I kind of liked the old, 2d logo better...it felt a lot smoother. It seems that most logos nowadays are oriented towards lowercase and non-3d items.


File mode got changed in no small thanks to Fugitive. This was a mistake and will not be committed again.


This one might actually be my bad since I made the initial skeleton for this task, but I think we should use arBaseTask vs. sfBaseTask.


lgtm


We also might need to call parent::execute at the top so it does all the sfContext jazz


@formanek I will do some minor tweaking to support conditional statements. Most of it is done.
I was thinking about creating a Wiki page to explain the design behind this implementation. High level design as well as details on how to reuse it.

Reuse is definitely an objective (XSS, hardcode values, etc.). I will keep you inform.


```
Utils.nonNull(v)
```


The comment doesn't help anyone. It either does or does not set the version_id.


'sideIThink' lol


"if not multipart" or "if multipart" and switch the branches.


oh ok, how?


We need to do checks/scaling/modulo on the x and y coordinates to keep them within the borders of the screen.


You're right, you're not duplicating. Still reading. Also, when I said looping backwards I was referring to my code.

I do like that you do the removal based on the return value in the goTo function.


In fact this could be a conditional expression.

https://docs.python.org/2.5/whatsnew/pep-308.html


It will be interesting to what and if a particular value set work


reack


:ballot_box_with_check: Assertion added


would it be possible to simply remove the
 tags and keep the text part between the tag
likehere!
 intact? Might look better to add a newline after each thing of text too, but not sure. Dan didn't think it's a good idea adding
 tags back in.


Yes, that is absolutely the correct thing to do — forget the state, so if the same placeholder is re-created in the future, it starts from a clean slate (new initial state value).

At some point, it *might* be nice if components could have constructors / destructors, so they can do things like AJAX calls on initialization, or saving data to local storage or the server on destruction.

```purescript
newtype Component s f g p = Component
  {
    initialize :: Maybe (f Unit),
    render :: State s (Html p (f Unit)),
    finalize :: Maybe (f Unit),
    query :: forall i. f i -> StateT s g i }
```

But this doesn't work since `render` is not powerful enough to call the children query processors with the initialize / finalize commands.

One could allow rendering to generate inputs to the query processor:

```purescript
newtype Component s f g p = Component
  {
    initialize :: Maybe (f Unit),
    render :: StateT s f (Html p (f Unit)),
    finalize :: Maybe (f Unit),
    query :: forall i. f i -> StateT s g i }
```

Then an `install` combinator can execute the initialize / finalize commands of children, in response to when they are installed / uninstalled, and `runUI` can run the top-level initialize / finalizer before and "after" the app. Any effects are forced into the query processor.

This would also provide a way to downgrade `StateT` in `render`:

```purescript
newtype Component s f g p = Component
  {
    initialize :: Maybe (f Unit),
    render :: ReaderT s f (HTML p (f Unit)),
    finalize :: Maybe (f Unit),
    query :: forall i. f i -> StateT s g i }
```

`install` would add its own query algebra to the parent, which would allow it to update the children state map, as well as call children query processors in response to their installation / uninstallation.

Or one can push all of this into `HTML` initializers / finalizers, which has always been a bit of a hack but is used widely.


Do you have any suggestions?


Right, I will wait to see how your test goes...than we can try my version.


This needs a changelog entry as well.


@McAviti: Is the ordered list starting with format loweralpha OK for you?

Regards,
Roland


Solo cambie el nombre, no cambie nada del codigo


I think you can rewrite this loops more concisely with streams.


By the way, what is the intention behind `??` here? Is that actually a typo? I assume so..


I would call this Text


Yeah, in this specific case I found it odd that rootElement is service but body not. I just thought there might be a reason for it.


Why are you doing this? What's wrong with context?


I'd call this just macro


@nasthagiri I addressed all your comments, ready for another pass


`get` is, in a sense, the right prefix. You're getting an ID string from a name string, or a variety of other types of data, kind of like how `getTemplate` gets a template from a species string or a variety of other types of data.


Awesome — thanks

2015-06-28 10:37 GMT-07:00 yburyug <notifications@github.com>:

> @tekknolagi <https://github.com/tekknolagi> check out the gitter room and
> feel free to drop some thoughts :) https://gitter.im/ybur-yug/gkv
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ybur-yug/gkv/commit/edf9fffa55054cbae3942203cb6b62bface0deb1#commitcomment-11900147>
> .
>


P.S. There's no such thing as an nREPL project - it's a lein or boot project. Unfortunately, there's plenty of work on be done to fully decouple CIDER from nREPL internals...


I'd call this MessageComponent (hint: MessageCompositor)


No m'havia fixat que aquests tambe feien falta :x


Ah, good point. The invalid argument is really `buffer`, which happens to have a sub-property with a bad value.

Logged https://github.com/aspnet/WebSockets/issues/44

Nice catch!


Since github doesn't show the diff, and I'm just curious, what did this commit do?


Frenzy is my middle name


@taisel, that works in Firefox but in Chromium and Chrome, this causes a cross origin request error. This works in all browsers.


I'd call this Argument and would split it into Flag (only name) and Parameter (name + value)


@jef-n Not sure how this escaped our testing, but this is a nasty one. Any chance of stopping 2.10 and including this in a 2.10.1 release? There's more discussion over at https://github.com/qgis/QGIS/commit/72c9830178e3903b77a36d30f3e69a5e0ca6533e


I'M CLAMMIN UP


@paulirish Ah, I missed that during code review. This has to be `innerHTML` because descriptions in the `protocol.json` contain html tags. After your change, descriptions look like this:

![html](https://i.imgur.com/LcT6tUv.png)


Immutable pleeeaase :)


Não entendi. Qual o problema?


LinkedList -> List and make final without initialization


CLAMS


take List instead of the concrete type


unmodifiable


@jhass @kirbyfan64 This is the 2D logo, that also serves for promotional material https://www.dropbox.com/s/r706edlgrxcl9od/Screenshot%202015-06-29%2019.21.53.png?dl=0


Does this affects also Linux? I use master on a regularly basis on Linux and I haven't seen this bug in the past days. Just noticed today on Windows/2.10.


I forgot to mention, this would also help to solve the following Macport issue: https://trac.macports.org/ticket/47429, without further fuzz ...


I think you missed some numeration here. :) I will merge this and fix.


See http://docs.travis-ci.com/user/ci-environment/#Virtualization-environments


Also I split it as follows:

```{}``` --> DefaultMacro
```{0}``` --> IndexedDefaultMacro(index)
```{name}``` --> NamedMacro(name [, args])
```{0:name}``` --> IndexedNamedMacro(index, name [, args])


LinkedList --> List


Indeed you can just use `*.zip` if you don't mind it also omits other .zip files in the repo.


this is broken on case sensitive file systems.


"Build Status"? :stuck_out_tongue:


No. I think it's supposed to be STUDENT_LEVELS. I can't find any instance of us using this either so feel free to rename or get rid of it.


Seems like the best approach!


TeamCity xBIM :: XbimScripting :: Build [Build 21](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1974&buildTypeId=XBIM_XbimScripting_Build) is now running


Thanks! Given enough pedants... ;)


Ugh, brain isn't working today.


Yes that's it. However, since only code for the false-path..


test


Sorry if i've mistaken it. Do you ever tried to cache those already patched files?


The commit pending can be found at https://gerrit.omnirom.org/#/c/13349/ but as humberos has switched focus from exynos3 devices it probably will never be merged.  All that it does is prevent checking for vfat if a certain boardconfig setting is defined


You can condense `this.getOption('filter') && this.getOption('filter') != ''` into just `this.getOption('filter')`, because the two checks are actually equivalent (the first implies the second).


TeamCity xBIM :: XbimScripting :: Build [Build 21](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1974&buildTypeId=XBIM_XbimScripting_Build) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:33


hpwh_residential (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (647 of 647 tests passed)

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


@voter101 Markdown supports "lazy list numbering" such that

```
1. foo
1. baz
1. bar
```

Renders as

1. foo
1. baz
1. bar

But either way will work :) Thanks for the quick response.


I see, that probably why I did it in OTS. All good then.


r+


I believe you mess up the logic here.  You need to reference rank....not the square.


This is PDF version(in Russian)


"Документы" means "the documents". I think it would be safe to leave it as "Документов".
Cannot change it in Transifex as I'm not in the Russian translation group, but the sequence is: 1 Документ, 2-4 Документa, 5..N Документов


Nice. Using unit-test style classes isn't necessary with py.test, you can just write one-liner functions at the module level with the test_ prefix, but it's compatible. Using a setup function is useful when you're going to write a lot of tests with similar preconditions.

Good job making sure to add zero -- check for any divide-by-zero.

The test you've put in place isn't terribly specific ( < m). That would hold true for a lot of operations, such as a buggy one that returned all zeros. It might also be worth checking a small, known example array against known return results.

Looking at the use of 1e-7 -- does that mean this is geared to a 32-bit int. Will it fall down on a 64-bit value?

Here's a broader thought -- theano from memory uses floatX, which can be either 64-bits or 32-bits. I am not sure what that means for testing.

My philosophy on code reviews is: the original author should take what's useful, ignore what's not. So you don't need to make changes according to every single thing in the comment, just reflect on them and make what changes you think are relevant. Let me know when you've done that and I'll take another look.

Feel free to let me know if you'd like me to do anything to chip in also.

This is a great start, it shouldn't be hard to add the rest of the functions.


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/07d8c7ecab91826d0a88bc8ff81949cb218204b0


merging graydon/stellar-core/auto-rate-load-gen = 07d8c7ec into auto


graydon/stellar-core/auto-rate-load-gen = 07d8c7ec merged ok, testing candidate = 57411560


@asterite That's the thing; the old transparent diamond looked nicer and cleaner, IMO. This one looks really strong, especially with the bold text and caps...


I don't think this was wrong ;) It's in hex, 1, 2, 4, 8, 10 (dec) == 1, 2, 4, 8, 16 (hex)


Hmm, okay... <strike>But can we make it ``getID``</strike>?


As long as no one starts asking me how the package works :-)


Style nit: put each nested loop on its own line (e.g. one `<-` per line, since that's easier to read).


what about the butts tho


Technically I suppose it's possible that the randomly-initialised example array may contain only values which are positive values in which case this would fail. It might be better to test on known values for the example array. It's a remote likelihood, but one day it'll happen to someone somewhere and confuse the heck out of them.


AFAIK we only really need to test the `codegen == true` and `unsafe == true` case, right?


Note, the identity function here just returns what was passed in. If you think it's worth documenting that, you could run some tests on values of 5, "hello" and None, just to make things stand out.


Also, I think `Seq(Seq(colB), Seq.empty)` is a bit more readable.


This was put in to truncate an extremely long channel name I believe. We need to find a way to keep the long channel name from out-growing its container while ensuring that everything else doesn't get chopped off prematurely. It might require trade-offs.


Perhaps we should just inline the implementation here?


I can't fix this until tonight...


Para excluir el nodo inicial también se puede hacer así:
 .evaluator( Evaluators.excludeStartPosition())


Do you want me to submit this for you?


Thanks. Fixed in eb224909.


Why would the dock sequence care about `WithHarvestAnimation` ?


Hi @peteles -- sorry about that, this function got refactored into the ProductGrid.php component as it makes more sense for it to live there than bundled with the general Controller.  We don't have a deprecation strategy with regards to internal functions at the moment, but I'll bring it up during our next retrospective.   Let us know if ProductGrid::createBookends() doesn't work for you.


That could be done but in future if we want to use some hadoop token facility to generate the token (like hadoop delegation token manager) we would have to rework it (like using a plugin adapter) to avoid dependency to hadoop for buffer server. I guess the token verification will also get affected in a similar way in future. Let me know if you still want to pursue this.


Better, but you have some extra logic you don't need.  Compare what you have with the following and see if it makes sense.

  // add pickLetter to all buttons on alphabet keypad
  $(".letter-button").on("click", pickLetter);

  // pick a letter from alphabet keypad
  function pickLetter() {
      var currentLetter = $(this);

      currentLetter.removeClass("letter-button").addClass("letter-disabled");

      //Next two lines to be refactored to the interface for checking if letters are in the word.

      currentLetter = currentLetter.html();

      verify(currentLetter);
  }

  // currently a placeholder for verifying the letter
  function verify(letter) {
      console.log(letter);
  }


Hey @alepolidori! It would be awesome to add a couple more notification types to this map, I know "mentioned" is `1` and has the icon `fa-at`. I'd submit a PR myself, but I'm getting ready to go out of tonight tonight and don't have time to get it done :smile: Here are a list of the original templates:
https://github.com/discourse/discourse/blob/837c0e267a8e6f9b2e3f4cd2d60f90324594ee6e/config/locales/client.en.yml#L833-L845


Same question. Also if it's a must, you can cache the trait. But in any case, this looks too suspicious.


In theory, we could, but it wouldn't match our capitalization convention.

And, of course, I'm open to better suggestions, if you have any.


I remember there was an edge case here which required us to check whether there was a model but I am not sure if it's valid anymore.


I'd suggest unpacking this into some stages, with comments, e.g.
squared = normed.eval()**2
sumsquares = np.sum(squared, axes=1)
reduced = sumsquares - 1
sqrted = np.sqrt(reduced)
maxabs = np.max(np.abs(sqrted)
self.assertAlmostEqual(maxabs, 0)

For the mathematically semi-literate like me, showing your working can be invaluable. I basically had to do that in my head to undertstand what is going on, I think it makes it more readable for some.


Are you sure this can still be overwritten in configuration ?


Yes I saw that fix also, but that is still working (still truncated). This is the item dropdown menu and as far I can see has nothing to do with the long channel name. BTW: The channel with the long name is on my hub, that's why I 'fixed' the admin pages in my previous commit.


You forgot to update the backends in docker compose file


The documentation requires a lot of changes for this.

Please next time, don't do mix release pull requests with big feature change like this one :)


`_updateComponentIfNeeded` is not re-entrant


Hey, fantastic! :) Hope you don't mind the comments ... they're just my 2c, don't take it too seriously :). Looks like things are well on track, we can do a merge request once you've had a chance to iterate on the coments. I'll pull in fchollet's latest changes as well.


My bad, I completely missed the 0x in-front of it


Issue solved had to do:
Application.LoadLevel (1);
Scenes start from 0.


[![Scholar's Mountain Retreat Amongst the Pines](http://api.artsmia.org/images/30978/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=30978)


[![Still life with grapes, apples, chestnuts, almonds, acorns and a red clay bottle](http://api.artsmia.org/images/121296/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=121296)


I don't remember...look it up :D But i know for sure its not passing null ;) is it in the javadoc?


Don't forget the code self-documentation... `@param` and `@return`. I'm not very familiar with the xmlps codebase, so check a few other classes for consistency.


:smiley:


That is actually the setup I play with except that I use VS 2015 community edition. Blut that shouldn't make the difference...


@alexstrel, I don't understand the changes you've made to BiCGstab.  The check for zero-rhs isn't optional, which will break pre-existing behaviour.  I think this should be optional.  Also, I don't understand why there are changes to some of the mixed-precision checks, e.g., with respect to heavy-quarks.


Is it possible to cache this computation for later use?


We could return "$start - ?" here.


What is this change!?!


The try/except was removed here, meaning that execution errors reach the Qt event loop without getting logged! (they just show up in the terminal)


Both string dates are removed from the mapping.


lets declare unsgined int instead of Count here


unsigned int


`tryAsynchronousUpdate` and `rootIdentifier` are not used here. Are you going to add this functionality in a separate diff?


Should the namespace stop being Serilog?


Removing the lifecycleManager will affect debugging too, as the debug helper tries to find the attached lifecycleManager.


You can generate an ssh-key and copy the public key to
/var/run/.ssh/authorized_keys in the container.

Something like:

RUN cat /dev/zero | ssh-keygen -q -N ""
ADD ~/.ssh/id_rsa.pub /var/root/.ssh/authorized_keys
RUN chmod 500 /var/root/.ssh/authorized_keys

The first run creates the .ssh directory with correct permissions (if you
mess this up, ssh will not use the authorzied_key file).

On Mon, Jun 29, 2015 at 4:15 PM, someinternetguy <notifications@github.com>
wrote:

> Hi, Thanks for all your work. I've forked the project and made a few
> changes that I need, including using a docker volume image for data and
> using the path /pocketmine instead of /mnt/pocketmine. Anyhow, one thing I
> would like to do is have *a way to set the supervisor password when the
> docker image is run*. Having it in the supervisord.conf file like this,
> works, but its insecure becasue anyone who looks at my repo on github, now
> has the password to mess with my server. Since I am running servers on a
> public IP, I would like more security than that. I am not proficient enough
> with programming to figure this out. Do you have any suggestions?
> *thanks!*
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mecolosimo/pocketmine/commit/84132cbfb4a42a4d451b3c360b92830f21f86bb6#commitcomment-11918328>
> .
>


rrr


The URL for this version was actually different from the others, as it used a `.zip` extension rather than `.dmg`.


You can push what you want, I have no more changes staged at this moment.
Just some very bad news from my phone: my i9300 power button did now really die, and prevents from booting. So I am completely grounded in doing any testing at the moment :-/


Nice catch!


You're welcome to that would be great


For heavy-quark, my fault. Yes it's not optional I thought about adding something like QUDA_COMPUTE_NULL_VECTOR extra argument in the invert parameter to trigger this case. Do you agree?


Meh. ``getId`` it is.

This ties with the ``Tools#Data`` refactor


@pfalcon Ok, glad you are not afraid of the dark!


Yes, sounds good.


add todo to implement our own random generator


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/68891122


The ```BOOST_USE_WINAPI_VERSION >= BOOST_WINAPI_VERSION_WIN6``` check is unnecessary, because ```BOOST_PLAT_WINDOWS_RUNTIME``` means we are compiling a windows (phone) store app which means in turn that the WinSDK version is at least WIN8.


A bit complex to use :)


Too bad we don't have something like "instance aliases"

```
instance QueryDSL g a = (Functor g, Inject (Coyoneda a) g)
```

to allow writing:

```
clickDecrement :: forall g. (QueryDSL g Input) => Free g Unit
```


Good comment :+1:


Why aren't we just including the composer auto loader ?


fast-forwarding master to auto = 57411560


The project being built is a dependency of installing it. Make will not rebuild the whole project if it doesn't half to. If you want to properly use a makefile for this. Maybe something like this is a better option
```
all:
	echo "A target should be specified. install | clean | build "

build:
	go get github.com/tools/godep
	godep go build -o runc .

install: build
	cp runc /usr/local/bin/runc
	rm runc

clean:
	rm runc
```


http://intellimate.github.io/IzouSDK/apidocs/org/intellimate/izou/sdk/frameworks/music/player/template/Player.html
it's stopMusicPlayback()


Change Name


Thanks @JesperTreetop for your feedback,


Changed


Thanks, @asmecher. Few of the other DAO methods seem to use this, but you’re right that they should be there.


thats simple, like how /x-accel-redirect/ is just part of the uri


i think that is fine.


Redid as Tabs. Not sure if it made a difference.


This PR makes chiainer depend on `pkg_resources` module, which is distributed with `setuptools`. I would like to discuss if it is acceptable. In my opinion, it is OK because user is supposed to install china via `pip`, which already depends on `setuptools`.


yup, no worries. i caught that later as well.


You could add a check at the beginning of if logic

 if (!serverSettings.getEnabled()) {
  // log.warn()
  }

and then have the serverSettings.enabled logic here.


Redo your translations. They're incorrect and would cause force closures as there are strings missing.
For example, attention
volume_rocker_screen_title is also used.


Does this work? I've never seen assigning arrays to a property quite like this before. I did a little test script for tools:run and didn't have luck (but perhaps I'm doing it wrong)

```<?php

$io = QubitInformationObject::getById(477);

$io->script = array('Arabic', 'Klingon');
$io->save();```


`psc-bundle`


See commit message: Page::ShortcodeColumns() should be Page::MyShortCodeMethod() sorry.


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

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


impl.: RqN000, RqN001, RqN002, RqN003.


I wonder if you type the changes by hand and don't use the script to get the text automatically?
Because there are a lot of differences compared to my last update using the script.


impl.: RqN005.


CacheExample


impl.: RqN004, RqN006.


se eliminaron dos arreglos de indices y se eliminaron constantes que no se utilizaban


commandline prefixes are gone for now:


Do we need a better message. I don't know how human friendly response.getInfo() message is going to be.


Awesome Good work!


Should be ```SPLITCHWRESSORET_H```


From what I remember the model check was pretty much always there, I'm pretty sure it's OK to remove it now


r+


Noice Wok !


Corrección? Performance?


I'd like to see these functions in the `ripple` namespace instead of `RPC`.


The Community Edition should work as well. The Bison version should be 2.4.1 and VC11 libs cannot be used anymore. They lead to errors like 'unresolved external symbol __imp____iob_func'.
See https://www.apachelounge.com/viewtopic.php?p=30547#30547

Do you get any specific error messages?


Great! Thx


saw approval from graydon
at https://github.com/MonsieurNicolas/stellar-core/commit/7c69fbc2c4b5ea77efbbf7e3ff3cc8c435bad214


merging MonsieurNicolas/stellar-core/scpLogging = 7c69fbc2 into auto


MonsieurNicolas/stellar-core/scpLogging = 7c69fbc2 merged ok, testing candidate = 3a9d7290


hm


Why are we catching null pointer exception ?


SO MANY ******

Good job!


ya te mido la performance, y la correccion se hace comparando el output con el del programa original


That sounds good


checkStoreEnabled perhaps conveys the intent better ?


Ok, podemos tener como base en cada commit, postear corrección y
performance.

2015-06-29 20:47 GMT-03:00 Emanuel Lupi <notifications@github.com>:

> ya te mido la performance, y la correccion se hace comparando el output
> con el del programa original
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/WaVEV/facultad/commit/b106f6edadd09075f5f56a2f81a1605fedc95bbe#commitcomment-11921630>
> .
>


breaks yaml, bad spacing.


wut


84691154_AutomateBuildingOfDocs (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1682 of 1682 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)


@akuefler Right, here is what I was talking about. "Forces" don't make sense to Fovea in general. You could instead call `mouse_event_force` something like `mouse_event_user_function`, wherein the user has selected a particular function (the part you've already taken care of, self.spatial_func) to be called when this mouse event occurs.


TeamCity NEventStore :: Raven :: Build [Build 6.0.0](http://build.prxm.it:8811/viewLog.html?buildId=3242&buildTypeId=NEventStore_Raven_Build&guest=1) outcome was **FAILURE**
Summary: Error running GitVersion Build time: 00:01:06


:scissors:


qualityOfService is only available with iOS 8, might want to wrap this as the support target is still 7?


@cyderize – can you come up with a more informative function name here please?


```Lua
x, y
```
 is slightly misleading. should probably be
```Lua
w, h
```


@keithbowes Have you seen this comment: http://php.net/manual/en/mysqli.close.php#97521 ?

Should we add msqli_kill()?

Or, on the contrary, should we **not** close in hope of being able to reuse the same DB connection in another PHP request?

Have you investigated persistant connections and related connection issues?

Thanks.


Merged changes from the original repo to this fork.


90946736_SHRgt1_4567 (Nigusse) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


does this route do anything for you?  Why is it here?  Don't leave dead code in your submitted files.


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


At the same time you set `$LUAC` set `$LUAROCKS`


84691154_AutomateBuildingOfDocs (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1676 of 1676 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)


re


This is just a style nit-pick, but variable names shouldn't start with a capital letter. (At least that's how I've made the rest of the project).


Again, why the random insertion of spaces after hyphens? (The hyphen usage here is wrong, but that's a separate problem.)


Since the error sent back to the settings page only shows up at the start of the page, this should probably clarify that it is the Custom profile pictures path.


The `*` operator is now a formatting gotcha. See how GitHub renders it now that double backquotes are now all changed to `"`:

 ![screen shot 2015-06-28 at 11 02 36 pm](https://cloud.githubusercontent.com/assets/1732/8400264/c947b55a-1de9-11e5-81db-e171bc9c5c08.png)


@SubJunk yeah, the bug circled twice but then flew away (9e42d58).


Wow these all came out decently


Looks ok.


Inline math annotations are all lost. :-1:


I think I will develop from your fork.

I agree, it doesn't matter if the official / original repo isn't up-to-date, so long as one repo is. However I worry that because everyone is looking to this, "official" repo that they may miss more developed forks and create their own and potentially develop something someone else already has.

> One day they will merge your pull request.  Maybe soon maybe in a very long time. But from my experience if your pull request is good, it will become the "official" 0.9 until it gets merged.

I find this unacceptable for development. If the pull request is good and effectively becomes the "official" then I see no going back. It is unappealing to switch a dependency from one repo to another, on the basis that the maintainer(s) intermittently have an opportunity to merge Pull Requests. There should be some consistency / standards. So if they are unable to maintain this repo, the proper practice is to look for another maintainer and have another repository take over, indefinitely. I would not like to keep track of the internal Issue tracker or multiple versions of a package, such as this, and each week pick the best one which is the further along. We should all work together, with one repository.

---

Essentially I was hoping for a response like "we are not maintaining this repo" or "we recommend using https://github.com/martin-magakian/influxdb-r as it is more maintained". I just want to make sure the community knows the status and direction, where should they look to and contribute to. What is that centralized, one repo, if not the official one.

Here is what I do *not* want to see:
![image](https://cloud.githubusercontent.com/assets/1885333/8400145/27533df8-1df0-11e5-8840-a591d97aec08.png)

Two different forks implementing the `write` API and another fork implementing an update for 0.9 API.
I would like to see everyone contributing to a single repo and we can all progress together.

I do not think my soft, inquisitive tone was expressed accurately over my original, cold textual comment. I understand that InfluxDB team is very busy, and sympathize. However, their users need guidance as to what is InfluxDB's plan or they are going to go somewhere else. I want to know what that one repository is, and I think the answer is yours: https://github.com/martin-magakian/influxdb-r

Thank you for answering my question :smile: .  I'll look at https://github.com/martin-magakian/influxdb-r for my prefered R package for InfluxDB :+1:.


Thanks. Should be fixed now: ade5d10


need to add the step to download app from demo-express github


a


Another doublequote nesting snafu here. :-1:


Oh my.  Does that mean we actually have 2 'pi', one in HReal and another in HProb?  Doesn't that seem awfully silly?


Currently there's `Arrays.copyOfRange` on the return for each hex method to extract the result, so I guess the override issue would not be an issue since what the UTF8String holding is a copy. (It might be bad if facing multithread clients use the same Hex object to eval hex). @davies what's your preference? give a separate buffer for each hex() instead of shared field?


asl197 >.>


I think creating an unitAdminTypeAndPersonGlobalDetail object servers no purpose beyond this method? Wonder if you just tried setting the fieldValues to the principalId....?


Yes. No sillier than having 2 'exp'.


The conditions in this if can be condensed further, not sure what's the purpose of adding the third condition?


Great catches! Thanks for the PR!


This is a "getter with a catch"... wonder if we can refactor the code a little to keep it simpler?


thie si


EFFECT_NO_EFFECT_DAMAGE is confusing name. it will not change effect damage to 0.


a new comment


No hay problema que este comentado esto?


Any progress on this commit? CentOS 7+ need this emergency, we even can not use `pm2 [action]`.


You should respond with no results here!


this is a lib


This game did not work in Repl.it.


Is there a way for users to detect this change to handle old and new versions in a non-breaking way? I'd rather not have to try/catch all use cases of this function if possible.


@mikepetersonccv , Shouldn't this still by "Take( 5 )"


no


jghgj


Accidentally left a debug statement in?


Copié el código tal cual de como te lo dan en la página. Así es


@mnishihan Bhai,
It was done accidentally. Sorry for that. Hope you fix it.


Looks OK.


This is just an example of course.  You would want error handling around this.


# karapace-web
web app underconstruction


:s/an instance/a slave/
:s/the instance/the slave/
[just to be consistent in naming across the plugin]


Thanks, that is true. It seems a common performance bottleneck and whether the sequence/array will be held with the AlignedSegment or lives as a separate python object should not make too much of a difference.
Maybe it is worth an experiment?
The alternative is to remove the qualities/sequence properties altogether to enforce users to manage their own copies, but that would break a lot of code.


We don't use this file at all. You don't have to add this here.


I'd say log message. If they're testing on a build that's already logged
in, doesn't hurt to just let them continue. Possible really nice case would
be to just allow for setting an auth token directly in BuildConfig so they
don't have to make an application on their account. What do you think?

On Sun, Jun 28, 2015, 9:56 AM Henrik Olsson <notifications@github.com>
wrote:

> So should we just give the user a gradle log message or throw an exception
> and stop the build?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Meisolsson/android/commit/25ce35719633430287046bbf5aa890ab97b5a772#commitcomment-11900006>
> .
>


@calcinai Nope, the project. XeroOauth-PHP makes me feel physical pain. I couldn't think of a better place to publically thank you. I'm sure each of the current 15 stars was thinking the same thing. You should expect others as soon, as they find this repo.


Simple and nice.


close [test](http://f7140a41.ngrok.io/w/JagJ_UnqaUj0lllyp3DCtYvS): xxxxx


@deepakaravindr, this commit breaks the build for me. Line 166 passes `JoinableTaskCreationOptions.LongRunning` as a second argument to `JoinableTaskFactory.Run` method but the assembly Microsoft.VisualStudio.Threading.dll that is in my SDK has neither an overload for `Run` with 2 arguments nor any definition for `JoinableTaskCreationOptions`. Any comments?


避免因为OSC和Boost同时发送数据导致的线程冲突。


Yes, I prefer to give a separate buffer for each hex(), which will be used to create UTF8String without coping.


Should update test word to demo.
This sample demonstrates CSP feature basic functionalities:
This sample sets following csp config in manifest.json
"csp": "default-src 'unsafe-inline';connect-src http://w3school.com.cn;style-src 'unsafe-inline';img-src 'self';media-src http://w3school.com.cn;frame-src http://w3school.com.cn"

Demostrate "connect-src" property,
Get "http://w3school.com.cn" content, access permitted
Get "http://www.google.com"content, access denied


这个逻辑应该放在show才比较合理嘛


When would you ever need more than 128 classes? Or need a list of 128 processes? it's called Reclass not ReClasses .


Commit message edition: `article entity form translation support related #742`


I believe you have included field="idleTerminationMinutes" here just to load the help file. Instead, you can point the help file by using help="/plugin/ARTIFACT_ID_GOES_HERE/help-projectConfig.html"


You won't need this if you update config.jelly with help=(help file location)


Hi,i want know how to bulid openwebrtc server?


Hi Ford,i'm learning openwebrtc native ios app,but the client connect to each other cast long time(about 10s),however webclient cast bellow 3s,could U give me a help?


Method has too many lines. [14/10]


Final newline missing.


Line is too long. [124/120]


The member variables are redundant. The MoveIt! parameters should be accessed by the Descartes_MoveIt adapter (?)


Hiiiii


could consider editor config (http://editorconfig.org/)


@miguelgrinberg I don't know how you fell about this (since the `room` is actually a set). I though most people would actually be converting it to a list, so I did them a favor.


Not sure I need contributor credit for proofreading a little bit of LaTeX :-)


> Zabbixのトリガーオブジェクトにはtrigger["hosts"]というのは無いので、エラーになっている

これは expanded_descriptions の方を見ていました。
すみません。

triggersの方には2.0でもちゃんとhostsが付いていますが、問題が発生しているトリガーでは
`hosts:[]` と何も入っていないのに`trigger["hosts"][0]["hostid"]` と要素にアクセスしようと
しているのでエラーが発生しているようです。

なぜhostsが空のtriggerが紛れ込んでいるのはまだ分かっていません。


line note on the right!


@svaksha wow! Thanks for adding this.


Doesn't get the reason for error


> なぜhostsが空のtriggerが紛れ込んでいるのはまだ分かっていません。

このトリガーはテンプレートですね。


line note on the left!


YW :)


Thanks you for the warm warning about that change. I would have appreciate to know it before. Like in a branch.

What the the **rationals** about that change?


debugging leftover


`/expired./`の`.`は，どういう意図でしょうか．
ピリオド一文字にマッチさせようとしている？ それとも，"expired"の後に任意の一文字が必要？


@andre-galdino It is not a good idea to reference a local path inside the code.
There is a test breaking because of this.


how about performance? have you give it a test?


fwiw, lgtm - excited about having this + auth!


Revert changes you made

https://developer.mozilla.org/en-US/Apps/Build/Manifest

Tips:
If your app is stored in the root of a Web server, for example mywebapp.github.com/, then launch_path must be set to /.

If your app is stored in a subdirectory, for example mymarket.github.com/mywebapp/, then launch_path must be set to /mywebapp/.

The same applies with the images


In my working branch, I've collapsed things down to just having one pi which lives in HProb; the pi that lives in HReal must be got by explicitly coercing the HProb pi. (Similarly for exp; coercing the output as need be.) Seems the simplest approach overall; but it will lead to weeding out these sorts of discrepancies whenever we switch over to using the concrete AST.


@Glavin001 I'm 100% agree with all you're saying. I would be nice if the InfluxDB team could give direction. I also agree with you, for the long run an official repo is more appealing. Anyway good luck for the dev!


looking better, please see my comment about upload the patch (i.e.: these two commits plus whatever else needs to be added) to JIRA so that CI (continuous integration) tests can run. thanks  @sugartxy !


From the man page:

       PermissionsStartOnly=
           Takes a boolean argument. If true, the permission-related execution options, as
           configured with User= and similar options (see systemd.exec(5) for more information),
           are only applied to the process started with ExecStart=, and not to the various other
           ExecStartPre=, ExecStartPost=, ExecReload=, ExecStop=, and ExecStopPost= commands. If
           false, the setting is applied to all configured commands the same way. Defaults to
           false.

This means that for `ExecStart` only, `User` is used. The other commands are run as `root`. Also, I ran it with this `.service` file, and it worked.

That being said, though, using the tmpfiles config seems like a better option.


Might be caused by O_RDONLY being 0 on most systems (so `open("path", 0)` opens for reading). I agree that not calling any of read(true) or write(true) is weird, and yet it's the example for OpenOptions::new(): `let file = OpenOptions::new().open("foo.txt");`!

The doc for append() and truncate() was fixed by #26642


This extra # should probably be removed.


So is the intent here to allow for uploading of emoji? Does this solution support unicode-based emoji for a wider set of possibility?


ref #5


I wonder if we should export SHARNESS_TEST_SRCDIR here as other variables that we define are exported?


Mick, you nay want to set up your local fdrv branch properly to track remotes/origin/fdrv so you do not need to merge it to itself ;)


Acepto tu commit pinche Alex


I learn something new everytime. For some reason that `.service` file didn't work too well on Arch. I do think the tmpfiles way would be more canonical though, as that's how systemd is supposed to build the `/run` directory. Thanks for the pointer to another way of solving the problem though.


Moved the createMenuOptions code from this file to the AppTabGroup.js file. The "Add" menu option code for Android needs to be attached to the Todo tab.


Git commit comment sent in French because it applies only to the Belgian market.


推荐用这个：
https://docs.python.org/2/library/glob.html


It is more pythonic to write `for value in values`, - from pylint C0103


I am fine to this fix.
Thanks.
--Wei


Good catch, thank you. All your points sound good to me.


@andresn - any chance you could help translate some of these?

Translation: Missing AIR_CONDITION for common in es-CO
Translation: Missing ARE for common in es-CO
Translation: Missing HEAT for common in es-CO
Translation: Missing HVAC_OFF for common in es-CO
Translation: Missing HVAC_ON for common in es-CO
Translation: Missing IS for common in es-CO
Translation: Missing NOBODY_HOME for smartthings in es-CO
Translation: Missing SOMEBODY_HOME for smartthings in es-CO
Translation: Missing SUNSET for smartthings in es-CO
Translation: Missing SUNRISE for smartthings in es-CO


#2 solved.


テンプレートトリガーが必ずしも混入するとは限りませんが，
要素が無いことによって止まるのはよろしくないですね。
try, exceptionでIndexErrorの場合はpassするだけでよいでしょうか。
for文の中にtryを仕掛けておけば，その問題のテンプレートトリガーだけスルーされるだけですし


you're trying to make excuses :D


PR up: https://github.com/aodn/aodn-portal/pull/1814


BUG: should be
```
best_est=j;
```


BUG: Testing this code on ip3model results in all chromosomes having equal fitness


Did you commit all your changes before the pull request?


This should be
```upload.maximum_size=1044464```
because we can only use lower 1M part for code at the moment.


仅仅是为了使用一下github


Just wanted to say thanks and confirm that this works with kml data I am using.


update https://github.com/fireball-packages/console/issues/2


It is required for correct function call at runtime.

The difference can be checked with a simple example. http://try.haxe.org/#FBA7F
```
import haxe.Int64;
using haxe.Int64;

class Test {
    static function main() {
        trace(test("aaa"));
        trace(test(1));
        trace(test(Int64.make(0x00, 0xff)));
    }

    static function test(value:Dynamic){
        if (Int64.is(value)) {
            //return Int64.toStr(cast(value, Int64));
            return value.toStr();
        } else {
            return Std.string(value);
        }
    }
}
```

## In case "value.toStr()"
"value" interpreted as a Dynamic and compiler generates the following JavaScript code:

```javascript
Test.test = function(value) {
	if(js_Boot.__instanceof(value,haxe__$Int64__$_$_$Int64)) return value.toStr(); else return Std.string(value);
};
```

This code will compile but will not work properly, because "Int64" instances does not have method "toStr" (it is static function).
You will have error at runtime:
```
11:45:57:408   aaa
11:45:57:408   1
11:45:57:409   Uncaught TypeError: value.toStr is not a function,http://try.haxe.org/program/FBA7F/run?r=0.18905784701928496,line:38
```

## In case "Int64.toStr(cast(value, Int64))"
Code contains an explicit "Int64.toStr" function call but it is require type cast.

```javascript
Test.test = function(value) {
	if(js_Boot.__instanceof(value,haxe__$Int64__$_$_$Int64)) {
		var tmp;
		var x = js_Boot.__cast(value , haxe__$Int64__$_$_$Int64);
		tmp = haxe__$Int64_Int64_$Impl_$.toString(x);
		return tmp;
	} else return Std.string(value);
};
```
Correct out:
```
11:49:14:141   aaa
11:49:14:142   1
11:49:14:143   255
```


Any changes since this one?


y SOAP?


Hi,

my build statuses "MiroI build statuses:" are no longer valid. Now your own travis-ci/appveyor badges of this branch should be put to README.md.


YES! YES! YES! :+1:


i think better way upgrade yii2-composer package and do installer on yii2-extension packages.


I can't seem to find a reference that specifies that, do you have a link so I can learn more?

Python documentation seems to use my structure: https://docs.python.org/2/tutorial/controlflow.html#for-statements


Should they reference the file or is that not needed?


最后译文要按照原文格式，例如你行首多了一行空行。另外用尽量少的commit，没必要删除source一个提交，移动到translated又一个提交。


What is this method supposed to be doing?
Right now the name is confusing and it just returns an empty array.


Does this compile? I don't see where you actually added the items method.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (351 of 351 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)


ユーザ一覧ページはpaginationしているので，`@non_admin`ユーザが2ページ目に載っているとき，このテストを通過してしまうのではないでしょうか．


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (351 of 351 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)


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-MacOS-10.9-clang: OK (1665 of 1668 tests passed)

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


I'm NewBie


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1671 of 1674 tests passed)

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


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (351 of 351 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 wish we knew this in Kitkat hahaha would of came in handy :-)


@MingcongBai To make the value ‘clean’, um…


I am fine to this fix.
Thanks
--Wei


Oh that's a good tip, thank you @leomindez


@Arthur2e5 simply excessive move...


You must use - it's incorrect, better: You can use (if you wish)
Forum name having the id number 1 - for category the same
better:
+// 	'FORUM_NAME_1'	=> 'Forum/Category name having the id number 1',
+// 	'FORUM_DESC_1'	=> 'Description forum/category having the id number 1',


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (351 of 351 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)


84691154_AutomateBuildingOfDocs (lefticus) - 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)


This is not a valid fix for a production server.

We are actually supposed to use the JMS, as the server needs to eventually create JMX objects for JMS queues.

Just fix your tests to have the JMS Manager instead of removing it from the production case.


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


@marcoRS  :O i didn't know it, is that a new method? I mean, i hadn't seen it in previous versions and recently butterknife was updated to 7.0 version.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (351 of 351 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)


Chỉ commit source code, những file không cần thiết thì ignore đi


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


por ahí cerca de NetBeans y Visual Studio, yéndose por la calzada de la soluciones de unos cuantos clicks, algo retirado del punto en el que estamos


Typo.


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (351 of 351 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)


if/else lồng nhau nhiều quá, rất khó đọc, em tìm cách giảm số if/else lồng nhau đi


It's been there for a while. It only applies to fragments because of their different life cycle.


@albertosouza 9af2516


Tìm cách tự implement toàn bộ, không dùng đến các class có sẵn.


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (351 of 351 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)


@leomindez Totally, only was a quick solution.


严重同意, EventBus 和 Broadcast 本质上是一致的,都是使得静态分析失效,难以跟踪和调试


personally i'm not a fan of this change at all and would like a reversion. setting aside the fact that push variables would need to be adjusted for this it is jolting to the player every time they are hit and for rapid-shot weapons like plasma, smg, zapper, flamer this is extremely annoying because it messes up aim and creates an effect like a longer-lasting stun.

however, conversely, this is not a bad idea:
http://redeclipse.net/forum/viewtopic.php?f=4&t=742


I'd love it if we could extend this a little more to include other metadata. Manpages usually have an author field and a copyright clause stating the license. Both should only be generated if such information is available/nonempty.


Going to force the client/user to have a `versionCode` and `versionName` in the Manifest is not a good idea. These values can and will be injected from the gradle build system, which also allows for dynamic numbering and naming.


Never add private data to public repository :)


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-AddressSanitizer: OK (351 of 351 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)


here is a simple repo script:

case class DecimalData(a: BigDecimal, b: BigDecimal);
val decimalData = sc.parallelize( DecimalData(1, 1) :: DecimalData(1, 2) :: DecimalData(9,1) :: Nil).toDF()
decimalData.agg(avg('a)).collect().foreach(println)


so if this is going to have its rightful part in the coalib we do want tests for it.


Internet spelling


This looks like it's made recursive. It's not abundantly clear to me why it's paniccing instead of just blowing up it's stack, but:

```
stack backtrace:
   1:        0x10e23886c - sys::backtrace::write::h453e09aaa6e11963Ncs
   2:        0x10e240784 - panicking::on_panic::hcd03717ee6ab0770uuw
   3:        0x10e2033d3 - rt::unwind::begin_unwind_inner::h537a46a93a092087Gcw
   4:        0x10e2041a5 - rt::unwind::begin_unwind_fmt::hd25c40e7531f0196Mbw
   5:        0x10e2400cc - rust_begin_unwind
   6:        0x10e2984b5 - panicking::panic_fmt::h08ef0ff0e76751ecl5B
   7:        0x10e291674 - panicking::panic::hecfda9226c0bd263S3B
   8:        0x10af144b3 - trans::declare::declare_fn::hce3f8e5cc9393547y3z
   9:        0x10af142a1 - trans::declare::declare_fn::hce3f8e5cc9393547y3z
  10:        0x10ae7a65a - trans::declare::declare_rust_fn::h7366ee277331f7153dA
  11:        0x10ae99d57 - trans::base::register_fn::ha48a555b45dc50a9kHi
  12:        0x10ae9ccbe - trans::base::register_method::hce7d57945ad74c2b69i
  13:        0x10ae9428f - trans::base::get_item_val::h0f886a453d3b8caf2Ti
  14:        0x10ae92caa - trans::base::trans_item::hb4bd81f90f1d29a7Swi
  15:        0x10ae92b8e - trans::base::trans_item::hb4bd81f90f1d29a7Swi
  16:        0x10aea18ee - trans::base::trans_crate::hff5ac3e2057c74deYkj
  17:        0x10ad29785 - driver::phase_4_translate_to_llvm::h5a7a1e139434187d0Oa
  18:        0x10ad24f11 - thread::local::LocalKey<T>::with::h1175059562303918311
  19:        0x10ad204f4 - driver::phase_3_run_analysis_passes::h10880160486980056080
  20:        0x10ad0a308 - driver::compile_input::hd66ed3e968244164Tba
  21:        0x10adcfb6d - run_compiler::h1828fb8d678316c3h6b
  22:        0x10adcd74e - boxed::F.FnBox<A>::call_box::h4949254292154653139
  23:        0x10adccfc7 - rt::unwind::try::try_fn::h11365387614610519203
  24:        0x10e2cc408 - rust_try_inner
  25:        0x10e2cc3f5 - rust_try
  26:        0x10e22d4b5 - rt::unwind::try::inner_try::h49b1a141ad8e5e45z8v
  27:        0x10adcd218 - boxed::F.FnBox<A>::call_box::h4580575424180275130
  28:        0x10e23f248 - sys::thread::Thread::new::thread_start::h862819a6cb201e5eTwv
  29:     0x7fff8bf5b267 - _pthread_body
  30:     0x7fff8bf5b1e4 - _pthread_start
```

I believe that frame 9 is the original call, and frame 10 is the inlined call to declare_cfn.


Font-Awesome and Code Mirror are not dependencies for w2ui. Only jQuery. It is in the project because I use it in demos. :)


- [ ] Can we capitalize Branch
- [ ] Also, change brach to Branch.
- [ ] Additionally, there's some strange grammar here: "Now you need to add it (them, if you want to do it for both your live and test apps) to your project"
- [ ] mainifest -> manifest


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (351 of 351 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)


Should be fixed by https://github.com/apache/spark/commit/24fda7381171738cbbbacb5965393b660763e562, could you help to confirm it?


Capitalize Branch


Good advice


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (351 of 351 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)


Description for the test cases of the Polygon and RegularOctagon class.


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (351 of 351 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)


multithreaded_performance (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


so why do you catch and reraise it? Unneeded


84691154_AutomateBuildingOfDocs (lefticus) - i386-Windows-7-VisualStudio-12: OK (1671 of 1674 tests passed)

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


Note: This change results in a `error: failed to commit transaction (conflicting files) texinfo-tex: /usr/bin/texindex exists in filesystem` failure when updating `texinfo-tex`. I think pacman thinks that `texinfo` and `texinfo.exe` are two different files. But who or what ever checks for these files is getting a positive answer when upgrading the package.

Workaround:
`pacman -R texinfo-tex`
`pacman -S texinfo-tex`


@lujunsincerely - please open an issue instead of commenting on a commit. I'll respond when you've done that as this is not the right place.


I am sorry. I honestly remember changing it before submission to what you suggested. I will submit a PR right away


这里实现的搜索是搜索文件内容，不是文件名


maybe we could split out a `Manpage` Object similar to the `HTMLWriter`. `manpage = Manpage(filename); manpage.add_h1("Some Title")` and so on?


Ok hmmm...I'll take a look at this when I get a chance, though at the moment I don't actually have a VM set up so making substantial changes is difficult. Thanks for taking a look at it, I appreciate it!


I think that adding only  '-' here is rather confusing. Passing '-Atc' from the caller is just simple and clear that it's an option.


84691154_AutomateBuildingOfDocs (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (644 of 644 tests passed)

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


Fixed in 46fbca6 thanks.


Demo of "style-src",
what is the effect if set style-src: 'unsafe-inline' ?


Just once I wish Tupac would stop trolling me from beyond the grave


and ideally we could make it share an API with the HTMLWriter which could be renamed to HTMLDocument (a writer isn't a good object, a Document fits just better IMO)


http://pylint-messages.wikidot.com/messages:c0103

It's true, the pydocs do use that structure. To be honest you'll find that people have different opinions about python and how it should be structured and written because of its open source nature. My linter complains about single character variables, (C0103) so I try to avoid them. Single character variables can lead to confusing code if you use too many of them, so I personally use them sparingly.

This would be a great question for Cris to get his opinion. He is your single greatest resource during your time in the bootcamp.

Joel@JoelStanner.com

On Jun 28, 2015, at 10:00 PM, Andrew Wilson <notifications@github.com> wrote:

> I can't seem to find a reference that specifies that, do you have a link so I can learn more?
>
> Python documentation seems to use my structure: https://docs.python.org/2/tutorial/controlflow.html#for-statements
>
> —
> Reply to this email directly or view it on GitHub.
>


OK d


wut


n.b. commit summaries should be in sentence case, not title case


> The monitors will, on startup, `refresh_from_paxos()` which will eventually hit the OSDMonitor's `update_from_paxos()`, which in turn will read the latest full version available.

so you suggest, instead of adding the rewrite paxos epoch to the leader, we should, apply the fix on **all** monitors. i.e. :

1. add the rewrite epoch to "paxos"
1. apply the rewrite transaction to the "osdmap"

i thought the idea was to fit the fix into the recovery mechanism. if that's what we intend to do, then the leader won't commit the proposal enclosed in the newly added paxos commit until it is accepted by the peons.
```
2015-06-29 12:49:42.678971 7fe887990700 10 mon.a@0(leader).paxos(paxos recovering c 1..14) leader_init -- starting paxos recovery
2015-06-29 12:49:42.679140 7fe887990700 10 mon.a@0(leader).paxos(paxos recovering c 1..14) learned uncommitted 15 pn 401 (1063 bytes) from myself
...
2015-06-29 12:49:42.779974 7fe88718f700 10 mon.a@0(leader).paxos(paxos recovering c 1..14) that's everyone.  begin on old learned value
2015-06-29 12:49:42.779977 7fe88718f700 10 mon.a@0(leader).paxos(paxos updating-previous c 1..14) begin for 15 1063 bytes
...
2015-06-29 12:49:42.879837 7fe888992700 10 mon.a@0(leader).paxosservice(osdmap 1..2) refresh
2015-06-29 12:49:42.879915 7fe888992700 10 mon.a@0(leader).paxosservice(logm 1..9) refresh
...
2015-06-29 12:49:42.880343 7fe888992700 10 mon.a@0(leader).pg v3 check_osd_map already seen 2 >= 2
```

> Having a new version, `last_committed+1` would only be required if we were doing this while the monitors are running, which we must not be doing (obviously because the state will progress if there's a quorum,

yes, i guess this is the crux. the fix won't kick in until the leader is elected, and the pending paxos commit is sent to peons.

will ping you over IRC.


Also, I'm not sure we should use "Fix Travis build error" as a commit summary for fixing any bug Travis detects. We should probably actually describe the bug, like "Fix bad style" or "Fix trailing whitespace"


FYI, in AS the option "Automatically Optimize Imports", for whenever you make the switch :)


Abbreviation 바꾼 업데이트만 하신건가요?


This package can be updated without removing with passing "--force" to pacman.


I treated "Travis Build Error" as a thing that was titled as a whole, because.

*shrug* next time


It uses `DXGI_FORMAT_R8_UNORM`:
https://github.com/bkaradzic/bgfx/blob/master/src/renderer_d3d11.cpp#L214


*WEEKEND* date breaking. Duh.


问题主要是：目录没有写完整，还是英文的。标点符号不要用英文的。代码要用按要求标注。


You have done very well nicejob


I'll have to add swizzle for these formats. Once I do that, I'll also add them to list of supported formats so that they can be used natively where possible, without swizzle.


should there be a `return` in front of this Serie table's `Episode.findBySerie`  statement?


Demostrate "connect-src" property,
 Get "http://w3school.com.cn" content, access permitted
 Get "http://www.google.com"content, access denied


Is the prefix 'postgres_' necessary?

It would be better if the name of $postgres_options clearly means an additional option only in this function, otherwise a code reader confuses the difference betwwen $psql_options and $postgres_options.

I think '$exec_options' (meaning only in exec_sql()) or just '$options' is rather simple and readable.


ok


Hey pawitp. Still following your work :-)
Isn't libwvm needed?


is `ID` in the Serie table  an alias for `ID_Serie` (or perhaps the other way around)?