Do you think it would benefit from adding an integration test? I do not think that this is testing that the option actually gets selected in the API call.

Other than this, it looks good.


Agreed, it could use a rethink with recent changes in distros. I recommend filing an issue, nobody but me will see this comment. :)


What does this commit do? If the tests are merged into WPT then there's a different procedure than modifying the to-upstream directory.


[Uploading BIMAGE.v2.10.14-g1.0.4-gp.WIXFMM-114.GPLogo.tar.gz…]()


I have some ideas on how to change the implementation a little to handle the colors and caching.
Would you like me to do a PR to your fork so you can include it in your PR or do you want to just merge in changes and I can do a separate commit?


UserException.permissionError?


I think the min/max value changed signals are now emitted at least once for all controls, when the control is connected. Is that an undesirable behavior?


this seems to work sometimes for me; other times, it shows
```
Uncaught RangeError: Maximum call stack size exceededipcRenderer.on @ /Users/yan/repos/browser-electron/app/content/scripts/aboutPreload.js:17
```

in the page console


I wrote the original tests that never made it to wpt. Replaced them with the ones I already have in a PR against wpt (recent).

The old suite I wrote would fail against the new code since some wrong assumptions were made and the spec was tweaked to make them more clear since.


Ah, sorry for missing that - I haven't bothered setting up a build environment so I didn't actually test these changes. I probably should do that in the future!


Nope. That violates the [WTFPL's conditions](http://www.wtfpl.net/faq/#main).


@solomonhawk I'm confused about this — the docs say that "format" should be an array, but you're expecting it to be a function.

They also say that "parser" should be a function, but make no mention of "parse".

How should I supply an array for the format of my calendar? Also, should `options.parser` in the docs be updated to `options.parse`?


Hmm... 'parent' could be in other places too, including the timeline graphic. I do think 'binary compatible' is better as that is what woof refers to, so consistency is important here.


I think so.  I think those signals should only be emitted if limits have actually been enabled on the AM side.


as you can use the qualified version e.g.
```fsharp
let params = [
    Platform AnyCpu
    FscParam.Target Exe
    Debug false
]
```
I don't think this is anything to be concerned over.


OK. It would be good to include a link to the WPT PR there.


If you really don't need anything in the test constructor, it's probably best to remove this.


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


Not at all. Still a rough WIP, not ready for PR :)


Hi @weisskrispies
Glad you find it useful.
I think you only need to change the custom class in your UIlabel in storyboard to CLTypingLable.
Please check the updated read.me for step 2.

Let me know if it works, if you find this is helpful. Please star the project on top right corner, thank you!


Interesting.  I updated CORS support on the heroku server to allow requests from all origins.  Are you seeing any errors in your console log?


I agree with @cprice404 on this. I think the manipulation of the timestamp is fairly confusing and makes me a little worried. While I guess having the timestamp could potentially be useful for debugging, I also think that it could make things more confusing if you can't tell whether the timestamp you're looking at has been manually manipulated or not. I like the idea of having both a cache generation id and a timestamp.

I wonder, if we decide this should really go in before we ship, if the timestamps in the access log might work well enough instead, and we could switch to just using an atomic integer, and later put back the timestamp for debugging purposes? Currently our access logs log at the second level, but we could change them to milliseconds? I guess what I'm mostly saying is that I question how useful the timestamp is vs. the confusion that the manual manipulation of it causes.

As a sidenote/nitpick, it took me a little while to realize that this function and the ` environment-class-info-cache-with-invalidated-entry` below were not required to fix the issue mentioned in the ticket/in the commit message, but rather because some of the unit tests were failing due to an unrelated issue. It would've been nice to see a note in the commit message about why these functions were added.


Without doing this check, a warning is thrown `Argument "" isn't numeric in multiplication` due to the operation ` $amount *= 100 ` being performed on a empty string.


If an editor is created before a PV connects, then how will it know when the limits have changed?


what's the reasoning with turning them off? does it result in double with the bootstrap `$(sele).tooltip(..)` call?


Can you delete the call to `releaseFrame` after it sends a system error in `dispatchInbound`  as part of this commit. It's double freeing the frame right now.. ideally, if you run the frame pool test on this commit, it should fail (and if not, let me know).


Remove??


Something is clearly wrong with this code. You can't demand for the robot to go 10 units in 1 second, because the robot might not be able to go that fast. We want to select the distance and the speed, not the distance and the time. The time shouldn't be a factor here.


これってなんでですか？


Likewise, this is a problematic test, since the maximum speed for the motors is 1.0, so if you're setting the speeds to 10, you're doing something wrong.


nit: mention that it registers it for the service that was specified when the created


why 2?


Maybe we need a flag?  If limits are enabled then the editor listens for the signals but if not it goes on it's merry way?  Since you have to take an action by calling `enableLimitsMonitoring()` to have limits anyway, it would be silly to not have a means of asking the control if it should wait for them or not.


cuz 2!


What has this to do with CPU usage? The timer should do the work? Did you have done mods on the brushed timer initialization?


About storage data structure: if you have to implement a faster way of searching/deleting then what other data structure would you think of using?


can you use `testutils.NewServer` here instead? (it supports flags to enable debug logging that way)


lgtm


Need better error message


Weren't we actually able to include a `SensorData` object here? Or was that one of the causes making the tests fail?


It doesn't appear the `enableLimitsMonitoring()` is called anywhere. I'm not sure what that means, that no controls have min/max values right now?


hmm we're in the tchannel package so we can't actually. maybe create a new file or find some other file to put the test in that uses tchannel_test as a package so we can use `testutils` for creating server/client


Ah, yeah. Good catch. Fixing.


Consider tracking block depth by test context to support nested modules.


This is testing functionality provided by ActiveRecord. I don't think that's necessary. As long as our app can delete images (verified by the controller test), we should be fine.


@stoeffel - look forward to taking a look at this this evening!  I updated the README on futurizer project - it got around 110 downloads yesterday so appears we are spreading the word! :smile:   Intend to move it into this Organisation and rename shortly too.


just ignore this commit this is just so I have something to go back to in case I mess up on this next part.


HTTP request to server here with array of items to download.


T


nit: just `time.Second`, don't need `1 *`


if you're outside of the tchannel package, you could probably use `raw.Call` rather than writing all the args explicitly


While you're here, could we change the >= to >, and get rid of the +1?


Let's rename this to Itinerary to prevent confusion.


Click the README.md file.
Click the  pencil icon in the upper right corner of the file view to edit.
In the editor, write a bit about yourself.
Write a commit message that describes your changes.
Click Commit changes button................


Because it wasn't totally outdated before.. The players hate the new pvp 1.8 brings.


As Bryce has commented, this shouldn't use `xhr`


Could you please explain some more what these types mean? For example if line, linear and strip are still cpu only? I guess I understand what line and strip means but I'm not certain what is linear means here. Interpolation?


Sure.


remotespark.utils.eventshandler.EventsHandler


Also, you can use `@image` here instead of `Image.last.id`


+1. LGTM.


@SigmaVirus Oh, I didn't know the default was 1500. I thought it was 2000 or 2500, for some reason.

Will update!


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


Congratulations on reaching maturity.    This is a fine project.

The philosophy of making it in pure Java as much as possible has helped me to use it more effectively and even provide feedback.  Thanks for that!


As Bryce has commented, this should be outside the `assert_difference` block. Additionally, we should use `assert_redirected_to` to check _where_ we're redirecting.


You need to separate the items by a meaningful character (not spaces).  So tab, comma, semicolon, something (ideally different than the default separator in the default logger)


Ignore.


You could add a success flash message here.


This commit adds back the sqlite files. Those should be gitignored.


Isn't it in my original PR comment?

Sent via mobile. Please excuse brevity / typos.

> On Feb 29, 2016, at 7:05 PM, Domenic Denicola <notifications@github.com> wrote:
>
> OK. It would be good to include a link to the WPT PR there.
>
> —
> Reply to this email directly or view it on GitHub.
>


Minor, but the confirm message should usually start with "Are you sure"


Yep, that worked! Thanks a lot.


there's a lock when setting `handler` but no lock when reading the `handler`. We should either use a lock / rlock here, or not lock at all (and have a contract that handlers should be set before calls are received)) and


This kind of change should be squashed. (However, as mentioned in the earlier commit, I don't think this test is necessary. But deleting that test will be easier if this commit has been squashed :wink:)


Why not add the new users to the existing test UserAuthenticator impl instead of creating a new test impl class?


This is a nice function.


That would seem to be the case.  We may need @iainworkman to weigh in on this to get further information.


@lvale, there's a parameter as well, so this change just ensures that the precision landing code is compiled in by default.


nitpick: spacing :-)


MasterEquation


addMiniDfsBasedStorage() is already adding a tmp workspace.


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


It felt weird sending in parts from the settings object as well as the settings object


