Thank you Andrea! :smile:


if (EventEngineManager.listenerOnUseItem(activeChar, item))


public static boolean listenerOnUseItem(L2PcInstance player, L2ItemInstance item)


ниже нарушение принципа инкапсуляции!


¿Por qué usás getEventPlayer(player) si el método ya pide un L2PcInstance?


public abstract boolean onUseItem(PlayerHolder player, L2ItemInstance item);


忽略 eclipse 的配置文件


We should elaborate on breaking.

1. malicious injection (maybe reference the china vs GH DOS attack)
2. broken assets


it is common practice for Python projects to name this file requirements.txt. Usually it is used by projects distributed in sources, but it does not make much sense for packages distributed from Pypi. I think it is safe to delete it


Buen aporte, saludos desde Perú.


public boolean onUseItem(PlayerHolder player, L2ItemInstance item)


we may also want to describe what you get by default, rather then immediately showing what can be configured.


SkillHolder[] skills = item.getItem().getSkills();


public boolean listenerOnUseItem(L2PcInstance player, L2ItemInstance item)


I literally said what the message should be


@twizmwazin Well I'm blind!


@sanemat Yeah, it is. Luckily just in the prototypes so it won't cause any issues.


Let's break it down:
if `jsonObject != null` is true, we short circuit the logical OR and don't execute the `!jsonObject.has("error")`
if `jsonObject == null`, we try to execute `jsonObject.has("error")` and fail with a NullPointerException.

IMHO this should probably be a `&&` instead of a `||`, but wanted @Carlinho89 to take a look at it, since it's his commit


cruby 1.9.3 uses a proc here, which mruby is based upon.
http://ruby-doc.org/core-1.9.3/Method.html#method-i-to_proc


I didn't think this was backported. Perhaps it should only be in the template for Rails master?


これかｗｗｗ
これ面白いわｗｗｗ


Minimized as https://issues.scala-lang.org/browse/SI-9408


dsd


Well statements with only expressions are indeed allowed.


hm, strange. ruby-doc doesn't seam to be correct. :+1:


A workaround would by to generate:

```scala
class fresh extends Lazy with Serializable { lazy val value = <splice> }
```

Rather than the current (seemingly convoluted):

```scala
Lazy.apply { /*byname*/
  class fresh extends Serializable {
    lazy val temp = <splice>
  }
  fresh.apply
}
```


But some may want to set different sub-command permisisons individually.


I had no idea you could do `instance` instead of `first`. This seems way more elegant and proper.


That did only update one Smack subproject. I recommend using a variable property for the Smack version, something like:

```groovy

ext {
  smackVersion = '4.1.3'
}

depenencies {
    compile "org.igniterealtime.smack:smack-android-extensions:$smackVersion"
    compile "org.igniterealtime.smack:smack-experimental:$smackVersion"
    compile "org.igniterealtime.smack:smack-tcp:$smackVersion"
}
```


Building a custom box is certainly an option to address the consistent network device naming issues. Still, relying on a specific interface is risky and should be avoided if possible.


current: haijing.heroku.com


please change the website URL


:exclamation: This is handled by Roslyn in cases where mscorlib contains `Array.Empty<T>`.  I don't want to make the code more complex to address this. If we need it, we can target .NET 4.6 or another framework which contains it.


Otherwise they bloat to 32 bits, which means sizeof(RDMProductDetail) is incorrect


Fixed


