> Even when these functions are made concurrent if the calls are still taking a lot of time we should implement a simple cache of ours. Basicly the cache will save the lastest response and will directly send that response to frontend which will take a few ms.

Yep, that's the way to go. :+1: You want to talk to external APIs in the background to update your internal cache with some interval between updates.

Then, for each incoming request, you quickly serve that internal cache.

That way, if your server is seeing 100+ requests per second, you can respond to them all without needing to talk to external APIs 100s of times.


also sorted all terms


Avoid using inline css, you should always keep your html and css separate.


Aren't both of these code-paths the same?


Thanks for that! It saved a couple of seconds.


I submitted a PR for that, just in case it is necessary: https://github.com/trotzig/import-js/pull/157


Right now just returns a string of the date as shown in okcupid's one of many formats, exactly how it is shown in the messages inbox.

Eventually will work on properly parsing all date formats okcupid displays here:
-If last contact was within past week, day of week is displayed (e.g. "Last contacted Tuesday")
-If last contact was more than a week a way, but within the current year, then Month (abbreviated) and Day is shown (e.g. "Last contaced Jan 11")
If last contact occurred prior to current year, then Month (abbreivated) Day, Year is shown (e.g. "Last contacted Dec 23, 2015")


asdas


Overall good job on the hw. A couple of things to note: you should have one css file for your website, and your .html files should not be inside of a folder.


I think the readme should have more information about our intended product. Test Comment?


Yes, my bad, hadn't really gone back thru to do any cleanup.


nice this is complex


opt.name is an alias for form so this is needed (i'm using name in some forms)


We changed Theano versionning. We now use a new Python pep 440. Now in the
truck, the version is tagget 0.8.0.dev0. We will bump Theano dev number
when needed (mostly, new user interface). This should help. You can check
at one point the Theano version to know if it is recent enough. If you need
this number numped for any reason, just ask. There is no constrain to bump
it.

On Sun, Jan 24, 2016 at 7:30 PM, Ehud Ben-Reuven <notifications@github.com>
wrote:

> it should be
> pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fchollet/keras/commit/ed1297b39310d37b384ba75e0eb771e9c0e4b80b#commitcomment-15639532>
> .
>


If we're going to change this here, can we change the other "private" methods to use the "_" convention as well? We should just be consistent one way or another.


Spelling - should be "Returns" instead of "Retruns". :)


In database.

> On Jan 24, 2016, at 2:20 PM, Beagle <notifications@github.com> wrote:
>
> @tchalvak This strtolower is redundant with the call to lower() inside the query. I think we should go with one or the other but not both. Do you have a preference?
>
> —
> Reply to this email directly or view it on GitHub.
>


*sure*

> On Jan 24, 2016, at 2:20 PM, Beagle <notifications@github.com> wrote:
>
> @tchalvak Could we encapsulate these inside a class as static functions and give it a namespace?
>
> —
> Reply to this email directly or view it on GitHub.
>


Hi Paul,

I ran into this problem during the development of a new service, as you
know, the Contextual Mail service.

The application uses Liquibase to migrate DB. During startup Liquibase
tries to verify DB by accessing database metadata, which caused the
StackOverflow error.

Regards,

Paul Smith <notifications@github.com>于2016年1月25日周一 上午9:45写道：

> Could you describe how you ran into this StackOverFlow? I can see what you
> mean here, but since this code has been in production for *years* I'm a
> bit weirded out by making a change without understanding a bit more about
> how you found it !
>
> Logically I can see how the invocation of the Proxy ought to then just
> come back into this same method causing an endless loop, but I'm not sure
> why it doesn't do that in Babylon right now... :-|
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/whimet/parfait/commit/9c545742e9b3dec0719f79345cf12348300ef678#commitcomment-15639057>
> .
>
--
--Yanhuili


Update


Prefer to encapsulate CLI logic into a class. For example, the only code in your bin executable should be:

```ruby
#!/usr/bin/env ruby

require_relative '../config/environment'

TicTacToeCLI.new.call
```

And then you'd have a file: `lib/tic_tac_toe_cli.rb` with:

```ruby
class TicTacToeCLI
  def initialize
  end

  def call
   # Your game logic
  end
end
```

The code in your bin would generally just go in `call`.


Great update.


keep this line in the code


Consider using ternary operator here
`opp_token = self.token == "X" ?  "O" : "X"`


This method is *very* long. Try to break down the logic into some smaller helper methods.


Good catch. I did test without providing any `modelPropertyNaming` and it worked. Maybe my test was not correctly performed. Will test again to confirm.


以前からそういう使い方があったかもしれませんが、途中で方が変わるのは、混乱を生じる可能性が高まると思います。このあとlast_event_idが使用されるのが、一箇所なので、そこだけint()でキャストするか、別名の変数を定義したほうがよいかと思います。


ssss


aaaa


sorry i am using my fork !_!


did you just call me a noob on your public contributions?

![screenshot 2016-01-24 17 34 06](https://cloud.githubusercontent.com/assets/53573/12540631/b6ca4f60-c2c0-11e5-9ed9-ca4d357f3dde.png)


did you just call me a noob on your public contributions?

![screenshot 2016-01-24 17 34 06](https://cloud.githubusercontent.com/assets/160713/12540639/d531bfce-c2c0-11e5-9e09-588db3363735.png)


Gg bro!


fix #3


UI looks awesome! 😀


Yes, you're right, twpark. I'm going to restore back the old functions along with the new.


@afragen I think there's another way to do this, where it adjusts the counter too. You know how the main "Plugins" menu shows the number of available plugin updates?


i <- NULL   #initialize the inverse


 +	uint32_t st_atime;  /* Breaks in 2038 */

No it doesn't. Like the best rock bands .. it's unsigned.


Warning, not checked yet, but it seems to me like there were more style dependencies shared via the mod2 class between mod2 & mod2b cf all those here: https://github.com/njoyard/the-law-factory/commit/1715b0d055a574413a46f9a9a66aac6ba91ffbf3


May I ask how to set up health Monitors to pool?


You need to remove the definition of `is_endian`. See https://ci.appveyor.com/project/briansmith/ring/build/1.0.401/job/pr8wpxn161s10dy2


# sign-off


use buffer-equals on npm to get node <4 support


@keithbostic, unfortunately, no -- that was my first thought but there are many uses of `WT_WITH_LOCK` that don't assign to `ret` in the body.  In particular, there are many uses where the body is not an expression.


@bqpd: does anything need to change for pip? I'm seeing `gpkit.egg-info/PKG-INFO:Home-page: https://www.github.com/convexopt/gpkit`


nice! thanks for this followup. doesn't look like it was as simple as the README made it out to be due the bugs, but it's super nice to have the ability to test RiakCS on travis moving forward


if you're feeling fancy this can be done with .every on the array.


vu qu'il ne sert que dans navettes et n'a pas de tempalte dédié je l'aurais plutôt inclu dans navettes.js pour moins éclater


Wouldn't it be better to do this on `CheckIgnoredDecls` pass?


oops, thanks


:100:, just got bitten by this.


Since 0.14 is out tomorrow, we should delay it more


I'll also invite more people


vulnerable to SQL injection


Was this an immediate reject? I'd be a supporter for this ep.


use r_sys_getenv to make this hting portable


I found that the default value ("CamelCase" in this case) is not set and therefore line 67 is executed.

I've updated the code to remove the default value for now. Will look into later on why defaultValue has no effect.


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


pourquoi avoir laissé le [] ici seulement? oubli juste?


Yes, that's what I was suggesting.


versioning is complicated, much more complicated than just putting in a number and hoping for the best. many formats have learned that the very hard way.

personally, i tend to think that pretty much nobody is willing to define versioning in a way that really works well on the web, i.e., to design versions as loosely coupled with the built-in ability for old code to process new content and vice versa. if you don't do this, then versioning becomes brittle and versions pretty much start become different media types altogether, just anecdotally connected through the fact that two different incompatible things happen to have the same name.

at the very least, my vote would be to say that if versioning should have at least a minimal chance of being useful and practical, then a version number should be mandatory. if it is not, then version sniffing becomes required and the value of explicit version numbers is close to zero.


y a pas besoin de lui dire ou lire les variables de config en fait ? je comprendrais jamais certaines magies d'angular...


@sahat I'm generally worried about a patch release which pulls down a new major version of `handlebars` (its main dep) breaking apps. I'm more in favor of rolling out a v3 — and if we do a v3, then I think we should consider the peerDeps discussion since it might be a better path forward… but I'm also fine with no peerDep.

What you do think, should we be safe and make it v3?


Yeah, let's keep `object.assign` as it's more correct since it covers the case when ES6 Symbols are used.


Crypto class, sure.

> On Jan 24, 2016, at 1:26 PM, Beagle <notifications@github.com> wrote:
>
> @tchalvak Could we encapsulate these inside a class as static functions and give it a namespace?
>
> —
> Reply to this email directly or view it on GitHub.
>


Out of curiosity—other than being more explicit, why is this actually better?


Good point.

> On Jan 24, 2016, at 1:25 PM, Beagle <notifications@github.com> wrote:
>
> @tchalvak All the variables being rendered by the template need to be escaped.
>
> —
> Reply to this email directly or view it on GitHub.
>


@RyanWittibschlager you will need to run "npm install" again to get the new dependencies. Apparently  a different port number counts as a different domain. Had to change how the files were served up. Added an Express server and a different file watcher. Can still can be run with "npm start"  and will auto reload the server but it doesn't auto refresh the page for you. You can run the old way using "npm run start-old" to get auto page refreshing but the API calls won't work even if the other server is running due to the different domains.


Hello @nmyahdmmm,

thank you.


Hashed nonce has a constrained length.


I couldn't get any difference in performance on trigrams on a 500MB file using vector vs deque. Stringstream was slower.


Titre II/Chapitre Ier - Environnement ouvert.md

Sous-Amendement 873 (Gouvernement)


Why? I think `force...` is better since it indicates that this is an unsafe method that can fail hard.


Amendement 704 (Belot)


забыла закрыть тег ) <article class="item">


建议改成clientId


Cool!


Waarom een `args = nil` hier? Het wordt niet gebruikt.


Amendement 688 (Erhel) et 705 (Belot)


Besides that one comment I posted, commit is approved.


Amendement 689 (Erhel) et 706 (Belot)


感謝~已修正


This fork `omake-fork` is for cross comiplation of MinGW OMake in Cygwin:

* The final binary `omake.exe` is MinGW native.
* Build system is Cygwin: build is by `make` of Cygwin.
* Using a cross C compiler available in Cygwin: In Cygwin64, it is called `x86_64-w64-ming32-gcc`.
* OCaml compiler is MinGW native.  I build it by myself but I think you could use http://protz.github.io/ocaml-installer/

Many OCaml software build scripts assume Unix. Porting them to non-unicies is very time consuming I am afriad.


@sourcefrog I would say this fix makes the code a little less straitforward. The binary search algorithm I'm using has a certain property - it returns a position in the array. The position is either the left most value we are looking for, or the position before which we need to insert the value. So, it seems to me much more natural to verify that the returned value is what we need rather than modify the standard algorithm.


Maybe refactor these three lines into a helper function (they are repeated in many tests)?


If you use `User.create` as you did on line 7, then you don't need to call `save` as well. `create` does that.

Instead you could write `if @user.persisted?`


Añadido soporte para SLES


I amended the last commit


Misleading comment. Looks like it actually works.


I find this bug, then I find the solution there.Thank you!


Done


The result at ultrashort tc is wrong, of course. Here the correct one:
```
Finished game 5000 (SF-Master vs SF-MC1): 1-0 {White wins by adjudication}
Score of SF-MC1 vs SF-Master: 1361 - 1220 - 2419  [0.514] 5000
ELO difference: 10
Finished match
```


Just a thought: shouldn't the calloc args for this and the next line be reversed ?

Size-wise that seems the same, but the layout may matter.


это, вероятно, самый важный заголовок на странице. Поэтому используй здесь h1. не смотри на первоначальный внешний вид тегов - h3 меньше по умолчанию размером чем h2 - потому что размеры можно поменять далее в CSS, а семантику, значение, важность тега нет


@ophian It would be better to perform this check with the USERLEVEL_EDITOR etc. checks instead of its integer values


Correct me if I'm wrong, but now that we have `JvmAssembly`, all those things share a single concept for compiling sources, and a single place where the toolchain for compiling is used. So I think that when we will make one use case work, the others will too.


OK, done


лишний div. можно просто добавить класс zagolovok к тегу h3 (который на самом деле должен быть h1)


Ok, seems to be nice. Good job ;)


