here we also need to check AdminUsers right, otherwise normal users will be able to see all the people's preferences


@devbharat Ok, I was a bit too early. With this tuning setup the quad does not do more than 0.1 m/s :-D
If you try to increase the velocity gains, see what happens. Do you have the same problem?


The private sets here aren't needed...I just removed them.


@vilchik-elena yes!


I talked with Paul in the chat room, once you add the constructor in Java, unless it is needed, you do not need to also add mutators(setters), as they become redundant. They will end up increasing the files size, and not really adding any benefit. So if it runs with just the constructor, no need to add the mutators.


Ok thanks.


That's what i thought. Are you just doing a merge of the branch or picking everything individually?


thanks! much clearer!


The idea is that `test-builders.sh` by default will clone the trees from your local tree by default. This can be changed to use git-export later.


I pick everything individually because we review via Gerrit. There's a CAF org that one of my colleagues maintains, which has the linux-msm updates too.

See here: https://github.com/codeaurora-unoffical/linux-msm


Discussed some improvements on this script with Ywata, basically to avoid duplicating default logic in each prepare script, he'll upload a new version.


After this, a single cure I, dia, haste, etc will steal hate off a player with multiple vokes/ws/dmg on a half dead mob.


The global value is already settable via BridgeConfiguration, but per cert is not yet.  #396 tracks this.


Quicksearching boolean field is wrong, I understand that and will restrict that in Quicksearch itself instead.


This math is f-ed up. We apply pitch correction to sinf part only for some reason. I guess I put the braces incorrectly when I was fixing it last time.


@cranmer I have started fiddling around with the approximate likelihood ratios. So far it is working well, though I have observed some strange things with tree-based methods as I feared. I'll continue with the decomposed ratios tomorrow.

With respect to the code, I think everything can be self-contained within a small module. So far I managed to only depend on numpy, scipy and scikit-learn.


Adds support for parsing relay response like:
1234ABCDEF: host relay.example.com[192.0.2.1] said: 450 4.1.8 <root@localhost.localdomain>: Sender address rejected: Domain not found (in reply to RCPT TO command)


I don't think you need this method at all. You have the variable `samlAuthorities` and all you have to do is

    Set<String> filteredExternalGroups = AuthorityUtils.authorityListToSet(samlAuthorities)


Thanks, my friend :) I have just did modifications for Ramdisk


Using consistent return types, and naming functions around that. I like it!


ain't nobody got time these days! haha. If you're ever tempted to
pair-program this project one of these days, message me!

On Tue, Oct 6, 2015 at 8:21 PM Forrest Townsend <notifications@github.com>
wrote:

> haha I just don't have time! [image: :disappointed:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Fofofosho/rps/commit/e13f61af6574b3d08c8090aa2477c3b29b1f2d88#commitcomment-13630084>
> .
>


The option that I specifically need is the "strictSSL" option - (there is a
bit a of a catch 22 going on with the corporate firewall, but don't worry I
am developing in a sandbox).

While the request package claims to look at environment variables for proxy
settings (which may or may not be set) it seems to only do this in its unit
testing:
https://github.com/request/request/blob/3733c61be8ce278d0ac0e9ca6405d9079aee8181/tests/test-proxy.js#L18.
It certainly does not do any checking for strictSSL:
https://github.com/request/request/blob/92b2ac03562c3e29d14946894183b629ede8cf04/request.js#L262

So basically, all of the npm configuration proxy settings are completely
ignored by the closure-util package via "get-down" and "request" for your
post-install update. You could mediate this by referencing the npm
configuration items - all of which become environment variables during a
package install (see
https://docs.npmjs.com/misc/config#environment-variables).

Please let me know if you'd like any assistance with this - I could provide
some example code if needed. I am very interested in seeing this working!

On Tue, Oct 6, 2015 at 5:27 PM Tim Schaub <notifications@github.com> wrote:

> @k00p <https://github.com/k00p> can you provide any detail on the options
> that you need?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openlayers/closure-util/commit/ee01478226e49c06548799d6355338a076b5f0cd#commitcomment-13628438>
> .
>


Hmm probably a good idea.


Good catch


Are these tests no longer needed?


I think this can be solved by moving out the _crlUriRelativePath out to the Initialize path, which lives in the IResource (which then is supposed to know about this). Let me toy with this a little... this was a relic left from when the entire URI was specified by the BridgeConfiguration, but I think I moved it one level too deep at the time to expedite this work.


@tamland fixed according to your recommendation: https://github.com/jimfcarroll/xbmc/commit/6a2c280dfd4fd414d7d685499fe69df9a5138505

I can submit a PR if you want.


Pour mes-aides, c'est en engagement difficile à tenir. Quand on implémente une nouvelle règle, savoir depuis quand exactement elle est appliquée et quelle était la règle précédente a un coût très significatif pour nous, sans améliorer notre produit.
Je l'avais fait pour des coefficients liés au TNS, et ça m'avait pris plusieurs heures à naviguer sur legifrance.

On peut éventuellement, en introduisant des modifications, les limiter à une période pour laquelle on est sûr qu'elles s'appliquent  (par exemple depuis 2014), mais on ne pourra pas chercher nous-mêmes, sans être mieux outillés, tout l'historique.

@MattiSG, tu en penses quoi ?


Do you know why the squished commits like this one stopped happening? https://github.com/DirtyUnicorns/android_kernel_moto_shamu/commit/5c5fa2ca5352b643301230e6b281edeee1635e2d

I don't push through gerrit so picking them all isn't really what I want to do. I think the squished ones stopped on DU around 70-71.


These tests are outdated and the analogous functionality should be tested on the querybuilder tests


I'm not sure about this. This whole commit assumes that the frame range is always open at the end, but as far as I know that's only the case for streaming data sources.


The test is tightly coupled with the implementation order here.
If the implemented code changes the order of validation to validate the command after validating money, this test will fail giving an error.
Test should not depend on each other and should not be aware of the implementation details.
Only the thing being tested is the thing that should be changed, so it is better to put the money amount greater so that the test does not fail for any other reason.


Stray comment


Same as above


why the cast here?


:goberserk:


@levackt I have renamed the test to explicitly indicate that it is an integration test.


I think here, after the pull step, we should run the build.sh script.  This will then make the slicer/slicerexecutionmodel image available, locally, if it was not available on DockerHub.  Or, it will update the image based on changes in the commit or branch for the test.  For the case for the slicer/slicerexecutionmodel is not available on DockerHub, the above line should be:

  docker pull slicer/slicerexecutionmodel || true


haha...I am not sure, will try soon. The controllers running by the way are **mc_pos_control_m**
**mc_att_control_m** , you increased the gains corresponding to these controllers, right ? (MPP or MPC, I am not sure.)


this kinda seems like an abuse of the setAndroidContext function, given it doesnt set the context?


@t4deu @joneslee85 Why these assertions were removed? Thanks :dancers:


Yes


@levackt I think that the best way to indicate that this integration test should be executed is to explicitly set `RUN_GPS_INTEGRATION_TESTS` system property (or environment variable) to `true` when running the test.


Shouldn't we keep the `class="codeph"` on the `<code>` element?


Fixed - FormatKeyValuePairsAsString


LGTM


Already pushed pretty much that here too https://github.com/tamland/xbmc/commits/fix_multiselect_leak :) And removed boost. We don't need that.

It compiles fine for me too so I don't know either. Lets create a pr and fire a jenkins build :)


Not enough


Thanks for the review!


Possibly should be staticly declared, rather than recreating the array every time


I wouldn't change the name for all artifacts... just the output image


I renamed these to be consistent with the Bridge (server) side. I wouldn't mind renaming them back, and i'll make a similar change to the Bridge server side.


What does this mean for building/installing on EL6?


names(userApiUrl.name);

I was trying to incorporate the json file into the `names ({ user : Karlo })`


Looks like all other layouts drophandler is missing them as well. Could you add them in the same PR?


Yes, I realized that. I modified the correct gains. Let me know if you get the same behaviour after you tried it.


No need for this comment as it does not make sense in the context of DragDropLayouts


Same for me


Tysm @Falkirks <3 :+1:


@jimfcarroll But `smart_ptr` is smarter, so go ahead an PR:)


Looks good


This changes only the name of the output image AFAICT


I stopped being lazy lol.


