- Лучше такие длинные строки разбивать для читаемости.
```Objective-C
[_employeeList setObject:[NSString stringWithFormat:@"%@, %@", _employee.lastName, _employee.firstName]
                  forKey:[NSString stringWithFormat:@"%d", [_uIDcounter intValue]]];
```
- Для более компактной записи можно использовать subscript:
```Objective-C
 _employeeList[@"keyName"] = obj;
```
- Нужно использовать литералы https://github.com/NYTimes/objective-c-style-guide#literals
- Желательно использовать `dot-notation` даже для "своих" свойств: https://github.com/NYTimes/objective-c-style-guide#dot-notation-syntax


TeamCity Jarvis Framework :: CI [Build 86](http://build.prxm.it:8811/viewLog.html?buildId=3199&buildTypeId=CqrsFramework_Ci) outcome was **SUCCESS**
Summary: Tests passed: 119 Build time: 00:01:22


Perex: Could you please change the following:

#: src/dvr/dvr_db.c:2166 src/dvr/dvr_autorec.c:1093 src/dvr/dvr_timerec.c:653
msgid "Creator"
msgstr "Kreator"

to:

#: src/dvr/dvr_db.c:2166 src/dvr/dvr_autorec.c:1093 src/dvr/dvr_timerec.c:653
msgid "Creator"
msgstr "Autor"

in polish translation, please?


Erhm. Plz roll this back.I've shortened that to KickAss so that the torrentdialog has more space in the header row. It' s just the setting in your config and possibly in settingsservice that needs to be adjusted.


@j0k3r en fait j'aimerais bien me passer de jQuery, mais je n'ai pas encore fait le nécessaire… :-/


Ha effectivement :)


Why is this necessary on 14.04?


Are you making this suggestion because 'type' is a shell builtin while 'which' is a separate binary?


> This will hopefully fix the jenkins build

And @DeepDiver1975 gets happy :)


I guess just `return not status['next_artefacts']` would also work, but maybe it's less clear


maybe
```python
pending_updates = status['next_artefacts']
return not pending_updates


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Deploy Update [Build 4](https://bsr.ict.je-labs.com/viewLog.html?buildId=29757&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_DeployUpdateWordpressCoreDirect) is now running


Sınıf yerine __CLASS__ kullanılsa daha iyi olabilir hatta static olarak __invoke sihirli metodu ile tanımlansa ve error handler olarak tanımlanması loader bi sınıf tarafından yapılsa daha kullanışlı olur bence


Kanskje dumt spørsmål, men må disse sjekkes inn i git egentlig?


Ah, you meant to still keep the explicit unlink, but also the while loop. Now I'm with you. Yeah, that's fine, I'll do that.


I like this change, but I'm thinking about whether we should also rename the plan description name to this `*ByRange` suffix style. What do you think?


I dunno. Didn't write the lib. It's this way on all the functions :]
Ah ok I looked: in Explorer_GetWindow it refills hwnd inline. See line 55 :+1:


**NOTE:**不能是简单的拼接，必须用`.`隔开。


### To do:

1) delete directory www and old
2) move content of directory pkg to root
3) "reverse engineer" directory man so that it is compatible with roxygen2

einar


Jeg mener, siden gulp pakker og minifiserer alt til en lib.js, så kan den vel også hente angualr fra nett i samme sleng?


不过不commit，会一直占用lock。


Well, I've runned check-style and there were no complaints. But I can change this, anyway.


事务开启。


Ok.


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


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 565](http://build.prxm.it:8811/viewLog.html?buildId=3201&buildTypeId=Jarvis_DocumentStore_CI) is now running


Elsewhere, it is expected that we return a new value (and not one by reference).


為了讓 user 可以客製化 error message，允許 `$this->currentUserCan` 可以回傳 boolean 或 [boolean success, string message] ?

譬如:

```php
{
     if (...) {
            return [false, "不是管理員"];
     }
     return [false, "不符合任何條件"];
}
```


Ack.


Done in transifex


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Deploy Update [Build 4](https://bsr.ict.je-labs.com/viewLog.html?buildId=29757&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_DeployUpdateWordpressCoreDirect) outcome was **SUCCESS**
Summary: Running Build time: 00:06:00


@onurozgurozkan @tayfunoziserikan page speed score cok dusuktu bootstrap js componentleri kaldirdim.


另外一種做法:

```php
function currentUserCan(...) {
        if (...) {
       return $this->allow("允許理由");
       }
       return $this->deny("拒絕存取");
}
```


This is a misnomer. Consider renaming this.


Have you seen https://www.nuget.org/packages/Topshelf.FSharp/?

The way you do a 'default' config is a nice way to start, that would make for a nice PR to the above.


Extra blank in ".  We ". I remember to have removed an extra blank from other comment I copied from fs_nir,
just out of curiosity, is it really a typo or it should be like that?.


Actually, I'm not sure if commits tree should looks like this. OOP start at initial commit. Shuoldn't it from the latest merge?


You could probably put the state in a `ref` cell and then return the ServiceControl. Start/Stop would then replace the contents of the ref cell.

Suave has a configuration option; the CancellationToken, which you should use to stop the server. It should be 'renewed' when Start called.


You can cast the resulting enum to int; done in the referenced nuget Topshelf.FSharp.


Consider using paket?


Looking good. If you fix the above mentioned things, I can move the sample to its own repo under Suave and give you access to it.


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


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


Rewriting conventional variables are often confusing.

Any reason you didn't change `os.Argd[0]` to the expanded path variable?


Saw the commit message, got curious, left dissapointed :(


done


@frenkield sounds good to me!

Probably if we find a better way to do this we can use it as well for the files (blob). Let me explain... When sending a file I am using a [similar mechanism for sending the filename](https://github.com/openworm/org.geppetto.frontend/blob/6ed6962608aadb5e0ddfc2ded2432b6fd3d9f02d/src/main/webapp/js/websocket-handlers/GEPPETTO.MessageSocket.js#L108-L111), i.e. the first byte, as discussed, allows as to distinguish between files and messages, the second byte is the file name length, then we send the filename and afterwards the file. I was googling a little bit in order to send this information in the header or in a more elegant way at least. I couldn't find anything and I implemented this ad-hoc solution but obviously there should be something better out there...

If you want to have a look at this and create a branch, I will keep an eye on it ( although I think you know much more than me about this :) Thank you!


:+1: :)


this ready for master, or were you wanting to test it yourself first?


@gidili  Have you forgotten to push this class FindDynamicVisulizationVariablesVisitor?


commit message has 81 characters.


Yes, it should. :) No worries...
```bash
git fetch
git checkout feature/oop-solution
git pull --rebase
git rebase origin/master
git push -f origin feature/oop-solution
```
Add me on skype, so we can make such things there


Can we have these two classes (ObjectProperties and PropertyMetaData) in separate files?


Nitpick: it should be PropertyMetadata


I think he did in 906847b6cc6241fbb23e944b6b11d59dcc035698


@pypingou `split_ref` is custom methods which looks for the files provided in the `filename` in the repository tree and returns the file object.


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 565](http://build.prxm.it:8811/viewLog.html?buildId=3201&buildTypeId=Jarvis_DocumentStore_CI) outcome was **SUCCESS**
Summary: Tests passed: 284 Build time: 00:13:00


TeamCity Jarvis Framework :: CI [Build 87](http://build.prxm.it:8811/viewLog.html?buildId=3202&buildTypeId=CqrsFramework_Ci) is now running


could you tell me why is it needed ???

this change should be reverted


Instantiating a QJEngine on that function is a big performance issue, since this function will be called many times and that object is very heavy. Could you move the QJSEngine as a member of  QQmlLevelDBReadStream?


Remove this, the `#join` will raise an exception if the Thread had one.


Well that's a bit rugh perhaps, i just changed the display name... but whatever suits you better