Misleading comment.


`xsprintf()`


Looks like a remnant of a previous incarnation.


`inet_addr` is deprecated in MSVC:

> warning C4996: 'inet_addr': Use inet_pton() or InetPton() instead or define _WINSOCK_DEPRECATED_NO_WARNINGS to disable deprecated API warnings

There *is* a `inet_pton()` function in MSVC, though it's a little obfuscated: https://msdn.microsoft.com/en-us/library/windows/desktop/cc805844(v=vs.85).aspx

>The InetPton function converts an IPv4 or IPv6 Internet network address in its standard text presentation form into its numeric binary form. **The ANSI version of this function is inet_pton.**

So, since it seemed the other platforms were already using `inet_pton` (in the #else), it assumed it would be okay for MSVC to use the same code path.

Quickly googling, apparently in MingW the function is `inet_pton4` instead. I guess we should put that in a MingW specific #ifdef.


Четыре div подряд тут не правильно - нужно разделить большую и маленькие картинки - так как у них разное позиционирование.
Оставь большую картинку как есть, а маленькие сделай лучше списком ul>li>img


Well, the thing is that this whole class structure exists only to create the zigzag segments; the code for generating lines infill makes calls to the zigzag connector processor. When you only want the lines infill, it makes calls to NoZigZagConnectorProcessor, which doesn't do anything.

Should I call the super class MaybeZigZagConnectorProcessor ?


I think this is a bit overly optimistic. Version is usually a lot more complex.

I would at the very least say either

a)	The member “version” MUST be present and contain the value “1.0” for this version of the specification.

or

b)	The member “version” and SHOULD be present. If it is not present then the value ‘1.0’ MUST be assumed.

