I think instead of using the object property, you can use `msg->response_body->data` directly. Regardless, there's some notes in the documentation about the response body maybe being `NULL` unless you call `soup_message_body_flatten()`.

https://developer.gnome.org/libsoup/stable/SoupMessage.html#SoupMessage-struct
https://developer.gnome.org/libsoup/stable/SoupMessageBody.html#SoupMessageBody-struct


You can use username instead of email so your players can use user instead email for login


I think this is true when the space available to the text is less than what is required, but not when the space available is more than what is required.

The skips which go into computing the two `allocations` (which is where the glue should end up) are `\gre@skip@nextbegindifference` (which really shouldn't be a skip) and `interwordspacetext...`.  The bar allocation could also pick up glue from the new bar spacing distances, but we're made those to not have any glue.  This leaves just these `interwordspacetext...` as the source of glue, but that will only actually get added when the initial allocation (the space which is naturally available) is insufficient.


It is magic :) I'll simplify what I'm doing and add a comment.


no extension


You should remove this directory from the repo too with `git rm .idea` because it already under version control.


wdyt about a compromise: both calls to backoffAPIRetry just pass 5. I might as well just hardcode that into the function, making it a const along with the other backoff parameters so that it can be easy to find to change later if need be.


Commenting to ensure we have a test for this conditional.


So we don't use assert. We use import static org.junit.Assert.assertEquals, assertTrue etc.
The reason is that these classes have a lot better error messages. Your assert will throw a general assert error, but won't really tell you what's wrong. By using assertEquals etc you get a much clearer error message.


Do you see last kmgs that show that crash was in ondemand code? If not, then its not the problem. To use this driver as is, you need to update cpufreq code and stats code. For wall timers. This is custom build gov. Not general for any source.


QuickFixIDD-Memos (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1563 of 1563 tests passed, 0 test warnings)

![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.71%25-green.svg)


All the previous functions are only used in `config.py` or nowhere. I think it will make sense to remove the ones are not used and to append a `_` in the name of the ones used to indicate that are private to this file. Or are you envision them to be used outside here?


There are explicit checks to prevent adding directories to the list of dependencies in FindCUDA/make2cmake.cmake.  If that check is failing, you need to figure out why and fix it there.  This is not the correct place for this.


ok


This would be called a TicTacToeResourceTest not just an application test. You'll have a test for each of your resources, etc.


regarding implementation 1, does moving into a separate suite reduce the inflation of visit and visitors?


```JS
else if (typeof config === 'string') {
  if (!process.browser) {
    throw new Error('Incorrect config, only object is supported in browser');
  } else {
    baseConfig = readConfigFile(config);
  }
} else {
  throw new Error('Incorrect config');
}


Here's an example of tests we write for simple resources
https://github.com/mainstreethub/task-api/blob/master/application/src/test/java/com/mainstreethub/task/resources/TaskResourceTest.java


Think this might actually be `<link rel="shortcut icon" href="../public/assets/ico/favicon.png">`
or even `<link rel="shortcut icon" href="/public/assets/ico/favicon.png">`

You'd need to see which works on this setup.


all the application/target stuff should not be here


Ok, I think we have a problem then... I'm now sure about the best way to deal with that... maybe making a `rubberbarnotes` for cases where `interwordspacetext...` is not added?


This stuff should all be not checked in


This should also not be checked in


```JS
else if (typeof config === 'string') {
  if (!process.browser) {
    throw new Error('Incorrect spec, only URL or object are supported in browser');
  } else {
    specRef = spec;
    if (!path.isAbsolute(specRef)) {
      specRef = path.resolve(process.cwd(), specRef);
    }
  }
} else {
  throw new Error('Incorrect spec');
}
```


alphabetize etc


By removing this, you can no longer use the "Compile" command for a CUDA file in VS.  There are also side effects for makefiles which I haven't been able to work around.  This line should continue to stay.  We might need additional information about what is failing and why before removing this.


Sorrysorry, "always compile before commit" missades.


@Altaf-Mahdi Perfect ! your suggestion fixed the issue :) thanks a lot


link in commit message should read:
https://github.com/susynt/SusyNtuple/commit/e290eb288abe1037e61b2e88fe6ad226de6b28d3


it is sufficient to close


or a trick like `\gre@rubberbarnotes=\glueexpr(\interwordspacetext-\dimexpr(\interwordspacetext))`?


The point of making the rule dependent on the script was to force a recompilation if the parameters to the script change.  Basically we need to update the output file if the input changes.  The script shouldn't change unless you are changing stuff such as the flags or other toolkit paths.

Please revert this line.


Oh how I wish rustc was faster at compiling winapi.


I think we could move the `page.has_selector?` into the private method too. Then this would be something like:
```ruby
assert label_is_present?('image_url')
```


This is worse.


If the ref is being enforced to use a value from the received keyfile below, then I think this line is unnecessary.


Inter-line alignment isn't part of any standard I'm aware of. Certainly not PSR-2.


I think we can just re-use 1ull<<57 ... which are all server-side jewel features.


What is the minimal nodejs required for J5? So far, it worked fined with node.js 0.10.x, but following this change this will no longer be the case as far as I can see - unless `Object.assign` is provided through some es6 shim.


remove: webpack should not resolve module differently than node does


That PR is about to land. It'd be a good idea to restructure this change in terms of that work to avoid introducing yet another mechanism.


No standard requires a blank line before the `namespace` statement.


Enabling CONFIG_BUILD_ARM64_APPENDED_DTB_IMAGE shouldn't prevent your device from booting.

Like I mentioned, the patches I linked you would prevent your phone from booting.

I've made several clean builds using this kernel, boots fine for me.


Please use nodes_copy (no capital letter) to be consistent with our naming policy. Or even better mesh_nodes.  The code looks good, but I didn't test it.


can you change the message? since getent hosts is not directly checking dns.


From https://build.hmdc.harvard.edu:8443/job/docs.zeligproject.org/257/consoleFull

```
Updating R packages
Loading required package: graph
Error in file(filename, "r", encoding = encoding) :
  https:// URLs are not supported
Calls: source -> file
In addition: Warning message:
In library(package, lib.loc = lib.loc, character.only = TRUE, logical.return = TRUE,  :
  there is no package called ‘graph’
Execution halted
```

http://bioconductor.org/packages/release/bioc/html/graph.html says "try http:// if https:// URLs are not supported".


What if we made CourseType a first-class thing (doesn't need to have database backing for now), so course.type has all the methods we're using here, but Type also provides the same methods?


You declare this but never use it.... (or maybe I missed where it was used).


Is there a risk with this that the we auto-select a style that does not exist?  like if the clock is wrong?  Should we have a list of actually defined styles and verify that a good one is selected?


You don't have to split up your read and write client. We did that for aviary because it was big, you can just have a TicTacToeClient.


Whatever "standard" you're purportedly following I'd surmise they're probably your own.


This should be called TicTacToeClientBuilder


And similarly here verify the value selected.


add
```HTML
<script src="node_components/swagger-diff/dist/swagger-diff.min.js"></script>
```


megjegyzest adhat a kollega hozza a modositashoz


@koddsson package.json needs babel-preset-es2015 as well


Ok.


@Altaf-Mahdi ,

Snap camera works 100% perfect but google camera 3 or 2 failing to work properly

Same google camera even though it shows scattered image but takes nice pics.

Anything you like to suggest for me here ? thanks a lot


Looks great for a getting started for newbies without too much background!
I guess what is missing is something like "hello world" for Lago, a first training task for any future lago developer, that is not related specifically to writing tests.

Also, maybe adding links to the other docs as 'next step', like trying out ovirt-system-tests or setting up Lago dev env...

But its a great start and I think any new Lago developer can start using it to start closing the gaps,
And I'm sure this will be a live document that we'll continue to improve and add too (probably even the new Lago developer will contribute to it :)


If `interwordspacetext...` is not used anywhere else (which I think is the case) then perhaps make it a dimension and then have a `rubberaroundbar` which is always added to the two `\gre@hskip` steps in spacing the bar syllables.  This rubber would simply have a 0 base and some reasonable stretch and shrink.  The `allocation`s and `requirement`s could then be dimensions all the way through (though we wouldn't necessarily need to make that change right this minute.


After note: we reset it to 65536 or 0?


Yes it is ok for you my friend but I need to import more changes before....:)


@gregvonkuster can we depend on perl_5_18? Meme is using some unicode feature in perl and it is saver to depend on our perl version. For example my system perl version was too old.


omg life is so much better with lambdas


@tinapotter @mikeflowerdew I've updated the Tau class to include the 3 WP flags as well as several truth matching quantities. I have yet to test out the ConeTruthLabelID and PartonTruthLabelID flags for when the tau is matched to a jet as the test samples I have at the moment (ttbar and stop decay) don't have any of them.

Let me know if the variables being stored are OK. I've implemented the vanilla [[https://svnweb.cern.ch/trac/atlasoff/browser/PhysicsAnalysis/SUSYPhys/SUSYTools/tags/SUSYTools-00-07-41/Root/SUSYObjDef_xAOD.cxx#L1843][baseline]] and [[https://svnweb.cern.ch/trac/atlasoff/browser/PhysicsAnalysis/SUSYPhys/SUSYTools/tags/SUSYTools-00-07-41/Root/SUSYObjDef_xAOD.cxx#L2115][signal] criteria in TauSelector as found in SUSYTools.


To confirm, is your following branch working for the calendar on php7:
https://github.com/practiceprovider/openemr/commits/php7-deprecated-methods
(if so, then I'll see what the differences are between that and the current openemr codebase to try to pinpoint the issue)


It was able to find the firmware files at /lib/firmware, but we can definitely move to /system/etc/firmware if that is the right path. Just saw that we don't actually have /system/etc/firmware in our image, so I'm expecting the fallback to be /lib/firmware still, which would explain why it worked fine.


Jak to bym wydzielił do zewnętrznych plików wszystkie RWD, np. style-1024.css, style-portrait-319.css


Crazy question - but why are we putting this at the end of the body instead of in the head?  And is there every the possibility of a race condition where this script does not get loaded because the browser or node.js or whatever is already done parsing the document?


This is out of order. We normall do
parent
scm
properties
build
dependencies
modules

Won't affect your pom working, but it makes it a lot easier to read, and get in and edit fast if we adhere to a standard format. See here
https://github.com/mainstreethub/task-api/blob/master/pom.xml


Przed klamrą piszemy spację :) Kod jest wtedy czytelniejszy.


I would say empty.


Ok, let's do that for 4.1. Do you think you could do it?


Any update on this by chance?


Moim zdaniem, nazwy projektów powinny być w konwencji `snake-case`.


"last 2 version" is that normal ?


Style: Prefer guard for early returns?


那个后两位bit位AND Binary 3 结果的挺屌的 哪看来的？


yesss! : )


yes :)


It's probably not necessary.  A Sink is light weight, so the copy cost is low.  But it seems like most users, other than Logs, hold a Sink by pointer or reference.  So I thought making the copy constructor explicit might be a good idea.

My usual heuristic is to mark a single argument constructor `explicit` unless I have a good reason not to.  If you prefer that it not be `explicit` I'm fine with changing it.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1563 of 1563 tests passed, 0 test warnings)

![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.71%25-green.svg)


Style: Semicolons are so Objective-C.


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


Ok, will continue my quest searching for a solution. Reverting this breaks the 0% setting anyway.


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


@MauricioMurga :+1: Necesitamos version bump a 2.0.3 con este cambio ya que tenemos un bug en 2.0.2 por que no encuentra la clase en Lang uppercase. `Class 'LANG' not found`


A single controller test shouldn't test more than a single request.


We should also assert that `images[2]` is not present.


This is a great test, but it is not an assertion.


Done.


@waylan The tests passed because I added `.encode('utf8')` - so I suspect this full thing can just be replaced with `pathname2url(path.encode('utf8'))` in the utils file I changed.


I'm confused that this is not already implemented in twisted. I think I would move it down the file, if you read the file up-down I think it makes sense to find first `Provider`.


Why the increase in the wait time? Can you do `utils.wait.giveTime ms, ->` just around the operation that is timing out?


Yes, Richard also mentioned this one. I will fix it.


lgtm


Shouldn't this be `/corp/MarketOrders.xml.aspx`?


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#IsgixG5FRciDV8HaJI_ANA


this hunk goes in an earlier commit i'm guessing?


I'm finishing my take-home mid term at the moment (it has taken a bit longer than I expected because I've had to go back and review material in order to answer the questions), but should be able to do something in an hour or so.  If you've got time right now, then you might want to make the changes which I'll then review (and correct if necessary) later on.


Does the email sending still work locally with these changes?


I didn't know `explicit` could apply to copy constructors, what effect does that have


Yeah.  We're reusing the constructor from a `Sink` in order to get all of the streams initialized properly.


Might be good to note that this is a "Closed Cursor"


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `coalib/__init__.py`, line 5.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/__init__.py
+++ b/coalib/__init__.py
@@ -3,6 +3,7 @@

 __version__ = Constants.VERSION

+
 def assert_supported_version():  # pragma: no cover
     if not sys.version_info > (3, 2):
         print("coala supports only python 3.3 or later.")
```


And to note here that this is where the cursor is actually opened.


Merging. Will deal with the rest of the FormatInfo stuff in another PR, probably after moving to DNX.


++ looks good


You may want to consider adding the following to your `.rubocop.yml` file:

```
Style/SignalException:
  EnforcedStyle: only_raise
```

This is one of the rules that we generally don't agree with the style guide on.


No it doesn't. I thought when we deploy it onto Heroku we no longer need this configuration so I just commented them out.


you mean in the diagram?  Or in glimpse messages?


Shouldn't it be the same thing (a table of constant pointers, vs. a contant table of pointers)? I guess that you're doing the change because clang doesn't like it. LGTM.


Any idea where to look next ? Where do the slider changes from LightsController.js/DashboardController.js end up ?


It probably can.  Which class in beast are you referring to?


Hardcoded `basedir`.


Hola estoy muy agradecido por el trabajo que le pones a Rosadin, desde que lo conocí se convirtió en mi reproductor iptv favorito, pero tengo un problema al cargar el canal cuatro de argentina
http://streaming.wirenetserver.com.ar:1935/canal4/live_1/livestream.m3u8
Se queda cargando eternamente y en otros reproductores como media player classic si funciona, solo he podido hacerlo funcionar con su enlace en rtmp (rtmp://streaming.wirenetserver.com.ar/canal4/live_1) pero quiero poder verlo usando ambos protocolos, ojalá lo puedas solucionar

Saludos


Shouldn't this be just part of the NODaemonExcetption, and not need to be passed into each call.


So just to make sure I understand-  in order to send no http body at all, I would leave both params and body args to `undefined`, right?  I think it would be worth it to add a quick blurb about that here.

We should also clarify that:

• GET requests will still encode `params`  in the query string, and this input will be ignored for non-GET requests (it's not just post and puts-- [we explicitly check for GET](https://github.com/mikermcneil/machinepack-http/blob/847b30282ee022b2cf09ab3044c49fc79788f827/machines/send-http-request.js#L154)).  DELETE requests [technically allow body to be provided](https://www.w3.org/Protocols/rfc2616/rfc2616.html), as do PATCH, etc.
• if `formData` is set, then the `body` input [will be ignored](https://github.com/mikermcneil/machinepack-http/blob/847b30282ee022b2cf09ab3044c49fc79788f827/machines/send-http-request.js#L170) and params will still be used.
+ the provided `body` will be JSON-encoded-- i.e. so maybe instead of "Body of the request (for PUT and POST)", it should be "Body of the request (will be JSON-encoded)."


why not static_cast here?


Yes Richard mentioned that I shouldn't change to `fail` too. But where does the rubocop.yml locate?


In the update diagram at line 58


`WrappedSink`


Thanks.  Will fix.


Heroku uses the production configuration so the settings here (development.rb) shouldn't matter. You will still want emails to be able to work locally, however.


Thanks, Piotrek! :-)


Oh.  Duh.  I wasn't paying close enough attention to my own work.  I'll try removing this class.  Excellent point.


Rows should stay selected if they push another view controller, and deselect on return.


You can add `.rubocop.yml` to the root directory of your repository.


Ok. Searched on the code, indeed this covers all the exceptions.


Thanks a lot! I will look over it tonight. It's a shame browsers don't optimize either. I suppose it's very difficult with such a dynamic language because you can't really analyze what you're actually getting very well.


Note that this means that if "wet" machine instances are passed in to Machine.build(), they'll now just be returned as-is, with no cloning/modifications/etc.


related to #10


For logging, don't always use log.init.

Log.init is for initialization stuff
Log.info is for random info
Log.debug is for debug that isn't important, as it will only show up if DEBUG_MODE is enabled
Log.warn is for warnings
Log.error and Log.fatal are for important errors that will still show up even if LOGGING_ENABLED is false.

I may put this information on the wiki page for the program on github later.


Are we changing the name? I thought we we're saying Add-Art was the platform and Addendum was the main program?


Why does this not support nanosecond level precision?


Exactly, I will move the configuration of sendgrid to production mode and bring this part back


Cool - we will change decimal scalars definitions to use that.


muito bem tomás, estás contratado


The function looks like this:
```
function hasGroupLock(pv,plotGroup) {
    return (plotGroup &&
    plotGroup.plotGroupId &&
    plotGroup.lockRelated &&
    pv && pv.plotGroupId===plotGroup.plotGroupId);
}
```
Maybe I am missing something but I don't see what lodash.get would do for me. It there a way to use it that I don't understand.


Ok


Git itself doesn't allow empty directories to be committed, [so empty files are a way out](http://stackoverflow.com/a/21422128/1670308) are a way around this


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


Even having made the design decisions that resulted in your having `Q1`, ..., `Q5`, you still have no reason to have these five fields `question_was_cheated_on1`, ..., `question_was_cheated_on5`. If you look at how you use them, they only have very brief lifetimes that local variables could handle just fine. (Also, you wouldn't even need five local variables if you were willing to alternate getting and setting rather than doing all the getting first and then all the setting.)


@petemoore, this broke tests... not sure how we landed it :) he he

Anyways, it's a quick fix just change to: `func (e engineProvider) PayloadSchema() runtime.CompositeSchema`
That's it...

@petemoore I just want to make sure you understand why I'm placing `PayloadSchema()` on the `Engine` and not the `EngineProvider`... By putting it on `Engine` instead of `EngineProvider` the configuration object given to `EngineProvider.NewEngine` can be used to customize the `PayloadSchema`...

I don't think that'll be a common case... But it'll be really nice to support it as it'll allow us to restrict features at configuration level.. and reflect such restrictions in the payload schema...

Obviously, this is exactly the same for plugin and pluginprovider... where it might be relevant in one or two plugins... It just allows us a but more customization and power in plugins and engines.


可考虑直接连接原作者博客内容。


@daurnimator Saw that.  Thanks for doing it, I totally forgot.


@tchalvak Why would you commit this directly to master when there are outstanding PRs and bugs that are blocking the merge?


This is great, but the xml looks a bit off. Did you run xar:format? The \r\n's are getting HTML-escaped and right angle brackets are appearing as `>` instead of `>`.


Isn't the `mIsCheater` redundant now that the `Question`s are keeping track of whether they were cheated on?


Looks great!


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


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 116](https://teamcity.labkey.org/viewLog.html?buildId=305022&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:52


dd


If the `savedInstanceState` tells you that the user asked to display the answer before rotating the device, wouldn't it be good to redisplay the answer? I can't think of any reason why I'd want the answer to disappear upon rotation.


You're right that the CI should be building release.


@rhyolight Subutai's response in your link is probably the best explanation.


Wouldn't it be good to also redisplay the answer, since it was showing prior to the device rotation?


Please omit `return` :scissors:


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


dsfd


Suggest putting a note here that the script will retry forever if it can't get a connection to keystone.


Having retrieved the value from the `savedInstanceState`, I don't see anywhere you put it back into `mAnswerIsShown`. That means if someone were to rotate the device a second time, the record of their cheating would be lost.


I think you were meant to use the "solve" function - you seem to have used a different function in the earlier version. This update does not appear to include the full code...


Why n-1 instead of n?


Oh, this is awesome, and what I wanted in this: https://github.com/cleanflight/cleanflight/issues/1268#issuecomment-186860339


That's the same case as the question whether adding two nullables should mean. There are several different possible answers (all self-consistent), and Julia decided to play it safe and not make a choice. Likewise, comparing nullables is currently not defined.

The basic question is whether comparing nullables should return `Bool` or `Nullable{Bool}`.

There's probably space for two different nullable types. Maybe we should introduce a type similar to Haskell's `Maybe` to Julia (with similar semantics), and make `Nullable` more comfortable to use.


fixed


I would consider it to be renamed as `defaultTimestamp`, but what could work better is `defaultProcessMessage(ProcessContext): ProcessMessage` and
`defaultStageMessage(ProcessContext): StageMessage`
Also ProcessContext could be defined as implicit, for reducing the number of duplication even more.

Test methods would look something like:
`val expectedMessage = defaultProcessMessage.copy(state = Processing.identifier)`


There is no reason to include explicit initializers of `false` because that's the default initialization for `boolean`s. Leaving that out, you should find it much easier to make the array whatever the correct length is to match the `mQuestionBank`, rather than hardcoding in the length of 5.


That does sound strange.


woops ignore this please :P


I think we use `/\A ... \z/` better than /^ ... $/. :dango:


It seems to me like the useful life of `mIsCheater` has ended. Now that you've got `mCheated`, it should be easy to get rid of `mIsCheater`.


small port numbers will get screwed up without quotes, so it's considered best practice according to some docker docs, which I am too lazy to link to


I actually tried this first and on my ubuntu 14.04 system it failed. I actually needed to install `erlang-dev` in addition to `erlang-parsetools`


pages


Ah, you, well, you see, I did this:

git checkout master
git hack hack
test test test
git branch fewer_redundant_requires
git push <--- whoops


@Youwotma

Can 'viewing live splash window' work on Mac from docker?


Just FYI, the "return None" isn't strictly necessary, it's implied if no return is specified. But I actually like explicit return statements.


Check this change, now falcon and titan uses the same kernel, read the description.

https://github.com/MaxiCM/android_kernel_motorola_msm8226/commit/79b7d5665f3340f2bb9d49a2f2f0467cf3384283

Greetings.


Isn't there a closing bracket missing at the end?


Suppose `mCheated[mCurrentIndex]` was already `true` from some earlier occasion of cheating on this question. If the user goes back into the `CheatActivity` and this time returns without looking at the answer, should `mCheated[mCurrentIndex]` be changed to `false`? That's what your code does.


Always make sure you use `var`, even in for loops.  This will still create a global variable.


FixDatasetFiles (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2119 of 2119 tests passed, 0 test warnings)

![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)


This seems to have an extra ')' at the end...?


For these, you could also consider using something like `document.querySelector(".album-view-title")`.  This will return the first matched, which is what you're also doing, except it excludes the need for the additional look up via index.


This is all new to me.  Its taking me hours just to figure out github!


Это не решает проблемы с обновлениями. Ни одни данные не подтягиваются.
vis.states[data.oid + '.alias.val'];
Не работает эта строчка в виз, только в edit.
Дело в том что в качестве oid я выбираю не само значение а ветку. А если в oid выбрать значение то оно обновляется нормально.


so types won't let you have `Int` here but you might be able to use `Num` ... although it might be overkill just to avoid the `NominalDiffTime` reference ...

`todayPlus :: (MonadIO m, Num a) => a -> m Day`


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


Ok so would it cause problems if `==` was implemented in a similar way to `isequal`?


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


then you could remove this type here so `NominalDiffTime` would completely be gone ... also probably overkill


SZVAV-Fan-Control (rraustad) - x86_64-MacOS-10.9-clang: OK (2114 of 2114 tests passed, 3 test warnings)

 * 3 tests had: AUD diffs.

![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)


Awesome, thanks! :heart_eyes:


Ok, thanks @tabalinas for your attention.
:thumbsup:


FixDatasetFiles (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1563 of 1563 tests passed, 0 test warnings)

![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.71%25-green.svg)


我向上帝发誓 我没有删 小心下次见到你我踢你的驴屁股 到时候一杯热奶茶也解救不了你

On Tue, Feb 23, 2016 at 2:09 AM, CHANG LI <notifications@github.com> wrote:

> 评论就评论了，还删啥。。。 我只是突然发现可以一次动两格就那么写了
> kick your butt
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CHANGLI0914/LeetCode_G-C/commit/cddd2dd8330f91cf88036637f49b66d3ead7adc4#commitcomment-16247897>
> .
>


You ought not have needed to change these from `private` to `public`. The book's design of accessing the extras through static methods seemed wise to me.


this function is provided my the user of the list and he will implement it, as we dont know what he wants to match in the data. Please remove this function.


Moroja?


can you add `:string` to the end of this line. I'll explain why on our next call.
```
remove_column :chess_pieces, :name, :string
```
Otherwise :+1:


What's happening here? does this fill as blue or some value that depends on the colourscale?


The `EXTRA_INDEX` is the first place in your code that a design decision crops up that I call into question on line 168 of QuizActivity.java.


ie what happens to the points when no overlay is defined?


We lost the header and footer


eflag 값의 위치는 다음 링크에 있으며 https://github.com/naver/arcus-java-client/pull/59/files#diff-4f807ffd8440527bbda08918420be34bR29
이 부분에서 할당하게 됩니다. https://github.com/naver/arcus-java-client/pull/59/files#diff-48ddf9e9b8ad8b5a0ef24bebc01308b2R161

기존에 eflag 가 list/set도 가질 수 있다는 가정하에 두었던 것으로 판단됩니다.
CollectionGet 있어도 문제가 없을 것 같고 추후에 절대 들어가지 않는다는 확신이 없어서 수정하지 않았습니다.
CollectionGet 의 eflag 를 BTreeGet 으로 이동하게 될 경우,
```collectionGet.getElementFlag()``` 가 아니라 ```((BTreeGet)collection).getElementFlag()``` 가 되어야 하는데.
전자의 호출 형태가 더 좋은 것 같습니다.

그리고 Comments 하신 PR이 최신 PR의 File Changes 가 아닌 것 같습니다.


yeah so I had the user page return both the user and session so that
session could be retrieved and saved in the trials table by each
task ( [0][1] would give you the session number)

On Monday, February 22, 2016, Christian Battista <notifications@github.com>
wrote:

> fixing busted unit test, @boomsbloom any idea on why this change is
> necessary?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/StanfordCosyne/math-intervention-api/commit/1adeb4e96e35b0aef708ec4caeaf4b6c065dea5c
>


You've gone to quite a bit of effort to get this index here, including tightening the coupling between the two activities. And what for? Couldn't you just have used `mCurrentIndex`?


keywords is don't have


in the future I'll run unit tests after changes like this (sorry for the
time sink in fixing that)

On Monday, February 22, 2016, Jonathan Nicholas <jdmnichol@gmail.com> wrote:

> yeah so I had the user page return both the user and session so that
> session could be retrieved and saved in the trials table by each
> task ( [0][1] would give you the session number)
>
> On Monday, February 22, 2016, Christian Battista <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
>> fixing busted unit test, @boomsbloom any idea on why this change is
>> necessary?
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/StanfordCosyne/math-intervention-api/commit/1adeb4e96e35b0aef708ec4caeaf4b6c065dea5c
>>
>


okeydokey - remember to run the tests before you push changes...

On Mon, Feb 22, 2016 at 6:30 PM, Jonathan Nicholas <notifications@github.com
> wrote:

> yeah so I had the user page return both the user and session so that
> session could be retrieved and saved in the trials table by each
> task ( [0][1] would give you the session number)
>
> On Monday, February 22, 2016, Christian Battista <notifications@github.com
> >
> wrote:
>
> > fixing busted unit test, @boomsbloom any idea on why this change is
> > necessary?
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/StanfordCosyne/math-intervention-api/commit/1adeb4e96e35b0aef708ec4caeaf4b6c065dea5c
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/StanfordCosyne/math-intervention-api/commit/1adeb4e96e35b0aef708ec4caeaf4b6c065dea5c#commitcomment-16257838>
> .
>


Dear @liewegas, glad you are back:-).   Well, I don't think we need to do the above change as you suggested here.  If you take a full look at the whole function again, you'll find the iterator bp is used everywhere within this function. Besides, the above change better matches the way bp is used elsewhere. But ping you if you insist:-)