TeamCity Jarvis Framework :: CI [Build 87](http://build.prxm.it:8811/viewLog.html?buildId=3202&buildTypeId=CqrsFramework_Ci) outcome was **SUCCESS**
Summary: Tests passed: 119 Build time: 00:01:33


Good call. I severely screwed up the release branch.


can we import variables inside this script? otherwise I have no idea about how to use the `${EXECPREFIX}` variable


These lines are too long, but although we had move the NULL down, it would be still too long. So I think it is ok to have it as it is.


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


I don't understand your system for `sys/Makefile`... This however is prone to conflict with other PRs that just squish the path defintitions to the end


stubs.Setを利用しない方がいいという話があったかと思いますので、mock.patch.objectで書き直してみましたので参考に張っておきます。
```
    def test_copy_image_to_volume(self):
        @contextlib.contextmanager
        def fake_temp_file():
            class FakeTmp(object):
                def __init__(self, name):
                    self.name = name
            yield FakeTmp('test').name
        context = None
        volume = self.test_data.TEST_VOLUME
        image_service = FakeImageService()
        image_id = 1
        patch = mock.patch.object
        with patch(image_utils, 'temporary_file') as fake_temporary_file:
            fake_temporary_file.return_value = fake_temp_file()
            with patch(image_utils, 'fetch_verify_image') \
                    as fake_fetch_verify_image:
                with patch(image_utils, 'convert_image') as fake_convert_image:
                    with patch(self.client, 'delete') as fake_delete:
                        with patch(self.driver, '_resize') as fake_resize:
                            self.driver.copy_image_to_volume(context, volume,
                                                             image_service,
                                                             image_id)
        fake_fetch_verify_image.assert_called_once_with(context, image_service,
                                                        image_id, 'test')
        fake_delete.assert_called_once_with(volume)
        fake_convert_image.assert_called_once_with('test',
                                                   'sheepdog:%s' %
                                                   volume['name'],
                                                   'raw')
        fake_resize.assert_called_once_with(volume)
```


why do we need this change?


tests works without this line,  so I deleted it


Same way as the other init scripts were modified - sed.  I will be submitting a pull request in a few minutes.


Cool, I didn't know that.  Thanks

On Thu, Jun 25, 2015, 01:03 Jonathan Colon <notifications@github.com> wrote:

> Just a thought :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/code-newbies/python-thursday-adventure/commit/ca6bdcb9c98ec226bc4f5d8cdfecb89d3bfda313#commitcomment-11853891>
> .
>


I think we can't import variables but we can generate this file from the
installer with good prefix.

2015-06-25 13:23 GMT+02:00 Fez Vrasta <notifications@github.com>:

> can we import variables inside this script? otherwise I have no idea about
> how to use the ${EXECPREFIX} variable
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FezVrasta/ark-server-tools/commit/13913709d4d6b7a997c2e97f45df3b919842dbc2#commitcomment-11858996>
> .
>

--
*Nicolas MARCQ*


yeah I just checked too. I restored the html folder and it's fine again.

You can probably remove the addtochangelog.exe file as that's probably a security risk. I couldn't find it when searching for it ctrl-f in byond editor.


No prob, done.


Je suis en mode "debug".
Il n'y a pas ou plus de $codeAnd Error[3] (alors que sur l'api officielle c'est [3] aussi).
Donc pour avoir les boutons de payment, j'ai remplacé par " return $codeAndError[1]; "

J'avoue ne pas trop comprendre pourquoi... j'ai déjà utilisé l'api et ca toujour été [3]... wtf?


Isn't this precisely what the Transifex 'review' function is for? So someone else can take another look, and refine the translation if necessary?


Guessing you mean having multiple REE file blocks per TEE file?
You can refer design document here: https://docs.google.com/document/d/1nvzAb472hBXsuZgLqEgDZIXFiefuHjN0XbD6iyxqNaM/edit?usp=sharing

We have 2 reasons to have multiple REE file blocks.
1. Improve data encryption performance, we calculate MAC for whole TEE file before. And now we calculate MAC for one TEE file block. (we don't need to read whole file in just for modify one byte)
2. Improve performance when supporting atomic update. To support atomic update, we need to do out-of-place update. e.g. copy file context and write new data to the copied version. split file into blocks can reduce the data to copy.

I am not sure if I understand correctly what you mean to having tee-supplicant manipulate blocks in a single file. To having tee-supplicant manipulate blocks in single file, we need to change REE/TEE interface from file-based to block-based, and implement a simple file system on it.


So do actually have *any* test that uses `weave launch`?


Mongoid is ODM so maybe `orm` is not best name for this folder


That exe was part of the tools dir that I removed I think. Gonna check
whats pulling in the html files in the code.
On Jun 25, 2015 1:27 PM, "Gutsy" <notifications@github.com> wrote:

> yeah I just checked too. I restored the html folder and it's fine again.
>
> You can probably remove the addtochangelog.exe file as that's probably a
> security risk. I couldn't find it when searching for it ctrl-f in byond
> editor.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stuicey/ApolloStation/commit/a030a65ca739a82cedddb8d5884108f2ac67a4f3#commitcomment-11859065>
> .
>


Traditionally this second parameter is called 'ok', and this pattern is called 'comma ok'.


Don't ignore your errors! Trying to decide what to do with it will remind you to pass it up to the caller.


"Fix: internet explorer compatibility"*
ScrollY doesn't work in IE11, used pageYOffset instead


@adrianq you comment disappeared? That new class seems to be in the commit, is it giving you problems?


license="BSD" should use vlicense for the exact text.


Yes, if people add extra spaces in between the range and the command itself, it will report unrecognized command without this skipping. For example:

:'a,'bm.
This works before the modification. And if the input is:

:'a,'b  m .
This does not work if no skipping spaces.


vertical alignment


Has anyone else lost the EPG info in the Electronic Program Guide with the latest git?
![Image of EPG tab](http://i.imgur.com/jioIy9o.png)


Gah. Thanks!


Yeah removing this line for me meant the machine would provision


This looks strange)


Thanks, I wasn't able to push to a branch or something while at work. Hosting on wololo.net makes sense :)


@gidili Yes, it was my eclipse having fun in the morning. Everything is fine so I deleted the comment :)


I think this fixes #2224. I haven't done extensive manual testing, but the tests all (obviously) still pass.

Example output:

```
cabal: Cabal sandbox file in $CABAL_SANDBOX_CONFIG does not exist: foo
```

the "foo" is the content of the env variable.

I should note that the error occurs quite "aggressively", i.e. when the settings are read, and not when they are actually used -- so even things like "cabal info" will fail if the env variable is invalid. I think a more relaxed would be a lot more error-prone since it would be easy to miss cases.


`return a.priority - b.priority`


I prefer to keep it generic. It would make switching to single precision shading computations easier in the future.


Привет Виталий. Я очень хочу написать тебе письмо. Пришли пожалуйста свой email на этот адрес nikdranik2@gmail.com. Я тебе писал на vsl@ontil.ihep.su, но помойму ты его уже давно не проверяешь.


The col_at in Vintageous returns the column number, here we need the row number.


@andyjmaclean:
* The changes to spec_helper.rb in are unnecessary because webmock_helper.rb in develop already has them
* We don't want the local non-headless testing stuff in capybara_helper.rb merged because it won't run on Travis


This should always attempt to stop *all* containers, not bail when one fails. It should return a non-zero exit code when any stop failed.


@russellsayshi Whoa there!  We can't use Windows specific calls unless you're planning to make an equivalent for 10+ other platforms!  Epitome wouldn't compile on Linux if we use `Windows.h`!


This is creating an ObjectResult unnecessarily if value is a Gtk.Widget or a Foundation.NSObject


Strict equality means this misses cases like `opts.priority === null`, which maybe you don't care about, but sometimes it's nice to just use `!opts.priority`.  If you're being really careful, though, I'd maybe do:
```if (!('priority' in opts)) { opts.priority = 0; }
if (isNaN(opts.priority)) { throw new Error...```


Shading computations should be done in single precision, even today. Since incoming/outgoing directions, normals, etc. are in double precision, there need to be a point where you switch from double to single, but that should happen fairly early.


Ah, thank you!


Sure thing.  I wasn't sure if you guys would consider it a real issue :)


@Sorien You can read http://blog.ircmaxell.com/2014/12/on-php-version-requirements.html to get an idea of why it's a bad idea to keep compatibility with [older version of PHP that are not supported anymore](http://php.net/supported-versions.php) In the case of PHP 5.6 it not /yet/ unsupported but will be in a few days, and very probably before silex 2.0 release


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


Yeah, perhaps we can just wrap the ByteString with `CI`.


Why do you new HBase artifacts?


Why do you need Pig? (More later...)


Long lines again. Not sure if we want to fix them though


30 is original size but there is an extra space on the right (list mode), 18 prevent this trouble


> This should always attempt to stop all containers, not bail when one fails

````
vagrant@vagrant-ubuntu-utopic-64:~/weave$ ./weave launch
vagrant@vagrant-ubuntu-utopic-64:~/weave$ ./weave stop-proxy
vagrant@vagrant-ubuntu-utopic-64:~/weave$ docker ps
CONTAINER ID        IMAGE                        COMMAND                CREATED             STATUS              PORTS                                            NAMES
f52f72ebdd03        weaveworks/weavedns:latest   "/home/weave/weavedn   12 seconds ago      Up 11 seconds       172.17.42.1:53->53/udp                           weavedns
8fa1347a1862        weaveworks/weave:latest      "/home/weave/weaver    13 seconds ago      Up 13 seconds       0.0.0.0:6783->6783/tcp, 0.0.0.0:6783->6783/udp   weave
vagrant@vagrant-ubuntu-utopic-64:~/weave$ ./weave stop
Proxy is not running.
vagrant@vagrant-ubuntu-utopic-64:~/weave$ docker ps
CONTAINER ID        IMAGE               COMMAND             CREATED             STATUS              PORTS               NAMES
````

> It should return a non-zero exit code when any stop failed.

If you mean failed as in 'has left containers running', then possibly. OTOH If you mean failed to stop because they weren't running, exiting with a failure code is debatable - the desired outcome has been achieved afterall.


Fixed indentation issues here.


The annoyance I was worried about was instances like `Binary` changing their behavior, but I suppose we can just address that carefully.


Can you formulate a regular expression to validate the string?

What if the string is ``some_not-allowed&&&chars$inhere`` ?


Why are we going through Pig here?


Ok.  I'm not sure where you draw the line, as I regard (almost) this whole PR as improvements to the existing code, that happen to have been prompted by the fdp work, but leave the code in a better state independent of that objective.

For instance, I regard the consolidation of the UDP forwarding code in fccd6095b as a huge improvement, and the fact that it also introduces the `InterHost` interface is almost incidental.  The reason I kept these DF flag changes in a separate commit, despite their awkward nature, is knowing that the awkwardness would soon be resolved.

I'd even say that introduction of `PacketKey` and `FlowOp` in 864b61029ce can stand alone, because the information flows within the router become clearer.

The changes that seemed only justifiable in the context of fdp, I have saved for the fdp PR (hence why the peer short IDs are not here, despite involving scattered changes to the router code).  Though I would agree that 8a3ca9b44256 is a modest exception to that, and one that could easily be dropped from this PR.

So I'd urge you to consider the PR in that light, before deciding that it needs to be split up into more PRs.  Otherwise I'm not sure what stops us ending up with a PR per commit.

But if you really want this commit breaking out, I'd still suggest looking at it in conjunction with 3bcabcb9.   They are thematically linked, and although there are significant intervening changes, they could be brought together (better after fccd6095b, I think, but before is technically possible too).


ah ok. I thought 'stop' bailed. Leave as is then.


Every bean should by default be a singleton. What is the reason for manually setting the bean scopes here?


Get rid of this class, and instead use two parallel arrays. Here, you are paying memory overhead of every object.
Why not just String[] and int[][] at top level?


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


@adrianq :dancers:


The line numbers deceived me and I ended up looking at the wrong method. It's _mountFilesystems_ that's the suspicious looking one.

You might want to mention the TypeError in the commit message. It's only in Python3 that NoneType and str type objects are incomparable. In Python 2 they compare just fine. Which is probably why no problem was observed until recently.

Otherwise, ack.


Each line only takes part in the substitution once, and in the line, the '/g' option will control if all or only the first occurence will be substituted. This is also how Vim does it.


I have made the corrections as you suggested and also added expires_in in ```Claims```. I have also included the tests for ```Claims``` from ```Tokie``` too. Also, I've added the tests for LegacyMessageVerifier which are same as that were there for previous MessageVerifier.


This should probably be an inner class to BFScan?


the retype can be removed and get_nir_src ( , type) used


Why is `private` removed?


hfhg


Intanto grazie... mi sembra che ci siano delle competenze qui. Ottimo!

Vorrei partire dai requisiti e raffinarli prima di procedere alla implementazione. Insieme a voi. Vado per sommi capi.

1. La ricerca sull'archivio dovrebbe prevedere un riquadro "search" nel banner (diciamo in alto a destra e per gli smartphone un design responsive).
2. La ricerca può essere semplice (ricerca su tutto) o raffinata con diversi criteri. Si dovrebbe aprire un riquadro esteso in questo caso con le opzioni.
3. Le opzioni di ricerca articoli sono:
  - per titolo (ricerca solo nel titolo o titolo e articolo)
  - per autore (uno o più)
  - per data (o range di date)
4. Le opzioni di ricerca commenti sono:
  - commento nel titolo (ricerca solo nel titolo o titolo e articolo e commenti)
  - per autore (uno o più)
  - per data (o range di date)
  - per destinatario (da commentatore A a commentatore B)

La ricerca estrae gli articoli selezionati e tutti i commenti che soddisfano i criteri di selezione. Se la ricerca seleziona più articoli presenta solo gli articoli. Se la ricerca seleziona un articolo e commenti presenta anche questi.

Poi se vi vengono altre cose ditemelo.

In generale la applicazione deve essere molto veloce. Se si fa girare un indicizzatore che consuma memoria e cpu occorre tenere sotto controllo questi due fattori. Quanta mem e quanta cpu. Ora gli articoli nel DB sono circa 6000 e un ordine di grandezza i commenti (vado a mente).  Quindi occhio alla crescita che può essere esponenziale. In generale l'indicizzatore girerà di notte (le 3AM) solo 1 volta.

Poi do una occhiata alle modifiche e nel caso faccio un merge nel main trunk.
Tenete conto che il mio è un hack che ha i seguenti obiettivi (oltre a quello di dare un service al progetto):
- essere manutenibile dal minor numero di persone possibile (ossia 1)
- essere molto veloce per occupare meno risorse possibili

Il compromesso è il mono-file in python che è comprensibile e manutenibile da chiunque con poche conoscenze di python e programming. Ho pensato soprattutto a me e al fatto che potendoci lavorare poco (1 oretta al mese) avevo bisogno di ripartire ogni volta da zero per ricordarmi come funziona il tutto. Fino a quando il progetto non scala direi di non scalare il livello di complessità della implementazione.

Per ultimo, mandatemi una mail al mio indirizzo. Vi mando un dump di test del DB per fare prove tecniche.

Saluti

PS: la ricerca è tutta vostra io non ci metterò quasi naso :)