I actually fix title case on slides 17 thru 20. From this slide:
![image](https://cloud.githubusercontent.com/assets/12142968/10342065/347d70e0-6ccc-11e5-90f7-34874be31429.png)
to this slide
![image](https://cloud.githubusercontent.com/assets/12142968/10342091/5235d85c-6ccc-11e5-91fd-e83ee7793cd8.png)


This is a MAJOR breaking change.  This just broke our production build because we weren't regression testing on IE11 and only breaks internet explorer when you update to this version.  isSupported changed from a variable to a function which means if your doing Notify.isSupported, its always going to evaluate as true and cause exceptions when you try and send out notifications.   Thanks for a great library, but can you roll back this change and move it to a MAJOR version change like 1.4.0 otherwise the default bower rules will automatically upgrade to this version as its a patch version.


I would do:
```
var votes = $('.dubup, .dubdown, .add-to-playlist');
if (currentdj === currentAuth) {
	window.initialSize = $(".infoContainer")[0].style.width;
	votes.hide();
	$(".infoContainer")[0].style.width = parseInt($(".infoContainer")[0].style.width) + votes.width()*3 + "px";
} else {
	votes.show();
	$(".infoContainer")[0].style.width = window.initialSize;
}
```
It is still perfectible (like avoiding using global variables..).
Note that hiding element don't prevent users to click on them with a bit of code..


Changed to *KeyName


What are these files for?


No, it doesn't change the API at all.


The `name` is only used [here]( https://github.com/rhcarvalho/origin/commit/40618be395c6bce2d020bdde2df184f24f1eeb8e#diff-f26900778a8353c1e3b16d70b2dce230R57).


Looks the same as mine except I consolidated the typemaps for smart pointers since they are all the same and I apparently account for pre C++11 compilers. It's up to you how you want to proceed.


Next time when you improve an existing method make it another function unless its a bug fix. That gives us more flexibility. Don't forget to update the documentation part in that case.
:)


Unfortunately this cleanup did not yet fix the uninitialized desc pointer nor the infinite loop.


It's great that you turned this what used to be ugly temporary class in a well refactored and well readable official one.


"開放"は"解放"ですね。


haha. Well, shit then. They were so much quicker to grab when squished.


Mid term sample code


Unfortunately it is not, Ahmet changed all the dataSources to retrun [0,1) as an empty interval... Which does not match what is happening in Fivox... We have to find an agreement here..!


Nice job cleaning up the code.


Cool to see how you've altered this to fit in with the rest of the project! I wouldn't have known to use a compounder like this.

However, `createCompounder` runs `deburr` on the string before sending it to `lowerCase`, which implicitly drops non-ascii strings.

```js
> var l = require('./lodash');
> l.lowerCase('THIS IS A TĒŠT');
'this is a t t'
> 'THIS IS A TĒŠT'.toLowerCase();
'this is a tēšt'
```


The max length would technically have to be v2.RepositoryNameTotalLengthMax - len(namespace)-1 which I guess we don't have here.


floorのとこ、別にdiv 10をしなくても文字列の最後の1文字削って数値に戻すとかでもよかったか？


D'accord mais il ne faut pas nuire aux autres. Et vous avez plusieurs niveaux de coopération:
  - tu cherches un temps limité à quelle date ta modification a été ajoutée dans la législation
  - si tu ne la trouves pas, tu ne fais la modification que sur la période dont tu es sûr (et tu l'indiques et l'annonce aux autres en ouvrant un ticket)
  - tu fais ta propre réforme

Est-ce que cela vous convient ?


Adds support for parsing relay response like:
1234ABCDEF: host relay.example.com[192.0.2.1] said: 450 4.1.8 root@localhost.localdomain: Sender address rejected: Domain not found (in reply to RCPT TO command)


nope have checked them before :)


@glima

To give some context: each builder currently have two different clones, a HOST clone (compile the host generator executable) and a TARGET clone (library code, JSON descriptions). It was also the case for many builders that the TARGET clone was different because added patches not upstreamable yet.

Whoever going to work on these further on, feel free to ping me to get help understanding how it works or details of the small OS build systems.


Sorry, you're right, I thought it was used for the other objects


Appoligies for the breakage. This is a required change but I will see if I can move it to a major version number.


@apotonick That clears errors, because if you call `#valid?` twice, you don't wanna see the same errors twice.

This could be avoided by using `Set` instead of `Array` and tweaking with `Error#==` or `Error#===` to ensure unique entries.


Test comment


> A possible approach, for instance, would be to add p or i to the GSdx resolution, and if i then also add the deinterlace mode. E.g. 640x480p or 640x480i (bob bff), since we only care about deinterlace mode only when the content is interlaced, but we don't care when it's progressive.

Yeah I think that's a fair compromise for it, keeps it compact but provides the required info. "auto" may look a bit ambiguous however, so we will need to possibly do something about that.

> removing save slot: I would miss it a lot at the title. I save/load and switch slots often, and it's much easier to observe at the title IMO. In fullscreen the title is invisible anyway, so it doesn't hurt anyone that it's there. But I'd be fine with moving it back to the end of bar. Also, if other than me no one finds it useful at the title and we're running out of room, sure, let's remove it.

Is this not a lilypad hack anyway? So it should be toggleable depending on user preference. I'm quite happy to leave it up to the user but not have it forced on.

> limiter: restore to its own "section" as before but replace "None" with "Capped", well, I think it's important to know the "mode" only when it's not in "normal" mode. But sure, go ahead and restore it.

Maybe, but showing something is more user friendly than nothing at all.  When I mentioned that I didn't think about the possibility that "Limiter:" might not be showing either :P

> Stuff which could be useful to have at the title, ordered by importance for most users.

Agreed, I think the 2 most important things are speed followed by resolution, everything else from a user perspective could be negotiable depending on the person. However i feel the importance is Speed > Resolution > Limiter > Renderer (although) i like this being at the start) > Usage Stats.


そういや出力に改行入れてないや~nで改行らしいな


@Kail72 same here please specify the changes, thank you.
@kirirur I will implement them in the sql dump.


I can assume that here because everything was required to be packed into the unit cell previously


そういや出力に改行入れてない気がする


Spent hours on this problem for my misunderstanding.


Don't leave the dead code.


The convention recommends to have a whitespace between keywords and parentheses to easily distinguish them from function calls (see [mw:Manual:Coding_conventions/JavaScript#Whitespace](https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript#Whitespace)).


Note that there is also [a similar plugin from friend of the show pat brisbin](https://github.com/pbrisbin/vim-mkdir)


for hillshade file it is ok


@omerkarj the $timeout is a bit of overhead here


Please improve your GIT commit messages!


Read this article carefully: https://robots.thoughtbot.com/5-useful-tips-for-a-better-commit-message


we might want to change the tmp dir so we can cache the download


Don't change the coding style! Please use `astyle` for consistent style.


@clepple : that's a good idea, though maybe hard to keep in sync (would need some tag to remind that this entry is tied to that PR / branch / ...)!
I indeed missed again the "@ version" in the commit message, sorry!


If you install Git hooks properly, git would check coding style every time when you are about to commit.


Please make a list of reference materials about "murmurHash".


Thank you :)


**no-warning-comments**: Unexpected todo comment.


**no-undef**: "expect" is not defined.


**require-jsdoc/require-jsdoc**: Missing @jsdoc for componentWillMount


**no-undef**: "describe" is not defined.


**no-trailing-spaces**: Trailing spaces not allowed.


**Eslint**: 'return' outside of function


**indent**: Expected indentation of 8 characters.


**indent**: Expected indentation of 4 characters.


**indent**: Expected indentation of 5 characters.


**indent**: Expected indentation of 9 characters.


Please improve your English expressions.


:+1: LGTM


"detail_entry" sounds too common to distinguish.


@astrofrog This sounds great. Let's put it here and see if any progress could be made later.


HAY


We usually use double understroke for special purposes, such as `__LINUX__`, `__GNUC__`, etc.
Don't use `__OPT__`, which is misleading.


@jrudolph is this change alright, so that the exception is thrown earlier (which I think is the case for the scaladsl)?


Don't leave the dead code.


+1 for adding it and calling it `andMap`, which is already used in `Task` and many third-party libraries.


Please write down meaningful GIT commit messages.


Oh, good catch. I'll patch master now.


Closes #4


wOw


Not quite what I was looking for, we can discuss later today.


Thanks for helping this, but I have some problems with building.
- CircleCI reported it failed building it https://circleci.com/gh/DragonetMC/Dragonet/169
- And if some edited G++ classes imported Dragonet classes, will it be fine?


This results in all messages for ALL ROOMS being cleared (a refresh brings them back).  Consider modifying this so it hides all messages for the room we're currently viewing instead.


Cannot reproduce #38


this is a hard work.... 〒︿〒


Can't spell initial apparently.


I've corrected the new oc behavior.  I've also included a note in latest.go about what carrying a kube api object entails, comment included below.

```
[pweil@localhost origin]$ ./old-oc get scc
NAME         PRIV      CAPS      HOSTDIR   SELINUX     RUNASUSER
privileged   true      []        true      RunAsAny    RunAsAny
restricted   false     []        false     MustRunAs   MustRunAsRange
[pweil@localhost origin]$ oc get scc
NAME         PRIV      CAPS      HOSTDIR   SELINUX     RUNASUSER
privileged   true      []        true      RunAsAny    RunAsAny
restricted   false     []        false     MustRunAs   MustRunAsRange
```

```go
	// deprecatedKubeApi is used to indicate types that OpenShift is carrying but still must be accessible
	// via the kube api rest mappings and the kube api prefix.  If you carry a deprecated kube api object in OpenShift code
	// you need to do the following:
	// 1.  register the type storage in master.go#installDeprecatedKubernetesAPI
	// 2.  add the printer to OpenShift printer.go
	// 3.  register any previous client shortcuts in the OpenShift factory.go#expandResourceShortcut
	// 4.  register the type here (and in the root scope map if necessary) to have it added to the kube rest mapper
```


I'm still working on this.


This change breaks safari submission (iOS included).

Submit buttons must be visible. I can see about hiding it better. What was the problem exactly?


fixed


`-nooutput` was removed in the next commit and replaced with `-stdin` and `-stdout`


@mastercoms Thanks. xD These days I've been busy so I don't have much time debugging it. Well thank you so much.


Could you put the badges on the same line?  See https://github.com/OneBusAway/onebusaway-android for an example with the travis and Gitter badges.


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


singular


Почему изменил тип?


Could you also please add a "Release" section to README for instructions of how to do a new release (obviously removing any username/password info)?  Can't recall if we discussed this yesterday.


@urkonn sí, flag es marcar un comentario como inválido
lo resolví en https://github.com/mxabierto/participa/pull/214


Unfortunately the infinite loop was not yet fixed in this cleanup.


missing oom check...


Почему сделано так? Ты понял, что было до этого?


And, move then to right of project title?  Again, see OBA Android.


> Is this not a lilypad hack anyway? So it should be toggleable depending on user preference. I'm quite happy to leave it up to the user but not have it forced on.

It was[*] a lilypad hack for many years, but already part of PCSX2 since some years now. The lilypad hack is also buggy since it can't track the slot change when you saved/loaded a different slow via the main menu. Also, when the lilypad hack is enabled, it adds `(lilypad)` to the title such that you know it comes from lilypad.

> Maybe, but showing something is more user friendly than nothing at all. When I mentioned that I didn't think about the possibility that "Limiter:" might not be showing either :P

Just play with it, and change it however you think it's best. I promise I won't change it back later without you wanting to :)

> I think the 2 most important things are speed followed by resolution

I agree about speed, and even while I personally really like the resolution to be displayed, TBH I don't see the value it brings to most users. But anyway, I didn't remove the resolution. You did in your suggestion/example  :)

[*] was, because yesterday also I updated lilypad to disable all the hacks which are not required for PCSX2. This "save state title" hack is included, as well as the screensaver hack and the fullscreen exit on escape hack.


quotegen.js isn't actually used anymore 😊 I kept it for historic reasons. Will attempt to copy over your changes & remove it permanently.


The location of the ProgramData folder is assigned to an environment variable in Windows because it's location can vary. It's usually (although not required to be), located at `%SYSTEMDRIVE%\ProgramData`, with `%SYSTEMDRIVE%` usually being `C:` (once again, not required). If the Git for Windows installer has a hard-coded reference to `C:\ProgramData`, then that's probably incorrect behavior and will cause problems for people who've installed windows to a drive other than `C:`.

IMO using the `%foo%` form of environment variables is appropriate here, as the docs are referring specifically to Windows, which uses that format natively. I personally use git directly from the `cmd.exe` prompt and rarely use bash, and I doubt I'm the only one. To somebody reading the docs who isn't necessarily familiar with the differences between POSIX and Windows might erroneously think that they need to use the `$foo` form, which will _only_ work in Git Bash and could potentially cause needless headache.

If desired I can update the PR and the commit to include extended information such as the usual location of `%PROGRAMDATA%`, but I hesitate to revert back to `C:\ProgramData` specifically, due to the issues I outlined above.


Why add thw original glow stone...  it was never included before, so why now?. The net.glowstone from dragonet is moved to dragonetmc/glow etc

@DefinitlyEvil I can successfully build it using maven myself. Just git clone --recursive (dragonet URL) and mvn package. Add maven to your PATH (On windows) or Linux just apt-get install maven so you can build from cmd line.


ooo my bad :P


Ok, cool -- let's leave this for v2.0.0.  Thanks!


Oh lol nvm


Sure, I had to push this unfinished readme in order to see how does it look like on github. I am going to push another commit including readme updates.


Fixed with the latest commit.


Why do we have 2 of the same url