Смари. У тебя вот тут `CSV.read(file, {col_sep: separator}).map{сделатьфильм}` — и в my_movies_list.rb аналогино совершенно `CSV.read(file, {col_sep: separator}).map{сделатьфильм}`. Ну некруто же :(


Not if the original value isn't a boolean. For pointer types, it's a fancy way of saying `(bool)ptr`, e.g. `ptr != nullptr`.


This was allowing me to choose whether or not I wanted logging at the work item level. So I could register 10 work item handlers and decide that I only want logging for 1 of them. Maybe that is unnecessary, but we did lose functionality here.


Are we validating the values anywhere? It would be good if we could validate the value before we apply the values.


The conflict goes the other way. See #1155.


I could not change constraint TervelData bits to all 0, so I did it in temp instead


It appears that NewServer starts listening for connections before it calls the function. Registering endpoints after the server is already listening for connections isn't supported by Channel.


this insert_row.cas() ... I am not sure I did not test it, I grab table last data and increment it for the new record


Cool. Down where you iterate up to `num_items` should be updated to match, too.


This on PlaybackServiceMediaPlayer.java: I'm puzzled by the fact that, on handleMediaKey(), pause() always ends up being called twice. Any reason in particular for such behavior?


I can move this to subchannel_test.go which is outside the tchannel package. Seems more appropriate.


Tervel complains about data type when I insert value


ну вот мне кажется что это слишком частный метод, который код в seen_recommendation не упрощает, а усложняет (это метод, который пригодится ТОЛЬКО там, не имеет осмысленного объяснимого смысла и требует дополнительного чтения). тогда уж лучше в метод не выносить вовсе, а в sort_by по месту всё обстряпать.


It does start listening, and technically it's a race if you receive requests before you register endpoints. However, we know there's no incoming calls in tests till we make the calls, so there's no race. (we do this in a lot of tests already). It shouldn't complain if you register after listening as it is (I think it even uses locks in a safe way so the race detector won't complain either)


Yep that sounds good


you little bitch...


old main create table and insert, schemaChecker gives appropriate errors


We are not planning to allow the compiler to specialize generics across modules. However, we are planning to add a new attribute (@_specialize developed by @atrick) that will allow library developers to force specialization of types that are known at compile time (of the library). You'll be able so ask the compiler to specialize generic functions for the case where K == String and V == Int.


Cherry picked from https://github.com/rvpatkar/angular.js/commit/3ad6d65210e563579fc8a8a89ec16bde56c51b9b


I think these overloads are a bit confusing. It's not very clear how they are expected to behave in response to various values of precision. For instance, what would `overflows(x, 30)` do for this version of the method?


I'm gonna drop the lock. I don't know if the overhead of yet another lock per request is worth it given that the previous code was not doing it.


God? Kinda pushing it.


@lorentey Your BTree implementation looks really impressive! I'd like to make sure that we are not regressing the performance of the library. Would you be interested in adding the library as a benchmark to swift/benchmarks?


yep, agreed.


just curious, why create a new action and not inject an empty payload?


Hmmmm.  This looks clever but confusing.  Especially when `curActiveI` changes value half way down.  I hope I don't have to debug this code in the future.


have you imported before? Possibly from when it was broken?


This is interesting... Typically we do this via: `@image = Image.find(params[:id])` and then `@image.destroy` (where the former often ends up in a `before_action` because it's common to `show`, `edit`, `update`, and `destroy`.)


As mentioned previously, consider using `:success` rather than `:notice`


标点符号错了...


I actually do not know; I went ahead and updated it for now but our Hive recipes are quite the mess still.


As you can see from the above, I changed other things in this same commit, which were not included in the commit description...
Still learning Git...


Corrected to `version`


No longer...


List of suffixes pulled from the following post: https://www.reddit.com/r/2nd_derivative/comments/3ao65n/interesting_and_entertaining_but_not_exactly_an/ct0kg0u


I might need you to debug this unless you have STR or can share a session-store-1 which reproduces.


Yes, compilation should fail there though - it's an actual compiler issue (you have two globals mutating each other). As for `ts-node`, yes, it uses `tsconfig.json` by default - you can say `noProject` though.


Corrected "Marvelled"


This is bizarre... We should let rails handle legitimate server errors automatically for us and respond with non-500 errors ourselves.


It may be worth a comment about why this should be an `assert`.  At first glance it looks like this could happen in the normal coarse of events.  What prior piece of code is responsible for making sure this shouldn't fire?


Is prefexing /usr/local/sbin before /sbin considered normal?  This being a system init script, would it be better to put /usr/local at the end?


`delete :destroy, @image` will work equivalently with less clutter.


[R/corrplot.R:310:12:](https://github.com/vsimko/corrplot/blob/0a74b9f2d693ea929151f3b50b28a3d3447c90c1/R/corrplot.R#L310) *warning:* **no visible global function definition for ‘corrMatOrder’**
```r
    ord <- corrMatOrder(corr, order = order, hclust.method = hclust.method)
           ^~~~~~~~~~~~
```


This is not exactly what I meant. I meant this:
```js
define([ // ...
  "require",
  "./Or"
  "./And",
  "./Not",
], function(Base, Element, TableView, arg, require) {
  "use strict";

  var Or, And, Not;
  // ...
```


Why remove the `assert_difference`? Additionally, adding the test for the flash message should appear in the same commit as the addition of that flash message.


As mentioned previously, we should assert _where_ we're redirecting.


This looks like:

<img width="889" alt="label" src="https://cloud.githubusercontent.com/assets/153/13413863/f9d15760-dfa1-11e5-98ac-1f203ad46562.png">


I actually think the test was better before making it rely on the created image.


unfortunatly we have a one more "non-pie" binary...rmt_storage.. :(
ı tried updated rmt_storage from SPH-M840 but it seems it didnt boot
if you have a chance, can you verify it mate?
here you can pull out updated rmt_storage binary https://github.com/sirmordred/rb3mod-raybst/blob/master/system/bin/rmt_storage
you can test it on any version of android
just replace old one with new one


(except without the word "all")


Should be "ADD INDEX " or "ADD KEY ", not "ADD " only!


As mentioned previously, I don't think this test is necessary.


Oh, absolutely! The code currently measures at 4400 lines without the unit tests, which is probably a bit excessive for a benchmark case, though. It would be easy enough to extract a smaller subset, so I'll do that. (But only after I finish work on `OrderedSet` and release a feature complete version.)

(BTW I used to have a simple benchmarking suite in the repo; I ripped it out when I blew up the original project into several smaller ones, and I haven't revived it yet.)


Removing the `sqlite` files should be squashed into the commit that added them.


return the first defined element?


I'm glad we got rid of this. It would be better to squash that change though.


Yes, fixing that now.


I think we should rely on Rails default 404 handling.


Great!


@dcleao Well, if I do this, the tests fail due to trouble loading the classes... I really seem to need to exclude Or, And and Not from the dependency list


no previous imports. FWIW I saw that error only on this commit, haven't seen it yet on later ones. Will debug if it happens again.


Love this cc: @mattwarren


Can't we just hard-code the servers now? Why the indirection?


It's not valid to hit more than one controller action in a single controller test. Fortunately, we can sitll test this case without doing that.


thankyou!


Where are we redirected?


If I was to implement a faster way of searching/deleting, I would use AVL tree data structure for its balancing of the tree and that their Big O complexity is O(log(n)) for worst case searching, deleting and inserting (with the exception for space complexity being O(n)). This is much better for the worst case, since the current array's worst case is O(n) for searching, inserting and deleting.

In order to implement this, there needs to be a numerical weight for each item in the list or some system that give higher weight for one over the other and create an AVL tree based on the weighed values.


init


Please delete this test so I can stop commenting on it :)


this is very little comment: we are moving to the style for lodash of:
```
import {debounce} from 'lodash';
```


Gee, that was bad.

There was an error in line 240.
Should be b instead of ei.kingRing[Them][0]

Fingers crossed !


git add images/*


I suppose it is, but that won't make it into the revision history.


hm, ça me paraît un peu lourd  les 515 lignes de Simple_nstream + redéfinition des tokens, au regard de la taille du code qui l'utilise. On ne peut pas se baser sur les nouveaux tokens et juste sauter les EOL ?


Shouldn't this method be inherited from AbstractFilter?
Why add to this.operands? Isn't it freezed?


For the ME909U-521 LTE Modem, the entry for u3g needs to be the following for serial access:
U3G_DEV(HUAWEI, ME909U, U3GINIT_HUAWEI),

Unless you have it working with the CDC driver?


There are several unused imports in the section of code. If you have eslint going in IntelliJ, it should tell you about them.  Otherwise, I can help you set it up.


恩，写得不错。


I know I mentioned the module ids... but I didn't realize these were upper case (unlike the module classes). I don't like upper case here. Would case this as camel-case, not Pascal.


`var` could be `const`


Yep, this looks perfect!


ここインデント
img {
とその閉じカッコの
}
のところもインテンド増やしっちゃってるね！


Done.


it's inherited, but here it is overriden, as the functionality is different.
what about
```javascript
var operands = this.operands.slice();
```


WIP :)


@weisskrispies Good idea. That is possible but haven't implemented. I will work on that and let you know :+1:


Good job using `find()` and `text()`


Idem. Shouldn't this be inherited?


I was worried we'd have to choose between dealing with slow generics or a scalability/code size explosion a'la C++ templates; this sounds like a very pragmatic solution to both.


See the test case before this "testThrowError"


idem


Is this necessary?


just wanted to say.. i like this component.  clean looking and very nicely organized.


This is identical except it throws the error in a process.nextTick() callback instead of synchronously.


better than this


Oops, meant to put that on PR #31552 :)


The tests should probably check that after 1 second that the robot has traveled approximately `speed` distance. You can make the check approximate by using a larger value in the epsilon for that `assertEquals`, i.e. `assertEquals(0.8, auto.getDistanceTraveled(), 0.01)`.


Does this code work? Seems to be some sort of problem finding the right directory. This is the error I'm finding:
Windows PowerShell
Copyright (C) 2015 Microsoft Corporation. All rights reserved.

C:\Users\Chris Baskerville\Desktop\crisprdesign [master]> python setup.py
Traceback (most recent call last):
  File "setup.py", line 3, in <module>
    import crisprdesign.util
  File "C:\Users\Chris Baskerville\Desktop\crisprdesign\crisprdesign\util.py", line 3, in <module>
    import sgrna
  File "C:\Users\Chris Baskerville\Desktop\crisprdesign\crisprdesign\sgrna.py", line 7, in <module>
    from Bio.Seq import Seq, MutableSeq
ImportError: No module named Bio.Seq
C:\Users\Chris Baskerville\Desktop\crisprdesign [master]> dir

    Directory: C:\Users\Chris Baskerville\Desktop\crisprdesign

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
d-----        2/29/2016   4:44 PM                bin
d-----        2/29/2016   4:45 PM                crisprdesign
-a----        2/29/2016   4:44 PM             29 .gitignore
-a----        2/29/2016   4:44 PM           1843 find_degen_guides.py
-a----        2/29/2016   4:44 PM           4097 find_genomewide_crispri_grna.py
-a----        2/29/2016   4:44 PM           1869 setup.py


Hey Charlie,

Thanks for the app. Very useful.  Just tried though, and it was returning all events with Status "Closed." I thus was unable to book an event that was actually Open as shown in my iphone app. I edited your code a bit so that it would always try to book an interested event, and then it worked.


The first line of this comment should be capitalized, and there should be spaces.


Also if you have a chance, could you modify the app so that it tries for 2 passes, and if that fails, then it tries for 1 pass?  Thanks!


is this file out of date with the version in https://github.com/borismus/webvr-polyfill/pull/33 ?


Would be useful to mention in the comments why we ignore missing boot files during verify.


I oppose this initial commit.


@andyneff I ran into this yesterday!  Thank you for fixing.


It should not work without `@@species`.


@diracdeltas @bbondy this is excellent. HTTPSE is immediately available now, and no spike in CPU+RAM on app startup like before


Was supposed to be a separate commits, o well mistakes happen.


Changing it to `prepend` still loads the template methods.


Switched to `testutils.NewServer` and `testutils.NewClient`.


I forgot to add gitignore when I created the repository. I have just added it to the repository which has just been updated.


Please confirm if NO_ADS_WRAPPER is still being used meaningfully in any project/solution files?
I could only find it in setenv_base.cmd.
I think it can be deleted as it is no longer used. .

Using FLAVOR_SDK=Debug still targets release folders for output files. i.e. build output is created as Debug flavor, but it is placed in a "Release" folder hierarchy. An invitation for confusion.


I don't know what this line does.  I don't think it is necessary.


they compete with the bootstrap call and the bootstrap ones look better to me


Such ignores should be done by developers globally.
You can find [example templates and guides about .gitignore](https://github.com/github/gitignore) in official github repository.


I don't think you need this method.  The component should mount long before it is every showing a mouse readout.


A few things here...

This Cleaner method is one that is not actually invoked from the code anywhere. It was written to clean up cases where the same assignment got into the database twice.

There's a mistake the in the original test; it should be expecting `Assignment` 2 not to exist, rather than `Article` 2.

I haven't needed to run this since I originally wrote it, and I don't think there are any such case variant titles that should be in production now. But I'm not completely sure.

But anyhow, we should either keep this test — and fix it to `expect(Article.exists?(2)).to eq(false)` — or we should get rid of the entire `Cleaners.repair_case_variant_assignment_titles` (along with `update_assignment_title_to_match_article` and `title_improperly_formatted?` which are just subroutines of that).

I think the latter is probably fine. Doesn't necessarily makes sense to do it as part of the wiki schema PR, though.


How can it be different functionality? Aren't they both on the prototype? You can't change meaning middle way.


You still do not address the case of individual users who say hey I own the place, I should know what my address is.

You need a big fat FAQ at top: I am just a little guy. Can I add my little address?


eslint tells me that you have not declared these in your PropTypes.


repeat of (scrollWidth * 1.5) in max()


this can be notated in jsdoc by `Object|Array.<Object>` or `Object|Object[]`


Should we be overriding WriteError as well or is that just piped through Write?


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


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


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


Ha


in the particualr case of an AND, I'm building a new AND with extra operands instead of building an AND  of an AND with something else.

a = new And(...)
a.and( new And(...))


Do we need to switch to another static ExitCode property that we use and if it is not zero then finish?


You wot?


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


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


Maybe make this a little more generic, like "Authentication failed: Bad username or password"?


Ok, but then you do need to slice the operands.


Should `return;` here


And here


We have a 6 server Redis cluster (3 masters/3 slaves) and would like to switch from SQL server to Redis for ASP.NET session management.  In addition, we are using the RedisAspNetProviders for the session provider.  All six nodes are in the connection string.  Last Friday, we uncovered an issue where IIS worker processes crash if the App pool is recycled and a Redis server node is down.
I found the 1.1.572-alpha release today, and it appears to fix the issue.  However, management does not want to use an "alpha" release.  When will the alpha release become the latest stable one?


awesome


look again... there's a new commit


Yeah, I'm almost certain that there aren't any of the problems that this routine addresses left to fix. So remove those methods from Cleaners would be fine. I'll just go ahead and do that on master, making this commit irrelevant.


> > Hey, btw, I was thinking that something along the lines of the test project I pushed for the docstring PR (https://github.com/SylvainCorlay/pbtest) could be a good example for pybind11 in general.

> Yes, that is pretty cool -- thanks for putting together a nice example. I will try to integrate or at least reference that in some way.

Cool, I updated the example with a clear readme etc. If you create a github org, it would simplify adding this type of thing aside of the main project. What do you think?


## Task 1

Write a program that includes 5 different functions each of which demonstrates one of 5 **different** string methods. As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses 5 functions to show 5 string methods and functions correctly (2 pts total; 0.4 points each function)

## Task 2

Write a program that processess these data to split them by line ending, replaces the tab separations with commas, and prints out the result - line-by-line. As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly processes data - it prints the data line-by-line, reformatted from tab-delimited to comma-delimited (3 pts.)

## Task 3

Write a program that takes the given list (using argparse) as input, and passes the list to a function.  Within the function, use iteration to remove every even-numbered item from the list, and return only the list containing odd numbers to the `main` loop.  Print the list of odd numbers from the main loop.  As always, your program should contain a `main` loop and an `ifmain` statement.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program correctly removes even-numbered elements from the list using iteration, returns this list, and prints this list from the main loop (3 pts.)

## Task 4

Why do you see these different behaviors between `function1` and `function2`?  Answer the question for each function by copying the code below to `task5.py` within your github username directory and adding your explanation to each function as a **function description**.

- [ ] Correct answer for function 1, as a function description (2 pts)
- [ ] Correct answer for function 2, as a function description (2 pts)


my thoughts with this file was to remove the global vars completely eventually, and just have the set statements nicely commented. That way we don't leak globals temptingly, thus new config variables don't strictly need the globals, since they're never used anywhere.

That idea isn't anywhere near finalised though, just a thought.


3color is fine.  works in all 3 browser.  However, I noticed that in safari, the image's scrollbar are visible.  they are not in other browsers.


This is not an advertised part of the plugin


But `new Mocha` is not documented, and this change requires people to dig into the source rather than rely on the existing documentation.


double negative is hard to parse, do you mean it should only work if @@species is present?


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


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


Thanks.


@brunobasto is this something we're going to need to add to every form using a management bar?


i see it falls back to defaultConstructor unless @@species has been set on the constructor: https://tc39.github.io/ecma262/#sec-speciesconstructor

is that what you meant?


Atualizado *


这样更有爱，^_^
`$method = REQUEST_METHOD . (IS_AJAX ? '_AJAX' : '');`


郵便番号はデータベースには入れない。
入力すれば住所が簡単に出る機能とのこと。


Why not?


That's actually less generic, and backend specific


@nathansobo That is weird. Pretty sure it used to work. I'll take a look. `position: absolute;` would be a good workaround.


This commit should fix the issue you encountered though right? I am re-working the internals to better support similar cases, I can make this change as i do that. Unless core-js still gets borked when it checks this.


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


ok!


想明白了，你应该用start.sh，而不是修改这个脚本，我会回滚这个文件
参考 #2198


Yes, Avalon Miner 6.0 is hardcoded.
I don't have other devices except Avalon Miner.
Add a new device is not very hard actually.


@adamwight I did that, which introduces a merge conflict. Should be pretty easy for you to rebase it on master and resolve that, though.


If it's green, then I'm happy with the file changes, and for you to merge it @tmbgreaves . We should probably note that two different APIs appear to be supported, with the scipy recommended version using lower case + underscores (the great thing about standards is that there are so many to chose from). But, given we've been using the Scientific.IO.NetCDF versions, we may as well stay with them unless they get deprecated.


nead review


I'm wondering about binning this function entirely, and just replacing it with a getter for the backing field, made more obvious like `getForcedIdentificationFlag()` or something.

Identification status shouldn't need to be checked in a security-critical way in any more than one place (InternalPageBase or related). Yes, there are other areas that could benefit like stats pages, user management display, but surely if we name things sensibly we don't have to go passing helpers into dataobjects?

I'm feeling more and more that these objects are handling business logic, data storage, remote requests, validation, and that list is growing. I'd like to push these back to the single responsibility of database encapsulation. Security checks should be handled elsewhere. That can be a later refactor though.


Hey Drew,

No. Only to those incorrectly printing hidden inputs with the wrong value
for the p_p_lifecycle parameter. Probably, all those using renderUrlPrams
taglib.
On Feb 29, 2016 10:07 PM, "Drew Brokke" <notifications@github.com> wrote:

> @brunobasto <https://github.com/brunobasto> is this something we're going
> to need to add to every form using a management bar?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/brunobasto/liferay-portal/commit/d5debce8de946540fd15bb6e7bc0ce904bd0cc8c#commitcomment-16406702>
> .
>


OK


昨晚试了一下，对于VIP资源链接不可得。
但的确不报错了，可以运行。


We should probably get the real data type and test on that - I dunno what PDO returns here - rather than testing both types, which does feel messy.


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


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


You forgot to change these to v1_9_R1


OK，注意commit规范，注意review


need review


> Pretty sure it used to work

Yeah I saw it working in your animation, but it wasn't working for me locally. Perhaps Chrome changed something in their flexbox implementation. Or I'm confused.


This shouldn't be commented out!


For scrollability, pull that config function call out into a local variable :)


I don't want to mutate the parameters.


yes. Is there a better way?


        // Cantor Ordering:
        // Construct the k-th permutation with a list of n numbers
        // Idea: group all permutations according to their first number (so n groups, each of
        // (n - 1)! numbers), find the group where the k-th permutation belongs, remove the common
        // first number from the list and append it to the resulting string, and iteratively
        // construct the (((k - 1) % (n - 1)!) + 1)-th permutation with the remaining n-1 numbers


I do wonder if this belongs here, or if it should just be put in the maintenance script.

Is it related to verifying a user's identity? Sort-of, but not really.


note added


Oops. Committed under wrong account.


I believe I set it up so that the connection to the value changed signals
of the limit PVs are only set up if limit monitoring is enabled. If
monitorin is not enabled then the signals are never emitted. I was working
from the assumption that any interfacing class would assume no limits until
such time as the were informed otherwise. I wonder if it might be necessary
to set the local stored range in the ampvcontrol to DBL_MAX and DBL_MIN and
emit this on construction, until a connection us established with the range
PVs, at which point the known range is emitted.


Hey Jon,

Thanks for the comment. Can you open a PR with your changes? I'll make the suggested changes for booking 2 tickets as well.


It seems there is a bug with this commit. Please check this commit.

https://github.com/jollaman999/jolla-kernel_bullhead/commit/b83ad7b8ddb8e4369f03e6144528375f4787cf45


segfault here.
`bld->site_id `and `bld->location_id` seems to be not set.


In general I think we're trying to avoid `catch (...)`, since coroutines throw a non-`std::exception`to exit.  If my understanding is correct, then this particular usage would be okay if the conditional (`if (!useFlowV1Output)`) were removed.


closed #579


This is still an antipattern. Please talk to @shiboying about using a concern.


It's interesting that this has the class `nav-link` but it's not a link.


Maybe `user_name_nav_item` is a better name.


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


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


Maybe `log_out_nav_item` is a better name.


@mohangk I checked the headers and it is because defining _XOPEN_SOURCE to 500 removes the strdup definition from the header files on OS X. There should be warnings about implicit declaration etc. In my branch I have moved the feature macros into its own header so it can be centralized.


오호


Oh my code is definitely not good. I just removed the check for status
being Open and just attempted to book no matter what.  That allowed me to
book an event that I knew for sure was available because I checked in the
iphone app first.

I'm not really a developer.. just a product guy hacking a bit here and
there.

On Mon, Feb 29, 2016 at 8:17 PM, Charlie Ou Yang <notifications@github.com>
wrote:

> Hey Jon,
>
> Thanks for the comment. Can you open a PR with your changes? I'll make the
> suggested changes for booking 2 tickets as well.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/charlieouyang/Jukely-Checker/commit/403c4d6ff70d1e579a6d8cdaa1852fea34dfafb7#commitcomment-16406866>
> .
>


Yeah, eww... maybe it's worth ripping that out to a setting. Although, if we're only specifying one title, that should be the only element in that array, right? Could we `array_pop` or something to ignore the key on that one? Might make it less fragile, or it may make it more fragile if that ever returns two pages for some reason. Dunno.


I intended for the last sentence in the new docstring above to explain this, but I perhaps it is not very clear... How should I clarify the explanation?


yay!


If you are going to up to Linux 3.10.98 check this commit.
It seems there is a bug in 3.10.98.

https://github.com/jollaman999/jolla-kernel_bullhead/commit/b83ad7b8ddb8e4369f03e6144528375f4787cf45


Comment on code


you implemented all  of functions in the serverfacade? I still can not see them at all.


I'd be tempted to override `ApplicationBase::setupHelpers` and add this there. Remember to do a parent method call


@mohangk pull request updated with the latest changes.


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


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 165](https://teamcity.labkey.org/viewLog.html?buildId=306444&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14 (3 new), passed: 17 Build time: 01:12:20
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAntigenVariableSelector: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyLogGutterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisFilter: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyAxisSelectors: <no details avaliable>

##### there are 4 more failed tests, see build details
```


Commit back


Also, calls to these methods is always followed by `throw new PrestoException(...)`. It might be cleaner to move them into these methods and rename them to `checkOverflow` (similar to the one below)


Can you please explain why? Is the Shim no longer needed? Thanks.


nice


Chao, this commit wiped out all my changes.


那个issue是start.command没有加执行权限，貌似不是我这个commit造成的。如果这个commit出问题，也应该是在弹出的终端.app中报错


ok i had to delete my old project and clone a new one. i can see it now. I will finish my part before the mid-night.


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


this is good.

[Uploading 白屏.txt…]()
[Uploading gulp introduces.docx…]()


[Uploading gulp introduces.docx…]()


On  1/03/16 08:23 AM, Pascal Obry wrote:
> I did not kept a stack trace. I could probably have it again if I put
> back memcpy().

It'd be really good if you could provide a stack trace, that
way we could have a chance of seeing if (1) you solved it properly,
and (2) where that bug's friends are.

James C. McPherson
--
Solaris kernel software engineer, system admin and troubleshooter
               https://www.jmcpdotcom.com/blog
Find me on LinkedIn @ http://www.linkedin.com/in/jamescmcpherson


:smile:


Since the only reason to go inside this `if` is to provide `trace` statements, perhaps it would be smart to check if `j.trace` early in the nest of `if` conditions.


It would appear that this merge wound up making a bunch of our work disappear. For instance, compare The ChangeLog entries under C.1 from this commit I did today: https://rawgit.com/w3c/aria/2e0df8a9aca83a63a38fcd0ba66eb2b270ad3d5e/aria/aria.html#changelog. Note that the last entry says: "29-Feb-2016: Added statement that authors must ensure aria-errormessage content is not hidden and is included in a container that exposes the content to the user."

If we look at the same thing from your commit here: https://rawgit.com/w3c/aria/9b5bf797d7427c4881544c89630d98cb67c805cc/aria/aria.html#changelog, you'll see the last entry says: "12-Nov-2015: Changed the superclass of application from landmark to structure; removed the accessible name requirement from document."

Another interesting thing to try is `git format-patch HEAD^`. This normally results in a single patch because it's a single commit. But with this commit, I get 147 files. (In other words, it seems like your merge undid 147 commits worth of work.)

Lastly, if you look at the commit message, it seems like you had some conflict resolution to do.

Any reason why I should not revert this?


Inlining the variable may make this a bit easier to follow:

```java
BigInteger bigInteger = BigInteger.valueOf(bigint).multiply(BigInteger.valueOf(tenToScale))
```


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


this is good.


I just mean it's incorrect test by the spec. `Promise#then` should use `this.constructor[@@species]` for subclassing, if `@@species` missed - just `Promise`, not `this.constructor`.


I'd rename the `bigint` variable to `value`


The code below suggests that our mechanism driver is capable of supporting network types other than opflex. Would we expect DVS to be different (except that it probably can only do opflex and VLAN -- but point being that it can do multiple types)?


I asked the GNOME release folks directly; they said not to pay attention to the version numbers and instead to whether it's in the release.


It would appear that one area for improvement is to avoid setting environment variables in both init.cmd AND in BuildEnv.props.

I propose that we extend BuildEnv.props to dynamically generate init.cmd from the properties it knows about. We can then put all the rules for the environment variables in one place: BuildEvn.props.

setenv_base can then execute a MSBUILD process to create init.cmd before calling it.

We still need to resolve where we want the output to go when doing RTM/Release or Debug builds. But there will be only one place where this is determined.

As a first pass suggestion, Build.props could be extended with the following: (for testing it creates   Build_env.cmd)

```
  <!-- Inlined task to set/export Environment Variables -->
  <UsingTask TaskName="SetEnvironmentVariable" TaskFactory="CodeTaskFactory" AssemblyFile="C:\Program Files (x86)\MSBuild\14.0\Bin\Microsoft.Build.Tasks.Core.dll">
    <ParameterGroup>
      <Name ParameterType="System.String" Required="true" />
      <Value ParameterType="System.String" Required="true" />
    </ParameterGroup>
    <Task>
      <Code Type="Fragment" Language="cs">
        <![CDATA[System.Environment.SetEnvironmentVariable(Name, Value);]]>
      </Code>
    </Task>
  </UsingTask>

  <ItemGroup>
    <BuildEnvItem Include="REM This file is auto generated. Do not edit" />
    <BuildEnvItem Include="SET FLAVOR=$(FLAVOR)" />
    <BuildEnvItem Include="SET Configuration     =$(Configuration)"/>
    <BuildEnvItem Include="SET SPOCLIENT         =$(SPOCLIENT)"/>
    <BuildEnvItem Include="SET CLRROOT           =$(CLRROOT)"/>
    <BuildEnvItem Include="SET SPOROOT           =$(SPOROOT)"/>
    <BuildEnvItem Include="SET COMMON_BUILD_ROOT =$(COMMON_BUILD_ROOT)"/>
    <BuildEnvItem Include="SET BUILD_ROOT_BASE   =$(BUILD_ROOT_BASE)"/>
    <BuildEnvItem Include="SET BUILD_ROOT        =$(BUILD_ROOT)"/>
    <BuildEnvItem Include="SET BUILD_TREE        =$(BUILD_TREE)"/>
    <BuildEnvItem Include="SET BUILD_TREE_CLIENT =$(BUILD_TREE_CLIENT)"/>
    <BuildEnvItem Include="SET BUILD_TREE_SERVER =$(BUILD_TREE_SERVER)"/>
    <BuildEnvItem Include="SET BHL_EXE           =$(BHL_EXE)"/>
    <BuildEnvItem Include="SET MDP_EXE           =$(MDP_EXE)"/>
    <BuildEnvItem Include="SET NetMfTargetsBaseDir=$(NetMfTargetsBaseDir)"/>
    <BuildEnvItem Include="SET FLAVOR_DAT        =$(FLAVOR_DAT)"/>
    <BuildEnvItem Include="SET FLAVOR_WIN        =$(FLAVOR_WIN)"/>
    <BuildEnvItem Include="SET FLAVOR_MEMORY     =$(FLAVOR_MEMORY)"/>
    <BuildEnvItem Include="SET CLRLIB            =$(CLRLIB)"/>
    <BuildEnvItem Include="SET VisualStudioVersion=$(VisualStudioVersion)"/>
    <BuildEnvItem Include="SET VcInstallDir      =$(VcInstallDir)"/>
  </ItemGroup>

  <Target Name="DefineBuildEnvironmentVariables">
     <WriteLinesToFile File="Build_env.cmd" Lines="@(BuildEnvItem)" Overwrite="true" Encoding="Unicode"/>
 <Exec Command="Type Build_env.cmd" /> <!-- optional reminder of what we are building to where --->
  </Target>
```

The above can be changed to generate whatever init.cmd is required with the final environment variables for the build and both the cmd environment (copy etc) and the Build processes will be in sync.

setenv_base would need to be changed to

```
msbuild /t:DefineBuildEnvironmentVariables .\BuildEnv.props
CALL "Build_env.cmd"
```


Whaaaaat. okay.

Not sure how that works.


Yup, looks good so far, just some minor refactoring I think, and a lot of testing :grinning:

So, my main concern here is that Maggie etc are marked as identified by this, which is fine currently where  everyone authorised to add people is currently authorised. What happens when someone leaves? Will we still mark them as identified (or is this too much of an edge case to worry about?)

We also don't verify *who* added the user. I also wonder if this is too much of an edge case to worry about, as anyone unauthorized adding people will surely be reverted rapidly. It's possibly something to tell tool admins to double-check before approving.


I guess something weird happened to this package, since NPM says that latest is `0.4.2` and `0.4.1` is in `package.json` + badge says `0.4.1` as a result my build just broke with the following error:

```
 ERROR in (webpack)-hot-middleware/client.js
    Module not found: Error: Cannot resolve module 'ansi-to-html-umd' in /Users/Restuta/Projects/tmp/react-static-boilerplate/node_modules/webpack-hot-middleware
```


can you push your stuff again. i am having hard time to pull your updates.


win10 下会报异常 #2192
need to try。。。cache?
  http://stackoverflow.com/questions/32083200/winreg-enumkeykey-i-does-not-workng


ACK.


thank you


Yea PR to the fork sounds good, but I will can review it in late tomorrow.
As you decide, i'm open, we just should get it work.


Hey- approximately how fast is the phone spinning and what length of string do you recommend using? Wicked project btw!


Also, quite a few of my comments are just random thoughts, and not necessarily things to act on, feel free to call me an idiot if anything is a stupid idea. :grinning:


Fix #195


Nit: this should probably also skip over the default moz bookmarks. looks like this in my exports:
```
<dl><p>
        </p><dt><a href="https://www.mozilla.org/en-US/firefox/help/" add_date="1449262701" last_modified="1449262701" icon_uri="http://www.mozilla.org/2005/made-up-favicon/1-1449262701704" icon="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAABmJLR0QA/wD/AP+gvaeTAAAACXBIWXMAAAsTAAALEwEAmpwYAAAAB3RJTUUH3gwMDAsTBZbkNwAAAB1pVFh0Q29tbWVudAAAAAAAQ3JlYXRlZCB3aXRoIEdJTVBkLmUHAAABNElEQVQ4y8WSsU0DURBE3yyWIaAJaqAAN4DPSL6AlIACKIEOyJEgRsIgOOkiInJqgAKowNg7BHdn7MOksNl+zZ//dvbDf5cAiklp22BdVtXdeTEpDYDB9m1VzU6OJuVp2NdEQCaI96fH2YHG4+mDduKYNMYINTcjcGbXzQVDEAphG0k48zUsajIbnAiMIXThpW8EICE0RAK4dvoKg9NIcTiQ589otyHOZLnwqK5nLwBFUZ4igc3iM0d1ff8CMC6mZ6Ihiaqq3gi1aUAnArD00SW1fq5OLBg0ymYmSZsR2/t4e/rGyCLW0sbp3oq+yTYqVgytQWui2FS7XYF7GFprY921T4CNQt8zr47dNzCkIX7y/jBtH+v+RGMQrc828W8pApnZbmEVQp/Ae7BlOy2ttib81/UFc+WRWEbjckIAAAAASUVORK5CYII=">Help and Tutorials</a>
        </dt><dt><a href="https://www.mozilla.org/en-US/firefox/customize/" add_date="1449262701" last_modified="1449262701" icon_uri="http://www.mozilla.org/2005/made-up-favicon/2-1449262701706" icon="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAABmJLR0QA/wD/AP+gvaeTAAAACXBIWXMAAAsTAAALEwEAmpwYAAAAB3RJTUUH3gwMDAsTBZbkNwAAAB1pVFh0Q29tbWVudAAAAAAAQ3JlYXRlZCB3aXRoIEdJTVBkLmUHAAABNElEQVQ4y8WSsU0DURBE3yyWIaAJaqAAN4DPSL6AlIACKIEOyJEgRsIgOOkiInJqgAKowNg7BHdn7MOksNl+zZ//dvbDf5cAiklp22BdVtXdeTEpDYDB9m1VzU6OJuVp2NdEQCaI96fH2YHG4+mDduKYNMYINTcjcGbXzQVDEAphG0k48zUsajIbnAiMIXThpW8EICE0RAK4dvoKg9NIcTiQ589otyHOZLnwqK5nLwBFUZ4igc3iM0d1ff8CMC6mZ6Ihiaqq3gi1aUAnArD00SW1fq5OLBg0ymYmSZsR2/t4e/rGyCLW0sbp3oq+yTYqVgytQWui2FS7XYF7GFprY921T4CNQt8zr47dNzCkIX7y/jBtH+v+RGMQrc828W8pApnZbmEVQp/Ae7BlOy2ttib81/UFc+WRWEbjckIAAAAASUVORK5CYII=">Customize Firefox</a>
        </dt><dt><a href="https://www.mozilla.org/en-US/contribute/" add_date="1449262701" last_modified="1449262701" icon_uri="http://www.mozilla.org/2005/made-up-favicon/3-1449262701707" icon="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAABmJLR0QA/wD/AP+gvaeTAAAACXBIWXMAAAsTAAALEwEAmpwYAAAAB3RJTUUH3gwMDAsTBZbkNwAAAB1pVFh0Q29tbWVudAAAAAAAQ3JlYXRlZCB3aXRoIEdJTVBkLmUHAAABNElEQVQ4y8WSsU0DURBE3yyWIaAJaqAAN4DPSL6AlIACKIEOyJEgRsIgOOkiInJqgAKowNg7BHdn7MOksNl+zZ//dvbDf5cAiklp22BdVtXdeTEpDYDB9m1VzU6OJuVp2NdEQCaI96fH2YHG4+mDduKYNMYINTcjcGbXzQVDEAphG0k48zUsajIbnAiMIXThpW8EICE0RAK4dvoKg9NIcTiQ589otyHOZLnwqK5nLwBFUZ4igc3iM0d1ff8CMC6mZ6Ihiaqq3gi1aUAnArD00SW1fq5OLBg0ymYmSZsR2/t4e/rGyCLW0sbp3oq+yTYqVgytQWui2FS7XYF7GFprY921T4CNQt8zr47dNzCkIX7y/jBtH+v+RGMQrc828W8pApnZbmEVQp/Ae7BlOy2ttib81/UFc+WRWEbjckIAAAAASUVORK5CYII=">Get Involved</a>
        </dt><dt><a href="https://www.mozilla.org/en-US/about/" add_date="1449262701" last_modified="1449262701" icon_uri="http://www.mozilla.org/2005/made-up-favicon/4-1449262701708" icon="data:image/png;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQCAYAAAAf8/9hAAAABmJLR0QA/wD/AP+gvaeTAAAACXBIWXMAAAsTAAALEwEAmpwYAAAAB3RJTUUH3gwMDAsTBZbkNwAAAB1pVFh0Q29tbWVudAAAAAAAQ3JlYXRlZCB3aXRoIEdJTVBkLmUHAAABNElEQVQ4y8WSsU0DURBE3yyWIaAJaqAAN4DPSL6AlIACKIEOyJEgRsIgOOkiInJqgAKowNg7BHdn7MOksNl+zZ//dvbDf5cAiklp22BdVtXdeTEpDYDB9m1VzU6OJuVp2NdEQCaI96fH2YHG4+mDduKYNMYINTcjcGbXzQVDEAphG0k48zUsajIbnAiMIXThpW8EICE0RAK4dvoKg9NIcTiQ589otyHOZLnwqK5nLwBFUZ4igc3iM0d1ff8CMC6mZ6Ihiaqq3gi1aUAnArD00SW1fq5OLBg0ymYmSZsR2/t4e/rGyCLW0sbp3oq+yTYqVgytQWui2FS7XYF7GFprY921T4CNQt8zr47dNzCkIX7y/jBtH+v+RGMQrc828W8pApnZbmEVQp/Ae7BlOy2ttib81/UFc+WRWEbjckIAAAAASUVORK5CYII=">About Us</a>
    </dt></dl>
```


(Informal) review completed, and verified this Task functions properly on US-3 branch. Good use of Javadoc comments, but perhaps would be better if they prefaced every function instead of just a few.


我在另外一台mac上测试了一下更新到2.9.1，发现start.command的执行权限的确没了。但这个commit并没有动文件权限


hrumph... you are correct. This presents an issue as this code largely predates the existence of `@@species` so it used `.constructor`. Consumers of es6-promise and RSVP rely on .constructor being sufficient, so changing this (although spec correct) will be a frustrating upgrading for users. As I am unsure if it is possible to provide a warning to the users.

Ultimately this will require a major version bump, but i wonder if we can guide users to upgrade their subclasses correctly...


Do you want me to create an issue for that?


Let's bundle lat + long into a CLLocation. This will be useful in mapview.


It's worth noting that certain password characters won't make it into the environment variable.


I have reverted it.


i can eventually see your stuff. ^ ^


Not related to your change but this table atsdb_ao doesn't seem to be AO. Not sure what above tests were trying to test. But if your moved tests needs AO table it doesn't seem to be the case. Unless I am missing something.


Why do attractions belong to a Trip Event?


Lol.  I was happy it was finished.


See #80 for the cause of the problem you mentioned


zxzxzx


[nellie]: Failure encountered:

```
Failed to locate logs
```


Grant www-data permissions on new location, i.e.
`/var/storage`

`$ sudo chown -R www-data:www-data /var/storage`


zxzxzxzxzxxz


(sorry about my inactivity, I've been busy with life stuff.)

So if I'm seeing correctly, the NBT data is automatically saved by Forge, and doesn't manually have to be manually saved by calling the saving and loading methods?


It would probably be better to have both tests, rather than just one.


I'll find out in a little while. Need to do some other things. Yes, I'm flying relatively blind, just trying to make sure that code that looks like it might be important in one section is available in the other... In fact, there are some things actually that should just be inside this code, but that's for later things as it is further refined. One of my latest modifications should correct some of the reporting issues...


keystoneTest init


Why is this not allowed in general? I can see how this would not be valid for casting, say, `12.3` to `decimal(2,1)`, but it should be allowed for cases like `CAST(double '1.23' AS decimal(2,1))` (by rounding the last digit)


It doesn't appear that this new method is used.  There is also another way to force a config into `Env` (e.g., https://github.com/ripple/rippled/blob/develop/src/ripple/app/tests/TxQ_test.cpp#L83).  So I'm guessing this method should be removed.


I would bring the enum to the global level so other files can also assign votes to models.


s


@flashbackzoo Iterates twice over each field?


Don't abbreviate variable names. I'd call this `stringValue`, and maybe call the argument above just `value`.


good!


# This is a markdown file


I added the required lines to cache a inverse matrix calculation.


Maybe just do:

```java
BigDecimal decimal = new BigDecimal(value)
    .setScale((int) scale, ROUND_HALF_UP);
```


@flashbackzoo This should be `$form->ID()`, doesn't have anything to do with request parameters


Similar comments as above


`$form->FormMethod()`


[nellie]: Job completed successfully! (chrisroberts/nellie-test@d5e5dc4e2697d836667db8a76291f48e4392dc17)


Console:
```
XMLHttpRequest cannot load http://willmaryellowbikes.herokuapp.com/api/statuses. Response for preflight is invalid (redirect)
Submission error:
Response {_body: XMLHttpRequestProgressEvent, status: 200, statusText: "Ok", headers: Headers, type: 3…}
_body: XMLHttpRequestProgressEvent
headers: Headers
status: 200
statusText: "Ok"
type: 3
url: null
__proto__: Response
```
Network:
```
General:
- Request URL:http://willmaryellowbikes.herokuapp.com/api/statuses
- Request Method:OPTIONS
- Status Code:307 Internal Redirect
Response Headers:
- Access-Control-Allow-Credentials:true
- Access-Control-Allow-Origin:http://192.168.56.103:8100
- Location:https://willmaryellowbikes.herokuapp.com/api/statuses
- Non-Authoritative-Reason:Delegate
Request Headers:
- Provisional headers are shown
- Access-Control-Request-Headers:content-type
- Access-Control-Request-Method:POST
- Origin:http://192.168.56.103:8100
- Referer:http://192.168.56.103:8100/
- User-Agent:Mozilla/5.0 (Linux; Android 4.2.2; GT-I9505 Build/JDQ39) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/46.0.2490.76 Mobile Safari/537.36
```


Yes, I'll add it.


`$form->FormAction()`


About a year ago I first came into contact with... This new kid on the block...


Specialization would allow [...] _in the case of `T: Copy`_


Shouldn't we also have a note like "Manual implementors of Clone must be careful to uphold this."?


... from installations of all...


So, I think what's needed is clarification on why boot files need special handling in ignore(). I'll investigate this more tomorrow.


I think this section should probably also include a "Explicitly define the behaviour/semantics in this case" aspect. Rather than just "add X to the documentation". The documentation reflects the specification, but it probably shouldn't *be* the specification.


Eh, really? What overrides this style that you need this de facto hack?


awesome!


Revert! I did a fix for some dpub roles and got a merge conflict on a typo from a previous PR when I went to push, but not sure why that would have turned into a revert of so many commits. It should have only changed the dpub.html file as that was all I was editing.

From: Joanmarie [mailto:notifications@github.com]
Sent: February 29, 2016 8:27 PM
To: w3c/aria <aria@noreply.github.com>
Cc: Matt Garrish <matt.garrish@bell.net>
Subject: Re: [aria] Merge branch 'master' of https://github.com/w3c/aria (9b5bf79)

It would appear that this merge wound up making a bunch of our work disappear. For instance, compare The ChangeLog entries under C.1 from this commit I did today: https://rawgit.com/w3c/aria/2e0df8a9aca83a63a38fcd0ba66eb2b270ad3d5e/aria/aria.html#changelog. Note that the last entry says: "29-Feb-2016: Added statement that authors must ensure aria-errormessage content is not hidden and is included in a container that exposes the content to the user."

If we look at the same thing from your commit here: https://rawgit.com/w3c/aria/9b5bf797d7427c4881544c89630d98cb67c805cc/aria/aria.html#changelog, you'll see the last entry says: "12-Nov-2015: Changed the superclass of application from landmark to structure; removed the accessible name requirement from document."

Another interesting thing to try is git format-patch HEAD^. This normally results in a single patch because it's a single commit. But with this commit, I get 147 files. (In other words, it seems like your merge undid 147 commits worth of work.)

Lastly, if you look at the commit message, it seems like you had some conflict resolution to do.

Any reason why I should not revert this?

—
Reply to this email directly or view it on GitHub <https://github.com/w3c/aria/commit/9b5bf797d7427c4881544c89630d98cb67c805cc#commitcomment-16407020> .


@sckott it's kind of a pain to have this name collide with dplyr, do you think that could be solved by making it an S3 method? I hate not being able to attach both packages.


Thanks! Looks great and user friendly.


@giogonzo @FrancescoCioria  Take a look of this.


@rafaelfranca Yes, like to see 4.1.1 release, I tested and jquery v2.2.1 will resolve [chosen issues 2506](https://github.com/harvesthq/chosen/issues/2506) as well.


[R/corrplot.R:310:12:](https://github.com/taiyun/corrplot/blob/9720558399e61fd9e7c577555db73a504203d287/R/corrplot.R#L310) *warning:* **no visible global function definition for ‘corrMatOrder’**
```r
    ord <- corrMatOrder(corr, order = order, hclust.method = hclust.method)
           ^~~~~~~~~~~~
```


[R/corrplot.R:310:12:](https://github.com/taiyun/corrplot/blob/2514db912082416afefbbcb60bbbbd02f356c5b8/R/corrplot.R#L310) *warning:* **no visible global function definition for ‘corrMatOrder’**
```r
    ord <- corrMatOrder(corr, order = order, hclust.method = hclust.method)
           ^~~~~~~~~~~~
```


huh? how does this change matters? could also be a separate PR, no?


ban đầu anh cũng dùng cái này mà github nó không cho chạy plugin


Can you fix?


chose `or`/`and` just to follow what elasticsearch does.  maybe open an issue about this and we can think about changing


The checkers should be checked...


resolves #5


Oh yeah, this is one of the things we were working on in #660


Shall your previous commit and this one be a temporary hack to be replaced by a better solution in 3.5.0?
If not, I suggest 2 changes:

1. Do this only if the actual system supports utf8mb4, and
2. do this also for the case of "alter table ... MODIFY ..." (1 elseif above"), and so it could be made a small private function for that.

Shall I make a PR?

If so, it should be "tested" by code review from my point of view, everything else would require a book with Harry Potter dimensions to be written to explain how to test.


J'ai un problème à ce niveau je suis bloqué. J'arrive a me connecter avec join en fesant /join localhost <port>
Mais il faudrait pouvoir se connecter à un client particulier et je sais pas trop comment implementer cela? tu aurais une idée? ou tu serais comment le faire ? (je supose que oui )


@dydek seems you were copying from github
and copied with `+` symbols :)


Accidentally committed this small change to the stylesheet to the master branch. Resolved by merging master into development, and development back to snapshot version as of ec5ec7639219edff59a3a91f021bf07d6b6593d1


Cool, thanks for checking!


okay, rechecked, it's actually the #in? method which turns ranges into arrays, cover works fine.


have added the image files. and refactored the code to pick the image files from the projects root directory.

thank you for the guidance and advice sir.


@JoeZ99

Why have you done this? The lock file is the snapshot of which libraries work right together to run the app and makes bundler work more efficiently installing all the stuff.

http://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/

> When developing an app, check in your Gemfile.lock, since you will use the bundler tool across all machines, and the precision enforced by bundler is extremely desirable for applications.
> ...
> When a gem lists a dependency, it lists a particular name and a range of version numbers. Importantly, it does not care where the dependency comes from.
> ...
> As a result, the application author will want to specify the precise gem servers that the deployment environment should use.
> ...
> Now, the application developer must be able to demand that the gem comes from that particular source. As you can see, there are critical differences between the desires of the gem developer (which are focused around longevity and flexibility) and the app developer (which are focused around absolute guarantees that all of the code, including third-party code, remains precisely the same between environments).

From Bundler docs: http://bundler.io/v1.11/rationale.html

> This is important: the Gemfile.lock makes your application a single package of both your own code and the third-party code it ran the last time you know for sure that everything worked. Specifying exact versions of the third-party code you depend on in your Gemfile would not provide the same guarantee, because gems usually declare a range of versions for their dependencies.

I can see two reasons for removing the Gemfile.lock file:

- Differences between platforms: https://github.com/bundler/bundler/issues/635 (at the end of the issue someone says that Heroku and other main providers remove the Gemfile.lock if it was not generated in UNIX so it's like not having Gemfile.lock at all).
- Generating a different list of dependencies based on an external file, like databases.yml: http://stackoverflow.com/a/5360120/316414

I think this needs more discussion...

@peillis @ecoslado @serguitus, please, give your opinion.


Unfortunately the fixup commit caused a Rubocop rule to fail. However, I then realised that `1` is the default value for the line number, so I don't need to specify it at all. See subsequent [fixup commit][1].

[1]: https://github.com/alphagov/smart-answers/commit/f6af32b942cf8ecd90e4eb886b7cf4abf859f864


I just had run those tests locally, so there is no CI link.
Try running FastGroovyTestSuite from within Intellij with clean working copy.
If you get errors, check test run configuration, it should look like this:
<img width="1073" alt="screen shot 2016-03-01 at 13 01 04" src="https://cloud.githubusercontent.com/assets/5519549/13423733/b9959be6-dfad-11e5-8d24-c818383fca80.png">
Vm options:
```
-ea
-Xbootclasspath/p:../out/classes/production/boot
-XX:+HeapDumpOnOutOfMemoryError
-Xmx512m
-XX:MaxPermSize=320m
-Didea.system.path=../test-system
-Didea.config.path=../test-config
-Didea.test.group=ALL_EXCLUDE_DEFINED
```


Unit test is missing


Are you sure that TODO comment with Jira Task ID APPLINK-.... must present in open source repo?


should be this.props.js === 'babel' or you will pass for typescript


I could make a PR to your branch to correct it, so you can merge it without the crowd see and discuss the hack and ask how to test.


hmm. It seems you are correct.
I will update pull request.


I have this problem with curl, wget does not have this. Curl can not handle this certificate.

```bash
Step 10 : RUN set -x 	&& cd /opt 	&& curl -o sonarqube.zip https://sonarsource.bintray.com/Distribution/sonarqube/sonarqube-$SONAR_VERSION.zip -fSL 	&& curl -o sonarqube.zip.asc https://sonarsource.bintray.com/Distribution/sonarqube/sonarqube-$SONAR_VERSION.zip.asc -fSL 	&& gpg --verify sonarqube.zip.asc 	&& unzip sonarqube.zip 	&& mv sonarqube-$SONAR_VERSION sonarqube 	&& rm sonarqube.zip* 	&& rm -rf $SONARQUBE_HOME/bin/*
 ---> Running in df1f38222b72
+ cd /opt
+ curl -o sonarqube.zip https://sonarsource.bintray.com/Distribution/sonarqube/sonarqube-5.3.zip -fSL
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
curl: (60) SSL certificate problem: unable to get local issuer certificate
More details here: http://curl.haxx.se/docs/sslcerts.html

curl performs SSL certificate verification by default, using a "bundle"
 of Certificate Authority (CA) public keys (CA certs). If the default
 bundle file isn't adequate, you can specify an alternate file
 using the --cacert option.
If this HTTPS server uses a certificate signed by a CA represented in
 the bundle, the certificate verification probably failed due to a
 problem with the certificate (it might be expired, or the name might
 not match the domain name in the URL).
If you'd like to turn off curl's verification of the certificate, use
 the -k (or --insecure) option.
```


Please keep attention with current change.


@guozhangwang @enothereska (and perhaps @ijuma as FYI): I observed that this backport/merge actually causes the classpath additions for rocksdb to appear twice in `kafka-run-class.sh` (e.g. [view the full shell script as per this merge](https://github.com/confluentinc/kafka/blob/b16e3b30b2bfda6974f948d6e4bd44e0f0dc9351/bin/kafka-run-class.sh#L55-L63)).

Perhaps other files are affected, too?


No need to get cynical here. How is the SDRAM controller relevant here? If it fails to ack a transaction in time that by itself has no effect on experiments.

`RTIOTimeout` does not only occur with gateware bugs (and a SDRAM controller would be well within its rights to delay an ack). It occurs with WB devices that take an unexpectedly long time either because they are busy or because the RT output triggering the read is wrongly timed (that could be considered a known bug in the rtio gateware). rt2wb could return a magic number (like ttl_input() does) but since we a) do expect a response here and b) the signatures would get uglier c) the magic number would need similar handling upstream anyway, I opted not to.

What do you propose?


Put a PR into here and me you and Eli can talk through it on Skype. If I understand you then I think I agree with what you are saying :)


I solved it via exceptions, because I wanted to separate the happy flow from everything else (the exceptional cases).
Also, just because nobody is calling `bar` with `null` value NOW, doesn't mean that we should be sure. But that's another topic :)

Will try avoiding the catch here (it was ugly before, as logic was mixed with error handling, but will revert it).


remove if not used.


Add coment in code line


I didn't add P_UseLines handling of sector portals. I think it's safe that the user has to rise/lower to the level of the layer above/below before using any switches there.


@Patater please replace both XXX's by "FIXME: replace by enum error code"


ahh ok. It does access the properties we submit.
Are you sure there is not other mechanism for an addon to know its name?
@tamland how can an addon get to know its name?


Gebäude löschen klappt nicht mehr, mouseover ist draußen, Skalierung nicht korrekt. Aber Transarenz eingefügt.


I agree with @rodrigc that the test is no longer working with current jenkins behaviour, scheduleBuild2 is not returning null when builds are merged. If it is due to that HudsonTestCase did not get updated to reflect that or it is the dependecy to 1.609 that did this I can't tell.
I suggest to assert that all tests succeeded instead.


Does this look OK on high-DPI displays with other OS?


Also kindly poke @namreeb on the subject to ensure that the commit above correctly replace ACE queue with the std one (without changing what the original code was doing, ofc). And if he can give me/us an hint on why playerbots can still read OPCode in the packets queue after https://github.com/blueboy/portalclassic/commit/de8177c28f1271219281c475a440b6e455a30418 but can no longer write into the queue. This is above my head, to be honest. :(


Ah, sorry. So only keep the +1. :-)


I am having problems installing Ice Builder 4.2.0 in Visual Studio 2015 (Installation via Tools->Extensions and Updates). The error in the ActivityLog.xml is:
SetSite failed for package [Package][Object reference not set to an instance of an object.]:{   at IceBuilder.Package.GetAutoBuilding() in C:\Users\pepe\Documents\ZeroC\ice-builder-visualstudio\IceBuilder\Package.cs:line 177
   at IceBuilder.Package.Initialize() in C:\Users\pepe\Documents\ZeroC\ice-builder-visualstudio\IceBuilder\Package.cs:line 429
   at Microsoft.VisualStudio.Shell.Package.Microsoft.VisualStudio.Shell.Interop.IVsPackage.SetSite(IServiceProvider sp)


This looks hard-coded to me.


@miguno Yup that looks like a duplicate, do you want to do a PR?


Yes. The threshold is making the 'Label' `red` in UI to be spotted quickly.


書くのがメインではなく、英会話フレーズ帳を他人とシェアできるような感じです。
1.単語帳の和文(Question)をみて、
2.脳内で英文を浮かべて、
3.口でつぶやく
4.英文(Answer)をみて答え合わせ。

普通のフレーズ帳と違う点は、人のつくったフレーズを自分のフレーズ帳に入れられるところです。


`confluent.version` should be `2.1.0-alpha1`, too.


May be remove this log?


@rolandgeider thank you for testing the xmppbot with the current master

Can you please clarify the differences between your commit and the changes that were made in previous pull-requests, for instance

* https://github.com/certtools/intelmq/pull/197
* https://github.com/certtools/intelmq/pull/158

did you have a look at the comments from https://github.com/certtools/intelmq/pull/197 ?
For instance there was also some talk about the name ACDCXMPPBot
Maybe you can integrate the improvements, discussed in the comments?


Skærm 2 ikke 1


Deckの機能は、遊戯王とかのカードデッキを想像していただきたいです！
「オフィスでつかう英語」みたいにDeckに名前をつけて、
その中にいろんなフレーズをいれて保存します。


To here means regular PR to Joomla! staging?


That complicates the automatic tools a bit, but at least we know...


It's garbage...


Please keep attention with current change.


Thanks @jollaman999


+1 to @LuxoftAKutsan


See also:
https://connect.microsoft.com/VisualStudio/feedback/details/2369346
where it is reported that the issue is fixed for the next release.

I haven't confirmed if this patch fixes the issue yet (I'm about to), but this comment was added
[[
Adding '#pragma once' inside ssl.h can probably workaround the issue.
Another option is to remove '#include "ssl.h"' in 'srtp.h' ('srtp.h' is included by 'ssl.h').
]]

So there is some kind of circular dependency between these two include files.


If some device downstream is busy and times out, the WB target core should "ack" the transaction with e.g. a bus error. The timeout should be implemented in the target core; it must ack all WB transactions - if you connect a target core to a CPU and the core indefinitely delays acks, the CPU will freeze, and this is normal behavior.

What exactly do you mean by "the RT output triggering the read is wrongly timed"? I would detect those problems by raising a "busy" exception when the next RT output command comes and the current transaction hasn't been acked yet.


Redundant star (*).


this is cool.good job


Indention changed. Why?


I'd rather let @guozhangwang handle this.


Either dump struct or class with intelligence - I'm a little bit black/white at such decisions.
dump struct has already gone away by specifing it as a class and allowing inheritances.

For me DemuxStream is a importand class inside kodi wich should live by itself - It should be responsible for anything what will be done with it.

B.t.w: this class needs some love, there are a couple of different variable name conventions in a single calss....

But sure, its a decision you have to make. Let me know...


Hello David,
 $_GET is used to retrieve url parameters. I have written all the cron jobs
in same file and the specified cron job get activated depending upon the
parameter which we pass as part of url
So using  $_GET[<parametername>], I am calling specific cron jobs.  I used
switch statement to call a specific code for execution.

Thanks,
Pradip

On Sun, Feb 28

On Sun, Feb 28, 2016 at 2:46 AM, songfarm-david <notifications@github.com>
wrote:

> Where is $_GET['method'] originating from? What is calling cronjobs.php
> with ['method'] key?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/songfarm-david/Songfarm-Oct2015/commit/2926489a886c8a224413732d6329b9d33a0f5860#commitcomment-16372574>
> .
>


need to ud


Nothing seems to have changed except for indention.


I noticed one inconsistency, but it behaved the same way before, so the commit is good... https://trello.com/c/iaBxxHme/603-propagate-avocado-test-stdout-stderr-into-avocado-test-with-stdout-stderr-prefix


Indents..


Maybe outside the scope of this PR, but consider making the function return a `Blob` with the data.


Maybe outside the scope of this PR, but consider making `haveNode` return a `boost::optional<Blob>` which allows you to eliminate the second reference argument (`Blob& nodeData`)


Should/could this return a `javax.jms.ConnectionFactory`? I see that even `ZeromqConnectorService` returns an `ActiveMQConnectionFactory`. Or perhaps you envisage other non-JMS connection factories being returned?


@hectorj2f @htr Added `run` here, since it was used further down. Think it's been forgotten here.


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


I was not 100% sure about that either, so I searched for similar regulations in other sports. One example are the [chess regulations](https://www.fide.com/FIDE/handbook/Standards_of_Chess_Equipment_and_tournament_venue.pdf). They define the size of the tables as length and width.
But your suggestion is a great way to avoid that. Let's go for that!

From my point of view, the difference between "must" and "should" lies in the question if a competition can be held when the particular regulation is ignored. Although there are several drawbacks when smaller mats are used, I don't think a competition should be cancelled when there are only smaller mats available.


@hectorj2f @htr Added `latest` here to be consistent with the way the image is called above.


What I mean is that there are alter table modify and alter table change statements, which are equal except change needs 2 column names, old and new, and so can be used also to rename columns, while modify uses only 1 column name, but the rest is equal (except position of parameters after the column names being shited by 1 in the "change" statement.

The statement in old update sql causing the problem here used "change" where "modify" would have been enough, so you hacked the "change" case.

But if in future someone else adds a new update sql using "modify" for some column of these data types "...TEXT...", then there is not your hack. So we need same hack few lines above again, too. And so we can put it into a small private function.

That's all explanation, no need to skype. I think it's clear we should do what I suggested if we wanna keep this hack for a while.

But if before 3.5. we wanna have a better solution for the problem (what I don't assume is your intention, it seems you wanna release soon, but just in case), there is no need to make your hack work for both statements "change" and "modify" because it will be reaplced by the right way, anyway.

Got me?


Forgot to tell you that you won't need this. The JWT gem does this for us. When you decode the jwt token, it automatically checks if the expiration is past and raises a `JWT::ExpiredSignature`. This is then converted in AccessToken.parse_auth_header to our InvalidTokenError. Therefore, you can remove any checks you have for exp.


What happened here?


Indention.


@vilchik-elena Do we need to change that?


The reason why I thought we better not do it in public with a PR visible in the issue tracker was that we should not point everybody with his/her nose on this hack and so not cause unwanted discussions.


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


Function parameter may be `const` ?


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


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


The commands must work without sudo for ordinary user.


@atfornes This is the commit we were commenting.


@pynicolas I have moved JavaScriptCheckVerifier into main dir and it uses `assertThat`.


Please check all `Send` methods. Looks that most of them can have `const` parameter `app_id`


@gschlager Thanks for all the work! The importer worked really well!


Wow cool das ist gut zu wissen, ich bin leider nicht weiter gekommen mit
dem Debian und jetzt grad im Urlaub.
Funktioniert Ubuntu Mate denn makellos mit dem Raspi 2 ?
Vielen dank für die info

Manuel

2016-02-28 20:56 GMT+01:00 Thomas301263 <notifications@github.com>:

> Hallo Manuel,
> also Simon läuft jetzt auf meinem Raspi 2, allerdings nicht mit der dafür
> vorgesehenen Debian Software.
> Ich habe Ubuntu Mate 15.10 installiert und Simon dann so wie Du unter
> Desktop beschrieben hast installiert. Danach hat es etwa 6 Stunden gedauert
> bis er compiliert hat und dann hat es funktioniert.
> Gruß
> Thomas
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ManuGithubSteam/LinJarK/commit/21f9244272aec08aafc7612d7e04efb415f9637c#commitcomment-16378279>
> .
>


我是围观的


@thijstriemstra yep :smile: I'm going to keep them in `gh-pages` as well as synchronise from time to time.


Will do, just working out some issues that appeared when i used 3.4 under windows


gives this error:

```
tolkien:dancer2ajax jason$ perl testschema.pl
{UNKNOWN}: {UNKNOWN}: Can't locate dancer2ajax/Schema/Artist.pm in @INC (you may need to install the dancer2ajax::Schema::Artist module) (@INC contains: /Users/jason/projects/dancer2/dancer2ajax/lib/ /Users/jason/perl5/perlbrew/perls/perl-5.22.1/lib/site_perl/5.22.1/darwin-2level /Users/jason/perl5/perlbrew/perls/perl-5.22.1/lib/site_perl/5.22.1 /Users/jason/perl5/perlbrew/perls/perl-5.22.1/lib/5.22.1/darwin-2level /Users/jason/perl5/perlbrew/perls/perl-5.22.1/lib/5.22.1 .) at /Users/jason/perl5/perlbrew/perls/perl-5.22.1/lib/site_perl/5.22.1/Class/C3/Componentised.pm line 150. at /Users/jason/perl5/perlbrew/perls/perl-5.22.1/lib/site_perl/5.22.1/Class/C3/Componentised.pm line 155
Compilation failed in require at /Users/jason/perl5/perlbrew/perls/perl-5.22.1/lib/site_perl/5.22.1/Class/C3/Componentised.pm line 150. at /Users/jason/perl5/perlbrew/perls/perl-5.22.1/lib/site_perl/5.22.1/Class/C3/Componentised.pm line 155
Compilation failed in require at testschema.pl line 7.
BEGIN failed--compilation aborted at testschema.pl line 7.
tolkien:dancer2ajax jason$
```


keep to good work


> dump struct has already gone away by specifing it as a class and allowing inheritances.

I have not written this class but I do the transformation. Means it's not too late.

> For me DemuxStream is a importand class inside kodi wich should live by itself - It should be responsible for anything what will be done with it.

This is the perspective from VideoPlayer but for paplayer it's meaningless. Selection logic is a VideoPlayer only thing and should to be owned by VideoPlayer.


In general I think I'd prefer to assume the list input
isinstance is a fairly heavy query & if this isn't a list then we don't need the bulk lookup (.belongs is more expensive than ==)


Looks good. Thanks.


It doesn't appear we set RequestInitializer anywhere prior to this?


That is just in the compiler development environment, it is not part of
the compiler CLaSH users interact with.

On 03/01/2016 11:11 AM, Gabor Greif wrote:
> This looks hard-coded to me.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/clash-lang/clash-compiler/commit/7652df6c118b71307cebaa79f58d668129746a69#commitcomment-16413210>.
>


cc: @brandwe


@valorkin  yes I know but you have to have an extra declaration for wepack's require if you don't use node typings? I'd been having problems without it earlier. Do you have better solution for that ?


These lines are duplicated. DRY:
if request.get_vars["option"] == "manage_recipient" and (has_role("ALERT_EDITOR") or has_role("ALERT_APPROVER"):
...
else:
....

(Note that I put the real check first & only do the role checks as a belt&braces in case of URL hackery)


wroung


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


Exactly as-above


commen


I do it on end :( lol


& again...


Nice :)


@mattdrayer fyi


ok, I'll make an own branch for it


This is missing an "e" "getSeasonScenExceptions".


Hi Vojtech,
Please see my changes regarding rebranding from HP to HPE.
BTW I left some comments in places I'm not sure we need to change.
Please don't hesitate with any comments, questions, additional suggestions

Regards,
Anatoliy


latout


This throws an error after successful installation

```zsh
~/packages
❯ pip2 install git+https://github.com/bjpop/methpat.git --user
Collecting git+https://github.com/bjpop/methpat.git
  Cloning https://github.com/bjpop/methpat.git to /tmp/pip-x6OQ37-build
Installing collected packages: methpat
  Running setup.py install for methpat ... done
Successfully installed methpat

~/packages
❯ which methpat
$HOME/.local/bin/methpat

~/packages
❯ methpat -h
Traceback (most recent call last):
  File "$HOME/.local/bin/methpat", line 9, in <module>
    load_entry_point('methpat==2.0.0', 'console_scripts', 'methpat')()
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 549, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2542, in load_entry_point
    return ep.load()
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2202, in load
    return self.resolve()
  File "/usr/lib/python2.7/site-packages/pkg_resources/__init__.py", line 2208, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File $HOME/.local/lib/python2.7/site-packages/methpat/methpat.py", line 8, in <module>
    from visualise import make_html
  File "$HOME/.local/lib/python2.7/site-packages/methpat/visualise.py", line 6, in <module>
    from version import methpat_version
ImportError: No module named version
```

Just a quick fix would do..


@vilchik-elena Shouldn't we extract that class to a separate maven module? Like https://github.com/SonarSource/sonar-java/tree/master/java-checks-testkit


We only allow FTP for self-managed subscriptions? Why?
NB Why call FTP 'Sync'?
- this seems highly confusing to me...I like calling spades spades otherwise we get into all sorts of trouble down the line.


questionとdeckが多対多で、中間テーブルがdeck_questionってこと？

-user: ユーザー情報を保存
- question: 「こんにちは」みたいな日本語が一覧で保存
- answer: 「Nice to meet you」みたいな英語の回答と、どの質問に対するanswerかを判別するid、どのユーザーの回答か判別するidを保存
-deck: 「オフィス英語」「恋愛英語」みたいなカテゴリ分け
- deck_question: deck と questionの中間テーブル

ってことかな？

もしこれがあってたとすると
・deckのconceptカラムの使い道がわからん
・deckにuser_idって必要？(このdeckを作成したユーザーは誰か？　みたいなことかな)
・deck has_many :decks, through: :deck_questions はやっぱおかしいw

上記気になりました！


Hello
My name is Mantesh chingude, I am computer engineer ,here I am doing a project is RFID Based Library manage system.
 This project is generally used in library where the security is needed.
The project aim is to protect the book from thief in the library.


We can omit `udp.c` for testing purpose.


@esben why is this recipe DEPENDING on cross:binutils_2.25,machine:glibc_2.22,cross:chrpath_0.14,cross:gcc_5.3.0 ??


here also, no need to include `gnrc_udp`


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


**Tip** Whenever you want to change topic; do this command.
`@topic ~[Insert Your Topic Here]`


Versiondate should probably be 2016-03-02


You don't have to set it to 0.


This change isn't related to what we want to fix, if we wanted to remove trailing new lines from files we should do it as its own change.


can be omitted


can also be omitted


I think when I clicked Administration->Other->Calendar->Clear Smarty Cache then README file deleted

Smarty path is correct but I am testing with the fix you have provided above


Command line for testing:

```
./results_cmp.py ../gunrock-output/BFS_coAuthorsDBLP_Fri\ Feb\ 19\ 112332\ 2016.json ../gunrock-output/DOBFS_belgium_osm_Fri\ Feb\ 19\ 111038\ 2016.json ../gunrock-output/CC_delaunay_n13_Fri\ Feb\ 19\ 122207\ 2016.json
```

Expected Outputs:

```
Parameters: algorithm=BFS mark_predecessors=False dataset=coAuthorsDBLP beta=6 quick_mode=False alpha=6 undirected=True
Wed Dec 16 18:48:25 2015:	m_teps=476.434326172 elapsed=4.10413742065
Fri Feb 19 11:23:32 2016:	m_teps=449.413146973 elapsed=4.35090065002 (+6.01%)
Parameters: algorithm=DOBFS mark_predecessors=False dataset=belgium_osm beta=6 quick_mode=False alpha=6 undirected=False
Wed Dec 16 18:08:59 2015:	m_teps=0 elapsed=0.237023994327
Fri Feb 19 11:10:38 2016:	m_teps=0 elapsed=0.103950500488 (-56.14%)
Parameters: algorithm=CC mark_predecessors=False dataset=delaunay_n13 beta=6 quick_mode=False alpha=6 undirected=True
Wed Dec 16 20:20:00 2015:	m_teps=38.5032081604 elapsed=1.27506256104
Fri Feb 19 12:22:07 2016:	m_teps=38.9032974243 elapsed=1.26194953918 (-1.03%)

```


Nazwy bloków daje z "-" zamiast "_".


@pynicolas I faced a limitation: `ParsingErrorCheck` is used here https://github.com/SonarSource/sonar-javascript/blob/master/javascript-checks/src/main/java/org/sonar/javascript/checks/tests/TreeCheckTest.java#L47. Indeed, may be I should remove this dependency and move test classes into new module.


row.pr_filter.query can be brought into scope outside the loop


rpl can also be omitted


In fact the whole line can? (Doesn't vary by pe_id)


& thus this check can too


This can also be brought into scope outside the loop


Thanks, spinnelein. I'll see about including that in the rewrite.


Hi @Tusky, what if you have configured LOGIN_URL to contain a named url pattern? I've done this in a project of mine and this code currently breaks.


@pynicolas Do you have ideas how to do it?:) All we need is to find a way to test ParsingErrorCheck


first


See #1016


@alerac Hello, j'ai avancé un peu, toujours sur la branche `light``. Pour le masque de la recherche avancée, on le fait en ligne, et la syntaxe aussi.

Mais il faut qu'on enrichisse un peu la partie avec les opérateurs en plus, avec 2-3 exemples/exercices

Et qu'on évalue le temps...


Great thanks, it works on POE MACOX version!


Sorry, thought I'd checked that option.


This works but it'd be nice to harness react-router, since we're using it elsewhere. E.g. `browserHistory.push( panoptesReturnUrl)`.


Subjective: I'm not convinced about this indentation style, but I'm not sure anything I can suggest would be any better. And, in any case, it looks as if the intention is for all this logic to get extracted into calculator methods in the long run, so I don't mind it being like this for now.


Founf reason why your hack does not work yet, see [https://github.com/joomla/joomla-cms/issues/9258#issuecomment-190659029](https://github.com/joomla/joomla-cms/issues/9258#issuecomment-190659029)


Just noticed this while looking at some other code


Usage:

```
./result_cmp.py <Gunrock JSON file 1> [Gunrock JSON file 2]...
```
Command line for testing:

```
./results_cmp.py ../gunrock-output/BFS_coAuthorsDBLP_Fri\ Feb\ 19\ 112332\ 2016.json ../gunrock-output/DOBFS_belgium_osm_Fri\ Feb\ 19\ 111038\ 2016.json ../gunrock-output/CC_delaunay_n13_Fri\ Feb\ 19\ 122207\ 2016.json
```

Expected Outputs:

```
Parameters: algorithm=BFS mark_predecessors=False dataset=coAuthorsDBLP beta=6 quick_mode=False alpha=6 undirected=True
Wed Dec 16 18:48:25 2015:	m_teps=476.434326172 elapsed=4.10413742065
Fri Feb 19 11:23:32 2016:	m_teps=449.413146973 elapsed=4.35090065002 (+6.01%)
Parameters: algorithm=DOBFS mark_predecessors=False dataset=belgium_osm beta=6 quick_mode=False alpha=6 undirected=False
Wed Dec 16 18:08:59 2015:	m_teps=0 elapsed=0.237023994327
Fri Feb 19 11:10:38 2016:	m_teps=0 elapsed=0.103950500488 (-56.14%)
Parameters: algorithm=CC mark_predecessors=False dataset=delaunay_n13 beta=6 quick_mode=False alpha=6 undirected=True
Wed Dec 16 20:20:00 2015:	m_teps=38.5032081604 elapsed=1.27506256104
Fri Feb 19 12:22:07 2016:	m_teps=38.9032974243 elapsed=1.26194953918 (-1.03%)

```


Should we rather use the CaseExportDataSchema for all domains?


impossible learn skill through transfer skills


@Kozoriz You must first delete object then set pointer to null !


@sokra I think this should do the trick. You already had the `null` check, but in the wrong order. If we check against `null` first, it should handle the case where a default export is `null` properly.


Uh, I got confused with an other commit: This one here is just to encapsulate the logic of initialzing the codec of the DemuxStream. Determining the codec is not trivial and some members has to be filled.

I don't see a better place for this in DemuxStream itself, because its the only one who knows what members has to be filled and wich "kodi" values are valid.

In my case: instead wrapping codecname in the addon to kodi style I would like more like to pass my avc1.xyz fourcc codec and let DemuxStream handle the transformation to its own style....


Gonna have to reject.

This sentence doesn't make sense.

> A Puppylinux distribution can also be built and assembled using packages
and components from another Linux distribution called ~~the~~ in Puppy _"binary compatible"_
distribution.


hmm... I think that there should be an error returned. `Err(Error::custom("invalid call"))`.


hey hey


do we have an update guide...? I can't see it and don't remember it being written...


@vcunat: Indeed. Still, there exists nothing to do that at the moment, right ?


gfhgf


There are sites that reach the MEMORY limit (128M) when opening SPM. The page tree build up should not use that much memory.


@vilchik-elena Suggestion for `TreeCheckTest`:
````
      Rule ruleAnnotation = AnnotationUtils.getAnnotation(check, Rule.class);
      if (ruleAnnotation != null && "ParsingError".equals(ruleAnnotation.key())) {
        issues.add(new LineIssue(check, e.getLine(), e.getMessage()));
      }
```


Alphabetical order


This is NOT good idea to add both test library and real library of Application Manager. You can get an ambiguity.