potentially with some verbiage on the meaning of the version value (e.g. following the usual industry standard approach of major/minor; with major versions breaking compatibility and minor version being strictly backwards compatible; e.g. https://apr.apache.org/versioning.html).

As that helps implementors not make things (too) brittle.

Dw.


это не абзац, абзац всегда самостоятельное (неслужебное!) высказывание, законченное по смыслу. здесь можно span или div, например


change to commit airport


How does the `PasswordBrokerManager` end up with the correct user provider?


1. Misleading comment.
2. Probably 'ResolveTestGenerated' is more suitable to test this particular fix.


Looks good to me!


и тут нужен container


In our construction (z axis upwards) and global frame as in figure:
![img_0324](https://cloud.githubusercontent.com/assets/10923418/12548045/f9649be4-c353-11e5-87e4-15f3e6744a0a.jpg) I think lines: 1:16-37-38 are ok.  What do you think about?


Thanks to Torvalds, i love Linux.


Yesterday evening I added exactly the same fix you done few hours after me (about that bug with wood counter). About perk change I humble bow before you to notice that unnneded + ^^


'Abstract' Unix sockets have their own namespace that is not visible in the file system ('man 7 unix'). I'll clarify the comment.


I thought our formatter would take care of this. Can you include a space before and after equal signs?
And for multiple parameters, can you also include a space after each comma?


блоки жилье-еда-сувениры лучше сделать списком, поставив их в ul и заменив div каждого на li - так семантичнее (это же действительно список из 3 элементов!)


@rubenvp8510 think that this dep on `hawkular-bus-common` can perhaps be removed too. Could you please try without it?


Was there a reason to comment this out rather then delete it?


wm de ai


Plz. move `</hawkular.cmdgw.itest.run.dir>` on the previous line


`hawkular.cmdgw.itest.mgmt.user` and `hawkular.cmdgw.itest.mgmt.password` can be removed, I guess.


у одинаковых по оформлению блоков класс должен быть один и тот же. т.е. везде просто reason, без цифры


We do not have `alerts` in our itest distro. The prop can be removed.


Ah, now I get it: it is to avoid a TOCTOU problem. `datafrm` is in shared memory so it could be modified after the hash has been verified. I need to double-check the decryption code because I think I have overlooked this problem.


We do not have `metrics` in our itest distro. The prop can be removed.


Would you like to see rebase or comment right here?


здесь бы подошел скорее article. div тоже можно, но он менее семантичен


cc @ozbek


If you call the iterator variable `alias` or `alias_iter`, I think it makes the next few lines more readable.


@cleverpanda regular armor, too, has no texture, no armor mod has no texture, but in the older version of the armor textures from there (the one that has already been added, leather, metal, diamond).

The log Forge says that it can not download the file texture, despite the fact that he is there, for example iron armor.


лучше type="date"


这里 _client 未初始化，还是 nil，会不会有隐患？


I'll elaborate on the FIXME then.


Sounds great @Xav. One point: would it be better to say: only honours integer note values in the description rather than only honours whole tones. It will play semitones :-)


лучше type="number"


и здесь тоже number


We tend to ignore any line length limit for `pom.xml` files. The whole `<dir>` on one line would be better.


I get an error "undef, erlang, timestamp' is this my erlang version?


We tend to ignore any line length limit for `pom.xml` files. The whole `<outputDir>` on one line would be better.


что это?


We tend to ignore any line length limit for `pom.xml` files. The whole `<outputDirectory>` on one line would be better.


OK, I admit that it's a bit wasteful for what it actually does. :D I always find the function-pointer tables easier to read and use than code: code has bugs, data structures usually don't; switch statements can differ on each line, but the arrays are always the same.

I'll move this into a separate function and replace it with a switch statement.


If compiler doesn't support NRVO then there will be just copy (in return).


(sorry added above to wrong commit to start with...deleted the other one)


We tend to ignore any line length limit for `pom.xml` files. All props uder `systemPropertyVariables` should be one per line.


This commits breaks my code on iOS, I'm extending ASMultiplexImageNode. I don't really understand why, but I get this error:
```
Undefined symbols for architecture x86_64:
  "_OBJC_CLASS_$_ASMultiplexImageNode", referenced from:
      _OBJC_CLASS_$__TtC10ios_nrc_nl25DefaultMediaRenderingNode in MediaRenderingImageNode.o
  "_OBJC_METACLASS_$_ASMultiplexImageNode", referenced from:
      _OBJC_METACLASS_$__TtC10ios_nrc_nl25DefaultMediaRenderingNode in MediaRenderingImageNode.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
```
I'm extending ASMultiplexImageNode in swift with DefaultMediaRenderingNode and MediaRenderingImageNode  is a subclass of DefaultMediaRenderingNode

I'm using Carthage to build ASyncDisplaykit


section должен содержать как минимум 2 элемента, это "список с заголовком". но александр першин относится к этому снисходительно, так что на твое усмотрение


Great idea!

Not sure if its possible, but maybe allow for a cancellation of the installation to do the updates first? If you do a clean Raspbian installation first and then fire up the pihole install script immediately you could benefit from doing the update/upgrade first and then install the pihole script.


One line


one line


I think it's better to amend to the current commit message


см. выше комментарий про p. здесь лучше span


Generally, we write `(*it).` as `it->` - same effect, but fewer parentheses so a bit more readable. I've checked and it looks like `->` is usable for iterators.


In order for these to be deployed on installation you need to put input.map into skeleton.zip and update it.


non -> none


You repeat too code with the BuildRealTimeX() functions. You can reduce the number of function in one, or minimum two


I'll add a comment about this. |daemon| would fork and the parent exits. I don't think that Android's init system would handle this correctly. It would probably tell Gecko that the daemon isn't running, while it actually is in the child process.


Can be improved by using `yield sum([<a list comprehension>])`


TeamCity ArachNGIN :: Build [Build 4](https://tc.phome.cz/viewLog.html?buildId=14&buildTypeId=ArachNGIN_Build&guest=1) is now running


Unusable to create migrations.

Assuming a config of ``$config['paths']['migrations'] = 'D:/modules/*/{data,schema}/migrations';`` and no ``D:\modules`` path, the response to attempting to create a migration is:

![image](https://cloud.githubusercontent.com/assets/12801/12548056/b534a3cc-c34b-11e5-8b7c-c341cc219777.png)

If a complete path is available, the following responses are generated:

![image](https://cloud.githubusercontent.com/assets/12801/12548108/051e7142-c34c-11e5-9585-39b01c1fd304.png)

Whilst using glob to retrieve migrations is entirely possible, there doesn't seem to be any mechanism to create migrations in specific paths.

I would suggest that if multiple paths are declared, then ``\Phinx\Config\Config::getMigrationPath()`` could/should get all the paths and present them as a choice during the migration creation, allowing the user to store the migration in the appropriate location.

Other potential options:
* Allow a class implementing ``CreationInterface`` have the ability to declare its preferred path. As the template generating classes are created by the user, I think it would be OK to allow them to be able to declare the location of the generated file. This could be by adding new methods (``string getMigrationPath()`` Returns the required path for the migration file of this class of migrations).
* A ``--preferred-path`` option that would override all paths.


:+1: for `with open()`, itertools, set, yield


TeamCity ArachNGIN :: Build [Build master-1.0.1-48-gd652bd1](https://tc.phome.cz/viewLog.html?buildId=14&buildTypeId=ArachNGIN_Build&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:45


TeamCity Gitversioner :: Build [Build 7](https://tc.phome.cz/viewLog.html?buildId=16&buildTypeId=Gitversioner_Build) is now running


I find it convenient not to have to update a cross for a pip requirement that we can cross compile. Pip already handles the multiple versions.
I agree for more complex dependencies or ones that require patching we can go the cross way but otherwise this is pretty straightforward for most common use case.
Here is the fix to force cross compiling extension with tornado: https://github.com/SynoCommunity/spksrc/commit/85efee8ccb85c08f7e697860f1e1a1f0a989a3b9#diff-109ac4a13b9676ba10fb0188ef5291bbR29

For the PIP thing, we can set it in python-cc.mk and do the eval like I did [here](https://github.com/SynoCommunity/spksrc/commit/8b42131bcab7c95e83c66a4c5f827a6e578b17c6#diff-a1a98d19209b47d3cc15cb2cd91a8827R36)
I think it'll PIP_WHEEL will then expand with the correct PIP then.


TeamCity Gitversioner :: Build [Build master-1.0.1-4-g8ad3c7d](https://tc.phome.cz/viewLog.html?buildId=16&buildTypeId=Gitversioner_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:00:09


LGTM :)


Sure, but we don't need a copy. It's incorrect behavior for us


That's a fair shout actually. So insert a "Would you like to continue with the installation?" prompt at L312?


When the end of the data stack is reached or when the initial data stack is empty or not long enough to need infinite scrolling, these lines trigger an incredible amount of `needLoadMoreData()`calls.

The angular app then hangs.


Well, adding this fix still adds the fragment to the Activity which shows a white screen. I have an Acivity where I have a Button. On Button onclick I am attaching the CameraFragment below the Button. Is there anyway to show a Toast message when this Exception occurs and restrict the Fragment to be attached ?


I moved the implementation details to the cpp side and explained that this only roughly describes how ZigZag works. The thing is that i wan't to give an idea of why there are certain calls to ZigZagProcessor within the code of generateLinearBasedInfill...


ok, will do.


Yes, in that case if you see the notice and think yes that might be a good idea you actually can. Not sure if you can do it like many other questions (Y/n) so default Yes and otherwise N for no (where the script needs to quit ofcourse ;-)).


noiiiiice


Unterminated string, missing quote...


nice! Lekker bezig met die randoms :smile:


it is not dependency, it just package version field :)


I'm pretty sure pkeyutl command line utility stayed the same... And that's the man page we're editing


Yeah, right! Thanks.


@paternal : après réflexion, tu as raison: cette PR n'a pas grand chose à voir avec tout ca.

En revanche, j’apprécierai qu'on le fasse après que cette PR aie éte mergée (par flemme de résoudre les conflits ;-)


Could also add bonus feature Rspec feature test


Andreas, once you're finished testing the files for its current optimization potential I recommend to reverse the approach of optimization configuration: Place O0 where it is absolutely necessary and leave the others to use the default optimization set by the build environment.

Background: O3 is not necessarily the "best" optimization setting. It is very aggressive regarding inlining etc. (good for performance and detecting loop issues due to loop unrolling)  but O2 or O1 might be almost as good performance-wise but are saving more memory size due to less agressive inlining/loop unrolling.


Maybe

    from, _, to, _, distance = line.split()
    instructions[(from, to)] = int(distance)

etc


These handler arrays always have 256 fields, in order to avoid the range check. Adding memory for data structures to avoid instructions usually seems like a good trade-off. The public callers below use a more obvious function declaration.


It's also an array with a fixed size of 256 entries.


I was trying to put a spinner in the action bar and had errors, so I tried shrinking the action bar and placing a spinner directly on the layout. It looks ugly that way though. I fixed the error, so I'm going to change it back tomorrow after I clean up the files.


Nice commit


thrilled that you caught and did this in a loop!


lgtm


The lengths and types are included, see https://github.com/tomato42/tlslite-ng/blob/master/tlslite/messages.py#L177

The RFC refers to lengths and types of the Handshake protocol, not to the Record Layer protocol lengths and types.


Laravel handles that part: https://github.com/laravel/framework/blob/5.2/src/Illuminate/Auth/Passwords/PasswordBrokerManager.php#L71

```
$this->app['auth']->createUserProvider($config['provider']),
```

When your auth config is correct, this will return `doctrine` (or any custom user provider you might have defined)


This check is covered by the two cases below.


Yes.


I understand that breaking changes are frowned upon since considering that version management can be a bit difficult. However creating and maintaining a custom transporter that timed out after some time was creating more problems than it was worth. It was originally put in there for a very specialized use case for the first project to use it.

In the future, how would you suggest handling a breaking change in the API? I'm still fairly new to maintaining a project used by more than a handful of people.


what r u doing /抠鼻


Remove dead code


check here


This is covered by the two cases below.


`Uncaught exception 'InvalidArgumentException' with message 'Identifier "orm.em" is not defined.'`

It doesn't work out of the box :( My app is configured


Won't regular `==` work here?


are you sure that removing absolute path will work properly all the time?


LGTM


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


我也看不懂  哈哈哈哈哈哈哈哈哈哈哈哈


eh?


don not understand why you jave remove this. Also there is nothing about the changes in ini file in commit message


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 224](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=710&buildTypeId=XbimEssentials_XbimEssentials) is now running


This should be protected not private.


@anosach-luxoft, could you please next time shortly describe what is fix about in commit message?


Is this change also related to the CVE mentioned above? I'm running Ruby `2.2.3` and I'm not sure I want to downgrade.


removing absolute path?


what property of the `EdgeMetadatas` is this testing?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 224](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=710&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:01:33


@JustAkan 인식이 안된다기 보다는 볼륨을 조절할때 락이 걸리는데 (커널 내부적으로 볼륨 버튼이 여러번 눌리는 현상 방지) 이때 마지막에는 볼륨 버튼이 눌리지 않고 진동만 울리는 현상이 있습니다.
진동이 울리는 시점을 조절해봐야 할 것 같습니다.


Maybe some comments on why you've picked this specific test data


I have remove it because it is hardcoded linux path. I can add it to description


Comment (or some other way to tell me) about what you're expecting to test here, please.


should be jenkins@taspar's public key. doesn't matter much for localhost though


a


The scan failed.
Included commits

Commit ID	Owner

84652d046be449561880fdd008307d81cc08a6d7	@BearSpace


We use Python 2, so print is still not a function. But in general you should never use print in this. What you should use is logging module. Here is 3 lines of sample code, but you can see it in other files how and where logger is initialised.

The rest of the things look ok.

```
import logging
logger = logging.getLogger('booktype')
logger.exception('Some kind of error')
```


@nyalldawson I think this method should have ```/TransferBack/``` annotation as it moves the ownership from c++ to python


$ curl -is http://mobile.kci.pangea.pub |grep Server
Server: nginx/1.4.6 (Ubuntu)

provisioning for apache doesn't make much sense


I don't think this is related. See https://github.com/pkp/pkp-lib/issues/1079#issuecomment-174466409


Need a check to see whether the plane is still there


> provisioning for apache doesn't make much sense

well, was copy of publisher-nci but yeah, will edit it.


Is this on purpose or just a missing one?


Add test of weather functionality


I'm just going to concat current working directory path with `app_config_folder_` as required. `MakeAbsolutePath()` call is unneeded


My comment


버전을 노출하는 것은 보안상 문제가 될 수 있습니다. 최근에 발견된 보안 취약점이 패치되지 않은 사이트를 누구나 쉽게 찾을 수 있게 되거든요. XE도 예전에는 버전을 노출하다가 이 문제 때문에 지운 것으로 알고 있습니다.


+msgstr "Pressen von links"
klingt merkwuerdig... in powerpoint wird das mit "Einfliegen von links" bezeichnet.
+msgstr "Einfach Vergrößern" sollte "Einfach vergrößern" heissen.


Updated comment based on https://github.com/qgis/QGIS/commit/8ad6ca08fe2556f787ef83cebaf11d93d16a25fa#commitcomment-15646270


Add test where plane succeeds in landing.


I think it would be nicer to pass the path into the ctor instead.

Less `dirname()` and less confusion if/when the class is being moved.


No it is on purpose - the `strError` is not needed and the `message` property is to be used instead just like the `StringValidator`


I don't think you need this because login.php is in templates/spacelab and overrides this file in the root.

That's the change I made yesterday to avoid touching the root. Have you seen that?


I'm just curious: Would you replace this normally with service objects and leave it for now just for "simplicity"?


ola ola


Good shout @englercj! Will tweak..


this is causing error "z_db error {error,error,<<"42601">>,<<"multiple assignments to same column \"creator_id\"


OK, this is the correct change.


You need to switch to spacelab here if you want to use that one instead of flatly.


User with economic role must have authorization to see economic grid
i suggest to uncoment line 21 .. 23


Unused methods.


This solution causes warnings in compileTranslationTable.c and logging.c. Maybe because they don't include config.h? My suggestion would be to keep these defines in the NMakefile but to add the explanatory comments from config.h there too.


I think you can delete this


I was browsing forks and I just want to let you know that if you want to fix this ancient bug, just do `pip install --upgrade discord.py`. This was fixed in v0.9.1, we're in v0.9.2 now. The master branch is outdated.


I think the original caveat with `@Each` remains:

> the word is already used in a lot of places as a single level loop construct (`array.each { ... }`, etc).

My vote would go for `@All` (recursive by default and equivalent to `@All(descendants = true)`) and `@All(descendants = false)` for direct children.


Bug. Some config options were being left out.


maybe leave br? or something?


Refactor with a fluent(ish) API.


OK, LGTM.


I had to move this within the generateRequestURL function to overcome errors I did not understand (new to iOS-Swift)


phi_convict_threshold cannot be less than 5


actually respected sir i have written there bottom of page that please do
correct this i am not getting this problem.

On Mon, Jan 25, 2016 at 3:08 PM, AcadGildProgrammers <
notifications@github.com> wrote:

> ishu4.java is incomplete. Please complete it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coderishu/java-assignment2/commit/13d8c1103d71ae8239348ce3aedc8620a3b2dae1#commitcomment-15644919>
> .
>


Не совсем понял, убрать прям обертки?  или именно article. Если убрать обертки, то как же блоки потом позиционировать?


<3, hate waiting Ctrl+Tab to show up 1st time, tnx!


Typo: _retrieve_


Yep, I like this.


Flaky assertion, can be true or false depending on thread scheduling.


@wei2912 Can you look at this: https://docs.travis-ci.com/user/deployment/google-app-engine and enable deploying on your side? Windows devices don't work well with the required commands.


Not the best solution to this problem: renaming the class ? (Checker, Rule, CheckTemplate) or changing a bit how the test is done ?


-уровни автояркости как на row и 1544dev


コインを取るとスコア加算 #64


ok


We cannot have dependency on log4j in camel-test.
Camel is using slf4j-api as the logging api.


I guess we shouldn't lint on reexports?


I know it's a bit too late now, but can you @Giantaxe make sure your commitmsg is better formatted next time, please?

http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html


If the token is invalid users should get routed to the login view anyway. Did that happen for you? If that's what happened for you then I think we're good. Otherwise, in general when a view/state/route is loaded perhaps we should validate/clear the cookie then.


Anyway, I listed this in #533 so we can get this properly bikeshedded soon


can you avoid creating the hasset and only create it if its in use


This small change breaks a lot of stuff...

Because feedfinder is imported in setup.py, it requires all it's dependencies (requests, beautifulsoup, etc.) before running setup.py itself... thus before installing the `install_requires` section.

It also breaks installation of other packages that depend on feedfinder2.


ok (but I still thinks that the composer in chatter should be reworked)


I was just lazy.  Fixed it!


das sieht schon viel aufgeräumter aus!


Code/FireStormHR/Python newest pygame version/PythonApplication1/PythonApplication1/FireStormHR version.pyproj

wat is deze


done ^^
Didn't pay enough attention at the time to refactor all...


Perhaps create new methods for different landing conditions (SRP).


You can add .settings, .project and .classpath to your gitignore to avoid them being submited to the repo.


beta 0.1


I think it'd be a good idea to move the 'generate' function to this class and make the rest of the member functions private.


niet, tovarisch!
Всё это наследование нужно для того, чтобы не писать вот таких case'ов. Ты просто определяешь поведение (в данном случае — выдачу информации о кине) в классах-наследниках.


Please remember to use curly brackets even for one lined blocks.


Are you running the newest Zotonic?  And if so, on which table to you see these errors? The edge table?


You can tell I'm used to working in an IDE that closes off quotes automatically... ;)


if(ertekek[5] >= 2) then -- Ha végzett az ertek 5.


Hi,
it should be >= 2.1.0 (acme-client have such requirements), I'll correct it
this evening.
Sorry for inconvenience.

kind regards,
Lukasz

2016-01-25 11:36 GMT+01:00 halo <notifications@github.com>:

> Is this change also related to the CVE mentioned above? I'm running Ruby
> 2.2.3 and I'm not sure I want to downgrade.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lgromanowski/letsencrypt-plugin/commit/bfca05f28924e20113f95b4a50d0b4e99de0067d#commitcomment-15646161>
> .
>


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


The intention of the condition is to check whether the dimension has been defined (i.e. explicitly set) as opposed to calculated. If the value is calculated there's still a possibility that it'll need to be changed (further stretched).

Within the if clause we save the current dimension value from layout into dimCrossAxis, then the new value is calculated and if it's different, we redo layout of the node.

Do you see any flaws in this reasoning?


Having also added `geteuid32`, I'm getting the following panic (from `whoami`):

`panic: unsupported kernel trap: 0xffff0fc0`
```go
goroutine 17 [running, locked to thread]:
github.com/lunixbochs/usercorn/go/arch/arm.setupTraps.func1(0xaac37628, 0x106ed5b0, 0xffff0fc0, 0x0, 0x4)
        /home/odroid/go/src/github.com/lunixbochs/usercorn/go/arch/arm/linux.go:43 +0x168
github.com/unicorn-engine/unicorn/bindings/go/unicorn.hookCode(0x4451e0, 0xffff0fc0, 0x0, 0x4, 0x10735600)
        /home/odroid/go/src/github.com/unicorn-engine/unicorn/bindings/go/unicorn/hook.go:27 +0x90

goroutine 1 [syscall, locked to thread]:
github.com/unicorn-engine/unicorn/bindings/go/unicorn._Cfunc_uc_emu_start(0x4451e0, 0x81d0, 0x0, 0xffffffff, 0xffffffff, 0x0, 0x0, 0x0, 0x0)
        github.com/unicorn-engine/unicorn/bindings/go/unicorn/_obj/_cgo_gotypes.go:125 +0x34
github.com/unicorn-engine/unicorn/bindings/go/unicorn.(*uc).StartWithOptions(0x106ed5b0, 0x81d0, 0x0, 0xffffffff, 0xffffffff, 0x10649a30, 0x0, 0x0)
        /home/odroid/go/src/github.com/unicorn-engine/unicorn/bindings/go/unicorn/unicorn.go:82 +0x70
github.com/unicorn-engine/unicorn/bindings/go/unicorn.(*uc).Start(0x106ed5b0, 0x81d0, 0x0, 0xffffffff, 0xffffffff, 0x0, 0x0)
        /home/odroid/go/src/github.com/unicorn-engine/unicorn/bindings/go/unicorn/unicorn.go:87 +0x78
github.com/lunixbochs/usercorn/go.(*Usercorn).Run(0x106f0100, 0x1066c108, 0x2, 0x3, 0x106f4000, 0x51, 0x80, 0x0, 0x0)
        /home/odroid/go/src/github.com/lunixbochs/usercorn/go/usercorn.go:210 +0x1030
main.main()
        /home/odroid/go/src/github.com/lunixbochs/usercorn/go/usercorn/cli.go:168 +0x1600

goroutine 34 [syscall, locked to thread]:
runtime.goexit()
        /home/odroid/go1.5/src/runtime/asm_arm.s:1036 +0x4
````


This line can be deleted, but I would leave it in there for now for debugging purposes.


Not ideal but it works without issues now!


Oh wait, I'm sorry. That's not possible.
I thought your comment was on Infill.cpp

Moving the generate function from infill to ZigZagConnectorProcessor would move the same problem to some yet unexistent class, cause the ZigZagConnectorProcessor functions should then be moved out of that class...


No need to say sorry, I'm very grateful for the time you're investing. Thank you very much.


@dwaynebailey

1. current printf check doesn't go into semantics, type part in regex is defined by just `\w`. So it will pass `%q` (which is not supported). This is good idea in my opinion because C printf and Python printf type definitions are not the same (there are for example types `a` and `p` that are used in C but not in Python) and since checker is not aware where the string will be used it should be up to developer to watch for semantics. So back to `% s` this is not semantically aware test, but it seem to work in both C and Python https://ideone.com/xmNV4J , apparently it is just ignored.

2. I have real example which was actually a product of mistake but we are using pofilter exactly to catch mistakes. Text was like this `%(discount)s%% of the price` and translator translated it as `%(discount)s% du montant total de la réservation`. So one percent sign slipped away and instead of `%%` it became `% d`. Which became +1 parameter and formatting failed with exception (not enough arguments).

3. String is valid (well working at least) printf string. But most importantly - in printf specification a single percent sign can never be outside of placeholder - this is true for both Python and C. This test should fail because of number of arguments mismatch. (1 vs 0)


You could have used Assert.assertfalse()


Yea


Changed it into os:timestamp().  erlang:timestamp() is indeed an Erlang 18.x function.


This is unrelated with LOAD CSV and could be enabled again


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 225](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=711&buildTypeId=XbimEssentials_XbimEssentials) is now running


use while loop?


The commit message needs to be updated.


1. This one is passing `passes(stdchecker.printf, "%%$", "%%$")`
2. `10%` gives exception in Python if formatting is used but that is what printf check validates. In C `printf("10%") `doesn't work properly as well: https://ideone.com/VvBjGL


> any client where time matters (normally servers creating tokens) will have an accurate clock

@mattheworiordan the issue is RSC14d (`the token has expired *or* the server has responded with a token error`) -- ie clients check if the token has expired themselves, so time matters for all clients, not just servers. (If we want to change that and rely just on realtime telling you you've expired, now we have that working reliably, we can, but that's what the spec says at the moment)

@alex-georgiou I don't this is the best way to solve this. Logic here is only run when creating token requests. In particular, it isn't run for an `authorise` call for a library initialized with a tokenDetails object, but that situation would still run into the problem we're trying to solve when checking expiry (line 155).

Use of serverTimeOffset is already abstracted by `getTimestamp`, so perhaps we could put this logic in there, so every place that needs an accurate timestamp so uses `getTimestamp` will automatically benefit?


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 225](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=711&buildTypeId=XbimEssentials_XbimEssentials) outcome was **SUCCESS**
Summary: Running Build time: 00:01:31


Nice test.


ok


Regarding Adminstration->ACL->Advanced

gacl has a deprecated constructor in /var/www/html/ppemr/gacl/gacl.class.php on line 55

Smarty has a deprecated constructor in /var/www/html/ppemr/gacl/admin/smarty/libs/Smarty.class.php on line 64

Smarty_Compiler has a deprecated constructor in /var/www/html/ppemr/gacl/admin/smarty/libs/Smarty_Compiler.class.php on line 35


The issue was that I was already at the login view.


Remember to put numbers in a constant to avoid "magic numbers".


@miduku Here why "In"? Don't we want to describe what these box are? As I did only list its components for now, I can imagine why you changed this but I thought it would come more later on (mechanics, design, position in the room, etc)


Great work Seba!


ternary operator?

tihs.messages_separator_position = (!msg) ? 'top' : msg.id


ой. переусложнил ужасненько. `detect` is your friend!


corrected..


rotine --> routine


По некоторым историческим причинам в руби существуют константы `TRUE` и `FALSE` (равные, соответственно, `true` и `false`), но использовать их вместо литералов `true` и `false` — очень дурной тон.


Поправь отступы. Вложенные теги должны быть "глубже" родителей.


btw: There is no such thing as a Connector, there only is a ZigZagConnector NoZigZagConnectorProcessor is to be parsed as (No ZigZagConnector)Processor


Okay I did some testing, this solution has a bug
When a new composite is created it is not getting selected visibly, because it's bounding box width and height is 0.


PSR


@boazy - this sounds like a great idea. Can you describe the changes you are proposing, how *default is working etc? Could you also update the README file (it has a section on Android which would have to change I think) creating up-to-date instructions for both Android and iOS?


It's a reasonable assumption, sure, but it's still an unusual pattern for ably-js, which generally doesn't share state between instances of the library. I don't think there's any downside to keeping it as an instance attribute, besides a very occasional duplicate `/time` query (which isn't very expensive). Your test problem will still work iff you make the change I suggest above (so that the second instance doesn't have to rely on the first instance to set the serverTimeOffset)


Выражение, сравнивающее нечто с `true` или `false` методом `==` (почти) никогда не имеет права на жизнь.
`movie.watch == false` — это совершенно то же самое, что `!movie.watch`.

Правильный вариант к тому же позволяет упростить весь select.


Yeah, moving it to a config would be better, which is set by the host OS (default to `/mnt/root/var/lib/rce`) - it'll mean that the host switching to use `/var/lib/docker` or anywhere else would be easier


You could have used a for each loop
http://examples.javacodegeeks.com/core-java/java-for-each-loop-example/


> It consists of four boxes situated in a room, where any box can record one person's voice and transfer it to its next sibling box.

@miduku I think this formulation is even more complicated than  the previous one. Does this _where_ here really make sense?

> [...]after the chain of translating the message four time is over[...]

I'm not sure about that as well… Kind of complicated _chain of translating the message four time_ Maybe we could find something else…


This was intended for bootloaders more or less. Can you use this for the semi-static strategy? Please share 😉


Вот эту логику (выбора фильма по году) хорошо бы вынести как метод класса Movie.


There is no mci, is there?
I thought we settled on mobile.kci.pangea.pub and now either
mobile.neon.pangea.pub or mobile.nci.pangea.pub...?


Это стандартный элемент, называется "хлебные крошки"  .breadcrumbs.


this is the first time I use git hub so bear with me.


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


Its not a good practice to modify the input parameters, you could create a copy of it and modify that one.


indentation please.


On acceptatie server is 0.13.6.where it didn't work with rsc updates.  on my development is 0.13.x where all worked. So its fixed in 0.13.7 ? :)


Check the formatting of if statements in this file.


ok


Haha il puotuere di petrolio aiuterà me in mio script. Да!


You could have used ArrayList.subList() method to avoid this.
http://beginnersbook.com/2013/12/how-to-get-sublist-of-an-arraylist-with-example/


Can we change the name here to RestComm-jEIR ?


Don't forget to handle the case where they are different classes (a return False would be enough)


Is this ok? It looks like a line break is missing


Не, пусть прям в классах будет прописан вес каждого типа фильмов.


http://htmlbook.ru/html/form
кнопка отправки должна быть внутри формы. И форма должна быть только одна. Поправь выше по коду.


Check the formatting of for statement below.


looks good


remove code block, this breaks the numbered list in rendering (and not all bullet point below is a command, the commands are already between backticks)


well that's the behavior of this method; `load_default_backend` is just the helper method used to provide that behavior.  if i had `load_default_backend` do both then i'd have to duplicate code in that method and i'd have to get rid of this one


yep, will update


Exception : open/close menu function (Not Set Left/RightViewController)


Isn't defining it as `provided` / `optional` an option so that projects using Log4j as logging backend can rely on it?

The idea is that, in some samples provided by Camel, it is interesting to assert the log sequence as it provides, in addition to be an obvious test assertion, some sort of _linear explanation_ of the test scenario. If `camel-test` is not the right place, maybe a test util module for the samples would be a better place to share this JUnit rule. An other option would be to rely on the lower level `System.out` to be agnostic to the logging backend. Else I can drop the idea of asserting the log if that does not make sense :)


на самом деле нам не нужно проверять на boolean, есть же ===, if (a === true)


Czy przypadkiem to nie bedzie zawsze tak ze SuperMutex bedzie otoczony Arkiem? Mozna by docelowo zrobic tak by new zwracal od razu Arca, to byloby jakoś tak:

    fn new() -> Arc::Arc<SuperMutex<T>> {

        Arc::new(SuperMutex{
            mutex : Mutex::new(None),
            cond  : Condvar::new(),
        });
    }

:-)


зачем заменила все одиночные кавычки на двойные? :) это потом будет очень мешать принимать патчи, откатить, нельзя помиловать


yeah, well public url is mobile.neon.pangea.pub but stll internally it is referred as mci. similar to kci, dci, and nci.


надо использовать подход if () {} else if {}, иначе программа выполнит все одиночно стоящие if ()


Why is it always added to the `Hedge` layer?


No, I was wrong.


Mind I ask, why would you force usage of one and only one subdomain? I can very much imagine users would like to use a subdomain for their own installation (me included, except I'm using nginx anyway).


**[unitTests_js_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_js_branches/23777/testReport/) for this commit**


Хз, нужно оно или нет. Склоняюсь, что нет - условие выполняется первое, не доходя до второго.


гораздо проще поменять условия местами, сначала проверить a instanceof Array && b instanceof Array ELSE a instanceof Array, тогда не потребуется делать отрицание


It works perfectly for the basic falling back to default case, but if we had case "basic" it would make sure it wouldn't appear as missing.


got it, thanks for the explanation.


Check the formatting here.


/lk;lkk


Hello World


Здесь и дальше используешь span - строчный элемент. Нужно использовать подходящий блочный элемент.
Дальше при стилизации, тебе придется использовать дополнительное свойство для того, что бы изменить строчный элемент на блочный. Пересмотри все теги span.


ok


No the classloader will fail because log4j is imported.

And yeah drop the testing of log


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


Sorry Boi


They computation is rotated and therefore wrong if you use the lines 37-38. I checked every other matrix and the issue is here. The main point I think is that the G in the diagram above is rotated to be on the top right of the figure. The diagram also distorts the angle of V0. Check my diagram from the screenshot of Gazebo (experiments/humanFixedBase/data/frames_diagram.png) The main difference between this diagram and what Im suggesting is that all the robot frames (0,1, and 2) must be oriented with Z going into the rotation axis. I will upload a new diagram to explain what I mean. Anyways, key is that with this frame assumption, all four measurements are predicted accurately : i.e. accelerometer, gyroscop, Forceplate force, forceplate moment. With any other rotation there are problems created elsewhere. As you know R_G_0 is the most important frame for all subsequent computations...


Shouldn't this line be within the `#ifndef DISABLE_NETWORK` block, too?


Seems like you forget to run minifier


just test


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


Whoops?


From https://www.kernel.org/doc/Documentation/arm/kernel_user_helpers.txt I only implemented `kuser_get_tls`. Looks like they're calling `kuser_cmpxchg`.


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


@lucasr
This modification was required because in some cases we cannot use simple stacking even if child.layout[dim[crossAxis]] is defined. Please also see below.

However, I'm not sure if this is the best way to make it work while preserving as much of the optimization as possible.


The only reason I had decided on using stdcall was because it forces callee cleanup of the stack, and I had originally been using some assembly.  It really doesn't matter whether you use cdecl or sdtcall since the function never actually returns.  Rather, it simply kills the cloned process once the user function is called.

I had originally started trying to use #pragma statements, but, after looking up some syntax in the manuals, it did say that using __stdcall (or __cdecl for that matter) was effectively a "predefined pragma" statement.  The #pragma syntax, though, might be more explicit in this case.

I like the addition of the macro to simplify loading the arguments into the child stack space.

Would it be alright if I make your suggested changes?  I'll push the changes sometime this morning.


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


Lines are too long :)


unsorted using


Hi! Yes, I'm doing work in the driver, I'm close to have prepared statements ready (text queries and all other works, I have implemented text queries as prepare/execute, with prepare which doing nothing).


yeah, i'm not sure how this happened. Any hints or advices are appreciated here.


By example in https://angular.io/docs/ts/latest/api/router/AsyncRoute-class.html you could go with:
`loader: () => Promise.resolve(About)` to make sure that type `About` exists.


ERROR: Utility classes should not have a public or default constructor.


ERROR: Name 'fincorrectUtilsClass' must match pattern '^[A-Z][_a-zA-Z0-9]*$'.


ERROR: [ClassNamingConventions] Class names should begin with an uppercase character
http://pmd.sourceforge.net/pmd-5.2.3/pmd-java/rules/java/naming.html#ClassNamingConventions


ERROR: [MethodNamingConventions] Method names should not contain underscores
http://pmd.sourceforge.net/pmd-5.2.3/pmd-java/rules/java/naming.html#MethodNamingConventions


Probably when changing branches? Make sure you have 30273cd602cb4ad2dbe669417769a9c17452f597 checked out in `pontoon/intro`.


ok


Javadoc should document the functionality, so you should provide some details.
For instance it should be something like "Deletes a user from the store", "listusers" doesn't means anything, that is just the name of an internal variable in your code.


What about using a weather method instead of creating a new instance of weather? Just check it ^^


:+1: :+1: :+1: :+1: :+1: :+1: Awesome!


this is probably broken, I think it needs to be `... + [Net::HTTPConflict]`


Bloody hell I've been looking at a version where `PasswordBrokerManager` uses the `CreateUserProviders` trait.

Looks like someone spotted the same problem as me and PRed it https://github.com/laravel/framework/commit/023f94bca038a1e9dfa93dc4d9759a2324dcae92

Changed in 5.2.11

GRRRRRRR


See also #485


this branch can be merged in the previous if block


Variable names should describe what they contain, instead of "u" use "user" and it would be clearer.
Also, in a javadoc you should provide descriptions.

@param user The user to delete from the store.

Adding final dots to the sentences in the javadoc is a nice practice too.


Этот вопрос лучше адресовать разработчику скрипта.
Я всего лишь немного его улучшил, чтоб он принимал больше урлов и автоматически их парсил.

Почему именно это видео не скачивается - ХЗ. Похоже, что Гугл что-то там запретил ибо по клику на видео формат выдает 403.


@lauraluiz not sure where do have document on ready event here?


Dobry pomysł. Trzeba to tak zrobić.


@return should specify what is returning


on ValueError, line 62 seems to set wrong corporation on "user".


What is the purpose of the wait group here? We are not synchronizing go routines. The saving of an event is a fire and forget activity, completely asynchronously. That is why we copied the request context. There is no need for a waitgroup.

We want the request to continue on its way while the event is being saved asynchronously.


You can use != to compare something is different


This will block this go routine, which is not the behavior we want.


typo in method name


I would use a double instead of landing a "string".


You can interate using "for each" blocks


Mistakenly submitted two times, the last one submitted to the "c6bd9577" is nothing to change, you can ignore.


frameworks/base/services/core/jni/com_android_server_location_GpsLocationProvider.cpp:1630:1: error: 'Status' does not name a type

getting this error while compiling for generic mediatek devices , it need to be fixed soon


Emmet еще не ставил, но читал немного про него. Я так понял, это для упрощения и скорости верстки, так?


Yo, Mr. Stalker här.
Detta borde vara
```HTML
<link rel="stylesheet" href="css/master.css">
```


To reviewer: Please expand the code to see whole thing.


버전을 표기하거나 하지않거나 취약점 공격을 시도한다면 당할 수 있는 건 마찬가지입니다.
어차피 nginx나 php버전도 외부에서 손쉽게 알 수 있습니다. (버전 표기가 심각한 문제였다면 php측에서도 php버전을 알 수 없도록 패치했을 것입니다.)


we would really appreciate if you could open a pull request for the exec validation change. :smiley:


Yep, kom på det igår natt *efter* att ha deployat till GitHub pages, haha :D


Comment


AKA `itertools.product(('-', '+'), repeat=len(arguments))`


в седоне один контейнер выносите перед хедером и закрывается после футера


выноси лого как обычно перед нав, потом стилями мы поставим куд нужно


просто класс main можно ,контейнер вы выносим здесь он уже не нужен


удалить


форма будет играть роль обертки потому верхние 2 дива не нужны (максимум можно обернуть одним дивом но по сути он не обязателен)


а вот внутри формы делаем 2 дива filter-group внутри по заголовку но не h2, а див и группа инпутов


див  и эту группу также обернуть в filter-group


форму удалить здесь, форма одна зкроется после кнопки показть, сколько не было бы групп инпутов если кнопка с типом сабмит одна значит форма одна, бывают целые страницы с multipleform  где много групп инпутов и это все одна форма


див и группу стоимость со слайдером оборачивает в див с классом cost , на консультации напомни объяснить почему не filter group


див


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


див и дать класс


здесь будет иконочный шрифт где иконки разметь вместо ссылок и изображений


лучше hotel-item внутри такая структура: 3 дива- hotel-left-part, hotel-middle-part, hotel-right-part


изображение будет лежать в hotel-left-part


в hotel-middle-part идет сначала ссылка с название отеля , потом див hotel-info внутри него 2 дива hotel-info-part, в первом пишешь без обертки гостиница и ниже ссылка подробнее, в тком же диве только справа без обертки пишешь цену, потом ссылку забронировать


в  hotel-right-part 2 дива с классами stars и rating


@KalebStumbaugh Unit tests will fail when trying to access SimGame.game because it is uninitialized--there's no Platform instance. You can prevent this from causing trouble in the tests by putting a guard around the log, as I have in ... another event class I think, I cannot remember which. It looks like this:
```
if (SimGame.game != null) {
  SimGame.game.plat.log().info(...);
}


не нужен


лучше назвать footer-hashtag


Hm, hade uppskattat om du svarade liiite snabbare nästa gång. :laughing:


I take that very very very serious. This code is crap, too much copy & paste.


k

;)


I use Chrome, Version 47.0.2526.111 (64-bit). I saw the comment in the other issue-thread, and it solved by change the function requireDynamic to only require.


@abbradar Thank you.


@rullzer how is this supposed to work? If I do not explicitly choose the right size the image will not be returned.


lol kek


После этой заплатки перестанет работать пункт доп. настроек графики "Частота 60 Гц".


As previously stated, the Type field in the event struct should be of type Int, not an aliased type (ie. EventType).


PIL can raise other errors here as well:

```python
        except (IOError, ValueError, TypeError) as e:
            ...
```

There is also a warning that PIL might raise for DecompressionBomb images. It would also be nice to log the exception at DEBUG level.


Could you put the version into the derivation in the future, I'm working on a PR to add version to all derivations to be able to use the attr for package version checking.


This isn't  a good way to use a mock, you could have used a normal User object for these tests.
A mock should simulate a functionality provided by some external service, for instance it could return OK when you save a user to the database or disk without really doing it.
mocks, should mock some functionality and then verify that functionality was called using verify.


I thought that it would be useful for the user to know this information.
Okay I will move it as a TODO statement.


tohle muze vratit i false


pls donot verify this pull request,as it contains the same indentation problem


The invalid count would be a nice addition to expose via an eval rule, and is not present in the SA plugin :). Can you open another issue to add this to our plugin?

E.g. a message with a lot of images that are invalid/corrupted is more likely to be spam.


You missing deep buffer in audiopolicy.conf?


You are using strings as a weather condition, what about using true or false respond from a method instead and therefore get to stub the test.


Looks much cleaner!


Thanks! One item less in my *“TheFuck ToDo List”*! :+1:


This comes with a problem I had. The SSL certificate I had on the old domain did not support the non-www version of my domain. Therefore, I had to redirect any non-www requests to www to keep it secure over SSL. My intention was not to "force" any user to use my setup. It is more an "example" how to set up the .htacces. Im not very experienced with Apache configuratin. Is there a more "generic" way for that redirect?


I will, nice to hear about your work! You may consider spreading the message via Github's NixOS/nixpkgs-committers and/or mailing list to save yourself from more commits monitoring.


Implementations of services usually have the same name that the Interface but with the postfix "Impl" for instance this class should be called "UserServiceImpl". Service implementations should be in the services package.


tato funkce nevraci rovnou chybovou zpravu, ale bud vysledek a nebo false pri chybe


You can use "for each" iteration blocks, they look way better.


nechybi tu return? :)


Kdy? pokud je to v případě nějaké chyby, tak je to OK, spadne to do vyjímky jen..


В JS принято использовать одинарные. Лучше использовать те, которые были изначально.


This group already got removed once in master (https://github.com/withassociates/slices/commit/8dcb45db58a89d23760c26db0b1f94a057265706). Then you add it back here, then it gets removed again later in the branch: https://github.com/withassociates/slices/commit/a46d8ce15cfd3362a54b719b20b5e498c758581b. Could just not add it here in the first place?


Added and removed. Needs tweaking...


I added kuser_cmpxchg in 5534747a3de821ee57309adc15133f8dc698488d (untested)


ehh, ano :D pridam


Hi @matrinox, Could you remove the ```require 'pry'``` lines? (there's more than one) I don't understand how you're using. And I think it's an unecessary dependency.


Prijde mi lepsi psat misto $this nazev tridy


Why did you use a Proxy pattern?
Always try to keep the code as simple as possible.
Your code is missing the javadoc, please remember javadoc is part of documenting.


Sooo most of this comit is just making like "                       {" to "         {"


opravdu int?


I already submit my changes. It is better to use #pragma aux for internal CRTL function because it enable define function property how we need. For user defined function is better to use predefined calling conventions, because it follow OS default calling conventions. On first view I didn't analyzed your code in details, later I understand that only necessary change for __callfn is to define parameters on stack and set proper stack cleanup, that simple #pragma aux resolve it efectively (now function is __watcall with parameters on stack and stack cleanup).


I dont have a repo setup on my my local machine, working directly on github . If you could send me pull requests regularly, that will help me keeping my fork updated.
Thanks.


@anosach-luxoft , in OnAllHMIResponsesReceived() DeleteChoices() will be called only if there is error from HMI.


Regarding Administration->Other->Database

I seems to be working fine, I don't see any error

I am not sure if phpmyadmin have custom error reporting or it is hiding errors other than that I don't see any error


Furuse-san,
Thank you for your response!


vraci int http kod


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


You could add the new product at the end of the list and it will perform better.
Adding it in the first position requires moving all the other elements.


Yes - was a problem in the rsc update, introduced by the extra merge functionalities.


ok, it's for you to help you to get back to work with spacelab.

I'll push something more at the end of this week. This week I'm busy and I can just rebase from master and develop/2.4 .


I find this comment rather harsh. The goal is to have a common style across a project to increase code readability, not to judge what's right or wrong. It's a matter of consistency, not of preference.


One more shorthand... :) I think it is a bit confusing to use shorthand while other blocks in this file use different notation ```$(document).ready```.


You could annotate this as @setup, that way values would be initialized before each test case run.


Ah, thanks - was beginning to worry you were prejudiced  against `geteuid'`s ;)

Unfortunately it desn't work yet:
````
invalid fetch: @0x400001d0, 0x1 = 0x0
[memory map]
  0xffff0000-0x100001000 rwx
  0x8000-0x125000 r-x busybox-armv5l [exe]
  0x12a000-0x132000 rw- busybox-armv5l [brk]
  0x60000000-0x60801000 rwx [stack]
  0x125000-0x127000 rwx
[registers]
   r0 0x00000000    r4 0xffffffff    r8 0x00000000   r12 0x607feb88
   r1 0x00000001    r5 0x0012a920    r9 0x607fef50       0x00000000
   r2 0xffffffb8    r6 0x0012a8d8   r10 0x0000015c       0x00000000
   r3 0x400001d0    r7 0x000003e9   r11 0x00000001       0x00000000
[stacktrace]
  0x400001d0
panic: Invalid memory fetch (UC_ERR_FETCH_UNMAPPED)
````


"better" english suggestion


Regarding "Is php7 now working"

The error that I see most is "deprecated constructor" and it is on many files so we have to fix PHP 4 style constructors first and I can start working on it as you give me go ahead


That isn't a good primary key, you could have two teachers with the same last_name.
You can have the DNI as primary key, or use composed keys (I don't recommend this last option)


Thinking about this further I  came up with two changes. In app.js if we are already login, state change to main.dashboard. In djangoAuth.js if our token is deemed invalid at any stage clear it. This covers the login page because 'resolve' has djangoAuth.authenticationStatus attached to it. Thoughts?

app.js
```javascript
      .state('login', {
        url: '/login',
        templateUrl: 'views/login.html',
        resolve: {
            authenticated: function(djangoAuth, $state) {
                return djangoAuth.authenticationStatus(true)
                    .then(function() {
                        console.log('Already logged in!');
                        $state.go('main.dashboard');
                    }, function(reason) {
                        return;
                });
            },
        },
        controller: 'LoginCtrl'
      })
```

djangoAuth.js:
```javascript
        'authenticationStatus': function(restrict, force){
            // Set restrict to true to reject the promise if not logged in
            // Set to false or omit to resolve when status is known
            // Set force to true to ignore stored value and query API
            restrict = restrict || false;
            force = force || false;
            if(this.authPromise == null || force){
                this.authPromise = this.request({
                    'method': "GET",
                    'url': "/rest-auth/user/"
                })
            }
            var da = this;
            var getAuthStatus = $q.defer();
            if(this.authenticated != null && !force){
                // We have a stored value which means we can pass it back right away.
                if(this.authenticated == false && restrict){
                    $cookies.remove('token');
                    getAuthStatus.reject("UserNotLoggedIn");
                }else{
                    getAuthStatus.resolve();
                }
            }else{
                // There isn't a stored value, or we're forcing a request back to
                // the API to get the authentication status.
                this.authPromise.then(function(){
                    da.authenticated = true;
                    getAuthStatus.resolve();
                },function(){
                    da.authenticated = false;
                    if(restrict){
                        $cookies.remove('token');
                        getAuthStatus.reject("UserNotLoggedIn");
                    }else{
                        getAuthStatus.resolve();
                    }
                });
            }
            return getAuthStatus.promise;
        },
```


in other comment, I mentioned to not use the string but a true or false method to stub the test. If you had this method you might have a guard conditions into this if statement, the code would be shorter and easier for you.


You sure? I get that error address at the end of every busybox armv5l command but the command still works.

For me it prints this on the line before (tested on 125d90e64ad917b7ccd20b14c65295e133311b9a): `whoami: unknown uid 501`


OK


Nice work


Dude are you using git bash your photo ain't showing


Привет, да

Вот тренажёр. :)
http://epixx.ru/emmet/ <http://epixx.ru/emmet/>

> 25 янв. 2016 г., в 14:46, roman-melnishin <notifications@github.com> написал(а):
>
> Emmet еще не ставил, но читал немного про него. Я так понял, это для упрощения и скорости верстки, так?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/roman-melnishin/153965-barbershop/commit/1d181861f50208e272d32f4d49c5303ae073b8a1#commitcomment-15647607>.
>


for the future, is good to write the function definitions as

```
var searchFunction = function (query) {
  ...
};
```


fixes #5


You could have used a prepared statement for this, check this out: http://www.mkyong.com/jdbc/jdbc-preparestatement-example-select-list-of-the-records/


The offsets 0x274 and 0x278 are wrong, here. It has to be 0x74 and 0x78:

```patch
From 92790a2928950025fcf128c0df8ad8c5cb4593b4 Mon Sep 17 00:00:00 2001
From: Dirk Behme <dirk.behme@de.bosch.com>
Date: Mon, 25 Jan 2016 13:09:08 +0100
Subject: [PATCH] board: salvator-x: Correct SD0CKCR and SD1CKCR addresses

Both registers are at offset 0x74 and 0x78, not 0x274 and 0x278.
Correct this.

Signed-off-by: Dirk Behme <dirk.behme@de.bosch.com>
---
 board/renesas/salvator-x/salvator-x.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/board/renesas/salvator-x/salvator-x.c b/board/renesas/salvator-x/salvator-x.c
index 4982301..e94755e 100644
--- a/board/renesas/salvator-x/salvator-x.c
+++ b/board/renesas/salvator-x/salvator-x.c
@@ -54,8 +54,8 @@ void s_init(void)
 #define SD2_MSTP312	(1 << 12)		/* either MMC0 */
 #define SD3_MSTP311	(1 << 11)		/* either MMC1 */

-#define SD0CKCR		0xE6150274
-#define SD1CKCR		0xE6150278
+#define SD0CKCR		0xE6150074
+#define SD1CKCR		0xE6150078
 #define SD2CKCR		0xE6150268
 #define SD3CKCR		0xE615026C
 #define SD_SRCFC_DIV1	(0 << 2)
```


I haven't tested/actually used this version (so correct me if I'm wrong), but I think it might be missing a few cases: say the config "row_size" was set to 3, and "fixed_row_size" is *false* (i.e. the user still wanted the old behavior). If there are currently no images in the gallery, and one is added, then the "size <= 0" condition here passes, rowSize is set to 3, and the gallery is drawn as if there are 3 images per row, so the single image is now drawn at 33% width, vs. the old behavior of 100%. Don't think this is a big deal, but it might throw off some people who still want the old behavior.


see https://task.beeznest.com/issues/10743


Closes #244


fixes #2


같은 이유로 아파치, nginx, PHP 버전을 숨기도록 설정하는 사람이 많습니다. 물론 작정하고 덤빈다면 큰 도움이 되지 않겠지만, 버전에 따라 자동으로 해킹을 시도하는 metasploit 같은 툴도 있기 때문에 초보 해커들을 곯려주는 데는 어느 정도 도움이 됩니다.


At the time I made this commit I was using sgit from my Kindle rather than committing over SSH from my PC.


Rails 4.1, but the gemspec says `~> 4.2`..?


Капслок в коде недопустим.


Vim, ze je to historicky, ale lepsii by bylo to pojmenovat jako setOrderId. Takhle to budi dojem, ze jich tam muzes dat vic. Co z toho udelat alias dat to jako deprecated?


OK


fixes #4


@jmalak The changes look great!  Thanks for your input!


Aco, I have pushed the updated version now


LGTM
@nieyy  Thank you very much!


nitpick: I think it is more natural to make `reason` a property if it is linked to `unsat`


Looks good and simplifies the code a lot.


Thanks .


u rock :)


там же идет просто пустой div сначала


you should use http://review.cyanogenmod.org/


mais il sait tout seul que c dans le module config qu'il doit aller chercher GOOGLE_ANALYTICS_ID et HOST_FOR_GOOGLE ?


There is no srccache on release-0.4. Please don't break the release branch.


+1 +2


je contrecheck parce que c le genre de truc ou on voit pas en dev qu'en fait analytics est plus appelé et on s'en rend compte que six mois après la mise en prod :)


release-0.4 makefiles use `=` not `:=`


@n1k0 could you have a look at the fetch calls in the file?


Maybe just `Map(f, r)`, and `Reduce(r...)` above?


f = np.linspace(0.001, 0.4, 100), cause HF is not greater 0.4


closes #244


Thank you for your work :+1:


Не понятен выбор span


Another case: if the user *doesn't* set the config option "row_size", nor the "ion-gallery-row" attribute, and the gallery initially starts off empty. Won't ionGalleryConfig.row_size be undefined? Maybe not, I'm not too sure how angular.extend works at L77.


new comment system.


This second line should not be indented. Otherwise the example will end up looking a little funky.


ok sory but emuCR.com said decaf is avaible


Should we check if content_type is set? I mean
if (response->content_type && streq(...))


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


That's really cool...


можно убрать комментарий


This shouldn't be added, any code that uses it is likely relying on master-only out-of-tree build rules.


will do


Ideally, should be a hash of the contents, no?


Refactoring is to make code easier to change in the future.

The time I spend refactoring is a gamble on how I'm going to spend my time in the future. If I refactor code today and need to change it tomorrow, my bet pays off. If however I never need to change that code again, the time I spent refactoring (apart from any side-benefit like practicing & learning the skill of refactoring & design) has essentially been a waste.

Because I only have limited time on this planet, and yet I am a self-confessed refactoring addict, I have learned to be constantly mindful of this trade-off, and try to keep a sense of when code is 'good enough'.

I'm confident that this code could be 'prefactored' if it did need to change in the future, so I'm comfortable leaving it as is.

Make sense?


@minond I can't really tell you what the decision was on these particular changes as I did not participate in them. Doing a minor version bump was probably an oversight when the version was bumped...

Still, in my opinion, the seeder feature was **unfinished** until this commit - you could not really call it a seeder - where one **breaking change** was indeed applied (`db:seed` was renamed `db:seed:all`), to better accommodate the notion of a data seed. In that same regard, I also added some saner defaults to the seed migrator, although these were just changes to defaults.


Let's do that: https://github.com/gradle/langos/issues/147#issuecomment-174457972


до этого места по аналогии с гостиницами


index-welcome это не лого потом див зкрываешь


здесь это h1 класс должен соответствовать заголовку первого уровня (main-title или что то подобное)


просто p можно без класса это стандартный набор текста


сейчас название класса "блок на главной" - это слишком абстрктно, лучше causes-item внутри 2 дива в левом h2 с классом, спан и p (без классов) , во втором диве img


Thanks


service-item


удали изображения будут псевдоэлементами,


