Hi Shreekanth,
 yest evening Satish needs Certificate but Struck in Approval stage
only.this API is throwing NumberFormatException while approving and not
able to do any transanction so  discussed with Malathi and changed to old
format as of now..

Regards,
Roopa

On Fri, Jan 29, 2016 at 3:57 PM, Sreekanth S Nair <notifications@github.com>
wrote:

> whats is this ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/egovernments/eGov/commit/abc3ecf6b15205bbf27efe737e7e1564a80a8a6e#commitcomment-15757490>
> .
>


Unnecessary test. Just fail…


not quite sure why ensureHeavy setting type and perform add operations rather name is misleading or it should not be here


Subshell means a `fork` for some shells. `{ [ "$CURRENT_USER_ID" -eq 0 ] || [ -O "$cmdline_file" ]; }`/


I don't see how initializing mValueUnit to null is "the only way of knowing for sure that the device has the TemperatureUnit in Celsius" ? Or are you referring to something else?
I think that it's the perfect commit for changing the default behavior to what was implicit before


Hey Greg,
sorry for nerving you a last time. Either I'm not experienced enough in js or I'm just stupid. (Probably something in between...)

Apparently your workaround seems to be avoided by Firefox & Chrome if I use Soundcloud sources.

In my example here ( http://christianlosert.com/test/greg/ ) I only added a Soundcloud Initiation at the beginning of your script + replaced the array-contemt with some valid Soundcloud URLs.

I'm suspecting that Firefox / Chrome don't like to run multiple createMediaElementSource operations each time handleCanplay() is called. But I'm not able to fix this.

It kind of works with Safari. (Most of the time the stream aborts immediatly after it starts and only works when I'm refreshing the page...)

Maybe you could give me a tipp, how to solve this. I promise, I won't bother you anymore.
Best, Christian


java doc


I don't mind who needs certificate or what, do not alter code unless you know whats is the impact or how to properly fix it, even if someone said so. Kindly keep this in mind for future development.


ㄹㄹ


I think we need to do the copy every time, otherwise we cannot handle the case that getinstdisk is changed.


Is there a reason to do this? It'll prevent you from using default routing later should you need to.


Damn, I can't read, please ignore me.


I agree with those three things.

Another thing that struck me this morning is that we might want to have a
stack of current_intents instead of just one which gets overwritten. At the
moment we can do 'slot filler' intents and 'nuke current_intent' intents,
but I think we would not be able to handle sub-intents. But it's probably
easier to fix this once we have a proper use case for it.

On Thu, Jan 28, 2016 at 6:26 PM, Niall Robinson <notifications@github.com>
wrote:

> Its looking really great - well done. Once this is merged in, what's next
> on the list?
>
>    - Proper refactor - at least some really rigorous commenting
>    - Bulk out the functionality? i.e. other kinds of searches and more
>    utterences?
>    - "Alias that?" - would be super cool to develop the utterences like
>    this once we finally get that echo
>
> We should have a nice system to show by end of Feb
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/met-office-lab/dre/commit/e2a2ec0b8f00027a1e473d15287a21bc574545b9#commitcomment-15741395>
> .
>

--
Please send all officially sensitive info to:
rachel.prudden@metoffice.gov.uk <alberto.arribas@metoffice.gov.uk>
(and let me know to look for it)


Note: none of these checks are seen in Rust ver. Just `|| continue`.


Inverses the appearance of the text - light grey background and grey text
https://www.npmjs.com/package/chalk


Oh right. I should have looked harder - I didn't realise that this was just  a feature request rather than a bug.


found java doc that says it doing that, so previous comment can be ignored


ok

On Fri, Jan 29, 2016 at 4:10 PM, Sreekanth S Nair <notifications@github.com>
wrote:

> I don't mind who needs certificate or what, do not alter code unless you
> know whats is the impact or how to properly fix it, even if someone said
> so. Kindly keep this in mind for future development.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/egovernments/eGov/commit/abc3ecf6b15205bbf27efe737e7e1564a80a8a6e#commitcomment-15757786>
> .
>


why you change it fastly


This, and probably another of changes to padding, cause the wiki page to be narrower -- this is causing significant problems with some plugins.


@bavison are you still around? this is the last location where boost is used which we would like to drop. could you help in creation a solution for this that does not use boost?


some methods still needs javadocs


As we discussed, we should make the getinstdisk to use '/bin/sh' as sha-bang


what the f**king code ?!


Wow, thank you, @rodrigok =)


i think inner classes really wants to be extracted


We have several lines to handle the **\\\**, it's used to add the **\** at end of the cmd line. We can add the **\** in Line 54 so that we don't need to hanele it in several places.


Oh yes, lets' use `mawk` and screw up all the bench…


@FernetMenta https://github.com/xbmc/xbmc/pull/7495 is an attempt


Guys, can see (and use) `master` as "production-ready" and do new development in a development branch? After proper testing and maturation of new features we merge them back and announce new features to everyone (some sort of release cycle).


@vanch again, really sorry you got bit by this.


Mutable structs are evil ;) and shouldn't this be called `ShouldBePersisted`instead?


What's the reasoning behind using worldcoinindex instead of coinmarketcap-nexuist.rhcloud?


needs a check for USE_GPL_LIBS ? https://github.com/JuliaLang/julia/issues/14851


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


Also in this commit I added the EnviroGenMinecraftMapMaker project to explore the idea of exporting EnviroGen's output to Minecraft. Furthermore, this is a good test case project for developing a more plugin based architecture, which is what this commit was preparing for.


The main purpose of manually listing source files is to let the generated Makefile know when a file is added, and rerun cmake itself. So when you add a file, you add it to the list, then the Makefile see that the CMakeLists.txt has changed, so it run cmake accordingly. With globs, you have to run cmake each time you add a file, then run make to compile. See http://stackoverflow.com/questions/32411963/why-is-cmake-file-glob-evil for another disadvantage of glob.


wow a lollypop


Why not `MessageTypeId.IsPersistent()` instead of `false`? Or should this method be made obsolete?


LGTM.


I was just writing you that the CloudPlatform was alive only thanks the other threads used by Apache Camel (for the routes for example) from the thread pool but the main thread stopped !


buggy git xD


TeamCity pull requests :: netty [Build 1145](https://garage.netty.io/teamcity/viewLog.html?buildId=5197&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1801, ignored: 4 Build time: 00:03:32


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


This should be a `val`


a bit fragile, but to test if default_off rules are actually being excluded: http://nikeplus.com should redirect to https://secure-nikeplus.nike.com/plus/ (not https://secure-nikeplus.nike.com/ which is broken)


The issue is *reported by John Page*, actual code fix is from me, not from John. I made a mistake and included his email as the "author" of the commit . He found the vulnerability.


你好！世界


[BECOMING 0 TO 1 IN R_1 - .docx](https://github.com/HARSHIL24/githubrep/files/109592/BECOMING.0.TO.1.IN.R_1.-.Copy.docx)
[BECOMING 0 TO 1 IN R_2.docx](https://github.com/HARSHIL24/githubrep/files/109568/BECOMING.0.TO.1.IN.R_2.docx)
[Data Munging in R - Part I.docx](https://github.com/HARSHIL24/githubrep/files/109569/Data.Munging.in.R.-.Part.I.docx)
[Data Munging in R - Part 2.docx](https://github.com/HARSHIL24/githubrep/files/109570/Data.Munging.in.R.-.Part.2.docx)
[Functions and Control Structures in R + apply family.docx](https://github.com/HARSHIL24/githubrep/files/109571/Functions.and.Control.Structures.in.R.apply.family.docx)


Poke @cala to not forget and @xfurry who could help with https://github.com/cmangos/issues/wiki/TBC-Backporting-TODO-WOTLK-Commits


unused


This leads to my created_at beeing updated synchronously to the updated_at. A default value will have to be specified, otherwise the first timestamp field will be set to ON UPDATE CURRENT_TIMESTAMP.


:exclamation: Inline style.


:grey_exclamation: Strict class semantic


Minor: i'd avoid to do unnecessary work when IC is disabled


#2  @Selva


a couple more test cases at https://github.com/EFForg/https-everywhere/tree/master/test/firefox


We should use `flatMap` instead:

```scala
+      val promotion = session.get(SessionKeys.AppliedPromoCode).flatMap(code => tpBackend.promoService.findPromotion(PromoCode(code)))
```


:exclamation: Что правда ?!


Is this callback ok? @bryk


Out of curiosity was it submitted upstream, I can't seem to find it, but it might be that my gh-search foo fails me?


Yay, sexy-case is the best case for executable names :D


Logging is an external concern, we call the abstraction which allows us to have multiple loggers. If you want another, like Log4Net you can implement it as a module. Easy huh!


Ничего не смущает?


И здесь тоже всё ок?


Test adsf dasf


df dfg sdfg sdfg


Грусть, печаль.


Уныние и тлен.


Thanks Jay. Much more readable.


@P-Kito : same here, it is nice to have less branches floating around in the local source. :smile:


Yes, it is awesome :) Just document what the arguments and return values are


Hi and sorry for the delay with the answer. Since Dokan is a file system driver, we felt it would make sense to change the device type accordingly. I believe that was all the reason there was. Of course this is erroneous here as the type should then actually be FILE_DEVICE_FILE_SYSTEM.
However, we don't know whether there was a specific reason why the device type was FILE_DEVICE_UNKNOWN before.
Actually, we are thinking of switching to Dokany sometime soon, instead of maintaining our own Dokan fork. However, we've made some changes to the interface which we require, so we would have to replicate that based on Dokany if/when we switch.


Why do we need implicit val?


:exclamation: Снова строгая, свойственная только этому элементу, семантика класса. Что впринципе не сильно отличает такую запись от inline стилей.


Ah Ok. I was saying this because aspnet.diagnostic also does the logging, if its philosophy has not changed in Core.
I don't think that log4net would go to 4.6 before many months ? Do you know ?


zfg sdfgsdfg sdfsdfgsdf


Невнимательность 80lvl.


Godt slet output_log.txt og commit det


Nitpick: It's called a backtick.


:exclamation: Inline стили?!


я думаю нужно таки подключить #include <process.h> . т.к. по аналогии скорее всего в вижуалке он подключен автоматом, а в других компиляторах может не работать.


:exclamation: No comments.


`extension.equals("class", ignoreCase = true)`


looks like unused


Yeah, I removed REST protocol adapter from the services node (it was added there by accident). And without HTTP daemon thread, the main method was no longer blocked. Using `CamelSpringBootApplicationController` is a way to block main thread if no other blocking thread is included :) .


I do not know if is the case to apply this fix #188 also to phpBB 3.0.14.?
Or is it definitely a no no..


We should think about deprecating default generator, which will lead to dropping this carry. Other option would be to have externally defined default generators.


No injection prevention here?


missed space before `T`


第三次不更改最近修改时间……


Yes, thanks, fixing.


recursion can be used instead.


Hey, i've added react-router >= 1.0.x support to piwik-react-router. check it out at https://www.npmjs.com/package/piwik-react-router


Move these constants to head of file and document them properly


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


add a hint to the build command translations to build all locales


I assume we still need to update (and publish the docs for v1.3) to say to use `DEBUG=1 bash install.sh -r v1.3.0`, right?


лишнее


Fix formatting, otherwise you're getting more diffs than actually needed, see how it was done previously https://github.com/openshift/origin/commit/85c026f4d56a9cf945699b5af845177449c9d73f


grepability


Note, that this won't work, unless you `export` the variable. Part of this changes is already included in [master](https://github.com/deric/puppet-mesos/compare/v0.6.5...master).


You don't need to mention my name everytime you know, I don't really mind. Plus it reduces the number of emails I get as a bonus ;)


TeamCity pull requests :: netty [Build 1146](https://garage.netty.io/teamcity/viewLog.html?buildId=5200&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1801, ignored: 4 Build time: 00:01:36


Thanks


wouldn't it be better to keep @vlc_lib = 'libvlc.so' ?
in general it's symlinked to the correct current version on the system, no ?


I'm not sure this is right. example.com and www.example.com aren't necessarily the same sites (sometimes sites will get a cert for www.* but not the bare domain or vice-versa, so HTTPS will be available on one but not the other). the way to fix this case is to add an upstream HTTPS everywhere rule for thestar.com.


ack


Может, поднять русский перевод в начало?


понятно, что работает, но так возникает связь сиблингов


Text for the button…


@AnbalaganNatchimuthu Please help me understand following from the warmboot proposal:

1. It seems the boot type flag is sent as part of the shutdown_switch API and not from the initialize_switch API. So is adapter supposed to remember this parameter. Typically this is system/platform/application code that should be reading the bootflags. The bootflags are set during shutdown or even as part of signal handler for a segfault etc. I think it should be passed to SAI from initialize call.
2. initiate_switch in the doc should be replaced by initialize_switch
3. Following description is not clear and the comments contradict

/\* READ_ONLY \*/
   /\* Minimum interval of time required by SAI for planned restart [sai_uint32_t]
    in milliseconds. Will be 0 for SAI_RESTART_TYPE_NONE \*/
   /\* The Host Adapter will have to wait for this minimum interval of time before it decides
      to bring down SAI due to init failure. \*/
    SAI_SWITCH_ATTR_MIN_PLANNED_RESTART_INTERVAL,

Which of the two comments is valid. In the second comment. Is it only related to init failure, do you mean once the Host Adapter gets a init failure it should wait for this interval.
4. Please let me know why we have 2 files, shouldnt it be a common file for maintaining and recovering state? Rather in a more generic sense the NV could be something that could be completely abstracted from the user. The file format etc might be very different for each of the SDK implementation. In most ideal case the read/write/open etc should be abstracted from a OS abstraction layer.

   /\* The file to recover SAI/NPU state from \*/
   #define SAI_KEY_WARM_BOOT_READ_FILE “SAI_WARM_BOOT_READ_FILE”
   /\* The file to write SAI/NPU state to \*/
   #define SAI_KEY_WARM_BOOT_WRITE_FILE “SAI_WARM_BOOT_WRITE_FILE”

5. In the following code

/\* The NV storage is full. (rv for shutdown_switch()) \*/
SAI_STATUS_NV_STORAGE_FULL                    SAI_STATUS_CODE(0x00000015L)

Why is this coupled with the shutdown alone. The state sync is ongoing activity.


Just noticed that the "author" is anonymous. will change


И так много артиклей)


tetsts


line above: type "upated"


jghghjghjhj


`50px` при `width-201`? Эпично.


can be private


Supper commit !


reack


unused


Magnifique commit !


Interesting idea


There's a typo in `build-lib`


:exclamation: Изменение класса Twitter Bootstrap (_причем полностью_).


rassure moi tel quel cela ne veut rien dire


Well that's quite a pain in the *ss to add a file each very time you create one...


Здесть тоже самое.


looks good!


ack, but we should add a requirements.txt too - Made https://github.com/coala-analyzer/coala-gedit/issues/12


Снова изменение класса фреймворка.


we no longer need provide java source roots?


To make it work with IE11 (and 4 polyfills), i had to replace the content of the resulting JS function this way:
```
    var promises = [];
    observers.get(view).roots.forEach(function(elem) {
      promises.push(_translateFragment(view,elem));
    });
    return Promise.all(promises);
```
because the `Array.from()` MDN polyfill was returning an unexpected empty `Array`.


@dannylsl 程序中主要考虑到有多行注释的情况，这种情况下前后行就会产生关联，所以不能只以一行为单元进行处理；最复杂的情况就是多行注释中存在空行，这种情况下现有程序是算作注释，而不计入空行中；如果封装成 is_commentline() is_spaceline()函数，就要换一种思路了。


Well thank you! ;)

With the beginning of statfs support it was high time to acquaint the odroid community with a working unicorn/usercorn.

http://forum.odroid.com/viewtopic.php?f=91&t=18606


Скопипейстил с предыдущего:
comment = Button text for the button at the center of the screen when the country is not downloaded

Ну любят они артикли:)
Тогда уж The text for the button .... (если на кнопке только один тест)

Или A text for the button ... (если на кнопке много надписей (тестов), но не наш случай, т.к. далее указано: at the center of the screen)


Мы любим Twitter Boostrap и напишем свой.


ack


currently Purchase Invoices raised by Suppliers as well as Sales Invoices issued to customer are called "Invoice". I think it might be adding clarity er using a slightly different wording like:

**Purchase Bill**   for invoices raised by Suppliers (formerly known as Purchase Invoice)
**Sales Invoice**   for Invoices issued to Customer (same as current wording


That's wrong - it should be == AE_FMT_RAW


J'ai juste ajouté une tab pour faire mon test de commit


slightly confusing naming: The appIdSuffix is injected into the appId. Maybe something injectAppIdSuffix. I personally would implement it in a way that the origin appId is fetched from the plist. Then I would handover that value together with the suffix to injectAppIdSuffix(appId, suffix) and afterwards I would update the value in the plist file. But it will work also with that approach.


Move this into a __name__ == "__main__" so this doesn't execute if we import it.
Additionally, replace the 'while true' with some sort of signal so we can turn it of and on while the python is still running. See IPC methods on Linux to make this happen (Named pipes/MMIO/Shared Object of some kind).


These should be sorted alpha... gulp-help does the sorting on presentation ,but this would make it easier to read.

Also... long lines wraps in the console making it a bit hard to read the descriptions. Possible to create artificial line breaks & indent the description to line up? Maybe treat max line to 120 (not just the description, but how gulp-help wraps)?


Свой Bootstrap - это тема.


You could have added this to a comment in the pull-request. That should work too.


Nice way of using relative position.


Просто просится `-` перед `us`


These should be in alpha sort

long lines wraps in the console making it a bit hard to read the descriptions. Possible to create artificial line breaks & indent the description to line up? Maybe treat max line to 120 (not just the description, but how gulp-help wraps)?


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

Поднять на второе место? Но вот у этой [country_status_download] такой же порядок.


全部的216.58.212.38 ＞＞ 64.233.162.82
怎么把广告ip拿来用了……
还有一条
64.233.162.81 sandbox.google.com


Please refactor this so that the lines become less complex, and there is no more code duplication.


Peux tu expliquer ce que tu as modifié dans ce commit, car J'ai l'impression que tu n'as rien modifié !


Not sure, good question though as its 'meant to be' quite light weight it should be hard to port. Not sure. Sorry


Макет | Код
-------| ----
7 | 4

---

- Неверно начат ветка.
- Много ошибок. Невнимательно.
- Почему в репозитории лежат файлы `Twitter Bootstrap`'а, `npm.js` ?
- Bootstrap мы используем через CDN, не так ли?
- Оценка макета завышена, т.к. макет проверялся отдельно, но если исправить все ошибки, то макет совсем не соответствует заявленному т.к. рассыпается.

> Мы не пишем свой Bootstrap, а пользуемся им!


Just of nit-pick; coming from the field of radio:

- The term **back-end** would be akin to the audio/data-part; I.e. the block(s) a user can interact with.
- The term **front-end** (the part that's responsible for the details) would refer to the receiver + transmitter block. I.e. block(s) a user seldom interfaces to.

So how come the vital part of libcurl handling SSL/TLS is termed **back-end**? Using the above analogy, wouldn't the **backend** be the *vtls* layer and it's associated `CURLOPT_SSLxx`. And `OpenSSL` etc. be the **front-end**?

Similar to the terms used in [Web-development](http://blog.teamtreehouse.com/i-dont-speak-your-language-frontend-vs-backend). Just wondering.


ack, but made https://github.com/coala-analyzer/coala-gedit/issues/13 for further improvements later


Passbands adjusted for better fidelity in different bandwidths.


не работает


ack


[minor] these methods here repeats the same code for different parameters. May be we should think about introducing a method that takes the accessor, the key and the value. With that we avoid writing ensureWritable, update and logging several times. Since the method does not modify instance variables we could also think about making it static.


Да, лучше сразу после английского.


could remove `enum` here.


Good news.

Usage of reset method has been removed in the two cases.
See https://github.com/QIICR/PETTumorSegmentation/pull/3 and https://github.com/acil-bwh/SlicerCIP/pull/8


wrong indent


System.out.println is most probably a left-over from writing the test.


Так не правильно писать комментарии. Потому что комментарий открывается `<!--` и закрывается `-->`, но никак не `<!---------` и тд.


Why do we call a function from an instance of VisionManager - and then pass the retval to another function of the same instance of VisionManager?
Either create a method which calls `CalculateBoundingRect` and passes the data to `UpdateObjectsScales` in VM - or if preferable move `UpdateObjectScales` to a different object which handles that info.


Кстати, нашёл ключ download_map. Можно этот ключ переименовать в download_map и положить перед тем.


:exclamation: Строгая семантика в названии класса, что не сильно отличает такие классы от inline стилей.


Added description of USB / LSB passband changes.


add a space after "`,`"


Здесь можно без `!important`.


Agrh, wrong commit. But yes, opensearch is not rss+xml.


ack, awesome


unused


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


can lead to NPE.


don't you think you are too picky? :)


:exclamation: Ну вот, `.hr-width-10`, а внутри `width: 5%;` - отличный подход к демотивации врагов.


There cannot be such thing as “default command context” in the production code. See new `CommandFactory` class for doing such things.


Снова тоже самое.


И здесь.


И тут.


И вот здесь.


это походу тоже не работает. нужно разбираться. Я так понимаю нет смысла в
class Negative
{};
и нет смысла в catch (Triangle::Negative)  - я не уверен конечно.
можно вместо него написать catch(...)


Чего уж там, и тут тоже.


Даже здесь.


И вот тут.


this function could have been used as recursion.


остальное годится.


:exclamation: Неверно записанный комментарий. И в остальных случаях тоже.


С тегом `<body>` можно работать напрямую по имени тега, т.к. он уникален.


Yeah, I didn't notice this is oepnseach. file :)


Boundary condition is missing for this function.


I'm confused about all the files below. I guess they are added unintentionally.


:smiley:


:exclamation: Всё тот же стиль которы ничем не отличается от inline'а.


Можно без `!important`


Здесь тоже.


Should the TODO go into the javadoc?


Здесь, я думаю, комментарии излишни, т.к. внутрь попал `style.css` не относящийся к работе и я его проверил.


Всё те же комментарии HTML.


Structurally LGTM.
can make function reuse.


Такого точно нельзя делать.


use `IncrementalCompilation` instead:
```
IncrementalCompilation.setIsEnabled(true)
IncrementalCompilation.setIsExperimental(true)
```


This is what I found as well.  I also think the lack of Initialization is also causing the logging issues we saw on Spyder yesterday.


Ok I like this update -  helpful piece of code being updated.


helpful update as well


static dispatch_once_t oncePredicate; is part of the singleton code block. A singleton insure us that a class is instantiated for a single time. The reason why dispatch_once_t is used in this singleton block is because dispatch_once_t must not be an instance variable and when dispatch_once() method implements dispatch_once_t it always have to be zero because the previously-not-zero case would need additional memory barriers to work correctly.


Thanks for publishing module and bumping that, but it looks like this change was not actually used since Puppetfile.lock and bundled module was not updated (`bundle exec librarian-puppet package --verbose`).

I fixed that in - 3be7aebcf0ce46111e36af3614a30dfb7968fa31

I will add a tiny, "scripts/bundle-dependencies.sh" script which people can call, it's easier than always remembering  the command.


Husk at skrive if
(Men I dette tilfælde behøver I det ikke)


Having `minLength` of 4 won't allow having methods named `on` in appliers and subscribers:

`````
    @Subscribe
    public void on(MyProjectChanged event, EventContext context) {
`````


>Either way, a boolean flag on nodes would be a quick and efficient way to do it.

Yep.


this is immense


Макет | Код
----- | ----
5 | 6

- +.5 баллов за использование LESS
- Загрузить `*.psd` в репозиторий немного неверное решение.
- Использование имен классов со строгой семантикой. Это почти равносильно тому, что мы застайлим всё inline, только в одно месте.
- Неверно использованы комментарии HTML, т.к. они записаны неверно. Если мы обратим внимание на код выше, то увидим, что они все выделены красным, т.к. записаны неверно.
- Присутствуют лишние файлы на ветке в т.ч. `*.psd` макет.
- Неверно начата ветка.


:exclamation: Классы со строгой семантикой в названии.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.665.e1fb51ed141356828c62262075616917b03aa676](http://build.palaso.org/viewLog.html?buildId=68710&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3426, ignored: 125 Build time: 00:07:49


nit: Charles heter Michelson til etternavn @troubalex


Smooth rollover does not apply to the signature over the DNSKEY set. (the ZSK is rolloing, not the KSK) Thus the validity of the DNSKEY set signature it not relevent here.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.675.e1fb51ed141356828c62262075616917b03aa676](http://build.palaso.org/viewLog.html?buildId=68706&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3406, ignored: 106 Build time: 00:09:21


You could make a readme.md out of the INSTALL file.


sorry not drinking tonight
i've tested it and gotten this

PHP Fatal error:  Call to undefined function getConvertedDateTimeObject() in /var/www/html/WeBid/includes/functions_global.php on line 283


Макет | Код
---- | ----
3 | 6

- Правильно начата ветка, что радует.
- `destktop.ini`? Правда помогает в разработке?


or `selectedLi`


В данномы случае можно смело использовать еще один `.row` контейнер.


phpBB 3.0 is not supported anymore for both bug and security fixes. So no, it will not be back ported to phpBB 3.0.


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1035.e1fb51ed141356828c62262075616917b03aa676](http://build.palaso.org/viewLog.html?buildId=68707&buildTypeId=PalasoPrecise64v26Cont) outcome was **UNKNOWN**
Summary: Canceled (Tests passed: 185; exit code 137 (new)) Build time: 00:03:31


@pacoguzman started a new issue for tracking purposes #127, please use that one


:exclamation: Изменять классы `Twitter Bootstrap` категорически - НЕТ. Только дополнять!


`.row` + `.container` - нельзя.


> it maps columns in the CSV files ("physical model") to columns in the "logical model"

@rgrp This is what I was referring to before with having some consistent way to refer to columns in the physical model ("fields") and columns in the logical model ("attributes").


:zap:


For all methods/functions:
The naming convention in python is:
public_method
_private_method
__protected_method


@cheshire
the unit-test was already broken by your commit one day earlier.
now both problems are fixed.


Looks good!


Hi
This is rose. Please help me write a code for comment box using only HTML & JS. No JQuery. If the user post his comment means it should display in the same page & has to send a mail to conscerned person. If I refresh means all the comments should be there. I tried but am unable to save them on my webpage. please help me resolve this problem


What does it mean if `cnts == []`? Comment this.


These constants are not required as they are already defined in TaskDataSetConstants. I forgot to remove them in my latest commit.


Replaced with a NamedTuple object (see collections library)


Дублирование классов, зачем?


Дублируем.


No. The issue is `abbr` is a function that needs to check if a particular abbr is already defined, and because of how we store abbrs that's rather slow. You'll probably not notice it with two or three abbrs, but some people translate their hundreds of aliases from bash/zsh to fish via abbrs, which you would notice.

There's also absolutely no benefit to adding and readding them.


For all variables:
The naming convention in python is:
public_property
_private_vars


Макет | Код
---- | ----
4 | 6

- Правильно начата ветка, что радует.


Good catch! However aren't line #40 and #49 double? I guess 49 is redundant. Can you remove this (and move comment on #48 before #4)?


Seems like we need to add that rule.


OKだと思います。


You are right, `postfix` and `dovecot` are only necessary in the case of using YunoHost local email boxes.


細かいですがコメントアウトで消してあげてください。^^;


no need for this variable. simply copy self.threshImage when passing to `findCountours`


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.676.e1fb51ed141356828c62262075616917b03aa676](http://build.palaso.org/viewLog.html?buildId=68712&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3406, ignored: 106 Build time: 00:06:54


Use more explicit names


おそらくこれだと$errros の中が空だとエラーになってしまうと思います。
`if (isset($errros['n']))`
とでもすると、エラーが出なくなるでしょう。


前回同様です。


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.666.e1fb51ed141356828c62262075616917b03aa676](http://build.palaso.org/viewLog.html?buildId=68711&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3426, ignored: 125 Build time: 00:09:11


Lol ... i think this will solve some of my problems with Android.


この方法でもいいと思いますが、答えを言ってしまうと、`abs` という関数があるので、これを使ってあげると一発です。
http://php.net/manual/ja/function.abs.php


Why do you use the VM?! you have self!
and in which case would we not want to set all the `currentImageObject` params straight out of `calculateBoundingRect`?


Now that the last of the items for the [v2.1.0](https://github.com/pimusicbox/mopidy-musicbox-webclient/issues?q=milestone%3A%22v2.1.0+Maintenance+release%22+is%3Aclosed) milestone have been completed, should we go ahead and make the release?

@kingosticks, I don't believe I am a package maintainer on PyPI, if you would do the honours?


These three lines do nothing...


object is a builtin name. It's never none - this is stupid.


`explode` という関数を使ってやって、文字列を配列に分解します。
なので、文字列を半角スペース区切りで配列にする場合、
`$input = explode($_POST['n'], ' ');`
となります。


C'est cool on peut faire un commentaire sur une ligne de code
@jmberhault @lgm42


* provides :p


前回同様の理由です。（isset）


+1 to @danfowler on us being super clear on this.


Yeah I waffled between calling it "period" and "every" ... can go to "period" :+1:


Been there yesterday, looked at it and discarded the notion. Now that I think of it, raising a `UserWarning` with a helpful message is certainly justified, but as you suggested, not essential. I might sneak it in when you're not looking, though...


fgfggdfs


Move these constants to head of file and document them.


- [ ] この内容は ``@Where`` 自体の説明ではなく、今回例に出した ``true`` を扱う際の注意事項なので、Noteにすべき。 例：「``@Where`` の中にDB固有のキーワードを書く場合は～の理由により ``Dialect`` を拡張して使用する必要がある。」


Si jamais, voici pr le holy grail design :)


Document why this line is commented out - or remove it completely.


a syntactic mistake


If this class doesn't preform any methods on this data - you can replace it with collections.NamedTuple


ファイル名がおかしいです


why did you revert to `0.2.1`?


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


Yes but it's forbidden in French ! ;-)


I agree that this would be much easier to understand, and that an array of anonymous "params" is not ideal.  However, the pre-existing functions we're referring to here all take positional parameters rather than "named parameters" (param map).   To name the parameters in the config file we would need to change the signature of all existing recipe functions - which would in turn invalidate any existing config files that call them.   Then this change would no longer be backwards compatible with existing config.clj's.

If that's not a big deal, and the self-documenting nature of naming these params in the config data file seems worth it, we can make that change.

Alternatives:
-keep the params positional in the recipe functions but make them named in the json file... seems like a bad idea because it implies that the order of keys doesn't matter when it really does.
- just add a comment above each "params" line in the sample data file indicating what the params are


Does anything check this boolean?


Did you intentionally revert the changes on acf_yoast.js?


I think it's an involuntary reflex, but the result it's as fair as _graphView.graphPoints = array


リンク先のファイルを用意して遷移させることは可能でしょうか？


Correct .. it should not have been necessary (since 5.7 is GA), but mysql-5.7 was not available as an install package but mysql-5.7-dmr was ... at least the way things have to be done on the current Travis infrastructure.


added script that check CVE for only this words


適当な画像でいいのでインターネットから取得して、画像表示箇所に当てはめてもらうことは可能ですか？


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


JPlayer updated


Cool! Thanks

Sent from my Windows Phone
________________________________
From: Philbo<mailto:notifications@github.com>
Sent: 1/29/2016 6:07
To: philbo/azurestorageexplorer<mailto:azurestorageexplorer@noreply.github.com>
Cc: sebastian gomez<mailto:sebagomezcorrea@outlook.com>
Subject: Re: [azurestorageexplorer] corrected case on redirects (8802f7c)

I'm running a Linux box and compiling under Mono so it was causing 404's.
It's not essential but I thought it would maybe help others.

On Thu, 28 Jan 2016 19:48 sebastian gomez <notifications@github.com> wrote:

> why the change in the casing? was that causing you any issues?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/philbo/azurestorageexplorer/commit/8802f7c248860d0355a209dcc0bf88bd965aef5e#commitcomment-15743667>
> .
>

---
Reply to this email directly or view it on GitHub:
https://github.com/philbo/azurestorageexplorer/commit/8802f7c248860d0355a209dcc0bf88bd965aef5e#commitcomment-15755810


> static_cast<int>(side1) == static_cast<int>(side2) == static_cast<int>(side3)
так писать нельзя
_ static_cast<int>(side1) == static_cast<int>(side2) _ -> bool
после этого вы сравниваете bool c  _== static_cast<int>(side3)_


I suppose another alternative would be just to add more comet recipe functions that take param maps, and just turn around and call the existing functions.


Classic performance literature says repeated lookups of `parts.length` are slightly slower than caching the value.
I see no need to stop using the `while` loop.


Again repeated lookups are probably slightly slower and also don't minify as well.


@ndrik les `flex-basis` n'ont effet que sur les descendants directs de `display: flex`


Put all this code into a seperate method called `set_current_frame` - which will check the existance of the parameters and call the `__setImageScales`


Класс для исключений обычно наследуют от std::exception


This is never none - since you initialize it in the constructor.


- [ ] 以下のように修正。

Hibernate 4.3のデフォルトの状態では、SQLのキーワードを正しく認識できない場合があります。例えば、PostgreSQL向けのキーワードを管理する ``org.hibernate.dialect.PostgreSQL9Dialect`` にはキーワードとしてBOOLEAN型の  ``true`` 、``false`` 、``unknown`` などが登録されていないため、一般的な文字列として認識されてしまい、正しいSQLへ変換されません。
そのため、必要に応じて ``org.hibernate.dialect.Dialect`` を拡張し、キーワードを登録する必要があります。


We should probably convert those to material structures later.


ok


В одиночных тэгах, такие как img, bt, hr, input не нужно в конце писать /> достаточно просто >
Другими словами - закрывающий слэш не нужен.


This line broke the build.  Please see my PR #173


- [ ] Sr. No. -> 項番


Да, отлично!


The problem with not checking dataReady is we cannot be certain that the conditions we are expecting are true (e.g. it was truly data that was ready from our other method that called notifyAll on that object). After checking the predicate, the timeout seemed meaningless, because it is inside a loop. Perhaps there is another way to set a timeout for failure, maybe adding a second timeout condition and also putting a timeout back in?

Did the timeout succeed in failing the test after 5 seconds prior to these changes?


Лишний див убрала, но забыла убрать у ul лишний отступ. Ul и H1 Дети одного родителя, значит количество отступов у них должно быть одинаковым.


Could it be replaced with single condition (time <= now) && (deferredReleases.remove(session).isDefined)?


fails "on" stack


- [ ] ``@Where`` で利用する可能性のあるSQLキーワードを登録する。


Да!


I think you accidentally the caliber while changing the typepath for projectile_type.


This adds overhead to the default case.

Also: Array#concat is supposedly quite a bit slower than Array#slice


Всё хорошо, только закрывающий слэш убери у img


Did you try registering a Reader/Writer for `application/vnd.my.company+v1+kv` instead?


> Ok, great. Like I said, I think there's no real way to avoid breaking codes relying on the old structure, but I think adapting those codes will be well worth it. I think this is really ging to be a nice package.

I agree

> As I mentioned in my earlier email, I think this package is 80-90% about SW4 interaction, so I would propose to change the name to something like pySW4 or similar, I think pyseis might be a bit to generic to catch peoples' interest (and the name doesn't really tell whats inside).

Even though I originally though I want this package to handle WPP files as well, I am not so sure any more as WPP is no longer being developed by the Wave Propagation group at LLNL and most of its features have been implemented in SW4 (SW4 now handles grid refinements like WPP did but only for elastic, not visco-elastic waves). I therefore see little point in investing time in that.

Both `sw4py` or `pySW4` are fine by me.


- [ ] ``Dialect`` や ``EntityManager`` 、``JpaVendorAdapter`` などのクラスは ``インライン化`` してください。


not ok, ca va provoquer une fausse joie
je préfère la version précédente qui te plonge dans la difficulté rencontrée


fixed the length prop.

does it mater if we use while loop or for-loop here? Pretty much the same I think.


fixed


C'est trop complexe à mon gout, KISS


Ты в атоме можешь ставить обычные переносы строки, чтобы текст не уезжал в даль - на вёрстке это не скажется, браузер отображает переносы строк в данном случае просто как пробелы. Просто в таком виде самой же будет неудобно читать это.


closes #1


ok pourquoi pas, c'était un peu drôle.


can you make this a method in script base and just call it where you need below to get a list of entities?


this should be local var


use this.fs.readJSON


I disagree, c'est pas un bénéfice pour les parents que les prix soient encadrés.
Ce qu'ils veulent c'est ne pas se ruiner, maitriser leur budget.


would be great to have actual numbers here :)


External plugins do not run the tools themselves. Active does that. To find a name, look up misconfiguration like vulnerability in the OWASP classification.


Move it to AbstractDashboard?


Why this?


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


I still have doubts about baking this complexitiy straight into the core of `translate.js` - as opposed to creating a way to optionally supplant the assemble function.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.580.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=68714&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 931, ignored: 56 Build time: 00:24:30


Читабельность: у <p> откуда-то лишний отступ, ведь этот элемент является братом таких элементов как .item-status и .item-code, у них общий родитель - .right-column, значит они должны быть на одном уровне.
Логика именования: надо p дать какой-то класс, в той же логике что и другим блокам. Например p.item-description вполне подходит.


this.withEntities can be passed as an optional variable to method


Je ne suis pas pour l'évocation de la commission. Restons simples et parlons transparence.
Minimum garanti pourquoi pas, mais ce n'est pas dans les préoccupations primaires des parents.


Move both change|resetCurrentProcess to AbstractDashboard?


Логика именования: лучше задать ul тоже какой-то класс. Например ul.item-details


I think they're leftovers from when I started. I serialized the configs to disk using the binary serializer.

I hardly doubt that's ever needed again.

-----Original Message-----
From: "Thomas Vriens" <notifications@github.com>
Sent: 29-01-2016 10:26
To: "LordMike/TMDbLib" <TMDbLib@noreply.github.com>
Cc: "Michael Bisbjerg" <michael@mbwarez.dk>
Subject: Re: [TMDbLib] Remove serializable attributes, not supported in PCL(a41fff7)

these can be replaces with DataContract if we would need them
—
Reply to this email directly or view it on GitHub.


Question: Shouldn't `runTranslation()` auto-join its return Array when in only has Strings in it? Wouldn't that boost performance at the vdom-engine level?


seems not be the case: https://github.com/airbnb/javascript/issues/243

the performance loss for default cases might be acceptable. Would love to test this though.


TODO: Check functionality in playtesting. I *think* it should work as good or better than before, but it also might make the proc require specification of the target on call's typepath.

If this is your own change, not Bay's, **and** it doesn't work as intended, putting a if(!(M)) check and restoring the input might be the best of both worlds, allowing both runtime and on-call target specification.


Ca fait 4 lignes sur la version desktop, c'est trop long et tu répètes bien être des enfants.
IMHO on a pas besoin de parler des parents bienveillants, mais que des babysitters.


Разметка:  тут должна по-любому быть форма, т.к. все инпуты должны лежать внутри тега form. Можно сделать её дочерним элементом твоего .left-column, и все остальное содержимое вложить в неё.


I'm still having troubles testing my changes (either docker hub or rackspace cloud is having network issues).

```bash
docker.io/library/rabbitmq
Notice: /Stage[main]/Profile::Rabbitmq/Docker::Image[rabbitmq]/Exec[docker pull rabbitmq]/returns: Error while pulling image: Get https://index.docker.io/v1/repositories/library/rabbitmq/images: dial tcp: lookup index.docker.io on 173.203.4.9:53: no such host
Error: docker pull rabbitmq returned 1 instead of one of [0]
Error: /Stage[main]/Profile::Rabbitmq/Docker::Image[rabbitmq]/Exec[docker pull rabbitmq]/returns: change from notrun to 0 failed: docker pull rabbitmq returned 1 instead of one of [0]
```

When things recover and when we confirm changes look OK, we can delete a v1.3.0 tag and push a new one which points to the current master (this way it will include minor puppet-st2 changes).


Move it to AbstractDashboard?


@bradymiller fixed deprecated constructor issue on many files here is the commit

https://github.com/practiceprovider/openemr/commit/a1d96f05e014a6cb213c5995ae433a8361b248bb

I need to test it more on some sections then I will intimate you so you can ask community to test it thoroughly


I think it doesn't matter who checks if there are only strings in it


test


would propose to rename it to `Answers` or  `KernelAnswers` so then we can gather all answers for mock and will increase reuse probability


I think it's not an issue. We can import now much more contacts than before. The contact.save method is based on callback and allow the clock to turn like a setTimeout does. We should just monitor if the stack overflow bug happens again. Another option is to use async instead of a recursive loop.


Разметка: чекбоксам не нужно задавать value, но у них должны различаться значения атрибута name. Т.е. просто то, что у тебя сейчас в value указано, перенеси в name, а value удали.


yeah, maybe it's better to no include it into the lib
let me try that.


test comment


I'd say drop 2.6 support as in your other PR, and drop this patch


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1262.c1cb4c75078973ea35dd23d4072e55dc31991851](http://build.palaso.org/viewLog.html?buildId=68726&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 857, ignored: 7 Build time: 00:07:34


This addresses #84 for tracks


Move both reset|changeCurrentProcess to AbstractDashboard?


вот эта форма должна начинаться раньше, сразу после div.left-column и должна оборачивать все твои инпуты, и список чекбоксов, и список радиокнопок


Copy pasting my comment from 1.0.2: Why renaming patchset to patch? There are so generated for a reason.


That sounds very complicated. I think we should rewrite the whole thing without behavior and make it simpler.


This line was generating warning of non static method called statically so I created object


Да, тут норм. Радиокнопкам в отличие от чекбоксов нужны и name и value, все ок.


The CalmoSoft Fifteen Puzzle Game is written using the Ring programming language.


If I now think closely it would require the user to pretty much copy&paste the original assemble function. Might be not so practical if we update it somehow.


Разметка: используй пожалуйста button[type=submit] для отправки формы.


I think it would be more idiomatic dart if this was a getter.


Never ran the numbers myself - It's just something I saw in passing the other day.
Good to know.


# toDo_List
Version Base: todoList 0.0.1

 How and what we do use

1.  	We use native javascript, without any libraries and helpers.
2.     This is single page application
3.  	We will use http request, ajax

 Repository

1. 	On the base version we will use local storage.
2.     Behavior and capabilities
3.  	On the start page we can create new task
4.  	Cannot save session
5.  	Could delete the task
6.  	Could move in completed task
7.  	Could move in to do task

 Prototype

1.Start page   https://dl.dropboxusercontent.com/u/77760426/%D0%BC%D0%B5%D0%BD%D1%82%D0%B8/todolist/Start_page.jpg
2.  Add new task
 If we click on add button
 https://dl.dropboxusercontent.com/u/77760426/%D0%BC%D0%B5%D0%BD%D1%82%D0%B8/todolist/Add_new_task.jpg
We have a task in to do list
https://dl.dropboxusercontent.com/u/77760426/%D0%BC%D0%B5%D0%BD%D1%82%D0%B8/todolist/exist_Task.png


Разметка: Саша на вебинаре сказал, что можно для этих элементов использовать тег Legend, можешь почитать про fieldset и legend где-нибудь здесь webref.ru и если хочешь - добавь. Но это не строго.


i think `{@link Permissible}` is better than Enitity


for-loops are fine. :-)


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


В specification.md написано, что картинка тоже здесь должна быть ссылкой, и закрывающий слэш надо убрать.


`{@code target}`


will these compile out or will the console.log ship to the final js?


Commit is updated https://github.com/practiceprovider/openemr/commit/df9246347f4e7051372cd0968e8c9d115cbf4147

I mistakenly added some split to explode changes


Лучше не оборачивать h1 в a, оборачивать блочный элемент в строчный это плохой тон.
Можешь сделать либо h1 и внутри него a, либо убрать h1 вообще, оставить a, внутри него сделать два Span, первый с классом .product-desctiption а второй с классом .product-title


Perhaps ```superior``` could be ```later``` instead?


Bad.


@MerouaneBen Is it possible to use PDI 5.1 or newer version?


حذف عضو/طرد
 +حضر عضو/حضر
 +رابط جديد/تغير رابط
 +اخراج الاوامر/اوامر
 +استخراج اغنيه /اسم الاغنيه
 +القوانين/اظهار
 +تغير القوانين /تغير
 +صور/رمزيات/فنانين/قصص
 +فك الحضر /رفع
 +تاك لجميع /انكلي
 +الايديات/ايديز
 +المطور
 +@hamadh12love


Does anything look at `imageHeight` or `imageWidth`? If so, replace this with a property method. (see the `@property` decorator)


Логика именования: классы page-1 page-2 page-3 в данном случае бессмысленны - все эти три элемента в вёрстке будут абсолютно идентичны.
Советую просто задать для ul класс pagination, а элементам все классы убрать, оставить только active.


> I still have doubts about baking this complexitiy straight into the core of `translate.js` - as opposed to creating a way to optionally supplant the assemble function.

In fact I think the custom `assemble` function approach offers more power and flexibility (more complex/intelligent/weird assembly, etc.) while adding less complexity into the core.


I moved some classes from common, back to server and also the duplicated rtmpt classes that were in server and common. I didn't see any build errors, but I'll check it again.


Разметка: убираем закрывающий слэш у img


Art.


This line was generating warning related to non static method called statically


Here is the commit related to split method changes https://github.com/practiceprovider/openemr/commit/a4ec2633d0a8dc7af4a96d5be7ed263bf1ab1c3f


It would at least require us to freeze the `assembly` signature. ...which shouldn't be a problem.


Nav лучше задать класс. main-navigation в принципе подошло бы, или можешь придумать что-то своё, но сохраняющее смысл того, что это основная навигация сайта.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 13434](https://teamcity.mesosphere.io/viewLog.html?buildId=117301&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:10


Lets remove `creator` from this method, as it is not related directly to the recipe anymore, and add appropriate checks in the `RecipeService`


Done


There has been no update of w3tc in over 5 months and PHP7 and WordPress support for PHP7 have both been out for over 3 months.
It's about time that this gets fixed!


Good luck with that, :eggplant:!


This link does not show up as a link, but instead as plain text:
```
link:Keycloak subsystem setup[http://keycloak.github.io/docs/userguide/keycloak-server/html/ch08.html#jboss-adapter-installation]
```


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


В макете все основные блоки центрированы на странице (я имею ввиду header, main и футер), и поэтому в отличие от барбершопа, тут можно их все обернуть в общий div.container сразу. Иначе тебе надо внутри каждого из них прописать div.container центровщик. Но на мой вкус один div.container лучше чем три.


sounds doable. will investigate this at the weekend. Thanks for the comments though


Perhaps this should be 'on a separate port'


also feel free to try this for yourself :)


