Did you run `git grep "coalib\.tests"` and `git grep "coalib/tests"` and check if anything was left ?
If yes, it'd be good to mention it to make us more confident of the changes :)


Good, this way you prepare a port option instead of a global option. This can be an advantage in some special cases of baud rate / CPU frequency / message frequency mixes.

If you want to save on variable space you can reuse ```maxCharDelay```, setting it to 0 for disable or to the appropriate value to enable, thus eliminating the ```rxBufferingEnabled``` array.


done.


a str *


Can you file a newcomer issue for this ? You dont need to modify it for something so small


remote first


Space needs to be removed. This needs to be changed now othewise sphinx docs will complain


space before :


우리진우잘해


do we need this function anymore? It is used only for test purpose...


Are you able to chance the color?

And it litterally says {0} is now glowing instead of the NPC name.


I'll give this another day for any more comments to come in and then commit it.


Не знаю после какого коммита, но в админке, если включить Объединить css или js файлов, то получаем ошибку
Warning: file_put_contents(Z:/home/site.loc/www/cache/static/css/styles.4c30f7217d96353d277603a0ae02bce3.css) [function.file-put-contents]: failed to open stream: No such file or directory in Z:\home\site.loc\www\system\core\template.php on line 779

Warning: file_put_contents(Z:/home/site.loc/www/cache/static/js/scripts.d89e47efa545d9cfc38265caa231e7c5.js) [function.file-put-contents]: failed to open stream: No such file or directory in Z:\home\site.loc\www\system\core\template.php on line 742


Salutare.
need help!

vreau sa declar un array care sa imi contina book-urile introduse intr-un BookSstock   "Book[] BookStock = new Book[100];"  si nu ma lasa.

cum adaug Books?
cum pot sa vizualizes arrray -ul meu de Book?

mersi


Nezd meg megegyszer


this last line is useless, it tests nothing


> No such file or directory

убедитесь что папка /cache/static/css/ и /cache/static/js/ есть и доступна для записи


:o Is it possible to map mobs? I mean...spawn ghasts, zombie pigmen, magma slimes in nether

Slimes in swamp

etc?


Has this actually been approved? :)

Great job for boost ing the core by the way :+1:


Cum adica nu te lasa?
Iti da eroare de compilare?
Sau ce ti se intampla?


We could just make this simpler with something like this:
```
Open the file _target/generated-slides/slides.html_ in your browser to see the generated HTML presentation.
```


Really picky thing :P
I am not sure about the use of italics, just for consistency. The rest of the README don't use it and it feels weird to me to introduce them here.


создал папки, теперь всё норм, спасибо


Read.md


@HookyQR So sorry for the delay, completely forgot about this! Your explanation makes sense, I think you can translate it into a test easily!


Did a bit of extra work to tidy up the `delete(Conference|Friend)` methods, using sweet alert for the confirmations. Also reactive push/remove of items from the instance.


You mean preventing a normal evade, but do something special?
I think there is no such thing - technical problems will likely be with the interaction between AI and MotionMaster (MoveTargetHome)
Addign such complexity into eventAI will in my opinion be a dangerous idea, as this will hugely increase the chance of errors in AI script (i.e. respawn/ another player attacking after evade, and such)


Disabling this Emoji JS seems to have fixed @21


Disabling this Emoji JS seems to have fixed #21


Disabling this Emoji JS seems to fix #21


Added validation for when there were no End of Day messages, but the time was greater than 6 minutes, so it took away 5 minutes from Wake time


Disabling this Emoji JS seemingly fixes #21


I do wonder if flake8 will give too many spurious warnings -- we'll see/


imi apare eroare si imi subliniaza in eclipse


as i said, its retarded

On 6.3.2016 10:01, grizeldi wrote:
>
> Pac main je cist obicna staric funkcija, ki jo klice tut Java VM.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mucamaca/vic-palacinke/commit/66706ad57c32bae51194f9db622d524d7a4f2e1d#commitcomment-16522386>.
>


@norvig I've run it and it throws a lot of errors at the moment, but most of them are valid errors.


Now the `timeSlotLengthLimit` can be changed in the variables section, if we want to have the time limits for Start of Day and End of Day messages be reduced when the `timeSlotLength` is less then a certain value


ce anume iti scrie in eroare?
Cu siguranta iti zice si cauza. Trebuie sa inveti sa citesti acele erori :)


The Porta*Turret's Base's Ammo is hardcoded, too. Wonder if we should add portapotties...


Instead of using static numbers (30, 60), these are now set as variables (`startOfDayAndEndOfDayMarginOne`, `startOfDayAndEndOfDayMarginTwo`)


@tkelman Thank you for that; I tried but the tests failed; since the first tests still worked, I think the code should work for v.0.3. However the issue seems to come from the following line:

    @test recvec([2^k for k=0:7], compute=Int128) == [1,-2]

and more precisely from the `Int128` part. Is there something wrong with this type in v.0.3.?


If you have include_package_data then package_data is ignored.

You can't have simultaenously MANIFEST.in, include_pacakge_data, and package_data. If you tell me what you want the behaviour to be then I will make you a working setup.py ;-)


This does not appear to work. I am not seeing the pills when click Documents.


> So how do you make pull requests with writing the code in eclipse without setting up the code properly?

@ThisTestUser I use eclipse to edit my github clone, not my project.


Moved the `DETERMINE AMOUNT OF MESSAGES` section closer to the wake time declaration section for ease of use


should be moved to appropriate position in the file!


@AndrewKralovec @TylerNethery1994  @TimLunik @MrMatumba Why all the code commits for the project is only from Andrew? The term-project must be a teamwork, where all the members need to contribute. Lack of teamwork would seriously impact your grades for the project.


The ideal point and nadir point are very important for normalization purpose. But in the original MOEA/D, it does not use any normalization strategy which makes it fails on problem with different scales on different objectives. And the original MOEA/D codes of jMetal only uses the ideal point, which is useful for aggregation function calculation.

In the MOEA/D-STM, its distance calculation method have two versions, one uses the normalized objective functions while the other does not. And in the submitted version, I use the latter one.

I think people who would like to tackle the differently scaled problem should use the nadir point and ideal point together. This can be simply implemented to normalize the objective functions before calculating the aggregation function. Anyway, if you insist to keep the abstract class the same as before, I will be cool to delete the nadir point related methods.

Cheers,

Ke

******************************************
Dr. Ke Li
Research Fellow
School of Computer Science
The University of Birmingham
Edgbaston, B15 2TT, UK
http://www.cs.bham.ac.uk/~likw
*******************************************

> 在 2016年3月6日，下午11:50，Matthieu Vergne <notifications@github.com> 写道：
>
> As far as I see, this nadir point and the new methods using it are not used anywhere but in your new algorithm. If I am right then I highly recommend you to put it back in the new class. You should not change an abstract class unless you have good reasons to think that it applies to all its extensions.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/JerryI00/jMetal/commit/01dee9c6130bfbfa35138773c9c7b86e66ac5c80#commitcomment-16526320>.
>


Personally,I think this line isn't needed,the other is good.


I will find another name for “QuickSort” soon.

Cheers,

Ke

******************************************
Dr. Ke Li
Research Fellow
School of Computer Science
The University of Birmingham
Edgbaston, B15 2TT, UK
http://www.cs.bham.ac.uk/~likw
*******************************************

> 在 2016年3月6日，下午11:46，Matthieu Vergne <notifications@github.com> 写道：
>
> It is true that I did not look at the use of this function, and regarding the implemented algorithms I am clearly the ignorant of the team (I am the Java guy here, not a metaheuristic guy). What I know is that a lot of people has "c" habits of re-implementing many stuff, so I kind of jumped on it when I saw the method and how it looks like.
>
> Maybe only the name is confusing, in which case I can only suggest you to find a better name for it. But I also strongly suggest you to improve the Javadoc of the method.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/JerryI00/jMetal/commit/01dee9c6130bfbfa35138773c9c7b86e66ac5c80#commitcomment-16526302>.
>


should use memcopy() for faster responce than for loop.  see https://github.com/f32c/arduino/blob/master/hardware/fpga/f32c/system/src/memmove.c


some comments


you could take this case out.


see https://github.com/pugjs/react-jade/pull/48


Awesome. :)


Yes


When 694 goes infinity and beyond...


nice use of dereferencing.


Yes.. have looked into audacity for waveform code. After spending an hour just to figure how to draw it... have closed it and started experimenting myself.
Have merged this into master now... Demuxing code seems stable and waveform looks nice (I think) please let me know what you think.
We can continue chat in issue #26


`isDrawLayerAttached` supports a undefined drawLayer parameter. The if could be:
```
if (isDrawLayerAttached(dl, plotId)) {
```


no. one is cosmetic and the other just syncs up a package in the circle ci test with what's used in the image build


Its so the right joystick on the controller drives it


fix #16


@deepu105 - Nice to see the development on v3.

Would like bring a typo to your attention:
packageor -> package


you would have to test it, but i think this line could be simplified to:
```
trS = this.state.hover || this.rowStates.nohover;
```


@ prettyblacc0504


Use es6 syntax, become:
```
var br = cell.split(newline)
             .map( (line, id) => (<span key={id}>{line}<br/></span>));
```


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


my bad?


On line 4357 "**if (_check_area(x0,y0, x1,y1, x2,y2) < 0.85)**", can you change that to "**< 0.9**" instead of 0.85? Did further testing and found a triangle could have an area between 0.85 & 0.9. So tightening the threshold works better. I've had random filled triangles drawing for over 8 hours without any corruption. I think the bug is patched.


Enga da unoda updates eh kaanum for a while!!


Credo invece sia interessante. Domani provo a scrivere un paio di metodi per un po' di cose. Se in
 un solo datatype mettiamo i dati interessanti, poi con i dati ordinati basta aggiungere metodi per ottenere le statistiche che ci servono. Per esempio Il metodo getPeriodigram che dato turb sputa fuori il periodigramma o un metodo makeSmooth che liscia ecc ecc. Cmq domani o dopo ci provo


Hell


Had to change this because of side effects from HaxeFoundation/haxe#3044 on SQLite


Yes - would probably be a good idea. I jump between so many code bases with differing style guides that my code ends up being a combination.


I think this will throw if `wrapAtCol` == -1. We probably also want to ensure that we're wrapping when `wrapAtCol` > the `// ` index. I'll merge and try some different inputs.


`워킹 디렉토리`는 고유 명사라고 생각해서 번역하지 않았습니다.
`Working directory`로 쓸지 `워킹 디렉토리`로 쓸지 고민은 했습니다.


Why is that more robust? (I believe it may be, I'm just not sure why)
This is the link to the most up-to-date magics docs: http://ipython.readthedocs.org/en/stable/config/custommagics.html#defining-magics


just to let you know it looks like you left some debug / testing code in


Hey @ismail44. There isn't an active project right now to pull Agents into gems, although I still think it'll make sense long-term.  I think working to parametrizing Scenarios should be a higher priority, if you'd be interested in working on that?


Thanks, good point - just changed that in 842e7a4


Hello


Test comment by me


>Hardcoding "1 base HP = 1 HP" makes it easier for mod developers to create 1HP pokemon

Similarly, hardcoding Shedinja makes it easier for mod developers to create Pokemon with a base HP of 1 (not fixed). The difference is, yours breaks mechanics.


我用電腦版sync有障礙TT，暫時緊急處理


This is going to cause Travis to automatically run phpunit, which we probably don't want since we don't have any phpunit tests (and likely never will). We could, however, definitely write some unit tests for the JS functions.


Sorry, I should've been more explicit: more robust in a sense that it doesn't require any IPython imports.


@leonardonr, SemestreProfessor  não vai mudar de nome?!


I am not the right one to say whether it makes sense or not to generalize it, as I am not knowledgeable about the advantages of having it at this level or another. My point is general and follows a good practice we think to be worth here: don't add things at general levels unless you have the *need* to do so, and here it does not seem so.

Don't forget that by doing so you increase the complexity of the class for any MOEA/D version. Factoring a piece of code should not be done because it *might* be useful, but because it is *required*, otherwise you just add and add and add many stuff until you get extremely complex classes where each actual implementation uses only a small part of it. This is one of the things we wanted to avoid in jMetal 5, and some classes have been completely redesigned precisely because of that. So as a rule of thumb, please don't touch the abstract methods unless the abstract method needs it.

So you might say that I insist for you to move it to your own class unless Antonio tells you the contrary, basically. You might as well introduce a dedicated class to manage that separately, so at least you don't increase uselessly the complexity of the abstract class.


Upload of the sweet, sweet logo.


this commit completes the method that returns_the_given_nimber_of_top_revenue_earners. It passes the spec harness wherever it is called in iteration 4. Our tests for the method use a small dataset because the tests takes overly long otherwise. We can choose to change that and then skip the test as we move on if that makes sense. The new method and it's helper methods are currently very messy and will need comments removed and sanitation guidelines applied.


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 130](https://teamcity.labkey.org/viewLog.html?buildId=308391&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabKey :: LabkeyModules15.3 :: build Build time: 00:00:00


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 133](https://teamcity.labkey.org/viewLog.html?buildId=308389&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Snapshot dependency failed: LabKey :: LabkeyModules15.3 :: build Build time: 00:00:01


Yeah I'm just waiting for the original reporter to confirm that the problems are gone before backporting it to 2.14


@leonardonr, Semestre não mudaria de nome para Período Letivo?!


Does this repo inted to be the "official" continuation of the Quickmud project?


Ah, got you.


@leonardonr, não mudaria para Plano de Ensino?!


ㅋㅋ 원래 이렇게 n^2 으로 푸는 문제야 ??


@jeromerobert I created an account for you on ARM64 machine. Please check your email.


Estou verificando com @WilliamAltoe & com @leonardonr


thank you Shep!


Ok, das als vorläufige Lösung für den Test zu haben ist für mich völlig in Ordnung. Aber am Ende sollte man da denke ich ein striktes Paketformat haben, im Sinne von "die ersten 128 bit sind die Session ID" und so weiter. Wir wollen schliesslich Bits sparen ;-)


I agree that's better. I remember trying that and going back to this, but let me try again; I'll document it if there's a legitimate issue


thats probably a good idea :+1:


Here are finaly the slides for tomorrow. Let's hope you'll all enjoy the demo... See you in a few hours.


update


Форма авторизации/регистрации пользователя


Eu só estava padronizando o formato mesmo, e essas alterações não seria só no modelo.


Please change this line to "Make sure you have read https://github.com/soimort/you-get/wiki/FAQ" (or similar words)

Legal like terms will scare people away. Don't be too harsh.


I just added a p tag.


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


meta comment: it's good practice to make more, smaller and logically separate commits. that way it's easier to review


unused import. it's also imported in the other file


No, still drops to the last declaration.


looks like there are a couple other binaries. i recommend updating your .gitignore to never look at ds stores and .pyc files


no need to add a comment about what you changed, since git diffs can provide that info


`용`으로 표현할 수 있으면 그렇게 했습니다(짧은 표현).


<img src="https://cloud.githubusercontent.com/assets/12206972/13558569/477de564-e3bc-11e5-8686-e39818e33abb.png" width="150px" height="200px" />


Fixes #5


Nope - Pic_Display searches for an ID passed as GET first, then defaults to the user's ID.


`되어`는 `돼`로 통일했습니다.


Good catch!  I just replaced a package that disappeared (to solve ````ImportError: No module named markup````) with a compatible package - but without checking whether it's still used anywhere.

Just removed markdown_deux in f6ec3c9


This should not compile? Is this covered somewhere later?


warning: This won't work


Awesome!


@duramato


He is my favorite soccer player


Are you sure we need to remove "stock_operating_unit" ?
(I think it is already inline with standard "procurement", which depend on stock)


是我修改的時候考慮不周，非常抱歉。

This version has some serious problems, and please update to the latest commit,


Why should this not compile? The only reason I could think of is something was broken and caused a conflict, maybe the `__rust_abi` shim.


@illicitonion This causes `ImportError: cannot import name BrowserLikePolicyForHTTPS`. In Twisted>=15.1.0 `twisted.web.client` does not seem to have `BrowserLikePolicyForHTTPS` class.


s/service/feature/ ?


Thanks for making `impl` @josharian, it seems useful!

I have a small suggestion. What do you think about adding an installation section that simply lists the import path of this package for easy copy and paste?

Something like this:

> Installation
> ------------
>
> ```bash
> go get -u github.com/josharian/impl
> ```

Not mentioning the import path in the README makes it hard to know if there's a vanity import path that should be used, or if it's just the github URL. I spent at least 20 seconds trying to find if the import path was mentioned somewhere... then had to look at the source code to check if there's an `// import` comment.

Most Go packages mention their import path in the README, so it's quite jarring to stumble upon one that doesn't.

Wouldn't it be nice to save all that trouble for all future users? What do you think?


`It`'s been fixed upstream. Next release this will be corrected. :P


g_build_filename is specifically for filenames and as such the separator may be different on different platforms (e.g. \ on Windows).


This is weird, all these lines have a trailing white space (PEP8 rule W291). What editor are you using? Some editors allow you to automatically trim the trailing spaces on saving (Sublime and Spyder can do that for you, and in Windows I think Notepad++ also allows you to do that automatically). This is the output of runing flake8 on `/dipy/align/test/test_parzenhist.py`:
```
test_parzenhist.py:3:1: F401 'ndimage' imported but unused
test_parzenhist.py:319:76: W291 trailing whitespace
test_parzenhist.py:320:78: W291 trailing whitespace
test_parzenhist.py:321:77: W291 trailing whitespace
test_parzenhist.py:322:71: W291 trailing whitespace
test_parzenhist.py:323:78: W291 trailing whitespace
test_parzenhist.py:324:76: W291 trailing whitespace
test_parzenhist.py:325:78: W291 trailing whitespace
test_parzenhist.py:326:73: W291 trailing whitespace
test_parzenhist.py:327:76: W291 trailing whitespace
test_parzenhist.py:328:72: W291 trailing whitespace
test_parzenhist.py:329:77: W291 trailing whitespace
test_parzenhist.py:330:75: W291 trailing whitespace
test_parzenhist.py:334:78: W291 trailing whitespace
test_parzenhist.py:335:74: W291 trailing whitespace
test_parzenhist.py:336:75: W291 trailing whitespace
test_parzenhist.py:337:77: W291 trailing whitespace
test_parzenhist.py:373:54: W291 trailing whitespace
test_parzenhist.py:374:53: W291 trailing whitespace
test_parzenhist.py:417:78: W291 trailing whitespace
test_parzenhist.py:418:74: W291 trailing whitespace
test_parzenhist.py:419:75: W291 trailing whitespace
test_parzenhist.py:420:76: W291 trailing whitespace
test_parzenhist.py:609:1: W293 blank line contains whitespace
test_parzenhist.py:618:1: W293 blank line contains whitespace
```


Can you add a note about that the default is false?


Well. I'd put in both places assuming that the cover letter won't appear in git history afterwards.


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


"Much update"?


I changed it to just media_url. media_base seemed unclear as to what it would be.


Yep

On Sun, Mar 6, 2016 at 8:06 PM, Pizza <notifications@github.com> wrote:

> "Much update"?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Happypizza/gear/commit/4f534f8734f82380a007b5ce87290e0d4a2a74b7#commitcomment-16526636>
> .
>


Booa


I don't think does what you intended
a connector Node happens at coordinates that have 1 odd and 1 even values
in other words they cant have equal parity (even/oddness)
`if( x % 2 != y % 2 )// then at a connector Node`


Commit message should have addons name. This will help me a lot when reading. For example,

[FIX] procurement_operating_unit, PO picking should be created before assigning the picking OU


...Well then. :package:


Now ```<container class="myClass">``` turns into ```<table class="container center">```

Shouldn't it retain the original classes?

```<table class="container myClass center">```

I changed this this:

```
component.children().attr({align:'center'}).addClass('center');
```


This is an awesome app.  I'm learning a lot just from looking through the source.

Can you explain this code in the view controllers?
`    var events: JSON? {
        didSet { self.reloadData() }
    }
}`

I understand that it's getting JSON data, but I don't understand it as the Alamofire code seems to need more than that.

Thanks


Watt is the box for?


Thanks for spotting this
On 7 Mar 2016 08:14, "Vivek" <notifications@github.com> wrote:

> @deepu105 <https://github.com/deepu105> - Nice to see the development on
> v3.
>
> Would like bring a typo to your attention:
> packageor -> package
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/deepu105/jhipster.github.io/commit/48f2005e6f62f70b477786f82ba8e346fa6d6fa5#commitcomment-16526403>
> .
>


Thanks for the feedback. I'm guessing it would also be possible to keep the schema in the old object name and just remove it from the new object name:

ALTER TABLE "${schema}"."${table}" RENAME TO "${table}_flyway4_upgrade";

I pushed a new commit with just this change: https://github.com/nathanvick/flyway/commit/7050f3f447de43eb11facf1963372135d91b1d91


