It might be confusing that offset is only taken into account if limit is set.


Why is this not name?


Why not BSD like ARTIQ?


I see that you have `default="ADDED"` in the XSD definition for these components.
Please, don't forget that we support JavaConfig anyway. There is no any XSD validation, default value populations.
Hence we really should have similar kind of default hooks for JavaConfig as well.
E.g. for this `cacheEvents`:
```
private Set<String> cacheEvents = Collections.singleton(CacheEventType.ADDED.name());
```

Please, revise all stuff to be sure that we are consistent with those default values from XSD to achieve the same default state from JavaConfig.


Any reason to revert these?


need updating


fast-forwarding master to auto = 1bd8e18d


Out of interest, in other implementations we allow a proc / block / callback to be used when calling methods like this so that you can say something:
```
connection.Connect(-> {
  print "Connected"
})
```

Is this type of approach going to be supported later and possible?


I'd use the `https` protocol as it seems to be more reliable with ssh keys and the like.

Also reference the [tag](https://github.com/gemhome/rmagick/releases/tag/RMagick_2-13-4) rather than head.

e.g.

```ruby
# http://bundler.io/git.html
:git => 'https://github.com/rails/rails.git', :tag => 'v2.3.5'
```


Cannot input Chinese, are there any other way? Can input character, like a, B, but the switch input method input after the Chinese never to display text in the input box


It was my opinion that the code is clearer this way, as you would have to write key[i] many times having the function inline. However I will defer to your opinion and commit the change to inline the byte reversal


@somebody32 думаю лучше юзать устовшиеся названия. Читается быстрее и понятнее


Yes, thanks =).


ㅇㅁㅇ


BSD now.


Test... did no other good phrase for this comes to your mind? :smile:


e.g.

> sorry I realise now that this was just a problem with my office firewall, where the git:// protocol is blocked
>
> – _https://github.com/mysociety/alaveteli/pull/2244#issuecomment-86606808_


Mathieu j'ai une erreur bizarre depuis tout a l'heure et je comprend pas,
je n'ai touché a rien ma je comprend pas. J'ai essayé de re-cloner le
projet mais rien a faire.
Je t'ai mis en pièce jointe l'image de l'erreur.

2015-03-30 12:03 GMT+02:00 Mathieu Vedie <notifications@github.com>:

> ok, je te laisse continuer.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jondzaghe/formation/commit/1c6bc2a28d2867ec3c63f5b097f5bcfc4c7999cc#commitcomment-10464345>
> .
>


@daniel-beck this is summary in general for @ikedam maintaining. I will prefer to revert last 2 releases and decline https://groups.google.com/forum/?hl=en#!searchin/jenkinsci-dev/flexible-publish/jenkinsci-dev/ovlA2DbwUa8/8jpl-gZs9RMJ .


My opinion not @mitchellh so i'm prefer not dedicate external function for small operations.


/me gives a green light


Sense of this line?


Nicely done.


404. What do you intend to put there?


Sphinx documentation.


There is a shorter syntax you can use in the context of a model:

``` ruby
def save_with_initial_vote
  transaction do
    save
    create_vote
  end
end
```

(does exactly the same thing, `transaction` is just a shortcut for `ActiveRecord::Base.transaction`)


makes sense, lazy me, let's continue on reviewable


:smiley:


It's at http://m-labs.hk/artiq/manual for ARTIQ. Should we use http://m-labs.hk/artiq/pyparser ? Or http://m-labs.hk/pyparser ?


coding style: spaces around operators


I don't understand the "doc" error...


@0xF013 @floriank we need to work out some naming conventions here.


The latter, I think.


How do I push documentation there?


2015-03-30 6:45 GMT-03:00 Mikko Koivunalho <notifications@github.com>:

> Not sure now that you say it. I think the first time Dist::Zilla did not
> point it out to me (dzil listdeps). Now it seems to be doing it. I must
> have missed something when I started work. Sorry about that. I'll make a
> revert commit.
>
Don't worry about it. I'm not going to merge you pull request as is. I'll
rebase and squash the commits to make the history more concise. :-)

In fact, I already pushed two commits to next.

--
Gustavo.


Perfect.


No, the + means it is a changed line. The difference is the URl => URL, the l is uppercase


This visualization uses d3.v3.js and nv.d3.js which are required under the lib directory. Would it be okay to add these files?


NodeDecorator.NODE_NAME_KEY should be used instead of 'name' key.


Useless assignment to variable - `errornous_records`.


Useless assignment to variable - `buffer`.


Annotation keywords should be all upper case, followed by a colon and a space, then a note describing the problem.


Why did you do this? :)


Kommentar


ORLY? Can't you find your task by action and delete it? And config value too


Update regarding the above comment: ceb24ce930d9ea5b7408b59b4c45aedbaaa45f09


Somehow, the date and revision in the `jstreegrid.js` header were not updated.


dict is a mutable type


No. I'll sort it.


The purpose of the change was to get file I/O out of the FHDL backend and into mibuild.


Up :-)


maybe that would be better on install time...


This still seems wrong. The term is lowercase and is not code. Reading this text you might easily be confused.


Added the zend application code files


Is there a particular reason for including Xinerama by default?


Again, dict is a mutable type, so this is unnecessary.


EDIF


Actually there was something else....I had to make a decision between "Test" and "Testo" ;)


Fonction dynamique de modification de graphiques.


Shouldn't need a nested loop.


There should be some range checking done here to make sure S-EL1 is not passing a dodgy x1 pointer to EL3. This might require a call into platform specific code.


有bug，灭活button的时候会把已经有数字的button激活。


@PiotrRudnickiTRP bardzo dobra zmiana!


Hm alright. Apologies, I just realized this commit is from 2013! I had a little trouble compiling the source, but managed to get it functional after a while, albeit with 3 compilation errors due to missing methods and java random import. Everything still seems to function without errors though. Did have to change plugin.Method to plugin.getMethod() as per the upstream changes. I'll have to see what happens if the rollover occurs twice in a row.

Just out of curiosity, why haven't you used this in ages? Found a new plugin or just not operating a server anymore?


Ajoute un graphique de manière dynamique.


Use newly defined fields on NodeDecorator
https://github.com/slipstream/SlipStreamClient/blob/cf5273fbab0e79dc3a7f31ff558217ef1ecf2293/client/src/main/python/slipstream/SlipStreamHttpClient.py#L434


Github add them.  There are no whitespaces in my local version.


We discussed that ``str`` should print the whole content of the conversion object, including data files.
http://irclog.whitequark.org/m-labs/2015-03-19#12165196;


Sounds good - thanks for the info


I think there is a problem of type.
`Traceback (most recent call last):
  File "/usr/local/bin/crisptime", line 116, in <module>
    rough_offset, flow = crisp.sync_camera_gyro(camera.image_sequence(), camera.timestamps, imu.gyro_data, imu.timestamps, levels, full_output=True)
  File "/usr/local/lib/python2.7/dist-packages/crisp/timesync.py", line 84, in sync_camera_gyro
    assert image_sequence_or_flow.ndim == 1
AttributeError: 'generator' object has no attribute 'ndim'`

The function `crisp.timesync.sync_camera_gyro(image_sequence_or_flow , ...)` is called by the script crisptime this way
`rough_offset, flow, frame_pair = crisp.sync_camera_gyro_manual(camera.image_sequence(),     camera.timestamps, imu.gyro_data, imu.timestamps, full_output=True)`
So:
`image_sequence_or_flow = camera.image_sequence()` which is a generator and not a ndarray as expected.


Alright. Don't publish the safari version just yet
 We'll do it during the meeting, whenever we have it.


nit: Ideally these asserts would be part of the earlier patch(es).


Shit. Forgot --no-preserve-root.


#acasc


Why not have a single conversion object? It should be just a container for the main source code + data files.


Use newly defined fields on `NodeDecorator`
```NODE_PRERECIPE = 'prerecipe'
NODE_RECIPE = 'recipe'
NODE_PACKAGES = 'packages'``` when referring to the scripts/packages, e.g. here:
https://github.com/slipstream/SlipStreamClient/blob/cf5273fbab0e79dc3a7f31ff558217ef1ecf2293/client/src/main/python/slipstream/SlipStreamHttpClient.py#L434


And I am Yuriy or Yurii. Seriously... Read my github. XD


Fonction qui démarre tous les graphiques défini, avec une durée prédéfini.


Yes
Create >>> "Create New discussion Pad" :-)


还是那个问题...所有button都获得数字之后，前四个button会被激活。


lol dafuq


@equivalentideas I think this is here as the way the help pages are marked up, we were getting a double top margin from the ```dd``` and the first ```p```.

However, looking at it again I see it has made things a bit cramped, and I'm wondering why this is in here at all. I'll open a ticket to do a quick review of this. Thanks for the feedback!


I thought that we might need to pass `limit=0` as encoding for *no limit` at some place. Not sure if this is relevant though.


Some things are not optimal. Please compare this to the Botania documentation I wrote here https://github.com/jaredlll08/ModTweaker2-Documentation/blob/master/Botania%20%5B%5BModTweakerBotania_SupportBotania%20Support%5D%5D.txt
* The headline doesn't name the mod.
* Why are there three *** under the headline?
* "Adding Recipes", "Removing Recipes" is not required, it's kind of self-explanatory.
* "//Examples:" is not needed either and just wastes space.
* Don't use InputStackSecondary but InputStack1/2
* If there are no commands, you don't have to say that, just don't list them.

Thanks.


Doesn't make much sense without limit for me, similar to the fact that `reverse` only makes sense if `sort` is set.


Is there bang version, `create!` too in associations? If then that should be used similarly as `ListingShape.create!` is used


Can you set the employee_id in the controller rather than resorting to a hidden field? Perhaps taking advantage of current_user?


所有button获得数字，求和之后，那些button还是可以点击的...


Push it! https://www.youtube.com/watch?v=YleXlgHI1oM


Well, this is actually a good style. Because developer need not to think over which quotes to use.

Do you know why Ruby developers chose to have both quotes for string? Because on very old ruby versions single quotes were much faster than double quotes (due to string interpolation feature). But now it is not true, so original reason to use different quotes in different places is gone.


Adding the most trivial solution that first comes to mind: `ZoomSliderCtrl`?

This would directly relate to the directive binding the controller. As always. *Naming Stuff is hard*TM


@bigson0504 Trampoline이 가속입니다;;


My preference is `%{}` and `%r{}` somehow.


Aqui foram incluídas tabulações para melhorar a organização.


_You_ might be confused. I doubt anyone else will be.

But yes, dropping the `<code>` bits (by turning it into a `dfn` instead of an `interface` definition, and no longer using the `{{...}}` shortcut syntax) is on my list. It's just more busywork than updating the link.


和前面三部分的问题相同


I created the tree, having string values, as in assignment 3. I printed the tree with the prettyPrint function. Then I created a recursive calc function which also computes the result.


It still doesn't work with this.


@mytch444 you might wanna do pull request XD


Looks fine. You may have to run the queries one by one.


in the details pane refactoring branch I have fieldController, displayPaneController and editPaneController to distinguish between controllers. Hello Java


Filed w3c/webappsec#234 so I won't forget.


Why?


@jmacmahon I see that this is merged in CDS's invenio repo: https://github.com/CERNDocumentServer/invenio/commit/7e361d7b7edc4f6665d2a6200e9e54e6b4884862 ([History for modules/bibdocfile/lib/bibdocfile.py](https://github.com/CERNDocumentServer/invenio/commits/prod/modules/bibdocfile/lib/bibdocfile.py)). However, changes to `modules/miscutil/lib/__init__.py` are not committed.


Shouldn't this be imported only if `CFG_ELASTICSEARCH_LOGGING` it `true`?


比较　gb_trees ,dict,list,map　插入和查找kv数据的效率


kannst wieder weg machen, denn du musst das ding auch mit sudo starten.

`sudo make start`
`sudo make stop`

ist aktuell doof gelöst, weil man sonst nicht in /var/run schreiben kann.
aber ist ja nur zum debuggen ;)


Please open an issue if you have one (commit comments are easily missed) and provide an [executable example code](https://github.com/libgdx/libgdx/wiki/Getting-help#executable-example-code) so I can see and fix problems without guessing.

It might make more sense to use BitmapFontCache instead of Label, since you aren't using scene2d.

It still looks like `lines` is correct. Eg, when "This is regular text\nwith a newline." then `lines=1`, which is correct for computing the height.


Nice use of fields_for


Suggesting an extraction of
```
listener = self._get_listener()
...
util.printStep(message)
listener.write_for(machine_name, message)
``` into `_print_step(msg, write_for=None)` if `write_for` is not `None`, `_get_listener()` and call `write_for()` on it.


also
#/bash/bin
Dah fuck Yuriy?


Could you tell me how it works in Log Dialog?
Why do you take parent N to compare with?


Will do tonight :)


Maybe better something like this:

**Note**: File or directory names are not expanded, e.g. `../path` is taken literally and not expanded.


@mytch444 I was mentoring mashup that day.


Indeed, removing channels before detached state is an oversight. I'll fix it.


da du den pfad sowieso in /tmp geändert hast ist das sudo in beiden variationen überflüssig :dancers:


:+1: ;)


One current difference is escaping. This is kinda confusing so probably another reason to use double everywhere.

```ruby
irb(main):001:0> '\n'
=> "\\n"
irb(main):002:0> "\n"
=> "\n"
```


Were these skill level designations worked out with the 'client'?


How was it? Tauranga one?


Doesn't this give a conflicting resolution of the old-to-new mapping of previous pages?


About the callbacks, I wanted to implement an api that is natural to .net developers. Receiving result (and acting upon it) is implemented in .net 4.0+ with Tasks API. If this is an important feature I can add an override with a predicate of type Action<T>. This way we can have an api like:

```csharp
channel.Publish("event", null);

// With callback
channel.Publish("event", null, (result) => { Console.Writeline("Message received" });

// Or (.net40 +)
Task<PublishResult> result = channel.PublishAsync("event", null);
result.ContinueWith((antecedent) => { Console.Writeline("Message received" } })
```

The point I wanted to make is, that in .NET there are currently [3 patterns](https://msdn.microsoft.com/en-us/library/jj152938(v=vs.110).aspx) for async methods and I wanted to conform to these. I feel that this style of callbacks is more like javascript programming, which is not a bad thing, but a little bit alien for .net. There are some examples of such an api in C# [here](https://socketio4net.codeplex.com/).

Martin, what do you think? @marto83


已经存在 -> 现有
重新用 JavaScript 实现 -> 用 JavaScript 再


Wie gesagt -> Eigenes Proejekt machen!
Mega cool :+1:


I think the `check_previous_import` should also consider the `Host` argument.


Edit parameters.yml file like;
https://gist.github.com/behramcelen/d53622eafc7700e34f41


Thanks Matt, I`ll look into it.


Add a conversion for previous mod_import_anymeta installs without the host/stub columns.


``in fdict``
keys is unnecessary.


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


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


Fixed in  91c4266


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


This function should be exported.


Not operating a server any more, haven't done for eight months. Server I
occasionally help out with now doesn't have a use for it!

On Monday, 30 March 2015, mibby <notifications@github.com> wrote:

> Hm alright. Apologies, I just realized this commit is from 2013! I had a
> little trouble compiling the source, but managed to get it functional after
> a while, albeit with 3 compilation errors due to missing methods and java
> random import. Everything still seems to function without errors though.
> Did have to change plugin.Method to plugin.getMethod() as per the upstream
> changes. I'll have to see what happens if the rollover occurs twice in a
> row.
>
> Just out of curiosity, why haven't you used this in ages? Found a new
> plugin or just not operating a server anymore?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dualspiral/Lottery/commit/5b7565916d4409a26cba0c84399c8013edba5eef#commitcomment-10465524>
> .
>


So you get a checkbox for a skill, and then four radio buttons to choose from? Do the buttons need to each be written out separately in the code, as here, or can you combine?


This is incorrect place for doing a cleanup. First of all this introduces a bug into galen when generating two reports at the same time (htmlreport and jsonreport). When it gets to generating a json report - it fails with an exception because the files are already deleted. That's the place where I moved all the cleanup https://github.com/galenframework/galen/blob/master/src/main/java/net/mindengine/galen/GalenMain.java#L314
This is executed after all test reports are generated. Basically that's the point where we already know that we wouldn't need the files anymore and they will not be used. If you find a problem in file cleanup then you should look in that place


noSuchEJBException is lost


Do the NS check before asserting (in case you have a malicious NS client).


I was hoping for [this sort of badgers](https://www.youtube.com/watch?v=EIyixC9NsLI)


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


Shouldn't there be an example to bind an address?
`ListenStream=127.0.0.1:443`


Prefer one var declaration per line for easier diffing


Empty line


Shouldn't there be a "if (ns) SMC_RET1(handle, SMC_UNK);" block like for TLK_ENTRY_DONE?


Ok - I'm planning to send you one big pull request with several changes soon, once I have everything I need for now.


nit: Ideally part of earlier patch


Do NS check before asserting.


Done in 702cdb0.


@jmacmahon Also, let's include in the commit the missing amendments to [`webstat_config.py`](https://github.com/CERNDocumentServer/invenio/blob/prod/modules/webstat/lib/webstat_config.py), i.e. not importing and calling `register_schema` anymore.


Sure, I will add the `config` object to the deployment package.  Then it will be something like this:  https://github.com/BitShares/BitShares-JS/blob/v1/src/client/js_client.coffee#L19-L20

I'll check this in after I test it:
```
diff --git a/src/deploy/web_wallet.coffee b/src/deploy/web_wallet.coffee
index 9e1aec0..8078e8e 100644
--- a/src/deploy/web_wallet.coffee
+++ b/src/deploy/web_wallet.coffee
@@ -8,3 +8,4 @@ module.exports =
   hash: require '../ecc/hash'
   secureRandom: require 'secure-random'
   logging: require './logging'
+  config: require '../config'
```


See 4f618b432c43aa1685357bab28376d6c9ad75d4f


Hi Anthony,
Is openocd is working with ashling and can be sustitue for mdb?

Noam


/usr/syno/bin gets purged whenever DSM is updated. /use/local/bin should be fine I think though.


I'm not really familiar with the threading model in .NET, it would be great to understand that at some point.  Don't bother replying in here, perhaps we can have a quick chat at some point.


Needs context.


Yeah, and the old code was better for that case. Also, the catch was limited for `binary_to_term/1`. It's probably a good idea to restore the old code, but change the message. However, I like that we print exactly what failed, which makes it much easier to debug this unlikely exception. So how about just turning the ?ABORT calls into ?WARN followed by a `file:delete`?


I don't know if it matters for the CSV syntax, but you could also remove the spaces after the comma:
```php
$output = '"Firstname","Lastname","Gender","E-mail","Active"'. "\n" ;
```


Wow! You're back!


As you can see, I have finished the extra assignment. Even so, I haven't used queues at all. Was that mandatory? I took it more as a hint. If needed, i can redo it with queues.
P.S. I've also slightly modified the compulsory assignment
P.P.S. I did a bit of team work with a group mate of mine, Alexa. Is that forbidden?


Convention used by Erlang/OTP and erlang-mode, allowing to quickly (un)comment.
Trailing comments use % and normal comments use %%. There's also %%%, but it's not used in rebar.


Together with my colleague Roxana Pop we did the extra assignment without using Q.
I first thought of putting the elements into a q and then using the function created in the assignment of "converting" the list into a tree, but it seemed redundant.


after addition


Main problem is the selection which I don't think was working properly. Can you check that!


Looks like these numbers are the wrong way round, should be status code first, see https://github.com/ably/ably-java/blob/master/src/io/ably/types/AblyException.java#L37


this should be refactored.  your `systemFirefox` string can be reused.


Because of #530


Again, think the error codes are the wrong way round, , see https://github.com/ably/ably-java/blob/master/src/io/ably/types/AblyException.java#L37


We could also put updateOrAddOne / updateOrAddMany into `./utils` like the rest. The only reason I left them here is because they are only used in this file.


Just so you know, a think a lot of the bloat is a non modular bitcore version.  I believe they have done some work in that area.  In the end, the library can potentially be Much smaller than it is now.  I suppose a separate uglify target would not break the development deploy task `web_wallet`, however, if we clean up bitcore (which I really should do anyways) then do you think you might not need uglify?

Do you have an updated bip32.org link?  It did not work.


should this instead be `opts.http_proxy || process.env.http_proxy` maybe?


2.46.0 or may be 2.45.1, who knows :)


vole vole, dobre, moc dobre


Mark for deletion.
Substitute with the correct service units.


Cool .. thanks


Running os:cmd is okay. The reliance on external tools is a concern for portability due to non-existence and differences in options/behavior. FWIW, Busybox seems to support -c%y.

Would sleeping 1500msec work?

If this is the only way to solve it reliably, it's okay.


Yes in Bluepower the original copper ore is like this
![copper_ore](https://cloud.githubusercontent.com/assets/9092453/6895812/5649826e-d6e4-11e4-973d-52f599015125.png)


add color


nit: So up until this patch, the wrong number is being reported. Not a big issue.


My concern is that the mtime change may not be reflected before the check, and that would make this brittle. Like a timeout that is sometimes exceeded.


Sure, I just wanted a smaller js file. If the non-ugly has managable size, I'd also prefer to keep it as is.

bip32.org seems to be hosted by github which experiences some DDOS atm:

* https://github.com/bip32/bip32.github.io

Thanks for all the great work you do!!


No - ```:``` is a poor separator for Capabilities like ```namespace:modalias(usb:v046DpC00E*dc*dsc*dp*ic*isc*ip*)``` or ```perl(XML::Simple)``` or ```locale(emacs:ja)```


Since for loops re-use the pointer, there was an intermediate factorization that failed because only the last item in the list was substituted in.  I think we'd probably be ok switching it back now.  Care you make a guess before I try?


Out of interest, was this a production key in your account?  If so, I recommend you delete it and issue a new one to invalidate this.


This looks good.


OK


That structure goes into the documentation, as it describes the HTTP serving API. I am hesitant putting conditionals inside.


保留 `native` 不翻译为 `本地`
访问平台全部功能的代码成为可能 -> 访问平台全部功能的能力
通常开发流程 -> 一般开发流程


Don't create a macro like this for *one* call.


That is tiny - no harm to leave it without conditional compiling?


Works for me, but shouldn't we make sure a syntax error is nicely reported, and isn't just an exception. I'm thinking of atom_value(attribute_name()).


It would seem that the change at line 70/71 of src/lib/php/common-auth.php makes for some confusion of the use of username. It might be clearer to retain the username setting in order to match the usage of the switches/arguments for other cli utilities.


Why is Thrift a dependency?


Us


I believe that I specifically asked you to modify control/kdl_conversion to add a SDF-to-KDL. The reason for this is that we will need it very very very soon. Please do it !


OK, fair enough. Let's get that finished and do the KDL stuff just afterwards.


I think ugly call member pointer function.


i've linked to /usr/bin/ for now...
yeah linking to /usr/syno/bin isn't a good idea either... thats for synology stuff... also i'm not really a fan of linking binaries, i think if the user wants to he can do it


omglol tjockt bred border jao


You really don't need to reserve in most cases. That's one of them. I bet that the performance gain is close to zero. In addition, you don't need to whether the children are empty, inserting an empty range works just fine.


显示要求 -> 明确要求


You basically search for the segment name twice here. That's bad form.

~~~
LinkIterator link_it = links_map.find(kdl_segment.getName());
if (link_it != links_map.end())
{
   sdf_link = *link_it;
   ....
}
~~~


To add to that, I think it's best practise to export all internal dependencies for testing. So for example, we'd do

```js
exports.toId = require('./utils/to-id')
```

so we can later in our tests do

```js
var updateOrAdd require('../path/to/update-or-add')
updateOrAdd.toId = function(){ return 'mymockedid'; }
// etc
```

It should just not be exposed to the public API.


Why not

~~~
osg::Node* new_hook = makeOsg2(kdl_segment, sdf_link, hook->asGroup());
~~~


@mareknovotny Removed in-container category. You are absolutely right - this is a plain Arquillian Weld SE test.


I forgot this when writing the comment, but the primary reason for using modified|unmodified is that it improves code readability. I find this use of `andalso` un-Erlang'ish, but that may be subjective. Restoring the Modified param and renaming it to `maybe_store_erlcinfo/3` could even be more descriptive and easier to understand.


is that space at the start intentional?


Yeah, had a rough time with this too. Several issues at play here:
* Stuff just worked when writing to a file share, not in 'native' mode
* When fileshare is enabled, you can not 'chown' the files because this is not supported
* The previously used ramfs used for temporary files does not support ACL

So I enabled ACL's and changed the temporary filesystem type from ramfs to tmpfs. The downside here is that the tmpfs needs a 'fixed' filesize (ramfs was an auto-growing one). So the last thing I need is a 'checking' script that checks if the typo3temp filesystem is almost full, and if so, will remount the filesystem with a larger size option. Taking care not to use more than x% of available system memory. That's on the top of the TODO


А может просто удалить блок с файлом, а не перерисовывать весь список? Это слишком накладно.


Okay.


Because the signature has changed - add documentation string for the parameters.


It's overall fine. However, there is obviously a lot of common logic between the URDF and SDF parts. I would have liked you to factor this logic out.

In other words, you're on the right path, but instead of buildings maps of name-to-SDF elements, create a local structure in which you extract the information from the SDF element, and work on that. This way, we can adapt the URDF codepath to do the same.

Basically, the OSG tree building should be split between:
 - extract information from URDF or SDF to a common representation
 - build the OSG from this representation.


@yavor87 I think when we chatted we thought the best approach would be to use Async because it was so widely supported, and then later as an additional bit of work add Task wrappers around the async methods as you said there was a generic wrapper?  I agree we should try and use the standard .NET way of doing things.


Super Commit


This should be static


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


[tests/testthat/test-r_random-generator.R:16:13:](https://github.com/scrm/scrm-r/blob/37dcb09aed16884c8a4d42b1f34f27886d830292/tests/testthat/test-r_random-generator.R#L16) *style:* **Variable and function names should be all lowercase.**
```r
    smpl <- test_RRG_sampleUnitExpo()
            ^~~~~~~~~~~~~~~~~~~~~~~
```


seq 22 9 ?


iiiiiiiiiiiiinteresting +1: :)


You're right, brain-fart. :)


Yes, we should aim to not introduce more of that, but `sed -i` behaves differently on each platform, and we should of course replace that chmod call with a call to the relevant Erlang function (in another patch). Can't you just copy a pre-existing version of the modified file to make it much simpler?


Having this tokens here is easier for dev and QA to run smoke test because you don't need the readinglist@restmail.net password, they are related to the readinglist only scope and we can revoke them in case of problem but I think it is simple enough.

I don't think loads let us pass them as CLI arguments, be we could read them from the ENV.


I am not sure. It comes from the REST version.


Addressed with ca2ac5bb23456988d5ae515556d9b9b33423cce6


Code review test. This seems a good change!


Hi - I think it was worse than... I have forked it somehow. I have never used git like this. Will try and undo, and then redo the changes from the command line. Thanks!


I belive this patch is already applied, on the suse branch.


You can use `git rebase`.


It was added by mistake. Visual studio adds configuration variables to multiple places for some reason.


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


It's not needed anymore.  We used to use Thrift as our binary protocol, we now use MsgPack only.


Trailing colon.  Maybe drop the bullet and add `, then perform the following steps:` at the end?


Arf...  Sorry again. I m completely lost.

"Ardoise de collaboration" is not displayed now. If you prefere I have this :

![2015-03-30_140409](https://cloud.githubusercontent.com/assets/3939543/6895929/ad5bd2c2-d6e5-11e4-8559-5ed3e3ec045d.jpg)

I found no option in the settings for "Create" >>> "Create New discussion Pad"

perhaps I have to make an appointment with a ophtalmo ?


@gck-ableton: nitpick, `#include <QtQuickTest/QtQuickTest>` is more `Qt`-correct


> Shouldn't there be an example to bind an address?

Those instructions are about changing the port, not the address. If you feel like starting another refsection about listening on a different address, then such a pull request would be welcome.


Good stuff.

Just curious why you are querying the DB instead of using the Wordpress provided functions?


No harm, but this is kind of pointless for a case where there are no computed parameters.


I used the URDF code as example.


JavaScript 后少个空格

这里的 only 不能译成唯一了，因为可以回调也可以发射事件，译成只能好些：
> 所以只能通过回调或发射事件把结果返回给 JavaScript


Hi Krikor, this is the commit of OC-3375.


I recommend that you used <= 4 or < 5 so if any value passed in less than 5 will trigger this response.


ok I found... :-)


LGTM


:ok:


Thank you!


@mareknovotny I also fixed the bug with executing tests. To sum it up: if test has 2 categories, you need 2 profiles to run it


I recommend that you use >= in this case, if they have that much money or more they are good to do business with not just that exact amount.


Ok, I will do it. I did not yet, because I want to put to work first, after I will do some improves.


This is a longish one. It would need testing.


@gck-ableton: I think the following would be more readable:
```
            TestUtils.withComponent(minimalScene, scene, {}, function(comp) {
                    compare(comp.foo, "B");
                }
            );
```


thank you


Ah yes, the two PRs interfered. Sorry about that! I won't be able to submit a fix for a bit since I'm not at my computer right now.


I've too have faced this issue. I get error if i change this back to require no errors appears to happen. PHP version PHP 5.5.9.


Added recovery info


In this case the check is correct but could be made stronger by
dollars >= 1000 && dollars < 1000000000

This way if they have 1000 dollars or more, up to the next level, they will be accepted.


@gck-ableton: nitpick, order alphabetically


Now I see what you did there. I don't think having slightly more verbose code here is a bad thing. This looks unusual and slower to read, but that may be subjective. I'm leaning toward using the minimally more verbose version. @ferd, what do you think?


Ceci est un commentaire sur le commit


If this change actually fixes a problem, then the bug is somewhere else.

[l]wmode should be set to NO_WRITE at the end of the timer interrupt, except when OWM_WRITE, OWM_SEARCH_ZERO, or OWM_SEARCH_ONE need to write another bit. So I set it to NO_WRITE at the top instead of in every branch of the following switch statement.


Why did you remove all test dependencies from package.json? o.O


Well done, it seems that you are understanding pretty well.


can you do something like `register_to(Node)(document.__class__)`?


Okay.


Strange, I'll look into this.


just test comment


Yup, but what I meant was is that this key is now effectively in the public domain, so if it's your key from your Ably account I would invalidate it.


For me it is OK!
You can finish the task with the remaining links on the navbar and merge the changes.
:+1:


Thinking about this a little - it is probably better to add onItemDoubleClick to the filesAdapter:
```go
 type filesAdapter struct {
 	gxui.AdapterBase
 	files []string // The absolute file paths
 	onItemDoubleClick func(gxui.AdapterItem)
 }
```
And register this function in Create:

```go
 func (a *filesAdapter) Create(theme gxui.Theme, index int) gxui.Control {
 	path := a.files[index]
 	_, name := filepath.Split(path)
 	label := theme.CreateLabel()
 	label.SetText(name)
 	if fi, err := os.Stat(path); err == nil && fi.IsDir() {
 		label.SetColor(directoryColor)
 	} else {
 		label.SetColor(fileColor)
 	}
 	label.OnDoubleClick(a.onItemDoubleClick)
 	return label
 }
```

That way, double clicking in some blank bit of the list won't cause the double-click logic to fire.

Alternatively, this logic might be common enough to justify adding OnItemClick / OnItemDoubleClick methods on to Tree and List.

What do you think?


1)Newline required after the declaration of variables
2)Error handling must be done for the realpath() function


就像这样 -> 比如

在这种情况下，是时候考虑 -> 这时可以考虑

就像这样 -> 比如


That's most likely another vestige of package module support. We need to remove it in that case.


Well I thought the unit test was running clean prior to check in. I'll have to check this again to be sure.


Also, inserting `gametic &&` into the `if` condition prevents the title screen from fading in.


What I meant was is that if this key was from your Ably account, then you should invalidate it as it is now in the public domain.


@gck-ableton: same here, the following would be more readable IMHO:
```
            TestUtils.withComponent(minimalCase, scene, {}, function(comp) {
                    compare(comp.textValue, "B")
                }
            );
```


You have to check whether the arguments have proper value or not.


Jerry,

I realized that I was taking the easy way as I was submitting it. My kids
were taking a nap, and I wanted to get as much done in that time as
possible. I can go clean it up, if you want.

On Monday, March 30, 2015, Jerry Edens <notifications@github.com> wrote:

> Well done, it seems that you are understanding pretty well.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bjwade1/ios-exercises/commit/fa86acf39074776f8f248abb8e666e2ab54acfe2#commitcomment-10466517>
> .
>


replace with `// Don't do anything.`


Really! I will change this.


You can maybe extract an ```addLast``` method from this function?


We cannot depend on a 3rd party SNAPSHOT version.


Good job. You can modularize it a bit if you want to


