Es wäre sinnvoller, nur einmal `magic="$(ubnt_get_mtd_part_magic)"` zu setzen und nicht zweimal ubnt_get_mtd_part_magic aufzurufen. Auf `cut` kann dann auch verzichtet werden, `${magic:0:2}` bzw. `${magic:3:1}` tun dasselbe.


это не ссылка


Ok, this commit broken compilation on Win:
`Error	6109	error RC2127: version WORDs separated by commas expected	E:\oregon\oregon_oryg\OregonCore\src\oregonrealm\OregonRealm.rc	55	1	oregon-realm`
`Error	6110	error RC2167: unrecognized VERSIONINFO field;  BEGIN or comma expected	E:\oregon\oregon_oryg\OregonCore\src\oregonrealm\OregonRealm.rc	55	1	oregon-realm`
`Error	6111	error RC2127: version WORDs separated by commas expected	E:\oregon\oregon_oryg\OregonCore\src\oregoncore\OregonCore.rc	55	1	oregon-core`
`Error	6112	error RC2167: unrecognized VERSIONINFO field;  BEGIN or comma expected	E:\oregon\oregon_oryg\OregonCore\src\oregoncore\OregonCore.rc	55	1	oregon-core`


This is of no importance it's the request_host, it's not actually making a request to it.


always give your variables clear and verbose names
`pre` and `cur` should be `previous` and `current`


`-= undefined` is `NaN`


Let's namespace this to affinity?

```
struct {
    int core;
} affinity;
```

This way we can throw other placement-related things in there (memory pinning might be useful or coarser-grained affinity...not in this patch, but just for the future).


closes #41


Die case-Blöcke sollten einen Default-Fall haben, damit bei unbekannter ID nicht nur noch "2" oder "5" in AR71XX_MODEL steht.


roy is behind me


fancy :)


Because this is a scheduled queue.  You Add to the end, but Remove needs to be conditional or you can race with other people adding the same key.

Controller adds image stream "foo" with uid "1".  Controller loop races against a deletion of "foo" with the recreation of "foo".  The value is "1" in the queue, and `Remove("foo", "1")` will only remove that entry, while `Add("foo", "2")` will purge "foo", "1".  It is ok to defer an item to be tested again, it is *not* ok to remove an item from the queue in a race.