I accidentally added the original Glowstone, but all is good now. Your repo structure was very flawed, and did not use maven correctly.


Right. I was on the fence about that too. I decided to align with the other casing methods (like `_.startCase`) which means it should be possible to run each others output and get back to its original output.


@marcioj
> Is this the expected behaviour?

No, this is a bug :), can you create an issue/PR about it, thanks :+1:


@gorhill you submit this to chrome web store?


Still not quite what I was asking, change the test to pass in `@[]` and see what happens.


Don't use `__OPT


As a request from the Learning Commons coordinator. The intended usage of the physical space has changed.


Yep seems to work fine again! thx!


@mastercoms So you like my idea to change to submodule?


wow, thorough, :+1:


Shadowing outer local variable - `m`.<br>Line is too long. [113/100]


Align the parameters of a method call if they span more than one line.


can we restore this one, and remove the other?


Use `each_with_object` instead of `reduce`.<br>Do not use semicolons to terminate expressions.


This disabled all tests in this spec. I guess that is not wanted.


We already define macro `IMPL` in `Makefile`.


Yes, but I'm fixing it up.


As you can see here: https://github.com/alkino/systemd/commit/4a1b20dc01b6e7ae782e4509febee3f3eaa05f8b#diff-f3f577c0bbbeb92b4c86bf42eb2a83beR628
The dbus is plural.


Missing callable typehint for `$out`


You should decouple such modifications from this commit.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.642](http://teamcity.monogame.net/viewLog.html?buildId=10475&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:06:14


Did it work for you?


indeed! you are right!


Can we avoid Stratigility dependency?


Fixed in 98872fc2a1b9dc03dea5c966d5d6a7d52b82dfa0


Though you've stumbled on a bug where those characters are not deburred but still removed.
```js
_.deburr('THIS IS A TĒŠT');
// => "THIS IS A TĒŠT"
```


I know there's some other regexes that don't specify a beginning sequence (`^something(Terminating app due to uncaught exception.*)`), but I'd appreciate if you could take a stab into this one.

The reason for this is performance, on some giant outputs it needs to scan each line until the end of the string.


Also you are pushing the full string "EnvironmentFile=/foo" now into this array, but that's wrong, it should just contain the path, possibly prefixed with "-".


Why bother with a memstream? asprintf() should suffice here.


Hey @bengolder - so question about this particular method - you're checking to see if a new item is in the `items` list. But because you're clearing the array at the beginning of `load_data` and also saving to a DB anyway shouldn't you be checking against the database table instead?


Can you send me a source sample of usage blur on VP?


Some comment


urgh please pr a fix?


I patched all these issues, including the infinite loop, back in August.  Unfortunately the infinite loop was reintroduced just a few days later.  I submitted another patch to fix this as well as a patch for another major bug over a month ago but Xilinx never responded.  Guess it is time to ping their mailing list again.


Is there template in Rcpp like ```is<NumericVector>(sexpr)`` ? We can use this for better readiblity


Sure


I was playing around with marshmallow last night. Do you know if `@pre_load` is available with `flask-marshmallow`? I wasn't able to get the decorator firing even with importing.


shbcfa


Please change translate for this line to:

"Недопустимое название для группы"


HELP ME !!!!
OS:Gnu/Linux Debian
GNOME: 3.14.1
ERROR!!!


Maybe explain why you do not ask for `tp1.typeSymbol.exists` instead.


Sorry you felt that way. It wasn't mean to be demanding at all, I was trying to be respectful of the project seeing as it's the first interaction I've had with it at all. I mainly just wanted to start a conversation about the topic. I guess I understand now that a PR is the way to do that, but I didn't know that before and your response felt unnecessarily hostile. I do apologize for the demanding tone of my comment, I can see how it'd come off that way. I was looking for this information for about an hour and was extremely frustrated (that's no excuse, I know) that it wasn't readily available, at least where I was looking for it.


I'll re-evaluate the deburring and word rules.


```php
// Avoid needless assignment + Yoda conditions + more readable
if (null !== $out) {
    $response = $out($request, $response);
}
```


This commit undoes a previous bugfix in ec210e10bd9272c971a4d66e163e33809d8a9ed1 dma: xilinx: Fix infinite loop in xilinx_dma driver. I haven't checked what else it reverted.


This should be 'ypserv'. But I will fix that manually.


Que tal José, noto que en tus cambios sustituiste en el archivo Datos/conexion.php la base de datos, cambiaste "ccjj" por "sistema_ciencias_juridicas", no se puede aceptar este cambio porque involucracia todos los demás archivos que dependan de este.


Hi Guillaume,

Did you check this script under high concurrence ? I suspect it to have a race condition on glabl vars $user and $password when two or more user are authenticating in the same time.

You should save each user/password informations for ongoing authentications in a hash using the session id ```$pg_msg->{'sess_hash'}``` as key.


Ready when you are. For the test, I am thinking of having a rectangular container with a certain water level. Then have particles which are at the bottom of the container (should float up) and also have particles dropped from above the container into the container. What do you think? However, I am still unable to get any 'effect' from having the buoyancy engine in a simulation. The gravity engine still seems to override any effect. I think going with the point on the liquid surface is a good idea. How do you want to proceed with that?


Yes, nice, good catch.


It sounds like a good rule of thumb, I'd be happy to follow it. And you're right, we should be consistent across the team. Would you recommend using something like Checkstyle to enforce some of those rules? We do suffer from severe lack of coding standards at the moment, looking at the existing code base.


I just noticed and commented on that bad commit as well.


Btw. wouldn't implementing with a `val` have the same effect? I.e. just changing the `def directive` to `val directive`?


Lets take that when it becomes an issue. Right now it just passes one file and I don't fully understand the complexity/purpose in the other headers.


> But anyway, I didn't remove the resolution. You did in your suggestion/example :)

No I didn't, in either :)

> I agree about speed, but even while I personally really like the resolution to be displayed

I know some people don't care, mainly those who want authentic emulation, but the majority of users like to flex their e-peen by how massive they can make the resolution and show it off (presuming we use the multiplied scaled resolution that is).  When in native it's of little concern to the user really, it was more useful from a dev perspective than anything else.


The sum (e^(-alpha * t)), where t is in 500ms increments and alpha = 1 / 60, converges to ~120.5, so I solved for t in sum (e^(-alpha * t)) / 120.5 >= 0.01, and sum (e^(-alpha * t)) / 120.5 >= 0.1


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


Which kernel base did you use to compile your latest build? Cos the audio bug is back compiling using base 2.1


this breaks in my firefox, which started to block http resources in a document retrieved via https.

you’ll have to self-host it (submodule) or find a https CDN


:+1: cool beans


Right. Those jscs rules that I've taken from other repository are following Wikibase convention, which is based on Mediawiki but of course differs in some spacing issues, including the one above (https://www.mediawiki.org/wiki/Wikibase/Coding_conventions#Whitespace).
But for this JS code it makes more sense to stick to MW convention, I guess.
This makes this pull request rather invalid. I'll change the ruleset and adjust spacing etc.


I'd also like to point out that while this change was unfortunate (but necessary), the likelihood of errors or changes creeping in on any project even in a minor release is easily incurable. May I suggest it might be a good idea to lock your dependencies if you are building for production like this?


原文: "Real World Business problems: Vertical Web Crawling with Scala"
訳案: 業務上の現実問題: Scalaを使ったバーティカルな Web クローリング

「縦方向のクローリング」とか怪しい訳になったら、 原文の "Vertical Web Crawling" でググッてみるのがオススメ。すぐに[Vertical search](https://en.wikipedia.org/wiki/Vertical_search) つまり、業務や分野に特化した検索のことで、定訳は[バーティカル検索](http://www.sophia-it.com/content/%E3%83%90%E3%83%BC%E3%83%86%E3%82%A3%E3%82%AB%E3%83%AB%E6%A4%9C%E7%B4%A2)だと出てくるはずです:

> バーティカル検索とは、Web検索エンジンなどの検索機能のうち、対象対象を特定の分野または特定種類のコンテンツなどに限定して参照できる検索機能のことである。対象分野の切り替えによって横断的に参照できるという意味合いを含むことも多い。


Ah, agreed then :) (did you just edit your example few comments up? or did I actually didn't see the resolution earlier while it was there??!?)


Good job @jpatel531!


Add a TODO: Should profile performance implications, and, if necessary make existsPart stop at static.


`getPrivateChannel` -> `getPresenceChannel`


@chelmuth: please add this and 7de82ff0aef50e8682fb38b25cc8b35f8461487c to staging


In deployment, this is another thing that should probably be placed in the Apache docroot. I think 'media' is fine.

I think /f/ was created just to have shorter URL's.


Bug fix


on it


The base patternfly css file has a pathing issue. So I am overriding it with the correct location for this application. This PR should address the apache build issues


Can avoid that multiple return hack?

```php
$sessionContainer = $this->getSession($request);
$request          = $this->injectSession($request, $sessionContainer);
// ...
```


Yes, it should be available. Our Schemas are inheriting from [`marshmallow-sqlalchemy`'s `ModelSchema`](https://github.com/marshmallow-code/marshmallow-sqlalchemy/blob/dev/marshmallow_sqlalchemy/schema.py#L100-L174)


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


Can I ask why you've removed this? You forgot to remove the line in the README that points people towards the demo for more information


At least he added the second line... :P


I don't even


@sklum @JordanCheney FYI this surfaced when I upgraded the OpenCV cascade models to those provided in 2.4.11 (a580a10014cd698da4bf1425137b531d16328203). IMO this represents a somewhat serious performance regression, so I think I will revert at least the alt2 model. Looking into it some more...


I may or may not have edited :P I did it earlier though before your last post, not after just to make you look like a douch xD


colocar o nome de um livro de verdade. como o publico é java, pega o livro de certificacao :)
pode por o valor real tambem


lembrar de evitar abstracoes :)


Thanks for this commits, yuri.

Also, i applied days ago this useful commit..

https://github.com/txuki2005/TaUrUs_Kernel/commit/834844e60f25e45a6cac77d61e2a445e6aba673f


Humm... nome estranho


Remove unnecessary blank lines.


Here I am just removing duplicates from this particular set of JSON data, before touching the database. For example, we have a Supervisor listed on each of 300 JSON objects, but there are only 5 unique supervisors. This makes sure that when we save to our database, we are only trying to check and save those 5 supervisors, rather than 300 copies of 5 different supervisors.


I guess so. I'm not familiar with the code so I used something that was obviously done first. I'll change it to a `val` and add a comment.


unitTests failure due to db connection issues.


Crie um programa de teste para cada tipo de teste que vai fazer. Talvez tenha feito isso (vou ver a medida que baixo nos commits aqui). Mas de nomes mais claros ao que o programa testa: LeXml. LeXMLComoEventos, EscreveXML etc


Remove unnecessary comments.


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


WOW ... bat research.... in a homemade cave ... that sounds interesting!
I didn't mean to press you, just noted. In fact, I think it is a good idea to keep only the install stuff in the README, at the very minimum, and all other stuff may be in a user manual (which has sort of duplicated sections now). The manual could be accessed from the Useful links section, for people who want to now more before installing.
Good luck with your exams.


I’m not sure what to do here. The output comes from a log in `xctest` so it has a long beginning sequence, e.g. `2013-12-30 17:08:56.197 SACoreTestApp[92319:6103] *** Terminating app`

Should I have a regexp like that?
```ruby
/^\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}.\d{3} .*\[\d+:\d+\] \*{3} Terminating app ...
```


Ditto.


From now on I'm gonna quote ref on everything :p (though I think ref and I do have permission to edit eachother's comments :p )


no fim, se possivel da um extract method nessa tres linhas assim voce fica com um codigo mais bonito pras pessoas. pode ser no ultimo video do primeiro capitulo


produto ficou repetitivo


Currently the controller will ignore that one rule that doesn't have a node port service. It'll setup all the cloud resources and when the node port on the service shows up, ill update the urlmap.

I haven't implemented the type=loadbalancer part, it seems backwards and very costly if someone is doing (l7 -> l4) but I suppose people might want to given a static ip a different url. Will ladd to wishlist and get to it sometime.


Are there any place where we should update that? repos, deployment, test script??


Add a blank line between functions.


a questao do nome :) boa que separou, mas lembra que quando voce ensinar stax vc ainda nao explicou o que eh stax, entao coloca um nome ligado com o problema...


so declara na hora que vai usar. tem umas 3 linhas ainda pra usar


where is this used in the docker code though?


原文: Basic Web crawling is very simple but it can quickly becomes a quagmire of complexity when you have the requirement of extracting and integrating information from the heterogeneous real world.
訳案: 基本的な Web クローリングはシンプルですが、多種の情報が入り混じった現実世界から情報を抽出、統合するという要求仕様を追加すると途端に複雑性の泥沼にはまります。


iterador eh um nome que da medo. tudo que é funcional da medo. tudo que é dominio é mais natural. melhor que iterador é "elementos", "tags"... etc


repara como ai ficar bem mais legivel? lembra sempre de atacar pelo lado do dominio, toda variavel


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


@kusera FYI the doc for `exec` is at https://nodejs.org/api/child_process.html#child_process_child_process_exec_command_options_callback


Yes, you are correct.  This is also an all-or-nothing comparison.  All thresholds must be exceeded to be considered a "high resolution" device.  Apps are welcome to change this and/or opt out, but by default, we don't want apps burning out batteries or making UI too small for people to interact with (since we've seen lots of apps ignore DPI completely).

If an app developer understands these concepts and wants to render at native resolution, they can just disable the scaling logic by setting SupportHighResolutions to true.  I'm debating adding an additional comment in the SceneRenderer file to further improve discoverability of this behavior.


原文: How well does Scala handle the challenges inherent in this surprisingly common requirement?
訳案: この意外と頻出する要求仕様に Scala はどれだけ対応できるでしょうか。


How about passing the digest object instead of its name? Then we can instantiate using `HmacAuth(hashlib.sha1, ...`), and don't need the custom error-handling logic.


Prefer not to use static methods--in Python, all they buy us is an extra layer of namespacing, which we already get at the module level. And smaller classes are nicer.


Hi,

Same comment than for commit 313d781b25ebfe07728ac6272924b4a2b2148493. You probably have a race condition on $start and $end.

You have a good example of tracking sessions in bin/pgs-badger. See the global hash ref $sessions and look for ```$sessions->{$hash}```.

About tracking the time the connection/disconnection time, you might want to use ```$pg_msg->{'timestamp'}``` instead of ```Time::HiRes::gettimeofday()```. This is set to the capture time for each messages.


citar de formatacao, cuidado com encoding. um video pra trocar para decimalformat


Fair.


ありがとうございます!!


aqui e no proximo voce assumiu que veio o conteudo certo... se viesse errado o que aconteceria? deixar esse tipo de decisao clara pro aluno. descrever na ementa. assim o aluno sabe que as implicacoes de cada chamada de metodo eh um erro diferente. e que ele tem que estar ciente disso


Yep we do :P


msg mais bonita :) tipo... validou ok


so are files in this directory expected to be named `\d+-[[:alnum:]]` or similar?
Definitely a feature that needs docs.


anotar aqui quais serao elas pra eu conferir?


Hola


da pra usar java 8 aqui?


Perhaps give another example in your commit message


usar new Produto(campos) nesse caso q vc ja tem todos os campos


Will look into that.


What is the other way?


:+1: GG


I can't cleanup any other way, it defies HA if I do. Tl;dr:
* In production this thing needs to be able to survive restarts without destroying and rebuilding everything (say node dies)
* Now say that while it's down, someone deletes an Ingress, if I just relied on Ingress deletes to cleanup I'd leak a loadbalancer.

I figured anyone doing a kubelet exec and hitting 8081/quit knows what they're doing.


citar que eh boa pratica... q essa abordagem permite usar good citizens bem facinho


perhaps this is implied?


For functions that return tuples, it can be helpful to use a `namedtuple` instead--that way, consumers can figure out the return type without having to read the docs:
```python
import collections
HmacResult = collections.namedtuple('HmacResult', ['match', 'expected', 'computed'])
hr = HmacResult(1, 'abc', 'def')
```


da maneira que foi implementado nao permite good citizen, precisa dos setters. deixar claro. citar que poderia armazenar todas as variaveis temporarias pra depois dar um NEWZAO... mas repara que seria chato? por isso mesmo que na pratica pessoas fazem como voce fez. citar.


remove essa linha :)


i am wondering if there is not a clean way to derive the path, rather than hard coding it. Just like `dockerinit` path can be relative to the `docker` binary.


faltou abrir html, title etc. deixa redondinho bonitao?


venda.html :) lembra de nomes que fazem sentido?


padrao


It's node backend record, you are right.
You are also right that all node backends records should be unique, but we cannot fall back to using set because node backend records ordering is important.
Fixed this in https://github.com/yandex/mastermind/commit/6315bd3278be4f31b14a02169ef585b7cb5a2f38, using it in https://github.com/yandex/mastermind/commit/a3d6cc7fcd4b3ac71d4e3634093f2a442d4b91b3 and https://github.com/yandex/mastermind/commit/1f78f5b35363e0e1a90b60b8e00d992e046853bc


Is that the way we're going to do it long-term? Should we document it in [rpc-api.md](https://github.com/OmniLayer/omnicore/blob/omnicore-0.0.10/src/omnicore/doc/rpc-api.md)?


@alexandreab this timeout seems a bit too much. 3 minutes should be enough for now.


 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:673](../blob/3e62a99f821433f690ee69a0ffa00b28ff70208f/include/chaiscript/language/chaiscript_parser.hpp#L673) Expression is always false because 'else if' condition matches previous condition at line 671. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:677](../blob/3e62a99f821433f690ee69a0ffa00b28ff70208f/include/chaiscript/language/chaiscript_parser.hpp#L677) Expression is always false because 'else if' condition matches previous condition at line 675. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/3e62a99f821433f690ee69a0ffa00b28ff70208f/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 675. (multiCondition)
 - __style__: [include/chaiscript/language/chaiscript_parser.hpp:679](../blob/3e62a99f821433f690ee69a0ffa00b28ff70208f/include/chaiscript/language/chaiscript_parser.hpp#L679) Expression is always false because 'else if' condition matches previous condition at line 677. (multiCondition)
 - __information__: [src/chaiscript_stdlib.cpp:1](../blob/3e62a99f821433f690ee69a0ffa00b28ff70208f/src/chaiscript_stdlib.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __information__: [src/main.cpp:1](../blob/3e62a99f821433f690ee69a0ffa00b28ff70208f/src/main.cpp#L1) Skipping configuration 'CHAISCRIPT_MSVC;max;min' since the value of 'max' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly. (ConfigurationNotChecked)
 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/3e62a99f821433f690ee69a0ffa00b28ff70208f/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/3e62a99f821433f690ee69a0ffa00b28ff70208f/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/3e62a99f821433f690ee69a0ffa00b28ff70208f/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/3e62a99f821433f690ee69a0ffa00b28ff70208f/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/3e62a99f821433f690ee69a0ffa00b28ff70208f/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


原文: How well does it help divide this complex problem into simpler problems and leverage existing libraries?
訳案: 複雑な問題を単純なものに分割したり、既存のライブラリを活用できるかなどの観点から Scala を評価します。


I know this came from the cookiecutter I recommended, but if we're not using anything in here, the file can go. I also like to use the `six` library for python 2-to-3 compatibility fixes.


o que acontece se usar o mesmo reader para ler dois xmls? vai acumular os dados certo pois o produtohandler é o mesmo, deixar claro


Could this bit be extracted to a separate function?

```
remove-if-not (lambda(w) (proof-associated-buffer-p (window-buffer w)))
				  windows))
