@Bladrak I tried that as well, and same result:

```
% make test

 ============
 Vows Results
 ============

  ✓ OK » 34 honored • 0 broken (4.235436s)

 =============
 Code Coverage
 =============

 ✗ tc_aws.loaders.presigning_loader.py  ••••••••••••••••••••••••••••••••••• 71.4%                27, 35, 37, and 3 more
 ✗ tc_aws.loaders.s3_loader.py          ••••••••••••••••••••••••••••••••••••••• 78.3%            15, 31, 32, and 2 more
 ✓ tc_aws.aws.storage.py                •••••••••••••••••••••••••••••••••••••••• 81.7%           58, 59, 67, and 8 more
 ✓ tc_aws.aws.connection.py             ••••••••••••••••••••••••••••••••••••••••••••• 90.9%      21
 ✓ tc_aws.storages.s3_storage.py        •••••••••••••••••••••••••••••••••••••••••••••••• 96.2%   79, 82
 ✓ tc_aws.__init__.py                   •••••••••••••••••••••••••••••••••••••••••••••••••• 100.0%
 ✓ tc_aws.aws.__init__.py               •••••••••••••••••••••••••••••••••••••••••••••••••• 100.0%
 ✓ tc_aws.loaders.__init__.py           •••••••••••••••••••••••••••••••••••••••••••••••••• 100.0%
 ✓ tc_aws.result_storages.__init__.py   •••••••••••••••••••••••••••••••••••••••••••••••••• 100.0%
 ✓ tc_aws.result_storages.s3_storage.py •••••••••••••••••••••••••••••••••••••••••••••••••• 100.0%
 ✓ tc_aws.storages.__init__.py          •••••••••••••••••••••••••••••••••••••••••••••••••• 100.0%

 ✓ OVERALL                              •••••••••••••••••••••••••••••••••••••••••••• 88.8%
```

If you copy/paste this test onto your machine, do you see failures?


@sandeshpl also, i dont see any changes to any files that would show your work done.


looking from the outside, w/o knowing much about how the security is handled in couch, I don't have a clue about *how* the security restriction is achieved. I see some dictionary assignments, but that's all.

maybe the problem can be solved by renaming the method to something more informative, I dont know, something like restrict_couch_access? ensure_security seems to vague, and I don't know what 'member' really means.

I was just asking for a more newbie-friendly explanation of what are these dicts configuring exactly, where and how. (ie, what's _database.security).

btw, is the assignemnt in the last line of the function really needed?


I installed oh-my-fish just before the repository was taken down. I was completely ignorant of any problems with copyright, or any other issues. Now that oh-my-fish has been taken down, `omf update` (for example) no longer works, and it's like being a wrongly implicated prisoner in my own shell!

The worst part is, without even the oh-my-fish README accessible anymore, let alone issue tracker, and no mailing list (or chatroom etc) linked off the https://github.com/oh-my-fish org page, users like me may be locked out of being able to look up or ask for help on basic things about what to do next.

@github Surely there must be some better way to handle these DMCA takedown requests that doesn't leave innocent users like us out hanging in the cold?

/cc @bpinto @bucaran


Well, I've put binding.pry in a couple spots. When I run just the **digest_email(user)**, it works. I need to figure out how to iterate through the user/email combos.

Is logic reasonably sound with this?

```
class DigestMailer < ApplicationMailer

  default from:"no-reply@theusthread.com"

  def digest_email(user)
    @historical_events = HistoricalEvent.monthy
    @user = user
    @people = @user.people
    mail(to: user.email, subject: "This month's events")
  end

  def digest_email_batch
    @user = User.all
    @user.each do |u|
      digest_email(u)
    end
  end

end
```


Looks good, flip these so that people can write a parser even if they do happen to include moment though.

```
if (opts.parser) {
  date = opts.parser(opts.field.value, opts.format);
}
else if (hasMoment) {
  date = moment(opts.field.value, opts.format);
  date = (date && date.isValid()) ? date.toDate() : null;
}
```


@fernandog This line ^

self.consoleLogging is false when running as a service (-d or -q), so it never got to the restoreDB call below!


Thanks, fixed!


remove the extra espace between from and err


But `kzd_voicemail_box:owner(Box)` will only return 'undefined' or a JSON object, no?


For anyone else who installed oh-my-fish and didn't realize it, it looks like the repo gets cloned to ~/.local/share/omf, where you can still access the README etc.


This looks good. Can you explain whether the `else if` is required in this circumstance as opposed to a regular `if`?


This is going to allocate a new delegate each time. You could make the `UpdateDateValue` method static and create a static cached `TimerCallback` delegate instance, and then pass `this` as the state.


Hey https://gitter.im/fishline/fin


What kind of value does the expression `number % 2 === 0` result in? What are its possible values, and when do they occur?


Instead of disposing the timer and allocating a new instance the next time it's needed, why not allocate a single timer once and then just use `Change(Timeout.Infinite, Timeout.Infinite)` to "stop" it and  `Change(_timerInterval, _timerInterval)` to start it?


@bucaran thanks! Finally a lifeline!


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/82075686


@jab
An email from @bucaran requesting the process to be terminated has already been sent. It shouldn't take long before the project is back 100%.

Please check back tomorrow, we hope to be pushing more code already. :)


测试提交


@bpinto Great to hear, thanks to you too! In the meantime, would now be a good time to add a link from the https://github.com/oh-my-fish org page to a Gitter chatroom, mailing list, or any other external place people in the user community can find each other so we don't get stranded again?


viết thêm các thủ tục đi


I like what you were trying to do here! Finding the center dynamically instead of hardcoding the indices would be a good challenge.


Don't worry. You will not be stranded again. We all swear to not fight over copyrights again. Basically we reconciled.


Thanks!    This should make the non-English wordnets easier to use and cite!


Yeah I thought about that too, but this path isn't performance critical so I wasn't convinced it warranted the indirection.


Again, not perf critical and nulling out the timer makes it convenient as a check for its state (is it null? need to create and start it then)


Instead, do something like this: https://github.com/VanessaE/pipeworks/blob/5822f60ba9cbebe45c1f82fa06dc93b747ba71a6/common.lua#L149


这种需要自定义包头，不交给inet去做。


Also we want to ensure there is no code running at all on our end if the app is idle. No timer at all is just that one step further to that end :smile:


Strange as in RPI I run as daemon.


I just read this old post, from 2 years ago when Joona explains how he tuned the king PSQT
to get the following result at LTC
LLR: 2.95 (-2.94,2.94) [0.00,6.00]
Total: 5890 W: 985 L: 857 D: 4048

https://groups.google.com/d/msg/fishcooking/1u30btUpa9M/cyhX_Udn1-IJ
https://groups.google.com/d/msg/fishcooking/1u30btUpa9M/A74Jai1wxuYJ

For endgame, he assumed symmetry between rank_1 and rank_8, and so on
Since then, there were a few tuning sessions, but the general pattern is still visible...

Good luck with your new attempts !


You can use this `[pluralize(number, "Star"), number]` .


That's good to hear, but oh-my-fish should still have its own Gitter in any case. @bpinto?


did you confirm this worked chris? Doesn't work for me. I'm now getting:

exceptions.TypeError: can only concatenate tuple (not "str") to tuple


Can someone please help me with this? I have 660 lists that I would LOVE to use on ANKI, but I can't transfer them. oohthemountainswinagain[at]gmail[dot}c0m

Thanks in advance!


@mhugent the INSTALL file should be updated to add the new dependency, namely libgeos++-dev.


Can someone please help me with this? I have 660 lists that I would LOVE to use on ANKI, but I can't transfer them. patriciusoenus[at]gmail[dot}c0m

Thanks in advance!


In the forum, @Thorsten made some toughs about the penalty of texture switching.
Maybe we need think about merging some objects in as few texture files as we can.


I changed explaining that this is the lowest access level and that this is achieved by a security design document, followed by a link to couch documentation on that. Seems enough now. :)

>btw, is the assignemnt in the last line of the function really needed?

lol! that's kind of magic, but I liked it. See: https://github.com/djc/couchdb-python/blob/master/couchdb/client.py#L378


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1360 of 1360 tests passed)

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


Should be `sdscmplex(range->min,range->max) > 0` :-)


Like above


This should be a file, use `false`


Does this need to be recursive?


Ah, the explanation is because annotator.js is mentioned explicitly while the extensions are an implicit rule.


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


> The gitter room will be restored when the repo is back up too.


Closer the first time! Thanks again for all the feedback.


why remove copyright?
Copyright (c) Microsoft Corporation
MIT License


The [0, 0] line converts from a 1x1 matrix to a float - which is more useful.

Sorry about the late reply - just saw this now


@B--B thank you so much mate, Scite kept saying that without the double backslash it was an error lol


OK, let's leave it as is. I'll keep it mind for future commits; thx for pointing it out !


是


replace needInit by isInitialized for the member name and accessors


> mount points on Unix don't show up as symlinks either, so I doubt that this would be a good idea on Windows.

Heh... The difference is that MSys2 *does* report those junction points as symlinks ;-)

```sh
me@work $ mkdir foo
me@work $ cmd //c 'mklink /j bar foo'
Junction created for bar <<===>> foo
me@work $ ls -l
total 0
lrwxrwxrwx 1 dscho None 26 Sep 25 09:44 bar -> /c/test123/foo/
drwxr-xr-x 1 dscho None  0 Sep 25 09:44 foo/
```

But when I mount it, MSys2 does *not* report it as a symlink:

```sh
me@work $ rm bar
me@work $ mkdir bar
me@work $ mount c:/test123/foo $(pwd)/bar
me@work $ ls -l
total 0
drwxr-xr-x 1 dscho None 0 Sep 25 09:46 bar/
drwxr-xr-x 1 dscho None 0 Sep 25 09:44 foo/
```

The reason is easy to see: the construct created by the `mount` call is actually not a reparse point at all. It is a mapping, and the directory has to exist already, and it keeps being a directory.

In fact, Git -- being a MinGW program and having no clue about MSys2 -- does not see the mount point. It sees the directory, and it handles it as such.

> Mounted volumes don't necessarily have a drive letter, and getcwd() is supposed to resolve all symlinks. Are git, msys2, perl etc. prepared to deal with absolute paths of the form \\?\Volume{GUID}\some\path...?

I have no idea ;-) But let's cross that bridge when -- and if! -- we need to. ;-)


I think I was trying to target imports as well, but given that flake8 itself already reports about that... your changes make more sense


@mieliep I would change this to:

`<h1>The Putter School Philosophy <span class="u-weight--normal">“Do what you want”</span></h1>`

Reason being that by default, an `h1` is bold, so I would leave it bold and rather specify the part that is not bold with a span. The result will be the same.

Also the inverted commas should be `“ ”` instead of `" "`


:+1: :beers: :beer: :beers:


Well, I let you do the 200th !

----- Mail original -----
De: "Holger Brandl" <notifications@github.com>
À: "mpicbg-scicomp/tissue_miner" <tissue_miner@noreply.github.com>
Cc: "etournay" <etournay@mpi-cbg.de>
Envoyé: Vendredi 25 Septembre 2015 11:53:15
Objet: Re: [tissue_miner] removed margin cell in get_cell_properties (3857c76)

I wanted to to do the 100th commit! Good work anyway. :-)

---
Reply to this email directly or view it on GitHub:
https://github.com/mpicbg-scicomp/tissue_miner/commit/3857c76695f71629745a541d9802fbbde62a8c4a#commitcomment-13434404