The following doc(s) have been deployed to the [ttrowellwp](http://ttrowellwp-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/wordpress/configuring-wp-config-php](http://ttrowellwp-static-docs.pantheon.io/docs/articles/wordpress/configuring-wp-config-php)


Unnecessary variable @IPRIT


Do we want this operator in `dynd::bytes` or `dynd::string`? I actually don't know the answer, it's a question of what semantics we want.


Then I suppose we should just make `MAX_CATEGORY_LISTINGS_COUNT = 100`, no?

What was the reason you changed it back from `100`?


The `extra_type` is no longer necessary, feel free to delete.


alles ganz schlecht ^^ lösch mal die binaries. rm -r bin/*  und haue "bin/*" in .gitignore rein.


Oh, good point. My bad.


this was a test!!!!!!!!!!!!!!!!!!!!


It's a utility class.  Map is generally useful.  I'll change the comment.


Kernel looks good. Where we should put concat_one_string is a good question.

Some kernels, like this one, expose functions that are generally useful. Another example would be all our special mathematical functions. If that is the case, we typically expose a `dynd::` version so people can use them independently. I'm not sure if concat is a candidate for this or not, just putting it out there.


Fixed typo in question 1.2:
return "Buzz" if n divisible by *5*, not 3.


Hi Andreas.

Excuse me by the big delay processing your request.

The today's nightly version contains your contribution to the project that fix some translations for de_DE language.

Can I add to you as current translator for de_DE?

Thanks in advance by your work.

Josep.


The following doc(s) have been deployed to the [ciphantomj](http://ciphantomj-static-docs.pantheon.io/docs) Multidev environment:
- [circle.yml](https://github.com/pantheon-systems/documentation/commit/f5b4c2d32c04403a23857a7dc49ffef915f2d00e/circle.yml)


Yeah -- I don't feel strongly either.  My thinking was "concatenating bytes doesn't really have to assume anything about the content, so probably makes sense there".  (In contrast to the comparison operators which *do* belong in `string` IMHO).


Looks good. You can drop the `dynd::` part. Also, `make_type<dynd::string>()` and `type(string_id)` do the same thing. No preference here.


:+1 :ship :shiptit


This seems hackier than explicitly setting an environment variable


Yes I am. I checked them with Atom. same happened with me as well.


The "+" prefix is getting common coinage for event hooking interfaces such as above. See the Zone.js proposal here: https://github.com/angular/zone.js/

The advantage of the separation is that it is clear when not to invoke the TS language service's implementation. For example, to produce a completion list I don't need to invoke the `getCompletionsAtPosition` method. If we just had filters then it would be invoked then the result discarded. However, for the diagnostics message methods I do need it as I am augmenting the list provided by the underlying service, not replacing them. I can also imagine cases where I would want to augment the list provided by `getCompletionsAtPosition` if, for example, I wanted to add a code snippet like system and have the code snippets appear in the list. I wanted it to be clear when you were replacing the functionality or augmenting and/or filtering it. It is also clear when you can stop. The first plugin to return a result wins the override; all plugins are consulted for filters.

BTW, one nice thing of the above is that if there are no plugins the code is transparent, only methods that are hooked pay the price for the hooks. It also is nice in that there are no "processing plugins" flags since the plugins get a non-plugged version of the interface they can use that will not call them back.


Well, addStack 1. does not accept a parameter for the parent and 2. calls addStackToCategory itself.
Should I update the addStack method?


ähh... ne


Straightforward. Easy enough! :+1:


Just a little mistake, need to also rename view on line 16 to string_concatenation. What we do here is really declare a singleton struct of type `string_concatenation` and a global variable of the same type called `string_concatenation`. The name after the declaration refers to the variable `string_concatenation`, but the struct can be referred to by `string_concatenation`.

The explanation is complicated, but the user interface is easy.


I'm sure you'd catch this, but the function body shouldn't be there.


Refactored via https://github.com/daemianmack/simulant/commit/f5eb27df4781facde898d86ca7dfb38e63971991


This is going to alter how BOLA works and I have concerns about this.  I understand why we would care about loading from cache from something like ABR throughput rule but bufferlevel rule is just setting targets.  We should not add latency to these targets IMO.


Fixed via https://github.com/daemianmack/simulant/commit/f5eb27df4781facde898d86ca7dfb38e63971991


@bgerstle yeah seems like a bit of nit - from reading your comment, it doesn't seem like it was too confusing.

>  I recognize early returns as a very specific and expressive pattern (don't do any of the work below, saves rightward drift), which (IMO) shouldn't be conflated w/ conditional branching (if X do A, else if Y do B, etc.). in both of these cases, there's a very clear if/elseif/.../else "tree" which could be expressed, but it's a bit muddled by the early returns.

This sounds more like a style issue more than anything else. While I see your point, I personally don't see the need to have a rule that says:
> Consecutive if statements with returns must never be used with the same intent of if/else statements

…but I am more than willing to discuss at code review and update the style guide.


I believe this would be 'Creates a Cordova project'. Same for all synopsis statements


Update settings to get credentias from VCAP_SERVICES


This is caught by validation, so we don't need an error.


This is a comment


Would it make sense to add this to some of the other ports as a choice?


Why so many quotes?


I misread / understood your messages...  My bad


We can use `EXPECT_ARRAY_EQ` for testing. If you do that, you don't need to use `.as<T>`.


Typo nitpick `mut`, here and below.


AUTO-MESSAGE - JSHint detected errors in the following files: webtask.js


There are tests in several places, the integration tests here cover the majority of this, and we test import extensively in hack/test-cmd.


`return` is always better.

Either handle it from where it's being called or add a new function to Catalog.


DEMAIS


WTF did the lawyers say??


@sourrust I'm sorry to be the cause of code modification. I can't help but identify to the guy in this [XKCD](https://xkcd.com/1172/)… But thanks a lot, that's neat!

I will wait for the commit to appear in the next release before I push a new version of the add-on, but I will definitely upgrade to v9+. Thanks again for quick answers and reaction!


I've fixed this up to use cmake/modules/Toolchain-foo.cmake, wll clean that up and push


I misread / misunderstood your messages... My bad


It's not used, as it's never called anywhere.


thanks @bbest! I'll sync my notes as well. Should we delete the master branch and just work from the gh-pages branch?


Ping @insertinterestingnamehere and @mwiebe.


I don't really feel this complicates anything. I agree developers could figure it out, but I don't think it hurts to have.


Yeah -- I was puzzled by this, too.  I think it's due to my forgetting to rename the *instance name* of `view` to `string_concatenation` above.


Lets get rid of it, as there doesn't seem to be an actual use case for it, and use a single Gradle user home for all projects.


AUTO-MESSAGE - JSHint detected errors in the following files: invalid.js


The constructor would be `public Boulder(){}` instead. Its purpose is to initialize the object.


Why mo-static? I understand changing the file's name for consistency's sake, but I'm not sure we want to reference mo-static in the generated project's source code. Thoughts?


update for ubuntu in Usb


>  would it be possible instead to use something else as an indicator for whether or not mutableItems are fetched or not?

Sounds a lot like having 2 different things to keep in sync. I could switch to to using a BOOL - but that doesn't make sync easier.

We can talk over solutions if you like.


httpd error logs where spammed by:
PHP Warning:  Illegal string offset 'id' in /var/www/html/cacti/lib/html_tree.php on line 314, referer: http://xxx.xxx.xxx.xxx/cacti/graph_view.php


Nice!


Instead of
```
    x = Ctor(x);
    x.s = 1;
```
I prefer
```
    x = x.abs();
```

Which I think it was originally until I inadvisably decided to go looking for some cheap micro-optimisations.


I have tried instantiating the BaseHandler (subclass of IPythonHandler) but it needs to have an 'application' and a 'request' parameter.  I don't know what data type these objects are (the IPythonHandler API doesnt specify it either) and I can't do `app = application()`


good question… I actually have this down for a ticket to write up - aside from the nearby section, we have not defined what should happen to a section if the fetch fails (do we remove the section? do we continue to show placeholders? do we show an empty cell?)

In the current implementation, if you return NO, and there are no results after a fetch, then the section will have no cells. (placeholders are only shown when prior/during a fetch and with no error)


What was the motivation for this?


Good catch, that's a cut and paste error.


So glad that somebody's taken this and updated it!


Concatenating bytes sounds fine to me. I don't really have a strong opinion there.
Aren't our strings null terminated though? That would make it so that we'd want a string-specific implementation of concatenation.


I noticed this higher-up too - did you make it so the Django ORM uses UUIDs for db pks?  Or are you switching the backend ids to the Django standard incremental integer pks?


You're not testing the collisions between classes and protocols this code was meant to fix, right?


we not need to keep entry in order. LinkedHashMap has more overhead than HashMap


Missing a comma (and deps are out of order). Adding deps via `npm --save` or `npm --save-dev` will avoid this problem!


It was basically just "Why use a data structure that does extra work to preserve insertion order if you don't need it to". It probably won't be noticeable but there's a StreamSerializer for this so I think it makes sense to optimize.


Duplicate wat


Not sure I follow.  You mean the function body should be in `string.hpp`?


I'm not sure I see why one way is cleaner than the other. Seems like a bit of a nit.


Тогда тут ошибка:
```
Panels.h:599:60: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  if (!sets.RSSI_raw && rssi < sets.rssi_warn_level && rssi != -99 )
```
Еще странное место:
```
../libraries/SingleSerial/MyStream.h:55:41: warning: no return statement in function returning non-void [-Wreturn-type]
     size_t _write(uint8_t c){ write(c); }
```


`dynd::string` is actually not NULL-terminated, which is the same as `std::string` (also not NULL-terminated). So we're actually cool, though I know that is a bit unexpected depending on one's background.


:+1: +1 :ship :shiptit


It should be scheduledController, consistent with everything else.


So, funny thing, there is no "Track User" command anymore. It's being replaced by the [permission system](https://github.com/SO-Close-Vote-Reviewers/SOCVR-Chatbot/blob/v2-dev/docs/functional-specifications.md#permission-system).

Also, we need to check that the person is 3k before they can be added to the Reviews group.


No, scheduled implies it is recurring.  `Schedule` implies a schedule of when it happens, or a single instance.


b'fore ;)


We already have this temporary function [here](https://github.com/kubernetes/kubernetes/blob/master/pkg/kubelet/container/helpers.go#L103)


Missed debug statements..?


Looks like there is another typo


You have this function defined twice, right? Once in string_concat_kernel.cpp and once in string.cpp?


I don't think the current string implementation is null-terminated.  It's allocated for *exactly* the size of the string.  It's completely reasonable that it would become null-terminated, however (to shrink without reallocating etc.) so indeed that probably is still a good argument for a string-specific concatenation implementation.


Ah, okay, I see. You don't even need this file `string_concat_kernel.cpp`. The reason the compiler isn't complaining is probably because that file wasn't added to CMakeLists.txt.


finally!


Ah, indeed.  I committed this file by accident.  In this case, I don't need a `string_concat_kernel.cpp` at all, I supposed.


I took a closer look and the feature matching should probably not be removed from `feature_set.h/cc`. However, I would suggest implementing a new two-view matching as `CascadeHasing::twoview_matching(viewport_list, view_1_id, view_2_id, &matching_result)` as the matching is quite different from what is implemented now. Does this make sense?


Aha. Got it. Unless I am losing my mind, the old way of making GitHub-flavored markdown from `.Rmd` did not create these inline HTML comments after figures. But it really doesn't matter now that this is resolved in `knitr` itself. Thanks.


reduces `init` implementation to:

``` objc
- (instancetype)init {
  return [self initWithItems:@[]];
}
```

_obligatory rant about how Swift's default parameter values are so much better than manually-telescoping ObjC APIs_


@alxndrsn We upgraded pouch so we could override the default 30 second timeout. They took it one step further and let you disable the timeout with the "false" as above.


if this does not work, use bindsym $mod+f10 or something


haha i am not seeing another one.  This command works locally


Note: This is 'reverted' in a subsequent commit.


Thank you, always great looking code...


I think the issue is that `Requires.@init` is called in this function unnecessarily. It would only be necessary if you had used `@init ex` elsewhere in the package in order to define multiple initialisation routines (which would be stored in the array `__inits__` – see the [source](https://github.com/one-more-minute/Requires.jl/blob/master/src/init.jl)).


I changed the width from 100 to 145 to have the names like `Cache Directory:` on the same line and not two.
However i'm not sure if this is the best practice, so please review.


also, wouldn't the current implementation result in `mutableItems` containing both placeholder & regular items?  on top of that, is anyone actually calling `initWithItems:` directly?


integration: <- Shouldn't that be integration without the colon?


Ah, right. That would explain a few things. So how *do* we add new users to the DB?

Checking the user's rep should be easy, CE.Net's User class has a `Reputation` property.


the pk's will just be auto-incrementing ints, and for any new models i created in postgres i had them reference this new pk as a foreign key. but there are still some other messaging models which reference the couch_id so we're also going to keep the couch_id on the backends (populating new ones with a random uuid) for linkage with those models until they get moved over as well.


sure, and to be clear, it'll also be clearDebug too then


We could call [`ConvertPodStatusToRunningPod`](https://github.com/kubernetes/kubernetes/blob/master/pkg/kubelet/container/helpers.go#L103) here, just like what is done [here](https://github.com/kubernetes/kubernetes/blob/master/pkg/kubelet/dockertools/manager.go#L1841).


No. Wasn't my intention to fix that. Just saw it as an opportunity to simplify the code. I spent a few hours thinking that this system wasn't injecting as singletons until I realized what was going on.


agree without this param data become normal (instead of RZero 350-400 become 40-45 and PPM 1-5 to 300-600)


Should be able to replace lines 242:247 with:

self.role.activities = self.activities

See myjobs/tests/setup.py for details.

Idea being that activities are available but not assigned by default. Made sense for when I originally made these tests, but could probably be refactored into something less redundant, especially if we just remove activities from tests that are expecting there to be none.


this is amazing


Thanks @alexkogon

Can you add that `HUB_PORT_4444_TCP_ADDR` has to be a hostname: example entry: `hub.example.com` and may not be `http://hub.example.com` (must not be prepended with http:// protocol).

On the other hand the `REMOTE_HOST` has to be a full uri - example entry: `http://node.example.com:5555`


@bgerstle it doesn't return the key, only the value.


Thanks it was very useful!


No, since `timeout` is a key under bundle
See https://circleci.com/docs/configuration#modifiers


hmm, it should accept a parameter. Maybe you need to pull the latest?

https://github.com/concrete5/concrete5/blob/release/8.0.0/web/concrete/src/Page/Stack/Stack.php#L219


@mvo5 now that google code is officially dead, this is causing https://bugs.launchpad.net/snappy/+bug/1539218


Я откатил изменения rssi и showArrow (вернее во втором случае синхронизировал типы прототипа с типами метода) и сделал новый pull request


The following doc(s) have been deployed to the [dev](http://dev-static-docs.pantheon.io/docs) Multidev environment:
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/54153e6deeaafadcd2654802316c6c3bbe636be5/scripts/deploy-multidev.sh)


My first github project


would be :)


Liking the expressiveness of this.


true… currently there isn't a great way to tell if nothing actually changed because it rebuilds everything. If we change the model to only insert new things instead of rebuilding, then this all makes more sense.


Does the latest version of material-design-lite not offer a "main" in the package.json?
What was the reason for changing the import here?


I've not seen the "+" prefix getting usage (zones notwithstanding). Just asked in the team (in case I'm just out of touch), and folks also agree its not common and seems a bad idea - especially for a function name (e.g. how would you write an ES6 export of the form `export function <identifierName>` with such a name?). In general we should try to stick to [valid identifiers](https://rbuckton.github.io/grammarkdown/es6.html#prod-Identifier).


or we can make it make more sense i should say…


my name is the light bringer I was created by aliens who captured living light and but the light being in a human and alien hybrid body.


@Gert-Jan The Hg subrepo fun begins :P


How about a "filter" postfix?


Resolve this merge conflict and push back up


If so, what about send the event outside the `func()`.


Default value?


That is a bash4-ism:  maybe more portable would be:

```bash
    $ gcloud beta emulators datastore start 2>&1 > log.txt &
```

Then again, I'm not sure who might still be running bash < 4.


The HTML comments were introduced by knitr to prevent figure captions from being rendered when `fig_caption = FALSE`. Apparently I don't have to deal with the case of markdown output, so I turned it off in knitr.


i think your problem is the `break-word` as its not a valid keyword for firefox..
https://developer.mozilla.org/en-US/docs/Web/CSS/word-break

and for your reference: http://caniuse.com/#search=word-break


A question; why `return 1` rather than `exit(1)`?


ah, I see, this is more of a "bimap" operation (give me the key for this object). either way, my point was that for these kinds of "search" operations we should prefer the "passingTest" variety of methods since they're faster than enumeration. in this case: `keysOfEntriesPassingTest:`


If we want to be consistent with the way things are done in the kernel, we should add "&& !defined(USE_BOOST_FOR_CXX11)"


heatUp() + coolDown() = adjustHeat(float factor, boolean redstone)


It's a bit pick on a comment that isn't even in javadoc! But I guess we should be clear about the documentation somewhere else.


Except for the question about bash4, this commit LGTM


same as before: "&& !defined(USE_BOOST_FOR_CXX11)"  ?


Get from?


s/status/get status


Should be GOTO/STEAL/GIVE


That's why switched to word-wrap, where it is a valid keyword!


@Rio517 is this better?


> currently there isn't a great way to tell if nothing actually changed because it rebuilds everything.

i think it might actually be easier to tell _because_it rebuilds everything. comparing two arrays of `WMFExploreSection` objects should be :cake:. ideally we could know ahead of time if there's changes and bail out of the expensive construction logic, but for now we can at least compare the two arrays of sections to see if they're identical.


Okay, that's a surprise to me. Implementing it in bytes sounds great for now then. We can change that if we decide to go for null terminated strings later.


Is this case still handled? :)


It's habit. After doing it I noticed exit(1) is used, I should have followed the style. In practice shouldn't make a difference tho.


I'm not sure we want to retain `!isset($articleRegistered)` here.  The effect of registration-by-article is to perform an update if already registered.  Shouldn't the effect be the same here with registration-by-issue?

I don't know if this counts as buggy behaviour or just odd behaviour.


GOTO?


Here is an [updated commit](https://github.com/chuckjaz/TypeScript/commit/b4425ecd2b0eec401d22e0863112c8b0936ebf14).


Thanks for the patch!

Could you give me some short description hav NagVis is integrated and why the patch is needed?


I would like to give you all a big THANK YOU for the quick work you did to patch this issue.


Cool - I'm keen for this to be merged in! :+1:


Thanks for the tip!


Maybe you missed this one?
[](https://github.com/official-stockfish/Stockfish/commit/4abe333e1fdb2fbe09975690cdb1f52e3e4b835b)


eslint is duplicated


Weird indentation here. Double-indent is the recommendation.


Single quotes for the escaped strings are missing.


TinyPtrVector? Also, is this really something that happens often enough to be worth caching?


number these to be consistent with the other sections perhaps?


The `fakeRuntime.PodList = tt.podList` is not needed now.


Can you give this a different name from the Module one, to make it clear that it finds *all* visible selectors rather than just the ones in this DeclContext?


Number are for multiple steps, dashes are for notes


<<Hello world>>


For test/modules/cos.js, I'd prefer to change
```
function t(n, pr, rm, expected) {
    Decimal.precision = pr;
    Decimal.rounding = rm;
    T.assertEqual(expected, Decimal.cos(n).valueOf());
    T.assertEqual(expected, new Decimal(n).cos().valueOf());
}
```
to
```
function t(n, pr, rm, expected) {
    Decimal.precision = pr;
    Decimal.rounding = rm;
    var x = new Decimal(n);
    T.assertEqual(expected, x.cos().valueOf());
    T.assertEqual(expected, Decimal.cos(n).valueOf());
    T.assert(x.eq(n) || x.isNaN());
}
```
and keep the
`t('-14044.44444444444', 10, 4, '0.04549771706');` you added.


Yeap. I sure did. Thanks!


So easy to code. They fly exactly like snowballs.


That's a weird time, now? It's 1.13h...

I am pretty sure this should max out at 30min or so (`30 * USEC_PER_MINUTE`), i.e. something round and easy to read...


s/was/is?


Yes, I'm aware this is excessively picky, so feel free to disregard :) But since the comment was intended for the future maintainers, I thought it would help understand that the on the producer side this does not have any effect at all.


It's a little wired to pass in empty name and namespace, but it actually works well with fake runtime, haha


Fixed


consider using format_timespan here...


Looks pretty OK to me, now. @teg?


İTEM FRAME?


This isn't being cached across modules, which means we do the collect/filter over and over again. What's the performance of that like? Is it just not an issue because Clang already has lookup tables for this?


Is there any particular reason for this to be present here, and not in init.el?


added keydown event listeners to specific text boxes/removed from global scope-per greg


No. The filter.dirOnly is not filter.fileOnly.


The following doc(s) have been deployed to the [evalbuild](http://evalbuild-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/drupal](http://evalbuild-static-docs.pantheon.io/docs/articles/drupal)


I apologize for being too presumptuous and rewriting the introductory paragraph. In this update, I've reverted the introductory paragraph to its previous state, changed the file name, and uploaded the .pdf document into the repo instead of linking to an issue


Should we remove the `amp-user-notification-id` from this document. Users shouldn't be using it. They should be using  `data-consent-notification-id` attribute.


A comment would make that clear to subsequent developers! :)


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


I don't think we should be referring people to a personal github repo for additional information. @jonathansick should we be including those types of iPython notebooks directly in the DMTN repo?


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


site web completé


Not sure where i screwed up in my git flow... but i believe i correctly modified the event listener placement.  check my most current .js file commit and let me know if it looks good.


done and the iostream too


I tried using afterOpen to create a popup over certain images but that didn't work, so I added a afterMedia setting to be fired after the slide is created.


noooooooooooooooooooooooooooooooooooo :,(


Resolvers is a key, when you are assigning it with +=, you get a setting. Move this into the settings Seq, otherwise it is disregarded.
Its strange that this compiles how it is


Why not just use a different prefix in fixits.swift?


@lakemalcom Thanks for the hint.


Eu ia rectificar isso, mas foste tipo Lucky Luke mais rápido que a sombra lol


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


Thank-you for accepting me.


Yes I agree that this will not be possible until docker support authentication, but I think if you are specifying a config file, you should take this into account now.


Rather than introduce an extra class, how about just adding this as an extra `protected` constructor?


Cool! :sunglasses:


@mckll DWMR (Done with my review). Good work here. Press ahead!


Ань, прости. Я дурак. Тут явный тег img еще должен быть вложен. внутрь спана


Thanks.


Thanks for finding this out. Will be fixed in the next release. This is a very rare case that you don't have multiple and you have allow-dir and you drop a single directory with no files inside, You will get an error and no file will be uploaded but the behaviour is expected since there is no file dropped so nothing is there to be uploaded, but the error should not show up.


The last two assertions should not have the same object instances on left and right hand side.


Indeed. Fixed.


Done


This also fixes rdar://problem/16192964 (warn on unknown selectors).


move aoid to the corresponding emu-clause


Yay! :)


Yeah I think we should commit to using only pod UID or the metadata (name, namespace and uid)


I will add a comment.


МММММ, смотри, ты пишешь текст как нормальный человек: первая заглавная, потом до точки строчными буквами=)) И в h3 тоже=)


Gah. Thank you!


Geez, this got hairy


nitpick - one `s` in `disappear`


aoid to clause


I'm being pedantic here, but wouldn't it be better to have a "CXX_STD=c++-xyz" option? 2 arguments in favor:
- It allows to use the c++14 or the upcoming c++1z std easily
- I feel like this code assumes that the std used by the compiler is C++98. This is bound to change (gcc-6 will have c++14 as default std). Hence, we could want to set CXX_STD=c++98 in a near future.


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


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


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


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


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


`Math.pow` -> `**`


LGTM. I never really used bullet or MBTB so I must have forgotten to make the relevant changes here


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


I think that is how it should be. You should see what your question looks like just after you create it.


@nadahar I have mixed feelings about the DLNA standard. I think it's good to default to the standard in most cases, and certainly to offer the ability to use the standard when renderers support it, but in my experience it's rare that a device follows the standard. So sometimes there's a choice between sticking to a standard or offering users the best experience we can, if their device chose to break standards, and in those cases I will always choose the user experience.

Offering the fallback images is something I've seen other servers do, and we do to a degree with formats but not sizes, so that would be a good change to trial and see if it breaks thumbnails on any devices. If it only breaks on a few then we can enable the renderer setting by default but set it to false for those few, and if it breaks a lot we can make it disabled by default.

Translations are looking great. I'll look at doing a release later on. I think I'll wait to merge the prettifying because we need to limit the threads it can run at once to query the web. I'm getting significant computer lag while the queries are running. Before we were getting around it by only querying for some files but with these changes we're doing more and it will cause more slowdowns.


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


DId you add the shutdownOutput() to fix a problem?

Based on the documentation, I would except that the preceding socket.close() would close both the input and output sides of the socket.. Is this not the case in practice?

Thanks!


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


For now yes, but I think it doesn't matter and I didn't time anything with it yet, so .... I'm just thinking out loud


https://monosnap.com/file/5z6Ps7Sr3G88pX3JKZLgLZwwIW5N0w - features-name  и blue - одинаковые по функционалу и все отличие - это цвет. То есть это явно один один тег. Предлагаю остановиться на h2. Сделать общий класс и отдельно класс для смены цвета. ЧТо-то типа: features-name, а для белого два класса features-name и features-name--color


цифры в названиях классов - зло


Apologies for the typo in the commit message.


`Math.pow` -> `**` in the frontmatter. Can you also remove the `es5id` and change `es6id` to `es7id`?

These apply to all other files too.


It turns out that my code is not efficient.  Using three separate if statements in a program is very inefficient as compared to if-elif-and-else statements.  The example code that the professor went over at the beginning of the last lecture is more efficient.  If the number is divisible by 15, for example, there will only be one comparison executed in the professor's program.  However, my program will executed three comparisons no matter what the input number is.  This is not good!  Do you want me to fix my code and re-submit?

Note: The reason why you did not see FizzBuzz is because I broke them into two separate words and display them on separate lines. In other words, if the number is 15, instead of displaying them as FizzBuzz on the same line, my program displays Fizz on one line and then Buzz on another line.  I guess that I mis-read the instructions.


For the purpose of publishing the technote on Zenodo and aDS, merging the two repos would be ideal, yes. When referring to notebooks in the text you can refer to their rendering on either GitHub or the jupyter notebook viewer site.


pour les reflet sur naruto c'est ce commit :)


there isn't a tile for item frame.


А на 63 то же самое?


Okay, so I had a closer look at glide but was a bit disappointed since it seemingly couldn't just be installed with `go install` and required an additional environment variable to be set (I'd expect at least the latter to get fixed in Go 1.6). I want Logstash Filter Verifier to be easy to build so I just went with GPM instead. See https://github.com/magnusbaeck/logstash-filter-verifier/commit/ae5df410b46f41322f863412f236f41ab57b0279.


@i32petoj @ivanportillo
Thank you very much guys for your support. Without you this project could not arrive a good port.
### @i32petoj I love u :heart: xxx
### @ivanportillo jjijijij can you measure my pulse? because i feel im in love :new_moon_with_face:


Почему у `body` такое `line-height`?


А здесь такой?


Refer to the README.md.  It's a commandline tool.  There is no GUI (Graphical User Interface) with it.  That means that you'll need to run it from the command prompt (Start->Run->cmd.exe->[ENTER]) and then pass in the required parameters (like username, course URL, etc...) so that it knows what to download.


Why renaming patchset to patch? There are so generated for a reason.


Componentes sim será o mais  indicado


@rhatdan sure thing! I will


Ok, this was before I joined.
Thanks.


Тут можно созвон организовать, и прям при мне сделать


никаких заглавных букв в классах. Мы боремся за единообразие.  То есть классы все с маленькой, никаких цифр и транслита


Почему такой межстрочный интервал?


Yeah, indeed. :)


Is `synnonym` a synonym for `synonym`? (philosoraptor)


@joselito5550 Thank you joselito, this means a lot for me and our relationship. :monkey:
### Always in my code :heart: xxxxxx <33333


not sure of this one. It may generate spurious movap* why are unwanted. I think we can leave with it.


why the change in the casing? was that causing you any issues?


The parenthesis aren not necessaries, infact PMD prints violations because it can be just if (jiraBaseUrl != null && !jiraBaseUrl.isEmpty()), beside that everything works fine.


> from reading your comment, it doesn't seem like it was too confusing.

unfortunately, you don't get to decide how confusing something is for me :stuck_out_tongue_closed_eyes:

I'll add it to the etherpad for discussion on Friday


It's working @amp5208


It's working.


reabrimos la rama para mezclarla en filmoteca/filmoteca.git dentro de la rama master


I left them separate because I expect we'll be providing an affordance for property getters/setters in the future, and wanted to call that out specifically.


removes SSSectionedDataSource yet the file shrinks by ~150 lines... very nice.


Good point.


This was fixing a typo that I introduced in an earlier commit.


No. Just where I set it up. It'd make sense to include from init.el though. lemme update, a moment


@Aerendir If I use `$this->setVerbosityLevel('foobar');` in a test, this line fails:

> 1) Liip\FunctionalTestBundle\Tests\Command\CommandTest::testRunCommandVerbosityOutOfBound
> constant(): Couldn't find constant \Symfony\Component\Console\Output\StreamOutput::VERBOSITY_FOOBAR
>
> .../LiipFunctionalTestBundle/Test/WebTestCase.php:170
> .../LiipFunctionalTestBundle/Test/WebTestCase.php:136
> .../LiipFunctionalTestBundle/Tests/Command/CommandTest.php:197
> .../vendor/phpunit/phpunit/src/TextUI/Command.php:155
> .../phpunit/phpunit/src/TextUI/Command.php:106

The test works with `if (!defined($verbosity)) {` (found [here](http://stackoverflow.com/questions/24159178/check-if-a-class-constant-exists/24159225#24159225)). Is there a reason you didn't used it? Thanks.


I like mentioning ObjectiveC because it reiterates that this is an Objective-C runtime feature we're working with. "Foundation" is cross-platform, "ObjectiveC" is not.


I think I was doing this to get a handle on the deploy-commit to push it to s3 but ended up not having time to implement that. Will change this to what you suggested since it's much cleaner.


TeamCity Waterfall :: Build [Build 45](https://tc.demonwav.com/viewLog.html?buildId=234&buildTypeId=Waterfall_Build&guest=1) outcome was **FAILURE**
Summary: Tests passed: 46; exit code 7 (new) Build time: 00:00:42


Good idea.


"use spaces instead of tabs" is an old Boost requirement. Tabs have different width in different text editors, so if you're using a tab to format code, there is a high chance that formatting will look messed up in some other editor.


I agree that we could code-complete differently here. That should be a follow-on.


Deveria ser
'while(scanf("%s %d %f", modelo[i].nome, &modelo[i].quantidade, &modelo[i].preco)==3 **&& count < N_PECAS**)' ;)
Boa simplificação nesta parte btw :100:


The markdown doesn't render correctly when previewed. I think the '#'s may need to be escaped.


кнопки плюса и минуса должны быть вокруг инпута. типа так: div в нем label, потом  button input button


О! А тут картинка=)))


@Rio517 would you prefer to have this method called with `access_token` explicitly?

```
def authorize_and_update(access_token:, attributes:)
  if authorized?(access_token)
    update(attributes)
...
```


I was pretty happy when I found out cmake had a solution. Otherwise it does get hairy. When do you need to apply fPIC? What platforms? What compilers? With this cmake takes care of it.


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


mmm


Not quite enough room but I'll line up the quotes so it looks something like this:
```
    if [[ "$TRAVIS_BRANCH" != "master" ||
          "$TRAVIS_PULL_REQUEST" = "true" ]]; then
```


Ну я бы прям в h1 не стала, хотя бы  h3


ссылка, а внутри h1


thank yoU!


Your link has no text (therefore there's no link)


That's your ID mapping file. You might have to copy it over. It was definitely tested.


This rocks!! Thank you so much!


Heh, this TODO should be deleted.


Обе кнопки в одном блоке. ТО есть hotel-buttons и там без дивов кнопки


s/fields/headers


Modifications for lønn


Yeah, good point. We should allow '!'; test cases in a follow-up patch.


Any update to the 3mm version?


thanks so much


Oops, thanks.


@bgerstle do or should we have a test double-checking this?


Sure.


trivial, but you could also write
```python
if len(result) == 1:
    return result[0] == backend_id
```


Okay!


Changed it so that symbol table is argument to the script now.


Should `&7FFFFFFF` to clear the high bit in this case


is the word "field" meaningful to HTTP spec?  `IsHTTPHeaderName()` is shorter and clerer to me


Nice!


That's smarter


Ahh did not look at the following line


Me parece bien.


naruto a bien progressé, on voit enfin les truc au loins, par contre Outrun marche plus j'ai une erreur au moment d'afficher le Ingame :/
Par contre j'ai eu ces 2 fichier a un moment, vertex_shader_passthrough.cso et pixel_shader_passthrough.cso si je les supprime j'ai droit a un fatal erreur maintenant, c'est normal ?
Pour naruto tu pourrait ajouté le commit ou tu a fixer les reflet de ta branch tmp, je sais pas le quelle cetait par contre.
L'erreur de Outrun c'est
>{rsx::thread} Exception: HRESULT = 0x80070057
(in file Emu\RSX\D3D12\D3D12GSRender.cpp:553, in function D3D12GSRender::flip)
Please report this to the developers.


This is just *my* opinion but the `== true` can go.  If the old version explicitly wanted false that means the new version implicitly accepts anything as true, as is the Ruby way.


^_^ 哈哈 Thx~


Please update link in repository's description too.
![proposal-object-getownpropertydescriptors a plural proposal for object getownpropertydescriptor 2016-01-28 11-29-14](https://cloud.githubusercontent.com/assets/19714/12633158/617f76bc-c5b2-11e5-908f-e57058d47120.png)

=> https://tc39.github.io/proposal-object-getownpropertydescriptors/


ところで ambiwidth な文字は使わない方がいいのでは。


tks for change,    i think self.process.wait()  should be keep in engine_kill() for subprocess resources recovery


Oh, right, I see, then yes, I think this a good plan, pass the context styles as parameters, while still maintaining them for the optimisations.


:turtle:


Woops, added the spaces within the string in other file, forgot to add them here. Good catch btw! =)
Fixed by 93608af; the exception messages are defined as `internal readonly static` fields within the exception type.


Given my understanding of the instructions, I thought that would be ok, but I could be wrong.


Yeah, this can be a simple `JLOG`


github is good!


It is not a bug - a micro-optimization that I stumbled upon as I was doing this.


我是说游戏的客户端。。。就像我那个一样


au ok, je viens de comprendre, c'est la compilation qui créé ces 2 fichier .cso ^^


@albertywu i think you updated the wrong file. you need to update src/grid/index.js, then run `npm run build`


对了你是doge里的哪个啊


The ole switch the b sig and v sig trick.


I pondered making the scheme more robust (including a thread-specific stack of journals, so you could "push" a journal in the callpath you're interested in) but ultimately, I decided that what we needed at this point was a very simple facility: a way to access a `Journal` outside of the normal logging contexts.

The design presumes that `setDebugJournalSink` will only be called early and in a controlled context. As of right now, it's only ever called from `Main.cpp` before we construct the `Application`.


Oops...


还有这玩意到底要不要Gui


@tasleson

AFAIK, this patch does not impact other editor user, but it do save me some time on switch code style between GNU(storaged project), Linux (multipath project) and K&R (libstoragemgmt project) whenever I open a C file.


That's definitely the best 500th commit that CEmu could have had.


@Maxopoly What is left after this?


18行目でチェック済みの$operatorを使えば、isset($_POST['operator']))を省略できます。

`<?php if(isset($_POST['operator']))if($_POST['operator'] =="+"){echo 'selected';} ?>`
↓
`<?php if($operator =="+"){echo 'selected';} ?>`

また、今回のようにifの内容が1行の場合は中括弧{}を省略できますが、
読みにくくなったり、改変がしにくくなるので
省略しない方がいいとおもいます。（人により意見が分かれるところですが。）


Nope.  I was surprised too.  It took me a while of debugging my own stuff until I realized maybe the aggregator was wrong.  My immediate concern was getting myself a working `recformin`, but I should go back and add tests for this (and other) aggregators.


# Assignment 3 Grading Rubric (15 points total)

## Task Set 1

- [ ] Regex matches entire dinucleotide repeat (AT)5 and only the repeat (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different from above (1 pt)
- [ ] Regex matches entire dinucleotide repeat (AT)5, only the repeat, and is different both above (1 pt)
- [ ] Regex correctly matches all instances of `R` or `S` (1 pt)
- [ ] Regex (find and replace) matches all instances of `R` or `S` and replaces them with `N` (1 pt)

## Task Set 2

- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 4,3,2,1 (1 pt)
- [ ] Regex (find and replace) correctly reverses the order of the columns from 1,2,3,4 (as you see them) to 1,2,4,3 (1 pt)
- [ ] Regex (find and replace) correctly replaces commas with tabs (1 pt)
- [ ] Regex (find and replace) correctly gives us the horrible format requested (1 pt)

## Task Set 3

- [ ] Regex (find and replace) correctly parses out only the lane number from the header (1 pt)
- [ ] Regex (find and replace) correctly parses out only the flowcell ID from the header (1 pt)
- [ ] Regex (find and replace) correctly parses any flowcell ID from the header (1 pt)

## Task Set 4

- [ ] Regex (find and replace) correctly extracts only the serial number from the fastq sequence (1 pt)
- [ ] Regex (find and replace) correctly turns the fastq into a fasta having the requested format (2 pt)

## Task Set 5

- [ ] Regex (find and replace) correctly extracts the (AT)5 repeat (Regex bliss)


Hey @frantuma,
Is the intention to mark subresources with an empty `@Api` annotation?  `readable` here is true for subresources when `(api != null && !api.hidden() && isSubresource)`.  Doing so generates a `Tag` in the following `if (readable)` statement, which in turn, duplicates the function in swagger-ui.

Just wondering if this should skip this `if` entirely when `isSubresource` is true.


I mean we have a feature to set a specific return-path header, does that feature still exist?


Show us your big pussy @spark008 @pwnall @ushadow


この方法でも初回アクセス時のエラー表示を消せますが、
本来は「初回アクセス時は処理自体行わない」方がソースコードとしては適切かと思います。


Not sure if sorting is a serialization concern.  Is there a reason the sorting should be performed in plist rather than via the creation of the Hash?


For some reason it refused to upload until I put the newline.


Now add this folder back to the change list?


Parameters are always optional in JS... we should test if the parameter was passed and if it is an array.
That way we can be flexible and allow passing a single ShowHide object.


Thanks for the review! Addressing feedback in a follow-up commit.


nah, the chances of this actually causing a problem are next to 0 by my guess.  if anything, us removing include guards is actually trading safety for aesthetics (i.e. having include guards is safer in the case where you might `#include` the file, but that's very unlikely for us)


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


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


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


Sure.


Sensor 18 was chosen based on experimentation. I monitored all the temperature sensors and did some tracing to see when the thermal-engine offlined the cores.  Sensor 18 best approximated the sensor used to offline the cores by the thermal-engine on bullhead.  It's not based on core temperature, but seems to be the temperature inside the phone casing. From a quick look, the thermal engine works quite a bit differently on angler, so there is probably a better sensor choice.


Yep, looks good to me :+1:


Ovo je uradjeno na masteru.
Pogledaj video koji sam okacio u topicu za cas 13.


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


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


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


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


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


updated - don't know what i was thinking there


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


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


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


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


Yeah, that's reasonable. The alternative to caching is "walk everything", so yes, we should cache it. Maybe we can do so more lazily.


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


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


This should be:

```
field.Invalid(fldPath.Child("httpHeaders"), name, fmt.Sprintf("name must match %s", validation.HTTPHeaderFieldNameFmt)))
```


may i eventually help you?
i can add your Y530 device specs to kernel.


Is there a corresponding block of old view code that needs to get deleted?


Not exactly -- wait_timeout or max_allowed_packet are unique to whois-server, and removing them would result is weird behavior for their respective cases.

While some of these might indeed be surplus, they are there however for a good reason, and that is to help fine tuning. It never hurts to share these kind of configurations. Maybe update with the latest ones?


#

> [`_**U R gey m7 just kys kid**_`](

- url

)


Alterei ali umas coisinhas se não estiver certo podem repor.
Ando a ver se consigo voltar a isto com mais  força mas o horário ainda não me permite.


(nevermind - looks like that happens in a few commits)


### Fu


yes it gets deleted in a later commit


Yeah, sure.


I didn't test it, but I don't think this code takes different permissions into consideration.

What if a user gives read-write access to another user, but only read-access to a group, the other user is a member of.
Will the user the calendar is shared with end up with a read-only or a read-write calendar?


Мы используем lowerCamelCase для переменных, а не underscore_syntax


Clang already has lookup tables for this, and is caching them in the (shared) clang::Sema, so the iteration itself should be cheap.


Here is what I found:

* use in `vtkMRMLChestRTColorTableNode`:
 * https://github.com/acil-bwh/SlicerCIP/blob/7bfe68a2df0ecf070990169342aa77b294dc78eb/Loadable/RegionTypeDisplay/MRML/vtkMRMLChestRTColorTableNode.cxx#L277
 * This class should probably be updated to derive from `vtkMRMLColorTableNode` and we should change code [here](https://github.com/Slicer/Slicer/blob/3e6327814ae21ef1a8cb901734de3095e872a1fe/Libs/MRML/Core/vtkMRMLColorTableNode.cxx#L1410) to be `if (this->GetType() >= vtkMRMLColorTableNode::User)`.
* use in `vtkMRMLPETTumorSegmentationParametersNode`
 * https://github.com/QIICR/PETTumorSegmentation/blob/e4100d649937e0ea4294acdcc208212be7102c0b/PETTumorSegmentation/MRML/vtkMRMLPETTumorSegmentationParametersNode.cxx#L56-L61

Search was done using `grep -rnsI -E "MRML.+\:\:Reset\(" .`

Cc: @lassoan  @jonieva @fedorov @chribaue @PietroNardelli


This only fires $ionicSideMenuOpen and $ionicSideMenuClose when pressing the side-menu button (ion-nav-button). The same event should be triggered when the user slides-open the menu.


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


I feel so ashamed right now... :flushed:


Oops. J'ai modifié ce fichier sur cette branche. J'ai créé une nouvelle branche pour les autres modifications concernant la langue anglaise. (Désolé, ces quelques années passées au Royaume Uni ont fait de moi un Grammar Nazi en anglais ! ;))


Надо выставить флажок bad_bit (если не считался /), а в случае со считыванием double-ов он выставится автоматически, а не бросать исключение


When I did the split, I had issues with the check lines matching... let's see if I can eliminate the need for it.


Thanks for reviewing!


:x:


при отсутствии слэша надо проверить, что в потоке выставлен bad_bit, а при его сбросе и последующем считывании считается знаменатель (проверить, что "споткнулись" на слэше)


Just a spot to note things that would be handy:
 1. multiple sorts, so I can get "openshift deps that are newer"
 1.  ```bash
cd $GOPATH/src/<godep name>
git checkout master
git checkout -b openshift-level
git reset --hard <openshift-level>
```
   puts the godep into the right state for a save
 1. It's possible to look for `UPSTREAM: ` commits to things under the godep path to note whether we've patched them.
 1. special-case cadvisor to always take the kube level
 1. The transitive dependency check should be less transitive.  I only want separate bumps for things that openshift directly depends on.


@mjpost Did you encounter an issue with the utf8 symbol table here? fstprint seems to fail on some words from the symbol table with something like :

No mapping for integer .... in the symbol table ..


Looks good


Like +1 So much sweg


does this require any changes to the api server?


I should add that the integer and its corresponding word clearly exists and that this works with the ascii (romanized) symbol table.


d


Not that I remember, unfortunately. Have you set LC_ALL and LANG correctly?


good catch


How cool is that? Thanks so much! :smile:

Although you need to fix something. Markdown uses '#' to make things capital, so you need to fix that!

-Sam :smiley:


Will update. Thanks. (Still doesn't help non-bash users, but is overall more useful)


Both set to en_US.UTF-8


It's all over the RFCs, e.g. https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html
But I agree that the longer name doesn't buy anything in practice.


Wrong comment! Do'h


TeamCity Waterfall :: Build [Build 46](https://tc.demonwav.com/viewLog.html?buildId=236&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 46 Build time: 00:01:00


Bugged in singleplayer, I wait that they are fixed


I'd like to do that eventually.  It's not clear to me how many people are actually using this package, though, so it hasn't been a high priority for me.


You certainly do if you want to test its behavior and the behavior of the ctlmgr classes under these conditions, especially when crashing etc. That stuff is currently not covered by tests. Also this is just a handful of lines and reimplementing it without the ctlmgr infrastructure would be just as much code.


рядом с параметром обычно пишут название параметра, а не его тип. то есть правильно было бы field, и если очень хочется, то потом описание на русском, что это такая-то штука которая обьект(или на английском)


It might in special cases, so we'd prefer to keep it in a separate field unless inconvenient. Thank you.


I know this is the first test to run and so doing `clearRequire('../')` is not absolutely necessary, but it might not always be the first test and I don't think it'd hurt to add this. Just a thought.


Sorry, I didn't have that problem, and am not sure what it might be...


Por mim está tudo bem, temos é que ver se coencide com as restantes strings no settings dos outros devices para as retificações ficar uniforms  na rep. Aparece sempre que puderes toda a ajuda é bem vinda.

Cumps


(I'm mostly offline today until tonight, can take a look then or tomorrow if you email me details and paths)


Too many butt hurt people, huh?


można też było po prostu tak:
`return properties.toString();`
ale wtedy klucze nie byłyby posortowane (przynajmniej w przypadku toString() na HashMap)


@lassoan What do you think of this:
* keep the signature `virtual void vtkMRMLNode::Reset()`
* this implementation would simply call `this->Reset(NULL)`
* mark it has deprecated
* update code in `vtkMRMLScene` (see [here](https://github.com/Slicer/Slicer/blob/3e6327814ae21ef1a8cb901734de3095e872a1fe/Libs/MRML/Core/vtkMRMLScene.cxx#L373)) to also call `nodes[i]->Reset();`


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


Hi @Praylin! I would test the command without any options and compare the output with your current command. You were able to use tail in the previous file (2-lastlines) without any options!


Then I'm not sure if you need threads - you can run asyncio operations one by one in the test routine using ``loop.run_until_complete()`` while the other tasks can continue running.


```
ERROR: On worker 2:
function model not defined on process 2
 in error at ./error.jl:21
 in anonymous at serialize.jl:526
 in f at /home/users/elijah/.julia/v0.4/DCEMRI/src/fitting.jl:89
 in levenberg_marquardt at /home/users/elijah/.julia/v0.4/DCEMRI/src/fitting.jl:30
 in nlsfitworker at /home/users/elijah/.julia/v0.4/DCEMRI/src/fitting.jl:91
 in anonymous at multi.jl:1358
 in anonymous at multi.jl:904
 in run_work_thunk at multi.jl:645
 in run_work_thunk at multi.jl:654
 in anonymous at task.jl:58
 in remotecall_fetch at multi.jl:731
 in call_on_owner at multi.jl:777
 in fetch at multi.jl:795
 in nlsfit at /home/users/elijah/.julia/v0.4/DCEMRI/src/fitting.jl:119
```
#4ee7882


There is a asyncio-friendly RPC client you can use in the test routine to call into the controller. ``run_until_complete`` will return the return value of the coroutine you put into it.


the `try...finally` is only there so we set and reset `runningChecksEnabled` when there is an exception. Keep in mind that you can have an exception that is not related to `expensiveChecks`. Eg with the expression `foo().bar`, as we do not have any control with what is going on inside `foo()`


Did you try reruning cmake?


@srinia6 Looks good.  I'd suggest adding test cases to check failure handling (i.e. handling for the case where the waterline calls fail.


nice abstraction here!


@SubJunk I agree that there's no point in sticking to the standard for the standard's sake, but from what I've seen of the thumbnails in my "investigation" so far we're breaking it quite badly and I think it's very likely that we'd improve general "workability" by adhering to it. If it's needed to "hack" for certain renderers that's always possible, but the way we do it now is plain wrong. The reason is that we send thumbnails under specific profile names (e.g PNG_TN) that's not formattet as PNG_TN is defined. I doesn't come as a surprise to me that some renderers show that wrong, when that particular profile is defined to be exactly 160x160. There are other profiles that allow other resolutions that we don't offer, and er also offer the same thumbnail as _TN and _SM while they are defined as 160x160 and 320x320 respectively.

I haven't looked into the prettifying, but from what I understand it involves lookups in opensubtitles. A good idea would maybe be to use the same logic for that as I've made for looking up album art. My code is fully thread safe and make sure that any one search is only performed once. New threads trying to do a search that is already ongoing are placed in a queue and isn't released until the previous query has returned and stored it's results in the database or a timeout is reached. After the search in question has gotten it's "green light" if first checks the database, so that if the previous query found something it will fetch that directly. I've also made a "grace periode" of 24 hours for searches without hits, so it will only retry failed searches once every 24 hours (in case the database is updated).

I've noticed no "laggy" behaviour when this is running, and it's relatively easy to duplicate and modify the classes used to do other things. Try to add some large music folders to your shared folders and try to browse them and see if you can spot any effects on your computer.

That would also make the information available for more than just prettifying, we would get our own "movie database" containing the information we want about the shared media. The difference is still that prettifying needs to answer before listing an entry while album art can come later. That could be solved by sending updates to the renderers. You can change items after they are sent, so it would be possible to initially (if the movie in question isn't found in the database) just do "basic prettifying" and then update it once the results were in.


Thanks Todd, I did exactly the same workaround. I also tried out to install the gem from source locally and it picked up the right dependencies.


:fearful:


is this helpful to keep in?


Is there a reason behind moving these two imports?


++ again


LGTM :+1:


The simplest way to leave the GPS in sleep mode is to leave the Force ON pin High.
You can anyway use the serial message for the same purpose.

"Force-ON is an input signal that can be used to wake up the SE868-A from the sleep mode. It is internally pulled-up. It has active-low logic, i.e. the module wakes up when FORCE_ON is tied to ground. When inactive, it should be left open drain or open collector"


Markdown fixed; thanks!


For completeness, would be swell if someone could patch and try this on java6, but I *bet* it's fine.


Thanks @syohex, I think separating package makes sense, will work on it.


кнопка сабмита ничего не засабмитит если она вне тега форм


We don't doesn't mean we shouldn't :/


good point


Done


Probably `uuid.uuid4().int` will be more optimal


this is confusing. I know that `_` should say "skipped" local variable, but isn't this changing behavior then? Where is the difference to https://github.com/jhawthorn/solidus/blob/7871b70410370ee994e39d8fec192703dadf5ff6/core/app/models/spree/payment_method/store_credit.rb#L31?


Looks good


Thanks for that Andy :+1: :)


godoc then explaining would be good.


We were able to knit it, but for some reason the dplyr section was not actually spitting out any results. We changed the "surveys = " line to just refer to our data name instead of using read_csv because we were having issues telling it that the data had headers so the data was coming out as one big jumble of numbers.


would make sense to get credentials using QgsAuthManager? bee inspired by Postgres provider


After the new change, this will require an extra `INVOICE_PAYMENT_ERROR` event.


это забыл исправить просто


Confused about the comment regarding the blocking event?


ah cool sure
ill get on it


Might be nice to make a named parameter here so this is easier to translate, something like `"Create {gateway_type} Gateway"`, for example (out of context, it's hard to tell what this might mean).


et j'insiste. :P


I've been able to repro the broken behavior and have written a test case for it. Working on fixing it now. (Yes, there are some unit tests, specifically around loading feeds, moving and deleting them. Amazingly this hasn't been caught before in the test cases which *do* work.)


This is not the same: "not nil != not truthy", I don't know if this matters here, but this changes behavior.

`if !current_value.nil?` would be the same behavior.


Instead of printing this, raise an error; I keep ndio's stdout clear so that a user can pipe output directly to a file if they want.


Sure, no problem!


sounds good.


This causes a restart for me. Try with Ubuntu 14.04.3 LTS and wifi card Centrino Advanced-N 6205

[-] Channel hopping failed: command failed: Device or resource busy (-16)

network-manager stop/waiting
network-manager start/running, process 4562

[!] Closing

Without apply "lower function" it's ok!


Wouldn't it have been easier to simply remove the line 661 from the changelog? We haven't it running in production, yet, thus changing the freshly added part of the changelog should be fine. Nevertheless, I'm fine with the found solution ;)


use basic or pki credential widget to add user&pwd as in App Postgres Layer interface


I'm unsure why git was showing this as modified, but I'm not taking any chances. :disappointed:


Yes. Not sure it worths to be implemented. Anyway, I added the code to mirror SPR read/write on dmac. Dma tests are now ok on the testsuite ^^


yeah that's a good point, i think i'd prefer to address that in another pr though (those translations were already that way, i just updated the text a bit)


Bien vu pour le nom dans la méthode main.


her eyou can heavly inspired by the same code of Add Postgres Layer


The init script is hardcoded for a Vagrant box, with the '0.9.0-dist' extracted in '/home/vagrant/'.

Copy the brooklyn init script to `/etc/init.d/brooklyn` and make it executable.

You can test it with `sudo /etc/init.d/brooklyn start` (stop, restart).

Needs properly implemented, status, pulling config in from sysconfig, possibly taking advantage of the lsb functions etc.


NOTE: the police_blotter_pipeline.run() at the end of the script was just to make it easier to run this trough my IDE's debugger.  It does work using the click commands.


discussed here: https://github.com/supercollider/supercollider/commit/71a90695e14cc720ebce6f376bbcc632286b496e


Your explanation makes sense, but since you're using mutex in all 3 crucial methods (next, Add, Remove) their entire body is protected by it, isn't that sufficient in that case?


Attempt to fix issue #54.


Yeap - every time. I was few days off with commits so I pulled last and - boom. So I try every commit since that and find this to be faulty.


Fair statement, I'd still move it to be part of the else body, in that case.


it's not safe to add user/pwd in the project and dump it in log... now you can use QgsAuthManager


I've added this middleware to the "routeMiddleware" array in App/Http/Kernel, but it threw me extensions, this fixed the problem for me.
If it's that you meant and that I should have added it to global middleware array, then i find my way much more convenient this way (Hello route groupes). Plus, it doesn't break any compatibility with the "normal" way


What is this line here in this file doing? There seems to be no corresponding ending statement...


Fixes #3 as well?


This is the commit that causes flicker on my machine.


Each of these tests serves different purpose. I'd insist on having some bits of testing at this level, though.


Добавлена новая строка


Fixed. It was an issue with Python's `json.decode` turning integers into strings.


Too much swagness


May be you are right. But I'm prefer to save compatibility if possible (where not bacside effects or code worsening). At first to be compatible with large amount of a ready to use external libraries.


I agree, probably a more sophisticated approach is warranted, but I thought this might be a good fix in the meantime.  On further thought it has occurred to me that this patch probably won't work for clang++, so maybe some further work is required. But the idea of using a binary "switch" to turn on boost-replacement code doesn't seem bad to me, regardless of which C++xy version it ends up using.  Perhaps it should just be called USE_STD


Actually sorry for the pedantry but how about `filepath.Join(source, dockerfile)`

Would you also mind rebasing against master and I'll get this merged. Thanks!


What if you added this to your theme's functions.php file instead?

```php
/**
 * Filter Force Login to allow exceptions for specific URLs.
 *
 * @return array An array of URLs. Must be absolute.
 **/
function my_forcelogin_whitelist( $whitelist ) {
  if( '/wp-json/frm/forms/' == preg_replace('/\?.*/', '', $_SERVER['REQUEST_URI']) ) {
    $whitelist[] = site_url($_SERVER['REQUEST_URI']);
  }
  if ( '/wp-json/frm/entries/' == preg_replace('/\?.*/', '', $_SERVER['REQUEST_URI']) ) {
    $whitelist[] = site_url($_SERVER['REQUEST_URI']);
  }
  return $whitelist;
}
add_filter('v_forcelogin_whitelist', 'my_forcelogin_whitelist', 10, 1);`


This is a very basic implementation of Faraday's error raising. We can move to more descriptive cases once the remote API handles statuses other than 500 for anything that goes wrong.


Done at https://issues.openmrs.org/browse/TRUNK-4819


Indeed... will fix.


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `python`.* *Related to file `test.py`, line 2.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,2 +1,2 @@
 def main():
-        pass
+    pass
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `python`.* *Related to file `test.py`, line 4.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -2,5 +2,6 @@
 Some documentation
 """

+
 def main():
         pass
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `python`.* *Related to file `test.py`, line 6.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -3,4 +3,4 @@
 """

 def main():
-        pass
+    pass
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `python`.* *Related to file `test.py`, line 7.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -4,4 +4,4 @@

 def main():
-        pass
+    pass
```


Ok, i'll add it before the merge.


-l is not a cflag. It's a linker flag.


прости мну, я дурак.. я перепутала это с каталогом.. а давай быстренько сотрем тег figure и figcapture и просто картинку отдельно в td, а текст в другой td .. очень стыдно=(


I do not have a strong opinion on this, I just made a little patch to prevent an error.
Personally I would leave the API view available on any interface.
Another possible solution would be to raise an exception (ApiError?), with a meaningful message that explains that the API is not working on the Zope root
But, as I said, I still have to figure out myself what the best solution could be :)


True, would fit better there, the per factory check is unnecessary in that case


-Wl,-z,defs or -Wl,--no-undefined

Please write a test to make sure this works before using. These flags don't exist on OS X.


Yeah, adding it - also saw some leftover transport lock stuff which I'll rip out


Hmm, with this change in place, the completion ignores the last pressed letter. That's why I had changed it to `connect_key_release_event`.
What exactly did not work with `connect_key_release_event`?


If I do that I need to go back and modify all of these methods, i.e.

```lua
if string.find(qU, "^SHOW[\n ]") or
string.find(qU, "^DESCRIBE[\n ]") or
...
```
as well as deal with comments...
```sql
SELECT-- this is valid SQL92
FROM SOME TABLE
```

... which is fine, but likely merits further discussion. In fact, the newline is implementation specific according to sql-92, but in practice it's probably just \n in MySQL.


done


@PunKeel haha 🙃


я ее тут даже в спан забыла перевести и размеры - короче первый вариант)


