are these numbers impossible phone number as the one we have right now?


no need for this line.
The defaults are set cassandra:cassandra already.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.246](http://teamcity.monogame.net/viewLog.html?buildId=8098&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:20:57


une vieille version que tu viens juste de commit :disappointed:


Should we order everything alphabetically? I have just been going in the order we announce them which is "random".


@spadilazm there was a fix, f2a745c41df44c32149abb27b80e92cede772551, for this applied some time ago.  If you just use the packages provided from the Ubuntu PPA there should be no need to patch anything.


aa


Yup I have wondered for some time whether I was being crazy, as there is a test setting `:unique` to `true` in `association_spec` but I guess the way it's done does not trigger a call to `validate_association_options!`

I'll see if I can fix that (and the tests)!


For fun xD this is because in the future there will be continuous modifications to the file xD


@sdt well speed is not one of the requirements for merging (although I would like to see at least native tests not timeout). On the other hand I would be really surprised if perl6 couldn't pass the native tests without timing out unless there is something accidentally polynomial/exponential in the implementation. Even the bash and make implementation can pass the native tests and they are VERY slow.

Are you certain it wasn't the self-hosted mode that was slow. Sometimes a bug in arg processing can result in the mal implementation hosting itself (3 layer rather than 2) which makes things really slow no matter how fast the underlying language. Although I don't think that bug happens until step6 as I recall. Also, if you run the same fib code manually after TCO is added (step5 or later) do you see the same issue. Perhaps it is a stack related issue in perl6.

That image is a great analogy for perl6 :-)


@JoeMattie
Yes, it's the same with node. Remember chrome and node uses the same js engine. V8. (If you tested in chrome).

Anyways heres an image after testing with benchmark module in node (same used in jsperf.com) and same code as I used in jsperf:

![selection_017](https://cloud.githubusercontent.com/assets/1419214/8010301/6e67e50c-0baf-11e5-90f2-e2e8333aadb7.png)


Seems that disabling CONFIG_EXT4/F2FS_FS_XATTR and CONFIG_EXT4/F2FS_FS_SECURITY was not the best idea - some selinux stuff in ROM depends on it - so, the ROM will fail to install without that. I'll look into it later.


kita perbakinya emang di index.html atau index.jade?
karena yang aku terus revisi ada di index.jade sama main.styl


Great! Feel free to run these benchmarks to test your stuff


should be good now @tchoulihan
ran it from scratch and it built on my environment.
If you get any errors please send along.


@KatherineKay you are changing the wrong file. We are on v33.


Should note this downloads from the Hub.  Something like:

Currently the container is uploaded to the Docker Hub, though future work will have it (and other layered images) come from Fedora  release engineering.  You can get it with `atomic run fedora/cockpitws`.  Then cockpit will be running on port 9090.


Copy and paste error.  I have fixed this.


오늘작성


Thanks lol - bad habits suck! I tend to use that operator a lot when I shouldn't:

https://github.com/anomalylabs/streams-platform/commit/1cdda87b938356da54ab84eb68e8f17d6e67b8b1


removed ~


Lets will -> Let's?


ok update readme


nope, I have no clue.
We could give the user a build, let me know what operating user he has.


Don't mutate the `path` field in a transformer function. There's no reason calling a function like `toAbsolutePaths(..)` should cause the caller to think, "Oh, the underlying object may be changing out from underneath me."

Instead make "`path` does not have a trailing slash" an invariant of the class. The logical place to enforce the invariant is in the `path` setter.


i think ordering by announcement time is fine


> `StateReference.identifier()` is necessary to know what was used to initiate the transition, but it is clunky to use because you still have to check for string or object and behave appropriately.

I think you're thinking of two different things. As a user, if I use string state names in my app, I'm going to use them consistently, both in `transitionTo()` calls, hooks, and elsewhere. Likewise with objects. Internally (or as a plugin developer), I can use the internal APIs to check (internal) state object identity or whatever else.

If we want to give people easier _public_ access to _their_ state definition (vs. the internal one), I can roll back to something closer to what I had in my working copy initially, i.e.:

```javascript
to: extend(function() { return to.identifier(); }, {
  state: function() { return to.state(); },
}),
```

That way you get `transition.to()` for whatever you passed to `transitionTo()` for doing basic checks, and `transition.to.state()` to get access to the whole (user-land) state object.


doesn't matter if it's final, if findTorrent returned null... null may have been assigned.


В первый раз лучше написать полностью, чтобы почувствовать это)


Pinging @fatbusinessman for the sake of nostalgia :hourglass_flowing_sand: :hocho: :partly_sunny:


Jenkins test this please


Смотря нужно ли тебе подтверждение пароля, зависит от твоего функционала


If during object construction is already null, you should get the NPE very early in the constructor, not in getName


Sí.

I can't think of anything off-hand. Maybe go through open issues one time (starting with milestone-marked ones for 1.0 or future) and see if we're missing anything?


Would be aided with some form of artificial target restriction


hi


This makes it necessary. Since we are constructing blank dataframe, it should pick the schema from query response.


Можем изменить схему. Если тебе будет удобнее, я могу после коментирования отписываться от уведомлений, но потом ты сам мне будешь присылать ссылку, где мне нужно ответить, в скайп или на почту


Please remove this and use the TestFactory methods that exist for this function.


I think this singleLine issue will be resolved with my next edit. We do not need four text-related views here.


you're right :p


I think you should keep both of getSnapTarget and getNearestContentElement. And by default dropdown search nearest md-content (or md-autocomplete-snap) for attach self to him. If in md-autocomplete there will be md-autocomplete-dropdown-at-body attribute than this search will be skipped and document.body used


Yeah, using "alex" as opposed to "Alex" works just fine.

For any name though it seems that using capital letters at the beginning won't work.


ack


@KatherineKay I removed you from write access, please use Pull requests until you are comfortable with working with github (Please consult the help pages "help.github.com" if you don't understand certain things).

Workflow:
1. Fork repository (Click on fork button on the top right.) https://help.github.com/articles/fork-a-repo/
2a. Do your changes on your own fork via webinterface of github.com (e.g. KatherineKay/openmalaria.snippets once you completed step 1)
2b. __Or__ commit local changes and push them to your forked repo.
3. Create a pull request (Click on the green button to compare your changes to another repo you forked) https://help.github.com/articles/using-pull-requests/


I caught that earlier, must have reverted locally, need to check version..


Bisous <3


I have been getting an error in the function weightmedian provided in practice. Please help.
![error](https://cloud.githubusercontent.com/assets/7758472/8010495/379d2cc0-0bce-11e5-93f0-fc6df87ed52e.jpg)


What I had in mind was to use [requirement specifiers](http://pip.readthedocs.org/en/latest/reference/pip_install.html#requirement-specifiers) instead of just stop using the version, but I get your point.


Can do, but, what does it mean to "come from Fedora release engineering"?  Will it be in a different registry altogether, a different repo / user in the Hub?


Why is this not a Boolean?


@lemire thanks, i was on the way catching up with the latest development, so just tried with vs2008 again and made some small adjustments. Now it's worky again there. As mentioned before, my good guess were that in 2 years or alike 2008 will be history :)

Thanks.


Yes I got it: the test calls `Neo4j::ActiveNode::HasN::Association.new(...)`, and the `HasN::Association` class does check for valid options: only the `has_one`/`has_many` methods do.


I'm not sure this has been discussed a lot but...why not `atomic run fedora/cockpitws`?  It's because we don't want the `-d` flag right?


I saw this diff and realized that it was a bug. This implementation is much better. It was the original intention so it doesn't break anything. Thanks.


just thinking ahead, not needed in this PR but maybe later, we should probably also allow in-memory file contents to be passed i.e. not read it from filesystem again.


`/` is linux specific! I'd just remove this test.


Ah ok.  I pushed a few minor updates and a patch release last night because the package still wasn't activating for some reason.


:trollface:
![batosurl eau](https://cloud.githubusercontent.com/assets/2819927/8010566/67ace36e-0bb1-11e5-81dc-7a9192680277.png)
Ca manque de relief vu que c'est juste une lumière ambiante, mais je trouve que ca rends déjà mieux :)


Good catch: https://github.com/daid/Cura/pull/1272


@vagnervjs yeah you are right, i'm gonna do that


uhm it actually lauinches the daemon, not only stting some env vars right?


давай. если возникнут вопросы, я тебе скину.


why `dataset` and not `data` ?


i.e. ceylon/ceylon-spec#1319 and ceylon/ceylon-spec#1318.


hi mom


Would it make sense to use `/v2/<name>/manifests/<digest>` like in the next line, rather than using `<reference>` and then clarifying that really we just mean digest?

Then again, the "Deleting an Image" earlier in the doc also uses `<reference>` and then clarifies that it must be a digest, so I could see it either way.

Just tossing the idea out there, no strong preference either way here. :-)


I'll pull this in, but please update the code to use the number-crunching methods from TestFactory. Thanks!


Yesterday I have some problems, with a addon, but increasing timeout value it it's settings, seems to resolve a problems.
Anyway I'll check again.


link first mention of  "distribtution" to https://www.drupal.org/documentation/build/distributions ?

How about reworking the introduction to sketch out the scenario a little more?
1) If your site reports there is a newer version of Drupal core or contributed modules (could be a link to a separate article on how to know this that includes screenshot of "status" tab?)
2) And you don't see a 1-click upstream update available on your Pantheon Site Dashboard
3) See if update has been committed upstream. Settings >> About Site >> Upstream URL. Look at recent commit. If it's less than an hour or two, wait and try accessing your Pantheon Site Dashboard again to see if 1-click upstream update is available. If it's been more than an hour or two and you DO see commit upstream, then open a support ticket.
4) If hasn't been committed upstream, then... (what you already have, pull in manually)

remove "alternate"


"the updates did not negatively impact your site" -> "that everything looks good"  ?


Hey @rachelwhitton. Looks great!

Added a few comments.

Also might be worth mentioning that people can manually upgrade contrib modules as well, and maybe link to https://www.drupal.org/node/1974964


No, it just sets the env vars.
You can actually run it on your computer ... it'll show the env vars it sets.

These env vars are used when python does `import dbus` to launch dbus.


As an example I did a Pull request on my own: https://github.com/SwissTPH/openmalaria.snippets/pull/1


Should kwargs be changed to match [docs](http://www.imagemagick.org/script/command-line-options.php#level)?

 - `black` => `black_point`
 - `white` => `white_point`


Excellent!


pooy


@dlonie yes. It should be in there, but that's going to change the baseline. And it's really not the purpose of that PR (remove deprecated AutoAPI). I should fix it in another PR. Or you can put the comment back in, and generate the new baseline in this on.e but it's kind of unrelated.


good solution!


The original is better than your update. What is with you company men trying to rehash the classics.


I changed the file structure (sorry Justin):

* Got rid of the "states" directory because they are all states. In the future there might be more than just states in there, but we'll leave it for now.
* Added an "encounters" directory where the encounters will live. Each encounter will have its own directory in here with an assets folder, etc. The encounter generator will write to here and make a directory for the encounter it's creating.


#define cellFsClose_symbol              0x297768 // old: 0x296B18 // bytes matched: 0x10  F821FF617C0802A6FBC10090EBC225B0


http://dbus.freedesktop.org/doc/dbus-launch.1.html ?


:ok:


I don't think this is going to actually work.  This isn't serialized to the database, so on the next web request it will be nil.

Also - how come we didn't use a pull request for these changes?


You're genius man ;) thank you so much ;)
On Jun 5, 2015 7:38 PM, "kodi1" <notifications@github.com> wrote:

> Yesterday I have some problems, with a addon, but increasing timeout value
> it it's settings, seems to resolve a problems.
> Anyway I'll check again.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kodi1/plugin.video.bscfusion/commit/d5ee9a8cb6c6e1f5cbf94aa571d6754a47730508#commitcomment-11544942>
> .
>


Oh o.O
Seems I was wrong


Ок


also where if not here do you start it? As it just starts the daemon maybe you can start it in the .install.dbus script and stop it in .success.dbus or so, isn't that cleaner?


i.e. no fiddling with the invocation


Only Addy is.


You've defined multiplier as an Integer type which has a max value of ~2E9, so it won't be able to handle the 1E12 number here.

The kilo- prefix should be a lowercase 'k' (though we haven't been there since before the 8080)

When you switch it to Long (or long) can you also refactor this piece into oshi.util.FormatUtil and write a Junit test for it in FormatUtilTest?


Looks good to me. Let's merge and deploy. Could you push it to Heroku? I'm not at my computer atm.


Works like a charm. Thanks!!!


Let's make sure we have newlines at the end of these files. You can configure Android Studio to do this for you.


It's ok to update baselines if your change affects a test. The test needs to change as a result of the intended change, and the baseline needs updating as a result of the changed test, so it is related ;)


RadianceRefactor (rpg777) - x86_64-MacOS-10.9-clang: Tests Failed

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


原来如此！！


So, again - it doesn't just start the daemon. My assumption was that it sets environment variables and the dbus-daemon is lazily started later when it is needed.

The environment variables are needed. And as travis runs each line in a new shell, it cannot be started anywhere else, and has to be on this line (I tried it earlier)

[link](https://github.com/favonia/pulse/issues/20) for similar issue


This closes #310


Right, if you use `atomic run fedora/cockpitws` in the unit file, the -d flag causes docker to daemonize w/o telling systemd which then thinks the service failed and runs `ExecStop` I mention that briefly in the follow on paragraph


university of washginton :see_no_evil:


Ideally I'd want a commit hash as "version". Also, some cpuinfo (eg. `head -1 /proc/cpuinfo`) would be nice but not portable.


We could probably get rid of some more cruft in these tests (like the "people_distinct_id" entry in the test properties)


The Integer is the Version.
When no specific version is requested it will take the highest version availiable.
TreeMap is sorted and has getLast()


you need to take account of any arguments that might be passed in the the function.

So
```
host[name] = function(...args) {
  originalFn.apply(this, args);
  cb.call(this);
};
```


my 2nd commit


Is there a reason why support for PHP <5.6 is dropped ahead of its EOL in September 2015?

> http://en.wikipedia.org/wiki/PHP#Release_history

The recent drop of support for PHP < 5.4, on th eother hand, is fine, I think.

https://github.com/sebastianbergmann/phploc/commit/0a4e48c772d51527a17cb5573bd9f22e96fae574


We may want to deprecate this and just call `identify`, or deprecate as a no-op and issue a warning.


改回。


Dude!


@geoff-wasilwa  the is_registred field in the database drip user table currently doesn't change from 0 to 1 after a user is created. Do we have an update function for the same?


There will not be a release from `master` before September 2015.


just checking you mean to test the `typeof` child.type (as opposed to `typeof` child)


Actually, I forgot that doing the teardown here is also incorrect because it prevents the `confess()` message from going to the correct logfile.


@geoff-wasilwa also getting this error http://stackoverflow.com/questions/24074749/spring-boot-cannot-determine-embedded-database-driver-class-for-database-type when building, I gues it has something to deal with the dependencies. Please advice.


lgtm


I think this is the same as `(hasTextContent || hasAltText)`


as I understand it, move_to is both the capture logic and the displacement for the new position, hence the self.update_attributes

i dont think it needs to be serializes in the database just to have a boolean switch to whether the piece moved from its initial position, at least from my understanding. The only place I call start position is after the game is created, and then @has_moved immediately is switched to true when a piece moves.

I messed up merging the pull requests by merging a branch I had open to master.


Style nit: `plugin_albums` instead of `pluginAlbums`


It may make sense to allow the user to specify the parent element in much
the same way that $mdDialog does.  How does that sound?

On Fri, Jun 5, 2015 at 9:27 AM, syam-ardas <notifications@github.com> wrote:

> I think you should keep both of getSnapTarget and
> getNearestContentElement. And by default dropdown search nearest md-content
> (or md-autocomplete-snap) for attach self to him. If in md-autocomplete
> there will be md-autocomplete-dropdown-at-body attribute than this search
> will be skipped and document.body used
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/material/commit/7f355f6d787520cc6a047f46067971b25d79ee73#commitcomment-11544769>
> .
>


to resolve ng-touch issue


I was speaking with my adviser on this and he was adamant about not tracking a first_move value in the db so this is the solution we proposed.


@weltling

Excellent!


Refactored the javascript that was in idea/project/search views so that modal-buttons everywhere will now just open their sibling dialogs.


Allows aligning text in different ways for different screen sizes.


It seems you mispelled "canola"


Should the word 'latin' be capital?


This was never synchronized before, but probably should be since mPersistentIdentity is advertised as thread safe. I'd like to propose
```
synchronized (mPersistentIdentity) {
    mPersistentIdentity.setDistinctId(distinctId);
    mDecideMessages.setDistinctId(distinctId);
}
```

DecideMessages.setDistinctId is synchronized but doesn't call out, so we should be ok wrt to deadlock.


Please @ me when you do so that I can download it. Just back from lunch, BTW

2015-06-05 18:16 GMT+02:00 mbonsma <notifications@github.com>:

> Also, I'm going to edit the example file to include some Cas genes so
> there will be an actual output.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JJ/phageParser/commit/6f99aa23dfbce6898538bc6f589c11e86d1ceba5#commitcomment-11544587>
> .
>

--
JJ


Avoiding HTML duplication. I made text-right- classes so you only have to write your elements one time but can have different alignment for different screen sizes.


What is std_misc needed for? It's a warning on OS X.


These lines need to be  sub(1, 7) and sub(8).


should be public_id, not plan id


It's updated @JJ - `Genbank_example.txt`


Clever, good work!


`host_name` a la rust style?


If we do figure out the proper old compiler fix for this, let's make a macro which encapsulates it rather repeating all these explicit ORs?


You'll also need to update `stdPeopleMessage` - it currently adds a distinct_id to the set if and only if we've got a people id.


We'll need to document what happens to folks who have currently set two ids, when they upgrade to installed customers. (We can either migrate, or just ignore it, but we'll be pushing the lib to devices that already "people_distinct_id" preferences on their drives)


thanks going to nuke this in a day or two anyway


![mobile mockup](https://cloud.githubusercontent.com/assets/11215496/8011051/1b735f20-0bbe-11e5-877b-6f2def20633a.jpg)


Ha


How come he was adamant about not storing it in the database?  I'm cool if we store it in a different place (cache, redis, something else, database is just most straightforward)

I'm not against storing it someplace else, but I don't think this method will actually work.  On the next request when we find the piece, it will lose the state of the instance variable.  the `has_moved` variable will basically always return nil between requests.

```
p = Piece.first
p.move_to!(4, 3)
p.has_moved # will return true

p = Piece.first # reload it from the database
p.has_moved # will return nil - has_moved hasn't been serialized to the db.
```


For `into_ascii_lowercase` (https://github.com/ecoal95/rust-layers/blob/37cda37cc60b26c2e25120d2bf6ee7c71e8dc6d6/src/platform/linux/surface.rs#L151)


@GrahamCampbell I've added v4 here


I like that you made your variables private.


access token expire hojata h shayad, mera toh working

On Fri, Jun 5, 2015 at 9:25 PM, Faizan Ayubi <notifications@github.com>
wrote:

> What was the registeration error?
> send me screenshot?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Swiftintern/Swiftintern/commit/523cdfcecad93159bc0e36187d70d0bcec3fd2f9#commitcomment-11544195>
> .
>


here, you can use `NS_ENUM` instead, then In swift it will be `ENGAVFoundationCameraMode.Video`


This was preventing my `build blocks` from completing. Found the solution here:
http://www.evotech.net/blog/2007/04/named-html-entities-in-numeric-order/#comment-26446


I think __strong is default


a parte de colocar todas as opcoes a gente vai ter q fazer sim
na terca, o daniel mostrou um pdf q listava todas essas opcoes (mas n ta disponivel esse pdf no moodle, e eu ja mandei email pra luciana ontem :/)
nao entendi mto bem o DAO; o web service (qdo ele n cai neh) ja retorna uma string no formato XML do bd


CSS3 Transitions not supported by: IE (8,9), Opera Mini (5.0-8.0)


and you can move all these ivars to m file, most people needn't see this ivars.
Or if you use property (in extension) you can just delete all these.


TeamCity YaccConstructor :: YC.Core General Build [Build 431](http://teamcity.codebetter.com/viewLog.html?buildId=192757&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Nit, inlining this is fine


Is there a stripe API call for this?


Why 100?


Fair enough.


alright, you're right from the looks of it. So I will just need to run a migration on the DB and setup a has_moved tracker of sorts?


I think support block would be awesome


This logic would allow `accton_as7710_32x-r1` as a valid machine, which according to your comments above is not a valid configuration.


@gdibble Why do you have `lodash.debounce` twice on lines 27 and 28? Did you mean to include `isfunction`?


I prefer static


Use `str.translate()` here, not regex.


why are we using `any()`/`all()` superposition functions for this?


You mean this? https://github.com/badges/shields/pull/453

Yes, I'll review it soon :)


Thank you @DomT4!


static


This is wrong (and breaking homebrew). Should be `\"` instead.


I'm not wild about overloading or optional parameters except when we're trying not to break legacy code. It's harder to discover the whole interface, it means that mocks and interfaces are more complex, you can't really rely on one method delegating to the other in base classes, and they often proliferate out of control. It's even a bit worse in the presence of class hierarchies, where thing(Object) and thing(MyObject) are ambiguous.

Mostly, I'm just being grumpy. This is fine.


yeah i wanted to know how it finds first post : )


cross out "to the page" . Visitors will learn


I see that you did that in 010b0b3579e2


Because in the [templatesources.json](https://github.com/ligershark/side-waffle/blob/autoupdate/TemplatePack/templatesources.json) file that we use the value is in quotes.


There currently is no infrastructure for running only part of the tests. But most of tests are written used “missed dependency is skipped test” approach, tmux is one of the rare (I hope) exceptions.


Sure. I never do this because all of these are in my global .gitignore file. :)


Looks good.


TeamCity YaccConstructor :: YC.Core General Build [Build 431](http://teamcity.codebetter.com/viewLog.html?buildId=192757&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: Error during Nuget update. ..\tools\Build.Tools\NuGet\NuGet.exe install "Z:\Builds\work\4138ce7c6332a78c\src\CYKOnGPU\packages.config" -OutputDirectory "..\src\packages" -ConfigFile "..\src\NuGet.config"   at FSI_0002.Packaging.rest... Build time: 00:05:12


It's outputting now

```
    -> 229..1590 ==> Type I
-> 229..1590 ==> Type I
-> 298983..299912 ==> Type I
-> 298983..299912 ==> Type I
-> 1750..2868 ==> Type I
-> 1750..2868 ==> Type I
-> 4418503..4421010 ==> Type I
-> 4418503..4421010 ==> Type I
-> 2942..5575 ==> Type I
-> 2942..5575 ==> Type I
```

I'll check if that is correct.


ok @hellopath, great! Thanks!


Yes, good idea


@arjunguha Yep, that works for me in my VM if I do "sudo make reinstall".  And it works in Travis.  So it can be merged in.


That's the most straightforward way to do it.  How come Sean was against putting it in the database?

I'm sure he had some good rationale behind it.


After npm install, you'll find all the .jsx files in the /src folder and their compiled versions in the /lib folder.


Needs a comment explaining the send queue strategy and when we disconnect, what problems we are trying to solve.


`db_create` is actually disappearing, so this is unnecessary.


It's not. I'll see what's going on.

2015-06-05 19:14 GMT+02:00 JJ Merelo <jjmerelo@gmail.com>:

> It's outputting now
>
> ```
>     -> 229..1590 ==> Type I
> -> 229..1590 ==> Type I
> -> 298983..299912 ==> Type I
> -> 298983..299912 ==> Type I
> -> 1750..2868 ==> Type I
> -> 1750..2868 ==> Type I
> -> 4418503..4421010 ==> Type I
> -> 4418503..4421010 ==> Type I
> -> 2942..5575 ==> Type I
> -> 2942..5575 ==> Type I
> ```
>
>
> I'll check if that is correct.
>

--
JJ


In my opinion,
what if keep `initWithFrame` with defaultValue settings, and  with  `mode`, `stillCameraMethod ` .. properties.
Just my opinion
PS: `id` -> `instancetype`


We do have that which-insert-to-use code in a few places now, don't we?  And the do-we-use-hashes preprocessor test in a couple more.  Mind if I merge this as-is and refactor in a separate PR?  I'm introducing yet another example in #597 and I'd rather get the ones from there and here in one swipe.


I like this code that Etesh helped us with to scale the image but I also like the code that we found the android docs too.
http://developer.android.com/training/camera/photobasics.html#TaskGallery


Racket's `modulo` has a different contract: it only accepts integers, hence the extra steps in Robby's code. `%` in javascript accepts any real, so I believe this matches the behavior of racket.


Closes #4


@gaearon I think this section requires a bit more explanation and what the `role` of this component should be (e.g.: should be used once because of the dispatching, ...). Also maybe list some tradeoffs by _not using it correctly_ ?

What do you think?


I think he just doesn't like to clutter the DB with one off boolean switches like this, but this was my initial gut feeling to store it in the database, but he over-rid my decision perhaps because of desire for simplicity? I didn't get to dive deep into his rationale because it was near the end of the session.


Tag is our friend. I learned something new.


> Mind if I merge this as-is and refactor in a separate PR?

I don't mind, that makes sense.


Um. I doubt it? But it doesn't change often and I think it's okay if we're off. I just updated it because I was thinking about it.


I agree. Will elaborate next time I edit docs. :+1:


Ok, right I see that now.  This would be nice to fix somehow with `atomic run --foreground` or something that knows how to delete the `-d` from the `docker run` but...hacky.


How did a period on a line by itself compile?


Heyyy, cool vimrc :+1:


rax retry tests


Well Done


Ah - makes sense.

You could potentially get the best of both worlds by changing the concept of "has_moved" to "piece_status".  This can potentially be used in other cases as well.

`piece_status` being one of `['unmoved', 'moved', 'pawn_moved_2']` could potentially be useful in the case of determining en passant, and more flexible (rather than requiring we use a bunch of `true/false` fields.


I'm going to brainstorm here without much knowledge about our API or even REST vocabulary, so please bear with me.

It appears to be that having /statistics as a first class object in our API is not the way to go. The stats seem to be like a property of an object not an object itself.

So shouldn't this be modeled reflecting this somehow? For example. This is what you're proposing:

```
/statistics/
/statistics/jobs
/statistics/last_job
/statistics/last_10_jobs
```

But we already have `/jobs`, so why not something like the above?

```
/jobs/<id>/stats
/jobs/stats
```

Finally, my impression is that `last` and `last_10` should not be part of the API (they could probably be provided via pagination instead, no?) I guess there's a proper "RESTFUL" way to do it, but like I said above, I'm not experienced enough to make an informed suggestion.


So you're saying that if `debug` is set to `False`, `use_reloader` will be ignored? I understand that this seems to be what the docs say, but it isn't true: running with `user_reloader=True` without setting `debug` to anything results in the reloader working just fine (and without the debugger running).


You should include a credentials.json.dist as a template that users can copy and paste as credentials.json.


`AssetsLibrary ` is not necessary for AVFoundationCameraController, It's just for Demo save photo, just add it to demo project


Yeah, it doesn't look like it. We could just download and parse their page every time, but that would be super silly. ;-)


@arjunguha Oh, sorry.  It's already been merged.  Nebber mind.


^ Fixed in https://github.com/Homebrew/homebrew/commit/2de0055c3606a9d0d54a9aec1eb2112a95f25154, Sorry!


The `fedora` creds only currently apply to the cloud image; if you do a baremetal install it'll be whatever you chose.

Maybe something like:  Log in as any valid user, although to actually perform administration you'll need to be in the `wheel` group.  The cloud image uses `fedora` as a username by default.  Hm...I'm not finding anything in the Cockpit docs for this ( http://files.cockpit-project.org/guide/guide.html ).


Ok xD


Thank you for this!


take look at http://necolas.github.io/normalize.css/ - it's initial stylesheet which makes default styles for all elements same in all browsers, it contains padding 0 and margin 0 too I guess


I think that's undefined currently.


The largest port number is 65535, so u16 will suffice.


remove whitespace


This enable using external images passing by parameter into report


Delete ViewController


+12065551212 is not an impossible phone number, that's very much a valid US phone number.

Not sure on +99, would be good to check which are truly impossible.


testing comment yo.


why not `HashMap<&str, &str>`?


Tssssss


Why exactly would disabling the scrolling cache be a good thing?


The first sight I saw AVFoundationCameraController, I think it's a controller rather than a view, it's not that apt.


Good call.


{ should be in a new line


position: relative is not needed here


Ooops.. it is! I always assumed it was unaddressable and that's why the movies used it.
http://en.wikipedia.org/wiki/555_%28telephone_number%29


Change phone numbers to `string`. Often, they do not fit in a `long`. And we may add ISD code so we need to be able to use stuff like '+91'.


That looks kind of right - so it's spitting out the gene location and which type it is? It would be good to also output which Cas gene it is too.

Also, each line should be associated with an accession number (i.e. NC_015138)


Oh, it certainly didn't.  But I got lazy and added the fix to this commit rather than rebase-fixup the last.


Why does this not have a `static` lifetime but `URI_SCHEME` does?


I haven't tested this on Python 2 at all. Will it even run?


Nice thanks for fixing it


Mmm. I doubt it. I probably would have done it so that md-autocomplete and his dropdown bind together throw autocompleteController, like input el's bind together throw ngModelController


fix whitespaces


use padding in `.top_background` instead, positioning every element by `top` and `position: relative` is bad idea


Ya, maybe we just use +12065550100 through12065550199?

That would work for up to 100 users, which I think is totally sane.


Please use the conventions as well. Change everything to camel case. Also I suggest changing Prizes to an array of 3 `int`s.
Rename duration to timings and make it a `DateTimeOffset` object.


set margin-bottom for `.banner` to make space between banner and logo


95652938-jsoncpp (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1670 of 1670 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)


:+1:  thanks!


:+1: Exactly what I was getting at. I would say "great minds" but I know mine is not.

Might not be necessary but maybe a `<script>` and `<link>` example under the compiled CSS usage? Like, "then just simply ...".


`Analytics` is undefined. Arguably, we should be tracking this at the UI layer. @dansomething, has some cool angular-directives that could be adapted to whatever analytics lib is pulled in at the app level.


Also, could you provide a CodePen or link so that I can see a sample of the
issue you are having?  I'm not totally clear on the issue, but I'd be glad
to help with a workaround or provide some sort of alternative syntax.

On Fri, Jun 5, 2015 at 10:27 AM, syam-ardas <notifications@github.com>
wrote:

> Mmm. I doubt it. I probably would have done it so that md-autocomplete and
> his dropdown bind together throw autocompleteController, like input el's
> bind together throw ngModelController
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/material/commit/7f355f6d787520cc6a047f46067971b25d79ee73#commitcomment-11545704>
> .
>


don't set fixed height on containers, let them adjust automatically to the content


oops


I've been really having issues showing the correct Visitor IP when using the Proxy Script to Hide the URL as defined here:

http://piwik.org/faq/how-to/faq_132/

However, when using this script it still DOES NOT show the correct Visitor IP.

I have tried doing the settings outlined here (http://piwik.org/faq/how-to-install/faq_98/), but still no luck.

It seems to be tracking correctly when I don't use the script.

Any ideas? I've been really having issues showing the correct Visitor IP when using the Proxy Script to Hide the URL as defined here:

[piwik.org]

However, when using this script it still DOES NOT show the correct Visitor IP.

I have tried doing the settings outlined here (http://piwik.org/faq/how-to-install/faq_98/), but still no luck.

It seems to be tracking correctly when I don't use the script.

Any ideas?


should have been MAuth...


Not available in Py2.6.


If you remove relative positions, elements will align correctly one below another


Hi @biiterlove,
this is so that rsync ignores permissions, owner and groups (otherwise warnings might show up).
Find some information here: http://unix.stackexchange.com/questions/102211/rsync-ignore-owner-group-time-and-perms


same as above


can you use `"test".to_string()` instead?


Let's just call this `scheme`, with allowable values `http` and `https`. The current name, `nonsecure`, is confusing because neither scheme is more secure than the other, with certificate checking disabled.


set some class on it, otherwise all lists will be styled like that and this is not good practice


same here, set some class


You shouldn't have to patch Angular itself. This is a iOS/WebKit bug so they should fix it.


Just a hello for now...new and don't understand much of it.......step by step instructions and how tos
could be found??......thank you .....


Did you measure any performance improvement?


don't set fixed height, let it adjust to the content


Do you use git pull --rebase?


Indeed, nvm


@subvertallchris #824


Good, will see into that.


@jeffbcross this seems to throw a xss warning when I run it against security scans, any thoughts on why this might not actually be a concern?


Fixed ab84cc3757b753cdbb648af2fe88718a8d708542


Closes issue #13


Works great :)


Resurrecting the Python testsuite and running it on Python 2.4 (RHEL5) confirms everything works as expected. Commit 691db8e312adb727f9e12392e62ed56ec95a9cc8 updates the minimum required version.


Yes, that's correct.


Great idea!


TeamCity Xbim :: XbimGeometry :: XbimGeometry [Build 3](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=12&buildTypeId=Xbim_XbimGeometry_XbimGeometry) outcome was **FAILURE**
Summary: Compilation error: Xbim.Geometry.Engine.Interop\Xbim.Geometry.Engine.Interop.csproj (new) Build time: 03:08:31


L and LA PNGs are still expanded to RGB, by the way.


What about double-double? :-)


hmm, yes, looking at this again, we should flush (and compact) with the 'to' inclusive

but we need to change the way we split the ranges since we now generate boundaries with `token.minKeyBound()`.
For example, say we own tokens a, b, c:
`(min(minToken) -> min(a)), (min(a) -> min(b)), (min(b) -> min(c)), (min(c) -> max(maxToken))`
we need to do:
`(min(minToken) -> max(a)), (max(a) -> max(b)) ... (max(c) -> max(maxToken))`

I will fix


Struggled with the user thing originally, I'll add some more to make it clear the user may change.  I didn't find anything in the docs either about user / group needs, but earlier testing I know `wheel` is needed.


Hi @xhacker,
This change - 231d4c318ed001ac1b36fe4ef99128280fec77a7 will enable you to achieve the desired result without adding the method `datePickerView:customViewForDate:`. Simply add the desired view in the subclass of `RSDFDatePickerDayCell`, and then use it in the subclass of `RSDFDatePickerView` in the method `collectionView:cellForItemAtIndexPath:`.


saved id in variable instead of using a flash to save id


Constants implicitly have static lifetimes since they're effectively inlined anywhere they're used. String literals are a special case where their static lifetime has to be explicit if the type is included.


Thanks, I guess


@ruslanskorb Cool, thanks for explanation!


Heh, nice.


No need for __restrict__ here


Hi

2015-06-05 19:26 GMT+02:00 mbonsma <notifications@github.com>:

> That looks kind of right - so it's spitting out the gene location and
> which type it is? It would be good to also output which Cas gene it is too.
>

You mean Cas3, 6 or nine? Not just the type?

> Also, each line should be associated with an accession number (i.e.
> NC_015138)
>

:check:


Instead of having this really long if block, I'd rather have:
```
if (!address.starts_with(URI_SCHEME)) {
    Err("MongoDB connection string must start with 'mongodb://'")
}

let addr = &address[10..];
....
```


Looks good. Can you also include the Cas protein name, so in this case something like:

```
->NC_015138: 229..1590 Cas3 ==> Type I
->NC_015138: 4418503..4421010 Cas3 ==> Type I
```


I met similar issue, and actually this fix had not perfectly fix that... since eventually after the last line pop out, then the length turn to 0, so again the lines[0] will report the out of range error;(

I fork and just insert one lenth check at least work for me to solve the issue, I will try to raise pull request then.

But, really good work/package you make:) thanks !


3rd party imports should be on their own, i.e.: with colors, kazoo, etc.


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: 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)


How is `'a` used?


I think it is better to just add a function called checkTypePredicate, which checks the parent to determine whether the type predicate is in a valid position.


Looks awesome


@xhacker you are welcome!


we need this in setup.py too (yeah i know it's duplicated, i need to fix setup.py so that it reads requirements.txt ...)


Except the bug will probably remain present on thousands of devices for months. So until the majority of users upgrade I will have to patch angular for their sake.


OK, I like how this patch, even though it overlaps with the previous one, tells how the idea behind the PR evolved chronologically.


Fixed #410


Yes, this is much better.


Yes, exactly - it'll be good to keep it around along with the type.


Ok.


FYI @garthwebb, I updated this comment since we're no longer doing compatibility normalization.


OK.


Other than the comments, lgtm. Thanks @franckcuny !


Initially had it that way, and moved to owning any strings contained within structs. I'm not sure what the expected standard is, so moving back to &str's for HashMap is an option.


@liewegas can probably put it in req_info::req_info(). Looking at the code, I'm not sure there's any other user for req_info::host, but will be cleaner to put it there. Also could change it to be a string, not a char pointer.


I'm going to revert this because this is causing test failures on almost all the branches


Please remove. Callers should use plain sumProd, with vectorization happening behind the scenes.


OK, this is indeed actually part of this commit, different from what I thought previously. I see it's needed a few lines below.


Please use plain sumProd here.


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Also use sumProd here. See SubList.H.


Again, I'm fine with this as a bugfix. I'm opening up a card so that we don't lose the idea of the framework code being able to use parameters (and its defaults) without command line arguments having been parsed (these things should, IMHO, be orthogonal).


Should not be needed.


That's done in the latest commit. I guess it's PR ready, right? :-)


I would like to add this back, if we implement all those `interfaces` in the `SystemStats` main class, we know for sure that we implement all the methods.


Hey @olivierlacan, I'm getting this same deprecation warning and couldn't find yet a way to silence it. Did you?
I found your build on Travis with the same warning message.


Yeah! Thanks!

On Fri, Jun 5, 2015 at 1:53 PM, Juan Julián Merelo Guervós <
notifications@github.com> wrote:

> That's done in the latest commit. I guess it's PR ready, right? :-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JJ/phageParser/commit/3fe5bd68c6fd1c985deef1594c33f4458a047da9#commitcomment-11546084>
> .
>


What is the reason for this? Not to compile rare paths? Do we add `CompilerDirectives.transferToInterpreter();` by convention or is required for some other reason I am missing?


The simple solution would be to `throw` with all the information you need to fix the bug (e.g. the team and format causing the crash).


You can also manually call crashlogger, which is like `throw`ing except the team validator doesn't stop working for everyone.


Interesting. So that's really a good usecase for https://github.com/nviennot/synapse (paper and slides available in the readme). I however don't recommend using synapse due to large operation overhead.

In your case, What about simply having the readonly app not touch indexes, i.e. it shouldn't call sync_indexes.


Yahoo! We can do much more in four hands :)


Depending on EBADF looks dangerous. What if something opens a file as we are shutting down and the file descriptor is reused?

It may be better to not bother with closing the pipe during shutdown. It matches the overall plan of not doing cleanup on shutdown that CoreCLR is on.


LGTM modulo comment.


@krishnaa110, I doubt I can be helpful in configuring the tool. I would recommend checking out `devel` branch and looking at our dev lab (`CloudFerry/devlab/README.md`), all the configuration gets generated with Vagrant scripts.


For spec-completeness, it seems like we should also add a `port` field, and implementers can make their own decisions about rejecting ports not in the set `{80, 443}`.


From https://users.rust-lang.org/t/to-string-vs-to-owned-for-string-literals/1441, it seems like ```to_owned()``` should be more efficient than ```to_string()``` when coming from &str, in which case I should move to using ```to_owned()``` everywhere.


@virgil : check this out! i've fixed few syntax errors due to wrong alignments in latest pull requests.

in addition now when one bind Tor2web to non standard ports (i use to while in testing) it can work without issues.


Yes, I think this is a good addition. It can also make TestMode in Boulder nicer, since it would just relax the port restrictions rather than hardcoding a specific test port.


Yeah the solution is not optimal, but the only way to do it for now without a huge refactoring.


There is a constant into SalesForce task use that constant


!! So this is why I kept getting 404s on lookup-user calls.


Use a Constant Class to put your constants


@SangeetaMitra Is this file being replaced w/ another? Is the relevant content already in another file? When we delete a file we need to 301 it so a user doesnt get a 404.


I think not, one minute is very little, troops use to attack take 30 minutes to get ready, the waste of maximum is two minutes per hour, unlikely that it will always have two minutes wasted.


Thank you.


Good! I also think this is healthy for the code organization.


Use constant from Constant Class


@jkotas I makes sense.
On the other hand the PAL initialization contains a lot of cleanup stuff that gets triggered when we fail in the middle of the initialization. So the pipe closing was one of those to stay in line with the other code.
I was actually wondering before if we should just get rid of all that cleanup code and be fully on the plan of no explicit cleanup.


That's right push comes before the tag. I will fix the example and tests soon !


should we return -EIO here? I'm not 100% sure


@troydai FYI - I just run build on my box an these entries were re-added.


It would be great if this was different for server and client.

The client could happily **remove**:
+		tls.TLS_RSA_WITH_AES_128_CBC_SHA,
+		tls.TLS_RSA_WITH_AES_256_CBC_SHA,

While we could support these for some time on the server for compatibility with older clients. Also splitting these would open the potential to make these configurable.


Add comments


or just "%s ...", __func__


Ping @NathanMcCauley @diogomonica


@saudaktr is CRM flowchart correct?
or any thing extra with gliffy app chrome...


Change println by logger


return rc;


Improve comment


why should we return -ENODATA here?


Delete semicolon


This change broke the documentation build. I have things like this in my user-config.jam:

    using gcc : : ccache g++ --std=c++11 ;

So now I'm getting errors like:

    sh: 1: ccache g++ --std=c++11: not found

It's easy enough to fix my configuration, but I imagine other people might have a similar problem.


I'd rather have it idempotent, unless there is a compelling reason


@gubatron this didn't work, because of this:

The libtorrent .a file is contained in `bin/gcc-4.8/`

But the `$GCC_VERSION=4.8.2`

so this line fails:
`  cp bin/gcc-$GCC_VERSION/release/boost-source/deprecated-functions-off/encryption-openssl/link-static/threading-multi/libtorrent.a $LIBTORRENT_LIBS/
`


improve comment


Hmm... this logic looks weird.


@rzarzynski note that the whole object iteration here and in the list case is pretty much identical, maybe can move it to an external function?


> so we're seeing Windows like we see SQLite. We'll support it, but it shouldn't be used in production. But also if it became too hard we'd drop it.

I'm fine with that.


I guess in this case it's unlikely this specialization will ever get hot.  I haven't really done specializations just for error cases before, but I was trying it out to see if it would make the fast path easier to read.  Generally the reason to do it is somewhere along the exception construction phase, a StringBuilder call is made with unbounded recursion and that messes up the PE process.

I think PE got a bit smarter in Truffle 0.7, but I believe we still need to guard against that.  @chrisseaton Is that still accurate?


Thanks, was going to do a similar pull request! :+1:


Can you bump this to the first build of 1.6.1 since it's a bug fix?


typo: "raise security error if http test attempts to connect to *local* ip"


Eh we no mames! ocmo le pones mother cukejfn as pendejo! e-e


Yay, thanks @Fishrock123


please also note: there are several MD not having a counterpart and they are done as well and pass tests. Please the the last post in https://github.com/spencermountain/nlp_compromise/issues/42


Ok, added the extra case in b32ec79 for this block and the one below.


Can we have a new function 'prepareDefinition' that wraps line 47 to 69?


aa


test


Please don't comment on individual commits, and :+1: and 'thank you's just make it harder to follow the code review


Stupid question: Why updating the `Gemfile.lock` at all when we are not doing a `bundle update`,  `bundle install`,  `bundle lock`... ?

Edit: Sorry!


@rzarzynski oh, I see now


Creation of project


ok ta chido :+1:


Creates package to all files changes?? I should said create a package empty?


1670-HeatPumpWaterHeater (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

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


this is the wrong fix


Yeah... I just feel like it makes sense for them to provide a programmatically accessible list of supported countries/currencies, right? Like what if we had to include a dropdown where you selected your country or currency and we only wanted to include ones supported by Stripe?

I guess the counter-argument is that your application shouldn't be permitted to add support for new countries without developer intervention.


I will change the format to yyyy-MM-dd so that log entries can be sorted. I hope that's OK for you, otherwise please tell me.


95652938-jsoncpp (jasondegraw) - Win64-Windows-7-VisualStudio-12: 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)


improve comment


fixes #55


@rzarzynski the question is whether it is possible to have multiple entries that share the same key_ext? Should that be an option?


another question is if we *should* add the "biggest" country abbreviations?
if so: is ca also Canada?


удалил то, что выше


@rzarzynski it looks very good!


Use string template to concatenate


OK.


@erg, with gcc you want -std=gnu++11


@virgil: this fixing was missing so that i think that the subdomain patch never worked. am I right?

now i've retested it with:
https://antani.bitmailendavkbec.antani.gov:8443
as well as with:
https://fuochi.fatui.bitmailendavkbec.antani.gov:8443


Not really, you'll have to convince me that I want gnu extensions enabled :P

On Fri, Jun 5, 2015 at 11:20 AM, Björn Lindqvist <notifications@github.com>
wrote:

> @erg <https://github.com/erg>, with gcc you want -std=gnu++11
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/slavapestov/factor/commit/1d58f827a2e7065ff1b6165d88b32b8440612bea#commitcomment-11546555>
> .
>


This line is unnecessary because the next line is using getInvalidFodlers function


Hello, thanks for the great work!! I love this feature!

Just curious if you have any timeline for a release tag including this feature?


Ca serait mieux de pas commiter le projet par contre, pour pas se binder a un IDE


@wuxingyi I think it's overall fine. My issue is mainly with the style that doesn't conform to the rest of the code. I'd get rid of the camel casing (isMultipartObj -> is_multipart_obj), and also add a very detailed comment that explains the whole race and why we do it the way we do it.


I think the rapdijson headers should be moved into a different directory, outside serialization/.  We should have a directory named 'external' (or 'thirdparty' or maybe something else?) to collect such external dependencies.  This would (1) help with maintenance and (2) allow cleaner code when multiple modules rely on the same external library.  As an example of the latter, when we will merge Chrono::Vehicle into Chrono, it would be awkward to include headers for rapidjson from under serialization/ since the use of JSON in Chrono::Vehicle is not related to serialization.


no new line before opening scope


are we expecting this to be something other than a string?


oh, damn I changed that locally. sorry. I'll push push my change to async-helpers...


thanks for fixing that


@wuxingyi also change in the comment 'fix:' to 'Fixes:'


no... im talking about any creature ... death grip .. doesnt pull anybody.. only makes them aggro..
I just want to see if someone else can confirm that this path works ...like on kvipkas vid.


95652938-jsoncpp (jasondegraw) - i386-Windows-7-VisualStudio-12: 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)


`funciton` > `function`


yeah, maybe. if there are some common abbreviations feel free to add them.


Heh, I saw this the other day and was quite confused :)


Here is my codepen of my issue http://codepen.io/anon/pen/doWYGd


Seems we don't anymore, will remove. IIRC I got errors when flushing during startup, I think we could flush before Keyspace.initialized was set to true, but this was a long time ago so things seem to have changed


Tu pourrais tirer dessus avec requirejs a place, idealement dans vueEmploye.js, en fait, on pourrait renommer ca, ca va devenir le fichier principal de l'application... m'enfin je pense que tu vois le genre :)


@wuxingyi this part is not clear to me. If we got EEXIST, the multipart object should not be part of the written_objs list. If not that's the bug.


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (646 of 646 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)


@mindfreakthemon It has been released in [v1.3.13](https://github.com/Parallels/vagrant-parallels/releases/tag/v1.3.13)


my hero


Maybe hydra.nixos.org managed to fail this as well. I just wanted to try a search of the message...


improve variable name maybe use folders


@mgenev Jumping in late here, but I think there is a much more elegant solution than firing actions that bubble up automatically. That kind of behavior tends to create inadvertent and hard-to-reason about coupling, _especially_ for developers new to the framework.

In my experience, anywhere that I would want a "global" action, I simply pull that behavior out into a service that gets injected into the component that needs it. Now you have an effectively "global" place to put behavior that components and controllers have access to, but it is much more explicit in your code about where that action eventually winds up.


that makes a lot of sense...


GNU extensions are like vendor lockin, very M$FT-1990s-esque. GNU extensions are the reason vanilla ``clang`` can't compile the linux kernel yet.

Only kind of kidding...


We usually report an error when the test passed to avocado is not present in the file system, “file not found”. So is there any other case to turnTestNotFoundError back?


ahhh ... thanks!


Isn't `pluck` just `_.map(obj, 'prop')` now? Do we even need the alias in "more"? @jridgewell


Use foldersName


improve comment


Isn't this code really biased towards iOS devices?


It's the same as `where` and `findWhere`? We could remove it, but that same argument would apply for remove the other two.


Since you do all these sanity checks twice in `create_dessert` and `edit_dessert`, you could put them into a common function if you wanted, to save typing and make it more flexible for future :)


I can pull this out into a separate PR if you'd like.


improve function name


Right, they're functionally equivalent, it'd just be a semantics question, right?


See https://github.com/nengo/nengo_gui/pull/339#issuecomment-109389069

The "more thinking than necessary" was because the toolbar used to just cover the main area, but with 114292c866d20b78a56e92a018bf65fc47e53e59 it has changed to only occupy the area which is actually visible.


"Fix out of bounds exception in lower and upper when the string contains xxx codepoint"


The encodings have to match for this to be correct. You probably want a fallback that iterates through the code points and compares them individually.


For multidevice support you should fix the global
static libusb_context* g_libusb_context = NULL;
it shall be not global but inside struct hackrf_device
You can also check airspy host source code: https://github.com/airspy/host/blob/master/libairspy/src/airspy.c
I have refactored all (with Youssef) and it support well multiple devices (tested with more than 3 devices at time)
Also performance of airspy host have been heavily improved in actual trunk, feel free to reuse anything anyway it was based on hackrf host at start.


H for Honda? HMMMM


What happens if you enter an ID that doesn't correspond to a dessert?


Why wouldn't the product still exist? This just seems odd because before this change we never had to check for this. It seems like there must be some other issue with the code that requires us to do this additional check.


It feels like your search view is trying to search and edit. Is that intentional? It's usually cleaner if one view does one thing.


Added flags to bootloader hanshake with 24-bits (check for off-by-one errors) for SBP protocol version, the rest reserved. Added 24-bits to heartbeat message (check for off-by-one errors) for SBP protocol version.


:puke:


It's might be a good idea to read them from a config file instead of hard code them, it will be really helpful for the editors.


:puke: again :D


CSS3 Transitions not supported by: IE (6,7,8,9)


Great work getting through all this. Super awesome! You might want to see if you could add some python tests for the edit method, so that you don't have to test it manually in the browser all the time.

![yay](http://www.reactiongifs.com/r/hdfi.gif)


mdp/rotp/issues/32 changed how otps are verified. I didn't get a chance to test it with v2+ right away but I'll try to do so soon.


Because we now additionally delete the product when we delete the element, but explicitly delete the product first. When the product fixture is destroyed it will attempt to delete the product again.


And you want it contained to the content box?

On Fri, Jun 5, 2015 at 11:27 AM, syam-ardas <notifications@github.com>
wrote:

> Here is my codepen of my issue http://codepen.io/anon/pen/doWYGd
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/material/commit/7f355f6d787520cc6a047f46067971b25d79ee73#commitcomment-11546668>
> .
>


This caused artifact on the screen for older Intel chip sets and was already incorporated upstream.


haha ok :)


Done


Hi,

not yet - i checked to ensure that it was actually using the rate control
logic correctly. Yes, it's using the rate control logic correctly now
(wlandebug +rate). I'll try iperf tonight at home.

The current way that the rate stuff is setup/used itself is wrong though -
the rate code needs to be setup when the node is created, not just when the
assocation occurs. There are other bugs there too to sift through. (It also
uses a static antenna configuration. Sigh.) I don't even know if frame
protection/rtscts is even being correctly used atm.

-adrian


braintree calls it a plan so I was just trying to log that part https://developers.braintreepayments.com/javascript+python/reference/request/subscription/create If someone is hitting this error we'll want to know the plan ID.


Is this correct? Where does connection come from?


Aha. I see you changed the `linux` define to `__linux__`. Hopefully that compiles without having to enable the gnu extensions.


It's not exactly equivalent. Both methods avoid `iteratee` and invoke `property` and `matches`, respectively, thus coercing some values and applying slightly different behaviour.

E.g.

```js
function x() {}
x.y = 1;

_.findWhere(obj, x)
// vs
_.find(obj, x);
```


Would you consider reverting on v4.5.4dev and working on a branch until the tests pass? I'm seeing some failures (which is ok) but I'm worried I'll miss stuff I've broken


Agreed, but, a) you wouldn't ever really use a functor like that, and b) you could always use the functional form if you really had a bug. I'm fine keeping them in `more`, I'm just saying keep the core small and idiomatic.


improve function name


I would definetly like to drop them in underscore v2, lodash v4 and allow
the user to call `matches`/`property` if they really need to

On Fri, Jun 5, 2015 at 2:42 PM, Adam Krebs <notifications@github.com> wrote:

> Agreed, but, a) you wouldn't ever really use a functor like that, and b)
> you could always use the functional form if you really had a bug. I'm fine
> keeping them in more, I'm just saying keep the core small and idiomatic.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/underdash/underdash/commit/54873f251397305e5f24a53662d451f664dd671f#commitcomment-11546939>
> .
>


This breaks if `host.getCanonicalFileName` needs a `this` reference. See https://github.com/Microsoft/TypeScript/issues/1545


CSS outline not supported by: IE (6,7)


"TestLoader"


@Robinlovelace good start but that isn't html, was trying to work out why it looked messed up... :wink:


Did you removed implementations of these functions for everything other than OpenGL + SDL platform? I think you should add windows only code back, since it is better than nothing when using non-OpenGL BaseManager. Moving implementations from tools code is ok.


yes please fix ASAP!
this is causing mayhem with my numerous browsers


Can you move it into linux/surface.rs then? Otherwise, this looks good.


ui-grid cells arranged so need to be "overflow:hidden"-ed. So if dropdown will be placed in cell than it will be trimmed. But if dropdown will be placed outside of grid it will be unbound from his input element (when input scrolled up, dropdown stay alone). I would have opportunity to place dropdown at row element (which not overlow:hidden) of ui-grid, but not cell element or document body


change filesName by files


Sure, but I think the log should be going in payments-service, not solitude which is why I didn't add it. You've got a detailed error back to payments-service and it seems wierd to have a log that doesn't match the line above.


@Makman2 : Scrutinizer says "unnecessary bracket after assert" :smiley:


We could also define a file with log messages / a const string, so that we easily can change them


As far as I can see, this abstract class is implemented nowhere. Did we overlook something here?


Just curious why 1.4 would be supported when 1.5 isn't?


Ah, so it sounds like the root issue is the scrolling, which is being
addressed.

On Fri, Jun 5, 2015 at 11:46 AM, syam-ardas <notifications@github.com>
wrote:

> ui-grid cells arranged so need to be "overflow:hidden"-ed. So if dropdown
> will be placed in cell than it will be trimmed. But if dropdown will be
> placed outside of grid it will be unbound from his input element (when
> input scrolled up, dropdown stay alone). I would have opportunity to place
> dropdown at row element (which not overlow:hidden) of ui-grid, but not cell
> element or document body
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/material/commit/7f355f6d787520cc6a047f46067971b25d79ee73#commitcomment-11547010>
> .
>


zkus to zvalidovat teď, mělo by to být ok


> I'm fine keeping them in more, I'm just saying keep the core small and idiomatic.

I'll merge if you submit a PR. :smile:


Профосмотры


It isn't necessary add another return


OK.


I think you are doing this on the wrong queue. Shouldn't it be the commonPropertiesQueue?


If I'm not wrong it can't be moved there, it must be in the crate file.


self.cn


Improve variable name "auxFile"


So have `=={T}(a::String{T}, b::String{T})`? The fallback would be to compare different encodings?


I'm a bit confused here, your are setting `avocado_params` to params and the PR cites `avocado_mux`.


Whoops! You're of course right. That's why we have code reviews… :)