Have you _not_ been paying attention to AT memes?


Sure. Send a PR?


I haven't. :P

On Sun, Mar 6, 2016 at 8:12 PM, Pizza <notifications@github.com> wrote:

> Have you *not* been paying attention to AT memes?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Happypizza/gear/commit/4f534f8734f82380a007b5ce87290e0d4a2a74b7#commitcomment-16526663>
> .
>


dd


Or you can just click the "Edit this file" button on the file to propose a change. The PR doesn't need Travis, anyway.


Once upon a time, the ATers used :stuck_out_tongue: (:p) too much. Then they went to github. Github autocompletes it to :package: (:package:). The end.


I was getting a NullPointerException in tiny_mce initialization code that was tring to construct TINYMCE_JS_ROOT through os.path.join calls using STATIC_ROOT as the base.

So for initialization of tiny_mce, it was necessary to define STATIC_ROOT


Okay. Also, can you add the bumper sticker/ brochure to gear (if you're done)?


Why are you deleting the iron and snow golem classes? What's the problem....


@walle-hong 以前这本书就看过，我这次是想用 LabVIEW 把所有的设计模式都实现一遍，哈哈


Okay, no worries - can you reply to the list with that? I'll resolve it in v5


Might not be there tomorrow. Gut issues.

-----Original Message-----
From: "Melinda Robertson" <notifications@github.com>
Sent: 2/20/2016 12:29 PM
To: "thelasteiko/422OperatingSystems" <422OperatingSystems@noreply.github.com>
Subject: Re: [422OperatingSystems] Refactored to abstract CPU and schedulerfunctions. (9c7e1ff)

Yeah, and if you could look at my code and make sure it runs that would be
great to. My computer decided to slow down and wouldn't run it so I
couldn't test it all the way.
On Feb 20, 2016 11:52 AM, "Shewan11" <notifications@github.com> wrote:

> I think so. I will check when I get home.
> Are you okay with my tasks? We can talk tomorrow more.
> On Feb 20, 2016 11:17 AM, "Melinda Robertson" <notifications@github.com>
> wrote:
>
> > Did you pull first?
> > On Feb 20, 2016 11:11 AM, "Shewan11" <notifications@github.com> wrote:
> >
> > > i can't commit. should i do branch?
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/thelasteiko/422OperatingSystems/commit/9c7e1ff372c150f7ae927229a63cd7e3f7a194f8#commitcomment-16222693
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/thelasteiko/422OperatingSystems/commit/9c7e1ff372c150f7ae927229a63cd7e3f7a194f8#commitcomment-16222723
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thelasteiko/422OperatingSystems/commit/9c7e1ff372c150f7ae927229a63cd7e3f7a194f8#commitcomment-16222943>
> .
>

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


style nit: you don't need the parens around the expression


Almost done. I'll commit when it's ready.


Okay!

On Sun, Mar 6, 2016 at 8:19 PM, Pizza <notifications@github.com> wrote:

> Almost done. I'll commit when it's ready.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Happypizza/gear/commit/4f534f8734f82380a007b5ce87290e0d4a2a74b7#commitcomment-16526697>
> .
>


Sorry, it was something I overlooked in the first round of fixing the contenttypes.generic deprecation.

At this point, I would rather avoid running rebase on this branch.  We now have this discussion history in the pull request attached to individual commits and we would loose all of that in a rebase.

Apologies about the mess, but I think this fix will stay spread over two commits....


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18518](https://teamcity.mesosphere.io/viewLog.html?buildId=155462&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:01


Apache는 음독하지 않았습니다.


I think this was the best I could do and hope it didn't make too much of a mess.


Renan

O ideal era que eles fizessem um git clone só no diretório do livro ... pois há muita coisa lá fora  do contexto. Posso criar um REPOSITORIO soh para o livro ... acho que seria melhor nao???


People were complaining about them but I don't see why he did it as they weren't causing any problems.


Ha thanks! Fully working now, at least locally...


Lovely.


`if (len>3) {`


Can't you check the axis on the driver station?
On Mar 6, 2016 7:14 PM, "Connor Adams" <notifications@github.com> wrote:

> Its so the right joystick on the controller drives it
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FRC2706/2016-2706-Rubiks/commit/419f7e1f42a6a0d28ab7caeafcda4f614e8e17dc#commitcomment-16526399>
> .
>


According to RelaxNG schema, `/` is not allowed here. But IMHO `services/ntpd` is a good idea, so schema should be changed.

Also - what is canonical meaning of values here? Should be also string? For boolean features what values should be true/false? This commit uses full variety of them (`''`, `'1'`, `True`, `None`). What about "VM default" (if it still should be supported)?


- [x] Docs


Time travel?


And shortly thereafter abandoning it


If this is suppose to be a `css` file we need to do two things!

* [ ] Rename the file to `name_tag.css`. We typically don't use spaces when we name files, instead we use `_`. Also since this is a `css` file, we should remove the `.html`.
* [ ] The correct syntax for CSS is below. Note, I removed the `" "` around a lot of the parameters. I also moved the semicolon towards the end of the style.

```css
.rectangle {
  width: 500px;
  background-color: red;
  border-top height: 200px red;
  border-middle: 220px white;
  border-bottom: 20px red;
}
```


- [x] DOCS


This line brought by the exact same outcome as putting `declare -A` in `arch/_common.sh`.


@Aeramor - I looked at different ways of writing this method, and it seems to me like the best way for now would be to use the Active Record method `Claim.where("created_at = ?", 7.days.ago)` for all environments, or to use  `Claim.where("created_at = ?", 7.days.ago)` for development and testing and `Claim.where('EXTRACT(days FROM now() - created_at) = 7')` for production. Ultimately, however, it sounds like it would be best to use a PostgreSQL database for all environments, which would make it possible to Extract in both cases.

While using different queries for different environments allows the use of Extract in production, the downside would be the inability to test the production code using the RSpect tests I wrote. I'd be curious to hear what you consider to be the best approach.


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


리뷰 완료했습니다.


"You can look at"


Now I am clueless.


thanks @jmansour


missing colon


thinking about it a little bit more, overriding these operators might not be the best way to implement these comparisons, because it might lead to the expectation that two dominos are equal if their point scores are equal. i'll suggest another way of doing this further down in the code where you use the comparisons


I always put the friday


"you can specifically target the exception controller" or "you are able to specifically target the exception controller"


I was referring to your commit being dated tomorrow


Thanks : )


@leninkp supongo que esto algo tendrá que ver con el backend, no? almacenamos estas preferencias?

Así nos evitaríamos de quemarlas en este arreglo ^