```


(isto eh, depois no ultimo video, citar)


e mostrar :)


Yeah, tots aware, stuff happens.  There are several things we should of been doing that would of caught this, but weren’t.  Our problems are our own fault.  Hindsight is a bitch as they say ;)  We already are planning several organizational changes as an outcome and whether versioning is changed or not will no longer effect us.  My suggestion is just to make you aware that this change may break anyone who is using bower for automatic patch updates which is the default.

Regards,

Chuck Holbrook

On Wed, Oct 7, 2015 at 8:54 AM, Alex Gibson <notifications@github.com>
wrote:

> I'd also like to point out that while this change was unfortunate (but necessary), the likelihood of errors or changes creeping in on any project even in a minor release is easily incurable. May I suggest it might be a good idea to lock your dependencies if you are building for production like this?
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/alexgibson/notify.js/commit/6133f4daf64f9e3ff94957b50f9aca04a5a434e1#commitcomment-13642310


原文:  What are the maintainability considerations?
訳案: また、保守性に関する懸念事項はないでしょうか?


quando ensinar isso, comecar com qName.equals("produto") e depois de um tempo no fim do video mudar. explicando o problema do null


Hmm... this is probably in the wrong place for RPU, because it's executed on the sender, which may be un-upgraded.


em geral extraimos esse codigo das quatro linhaspara um novoProduto()


nomes reais


Does `(< 1 (length (frame-list)))` mean the same as `(not (frame-list))`?


Titile of commit is started with upper case letter.
s/applied vcpu concept to guest_struct/Applied vcpu concept to guest_struct/
Anyway, this commit does not look like applied vcpu concept into source code.

Think again what did you do.
This commit shows two things, merging sub-functions into a function and move guest structure to proper file.


Appreciate the feedback, thanks :+1:


Seems like this would be captured well by an `Enum`: https://docs.python.org/3/library/enum.html. It's in the standard library in python 3 only, but it's been backported: https://pypi.python.org/pypi/enum/.


@ACI-leecooper btw it's easy to avoid "Merge branch 'master'" commits like this one - just use git rebase, e.g. me and @jcalfee are using the following alias instead of direct "git pull":
alias gspp='git stash && git pull --rebase && git stash pop'


acentuacao


nomes dos arquivos. se possivel evitar "2"


Seems like you fixed it before me, good job. <review


========== Build: 1 succeeded, 0 failed, 0 up-to-date, 0 skipped ==========


Ahhhhhhh. I was thinking of a different scenario, where `socrata_query` may
be called more than once every three days and then there may be duplicate
inspections in the table if they're pulled more than once. I'm assuming we
just get the script to call every 3 days and that'll be AOK?

On Wed, Oct 7, 2015 at 9:01 AM, Benjamin Golder <notifications@github.com>
wrote:

> Here I am just removing duplicates from this particular set of JSON data,
> before touching the database. For example, we have a Supervisor listed on
> each of 300 JSON objects, but there are only 5 unique supervisors. This
> makes sure that when we save to our database, we are only trying to check
> and save those 5 supervisors, rather than 300 copies of 5 different
> supervisors.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/codeforamerica/mdc-inspectors/commit/2769b6940324bea0446a8c422ad5b291a8c284e7#commitcomment-13642476>
> .
>


s/guest/vcpu/


Ditto.


大丈夫だと思います。


so the old way didn't work for you?


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


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


Let's leave it a `boolean` for now.


saw approval from macintux
at https://github.com/basho/riak_kv/commit/90ebb561ec99b63aaf2e948bcc699bf5a3567276


merging basho/riak_kv/bugfix/bch/update-riak-ql-repo = 90ebb561 into borshop-integration-1210-bugfix/bch/update-riak-ql-repo


basho/riak_kv/bugfix/bch/update-riak-ql-repo = 90ebb561 merged ok, testing candidate = e36ffd91


We are still seeing this issue in versions of Rails containing this code.


s/guest/vcpu/
s/vcpus/vcpu/
s/struct vcpu/struct vcpu_t


I appreciate the library.  Thank you!

Regards,

Chuck Holbrook

On Wed, Oct 7, 2015 at 9:11 AM, Alex Gibson <notifications@github.com>
wrote:

> Appreciate the feedback, thanks :+1:
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/alexgibson/notify.js/commit/6133f4daf64f9e3ff94957b50f9aca04a5a434e1#commitcomment-13642745


Isn't that like legit error? What's the point of initializing both members of union?


Cool; looks good.

Matt

++++++++++++++++
*Matt Carroll*
Cell 617-462-0631
@MattatMIT
++++++++++++++++

On Tue, Oct 6, 2015 at 12:29 PM, Joseph Goldbeck <notifications@github.com>
wrote:

> Add newspix
>
> On behalf of @mattcarroll
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/c4fcm/civic-parade/commit/8ef42494a382bde515ca7f4face3ca359dd0641b
>


@troianoandres
I would like to add a directive property in order to remove 'ng-controller: MovieController as movieCtrl' in my html, in order to have a more legible sintax.
Do you think it's posible by doing it here, in the movies module?

And by doing so, my goal is to share this controller in the other modules.

Could you tell me whether this aproach is good? Thanks!


In this commit, I modified the DHT_SAMPLE_INTERVAL to get the data from the DHT22 sensor at a rate of 1 per 15min. Also added a simple uptime() function that, when called, sends me a notification with the current uptime of the Photon.


change spelling


May I suggest using `for ... in p.TransitionsFrom p.Initial do` here? Would make the intent of the loop slightly clearer.


I've updated the [wiki](https://github.com/killbill/killbill/wiki/Kill-Bill-Migration-0.14.x-to-0.16.x/_compare/1597e01%5E...1597e01). I don't think it was overridden anywhere though.


Maybe we do not need those `import` anymore


We could avoid this boring stuff by passing `true` as second argument to `json_decode()` [there](https://github.com/lcobucci/jose-parsing/blob/1ea9a54dcbf3a92240313b3dc1dde6e816576264/src/Decoder.php#L28).


Is this already in tagged version?


Oh well, good thing you noticed it. You have amazing eye for these.

Let's make a release when we get this in.


@kayousterhout, :)  thanks :+1:


Looks great! :shipit:


Since this is WSGI-oriented, including a WSGI middleware would be nice. Should be a simple wrapper around `HmacAuth`.


Ouch! :grin:


I removed this @addyosmani  and pushed the change onto your branch :)


Maybe we should add unit tests at some point :)


`s/omited/omitted/`


sorry this was by me.


Oops, my [bad](https://github.com/killbill/killbill/commit/d0ad5c2e0fdfe43f7f6cdea08a32aacb3cef891b). Thank you GitHub search!


Este projeto foi desenvolvido utilizando HTML, PHP e JavaScript.


I'm thinking we should create three types:

1. `Balance` - contains just `balance` and `reserved`
1. `AddressBalance extends Balance` - adds an `address` field
1. `PropertyBalance extends Balance` - adds a `propertyid` field

`omniGetBalance` will return `Balance`
`omniGetAllBalancesForId` will return `AddressBalance`
`omniGetAllBalancesForAddress` will return `PropertyBalance`


Recently I just made the same change in my project :)


At line 589 the code now uses XILINX_DMA_REG_DSTADDR instead of XILINX_DMA_REG_SRCADDR. On first sight that seems correct, but unfortunately the value for XILINX_DMA_REG_DSTADDR is incorrect. It should be equal to that of XILINX_DMA_REG_SRCADDR. That is why this worked before and will now fail.


Really nice work @peterjc, thanks for continuing to invest this much time and effort in Galaxy and related efforts :smile:.


Right! Will do so from now on.


I am whinaral paul and as a profession doing development  from last 6 years. anything more @Gorash  ?


@mziccard do you see a value in adding to Blob and Bucket static methods that would act upon multiple instances (and internally use batch). Such as `Blobs.get(...), Blobs.delete(...), Blobs.update(...)`. Batch is more flexible but
also harder to use and I wonder if would could simplify it for some common use-cases. What do you think?


I think the more likely correction lacks curly braces, if the @class is the word "omission":

`<span class="omission">`


@thomasflynn, just realized the definition of GVRCameraClippingDistanceInterface is missing from this commit.


fix in https://github.com/zdict/zdict/commit/55c6d60aad049141357fa1318c4de043d5c89abc


A classe do arquivo SolicitacaoColetaViewPage.groovy deveria ter este mesmo nome, mas está com o nome igual ao de outra classe que já existia no projeto.


Think it would make more sense to use the RUNMODE_* enum's values here instead of the arbitrary string.


Your indentation is off in these two files.


Faltou só o circunflexo depois do "~'" nos steps.


Is this API complete? Or there are other things that have to be added?

If not, are you planning to expose the entire Skia API as C?

Thanks


Ah. I see.  Yup, okay with this for now -- we can address if/when we refactor to independent service containers.


噫。。游离于太多branch之间了。。


Oh, three actually.


#data clearn

library("dplyr")

house_power<-tbl_df(read.table("household_power_consumption.txt",header=TRUE,sep=";", na.strings = c("?","")))

house_power$Date<-as.Date(house_power$Date,format="%d/%m/%Y")

Project<-subset(house_power,Date=="2007-2-1" | Date=="2007-2-2")

Project$datetime<-paste(Project$Date,Project$Time)

Project$Time<-strptime(Project$datetime, format = "%Y-%m-%d %H:%M:%S")

Project$Voltage<-as.character(Project$Voltage)

Project$Global_active_power<-as.character(Project$Global_active_power)

Project$Global_reactive_power<-as.character(Project$Global_reactive_power)

Project$Global_active_power<-as.numeric(Project$Global_active_power)

Project$Global_reactive_power<-as.numeric(Project$Global_reactive_power)

#plot1

png(file= "plot1.png")

par(mfrow= c(1,1))

hist(Project$Global_active_power,col = "red",xlab="Global Active Power (kilowatts)", ylab = "Frequency",main = "Global Active Power")

dev.off()


Why is this needed, when RS_Init call is under the same #ifdef ?


v7 support library from what I heard is necessary if we want to transition to material design on new devices while supporting the older ones. It would be nice if we could slowly move that way anyways.


Please check if adding `#include <map>` somewhere near the line saying `#include <set>` solves your problem.


