Hi,
I had a problem here, when using  'outer-drag'.
image is not rendering and image is getting blurred as much as i scrolled.


--network-plugin


No inventory


I know this is POC, so mostly brainstorming here. For append-only sinks like HDFS, do you think this will always provide exactly once semantics? You could add a batch to HDFS sink and then this process/thread may crash without updating the watermarks. On resurgence, the batch would be re-processed and re-added to the sink. What do you think?

I think, perhaps, the HDFS sink in this case would have to write to a temporary file and do an atomic move to 'commit' the batch. So, perhaps, we need a commit call to the sink?

Again, just throwing some ideas around.


I transposed two numbers in the issue. This fixes #1209.


We need to guard against appending it twice.


thx!


This formatting thing, I need to take care of it, looks like my personal formatting "preferences" differ from the rest of the world...


remove space here.


use new private method with broadcasting argument


@CliffordAnderson I'll switch it to two `<app>` tags.  I felt it could go either and of course as you said it all comes down to how we'll eventually visualize the changes in the interface.  All these little details :confounded:


remove space


remove space.


Antras praktinis darbas. Programa JAVA kalba/Kreditas nr. 2/src/praktinistrys/NewJFrame.java

    /**
     *
     * CONSOLIDATE CONDITIONAL EXPRESSION
     *
     */


use new private method by default call


It's actually quite important for Taxonomy to be abstract; it's not the Taxonomy class that defines which collection and other meta-data to use, it's up to whatever concrete implementation (Asset) that uses Taxonomy.


This should be done after ` if (!path.endsWith(".py", Qt::CaseInsensitive)) {`


I pulled this in and it did not work for me. Is this still working for you?
Im running the following
vbox: 5.0.10
VVV: 1.2.0
Vagrant: 1.7.4

I've got a symlink on my host referencing ~/sites/cool-site to ~/vagrants/www/cool-site/htdocs
Any help getting this running would be much appreciated.


Returning an error here breaks rkt on every systemd that doesn't run systemd, right? Is this intentional?


nit: what's a ServiceUser? :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9510](https://teamcity.mesosphere.io/viewLog.html?buildId=93341&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 174 Build time: 00:03:42


Добавлена заглушка на сообщения


Remove the commented code.
Personally, I would rather use something similar to following code in order to obtain _confimartionData_ (but it does not matter much):
````
var confirmationData = String.Join(
	Environment.NewLine, // or whatever string you prefer
	new [] {
		model.BudgetId.ToString(),
		model.AssignedUserEmail,
		model.Permission.ToString(),
	})
````
It it longer, but I bit less messy and more easily extendible (in IMHO even readable). The reason I use separate is that in some cases there could be very same hash of an "luckily-formed" GUID and "well-picked" email address.

You should also add some value that is known only to your application (such as _MachineKey_ that is also used as a salt by identity manager) to ensure value is not easily (~ too straight-forwardly) spoofed. You could actually go a bit further and symmetrically encode URL parameters – check the static [_MachineKey_ class](https://msdn.microsoft.com/en-us/library/system.web.security.machinekey.protect(v=vs.110).aspx) out.

Nice thing about .NET is that you can easily serialize an POCO object (with all your parameters) to a string, "protect" it with the _MachineKey_ and send it out as a link. Then get this link back, "unprotect" it and deserialize the string back to the POCO object. Ideal candidate for 2 URL extension methods :wink:.

But it is completely OK to just protect and unprotect your _confirmationData_ or add your _MachineKey_ (or any other unique key e.g. from web.config (do not store it in GitHub!)) to the _confirmationData_ before they are hashed.

I guess I do not have to add that these methods that care about hashing and its verification should not be part of a controller…


a lot of controllers not being used here.


Мораль: в 1.20 ночи полезно смотреть на код перед его коммитом


lgtm


I just downloaded and installed Node Tools for VS and it became clear that node projects weren't supported. I added the support. Please install latest CI build and verify the fix http://vsixgallery.com/extension/36bf2130-106e-40f2-89ff-a2bdac6be879/


This should be moved outside the if-else since it is a common step.


(Achtung! Langer Text)
Das Problem ist, das nach dem Abrufen des Benutzers das User-Objekt immer noch von dem EntityManager verwaltet wird (es ist "**attached**") in diesem Zustand werden alle Objekte automatisch durch die Setter upgedated.
Aus diesem Grund wird das Passwort nicht gehasht, es wird durch den Setter direkt in der DB auf 'test123' gesetzt.
Die IF-Anweisung in der update Methode im UserService liefert daruch true zurück -> keine generation und setzen des Hashes!

Im 'normalen' Einsatz der Applikation passiert das nicht, weil das Objekt dort noch per JSF an den User gesendet wird und dort dann von einer anderen Methode 'übernommen' wird. Dadurch wird eine neue Transaktion erstellt und das Objekt ist noch nicht attached (weil auf ihm noch keine Datenbankoperation, welche zu einem attachen führen würde ausgeführt wurde) -> Das setzten des Passwords über die Setter ändert **nicht** das DB-Objekt.

Zur Lösung des Testes kannst du:
* Entweder die IF-Anweisung aus der update-Methode des UserServices entfernen oder dort allgemein die Logik    abändern
* Dir in den Test einen PersitenceContext injecten und mit diesem einen detatch auf das User-Objekt ausführen

P.S. Der Name der Klasse "UserManagement " stimmt nicht mit dem Namen der Datei "userManagement " überein

 :-P

:palm_tree:


Now it's all your!!!! :P


faut que tu m'explique. Pourquoi des attribus public et protected? Pourquoi mobtype sachant que les enemis sont des sous classes?


FYI, these are the default values of `$compact` and `$regular`, so strictly speaking it's not necessary to reassign their values.


add highlight


This should fix #1 and #2.


Why not just a QFile::rename()?


@hulpke, I guess from the neighbouring  code that  `ElementayAbelianConsider` is a function to be used with `SubgroupsSolvableGroup`? In this case may I have two suggestions?
* Both SizeConsiderFunction and ExactSizeConsiderFunction are documented, so the same should happen with the 3rd one
* Rename ElementaryAbelianConsider to ElementaryAbelianConsiderFunction to follow the same naming style


Fix'd
https://github.com/aries-auto/ariesautomotive/commit/6c4ae3358a9eae3c5ccf30f4b4415a5f7c858056


EconomizerIntegrationNew (mjwitte) - x86_64-MacOS-10.9-clang: OK (1956 of 1974 tests passed)

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


Why not `Utils::Fs::forceRemove()` here?


m_pFrame might be invalid at this point. you want to move this further up, where m_width etc is assigned.


Looks pretty solid at a first glance - there are some things to clean up though. Like all of the logic after that first return statement in the test function?


Wouldn't it be better to check `ForAll(cgens, IsOne)` instead of `ForAll(cgens,x->Order(x)=1)` to avoid calculation of the order (which may take some time)?


109626134-VariousDocumentationIssues (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 tests passed)

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


Good point. Let me add that in


Why not use aws-sdk ruby gem?


@theoryshaw: Just in case your interested. I started trying to make requirements for a rewrite of draw. Was going to try and incorporate the functionality of draw and the enhancements listed on the issues.


ah, thanks


I think this check is superfluous


Cade o TEST pra essa funcionalidade ? :)


