For me that indicate that you use a nonstandard space, can you verify that's relay a standard space?


What's wrong with the default generated copy constructor?


picked


@vilchik-elena My understanding was that symbols were declared in `SymbolDeclarationVisitor`.


The build script not noticing these is not great.


I wish there was a better way than casting `nullptr` to `void*` only to force a particular overload to be selected.


What? How? Is that hardcoded somewhere?


you can bind with lambdas to avoid the anonymous class


This is a code duplicate, could it be encapsulated in a guard method and reused, while better capturing the intent? e.g. EnsureNotUsingChildContainer or some other name.


knm,


没错
记得按时吃午饭啊……


zzz


ah woops your right, need to run most of it thru a formatter any ways as alternates between brackets on new line and not, same for indentation, few sections missing. Will fix this up, once i hear if the remaining issues are Particles end or not.


Nice work @andela-toduah, however there are some issues with the UI. On the page listing the products, there is no consistency in the number of products on each row (varies from 1..4). Also, I think the page numbers would look better without borders


The color is 16


Yes, I'll take a look at this.


See http://groovy-lang.org/structure.html#_import_aliasing


misaligned condition. To this spi freq, is it planned to make this calculation dynamic?


@vilchik-elena How useful is that method? It's called only in a test.


We're officially doomed now.


picked


@pynicolas SymbolDeclarationVisitor creates symbols which are hoisted (created before executing scope). That's not the case for class:
```
var x = new A();  // this will fail

class A {}
```


She stolen my account and this is my email
2016/2/15 下午3:18於 "László Szemcsák" <notifications@github.com>寫道：

> Open Internet Explorer and trying
>
>
>
> 2016-02-14 22:18 GMT+01:00 Benone Bitencourt <notifications@github.com>:
>
> > I'm trying to join/sign up Epic organization to fork Untreal Engine, but
> I
> > don't get access and still getting 404 in Unreal Engine git page. Someone
> > can help me out here?
> >
> > Thx in advanced!
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/EpicGames/Signup/commit/91dc07dabbca8ad80a3fed5e56335609c2cc9bb7#commitcomment-16085624
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EpicGames/Signup/commit/91dc07dabbca8ad80a3fed5e56335609c2cc9bb7#commitcomment-16088731>
> .
>


tc_init() is a device specific timer init and not all the device supports return value so we removed it.


Thanks for the clarification!  I plan to convert over to Elixir / Phoenix to get immutability plus better performance rather than worse, but the need for that has not yet arrived, and in the meantime, our app cannot afford to perform any worse than it started out doing, which was pretty poorly but is now hard to make significantly better for the majority of requests -- though there remains a badly performing minority that does affect a certain population of users.


`current_idx .. ;true;false]`


Is there an alternative solution for `NeoBundleClean`?


thanks!


@pynicolas Because any class declaration which is not global was parsed as expression, as CLASS_DECLARATION is lower that EXPRESSION_STATEMENT.


OK


IMHO, we need to centralize the validation you made at 2 different spots. You probably haven't noticed it, but you can do it through the ```@wsme_pecan.wsexpose(Audit, body=Audit ...``` decorator which basically says that the body of the request is of type `Audit`. So by adding the `@wsme.validate(types.uuid, Audit)` decorator for both `post()` and `patch()`, I guess you can implement the `validate()` function just like for `ActionPlanPatchType` in `watcher/api/controllers/v1/action_plan.py` to do the work.


@vilchik-elena Should we then change the name of SymbolDeclarationVisitor to reflect that?


The text should be `Invalid frame` as shown in the README. Sorry for nitpicking. But consistency helps understanding issues.


Pour forcer la lecture complète du fichier avant son écriture.

Avec `cat $file | something -o $file` on risque (fortement, suivant sa taille) de commencer à écrire dans "$file"  avant d'avoir fini de le lire, et ça ne peut généralement pas bien se passer.

`tac` donnant les lignes de la fin au début, le fichier est lu entièrement, un autre `tac` les remet dans l'ordre, à ce moment là tout le contenu du fichier est lu, on peut commencer à envoyer le début dans `msgcat` qui à son tour pourra écraser le fichier: Vu qu'on l'a lu entièrement.


test


Text should be ` Missing filename, and line number` as both are missing.


@vilchik-elena Should we have a parsing unit test to check that?


picked


@atd This is the point loginRequired is failing. $timeout is already used inside it, so do you have any idea of what is missing?


@pynicolas We can.. Any suggestions?:)


why the omod? are we depending on classes in omod, and not the api?


@vilchik-elena HoistedSymbolDeclarationVisitor?


Only BarChart here, forgot to mention on commit.


@ahoarau  You can use the `CMAKE_POSITION_INDIPENDENT_CODE` CMake option for this, see https://github.com/robotology/yarp/blob/c0682dabab4904a4a4a51553caacee1b7fc0d7eb/conf/YarpOptions.cmake#L307 .


I'm thinking you invited Administrator page. Write Administrator mans or
waiting e-mail than your account is ok and allow admin you can write your
Tournament activity.

2016-02-15 9:59 GMT+01:00 lewisine <notifications@github.com>:

> She stolen my account and this is my email
> 2016/2/15 下午3:18於 "László Szemcsák" <notifications@github.com>寫道：
>
> > Open Internet Explorer and trying
> >
> >
> >
> > 2016-02-14 22:18 GMT+01:00 Benone Bitencourt <notifications@github.com>:
> >
> > > I'm trying to join/sign up Epic organization to fork Untreal Engine,
> but
> > I
> > > don't get access and still getting 404 in Unreal Engine git page.
> Someone
> > > can help me out here?
> > >
> > > Thx in advanced!
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/EpicGames/Signup/commit/91dc07dabbca8ad80a3fed5e56335609c2cc9bb7#commitcomment-16085624
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/EpicGames/Signup/commit/91dc07dabbca8ad80a3fed5e56335609c2cc9bb7#commitcomment-16088731
> >
> > .
>
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EpicGames/Signup/commit/91dc07dabbca8ad80a3fed5e56335609c2cc9bb7#commitcomment-16090036>
> .
>


Sorry for such a late reply. The description was added with proper explanation in Dec, 2014 which is before you've asked this question. There are examples and explanations as needed.

Though I'm not sure if this works anymore. I haven't used it for a year now. Maybe to get it to work you need to do some modifications in selectors to extract data.


Interface Implementation


здесь надо добавить кавычки - а то не понятно, как будто бы array тоже может вернуться,
а на самом деле вернется "array"


wrong indentation, it's space based, not tab


может переименовать x в value?


Quick question: could we even just not even reset when it is from our parent? I guess this will anyway lead to a parent switch? Which then will automatically do things (have not check the full flow). But anyway - I agree on removing as many trickle resets as we can. Maybe the trickle timer should be reset only if the new rank of the parent and our own rank is significantly changed so that is worth quickly informing any of the other nodes.


Could you squash the two changes here into 7eb3f9f ("increment bosh release") and 9c69e0b ("set max retries") respectively?


Это соответствует макету. Сделал 460px(те же 98% только в пикселях) чтобы глаза не мозолила)


why after start ?


@mikemcquaid: it seems with the `var/{postgres}` directory creation, the conditional check in the following line always returning false and the PostgreSQL cluster is no longer initialized post-installation? Is this change in behavior expected?


It will be really annoying, we should be able to pass a provider


Looks like it only works with default stuff like name :/ well, either way, its gone completely now ;)


随分中身変わったんだなぁ… うちも倣って直そう。PRデプロイはできてるけど。


There seems to be an issue with the pagination after deletion. By default there are 5 polls displayed per page. If I delete 2 polls from  first page, currently it displays 3 polls, but instead should display 5 polls (if there are sufficient polls remaining). Since, if I navigate to the next page, it currently does not display the polls which it assumes to be on the previous page after deletion. (I'm sorry I hope i am not too confusing here!)


Yes, that looks good. I guess local repair will only be useful first time - then it will find new parents via DIO's anyway - without repairing lots of times.


No, I never let others use my account
2016/2/15 下午5:06於 "László Szemcsák" <notifications@github.com>寫道：

> I'm thinking you invited Administrator page. Write Administrator mans or
> waiting e-mail than your account is ok and allow admin you can write your
> Tournament activity.
>
>
>
>
>
> 2016-02-15 9:59 GMT+01:00 lewisine <notifications@github.com>:
>
> > She stolen my account and this is my email
> > 2016/2/15 下午3:18於 "László Szemcsák" <notifications@github.com>寫道：
> >
> > > Open Internet Explorer and trying
> > >
> > >
> > >
> > > 2016-02-14 22:18 GMT+01:00 Benone Bitencourt <notifications@github.com
> >:
> > >
> > > > I'm trying to join/sign up Epic organization to fork Untreal Engine,
> > but
> > > I
> > > > don't get access and still getting 404 in Unreal Engine git page.
> > Someone
> > > > can help me out here?
> > > >
> > > > Thx in advanced!
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/EpicGames/Signup/commit/91dc07dabbca8ad80a3fed5e56335609c2cc9bb7#commitcomment-16085624
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/EpicGames/Signup/commit/91dc07dabbca8ad80a3fed5e56335609c2cc9bb7#commitcomment-16088731
> > >
> > > .
> >
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/EpicGames/Signup/commit/91dc07dabbca8ad80a3fed5e56335609c2cc9bb7#commitcomment-16090036
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EpicGames/Signup/commit/91dc07dabbca8ad80a3fed5e56335609c2cc9bb7#commitcomment-16090131>
> .
>


Well, I mostly write XQuery, so I'm not that much interested in javascript or CSS. Unfortunately, there are no external editors for XQuery I can actually work with. Even though I wrote a plugin for Atom, I still switch back to eXide for serious work. eXide also transparently compiles less to css, so no external tools are needed. And since I often switch between projects and places, I like to be able to do all development online.

I could probably live with working on the js on the command line, but I would prefer if we could keep an option to produce a dev xar with unminimized sources.


说错，是调整登录框


picked


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


Thank you


Since we have AdminEmailHandler in our logging configuration, should we document setting ADMINS somewhere?


Unblock setspawnworld & setworldspawn.


На курсах говорили что h1 желательно чтобы один был. У меня на странице несколько раз заголовки одного размера встречаются поэтому сделал их h2 и h3. Есть заголовок на странице каталога(он у меня через <p> сделан). Можно было бы его через h1 сделать.
Это ничего, что на главной h2 и h3, а на странице каталога h1 h2 h3? Просто h1 самый весомый, а на второстепенной странице - как то хз)


Happy for you to merge this @dallinb


вот смотри - у тебя тут после тире где-то описание с маленькой буквы, а где-то - с большой


I speak about the used character not the language syntax!
But I just verified and it's effectively correct :-)


Any comment on the logic behind changing the EWMA_ALPHA to 15? We have been using 2 on the ALPHA to get more stability and less switching - but 15 is interesting to know how you came up with (we have just started to test with this new link-stats module so maybe there is something different than the old behavior).


Создание полнофункциональной админки.


This commit removes handling of series. Is that intentional? If so series are tricky. Does this code handle al the variations?


`query + '*'.freeze`?


I've run into this unexpectedly on nightly. As it's configurable, I don't mind at all, but I can't help but wonder what's the reason for the new default behavior?


ну тут тоже, собственно, надо определиться с соглашениями, камел-кейс или питон-стайл


得到


Ah sorry, I misunderstood.


得到2


Inconsistency detected. http_parser.c:

```
/**
 * Verify that a char is a valid visible (printable) US-ASCII
 * character or %x80-FF
 **/
#define IS_HEADER_CHAR(ch)                                                     \
  (ch == CR || ch == LF || ch == 9 || (ch > 31 && ch != 127))
```

Real example (nginx + maxmind) with `GEOIP_CITY: Düsseldorf` - got an empty reply, so nginx will send the request to next upstream and then it will end with 502 bad gateway cause nobody replied.