Why is the DMX padding option made dependent on using FluidSynth?


@BenFlanagan I might try running `npm cache clean` first - any npm version in the 2.x range should be able to handle this.


It seems like the following three classes can be merged into one?


Yup, the Id is null - so the editor will make sure to create a datatype in that case


same here, the retype could be removed and get_nir_src(.., type) used


Does this version work? How does efficiency compare to other version?


ヘルパ化了解しました！


We need to add (!prog), to allow dump_ir() to still run for ARB_vertex_program's, where prog and shader->base.ir are both null.


By this, class variable ```expires_in``` would have default value of 1 month, i.e., when we don't set expiry. Is this the way to go?


There is no distinct license file. They put the license text into the source files.  About 30 different authors altogether.


Why do you need to create such a large array of objects?


I think it is a typo, but wonder it these kind of errors are worth fixing.


Good to talk last evening, and to hear Sophie even if I did not see her.
Thanks for taking the time to get me started. I am sufficiently intrigued
to do some experimenting although I actually do very little coding. I am
also very taken up with running a project in the village to manage a £130K
church re-roofing which is being mostly funded by Heritage Lottery Fund who
are extremely demanding. We are at the critical point where they have to
sign off the planning stage and agree to release the real money. Oops.

Best wishes, P

On Wed, 24 Jun 2015 at 22:17 David Haywood Smith <notifications@github.com>
wrote:

> Marvellous!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fullacott/Test1/commit/8895f15909a79990049e34c306bd228080e01334#commitcomment-11848073>
> .
>


Why use List objects? Why not directly arrays? You know the query length, no?


> as I regard (almost) this whole PR as improvements to the existing code

Perhaps, but the commits get progressively more complex and, conceivably, debatable.

I want to get the easy stuff out of the way, first.

Re the ICMP change. I'd love to bring that forward, pre sleeve, but treat it as a separate change to the DF work.

So in summary, as the next step I'd really like to get the DF changes merged. And for that, if you can think of any relatively easy improvements for containing the `DF in two places` ugliness that would be great.


It's moot as there's no `name`, so it's impossible to publish it anyways.


@Sorien this matches the decision done for the next Symfony major version


ok, I will try to fix this and the 2 similar lines below.


Why not pass in the query object to make this cleaner?


Remove dead code.


get_nir_src already returns a src_reg, but we are doing src_reg(get_nir_src ...
Is this on purpose?


@leachbj you are correct that is the code that determines (via proton) the next deadline by which some data needs to have been sent and schedules a heartbeat to potentially happen at that future time (in case no other work happens). If a client is completely idle then that would only be expected every 7.5 seconds. If you just trace the TimerServiceImpl in logback.xml are you seeing that happen on a more frequent period?

``` <logger name="com.ibm.mqlight.api.impl.timer.TimerServiceImpl" level="TRACE"/>```


Ok, I just removed 'TODO: implement' from original comment. What's your suggestion here?


You return back to version 0.1.1, why?


What's this class for?


Dit was idd puur om inconsistenties uit ldb te halen, is niet voorgekomen.


ok


Vim does not have this error, like in the comments part above, Vim just read the current file into the buffer if no file name is given. Maybe we just use the show_not_implemented()


Fixed.


Ok, will change fname => file_name, and dname => dir_name


Why you are adding composer.phar. Directly add vendor folder.


Ok, good habbit


I protest ;)


We should use our own pcre as well.


The broken link's new location is
https://docs.acquia.com/articles/importance-user-roles-and-permissions-site-security
Should it be readded?


POST SCRIPTUM: ho visto i link ai motori semantici. Mi sembra una idea carina R&D... per me fate pure... può perfino rischiare di funzionare :)


And enable IPv6.


это пока единственная проблема? а те примеры. которые я указал в таблице идут?


Don't quite understand 'WS', there are too many acronyms for that. :-)


```
 * 2. Redistributions in binary form must reproduce the above copyright
 *    notice, this list of conditions and the following disclaimer in the
 *    documentation and/or other materials provided with the distribution.
```


I have a question:
If I was in an interview and was writing code, but suddenly, I occured some syntax problems that I was not familiar with, such as the throw expression. Then, I told the interviewee that I am not familiar with the syntax in this place, but my total code was right except the doubt syntax. Would the syntax problem affect my interview result?


судя по изменениям выше. тут не все изменения прошли


Thanks for the hint, I'll take a look at both of them.


ignore


\o/


This is the chillest pull request I have ever seen.


Okay disregard what I said before, I see the problem now. Thanks for reporting!


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


Fixed them.


Why the factor is the average value, shouldn't it be calculated depend on the distance type (horizontal/vertical/both) ?


Ok, Vim returns 'Can not open file <file_name>', will change


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


A comment explaining the dependency between snapshots and formats might be helpful here. I'm guessing that's why it had to be moved above super constructor call.


Please add some note not to remove these out commented lines. People who like "refactoring" and "celaning up" code, often tend to think that everything that is commented out should not be there, and there was no thought behind it at all...


Ok


что значит единственная? у нас >100 примеров эксепшены выдают.
не знаю, о какой таблице идёт речь.


il manque des choses...


How about `removeAt`?


Any specific action to be taken upon this IOError exception? I just couldn't think about anything to do for that.


This is the screen scale factor, in below code section, we calculate the offset with (hirizontal/vertical/both) touch offset.


You are right, I just simply forget it will read the whole file if no size specified.


Cant you just do something like `mActiveTransform.postScale` in `DefaultZoomableController` on double tap?


Yep, better name


But when the screen scale factors are different in x and y, shouldn't it be applied differently


Vim returns the same 'Can not open file <file_name>' as earlier. Will change.


This should fix the issue of the blueshield being unable to access the channels they need.


@loechel there is no text below this headline.


Just lowering the barrier for users. Introducing Paket will confuse some people.


Added it to the wrong place, I corrected this in a later commit


I think that all the pylint no-member annotations should be on the same line as the code that they effect. Right now, the one on line 883 is totally redundant, since the one at 879 has already disabled all no-member warnings.


LinkCrawler, so deal with links, not base urls and year. Why should a linkcrawler care if it is a year, month or some other way to construct a new link.


Again, nothing else is meaningful to be done with any type of Exceptions.


Without this line, one of the integration tests fails like this:
```
yadt-minion$ cat target/reports/integrationtests/should_not_raise_any_exceptions_when_calling_status_tests.err
error: cannot open Packages database in /.rpmdb
E
======================================================================
ERROR: test (__main__.Test)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/mock.py", line 1210, in patched
    return func(*args, **keywargs)
  File "/data/home/snordhausen/repos/yadt-minion/src/integrationtest/python/should_not_raise_any_exceptions_when_calling_status_tests.py", line 17, in test
    yadt_minion = Status()
  File "/data/home/snordhausen/repos/yadt-minion/target/dist/yadt-minion-0.5/yadtminion/__init__.py", line 214, in __init__
    self.yumbase.conf.cache = not(is_root)
  File "/usr/lib/python2.7/dist-packages/yum/__init__.py", line 896, in <lambda>
    conf = property(fget=lambda self: self._getConfig(),
  File "/usr/lib/python2.7/dist-packages/yum/__init__.py", line 300, in _getConfig
    startupconf = config.readStartupConfig(fn, root)
  File "/usr/lib/python2.7/dist-packages/yum/config.py", line 880, in readStartupConfig
    startupconf.releasever = _getsysver(startupconf.installroot, startupconf.distroverpkg)
  File "/usr/lib/python2.7/dist-packages/yum/config.py", line 1025, in _getsysver
    raise Errors.YumBaseError(msg + "Error: " + str(e))
YumBaseError: Error: rpmdb open failed

----------------------------------------------------------------------
Ran 1 test in 0.005s

FAILED (errors=1)
```


Singletons cause pain in tests. I'll do
```ruby
def self.task_registry
  @registry ||= TaskRegistry.new
end
```
in the module. It behaves the same but is easier to test.


Would you ever guess that a linkcrawler can tell you something about year?


Nice. I'll change, tomorrow or in the weekend. This was something I threw together after breakfast:)


yep, but this algorithm is borrow from CCScrollView and it is used for many years.
So let's just keep it here. I think the `factor` here is always 1 in most cases. Developers seldom change the view scale.


> So in summary, as the next step I'd really like to get the DF changes merged. And for that, if you can think of any relatively easy improvements for containing the DF in two places ugliness that would be great.

What is easy is not better, and what is better is not easy.  So I can put this commit into its own PR, if you are willing to merge as-is.

Could you take a look at the complex/debatable changes while that happens?  I think you will find a lot of the apparent complexity is superficial.  fccd6095 moves a lot of code around, but it remains close in spirit to what went before.  And I hope the consolidation will make it more readable, and so straightforward to review.  864b61029 is far reaching, but pure refactoring.  And the other commits are fairly modest.


I couldn't simulate such error in a normal Vim session. I would suggest using the same "Can not open file <file_name>".


Т.е. больше 100 примеров не работает?


Ok.


Change `throw new Error()` to `reject(new Error())`.


@sraue why? Cause latest and greatest - my BMW is faster than your Fax?


Reviewed-by: Pascal Brand <pascal.brand@linaro.org>


I see now that you wrapped it all in a `try/catch` and the `catch` (see below) calls `reject`. This is fine. Thanks


Also fixes accidental SQL injection


Haven't seen it. I'll can have a look and make a PR there too. I'll see if add it to the sample, it's nice to have as few dependencies as possible in the sample providing just the necessary bits.


ugh, very sloppy.


Since the description is not yet set, I would say "for" instead of "of".


Logging an exception is not exception handling, wrap up and let it rip or add the url to some failure object which I can examine later. Think about this, if when you are crawling 100 urls and two of them gave an exception, will you be able to make any sense of it, till I told you which urls were they, exception stack trace is great in showing the stack, not values in the stack and in this case it is data which causes the exception not the program.


ok i can do separate request !

i dont see the link with txt files & i don't know CodeMirror, i am going to check


Will try to fix later this week. :)


I'm working on position management service that is designed to work with
your transaction managers. Once I have this, I'll have a better idea of
possible ways to handle failure.