Good decision.

I'm also certain a note about disabling vsync and tripple buffering should be added to https://github.com/vurtun/zahnrad/blob/master/demo/Readme.md and probably a new Readme.md with similar content in https://github.com/vurtun/zahnrad/tree/master/example should be introduced as well.


I'll revert this, I need to go ahead and add a JIRA for several issues in the proxy in any case.


Prosze podac moj adres: grzanka@agh.edu.pl


Czy mozna podac dwoch autorow ?


Interesting, no redefinition error, because of different return?


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


Can you explain me the codesniffer here in a little more details? I.e. talk me through the code. Maybe offline will be easier for that...


If it helps, we can infer that the Conjur server is 4.5 or earlier, since there's no info service


Public User Switches visible guidelines


Your welcome, my main goal was to prevent an error when the API was mistakenly queried with repeated parameters (e.g. API/route?a=yes&a=yes).
In that case get('a') was returning ['yes','yes'] and the .lower was failing with an attribute error.
that's why I had to add "if isinstance(value, list)".
Being lazy I wanted to add it just in one place :p
Next time It would be better if I file an issue before making a PR... explaining the patch later feels a bit strange :)


it's quite possible - at the time I was worried about removing the outer wakelock but when I tested it I couldn't repro it and so I didn't keep it - if I gave you a gerrit patch, do you know how to build and test it on your device?