The following doc(s) have been deployed to the [drupalizem](http://drupalizem-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/drupalize-me-series](http://drupalizem-static-docs.pantheon.io/docs/articles/drupalize-me-series)


I'd rather focus on acceptance tests, personally. I am guilty of doing lots of refactoring after code is written, meanwhile the front-end more or less stays the same. Acceptance tests would less fragile overall, I think -- and also cover the more critical functionality that we care about most. Nightwatch.js looks like a pretty solid Selenium-powered solution.

I changed this to PHP because it didn't know how to run `composer`. My hope is we can get Travis to operate on a PHP environment but run tests as Node. Haven't sat down and tried to figure it out, but I'm sure it's possible.


I don't see it


```
error: The option `containers.isso.services.isso.config.general' is used but not defined.
(use ‘--show-trace’ to show detailed location information)
```


fffffffffffffffffffffffffff


Look at the commit list.  Your commit is dated 12:47 AM tomorrow


vvvvvvvvvvvvvvvvvvvvvvvvvv


@fabricioflores puedes ayudar a revisar este algoritmo? si `selected` está indefinido, lo pone a `true`. y en esta línea @leninkp lo cambia a `false` (de nuevo :WAT: ) y luego tenemos de nuevo el mismo condicional más abajo


Oh good catch I'll fix that.


...no?
http://i.imgur.com/ZNnW6WA.png


@leninkp igual, preferiría que saquemos esto del backend. así no lo tenemos que cambiar dos veces más adelante
https://github.com/GrowMoi/moi/blob/master/app/models/content.rb#L27


@zet809

* We can add the part of code which get the real gateway if it's a key word **<xcatmaster>** into the xCAT::NetworkUtils->getNodeNetworkCfg() instead of running it each time for a node.

* A fact to support comment 1 that the running of 'xCAT::NetworkUtils->my_ip_facing($ip)' has to be executed for each node.


affects


Alberto me comunico que en este primer release, no vamos a almacenar estas preferencias, llamadas intereses, asi que de momento las vamos a quemar


Hey There!

I'm glad you’ve been following the app!

The "events" variable is actually being set from the EventsViewController inside the loadEvents method. LoadEvents is called when the view appears, and the method uses Alamofire to retrieve the required data from a database. Once the data is retrieved, it is converted to JSON and stored in the “events” variable in the EventsTableView. Once the "events” variable in EventsTableView is set, self.reloadData() is automatically called and the tableView is populated.

Does that clear things up? Let me know if you have any more questions!


Somehow `git log` is unaffected....

I'm seeing the wrong date from GitHub and gitk


This doesn't make sense……(denial


is your space key broken?


喔 老師這個檔案不是我創的
好像是在用git時
使用錯誤造成的


@zet809

* Could we using the runxcmd() to call the xCAT command internally?

* I think we need check the output of xdsh to ensure the operation to configure IP has been successfully.


Well created_at is a datetime and 7.days.ago will be the current time 7 days ago (which is why you need to extract days first).
There are a lot of ways to solve this, an easy way that doesn't require extract would be to get all claims between 2 timestamps like `Claim.where(created_at: 8.days.ago..7.days.ago)` This will work on both databases but still have the issue of being time dependant on run (if you run at 1pm you could miss something that hit it's window at noon for example). So instead I recommend using: `Claim.where(created_at: (Date.today - 8.days)..(Date.today - 7.days))` This will not be run time dependant and is effectively what you're looking to do. The only benefit to using extract is that it's slightly faster than doing a less than greater than.

Why are you unable to run rspec against postgres? Normally in your db config you just specify a different database (in your local postgres) as the test/dev and run against that.


you are really a very prolific coder @johnclause ! well done!


obviously we've been running without this.  Any particular reason you added it?


:+1: (assuming that also shell escapes)


thanks for fixing that @jmansour .
was there an update on the status of MeshVariable_projection


This should only ever be called once, on include, so we probably want to memoize it at that time.


`if len(self.board) == 0` is equivalent to `if not self.board`
reference: http://stackoverflow.com/questions/9787890/assign-class-boolean-value-in-python

in general, though, i recommend avoiding returning None, because it has a tendency of getting passed around and then causing some sort of exception far from where the actual error happened. there are almost always better ways of implementing this behavior. i will point them out as i see where left_end and right_end are used


`if game.team1.nil?`


`@url_helpers || self.url_helpers = Module.new`


you don't like my dash spacing conventions, huh? :)


any reason we wouldn't want to memoize?


Hmmmm, I might have done this incorrectly. It looks like we might need to standardize naming conventions or install a pre-commit hook? I am not too familiar with this: http://stackoverflow.com/questions/19151247/ignoring-folder-meta-files-on-version-control

Also, you are correct about not ignoring meta files. We have to let everyone know to push and pull meta files everytime a commit has been added: http://answers.unity3d.com/questions/932348/visible-or-hidden-meta-files-with-git.html


Just using the same coding conventions as radare developers do ;-)


Don't forget to add an `alt` attribute to your `<img>` tags for better search engine optimization :)


Fix it !


no need to make `attribute` have a default value. it will only kick the exception associated with not providing one further down the road, making it harder to find where the error is. also, you should consider giving it a more descriptive name


I like the instance config as well, that might be

```ruby
def initialize(request, options = {})
  # snip
  @url_helpers = options.delete(:url_helpers) || self.class.url_helpers
  @default_url_options = options.delete(:default_url_options) || self.class.default_url_options
end
```


Great job


as we discussed, this could be kept outside of the class definition since it will always be the same

also, python style conventions: prefer single quotes to double quotes, and spacing around key, value pairs should be key: value


Exactly, and you're not passing an id on the profile page, so it'll show to the current user's profile pic no matter whose profile you're looking at.


Thanks for this work @jeff3456 but one thing: I don't want the notebooks in a separate directory. Because that might lead to problems when the notebook does an "import" statement -- if the module is in another directory, the notebook may not find it, unless paths are set. I think it is safwer to have all the files in one directory.


`assert_equal Active...`?


you could rewrite this as `return not self.hand`

putting a `not` in front of something will automatically cast it to a boolean, and lists automatically use length to determine boolean value


Yeah I'm not fully understanding the purpose of these meta files yet.

Are you able to runt he client at all? Mine just hangs.


these are already included, no?


I'm sorry but I have to say, the fix of temp id is almost meaningless to anyone, but it will make the user feel confusing after update.

> DO NOT CHANGE A CARD ID ONCE IT HAS BEEN ALREADY ADDED. It is very troublesome when a card has to be renamed, this is more important than proper accuracy on the ID.

The readme of this project says the same thing.

@destdev Had Devpro updated this id change?


the 'pythonic' way to do something like this is:

`return sum([domino.point_sum() for domino in self.hand])`


instead of having printing methods, i recommend having string representations for your objects that you can print whenever you want


Why is this section named disabled ?
Wouldn't a name like `invalidlinks` be better ?


棒！


If user table is not truncated, you'll have to check for an existing user with this user_name


If traffic allocation is set to 100% then 'ignore' is not set to 0 but it is left out of the object so you need to handle that case too


same thing, if not truncated, you'll have to look for an existing entry for user_name


Делю страницу на блоки,мне так легче ориентироваться,что откуда выходит,иначе путаюсь,это ж не нарушение)?


@IonicaBizau np! Keep making cool stuff and I'll keep playing with it. 💁🏻 lol


Можем


Там красная лента в дизайне,я думал,что это ссылка


lol


Unclosed ``.


Goed zo


Temp files are most likely created by this code:
https://github.com/Ultimaker/Uranium/blob/2.1/UM/SaveFile.py#L54

Why they are not properly moved towards the resulting file I do not know. But with how Windows handles files, it could be that the final os.replace actually fails.


It's work for me


Maybe I am missing something here, but by specifying "called at the same time" it's possible again to have different group constructs: when you use a heat system where the heats are not called at the same time, 1h+++ is not relevant because they are not called at the same time.
I am happy if you can prove me wrong, but this seems to make the complete regulation change pointless.


'use' umesto 'Use'


another mention: @zaboco


ime klase pocinje velikim slovom


yet another: @zaboco


Good point!


About `(entity, 0)`: `0` = ?


pazi na identaciju i PSR-2 coding style


是的. 所以我就要借鉴你这个 `event` 和让用户自己执行`start` 行为, 但是绝对不能暴露不相关的函数给使用者.

最后, 感谢你的支持! 么么哒!


@jelovirt: This reverts 914a5d3a9648a8d9bc4df58a9a616ef9c0df2c3b, which causes builds on Windows to fail.


Why remove taskNum-1?


Why don't you return the partially applied function without `state` applied here, like in other cases?


Перепутал img с alt,это значит,что надо больше спать)


IIRC I made this because Compass builds didn't work anymore because of dependencies. Sigh.


pazi na indentaciju


Same as for other PR, why taskNum-1?


Looks like there's a small indentation error here (leading space)


Hello @madmatt,

Yes, this makes a lot of sense to me.

Making default behavior explicit was exactly what I had in mind, so I second your suggestion to limit the possible fields to primitive types and set the default type to string.

This would result in a something that I think is an expectable behavior for such a lib.

An exception would only make sense, if we would also allow complex field types (which we shouldn’t), since someone made an explicit decision, which cannot be delivered.

> Am 07.03.2016 um 10:37 schrieb madmatt <notifications@github.com>:
>
> Hey @fanatique <https://github.com/fanatique>,
>
> Thanks for the comment. I think having a default: option that does nothing is a good idea to make it clear what is happening - I'll amend this to show:
>
> case 'string':
> default:
>     $currentLine[$name] = (string)$currentLine[$name];
> break;
> I think this should make it clear the default is string, which is also backwards compatible, so would require a new major version if you were releasing versions under semver.
>
> I'm with @Firesphere <https://github.com/Firesphere> that I don't think raising an exception in this case is a great idea, as generally I wouldn't expect to specify field_type for every field in the chopping map - only when it's required.
>
> I was pondering adding other field types like date, datetime, but our current project doesn't really call for it, so I haven't yet. What do you think about these quasi-types - I feel this is getting too deep into implementation-specific code, and we should just deal with PHP primitive types like int, float and string, but keen to hear your thoughts.
>
> Does this make sense to you as well?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/madmatt/php-fixed-length-file-parser/commit/6c232f34d1862835cfc8ae5ae093c78a77dd9c87#commitcomment-16530734>.
>


@mercury233 I was more confused when I found out that I accidently overwrote our Navigate with Combination after getting all the recent fixes.
If you or others want to revert this, sure, no problem, we can do this. I just think it's less confusing if the scripts have an ID matching the number.
However, yes, I have to admit, I forgot that part of the readme here :/


Bien joué =)


Now i get this bug:

![image](https://cloud.githubusercontent.com/assets/5549913/13566368/83004c64-e455-11e5-91e5-7896e22735af.png)


Yeah, I think this commit was to fix https://github.com/robfletcher/gradle-compass/issues/54.

Is it so that if you have both the `buildscript {}` block and the `configurations.all {}` block, the build breaks?


fix


Why not

``` python
config["retractionZHop"] = timelapse.retraction_zhop
```

like the others? Or moving the others over to

``` python
config.update(dict(type="zchange",
                   postRoll=timelapse.post_roll,
                   fps=timelapse.fps,
                   retractionZHop=timelapse.retraction_zhop))
```

Might be missing something here...


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18544](https://teamcity.mesosphere.io/viewLog.html?buildId=155580&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:04


@akariv decimalSeparator is not a constraint i think - i think it should go on the `number` type entry. I'm happy to craft this if you want - or you can take a second stab :-)


test comment


written


Thank you Drew, this feature will be in fact added to next releases. Please stay tuned.

Please for any feature requests, issues or bugs contact with our support team:
support3d@bq.com


Thanks! :)


Consider using:
```
if ##class(%Dictionary.PropertyDefinition).IDKEYExists("%Dictionary.ClassDefinition","SystemMethods") {
    set countSysMethods = ccdefs.SystemMethods.Count()
} else {
   set countSysMethods = 0
}
```


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


Do we need `immutable-js` any longer? If not we should remove it from deps.


Там таблица в 3 раза больше,я не понял этого сразу


So at the very bottom there is it supposed to be "eitm" or is that "item"?


리뷰 완료~


php


@tim37021 : Great. I like your idea to consolidate the structure intersection (and even generic form).


I will check it, @marcellustavares. Thx.


JS izdvojiti u odvojene .js fajlove. Svaki deo modula je odvojen js fajl.


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


If I restore the `buildscript` block, builds succeed for me on both OS X and Windows, but IIRC the `inotify` error might've been sporadic, so...


@chelmuth: please consider for staging


Right - I've missed this variant. Furthermore, I don't know how `try catch` changes performance.

Will be able to fix this in the evening.


Rename **$monitorId** to **$componentId** for clarity


Have you seen this #define in other projects? For me it resulted in an unfruitful search on a newly introduced C++17 (?) `foreach` keyword. I'm already brainwashed by the `begin() end()` boilerplate I hardly care (although it can be error-prone when nested and incrementing the wrong iterator), but for those who mind the typing I would propose using the vanilla `BOOST_FOREACH` in sources. The C++11 range for construct does not seem to be supported by MSVC 2010 [1], which is still used by quite a few.

[1] http://en.cppreference.com/w/cpp/compiler_support


The above makes no sense to me.


It is unused file.


and it is going to simplify some of the code as well... going to push a patch for using `checkout_tree_at` instead of `checkout_tree`


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


Teste pT


Given the fact these are all single characters I would rather implement this as a foreach loop over a string consisting of the characters you intend to escape (and skip the bottom 2).


@championswimmer , I am not finding anything to add in gulp.js, app/components is already there ,want help in gulp , all done except that :)


Don't touch existing files for now. Just create one file that

- fetches, builds and installs beakerlib
- calls other scripts with the actual test code

Once I can successfully execute that script manually on the CI machines, I will include your top-level script into slave-control.py myself.

As I said, the biggest problem right now is that

- you use a tarball instead of plain files
- the test does not actually succeed (see the logs I posted)


其中图片上传功能,还未能将图片名称写入数据库.登录注册的js数据验证还未做.


note to self: stop mentioning yourself @zaboco


I think `region` should be an app-wide variable.


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


Any plan on adding support for handling specific routes ?

For example, if I want https://example.com/products/* to open in app but want https://example.com to open in browser regularly.

See http://stackoverflow.com/questions/35840262/handling-specific-routes-in-android-m-app-links for more details.

Otherwise, every url form the domain will be matched and that's a problem for most use cases out there


Hash size is different for base and branch


this is what


Here's [pull request](https://github.com/intersystems-ru/webterminal/pull/59).


Like set in this file:
https://github.com/alphagov/rummager/commit/5a078e60c055d8a0aa6491aff013dd97287b28d0#diff-d6321e1bd14503c48adb4cd416e1b783R7


this is why the build fails. it works on my machine. perhaps because i have a newer version of bower?


I've added libsoup as a hard dep to the PPAs/repos


How will this effect the smart answer flows that don't pass response to next_node? eg,


this was unused? I was sorta relying on it when playing with the oh-my-fish code


Can't these (`CENTER-` and `-UVS`) be iterator settings instead? Imagine this is very useful for all other 'clients' such as the modeller plug-ins.

With those two settings removed the usecase for separate convert settings becomes a bit thin maybe. Besides, I can imagine that USE_NAMES/GUIDS can be useful for the plug-ins as well. The Blender add-on has a similar setting.


What is this `$transform` thing and why do we need it? Not enough functional programming running in my blood yet, sorry :)


Try catch does not particularly affect performance IF catch block does not get hit.


Yes, thanks! Fixed :+1:


fali ;


the user will the a list starts from 1, but internally the index starts from 0


I see no changes in the Drawables.md file. If you want to edit the Wiki, go there on the codepath repo and change it there. No need for PR


You also changed the submodule to point to your repo.


As we have a `@client.create_repository` this could be renamed to `@client.create_snapshot`.


What's your question? If it is regarding the `typeof spec.v` you are right: it should be just `typeof spec`, because `returnFormattedValues: false`. Will fix it.

If it is about the `simple.name`, we're inside of the foreach... `simple.name` will be `"boolean"`, `"number"`, `"string"`, `"object"` or `"function"`. Just checking that the returned primitive value is of the expected type.


Hi @Ry-K
Can you please explain why we need this?


To learn more about the Kubernetes Dashboard, ...


Because we humans count from 1, so the list should display from 1 right? but the index starts from 0


Qt has `foreach` macro by default (which can be disabled) macro but other than that don't think there's a big change of collision. Personally I think foreach increases the code readibility immensily (and BOOST_FOREACH everywhere doesn't look that nice) but if you object we can drop this.


In the meantime, I have read through the corresponding branch in FIAT. I see you invent a new convention for numbering entities on tensor product cells as a tuple, combining the entity number for the corresponding entities of each subcell. This convention is at variance with what we use for `entity_dofs`, for example (P2 on prism cell):
```
{(0, 0): {0: [0], 1: [1], 2: [3], 3: [4], 4: [6], 5: [7]},
 (0, 1): {0: [2], 1: [5], 2: [8]},
 (1, 0): {0: [9], 1: [10], 2: [12], 3: [13], 4: [15], 5: [16]},
 (1, 1): {0: [11], 1: [14], 2: [17]},
 (2, 0): {0: [], 1: []},
 (2, 1): {0: []}}