Don't merge yet. Need to fix crash on empty subreddits. :)


Perhaps I’m stupid, but where’s the `i++`?


This seems to be crashing GeckoView during initialization. Since context getter is being set later.

    java.lang.NullPointerException
            at org.mozilla.gecko.GeckoAppShell.getContext(GeckoAppShell.java:2067)
            at org.mozilla.gecko.GeckoAppShell.getDpi(GeckoAppShell.java:1365)
            at org.mozilla.gecko.gfx.JavaPanZoomController.<clinit>(JavaPanZoomController.java:61)
            at org.mozilla.gecko.gfx.PanZoomController$Factory.create(PanZoomController.java:29)
            at org.mozilla.gecko.gfx.GeckoLayerClient.<init>(GeckoLayerClient.java:138)
            at org.mozilla.gecko.gfx.LayerView.initializeView(LayerView.java:125)
            at org.mozilla.gecko.GeckoView.init(GeckoView.java:125)
            at org.mozilla.gecko.GeckoView.<init>(GeckoView.java:118)
            at java.lang.reflect.Constructor.constructNative(Native Method)
            at java.lang.reflect.Constructor.newInstance(Constructor.java:417)
            at android.view.LayoutInflater.createView(LayoutInflater.java:587)
            at android.view.LayoutInflater.createViewFromTag(LayoutInflater.java:687)
            at android.view.LayoutInflater.rInflate(LayoutInflater.java:746)
            at android.view.LayoutInflater.inflate(LayoutInflater.java:489)
            at android.view.LayoutInflater.inflate(LayoutInflater.java:396)