Excellent idea! The rest of the code would be much simpler. I did not know there was such a method in the JDK.
The only problem I see is that toLongArray has been added in java 7. Using this method would break backward compatibility with java 6.


Could it be that you meant pos.game_phase() <= PHASE_ENDGAME?


The only way this would be different is if `current_value` was `false`. `current_value` can't be `false`, only `nil` or an `option_value`


это был result, но смысл поняла)


Also the name of the annotation could be discussed. As I was not sure whether I would have more time this evening, I wanted to have it in place nevertheless.


Are you sure the old code didn't work, and the new one does? I would think the new code would wrongly discard timezone offset, which might get the wrong day for 2002-02-02T23:00:00-01:00, which is actually the 3rd.

Looking at the old code, wasn't there an extra `)` which might cause the bug?

    bday.setTimezoneOffset((new Date()).getTimezoneOffset() * -10/3);
                                      ^

And I would expect it to actually throw a parser error...


I added this code to my fresh dev-next installation. The error doesn't show up anymore. There comes up a loading screen if you open a ticket and then it opens the ticket overview. Problem now is that the ticket isn't created. Can somebody comfirm that behavior?


I've a funny feeling there is a reason why we didn't do this, but I can't think of it


I forgot to mention in the push message that there was an addition of a carousel added to the home page. With some custom styling.