Based on the [Connection implementation](https://github.com/lsst-camera-dh/eTraveler-clientAPI/blob/master/python/eTraveler/clientAPI/connection.py), instead of this line, the following should work:
```
myConn.env['LCATR_ASPIC_LOGFILE'] = log
```
See [line 269](https://github.com/lsst-camera-dh/eTraveler-clientAPI/blob/master/python/eTraveler/clientAPI/connection.py#L269) where the environment is passed to the `.execute(...)` function.


@stockfishdeveloper That's nice to hear, but I [do not see](http://github.com/stockfishdeveloper/Stockfish/pulse/halfweekly) your new (non-`master`) development branch containing your changes and I don't see your test pending either.

@Rocky640 I don't know whether or not @stockfishdeveloper considered how the Threats weight affects his change.  Given the level of detail in [this explanation](http://github.com/ddugovic/Stockfish/pull/22#issue-122120259) I would be shocked if he considered that... so I'm guessing he's not interested in reintroducing the Threats weight?


This looks great!


EconomizerIntegrationNew (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (1961 of 1980 tests passed)

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


Check


Looking good now.


We disallow resetting it now (since nulls are never transferred). This follows the behavior.


Yes, this was before we came to that guideline. But you're welcome to do a PR bumping the version number.


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


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


are you suggesting thay I should mention that before the actual code or just not have any comments? :p


hahah thanks @salavert ;)


This should throw an exception if the write fails (the implementation just returns `false`) - its too important to leave this unnoticed to the dev.


Why `.toUtf8().constData()`? You already store it as QString.


I noticed you kind of go back and forth here.  Sometimes check for settings == null, sometimes check for descriptor == null.   I think fxcop will break on either of them not being checked.

Did you run fxcop on this to resolve those?


`zeroline` appears at 1 (i.e. e^0) on log axes so this removes the `zeroline` from log axes


Changed


remove these random spaces


Again no need for the check


Windows


Did you mean to use `Utils::Fs::fileName(filePath);` ?


Oh that's right, i was referring to https://github.com/kubernetes/kubernetes/blob/master/cmd/kubelet/app/server.go#L321 by struct member name, but it is indeed --network-plugin


@benrudolph is there a reason why these got deleted when https://github.com/dimagi/commcare-hq/blob/master/corehq/apps/reports/standard/inspect.py#L84-L89 still expects them to exist? adding it back in for now because it's throwing an error in reports


It's causing errors for that person, yeah. We're not changing our defaults; instead we're providing a way for them to override them and get around things like this.


This change would be a proper candidate for a PR, because this could be debatable (see [Obtaining LaTeX Representations of Objects](http://www.gap-system.org/Manuals/doc/ref/chap27.html#X78024C8087F3E07F)). In the past, we even moved several LaTeX generating functions to obsoletes, just keeping them as examples for other users that may want to generate LaTeX code. Of course, if `LaTeXTable` is nontrivial and reasonable, we can add it, and in this case the link to the ManSection should be linked from one of *.xml files (this is not done yet).

By the way, you can easily post here PDF with the example produced with the help of this function - just drag and drop it onto the text input area.


Did you mean `tr("Failed to update the plugin file. %1")`?


I found in some core popups (f.e. smiles) with this functionality. I haven't checked any modules.

In any case I think both these functions should be deprecated first and than should be removed in next version. I think it's a better practice than remove everything right now because you never know what custom non-public modules uses.


doc would be great about what we're doing here. might be nice to also put a comment in java


:+1: Love this assignment.


nit: spacing before """


OK forget the above. I saw how it is used in the code below.


perhaps fix this one too :)


It corrects a deprecation warning in the javascript console that I happened to notice while debugging a bunch of other stuff.


Huh, that's why you shutdowned Pham right? Looks it's an interesting update.


tu as un script de conversation instantanée ? :+1:


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9511](https://teamcity.mesosphere.io/viewLog.html?buildId=93351&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:11


Nicely done. I think this the less-hacky way of doing this. Good :eyes: for catching this log-axis issue.

:cow2: I've been trying to remove all the

```js
var a                            =  'hello';
```

patterns in the plotly.js code. I guess I missed that file.

@cldougl could you re-write your patch without those big space gaps? And maybe mention in the comment that without this if-block, a zeroline is incorrectly drawn at `1`.


well yah, we don't set it, but it is going to be relative to the div a chart is in so all works fine.

The main reason I remove it is when i do an ajax request the new chartkick that gets generated does not get those properties, the gem doesn't load them unless you reload the page so..kinda useless in this case


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1962 of 1980 tests passed)

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


You might get away by just doing it like it is. The (shared) pointers are used for the Reaper actions because it's not clear how the the action entry objects should be copied or even even moved around. (Which is needed for at least std::vector push_back.)


Reload is called from both cni.go (loads any existing net.conf for stock cni plugins, does nothing for net.kubernetes: https://github.com/kubernetes/kubernetes/pull/18690/files#diff-5edfa9e0b8826e636c36d1c78377858bR135) and periodically from syncNetworkStatus in the kubelet (https://github.com/kubernetes/kubernetes/pull/18690/files#diff-bf28da68f62a8df6e99e447c4351122dR2696).

In the latter case, the kubelet picks a network plugin and gives it to cni.realod to write the netconf as a WriterTo interface. I wanted the kubelet to pick a network plugin without having to know about where/how to write out the netconf.


This seems to be the cause of https://github.com/google/closure-compiler/issues/1299


Good call...updated tc.


Shouldn't this be in the "add cancel buttons" commit? Also, didn't we decide to use hard coded locations for cancel buttons for now since the referrer isn't always valid?


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

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


I can't get this to match anything, what is it trying to achieve? http://regexr.com/3cds0
Maybe use `#` as a regex delimiter to remove the noise a bit?


Fixed.


So rSyscall will be copied to all source files including syscall.h?


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Dafuq este site tem comentários?


@z3cka I appreciate your proofreading. I really do!


what does "default" mean?


I think @dougbu is right, I didn't mention this case, but if an intermediate `_ViewStart.cshtml` doesn't define the layout, it should not override the previous one.


I prefer Mac OSX


FxCop only cares if you actually use the parameter. If you do, you need to check it. If I don't use it, I don't check for null -- passing along a null to a submethod is fine; if it needs it to be non-null, it can check.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9512](https://teamcity.mesosphere.io/viewLog.html?buildId=93354&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:09


shouldnt OPENCL be removed from options in GSDX unless it is specifically selected too?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9513](https://teamcity.mesosphere.io/viewLog.html?buildId=93359&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 206 Build time: 00:03:51


See 19593a97d30de5a23c3712c3f0edb2ac76a02b0b.


这 i18n 画风突变


109626134-VariousDocumentationIssues (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (0 of 0 tests passed)

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


Did you intend for this to be "dely" or "delay"?  Not sure if you wanted it to be only 4 characters for alignment reasons.


Thanks! Appreciate the help and additions.


cool. glad, this is just so much easier to read: https://github.com/UCLALibrary/ndlc.info/blob/4ef63ee55917788053c720cd647ff12b8867dbf1/pages/02.ndlc-schedule/program.md


This will not change anything.


yes, much better!


Could you beef this up to explain the confusion I had before?

"If the value of this flag starts with the prefix, the final token implies the CNI plugin with a specific CNI driver."  Something like that


These 2 tests will never work because you do ` QString novaFolder = Utils::Misc::pythonVersion() >= 3 ? "searchengine/nova3" : "searchengine/nova";` (due to the changed .qrc file)


also --net-plugin not --net-plugin-name


No I was not aware: but now that you mention it, it is obvious, isn't it?  I guess I still have to internalise the abstraction away of execution _sequence_ that streams represent.  Thanks for that. I'll get back to this.


Well, this one was just adding a new object and changing the return value of `describe` messages, so tedious but easy :)


According to GCC documentation /usr/include and /usr/local/include are automatically searched for include files during preprocessing, as well as /usr/lib and /usr/local/lib are searched for shared and static libraries to link with. See https://gcc.gnu.org/onlinedocs/cpp/Search-Path.html for some details.


I am not testing the code, this is a code review. (C++/Qt topic)

>   CacheEntry(QString userIp, QString userName, int channelID); //construct a entry using default values

no need this overloading, C++ supports default parameters

> CacheEntry(){}//default construcot just to use in QMap without pointers

CacheEntry is not a pointer type, always making the copy when it was passed as arguments or returned as value. Is it an expected behavior?

Qt provides QObject for the object's memory management.
http://doc.qt.io/qt-5/objecttrees.html

As map key, user IP and name does not promise the uniqueness of id. (rare case but happen)
channelID does ?


I am unhappy that this is part of the interface - netconf is a CNI-specific thing, isn't it?  Can't we keep that internal to CNI?  The writer doesn't seem to be used anywhere, but iassume it's dep-inj for testing?


means: Kubelet, just pick whichever you think is right. currently bridge.


not sure what you mean .. :grey_question:

Brendan said presenters is the thing to use in this case :smile:


It is not okay - that is why we included the library in the first place. Please revert the removal.


-Werror flag was changed to -Wall because OpenMPI implementation produces some warnings during compilation and these become errors. -Wall will pedantically inform about warnings (though will not stop compilation when occurred).


hi that script is for version xiaomi yi fw 1.2.13 ? like this ? :

https://github.com/PJanisio/Xiaomi_Yi_autoexec/tree/master/sharpening%20%28method%20-%20denoise%29

cause am not shure and am not shure about my self to make a script myself and ruin the camera...am looking for finished autoexec


Do we have any assets to install?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9514](https://teamcity.mesosphere.io/viewLog.html?buildId=93360&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:05


My code is 1127717387
12/15
19:00
my email is plachilly@hotmail.com


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9515](https://teamcity.mesosphere.io/viewLog.html?buildId=93370&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:08


Not sure, both is correct, maybe `'one'` would be even better.
http://english.stackexchange.com/questions/153327/why-do-we-say-she-referring-to-a-general-user-of-a-program
http://english.stackexchange.com/questions/28508/reason-for-the-current-trend-to-use-she-as-the-gender-neutral-pronoun


Beware of `?` quantifiers on the left side of `s///`.


Thanks, you saved my evening. Couldn't figure out why `choose-tree` and `link-window` don't work together, turns out that a dummy `-b` option was required.


adding whitespace, new lines and using def statement instead of an assignment statement that binds lambda expression directly to a name


git cherry-pick --continue as title? Ah? xD #githubtroll


My code is 1127717387
12/15
19:02
my email is plachilly@hotmail.com


To clarify, it is the easiest way to differentiate a user request for stock cni bridge plugin, from a potential kubernetes meta plugin + bridge/flannel/whatever.


@tailhook will there be another blog post detailing new changes to the rotor library?


This can be greatly simplified by only doing `return m_plugins.keys();`


lua.Run won't do anything, but I removed it anyway.


Win 10 x64 - This fixed my current issue with Fibers. Fibers originally worked fine as a module within NativeScript-CLI (it worked for weeks... and over several months with a couple of re-installs of windows). Then some event in late Nov caused this problem to start happening (any build or rebuild of fibers refused to work). If anyone would like to know any details about my computer and software configuration then im happy to share that information.


Hey do you want to do what I did and #include softtakover.h so you can use the constant defined there (times 2 for this case) rather than hard-code it?


:birthday:


You are totally right that we will have to make this atomic, probably by doing some kind of move or maybe a separate manifest / metadata store (though I haven't look into this in detail).  Is there any reason though that this can't be the responsibility of the `addBatch` function?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1135.3f08761c54c73fd109f13e11a4f10a0cd0899b24](http://build.palaso.org/viewLog.html?buildId=67332&buildTypeId=bt431) outcome was **FAILURE**
Summary: Execution timeout (new); tests failed: 1 (1 new), passed: 774, ignored: 6 Build time: 00:35:19
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


Ovo je dijelo Nenada :D


While we're here, it might be worth changing this to `0o022` since this style isn't allowed in strict mode.


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

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


So the thing is that I update the charts with ajax to see previous polls and when I was writing that I was planning on updating the `Total number of votes` string on the page as well. So I thought that having the method in the presenter and calling it in two places (page and ajax) is more efficient than duplicating `Total number of votes: #{ Poll.last.ballots.count }`

but I might not understand something..and also I haven't included that into the ajax code yet because i'm running out of time :cry:


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


I thought readMay checks for an empty string and returns Nothing for empty strings. I guess I was wrong, use case for mfilter ?


What/how is this used (additional_bbc)?


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

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


Note to self: Later I need to check later the old code. I think for forward compatibility we need to just ignore the extra fields and not return early.


```/deterministic```, not ```/feature:deterministic```


Redundant, mentioned lower.


should that be `$channel_credentials` to be consistent with the rest?


I'd mention that there is an option to override this.


Sounds like a bug, should we link to an issue?


Second sentence: "Native apps no longer require Cordova."


I don't think the comment adds anything. It's a good idea to help the reader understand why you're doing something, but I think we have to assume that they know a certain amount about the environment.


We'd be better off binding this in the constructor, because otherwise a new function is created in each render cycle which is much less efficient. Also, handleChange needs the event argument, which this would strip...


Remove periods after the library names in the list


As per my comment on f9a2d36abf7b2e400014f1b69de8a62be84a2f44 we should really avoid creating new functions in `render()` when we can


4959-plant-HX-CoolDifferential-stuck-Off (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1448 of 1448 tests passed)

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


Получилось так , что я случайно удалил папку с курсачем :(
по этому сделал еще раз коммит :+1:


I added the loclaStorage on my calculator project but was in master branch. moved it over to my new w03/d01 branch and then pushed. doesn't look like the changes showed up here.


109626134-VariousDocumentationIssues (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (0 of 0 tests passed)

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


109626134-VariousDocumentationIssues (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Sentence one: "The new SalesforceReact library provides the ability to use React Native."

(Notice the space in "React Native".)


Super

> Am 15.12.2015 um 22:32 schrieb Ningfei Li <notifications@github.com>:
>
> I saw that the voxel size and affine params of the previous aal.nii is slightly different from the standard one. So I just took the ROI_MNI_V4.nii from spm (same as that from fieldtrip) and changed it accordingly.
>
> —
> Reply to this email directly or view it on GitHub.
>


109626134-VariousDocumentationIssues (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed

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


third-party needs to be 9 because it depends on gms which depends on 9


109626134-VariousDocumentationIssues (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Reword so that it can't give the false impression that TLS 1.0 is already disabled on Salesforce servers.


theoretically, sure.

right now this only takes one URL (or spits out the raw input). unfortunately, a lot of the data we had imported over has multiple URLs for videos in the same text field. ideally, we'd keep using this, and change the field type in Craft, but that's going to make a lot of manual re-adding of data (which is a million times more boring that making this work with multiple URLs)


Removing these imports breaks the home page. I haven't check which one.


@eboyer I realize that this breaks the "Filtering" and "Footer" sections since the right-side overlays both. Hopefully there's a simple CSS fix, otherwise I can add more logic for hiding the right-side when transitioning below the main/feature area.


That is actually needed :(
I will try to make it simpler but I couldn't quite get the results I wanted using the existing methods :(

Do you have any environment with 2 domains I can use to test one of the scenarios?

Sent via iPhone while out and about

> On 16 Dec 2015, at 10:01 AM, Brian Farnhill <notifications@github.com> wrote:
>
> Looks pretty solid at a first glance - there are some things to clean up though. Like all of the logic after that first return statement in the test function?
>
> —
> Reply to this email directly or view it on GitHub.
>


I'm going with Arch Linux


dammit. don't know how i missed that. sorry. basically every call passes in `True` for `simplified_users` and `combined`


Did this sentence never have a period? ;-)


Yeah, just safeguarding against another edge case.


Period at the end of the sentence. :-)


From an API point design point of view, serialEvent() is a disaster.  Weak functions should not be used an a normal API.  Libraries that take a Stream or HardwareSerial reference or pointer in their constructor or begin() function can't use serialEvent.  It's only usable within sketches.

This sort of API really should have been defined with a function like Serial.attachAvailableEvent().

I can tell you with certainty that some people do use serialEvent.  I know, because I initially didn't implement it for Teensy.  People complained.  I had to implement it.

Whether or not serialEvent should remain in Arduino is a good question.  My main point is it really should be done with an attach function, and the weak reference should be eventually removed.


Why cp and not symlink?


109626134b-AvailabilityManagerOutputUnits (mjwitte) - x86_64-MacOS-10.9-clang: OK (1978 of 1979 tests passed)

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


You didn't actually invert the condition, so this pass is still running. Isn't it?


Remover periods after all library names in the list.


It's not unused we're using it in production in fact :)
It's useful with IoC


no problem!


@BrianFarnhill
one of my concerns with this resource is that -Force parameter. Whilst it
solves the presented problem I think it kinda goes against the DSC model.
Why do I say that? I don't know, just think perhaps we shouldn't force
something.

On Wed, Dec 16, 2015 at 10:18 AM, Camilo Borges <camilob@gmail.com> wrote:

> That is actually needed :(
> I will try to make it simpler but I couldn't quite get the results I
> wanted using the existing methods :(
>
> Do you have any environment with 2 domains I can use to test one of the
> scenarios?
>
> Sent via iPhone while out and about
>
>
> On 16 Dec 2015, at 10:01 AM, Brian Farnhill <notifications@github.com>
> wrote:
>
> Looks pretty solid at a first glance - there are some things to clean up
> though. Like all of the logic after that first return statement in the test
> function?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/camiloborges/xSharePoint/commit/df91dbe19dbd5f1504651d12d7e86391d1a9ea60#commitcomment-14999431>
> .
>
>


Awesome!!!
hmm, Editing ESlint file still ignores the project .eslintrc and right click and run WebAnalyzer appears to do nothing at all, Is there somehting else I should try?


This is not happening after the AJAX. Since it is not in a callback, it happens immediately.


Ah dammit... since I can't test WP it's a pain :\
Thanks :+1:


@shminer dont worry about this one.
i did this because i did not want the caf lmk.


Cocoa Touch (capitalization)
Cocoa Touch dynamic framework targets have been added to our libraries. Going forward, this will be the recommended way to consume Mobile SDK libraries.


Wow, sorry my bad it should be {{cookiecutter.repo_name}} ... I'll update the PR tonight


reset_mdcache is not used. We have to check the flag here, and reset MD cache is flag is set:
```
else
	{
		// If a catalog change was detected, purge MDCache contents
		if (reset_mdcache)
		{
			CMDCache::Reset();
		}
	}
```


"React Native" (space between the words, x 2)


I see this is used (not in this PR) but how was it not defined?  How did it compile?


Please add a period.


Can you clarify in the docs that the randomization only applies to the initial offset?


Sweet!


`map` will already return this array. You don't need the next line


Zeile 806,
Da steht jetzt:
return "Unknown argument $a[1], choose one of " . join(" ", sort keys %sets)

Damit werden nur die Keys aus dem Hash ausgegeben.
Wenn man ein dropdown möchte, dann müsste man es in etwa so schreiben:
return "Unknown argument $a[1], choose one of MS:on,off MU:on,off " . join(" ", sort keys %sets)

Damit das klappt, wäre etwas in der Art:
  return "Unknown argument $a[1], choose one of " . join(" ", {$_.":".$sets($_)}  sort keys %sets)

notwendig. Die Syntax passt aber nicht... bin jetzt doch zu müde, fehlt vermutlich ein map


@Dashed sure, probably after I get working HTTP :)


typo, should be `true`


Same clarification that this is an initial offset, please.


I just did a complete search for 'icms_header' and icms_footer' in the core and they only show up as template variables, not as functions - php, js, or otherwise. I also search the latest 1.3 release and didn't find them there, either.

There are icms_cp_header and icms_cp_footer, though, in include/cp_functions.php


@laurelfan was removing these strings intentional? Are they no longer being used in the teacher dashboard? Or were they double included in different files and you're removing them from this one?


I think the intention is that it could be somewhere outside of assets, but if that's the case it should probably be outside of the web root (which this config doesn't even support).

In that case I guess making it relative to ASSETS_DIR is probably fine.


You can certainly use the gem for that ... 3Hub is just a file viewer.


Agreed. :)


In the second test of the trainingControllerSpec, you assume that there is a training in DB, and it should not be so!


This should also remove the `Folder` entry in SilverStripe, right?


The failsafe_kill_switch should also be conditional and moved down to line 1212


It should be UP Social. :)


The second field here is the name for the long command-line option name, which normally shouldn't include a space. Looking at the manpage update, this should probably be just `"randomize"`.


anywhere where a period isn't the start of a urlsegment.

E.g. `path/parent.dir/file.ext`

but not `path/.parent/file.jpg` or  `path/parent/.hidden`


We also don't want to generate invalidation messages for catalog tables updated during DML. ORCA only cares about schema or statistic changes, not data updates.

In the original commit we had this check:
```
   /*
    * We don't track(bump command id) catalog tables changes in the AOSEG namespace.
    * These are modified for DML only (not DDL)
    */
   if (IsAoSegmentRelation(relation))
   {
	return;
   }
```

Do you think we can add that exemption in here as well?


Maybe "randomize beginning of -t interval" might be a little clearer.


EconomizerIntegrationNew (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (906 of 906 tests passed)

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


easy - just tell PhpStorm to format it ;-)


@KrauseFx Doesn't quite fix it :( The agents default temporary dir is `/path/to/BuildAgent/temp/buildTmp` and so it still will fail these two checks


Can we use a #define constant instead of -1? Numerical constants are hard to search for in the code. Thanks.


I think unit tests would make this method a lot clearer.


@systembugtj a PR would be accepted very quickly :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9516](https://teamcity.mesosphere.io/viewLog.html?buildId=93395&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 174 Build time: 00:03:13


merge!


readme


Ooooh, fancy!


Right here. Remote execution has failed, but you caught exception, complained a bit but continued


collation: utf8mb4_general_ci で大文字小文字区別しないになっているので
ここを `name = ?` にするだけではだめでしょうか?


@MMajko Are you aware of the fact that `groupEventsByDays()` actually produces an object with keys (day numbers) which are Strings, not Integers?


Should name these parameters properly now, something like `Object cause, Object... context`


looks good. thanks @ShadowManu!


`object` should not be Nullable


Welcome Swift!


I like your solution, nice work!


@yujuhong Why we use a pointer as c.timestamp? If we don't use a pointer for the timestamp, then we can get rid of this.


Was this removed again by mistake?


Full sentence is emphasized, so period should probably be included in the markup.


Clarifying: The data set doesn't have only full missing days; it has missing observations within days, which are important too.


Looks like this was a run-on sentence.


This is missing a parenthesis.


@JohnHamby This is largely an experiment of mine, having a single array initlizer type feels more natural to me, as a bonus we now can use existing bound node (`BoundArrayInitialization` in both CS and VB ) for it tosimplify the logic. The best part is it seems works OK :)
Just curious, what's the reason for having `IExpressionArrayInitializer ` and `IDimensionArrayInitializer`? Did I missed something important here?


ROCKSDB_PRIszt requires % before it just like any other PR formatting


can we just pass the pod's UID?


Parallel construction: adding "." at end of imperative-sentence list items that lacked them, to be consistent with similar list items toward the end of the document.


Italicizing since it's placeholder text/metasyntactic variable.


I feel we really should return nil if the podstatus is not found.


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

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


If you had more than 1 beer, it would display X bottles of beer on the wall (2 times). the Instructions show it's one time per repeat. with this mod it works as per the instructions.
![screenshot from 2015-12-15 22 52 04](https://cloud.githubusercontent.com/assets/8425834/11827555/e373d71c-a384-11e5-82d6-52a8a7d9904f.png)


What do you think about prefixing this name with a `$` or something to make it really clear that its not an actual name if you're looking at the name map that its an internal name. Should probably also be the simple class name not the full name.


This comment is no longer true, right?


formatting noise.


$query = "DELETE FROM interests WHERE offersid ='".$id."'";
ist falsh


LGTM


I agree


@RJP43 @spadafour  If you guys want to jump in, here it is, by 7PM... Let me know if you're going to take hold, or if you are going to work on this. I edited up to line 91. Thanks.


I agree by email

On Tue, Dec 15, 2015 at 8:59 AM Jonathan Garbee <notifications@github.com>
wrote:

> Well, I knew the test was somewhat buggy, but I thought I fixed it by
> checking the class vs actual input focus state (since that never works in
> the test system it seems.) It also passed in the PR with the fixed test, so
> it must have just been outright buggy to work locally and for PR then fail
> once merged.
>
> Active element testing is hard. ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/google/material-design-lite/commit/5b491f6f9879b14f78f869ffab17c1043029ad05#commitcomment-14986068>
> .
>


If we created a `PathMatch.fromAuxiliaryRouteOnly(urlParse.auxiliary)` then would be ever use this elsewhere?


Just a test comment!


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


Use 2 (not 4) spaces for indenting an expression spanning multiple lines.


Do not use prefix `_` for a variable that is used.


Line is too long. [109/100]


Use the new Ruby 1.9 hash syntax.


It could be be a part of addBatch too as long as it's atomic, like you said.

Also, I haven't thought much about this but if we ever foresee the cadence of addBatch to be different than the cadence of committing, it may make sense. I, personally, don't see why we wouldn't do a commit every time we add a batch.


if rPodStatus is an empty PodStatus, and err is not nil, then why not just fall through and call runtime.GetPodStatus() again?


Why did you remove the `AddHandler` and `php_engine` settings from the previous `.htaccess`? https://github.com/silverstripe/silverstripe-installer/blob/master/assets/.htaccess


I suggest the setters return new instances, like Transform. Otherwise modifying one variable requires creating a builder and feeding it each of the previous explosion settings.


It isn't a bug. It is permitted by the CLR specification (much to my dismay).


I don't understand why we need to cache the error?


@esteve I noticed this test fails on OSX consistently.

Do you think maybe the reason it fails is because the test matches this line from add_two_ints_server which happens in the callback, and the process gets killed by the test launch handler before it can send the response message?


...And I am ALSO heeeeere!


resolves #48 (I hope)


@stevekinney would you mind taking a look at this and seeing what you think? need to figure out a way for that leftStart function to evaluate correctly..


This should be 1.1.0


done

On Tue, Dec 15, 2015 at 3:34 PM, Sean M. Pappalardo <
notifications@github.com> wrote:

> Hey do you want to do what I did and #include softtakeover.h so you can
> use the constant defined there (times 2 for this case) rather than
> hard-code it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mixxxdj/mixxx/commit/1476879743c437972322264394456d022895e98c#commitcomment-15000133>
> .
>


I'm not sure this will work at all. Won't bundler get mad? And wouldn't it get mad if you had `gem "chef", git: "blah"` in a gemfile somewhere else? Maybe in-repo we always have `MAJOR.MINOR.0.dev`, but build overwrites that with `MAJOR.MINOR.BUILD` ?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9518](https://teamcity.mesosphere.io/viewLog.html?buildId=93398&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:03:58


Thanks!


Did you notice the NULL within the above ```#ifdef``` is still present?


Yeah that sounds reasonable (getting rid of `getTupleConstant`).  This will create the tuple once for each time we compile the method, but that's probably not too big a deal.

The other reason we have those functions is for GC purposes -- we don't automatically track references to objects that are embedded directly into code.  Instead we have to register it with the `BoxedModule` object that owns the code; the `getIntConstant` and similar functions take care of this.  We'll probably need to add a `BoxedModule::registerConstant()` function, which could just append the object to the `keep_alive` list.

Other than that, I think it's just a matter of adding a test and then creating a PR.  For the test, we could do something like
```Python
# stat_check: noninit_count('gc_registered_bytes') <= 1000000

for i in xrange(1000000):
    (1, 2, 3, 4, 5)
```


We have to use ngdocs for documentations


cc @mrshu


This isn't portable, looks to be unused and should be removed.


Fix-allocation-for-feneSolarDelaySeq (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1983 of 1985 tests passed)

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


I was wondering about that. If you use the templates as written I don’t think anything ends up outside a frame. So there’s nothing to ignore. However, the issue this was designed to address in pull request #86 was a purely cosmetic one about numbers on the title slide which might not be worth changing, and could be changed in a different way by adding `plain`.


would using a different name cause too many problems?  It would certainly flush out the places that assume a name, but I guess it would be to breaky.


no comments on this commit


I have tried to reproduce what you apparently have seen without the patch... but failed.  I did not need any changes to use the standalone NDK toolchain(s) to build flashrom. The `linux_spi.c` change was already added as part of r1898 and I don't think the endianess patch is necessary at all. I have not tried with the android build system though... did you actually see any `hwaccess.h`-induced breakage, @mbolivar?


I bet that breaks host networking with CNI.


Fix-allocation-for-feneSolarDelaySeq (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


LGTM


Can do this to avoid using c style enum and still use c++11 strongly typed enums :D.


[![Apothecary bottle with mynah birds and peonies](http://api.artsmia.org/images/122413/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=122413)


[![Figure A (Geometrical Patterns)](http://api.artsmia.org/images/121655/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=121655)


[![Fire Escape, No 2](http://api.artsmia.org/images/121832/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=121832)


[![Fire, from the series The Four Elements](http://api.artsmia.org/images/2/300/small.jpg)](http://collections.artsmia.org/?page=simple&id=2)


Ah good point, I'll fix that!


these sort of things happen all over this code... anyway, coordinator only is a single node, so it is very safe to ignore


I did not change any of the messages


Might as well since I'm modifying the javadoc anyway


Corrigido.


添田です。
ここも良くないですね。そもそもここを通過しないため、問題は起きていませんが、仮にwhile文の条件式が正しく設定された場合、永久ループになります。他のソースコードではきちんとできているので、うっかりミスだと思いますが。


I'd change the name to just Copyright, doesn't make sense to have possessive pronouns for something like this.


Note that we don't use github for issue or patch tracking, see the README file for more info.

I have committed a modified version of this (our style guide says the sys/*.h includes go together, and not all platforms have a sys/time.h so it needs to be wrapped in an ifdef).


添田です。
「atsushi」などは、文字列ですね。文字の場合は、括ってあげてください。


インスタンスメソッドの公開の精度については、テストをする時に必要だと思えるメソッドが呼び出せればいいかなと思っていました。
仕様を見ると、確かにparseメソッドのみが呼び出されるようにするべきな気がしてきました。


FYI `cameras` and `lights` blocks are optional, you will get a sensible default if you leave them off (e.g. nothing in this scene is really affected)


this one was fixed in c528c1b4af2f06712177b3de9b30705752f7cbcb via https://bugzilla.mindrot.org/show_bug.cgi?id=2296


TeamCity ReOpenLDAP :: ps-stable [Build 99](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1415&buildTypeId=ReOpenLDAP_PsStable&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 00:25:32


Hey!


nice !! #lol


添田です。
引数を用意してあげてください。このままだと、引数を受け取れないですね。


In case you didn't know, a width without units is going to be in meters, fixed as you say in the docs. If you wanted to fix it in pixels, you could do `5px` or whatever. Either has its uses.


Need to test with concurrent scroll views


Hey Shane


`CSVParser` を見たときに利用する人は、どのメソッドが呼べるんだろう？というのを考えます。
そのとき、CSV(ファイル)をパースする以外のメソッドがあるとだいぶ混乱しますね。


It might be a more complete solution to apply a "white-space:nowrap" style to the tr or thead to keep all the th's from wrapping the sort button to the next line.

Additionally I like allowing the message column to use up any unused space on the line so that fewer of them wrap (style="width:100%").

					<?=gettext("Time")?>
					<?=gettext("Process")?>
					<?=gettext("PID")?>
					<?=gettext("Message")?>


添田です。
ここは、作業途中ですかね。。。？


添田です。
ここも名前が文字化できていませんね。


添田です。
不要なelseは書かない方が良いですね。defaultと同じです。


I find the name a bit too similar to `"registry/handlers".imageManifestHandler`


添田です。
ここの$numは、ちょっと危ないですね。その時の中身を出力したいのであれば、echo $shokiで大丈夫ですよ。


cause string 이 적합한가 ? 아래 string아 나아 보임.
- "interrupted in the process of authentication"


We don't need to keep the "original" value comment here. It's the Peer Reviewer job to compare the changes made. And even if she misses something important, we can go back on the repo history to check how it was before, or create a new fix, so don't worry about keeping history that way ;)


all tests pass:
success: http://buildbot.bos1/builders/python-client-test/builds/429


saw approval from alexmoore
at https://github.com/basho/riak-python-client/commit/5cefff83fffb0f9484228ebf39e691756d02768b


merging basho/riak-python-client/feature/bch/2.3.0-release-notes = 5cefff83 into borshop-integration-419-feature/bch/2.3.0-release-notes


basho/riak-python-client/feature/bch/2.3.0-release-notes = 5cefff83 merged ok, testing candidate = f99e4277


> @miketaylr request_headers are defined here but never used.

@karlcow isn't it used here on line 54 and 51? :see_no_evil:


is it necessary to have rows and col?  isn't nested enough eg fridge -> shelf -> col -> row are all nested so why do we need an extra fields?


"Cancelled by application" 이란 string이 더 맞지 않는 지 ??
위의 다른 부분에서 cause string이 정확한 문장이 되는 지 확인하세요..


例外については、指摘が欲しかった点です。
今回の場合だと`transpose`を使っていて、配列の要素数が合わないと`IndexError`を発生させると書いてあったので、テストに書いてみました。
確かに、どういう時に`IndexError`が発生するかがソースを見る限りだとわからないですね。


Should member variables be instantiated in the `m_var` portion of the constructor, i.e. `m_workspace`, etc?


Nitpick: seems weird to have these ctors since `TypeName` is implicitly convertible from `Type`and `string`.  Maybe we should just have the one ctor that takes a `string` for the sake of CLS compliance?  That's the only one really being used anyway.


Yeah, I'm sure we can come up with something better. It's not a store, and it's not a service, so I'm sure what to call it. `ManifestSerDes`?


This setter is not necessary, the constructor is the best choice.


im the best


SurfaceProperties (macumber) - Win64-Windows-7-VisualStudio-12: Build Failed

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


This is still in the Java part, right? If so, then it's wrong, Java doesn't have keyword sealed. However, it has others keywords that may be applied to variables/methods: volatile and synchronized.


I think that is missing a constructor with the parameters, to initialize an instance we should always pass that values transactionName and applicationName


This looks like a whitespace change - can you revert this and leave the rest


인자로 주어진 cause 스트링과 prefix 붙는 "Cancelled because"가 연결이 되면 정상 문장이 되는 지
cancel() 하는 모든 곳에서 확인하세요..

예를 들어, "Cancelled because by application" 이란 스트링이 나오게 되요..
"Cancelled by application" 이면 적당한 문장이 되네요..


なるほど、クラスの名前を超える機能が公開されていると使う人が混乱する訳ですね。
テスト中心に考えていて見落としてしまいました、気を付けます。


so this change was necessary on my computer to guard to run correctly.

After you do:
- git pull

do also:
- bundle


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9409](https://teamcity.mesosphere.io/viewLog.html?buildId=92752&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:03:36


:+1: thanks for this fix, @atkinsj


여기서 만들어지는 string의 예를 한번 들어주면 좋겠어요 ?? "by"가 맞는 지 모르겠네요..


Not just methods, fields may be static too (class variables).


@yracine I understand your frustration to a degree. And I do appreciate all that you have done with your work and the help you have provided users on the forums.

I would have been willing to donate before, heck I was even offering to help you _for free_ to structure your code to make it easier for you to support for others (including myself) and asked for nothing in return.

However, given the way that you reacted to the situation, why would any of us trust giving you money now? What is to prevent you from raising the fees later? Or to simply stop supporting the software at any point in time in the future (at which point we would all be left helpless)? You have said it yourself that this is a hobby and that it starting to take too much time, so what will we do when you end up having to prioritize your family over a few donations that you received?

What if we were to spin this around? How much in donations would you need to consider releasing the code with an open source license? You still own the copyright to anything you wrote, but then others would be free to contribute and not live in fear that the code will simply disappear some day. And since you are still the most knowledgeable about the codebase you still could monetize your support.

Just a thought


I didn't understand why this merge was here.

This is validating the parameters that are necessary to create or update a recipe.

If you want to merge the current_user you should not permit user_id and then you need to change the tests to not send a user id and then the merge makes sense(and that's better :) ).

Do you understand what I mean? try to do it. otherwise I will do it and show you.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9408](https://teamcity.mesosphere.io/viewLog.html?buildId=92739&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:08:29


もちろん意味があって使えるメソッドが公開されている分には問題はありません。

だた、公開されているメソッドが他のメソッドのアウトプットに依存しているので、むむむ…ってなる、という感じですね。


you already found the recipe once in the find_recipe before filter. no need to do that twice.


Can you get rid of tabs please.


there was missing a couple of important stuff and you can always erase comments.


you should never do it.

But because this code is not in production(meaning that nobody is using our application yet) and because it's only comments that were too much on a old migration I did it.


Just updated the post on the raspberry pi site and the readme on the water measurement github.

________________________________
From: Steve Lambert <notifications@github.com>
Sent: Thursday, December 10, 2015 3:50 PM
To: dannyfabricant/distance-measurement
Cc: FABRICANT, DANIEL
Subject: Re: [distance-measurement] Merge pull request #1 from slambert/master (f321c55)

This was meant to be so embarrassing you wouldn't leave it for more than a day.

-
Reply to this email directly or view it on GitHub<https://github.com/dannyfabricant/distance-measurement/commit/f321c55cb09c85c07e6743bdcdf1360f93170236#commitcomment-14911022>.


I moved this to the top because the files that are included in base.inc.php are going to need autoload setup before they run as we move towards namespacing our work.


@stephenlb do you know what was not working? I will like to fix it if you think this functionality is useful...


It would probably be best to preserve the enum type here (i.e. change this to `SessionState m_session;` and make `getSession()` return type `SessionState` and `setSession()` accept an argument of type `SessionState`), rather than implicitly casting it to and from `int`.


fast-forwarding master to borshop-integration-418-features/lrb/riak_pb-submodule = f558141b


Merge sha f99e4277 is stale.


merging basho/riak-python-client/feature/bch/2.3.0-release-notes = 5cefff83 into borshop-integration-419-feature/bch/2.3.0-release-notes


basho/riak-python-client/feature/bch/2.3.0-release-notes = 5cefff83 merged ok, testing candidate = 59ddc399


all tests pass:
success: http://buildbot.bos1/builders/ruby-client-test/builds/418


This is an attempt to start a namespace for this project. Generic classnames related to common things like sessions scare me now, so I figured I'd make it clear that this is a NW specific factory.


与 node sdk 保持一致，云函数调用

```php
Cloud::runFunc(...);
```
调整为
```php
Cloud::run(...);
```


Mistake, this fixed #4 not #6.


そもそも利用者視点で見た場合、 `parse` した結果 `IndexError` が発生すること自体に違和感があります。
`IndexError` は、http://docs.ruby-lang.org/ja/2.2.0/class/IndexError.html にあるように添字が範囲外のときに発生する例外です。
`parse` を呼び出すのに添字を使用していないのに何故 `IndexError` が発生するんだろう？
っていう気持ちになります。


本当ですね、`parse_line_header`は`parse_csv_header`に依存してました。
依存関係にあるメソッドを公開するのは、ユーザを混乱させるのでよろしくないんですね。


applied upstream, thanks.


Despite the doc seeming to indicate that session_destroy() === $session->invalidate(), testing showed that $session->clear() is required.


the reason why this didnt work was because when you create a recipe:

```ruby
let!(:recipe) { create(:recipe) }
```
the code run by this is the one in the factory:

```ruby
FactoryGirl.define do
  factory :recipe do
    title 'recipe title'
    description 'recipe description'
    user
  end
end
```

that user line inside of the recipe factory is telling that it should create a new user from the user factory.

So when you did:
```ruby
login_user
```

and you had this:
```ruby
def login_user
  before(:each) do
    @request.env["devise.mapping"] = Devise.mappings[:user]
    user = FactoryGirl.create(:user)
    #user.confirm! # or set a confirmed_at inside the factory. Only necessary if you are using the "confirmable" module
    sign_in user
  end
end
```

you were creating another new user.

And when you compared both here:

```ruby
def require_permission
  if current_user != Recipe.find(params[:id]).user
```

They were not the same, hence this didn't work.

How would you have found out? Debugging for example:

```ruby
  raise (current_user != Recipe.find(params[:id]).user).inspect
```

if this returned true it means that the users were not the same, then you could have thought why.


Are you sure about this? I think that you wanted to use attr_accessor.


Sorry if I broke something...

I fixed [this bug](https://github.com/fpillet/NSLogger/issues/168), basically `[message description]` was defined only in `DEBUG`, so it couldn't work if the app was not compiled in debug.

When I test it everything was working well and I had the colors on my mac.


@tchalvak Please review this line for intended behavior of whichever. In testing this patch works but the use of whichever() worries me that there may be edge cases not in the test suite that you might know about.


あー、依存してるから、というかたぶん意図しない使いかたをされるかもですね。

たとえば、いきなり

```ruby
CSVParser.new.parse_line("A,B,C,D,E,F")
```

みたいに使うとか。
そういうのって、 **お題が** というよりも、作成者であるところの @yuta-muramoto さんの意図通りですかね？


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9410](https://teamcity.mesosphere.io/viewLog.html?buildId=92759&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:03:58


This sets the static instance of the session up with a mock storage engine for testing. All subsequent calls to ::getSession() throughout the code will get this fixture.


@benjamn be aware - I think this was the intended logic and it fixed my bug, but you might want to check if this is correct.


There should be class or module, not def.


@tchalvak This might be simplifiable with the default parameter to $session->get(). Please review for intended behavior and maybe submit a simple patch if it will do what you expect it to do with less complexity.


We also should run the same steps (i.e. run the session close algorithm on all open MediaKeySessions) in the other case that handles decryption failure, i.e. after the "Use the cdm to decrypt the block using block key" step in the "If decryption fails" block.

Maybe we should add a new "Handle unresponsive CDM" algorithm, which performs the steps you outlined here, and call it here and below where we also handle decryption failure?

Or, you could add this failure/unresponsive CDM handler at the end of the algorithm, prefixed by "If the CDM is unresponsive or any of the preceding steps otherwise failed...".


This is your code according to git...


なるほど、ファイルを渡しただけなのにindexがおかしいというerrorが帰ってきたら、違和感しかないですね。
作っている自分はわかっているので、当たり前に分かっていると思ってtestを書いていました。これはダメですね。


I fixed your problem and left a lot of comments that I hope helps you understand what you were doing wrong.

There is a couple of notes:

- individual permissions should not be a problem to tackle now. It's more complex and even though the way you implemented is simple and works for now. later will turn too complex and we will be using something like this: https://www.phase2technology.com/blog/authentication-permissions-and-roles-in-rails-with-devise-cancan-and-role-model/
https://github.com/RolifyCommunity/rolify/wiki/Devise---CanCanCan---rolify-Tutorial
https://github.com/CanCanCommunity/cancancan

- you should probably worry more about, what pages can we have access only if we are logged in:
https://github.com/plataformatec/devise#controller-filters-and-helpers

But everything else seems to be fine on this PR.


Wow, this is the last time I'm using the GitHub web editor.. it leaves incomprehensible diff reports.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9411](https://teamcity.mesosphere.io/viewLog.html?buildId=92766&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:03:56


Test comment


That is an interesting idea...

I prefer the name "use case" because that has a matching concept in the language of software design. The word runner doesn't really have a clear matching concept in the same way.

If you wanted to change something, maybe change .run to .perform or .do so that it seems a little more "use casey".

Putting it under models is fine. I'm not super attached to having it it's own directory.


This appears to have broken the shebang processing.  Running tests now.


@tchaikov .  For reserved replicas, it own free using clear_scrub_reserved. For the on_shutdown, there is only one implementation which don't care ec or no-ec.


thanks, fixed.


Looks good


I believe this introduces #20.


Hi @martinbomio the main demo https://stephenlb.github.io/webrtc-sdk/


そうですね、`parse_line`に関して言えばそういう使い方も出来るように書いていました。一応公式のCSVモジュールのリファレンスを見てちょっと`parse_line`メソッドがあったので真似してみようと思って書いてみました。
しかし、`parse_line_header`の場合はそのように直接呼び出される事は意図していませんでした。


@jrdietrick 105 110 and 111 are all issues with the pypi release of 0.8, current master (and my package 0.8.1) works with django 1.8, I am using it already.  Your PR #124 does add tests though so I will look into using it if I have any problems with 0.8.1


How can I help with this?


woo!


:+1:  enumerable


The autoconf guidelines advise using exit rather than return because on some platforms the return value is not propagated.

http://web.mit.edu/gnu/doc/html/autoconf_5.html:
" Test programs should exit, not return, from main, because on some systems (old Suns, at least) the argument to return in main is ignored."


lol oh this is old hahaha


Not this commit, it's the stderr handling (can be caused by unrelated login shell issues.)


I think servers and replicas should be added to the `servers` and `replicas` field here rather than in `createServers` or `createReplicas` so that one can call `createServer` or `createReplica` to test adding/removing servers, as is done with clients.


Servers are already added in `createServer`


ナイスチョイス! SHA1とか弱そうなハッシュ関数よりはこっちのほうが断然いいb


Merge sha a3fa7bb6 is stale.


merging xing/riak-ruby-client/PR-microseconds = 7b6e33d5 into borshop-integration-247-PR-microseconds


xing/riak-ruby-client/PR-microseconds = 7b6e33d5 merged ok, testing candidate = d51ac334


Why is there a single-quote before `$char`? Could that be the cause of your unexpected "fish: Wide character 776 has no narrow representation" error after adding __-x__ to the set command above?


This is going to be so awesome!


вот этот еще не сделал, не понимаю как и есть проблемки которые мешают: список обновляется дважды и забивается одинаковыми фото например , при том что метод onLoadMore() дважды НЕ вызывается , то есть проблемы на уровне SuccessLoadedUrls()


Unfortunately, this introduced a warning for other platforms:

src/lib.rs:17:45: 17:57 warning: unused or unknown feature, #[warn(unused_features)] on by default
src/lib.rs:17 #![feature(core_intrinsics, core_slice_ext, core_str_ext)]
                                                          ^~~~~~~~~~~~

Any ideas on how to fix it? Does #[cfg(feature = ...)] work for this case?


My phone


This is a test comment


Can not we have this constant in a common file as (3 secs or 30 secs) used in other places like auth_test.clj, drpc_auth_test.clj, nimbus_auth_test.clj?


The new version with all of the macros wrapped correctly (I think) can be found in 6220b8d


I don't think you meant this.


I don't know what you are talking about.   We did not knowingly strip
anything and website links in via ABOUT tab and GIT HUB tab to main hashkat
with attributions and license.  Further this is not a live site and is not
complete.  We welcome your input but please be more clear about what you
are talking about

Linda
On Dec 14, 2015 2:39 AM, "ludamad" <notifications@github.com> wrote:

> Why have you stripped the license off the file copied from hashkat? This
> isn't something you should do, or need to do.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JPaveg/HashKat-Mockup/commit/d6b28e582ac9af1af6f635345e98be80cf2f748e#commitcomment-14949658>
> .
>


Or the sort button could be forced below each header.  Really just depends on how a person wants it to look.

`

		<?=gettext("Time")?>

		<?=gettext("Process")?>

		<?=gettext("PID")?>

		<?=gettext("Message")?>

`


Fixed.


tretete


@biologicaldynamics

Accidental commit to master. Same code as PR. Please remove.


Thanks @stephenlb. I think I found the issue. How do you think I should test it with the demo?


Does the column belong to a row, or the row to the column? And what is it that gets a barcode?

There are a couple of good use-cases for why you’d like to define a row/col position in a location.

For example: you are going to put a tube in a box. The box is a 8x8 chipboard box. You scan the tube and CBP-LIMS asks you where did you put the tube? You scan the barcode of the box. CBP-LIMS now asks, which position in the box? And you type C3 - or you show them a nice grid of rows x columns and let the user click it. (You’ll still sink my battleship - I always hide it at C3).

If you just used nested locations, you’d have to scan the box, select the row, select the column. Instead of that nice grid, you’ll get a wizard interface with “Next >>”, “Next >>”, “Next>>” all the way down.

Example two: plates.  Where did you mix those PCRs? Oh, they’re in a plate? What’s that barcode again? Where in the plate? Oh all the way down in H12?  Plate #123 (H12) makes more sense than Plate #123 > Row 8 > Col 12.

Practical reason: If you use rows/cols in locations, you don’t have to pre-populate the database with all possible rows / columns.

Let me know if you think I’m wrong…

> On Dec 14, 2015, at 10:14 PM, tree101 <notifications@github.com> wrote:
>
> is it necessary to have rows and col? isn't nested enough eg fridge -> shelf -> col -> row are all nested so why do we need an extra fields?
>
> —
> Reply to this email directly or view it on GitHub.
>


fast-forwarding master to borshop-integration-247-PR-microseconds = d51ac334


identation 확인


warning 메세지와 debug 메세지도 함께 있네요..
warning 메세지이면 충분할 것 같고 operation 정보가 포함되면 좋을 것 같네요..


Yes that's my understanding as well.  But that's also the reason for the switch to the DCO, is the realization that this might be a better approach from the previous CLA.


Sorry. I tested with zsh 5.2 and bash 4.3.42, and it tries to find a file called `$URL`. `"` already does shell expansion so you don't need the backticks (``` ` ```), just `echo "webhook-url=$URL" >> $DIRNAME/crust.conf`.


thank for fix this question,i have the same question too
but  if evt.sessionStolen is true  ,  there is   other connection with same clientId ,why remove clientId from m_clientIDs,  this is possible cause  delete  the lastest connection desc  which created  in ProtocolProcessor#processConnect   from m_clientIDs  too


This would actually just call "Undefined" not handleUndefined. I think you're close. I would just change it to something like this.

```php
$event = isset($event['event']) ? $event['event'] : 'undefined';
$method = 'handle' . studly_case(str_replace('.', '_', $event));
```

That way it will actually call handleUndefined. I think that was your intention. Also do you know which event returns without this key so I can write a test case for it?


Thanks!


why removing comments cause the checks to fail?


We have to catch this exception. So I will throw a different checked exception and catch it. Thanks


fixed


Hmmm, my previous comment appears to be bogus. I see you're relying on some undocumented behavior of the builtin printf command (that is also undocumented in the BSD printf command on OS X) that treats a char prefixed with a single-quote as its ordinal (e.g., Unicode code point) equivalent. But this implies that you really want the change to __LC_ALL__ to only apply to the `read` and `for char in $chars` portion of this function but not the fish builtin __printf__ command run by the parenthesized expression. And that seems wrong and suggests there are still problems with this solution.


添田です。
良いチョイスですね！この設定はPHPの設定ファイルでも設定でき、プログラム上でも変更ができるのです。
ついでにPHPの設定ファイルにつてもちょっと勉強してみると良いですよ！


Better to use `usleep`.

```php
$sleeptime = $request->header('Retry-After') * 1000;
uleep($sleeptime);
```

`sleep` only sleeps for whole seconds, `float` to `int` conversion truncates, which means that if `Retry-After` is set to say `2900`, PHP will only sleep for 2 seconds, sending the retry 900ms early.


添田です。
この開始時間と終了時間ですが、この間に自分で調べた関数の処理を挟んでください。
（今回で言えばdate_default_timezone_setですね）
処理があっという間に終わるため、開始と終了の時刻は同じになるかもしれませんが、
たとえばループ処理などを挟むと、その処理にどれくらい時間が掛かるかわかります。


I'm in Spain pre Xmas. I will look into this as soon as I return.  Can you
send me a screenshot of what is missing?  Thanks.
On Dec 15, 2015 5:06 AM, "Linda Kettle" <linda.kettle@uoit.net> wrote:

> On Dec 15, 2015 4:58 AM, "Linda Kettle" <linda.kettle@uoit.net> wrote:
>
>> I don't know what you are talking about.   We did not knowingly strip
>> anything and website links in via ABOUT tab and GIT HUB tab to main hashkat
>> with attributions and license.  Further this is not a live site and is not
>> complete.  We welcome your input but please be more clear about what you
>> are talking about
>>
>> Linda
>> On Dec 14, 2015 2:39 AM, "ludamad" <notifications@github.com> wrote:
>>
>>> Why have you stripped the license off the file copied from hashkat? This
>>> isn't something you should do, or need to do.
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/JPaveg/HashKat-Mockup/commit/d6b28e582ac9af1af6f635345e98be80cf2f748e#commitcomment-14949658>
>>> .
>>>
>>


添田です。
細かいですが、「/」ではなく「-」で区切ってください。それ以外はOKです！


Agreed.  I put in the -k fairly early in the development when I had problems with containers shutting down (they were probably misconfigured).  Forgot to remove this when the problems were resolved.


@stephenlb see https://martinbomio.github.io/webrtc-sdk/. If you feel comfortable with the solution, I can create a PR


Hmm I guess my logic has always been plate 123, then the nested address is H12; so there are no nested row 8 >col 12;  so it would just be you create a box called h123, then you just make nested addresses of however you have it set up.  It would be flexible too since this could be for any shape size container because is up to the user to assign it.  But I'm fine with either way.
* would it work if row and col are preassign in another table and you could choose a predesign box?


That's a bit beyond my current level of C++.


I think this should accept status 0 under some conditions, because Chrome 47.0.2526.80 with `--allow-file-access-from-files` gives status 0 for successful loads from file:// URLs. I submitted https://github.com/kripken/emscripten/pull/3965 to fix this.


please don't use mu handle to document your code :p thanks!


Applied upstream.  Thanks.


@skolsuper, yeah I can see it working alright for you at 0.8.1, and it almost did for me. There are two lingering issues which depending on your environment may or may not affect you.

1. configurations 0.8.1 still doesn't correctly cooperate with the old shim code for adding the `--configuration` option to apps which have not upgraded to using argparse. If I use, for example, `django-extensions==1.5.0` and try to run `python manage.py shell_plus --configuration=John`, I get `./manage.py: error: no such option: --configuration`, because **django-extensions** at this version is still using optparse, and configurations doesn't fall back correctly. One answer is "dude, you need to upgrade your django-extensions", but there will be plenty of 3rd party packages out there which aren't changed to use argparse, for a few more months at least. **django-extensions** is not the only one even in my stack where I encountered this issue.

2. Recursive management command calls are not handled correctly in 0.8.1. This only affected me when I used `django-nose`, because it has some management  commands which call other management commands.


The only


@miketaylr it was… but not anymore. It's why I noticed it. Check the current code.
https://github.com/webcompat/webcompat.com/blob/master/webcompat/api/endpoints.py#L39


Would it be better to stop "using namespace llvm;" ?


プログラミング基礎学習２　改訂版.php

添田です。
細かいですが、全部文字ではなく数字と文字の混在をやってみてください。


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9412](https://teamcity.mesosphere.io/viewLog.html?buildId=92781&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:08:05


not bad.


Find the sum of all numbers natural numbers below X that are multiples of 3 or 5, where X = 1000.


Let me write a little PR.. You can actually use the backup dir for disk usage…


Yeah, you're right, this is a bit sloppy. I have revisited it and I _think_ I mostly introduced it to fight off a race in callbacks scheduled by `scheduleAdvance` and `scheduleRetry`. What I've done instead is remove `scheduleAdvance` and call `advance` synchronously in those contexts where it was previously scheduled. This change seems to remove the need for the flag.


Is there any other information you want to display about a a user?  Maybe their name?  What other properties might it be nice to have?

Remember the requirements for projects are purposefully vagure.  It's up to you to decide on some of the specifics.


添田です。
中村さんのPHPファイルは必ず<html>の記述がありますが、これは必須ではありません。
こういったutilファイルは結構PHPの関数のみだったりします。
利用しないHTMLは削除する用にした方が良いですね。


@martinbomio PR = :+1:


Do you really want to create 384 blank locations for a plate? Or do you want to traverse the child-list to find which one is Row #2, column #7? You’ll have to have the order of children embedded in there somehow anyway (either as a field or as the name). Might as well just include it as fields to begin with. And I don’t trust names in a database to include information… H12 means something to me, but it doesn’t mean anything to the database. You’ll want to draw a grid for the location to let Stan click where he put the tube. Which is easier to code => interpret H12 as a position, find the # of children and draw an appropriate grid based on the *name* or hard code rows/cols?

Fully nesting things has a certain elegance, but you start to lose the elegance when you have to keep children in a proper order, which you’d have to do in a drop-down box.

Try it with the row/col fields first and see if there is something in there that you vehemently don’t like. I think it will make your life easier to include them.

> On Dec 14, 2015, at 11:11 PM, tree101 <notifications@github.com> wrote:
>
> Hmm I guess my logic has always been plate 123, then the nested address is H12; so there are no nested row 8 >col 12; so it would just be you create a box called h123, then you just make nested addresses of however you have it set up. It would be flexible too since this could be for any shape size container because is up to the user to assign it. But I'm fine with either way.
>
> 	• would it work if row and col are preassign in another table and you could choose a predesign box?
> —
> Reply to this email directly or view it on GitHub.
>


### Debug this simple game program. The game should generate a random number between 0 and 100
### when the user hits Enter. The user can stop the game by typing "Stop".
### A score over 50 should result in a win.

import random

def roll():
    roll == raw_input("Press Enter to Spin the Wheel or Type ""Stop""!")

roll = 0
while roll != "":
    prize = random.randint(0, 100)
    print "You're number is " + str(prize) + "!"
    if prize < 50:
        print "Sorry, you didn't win. Try again"
    if prize > 50:
        print "Congratulations! You're a winner!"

    roll = raw_input("Press Enter to Spin the Wheel or type 'stop'!")

elif roll == "Stop":
    print "Thank you for playing"
else:
    print "Well, so long!"


It'd only become invalid if the `Work` was removed from its parent, and since callbacks don't hold references, I was assuming (when I wrote that bit) that parents would be the only parties holding references to children. In fact it seems like we keep quite a lot of auxiliary references to work elements to track their execution sequence, so you're right: I should make the parent pointer a `weak_ptr` not a ref; I can't assume anything about its lifetime. Will fix this.


This code is useful... it probably should be in a separate method though that you call in a `before_action`.  Something like `must_be_current_user!`.  You could also add it to `ApplicationController` or a module for reuse throughout your application.


添田です。
require_once良いですね！これ重要なので、是非覚えておいてください。
一度ロードしたファイルは再度ロードしないための関数です。


#4994-ControllerWaterCoil-TemperatureAndHumidityRatio-control-active-when-it-shouldn't-be (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1965 of 1985 tests passed)

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


I originally did the PR for this quite a while ago, and at the time was just trying to do a quick fix for LinkedIn's weird behaviour of requiring the state parameter to have some value. Don't remember why I hard coded the string though, what a bad idea haha. Sorry to you @baloo32, @markbates and anyone else who might've run into this.


Solution verified by www.projecteuler.net.


I hope you noticed that this library has absolutely nothing whatsoever to do with Adobe Photoshop.


Ah, you're right! (I suspected I was missing something). I got confused by the old diff here.

`request_headers` is included by default inside `api_request`, but we forgot to remove this instance. Want to file a bug?


I created a new issue for this bug - #170 - there are more places than just mysqli


Why did you commit my code?


yes but you can minimize usage of swap by set swappiness value to 10 in init script, it only touch swap if ~90% memory is used


Good catch, thanks.


Solution verified by www.projecteuler.com.


I was thinking, that is should be as "natural", as possible, but I do not have strong preference.


if文の条件を満たしている場合はreturnして、このメソッドは終了するので、elseは書くだけ無駄ですし、ブロックが深くなるので消したほうが良いです。


yes, I can do this


Updated in 4fd07e28119858cb84e6e4d905fa725c2dc7c180.


If you really want to highlight the main problem Swindler solves, you could list a few examples of bugs that people who have tried other OS X window managers will be familiar with.


ここもif文の条件を反転させてエラーメッセージを出力して-1をreturnすれば、mapにputする処理はブロックで囲わなくてもよくなります。


Test comment


I don't really get how Swindler's state consistency helps me get events about windows before the WindowCreated happens.


presentation pdf


ipython notebook 자료


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9413](https://teamcity.mesosphere.io/viewLog.html?buildId=92796&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:07:39


대문 edit


I agree about manually entering the address; could be troublesome   ;)  I don't dislike it - just trying to understand it -  So let me see if I get this correct.   Example is, say Stan wants a new location, box123, this box has 8 row and 8 col, so he enters that.   Next when he wants to enter samples for this box, he clicks on it and the program draws out the 8x8 and allows him to pick from it?    that kinda cool - so in other words the col and row fields allows us to know what the dimensions are and be able to draw it - theryby still retaiing the flexibiility of further nesting and different size box?  whoo i think im starting to see it.


salesVlue <- スペルミス


添田です。
細かいですが、「年-月-日 時:分:秒」で出力してほしいです。
例：2015-12-31 12:54:44
dateの使い方としては間違っていません。


Hi @seliopou, I just came across this looking through the frenetic-lang folder. Have you heard about OCaml's private types? I just read about them the other day: http://caml.inria.fr/pub/docs/manual-ocaml/extn.html#sec221 With private types, one can destruct, but not construct values. They may come in handy here. I'm using them in the frenetic repository now: https://github.com/frenetic-lang/frenetic/blob/master/lib/Frenetic_Vlr.mli#L107-L110


This fixes the following error:
ERROR: MethodError: `convert` has no method matching convert(::Type{DICOM.DcmElt}, ::Array{UInt16,1})
This may have arisen from a call to the constructor DICOM.DcmElt(...),
since type constructors fall back to convert methods.


添田です。
良いですね！自分でこうした方が良いと思う形を色々取り入れてみてください。
自分で遊ぶと、より身に付くと思います。


* starting to like this more because whenever their is a dimesion define; the program can draw a grid for data entry


Ah, that seems like a better idea, I'll do that.


hm good point. it's also possible people don't call this folder 'shopstyle-node-common' locally too

perhaps we need a different approach here in general. e.g. something in the configs of the parent repo to say "hey also load configs from common (and potentially other repos too) please", e.g. an `inheritConfigs: ['shopstyle-node-common']` property which can be default included in the boilerplate

thoughts?


I made the change to x: 3 because that is how queenside castling works.  If it isn't at 3 then the rook and king land on the same spot.  This link https://en.wikipedia.org/wiki/Castling has pictures with queenside castling and you will see what I am talking about.


ここではFileクラスのインスタンスを変数に入れる必要がないです。
PrintWriterのインスタンスを作成することにしか使ってないですし。

    pw = new PrintWriter(new BufferedWriter(new FileWriter(new File(args, outFilename))));


Just curious.. why did you call this `direct`?  This can be any descriptive name you want like `find_by_entry`, etc


I hosted this locally. This looks better already on my mobile device. I run Android 2.2 Froyo.


Lines 62 and 63 should not be indented.  You need to indent starting at line 65 due to the `if` statement


添田です。
細かい事ですが、変数の中身が文字とわかっているのであれば「"」で括る必要はないですよ。
str_replace('I', 'い', $mozi);
でOKです。


@mbj just to be clear `restrict_with_error` adds a validation error to the object. `restrict_with_exception` actually throws an exception when the a dependency is deleted.


添田です。
file関数は存在を知りませんでした。状況によっては、これを利用するのも良いですね。
こうやって自分から色々試す姿勢は素晴らしいです。


This doesn't really tell me much. I would suggest using a string to indicate what the slug was before destruction.


    n == 3 && line.length() > 10
で同時に問えば良いかと。
でも、2点ほどマズいことが。

1. 「10桁を超える」は9桁以下なら良いのか？
2. 売上ファイルに書かれている金額が10桁を超えているかを問うているが、仕様は「合計金額が10桁を超えているか」なので誤り。


I would align the opening curly with the other values in the Hash.


br = new BufferedReader(new FileReader(new File(args, listFileName)));


So I accidentally built a zImage with the wrong config but got lucky and exposed a bug :)
If IPMI isn't available on the system (either because you're running on an FSP-machine or you messed up your config...) `platform->ipmi` isn't set. But here we unconditionally call `get_ipmi_bmc_mac` and we segfault pretty soon after that.
Easy fix by just wrapping that in an `if(ipmi_present)` or preferably `if(platform->ipmi)`, but otherwise everything looks good!


What id you used `address.attributes` to iterate over all the attributes? This might remove the need to use `address.public_send(name)` below.

The only problem might be that if there are more or less attributes than the ones defined here.


Paul is a bitch.


join in


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9414](https://teamcity.mesosphere.io/viewLog.html?buildId=92801&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:03:31


    String args

コマンドライン引数の最初の要素を受け取っているので複数形ではないはず。
※ argsはarguments の略。たぶん。


The repeated `sec.parent_program._moduleExtension = {}` was the old test's way of making sure that `sec.parent_program.getModuleExtension('StudentClassRegModuleInfo')` was updated when `options` was updated, so since there's now only one copy of the SCRMI floating around it's unnecessary.  The explicitly specifying things as decimals is for the same reason -- it turns out setting a string as the value of a `DecimalField` works just fine, but if you try to access that attribute on the same copy of the object, you'll get the string back instead of a decimal.  Since I'm now manipulating a single copy of the object rather than getting it from the DB again each time, this no longer worked -- the code expects a number.

In both cases, the old test was depending on some sketchy things and this is a cleaner way to do it, I think.  In real life, if a SCRMI changes in the middle of a request and we don't update things, that's fine.


GitHub - capital 'H'


変数名i, jは、主にループカウンターで慣例的に使われる変数名です。
ほかのことでは使用しないほうがいいです。

i, jだと何が入っているかもわかりづらいですし。


@afcaJordi I saw that you created test cases in advance to writing actual code - very good!
Could you try to reference the respective issues in your commits as well (sometimes you do but sometimes I cannot find the ref). thx!


It is not really of consequence to HASHKAT if random files get posted without license; sorry if it came off as overly critical. You don't really need to do anything about it, however just be careful in the future. This commit strips the license comment from config_static.h. The most up-to-date version of config_static.h in the repo does not have the license.


You're mixing git with GitHub, git is the software for collaboration, GitHub is a server that hosts git repos


hi,uuxia.could you give me your email or other contact 。
and my email is xiaxiaoli_2005@163.com.
i have something ask you,please call me!


Addresses #15, though the code could be cleaner.


これだと例外が発生したときにsalesをcloseできないです。


@trumpetchris you can do this in one shot: `Get-ApcManagementCredential 'biz.dfch.PS.Activiti.Client.Setting' -As PSCredential`


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9415](https://teamcity.mesosphere.io/viewLog.html?buildId=92802&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:04:03


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9416](https://teamcity.mesosphere.io/viewLog.html?buildId=92803&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:00:42


I'm more curious. I know the original code used `specify` here, but I'm curious why you didn't change it to `it`. Do you think specify is a better command to use when there is no description?


For some reason i can t commit the actual project to get hub.


Quick note: No git tutorial is complete without diagrams. Perhaps we could make our own or cite other tutorials for a deeper understanding?


1. 文の折り返しの起点がわからないです。例えば80文字を超えたら折り返すなど、一貫性があるべきです。
2. なるべくブロックは浅くなるようにコード書くと読みやすくなります
3. 変数名は、何が入っているのかわかるような名前をつけてください


Actually, no. It won't work that way. In this example they have one single field and one single value, and we've already discussed that we do not want to disassemble into array. If I pass a list of column names and a list of values, I will have to parse the string first, otherwise it will be like this:

INSERT INTO bi_temp_tables.devices("device_id, device_descr",effective, asserted) VALUES ('4, ''desc_with_literal''', '["2015-12-01 00:00:00","2016-01-01 00:00:00")', '["2015-12-14 00:00:00",infinity)')


My general preference is to use `{` .. `}` when chaining, I would probably write this as:

```ruby
expect { order.contents.add(shirt, 3) }
  .to change { order.adjustments.eligible.pluck(:amount) }
```


There's an extra space before the opening curly brace.


Also to note. Perhaps a useful tool for the _complete_ beginner (they've heard of the 'command line' but that's as far as their knowledge extends), I suggest taking a look at GitHub Desktop https://desktop.github.com/


YES! Or at least, that’s the thought...

my_rows and my_cols tell us how many rows/columns a location has. (feel free to change the names) This way we can make something that is easy to click on.s

parent_row and parent_col tell us where in the parent *we* are.

Similarly, samples will get location_id, location_row, location_col fields.

> On Dec 14, 2015, at 11:34 PM, tree101 <notifications@github.com> wrote:
>
> I agree about manually entering the address; could be troublesome ;) I don't dislike it - just trying to understand it - So let me see if I get this correct. Example is, say Stan wants a new location, box123, this box has 8 row and 8 col, so he enters that. Next when he wants to enter samples for this box, he clicks on it and the program draws out the 8x8 and allows him to pick from it? that kinda cool - so in other words the col and row fields allows us to know what the dimensions are and be able to draw it - theryby still retaiing the flexibiility of further nesting and different size box? whoo i think im starting to see it.
>
> —
> Reply to this email directly or view it on GitHub.
>


Thanks for the kind words!


The HK files stripped of #comments were just so we could see, without
fluff, what exactly needed to be the end product of our Web form.
On Dec 15, 2015 5:48 AM, "ludamad" <notifications@github.com> wrote:

> It is not really of consequence to HASHKAT if random files get posted
> without license; sorry if it came off as overly critical. You don't really
> need to do anything about it, however just be careful in the future. This
> commit strips the license comment from config_static.h. The most up-to-date
> version of config_static.h in the repo does not have the license.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JPaveg/HashKat-Mockup/commit/d6b28e582ac9af1af6f635345e98be80cf2f748e#commitcomment-14976944>
> .
>


The code in the context above uses `#update_attributes!`. I would suggest choosing one or the other for consistency.


Exactly. Grids are so pervasive in the lab, that it makes sense to just adopt them in the program directly. It’s like the difference between a linked-list and an array. Both have their places. Linked lists are ultimately more flexible, but arrays are more efficient.

> On Dec 14, 2015, at 11:37 PM, tree101 <notifications@github.com> wrote:
>
> 	• starting to like this more because whenever their is a dimesion define; the program can draw a grid for data entry
> —
> Reply to this email directly or view it on GitHub.
>


添田です。
処理はあってます！ただ、謎のsが入ってますね。


Not sure about how to download the private key in P12 format. I can only find how to download JSON file. Could you please clarify how to download the private key?


It just gives you the WindowCreated event before giving you the other events.


添田です。
細かいですが、文字の場合は、「'」か「"」で括ってあげてください。


Do we know if it is actually `false`, or is it `nil`? I would match on that specifically for now. If the implementation changes I would want for this spec to fail so that we would be forced to fix it to be precise.


１.定義ファイル「.lst」ファイルの支店コードや商品コードの判定が不完全です。

２.rcdファイル読み込みの処理はできていないです。
  予想される起こりえること。
    ・0000000Z.rcdというようなファイルがあると、エラーメッセージ無しで終了することが予想されます。
    ・「a.rcd」「b.rcd」「c.rcd」という売上ファイルの構成でも集計する可能性がある。（試してはいないけど）

３.集計ごとに出力していませんか？


添田です。
毎回Aをechoすると連結しているように見えますが、そうではなく１つの変数にAが30個連結された文字列が収まっている状況を作れるように調整してみてください。前後の課題はできているので、たぶんすぐできます。


添田です。
1000は数字なので、「'」で括る必要はありません。
$key = 1000;
で大丈夫ですよ。
処理はあってます。


In the view of the change made to rocc.scala does this mean we can have 4 separate accelerators within rocc and each addressed by a different custom opcode?


What do you think about having one commit that changes this value *and* replaces the `schema.rb` with `structure.sql` ? The other changes in this specific commit could come after, but changing the schema type could be isolated on it's own; reducing the diff of this commit.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9417](https://teamcity.mesosphere.io/viewLog.html?buildId=92808&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:03:52


TeamCity pull requests :: netty [Build 865](https://garage.netty.io/teamcity/viewLog.html?buildId=4076&buildTypeId=pulls_netty&guest=1) is now running


I would probably use `#any?` here.


index


I would remove the unless and change this to be:

```ruby
order.create_proposed_shipments if shipments_attrs.empty?
```


On Mac it's also too big, and I can actually see it grows:

![visual mov](https://cloud.githubusercontent.com/assets/30809/11802510/89ee1b5a-a2bf-11e5-83c0-dfb6f5f901c7.gif)


path can omit query, which will be represented in queryItems


You could use receive_messages here.


Awesome. :+1:


why use this as a class??


This was just me fixing my stupidity---for the second time (I pushed to master without first pulling, forgetting I had previously pushed another commit from a different machine). Anyway, all is fine. I hope!


How can we differ the tag name in hub.register() method?I want to send the notification to the individual user only.Kindly, help me as soon as possibel.....


- s/Genertors/Generators
- s/developpers/developers
- s/a lots of/lots of
- s/Thumbnail/thumbnails
- s/from/for


Ok, I moved the signal creation function to `esp.program.models` since it has a lot to do with `Program` and `ProgramModule`, and then put the calls to it in `esp.application.signals` and `esp.program.modules.signals` which are imported by their respective `.apps` files.


Thumbnail generators allows previewing different kinds of assets by generating thumbnails for them.


What happened here?


The spec compatibility does *not* depend on the JVM version. The jars that we build do. The versions that I bumped are those of the jars, not of the specification. In fact, the specification doesn't currently have a version number.


It seems like bigger issues include event-only shininess.


of Asset
file MIME type?


I think that it should be marked "inline" to allow multiple translation units to contain (identical) definitions.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9418](https://teamcity.mesosphere.io/viewLog.html?buildId=92809&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:09:44


should these lines be indented like the other sections?


While this may work you don't need to create a navigation bar or a bar button item. You could just create a regular `UIButton` and add it to the view.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9420](https://teamcity.mesosphere.io/viewLog.html?buildId=92827&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:39


Here you basically want to do what you also did in the other view controller when you shared data.


most cases


Trasient


example**s** of tr**a**nsient


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9419](https://teamcity.mesosphere.io/viewLog.html?buildId=92826&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 205 Build time: 00:03:53


whoops, fixing


you**r**


open source


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.953](http://teamcity.monogame.net/viewLog.html?buildId=12337&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Mac) Build time: 00:00:14


needs to mention the ``disable`` option as well


Yes, that is correct.


oops one last thing add a note in the CHANGELOG for 8.13

Thanks tons for making a compatible way to address this feature!


They are there for a reason :(.


好惨。。只能malloc free....


your file seems can't work,mine:
description=bigdesk - Live charts and statistics for Elasticsearch cluster.
version=master
site=true
name=bigdesk


Oh oops lol I'll add them back


Unfortunately, I have to change the default Gradle build, because it conflicts with the Gulp one (or, more likely, the one used by the Liferay Gulp tasks) :cry: