The other strings could disappear after refactoring from edX, I cannot
guarantee that!

On Sun, Nov 1, 2015 at 11:48 AM, Ahmed Aljazzar <notifications@github.com>
wrote:

> @OmarIthawi <https://github.com/OmarIthawi> Why you specified it for
> Edraak? "My Courses" strign is already in edx-platform; check this link to
> make sure of that:
> https://www.transifex.com/open-edx/edx-platform/translate/#ar/$/62608489?q=My%20Courses
> <https://www.transifex.com/open-edx/edx-platform/translate/#ar/%24/62608489?q=My%20Courses>
>
> However, it is functioning well.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Edraak/edx-platform/commit/0f8c247f8a4cf367219328f98824ad38792aa6c8#commitcomment-14113272>
> .
>

--
Omar Al-Ithawi

Senior Engineer, Edraak.org <http://www.edraak.org/>

Queen Rania Foundation

for Education & Development

T +962 6 4016464  Ext. 700

M +962 7 90574405

E oithawi@qrf.org

<https://twitter.com/edraak> <https://www.facebook.com/edraakorg>
<http://www.youtube.com/user/edraakorg> <http://instagram.com/edraakorg>


dann kann es in die sub SIGNALduino_IdList($$$)


ja, so ist es besser


Конечно, с использованием ООП многое можно сделать проще, но в рамках процедурной реализации, по-моему, возможно только 2 решения: либо выделять логические переменные для хранения последнего вызванного метода (lastCallWasByte, lastCallWasString и т.д.), которые нужно будет проверять в каждом открытом методе (минимум 3 if'а); либо  реализовать Logger как конечный автомат с набором состояний.
Последний вариант, по-моему, более понятный.


Actually, I was thinking we should check the "extras" on the posters and use the one that has the "latest" value?


@mjackson Won't `push` and `replace` support second `query` argument?


Correction of typos in the file naming scheme


I opened a pull request with the code that was tested.


Еще раз поясню. Вы сейчас зачем-то ограничиваете себя в использовании стандартных ссылочных типов, этого совершенно не нужно было делать. Достаточно, ограничивать себя в ООП дизайне своего кода. Хотя и это делать не обязательно, вот что мне на уточняющий вопрос написал Женя:
> мы начали говорить об ООП.
> я им разрешил попробовать по нашей модели “сначала сами – потом правильно вместе”


Du coups pourquoi tu en cree un ? un mec peut tres bien avoir comme pseudo "anonymous". il faut ne rien mettre

    user = ""


Could you please bump the version and release an update for bower?


Oh oh, a typo I did while batch replacing all with sed on command-line. Mea culpa. Thank you for pushing the fix!


ja, funktioniert
2015.11.01 10:01:16 3: Calling Getting Attr sub with args: del whitelist_IDs =
2015.11.01 10:01:16 3: sduino IDlist MS 0 1 13 14 15 2 20 22 23 3 4 6 7
2015.11.01 10:01:16 3: sduino IDlist MU 16 21 5 8 9
2015.11.01 10:01:16 3: sduino IDlist MC 10 11 12 18
und nach dem restart:
2015.11.01 10:02:55 3: sduino IDlist MS 0 1 13 14 15 2 20 22 23 3 4 6 7
2015.11.01 10:02:55 3: sduino IDlist MU 16 21 5 8 9
2015.11.01 10:02:55 3: sduino IDlist MC 10 11 12 18


Thank you, I'm updating the server right now with all changes :)


Ah, thanks for the update! Sorry for missing this morning!


Thank you for pushing/polishing for Talime! I'm updating the server now.


Обычно статусбар в играх скрывают, полноэкранных приложений ни разу не видел, а насчет экономии места, то можно использовать CollapsingToolbarLayout


В данном случае основной мотив - улучшение читабельности: у метода isSumOutOfRange(...) 4 аргумента, а у isSumOutOf(Integer|Byte)Range(...) 2 аргумента + название, говорящее, с каким именно диапазоном работает метод


Ideally, we need to come up with a better solution for user control of priority then well, priority or install date


Nice.


@fefifofum i did not even cloned you source I can't synch 1gb of source just to comply with your demands..if you want I can just leave the development for you guys..I'd be happy to do so..


implemented in https://github.com/alexeykudinkin/intellij-rust/commit/c7951afa8c0f19063fd0d72d7a44830a30ba4216


This is a comment.
(Just testing.)


Another comment.
(testing)


Since you mention this kicks in at depth >= 18.  Do we have any info on what depths get reached on fishtest at STC and LTC?  I have always wondered.


General comment.


Test comment.


Also, added bilateral and gaussian filter for sigma values of 2.


Hey I am Tempus Thales from SA, and I am trying to use this Auth for my Goon Team but I need to ask you a couple of questions...  Could you shoot me a PM with your SA name so I can ask you a couple of questions?
Thanks!


Any reason why you removed moment from globals? essentially breaking code that depends on moment.


that might make merging changes from the real memtest more difficult...


Die Currencies verwenden die ISO-4217 Codes als ID. Es sollte möglich sein, darüber die Wechselkurse zu laden. Genaueres zu den Codes hier: http://www.xe.com/iso4217.php


Ben Kothe


Much better! :)


Hi @xbmc/team-kodi have there a stupid question ;)

I'm still on rework for the libKODI_guilib to make developer friendly (see https://github.com/AlwinEsch/kodi/blob/internet-addon-handler-2/addons/library.kodi.guilib/libKODI_guilib.h), which becomes done due to changes for internet browser where also another Kodi Dialogs must available for add-on and was then so big that nobody can see what it do.

On places only handled by headers are the ```const char*``` replaced from my to ```const std::string&``` but was a bit confused that on compiled lib side always std::string becomes used for "GetDescription", additional is related to there a memleak (string array becomes before allocated from Kodi).

How you see the use of ```std::string``` on library? Is this acceptable? I'm absolutely not sure now with this place.

If acceptable (where I think no) can it be replaced by something like this on library:
```C++
string cGUIProgressControl::GetDescription() const
{
  if (!m_SliderHandle)
    return "";

  std::string text;
  text.reserve(1024);
  ((CB_GUILib*)m_cb)->Control.Slider.GetDescription(((AddonCB*)m_Handle)->addonData, m_SliderHandle, text[0], text.capacity());
  text.shrink_to_fit();

  return text.c_str();
}
```
And to this inside Kodi:
```C++
void CAddonGUI_Control_Progress::GetDescription(void *addonData, GUIHANDLE handle, char &text, unsigned int iMaxStringSize)
{
  CAddonCallbacks* helper = (CAddonCallbacks*) addonData;
  if (!helper || !handle)
    return;

  CGUISliderControl *pControl = (CGUISliderControl*)handle;
  strncpy(&text, pControl->GetDescription().c_str(), iMaxStringSize);
}
```

<b>Currently is this with memleak inside!:</b>
```C++
const char* CAddonCallbacksGUI::Control_Progress_GetDescription(void *addonData, GUIHANDLE handle)
{
  CAddonCallbacks* helper = (CAddonCallbacks*) addonData;
  if (!helper || !handle)
    return NULL;

  CGUIProgressControl *pControl = (CGUIProgressControl*)handle;
  std::string string = pControl->GetDescription();

  char *buffer = (char*) malloc (string.length()+1);
  strcpy(buffer, string.c_str());
  return buffer;
}
```
Thanks a lot to all


it work!


Omg...a regression will sneak in just because of 2 lines of code.


I think Marco is right.   Any failure mode is benign(unless someone can think of a case) so why defeat any optimizations by using atomics?


Oh, I need to push a new version. Ight yo.


creo que hay que salta para atras a este commit. aqui se borro busqueda_costo_uniforme


It doesn't.


Well, ok now.
I ve pushed Test1 for some items xlation.
Thanks n rgds / jbdwi


Should I add the configure-time fu to check for `Term::ReadKey'?


It needs to, otherwise this change will trash validation performance for miners.


Remove "one" imo


sharif


Do you see any reason not to move the `operator->` specification from `common_iterator` to the head of the Adaptors section, and re-use it for `counted_iterator`, `move_iterator`, and `reverse_iterator`? (A la https://github.com/CaseyCarter/cmcstl2/compare/operator_arrow)


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


We could also just do a runtime check.


compiles and works locally


@phase Well, it needs to compile C code, since the interpreter is written in C...

Can you try the [newest Fbuild commit](https://github.com/felix-lang/fbuild/commit/61ca5dd20138a02ad7604f59308d1c16fcd09f2f) and see if it works?


That was a mistake :speak_no_evil:


Use tabs instead of space.


ありがとうございます！
deferとguardについては若干避けていたので、このように提案頂けると使用意図が分かって「何のために使うのか」というのが非常にスーッと入ってきました！本当にありがたかったです！
もうちょい自分の中で揉んでマージしたいと思います！


Needing an El Cap bottle is legit cause to pull new ones :wink:


Please don't leave commented out code in the commits, after a while it won't be apparent if it is there as a left over, for debugging reasons, or if it is accidentally commented out.


Two comments about this block (and the testscripts one).

1. Really not convinced `respond_to?` is necessary.
2. If the main point is to produce an array without duplicates, use the `Array.uniq` method. It'll simplify the code quite a bit.


OK, then. For other users, please provide some documentation.


thanks!


Was leaving debug arguments in the committed code intentional? :wink:


LGTM, good job! :+1:


I'll take that as a yes :-).


Typo.

vvendor -> vendor


good


This commit again contains a lot of commented out code... Let's clean this up; either keep it if needed or remove it completely. If for some reason you will need to get back to what was originally there, you can see the project's history.


LGTM


1: M110 missing component name


For anyone confused by the description, it means that the ```is``` property will now be set automatically.


In some places you removed the @package tag and in others you do not. For a proper docblock, I believe it needs to be there. I see that @category and @subproject have been removed by PHPDoc and that @project supports multiple levels.

What changes need to be made in the wiki?


Note: this does need testing.


Always update with AO Date


Please test this fix in sybase database and make sure it works in sybase database.
Thanks,
Dale


ah, I was trying to use OpenAL ( w/out Soft ) 1.1
http://www.openal.org/

instead of OpenAL Soft 1.12+
http://kcat.strangesoft.net/openal.html

okay headers are in subdir "include/AL" -- on the Compile webpage, suggest changing "OpenAL" explicitly to "OpenAL Soft" ...


Fixed.


Fine


Not a tipo... since ``User#toString()`` returns the ``userid``, it should be equivalent.


Build time reduced dramatically as expected.

The npm install for test dependencies still takes time but in most real applications I feel like that wouldn't be a huge problem.


Your commit messages are getting shorter and shorter :D


I'll give it a shot


You forgot this recording:
+* `June 2, 2015: Server-side Backbone <https://www.youtube.com/watch?v=dm6L-CfrmwM>`__


