Thanks, I forgot about this, too.


@altidb


This is sorta required if you want `./setup.py command` to work.  If you want to drop support for this, `chmod a-x setup.py` as well.


This indent level is not necessary, switch to something like:
```
if res.body.nil?
  return
end
```


Fixed by 9792ec64e7752469162eec4c5459a2b20df00f43


@IncoCode Good idea, done: c41b7b4580ffd89cf38ba273bc4666cac0a73fa0.


You left in a console.log :)


Fixed by 7fb7edda4cb82b925b93af7e2e0cd80b85a0e496 and d6696753092855309c07eba6c8c2fe6bf9e98f48


@mgedmin Please see https://github.com/SirVer/ultisnips/issues/305. Not a bad idea per se -  but it won't read your mind what you want if both actions are possibilities and I am tired of the bug reports.


@altitdb


> I think there was agreement we could loosen these key names,

Can you point me at the issue that described the new key restrictions?


Doh, again -.-


@jakob-WMDE oops, I wanted to submit this as PR so you could double check but accidentally put it onto master...


The thing to check being PHP 5.4+ -> PHP 5.5+


I'd much rather see all of this "driver code" in methods as well.


Fixed on `master`.


Yes, it implements the Resizer as the respective `ReplicationControllerResizer` upstream does.


Ok, verifique se está correto antes de fazer o pull request por favor: b2ccf897cf6cb168f3ffc792fd5af27f488cf8dc


@nieltg It's still broken.


This is a great start! I recommend using trusty64 as our basebox: https://atlas.hashicorp.com/ubuntu/boxes/trusty64


Looks like this is out of sync with the @brand-warning .less variable now - not sure if that's what you anticipated or not.

https://github.com/USPTO/designpatterns/blob/gh-pages/usptostrap/less/_variables.less#L14


I know, you told me, but I was just doing it anyway because it makes it easier to work with the strings in the future if one decides to add more stuff in different places.

And no, I am not going to go crazy on replacing all of them ;-).

On 11 May, 2015, at 11:21, Richard Gowers wrote:

> It's not important, but "{}".format() (ie without {0}) is supported in 3.1+. Just before you went crazy trying to fix all the empty ones.
>
> https://docs.python.org/3.1/library/string.html
>
> �
> Reply to this email directly or view it on GitHub.
>

--
Oliver Beckstein * orbeckst@gmx.net
skype: orbeckst  * orbeckst@gmail.com


ok


When I ran the tests I found a few small issues in the way things were printed. You might want to review the fixes I made:
https://github.com/exosite/exoline/commit/1286fc595e53995b7542fe83bdd08d25e19102b2

Also, we should make tests for this command. Here's how to run tests:
https://github.com/exosite/exoline/tree/master/test


I am confused on that. My code is different than what was in the RP book. Command Line tells me it is 2.7.6 but maybe when I used IDLE, it is 3? I think that must be my issue, as I worked through RP exercise in IDLE. Do you know if I can revert IDLE for 2.7.6 ?


**%text-in-one-line** need to move into core/_util.scss, becouse it used not only in this file.


Okay, I just pulled the latest source and compiled and now I get the same list of errors as @danicampora
I certainly didn't get that the last time I tried to build it (2 - 3 weeks ago); at that point the build still made it all the way to `input.c` where it complained that it couldn't find `readline/tilde.h`
This is a new set of errors for me. Thanks for checking, I'll try to dig in to this.


Nit, make this "resizeLong" and "resizeExample".  Upstream is wrong.

Otherwise, LGTM and I'll merge you after you make that change.


Фон с точками под шарами убран и текст вроде как должен читаться в Firefox, Chrome и IE начиная с 10 версии. Шарики не получилось равномерно сделать https://github.com/Vika-Sokhinova/Web-pril/tree/master/%D0%BE%D1%82%D0%BA%D1%80%D1%8B%D1%82%D0%BA%D0%B0%20new


Ok
Посмотрю


This follows the pattern elsewhere in the file. `shared_from_this` returns a `Peer`, so a downcast is needed to access methods that exist on `TCPPeer` but not on `Peer`.


dout(20) << __func__ << " final fiemap " << m << dendl;
(otherwise we loop even with debug = 0)


Thanks Semyon, will take a look. Would it be possible to create a test (in test/index.js) that covers the new functionality?


https://github.com/GoogleCloudPlatform/kubernetes/issues/6589

----- Original Message -----
> > I think there was agreement we could loosen these key names,
>
> Can you point me at the issue that described the new key restrictions?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/deads2k/kubernetes/commit/dc17844fb29e004c7e8f6d8f0e60605e0daf3935#commitcomment-11140005


la clase Location que retorna LocationManager tiene datos de tipo "double", respetando eso, se creó la clase LocationService, pero DatabaseType los define como REAL y en DatabaseTableRecord se llaman el setter y getter es "float", si float va ser el tipo de dato definitivo, habrá que modificar ambas clases para que coincidan y no tener que estar haciendo casting.


Actually, I'd make the last line be the usual args + return code, and include m at the end:

  dout(20) << __func__ << " " << offset << "~" << len << " size = 0 (" << m << ")" << dendl;


Hi,

can you make a pull request for that?


can you move this to a predicate method so I know what the meaning of this condition is?


Thanks. Zoin is waiting for Anh’s specs and instructions for those. We probably do want to get those to Zoin pretty quick, Anh, since people are referencing the pattern library and we made the irresponsible move of changing the color without a release.

Thanks,
Arva

From: Brian Mathews [mailto:notifications@github.com]
Sent: Monday, May 11, 2015 2:29 PM
To: USPTO/designpatterns
Subject: Re: [designpatterns] gold color changes (9870c08)

Looks like this is out of sync with the @brand-warning .less variable now - not sure if that's what you anticipated or not.

https://github.com/USPTO/designpatterns/blob/gh-pages/usptostrap/less/_variables.less#L14

—
Reply to this email directly or view it on GitHub<https://github.com/USPTO/designpatterns/commit/9870c08094c5e20bd46458a2d863f87dbd7e00f4#commitcomment-11140060>.


Hmm, I'm not sure how I came to the conclusion I did originally, but it would appear I was wrong.  In testing thus far, I've found Marco's approach to be more or less equally quick & effective (perhaps even moreso given the fishtest results), while also being much simpler.

One place where I know it can be improved (at least tactically) is by also checking for a limited number of legal moves as I do in matefinder.


I would love to see this commit in upstream repo. I've just debugged a chocolatey issue https://github.com/chocolatey/choco/issues/277 and increased the same value to fix StackOverflowExceptions while running choco.exe.


It could be that stdin/stdout/stderr are being redefined by some macro (that's not our macro).  Can you look in build/genhdr/qstrdefs.preprocessed.h and see if you can find `Q(stdin)`?  It should be just before `Q(version_info)`.


Great @paly2 :+1:


Put these above with the other select functions


Remove newline from query


Those changes might better be separated into 2 commits.


We need to check if there is already an index set, and if not use the inferrer to get the index name.  So it should be something like:
```
if (pathInfo.Index.IsNullOrEmpty())
    pathInfo.Index = connetionSettingsValues.Inferrer.IndexName<T>()
```


I see, GainAbilityAllEffect applies only to Slivers on /the battlefield/. What would be the correct way to make it apply to Slivers in the players' hands?


Now ok for Master Levels :-)

https://github.com/a-detiste/game-data-packager/commit/9b918194e05a1a726104f9b8c5714c3810df21ca

Now for nerve.wad it's a bit more tricky; as the .desktop file is generated that way:
`Exec=boom -iwad /usr/share/games/doom/doom2.wad -file /usr/share/games/doom/nerve.wad`

Depending on `doom2-wad | freedoom` in the yaml without changes to the python code
would mean having possibily broken .desktop files.

Is this -iwad parameter mandatory ? For all engines ?


tested and working


This is only used in one place. Better to inline it.


When calling the testType() methods, I know they will run quickly, but because of the async nature, there's a chance the promise could return before the test finishes. What about using a callback or the Async library here to control flow?


The logic looks good to me. I think it would be nice if it didn't have to run through the whole deck when the halves are actually not a match for the deck.

I would recommend breaking out your conditions into named methods.

Remember that ruby allows us to write expressive code. Use that as a tool to make sure people don't have to read every step to understand the higher level function of the code. As in, at first glance, I would like to be able to understand the general strategy, even if I don't know exactly every detail.


This hunk makes the line fit the comment that's just above.
```
        // Create a timer to wait for current SCP timeout / 2 before accepting
         // that ballot.
```
Without it, the large-network simulation tests fail when the timer triggers in the wrong moment, namely when other nodes are in COMMITTING phase.

This problem might very well be preexisiting to the prefetch code, since before prefetch large-network simulation tests were failing with 4 or 5 nodes. With this `mBallotValidationTimers` fix, we can run much larger networks.


So, if I'm reading to correctly, it means that the returnUrl should be in the format provider.fragment? So, for example, "coreapps.patientdashboard/patientDashboard"?  Even though I generally like this better (though I'd use a colon instead of a period to maintain the same pattern we use for htmlforms, ie "coreapps:patientdashboard/patientDashboard") I think elsewhere "returnUrl" is just interpreted as a URL, so we probably want to go that route for consistency?


Might be a micro-optimization, bit if you did this in your iff on what is now line 58, you'd avoid all the existing checks when it is avro.


It's `color('grayscale', 80)`, not `setting-get`


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


Should be `SRC_LUACORE` instead of `SRC_LJCORE`.


You're such a moron, I swear H3C4T3. Bitching about the .gitignore like a total noob on multiple SWG projects. It's one thing to be inexperienced, but then to start drama, sigh.

http://swganh.com/anh_community/viewtopic.php?f=21&t=1536&sid=4d261c2c5d55d5850a3a50df47837283#p5958


NIT: These really should be after the imports


Use `.sorted()` in the stream before `.collect()`


Can use `toList()` with `.sorted()`


Now that sounds like a nice solution.  I'll do that.  I like to make eclipse happy.

--- Original Message ---

From: "Jeff Butler" <notifications@github.com>
Sent: May 11, 2015 9:26 AM
To: "mybatis/generator" <generator@noreply.github.com>
Cc: "Jeremy Landis" <jeremylandis@hotmail.com>
Subject: Re: [generator] Remove build helper plugin.  Not needed and it obscures a function of (2ef881b)

If this really bugs you, you could add the build helper in a profile that is only active inside eclipse - just make sure it isn't active by default.

---
Reply to this email directly or view it on GitHub:
https://github.com/mybatis/generator/commit/2ef881b7001a11f9a3a1b97579172565a8c8b725#commitcomment-11133711


You can jettison test-helpers as well


All DTO(s) from my point of view are more related to domain package. Because this ones are going to be populated by response model from db query.