lolyup

On Mon, Feb 22, 2016 at 6:31 PM, Jonathan Nicholas <notifications@github.com
> wrote:

> in the future I'll run unit tests after changes like this (sorry for the
> time sink in fixing that)
>
> On Monday, February 22, 2016, Jonathan Nicholas <jdmnichol@gmail.com>
> wrote:
>
> > yeah so I had the user page return both the user and session so that
> > session could be retrieved and saved in the trials table by each
> > task ( [0][1] would give you the session number)
> >
> > On Monday, February 22, 2016, Christian Battista <
> notifications@github.com
> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
> >
> >> fixing busted unit test, @boomsbloom any idea on why this change is
> >> necessary?
> >>
> >> ---
> >> Reply to this email directly or view it on GitHub:
> >>
> >>
> https://github.com/StanfordCosyne/math-intervention-api/commit/1adeb4e96e35b0aef708ec4caeaf4b6c065dea5c
> >>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/StanfordCosyne/math-intervention-api/commit/1adeb4e96e35b0aef708ec4caeaf4b6c065dea5c#commitcomment-16257849>
> .
>


SZVAV-Fan-Control (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1006 of 1006 tests passed, 0 test warnings)

![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-29.09%25-green.svg)


Is there some room for factoring common code out from the `onClick` method and here?


Yes, I know how `var` works now, thank you. It was a typo which would have been caught be a real syntax parser. ;-) Instead of talking about typos how about you help me learn how to configure the environment to automatically catch these types of issues? Is `'use strict'` enough or are there other things you do in production code? Thanks!


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


Not all of the changes to this file are relevant to the switch to browserify and should have gone in a different commit.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 114](https://teamcity.labkey.org/viewLog.html?buildId=305024&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:04:29


A test case ...


It looks like you display the answer independent of the value of `mAnswerRevealed` that you retrieved from the `savedInstanceState`. So, if someone goes into the `CheatActivity`, doesn't click the button to show the answer, but rotates the device, they get to see the answer anyhow.


Maybe we can call a method just to define a default value for Execution Type


Should this still be here? Doesn't look like legitimate todo code.


Calling a method to put a default value


All of this looks questionable.  Should it be here?


Deleting `checkAssign` in the other two parts of the file and putting them here was what fixed #14.


Good call. Since this issue is pre-existing I've created a bug ticket for it here: https://trello.com/c/HefLlVPp/116-filter-out-inactive-contacts-from-chart-results


So, there's not a way to configure the _environment_ to throw errors.  That's one of the pitfalls of JS is that if the interpreter hits invalid lines, it's just going to run them.

Strict mode would have thrown an error in this scenario, but I think what you're looking for is a linter of some sort.

What text editor are you using?


Please forgive my completely not advanced math related file.. :)