hi, really great job, but maybe you can help me.. it doesnt work :(.. Only the background move, nothing else, no bird nothing.. can you tell me about what i can do ?


That's in simple_model.c, which will be in the next PR


ack


I spent most of yesterday trying various permutations of this sort of change ... I don't think I tried this variant though ... will report back.


-->主页的图片轮播没有完成，页面两侧的标语没有完成<--


Beau boulot


I don't think we want to capitalize Option.


Hmm, maybe `fresh` will get an outer pointer and you'll still be in trouble...


если нужно сравнить ссылки, может есть смысл использовать ReferenceEquals?


Thank you for the mention @freak97. Appreciate that alot! :+1:


Tested :+1:


add detail


double check this


"that the either the graph" -> "that either the graph" or "that the graph"


I was wondering why it is only possible to say thumbs up/down once per job? Why a user cannot change it? It is possible on the OZnzb website.


Add test cases, and results.


merged


Just wondering, can we glob all config/*.yml here?  I'd like to add reference data in a new yaml file, and this seems to be the only place we hardcode the list of files.


So this is why you stopped working on agariobot

Seriously though, looking foward to this!


Sorry - missed that we already had a TravisCI file.
Overwrote by mistake.


I don't see why not.


Sé que esto no lo modificaste pero ya que estamos suma. Estaría bueno que este mensaje varíe dependiendo del estado del engine. Si es tiempo de votación, te invite a votar, si es registro, que te registres, etc.


Considering that constants are used in the for loop above, a reference to the same constants should be used here (in case their value changes in future).


maybe someone can go ahead and update the level and faction of the creatures

(
SET @CGUID := 371271;
select * from creature WHERE `guid` BETWEEN @CGUID+0 AND @CGUID+280 order by `id`;
)


Maybe ```if not user:``` ?


I believe elsewhere is using the canvas.Graph style, rather than array element style.


метод Equals не выбрасывает исключений


you can leave an inline comment


??? Tiene que ser estático sino no carga el Set y no muestra la cuenta regresiva.


There is no zeny check between the last `next` and this line.


not user get True when is 0, '', [] etc etc. I would check if user not exists


лучше присвоить  string.Empty


One more secret handshake to try: declare the fresh classes as `final`, that lets the `constructors` compiler phase elide the outer pointer if it isn't used.


What do you mean? There was a command whitelist before this commit. Some people requested it also on messages, so I implemented it.


yeah you're right... awesome, thanks! :+1:


There are no checks on this item between the last `next` and this line. There can never be `next`, `input`, `prompt`, `select`, `sleep`, `progressbar`, etc between the checks and `delitem`.


<dasCaptcha> needs to remain as the original: <theCaptcha>

We use it as a Placeholder to replace the string with the actual captcha to type.


не весь (в смысле get set ) индексатор  должен быть private


The assert is no longer needed, I've removed it.


too heavy


There is no need to scope, what ultimately becomes, the whole application with the controller. Better would be to specify the controller on line 55 - `<ion-side-menu>`. Also, please use the `XYZController as xyz` as much as possible.


box -> popover


fixed


Made a few comments. Other than that, +1 from me. Thanks @MatthewHiscock !


No newline at eof


other может быть null, а может указывать на тот же объект, на котором вызывается этот метод, все это стоит проверить


Ideally, we would like to avoid hardcoding a hide action for one particular screen. I think there should be a better solution to make it work with the login other than hardcoding a hide for that particular view. Could you please re-visit the issue and see if you can find a better solution? Thanks


The thing I had which fixed the serialization problem was,
```
q"""
val fresh: $actualType = $tree  // alternativel lazy val
_root_.shapeless.Lazy[$actualType]($tree)
"""
however, this caused a couple of other instances to fail at either compile or runtime, presumably because it affected the way the recursive derivation was happening somehow.


automitically to automatically


apprved


No se usa sólo para anunciar el próximo evento sino para todo tipo de mensaje que tenga una cuenta regresiva (bah, tenga un tiempo).


Changes:
- "strip" operator replaced by "before strip"
- added new operator "before replace"
- new example filters in rules.txt
- reworked blocking chat until moved, allowing to specify distance, also moved to anti_bot
- added an option to specify that if a message starts with something from the list it will be ignored from the delay/similarity check + supports regex
- added an emergency option to communicate with the developer if something wrong occurs and I cant access source (nerfed in the later commit)


не очень хороший код


MS рекомендуeт использовать вполне определенные имена здесь Add


The side menu is de-facto a separate module if you think about. The idea is, if we choose to get rid of the sidebar all together, it should be as simple as removing the whole directory and removing one module import. This way, we would have to re-visit the top level application file and also make sure there are no other components referencing the Menu/SideMenu constant.

Also, as per comment above, try to avoid the use of `$scope` in favour of the object oriented style of writing controllers using `this`. Angular documentation has decent info on this.


Nah. I just got bored, as I often do. Though I've had this idea simmering for over a year. :)


I think this should be defaultLoader(), wdyt?


не уверена, что сеттер нужен, даже приватный


I fucked things up resolving the conflict


fixed


The indentation on this file is now completely broken!


if (EventEngineManager.getAllRegisteredPlayers().size() < ConfigData.MIN_PLAYERS_IN_EVENT)


I played around with the code and now you can change your up/down vote!


Do u mean canola oil?


Looks like you spelled 'canola' wrong.


I cannot merge this code right now. Sorry! You should check your spelling first before u make a pull request.


Не хотелось бы повторять дважды они-они. Заменил на "Холм Ветряков стал
достопримечательностью Казегауры"


because it was in there twice and the TileRenderdistance was missing? >_>


Changes will match this: https://github.com/jonathanKingston/ember-cli-sri/commit/626f4b8801f343ff03fb878a7ab2696b60c71dcf

Is that enough?


I have been clear to avoid potentially inaccurate information: https://github.com/jonathanKingston/ember-cli-sri/commit/626f4b8801f343ff03fb878a7ab2696b60c71dcf

If that is ok I will update this pull too (I want both ember-cli docs and my README to match where possible).


Thanks for this, @ryanjhodge!


Вставил "реконструкцию"


I didn't really have enough time to do a build so I took the libaudiopolicymanager.so and just put it into /system/lib and the device bootlooped. Did it need the USE_CUSTOM_AUDIO_POLICY flag?


作成されたChatClinentHandler.javaについて、「接続してきたクライアント側のオブジェクトを作成」の機能が実装されていることを確認しました。


This commit fixed a pretty strange boot issue I had been having all along. Since it *fixes* the issue and I have no problem anymore, this is just for your information and because it might give some insight into issues people might have.

When I had the terminal_usb keyboard converter plugged in during boot, Linux would often fail to bring up some of the cores of my quadcore Intel CPU, or even crash altogether during this phase of the boot. The kernel log messages are in a [gist](https://gist.github.com/DigitalBrains1/63a9ea08597b890b0599), specifically getting interesting around [line 255](https://gist.github.com/DigitalBrains1/63a9ea08597b890b0599#file-gistfile1-txt-L255).

When trying to boot to Windows 7 to play a non-Linux game, Windows would often reboot even before the start screen faded in completely, and thus end up in a reboot loop.

I presume this is a behaviour shown specifically by my system. The motherboard, likely to blaim, is a Gigabyte GA-Z97X-UD3H.

I ended up unplugging the keyboard when I shut down the PC and plug it in again after it has booted, which was very easy as the hub was next to the keyboard anyway.


Согласен


fixed


lol


Added the https://github.com/BankFacil/vanilla-masker project to the files, open source lib for filtering/masking input text.


m) NOW I see it. I guess this sort of blindness explains how you broke it in the first place ;)


I do not think that this is right. In BaseConnection::createChannel, TelepathyQt/base-connection.cpp, mPriv->createChannelCB() is called before channel->setTargetID(). But the createChannelCB creates the TextChannel, so that targetID is not set when the constructor is run.

We would either have to replace the uses of m_contactJid with baseChannel->targetID() and store the baseChannel pointer or revert this.


@will I just see this. Why `CGI.escape` is used here? Ruby's URI doesn't do that. I think URI shouldn't do any conversion. In any case you should manually invoke CGI.escape when constructing an URI from its components. I'm also not sure we should have `userinfo` as a separate argument in the constructor, because `user` and `password` make that up... or no?


Inital draft for Manifest


Fabulous work!


Вот и я хз. Там вообще ещё много физики/аэродинамики будет. Пока так оставил


At the very bottom (the link target above) are the instructions for installing from a tarball which covers that case.


It seems this commit causes problems for admin menu's layout

![quokka-menu](https://cloud.githubusercontent.com/assets/1086947/8762377/f7a1af0c-2d3c-11e5-8f18-cab9c1f0bf61.png)


test


looks like it is duplicating the categories @realli


Тут снова что-то воздушное)


That makes absolutely no sense at all.  Might?  Please explain how this can affect boot.


not sure if this is impactful. Travis seems to be failing.

@RobLoach ?


I just bitrotted this with my top sites local visits thing. This'll turn into `(siteID, is_local, date)`.


These can all be `static let`, right? Or `private let` at the toplevel?


This commit caused
[Regression in current folder tab completion (./executable)](https://github.com/scopatz/xonsh/issues/307)


Shouldn't this go to the plugin providing those animations?


All this does is makes extra libs available for inline kernel building.  If the libs in the kernel toolchains are newer than the rom then this is pretty much required.


fixed


Thanks :)


I think this is still worth leaving in; remember we had trouble when the OS upgraded sqlite? (We don't use our own sqlcipher version for everything, do we?)


Trailing comma.


pretty much


Oh, nice. That makes testing easier!


Drop the views *and any indices* before modifying the tables.


Why use a tuple? Why not
```
NotExhaustivePattern [[Binder]] Bool
```
?


Maybe just indent once: `indent $ paras $ ...`


Пока оставлю так, чую тут какую-то метафору)


If anyone's wondering, this specific line is the change that's causing a lot of Stylish themes to break.


Действительно)


And `PRAGMA foreign_key_check`.


rebuild index


fixed. Мне не нравится "зачесала" =\


The four dates that you added in this method probably shouldn't be now. Very likely they should be the last visit date of the history item, which should be something we know at this point.


@begizi so Echo no longer retains the original `error` thrown?  It looks like `err` just gets thrown away completely now?


I'm not sure what this change is for… could you explain?


Я вообще не понял, что он хотел сказать)


http://stu.sernam.ru/lect_alg.php?id=31 Считается, что степень нулевого многочлена  - это -∞


It does retain it. You can set the error handler function to manually
format the error thrown by the middleware or handlers. It's
`SetErrorHandler()` or something similar.

On Sat, Jul 18, 2015, 12:23 PM Craig P Jolicoeur <notifications@github.com>
wrote:

> @begizi <https://github.com/begizi> so Echo no longer retains the
> original error thrown? It looks like err just gets thrown away completely
> now?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mojotech/situation-room/commit/64ab6af77c927f8faefc1b8984197ae33d30b6d4#commitcomment-12238842>
> .
>


nitpick: the dog is called "The Low-Profile Dog"


Please add the requirement to provide these widgets DLL's:

wxbase28*_xml_vc_custom.dll
wxmsw28*_qa_vc_custom.dll

Unmodified HEAD will not run without those.


@szaparenko, @mysthea  (wywołanie takie następuje poprzez @, spróbuj mnie wywołać), wszelkie zmiany na modelach (pliki models.py) pociągają za sobą stworzenie pliku migracji. Jeśli nie wiecie co to, pisać, opiszę w mailu. W każdym razie dorobiłam migrację i będzie w następnym PullRequeście, aczkolwiek na przyszłość, jeśli widzicie, że w modelu jest coś nie tak, zgłaszać mi to, a ja poprawię łącznie z samym plikiem migracji, żeby nie tworzyć wielu plików migracji (im mniej tym lepiej).
@szaparenko, nawet nie wiedziałam, że tu można coś zmieniać, nigdy tego nie robiłam :) :+1:


The second commit changes rounding modes in the file `rtl/rel11/rel9-rtl-pkg/support/lib3.delta3/sqrt66.lisp` from old to new. Then it moves this file to `rtl/rel11/support/sqrt.lisp`.


You're calling it `hoodie-server` throughout the document, while in the future it will either be `hoodiehq/server` or `@hoodie/server`. Can we find a common way to refer to components now?

Also applies to `hoodie-client`


fixed


typo: environment


If I had the answers to a couple of questions I could probably pull together a PR.

1. is `cwd` supposed to be a temp directory here?
2. if so, is that temp directory supposed to contain the contents of `benchmark_dir`?


I'd convert that from prose to a code block, because that's what I scan readmes for, when I want to contribute.

```bash
git clone git@github.com:hoodiehq/server.git
cd server
npm install
```


It seems mariadb has problem with ``enableParallelBuilding``, or do I get this wrong? http://hydra.nixos.org/build/23703925/nixlog/1/tail-reload /cc other maintainer: @wkennington.


типа fixed


Thanks for tracking this down.


How do we make sure this doesn't get out of sync? The hint might be enough.


I'll have a look

Il giorno 18:31 sab 18/lug/2015 Anthony Scopatz <notifications@github.com>
ha scritto:

> Thanks for tracking this down.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/scopatz/xonsh/commit/e130c6f96fedfb8228ec905fad9cba6152f87f97#commitcomment-12238873>
> .
>


#include "stdafx.h"


1) Globally this is good, but I think that you should raise an S3UploadError instead of the S3Error. Otherwise, continue on error / partial thing will not work (as there is only the following exc that are catched: invalidFileError and S3UploadError. The other are fatals).

2) I saw that there was the same kind of code used in object_copy/_modify, but there are 2 calling functions that doesn't care so much (that the exception be S3Error or S3UploadError) and 1 that is using the fact that it is an S3Error to look at the code to see if it is a "NoSuchKey" error.
(See: https://github.com/s3tools/s3cmd/issues/584)
-> Maybe you can add to this PR, the change to parse the code more early, in "object_copy" "object_modify", to also return an S3UploadError in the case of "NoSuchKey" and the S3Error otherwise?

3) I don't think that it is for this PR, and I'm not sure if it is a good idea, but we should probably modify "send_file_multipart" to have the same kind of error handling and retry as what is in "send_file".
In the current code, if i'm not wrong, we retry small files, but not big (multipart) files. That is not great :-s


I'm in Boot Camp


ЧЯДНТ?)


It also handles the `if` check ;)


I don't think v-= v/8 is a good way to decay.  Since the values will never be able to decay below 7.
I believe v*=80/100 you will get better results.


What's this doing? Don't we already have an array with all the lines in it?


1. yes, 2. no


Nice use of a list comprehension


good


Было уже за полночь


ЛОЛ


The reason is for the "extras" API: http://pouchdb.com/api.html#extras

I know it adds a lot of size to the npm install, but I think it was really the easiest way to solve that issue. Making the extras an external module was something we considered, but due to complexity plus the inability to modify pouchdb itself when using the adapter plugins made us decide on the "extras" API, similar to `babel/polyfill`.


@tdhock This commit makes geom_point version of FluView viz works. See the following gist

http://bl.ocks.org/caijun/raw/b53842f22fe0c75f26db/


fixed


@sindresorhus will this be quite confusing to see just `unzipResponse` in code?


@RobLoach this seems to be failing Travis?


#include "stdafx.h"
#include "conio.h"
#include <iostream>
using namespace std;

void main()
{
     cout<<"Hola Mundo!";
     getch();
}


updated


There is still room for recovery, but my 8-thread test is currently showing a large regression (against Komodo).  I suggest running a 7-thread, LTC test.  My test has ample hash.  I know it has already had an extra test, but I think it is a reasonable precaution.


I forked your repo and updated the files in `3rdparty/poppler/git` to the latest Poppler (0.34.0) but it doesn’t compile:

```
/tmp/pdf2htmlex20150718-77133-aun0to/pdf2htmlEX-0.13.6/3rdparty/poppler/git/CairoFontEngine.cc:37:10: fatal error: 'config.h' file not found
#include <config.h>
         ^
/tmp/pdf2htmlex20150718-77133-aun0to/pdf2htmlEX-0.13.6/3rdparty/poppler/git/CairoRescaleBox.cc:54:10: fatal error: 'goo/gtypes_p.h' file not found
#include "goo/gtypes_p.h"
         ^
```

Is it sufficient to just dump the files in there or did I miss a step?


I didn't think so, but I made it so I might not see it objectively. Lots of common methods are passthrough if it can't do what it need. I could have named it `unzip-response-if-needed`, but that would have been very verbose. Any suggestions? Rename module? To what? Or just use a longer variable name? I really think the if logic should be in the module. Do the case here is how to make it clear what it does.


博客终于支持评论了


测试测试


This piece of code was a copy/paste from the FIELD_PARAMETER_SET_UPDATE_ELEMENT subroutine, but you're right, here it should also be possible to interpolate for a Gauss point in a ghost element.


ok


What happens when the SSL session comes from the session pool? As I stated before, this is still the wrong approach for this problem, SSL hostname verification is a property of the origin server, not a property of the transaction.


Yes, this is a mistake. I don't know why this happened.


you can keep:
```python
try:
  import numpy
  return [(n[i], -1 * f[i]) for i in numpy.lexsort([ln, f])]
except ImportError:
  # your implementation
```


Yeah, I’m not sure about this. I think I want it to be `Hoodie Server`.


показала своё истинное лицо >_>


FUCK this code.


You can ignore these changes.


I phrased it wrong. It's not so much irrelevant as it is default values. There's no reason to export "alignX", for example, if it's set to "none", since "none" is the default value — https://github.com/kangax/fabric.js/blob/2171dd13798458d4cfa383a8b2030ae1fa216ffe/src/shapes/image.class.js#L47

Does that make sense?


just push commits onto `next` to fix this, or PR if you are unsure.


I also noticed that we don't call `removeDefaultValues` in all `toObject` methods, so there's a chance we missed something else.


yeah, let’s not worry about this too much.


@leonkyr i forget if we have discussed about this, but i am looking at it again, and this code is just doing nothing. `winston.Logger` doesn't accept a `enable` option.


No need to put a commit message for a merge :)


As with the rest: just push commits onto `next` to fix this, or PR if you are unsure.


fixed


I got bored too, except I have no ideas. :P


I computed just the numbers on the diagonals eventually. I left all the
rest just because it took time to be written and it would have been cruel
to delete it.

On Sat, Jul 18, 2015 at 10:05 AM, Cosma Alex <notifications@github.com>
wrote:

> Check out this post
> <http://math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000>
> to see how to compute this in constant time instead of O(n)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SIonut/JavaMainRepo/commit/56163bc21b34de7f6885a559e2802df455a30d1d#commitcomment-12237172>
> .
>


Thanks for the link, Chris. I didn't know about the code style guide.


Notiz: "Revert" geht nicht AUF einen Commit zurück, sondern macht genau diesen rückgängig.
revert HEAD^ löscht also die Änderungen des vorletzten Commits, statt des letzten.


Hey - it is my!


I would leave checking to user, but this have same reverse side of the coin. So let it be this way. :)


@geodotka, czułem, że za dużo kliknąłem ;) Dzięki za nakierowanie!
(To był test wywołania :))


10 lords a leaping :tophat:


@tomwhipple if they not supported why `.so` files present in the sdks? i.e. https://github.com/paypal/PayPal-Cordova-Plugin/tree/master/src/android/libs/mips64


So it should be a dev dependency...


i just do not understand if it will be exported if alignX is set to Mid. the user will need to add it to "propertiestoinclude" or it will be automatic?


Rather than passing `jsonParser` here and then again on [line 77](https://github.com/reybango/thinkful/blob/232d20c92db77e5668afb81298a86347b06ecf71/lesson2/shopping-list/server.js#L77), you could also call `app.use(jsonParser);` once before you define any of your routes (i.e. around line 56)


One of the points of the module is to make the user not have to care how to check if it needs unzipping or even that you have to do such thing.

I'm all for ideas on how to make it clearer, in the name, docs, etc. but I strongly feel the if checking is the responsibility of the module. Same as we do a lot of things for the user in `got` so they don't have to care.


Yes, a mistake.


Заменил на "втянули", хочется сохранить аналогию с черепахой


Shouldn't this be moved to lsst-sqre?  I don't think we should ever depend on something in a personal repo.


updated


@bfontaine For now, as a walkaround, please change that problematic `gFlase` to `nullptr`.


I think this should work (assuming promise.value() is how to get the value in bluebird), the other pattern for this kind of thing is starting around line 38, where I redefine inspect, isFulfilled and isRejected for es6-promises.

For setting up the test, I am using the jasmine gem (there are also mocha smoke tests). There is a single shared example `itImplementsContracts` that I run for each supported library. It is all inside of mock-promises_spec.js. You need to implement an interface with `PromiseClass` (that returns the constructor), `PromiseWrapper` (equivalent to `Promise.resolve`) and `getDeferred` (returns a Deferred object with a `promise` property and `resolve` and `reject` functions).


Evgeniy, I dont think that it is a good idea.


ок, жильцы)


You can't make this change at the eleventh hour when the documentation doesn't state this is how it is going to work.  Using the "sweep" AI to determine the balancing is flawed, because the sweep AI really isn't an AI.  Scores can get out of wack if one sweep gets ahead of the other sweep.  I've created 2 AIs, and the better AI might only win by 5 to 10 anyways.  Artificially favoring the underdog isn't in the spirit of the competition, and what meeple would want to get on the losing elevator anyways?  Perhaps implement the possession arrow (to keep it 50/50) or revert it back to being random so one side isn't favored.


Good!


ДЕШКА fixed


@soniayang97 @imurchie , this seems to export nothing because `lib/util.js` does not have a default export.

Would be a good idea write a simple `index-specs.js` and check existence of exported method.


fixed


Есть же более новая версия коммита, там многое исправлено (была выложена до дедлайна).

Остальное исправлю в ближайшее время.


`Unexpected redirect to magnet URI.`


оставил


тут в значении, что всё устаканилось


This is fixed in commit [a58ac10](https://github.com/SanderA/cm/commit/a58ac10916f459b6c83db90a3c0a31fa5d3ff407).


вуш! Есть правила на порядок восклицательных знаков?!!!!1адынадын?!11)))


No, it will be automatic. Check it out — https://github.com/kangax/fabric.js/blob/2171dd13798458d4cfa383a8b2030ae1fa216ffe/src/shapes/object.class.js#L827-L838


Согласен


wasnt able to write the full summary:
*Confirmation Box shows up when you click Exit
*Added "Add Task" Button
*Added "Add Task* Window


The count of various words is surprisingly low. Across the two papers we are using right now (Farzan and Matthias), no word is counted more than twice! This means we will need a large number of papers to significantly learn from them.


Can't accept this. If the settings returns null or zero the loop will be screwed. Leave the default value.
Why change the name? The full-stop separated format is preferred, as the 'setting' field will be removed in the future so using those values make more work for converting.


updated, но синоним же


updated


look at the variable name ;). Either you've modified the wrong one orr the wrong one has been used in ui.R


Рома, спасибо!


automatic removing ok, but not automatic exporting.
alignX alignY and meetOrSlice are not in default object.toObject

or am i missing something?


updated. Но точно это
"приветствие"? Типа официального знакомства же


да, у меня чет его удалить не получается


В порядке бреда оставлю так)


Align it properly.


Nie szkodzi, najszybciej się człowiek uczy, jak sobie poklika i sprawdzi, co będzie. W razie czego ja mam aktualny kod, wszystko można naprawić ;)


This is unnecessary IMO.
The setting was deliberately set to be the same as the short-hand format so that it could be used even if the expanding method wasn't available.


They escape in https://github.com/ruby/ruby/blob/trunk/lib/uri/generic.rb#L552-L556 which references the RFC
> Within the user and password field, any ":",
   "@", or "/" must be encoded.
So using full blown `CGI.escape` is probably escaping more than necessary. I can change it to do just that

with regards to `#userinfo` and `#userinfo=` I could:
* drop `#userinfo=`, moving the logic to split the parts into `.parse`
* and make `#userinfo` private just used for `#to_s`

Does that sound good?

Also this reminds me, I was very surprised that I needed to call `not_nil!` inside the `if`—I've already proven that inside that branch it's not nil


\o/


Да тут вообще бред)


This gets compilation warnings:  should be a size_t.


'ensure' needs a final 's'


"в течение" калит меня) Не могу запомнить как надо)


This is for a hexahedral element, but these hardcoded face xi-directions were only temporary. I changed this into something more elegant in commit [a58ac10](https://github.com/SanderA/cm/commit/e368542b99bfb51464a3717d6dbdcf793fdad71c)


fack ton merge :!


fixed


ObjectGuid Guid = pet->GetGUID()


лол) ок, исчезла так исчезла


Sorry but how is this suppose to contribute?


Not a problem, its nice to have you on board. I am on holiday just now and will get back shortly


You mean the fact that we added them to `stateProperties`? I'm pretty sure they won't be exported. It's only for `_removeDefaultValues`


And what about void methods?


Hi @kmorel! Are you sure "cmake_minimum_required(VERSION 2.8)" or even "cmake_minimum_required(VERSION 2.6)" are not good enough here?


Извините, недосмотрела. Сейчас гляну новую.
--
С уважением, Кравчук А.И.

> On 18 Jul 2015, at 20:09, HelgaGebremariam <notifications@github.com> wrote:
>
> Есть же более новая версия коммита, там многое исправлено (была выложена до дедлайна).
>
> Остальное исправлю в ближайшее время.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/HelgaGebremariam/ASP.NET.GEBREMARIAM.DAY3/commit/e9f72aa5931a7644b7f5071f0bbb507786e832df#commitcomment-12239037>.
>


да о чём речь то?) Что она вообще хотела этим сказать?))


Согласен


I agree.


Looks like this just counts how often LMK is actually used, and if this is true, it would only be good for debugging


нет, не повторила, он ей не дал)


Whats the significance of the different return values here?


Спасибо) Не силён в ихних ругательствах


updated


Students/Goia I. Paula Ioana/Assig 1/Assig1.1/src/source/Main.java

Careful on the complexity here. Check out [this post](math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000) to see how to reduce it from `O(n)` to `O(1)`


Good job otherwise


1.Brady I made the changes to address above issues. And rebase the commits into branch
CWE-89-SEC_problem_encounter_improve.
2. I'm close to finish interface/patient_file/encounter/delete_form.php. After this file is commit I can work to completely convert problem_encounter.php to the new security model.


fixed in 55bc479


IClonable?


Okay, good to know!


Exactly, it's just for debugging ;)

I just added it as I was chatting with arter who is facing a pretty high lmk activity on S6, and we just wanted to compare.

I've had exactly 0 in ~4h.

This really doesn't need to make it into all kernels ;) Unless you're looking into lmk activity.

JP


Using sweep likely wasn't the best idea. I like the idea of adding the possession arrow to keep it balanced. I'll try to get that implemented asap.

—
Sent from Mailbox

On Sat, Jul 18, 2015 at 12:05 PM, randombits01 <notifications@github.com>
wrote:

> You can't make this change at the eleventh hour when the documentation doesn't state this is how it is going to work.  Using the "sweep" AI to determine the balancing is flawed, because the sweep AI really isn't an AI.  Scores can get out of wack if one sweep gets ahead of the other sweep.  I've created 2 AIs, and the better AI might only win by 5 to 10 anyways.  Artificially favoring the underdog isn't in the spirit of the competition, and what meeple would want to get on the losing elevator anyways?  Perhaps implement the possession arrow (to keep it 50/50) or revert it back to being random so one side isn't favored.
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/eonarheim/BellTowerEscape/commit/e923998e5af4a4ecdb0032fdafdf9e0830e4d065#commitcomment-12239026


also missing inicialization


So you constructed the whole matrix and then checked each element. Yes, that is one possibility, however, you are only interested in the numbers on the diagonals. Are you sure there isn't a more efficient way of computing only those numbers and not the others? Try playing around with this `side length` and see if you can find any little formula which might give you the numbers on the diagonals using this side length


Very well done otherwise!


Students/Alungei C. Paula Carla/AsignmentsFor1Chapter/src/MultipleOf3and5.java


intex -> index


[This post](http://www.geeksforgeeks.org/program-for-nth-fibonacci-number/) should help you with the Fibonacci computing :smile:


yea the bluebird promise already has isFuflled and isRejected on it, you have to access the state using value or reason.


Игорь, привет! :)
Мы собираемся завтра в кафе "Гараж" на ул. Кальварийская, 17. Присоединишься к нам?


Doesn't this mean it only checks map rules?


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


> small reusable pieces of code used throughout the code base


IMO this allows to write shorter and clearer code for integrating over faces, especially when calculating an element matrix/jacobian where there is a nested for-loop.


Привет, Андрей!
Завтра собираемся в кафе "Гараж" на ул. Кальварийская, 17. Присоединишься к нам?


just me being a klutz! fixed (again)


mistakenly used the same comment. This code generates the new files for the chord diagram


No problemo, you can make a different class for that or a new project. I don't mind


Would be cool to use relative links here


please move style to css, don't put it like this


compilation*


TEST


plz ignore this ;)


Rename this file to README.org -- I dunno why but it bugs me :)


https://github.com/robertogagliotta/openemr/commit/7bf7a45eacde5bab377f195154cac3bc1f54de60#diff-1da5e3a05ba3c35a0efcd2a655b5bdff
new branch commit


@vinniefalco Nice, thanks! We're not using boost directly but I might copy-and-adapt this into our code if we wind up using the pattern much more.


remove those white spaces ?


Students/Sporis A. Ioan Catalin/Palidrome/src/Twist1.java

you can write
`return (nr2==nr)`


Students/Sporis A. Ioan Catalin/SumOfMultiples/src/Multiples.java

Careful with this:
- 1) what if the number is a multiple of 3 `and` 5? it is counted twice => wrong result!
- 2) it's a very inefficient method. Check out [this post](math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000) to see how to compute it in liniar time


this should be showing the readme


good job otherwise


I had a couple of old suse enterprise boxes out of a few hundred that I just could not get to compile popt (and thus librsync) and this fixed it perfectly.  Thanks for this fix!


Thanks) its my first day on  githab


Now save common chunk by considering `nest_order` rather than only `group`. see following gist

http://bl.ocks.org/caijun/raw/4f3b8637b07f30934aab/


you re sure? the guid is empty for remove !?

I will rewrite all this packet in fiew time.


idem for remove !


Type on the class name.


This does not match the title below.


We try not to give multiple potential reasons for a rejection unless we also include the details so that the caller knows why it happened (as long as exposing why the request was rejected wouldn't create a security vulnerability). Remove 'disabled/', and you're good. For example, we try not to throw these sorts of errors: "You couldn't login because your username, or password, or date of birth, or account has expired". The caller has no idea what they should fix in the previous case.


makes geom_point version of FluView viz work.


@jlnwebster this `placeholder` is just padding, right?


You will want to include the host header that was sent in the request in the exception. A server might be behind a proxy and the proxy might rewrite the host header (thus invalidating the signature), and the only way the client will know is if we mirror the host header back to them in the error.


Fine, thanks a lot, I do that right now.


FIXMEs and TODOs usually have a colon following them (and people include that colon when grepping). Add the colon, please. It's important to have healthy colons.


:D We have two menus now, good.
Anyway we need a way to not display a menu at all, as for now isn't possible :D


Does Date() parse ISO8601 properly? I think it does, just wondering.


Great commands listed! This will be helpful if we ever need to look back on our work!


Add a TODO: for this line - check to see if we need sysStatus.


Your 'validation' error has a vadidation error.


This is a global const, so I would name it with capital letters CAPITAL_DIR_MAP


function removeCurlOption is not defined. Add this to this class:

public function removeCurlOption($key)
        {
            unset($this->curlOptions[$key]);
        }


Yeah think so. Not that it achieved anything


lklñkñlkñlkñlñ


#include<iostream>
#include<conio.h>

using namespace std;

void cargar(int v[],int m);
void mostrar(int v[],int m);
void ordenar(int v[],int m);
int Mayor(int v[],int m);
void Eliminar(int v[],int &m,int e);
void Inserta(int v[],int &m,int d);

void main()
{
	int n,op;
	int v[50];

	do{
		cout<<endl<<"****   MENU  ***"<<endl;
		cout<<endl<<"1.-Cargar ";
		cout<<endl<<"2.-Mostrar";
		cout<<endl<<"3.-Ordenar";
		cout<<endl<<"4.-Elemento mayor";
		cout<<endl<<"5.-Eliminar un numero ";
		cout<<endl<<"6.-Insertat un digito";
		cout<<endl<<"0.-Salir";
		cout<<endl<<"Introduzca una opcion:\n";
		cin>>op;
		switch(op)
		{

			case 1: cout<<endl<<"Introduzca la dimension del vector: ";
				    cin>>n;
				    cargar(v,n);
					break;
			case 2: mostrar(v,n);
					break;
			case 3: ordenar(v,n);
					break;
			case 4:int may;
					may=Mayor(v,n);
					cout<<"El elmento mayor es: "<<may;
					break;
			case 5:int e;
					cout<<"Introduzca el numero que desea eliminar ";
					cin>>e;
					Eliminar(v,n,e);
					break;
			case 6:int d;
					cout<<"Introduzca el numero que desea insertar ";
					cin>>d;
					Inserta(v,n,d);
					break;
			default:cout<<"Introducir una opcion valida: ";
		}
	}while(op!=0);
getch();
}
void cargar(int v[],int m)
{
	int i;
	for(i=0;i<m;i++)
	{
		cout<<endl<<"V["<<i<<"]  ";
		cin>>v[i];
	}

}
void mostrar(int v[],int m)
{
	int i;
	for(i=0;i<m;i++)
		cout<<endl<<v[i]<<endl ;

}
void ordenar(int v[],int m)
{
	int i,j,aux=0;
	for(i=0;i<m;i++)
	{
		for(j=i+1;j<m;j++)
		{
			if(v[i]>v[j])
			{
				aux=v[i];
				v[i]=v[j];
				v[j]=aux;
			}
		}
	}

}
int Mayor(int v[],int m)
{
	int i, may=-999;
	for(i=0;i<m;i++)
	{
		if(v[i]>may)
			may=v[i];
		}
	return may;
}
void Eliminar(int v[],int &m,int e)
{
	int i,k;
		for(i=0;i<m;i++){
			if(v[i]==e){
				for(k=i;k<m-1;k++)
					v[k]=v[k+1];
			m--;
			i--;

			}

		}
}
void Inserta(int v[],int &m,int d)
{
	int i,aux;
		for(i=1;i<=m+2;i++)
		{
			if(v[i]>d)
			{
				aux=v[i];
				v[i]=d;
				d=aux;
				i--;
			}
		}
}


@tdhock this commit makes the compiler save common chunk by considering `nest_order` rather than the only `group`. Besides, the renderer successfully renders the geom_point version of FluView viz. See following gist

http://bl.ocks.org/caijun/b53842f22fe0c75f26db


No, it's just because redditbooru doesn't have SSL, and I want the subverses to load as securely as possible, instead of loading the page over SSL while the images aren't.


By the way there was also a small mistake in one of the tests which I fixed here.


Isn't it already?
On 18 Jul 2015 17:57, "Oliver Faircliff" <notifications@github.com> wrote:

> So it should be a dev dependency...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/itsapi/github-listener/commit/095f2addf9b8599227c1370a6eb3b9beae8b7bf7#commitcomment-12239004>
> .
>


The change of damimage.com should not be here, I apologize for this.


remove this option (explicit is better)


That answer combination does not admit a solution, as long as the `root` variable contains a relative path.


Yeah it would loop a lot in some cases. But I think it's ok because a) it's not a very costly loop b) it will only be triggered in exceptional circumstances like you said c) it won't get stuck because it will begin to return the original polygon which always has at least 4 vertices as the tolerance reaches 0.

I'm not sure if reducing by 1 percent each iteration is right though. You could jump from 2 to some number of vertices greater than 3 in some cases.


Wouldn't this need to be `#include <windows.h>`? (I'm surprised it's not an error; it searches globally with <>)


Why did you remove "/main.lua" on line 14?


ありがとうございます． 15d725080bcadfb4ae3cb4bc2ec9cc2d0831bb57 で修正しておきました♪


test


@levicole I added this back in as a way to show an individual bookmark page. This probably needs to be done a different way so we can discuss further.


@levicole this needs work because it's deleting the topic instead of the bookmark so I commented it out for now.


js是直接搬运的。留个坑以后填


Yeha, you are right. I wrote that above already, that it needs an **and**.


Edited: `getConfigPath()` -> `getConfigFilePath()`


I could be wrong, but...

I don't think you can directly modify `tags` this way... It's one of those things that might seem to work initially, but the iD code makes assumptions about the immutability of entities and so modifying them directly will break the user's edit history.  You haven't written a history state, and `entity` might not even be to a current entity anymore.

I *think* you'd need to do something like:
```
var id = entity.id,
    ent = context.entity(id),
    tags = _.merge(_.clone(ent.tags), { wikidata: qid });

context.perform(iD.actions.ChangeTags(id, tags),
    t('operations.change_tags.annotation'));
```

But even that is a problem when called asynchronously because the user could be moving something, or doing an edit within a bunch of `context.replace` calls, or even in save mode already - so having a history entry written with `context.perform` could be bad.

I'm not really sure the best way to perform an asynchronous tag change.  Maybe @jfirebaugh can suggest something?


This was already added in the develop branch, looks like you are behind.

https://github.com/jarias/stormpath-sdk-go/blob/develop/account.go#L66

Maybe the name I use is not the best, but the function is the same as yours


@levicole this successfully deletes the bookmark but in doing this I've lost the way of actually linking to the bookmark url. I'll need to refactor the topic#show view so the actual bookmark link works and I can edit, delete and view the bookmark.


They aren't present in the latest PayPal Android SDK, so it is consistent to take them out here.

In any case `decider.so` just returns `false` to an `isArchitectureSupported()` for anything other than ARMv7. It was broken out separately due to a bug in Android 4 that caused ARMv7a to incorrectly return `false`. In other architectures like MIPS we can just deduce the non-supportedness by the lack of a library. And reduce binary size at the same time.

I'm not sure anyone actually makes a MIPS Android phone anyway. ;)


I'm first going to point out that you left out the underscores from those and used spaces, which breaks their use in sorting the sitreps; I would try to never leave them out.

As for skipping "Custom_3", I don't see what you're talking about, except that I didn't have an example that uses it, is that it?  I just gave the three examples the priorities I thought they should have.  So, I guess the answer is "yes", I think it is normal that you might not use all the possible labels :smiley:


Its actually not in boost yet (and might never be) so you would need to copy it anyway


1. These folders should not be commited
.gradle
.idea
build/

2. app/src/main/res/layout/fragment_main.xml
 android:text="A confidential Safety resource for Peace Corps Volunteers"
should be added in strings.xml

3.

<string name="hello_world">Hello world!</string>
Unnecessary string

4. These files should not be committed

local.properties


Done.


App and DB dockerized for development

### Dependencies

* install [boot2docker](http://docs.docker.com/mac/step_one/)
* install [docker-compose](https://docs.docker.com/compose/)

### Setup

#### boot2docker
* run 'boot2docker up'
* run 'boot2docker start'

#### docker-compose
* run 'docker-compose build'
* run 'docker-compose run rake db:setup'
* run 'docker-compose run rake db:seed'
* run 'docker-compose run rake db:populate'
* run 'docker-compose up' (should start webrick on port 3000)

### View app

* run 'boot2docker ip' to get VM's ip
* visit vm's ip port 3000 in browser

Possible bug: starting/restarting the server might not work due to "server already running". Simple delete '/app/tmp/pids/server.pid' file to be able to start it again.

@tawan App im Browser bei mir extrem langsam (ca. 30 sec pro Request). Evtl. ein Problem mit DNS - siehe [hier](https://github.com/docker/compose/issues/356).


Thanks to @irobin591 for the corrections :)


no need


@vedi can you take a look?
@jonyrock  what do you suggest?


No. I meant people who makes the changes and work it out themselves not
afterwards which causes headache(and extra work). Especially Windows
people, there is git config for this
https://help.github.com/articles/dealing-with-line-endings/.

Velho

2015-07-18 9:34 GMT+00:00 Trond B Krokli <notifications@github.com>:

> @Velho <https://github.com/Velho> : the change in this commit is to make
> the entire source follow the same standard.
> Would you rather like that the source code is a mix of CR/LF and LF-only
> text, randomly based on what each user chooses to set in their version of
> git?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TrinityCore/TrinityCore/commit/57091cc5f105f5196ffaed1ec91d8dddc2b360c3#commitcomment-12237573>
> .
>


I thought map.Rules were already merged with the default ones. This mirrors https://github.com/Mailaender/OpenRA/commit/bac95b3aab7ee431a5e9ee16a290842e62537c18#diff-798e91ffea524d28cd5569c232e8d368R36 btw.


can you make this a generic username? for most out-of-the-box Postgres installs, you'd want to use `postgres` as the username here


> I don't check if it was fleet with different missions but it already works in that way.

Oh, I wasn't clear enough-- I meant that they would wait there for all (or at least some significant portion or something) of  the mission members to arrive.  Right now fleets whose target is the next system and which have compatible missions (like attack and secure) will group up if they happen to be in the same system at the same time.  But currently if they start at different points they might easily head to the target system via different paths.  It might not be feasible for them to all stage at the same system, but they could at least check that the other fleets are in the last system prior to heading into the target system, etc.

> Can you say little more about abilities?

I just meant generally however we have the code accomplish that behavior-- whether via special orders, or whatever.


ukinana


Where do these values come from? Normally we use `.env` for this, which is in the `.gitignore` file. This file should probably be listed in `.gitignore` also if we keep using it.

Remember this is a publicly-accessible repo. So this token should be considered compromised even if it was removed from the repo


думаю, что лучше таки выбросить ArgumentOutOfRangeException


You're mostly right. I definitely broke things here.
I refactored out this commit from my WIP on MUC, where the code was right, because I needed to resolve targetID early, so I could set it right after base channel construction.
https://github.com/Kaffeine/telepathy-nonsense/blob/master/connection.cc#L725

You're right. I broke it and I really have to come back and fix it, probably with muc support.
Thanks for pulling me back!


Yeah, I had not yet looked at other lint checks when I made that comment, nevermind.


:+1: That seems like fun :exclamation:


> You can just remove 'elif main_mission_type' condition and have same effect.

Ack, no, I think I need to fix the line, not remove it lol.  I'll have to look at it more carefully, I think that is supposed to be "main_mission_type not in" rather than "main_mission_type in"


So, I haven't gotten to the bottom of it yet, but I have definitely noticed something strange and seemingly dangerous about the decorator I wrote. There seem to be conditions in which this decorator can be called and the entire cache directory contents are deleted.

It could be that the way you implemented this in Astropy does not produce this behavior, and this is specific to something dumb I have done with my own implementation. I'll try to get to the bottom of my end soon, but I thought I would mention this asap before your PR is merged, in the event that this bug also plagues your end.

cc @embray


They could suffer from the same problem. I don't know if I have to do manual rule merging here.


1. These folders should not be commited
.gradle
.idea
build/

2. app/src/main/res/layout/fragment_main.xml
 android:text="A confidential Safety resource for Peace Corps Volunteers"
should be added in strings.xml

3.<string name="hello_world">Hello world!</string>
Unnecessary string in strings.xml

4.These files should not be committed
local.properties


From my experience the other yaml checks work fine, the issue @pchote pointed out should be the only reason why this doesn't work properly.


This would not be legal on MSVC. For now I'll tolerate it since this is not likely to be compiled on Windows.


1. я понимаю, что здесь была попытка оптимизировать хранение коэффициентов, исключая нулевые, но не согласна с логикой, поскольку теряется смысл, основанный на массиве - коэффициент при нулевой степени в нулевом индексе и т.д.
2. непонятно, что такое сравнение double числа с 0 (coefficients[Degree] == 0), например, 0.0000000001 равно 0 или нет?


Are you really sure it impacts performance that much? I mean one extra conditional check compared to the whole processing should be negligible.


Where this library is coming?
Why do you need it compared to `net/http`?
I'm not very confident using a different implementation than the original go one...
Are you sure the missing feature (or bug fixes) haven't been applied now to the regular net/http package?


they should be exported.
we are fixing in another PR a pathgroup that can t be exported to object because transformMatrix is missing, and the same day we create another situation in wich the object will not be restored correctly.

they are for sure rarely used options, but worth exporting


I would stay away from `__forceinline` unless it's clear that the compiler is screwing the perf badly.


At least it will trip some code from resulting binary.


Ok, I can remove that. It is just that inlining those functions make sense because they are used only in one place. But again function call is cheap too.


Thanks @irobin591
Going to fix that :+1:


gloup...


I tried extending `ObjectConstructor` so that the `<any>` would not be necessary, but as far as I can tell you cannot currently add static methods to a TypeScript interface?


This is first repository.


Your regex can throw NullReferenceException.
For example if the play.spotify.com link is like:
play.spotify.com/album/<albumid>
so it does not contains user or an userid


this breaks the test suite -- can you change to `postgres`? bad form for me to use my own username!


> It is funny that MSVC even allow to pass r-value as non-const reference.

They allow it has an extension. About the change itself, I wonder what I missed previously when I wanted to change that. I clearly remember that something made me think it was not possible to use a const reference here.


e.g. I know I set a bad example here


This is the first repo.


методы Equals немного "мутноваты"
1. сначала проверяем ссылки - ReferenceEquals
2. потом есть смысл проверить степени полиномов, если они не равны, то дальше нет смысла сравнивать
3. а вот если степени равна, то сравниваем при одинаковых степенях, и максимум искать не нужно
4. код следующего метода public override bool Equals(object obj) будет читаться легче, если заменить условие на обратное obj != null (кстати, Ваш код этим периодически грешит:))


I was also sure that we edit style at some point there. But apparently not. :)


Are you sure they aren't inlined anyway? Compilers are generally not bad at inlining functions. Generally MSVC is a pretty decent compiler regarding optimizations as far as.


> They allow it has an extension.

It's not a bug, it's a feature. I see.


Hello Peter this is terrific!   Is there any chance I can convince you to post a .gem file some place?    I'm working in cygwin, and the bundler really doesn't want to work with your branch.

Thank you,
Jason


Не нашла перегруженных операций сложения, вычитания,умножения


Yes, I explicitly checked that. Even `inline` doesn't inline. But I also doesn't like being that intrusive with those `__forceinline` Thought we already does that for all `pixmix` and related functions.


I'm not completely convinced but if you think the gain is worth it I won't argue.


I guess those `Draw...` functions are too big to be considered useful candidates for inlining.


I actually prefer h3's the way github formats them...


figures added


Weird. All I changed was the url for that link ...


I agree, the loop is not costly, if we try alter the 1 percent, we could reduce the amount of looping, but I couldn't think of an easy way of doing it. We're essentially solving for tolerance, so we could do a simple binary search for the right tolerance that returns a triangle (or even better, that returns something better than a triangle if the input allows).

I think with documentation, we could leave the onus up to the user to make sure they adjust tolerance if they get triangles ... So I'd leave current implementation as is.


I was using the "go depth" command today, and I noticed that at depths below 9 this is much faster.  Maybe we can implement this just for those first 8 plys.  I tested it again at 8 threads and several hundred positions using "deep position analysis" in Fritz. This took 1 min 50 seconds to complete the particular depth 8 analysis, while the current master took 5 minutes and 54 seconds with the same position and settings.  It probably does not amount to a hill of beans as each individual search to depth 8 is very fast regardless, but it is an interesting result.
If someone was doing a Monte Carlo, it would almost certainly help.


I'll resubmit. Don't pull


If this is the issue, it's probably relatively straightforward to solve
by replacing `self._benchmark_dir` by
`os.path.abspath(self._benchmark_dir)` on lines 513, 526 in
asv/benchmarks.py:Benchmarks.run_benchmarks
.
If this is the problem, it's rather strange that it was not seen before.
Is there something special in your benchmark setup?


figure 1


Crap, this means we need to move genReplicationId to the `extras` API, because I am using it in pouchdb-load via copypasta currently...


Nice.


`"Could not load GeoIP database. Reason: %1"`


Why there?
Should stay before the "Sticky header" section I think.


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | TIMEOUT | Skip
 ---------|---------| ---- | ---- | ------- | ----
 basic_test|suite_setup|1|0|0|0
basic_test|basic/test_create_vm|1|0|0|0
basic_test|basic/test_stop_vm|1|0|0|0
basic_test|basic/test_start_vm|1|0|0|0
basic_test|basic/test_reboot_vm|1|0|0|0
basic_test|basic/test_add_volume|1|0|0|0
basic_test|basic/test_add_vol_to_stopvm|1|0|0|0
basic_test|basic/test_add_volume_reboot_vm|1|0|0|0
basic_test|basic/test_add_multi_volumes|1|0|0|0
basic_test|basic/test_add_volume_negative|1|0|0|0
basic_test|basic/test_crt_temp_from_volume|1|0|0|0
basic_test|basic/test_crt_vm_with_volume|1|0|0|0
basic_test|basic/test_vm_securitygroup|1|0|0|0
basic_test|basic/test_create_iso_vm|1|0|0|0
basic_test|suite_teardown|1|0|0|0
virtual_router_test|suite_setup|1|0|0|0
virtual_router_test|__host/test_reconnect_host__|0|1|0|0
virtual_router_test|novlan/test_create_vm|1|0|0|0
virtual_router_test|novlan/test_reboot_vm|1|0|0|0
virtual_router_test|novlan/test_ping_vms|1|0|0|0
virtual_router_test|__host/test_reconnect_host__|0|1|0|0
virtual_router_test|vlan/test_create_vm|1|0|0|0
virtual_router_test|vlan/test_reboot_vm|1|0|0|0
virtual_router_test|vlan/test_snat_create|1|0|0|0
virtual_router_test|vlan/test_dnat_reboot|1|0|0|0
virtual_router_test|vlan/test_ping_vms|1|0|0|0
virtual_router_test|vlan/test_create_3vms|1|0|0|0
virtual_router_test|vlan/test_start_vm|1|0|0|0
virtual_router_test|vlan/test_3l3s_vm|1|0|0|0
virtual_router_test|vlan/test_remove_dhcp|1|0|0|0
virtual_router_test|vlan/test_2l3_vm|1|0|0|0
virtual_router_test|vlan/test_crt_vms_with_limited_ips|1|0|0|0
virtual_router_test|vlan/test_crt_vms_with_limited_ips2.0|1|0|0|0
virtual_router_test|vlan/test_crt_vms_with_limited_ips2.1|1|0|0|0
virtual_router_test|vlan/test_start_vms_simultaneously|1|0|0|0
virtual_router_test|vlan/test_crt_static_ip_vm|1|0|0|0
virtual_router_test|vlan/test_crt_vm_with_hostname|1|0|0|0
virtual_router_test|volume/test_add_volume|1|0|0|0
virtual_router_test|volume/test_add_m_vols|1|0|0|0
virtual_router_test|volume/test_add_volume_vr|1|0|0|0
virtual_router_test|volume/test_crt_vm_volume|1|0|0|0
virtual_router_test|volume/test_crt_vm_2_volumes|1|0|0|0
virtual_router_test|volume/test_crt_vm_24_volumes|1|0|0|0
virtual_router_test|volume/test_volume_ops_on_stopped_vm|1|0|0|0
virtual_router_test|volume/test_crt_vm_from_dvt|1|0|0|0
virtual_router_test|template_image/test_crt_temp_image|1|0|0|0
virtual_router_test|template_image/test_del_temp_image|1|0|0|0
virtual_router_test|sg/test_ingress|1|0|0|0
virtual_router_test|sg/test_egress|1|0|0|0
virtual_router_test|sg/test_icmp|1|0|0|0
virtual_router_test|sg/test_udp|1|0|0|0
virtual_router_test|sg/test_stopped_vm|1|0|0|0
virtual_router_test|sg/test_vm_ops|1|0|0|0
virtual_router_test|sg/test_2vm_egress|1|0|0|0
virtual_router_test|sg/test_2vm_ingress|1|0|0|0
virtual_router_test|sg/test_4vm_complx|1|0|0|0
virtual_router_test|sg/test_detach_l3|1|0|0|0
virtual_router_test|pf/test_2l3s_pfs|1|0|0|0
virtual_router_test|pf/test_attach_pf|1|0|0|0
virtual_router_test|pf/test_delete_pf_after_destroy_vm|1|0|0|0
virtual_router_test|pf/test_del_vip_before_add_pf_to_vm|1|0|0|0
virtual_router_test|pf/test_multi_pfs_sg_tcp|1|0|0|0
virtual_router_test|pf/test_multi_rules|1|0|0|0
virtual_router_test|pf/test_pf_after_destroy_vm|1|0|0|0
virtual_router_test|pf/test_pf_sg_tcp|1|0|0|0
virtual_router_test|pf/test_pfs_tcp_udp_same_vip_ports|1|0|0|0
virtual_router_test|pf/test_pfs_tcp_with_same_vip|1|0|0|0
virtual_router_test|pf/test_tcp|1|0|0|0
virtual_router_test|pf/test_udp|1|0|0|0
virtual_router_test|pf/test_vm_ops|1|0|0|0
virtual_router_test|eip/test_2_eips|1|0|0|0
virtual_router_test|eip/test_2l3s_eip_pf|1|0|0|0
virtual_router_test|eip/test_2l3s_eips|1|0|0|0
virtual_router_test|eip/test_attach_eip|1|0|0|0
virtual_router_test|eip/test_create_eip_vm|1|0|0|0
virtual_router_test|eip/test_del_vip_before_add_eip_to_vm|1|0|0|0
virtual_router_test|eip/test_eip_sg_tcp|1|0|0|0
virtual_router_test|eip/test_pf_eip_attach|1|0|0|0
virtual_router_test|eip/test_vm_ops|1|0|0|0
virtual_router_test|eip/test_vm_ops2|1|0|0|0
virtual_router_test|__snapshot/test_backup_snapshot__|0|1|0|0
virtual_router_test|snapshot/test_create_100_snapshots|1|0|0|0
virtual_router_test|snapshot/test_create_snapshot|1|0|0|0
virtual_router_test|snapshot/test_create_sp_after_delete|1|0|0|0
virtual_router_test|snapshot/test_create_image_by_snapshot|1|0|0|0
virtual_router_test|snapshot/test_crt_sp_for_both_root_...|1|0|0|0
virtual_router_test|snapshot/test_crt_sp_in_stopped_vm2|1|0|0|0
virtual_router_test|__snapshot/test_crt_sp_in_stopped_vm3__|0|0|0|1
virtual_router_test|snapshot/test_crt_sp_in_stopped_vm|1|0|0|0
virtual_router_test|__snapshot/test_delete_snapshot_on_ru...__|0|0|0|1
virtual_router_test|__snapshot/test_crt_sp_in_live_vm__|0|0|0|1
virtual_router_test|snapshot/test_crt_sp_from_vol_crt_f...|1|0|0|0
virtual_router_test|snapshot/test_change_max_sp_depth|1|0|0|0
virtual_router_test|__snapshot/test_change_max_sp_depth_l...__|0|0|0|1
virtual_router_test|regression/delete_sg_with_2_attache...|1|0|0|0
virtual_router_test|regression/rm_nic_sg_on_stopped_vm|1|0|0|0
virtual_router_test|regression/test_data_volume_with_vm...|1|0|0|0
virtual_router_test|vr/test_ping_vms_reboot_vr|1|0|0|0
virtual_router_test|vr/test_reconnect_vr|1|0|0|0
virtual_router_test|vr/test_start_vrs_simultaneously|1|0|0|0
virtual_router_test|__host/test_reconnect_host__|0|1|0|0
virtual_router_test|suite_teardown|1|0|0|0
Total|102|94|4|0|4


As github does not have "issues" page I am going to post my bugreport here:

The gem posted at rubygems.org is broken:

The gem posted at rubygems.org is broken

```
]$ gem install wikitext
Building native extensions.  This could take a while...
Successfully installed wikitext-4.1.1
Parsing documentation for wikitext-4.1.1
Installing ri documentation for wikitext-4.1.1
Done installing documentation for wikitext after 0 seconds
1 gem installed
$ ~/.gem/ruby/2.2.0/bin/wikitext
/home/anatol/.gem/ruby/2.2.0/gems/wikitext-4.1.1/Gemfile not found
```


This needs fine tuning.
From this [FAQ](https://www.maxmind.com/en/faq#howoften):
>**How often are the GeoIP2 and GeoIP Legacy databases updated?**
>
>The GeoIP2 and GeoIP Legacy Country and City and GeoIP Legacy Region databases are updated every Tuesday. **All other databases are updated on the first Tuesday of the month.**


I ended up removing the route, found a way to elegantly not need it


We could also resolve templates for `edit` and `new` views and handle them in Twig, by passing all needed arguments to render a form field, I still don't understand why ya'll want to restrict this great behavior to only `list` and `show` :confused:


I think we can replace `to use` by simply `use`.
*semantics* :laughing:


Ok, thanks, I can reproduce this easily. Fix incoming... :)


For which reason? I doubt that the numpy version is significantly better and it blows up code. Furthermore you do an import in a function which might delay the request.


Fixed. Thanks!


figure 2


Thanks! Yeah,I wasn't really thinking about the complexity when I did that problem. Oh,I checked the link you sent me but it doesn't work ,but I'll look for a better solution for the problem :D.


figure 3


getEventPlayer devuelve PlayerHolder y eso es lo q se usa en el evento


config_types.h and ogg.pc are generated in the build directory (CMAKE_BINARY_DIR).


figure 4


Nice separation.


porq usar L2ItemInstance en lugar de L2Item?
Estoy usando casi lo mismo q los itemHandlers.


Maybe I am paranoid and I am doing microoptimizing which is bad. Here is my suggestion for this segment
```c++
QString *country = m_cache.object(hostAddr);
if (country)
    return *country;
```

It avoids one lookup in the cache...


Hi @JGAndersen! I'll fix that now. In the future, you can just do `bundle install --without development test`


No dije nada.


initial template tht was forked


o de checar o mapa não vai funcionar, o getmapusers não é checa se o mapa existe e sim se há jogadores, se não haver jogadores vai retornar com 0, se não haver o mapa pode haver até um travamento no npc, recomendo fazer uma função especifica na source para fazer isso.

Na verdade o algorítimo é até simples, segue o nopaste:

http://nopaste.linux-dev.org/?646133


Fixes search display in spear. #366


No sé exactamente la diferencia entre uno y otro pero sí sé que L2ItemInstance tiene más información sobre el ítem en sí (de hecho, en la clase que se toca del core usa L2ItemInstance). Yo creo que si la gracia del engine es que sea abstracto y extendible, debería abarcar la mayor cantidad de info los listeners, quizás en algún momento se usan.

Además no cuesta nada.

(Chequeá que quedó mal el parche del core, falta un parámetro).


I appreciate the refactor.  Nice extraction of header and section processing.


IIRC, `catalog` is mostly used internally. The "public" name of the property is `translation_domain`. I'd recommend to change it for consistency with Symfony docs about translation


That is a bug but it only effects the "link" listed on the job, it was pulling from the lsst-sqre/ec2-snapshot repo.  It was fixed about a week ago by https://github.com/lsst-sqre/sandbox-jenkins-demo/commit/be752b1f5a9346094fa1fc641939822ae40a755a


(Idea from FSM on IRC)


Ah. I see.


Love the refactor into re-usable, well named, functions.  It makes readability much better.


Yeah, possession arrow would keep it balanced.  Random should keep it balanced over 500 turns.  Possession arrow works perfectly when there is 1 and 1 or 2 and 2 stopping at the same floor.  However, if 2 and 1 stop at the same floor you would expect one AI to have a 2/3 chance of getting the meeple instead of 1/2 with the possession arrow.  This is getting complicated...


You either delete `%1` or you use arg().


Using devices such as Jawbone Up, Nike FuelBand, and Fitbit it is now possible to collect a large amount of data about personal activity relatively inexpensively. These type of devices are part of the quantified self movement – a group of enthusiasts who take measurements about themselves regularly to improve their health, to find patterns in their behavior, or because they are tech geeks. One thing that people regularly do is quantify how much of a particular activity they do, but they rarely quantify how well they do it. In this project, your goal will be to use data from accelerometers on the belt, forearm, arm, and dumbell of 6 participants. They were asked to perform barbell lifts correctly and incorrectly in 5 different ways. More information is available from the website here: http://groupware.les.inf.puc-rio.br/har


It's `/tmp` ;-) it's the usual temp system folder


ok, lo volvere atras en mi branch o almenos q quieras cambiarlo tu ahora mismo


You could store ´/tmp/$SCRIPTNAME` in a variable, so you won't have to repeat it here


Lo arreglo yo, don't worry.


cuando termine con el tema de los buffs retomare este listener y estudiare mas el tema


Logger::instance()->addMessage(tr("Successfully updated GeoIP database"), Log::INFO);


This date being present was causing Airmail 2 (an OS X mail client) to incorrectly change the dates on emails.


`"Couldn't load GeoIP database. Reason: %1"`


Repo for Coursera data science toolbox project


`"Couldn't download GeoIP database. Reason: %1"`


LF EOLs are already being enforced ever since https://github.com/TrinityCore/TrinityCore/commit/5b078786a21ccc0ca27835e52cfaf3d381ee7f50

Half if not most of the issues addressed in this commit were caused by me back when I used shitty text editors like notepad to edit SQL files.

Now the point of bumping week old commits to point out issues that were already solved I don't really know. That's why I say,  there are both more important and interesting things to be done at TC.


Post is a draft – let’s discuss when or if to publish. @JonHMChan


Some more clarification over IRC:

```
12:27 <+Not-1f01> [PyLink] GLolol 024ac16 - relay: Only allow messaging users in common channels / channels that you're in The old behavior, which iterated over the pseudoclients representing the sender, dropped messages to clients without a common channel without any warning. Now, the sender will get a lovely notice from the PyLink client. Trying to send to a '-n' channel without being in it: -PyLink-devel- Error: You
12:27 <+Not-1f01> must be in '#channel' in order to send messages. Trying to message a user without sharing a common channel: -PyLink-devel- Error: You must be in a common channel with 'GLolol/testnet' in order to send messages. Closes #62.
12:27 <@Satin> shouldn't that be a +n channel
12:27 <@Satin> not -n
12:28 <@GLo|o|> Satin: +n prevents sending to channels you're not in
12:28 <@GLo|o|> it's a default mode on every sane ircd, but it is possible to turn that off
12:28 <@Satin> yes
12:29 <@Satin> so wouldn't sending to a -n channel without being in it be allowed
12:29 <@GLo|o|> yes, but pylink doesn't allow that - it only spawns a client for someone if they're in a channel on the remote network
12:29 <@Satin> oh
12:29 <@Satin> i see now
12:29 <@GLo|o|> i'm following janus's style of not polluting /lusers by quitting users without a channel :O
```


this should probably call `emit_static_alloca`, although if you are allowing non-SSA, you'll need a way to handle the extra copy required for cases like `x = new(T, getfield(x,2), getfield(x,1))`


GitHub is hard to use


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 167](https://teamcity.labkey.org/viewLog.html?buildId=261560&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 20 Build time: 00:36:11
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

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

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

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

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

```


I apologize for forgetting php!


Mutex was not unlocked.


@FernetMenta

In order to solve the old drama about setMimeType and distinguishing to not do Head Request, should the python API be extended now to add a method: setContentLookup(true | false) - so that addons can use this functionality?

Did you have that in mind back in our discussion here: https://github.com/xbmc/xbmc/pull/6753#issuecomment-84855153


I think this is invalid yaml.


lol


un NPE que raro, igual asi como esta no cumple ninguna funcion....voy a testearlo en mi ultimo branch


I think there are APIs two and from the web/wallet. I'm not sure "from the Web to the wallet" covers it.


not better store as con?


this assumes that only one allocation from this alloca is live at any given point, so static_alloca should be fine. Then, wouldn't your case be covered by simply emitting all the fields before the object in the on_stack case ? The only downside would be a bit of additional gcframe traffic but well.


¿Qué cosa no cumple ninguna función? Tira NPE porque si está en VOTING todavía no hay ningún getNextEvent


:) я здание не допонял :) я понял что надо вывести все которые не повторяються (следуя этой логики всё правельно работает) но свою ошибку уже понял


No modification for errors?


Seems that the Dark theme in section pages is a cool stuff that everyone wants.. :)


me refieria al "replace"


Why bother :) Errors might be pretty verbose as is.


Sorry, I don't think I understand. read_more_link doesn't work? Did my contribution break something?


right, but in that case you need to take up two gc-slots since the old and new object are alive simultaneously during that instruction and we use new the object itself as the temporary gc-root (assuming the object is not isbits)


Ah, claro.


After this change, the hash on 'internal' parts of the expression are taken into account when computing max length of password.  Also, SIMD logic added.  32 bit SIMD can only use 247 bytes of the 256 byte buffer (we have to add the 0x80 and bit count), and the 64 bit SIMD can only use 239 bytes of the buffer.

Here are some sample runs after this fix:

```
$ ../run/john -test=0 -form='dynamic=sha512(md5_raw($p).$p.$p.$p),debug' | grep MaxIn
MaxInputLenX86=74
MaxInputLen=74
$ ../run/john -test=0 -form='dynamic=sha256(md5_raw($p).$p.$p.$p),debug' | grep MaxIn
MaxInputLenX86=77
MaxInputLen=77
$ ../run/john -test=0 -form='dynamic=skein512(md5_raw($p).$p.$p.$p),debug' | grep MaxIn
MaxInputLenX86=80
MaxInputLen=80
$ ../run/john -test=0 -form='dynamic=skein512(md5_raw(md5_raw($p).$p.$p.$p).$p.$p.$p),debug' | grep MaxIn
MaxInputLenX86=77
MaxInputLen=77
```

So here the first expression is limited by the sha512.  It can use only 239 bytes, and 16*74*3 is 238 bytes.
The 2nd is limited by the sha256 to 247 bytes.  16+77*3 is 247 bytes.  The third is full length (256 bytes), since skein512 is not SIMD hash.  So 16+80*3 is 256 bytes.  The forth even though the skein512 expression could handle 80 byte values, is limited by md5 within the inner-inner expression. md5 is a 32 bit SIMD, so it has the same limit as the sha256 in this case, 77 byte password (and the 16 byte raw-md5 residue).

So, this is showing that the code is now computing the proper max length passwords we can use for a given format within the constraints of the Dynamic format.


Could be. I'll consider this when refactoring the thing.


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


why would you do this


well, I meant just emit every argument to the new() into a "register" and root it in the arg temp space (i.e., stop using x as a root for all of those in the stack case)


Ignore me


угу, понял


Во всех задачах под “дано” подразумевается ввод соответствующих данных через параметры командной строки. Если не указано иначе, предполагается, что в переменных хранятся целые числа.

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


We have ready trait with .map_any


This is a bit of shit. But I really wanted to make it work :p and I'm about to go out for dinner. So I might try to make it look a bit better afterwards. :) :dancer:


lovely .lib files u hav there


ah. yes, i guess that's not bad


наверное надо заставиться циклом это переделать, угу понял


yes, something like SetContentLookup needs to be added to CFileItem.


btw sorry if i missed up your gradle file with that path change.  i didn't realize i had done that.


here is the log

http://codepad.org/gtssx33f


also, moved the part about changing account balance from User.pay to Payment.update_record to accomodate paying to multiple person


You don't want this to be done implictly by `filter`?


I'll switch to that if you like. It felt like this would only make the errors harder to read.


Fantastic changes! I love the biome stuff!


I rather be explicit here.


:D


:P


1718_setpoint_managers (mbadams5) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


(NSString *)componentHierarchyDescription uses the topview (window).
This method allows you to specify a view as well as whether or not to print upwards or downwards.
Not quite sure how chisel works, but I'd imagine this needs to be public.
But you would know better than me, I've sent you a link to the in a private message to the line in chisel.


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/71510891


> I think it's for portability, and so they can run even if `zsh` is not installed on the system.

That was my assumption, too. But what's the real-world use case for installing `oh-my-zsh` without also installing `zsh`? To my slow-witted mind, there are none. How could there be? `zsh` is a hard runtime dependency.

I have no dog race in this race. It just seems a bit... *peculiar* not to leverage `zsh` for `oh-my-zsh` installation. Doing so would simplify most of the installer's current portability concerns. For example, `zsh` provides builtin access to the current terminal's `termcap` capabilities (e.g., via the `echotc()` command and `zsh/termcap` module).

> There are separate PRs open to address the installer's echo portability issue (#3664) and other Cygwin portability stuff (#3646). They also replace the hardcoded ANSI escape sequences with terminfo lookups.

**Excellence.** I'll finesse this commit out of the pull request.


expose a constructor function called Symbol::Create for c++ user might be better.

Move this function to mxnet_api.cc call Symbol::Create and set the params


consider using full name Symbol, would make it more clear


ok


fast-forwarding master to auto = 255fa8de


cleanup_unit_test (mbadams5) - i386-Windows-7-VisualStudio-12: OK (1759 of 1759 tests passed)

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


This is used not only for api, creating symbol from config could use it too which can be c++.


I know what you mean, I guess the function was the same, maybe we should reconsider the name of api_registry and change it to some general name of registry. In general I think one place of registry would be better


Hi @plfiorini I fixed greeter CMakeLists QtAccountsService OPTIONAL issue ;-)


@levicole the previous suggestion works to save a new bookmark without generating the error; however, for some reason the topic_id is "nil" when the bookmark is saved.
```bash
2.2.0 :006 > b = Bookmark.last
  Bookmark Load (0.1ms)  SELECT  "bookmarks".* FROM "bookmarks"  ORDER BY "bookmarks"."id" DESC LIMIT 1
 => #<Bookmark id: 20, url: "abc", topic_id: nil, created_at: "2015-07-18 02:10:06", updated_at: "2015-07-18 02:10:06">
```


Make sure you have the @topic.bookmarks.build as well, and associate it with the user too.

Sent from my iPhone

> On Jul 17, 2015, at 9:16 PM, Steven Filler <notifications@github.com> wrote:
>
> @levicole the previous suggestion works to save a new bookmark without generating the error; however, for some reason the topic_id is "nil" when the bookmark is saved.
>
> 2.2.0 :006 > b = Bookmark.last
>   Bookmark Load (0.1ms)  SELECT  "bookmarks".* FROM "bookmarks"  ORDER BY "bookmarks"."id" DESC LIMIT 1
>  => #<Bookmark id: 20, url: "abc", topic_id: nil, created_at: "2015-07-18 02:10:06", updated_at: "2015-07-18 02:10:06">
> ―
> Reply to this email directly or view it on GitHub.
>


add destructor to the entries, or use unique ptr on entries


forward OnSpawnDrugs()


No.


cleanup_unit_test (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (727 of 727 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-7.03%25-green.svg)


That sounds great, but my concern is all of the existing calls that try to use printf formatters instead of the `{}` that plFormat uses.


cool


I'm not actually an experienced Python developer at all - this probably needs tidying somewhat before merging, and it may even be possible to improve performance even further by removing the loop on line 446/455 completely


test


Thank you <3 <3


I think the sleep should go in onTouchEvent, they way it works right now is kinda weird. Basically a touch event is invalidating the view which triggers onDraw where the move is made. I think onDraw shouldn't change the state of the game and just worry about drawing whenever it needs to. You could invalidate before you make a move which will draw the latest move then wait and make an AI move and invalidate again (I think) for the same effect. Then again, the code was already kinda convoluted, I should probably refactor into a board class that has nothing to do with drawing and have the UI change the board state and the board trigger invalidates to redraw.


Hi Krzysztof, I will see when the error happens,  I think  it was after I
have edited some parameters.

Will do some more typo corrections soon.

*Tweaks.*  The weight works well with many of the groupboxes .  So a user
can "grow" or "shrink" a groupbox  influence over  maybe 1 keyframe
interval and  another over  3  keyframe intervals. The possibilities are
infinite. Most of these tweaks will be very good for animation

Weight could also be subdivided into weight.x, weight.y , weight.z.    But
there are already so many tweaks.

A tool tip may be needed to advise users that with mandelbulbs, the fabs()
 type functions often work best with low powers .

BTW in my current temporary UI layout, the last boxfold is not required,
because I think setting the first boxfold to start at iteration 1 produces
the same effect.

If any user wants more tweaks, then they could have the option of following
the "Basic guide to write your own Mandelbulber Formulas, using Ubuntu and
QT Creator." I managed to do it, so I think a lot of people should be
capable. BUT this may generate a lot of "please help me " emails.

There are also Initial Conditions tweaks to be considered.

Which and how many tweak groupboxes that Mandelbulber should have, has to
be decided. I am not sure how it should be decided. I think I may have
already produced too many and maybe  a maximum of 7 or 8 groupboxes would
be good amount to start with on the hybrid tab, like:-

Box fold
Rotation
Spherical Fold
the "4 constant" fabs() group box

*go to MAIN FORMULA, then:*

Rotation
constantmultipler
additionConstant
the "4 constant" fabs() group box

*go to ADD JULIA else CONSTANT MULIPLIER.*

More can always be added later.

It is for you to decide.:)

Cheers

Graeme

On Sat, Jul 18, 2015 at 4:07 AM, Krzysztof Marczak <notifications@github.com
> wrote:

> Error like:
>
> Warning: slotLogLineEditChanged() error: value from "logedit_DE_factor" is
> not greater zero</i>
>
> can happen when you edit some parameter manually (not by slider) and for a
> while it's not a valid number. When that errors are generated? When you
> start application, when you edit some parameters, or when you close
> application?
> I have pulled latest revision and I can't see that errors. So it's not
> because of your latest edits. You did everything very well.
>
> About this problem:
>
>
> *The only current problem is that the tooltip for color button adds a new
> parameter description each time you use it.*
> I need to check ColorPaletteWidget class. There must be some bug.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/buddhi1980/mandelbulber2/commit/9620b313cfc8011a7f9d5620dcc5e9f395c9bbca#commitcomment-12228425>
> .
>


:gun:


I love coffee : , pizza , and sports


cleanup_unit_test (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (1759 of 1759 tests passed)

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


this is a comment


Explanation of the `; 1` seemed a bit unnecessary given that it was used just a few lines above the diff for pretty much exactly the same purpose.  And that's a *lot* more verbose than any commit message I've seen on any other project.  But ok, I'll amend it.  It's the weekend now though so it'll take me a couple days to get to it.


You give more meeple to the player with the lower score?...


rax retry tests


Whoops, @greduan not @eduan - sorry @eduan


@ccx-testerbot runtests master


Missing an indent there


note to self: this branch is forked from files-cache ... rebase b4 PR


commit


Update


Merged. Thanks!


Technically https://hg.adblockplus.org/easylist/rev/d6ff372bbece should take care of these filters


change usage (example) with correct command


where dem resources at


@patricknelson Hmm, three years ago ;) It'd have been something around switching from one area to another, either from a ModelAdmin, `admin/pages` overview, or (most likely) and `admin/pages/edit/show/<id>` detail view. Shouldn't serverside form validation render both for full page and PJAX reloads thougH? Happy to review a pull request.


in uart.h
#define TX_BUFF_SIZE    100
may be
#define TX_BUFF_SIZE    0x100


此修改是错的，看Lua群


Here you go! :D https://github.com/silverstripe/silverstripe-cms/pull/1238


I asked since I was trying to carefully investigate the basis here so I didn't break anything. Hopefully I didnt.


I observe CodeIgniter changes, they prefer elseif. :-) Not so important.


The digests are always cached, and this doesn't make sense anymore.

(We're clearing them on every request, so no biggie.)


@jksg I'll update to max-height


In this case `skip_download` should be set in order not to waste time downloading.


I have to revert this change in order to get the plugin to actually work. $this->link is empty when save and getInfo are called, so the db_escape_string returns nothing, and so the consumer key & access token are never set.

I'm running tt-rss 1.9, FWIW.


Seemingly this alone wasnt the problem. We are still investigating what else Airmail could be creating magic dates out of.


My best guess at this point is our Message-Id. I don't think these are unique, since it's `<group/{groupid}@...>` instead of unique for that message.


Can you give this method one parameter.  And this parameter accepts a number.  It should replace that 100 you have there.
example
sayHello(23) should print out "Hellox23"


Just for the record, this commit is too big for my taste which makes it difficult to know why certain things are the way they are. For example, the blogs-api module seems to be new and it's build, gradle, ivy files just an skeleton, in which case it should have been its own commit. I also think that using the settings API should have been done in small steps explaining why each class was added.


what  source ? latest  rom from ivan?


This if clause is weird. I'm guessing that you copied it from somewhere else. I'm also guessing that it was done because if portletResourse exists then it means this is the configuration portlet and if it's the configuration portlet then there is a form and that form is for changing the settings and we want to keep the values in the params in case of failure....

Too many assumptions. The if clause should be much more clearly related to the consequence (reading request params before the settings), or else a small change in the chain will break this code.


их просумировать надо или просто вывести рядом? 2 числа (1 число из массива 2 число то самое вещественное "В") я переделал под этот вариант


Looks good! +1


Not quite right...  it needs to take in a parameter

sayHello(9) should print out "Hellox9"
sayHello(45) should print out "Hellox45"
sayHello(101) should print out "Hellox101"


ㅇㅁㄴㅇㅁㄴㅇ


The changes has being tested  at: select patient- select encounter – add form WORK NOTE  – save it—and Delete


how to add custom http headers in request?


This line was causing the MSb range bit to overflow into the justify bit


I hadn't realized that some of the data might be REPL-only. I think the best approach would be to do a first pass were everything is included in the `FooExpr` records to make things homogeneous and easy to refactor. After we settle on how it should work, we can go back and remove/adapt those REPL-dependent records.


Hello @hboutemy,

I think I found a solution for this:
* Set `<pubScmUrl>${project.scm.developerConnection}</pubScmUrl>` in the configuration of `maven-scm-publish-plugin`, see dfac2d7b7bd4eeffa6f33988be8d567f65028669
* Derived projects need to set the property: `<scmpublish.content>target/staging/${project.artifactId}</scmpublish.content>`


This doesn't look right to me... the method was named `fail`, not `raise`...


Test Comment


Ah, I see. Please squash the later commit that fixes this into this commit using `git rebase -i origin/master`. Thanks!


I have to revert this change in order to get the plugin to actually work. $this->link is empty when save and getInfo are called, so the db_escape_string returns nothing, and so the consumer key & access token are never set.

See also https://tt-rss.org/forum/viewtopic.php?f=1&t=1859&p=9279#p9279

(I'm running tt-rss 1.9, FWIW.)


clean up


Cool


Understood


Stair issue fixed with 7fb5c25


Just realized this has some flaws, related to not checking Location conditions (which hadn't mattered for current weapons); am working on fix.


Great!
 job that is right!


instead of this, u should assign a default value in the schema / db


@murtra I replaced the Mailchimp signin form with a ConvertKit one. Please make sure you are not conflicting or if you are, that the signin form still works after resolving the conflict. Thx!


If a format can have various max. password lengths, do we report the actual password length? Should we? For which --verbosity?


@item.mark_as_complete
then do the normal ... if @item.save


u should be using .update not update_attribute as update_attribute skips validations.
http://api.rubyonrails.org/classes/ActiveRecord/Persistence.html#method-i-update

I think u should call it .mark_as_complete


`user` and `password` remain nillable on line 87 because they're calls to nillable methods, and also because of the possibility of concurrent access to the underlying nillable instance variables.

http://crystal-lang.org/docs/syntax_and_semantics/if_var.html (at the bottom)


shouldn't


Should probably point out that internal consistency checks should only fail if there is a failure which may result in incorrect operation and can only result from hardware failure, miscompilation, memory corruption, or a serious bug in the library.


I think you should remove these comments. It is very self-explanatory what checks are needed here if we support more record sizes in the future.


Not new but its a bit weird that `self.tables` is a list of adapters.


higher than usual liklyhood of some namespace conflict-- e.g. some fool system header. Keep in mind ending with _t is a posix reserved namespace.  Can I suggest secp256k1_callback_type instead?


Checkout [this commit](https://github.com/sshaw/gendered/commit/121cd1a21ff075692c073f6a167aeb4ee222a628) (pull request [here](https://github.com/barelyknown/gendered/pull/7)). It allows one specify connection options.

Check http.rb's wiki for [how to use HTTPS](https://github.com/httprb/http.rb/wiki/HTTPS).


Feel free to fix that convention library-wide.


The like is kind of buddy if the caption is too long, but works fine if it on one line.


<b>Using the median filter:</b>

let imgray = img.gray();
setLeft(imgray);
setRight(imgray.medianFilter(2));


I don't fully understand why we need the Helpers introduced in this commit. They add quite some complexity and I'd like to be sure of whether it's really needed.


Specifying the index should not be needed, but I kept it anyways, maybe I'm missing something...


This is a much better example of what applying settings/config admin should look like.


Is there a reason why portlet.properties is still needed?


This change breaks the recursive nature of checking directories added in e03c7b823eadae89ee6405ad0359f96a8132df69. It now only checks the Go package in specified directory, rather than it and all children directories.

Is that intended? I think I can still achieve the same effect by doing `./...` but I find it strange that this commit changes recursive behavior of checking directories and does not mention it at all in the commit message (making me think it was unintentional).


and spaces


Update: auf Linux normale Responsezeiten. Also DNS / boot2docker Problem.


I will change this right away


TNX for your effort


To be complete I think there are a few more places in scope manager that could be incorrect..
`usedPredefinedAndGlobals` uses `Object.keys` to get the list out for data.. as I understand your research, this would be broken in old javascript environments. I guess its difficult to fix without adding a specific case for it.. Are we just happy this is such a corner case it isn't worth fixing?

The same with `impliedGlobals` (it uses `_.values` to get data) which also, is inconsistently created with `= {}` which I guess means that additionally it could cause a crash on this line `impliedGlobals[usedLabelName].line.push(undefinedToken.line);` - would you mind changing it to `Object.create(null)` in this PR?


I think we should change the mode to 0700, we're mounting the user's backups there (although probably noone will be able to read them other than the current user due to limitations in fuse).


Not good way if we not in current directory i make new ome


Thank you for fixing the translation


Les fonctions de loading qui sont dans le global file c'est pas bien

Faut utiliser les variables de appclass qui sont faites pour


Sounds good! I have some thoughts on the testability of this change; I'll include them in a separate comment


They will be exported only if they're different from default values. Basically nothing changes right now. The behavior is the same. There's no point in exporting "none" value if it's a default value. If it's missing, it will become "none", if it's "none" it will become "none". No difference. Just like we wouldn't want to export transformMatrix == [1,0,0,1,0,0] (identity matrix that has no effect on transformation). You understand what I mean?

Sent from my iPhone

> On 18 Jul 2015, at 14:42, Andrea Bogazzi <notifications@github.com> wrote:
>
> they should be exported.
> we are fixing in another PR a pathgroup that can t be exported to object because transformMatrix is missing, and the same day we create another situation in wich the object will not be restored correctly.
>
> they are for sure rarely used options, but worth exporting
>
> —
> Reply to this email directly or view it on GitHub.
>


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 168](https://teamcity.labkey.org/viewLog.html?buildId=261574&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 0 Build time: 00:07:07
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

```


So right now this can't be used, e.g. to turn an argument failure into a handable error code because execution continues after the check fails (unless the callback does something insane like manually rewinds the stack!).

I don't know this this is a /problem/ but I bet the first thing that users are likely to do with the argument checking callback is convert it to error print and don't crash, and then be really surprised when their program crashes anyways.

We could make arg check return after calling the callback for those cases, but I haven't completely thought that through.


Maybe that's because of s.h.i.e.l.d right?


Thanks for the heads up. I've turned issues on, and I'll investigate the cause of this issue within a few days (currently at hospital with a newborn baby).


There should be some kind of unit test to demonstrate this problem before fixing, right?


Nice Øne


Hi,
Thank you for the improvement. Sure, please create a PR. I will update the npm.


I'm mimicking the validation in the wp_cache_sanitize_value function, as well as other places in wp-cache.php when sanitizing the inputs. The strip tags is there because none of the options require tags. The esc_html turns quotes into their html entities to avoid escaping of the string.

Given the limited number of options/settings, each setting that imports a string could be treated uniquely with their own sanitization filter. That should strengthen the sanitization of the options instead of relying on a single esc_html for all the string options.


Congrats with this event!!

I'll post it to issues page to easier tracking


Students/Fericean G. George Vlad/Assignment1.1/src/first_pack/Sum.java

Careful on the complexity here. Check out [this post](math.stackexchange.com/questions/9259/find-the-sum-of-all-the-multiples-of-3-or-5-below-1000) to see how to reduce it from `O(n)` to `O(1)`


Careful with the formatting but looks good otherwise


Good point. I create a list of options/settings to compare against before importing. f01bff6fb82dc816d28c802c9e2eb9dc052f5b8a


@fernandog yep I think so, now just to source them properly.


This could be `declaredInCurrentScope`..
You might be able to tidy it further, removing the variable and moving the check outer shadow check to the `else`


Why not setting it to false?


Great and efficient approach


do you not need to set a flag that it is *not* a directive? You want this so that it always calls validateParams? at the moment doesn't it still do

```
       var directive = state.tokens.curr.value;
       if (state.directive[directive] ||
           (directive === "use strict" && state.option.strict === "implied")) {
         warning("W034", state.tokens.curr, directive);
       }

       // there's no directive negation, so always set to true
       state.directive[directive] = true;
```


FYI, MIT is more of a code license.