It finds the date the user joined, converts that into unix time. Does a post search with that and if a post is found, gives you the link, otherwise search again with a time that is one day after the day you joined.


@rodrigopavezi technically no, but good point. I will add checks for that as suggested. Thanks!


Hooray! We finally got the code on Github. I can't believe Git is that finicky sometimes. Makes me want to rip my hair out... At any rate, now I can comment inline and show you exactly what I'm looking at.

So, let's look at line 18 first. Can you tell me why you indented this line?


I'd recommend throwing, or at least logging any error that the dataFunction throws. Otherwise this will end up causing nightmares for someone.


@ht-malama did you properly test it before pushing it?


Use addAll


forgot candidateID in the index...


I had to add these or else I got ambiguities elsewhere in the code when comparing `StaticString`.


I think you should call fromDb inside the `save` method.


NICE


@hthakur034 yes. I did.


What problem do you try to solve? Is it related to the default value override?


I think it's an erroneous deprecation warning that will be removed in future Rails versions.

> On Jun 5, 2015, at 19:54, Willian Fernandes <notifications@github.com> wrote:
>
> Hey @olivierlacan, I'm getting this same deprecation warning and couldn't find yet a way to silence it. Did you?
> I found your build on Travis with the same warning message.
>
> —
> Reply to this email directly or view it on GitHub.
>