Can we get things done without this? This looks like it's really just a wrapper for the nfq/nflog runmode functions, so perhaps we can just use those directly?


Visual C++ has many bugs with constexpr classes ;)


É que considerei livro da casa do código algo como menos abstrato do que só "livro", mas blz vou pegar os titulos da casa do código mesmo.


This is backwards, since videos belong to a category, the videos table should have a `category_id`.


No curso não será esse, durante o vídeo vou dar nomes melhores nas variáveis.


Second and actual mode the same.


Good job! :D


"Julia bundles the following external programs and libraries on some platforms" OpenSSL on what platforms? If "library", I thought OpenSSL was GPL-incompatible, except under the system exception, that doesn't seem to apply to Windows(?). Is this for libgit2?


*Would like feedback on what I've done so far and if it's what you wanted for this assignment, if I'm heading in the right direction, etc.*


Also can you amend your commit and define what the bug was in commit message description?


you know that the "hibernate.search." prefix is redundant right?


Great. Can you add a comment to explain that?


Same for here


library("dplyr")

house_power<-tbl_df(read.table("household_power_consumption.txt",header=TRUE,sep=";", na.strings = c("?","")))

house_power$Date<-as.Date(house_power$Date,format="%d/%m/%Y")

Project<-subset(house_power,Date=="2007-2-1" | Date=="2007-2-2")

Project$datetime<-paste(Project$Date,Project$Time)


So in one place it's completely acceptable to add forward prototypes (even put them deep into the file), but in other place the whole static function must be relocated rather than adding a prototype like here... Strange.


Was just about to open a PR for this one :+1:


@maxiloc why are you adding the indexing in the queue for store which has "Indexing Enable" as "No". All these records just increasing the cron time and seems to be garbage records in queue table.


I would like to shift back to `$(e.target)` and let this function independent of the dom element.


hello


okay - we should then remove it and make the corresponding changes...but only in master, and eventually moved to 2.x. Let's not backport this to 1.10.


если нужно сделать класс Dao сериализуемым, то надо вынести implements Serializable из женериков T & id. Serializable - это интерфейс, поэтому Implements.


So it's required for VS15 to work?


Is this function really necessary, all the stuff can be done in the below function. I don't see a need to differentiate.


This breaks building on 32-bit, because findstring finds pentium inside pentium4


Ist ne echt Interessante Idee, die frage ist von mir jz aus willst du das als Handy App machen sodas man das Handy Bewegen muss oder fürn Computer und dann mit Maus oder Tastatur?


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


It is currently not needed, but does no harm. When I have finished my radio measurements, I'll check if debug output can be enabled via RS232 without much power consumption, so I might reenable RS_Init() again and don't want to forget about the receiver.


Hi Marco,

For Mac OS X, the command which makes the fastest binaries is just
     CXX: g++
     make build ARCH=x86-64-modern

It uses "x86-64-modern" because there are no Intel Macs with old Intel processors without popcount, and it uses the "g++" compiler alias because using the "clang" compiler results in compilation problems on some versions of OS X and in slower executables than using the "g++" compiler.