3.4.7 was pulled as per http://comments.gmane.org/gmane.comp.java.zookeeper.devel/31293.  I'm pushing up 3.4.8 images now


Thanks, I appreciate it. I'm lagging behind this week. I've been fighting the flu or something since Thursday.


This seems to set the `mQuestionsCheatedOnBank[index]` to `false` if the user didn't cheat on this question on this particular occasion, even if they cheated on it previously. Shouldn't it be left `true` once it is `true`?


Markers have images. Those images are requested from a server, that that server must support cors.


change


It's a good point but like you're saying, I think it's a little _too_ generalized (hi Haskell).


Given that I wanted `==` not to be defined, it would certainly harm that goal. Eric's comment above really hits the nail on the head: there is no uniformly correct behavior for this function. Very different interpretations of what a Nullable ought to be are possible; we tried to provide the minimal core that all interpretations share. Once you define `==` to match `isequal`, you've removed our ability to support the other interpretations.


TODO: distinguish between sync. and async. replication?


ios::sync_with_stdio(false);
   Test 1: TEST OK [0.000 secs, 4196 KB]
   Test 2: TEST OK [0.000 secs, 4196 KB]
   Test 3: TEST OK [0.000 secs, 4196 KB]
   Test 4: TEST OK [0.000 secs, 4196 KB]
   Test 5: TEST OK [0.000 secs, 4316 KB]
   Test 6: TEST OK [0.000 secs, 4324 KB]
   Test 7: TEST OK [0.022 secs, 4328 KB]
   Test 8: TEST OK [0.054 secs, 4328 KB]
   Test 9: TEST OK [0.086 secs, 4328 KB]
   Test 10: TEST OK [0.086 secs, 4328 KB]


Use case: https://api.apiembed.com/?source=https://github.com/NarroApp/docs/raw/master/source/base.json&targets=all


I traced through this commit line by line and these last two lines

```yaml
- match: '(?=\S)'
  pop: true
```

Are what are triggering my Sublime Text to hang forever/crash in #200

I notice these are all over the place (lookahead for any character that is not a space) What exactly is this supposed to be doing here?


No,  this Makefile isn't a standard one.
It misspelled uses 'DSTROOT'  instead of  'DESTDIR'  or 'prefix'


@mashpie no worries :)


SZVAV-Fan-Control (rraustad) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


SZVAV-Fan-Control (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2152 of 2152 tests passed, 6 test warnings)

 * 6 tests had: AUD diffs.

![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)


@tchalvak We should not require any libs in our tests unless we're testing that lib specifically (which we [almost] never do). Including in the test itself can mask failures if base.inc.php doesn't already include it.


I agreed with you that aneuploidy is not the right word here as it refers to the overall number of chromosomes in the sample, not the number of alleles required to determine a genotype.. how about:
"a cache of the PL index to the (list of) alleles it represents for any ploidy (the map key). So, for example the beginning of the entry 3 is {1,1,1}, {1,1,2}, {1,1,3}"  (right?)