Todo: add download link for AS


I would make this 'predators become prey, but price is unfair'. I'd even consider getting rid of 'but', but I'm not sure


Fixed in v1.21.0


@jomo  huh? lol


Damn, beat me to my PR.


lol.  True! :D


Hola


Sounds like a bug; can you open an issue to help us track it, please?


This will loop forever if `getCompactionSet()` returns an empty list, which can happen if a shard is larger than `maxShardSizeBytes`.


Maybe with a full stop?
predators become prey. price is unfair


As things stand, this change breaks Mac OSX compatibility as the library is named "libcconv2_cudanet.dylib" on those machines (and part of our reasoning for using `find_library` in the first place).  Perhaps we could utilize `sys.platform` and set the extension based on underlying platform?


You're right, it can just be passed directly to BaseInstaller. Guess I thought I might need it again but ended not needing it.


This issue should also reference https://github.com/BroadleafCommerce/BroadleafCommerce/issues/1348


@marek-safar This breaks building with monolite:

```
Creating the per profile list ../../build/deps/basic_System.dll.sources ...
 mkdir -p -- ../../class/lib/basic/bare/
 MCS     [bare/basic] System.dll
 ../../../external/referencesource/System/compmod/system/collections/specialized/nameobjectcollectionbase.cs(86,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/collections/specialized/nameobjectcollectionbase.cs(99,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/collections/specialized/nameobjectcollectionbase.cs(156,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/collections/specialized/nameobjectcollectionbase.cs(195,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/collections/specialized/nameobjectcollectionbase.cs(211,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/collections/specialized/nameobjectcollectionbase.cs(764,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/collections/specialized/nameobjectcollectionbase.cs(814,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/collections/specialized/nameobjectcollectionbase.cs(834,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/collections/specialized/namevaluecollection.cs(64,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/collections/specialized/namevaluecollection.cs(107,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/componentmodel/IComponent.cs(33,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/componentmodel/TypeDescriptor.cs(3467,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/compmod/system/componentmodel/TypeDescriptor.cs(3513,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/sys/system/collections/concurrent/BlockingCollection.cs(52,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 ../../../external/referencesource/System/sys/system/threading/Barrier.cs(104,25): warning CS1635: Cannot restore warning `CS0618' because it was disabled globally
 warning CS8001: SDK path could not be resolved
 warning CS1685: The predefined type `System.Security.Permissions.HostProtectionAttribute' is defined multiple times. Using definition from `mscorlib.dll'
 ../corlib/System.Security.Permissions/HostProtectionAttribute.cs(38,22): (Location of the symbol related to previous warning)
 ./../../class/lib/monolite/mscorlib.dll (Location of the symbol related to previous warning)
 ../../../external/referencesource/System/compmod/system/collections/generic/throwhelper.cs(62,53): warning CS0436: The type `System.ExceptionResource' conflicts with the imported type of same name'. Ignoring the imported type definition
 ../../../external/referencesource/System/compmod/system/collections/generic/throwhelper.cs(327,19): (Location of the symbol related to previous warning)
 ./../../class/lib/monolite/mscorlib.dll (Location of the symbol related to previous warning)
 ../../../external/referencesource/System/compmod/system/collections/generic/throwhelper.cs(66,57): warning CS0436: The type `System.ExceptionArgument' conflicts with the imported type of same name'. Ignoring the imported type definition
 ../../../external/referencesource/System/compmod/system/collections/generic/throwhelper.cs(304,19): (Location of the symbol related to previous warning)
 ./../../class/lib/monolite/mscorlib.dll (Location of the symbol related to previous warning)
 ../../../external/referencesource/System/compmod/system/codedom/compiler/CompilerParameters.cs(350,34): error CS1061: Type `System.Security.Policy.Evidence' does not contain a definition for `Clone' and no extension method `Clone' of type `System.Security.Policy.Evidence' could be found. Are you missing an assembly reference?
 ./../../class/lib/monolite/mscorlib.dll (Location of the symbol related to previous error)
 ../../../external/referencesource/System/compmod/system/codedom/compiler/CompilerParameters.cs(357,38): error CS1061: Type `System.Security.Policy.Evidence' does not contain a definition for `Clone' and no extension method `Clone' of type `System.Security.Policy.Evidence' could be found. Are you missing an assembly reference?
 ./../../class/lib/monolite/mscorlib.dll (Location of the symbol related to previous error)
 ../../../external/referencesource/System/compmod/system/collections/generic/debugview.cs(105,17): warning CS0436: The type `System.ThrowHelper' conflicts with the imported type of same name'. Ignoring the imported type definition
 ../../../external/referencesource/System/compmod/system/collections/generic/throwhelper.cs(49,27): (Location of the symbol related to previous warning)
 ./../../class/lib/monolite/mscorlib.dll (Location of the symbol related to previous warning)
 ../../../external/referencesource/System/net/System/Net/Internal.cs(938,17): warning CS0472: The result of comparing value type `System.ArraySegment<byte>' with null is always `false'
 ../../../external/referencesource/System/regex/system/text/regularexpressions/Regex.cs(1356,13): warning CS0162: Unreachable code detected
 Compilation failed: 2 error(s), 22 warnings
 make[9]: *** [../../class/lib/basic/bare/System.dll] Error 1
 make[8]: *** [../../class/lib/basic/bare/System.dll] Error 2
 make[7]: *** [do-all] Error 2
 make[6]: *** [all-recursive] Error 1
 make[5]: *** [all-recursive] Error 1
 make[4]: *** [profile-do--basic--all] Error 2
 make[3]: *** [profiles-do--all] Error 2
 make[2]: *** [all-local] Error 2
```


This naming seems confusing. This function doesn't care about new or previous. Can you just use "traitCollection"?


@dpgeorge - I have this section in `build/genhdr/qstrdefs.preprocessed.h`; no `Q(stdin)`
I'll email the whole file to you as an attachment in case it helps.

```
Q(<module>)
Q(<lambda>)
Q(<listcomp>)
Q(<dictcomp>)
Q(<setcomp>)
Q(<genexpr>)
Q(<string>)
Q(<__stdinp>)

Q(encode)
Q(decode)
Q(utf-8)

Q(argv)
Q(byteorder)
Q(big)
Q(exit)
Q(little)

Q(platform)

Q(__stdinp)
Q(__stdoutp)
Q(__stderrp)
Q(version)
Q(version_info)

Q(name)

Q(implementation)