Yes, you are right,


How did you release this? There should also be a tag created...


Ist ne gute Idee aber nicht der Bereich in den ich später möchte wünsche dir aber viel Erfolg.


merging


Thanks. I shouldn't do pull requests at 4 am :x


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


Honestly I think this should be even lower -- i.e. persistent connections are great, but 10 minutes of no activity? That seems unnecessary to me. I would go lower to even something like 2 minutes.


Formatting looks off here.


Oh, forgot about debug output on serial. Good point. OK, I'll wait for the measurements, and then I will merge the 'final' version.


Oh sorry you did that =).


@satya164 And we're live! :) I did one more pass of the Java code, mostly style (removed unnecessary `final`, extracted some repeated code into `notifyWebSocketFailed`.


This unfortunately breaks when a model is not of type `object`, which is valid—using an array is allowed, for instance, in which case the model becomes a subclass of `NSMutableArray` and then this doesn't compile. Or a primitive type is possible.

This is affecting me so I'll try to work up a PR for a fix and hopefully a test case to exercise it, but not sure if making changes to the Pet Store spec is going to be a difficult process?


Hammer geil für Leute die an viel befahrenen STraßen wohnen oder einfach keine Lust haben nach draußen zu gehen.
Hab nur die gleiche Bedenken wie bei meiner Idee das diese Spiele dann schnell das Kommunikative beim treffen nimmt.


LGTM!


@Ravie у вас не завалялся последний релиз? Не могу войти на форум YTDB, заранее спасибо.


Does not work @64FF00 read issue section


I think you have the parameters reversed in your mind: `pentium4` does not appear as a substring of `pentium pentium2 pentium3`


This is not very clear to me. Not sure what to to differently now. I got stuck with interface, which lead me here.

Could you add some before/after example?


This should be `.shower .slide` to be in line with the actual selector.


Die Spiel Idee finde ich richtig geil das es vollkommend auf den Spieler ankommt wie das Spiel ausgeht. Aber erfährt der Spieler dann auch ob er den richtigen hat oder bleibt das offen?  Obwohl ich selber finde beide alternativen so seinen Reiz haben.


This is my README for my first assignment for Exploratory Data Analysis


@mcnewton

```
---- freeradius-server.orig/scripts/monit/freeradius.monitrc
-+++ freeradius-server/scripts/monit/freeradius.monitrc
```

*.orig, are you sure?


consensus/params.h should be sufficient, I think.


In my opinion the env variable isn't necessary here. I don't really like the idea of a side effect of the environment clobbering a user entered value.


No kidding! Corrected, but cleaned up the code somewhat.


What about namespace Consensus::SoftForks?


self needs to be (self)
See https://github.com/pfsense/pfsense/pull/1952


Idee finde ich super und wie ich schon in der Schule gesagt habe wäre ich mit bei der Partie.


do you think it would be useful to have an indexesByName on TableMetadata like there is for columns, or is the use case for that probably too narrow?


I see, got wrong version from composer. All works fine now!


@mkonicek Cool :)


That it does, thanks!


Viel Erfolg dabei.


I wouldn't bother unrolling it - might accidentally break something and actually a lot of it is quite common.


Well actually, I'd appreciate it, if you would merge it. We might need some discussion on how to handle the debug output. Most people probably don't use it and leaving it out saves a lot od space.


Misnamed commit - should be:
 Enabled pattern lab styling of pdfs
Not:
 Enabled pattern lab styling of iiif.js (tidy)


Hrm, this isn't actually true yet.


Since this is a continuous type of gesture we probably want to use `UIGestureRecognizerStateChanged` here instead of recognized to get a stream of updates to make this a bit more smooth.


Gute Idee kann es mir Persönlich so nicht vorstellen wie das abspielen soll. Wünsche aber viel Erfolg und hoffe das man es dann am ende auch mal spielen kann.


@Caberhagen ich würde in dem Fall eher mit Außenteam übersetzen. Passt auch eher zu Star Trek.
In etwa "Das auf die Rettungsmission geschickte Schiff hat nicht genug Platz für das Außenteam".


Yes I see, but you're not using classes here on purpose.

Mixing OO and functional paradigm seems a bit arbitrary here.
I wouldn't be sure when to use which if I worked on such project.


Ich finde das ist eine super Idee. Hat mich irgendwie an The Book of Unwritten Tales erinnert, was meine lieblings Spielreihe ist. Ist zwar vom Prinzip her nur in manschen belangen das Gleiche aber ich finde deine Idee gut. Hätte auf jeden Fall lust mir da was gutes auszudenken. Irgendwie gab es doch auch mal so ein ewig altes Spiel was exakt das war... auf windows 98 oder so :D


Hi, I'm no java developper but as far as I tested, shouldn't "equals" be replaced by "contains" on line 38 ?. A job can be launched on a mesos cloud if the job label is contained in label field in the cloud definition. Currently if the cloud label field contains more strings than the job label field, the job runs but the workspace browing failswhile the slave is offline.


change to the file


Klingt super lustig! Eine tolle Idee.


Viel Erfolg :P


Gute Idee viel Erfolg dabei.


Add pi function for test pref.


The current form also contains `<select>`.


@jython234 Sorry, it wont work for me if I dont do that and sometimes I forget to remove that. I wonder why it wont work for that if I dont have that there, but it will work for you? (Yes, I did generate a fresh world)


Looks good!


Interessante Sache wünsche Viel Erfolg.


This commit also implements sigma-clipping into KDE calculation.


Yes, we are going to stick to this.

Extending the documentation is a good idea.


That makes a lot of sense.
Em 07/10/2015 18:25, "Elisa Beshero-Bondar" <notifications@github.com>
escreveu:

> I think the more likely correction lacks curly braces, if the @class
> <https://github.com/class> is the word "omission":
>
> <span class="omission">
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ebeshero/Amadis-in-Translation/commit/22462e7e1395bf7e097a278ccd0e9f71e2ed7918#commitcomment-13642989>
> .
>


Meh, that's irrelevant as it's a -p1. It's what quilt refresh added; I only updated the redius/freerad bits. It's changed "freeradius-server.orig" and "freeradius-server" to "a" and "b".

quilt push/pop apply and remove the patch fine.

Cheers,
Matthew


good catch!


Sowas ist ja immer Interessant daher viel Erfolg.


OK, makes sense. In that case, though, could you, please, add a small comment in the code explaining why it is there? In case the discussion takes a bit longer, as is unfortunately common on this project :-).


Klingt gut! Fände sowas in Kombination mit einem strategischen Kampfstil wie bei Heroes of M&M irgendwie geil. Hat bestimmt auch so Potezial das Projekt.


Quick q? What was the date of this meeting?


Wow, this is perfect.

Was exactly what we've been looking for since the introduction of the boilerplate directory.


Sind nen paar gute Sachen bei, viel Erfolg bei der Verwirklichung.


Nope. Not done


fast-forwarding merge/2.0-2.1-end-to-end-timeseries to borshop-integration-1210-bugfix/bch/update-riak-ql-repo = e36ffd91


We want to make sure the scale is set back to `1` here...


@jcelerier once again I would like to highlight this commit because I think this would allow to query value on the fly from any address and to use this value to compute curve value.


Note that the change from `Boolean` to `boolean` created an issue with the consensus tests.


Рабочий вариант? Можно забирать?


Ist ne super Idee. Musst du mal schauen. Anscheinend sind da ja Kollegen aus den höheren Semestern schon hinterher eine Plattform zur Projektplanung zu programmieren, nicht das die genau das Gleiche machen. Sonst viel Erfolg.


but as it is done now it seems you will not be able to save/load it... I need to work more on this


We backed something out of master? I thought that was just on the release branch.


On **kwargs: in this case generalization is not worth it because common attributes are generally set explicitly, and default values are useful mostly when temporary objects are created, not the ones that will be saved to mongo.


сущность Address имеет id того типа, который передается в женерике: Integer. дальше в коде в одном месте id в аргументе имеет тип int, хотя в женерике указан Integer; в других местах тип id указан верный.


According to http://docs.opengeospatial.org/is/12-063r5/12-063r5.html, section C.4.1, the DATUM is mandatory in a GEOGCS WKT, and the SPHEROID is mandatory in a DATUM WKT, so there should be no need to handle this case.


This does not work in IE9 and below. We either need a polyfill, or drop the HTML5 validation in favor of a javascript implementation.