Yeah, Creative's OpenAL stuff for Windows (which, I think, was supposed to do hardware EAX on Creative cards?) is outdated, partially broken and abandoned. It's dead, Jim. In fact, the openal.org URL expired earlier this year (last year? don't remember) and Ryan Gordon (icculus) got the URL in an auction, and is currently hosting it, AFAIK, with a mirror of the old data.

For Windows, Linux, and probably the BSDs too, OpenAL Soft is the prefered OpenAL implementation.

> on the Compile webpage, suggest changing "OpenAL" explicitly to "OpenAL Soft"

Well, we're still using Apple's OpenAL on Mac OS X, which is not OpenAL Soft [1]. Still, I should maybe explain that on the compile page in the wiki. I'll add that tomorrow.

[1] Though I heard it's limited somehow (in what way? no idea). Maybe we'll have to switch later.


@iamfat  记错了， `qf` 是 `TRUE`, `gini` 是 `true`. 记错了，记错了


Just tried on a clean machine  -- same issue.

```
make[1]: *** No rule to make target `build_lib_static'.  Stop.
make: *** [x86_64-apple-darwin/rt/jemalloc/lib/libjemalloc_pic.a] Error 2
```

Not sure why this is happening for me. I'm sure other people are able to build rust...


It would surprise me if this did not pass.  With this more accurate king safety, it might permit a slightly higher king safety as well.


`buy_order` and `sell_orders`could be methods, would save passing them around as arguments.


I'd prefer if you kept the deps using the non-compiled sources.  would you mind running depgen.py again?


Thanks!


Looks good


        tur.setheading(0)
        tur.forward(40)
        tur.right(90)
        tur.forward(60)
        tur.right(90)
        tur.forward(40)
        tur.right(90)
        tur.forward(60)


Can you add this file to the ones that depgen.py knows about? There's a hard-coded list in the file itself.


Done, I didn't understand the difference between `--full` and `--full_debug`.


Nice! :)


After you add the file to depgen.py, you should be able to run src/depgen.py and get the full list of uncompiled deps.


@ppeczek Out of curiosity, which OAuth 1.0 provider are you using this library with?


Thanks :sweat: !!

BTW, it's a little bit gnarly, so if you have any ideas or machinery that would make it easier or cleaner, feel free to replace it or otherwise refactor it! :smile:

I'm pretty happy that this worked, and I'm also really glad you thought to put this stuff into Agda, since I have found and fixed a few bugs(?) in the paper in the course of the formalization effort.


Historically we've _always_ updated the docs after the fact.


Yes, it's not well documented.

running with no flag, regenerates html files with all the deps.
running with --full_debug concats all the js and then uses that in the HTML files
running with --full cancats all the js, compilesd, and then uses that in the HTML files.

I probably should make a note in the README


Test should be stopped, there is no reason to continue it. It's wasting a lot of resources.


:sparkles:


Fixed


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


A smiley face!


As some of the inner classes of the HeapCodecFactory were used from the Direct one anyway I just moved everything back to this base class and removed the Heap one completely.


fixed


Might have something to do with [this](https://github.com/rust-lang/rust/issues/24887)?

Not sure why this would be happening… :(


Ben helped me figure out what was wrong with it by the way! Also I did a sorta hacky move by just wrapping the python library heapq with a class & replacing our other min priority queues with it.


Cool… yeah it looks like it should be possible to tidy it up a bit but I'll have to play around with it and see what works. I was hoping the Agda would help us with some of the details but I'm actually a bit surprised it came together so quickly. Pretty happy about that too :)


The NoOpCompressor that is currently defined is implementing the BytesCompressor interface, here I would need a similar one for the hadoop compressor interface, should I add it here?


Same question as the similar comment above.


To keep the format is the same as ClientCertRequestHandlerInternal.java, I add this empty line.


Sometimes our advanced pawns do help particularly if a rook is behind them.  The rook can support pawn pushes, so long as the pawn is mobile.  The can bust up a pawn shield or promote holes.  But in general, it should be good.


O(logn) for sort string and O(logn) for each string. Could be faster.


Retirar o rodapé que está duplicado no arquivo index.php


FuncPtrStubs は確かに fptrstubs.cpp とかに実装のある識別子ですね


console logs in above code yield precisely the necessary information.

However, in the Mustache template I needed some means of identifying "courses," (keys) that were at the top level of the object, without explicitly naming them--a variable, in short.  It was easy enough to create an array of those header names, but menu.map returns an error (there's a layer in between the menu variable and the objects inside the soup and salad arrays).  Needed a way to create an array variable and to be able to iterate over it, yielding both the names of the top-level keys, and the key/value pairs below.


Very interesting anyway!
May I suggest the following simplification:
    Depth d = std::min(1, abs(18 - depth));
...
    depth = std::min(1, abs(18 - depth));


お疲れ様です。
レビューありがとうございます。スペースとtabの混在、演算子前後のスペース、以後気を付けます。

2015-11-01 0:33 GMT+09:00 kazuaking <notifications@github.com>:

> 中身についてはもう少し見させてください。
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/a3095194/algorithm/commit/fbeea4fc82171c97de52ddb3681bfedf9e9bc677#commitcomment-14109585>
> .
>


@contentfree I pushed [backburner 1.2](https://rubygems.org/gems/backburner). Thanks for your hard work making backburner more reliable. I would definitely +1 moving our workers to https://github.com/ruby-concurrency/concurrent-ruby.


Okay, so I implemented the pitot tube cover in the branch `bug-303`:

- added pitot tube cover 3D object from the bonanza + improved texture
- click on the cover to remove it, click on pitot tube to put it back
- if left on, airspeed indicator doesn't respond
- state saved between sessions (as well as the airspeed indicator condition)
- hoovering tooltips change if pitot tube cover is there or not

Maybe in the future someone can help me improve this 3D model, I think it should look a bit more like this:

![eg](http://airfactsjournal.com/files/2011/03/pitot-tube.jpg)


While this works fine when querying manually, to use ActiveRecord's associations you need to name the table `lists` for `list_id` to be an appropriate foreign key.


I think the diminishment from there being two files is contingent on the opponent having a rook to put on the other.


PV_measure-sketchup (jmarrec) - i386-Windows-7-VisualStudio-12: OK (2338 of 2366 tests passed)

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


`Task.update` normally operates on the entire table, not an individual task. You probably wanted to have the user pick a task_id, then do `task = Task.find(task_id)` and `task.update(yadda: yadda)`.


Why 1?


Up a line...


agree. but it's up to @ajithcj to stop it.


The way to write it would be: `tasks = tasks.where(completed: false) unless options[:completed]`


you can pre-alloc jobs with batchSize before.


Will you try the same trick at line 286 ?


Are these the only alerts with this subject? Maybe we can change it to highlight missing active subscriptions for retries?


Looks good. Approved.


Yeah, we are using a different style here it turns out


Only my local db :)


Wrong Condition constrain, should be ||
Try:
		int[] A = { 1, 2, 3, 4, 5, 6, -1, -1, -1, -1, -1, -1, -1 };
		int[] B = { 1, 5, 7, 8, 10 };
you'll see


![fbuild-failure](https://cloud.githubusercontent.com/assets/7685133/10872937/5dff3256-80c1-11e5-9590-1453fe64afd1.gif)
Nope.jpg


I'm not sure what Im missing here--

  var drinksArray = favoriteDrinks[characters]

    return drinksArray


@huangp This may get rid of the "upper bound" error, but presumably at least one of these libraries requires a newer version of Validator than the one we have.

We may need to look at downgrading some libraries.  Which libraries were bringing in the newer version of Validator?


Go ahead


nvm, I guess the log msg is descriptive enough


There's no need to use `belongs_to` and `integer :foo_id` together. One or the other will do, I prefer the latter style because there's less Rails magic to think about. :P


yes, you are right.


AllCols will get all the columns in table, yes, it is in  in delete only state, also we can remove data, u can see line 500, i only check  `col.State != model.StatePublic`


Commit vocabulary is correct, diff is correct.  Thanks!


QoI -> qoi


Great! Thanks!


Looks like github is caching the old logo image. We may need to modify the logo to bust the cache, append `?abc123` or something.


Ahh, yeah. That's probably not needed... will need to look at the usages of it here though: https://github.com/search?utf8=%E2%9C%93&q=GIT_SHA+user%3Asoundcloud&type=Code&ref=searchresults


@knagato :tada:


(おっさん、そこ文字列や)


二宮くんもだったんですけど、gem のrmagickって画像の編集用ですか？
僕使った記憶が無い& 簡単なバリデーションだったらcarrierwaveの設定ファイルで事済んでしまうので、
特に使用用途が明確に成っていない場合は消してしまって大丈夫だと思います。


It is not used because we commented out the feature to find non-unique request IDs (which is part of the addRmRecordToDataset method and creates a NonUniqueRequestIdEvent). Would you recommend for the next commit to comment the related unused variable too?


Why did you remove these docblocks?


Just curious, but shouldn't this `edict-change-dictionary` call be part of flyspell-mode-hook too?


これ一気に書いてあげましょう！


ok.


Done: e3cca4286205e8d29c58918145f0c1a4b83270fa.


work's check


Connecte toi sur skype auj !!


Hi folk. When will the multiple search functionality be available through composer?¿


If the position locked earlier,it would brake harder than be commanded for zero velocity.When the velocity close to zero,lock the position again,so the quad wouldn't go back but just stay there.How about this solution?


Have you reported this? I'm pretty sure we have some in our regular translations, for emails and such. Probably not an issue for the FAQ, but still.


@ldonis yo opino que si, para que vaya de la mano con el dominio? Siento mas amigable gt que GT... me contas.


userのコントローラーは今回devise内蔵のやつで事足りるかと思うので、
わざわざeditアクションを定義してあげる必要はないです。


dh


改行


trueはコーテーションつけてもつけなくてもどっちでも大丈夫ですよ。


atoi is a C and C++ function.
http://www.cplusplus.com/reference/cstdlib/atoi/
http://en.cppreference.com/w/cpp/string/byte/atoi

So it's both in the std:: namespace as well as in the global namespace (depending on your included headers)

std::vector is only in std::, same for stoi (which is a C++11 introduction) so won't compile without std::

isinf is a macro in C from the <math.h> header. And a function in C++11 in std:: when you include <cmath>

(Note that visual studio totally fucked up C compatibility and added underscores for just about any C function in C++, expect to break visual studio compatibility at ever step you take)


form_forでeditを行う場合、明示的にvalueで値を指定してあげなくても引数に指定するインスタンスから自動で値を読み取ってくれます。
なのでここの、value: @user.name　の記述は不要になります。


The standalone test works – great! That should cover the vast majority of cases.

But I cannot get the `@WebAppConfiguration` case to work if one of the controller’s dependencies has a dependency of its own.


ここもですね。
form_forの便利さに感謝しながら消しましょう。


Finally :D


Could you please clarify a little bit? I have assertions in catch block. Maybe you could give some example how could I use ExpectedException and preserve existing assertions?


Oh yes.


Hi @Q-rec also dein Code geht aber wenig in die Richtung von dem was du vor hast. Ich habe mal 2 Beispiele geschrieben dir dir weiterhelfen sollten. Siehe [Bsp. 1](https://github.com/FH-Potsdam/steel-ant-input-output/blob/master/examples/lines-from-the-middle/sketch-01.js) und [Bsp. 2](https://github.com/FH-Potsdam/steel-ant-input-output/blob/master/examples/lines-from-the-middle/sketch-02.js)


感謝&削除


@MappleCastle Did you flight test the current solution? If you provide us logs and show us where the problem is then we can discuss another solution.


@vladshub  Could you review?


What is the advantage of doing it like that Max? I've seen the glob function before.


This test is on 2.64 after less than 7500 games. I guess this is one of the best tests ever. So,
the main question is: can we afford 24 cores testing? This should be answered with simply yes or no.


Is it just a regex type search function?


user情報操作関連のルーティングはすでにdeviseのdevise_for :usersで実装されています。
今回userに対して明示的にルーティングを設定する事で、
結果deviseのものと自身で定義したものが二重となってしまっているかと思います。
無駄になってしまうので、deviseのものを使用し、自身で定義したものは消しましょう。


Well, I guess my quick browse was a bit too quick!


`esri.geometry.contains` -> point geometry only
`esri.geometry.geometryEngine.contains` -> all geometry type


これprototypeのimageと被るので、avaterの方がいいです。


Does this no longer filter out random fields when saving a form?


Fixed in 7ced16acbfa25d05e8aa67095155672c75950dd2


This commit breaks spacemacs on cygwin.
I uncomment the quelpa stuff and it works again.

I can get you a traceback if you want, but it basically complains about /bin/bash not existing.


@wumouse they are auto generated in the stubs and online docs.


@chelmuth: please add to staging


I think there is stil a bug (already exists in old version).

If we clone propal from company X to company Y, if propal is linked on a project and project dedicated to a company, the clone has no project, if project is not dedicated to a company, the clone has same project.

So sometimes, clone has a project, sometime not. It is sure if we change of company, if source project is linked to company X, we should not keep the link for cloned proposal. But why keeping it when project is a project common to everybody ?
Don't you think we should not just set  $clonedObj->fk_project = ''; in all cases ?


I thought a little bit about this. It should more probably be in `auto-dictionary-mode-hook`, so when `auto-dictionary` is enabled, it will not override the dictionary if `ispell-local-dictionary` if set.

I've put this in my TODO list, will PR it soon or within next spell-checking's PR.


Question: Is the pollDelay member always mandatory?


Not sure whether this'd fall under your or smea's jurisdiction, but I've been having issues getting the hbmenu to launch via the Smash demo on a New 10.2.0 EUR system. I've successfully compiled a pcap with the PAYLOADURL option pointing to my own, personally hosted copy of the latest relevant otherapp payload at http://threela.ws/otherapp.bin and successfully broadcast it - the payload seems to trigger, "ropbin is ready" messages etc appear on screen, but the hbmenu hangs at the "hello" boot screen. Some attempts get further than others, but none progress to the actual menu. Wondering if you might have any insight.


Add `LEVEL3_BOUNCEBALL_REGEN_BOOSTER` instead.


Mispeled canola oil


Add `LEVEL3_BOUNCEBALL_REGEN_CREEP` instead.


Looks good to me! I am not very experienced with composer yet, but I think that `version` should better be omitted. The repository from [packagist](https://packagist.org/) (or your local composer executable, if you point it to a git repository) usually infers the version from a tag from your git repository. This way you don't need to up-date `version` every time you make a new release, which you might forget. Also see [the docs](https://getcomposer.org/doc/04-schema.md#version).


テストファイルがまだ生成されてしまっていますね。
application.rbに不要なテストファイルを生成しない記述ってもうしましたか？


I will test this solution within the next few days and provide logs to you.Thanks for replying.


Poniedziałek rano, 153


これもですね。helperファイルは不要になるので生成されない記述を施してあげましょう。


Changing the key (index of `options.sources`) to refer a target layer in according to a select UI.


What do you mean by saying "does not test time of timeout"? There is timeout = 1500 attribute in test annotation.


ん？　このコミットはなんだ？


Init SelectLayer UI.


Do we want external modules to inherit from out class or do we want a facade-style interface?


コミットは細かく...　と言いましたが、さすがに１行改行しただけでコミットする必要はないです　笑


great!


Misleading variable name. Maybe `wantsBoosterService` is better. Read the comment on the pull request though, I'd prefer not to use the red healing mist for barbs.


What is 4th party compared to 3rd party?


Not, this is not OK: the machine_load() function really shouldn't fail on parse errors. We should log, but continue. The serialization format might change between releases, and we shouldn't completely die in that case...

Failing the deserialization due to OOM is fine, we are fucked then anyway, but parse errors are really something to just warn and continue as wel as possible...


Because aliasedType is of type TypeResult, not Option.


It will not brake harder because its a nested loop controller. Zero velocity is already braking completely with maximum power. If you need the system to be more aggressive you would have to tune the velocity loop more aggressively, not change the outer loop.


Hi
The docker_url in blueprint files is not working (yet ?).
==> 404 Not Found as a reponse.


Indentation


why underscored?


Why is this message being logged at ``INFO``?  It seems like it should be either ``WARN`` or ``ERROR`` since it is communicating an invalid system state.


MediaTests is failing here, specifically testing for the caption part.


complete->unpaid is missing


TeamCity pull requests :: netty [Build 551](https://garage.netty.io/teamcity/viewLog.html?buildId=2897&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5643, ignored: 21 Build time: 00:42:48


Fixed in 33a42409dbfd80f9ff969846bbd425614b414d57


- `Function.prototype.apply` is 2x slower than a normal function call.
- Most functions have 0-3 arguments.

So this `switch` does avoid to call `Function.prototype.apply`.
This optimization is well-known, e.g. `EventEmitter` is using it [here](https://github.com/nodejs/node/blob/master/lib/events.js#L162-L183)


since when are we styling by id?


Can you be more specific than Exception?


I would suggest you do this cast inside of your initialize method. It's quite a common convention and will ease adding other methods in future.

```ruby
  def initialize(stock:, quantity:, market_price:, cost:)
    @stock = stock
    @total_quantity = quantity.to_i
    @market_price = market_price.to_i
    @cost = cost
  end
```


As above I don't quite understand. Please clarify.
Added minimum inactivation time check in 33a42409dbfd80f9ff969846bbd425614b414d57


@chexee Did you intend for these cases to drop-through?


To keep consistency with `update_state`. I underscored it in `update_state` because the non-underscored kwargs refer to actions that happened and `skip_moderation` is not one.


Unless we're sure the exception means only this, we should at least log it and preferably communicate it to the client in case there really was a different error that needs fixing.


Looking good.


Why are all checked and runtime exceptions being caught and not rethrown?  Not only could this broad catch could lead to the system become unstable, but it would catch and swallow the ``ConfigurationException`` being thrown in this block.  It seems appropriate to refactor to a multi-catch for the expected checked exceptions.  Also, should those exceptions be re-thrown?  If not, the log message be a ``WARN`` not ``DEBUG``.


I ended up using `ipc` because apparently `executeJavascript` is undefined on `WebContents`'s prototype. I think both methods are asynchronous, though, because, as far as I understand, [there's no way to synchronously control renderer processes from the main process](http://electron.atom.io/docs/v0.34.0/api/web-contents/):

> There is no way to send synchronous messages from the main process to a renderer process, because it would be very easy to cause dead locks.

I am not sure this is the case for `::executeJavascript` as well, but [I would expect so](https://github.com/atom/electron/blob/master/atom/browser/api/atom_api_web_contents.cc#L699-L702).

I can think of two distinct ways of making this work in a synchronous manner:

1. Introduce a way of acquiring a (logical) "lock" on the main process, and on `app 'before-quit'` check if a renderer process is the owner of that lock.
2. On window creation, the main process decides which window should save the blob store on exit and passes it an argument accordingly.

I can see drawbacks in both approaches, mainly because we'd introduce some complexity and, honestly, I'd prefer to keep code as simple as possible. As of this commit, although asynchronous, the blob store seems to always be saved on exit. There's probably the chance that, sometimes, this won't happen: however, this will be a non-destructive action and I think it's probably okay if that occurs sometimes.

@maxbrunsfeld: the TL; DR from my side is that we can keep this as it is. What do you think? :thought_balloon: I am going to :ship: this as soon as we've settled on a solution to this issue. :+1:


@maximilien The Travis CI build failed after this commit, since unit test is not changed accordingly. Furthermore, we should use "VmNamePrefix" for backward compatibility, so we'd better roll back this change. @jianqiu will update eCPI code to provide a different way to handle "hostname" for for deployment VM and director.


Facades are an interesting approach. The way I envisioned it is that external modules only have to write a component-layer (wrapper/adapter) that inherits from our (interface)-classes.


Keywords could have been fixed too)


kay i will make a check between the bossscripts and give information on what has to be scripted.
issues will be reported in https://github.com/cmangos/issues then


The value can go away ;) We are using HTML5 and not XHTML :)


If you believe this should be improved, add a TODO here, as few people will find the commit message in the future.


I don't think using these methods here is a good fit. Usually DELETE has no payload..


Regardless payments being enabled?


we 100% sure posters can not be null?


Can somebody provide some info if this is still a valid notice? Is it save to upgrade to Windows10?


^

I don't think anything needs to be done.


same logic as in my quick fix on friday i'd say

            elif not (paid or self.is_paid) and payment_required:
                self.state = RegistrationState.unpaid


@sgaide Ah, I see! Thanks for explaining!


@KozhinM Thanks for this. I was playing around with this too on both IE and FF.
But it seems even with 0.5 it doesn't work on FF at times. Though this always works on IE


for the single-registrant mode it's a different thing though. The following would actually make sense (also for paid/unpaid):

```
PUT /manage/registration/<int:reg_form_id>/registrations/<int:registration_id>/checked-in
DELETE /manage/registration/<int:reg_form_id>/registrations/<int:registration_id>/checked-in
```


please remove comments


The auto configure stuff was cruft from another commit that got included by accident. You're right, will probably review.


This is wrong. as_string should return the delivery time as it appears on the order.


Good point. Fixed in d0fe4fc4b5d4f45f10c6fed133968bd50ad61cfe


do we even need the href attr? I think by having it you end up with an ugly # in the url if it's disabled. and styling-wise it should work without it just fine


stdout_stream.class = ExtendedJournalHandler.circus_journal.JournalStream


I agree with that in principle, but what I wanted to do is to replace the currently existing generator in place without any other code changes.
And this is what current generator does (and what the infrastructure requires).

So what I would suggest is:
- leave as is for now
- in the follow-up PR, change the infrastructure to accept methods taking in as a pointer, and fix it here (this change should be pretty small here)
Does it sound ok?


Well it is to defeated any "It is not performant" remarks. Plus it is trivial. :+1:


There's something weird here but there's still a bug I believe :D


Done :+1:


Like this one, a specially that you are using it in many places.

    public static class SocketTimeoutCauseMatcher extends BaseMatcher<Exception> {

        @Override
        public boolean matches(Object arg0) {
            return ((Exception) arg0).getCause() instanceof SocketTimeoutException;
        }

        @Override
        public void describeTo(Description arg0) {
            // TODO Auto-generated method stub

        }

    }


Ping @reduz. This PR should be mergeable if you agree with the above explanation.


I like you cute Japanese developper for support 6 or 8 cores XD


Line Added


:+1: Sorry, I didn't necessarily mean to change all the existing spec descriptions, but nice anyway! :)


Hm, probably not. This ad-hoc API isn't supposed to end up in CAF anyways. It's just for fast prototyping. :)


You might instead use that filter directly on the `{% set sceneID %}{{scene.identifier|swiftIdentifier|lowerFirstWord}}{% endset %}` instead, that would let you avoid repetition


Yup, I thought I got the recommendation from [1], but upon closer inspection I noticed that you only need it under certain conditions. So i'll remove it.

[1] https://www.mediawiki.org/wiki/Composer/For_extensions


But what with other assertions?
assertTrue(System.currentTimeMillis() - start < 1500);
proxyRule.verify(1, getRequestedFor(urlEqualTo("/test?oauth_token=token")));
mockSpidRule.verify(0, getRequestedFor(urlEqualTo("/test?oauth_token=token")));


Well all our current non-regressing tests use this code...anyhow I have
pushed a new patch with atomic, now under test:

http://tests.stockfishchess.org/tests/view/563712790ebc59227df16dc0

We have an additional lookup of a boolean atomic variable, in this way the
counter is now fully local and so we avoid to read/update/write the counter
in an atomic (and possibly slow) way in the hot path. In the hot path we
just (loosely) read an atomic flag.

On Mon, Nov 2, 2015 at 10:10 AM, Joona Kiiski <notifications@github.com>
wrote:

> Because:
>
> - It will be extremly difficult to find and debug. You'll need to go down
> to assembler to actually see how the compiler has optimized the code...
> - With TT, we are protected by the fact two threads will very rarely access
> the same slot simultaneously. So even if the result is garbage every time
> this happens, it doesn't make a difference.
> - Here we'll ask threads to access same variables concurrently >100 times
> per second. If the result is garbage every time this happens, it completely
> breaks the behaviour.
>
> On Mon, Nov 2, 2015 at 12:18 AM, mstembera <notifications@github.com>
> wrote:
>
> > I think Marco is right. Any failure mode is benign(unless someone can
> > think of a case) so why defeat any optimizations by using atomics?
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/mcostalba/Stockfish/commit/edac860627b6706e1c0722812c72731dc02d463d#commitcomment-14117263
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/edac860627b6706e1c0722812c72731dc02d463d#commitcomment-14121831>
> .
>


I don't think we should default to any specific region. Users can use env `AWS_DEFAULT_REGION` variable to specify a region or via command line.


Guess it would be an idea to use brackets for just one line too. Makes these easier to spot


This is indeed a sensible approach. Does this mean that an (intentional?) side-effect is that we can model only 1 connection between components (in each direction)?


@elia I am struggling to understand this one. There's a bug that results from it, and I cannot really fix it without understanding *why* there's two sets of filters. You can repro the problem if you run `$ bundle exec rake mspec_opal_node PATTERN=spec/rubyspec/core/string/sub_spec.rb` In this case the pattern is loading rubyspecs, but rubyspec filters are not applied, since the suite is set to opal. I ran into this because in CONTRIBUTING.md, I say that you can do `$ bundle exec rake mspec_node PATTERN=spec/rubyspec/core/string/sub_spec.rb`, but this now blows up because it is trying to do `mspec_opal_node` with that pattern first, without applying rubyspec filters. Any reason we cannot have *all* filters to be applied regardless of the suite?


So the problem lies not with compilers which don't allow for the normal C functions.

I suppose not all such functions are also defined in std:: in C++. In that case the bug is insoluble.


Those could be put outside catch block without modifications
2 lis 2015 11:49 AM "j-gurda" <notifications@github.com> napisał(a):

> But what with other assertions?
> assertTrue(System.currentTimeMillis() - start < 1500);
> proxyRule.verify(1,
> getRequestedFor(urlEqualTo("/test?oauth_token=token")));
> mockSpidRule.verify(0,
> getRequestedFor(urlEqualTo("/test?oauth_token=token")));
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/schibsted/spid-client-java/commit/7d9ef4985d52705f440cfb42dc76b46ebd437670#commitcomment-14123743>
> .
>


1: M101 signature is missing
5: M102 unrecognized bullet/signature


Thank you for taking the time to explain.  I shouldn't be so quick in calling things benign.


This should have been a PR to master, as it's a bugfix but don't worry, I will add this code to master on my own :-)


Yes, agreed, this comes from Rails tests, which should not test this in the first place.


Voici mon joli h1


[FIXED] means I changed in code
[CHANGED] means I changed test case

Test case 1 )
[CHANGED]When all capitalized, subject picks Protest Locations instead of Protest.
[FIXED]Then if plain organization is not cleared after being used as Location, it gets added again in EventName.

Test case 2 )
FW: Wildfire in CA (Highway Fire – Corona) – Updated
Din't pick up any Location. Only Corona is tagged as Person.

Test case 3 ) PASSED
Police Activity in Los Angeles, CA
[CHANGED]"CA" is not picked up as Location.

Test case 4 )
Canada (Vancouver) - brackets are not ignored.
Canada Vancouver should be picked up , not just Canada. Need to debug and fix as doesnt fix even after adding parenthesis in unwanted list.

Test case 5 )
Robbery – Attempted of Banking Centre – 1430 Wilshire Blvd Santa Monica, CA – West
Location picked up as "CA". 1430 - date, Wilshire etc - Organisation
Reason why address is picked up as Organization is because Blvd = shortform used.
When I change to full word, the phrase is identified as Address (including number)

Test case 6 )
Life Safety Issue Air Quality – 275 S Valencia Ave Brea CA – West
[CHANGED] Location taken as: Valencia Ave Brea CA
275(Num) S(NN)

Maybe we dont need these specific details part of address so much. How much is air quality/any incident impact going to change across the street?

Test case 7 )
Rockridge Protestors enter branch Video
Rockrigde is Miscellaneous here, not a Location. So no event created.

Test case 8 & 9 ) No incidents created as expected. so PASSED

Test case 10 )
Any concerns with the march in Minnesota?
march tagged as date wrongly.

Test case 11 & 12 )
U.S. Advisory: Tonawanda, NY – Bomb Threat During a Robbery – April 20, 2015
Location parsing mistake - same as below.

Updated DRAFT: Travel Advisory: United States – May Day Protest Activity on May 1, 2015

Date tagged as May Day. [CHANGED]

but another issue with first occurrence date parsing (& similarly location) -
if (currNamedEntity.equals(IncidentMonitorConstants.NLPDateEntityIdentifier)) {
				if (prevNamedEntity.equals(IncidentMonitorConstants.NLPDateEntityIdentifier)
						|| nlpout.getDateMap().size() == 0)
					nlpout.addDatePart(wordpos, currWord);

this caused date to be May Day 1 2015 (prev Entity was date entity in second date) so condition above doesnt work.

Test case 13 )
Location not identified by NLP: only CA
Suspicious Package/Substance – 685 Highway 65 Lincoln, CA – West

Test case 14 ) same location parsing error as 11, 12

Test case 15 )
Baltimore Civil Unrest - no location identified
Baltimore civil unrest - location identified
Civil Unrest at Baltimore - location identified.

Test case 16 )
Demonstration at State Palace on Jl. Medan Merdeka Utara ( Jl. = shortform used is the problem)

Even if I change to full form, current parsing is a problem as Demonstration at State Palace on Jalan Medan Merdeka Utara - 2 locations marked.

Test case 17 )
Docklands, Australia – Protest Outside the Transurban Office – May 27

Docklands not picked up (is tagged Miscellaneous).

Test case 18 ) PASSED

Test case 19 )
Subject parsing is haywire. Same H1N1 cases "will be" problem

Test case 20 )
Delhi is not a location but New Delhi is. Anyways India is picked up, which will naturally be marked as New Delhi, the capital as location.[CHANGED] non issue so changed to New Delhi.

Test case 21 ) PASSED

Test case 22 )
Earthquake M5.6 -- only M5 is taken .6 is considered next sentence. Do we want to resolve this in event name?

Test case 23 ) PASSED

Test case  24 )
Alert: Hong Kong: Flights affected

Event name issue : Pick up "Alert Flights affect. So we want to filter these headers in a subject?

Test case 25 ) PASSED
Only thing here is event name is "Red Alert Announcement Signal issue" - issue is lemmatized form of issued. This may change after event parser changes. Along with test case 24 above.

Test case 26 )
same as Canada (Vancouver) case - Violent Crime in India (Mumbai) ...
Mumbai is not getting added.

Test case 27 )
Delhi CM meet public for feedback on governance - too much details in subject. Should be reduced with event name parser enhancement.

Test case 28 )
Malaysia: Kuala Lumpur: Avoid vicinity of ongoing rally by Rohingya protesters outside Myanmar embassy

-- myanmar embassy is not tagged location = only myanmar is tagged location. But incident is tagged to be in KL correctly

Test case 29 ) PASSED
we expected "Traffic Congestion Expect" but got Traffic Congestion Expected as all caps, NLP tags as Organisation - minor issue so for now changed test sheet [CHANGED]

Test case 30 )
We've hit an exception where Organization after proposition is wrong. Firstly its not an organization, but since the words are capitalized in subject randomly, NLP takes it as org.

Delhi, India – Protests against Hashmipura Massacre on May 22-23
Location was Delhi India Hashmipura Massacre

Test case 31 ) PASSED

Test case 32 )
Issue parsing event name & location was expected to be northwest of Guam -- I changed this to Guam as northwest of Guam on geogle map also returns Guam. (BTW looks amazing, we should visit.) Parsed event name is too long...

Test case 33 )
Murder on Phillip Street -- is this a valid case? We can't figure out the location. NLP says Phillps St/Philip Street is a Person entity. I think we shouldnt create event as we dont know location clearly anyways?

Test case 34 )
Body of email: The Government College Teachers' Manram in Chennai plans to stage a demonstration today, May 14, outside of Directorate of Collegiate Education to demand the dismissal of Collegiate Education Director M Devadoss over late payment of salaries.

NLP tags Collegiate Education Director M Devadoss as an organization.
Furthermore the event name is ...... messed up.

Test case 35 ) PASSED

Test case 36 )
Mumbai LIC Employees Protest Against Layoffs on May 11, 2015
Tagged as Org due to capitalization : Mumbai LIC Employees Protest Against Layoffs

Test case 37 )
Switzerland: Zurich Bomb Threat reported
NLP doesn't know Zurich is a location here.

Switzerland: Zurich bomb threat reported
Now NLP will know that Zurich is a place.

Zurich Bomb Threat reported
Now NLP marks Zurich Bomb Threat as an organization.

Capitalization is messing up stuff for us... any way to make non locations/person/org lower case.

Test case 38 )
Location parsing error
Draft: Athens,Greece – Protestors clash with police in Exarchela district, May 23

in this case we are expecting both Athens Greece and Exarchela (district not marked as Location unless capitalized like Bendemeer Road) Do we want to start adding all Location entities in a list despite their position in sentence?

Test case 39 )
Date parsing error
Rome Italy – Trade Union protest march, May 23
Date: march, May 23 --- we need to address word "march" specially, it appears often in mails.

Test case 40 )
Location parsing error
Same as error parsing multiple locations - Location was Istanbul Turkey Square
Istanbul, Turkey – Rally at Taksim Square tonight, May 18
Test case 41 )
Subject parsing error
MSA Special Analysis: Pentagon Accidentally Ships Live Anthrax, Dozens Exposed
too lengthy subject.. should be fixed with our enhancement on event name

Test case 42 )
Location cant be parsed
Capitol Building Evacuated/ think this is another invalid case -- as there is no clear indication on where this Capitol building is (could be Singapore/US/anywhere)

Test case 43 )
Address problem... need to make the num or alpha before address thing more robust.
FL8-147 Deland, 230 N Woodland Blvd, Deland FL – Smoke in Building – IMPACT – UPDATE #2 FINAL

Test case 44 )
TSSA Calls off strike - event name parsing: TSSA Calls strike (meaning is inverted)

Test case 45 / 46 / 47/ 49 ) PASSED
with minor tweaks to event name -- which i think we will improve with event name enhancement.

Test case 48 )
Location tagging is weird for this: check NLP

FL9-600, Jacksonville Operations Center, 9000 Southside Blvd., Jacksonville, FL – Chlorine smell in building – Impact - 4th Update

Test case 50 )
Location tagging should include Misc and 2nd Location for this:
Japan: Kagoshima prefecture: Defer all travel to Kuchinoerabu-jima island following eruption of Mount Shindake volcano

Test case 51 ) PASSED
Curfew lifted in Baltimore after riots

Test case 52 )
Same multiple locations error: Europe & CIS:Denmark, Norway, Sweden: Reconfirm Scandinavian Airlines flights from 21 May due to threatened pilots' strike

Test case 53 )
Subject parsing error, should be fixed with event name.

Jakarta, urban centres: Anticipate heightened security, travel disruption during 21 May student rallies
Expected: travel disruption student rallies ( but I think we should only parse student rallies as event name)
Actual: urban centres anticipate heighten security travel disruption student rallies


I'll change it to `AutomationException` That's what's thrown when the element is not found in `FindElement` when the element wasn't found. The reason for re-catching it here rather than just letting it propate is that we don't know the name of the element we were trying to find in `FindElement`


How about adding sound effects when an action happen?


Plz write the correct values as mentioned in the image.. This is the correct one

RewardHolder reward8 = new RewardHolder("Jacks Place", "0", "3000", "30", scaledImage8);


Ugly!


I think I can remove it entirely actually. I didn't notice that it's already handled in `FindElement` Which means that this commit can be discarded altogether


Should probably read `$AWS_KMS_ID` ?


Same as a above.


Should password be unique?


New typo - Uuninstall :)


This also removed the whole spec for `File::Stat#zero?`, is it intended?
Is the shared examples in `:file_zero` problematic on solaris as well?

(Removing the `platform_is` is totally fine here, and maybe it should be done in `core/file/zero_spec.rb` too)


Remove comments


Debugger entered--Lisp error: (file-error "Searching for program" "no such file or directory" "/bin/bash")
  call-process("/bin/bash" nil t nil "-c" "c:/cygwin-1.7.31-3/bin/timeout.exe --help")
  apply(call-process "/bin/bash" nil t nil ("-c" "c:/cygwin-1.7.31-3/bin/timeout.exe --help"))
  process-file("/bin/bash" nil t nil "-c" "c:/cygwin-1.7.31-3/bin/timeout.exe --help")
  shell-command-to-string("c:/cygwin-1.7.31-3/bin/timeout.exe --help")
  (string-match-p "^ *-k" (shell-command-to-string (concat prog " --help")))
  (and prog (string-match-p "^ *-k" (shell-command-to-string (concat prog " --help"))))
  (if (and prog (string-match-p "^ *-k" (shell-command-to-string (concat prog " --help")))) (progn prog))
  (when (and prog (string-match-p "^ *-k" (shell-command-to-string (concat prog " --help")))) prog)
  (let ((prog (or (executable-find "timeout") (executable-find "gtimeout")))) (when (and prog (string-match-p "^ *-k" (shell-command-to-string (concat prog " --help")))) prog))
  eval((let ((prog (or (executable-find "timeout") (executable-find "gtimeout")))) (when (and prog (string-match-p "^ *-k" (shell-command-to-string (concat prog " --help")))) prog)))
  byte-code("\303\304N\211\203


My bad. I thought that is a good idea to remove duplicate lines because we have global @package, @category, @subproject declarations. But it seems that doesn't work like I expected if in files there are no namespaces. I will fix it. Thank you for the note.


Remove app/assets/stylesheets/destinations.css.scss~


For what reasons, these have been used.?


а если вызвать просто print, без блока, что произойдёт?


ack, nice it works again


хм, интересно, в чем разница?


Typo `presently` ?


1) давай всё же sorted_by будет выполнять одну только вещь — возвращать отсортированный список фильмов (думаем об этом как и раньше: чтобы метод был «теоретически полезным», а не просто выполнял задание).
2) а если вызвать его без блока, что произойдёт?


Could you implement this Video Completion Algorithm?
It would be of great help.
Thank you very much~~


I'd say Sturdy is overrated but I don't participate in the community enough to know what they think of Sturdy.

I do think Sturdy is nearly completely useless on slow walls, though. Being left with 1HP, especially if you're slow, especially if your main use is soaking hits, pretty much leaves you useless.


```go
Already does ...
```go
cli.StringFlag{
   Name:   "kms, k",
   Usage:  "the AWS KMS ID you wish to use to encrypt the file/s",
   EnvVar: "AWS_KMS_ID",
},
```
Maybe my examples should be more clear .. perhaps -k [ID | $AWS_KMS_ID]


But why it is 100 ms margin in one direction and 500 ms in other one?


AWS supported env variable is called `AWS_DEFAULT_REGION`, not sure, maybe `AWS_REGION` works too.


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


good catch, thanks


Done


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


@ryanpepper Yes, exactly. To quote [the Python docs](https://docs.python.org/3.5/library/glob.html): _"The glob module finds all the pathnames matching a specified pattern according to the rules used by the Unix shell."_

The advantage is that you avoid reinventing the wheel and can write a much more compact expression instead of having to hand-code an entire `for` loop yourself. This makes it less difficult for the reader to understand what's going on, too. Remember that testing your code is not the only purpose of your tests - they serve as documentation of your code's behaviour at the same time, so making them short and readable is a good thing. In fact, maybe I should have split my suggestion up into three lines to make it clearer what's going on:
```
filename_regex = str(tmpdir.join('bigbar-Oxs_TimeDriver-Magnetization*.omf'))
omf_files = glob(filename_regex)
assert len(omf_files) >= 1
```
(There is probably a more expressive variable name than `filename_regex`, but I can't think of it right now.) There is certainly a lot of freedom in here, but it's good to try and think about how your tests may be read and interpreted by someone in the future.


This line needs to be updated after the previous change. Github is currently only reporting on the success of the build, not of tests, unfortunately...


The `package.json` has the version. Why do this?


nothing


J'ai mis le code en commentaire pour que tu ne soit pas bloqué.


_Publisher_ is an overused word in the domain of GOV.UK. Can we change this to be more specific please? Something like `GuidePublisher` or `PublisherOfGuide` would work. I'm open to alternatives but a little more specificity would be nice.


@andrew-js-wright looks good!


Добавить TODO когда будет исправлено.


Closes #14


Test remove it !!!


`data-` missing


Вынести в cpp модуль в unnamed namespace?


well that's your choice as dev to use aegis or official desc
i just giving some notes


I think I did that because I added some specs to the opal suite and saw them pass, but were actually filtered as they matched some rubyspec filter. IMO the opal suite shouldn't have filters at all being completely under our control (e.g. I'd cherish commented code in the case of a file that's copied from rubyspec)


hi is this working still or not ?


Почему убран `'\n'`?


It was initially used by pg-monitor, to be able to easily detect the version and apply formatting based on that. Currently it is not being used, but was left there for compatibility reasons. I should have dumped it during the 2.0 release, but it escaped me at the time :)


Shouldn't we set the values from src/main/resources/zookeeper.properties were ip and port are set?


reack


Good work :+1:


Next line ?!


What is this person? The 'beneficiary' of the case? The person actioning the activity?


`m_update_status_duration` в этом коммите нигде больше не присутствует, описать изменение.


Описать сам алгоритм в комментариях.


Adding Logos Syntax highlighting to package list


blah !!!


I don't currently send assigned_user on auth's of existing tags, only on creation where it is set null. Does that matter?


Test comment


Probably.


@mattcui thanks for checking on this fix!

I would not agree with 'unit test is not changed accordingly'. I updated `softlayer/vm/softlayer_creator_test.go` as you can see. Also I verified that `bin/test-unit` works fine, just checked it one more time. Build fails because of another commit which uses `VmNamePrefix`

https://github.com/maximilien/bosh-softlayer-cpi/blame/05d58f5c66e9a0add82d7b5b5523e54848ca6649/softlayer/vm/softlayer_creator_test.go#L156

I believe it was submitted in parallel with my pr.

If 'VmNamePrefix' should remain I'm ok with that, we just need to make sure that its value is passed as `hostname` in Softlayer API call.


Hey, thanks for your reply.
Thats reasonably.
But then, how to change the "level of monitoring per default".
I tried
-O="-tall"
-T all
even
-i all

I believe, for the moment, you cant change the default.
Cheers,
Jaime

On Mon, Nov 2, 2015 at 7:17 AM, Georg Schönberger <notifications@github.com>
wrote:

> Hi Jaime,
> a lot of users had problems with false positives in the System Even Log
> (SEL). As we focus on hardware monitoring with the IPMI plugin we decided
> to keep an eye on RAM and CPU failures, these are the most important SEL
> messages.
> I agree with you that hard coding event types is not the best solution in
> every case, but we want to provide a good level of monitoring per default.
> Therefore we stick to "Memory,Processor" per default, a good standard level
> for every one.
>
> Cheers, Georg
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thomas-krenn/check_ipmi_sensor_v3/commit/867d0dbab8e31bd194d97c9a0f346b842fa20233#commitcomment-14120293>
> .
>


Mogę usuwać komentarze!


I don't think we want to lock ourselves to 2.2.x?


The beneficiary. It's a component of the person perspective of the case, hence the person_id.

Activity assignments to staff members are not in the requirements spec - but would make sense, yes.


test test test


https://robots.thoughtbot.com/rubys-pessimistic-operator :)


Not currently; but it will in future. At the moment we just add anything that's unrecognised to the database; but that's dangerous because we don't currently _require_ secrets in newly added tags.


Dang, wasn't covert enough! :stuck_out_tongue:


Umh - from what I see in the spec, the activities are assigned to offices rather than individual staff (not implemented yet, but on the list). Makes sense too.


Ah, bad memory. Do we want to lock ourselves to < 3.0? ;).


观光团。。


Nicely written, however looping through undecided listeners is not the best way to dispatch event. If the listeners are not added and removed dynamically really fast, there is other way without looping through possibly large amount of listener objects. Imagine structure Map<SkypeEvent,List<HoldListener>> for every SkypeEvent that can occur. When adding listener, loop through keyset, determine whether or not it is suitable for that event and then add it to corresponding arraylists. Dispatching event is then only delivered to objects that are willing to accept it. Not to mention that "listener.getEvent().isInstance(event)" would not be required to be determined during every event dispatch (It would be already determined for every event in add method), like it is now, which consumes too much time unnecessarily. It is of course just an idea for performance improvement, not really big deal and up to you (I cannot get it by myself as this contains too many exotic constructions not supported by pure java).


NB the person_id is not visible (since component link). Haven't adapted the case perspective yet, but will do so.


Added missing semicolon


Btw, I mixed it up with: http://guides.rubygems.org/specification-reference/#required_ruby_version which I had fresh in memory (see their examples).


That makes sense. Thanks for the info.


Thanks this will be in the next commit. I need to stop trying to do this
stuff early in the morning. :)

On Mon, Nov 2, 2015 at 5:59 AM, Adrian Jones <notifications@github.com>
wrote:

> New typo - Uuninstall :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ryancramerdesign/ProcessWire/commit/65f405294f68ea5b6d3d77c3307c4e3c192116c4#commitcomment-14123965>
> .
>


Should we rename it content schemas too?


That's exactly what I meant there.
As for me, after `:empty_command` change everything is just great. :smile:
There is definitely nothing criminal to raise in `pipeline/2` for "no command" case (there are many examples of same thing in the wild).


This array is a list of events where a tag was authenticated and assigned to a user, and that user was alloewed or denied access. eg:

<pre>
[
    {
        "date": 1422222221,
        "location": "door",
        "result": "denied"
    },
    {
        "date": 1422222222,
        "location": "door",
        "result": "allowed"
    }
]

It would actually be trivial to send you scans of unallocated and alien cards too, but I don't right now.

I don't think any of this is a big issue?


**Code Functionality: Critical**

Nothing here caters for trying to log in with a wrong password. To help with this, I would recommend you create an `auth/register` endpoint to avoid unnecessary complexity.


> Btw, I mixed it up with: http://guides.rubygems.org/specification-reference/#required_ruby_version which I had fresh in memory (see their examples).

Aha :)


Не нужен?


It's only used by topographic ops so a name D8AreaOpImage will be better no ?


In your first comment you said "It would let you rid off try-catch in the test" and now "Those could be put outside catch block".
Still don't follow what you would like me to achieve.

Please provide sample code of whole test as you see it. Otherwise we will not move forward.


why _views_ section needs to be added in app.js.
So far it's first time when you have added _view_ sectrion in _client/app/app.js_


Организация корзин плохо подходит для анализа, тут скорее нужно вычислять квантили.


Line is longer than allowed. (99 > 80)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*


Почему `value` участвует в проверке?


rename to D8SlopeOpImage


Because additional CPU load/GC event etc. would not make test running faster but may make test running slower.


По построению затухание (уменьшение `count` в корзине) не предусмотрено, т.е. `count` будет всегда возрастать при мерже.


Il faut que je teste cette nouvelle vue pour voir ce que ca donne pour la taille ou tu as prévu plus de changements au modele ? La vue co_propriete_parcelle n'est pas encore intégrée dans le script de creation du modele, c'est normal ?


Алло


@jglick
This change was backward-incompatible & caused a regression in one of CloudBees internal plugins. https://issues.jenkins-ci.org/browse/JENKINS-31341


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


`Network Flag` was my thinking. I'll add javadoc to them


HYNetworking


не понимаю


thanks for this pull request... this also fixes embedded script tags get executed twice  +1


@lexmag alright, let's leave it like this then :)


why relations between auction and user is commented out?


good point.  We can probably start using tombstone commits to verify local integrity as well, in this series I have used them only to verify the integrity of a remote repository


id definition commented?


$document - is a global AngularJs var ? Is it js document analogy?


silly me!


This needs comments; it is difficult to follow the intent.


Merged here, right now.


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


I that a place where "ContextMenu directive" comes in play ?


@ajithcj: please stop wasting resources and stop this test.


I understand that we don't need decimals in price but as long as it is a book auction I don't think that anyone will bid 9,223,372,036,854,775,807$ for one book :)


This particular line, MARenderBackend.d3ddev is crashing the programs. Anything else doesn't appear to do it, how ever...


,jj


What does "outline" stand for ?


Is "SECRET" the name of the cookie?


I think it's better to use your own exception if you know when exactly it's thrown.


It's an edge case, it passes sometimes, in the real world I think there is no concurrency problem.


what the difference between _visibility_ and _display_ properties ?


This caching seems a strange thing to do. There are other/standard caching mechanisms between the browser and the source server.


use uppercase for finals


So, wrapping up for 3.3 release? Should translators start doing their job?


analysisDate is available from `AnalysisMetadataHolder`, `reportReader` dependency could have been removed completely from this Step


why do we need browser dependent properties ?


Isn't this line now useless? As it get's set to at leas an empty object on top it will always be there, right?


I'm already working on fixing it somehow so it's only temporary.


надо


кто испортил файлик


is_superuser is not needed in this case.


Change argument name to a more generic one :)


Why are we returning empty auction if there's no such in DB? Shouldn't we return empty optional?
return Optional.ofNullable(auction);


Hmm, so security/privacy is the official reason https://help.github.com/articles/why-do-my-images-have-strange-urls/. That had come into my mind, but I found that a non-reason (and I still can't see it).

Anyway, they offer a way to purge the cache, which I did now :-)


Hi,
Please use the manager blueprints under the /new folder as we're dropping docker based managers.


I did it accidentaly when the IDE refactored the class...


@IIvec @ajithcj  @jhellis3: If there were new SMP efforts I would suggest 3 cores, 7 cores, and not to exceed 17 cores and to cooperate closely in order to save resources.


Please, call commits not 'Update  <date>', call by changes.


Use Optional.empty() instead of returning null.


if there's no bids we'll get nullpointer exception. Use Optional.ofNullable()


Hmm, this change probably breaks the API because the method `flags_from_text_set()` is moved from one module to the other. Could you let the methods here or alternatively provide a wrapper functions in new DNSKEY/CDNSKEY modules to maintain backwards compatibility? Thanks!


before 与 after 伪对象双冒号写法的兼容性，不支持兼容写法


final with uppercase


Will return false if `name` & `version` present, but `url` not present, so the name is a bit misleading.


blockquote 与 q标签的区别


Здесь лучше написать f суженное на x=\eta_i


@alwey: agreed. I'll do my investigation of q these days and share information obtained here.


Shouldn't that 17 be bumped?


Nice catch, yes that would be a consequence. So this means component handshake is reduced to calling required functions on the input object to check if they are available and that any other information will have to be contained in the parameter map on the component or the parameter map on the connection. We will probably have to write out some registration examples to identify if this is always possible.


@arie-stratoscale  builtin modules do not have __file__ attributes. Want to fix or we'll revert?
For example: sys.modules[os.listdir.__module__]

@eliran-stratoscale  @mickey-stratoscale


@salva: many thanks for taking a look and giving feedback!

Yes, if there's no newline character (or more precisely an `$/` or `$INPUT_RECORD_SEPARATOR` character) coming, it could hang. So strictly speaking, we shuold implement a way to read chunks of data from those pipes, and detect the record separator on our own to determine when we just saw a whole line, and process that line later with `$self->execute_line_based_operation`.

I would also like to merge it as it fixes a bug, and I think the risk of "characters come in without a newline" is lower than "somebody writes more than 64k to STDOUT/STDERR".


知乎观光团第二波


@agorrod, I would urge the comment include a description of the race, or at least a pointer to the issue?


Thank you for this!


The whole reconstruction is now stored as plane python objects and serialized to json when saving. To replace that with classes, big changes in `reconstruction.py` are required.

Essentially, every time a reconstruction object is accessed, the code must be modified to use the new reconstruction object.


ack


Hmm probably better not to remove as of now since settign an id has consequences wrt saving/restoring of views and leaving it away will require testing on all supported platforms.


The warning in ContestAdmin...Mixin should not be shown to contest_admins/teachers. They may not know what other contests exist at all.

Unfortunately, even then it may happen that a contestant participates in more than one contest with overlapping exclusiveness. If we allow such a situation, the superadmins would get notified because of error 500 or something similar. We don't want superadmins to handle such cases.

We'd accept a change if this logic is changed for example so that both overlapping contest admins and superadmins are notified, but not too many times, and only if actually there is a participant of many overlapping exclusive contests (or if this participant tries to access the contest).


But it can be not a dollars )


Any specific reasoning for choosing smashcase?


Hmm, it seems that you can move everything to `dnskeybase` module and import it back to `dnskey` module - that should be enough for maintaining backwards compatibility. This is applicable to constants listed at the beginning of the dnskey module, too:

    from dnskeybase import flags_to_text_set
    from dnskeybase import SEP


it's not mine (


Alternative fix (matching the original intention) is to hide this Excl*Inline for non-superusers.


`equals` will make the != `null` check so not really necessary, but fine either way


Which can occurs for instance during concurent renames...
A => B
B => A
Note : I first tried to implement this lock free, but it now seems impossible to me, hence, I choosed to use google implementation. We can fall back in this class to the previous implementation.


Are you not pitting an increase in the growth rate of BestMoveChanges that is at best linear in the thread count against an exponential increase in the decay rate? With this I expect that BestMoveChanges will generally be a lot LOWER than in the single-threaded case (in fact I would expect it to hover very very close to 0 all the time), so combining that with the division by Threads.size() the effect of this would be to drastically reduce time usage.


> On 02 Nov 2015, at 12:54, ab5tract <notifications@github.com> wrote:
>
> Any specific reasoning for choosing smashcase?
>
> —
> Reply to this email directly or view it on GitHub.
>

http://irclog.perlgeek.de/perl6/2015-11-02#i_11467474

Liz


:+1: Isso aí, duas funções independentes, cada uma com sua tarefa, e cabe ao usuário combinar ambas se precisar. Show!


@julienlancelot @simonbrandhof, I wonder if it is correct that `findByFindAnalysis` may return `null`.

When this method is called, we know this is not the first analysis, so returning `null` should not happen. In such case, we should fail rather than silently return `null`

In addition, this means that we do not really implement what we intended, which is always have a snapshot for the 'since previous version' period


**Code Quality: Nitpick**

I think these test names should be changed to reflect changes on the codebase. Also, let's try to be consistent about adding at least a single space between method declarations.


reack


Sure, but that doesn't really explain why it can't be int-bounds :)

Anyway, it is hardly the only smashcase in CORE.. I just wish it would all be consistent by 6.Christmas..


ack


1) Не використовуєш лейбли для текстових інпутів
2) Для лейблів треба використовувати атрибут for
наприклад: "
 <label for="male">Male</label>
  <input type="radio" name="sex" id="male" value="male">
"
3) В мультиселекті накопіпастив всі опшини чекед
<select name="position" size="3">
 <option selected="" value="position1">Первая позиция</option>
 <option selected="" value="position2">Вторая позиция</option>
 <option selected="" value="position3">Третья позиция</option>
</select>

http://www.w3schools.com/tags/att_option_selected.asp

4) Таблиці - не використовуєш


TeamCity YaccConstructor :: YC.Core General Build [Build 503](http://teamcity.codebetter.com/viewLog.html?buildId=201652&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Doesn't this kind of defeat the purpose of the define in base.h? And this probably also doesn't work on POSIX, no?


Hmm? Children are also required for the normal mode.
They're only not required in leaf node, but needed to store the children of e.g. ``/``.


$name is not defined (and not used in the message).


The same issue exists in feed_atom0.3.tpl .


I'd rather we prevent triggering those warnings in the first place, than turning them off...


If you want to remove the error just cherry-pick this commit into your branch.


@omryYotpo getShowBottomline is a great name with a lot of actions ... a better name would be only one action :taxi:


@lrusak: ping


Do you need that still?


Для лейблів потібно писати Фор <label for="male"><input type="radio" name="sex" id="male" value=""


Ugh, no please. This is exactly what the define up there tries to do. But perhaps the snprintf-define should be made conditional.


There is another issue with using `<$fh>` that's going to bit you sooner:

As `<$fh>` reads data and buffers it inside the PerlIO layer, full lines may remain hidden there and still IO::Select will not mark the channel as readable (because it only looks at the socket).


бажано через крапку не називати елементи, особливо коли використовуєш метод ГЕТ
можуть бути проблеми з передачею . в урлі


@omryYotpo isRenderBottomline is a better name than getShowBottomline...


reack


Not me, but it is imported in the main.js


@omryYotpo  Remove comments that are not actual comments..


So `Engine_instructions` then :)

(in DB jargon everything is often called a query even wiping out the DB)


In the main.js I changed the following functions and made the following adaptations:
function rezise pizza:
1. Removed the determineDX function altogether. Values were staying the same. Changes are now based on the values in the sizeSwitcher function.
2. Changed the changePizzaSizes(size) function to iterate through the divs and change only the style.width based on the new value. I also put some variable creation outside the loop, saves memory.
3. Created a function isScroll to generate pizzas only when there is a scroll.
4. Created a function requestTick to check if there is a scroll change, if so the browser can update the pizzaPositions
5.  Therefore also the function updatePositions is slightly altered
6. The eventListener is changed so that we calculate the amount of pizza that can be on the screen and only print the background pizza's necessary, depending on the screensize.


@DU-jdto: fixed


Cadê os testes? ;)


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


Generally, we should remove it. If smth breaks after we remove it, we should fix that. `l10n` service should replace this ember-i18n


In order to work properly in the present of an IOMMU, you need to use the dma API, more precisely that of PCI devices (pci_dma_*).


всі опшини вибрані як selected, потрібно обирати лише один
для 4 версії html потрібно використовувати selected="selected"


Yes with the type it is better, but with the label it is the right thing.


Отступи повинні бути однакові


@Dani723 please visit Gitter to see my ideas regarding services, sent on Friday


Line is longer than allowed. (99 > 80)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*


*This commit introduced an issue unrelated to the diff. This is probably a logically related issue. It occurred in `setup.py` in line 3.*

Line is longer than allowed. (99 > 80)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*


Sorry for pushing this directly to master. I was going to create a pull request, but then I suddenly forgot it unfortunately.


I ment, that this not tetst, that connection isn't timedout before it should be. Sorry for misunderstanding.


@omryYotpo code review points:
- [ ] Rename the function getShowBottomline to something more readable
- [ ] Remove useless comments.
this this and :shipit:


теги повинні закриватись і відкриватись на одному рівні
отступи мають бути однакові


reack


so i'm ok with that.


Ніде не використовуєш тег thead


could we replace this backward jump with a loop?


so, now the patient list is not retrieved anymore every time we open a form?


I have a problem that might be related to this. In my case, it does not get fixed unless I set `ctermbg=250` and `ctermfg` to 243 or lower... I know nothing about vim configuration so not sure how this makes sense. See the screenshots

Before (also with the original `hi VertSplit ctermbg=247 ctermfg=247` and with the proposed patch)
![before](https://cloud.githubusercontent.com/assets/2887858/10881376/7e704cc2-8162-11e5-8fb0-965d5beaf9a7.png)

After (with `hi VertSplit ctermbg=250 ctermfg=243` or lower `ctermfg`)
![after](https://cloud.githubusercontent.com/assets/2887858/10881402/9f9b4104-8162-11e5-901c-2e56218ffb4a.png)

Regards!


komentarz :+1:


Found the issue on Windows. Will do a new PR soon after testing on Mac OS X once more.
The default action of a AWT tray icon is triggered on double-click usually, but it can also be done on pure left-click.


@Nelspike Is it safe to remove or Is it used in your comps/services somewhere?


No, not yet. That's coming soon :)


ok


This commit is just removing a parameter that wasn't being used by the client.


See #2272, I've fixed some strings in the updated inspector UI


知乎观光团第三波


Pourquoi est ce que tu as commenté cette ligne?
raise TypeError("This is not base64/32/16 encoded string")


Your fix should certainly address the most significant reduction in time usage, but I'd still expect the new version to essentially eliminate the impact of BestMoveChanges on time usage for the reason given above (linear vs exponential and division by Threads.size()).


b


> On 02 Nov 2015, at 12:58, ab5tract <notifications@github.com> wrote:
>
> Sure, but that doesn't really explain why it can't be int-bounds :)
>
> Anyway, it is hardly the only smashcase in CORE.. I just wish it would all be consistent by 6.Christmas..

Good point.  Will change in a bit  :-)

Liz


Excelente ideia! :+1:  :+1:  :+1:

Tem que anunciar lá no twitter, o pessoal vai gostar dessa.


Poniedziałek, 213


label for="name"
for ставиться до айді
input type="text" name="name" id="name"  value=""

http://www.w3schools.com/tags/att_label_for.asp


OK, I think I understand why now. VS2015 (at least) seems to be drunk, and recommend we not use `_strdup` in favor of `_strdup`. Yeah, nice work pre-processor, you had *one* job...


check your aggregation's code once.i think modified code should be like this:

if r.status_code == 200:
				return r.json() #modified line
			else:
				r = {}
				r['error'] = 'Something went wrong, Looks like the server is down.'
				return json.dumps(r)
		else:
			r = {}
			r['error'] = 'No Query string has been given to run an aggregation query for'
			return r.dumps(r) #why aren't we using here return json.dumps(r)


@lucasart When I raised the pull request @mcostalba  had asked to continue running at 24 threads...
Also at that time the LLR was 1.43 and it made some sense to continue running it for more confidence.... Now that the LLR is 2.60 and even Marco agrees that it can be stopped... So, I've stopped the test.

Thanks to  jojom, vdv and fastgm for donating their machine time... And to all those who provided inputs and supporting tests...


Спасибо,что глянули. Исправим :)


для цього питання потрібно робити список в списку


All fixed in 8ae99e93bccb97a8d502c86b8ccd2c1571cf2815. Thanks! :+1:


Gikk ikke disse grønt lenger, eller bare glemt uncomment?


там де чекбокси та радіобатони потрібно робити додатковий список, в якому лі це варіанти відповідей


4(!) отступа?!


Question mark here helps if trip is null but not when direction is null?


I think it's okay like this -- if someone sets up docker to work as non-root (insecure...), he can still use this script.


не використовуєш теги  <thead> <tbody>


Он там ни к чему.


TeamCity YaccConstructor :: YC.Core General Build [Build 503](http://teamcity.codebetter.com/viewLog.html?buildId=201652&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: NUnit test failed (9).   at Fake.NUnitSequential.NUnit(FSharpFunc`2 setParams, IEnumerable`1 assemblies)   at FSI_0002.Core.commonConfig@178-12.Invoke(Unit arg10@) in Z:\Builds\work\4138ce7c6332a78c\tools\Build.Tools\Core.fsx:line 1... Build time: 00:17:00
### Failed tests
```
GLLParserSimpleTest+GLL parser tests with simple lexer.Bad left rec: <no details avaliable>

GLLParserSimpleTest+GLL parser tests with simple lexer.Complex right null: <no details avaliable>

GLLParserSimpleTest+GLL parser tests with simple lexer.Cond: <no details avaliable>

GLLParserSimpleTest+GLL parser tests with simple lexer.Eps: <no details avaliable>

GLLParserSimpleTest+GLL parser tests with simple lexer.Eps2: <no details avaliable>

GLLParserSimpleTest+GLL parser tests with simple lexer.List: <no details avaliable>

GLLParserSimpleTest+GLL parser tests with simple lexer.ListEps: <no details avaliable>

GLLParserSimpleTest+GLL parser tests with simple lexer.Longest: <no details avaliable>

GLLParserSimpleTest+GLL parser tests with simple lexer.Simple right null: <no details avaliable>

```


Typos:
* Use capital 'T' at the beginning of the log message.
* 'Transfer' is not written 'Transfert' in English.


Same typos.


Looks like all the law used stuff was meant to be committed in https://github.com/mysociety/alaveteli/commit/1a6e36ee8bf2c1d17ca916fa99bc959eef4ff6a8? I'll review it over both commits, so excuse any comments that later get changed – I'll try to keep track! :)


Since this is being abstracted in its own library, it's probably worth making the split delimiter an option to the function. Some cases may want to split on underscores and spaces for instance.


does run really work without deleting the old container?
I guess `./manage.sh up && ./manage.sh stop && ./manage.sh up` will fail...


заголовки колонок вирівняні не по лівому краю


Great tests!


Why not calculating `total_registrations` within the function?


Typo: start log message with capital 'C'.


Same as a plain string so whether to write "datatype=" should get to the same outcome.  Maybe have a version without the explicit xsd:string as well.


- Missing the `:act` mapping.
- Constants live at the [top of the file](https://github.com/bbatsov/ruby-style-guide#consistent-classes)


Not approved!


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


I might be missing something, but I think the logic in both of the if and the else are the same. So maybe the if statement isn't necessary. Just run through the logic no matter what.


else is not required here (if block returns anyway).


The logic inside a callback should always be about doing something with the result. Like, informing the client off the app through a websocket or some other push technology.

The callback should not be used to put additional business logic since that should be inside the command handler. Remember that the primary user of the eventPublisher interface would be a controller of some sort which is responsible for transforming the user input to a command and then dispatching it. If you add logic to for example dispatchToRabbitMq that would imply that that controller layer is containing business logic, or at least the orchestration of that business logic, which imho is wrong.

Furthermore, the logic put in the callback is not part of the transaction in which the command handler is executed. so that would mean that if you start using a transactional resource in your callback, you could have inconsistent state because of failures of the resource.

Bottom line, a callback is just another means to get to the result of the command execution. It should not be (ab)used to add some business logic and therefore I don't see the point to adding the possibility to have to ways to get to the same result which in my opinion is confusing. you either use callbacks(does not block the calling thread), or futures(blocks the calling thread if you want the results for post-processing).


method implemented actually


I had a hard time seeing the difference between the dim and the bold, maybe try using the underline or standout attributes found here. https://docs.python.org/3/howto/curses.html


@itamarnet Uau, começou a programar no bc agora? Legal :)

Só achei um problema bem pequeno, uma linha em branco a mais em um único teste:

```