On Wed, Jun 24, 2015 at 9:51 PM, Jared <notifications@github.com> wrote:

> Thank you yes for sure, I'm just asking to understand problem but will
> solve problem in way that will help the most people. And potentially even
> we can design the solution together so it helps you + the Lean community.
>
> Thats why I ask about state: It seems like the intent of fall-over
> reconciliation is really to get back trading or pick up where it left off.
> But if there's other reasons I don't know its good to be made aware... like
> maybe we have a "OnFallOver" error event handler to handle this etc.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dpavlenkov/Lean/commit/d772016df95710eee1b0f3b80572c43f6e7cd153#commitcomment-11851757>
> .
>


"Toggle flag on/off" I think you used "Toggle" to describe a similar things, somewhere else.
"Toggle" is also the user friendly name for ARBoolean


This was not meant to be the dynflow task instance but rather an object that holds information about the task as a whole and provides the dynflow task and other info. I'll explain it on the code review. I'll prepare a diagram too.

We definitely need to set up a glossary of terms. I have a feeling that it's very messy now.


Either version is fine with me. I chose the slightly more verbose code because I think it is more readable for beginners.


Pull out `BoyerMoore.new` into a let block to dry these tests up


I think using `atom.project.relativizePath` may allow us to support the `.rubocop.yml` file in any of the directories within the project's path. See https://atom.io/docs/api/v0.196.0/Project#instance-relativizePath

Let me know if that works better.


I didn't try making `_getValue` work because the commit log indicated that the `preventDestroy` option was added to `region.empty()` be consistent with `region.show()`, which does not use `_getValue`.

I don't believe that switching `this` and `'preventDestroy'` would help anyway. If someone is passing in a function for `options`, the `_getValue` call would return the entire object returned by `options.apply(this, ['preventDestroy'])` rather than just the 'preventDestroy' value on that object. Plus, passing in a value of `{}` for `options` would still fail. If anything, we'd want to do something like this (untested):

```
options = Marionette._getValue(options, this);
var preventDestroy = !!options.preventDestroy;
```

But that still doesn't seem consistent with the way `_getValue` is used anywhere else in Marionette. It seems like it was just used by mistake in this case.


It is rude to usemcommemts on commits instead of using an issue.  Ask in the right place and i might answer


@SubJunk ```zerolatency``` is a libx264-specific option, this probably isn't the right place for it.


Oh okay ... took me a sec to realize the difference.


Excellent! So we can support both. I like this.


Sure, I get that. It's just a single file though: https://github.com/haf/Topshelf.FSharp/blob/master/src/Topshelf.FSharp/FSharpApi.fs so you could compile it into the example through https://fsprojects.github.io/Paket/github-dependencies.html


This description could also be interpreted as "You are showing the flags to the members of said faction". A better description could be "The faction whose flags are displayed" but I don't believe that is an optimal message either.


aaaaaaaaaaaaaaaaaaaaaaa


这个地方可能有问题，不是每次新建 work 都把之前的全部删掉。我们一个一个新建，每个 work 是一个单独的 from 来提交


Ok, absolutely.


Yeah, I think it'd be best to wait for node-pre-gyp to support Electron compiling. In the meantime your fork is simple enough to implement whenever I need to compile changes. Thanks!


Reviewed-by: Pascal Brand <pascal.brand@linaro.org>


Change `console.log` to `@verbose` or some other logger method.

See https://github.com/Glavin001/atom-beautify/blob/master/src/logger.coffee#L63 for the list of available logger methods, exported from [winston](https://github.com/winstonjs/winston).


Please make a change and do not load libraries from external sources. This centralizes the failure point and enables the external provider to track all visitors, or worse, inject code in a targeted manner via referrer, domain, IP and public cookie matching. Please include these resources locally with the wordpress installation and make using the local copy the default. In case you'd like to provide users with the option to use a CDN, please do it in a manner which allows and encourages those managing multiple wordpress installations to 1. use their own, 2. verify the script loaded is the right one (lazy load it with JavaScript and verify a checksum) and 3. avoid leaking user's browser behavior to third parties.


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


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.423.c742fbf6ab48267c047e3b836fc6257b2509f6bc](http://build.palaso.org/viewLog.html?buildId=58880&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3229, ignored: 119 Build time: 00:06:54


Just skip the parentheses around the "s" IMO it is source to more confusion than help. Could "be shown" be replaced with "see"? And could the "you want" be removed.


Both have been done and verified here:  https://github.com/the-t-in-rtf/unified-listing/commit/53b321527606bde71c338c35f481a5b655e838b1


wwwwwwwwwwwwwwwwwww


changes done


Comprend pas celle la ? Quoi qu'il se passe on passe par la ? :)


So cool!


I would move this part of the commit where we activate lower_vec_to_movs to the next commit "vecN operations are marked as unreachable", because  the lower_vec_to_movs is in fact the implementation of the vecN operations and it feels more natural and easy to justify. And these two lowering passes are not related.

If we do this, the commit message should be updated to remove the mentions to lower_vec_to_movs


If we split the previous commit and add here the activation of lower_vec_to_movs I would change the commit message to:
"i965/nir/vec4: Implement "vecN" instructions"
"nir_op_vecN instructions are implemented by the lower_vec_to_movs() pass"


Now you say "or players" instead of adding an "s" in parentheses. We should be consistent.
I still believe that we should just use the plural form. Then we also subtly suggest them to type more than one player at once. After all, they might not know that they can type in multiple at a time,


was `tests/background-image/style.json` completely removed intentionally?


Put suggest in its own line.


ElementType.TYPE can be scrapped out.


Aha, thanks. I was already looking for an obvious replacement to create a PR, but could not find anyone in the sources. However, Mike @m6w6  uses this at 17 different places and I am not really familiar with the pecl http code, so he'll have to do the fixing himself.

	Creating library C:\php-sdk\php70dev\Release\php_http.lib and object C:\php-sdk\php70dev\Release\php_http.exp
	php_http_querystring.obj : error LNK2001: unresolved external symbol _RETVAL_ZVAL_FAST
	php_http_env_response.obj : error LNK2001: unresolved external symbol _RETVAL_ZVAL_FAST
	php_http_header.obj : error LNK2001: unresolved external symbol _RETVAL_ZVAL_FAST
	php_http_message.obj : error LNK2001: unresolved external symbol _RETVAL_ZVAL_FAST
	php_http_params.obj : error LNK2001: unresolved external symbol _RETVAL_ZVAL_FAST
	php_http_client.obj : error LNK2001: unresolved external symbol _RETVAL_ZVAL_FAST
	php_http_client_request.obj : error LNK2001: unresolved external symbol _RETVAL_ZVAL_FAST
	php_http_cookie.obj : error LNK2001: unresolved external symbol _RETVAL_ZVAL_FAST
	php_http_env.obj : error LNK2001: unresolved external symbol _RETVAL_ZVAL_FAST
	php_http_message.obj : error LNK2001: unresolved external symbol _RETURN_ZVAL_FAST
	php_http_message_body.obj : error LNK2001: unresolved external symbol _RETURN_ZVAL_FAST
	php_http_params.obj : error LNK2001: unresolved external symbol _RETURN_ZVAL_FAST
	php_http_querystring.obj : error LNK2001: unresolved external symbol _RETURN_ZVAL_FAST
	php_http_client_request.obj : error LNK2001: unresolved external symbol _RETURN_ZVAL_FAST
	php_http_client_response.obj : error LNK2001: unresolved external symbol _RETURN_ZVAL_FAST
	php_http_cookie.obj : error LNK2001: unresolved external symbol _RETURN_ZVAL_FAST
	php_http_env_request.obj : error LNK2001: unresolved external symbol _RETURN_ZVAL_FAST
	C:\php-sdk\php70dev\Release\php_http.dll : fatal error LNK1120: 2 unresolved externals


Same, ElementType.TYPE can be scrapped out.


Please remove the `extern` specifier. It's the default storage class for C functions. Don't forget to update the man page (should be done automatically when building).


`ngx.var.request_time` is the time elapsed since the request was received:

```
local request_time = ngx.now() - ngx.req.start_time()
```

Which is not accurate when describing:

> request is the time elapsed between the first bytes were read from the client and after the last bytes were sent to the client. Useful for detecting slow clients.

Because the time the last byte was sent to the client is in `body_filter`, not later during the execution of `log_by_lua` that can happen later on. One can detect the streaming to the client has ended like follows:

```
-- in body_filter
local chunk, eof = ngx.arg[1], ngx.arg[2]
if eof then -- latest chunk
  ngx.ctx.total_time = ngx.now() - ngx.req.start_time()
end
```


Ligne 434 il y a
$campaign->id_voucher
mais $campaign n'existe pas :) il est après et du coup ça crer des bon de réduction à chaque fois que tu édites au lieux de mettre à jours le bon


extra blank before op[0]


@cxwshawn, imo, i'd suggest you to change the interface of
```
ObjBencher::aio_bench(
  int operation, int secondsToRun,
  int concurrentios, int object_size, bool cleanup, const char* run_name, bool no_verify)
```
to
```

  int operation, int secondsToRun,
  int concurrentios, int object_size, bool cleanup, const std::string& run_name, bool no_verify)
```
instead.

and the title of this commit could be more informative, like "check `run_name` for empty string instead of NULL". and you might want to prefix the title with "common: ".


Nope. There's no gain in waiting.


That than I also have to add Paket, nothing against Paket but I try to make it as easy as possible for John Seesharp to follow :)


Use ```Utils.nonNull``` on regionStartLoc.


What about "invites" instead of "the invites".


@agentzh Cool, thanks :+1:


Good idea!


oh, it's background-pattern now. For some reason the `/background-image/*/actual.png` files were sticking around and causing problems.


What's the purpose of this file, and how are we keeping it up to date? (there are a number of test files missing)


Thanks Peter!


Plural bs singular, again.


Los ejemplos mostrados en las diapositivas únicamente funcionan con la versión 2.0.0 de sequelize:
https://www.miriadax.net/web/javascript-node-js/foro/-/message_boards/view_message/34207346?_19_redirect=https%3A%2F%2Fwww.miriadax.net%2Fweb%2Fjavascript-node-js%2Fforo%2F-%2Fmessage_boards%2Fsearch%3F_19_keywords%3DProblemas%2Bcon%2Bquiz.validate%2528%2529.then%2528function%2528err%2529%26_19_threadId%3D0%26_19_breadcrumbsMessageId%3D0%26_19_searchCategoryId%3D34083430%26_19_breadcrumbsCategoryId%3D34083430%26_19_searchCategoryIds%3D0%26_19_redirect%3Dhttps%253A%252F%252Fwww.miriadax.net%252Fweb%252Fjavascript-node-js%252Fforo%252F-%252Fmessage_boards%253F_19_mbCategoryId%253D34083430


Why don't we have  `goto err;` as it is done in other part of the code so that error codes and panic are not spread in the code?


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


I believe the word "in" is a bit redundant


Ah yes bien vu :)


"The player who should join"


ighu


Why do you still need the above 2 lines?
Where is check_format defined?


"Toggle map auto updating on/off"