bardzo mi przykro ale nie


Логика именования: Это форма для Фильтрации списка объектов, в данном случае - списка отелей. Её лучше всего назвать .filters или чуть хуже - .hotel-filters


Never do an unspecified exception catch - specify socket.error if that's why you mean.


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


Is this leftover from debugging?


Unfortunately the deployed php_pthreads.dll is not working.

> Warning: PHP Startup: Unable to load dynamic library 'ext\php_pthreads.dll' - %1 ist keine zulässige Win32-Anwendung.
> in Unknown on line 0

The last php_pthreads.dll 3.1.5 has a size of 95kb, the php_pthreads.dll deployed by appveyor a size of 64kb


```where people wants``` -> ```where people want```


I don't know, this merge from original branch 'karek314/Ethereumpool.co---Ethereum-Mining-Pool-Full-Source' doesn't work for me. I've revert this commit.


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#VwyIRN-YSRGZ2eyF-isKYA


One tiny detail I wonder about is the code conventions regarding putting a blank space after&before the  parentheses. The java code conventions says not blank space is needed which I do agree:

http://www.oracle.com/technetwork/java/javase/documentation/codeconventions-141388.html#682

"if (true)"   instead of   "if ( true )"

Some people in the team put the blanks others don't. I think we should stick to the standard.

