1: D400 First line should end with a period (not 'n')
1: D205 1 blank line required between summary line and description (found 0)
27:1: E402 module level import not at top of file
28:1: E402 module level import not at top of file
29:1: E402 module level import not at top of file
31:1: E402 module level import not at top of file
32:1: E402 module level import not at top of file
33:1: E402 module level import not at top of file
34:1: E402 module level import not at top of file
35:1: E402 module level import not at top of file
36:1: E402 module level import not at top of file
37:1: I101 Imported names are in the wrong order. Should be EmailMessage, EmailMultiAlternatives
37:1: I201 Missing newline before sections or imports.
37:1: E402 module level import not at top of file
38:1: I101 Imported names are in the wrong order. Should be AbstractFormatter, DumbWriter
38:1: I100 Imports statements are in the wrong order. from formatter should be before from flask_email.message
38:1: I201 Missing newline before sections or imports.
38:1: E402 module level import not at top of file
39:1: I101 Imported names are in the wrong order. Should be StringIO, iteritems
39:1: I201 Missing newline before sections or imports.
39:1: E402 module level import not at top of file
40:1: I100 Imports statements are in the wrong order. from time should be before from six
40:1: I201 Missing newline before sections or imports.
40:1: E402 module level import not at top of file
42:1: E402 module level import not at top of file
43:1: E402 module level import not at top of file
44:1: E402 module level import not at top of file
45:1: E402 module level import not at top of file
47:1: E402 module level import not at top of file
49:1: E731 do not assign a lambda expression, use a def
84: D400 First line should end with a period (not 'd')
84: D205 1 blank line required between summary line and description (found 0)
97:25: E124 closing bracket does not match visual indentation
127: D301 Use r""" if any backslashes in a docstring
127: D400 First line should end with a period (not 'y')
127: D205 1 blank line required between summary line and description (found 0)
210:24: E127 continuation line over-indented for visual indent
211:24: E127 continuation line over-indented for visual indent
212:24: E127 continuation line over-indented for visual indent
236:53: E502 the backslash is redundant between brackets
237:25: E128 continuation line under-indented for visual indent
237:50: E502 the backslash is redundant between brackets
237:28: E251 unexpected spaces around keyword / parameter equals
237:30: E251 unexpected spaces around keyword / parameter equals
238:25: E128 continuation line under-indented for visual indent
238:50: E502 the backslash is redundant between brackets
238:28: E251 unexpected spaces around keyword / parameter equals
238:30: E251 unexpected spaces around keyword / parameter equals
239:25: E128 continuation line under-indented for visual indent
239:44: E502 the backslash is redundant between brackets
239:31: E251 unexpected spaces around keyword / parameter equals
239:33: E251 unexpected spaces around keyword / parameter equals
240:25: E128 continuation line under-indented for visual indent
240:46: E502 the backslash is redundant between brackets
240:35: E251 unexpected spaces around keyword / parameter equals
240:37: E251 unexpected spaces around keyword / parameter equals
241:25: E128 continuation line under-indented for visual indent
241:35: E251 unexpected spaces around keyword / parameter equals
241:37: E251 unexpected spaces around keyword / parameter equals
257:40: E127 continuation line over-indented for visual indent
283: D200 One-line docstring should fit on one line with quotes (found 3)
294: D209 Multi-line docstring closing quotes should be on a separate line
294: D205 1 blank line required between summary line and description (found 0)
397:21: E266 too many leading '#' for block comment
409:13: B901 blind except: statement
434: D401 First line should be in imperative mood ('Remove', not 'Removes')
434: D400 First line should end with a period (not 'n')
434: D205 1 blank line required between summary line and description (found 0)
456: D400 First line should end with a period (not 's')
456: D205 1 blank line required between summary line and description (found 0)


26:1: F401 'CFG_DEVEL_SITE' imported but unused
80: D102 Missing docstring in public method
109: D209 Multi-line docstring closing quotes should be on a separate line
109: D203 1 blank line required before class docstring (found 0)
109: D400 First line should end with a period (not 'f')
109: D205 1 blank line required between summary line and description (found 0)
115: D102 Missing docstring in public method
155: D102 Missing docstring in public method
160:1: E303 too many blank lines (3)
160: D210 No whitespaces allowed surrounding docstring text
160: D209 Multi-line docstring closing quotes should be on a separate line
160: D203 1 blank line required before class docstring (found 0)
160: D400 First line should end with a period (not 'l')
160: D205 1 blank line required between summary line and description (found 0)
185: D102 Missing docstring in public method
206: D102 Missing docstring in public method


See now it performs as expected, awesome work bro !


@lindamartin yes, that would be fine.


+@hasPermission('user.index')
     aqui mostra algo relacionado a essa permissão
 @endcan

？？？？？？


Good question. jsjws uses yui but only in the ASN1 code. I'll see if I can remote it in the node module.


If not value, then you don't increment the column index and the subsequent column offsets will be off.


Syntax error


took out height


Note that this was already reverted in be692a2ceb183734653d6d5167f43562efbe871a due to the build failing.


I think reverting is the good way. The patch was a plain text approach. It does not work properly in html context.


I will add acceptance test for this bug


@pynicolas OK!


Thanks for noticing @brasmusson - I just merged the PR without paying enough attention. I've cleaned up my mess.


saw approval from shino
at https://github.com/basho/riak_cs/commit/5c80e221e08b3e68675841efb65cc605698e394e


merging basho/riak_cs/feature/no-timeout-delete-fsm = 5c80e221 into borshop-integration-1249-feature/no-timeout-delete-fsm


basho/riak_cs/feature/no-timeout-delete-fsm = 5c80e221 merged ok, testing candidate = eaddb487


Да я понимаю, основные эксперименты с конфигом я сюда и не коммитил, но иногда просто лень )


@thet see #108 ready for 1.x branch ...


Tangina bat ganyan ginawa mo? Hahahaha


If I understand right, we don't need some actions: they are define in reducers/mesages.js and channels.js


Are you going to integrate this very change (setting a timer) anyway? (Or I should be on your behalf.) Otherwise I'm going to remove this branch.


Very nice that you added comments near `if` statements :+1:


Loops do not create a new scope.


Forgot to mention that it is for Permalinks'.


oh, good to know. I'll pull that out.


Wouldn't it be better to kill validation completely from Susy?


@sasrin I've pushed the changes to this branch https://github.com/jsdelfino/d-abacus/commit/083dbeb188fd83750f28befee77052a14b398650

With these changes the tests run with batching enabled and handle out of sequence usage submissions.


Pass `$validate` directly.


Already done: 780d7b5aba39806655e2f96f20f247a1b6f5c997

Go ahead and remove the branch.


benutz da ng-src="signatureImage", es funktioniert zwar au so, aber es macht en http call uf {{ signatureImage }} bevors die variable füllt, was ja ned so schön isch. ng-src verhinderet das


It is a typo here:

    \cs_new_protected:Npn \@@_unit_format_literal_extras_aux:nN #1#2
      {
        \exp_args:NNf \tl_replace_all:NnV \l_@@_unit_tl
          { \char_generate:nn { #1 } { 12 } } #2
      }


wetsch ned im fehlerfall en button azeige und soschd nur d'signature? denn chöntsch da en $scope variable uf true setze


Better for what? We might get some speed improvements (we could test to see how much) — but how else do we consistently give useful errors when bad values are passed?


blog/_posts/2015-09-11-BaRuCo. It’s not just the talks, it’s the people.md

can you add the date in the permalink, too? like this `blog/2015-09-11-baruco-its-not-just-the-talks-its-the-people`


Don't worry, it doesn't matter because you won't use DQL soon ;-) The best way to build Doctrine querys is with an object-oriented way, be patient...


@actionless
Can you create an issue for that, please?
Have you bisected it?


Thanks! BitHub has sent payment of  $34.45USD for this commit.


please add all images to the `/img/blog/2015/` folder, like you did in your first blog post, and change the link accordingly. See more information [here](https://github.com/rails-girls-summer-of-code/summer-of-code/blob/gh-pages/README.md#images)


it seems -client is always set regardless of what I specify in
--host_jvm_args, am I'm right here?


Please integrate the benchmarks into one program.
More programs -> more compile time.


Why do we have duplicate validation?


I moved the variable definition outside the inner scope, the re-assign a new value to an existing variable on each execution instead of creating new variables. This change helps avoid a lot of garbage collection in a web audio app that calls a bunch of memoized functions, on every tick.

https://github.com/adamrenklint/meemo/blob/master/index.js#L135


I still could not figure out how to run bazel on a VM that does not support -client.
Running
strace bin/bazel  --host_jvm_args="-server"
I can see that the -server option is picked up.

execve("/usr/java/latest/bin/java", ["bazel()", "-client", "-Xms256m", "-XX:NewRatio=4", "-XX:+HeapDumpOnOutOfMemoryError", "-XX:HeapDumpPath=/sapmnt/home1/d"..., "-Xverify:none", "-Djava.library.path=/sapmnt/home"..., "-Dfile.encoding=ISO-8859-1", "-server", "-jar", "/sapmnt/home1/d041705/.cache/baz"..., "--batch", "--install_base=/sapmnt/home1/d04"..., "--install_md5=fc73563854b9d52217"..., "--output_base=/sapmnt/home1/d041"..., ...], [/* 103 vars */]) = 0

but -client is still used

which leads to an error on my vm (-client not supported)


achso, ja du chasch natürli au im html luege eb de successhandler-wert gsetzt isch. in javascript wird ja jede non-null wert als true ghandlet, darum wird au es bild als true ghandlet und darum tuet das so wie du das gmacht hesch


you have to change the java_toolchain (default one is declared in tools/jdk/BUILD) to add a `jvm_opts` attribute set to `[]` and that should work,


this is a new class


Are you using `resolver`? The values are still getting set and reset so garbage is still created though one is a string (minimal) the other is the cache reference (never collected) and then the `args` alias of `arguments`.


Susy calls several of these functions directly. In any case where the actual grid system (or a future plugin) needs to call a Su function directly, Su should be able to handle validation. But, anywhere Su calls itself, we should have the validation handled already. So we don't know where the validation will need to happen — but we only want it to happen once.


That makes sense I guess.


```ruby
name = {
  name1: 'test1',
  name1: 'test2'
}
```


```ruby
numbers.each { |num| print num }
```


Ok, Thanks! I had hoped I could   be overridden.


На сотовых браузерах эта функция не работает


Would prefer this to be `'You\'re currently opted out of private messaging, go to the <a href="@url">account settings form</a> to opt in.'` for accessibility purposes ("click here" links are not helpful to screenreader users)


Well the location of the java_toolchain itself is over-writable with
--java_toolchain
On Mon, Sep 14, 2015 at 10:34 AM kohlerm <notifications@github.com> wrote:

> Ok, Thanks! I had hoped I could be overridden.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bazelbuild/bazel/commit/1fa163c42ee26a72c21c118ec820d677a431ee49#commitcomment-13207150>
> .
>


See, it was simpler to update than on the older formspec ;)


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1631


saw approval from shino
at https://github.com/basho/riak_cs/commit/6b76e4386faded570eef99cd1ac505eab3d37d6a


merging basho/riak_cs/kuenishi-patch-3 = 6b76e438 into borshop-integration-1251-kuenishi-patch-3


basho/riak_cs/kuenishi-patch-3 = 6b76e438 merged ok, testing candidate = 91cb212b


han bar mini changes gmacht (form-control-static klass hinzuegfüegt und ng-src) aber susch isch super ;)


fix for calling removeProvider when RuleProvider service is unregistered
https://github.com/marinmitev/smarthome/pull/90


I wrote it this way for readability. The other arguments have clear meanings in the variables we pass. Using a named argument for `false` makes it clear what's going on. Are you proposing to remove it for performance, simplicity, or consistency? I'm open to removing, but not sure what the advantage is.


lgtm


有空再面咱一发吧XD


The flight code should not be able to arm if the EKF is unhealthy so I will move the vehicle arm status check as suggested. If we do end up with some race condition and the vehicle is allowed to arm when the EKF is unhealthy, they an immediate reset would not be a bad option.


変更完了です。


According to [this SO post](http://stackoverflow.com/questions/23274679/renaming-multiple-columns-in-one-statement-with-postgresql) you can combine ADD COLUMNs but not RENAME COLUMN. Do you think there is a worthwhile performance benefit to combining the ADD COLUMNs into a single ALTER TABLE?


+1
If someone knows an up-to-date fork I would be pleased to know


May wish to incorporate or point to Matt's example on how to utilise metered components at Signup.
Great one-off signup charges such as equipment, itemised setup/install charges etc.


I'm assuming this first check is really checking to see `if object is a string`? I may be wrong, but I assume that `hasPrefix` is only valid for NSString so would it be more explicit to use `[obj isKindOfClass:[NSString class]]`?


Cool !
Maybe it would be benifitial to add the schema and validation to the book model itself?
Theres not right or wrong ;)


Thanks for fixing that!


i will create one when i'll bisect the particular commit


@afedchin if you are bored my OCD would love to see this adressed =)


If the line fits, no need to break it.


Let's always leave a leading space for the self-closing JSX tags


> what does "65 / 91 ( 71%)" mean?

I don't know.


Let's avoid `let` when in function scope


Proper indentation


Oh, I see you had this before.
I really prefer this way to hold the URL clean, if there is no force-parameter.
But defining the default-parameter (```force = false```) additionally makes sense for readability.


thanks


Performance improvements on DecisionTree with this change was 3x time.
Data size was 100K dataset, with 100 features, and other default values for DecicsionTree, on Curly.

All review comments except documentation change has been addressed.


These are debatable – should we always nest even when the line itself fits?


String 안에는 뭐가 들어가도 상관 없지 않나? 물론 수정 내가 한 건 아님


- Vâng, nếu làm theo kiểu batching thì nhanh hơn và scalable hơn
- Nhưng sao lại không dùng Promise.all hả anh, cái Promise.all em đang dùng là để biết khi nào việc update toàn bộ các tile trong một thao tác xóa là hoàn tất.
 -  vì khi xóa một điểm hoặc môt lúc nhiều điểm sóng thì phải update lại toàn bộ tile chứa các điểm sóng đó, Mà các hàm update tile là các hàm asynchrouns, nên để biết khi nào một thao tác xóa kết thúc thì phải biết khi nào tất cả các thao tác update đó kết thúc.


fast-forwarding develop to borshop-integration-1249-feature/no-timeout-delete-fsm = eaddb487


Merge sha 91cb212b is stale.


merging basho/riak_cs/kuenishi-patch-3 = 6b76e438 into borshop-integration-1251-kuenishi-patch-3


basho/riak_cs/kuenishi-patch-3 = 6b76e438 merged ok, testing candidate = 661fb4e6


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/626e59434a0537044c8c7063c85e371dcd1106a2


basho/riak_cs/merge-from-2.0 = 626e5943 merged ok, testing candidate = 5dc5ac2a


merging basho/riak_cs/merge-from-2.0 = 626e5943 into borshop-integration-1252-merge-from-2.0


If it fits I like to have it on one line.
No forceful nesting for me.


No reason. Code amended as suggested.


should the pictures have to have some special names?

2015-09-14 10:24 GMT+02:00 Laura <notifications@github.com>:

> please add all images to the /img/blog/2015/ folder, like you did in your
> first blog post, and change the link accordingly. See more information
> here
> <https://github.com/rails-girls-summer-of-code/summer-of-code/blob/gh-pages/README.md#images>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Czarrrna/summer-of-code/commit/88b66e7d3aaf496f16905a5f3876d5ae414acd9d#commitcomment-13207005>
> .
>


@porcelli What's the ```kie-wb-common-showcase``` all about? Another web-app to maintain?


You can call me picky, but again... the 2nd check might be cutting corners a bit too much? :(
`... && [[NSURL fileURLWithPath:obj] isFileURL])...`

(unchecked code)


I changed it because I saw at some point in the code that it was the convention in Mars. Having them on the header helps to make the code more readable and maintainable because all the dependencies can be seen by just taking a look on the header.


I think this change has made the mapping take *ages* now...
```
Phase  4  : 50818 unrouted; (Setup:0, Hold:173538, Component Switching Limit:0)     REAL time: 3 mins 4 secs
  Intermediate status: 371 unrouted;       REAL time: 33 mins 9 secs

Updating file: atlys_hdmi2usb-hdmi2usbsoc-atlys.ncd with current fully routed design.

Phase  5  : 0 unrouted; (Setup:12863, Hold:181573, Component Switching Limit:0)     REAL time: 37 mins 14 secs
  Intermediate status: 24 unrouted;       REAL time: 1 hrs 7 mins 20 secs

Phase  6  : 0 unrouted; (Setup:1248, Hold:181573, Component Switching Limit:0)     REAL time: 1 hrs 7 mins 54 secs

Updating file: atlys_hdmi2usb-hdmi2usbsoc-atlys.ncd with current fully routed design.
```


Wrong order checking, it could not check until clDbalLibInitialize calling to initialize


@miigotu I keep forgetting and by the time I remember it's already too fucked. :cry:


Ừ, nếu các việc update các tile cũng là async thì dùng Promise.all là đúng rồi. Anh quên mất là update cũng là async


Ok, so I'll keep the force = false for readability and then create the url with ``` var url = `${config.apiURL}v2/apps/${appId}```` as it was before.


Weet je zeker dat je de implementatie van TSNE van sklearn wilt gebruiken? Ik heb daar gekke dingen mee gezien... Beter is een van de originele implementaties en voor grote datasets zou ik de barnes hut implementatie gebruiken. Zou je eenvoudig tussen kunnen kiezen door een cutoff te gebruiken bij bijvoorbeeld 1000 dimensies.


I see yours


* you need `stringByExpandingTildeInPath` ;-)
* can we make `[@"~/Library/Application Support" stringByExpandingTildeInPath]` a const in QSPaths.h?


this is not making sense, it's an infinite loop.... cause the conditions ```$i < $i + 1``` will always be true


is it worth adding `if(url) {...}`?


failover not fail over


again, tilde needs expanding, plus make it a const. Maybe the whole thing should be?


a way to figure out which post they belong to would be great, so maybe something like a prefix for all the images you use, such as `baruco-warsaw` or `baruco`, and then add what the image is about, eg: `baruco-warsaw-aaron-patterson` and `baruco-warsaw-matz`


I think these strings can simply be removed from here, and the framework strings referenced directly where these are used.


Does it work yet?


Should we pre-create the table in ARM templates just like pre-creating the container "bosh" and "stemcell"?


Yes, cool, thx.


This probably trouble if > 10 databases loaded and cached.


The functionality is there but we don't know why it is not in the documentation anymore.


the first one is the primary and the second is the backup


That was not my point, I do like named arguments. My point was to pass the `$validate` variable from the function call directly without altering it but I misunderstood what you were doing when writing those comments.


Yes. Have Updated in next submit.


according to http://hibernate.org/validator/documentation/getting-started/


Done.


It works with that DLL in `C:/windows/system32` thanks!


Yes


Does this fail on the current version?


Setup 10 databases with > 1.000.000 records to verify it works.


共三处
1.67-70
2.82-84
3.101-104


Maybe add ```json for better syntax highlighting.


Maybe add ```bash for better syntax highlighting.


¿Esto hay que quitarlo por algo?


Về mặt API, cái file L.GridLayer.MaskCanvas.js là implementation của tất cả các tính năng liên quan tới bản đồ, vậy thì việc insert/remove điểm sóng phải implement trong file này chứ. Còn main-1.0-dev.js chỉ gọi tới các hàm của layer thôi. Em nên thiết kế sao cho việc insert/remove phải thực hiện hoàn toàn bởi layer thôi.


The jQuery version is too high for us. It's good for jQuery 1.7+.


Perhaps it should not be commented out then?


There is a windows system call that can update the load paths at run time if you wanted to bundle it with the gem: https://msdn.microsoft.com/en-us/library/windows/desktop/hh310513(v=vs.85).aspx


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


I need bit help. I am trying to add created_on field on api sothat i can fetch tickets depending on date range. I added below command. but in vain. can you please help me?
curl -H 'Accept: application/xml' -H 'Content-type: application/xml' https://open.panduitdms.com/api/tickets/search?query=queue.id:1233%20AND%20status.id:\(\<50\)\&sf=created_on\&r=1\&access_key=dd\&per_page=500\&page=1\&created_on=2014-12-31T24:60:60Z%20TO%20*
but its not filtering by date


yes


this issue occurs on single values enum without values (last row in acceptance test)


@borshop: retry


spelling


Great. And what about multiple-valued with null values? That looks like a different code path.


I reverted 1080p support. We first have to work on the encoder: https://github.com/timvideos/HDMI2USB-misoc-firmware/issues/53


I'm using it both with resolver, generating a rather long string by ```JSON.stringify```ing the whole arguments array (https://github.com/adamrenklint/bap/blob/apollo/lib/Params.js#L60), and sometimes simply with ```args[0]```.

It did look like it had an effect by reducing GC events, but now I can't reproduce the result, and my original problem (lots of GC in Firefox causing audio jitter) still remains. I'm gonna keep digging...

One idea that I had was this: passing an array of possible cache keys into ```memoize```, and setup the cache object with these keys defined, but empty, to begin. This way I could avoid hidden classes in V8 being created when increasing the size of the cache object as time goes. Also not sure about the effect of this, has to be benchmarked.


See https://github.com/timvideos/HDMI2USB-misoc-firmware/issues/53


This function as no test in ParticipantTestCase.php


SoFiA currently crashes on my machine at exactly that position with the following error:

    Traceback (most recent call last):
      File ".../sofia_pipeline.py", line 147, in <module>
        sampleRms=max(1,int((float(np.array(np_Cube.shape).prod())/maxNrVox)**(1./min(3,len(np_cube.shape)))))
    NameError: name 'np_cube' is not defined
    Pipeline failed with exit code 1.


- Vâng, em sửa nốt cái lỗi update trong db rồi sửa lại.
- Giờ em vẫn còn lỗi update những tile trong db.
- Nãy em mới update lại tile cache trong tiles với hugeTile thôi.


for multi-valued it's FALSE, as you can see in acceptance test we don't enter anything for it (no value).


But then you have no test when a mutliple valued attribute is present?


@Godin Well, @dbolkensteyn and I noticed that behavior two weeks ago. Without specifying a branch, the icon was red as soon as one of the other analyzed branch/PR was red. I can't figure out why it's currently not the case.


Git pull


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1635


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1636


Hi Fred - ah, thanks for digging this out.

In this case, I suggest we have one transaction wrapping one add statement and one rename statement:

```sql
BEGIN TRANSACTION;

    ALTER ...

    RENAME ...

END TRANSACTION;
```

Could you make those updates?


These relate directly to values in the StoreType enum


That looks good! I'll make the updates.


yes, because our test code doesn't understand comma-separated format in table yet. I will add it but it needs a bit extra time.


Hi @korbinianHoerfurter,

thanks for your contribution! Your code unfortunately caused some existing test cases to fail. You should make sure, that all test cases are passing, otherwise we are not able to merge your changes. You can run our test suite from your console like this: `grunt auto-test` (then the test runner is executed automatically every time you change your code).

You should also create a new test case to make sure, that properties actually get copied while replacing. The tests are located here: https://github.com/korbinianHoerfurter/bpmn-js/blob/master/test/spec/features/replace/BpmnReplaceSpec.js

If you have any questions feel free to ask (here or in our [Forum](https://forum.bpmn.io/)).


Fix.


It is normal order ;)


vis? no more vis :)


@otkmnb2783 @alexleigh, I make a new changes, i could be work now. you code is not working in fact
https://github.com/wuqunfei/jersey/commit/2effce3fea521dee026a9d7103ee4e7099d9dbf9


## This is a markdown file


gj


Where did you find that convention? All the files I looked at seem to try to reduce the includes in the header as much as possible.

Anyway if it's a convention I am fine with it :)


effect not confirmed, review again


review again


Ok


ObjectID in mongodb is not an IntegerField, you should treat this filed as a string field or a referenced field.


Line 24: "can not" escrito incorretamente (correto seria "cannot)


test


no it should not. It is necessary for the test (this is how the source will look like in generated files: comments after the Twig_Template subclass)


Thanks :+1:


one thing I did for performance seems lost here... rather than removing a light to move it one block over, I would align them to a grid, like spacing them every 3 or 4 blocks, and keepign the same grid even when the player moves less than 3 or 4 blocks. I think you removed that by moving the rounding outside of the player pos code. Line 13-16 does the rounding independent of player position, then line 110 is adding without rounding. The math is more expensive in my version... but I would think the removing light and adding it would be more expensive overall. Did you test that?


Thanks. Is it alright that `Like` model implemented like this?


I am interested to integrate this code into ipdb.

I do not understand what you mean by

"If it is an embed shell and no IPythonApp is running, the logic is usually in a
python script and you will have to change the code in order to make it works."

in your log message.


fast-forwarding develop to borshop-integration-1251-kuenishi-patch-3 = 661fb4e6


Merge sha 5dc5ac2a is stale.


merging basho/riak_cs/merge-from-2.0 = 626e5943 into borshop-integration-1252-merge-from-2.0


basho/riak_cs/merge-from-2.0 = 626e5943 merged ok, testing candidate = 79c0152d


Line 24: "generetes"escrito incorretamente (correto seria "generates")


ah yes you are right...


Hi there,

Is there a way to override this last change in some vim.before.after kind of thing? Or should i just remove them from my keymap file? I use Ctrl x to decrease numbers and Ctrl z to switch from vim to terminal.

Cheers


`iteritems` doesn't work in python 3.0 - 3.4, see [PEP 0469](http://legacy.python.org/dev/peps/pep-0469). It might work again in 3.5, but as it stands now it breaks in 3.4.


Wow, you fixed the build! I only started working on that last night!


if u provide an add method, u should also provide a del method.


Is there a reason you did not submit this as a PR ?


Hmph, no idea why it works on my machine without this, but I agree it's necessary.


Of course I will.


@lsnape This pull request has the side effect of re-introducing clojure metrics 2.5.1 which we know has a bug recovering when graphite is unstable. We'll need to downgrade the version of clojure metrics in radix and re-release a new version. Previously the version was downgraded by simply using an old version of radix for some reason.


@Maxwe11 - Okay.  I'll do that when it's time for the final rebase/squash.


Don't think we need to do this with SNAPSHOTs. Whole point is they're unstable.


there should be a sample html page with embedded detect cms library.


Nyttig!


Hello Andy,

Seem likes the command does not work on Ubuntu (i.e 13.04), can we fallback to 'modprode tipc' if above failed?

Regards,
Hoang


第一次 修改 README.md 文件


@gavinking : sure, I clearly find it better to use the specialized `Unit` and `PhasedUnit` classes. They are made for this type of need : in particular avoiding to *always* iterate the project `PhasedUnit`s or `Unit`s.

However, the `PhasedUnit` returned by a `CeylonUnit` might be null, for exemple if it's a binary archive (CAR file) for which no corresponding source archive has been found in repositories.

So I would  rather add a null-check just after this line. And of course in all the other similar cases below.


Flottings.
`org.apache` ser mer vennligsinnet ut en `com.hp.hpl` i mine øyne :)


Not sure if this name is valid / usable here... but sounds more logical than suse-13.2 or anything alike,


could we just re-use the `node-monitor-period` flag name instead of creating a new flag name that serves the same purpose?


Well thought out :+1:


I'm not a big fan of those kind of methods. I prefer to use [constructor injection](https://en.wikipedia.org/wiki/Dependency_injection#Constructor_injection) so that testing with mocks is simpler - see the [subsequent commit](2cc2110f5cd8ccb27d3bd85250066b15c20e1072). If I went down the memo-ized private renderer method, I'd have to stub that method in all the tests.

It feels to me as if the bit that's "heaviest on the eye" is the list of helper modules. If the `OutcomePresenter` was in the `SmartAnswer` module or the helpers were not, then I think the list of 3 helpers could be collapsed onto a single line which might help.

Anyway, thanks for the feedback, but I think I'm going to leave this as it is for now.


I think this is not working anymore. Apparently the model does not translate this timestamp into Unix timestamp. I may be wrong, but I think this is the reason why the timestamps are not being handled anymore.


:star:  i boka for å ha oppdatert diagrammene i samme commit


Yeah. Sorry. I could probably have curated the git history a bit better here!


This is not fixing the problem for me.
Can you verify this?


@killpanda I think it's good, and this cls could be used in more general case. such as following people .etc.  With the created_at field, all operations are tracked. Thus we could use this table to do something like timeline.


Why would the KEEP function itself need to be transparent?

Incidentally, I've got the mechanics working so that you can say **output: insert/:only :value** and not use the APPLY.  But the compatibility mode I have set up for r3-legacy that lets you switch back to #[true] for refinements captures the convention for a function at the time of definition (by encoding whether the refinement accepts LOGIC! or WORD! in its typeset).  But if a dynamically created keep function like this tried to take advantage of the feature, then switching into compatibility mode would break it in the dynamic definition and **insert/:only** would try to run **insert/#[true]**.  :-/


:ok:


Good


This should not be needed, as it is compiling fine standalone and as part of Sming on both linux and windows. Also, rBoot is an inclusion in Sming, not a fork, so any changes in rBoot should be suggested upstream and reintegrated into Sming.


Ironically, I'd like @yogthos opinion on this one. 1) I think it's probably better to emphasize ClojureScript and 2) That lower-case s at the start has been kept since the original commit but perhaps it's time to retire it.


As of version 6, the osgi compendium bundle is no longer a bundle that can be resolved at runtime. The reason being that it provides API classes that should be provided by the implementation of that API. Using the compendium bundle at runtime was already considered a bad practice for a long time, so we might need to look at what bundles have a need for the compendium bundle and how we can solve that.


fast-forwarding develop to borshop-integration-1252-merge-from-2.0 = 79c0152d


@halfcrazy Thanks, I will keep going based on it.


If the interpreter is directly run with the embedded console, it means that you have a python script to start it up, so my patch won't concern your use case and you will have to add things in the python script in order to load whatever you want in the interpreter.


:cool: :+1:


Yes :)


Is this a typo(Should it be DungeonInformation = 0x27) ??


i don't get what this function does, could u give a brief explain?


>H owever, the `PhasedUnit` returned by a `CeylonUnit` might be null, for exemple if it's a binary archive (CAR file) for which no corresponding source archive has been found in repositories.

I'm pretty sure that's impossible in these cases.


what he said :P


Just F* great! This broke my dev machine completely, especially since virtualenv doesn't work with this version. It's not just some minor patch to push it on top. There is no easy way to revert to 3.4 without some hacking. This should be introduced as another .rb in my opinion.


I assume appveyor is used for windows builds; but shouldn't '4.0.0' be added here instead?


pbr and setup.cfg?


I think the licensing of this hash function may be a moot point.

I noticed the Python generator was also generating a `hashCode` method, simply copied over from the Java generator (where it is useful, AIUI).

So I think this is a similar case of copying over something Java-specific which isn't useful in Javascript.  At least I can't find anything suggesting it's wanted in Javascript.  Unless it is a JSX thing?


https://github.com/envire/mars/blob/master/sim/src/core/Simulator.cpp

Just above a lot of includes ;) So I guess the convention is not very strongly enforced...


coucou


taggle


Grote datasets lijken me in deze specifieke context niet echt een issue, maar de sklearn aanpak is idd niet de stabielste... Ik bekijk het het nog want mogelijk vliegt die tsne er nog uit. De de ete-dendrogrammen zijn nu ook toegevoegd al heb ik daar wat viz-probleempjes.

> On 14 Sep 2015, at 10:55, Folgert Karsdorp <notifications@github.com> wrote:
>
> Weet je zeker dat je de implementatie van TSNE van sklearn wilt gebruiken? Ik heb daar gekke dingen mee gezien... Beter is een van de originele implementaties en voor grote datasets zou ik de barnes hut implementatie gebruiken. Zou je eenvoudig tussen kunnen kiezen door een cutoff te gebruiken bij bijvoorbeeld 1000 dimensies.
>
> —
> Reply to this email directly or view it on GitHub.
>


Might be. I just wanted to make clear that there are cases when it returns `null`. In Ceylon we would have probably added an `if (exists)` test without any problem :-)


Remove gettext and use it from utils


Add some tests for this one?


Nice :+1:


This was another Java-ism copied over into the Python generator.  Is it actually useful here?


Good. I noticed the profiler sometimes reported excessive time in this part of the code (more precisely the decrement of the ttl), I don't know why as apparently there's nothing strange there. I think having it wrapped can be handy.


LGTM.


@jameslnewell , this was a good catch, updating the regex to support windows paths.

However, the rootPath still only outputs unix-friendly paths `../../../../../` (see [index.js line #27](https://github.com/jameslnewell/metalsmith-rootpath/blob/master/index.js#L27)) so there needs to be a check to detect the OS (or path type: *nix vs windows), and output the correct rootPath accordingly:

*nix: `../../../`
windows: ..\..\..\`


Yes. Thanks. I've added the following to the commit note:

> Note that `StartNodePresenter` relies on the following `NodePresenter` methods:
>
> * `#has_title?`
> * `#title`
> * `#has_body?`
> * `#body`

In hindsight it would've been better if I'd introduced unit tests for `StartNodePresenter` at this point rather than in [this later commit](https://github.com/alphagov/smart-answers/commit/3be14f72251bdc681a074964d4f941acbfe82648#diff-85d6d218c976728de4211b3043ec0ff2).


Googleで調べたところ、この機能が役に立ったという報告が特になかったため廃止しました。
（もし必要でしたら復活させます）


yes, saw that in the comments of a blog post about the new cmpn version [0], but wanted to have a quick solution first on saturday morning, because Gabriele run across this problem. Wanted to have a look at this again (that's why I wrote "for now..." in the commit, but I admit that it could have been a bit more verbose ;) )

[0] http://blog.osgi.org/2015/08/release-6-of-osgi-compendium-osgi.html


You have no idea how much time I spent trying to get this to work last night...


Thanks: I'll fix with a slight alteration (not sure why I had `\tl_replace_all:NnV` in the first place!)


Has a PR to the master fork been created for this?


As I wrote in the comment here https://github.com/codemix/yii2-localeurls/issues/44
what about
```
return $url . ($this->suffix == '/' ? '/' : '');
```
I'm not an expert on URLManager, but this still looks like a dirty patch. I would hope, there is some moment, where you properly add the “suffix” property no matter what it is – slash, string, anything.


Fix here: https://github.com/netguru/reminders/commit/b5f0e92f91ba899940c2653efcfde9c64bffe2de


As Arne said, it is always better to keep the number of header includes as low as possible in the header files. It is even better for readability when includes not used in header file, they shouldn't be included there, as also "others" using the header do not need to have these "additional" includes.


Is this the display fix where we all were waiting for?


Fix here: https://github.com/netguru/reminders/commit/b3dc469e1b36a1bf6fa529223680a7eacada96ed


does this fix your CCE ? strange.

Anyway, an `EditedSourceFile` should normally always be seen as `equal` to the corresponding `ProjectSourceFile`, unless the corresponding `ProjectSourceFile` is `null`.

Also the `VirtualFile` of an `EditedSourceFile` should always be the same as the `IFileVirtualFile` of the corresponding `ProjectSourceFile`, unless the corresponding `ProjectSourceFile` is `null`.

So it doesn't seem to me that this change solves anything, as far as I understand, since it doesn't manage the case when `getOriginalSourceFile()` returns `null`. The exception will probably be an NPE instead of a CCE.


This shouldn't be here, I'd say ;)

Had to change this back for the Webinterface to work again


Oh, and another Java-specific thing copied for Python.  Is an `equals()` method wanted in Javascript?


If you add an `open Fun` at the top of the file, the `|>` function will be available and these lines should compile as-is.


提交到doc文件夹下


The main purpose of rewriting `__get__` is to get the real instance of bounded `post1` object.

There's the  example:
```python
post1 = Post.objects.filter(id='xx').first()
likes = post.likes
likes.product_id = 'xx'
```

In this case, when calling `__get__(self, instance, owner)`, `self._instance` is the `post1` object. Then `like.product_id` could gets `post1.id` correctly from `post1.prodcut_id()`.


Hehe, yes, copied it from a test installation, sorry.


I think this part of the commit should be squashed into 684ce96295f2383554d9a2b09f396ba402646f51


Moved by you here:
https://github.com/netguru/reminders/commit/6457b2f8cfe10800373938ea950bfcc3ed2790a0

Can you explain why it shouldn't be here?


Renamed file.


Ples baus, gibe me ~~the pussy~~ a function baus.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 19](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=172&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


It looks ugly on the page, but you can at least identify the tile


Crossroads Caravan Delivery: You get this quest from Grol'dom Farm in Northren Barrens and you have to talk to Rocco Whipshank to mount you on the caravan up to Crossroads , but you can't cause in the Rocco's dialog panel there is no command that mounts you on the caravan .

Fade to Black : this quest can be taken from Kalec in Azshara in the Northren Rocketway and the objective is to kill Katrana and Malicion . The thing is that after you kill Katrana , Malicion will show up to kill you but he gets killed by Kalec . It doesn't happen that and you can't kill Malicion untill the gm will fix the problem.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 19](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=172&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **FAILURE**
Summary: Compilation error: PluginBCF\xBIM BCF Plugin.csproj (new) Build time: 00:00:42


Oh, good idea. I'll revert the change and push it.


If nothing changed in API/ABI, then it should be user decision whether update git-client with some fix or not. For example i didn't hit time issues, but i have terrible infinite builds for polling and wants revert git-client. When you bump it in core for your local development work it says to jenkins that git-plugin works only with minimal version that you bump and it limits people in version variations.

Of course for testing and having newer version it's ok, but :atm: i can't figure out working variants for git-plugin + git-client-plugin versions.


Display was fixed friday
This commit adds Touch Screen support


Merge from Master


Replace with `//`. `#` doesn't introduce comments in Groovy.


test


Is it common practice to use `#` as a comment character for these log files?  The code ingesting the log file seems to simply parse each line as JSON and ignore it if it does not parse correctly.  (Looks good to me to add this comment, just asking whether there is some convention on how to do it.)


So what I want to do is better understand the really-specific situations when and `EditedPhasedUnit` is created with a `null` `ProjectPhasedUnit` though it indeed *has* a related project source file (case I spoke above). And make so that this case will not happen anymore. because it's *not* a normal and consistent situation.


Done


why is this path (`./scheduler/scheduler`) different than the one below?


sorry, but we shouldn't do release until i solve polling issues with 2.4


I've pulled this in, but I'm concerned it may slow up build times (which are already too long for such a small project).


LiteLoader doesn't use it, it's a system it makes available for mods. It makes it very easy for litemod authors to allow parts of their mod to be disabled by the server, for example disabling entity radar functionality in VoxelMap or disabling more advanced script functions in Macros.


This does not work. It breaks the mail functionality completely.


Rework this commit, the `this` parameter should be the first parameter (not the last)


I'd probably put the comment above this line that its handling back-compat for pre-12.3 servers


again, probably throw a comment above this line that its for pre-12.3 servers


works fine!


101280704_lowTempRad (Nigusse) - x86_64-MacOS-10.9-clang: OK (1850 of 1865 tests passed)

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


#7 related


Mate. thanks for taking the time!
Yeah we're not going to be in that vicinity. We're patching a feature hole, whilst migrating some solutions.

Thanks for stopping in :+1:


QTabBar::setAutoHide() requires Qt 5.4 or newer.
Should we support older versions? If not, the file ainclude.m4 should be updated again.


Even without this (needed) code, the test *keeps the desktop shortcut deleted on updates if it was previously deleted after install* in *squirrel-update-spec.coffee* still passes.


adição de novos arquivos


`<header>` tag must be used to contain content related to header info of the page, like title, navigation bar, signup options etc., not to be used in the content of the page. Remove this tag,


go to OUR new website? our?


I think we want the print() function invocation not under the "if debug".


Ha, clever trick here!


The way it is isn't too bad, is it? For most parts, it's all automatic. If we could make `CreateChromaFilter` somewhat similar to `MakeFilter` then it would be just as automatic wouldn't it?


I think we need a module global definition for `_DEBUG`.


We should be consistent about using a value in an `&&` statement or `!= null` statement when checking existence.  I think I prefer the latter since we have had trouble in the past with using the raw value.


Failing test subjects are burning metaphysically; increase prototype production of fever reliever medication brand: (yellow) WATSON [#825]  / 325-1000 mg; MAXIMUM (13 a Day) Every 4 to 6 hours as needed for chronic pain management due to fevering spiritual disalignment within the body's earth realm. [OBE]: disruptions within chest, gravitational weight of soul is exactly 21 grams (downward g-force earth tons)


PACIFIC STANDARD + GREENWHICH MEAN TIME + MOUNTAIN TIME + ANOS DOMONIC


Thanks again @bugada. Everyone can probably see I am new to this pull request thing...


And in my opinion we should share the settings between the GTK/Qt interfaces and the statusicon plugins. Other thoughts?


LGTM.


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1868 of 1870 tests passed)

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


I am sorry that you had to and clean up after my mess


Well, maybe i should add this to readme but due to simplicity of this task the main goal is to see HOW you do it, no IF you do it. So CS and git basics are also important.


I happened to look through this and I think the colors are opposite of what they should be, i.e. black == 0.


My intention here was to see if I could get the paragraph to say '1'.  My overall intention is to have the value of the variable displayed that will control the time lapse to change the block skin from one skin to another.


Does the `--delete` do anything when we are rsync'ing specific files?


Amen


Need to move a couple things to the config (and decide on sensible defaults).

The LazyRequest will need updated to avoid this stuff too, but it can get a simpler version since it doesn't care about encoding.


mouais, looks ok, but please resolve conflic 1st!


This class shouldn't be here


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


80 character limit per line


https://lkml.org/lkml/2013/11/23/148 taken from this


this can be further simplified to

```javascript
this.get('scrollable').animate({
  scrollTop: this.get('target')
}, this.get('duration'), this.get('easing'), Em.run.bind(this, this.sendAction, 'afterScroll'));
```


80 character limit per line on all the changes in this file


Thanks! Changed the attribution in 28461e7b7219c889db52bd34ad910e6564a6019b


Also addition to the prototype function that will work as the vector disease models


No F1 because Note Blocks only support tones from F#1 to F#3 http://i.imgur.com/NhvXZru.png


Please limit max 80 characters per line for all the changes in this commit


94006714-MultipleTowers (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1875 of 1875 tests passed)

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


101917310_waterCoilHumRatFalseWarning (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1335 of 1335 tests passed)

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


In the commit comment, you have a reference to "symling", which I think you want to be "symlink".

LGTM otherwise.


101280704_lowTempRad (Nigusse) - i386-Windows-7-VisualStudio-12: OK (0 of 0 tests passed)

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


Missing scenario names.


Why ?


Unhandled Fundamental Failure


Both the GUI and controller scenarios have the correct focus on the internal system state and user interface behaviour, respectively.


Getting RenderChains used in other Renderchains to work correctly is a bit of a pain though. And getting the previous method took far too many fiddly bits and tweaks to make it work IMO.

In theory a system which simply keeps track of how the RenderChains are connected, should be easier to maintain and work with. But I'm not yet entirely sure how to make such a system yet, I'll let you know when I've found something interesting.


Is Skip needed ?


> NOTE: on wx.EVT_CLOSE... do NOT put id=ID_WHATEVER with this event, as

where is that ?

Is this commit tested ?


I am not sure we need this attribute.  What would happen if we don't use it?


remember in C++, reference means "the object" or alias.

following code should print out 1, and causes no memory leak, because p1 is the same as p0.

```C++
	int *p0 = new int(0);
	int*& p1=p0;
	*p1 = 1;
	std::cout<<*p0<<std::endl;
	delete p1;
```


the zs elements at the end has ......... for the rec id;  some of them aren't 8 chars ...


heu ya un soucis là, "une voix", mais apres "elles me rassurèrent" et puis aussi "détend"  :dancers:


Javadoc needs to be updated.... :-(


Good catch! Looking at it now, I don't think that cleanup code is needed anymore as the unit tests shouldn't really be working off a persistent database anyway now that we have a set of integration tests for that purpose. I'll just remove that code section - which will solve the problem with it :) - and will try to merge these changes a bit later today.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 623](https://teamcity.labkey.org/viewLog.html?buildId=272076&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 32 (1 new), passed: 7 Build time: 00:55:55
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.tests.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

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

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

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

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

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

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

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


This ending "/" was adventurous. Try removing it :smile:


TAI-UTC is 30-odd seconds, , which is ~12 arcsec for 10deg/day. So that's a big deal and could explain the entire discrepancy.

Can somebody clarify if the OpSim visits are tabulated in TAI or UTC? Maybe it doesn't actually matter, and we can just declare that they are UTC. In any case, the detections need to be generated in UTC, at least for most ephemeris applications. And at the end of the line OD requires UTC positions.


I don't think this is Crosswalk change so not needed here. You are welcome to push patch upstream to suppress this warning.


How does this work on pre-4.4 devices, as I understand it, android.app.ResourcesManager is new for android-4.4? There's no try/catch surrounding this that would prevent crashing.


How does this work pre-4.4? As I understand it, android.app.ResourcesManager is new for android-4.4. There are no obvious try/catch that prevent this from crashing when running on a pre-4.4 device.


なるほど
了解です！


@Utumno review this for py2/3 compatability, which py33+ is relavent to wxPhoenix.
http://docs.pythonsprints.com/python3_porting/py-porting.html#handling-exceptions


This breaks the install from YUM repo on CentOS.


try the "auto const&" for pointer:

```C++
	int *p0 = new int(0);
        auto const& p2=p0;
	*p2 = 1; // okay
        //p2=nullptr; //compiling time error, p2 is const
	std::cout<<*p2<<std::endl;
        delete p2; // ok
```


Why is the resource declaration required here?


LGTM, thanks.


Correct approach to abstract definitions of visible elements in GUI scenarios.


s/*das*board/*dash*board/g

this is causing an error when attempting to delete an applab project from within that project's header (as opposed to from the My Projects page)


Controller scenarios adequately describe the server system states.


@vahidreza I have solved the pesky issue we have been having with v0.1.0. All you have to do is to add the Android schema declaration to the config.xml widget tag, then you'll need to declare a gap:config merge section, and add the tag for <application android:name="com.medlei.App"/>


Relevant, measurable and attainable feature.


Take time to learn how to use map and fold.  It will pay off benefits for a long time in your career. :)


Parametrized, testable steps.


(Y)


Abstract references to interface elements.


There are 4 errors when exporting .


dw


Yes, What I am trying to say is that
do NOT write this
```
self.Bind(wx.EVT_CLOSE, self.OnCloseWindow, id=ID_MYCLOSEID)
```
the wx.EVT_CLOSE uses the built-in wx.ID_CLOSE, so don't bind a user defined ID with these events or even write id=wx.ID_CLOSE. Other events are ok doing this. Personal debugging experience that caused a lot of headache until I figured out the issue.


Hello @sintetizzatore
cc:  @jonhanke  @julialintern

----
Good job in completing these advanced python exercises.

Thanks for formatting explanations and results in an easily readable manner.  It is greatly appreciated and makes my life easier.

----
Some optional extras if you have the time:

a.  [Explore 'defaultdict' option in python](http://stackoverflow.com/questions/5900578/how-does-collections-defaultdict-work)

defaultdict means that if an key is not found in the dictionary that instead of a KeyError being thrown a new value is created. The type of this new pair is given by the argument of defaultdict.

b.  Explore pprint in python
Prints output in a more readable and structured format.


@Gilber7kim The source is updated more frequently than the binaries, because the binaries (what you use when you install something) are created from the source at certain intervals.


Alguns erros de sintaxe, mas o resto tá bom.


Maybe the node.id property can be made immutable to prevent bad things?

@oldschooljarvis @steveblue @gadicc @talves @andrewreedy


94006714-MultipleTowers (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1875 of 1875 tests passed)

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


Famous typo! Should be `inOutExpo`


Just a comment about the use of this event. The `onload` event is a standard event in the DOM, while the `ready`
event is specific to jQuery. The purpose of the ready event is that it should occur as early as possible after the document has loaded, so that code that adds functionality to the elements in the page doesn't have to wait for all content to load.


Should be in the same chunk, but it is probably better to check the potential block


Note: this change leads to backwards compatibility break!
Anyone who will update SDK to 1.1.4 (like me) will have hidden surprise in production: all scheduled pushes will begin to arrive to users not on their local time, but on UTC instead. Which is a big difference (at least for us).
Probably flag 'utc_time' will serve purpose better, with meaning “if set to true, format push_time as UTC time”.
Also note there is no word in docs on this: https://parse.com/docs/php/guide#push-notifications-scheduling-pushes


Hey i'm sorry i just realized i kinda messed up.

So the actual repo for cryptlib is

https://github.com/invalidred/cryptlib

So i should have asked you to submit your pull request here.

Since you did help fix an issue, I would be more than happy to add you as a
contributor of cryptlib.

If you could submit a pull request with update package.json where you
include yourself in the contributors section, I will merge the pull request.

On Mon, Sep 14, 2015 at 9:57 AM, 王庭茂 <notifications@github.com> wrote:

> #25 <https://github.com/Pakhee/Cross-platform-AES-encryption/pull/25>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Pakhee/Cross-platform-AES-encryption/commit/f94bdc310c78406ac64291d3c47691f576801a38#commitcomment-13213592>
> .
>


Can you let me know, what are those issues? As per me its going good.

On Tue, 15 Sep 2015 10:38 am José Herrera <notifications@github.com> wrote:

> There are 4 errors when exporting .
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/niccs/PanicButton/commit/7927a0de9bb161bfb34f043f703a4b7ef0ee5dc8#commitcomment-13226553>
> .
>


To ensure that this code will work correctly you need to include it inside of the callback that it's executed after de document it's loaded


Hello @jperelshteyn
cc:  @jonhanke  @julialintern

----
Really nice job in completing these advanced python exercises!
Good for finishing the optional Markov!

Thanks for formatting explanations and results in an easily readable manner.  It is greatly appreciated and makes my life easier.

----
Some optional extras if you have the time:

a.  [Explore 'defaultdict' option in python](http://stackoverflow.com/questions/5900578/how-does-collections-defaultdict-work)

defaultdict means that if an key is not found in the dictionary that instead of a KeyError being thrown a new value is created. The type of this new pair is given by the argument of defaultdict.

b.  Explore pprint in python
Prints output in a more readable and structured format.


Kaity asked if we could add "Redirected from: <title><line break>" to beginning of description for redirects. This does that. Will need to clean this up tomorrow...


@thenickcox @ragesoss I probably added this in anticipation of future use. I don't believe I ever built a component that used it, in any case.


You have Skype?


"In the following example assume the SN interface to the MN"

should be "The following example assues the SN interface to the MN "


Adding the example to the documentatio


Good point, will look a lot more cleaner that way


I think it could cause problems (e.g. accidental click) - do you still think we should add it?


@arter97 I have found a problem which I mention here: https://github.com/Tkkg1994/Hulk-Kernel-V2/commit/47e3d1dd7037adb5599b9837d0e1323dbcd0f9ae
It is only a small compiling error, easy to fix (it only appears with cpuboost driver) hope this helps


Is there missing a step to connect SN27 and group sn1group?


Skip is needed so that higher level event handler are gotten to, otherwize the event is eaten.
The placing(wherever in the def) of the event.Skip() is not important, just that it is there.


           Hi Steve,

I think in the narrow context of opSim it doesn’t matter and we could, at
least in principle, declare its time to be UTC. But I’m guessing that
variable star astronomers and others who care about taking time
differences on long time scales would argue for TAI. Given that LSST
should really have only one choice, now with opsim and eventually
with real data, I am more in favor of TAI and letting solar system
people do their own (tabular!) conversion to UTC.

I would appreciate to hear your counterarguments if you would argue
otherwise, as your experience vastly outweighs mine!

        Cheers, Zeljko

> On Sep 14, 2015, at 5:30 PM, Steve Chesley <notifications@github.com> wrote:
>
> TAI-UTC is 30-odd seconds, , which is ~12 arcsec for 10deg/day. So that's a big deal and could explain the entire discrepancy.
>
> Can somebody clarify if the OpSim visits are tabulated in TAI or UTC? Maybe it doesn't actually matter, and we can just declare that they are UTC. In any case, the detections need to be generated in UTC, at least for most ephemeris applications. And at the end of the line OD requires UTC positions.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/testMOPS/mops/commit/f995cb0253c75dcac003e68a62c813c8c5fb22b7#commitcomment-13226461>.
>


This part is still correct (line 202 to 204).

The attach portion is the only portion that you should be changing


The new version of `p` exports `head :: [a] -> Maybe a` which conflicts with this, so I took this as an opportunity to get rid of `head` altogether.


Restore to previous version


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (800 of 800 tests passed)

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


Oh yeah, I totally forgot that Adaptive uses the `body` key! If you’re adapting this to use with Adaptive 1.x or 2.0, that would be a good idea.

In Adaptive 2.1, however, we don’t need to worry about that within component templates. One of the design goals for Stencil/Adaptive Components was to avoid this problem entirely. In 2.1, all component templates are completely isolated from the outside view context. You have to be explicit about what is passed in, using either “params” or “bodies”. Here’s an example using both:

```html
{@c-tabs tabs=tabData}
    {#tabPaneData}
        {@c-tabs__pane}{.}{/c-tabs__pane}
    {/tabPaneData}
{/c-tabs}
```


I see. Ok lets keep it as is for the moment and see where it goes.


It's adding, not updating.


I don't think this will do anything because the query isn't actually issued until the .get() call in the loop.

https://cloud.google.com/appengine/docs/python/datastore/queryclass#Query_get


Thanks.


I love line 24-26


Its the greatest


I don't like allocating on the heap and not deleting it in the same scope: something like ```int (or void) THBitmapFont::drawTextWrapped(THRenderTarget* pCanvas, const char* sMessage ... const THFontDrawArea* drawArea)```

Then when you call that function you can allocate the structure on the stack with something like
```
THFontDrawArea drawArea;
drawTextWrapped(pCanvas, sMessage, ... &drawArea);
```
and no manual deletion is necessary. As is your implementation leaks memory.


I have a question here:
In above description, we use "primary" and "backup" service node. But here we use "source" and "destination" service node. Do we need to conform the same descritipion?


Is Asterisk a better name?


is 66 correct? The original script had 67 (and maybe that was a bug)


weird getting a notification for this so many years on...


If the SN is needed to be reinstalled,


Good point. I'd probably try to do these calculations in the db, but at least we should be able to optimize it by loading batches of users with the [run()](https://cloud.google.com/appengine/docs/python/datastore/queryclass#Query_run) command.


or, if the SN's configuration is not lost,


Do not include a constructor for a struct. Initialization behavior can be controlled at initialization, THFontDrawArea area; defines an uninitialized struct (values unknown), and THFontDrawArea area = {}; defines a struct with all values 0'd out. You can also do THFontDrawArea area = { 1, 1, 1, 1, 1, 1 } if you wanted to set all the values to 1 on initialization.


false, not FALSE


These changes should have been done in the variables.scss file, not by changing them here.


See above


Not sure why this happened :/


Should there be a link for "Set Up the Service Nodes for diskfull Installation"


A


See below, changes should be in variables.scss


We could get rid of this if colors are changed in variables.scss, it will format correctly automatically


Criteria is a plural form for criterion.


In my experience, SQL queries are usually read as ["count star"](http://www.sqlskills.com/blogs/conor/countstar-vs-countcolumn-vs-countdistinct-vs-countcol-over/) so I think this name makes sense.


Link for "Build the Service Node Stateless Image"
"


> self.Bind(wx.EVT_CLOSE, self.OnCloseWindow, id=ID_MYCLOSEID)

Have you seen it in the code ? _Is it related to this commit_ ? Random wx tips _do not belong to commit messages._

Re: skip: I know what skip does thanks - and I very well know sometimes it is expressively omitted. Have you tested _this particular one you added ?_ Document what is the difference if it is not present - otherwise do not add it.

In the wx stuff I trust you know what you are doing - and that you tested that nothing broke - right ?


Link for "Set Up the Service Nodes for diskfull Installation"


Note that this is preventing the "retry thumb" functionality from working. The current case where this is happening for me is here: https://www.reddit.com/r/leagueoflegends/comments/3kyoqd/apollo_on_tips_break_everyone_has_moved_out_of/


Nooo, goodbye 1.7, you'll always be hated by us (server owners) but loved by players


Nên đưa ra thành một câu lệnh xử lý link trước


Nice commit, this was killing me a little earlier today, and I was looking to make this change myself.  Cheers :)


Links for "Add Service Nodes postscripts"


:sparkles::heart::sparkles:


Em todos os cenários, o começo "Given" é redundante, pois já está estabelecido no nome do cenário que o tempo limite passou ou não.


Não segue um padrão textual na escrita do ":" após "Feature" e "Scenario"


Uma possível alteração nos cenários de GUI poderia ser mudar o começo para :
Given that I am on the auto evaluation Page
And there is a criterion "Requirements"


Hi Halil,

The question I have is that if the factory method is called with the record to find and in fact it does not find something it has no choice but to return null. At which point the exception would be raised.

It might be worth considering changing the prototype to something like:

bool Factory(TKey aKeyValue, out TItem aItem)

so factory can return TRUE if it did find the item to add, FALSE otherwise.


Dòng quá dài


either I am missing something here or `[${PYTHON_USEDEP}]`is missing on almost every dep?


There is a configure options, --enable-gcov, which enables the necessary gcov flags.

Perhaps the coverage target should depend on the check target.


Those should be in the commit message.

2to3 conversion is something  do not want to get into - especially in a _random_ manner - I have said and explained this already a million times so here its is for the last time:  _no more 2 to 3 commits_ exception: well documented wx commits - that do not break existing code while making it phoenix compatible. Dedicate a branch to them _work on it for some time_ and then show it to me for review.


Thừa ?> cuối file, thiếu dòng trống cuối file


@jagadeshbabu what does `to_representation` do?


Thiếu dòng trống cuối file


revert this change


@LarsBuur Looks like the issue is `options` being undefined (not `redirectUrl`). Will submit a patch, but

`Meteor.loginWithFacebook({})` would fix the issue also.


My email is neetikasood@gmail. Ping me ur issues, I will look into it

On Tue, 15 Sep 2015 10:44 am José Herrera <notifications@github.com> wrote:

> You have Skype?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/niccs/PanicButton/commit/7927a0de9bb161bfb34f043f703a4b7ef0ee5dc8#commitcomment-13226625>
> .
>


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1333 of 1333 tests passed)

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


咦？Senior 不恢复嘛？


`escape` is deprecated. I'm also a bit confused by the `decodeURIComponent` and `escape` composition, may I ask what your goal was here?


group["picture"].present?を23行目に移動してください。15行目は特に必要ないとおもうので、23行目にはgroup["picture"]を直接いれてください。


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1875 of 1875 tests passed)

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


Seems reasonable.  Do you think maybe we should just always include the cx info, like put it in the compulsory subreads table?


+ To allow you to open the rcons from the Management Node running the
+ conserver daemon on the Service Nodes, do the following:

To allow you to open the rcons from the Management Node using the conserver daemon running on the Service Nodes, do the following


Merging Nick's changes.


Yeah，that should be int64_t, updated


@pcingola we don't really have a test framework for this kind of problem, which is why it slipped by.  I could imagine some kind of mock-based test in `test_cli.py` that called each Abstract*Runner's run method and checked if the attribute called on the client was an actually existing attribute.  Or perhaps you can come up with something better than that.  In any case, more useful tests are always welcome!


I added back in the dummy data so Nick can see how the graphs behave.


leaguesharp


The installdir  directory is mounted or copied to the service nodes during the hierarchical  install of compute nodes from the service nodes.

Please remove  of compute nodes from the service nodes. ""


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


And I just sent photographic errors, check your email.


That is, the problem here is the connection between the cli and the client.  We don't really test that much.  The test files you mentioned already have tests for the process of the client getting an individual variant (the analogue of tests you wrote for getting a dataset).


Looks much better at newer version +1:


remove tailing whitespaces


Have sent a test mail to joselikoxulo@hotmail.com.
I haven't rec'd any emails yet
On 15 Sep 2015 11:23 am, "José Herrera" <notifications@github.com> wrote:

> And I just sent photographic errors, check your email.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/niccs/PanicButton/commit/7927a0de9bb161bfb34f043f703a4b7ef0ee5dc8#commitcomment-13227028>
> .
>


Yep! Hulk verrrrry angry!!


I don't think we use .then() correct here, actually then() will take two parameters, one is onSuccess callback an other is onFailed callback. We don't need the second .then(). please double check.


You are correct, that was what I was referring to as "snap to grid" that wasn't complete.  I've fixed a bunch of things, including that.  The math is still faster, and the nodes are staying put until they really need to go.  I also brought the helmet textures over from your repo.

My master branch has everything except megatorch, which has it's own new branch.  The differences are minimal between the two.  I have a feeling that many won't want that so for the moment I'm not including it on master.


94006714-MultipleTowers (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1875 of 1875 tests passed)

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


cool!


remove the tailing whitespace.


ahhh....

Then it could be done for the other ones on that form - status_id & campaign_id ... (inclination dependent)


Intro animation. http://img.memecdn.com/disco-adolf_o_692330.gif


REVERT IT FOR NOW IF YOU BUILDING


Why dropped msm7x27a support?
You not will drop completely support, why make this?


line 111 just switches the syntax to the new style. No, I haven't run across any odd looking event bindings in bash yet, but I see it done rather often in other folks code and it causes closing issues, which often are related to saving of settings.
I have noticed older bash relz to have a issue like this upon closing and settings(usually with BAIN installs infos, not getting recorded and the mods tab not showing what was actually installed with BAIN).

Re: I understand Skip too, and unless there is a specific reason to omit it, it likely should be there. In instances where skip is omitted or worked around in a loop or whatever, it is these instances that should be documented right in the def docstring as to why.

I work with wxPy almost exclusively nowadays. SourceCoder even has a custom pyshell and was designed exclusively for developing python(wxPy/pyQt/PySide/etc) GUIs dynamically on the fly. ...so yeah, I recall the wx2.8 years very buggy with many unneeded workarounds. My new codebases don't even touch wx2.8 verbally, but often will run with it.
Updating the wxPyDemo to phoenix taught me a lot about the syntax and changes and what works and what doesn't, but more importantly, what is CROSS Compatible. There are many ways to write these things, but was decided on as with the zen, that there should be one correct way to do it obviously.
The syntax I changed with the event binding was "ancient" and stuff like that always is floating around the wxPyWiki too.

I normally edit the code actually running my bash first, then drag and drop it into the github GUI and commit, so yeah, it is tested so to speak. My bash is running from the steam dir, not the github local repo, as there are many branches to work with.


What do you mean?

I have answered the test message .


This page looks odd. We'd better add some lines to explain that this page is for presentation unit test, to give other developer enough details about the background of this page, even if it never displayed.


Hiyo!

So I just Purchased your infistar anti-hack/admin tool and I keep getting kicked from my own server.

Do you have a proper tutorial for A3Wasteland? I seem to be missing something or doing something wrong.
https://github.com/Fractured-Gaming/Frac_v1.2.Altis/commit/9db343b04b3a48b0eef8bdfa4c5e84328fe2e12b

Did the above and followed your instructions.

14-09-2015 19:01:17 infiSTAR.de | 311h 35min | [FRAC]Mokey(76561198008596823) | Memoryhack RscDisplayConfigure onLoad changed: ["onLoad",_this,"RscDisplayConfigure",'GUI'] call 	(uinamespace getvariable 'BIS_fnc_initDisplay') (v0218)
14-09-2015 19:01:17 infiSTAR.de | 311h 35min | [FRAC]Mokey(76561198008596823) ADMIN - would have been banned now! (v0218)

I'm sorry if this is the wrong place to find you.... But help would be greatly appreciated.


@cyanogen


I'll fix this tomorrow and update. Thanks!
On Mon, Sep 14, 2015 at 7:22 PM Dean <notifications@github.com> wrote:

> 5.x firm w/ 4.x-MSET
> 7.x-8.x and 9.0-9.2 firm w/ 6.x-MSET are also supported.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/b-/rxTools/commit/927243da89f6fa1a4ccd7322ec372876fe1da71c#commitcomment-13225612>
> .
>


Sure why not.

On 09/14/2015 09:27 PM, Eileen McNaughton wrote:
>
> ahhh....
>
> Then it could be done for the other ones on that form - status_id &
> campaign_id ... (inclination dependent)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/eileenmcnaughton/civicrm-core/commit/7fbb65da3ff6dec979f41ecb099444e0afdee128#commitcomment-13227108>.
>


:+1: nice work


Rejected - no coding style commits.


`bundle exec cap deploy` indeed works. I'm pretty sure I was running with bundle exec before, but... Anyway, good to know it is possible to manually set asset_manifest_prefix, just in case.


good:)


Awesome, thanks @lkuper ! :)


what is the benefit of this option and is this working at all ???


http://www.w3.org/TR/html-markup/h1.html#h1-content-model

Only phrasing content is permitted inside of the header tag. <span> is okay, <p> is not. See this:
http://www.w3.org/TR/html-markup/terminology.html#phrasing-content


Note: It isn't 2to3; it is 2and3. It works with both, and all the other exception instances in code that where not written by me do this also, so why not change them back to py27 only code?


IMO, best to move this off to a long term development branch and work on it there. There are much higher priority things in need of work for 307 unless this stuff is causing critical issues with Bash on the supported modules it currently calls for.


This is generally a bad idea. Your tracking a value that should always be equivalent to $scope.completed.length. And by adding this new variable, you are assuming that everyone who modifies the "completed" list will also update your variable.

Also, watch your indentation.


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: Build Failed

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


There are a few issues but I'm going to go ahead and merge, because we are running out of time.


it creats not created


Is this an old bug in the code? If so, this bug seems to be major one.


never mind, I merged and made a new bug. Whener ever the completed button is clicked, the number increments because of that $scope.completed_count variable, which is just all around a bad idea. Click it 500 times and look at what happens. I'm going to fix it in the master branch now.


Well, bash almost runs fine for me on wx3.0.1.1+. There are some issues with progress dialog and thinsplitters and such. Taking the time to fix these in smaller gulps is better than jumping off the bridge so to speak. Besides tackling these issues, bash would run fine on wx3 classic now.


@AmberPoo1 we've apparently done something strange with this branch. We keep making the same commit, essentially, over and over. I think it is because of the way I merged changes when I did the hotfix over the weekend. I'll have a look at it, but for now commit *only* to the vt_5_x_dev branch.


What happens when i I click the completed button twice? this goes up by two.


This chunk seems to be useful. Remove?


I noticed wrapping a `case` with curly braces allows folding with text editors -- with Notepad++ at least. But I rarely use folding, and anyway text editors should be intelligent enough so we don't need to adapt our coding.


> There are much higher priority things in need of work for 307

Exactly my thoughts

*If you really want to help you must work on __already open issues that are a priority__. Otherwise you just hinder me.* Coding style and corrections to wx belong to a dedicated branch where you will work systematically _for a while_, while keep rebasing it on dev.


Following here: https://css-tricks.com/complete-guide-table-element/


If the method `blob_uri` is a protected one, we should add `protected` before the definition of `blob_uri`.


Never thought of using `overflow: auto` to deal with this margin issue. I usually just resort to padding on a child element – like I used on the `.hero h1`. Figured I'd give this technique a shot. For reference check out [collapsing margins](http://www.w3.org/TR/CSS2/box.html#collapsing-margins).


Wasn't able to get back on the cluster 9-14-2015 to get the fastest blocked version of dgemm_mine run.
Also haven't gotten the chance to run the blocked fortran code yet on the cluster to see how much of a better performance it gives.


`req.path` is always `undefined`


> Well, bash almost runs fine for me on wx3.0.1.1+. There are some issues with progress dialog and thinsplitters and such. Taking the time to fix these in smaller gulps is better than jumping off the bridge so to speak. Besides tackling these issues, bash would run fine on wx3 classic now.

Then set up a dedicated branch off dev where you work on those glitches.

If you are looking for a task - see issue 200. Especially splitting a details panel out of the Installer one is suited to you and will keep you busy for a couple weeks. I want this done


> Well, bash almost runs fine for me on wx3.0.1.1+. There are some issues with progress dialog and thinsplitters and such. Taking the time to fix these in smaller gulps is better than jumping off the bridge so to speak. Besides tackling these issues, bash would run fine on wx3 classic now.

Then set up a dedicated branch off dev where you work on those glitches.

If you are looking for a task - see issue 200. Especially splitting a details panel out of the Installer one is suited to you and will keep you busy for a couple weeks. I want this done

And no more random issues and pull requests


Why did you make this change?


To clarify to anyone coming on this later, I didn't use PHPUnit's `@require` annotation because I could not find a way to use operators with it. If anyone knows a way, please let me know! Either way, the result is the same.


I might be able to get the locker code with a little reverse engineering but I don't think that would be very ethical.


consider changing to:
`value = $(window).scrollTop();`


`.introduction` is recommended as the class name for this container. `.hero` is another commonly user class name for this type of treatment.


Hi @mattcen,

I know that the CSV output of the scripts tends to get used/consumed by other scripts at Cyber. Are you happy to confirm that it is AOK to make this change as far as impacting the good people of Cyber goes?


Ok. I see what I can do. Re: 200 You want it in a separate module I am guessing?


Write complete docs following initial browser tests passing


btw: i'm using mfpu=vfpv3 because "neon" prevent vectorization due of not precise instructions. but if u'll use this flag + -funsafe-math-optimizations .. then it's "forced" vectorized. And with "vfpv3" vectorization is performed ... without using -funsafe ...
So if u don't care about some ISO Rules, then u can still use the old value: neon-fp16 .. because that should give best performance :-)


Eh, sure.  That seems like the least-bad option.

I wanted to say we should add the FullPass flag as well, see if anyone
still needs the CX tag, but there are no existing per-Subread Flag
variables and we're not going to efficiently pack it below a uint8 ...
which is already the size we're using for the CX tag.  So we might as well
just pass along the whole shebang.

-Brett

On Mon, Sep 14, 2015 at 6:19 PM, David Alexander <notifications@github.com>
wrote:

> Seems reasonable. Do you think maybe we should just always include the cx
> info, like put it in the compulsory subreads table?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bnbowman/PacBioFileFormats/commit/559f2a0481e6eca2606a383e8cecf3081b181bef#commitcomment-13226988>
> .
>


Unhandled Fundamental Failure


@mattcen this seems like a matter of preference. Is there a technical reason why this is desirable?


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (801 of 801 tests passed)

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


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1336 of 1336 tests passed)

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


@mattcen and you've tested this one extensively?


This 'Watch the Video' button wasn't in the mock, but was discussed in the project specs. Decided to go ahead and add it. Also, I opted for these to be links as opposed to traditional buttons. Would revert back to buttons if they didn't redirect the user to a new url.


It's still there :crying_cat_face:


#4854-ZoneHVACOutdoorAirUnit-does-not-autosize-child-objects-correctly (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1336 of 1336 tests passed)

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


I tuned those 4 parameters starting with something almost identical to Stephan G values  16, 7, 14, 50
I was quite puzzled to see the p2 going from 7 to 1 in less than 2000 games.

Here is the effect of the "test" parameters. Much less change than Stephan version.

![2015-09-14_21h57_21](https://cloud.githubusercontent.com/assets/8949638/9866038/a133c7b4-5b2b-11e5-9d80-c9f993a8d7cc.jpg)


The comment is confusing. The logic of "waiting until the copy succeeds" is in the mothod `copy_blob` of `blob_manager`.
Here it is only a retry of `copy_stemcell`. So I suggest removing the comment.


Thanks, do you mean passing the whole logic of parseDirectives to the fold and keeping state being concatenation of  IncludeDirective in a F# list?

I just can't wrap my head around it yet so I've put a "todo for evaluation" for now.


Non c'è bisogno del metodo getGameSystem! ce l'hai già nella classe (si chiama "turno", il nome è un po' infelice ma vabbè, se vuoi rinominalo)

Per prenderci i pezzi di gioco usiamo Game.Services.GetService<T>().

Funziona così:
1. Creo i vari oggetti che ereditano da GameComponen in Quantum.cs, e li aggiungo alla raccolta di Components con Components.Add (elemento). In questo modo il Game chiama automaticamente le fasi di initialize, update ecc di tutti gli iscritti

2. Ne loro costruttore i componenti aggiungono sé stessi alla lista dei servizi

3. Tutti i componenti che hanno bisogno di un servizio fornito da un altro componente se lo possono pescare nel loro Initialize, chiamando game.Services.GetService<T>()


Зачет!


Пффф... Начинай снимать розовые очки :)

Если бы я стал перед выбором: соответствовать REST и притащить какой-то код (который еще и фильтром будет на всех запросах висеть) или не соответствовать, то я бы выбрал второе. Какой мега бенефит ты получишь от соответствия ресту? Пока не видно ничего конкретного, кроме того, что можно сказать, что ты соответствуешь.

Короче, есть случаи, когда в REST не получается уложиться технически. Вон, Dropbox, например, стали использовать GET вместо POST, когда приспичило: https://blogs.dropbox.com/developers/2015/03/limitations-of-the-get-method-in-http/ Так что догмы в топку, да здравствует разумный и прагматичный подход.


Ok, I was reluctant to open it as a sign I don't want to do direct IO but I think it was a bit of wishfull thinking on my end, for now this piece will use IO unless we have some virtual filesystem stuff.


Each variable assignment SHOULD be declared on a separate line, variables that are only declared but do not get a value assigned too


Good point, replaced with
```fsharp
let getDirectoryOfFile = Path.GetFullPathSafe >> Path.GetDirectoryName
```

I tried and `Path.GetDirectoryName` wouldn't resolve relative paths


Got it, thanks.


Should be translatable


Ой.
Был поторопился и неправильно тебя понял. Да, эта ссылка про BOM и тут непричем.


That's better yes :)


Хэх :) Хорошо сказал! Переделаю на POST.


@vahidreza I do not have access to the file right now. Please see the documentation from PhoneGap Build, and it should make things clear: http://docs.build.phonegap.com/en_US/configuring_config_file_element.md.html#_android


Fixed in https://github.com/galaxyproject/planemo/commit/ec0f170b0aaedb9a1f6631aea06ebbff4a61545c


Missing @returns jsdoc


5194_fix_failing_unit_test (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1869 of 1869 tests passed)

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


I've added these tests, because they are failing now after the change in https://github.com/bcosca/fatfree-core/commit/d0b4432252b965846cd23ac9b4271ab6d5fbc9b0


I would prefer `javax.inject.Singleton`


P.S. Отвечаю сам себе -- нет, не нужен, т.к. `The name of the configureGlobal method is not important.`


I added this and tried to render `proceduresView` the way `procedureEntryView` is rendered. Not sure why these two are different.


Ahh cool... i tried and it works....


I don't understand this construct. Why not to use `session.cmd(cmd)` directly?


Roger that.

On Tuesday, September 15, 2015, Artur Malabarba <notifications@github.com>
wrote:

> Let's wait a bit before we merge this. I have my hands full right now and
> I like to have some free time on my hands when making but big changes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Malabarba/clojure-mode/commit/8431e0de8393fd0ec2027167853ece57749a05ec#commitcomment-13232758>
> .
>

--
Best Regards,
Bozhidar Batsov

http://www.batsov.com


what was the intended fix here? I've added some new unit tests that probably should sum up this method's features. This regex is the key. But it doesn't fit into the keyhole yet.


I guess it's better to call it confirm_turnon_options then.
We already do confirm_usb_storage/cancel_usb_storage, so it would be more in line.


I don't think it is a problem to mark a feature as experimental.
In such cases, it is important what the default behavior is. If the default is "as before", the new behavior is very non-controversial as only those turn it on who hear / read about it, and do it consciously.  The drawback is that it is not seen by the vast majority.   I strongly still advocate to introduce experimental features that way.  So the ESS release (which makes it also into debian/ubuntu/fedora/..) should have the experimental feature turned off,  but after that we can turn it on for the advanced users who work from the bleeding edge (github, MELPA, ..).


5194_fix_failing_unit_test (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


FDGroundTempModelBugFix (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1335 of 1335 tests passed)

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


Submitted it already. It will not be accepted :P


Maybe a textual description of the test would be useful, it helps to understand the test without needing to read the code in details


I know you said not to change test app, but… When rendering `procedureEntryView`, we pass in `el` for `new` options, and view itself calls `el.html` directly. However, that is not the case with `proceduresView` and `Navigation` was passing `#tab-content` to `openInMain` previously.

Not sure how it worked before, since it was equivalent to something like this:
``` js
var tc = $('#tab-content');
tc.html(tc); // since it is not a function and not a String, removes everything, unbinds events.
```

I maybe wrong here, hope you can clear things out.


nitpick: Yes, it is possible to do it with regexp, but is it really necessarily? The format is given, it's "$previous $current". So IMO `ori_runlevel = ori_runlevel.split()[-1]` should be enough. (anyway I'm not against regexp, it just seems unneeded to me)


Thanks for thorough review @vasily-kirichenko.

I tried the updated code again and it's still working, will probably work on tests for this piece before moving upward to expose the functionality to GoToDefinitionFilter.


Tellraw wasn't accepted either


How about `"Guest runlevel is already %s", ori_runlevel`?


Hello, I'm trying to use your package in my project. I use npm-container in my project. However, the lastest version(0.3.0) seem still not updated with this commit. But when I clone it into a new folder, it has updated with this commit. That's so great if you can help me to solve this problem.


Underscores ? Job_blabla_Task_blabla


This does not seem right. Russian is replaced by English in the `_ru.properties` file.


See...


Comment on text etc etc


what ?


Yep yep yep


WAT? Maybe just add it in Slapper?


nope area is not mesured in lat / long


idem


It might be worth adding some debug info here: `"Changing runlevel from %s to %s failed (%s)", ori_runlevel, expect_runlevel, tmp_runlevel)`


однозначно стоит


This means ```include_once( __DIR__ . '/vendor/autoload.php' );``` becomes ```include_once __DIR__ . '/vendor/autoload.php';```.


Should be 1.7 as the project has prerequisite of "Java 7" or Java 6 is actually fine?


@henryso it was this line I belive


The bug I am facing for now, is that if you add couple of procedures, `proceduresCollection` gets all messed up and will contain duplicate models and not represent actual contents of local storage. Way to reproduce:

1. clear everything in `localStorage` out;
2. add procedure (name patient '1');
3. add procedure (name patient '2').
4. you'll see 2 identical models in collection itself, local storage looks all right
5. refresh — everything is fine;
6. click on '1', go back, click on '2', go back;
7. collection messed up again, local storage still fine.
8. refresh — everything is fine.

Not sure if `collection[0] === collection[1]` (same reference).


Not all services have been started when the guest get IP addr.
The guest runlevel maybe "unknown" in that time and the cmd exit status is 1, which will cause the cmd execution failed.

Wait for some time to make sure the services have been started.

Thanks.


You're right that as ids are decoupled we need to have a way to filter logs by job/taskid. Underscores maybe ? Cosmetic I know.


I'm not sure about which use case this is useful.


Ok, that's fair enough, will update it.


Yes, show the existing runlevel is better, will also update this one.

Thanks.


Hi Damian,

https://github.com/jitsi/jitsi-meet/blob/master/modules/UI/videolayout/LargeVideo.js#L281
The name 'activeSpeakerAvatar' is misleading and it actually is the avatar displayed on large video area. In this test there are only 2 participants(or at least should be 2 - maybe we should verify that at the beginning). It means that there are no active speaker switches. Once the conference starts participants is seeing remote peer all the time(even if there is active speaker notification). If the tests fails it means that UI is broken as we should never switch to see our local video on "large". This check is to verify if after our avatar has changed it has also changed for the remote peer who is watching us on large video.


Space before the colon.


In general : i use only CSS with a lot of class. Update the css file (and test with FireBug).

After : removing extra uneeded class.

Each one is method


Please indent all of the `|>` in by one. And same in `readCookie` below.


why do we need to clone ? (maybe a comment would help)


Hi Marco,

your words are just OK. It's easier to be more humble when you completely
understand the rules.
Please don't worry about me: I do understand what you are talking about.

On Tue, Sep 15, 2015 at 10:12 AM, Marco Costalba <notifications@github.com>
wrote:

> Question: "Can we see this on longer time controls, please!"
>
> Short answer: "No, we can't"
>
> Long answer: You have just arrived here out of nowhere, you know nothing
> of Fishtest and you pretend to just start inventing your own testing
> condition. This is very pretentious and arrogant from you. There is people
> much more involved and with proven success record that sticks strictly to
> the rules, I really don't understand why you (that know nothing) just
> don't.
>
> You should be more humble and follow the rules. Then, when you will be
> able to at least understand a bit of what you are doing (i.e. in some
> months time) you can ask for an exception *before* to submit and *wait*
> for an answer.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IIvec/Stockfish/commit/bf3023f13b50ac02808d51e6e50c29917004defb#commitcomment-13231122>
> .
>

--
IQ World <http://www.ultimaiq.net/>


Gotta catch them all !!! Millipede !!!


Cannot merge this as the change is incorrect. Readers and writers were going to the correct place in the original version but the placeholder text was on the wrong item. Please just just change the placeholder text.


Yes, but here , it's the old code. Update old code to use exception, maybe (but unsure we need exception here : we return an informatio to user, not an exception (I think)).
The question is : why move to -1 ? Why not leave null ?


As I understand it, including a newline in a header fields is an error in the WAI application. The user did not necessarily sent an invalid request, so is it really a good idea to return a 400 in that case?


其实好像大概也许是有的


LGTM. Thanks!


cat /data/serial_number?
How can you read this file which is not there?


I tried restoring this line and the call to this macro in `gregoriotex-syllable.tex`, but it didn't make a difference for the clef_change.gabc (nor any other) test.


One way to prevent an xpath selector here is would be to tap on the <span> element next to the the <img> (it's weird that one can't tap on the img area).
So I think (By.CSS_SELECTOR, '[@class="menu-item"] img[src^="app://camera.gaiamobile.org"] + span') could work as well.


if($iNewUID = $this->_createNewUser($username))
     // Do the job


@kazu-yamamoto would it make sense to revert this commit and turn it into a pull request, so that we can discuss it before merging?


And there's a way of using FormHandler classes (as it was in FOSUserBundle 1.3.x), which isn't a great option, but  I've seen some projects, written that way.

I can refactor this to the manager class if it's needed. Declare it as a service, get it from DI container in controller and pass the $author, $post and $comment to it. It will save the data to db and initialize + dispatch newCommentEvent. In controller there will be only 2 lines of code inside $form->isValid().


ok... no other clue then...


Here how you can cancel requests : https://github.com/mgonto/restangular/issues/387