Q(maxsize)
```


name: "Not North: Performance"


yes, my fault, will change soon


I have issues with it when passing a byte string.


why does this belong in the core library. Seems like a problem that's specific to a single file.


Instead of the `else` branch below I'd add the `followedByCurrentUser` property here with initial value `false`.


well you removed the `.html` part of the links but hard-coded the domain, why is that?
```diff
- <li><a href="issues.html">Open an issue</a></li>
+ <li><a href="http://plotme.worldcretornica.com/issues">Open an issue</a></li>
```


I don't see how "provider.page" is better than "/provider/page". Generally it's best not to explicitly depend on the uiframework structure unless there's an actual benefit to doing so.

(Perhaps the issue here is ensuring that the framework adds the webapp context path first?


I looked, and I have:
```
Q(__stdinp)
Q(__stdoutp)
Q(__stderrp)
Q(version)
Q(version_info)
```


:trollface:


np, bro.


This doesn't seem like it belongs in this commit. Also, it's removed in the next commit.


This is creating a lot more objects per row. Is this a problem?


Why would you want the holocore gitignore to be empty? We can't upload clientdata and the odb is per server so there is literally no point in uploading them. And settings? That's obvious why you don't need those commited. Go away H3C4T3, you're trolling isn't accepted.


@jriguera: No, I'm not.  However, my tests have all been against the newest NetApp API.  Actually, I'm trying to turn your code into a library so I can use it for collectd: https://github.com/collectd/collectd


Good to place in the constructor the size of schools.
List<SchoolDTO> schoolDTOs = new ArrayList<>(schools.size());


both modified:   app/views/consumer/employee_dependents/_dependent_form.html.erb
deleted by others: app/views/consumer/employee_dependents/group_selection.html.erb
both modified:   app/views/consumer/employee_dependents/index.html.erb
deleted by me:   app/views/consumer/employee_roles/_welcome_msg.html.erb
deleted by me:   app/views/employers/employer_profiles/_welcome_msg.html.erb
both modified:   app/views/employers/employer_profiles/show.html.erb
both modified:   app/views/employers/family/_address_fields.html.erb
deleted by me:   app/views/employers/family/_form.html.erb
both modified:   app/views/employers/family/_member_fields.html.erb
both modified:   app/views/people/landing_pages/_family_members.html.erb
both modified:   app/views/plan_shopping/_plans.html.erb
both modified:   app/views/plan_shopping/thankyou.html.erb
both modified:   app/views/shared/_census_dependent_fields.html.erb


There's no doubt that whiptail and dialog are the nicer options. The select is only hit if neither of those are installed, but I still think attempting to show the user all the available options via select is better than falling back to the really old method of making the user know ahead of time which machine they want and what its OE name is.


No problem. Thanks!


Here's a simpler way of doing this.

```javascript
var following = loggedInUser.following || []
if (following.indexOf(this.id) !== -1) {
```


This was the first thing I saw, sorry I didn't see the 'resources' label :P :tropical_drink:


How elegant!


This may or may not be something we want to have a production. I feel safer leaving it out until we know for sure.


He's just challenged, it isn't the first project where he complained about the .gitignore, he totally doesn't understand it. He literally is a skiddy if you look his name up.


  > C++ language truly doesn't have a standard for this in the language definition.

Having worked /on/ two separate compiler/debugger chains in the past, I'm pretty sure it does.
Specifically, section 2.10, "Identifiers," point 3, says the following:

    (3.2) — Each identifier that begins with an underscore is reserved to the implementation
    for use as a name in the global namespace.

Note that a "name in the global namespace" may be a preprocessor define, and thus you cannot use these names in any namespace (including private class members) for conforming code.

http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2014/n4296.pdf

  > Well, even if it was left alone at its 1024 value, that single packet could be split up by
  > routers that only have a 576 MTU.

Having worked on device drivers and network internals, and written several book chapters on the subject, I believe I am reasonably well versed with how MTU works.

There is no coupling that I can see between the multiple buffers used to hold user data for transport, and the MTU used to talk to the network interface. If there is such a coupling, please point me at the code, because I don't want to miss it. Else, I will re-submit this request with a lower default per-buffer size. (Again, the implementation looks to me as if it supports splitting per-packet payload into multiple local heap buffers, presumably to manage fragmentation.)


This is probably fine, but I wanted to check it out and haven't had a chance to do so yet.


In bcrypt's documentation it says * if salit is specified as a number then a salt will be generated and used (see examples).*

So you can delete `bcrypt.genSalt(10, function(err, salt) {` and do only:

` bcrypt.hash(password, 10, function(err, hash) {`


huge effort AJ!


Very nice!


This commit doesn't include the updated version of `ax_python_module.m4`, so the `$3` argument will do nothing.


Right, I forgot this one. Will do.


Looks great! Merge away IMO.


@poppywis3 you might have two versions of IDLE installed then. There's the default 2.7.x that comes with OS X and then maybe a version of Python 3 that you've installed? You may try googling where the default IDLE lives on your comp. Or you could run things directly from the command line?


Never mind, I see why it was done.


we have a release-candidate target that this might interact with... that could probably be renamed if needed


This one can be implemented as a method object. And when you use it inside the route you'll be able to do

`req.user.follow(followUser, ..)`

It's more elegant to call it on the user object that wants to follow another user.


@pcottle You misspelled. Also, I would actually initialize it to `curses.color_pair(1)`. You didn't answer the question in the comment I had so I didn't know what you wanted ;)


bump. this is still confusing.


Great, I just setup git at school :D
Can you please delete the old RenderManager files as well as FurnitureShapes.h? I haven't figured out how to delete files via git.


Timbad, your mom did not you learned the courtesy ?
As regards the rest, indeed it is a statement, that you constantly lock a lot of files without reason.
Zachman61  i never say the gitignore to be empty, i say a lot of source file is lock with him.
but if you are not agree , you are free to go back in Closed Source.


I'm confused over why one is an `Identifier` and the others are declarations;


Didn't catch this the last time, but the other resources have listed this type as a "string" (for both params).  To your credit, the spec does refer to these as reference types.  However, these documentation pages have been using more generally understood terminology (i.e. string, numeric) thus far as part of modeling after https://developer.github.com/.  The case could certainly be made for sticking with FHIR types, which would require updating all resource pages.  CCing @kpshek and @daliboz for thoughts


Where exactly can I download the stable version? I'm new to github and the navigation confuses me...


When will these not have a name?


@escowles Is the discovery_permissions method being overridden? Where can I find it being called?


this also breaks appveyor https://ci.appveyor.com/project/StefanKarpinski/julia/build/1.0.4599/job/pwt725wp2rmg01an


Will do! :)


None of the source files are kept out via the gitignore. Only game data that is illegal to share, server settings, and IDE settings.


Thanks, this worked!


Cool, this works for me.


Hello Simon,

Yes, sure. But currently I have another tasks so I will do it once I will
have a bit more time.

Regards,
Semyon Churyakov
On May 11, 2015 9:33 PM, "Simon Guest" <notifications@github.com> wrote:

> Thanks Semyon, will take a look. Would it be possible to create a test (in
> test/index.js) that covers the new functionality?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/semyonchuryakov/swagger-mongoose/commit/dd8e70b898ba5f91d89e73fa9e3066fffe6f817f#commitcomment-11140115>
> .
>


It can be undefined. Undefined means the features is toggle off, unless it has a throttle > 0%.


I assume followUser is an id so the name should reflect that. I'd name it userId since we're refactoring the follow function into one called on the user object and it will be the only id argument.


@pcottle, this doesn't work.

1. If `colorPair` is in `self.colors`, `colorIndex` is still set to `newIndex`, which would be undefined and I think python would backtrace
2. If `newIndex` is `>= curses.COLOR_PAIRS`, indicating that there are no more color pairs to allocate, `newIndex` is still equal to `len(self.colors)`, and not even an actual valid color index; I would expect a curses exception to throw in that case. In general, this wouldn't actually happen, because on my terms there's 2^8 color pair indices, but there may be pathologically low terms out there we should support.


@phillipgray ok will do. I will have it sorted by week 2. I prefer IDLE but if I need to do CL from here on out, I will do it.


I think you should revert this commit


No need to implement the callback function `function(err, user)` - just pass done as an argument.


Oh, I forgot to mention, for the "dialog" and "select" options that I've added with this patch, both of them show the MACHINEs as well as from which layer they come. I'll look into adding it to the whiptail version too. I think it's a very good suggestion.


Same comments as for the follow operation.


Yup, it doesn't make sense. Thanks.


Is this comment true in the case of cells below the bottom of the map? If they are outside before being height adjusted it would seem we return false above before we get a chance to check against the fixed height.


@stulees I think this line can go as we don't have tests for this app.


`RealmSwift.Object` ?


Since we'd added config.dashboardUrl, we could even get rid of this branch in the if/then and have the default always be the old patient dashboard (for now, at least) and add the clinician dashboard as the dashboarUrl in pih-config-lacolline.json


Yes! :) Will change.


This is not the correct fix. You need to null check model.getDevicelist() as this returns null.


Yup, the same.


Yup. Will change it.


We might want to rename this also also keep a version that does just the simple ```map.Bounds.Contains(uv.U, uv.V)``` check and give them more apparent names regarding if they respect the heightmap or not.

I say this because you do the ```map.Bounds.Contains(uv.U, uv.V)``` check in several places so it would be worth wrapping into a clearly named method (in my opinion).


(shipit)


no Zachman61 look again on it closely and you will understand what I mean :)


Looks like this comment is redundant now.


Ok


Courtesy? You've been snotty and insulting on multiple projects to others, all based on your own incompetence.

As with ANH, there is everything you need, server side that doesn't stem from the clients, to host a server.


ты против "ведущей запятой"?


I'll change it to trusty64


"they" instead of "their" here in the firs occurrence


fyi, http://git.haskell.org/darcs-mirrors/transformers.git is an automatic mirror which sadly from time to time does non-fast-forward updates... you shouldn't rely on that as a submodule


100x faster to rebuild now @thomasdavis :smile:


oh ok, I also saw the need to install the Windows 10 SDK https://msdn.microsoft.com/fr-fr/windows/hardware/dn913721.aspx here for the include file d3d12.h .
But the problem, I do not know where to put the variable or is sdk for it to find the include file d3d12.h


as it is only testing code i don't mind :)


Thanks!. That makes sense. Should I make this change now?


Isn't this error message misleading if the user changes any of these settings? security.passwordMinimumLength, security.passwordRequiresUpperAndLowerCase, security.passwordRequiresDigit


Consider also ```yield break```ing when the height is zero, as the rest of the code would just return the same position again. (This assumes when you sort out the TODO below, this statement is still valid).


Yeah, that's probably much smarter ;)


"building of a project" sounds better


I guess here you want to pass the loggedInUser as an argument to the toClient method.

```javascript
.toClient(req.user)
```


@tkelman Could you give it a try?


Changes Added for workable reservation form


The same applies to all these other error messages. They should base on the user settings values.


Leftover debug output


This might be able to be an instance method and use the ```map``` field.


Same here


The Wmap mixin isn't necessary


You shouldn't need the cgi library either


Would be better to keep the old naming schema. e.g. cb-centos66-amb200-2015-05-11


I think this one is not needed. Can we advice `back-to-indentation` instead?


"the classpath of all depending scopes"


This makes sure each instance of test case has its own copy of sample projects. Otherwise, when parallel test is turned on, we'll have several tests working on the same project in parallel. Note that some test case need to modify project.json, while some others don't do this. And all test cases need to restore to generate correct lock file.


Hang on... if you need to hardcode something that was previously configurable just to make your tests work, then it means that your tests aren't flexible enough.


Test


I don't think it's critical, maybe just next time we are working in there.


I think in this case we never want the users to tab to the outside of text editor, so it's ok to disable the default tab function for all


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


Timbad...My incompetence? I must surely you recall who deserted a lot of the project and deleted a lot of file by rage and jealousy ? you has a short memory not me :)


I think GitHub is hoodwinking you. This is the last commit in this pull request even though it appears second in the list.


You need to pass the loggedInUser as an argument right?


I think an explanatory xml comment against this and IsExploredProjected explaining the projection is called for, given their non-projected but still public counterparts which would confuse consumers.


We should consider caching this delegate.


Pulling changes from upstream


The 4001 should be Variable.HELP_TEXT_MAX + 1


@phillipgray ok I think I have it sorted, so my code from here out will be OK. Since I probably won't make the hangout tomorrow, I will just leave this file as is. Thanks for pointing it out!


Ready


Here is the problem.  With dependency injection, we want to set the value of the context.  Instead, you created a _new_ variable `blogContext` (line 16), and are therefore never using the mock.  Instead, just overwrite db with the injected value.


Can you give an example how paragraphs are defined now? I am using my own paragraph delimiters so I have no idea how it works right now. Given the following example:

```
#' @export is.Date
#' @param x an R object
#' @return TRUE if x is a Date object, FALSE otherwise.
#' @seealso \code{\link{is.instant}}, \code{\link{is.timespan}}, \code{\link{is.POSIXt}}
#' @keywords logic chron
#' @examples
```
Will all those single-line roxy entries define a paragraph? If no I am not sure that's ideal, I think all those roxy one liners are a paragraph.


The double casting is silly. Avoid it is possible or remove the ```var``` if it can't be inferred.


looks good


Делал docstrings по образцу:
http://sphinxcontrib-napoleon.readthedocs.org/en/latest/example_google.html

Похоже в PyCharm есть свой образец.

Надо ли писать, что возвращается None?


oh yes, makes sense


Slightly cheaty way of doing this


highest big -> highest buy


apparently fixing a typo revealed a few other bugs...


# Git Exercises

## Tasks
- **Getting Started**
  - [ ] Follow the assignment instructions to create a new issue.
  - [ ] Fork repository `2.1-git-exercises` from the class organization page.
  - [ ] Go through [Learn Git
    Branching](http://pcottle.github.io/learnGitBranching/)
- **Git**
  Using the concepts you learned in the git branching exercise, make the
  following changes to your fork of the assignment repository. You will be
  pulling history from the `develop` branch that already exists.

  - [ ] Create a branch called `exercise-1`. The contents of the branch should
    be `index.html` with only the boilerplate code and the heading.
  - [ ] Create a branch called `exercise-2`. The contents of the branch should
    be `index.html` with only the boilerplate code and the anchor tag.
  - [ ] Create a branch called `exercise-3`. The contents of the branch should
    be `index.html` with all of the content from `develop`, as well as a
    `<h2>` with your name below the `<h2>`. The commit containing your name
    should appear in history immediately after the commit adding the `<h1>`.
  - [ ] Create a branch called `exercise-4`. The contents of the branch should
    be `index.html` with all of the content from `develop`, except the paragraph
    of text should be removed from the page without being removed from the
    branch's history.
  - [ ] Publish your `exercise-3` to GitHub Pages.


I have no idea what you're talking about now, are you trying to say I deserted a lot of projects and deleted files? What?

But yes, your incompetence, as one can clearly tell by your posts here on git, on the ANH forums and then the PSWG forums. This whole drama about the .gitignore is pathetic.


`.trim()` would probably be sane here.


I just tried installing a pure css bower package (with a compass project)
```bash
bower install --save animate.css
```
and it's css file gets injected in the vendor.js section that is removed in this commit.
```html
...
<!-- Place favicon.ico and apple-touch-icon.png in the root directory -->
<!-- build:css assets/styles/vendor.css -->
<!-- bower:css -->
<link rel="stylesheet" href="bower_components/animate.css/animate.css" />
<!-- endbower -->
<!-- endbuild -->
<!-- build:css assets/styles/main.css -->
...
```

So this is not the correct solution to the problem. I'll try to figure something out. I think I now how to fix this.


Fails with

```purescript
Compiling Main
Error in module Main:
Error in value declaration clickHandler:
Error checking kind of type
  forall i. Prim.String -> (Halogen.HTML.Events.Monad.Event Halogen.HTML.Events.Types.MouseEvent -> Halogen.HTML.Events.Handler.EventHandler i) -> Halogen.HTML.Attributes.Attr i
Error checking kind of type
  forall i. Prim.String -> (Halogen.HTML.Events.Monad.Event Halogen.HTML.Events.Types.MouseEvent -> Halogen.HTML.Events.Handler.EventHandler i) -> Halogen.HTML.Attributes.Attr i
Error checking kind of type
  Prim.String -> (Halogen.HTML.Events.Monad.Event Halogen.HTML.Events.Types.MouseEvent -> Halogen.HTML.Events.Handler.EventHandler i) -> Halogen.HTML.Attributes.Attr i
Error at src/Main.purs line 3, column 1 - line 4, column 1:
  Cannot unify kind
    !
  with kind
    *
```


Why do we need a comma after _cancellation_point(lambda: self.quit_condition) ? I can see from the previous code that it was a part of the args, but - why is it necessary?


Не поставил pep8, но исправил всё согласно pep8 в подсказках PyCharm


"parent.parent"

:^(


Nope. We should turn the whole statement around.

**Unless** you prefer this, @jgorset:

```ruby
unless ...
  ...
elsif ...
  ...
end
```


What is detail_text
I can't find it in another xml file.


It's because we've got
```c
#define	stdin	__stdinp
#define	stdout	__stdoutp
#define	stderr	__stderrp
```
in <stdio.h>

As a hack, putting
```c
#undef stdin
#undef stdout
#undef stderr
```

at the start of qstrdefs.h fix this issue. But there must be a better way...


Turning it around might get weird as well, actually. Because of the `elsif`.


Uh-oh.


@wkennington Looks like this change broke Darwin too.  This prevents cmake from building, which means clang can't build, so that almost nothing on Darwin can build...


please contact me about this project i like to get it working.


The current shroud renderer avoids updating vertices that are offscreen for performance. Generating verties for the shroud is really expensive in general and best avoided where possible.

The old ```shroudDirty``` layer tracked this stuff (its removal is welcome) but this does mean this loop now updates stuff that is offscreen which is bad.

This should probably loop over the visible cells, and check & update ```cellsAndNeighborsDirty``` as needed.


At CoreOS we vendor all dependent libraries using godep. I must admit that it has its pluses and minuses but overall, it works. I'd like to be consistent  with other projects and other dependencies that are vendored by flannel.


I am also unhappy about this. Will make some experiments.


Updated: Updated to API 10
Added: Zaresir to Authors


Please have a look at: http://www.oracle.com/technetwork/java/transferobject-139757.html
There is a good description of implementation and why it's needed DTO :)


add some punctuation, if you have time.


```cellsDirty.UnionWith(cells);```


Outdent this one a bit? :smiley:


You are a dumb boy


Opened an issue: https://github.com/Homebrew/homebrew/issues/39649

CC @tdsmith


Oh, good catch.  That test above should be checking against `MapSize` (guarding against OutOfRangeExceptions), not the bounds.


Lang Turkish Icy Phoenix 2.0.0.86


переодически -> периодически
"планировать ~~событие~~ поступление"?


Thank you :-)


Most of those checks are wrong (see my last comment), so this isn't really a problem anymore.


Props to @westonruter for this method


Is this nicer?

```ruby
::Tasuku.configure { |config| config.update_answers = true }
```

#codegolf #notsomuch


`const char ** in argv` is non-standard and makes code C++-incompatible. It should have been `char ** argv`.


Boa tarde pessoal,
Esta solução já funciona com o Demoiselle 2.5 (HTML5)?
Ao executar, me deparo com o erro (org.openqa.selenium.WebDriverException: RichFaces is not defined


yes they become paragraphs. I have always found it weird that paragraph motion would jump over the whole set of entries, this makes it hard to navigate inside the roxygen doc. Auctex also works like this with items: one-liner items are considered paragraphs.


```wr``` is unused (and available as a field, besides).


Can you show me the place where we're defaulting to mysql?
Right now the default is SQLite (see https://github.com/xdarklight/cm-update-server/blob/3aa9a242e53c4245b529102ae04f9345bd837b68/config/default.js#L179) so this change should not be required.


Merging upstream changes


I think i resolved the issue... Let me know if I've done right please... If
I wronged again I'll beg you to help me.

2015-05-11 19:24 GMT+02:00 Sott 95 <mrnegro1@gmail.com>:

> Sorry I'm not keen on github.. How can I change the commit accordingly to
> my language? Must I do it all again?
>
> 2015-05-11 16:45 GMT+02:00 William <notifications@github.com>:
>
>> This commit changes the strings for English not for Italian.
>> Look at your commit and change accordingly based on your language because
>> this is totally wrong.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/zephiK/android_packages_apps_Settings/commit/a315d2220d0ab6164ccbb7afe55592863a13c34a#commitcomment-11135415>
>> .
>>
>
>


Thanks!


Fixes #1.


This shouldn't be too hard to address.


Sorry! This was something super quick and dirty written over the weekend. We'll get this cleaned up once the docs work for it has landed. Appreciate the head's up.


Thanks. After applying that hack, I get the original error:
```c
CC input.c
input.c:38:10: fatal error: 'readline/tilde.h' file not found
#include <readline/tilde.h>
```


You are right. Will be back with changes.


Remove the whole method of updateHour. The method addHours does exactly what that is trying to do


See my later comment about referencing the image field name instead, and querying the value using reflection.


Can you, please, clarify why this bit of code was unnecessary? The same pattern can be observed on lines 638 / 639. Cannot we make the code simpler the same way also there? If not, why?


I know you all don't use it, but this commit actually broke building via Bitbake (I use Yocto specifically). It leads to circular dependency errors. I'm not fully sure why.


I think the issue here is that person attributes only support VARCHAR(50), but worth seeing what happens.  I made some commons on this on the "Builds" thread...


I've made a sample port of CastAsThoughItHadFlashAllEffect, but I'm not sure of whether it's correct:

```java
Ability haste = new SimpleStaticAbility(Zone.BATTLEFIELD, new GainAbilityAttachedEffect(new CanAttackAsThoughItHadHasteEnchantedEffect(Duration.Custom), AttachmentType.AURA, Duration.Custom, "Enchanted creature can attack as though it had haste."));

class CanAttackAsThoughItHadHasteEnchantedEffect extends AsThoughEffectImpl {
    public CanAttackAsThoughItHadHasteEnchantedEffect(Duration duration) {
        super(AsThoughEffectType.ATTACK, duration, Outcome.Benefit);
        staticText = "Enchanted creature can attack as though it had haste";
    }

    public CanAttackAsThoughItHadHasteEnchantedEffect(final CanAttackAsThoughItHadHasteEnchantedEffect effect) {
        super(effect);
    }

    @Override
    public boolean apply(Game game, Ability source) {
        return true;
    }

    @Override
    public CanAttackAsThoughItHadHasteEnchantedEffect copy() {
        return new CanAttackAsThoughItHadHasteEnchantedEffect(this);
    }

    @Override
    public boolean applies(UUID affectedSpellId, Ability source, UUID affectedControllerId, Game game) {
        Permanent enchantment = game.getPermanent(source.getSourceId());
        if (enchantment != null && enchantment.getAttachedTo() != null) {
            Permanent permanent = game.getPermanent(enchantment.getAttachedTo());
            if (permanent != null) {
                return true;
            }
        }
        return false;
    }
}
```


Thanks for making the switch. We've been locked in to 0.8 RC[last] deps due to some gesture breakage of late.


Oh, sorry. I will revert this change.


If they are not API breaking, why the minor version change?


Thanks! I got it fixed.


@mgp25 this dosn't solve the problem tho.. I have 3 friends which I haven't added to the account, and getUnconfirmedFriends() dosn't show them at all..


(Not related directly to the security bug...)
Can the code duplication between this function and `r1cs_to_qap_instance_map` above be reduced?


Third intent:

To determine if a data set could be analyzed using multifractals there should be a linear relationship between $log(Z_q)$ versus $log(\epsilon)$, but oscillations might be present and the data sets still qualify as self-similar [@Liebovitch1998]. For this reason it is difficult to use tests of quadratic trends as proposed by Yakimov et al.[-@Yakimov2008], because the curvature of a quadratic function can be fitted by a periodic function with the correct amplitude. To work around this problem, I used the coefficient of determination ($R^2$) as a measure of goodness of fit [@Borda-de-Agua2002]; in general this is not a good indicator of linearity, but since linearity is not exclusively required, I use it here in a descriptive way. The C++ source code to perform the multifractal analysis is available at <https://github.com/lsaravia/mfsba>.


Can we add in `@event` docs for google-feeds-response, google-feeds-error and so on?


this will also pick multivolumes, which currently cannot be selected as reference. Is this intentional?


Mike was here. ;-)


yay! thx.


more potential to reduce code duplication here


<3 <3 <3


`for setting in pattern.metadata.settings`


Closes #12


So I think we actually need to think about this some more. Rather than merge hardcoded things like this that are much more custom than "features.*" for GeoJSON, I think we should make this a CLI option and work towards modularity of the code so that we can easily create custom utilities to handle different data


I let you do the man who does not know of what I speak...
I return to work for now.
see you soon ;)


You don't need this here. Just up in the main menu.


Yo what is the Thumbs.db file?


You should check the .xml layout file under /res for the revision you have checked out. The detail_text should be—but you should verify—in the fragment_detail.xml. The detail_text will the weather for a particular day that is sent via ShareIntent to another person via another app, email, text, Google Hangouts, etc.


@koenkooi the scripts are sources and not executed into users shell envrironment. May folks use dash and zsh as well and it is nicer to support wider set thats why whiptail we can require whiptail instead of bash


This doesn't work. `)` should be moved to the end of the line.


Here, too.


@twoerner think of a case where someone uses zsh and has not installed whiptail and dialog, select will confuse him further. Instead its more user friendly to hard error and say please install whiptail or dialog and quit.


обслуживание -> обслуживания


This also fixes issue manifoldjs/ManifoldJS#54


```begin``` -> ```begin()```


I am working on updating cmb for paraview master. This needs to be resolved for CMB to get any new features in smtk. Also, I think it's best to update paraview version for smtk and cmb the same time when possible, so that developers of both cmb AND smtk (using the cmbsuperbuild) don't have to build the different version of paraview.


Isn't async the only modifier that can precede an arrow function? I feel like the only difference for an arrow function is the preceding line break? The other cases (like open bracket and asterisk) won't occur for an arrow function anyway.


Making it fail safe :)


I tried not to define a new method for this but I failed. If `Her::Model` was not a mixin but a base class, I'd put `@attribute_methods_mutex` initialization in `self.inherited` method. But here I cannot do such thing. If I even put a `@attribute_methods_mutex` initialization in `included do ... end` then some of the inheritance specs will fail. (btw i think that making `Her::Model` a mixin was a bad design choise)


Your Menu seems to be working from the front end... but you don't have the function here? Did you forget to upload your latest code here?


If you don't have a comma there,  it would be args=(f)  that is equivalent
of args=f

(f,) forces it to be a tuple,  the same type as (f, a)

It is a Python idiom

On Mon, May 11, 2015, 12:16 PM jakubbrindza <notifications@github.com>
wrote:

> Why do we need a comma after _cancellation_point(lambda:
> self.quit_condition) ? I can see from the previous code that it was a part
> of the args, but - why is it necessary?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/getting-things-gnome/gtg/commit/c3c04a28af87f75f7ceecbbd4767fff7630354cc#commitcomment-11140895>
> .
>


Is there a scenario in which you don't want the pull secret names to be inferred by the kubelet outside of the user spec?  I.e. ONLY the user of the pod knows what secrets the Kubelet should use?  For instance, if we attached a service account, then the only pull secrets available would be those specified by the service account, and it would be impossible for the user to alter what those secrets are.  Is pull secrets something the pod author knows about, or is it a detail of the infrastructure?


The second ``map`` should be a ``foreach``.


`cm-update-server$ node_modules/sequelize-cli/bin/sequelize init:config --force` above generates that config.json file with mysql, and that's all I ever experienced. If you clone fresh do you get something different?


pesquisarMotoEspecifica tbm foi alterado


i will have some week offs soon and will try something for dual sims on aosp . CM renamed so much stuff in 5.1 for caf , it's time consuming to fix dual sub ID's . let's see


This still seems out of place. Both the starting and ending should be inside the loop or outside the loop.


Identifier corresponds to names in variable declarations, others are declarations captured as is.


default exports can omit the name


Okay, I'm trying it out now. If it works I can send a pull request.


Quick note, Dave. This might be a good time to rebase on upstream master (https://github.com/hyperstudio/Annotation-Studio), as we've merged your previous pull request in, and a fair number of other commits too. Drop me a line if you have any questions. Thanks!


Do we have a relevant test for default exports then


target.kickPlayer("You have been banned by " + sender.getName() + "\n If you think you have been banned wrongly, appeal here: " + TFM_ConfigEntry.SERVER_BAN_URL.getString()););

Just changed the forum url to this so it works for any clone if it is implemented.


Mike was here. ;-)


@mariofusco I am thinking about removing the features with examples (``jbpm-spring-persistent``, ``jbpm-example``, and possibly ``lib``) - these features are currently not tested. They could probably be moved to another feature file, if they are needed?


You might still want this. The target modules are still opt in for clang (host modules default to clang though).


How did you get pass the audio build error?


target.kickPlayer("You have been banned by " + sender.getName() + "\n If you think you have been banned wrongly, appeal here: "  + TFM_ConfigEntry.SERVER_ BAN_URL.getString());


> Is there a scenario in which you don't want the pull secret names to be inferred by the kubelet outside of the user spec? I.e. ONLY the user of the pod knows what secrets the Kubelet should use? For instance, if we attached a service account, then the only pull secrets available would be those specified by the service account, and it would be impossible for the user to alter what those secrets are. Is pull secrets something the pod author knows about, or is it a detail of the infrastructure?

It seemed reasonable to me to support having pull secrets directly on pods **and** on service accounts.  The list of valid credentials would be unioned together and tried in order of specific to general matches.  Service accounts felt like a more advanced usage that was complementary, not supplanting.


This image is WAY too big. Crop it down to 1024-ish.


The error message is also not clear.
I confused by  the error" rustup: need file", the word "file" is a file missed or the file command.


the projection and camera can be optimised out


How does doing it this way cause the child data to be persisted to the parent? Seems like you will set ThisNodeOrAnySubNodesHasError and HasAggregatedChildData to false when you come back out of func?


FYI `unlink()` won't issue a warning, based on file non-existence. But I also like the readability of `file.remove()`.


The error message is also not clear.
I confused by the error" rustup: need file", the word "file" is a file missed or the file command.


Uncaught exception - Error running: cp -R android-sdk-linux/* /opt/android-sdk []


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


----- Original Message -----
> > Is there a scenario in which you don't want the pull secret names to be
> > inferred by the kubelet outside of the user spec? I.e. ONLY the user of
> > the pod knows what secrets the Kubelet should use? For instance, if we
> > attached a service account, then the only pull secrets available would be
> > those specified by the service account, and it would be impossible for the
> > user to alter what those secrets are. Is pull secrets something the pod
> > author knows about, or is it a detail of the infrastructure?
>
>
> It seemed reasonable to me to support having pull secrets directly on pods
> **and** on service accounts.  The list of valid credentials would be unioned
> together and tried in order of specific to general matches.  Service
> accounts felt like a more advanced usage that was complementary, not
> supplanting.

But what's the use case that drives it?  I just want to be crystal clear we *need* to add PullSecretNames here, vs it just being something we can do.  If we *ONLY* added it for service accounts, and required a service account to get pull secrets, what do we lose?


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 34](http://teamcity.otenv.com/viewLog.html?buildId=1299112&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


@thierryvolpiatto Thank you for reverting this comment. :)

I think that a redefinition is harmful, because it leaves no traces:  `find-function` doesn't find the original definition anymore, `describe-function` doesn't show the original docstring, etc.  Worse, you cannot restore the original definition programmatically, from Lisp, that is.  It's very hard to figure out what's actually going on, and it's even harder to restore the original definition in case the redefinition has a bug or breaks some other package:  You can't just “revert” it from Lisp—as you can with advices—and you can't even find the original definition, since there's no trace of it left and Emacs can't tell you were it came from anymore.

Hence, I think it's harmful to load dired-async implicitly.

I agree with @michael-heerdegen, though:  The real issue is in dired-async, which should advise `dired-create-files` instead of re-defining it.  Advices appear in `describe-function`, and can be removed programmatically, which is a lot more transparent and safer than a forceful redefinition.


I would change `max_calls_to_congress` to 3 in order to allow it to go to two senators + 1 rep.

Also delete the entire `extra_last_calls` block.


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 34](http://teamcity.otenv.com/viewLog.html?buildId=1299112&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


I was going to make a release of Annotator and would have slightly preferred if we could have waited for that.


Gotcha! I am new to Python. And yes, it is pretty lean and readable.


How about:

```rust
try!(self.log(req, &res);
Ok(res)
```


Jajaja! todo por 6 créditos! :v:


the projection and camera can be optimised out, since it's being recalculated every loop


Same `try!` trick here.


Oh, revert it then.


I favour heap::EMPTY for this sort of thing unless there's a reason to avoid it.


88737258-EPW-Ruby-Access (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


something like `This problem can be fixed by modifying the Julia build configuration or by` ?


Let's have no space between the `v` and version nnumber e.g. `v2.5.0`, which is the typical format. In UK English at least, the ` v ` is shorthand for "versus".


thumbsdb file?


That would add even more noise to the history ;)


nice


Will do when the whole pull request is ready to merge :)


Mike was here. ;-)


Thumbs.db

The file that is visible in this commit directly above my first comment.


It seems weird to have a macro for these but not actually generalize of `mut?`. *shrug*


Why did you leave out `generate_pebblejs_manifest_dict()` and `generate_simplyjs_manifest_dict()` here ?

Can't build my pebble.js anymore at the moment :crying_cat_face:


You sould change the existing LookLibraryTopCardTargetPlayerEffect from the framework and add a parameter for the number of cards.
Defaulting the old signature to 1 and use it in the new cards.


Regarding readline, OS X version is (based on ?) NetBSD's libedit, probably to avoid GPL issues.
It's unfortunately not a "drop in" replacement of gnu readline, so we need to install and use the real gnu readline (thru macports, homebrew, ...)

I don't think that adding 'common' search path for macports, homebrew, etc is a solution, as the path may change.
For exemple, with homebrew, the library is not linked to /usr/local/include, to avoid conflicts, so the path will change when a new version is released.

For information, CPython on OS X solves this with readline : https://pypi.python.org/pypi/readline, a package statically linked with gnu readline.


88737258-EPW-Ruby-Access (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


b'' tells python to interpret the string as a bytestring.


The version used should be min(version, 4), there is no reason to fail if the compositor doesn't implement wl_seat v4.


!!summary is a very good idea :+1:


I'm close to doing another push for review, and I have added a call to consolidateBlanks when undoing, but I'm still not sure if I did the right thing to reproduce your scenario. Here's a quick video I recorded, could you have a look? http://youtu.be/QPptsEv7cDk


i dont know


This has been added @dheniges.  I had to manually merge this change due to a conflicting PR.


just above this we filter out shards that are larger than maxShardSize so everything in shards is guaranteed to be ``< maxShardSize``


@Velensk:

I added some code so that if images cannot be found, or the card keys don't match their names, the game will make an error when it starts up, rather than when the card appears.

When I did this, I was getting crashes here and I couldn't find the "secret-glyph.png". Did I do the right thing above?
I.e. was the image really meant to be "veiling-glyph" and it wasn't changed from some temporary state, or was there some file that accidentally didn't get committed to the repo?


good question I dont have a clue atm. What is a db ending?


NodeBase is a key implementation detail (hashsets containing node ids) and so I did not leave it up to be trait based, instead the usage is currently as a base inheritance with an accessor trait (get_base, get_base_mut)


FacilityTabGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

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


This seems wrong - React docs say that `this.state` should be treated as immutable. However, `_.extend` would change the property of `this.state`, for example see this:

http://stackoverflow.com/questions/26253351/correct-modification-of-state-arrays-in-reactjs

So... This line should be rewritten as:

`tilestates: _.clone(this.state.tilestates)[index] = 'revealed'`

It seems more readable as well. What do you think?


I thought 'vs' was the shorthand for "versus" ? Anyway, changed. Now waiting for @travisci to finally start it's builds... getting pretty annoyed with how incredibly slow it is today.


Typo ;p


use always [object].hasSubtype(...)  instead of contains for such a check because of Changeling, that also works on players hand!


I removed a redundant line for rspec-rails.


> For information, CPython on OS X solves this with readline : https://pypi.python.org/pypi/readline, a package statically linked with gnu readline.

Any chance we could use that same solution?


I am not sure $_SESSION is the way to go for the config file.  My experience using $_SESSION was when a user logs in, their email / pw can be saved via $_SESSION so that when they pass from one link to the next while signed in php.  Why not just use regular variables for these fields?


It is creating a wrapper object for every object that we need to create (and use to create anyway). I don't think this should be a huge problem, if we see that this is a problem, we can come up with an alternate scheme for identifying the row size.


Use `Util.keyCode` instead? https://github.com/daviferreira/medium-editor/blob/master/src/js/util.js#L62


good point.  i guess this is intertwined with my plans to allow for contouring on multivolumes. once multivolumes are supported as reference, we will want to be able to propagate to a multivolume.

two options i suppose: 1) ignore multivolumes in this list until we can support them in the editor, or 2) wait until we can support multivolumes in the editor and then enable label map propagation to everything.


That is fair, I thought it better to be explicit in this case than simply silencing the warning, perhaps using `unlink` would have bee cleaner though... I am not sure


Couple things here.

1. The module name should also reflect that this is part of Giant Steps.
2. Storing all of this pitch information in tuples (or tuple-like arrays) **and accessing them outside the module that way as well** is a good example of [primitive obsession](http://c2.com/cgi/wiki?PrimitiveObsession)

A more OO approach here would be

```ruby
require
module GiantSteps
  class PitchMap
    ROOTS = [[:B, 3],
                     [:D, 4],
                     ...]

    def root_pitches
      @root_pitches ||= ROOTS.map { |pitch_tuple| Pitch.new(pitch_tuple[0], pitch_tuple[1]) }
    end
  end
end
```

This way, classes outside of this don't need to know about the **internal representation** of the pitch information, any consumer of this class can refer to the documention on the public API for `Pitch` for how to interact with it.


We already have a "name" string for a different thing in `tags/view_all.ctp`, so we’d better use `__p()` instead.


I like the key shortcut idea. ill have to look into that.


@rhc54 this commit broke all tests that involve inter communicator.
for example, the barrier_inter test from the mpi test suite now fails.
if invoked with two tasks on one node, task 1 fails in orte_oob_base_send_cb, since task 0 is not addressable at that time.


Ok


Why? :1234:


ddd


ㅇㅋ...seed.rb내용은 비우고 Factorygirl을 써서 테스팅하는쪽으로 하겠삼


This fix actually now breaks IconButtons... I liked it more as a typo ;)

![iconbutton](http://i.imgur.com/AGdQpBv.png)


test


Copy List With Ramdon Pointer


I tried to do this before. Probably you got the same error as mine. :smile: @ed-xj


np :wink:


http://www.clipular.com/c/5440333862141952.png?k=I6bJg6ZmB907wg_LY-McHj5nKSo
Please look into this...
You renamed the file but forgots to change the image src


Ignore my last sentence...


How about using a GET form in the first place?


Same here.


This is brilliant. Can you rebase so that I can merge?


Why were these removed? This is a very important part of the addon to my server.


Yep - fixed with d3e888387a1f51b4bb0702e54a12b9821addce12


grrrrrrr you really need to add .rtl to the front of that


Nothing, git just had me push changes, I don't think anything broke so it
should be all good

On Sunday, May 10, 2015, nanflasted <notifications@github.com> wrote:

> What did you do?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nanflasted/ResidentMedieval/commit/089948916c5c5d781079c04519eba30b111b6b03#commitcomment-11125947>
> .
>


give me a sec


You sould change Mpdf.php to MPDF.php because you have this Kohana_MPDF in your class.


If it wouldn't be too much trouble, a PR would be greatly appreciated.


完全にVimに移行したんですか？


This is needed for eclipse.  It otherwise shows 1000s of errors.  What was this obsuring?  At the moment, the level of errors makes it nearly impossible for me to work with this in eclipse.  The point to this was that during build the generated code was added to the build as a source file so that the test classes can resolve imports and thus not have errors.


init


OK champ. Can you walk me through the rebase?


What happens if the user isn't logged in?


Fix in fde0da2


target은 너무 딱딱해보이지? listener도 이상하고...

story about player 니까 player? 이건 계속 생각해보겠음


Errors are probably less than 1000s.  This fell under a catagory where I have lots of warnings due to elevated settings.  None the less, issue is that eclipse shows the test packages full of errors on imports.  So if I can understand the issue this caused, maybe we can figure out how to fix so everyone is happy :)


wdt


LGTM.


就是user_edit.php。。上百个用户还要动数据库，麻烦的要死。。/(ㄒoㄒ)/~~


One issue I see is the lack of a canTransfer or a preview of the transfer before it happens.
The other problem is if there are multiple transfers the state of the economy could change, so results could differ depending on whether transfers are applied one after the other.

I'd like to see some way to preview and commit changes or not, regardless of how that form ends up taking.

A query, which way is better?

from.transfer(Account to)

or to.transferFrom(Account from)

Will both resolve similarly with contexts?


新旧结合来用，目前我就是这样，没办法中的办法，把旧版放在子文件夹里。


SORRY FOR DOUBLE COMMIT (tried to update ONLY commits commentary)


AWK  查看资源
          快速指南
内容没有了，网页显示404


efefe


The role should be `:attr:`Image.alpha_channel <...>` instead of `:class:`Image.alpha_channel() <...>`.


(╯‵□′)╯︵┻━┻ 关键旧版我没有了。。


Broke my fork's DB. Error dump [available here](https://gist.github.com/SoftwareGuy/8e68a56947d3430a6f91). I assume it didn't commit and make changes to my DB.

I'll wait until this is rectified before upgrading again.


It's working


Do you mean **expect**?


:class:\`True\` → :const:\`True\`


:class:\`False\` → :const:\`False\`


//Arhur


Warning - Definition of {%settings-key-name-setter in system-internals} is declared open, but it is not exported from this library


right. thanks!


Fantastic!


I'm not sure I agree with this. Given the way that retries can happen within an upload, the last error is typically the one that was "not retryable" and therefore caused the error.


https://github.com/orvice/ss-panel/archive/68dab6061939e2ad2fbb792f18ee2c476c10b129.zip
2015年4月24日提交的，这是2.0.1版，用户管理写入数据库代码在，应该没问题。

https://github.com/orvice/ss-panel/tree/68dab6061939e2ad2fbb792f18ee2c476c10b129
这是原地址


Thanks man lol


Esse `else` é dispensável. :scissors:


Delete empty line  17 and 16


`i < 30` ? e se testarmos com grandes strings ?


Was the version roll-back intended in this commit? Just pulled master and got a "cannot downgrade" issue within Moodle.


Delete empty line 10


rax retry tests


are you guys alright with removing this variable?


ok，我合并进去


Cant see the variable
On May 10, 2015 7:55 PM, "goldfish" <notifications@github.com> wrote:

> are you guys alright with removing this variable?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fishtez/blackjack/commit/afbf226c744005ea0a37f68b3204958919dfe666#commitcomment-11126293>
> .
>


Secret glyph was a scrapped concept. If there was any reference to it, it was a mistake. Veiling glyph is not and it should have an image in the game.

> On May 11, 2015, at 3:55 PM, Chris Beck <notifications@github.com> wrote:
>
> @Velensk <https://github.com/Velensk>:
>
> I added some code so that if images cannot be found, or the card keys don't match their names, the game will make an error when it starts up, rather than when the card appears.
>
> When I did this, I was getting crashes here and I couldn't find the "secret-glyph.png". Did I do the right thing above?
> I.e. was the image really meant to be "veiling-glyph" and it wasn't changed from some temporary state, or was there some file that accidentally didn't get committed to the repo?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/davewx7/citadel/commit/18e9c6c340fd8b9d996998623ac58fde60ef0c0d#commitcomment-11141551>.
>


On 05/11/2015 03:15 PM, Santosh Kumar Gupta wrote:
>
> the last version v2.6.7
>

Last Cobbler version is 2.6.8. Could you update, retry and provide
feedback please?

btw EPEL was not updated with 2.6.8 yet. Orion or Jorgen, could someone
do it?

Regards,
Alan Evangelista


I would go for more declarative variable names instead of vv, www etc.


either way is fine, I will let you pick!


Yes, thanks @Anton-2
I can now build the unix port with two small hacks; inserting `CFLAGS_MOD += -I/opt/local/include` in line 39 of `unix/Makefile` and these `#undef`'s at the beginning of `qstrdefs.h`
One of those few times that I wish that Apple was a little bit less different...


Hello Darius,

Thank you very much for the pointers - work like charm.

Just to let you know that I have been using - last 1-2 months your package and like it a lot (used to use nelmio on S2 - but my current project is L5 hence my interests in the package) - specially configuration options and ease of populating documentation. I also tried today jlapp/swaggervel as my composer was broken and could not fetch your package and it was not as good as yours - I am sure other developers will also appropriate your dev.

All the best,
Darek


Really big method, maybe break it? :)


grandparent?


Character limit is 120 currently. Is that a good length do you think?


ok no need to be so rude : there is no so big issues to fix

Be sure that I will do my best to help to upstream support for imx


I have a branch waiting to be reviewed that updates cmb to work with
paraview master.
https://gitlab.kitware.com/cmb/cmb/merge_requests/27

On Mon, May 11, 2015 at 3:37 PM, Yumin Yuan <notifications@github.com>
wrote:

> I am working on updating cmb for paraview master. This needs to be
> resolved for CMB to get any new features in smtk. Also, I think it's best
> to update paraview version for smtk and cmb the same time when possible, so
> that developers of both cmb AND smtk (using the cmbsuperbuild) don't have
> to build the different version of paraview.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Kitware/SMTK/commit/afdcb4ffae3ee4d5c697ec2f2d75473ee759e7dd#commitcomment-11141249>
> .
>


:rage:


break? :hammer:


How is that fail safe?
It just causes more trouble, such as testing on localhost, or using the [GitHub pages](https://worldcretornica.github.io/PlotMe-Core/) version which has SSL.


This change breaks non-debian platforms.  Needs moved to OS specific include.


pow(2.0, ballot.counter) / 2 is the current SCP timeout - this change basically breaks the timeout that should trigger during a round if the node supposed to be king is slow/not responding.

I would say that if you have some topologies failing with this, just open an issue and mark them hidden: I'd rather not tweak the coronation algorithm as we're replacing it anyways and I don't know if we're creating more problems by messing with those timeouts.


I think it is good but for the index might be still a bit long - you know we have this very long signatures with all that types ;)

Anyway -when all is up and running we can see how it works out - idea is very good indeed as  one does not need to extract the first line like I did previously - just get the meta that's all.


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


Please, don't do this! `Load` is not promising anything is async and even if it were `async` you are lying about it being so by simply wrapping stuff in `Task.Run`. Also you are not awaiting anything here, adding a warning and a potential bug as this code will just continue execution. So what you are doing here is a fire and forget.

Maybe the interface should get some async versions of its method added instead?


The key here is intentionally "Rihn's Anointed#premium" but the name is just "Rihn's Anointed".

This is how we do 'variations' on cards in the game -- i.e. cards that are the same but have cosmetic differences. "Rihn's Anointed#premium" is the same as "Rihn's Anointed" with cosmetic changes.

The rule is that name: should be the same as the key up to any # found in the key.


Pourquoi bug yibo...


OK, I am opening an issue to track the use of dynamic_cast in places where we should not need them


This is great, thank you.


Also here...


Drawabale? ;)


Teglite ot saita tomEE vtoria link koito e za jax-rs i go konfigurirate sus apache tomcat 7 po klipcheto koeto e vuv saita


I was trying to make it work that way, but I got some wierd type system errors. I decided to just commit for now and come back to it:

https://github.com/davewx7/citadel/commit/18e9c6c340fd8b9d996998623ac58fde60ef0c0d#diff-e4c1d8508d510a63d6404a8b5130b5e1R640


This code is in `libcore`, `heap::EMPTY` is in `liballoc`.


> very long signatures

Yeah, we need to solve that one at some point.

> does not need to extract the first line like I did previously - just get the meta that's all

Less magic and more control to the author of the docs. I'm definitely finding it nice to use already.


Great, that worked.  Thanks!


Yai, thanks :smile_cat:


parce que tu as synchronisé avec une ancienne version du fichier rooting donc j'ai perdu des données que j'avais pourtant modifié !


Well, I tried running the program after syncing and it's telling me I have typos, however it's not telling me where and I'm having trouble finding it.

In the meanwhile I can't get my program to run.


Well it's kind of there :-)
(in the Makefile) scripts/translate/collect.py

placed the ACL descriptions back into place, rest of the language stuff needs some work....


Octopus-parent?


Excellent explanation!


Argh, my fault, thanks


Indeed but it's a real help for debug and clean up so can be tweaked later


No problem at all. The browser build is kind of a tricky thing.


Thx much, please add space arround the operators. b ? a : b; please for better readability.


We need check if metadata['mime_type'] exists.


@mrjgreen this can't actually work: line 4332 specifies 3rd capture group for replacement, but the regex in line 4329 specifies only 2 capture groups. Any JVM-based device parser which uses or wraps the standard Java regex libs - I'm writing one now in Clojure - attempting to use this section will fail with `java.lang.IndexOutOfBoundsException: No group 3`


ouh. rude. I didnt meant to be ;)

however, it was my fault, now it's upstream, and @sraue tested and reported working.


New-Features-8-3 (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


check if object_id is None


So, there is still no @user_id variable in this controller. user_id should be the ID of the user that's created/found above.


There's a system test for it.


Within the documentation this commit also clearly defines the relationships between xPress refIds and entity model refIds for each object.


Hi Feilen,

You can check VROGL.cpp at line 34.  That should hopefully point you in the right direction!


VarVar?


Move this check to get_bundle(). In the future we can use get_bundle() from other places.


There was a missing column insert that the API needs


It should give you some list on the command line of what's broken. For instance if I insert a typo I get this:

INFO: graphical_font.cpp:196 : LOADING FONT: data/base_fonts.cfg -> data/base_fonts.cfg
INFO: graphical_font.cpp:196 : LOADING FONT: data/fonts.cfg -> data/fonts.cfg
INFO: formula_function.cpp:3796 : CONSOLE: 'These cards have card name vs. ffl key mismatch: [~key: Rihn's Anoinzzzzted        value[name] = Rihn's Anointed
~]'
CRITICAL: formula_object.cpp:739 ASSERTION FAILED: UNIT TEST FAILURE FOR CLASS game TEST 1 FAILED: "size(vars.cards_with_typos) = 0"

CRITICAL:


The case was that data was there (available using API), but even though the chart was flat.


The commit referenced in the commit message no longer exists, the hash should be updated in the final PR before it gets merged.


Yep, i've already created a pull-request for this.
Pending merge to master-branch

https://github.com/MattHodge/Graphite-PowerShell-Functions/pull/47


This huck is not optional, in the sense that the testnet will break if the original line is in. The alternative are to
* rework the `mBallotValidationTimers` along the same line as the prefetch code, so that the flow is no longer involves a suspension in the middle of `scp`.
* Rebase atop your work where `mBallotValidationTimers` doesn't exist anymore.
Either way, both these options mean delaying fixing the testnet further, which is not necessary.

Note that this fix existed in the pr last week as well. It is highlighted here since it fell off during a merge when squashing and I had to reintroduce it.


Fixed here: https://github.com/davewx7/citadel/commit/b37928e4ed0947fa04eb4bed935afbd8cafdbeb2


@acmenucms Textareas never cleared, because .html('') does not clear textareas. .val('') works for both textareas and inputs.
Also now when the user clicks on submit on the item part of the screen, that part closes.


No ``FSLabelError`` class added by the commit.


call the first parameter objectId and check it.
That should be enough.
```
public boolean applies(UUID objectId, Ability source, UUID affectedControllerId, Game game) {
        Permanent enchantment = game.getPermanent(source.getSourceId());
        return enchantment != null && enchantment.getAttachedTo() != null && enchantment.getAttachedTo() == objectId;
    }
```


Лучше сделать в одну строку album.Id =  response["aid"] ?? response["id"]


I encountered this loop of death today.

This fixed my issue


Damn! That's a good one


This should use ``with`` and iterate over the file instead of reading everything into memory first, but that's out of the scope of this commit (and the patch set).


Hi @cgeorg, is this `$` a typo? Or does it have a special meaning?


I've opened an issue on this: https://github.com/ua-parser/uap-core/issues/51


Is there a particular reason for making this change here rather than being explicit about it when you create a loop action?  I feel this change loses generality.


Could I get a log of the error?

On Mon, May 11, 2015, 12:17 John Wiegley <notifications@github.com> wrote:

> @wkennington <https://github.com/wkennington> Looks like this change
> broke Darwin too. This prevents cmake from building, which means clang
> can't build, so that almost nothing on Darwin can build...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/3550d117cc0a955b370a448e4aa155b9bf48268e#commitcomment-11140922>
> .
>


Found it. Thanks. The problem was that I was checking only the portraits. The mismatch was elsewhere.


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


Modificação funcionando.


Still referring to **an application**


that is fine


@millerdev I made these changes to fix broken tests between copy-paste and markdown


Just on staging. not on a branch


How did you get pass the audio error?


Hey Nelson, you don't understand what's going on here. The cursor tab
outside the editor only happen under specific cases. You want to prevent
Default under those condition, not unconditionally at the beginning of the
code.

We can talk about this in person.

On Mon, May 11, 2015 at 2:10 PM, nelsonleung <notifications@github.com>
wrote:

> I think in this case we never want the users to tab to the outside of text
> editor, so it's ok to disable the default tab function for all
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nelsonleung/textAngular/commit/30bc0bf6e7cc6425475e5d5fe919f3bb9725fc6a#commitcomment-11140764>
> .
>


I'll wrap it in an if stmt for darwin. It should all be going away soon
anyway

On Mon, May 11, 2015, 07:58 John Wiegley <notifications@github.com> wrote:

> @wkennington <https://github.com/wkennington> These lines do not work on
> Darwin.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/5e3fe3916a1aef6c2b441d9009f8a8d7f14030d6#commitcomment-11135740>
> .
>


I'll make the same patch as curl.

On Mon, May 11, 2015, 07:55 John Wiegley <notifications@github.com> wrote:

> @wkennington <https://github.com/wkennington> These lines do not work on
> Darwin.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/e9ee5f99615d0980538e60b1079c8438f8d8135c#commitcomment-11135659>
> .
>


Формат записей, используемый в docstring не очень важен, главное, чтобы во всей программе был только один формат.

Думаю, если функция всегда возвращает `None`, можно не писать этого в docstring


@valib thanks!


@isaacmediacore Can you revert this change? This is necessary for only local dev.


I would prefer to leave that in. The reason being that for people who already have cloned the repo might need to update that branch. This is a preventative redundant pull to prevent those kinds of issue.


why the dict unpack for 1 arg?


Bring this back


besides this looks good to me, thanks @kingjr
@agramfort do you want to take a look?


Oh hell, I forgot about that.  Good catch.


Basicamente, esta linea hizo la magia... solo habia que abrir el archivo para leer de el, no para meterle mas cosas.


Please check that this works with both Python 2 and Python 3 (I remember some issues with python-rpm and Python 3)


Are you sure ?


https://raw.githubusercontent.com/OpenELEC/OpenELEC.tv/master/packages/devel/libcec/patches/libcec-00-imx6-support.patch

here, seems to be good as is, required (header) stuff were re-added to Pulse-Eight/platform


LGTM


If 'LINKFLAGS' is not a keyword arg won't this end up adding env['LINKFLAGS'] twice?  And if it is specified, isn't overriding the environment variable the expected behavior?


I wasn't sure about the assert_raise syntax.


Лучше написать тест и проверить каждый возможный случай.


docstring для переменных не работает, а комментарии подобного плана (объясняющие назначение блока) лучше размещать перед блоком, к которому они относятся


```chance='foo'``` is enough.


oh man, I'm a major derp for doing that


Congrats on 1000!


Тебе не нужно поле, отдельно хранящее голову змеи. В соответствующем методе ты можешь получить голову, используя список


I think these imports could be squashed into fewer lines (2 or 3 at most)


Поэтому пока так и написал, так я проверял и знаю что работает, а если все равно будет ошибка и не будет не id не aid то все равно ошибка выскочит)
В общем это будет работать, а там как нибудь зайду в VS и изменю, а вообще если быть честным, то скорее всего так оно и останется пока API в очередной раз не измениться :D


Because you never want to commit your logs.


sorry not following.......

----- Original Message -----

From: "Levi Cole" <notifications@github.com>
To: "TeriPastorino/KnoWater" <KnoWater@noreply.github.com>
Cc: "TeriPastorino" <tpasto@comcast.net>
Sent: Monday, May 11, 2015 1:17:55 PM
Subject: Re: [KnoWater] change to uid to user_id (4611f19)

So, there is still no @user_id variable in this controller. user_id should be the ID of the user that's created/found above.

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


Тем более, что ты уже это сделал


oh the @

should it be just session[:user_id] = user_id

----- Original Message -----

From: "Levi Cole" <notifications@github.com>
To: "TeriPastorino/KnoWater" <KnoWater@noreply.github.com>
Cc: "TeriPastorino" <tpasto@comcast.net>
Sent: Monday, May 11, 2015 1:17:55 PM
Subject: Re: [KnoWater] change to uid to user_id (4611f19)

So, there is still no @user_id variable in this controller. user_id should be the ID of the user that's created/found above.

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


Looks like you need some pussy cum in that recipe ;))