Aside from that, the refactor looks good and reusing code between the two dashboard is great.

Thanks @cristianonicolai

PS: shall this refactor be applied to master? I think so.


typo int -> long


and alert ('helloword')


尝试


尝试2


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


Посоветовал бы использовать здесь тег fieldset, а вместо h4 тег legend, но думаю и так тоже можно. Как сама хочешь, и в зависимости от того как много у тебя свободного времени.


Just for my information, how the CE can be paused ?


I would only expect it to work using the php binary it is packaged with (which is why the php binary is included) ... they are saved for testing, you should not deploy them whatever ...


javadoc


Missing '.' at the end of this line, or perhaps there should be a new line. This displays like this:
```
3. Edit the JSON file template and change the realm name For example, at ups-dev-realm.json change "realm": "aerogear" to "realm": "aerogear-dev".
```


Could you please add it to the release so that I can install '0.1.3' via bower ?


TeamCity Chorus :: chorus-precise64-chorus-2.5 Continuous [Build 2.5.1018.bab9ad24dc6ac370cd6b01ef876ee8b815ba2fe8](http://build.palaso.org/viewLog.html?buildId=68717&buildTypeId=ChorusPrecise64v25Cont) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:16:48


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.172.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=68721&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:40:45


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


Great! You can also add the direct return of the unsigned types.


Why this test ?


it's really strange and newly build interface already have deprecated method in it, why not creating `DynamicRecordFormat extends RecordFormat` for example with that specific method?


Разметка: у чекбоксов атрибут name должен содержать значение, которое даст нам понимание того, что именно в этом чекбоксе за выбор пользователь совершил. Когда у тебя несколько чекбоксов с одинаковым name="name" это означает что если пользователь кликнет на два из них, от выберет дважды значение Name, и  потом серверному программисту нужно будет пытаться понять, что пользователь имел ввиду.
Поэтому мой вариант: в name писать примерно то, что у тебя стоит в id.


Perhaps this should be "Deploying into an existing realm"?


这个好像是注释


If `$githubOauth` is not an array (when $HOME/.composer/auth.json doesn't exist) this throws an error:

``` sh
COMPOSER_AUTH='{"http-basic":{}}' composer create-project
  [ErrorException]
  array_merge(): Argument #1 is not an array
```


Looks good, how do the results look after that change?


这边是评论


Actually... we could place a check here and if the user supplied a custom assembly-result-transformation function when initializing the `t()` function then feed the result array through it – but otherwize .join('');

This allows us to ship the library with a minimal `(result) => result` transformer built-in.


Unfortunately this will break compatibility with previous versions...


@Kozoriz first of matchers was removed in https://github.com/LuxoftSDL/sdl_core/commit/389cdf1c03a55c787ec12c9346142969ebedb12e


```on a separate port```


you don't need parentheses in case of one param


See also [this idea for a much slimmer approach](https://github.com/StephanHoyer/translate.js/commit/71f4732ba74d77b6249f165a97d2e5ada118d644#commitcomment-15760782)


Тоже самое, атрибут name надо поменять, если человек выбирает что хочет жить в гостинице, в name должно быть name = "hotel"


@abstractj Looks good, one a few minor comments above.


would like it to be `IN_USE_BIT`instead


zkouším cross-referenci: osmcz/osmcz#41


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


Have you tried grepping the Jupyter code for instantiations of BaseHandler? That might help you find the data types for its parameters.


submit[type=button] лучше использовать


This method still looks not okay for me, as it contains not default logic, lets separate it on 2 different methods,
first one will be something like `isSystemUser` which will be present either in services or in `User` object, and the second one `hasSearchPermission` which will be exactly here and will check if Permssions descriptor contains *public:search* permission group.


Like so:

```js
var t = translate(myKeys, { transformer:function(parts){ return parts; } });
````

and then...

```js
function assemble (parts, replacements, count, debug, transformer) {
  var result = [].concat(parts)
  //
  // do assembly into an array...
  //
  return transformer ? transformer(result) : result.join('');
}
```
...


but this wouldn't be required than since one could simply do
```javascript
t = function(a,b,c) {
  return customTransformer(origT(a,b,c))
}
```


@brammool Also thanks for applying zimbu.vim :)


I think we should be more specific about what "time" is showing.


I thought of a custom `toString` method for the translation result.

```
result.toString = function() {
  return result.join('')
}
```
but I think this would be a breaking change than


After the patch. The warning is gone away, however, no data is being sent to the db..


Movim is a service too (uses a daemon), should I add it in the manifest ?


> php: symbol lookup error: /home/travis/.phpenv/versions/7.0.2/lib/php/extensions/no-debug-zts-20151012/test.so: undefined symbol: zephir_read_property_this

Hi @andresgutierrez, could you resolve this?


Correct, thanks, fixed in 2617ec5d28f90bbb41892588baa4da8c2f627d98


I think Memcached is used in php for Players.dat and Worlds.dat load cache
I found when loading a world or a offline player data , php will lag in performance in disk read .
Not every server use SSD , if we can catch high-freq useing files to memcache . The server won't lag at I/O things .


Hi,
  how can i write code for multiple Files attached to a single Document..

$file = EchoSign\Info\FileInfos::createFromFile('demo.pdf');


I find name and package Current very confusing would prefer other recognized name for format, that will simplify common understanding of what format is using where and when, especially when we changing what is _current_ next time.


> but this wouldn't be required than since one could simply do
>     t = function(a,b,c) {
>       return customTransformer(origT(a,b,c))
>     }

Hmm...?
It would still be required because `origT(a,b,c)` would always return a `join()`ed result string.


Use `take` instead of `.first/.last` when ordering isn't needed.


I'm not able to run the code on my side since I do not have a corresponding setup locally. But I can confirm that it corresponds to the disscussions we had by phone. Some stakeholder should try to run the code and confirm that it works.


At the moment the device always sends the temperature in Celsius, but it exist the possibility that in the future devices sends the temperature in Kelvin. (Was a pending update)

If the device connects to a Gadget configured in Kelvin, we would receive values in Kelvin while sending the elements in Celsius.


Please make sure all the methods are defined in the same style, for example you say `Create new Stack` but `Returns ...`


Implemented stamina usage.


Is `CFG_TA_FLOAT_SUPPORT` not needed anymore?


if I do that, yeoman shows a diff on the .json file when committing files to disk  because the .json file has a 2 space tab and yeoman wants to write it with a 4 space tab. I think I could do this.fs.write(..., null, 2) on the file to prevent this but it seems a little weird...


Why? By default the "override" option is activated. The behavior will stay exactly the same.

But you can turn override mode off and then get the Ant-like behavior:

```xml
<configuration>
    <override>false</override>
</configuration>
```


Саша в последнем вебинаре попросил эти элементики сделать все-таки ссылками. Переделай если не сложно)  и class лучше задать sort-up и sort-down а то просто up вообще не понятно о чём )))


we always return array than. So basically we support 2 modes, `array` or `string`. `string` mode is default, `array`-mode allows better post processing.


No, it should not be hard coded.  @wtholt, this needs to be en.yml

Andrew Cates
Ruby Development Manager
Biomedical Informatics Center
Medical University of South Carolina

From: <owner-sparc_request@musc.edu<mailto:owner-sparc_request@musc.edu>> on behalf of robertjamesmiller <notifications@github.com<mailto:notifications@github.com>>
Reply-To: sparc-request/sparc-request <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Thursday, January 28, 2016 at 11:03 PM
To: sparc-request/sparc-request <sparc-request@noreply.github.com<mailto:sparc-request@noreply.github.com>>
Subject: Re: [sparc-request] deleted ask a question and feedback buttons, added contact us button along with contactform model and controller with specs (9e4c8ab)

@wtholt<https://github.com/wtholt> @amcates<https://github.com/amcates> Did you all intend to hardcode the REDCap survey into the "Feedback" button? Or, should it be moved to the locale file en.yml?

—
Reply to this email directly or view it on GitHub<https://github.com/sparc-request/sparc-request/commit/9e4c8abf13398abdd505cd1773f9bf7a74b391ec#commitcomment-15752245>.


Found it a bit confusing. Would assume that RecordFormats is actually holder/factory of all available formats with some current format and not vice versa.
Something like:
public static Format DEFAULT_FORMAT = new FuturamaRocksFormat(); in RecordFormats and then we can reference it by using RecordFormats.DEFAULT_FORMAT


This interface is not formatted


i think `when {@code stack} is null` is simpler and better actually


я бы сделал это двумя разными span. Из соображений единообразия, и чтобы управлять ими из css было удобнее.


ok.  :-)


can we please extract it from constructor


Thanks, I'll fix.


yes that was my question in the PR :smile:


@joao-manywho state could be null here so you may want to put a || {} when instantiating state


h3, p, вместе с кнопочками Подробнее и Забронировать лучше обернуть в один блок, т.к. они вместе одним куском находятся. А в другой блок обернуть ту часть которая справа, где про рейтинг. И первый кусок мы потом сфлоатим влево, а второй - вправо.


There are 649 usages of data-target across the HTML of the HMRC internal repos, mostly prototypes. Would this cause a conflict? Perhaps not. But you could you consider something slightly less generic, like "data-password-target".


We try to keep the Debian version the same as the upstream one. We've not actually released since bumping to `2.9-0` so you can just put this there.


@pannous See the PR notes; apparently pip doesn't honor this and reject it in the way I thought.

https://github.com/fabric/fabric/pull/1416#issuecomment-170086930


don't see why we need it here, called only in `initialiseNewStoreFile` under condition `getNumberOfReservedLowIds() > 0`


Should be ASSERT here?


I think you should explain this `fully update better` see the [example](https://github.com/codenvy/che-core/blob/2f6667b4a147ea1c229722b74c98f2b5c346e27b/platform-api/che-core-api-workspace/src/main/java/org/eclipse/che/api/workspace/server/spi/WorkspaceDao.java#L43-L55)


I certainly wasn't planning on changing it w/o understanding it.

I have no way to confirm. I can't find any page that references these tokens (I searched for `tradnl`+`radstr`+`pronun`).

I presume you can take a partial subset of a pull request (i.e. everything up to but excluding the notes commit).
If not, let me know, and I can lop this commit off...


What was this required for?


java doc


I have a preference to use data attributes instead of CSS classes to control state, just to keep class solely for styling. However, I know we're already heavily into using ".js-" prefixed classes with JS. Just throwing in my two cents. :-)


そうなんです！チェック早いー笑


@zabbal: The patch was rejected, see issue #716.


My initial guess was `transistor`, but it doesn't fit in context, so I gave up...


Ну, конечно это будет не так реализовано, тут будет иконочный шрифт, советую пока переверстать это в ul.stars > li , и li будет по количеству звёздочек.
Либо же figure.stars>span.stars-4


Эти кнопки надо перенести наверх, как я писал в комментарии выше.


don't used anymore


This line can be deleted


```scala
scala> import scalaz._, Scalaz._, effect._, IO._
import scalaz._
import Scalaz._
import effect._
import IO._

scala> val action1 = for {
         _ <- putStrLn("Hello, world!")
       } yield ()
action1: scalaz.effect.IO[Unit] = scalaz.effect.IOFunctions$$anon$4@149f6f65

scala> action1.unsafePerformIO
Hello, world!
```


Wrong


Please start a thread on the forum to discuss, this commit is nearly four years old and the code has likely changed since then.


According to the javadoc you wrote please add `@Nullable` to the tags argument.


Wow, that "aprMapPath + @" is a master piece, good job Laz!


Pushed up a cleaned code to remove unused variables warning error messages.


I've been thinking about this for quite some time too, but not yet been
able to dig deep.
Thanks for looking into it.

From the reports I've seen, it seems sometimes an invalid ID is provided.
Some reports are like this:

>java.lang.IllegalArgumentException: accounts Record ID
-9223372036854775808 does not exist in the db

Which as you rightly suggest is the value of Spinner.INVALID_ROW_ID.
(hadn't realized this before).

I'm also not sure how we arrive at this condition. The Spinner is always
initialized with an adapter.
(might have something to do with the OnItemSelectedListener I added to
mTransferAccountSpinner to handle multicurrency stuff - but I'm not sure)

So even though we're not sure how this state is arrived,  I think your code
suggestion would fix the issue.
Could you make a fix to hotfix/patches and test it in single-entry mode?

What worries more are the cases (and there are tons of them) where we get
exceptions for DatabaseAdapter.getID() with valid looking UIDs and IDs.
I think these cases have to do with list refreshes and non-existing
(deleted) accounts/transactions. But I'm not sure. Anyhow, that is a
separate issue from this one.

On Thu, Jan 28, 2016 at 7:08 PM, Àlex Magaz Graça <notifications@github.com>
wrote:

> Hi @codinguser <https://github.com/codinguser>! I've arrived here from this
> crash report
> <https://fabric.io/gnucash/android/apps/org.gnucash.android/issues/5620ba73f5d3a7f76bd6cc13>.
> It seems lots crashes go through this code path.
>
> I was trying to understand the condition under which the application
> crashes, but I got confused with the change in the conditional. Could you
> explain it? For me, it would make more sense something like this:
>
> if (mUseDoubleEntry && transferAcctId != Spinner.INVALID_ROW_ID) {
>
> Even so, I still doubt about this. getSelectedItemId() should only return
> INVALID_ROW_ID if nothing has been selected in the spinner. However, if
> it has items, I don't see any way to arrive at this condition. I'd say it
> could only happen in the case the spinner was empty but, again, I'm not
> sure it's possible with a sensible accounts structure.
>
> What do you think?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/codinguser/gnucash-android/commit/93cc38df92f2ef957218d74642c3de89706c016e#commitcomment-15740876>
> .
>


What if `skipCount` is < 0, i can see that `IllegalArgumentException` will be thrown, please add it to the javadoc and document


NoStoreHeader is closed to NeoStoreHeader, would propose to rename all `no` related classes to Empty_something_ or Blank_something_


И гостиница, и мотель, и аппартаменты для нас в данном макете идентичны, их вёрстка состоит из однотипных элементов, и даже .psd макет называется hotels.psd, поэтому у всех трёх будет общий класс .hotel, а уже внутри него, в абзаце, в котором у тебя тип - мотель, обернуть слово Мотель в span.hotel-type и всё.
Или если хочется чтобы было ещё логичнее, то надо эти блоки называть не .hotel, а, например, .element, тогда у него будет span.element-type в котором будет написано что это: гостиница, отель, или апартаменты.
Но общий класс должен быть одинаков, потому что вёрстка не отличается, и значит css Будет один и тот же.


Скажи, плз, почему здесь нужен именно Provider, а не сами Limits?


I think this is also Wrong


Та же история с .apartment что и с .motel - нужно заменить на какой-то общий класс.


Seems like name is passed twice (expand, which is ok) and as a Map


Perfect.


oh right, I'll refrain from increasing the version.


Можно сделать nav.main-navigation, чтобы в css класс использовать, а не тег


The ironic thing? This ended up not even being used, as the video timing actually starts before the first cut.

:laughing:


May I suggest using something like `npm install --save --save-exact pkg` to avoid the issue of having "hats" in the versions. You might even create an alias... :new_moon_with_face:


Логотип это ссылка на главную.


Seems like name is passed twice


uhh


looks strange, we probably want to get some value from header? like: <T> value()?


У header  main и footer должна быть либо общая центрующая обёртка (.container), одна на всех ( в барбершопе так нельзя было сделать ) или нужно внутри каждого из них ставить центровщик .container.


This still needs to be addressed. We need 2 more relations (they'd have 2 distinct URIs)


Центровщик тут, или после открывающего body


hmmmm, it seems that we can define the icon exactly in the implementation and avoid this interface.


CC @SergeyZhukovsky for testing these.


Закрывающий / надо убрать. Этой картинке лучше дать класс .banner  - Это такое стандартное слово для похожих блоков..


Build would stop with error when building on the RPi without it, a pity I didn't keep the complaining log but it was quite long.


Лишний отступ у .first-reason, он внутри media-block на том же уровне вложенности что и header, значит количество отступов должно быть таким же.


Technically Redux works in IE8 but, just like React, it assumes you have an ES5 shim. Any way, we are dropping IE8 compatibility after React 15 comes out because it drops it as well.


This is a BC break when using the DomCrawler. The behaviour have been changed (the tests have been modified). I used to make this kind of request:

```php
$crawler = $this->client->request('GET', ''/secured/login', array(), array(), array(
    'HTTP_HOST' => $this->container->getParameter('my.host'),
));
```

Know I need to provide the full URI (with path for environment) or there is a best way to do the same as before?


I think that even uint8 or char would do ^^'


> Out of curiosity was it submitted upstream, I can't seem to find it, but it might be that my gh-search foo fails me?

Submitted and rejected because it displays a secrets on the console.


Very good job, thank you Jc!


Возможно, уже не нужно. Попробую без.


Why don't you liek `const`? =)


Why `locals` changed from `query.locals` to just `locals`?


@jreback: the comment from @cel4 is about the wording
`ImportError: C extension: No module named numpy not built.`
Clearly, `module == 'No module named numpy'` and substituting it makes confusing message "No module not built".
You could just print the message from the caught exception as is and then print your added explanation.


Do we realty have a `bool` value constants?


we never create destination queues (except for audit & error)


Looks nice.
Question / Remarks:
 - F/R in the first module stands for Forward/Reverse?
- Maybe modules could be a bit more separated ?


Updated the method, had forgotten to change the method to use the new parameter name (fieldId).


Almost :D


Closer, but still not exactly that ^^


hm....is it correct?


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#Gii8r-fbT_a5SOMOr-NVZg


Setting Focusmode on TFocusMode.ContinuousAutoFocus (without your workaround) did work like a charm at my place


Вспомнил. Экземпляр этого класса нужно создать во время вызова configure в модуле guice. Можно ли это сделать без использования Provider - не уверен, в других таких же местах тоже передаем Provider.


I've got one remaining itch, what do you think about https://github.com/pimusicbox/mopidy-musicbox-webclient/issues/146 ?


Why have you removed that? You should be able to provide custom transport service via registry reference. Please revert that commit.


mental note to self: host must do this, not transport


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


Why did you remove all the `autorelease` methods in this file? Everything should work like before.


это будет фоном для promo-block а здесь должно быть изображение слов на прозрачном фоне


there is no way.
It's been coded by @simonbrandhof but it's not used


rename to message pump, for consistency with other transports?


I used tis naming just because this class contains the singleton, but I don't save nothing locally from this class. I have to change the naming.


# Just  do whatever you want


nice catch


java script has been applied to the details page only
jquery has been applied to edit page
the jquery folder contains the jquery code as well as the edit page


I just checked : 4 spaces is the new indentation rule for entity files. So this is actually a good thing to reindent it :smile:


Web service is missing. I still suggest to keep the code as the feature will come soon.


Can you revert this commit and use this https://github.com/nxcm/nx_device_wingtech_wt88047/commit/ed5f5093569c4a385ca5a2a3726cee4975850b19


Why do we have both peek & receive? Mind that each operation you do costs money, peek is not required


testing the first call to `addReceived` does not guarantee that multiple successive calls give the right result.


I just realized that like the state color info the parent info is also quite irrelevant within the context of this specific task. I suppose there is not much point in removing it now though.


Hi guys,

with all due respect, but this is unacceptable.

This API change breaks clients that rely on the old semantics of the API (i.e., the "description" field being plain text). Silently changing API semantics is just not very nice.

It's also totally unnecessary IMHO: You could have just done it the other way around, and have the new field be "descriptionHtml" or something, so that clients that _want_ the new feature can use it if they want to, and not break the old ones.


Careful with removing the init sleeps without thorough testing, they are occasionally necessary to allow the server to process everything on fresh restarts with players waiting to join, otherwise the comm menu has a chance to throw errors and not initialize correctly.


Here is the log i'm observing: https://builds.apache.org/job/Red5-client/512/console


spacing is off.


AzureStorageQueues


# Not that you should add your API KEY in gradle.properties


spacing is off


The Device Identification isn't enough for a hashcode of DeviceAuthorization. The Organisation and Function Group should be included.


Works for me: https://travis-ci.org/rstats-db/DBItest#L447


I think a comment why we need to do this bash trick would be good.


Name Dispatcher, just to be consistent with other transports


Здесь что-нибудь поменялось по сравнению с SimpleGrizzlyWebServer?


assume address only, maybe with @ namespace name, but we should not pass connectionstrings (with username/pwd) around in the future


Good point, I'll change the members to readonly.
Concerning the naming, no, the point is to give the information that the message was sent to the persistence and should be flagged accordingly.


Можно выкрутиться через requestInjection

https://github.com/google/guice/wiki/AOP#Injecting_Interceptors


Fred and Caleb,

This looks great! I noticed a couple of minor grammatical mistakes in the introduction. I'm using the Github GUI, so should I just make the corrections on my local copy and "sync" the changes to github? Also, where should I submit my author bio? Thanks!

-Jake

________________________________
Jacob Wes Greene
PhD Student
Department of English
University of Florida
jacobwgreene.com<http://www.jacobwgreene.com>

________________________________
From: W. Caleb McDaniel <notifications@github.com>
Sent: Thursday, January 28, 2016 9:29 AM
To: programminghistorian/ph-submissions
Cc: Greene,Jacob W
Subject: Re: [ph-submissions] Change AR lesson slug, fix jekyll build errors (8413326)

@fredgibbs<https://github.com/fredgibbs> @jacobwgreene<https://github.com/jacobwgreene> I fixed some errors that were causing the site to fail. There's now a live preview of this lesson:

http://programminghistorian.github.io/ph-submissions/lessons/intro-to-ar-development

You should be ready to create an issue for the lesson now, including that link so reviewers can easily find it. I changed the lesson slug to make it shorter, but feel free to change back.

-
Reply to this email directly or view it on GitHub<https://github.com/programminghistorian/ph-submissions/commit/841332686d5189b27e7041836c5b1fb51e1b5d4b#commitcomment-15734041>.


I'll add that to the read.me

2016-01-29 14:24 GMT+01:00 pbhofstede <notifications@github.com>:

> Setting Focusmode on TFocusMode.ContinuousAutoFocus (without your
> workaround) did work like a charm at my place
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pbhofstede/ZXing.Delphi/commit/b4aaaef70f7d0b5ddb9bd21e24b5c8d4a8c9ca7d#commitcomment-15761525>
> .
>


@PocketKiller N++ FTW.


Doesn't this break wire compatibility for xml users?


In finally managed to get a clean run without any docker hub related errors.

It looks like the changes are working so we could re-tag v1.3.0 release.


Смотри, у тебя на этой странице есть 5 похожих блоков с причинами(.reason). Всем им можно дать общий класс .reason.
Два из них отличаются цветом фона. Им можно добавить дополнительный модифицирующий класс, например .reason-special, но нам это не понадобится, и я сейчас объясню почему.
Эти отличающиеся блоки причин, лежат в одном ряду с картинкой, в одном случае сначала блок потом картинка, в другом - наоборот.
Я предлагаю выделить блок с картинкой в отдельную сущность, назвать её как-нибудь вроде Блок С Медиа Содержимым, т.е. .media-block.
И затем ты в css просто напишешь для .reason { ..какие-то стили.. } а потом для .media-block .reason { ..переопределишь цвета.. }
Картинку внутри .media-block можно даже в figure не оборачивать, я думаю.

Ещё у тебя есть три однотипных блока с опциями, с фичами, называй как угодно, но у них много общих стилей - значит у них должен быть общий класс. Например - .feature.
Единственное чем они отличаются - разными картинками наверху, но эти картинки будут css-фоном сделаны, т.к. они оформительские, поэтому их можно просто задать через css. Для этого каждой Фиче добавляешь ещё по классу: .feature-housing   .feature-food  .feature-souvenirs и в css у них будет только background-image: url('img/название_картинки_№x.png'); отличаться


ok, thanks guys


Это не configureJerseyServlet, а скорее registerWebappContext.


Новое только в configureJerseyServlet


Was detected by Intellij :)


spacing is off


Since I use the finder at multiple places, I can't inject it. The Finder has an additive state when filters are added.


ok


This needs to be Receive, queue/peek/lock = receive transaction


.media-block не для всего контейнера, а для блоков где причина+картинка, в комментариях дальше почитай. Этот общий можно назвать .promo


Whether the message should be persisted is not `ZmqTransport`'s responsibility, and if not told otherwise, it assumes the given message was not sent to the persistence.


s/.././


> Since I use the finder at multiple places, I can't inject it. The Finder has an additive state when filters are added.

As said before, I can't use the injected Finder. I will remove it from the constructor.


переименую


В мире где жильё и еда попадают в категорию развлечений - мне было бы жить очень грустно :)


spacing


Почему жильё h3 а еда и сувениры h4 ? Ну, не в филосовском смысле, а как элементы на этой странице?


Картинка - закрывающий слэш не нужен


#150 :
Here we could autoswitch the "active" instance to the "correct" local one.
I mean that if the current active server is the A one, and the android device is connected to B's wifi, we might want to switch to server B.
(but we need to keep in mind that the user might want to change something in home A)


what is it!?


You need to change the href in the head to "/favicon.ico" or "http://urn1350.net/favicon.ico"


@kmob2
+1


we should continue to use the serializer defined by the user, this will blow up for existing xml users that have messages on the queue


In general, check all lines of code for spacing. There are a lot of inconsistencies and I only took the time to specifically note a few of them.


我最近把 R 接口的 cppjieba 版本从 v2.x.x 更新到最新的 v4.4.x 。在 Appveyor 跑 Windows CI 的时候发现，CI 上 Decode 函数会报错，但是我本地不报错。

后来对比 CI 环境和本地环境，主要是 session charset 的不同 本地是 CP936， CI 是 session charset: ISO8859-1。

我把 Windows 环境切换到 美国 的语言环境，就能重复这个错误。

后来对比切换前后的差异，主要就是这个新的 split 函数。

我把旧的 split 函数体替换新的 split 函数体 用在 v4.4.x 上，中文环境，英文环境下不同的  session charset  里都不会报错，均能正常运行。但是，使用新的 split 函数用在 v 4.4x 时，只能在中文环境里运行。

Linux 和 Mac 下没有这个问题。

我是在 MinGW 下编译的，MSVC 下不清楚会不会有类似问题，我没有在 MSVC 上编译过。

编译环境不会影响结果无关，与运行时有关。也就是我用新版的 split 在英文环境下编译，在英文环境下 split 的结果会有问题，同一个程序，我切换到中文环境就能正常运行。

很奇怪的一个现象。不过 Windows 编码问题很多了，见怪不怪。。。各种不标准。

在 R 接口里我打算先用旧版的 split 函数，在 Windows 兼容性稍微好一些。


actually there are some other things to consider:
* if a field is missing (such as "service"), the  yeoman will want to add it
* for some reason, yeoman wants to remove the entity javadoc (not the field ones).


https://github.com/Funwayguy/DOTV_Tweaks/commit/7d3361bd6b1933aca085f18920b708081a206b26#diff-4aae6ee9e56086d9b8d687b780890c79R32

This may be causing a the problem @Darkosto was having - perhaps adding 0.25 to the Y height would prevent any sort of glitching when moving to the end location. It looked like he was spawning either below or in the main platform.


Unfortunately, the `sigint` program doesn't work on our CI machines: it just hangs, probably because the tests are not run in a console, so I have disabled the `signal.ml` test again. (commit af2669f0a031af44fbb12769976380d174ebf00c)


I made one line comment regarding the hashCode for DeviceAuthorization. When that's fixed, the pull request can be merged.


Better use
```rust
if let Some(command) = c.running {
    ...
}
```
instead. It's more idiomatic.


Oops sorry I thought this was in `PersistentTransport` - my bad


ой все


112259371_UnitHeaterHWCoilUASizing (Nigusse) - x86_64-MacOS-10.9-clang: OK (2051 of 2052 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small 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.95%25-yellow.svg)


Imo changing existing comets to take a map and abandoning positions args is
a fine step to take now given number of users we support.

On Fri, Jan 29, 2016 at 7:03 AM Matthew Forsyth <notifications@github.com>
wrote:

> I agree that this would be much easier to understand, and that an array of
> anonymous "params" is not ideal. However, the pre-existing functions we're
> referring to here all take positional parameters rather than "named
> parameters" (param map). To name the parameters in the config file we would
> need to change the signature of all existing recipe functions - which would
> in turn invalidate any existing config files that call them. Then this
> change would no longer be backwards compatible with existing config.clj's.
>
> If that's not a big deal, and the self-documenting nature of naming these
> params in the config data file seems worth it, we can make that change.
>
> Alternatives:
> -keep the params positional in the recipe functions but make them named in
> the json file... seems like a bad idea because it implies that the order of
> keys doesn't matter when it really does.
>
>    - just add a comment above each "params" line in the sample data file
>    indicating what the params are
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/twosigma/satellite/commit/f82f621233fa10657ce68183788b9211eccf525c#commitcomment-15760062>
> .
>


Dear Williams,

Thank you for the word-smithing, but I deliberately wrote "number" instead of "integer", because - as my example shows - range works also with floating point numbers.


As long as `dr` doesn't have a problem with this being included then I'm okay with it.


The two lines `stderr` and `stdout` are not necessary. They are inherited by default when using `spawn()` in the end. See [documentation](https://doc.rust-lang.org/std/process/struct.Command.html#method.new)


some variation of "as many as we can find" might be stronger here than just "any".


:smile:


Well, in many cases it's not really the **complete** details, as there's often another level of indirection via morph…


Yes, having this issue using OAuth with 1.2.0 as well. loadstart event is not fired
Thanks


cool! nice and clean :shower:


Not a big deal but it should be "principal".


> Even though I originally though I want this package to handle WPP files as well, I am not so sure any more as WPP is no longer being developed by the Wave Propagation group at LLNL and most of its features have been implemented in SW4 (SW4 now handles grid refinements like WPP did but only for elastic, not visco-elastic waves). I therefore see little point in investing time in that.

I never knew or used WPP but yeah, from reading the docs I also had the impression that it was kind of an replacement.

> Both sw4py or pySW4 are fine by me.

Ok, cool, I really think that a name change makes sense. And it doesn't mean that those other helper routines (geo etc.) you have in there can not stay. If you have a preference between the two, let me know..


This line seems to be unnecessary ;)


That's not really true — in most cases we just rebuild at scheduled intervals.


It makes command unsuitable on CI, because exec won't report exit code of the command.
Perhaps it would be less confusing to stick with one command and update documentation accordingly?

    RAILS_ENV=test bundle exec rake spec:javascript


@leplatrem @n1k0 @natim @almet made the new changes, any other changes I should make?


можешь тут делать перенос строки, в тех же местах где в макете, браузер просто воспримет это как пробел, а тебе будет проще ориентироваться в коде. Принудительный перенос браузер сделает внутри P только там, где есть <br>


this is awesome


imo it should be vice versa. return an error code but disable the test


'Wikidata' doesn't capitalise the 'd'. It's also not really "_wikipedia_'s wikidata" — it would be good to find a way to say this that is correct, without being needlessly pedantic too.


Про это я уже писал выше. Делать списком или нет - на твой выбор, но элементы списка должны иметь тот же класс, что у аналогичных элементов в медиа-блоках, где тоже указаны причины.
Но если делать списком - тогда обёрточный див тебе не нужен, можешь сделать просто ul.reasons-list например


Please tell @CSurieux which recent actions of Lombiq were breaking for you.


shouldn't this use ```<<``` instead of  ```+``` since this is a stream?


You need to add the other attributes here, like volume_type / volume_size etc


Should i put in semipassive?


This block got the indentation messed up - at least in the variable identifier alignment.


This makes it sound like it's a _bad thing_ that different sites might have different types of data, rather than just an entirely normal thing that isn't really an issue in most cases at all. I'd go more for just a simple statement that often different sites have different types of data.


haha, drunk late tipying at css meetup.

I'm sure that's why there is an aligment problem.
@pixelastic vengeance :D


OK @tnn1t1s sounds good, I'll do that :+1:


очепятку нашёл))


Need more arguments here also


it already disables the test.
It's not returning an error code so tests don't fail.


Well, no, not really. We're not trying to be complete either. There's lots of data on these other sites that we're not going to include either — which is why we want to give you the URLs or IDs for those sources too…


I like this change!


why not USE_CREG? :-)


Do we need it to the interface?


@liewegas , s/if/else if/

please cherry-pick new patch from https://github.com/ceph/ceph/pull/6536. thanks


ну правда же, можно перенос сделать.
Единообразие: вот до этого ты этот отступ не делала, а тут почему-то сделала. Я имею ввиду что у тебя содержимое абзаца на 2 пробела сдвинуто вправо. По-хорошему надо также сделать везде, где то этого есть аналогичные абзацы.


@ahshahid we need to rationalize changes to Spark. Are you sure this is a Spark bug? If so then need to file a bug in Apache Spark and propose this fix via a PR. The snappy-spark repo is now public (and will make it a public fork soon).


Why are we using Kenya here?

If we want a good worked example, Australia (Representatives) is probably the best, as it uses so many different source.


good catch!


specs := make([]*ec2.SpotFleetLaunchSpecification{}, len(user_specs)) # And move this line down so it compiles :)


should be ```nullptr```


for i, user_spec := ....


These were removed in a later commit


Через композер устанавливается без этой правки


specs[i] = opts # By pre-allocating the array at the top you're more efficient, as you don't need to re-allocate it as it grows


я бы назвал по смыслу - .hotels-search


откуда знаешь что она на index.html будет отправляться?))


Are we at all concerned that we will not be notified of breaking changes? It seems like taking our own path leaves us at risk.


player.holdAlert.isDefined


... change this `wait_with_output()` to `wait()` to avoid capturing stdout. Should work.


Regenerate deck button added to the action bar (the circular arrows icon).


Перед POST запросом нужно всегда делать GET.
В ответе на GET запрос могут устанавливаться куки, которые могут участвовать в дальнейших проверках на сервере.


commit message: absoloute -> absolute


Maybe you want %+v here instead of %s?


No unit test of massSubmit ?


TeamCity Xbim :: Xbim Geometry :: XbimGeometry [Build 136](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=729&buildTypeId=Xbim_XbimGeometry_XbimGeometry) is now running


По-моему здесь никакой Jersey специфики не используется, и можно просто передавать HttpServlet mainServlet, а не JerseyHttpServlet jerseyHttpServlet, но как знаешь.


commit message: please place a `Fixes <full URL>` note in the body so the issue gets auto closed


Thanks!


Maybe a // FIXME comment here, as I can see this being something you'd *not* want.


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


Probably because I'm defining "NOT_USING_CREG"?


bracket alignment


Correct. Take a look at [this](http://stackoverflow.com/questions/958908/how-do-you-reverse-a-string-in-place-in-javascript/) for an alternate approach.


Разметка: с этими кнопульками будет хитрость, Надо input[type=date] и последующий button обернуть в родителя с классом ну например .date-with-button
Button - ему не нужны name и value, это не поле формы, оно никак с ней не связано, а эти атрибуты нужны только для формы.


All fine to me


Add spot_request_state


Oh, you can do this in /rg flag... But I'll add these commands.
/rg flag, allow you do anything with code of region permission. See https://github.com/FunCraftNetwork/NuclearPlugins/blob/master/NuclearRegions/src/com/fcmcpe/nuclear/regions/permission/RegionPermission.java#L11-L23


it's unsound to default to white here. If `user` didn't play in this game, then the white player shouldn't be blamed.

The function must return Option[JsArray]


lgtm!

None of my comments are critical, ship it :)


This is named `value` as in one more idea from https://github.com/LeaVerou/awesomplete/issues/16821#issuecomment-176750717


Перепутала, for у label должен ссылаться на Id а не на name


It's better to declare all variables explicitly using the keyword `var`


in fact it's not related to the fs method, it comes from the fact that we use the --regenerate flag on the entity gen. I think there is a bug with javadoc.


112259371_UnitHeaterHWCoilUASizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2089 of 2090 tests passed, 0 test warnings)

 * 6 tests had: AUD diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small 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.95%25-yellow.svg)


I think it is "descendants" (see http://grammarist.com/usage/descendant-descendent/)


this will drop the whole socket, which you probably don't want to do.
what you'd actually want to do is unsubscribe from `/centres` but we don't currently present a way to do that in the API


Тут тоже хитрость: будет общая обёртка, ну скажем .number-with-buttons, и внутри будет button + input[type=number] + button.
Между атрибутами тегов должен быть пробел.


Prof. Bruno, esse commit do Gabriel deletou o arquivo de outro aluno: class-03-resolved-AlexBrasileiro-Alex-Brasileiro.md


implementation and prototype have to either both be DLLEXPORT or neither for msvc compatibility


@gervasek  Please review this thank you!


Feeling great with **O(n)** time!


Картинки это одиночный тег - закрывающий слэш не нужен


Why do you `App::make` it instead of just calling `new Finder`?


EDIT: I fixed this madness by realizing that I'm just not experienced enough to deal with those cross browser issues. So I took the liberty to make advantage of your audiostreamsource.js. Now it seems to run smooth on each browser...

Best,
Christian


@matcracker pensandoci bene credo che ora funzioni xD


コメント行の扱いはどうなりますか？


This is not copying the text if I delete the text from first textbox. Need to fix this.


Если убирать упоминание jersey, то внизу их еще несколько. Тут можно было бы, наоборот, всю фабрику перенести в модуль, он кроме ее вызова все равно ничем не занимается.


Thanks!


112259371_UnitHeaterHWCoilUASizing (Nigusse) - 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)


timeout keyword argument was introduced with bt 0.8.4 which is the default on up-to-date quicklisp repositories. I've checked, it wasn't there one release earlier.


На вебинаре Саша предложил сделать это через fieldset (вместо section) и legend (вместо b), если есть время и силы посмотреть что это - можешь переделать, так будет более семантично. Нет - так приму.


谢谢反馈，能否把CI上Decode函数报错的具体信息贴一下给我呢？这个问题太诡异了。


humm, it's the style formatter doing things being the scenes....


@salmanm Just a hack for now, I'm looking at it all tonight to figure out why we are missing the message


Вот тут могли бы быть и imput'ы, но вы будете такую штуку делать ещё, так что пока норм


Will do.


you forgot our brain are so good at fuzzy matching, and fuzzy typying.

haha.
_Pattern matching powaaaaa_


Nice :clap:


ups sorry, fixed.


Thanks, i upgraded to 095f3d5 yesterday and noticed now this indeed works :)


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


ok pls make an issue / PR for this


@joto - good stuff outlining intentions etc


Привязывайтесь к названиям, а не к URL, например, так: https://regex101.com/r/gR9rX0/2 (упрощённый пример)
В этом случае:
a) код проще для понимания
b) вероятность изменения URL больше, чем названия