класс не обязатален, стандартный текст


по аналогии с первым только зеркально развернуто


Nice, thanks


features  будет короче и одним словом далее внутри по аналогии с вышестоящим


features-item


Looks great. Thanks!!!


p


всю форму перенеси вконец футера можно за футер вынести


ссылка и это вне формы, идет после текста в блоке фидбэк, форму расскажу на консультации как верстать долго писать


удалить остальное аналогия с футером в гостиницах


Oh, you meant the spacing. I misunderstood the previous comment, I'll have to fix this in topic 2 too.


oof. This is a big chunk of global state.


Это можно переписать короче и понятнее (select not something == reject something, huh?)


text changed to "This file was made for a test"


lepší je `@return self`


...а также для красоты таких выборок я бы сделал кину дополнительный метод `watched?` (с вопр.знаком). В руби обычно так называют методы, отвечающие true/false.


не указывай название тега в классе, просто sub-titile (означает вспомогательный , не главный заголовок)


We could also move up until we see the `composer.json`.


Any kind of performance penalty from all the reflection?


No prob.  Hope you have time for more contribs, we're big fans of the work you did so far!:)


Wiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiiii \o/


I added a compiler message. Is that what you meant?

https://github.com/aothms/IfcOpenShell/commit/801c7da40ff7edc207e3f4721fe3c6b8ad3f31ef#diff-662210383372f94d2d123befe3de6b95R108


