Nah, not urgent at all. If its interesting/helpful, here was where I got to roughly:

```
        # Sum of ballots given
        votes_given_query = self.get_annotation_metric_query_str("votes_given", "SUM", round, False)
        # Sum of panellists across rounds
        votes_possible_query = self.get_annotation_metric_query_str("votes_possible", "SUM", round, False)
        # Straight count of debates completed
        debates = self.get_annotation_metric_query_str("ballot_submission_id", "COUNT", round, False)

        query = "(({given}) * (({debates} * {expected}) / ({possible})))".format(
            given=votes_given_query, possible=votes_possible_query, debates=debates, expected=self.adjs_per_debate)
```


哦哦

原来如此


Yes, it is. I've hit this several times now in the Swift source as I refactor or do cleanups. Unrelated files break because I remove no-longer-needed includes from headers that I have modified.


Shoot I was asleep on the job


[Standard JS](https://github.com/feross/standard) recommends no semicolons and ["eslint:recommended"](http://eslint.org/) doesn't complain about them not being there. Are there any general style guides that are more widely accepted?


@naveenrana1309 Hi, the `pod 'Firebase'` can not find with, can you update it, thanks.


原来如此


Have added a check in 977b15b3025d385e0cded01067c903fcce1c3d08, although it would be interesting to see in what circumstances it could throw a 0.


ideally this will move to the store layer eventually but this seems ok.


Adding `"html.parser"` to every file may seem like a arduous change, but it's literally 15s in atom editor to change, and this is how the upstream maintainers have chosen to update the bs4 libraries. Creating a "utility module" or wrapper script will unnecessarily obfuscate the code and make it incompatible/unlike other current BeautifulSoup code readers will come across.

Please consider the original pull request, as submitted. Perhaps we can add an explanation to the repo README?


Amazing! SF will be unstopable with that!

Congratulations!!!


can you also do this for bravery stat messages?


Непотрібні файли


This project Mainly related to cscart to counter point Intigration


Hello Woirld1


It's simple!


`process.env` в допомогу


You inserted hello world


wtf `}`?!


past link here


You may should add a check, if the player **has** an inventory opened to prevent NPEs.
This would prevent malicious players from sending harmful fake payloads to the server.


If I didn't do this, the transparent activity was in front of the game and would "freeze frame". I had to finish() it before sending the intent for the user to be able to see the animation while working with the dialog.

I actually liked the freeze frame and it maybe even saved power (not sure) because the screen didn't need to be updated, but I didn't want to change the app behavior and confuse people. So, did it this way.


Fine by me.

On Wed, 24 Aug 2016, 02:54 Samuel Clay, <notifications@github.com> wrote:

> To be honest I don't really do a thing. I upgrade Android Studio and it
> does it automatically.
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/samuelclay/NewsBlur/commit/21882a7fc24ed710a0afb2972ace9722a322a659#commitcomment-18753203>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABm0ofb4Wd18A39HtFo9ED10_EUEDUE4ks5qi6RcgaJpZM4JrbHm>
> .
>


CoilHeatingGasTransition (mitchute) - i386-Windows-7-VisualStudio-12: OK (2331 of 2331 tests passed, 0 test warnings)

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


Тут крвще темплейти юзати

```js
`/oauth/access_token?client_id=${conf.client_id}&client_secret=${conf.client_secret}&grant_type=client_credentials`
```


Fast calls will make each JS object slightly larger (and execution slightly faster)


Знову темплейти


# Continous Integration Result: SUCCESSFUL

Congratulations, this patch passed basic tests

Tested-by: NetDEF / OpenSourceRouting.org CI System

__CI System Testrun URL: https://ci1.netdef.org/browse/QUAGGA-GITHUBOSRQUAGGA-91/__

This is a comment from an EXPERIMENTAL automated CI system.
For questions and feedback in regards to this CI system, please feel free to email
Martin Winter - mwinter (at) opensourcerouting.org.


ok


Перенос строки у кінець файлу


@anuragjain67 I installed the recent version of django-import-export, and added a folder like templates/admin/import_export/change_list_export.html, and copied your code above, but still it doesn't work from me, i can't see Export option


2.0   重构不合理地方  数据操作完全异步线程


#4262-Cooling-Towers-are-not-being-sequentially-loaded,-all-towers-are-running (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1759 of 1760 tests passed, 0 test warnings)

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


not sure if my fix was the best, added it to the item issue list instead for someone else to take a look at
https://github.com/babymannen/theforgottenserver-7.4/issues/44


Ти глобальну змінну змінюєш. Так не можна

До того ж. респонзу власитиво приходити частинами , тобто, тре `+=`


This is a test for practise


Чому ти не об’єднаєш тіло `getData` з цією анонімною функцією?


add_sketchup_plugin (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Ніколи не експортуй `module.exports.NAME = NAME`, бо ти так експортиш у глобальну видимість, а ти не можеш гарантувати, що хтось вже не юзає щось з таким самим іменем.

Тому юзай `module.exports = NAME`


те саме


This needs a `@history` annotation.


Навіщо ти його рейквариш? Ти ж не юзаєш його


@RippedThemer Thanks!


added comments to the interface IStudent.java


можеш видалити


plklk


add_sketchup_plugin (kbenne) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg)


;lk


`exports.data = res.data` або `+=`, глянь доки


os_2_0-osw-outputadapters (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Має об’являтися тут -- https://github.com/wack17s/fb-parser/commit/35ba667da6646c4aef75a80d125835c249d062b2#diff-fa952fa00047d8b7c987e18f203621a1R19


場所ww


cc @yiminghe 整理了下文档，不知道有没有理解错的地方。。。


os_2_0-osw-outputadapters (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg)


The main point is that it is only available on only a subset of current save layer possibilities.
If we encrypt at the directory level, it should work for all save layer mechanisms, since in the end they all have to result in a directory that is added as a branch to the aufs stack.

By the way, in shutdownconfig, that "CRYPTO='-E 1'" should be "CRYPTO='-e 1'".


This fails 1 out of 256 times, which seems too much!


can you remember what it was we needed to upgrade tar for? xz support for newer llvm maybe?


It posts things like commits to the github channel, but does it post comments to the github_issues channel?


unable to resolve host address `blog.getify.com` 🤔


ok, 强调下通过 rc-util/lib/xx 使用吧


close #379


fix #375


Yes, I think that's exactly what it was.


close #380


test


I thought that was in January though, not February?


my web page


Whoopsie


CoilHeatingGasTransition (mitchute) - Win64-Windows-7-VisualStudio-12: OK (2331 of 2331 tests passed, 0 test warnings)

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


Peeps to check if he can steal something from here for [is-pwa](https://github.com/hemanth/is-pwa#is-pwa) 🤔


Try to commit


That should be usually in global `.gitignore` not in a project one.


Derp


Each file should contain a new line at the very end.


Bra hede


The `otOperationalDataset` structure represents an Operational Dataset, not a command.  I don't think we should be overloading this structure to support get commands.


Its very useful for newbies like me.Thanks.


You cannot add those Dockerfile changes without braking the original image


A few general comments that apply to all your commits:

* Please fix your git setup, so your commits bear a e-mail. If you don't want to use a real one, you can use any fake one - just generate one that would identify you (i.e. not `localuser@localhost`).
* Please don't write just `Implemented processZlib() method` in commit message, and then slam tons of heavy changes in that commit. Either list them all, or, better yet, do commits in smaller independent chunks.


asd


Again, as with `$terminator`, `$key` is guaranteed to be an integer here.


I heavily suspect that it's a bad idea to append characters to the string one-by-one. For several gigs of bytes, it would be either very slow or it will take tremendous amounts of RAM. Or both.


I think it's better to have separate commands for each function.  For example:

- `otSendActiveGet()`
- `otSendPendingGet()`
- `otSendCommissionerGet()`


Has this been tested?
I am still experiencing the prefix duplication bug in 3.1.0


close #4


Seems that @nanosoldier can't run daily job if there isn't a previous run?


test sfsdf


I have a patch for shutdownconfig, I'll do some testing and then push to 'rationalise'.


I think it's better to have a cli that lists TLVs, rather than having to set each flag.  For example:

`dataset send activeget activetimestamp masterkey`


links on About Us page not working
need to put in links for everything in the nav bar & footer


According to `gzuncompress` docs, it returns false on error. Shouldn't we check for it and throw an exception?


I would call this `binary` rather than `userdata`.  Also, is there a use case for feeding the binary instead?


This contains fix for issue #1


You could append directly into the message, rather than staging on the stack.


Prefer pass-by-reference if `NULL` is not allowed.


Are you aware that you can set your own default delimiters for specific languages using a configuration option rather than having to keep a fork of the plugin?


Are you aware that there is already a configuration option to set whether or not to include a space inside comment delimiters? Changing the delimiter in the plugin source is not necessary to achieve this behavior.


I don't think this works - this does not encode in network byte order.


Also see [Sun Studio 12.3 and BLAKE2b failures](https://github.com/weidai11/cryptopp/issues/247).


+1 for this but I think tha


真的很好


哎呦，不错！


Guten Morgen,
Ich habe mal eine Frage zu dem Script von config.inc.php.
Und zwar bin ich bei one.com angemeldet.
Leider funktioniert der von ihnen mitgebende Code nicht.
Ich Abe raus gefunden das dieser nur funktioniert

> `<?php

$con=mysqli_connect("Hoster", "Benutzer", "Passwort", "users");`

jetzt komme ich leider nicht weiter wenn ich mich regestriert habe.

könne sie mir sagen wo der Fehler liegt. Ich habe nur die Datei config.inc.php geändert.

Lieben Gruß oli


ok?


Added Discord Bot created by Devsome


I thought we could use Game.CosmeticRandom for graphical stuff / stuff that are not synced. It does not really matter if the flames / smokes are not on the same location for all players.


There isn't much physical sense , but I thought it would be an expected response if a Time object was the operand. The only use I can see for this is `(time1 - time2).to_f` which can also be done as `time1.et - time2.et`.


Hmm that's actually broadly what I had in mind, did it work?


You are perfectly right! Will fix shortly


Make sure you have entered your instruments in the application.properties file, something like:

oanda.instruments=EUR_USD,USD_JPY,EUR_JPY,AUD_USD,AUD_SGD,USD_SGD,AU200_AUD,AUD_CHF,BCO_USD,JP225_USD,SG30_SGD,SPX500_USD,UK100_GBP


fgh


This is not an academic document .


Good point


This shouldn't be marked as `javascript` because `npm install` is a shell command, not a JavaScript code.


my first remote test


Busy Travis made it around 0.25s. I've fixed it with strict `<0.3`.


"Indicator", "Marker", "Chunk name", that's all basically the same thing, no? :)


Awesome, thanks.

Would be great to hear how to improve the code

Thanks,
Kuba


```javascript
gg(__dirname).setup(gulp);
```

is a suggested valid setup code.


As it's a JSON, then `JSON` should be used there.


How will this commit affect satfinder, fastscan, service searching, TuneTest.py, etc? Are you going to need to reconfigure the tuner everytime? And what about recording conflict management?


#4262-Cooling-Towers-are-not-being-sequentially-loaded,-all-towers-are-running (rraustad) - i386-Windows-7-VisualStudio-12: OK (2332 of 2332 tests passed, 0 test warnings)

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


existence*, dummy.


`fixes` is not a valid commit message. What exactly are you fixing? There are multiple things including extending the documentation, adding dependencies, changing HTML code and even modifying the configuration. Nothing really seems like a "fix".


Fixes #15


Fix:
https://github.com/hustler79/wou_concept/commit/6923390f37d7b35d223ffa5a6ecb7087a3f21183#diff-04c6e90faac2675aa89e2176d2eec7d8L15


Blank line at the end of the file is missing here.


Fixes #12


i don't know what i am doing here


What is this empty function used for?


@keymane This is customized logic that can break if we use this application for any other module where there is no pkchild and pk woman tables


What exactly is being fixed here?


Create new branch from master


try to change something


@keymane This is customized logic that should not be in opensrp-app


@keymane This is customized logic that would only work for vaccinator app


Update homepage's welcome words


What do you need `useAva` for?


The "press any key to continue" should probably be removed.
You can just put the text there, instead of an image, too


derp


There's no script like that in `package.json`.


What is the reason to have this key uppercased?


11756 yields a different bench though.
Now at least  2Q 4R 3N are needed, or 2Q 3R 4N 1B, or 2Q 2R 4N 2B, or 1Q 4R 4N 1B (of course turn Ns into Bs and/or add minors to obtain all the remaining combinations).

@crunchyNYC pawns are not counted here!


###  What's new in Mobirise v3.05
* Added 30+ new blocks for Mobirise 3 theme
* Added tags filter to gallery blocks
* Added new 2100 perfect icons to 'Icons' extension

<https://mobirise.com/assets3/images/v305_2.jpg">

###  What's new in Mobirise v3.04
* "WOW Slider" extension - The famous image slider with 100+ transitions and skins, is now available as a Mobirise extension. See also http://wowslider.com/
* Minor GUI changes
* Bug fixes

###  What's new in Mobirise v3.03
* Paddings for article blocks
* Background color for article blocks
* Changes in menu and footer apply for all pages automatically
* Fixed "non-loading" bug with old extensions and themes

### What's new in Mobirise v3.0
* Brand-new free default theme - new blocks, colors, fonts, icons
* New image gallery and slider - now with video slides
* New drop-down menu - included by default
* Added animation on scroll
* Fixed Google Maps
* Improved inline editor - works faster
* Massive bug fixes

### What's new in Mobirise v2.14
* 'Icons' extension updated:
* Now it supports several fonts
* Added 'Font Awesome', 'Linecons' fonts
* Added 'Show Bullets' parameter in gallery blocks
* Improvements to overall application stability
* Fixed bug with links in gallery captions
* Other bug fixes

* Updated "PayPal Shopping Cart" extension - To update see "Menu" -> "Extensions & Themes"


@maimoonak yea, true. I'm testing fts search on the vaccinator app. If it works out well, I'll will refactor accordingly


There is another commit in @nbd168 staging tree https://git.lede-project.org/?p=lede/nbd/staging.git;a=commit;h=cda91f352e10ec8677537c5c7a354b2f888a7433
IMO the mentioned commit is incomplete, because freq that is being changed to 1200000 is underclocked, for ipq8065 it should be 1400000, afaik. The support for ipq8065 crew should be implemented elsewhere I suppose.
If the mentioned commit is a workaround, shouldn't max microvoltage also be decreased to 1250000 (as in ipq8064) from 1275000 then?
@jow-


What does `latest` variable mean here? Shouldn't it be `currentSections` for example?


I always put equals validations in multiple lines, but I will put them in one line.

Thanks ^^


Create nightly dumps in a csv file in a predefined location.


What was the reason to not sue the native function here?


sudo_users = {245333470},--Sudo users


This actually won't work. See:
https://jsfiddle.net/xph2chn0/


https://openqa.suse.de/tests/525891#step/first_boot/6 shows a console so what does *not* work?


Right, but I was thinking that we can permit have multiple entries in the same day.

What do you think?


Fixes https://github.com/OctopusDeploy/Issues/issues/2648


That should be https://github.com/w3c/css-houdini-drafts/issues/252


Is this the code the same as:
```javascript
latest[section.id] = section;
```
?


Wouldn't it be better to just calculate list size in a regular way rather than keeping an additional variable here?


>How will this commit affect satfinder, fastscan, service searching, TuneTest.py, etc? re you going to need to reconfigure the tuner everytime?

Currently in order to scan for C/T you need to change tuner type first, later UI will be changed to support scanning for C/T without changing type.

Satfinder, fastscan, etc should not affected since we are configuring tuner as S.

>And what about recording conflict management?

Since there is only one frontend (with multiple delivery systems) confict manament should work. If this one tuner was presenting two frontends (where only one could used at a time), then for sure there would be some problems.

Till now only one issue reported for non multituner devices and was fixed in https://github.com/OpenPLi/enigma2/commit/60880db3879250f6e253058eedfaf4e24ab92d78

Please feel free to suggest improvements. Enigma2 always assumed that tuner will be S or C/T but never S/C/T.


The errors from Google do not appear to be temporary.  I believe they now require certain cookies to be set, but I haven't had any luck figuring out how to get this working again.  I copied the working `https://xxx.drive.google.com/videoplayback?...` URL from Chrome as a `cURL` command and removed all of the cookies one by one to see which was required.  The only one that appears to be required is `DRIVE_STREAM=xxx`.

I'll keep digging...


в main конфиге ключи нельзя хранить - только в локальных и ни в коем случае не в гите


居然和我之前尝试写的方式思路一样 😃


AND IIIIIIIIIIIIIIIIIIIIIIIIIIII WILL ALWAYS LOVE YOUUUUUU


please change `<>` to `!=` or `!==`


lol sorry, it does work


какое-то странное форматирование. Стоило оформлять как return `$this->render('pay',` []);` без размазывания пустого массива по строкам, а если массив полный то что-то как `$this->render('pay',` [
]);`


i am getting few errors while executing this script in geo_train.m file ..can you please let me know how it can be resolved


i'll update it after the first few shows


Isn't it the same as:

```javascript
{ [this.state.listSize]: {}}
```

?


[errors_in geo_train.docx](https://github.com/sachinkariyattin/HWCR/files/434203/errors_in.geo_train.docx)

please find the file for errors


You shouldn't ignore these... At least not the regular Gemfile.


please merge these changes to master


Looks cleaner because self._texture is now always present ^^


+1, we package from github anyway, so the package contains the commit introducing punnycode@2.0.0 and won't break anymore.


Yes, of course. Discontinuing HTTP without an **official announcement** would break tons of different clients and would just be insane. Please consult the osm dev mailing list or Overpass API status page on the OSM Wiki first before jumping to conclusions.


Don't comment it.


批注一下，你看这个文件是乱码，说明你的二进制文件的格式不是UTF-8的，你自己转好


Rename to `RecordConfigurationModuleProvider`.


Undo this empty line.


删除jenkins构建历史记录


Cleanup dead code.


此外，不要添加 .exe 文件。版本库里不需要二进制文件。


Use spaces for indentation.


The interface has another problem, because then the `UfElimination` would need to cast the given object to its own subclass, which is also not nice if it can be afforded.

If we add utilities to `SolverContext`, then how do we decide where to put what? Why does something go to utilities vs. to a formula manager? If the `SolverContext` and the utilities are discoupled then it is clear: algorithms that are implemented on top of the regular API are utils.

I would not consider the utilities part of the core API of JavaSMT and thus not put them into `api`, just like we don't have the testing *utilities* in API.


I guess I'm outdated here, I thought only ES6 and higher allowed omitting semicolons. I think we've also enforced semicolons in the past because minifiers/bundlers tended to have issues with missing semicolons, but that's probably a problem long solved.


This isn't needed, remove it.


Свойства: HasSelectedAccessTemplate, HasSelectedSchedule, HasSelectedPassCardTemplate лучше удалить, чтобы сделать вьюмодель "чище" и воспользоваться DataTrigger'ом, чтобы не плодить "одноразовых" свойств во вьюмодели, которые засоряют код вьюмодели. Создание подобных свойств - workaround для того, чтоб не пользоваться триггерами. Хотя никаких оснований для этого нету. Ниже в комментариях описан подход, который лучше использовать, вместо создания подобных свойств.


```PHP
isset($_POST['destination'])
```


It's cool you're using PropTypes :+1:


```PHP
$result = isset($_POST['data']['result']) ? $_POST['data']['result'] : null;
```


TeamCity Ms Demo :: Build [Build 22](http://localhost:8111/viewLog.html?buildId=28&buildTypeId=MsDemo_Build) is now running


TeamCity Ms Demo :: Build [Build 22](http://localhost:8111/viewLog.html?buildId=28&buildTypeId=MsDemo_Build) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 3 Build time: 00:00:35
### Failed tests
```
com.epam.clientprofolio.dao.ClientProfolioDaoTest.shouldGetClientProfolioById: <no details avaliable>

```


May goto die in a fire


这不是exe，是我的笔记，还没做完


You lost bracket here "}"


问题是，你的版本库里有exe文件。你要我给你指出来嘛？你的笔记是编码不对。两个问题你都有。


Примерный код использования триггеров, вместо "вспомогательных" свойств:
```
<Hyperlink Command="{Binding SelectPassCardTemplateCommand}" Style="{StaticResource HyperlinkStyle}">
							<TextBlock
								TextDecorations="Underline">
								<TextBlock.Style>
									<Style>
										<Style.Triggers>
											<DataTrigger Binding="{Binding SelectedPassCardTemplate, Converter={StaticResource NullToBoolConverter}}" Value="True">
												<Setter Property="TextBlock.Text"  Value="{Binding SelectedPassCardTemplate.Name}" />
											</DataTrigger>
											<DataTrigger Binding="{Binding SelectedPassCardTemplate, Converter={StaticResource NullToBoolConverter}}" Value="False">
												<Setter Property="TextBlock.Text"  Value="Нажмите для выбора" />
											</DataTrigger>
										</Style.Triggers>
									</Style>
								</TextBlock.Style>
							</TextBlock>
</Hyperlink>
```
Конвертер **NullToBoolConverter** можно найти в **Controls.Converters**.


Ничёси заморочился


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


лучше строго


I don't have good answers for what it should be, but it did surprise me. The rest of the context is "real world" (the internal id sounds like a legit thing). Thinking about it though, I don't know what you'd replace it with that wouldn't be equally confusing. It is probably something that just requires more effort to read than I put in, which is fine.


нет проверки на успешность открытия


What's questionable about these?


нет проверки на успешность записи


LGTM


нужно закрыть открытый дескриптор файла:
```PHP
if (! fclose($file)) {
    throw new \LogicException("can't close the file");
}
```


ｄｓＡｖＶ


DBにユニーク制約を付けて同じuser_idとsoukai_idの組み合わせのレコードが重複しないようにしておきましょう
http://qiita.com/wadako111/items/958dded40a840c35c5ec

DBのスキーマ設計についてもある程度勉強しておいたほうが良いです
（正規化とか主キー・外部キー等、基本的な知識と用語は押さえておきましょう）
http://qiita.com/shakkee/items/85e7b29a9fd2acfaf9b5


The first bit of the FLIF header is already reserved for future extensions, so if we need more things in the header, we can set that bit to 1 and do whatever changes we want.

Also, if future header data is needed for whatever reason, we can always define a new chunk for that.

> The FLIF chunk is technically variable length (due to the VLE of width/height), so a tool that simply wants to skip it will have to decode the image width/height to figure out where it ends, adding code complexity to e.g. a tool that's only interested in the EXIF information.

Tools that deal with FLIF chunks will need the code complexity to decode VLE lengths anyway, since chunk lengths are encoded that way. So decoding the header info (VLE encoded width/height) to figure out where the chunks start should not add too much complication.


I'm not sure if adding some random people to the list of translators and expecting that they found themselves and remove from this list is a good approach. I'm pretty sure that on this list there is still lots of people, which never translated anything.


This commit made scss themes work; however, it also broke mixins.


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


It's better to use ArrayList() in this case. Because we don't access this lists from different threads - so we don't need thread-safe collection.


можно отлавливать `PDOException`


hello hello hello hello


Thank you.


#4262-Cooling-Towers-are-not-being-sequentially-loaded,-all-towers-are-running (rraustad) - Win64-Windows-7-VisualStudio-12: OK (2332 of 2332 tests passed, 0 test warnings)

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


Does this line actually servers any purpose?


1. You use `forEach` from `lodash` but `forEach` is available on Array instance.

```js
[1,2,3].forEach(n => console.log(n))
// log 1
// log 2
// log 3
```

2. Kamil is right, there is no need for a loop here.
3. Component state shouldn't be mutated do to how react optimization works. `latest` is a reference to `this.state.sections`, not new array.


и, если нужно, можно выше добавить проверку на необходимость записи, когда `$result === null`


sqlInjection. нужно экранировать


test


Couldn't find a Distutils orphaned packaged in Fedora, maybe something I miss here? What is the purpose of providing that?


Well, then you also can remove the floor() call too...


в контроллере не должно быть формирования SQL для выборки. Подобные условия должны быть сворммированы как отдельные методы в классе ActiveQuery. И никогда не используй числовые или строковы значения в условиях да и впринципе где-либо где побобное значение что-то значит. Вот к примеру, что значит `post_status = 1 `? Если бы ты написал что-то вроде `post_status = PostStatus::ACTIVE` то все бы стало на свои места. Всегда используй константы для обозначения постоянных значений. А сама выборка должны была бы быть наподобие Posts::find()->active()->all().


Components shouldn't be kept in state. You can store only data and map through that data in `render` method.


проверка на успешность выполнения нужна:

```PHP
if ($db->exec(/**/) === false) {
    throw new \RuntimeException('detected db conflict ');
}
```


@Override
    protected List<ReactPackage> getPackages() {
      return Arrays.<ReactPackage>asList(
          new MainReactPackage(),
          new BluetoothIOPackage(this), //<- How pass the activity
                                        // from ReactApplication ?
          new VectorIconsPackage()
      );
    }


почему тут не использовался DataProvider?


recover


I don't think you want to do this here because you will be adding the middleware every time a request to this url is made. I think you want to do this outside of a route as you are with `body-parser` and `cookie-parser`


add_sketchup_plugin (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Minecraft avatar finder


Stone someone


Giphy search


Random-Game status


URL Shortener


This is great. It is totally better not to put all that file system stuff in this file which is primarily concerned with setting up our server and routes


This is a endless loop, when the client got a weapon in that slot.


я бы за `if-else` вынес инициализацию `$output`


Не увидел, где используется свойство **parentViewModel** в методе **Initialize**. Если не используется, то лучше его не передавать в метод.


出席済みの確認はモデル側のメソッドにしておいたほうが良さそうです
（Userにsoukai_idを渡してboolで返ってくる感じのが良いかな？）

```
if current_user.attended?(soukai_id)
```
みたいな感じで表現できると、英文読みで一目で「current_userがsoukai_idに出席しているかどうか」というのが分かるので。
（後は出席確認は頻繁に行われる操作だと思われるので、メソッド化しておく事で変更修正しやすくなるというメリットもあります）

処理の流れを英文のように表現できる形にしておくと、可読性が良くなる事はもちろん、修正にも強くなると思います。


Не увидел, где используется свойство parentViewModel в методе Initialize. Если не используется, то лучше его не передавать в метод.


add_sketchup_plugin (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Please pull again and recompile. Let me know if works or not.


не создавай условия большой вложенности. Если условие получается длинное то его лучше вынести в функцию с читабельным именем(в ней тоже можно условие раздробить на несколько доп функций и т.д.). К примеру `if ($this->canLoadModelAndUpdatePostStatus($model)) {
   .....
}`


Np, thanks for your contribution! :-)


```
\Illuminate\Session\TokenMismatchException::class,
```


Are we ever resolving that Deferred with false? No? Then resolve with `null` (aka `$tmp->resolve();`).


When implementing a lazy cons callback in CSIP, the user does not need to distinguish between `MIPNode` and `MIPSol` per se, because he can set the `fractional` parameter accordingly.

Of course it can be done, but I guess I needed a way to vent my frustration about the unmet assumptions and mismatch between interfaces that are stacked up here.


os_2_0-osw-outputadapters (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Maybe I'm doing my math wrong or misunderstand the function but isn't this method producing a single result across multiple rounds? IE it does `(({given}) * (({debates} * {expected}) / ({possible})))` once per team for all of its results? Whereas the method I went with can work out the ratio on a per-round basis before finding their sum.


если ты уже используешь один подход по расстоновке открывающей и закрывающей скобки то должен его придерживаться везде и не перемешивать подходы. Код должен быть однообразным и если у тебя в каждой функции разный стиль используется то это не код - а прокисшая каша выходит. При работе в команде особо важно придерживаться одного стиля т.к. если один и тот же класс будут редактировать программисты с разными стилями кода то кофликт(я имею ввиду Git конфликт, но до мордобоя тоже может дойти=D ) гарантирован. К тому же разнородный код смотрится ужасно


Sure this shouldn't be thrown into the loop instead? At least, this only happens if you don't `yield` before resolving … and that Error will go unnoticed in most cases => hard to debug then.


These names are really meaningless or rather hard to understand at the first sight. There should be more attention put to naming and their conventions.


os_2_0-osw-outputadapters (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


This seems like a reasonable alternative to using `app.use` but there's a problem. Because this is a route the function will run after all the functions passed to `app.use`. So if we want can't do stuff in this function that we want to happen before the `express.static` middleware serves static files


Shouldn't these be `private`?


greattt


You shouldn't share that private key publicly.


this works great, except that you first need to convert the parts from strings into integers:

```groovy
def versionCode = versionInfo.major.toInteger() * 10000 +
                  versionInfo.minor.toInteger() * 100 +
                  versionInfo.patch.toInteger()
```


+小学僧


ok


не смешивай верблюжью нотацию и нижние подчеркивания. Ты должен использовать или один подход или другой, а при применениии обоих одновременно получается каша.  В большинстве баз данных также можно использовать верблюжью нотацию так что у тебя даже монели не будут выделять из кода и вся система будет написана в единообрахной форме. Единообразие кода в проекте один из важных показателей качества кода и за его нарушение в некоторых командах могут жестоко карать=)


あと、whereとかでクエリにパラメータを直で入れて投げるとSQLインジェクションの脆弱性が生まれるので、プレースホルダーを使ったり、キーワード引数で渡しましょう

http://qiita.com/sutetotanuki/items/5eda6bbb5532dd64529a


Why?


Will satfinder tunes satellite, cable and terrestrial.  So is that not going to be possible?

And what about plugins like ABM that scan consecutively on different tuner types. Once configured is python not going to be able to recognise or tune the T/C capabilities of this tuner?

Also how is cablescan going to work if the identifies itself as DVB-S?


@Mr-DaveDev  I don't think there's any reason to make it a autoconf option.

How it is at the moment it works quite well; if the configuration file option isn't specified then the behaviour defaults to the current behaviour by leaving a NULL as the thread name.

Then where you currently see "Thread 1" in the web interface, you now see "Thread 1 -- Front" (front being the camera name). Or you still see "Thread 1" if no name is set.

If we default the thread name to "Thread1" etc, you'd see "Thread 1 -- Thread1" in the web ui the way the code is now, which seemed odd to me.

We could change the web ui to simply say "Front" if a name is set, and "Thread1" if not. But I wasn't sure if it was important to retain the thread number for people to see in the web ui? If that isn't important this seems like a good option.


@ThomasArdal This is done automatically by running `tools/paket.exe install`


Thanks!


Малоэффективная реализация. Выполняется 3 синхронных запроса к серверу при инициализации объекта. В идеале, лучше обойтись одним (асинхронным), либо 3 асинхронных


Эдсгер Дейкстра вывел замечательное правило:

> модуль (в данном случае функция) должен иметь только одну точку входа и только одну точку выхода

Почитай вот эиу стаью https://habrahabr.ru/post/40841/

Теперь порос, что не так с этим методом?


вот это имя переменной вообще в разрез со всеми стандартами идет


dadsadada


в PHP принято именовать все в верблюжьей нотации.
вопрос, что не так с этим классом?


опять же, между функциями должен быть пробел


Pro Tip: You can remove `$languageList = $languageList->getObjects();` because `DatabaseObjectList`s are Iterators.


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


Лучше использовать именованные аргументы. (см. комментарии выше)


И как ты предлагаешь поднимать CI в дженкинсе если эти файлы здесь?


@benjello Tu avais bien commencé mais il doit manquer un truc quelque part. Je regarde.


Shouldn't there be some punctuation after name?


New line instead of space at end?


Fixes #261, but still needs testing!


@mathiascreutz: Shouldn't there be punctuation at end?


этот контроллер перегружен.  Ты собрал всю логику фронтенда в один класс - так нельязя. Грубо говоря у тебя на каждую модель должен быть свой контроллер. Каждый контроллер должен отвечать за какую-то одну часть предметной области. Кпримеру контроллер постов, контроллер оплаты, контроллер категорий и т.п. Этот контроллер надо дробить на специализированные контроллеры.


@cbenz: je n'arrive pas à traiter avec les converteurs SEUIL et TAUX !


Is there some typo in line 73?


Maybe a loooong lime ago, should be safe to remove, but in another commit maybe?


Не понятно, почему мы выполняем проверку на null, вместо передачи null'а в качестве Guid'а в конструктор DepartmentSelectionViewModel().
Передача **null** в конструктор ничего не аффектит (если я не ошибаюсь), зато не нужно будет выполнять подобную проверку.


No idea :D


I also suggest adding a explanation comment to every Obsoletes you create, so in 50 years, we still know why that's there.


It is import!


OK fixed in subsequent commit


Use full descriptive names for functions


Please merge the latest master otherwise you import bootstrap-select 1.11 that is buggy and causing an issue with sale register and similar cases.


Looks fine.


Лучше проверку **departmentSelectionViewModel.SelectedDepartment != null** вынести на уровень выше в строку 551. Тогда код будет выглядеть:
`if (DialogService.ShowModalWindow(departmentSelectionViewModel) && departmentSelectionViewModel.SelectedDepartment != null)`
Данный рефакторинг позволит избежать нескольких проверок SelectedDepartment  на null


If `cache="true"` it can not override by `useCache(false)`. When I import seed data again on a database, It report StatusItem entity is immutable. I think it is because EntityAutoServiceRunner loaded entity from cache.


Могут ли выполнятся условия: **Employee == null** или **Employee.Cards == null**.
Если да, то лучше добавить проверки на null.


this needs to be fixed and be public/license I believe


OK I give up: which change caused this change to the test necessary?


UI requires more changes in order to be able to scan in S/C/T.

Currently UI doesn't support scanning other modes without first changing type to C or T.

In order to support other modes, we should change [isCompatible](https://github.com/OpenPLi/enigma2/blob/HEAD/lib/python/Components/NimManager.py#L542) with [canBeCompatibleWith](https://github.com/OpenPLi/enigma2/blob/HEAD/lib/python/Components/NimManager.py#L547) in several places and rewrite lot's of UI parts.

So please consider that commit as work in progress, currently it handles S/C/T in C++ and more work required for UI. OK?

The only device with multituner (S2/S/T2/T/C) is osmini+, so if you have such receiver we can improve all together UI to fully support such modern tuners.


Currently the only idea that comes to my mind is the one I've described earlier:

> I'm using the scrollTop property to calculate the space above the edited cell "inside" the scrollable container. Let me explain this on an example:
> http://jsfiddle.net/js_ziggle/3300bcy8/

There are probably more situations like that, but they shouldn't happen too often. I guess if you have a reasonably sized table, this shouldn't be too much of an issue.


Of course! When having an opposite bishops endgame, only a check for the strong side is necessary. I would be surprised if this doesn't pass.


#14 じゃなくて #17 だった


ok


I think you should set the pointer to `NULL` after deleting it. The `NULL`  can avoids crashes on double deletes.

```
delete aPointer;
delete aPointer; // undefined behaviour.

aPointer = NULL;
delete aPointer; // won't do anything.
```


Give broad & meaningful description on what the function does. This helps anyone reading your code easily understand what you are trying to do and how to use it.


Oh, good catch!
I've implemented something even simpler in #591 (always-fallback-on-error)...


Hi David,

The provided instructions should get you up and running with jQuery. Storing the response into a variable, catching errors and/or updating DOM content is the same without using CORS Proxy. These information should be available in jQuery docs. Last, the [curl](https://curl.haxx.se/) examples can be executed through a terminal (not javascript code).


なんとなくそんな気がしたので一応書いておきますが、@付きの変数はインスタンス変数で、
RailsのController上でのインスタンス変数は、Viewに渡す為に使われます。
Viewに渡す必要の無い変数はローカル変数にしておくのが良いです。（つまりなんでもかんでも@を付けてインスタンス変数にする必要は無いという事です）
http://www.rubylife.jp/rails/template/index6.html
http://qiita.com/kansiho/items/f5ab9b6eeb990e6af327

他のコードをよく見てないのでもしかしたらViewで使ってる変数なのかもしれませんが、一応。


@apmuthu This is just an outdated  fork. In the original they already added SQLyog by now.


would fix it


Fixes https://github.com/symphonyoss/hubot-symphony/issues/10


No. It's private in the sense that you probably shouldn't use them (`_` prefix) but `private` keyword is completely irrelevant here. If we were to document it as `private`, we may as well not document it at all instead since it would be the same thing technically. However, these methods are used in production and the API is frozen. For instance, `Module._extensions === require.extensions`, `Module._load` is used in `ts-node`.


why do you do this iteration instead of just simply
```javascript
    return Object.freeze(Object.create(this._extraLibs));
```


`threadPoolSize` would be more accurate.


extend readMe with features


@sammerry This commit contains some very interesting changes (in particular the market selection in the Market class). Would you be willing to work towards raising a PR against the base repo?

We'd probably have to make some minor changes (e.g., make "/themis" path configurable etc), and add some tests for the new functionality. But it would be awesome to work on this together (Themis is still a very young project and everything is kind of "work in progress", so why not streamline our efforts :) ). Thanks


@DefCon-007 : Umm, I don't know what went wrong - but I can see lots of duplicate posts on searching for posts...


Малоэффективная реализация метода AfterAdd. В методе выполняется большое число синхронных запросов к серверу.


Should work, feel free to implement it.


Hi Benjamin, this commit contains some very interesting changes. If you believe that these changes are of general interest, would you be willing to work towards raising a PR against the base repo?

We'd probably have to make some minor changes (e.g., remove print statements etc), and potentially add some tests for the new functionality (if any). But it would be awesome to work on this together (Themis is still a very young project and everything is kind of "work in progress", so why not streamline our efforts :) ). Cheers


We actually have this?


1.相關圖檔放到android平台resource下
2.server 位址調整
3.活動通知頁面字型顏色調整


Is it possible to correct the following build warnings/errors:

/Users/travis/build/domoticz/domoticz/hardware/GoodweAPI.cpp:171:10: warning: using the result of an assignment as a condition without parentheses [-Wparentheses]
        while(c = str[i++])
              ~~^~~~~~~~~~
/Users/travis/build/domoticz/domoticz/hardware/GoodweAPI.cpp:171:10: note: place parentheses around the assignment to silence this warning
        while(c = str[i++])
                ^
              (           )
/Users/travis/build/domoticz/domoticz/hardware/GoodweAPI.cpp:171:10: note: use '==' to turn this assignment into an equality comparison
        while(c = str[i++])
                ^
                ==
/Users/travis/build/domoticz/domoticz/hardware/GoodweAPI.cpp:447:11: warning: use of unary operator that may be intended as compound assignment (+=)
                ChildID =+ 10;
                        ^~


Добрый день, не могу понять что устанавливать, два архива у Вас, в обоих пакетах везде в install.php версия 2.0.3.
Подскажите пожалуйста, что скачивать и устанавливать? Или самому мержить надо обновления?


Omg, just noticed these global GT_Bot* callbacks... What an abomination.


JQuery defines its AMD module as jquery, i.e. with this code (version 1.12)

if ( typeof define === "function" && define.amd ) {
	define( "jquery", [], function() {
		return jQuery;
	} );
}

Therefore dotdotdot should use the default module name.


It would be better to initialise the `ExecutorService` externally and inject it into `MailService` - this way this class would be more testable and flexible, as we gain better control of what kind of `ExecutorService` is used. Additionally, I think we don't need the entire `TdpRecruitmentApplicationConfiguration` here - `TdpRecruitmentEmailConfiguration` would be enough.


Yes. It's a typo.


Interresting idea, but following line makes no sense:

s & e->pawnsOnSquares[Us][WHITE]? e->strongSquares[Us][WHITE]++ : e->strongSquares[Us][BLACK]++;

e->pawnsOnSquares is not a bitboard. Its count only the number of pawns on this color (and even if this was a bitboard the operator order is wrong, because only Bitboard & Square is overloaded as expected).

So following line could be used:

DarkSquares & s ? e->strongSquares[Us][BLACK]++ : e->strongSquares[Us][WHITE]++;

Good luck for your further tests.


This keys should be stored in a different module, perhaps call it `configs.py` (which should be git-ignored). It's not best practice to publish your API keys on public repository since anyone can use them to access your services as you.


@hargup  I have made some changes please have a look.  Particularly the for loop.


Лучше везде **Margin'ы** задавать не меньше 5ти. У нас есть много унаследованного кода, где в кучи мест используются отступы, равные 2-ум (в качестве отступов по-умолчанию), но, как показывает практика (а на данном проекте это стало ясно очень давно), 2 - это слишком мало, для реализации более-менее симпатичных окон. Лично я использую margin=5 в качестве стандартных отступов по-умолчанию.


`user = ` / check your spacing.


В блоках код куда-то уехал на один **Tab** вправо.


Ah thanks, will try it later today
Il 24/ago/2016 10:47 AM, "Stefan Geschwentner" <notifications@github.com>
ha scritto:

> Interresting idea, but following line makes no sense:
>
> s & e->pawnsOnSquares[Us][WHITE]? e->strongSquares[Us][WHITE]++ :
> e->strongSquares[Us][BLACK]++;
>
> e->pawnsOnSquares is not a bitboard. Its count only the number of pawns on
> this color (and even if this was a bitboard the operator order is wrong,
> because only Bitboard & Square is overloaded as expected).
>
> So following line could be used:
>
> DarkSquares & s ? e->strongSquares[Us][BLACK]++ :
> e->strongSquares[Us][WHITE]++;
>
> Good luck for your further tests.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/Stefano80/Stockfish/commit/49366c5d547eec37b527979e706ea51c7edd14e2#commitcomment-18756251>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AIWmz9-mKxuz1GykRJUoi68kBjDYhWsEks5qjAUjgaJpZM4JrxtE>
> .
>


Alright. So are we gonna add stuff like `_debugProcess()` to `Process`?


`sendMessage` is redundant and confusing - it's enough to pass the `message` in the constructor and give `pool.execute` a `MailingTask` instance.


@ezyang fyi, you may want to align the message with the pre-existing usage message here:

```
$ cabal new-build :lol:
cabal: Unrecognised build target syntax for ':lol:'.
Syntax:
- build [package]
- build [package:]component
- build [package:][component:]module
- build [package:][component:]file
where
package is a package name, package dir or .cabal file

Examples:
- build foo -- package name
- build tests -- component name
(name of library, executable, test-suite or benchmark)
- build Data.Foo -- module name
- build Data/Foo.hsc -- file name

An ambigious target can be qualified by package, component
and/or component kind (lib|exe|test|bench)
- build foo:tests -- component qualified by package
- build tests:Data.Foo -- module qualified by component
- build lib:foo -- component qualified by kind

```


The thread pool size should not be a part of SMTP configuration, it can be just another simple configuration property.


Thanks ;)
Will check and merge soon.


What is tested by this test?


お、 `reduce` を使ったのはさすが。実はもっと短くできます。


It seems that these changes caused the ATTRIBUTES from not displaying in MCP when viewing a forum.


"matrix" is not necessary. It's when you want to run tests on different environments. We run just on oraclejdk8.


@GerbilSoft yes, I am aware of this. (You told me before ;)
I actually have an unpublished commit on this branch, but I ended up reimplementing it on feature/rpcli. I guess I'm just going to reset this branch back to master (once I'm done slacking off, that is)


@benjello I added a0f5490237b6ebdb1c69f43fa4e7e5930a8266ec


Да, забыли обновить версию в install.php.
Поправили.
Архив для установки - ulogin.install.zip.


Good, I was going to do this


We should declare jdk version explicitly in case the default changes before we want.


Спасибо за быстрый ответ!


Les tests passent ?


Yeah i know i'm working on refactoring mcp stuff, this commit was unwanted


I see, I updated my reply above.


И так подитожим.

- Система не работоспособна - ее нельязя развернуть и даже при починтке явных ошибок в миграциях система все-равно падает.
- Качество кода плоховатое - код не единообразен, используются разные подходы к оформлению кода, код плохо структурирован.
- Присутствует обильное число логических ошибок
- Архитектура приложения плохо продумана
- во многих местах используется ленивая загрузка, когда можно и нужно было воспользоваться жадной загрузкой. Рекомендация перечитай http://www.yiiframework.com/doc-2.0/guide-db-active-record.html и материал по джоинам, к примеру https://habrahabr.ru/post/44807/ https://habrahabr.ru/post/123636/ https://hsto.org/getpro/habr/comment_images/e60/fde/c23/e60fdec2379da78d91098f3f39e7f67d.jpg

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

Первое что нужно будет сделать это привести в порядок скрипт инициализации и миграции. Потом проыести реструктуризацию контроллеров во фронтенде. По ходу дела нужно делать рефакторинг в классах с которыми будешь работать опираясь на замечания что я тебе писал(прочитай вот это перед следующей работой  с кодом http://webmechanic.com.ua/software/phpstorm-reformat-code/).

**Добавь меня в Collaborators в настройках репозитория, чтобы у меня был необходимый доступ**

Работать будем следующим образом - выделяется задача, ты создаешь под нее ветку и там работаешь, по окончанию работы делаешь Pull request в котором я буду делать ревью.

Оформляй комиты по следующему шаблону:

```
сообщение комита

----------------------------------------
#Номер задачи : назване задачи
https://github.com/Chantix/Publications/issues/номер задачи
```

К примеру:

```
Создал htaccess файлы в папке окружения для dev режима обеспечивающие полный доступ к ресурсам и не ограничивающие вывод ошибок бля более удобного процесса разработки.

----------------------------------------
#1 : Привести в порядок скрипт инициализации
https://github.com/Chantix/Publications/issues/1
```

Делай комит в конце каждого рабочего дня и оформляй сообщение так как будто ты 10 лет не мог писать и теперь у тебя появилась возможность высказать все, что ты хотел и не хотел сказать=)

Начинать стоит с задачи по миграциям, потом по скрипту инициализации и только потом по разбиению контроллера.

_Теперь вотпрос. Я тебе давал статьи по SOLID и DRY. Основываяь на этих принципах ответь мне, какая большая проблема присутствует в контроллерах админки и что нужнос делать, чтобы ее исправить?_


Hm, on a second check, we're already beyond the "don't-sync-all-effects" PR - https://github.com/OpenRA/OpenRA/pull/11396 - so you're right and this might probably work then.

I'm a bit worried about different object amounts desyncing the game though.


Le pb de validation de la législation passe, mais après j'ai :

```
(openfisca-baremes-ipp) ~/D/o/openfisca-france-baremes-ipp (baremes-ipp) $ python -m openfisca_france.tests.test_basics
ERROR:openfisca_core.formulas:An error occurred while calling formula ppa_fictive@familles<2015-12> in module ppa
ERROR:openfisca_core.formulas:An error occurred while calling formula ppa@familles<2016-02> in module ppa
ERROR:openfisca_core.formulas:An error occurred while calling formula mini@familles<2016> in module mesures
ERROR:openfisca_core.formulas:An error occurred while calling formula psoc@familles<2016> in module mesures
ERROR:openfisca_core.formulas:An error occurred while calling formula revdisp@menages<2016> in module mesures
Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/tests/test_basics.py", line 52, in <module>
    check_run(simulation, period)
  File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/tests/test_basics.py", line 32, in check_run
    assert simulation.calculate('revdisp') is not None, "Can't compute revdisp on period {}".format(period)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 74, in calculate
    return self.compute(column_name, period = period, **parameters).array
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 159, in compute
    result = holder.compute(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
    formula_dated_holder = self.formula.compute(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 555, in compute
    formula_result = self.base_function(simulation, period)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/base_functions.py", line 60, in requested_period_default_value
    return formula.function(simulation, period, *extra_params)
  File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/mesures.py", line 86, in function
    psoc_holder = simulation.compute('psoc', period)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 159, in compute
    result = holder.compute(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
    formula_dated_holder = self.formula.compute(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 555, in compute
    formula_result = self.base_function(simulation, period)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/base_functions.py", line 60, in requested_period_default_value
    return formula.function(simulation, period, *extra_params)
  File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/mesures.py", line 323, in function
    mini = simulation.calculate('mini', period)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 74, in calculate
    return self.compute(column_name, period = period, **parameters).array
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 159, in compute
    result = holder.compute(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
    formula_dated_holder = self.formula.compute(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 555, in compute
    formula_result = self.base_function(simulation, period)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/base_functions.py", line 60, in requested_period_default_value
    return formula.function(simulation, period, *extra_params)
  File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/mesures.py", line 370, in function
    ppa = simulation.calculate_add('ppa', period)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 79, in calculate_add
    return self.compute_add(column_name, period = period, **parameters).array
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 176, in compute_add
    return holder.compute_add(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 164, in compute_add
    dated_holder = self.compute(period = requested_period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
    formula_dated_holder = self.formula.compute(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 269, in compute
    dated_holder = dated_formula['formula'].compute(period = output_period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 555, in compute
    formula_result = self.base_function(simulation, period)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/base_functions.py", line 60, in requested_period_default_value
    return formula.function(simulation, period, *extra_params)
  File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/prestations/minima_sociaux/ppa.py", line 329, in function
    for period2 in [m_1, m_2, m_3]
  File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/prestations/minima_sociaux/ppa.py", line 329, in <genexpr>
    for period2 in [m_1, m_2, m_3]
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 74, in calculate
    return self.compute(column_name, period = period, **parameters).array
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 159, in compute
    result = holder.compute(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
    formula_dated_holder = self.formula.compute(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 553, in compute
    formula_result = self.base_function(simulation, period, *extra_params)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/base_functions.py", line 60, in requested_period_default_value
    return formula.function(simulation, period, *extra_params)
  File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/prestations/minima_sociaux/ppa.py", line 285, in function
    elig = simulation.calculate('ppa_eligibilite', period, extra_params = [reference_period])
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 74, in calculate
    return self.compute(column_name, period = period, **parameters).array
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 159, in compute
    result = holder.compute(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
    formula_dated_holder = self.formula.compute(period = period, **parameters)
  File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 589, in compute
    column.name, entity.key_plural, str(period), str(output_period), array).encode('utf-8')
AssertionError: Function ppa_eligibilite@familles<2015-12>() --> <2015-12>{(Period((u'month', Instant((2016, 1, 1)), 1)),): array([ True], dtype=bool)} doesn't return a numpy array

```


This is not needed. Globals get brought in automatically via the upgrading mechanism.


Please do NOT create plugins an admin user can not deactivate.

Am 23.08.2016 um 13:50 schrieb Olle Härstedt:
>
> You can edit the database.
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/LimeSurvey/LimeSurvey/commit/62be5fca75bebf2440d5e6a4bf82574d1aa51ba8#commitcomment-18741638>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABXuggoEvK0qD-XSbXt22kNwoUQ05OyPks5qit5sgaJpZM4Jqxb0>.
>


should this default to 24


cc @afc163


Will be nice to skip one line here.


All of these should come as optional function pointers akin to thinkFunc, dieFunc, etc in g_ascript.cpp


Should this default to 20?


Please always specify the jdk version (it is oraclejdk7 by default, but we do not want to rely on that in case travis decides it to be oraclejdk8 before we want to do it).


@athityakumar : Made the necessary changes and ran the deploy script. Please have a look at `https://metakgp.github.io/naarad/`


Do you need this?


OK！
あと基本的には `()` 省略しない方向でいきましょ〜


First File


is there any reason you are not using self.instruction_hist here ?


Best practice: throw an exception instead of returning a string message.


Well adding the losetup crypto stuff was rather a waste of time, the losetup-FULL in LxPupSc_16.08.99 does not support -p or -e, so the creation of the encrypted savefile fails. The code worked fine in tahrpup 6.0.5.


Basic cockpit from B1900d, still needs coding.


へい！


Use _docString_ comments instead of the general comment. You can move all these to _docString_.


best commit 2k16


Ok. Je vais regarder cela. Merci !

Le mer. 24 août 2016 10:57, Christophe Benz <notifications@github.com> a
écrit :

> Le pb de validation de la législation passe, mais après j'ai :
>
> (openfisca-baremes-ipp) ~/D/o/openfisca-france-baremes-ipp (baremes-ipp) $ python -m openfisca_france.tests.test_basics
> ERROR:openfisca_core.formulas:An error occurred while calling formula ppa_fictive@familles<2015-12> in module ppa
> ERROR:openfisca_core.formulas:An error occurred while calling formula ppa@familles<2016-02> in module ppa
> ERROR:openfisca_core.formulas:An error occurred while calling formula mini@familles<2016> in module mesures
> ERROR:openfisca_core.formulas:An error occurred while calling formula psoc@familles<2016> in module mesures
> ERROR:openfisca_core.formulas:An error occurred while calling formula revdisp@menages<2016> in module mesures
> Traceback (most recent call last):
>   File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
>     "__main__", fname, loader, pkg_name)
>   File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
>     exec code in run_globals
>   File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/tests/test_basics.py", line 52, in <module>
>     check_run(simulation, period)
>   File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/tests/test_basics.py", line 32, in check_run
>     assert simulation.calculate('revdisp') is not None, "Can't compute revdisp on period {}".format(period)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 74, in calculate
>     return self.compute(column_name, period = period, **parameters).array
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 159, in compute
>     result = holder.compute(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
>     formula_dated_holder = self.formula.compute(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 555, in compute
>     formula_result = self.base_function(simulation, period)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/base_functions.py", line 60, in requested_period_default_value
>     return formula.function(simulation, period, *extra_params)
>   File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/mesures.py", line 86, in function
>     psoc_holder = simulation.compute('psoc', period)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 159, in compute
>     result = holder.compute(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
>     formula_dated_holder = self.formula.compute(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 555, in compute
>     formula_result = self.base_function(simulation, period)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/base_functions.py", line 60, in requested_period_default_value
>     return formula.function(simulation, period, *extra_params)
>   File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/mesures.py", line 323, in function
>     mini = simulation.calculate('mini', period)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 74, in calculate
>     return self.compute(column_name, period = period, **parameters).array
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 159, in compute
>     result = holder.compute(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
>     formula_dated_holder = self.formula.compute(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 555, in compute
>     formula_result = self.base_function(simulation, period)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/base_functions.py", line 60, in requested_period_default_value
>     return formula.function(simulation, period, *extra_params)
>   File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/mesures.py", line 370, in function
>     ppa = simulation.calculate_add('ppa', period)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 79, in calculate_add
>     return self.compute_add(column_name, period = period, **parameters).array
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 176, in compute_add
>     return holder.compute_add(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 164, in compute_add
>     dated_holder = self.compute(period = requested_period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
>     formula_dated_holder = self.formula.compute(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 269, in compute
>     dated_holder = dated_formula['formula'].compute(period = output_period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 555, in compute
>     formula_result = self.base_function(simulation, period)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/base_functions.py", line 60, in requested_period_default_value
>     return formula.function(simulation, period, *extra_params)
>   File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/prestations/minima_sociaux/ppa.py", line 329, in function
>     for period2 in [m_1, m_2, m_3]
>   File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/prestations/minima_sociaux/ppa.py", line 329, in <genexpr>
>     for period2 in [m_1, m_2, m_3]
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 74, in calculate
>     return self.compute(column_name, period = period, **parameters).array
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 159, in compute
>     result = holder.compute(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
>     formula_dated_holder = self.formula.compute(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 553, in compute
>     formula_result = self.base_function(simulation, period, *extra_params)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/base_functions.py", line 60, in requested_period_default_value
>     return formula.function(simulation, period, *extra_params)
>   File "/home/cbenz/Dev/openfisca/openfisca-france-baremes-ipp/openfisca_france/model/prestations/minima_sociaux/ppa.py", line 285, in function
>     elig = simulation.calculate('ppa_eligibilite', period, extra_params = [reference_period])
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 74, in calculate
>     return self.compute(column_name, period = period, **parameters).array
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/simulations.py", line 159, in compute
>     result = holder.compute(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/holders.py", line 137, in compute
>     formula_dated_holder = self.formula.compute(period = period, **parameters)
>   File "/home/cbenz/Dev/openfisca/openfisca-core-baremes-ipp/openfisca_core/formulas.py", line 589, in compute
>     column.name, entity.key_plural, str(period), str(output_period), array).encode('utf-8')
> AssertionError: Function ppa_eligibilite@familles<2015-12>() --> <2015-12>{(Period((u'month', Instant((2016, 1, 1)), 1)),): array([ True], dtype=bool)} doesn't return a numpy array
>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/openfisca/openfisca-core/commit/deafd0bdc5fbe7e9082e9cdf9863dd8843099968#commitcomment-18756367>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABHxd2QJ28zfdSZTNDKNd03ZkGZaekbSks5qjAdggaJpZM4JrwGe>
> .
>
--
Mahdi Ben Jelloul


Wouldn't it make more sense to defer to the POST setting and if that does not exist, to use the global:
something like
``````
$left_margin = isset($_POST["left_margin"]) ? $_POST["left_margin"] : $GLOBALS['cms_1500_left_margin'];
$top_margin  = isset($_POST["top_margin"] ) ? $_POST["top_margin" ] : $GLOBALS['cms_1500_top_margin'];


So many nested `for`s here, is there any other better/cleaner way?


Read on the order of import statements in PEP8


This commit also updates for Swift 2 syntax.


Functionally, this is an entity callback, not a _class_ callback. Please remove the mention of player classes from this function.


Closes #327


Only here, it's okay to not put spaces before and after `=`: in method declaration and calling i.e. `nargs=2, type=str, help="...."`


In the frameworks? good question. @xdevs23


cf https://travis-ci.org/openfisca/openfisca-france/builds/153607607 en cours


You didn't want to use SQLAlchemy?


Sorry, I have confused the issues a bit. But the root cause of the problem is same. However we might allow relative links starting with ./. It's probably time to factor out the URL validation (see #12479) and add this condition to it...


Closes #325


Maybe I open too many issues these days?:D


Maybe: `os.getenv("AMQ_QUEUES", "")`?


This not fixes disconnects.


hello


You sure do, but nope, it's not a bother :)


Thanks


"found" - "founded" means "created by" :P


I don't like the naming here, which makes you assume the function is intended for setting weights for external entities.


I don't see tests for these components.


__apiary.manifest contains reference to non-existent file in repository.__ (Autogenerated message from apiary.io)
+ apiary.apib


show me u warface!


In my case it has also given me error and I have resolved as follows $value = Gump::sanitize($value)


> The interface has another problem, because then the UfElimination would need to cast the given object to its own subclass, which is also not nice if it can be afforded.

That is something I really don't care about.

But in any case, OK, I don't particularly care about this whole issue either, and I just want to be done with it. If you guys like `utils` package, let's have a `utils` package.


Thanks for the tip...i changed the name of the function yesterday and added some coding...but i am having a prob pushing it...ill work on it today and use the pawn.current...

Sent from my iPhone

> On Aug 23, 2016, at 11:23 PM, Mohieddine SAADEH (Dino for short) <notifications@github.com> wrote:
>
> @najidagher:
> you can always use pawn.currentPossibleMoves instead of re-reading it from the buttons which in turn is reading it from pawn. Please format your code. I added a //WIP: Naji until the code is finished and working.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


language is missing


this file should go to public/license


can we get a new nuget package please?

BR,
John


写了 typescript，各种奇巧淫技。


The test broke because instead of pushing all places we're now only pushing one level. This makes the API handler compatible with the webapp view.


There's no advantage in keeping `instruction_hist` in the object's scope, so I rather not.


Support arbitrary-precision numbers #4


-control/LICENSE


Oh yeah I forgot it offers that. Thanks!


I guess same behavior should then also be for assets with `filename`


change node1


I recommend bumping minimum too, to align with TTK https://github.com/translate/translate/blob/master/requirements/recommended.txt#L6


Why was the selector argument removed? As far as I understand this, it breaks nested lists...


#554 should be closed by this, but that went wrong, and on GitHub, EVERYTHING must go right in one go :(


@antydemant what a fix?)


Yes, thank you! But is there a web page where I can look at it, if need be? Or is it the documentation that's on the UV-CDAT web site?


Move to route config


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=14)


Fix


Collaborator:
Corresponding review in collaborator has been deleted.


Cambios para los iconos


`https://github.com/ethanet/bc-19-budget-analysis-application` would just be fine


ng-src


Сделай protected-метод в подклассах, который принимает объект вагона, сравнивает его тип и возвращает  true/false, а здесь просто вызывай этот метод. Тогда ты избавишься от проверко на train_type и вообще от вложенного if/elsif. Код станет короче и понятней.


Complete C programs


gfhfgh


fgh


vm.rule -> vm.rules


Константы задаются внутри класса и  обычно в самом его начале.


Don't publish your keys publicly. You could put all these in another module say `config.py` then git-ignore the file. The stanard way is to have the keys put in an environment variable, see details [here](http://stackoverflow.com/questions/4906977/access-environment-variables-from-python)


What about the dice collisions? we don't want to end up with several
unfinished work.

Regards, *Dino SAADEH*

On Wed, Aug 24, 2016 at 12:11 PM, najidagher <notifications@github.com>
wrote:

> Thanks for the tip...i changed the name of the function yesterday and
> added some coding...but i am having a prob pushing it...ill work on it
> today and use the pawn.current...
>
>
> Sent from my iPhone
>
> > On Aug 23, 2016, at 11:23 PM, Mohieddine SAADEH (Dino for short) <
> notifications@github.com> wrote:
> >
> > @najidagher:
> > you can always use pawn.currentPossibleMoves instead of re-reading it
> from the buttons which in turn is reading it from pawn. Please format your
> code. I added a //WIP: Naji until the code is finished and working.
> >
> > —
> > You are receiving this because you were mentioned.
> > Reply to this email directly, view it on GitHub, or mute the thread.
> >
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <https://github.com/dinosaadeh/Barjis/commit/554db560f6c0539968a6ade7d828a9667b15eee3#commitcomment-18756603>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AOhVV3bbhmezBT_5TyGhZlxVryDobyWIks5qjAqqgaJpZM4JrWND>
> .
>


That's good. Now the code is clear and Easy to understand


Favicon的支持建议考虑使用这个模块 https://www.npmjs.com/package/serve-favicon


А вчерашние комменты по этому методу ты пропустил? Или я их не отправил?.. `select(&block)` и «что будет, если ни блока ни фильтров не передано?»


Awesome! That sounds great. Thanks for the comment on the `marketo_ma` issue. I will reply there.


Хорошо бы тут тщательнее проверить (что оно соответствует прям всем условиям в блоке).


this is third file


Fix


Leaving a comment in case I want to recover this for future uses


? res.data


thanks for the mail! I will try to work on the Python questions this
weekend. Currently quite busy with work...

Best,
Nils

On Tue, Aug 23, 2016 at 9:45 PM, Ramesh Sampath <notifications@github.com>
wrote:

> Looks good @Nils0 <https://github.com/Nils0> . Looking forward to your
> Python submissions. Post in Slack if you have any questions on Python
> section.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/Nils0/dsp/commit/1d07a4f16f287c11944c2f0fad6152125c82275d#commitcomment-18753468>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ATUf2NdUOkjhkd2KjdrQEwkANUYnrR3fks5qi7AxgaJpZM4JrlVM>
> .
>


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=16)


Thank you ;)


Can't wait for this!


А ежли и то и другое одновременно? `filter(new_sci_fi: true, country: 'USA')`?


today when you add a domain to your yunohost config, it is used for all three at once. yunohost only configures stuff for web, mail and xmpp. I think the idea behind all this is to choose, when you add a domain, to configure it only for a subset of these (say one main domain for mail and xmpp, and several web domains for serving your own cloud stuffs, your public blog on another domain, etc).


Хм. Т.е. в первой ветке if у тебя деньги снимутся и фильм покажется, а во второй — что-то совсем другое произойдёт. По-моему это непорядок!


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


А если второй фильтр определить, с другим именем, чо будет?


да, действительно, этот метод абсолютно не имеет смысла.


wrapPicker 可以定义一个返回值吧，用 as 好奇怪


:heart: As-tu dû modifier le template de base?


@akbertram please review


That said, and reading your comment, I think 'im' is badly named (there are numerous IM protocols), and should better be named 'xmpp'. Also 'im' is not really self-speaking and will feel quite obscure for new comers.


This wasn't removed, it was reverted because the original patch didn't account for a proper fallback.


thanks =D


After netServiceBrowserWillSearch called, then nothing happen. I need look into this.


Nice...


@MarcScott Huh? Was this accidental?


因为返回值是一个 Union 类型。


I'm going to add the functionality soon. First I need to finish the frackin' Java project.....................................


Please learn to write proper norwegian with correct grammar and correct translations before you push something into a live project.

This commit is terrible.


If someone is using it in production, sure. That part of the API is less stable, as far as I know. Logically, this is because once people started writing custom extension loaders and CLIs for node they can not change it (even though it should have been private).


regarding "this feels like a breaking change for existing sites" -> existing sites would have their own config already and hence would not fall back to default one, no?


Ideally, I think I agree with what you're getting at, but sadly these APIs are used in production (also by myself for `ts-node`). I don't really want to block anyone from using these typings, so I'd err on the side of inclusion for any semi-documented APIs people are using.


Thanks, you're right


i am happy , because i finished with documentation for today.
this is just first step in one million km journey.


Hi.
That import was placed in the middle, just after `def createFolder`, because `cgruconfig` uses `createFolder` function. What better to do in such cases? Place import in the middle, or there is a better way?


Rather than filtering on the deleted flag, I think it will be more efficient to simply remove the `FormRecord` entity. The `FormRecordSnapshot` entities will retain all the information.


Prominent Header (no me)


Yep - my ctrl key is dodgy so ctrl+n / ctrl+p occasionally adds stray
characters :)

On Wed, Aug 24, 2016 at 10:44 AM, Andrew Scheller <notifications@github.com>
wrote:

> @MarcScott <https://github.com/MarcScott> Huh? Was this accidental?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/raspberrypilearning/burping-jelly-baby/commit/52e3d0917bcbf33da220b72531abf283d3e328d7#commitcomment-18757085>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AETp8vCkYB3Vdxn-VfOmzxxwzD0QlyvOks5qjBJzgaJpZM4Jr027>
> .
>


(add linebreak)


任务十四


Thank you, I just found this too... update coming soon (just testing it now)


If I recall correct, it should jump to +$3133, at least that's what the direct assembler of the C65 built-in monitor claimed. I was making up this statement as a helper: A 16-bit branch jumps to the same address as the 8-bit branch, if the offset is the same, just encoded in 16 bits.

You can test this yourself using the direct assembler of the MEGA65/xemu repository.


Linking to #6


remove line?


Im working in parallel ..but i dnt want a working day passes without committing anything

Sent from my iPhone

> On Aug 24, 2016, at 12:25 PM, Mohieddine SAADEH (Dino for short) <notifications@github.com> wrote:
>
> What about the dice collisions? we don't want to end up with several
> unfinished work.
>
>
> Regards, *Dino SAADEH*
>
> On Wed, Aug 24, 2016 at 12:11 PM, najidagher <notifications@github.com>
> wrote:
>
> > Thanks for the tip...i changed the name of the function yesterday and
> > added some coding...but i am having a prob pushing it...ill work on it
> > today and use the pawn.current...
> >
> >
> > Sent from my iPhone
> >
> > > On Aug 23, 2016, at 11:23 PM, Mohieddine SAADEH (Dino for short) <
> > notifications@github.com> wrote:
> > >
> > > @najidagher:
> > > you can always use pawn.currentPossibleMoves instead of re-reading it
> > from the buttons which in turn is reading it from pawn. Please format your
> > code. I added a //WIP: Naji until the code is finished and working.
> > >
> > > —
> > > You are receiving this because you were mentioned.
> > > Reply to this email directly, view it on GitHub, or mute the thread.
> > >
> >
> > —
> > You are receiving this because you commented.
> > Reply to this email directly, view it on GitHub
> > <https://github.com/dinosaadeh/Barjis/commit/554db560f6c0539968a6ade7d828a9667b15eee3#commitcomment-18756603>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/AOhVV3bbhmezBT_5TyGhZlxVryDobyWIks5qjAqqgaJpZM4JrWND>
> > .
> >
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>


I for example use `_debugProcess(process.pid)` to enable remote-debugging in staging env on-demand. I'm okay with it being private, because I only use it in staging and it is not critical for the functionality of my app. [It is also advised in the docs of node-inspector](https://github.com/node-inspector/node-inspector#windows). But I had to augment the node typings to use it. Which leaves us with a the general question, should the typings care about wether an API is supposed to be "private"? Should it be the user's responsibility what he uses and what he doesn't, and we only show what is available?


Yes, as I was writing the commit message I thought of the same thing, and went and double-checked and confirmed it's all good. And then forgot to change the commit message :)


This will not work, as echo_n is not defined. Use 'printf' instead.


Здорово ) Кстати, есть такая штука *Here documents*, которая позволяет удобно форматировать строку с сохранением всех отступов:

```ruby
puts <<~TEXT
Station:
  accept_train(train_name, train_type)
  depart_train(train_name)
TEXT
```

Здесь `TEXT` являются ограничительными символами (могут быть любыми), всё что между ними воспринимается как строка. Интерполяция тоже работает.


sdfsdfsdf


I actually didn't know that exists. Tho, since the data ends up in the prototype `Object.keys(clone)` will give you `[]` which might be unexpected


dsfdsfs


sdfs


sdfsdf


Collaborator:
Corresponding review in collaborator has been deleted.


xdfdsf


sdfsf


This is fixed in commit #3689612.


You'll want a `var` in front of new variables, otherwise JavaScript makes them global. It's ridiculous, I know. :weary: The pain is real.


@awig I found this to be very useful for compiling single files easily for debugging while developing.


blm mengertii


No, send this to Mr. Harness.


GOD NO.
This is just me having fun. He's been in memes. What can be worse


Too late, saved it. It will end up going to him one way or another.


Dude, I can just change my github.io!


@ojwb With this commit, the build fails with: "error: ‘str’ cannot be used as a function". What could possibly be going wrong here? It built fine at https://github.com/ayshtmr/xapian/blob/str/xapian-letor/api/letor_internal.cc#L129


XD
SSSHHH


The instructions are fixed, I FORGOT A DANG BACKSLASH!


Yeah, I'm scanning only a small area but still get 60000 pokemons/day and I'd like to send notification for about 3000-4000 of them, so 2 keys would be enough for me


Why SHHH?


@Simn rebase your branch on this and the c# issue should be fixed.


Sent.


DUDE, YOU IDIOT!


WHAT ARE YOU DOING!


JK XD


I'll add a msg to him in the github.io
JUST in case


Um, its probably not worth flagging the incompatibility, as the only person I know who upgrades preexisting sites is me. It will be interesting to see if this affects previous big tournaments though.


Home es un container.
NotFound es una página de error, no hay razón para que este al tanto del estado de Redux. Es lo mismo que una página estática (de hecho, ahora mismo es un componente funcional).
Seguidor esta ahi para testear, idealmente en un futuro lo cambiaríamos por otra cosa. Sino lo volamos y reemplazamos por un container ya, pero no me parece algo hiper urgente.

No soy un experto en lo de React-Router, pero ahora mismo no lo afectan los cambios de URL (había un tutorial de React-Router que indicaba hacerlo así). De hecho, si lo sacas de la ruta raiz creería que no te lo toma como válido (no, no hice el experimento).


Don't even joke about that!


spenaranda


Did you do a Force update?


BTW, I downloaded the version before u fixed instructions on my pc. Mr. Harness will love this!


Just change the ' in the html tag to \', the one after Tino/Valentino.


The changes to lein.bat was reverted upstream because it broke lein.bat upgrades, but `lein.cmd` was unchanged. Is the issue there as well?


Can't remember what I put.
And It can't be IE...


The initial plan of how a menu will be designed.


This was what I tried originally instead of using regular expressions. It didn't work what-so-ever.


NOTE: account file has owner csv file instead of account csv file. Refer to next commit


再来报告bug。。。
大乔贴的乐被马谡拿回来以后，在手牌区这张牌依然会有国色字样


Changed to using regex, but at this point I didn't know what I was doing.


EMS (brianlball) - Win64-Windows-7-VisualStudio-12: Tests Failed (2192 of 2456 tests passed, 0 test warnings)

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


Couldn't alignment be off here?


NstDatabase.xml is not "written to" by the emulator.  It's an .xml file that dictates "quirks" for games/ROMs based on CRC32 and SHA1 checksums.  It's commonly used to work around problems with broken/bad ROMs (such as forcing a mapper, limiting PRG/CHR sizes, or fixing mirroring).

As for "how" the file is "maintained": it comes with Nestopia.  See Options -> Database.  Internal means to use the NstDatabase.xml, External means to use another .xml file (of the same syntax) which can complement (and/or override, I believe) whatever's in the Internal.

Extending NstDatabase.xml is more than just C code.  It also involves manipulating and working out the details in the .xsd (XML Schema) files, which are incredibly important.

I believe this merge/pull req has run its course.  If someone wants to actually implement what has been discussed here, which should absolutely be classified as a feature at this point, fork + make a branch + work on it + submit a pull request.  Further discussion, IMO, should end up in an Issues ticket (since it's a feature request).  I suggest ignoring the NstDatabase.xml feature request for the time being -- this is a lot of work -- and instead just focus on adding something similar to what was done in FCEUX: implementing a selection under Machine -> Options -> RAM Initialization that lets you pick between $00 (default), $FF, and random.  This setting would get saved in nestopia.xml (the settings/configuration file for Nestopia).  Look at Options -> Region for an example of "how" to do this (Win32/GUI-wise).


This is usually called `with`, fwiw.


Good catch, it probably should! PR please for further discussion?


Salts for salting passwords (also later used for password generating).


An actually functional regular expression (because it doesn't use look-ahead).


os_2_0-osw-outputadapters (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


@wondie, please check if the issue has been fixed in the latest commit. The template selector filters the templates for the current profile since the user should not be able to load/use templates for a different profile that is not the current profile.


os_2_0-osw-outputadapters (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Preliminary tests for hashing


@wassimlababidi

## style comments:
* please remove unused spaces
* please remove unused code (commented code)
* please fix the alignment (tab and spaces)

## implementation comments:
* we should use comment syntax such as Doxygen or Sphinx so that we can automatically build the doc
* on the createHEADER function:
    * when checking header values (ex: ```if (tlvtype==1)```), don't use numbers but constants (ex:```if (tlvtype==TLV_TYPE_MULTI)```) (like you did with timestamp)
   * lenPresent variable name looks like a boolean, but it is an integer
* on the insertObject function:
   * MTTLV_MAX_SIZE-5 => what is "5" => can you replace with a constant ?
   * \*offset+=8; => replace with constant
   * you should use memcpy instead of "=" assignements

## other comments:
* be more specific in your commits comments
* I really like your GET SET for the Header !

Good luck !


I'll make pull this and merge it.


Attempting to reduce errors by catching keyboard interrupts.


@africker for the change from "t_mob" to "tmo", I think it will be more flexible if it is set to "t_mo".

The current line that identifies the carrier checks for the carrier name both with and without the underscore because I saw examples files with either "t_mob" or "tmob". So setting the carrier to "t_mo" will catch all those possible configurations.
This is the code (line 125) that finds the carrier within each filename:
carrier = next((x for x in carrierList if (x in f.lower()) or (x.replace("_","") in f.lower())), None)


Attempting to test latest commits. Getting an args error in the search_overseer_thread. "takes exactly 6 arguments <7 given>. I copied your files with changes precisely. Any idea where it's attempting to pull in the extra arg?


In order to be consistent, that `.AddrMode` value should be `0x0040000` (which is `AM65_REL_LONG`).


Plans for getting both strings and single numbers.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1185 of 1185 tests passed, 0 test warnings)

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


致A3Coin爱好者，作为A3初期团队，A3作为二代预挖掘币，开发团队已经再推广初期挖掘了30%币，分发了30%币，剩下币保留在数据链中。
如果后继者继续愿意推广A3,我们愿意贡献第一个数据块。
A3设计初衷是为了研究资产合约，数字交易等问题。
如需帮助请联系aspnmy@163.com


Doh, yeah I'll fix thanks :)


This is what I was using at the time to help figure out how my encryption would work.


You there?


Ok so this is where stuff gets interesting at dupes..


It turns out that this bit of code is dead and I added it for no reason.


i added that line on my base file to see if it corrected the issue, but doesn't appear to have fixed it. Still no casting on debilitation nor targeted spells.


single step your way through it in degugger


I planned to encrypt and store the data every time the program was exited.


HTTPS Everywhere would use https here ;)
@Hainish @cooperq


sorry i take that back, looks like its working!


I agree! I particularly enjoy how you label the study of history as fun and the reason for that being that as students of history we have the ability and privilege to analyze the writings of historians that came before us and develop our own perspectives and draw our own conclusions. As Jenkins postulates that historians do not invent history but through their and subsequently our study of a topic we are able to give it meaning. The idea of our work (even as undergraduate students) giving historical happenings a sense of meaning is an attractive idea that serves to valid why studying history is perpetually important.


should this line be removed?

 [WARNING]: While constructing a mapping from /root/openshift-
ansible/roles/openshift_repos/tasks/main.yaml, line 43, column 5, found a duplicate
dict key (src).  Using last defined value only.

TASK [openshift_repos : Configure origin yum repositories RHEL/CentOS] *********
fatal: [10.10.10.10]: FAILED! => {"failed": true, "msg": "the field 'args' has an invalid value, which appears to include a variable that is undefined. The error was: 'item' is undefined\n\nThe error appears to have been in '/root/openshift-ansible/roles/openshift_repos/tasks/main.yaml': line 41, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Configure origin yum repositories RHEL/CentOS\n  ^ here\n"}


This was also the version of the code where I renamed all of the functions to no longer have an underscore in front of them.


reset


What other fields are there where the opportunities continue to balloon? To expand outwards faster than we can learn, and it's all we can do to keep up and keep learning? Few other fields are like that, where the sky's the limit... in fact, the limit's far beyond that.


@ikhoon 깃헙 들어가지 말고 휴가 즐기다 오시오!!


The main attraction of this commit -- better password management and verification.


is this needed?


I remembered that I kinda need to store a username.


Whats the reason for this? Now the image test don't work anymore (but the wheel is probably much smaller?)


@zenithtekla any response to @mleanos's above comment?


Trying to bring repository up to date with praxes.


Very occasionally I would encounter an uncaught keyboard interrupt so I decided to attempt to catch them, just in case.


More dead code.


See 2c8ae29538b4b335f0587a0519dfbb18a040e0eb.


lol - I'll fix that :)


Pranshu, you need to roll back and re-commit.
1. Roll back the last commit.
2. Commit the unchanged theme.
3. Commit the theme with your changes.

Use meaningful commit messages. If you don't know what that means, please read http://chris.beams.io/posts/git-commit/


I'm testing commenting on my README-Edits file.


121784677-OAControllerFixes4633 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2362 of 2365 tests passed, 570 test warnings)

 * 573 tests had: AUD diffs.
 * 8 tests had: EIO diffs.
 * 3 tests had: ESO big diffs.
 * 3 tests had: Table small diffs.
 * 1 test had: ERR diffs.
 * 1 test had: MTR big diffs.
 * 1 test had: Table big diffs.

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


Because the constructor is actually a public API, we probably want to keep the old one and just add a new constructor which takes an additional memoryPool.


This applies to other public classes as well.


I totally agree koitsu, would be much easier to implement this through the source and have a menu option. Makes the most sense. I think messing with the database is going to open a brand new can of C++ goo worms.:P

Looking at rainwarriors implementation seems like it would be way easier.


Eventually, if I continue working on this after the assessment, I may change to comparing hashes instead of doing this method.


Maybe add a comment explaining why we need to do the shuffle?


121784677-OAControllerFixes4633 (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Right - this was what Tom recommended for the other wheels - OSX and Linux.   I see that the current Windows wheels up on Pypi are also small, so I guess they don't have the images either.