@thomasburke1 Probably a good idea to learn how to use the github interface, to speed the process up a little bit.


У меня здесь в строке 76 this.container = document.getElementById.
Я признаю недоработки, проверок практически не поставил. Есть и другие недостатки. Учусь пока, и время на выполнение ограничено. ДЗ - 3 ждет :). Но я многое исправил в коде, спасибо Вам за ревью. А выполнить проверки перед созданием объекта и отдать в объект чистые данные не проще?


TeamCity pull requests :: netty [Build 1136](https://garage.netty.io/teamcity/viewLog.html?buildId=5175&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6393, ignored: 23 Build time: 00:42:51


huh... The param "hidden" and the value "showOverlayButton" are counter-intuitive :)


These are all some kind of messages, remove the `Msg` suffix.


no need for curly braces here


Test comment


@emschwartz why is Babel necessary?


Should be `phenotips.patientSheet.measurements.age.placeholder`.

See http://dev.xwiki.org/xwiki/bin/view/Community/L10N+Conventions#HRulesforstandardformelements


Right, then keep it, enhances readability


@bocharsky-bw We *also* need to update the `metadata.yml` file on the main repositories to point to the challenge repositories. This is what that looks like :). I'm putting these on a different branch - I'll merge them all in on deploy


I guess this is going to hold permanent file handlers to the jars?