will it work if key2 has not been allocated?


cai gi day


Here you are explaining the commands behavior, and not the required information for the argument.


đặt tên theo chuẩn


PErcolationStats


Hm, shouldn't this have been fixed by making getBoard return null on invalid input? Or is there a valid usecase for getBoard returning a different board than the one passed in?


Too long line


Please correct it in the near future...


You could also copy-and-paste it ;)


Yes.


I noticed something else: For a project that has the jacoco plugin applied, 0.11.x started listing the `org.jacoco:org.jacoco.agent:0.7.1.201405082137` and `org.jacoco:org.jacoco.ant:0.7.1.201405082137` dependencies. Is this intentional?


it works for ipython 1.1, 2.4, 3.1 and python 2.7, and 3.4


Double f


weird indentation


Typo


wow, what a test coverage :)


If this code came from somewhere else, you need to identify where it came from. Also, _BSP_outch is not a properly named function. See https://devel.rtems.org/wiki/Developer/Coding/NamingRules which isn't complete, but for BSP functions we try to use something like either raspberrypi_xxx() or bcm2835_xxx()


Shouldn't it be "who's"? I am not 100% sure though.


What does this fix?


:facepalm: @georgehrke you are the one with the most inconsistent email in your pulls


Could it be "The new powers new value"? Actually I am not too fond with either description.


Use {} in the to ifs statements above.


Yep, we should not need to do this. I was just trying to get things working and pushed this to the spike in haste.


Isn't the word "the" a bit redundant?


### Seems like the following does most of what is needed (i forked the stuff onto my repository first):