Good. Just an idea that it might be easier to maintain just one build system.


@David-Development Does this affect f-droid?


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


@viebig esse commit já está no master a 22 dias, agora.


Thanks Johnny.  Submitting it through Github was actually harder than the programming assignment.  Hope all is well.

From: Johnny Ocampo [mailto:notifications@github.com]
Sent: Sunday, January 24, 2016 5:06 AM
To: DRavaglia/ProgrammingAssignment2
Cc: Ravaglia, Daniel C
Subject: Re: [ProgrammingAssignment2] Update cachematrix.R (9c73ccf)

Nice work! Being review by Johnny O. :-)

—
Reply to this email directly or view it on GitHub<https://github.com/DRavaglia/ProgrammingAssignment2/commit/9c73ccf4b5b63e5ebb5b1cbf46beeeec1240029a#commitcomment-15635658>.

________________________________

This email is intended solely for the recipient. It may contain privileged, proprietary or confidential information or material. If you are not the intended recipient, please delete this email and any attachments and notify the sender of the error.


I agree that there's more work required for completely supporting a custom ecosystem than what can be outlined here but given that dependency reports:

> Should also show link time dependencies for native binaries, and runtime dependencies for executable binaries (including test suites, jvm applications, native exe, etc).

And:

> The implementation should *not* have any knowledge of the Java ecosystem. It may understand the abstract software model.

I think it's beneficial to design it with custom ecosystems in mind if only to make sure it remains decoupled from the JVM and Native ecosystems.


You don't need to repeat this in every method. Actually I don't think this works. You should put it only ones and add except: [] if you don't want that action for a certain method


Hmm, this isn't quite what I meant (for tbody) - you only need one tbody per table, so place it inside the <table> </table> tags below, and then load the rows inside that.


once*


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


Good points.


It would be nice to use other :comment or 'comment', not both. Just for you to know, you don't need to change it in the training


Better to use slim


Also, I think you'll need to swap these - oldValue is [key][1], and newValue is [key][0].


I was guessing so, but I'm not understanding how elements in _data/people are linked to the full names. Just by firstname?
I guess I'll try.