When I try to solve that for scalac I'll see if I can come up with a general solution, I suspect it's going to involve copying exported jars into a temporary location (md5 name or something)


Yet another comment, on the fourth line of text.


no backup needed


Hm.  This switch case duplicates information already encoded in
https://github.com/KhronosGroup/SPIRV-Tools/blob/master/source/operand.cpp#L947


Yeah sometimes I commit stuff, I know it's surprising when it happens :p


Great, very nice to hear :)

So I added an graph example for you! It uses a gpu array for the positions, which could be in theory written to from the GPU (with OpenCL, CUDA, OpenGL).
To get this running, you need to check out a few things.

```Julia
Pkg.clone("https://github.com/simondanisch/StructsOfArrays.jl") #my experimental fork, might change
Pkg.checkout("StructsOfArrays", "sd/fsa_structs")
Pkg.checkout("GLAbstraction", "sd/next²")
Pkg.checkout("GLVisualize", "next2")
Pkg.checkout("GLWindow", "sd/rebuild")
Pkg.checkout("GeometryTypes", "sd/boundingbox")
Pkg.checkout("FixedSizeArrays", "master")
```
I hope this is all, haven't tested it yet on another machine.
More later...


On the line above, are you adding a collection to itself?


Is Ruby required just for jekyll?