```
As you can see the dimensions are numbered with pairs, but the entity number is a flat number. I don't think there is a strong reason to use two different conventions in the same codebase for the same thing.


This should be included in cors block. I'm thinking about removing the cors flag and always intercept OPTIONS and setting headers on response. What do you think?


不要的东西直接删掉，别用注释


Sure


已阅


Just a minor comment: please use articles
PTAL


The `'` ins `s'` needs to be escaped.


@matthewtckr this commit introduced check-style violations in SalesForceInsertDialog. Please review and clean up


Nice!


Shouldn't these methods just take a `Setting` argument rather than `SettingField`? I can imagine this results potentially in compiler warnings (signed vs unsigned or others). Haven't checked though. Accepting `Setting` also seems more readable to me. In the method body you can do a static_cast<unsigned>.


Awesome, thanks!


Now corrected.


I was thinking to check based on the numerical value, but since it has been already rendered you're right there's not a lot that can be done.


use `\t`, otherwise coala will complain


I don't think there's a single line in this class that does not violate check-style.

@SergeyTravin, fix .


UV option could definitely be a geom iter setting. Model centering possibly also. Let's revisit this when I've actually implemented the features.


which is why your test fails


Izbaci overwriteovanje user_id-a na tasku. Note: ne postoji provera da li task koji user edituje zaista pripada tom useru. Bolje resenje bi bilo $task = Auth::user()->tasks()->findOrFail($id);


this is untested


Add a protected `unsigned num_settings() const` or similar? But as mentioned, I think the IfcConvertSettings can also be merged into the IteratorSettings as other iterator consumers can benefit from them as well.


Yes, definitely.


I would remove or reword this line as this is not possible to do yet.


Most style guides (including mozilla's) recommend against single-line control statements without braces.


Yeah, this was mistakenly pushed to master by me. I will be adding the config file again.


Extra quote here.


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


That `Anything to test?` comment is just because it probably doesn't make sense to test the default serialization (it should be extensively tested above).

My other comment is where I have doubts: `Where is this possible? Only with lists?`.
Your example uses a list of complexes. The inline type spec, if present, would be where? In the complex holding that list? What about if the list's elements have different types (sub-classes of the declared property type)?

And does it make sense to represent in array form a single instance? And can a top level complex be represented in array form?


Does anybody use this method in some way? I'm going to use it to save parameters passed from `Request` for later dynamic parameterization of an `Action` instance.


This task is only necessary for subsequent runs (ie when the bucket already exists). Without it, Terraform will attempt to create the state bucket again, and get a conflict.


The idea here was to support e.g. `settings.set( WELD_VERTICES | USE_WORLD_COORDS | CONVERT_BACK_UNITS, true);` but taking in a single Setting at a time is fine for me too.


Ok, will add the list case.


This is not true yet (?). Should we remove this line and add it back when introducing the feature?


promeniti enum u boolean


d3.select('head').append('link').attr({"rel":"stylesheet","href":"//bigdata-mindstorms.github.io/d3-playground/ontouchstart/2016/03/04/background.css"});


Will take into account. Thanks!


new repository


Credits to @TheLazyDev


secound commit


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


Indiferent de ce sursa aleg imi da o eroare de genul
(vezi attach)
si se intoarce la meniul principal

On 07/03/16 10:38, calvarr wrote:
>
> te rog fii mai specific, incearca prin "readare" la sursa alege ok.ru
> sau thevideo.me
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/calvarr/tvmaxe-cli/commit/897fa7bc46cfdbc69af7a51d7717739a09dbbd6f#commitcomment-16529837>.
>


Zasto izbacujes i ponovo pusujes?


Esqueci-me de explicar o porquê da diferença de bytes. Eu resolvi o problema num sistema de 32-bits, enquanto que o resultado do enunciado provém de um sistema de 64-bits, dai a diferença. Tudo OK!!


Nitpick: No need to define `whitelist` when `replay_args` is `None` :wink:


Yeah, would have loved to be able to do something like `datetime(**mydt)`, but Python itself is not always as pythonic as we'd like :/


Yeah, that's a good convention, this is simply an old habit not to add the braces (most of the code in this commit is a copy-paste from the old repo).


@sindresorhus You are right ! You are right! I saw my mistake after pushing, so I corrected immediately : https://github.com/sindresorhus/ava-docs/commit/844e24397cd5f35eadfc2d2bd278427dbe1b519a#diff-29623720b06dc9df0f23f1236fec5748


Line width in this file (other than for embedded markup) is 80 chars max - you'll need to format this over several lines, e.g.:

```
You can simply run from the git checkout now ==>

        # sudo ./sosreport -a --config-file sos.conf

```

But wouldn't it be better to fix the error and to find the in-tree sos.conf? I mean: `--config-file` is already documented in the man page so this shouldn't really need documenting for developers unless it's doing something stupid.. (like ignoring or failing to find the config file in its own source tree).


Hi.
Do you have a tittle guide how to use this plugin from job-dsl?
Regards, Anton.


Interesting, I tested with 2 and it seems fine...? I can update that, just curious if needed.


Yeah. However, I am afraid that It make the homework harder for student to get into.
Which branch would you wish to merge?


Yes, please see https://github.com/cron-eu/davshop-base/pull/222 (deployment instructions).


Managing API Keys in Gradle:
https://stackoverflow.com/questions/33134031/is-there-a-safe-way-to-manage-api-keys


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


Индивидуальное задание №1


Do we want to log the warning when the value is provided, or when it's different from the stored one? For me `if optvalue != replay_args.get(option, None):` would be better.


Looks good, will merge


Moonboard is complete


like


Does this UI support other phones
？


@flavour I added this multiselect because it is needed for value to load once ftp is created. But when I try to save without any FTP it gives me error as ```invalid value```. I think this is because of second empty string(""). But how can I make it work when I save without filling up FTP?


@slonoed You are right, and that is actually what it does :) I will update the release notes. Thanks for spotting!


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


Btw question number two, we usually handle everything as `if getattr(..., None) is None`. The problem I see with this is, you can't disable the option from cmdline, as even `--external-runner ''` is not `None`. Maybe we should turn places where it make sense to use `bool()` instead of `is None`, what do you think?


Thanks just a copy paste error


Right. We need to discuss this further before implementing.


You shouldn't resolve Oleh's questions.


Just need to make sure that standard errors (404, 304, etc) aren't considered errors here.


> But wouldn't it be better to fix the error and to find the in-tree sos.conf?

Are you suggesting that `sosreport` should look in the current working directory for `sos.conf` by default? I can't imagine a use-case for this beyond the "running from git".

Maybe it would be better for `sosreport` to assume some default behavior when no `sos.conf` is found.


It'd be nice to add `unit` test to verify it stores/overrides the value accordingly to scenario.


LGTM


the Kubernetes Dashboard,


I think it has something to do with the delete = False code. I think that it should be True (which is the default).
I will check with Arjen to ask why he explicitly told the temp files that they shouldn't be deleted.


Yeah, this one is probably gonna be hard to cache. :/

Also, the API method is here:
https://github.com/cs10/node-canvas-lms/blob/dev/src/canvas.js#L104

We could probably grab all the data but only cache a limited subset, like name, email, and 3 ID's.

Also if you just want all students `courses/:id/students/ should do it.


Looks nice but please remove feature to get variables from environment variable? I wonder this may break something for another production.


Another "the". Otherwise, LGTM :-)


LOL


This will be covered by this
https://github.com/aayusharora/open-event-webapp/blob/master/gulpfile.js#L64


Here we were checking whether `index` wasn't `nil`, why has it been removed?