(see quilt -p n and -p ab, though I didn't explicitly set the latter, but that accounts for the change it made)


master [still builds on Xcode 7](https://github.com/mapbox/mapbox-gl-native/blob/3a1f62e6b9e1e204dde3dd3cf3736cebc1ac0f45/.travis.yml#L31), so it’s Bitcode-enabled.


Finde die Idee auch sehr vielversprechend! Wie schon gesagt könnte man das ja sehr gut mit meiner Verbinden. Wäre je nach dem wie es sich sonst so ergibt dabei.


Mmm... I dont think many secondary indexes per table is a common use case.


Agreed.

I gave a quick look at OMPI's uses of obj->*cpuset. I couldn't find anything that uses disallowed resources, except opal_hwloc_base_cset2mapstr(). This function will now report things within the cgroup.

By the way, this change will simplify things since you currently have to do and(allowed_cpu,online_cpuset) multiple times to get the set of allowed and online PUs. You will just use obj->cpuset instead everywhere (and that's something you would have to change for hwloc 2.0 anyway because online_cpuset is gone).


Yuck.  Why are we now catching this?  Could we make it assert in an exception filter, but still throw?


Buuuuu...., it still happens! So it's still not fixed. I need to continue investigation.


Awesome job! Weren't you able to make the standard ES2015 imports work?

i.e.:

```ts
import * as runSequence from run-sequence';
```
instead of
```ts
import runSequence = require('run-sequence');
```


2011 :smile:


That is actually pretty cool, not sure why platform don't use it, though :/.
IIRC, ctimer_init() is just a wrapper for a process_start(&ctimer_process) ... should we keep it?


@kumajaya http://postimg.org/image/g1gqs9mu3/
Ridiculous... 9h


:tada:


Ich finde die Idee auch richtig geil. So "Entscheidungsdinger" sind immer gut. Würde dann halt auch zu irgendeiner Auflösung tendieren und schauen das man einen Ansporn hat. Aber mit ein bisschen Atmosphäre und dem richtigen Look hat das bestimmt gut Potenzial.


Indentation is off here too


Ah you got it fixed :smile:


this is super cool


Okay ...

1. Are you saying that this code is badly mangling something it shouldn't when iATS is co-existing with other payment processors? Is this new for 4.7 due to some change, or an issue with earlier versions as well?

2. Maybe I'm not reading your code correctly, but it looks like your new conditional is never true - the array keys of the _paymentProcessors array would be the full class names I think.

Conclusion: I'm confused about what problem you are solving and how your patch actually achieves it.


Not sure if I think that this is a good idea.


Dito. Hätte da nicht so die Muße mich dahinter zu setzen die Aufgaben zu entwerfen. Aber wenn man es gut macht bestimmt ne super Idee. Viel Spaß!


The message should be i18n-able, or in German. Maybe it should even be provided with the server response.


foo


Do we need cold tasks here?  Could we just change to `Task.Run`, or `TaskFactory.StartNew`?


Can you add "on" here to match the other one? `(on line 2)`.


isGoToHistoryCalled is better (make sure to follow the complete camelcase or underscore)


@manish172 I am quite sure it's not a big cost, but you are right => e575083ff21d552720228a8eb5ebb4054b02eb48


No no no. Bucket names should be splitted like in link I'm provided. Unfortunately, there are no other ways to translate them properly into other languages. All mods which I know have separated names for each bucket name.


hey


Looks good!

It would be nice to preserve comparison operations though.

Maybe something like:

- `AnyBalanceType x AnyBalanceType  -> compare balance, reserved`
- `AddressBalance x AddressBalance -> compare balance, reserved, address`
- `PropertyBalance x PropertyBalance -> compare balance, reserved, propertyid`


According to reveal's doc:

> Flags if the script should load after reveal.js has started, defaults to false

Because it is a plugin, you want reveal to be loaded first, so `async` to be true.


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


by putting a dollar sign at the end of the preceding regex, we only match if partition number is at end of block device pathname string


No, that would be (< 0 (length (frame-list)))

Here I test if there is less than 2 elements in the list (there must be at least one in this context I believe). Motivation is: if we are on the root window we should delete the frame (and not only the window), but we can only do this if there is another frame somewhere. Otherwise we get an error.


"adde more thigns to lecel 4" Did you write the commit with your fucking head, dude? xD


another foo


but this is solved by this commit 371d1df8036b6993b8d860324408e8a94d038c67


When are you going to create a new version in NuGet for this change?


@Adaptivity That is a crap ton of buckets though (average 4 per booze, so around 20 buckets (+ 1 extra for ale)), I'll see what I can do though :)


But we still need to be able to construct a  drmModeModeInfo object for drmModeSetCrtc(), so we can store it as it is as well.


Debugging statements should be completely cleaned up rather than just commented out.


Agreed, but then I suggest a more high-level way of doing so. This is a compile-time decision, so even better if we can do something similar to what is done for rtc, rtc_init() in platform/galileo/core/sys/clock.c . Then on the eth code you check on compile-time before populating pci_addr accordingly to the chosen interface.

Rationale: I will be slowly proposing that we move more and more to compile-time configs like that whenever possible. Goal is to end up with a cpu-conf.h or soc-conf.h on the platform layer for the cpu layer (drivers) to consume.


@RubenVerborgh, [fixed](https://github.com/shower/core/commit/5617774faee73cc1376c70f619dbb4b75f4db44e), thank you


I found a bug: when player 1 dies, player two will be drawn as player 1, i.e. becomes green instead of blue.
<bug


for kernel building you can use build repo from Cyanogenmod https://github.com/CyanogenMod/android_build/tree/cm-13.0


hey now foo


Thanks bud.
I was able to use ES2015 syntax but for some module (can't recall which ones) it was generating an tslint error, so I kept the hybrid one, will re-investigate.


@alombarte I was waiting for the merge to devel to start moving stuff around to follow PSR conventions, specially taking some examples from the sifo3-beta. I have some tests but as @obokaman-com says the autoloading is an issue.


This also fixes https://github.com/OpenDataServices/cove/issues/97


これ、structじゃなくてObjective-Cのクラスとして作り直したほうがやった感が出るカモ？


As longs as you are following conventions (`Video` and `Review` class in this case), you don't need to set `foreign_key` manually.


Addressed latest comments.


You got I booting yet, my first build hanged on Google logo and fucked up storage a bit, my second one I will test in approx. 2 hours because I have driving school now.


Well, this isn't too much actually. At least better than this ton of names: https://github.com/Glassmaker/ExtraTiC-Localization/blob/master/ru_RU.lang#L833-L4831 :smile: (en_US.lang only 500 lines :) )


If you build AOSP rom you can implement button light like in lollipop

https://github.com/ProjectG/platform_frameworks_base/commits/aosp-5.1

sorry for very bad commits titles..


Fixed: c758c9f6501a4636b0657fae856143ea554a9ad1
(Using stronger typing flushed out some cases where we are cheating, but I don't want to rewrite all those tests, so I converted it back to `Boolean`.)


hey now


Hopefully in the next hour or two
On Oct 7, 2015 11:59 AM, "George Alatrash" <notifications@github.com> wrote:

> When are you going to create a new version in NuGet for this change?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lefthandedgoat/canopy/commit/a5705ca997e2704d2969ed1191f857bae3050cfc#commitcomment-13643779>
> .
>


You can combine these assertions using the `be_a_new` matcher.


Looking at the PROD logs, I saw this `session.domain is deprecated, use play.http.session.domain instead` warning:

```
2015-10-07 16:53:26,334 [main] WARN  application - PROD.conf @ file:/subscriptions/subscriptions-frontend-1.0-SNAPSHOT/conf/PROD.conf: 14: session.domain is deprecated, use play.http.session.domain instead
```

...which led me to this line of configuration above. As it turns out, we don't want this configuration, whether for `session.domain` or `play.http.session.domain` - because it causes the play session cookie to be dropped on the wrong domain, leading to these errors that @mario-galic and @afiore experienced earlier today (basically, could have happened any time since #236 went out): `Sorry, we are having problems authenticating your email address
Anti forgery token missing in session` :

![image](https://cloud.githubusercontent.com/assets/52038/10344796/363e2c0c-6d1d-11e5-815d-dedd81e4bbd7.png)

See the cookies we end up with:

![image](https://cloud.githubusercontent.com/assets/52038/10344793/2d7ea376-6d1d-11e5-895e-e906b125bb1f.png)

We need to _not_ drop the Play session cookies on `.theguardian.com` - they should just be on `subscribe.theguardian.com`. The Identity cookies _should_ be dropped on `.theguardian.com` so that they can be used for authentication across the site.


Looks sane to me, but can we have a test for this?  It shouldn't be too hard I think in `tests/admin-test.sh`.


foo


Can just do `ostree_repo_set_ref_immediate` here.


Your spec says it should redirect but your assertion is for a render...be careful of out of sync spec descriptions.


I do!

I understand the value of what we learn from the other ports on Contiki, but I believe that we should try to keep the current design for x86 ports as much as possible.

platform/ is the layer that knows Contiki core (kernel). cpu/ is pretty much an agnostic layer that knows everything about the soc/cpu being used and its architecture.

platform/ provides information to Contiki through platform-conf.h. platform/ initializes the cpu/ layer with what it needs, choosing its configuration details. platform/ one day may provide a *-conf.h to be consumed by cpu/.


:+1: Thanks!


As long as `to_param` returns `slug`, you can just pass in `video1` and let `to_param` to the work for you.


Не очень выразительное имя. `executeResponses`?


In the future, consider using underscores for your names, they are great replacements for spaces! eg. shape_reported


I would prefer this encapsulated into the utils function like `is_nonstr_iterable` and `is_str`, so that we can reuse it elsewhere.


First need to get it booting


А при ошибке как вернуть ответ?


Pedantically speaking they should be (*volatilue unsigned int *)  or better yet uint16_t so its obvious to anyone what size is wanted. In this case it doesn't matter as they have to complete before the function returns.


Be sure to include as a comment the answer to the question, but in this case you did the right thing for your procedure!


@relakx DWMR (done with my review). It works! Good good. Lean into this work.


Pretty sure partnet.com is not in the about page.  Need to figure out how to deal with Companies as contributors


I'm not so sure you need the stacked argument here!


No. `if (defined %Exporter::)` should not import Exporter. Only `use Exporter` or `require Exporter` should do that.
`if (defined %Exporter::)` should return undef


Use `$(CC)` rather than plain `gcc`, so that we can switch to other compilers like `clang`.


Should be EINVAL - it's a known ioctl with a bogus argument...


The local variable assignments are not necessary. The variables are immediately discarded anyway.

```ruby
let(:the_league) { Video.create(title: "The League", description: "Bros", category: comedy, created_at: 1.day.ago) }
```

is fine.


Nice!


Use `$(RM)` rather than plain `rm`, so that we can clean targets properly.


Yep, I will try to boot up this today


@eneim I should have told you that refresh is:

```

    public void refresh() {
        removeAllViewsInLayout();
        layoutChildren(0, mAdapter.getCount());
        setTopView();
    }
```


The "#2" was meant as the second commit, related to the one just before it. Not related to what it links to.


Ok, sounds good. Updated.
Thanks for the link to the spec, that will be quite helfull if I run into something similar in the future. I was having a bit of trouble finding it.


Rather than do this to instantiate your variables, use `let!` which is not lazily evaluated.


hey foo


@Adaptivity Touché, *grabs some eye bleach*


`let!` will mess up this spec, but that means that you should probably just create your videos inside of your specs.


Well, to be honest, that pinterest stuff doesn't look that great, haha. I could have used some variables to declare sharing descriptions and image.

The pinterest API is very messy itself though. First of all the needed preg_replace to handle JSON and if you'd like to use it in a more flawless way, you'd need up to four requests, if I'm correct on this.
It only handles the accurate URL without adding up the different ways of reaching the target. (http://www.my.url doesn't add up with http://my.url )

Have a look at the URL Breakdown at the bottom of the page for example:
http://pinplugins.com/pincount/?url=http%3A%2F%2Fwww.google.de

//

Linkedin is not a big deal, i guees and Xing is a very common business platform in Germany, which has the same workaround as Google+ for collection the share count.

//

Btw: Heard of the planned [shutdown of the Twitter Sharecount](http://www.i-programmer.info/news/155-social-media/9048-twitter-switches-off-json-api-no-more-tweet-counts.html)? Let's see, where this is going.


nice!


Doesn't look like you came back to fix it... ?


Could you please handle the `RCTWebSocketManager` to `RCTWebSocketModule` name change in JS?

```
var NativeModules = require('NativeModules');
var RCTWebSocketModule = NativeModules.WebSocketModule || NativeModules.WebSocketManager;
```


Big question! I like it


More local variables you don't need.


Did mako guys got it booting yet?


hey


Should we maybe go with uint32_t for freq?


I love this Fabian, a lot. I can't wait to help you out and see the final result! Try looking at weird things like animal behavior as well! They have a good sense about stuff like this.


@Caberhagen super, danke für die Anpassung ! P.S. ein ß geht in html so: ß


yes, I even used it unprefixed in other tests, will get rid of the remaining ones...


@paramaggarwal It's called `WebSocketModule` now. What's the need of `NativeModules.WebSocketManager`?


nope.. not yet


done


I considered doing this for the SimplePwo too...let's talk about options here.


Of course. Sorry for the confusion.


Может тут лучше ошибку кидать, чем проглатывать молча?


Thanks for spotting my error in my directions for this commit and doing the right thing re "set by LRS"!


Nit: the commit message should be imperative ("Add" instead of "Added")


I did amend the commit message. There were actually two issues:
1. If one clicks on the send button, "Object [Object]" is displayed on chat.sdslabs.co and a random number is displayed in the public channel of slack. One needs to type a message and press Enter instead of clicking on the send button.
2. If someone sends a null value to server, instead of the message string (by typing sendMessage(null) in console), the app crashes and doesn't work until restarted.


A bit confused by these two lines of code here. I think you meant Unemployed majors?


recent comment package


Only .js?


Let's talk about why we need an interface for this


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


@maxiloc Thanks! I mentioned this because if "Queue Enabled" and "Indexing Enable" is set as "No" then it runs the indexing directly for that store and it loop through all stores again due to which It was increasing the DB overhead.


@palamago aca yo estaba pasando los options para que se desabilitaran los controles y tal. Donde querés ponerlos ahora?


Ok I've just seen the other commit :)


für "getPrices_multi" braucht es noch die beiden Funktionen, nicht für "getPrices".


That was fast, thanks!


@manish172 Yes I got your point. Thanks for reporting that


Also add codecov.io


Ruckdeschel ;)


fstab.qcom - my mistake


sounds good, :+1:


Ah yes, reacquainting myself with the problem — we technically build with bitcode, but it's broken and you have to disable it everywhere, per #2332. Will we add a "(and actually works)" to this line? :wink-wink-knudge-knudge:


Actually there isn't any description in your commit message that is why I am asking you.
Mention the same in commit summary.


@jramscr

commit: 966351c

1) Change fix with keep space one line on the view
2) Performed testing to Project Model.


@relakx DWMR. More success. :cherry_blossom:


Those ones have broken the headlines again:

https://doc.owncloud.org/server/8.1/admin_manual/configuration_user/user_auth_ldap.html#owncloud-ldap-internals

Those have to be sub-headlines of the "ownCloud LDAP internals".


@cvrebert there were a couple of uncredited cut & paste from bootstrap's docs, I hope the attribution is fine now


Sehr coole Idee! Unglaublich praktisch um ein bisschen Oranistation in Projektarbeiten zu bringen.


기업에 대한 마크업이 추가되어야 함


All in all, this is fantastic work, Anil! Seriously I have very few comments on your code, it is very well documented and very clear.


Will do.


Yep.  That works as well.  I cloned a new version of your repo, made the change and it processed my collection without error.  Feel free to kick back my pull request.

Well done!


Seemed to fix it for us! Still have yet to test with wp-super-cache enabled, I'm still suspicious that it would cause problems for this.


Just a note about i18n properties file guidelines: Try to always have a new line at the end, so that the GitHub comparison will not have a false positive that something changed on the last line. Otherwise, this looks awesome!


@satya164 We ship JS updates to older versions of the app shipped to customers. So for example while the app itself has RN 0.9 - we ship JS bundle updates that use JS from RN 0.11. We had a similar issue when `RCTReachability` was renamed to `RCTNetInfo`.

It's ok if this is not a pattern that the team would like to follow. I generally make a copy of the JS file with the appropriate backward compatible handling and use that instead. Thank you for adding this to Android also!


I think we should check what google changed in ramdisk in device tree for nexus 7 2013.. if you can see only google logo propably this is ramdisk issue


another comment


@jneidlinger I can't open HTTP link, `c` is missing, should be: https://codereview.qt-project.org/#/c/121545/


Nit: now that you live in `Memory.size` you might want to undo this change.


Gefällt mir gut die Idee. Auch das man daran immer weiter ausbauen kann ist ein guter Aspekt. An sowas würde ich mich auch gerne beteiligen.


Just a slight wording difference for the keys - maybe in the i18n refer to them as 'species' instead of pokemon, just to keep with convention for the rest of the program.


Same here. Why is this plain `int`, btw?


@mglukhikh there's a typo in some of the messages: "that" instead of "than". Also why "better" and not "greater"?


foo


gnar gnar. It would be helpful to have some notes in the video tutorials about these breaking changes. I know how fast things move so maybe perhaps notes in the README


I have worked on this project before with many students before so I can DEFINITELY be a big help if you ever need to bounce ideas off of me :)


:clap:


Why not just this instead?

```js
return map.mappings || map._mappings;
```


I don't think those comparison operations follow Java conventions. If objects have different fields they should probably not compare as equal. I do see why it would be convenient, though.

Maybe a better implementation would be to use only one type (`Balance`) and have the following:

`omniGetBalance` will return `Balance`
`omniGetAllBalancesForId` will return `Map<Address, Balance>`
`omniGetAllBalancesForAddress` will return `Map<CurrencyID, Balance>`

And `Map` should probably be a `SortedMap`.

This would actually simplify some of the code in the consensus tests because the lists are being converted to sorted maps anyway. Also we have the Groovy type `ConsensusEntry` which could be replaced by `Balance` (if we write an iterator in Java)


Added some docstrings: https://github.com/yandex/mastermind/commit/fe4d0bd4a27c27f9aa36d19738b12fc2e1540b20


Yeah, I reverted storage changes: this is build which is up to date with current device and kernel tree marshmallows release: http://dl.kevinjoa.club/marshmallow-test-only-for-me/geehrc-bricked/aosp_geehrc-ota-8.zip

Will test it at 9pm GMT+1 (u can flash on your own risk)


Deine Idee ist bestimmt für kleinere Künstler besonders interessant, wenn diese ihre Musik so leicht weiter verbreiten können ohne dass gezielt danach gesucht wird. Viel Erfolg damit!


If I activate an environment and then just use "python" I only can get the `active` python. For installing python2 into python3 kernel I have to use the full path to `the other` python here, don't I?


It does. Works fine. Thks


@jneidlinger Looks like patch applied partially. For example I see changes in qcoretextfontdatabase_p.h file: https://codereview.qt-project.org/#/c/115138/2/src/platformsupport/fontdatabases/mac/qcoretextfontdatabase_p.h


this can be done a lot more elegantly. Check [this post](http://math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000)


This is great! well done


kann dem nur zustimmen und finde die Idee auch mit unter am ansprechendsten


Лучше назвать как-нибудь стандартно, как называется задача – её название и соответствующее сокращение широко известно. Например, "(find) longest common subsequence". Аналогично для других имён вроде compare_params, compare_matrix.


`display-name`, how cool! :+1:


Nice job!


LGTM


what will this metric name end up looking like?


pandas has a built in .count() method that is faster than len because it is built on top of C!

`killings[killings["year"] == 2015].count()`


LGTM <3


Oops, this is just a leftover from some earlier churn.


hey foo


Great thank you!


"list" method acts like mongo query filter, i.e. it will not filter by group_id and will return every group history document. So if the goal is to get all group history records from mongo this method works fine with ids == None


Good question :smile: Hard day at work, ill fix it tommorow.


Here is a good example of pulling in the environment for proxy/ssl:
https://github.com/ChadMartinson/RV-Dashboard/blob/d49c294562662305db0c5dea8cbb442b742e83db/jspm_packages/npm/bower-config%400.4.5/lib/util/defaults.js#L5

You would need to be able to pass the "request" option array through "get-down", but you could get the correct options from the environment like this:
```javascript
var options = {
   proxy: process.env.HTTP_PROXY || process.env.http_proxy || null,
   env: { HTTPS_PROXY : process.env.HTTPS_PROXY || process.env.https_proxy ||  process.env.HTTP_PROXY || process.env.http_proxy || null },
   strictSSL: process.env.strict-ssl || true
}
```
Realistically, that should probably be done in "get-down" if possible.


Should the factor here be 0.5f instead of 0.5?


Adding team members to ReadMe file for pull request and merge practice


comment on the last line


In the future, please try and include the file affected. It is definitely great that it is on the BRANCH named after the module. But it helps later when searching through the log to see the files affected in the title line.

The first line is usually considered the title of the commit. So you want to limit to a brief summary:

```
Revise, Reorganized Slides and Fixed URLs in Further Resources

### BODY OF COMMIT ###
```

Within the body of the commit talk about the work that you did and answer the following questions:

* Why is this change necessary?

This question tells reviewers of your pull request what to expect in the commit, allowing them to more easily identify and point out unrelated changes.

* How does it address the issue?

Describe, at a high level, what was done to affect change. Introduce a red/black tree to increase search speed or Remove <troublesome gem X>, which was causing <specific description of issue introduced by gem> are good examples.

If your change is obvious, you may be able to omit addressing this question.

* What side effects does this change have?

This is the most important question to answer, as it can point out problems where you are making too many changes in one commit or branch. One or two bullet points for related changes may be okay, but five or six are likely indicators of a commit that is doing too many things.

Your team should have guidelines and rules-of-thumb for how much can be done in a single commit/branch.

> [Reference](https://robots.thoughtbot.com/5-useful-tips-for-a-better-commit-message)


Sehr gute Spieleidee. Klingt spannend. Viel Erfolg bei der Umsetzung!


Travis docs say that `phantomjs` is in `PATH`, so it should work without `./`.


Did you mean this:
if (!IRL_BLOCKING || !IRProtocol) {


should display


o/


Because that would mean that you can pass any datetime-like object and expect this method to return you desired records. As for now, this is not the case.


Updated commit summary.


I would make this abstract along with the class


This is wrong that line (and the following) should be:
```java
        ClusteredCacheManagerEntity clusteredCacheManagerEntity = null;
        if (accessMode.compareTo(AccessMode.VALIDATE) < 0) {
          EntityMaintenanceRef<ClusteredCacheManagerEntity, ServerCacheManagerConfiguration> mmodeRef =
              connection.acquireMaintenanceModeRef(ClusteredCacheManagerEntity.class, EntityVersion.getVersion(), name);
          try {
            switch (accessMode) {

```


hey this is a comment


license added per GitHub recommendation


test d'un push de mon bord
Kismony Ladouceur


Это не print, по крайней мере насколько я понимаю, основное для этого метода – вернуть результат. Печать там только отладочная.

Строка не mutual, а common.

И не string, а subsequence.
(Под)строка это последовательность *подряд* идущих элементов. А подпоследовательность – это последовательность элементов идущих в том же порядке в исходной, но не обязательно подряд.


I wonderful observation. There are pros and cons in this but I like to think about it like this to help me make a decision.

If I was going to make the `readAddressInfo` into a class object would I define scanner in the constructor to supply a field or would I declare it in my methods.

I would probably declare it as a field and let each one of my methods utilize the field. This is similar to passing `keyboard` in as a parameter.

:+1:


Just putting this here for your consideration. It doesn't remove any lines of code, but it does remove the redundant usage of allowedMethods.

```
allowedMethods ++ resources.find(_.path.findFirstIn(path).isDefined) match {
  case Some(matchedResource) =>
    logger.debug(s"Matched path $path with configured resource: $matchedResource")
    matchedResource.methods
  case None =>
    logger.debug(s"Did not find a configured resource matching path $path")
    Nil
}
```


Possibly.  I was trying to keep the change minimal here to make it easier to find the deadlocks.