I think **manyToOne** section should not be repeated twice in this case. As a result instead:

``` yaml
manyToOne:
    root:
      targetEntity: Entity\Category
      joinColumn:
        name: tree_root
        referencedColumnName: id
        onDelete: CASCADE
      gedmo:
        - treeRoot
  manyToOne:
    parent:
      targetEntity: Entity\Category
      inversedBy: children
      joinColumn:
        name: parent_id
        referencedColumnName: id
        onDelete: CASCADE
      gedmo:
        - treeParent
```

Should be
``` yaml
manyToOne:
    root:
      targetEntity: Entity\Category
      joinColumn:
        name: tree_root
        referencedColumnName: id
        onDelete: CASCADE
      gedmo:
        - treeRoot
    parent:
      targetEntity: Entity\Category
      inversedBy: children
      joinColumn:
        name: parent_id
        referencedColumnName: id
        onDelete: CASCADE
      gedmo:
        - treeParent
```


`@player_name` would work here too. I'm still confused though. This isn't a problem in my dice game. I wrote some basic tests to test instance variable access from within instance methods and they all seem to behave like this. i.e. require `self` or `@`


same with repeated section..


Done, for all region code comparison statements.


While the scales can maybe somehow be <= 0, it's very unlikely for the squares to be <= 0 :)


ok


Having issues with this commit. This will break the veh interaction menu.


lol


I disagree, the map() statements makes it absolutely trivial to add a new entry here.


I think you need at least 2 examples. Simple one and the one where credentials could be read (enriched) from env variables. Currently i use those:
```
const credentials = {
    client_id: "myClientId",
    client_secret: "myClientSecret",
    project_key": "myProjectKey"
}

const clientConfig = {
	config: credentials,
	user_agent: `${pkg.name}-${pkg.version}`
}

const sphereClient = new SphereClient(clientConfig)
```


That's LengthSq(), no?


Thanks boss. It was old code left in the core that I took out.


Well, *libblockdev* has this function: ``bd_lvm_cache_detach (vg_name, cached_lv, destroy)`` with ``destroy`` being ``TRUE`` or ``FALSE``. So it would be nice to have methods for both.


Hi TheGaABoO,

As you can see from [here](https://github.com/adobe/brackets/wiki/Extension-Locations), the extension I created will be in this folder **Mac: ~/Library/Application Support/Brackets/extensions/user**.
The file you need to edit is in the **template** folder.


Might be wise to un-abbreviate "analysis" too. I used to have a folder for analysis (abbreviated) on my desktop. Was nearly embarrassing when I went to give a talk on one occasion and realised what the audience might see/think! The code above was originally written before that realisation! ;-)


The current docstring doesn't indicate anything at all about the type of the returned value.


here is a typo


Alright. If, by chance, Blender falls back to 'ON' when 'ADAPTIVE' is not available, we could have a boolean (True -> 'ADAPTIVE' with possible fallback on 'ON'; False -> 'OFF')


#4954 is closed now


@TR4Android I did that, but it always returns null which indicates to use the default behavior. I need to get a hold of the view before the default behavior.


Why has the set of outputted names changed?
(Also, is `dump_module` actually "useful for debugging"? Maybe it can be dropped?)


done


@danielgindi could you review this real quick?
I'm not sure I did everything correctly since it's your implementation of the rotated labels.

I just separated the "spacing" from the actual calculation of the label width.


Are we checking customer-return-path from feature configuration?


actually it would make more sense to run "bundle update"


@hvelarde this way it worked!


@Jiri-Kremser this `javax.inject.Singleton` will be instantiated just once per JVM and will be shared among multiple threads. I suppose that you deliberately ignore that `this.md5` can be assigned more than once, because it should have no harmful effects. Anyway I vote for noting that in a short comment.


@Jiri-Kremser  `0.4.0` does not sound right


`lib.rs` LGTM


@Jiri-Kremser these commented lines can be removed?


Thanks for finishing up what I had forgot to do!


0.2.0 does not sound right


So it looks to me like the problem was that we were ignoring the third output from `extractBoundaryRoots()` in the original code when it could have been nonzero.  The new code doesn't attempt to extract roots from the right endpoint because it's not necessary.  That's probably a good thing, since `extractBoundaryRoots()` is a nasty operation.

The old code, however, would extract as many roots as possible from the left endpoint, while the new code only extracts one.  Is there any potential issue here?  Or is this something that will never happen?  Should the last line just be `g = extractBoundaryRoots(g)` with no second argument?


@vpodzime So *libblockdev* uses `--splitcache` when destroy == `FALSE`?


I reverted the changes with 090753281cb902459982e83b9592d8f970d9db84.
Current Channel.packageByFileName query already preferred the original channel. I just added the secondary sorting according to build_time in 07949c54d18abd4947a9019e0d86e559b14e8a66 like you suggested.

Thank you.


I will add it right now thank you.


Fixes #921.


Huh, this is a bit confusing for me. When would you write a final field in the target class (or better said, how can you even write to a final field in the target class?)


Tuong-Van Vu
t.v.vu@vu.nl
VU University Amsterdam


The hash is not part of the message, because then sites could say they signed one thing, but send us something else. So it has to be computed by the tray to verify the signed data is correct.

I would not add the flag to each message, rather as suggested earlier, as a variable sent through `qz.websocket.connect`. I don't think we should support this though, but maybe I don't understand the concern over the added dependency, it's small (<4kb) and it is overrideable by various node modules if needed.


@Jiri-Kremser do we really have to copy this from Inventory?


@dylanahsmith

Hi,
I wanted to know why rails scope support is removed from identity cache?
If I want to cache only some items say  only active items and not all items, how can I do that after this change?
Can you please help me for that?


Do you think the dots are a necessity here? Imho "26 Jan 2015" would look better than "26. Jan. 2015". On top of that, the second dot in "26. May. 2015" makes no sense anyway.


we should help the reader by being specific about what is wrong, I would say `'subject' field in token is mandary`


@gabrielamatias should the E on executive be capitalised?


@Jiri-Kremser ambiguity between what and what? Could you please merge this class level JavaDoc with the above `@author Jirka Kremser`?


@Jiri-Kremser What is UpdateCommandLiteral for?


Now I see that breakpoint not set on windows, but I think that is plugin issue. @dlsniper
![image](https://cloud.githubusercontent.com/assets/3740898/12617164/73e52da8-c51f-11e5-8f32-607fc4232263.png)


lol :)


@gabrielamatias do we need to specify UK here? In the past we've only specified the country when it's not the UK, so if we do it in this ad then we need to do it for enterprise AE and SDR too?


What is WithValues for?


v3 so far yes.
v4 on Android 4.1+


To add on to that, remember that your controller's purpose is to control your $scope. A lot of this other logic (which you are using these variables in) should be put in a factory.


agree


Does it make sense to add emptly `package-info.java` files?


Fair enough, keep it as is.  :)


What is AbstractHttpCommand for?


I mean we do have a `fillAlpha` property, it would be best if it could affect the drawable fill. It's not a must, just makes thing clearer and easier to use...


Why?
Does the order matter in any way?


Will this brake OS installations?


Only compiles when adding :
```__declspec(dllimport)``` to  https://github.com/amakaseev/sprite-sheet-packer/blob/master/SpriteSheetPacker/3rdparty/PVRTexTool/Include/PVRTString.h#L47