$ ./run zzmat.sh
.
--------------------------------------------------------------------------------------------------------------------------------------------------------------
[FAILED #1, line 2] zzmat	fat		5
@@ -1 +1,2 @@
 120
+
--------------------------------------------------------------------------------------------------------------------------------------------------------------
..........................................................

FAIL: 1 of 59 tests failed
```


Everything is one tab indented, is there a reason for making this block two tabs indented.


The deprecation message says `law_used_short` will be deprecated, but the method name is `law_used_with_a`.


Appologies..My typo error..i am using 0.8.6 itslef.

Sent from Yahoo Mail on Android

From:"Gonzalo Peci" <notifications@github.com>
Date:Mon, 2 Nov, 2015 at 4:50
Subject:Re: [packer] Add quickfix to restart provisioner as existing one was not working. (af4d8b9)

You dont have latet version. 0.8.6

—
Reply to this email directly or view it on GitHub.￼


Yeah just remove this method lol. It doesn't do anything. That was my bad. Just wanted to simulate user input being put in.


Typo: start log message with capital 'T'. Correct also below.


@zulmat hi, don't use the companion project
I suggest any other keyboard mapping software like joy2key or SDL2 Gamepad Tool http://www.generalarcade.com/gamepadtool/


[Added `__eq__`](https://github.com/biolab/orange3/pull/817/files#diff-558dd573cc7f5c0af810e126873e2c69R40), thanks.


Useless catch clause (throws the exact same exception that it catches).


LGTM


Use symbol keys here – don't need to convert then


Useless catch clause.


This closes issue #2 and moves to issue #3


Puxa, que pena ter que eliminar o código anterior :(

O lado bom é que já tinha a `zztranspor` pronta pra usar!


@zulmat you don't have to use it, because it's unsupported and I'm going to remove it from github


Yes, but a trip always has a direction id


I have a very minor preference of `law_used_human` for the method name, but don't worry if it becomes a real pain to change, or if you disagree.


Suggest deleting the RDF 1.0 versions from the revised test suite.
Some tools don't look in the list of tests, but look for e.g. rdf:type. And mixtures seem to cause confusion as people see them in the file and expect to pass it (human version of the first case).


Для лейбла бажно всюди писати фор

label for="name"
input id="name" type="text" name="name" value=""  size="35"


Could just set key to default to `:full` I think.


I am not totally sure. We are using both, but I think mostly the above syntax. Is `super` more pythonic? It seems differences are mostly related to multiple inheritance which we are probably wise to avoid anyways :) When using super, I quite often make mistakes when I rename a class as I have to make updates in several places, not just changing the class name.


Remeber the difference between `^` and `~` so you don't end up with a version mismatch again.


Wrong indent.


Again.


I think that this line deserves a comment so no one would remove it by mistake


I was just using print statements for debugging. But you're definitely right, we should probably be removing print statements when we put up a new pr.


pull request 1


Put one with a trapped mob in it on your hotbar... turns my screen brown and dark.
May be something to to with render passes.


@fundon the new [2.2.5 update](https://github.com/vitaly-t/pg-promise/releases/tag/v.2.2.5) took care of this ;)


не є окремим списком потрібно ьуло не закривати попередній список і писати в одній li все

Заголовок h4 та select є елементами різного змісту тому не можуть бути списком


I think this would make more sense in `applicable_law`:

```ruby
def applicable_law
  LAW_USED_READABLE_DATA.fetch(law_used.to_sym)
  rescue KeyError # Can't remember the exact exception it raises
    raise "Unknown law used '#{law_used}'"
end
```


Unfortunately, Swift doesn't allow default parameters in protocol methods so I had to adapt all callers of this methods.


структура цього fieldset -у повинна бути як список списків а не окремі списки
на уроці повторимо детальніше


i also got bitten by this just now, holy crap, that was annoying


Typo: start log message with capital 'T' (see also below).


I think for now its fine for this method to be here, but I think at some point we might want to break this out into another class that is specifically for querying different databases.


не є списком має лише 1 елемент


Very nice spotted :)


```C++
Class A: QObject {
Q_OJBECT
unique_ptr<QObject> child{new QObject{this}};
};
```
seems to be safe:

the child is destroyed before the the instance of A, and the child node is removed from the child list of its parent, *i.e.* the instance of A. The cleaning mechanism of QObject trees is not used here.

I feel it's a good practice to use standard C++ features instead of Qt ones. For example, try to use std::vector instead of QVector, unless there's a strong reason to use QVector.


why?


I think there might be some other code in this test which is there specifically to handle Smartdown flows e.g. https://github.com/alphagov/smart-answers/blob/master/test/regression/smart_answers_regression_test.rb#L99-L103

I reckon it'd be worth having a quick look through the history of this file to see whether there's any other code which can be simplified.


You don't need this one third argument.


This doesn't make a lot of sense. You're setting `req.body.brief.archived` to `true` if it's already `true`?

This would probably be more efficient:

```coffeescript
# Coffeescript input
req.body.brief.archived is false unless req.body.brief.archived is true
```

```javascript
// Javascript output
if (req.body.brief.archived !== true) {
  req.body.brief.archived === false;
}
```


no preference, should be a simple change


“However, because I've pointed a Object pointer at this block of memory, and because I put an Object proto field at the front of Room, I'm able to treat a Room like it's an Object.”——learn C the hard way


В таблицях не використовуєш thead


Yeah ok, sounds very reasonable.


I use Swift 2.0


Also it would make sense check that if (isset($params['database'])) and only then set database. If some upgraded users will be missing it :)


Is this still necessary if we use [position: sticky](https://github.com/freenet/website/pull/16#issuecomment-152979338)?


What is the value of hard-coding the supergroup in here when that is default behavior when using None and when the name of the group is actually configurable via `dfs.permissions.superusergroup` in hdfs-site.xml?


We've been doing this as a nice long if statement in other bits of the framework.

```
if (this.model.get("_globals") && this.model.get("_globals")._menu && this.model.get("_globals")._menu._boxmenu &&
...
```

rather than the try+catch. its not easy to see what the try/catch is for unless you leave a comment.


I think the whole concept of data partials as something separate from partial templates has pretty much gone away, apart from the fact that some partial templates are in the `data_partials` directory. I think that makes this test a bit redundant or at least its intention should change. I'm not suggesting you do any more in this commit or PR, but it would be good to capture somewhere that we need to tidy this aspect of the code up a bit...


If we were really counting operations, this could be done as part of the loop, after a failed parse.  But it would put 100+ lines of code into the loop, and require a pretty serious code juggle.  So...


http://www.w3schools.com/tags/att_label_for.asp

потрібно використовувати атрибут фор для  лейбла


Good question! I think it's because we think of the UI in QT designer as a "class" - `Dialog` that is then instantiated in the code:

```
from .ui.dialog import Ui_Dialog
...
# now load in the UI that was created in the UI designer
self.ui = Ui_Dialog()
self.ui.setupUi(self)
```

Do you think we should change this?


Проверял эта ли строка выдает ошибку по #114  - оказалос именно она.


Rename to 'activated'?


`ImmutableList` seems more appropriate.


this commit is not the latest version of the code. This readme has since been replaced with a more recent version of the code....


isActive?


I also think there should be a nil check here because at this point there is a chance that both conn and cur are nil. The branch database stuff isn't working for me and so my log looks like this:

error to connecting to users database
error creating cursor
error executing query
error executing fetch
Traceback (most recent call last):
  File "main.py", line 19, in <module>
    login_success = screen.login()
  File "/Users/colinsmith/Developer/backEnd/CS419_Group_1_Project/code/loginscreen.py", line 89, in login
    rows = LoginScreen.user_query(self, query, 1)
  File "/Users/colinsmith/Developer/backEnd/CS419_Group_1_Project/code/loginscreen.py", line 136, in user_query
    cur.close()
UnboundLocalError: local variable 'cur' referenced before assignment


потрібно робити список списків а не окремі ul-и


setActive?


for Issue #88


Didn't notice it was a string.

In that case you could just use `JSON.parse()`

`JSON.parse('false') # => false`
`JSON.parse('true') # => true`


I think this makes `RegisterableSmartAnswers` or at least `RegisterableSmartAnswers#smartdown_registrables` redundant. If you don't tidy this up in this PR, it would be good to capture the fact that we should do it somewhere.


QuorumOutputStream should flush the buffer when satisfied.


for issue #88


те саме +
селектед атрибут не використано


Again i think you are looking an older version of the code - just like you say, this was changed to "Shotgun Panel" quite a long time ago!


globals need to be defined in the schema as they are kinda default plugin settings and are not relevant to the content.

this shouldn't be in the ``_globals`` section, nor on by default, it pertains to a menu object and so should either go in the course.json or the contentObject.json depending on whether you're using submenus. when we get to multiple menus it'll cause a problem having it in the ``_global `` namespace


Rename to resetSSHConnections


This should have been `%(display_name)s`. I've added as a suggestion in Verbatim: https://localize.mozilla.org/ru/mdn/LC_MESSAGES/translate.html#filter=suggestions @unghost Could you please apply the change?


<thead>,  <tbody> не використано


However you are... Thanks!


this method doesn't exist in the latest version of the code


Missing upstream pull number


It might as well be useful to reset only a particular ssh connection rather than all of them (could add another Redmine ticket).


missing pull request number


see #1902


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


thanks krzysztof, well done!
i will look into progressbar stuff your ideas make absolutely sense.
progressbar for queue is only needed in cli mode, (though still pretty useless, since nth item in queue does not say anything about how long it will take to render).


typo --> input (single t).


I think I ignore scans from unknown/unlinked cards. Maybe. If you ever plan to change this then file an issue and I'll fix the tests and then the logic.


Yep.


Using a HashTable would be clearer than a List of Lists.


互用性。。。中文有这么说的么。。。我觉得这种地方还是意译更好吧
此外，requirements不应该是要求而非依赖么？
以下是个人翻译，供参考：
> 下面是使自动加载器能够协同工作所必须要遵守的要求


@kemiKondratenko залишилось від старого, не використовую


:thumbsup:


My guess is that the terminal does not have enough colors. http://vim.wikia.com/wiki/256_colors_in_vim E.g. there should be two different types of red, there, too, but seems like all red is the same. So in case of vertsplit, 247 248 250 are all the same color.


必须**符合**如下结构
不然少了谓语


Ей преписваш си услуги :D


`DIRECTORY_SEPARATOR`**不**建议翻译，这在PHP中是一个常量，参考其他语言也并未进行翻译


Why is this needed for the prune?  This pull isn't a place to just stuff things you thought should be re-ordered.  Especially not after I've reviewed most of it.


Probably a copy/paste issue, I take no credit for implementing this feature... :smile:


Why we need lambda?


瓦特


`_` 字符<del>中</del>**在**命名空间中没有特殊意义。


this doesn't exist in the latest version of the code


정규표현식이 들어가야 합니다.


How do I obtain the reference to getNumberFormat() from the DoubleParameter?


148 ~ 150 3行に渡る翻訳が、148 ~ 149行になり1行減ってるようです。


Don't you think this call is redundant? Anyway we get JS rules from the filter lists.
You then choose to apply it or not regarding to the scriptSource property.


>The fully-qualified namespace and class is suffixed with .php
我的理解并不是说要求命名空间和类名后缀为.php，而是说会自动添加上.php后缀