@mattbenjamin seems that we only take the lock if that's a leaf node


I don't think you can send the params from universal middleware.  The serializedApiMethods gets sent from a middleware parameter (`apiguard:SERIALIZED_ARRAY`) which can only be invoked from routeMiddleware.  Here's more info on it: https://mattstauffer.co/blog/passing-parameters-to-middleware-in-laravel-5.1

If your code, the options set in `protected $apiMethods` never get sent to the middleware.


Men sorry. Im not a developer.
I try to buil my own version of this whitout success (some missing libraries).
So i can´t build but if you pass me an apk i can test it


Derp, yep :P


why is this commented out?


good work


This duplicates data encoded in tables in spv_operand_desc.
See operand.cpp.
Should be doing a lookup in the AssemblyGrammar.


OMG THANK YOU SO MUCH! I was too lazy the first time around to do this proper :) Also, thank you for fixing the WriteLine on World Loading messages :)


No, `handle_action_call` calls with the same store credit that is passed in


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


there's no need to end those expressions with a final ;


got the whole redirect thing "working". still not sure how to send fields to the upload page to pre-populate information. The issue I ran into was posting to the upload page does not actually render it/redirect it. But redirecting to it like this does not post to it. I am not sure of an alternative solution to this.


How about using AdminConfig.list() with the server config id instead of just the node config id?:
```
def getObjectByNodeServerAndName( nodename, servername, typename, objectname ):
    """Get the config object ID of an object based on its node, server, type, and name"""
    m = "getObjectByNodeServerAndName:"
    #sop(m,"Entry. nodename=%s servername=%s typename=%s objectname=%s" % ( repr(nodename), repr(servername), repr(typename), repr(objectname), ))
    server_id = getServerId( nodename, servername )
    #sop(m,"server_id=%s" % ( repr(server_id), ))
    all = _splitlines(AdminConfig.list( typename, server_id ))
    result = None
    for obj in all:
        #sop(m,"obj=%s" % ( repr(obj), ))
        name = AdminConfig.showAttribute( obj, 'name' )
        if name == objectname:
            if result != None:
                raise "FOUND more than one %s with name %s" % ( typename, objectname )
            result = obj
    #sop(m,"Exit. result=%s" % ( repr(result), ))
    return result
```


Why not just call Context.getPatientService().voidAllergy(allergy, reason) ?


Wow, great work! I had no idea this defect was going to get that complicated :)


Any way you would be interested in making this work for version 9?


Can you add some tests for this?


Unfortunately not but I think you are on the right track with --type=soa.
I've figured out what my problem is, it's that my local DNS-server (BIND)  is both a resolver and an authorative nameserver. This is a common setup for small networks that use a "real" domain.

My domain is "gielen.name" and my local dns-server is "ns2.gielen.name".
This doesn't work:
...
$ nslookup -type=soa www.gielen.name ns2.gielen.name
Server:		ns2.gielen.name
Address:	2a02:898:62:f6::7e#53

www.gielen.name	canonical name = webserver.gielen.name.
...

Using an outside resolver does work:
...
$ nslookup -type=soa www.gielen.name 8.8.8.8
Server:		8.8.8.8
Address:	8.8.8.8#53

Non-authoritative answer:
www.gielen.name	canonical name = webserver.gielen.name.

Authoritative answers can be found from:
gielen.name
	origin = ns1.gielen.name
....

A work-around would be to make the resolver configurable:
```
nslookup -type=soa ${DOMAIN} ${RESOLVER}| grep origin | awk '{print $3}'
```
It's not perfect though, not all networks allow access to outside resolvers.

However I don like nslookup. It's not as reliable as dig. For example it fails for records that are not yet available:
...

$ nslookup -type=soa fail.gielen.name 8.8.8.8
Server:		8.8.8.8
Address:	8.8.8.8#53

** server can't find fail.gielen.name: NXDOMAIN
...

I've changed my version to:
...
dig +noall +authority -t SOA ${DOMAIN} | awk '{sub(/.$/, "", $5); print $5}' | head -1
...
That work's for everything I've tried. Does it work for you? If not, could you give me some examples of names that do not work?


Thanks, @gbinal! This has no effect on how the front end of Methods renders or functions, right? If not, we probably just need to give the milestones and team a quick once-over and then pull in.


thanks :blush:


Good catch, I should be adding allergiesList