lol trop bien !


We do not use @author tags in javadoc at Apache


Do you mind adding a simple comment about the `op` parameter so that's it's easier to understand why 1 and 2?


how does it works on staging


It is stated in the comments that `[W, H` is specified in pixels.
Since the third parameter is in "pixels per unit" (where unit can be inches or centimeters or pixels or ...), I would say it makes sense to use "1 pixel per pixel".

However, the name `PPI` is misleading in that case.


@gck-ableton: the commit message is quite hard to read, mostly because there is something in the first sentence...


Ubuntu 12.04 LTS, so not the newest. dpkg says libxcb-ewmh1 **0.3.8-1build1**.
I currently can't debug, but I'll get back once I can.


Thanks!


@gck-ableton: same thing here about readability (see other comments).


Instead of sync we encourage users to use doStart / doStop to implement logic for this kind.


@gck-ableton: same comments on this commit as on the previous one


1) Give a meaningful name to the variable p. Other functions also have the same variable p change it.
2) Newline is required after the declaration of variables.


A component should favor storing state on the message - eg as header / body as that is message information.

State on exchange property is often only for EIP patterns.


Check the value of length before allocating memory.


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


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


else block is not required. Write the return buffer; statement outside the if block.


Just FYI, the stmp stuff shouldn't be needed. With the results_db.analysis.find_one() we return the same JSON that you'd get from report.json. You should be able to access this type of data by just removing the len().


This should be @UriPath as its part of the context path, not as a uri query parameter.


I need to add a PhantomJS launcher!


This pattern is specifically used to modify a JSON object in place. We construct a reference when we initialize the object to its "empty" state so that we can populate the object in its rightful place. This idiom started before we had move-aware JSON objects, but I still prefer it.


Throw an exception instead of logging


I really think that we should make it clear in the infrastructure section that the "current document" is the document in the current browsing context (i.e. includes subframes as well as the tlbc). This basically follows from HTML, if we are careful, but should be called out explicitly. That can be a followup commit though.


I think you want a z here, unfortunately.


Apply this filter globally and use "AllowAnonymous" attribute on Index() in LoginController


这里 数字 和 字符串 不建议翻译

一个 `NSArray` 同时包含 `NSNumber` 和 `NSString` -> 一个包含 `NSNumber` 和 `NSString` 类型的 `NSArray`


Move model declaration to the top of the page