Qu'est-ce que c'est 'peb'?


Wouldn't this be better using `chrome.storage.sync.getBytesInUse`? I'd assume that the object wrappers take some space that counts towards the quota. Equivalent using that:

```coffee
  chrome.storage.sync.get null, (items) ->
    for key of items
      chrome.storage.sync.getBytesInUse key, do (key) -> (bytesInUse) ->
        console.log key, bytesInUse
    chrome.storage.sync.getBytesInUse null, (bytesInUse) ->
      console.log "total", bytesInUse
```

It also might be better to have this as a function, so we can call it as we need the data instead of doing it blindly every startup.


Could you use GNU/Linux instead of Linux?
http://www.gnu.org/gnu/linux-and-gnu.en.html


What do you want me to do here... just not call it a call back?


=> GNU/Linux


Done.


There is catch-all logging in the service for any client 4xx but it's not as low level as this. We can just wait until we have to debug something and see if we're missing any data.


The replace call will miss `<html>`


Here you can just write `pass`.


Missing two blank lines between class defs.


It actually seems like this PR has been merged already (though in slightly different form)?


What about setting better function name? E.g. askForName()


I believe that this shows up in user error messages. We should probably make this conditional on `cx.verbose` at least.


What kovak meant bu not using enum, is that in python there's no benefit to using a enum that map to ints. Instead, you use the actual text. I.e. this should be `if self.inkc.mode == 'draw':`, so you can get rid of the enum entirely.