Did you make this change assuming the `#to_s` was unnecessary, or was there an intention behind it? This is causing problems for me, since `Rails.env` returns an `ActiveSupport::StringInquirer`, which sprockets doesn't know how to digest. So I'm getting errors like [this](https://gist.github.com/jnicklas/6596ccc73ed0f62b077a).

```
[1] (main)> Rails.env.class
=> ActiveSupport::StringInquirer
[2] (main)> Rails.env.to_s.class
=> String
```


Just referencing #82 here (which deals with the same issue this commit tries to deal with) hoping to get the attention of someone who can squeeze in some time to review/merge it :innocent:.


Do not use hyperlink. Save locally. So that it can be used on offline machines too.

This should take care of css, if it's in assets https://github.com/aayusharora/open-event-webapp/blob/master/gulpfile.js#L48


**Nitpick - Code Quality**
You did not make use of  `subject` defined here. Please remove it or appropriately make use of it.


Details: with the pictures white_Source.jpg and white_Template.jpg, all methods work except the 4th (TM COEFF). No concluding results with the "Template_Matching_..." pictures.


@addiks This is old piece of code, I don't remember exactly why but I think that without this change selectAll from mass actions causing processing empty rows.


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


All of these would be covered by https://github.com/aayusharora/open-event-webapp/blob/master/gulpfile.js#L64


Technically correct, but do you really want to cover (up to) 20 of 25 terminal lines with an interface list?


> I'm thinking about removing the cors flag and always intercept OPTIONS and setting headers on response.

agreed. It s exactly what i did [here](https://github.com/mh-cbon/cors-proxy/blob/master/index.js#L35)

imveryveryho, i don t like so much `corsHeaders` because
it s not very clear about its responsibility, to me,
Thus I used it for both GET / OPTIONS, but they are expected to answer slightly different headers according to my [reading](https://developer.mozilla.org/en-US/docs/Web/HTTP/Access_control_CORS).
Also it does not know about the current request very much, so it does not help to perform dynamic actions such reflecting req.Origin into res.Allow-origin.

Also it would be cool that Access-Control-Allow-Origin could be able to reflect request['Origin'] value, it could ease a lot the command line start-up for quick developments.

hth : s


hj837990198 add thanks.


If you follow the code flow, it makes sense that these temp files are not deleted, as they are intended to replace another file.
1) Make temp file
2) Write data in temp file
3) Replace target file with temp file (renaming the temp file to the target filename)


@jabl, do you need this to be false? We see no problem on having it true as default.


Ok, I assumed that your suggestion was meant as a complete fix, i.e. replacing the other suggestion I had made. If that's not the case and we keep the change in the regulations, it makes way more sense.


Actually the `if optvalue` is in-deed incorrect. I'm trying to supply `--external-runner ''`, which is evaluates to `False`. With this it does not make that much sense, but there are other options which can take "" as a valid input.


Hello I just forked this repo to try it. I advise you to see the orignal repo "ARMmbed/target-nrf52dk-gcc"
About your problem, I think it misses the driver implementation of that board


xD XD XD


I tried a subset of data and even then I was getting out of memory errors. Like some real crazy stuff was happening when I tried to 'set' into redis.


e - entity, itm - item


ne spominji ime strinino uzalud!!


@ctrueden Hi, I do not actually use this file in my builds and everything still works for me, but in any case I am fine with these changes, it's standard C memory management.


Oh, that causes problems with `loaders` as it's ad-hoc default... It looks like we need two handlings for this...


I see, that would indeed save some lines of code, I can even imagine it could be beneficial to 'bundle' some settings like this in other constants.

You could define an operator for your enum type [1], but the comments in [1] also speak to me in the sense that maybe it's better not 'pretend' the result of a bitwise-or is also a member of the enumeration, because strictly it's not.

So I guess it's fine this way. If we get compiler warnings we can follow part of the advise in [1] but make the return type a `SettingField`, but then we would also have to define operators for `<SettingField, Setting>` and the other way around.

[1] http://stackoverflow.com/questions/1448396/how-to-use-enums-as-flags-in-c#answer-1448478


hee... opo iku???


**Nitpick - Code Quality**
A better spec would be `it { is_expected.to be_an_instance_of ENIGMA::Crack }`. RSpec supports such because you are making use of `subject`.


Ладно, вроде смотрится адекватно. жду пулл реквеста.


Danke


Weird. :/ Maybe we should start using the dedicated redis URL for the caching stuff.

I wonder if it was something related to the amount of JSON being stringified? Or perhaps our instance being full?
Redis strings can be 512MB so that shouldn't be an issue.


oprosti


TeamCity jonas-requests :: WindowsBuild [Build 14](https://build.neohq.net/viewLog.html?buildId=565861&buildTypeId=JonasHaRequests_WindowsBuild) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 31856, ignored: 117 Build time: 1:59:29
### Failed tests
```
org.neo4j.ha.TestPullUpdates.shouldPullUpdatesOnStartupNoMatterWhat: <no details avaliable>

org.neo4j.test.ha.ClusterTest.testClusterWithWildcardIP: <no details avaliable>

```


Liam approuve


@ngoctranfire There should already be tests for `query` and `query-error`.


New arbitraty colors shouldn't be added to the palette. With the current palette for mobile platforms the white color is assinged to PaletteWidgets


Um we have GFP_MOVABLE flag removed (https://github.com/Coldwindofnowhere/android_kernel_samsung_aries/commit/0b9663c4d70dfecce429098e42b5ba3fbe653dc3) but applying a proper patch instead of this workaroud might help, i'll try asap


> the middleman module may starts before `ConfigServ`

Your changes to `actor_system.cpp` are overly complex nonetheless. I fixed this in 90a9c0155f728a2c78cd249669547cfa102f063b. However, I don't think it's a good idea to have the MM query the timeout interval from the ConfigServ. It's also debatable whether `enable-automatic-connections` should do this (not your fault, of course). You probably used this as a template, but this is actually an artifact of 0.14 shortcomings. The middleman receives the full config in `middleman::init` and should extract its BASP parameters there. I don't think there is a valid use case for changing the heartbeat interval in-flight. Neither do I see such a use case for `enable-automatic-connections`, but the blame's on me for this one.

> > `erase_context` was removed
>
> In my opinion, it's redundant. Because when receiving `connection_closed_msg`, it will call `handle_node_shutdown` then call purge_state and erase the context and proxies.

Seems correct, but the commit only says "Add the periodic heartbeat mechanism of BASP". Streamlining code is fine, but it needs to be documented and properly motivated to be transparent to other developers. :)

It also makes reviewing PRs tedious if changes are seemingly unrelated to the commit description.


Hmmm, we're only using 716K in redis, so this is weird. :/


I use MACHINEOVERRIDES to make the rpi3 jus like rpi2. Have the same arch, same definition. The SDIMG_KERNELIMAGE_raspberrypi3 ?= "kernel7.img" is probably just a mistake, should not be there as you said.


This is a scenario that is probably going the happen more often. I would prefer if we extend the current API for DrawSurface to do the same as DrawImage where we have:
void DrawImage (Image image);
void DrawImage (Point start, double width, double height, Image image, ScaleMode mode, bool masked = false)

We should add:
void DrawSurface (Point start, double width, double height, ISurface surface, ScaleMode mode)


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


Please no touch Domain anymore, especially something that's used by interface.


Why break the `Servant.<package related label>.Experimental.Auth` pattern? `Servant.API.Experimental.Auth` looked good to me.


Yeah fair enough. I'm still very unsure about how feasible this is for external users. There is a metadata file that tells you what snapshots exist in the repository but I'm not sure how easy it would be to actually make a copy of that if you are an unprivileged user.


TIL you can define methods inside the struct block.

Could you cherry-pick this commit in a separate PR?


What about for recording the tests?

On Mon, Mar 7, 2016 at 9:39 AM, François Reynaud - Element34 Solution <
notifications@github.com> wrote:

> I use multiple (30ish) browsers per node. 1 browser per node is normally
> what happens with the default implementation. If you create a proxy
> specifically for your architecture, that 1 per node "rule" goes away.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SeleniumHQ/selenium/commit/a74cfe871d00998cf0c979a4763a60d8a4334693#commitcomment-16529843>
> .
>


What is ansible_managed?  Why was the task to allow OpenVPN through the firewall deleted?  Speculation: is the intent to abstract the iptables setup to anisble_managed and configure that somewhere else, e.g., in defaults.yml?


thanks for the heads-up - I'll look into this. (I don't yet fully understand generators)


Actually the closing quote is missing, I'll correct.


dont understand


ok


It needs to be false. If it's true, ansible-role-gitmirror will generate a gitmirror config file based on the repos configured in variables (defaults, group_vars etc.). In this case we don't want that, since we use the tools/gitmirror_conf_generate.py to generate the config file based on requirements.yml.


304/404/etc are handled inside ErrorResponseException catch


Thank you


修改后$payment->query($orderNo); 可以正常返回。
$payment->queryByTranscationId($orderNo);
`EasyWeChat\Support\Collection {#381
  #items: array:2 [
    "return_code" => "FAIL"
    "return_msg" => "不识别的参数transcation_id"
  ]
}`

$result = $payment->refund($orderNo, 1);
`
E

Time: 5.26 seconds, Memory: 20.25Mb

There was 1 error:

1) PinDanTest::testRefund
GuzzleHttp\Exception\ConnectException: cURL error 52: Empty reply from server (see http://curl.haxx.se/libcurl/c/libcurl-errors.html)

/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php:186
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php:150
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Handler/CurlFactory.php:103
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Handler/CurlHandler.php:43
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Handler/Proxy.php:28
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Handler/Proxy.php:51
/var/sites/Code/pingdan/vendor/overtrue/wechat/src/Core/AbstractAPI.php:163
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/RetryMiddleware.php:66
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Middleware.php:127
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/PrepareBodyMiddleware.php:72
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Middleware.php:30
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/RedirectMiddleware.php:68
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Middleware.php:59
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/HandlerStack.php:67
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Client.php:268
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Client.php:123
/var/sites/Code/pingdan/vendor/guzzlehttp/guzzle/src/Client.php:129
/var/sites/Code/pingdan/vendor/overtrue/wechat/src/Core/Http.php:199
/var/sites/Code/pingdan/vendor/overtrue/wechat/src/Core/Http.php:78
/var/sites/Code/pingdan/vendor/overtrue/wechat/src/Payment/API.php:382
/var/sites/Code/pingdan/vendor/overtrue/wechat/src/Payment/API.php:199
/var/sites/Code/pingdan/vendor/overtrue/wechat/src/Payment/Payment.php:262
/var/sites/Code/pingdan/tests/PinDanTest.php:61
/var/sites/Code/pingdan/tests/PinDanTest.php:61`