TeamCity Xbim :: Xbim Analysis :: XbimAnalysis [Build 14](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=198&buildTypeId=Xbim_XbimAnalysis_XbimAnalysis) is now running


ok thanks


as


I don't know


TeamCity Xbim :: Xbim Analysis :: XbimAnalysis [Build 14](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=198&buildTypeId=Xbim_XbimAnalysis_XbimAnalysis) outcome was **SUCCESS**
Summary: Running Build time: 00:00:59


is it mandatory to replace RM_DB, with db/rm ?
This can imply many transitive changes


@mieliep The blockquote is cool, but it's not really producing the same result as the design, e.g.

VS

Thinking that maybe just using the `box` pattern might be better, and maybe the text inside the box could be `h2`, to mimic the design?


@JoakimSoderberg Yeah, note that I haven't changed anything, just added a better timing function that I haven't tested properly yet :-) I think I should switch to RAW actually to avoid dven slight skew corrections. What do you think?


Thisi -> This


Any reason this function is defined in another function? It seems like a pure function, no?


Trailing spaces.


Cool. This fixes CI!


@jite Yea `CLOCK_MONOTONIC_RAW` is probably a better choice in my opinion :)


Closes #592


@mmajko said that it will, because some users think that the current state is final...


This sentence no verb.


I think it should be max(2, numberOfCoreAvailableLocally-1) (one kept for the scheduler process)
Additionnally, we should by default use Multiple-Nodes in a single JVM for the LocalInfrastructure to avoid exhausting RAM with java processes.


Same here, the GEOS_CPP_LIBRARY variable (only) appears after flushing my build directory.


DBに保存する時に変な状態で保存されてたのかな？保存で失敗して欲しい気はするけど・・・日付けはDateTime型でセットするのが懸命だと思います。


@mieliep There's an id on this section called "andrew-putter", which might have been copied from the landing page. I think it's a better id for this section since "whos" should technically have an apostophe, but you can't have an apostophe in an id. So I would remove the id from `l-section--short` and replace the `l-section` id as well as the `l-section__action` link with "andrew-putter" - also for consistency with the landing page.


The entire point of the Name mechanism in NEST is to make comparisons fast by mapping strings to integers. So it should be `if ( name != names::gaussian )`


Oh, okay.


See above.


rax retry tests


You should pass a reference so you do not need to guard against null-pointers.


As a private member, `name` should be `name_`.


There's an extra space in the commit message after the first description-sentence ;)


Tuples should not be forced to s-c-s users IMHO


The tag is always generated belonging to class `paramtest3`, even if it doesn't exist.

From a SystemVerilog standard point of view it is possible that `paramtest3` is not a class, but that is not possible to determine that without first identifying all types within the compiled code and then searching for `paramtest3`. Since this is currently not possible in ctags, I've implemented the support for what I most commonly see: out-of-block declarations.

I still stand on the position that it is not our place to verify the code, we simply need to provide as much information as we can possibly gather :)


It's kind of bad practice to catch an exception without logging it.. something like a logging call or sys.stderr.write() would be cool. The cost of having faked robustness by catching everything will be a programme that behaves odd and is hard to debug... but at least being somewhat verbose makes it easier to see through whenever things fail for unforseen reasons... for instance because local_settings is broken or undefined or someone overwrote the `getattr` builtin or whatever :)


Oh no, I must have been doing the readme tired late in the night. I think I could merge this :+1:


typo, resgisterTarget should registerTarget


This should not be handled like this, but rather through `@ConfigurationProperties`


Why not:

```
base_plot_options = copy.deepcopy(base_plot_options)
new_plot_options = copy.deepcopy(new_plot_options)
return dict(base_plot_options, **new_plot_options)
```

This seems like unexpected behavior:

```
update_plot_options({}, {'use': 'me'})  # outputs {}
```

Maybe it's to prevent bad keys, but then we're just silently failing, I'd rather we explicitly strip or error on them.

Also, it's not really and *update* this way since you're destroying the original object reference.


Updated title.
More descriptive title for the landing page; changed inner pages' title.


Yes, see comment in the needinfo-ed bug.

Regards,
Martijn

On Fri, Sep 25, 2015 at 9:37 AM, mermi <notifications@github.com> wrote:

> Yea you are right here, but @mwargers <https://github.com/mwargers> this
> test has a failure too https://pastebin.mozilla.org/8847471 there are
> some tests failure and it's not related to this PR
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mermi/gaia/commit/a3c88a373688bbe69855cde49cbc46bdd5d9cc2d#commitcomment-13431739>
> .
>


not too worried about it though :) just reviewing out loud


This whole thing feels very intrusive. Can't it be decoupled _via_ an interceptor or something?
All this code does not belong in `BindableProxyFactory` IMHO (which I already complained about doing too much in a previous PR, now it would have yet another responsibility!)


Yes, that looks better.


name the variable with fullname _skinningAttribute


It's probably not required. The idea was to put all things related to the Database (hsqldb) in the same directory. What are the transitive changes you are thinking about?


Reviewed by Dietmar Zabel.


If you want to share the workload you also need to give up some control. (Yes, it's hard, I know.)


Gets removed later so considering as fixed


reactor leaks occurring again

                $1.00;0.22;0.25$ <(|, %3, a) --> d>? {?: 4} ((<%1 --> %2>, %2, task("?")), (<(|, %1, %3) --> %2>, (<StructuralDeduction --> Truth>)))


Thanks for the reply
I've try it and i'm afraid this didn't help at all. I'm still getting a plain box following the given RGB color code, but still no text with alpha 0.9. Changing the alpha propriety didn't shows any changes.

many thank for this app, i'm using it with max/msp in order to bild an interactive interface for research purpose, and it's very helpfull. I'm trying to share my modifications on Github as clean as I can, so that you can use them.


Не, как-то оно может рабоать не так как надо. Чем не подходит `@Bean(destroyMehtod)`? (См. http://docs.spring.io/spring/docs/current/javadoc-api/org/springframework/context/annotation/Bean.html#destroyMethod--?:)


I had to loosen the policy. This only happens on backward inference, if a question derived task contains a "%" we can skip it for now.


conetents


header


Ага, решили сделать по-проще и объединить конфиги? Работать будет, но лучше бы не смешивать разные штуки в одном классе. Надо было оставить как есть, инжектирование `DataSource` это правильный подход.


hrittt


It seems we need this "cat" param on mobile: ADEN-2476


What about killing the queued staging builds (for Linux)?


its more important to get NAL working at first. NAL1-5 is now fine.


who cleans it up after test was executed?


cool, this section is much clearer :)


Зачем этот метод? Выглядит абсолютно бесполезным.


Done!

GitHub must've made for me when I started editing the README, then aborted when we decided to go a different way. Thanks.


Предлагаю убрать `hibernate.id.new_generator_mappings` -- во-первых, неясно зачем она понадобилась. Во-вторых, судя по доке, это значение по умолчанию.


Check this sentance


How about Mockito.anyObject() ?


:goat: `Deprecated` not `Depreciated`


... тогда нужно было ссылаться из Liquibase на JpaConfig?

25 сентября 2015 г., 13:20 пользователь Slava Semushin <
notifications@github.com> написал:

> Ага, решили сделать по-проще и объединить конфиги? Работать будет, но
> лучше бы не смешивать разные штуки в одном классе. Надо было оставить как
> есть, инжектирование DataSource это правильный подход.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AleksSPb/Spring-demo-App1/commit/3626640ff11a03b533fe461a756056d9a78d3531#commitcomment-13434803>
> .
>

--
Всего наилучшего, Пархоменко Александр


:goat: see https://github.com/plotly/python-api/commit/38762b42653c05cc39a1f142ca7acc54e30c836b?diff=split#commitcomment-13434846


reservedmapkeyLineageId is never sued


>тогда нужно было ссылаться из Liquibase на JpaConfig?

Достаточно было просто заинжектить `DataSource` и не важно кто и где его создает.


How did you get Doze to work properly with this cofig?


I believe, the check for leading `.` and trailing `(` should happen in the specific rules. Sometimes it is not a method we want to catch, but a tool.
What do you think?


As discussed, we'll keep this one, but all the next migraions will be done in Java


This seems strange to have a dynamic default based on the machine? Mind adding a comment?


Err, I mean windows


emsadenhamraksh


q


Sounds good


For now, I added the sleep so that host get discovered successfully (by loading kernel in ram) and get registered with server. Once its registered with server, we can see that in webUI.


How to use it for popup
Because when i call popup and call it, $(window) will doesn't work


Unfortunately, you cannot make a value class constructor private. It'll just ignore the visibility modifier :(


Убрал
(раньше почему-то Liquibase выдавал ошибку по поводу автогенерации значения)

25 сентября 2015 г., 13:23 пользователь Slava Semushin <
notifications@github.com> написал:

> Предлагаю убрать hibernate.id.new_generator_mappings -- во-первых, неясно
> зачем она понадобилась. Во-вторых, судя по доке, это значение по умолчанию.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AleksSPb/Spring-demo-App1/commit/d0b3c3c154b6a30d41f29925d84a735b8275e00c#commitcomment-13434839>
> .
>

--
Всего наилучшего, Пархоменко Александр


These test stubs are already covered in `test_discoveryrule.py`
https://github.com/SatelliteQE/robottelo/blob/master/tests/foreman/ui/test_discoveryrule.py


Вообще говоря, этот коммит не вписывается в версию `1.1.х`, потому что здесь ломается обратная совместимость. Методы `LoadJevixConfig` и `LoadQevixConfig` удалены, а метод загрузки конфига `loadConfig` перенесён в отдельный класс, который не является модулем системы (соответственно, нельзя привычным способом расширить его методы в плагине). Это повод еще раз напомнить о не очень удачном подходе к версионированию. Все виды изменений сваливаются в кучу, добавляя палок в колёса юзерам.
Я пишу это не просто так. Дело в том, что у меня использовались методы LoadJevixConfig и LoadQevixConfig для добавления своих тегов в конфиг типографов в плагине. Я его не опубликовал в каталоге потому, что для него нужна была правка в https://github.com/altocms/altocms/pull/734. Так бы он устарел еще в момент выхода.


I'd say: "name, description or :tag"


kaaaaanaaaaa


Sure, but I need to finish testing the rest of the release before I start on doc updates.


yeah better, I thought it could be confusing because you need to specify ":tag" and for name or description, without the ":". In any case, yep.


This is causing an exception:
Exception loading list from XML: System.MissingMethodException: Cannot create an abstract class 'Verse.Genstep'

I would have dug deeper, but I don't have your modified source


hello


much nicer


if skin contains the attribute, name it. this._skinAttribute, it's easier to understand in the code later. same thing for morph. name it _morphAttribute


NodeSandbox.documentCharset = NodeSandbox._parseDocumentCharset();


Suggestion applied for the number of nodes.

@fviale Do you know how to enable this feature?


static _parseDocumentCharset () {


remove


你好，该项目为公司私有资产，请删除该项目，谢谢


```
if (config.require) _.extendProto(FacetClass, { require: config.require });
if (config.configSchema) _.extendProto(FacetClass, { configSchema: config.configSchema });
```

is the same as

```
_.extendProto(FacetClass, _.pickKeys(config, 'require', 'configSchema'));
```

at the low price of always calling the function that happens only on facet class definition (that will happen anyway as we want schemas for everything probably)


What's the issue with using aliases in enums? It seems like bindgen converts those just fine.


Thanks


NodeSandbox.documentCharset


Looks good.


import NodeSandbox from '../../sandbox/node';


select datepart(year,OrderDate) as YearOfOrderDate , count(SalesOrderID) as CountOfOrders
from AdventureWorks2014.Sales.SalesOrderHeader s
group by datepart(year,s.OrderDate)
order by datepart(year,s.OrderDate)


"PREPARING the facilities for inter-punctuation rules. (if it turns ou…
…t we don't need them, this can be reverted)


plzzzsirrr


Should be named Morphing without the attribute, like other attribute:
ShadowReceiveAttribute -> type: ShadowReceive
BillboardAttribute -> type: Billboard


I know. But the arena alloc flag prevents them from freeing


That's why I have a separate dtor in that table......


Давай улучшать:

Во-первых, добавим полные имена классов. Вернее не классов, а ключи для локализации. Например, для вместо `NotBlank` должно быть `org.hibernate.validator.constraints.NotBlank.message`.

Во-вторых, вместо `{0}` и `{1}` начнем использовать более внятные названия. Кстати, сходу не знаю чем заменяется `{0}`. Если узнаешь -- расскажи :)

См. также http://docs.jboss.org/hibernate/validator/5.1/reference/en-US/html/chapter-message-interpolation.html


Done.


Apesar de ter tido uma idéia similar na época da confecção da zztrim, fui motivado pelo artigo do Rubens Queiroz de Almeida no site do Dicas-L ( http://www.dicas-l.com.br/arquivo/removendo_ou_substituindo_espacos_em_branco_consecutivos_com_o_comando_tr.php )


test


The `.swp` file has nothing to do here!


Why not using ```DOCROOT.'vendor/autoload.php';```? :-)


I've tried on aosp and works well , without this don't know why but even if the updater script is correct crashes at line 5 column 11 with bad syntax error...! :-P


How well does this work compared to `<input type="checkbox" checked="{{hostChecked::change}}">`? If we leave out the `::event` syntax, we only get one-way, host-to-child binding for the native element. This might not be what you need. Just checking.


Demo


This is gnarly. Do we have an upstream bug open about this or was one opened and closed as this behaviour being intentional?


first issue of new... in this case i prefer a less defensive speach :-)
we are here to stay!


i wanted to stress the fact that they were both proposed and accepted... in the general case merged would have been better...


Looks good to me :)


## This is a markdown file


i leave the second part as is: i want to suggest to the user, that creating a file with a set of value and passing it to the script is a better option than adding lot of arguments.
escaping parameters for the command line is really no fun.


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


Build failure in chromium-dev: http://hydra.nixos.org/build/26256724


@wlbragg @thevirtualfer The thing is there was no space on `floats.png` to add this new texture. As for merging things into as few textures files as possible, that's fine with me but I'd like to do it only once: e.g. each livery have their own `float.png`, so that if we would manage to find some space for this new texture I would have to add it to several different `float.png` files, then optimize them and push them, but if later you guys will decide to also change the merge the floats with another texture, the same work will have to be redone. As I have absolutely no clue of what kind of work I will have to do with the textures once @thevirtualfer finishes his 3D remodelling, I'd say not to do anything right now and wait for it.


提交项目


rax retry tests


Just have to dig into a better solution for OSX since it doesn't seem to implement clock_gettime and after the backend unification (almost anyway), OSX uses sys_posix too.


Не ожидал, что кто-то уже успел на эти нововведения завязаться. Проблему осознал. Но, чтоб не откатывать назад, предлагаю такой путь ее решения: в классе `ModuleText` добавить два публичных метода - `CreateTextParser()`, который будет возвращать объект парсера, и `LoadTextParserConfig()`, который собственно подгружает в парсер конфигурацию.

Тогда будет возможность плагином как собственный парсер подключать (кроме Jevix и Qevix), так и конфигурировать парсер дополнительно.

Понимаю, что это потребует исправлений для неопубликованного плагина, но они будут небольшими и саму проблему собственно решит


Maybe you meant assessment here?


mmm... no, it's not about filling configuration fields, but fields in the .sla...
i mean: name, address, ...
if you have better name for it, i'm not 100% happy with the term "fields"... but i think that people who are in need of command line scripts understand what i mean :-)
changed the first part of your suggestion in "with multiple arguments" since it's more about passing values than options.
i don't think that scripts that get passed to the command line will need lot of options (except for the pdf version, but in that case, there are so many options that one should really create a file with the one she choses)


this is a cfor change


Hmm, chromium builds fail so often – do people test the builds before pushing? I don't think it's good to just push untested stuff into master.


test


`objevt` -> object. Beginners will likely not be aware that working against arrays requires a little more work. It may be worth mentioning https://www.polymer-project.org/1.0/docs/devguide/properties.html#array-mutation


done


Now works with existing bundles without recompiling.


does this happens in real world? if not we might want to lower the log level to log the unexpected behaviour.


I didn't want to change the interface to `InfoRequest#receive`, so I'll leave it as is. We'll think about changing it if/when we remove the `raw_email_data` argument.


Thank you @eliotsykes, I'll include it in the next release : )


There is a constant for `framework`.


@h2o64 Nope, it requires CM hardware remapping in frameworks/base


Not really an issue at all: this could be constructed as:

```
<template is="dom-bind">
  <todo-app></todo-app>
</template>
```

with your `td-model` and `td-todos` being children. I assume for styling or simplicity opting for the pattern you used makes sense.


Please [prefer css selectors over xpath whenever possible](plone/plonetheme.barceloneta@e337ed0138e40299586cc25cca8ec4c842787a36). So at least:

    <after css:theme-children="head" css:content="/html/head/link[@data-bundle='basic' or @data-bundle='plone' or @data-bundle='plone-logged-in' or @data-bundle='diazo']" />
    <after css:theme-children="head" content="/html/head/script[@data-bundle='basic' or @data-bundle='plone' or @data-bundle='plone-logged-in' or @data-bundle='diazo']" />

I don't know whether the `or` xpath operator is convertible. @lrowe @smcmahon, do you?


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


I've just pushed my changes. Please review this commit.


bra namn!


How about making this generic method ```public static```?


Rename "values" argument: right now it's both values and attributes. Maybe "valuesAndAttributes" is a good name (:
Same for _values


Rename to note the attribute returned is the server one


+ Added Firewall


Ah, but if you change the code so that it doesn't ignore blank string values anymore, as I think you do in the commit this is fixing up, then you're changing the behaviour of the code as a whole, unless the default `rejected_reason` param of `InfoRequest.receive` is also changed to `nil` - requests will get `EventLogInfo` params with a blank value for `rejected_reason` whereas before `rejected_reason` was not in the hash - which I think is the desired behaviour. So something needs to be done to resolve that.


Maybe rename as getModelAttributes?


I think the usual convention is "@since 1.9"
http://www.oracle.com/technetwork/java/javase/documentation/index-137868.html#@since


Äh. The proofs are remembered even when you reload or restart the browser, as they are stored in `localStorage`. But they are tied to the browser, and there is no way to access them from another browser or to transfer them.


I would call the other constructor from here. But maybe that is just me. :-P


Imo the last one should be an ```if``` because it's possible (at least I think so) that ```m_currentFile->HasVideoInfoTag()``` is true, but also ```item == VIDEOPLAYER_TITLE```.


Final


I was going for the simplest formulation with the weakest guarantee here -- since people might be using private sessions and whatnot. The whole point being not to overcomplicate this section, since technically experienced users will probably have reasonable expectations anyways.


I didn't think as far as to make this lazily as well. Way more consistent this way, good catch!


I think this is over-cautious. In most cases, they can come back and have the proofs still there; just warn them that in some situations (including incompatible updates to the code) the proofs can get lost.


@gideon-maina haha this commit is awesome... dyu think that fix is viable?


This upgrade breaks my vanilla webpack setup, due to postcss5 using es6 (thus potentially introducing some unresolved babel dependency). Need to fix version 0.18.0 now, using "latest" broken.


done


Nice job, man!


I mucked this one up, forgetting the **.tid** extension. This file was replaced by [Contents.tid](https://github.com/hegart-dmishiv/my-TW5-dev-instance/blob/master/wiki/tiddlers/Contents.tid) in [commit #df23b33](https://github.com/hegart-dmishiv/my-TW5-dev-instance/commit/df23b33d8257e203efc61555ff7f06356e2672e5).


Are you sure about this? We're possibly gonna break user scripts. Isn't it better to keep compatibility for some time, and then remove the option from arguments at all?


@mieliep I just added a "primary" class to the anchors on the white backgrounds. That class overrides the color properties implemented in the `layout.less` file.

This is where that class comes from: https://github.com/putterschool/styleguide/blob/master/src/assets/less/base/utilities.less


Well is the same style it is in other places of OTRS, so I try to keep it consistent, but yes we can change it to $DynamicFiledName, what is is provably the most correct term, we also need to change the Label in such case


The reason for generating the little bit of CSS in the controller class is to add theme aware colors only.
This is since different themes have different colors for info, danger, warning etc..

The 'real' CSS implementation for the chat stuff is in the stylesheets as can be seen here.. https://github.com/aliasgherlakkadshaw/mofowallet/blob/master/app/styles/main.less#L291

So if there is a need to change the css always do it in the main.less not in the position above.


Done


@alexanderlz
Have you already released the plugin to Jenkins repo? If yes, you should not change artifact IDs


Hmmm... It gives me `master`. Can you elaborate a bit, please?


Because of the scope and since the variable is no changes it should not cause any problems, but for sure might be co fusing


See comments on https://github.com/mysociety/alaveteli/commit/50f22253b7f960ec05465ad5a6973df67e99e449#commitcomment-13435425


Fixed in 0c11567.


Not exactly sure here if I'm getting this right, but shouldn't this return an error if either `old_file.path` or `new_file.path` is `NULL`? Currently it does only return an error when both are `NULL`. If I am mistaken I'd clarify the error message (e.g. "git diff header lacks old and new paths").


* Bugs:

* [ ] while thinking about your commandline option, maybe we need to have two iaas subdires, one where we use commandline (as you already have done it) and one where we use api.

* [ ] reread who split works
       (d[i]["Name"],d[i]["Used"],d[i]["Max"])   = value_name.split("|")

* you can do all three splits in a single split


I've tried to clarify this in 5fe343cc731103c4cca640be84534ccb58f03db3. Feel free to refine further, though!


Can you also add the actions for opening and closing files in NB, and opening and closing NB.


Сообщения не передавались по цепочке, если отсутствовал метод в `then`. Мда, половину логики простых промисов приходится прописывать. Всё ждал, пока добавят в стандарт, а там молчание: https://esdiscuss.org/topic/cancelable-promises-proposal


Excellent work, thanks!


emitiharabr


ishopUa


Ah, so I think the change you mean is https://github.com/mysociety/alaveteli/commit/9f0ba42bb5871cf6cf90bd6e2b6ca37e2631569a#diff-5f7dfe1185cac97cbc9f9dc356abce7cL1500?


done


Bugs:

* reread how split works, see comment i made to Guravs similar code


alalaa


<?xml version="1.0" encoding="UTF-8" ?>
<!DOCTYPE ZOOshop>
<ZOOshop name="Зоомагазин Айболит">
	<title1>
	Рыбка Гуппи
	</title1>
	<description>
		<species> Вид: <speciesname> Гуппи </speciesname> </species>
		<price> Стоимость: <pricenum> 450 руб. </pricenum> </price>
		<color> Цвет: <colortype> Красно-оранжевый </colortype> </color>
		<ages> Продолжительность жизни: <agesamount> 5лет </ages>
		<description> Гуппи сравнительно легко поддаются селекции, что привлекает опытных владельцев аквариумов. </description>
	</description>
	<options>
		<option>
		Выберите животное
		<!--
		Затем скажите продавцу
		-->
		</option>
		<option>
		Подойти на кассу
		</option>
		<option>
		Оплатите
		</option>
	</options>
	<title2>
	Аквариумные черепахи
	</title2>
	<description>
		<species> Вид: <speciesname> Черепаха </speciesname> </species>
		<price> Стоимость: <pricenum> 1000 руб. </pricenum> </price>
		<color> Цвет: <colortype> Коричнево-зеленый </colortype> </color>
		<ages> Продолжительность жизни: <agesamount> 10лет </ages>
		<description> Эта аквариумная черепаха популярна благодаря несложному уходу и своей красивой окраске. </description>
	</description>
	<options>
		<option>
		Выберите животное
		<!--

		Затем скажите продавцу
		-->
		</option>
		<option>
		Подойти на кассу
		</option>
		<option>
		Оплатите
		</option>
	</options>
	<title3>
	Белка
	</title3>
	<description>
		<species> Вид: <speciesname> Грызун </speciesname> </species>
		<price> Стоимость: <pricenum> 1500 руб. </pricenum> </price>
		<color> Цвет: <colortype> Тёмно-бурый </colortype> </color>
		<ages> Продолжительность жизни: <agesamount> 9лет </ages>
		<description> Имеет удлинённое тело с пушистым длинным хвостом, уши длинные, цвет тёмно-бурый с белым брюшком, иногда серый (особенно зимой).  </description>
	</description>
	<options>
		<option>
		Выберите животное
		<!--

		Затем скажите продавцу
		-->
		</option>
		<option>
		Подойти на кассу
		</option>
		<option>
		Оплатите
		</option>
	</options>
	<title4>
	Хомяк
	</title4>
	<description>
		<species> Вид: <speciesname> Грызун </speciesname> </species>
		<price> Стоимость: <pricenum> 300 руб. </pricenum> </price>
		<color> Цвет: <colortype> Серый </colortype> </color>
		<ages> Продолжительность жизни: <agesamount> 3года </ages>
		<description> Длина тела варьируется от 5 до 34 см, хвоста от 0,7 до 10 см.  </description>
	</description>
	<options>
		<option>
		Выберите животное
		<!--

		Затем скажите продавцу
		-->
		</option>
		<option>
		Подойти на кассу
		</option>
		<option>
		Оплатите
		</option>
	</options>
	<title5>
	Морская свинка
	</title5>
	<description>
		<species> Вид: <speciesname> Грызун </speciesname> </species>
		<price> Стоимость: <pricenum> 1300 руб. </pricenum> </price>
		<color> Цвет: <colortype> Разный </colortype> </color>
		<ages> Продолжительность жизни: <agesamount> 4года </ages>
		<description> Несмотря на название, не связаны с семейством свиней, так же как и не являются морскими животными и не связаны с Гвинеей.  </description>
	</description>
	<options>
		<option>
		Выберите животное
		<!--

		Затем скажите продавцу
		-->
		</option>
		<option>
		Подойти на кассу
		</option>
		<option>
		Оплатите
		</option>
	</options>
	<title6>
	Кролик
	</title6>
	<description>
		<species> Вид: <speciesname> Заяц </speciesname> </species>
		<price> Стоимость: <pricenum> 3000 руб. </pricenum> </price>
		<color> Цвет: <colortype> Разный </colortype> </color>
		<ages> Продолжительность жизни: <agesamount> 7лет </ages>
		<description> Кролики - прекрасные домашние животные. Они чистоплотны, хорошо обучаются, интересны, послушны и общительны. </description>
	</description>
	<options>
		<option>
		Выберите животное
		<!--

		Затем скажите продавцу
		-->
		</option>
		<option>
		Подойти на кассу
		</option>
		<option>
		Оплатите
		</option>
	</options>
	<title7>
	Кот
	</title7>
	<description>
		<species> Вид: <speciesname> Кошачьи </speciesname> </species>
		<price> Стоимость: <pricenum> Варьируется в зависимости от породы </pricenum> </price>
		<color> Цвет: <colortype> Разный </colortype> </color>
		<ages> Продолжительность жизни: <agesamount> 13-15лет </ages>
		<description> Котята здоровые, веселые, активные ждут своих маму и папу.  </description>
	</description>
	<options>
		<option>
		Выберите животное
		<!--

		Затем скажите продавцу
		-->
		</option>
		<option>
		Подойти на кассу
		</option>
		<option>
		Оплатите
		</option>
	</options>
	<title8>
	Собака
	</title8>
	<description>
		<species> Вид: <speciesname> Псовые </speciesname> </species>
		<price> Стоимость: <pricenum> Варьируется в зависимости от породы </pricenum> </price>
		<color> Цвет: <colortype> Разный </colortype> </color>
		<ages> Продолжительность жизни: <agesamount> 13-15лет </ages>
		<description> Щенки здоровые, веселые, активные ждут своих маму и папу.  </description>
	</description>
	<options>
		<option>
		Выберите животное
		<!--

		Затем скажите продавцу
		-->
		</option>
		<option>
		Подойти на кассу
		</option>
		<option>
		Оплатите
		</option>
	</options>
	<title9>
	Хорёк
	</title9>
	<description>
		<species> Вид: <speciesname> Куньи </speciesname> </species>
		<price> Стоимость: <pricenum> 10000руб </pricenum> </price>
		<color> Цвет: <colortype> Темно-коричневый в полоску </colortype> </color>
		<ages> Продолжительность жизни: <agesamount> 10лет </ages>
		<description> Щенки здоровые, веселые, активные ждут своих маму и папу.  </description>
	</description>
	<options>
		<option>
		Выберите животное
		<!--

		Затем скажите продавцу
		-->
		</option>
		<option>
		Подойти на кассу
		</option>
		<option>
		Оплатите
		</option>
	</options>
	<title10>
	Волнистый попугайчик
	</title10>
	<description>
		<species> Вид: <speciesname> Попугаевые </speciesname> </species>
		<price> Стоимость: <pricenum> 2500руб </pricenum> </price>
		<color> Цвет: <colortype> Разный </colortype> </color>
		<ages> Продолжительность жизни: <agesamount> 7лет </ages>
		<description> Красивые и здоровые птенчики хотят дружить в Вами и Вашими детьми..  </description>
	</description>
	<options>
		<option>
		Выберите животное
		<!--

		Затем скажите продавцу
		-->
		</option>
		<option>
		Подойти на кассу
		</option>
		<option>
		Оплатите
		</option>
	</options>
</catalog>


Can you use `PlotlyTestCase` which will maintain sandboxing from test-to-test? It stores and resets all the config/session/blah/blah info.


The patch can be a rename, so you only need one of the paths.


@andyberry88, do you mean to leave these tests commented out?


cheers


Sorry I misspoke; it could be an add or a delete.  A rename should have two (different) paths.


I am a whitespace nazi, and I approve of this commit :+1:


Fixed. I find name `contents` a bit misleading. May be change (mod|item|trait)_contents -> (mod|item|trait)_element?


This test is pretty hard to read through, can we break it up? Also, why do you need to call `tearDown`, why not just allow the test runner to do that.


removed


That commit is contained in master already, so I guess we should also kill the 8k jobs in there, too? However, I don't know what to have on master now instead of that...


`rescue nil` on render is very bad.
just had hard time figuring out an admin problem and it just was a typo on my calculator class, without exceptions because of this.


The open and close are kinda implemented; by logging when snapshots were started and stopped. I'm still looking for a nice way to detect NB open/close/activate/deactivate


I would even make a function to get the schema file in order to prepare the export schema file retrieval


building the ppa's from this version, not sure when i can update next. if theres any uber change i'll update asap.


done


Evidently, [Lua doesn't do hoisting](http://www.luafaq.org/gotchas.html#T3.2), so I've rearranged the functions.


I would rename `region` to `region_id` and `my_region` to `my_region_id` for clarity.


ok mistake by me: it throws ConcurrencyExceptions when using `TransactionScope` but happily works when not using transactions.


@milosstojanovic you're back!!! How's London? :D


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


No, trying to do too many things . Fixed now.


Why is TICK `17`? Is it related to frame rates / refresh rates for 60fps?


Gowtham testing commit comment


> Heh... The difference is that MSys2 does report those junction points as symlinks ;-)

Not always...I just tried ```mklink /j xtemp \\?\Volume{...}\Temp``` (where Volume{...} is a drive without a drive letter). Worked like a charm and MSys2 says its a directory.

> The reason is easy to see: the construct created by the mount call is actually not a reparse point at all.

I wasn't talking about MSys2 mount emulation, but real Windows mount points [1]? AFAICT, with your change, these would be symlinks as well.

[1] https://technet.microsoft.com/en-us/library/cc753321.aspx


properly rebase to master, this commit should not be in pull request


remove trailing whitespace


1.12.1 or 12.0-0? It is quite unusual start versioning with 1. But at the end this is just numbers...


place opening brace in new line


Yes, the logging is common sense driven.

About max_headers, if the admin changes only the headers and wants to reload the new config, your code fails to update what he wants. In other words, that code was executed when it was last enabled but with a different header than what the new config has.


You should probably squash this and previous commit


Нашёл, проверил, нHIS прекрасно работает, получается него/неё.


There is no "8 Recordings" and "9 Recordings". is this intended?


rax retry tests


new line


Guess I'm somewhat confused by the checks in lines 878 to 882, which seem to imply that it is only valid to either pass both `patch->old_path` and `patch->new_path` or none of them.


Added Back


Нет, почему же, если `$oSkin === null`, то условие `!$oSkin` равно `true` и остальная часть выражения уже не будет проверяться.

Отличие от вашего варианта: у вас `$this->Assign('oConfig', Config::getInstance());` будет выполнено, если `$oSkin !== null`, но оставшаяся часть выражения равна `true` (это старый скин). В моем случае `$this->Assign()` будет выполнено еще и тогда, когда `$oSkin === null`. Логика немного изменена (я считаю, что `$oSkin === null` это тоже может быть признаком старого скина), но в обоих варианта при пустом $oSkin ошибка возникать в этом выражении не будет


gunpowder


Debug code?


this thing scares me :)


Pipes halfway done? So… half-pipes? \*scnr\*


I did it again <.<


There is another comment /* TODO ... API 1.9.7 */ left https://github.com/sub3/pvr.nextpvr/blob/dca254bf3480d23a815190fd854ec543e76b4f51/src/client.cpp#L531 You should remove that comment as you just implemented this. ;-)


Half Pipe 3 confirmed


:+1: :100: / :100:  på den


Right.  Those are the values of the `+++` and `---` lines, which may be absent.  But if they *are* present then they must *both* be present.


You shouldn't set class dynamically this way with interpolation. You need to use ng-class directive (you can see examples above). IE doesn't like interpolation on classes sometimes and it's just best to use a directive that handles this for you already.


Ah, thanks for the clarification.


I added these back also


yep


Actually, here and elsewhere, these variables are used once in your code, so they're probably not needed at all.


Done


See: https://our.umbraco.org/Documentation/Extending/version7-assets

... but once the model builder is in core we'd just modify the URLs in the BackOfficeController


done


moved


not that really matters, maybe a compiler warning... but shouldn't coercion be explicit?


I don't get where this class is set. Seems superfluous to me.


the `op->tid` should be always `0` here, i believe.


ok... more than a couple of scaring things, looks ready to go :)


Codename: Pipe 1.5.


cc: @durran @estolfo This is blocking a Mongoid5-compatbile Kaminari release which a lot of people are using.


Actually I think that would work:

    <after css:theme-children="head" css:content="head link[data-bundle='basic'], head link[data-bundle='plone'], head link[data-bundle='plone-logged-in'], head link[data-bundle='diazo']" />
    <after css:theme-children="head" css:content="head script[data-bundle='basic'], head script[data-bundle='plone'], head script[data-bundle='plone-logged-in'], head script[data-bundle='diazo']" />


thx m8


It could absolutely be a letter_opener issue (I did just scan the repo for one). I would not obsess about it just in case it is revealing fake data. Pry wins all. Now, on to testing your emails.


On 09/24/2015 09:46 PM, Agustín Bergé wrote:
> `id_type` to `locality` should be explicit, as not every id represents a locality.
> `locality` to `id_type` can be implicit, every locality should be represented by an id.

I agree with that. However, my rationale to make both implicit was to
ensure maximal backwards compatibility. If a `id_type` which does not
represent a locality is converted to `locality` an assertion is triggered.

>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/STEllAR-GROUP/hpx/commit/aaf5423395f855b6d9f08dc7c8a50e525594ab66#commitcomment-13423687
>

--
Thomas Heller
Friedrich-Alexander-Universität Erlangen-Nürnberg
Department Informatik - Lehrstuhl Rechnerarchitektur
Martensstr. 3
91058 Erlangen
Tel.: 09131/85-27018
Fax:  09131/85-27912
Email: thomas.heller@cs.fau.de


Я бы сделал "королевская кровать", серьёзно.


There's still an easier way to do this... you don't need this line at all and then you can change the below line to not pass in a culture and it will just work.
You need to reference namespace `Umbraco.Core.Services` since that is where the extension method exists.

It's also worth noting that you can just get the culture by doing this:

`Thread.CurrentThread.CurrentUICulture`


I was unsure about awaiting that because wouldn't it mean it'd take some time to boot up? Then again it means once it's booted up forecasts are available instantly.


This doesn't work on Windows - it needs `\` in command path and `.cmd` extension. But I don't know how to solve this nicely for all platforms.

The original `bg('nodemon', './src/server')` works well with `nodemon` installed globally.


On 09/24/2015 09:33 PM, Hartmut Kaiser wrote:
> Ahh, ok - makes sense. However I'm not sure if a `locality` should be an executor type (I admit however the idea is compelling).
>
> We already have the `targeting_distribution_policy` to turn an `id_type` into a distribution policy and the `distribution_policy_executor` to turn any distributon policy into an executor. So I think the code should do something like:
>
>     locality l = ...;
>     targeting_distribution_policy tdp(l);
>     async(make_distribution_policy_executor(tdp), action_type(), ...);
>
> That would also simplify the implementation of the locality type itself.

That's a good idea. I have to think about how to best integrate that
with async and friends then.

>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/STEllAR-GROUP/hpx/commit/aaf5423395f855b6d9f08dc7c8a50e525594ab66#commitcomment-13423437
>

--
Thomas Heller
Friedrich-Alexander-Universität Erlangen-Nürnberg
Department Informatik - Lehrstuhl Rechnerarchitektur
Martensstr. 3
91058 Erlangen
Tel.: 09131/85-27018
Fax:  09131/85-27912
Email: thomas.heller@cs.fau.de


nack. We should check this inside the `data_dir` module.


This line can be removed as you overwrite the value of tag.iClientChannelUid in the next line.


Did you check to see who has access to this function, and who is using it?


use `/home/example/myscript.sh`, to check the `basename` aspect too?


This line can be removed as you overwrite tag.iTimerType value in line 1071


Btw isn't this already handled by the `tempfile.mkdtemp`? I think the function guarantees empty fresh directory writable only by the user.


không được bỏ đi $cat_data, bỏ đi sẽ ko có tính năng upload nếu chru đề ko có chủ đề con


AFAIR you can do it easier by
n_nodes="`cat $PBS_NODEFILE | wc -l`"
and then use $n_nodes in -np


Perhaps add a description like: "Values used to count and index unique instances of this cell throughout the geometry if referenced in a tally's "distribcell" filter"


Would like to test, though: I rather not have two full eXist-db source trees locally… Is there a way for me to include this commit into stock eXist the git way? How do I download the patch, there is no link here?


I would rewrite line 1071 to 1075 like
tag.iTimerType = pRulesNode->FirstChildElement("EPGTitle") ? TIMER_REPEATING_EPG : TIMER_REPEATING_MANUAL; Then, you do one assignment in any case.


Spaces instead of tab here.


Are these examples correct? I thought the accepted types were "x-plane", "y-plane" etc with all lower case and hyphens between words.


Even without `git gui`, it should be easy to avoid adding unrelated files.
Even a `git commit -a` would just commit changes to files that are already part ot the repository, i.e., it would commit everything you'd see when using `git diff`. and not add unrelated files.
To add unrelated files without noticing, you'd need to do something like `git add *.c *.h; git commit -m "commit message"; git push`


Ditto an explanation similar to the one I provided above


is "closei" correct or is that a vim-insert typo?


Thanks! This would help in tracking the actual sessions.


also, try to keep lower-case / mixed-case separated. stick to one or the other


`$HkApply` would be easier to read.


Ah. This commit looks good!

Thanks for taking a look so quickly, Erik and Tom.


After this modification the game crashes here.
![2015-09-25 7 51 35](https://cloud.githubusercontent.com/assets/5671495/10099724/edefa802-63be-11e5-9626-60cb5b39e056.png)


`enable_if_type` is same as `has_type`


If you have a git source base already, you can do -

```
cd $EXIST_HOME
git remote add adamretter https://github.com/adamretter/exist.git
git fetch adamretter
git checkout fop-2.0
```

The last command will switch you from origin/develop to adamretter/fop-2.0. Remember to switch back when you are finished by running `git checkout develop`.


hi i am using this api in my application its works great but i have some concern about how to use "Places Autocomplete TextBox" with this control, i am not find any example related this.


MonkeyPatchedDelete has an __enter__ method that patches numpy.delete with a function that tests for negative indices and an __exit__ method that undoes the monkey-patch. This test failed before the code modification, detecting the negative index for this case.


Same here: use "Close & Keep-Alive" or "close & keep-alive"


On which version did you test this?


А, да, это инверсия моего `$oSkin &&`. Но у меня операций меньше (нет отрицания) и читабельность выше :grinning: Ну ладно.


Hmm, I thought better implicit, then explicit. `__del__` is not always called... If we want to use `__del__`, how about modifying the `data_dir`:

```python
_tmp_tracker = None
class AvocadoTempFile(object):
    def __init__(self):
        self.tmp_dir = tempfile.mkdtemp(prefix='avocado_', dir=BASE_TMP_DIR)

    def get(self):
        return self.tmp_dir

    def cleanup(self):
        shutil.rmtree(tmp_dir, ignore_errors=True)
        _tmp_trakcer = None

    def __del__(self):
        self.cleanup()

def get_tmp_dir():
    if _tmp_tracker is None:
        _tmp_tracker = AvocadoTempFile()
    return _tmp_tracker
```

This should do the same on the module level while you can still safely call the cleanup manually.


Why commented headers? If they can be removed, you can removed them. Otherwise, uncomment them.


Perhaps something like: "The number of entries in the array of offsets for this lattice used to count and index one or more cells referenced by a tally's "distribcell" filter. The size of the array accounts for the number of cells as well as the number of lattice cells where one or more instances of each cell is found."


Ok, so we're all good then since we'll have a scope when running under DTC
transports

On Fri, Sep 25, 2015 at 1:28 PM, Tim Bussmann <notifications@github.com>
wrote:

> ok mistake by me: it throws ConcurrencyExceptions when using
> TransactionScope but happily works when not using transactions.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Particular/NServiceBus/commit/3b5158d30980783a10e3fb70f6fe970e034ac792#commitcomment-13435789>
> .
>


@terrajobst did you take a look at existing solutions like https://github.com/gsscoder/commandline (which is extremely popular) before designing this?


No period at the end of this line, seems like you are using periods elsewhere.


Merci Nicolas!


thks for all your late remark, we've been a bit ravenous on merging this. I'll clean up all the small things :)


This is a conservative fix. An alternate would be `idx[1:][idx[:-1] != idx[1:]] - 1` which would delete only elements with length > 0. But this fix just asks to delete the same thing twice if idx > 0 and it fixes the case where  idx starts with [0, 0, ...]


今はローカルで。


Why were maps removed?


JNM


lgtm


I'll be honest that my recollection of the workings of distribcell with maps and offsets is a little rusty, but I think I need this to index distribcell tallies. That being said, if you were able to run the Pandas DataFrames IPython Notebook after this change, then it must be fine to remove maps.


Why  -1 ?


Стандартная ошибка. Что будет, если будет не 27, а 21 или 15 городов? Задача была в том, что нужно найти объекты с подходящими параметрами, то есть 27 должно быть параметром. Далее то же самое


This was `new int[theModel.groups[i].length`. You forgot the "new" (and the type annotation, but maybe that's optional)
This mistake caused a nice domino effect of problems... including the cause of your JVM crash, too: Since it put a value other than 0 that wasn't a VBO Id into the VBO array... it crashed.


А я бы ничего не менял, один перевод есть и пусть будет. Уже привыкли и пусть, тем более там не big, значит она роскошная, а королевская это банально. Осторожно ИМХО!


Why is the leading comma part of the property value?


This limitation seems surprising and unnatural for users.

This could limit what could usage of types that in public APIs by library developers, as changing them fill slightly could make type not be eta-reduced to class type.

Note that scalac accepts such code.


Смотрю серьёзно занялся делом (:


This does not make sense. This should either read TIMER_ONCE_MANUAL or TIMER_ONCE_EPG, depending on whether the timer is epg-based or purely time-based.


Is there a reason not to use `gulp-nodemon` programatically, without the `spawn`?

```

// ...

gulp.task('nodemon', () => {
  nodemon({
    script: './src/server',
    env: { 'NODE_ENV': process.env.NODE_ENV || 'development' }
  })
})

gulp.task('server', ['set-dev-environment', 'server-hot', 'nodemon']);
```


You are comparing `oldPod.DeletionGracePeriodSeconds` with itself, I guess you meant `differentPeriod(oldPod.DeletionGracePeriodSeconds, pod.DeletionGracePeriodSeconds)`


To repeat my comment from https://github.com/eapache/starscope/issues/88#issuecomment-143096392 - Starscope is a cross-platform tool, not a UNIX-specific one. Doing config placement "right" (e.g. using `%APPDATA%` on Windows, wherever it's supposed to be on OSX) is a lot of work and a fairly contentious decision anyways given that such files are hard for the user to find and edit.

I would note that git still automatically reads `~/.gitconfig` even on platforms that support XDG.


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


Oh yes.


You should not read more bytes than "size", because you would overflow your *data buffer. So make sure that can not happen. Then, bytes_read will never be > content-length.


Can you give full archive of your save dir. and build type. I need to reproduce this.


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


I'd add a newline here


See followup change: 31b4344ced0c3ea8c5de1f3034370b90cb502689.


Whatever we do here, I strongly object to having both conversions implicit as this is prone to unexpected and unwanted conversion paths.


// Пробный коментарий


A check would indeed be good, but I don't find anything in the ALSA headers that is suitable for an `#ifdef` and the PCM formats themselves are enums.

Maybe conditionally enable this through `setup.py?`

Like setting `extra_compile_args` to `-DALSA_HAS_DSD_FORMATS` if  `aplay --version` indicates alsa is recent enough?

Can you look into that and update the pull request? My Linux dev machine is offline right now.


I would not delete this line as it's presense is a clear statement that you know about this flag (it has just been introduced with PVR API 4.1.0) and actively decided to have no special flags, atm.


you can use `spro<long>(str)` from string_algorithms.hpp


// test comment


If I understand correctly, people who use `TakeSnapshot` class directly will have a nice surprise when they realise that it now takes snapshots when the test fails NOT. I'd rather have a new class with a new name and leave the old one as is or even better merge the two


Does not look like Dutch to me :)


test


I always get confused every time I go back and look at distribcell variables. In this case, "maps" was simply the size of the offset array, so it is superfluous.


Restored in SHA 922cf9f85b


nice and compact nil-safe check, I like that :-)


Why do we need the ID in the key? Are we going to have a registrations list which will combine all registration forms, and the column selector should allow the selection of *any* item of the registration forms?!


Fixed.


They are correct according to the code that's there currently. I was going to ask you about this actually. I think they should be lower-case and hyphenated to be consistent. If you agree, I'll go ahead and change them.


Yup, the only one missing, will be done next week. It was a conscious decision, since we do not have any NL wikias that have this extension enabled yet. And the Dutch are pretty good with English. ;)


If you change the view for form 1, you don't want to change  it for form 2.


You have the point.
В самом деле, то, что уже было переведено нормально менять особо не стоит. Она и до 914 была "роскошной".


I would have expected our API group to "".  Does this field not mean what I think it means?


ah, of course


Yes, I think they should be changed to be consistent so that we can rebuild
the CSG tree from the summary file, convert it to OpenCG geometry, etc.

On Fri, Sep 25, 2015 at 8:08 AM, Paul Romano <notifications@github.com>
wrote:

> They are correct according to the code that's there currently. I was going
> to ask you about this actually. I think they should be lower-case and
> hyphenated to be consistent. If you agree, I'll go ahead and change them.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paulromano/openmc/commit/759f96c9e05f6bcc8ab588b93c1ddbda2edb6e9e#commitcomment-13436396>
> .
>

--
Will Boyd
Nuclear Science & Engineering
Massachusetts Institute of Technology
Cell: 423.413.8469
wbinventor@gmail.com


Fel


en general estamos poniendo la llave debajo


The latest Magento 1.9.2.1


To express that this a manual (time-based) repeating timer, you should add PVR_TIMER_TYPE_IS_MANUAL and to be able to enter start and end time and channel in Kodi Timer Settings, also PVR_TIMER_TYPE_SUPPORTS_CHANNELS and  PVR_TIMER_TYPE_SUPPORTS_START_TIME and       PVR_TIMER_TYPE_SUPPORTS_END_TIME


dentro de este if faltan 3 espacios más


size, coming from the user, should be clipped at MIN(size, bytes_left)


This flag makes me so incredibly unhappy.  Also, open an issue to fix this.  It won't work  properly once we enable experimental (which you probably already did).


rax retry tests


Needs update, just remains of c&p


You can't do that!
`$this->getCreatedAt()` return a date like that: `20150917120641.0Z` _(YYYYMMDDHHMMSS.0T)_
I use this code to parse the date: `preg_replace("/([0-9]{4})([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})([0-9]{2})(.*)/", "$3.$2.$1 $4:$5:$6", $attribute["whencreated"])` but you can simply return a timestamp after that :)


@derekwaynecarr the `experimentalMode` is a fragile way of getting correct controller behavior.  Is there an issue to correct this upstream to make it more idiot-proof?


Hi @pekeler,
thank you very much for contributing the App, good work!

I have reviewed it and found two minor issues (created a PR)
1. It has the hardcoded mount point in setup.js and failed if installed anywhere else.
2. It used arrow functions in setup.js which does not work with out-of-the-box ArangoDB.

Would you be so kind to merge these fixes? They will not break your local installation.


still using raw, even if ZALLOC failed!


Shouldn't we change this in the generator?


Should fill in layout from ebay pics


What about mentioning it also in the README, when we show the Sinatra snippet?


Followup issue to consider exposing this.  If there's an existing kube flag, we'll want to plumb it through.


Nice, a bit slower at the beginning, but native performance afterwards :smile:


# Using EmbeddedFilePathEntity for EmailAttachments

Since a month Signum.Files has a new guy in town: EmbeddedFilePath. If you know that EmbeddedFile is, and what FilePath is... just combine them :)

With this commit EmailAttachment starts using EmbeddedFilePath instead of  FilePath, so at some point you'll need to synchronize.

```SQL

--- Adding new fields (auto generated)

ALTER TABLE dbo.EmailMessageAttachments ADD File_FileName NVARCHAR(260) NOT NULL DEFAULT ''
DECLARE @Constraint_EmailMessageAttachments_File_FileName nvarchar(max)
SELECT  @Constraint_EmailMessageAttachments_File_FileName = 'ALTER TABLE dbo.EmailMessageAttachments DROP CONSTRAINT [' + dc.name  + '];'
FROM sys.default_constraints dc
JOIN sys.columns c ON dc.parent_object_id = c.object_id AND dc.parent_column_id = c.column_id
WHERE c.object_id = OBJECT_ID('dbo.EmailMessageAttachments') AND c.name = 'File_FileName'
EXEC dbo.sp_executesql @Constraint_EmailMessageAttachments_File_FileName

ALTER TABLE dbo.EmailMessageAttachments ADD File_FileLength INT NOT NULL DEFAULT 0
DECLARE @Constraint_EmailMessageAttachments_File_FileLength nvarchar(max)
SELECT  @Constraint_EmailMessageAttachments_File_FileLength = 'ALTER TABLE dbo.EmailMessageAttachments DROP CONSTRAINT [' + dc.name  + '];'
FROM sys.default_constraints dc
JOIN sys.columns c ON dc.parent_object_id = c.object_id AND dc.parent_column_id = c.column_id
WHERE c.object_id = OBJECT_ID('dbo.EmailMessageAttachments') AND c.name = 'File_FileLength'
EXEC dbo.sp_executesql @Constraint_EmailMessageAttachments_File_FileLength

ALTER TABLE dbo.EmailMessageAttachments ADD File_Sufix NVARCHAR(260) NOT NULL DEFAULT ''
DECLARE @Constraint_EmailMessageAttachments_File_Sufix nvarchar(max)
SELECT  @Constraint_EmailMessageAttachments_File_Sufix = 'ALTER TABLE dbo.EmailMessageAttachments DROP CONSTRAINT [' + dc.name  + '];'
FROM sys.default_constraints dc
JOIN sys.columns c ON dc.parent_object_id = c.object_id AND dc.parent_column_id = c.column_id
WHERE c.object_id = OBJECT_ID('dbo.EmailMessageAttachments') AND c.name = 'File_Sufix'
EXEC dbo.sp_executesql @Constraint_EmailMessageAttachments_File_Sufix

ALTER TABLE dbo.EmailMessageAttachments ADD File_FileTypeID INT NOT NULL DEFAULT 0
DECLARE @Constraint_EmailMessageAttachments_File_FileTypeID nvarchar(max)
SELECT  @Constraint_EmailMessageAttachments_File_FileTypeID = 'ALTER TABLE dbo.EmailMessageAttachments DROP CONSTRAINT [' + dc.name  + '];'
FROM sys.default_constraints dc
JOIN sys.columns c ON dc.parent_object_id = c.object_id AND dc.parent_column_id = c.column_id
WHERE c.object_id = OBJECT_ID('dbo.EmailMessageAttachments') AND c.name = 'File_FileTypeID'
EXEC dbo.sp_executesql @Constraint_EmailMessageAttachments_File_FileTypeID

--- Move columns (manual)

GO
update a
set File_FileName=f.FileName,
File_FileLength=f.FileLength,
File_Sufix=f.Sufix,
File_FileTypeID=f.FileTypeID
from dbo.EmailMessageAttachments a
join dbo.FilePath f
on a.FileID=f.ID

delete f
from dbo.EmailMessageAttachments a
join dbo.FilePath f

--- Add FK ( auto generated)

ALTER TABLE dbo.EmailMessageAttachments ADD CONSTRAINT FK_EmailMessageAttachments_File_FileTypeID FOREIGN KEY (File_FileTypeID) REFERENCES dbo.FileType(ID)

CREATE INDEX IX_File_FileTypeID ON dbo.EmailMessageAttachments(File_FileTypeID)
```


notify user that the new connection failed using the callback


Thanks, although PXL is doing all of the story translations. This github is just meant to be an archive of the files for now. I'll keep your pull request open in-case PXL is unable to complete the translation. But until then please don't translate anymore of the story. If you would like to help the a lot of the game text needs to be translated at: http://fgo.arimil.com and if you have a good grasp of Japanese you can be a validator which allows you to check translations and accept them.


no


Hmm, OK


this is a very long comparison, how about breaking it up for readability:
```
modify := oldPod != nil
modify = modify && pod.Status.Phase == api.PodRunning
modify = modify && (differentTime(oldPod.DeletionTimestamp, pod.DeletionTimestamp) || differentPeriod(oldPod.DeletionGracePeriodSeconds, pod.DeletionGracePeriodSeconds))

if modify {
...
}
```


esta estructura de define en 2 archivos distintos y no se usa en ningun lado


If you define the timer type like this, you will not be able to enter any "search string" in Kodi Timer settings dialog (the resp. attribute is not set, see xbmc_pvr_types.h for the list of attributes) - you can only specify the days of week this timer rule should match. Not sure whether this is your intention.


To get a NUL terminated string in all cases without buffer overflow.
In fact, should be sizeof(parameters->infile)-1U but infile & infilename are both OPJ_PATH_LEN bytes.
I forgot to add the NUL character at the end of the buffer... (not appended if infilename len is longer or equal to OPJ_PATH_LEN). That should never append but is signaled by coverity as a security issue.


was going add the an issue and give a pull request on the same error :+1: .


вообще объявлять переменную нужно перед ее использованием, тут оно работает, наверное, только изза промисов


Vish,
Have you had time to revisit this?  Was this tested with Nova-network or with Neutron?  Nova-compute only seems to crash if Nova-network hasn't started.  The problem is present when you try to restart Nova-network.  Nova-network crashes every time.  After reverting this change all works quickly and normally.


yep, although the `rc` is not really needed (`finally` is processed even after `return`)


@gjn how can we get the currently used protocol


Yes. However, I wanted to prevent conditions where the dir might have been deleted because of some funky code introduced in the future. With the assert, we catch that condition right away.


General comment: May I suggest that you once open Kodi Timer Settings dialog and check for every type you defined that actually all timer attributes you expect actually show up? I have the feeling that your type definitions might still be lacking some attributes.


omg yes, yes, yes... No for each `sleeptest` in selftests...


>> if (minLevel <= minDepth)
this statement will always be true. Because minDepth cannot be less than 5 and it is impossible to have minLevel greater 5 with 4 threads or 7 threads(or even with 16 threads).

Unless I'm greatly mistaken this test could could run for a long time and show no ELO gain/loss(because the else(optionSp) condition will never fire and patch will behave exactly like master).
Can you check locally if this condition ever fires?


@gjn you know a better way to build the identify url ``rest/services/api/MapServer/ch.swisstopo.zeitreihen/bv80032194_1970``


Please use `1.day.ago` instead.