thanks!


This needs an argument!


Эммм не. Когда я говорил «сделай метод description», я имел в виду следующее:
```ruby
class AncientMovie < Movie
  def description
    "старый фильм (#{@year})"
  end
end
```
...и потом кто-то вызывает просто `movie.description`.

Можно сделать либо так (как я пишу), либо перекрывать таки у каждого класса initialize (вызывая super, чтобы инициализировать всё, кроме описания). Так как ты сейчас сделал, у тебя получается что класс Movie должен «знать» что у него будут потомки, которые определят метод description — это не очень хороший объектно-ориентированный дизайн (да к тому же теперь класс Movie сам по себе использовать нельзя — попробуй!)


could we have this array in image.c, translatable but not translated and non-static ?

*  usually when you define an enum it's a good habit to have the enum<=>string table defined in a related file
* that would be pretty usefull for for lua...


nope, our client implementation ensures that there is a  `content_type`.

If any `content_type` is given we put `application/octet-stream` before call the response callback.


I don't support this change. I'd suggest the actual fault is the getter (getDatePattern) should be removed as it's weird to access a static constant via a getter.


I think I was afraid of overflows. NAME_OFFSET and siz could overflow when added (in principle at least).


Also, if using <div>, you won't need <br> - since the div takes up the full width of the parent element, the next list element will automatically go on a new line.


== 0


This file could be tidier. Instead of adding all in the same lever you could do sth like this:

```
sessions:
   login: 'Iniciar Sesion'
   logout: 'Cerrar Sesión'
```

Just for you to know. You don't need to change it in the training


This is the same as the above. Why is intellij removing static constants. Is it because they aren't declared with the usual capitals?


There are a lot more than just this one that need updating.


How is `PositionExpander` related to this PR?


I think it loops through all the content in _data/people until it finds the one with same full name


We dont use this tests. We use rspec. You should remove this files and add the folder to the gitignore


WERGWERG


Fuck, it's not supposed to be there. I accidentaly staged it, sorry, will fix.


Почему у тебя все комиты называются initial commit


Ahh I see, I'll change it.


А эта штука одинакова у всех экземпляров класса AncientMovie. Из чего можно сделать вывод, что это должно быть константой в классе, например!


I created this function to make it easier to init a struct sol_lwm2m_resource. However since the struct is public the API version will need to be set. If this function is not on the header and not inline and
we may set the wrong API version.
Imagine the following case: Soletta is update and struct sol_lwm2m_resouce has its layout changed, thus, its api version is changed to 2.
If sol_lwm2m_resource_init() is not a public inline function, the user would need to check the API version by himself/herself like this:
```C
struct sol_lwm2m_resource res;
sol_lwm2m_resource_init(&res, 0, 1, MyIntNumber);
```


In my opinion, it is much more desirable to have a process that makes no progress than to have a process that keeps trying to create a connection repeatedly, each of which consumes resources (e.g. memory, sockets, etc.).

`gun` uses [`proc_lib:start_link/3`](https://github.com/ninenines/gun/blob/master/src/gun.erl#L461) to spawn its workers. According to the [documentation](http://erldocs.com/current/stdlib/proc_lib.html?i=0&search=proc_lib:start_link#start_link/3):

> If the `start_link/3,4,5` function is used and the process crashes before it has called `init_ack/1,2`, `{error, Reason}` is returned if the calling process traps exits.

It seems that if the `gun` process crashes upon initialization there will be either a crash or an `{error, Reason}`.


You could use `keysView()` instead of `keyBag().toSet()`


@byteandahalf When I use SGit my photo is fine have you gone to the "Big Profile" button and set up that?


haha no, it already got fixed up with linux 3.10.95 ;) I was not in full
consciousness when I pushed that commit, I saw that commit on cm's github
tried cherry picking it, and failed lol
manually copied the commit and the commit msg and thn I realized that it
was already fixed with linux patch 3.10.95
On 25-Jan-2016 15:23, "Alex B." <notifications@github.com> wrote:

> So basically the fix consists in entering one blank space? ^^ It seems the
> vulnerability was fixed by Asus with the latest source update?
> Fix for vulnerable kernels:
> (CyanogenMod/android_kernel_cyanogen_msm8916@87066d3
> <https://github.com/CyanogenMod/android_kernel_cyanogen_msm8916/commit/87066d33ef6e4347ea24108260bbbe3b944ef130>
> )
>
> Our kernel (latest source):
>
> https://github.com/friedrich420/Ael-Zen-Kernel-Asus-Zenfone2-/blob/master/security/keys/process_keys.c#L795
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Zenfone2-Dev/android_kernel_asus_moorefield/commit/347de1d212d6be8bc103aba02b00f615c95271e8#commitcomment-15645211>
> .
>


Can't check right now, I put it on my list

On 25.01.16 13:01, Ian wrote:
> fix nasty bug with high PHP versions
>
> @garvinhicking
> May this be the "real" cause of http://board.s9y.org/viewtopic.php?f=10&t=20639 ff, not loading the editor.js file? He is using PHP 5.6.17.
> What I do not get on this: Why does this only happen with the Standard Editor on his site? And why does this not happen testing both groups to me locally with nearly the same PHP version?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/s9y/Serendipity/commit/1b67de854de371be38bc1a60858d6c3c89279730
>


Sure, I have to call it only in this case :smile:


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


This change should be reverted. It wouldn't be usual to define the service being tested in each and every test method.


Simple :) I've just re-pushed the PR without the log testing.


mergi


Is it will more useful to do
features = {}
?


Ah, missed that, Yeah, that's good too, but was thinking about seeing log print during the IfcCovnert conversion.


:+1:
One concern here: calling methods to get bot's name, id, team  (Ex: `#bot_user_name`) before bot is started will raise exception.


OK. Just to add this here:

We have other occurrences in bundled-libs WIKI and PEAR and in lib2 of serendipity_event_textile. What shall we do with it: fix or upgrade?

https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Function.php#L43

https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Phplookup.php#L39
https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Phplookup.php#L51

https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Url.php#L40
https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Url.php#L49

https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Latex/Wikilink.php#L41
https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Latex/Wikilink.php#L42
https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Latex/Wikilink.php#L43

https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Interwiki.php#L56
https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Interwiki.php#L64

https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Image.php#L106
https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Image.php#L126
https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Image.php#L127

https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Wikilink.php#L62
https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Render/Xhtml/Wikilink.php#L64

https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Parse/Default/Heading.php#L66

https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/Text/Wiki/Rule/entities.php#L53

https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/PEAR/Exception.php#L332
https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/bundled-libs/PEAR/Exception.php#L358

https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/plugins/serendipity_event_textile/lib2/classTextile.php#L842
https://github.com/s9y/Serendipity/blob/9e49ecc8c3362f3bb4522896d4be3d89058460f7/plugins/serendipity_event_textile/lib2/classTextile.php#L1791


А в задании-то было — вывести здесь, сколько ещё фильмов того же режиссёра в нашем топе всех фильмов.


Cool. I've never used Objects.equals before.


@LuckyCyborg one space to much? :blush:


@LuckyCyborg

> & Server Headers

compared to line [460](https://github.com/LuckyCyborg/simple-mvc-framework/commit/09b59bf02abfde625b0d150b2e40bdb7d1055003#diff-a2531a61dddd30f35bfefc16aaba5531R460)

> Server Headers


TeamCity Civic Assembly :: Build backend + frontend [Build 23](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=430&buildTypeId=CivicAssembly_BuildBackendFrontend&guest=1) is now running


features.append([bounds_names[i] + 'rel', np.mean(pgram[idx]) * (frequency_bounds[i+1] - frequency_bounds[i]) / power])


features.append([bounds_names[i] + 'normalized', np.mean(pgram[idx]) * (frequency_bounds[i+1] - frequency_bounds[i]) / (power - features[3][1])]) #features[3][1] is VLF and is already calculated


see [comment](https://github.com/LuckyCyborg/simple-mvc-framework/commit/09b59bf02abfde625b0d150b2e40bdb7d1055003#commitcomment-15648728) above :point_up:

I would also revise the language files with the same ?!


It is good to hear from you Dan.  It can be a bit of a challenge at first to use Git bash alone with Git Hub since one is not constantly keeping version control on documents, but, once you use it twice or more times then it becomes fun to work with.  I like the fact you provided a test case for the programming assignment. Out of the four individuals I had to grade their assignments, you were the only one that provided the test case. How long have you been programming for?


I added an assertion to verify the value against the array length. The length is passed in by the caller.


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


You didn't miss it, added it as per your comment. Don't think a log print is necessary. Most users will use precompiled binaries from the website, which doesn't include the message now, and will hopefully be linked to newest OCC for the upcoming builds.


Luckily the change I made to implement the BaseEntity removed all these changes. I prefer the long format because it is much easier to read.


Fix the commit message please


I'm just following the same way it is mostly used in this file.


Can we make it `-ggdb2` on release build? It is significantly harder to debug the release build now (and certain bugs are not always as reproducible on release and debug builds).


>Just to let you know. When I cross read texts, I turn myself into a total newbie and then I try to follow the ideas and steps and see how far I can go before I refuse... :smile:

And that's what one should do. The best thing is to work on the Wiki. A definition section would probably help.


Instead of first saving objects, can you use existing ones? Look at standardTestDataset.xml for existing ids


Can we also always include space before and after equal signs for improved formatting?


Not necessary to revert this change, but I did also prefer the long version as easier to read. I believe we can leave optimisations like these to the compiler and keep the code easier to understand.


actually the message just below is one I'm trying to figure out what to do about, and I hadn't noticed that about comm_err_flag yet, so that's good to know.


@Slaals As an argument of exiftool, what does `-php` means?
I cannot find it in the man page (`-p` is a print format but...).


Shouldn't this be outside of the context manager?


Yeah. Wonder if OCE folks are planning to update soon. I wonder if it would be sensible to use the official OCCT instead of OCE for Windows build scripts?


I agree... these blood test status checks are hard enough to understand/follow :)


TeamCity ArachNGIN :: Build [Build 5](https://tc.phome.cz/viewLog.html?buildId=18&buildTypeId=ArachNGIN_Build&guest=1) is now running


Вот тут у тебя `new(movie)` повторяется много раз. А в руби, как известно, мы повторений не любим (принцип DRY — Don't Repeat Yourself)


@Gillou68310 Do you think this is something that upstream would take as an optional setting?

There are a lot of times where the core is not taking enough CPU or the CPU utilization is kind of strange. This results in a lot of newer ARM devices under clocking their CPUs which causes slow downs in games.

A busy wait, however battery inefficient it is, seem to fix this situation.

Are you aware of any better way to prevent an android device from under clocking?