However it crashed immediately on execution.


I think the point is that an extra zero gets introduced at the left (by definition of `cumsum()`). This is the root we extract out.


I used a brute force fitting method to try to match the previous values.


What are the subclasses supposed to provide through `getSetters()` ?


Not sure, the order by statement looks like this:
```
ORDER BY CASE WHEN C.id = :channel_id THEN 1 ELSE 2 END, p.build_time
```

> order by 1

means order by the first column and

> order by 2

means order by the second column. Do you know that first and second column is ? Here in hibernate?


Java doesn't have native constants for charsets, and this avoids having to add an `UnsupportedEncodingException` catch block everywhere we specify an encoding.


First, everyone should open up their preferences in their text editor of choice and change the defaults for formatting to something that the team agrees on and matches general convention. You can find many online tutorials for this.

Second, there are plugins for most editors that will 'prettify' the code you wrote, based on preferences you can specify.


Looks like you removed them with this commit: https://github.com/pentaho/pentaho-platform-plugin-common-ui/commit/a1cbbdeff819a6068be29a3ee97e4f82efbf38b7


Sure thing.


You need to remove the `run_cmd` logic above


copy Windows_x86_64/PVRTexLib.dll to compile dest path

need fix
export_dll.files = $$PWD/Windows_x86_64/PVRTexLib.dll
DEPLOYMENT += export_dll

for copy dll


@gabrielamatias "close new business in an exciting, fast-paced environment"

just noticed the next line says growth in it already


Hey Matt. Everything is working correctly on your quiz, however there is a small section you may have overlooked. In the directions it specifies the following: Once the user enters in a number, and a character, the user can either then just press the enter key (as long as the cursor is in one of the input fields). Resubmit after writing the code that will check for the keyboard event.


Dup


Fixed #3.


@Jiri-Kremser s/than it's needed/than it neededs/ ?


I asked already, here: https://github.com/molinch/FFImageLoading/issues/130#issuecomment-174606585


In order to do this we have to be completely certain that all references to this buffer will be gone by the time the write is complete.  I think that may not be strictly true... for example,

- send write to osd.0
- crush layout changes
- send write to osd.1, write completes, Aio write goes away
- message finally gets around to writing message to osd.0 socket, referencing unallocated memory


Here you may choose to keep the `run_cmd` logic, but instead of `loader.remoter.run`, you must pick `run_cmd`.


@gabrielamatias "are are looking" (no 'we' beforehand)


This no longer is generated. (Moved to vendor sony kernel)
Therefore, BoardConfig.mk should have
--dt_dir $(OUT)/obj/KERNEL_OBJ/arch/arm/boot/dts
As kernel will not be built from source due to missing dtbs folder:
```
if vendor/sony/kernel is included in the android tree
we use prebuilt kernel, if folder is not prezent then
kernel is built from source
``


@gabrielamatias no comma between telecoms and accountants I don't think


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


Replaced useless info with readme


typo fix for text search: generate_reset_password_token


Again, some of the formatting in this file makes for a confusing read.


OK


Sample times from the AsyncRequestTest:

sync time to get 3 matches: 1398ms
async time to get 3 matches: 1038ms (limited to 1 async thread)
async time to get 3 matches: 487ms (no async thread limit (or limit >= 3, obviously))


Don't we have something along these lines in: https://github.com/pentaho/pentaho-platform-plugin-common-ui/blob/master/package-res/resources/web/pentaho/util/error.js? I think it is a good idea but I think this would be a separate story...


good catch


@Jiri-Kremser plz one sentence what is initialize doing?


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


That's a "maximum" that you are adding to, but you are trying to continue a running mean on it. Should be just Math.max...


scope support wasn't removed. I was just ignored before, which is more dangerous if the scope was to avoid a user from reading another users data


Should rename `xValAverageLength` for clarity :-)


worked :+1:


Yea, sorry, I didn't test -without- basic auth. It will be good not to patch this next time I upgrade though!


@Minecrell the contract of `final` on fields is only enforced by the compiler and by reflection, actual accesses in bytecode aren't validated and thus it's possible to mutate a final field's if you can generate the bytecode to do so.

In general, you cannot simply blithely remove a `final` annotation from a field without considering the repercussions (since `final` stipulates a contract of *write-once, read many*) as the original code may not expect the value of the field to change and thus may incorporate assumptions which are invalidated by removing the `final` modifier from the field. In other words, just because the general approach to using access transformers to remove `final` decoration from fields is pretty gung-ho, that doesn't mean it should be that way!

Quite often in mixin code, when adding an `@Shadow` field, the `final` modifier is removed from the field in order to avoid an unwanted field initialiser. However this has the side-effect of allowing code to be incorporated in the mixin which mutates the field, in breach of the `final` contract on the original field. This is undesirable because accesses to final fields should - in some way - be excplicitly declared (eg. via Access Transformer) and someone later modifying a mixin may not realise the underlying (shadowed) field is actually `final`.

The `@Final` annotation is thus designed to regain `final`-contract parity with the target class for `@Shadow` fields, enforcing a policy of adding access transformations for fields which need to be mutated, and preventing accidental write access to those which shouldn't be.

This was all hunky dory and @gabizou [added the decorations to all the relevant fields](https://github.com/SpongePowered/SpongeCommon/commit/61239c87285052a752027c339b6b794b39e3d8ad) and in the process discovered the fields which had been being mutated without being access transformed. However he discovered that some fields were being accessed which *cannot* have ATs applied because they are in the FML codebase which is exempted from ATs. Thus, with this annotation it is now possible to explicitly say *"this field is actually final, but we're choosing to mutate it anyway"*.


@kvz I used this to redirect traffic from port 80 to 8080 so we can do http://master.tus.io/files instead of http://master.tus.io:8080/files/ Is this a proper way to achieve this?


pour Outrun, comme je t'ai dit sur la PR j'ai trouvé la bonne valleur, et j'en est tester une autre et ca marchait pas du tout donc au moins on est sur que c'est la bonne ^^


[FindBugs](http://findbugs.sourceforge.net/) pointed to all the code that was altered in this commit. I was focusing on the "Using default encoding" issues that it found.


Interesting, I'll read this more carefully later (my C++ foo is not great, so it'll probably take a while to parse it).


:+1:  but mingw not linked this libs :(


Nice nice good work


OK, if we do that, maybe we should change the names again.

.value could be any type, but then instead of base_value and base_unit,
we could use num_value and num_unit.  That way we know that those
pertain only to numerical values.

Scott

On 01/26/2016 10:04 PM, Jing Luo wrote:
> I think thant makes more sense. Sine it is defined as a pure number. If
> .value is not a number it coulee be None or NAN. I believe we are not going
> to update this type of .value from base_value.
>
> -Luo Jing
>
> On Tuesday, January 26, 2016, Scott Ransom <notifications@github.com> wrote:
>
>  > So are you saying that .value could be a string but .base_value can only
>  > be None or a number? That might work....
>  >
>  > —
>  > Reply to this email directly or view it on GitHub
>  >
> <https://github.com/luojing1211/PINT/commit/5ca25e27a72cc1dbcb472c5b4ba13f0849f30ec1#commitcomment-15695302>
>  > .
>  >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/luojing1211/PINT/commit/5ca25e27a72cc1dbcb472c5b4ba13f0849f30ec1#commitcomment-15695355>.Web
> Bug from
> https://github.com/notifications/beacon/AAIHIlLXJ1UVS3gsB9fgxsFqImFnwUu_ks5peCtLgaJpZM4HNCac.gif
>

--
Scott M. Ransom            Address:  NRAO
Phone:  (434) 296-0320               520 Edgemont Rd.
email:  sransom@nrao.edu             Charlottesville, VA 22903 USA
GPG Fingerprint: 06A9 9553 78BE 16DB 407B  FFCA 9BFA B6FF FFD3 2989


Since the `__attribute__` macro is GCC is you might want to include the following so it plays nice with other compilers:

    #ifndef __GNUC__ #define __attribute__(a)


:bird: :eyes:


On 27/Jan - 06:53, Séverin Lemaignan wrote:
> Alright. If, by chance, Blender falls back to 'ON' when 'ADAPTIVE' is
> not available, we could have a boolean (True -> 'ADAPTIVE' with
> possible fallback on 'ON'; False -> 'OFF')

Unfortunately, it is not documented, and from reading the code source, I
don't think it implements that (in particular, because nothing reads
return value ...).


Fair enough. Is it too troublesome to squash the 3 commits together (the original ones that introduce the method and this one)?


@kolkov that _might_ be the case. Like it was reported in the issue, it might be that the IDE has to do some work as well.


Yes, for now. The scaler needs some sort of initialization code, but it should be handled in a separate issue.


Rails version? 4.2.3?


@kzhdev
lg


32-bit discrimination! ;)

```go
# github.com/lunixbochs/usercorn/go/native
go/native/statfs_linux.go:9: cannot use s.Type (type int32) as type int64 in field value
go/native/statfs_linux.go:10: cannot use s.Bsize (type int32) as type int64 in field value
go/native/statfs_linux.go:17: cannot use s.Namelen (type int32) as type int64 in field value
go/native/statfs_linux.go:18: cannot use s.Frsize (type int32) as type int64 in field value
go/native/statfs_linux.go:19: cannot use s.Flags (type int32) as type int64 in field value
make: *** [usercorn] Error 2
````