88737258-EPW-Ruby-Access (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2115 of 2208 tests passed)

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


The data was not there in specific circumstances via API (eg when url parameter `column=avg_order_revenue` was set) and for this I added a system test. Also made sure the values are no longer formatted. I'm trying to avoid to add too many system and UI tests that cover more or less the "same code". I know the code that get's executed can change over time...

Maybe we should write a script to request all graphs with all columns (this would have actually not detected this case, we'd need to create a screenshot test for each column of each graph). @diosmosis any thoughts?


Я предупреждал, что метод copy-paste приводит к ошибкам в коде.


:de:


If it isn't actually possible to buy nerve.wad without also getting doom2.wad, then I don't think it really matters.


I've added an issue for the qstr problem, which is not specific to Mac; see #1252.


Реализуй ещё метод `__iter__`, который будет возвращать итератор от списка координат, чтобы можно было итерироваться прямо по змее.


map::find retorna un iterador http://en.cppreference.com/w/cpp/container/map/find


This part of the docstring doesn't belong here.


@jordibc I think you should commit these bugfixes first in v1.0 branch and them merge into master


Ты просил, чтобы в конструкторе Snake принимал координату головы змеи


But the default log file goes into `/storage/logs`...


Да, но это же не обязывает тебя хранить её


This annotation is incorrect since the method is used by `ShardCompactor`


Great! Doing its job :)