similarly here


Should this commit be reverted? Anyone have context?


Remove this extra line.


This comment is not needed.


that disables the MuSQL server... is it possible to run them both?


Should be fixed in d2adb82.


Why is this method required?


It isn't an Integration test because it doesn't deploying code


Checked.


Gracias me lo apunto!


Right. The same string looks different in a different encoding in memory, so you can use `memcmp`.


yep


I think `asis` is not needed since http://blog.rapporter.net/2014/09/pander-tables-inside-of-knitr.html?q=asis


Review that parameter aren't necesary


I don't know this this comment syntax will work in css


Normally, params to init are not documented in `__init__` but in the class definition. See e.g. https://github.com/kivy/kivy/blob/master/kivy/core/text/__init__.py#L65.


Yes we want to not compile exceptional code paths. If you're going to throw an exception, you might as well transfer as well, and if we don't throw then this code path doesn't need to exist. I think it has a knock-on effect where all the things that could happen in the exception code path - like objects escaping - can't happen, which simplifies optimisation of the non-exceptional path.

I think we still need them with the current PE.

In the future we might work out that some exceptions need to work on the fast path, but we aren't there yet.


sure I can(:


I'd take a branch for that. I'd wait til 4.4/external-storage lands in master (which should be soon, since it has passed internal review) and branch off that.


Remove the blank line above, and typically you don't start the text on the next line, but rather on the same line.


looks good


If and else cases differ only by the alert string "Welcom, welcome..." vs "Again you..". Try setting only this var - the rest code should not be repeated. Try to reuse the code.


@zephik thanks my friend :)


some mock are't needed


some mocks aren't needed


i can do that easily


Thanks for the reminder, forgot to accept the repo invite!

Side note: we're only discussing what methods should make it in or be left out at the moment. What about their signatures? Have we decided how we want to tackle overloaded methods yet?


You don't seem to be actually using `shortstraw_const` anywhere. Similarly, the class attributes e.g. `points`, `color` need to be declared as E.g. a ListProperty, ObjectProperty etc and then fully documented. See how it's done e.g. https://github.com/kivy/kivy/blob/master/kivy/uix/widget.py#L830.


This note and the line above it are no longer true / relevant now that you published the gem.


This should probably inherit from EventDispatcher so you can make the attributes I mentioned below kivy properties.