Ah sorry, I missed that discussion, so I apologise for whatever duplication I added. I would think, however, that with Starscope being a **command line tool*, only two kinds of handling are necessary: the one for \*nixish systems (which include OS X, whose terminal components are forked off BSD and as such fall in the same category – its command line tools do not use plists or store data in the `~/Library` hierarchy) and one for _The Odd Man Out™_, aka Windows.


ну и я бы ее назвал как-нибудь `rejectIfNotCancelled`


I agree, those variable names are tough to decipher. Ok, removing maps
should be fine then.

On Fri, Sep 25, 2015 at 8:06 AM, Paul Romano <notifications@github.com>
wrote:

> I always get confused every time I go back and look at distribcell
> variables. In this case, "maps" was simply the size of the offset array, so
> it is superfluous.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paulromano/openmc/commit/9bc3b3358c6630f5756fb4eab0648c71bbffd213#commitcomment-13436365>
> .
>

--
Will Boyd
Nuclear Science & Engineering
Massachusetts Institute of Technology
Cell: 423.413.8469
wbinventor@gmail.com


Could you add `@formatter:off` here?


Could you add `@formatter:on` here?


How about breaking this long expression for readibility:

```
modify := oldPod != nil
modify = modify && pod.Status.Phase == api.PodRunning
modify = modify && (differentTime(oldPod.DeletionTimestamp, pod.DeletionTimestamp) || differentPeriod(oldPod.DeletionGracePeriodSeconds, pod.DeletionGracePeriodSeconds))

if modify {
...
}
```


Thread.CurrentThread.CurrentUICulture is not the correct culture, when I have a user set as danish, I still get a en-UK culture from it


Could do.
Am 25.09.2015 19:05 schrieb "Emux" <notifications@github.com>:

> How about making this generic method public static?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mapsforge/mapsforge/commit/ecba4c17f72091cddbd3b2ca2b7e81bcb80d8f06#commitcomment-13435407>
> .
>


Fixes php Notices


Okay, great let's do that.


And I had to rack my head for about 2 hours to figure out where lies the error.


Yea OSX/BSD has a completely different API for this. Here's an example from Libevent (it supports monotonic clock for Linux/OSX/Windows with a uniform API... A bit of a hassle to borrow directly though):
https://github.com/nmathewson/Libevent/blob/6e7a580c15a7722f94630184189ae3af6afabdd9/evutil_time.c#L321

`mach_absolute_time` is what you want basically... One issue (if I remember correctly) between these APIs, is that they handle state quite different.


variable**ni**?


you are right, this was added for api groups: https://github.com/kubernetes/kubernetes/commit/fab367230f0e2963b2b4ef4a35159134ce955f95