Could you link an example where a normal marker would work? I've been trying this morning, but the [markers don't get rendered](https://cloud.githubusercontent.com/assets/708395/13240049/3da87c3a-da33-11e5-814e-8b7d929ef2d1.png). My [marker is customised](https://cloud.githubusercontent.com/assets/708395/13240034/198ac20e-da33-11e5-9820-9697241c514e.png) and doesn't have images. (Overlay too, but I suspect that's a separate issue.)


Well it worked here.

```root [ dmd2 ] ! dpkg -L mxml
/.
/usr
/usr/share
/usr/share/man
/usr/share/man/man3
/usr/share/man/man3/mxml.3
/usr/share/man/man1
/usr/share/man/man1/mxmldoc.1
/usr/share/doc
/usr/share/doc/mxml
/usr/share/doc/mxml/D.gif
/usr/share/doc/mxml/mxmldoc.xsd
/usr/share/doc/mxml/1.gif
/usr/share/doc/mxml/3.gif
/usr/share/doc/mxml/4.gif
/usr/share/doc/mxml/COPYING
/usr/share/doc/mxml/B.gif
/usr/share/doc/mxml/0.gif
/usr/share/doc/mxml/2.gif
/usr/share/doc/mxml/CHANGES
/usr/share/doc/mxml/README
/usr/share/doc/mxml/C.gif
/usr/share/doc/mxml/A.gif
/usr/share/doc/mxml/mxml.html
/usr/lib
/usr/lib/pkgconfig
/usr/lib/pkgconfig/mxml.pc
/usr/lib/libmxml.so.1.5
/usr/bin
/usr/bin/mxmldoc
/usr/include
/usr/include/mxml.h
/usr/lib/libmxml.so
/usr/lib/libmxml.so.1```


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


I don't buy your argument that luring the cheater into a false sense of security is a good thing.


Ah yeah it should be n, but JS let me get away with it because arrays sizes are dynamic..


This is fixed in a commit I haven't pushed yet because it involved other changes.


ios::sync_with_stdio(true);
   Test 1: TEST OK [0.000 secs, 4204 KB]
   Test 2: TEST OK [0.000 secs, 4204 KB]
   Test 3: TEST OK [0.000 secs, 4204 KB]
   Test 4: TEST OK [0.000 secs, 4204 KB]
   Test 5: TEST OK [0.000 secs, 4204 KB]
   Test 6: TEST OK [0.011 secs, 4204 KB]
   Test 7: TEST OK [0.032 secs, 4204 KB]
   Test 8: TEST OK [0.054 secs, 4204 KB]
   Test 9: TEST OK [0.097 secs, 4204 KB]
   Test 10: TEST OK [0.097 secs, 4204 KB]


these links don't seem to be used


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


Thanks, this is something I was meaning to do. I think the paths to the Handlebars templates also need to be added to this though.


Thanks a lot. It worked.


"alleles of general ploidy"?


@danski I can successfully pull down profile information as people login


Shannon had me using Brackets which is cool since it helps when I'm messing with CSS (and yes, I know I can do that in the dev tools too) but in the past I used Sublime Text 3 and iTerm. One of the big things I'd like to go over during our first meeting is your dev env recommendations/setup/etc. Right now I feel like I'm programming with oven mitts on.


draw_sprite already rounds inputs i think. it could be the case that the default behavior could be to floor it (floor it? flooring it!). so yeah idk just my pennies


test


@btshimizukza コメント

- [ ] この２行（CSRF対策及びセッションの利用に関する説明）も不要なのでは？

---

削除しました by btsatoutkzy


I was just having fun with hello-csv. And of course, I will follow the company's style.


@danski added white list validation to keep application logins limited


When no overlay is defined, the default colour scale is used (always maps to the default blue colour no matter what value is passed to it).


This form reads more directly, because we can clearly see how the `true` response depends on the `current_user`.  It avoids a tiny bit of mental gymnastics to get past the `unless` and avoids the block structure around `render`.


> What files in autoload.php? That's an automatically generated file?

No `vendor/composer/autoload_classmap.php`, `vendor/composer/ClassLoader.php` and `vendor/composer/autoload_files.php`


 Yes, it is a bug that should be fixed.
The current test cases of DIFT does not cover the `len > len_pt_max` condition.
Please open up a issue for the bug fixing and the test cases improvement.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16768](https://teamcity.mesosphere.io/viewLog.html?buildId=141744&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:10


Given your eminently reasonable design decision to keep the record of whether each question was cheated upon within the corresponding instance of the `Question` class, you really don't have any reason to keep `mCheatLog` around as an ongoing field within the `QuizActivity`. It only gets used during two brief localized pieces of code while saving and restoring the data. You could use local variables in each of those two places, which would save your reader from having to read the whole class to understand how `mCheatLog` is being used.


Sorry, I made the mistake of commiting to master on the GitHub UI. This was fixed with https://github.com/fastlane/countdown/commit/f85e705c74941391bc618a12dfc48bfbae819468. Sorry about that


更新readme


left out the http:// on line 32....


Suppose the user had cheated on this question before, but now goes back into and out of the `CheatActivity` without taking another peak at the same answer. Does it make sense to be setting the flag back from `true` to `false`? That's what your `setUserCheated` is doing.


ampersands!


As we add more outputs it'll be nice to have some kind of factory pattern for resolving some kind of general transfomer implementation, but lets worry about that later.


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


Why is this declared here? Why not have calculatePLcache return a new `List<List<Integer>>` object (when it is called) and put into the Map?

Infact, I would suggest that it will not work correctly a written way since there is only one `List<List<Integer>>` that is being reused so all the map values are the same object.

Could you please add a test that uses two different ploidies at the same time (that is, initialize the map for two different ploidies, and then check to see if the values returned are correct)


Small nit, in some places brackets are on the same line, others not.  Should be consistent but I don't really care which way.


Thanks for this. Honestly, I didn't think that anybody would bother to look at this. This was mainly for my own amusement.


I'll add a test case to confirm.


Maybe cancel the process if there are no draw commands?
```
if (drawCommands.isEmpty()){
	return;
}
```
Just a suggestion.


Awesome.


comment that the recursion is bound to terminate in depth = ploidy.

this is much clearer, thanks.


Change these to `<strong>`, `<b>` is extremely deprecated.


Damn it, I though I had deleted both of those.  Anyway, I'm game with removing includes in tests as much as possible, sure, though I don't want to load up bootstrap to accomplish that.


See, you are doing all the hard stuff.


Fix escaping.


What is the exact syntax (from the user) required for the dice roll? I'll see if I can optimize this.


Vẫn còn lỗi đấy ạ.
Cái này kiểm $catid > 0, nhưng nó NULL từ bên trên mà.
Các hàm truyên biến vào func thì ko lỗi, chứ global thì lỗi.


Wouldn't this commit break in the benchmark classes in the case there is no `osd_ra` in the config file? For example, the following line is in the KvmRbdFio class (as well as in most other benchmark classes):
`self.run_dir = '%s/osd_ra-%08d/client_ra-%08d/op_size-%08d/concurrent_procs-%03d/iodepth-%03d/%s' % (self.run_dir, int(self.osd_ra), int(self.client_ra), int(self.op_size), int(self.total_procs), int(self.iodepth), self.mode)`

When `osd_ra` isn't specified in the yaml config file, it will default to `None`. When the above line is executed, it will attempt to parse an int from `None` and break with a `TypeError`.

Should a note be made about always specifying a value for `osd_ra` in the config file, or should the self.run_dir line in the classes as above be changed to not include `osd_ra`? Or is this the desired effect?


also @Robo0603179, why exactly did you need to comment out basically everything in GetStuffIn.java


saw approval from mrallen1
at https://github.com/basho/bitcask/commit/cffc2f7fe91f24e67bfa0b6ff010e4f41a199691


merging basho/bitcask/dr/apply_memory_leak_patches_to_1.7.2 = cffc2f7f into borshop-integration-232-dr/apply_memory_leak_patches_to_1.7.2


basho/bitcask/dr/apply_memory_leak_patches_to_1.7.2 = cffc2f7f merged ok, testing candidate = 3e1b29e0


TeamCity Open-source Libraries :: Sourcerer :: CI [Build 0.0.38.0](http://build01:8888/viewLog.html?buildId=14369&buildTypeId=OpenSourceLibraries_Sourcerer_CI) is now running


TeamCity Open-source Libraries :: ThirdDrawer :: CI [Build 1.1.45.0](http://build01:8888/viewLog.html?buildId=14371&buildTypeId=OpenSourceLibraries_ThirdDrawer_CI) is now running


This is a good change


TeamCity Open-source Libraries :: YACLP :: CI [Build 1.0.38.0](http://build01:8888/viewLog.html?buildId=14370&buildTypeId=OpenSourceLibraries_Yaclp_CI) is now running


Should this use the 'RegexpSinglelineJava' module?


TeamCity Open-source Libraries :: YACLP :: CI [Build 1.0.38.0](http://build01:8888/viewLog.html?buildId=14370&buildTypeId=OpenSourceLibraries_Yaclp_CI) outcome was **SUCCESS**
Summary: Tests passed: 20 Build time: 00:00:48


TeamCity Open-source Libraries :: ThirdDrawer :: CI [Build 1.1.45.0](http://build01:8888/viewLog.html?buildId=14371&buildTypeId=OpenSourceLibraries_ThirdDrawer_CI) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:00:50


TeamCity Open-source Libraries :: MvcNavigationHelpers :: CI [Build 0.0.43.0](http://build01:8888/viewLog.html?buildId=14373&buildTypeId=OpenSourceLibraries_MvcNavigationHelpers_CI) is now running


TeamCity Open-source Libraries :: BlogMonster :: CI [Build 2.0.60.0](http://build01:8888/viewLog.html?buildId=14372&buildTypeId=BlogMonster_CI) is now running


Commented out actionListeners to allow for default start button to override 3 games issue


don't forget to add a semicolon at the end of this line


@Robo0603179
For proper method documentation, use the /** comment with 2 stars. The documentation will then show up if you highlight the method in eclipse.


That looks like it fixes the problem. Thanks!


TeamCity Open-source Libraries :: MvcNavigationHelpers :: CI [Build 0.0.43.0](http://build01:8888/viewLog.html?buildId=14373&buildTypeId=OpenSourceLibraries_MvcNavigationHelpers_CI) outcome was **SUCCESS**
Summary: Tests passed: 21 Build time: 00:00:24


Tady taky... @Chleboir-CZ


TeamCity Open-source Libraries :: BlogMonster :: CI [Build 2.0.60.0](http://build01:8888/viewLog.html?buildId=14372&buildTypeId=BlogMonster_CI) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:00:27


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.529.0](http://build01:8888/viewLog.html?buildId=14374&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.64.0](http://build01:8888/viewLog.html?buildId=14375&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) is now running


TeamCity Open-source Libraries :: Sourcerer :: CI [Build 0.0.38.0](http://build01:8888/viewLog.html?buildId=14369&buildTypeId=OpenSourceLibraries_Sourcerer_CI) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:01:38


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.529.0](http://build01:8888/viewLog.html?buildId=14374&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:34


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.64.0](http://build01:8888/viewLog.html?buildId=14375&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) outcome was **SUCCESS**
Summary: Tests passed: 3 Build time: 00:00:58


Merge met de Master om gelijk te maken voor aan maken van develop branch


Is it possible that this.logger is undefined somehow? I'm getting a "Cannot read property 'info' of undefined" when running my tests with phantomjs.


Doesn't this just match tab characters?  Don't you want `[\s]+$`


Thinking about this, my final comment would be that "pr must contain tests to be considered" may be a bit discouraging. Maybe "must contain tests to be merged"?


nit: "high-level"


better use `log_debug_errno(errno, "…` here...


Whatever you want to do. I wasn't 100% clear on the add-art/addendum distinction.  Feel free to change the language.

I did change the logo at the top of the popup. Should I change it back to add-art?
![replace_script_based_google_ads_blocked_by_other_ad_blockers__closes_ _ _coreytegeler_add-art-chrome_e349d9f](https://cloud.githubusercontent.com/assets/105414/13264989/8c1240ee-da25-11e5-88c1-d47e915a8117.png)


(object)?


Thank you! :D I've accidentally overlooked this one when converted scripts to Pleased Giraffe format.


nitpick: spurious newline before closing bracket...


Instead of declaration similar code for  $menuItemHasChild and   $subMenuItemHasChild you can use has-child class which you have added before.


why does this go away?  unrelated to this change?


Good code style


cancellation


Sean, Dan W.


upload R script to github


Style: `for (index, button) in buttonsArr.enumerate()` would be straightforward


`save!` should probably be used here just incase it fails for some reason that `valid?` doesn't handle.


Oh yeah, I missed the `function-declaration-parameters` include.


Not necessarily, In Python 3 a Unicode string is expected by `quote`, which is encoded as a utf8 bytestring before passing to the new `quote_bytestring` function. Passing a bytestring to `quote` in PY3 may raise an error. I don't know though. Might be worth testing.


I prefer to up the version number just before doing a release, but I've not read up on the trade-off. Why do you prefer to increment the version number early?


Would it make sense to move this to a static method of `IndexNotFoundException` (i.e. create a factory method)?


Any reason this needs to be an NSObject?


:sunglasses:


It is more complicated than you think. One depth path isn't properly supported yet.


Controller tests should only test a single request.


@borshop merge


@bgruening Sure, I had no idea meme is using perl.  I've never encountered an error in the many meme tool executions I've run.  I wonder how I have not seen this.  Is there some special case that I am missing?


@MikereDD Try it now


What assertion can we use to verify that the database has one less record in it?


cool


Tabs and spaces (stupid brain)... but still: http://docs.oracle.com/javase/specs/jls/se8/html/jls-3.html#jls-3.6


@tinapotter @mikeflowerdew Here I've added the flag for the shower-shape, LAr cleaning as in [FillPhoton](https://svnweb.cern.ch/trac/atlasoff/browser/PhysicsAnalysis/SUSYPhys/SUSYTools/tags/SUSYTools-00-07-41/Root/SUSYObjDef_xAOD.cxx#L1657). In a previous commit https://github.com/susynt/SusyNtuple/commit/473540644a22997fed0fa78baaf1ffb91c845aab I've added the author information as well.

I'm starting to think that it might just be simpler in the case of photons to just use SUSYTools' "baseline" flag instead of storing these separate cleaning flags, etc... for SusyNtuple photons. This would work under the assumption that we do not need separate working points for photons (so just photons satisfying the "Tight" ID, which is default in SUSYTools). All other selection criteria done in "FillPhoton" in SUSYTools is the same for every analysis (i.e. is not configurable) so it is not so necessary to store it in the SusyNt. What do you think?


I had the same change before without the `.encode()` calls and it failed. I added them as an experiment. I force pushed, so that is lost - I maybe shouldn't do that when somebody else is paying attention but I can't stand a dirty git history :) (I should just tidy up at the end tho').


Since we depend on the image actually being the database for its id, can we use `create!` here?


This would be a tad safer in a defer


I'm working on a fix right now.

Feature "unicode_strings" is not supported by Perl 5.10.1 at /usr/local/tools/meme/4.11.0/iuc/package_meme_4_11_0/6ee2e1225125/lib/perl/Alphabet.pm line 5
BEGIN failed--compilation aborted at /usr/local/tools/meme/4.11.0/iuc/package_meme_4_11_0/6ee2e1225125/lib/perl/Alphabet.pm line 5.
Compilation failed in require at /usr/local/tools/meme/4.11.0/iuc/package_meme_4_11_0/6ee2e1225125/lib/perl/CheckingSAX.pm line 14.
BEGIN failed--compilation aborted at /usr/local/tools/meme/4.11.0/iuc/package_meme_4_11_0/6ee2e1225125/lib/perl/CheckingSAX.pm line 14.
Compilation failed in require at (eval 12) line 3.
	...propagated at /usr/share/perl5/base.pm line 94.
BEGIN failed--compilation aborted at /usr/local/tools/meme/4.11.0/iuc/package_meme_4_11_0/6ee2e1225125/lib/perl/MemeSAX.pm line 3.
Compilation failed in require at /usr/local/tools/meme/4.11.0/iuc/package_meme_4_11_0/6ee2e1225125/bin/meme_xml_to_html line 48.
Warning: meme_xml_to_html exited abnormally and may have failed to create HTML output.
Error running MEME.

Yes, get perl 5.10 installed :)


Yeah, I'll fix it in the rebase.


Can we keep these as separate tests? Makes it easier to see what fails when a test does not pass.


Yeah, fixin


This was meant for #2.


ok


Sorry I got it wrong in the first place. I'm particular about my middle initial in stuff so you'd think that I would have paid more attention to detail there ;-)

Looking forward to next week's show!


all tests pass:
success: http://buildbot.bos1/builders/test-bitcask/builds/576


what?


Consider using `path_to my_image` instead of `/images/' + my_image.id.to_s`.


Also, lodash has been upgraded in this commit.


gofmt did this


Hi human!


When localhost isn't specified, running the tests on os x causes a security firewall warning that must be approved before continuing, however, the test completes with a failure before you're able to do that.  Explicitly specifying localhost allows incoming connections.


+1 looks good.


Cool, Thanks!  Let me know if I can help.


No worries. I go by just about anything.


This shouldn't be run the first time around, check for last being zero.


Building now after syncing up. I'm at work though won't be able to flash until around 5:30 PAC time. Will let you know.


Omer Zahid


done


It's not necessarily true anymore, the lock_type becomes a property of the operation rather than a record of the lock to release.  Once the locks are released, the lock_manager will be empty.  This field, however, does not get cleared.


`git commit -am "Change @jdalton's name to 'Your Majesty'"`


Done


Take your time


Is this mutex needed? It's only used in this function


> The GP has huge problems with concave objects, so I
  temporary disabled centroid adding to gp, to see if it is better, but
  looks like it isnt.

Ok, let us know.


Yeah, a clean git history is a beautiful thing. And your most recent patch is much cleaner looking--if it works. We'll see soon enough. Too bad AppVeyor is so slow.


That's is true.
I have seen now the video.
There are several issues with that calibration on this video and the
result is not good, but for that quality of reference, fantastic.
Issues are:
Motion blur on taking image, (because no global shutter and you'r move)
Print not mounted on cartoon or something and further bend, this
create wrong calibrations.
Illumination maybe not optimal.

Now, that procedure is unlike it can be done on machine and further if
you modify lens position the resulting distortion changes.
Some have microscope cameras or additional lenses.
The problem is visible if you click in the image with correct
calibrated steps/mm and camera mm/pixel.
If you click near center at 1/3 away from center and the resulting PCB
or component position is off 0.15 or more mm after cam calibration
your know, there is a problem.

I assume, calibration is applied or remade when microscope cam or and
uplooking cam is setup. Certain parameters are not possible to extract
using this setup, but it don't matter
as on this setup, you don't make sheared images where it is something
out of focus.
You have always the same "Lab" environment.
Normally calibration is made in set's , example one calibration is 6
sets and one set means 10 images, total 60 images. For the set, it's
important to cover all the image equality, with
no preference to certain boarder.
If we could say, we make X sets where the user or for uplooking cam
the nozzle changes the Rotation, and the machine recognize the
calibration grid, and moves 1mm up , and
take next calibration grid, until the calibration grid cannot be
recognized. Then it goes to center and do the same on down direction,
go 1mm left, and take the same row of calibrations. Same for right
side.


`Omer Zahid`


Try using these values: (not because of version #s, but because of coverage/use)

```
  "dependencies": {
    "hapi": "^7.5.3",
    "path": "^0.4.9"
  },
  "devDependencies": {
    "connect-modrewrite": "^0.7.9",
    "es5-shim": "^4.0.1",
    "grunt": "^0.4.5",
    "grunt-cli": "^0.1.13",
    "grunt-contrib-clean": "^0.6.0",
    "grunt-contrib-copy": "^0.5.0",
    "grunt-contrib-watch": "^0.6.1",
    "grunt-hapi": "^0.8.1"
  },
```


Årets commit-besked!


I see no errors, but I think we've moved away from [two spaces after a period.](https://www.google.com/search?q=two+spaces+after+period&ie=utf-8&oe=utf-8) :man:


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


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


Does this merge imply that any inequality query on the datastore cannot be parallelized?


@dhalperi


Expand image sign upside-down


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


If it helps, subop 11 is QUEUE:
https://github.com/OmerMor/SCI16/blob/master/INTERP/AUDIO.C#L195


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


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


This time should be passed down from run(), rather than having the scrape have data with two different timestamps.


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `tests/scala/ScalaLintBearTest.py`, lines 15 to 17.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -12,9 +12,9 @@
 """.split("\n")

 ScalaLintBear1Test = verify_local_bear(ScalaLintBear,
-                                      valid_files=(),
-                                      invalid_files=(test_file,),
-                                      tempfile_kwargs={"suffix": ".scala"})
+                                       valid_files=(),
+                                       invalid_files=(test_file,),
+                                       tempfile_kwargs={"suffix": ".scala"})

 ScalaLintBear2Test = verify_local_bear(ScalaLintBear,
                                       valid_files=(test_file,),
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `tests/scala/ScalaLintBearTest.py`, lines 20 to 22.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -17,8 +17,8 @@
                                       tempfile_kwargs={"suffix": ".scala"})

 ScalaLintBear2Test = verify_local_bear(ScalaLintBear,
-                                      valid_files=(test_file,),
-                                      invalid_files=(),
-                                      settings={
+                                       valid_files=(test_file,),
+                                       invalid_files=(),
+                                       settings={
                                           'scalalint_config': 'config.xml'},
                                       tempfile_kwargs={"suffix": ".scala"})
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 24.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -21,4 +21,4 @@
                                       invalid_files=(),
                                       settings={
                                           'scalalint_config': 'config.xml'},
-                                      tempfile_kwargs={"suffix": ".scala"})
+                                       tempfile_kwargs={"suffix": ".scala"})
```


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 9.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -6,7 +6,7 @@
   def main(args: Array[String]): Unit = {
     println("Hello, world!")
     var x = 10
-
+
   }
 }
 """.split("\n")
```


You probably need to mock the repository instead of the loader. (Since a mock returns null or the default value on all its methods...so mock.getRepostiory() returns null if not otherwise specified)


Added this condition if ($result['billing_note'] != EMPTY) {


@rmarau oh. yes you are right.


This is good tip. I will change it.


Done


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


Nit: You could create a local map, put that in an immutable wrapper and then assign this one to the `MAP` constant.


- no bluestore update?
- the collection_move_rename operation is already stale in that the cid is always the same.  let's make a new op (just rename?) that only encodes teh cid once, and maybe has a flag indicating whether to ignore enoent?  or keep it a separate op.. but at least lets not keep the two cid's that we assert match later on.


Further on in the style guide we see the suggested naming convention for a model.

```
// Contains the ShoppingCart model. Notice that there is no suffix for the models.
shopping-cart.ts
```

Since this states that there is no suffix used for the models, it might cause some confusion because the example directory structure is using a model suffix for some models.

I removed the model suffix to keep consistency across the whole style guide.


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


Should be 2016-02-10


fast-forwarding patch-1.7.2 to borshop-integration-232-dr/apply_memory_leak_patches_to_1.7.2 = 3e1b29e0


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


Yay, a `final` class declaration. I like that. :)


Please note that the changes I made to fix the dark pixels in the rocket launcher muzzle flash were actually the `MISFA0` and `MISFB0` lumps I included in `doomretro.wad`. The change to the offset in `info.c` is because said lump is not as tall, and the change to `i_tinttab.c` is merely reformatting.


Ranieri,

Temos que ver o lance da sobreposição com o menu superior


The one below is GTMNSDictionary+URLArguments; this one is GTMNSString+URLArguments.

Do you think it would make sense to move this down there?


Same here; you could put a local map instance into an immutable wrapper first.


Prinz is also a milk liquor. It needs #Citron Lime


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


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


this was created on github - deleting so that i can push same name file from local


made a line change.


Roger.


It looks like `cabal-install >= 1.20 && < 1.23` can't download the index or packages when run on AppVeyor, and `--allow-newer` was added in 1.20.0.0.  I can't think of a solution.  I guess we should stop building cabal-install until after the release.  Would you like me to comment it out?

1.20.0.3: https://ci.appveyor.com/project/grayjay/cabal/build/1.0.43
1.22.0.0: https://ci.appveyor.com/project/grayjay/cabal/build/1.0.45


Remove the `link = mistune.escape(link, quote=True)` in L42


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `tests/scala/ScalaLintBearTest.py`, line 8.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/scala/ScalaLintBearTest.py
+++ b/tests/scala/ScalaLintBearTest.py
@@ -5,7 +5,7 @@
 object HelloWorld {
   def main(args: Array[String]): Unit = {
     println("Hello, world!")
-    var x = 10
+    var x = 10
   }
 }
 """.split("\n")
```


Done


It should be, "Dit daglige resumé" :P


Hello!

New message, please read <http://wellnesshabits.com/which.php?g>

sveni_lee@gmx.de


Its no problem :P I appreciate the help, sorry if i seemed short i was rushing from meeting to meeting at the time :) It's taking longer than a normal release due to a few factors including a slow response as none of the regulars are dedicated to looking after SW, if your interested i listed some of the causes here #1901


Hi Cri,

A few points:

The print is mounted on cardboard and was pretty straight. Maybe not as
good as it could be, but it wasn't bending.

I agree, the lighting could definitely be better.

This is really a first attempt at this. Improvements are certainly welcome,
but now I am moving on to other tasks as what I've got now gets the job
done. I'll revisit it if I run into limitations down the road.

Jason

On Tue, Feb 23, 2016 at 12:17 PM cri-s <notifications@github.com> wrote:

> That's is true.
> I have seen now the video.
> There are several issues with that calibration on this video and the
> result is not good, but for that quality of reference, fantastic.
> Issues are:
> Motion blur on taking image, (because no global shutter and you'r move)
> Print not mounted on cartoon or something and further bend, this
> create wrong calibrations.
> Illumination maybe not optimal.
>
> Now, that procedure is unlike it can be done on machine and further if
> you modify lens position the resulting distortion changes.
> Some have microscope cameras or additional lenses.
> The problem is visible if you click in the image with correct
> calibrated steps/mm and camera mm/pixel.
> If you click near center at 1/3 away from center and the resulting PCB
> or component position is off 0.15 or more mm after cam calibration
> your know, there is a problem.
>
> I assume, calibration is applied or remade when microscope cam or and
> uplooking cam is setup. Certain parameters are not possible to extract
> using this setup, but it don't matter
> as on this setup, you don't make sheared images where it is something
> out of focus.
> You have always the same "Lab" environment.
> Normally calibration is made in set's , example one calibration is 6
> sets and one set means 10 images, total 60 images. For the set, it's
> important to cover all the image equality, with
> no preference to certain boarder.
> If we could say, we make X sets where the user or for uplooking cam
> the nozzle changes the Rotation, and the machine recognize the
> calibration grid, and moves 1mm up , and
> take next calibration grid, until the calibration grid cannot be
> recognized. Then it goes to center and do the same on down direction,
> go 1mm left, and take the same row of calibrations. Same for right
> side.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openpnp/openpnp/commit/0143f8ebe02d584081b720a2e65d38878ad54f2f#commitcomment-16278780>
> .
>


Oops!


Like this:

log_debug_errno(errno, "/proc/device-tree");

?


I feel that change goes too far. I appreciate colors, but I feel a bit lost when [INFO], [WARNING], [ERROR] disappear. For many years I used to see [INFO] in output and suddenly output is changed.

I treat colors as my local computer bonus. My colleagues, my Jenkins will probably use default maven installation, and I would like my output is the same as others output. It means I expect color will improve my output rather than modify it.


ohhh, this is a crutch :( i not understand, why bug at #5 it is blame a ZfSnapPhpDebugBar? imho, it is blame author #5.


You're not expecting an exception, so if an exception is raised, the test should fail.


Unfortunately, skitch is dicontinued and these URLs don't work anymore :disappointed:


Suggestion, add another column with boolean values (true or false) which represents whether the resource is valid.


@mikepenz I was about to fill a PR but I'll skip it, now that you've seen it. BTW, ViewCompat.setRotation does nothing in APIs prior 11. The arrow will still be pointing sown after expanding. IMO it's nothing to worry about. ViewCompact will at least prevent it from crashing in old devices.


Noted.


I'm noticing in my local build that the NETCore50 configuration is still getting the wrong packages.  I think we need another section in the project.json.  /cc @mellinoe


Das muss
"Für diese Maschine müssen Abbilder in die folgenden Geräte geladen werden: "
heissen (Das "n" am Ende von "Geräten" ist zuviel


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


this mac system file


@rmarau yes you are right. might worth of notting somewhere in the javadoc of the item.

I have already commited the change. Thanks for pointing it out. Might just was to late yesterday at 01:00 am ;)


The question was, is there a reason that it is coded inside
ReferenceCamera and not
inside Wizard ?

2016-02-23 20:29 GMT, Jason von Nieda <notifications@github.com>:
> Hi Cri,
>
> A few points:
>
> The print is mounted on cardboard and was pretty straight. Maybe not as
> good as it could be, but it wasn't bending.
>
> I agree, the lighting could definitely be better.
>
> This is really a first attempt at this. Improvements are certainly welcome,
> but now I am moving on to other tasks as what I've got now gets the job
> done. I'll revisit it if I run into limitations down the road.
>
> Jason
>
>
> On Tue, Feb 23, 2016 at 12:17 PM cri-s <notifications@github.com> wrote:
>
>> That's is true.
>> I have seen now the video.
>> There are several issues with that calibration on this video and the
>> result is not good, but for that quality of reference, fantastic.
>> Issues are:
>> Motion blur on taking image, (because no global shutter and you'r move)
>> Print not mounted on cartoon or something and further bend, this
>> create wrong calibrations.
>> Illumination maybe not optimal.
>>
>> Now, that procedure is unlike it can be done on machine and further if
>> you modify lens position the resulting distortion changes.
>> Some have microscope cameras or additional lenses.
>> The problem is visible if you click in the image with correct
>> calibrated steps/mm and camera mm/pixel.
>> If you click near center at 1/3 away from center and the resulting PCB
>> or component position is off 0.15 or more mm after cam calibration
>> your know, there is a problem.
>>
>> I assume, calibration is applied or remade when microscope cam or and
>> uplooking cam is setup. Certain parameters are not possible to extract
>> using this setup, but it don't matter
>> as on this setup, you don't make sheared images where it is something
>> out of focus.
>> You have always the same "Lab" environment.
>> Normally calibration is made in set's , example one calibration is 6
>> sets and one set means 10 images, total 60 images. For the set, it's
>> important to cover all the image equality, with
>> no preference to certain boarder.
>> If we could say, we make X sets where the user or for uplooking cam
>> the nozzle changes the Rotation, and the machine recognize the
>> calibration grid, and moves 1mm up , and
>> take next calibration grid, until the calibration grid cannot be
>> recognized. Then it goes to center and do the same on down direction,
>> go 1mm left, and take the same row of calibrations. Same for right
>> side.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/openpnp/openpnp/commit/0143f8ebe02d584081b720a2e65d38878ad54f2f#commitcomment-16278780>
>> .
>>
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/openpnp/openpnp/commit/0143f8ebe02d584081b720a2e65d38878ad54f2f#commitcomment-16279066


I've logged this as an issue here

https://github.com/driftyco/ionic/issues/5576


great


for the individual configuration a tests


:) sure!  ...and great work with all the libs. Thanks for that


The reason why your pull request only displays a small portion of your code is it only includes the portion you added since you created the separate branch.  I addressed this problem by deleting my entire code, pushing it to the repo, then readding my code and pushing again.  Now it displays all the code in one solid chunk.  In the future, we won't have to do that because now we know how to use pull requests. =) For now, I will peruse your code outside of the pull requests and write my comments here.


@rmarau My plesure ;). thanks ;)


Danke für den Hinweis, ist behoben.


Hier gehe ich mit der Übersetzung nicht konform.
"Wahrscheinlich" ist zu hart.
Ich würde hier "Eventuell" bevorzugen, da dies das "may" m.E. besser wiederspiegelt. Im Original ist norgends von "Probably" die Rede...


I agree. Basically it will be the same is it is now on dev, except for re-using the `IsBackgroundHttpContext`, which is good, and using LogicalGet/Set. What is the benefit of the latter again?

A PR would be awesome.


Is it possible that different partitions have different InputFormat?


I think that's the wrong way to approach the problem. Over four months ago I [posted an issue](https://github.com/retep998/winapi-rs/issues/218) about how to make compile times 4x faster, and since then very little of that has been implemented. The compiler will never magically get 2x faster, this is also just indicative that the design of winapi may be wrong in terms of scalability.


Testing 1.1


Hej tak to kašlat.


(And **if** it makes sense to have toCLO,fromCLO,cimprove alongside toLO,fromLO,improve, won't it also make sense to have RoundTripCLO alongside RoundTripLO?)


Ah ok, found some examples, should be
log_debug_errno(errno, "/proc/device-tree: %m");
Merging and rebasing follows soon.


I actually don't get the note. Line 42 escapes the title and not the link, right?


Seems to work.  :+1:  Release it!

Is the following expected output?  (I've redacted "live" and "totalsubsity", but they look reasonable).

```
{
  "poolsize": 33998,
  "difficulty": 2,
  "allmempooltix": 20,
  "ownmempooltix": 0,
  "immature": 28,
  "live": yy,
  "proportionlive": 0.009029942937819871,
  "voted": 19,
  "totalsubsidy": XX.xxxxxx,
  "missed": 0,
  "proportionmissed": 0,
  "revoked": 0
}```


sneha


Thanks for spotting the spelling.  Not intended to be a reference to Snoop Dogg...


change to cut, not awk


QuickFixIDD-Memos (mjwitte) - i386-Windows-7-VisualStudio-12: OK (2119 of 2119 tests passed, 0 test warnings)

![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)


The only reason there was to get the std::string copied? why not using something like:

std::string(old_string).c_str()?


Can we get rid of `java.io` here too?


Diese Übersetzung erscheint mir selbst nun etwas ungelenk.

Was haltet Ihr von:

Sie können nichts tun, um dieses Problem zu beheben. Sie können nur darauf warten, das die Entwickler "
"die Emulation verbessern.\n"

?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1329.3df74fe66670df962c5d2c954796b40f3c95bf5b](http://build.palaso.org/viewLog.html?buildId=69568&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1020, ignored: 12 Build time: 00:06:33


@thongsav-usgs the only reason this `+1` was added was to fix the colors correct?


This https://autotools.io/libtool/version.html indicates that this is the wrong thing to do.  This changes the soname at every change in release and requires all users to rebuild.


There is no rule against that AFAIK.


the scope gets lost in the next loop in the for() loop


@thongsav-usgs why didn't it work here? Because it is messing up the labels for the callouts


Gefällt mir persönlich sehr gut, kann meinetwegen gerne so übernommen werden.


not exactly.


Wenn "Minimum", dann aber auch wieder "Maximum", oder ?


clearButtonStyle, right?


Certainly, these things are equi-consistent.


 ----------------
	User:
	Device Name Dev2787
	Model samsung, SM-T235
	App: TrackView, v. 1.4.2-pro
	SDK Version: 19
	Language: vi_VN
	Carrier: VN MobiFone

Mon Feb 15 00:41:38 GMT+07:00 2016:
[12:41:38 AM]: VApplication.init
[12:41:38 AM]: Loading settings {"m4":0,"ry":0}
[12:41:40 AM]: valid date: 2013.9.30-2016.6.30
[12:41:40 AM]: parsed date: Mon Sep 30 00:00:00 GMT+07:00 2013 Thu Jun 30 00:00:00 GMT+07:00 2016
[12:41:40 AM]: Command: enablepaidversion

[12:41:40 AM]: Command: setrelay

[12:41:40 AM]: onStartSession com.trackview.activity.TrackView@41fe65e8
[12:41:41 AM]: onStartSession com.trackview.login.LoginActivity@4202a8c0
[12:41:42 AM]: onEndSession com.trackview.activity.TrackView@41fe65e8


What I saw from the previous tests is that LTC like somewhat lower reduction (although slightly) than STC.
I'm afraid that the best constant here (for LTC and XLTC) would not pass STC. But maybe you can solve this by changing something else. Anyway, 2.10 is a constant to try.


Einverstanden.


is the replyTo really needed?


Warum eigentlich Minimum/Maximum und nicht minimal/maximal?


so the std::string goes out of scope, too and the c_str() is a dangline ptr, right?

This is a design problem here when mixing std::string and old _cstr() that needs something else.


http://www.gnu.org/software/libtool/manual/html_node/Release-numbers.html also states that -release should be used *instead* of -version-info and only for internal libraries.


Das muss mir wohl beim Überprüfen durchgerutsch sein. Ich würde "Minimale Strahlbreite" und "Maximale Strahlbreite" ggü. "Minimum Strahlbreite" und "Maximum Strahlbreite" bevorzugen.


Frag fabs ob er einen alias im pool dafür erstellt. Ich bin mir nicht sicher ob es klug ist das auf die Website zu packen.


I can fix that locally, no reason to reset this PR.  We need this in so that Davis's change can go in.


lgtm.


@monnerat can you review this line, I did this by copying the line above but it's occurred to me I don't know what the leading 1 is for, I assume it's a type specifier. this option takes type `long`. i notice options start with 0, 1, 2


Biggest reason was that we needed to be able to capture images directly
from the camera before they were processed. If you feel it would work
better in the wizard that is fine with me. We can just supply the images
via a callback.

Jason

On Tue, Feb 23, 2016 at 12:33 PM cri-s <notifications@github.com> wrote:

> The question was, is there a reason that it is coded inside
> ReferenceCamera and not
> inside Wizard ?
>
> 2016-02-23 20:29 GMT, Jason von Nieda <notifications@github.com>:
> > Hi Cri,
> >
> > A few points:
> >
> > The print is mounted on cardboard and was pretty straight. Maybe not as
> > good as it could be, but it wasn't bending.
> >
> > I agree, the lighting could definitely be better.
> >
> > This is really a first attempt at this. Improvements are certainly
> welcome,
> > but now I am moving on to other tasks as what I've got now gets the job
> > done. I'll revisit it if I run into limitations down the road.
> >
> > Jason
> >
> >
> > On Tue, Feb 23, 2016 at 12:17 PM cri-s <notifications@github.com> wrote:
> >
> >> That's is true.
> >> I have seen now the video.
> >> There are several issues with that calibration on this video and the
> >> result is not good, but for that quality of reference, fantastic.
> >> Issues are:
> >> Motion blur on taking image, (because no global shutter and you'r move)
> >> Print not mounted on cartoon or something and further bend, this
> >> create wrong calibrations.
> >> Illumination maybe not optimal.
> >>
> >> Now, that procedure is unlike it can be done on machine and further if
> >> you modify lens position the resulting distortion changes.
> >> Some have microscope cameras or additional lenses.
> >> The problem is visible if you click in the image with correct
> >> calibrated steps/mm and camera mm/pixel.
> >> If you click near center at 1/3 away from center and the resulting PCB
> >> or component position is off 0.15 or more mm after cam calibration
> >> your know, there is a problem.
> >>
> >> I assume, calibration is applied or remade when microscope cam or and
> >> uplooking cam is setup. Certain parameters are not possible to extract
> >> using this setup, but it don't matter
> >> as on this setup, you don't make sheared images where it is something
> >> out of focus.
> >> You have always the same "Lab" environment.
> >> Normally calibration is made in set's , example one calibration is 6
> >> sets and one set means 10 images, total 60 images. For the set, it's
> >> important to cover all the image equality, with
> >> no preference to certain boarder.
> >> If we could say, we make X sets where the user or for uplooking cam
> >> the nozzle changes the Rotation, and the machine recognize the
> >> calibration grid, and moves 1mm up , and
> >> take next calibration grid, until the calibration grid cannot be
> >> recognized. Then it goes to center and do the same on down direction,
> >> go 1mm left, and take the same row of calibrations. Same for right
> >> side.
> >>
> >> —
> >> Reply to this email directly or view it on GitHub
> >> <
> https://github.com/openpnp/openpnp/commit/0143f8ebe02d584081b720a2e65d38878ad54f2f#commitcomment-16278780
> >
> >> .
> >>
> >
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/openpnp/openpnp/commit/0143f8ebe02d584081b720a2e65d38878ad54f2f#commitcomment-16279066
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/openpnp/openpnp/commit/0143f8ebe02d584081b720a2e65d38878ad54f2f#commitcomment-16279168>
> .
>


Why should we add this message argument?


Line 15:

I think we shouldn't include code that demonstrates our functions within the .py file, but rather should put that into a notebook file.


Fixes #22


Numm am besten einen link Text nicht die URL

Eher sowas wie "hier klicken um zur Liste zu kommen".


? the std:: vector does'nt go out of scope during the call to the addon.
The addon MUST copy the char ptr, thats true - but OK in my eyes


OK, denn englisch "minimum width" ist deutsch "minimale Breite" (oder "Minimum-Breite" mit Bindestrich, denn "minimum" gibt's im Deutschen nicht als Adjektiv :-) )


that's great


You might need to throw an error if `BACKEND_URI` is set but `BACKEND` is not set to `"ilp-quoter"`, or document that you need to set both


Ich werde es zu "minimale Strahlbreite" und "maximale Strahlbreite" ändern.


Lines 22-23:

When you use triple quotes_ inside _a function, this becomes the documentation for that function.  That's why triple quotes are used in functions as well as at the top of the file.  This is preferred over #.  So, if someone were to type in set_size_range?, no documentation would appear in the current format.


Didn't realise #92 was another fix to the issue; will look at it to try to fix it up.


Hier gehe ich mit der vorgeschlagenen Änderung nicht konform.
Eine "Vorrichtung" ist etwas physisches, während eine "Massnahme" auch in Software erfolgen kann.
Bei der hier gemeinten "Protection" kann es sich durchaus um einen reinen Softwaremechanismus handeln, der noch nicht richtig verstanden und emuliert wurde.
Aber vielleicht können wir uns auf "Schutzmechanismen" einigen ? :-)


Yes, but it's an ugly workaround cause we rely to assign a c_str() a const char* while assignmed by simple copy would be sufficient. Your workarounds of course works ... but is overkill.

that's why I said: a different design approach is needed to solve that without that much overhead.


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


Probably should add Time types here too, a growing number of providers support those


mytrip


"Schutzvorkehrungen"? Aber Schutzmechanismen ist auch gut.


nu mfw :egg:


Sounds good !

2016-02-23 21:42 GMT+01:00 Lothar Serra Mari <notifications@github.com>:

> Ich werde es zu "minimale Strahlbreite" und "maximale Strahlbreite" ändern.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rootfather/mame/commit/ebf2ee3705279c7fc5ba30ecc708036c3ee99aab#commitcomment-16279435>
> .
>


Audacity would like to thank Chris Navarrete from Fortinet’s FortiGuard Labs for notifying us of this bug and associated security risk (CVE-2016-2541).


Mit "Schutzmechanismen" wäre ich einverstanden.


Will do this as soon as we'll get stable test results and will show them in build summary along with mobilespec results. For now, it is needed to see if there were any failures.


I have a fix locally for this and will check it in with my work in TFS later today.  Please don't reset this PR.


Lines 22-23:

Furthermore, your documentation is very scanty.  It doesn't tell me anything about the inputs.  I'm pretty sure they are scalar floats.  But you need to specify what formats the inputs need to be in.  It also isn't clear without reading into your code what 'tick' even is.  This is also bad formatting, because you don't include the dataframe itself as an input.


@sgress454 k my last change is: https://github.com/treelinehq/machine-as-script/commit/ba82f443a5c2749bc322314a1e7eaa82748488d7


Will check. Also happened with conservative governor. Interactive is the only Default governor working. I'll leave feedback if I sorted. Thank you dorimanx!


Thanks for your advice, I'll respond in the forum.


sorry, I still haven't seen your solution. I can imagine 2 ways to pass strings from kodi to addon:

1.) char[] buffer instead const char*
2.) std::string (bad idea because std::string differences in release and debug for example)


Audacity would like to thank Chris Navarrete from Fortinet’s FortiGuard Labs for notifying us of this bug and associated security risk (CVE-2016-2540)


"Schutzvorkehrungen"? Aber Schutzmechanismen ist auch gut.

Mit "Schutzvorkehrungen" wäre ich auch einverstanden.

Machen wir Best of 3.
rootfather entscheidet, was Ihm besser gefällt.

2016-02-23 21:44 GMT+01:00 Michael Zapf <notifications@github.com>:

> "Schutzvorkehrungen"? Aber Schutzmechanismen ist auch gut.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rootfather/mame/commit/ebf2ee3705279c7fc5ba30ecc708036c3ee99aab#commitcomment-16279493>
> .
>


Filed #436 for you.


Line 20:

A continuation of the above comment.  In your code, you create a variable called 'prop,' then proceed to use it in your function.  But if I tried to use your function right now outside of this notebook, it would be useless because I don't have a prop defined.  Your function needs to stand independent of the notebook.


Just one question. Overall LGTM.


Crap, forgot to tick the "make this a branch" box


Sure, would be a good thing to refactor all these classes.


Ich hatte hier auch "eingelegt" statt "geladen" geschrieben, weil ich eine Disk in ein Laufwerk einlege (während es im Englischen wirklich "load a disk in the drive" heißen kann). Es kann auch als "geladen" da stehen, aber weiß dann jeder, was zu tun ist?


done.


2.) is clear it won't work
1.) The real problem here is the mixture of C / C++ interface in a more perfect world one would not need to asign the c_str() but could use a copy of an std::string directly as this is how it's used anyways.


from @25:25 of Tip Calculator video
https://vimeo.com/102084767


Oops, you are right, nevermind. It's seems mistune (the overrided method) also does not escapes the link.


I see.


Yes, best would be always to use .getPlayersInRange(source.getControllerId(), game) for this kind of task.


hello


Vi burde ha et Preference eller Settings objekt som kan provides i steden for å ha logikk i provide metodene. Da blir det bl.a lettere og skrive tester med forskjellige cacheSizer


**HEAD commit has no message.**
*GitCommitBear, severity 1, section `commit`.* *Project wide.*


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `docs`.* *Related to file `docs/Users/Tutorials/Tutorial.rst`, line 327.*

The issue can be fixed by applying the following patch:

```diff
--- a/docs/Users/Tutorials/Tutorial.rst
+++ b/docs/Users/Tutorials/Tutorial.rst
@@ -324,7 +324,7 @@
 -  The settings it uses (optional and required)

 There are two types of Bears which are there-
-- Global
+- Global
 - Local

 The Local Bear can analyze only one file at once. It can
```


Its not three spaces. It is in fact a single tab.


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `docs`.* *Related to file `docs/Users/Tutorials/Writing_Bears.rst`, line 56.*

The issue can be fixed by applying the following patch:

```diff
--- a/docs/Users/Tutorials/Writing_Bears.rst
+++ b/docs/Users/Tutorials/Writing_Bears.rst
@@ -53,7 +53,7 @@
 The Bear so created is of the type LocalBear which can only read from a single file.

 To write a code which can read and extract facts from multiple files,
-A bear of type GlobalBear needs to be created.Here is a simple example
+A bear of type GlobalBear needs to be created.Here is a simple example
 for GlobalBear:

 .. code:: python
```


**Line is longer than allowed. (84 > 80)**
*LineLengthBear, severity 1, section `docs`.* *Related to file `docs/Users/Tutorials/Writing_Bears.rst`, line 53 column 81 to line 53 column 85*.


[SQL]
ALTER TABLE CATEGORY ADD FORCE_START BOOLEAN DEFAULT FALSE


No it is not in the spec yet. But it could be reintroduced to the spec soon. Commenting out this case for now. Thanks