Have you seen this funny [bug?](https://github.com/unicorn-engine/unicorn/issues/398#issuecomment-175028959) in usercorn?


Привет.

Выглядит хорошо! Есть несколько замечаний.

Когда в строку подставляешь переменные, учти, что вокруг значения переменной нужны пробелы, иначе текст слипнется. Проверял вообще работу кода как сказано в задании?

Вызов `typeof` для массива возвращает `object`, так же как и для объекта (`{}`). В этом задании дополнительные проверки не обязательны, так как весь проект теоретически ты контролируешь, но имей в виду. В качестве дополнительного упражнения, можешь попробовать обработать это исключение и возвращать какое-нибудь сообщение в этом случае или, может быть, `false`.

Старайся форматировать код одинаково, в частности отступы пляшут.


Seems correct!

The idea is we rotate the rect of the label itself, but add the spacing without considering any angle. So adding the spacing to the rotated size is the correct approach :-)


Yes I did :-) The actual spacing is correct, but there are a few issues with the avg property that became max...


hmm, I must be getting paranoid about divisions


Should NOT be greater than 1


@afollestad And you did try `onCreateView(...)` not only `createView(...)`? Strange.


:+1:  Nice comments and explanations. Please go ahead and merge if no other comments from afsal. Was using all these examples with presentation, looks like we can avoid ppts if we make comments like you have given.


The old method wasn't specifying an encoding (the cause of this entire issue).


I see. One thing we could do is to download scylla-server, get its contents and modify it in python.

But really, this is not really necessary. I was just thinking that this could become a cluster method in the future.


@Mumfrey Ah, thanks for the explanation! I was basically wondering about the use cases for this considering we have access transformers, but if it's actually only supposed to be used in places where we can't apply our access transformers it makes sense to use the annotation there.

I was just a bit worried we would soon have a mixture of fields modified with an access transformer and some without, which would be really inconsistent any confusing, but as long it is only used in places where ATs **cannot** be applied this sounds reasonable to add to Mixin.


Hi,

I just took a quick look at your code. I see some errors in the randint () functions. Don't convert topLimit to string. Also, you should use return statement to return the value that random.randint(numbe1, number2) creates.


I experienced the issue when my vertical slider was placed into Y-scrollable <div> element. The slider was at the right place, but mouse position after scrolling was incorrect (lower than real cursor for same pixels as scrollTop element property). So I fixed this. Please rewiew.


@Jiri-Kremser I wonder why hawkular-rx is a good name for this module? I suppose `rx` stand for "reactive" - a kind of a paradigm in programming, right? Is that really this paradigm that makes this module distinct from all other modules (dist, e2e, pinger, ...) in this git repo? I do not think so. Would not `hawkular-api` be a better name?


@TR4Android I thought so too. But the reflection code I use is what `LayoutInflater` runs if the factory returns null.


@nitincchauhan I originally included that, but then I realized that the only values I have are ``previous`` (which is ~30sec before) and ``now``.  If ``now < previous`` that means the clock shifted backwards by more than 30 seconds!  We have no way to detect a clock drift of ~128ms magnitude.


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


@bryanoltman do you want to set the '_url' if you're going to return?


Another thing that occurred to me is that this setup code should be moved to `setUp`, for roles separation purposes (try to not mix setup and actual test whenever possible).


I have also noticed this befor. Will this be fixed in any future?


@gabrielamatias "The need to take recurring payments is increasing, and it is becoming..."


@gabrielamatias remove "creating new markets" - don't think it reads right


Maybe this should be in try-catch block, cause if the content blocker has been loaded it probably will raise an exception?


Please don't update the version as part of a pull request. I do version management as a separate commit, to make it clear what each commit is about.


the generated atlas is now completely different. All sprite are now packed in one line


This method is unused. please remove it.


Sorry! My mistake - got the link from someone else and didn't check it.


I was about to ping Sergey for the issue but I've seen I missed the ping in our message. He'll know what needs to be done and where. Unfortunately I don't have a Windows machine so I can't help too much right now and I'm using AWS to make things work when needed :) If you know anyone working at Microsoft...


....Nice...I hope we can promote a way to solve this rare case... Extra copy always annoy me...


In your example, it looks like the response is a JSON formatted string. How about parsing the response here, if possible?


This method is unused. I guess it can be removed ?


https://jira.duraspace.org/browse/ISLANDORA-1594 is a higher level ticket. it might need to be broken out into smaller tickets as it would appear none of the calls in this SP are respecting solr defaults


I got it now, thanks for the clarification.

And yes, we should never really stream anything to cout in a function that is also called from a wrapper. An exception is the way to go.


タイトル画面の上のほうに，時計と電池表示のぶんだけ余白を作りました。
title.png ... 元の画像
title_1.png ... 余白あけた版その1。余白は薄い緑色。
title_2.png ... 余白あけた版その2。余白は白。


このへんの資料。しかしこれ動くんかいな・・・
https://www.playframework.com/documentation/2.2.x/ScalaResults
https://www.playframework.com/documentation/ja/2.3.x/ScalaSessionFlash


getOperand is not used anymore and as such should be removed from all this hierarchy (this is rightly encoded in kind)


Thx.