In terminal:

    git clone https://github.com/einarhjorleifsson/geo.git
    cd geo
    git rm -r old
    git rm -r www
    git mv pkg/* .
    git commit -m "Remove duplicated directory"
    git push origin master

In R:

    library(Rd2roxygen)
    Rd2roxygen("/home/einarhj/r/Pakkar/geo")

In terminal:

    git add *
    git commit -m "Rd2roxygen test"
    git push origin master

see my [github space](https://github.com/einarhjorleifsson/geo)


Oh, sorry spatie tussen daily en meeting ipv. streepje, zelfde voor sprint reflection.


Gradle was failing in compilation: so you couldn't run tests from the command line. Something that will need to be sorted out.


space indent here


Just a small FYI here:
This breaks compatibility with the Visual Studio Bootstrapper, which will pass a project name for each included project.
I know that that plugin is deprecated right now but you might have to use it until the non-TFS Sonar MSBuild Runner is ready (version 0.9 depends on TFS)


slightly ^^ yes :)


Space indent here


The brackets are unnecessary. A switch statement could also be better, because this is an enum.


because the focus? Ik had de eerste keer de zin niet helemaal gelezen, maar het is because **of** the focus on features.


I have done this, you can see the results here: https://github.com/the-t-in-rtf/unified-listing/commit/1ff5af7916ff0dee0aac5c696b5e0aee7cc57394


They are required so that the `aspect.after` callback is fired so that the `selectable` class can be applied to the correct rows.


Why implementing so much custom content lately?


"The player to see/set rank for"


It has been dealt with appropriately, and removed in favor of your suggestion below,  see https://github.com/the-t-in-rtf/unified-listing/commit/1ff5af7916ff0dee0aac5c696b5e0aee7cc57394


work is done


static import


What we are actually sound is to set their faction. This message isn't very precise.


Please move this big if into a private method.


Fixed.


Possibly. I removed it when I switched to bootstrap-stylus but it looks like a bower install puts it back.


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


OK, I'll do this when I've booted into linux


Should this be here, or is this a stray import from testing?


@cxwshawn i am not sure i understand this fix. but `--run-name` is also documented at https://github.com/cxwshawn/ceph/commit/39d2390ea93aa8f3225e6c1c3850f95f3e4240e9#diff-dd76bf654ce550a2c7b66c2bc1a9430dL38 . and we take `--run-name` an alias of `--run_name`. so i believe they are the same thing. am i right?


You are absolutly right. It solved the problem. Having a fixed Y position for the label wont work in every situation.

I am now facing another problem....

![rplot01](https://cloud.githubusercontent.com/assets/13031900/8354284/e0264e38-1b3e-11e5-9b33-85de340881a3.png)

This report originated from this set of short term data:

[1] 29 30 30 30 30 30 30 30 30 28 30 27 28 31 30 30 30 30 28 28 28 28 28 28 28 28 32 32 32 32
  [31] 32 32 32 32 32 32 32 32 32 32 32 32 32 32 31 31 31 31 30 31 31 31 30 31 31 31 31 31 31 31
  [61] 30 31 31 31 31 32 32 31 31 32 32 32 31 31 31 32 32 31 31 32 31 32 31 31 31 31 29 30 29 29
  [91] 30 29 29 30 30 30 29 30 29 30 30 29 30 31 32 31 31 31 31 31 31 31 31 31 31 31 31 31 32 32
 [121] 32 31 32 31 31

I think that the binwidth is not well adjusted to this particular example and there is something odd in the way the density lines are beeing plotted. Is it normal for the p-value to be that low? This distribution is far from being a perfect gaussian but at the same time not that far.

Note that the system that generated this data can only detect changes in temperture of one degree.


also you might want to polish the title of your commit message.


pushed -f (comma, ok)


My fault, this line is too long, better:
    return emit(LRP(dst,
                     fix_3src_operand(a),
                     fix_3src_operand(y),
                     fix_3src_operand(x)));


I think you mean : 'use findAllDeletedInLastVersion'


посмотрел, действительно сломался solve(log(x**2)) , кажется понял на чем


That's true :)

-----Original Message-----
From: "Henrik Feldt" <notifications@github.com>
Sent: 2015-06-25 14:38
To: "mastoj/SuaveTopShelfDemo" <SuaveTopShelfDemo@noreply.github.com>
Cc: "Tomas Jansson" <tomas.jansson@gmail.com>
Subject: Re: [SuaveTopShelfDemo] initial commit (9e5f065)

You could also copy-and-paste it ;)
—
Reply to this email directly or view it on GitHub.


align parameters and add type of siteId(string) in comment


because this is a nonlinear optimization, it might be valuable to initialize with the positive weights from the SVD solution. obviously, though, if this works as is , then the extra computation is not worth it.


The first should be ""The player to see/set rank for"
The second should use "perform" instead of "do" (IMO)
The last is just plain wrong. What we do is to set the players faction.


现在我们感兴趣 `的是` ...


add an empty line


Why?


add type of parameter in comment


I'm not seeing the issue in this code, but it was the only change yesterday

I was diffing yesterday's and today's arm_5.1_stock zip files and the update-binary had '0' for the density for the 480 playgames (see attached image).  I assume it should have stayed and '8'

![image](https://cloud.githubusercontent.com/assets/376370/8354580/d556e916-1b0e-11e5-8d50-05d16b5007c4.png)


The "in numbers" at the end sounds a bit weird.


у нас есть тесты, их полезно прогонять при попытке что-то "поправить в консерватории"


Slim device repos do not have android_ in the beginning, also we have device_qcom_common in our main manifest


Fixed.


Players might be confused about what an "entity" is, what about "player/faction"?


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Deploy Update [Build 5](https://bsr.ict.je-labs.com/viewLog.html?buildId=29769&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_DeployUpdateWordpressCoreDirect) is now running


@SalamiArmy could you send the error message you are facing with the VS Bootstrapper plugin? I fail to see how this change breaks compatibility


add type of parameter


Why not added in brw_nir.cpp. Are the text shader opcodes different or not used in fs_nir?


`router` is not a widget and thus doesn't need to be here.


What about "inspect" instead of "look at"?


ah ill remove them thanks for pointing it out :)


Да мне прислали письмо я их на выходных скачал


Fixed. Also took the occasion to add extra debug info.


Or is it "whose" I am not sure.


Can we do this via our stylesheet?


fixed #7


Line 3 should say Slim

You should always call device specific after generic ei
$(call inherit-product, vendor/slim/config/common_full_phone.mk) should be before
$(call inherit-product, device/motorola/titan/full_titan.mk)


Shouldn't it be "who's" or "whose"? I am not sure what is correct.


The selector doesn't need to be quite so verbose:

```css
.zci--snow {
  ...
}
```

Should be fine.


likewise:

```css
.dark-header .zci--snow {
  ...
}
```
is good.


Main file contain curl class


It should be "relation" instead of "relationship". If it was relationship, that would mean you could set it for individual factions.


I am not sure if this is the optimal message. I can however not suggest a better one.


校对完，没什么问题


но тут еще проблема в том, что я вижу в diff 3 последних строки правки. почему все остальное не дошло пока не понимаю.


done


That message sounds a bit clumsy.


allright !


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


build


I suggest you search for "you want" in you IDE. It should not be used anymore.


TeamCity Wordpress :: CorpSite :: CorpSite-Staging :: Deploy Update [Build 5](https://bsr.ict.je-labs.com/viewLog.html?buildId=29769&buildTypeId=Wordpress_CorpSite_CorpSiteStaging_DeployUpdateWordpressCoreDirect) outcome was **SUCCESS**
Summary: Running Build time: 00:05:34


Fixed.


The message doesnt seem very intuitive. I suggest a wording like the following:

> WARNING: SSL disabled and configured to listen on the network. Passwords and other information might me eavesdropped by third parties.

My wording isn't exactly optimal, but at least tells the user the conditions that trigger the issue.


It was because @DJAlik had already committed the new Google Play Games to the repo, before a universal version was updated (and still not is, btw). So now it doesn't see the same version for xx8 available, and thus it skips this DPI variation. Normally the best is to update all variants at the same time, but this is not always possible.


и ещё полезно к коммитам вменяемые коменты писать


Well, then i have to `grep -h 'Copyright (c)' src/*.c` to at least try to satisfy that and build my own textfile i later move with `vlicense`.
Debian for example just picks the name common in most files (but not all) and adds `...and others`.  Which cannot be ok either, if being pedantic.
Or `cp src/*.c /usr/share/licenses/tin`.


This should be `F<...>` no? The file does exist in the tarball... If no objections I will apply with that change


If you don't like it, I can make it more generic. I can add here named parameter before_render_kwargs, which will be dict of parameters for before_render function.


Some of eye money messages are inconsistent. Some says "of money" others don't. Also should it be "The faction to deposit it in" or "The receiving faction"? I prefer the shorter descriptions.


wf


I had removed it, cause after Matthieu suggested I run make after every rebase.
Will remove it!


Yes, when you start the IDE and you have removed/edited/screwed up your hardware, selecting a default board makes everything work all the same


TeamCity YaccConstructor :: YC.Core General Build [Build 460](http://teamcity.codebetter.com/viewLog.html?buildId=194006&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Agreed with all that, revert or modify this I think


No objections.  I'd not thought of using `F<>`: good catch!


setDescriptionEntity(array $descriptionEntity)


@tchap On line 302 below the error is `&ErrRefNotFound{remote + "/" + branch}`. I wonder why here you use `target` and `remote` there. What's the reason for the inconsistency?


instead of `self` , you could put `extensionItem` directly in the `activityItems` array. If you do that, then you won't need to check for the bundle-id, i.e., you won't need to implement the UIActivityItemSource protocol. Since other activities do not return actionable data (if any at all) to the completionWithItemHandler callback, you could feed whatever data is returned to the fillReturnedItems method in that callback. You can then eliminate the "isPasswordManagerActivityType" function altogether.


Ok , so  i will create a properties bean for mail link crawler, and use it in every specific implementation of link crawler to get additional parameters


changes made


Try calling `updatePageInfo` in `updateTitle` method instead.


Perhaps `...each C<*_ok> function uses` (as opposed to `sub`)?


may as well keep these to be complete


'hi internetz'


That whole loop looks quadratic, since both the `sort.Search` and and sum updating are linear. The former is fixable with a different data structure, e.g. a [skiplist](https://github.com/msackman/skiplist), but the latter is inherent. Is there no better way? I do quite like the 2nd answer from http://programmers.stackexchange.com/questions/233541/how-to-implement-a-weighted-shuffle. It's not perfect, but perhaps good enough? I suggest you ask your colleagues for ideas.

And I also do wonder about the complexity of append. Different data structure would help here too.


Hello Massive Team,

did I get it right?
01. Munit does not like .hxml files rooted more deeply than the main directory.
02. The src reference can only point to one directory.
03. The src reference also is not supported for a deeper structure than (root/src). So no subdirectories of root/src can be referenced.

Is this mandatory? Or can this be fixed by additional arguments? Or did I get it totally wrong and I expose myself to be an absolute beginner?
;-)

As I would have to restructure a very complex project, I would be very gratefull to have certainty.

Thanks for your support. And thanks for this cool tool, by the way.

Kind regards
Markus


@snordhausen You don't have a rpmdb on your machine (ubuntu with yum installed), you can create one with:
```sudo rpm --initdb --dbpath /.rpmdb```
see: https://github.com/yadt/yadt-minion/pull/6


It's kinda unfortunate they don't have a license file. I'd just select one snippet with the main author.


That's more accurate, yes.  Although, I think that `sub` is used throughout the document where `function` is really meant.  Perhaps it's better to leave as `sub` for consistency?


Added functions to write restart files in Parameter & children and Model & children.
Added getCovarianceMatrix to CovarianceMatrix.cpp (for the write restart file method)
initBaseValuesFromRestartFile is working correctly.

Bug fix for the looping in proposeCodonSpecificParameters


@glitsj16 I mean that `_` is in `GEANY_WORDCHARS` but you also added it (after `+`), so it's actually there twice in the resulting string.


I only commit from one machine. What is the source for those e-mail addresses?


Explain me please condition in these methods WithCustomState, WithMessage and WithErrorCode: "if (count > 0)  throw new ValidationTestException..."
Is it means, that you don't expect an validation failure with specified value (state, message or errorCode) ?


@tchap I suppose the naming could be better:
`EnsureLocalTrackingBranch` -> `MirrorRemoteBranch`
`EnsureBranchSynchronized` -> `SyncExistingLocalBranch`
or something along the lines. Current naming doesn't actually tell you anything about what happens or about distinction between creation and sync.


please fix the doc, it says "configuration configuration" by mistake


Fair point, applying as-is


use ref


@HowardHinnant Curious to hear your thoughts about the choices Nik made in this pull request with respect to how he partitioned the business logic. How do you feel about the domain-specific responsibilities of the new `Consensus` class?


ditto


The `-` suggests you're supporting both the prerelease `0.3` _and_ the official release starting with `0.3.0`. This is newly documented.


@st3fan Exactly. Except for one-password's extension, the idea is that all others use `.find-login-action.` in their name to indicate compliance with one-password's SDK (even for other activity types). Another options is to feed the fill item into the activity view controller beforehand (see my new comment below).


hehe


Lovely.  Thanks!


aa


*following


With this commit:
```
09:20:26.821 INFO  - Sensor org.sonar.plugins.resharper.CSharpReSharperProvider$CSharpReSharperSensor@b1af58
09:20:26.821 INFO  - Got Dev2.Server as the sonar.resharper.projectName setting
09:20:26.821 WARN  - ReSharper plugin is running in deprecated mode. inspectcode.exe should be ran outside the plugin and the report imported through sonar.resharper.cs.reportPath property.
09:20:26.837 INFO  - Executing command: C:/jetbrains-commandline-tools/inspectcode.exe /output=C:\NightlySonar\Dev_Sonar\Dev\.sonar\Warewolf-ESB-TestingNewAnalysis_Dev2.Server\resharper-report.xml /no-swea /project=Dev2.Server /profile=C:\NightlySonar\Dev_Sonar\Dev\.sonar\Warewolf-ESB-TestingNewAnalysis_Dev2.Server\resharper-sonarqube.DotSettings /no-buildin-settings C:\NightlySonar\Dev_Sonar\Dev\AcceptanceTesting.sln
INFO: ------------------------------------------------------------------------
INFO: EXECUTION FAILURE
```
Without:
```
14:45:53.334 INFO  - Sensor org.sonar.plugins.csharp.CSharpSensor@cbd2de (done) | time=5387ms
14:45:53.335 INFO  - Sensor org.sonar.plugins.resharper.CSharpReSharperProvider$CSharpReSharperSensor@18bacb0
14:45:53.335 WARN  - ReSharper plugin is running in reportPath mode, other properties other than reportPath and solutionFile can be undefined
14:45:53.335 INFO  - Parsing ReSharper report: ...
```
Note the custom logging I added to my build of this Resharper plugin (I've since lost the code but I think I added it to ReSharperSensor.java at the point it checks if that property exists, I log what value it got) this proves VS bootstrapper is passing the project name through:
```
09:20:26.821 INFO  - Got Dev2.Server as the sonar.resharper.projectName setting
```
(Dev2.Server is an included project in my solution)


I got that setting and I didn't change since for years.
https://github.com/georgehrke/dotfiles/blob/master/.gitconfig


@tchap Forgot to update the comment.


@mbernat Yeah, I was also thinking about it...


That doesn't look right. You're not calculating the value that you expect. In the formula, each P is a 2d point, and what you get is a coordinate of the point on a curve based on on parametric value t = 0..1, where X axis is time and Y axis is progression. Instead you need to get Y from X, which is more complicated. In GL JS, we use https://github.com/mapbox/unitbezier/ for that.


This import does not exist in my dev environment.


je montrais a un copain comment s'utilisais github et android studio :p
donc je montrait le systeme de commit/push :D

Le 24 juin 2015 23:47, Antoine Paris <notifications@github.com> a écrit :

> Mec c'est un peu tard non? [image: :dancer:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/anpar/LSINF1225-projet/commit/167ce7800c5681aab154ba8aacebe9396b52d738#commitcomment-11848614>
> .
>


Shouldn't EmulatedUnscopedShadowDomStrategy also be exported here, just for the sake of completeness?


see issue #23 for this function


@mbernat Really?


TeamCity YaccConstructor :: YC.Core General Build [Build 460](http://teamcity.codebetter.com/viewLog.html?buildId=194006&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Building ..\src\YC.SDK.sln failed with exitcode 1. Build time: 00:10:18


Right now, CodeMirror is used for editing files. You can try on the online demo : http://fm.linea21.com/

Your proposed feature sounds like a duplicated functionality to me, since the FM already offer to visualize editable content. Rather than duplicating the feature we could use a new parameter option proposing to visualise content (in place of editing).
Actually I would give you the advice to PR only PDFViewer right now.

You can also do PR  for :

* Added min width to splitter
* fix default loading shift …
* Added purple theme
* Hide contextmenu on file click
* Fix scroll size …


throw an error here instead of hive default


lgtm % comments


`git log | grep devgeorg@ownCloud.com` is empty. That e-mail address doesn't even exist. There must be another source for those addresses besides the git commit history.


Add the @return tag to the javadoc indicating that it returns an unmodifiable and inmutable copy of the reads in the assembly region.


Use ```Utils.nonNull```.


Is the project name too long? I can't think of a better shorter name at the moment.  Either way, I really like the site.


Done.


Add the {} to the if.


But extracting a variable would be even better then. It's not necessarily clear that status['next_artefacts'] contains the updates


Absolutely. The css filename threw me for a loop.


Try calling `updatePageInfo` in `updateTitle` method.


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.396.ffa1adf7a195c6750c13fd12c71542fa59e5be99](http://build.palaso.org/viewLog.html?buildId=58881&buildTypeId=bt322) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:03:47


Add the javadoc indicating that this is equivalent to trim(span,span).


Well if inheriting the environment fixes this it looks like yum uses an environment variable to know which rpmdb to use? So it wouldn't help to create an rpmdb and this patch is the only way to get the tests to work everywhere.


会绘制前
去掉 `会`


Number of clusters already starts from 2, so `vrcs_diff[0]` is `(VRC_4 – VRC_3) – (VRC_3 – VRC_2)` (k=3).
I've update the line for `k_opt`.


Why is the order yaw, pitch, roll? Normally it should be pitch, yaw, roll to match the axes.


It's ready, I just managed to push it up to `staging` before rushing out the door.  :P


@annacurrey


Agree, fixed.


### How did i get here?

    git clone git@github.com:Hafro/geo.git
    cd geo
    git checkout -b iss1
    git rm -r old
    git rm -r www
    git mv pkg/* .

    git commit -m "Moved stuff to root and deleted redundant directories"
    git push origin iss1

    library(Rd2roxygen)
    Rd2roxygen("/home/einarhj/r/Pakkar/geo")

    git add *
    git commit -m "Rd2roxygen test"
    git push origin iss1

Problems: @export seems missing in help files
Next step: fix that and then run cmd check

einar


@stringparser thanks for that great example, just fantastic. this would totally be worth adding the wiki as an example of how to customize the output! we'll integrate this immediately!


Did I miss an existing way of doing this?

I want need to put an extra field in the response json from the event handler. What I have in scope is the created relationship object (see above).
I kind of lost tract digging through the very general json builder so I implemented this very ad-hoc.
Suggestions very much welcome.


love the name


@tchap Whaaat? Why is this madness necessary again?


Check readsToRemove with ```Utils.nonNull```


I'd suggest you add a get/setExtent and specify that setting the location sets both the position and extent. Also, is their any reason why we need to use an interface for Transform?

I'd also suggest adding a getter/setter for the rotation as a Quaterniond, and a getter/setter for all the transform information (except the Extent) as a Matrix4d.


I already changed it as part of your comment to the previous commit. What do you think? https://github.com/Igalia/mesa/commit/86ea3d21e0e73db0ed96f1244dcc4e8adb8bfb14.


@woppa684 good point actually. I've imported only them because `EmulatedUnscopedShadowDomStrategy` is used by default anyways (https://github.com/angular/angular/blob/master/modules/angular2/src/core/application.ts#L37-L390)


校对完，没什么问题


@tchap Well, the code says "stable branch" everywhere but the comment says "master branch".


This looks good, so the next step is to make this search by second letter typed, third letter typed, etc?


I find this structure harder to read than the wrapping on every line, but I'll let @liggitt break the tie.


> git log | grep devgeorg@ownCloud.com is empty. That e-mail address doesn't even exist. There must be another source for those addresses besides the git commit history.

Maybe no longer - but is was present at some point in time. Anyway - this is not critical.


Fixed.


I prefer each wrap being an assign on it's own line as well


no template attribute


config/database.yml.sampleをpushし直しました！


Does it make sense to rewrite the commits with (owncloud)dev@georgswebsite.de and my gmail address to make use of my current address?


Let's try it out in praxis. We can adjust them if it's not working well.


That's a nice improvement indeed. Thank you for the heads-up.


Why you implemented this class? As I see, there are all native devise actions w/o redefining?


@SalamiArmy If I correctly guess - you are trying to use the re-use reports mode of this new R# release while using the Visual Studio Bootstrapper plugin.

That use case is not supported, the re-use reports mode is being introduced only to MSBuild Runner users.

Meanwhile, nothing is changing for Visual Studio Bootstrapper plugin users - R# is still being executed.

FYI - we'll release the MSBuild Runner 1.0 shortly, which will generalize its use to all .NET users (even if you're not using TFS).


@bpoplauschi can you check this https://github.com/mythodeia/SDWebImage/commit/3c162d2f9c42e77395a07850eb88b6aacfde9b23
and let me know if that works for you.
i have used constraints and did not overwrite the ```layoutSubviews```


Sure, that makes sense. It seems like a fine intermediate step.

On Tue, Jun 23, 2015 at 2:42 PM, Philip Durbin <notifications@github.com>
wrote:

> For comparison, here was an earlier commit I made (in a separate branch)
> for option 2 about making fragsize configurable: ff77e62
> <https://github.com/IQSS/dataverse/commit/ff77e623615199cd7db943a7432f2bca1aeb82ff>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IQSS/dataverse/commit/40697f92b74dc3df7d204b3dad5e3d20d9c971db#commitcomment-11823103>
> .
>


I literally copied/pasted from Entity. Guessing it was wrong there.

Granted we should match whatever MC does this as.


for example, with this structure, it's harder to see the two args to `cacheControlFilter` are the retval of `namespacingFilter` and `"no-store"`


pretty sure this will break on a nil `handler`. I think you need to initialize `handler` the container.


Thanks Ninja.


need sort - type, number
bug 1
bug 2
bug 8901


no need for else.  yay go.


since this is a file it wont work


> Does it make sense to rewrite the commits with (owncloud)dev@georgswebsite.de and my gmail address to make use of my current address?

there is no further action required here - thx


Minor suggestion - but should now use SpatialReferenceChanged instead of PropertyChanged and checking for SpatialReference.


The same situation


oh, sorry!

I will fix this after a while. thanks

Best Regards

hightman/海鳗
**********************************
微信/微博：hightman
Github：https://github.com/hightman

> 在 2015年6月25日，下午9:23，lynicidn <notifications@github.com> 写道：
>
> need sort - type, number
> bug 1
> bug 2
> bug 8901
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/hightman/yii2/commit/f80d71cee282643aa01f67552e906f1bc62f31ac#commitcomment-11861004>.
>


looks good thanks :+1:


Might as well log remote_address on this line.


Ah, you're right! Good catch. Might as well fix that while addressing this duplication.


:D


Well that made the prefs dialog buttons lose their icons completely...


Was this tested? I thought it was strange that the patch references `unknown_profiles` but does not define it (I did not remember ever adding a variable of that name).

The patch certainly catches bad profiles:

```
# sosreport -v --batch --profile=bogus

sosreport (version 3.2)

Unknown or inactive profile(s) provided: ['bogus']
The following profiles are available:

 boot
 cluster
 debug
 desktop
 hardware
 identity
 java
 kernel
 mail
 memory
 mrg
 network
 nfs
 openshift
 openstack
 packagemanager
 performance
 security
 services
 storage
 sysmgmt
 system
 virt
 webserver

 24 profiles, 0 plugins

```

But is a bit sub-optimal for actual existing names:

```
# sosreport -v --batch --profile=system

sosreport (version 3.2)

plugin abrt does not install, skipping: global name 'unknown_profiles' is not defined
plugin anaconda does not install, skipping: global name 'unknown_profiles' is not defined
plugin anacron does not install, skipping: global name 'unknown_profiles' is not defined
plugin auditd does not install, skipping: global name 'unknown_profiles' is not defined
plugin boot does not install, skipping: global name 'unknown_profiles' is not defined
plugin cgroups does not install, skipping: global name 'unknown_profiles' is not defined
plugin cron does not install, skipping: global name 'unknown_profiles' is not defined
plugin dbus does not install, skipping: global name 'unknown_profiles' is not defined
plugin general does not install, skipping: global name 'unknown_profiles' is not defined
plugin hardware does not install, skipping: global name 'unknown_profiles' is not defined
plugin i18n does not install, skipping: global name 'unknown_profiles' is not defined
plugin kdump does not install, skipping: global name 'unknown_profiles' is not defined
plugin kernel does not install, skipping: global name 'unknown_profiles' is not defined
plugin keyutils does not install, skipping: global name 'unknown_profiles' is not defined
plugin krb5 does not install, skipping: global name 'unknown_profiles' is not defined
plugin kvm does not install, skipping: global name 'unknown_profiles' is not defined
plugin ldap does not install, skipping: global name 'unknown_profiles' is not defined
plugin libraries does not install, skipping: global name 'unknown_profiles' is not defined
plugin libvirt does not install, skipping: global name 'unknown_profiles' is not defined
plugin logrotate does not install, skipping: global name 'unknown_profiles' is not defined
plugin logs does not install, skipping: global name 'unknown_profiles' is not defined
plugin lsbrelease does not install, skipping: global name 'unknown_profiles' is not defined
plugin megacli does not install, skipping: global name 'unknown_profiles' is not defined
plugin memory does not install, skipping: global name 'unknown_profiles' is not defined
plugin multipath does not install, skipping: global name 'unknown_profiles' is not defined
plugin networking does not install, skipping: global name 'unknown_profiles' is not defined
plugin numa does not install, skipping: global name 'unknown_profiles' is not defined
plugin openhpi does not install, skipping: global name 'unknown_profiles' is not defined
plugin pam does not install, skipping: global name 'unknown_profiles' is not defined
plugin pci does not install, skipping: global name 'unknown_profiles' is not defined
plugin pcp does not install, skipping: global name 'unknown_profiles' is not defined
plugin ppp does not install, skipping: global name 'unknown_profiles' is not defined
plugin process does not install, skipping: global name 'unknown_profiles' is not defined
plugin processor does not install, skipping: global name 'unknown_profiles' is not defined
plugin psacct does not install, skipping: global name 'unknown_profiles' is not defined
plugin python does not install, skipping: global name 'unknown_profiles' is not defined
plugin rpm does not install, skipping: global name 'unknown_profiles' is not defined
plugin sar does not install, skipping: global name 'unknown_profiles' is not defined
plugin selinux does not install, skipping: global name 'unknown_profiles' is not defined
plugin ssmtp does not install, skipping: global name 'unknown_profiles' is not defined
plugin startup does not install, skipping: global name 'unknown_profiles' is not defined
plugin subscription_manager does not install, skipping: global name 'unknown_profiles' is not defined
plugin system does not install, skipping: global name 'unknown_profiles' is not defined
plugin systemd does not install, skipping: global name 'unknown_profiles' is not defined
plugin sysvipc does not install, skipping: global name 'unknown_profiles' is not defined
plugin tuned does not install, skipping: global name 'unknown_profiles' is not defined
plugin udev does not install, skipping: global name 'unknown_profiles' is not defined
plugin yum does not install, skipping: global name 'unknown_profiles' is not defined
Unknown or inactive profile(s) provided: ['system']
The following profiles are available:

 boot
 cluster
 debug
 desktop
 hardware
 identity
 java
 kernel
 mail
 memory
 mrg
 network
 nfs
 openshift
 openstack
 packagemanager
 performance
 security
 services
 storage
 sysmgmt
 system
 virt
 webserver

 24 profiles, 0 plugins

```


Same as in geyes, the prefs dialog buttons don't have icons anymore so they look unusually small...


@yuchi Simply for 2 reasons:
1. Since these files are transpiled into AMD format, it makes no sense to automatically transpile everyone's code into AMD (for instance, our current YUI modules would no longer work, modules which are expecting to be global wouldn't work, etc). We don't want to break everyone's code, and this es6 transpilation should be opt-in.
2. .es (or es6) isn't a recognized syntax (at least as far as I've found in any of the JS packages around), so everyone's IDE and text editor would have to be modified to add support for it as an extension. That seems more of a pain than needed.
This is simply a flag to indicate which files should be transpiled.

Before deciding how to flag it, I did some research and it's not an uncommon pattern (though some others use .es, .es6, .jsx).

What's the downside to using this pattern, and do you see any other way to flag these files to indicate that we should transpile them?


better use R_NEW0 to avoid possible issues with uninitialized fields


I found it odd that the `tutorial` link goes to the same wiki page as the `Documentation` link that precedes it. Perhaps changing the `tutorial` link to this would be better:

https://github.com/clojure/clojurescript/wiki#tutorials


But when the default changes also changes to these exports should me made, and that's not really obvious, so including the unscoped version here also would be better I think.


@tchap so here it just silently doesn't do anything? Perhaps it could at least print a warning that no accepted issues were found.


Fair enough. The pass-myself-in-get-myself-out previous implementation looked weird and didn't read well at all, it was not immediately obvious to me that by doing
```
handler :=
handler =
handler =
handler =
```
there was a chain happening and not a re-assignment of some sort.


When returning an object / hash (`{}`), you should provide both keys and values, but the keys can't reference `this`. So it would be:

``` js
return {
  run: function() {
    animate(step);
  }
}
```


Fixed.


The `Handle` and the `InstallAPI` calls have swapped order.  Does that affect the result?  I would sort of expect it to, but I'm not completely certain.


is it all possible for this to be chainable? would that refactor be worth it?
` handler := indexAPIPaths().filterCacheControl().filterNamespaces()...`


Oh, you fixed it eventually. Good :+1:


Deriving a copy is broken. An error is occurring in links.coffee. Please make sure you test deriving and editing.

Also, when I first navigate to the first page in a book, the version, page and title are not apart of the URL.


@tchap and what's the problem with sending the empty array even when you don't want to update it? Does it hurt anyone? I mean, AFAICS, it's just a few bytes overhead with no effect.

If at all possible, I'd prefer to avoid null and to always have an array in that field.


Feels weird doing it. In any case, this else block assignment ensures `nil` is never returned. Sloppy?


I'm happy to update this PR accordingly once @tbosch decided what to do.


I think this needs indenting.


The read_domains/write_domain approach was modelled after i915's set_domain ioctl (which was also part of the original GEM API proposal -- according to the LWN article). The UMP msync code does not make any distinction between readers and writers, it simply has the concept of ownership and assumes that the owner is going to have exclusive RW access. We could either live it here unused to be aligned with other drivers' GEM API, and then think about what meaning should we give to it, or remove it now and add later on if we figure it would add any benefit.

I don't have a strong opinion on which approach would be better or preferred by upstream, so I intentionally left it here more as a reminder to have this discussion than anything else.


This package still officially supports PHP 5.3 which doesn't support array shorthand. This commit corrects an oversight.


we're `Run`ning, not `Drive`ing so it's legal


Thanks!  I've got a bunch of performance enhancements I want to make, so I can work with bigger datasets.


fs_nir reuses the fs_visitor, IR-based emit_texture(). So they have a similar block of code but from NIR opcode to their IR counter-part, not to the SHADER_ opcodes directly.


The latest/greatest version of my skeleton layout.


Is there a way to make this a bit DRYer?


good


Are these things really-really necessary? We may create some cross-platform macro wrapper around `strcasecmp` / `stricmp` if necessary. Or even [such](http://stackoverflow.com/a/2886589/2684760) implementation of `char_traits`.


@DDoS

I didn't make this a class solely because isValid is implementation-specific. The base game would flag an invalid transform if coordinates are outside 30m, n, 30m (well in our implementation) but another implementation that is completely custom may not have such restrictions.


All those dudes forgotten forever.


汉字和Unicode互转工具


Also you can use this method:
```ruby
def set_team
  @team = Team.find params[:id]
end
```
to not duplicate this line `team = Team.find params[:id]`
also you must add `before_action :set_team, only: [:show, :edit, :update, :destroy]` for that

It's a standart trick in Rails4


Right, MC doesn't use roll, so any roll other than 0 would be invalid...


I suggest you re-base on the pass lmr tweak.


@DDoS

Actually, ArmorStands have roll now I believe.


return true; // FIXME: This will need changing


This is great. It is elementary enough that it all works with `lens-family`, of course; if incorporating it into `lens` makes too much trouble, there is always the possibility of a `Lens.Simple.Tutorial` !


Thank you!

However, it looks like you copied the entire `time()` method, which ws designed to handle both start and end times.

It would be better to add two new methods: `endtime()` and `enddate()`. Both methods would just call their generic equivalents, but with the right `$args`:

```
$endtime = $this->time(
  array(
    'start' => false,
    'html' => $html,
    'filters' => $filters,
  )
);
```


Everything should use Vector3d.


Looks like we need to install a white-space trim plug-in in Sublime for you. Hit me up when you get it, and we'll get this taken care of.


Hi, could you clarify why we need to remove this ?


I'm getting this undefined constant being spat out, does this actually cause problems? Cos you've not released a newer gem containing this yet...


This loose the borrow parameter of SymbolicOutput. I think it is better to keep it, (I don't know if it is used at this point, but even in that case, for consistency, I would keep it).

I think the best long term way of doing this, is to add a clone() method to that class with a parameter variable that allow to change the parameter when doing the clone. If the parameter isn't set, it copy the member variable.

That way, if we add other attribute to that class, we will be sure to have the copies well done.


This is me being indecisive again, but I think let's change the sizes on checkboxes to be medium, large, and extra-large. That way the default on all skins (buttons, items, etc...) is medium instead of small.


lol


I think MC just gives it out as independent coordinates. In BlockRay I convert the Euler angles to a Quaterniond and indeed yaw is the y rotation. So they are out of order.


Intanto.. prego! Con tutto quello che lurko, è il minimo dare una mano.

Alla parte di ricerca non ci ho ancora pensato bene, ma i requisiti che hai scritto mi sembrano buoni.
Comunque a naso penso che utilizzando solr la parte di indicizzazione/ricerca vada messa su un'altra VM.

Non sono molto d'accordo sul fatto che un unico script sia più facile da manutenere, ok se è molto breve ma avendo da gestire la connessione al db, il parsing del contenuto per l'embed e l'output delle pagine html penso sia meglio separare il codice, specialmente l'html.

Per migliorare il consumo di risorse sarebbe anche da limitare il numero di query al db, adesso ne viene fatta una per ogni commento.

Il branch su cui sto lavorando è qua https://github.com/crazyhg/hookii/tree/refactoring


typfout!


@edmistj ASS, BUM, BUT, COK, COQ, CUM, DIC, DIE, DIK, DIQ, FAG, FAP, FUC, FUK, FUQ, KOK, KUM, LSD, NIG, PIS, PRN, SEX, SUK, TIT, VAG, WTF, XXX

This list has a few more but those are the most offensive:
https://github.com/klhurley/ElementalEngine2/blob/master/Common/Databases/BadWords.dbx


I don't understand the last sentence. Do this mean that the maker still reference the old shared var?

From memory, I think it only work when shared var are an input of the graph and not in the updates. If that is right, doc this restriction (and make sure that you raise an error if there is an update of the correspoinding shared var)


Blog site


Could you put parentheses around the appropriate parts of this - I'm always reluctant to have && and || inline with no parentheses. Thx.


Coffee blog


s/WeaveDNS containers/weaveDNS


kan ook wel weg :p


Huh?  What's the reasoning for this???


Sorry, moved my comment to PR itself.


When I deployed to a jellybean device, the focus no longer worked because it had a density of 1.5, but was getting reset to 1 due to these lines.


This seems to be separate from the rest of the code - can we integrate it with how the rrds are generated somehow?


priority: required?
required pl. a bash, a grep, a gcc és ezek.
Ez max. optional, amit viszont nem kell megadni a priority fájlban, mert ha nincs priority, akkor az automatikusan optional.


also fixes blank text when hosting a game
very strange issues, solved by a very strange solution


openshift `healthz` should always be about openshift's health.  When we have internal kube we should include both.


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


@palinek Maybe this string doesn't need localization. It's useless for a non developer.


AWESOME


You can simplify this loop:

````
reads.removeAll(readsToRemove);
for (final SAMRecord read : reads) {
    spanIncludingReads = spanIncludingReads.union(genomeLocParser.createGenomeLoc(read));
}


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


raise an error. move exist_svs out of the loop.


typo, maker.inputs


shouldn't this get called from somewhere?


you forgot to include TokenizeTask.cs in the commit.


хорошо учтем


Is it a good idea to not throw an error? in this and the next case below?


Looking again at i915_gem_set_domain_ioctl(), they also enforce read_domains == write_domain when write_domain!=0. Since it looks like we don't care about changing read_domains, I think we should remove read_domains from the API and simply update gem_obj->read_domains with the value of write_domain.
```
/* Having something in the write domain implies it's in the read
 * domain, and only that read domain.  Enforce that in the request.
 */