I'm sure the old code didn't work, there was a two hour difference between the two dates.

There is no problem with that `)`, it's closing this one:
```
bday.setTimezoneOffset((new Date()).getTimezoneOffset() * -10/3);
                       ^          ^
```
The case you describe isn't possible in our UI, where we only pick date and not time. Assuming time is not set anywhere, it can safely be discarded.

The reason this problem exists is because the fuzzy date picker and DateTimePicker have different behaviour. Both generate ISO8601 strings to represent their values, but the first assumes the chosen date is in UTC time, while the second assumes it's in local time and generates an ISO8601 string in UTC, offset using the local timezone.

This solution works well in my testing, but now that you point it out, I see a potential case where it would fail: timezones greater than UTC. I will work on a solution...


so what ini have to edit?


Can you add some tests for this? Something similar to AllergyController1_8Test


It's mainly because we're a bunch of cheap frauds.
Deal with it chimp!


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


I have to check API, but it is possible that voidAllergy does not update the allergies list in patient.


This wouldn't work with nested resources, would it?


Can you add some tests for this method?


When is this fix going to be released? The last release 2.6.1 was before this fix (October 19, 2015).


Andre, I tried the link I still get the "Not found" issue.


gsdx.ini but you need first the new driver that contains the bug fix (which isn't released yet)


`wwtion2answer.org` huh?

Theoretically you can parse the requested URI with JavaScript, build a new URI for th subdomain and redirect to it.


I am coding the nested resource case at this very moment @ryanmortier


Do not assume. :)


Yep! Please do or if it's going to be a while, it'll be good if we can go ahead and merge and then make edits after.  :)


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


It's a bit late but better late than never. ```UIA_support``` is not PEP8 complaint name, even Git highlights is as a class name. A name starting with small cap would be better: ```is_UIA_support```, ``` uia_support```


fucking amd drivers...
:facepunch:


Seems like we are adding active flag to answers table here. Whereas in the migration above `active` is supposed to be added to `users`. Is there any relationship between the migration file above and this?


Yeah, I realized, sorry for that!  I just pushed another change...

Thanks, Andre.


I'm not sure about this one, I think we should stick with bash.
In some environments sh is a symlink to dash (Like Ubuntu) and they are not compatible.


good catch :+1:


Looks good to me.


Yes, it does. Originally I used an xmldb_table as the first parameter, but it threw an exception. I'll look into using recordsets.


lalal


Is there a performance benefit for using `find_by(id: payload["user"])` rather than `find(payload["user"])`?


How did I not know this existed?  We had to jump through a few unpleasant hoops in the ZFS code to emulate this exact functionality, for example `arc_tuning_update()`.  We should totally undo those hacks shift to this interface where appropriate.  That said, it looks like `module_param_cb()` is currently the preferred version of this so this obsolete version might be retired at any time.


16 different challenges*


Let me clarify that terrible commit message...

If you include a top-level file, e.g., `include native.py;`, this commit adds that file to py_modules in setup.py so the file gets packaged and installed.

if you include a package by including its component files (since we don't support including directories at the moment), e.g., `include foo/__init__.py; include foo/stuff.py; include foo/morestuff.py`, that information is not yet captured in setup.py.


var table


@rudhbaskaran basic weight noise during training without annealing


Is there a performance benefit for using `find_by(id: question_params[:id])` rather than `find(question_params[:id])`?


This looks good, please merge.


Please add license headers


@thet I have a forms application that uses dynamic schema (with plone.alterego, similar to what Dexterity does), but while Dexterity can address the schema by a stable FTI name (portal_type, IIRC) that can be serialized inside the saved plone.supermodel schema, my use case cannot have a __name__ set.  The reason is that setting the __name__ gets the name serialized into supermodel XML output, and my name/identifier happens to be the md5sum of that output, which means I have an impossibility on my hand:  I cannot have a name stored in a content-addressed/hash-addressed system and put the id/hash in the content being serialized.  More: https://github.com/zopefoundation/zope.interface/issues/31#issuecomment-176416738


چی کس نوشتی اشکان کمک کن نصبش کنم برنامه نویسی لمر کردی هیچیش معلوم نیست


This check should be reworked to use `time_after()/time_before()` to properly account for jiffie wrap.


@Berdir what's this?


Hi @wykapedia

Sure, can you file an issue in issues.jenkins-ci.org, with component=active-choices-plugin, please?


Do we really want to return the specific error to the consumer? Just thinking.


What's the `job_spec` for? Something doesn't seem right or am I missing something?

Just skimming the docs to see if quantum is a good fit for me...


Fix https://github.com/scls19fr/windrose/issues/14


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


not sure where this gets called from but may want to consider wrapping this in a `CriticalSection` to prevent race conditions from creating two of these


Typo 'agoritm`


I think this commit shouldn't have been in #5571, right?


This only happens when player using auto-clicker to increase interact speed.


I think we need to verify that the owner of the question is the person attempting to update it. A `user` object or id could be handy here.


Same comment as  I noted above.


This won't make much sense in the general case since all values outside the defined range will be ignored, you need normalize the value inside the defined priority range.


new line


I don't think you want to actually set this so it's persistent.  Or if you do, perhaps it should reset to 0 after the function is complete.  That way it's more of a status indicator that a kick is in progress.


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


nice, clear test


TeamCity Waterfall :: Build [Build 47](https://tc.demonwav.com/viewLog.html?buildId=237&buildTypeId=Waterfall_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 46 Build time: 00:01:00


You can probably drop this comment since it doesn't reference anything specific


Fixed also **exportAllUrls** due to changes in admin console styles.

**Tested:**
./gsa_admin.py --export_all_urls -n host.com -u xxxx -p xxxx --output="test.export.txt"


I am not convinced we should keep both events: In the past we did not have the `InvoicePayment` event so we needed   to send an `Invoice` event to reflect that balance may have changed. Now listening to `InvoicePayment` should suffice to get that information. I guess this is a semantics issue. What bothers me is to send 2 different events for the same action.


Is this supposed to be `page.Length`?


Generally looks great. Two small comments.


shouldn't we add a minimum time to wait
```
assert that time.Since(n) > 50 * time.Millisecond
```
and get rid of the test below?


The remove flag is not required when unprotect a channel... That was intended to leave it there....
1 min remember is I suggest for testing only....


**recrawlPattern Tested:**
./gsa_admin.py --recrawl --urlpattern="http://some.host.com/lang/" -n host.com -u xxxx -p xxxx

./gsa_admin.py --recrawl --collection="default_collection" --urlpattern="http://some.host.com/lang/" -n host.com -u xxxx -p xxxx

Fixed exportAllUrls due to changes in admin console styles.


The following doc(s) have been deployed to the [1275tokens](http://1275tokens-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/local/cli/machine-tokens](http://1275tokens-static-docs.pantheon.io/docs/articles/local/cli/machine-tokens)
- [source/docs/assets/images/machine-token-ready1.png](https://github.com/pantheon-systems/documentation/commit/1c3ccbe120645fe55bb8dba4de536338fd5fb418/source/docs/assets/images/machine-token-ready1.png)


There is a StandardValidators.ATTRIBUTE_KEY_VALIDATOR that should probably be used here.


That sounds like a good place to put them. I'll move them there.


We could create an automated test based on this dummy project at some point.


Okay i don't mind it if it's necessary for some biome systems.


That worked! :)


Nah let's keep it simple with a y value.


The following doc(s) have been deployed to the [1124envco](http://1124envco-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/wordpress/environment-specific-config-mu-plugin](http://1124envco-static-docs.pantheon.io/docs/articles/wordpress/environment-specific-config-mu-plugin)


Are you suggesting to remove the adjustment event? It's a bit weird in the case of refund with item adjustment, no?


Puh :)


I tried creating an account but it says I am a spambot.  Can you create the issue or instruct me on how to get an account?


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


so whats the difference between this and 2cho. Does 2choo not work?


:+1: these tests are really easy to follow


Just like with PutElasticsearch, does it make sense to have all of these catch exceptions when they do just about the same thing?


Great job @zenspider  - care for a new release? That would help us tremendously in [RubyCritic](https://github.com/whitesmith/rubycritic/issues/81) ;)


@ejeffrey , this test made looking at the code so simple.  Thank you.


sure - gimme a sec


CocoModel has a "getOwner" function. You could use that instead to check if the return value is me.id.


@bitHero : since you already forked and made the change, do you mind submitting a PR? Thanks!


looks good


Context.yield is for when the processor configuration is causing problems. Are all of these exceptions due to processor configured problems or are some due to the flowfile (EL index is wrong, EL type is wrong, EL doc id, etc).


Or we could extract it from accountRecordId from `InternalCallContext ` assuming it was correctly populated.


Since lava only occurs in large amounts in large caves we can use 'large cave depth' for 'lava max height'.


arquivos sem blocos de código fica ruim de visualizar... seria bom não aceita-los e mostrar para o aluno o [styling with markdown do github](https://guides.github.com/features/mastering-markdown/#examples) para que ele possa ver como faz para adicionar blocos de código no arquivo


It works on some.
>dig +noall +authority -t SOA example.com

seems to work on all older computers I tested, however recent ( CentOS 7.2.1511, and Ubuntu 15.10 ) both gave a null answer

Without the "+noall"  you get a result, however I suspect that the server isn't claiming it's authoritative, hence there is nothing in the "authoritative" section, and hence nothing is returned.


Sorry, good title is : “Add _(add|rm)Class for HTMLElement ”.


Oh, I didn't notice the obsolete comment. `module_param_cb()` doesn't exist until 2.6.36 though.


Good catch. That is an error in the documentation.
See https://github.com/c-rack/quantum-elixir/blob/master/test/quantum_test.exs#L26-L35 for a better example.
We will fix it in issue https://github.com/c-rack/quantum-elixir/issues/45


@ivan19871002 can you please  update the xiaomi-dev  repos when you have enough time  for it ?


I feel that letting go of Java 6 at this point is reasonable. We do not really support it anyhow (the travis tests start at Java 7). It seems that Google is adopting OpenJDK for Android so they will be Java 8 compatible for sure.

``toLongArray`` is very handy. For example, it could be used to turn a ``BitSet`` into an ``EWAHCompressedBitmap`` very quickly too!