I am writing profiler_num_samples into the PROFILING memory region here, first switching the write focus. However, when I read that on line 18 of profiler.c, I get the wrong value. Is this because write to SDRAM occurs not before I read that memory location in lower level? Thanks.


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.5 [Build 100](https://teamcity.labkey.org/viewLog.html?buildId=303382&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 12 (2 new), passed: 38 Build time: 01:18:51
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

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

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

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

##### there are 2 more failed tests, see build details
```


why is this using seconds squared?


This is unrelated to your pull request. See #640.


I recently saw another GitHub harboring a similar project (Natura - Minecraft Mod) which tries to put the code under CC0 (which I am okay with) and the textures/art under a separate license.

I'll be honest; I am not prepared to let go of it yet. One day I want to release the entire project in GitHub under CC0, when I am no longer interested in updating it. I made this mod for people to use, not to rot away due to my selfishness.

I feared that possibility since I removed the license. As you have said, I cannot invalidate what I did (an example: like stating something offensive on Twitter). I feel both a relief and a regret as I was able to do what I could not do, but fear that I may lose interest in the future of the project should anyone try to make a project near similar or misuses the assets of this mod.

Thanks for pointing it out. I would have never known.


Hi Tim,
I think it's better to replace the current test by the following:

```
def test_check_migrations(self):
    requires_migrations_checks = dance.Command.requires_migrations_checks
    try:
        with mock.patch.object(BaseCommand, 'check_migrations') as check_migrations:
        	dance.Command.requires_migrations_checks = False
            management.call_command('dance', verbosity=0)
            self.assert(check_migrations.called)
            dance.Command.requires_migrations_checks = True
            management.call_command('dance', verbosity=0)
            self.assertTrue(check_migrations.called)
    finally:
        dance.Command.requires_migrations_checks = requires_migrations_checks
```

It's only making sure that even if we change the value of `requires_migrations_checks` the test will not fail.


picked


форматирование пробелами - тоже не очень


is the color white intentional here? It makes the text invisible (white on white).


What happens when patient program is not provided?


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


It is worth asserting the data we get is correct.


is_last -> isLast?


Ok @lucaderi


@sbesson It is not, I pushed the wrong version, sec.


возможно придется сменить пароль в spring.propirties


Please add the image file as well.
Also, wouldn't it be better to leave img-center class and change the selector in css?


there is no strict requirement, 2.0.6 was added here since advanced app changes depended on changes in the framework.


"." at the end...


都用英文交谈 我还能说什么呢 都听木懂


Remove these multiple "class=uppercase" please. Substitute with ID for those that are NOT uppercase, and add uppercase for all <li>-s in this ul class.


I confirm that this is the case. Somehow it's `git`'s fault, but trying to push a change there would be more difficult. Maybe this should be reverted? :(


I don't think adding the package used add much value here, but do you know what would be incredible?
A sample .env file! :trophy:
The rest looks good to me. :+1:


I don't know why :(
Under Win i tested :
1) "..\hardware\1Wire\1WireCommon.h"
2) "\1Wire\1WireCommon.h"
3) "..\1Wire\1WireCommon.h"
and all worked.
Under travis only this first passed. But you're saying that under Linux not compiling :(
Please check/repalce for other include of the above.


No inline style please.


Имена `Activity` не описательные.


As I remember a cell is every round on the same position...or not?
It just changes their alive state.

A better sentence would be: The GameFieldController for calculationg alive states of all cells for each round.


Please add the logo file as well.


@byronmccollum, we usually set 4GB with systems that have at least 48GB of memory.
This is already configurable since it's a set of variables.
We could potentially calculate this based on the amount of RAM in the machine.
I'll see what I can do :)


abc


@sbesson Fixed in b40f107, thanks!


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


By the way. I think this item had an onclick event described in the design?


So that GPU bar doesn't look like a sliver as compared to the CPU bar. Is there any other measure you'd recommend?


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


picked


for number values


С переименованием потерялся смысл названия метода. Native подсказывает, что functorPointer - это сырой указатель на функтор в ядре. Нужен комментарий или другое название метода.


No problem, thanks for fixing!


I don't really like having duplicate code everywhere.

It would be better to have a `PathUtil` utility class with a `basename` static method. This way, tests could be added to it and behavior checked in case of subtle bugs.

I can see there would be some issue with your method when `$path` does not contains any `/` or `\`. I dit not checked whether this is possible in current Zippy implementation.

A better approach could be:

```php
abstract class PathUtil
{
  public static function basename($path)
  {
    return (false === $pos = strrpos(strtr($path, '\\', '/'), '/')) ? $path : substr($path, $pos + 1);
  }
}
```


How is the ü in Düsseldorf encoded?  If you can get the raw bytes for that header, that would be helpful.


The other display controls created have used a different approach.


どうやら原因はここだったみたいです


@BreakingBad69. Indeed, it works properly with some other products. Then, it could be an issue with the product configuration. Do you face this with some other products as well? Did you try to delete and re-create it?


sdsd


Weel done!


Why not `in_path_command` too?


aavvbb


Please, keep imports in alphabetical order.


Newline is not necessary


new comment


There is an extra comma in the script
`STRING_OUT+="          0.0, 0.0, 0.0, 1.0],\n"`


```
$ curl -vvv http://localhost:8000/something -H 'GEOIP_CITY: Düsseldorf'
*   Trying 127.0.0.1...
* Connected to localhost (127.0.0.1) port 8000 (#0)
> GET /something HTTP/1.1
> Host: localhost:8000
> User-Agent: curl/7.43.0
> Accept: */*
> GEOIP_CITY: Düsseldorf
>
* Empty reply from server
* Connection #0 to host localhost left intact
```


> イメージとしてはプレビューといいつつも layout とかはそのまま (editor 用) で、本文にだけスタイルがあたってて、他の公開日時などといったメタ情報も普通に出ている

私もこの方法なら`posts#show`でいいと思いました！

`/editor`を記事編集画面として考えると`posts#show`でプレビューされたほうがよさそうだし、管理画面として考えるとメタ情報も表示したほうがよさそうだし…と考えていたので、本文だけスタイルを当てるというのはよさそうですね。

ページングを付けなくて大丈夫かというのと、競合しないようにスタイルを当てられるかが少し心配ですが…


Please, fix indentation and lint error.


плохо звучит value of null


Thanks!
My comments are above.


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


@nobu oops. I missed it. Thank you notification.


LGTM


Ok

P.S. This repo installing by `create-project` command of Composer usually, because `yiisoft/yii2 v2.0.6` never be installed. The offset need to improve - occurred to me.

P.S.S Thank for 2.0.7 release :wink:


tree? тоже не очень удачно


最初の版


Old code


This test fails, there is no null check in getFileBytes


м.б. [+]"name": value
как-то так?


Sorry about that.  Thanks for performing the colonectimy.


This test fails too, you're not checking the correct returned contents


@ekachandrasetiawan :
```python

 +	def create(self, cr, uid, vals, context=None):
+		res = super(order_preparation, self).create(cr, uid, vals, context=context)
+
+		m  = self.pool.get('ir.model.data')
+		id_group = m.get_object(cr, uid, 'sbm_order_preparation', 'group_admin_ho').id
+		user_group = self.pool.get('res.groups').browse(cr, uid, id_group)
+
+		for x in user_group.users:
+			if x.id <> uid:
+				if x.partner_id.id:
+					self._set_followers(cr, uid, res, x.partner_id.id, context=None)
+		return res
+

```

create dipisah aja can,, jadi kayak gini kira2
```python
def create(cr,uid,vals,context={}):
     res = super(order_preparation,self).create()
     self._set_followers()
     return res

```


@vilchik-elena Shouldn't all instances of a class have the same type?


I like the option to put the token into an environment variable.

How much shell interpolation does `.npmrc` do?

Could we put a read token in there as a default for when no token has already been exported?

    $ cat .npmrc
    //registry.npmjs.org/:_authToken=${NPM_TOKEN:-<READTOKEN>}

...where <READTOKEN> is to be replaced with a hardcoded token.


Fixed, thanks @elieux


breaks function


Dialyzer told me that this clause can never match. It should be removed.


You can skip this, I'm just exploring this site.


Al di là che sia corretta o meno la soluzione applicata, questa modifica cambia il comportamento del programma. Nella nuova versione solo `throw new RuntimeException(e);` verrà eseguito in caso di gestione di eccezione, mentre la vecchia gestione `CommandLineUtil.printHelp(group);` verrà **sempre** eseguita dopo l'intero blocco try/catch. Bisogna sempre stare attenti alla gestione delle parentesi graffe. Un blocco `{expression;}` in java ha la stessa valenza di `expression;`, e visto che gli spazi bianchi non contano, non bisogna lasciare che l'indentazione ci inganni.


Yes indeed. Just pushed a fix.


There are two people porting your mod to 1.8.9. That's amazing, in a way.


:hug:


This line is very long.


colon - двоеточие. Причем здесь твоеточие?


And so is this one.


@redox sorry hadn't seen your reply.
In Markdown to add a line feed, you have to put two spaces at the end of the line and then go to the next line.
So this would not fix the issue I had with not being able to add line feeds ^^


Thanks man :+1:


why removed?


Extra comma at the end of the line. Must be removed.


тут еще надо выяснить, является ли простой тип в коре допустимым.
вот, например, JSON.parse его возвращает без проблем.
надо ли и тут так сделать? а то сейчас возвращается
{
"null": "string"
}


the old test used to press 'up' here


Are you stuck with something related to pg-promise? Need help? Looks like you are executing a query, passing it an empty string or a non-string value...


@phlax @unho re #4482: there was no need to run refresh_stats before.
seems the issue is a regression introduced by this commit.


We now have [![Build Status](https://travis-ci.org/ocra-recipes/ocra-recipes.svg?branch=master)](https://travis-ci.org/ocra-recipes/ocra-recipes) on OSX


picked


OK, let's leave it. I'll rework it later.


тут, наверное, можно не создавать child?


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 293](http://ci.infinispan.org/viewLog.html?buildId=35514&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for 8d4ddefd5fe9d9b28e614d37358fbc90598057a8
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:01:27

Process exited with code 1


Here is the reason:

ActivityRestResourcesV1 use EntityBuilder class to build the response
The ActivityRestResourcesV1 is declare as a component key  without type in service-configuration.xml
EntityBuilder is sample class not declared in configuration.
What we need to change current behavior is to modify getActivityStream to add the needed permission
https://github.com/exoplatform/social/blob/4.3.0/component/service/src/main/java/org/exoplatform/social/rest/api/EntityBuilder.java#L415
So in order to overload the needed method in the rest service we need to do it for all methods calls :

getActivitiesOfCurrentUser : https://github.com/exoplatform/social/blob/4.3.0/component/service/src/main/java/org/exoplatform/social/rest/impl/activity/ActivityRestResourcesV1.java#L101

getActivityById : https://github.com/exoplatform/social/blob/4.3.0/component/service/src/main/java/org/exoplatform/social/rest/impl/activity/ActivityRestResourcesV1.java#L140

getCommentsOfActivity : https://github.com/exoplatform/social/blob/4.3.0/component/service/src/main/java/org/exoplatform/social/rest/impl/activity/ActivityRestResourcesV1.java#L256

getLikesOfActivity    :https://github.com/exoplatform/social/blob/4.3.0/component/service/src/main/java/org/exoplatform/social/rest/impl/activity/ActivityRestResourcesV1.java#L347

Same thing for SpaceRestResourcesV1 where we need to ovveride the method buildEntityFromSpace from the class EntityBuilder called by almost methods in the SpaceRestResourcesV1

So I see it's more sample and cleaner to override then to ovverload all methods

WDYT ?


For already provisioned boxes without GeoDjango installed, please pull latest source and run this manually to update:

1. sudo apt-get install -y postgresql-9.3-postgis-2.1
2. sudo apt-get install -y binutils libgeoip1 libproj-dev gdal-bin python-gdal
3. sudo su - postgres
4. psql -c "CREATE EXTENSION postgis; CREATE EXTENSION postgis_topology;" opendims
5. Done

Check if PostGIS is enabled on opendims database:

1. psql -U vagrant opendims
2. select postgis_version();
3. \q


Hi Kushank, Thanks for taking care of multiple tickets. The new functionality appears to be good, please see my comments inline. Also, could you add some tests related to the new deletion functionality ?


@ta2-1 - im not sure if there was discussion about it at the time - but i removed it intentionally as i thought that was what was wanted.


No Guava version yet in mpt.


Ah, that would have been a cool idea!


padding:0 0 0 0 ;


breaks function


let's begin!


For me, curl uses UTF-8 (probably the system encoding actually), and `ü` looks like `[0xC3, 0xBC]`.


sgdgsgeggre


processNativeFunctor?


Very nice!


sdgregdfdhd


Nice work :)


A cosa serve questa modifica? Per quale motivo è stata aggiunta una classe non richiesta con lo stesso nome di quella della libreria di Google?


@pynicolas Hm... good question. To be precise no, they are different, see:
```
class A {
  foo(){}
}

var x = new A(), y = new A();
x.bar = function(){}
x.bar(); // ok
y.bar(); // fails
```


picked


Why do we need this interface? Shouldn't the "kind" of subform be determined entirely by it's key field?


Corresponds to PR https://github.com/joomla/joomla-cms/pull/8676


I have removed the gray background of images because they look like really ugly with it. I just removed it, now logos have transparent background (if they don't have it yet) and they look beauty ^,^


well welp


@phlax it seems we talked about other things. I didn't notice this change then. But we can see now that we need to update cache after we actually recalculate checks and they are changed.


@adaur status column needs to be smaller. ping me when in IRC


I failed to find an answer in RFC 6550. I agree in essence but there is one case where this reset might still be needed: if we do not have any other suitable parent to switch to. In this case it is a good idea to reset the timer and let the rest of the network know that there is something going on, hoping to find a way to join the DODAG soon.


opkop


Angler :P


Perfect!


Due errori in una modifica sola:

1. fare downcast di `weight` da `double` a `float` potrebbe far perdere precisione al dato e quindi generare un errore; esiste un metodo `Double.compare(double, double)` che fa esattamente quanto necessario.
2. il confronto va fatto con `1.0` non `0`


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


@NicolaeGrigoriu: `administrator` should have the same permission.


Found another option:

The token can also be passed as a cli argument, like so...

`npm --//registry.npmjs.org/:_authToken=<writetoken>`

This allows to alias npm and thus overrule the tracked read token:

`alias npm=npm --//registry.npmjs.org/:_authToken=<writetoken>`


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


Errore. Questa modifica cambia il comportamento del programma


Remove this module.


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


log scale?


Check style violation, @e-cuellar please fix


yep

```
...
0030: 55 73 65 72 2d 41 67 65 6e 74 3a 20 63 75 72 6c User-Agent: curl
0040: 2f 37 2e 34 33 2e 30 0d 0a 41 63 63 65 70 74 3a /7.43.0..Accept:
0050: 20 2a 2f 2a 0d 0a 47 45 4f 49 50 5f 43 49 54 59  */*..GEOIP_CITY
0060: 3a 20 44 c3 bc 73 73 65 6c 64 6f 72 66 0d 0a 0d : D..sseldorf...
0070: 0a                                              .
== Info: Empty reply from server
== Info: Connection #0 to host localhost left intact
```


Well this mostly comes from playing with the Travis non-regression tests we have in the repo. Several of them expect relatively quick recovery from temporary disconnections, and a low alpha is too slow to pass the test. The link-stats module shouldn't have (too much) effect there. For real deployments it still is a bit unclear to me (1) how much does alpha matter and (2) how to pick the best value.


You should use `rem-calc` here as well.


@NicolaeGrigoriu: Do you need to export all the modules with `'value' => 0,`?


Check style violation, @mbatchelor  please fix


Тогда и на 17 строке давай это же укоротим)


Hello Aerendir,

I have just created the pr: https://github.com/commerceguys/guzzle-oauth2-plugin/pull/29 but as we don't use travis, I didn't update it and now it failed, I will try to fix it during the week. Meanwhile, let's hope the authors will answer :)


we alreday have such constant in DateTime. I believe it better to use them


Ok. Il modo standard di eseguire questa modifica è invertire la condizione dell'if, mettendo all'interno l'assegnamento, e fuori dall'if ritornare in modo incondizionato il valore dell'attributo.


Should we operate with `roleId` or `roleNames`? /cc @AnatolyMuntean


Hmm, might be.
But what is more important is that 7f98e1635f9d61292ae2dbbebdc73849c4208e17 is completely wrong.
Will push proper code soon..


Did you check if SHARE_DOMAIN is set in all new-squerb layouts and squerb-frame?


Логично


Errore. Cambia il comportamento del programma. Un modo di fare questa modifica è trasformare il while in un for.


This value should be 1.


Very good! :) Thank you for the fast response! :) Very good work! :+1:


Log scale would throw all bars into negative as all perf numbers are subzero. Are you suggesting a variation of the log scale?


This module is not in a default DDB CMS build. It should be removed.


OMG, gotcha.


:ok_hand: I just copied these css from squerb repository 🙂


Note that

```
<br />
```
is only valid is you use xhtml, if html4 then
```
<br>
```
is perfectly valid.


@silentroach This is most certainly not the commit that you are looking for, btw — it deals with outgoing headers, and your issue is with incoming headers, it seems.


I agree on resetting Trickle only if the rank changed significantly. I thing resetting at every parent switch was mandated in some early draft of RPL but doesn't seem to be in there in the final document.


So:
```C++
//1)
	const_iterator find(const key_type& _Keyval) const
		{	// find an element in nonmutable sequence that matches _Keyval
		const_iterator _Where = lower_bound(_Keyval);
		return (_Where == end()
			|| _DEBUG_LT_PRED(this->_Getcomp(),
				_Keyval, this->_Key(_Where._Mynode()))
					? end() : _Where);
		}
//2) assignment of "auto it"
//3) comparison of "it" with "overrideSettings.end()" which internally;
const_iterator end() const _NOEXCEPT
		{	// return iterator for end of nonmutable sequence
		return (const_iterator(this->_Myhead, this));
		}
```
versus checking alone boolean variable. So, usage of ```overrideArrayIsUsed``` increases performance


picked


IMHO the fieldsets in mercator forms look a bit strange with the double border:

![fieldset](https://cloud.githubusercontent.com/assets/202576/13045202/5903e756-d3d2-11e5-8fcf-6fc53113485a.png)


Да, ты прав. Добавление embeds_many решило проблему


I liked the tinted background in fields. Maybe we should ask @stadine about this.


Mmm ... I'm not sure I got it. Right now we have ```rootFormClass``` that has ```FormField``` with type ```SubformType``` that references ```subformFormClass```. This ```subformFormClass``` has ```keyField``` that keeps ```SubFormKind``` id. By id we have subform kind definition which is again ```FormClass```.


well those slashes are in the wrong direction for Linux, I changed it to:
`#include "../hardware/1Wire/1WireCommon.h"`

and seem to be having much better luck.


OK


Should probably only migrate settings explicitly changed by the user with `g_settings_get_user_value()`


> `encodestring()` returns a string containing one or more lines of base64-encoded data always including an extra trailing newline (`'\n'`).

> `b64encode` <...> The encoded string is returned.

[doc](https://docs.python.org/2/library/base64.html)


IMHO these are quite specific to mercator. Why move them to core?


breaks function


@NicolaeGrigoriu: It doesn't look like you exported from a plain `ding2/ding2` instance.


"Deprecated API usage: getDocument is called with pdfDataRangeTransport, passwordCallback or progressCallback argument"


This comment doesn't appear to match the `#if`. I'd recommend deleting the comment. I would rather we not add these sorts of comments and not require them in our style guide, since they are a pain to maintain and any decent text editor can indicate which `#endif` goes with which `#if`.


@andreaskring TODO: fix the indentation on ```siard/out/metadata/TableIndexFileStrategy.java```


ack assuming it works


ack


I guess this one should never be used except in some very rare use case. We should try to see if we can hide it a bit more.


**Project Tracking software**


Realice los cambios en la maquina de Carlos Valerio por eso se subio con su usuario.


fellow udacian here. There is a problem with the repo for windows users about not allowed characters in "\slides\1-8-02-Icicle:Head Collision Detection.tif" I think it would be beneficial for everybody to find a replacement for the ":"


Should be removed.


Sale here, i am wondering if we could have a CTOR that only accepts non null, and some kind of factory method that makes it clear in the name that we build an incomplete context (e.g catalog use case where accountId does not make sense,...)


Needs to be translated:
``````
<b><?php echo xlt(Backup); ?></b>


Cool... that will help resolve issues in a quick manner...


CucumberOption의 주석처리문을 제거해도 될 듯합니다~


Well, they... do need it.  Perhaps this is poorly worded.  The regular context for `apt*` tools is for them to run on `/` at large, and touch many files which have owner uid=0, and so for that purpose, they need "root"... but it's also seeming quite typical ("in practice", on my wildly non-scientific sample of things-I-happened-to-do-last-week) no other capabilities are needed.

I'll work on words'ing better...


The explain plan tests looks good.


What does 'home' do here?


What made you choose to declare a global variable for f1 and f2?


/cc @jdalton

Still have one failing test with `transform` for object realm. When transform is provided a `function() {}` the `new Ctor instanceof Ctor === true` (when would this not be the case)?

```js
return result instanceof Ctor || !(new Ctor instanceof Ctor); //returns false for function() {}
```


Great! Thanks for checking if the file exists!


Removed it into the controller.


It is done


`admin.create`?

Are you creating an admin or a user? :smile:


TODO


I do believe that a better import would be `from ggrc import app` since it's already used elsewhere to fix the import issue and it's basically the most propper fix for this. It will also make it easier to remove all these "import fixes" once we do a refactor.


@WellingGuzman Fair enough – though I think it was added in for when errors have text with no spaces... those will now run off the page and not be visible.

Either way it'll be ugly... :disappointed:


Hi John, just wondering why you reverted this config file to comply with RuboCop v0.36.0. Any inputs?


This seems odd to do


들여쓰기가 이상해요


Dick...

Shoot. I wasn't really thinking about compaitability, this code is only compaitable with Cablecast > 6.0 as it uses new APIs. I'm a bit swamped right now, but I'll branch this into `7x-2.x` and revert the commit on the `7x-1x` branch.


I wonder how many statistics will count this as accidental commit of sensitive data? :D


I did not verify that this is just a copy, but LGTM assuming it is.


May I suggest you put this after the devise, I like having the entry point of the application to be defined and visible early in the routes... but thats my personal preference.


Yup, it's in the works ;-)


You're right, there's no need to modify the array. I'll submit another PR.


```
-106	images/backgrounds/bg-tile-uniform.png
-553	images/backgrounds/bg-tile1.png
-6725	images/gist-logo-white.png
-41911	images/screenshots/after_red.png
-41686	images/screenshots/before.png
-53785	images/screenshots/code_solarized_dark.png
-55309	images/screenshots/code_zenburn.png
-8719	images/wiki/add-syntax-theme-option.png
-2846	images/wiki/background-image-removed.png
-4750	images/wiki/background-image-select.png
-1774	images/wiki/find-and-replace-base-color.png
-7509	images/wiki/find-and-replace-base-color2.png
-4627	images/wiki/login.png
-3785	images/wiki/open-editor.png
-24989	images/wiki/select-and-paste-code.png
-10006	images/wiki/sort-theme.png
-726	images/wiki/syntax-highlight-removed.png
-12138	images/wiki/syntax-highlight-selected.png
total -281944
```
Still have to [zopfli](https://github.com/google/zopfli) them, but that's going to take a few hours or days :stuck_out_tongue_winking_eye:


+100. LGTM.


Shouldn't this just be `DUK_USE_DEBUGGER_INSPECT`?


These constructors should never be called directly (the factory is `InternalCallContextFactory`).


Correções adicionadas.
Claro que sim.Agora eu vou começar a esperar o commit antes de mandar outro PR já que não consegui enviar separado :(


This is great!


@chintanb Please review! :*


Looks like that was a lot of fun .... :+1:


added HTML files for twitter presidential election candidates


Remove these comments. (Only `PourTransaction.cpp` should include them, which is why they were commented out.)


Please open a new issue with your question over here:
https://github.com/Homebrew/homebrew-science/issues/new


Indeed! :sweat_smile:


> .NET core platforms

.NET Core platforms (capital C)

I noticed that two spaces are used between sentences throughout, and a number of style manuals would call for a single-space when using a variable-width font, as will be the rendered output; besides all of that, I think the two spaces are only rendered as a single space anyway in the output, which makes this comment a **total** waste of our time. :smile:

> ... (eg: BindingRedirects ...

somewhat non-standard ... are you sure you don't want the more usual:

... (e.g., BindingRedirects ...


LGTM.


:mount_fuji: - I am familiar with emacs' indentation.

Here is master:
https://github.com/clojure/math.combinatorics/blob/master/src/main/clojure/clojure/math/combinatorics.clj#L212

Which (presently) is this SHA:
https://github.com/clojure/math.combinatorics/blob/602a1128340989ae1829336394ecef65d5cc7662/src/main/clojure/clojure/math/combinatorics.clj#L212

Anyway, this looks like a neat library - I look forward to depending on it some day.


More comments to clean out.


> ... Standard application, and other target ... (comma splice)

... Standard application and other target ...


This is final


A feature that requires a reboot doesn't require a call to recreate statusbar. It is unnecessary.


Read through it; seemed very mechanical. :+1:


See https://github.com/killbill/killbill/commit/cb62783039a04500605329b23c155944e246dec2.


What happens when you interpret an `isize as i64`? Can you suddenly count further?


BUT WHERE IS THE PART THAT ACTUALLY SHIFTS THE GEARS TREY DAMMIT


your new index.html looks good, but the Spanish game html, we want change the <table> to <div>, and make the layout better. Never mind, I will do the rest changes.


Why are these not needed?


Mentioned in my previous comment. They're not exposed outside of libzerocash.


Ah, that's definitely an oversight.  I'll fix in next version, if not before.


Why are zerocash/zerocash_pour_{gadget,ppzksnark}.tcc not needed here?


> Why is there only one version of `.NETStandardApp` (1.5), but there are many of `.NETStandard`?**

Not technically a comma splice (subjects & verbs ok), but it seems a little clunky phrased this way. How about ...

Why is there only one version of `.NETStandardApp` (1.5) but many versions of `.NETStandard`?**


I think the commented-out lines can just be deleted.


In case you want to make the `if`s less repetitive I'd use something like:

```nix
with (if enablePlexPass then {
  version = "0.9.15.3.1674";
  vsnHash = "f46e7e6";
  sha256 = "086njnjcmknmbn90mmvf60ls7q73g2m955yk621jjdngs4ybvm19";
} else {
  version = "0.9.15.2.1663";
  vsnHash = "7efd046";
  # Compacted this to base32 using "nix-hash --type sha256 --to-base32"
  sha256 = "1kzr826khn0n69mr2kbr5hxcb56mj12fryhwr95r3132gj02aqph";
});
```

Or maybe instead of the `with ...;` use an attribute and `inherit` it in the `mkDerivation` attributes.


Hi


Otherwise LGTM.


this Prof Kanan's website http://moudgalya.org/ check his name


It would be better to say 'Enter 1 to send a thank you, 2 to create a report...etc' so the user doesn't have to remember which option was which.


Looks great when new theme is used, but not very functional with the older themes.


The basic notion of creating a `Databaser` type which is then used to perform database actions is taken from jex-events, but the specific implementations are unique to this service.


We were thinking of doing the talks as .rnw not .tex, so that they are fully automated. Which is essentially just taking code from main-figures.rnw and main-tables.rnw and adding the frame titles like you've done above. The beamer style will then handle the formatting.

I originally first thought that re-making the figures and tables would be overkill (yes, they're already made in knitr-cache so why do knitr again). But you need to be sure that your knitr-cache is based on the model run that you're talking about in the beamer .rnw code. And it sounds like we may have to re-run the model with new catch levels during the meeting - having the talk automated will make that easier. Sorry - Chris and I may have chatted about this briefly when he was still in my office after the conference call on Friday.

So the above file should be helpful as a start to see what should go in the talk. But maybe let Chris start on a structure tomorrow so we're not duplicating effort. Once the structure is laid out it will be easy for us to add bits and change frame titles etc. I expect tables will be somewhat easy also.


casefold... NICE!


@benhaynes Ohhh I understand. so. what should end up with?


These buttons on this script look bad on the old themes.


La variable "_" debe ser mínimo 3 caracteres para que no marque error de pylint


I'd leave it in I guess – better to read the text and it's worse if it runs over.


This is basically to mimic a lazy sequence. This is used in `templeton/elasticsearch` to crawl the database.

An alternative implementation would have been to mimic the `sql.Rows` behavior: create a new type which supports a `Next` method which returns either true or false which can then be looped over to update some other method of getting at the actual data (for `Rows` it's `Scan`), and which stores errors to be pulled out of an `Err` method, plus having a `Close` method to call at the end. I'm not sure if this would be better or worse.


x


This is here because while crawling, we always have to read a row past the end of the rows corresponding to a given file. So when we start the next one, we need to pull in that extra row.


Obviously not - that was just a joke.  You'd need a function that would interpret the wrapped around negative value.


To Avoid Clang Warning Error.


Fixed by commit: f3e53d7401e7cc92a129ed38bea45f5134e986b7


This is a slightly weird thing to do; I'm trying to mimic what the `Databaser` type does with `Ping` -- make sure that the connection basically works. So I just ask for it to get a status of red or better (i.e. anything).


PR: https://github.com/JuliaLang/julia/pull/15091


I'll check this.


Try commenting this line and the following line out and printing the value of Amount to make sure your function is doing what you wanted it to do...


@tobrun Can you help me understand why `removeAllAnnotations()` was changed to `removeAnnotations()`?


So... this works perfectly for Google Maps, but the Google Fonts don't get applied to the things from the initial page load (they work ok on everything thereafter, however).

Is there a function I can call to have the browser re-render text or some other way I can apply the newly fetched font to things already onscreen?


Can you add an exception here to make sure the file exists? If not, can you ask the user for a name?


By the way, I just discovered that I can grab the commit SHA from git log rather than searching for it on github. Much easier! It would be cool if you could configure git to dump the SHA on the command line after each commit. Maybe you can. I'll have to do some research.


Change README.md


> `.NETStandard` is an abstract representation of API that covers all historical platforms that have ever supported that API.

I wish that could be stated a little clearer ... perhaps it just can't. This might be one of those cases where going ahead and using three sentences instead of one to explain the concept would be a good idea.

> We are shipping ... (contraction used elsewhere, so use them everywhere)

We're shipping ...

> ... implementation at this point and it supports `netstandard1.5`. (Independent clauses normally require a comma at the conjunction; however, the second clause is so short that this is probably one of those optional cases where you could just go with what you have. Alternatively, you could go to "which" or "that" here.)

... implementation at this point, and it supports `netstandard1.5`.
or
... implementation at this point, which supports `netstandard1.5`.
or
... implementation at this point that supports `netstandard1.5`.

> As we version `.NETStandard` in the future we will ... (comma required for adverbial introductory clause)

As we version `.NETStandard` in the future, we will ...


Thanks. That was bugging me too. lol


:+1:

Finally got the fixes for the blocker issue working - I'll take a look into this in the morning and if it works I'll include the fixes with my PR.


yes, but not works, waiting fix :/


With the alternative implementation, this would do something like pull a `cursor` object, `defer cursor.Close()` and the loop below would be (roughly) `for cursor.Next() { ids := cursor.Get() <format, marshal, queue to be indexed> }; return cursor.Err()`.


I don't think half of those are still being used anymore.


> For compatibility purposes this moniker ... (prepositional introductory clause needs a comma)

For compatibility purposes, this moniker ...

> ... for “.NET Core applications”. (period inside quote)

... for “.NET Core applications.”


unfortunately, It does not work just to bind it to the variable listOfDefaultQtys. It needs to have a option tag to populate the Select list. Just I could define a array listOfDefaultQtys for 1-10 numbers.


Putting the spaces like the above way does not work at all. I was wrong.
Removed them.


done


> ASP.NET V5 applications

V5?

> As such the TFM already had more characteristics ... (adverbial clause ... comma needed)

As such, the TFM already had more characteristics ...

... also RE: "TFM" ... that's used here a second time, but I don't recall seeing it ever defined as "Target Framework Moniker" anywhere above.

> ... with .NET standard application.  .NET standard applications need ... (capitalize "Standard" as above?)

... with .NET Standard application.  .NET Standard applications need ...


All done


should propagate the error instead of asserting


Now directive is working as it should :) :+1:


this looks bad when use the old themes


I believe we just want to check the _remainingCellsToReveal is zero and not check the flaggedMineCells.Count(). Apparently, flagging mine cells is completely optional! http://www.minesweepers.org/object.asp


should probably log the error (maybe later to a mirroring error log) rather than asserting


Yeah, some of these are ancient. Had to remove the commit as I was misusing pngcrush and it zeroed out some. Will follow up with something better :)


Feels out of order. Shouldn't this Q&A be above the ...

> **Q: Why is there only one version of `.NETStandardApp` (1.5), but there are many of `.NETStandard`?**

... question?


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


The state query parameter is a security feature to pair up requests and callbacks towards the server.
This is a security feature. [Relevant Google search](https://www.google.ro/webhp?sourceid=chrome-instant&ion=1&espv=2&ie=UTF-8#q=oauth2+state+parameter)

I removed this piece of code because passport already does these checks, thus this is redundant code.

Relevant parts of Passport that do this: [1](https://github.com/jaredhanson/passport-oauth2/blob/9b69f06de16287a5254e18c1f6a419015b53f213/lib/state/session.js#L36) [2](https://github.com/jaredhanson/passport-oauth2/blob/9b69f06de16287a5254e18c1f6a419015b53f213/lib/state/session.js#L62) [3](https://github.com/jaredhanson/passport-oauth2/blob/28470d8c7fb2ef89662ac198138ae76c5cf95b78/lib/strategy.js#L216) [4](https://github.com/jaredhanson/passport-oauth2/blob/28470d8c7fb2ef89662ac198138ae76c5cf95b78/lib/strategy.js#L152)


REVIEW PLACEHOLDER


Sorry, I read the `!==` as `===`.  How about

```
({MSIE:1, Safari:1}[MathJax.Hub.Browser] ? '.mp3' : '.ogg')
```
or go back to your original.  Sorry about the error.


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


vcsb


Hola, gracias por tu trabajo.
Siempre que tento abrir qualquer stream tengo erro: Error de reproducion.
Se dices que los streams funcionam, pudes indicar una lista m3u que lo saibas que funciona?
Otra pergunta: hay logs para puder analisar?

Gracias


testing import at least (and only) assures that the module exists and doesn't have syntax errors.I always start with a really simple test that imports, and usually one that instantiate the class if there is a class int eh module -- basic tst driven development -- I do that before even writing any code.

Once you have  a bunch of real tests in place those become superfluous, but it's still nice to have them there to fail early if something catastrophic happens.


How about making x randomized and calculating `y` with the formula `det(A) = ad - bc`? This change makes the tests more robust.


 - support reveal spoilers in 4chan x
 - start supporting bottom header (why?)
 - start styling the thread watcher
 - start styling the settings
 - copy paste ccd0 styling for quoting


Den Haarscheidt habe ich im Betrieb umbenannt, so etwas müssen wir auch automatisch abfangen.


Huh!? Why did this end up attributed to user "yashikno"??
I dunno quite what I've done wrong, but somehow, after messing around with git config,  I got the same commit but with the correct author.
Ah well...


>  As such `NETStandard` assemblies ... (comma)

 As such, `NETStandard` assemblies ...

> ... that assembly, for example: .NET Desktop, Windows Phone, Universal Windows Platform applications, .NET Standard applications, etc.  (suggest parenthetical note, standard abbreviation e.g., and period at the end)

... that assembly (e.g., .NET Desktop, Windows Phone, Universal Windows Platform applications, .NET Standard applications, etc.).

> ... .NET standard applications ... (consistent capitalization)

... .NET Standard applications ...

> .NET standard applications are runnable on their own.

"runnable" ? ... "executable" ? ... would that be better here?

.NET Standard applications are executable on their own.

> .NETStandard ... (space here)

.NET Standard ...


What does exit() do here?


Actually can you make it so that the hatchet binary is put into a bin folder and we just ignore all bin folders?


I'm good for whenever. I just need a little notice. Chat, I think, would be best.

Sent from my iPhone

> On Feb 15, 2016, at 2:29 PM, mariahcann1995 <notifications@github.com> wrote:
>
> So everyone we have something due for this on the 23rd. So we need to make a plan what we want to accomplish and all. I would like to do a chat when everyone is free I am free everyday Monday-Wednesday then anytime after 5. We need to get this done.
>
> —
> Reply to this email directly or view it on GitHub.
>


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


FYI usually we put the TODO above the line with the needed action vs below it.


You are absolutely right! I got confused with an app that I'm upgrading. I apologize.


This commit was fixing stuff hat was already broken (at least as far as the Flake8 was concerned) Sometimes is better to just get master working and not get hung up on process too much.

The `io_util.py` module is a work in progress, so yes, should probably be in a branch , but it's also not touching anything else, so not getting in the way. On a branch, it would likely get lost in the shuffle.

"branch early, branch often" only works if you also merge early.


Fixed by commit cb09bcaa9935dc0cdbb596dbf86e566c27a7fe72


> **Q: How is this different than `.NETFramework`, AKA: Full .NET, Desktop .NET, Big .NET, old .NET?**

"AKA" ? ... how about a slashed list or comma-separated list? Shouldn't there be a space between ".NET" and "Framework?"

**Q: How is this different than `.NET Framework` (Full .NET/Desktop .NET/Big .NET/old .NET)?**
or
**Q: How is this different than `.NET Framework` (Full .NET, Desktop .NET, Big .NET, or old .NET)?**


> "branch early, branch often" only works if you also merge early.

Now that I have merge rights I can review and merge your PRs.  Feel free to ping me.


> Just as sanity check...

I don't deny that your are right.  Only that it didn't always work that way in the past (see [quirks mode](http://www.quirksmode.org/js/events_early.html) for instance, or google ["javascript event handler return false"](https://www.google.com/?gws_rd=ssl#safe=active&q=javascript+event+handler+return+false)), and that's where I got in the habit of doing it.  I don't know which version of IE saw the change (I would expect IE9), and since we have still been supporting IE8, that's why it has continued to be used.  But this code includes other features that are not IE8 compatible, so it is not a big deal.  Go back to the original form if you wish.


I added `support/build.test.js` in a later commit that does a simple smoke test of the build artifacts.


You are showing all of the right content, but it does not seem to be styled properly.

**Your file:**

![image](https://cloud.githubusercontent.com/assets/1809908/13064059/168e10f4-d41d-11e5-8cfc-40e9d94f178b.png)

**What it should look like, with each school represented as a distinct box with a grey background color:**

![image](https://cloud.githubusercontent.com/assets/1809908/13064065/2eea3010-d41d-11e5-93ec-40c20b36c3e4.png)

To achieve this, you will want to create a reusable component by wrapping each school in a <div> element. Then give each <div> element a class and define that class in CSS.

Here is an example of what this could look like: http://codepen.io/lknarf/pen/OMqvjq


Jep - fällt ja hier sowieso nur auf, weil wir das noch von Hand pflegen. Wenn das automatisch läuft, würde ich die Node-Namen aus dem Dateinamen rauslassen, die können nur Probleme machen.
(Statt dessen vielleicht als Kommentar in die Datei rein, falls da mal ein Mensch reinschaut.)


> would that mean that line 222 should return true?

That would probably be best, but I'm not sure any longer what the result for returning null was.  I don't want to start up a Windows VM to test it out, since it probably doesn't matter any more.


Hola, gracias por tu trabajo.
Pero siempre que tento abrir un stream de qualquer lista m3u, tengo "Error de reprodución :("
No tengo firewall e analiso puertos com curports e aparente no haber problemas.

Se dices que el problem es de los streams, pudes dar un url de ejemplo de streams m3u que funcione?

Outra pergunta: hay logs para poder analisar?

Gracias

![rtv_error](https://cloud.githubusercontent.com/assets/17261818/13064102/9dd0b86c-d447-11e5-91b6-cbcbfbecfe16.jpg)


> **A:** `.NETFramework` is another ... (space?)

**A:** `.NET Framework` is another

> ... a different set of API. (plural?)

... a different set of API's.
or
a different set of APIs. (more common in the official docs not to use an apostrophe, but I sort of wish they had gone the other way with it :smile:)

> As such a `.NETStandard` assembly ... (comma)

As such, a `.NETStandard` assembly ...


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


hello


pngcrush'd and zopfli'd  :heart_eyes:

```
for f in *.png; do pngcrush -rem allb -brute -reduce -ow $f; done
for f in *.png; do zopflipng --iterations=1 --splitting=3 --filters=01234mepb $f $f; done
```
```
-48065	images/screenshots/after_blue.png
-1835	images/screenshots/after_red.png
-1728	images/screenshots/before.png
-1462	images/screenshots/code_solarized_dark.png
-1670	images/screenshots/code_zenburn.png
-234	images/wiki/add-syntax-theme-option.png
-72	images/wiki/background-image-removed.png
-78	images/wiki/background-image-select.png
-16	images/wiki/find-and-replace-base-color.png
-16	images/wiki/find-and-replace-base-color2.png
-28	images/wiki/login.png
-49	images/wiki/open-editor.png
-852	images/wiki/select-and-paste-code.png
-152	images/wiki/sort-theme.png
-17	images/wiki/syntax-highlight-removed.png
-685	images/wiki/syntax-highlight-selected.png
-25249	images/wiki/userstyles-options.png
total -82208
```


@rochefort Fixed! Thanks.


Als Dateinamen dann nur noch die ID da die in aller Regel unveränderlich ist. Schlecht zu lesen aber sinnvoll, in den Kommentar würde ich den Knotennamen dann aber aufnehmen.


Fix #14


It's because mu' with 0. Perhaps we should add another test?


Since each person chooses the Dog vet game a couple ideas for starters ...
1. breads of dogs
2. bathroom bubbles for dogs
3. simple medical tools(not extravagant though age appropriate)
4. dog clothes
5. sicknesses (age appropriate simplified) for exemplar...my dogs sick he's throwing up food-- or my dogs limping because he jumped a stool and didn't clear "could bandage the leg"?
6. Hair cutting styles
7. could make a location for a vetnary clinic in a certain rural area ,designing and choosing colors, building structure, dog baths, beds, tools, play houses, as well as dyes for your dogs hair..mo hawks, bows.


## Task 1

- [ ] Program is correctly formatted with shebang, description, code (1 pt)
- [ ] Program includes `main()` function and "ifmain" statement (1 pt)
- [ ] Program includes additional function(s) to compute "palindrome-ness" (2 pt)
- [ ] Program correctly determines whether **any** phrase or word is a palindrome (6 pt)

## Task 2

- [ ] Program is fixed - it is still recursive and works correctly with no errors (5 pt)


Woops, sorry about that @Yive !


This shouldn't happen in ``dryrun``.  Don't collect data in dryrun


@khanson679 I'm pretty sure you can put colons in a map key with quotes. You definitely can do it with values. From what I remember reading of the spec, I think you can also do it with keys, too, but we should check.

And gotcha, that makes sense. Though not all of our entries will need to have anchors. Only those that we want to reference in other entries.


I've built a test to confirm this (https://github.com/tarkatronic/django-axes/commit/b245ce2fac04f4b8a8f542ddbf119eaddecfd9c9). I will work on fixing it soon.

I also added a tox.ini on that branch, and updated the Django versions to test against in Travis.


@khanson679 :+1:


Hey dan, all I could think of when you were explaining it was this.
https://www.youtube.com/watch?v=PLxPj25Frm4


Final fonts


@khanson679 Sorry, I don't see the commit that you're talking about? And why should blank fields be removed? In either case, though, we'll still need to figure out how to handle multiple names for the same author. Perhaps we should open an issue for it?


Part 1
Write a function named 'pollutantmean' that calculates the mean of a pollutant (sulfate or nitrate) across a specified list of monitors. The function 'pollutantmean' takes three arguments: 'directory', 'pollutant', and 'id'. Given a vector monitor ID numbers, 'pollutantmean' reads that monitors' particulate matter data from the directory specified in the 'directory' argument and returns the mean of the pollutant across all of the monitors, ignoring any missing values coded as NA. A prototype of the function is as follows

You can see some example output from this function. The function that you write should be able to match this output. Please save your code to a file named pollutantmean.R.


hello world


closes #1


Hi, thank you, sorry for that, I didn't look well.


+1 filter operation


+1 "data" is the response


sport huge amounts of?


+1 factory


+1 factory-like


+1 ancillary parameter


delete "are"


sda


+1 filter


上部のform_for@restaurantでは店名登録が可能であるのに下部のfield_for@restaurantでは登録できない仕組みの違いについて教えていただきたい
扱っているのは同じrestaurantではないんですか？？


+1 factory-ish


this footnote may not be necessary?


+1 query


+1 factory/query


This bug is also present in 2.8.0


I think this polarity change broke the reset when using Vivado 2015.4.
Was this tested with a Microblaze, clock wizard, reset setup?


it emailed me today for this for some reason

i remember this

good stuff


What @Tigercat2000 said, applies here.


let勉強してきました！が、今回は局所的に変えたい変数は（おそらく）ないのでvarだけで行きます


Why those files were removed?


@MarkehMe Nice check for modded status, all plugins should be doing checks in this manner rather than checking the server build.


そしてコミット大きく読みづらくてゴメンなさい。


incomplete sentence?


I don't know that this will work. The index only shows groups for Unresolved errors. If we've just switched all of the errors to Archived then the group no longer exists to be queried. https://github.com/Particular/ServiceControl/blob/master/src/ServiceControl/Recoverability/Grouping/Raven/FailureGroupsViewIndex.cs


发呆时发生的发生地方时代发生地方


Yeah, I slipped up on that one. I'll get it fixed soon!


I think so --- have, provide, feature. (I still changed the wording).


Interesting comment@LegNeato. I don't recall tracking ever being used as blocking. It has always been a list of important bugs that relman want to keep track of for a release. (This use predates my tenure in this position and was carried forward.) At this point engineering sees this as a non blocking list. The point of introducing blocking is to make this point clear. Perhaps long term, if relman doesn't see value in a '+' value, we can just drop it and leave blocking in place.


are --> as


-1 "display" is a noun, not a verb, so this reads like "display name of the location"


@Tkkg1994
If you don't add sched_enable_hmp=1 to your cmdline, you will meet this problem.
But after you add it, this problem will disappear.


+1 "value" is the response


ブロックの中でthisの値が変わってしまうので、ここで変数に入れました。
しかしよくよく考えてみれば引数をただ変数に最代入してるだけで、二度手間なので、後ろのコミットで修正します。


hello world


Another patch to cpu_boost is needed to shut it up.


+1 query


+1 factory


```
$ cat test.cpp
#include <cstdlib>
int main(){void *p = alloca(10);}
```

```
$ clang++ test.cpp -fno-builtin
/var/tmp/test-7dc7d6.o: In function `main':
test.cpp:(.text+0x10): warning: Warning: reference to the libc supplied alloca(3); this most likely will not work. Please use the compiler provided version of alloca(3), by supplying the appropriate compiler flags (e.g. not -std=c89).
```


-1 argument is not the direct object of the base name


@satya164, I'm quite sorry about that! I made a mistake while syncing the code and missed your latest contribution.

I'm busy integrating the UIExplorer, so I can't get to it quite yet. Would you mind rebasing and putting up a PR to RN master?


+1 parameter is ancillary


-1 we're "comparing...with" the argument


-1 splitting fail


@Tkkg1994
I am analyzing Samsung's changes to CAF Kernel.


Can we make this [20]byte instead? We're kinda abusing Go's leniency of utf-8 codepoints by storing these in strings.


Need to test this


"experiment manager tool" -- > experiment manager


"experiment management tool" --> experiment manager


ibus-rime等前端也要`rime_get_api()->set_option(_session, "_horizontal", ..)`嗎？


needs to get put back in


+1 argument is direct object of base name


why they need these interfaces?


make, New and Developing capitalized


verify this is ok


Make New and Developing Technologies, capitalized


+1 query


[Meet-Meet designdoc.docx](https://github.com/Byuunion/Senior_project/files/131719/Meet-Meet.designdoc.docx)

Corrected some typos and reworded a couple lines


I dislike that this package conflicts with the stdlib's errors package.


I moved in this page from commit 33af50614c028b598afc0abd6bd3ba1a2d3436b8 @stillfreefarm


-1 argument is not the direct object of the base name (would prefer "matching" to "with")


+1 factory


pop-up?


@Robotia Thanks, I thought it was the safest method. Unsure of other methods at the moment that would be more adequate. :smiley:


viewの画像投稿フォームにそれぞれ
.thumbail-0 .thumbnail-1 .thumbnail-2というクラスをつけました。
eachで短く書こうという算段です。


+1 argument label required here because argument is not the direct object of the base name


When you debug microcontrollers / FPGAs / wireless routers (for that matter), you'll often need these interfaces to connect to the devices. Not so with S.T., where it's all plain Internet grade technology.


+1 argument label indicates that argument is not the direct object


+1 argument is the direct object


+1 argument is the direct object (BUG: we should drop "points" here)


+1 center/radius/etc. have equal weight


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


+1 arguments have equal weight


Do we really need this? Seems pretty coupled to HTTP.


You can use 'return' here, as well.


'kay. (Wikipedia lists either.)


This could do with a read and corrections by a native speaker ;)


why only x?


このdivを生成するコードは消去して、htmlにあらかじめからのdivを書き込みました。


why only local?


+1 argument is the direct object (should be: appendGlyph)


@clbemre

Yes you can, as long as you follow the license ([Apache License 2.0](http://www.apache.org/licenses/LICENSE-2.0)), which is the most common license for Android libraries.

But please create an issue instead of commenting on an irrelavent commit next time :/


+1 argument is the direct object (should be: appendGlyphs)


@dcarral At the time, our engine was running 0.35.1. We will update this to our new defaults shortly.


+1 argument is the direct object (should be: appendPackedGlyphs)


What is this function doing? Seems like a possible typo.


+1 argument is the direct object (should be: appendRoundedRect)


-1 argument is not the direct object


+1 query


I <3 open source!

https://blog.mozilla.org/channels/2011/06/01/more-details-about-how-to-use-the-tracking-firefox-bugzilla-flag/


Can we use `sizeof(*something*)` instead of MAGIC_NUMBERS in these locations?


It does both: local and distributed experimentation. Thus, "in addition to local", .... "distributed experiments as well".


-1 preposition splitting is failing here (could be: noteHeightOfRowsChanged(_:inColumn:))


it looks like if the string starts with `<?php ...` its possible to end up with `?><?php\n<?php ...`


k


+1 argument is not the direct object


@elmindreda: can you upstream this so that other projects using tinycthread also benefit from the fix?


+1 argument is the direct object


Guess we didn't learn our lesson the last time we tried to pack a bunch of variability into an item's tooltip...


IPFS_API environment variable is not actually used.


Confused by spatial axes, that's y! I remove the gravity thing altogether.


should be z!


You need to touch the panel for powersuspend to work properly. See here: https://github.com/SlimRoms/kernel_lge_msm8974/commit/28b32c1175738b29ebce48be6234f30308742a69


本パッチとは別件ですが、
Python２の場合に、portが unicode となっているせいで
== str に引っかからない件についてです。
Py2,Py3両方でうまくエラー検知できるように作りこむ必要がありますが、
以下URLあたりが参考になりそうです。

[python3のbytes型とstr型の比較と変換方法]
http://python.civic-apps.com/python3-bytes-str-convert/

[Six: Python 2 and 3 Compatibility Library]
https://pythonhosted.org/six/
数値の例で言えば、
six.integer_types
    Possible integer types. In Python 2, this is long and int, and in Python 3, just int.
なので、sixを使うと上手くバージョン差分を吸収することが可能です。
もちろん、岩瀬さんに共有頂いた
http://python-future.org/compatible_idioms.html#long-integers
利用でも可能です。

ご参考になればと思います。


承知です。assetsの方の記述削除します


+0 "frame" is redundant here (BUG: look for blatant redundancy between base name and first argument label)


I don't think we should tolerate null path. That's likely a programmer error.


+1 factory


-1 argument is not the direct object


Brackets { } and remove the ;


+1 query


Too many brackets here?


+0 consistency with related APIs


```@"c:\".Length``` is odd, given that the line above has constants 3 and 2. Just use a constant as well.


Osu!direct works as well.


hello


sed -e "s/^.*PRIVMSG $C.*'$2@$N//g" -e 's/^ //g' .....


+1 argument is direct object


`$'\n'


Isn't that effectively disabling the original `render()` call from above if `isComponentRoute()` returns `false`?


Single if?


Actually the entire method body can be a simple expression.


`case "$*" in (*$i*)`


Read it into an array ahead of time.


assert path != null instead.


The logic of any is wrong, this will never match I think


いると思います。passwordなしでeditできるようにするため、deviseの標準コントローラにメソッドを追加する必要があって、その記述がcontroller/users/registrations_controllerに記述してあるため、呼び出す必要があるかと。


`"$*"` (py:`IFS.get(0,' ').join(argv[1:])`) screws up all extra printf format chars.


Array +=.


Listen, use `[[` or case.


Isn't this the same as

```C#
if (IsDirectorySeparator(path[i]) && !IsDirectorySeparator(path[i - 1]))
{
   break;
}
```

rootLength is >= 1 and i > rootLength no?


Me too! Thanks for the link. Looks like that meaning was lost (quickly?) with tracking. Shame that we're repeating history here. Many tracked bugs have carried over. This flag has not had the blocking weight that it was clearly intended to have. We have also not consistently applied criteria and have tracked bugs that do not have actionable steps, which seems to be correct according to the criteria although perhaps the intention is that we know the next step (investigation) as opposed to we have no idea what to do next (after investigation).

@sylvestre wdyt? Should we stick with blocking or try and get back to the original intention with tracking?


assert path != null


@martell, I think that change is already upstream as of https://github.com/tinycthread/tinycthread/commit/45845262de145e734354890cad33f4c0527a111a.


What happened to the formatting here?


Why don't you just use zsh


Unused `syncHistory`, `routeReducer`.


Also formatting here??


後でkaminariでページネーションを実装した時に直す方が、コミットとしては分かりやすいかと思いました。
今、全てlink_toにした方が良いですか？


Command to run the program on the board ?


rename to buienradar


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


Unused `routeReducer`.


Unused `routeReducer`


huh. Good catch...


I think we already have this implemented below: GetPathRoot


actually i like the unless as it doesn't curl twice. i know this is a bad pattern for readability but this script is dumb anyway.


In test scenarios we have PEReferences without filepaths.


Better use the action creator function, not the raw constant and object literal action.
The constant is exported for someone who wants to react to it in his reducer.


0.3.18 でやっていたんですが、例のWARN解決のために gem cleanup をしたところ、permission denied となってしまい、出来る限りの手段を尽くしてもインストールできませんでした。なぜか0.4.2ならインストールできたのですが、0.3系の方が良いかと思って0.3.20でトライしたところ通ったので3.20にしました


恐らく特に変わったことはないと思います


good catch, I'll fix that asap, thank you John!


@ericvicenti No problem. Will do a PR against master.


Note: that's not how .net works.  This prevents this from being a drop in replacement.  It also makes code that loops really simple.


少し前から気づいていたのですが、このブランチ内でコントローラ名の修正コミットしてもいいですか？
ブランチ切った方が良いかと思って保留しているのですが…


No, root length could be when you have no root.


This is the first step to fixing #8, but it requires that you change the level scale


OK. If .net does it then let's do the same.


Ah. In that case, :-(


commit is wrong. This adds user face icon to repo


And please also remove wtf/build_config.h if it is no longer necessary.


@MarkehMe This is is a fail-safe way to detect Forge. Most people just check if the server type is "cauldron" which doesn't work when the server type is renamed.


(Elaboration: we should figure out how to make one import the other.)


Hrmm... i just tried this out and GetPathRoot doesn't seem to follow the same rules as Windows.  For example, it doesn't think there's a root if you have ```C:``` whereas Windows does think there's a root.


i can change this, but i really don't like magic constants.  using string.length makes it very clear as to where that value is actually coming from.


The idea was to use HTTP status codes but in a generic way. We could use our own status codes and map them to http in the http package, I don't mind either way.


Sure.  Though that isn't necessarily a good thing :)


ImmutableMap.copyOf(definitions)


Yep, you are the experts now and living in the day-to-day...I just wanted to pipe up in case you were not aware of history repeating :smiley: . I personally often find what is needed is _fewer_ options / more clarity rather than _more_ options or added process.

Of course, everywhere I have shipped software has this "blocker drift" naturally happening. People see those bugs getting attention, no one wants to ship bugs, etc. I think it is release management's job to be the "keeper of reality" for the blocker list and find the minimal set of bugs needing to be resolved for a release. To do this they need to defend that process rabidly and make it trivial for everyone else to follow it :beers:


I think this is doing sampling with replacement on the rids, which isn't the same thing as the permutation approach (which is sampling w/o replacement). As xun indicated in #732 earlier this could lead to non-unique rids and neighbors.


So check there that it's not null.


Everything above should be created per-test rather than for the entire environment.


Your createEquipment() method is tagged with ```@Before``` I don't see any reason to call it here.


Cool, I will transfer it then. Need to sleep now 2am.

On Mon, Feb 15, 2016, 23:21 Simon Richardson <notifications@github.com>
wrote:

> I'll give you contributors access tomorrow
>
> On Mon, 15 Feb 2016, 20:55 Christoph Hermann <notifications@github.com>
> wrote:
>
> > I think you will either have to move it or I will have to create a PR
> > against master, since I don't have the rights to push to a new branch on
> > fantasyland.
> >
> > Thanks for the hard work!
> >
> > np, I plan to contrib even more in the future, but I still have to learn
> a
> > lot :-)
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/stoeffel/fantasy-monoids/commit/1439b39be02b6768b49c13abea729928185dceb4#commitcomment-16104374
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stoeffel/fantasy-monoids/commit/1439b39be02b6768b49c13abea729928185dceb4#commitcomment-16105659>
> .
>


This paragraph is no longer correct?


I'm consistently receiving this error "./startup.sh: line 93: docker-machine: command not found"
 upon executing the command
./startup.sh -m mymachine -a myaccess -s mysectetkey -c vpc-7e714e1b -r eu-west-1


@jamigibbs I think it would be very interesting if you added ga (google analytics) to this app along side your custom tracking. You'd see the value of both more clearly.


Unit test method names in Apollo are not prefixed or suffixed with "test"


```C:foo.bar``` is actually a drive-relative path. There is no root. These paths are odd, I don't think we care about them.


In any case, we should not have 2 implementation of GetPathRoot


hehe


I thought there must be at least 10 lines of test code for every line of emulation, but it's very apparent now that most of the effort is in the testing. Even with your experience on the ESS, and it must be invaluable, the rate of robust code addition is beyond impressive.

I'll definitely look over the support routines, installing drivers and anticipating the sound output via the guest OS drivers.


No, just in this case it is ;)


Thanks for review! Now this are right.


The 'header' tag has a display value of 'block' by default, so you shouldn't need to set its width to 100%, as it should automatically fill its parent container's width :)


Yes, something weird must have happened.
It did not look like this in my local files.


k


@tf I made this code very robust, because there might be (edge) cases where there is a published entry without a revision? If this is guaranteed never to be the case, this condition can be dropped.


Repository for Java code development


чувкак можно же написать
if (dallasSensorStructArray[i].currentTemper = DEVICE_DISCONNECTED_C)
{
 continue;
}
... if который был до этого внутри
у тебя сразу же станет на один уровень вложенности меньше


~~rewrited~~ rewritten


I'm not sure I understand why we need these changes now (see also https://github.com/gradle/gradle/commit/e43e6aa8e012b38528c96235db739911beaff4f9), but they seem to make our rules less expressive, by targeting a wider scope than necessary. Is there something we can fix, perhaps a feature to implement, that would allow us to revert these changes?


Please use the reverse domain name notation instead of the "Box 0" string - for example "com.mbed.uvisor.main"


Hi,

 Simply remove Newtonsoft,Json.Net.dll and keep ConvertJsonStringToXmlNode, let Unity recompile and DataMaker will find the PUN own Newtonsoft,Json.Net.dll code.

Bye,

 Jean


yes, it's there: https://github.com/pimcore/pimcore/blob/master/update/3654/postupdate.php#L3-L3


Ich weiß nicht, ob der Screenshot hier Sinn macht. Man sieht darauf nicht viel. Die Schriftgröße sollte größer sein, aber das Bild ist im Handbuch dafür zu klein.


Why not providing a more solid reference for the WKT format ? The BNF gramar is defined in the  "OpenGIS Implementation Specification for Geographic information - Simple feature access - Part 1: Common architecture " standard,  06-103r4, http://www.opengeospatial.org/standards/sfa.
The above format expression is even incomplete for polygons with holes, polygons with Z, etc.


I think this `next_node_calculation` did add a little to the readability of the code, but perhaps the way to solve that is to push the call to the `data_query` method into a method on the calculator.


public RenderView(Context context) {


Line is too long
But ill fix it when i cherry pick


здесь также
if (dallasSensorStructArray[i].lowThreTemp == UNDIFINED || dallasSensorStructArray[i].upThreTemp == UNDIFINED)
{
continue;
}
if'чик который был внутри
еще один уровень вложенности уйдет


See my [previous comment](https://github.com/alphagov/smart-answers/commit/fe5637407eb853b86eab3e93253b2a3f87efbde3#commitcomment-16113486).


But you're right, there's something wrong because it's not executed by the updater... strange.


The label is used by the `usb_drv` for matching the policy. So you can use `label_suffix` at the `usb_drv` to select the right device in addition to just looking it up by the name of the component (`label_prefix`). I am not entirely sure if that is such a common use-case though. Instead of making this attribute mandatory we should probably make it purely optional.


Hi @gsidebo, I've implemented the backoff-retry mechanism for that exact reason; especially when running unittest, it sometimes took some time before the database (Cassandra) became consistent.


One such way is http://giswiki.hsr.ch/GeoCSV


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


Also commit message tense is wrong


This does not have any effect here. Remove it.


здесь также заменить как и до этого писал в примерах


Paint paint;


How else i'm supposed to fix build failures with older compilers, non-gcc5, if i can not compile DT with them?


hello! Where could I read about what is '2.1', release dates, to-do plan?


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


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


Paint paint;2


Paint paint;3


Звёзды нужно задать иконками, они есть в иконочном шрифте, например список, внутри li-шек иконки, и правильные маргины им надо сделать, а то они у тебя слипаются


test


![1](https://cloud.githubusercontent.com/assets/16293594/13073102/147d8158-d4d8-11e5-9f53-4b79f166e190.jpg)
![2](https://cloud.githubusercontent.com/assets/16293594/13073104/1481556c-d4d8-11e5-9d07-fc34d673893f.jpg)
![20151010_173933](https://cloud.githubusercontent.com/assets/16293594/13073103/148075de-d4d8-11e5-8fba-3648b25ae11d.jpg)
![Uploading 20151014_201105.jpg…]()
![Uploading 20151120_104825.jpg…]()
![Uploading 20151120_112006.jpg…]()
![Uploading 20151218_071052.jpg…]()
![Uploading 20151218_073536.jpg…]()
![Uploading 20151218_074929.jpg…]()
![Uploading 20151218_081434.jpg…]()
![Uploading 20151218_082838.jpg…]()
![Uploading 20151218_083414.jpg…]()
![Uploading 20151218_083629.jpg…]()
![Uploading 20151218_083819.jpg…]()
![Uploading 20151220_043452.jpg…]()
![Uploading 20151222_190105.jpg…]()
![Uploading 20151222_190144.jpg…]()
![Uploading 20151229_182545 - Copy.jpg…]()
![Uploading 20151229_182545.jpg…]()
![Uploading 20151230_003819.jpg…]()
![Uploading 20160102_001901.jpg…]()
![Uploading 20160102_001919.jpg…]()
![Uploading 20160102_001936.jpg…]()
![Uploading 20160102_002219.jpg…]()
![Uploading 20160102_002236.jpg…]()
![Uploading 20160102_002257.jpg…]()
![Uploading 20160102_002355.jpg…]()
![Uploading 20160102_002611.jpg…]()
![Uploading 20160102_002655.jpg…]()
![Uploading 20160102_002712.jpg…]()
![Uploading 20160102_003620.jpg…]()
![Uploading 20160102_003723.jpg…]()
![Uploading 20160102_083900.jpg…]()
![Uploading 20160102_083955.jpg…]()
![Uploading 20160102_084038.jpg…]()
![Uploading 20160102_084323.jpg…]()
![Uploading 20160104_072320.jpg…]()
![Uploading 20160116_181134.jpg…]()
![Uploading 20160116_182631.jpg…]()
![Uploading 20160116_183239.jpg…]()
![Uploading 20160127_044642.jpg…]()
![Uploading 20160127_045347.jpg…]()
![Uploading 20160205_103428.jpg…]()
![Uploading 20160205_113407.jpg…]()
![Uploading 20160205_175749.jpg…]()
![Uploading 20160206_025905.jpg…]()
![Uploading 20160206_030244.jpg…]()
![Uploading 20160207_221339.jpg…]()
![Uploading 20160207_221343.jpg…]()
![Uploading 20160207_223331.jpg…]()
![Uploading 20160207_223352.jpg…]()
![Uploading 20160207_223416.jpg…]()
![Uploading 20160207_223423.jpg…]()
![Uploading 20160207_223441.jpg…]()
![Uploading 20160207_223540.jpg…]()
![Uploading 20160207_224113.jpg…]()
![Uploading 20160210_095144.jpg…]()


здесь такая же песня с вложенностью


peut-être charger le module explicitement dans l'image docker-shibboleth-sp ? avec ``a2enmod headers`` ?


Paint paint;4


I have and to be honest my reasons for not using it are purely irrational ;-) Well, I will use `host_to_big_endian` instead.


why?


Thanks, I have incorporated your remarks.


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


mg


이런 식으로 SQL 문장을 문자열 더하기로 조립하면... 작성하면 sql injection에 취약함.

참고:
* https://ko.wikipedia.org/wiki/SQL_삽입
* http://beyondj2ee.tumblr.com/post/13861300836/mybatis에서-like-검색하기


@Paolo Messina
#Paolo Messina
I do not know the code.


@pynicolas  :) I will fix that


Good to hear this is fixed! Any thoughts on when to expect this release? :)


@PaoloMessina


Hi Jean,

If ConvertJsonStringToXmlNode.cs is kept, the following error is shown while recompiling:

Assets/PlayMaker DataMaker/Json/Actions/ConvertJsonStringToXmlNode.cs(9,7): error CS0246: The type or namespace name `Newtonsoft' could not be found. Are you missing a using directive or an assembly reference?

Assets/PlayMaker DataMaker/Json/Actions/ConvertJsonStringToXmlNode.cs(10,7): error CS0246: The type or namespace name `Newtonsoft' could not be found. Are you missing a using directive or an assembly reference?

It seems that ConvertJsonStringToXmlNode.cs cannot lookup to the Newtonsoft.Json.Net.dll provided by ExitGames.

Wilson


@butenkor


@ppescher


inherit from object, you don't want old-style classes here


No problem, one can dream... :relaxed:


@ppescher
#ppescher
I do not know the code.


Not just that but if we have many pawns with opp bishops and one side has at least 1 more. I'll take a look at the link however i'm currently trying to target some endgame phases that could be scaled better (these are stages that will eventually lead into an endgame).


Although I'm sure that the `next_node_calculation` blocks weren't the best place to do it, I think they were serving *some* purpose if only in providing an abbreviated way to refer to the logic.

I would say that in this case in particular, the resulting code in that `elsif` is *harder* to read than what was there before.

I wonder if your real motivation here is more about getting rid of the `next_node_calculation` blocks and/or preparing to extract these bits of logic into the calculator. If so, I think it would be good to explain that in the commit note.


What about simply adding "$$JAVA_HOME/include/darwin" to the INCLUDEPATH as a third line. I think it doesn't hurt if "$$JAVA_HOME/include/linux" does not exist under OSX. Less stuff to read.


@pynicolas fixe 77e0e893ae2e75f70ea22028d516da16bd77bfaf


Well spotted!


Paint paint;


Odd. I don't have time to look at this right now, is that a show stopper that you can't use ConvertJsonStringToXmlNode?


Couldn't the metadata also include the possibility of specifying the spatial reference system ? Coordinates are not necessarily longitude/latitude over the WGS 84 datum (although this will be the most common case, and what is enforced in KML or recommended in GeoJSON).
A way of doing this (restricting to the database of spatial reference system published by the EPSG) would be to  refer to URL pointing to the registry maintained by OGC, with URL in the form of http://www.opengis.net/def/crs/EPSG/0/code, like http://www.opengis.net/def/crs/EPSG/0/4326 for the EPSG:4326 (latitude/longitude over the WGS84 datum)
Although I should mention that there are potential caveats in using EPSG codes, since for example EPSG:4326 mandates an axis order in latitude, longitude order whereas the general practice (enforced in KML and GeoJSON) is to have longitude,latitude order.


Shouldn't that be `set` ?


you're right!


@ppescher
I can not platform.h code files.


а в одну строчку нельзя это написать?


If no maxtextlength is set, then this seems to fix a very low max  (~3500 characters). Does Integer.MAX_VALUE * 2 cause an overflow?


vielleicht das `tonumber()` jeweils in die return values von `r:count` und `r:getLimit` einarbeiten. dann sieht es etwas aufgeräumter aus


`// is the same...` is it?


Hello, please rename SL to SI, because SI is alpha2 code for country Slovenia and SL is alpha2 code for Slovenian language... SL code for country actually represent Sierra Leone. Thank you