Farid Zakaria

On Mon, May 11, 2015 at 12:48 PM, alexrengel <notifications@github.com>
wrote:

> Gotcha! I am new to Python. And yes, it is pretty lean and readable.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fzakaria/HypeScript/commit/4ecdbbae6e72e486d02c7866fd1fbaa34f9367f2#commitcomment-11141430>
> .
>


you are so right.


I thought that statically allocating device numbers was deprecated. See https://github.com/raspberrypi/linux/blob/rpi-3.18.y/drivers/misc/vc04_services/interface/vchiq_arm/vchiq_arm.c#L2811 as an example of how to allocate them dynamically.


OK, I've found the commit. Thanks for clarification.


OMGOGMOGMOGMGOMGOG !! Les rotations sont la !!!


This looks correct. Are you handling the cancellation of blockfaces separately?


Sebastian Wiesner <notifications@github.com> writes:

> I agree with @michael-heerdegen, though: The real issue is in
> dired-async, which should advise dired-create-files instead of
> re-defining it. Advices appear in describe-function, and can be
> removed programmatically, which is a lot more transparent and safer
> than a forceful redefinition.

Now you can enable/disable dired-async with `dired-async-mode'.

This minor-mode activate or deactivate the advised function
dired-create-files.

--
Thierry
Get my Gnupg key:
gpg --keyserver pgp.mit.edu --recv-keys 59F29997


Плохо, что змея может самостоятельно решить закрыть программу. Это не её обязанность. Остановку программы должен совершать класс `Game`, как и писать сообщения о победе/поражении


Лучше написать через цикл, а не через рекурсию.


It is stupid though because the code had to fallback to making hundreds of draw calls per zorder repeatedly changing textures in order to support that. Whereas now it makes one texture change and one draw call per zorder.


Oh yeah, I actually fixed the problem, but forgot to re-add these flags. Thanks for the reminder!


This also broke all of the buildbots. http://buildbot.e.ip.saba.us:8010/builders/ And my local build. Do you have a fix, or should we revert this?


yeah, that was totally wrong.


Although more readable, isn't adding these braces contrary to the preferred Linux style (and checkpatch's advice)?


Formatting nit: space before {.


Чтобы не отнимать единицу, проще использовать функцию `randrange`


Also broke my local build


No problem, I guess it was a little frustration at the moment ;)
That's great for openelec fix
Now I will try to push to libCEC so that this kind of trouble is less likely; thanks again for porting to V3.0.0


Так как из `random` используется только одна функция, и её имя легко узнаваемо, можно импортировать не весь модуль, а конкретно только эту функцию:
`from random import randrange`


Закомментированный код должен быть удалён. При необходимости следует использовать сохранения кода в репозитории.


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Most recent release (2.4.3) has these instantiation calls commented out - just a suggestion but might want to swap the commit for release 2.4.3 to be this one (084332229e6bdf9d1271e4ac98bb53ec48b30c34) where you uncomment them?


Вынеси получение новой координаты по старой и направлению во внутренний метод этого класса (начинающийся с `_`)


No, I see you are correct:

> This does not apply if only one branch of a conditional statement is a single
> statement; in the latter case use braces in both branches:

Thanks - I've learnt something.


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Контроль проигрыша должен быть где-то здесь, а не внутри змеи


Can state ever be 0? Maybe when the call is paused? If so, document it in the comment above.


Following a change to a similar allocation in VCHIQ, does this allocation need to be atomic?


/cc @jeregrine @lancehalvorsen


В целом, уже гораздо лучше!


@lukasgeiter true, but you can change log file path. And as you probably know, each line in a .gitignore file specifies a pattern, so lumen.log won't be committed regardless of where it placed (in default folder or not).


Awesome, thanks!


In 7.x, it was a no go to use entity API in update functions, because you might add an update function later on that added/removed/changed a field and that completely exploded. Not sure about that in 8.x, given that field update functions always run first. But that doesn't work for changing stuff with data. So I think it is still necessary to do a man update query here? Should also be considerably faster...


oops. tests failing. will check tomorrow


`path_to_input` тоже не очень хорошо звучит. `input_filename` гораздо более широко используемое имя для входного файла


in the previous commit


"Rusts" => "Rust's"


"rust meetup" => "Rust meetup"


I'm not a big fan of toggles. All controls except these toggles have a single function: they deterministically bring the call from a state to one other state, or fail (and do nothing). They also deterministically fail if you send them twice, so controls are idempotent: sending them twice has the same effect as sending them twice. Toggles violate that.


This turned out to be easy, it should now give the error messages in the popup as well as on command line.

https://github.com/davewx7/citadel/commit/6d362b8c86c623b88391e3dc1f68b5bccc17228d


Cool.


With this commit I was getting error "'-std=gnu90' is valid for C/ObjC but not for C++"
We can fix that with CONLYFLAGS e.g. :
-LOCAL_CFLAGS += -Os -std=gnu89
+LOCAL_CFLAGS += -Os
+LOCAL_CONLYFLAGS := -std=gnu89


Don't forget to increment the version number in in kOS.version.


@limonte No. See that slash at the beginning? This only ignores `lumen.log` in root directory.
I'm not saying this line is useless but it's not really needed.


Should this allocation be atomic?


Error compile on D3D12PipelineState.cpp
<2>------ Début de la génération : Projet : emucore, Configuration : Release x64 ------
2>  D3D12PipelineState.cpp
2>Emu\RSX\D3D12\D3D12PipelineState.cpp(246): error C2653: 'Shader' : n'est pas un nom de classe ni d'espace de noms
2>Emu\RSX\D3D12\D3D12PipelineState.cpp(246): error C2065: 'SHADER_TYPE' : identificateur non déclaré
2>Emu\RSX\D3D12\D3D12PipelineState.cpp(246): error C2146: erreur de syntaxe : absence de ')' avant l'identificateur 'st'
2>Emu\RSX\D3D12\D3D12PipelineState.cpp(247): error C2143: erreur de syntaxe : absence de ';' avant '{'
2>Emu\RSX\D3D12\D3D12PipelineState.cpp(247): error C2447: '{' : en-tête de fonction manquant (liste formelle à l'ancien format ?)
3>------ Début de la génération : Projet : rpcs3, Configuration : Release x64 ------
3>LINK : fatal error LNK1181: impossible d'ouvrir le fichier en entrée 'G:\GIT\rpcs3 Zangetsu\x64\Release\emucore.lib'<


I adjusted the comment to match reality


Пара замечаний:

 1. Проще импортировать непосредственно эту функцию из модуля:
`from collections import namedtuple`
чтобы не писать такой долгий вызов. Имя функции достаточно уникально, чтобы не бояться спутать её с другой.
 2. Обычно, при создании `namedtuple` имя созданного класса совпадает с именем переменной, в которой его сохраняют (`mRNA_tuple` -> `RnaStructureInfo`)


You're right it doesn't. It's even protected my a mutex which can sleep.
I'll add a commit that changes this to GFP_KERNEL.


I think the proper implementation for this is actually as simple as:

```python
def __iter__(self):
    """Yield an infinite stream of messages from this consumer."""
    while True:
        yield self.consume()