Just pushed the fixes. I just saw the implementation for XAxisRendererHorizontal is a little bit different...


The following doc(s) have been deployed to the [testmultid](http://testmultid-static-docs.pantheon.io/docs) Multidev environment:
- [circle.yml](https://github.com/pantheon-systems/documentation/commit/7f629db0e8198eb55661c1277627efcb6ef72abb/circle.yml)
- [docs/articles](http://testmultid-static-docs.pantheon.io/docs/articles)
- [docs/articles/drupal/8/configuration-management](http://testmultid-static-docs.pantheon.io/docs/articles/drupal/8/configuration-management)
- [docs/articles/drupal/8/create-services-yml-file](http://testmultid-static-docs.pantheon.io/docs/articles/drupal/8/create-services-yml-file)
- [docs/articles/drupal/8/drupal-8-performance-and-caching-settings](http://testmultid-static-docs.pantheon.io/docs/articles/drupal/8/drupal-8-performance-and-caching-settings)
- [docs/articles/sites/quicksilver](http://testmultid-static-docs.pantheon.io/docs/articles/sites/quicksilver)
- [docs/articles/sites/updating-payment-methods](http://testmultid-static-docs.pantheon.io/docs/articles/sites/updating-payment-methods)


I'm also curious about this commit, especially now that it's part of the audit mode in Chef...


Hi,

In the evaluateAnswer function, you declare the first parameter is "userGuess," but then you use "Userguess"  (the first letter is capitalized) in the body of the function.


Am sorry, could not see Issues enabled so post it here.
After `make clean` no `_target*/{lib,dep,obj}` survive and this makes redirections unhappy.
Please consider fixing
TIA


Marvin,
Is this master branch still functional? I'm about to start the next book and wonder if it's safe to $ git pull ??


Oh yeah, and it's working :+1:
Thanks


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


Reviewed.Please merge


Please add this to your [global gitignore](https://help.github.com/articles/ignoring-files/#create-a-global-gitignore)


in fact removing it put Relation in isolation from programState and constraints which is actually really great !


Yes. We want to update the backing property but don't need to do any further processing if the url was set to `nil`


This </p> doesn't have an opening tag!


@gabrielamatias should be full stop at end of all bullet points


@bogas04 @apicellaj Please review this new logic and let me know if there are any gotchas


Hello tsmr,
Could it be possible to have more info, as it seems not to be linked to lock mechanism?
Thank you
regards,


Componentes ou recursos? Sugestões?


Thanks for replying

My issue is scope is not working in any case now...
Before it was working fine that if scope is to avoid a user from reading another user's data, scope will not load result and fetch method will result into nil..

but now scope is firing error in all cases.. for eg: Model.fetch(id) is working but if we do  Model.some_scope.fetch(id) firing error
```
IdentityCache::UnsupportedScopeError: IdentityCache doesn't support rails scopes
```

In my case I want to cache only active items not all, so am calling it via scope but now its not possible..
Is there any way to achieve  this Model.some_scope.fetch(id)


@gabrielamatias should the </li> be on the line below?


The entity "uuml" was referenced, but not declared.


@gabrielamatias are they going to be asked the fintech question here as with the SDR role?


No they aren't just variable names, they have some internal structure, namely `lines[current_polygon]` and `polyStart[current_polygon]`.
I would like you to store those in a new variable.

Also don't forget about the mistake/bug in this code!


The "config.js.example" snippet above will only work if you have a "DeltaSol BS Plus v2" controller. If you do not have such a controller you will most likely have to modify the VBus field IDs listed in the `xivelyPacketFieldMap` in lines 64 and 65 of the config file (and optionally add more fields). The get a list of fields provided by your controller you can remove the comments on line 294 in the `examples/vbustouch-proxy/index.js`, start the example and wait for a couple of minutes. You should then see a list of field IDs, their name and their current value. Then just copy the field ID into the `xivelyPacketFieldMap` object in the "config.js" file, add the comments back in the `index.js` (to stop the output of the fields) and restart the example.


This is all now in the cart service, correct? If so, you can go ahead and delete these comments. Makes it cleaner.


Nope, it's broken at the moment. Skip it for a day or two, but I'll also notify when it is ready.

- m

---- On Wed, 27 Jan 2016 10:25:16 -0500 Craig Leat <notifications@github.com> wrote ----

Marvin,
 Is this master branch still functional? I'm about to start the next book and wonder if it's safe to $ git pull ??
 —
Reply to this email directly or view it on GitHub.


ok, done


I think it's still too complex.


I actually meant that it doesn't expose methods which could generate the `reset_password_token` which is a property of `Person`


I also got a lot of errors during migration from 0.83.8 to 0.84 that I didn't get in the past feek weeks during which I made a lot of migration tests...


Yep


Instead of doing something like `Model.where(active: true).fetch(id)` use

```ruby
record = Model.fetch(id)
record = nil unless record.active
```


If this type doesn't hold an IntPtr, and is just a container-wrapper for a SafeGssCredHandle, why does it exist?


Ok I guess this will be another line


Thanks. Glad I asked!


Thanks mate! :+1:


new badge: ![badge ](https://cloud.githubusercontent.com/assets/1520891/12618267/677027dc-c513-11e5-90ff-f9b78c06cf74.png) ;-)


in_buffer doesn't exist, does not compile


I am wondering why this is not constructed and stored when calling the constructor of BinaryRelation rather than on each call to hashcode/equals


This patch cannot be accepted.... too complicated....


How different is this test from the one before?


@AMikhniev It MUST not, but SHOULD.


This assert seems to have been lost


did you manage to compile and run the tests?
should I wait before I merge?


Можно ведь заменить на один TextView, нет?


You are right that if I use the Diamond Operator i can drop the SuppressWarnings annotation. But I don't understand the problem. Is there an issue with any IDE?


1. Make your classes more descriptive than style1, style 2...
2. best to avoid inline styling because it's not modular and easily overlooked. This makes it harder for others to modify the design. If you want to make quick fixes to styling and don't want to make a new class, checkout bass css. It's a library that gives you classes to do many individual styles. It's the same idea as bootstrap, but more lightweight.


Does the test pass if you do not cast to Patient but keep them as Objects?


// TODO: Make this a reuseable laravel package


* TODO: Refactor the validator to reduce duplication


This part looks good.... But I thought I already pushed it...


давай вынесем блоки div class="logo" и div class="user-block" из блока nav class="main-navigation"


Unused: No call sites? Then get rid of the method.
Unused: Shouldn't be possible? Call Debug.Fail so we find if that assertion is ever wrong.


Tested it, and it worked like a charm!
It think that this was one of the most play making issues, and now has been tackled by this.

Thank you so much for fixing this!


Here is the result of my migration:

C:\Users\me>"C:\Program Files\PHP\v7.0\php.exe" -f L:\inetpub\wwwroot\glpi091\tools\cliupdate.php -- --upgrade
Current GLPI Data version: 0.83.8
Current GLPI Code version: 0.91
Default GLPI Language: en_US

========================================== Update to 0.84 ==========================================
To be safe, we are working on origin_glpi_networkports. It is a copy of glpi_networkports
To be safe, we are working on origin_glpi_networkequipments. It is a copy of glpi_networkequipments
network warning - NetworkPort[19604]=Computer[32781]: Invalid gateway address @ network warning - NetworkPort[21078]=Co
etwork warning - NetworkPort[22515]=Computer[33677]: Invalid gateway address @ network warning - NetworkPort[25371]=Com
etwork warning - NetworkPort[30266]=Computer[35617]: Invalid gateway address @ network warning - NetworkPort[30933]=Com
etwork warning - NetworkPort[10640]=Computer[6143]: Invalid gateway address @ network warning - NetworkPort[30869]=Comp
@ network warning - NetworkPort[30107]=Computer[35791]: Invalid gateway address @   *** PHP Notice(8): Undefined index: glpiactive_entity
  Backtrace :
  inc\ipnetwork.class.php:560
  inc\ipnetwork.class.php:263                        IPNetwork::searchNetworks()
  install\update_0831_084.php:2003                   IPNetwork->prepareInput()
  install\update_0831_084.php:80                     updateNetworkFramework()
  tools\cliupdate.php:278                            update0831to084()
PHP Notice: Undefined index: glpiactive_entity in L:\inetpub\wwwroot\glpi091\inc\ipnetwork.class.php at line 560
...
kPort[35869]=NetworkEquipment[2983]:  @ invalid IP address - NetworkPort[35870]=NetworkEquipment[2984]:  @ invalid IP address - NetworkPort[35871]=NetworkEqui
ent[2985]:  @ invalid IP address - NetworkPort[35872]=NetworkEquipment[2986]:  @ invalid IP address - NetworkPort[35879]=NetworkEquipment[2993]:  @ invalid IP
ddress - NetworkPort[35882]=NetworkEquipment[2996]:  @ invalid IP address - NetworkPort[35883]=NetworkEquipment[2997]:  @ invalid IP address - NetworkPort[358
]=NetworkEquipment[2998]:  @ invalid IP address - NetworkPort[35901]=NetworkEquipment[3015]:  @ Change of the database layout - glpi_networkportaliases (5 min
** You should have a look at the "migration cleaner" tool !                                         )
** With it, you should re-create the networks topologies and the links between the networks and the addresses
** You can delete ocs_* tables if you use OCS mode ONLY AFTER ocsinventoryng plugin installation.
** You can delete ocs_* tables if you do not use OCS synchronisation.
Task completed. (16 minutes 17 seconds)
========================================= Update to 0.84.1 =========================================
Task completed. (23 minutes 56 seconds)
========================================= Update to 0.84.3 =========================================
Task completed. (23 minutes 57 seconds)
========================================= Update to 0.84.4 =========================================
Task completed. (23 minutes 57 seconds)
========================================= Update to 0.84.6 =========================================
Task completed. (24 minutes 3 seconds)
========================================== Update to 0.85 ==========================================
glpi_dropdowntranslations table already exists. A backup have been done to backup_glpi_dropdowntranslations.
glpi_knowbaseitemtranslations table already exists. A backup have been done to backup_glpi_knowbaseitemtranslations.
** You can delete backup tables if you have no need of them.
Task completed. (27 minutes 2 seconds)
========================================= Update to 0.85.3 =========================================
Task completed. (27 minutes 43 seconds)
========================================= Update to 0.85.5 =========================================
Task completed. (27 minutes 43 seconds)
========================================== Update to 0.90 ==========================================
Task completed. (27 minutes 45 seconds)
========================================= Update to 0.90.1 =========================================
Task completed. (27 minutes 45 seconds)
========================================== Update to 0.91 ==========================================
Task completed. (27 minutes 47 seconds)
Migration Done.


this can be removed


static is implicit, no need for it.


I would rename patient1 to proxyPatient and patient to nonProxyPatient


I see that the hard coded values are temporary, but would it not be better already at this stage to make the constants private as defaults and expose them via non-static getters? I suppose that's how you'll have to access them, regardless of how you'll implement the loading of the actual values.


please rename it to `SetSelectedItem`, `GetSelectedItem`


Is `isServer` something the user can control? If code can get this far with it true, this should be an exception, not an assert.


@piton Да, можно и так. При выборе типа поля исходил из следующих соображений:
* integer занимает меньше места;
* сортировка по полю типа integer несколько быстрее.


@piton В принципе оба варианта эквивалентны. Дело вкуса больше, хотя возможно более короткий вариант кому-то покажется предпочтительнее.


I assume by `uuml`, `ü` is meant? Which is the `ü` HTML-escaped. This char is only used in `<string name="credits_turk_description">Türkçe</string>`, which is not changed in this translation. Thereafter, only [these five characters](https://stackoverflow.com/questions/1091945/what-characters-do-i-need-to-escape-in-xml-documents) have to be escaped in XML.


Exception text needs to be a resource string


Resource string


Этот функционал стоит унести в holder


It works but will be lost upon reboot


Thank you for your advice and the fantastic sample code! Unwrapping at the same time is exactly what i wanted to do. You are my Stack Overflow! lol
I'll try your code soon^^


OK sorry I see now what you mean....


:eyes: :+1: :two: me!


Don't need to set $scope.error = null twice


Скобочка


I assume by uuml, ü is meant? Which is the ü HTML-escaped. This char is only used in <string name="credits_turk_description">Türkçe</string>, which is not changed in this translation. Thereafter, only these five characters have to be escaped in XML.


`connectionsPerIp` vs. `connectionsPerIP` vs. `connectionsPerClient`?


Этот элемент используется настолько часто в вебе, что у него есть общепринятое название - breadcrumbs или хлебные крошки. То есть это та цепочка ссылок по которой он может вернуться обратно.


The order doesn't matter.  Pete and I talked about it this morning.  It was just a misunderstanding about the responsibilities of go-extpoints and go-import-subtree.


try to limit the texture size


Clang supports __attribute__(format) as well, but it also defines __GNU__C, so that makes sense.


Don't merge yet. I just wanted to make sure that the PR was working fine.


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


i just started my own phone repair shop and i wanted to learn about hot to root and flash a phone to any services is there any  groups for a starter like me so i don't mess up someone  phone


There are some `isNtlm` asserts that I also wonder if they should be exceptions, I'm not going to point them all out, though.


this looks like a merge error


no need to add this method explicitly to `CGUIMediaWindow` because it's virtual.


@dkayiwa  that is what I was asking you in IRC.. the prob are 1) if we use same session we doesn't need this test case at all
2) or we can convert a hibernateproxy object to nonhibernate object which i tried and end up getting a Class cast exception .


убирай все br из формы - они тут не нужны будут


@Jiri-Kremser does it make sense to add an empty package-info.java ?


and NPM. Could you add this one ?


辛苦 zz 菊苣～
<img width="218" alt="2016-01-27 11 36 23" src="https://cloud.githubusercontent.com/assets/1846356/12618470/ea87b73a-c54e-11e5-8c9a-fbd9d7fe08d0.png">
文字有点不垂直居中的感觉，默认还是用原来 tag 的颜色吧，鼠标移上去后变色，分成两部分变色，鼠标在团队上就只有团队变色这样～


typeof safari == 'undefined'


Where do you use this concept?


I see that you already did some modifications. Thank you. I changed a bit the code to be closer to my solution.


And what about installing the package during the compile phase? This is the common way of solving the problem of resources needing installed stuff.
See the "run_action" section at:https://docs.chef.io/resource_common.html#run_action


@piton На самом деле происходит и создание новой сущности и изменение существующей. [Пример в википедии](https://ru.wikipedia.org/wiki/HTTP#POST) с добавлением комментария к записи использует запрос POST(ситуация аналогичная). В принципе тоже дело вкуса. На мой взгляд приемлемы оба варианта.


Diria componentes.
Estamos a dizer que um recurso é uma função logo aqui deverá ser um componente da aplicação.
Mas é dúbio...
Muito dúbio..
Fica ao teu critério..  :)


how about "...between regular and rushed mode"?


I like this line


this line is also good.


@brettwooldridge ~30s is when everything is alright. but for clock change, we should add 128 tolerance as:
if (clockSource.plusMillis(now, 128) < previous || now > clockSource.plusMillis(previous, (2 * HOUSEKEEPING_PERIOD_MS))) {


The reverse says that if this flag is set that both accept and init will be returned.  Can we have a state where only one of accept or init is set here? Does it make the universe stop existing?


I also don't think you need to set $scope.login = {}  (Joe correct me if I'm wrong)
ng-model should create the object.


не разделяй на две формы - можешь div class="filter-producers" и div class="filter-groups" обернуть одной формой и в конце формы вставь кнопку "Показать"


I have looked but not implement them.

I could work on these task

1) Your pattern matches are incomplete. Pity if someone entered something other than "CA" or "DA" :p

2) The most generic advice ever: Whenever you find any repeating patterns in your code, write proper abstractions around those patterns. Example 1:

putStr foo
hFlush stdout
bar <- getLine

put it in a separate function! add the "is valid input" check to that function!

Example 2: Every branch in your Main:main:runloop ends with runLoop newAccounts. While newAccounts might be different, you sure can factor out the runLoop. This leads to:

and this one :

[–]BethAr 1 punt 21 uur geleden

I didn't look it too deeply, but from the types, it seems there are a few functions that could be collapsed into one, to avoid code repetition.

You also defined many repetitive aliases in the where clauses, like l and r, which could be refactored into another helper function.

I have to think wel how to implement these so I will not break things.
Maybe a seperate issue.
This one was purely on displaying the reports.


What the fsck…


опечатка ```</lavel>```


This was always intended that way, so it's truly a bug. However, it will require users to re-tweak their flags, so I'd call it out in the introduction of the announcement mail.


@piton Да, тут полностью согласен, но конкретно в этом проекте не стал использовать gem simple_form. Захотелось немного поверстать руками:smile:


Added a boolean `destroy_cache` with commit 0440cb6448c5777193a519819d8eea5cc6cc1a2e


was there a failure? adjust require if you need to drop 0.4 support


Also add anchoring of regexps in PromQL.


I still do not get it because the verifies text and method names mean the same. Appending "version" does not make it any different.


`@see {@link..... | SO 610406}`


@piton Согласен. С другой стороны такой вариант избегает лишнего запроса в БД.


I have seen a similar function for `State` named `zoom`, in case you like that name.


@soundanalogous :+1:


Actually, that is what I've done with the LWRPs and the abiquo-api gem. I'll give it a try.


Why did this break the tests?


이거 제가 고치고 커밋만 안하고 있던건데 ㅋㅋ


@asbjornu We have ported the commits that we are your sure of will work to the new unified module above, that includes tax for discounts and modman. (Very grateful for those commits) There are some issues that we are not sure of and some that is plain wrong, therefor I don't recommend to accept the pull request as a whole


- that it


@piton Я всё-таки считаю, что размер файла и прочее это атрибуты приложенного документа. Uploader - просто делать реализации.


Sí este test no tiene sentido aqui... porque el rango se valida más temprano en el CLI del Crane según el framework


haha


Hi,

I think the error is that you call the askUserToGuess function in the playGame function. The previous one returns a boolean value.


Такую галерею лучше разметить иначе, по скольку подразумевается следующая функциональность: Пользователь видит большую картинку (некий вью-порт) и несколько маленьких картинок под ней (некое превью), когда пользователь щелкает в одну из маленьких картинок, большая должна поменяться. Я бы сделал как-то так:
div.gallery
___div.view-port
______img
___div.controls
___и тут все маленькие картинки

Причем в данном случае лучше использовать просто теги `img` не оборачивая их в тег `figure`


Смотри, все блоки с классами block1...block6 - одинаковые - значит мы будем для них прописывать одни и теже стили - значит у них у всех должен быть одинаковый класс, например, div class="product-item".


Весь layout можно ведь заменить на один TextView, нет?


Also, arguably an [ENHANCEMENT] and not a (breaking) [CHANGE]. You might want to mention that it has also improved, suggestion:

* [ENHANCEMENT]  Storage switches less often between regular and rushed mode.
* [ENHANCEMENT]  Storage switches into rushed mode if there are too many memory chunks.


Этот функционал стоит унести в holder.


этот список парсинг того что в главном меню на кнопке `mods` показвает


this won't work as we call the parent at the very last. why not use `CGUIWindowVideoBase::OnDeinitWindow` as you have done before?


I'd not have a short option, to remain consistent perhaps?


FYI, I applied this patch to my hxcpp (3.2.193) and it indeed fixed @Simn's enum testcase (for me, on Linux.)


你有多爱字符串？


PerClient is confusing as we are assigning one client object per connection internally…
`connectionsPerOrigin` would be fine too.

Also, intuitively I regard I(nternet) and P(rotocol) as two words, hence both uppercased …
Thus, using another identifier could be favorable, but not `connectionsPerClient` … other suggestions?


Great! Note that you could also use this to track the number of input/output channels.


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


Yeah, separate issue. This is refactoring. You need to scan the codebase for repetition, and maybe write it down in an issue for each chunk of repetition you find... then we abstract these repeated parts into functions (that's the job of functions):) Yay! someone else explaining this stuff too is good :)


Часть названия товара, которая в кавычках («American crew», «Baxter of California» итп) нужно обернуть в span (для всех подобных элементов block1...block6). Это нужно будет нам при стилизации - так как все назнание товара обычными буквами,  а название бренда будет большими буквами.


Sorry for responding so late in the game, but I would personally like to keep the change to the PS1 telling what the venv we are in is or at least a variable that will allow us to keep it if we want. This is the same functionality as if you were to just use virtualenv and I feel it makes it easier to know if you are in a venv or not without having to even type in any command. Just my 2 cents, but I feel that the functionality should be the same as the package it is mimicking.


This is still incorrect. What you tried to do here is set a variable called ```setName``` when toy need to be using methods. Here's how it should look:
```java
public class Pork extends Food {
	setName("Whipped Cream");
	setCost(0.40);
	setStoreCost(1.00);
	setQuantity(20);
	setExpiration(7);
	setDesirability(1);
}
```


A


Поля можно сделать final


Did you get any more info on this?


My IDE (IntelliJ) warns when using `HashMap<String, String>` with Java 8 as it's more verbose than necessary. When switching to Java 8 recently I replaced all the old syntax with the diamond operator.


Тоже final


There's no need to update sample rate on route change. The stream format callback will always be called if the sample rate changes, regardless if it's because of plugging in/out hardware, or this or another app changing the sample rate (by setting session preferred rate), or a connected IAA host changing the stream format of the node.


Cool.  Thanks!  ;)


Could we consider to add an info page to the menu for these kind of stuff somehow???


Ah, okay...
Den 27. jan. 2016 7.36 AM skrev "Greg Arndt" <notifications@github.com>:

> The order doesn't matter. Pete and I talked about it this morning. It was
> just a misunderstanding about the responsibilities of go-extpoints and
> go-import-subtree.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/taskcluster/taskcluster-worker/commit/42d51c373ac1b7bbe8233ae9a836b50ce588c3a8#commitcomment-15708778>
> .
>


@piton Не очень люблю я эти fixtures и фабрики(factory_girl):smile: Есть пара конкретных претензий:
* Лишние зависимости;
* Не очевидно сразу откуда взялись эти значения и как они были созданы(что содержат);
* Есть некая "магия" в этих фабриках. Вроде и понятно, что где-то в другом месте это объявлено, но не сразу ясно где, а так сразу видно.


I think the custom_filter.js is fine, not too sure though. But I think it should be a factory. It creates a filter object which contains methods that can be used by our filter controller or any other controller as long as it's injected. If not a factory, what do you think it should be?


Had you removed the error class previously?


Not sure this is needed. I think we can assume they are string :).


@piton Согласен, можно и вынести.


I miss a string[] typehint :'(.


Can write a generic `KeyValueStore[A, B, ?] ~> AtomicRef[Map[A, B], ?]` and then use that instead?


TeamCity Civic Assembly :: Build all [Build 31](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=502&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


I am hoping it would be easier then to port and incorporate SMBX65-38A features into LVLX file and then into PGE engine, once skeleton for action-based event system is complete.


@asbjornu I will be happy to add this feature to the new unified module https://github.com/PayEx/Magento-modules/tree/PayEx_Payments
+ the configuration option if that's the final consensus. I don't see any big problems with it.


TeamCity Civic Assembly :: Build all [Build 31](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=502&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Exit code 2; exit code 1; exit code 123 Build time: 00:00:41


Where did this come from?


Есть мощный курс по формам https://htmlacademy.ru/courses/74 рекомендую его пройти.


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


@dkayiwa we need this line  to initialise the concept object in the database with the same uuid  as the patient object uuid , and then retrieving the proxy object  of same concept object in the 259 line.


Tenho dúvidas... pode ser que numa próxima ronda acenda uma luzinha...

Thanks :-)


@piton Не стал сильно перегружать такое простое приложение библиотеками, может попробую как-нибудь в другой раз:thumbsup:


spoke with @snordhausen, agreed to take the first element (see comment)


The third number should be 10?


забыл добавь пагинацию - переключатель страниц)


the UTF-8 encoding struggle from issue #43 & #43


I worked through some things with Paul ... as it turns out I was on-track, but incorrectly writing my console.log functions (which I'd deleted entirely in what I originally turned in because I was so frustrated).


В целом вариант оборачивания `input` в `label` верный и рабочий, но все же более гибкий и правильный вариант - это размещение их независимо и связывание через атрибут `id` у тега `input` и атрибут `for` у тега `label`. В курсе "знакомство с формами" этот вопрос рассматривается https://htmlacademy.ru/courses/46/run/5


c'est pour quelle jeux ? Pourquoi il y a les hack la ? ^^


Cool ;)


Yep, you're right


Wat. Is. This? :D

padding, ftw!


divide this long row


Caution. It seems this image can only be built on environments with `/home/hwk/demo`. I guess this contradicts the docker guidelines for building images, e.g. images should be able to build anywhere (except with internet ;-)


Sure


@sapieneptus Is this example correct?  If it is not, please update.


Merci!


OK


Fixed it, thank you!


And the copyright lasts until your dead, plus another 70 years, so no need to worry.
Only update the year if you add anything new in 2016.


I think just focusing on the persistence would allow for more reuse here. We want an interpreter like
```scala
def persistentConfig[S[_]: Functor](loc: FsFile)(implicit S0: Task :<: S, S1: ConfigRefF :<: S): ConfigRef ~> Free[S, ?]
```
i.e. we just tack on the persisting and leave the final interpretation of the `ConfigRef` for later. If we then want to have a version of this that interprets into `Task` using the `AtomicRef.fromTaskRef` we can, for convenience.


I'm getting this error on this string :     <string name="status_updated">Status geüpdate.</string>


убрать br


you don't need it. Set something to class TestSetting every request. I wrote about that on my previous review. But you ignored


Забыл проставить текст для alt


okay, watch the next commit.


:beer:


Looks fine. I wonder what are the implications down the line (I'm hoping actual none) :)


imf! картинка фоновая


Please check the spelling.


I believe you need to interpolate the string here. `"#{role}"` as against `"(role)"`


is it correct?


lol


заголовки начинаются с h1


Turns out the byte array version is for data sent in binary form, like uploads, and meant to be used alongside the string version ([Websocket Annotations](http://www.eclipse.org/jetty/documentation/9.2.10.v20150310/jetty-websocket-api-annotations.html)). However the docs state that utf-8 is explicitly used ([OnWebsocketMessage](http://download.eclipse.org/jetty/stable-9/apidocs/org/eclipse/jetty/websocket/api/annotations/OnWebSocketMessage.html)), at least for the `Reader` version, so I'll change to use that so we can be certain of the encoding received.


Galaxy, not Tower integration


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


I know that when `self.filename` is None this will 'magically' return None, but you might consider being more explicit?


You should have made that pull request for ZDoom... ;)


Added contact


If all a callback does is invoke a function with the same arguments, you can replace the callback itself with the function:

```
exec(options, JSON.parse(data))
  .then(function(result) {
    resolve(result);
  })
  .catch(function(err) {
    reject(err);
  })
```

Can be replaced with:

```
exec(options, JSON.parse(data))
  .then(resolve)
  .catch(reject)
```

A larger optimization is that the entire `return new Promise(function(resolve, reject)` can be removed and you can simply return a promise from anywhere you would resolve or reject, e.g. `return exec(options, JSON.parse(data))`.


Не люблю его, вёрстка и танцы с паддингами заметно сложнее становятся. И здесь мб будет усложнение лайаута.


Donuts :)


На самом деле тут вроде должны быть еще элементы. В макете "Группы товаров" имеют вложенные элементы


@clebergnu, I'm guessing you'll fix that bug reported by @ldoktor down the line on this series.


Как ставятся правильные(горизонтальные) кавычки? Потом через шрифт?


Thank you verymuch. :+1:


UPD: lib, dep, obj just don't get created on repo clone


Yes, I think this is currently O(2^n) because at every level of the recursion the operation recurses twice. You can reproduce this with `Array<Dynamic>` directly as well:

```haxe
class Main {
	static function main() {
		for (k in 0...100) {
			var files = [for (i in 0...k) "foo"];
			var list:Array<Dynamic> = [];
			for (f in 0...files.length) {
				list = ["foo", list];
			}
			trace("Testing with " + k);
			trace(list);
		}
	}
}
```

Sadly it's not the cause of that mac-enum-serialization-crash.


Can you try this?
http://gerrit.aicp-rom.com/#/c/22352/


Dont' forget to add this Test in in Category1Suite, otherwise it will never be executed !


OK


Would `.Mounts 0` only print the first mounted volume?


@stas-vilchik


Neither does `out_buffer`. Replacing them with `nullptr` fixes the compiler issues, but somehow I doubt the solution would be that easy


Sure


Gud jokes.


Ромбики можно сделать через list-style?


Thank you :) Just to note, currently it will overlap with the page tools https://ptpb.pw/p5zE.png


Hmmm.... OK, I agree with your reasoning.


В карточке товара, картинку тоже лучше не оборачивать в `figure` и лучше обернуть в `a`, так вероятнее всего или она или название товара ниже должны вести пользователя на страницу этого товара (или даже и то и другое).


it seems to me like you have all the wiring in place already, since `props` is an Observable, no? Just that you have to feed your `.disable` button's DOM events into the `props` stream, as part of your 'intent' rather than down here after `props` is already constructed.

 something like:
```javascript
const toggleDisabled$ = DOM.select('.disable').events('click').map(ev => true).startWith(false);
const toggleProps$ = toggleDisabled$.map(isDisabled => {options, value: options[0].value, isDisabled});
```

if you want to avoid overwriting the other props when this happens, split the props object into several streams that can be controlled independently: `Toggle({DOM, disabled: toggleDisabled$, selection: selectionProps$})` (or make props a metastream$$)


Ugh.  That last sentence should say "when the var is EQUAL TO the size of skel_data..."  Right now it looks like I'm talking about an assignment.  Sorry about that!


Через что делается чекбокс нужного размера и формы? (с крестиком вместо галочки)


Please review this, @casey-collab


Hi @juliocamarero, this registration using dynamic references doesn't work.

When portal is started with portlets initialized and then I deploy/redeploy the module, `addPanelApp` is called before `setPortletLocalService` causing NPE.

I'm not sure if there is am option to tell OSGi container to load the references in a specific order.

We have luck with `companyLocalService`, which is set before. Probably because `CompanyLocalService` in alphabet is before `PanelApp` :)

Do you have any trick up your sleeve how to solve it?


it's fine to leave in `stable-v1`. We will probably move away from cf-blue-green on master eventually, at which point we can re-think whether the `cf target` call should be part of deploy.


I'm not sure exactly, but I think the tests are failing on Python3 because this was removed.


Are you sure? I was using TrackballControls  on purpose... Now the rotation is limited :-1:


@larrycai I created [niccokunzmann/samba-share](https://hub.docker.com/r/niccokunzmann/samba-share/). If you like to have a look at it. I would like to include your changes, too.


Hmmm... you're right, I should drop the cast too.


Can probably just reference the prototype (Database.prototype.adapter.types)... but overhead is minimal so OK :+1:


That's what the computer expects. If you instead do the unexpected you may sometimes find more bugs!


@abhinandanprateek We can actually remove the setId and setDomainId in all the testxxxLockableAccounts; create like a single class leve AccountVO and then in each of the testxxxLockableAccounts methods simply call the setType method and then call assert on them.


Thanks. Makes sense. Can you make a pull request. Please add Signed-off-by line.


isn't `toggle.DOM` an Observable? so it should go into the construction of this DOM Observable with combineLatest or whatever, rather than inserting an Observable into your final vtree


Для этого элемента тоже есть устоявщееся название - это paginator.


@lmr I may push something on the tail of those patches, or better IMHO, at a separate PR with proper tests.


:lollipop:


I don't see the merit in this change. I'm a believer in only changing the access rights of variables when we need to.


另外对于列表页的和种子 tag 上方的字幕组点击跳到链接 `/team/[team_id]`，
对于列表页的和种子 tag 上方的用户点击跳到链接 `/user/[user_id]`
我们完成搜索的功能之后做再处理一下这两个专属页面～


remove google scholar


let's talk in person.


Oh, sure!


you can remove that. It is not useful.  Am I right ?


minor, but needs a newline


Problem is need of animation for TrackballControls.
Even if you do not touch mouse, rendering always runs.
I replace it with OrbitControls - it has more or less similar functionality.
Could you try this:

http://jsroot.gsi.de/dev/examples.htm#tgeo_prim


Tests? For some reason I don't instinctively trust the behavior of `new [bound method]` :)


1. Need to change the kernel/ram offsets
2. Get a dt.img
3. Get a prebuilt kernel
4. Get the OTA asserts
5. Get partition sizes
6. Figure out if we need to be a unified device


try to use position centre if you want to show it on the middle or some `% values`, like `left: 30%`.


логотип всегда ссылка


why two if's if you don't do anything in between them?

I'd write it this way:

````
if (!$this->form->isSubmitted() || !$this->form->isCorrect()) {
    return;
}

// your other code for submitted correct form
````


Hey, that's the same md mistake I always make!


can't you delete them instead of setting them to an empty string?


It shouldn't be necessary to ever specify -m32 or -m64 here. The top-level autoconf script chooses the right flags to use, and we have to compile src/backend/gpopt with the same flags as the rest of the server, or it won't compile.


maybe you should add validation instead? If your mailengine is set, the api_key and list_id should probably be set too?


could it possible to use top:20 instead of top:30 with margin-top: -10px?


Well, OK, it works fine, but I hate being restricted, and then we have to be very careful about the original orientation and the position of the camera (see my previous example)


not necessary I think


Please, convert the indentation to spaces. Our tab width is two spaces each.


## Caching the Inverse of a Matrix:
## Matrix inversion is usually a costly computation and there may be some
## benefit to caching the inverse of a matrix rather than compute it repeatedly.
## Below are a pair of functions that are used to create a special object that
## stores a matrix and caches its inverse.

## This function creates a special "matrix" object that can cache its inverse.

makeCacheMatrix <- function(x = matrix()) {
        inv <- NULL
        set <- function(y) {
                x <<- y
                inv <<- NULL
        }
        get <- function() x
        setInverse <- function(inverse) inv <<- inverse
        getInverse <- function() inv
        list(set = set,
             get = get,
             setInverse = setInverse,
             getInverse = getInverse)
}

## This function computes the inverse of the special "matrix" created by
## makeCacheMatrix above. If the inverse has already been calculated (and the
## matrix has not changed), then it should retrieve the inverse from the cache.

cacheSolve <- function(x, ...) {
        ## Return a matrix that is the inverse of 'x'
        inv <- x$getInverse()
        if (!is.null(inv)) {
                message("getting cached data")
                return(inv)
        }
        mat <- x$get()
        inv <- solve(mat, ...)
        x$setInverse(inv)
        inv
}


We should include a contact email here.


tabs as indentation?


Fixed in 7cbe27a5164766c2cb58054c0a0304aa17875877
Good call!


Keep in mind the old method also used an access transformer, I just simplified that method a bit by replacing it with a different one:

```diff
-public net.minecraft.world.storage.SaveFormatOld field_75808_a # savesDirectory
```


I hate deleting them because that means that I always need to define a "default-value" when getting the value.


translation !


Yup sorry.  I had an old version


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


Orientation was (is) real headache.
Geometry was rotated several times.
And it was not consistent with ROOT drawing.
Therefore I start with geodemo.C macro to reproduce basic drawings.
By default Z axis looks up in ROOT - this is now how camera configured.
With draw option 'y' one could select Y for up direction (default for WebGL). Like:

http://jsroot.gsi.de/dev/?nobrowser&json=../files/simple_alice.json.gz&opt=y


If I remember well, we don't use whitespaces around the dot operator in AtoM. We don't have a complete style guideline but we try to be consistent with the way we do things in our codebase.