if (write_domain != 0 && read_domains != write_domain)
        return -EINVAL;
```


Changes all uses of ```getLocation``` to ```getSpan``` and remove this method.


If we're running standalone there's not a good use for Kube-style healthchecks, is there?


[WIP]


Typo "Adiitional" -> "Additional"


That's a bit disappointing. Well if the data set is to large, the user
still has the possibility of splitting it and use warm starting.

On 2015-06-25 06:53 AM, Toby Dylan Hocking wrote:
> interesting idea but if there is a data set so big than probably it is too
> big for R. for example Mark's test data sets are not that big. I would
> recommend to just use regular ints for now (don't worry about unsigned
> ints
> or size_t).
>
> http://stackoverflow.com/questions/10640836/max-length-for-a-vector-in-r
>
> On Wed, Jun 24, 2015 at 8:32 PM, Ishmael Belghazi
> <notifications@github.com>
> wrote:
>
> > @tdhock <https://github.com/tdhock> This bothers me. The package is
> going
> > to deal with large arrays so int could potentially not be enough to
> index
> > the arrays. The recommended C pratice is to use size_t instead of int.
> > Still I can't find a way to pass the data as unsigned integer from R
> to C.
> > The only storage mode R relevant storage mode that R allows are
> "integer"
> > and "double" (
> >
> https://stat.ethz.ch/R-manual/R-devel/library/base/html/typeof.html). Any
> > ideas?
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/IshmaelBelghazi/bigoptim/commit/6858d198f100ad0c8d4eaad12a3c0f270f9b5552#commitcomment-11850994>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/IshmaelBelghazi/bigoptim/commit/6858d198f100ad0c8d4eaad12a3c0f270f9b5552#commitcomment-11858459>.
>


How are we supporting legacy 45 stuff without this call?


If our only concern here is revocation, do we really need a separate method? Is there anything wrong with using `makeToken()` instead, after appending an additional `tokenValid` requirement?

In my understanding, the only reason to ever prefer a `parentToken` provider is in the case when the original token has some additional mutable state beyond revocability, e.g. a role assignment in the case of UiView tokens.


Change all uses of ```getExtendedLoc``` for ```getExtendedSpan```` and remove this method.