We should talk about this on IRC.


some random stylistic feedback:

- (def some-fn-name (fn args ...)) can also be written as simply (defn some-fn-name args ...)
- deeply nested calls can often be re-written using the -> or ->> macro to re-arrange forms in a more intuitive manner
- there's also a shorthand available for function literals that can be used to shorten the filtering function somewhat

putting it all together, this is one way to re-write the function:

```
(defn sum-of-multiples-of-3-or-5-below [n]
  (->> (range n)
       (filter #(or (zero? (mod % 5))
                    (zero? (mod % 3))))
       (reduce +)))
```


Not really. I ended up fixing the need for this some iterations back.


The way we register events in kivy is to do `__events__ = ('on_stroke_added', 'on_stroke_removed')`, like https://github.com/kivy/kivy/blob/master/kivy/uix/widget.py#L250.


This color needs to be configurable, so it should be a property.


@rvallespin this is more legible this way `rgba(white, 0.2)` :)


Although strictly speaking, we still have `avocado.test.MissingTest` needing it.


Please reconsider this.

1. It is not good practice to run the entire program with more privileges than it really requires.
2. Once N++ has restarted as admin, you can no longer drag and drop files.

The idiomatic way to solve this is to launch a separate process as admin which deals only with saving, while the original N++ process keeps going as before.

I don't know if you remember, but I submitted a patch for this a couple of years ago but it was rejected. I've kept it up to date since I consider this feature absolutely essential:

https://github.com/andreas-jonsson/NotepadPlusPlus-Privileged


Awesome!!! It's getting better and better!


I was thinking about reducing code, but as we have 2 buttons on confirm dialog box have no idea how to do that...but will try hard(:


that's fine, but shouldn't it be in a branch until it's ready?
Just curious as now you get:
* A huge commented HTML block when you view source
* A dependency on a file that is never used
* Duplicate files leading to some confusion if anyone starts digging around

Thanks.


Fixed:
https://github.com/ccd0/4chan-x/commit/d7b84817b0bb779cd7881f6e4f4cb8641beb1f00


I'd argue we can review more thoroughly the need for `avocado.test.MissingTest` to still exist, but for now, I'd argue it's best to simply leave it as is and just add the missing definition.


Good catch!


Need three slashes, like `file:///path/to/loca/repo`.  At first I tried two and it looked for `/to/local/repo` in the filesystem.


@musamusa if a function that returns a promise is called inside $state resolve block and it through error, the state transition will not complete so in this case if `return appConfigService.getCurrentAppConfig();` or `return utilizationService.all();` fails, immunization UI will not be show, hence the need to catch errors and return empty array or object or show appropriate error message.

for instance, if i have a view that shows list of Vaccines  i.e an array of vaccines and i need to resolve the list at state transition resolve block, i need to add `.catch(function(error){
    here i can decide to show alert so user will know why the state transition failed or
   return an empty array, hence in the view the table will be empty or an alert label will be shown that says "No vaccine is available" etc.
})`


this is another one where ->> can help untangle the nested calls.  a (let) form can also help clarify things a little:

```
(let [fibs (fn f [a b]
             (lazy-seq (cons a (f b (+ a b)))))]
  (->> (fibs 0 1)
       (take-while #(< % 4000000))
       (filter even?)
       (reduce +)))
```


You could replace it with

```js
entryData = dataFunction ? dataFunction(entryData) : entryData
```
That handles the case dataFunction is false and lets it throw if it has an error. (I think v8 might also have an easier time optimizing it because it won't be throwing and swallowing an error in the normal case.)


i changed the (partial) to #(< % 4000000) so we can use < instead of >, but it's a subjective call.


Finally, seems the tests are failing but we can still add test, lots of functions are created in this PR services and controllers, there is still no Unit test for almost all the new Controller and Service functions.


Because 1.4 is a long-term-support (LTS) version, and 1.5 is not, and this change is from several months before 1.8 came out.


Basically here I'm injecting avocado specific params into the params of a non native test loader plugin (vt), so that people running virt-tests can use 'Frankenstein Mode' -> that is using avocado multiplexed params from inside virt-tests. It was the way I found to support a feature requested by @adereis.


Oh I understand it now. Not exactly. It was never in argparse, so instead removing it from cmdline this only uses the only location, where `disable_restore_image*` is set. So it's a bugfix, not change of the behavior.


LGTM


yes, I wanted to make it easier to revert the patch in case something is wrong...


serviceName always exists. Feel free to ProtocolError the connection if it does not exist.


oh yeah, and zero? is a convenience function for (= 0 ...).


@erikh, Overall I think this is the right structure!

As this is an early implementation, I mostly have a comment around using netmaster/netplugin for posting the configuration states. I realize you might already be aware of this but thought I will share so we are on same page :blush:.

Right now you are managing the ovs specific configuration state (we usually refer it as **concrete or network-function's** configuration) directly as part of the Create/Delete requests. We actually want to create/delete the configuration state that is understood by the netmaster instead (we usually refer it as **logical** configuration). This can be achieved by invoking netdcli, as netmaster is not a daemon yet. This will ensure that we make use of all the logic built around resource management (like allocation of IPs, vlans etc) for networks and endpoints and also to correctly trigger the Netplugin. For instance checkout one implementation of (now outdated version) `driverapi.CreateEndpoint()` at https://github.com/contiv/netplugin/blob/master/mgmtfn/pslibnet/libnetdriver.go#L76.

Following is one way I think we can hook up libnetwork events with netmaster configuration. But there might be other ways as well so please feel free to suggest/correct if I missed something. **Note** that some changes might be needed in a way we invoke netmaster once we move to a multi-host environment, but we can take them up later once we have basic integration working.
- **Driver Init (or main())**:
Here we can possibly push the logical config pertaining to the initializing the driver itself. For instance for vlan-based networks we need to push this configuration today to add host's data-path netdevice to the ovs bridge:
```
# netdcli -add-cfg - <<EOF
{
"Hosts" : [{
        "Name"                      : "host1",
        "Intf"                      : "eth1"
    }]
}
EOF
```
- **Network Create**:
Here we can push the network's logical configuration (derived from the labels/options passed by libnetwork). Note that network and tenant names shall come from labels passed by libnetwork to the driver. I am also assuming that Tenant was created before hand with it's logical configuration pushed separately by calling 'netdcli'. So equivalent netcdli calls from shell it will be:
```
# netdcli -add-cfg - <<EOF
 {
   "Tenants" : [{
        "Name"                      : "tenant-one",
        "Networks"  : [
        {
            "Name"                  : "orange"
        }]
   }]
}
EOF
```

- **Endpoint Create/Join**:
Here we can push the endpoint's logical configuration, mostly it will be container's name or uuid. And return back the interface name, IP etc as assigned by the netplugin/netmaster by reading it using netdcli again. So equivalent netcdli calls from shell it will be:
```
# netdcli -add-cfg - <<EOF
{
    "Tenants" : [{
        "Name"                      : "tenant-one",
        "Networks"  : [
        {
            "Name"                  : "orange",
            "Endpoints" : [{
                "Container"         : "myContainer1",
                "Host"              : "host1"
            }]
       }]
    }]
}
EOF

#netdcli -oper get -construct endpoint orange-myContainer1
```

For **delete** events netdcli can be invoked with -del-cfg flag

/cc @shaleman @jainvipin


¡Cierto! Se me pasó current_user. En cuanto a build vs new... ¿Tiene alguna ventaja? ¿o son sólo alias?


Wait  10 to 7 Days.... :-)


Just as an FYI. @roncain I didn't remove the [Contributing.md](https://github.com/dotnet/wcf/blob/master/CONTRIBUTING.md) at the root since that is the target of the contibuting guideline url that shows up when someone creates a pull request.


Looks like a very good idea. Let me try this out first before merger.


Misspelled children in commit message


For anyone seeing this: despite the commit message, the preDispatch will not be removed.


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1162 of 1162 tests passed)

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


reack


Got it.


OK.


I think you should fix the typo in the commit message.


also breaks with `order`, haven't tried others.


FYI, fixed in a74465d


This name seems odd, it doesn't return a user, just a downcasted context.


If I override lag, I could make it an S3 generic, or I could make it an S4 generic, or I could make it not a generic.  I think having two different lag S3 generics is actually more confusing than the alternative.


it should have the number of the GCC in your system. Make sure you have the latest version.


This works great, except for that the scrolling still doesn't work on touchscreen devices. Any tips?


Good question @sadmac7000


I tried it out and discovered this too.  It's fixed in a later commit.  the `file:///` prefix isn't necessary and the repo can just be referenced by the path.  it tested this and everything worked.


@rpdubose We have to pull this article out. Please suggest how it can be done. Nathan had told me I can just delete it,so I did.


Seems like, with some creative patch ordering manipulation, you can drop the need to both add and remove these things as part of the same patchset.


What do you mean? The two getColorCameraParams and GetIrCameraParams is not part of this PR ? I can't see its merged in any form :) ? - but I'm not sure if it is finished either.


I agree, but I couldn't find a better name :+1:
Maybe the profile info should be decoupled from the authenticated context and simply added as a property in the authenticated context?


try to not use `className`, prefer [data-focus='component-name']


I do, but however the way libtorrent compiles, it leaves out the minor dot.


maybe make this TODO(zachmullen)


You don't use this?


One of the chapters I downloaded contained a PDF format as well. This was last year, so I don't remember the exact one.


use lodash `keys` because Object.keys is not transpiled


Do you really want __raw_writel()?  Something tells me this will fail on BE systems.  Also, do you need a barrier?


if I do this raise and exception:

AttributeError: 'StrokeCanvasFloat' object has no attribute '_trigger_layout'


I love that everything else gets cleaned up, but I don't like that the resulting user experience requires a downcast. Using extensions to mask the downcast isn't much better because you'll have the Facebook and Twitter extensions show up on the same object.

When it's a trade off of pain for us vs pain for consumers, the pain should be on ours side. Stick with the generics.


const struct zynq_fpga_priv *priv ?


Ah, excellent. Just double-checking. Thanks!


Two validate that two points are equal based on values.


Okay, let's forget this approach then :smile:
I'll clean the PR and generalize the design we adopted to Twitter.


I have trouble with this. We have DNSName so we are sure what type of data we are dealing with, and this method meddles with that guarantee. I would rather see a ReversedDNSName, or a method that just returns the string we need for NSEC ordernames, etc.


Hello Brady,

1 - Not exhaustively. I can engage in a more complete testing of the code if it will help.

2, 3 - Thanks, I will update the code and post to forum

Also...

I have not made it clear about the missing appointments, recurrtype=2 appointments ("last Friday", "2nd Monday" etc) were not appearing in the report when the pc_eventDate variable was set to a value less than the start date (from_date) for the report. The SQL query was faulty.


I am not sure using the testunithelper like this is the right thing to do here.
One of the problems is that we are only checking the unit attention once, and thus depend on that any target that sends multiple unit attentions for this event will send a bus reset as the very first event.

I think a better solution would be to switch to using
task = iscsi_testunitready_sync(...)
and then iterate over it until all async events have been consumed.


I was expecting the author of the commit to say anything before we proceed to any rollback, but he didn't say anything until now.
I might have some time at this weekend. So, is it unanimous that a rollback is required? Should I do that at the next days?


Was it, though?  You have no guarantee you did anything.


I also have trouble with this - DNSName holds absolute names, this makes it hold relative names without any way to tell from the outside.


now you show it, looks easy. :)