Yep right  here @CodeBrauer [Async Defer](https://www.igvita.com/2014/05/20/script-injected-async-scripts-considered-harmful/)


If PinMode is an enum class, then Camel has out of the box type converter to/from String for that guy, and therefore not really needed.


Did you forget to change this one?


z


Use Layout for this. You can check IsAuthenticated to decide what to display.


so the tree2list to be contructed of 2 functions? 1: addLast and the oher one the "tree2list"?


Hmm, perhaps a more 'shield generator'-esque image would be appropriate here?


Using a description as a blurb. That can't possibly go wrong...


Well, from the [history of xcb/util-wm](http://cgit.freedesktop.org/xcb/util-wm/log/) it looks like there was [a bug fixed after 0.3.8 was tagged](http://cgit.freedesktop.org/xcb/util-wm/commit/?id=23a4eb6059e0a694e342c3ff41ab076dad50b745) relating to `xcb_ewmh_get_supporting_wm_check`.


I first wrote it like that but something went wrong and my list was not constructed as I wanted to be


I'll give it a try later ;)


Why not also enable `icc-profile-auto` in this profile?


Hi Madhura,
See 0d94439eda08703880ea77fa43474b599c78a02b; I hope you agree.


sortof, what I meant is that you could have ```tree2list``` call a function called ```addLast``` instead of having all that code there


Ok, I will remove it.

On Mon, Mar 30, 2015 at 3:13 PM, mattheworiordan <notifications@github.com>
wrote:

> It's not needed anymore. We used to use Thrift as our binary protocol, we
> now use MsgPack only.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ably/ably-dotnet/commit/69e59dbaac0f50949731bde43a9ea4f3fc57a1b8#commitcomment-10466420>
> .
>


Unfortunately, `PathResolver` resides in `DynamoSandbox.exe`, not something `DynamoRevit` can depend on :(

Would you recommend doing this through other ways like `DefaultPathResolver`? The only problem with that is it will be misused in no time and go out of control.


when you update a lib version explain in the commit why. It might be useless later


looks good but now you have to mark it as explicit - otherwise you my get some nasty implicity conversions.
Maybe you should also use the default n and c-term gains here instead of the empty formula.


При добавлении комментария нужно автоматически запихнуть это Yii::$app->user->identity->username в id_user,и тут у меня проблема


aha...got it ;)


As discussed internally with @drjova
* `CFG_ELASTICSEARCH_EVENTS_MAP` should also contain the configuration for visualization
* `CFG_ELASTICSEARCH_EVENTS_MAP` could be renamed to simply `CFG_ELASTICSEARCH_EVENTS`
* In order to avoid doing pull requests to Invenio every time we want to define a new event, we could save the configuration in a config file, much like `webstat/webstat.cfg` which we will then overwrite with the cds overlay. The new file can be called something like `webstat/events.json`.


We are currently improving the way authentication works, including LDAP, for Indico v1.9.2, which will be released in the near future.
We plan on making the LDAP configuration more flexible. We have noted your request and we will expose this parameter in the configuration.

Thanks for the feedback.


Merged into master.


Зачем тут enter'ов так много?


`pathResolver` is initialized as `RevitTestPathResolver`, but it is declared to be of type `IPathResolver` so it needs to be cast to `RevitTestPathResolver` before its `InitializePreloadedLibraries` method can be used. The cast `as` will not fail since the assignment was done right before the cast, in which case we know for sure that the `pathResolver` is an instance of `RevitTestPathResolver`. Hope that helps.


其它 -> 其余


button(ng-click='createTag()')=env.t('add')


Cool.


They don't *need* to be in source control, it's just a way to test the service without having to enter anything.

We could also get the token dynamically by doing the authentication dance, but I don't think the tooling is ready for this just yet (in PyFxA).


Мы же решили, что оценка не зависит от написания отзыва!
Зачем ее прилеплять к отзыву тогда?


There shouldn't be a reason to change this (see issue https://github.com/SpongePowered/Sponge/issues/210)
See section [2.3.3](https://google-styleguide.googlecode.com/svn/trunk/javaguide.html#s2.3.3-non-ascii-characters) of the Google style guide.
> __Tip:__ Never make your code less readable simply out of fear that some programs might not handle non-ASCII characters properly. If that should happen, those programs are __broken__ and they must be __fixed__.


`Cockpit starts on demand via socket activation. To change its port and/or address you should place the following content in the [...]`

So yes, I still think this belongs in this refsection. I see no point in changing this to `To change its port you should [...]` and creating another refsection which is just about the same configuration directive.


But i would change variable name to locales instead of locale. Cuz its array of locales. But i dunno :D


```js
$scope.createTag = function(newTag) {
  User.user.opts.addTag({body:{name:$scope._newTag.name, id:Shared.uuid()}});
```


Заставить пользователя вводить оценку в инпут-бокс?!


this.getDOMNode was deprecated ?


Done :)


In dogm_lcd_implementation.h use volumetric_multiplier because this valor varius with filament width....


```js
input(type='text', ng-model='_newTag.name', placeholder=env.t('newTag'))
button(ng-click='createTag()')=env.t('add')
```


```js
button(ng-click='createTag()')=env.t('add')
```


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


Shouldn't this method be in some kind of `BaseRepository` class? Seems to me like you're going to repeat this method in every Repo


why do you need to manually update it ?


Actually, it seems possible in PyFxA to create new accounts and play with them: https://github.com/mozilla/PyFxA#testing-email-addresses


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.115](http://teamcity.monogame.net/viewLog.html?buildId=5952&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 469, ignored: 6 Build time: 00:15:16


looks like we'll have to update for the new uploader too!


После удаления лучше возвращать админа на главную, зачем ему график премьер?


Or use `fetch` - it more specify that you're gonna fetch something from API of some sort


2 rest in the same object what does it means ?


makes sense


Not worth spacing


Не надо копировать одинаковую форму для создания и редактирования.
Используй render partial


It is indeed a problem with MySQL, just checked an with PostgreSQL everything works like expected.


having the colon on a separate line will create a space between it and the word 'Revised', not a biggie but the colon should stay immediately after the closing php tag


@nouveller is this sufficient email encryption?


No idea how WP copes with this - it's an SVN post commit tag that's meant to get replaced per revision. Let's see what happens first


The only motivation there was the defaults provided with S3Connection (ie `is_secure` is default to True) but if it breaks STS then I'm okay with `connect_to_region`.


同样地，耗时的操作不应该阻塞 native 模块。


As far as my case, this won't solve the problem.

I have not have this problem before.
Today it was the first day and i won;t fix.
Any other possible solution?


I'm not sure about this. The problem is that snf-image-creator works in CentOS 6 which comes with python 2.6 and argparse was introduced in python 2.7


afaik using `getPage` is a recipe for disaster because it doesn't set (or allow you to set) any http connection/request timeout.

We've dealt with this before:

https://github.com/yadt/yadtshell-plugins/blob/master/src/main/python/yadtshell_plugins/rest.py


I think (and I could be wrong here) that `go get github.com/jayjanssen/myq-tools` is sufficient (assuming you are doing `go run myq_status.go` from inside that dir.


- please use camel case for entity and attribute names
- http://www.purl.org/ogit/AccessControl should be fine


:D


If you're using attr_accessor then use it also for a setter here: https://github.com/netguru/people/blob/98af0c6a59cc5eb3b245d8a61eebbd117e0f6174/app/services/trello/project_start_checker.rb#L34


Ok I can see it's changed later


I'm debating over whether or not I should just replace all those utility functions with raw XCB code instead, then the only library left to link to would be XCB itself. Don't know how many of them I can get rid of this way though, but the (until now) troublesome ones ICCCM and EWMH can be replaced easily.


Backwards compatibility. We should deprecate the method, though (forgot to do that myself).


The error was:
```
Unexpected error: couldn't get version/kind: invalid character 'a' looking for beginning of value
```
It didn't matter what the first character was in the file, the error used to say 'P' until we merged the missing json def for policy.  The file looked like this: https://gist.github.com/pweil-/5acb7f6b7be023523088.

I will take a closer look this morning and also fix the unit test that was commented on.


Using the octal code means that regardless of the selected encoding in the IDE the code will work, I don't see a readability issue here that would be a good reason to include a non-ansi character as its UTF8 literal when this minor change makes it work on all platforms.


Missing space between `expect` and `{`  and between `change` and `{`}


I imagine this would be fine, though I'd put in a separate JS file. Therefore you wouldn't need to worry about bots scraping the raw JS in your HTML and possibly piecing your email together.


> Its value MUST be `1`.

Its value MUST be `1` in this case.


Please squash with other README commit.


Divide this into two seperate specs


Remove space


Tell me if I wrong, but rows 64 and 71 dublication each other.


okay.


Hmm.  The need for this suggests that the test case coverage is inadequate?


Dublicating 64-th row


There is a more complete fix for this on tickets/HSC-1196 at Mitaka.


8)
On Пн, 30 марта 2015 г. at 6:32 Evgeny Safronov <notifications@github.com>
wrote:

> OMG
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cocaine/cocaine-core/commit/3ed32742c287199e2fa47c5155e81215536b04ba#commitcomment-10464781>
> .
>


Thank you for bringing this to our attention. We looked into it and it appears that actually Acrobat's behavior is a bit more complex than just interpreting the numbers with several minus signs as negative ones. It is true in case of real numbers, but in case of integers something probably goes wrong in Acrobat's logic and the number is interpreted as zero. So, for example, "--40." is interpreted as -40.0, but "--40" - as 0.
Thus we've created a fix that mimics Acrobat's behavior, but it's a bit more complex than the one that is proposed in this pull request.


Can't it be just `User` ?


FWIW, I pushed this and a bunch of other small readability fixes out as a new rendered PDF: https://github.com/tartley/python-regex-cheatsheet/releases/tag/v0.3.3


I think you are right.. I tried to use this function: http://docs.scipy.org/doc/scipy-dev/reference/generated/scipy.ndimage.interpolation.map_coordinates.html  instead of interpn.  For me it was easier to imagine.


oh, crap.  Yeah, I commented it out to make sure it wasn't being used and meant to remove it after I ran the tests.  Will fix


Actually, the second half of that error message is probably not justified. I'd assume there was _no_ stderr, socket or otherwise. I'm guessing all stdout/stderr output will just be silently dropped...


`expose_decorated(:users, decorator: AvailableUserDecorator)`?


Please add a new line


No need to do this just include the recipe as it will take care of what it needs to do in windows and linux.


运行时环境下立刻能访问 -> 运行时能直接访问

，这通常非常有用 -> 很有用


yeah, the biggest advantage would be to make sure the user doesn't think he's using DAV while it's disabled


I am used to capitalize the names of controllers - as well as not spelling out `Controller` completely, just because I am lazy and will have at least 6 typos, while typing `ncontroslller`


Why is there a space?


> e["#{type}_difficulty"] != 0

難易度がwikiに表記されていない時に0になります。今回のケースでは、難易度の情報が欠落していることはなかったので、大抵の場合は譜面が存在しない時です。(例:トリコンDPA)

> e["#{type}_notes"] == 0

ノート数がwikiに表記されていない時に0になります。新曲のノート数は情報不足のようで、空欄になっていた箇所が複数ありました。現在はTexTageを参考に補完してあります。

どちらも、あまりstrictな実装ではないですが、開発者向けのツールなのでいいかな、と思ってます。
ただ、0ではなくnullを使ったほうが良いかもしれないことに気づいたので、多分修正します。


Good addition


I think the name of the controller should be capitalized when you define it since it's a constructor function, but in the template it's an instance


[CN][BSS]などのマーカーが入っていた時に、帰ってくる値が0だったような、そんな記憶があります。


Please use

```ruby
if @own_address
```


Thank you. Great example!


Looks good to me.  There was one thing I noticed, which was you weren't specifying `stderr=subprocess.PIPE` like the other calls - but I think that they're wrong, because we're creating a pipe but then doing nothing with it.  That could lead to the child process blocking if it filled the pipe.

I'll file a separate issue.


> Not sure if this is a bug or a feature

It's a feature :) https://github.com/smarter/dotty/commit/7d43180e834aa2a576af360442e3cc0e8cd649df#diff-06714312372903b381147b591016b0cfR39

> you could future proof this code by guarding on sel.symbol.isSynthetic

@DarkDimius suggested adding a run test which should be enough, but that'll have to wait until we have run tests in Dotty.


I plan to another update index.html.


Add control spec for New Bright Baja Extreme


Rename this file to agent_windows.rb


okay man that is good of me learning to use the git method


You've committed a merge conflict, that's what these <<<  and >>>> things are. This file won't work without you fixing it.


Rename this file to linux_agent.rb


The HeatRecoveryTest runs clean at home and at here at work.
![hrtest](https://cloud.githubusercontent.com/assets/5778156/6896627/52fdb3ce-d6bb-11e4-9dd0-2528bf80a1ab.jpg)


```
s/to/for
```


so that would be

```javascript
app.controller('ZoomSliderCtrl', ['$scope', function(scope) {
  // [...]
});
```

and in a template:

```html
<div data-ng-controller="ZoomSliderCtrl as zoomSliderCtrl">

</div>
```


Python-argparse is also available for Python 2.6:
https://pypi.python.org/pypi/argparse

And it is even packaged as an RPM.
In fact, it seems even GRNET packages a python-argparse RPM?

From http://rpm.pbone.net/index.php3/stat/3/srodzaj/1/search/python-argparse:
"ftp5.gwdg.de/pub/opensuse/repositories/home:/GRNET:/synnefo/CentOS_CentOS-6/noarch/python-argparse-1.2.1-5.1.noarch.rpm"


This makes it impossible to monitor a running system.


Use repo


Add new line


And use repo


Etc etc use projectrepo :)


Why do you use semicolon here?


It is going to be a major update btw.


What does it mean?


If you're using attr_accessor then use the setter method already. Otherwise use attr_reader


It's a bit dangerous to bind mount all of `/dev` - at least one issue I've hit is around `/dev/pts` - if you run a container interactively, the child container ends up overwriting the pty permissions on the host, which leads to really weird issues.  It might only be `/dev/pts` that has this bug though.

There is a patch in progress for Docker to have `--dev=host` I think.  In the meantime though, can we identify the devices that this container needs?  Or is it something difficult like "all block devices"?


fetch cards


Change the description.


This should be added via composer. I'm fixing this.


Uh yeah, that would have been a better description; it all disappears.
Interestingly though, piping still seems to work as input to a GUI app, which means std::cin must be there...?


I agree on the separation going forward.  What I did here was to rename `check_rebuild_first_syllable` to `rebuild_characters`.  I made this new method call `gregorio_rebuild_characters` and used it instead of `gregorio_rebuild_characters` throughout the code (three places).


Fixes #315


To be clear, that commit was already cherry-picked: http://review.cyanogenmod.org/#/c/92643/


re-running some travis-ci tests recently with this commit reveals that it introduced issues :
- https://github.com/kares/jruby/compare/test-pre-2581-fix...test-2581-fix (maybe due to snapshot updates)

these 2 branches are essentially the same except for the commit :
- https://travis-ci.org/kares/jruby/builds/56385102
- https://travis-ci.org/kares/jruby/builds/56385143

... most of the failures are not on latest **jruby-1_7** but at least [one encoding](https://travis-ci.org/jruby/jruby/jobs/56343844#L1262-L1267) regression spec seems to persist ;(


yeah. I would go with ctrl or vm for a view with one controller and with more specific naming in case of transcluded templates (e.g. tabCtrl / tabController + paneCtrl / paneController)


Looks fine unless the users decide to go back to the table/db/server import pages and submit the file from there. That way the parameters passed in the link will be lost and the upload will be considered as a fresh import.


I guess you're right - must admit I haven't thought about this too hard until now...


`diff.music_name`と`diff.type`は、「変更を適用する曲データを特定するための」もので、addedは変更対象になる曲がそもそも存在しないので、現在の仕様の方が自然かな、と思っています。
(それとも、`diff.original_music_name`, `diff.original_type`にすると伝わりやすいでしょうか？)


this third line was just for test and I forgot to delete it...


Same 2 issues


Hmm, are we trying to load e.g. `scsi_transport_iscsi.ko` from the container onto the host?  At the moment we don't have the infrastructure to ensure the container's kernel modules match the host.  We may need a more general infrastructure for this, but the current hack may need to be running `chroot /host insmod scsi_transport_iscsi.ko` or whichever is needed.


Use interpolation. It's faster than `+`


I'd rather add this to ```ResPool``` (on top of sat::Pool).


Looks good. Thomas, could you please merge this into the master? Thanks!


Nexus sensor added


啊我是说这个history...


Fixed in latest commit and push on master


Use `.map { |m| m.role.name }` to iterate once, not twice over same array


@gitster Yeah, I had nightmares of INTERCAL while writing it. I'm not sure how duplicate `redirect-from`s are handled. I don't actually have Jekyll running on my machine, which makes experimenting a real pain. Certainly it's not an ideal system for version-control, as you will not even get a conflict when merging two topics with competing redirects. :(

I think the main motivation for these `redirect_from`s is not what we are doing here, but rather giving "permalink" URLs for individual posts. So we are somewhat abusing the mechanism.

I think in the long run it would make more sense to have an index at the `rev_news/` root rather than a redirect, anyway, so I wasn't inclined to dig much further into it.


@douglascamata da onde saiu esse `zero_interest`?

tentei achar aqui nessa doc mas nada :confused:

https://www.userede.com.br/pt-BR/produtosservicos/Documents/Ecommerce/05_Manual_E-Guia_do_Desenvolvedor_eRede_140602.pdf


Thanks for pointing this out!  I've opened PR #7515 to fix.


But this is not due to my change, right?


README 文件请用 MARKDOWN格式书写


EasyAlign愛用してます。その部分まで気配りできませんでしたスミマセン。

ただ、これをきっかけに、複数文字を整列対象に入れる方法を調べたので、その点は感謝します。


1: M111 unrecognized component name: docs


Yeah, I figured this out through the next assignment.  The directions of the assignment were too vague for me.


1: M100 needs more reviewers
1: M101 signature is missing
1: M110 missing component name


46: D102 Missing docstring in public method
49:5: F999 redefinition of unused 'visit' from line 45
50: D102 Missing docstring in public method
53:5: F999 redefinition of unused 'visit' from line 49
54: D102 Missing docstring in public method
57:5: F999 redefinition of unused 'visit' from line 53
58: D102 Missing docstring in public method
65:5: F999 redefinition of unused 'visit' from line 57
66: D102 Missing docstring in public method
69:5: F999 redefinition of unused 'visit' from line 65
70: D102 Missing docstring in public method
74:5: F999 redefinition of unused 'visit' from line 69
75: D102 Missing docstring in public method
78:5: F999 redefinition of unused 'visit' from line 74
79: D102 Missing docstring in public method
82:5: F999 redefinition of unused 'visit' from line 78
83: D102 Missing docstring in public method
86:5: F999 redefinition of unused 'visit' from line 82
87: D102 Missing docstring in public method
90:5: F999 redefinition of unused 'visit' from line 86
91: D102 Missing docstring in public method
94:5: F999 redefinition of unused 'visit' from line 90
95: D102 Missing docstring in public method
98:5: F999 redefinition of unused 'visit' from line 94
99: D102 Missing docstring in public method


24:1: I100 Imports statements are in the wrong order. from intbitset should be before import numpy
25:1: I201 Missing newline before sections or imports.
27:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from six


「どのフィールドが変更されたか」という意味でしょうか？

そうでしたら、@f5355c103b762c37c1f3995d2b00148adcefccab でメッセージを表示するようにしました。


22:1: F401 'search_unit' imported but unused


Could not evaluate contents of __all__. That means pep257 cannot decide which definitions are public. Variable __all__ should be present at most once in each file, in form `__all__ = ('a_public_function', 'APublicClass', ...)`. More info on __all__: http://stackoverflow.com/q/44834/.


409:9: E731 do not assign a lambda expression, use a def
415:13: E731 do not assign a lambda expression, use a def
419:17: E731 do not assign a lambda expression, use a def
421:17: E731 do not assign a lambda expression, use a def
470:9: E731 do not assign a lambda expression, use a def
476:13: E731 do not assign a lambda expression, use a def
480:17: E731 do not assign a lambda expression, use a def
482:17: E731 do not assign a lambda expression, use a def


Why was widescreen support removed and this pull request accepted?


Hey, what i am trying to do is to interface dataflow decoders (components) in to ffmpeg library. The main reason for doing so is to replace the source and display components with ffmpeg's infrastructures and also introduce the concept of dataflow to a wider audience.


seems to work not correctly, as still the screenshots remain in temp dir


In fact we found dropping singleFrameTime to 15 to make GUI performance much worse - it was causing Animations run in slow motion. On the other hand increasing it to say 60-100ms seem to help - the GUI did not become more fluid but intermittent animation slow downs at 40 no longer occur. Not sure that makes sense though looking at the code...


Holy shit how did you tap into the security feed in my house?!?!


For now, I prefer to check host type since I don't do it in all the tests - just a few places.


result should be ` "\"\"aaa\"\""`


Incorrect fix!

Better would be:

$file_extension = strtoupper(pathinfo($file_path, PATHINFO_EXTENSION));

        // file exists?
        if (!file_exists($file_path)) {

            throw new \Exception('File '.$file_path.' doesn\'t exist', SimpleExcelException::FILE_NOT_FOUND);

        // extension valid?
        } elseif ($file_extension != strtoupper($this->file_extension)){

            throw new \Exception('File extension '.$file_extension.' doesn\'t match with '.$this->file_extension, SimpleExcelException::FILE_EXTENSION_MISMATCH);


В таком случае я заведу отдельный метод returnPriority и в случае если надо преобразовать OperationsLabels в int буду использовать его ( соответственно отдельная функция будет в другую сторону)... Проблема кастов так решится, а вот с назначением конкретных чисел енуму... У меня всё равно получится что приоритеты это некие конкретные числа - так их проще сравнивать. Это подойдёт?


The examples section later still uses solr-location. Is there a specific reason for this or just an oversight?


Is it possible? Maybe it would be better to make parameter always not null?


It turns out that I was just pushing the commits not deploying the app.


Hahaha, the kind of syntax error that breaks everything


Is it correct? Are you checked CSV specification?


Use `delete` to free memory for objects allocated with `new` (also it'll call destructors).


Thank you very much for this fix!!


If it's easy and doesn't clutter things, why not? :wink:


missing javadoc :)


Thanks Denis. Didn't check what you did exactly but good to have someone do some cleanup :)


Thanks for this. I am pushing out now, but not bothering to npm publish, because that would mean a new version number.


not unknown, but not found...


You gui issue should be fixed by the two top patches. The condition you removed, namley m_bPresentFrame caused that issue, which had the inverted effect of what you wanted to achieve.


isSameAs, we want strict reference not object equality.


Regarding the naming of the variable: did we drop the rule to call config variables CFG_<module_name>_* ? If we use ES for another module, like the search engine, there might be some confusion regarding the naming.


One of these days, I will turn it all into an angular directive and then a Web component that works with Polymer or X-Tags...


Because I have replaced Karma with Tape?


@serby this look ok to you? If so I'll version and publish.


Was the plan to replace FactoryGirl with something else?


Which Plone version os compatible with p.a.widgets 2.0.0? (I thought that Plone 5 will work without p.a.widgets and only 1.x would work with Plone 4.3.x.

Puzzled :)


>  I would go with ctrl or vm for a view with one controller and with more specific naming in case of transcluded templates

agree with @0xF013 .. would rather keep naming short + simple, except in cases of ambiguity.


Where is this used? It is not considered API, so ok to change...


<img src="http://i2.kym-cdn.com/photos/images/facebook/000/138/246/tumblr_lltzgnHi5F1qzib3wo1_400.jpg" width="150">


Might as well use a single add_cmd_output for the three commands (it supports lists since 3.2).


Ah, I see now.


@teacheRforV --out 没起作用，生成正确名字的文件才能做后面的分析，否则没法调用啊~


Yes, things might break. But like I said `appcast` isn’t yet functional and having a bunch of these to test on is good, to find the limitations. Keep in mind `appcast`s aren’t user-facing: they will be used to automate cask updates, so breakages should be caught early on.

I do agree with your points, but you’re commenting on a single one of the 160 casks changed at the same time. This is all in git. It’s safe. Let’s see what we can make work and revert what we can’t. It’ll be quite a while before `appcast`s are working.


@axtimwalde

> trakem2/TrakEM2@5584415 and axtimwalde/mpicbg@9d434ac

Sadly these did not actually release successfully. You should either [subscribe to imagej-builds](http://imagej.net/mailman/listinfo/imagej-builds) or we can add `trakem2@imagej.net` to the reporting list for the TrakEM2 builds so you can be immediately notified of these failures. Let me know your preference.

Both attempts([1](http://jenkins.imagej.net/job/TrakEM2/74/), [2](http://jenkins.imagej.net/job/TrakEM2/76/)) to re-release pom-trakem2 1.3.2 failed because of the sequence of events I described above. The changes in git ```have not``` been deployed to Maven, so nothing is able to consume your re-releases (e.g., [deployed mpicbg-1.0.1](http://maven.imagej.net/index.html#nexus-search;quick~mpicbg) is still from Mar 14)

```
Rolling up the PATCH number of mpicbg to correct the pom would have required me to make releases of ALL dependents which took me a day for the 1.0.1-release which I am really really hesitant to do. I understand that this is against the reproducible build rule which I regret, but the alternative is to not fix.
```
It's not immediately clear to me what the state of each dependency is between release and re-release, but if you did the re-releases just to add bug fixes and updated dependencies, the simplest solution is likely to just bump the PATCH version on all affected components and do another release and remove all uses of pom-trakem2.

I'm also happy to chat (here, e-mail, skype, whatever you prefer) about how to make the release process easier for your software so that doing PATCH re-releases isn't so burdensome and time consuming.


bwaahahaha, commit messages with emojis, oh lawrd!


In my bashrc, I have an entry that looks a little like this:
```bash
...

# enable programmable completion features (you don't need to enable
# this, if it's already enabled in /etc/bash.bashrc and /etc/profile
# sources /etc/bash.bashrc).
if [ -f /etc/bash_completion ] && ! shopt -oq posix; then
    . /etc/bash_completion
fi

# If git isn't installed yet... these may fail
export GIT_PS1_SHOWDIRTYSTATE=1
export GIT_PS1_SHOWSTASHSTATE=1
export GIT_PS1_SHOWUNTRACKEDFILES=1

set_bash_prompt()
{
    if [ "$color_prompt" = yes ]; then
        NC='\[\e[0m\]'
        C11='\[\e[1;32m\]'
        C13='\[\e[1;34m\]'
        export GIT_PS1_SHOWCOLORHINTS=1
        PS1="${debian_chroot:+($debian_chroot)}$C11\u@\h$NC:$C13\w$NC$(__git_ps1 ':(%s)')$NC\$ "
    else
        PS1='${debian_chroot:+($debian_chroot)}\u@\h:\w$(__git_ps1 ":(%s)")\$ '
    fi
}

# This tells bash to reinterpret PS1 after every command, which we need because
# __git_ps1 could be outputting color codes
PROMPT_COMMAND=set_bash_prompt

...
```


Yes, use ToJSON

----- Original Message -----
> The error was:
> ```
> Unexpected error: couldn't get version/kind: invalid character 'a' looking
> for beginning of value
> ```
> It didn't matter what the first character was in the file, the error used to
> say 'P' until we merged the missing json def for policy.  The file looked
> like this: https://gist.github.com/pweil-/5acb7f6b7be023523088.
>
> I will take a closer look this morning and also fix the unit test that was
> commented on.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/pweil-/origin/commit/bc3e1da9dcab6331442734ed390eec9add588ccf#commitcomment-10466977


failed.


Use `delete` to free objects allocated with `new`.


Good stuff, as always. Is there a way to get in touch with you? :)


this should fix errors that come from #5210
+1 for this.


This is just a formatting change. Commit hashes are displayed with an extra set of paren's on either side, however I have already included a set of paren's in my .bashrc. So I removed this specific set here, to prevent two sets of paren's from showing up.


I didn't find any usage of it but I couldn't determine with 100% certainty that it wasn't API so I thought better safe than sorry since it's 3.8.


> Could you tell me how it works in Log Dialog?

1. right click a file in file list of Log dialog
2. click "Mark to compare"
3. go another commit and right click any file in file list, got one more context menu item "Diff xxx", click it.


That's why I specifically added 'click here', but I guess still some percentage of users would miss that.


Double-deleting child nodes? Also it's useful to nullify deleted pointers.


Why do we need `final` here? Wouldn't it be better to allow extending classes to override, or at least provide a hook by which additional initializations can be done?


I like the direction this takes for localization of terms.

We should try to make the definitions broadly applicable. For example, use of the term "certified" in  "A certified person directly involved with student instruction," sounds like it fits into U.S. primary-secondary education, but might not be appropriate for early learning, post-secondary, or adult education. I think we could use a definition for educator that applies across contexts and levels of qualification. The nuance of a qualifications, e.g. certified teacher vs. paraprofessional, are important for other kinds of education accountability data, but not for learning resource audience role.


@jkr

>Note that this corruption only occurs if we rename it as reference.docx and put it in the pandoc dir. If we give it as a command line option, it's fine (because then it only uses styles, and leaves out settings.xml).

I realized that I had overlooked this (wasn't aware of the difference).
I can confirm the corruption with 1.13.2 following the steps you outlined (I said I couldn't reproduce it earlier).
Of course it works fine with after @lierdakil's fix.


Ok to keep them removed. But for you knowledge you can catch multiple error like this:

except (AssertionError, RuntimeError) as e:


first test


Blank line?


Seems a bit short


or '23 sponsor packages purchased'


1: D400 First line should end with a period (not 'e')
28:1: I101 Imported names are in the wrong order. Should be task_sleep_now_if_required, task_update_progress, write_message
30:1: I101 Imported names are in the wrong order. Should be Error, deserialize_via_marshal, run_sql, serialize_via_marshal
34:1: I100 Imports statements are in the wrong order. import invenio.legacy.template should be before from invenio.utils.date
41:44: E251 unexpected spaces around keyword / parameter equals
41:46: E251 unexpected spaces around keyword / parameter equals
41: D401 First line should be in imperative mood ('Return', not 'Returns')
41: D209 Multi-line docstring closing quotes should be on a separate line
41: D205 1 blank line required between summary line and description (found 0)
52:75: E502 the backslash is redundant between brackets
53:65: E502 the backslash is redundant between brackets
54:28: E225 missing whitespace around operator
65:78: E502 the backslash is redundant between brackets
66:83: E502 the backslash is redundant between brackets
67:32: E225 missing whitespace around operator
80:44: E261 at least two spaces before inline comment
80:44: E262 inline comment should start with '# '
80: D401 First line should be in imperative mood ('Return', not 'Returns')
80: D400 First line should end with a period (not 'm')
86:30: E261 at least two spaces before inline comment
95: D401 First line should be in imperative mood ('Return', not 'Returns')
95: D209 Multi-line docstring closing quotes should be on a separate line
95: D400 First line should end with a period (not 'n')
95: D205 1 blank line required between summary line and description (found 0)
104: D401 First line should be in imperative mood ('Find', not 'Finds')
104: D400 First line should end with a period (not 'g')
104: D300 Use """triple double quotes""" (found '''-quotes)
113:66: E502 the backslash is redundant between brackets
114:46: E225 missing whitespace around operator
132:8: E114 indentation is not a multiple of four (comment)
146: D209 Multi-line docstring closing quotes should be on a separate line
146: D400 First line should end with a period (not 'y')
146: D205 1 blank line required between summary line and description (found 0)
146: D300 Use """triple double quotes""" (found '''-quotes)
157:68: E502 the backslash is redundant between brackets
158:24: E225 missing whitespace around operator
169: D401 First line should be in imperative mood ('Return', not 'Returns')
169: D400 First line should end with a period (not 'd')
169: D300 Use """triple double quotes""" (found '''-quotes)
177:91: E502 the backslash is redundant between brackets
178:28: E225 missing whitespace around operator
181:72: E502 the backslash is redundant between brackets
182:60: E502 the backslash is redundant between brackets
183:24: E225 missing whitespace around operator
187: D209 Multi-line docstring closing quotes should be on a separate line
187: D400 First line should end with a period (not 'y')
187: D205 1 blank line required between summary line and description (found 0)
187: D300 Use """triple double quotes""" (found '''-quotes)
192:73: E225 missing whitespace around operator
193:96: E502 the backslash is redundant between brackets
193:56: E231 missing whitespace after ','
193:76: E231 missing whitespace after ','
194:56: E231 missing whitespace after ','
199: D401 First line should be in imperative mood ('Return', not 'Returns')
199: D205 1 blank line required between summary line and description (found 0)
209:63: E502 the backslash is redundant between brackets
210:28: E225 missing whitespace around operator
212:76: E502 the backslash is redundant between brackets
212:70: E225 missing whitespace around operator
219:58: E502 the backslash is redundant between brackets
220:28: E225 missing whitespace around operator
222:65: E225 missing whitespace around operator
233:80: E502 the backslash is redundant between brackets
234:24: E225 missing whitespace around operator
238: D400 First line should end with a period (not 'n')
238: D300 Use """triple double quotes""" (found '''-quotes)
244:34: E261 at least two spaces before inline comment
244:34: E262 inline comment should start with '# '
247:50: E225 missing whitespace around operator
253:60: E502 the backslash is redundant between brackets
254:24: E225 missing whitespace around operator
256:1: E302 expected 2 blank lines, found 1
256: D401 First line should be in imperative mood ('Identifie', not 'Identifies')
256: D400 First line should end with a period (not 't')
263:1: E302 expected 2 blank lines, found 1
263: D401 First line should be in imperative mood ('Doe', not 'Does')
263: D209 Multi-line docstring closing quotes should be on a separate line
263: D400 First line should end with a period (not 'e')
263: D205 1 blank line required between summary line and description (found 0)
271:80: E502 the backslash is redundant between brackets
272:24: E225 missing whitespace around operator
278:17: E127 continuation line over-indented for visual indent
279:76: E502 the backslash is redundant between brackets
287:81: E502 the backslash is redundant between brackets
292:86: E502 the backslash is redundant between brackets
299: D400 First line should end with a period (not 'A')
311:17: E261 at least two spaces before inline comment
312:65: E502 the backslash is redundant between brackets
313:60: E225 missing whitespace around operator
320:43: E502 the backslash is redundant between brackets
321:13: E128 continuation line under-indented for visual indent
321:77: E502 the backslash is redundant between brackets
324:80: E502 the backslash is redundant between brackets
325:50: E225 missing whitespace around operator
327:75: E502 the backslash is redundant between brackets
332: D400 First line should end with a period (not 'S')
334:64: E502 the backslash is redundant between brackets
335:50: E502 the backslash is redundant between brackets
336:20: E225 missing whitespace around operator
337:60: E225 missing whitespace around operator
342:119: E502 the backslash is redundant between brackets
345:17: E261 at least two spaces before inline comment
349:62: E502 the backslash is redundant between brackets
354:43: E502 the backslash is redundant between brackets
355:13: E128 continuation line under-indented for visual indent
357:81: E502 the backslash is redundant between brackets
358:58: E225 missing whitespace around operator
360:57: E502 the backslash is redundant between brackets
361:50: E225 missing whitespace around operator
365: D209 Multi-line docstring closing quotes should be on a separate line
365: D205 1 blank line required between summary line and description (found 0)
369:66: E502 the backslash is redundant between brackets
370:20: E225 missing whitespace around operator
378:42: E225 missing whitespace around operator
382:38: E225 missing whitespace around operator
387: D209 Multi-line docstring closing quotes should be on a separate line
387: D400 First line should end with a period (not 's')
387: D205 1 blank line required between summary line and description (found 0)
392:54: E502 the backslash is redundant between brackets
407:66: F821 undefined name 'six'
427: D401 First line should be in imperative mood ('Return', not 'Returns')
427: D209 Multi-line docstring closing quotes should be on a separate line
427: D202 No blank lines allowed after function docstring (found 1)
427: D400 First line should end with a period (not 'r')
427: D205 1 blank line required between summary line and description (found 0)
433:24: E261 at least two spaces before inline comment
433:25: E262 inline comment should start with '# '
442:73: E502 the backslash is redundant between brackets
443:51: E225 missing whitespace around operator
452:63: E502 the backslash is redundant between brackets
453:49: E225 missing whitespace around operator
458: D401 First line should be in imperative mood ('Return', not 'Returns')
458: D400 First line should end with a period (not 's')
460:53: E502 the backslash is redundant between brackets
464: D401 First line should be in imperative mood ('Return', not 'Returns')
464: D209 Multi-line docstring closing quotes should be on a separate line
464: D400 First line should end with a period (not ':')
464: D205 1 blank line required between summary line and description (found 0)
473:13: F841 local variable 'dummy' is assigned to but never used
478:27: E261 at least two spaces before inline comment
478:28: E262 inline comment should start with '# '
483: D401 First line should be in imperative mood ('Return', not 'Returns')
483: D209 Multi-line docstring closing quotes should be on a separate line
483: D400 First line should end with a period (not ',')
483: D205 1 blank line required between summary line and description (found 0)
495:75: E502 the backslash is redundant between brackets
496:23: E128 continuation line under-indented for visual indent
496:24: E225 missing whitespace around operator
510:72: E502 the backslash is redundant between brackets
511:69: E225 missing whitespace around operator
513:83: E502 the backslash is redundant between brackets
514:23: E128 continuation line under-indented for visual indent
514:24: E225 missing whitespace around operator
521: D401 First line should be in imperative mood ('Thi', not 'This')
521: D209 Multi-line docstring closing quotes should be on a separate line
521: D400 First line should end with a period (not 'd')
521: D205 1 blank line required between summary line and description (found 0)
528:5: B901 blind except: statement
532:1: E302 expected 2 blank lines, found 1
532: D401 First line should be in imperative mood ('Thi', not 'This')
532: D209 Multi-line docstring closing quotes should be on a separate line
532: D400 First line should end with a period (not 'd')
532: D205 1 blank line required between summary line and description (found 0)
547:20: E261 at least two spaces before inline comment
551:1: E302 expected 2 blank lines, found 1
551: D401 First line should be in imperative mood ('Thi', not 'This')
551: D209 Multi-line docstring closing quotes should be on a separate line
551: D400 First line should end with a period (not 'e')
551: D205 1 blank line required between summary line and description (found 0)
561:1: E302 expected 2 blank lines, found 1
561:59: E251 unexpected spaces around keyword / parameter equals
561:61: E251 unexpected spaces around keyword / parameter equals
561: D401 First line should be in imperative mood ('Update', not 'Updates')
561: D209 Multi-line docstring closing quotes should be on a separate line
561: D202 No blank lines allowed after function docstring (found 1)
561: D400 First line should end with a period (not 'd')
561: D205 1 blank line required between summary line and description (found 0)
572:69: E502 the backslash is redundant between brackets
573:46: E225 missing whitespace around operator
582:75: E502 the backslash is redundant between brackets
583:24: E225 missing whitespace around operator
584:21: E261 at least two spaces before inline comment
584:22: E262 inline comment should start with '# '
588:83: E502 the backslash is redundant between brackets
589:24: E225 missing whitespace around operator
603:14: E261 at least two spaces before inline comment
617:60: E502 the backslash is redundant between brackets
618:28: E225 missing whitespace around operator
621:69: E502 the backslash is redundant between brackets
622:33: E128 continuation line under-indented for visual indent
624:60: E502 the backslash is redundant between brackets
625:28: E225 missing whitespace around operator
627:69: E502 the backslash is redundant between brackets
628:33: E128 continuation line under-indented for visual indent
630:59: E502 the backslash is redundant between brackets
631:28: E225 missing whitespace around operator
641:68: E502 the backslash is redundant between brackets
642:42: E128 continuation line under-indented for visual indent
650:85: E502 the backslash is redundant between brackets
651:28: E225 missing whitespace around operator
656:117: E251 unexpected spaces around keyword / parameter equals
656:119: E251 unexpected spaces around keyword / parameter equals
656: D401 First line should be in imperative mood ('Update', not 'Updates')
656: D400 First line should end with a period (not 's')
660:62: E225 missing whitespace around operator
665:64: E502 the backslash is redundant between brackets
666:24: E225 missing whitespace around operator
687:17: E125 continuation line with same indent as next logical line
703:72: E502 the backslash is redundant between brackets
715:66: E502 the backslash is redundant between brackets
716:21: E128 continuation line under-indented for visual indent
720:66: E502 the backslash is redundant between brackets
721:21: E128 continuation line under-indented for visual indent
722:64: E225 missing whitespace around operator
725: D401 First line should be in imperative mood ('Modifie', not 'Modifies')
725: D209 Multi-line docstring closing quotes should be on a separate line
725: D400 First line should end with a period (not 'n')
725: D205 1 blank line required between summary line and description (found 0)
734: D401 First line should be in imperative mood ('Insert', not 'Inserts')
734: D400 First line should end with a period (not 's')
743:50: E261 at least two spaces before inline comment
743:50: E262 inline comment should start with '# '
749:34: E261 at least two spaces before inline comment
749:35: E262 inline comment should start with '# '
756:23: E261 at least two spaces before inline comment
756:24: E262 inline comment should start with '# '
763:38: E225 missing whitespace around operator
767: D400 First line should end with a period (not 's')
775:38: E225 missing whitespace around operator
779: D209 Multi-line docstring closing quotes should be on a separate line
779: D400 First line should end with a period (not 'w')
779: D205 1 blank line required between summary line and description (found 0)
789: D400 First line should end with a period (not ')')
806: D401 First line should be in imperative mood ('Update', not 'Updates')
806: D209 Multi-line docstring closing quotes should be on a separate line
806: D202 No blank lines allowed after function docstring (found 1)
806: D400 First line should end with a period (not 't')
806: D205 1 blank line required between summary line and description (found 0)
816:66: E502 the backslash is redundant between brackets
817:59: E502 the backslash is redundant between brackets
818:20: E225 missing whitespace around operator
844:27: E261 at least two spaces before inline comment
851:61: E225 missing whitespace around operator
852:71: E225 missing whitespace around operator
855:82: E502 the backslash is redundant between brackets
856:28: E225 missing whitespace around operator
861:32: E261 at least two spaces before inline comment
865:28: E261 at least two spaces before inline comment
865:29: E262 inline comment should start with '# '
868:82: E502 the backslash is redundant between brackets
869:27: E128 continuation line under-indented for visual indent
871:84: E502 the backslash is redundant between brackets
872:19: E128 continuation line under-indented for visual indent
872:20: E225 missing whitespace around operator
877:56: E225 missing whitespace around operator
878:58: E225 missing whitespace around operator
881:87: E502 the backslash is redundant between brackets
882:27: E128 continuation line under-indented for visual indent
882:28: E225 missing whitespace around operator
887:38: E251 unexpected spaces around keyword / parameter equals
887:40: E251 unexpected spaces around keyword / parameter equals
887: D401 First line should be in imperative mood ('Rebalance', not 'Rebalances')
887: D400 First line should end with a period (not 't')
899:58: E225 missing whitespace around operator
901:23: E261 at least two spaces before inline comment
903:62: E502 the backslash is redundant between brackets
904:24: E225 missing whitespace around operator
910:56: E225 missing whitespace around operator
911:70: E502 the backslash is redundant between brackets
912:27: E127 continuation line over-indented for visual indent
912:28: E225 missing whitespace around operator
914:33: E128 continuation line under-indented for visual indent
915:33: E128 continuation line under-indented for visual indent
916:33: E128 continuation line under-indented for visual indent
918:72: E502 the backslash is redundant between brackets
919:28: E225 missing whitespace around operator
927: D401 First line should be in imperative mood ('test', not 'tests')
927: D400 First line should end with a period (not 's')


Nowadays, users are able to recognize a link when they see one.


rme: ctrl+shift+F
have fun replacing stuff manually
not everything is lost yet

@EPuncker I've never donated a single cent to otland and I agree with others. If he doesn't want people to use unofficial otb, he should have released official one earlier.


TBH I need to rename the repo. The package.json as the right details in it.

spotify-websocket-cli — A pure JS implementation of the Spotify protocol, that _does_ _not_ need the desktop GUI client installed. (which is good, as the MacOS variants seem to steal focus randomly, and the Linux client itself is balls, and hangs all of X ALL THE TIME!) I have no idea about windows. I hear you can run Javascript on it. Not sure how. ;-)

I wrote the player last night. Now to refactor my several hundred lines of JS into something usable and where every third line is not console.log —  And not hard code my own username and password in :)

Also, entire thing doesn’t use libspotify, at all, so doesn’t need an API SDK key. Just a spotify premium user account. :)

(entire thing came out of a kickstarter I am helping someone prototype, btw)

-Dx

> On 30 Mar 2015, at 13:09, Graeme West <notifications@github.com> wrote:
>
> iiiiiiiiiiiiinteresting +1: :)
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/Xalior/spotify-websocket-api/commit/8ae0f4a4cea8e5fcee542180756fb7eec31a8217#commitcomment-10466368>.
>


@rraustad See #4823  comments.  The tests apparently run differently on the CI systems.  You may be able to reproduce the failure by running just the HeatRecoveryTest.  You can do this with a command-line flag
`energyplus_tests --gtest_filter=HeatRecoveryTest`
The filter string can include wildcards like *
`energyplus_tests --help` shows more options.


all of this strings should be pulled out as constants.


Well, I know.
But I do not understand why parent N is taken. e.g. `abcdef^1`


I've changed it to use OpenOrCreate, but added a File.Exists call prior to it.  We still want to be able to delete the temporary file created, and we can't tell with OpenOrCreate whether it opened or created.  There's of course a race condition here with usage of File.Exists, but the negative consequences of that are very minor, and the chances of it happening are close to none.


http://www.iana.org/domains/root/db lists domains with longer TLDs


Thanks


@ludmilamarian you mean we should name the variable `CFG_WEBSTAT_...` instead of `CFG_...`?


im not sure where that came from i'll drop it in my next push


а в чем конкретно проблема? Чтобы не искать, добавь коммент к нужной строке в коммите


@kalashnikovisme лучше сделать скоуп Eveng.active, который будет по дате как current плюс future. Потому что тут ещё ты берёшь first(8) от коллекции декораторов - это в запрос к базе не входит


@marijnh thanks for fixing this btw ! really appreciated :)

I was thinking that this line could maybe also reflect the completionSingle option

```javascript
if (picked && data.list.length == 1 && completion.cm.options.completeSingle
) completion.pick(data, 0);
```


Add hi-score tracking (display the highscore somewhere in the menu/title and it should persist even when the game is restarted)


Argh, it should only be considered if a merge took place. Will fix that. Thanks for pointing that out.


reset is unnecessary here since the object was just instantiated and the count should be already at 0.


test


kde5/kde-overlay/etc/skel/.config/plasma-org.kde.plasma.desktop-appletsrc problematic. I will reconfigure this file...


Don't like throwing a runtime exception in a logging function.  Just log, and return a null object.


Wow, that's more than I wanted you to change, thanks so much for also (kind of unrelated ;) introducing labels for the other strings!


12: M102 unrecognized bullet/signature
3: M190 line is too long (75 > 72)
6: M190 line is too long (79 > 72)
10: M190 line is too long (74 > 72)


вместо этой строки можно begin_date.to_date == end_date.to_date


I don't think that's relevant. The user has pressed enter. `completeSingle` is about not having to press enter to complete a single result.


Nice :)


> Keep in mind appcasts aren’t user-facing: they will be used to automate cask updates, so breakages should be caught early on.

Oh, I thought it was meant for something like "brew cask update" that would check the appcasts of one or all installed apps locally and would update them to the latest version. In that case that's a different story, especially if there will be someone doing sanity checks on this manually.


Thanks. You pay attention to wrong parts, such typos don't mean, don't fixes and don't improve anything ;)


oh wait you are right, thanks again !


I like where this is going. A few changes I would suggest.
- Add some global flags at the top to make it easy to configure the extra lines on/off (I like to use a one line status set to a really large font (like 6) so that I can easily see it from a distance)
- Add a global field so that the user can easily configure the FULL cutoff value.


hi franco,
actually - I think this is just a snafu from merging / remerging my branch back to justingardner/life - i was out of sync with the remote because of a missed `git fetch`. if you can ignore change to this file that would be good.
thanks / sorry, d


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


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


I think this would also be good to reliably solve issues concerning encoding, e. g. #662.


I agree that we need a proper configure system. Mathieu has implemented a CMAKE based system in the 'python' branch. I am not sure how difficult it would be to back port it to the master branch. Eventually the `python` should be merged to `master` so we could perhaps just wait for this.


"we got 94 individual donations and 23 sponsor packages were purchased"?
It sounds a bit weird to switch from active to passive in the same sentence, so I think I prefer
"we got 94 individual packages and sold 23 sponsor packages" unless we can find a better sentence altogether (eg. using something else instead of the word "got"). wdyt @sareg0?


Yes.


If anyone wants to set up the site locally, btw, here's what I learned yesterday (ref #30) for getting it running on Ubuntu:

* Make sure you've got ruby2 (packages are available, have to replace `/usr/bin/ruby` and `/usr/bin/gem` with their v2 options)
* `sudo gem install bundler`
* Clone this repo
* If I haven't committed the Gemfile yet, create it with these lines:
```
source 'https://rubygems.org'
gem 'github-pages'
```
* `sudo apt-get install zlib1g-dev` # ref [1]
* `bundle install`
* `bundle exec jekyll serve`
* browse the site on http://localhost:4000

Based on https://help.github.com/articles/using-jekyll-with-pages/

I better copy the above into the repo's README at some point.

[1] http://www.nokogiri.org/tutorials/installing_nokogiri.html#ubuntu___debian


@ryanty0304 욕설은 자제해 주시기 바랍니다. 관심은 다른데에서도 충분히 받을 수 있어요.


thanks, didn't realise, oops :+1:
it's been updated already.


Looks good to me (verified all CCEs to be removed are mentioned elsewhere). ACK for this change.


This makes it much simpler!


De rien :)


please do not commit comments like this one


I searched through both coreclr and corefx... these were the remaining ones I could find.


Great! Thanks for the link.


ACK.


The API is just one (versioned with ```v1```, currently) please avoid ambiguity and use different term to describe what you add (url or functionality ...).


Need to find a way to prevent excessive zCulling of objects behind closest wall


17: M102 unrecognized bullet/signature
18: M102 unrecognized bullet/signature


Ah yes! I had to do some digging to find the skip functionality which works with the latest version of Specs2...


comment like this one are obsolete because we will not export a javadoc from our classes. please lets stick to the clean code principles.


I'll just leave the license headers to you guys then. I'm not touching Gradle with a 10-ft pole. Gradle fills me with a rage every time I want it to do something. I don't want to be in a rage. I just want to write code.


please remove


No, it does not work in normal case (not merge point).
It works if I run the 2 "else" clauses. (line 2079 and 2091)


Weird
===========
###### Actually, I don't really understand....

The doc I just commited (the .md opened by Mostafa with Word) does not present weird caracteristics when opened with atom or notepad++, but **DOES** look weird in Git. Tell me if its ok or not. In any case, we can convert it in Pdf on my pc and have a proper version of it.

###### More Generally

Guys... don't open the .md with word or anything (notepad might work, but...).
If you want, the more simple thing is to modify it (in order to put your signature for example) directly in Github Web. There is a button to modify a file you are looking at. Otherwise, open the .md file with atom, do the modification, save it and commit it.

###### All yours,

**Bruno**


Sounds pretty cool! I shall keep an eye on it. Hope you're well! G


@kalashnikovisme begin_date.to_s.match(/[0-9]{2}:[0-9]{2}/) -  begin_date.strftime("%H:%m")


This lifecycle method fires after the component received new props and/or state and re-rendered.

This component needs to know the width and height of its DOM Node, but there's no way to know it before having rendered it a first time. Thus, every time the component re-renders with potentially a different width and/or height, we keep track of that width and height to position it correctly.


Put it on github: https://github.com/HyperSuprime-Cam/afw/commit/2c2f1055a65cd29634ce52a4049bc664faff6ee4

It's awaiting review on the HSC side, and then the plan was to cherry-pick it to LSST.


This is equivalent to `style = _.assign({}, style, this._getStyle());`


哈哈，大神你终于提交了


How do we specify the bounds of the popup widget? Currently the widget that pops up is the default widget size.