```

This works because you're returning a generator and this ensures that the actual method is called on each iteration.

In general, you want an object to call it's own public functions where appropriate. Here it'd avoid a subtle and annoying bug, but in other cases it reduces duplicated code.  It's not as obvious here, but consider if `self.consume` had to perform a few checks. In that case it'd be more obvious that the right call was for ``__iter__`` to call it.


looks good - the tp_alloc call matches the code for `weakref___new__`


this exercises the proxy_getattr fix, is there any chance you can add something that exercises proxy_setattr as well?


There is likely some latency between the above writes and this check.


I see there are a lot of places that need reformatting, so I'm not going to comment on all of them. I'll assume you're going to make a pass over it to fix them.


> increased the flash sector size.

Ok, is it really "flash sector size"? Unless can be proven it is, I suggest just calling it "irom0_0_seg", which is at least 100% true.


It looks like this line was accidentally added here:

  https://github.com/KitwareMedical/TubeTK/commit/d0bb7fa0b412d3f18e63909854469d3e490e0713

and it is fine to remove.  The filter does own the data, and it will be deleted when the filter goes out of scope and all SmartPointer reference counts are removed.


can you please use the 'path' builtin module instead of concatenating directories with '+' - I know this is on unix/mac but still let's use path.join for this thanks!


Less urgent than the other complaints, but before this commit setting `JULIA_DEBUG` in `Make.inc` would allow debug builds by default (if neither `release` nor `debug` is specified explicitly on the `make` commandline).  It would be great to find a way to restore that functionality.


Of couse, this is a source of inefficiency. We would like to maintain everything in registers, and the compiler is likely unable to do so if one uses _write_16bit_avx_d1.