with tilt > 2.0
Completed 500 Internal Server Error in 14ms (ActiveRecord: 0.0ms)

ActionView::Template::Error (Constructor Generator must return callable object):
  temple (0.7.5) lib/temple/engine.rb:62:in `block in call_chain'
  temple (0.7.5) lib/temple/engine.rb:60:in `map'
  temple (0.7.5) lib/temple/engine.rb:60:in `call_chain'
  temple (0.7.5) lib/temple/engine.rb:50:in `call'
  temple (0.7.5) lib/temple/mixins/template.rb:10:in `compile'
  temple (0.7.5) lib/temple/templates/rails.rb:8:in `call'


in #712, that’s where :)


Nice factoring.


I know there is an accessor that will do this with a timeout that was recently added to the kernel, but I can't remember exactly what it was called...


Interesting, are you going to make the game like we talked about?


Yeah, you just need to know where to look


Good idea!


Nice!


a `n` is missing. ;-)


There is still some issues remaining. It seems much more stabler than previous commits
https://wrench.internalx.com/Wrench/ViewLane.aspx?lane_id=1868&host_id=233&revision_id=555513

Check the test-iOS step


Now, the comment here is fair, and I've changed the commit entirely to not make reference to `mux` anymore. Sorry I forgot that.


I think I'll go one better and have `example.py` prompt the user for username and password and store it in credentials.json if it doesn't exist.


Indeed, feel free to fix any typos you find :-)


I indented that line 4 spaces as it's best practice to make it easier to read and help identify which code each brace completes. For this particular one, I used the example set in the notes on Bloc as a guide.


I mean instead of the __init__function just write `pass`.


It seems it needs native English speaker review.


Might be useful to modify the FPGA manager to not force the use of init()?


@abingham Sorry that it took so long to reply but I didn't have much time to work on `emacs-ycmd`.

I couldn't find a way either to detect when a buffer gets first focused. There is the hook `window-configuration-change-hook` which gets called when switching buffer. When using a timer to defer the parsing request we can achieve that when loading buffers with `desktop.el` we only parse when a buffer is focused for a short time.
See https://github.com/ptrv/emacs-ycmd/commit/72596752f3d7efa9a7db7b86cc9686bf6a0ed095


Did you mean `PaginationAdvanced` ?


Should be "wildcard", not "wildcart".


Wasn't there an implementation of this prior to the FPGA manager?  xdevcfg?  Please provide proper attribution.


Group/"module" names are now translation keys, rather than translated strings.


間違えてあげてしまった


Hopefully this convinces more FOSS projects to :hankey: on SF


haha, maybe you can use `- export TZ=Asia/Shanghai` to correct the commit time to GTM+8.


Well, it doesn't matter ,2333


I am sorry that title was wrong. #273 should be modified to #373.


the latest version of glob is used in `grunt-legacy-file`, this was the only unit test where the order of expected results was affected by the version change.


whoops


Any ideas on my @TODO comment here?


然并卵

强行示范comment on commit功能


...我这是软件工程大作业


TODO: currently system updates only nearby existing gameobjects, npcs aren't updated. Its generally wrong - it is not depend on its type and every object should be updated correctly


その通りですね！
私、ここ数日立て込んでいて、直すのが来週以降になりそうなので、もし、手が空いていたらpull requestにしてもらえると助かります。手が空いていなくても私がじきに直すので大丈夫です。


Please could you make this function the same as https://github.com/GeoNet/mule-quake-client-felt/blob/master/src/main/resources/db.xml#L21

It's the one that's in prod and I would rather not change the DB :-)


Support for a light-weight PDF generation implementation has been something I've been looking for a while, with one of the requirements being it supports xsl-style-sheets. This seems to be the first one that comes close to fitting that bill!

I see the XSL support is brand new and likely not well-tested (hence it's lack of inclusion in the master branch) --- is there anything specific as to the implementation you need handled. I'd very happy to focus specifically on that feature. Can you give me an idea of your intended implementation?


我说呢。。Tags打得飞起。。
23333，赶紧匿了。。。


So the point I was going to make was that the OS is the one sending back filesystem events from a single schedule, which is very efficient. But turns out, unlike [FSEvents](https://developer.apple.com/library/mac/documentation/Darwin/Reference/FSEvents_Ref/) and the Windows API, [inotify doesn't work this way](http://www.quora.com/Inotify-monitoring-of-directories-is-not-recursive-Is-there-any-specific-reason-for-this-design-in-Linux-kernel). That's pretty terrible, and your point stands on Linux because of this.

Despite that, I'd still rather pull in every other part of your PR except this one, because it's still relatively complex and incomplete (meaning it's only going to get *more* complex).

If you rebase this commit out, I'm ready to pull it in (I can implement the `[directories]` one if you didn't already). We can discuss what the best approach to ignoring directories in a new issue / PR.

That seem ok?

(Thanks for your patients btw, I do appreciate it.)


开个pr 先merge掉 你先搞下那个submodule…现在不能auto merge我得手动diff


Looks good.


Any more thoughts on this one?


Yes


Yes, sorry what step fot stumber?


Fixed it :+1:


@jiangtiandao 的确没更新，传不上来T_T，不过这个后来测试发现这种方式会让汉化补丁额外增加2%~4%的性能损耗，还需要像办法


「おこだよ！」の間違いでは？？


Thanks Rosemary, everything looks cleaned up now


should be ==0


In line 310 is another "RebuildKeyspaceGraph" between

DeleteTablet(s)
DeleteShard

That one is also mentioned in the doc: https://github.com/youtube/vitess/blob/master/doc/HorizontalReshardingGuide.md#step-62-rebuild-the-serving-graph

Do we actually need that one?


Readme


rax retry tests


I want to specific both data type and value so should be === 0


Today my generic rss seems to be broken after this commit. But the test button wasn't complaining. I had to delete the indexer and add it again.


Hrm do you get the error consistently? sometimes the tests time out since its async only. and is this with or without the jquery version change?


Hi,
I have written a web socket application in spring boot. Now a GWT based web client is expected to connect with this server and receive messages as soon as server sends some specific messages. Can I use this gwt-stomp client for this purpose? Any simple example, how can I include and use it in my GWT application? Thanks in advance.


@Ahbee I get a compile-time error with this file at:

```c++
class NTOperation::public juce::UndoableAction {

c:\cplusplus\juce\modules\juce_icarus\utils/NTJuceUtils.h(6): error C2653: 'NTOperation': is not a class or namespace name

```


/cc @manasdk - just FYI.


great!


I fixed those Forgot to push, try now


yes sure, if you want to specific either value or data type, you can


채영 - 사라진 커밋들1


#ALtditemheader()
などは共通なので、分岐の範囲を狭くしておくとベターです。


My case I want to specify that's why I put like this


perfect


`listCollections` only work on 2.8+, you need to query the `system.namespaces` collection to support older versions.


pleas don't hurt me.


зачем это?


@chipotle rethinkdb/rethinkdb#4349 was just merged, so I think we're good now.


лучше это делать инициализацией


Sure. Ground is odd but has some interesting use cases.

It is possible with a series of overlapping ground level images to reconstruct plan view (_ordinary_) orthophotos. Part of what is intriguing about them is ridiculous resolution -- in our tests up to 0.25cm resolution with an ordinary point-and-shoot.

For parks, the use cases are documenting linear assets like trails and roads, pipes, buried wires, etc..

As far as humanitarian applications, I imagine things like a photo series of a sanitary canal. Post-process those images, and now you have not only visually interpretable documentation of that canal, but measurable output in the form of an orthophoto. Also, this is a sub-canopy, sub-building orthophoto, so it has additional interesting characteristics.

Ties back to Mapillary use cases as well.


@mhugent happy to report that this has indeed fixed the issue I reported. Thanks :)


Before I commit this, just want to get an idea of how extensively you've tested it.


по устройству полигон и полилиния -- очень похожи, не думали сделать так, чтобы подобные вещи делались одним кодом?


порядок методов умопомрачительный :)


Also for function libraries, best to not rely on variables for paths. Better to use dirname(__FILE__), for example:
``````
require_once(dirname(__FILE__) . "/patient.inc");


Also, would always use require_once and not use include_once.


ого! это что такое и зачем оно?


where to find export and import code, missing commit?


почему указатель и почему нет конст-а?


Внимание: я просил XList<Point>, а не XList<Point *>. Поправить сможете?


возвращаете копию, это плохо


бред какой-то


по месту нельзя было написать?


Is $provider_facility_filter being used in this report, though?


rax retry tests


фигурные скобки здесь только съедают место


For security stuff for appointments_report.php, place that in another branch and will do it later.


@smarterclayton do these conversions for ISImage and ISTag between v1beta3 and internal look correct, now that I've de-embedded Image?


нормальный компилятор здесь скажет ворнинг; тогда уж:
#endif //_XLIST.H


это для подключения математических констант, у меня площадь окружности считается через M_PI. По заданию, надо было разнести классы в Base и Shapes, и я забыл перетащить это перетащить в Shapes. Библиотека <string> не нужна вообще, тоже что-то с ней делал, но в дальнейшем изменил.


Interensante...jamás pensé que esto fuera de verdad XD


r+


Or, you know, you could use a u32 * and not have to do math :).


I don't understand the use of the word "Nominal" here?


C'mon I have my recall on H so I can ult/heal with B.


tests just began to fail due to [recent upstream changes](http://stefanchrist.eu/blog/2015_05_01/Choosing%20Diffie%20Hellman%20Key%20Exchange%20Parameters.xhtml)


Yeah, you should really be handling errors.  Hold some state you set in your priv object that you set in your isr, and check it here after you've been woken up.


taz probando???


You're clock probably doesn't actually have to be enabled until right before you kick the hardware.  No need to clock it while you're doing manipulations in host memory.