Use agnostic url (No protocol, just start with // and the browser will replace with current protocol).


It's a core issue that is fixed in latest. Every request needs to set this culture for a user in the back office.


Integration scripts installed at client side specifically try to delete
RM_DB, SCHEDULER_DB repository, some gradle builds maybe. I don't know
precisely. Anyway, it has already changed in the past, like moving from
scheduling/SCHEDULER_DB to scheduling/data/SCHEDULER_DB so I think we can
live with another change.

On Fri, Sep 25, 2015 at 12:16 PM, Laurent Pellegrino <
notifications@github.com> wrote:

> It's probably not required. The idea was to put all things related to the
> Database (hsqldb) in the same directory. What are the transitive changes
> you are thinking about?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lpellegr/scheduling/commit/72dbdb54d945ac75660d703f35cae3e67844de8e#commitcomment-13434741>
> .
>


Just linking in a pull before the push it get's me everytime. We have a bunch of shared repo's at work with lots of contributors.


Perfect, so when dev-v7 is merged in the next time it will just work? - will change to this then


This is good IMO.

Might use https, but not sure urrlib2 supports this well.


Fixed here: 779dd26527907a59d246c95fb1f30d0c386f43b7 If you are using v-latest and this is still an issue then i need to investigate further.


      No, nothing was missed.  I examined this question carefully, and
    none of the ice_ocean_SIS2/* answers changed. The answers only
    changed if the guess at the ice temperature at the start of the
    fast-thermodynamics step was above the freezing point. I suspect
    that this situation only arises with multiple short atmospheric
    forcing and sea-ice fast-thermodynamics timesteps between the slow
    timesteps - i.e., the answers would only change in coupled models.

      - Bob Hallberg
     Dr. Robert Hallberg
 Oceanographer and Ocean Group Head
 NOAA Geophysical Fluid Dynamics Laboratory

On 09/24/2015 11:26 PM, Alistair Adcroft wrote:

      Did none of the ice_ocean_SIS2/* answers change, including
        OM4_025? I haven't got a session to test for myself but I am
        suspicious that something was missed ...
      —
        Reply to this email directly or view
          it on GitHub.


I did look for a gem which implemented the "detect my OS and figure out where config files should go" pattern and couldn't find one, which made me think that either it was more complicated than I thought, or nobody actually cared.

Given the number of pings I've received people clearly care, so I'm more surprised than ever that I can't just import a gem and let it do the right thing. Do you know of other gems that have solved this problem?


hehe yea we need to polish some of this little stuff


Hopefully how I have pulled the other branches I'm working on are a bit easier for you now. Don't worry about deployment they are happy to wait until you return now. Just going to keep working through the fixes left and hopefully they can be checked in next week to be sent to testers.


are you able to compile it and boot?
kindly let me know


now if you can find a way of also changing the logo on the top-bar to something good ... :smiley:  that would be awesome


... but again, you don't need to do that at all! you don't have to pass the culture in, it will do it for you!


@smarterclayton  Is this indicative of a lost patch?


@ixeetProject Please review..


Please fix the type in `presents` word.


Hi @sebersole , why the EnversSchemaGenerator class was removed in this commit?

Cheers


So parameters->outfile[4095] is guaranteed to be /0 ?  Where is this set in the code?


@ramsudharsan https://github.com/D820-dev/android_kernel_htc_a51dtul/commit/054a2fe1325b5e82a383674e6dd8dd7c768a5abb
Copy pasting HTC's tarball onto mainline should work right? Exact same version


The java property "proactive.node.workers" must be set when starting the
NodeJVM. This property is defined in class RMNodeStarter

On Fri, Sep 25, 2015 at 12:36 PM, Laurent Pellegrino <
notifications@github.com> wrote:

> Suggestion applied for the number of nodes.
>
> @fviale <https://github.com/fviale> Do you know how to enable this
> feature?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lpellegr/scheduling/commit/f63242d9f4cd31188c9fe0d843fc156f1316d7f1#commitcomment-13435005>
> .
>


It's probably best to continue using this 'water' variable, since you defined it above.


Filed SERVER-912 for metrics.

I think I agree that consistency between these two is good, and that as a consumer it's easier to use this as a macro.

I've implemented it using a macro in https://github.com/rlinehan/puppet-server/commit/4d21570b4ae525186593eb1793e0ab4059149ac8. I've implemented it a little differently than `with-jruby-puppet`; `with-jruby-puppet` requires the jruby service to be passed in as a command. I feel like this actually makes it somewhat harder for the consumer to use (I always forget what function to use to get a reference to a TK service). Instead I've just included the macro for `with-lock` in the `jruby-core` namespace, and still have a `with-lock` service function. Is there any downside to doing this or reason I shouldn't implement it this way?


godoc explaining why this is important.


this isn't ssh.weekly-review.bocoup.com because of the weirdo DNS issues you were having yesterday, yeah?


It's possible to rewrite this using AuthenticatableContact.objects.select_related('relation').get(user=request.user)
The old code didn't do this, and it effectively never used allow_id=True afaict so the current implementation is equal to the old one. The old code also had nested imports which are not needed with the current code, this may change if we import the model in this module.


This if condition could be moved to a separate method. What's your opinion about that?


@ixeetProjects  Error messages are changed...


Sure ;)


@ixeetProjects  New added services...


Ha, if there's gulp task for that, feel free to implement. Haven't checked it to be fair! Would love to see a PR from you implementing that @ViliamKopecky :)


In that case it would probably make more sense to make AuthenticatableContact a swappable model.


No, it does not control bln for messages.
It is to not light it up at all or when only screen is touched.


Let me see if there's other things that should go into 2.1.1 and then see about a Tuesday release. The updated retry needs to go in as well.


This still expects the period to be provided.


That would be a good note to add for best practices for defining variable names.


@ixeetProjects  Please merge the code..becoz new addition have been done.


I'm not familiar enough with github to know if I've actually submitted these two changes.


1: M101 signature is missing
3: M102 unrecognized bullet/signature
5: M102 unrecognized bullet/signature
7: M102 unrecognized bullet/signature
9: M102 unrecognized bullet/signature
11: M102 unrecognized bullet/signature
13: M102 unrecognized bullet/signature
14: M102 unrecognized bullet/signature
1: M111 unrecognized component name:  Crossrefutils
1: M190 line is too long (62 > 50)
5: M190 line is too long (101 > 72)
7: M190 line is too long (75 > 72)


Good idea broh


It was my own fault. I had done a git diff with some wild cards (since i had many files).  The diff looked fine, so I simply hit up-arrow, home, and changed git diff to git add.

I do not do a git commit -a because I have several files that are modified, but not to be checked in yet (such as formats.c with the extra logic for -test-full, and a modified single.c and modfied x86-64.h).


@tomwhite.  Hi Tom:
I am not sure how I ended up with some of your spark files in my integration test.
I am not a software engineer.  Any insight you can provide would be greatly appreciated.  Thanks in advance,
-david


@fviale Thank you. I will create a specific pull request for that once this one is merged.


FFS... apparently that fix didn't fully fix it, will fix now


This commit is being reviewed as a part of review issue #63.


Your commit message doesn't contain the correct Story ID format.
Just a notification though, let it pass, no rewriting history.


updated to set this the way its already happening upstream, based on if autoscaler (or deployments - not included in this rebase) are enabled.


Wo ist denn diese Änderung? In is_guard kann sie nicht sein, weil die Funktion nur einen Parameter bekommt, und sie ja beide Einheiten betrachten müsste. Es gibt noch is_guardian_u, was zwei Einheiten betrachet, und... aha, da ist es. Prima. Allerdings static innerhalb von move.c, muss ich mal gucken, wie ich da dran komme.


Ну, добавил такой вариант https://github.com/altocms/altocms/pull/745. Возможно я не совсем уловил суть идеи, но этого достаточно, по крайней мере, для меня.


lol, nope never mind... it works :)


Now that I have keccak and sha3 fully working within dynamic, I can remove this test file I wrote to help get me there.


We also still have this line in opj_decompress:

	strncpy(parameters->infile, infilename, sizeof(infilename));


Hi AK,
interesting commit. What does it do exactly?
Tx, Andi


`@wraps(f)`


OK, this is going to fail on my machine, but it's good to have it in selftests. (at least until we start running selftests in parallel...)


camelCase


Not a big fan of this name or it being a decorator.

Why not `camelize_keys(dict_)`?


Neat. Specially the gif!


I don't even have Midori installed, so I'm not interested in searching its bookmarks. And, even though Firefox is installed, it doesn't have meaningful bookmarks. I might not want to search bookmarks at all.

Can you use a different notification mechanism (e.g., a text field or a list entry inside Gno-Menu) for this information? Or perhaps introduce an option to disable bookmark searching? For some reason I'm seeing these two (!?) notifications even if I'm not consciously performing a web search.


提交DMGCD框架


this will just return null when cast


Can we have a cellprofiler/cpmath/cpmorphology.py that's the following?

    import warnings
    import exceptions
    warnings.warn(
        "cellprofiler.cpmath.cpmorphology is deprecated, please use centrosome.cpmorphology instead",
        exceptions.DeprecationWarning)
    from centrosome.cpmorphology import *

and similarly for the others that remain?


Hi David,

Not exactly sure, but you probably need to checkout the latest master branch and use git rebase to rebase your branch on it. You could also just delete the errant files from your branch - though rebasing is generally good anyway.

Tom


> but post-hammer is much more careful about this sort of thing.)

@liewegas just want to confirm with you: is the reason of this "carefulness" the `cmp_bitwise()` we are using now. which is different from the tuple of `<hobj, shard_id, generation>` to compare the `ghobjects` when listing objects in a collection back to hammer. am i right?


OK.  Will do.

Thanks,

-david

> On Sep 25, 2015, at 8:43 AM, Tom White <notifications@github.com> wrote:
>
> Hi David,
>
> Not exactly sure, but you probably need to checkout the latest master branch and use git rebase to rebase your branch on it. You could also just delete the errant files from your branch - though rebasing is generally good anyway.
>
> Tom
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/broadinstitute/hellbender/commit/f6ca592b23e5a1e07f163f096bf957f5a0959f47#commitcomment-13437182>.
>

—
David E. Kling, Ph.D.
Senior Technical Writer
Broad Institute
75 Ames Street, #7003DC
Cambridge, MA 02142
O (617) 714 - 7351
C (617) 291 - 1617


Using `parseInt` without specifying a radix is a bad practice. If no radix is specified, some browsers might use 8 instead of 10 for strings starting with `0`. Also see the docs at MDN: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/parseInt


```
Server/WorldSession.h:810:51: error: address of array 'this->m_Socket' will
      always evaluate to 'true' [-Werror,-Wpointer-bool-conversion]
        bool PlayerDisconnected() const { return !m_Socket; }
                                                 ~^~~~~~~~
```


See https://github.com/readium/readium-js-viewer/issues/412


You're probably right. Will change it :)


1. This will return null as `"presence-thing"` does not begin with `"private-"`
2. I think so too


Yeah, I'd agree that the truncation should be party list only.


I merged the changes into develop locally and pushed to origin. We can have a look at our git workflow on Monday. In the meantime, sprinting on Drupal 8 with a dire hangover. :)


Yeah looks as though it never fired :(


Morrison A, Straube S, Plesser HE, Diesmann M (2007) Exact subthreshold integration
with continuous spike times in discrete-time neural network simulations. Neural
Comput. 19(1):47-79.


Hanuschkin A, Kunkel S, Helias M, Morrison A and Diesmann M (2010) A general and efficient method for incorporating precise spike times in globally time-driven simulations. Front. Neuroinform. 4:113.


so complicated XD, good to learn


Looks good, you have my vote.


Plot legend only once and x-tick labels only in lowest panel, which prevents overlap with title.
```python
    if v==2:
        plot.legend(loc=4)
    else:
        plot.set_xticklabels('')
```


For maximal backwards compatibility via conversions, you ought to do the exact opposite of what a proper design would mandate. How about splitting functions instead and having thin asserting wrappers forwarding to the properly designed function, macros even? Or designing `locality` properly from the go, and using a helper `backwards_compatible_locality` type in function arguments that implicitly convert from old and new types? Given that any solution would be temporary, just one release until `id_type` is phased out, it might not be worth the cost.


requirements for the app


This may not be ideal, because `debugger-ui` is to use 8081 port.


Asi nikdy, ale proc to omezovat? Kdyz nekdo zada dve data, proste se pouzije to druhe.


continuous-time -> precise
or off-grid


@oleg-nenashev the releases were not to Jenkins repo.


ok - updating


[1] [2] -> [1,2]


hi


rax retry tests


This is used to unplug/plug cores based on battery temperature/amperage/voltage etc. we disable it because it is not really something we want to encourage. It seems to be useful sometimes, but not often enough, it usually results in huge performance regression. We have seen issues with it being enabled on shamu because we also use dynamic hotplugs.


OK, then everything is fine here.
:+1: for the plugin


ok, thanks


what I wanted to achive was add something to umbRequestHelper.getApiUrl(...) urls _without_ changing the BackOfficeController... in a way, make the whole thing pluggable - is that feasible?


Thanks, Dannii. I’ll plan on making a general announcement about this to Inform-using entrants as well.

> On Sep 25, 2015, at 1:36 AM, Dannii Willis <notifications@github.com> wrote:
>
> I've fixed a pretty major bug with the status window, so it would be good if you could update Parchment. If you want to use the Inform template it is at https://raw.github.com/curiousdannii/parchment/master/lib/parchment-for-inform7.zip <https://raw.github.com/curiousdannii/parchment/master/lib/parchment-for-inform7.zip>
> Let me know if I can help with anything further.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/jmacdotorg/ifcomp/commit/315610feb47569b94ba372c1824607b65a52a55c#commitcomment-13430460>.
>

--
Jason McIntosh

http://jmac.org • jmac@jmac.org • @JmacDotOrg


**README - Nitpick**
The correct wording for _folders_ is **directory**.


How about a git utils file?


I fixed it so I get a feedback when bad info is provided


as ResourceMap & Catalog tests output is ignored, it should not silently fail, please remove the try-except


For some reason the rule set in css_styles ```{1,select,1107:color:#ccc}``` does not seem to be applied on the chart (e.g. see "Vomiting" for Patient ID 5000 on dev - Vomiting = None is coming out black instead of grey.


commit message neglected code in cachematrix.R,
which should have been in earlier commit.


not very important but backoffice or backOffice ?


Not sure it's getting much simpler this way.


you don't need to rearrange the functions, just write
```local near_surface, add_shrooms, make_worms```
on the beginning of the file and these are accessible and localized everywhere under this line.


ok


Unless we're going to be maintaining the Java bindings, I would suggest it be removed from here for now -- it's not tested, so it shouldn't make it into the next release.


**README - Nitpick**
The correct wording for _folders_ is **directory**; and when siting a file or a directory, put it between the back-quotes, e.g. `/tests` directory.


Thanks, @jonatack! :heart:


ok

On Fri, Sep 25, 2015 at 2:37 PM, Laurent Pellegrino <
notifications@github.com> wrote:

> @fviale <https://github.com/fviale> Thank you. I will create a specific
> pull request for that once this one is merged.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lpellegr/scheduling/commit/f63242d9f4cd31188c9fe0d843fc156f1316d7f1#commitcomment-13437076>
> .
>


**General**
- Well composed commit message, keep it up!
-


Added global "portal_onsite_document_download" to control feature.


Unnecessary spacing detected.


yes, it was too strange to be true! My change _almost_ fixed the issue because it's slowed down the loop: the real issue was a race condition in Qt's QUdpSocket, causing the readyRead signal to stop being emitted sometime. It has been fixed in Qt 5.5.1 that should be released around next week. cf: https://bugreports.qt.io/browse/QTBUG-46552


Awesome!


unexpected token tCONSTANT<br>dynamic constant assignment


Line is too long. [108/100]


merge


**Functionality - Satisfactory**
It's good you used PSR4 autoloading.


I think it's a bit misleading that the voltage traces of the precise neuron look different in the three panels due to the different recording intervals. Maybe it's enough to adapt the comments accordingly.

Apart from that, no further comments from my side and :+1:


Yes, but you'll need to replace it on each iteration of the loop.
I think you can do this with resource.clear_query()


just a proposal:
setSchemaFolderLocation(docSource.getSchemaUri());


getSchemaFolderLocation()


>So parameters->infile[OPJ_PATH_LEN-1] is guaranteed to be /0 ? Where is this set in the code?
Not yet, coverity hint that suggested that strncpy would NUL terminate the string and using sizeof(parameters->infile) would overflow. As I said,  I didn't add the NUL character at the end of the buffer, needs to be done as `strncpy` does not always do this.
>We also still have this line in opj_decompress
Well, coverity missed it I guess... So not corrected. I'll check all `strncpy` calls.


**Code Quality - Nitpick**
You did not need to `require` PHPUnit, all you need is to give instructions in _README_ for users who have not installed it to do so.


`systemd.exec(5)` needs an update as well if we support this.


CAP is a 'transformable' - shouldn't need a special case here.
I pointed to the handling of these here:
https://github.com/flavour/eden/blob/master/modules/s3/s3rest.py#L432


This is what I was recommending, yes :)


Should we not send a response at this point?


Thanks @sergiogonzalez. I'll try using ResourceBundle.

On 9/25/2015 3:29 AM, Sergio Gonzalez wrote:
>
> Hey @jhinkey <https://github.com/jhinkey> maybe we need to use
> ResourceBundle.
> We have a similar use case in
> document-library-google-docs/src/META-INF/resources/portal_settings/google_apps.jsp
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sergiogonzalez/liferay-docs/commit/95d9685247d696b9d8a05d25605dcef50e364bf4#commitcomment-13431549>.
>

--
Jim Hinkey
Sr. Knowledge Engineer
Liferay, Inc.
www.liferay.com <http://www.liferay.com>

*Liferay Symposium North America*
November 16-17, 2015 | Chicago, IL
www.liferay.com/northamerica2015 <http://www.liferay.com/northamerica2015>
---

Visit Us: www.liferay.com <http://www.liferay.com> | Like Us:
facebook.com/liferay <http://facebook.com/liferay> | Follow Us:
twitter.com/liferay <http://twitter.com/liferay>


This looks odd? 2 identical queries but the 2nd just gets the 1st row
ftp_row = ftp_rows.first() saves a DB query


Shouldn't this also be updated in `sass/susy/language/_susy.scss`?

Still says `2.1`.


Any idea when this gets reviewed and integrated?


Is this correct? In what situation we have a token used to create a headline, without the type being set?


**p,tht;tys**


Some services which are often used, if you want to take a look at them(many aremuch easier than using Travis + Scrutinizer).

https://insight.sensiolabs.com/
https://codeclimate.com/
https://codecov.io/


When do you intend for this to be called?  If it is meant to be used as a fitness function or utility score, you might run into some issues.  It simply won't play any card with value less than 3 (4.8 = hp*attack) since it is worth more points to hold it in hand.  Another thing is that the Coin messes with hand values, after P1 draw on T1 the net score is 0, but after P2 draws on T1 the score has him up by 6.


What do you mean 'works through CRUD'?
It isn't possible to test via /eden/sync/task currently since there isn't a controller for that...you could add one to test I suppose.
If you really need to reconfigure the options here then you can do that in the field.requires
Either pass a set to IS_ONE_OF instead of db or else add a filter.
However I think permissions should just work here if set correctly


Інш**f**


the change


I did put a cleanup stage on the


their topics


@toobulkeh I co-opted your google maps code for the photographer profile - had to make a few of the classes more generic


кожнен -> кожен


Comment should be at the top of the else branch not outside it:
else:
    # First Run


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.981.c2434a7519ab3aa2734fb343c605ed03c747cb2c](http://build.palaso.org/viewLog.html?buildId=63796&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 532, ignored: 6 Build time: 00:06:14


if needs updating


It can safely default to None I think...there hasn't yet been a push has there?


@VlasovVitaly
I have created some preset characters before, and I compiled the source, ran the game, then chose [New Game] -> [Preset Character] in the main menu, then the game crashed. I discovered the vector `world_generator->all_worldnames` is empty, while `sel3 == 0`.


Shouldn't it be https://api.smsapi.pl/ ?


respect both for backwards compatibility


`мітах_->мітах__`

тобто замало підкреслень