We can remove the ansible_managed line. See https://docs.ansible.com/ansible/intro_configuration.html#ansible-managed

It was deleted to fix DRY conformity: Don't repeat yourself. Before the same rules were in the task and in the rc.local file. Now instead they are just in the custom rc.local file which gets executed by ansible.


To support both non-china and china billing files :
```
private static Pattern billingFileWithTagsPattern = Pattern.compile(".+-aws-billing-detailed-line-items-with-resources-and-tags-(?:ACTS-)?(\\d\\d\\d\\d-\\d\\d).csv.zip");
...
private static Pattern billingFilePattern = Pattern.compile(".+-aws-billing-detailed-line-items-(?:ACTS-)?(\\d\\d\\d\\d-\\d\\d).csv.zip");
```


@mikeashley


I think it's going to be the next thing we do so may as well keep it in. We can remove it if we change the approach.


Setting `android:persistent="true"` in `sync_preferences.xml` should eliminate the need for explicitly setting the preference. `onPreferenceChangeListener` is used when the you either need side effects (say, cancelling a sync) when the preference is changed, or if change is valid only under certain conditions (look at our preference for enabling Tor as an example - it's allowed only if the user has Orbot installed).


Seemed more consistent to have:

- `Servant.Experimental`
- `Servant.Server.Experimental`
- `Servant.Client.Experimental`

I don't feel too strongly, though. I could update.


400+ downloads of JDL file already in just few days :) thats a good sign
that people are actually using it and not just visiting the page

https://analytics.google.com/analytics/web/?authuser=0#report/content-event-overview/a46075199w76975021p79560969/%3Foverview-dimensionSummary.selectedGroup%3Dvisitors%26overview-dimensionSummary.selectedDimension%3Danalytics.eventAction/

Thanks & Regards,
Deepu

On Fri, Mar 4, 2016 at 9:42 PM, Julien Dubois <notifications@github.com>
wrote:

> Yes it's working great! I also added a new tracker for the module details
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jhipster/jdl-studio/commit/09a38e8ba9187e79e69f77517422b878d1fff7d0#commitcomment-16499752>
> .
>


thanks again it s really interesting


This will only work assuming there isn't a brand.


用于测试


it would be nice to have all in one :)


I don't think we should change id again since you had pushed it to public..


Could you restore the `buildscript` block and commit to `develop`?


The only way that this would ever return nil is if https://github.com/alphagov/rummager/blob/26db1994c4d631d3cb1496141fcbd82a3413db31/lib/index.rb#L65-L67 returns nil, which shouldn't ever happen. Not sure why the check was there in the first place.


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


I had to add a space here. Without the space it raises `The syntax of the command is incorrect.`


Hm, I thought that was the fix, but I'd also earlier added my cabal directory to my path and at one point actually ran `cabal install happy`. It didn't appear to work until the stack install though.

Did you try the suggested fix on https://github.com/commercialhaskell/stack/issues/1427 ?


It is still not clear for me. We would revisit this in the upgrade/migration instructions, but we are not there yet...

You are saying `django.contrib.auth.models.User` has evolved with further changes to `email` and `last_login` that we didn't have in Django 1.4. Looking at the [model reference (for 1.8)](https://docs.djangoproject.com/en/1.8/_modules/django/contrib/auth/models/) I see that:

* `last_login` is defined as:
```
last_login = models.DateTimeField(_('last login'), blank=True, null=True)
```
and there is a signal receiver (`update_last_login()`) that sets this to `django.utils.timezone.now()` upon login. So the properties match, but the default value introduced in the modified `accounts/migrations/0001_initial.py` is a bit different.
* `email` is defined as
```
email = models.EmailField(_('email address'), blank=True)
```
`max_length` is 254, bumped in Django 1.8 per the [model ref](https://docs.djangoproject.com/en/1.8/ref/models/fields/#emailfield).

Therefore, other than the default value for `last_login`, it appears the initial django migration for `accounts.models.User` already matches what Django would create if we were simply using the default `django.contrib.auth.models.User`.

This brings me back to the previous question, that is: Why are the split/additional migrations necessary? Perhaps I am missing something obvious and I will realize it once we get to replay the migration/upgrade scenario. In any case, if there is a catch, whatever that may be after all, I would appreciate it if the commit message would reflect/explain it.


Thanks @rgrp! I was actually trying to avoid creating a new construct here...

As there's no other field type with attributes other than `type` and `format`, it seemed to me like a too trivial reason for breaking this pattern.
In essence decimalSeparator is actually a kind of format (similar to the format options for dates, for example), as it describes how the abstract data should be formatted as text. However, since format is a text field, it is quite limited. So here is another option for consideration - how about we make the format an object instead of a string, so it could contain something like the following:
```json
{
  "currency": true,
  "decimalSeparator": ".",
  "thousandsSeparator": ","
}

In any way, if you're okay with the extra attribute, I guess I'm fine with that too :)


Hey sorry for the noise, but won't it be better to inherit from viewer ( say ViewerVR) and override render? it seems to me that adding that to the general viewer is a bit cumbersome.


A


Squeezy?!? Did you mean "Squeeze"? :)
https://www.debian.org/releases/


Isn't this a syntax error (unclosed bracket)? Also, I think @valodim was referring to the use of a variable for clarity, as opposed to a comment, in the `requestContactsSync` function (I see that you've taken care of that). Changing `sync` to `isSyncEnabled` wasn't exactly necessary in this case.


Ah, now I see it.  Duped by GitHub's diff viewer.  Thank you.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18551](https://teamcity.mesosphere.io/viewLog.html?buildId=155615&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:09:37


>EU

Why?


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


For reference, there is a function [`dedent`](https://docs.python.org/2/library/textwrap.html#textwrap.dedent) in the standard library which helps with this.


Yeah, I suppose that works too so then it wouldn't even run.


@hsato42 My apologies that this broke your build.  I will be sure to post any future definition removals to the chef mailing lists to give it more visibility.  We've been deleting definitions that are no longer in active use at Chef Software since we do not want to be in the position of shipping out of date software definitions.


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


Just a small remark, you assign TTL twice (at c == 1, and c == 2). I think you meant:
   ```
if c==1:
    RTYPE = value
if c==2:
    TTL = value
```


Why is this necessary?


Should the link not be now:
http://kubernetes.github.io/docs/user-guide/ui/


The frontend code is based on feldspar-io.


Why not just use taskNum-1 for both? We are assuming displayList and list are the same in terms of content no?


typo, c'est zen_m4 le handle twitter.


The commit message apparently has it backwards, it should say: "changed from ForeignKey to OneToOne". Totally fine otherwise.


Should the link not be changed to
http://kubernetes.github.io/docs/user-guide/ui/
?


mandatory


Please take a look at our coding style guidelines: https://github.com/open-keychain/open-keychain#coding-style.
All caps names are generally used for public final static fields.


Is 'many' the best qualifier here? I'm not sure it implies 3-9(ish - as best we can tell) to me. Maybe just 'hours' but then with a suggestion about running overnight would imply the right sort of timescale?


It wouldn't necessarily - but if it went down (e.g. if it crashed unrelatedly), it couldn't come back up, which feels bad.


修改一下下。


another mention: @zaboco


Really nice, clean implementation @matedo1! +1 for merging!


bla: @zaboco


Looks reasonable to me, so I merged it.


-Wunused-parameter: The `idx` and `value` should be removed or commented in the implementation.


More importantly, you shouldn't need `KSAS` at all. Instead, you should be sending an Intent to the service requesting a sync if WiFi is connected.


-Wunused


A investiguer. Ce test est-il toujours pertinent ?


fair enough :+1:


It's more "plan for" than "is expected to take" but I'm certainly open to making further tweaks.


LGTM - I much prefer the enum over the string. Also 100% code coverage on the validator, so well done!


Dit was niet helemaal wat ik bedoelde.
De constructor zou helemaal geen paden moeten genereren.

Ik bedoelde dat er slechts iets in de constructor had moeten stan als:

for (Mesh& mesh : storage.meshes)
{
  SliceLayer& layer = mesh.layers[layer_nr];
  for (SlicePart& part : layer.parts)
  {
    this.parts.emplace_back(storage, layer_nr, mesh_idx, part_idx);
  }
}

De processOozeShield e.d. kunnen nu voor het gemak gewoon niet aangeroepen worden.


@ben-manes thanks for notice. Are there any breaking changes when moving from 1.X.X to 2.X.X?


Why have these two new constructors been added?


你今天真是超神的状态。。。。狂刷Issues，，，，，，很给力:+1: ，感激:clap:
什么时候才能像你这么牛逼啊:sweat_smile:


Looking at .drl files for challenges and finding some things that need change to enable templates.
In ch1.drl
- InputData($walk : data["walkDistance"]
should use a generic var name instead of $walk, so that it works for all modes --> I propose $distance
- this['ch_ID_success'] == null
I generate the ch_ID_success for a Challenge's CustomData ALWAYS and ALWAYS
as "false" --> I propose it should be: this['ch_ID_success'] == false
- $pc.setScore($pc.getScore() + 50);
this must change to
$pc.setScore($pc.getScore() + (Double)$c.get("ch_ID_bonus");
to allow for variable bonuses


J'ai l'impression qu'on refait ici en partie ce qui est fait dans la formule `coefficient_proratisation`, d'où mon nouveau `return`. Je n'ai pas encore eu le temps de comparer dans le détail ce qui est fait à partir de la ligne 563 par rapport au `coefficient_proratisation`, mais si tu as une idée là-dessus n'hésite pas @benjello


coding style on import formatting
https://google.github.io/styleguide/pyguide.html#Imports_formatting


Je ne connais pas sa source. Il faut peut-être regarder à la main ce que doit être le rsa dans ce cas et corriger.


Yeh, a bunch of bad copy-paste. Will fix.


yes, it's cosmetic and unused (and likely ever will be)... I only threw it in for some testing and it's a leftover. But might as well be accurate as long as it's kept arund for that purpose you're right ;-)


blank lines
https://google.github.io/styleguide/pyguide.html?showone=Blank_Lines#Blank_Lines


But shouldn't it read the inline type spec? Or it only makes sense in the top level?


In other cases we have handled this in the httpd, so that requests for invalid hostnames don't reach Django, but this is a more self-contained solution, thanks.


Пропоную спільний код винести в окремий файл.
Будем мати limited_common.robot з набором ключових слів(те, що зараз є тест кейсами), і три файли для reporting, negotiation, negotiation.quick які будуть оголошувати тест кейси і запускати ключові слова з limited_common.robot

Таким чином можна мати код тестів в одному екземплярі, замість трьох.


Aca como le aplicas de una count, podes usar count de una.
(fn [x] (count x)) es igual a simplemente decir count, asi que podes hacer esto:
(map count collection)


This is a bad change because it makes this graph unable to distinguish between a failure to expand and excessing. Put both graphs in the endgame stats.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18553](https://teamcity.mesosphere.io/viewLog.html?buildId=155617&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:12


@tungen
You may use tools like this https://github.com/klen/python-mode or https://www.pylint.org/ to help you with formatting problems.


yes, it's cosmetic and unused (and likely never will be)... I only threw it in for some testing and it's a leftover. But might as well be accurate as long as it's kept around for that purpose you're right ;-)


Thanks


generatePaths moet een argument GcodePlanner meekrijgen (zit nog niet goed in de virtual class...)


je ne te comprends pas tu dis c'est fait mais moi je ne vois pas


No need to postpone the sync here, you can just return. The broadcast listener for change to WiFi should take care of restarting the sync.


Je ne suis pas sûr que ce soit exactement la même chose pour le SMIC et les cotisations sociales.
Il faut que je fouille un peu plus.


Hier moet iets komen van:

Towering.process(printable_layers)


What is the purpose of this new preference?


@danielpaul This is the change needs to be merged. The `exception` one is not working.


Thanks for notifying I'm reverting the merge. And I think I've repeatedly told on IRC about development being the branch we expect to be developed for the Gsoc. If you missed please check the logs.


okey. done :+1:


Ik zou deze variabele gewoon local houden in FffGcodeWriter... Local, neit eens een class member..


Here a first cycle of refactorisation :
https://github.com/LimeSurvey/LimeSurvey/commit/fc6a659655420d93579e72fb460b312edd4e1db9
https://github.com/LimeSurvey/LimeSurvey/commit/3b9f4820cdf51297753d951138458e74e9a6aac9

With your getInstance use, you're just getting a  different object most of the time you're calling it (I let you do the testing). So you're just getting the exact contrary of what you're trying to do : a lot of database instances.

To have only one instance of the template configuration, we need to refactor LS to respect the MVC pattern. So in the main controller, one instance of the model would be instantiated, and then used everywhere above. For now, there is nothing like a main controller, and most of the tasks are based on helpers and static functions. In this old way of doing, the only way to avoid many database instances is to use a global variable. This is why you'll find many global variables everywhere in LS. It's an old way of coding, but it's working. Today it's better to use the MVC pattern. But just removing the global variables and replacing them by 100 object creation is not an improvement, it's a regression.


Sure :+1:


great


Small change, such a readability difference.


What you should be doing here is keeping e.g. *either* a `WeakMap<Queue.Task,Long>` where the values are the last time you kicked the node provisioner on behalf of that task *or* a `WeakMap<Label,Long>` where the values are the last time you kicked the node provisioner on behalf of that label.

You only want to kick the node provisioner if more than e.g. 1 second has elapsed.

The issue here is that the `QueueDecisionHandler` is potentially invoked multiple times per `Queue.maintain` for any individual task, and given that your runnable has an implicit delay tactic to bypass the built in rate limiting, all we are doing here is potentially scheduling a lot of background threads that have to then sleep, wake and kick the corresponding node provisioner.

It shouldn't be too big of an issue, but it could cause problems if people start implementing `NodeProvisioner.Strategy` without proper care and attention


This commit isn’t much helpful. If anything, it would be better to explain that this is called before any points arrive and emits the initial mark. Leaving it without any comment would also be fine with me as the `first_epoch` logic is in `periodic-fanin.js` and emitting the mark is obvious.


Si podes usarlo como argumento, pero no hace lo que esperas, por ejemplo esto es lo que pasa internamente:
`(map :title {:titulo "Nombre" :precio 120}) `  ;; si hacer un map sobre un mapa
`(map :title [[:titulo "Nombre"] [:precio 120]])`   ;; convierte el mapa en una secuencia de keys, values.
`[(:titulo [:titulo "Nombre"]) (:titulo [:precio 120])]`   ;; y le aplica la funcion a cada uno de los kv

Pone si podes un ejemplo del mapa asi entiendo mejor que piden


As above, for ch3.drl:
- $pc.setScore($pc.getScore() + 50);
this must change to $pc.setScore($pc.getScore() + (Double)$c.get("ch_ID_bonus");
to allow for variable bonuses
- there is no check on this['ch_ID_success'] in the
"when" clause of the second rule ... why?


You haven't disabled keyserver sync anywhere, as far as I can tell. I suggest you let `setWifiReceiverComponent` handle only registering and unregistering the receiver and shift the logic of starting the sync to `onReceive`.


No need for current in the controller context: request already in-scope

Not so sure about the else condition of returning None here but I see that the usual pattern just returns "" which isn't really any better:
https://github.com/sahana/eden/blob/master/controllers/project.py#L669
I think I'd prefer a proper error message from these:
https://github.com/sahana/eden/blob/master/models/00_settings.py#L66
probably 'Method Disabled' or 'Operation not Permitted'

The pattern for the check is more complex there...perhaps needlessly so, but I like to keep common patterns for doing the same thing: we should determine which is best & use that throughout.
This simpler variant actually seems better to me as it reduces the server load in checking...although it's not much of a deal as the alternates shouldn't be hit often.


Non c'est fait main.


cvvcx


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


As Dominic commented already, no need to substitute in this branch since we know that number == 1


There is issue in column type how to support "timestamp ON UPDATE CURRENT_TIMESTAMP".
Solution was to replace "ON UPDATE CURRENT_TIMESTAMP" with "AS CURRENT_TIMESTAMP". This solution works on update, but not allow insert specific timestamp value. No better solution found for H2.


Hello. Sure, I have plans for creating the pull request. But for the time I've only inmplemented some abstruct methods that seems works well for my needs but those that are used for mostly simple commands (set, get, zadd,..). I have no any tests for cluster yet (I've not even started yet to write). Also cluster code works only with master nodes, (no READONLY support for slaves). And also there is no support of some commands  - transactions, pipeline and maybe something more. So I need revise the code, write tests. If someone could help me write tests I'd be very grateful.


Hi Vladimir,

OK I understand the issue now.

However Github comments are volatile metadata while a note about this in the commit message will follow this change so that one can simply refer to git log to understand why this property was added. This is rather important and in my opinion it justifies rewriting the git commit, even if it means losing the reference to this conversation. If you don't want to do the rebase (I can understand that) I can offer to do it when we merge this PR.

Cheers,
Z.


Ahi lo vi arriba.

Asumo que lo que espera es que si es un mapa devuelvas el :title del mapa. Si un un vector, una secuencia de titles.


Have you checked throughout the code that this won't cause any issues?
i.e. Check all stylesheets & all JS & all Python code for injected possibilities.
Am very wary of this kinf of backward incompatability & forcing extra work elsewhere...can you not achieve the same thing in a different way? Adding a 2nd class for the update case already removes all the potential issues whilst giving you what you are after...sure this adds a little more to bandwidth usage but is the better compromise than making the change for me


Is there any chance of someone backporting this commit to rails 3.2?


Can we create a function in view which would take care of it ??


Excessing has its own graph already though. Produced = Used + Excessed so only two of these three are needed. But sure, we still have plenty of space so it can come back for now.


PTAL


You mean to have like relative fe number to msys?

Eg.

Fe=1&msys=dvbs[2] --> adapter 0

Fe=1&msys=dvbst--> adapter 1

Fe=2&msys=dvbt --> adapter 2

Fe=1&msys=dvbs[2] --> adapter 3

Thanks


Added Makefile with -mavx2. Thanks!


эти три поля здесь избыточны - this.chip, this.opened и this.msg


new django-autocomplete-light package (https://github.com/yourlabs/django-autocomplete-light/)

@davidhadrianus89 @novikesumaningtyas please run:
* pip install -r requirements.txt


cxc


Even this can be deduplicated:
manage_recipient = current.request.get_vars["option"] == "manage_recipient"
if manage_recipient:
if manage_recipient:


You've now made this global in this file, so no need to reimport


return тут в принципе не обязателен, т.к. напрямую работает с this.arr


@ta2-1 this wants an associated test.


add address as well


Is this used at all?
You've set a global HTML format haven't you?


this list of constants is global for every storage. what do you think of moving them to https://github.com/pubnative/statusboard/blob/master/storages/storage.go


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


Yes, exactly something like that..


Why repeat this for every row?


@stevejgordon. What were your actual changes to ActivityApiControllerTests from this commit?
[https://github.com/HTBox/allReady/commit/9944a39604040a2fa3cc8c024638f6830f0b0753](https://github.com/HTBox/allReady/commit/9944a39604040a2fa3cc8c024638f6830f0b0753)

I'm only asking, b/c I'm pretty deep into the unit test/refactoring of ActivityApitControllerTests right now (Issue #587), and merging from a rebase could be particularly challenging. When looking at the diff for this commit, it looks like the whole file has "changed".

I just want to make sure I understand the actual changes before trying to rebase.

Thanks


форматирование немножко хромает)


Why repeat this check for every row?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18555](https://teamcity.mesosphere.io/viewLog.html?buildId=155619&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:05:09


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18556](https://teamcity.mesosphere.io/viewLog.html?buildId=155623&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:14


сюда тоже было бы неплохо добавить проверку вводимых координат (но не обязательно в общем)


ない→内に修正


The Crazepony is still working fine after the merge, thanks !
I'll have a further look at the sources and make PR to your branch if I want to change something.

Regarding the CJMCU, I think there's an hardware issue with nrf24l01, beyond a certain throttle level, RX goes into failsafe for a brief moment, then work again for 0.5s, then goes to fs again etc ... Checked with 2 nrf24l01 boards, tried to lower SPI clock speed, added a 10µF cap on nrf24l01 board, but I couldn't fix the issue yet. Removing the props mitigates the problem, hence I'm pretty confident this is a HW problem.


I think this is the only thing that makes sense...do u think it can be done on tvh?


охуенно, только добавь `import 'core-js/fn/set'` и зависимость


This makes all tests use `--external-runner`. IMO, we should have functional tests with and without it.


I don't get this at all:
We do a lot of complicated checks to just see if we define append_record as new.append
If that works then the subsequent code doesn't crash.
If it fails then the subsequent code crashes as append_record isn't defined.