Indeed that is what I did. I think you are right, I'll see if I can solve it that way. Thanks.


"search for [...] the secret tracking code of a trackable" This only works for TB/GC not GK


gtfgtfrtr


This is awesome!!


testchat


I am not ok with this. A swap size smaller than available RAM breaks hibernation.
Please either revert or make it configurable.


Instead of doing this fixup after the swap, could you do the if above before creating f_cpy? This way we are more future safe in case of other change in Function and won't need the fixup.


Same as above, maxSwapSizeB could be smaller than the installed RAM size, which breaks hibernation.


Looks good :+1:


rax retry tests


I find this JSHint hobbyhorse particularly annoying, since the format it complains about seems clearest to me. But I must be deluded, and will toe the line.


umm. just invert the `Less` predicate?


To clarify: I don't mind smarter swap size calculation, and I am not against a swap smaller than physical RAM in principle, I just believe that if swap is allowed to be smaller than RAM, this should be an option and not a hardcode.


I think you can replace everything after "is ignored" in the first sentence with "is now ignored", and remove "You should" from the second sentence?


Started...


Can you please also update the javadoc for this function so that the locking semantics are clearly defined? This will enable callers to understand the expectations on them for releasing locks.


change name to ```equalsIgnoreReads```; after String's equalsIgnoreCase. In any case it should be equals and not equal.


#1
mojombo#1
mojombo/github-flavored-markdown#1


some ObjetGuids are stored as uint64
also gossips are hardcoded


i am not sure to understand, maybe you are talking about embed stuff ? embed just add the content into the viewer in iframe (so you can visualize html, asp, php, etc ... if you add them into embed parts, but you can't edit embed files.


Thanks for the explanation. I agree that enforcing atomicity of block-size updates in tee-supplicant may be difficult.


It might be worth initialising this with a capacity of `len(totalSpacePerPeer)`


I'll make this adjustment this evening.


Are you planning to try Lucas suggestions ?

If so, I will wait that you are finished with the suggested tests,
and in any case will wait for them to be in the master before merging anything.


perhaps its phrased better with "list containing an override for the empty hand".


Agreed, I'll change this.


And perhaps we add two more methods:

`starttime()` and `startdate()`.

And to make it event more user friendly, we could turn it into 8 new methods:

```
starttime()
starttime_html()
startdate()
startdate_html()
endtime()
endtime_html()
enddate()
enddate_html()
```

Where all '_html' method return the HTML equivalent of the corresponding field.

Jeroen