Here are my comments for the french translation
Key    : 'List all autosign entries'
Change translation to: 'Afficher toutes les entrées signées automatiquement'

Key    : 'required for managed host that is bare metal, not required if it's a virtual machine'
Keep 1.10 string: 'requis pour les hôtes gérés qui sont des serveurs physiques, non requis pour les machines virtuelles'

Key    : 'Force a Puppet agent run on the host'
New Translation: 'Forcer une exécution de Puppet agent sur l'hôte'
Comment: rewording

Key    : 'List all users for location'
New Translation: 'Afficher tous les utilisateurs d'un emplacement'
Comment: Lister -> Affichier

Key    : 'Create :a_resource'
New Translation: 'Créer : a_resource'
Comment: missing space

Key    : 'Update :a_resource'
New Translation: 'Mise à jour : a_resource'
Comment: delete double space

Key    : 'Delete :a_resource'
New Translation: 'Suppression : a_resource'
Comment: uniform style

Key    : 'Unlock'
Back to 1.10 string: 'Déverrouiller'

Key    : 'Lock'
Back to 1.10 string: 'Verrouiller'

Key    : 'in %s'
Back to 1.10 string: 'Dans %s'

Key    : 'URL hosts will retrieve templates from during build (normally http as many installers don't support https)'
New translation: 'L'URL de récupération de modèles des hôtes durant la phase de construction (normalement en HTTP car beaucoup de programmes d'installation ne prennent pas en charge HTTPS)'
Comment: delete trailing space

Key    : 'To enable a provider, either install the OS package (e.g. foreman-libvirt) or enable the bundler group for development setup (e.g. ovirt).'
New transltion: 'Pour activer un fournisseur de ressources de calcul, soit installer le paquet logiciel (p. ex. foreman-libvirt) ou activer le groupe bundler pour les configurations de développement (p. ex. oVirt)'
Comment: merge 1.10 and D.S. translations.

Key    : 'e.g. mail'
New transltion: 'Ex : mail'
Comment: space added to 1.10 translation. "mail" is not translated because it is the LDAP attribute name.

Key    : 'AuthSource|Onthefly register'
Keep 1.10 translation: 'Enregistrement à la volée'

Key    : 'Full trace'
Keep 1.10 translation: 'trace complète'

Key    : 'There was an error listing VMs: %s'
New translation: 'Il y a eu une erreur en listant les VM : %s'

Key    : 'SCSI controller'
Keep 1.10 translation: 'Contrôleur SCSI'

Key    : 'Eager zero'
Keep 1.10 translation: 'Provisionné à zero'

Key    : 'Edit Template'
New translation   : 'Modifier le modèle'
Comment: capitalization


WORKS ON MY COMPUTER


erledigt.


Il processore grafico è pronto per l'utilizzo ALLA FINE di Initialize, SUBITO PRIMA CHE VENGA CHIAMATO L'INITIALIZE DEI COMPONENTI!! (quindi i componenti possono crearsi gli spritebatch e ion generale fare cose grafiche nel loro inizialize())


Typo "prasing"


But plural is more readable in `for concentrator in report.concentrators`.


Resolves #44


Logger hinzugefügt.


E' un giro un po' strambo, poi ve lo spiego dal vivo


Newline


I think it should be possible, but would require extra bookkeeping in Dialyzer, hence it was not added. Unfortunately it is not possible to automate the inheritance of the callback either...


Sublime prefs -> `"ensure_newline_at_eof_on_save": true,`


Fixes #78 ?


I checked on a hand-made toy example and in [documentation](http://git-scm.com/docs/gitignore) that these entries are redundant.

Entry `**/*/target` ignores dir/file named `target` wherever in the tree directory it is a subdirectory of an existing directory while `/target/` ignores dir/file `target` in the root directory. Both of these entries combined make `*/target` redundant. What's more, instead of these two entries (`**/*/target` and `/target/`) you could just use `target` entry which will ignore dir/file `target` in the whole directory tree.

However, I think that ignoring `target` dir/file wherever it is in the directory tree is not our goal and we want to just ignore target directories in all sudirectories of the root directory and `iis-workflows` directory. In such case we should place two separate `.gitignore` files in root and `iis-workflows` directories with the same content:

    /*/target
    /target

The situation is the same with `build` and `bin` directories below.


E' un po' scomodo come sistema; a questo punto è meglio fare lo sfondo come estensione di DrawbleGameComponent (ma bisogna stare attenti che venga disegnato prima del GuiManager in questo caso)


I wouldn't lowercase a model name, as information gets lost (there is no way to recover 'camelcase' to 'CamelCase')


Shut up bitch I don't care about buildbattles anymore lol


Hey @bausmeier yup, it would seem so. Thanks! I didn't know about that.


Shut the fuck up loser


task_args is a misnomer, they can be like --scheduler-host etc as well, right?

Make --help always be a known arg and avoid writing filter methods?


please add to step 3
Import Package->Custom Package...

That compiled unityproject exists here: Unity scripts package for QTM realtime streaming using the .Net realtime Sdk (http://www.qualisys.com/download/QTMUnityRealtimeStreaming.unitypackage)


This is edit in Branch 2


I'm not sure about dealing with prefixes. Or maybe you can use the Vendor module from PostCSS: https://github.com/postcss/postcss/blob/master/docs/api.md#vendor-module


Specific note for the key: 'IDs of associated partition tables'
1.10   : 'IDs des tables de partitions associées'
D.S.   : 'ID des tables de partition associées'
New translation: 'ID des tables de partitions associées'

Comment: "ID" stays without "s" on french. But partitions *do* require an "s" at the end. Partitions' table always contain more than 1 partition and their number can be easily counted.
Ref. http://www.logilangue.com/public/Site/clicGrammaire/AccordApresDE.php


@ddiss Integrated yours and Filipe's feedback in commit 44c16cbf633d. Thanks!


I have basically used the existing Cassandra Storage as a base template and just adapted the parts where data gets written to the database.


dakkar: I'm unclear on what benefit I would get by adding an XS dependency? The code above works fine for all versions of Perl for which dbic is supported. If there's a bug with it that `Sub::Name` would fix, I'd be delighted with a use case and patch :)


Damn it. Yes, yes it should :)


The <code>...
Generally something is wrong with this sentence. Maybe try another way :
... is used to create a Motech event. The event payload contains details about user enrollment


Weil es momentan wohl leider keine andere Möglichkeit gibt die momentan im Focus stehende ZElle programmatisch zu setzen. Siehe Bug-Report:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=198260
Sollte ich das noch im Code vermerken?


sorry mod keizer


Similiar to previous comment. I think you should use the 'payload' word and remember about 'the' before class name


Uhm.. 0.4 < 0.34


you should use here article, a or the (i'm not sure which one is better in this example)


the same


Thanks Thomas. However, it looks as though the chattr +/-c mechanism for enabling/disabling compression on a per-file basis still isn't documented.


Ja bitte. Sowas spart nervige Rückfragen ;-)


Hi Martin! In my PR, I also removed the line from the Perldoc that shows the TicketNumber in the response data; the web service only returns the TicketID and an ArticleID if one was created.


`\luatexluaescapestring` is an expandable function. Temporarily `\let` it to `\scan_stop:` is dirty trick to protect it in the following `\cs_gset_protected_nopar:Npx`. Since it is not appropriate to be expanded at that time.


a Motech event. Try to stick with one convention MotechEvent/Motech event


How did you choose these values? What properties need they have?


The thing is that in some sections it is necessary to have it in each section (eg plugins, display settings, etc.) because they do refer to different parts of code.
So in settings it is done the same way for consistency reasons not because it is necessary...


@jsenecal: What do you think? What should the default behaviour be?


Hey, would love to hear about your plans for minidumper. Anything I can do to help?


why just not a `_.isPlainObject(req.params) ? req.params : {}`
easier to read


use an article before 'Motech event', here and in the other examples


That is indeed better, I can change it


이건 잘못된듯


Please add ``.idea``


Do we even need to keep this copy of the attribute? Couldn't what follows just be `if (opts.os === 'win')`?


Should I change all of them?


Should be called "Initial commit"


I have updated the PR


Diese Prüfung kann in Contao 4 doch gar nicht funktionieren?


Thanks for adding this!


seconds?
minutes?
hours?

please add doc


It is used in other places in `device.js`, so we do need to keep a copy.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (806 of 806 tests passed)

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


I thought my test opens multiple connections (as it starts multiple node clients)...?


They are large primes.


how can something be "half MVC" because of the "h?" in the regex


**Functionality - Critical**
You should check if the word already exists, what happens to duplicate words?


I don't like *italicising* the quote marks...


Yeah hardcore noob mode


@josemarluedke Any reason for it? It is breaking the addon for me.
Check the dummy app as well


Maybe this could be written more formal, by rewriting sentences containing "we" etc.


Guess what else we missed! This was used by Jenkins to upload git builds to the CDN. Accordingly those builds haven been broken since: http://jenkins.jquery.com/job/jQuery%20UI/


I got two errors when trying to build https://github.com/cmangos/mangos-wotlk/commit/f37049cc309bb9b3e96278073bf9b6067503192a

```
/cmangos/src/scriptdev2/ScriptMgr.cpp: In function 'void InitScriptLibrary()':
/cmangos/src/scriptdev2/ScriptMgr.cpp:105: error: '_MANGOSD_CONFIG' was not declared in this scope
/cmangos/src/scriptdev2/ScriptMgr.cpp:118: error: '_MANGOSDCONFVERSION' was not declared in this scope
```


Just making things a little tidier after updating.


Which I think we can replace with the new requirejs task. Trying that...


looks good, just a question: should this not just be microsToMillis( .. )? or is their a reason for the UUID naming?


GroundDomainUpdate (mmatts) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1340 of 1344 tests passed)

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


Are you going to make a PR for this fix?


ao invés de registrar o event em cada player que logar, coloque para registrar ao entrar no evento (tp)


> This prop not really used. it's only for frameworks/ex/camera2/portability/src/com/android/ex/camera2/portability/CameraAgentFactory.java  You can set any value 0/1/2 and camera will work anyway. msm8226/8974 not support v2 api, even 5.1 blobs and v1 forced anyway.

5.1 "feature"


Also, this issue https://github.com/cmangos/issues/issues/740 still exists (do not break build, but very annoying but outputting thousands of warnings).


**Code Quality / Testing - Critical**
Use absolute referencing so that tests can be run from anywhere. This line should be `include __DIR__ . "/../src/rankingSystem.php'


Doch: https://github.com/contao/core-bundle/blob/master/src/Resources/contao/library/Contao/ModuleLoader.php#L58-L68


`_build_parser(...).parse_known_args(...)` is called so many (4) times through very confusing indirection each time through a different method(s) that alter state... Hard to grasp the order, or how each invocation's semantics differ.

Prefer to lay those 4 steps out in one flat method (__init__ itself perhaps), and comment on the relevant state invariants around each?

(I think we definitely don't need so many as 4 re-parses here yet, but that would be another PR perhaps.)


done


OK, then :shipit:


essa parte não precisa ser a cada player logar, pode colocar ao iniciar o evento.


remote


thanks @motvicka we merged and tagged it with 0.1.1


remote comment


My understanding is that 'deviceorientation' event can still be true or false, with a preference for false (i.e. not using the geomagnetic data). The implementation can still decide to provide absolute orientation if relative is not available or the resulting data is more accurate.


This commit is being reviewed as a part of review issue #63.


Thanks for the help. Documentation fixes (for the API) should be applied in the code of [telepat-api](https://github.com/telepat-io/telepat-api). Every API endpoint has a doc block.


it's not uncommon for an xbt file to be larger than the sum of the individual images.
it's not a compression tool, but a conversion tool to put the images in a format that can be loaded faster by the gpu.

according to jm (iirc) only very large images should be kept out of the xbt file.
i'm not sure where that line should be drawn exactly.
for now, i've set compile="true" to all packs with images smaller than 512x512 px.


When I tried to run the docker file on AWS received an error similar to "ulimit: open files: cannot modify limit: Operation not permitted".  It looked similar to the issue at http://stackoverflow.com/questions/24961576/docker-error-while-creating-couchbase-ulimit-open-files-cannot-modify-limit and moving the ulimit command from the startup script to the docker file fixed the problem for me.
I am very new to docker, neo4j and bash scripts, so what I have done might be sub-optimal :-)


Thanks.


额。。这里的Markdown语法写错了


Where does the core() config class remark come into play? Seems like a note on implementation detail that's very far from the implementation itself - remove?


In my view 'absolutedeviceorientation' would always be absolute=true (hence the name). If the implementation is unable to provide absolute orientation a one shot null event should be fired (analogous to the regular 'deviceorientation'). Does that make sense?


That fixes the build (which doesn't look too bad either: http://swarm.jquery.org/job/1295 ), but we still lost `dist/jquery-ui.css`.


I think these need to be `check_call([PYTHON, EMCC, ...` in order to be Windows safe, e.g. https://github.com/kripken/emscripten/commit/f13a1ea175b06dcbb110761d2f2dd00e27155279


It is, this range should allow any of the betas or rc releases to be used. See http://semver.npmjs.com


you're the expert here :)
there was one that became 40mb->880mb hence I added this.


After changing the axis colors we need to change also the terminal joint color from blue to something else. Otherwise it is difficult to distinguish it from the axis in some configurations (for example for the default 2R robot)


Added in Tests


não precisa disso: creature:isPlayer(), pois só esta registrado em players ....


whoops, that was probably me... thanks for fixing! Ack


não precisa disso, é só colocar para fazer isso ao iniciar o evento.


Also, looking at our needs internally, I think it would be interesting to extract method from handle() allowing to easily serve a `Resource`

Something like :
```
    protected void serveResource(RestxResponse resp, URL resource) throws IOException {
        resp.setLogLevel(RestxLogLevel.QUIET);
        resp.setStatus(HttpStatus.OK);
        String contentType = HTTP.getContentTypeFromExtension(resource.getPath()).or("application/octet-stream");
        Optional<CachedResourcePolicy> cachedResourcePolicy = cachePolicyMatching(contentType, resource.getPath());
        if (cachedResourcePolicy.isPresent()) {
            resp.setHeader("Cache-Control", cachedResourcePolicy.get().getCacheValue());
        }
        resp.setContentType(contentType);
        Resources.asByteSource(resource).copyTo(resp.getOutputStream());
    }
```

Here, I'm just not sure about the call to `cachePolicyMatching(contentType, resource.getPath())` as I'm not sure if `resource.getPath()` will return the same path as the initial `relativePath`


bacon?  Stupid autocorrect...  That should have read:

catch counterparty daemon errors and retry later with backoff.


Hmm, I'm not sure treating a negative value practically as a "assume output buffer has infinite space" is safe, since the user could pass length as result of some computation that happens to go negative. Could this just be signed, and negative values denote no space to write to?


do you mean moving the "CompileDefineFlags", "CompileFlags" and "CompilerOptions" out of the loop ?

Or is there more to the simplification you have in mind ?

Iknow it only supports a single level, but I felt it was good enough for now, and putting the logic inside the 'GetLastFolderName' function allows for a change at any point later without too much trouble.

I tried using the full path and it worked, but I'm not sure it's much better.


That "[type=item,name=tritanium_ingot,mod=mo]" and another like that doesnt work!


b13ee9da2aff4fa104185e0050697204d4fa6742 should improve it a bit.


>people may wonder if it is necessary to submit changes before changing section
while this is true, having an incosistent behaviour is much worse than hitting submit once to much imho...


yes, it was a copy/paste error


essa gStorage é só pra guardar valor das duas de cima?!


Version for Bower is deprecated: http://bower.io/docs/creating-packages/#version.


ah yes. It has to be [item name=tritanium_ingot mod=mo]


@dsanders11 thanks for the background information.  I realize my question was a bit loaded... When I say "help get functionality upstream" I mean less from a coding perspective and more from a use-case and documentation perspective. Would you be opposed to continuing this dialog in #91?  Also, would you be opposed to being a test bed for these?  We're even willing to work out a support agreement, although if you're comfortable making custom builds, this may not be a concern.

Hypothetically, this would eventually get your company on an upstream build so you don't have to divert the code. :palm_tree:


Yeah I mean where you have the loop...
My suggestion is to leave the original code mostly unchanged, and just add the loop around the "InputFiles" variable, and put the ObjectList command inside that... Does that make sense?


IMHO it's more readeable in plural, because plural means a list: `report.concentrators` it's plural, then I can access an item by index.


da pra usar: for _, online in pairs(Game.getPlayers()) do
depois online:getStorageValue, online:Player(cid):sendTextMessage


OMG :D


shouldn't the two above be static in `sicslowpan.c`?


Thank you for breaking REST API :(


I'd remove the commented out line


Yep. Will remove the h?


```php
elseif ($object instanceof Ticket
    && ($C = $object->collaborators->filter(array(
        'user__emails__address' => $mailinfo['email'])
    )->first()
) {
.....
}
```
`$object` and a check, right?


I see your point. To me though I prefer to have two different buttons if they are needed. As I said this is my opinion, and if you think it's better in another way, no problem to revert the changes !
In any case I'll have again a look at all the relevant pages


My bad. It's a thread...


Based on your usages of this, I might have even gone one step further. Helpers are always used in views, and therefore it's ok to have view code in there. I might have written a method like this:

```
def render_topic_name(topic, link = false)
  output = ""
  output += link ? link_to(topic.name, topic) : topic.name
  output += "<small>private</small>" unless topic.public?
  output.html_safe
```


OK. Can i change it in russian "info" files?


@PaoloRotolo good work!


Warface VN update
http://warface.goplay.vn/news/details/2850/ra-mat-phien-ban-warface-big-update-3-3

game_version
"1.1.1.279"


@ivank I think only the Composer badges from poser should use the package name, other providers work with the github owner/repository name.


como você não vai usar as funções do evento em outras coisas, seria bem melhor chamar por dofile a lib de cada arquivo do evento .... ficaria bem mais "leve".


Sure. I'll fix it in the English version


我碰到的問題是下載連結（/firefox/download/latest-win.html）
browserSync / connect-modrewrite 會把 `*.html` 轉去 `*.shtml`，所以不再需要 .html，
這時 server (nginx) 找不到 html 就會錯誤，所以才 rewrite 到 shtml。


Hi,

After pulling the latest from master, rebuilding and caffe train on my LMDB I was getting this error:
```
I0917 12:13:07.674737 26390 net.cpp:433] data -> data
I0917 12:13:07.674770 26390 net.cpp:433] data -> label
F0917 12:13:07.675629 26393 db.cpp:20] Unknown database backend
```

Turns out my old Makefile.config is not backward compatible with this PR (`+USE_LMDB := 1` was missing).

Solved it easily by copying again the new Makefile.config.example to Makefile.config, and making my regular modifications.

putting this comment here for anyone who might encounter the same issue.


Why use babel，it's toooooo large.


Yeah, you might be right ... I was not reading your test code carefully enough.

But I looked at your changes to `HttpService` carefully and you know what: Your approach is more or less the same like mine! This is because a `Source`, like any other processing state in Akka Streams, is just a blueprint of a stream – it's cold. It can be reused for multiple stream executions and only becomes hot through materialization which happens when completing the request with the source. So, you create a stream for each request, too.

Having said that, I indeed think that building (cold) `Source` once and storing it in a `val` is a good idea, even though building the blueprint should be a lightweight activity.

Does that help?


@kidwm 是啊。


OK!


I'm going to refactor them. Thanks


Yes, I merged it 1:1.


`2.0.3` is better.


and can you make star_name, star_prefix, star_suffix translatable


**no-undef**: "TestUtils" is not defined.


**no-undef**: "sinon" is not defined.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "expect" is not defined.


**Eslint**: 'return' outside of function


**no-undef**: "componentHandler" is not defined.


**new-cap**: A function with a name starting with an uppercase letter should only be used as a constructor.


@kidwm 贊成，比較清楚。


**Code Quality - Awesome**
Great use of the least known ternary operator in PHP, and keeping the code _short_.


Why might you want to always pull changes manually rather than having Git automatically stay up-to-date with your remote repository?


Checkmarx scan results 17 Sep 2015 15:35 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
5
11
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

90003e56d4511b9e1f1b097c10578cb41f77ca30	@CxTest


Actual test is missing?


These could just be functions that take `iter` as an argument.


control the issue


Hello @pamval
Please review and update the description of our plugin


Looks like this is called with `buf` pointing to a fragment rather than a buffer of size `UIP_BUFSIZE`, am I missing something? Is this guaranteed to not overflow because we only access the IP header? A bit of clarification in the code is probably needed :)


:lollipop:


i fixed this problem in main repo: 14bda9865b80b5a346c7893cde99a51ef8e2656f


I really like this .editorconfig thing :+1:


You should copy your config.ini in a config_local.ini otherwise it will erase other's confi files


Here it is not checked that we are in batch mode. Does that mean it will be skipped also when we are not in batch mode?


This commit is being reviewed as a part of review issue #63.


This is the part you shoud have PR


typo


I forgot that it's XS ☹
The idea is to have sensible names in stack traces, but the caller can always call `set_subname` before passing the coderefs through. So yes, ignore me 😜


I respect your opinion but i also ask you to respect my work over the last years/months to make the look and workflow of redmatrix/hubzilla consistent...


Honestly I don't think that this is correct, but at least it is the solution for the file reading.
I started looking at the layout code (which calls this function), It seems to be a little bit overcomplicated. Too many functions with boolean arguments that call other functions depending on their values.

But I would say that the principal problem here is that the same function does two things -- resets defaults completely and resets one variable when sigma is changed. These actions must be separated  imho.


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


test a commit


Just reminder that the test "failures" here are expected because the models use the setpoint manager that was changed.


TeamCity Xbim :: Xbim Web UI :: Xbim Web UI [Build 19](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=178&buildTypeId=Xbim_XbimWebUi_XbimWebUi) outcome was **SUCCESS**
Summary: Running Build time: 00:01:17


più o meno ho capito. La cosa che mi puzza però è che la classe quantum tali metodi li possiede già e mi verrebbe da implementare flussodigioco come interfaccia di quantum. Ad ogni modo, al prossimo giro cerco di capire bene le classi gamesystem e flussodigioco per capire ancora meglio.


does this need to be unique_ptr<char[]> so that delete [] gets called?


linux build is broked currently i am trying to figure out what's happening. About warnings i'll send that for later.


Solves #14 (for real this time...)


is this a CMake 3.1 feature? right now the minimum is 3.0, though there are some 3.1 features like this that I'd love to use. maybe a poll on the mailing list to ask about a change


Hi, it would be nice if you merge this one into p700/710/720 -  The old code won't updateLoncharCounter() on pasting text with mouse.


Do we need this explicit directory creation?  Shouldn't the temporary directory already be created by R?  Or is this a weird windows 'feature'


oh, and for this purpose you can use the target named something like OSVR cxx11 flags right now


MiscReleaseFixes (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1874 of 1874 tests passed)

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


With Redis 2.8.2103, I'm unable to compile `RedisServer` with `Release` and `x86` target configuration.
The error `Error	6	error C2065: '_WIN32_WINNT_WIN8' : undeclared identifier	C:\Users\bodet\Documents\redis-2.8\src\Win32_Interop\Win32_FDAPI.cpp	198	1	Win32_Interop
` occurs.
If I replace this line with:
`if (true) {`
The compilation works without errors.

NB: Compilation was OK with Redis 2.8.2101


Great job! this is really well structured!


mi sono limitato a riordinare il codice, non ho aggiunto istruzioni qui.
Cmq se non sono necessarie entrambe provo a rimuoverne una.
La scocciature di Components è che non c'è un metodo get


May as well do a =delete on those.


But why deprecate this? How can one know what server failed in the cluster?


Jap, isch no a Baustell wiet gsehsch - modernizr wer z'ziil ir endversion ;-)


think the return type is usually by ref, though I don't know if it matters for the purposes of disabling assignment


Thanks for clarification! Btw. can you say something about scalability of SSE using akka-streams/actors (serving thousands of clients concurrently)?


OK, my bad, I didn't notice `(struct uip_ip_hdr *)`, things actually are pretty clear as is


Is this the first hackathon commit of the day? Awesome. :smile:


=delete again presumably.


I'm afraid I have to agree with @git-marijus. I rather have people to submit their changes twice, than not at all. The latter will create more support questions.


Haha Sebi, gisch alles :wink:  Wie du richtig gseh hesch ischs würkli no a Baustell :wink: Sobaldi Ziit hann wirds fertig...


Seems to be a generic CM problem:
https://gist.github.com/tuankiet65/121f723194558913a6ca

Does anyone have a solution?


Do not forget the Namespace!


srsly we had an unneeded include in there? oops. presumably I was thinking of wrapping callbacks at some point.


same deal with the delete, think the array thing is needed


yes, sure! Sorry, tat wasn't my intention, please revert the change, don't worry!


This revision is clean.


You cannot tune against master: test deleted.


Oh -- the tempdir exists, but tempfile returns only a file name for a file that most likely doesn't exist yet. We use that name as libpath and really need to create the directory, otherwise `.libPaths()` just ignores it.


To save calling calculateObjectSize (which is fairly expensive) you can just use sz += line.length -- it's a good enough for a rough estimate.


again the CMake version, can just use that target instead, look in the example plugins CMake to find out what its name actually is -can never remember myself and I'm on mobile right now so looking it up is a pain


I'm curious as to why our vfs modules were different sizes. I compiled by your directions in the INSTALL file, but your module was 43.3kb, whereas mine was 42.1kb. Also, new in 4.3 and might be beneficial to Greyhole, "New FileChangeNotify subsystem". Check out the release notes for Samba 4.3.


Ah yes you are right, wasn't thinking through what we were trying to accomplish with this function, sorry!


The archiver is called Compress and is available on the CC forums. By the way, Aware was a project that was meant to be like Siri: "do this", "open FileX", "what time is it".


You don't need to put these 3 lines here; jQuery is already included in the project, and any other css or js files in the 'client' folder are automatically loaded for all client pages.


`"signed in"` would be a good description to use on line 29. "visiting own profile" is covered by line 34's description.


Looks pretty nice! I'd bring up the topic on the mailing list before merging, but I think this is good stuff. And I'll be really happy if it ends up giving me permission to use the rest of c++11 as supported by msvc2013 in the ClientKit c++ bindings


Si. In realtà volevo fare una cosa del tipo
 foreach (IGameComponent component in Components)
+           if (component is DrawbleGameComponent ) ((DrawbleGameComponent)component).Draw(gametime);

ma ci stavano un po di casini con varie classi e ho lasciato stare.

Per i commit io vi consiglio di aggiornare prima e dopo ogni sessione di lavoro. Così stiamo tutti più o meno sulla stessa versione ed è più facile fare i merge.


What's a domain hack?


Is there a way they can "fake" purchasing a domain and using one?


```go
if invitation != nil {
	return
}
```

saves you one indentation level.

Also, panicking was fine in `init`, but is not anymore in the handler.

Please make this function `func() (*template.Template, error)`, which will cache the template if it is successful.


excuse me. I use “PocketMine-MP_1.6dev-31_php7_8768b7fd_API-2.0.0.phar”. But ti shows that:‘You  must use PHP >= 7.0' and ’Please use the installer provided on the homepage‘. So here is my trouble:What is the homepage' url? Or how can i download the installer. I am a user of Windows8.1.
By the way, I am not a English native speaker. Therefore I hope you can get the point i said. :D


Line 31 can be removed, and make use of the existing `@user` instance variable on line 32 in `login_as(@user, ...)`


https://github.com/nickycadavillo/nickycadavillo.github.io/commit/9c639827c2ecb88e5b48f6f508ffd351e1c66289
Really great. But I was hoping to see more personal frustration..lol..But I think it's good you are writing this stuff to keep it fresh on the radar screen.


Solution alternative :

~~~python
def get(self, key, default=None):
    return self._values.get(key, default)
~~~


This represents a patch.
I'd like to better inquire the problem though, in order to find a definitive solution.


At least add a `FIXME` here. Tests of this package will break depending on some files in it's parent directory. This is not good. Tests should have their own template files.


oh, and apparently the preferred new style for stuff like this is auto buf = type (parameters) though I don't recall if that applies to unique ptr. I probably should pull in a make unique implementation if I can figure out how to detect if it's needed with just the preprocessor, or maybe just stick it in a namespace.


EU/NA/TR update
"1.11300.3691.32100"


What uses `/`?  Note that with `/usr/bin/atomic` we've started a convention of `/host`.


Right, i reviewed again. On any capable system the 3.7 policy was loading the autorun library (which in turn loads files in services/autorun)

"inputs" slist => { "lib/$(sys.cf_version_major).$(sys.cf_version_minor)/autorun.cf" };

So your patch brings us back to that same state. Thanks.

The autorun lib does have the services_autorun guards before activating the bundles, but we are still stuck loading the policy in all cases. I will open an issue to see how we can best resolve that part.


Could this be cherry-picked into `hotfix/1.8.6`, too, or do you think that's too risky?

I don't see how this would break anything, but you never know with classpath stuff...


@luanluciano93
Foi como eu disse, eu peguei este evento pronto, pois foi funcional até agora, porém não consegui modifica-lo ainda pois estou com algumas prioridades no sv.

Se puder abrir um pull request com essas modificações, agradecerei muito.


`\c_backslash_str` is in `l3str`. When can it be merged into `l3kernel`?


I see this does bring the code in line with the on_failure docs.  However, if we json encode shouldn't we decode again in `scheduler.py` (https://github.com/spotify/luigi/blob/master/luigi/scheduler.py#L997)?

If we deal with this consumer-side we will have to wrap any `json.loads(your_string)` in try/except to support old clients.


Apparently none. Feel free to submit a pull request refactoring this. Thanks!


I think we need to add `capitalize` in addition to `underscore`, `dasherize`, `camelize`. Main reason being that `camelize` uppercases the first letter unless `False` is passed as the second parameter. Then using `camelize` would lowercase the first letter and uppercase the first letter of each subsequent word. `capitalize` would uppercase the first letter of every word. Thoughts?

* camel
* camelCase
* CapitalCase


No, I don't think so. As that is not creating any new network interface.


I'm planning to do this at the weekend. (I'd set up some 'notes' for code changes, and forgot I'd used that here). An alternative of course is `\cs_to_str:N \\`, but that will be slightly slower so clearly better to have a pre-defined constant.


For example with kartik-v/yii2-widget-select2 when trying to set "prompt" in the widget's "options" property


For lines 48 through 57, don't use aria-label or aria-hidden. They are for accessibility tweaking. I suggest just removing them wherever they appear, but you might want to replace them with something else.

Also, on lines 48 and 56, the href to categoryPage doesn't work like that. I suggest just removing both href's.


AFAIK, `\c_colon_str` is not defined :-)


Le variabili $kernel_loc e $rootfs_loc sono definite nel file di configurazione /env/config, però all'inizio dello script ```boot_board``` questo file non viene incluso in questo modo:

```. /env/config```

Sei sicuro che funziona?
Cioè se cambio quelle variabili in /env/config, cambia il default del boot?


mejorado [aca](https://github.com/datal-org/datal/blob/develop/core/daos/preferences.py#L54-L58)


please do not ignore "message", store each of them to list , and then check all messages in UTs with expected list.


This removes the feature not to create DB in case `MYSQL_DISABLE_CREATE_DB` is set.


Just a question, where is the right place to discuss this stuff?


Same comment as for the other case: this will get added soon I hope, sorry for the confusion. (I think it will be useful and clearer than `\token_to_str:N :` - agreed?)


It's Misspelled, man.


Oh, and thank you for making the commit nice and clean and easy to review! Dunno if you used clang format or if the changes were just small enough that you matched existing style, but that was a very nice diff to look at.


A nameserver is like a phone book for the Internet. A server is like a phone, and a domain is like that phone's listing in the phone book. What's missing? The actual phone *number*.

To actually access a domain, you need to know its server's IP address, e.g. 172.20.4.117. These are difficult to remember, so instead we use domains -- generalassemb.ly, google.com, and so on -- which are much more human-readable. For instance, if you type `162.243.23.12` into your address bar, it takes you to the same page as if you typed `jshawl.com` into your address bar.

When you type `jshawl.com` into your browser's address bar and hit "send", that request first goes to a special computer called a Nameserver. Its whole job is to look up domains in the Domain Name System -- the "phone book" of the Internet, which has every domain listed alongside its IP address. It sees that `jshawl.com` is listed alongside `162.243.23.12`, similar to how "Jenny Brown" is listed in the phone book alongside "867-5309".

Once it's found the IP address, the nameserver forwards your request onto that IP, and business continues as usual. This all happens extremely quickly.


hgjgj


Can you put a blurb describing them in here?


> Here it is not checked that we are in batch mode. Does that mean it will be skipped also when we are not in batch mode?

An explicit check would be easier to understand. But its not needed,   if there is no batch request  the "last_new_version" in transaction Is None, so this code is not called.


Oh, I missed the email discussion. Thanks for the heads up! Now that I read it I understand the reasons. The look-up methods are, however, needed because clients like *libblockdev* won't run a main loop to use the signals for object updates. And it won't hold any persistent state mirroring the state of the LVM DBus daemon. So it needs a cheap and easily usable way to get the object paths for devices (PVs), names (VGs) and vg_name+lv_name (LVs).

The ObjectManager API is not really useful because iterating over the all objects it returns, getting their properties and comparing them with the given value is really expensive and quite a lot of code in C.


I personally agree with @jerel on this, and the default should be to not do anything, use the model name as-is.


fixat kommentarer i avsnitt 2, 3.2, 4, 4.2.1 och 10


thanks for the link. I might have a mistake, but `react-select` installed a duplicate react as dependency and caused issue (and worked when changed the version of react in peer deps to 0.14.0-rc1). will try out again.


first version


We need a documentation paragraph here.


Lets leave the push/pop scope around this stuff, so these variables clearly don't leak


And fails to work due to some Byte Order Mark issues. It was thought as a temporary patch, to see that this is really the issue. Unfortunately, it does not seem to be that easy :)


We also need a test for this new functionality.


looks ok. you need to issue a pull request, now.


@ignatov It's needed for proper highlighting todo items inside multiline comments


Nice


Yes. Thank you very much again!


err, huh?


**no-undef**: "describe" is not defined.


**no-undef**: "before" is not defined.


**no-undef**: "expect" is not defined.


**no-undef**: "it" is not defined.


**no-undef**: "TestUtils" is not defined.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "ReactDOM" is not defined.


**no-undef**: "sinon" is not defined.


I guess GitHub doesn't let you place review comments in files that weren't changed in a commit, so I'll place  such comments here.

Please change string `iis-core-examples` to `iis-workflows-core-examples` in `iis-workflows/docs/workflow_description_language.markdown` and `iis-workflows/docs/oozie-installer.markdown` files.


@aharley40 this change isn't warranted, the verb is supposed to be delete. This isn't a recordsrequesthandler endpoint, working out-of-the-box with the records proxy isn't a concern


The variable was just renamed to MYSQL_RUNNING_AS_SLAVE to be consistent with 5.5 and more accurate. This variable is only used internally, so it shouldn't affect anything outside.


Okay, I'll look into it. This may be because I switched to Enforcing on a Permissive build.


change 0  to nullptr


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1344 of 1344 tests passed)

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


@aharley40 we're trynig to move away from using paths as verbs, recordsrequesthandler offers it in order to support form-based interaction but I want it to support proper verbs too for APIs soon


@whatyouhide oh, thank you for the great post! It would be harder for me to implement this without your blog post! :)


The only issue I found is removing `MYSQL_DISABLE_CREATE_DB` functionality -- we would like have option to not initialize DB also in non-slave scenario. Why it was removed? Actually it was already done as part of https://github.com/openshift/mysql/pull/85 ...


did this fix it? even though ur change is better, i was still curious to know why it failed for you guys and not for me.


Oh, good catch


@jgillis @ghorn Have a look at the above lines. Allows you to place stuff in a location relative to the plugin currently in use. Here, I place the JIT headers there.


esta bueno!!


@aharley40 verify that these events still work, i've seen the string syntax for naming functions break down in some ExtJS versions when you use an object with advanced options


great, thanks!


Thanks for the offer. I may well take you up on it for db2/sybase testing. For now, I seem to have the oracle docker image running on travis by retrying 2 or 3 times.
Build is green. Just need to sort out the todos and we have an Oracle driver.


nope keeping describe as it lets you specify a particular stack name. [listStacks](http://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/CloudFormation.html#listStacks-property) does not..


@aharley40 test these too


But two new lines!


Could this be elaborated, and/or point to the doc for ChooseHashFunction.


Ok,

On Thu, Sep 17, 2015 at 10:03 AM, Andres Vazquez <notifications@github.com>
wrote:

> mejorado aca
> <https://github.com/datal-org/datal/blob/develop/core/daos/preferences.py#L54-L58>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/datal-org/datal/commit/90c776e4ae25d48ef0f079613a64cfc456809a34#commitcomment-13286326>
> .
>


@esteele Can send me the attributes you moved to the registry after the sprint, so that I can update the docs?


@tisto I think this is going into the right direction, no?


#1086 will now handle json encoded or non-encoded error messages.


This is the script I used to build:

```
export GREYHOLE_INSTALL_DIR="/home/gb/Greyhole"
export HOME="/home/gb"
cd samba-4.3.0
    if  [ ! -f source3/modules/vfs_greyhole.c ]; then
        patch -p1 < ${GREYHOLE_INSTALL_DIR}/samba-module/wscript-samba-4.3.patch
        ./configure --enable-debug --enable-selftest --disable-symbol-versions --without-acl-support --without-ldap --without-ads
    fi
    rm -f source3/modules/vfs_greyhole.c
    ln -s ${GREYHOLE_INSTALL_DIR}/samba-module/vfs_greyhole-samba-4.3.c source3/modules/vfs_greyhole.c
    make -j
cd ..
cp samba-4.3.0/bin/default/source3/modules/libvfs_module_greyhole.so "${GREYHOLE_INSTALL_DIR}/samba-module/bin/4.3/greyhole-`uname -i`.so"
```


Very welcome! And now to make my app code... three characters shorter! WOOO!

On Thu, Sep 17, 2015 at 8:09 AM, Matt Patterson <notifications@github.com>
wrote:

> bump version
>
> So we can release @pobocks' params key handling changes
>
> Thanks, Dave!
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/fidothe/saxon-xslt/commit/527180bb1bbd5dc32fa48e8f93c4a7b26aadd9dc
>


Checkmarx scan results 17 Sep 2015 15:43 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
5
11
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

90003e56d4511b9e1f1b097c10578cb41f77ca30	@CxTest


@jmchilton thanks John - btw the `planemo` docs and "how to" were invaluable for getting this done, it's a great resource.


please remove this file from repository


All the crap that the logger puts out for prefixes ruins any cool factor it has, but now that we changed how it displays we can probably change that too


Looooooool. That was *ridiculously* ambitious for that time, you know. Hm hm. Interesting.


**new-cap**: A function with a name starting with an uppercase letter should only be used as a constructor.


**no-undef**: "before" is not defined.


**no-undef**: "ReactDOM" is not defined.


**react/jsx-sort-props**: Props should be sorted alphabetically


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "describe" is not defined.


**no-undef**: "expect" is not defined.


**valid-jsdoc**: Missing JSDoc @returns for function.


**no-undef**: "TestUtils" is not defined.


**no-undef**: "it" is not defined.


**Eslint**: 'return' outside of function


**no-undef**: "sinon" is not defined.


MiscReleaseFixes (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1879 of 1880 tests passed)

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


You could add some description for what the helper function does.


Je savais bien que ce n'était pas encore assez pythonique.
Merci, c'est corrigé!


Checkmarx scan results 17 Sep 2015 16:24 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
5
11
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

90003e56d4511b9e1f1b097c10578cb41f77ca30	@CxTest


Sweet! Could you add in asserts for unexpected HRESULTs? Would make it easier to catch errors of the application just crashes.


I know this happened a while ago but AHAHAHAHAHHAHAHA this still cracks me up


Sorry, I didn't answer your question concerning the selection of a locale.

In my opinion there should be no options to select a language. The peferred language is choosen in the browser-options and transferred as http-header:
"Accept-Language: de-de,de;q=0.8,en-us;q=0.5,en;q=0.3"

In a Java-Servlet-Environment this Locale is set as default

Just
```java
final ResourceBundle bundle = ResourceBundle.getBundle("Messsages")
```
will do the job.


Looks good to me!


@mjwitte The one diff is the EcoRoofOrlando error file text diff.


你要儲存的是`Military.ArrivingDays`


It appears `response` in this case is the TestResponse object created by the test suite after a `post` (but there may be a delegator involved). Changing the test to `expect(response.body).to include "text"` resolves the issue, however I think it's still a bug that the test suite hangs forever. Not sure where the error lies, but .flatten on the array with the TestResponse in it also hangs forever. .flatten(1) returns something like `[{200, "response body"}, <TestResponse#..>...]`. running .flatten(1) again on that returns `[{200, "response body"},{200, "response body"}, <TestResponse#..>...]`, thus the hanging issue in a recursive loop that never ends.


change the Readme file


Checkmarx scan results 17 Sep 2015 16:27 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
5
11
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

90003e56d4511b9e1f1b097c10578cb41f77ca30	@CxTest


:D


Much better naming.


It's not available for successfully completed commands, which is in line with all the other helper methods that the driver provides.   A call to CommandResult#throwOnError will throw an exception if the command failed, and the exception will include the address of the server that the command failed on.


This commit is being reviewed as a part of review issue #3.


This commit is being reviewed as a part of review issue #63.


Ugh, can't make a two-word change without a typo.  Pbbt.


change this in ProgressWheel.java

protected void onDraw(Canvas canvas) {
    super.onDraw(canvas);
    //Draw the inner circle
    canvas.drawArc(innerCircleBounds, 360, 360, false, circlePaint);
    //Draw the rim
    canvas.drawArc(circleBounds, 360, 360, false, rimPaint);
    canvas.drawArc(circleOuterContour, 360, 360, false, contourPaint);
    //canvas.drawArc(circleInnerContour, 360, 360, false, contourPaint);
    //Draw the bar
    if (isSpinning) {
        canvas.drawArc(circleBounds, progress - 90, barLength, false, barPaint);
    } else {
        canvas.drawArc(circleBounds, -90, progress, false, barPaint);
    }
    //Draw the text (attempts to center it horizontally and vertically)
    float textHeight = textPaint.descent() - textPaint.ascent();

    float startingOffset = (textHeight * splitText.length) / 2 - (textHeight / 4);

    for (int i = (splitText.length - 1) ; i >=0  ; i--) {
        float horizontalTextOffset = textPaint.measureText(splitText[i]) / 2;
        canvas.drawText(
                splitText[i],
                this.getWidth() / 2 - horizontalTextOffset,
                this.getHeight() / 2 + startingOffset,
                textPaint);

                startingOffset = startingOffset - textHeight;
    }
    if (isSpinning) {
        scheduleRedraw();
    }
}


How about replacing `set(range(106, 107))` with `{106}`?


There is a (huge) problem here... @caymard @fabiencastan
'loadDescsFromBinFile' assumes that the data saved in the file is of the same type as descriptor. Which, so far, for us is not the same, as we were using float to manage descriptors in the application, and we were able to load the descriptors independently from their storage representation.


I'm OK with this API but consider having a generic function, e.g. `to_binary`, which is an identity function for binaries and uses `list_to_binary` for lists. But I see why you may want to do it this way, so this is just an idea.


**Github**
- Fix your email settings for `git config`, your identity is not reflected in the commit message. The `user.email` should match the email you used to sign-up for GitHub.


Why channel 106? Why are we referencing IDs at all? Is there some more meaningful way we can address these channels?


Thanks for the support fellas...appreciate it..


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

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


After which CMSSW version is `maxSecondsUntilRampdown` supported? I think we need to be more flexible so that users aren't restricted by that cutoff.


**Code Quality/Comments - Nitpick**
Please use the formal docblock syntax with tags, see http://www.phpdoc.org/docs/latest/guides/docblocks.html


Why are we referencing specific organization IDs? Do we care whether these organizations exist?


**Code Quality - Nitpick**
Use descriptive name, I'll suggest something like `UrbanDictionary`


7.4 and greater, I think.  It might have been in 7.3 under a different name.  I don't think it's a restriction, as this should be ignored when running with a lower CMSSW version.  If not, I'll add a check.


fine


I'd vote to test that before merging.


I've got a few fixes, still, too.


That's the start position of the substring, that is, it will return what's after `charset=`.

Not sure about the use of `HttpHeader.getCharset()`, there's no valid `HttpRequestHeader` at this point (I guess the statements could be reordered to create first the `HttpRequestHeader` but then the `Content-Length` would have to be set again). Or, you mean expose a class method?


I was able to completely scan OWASP Benchmark after the changes.


This can be made more compact by replacing this:

```python
return (
    {
        u'users': [{
```

With this:

```python
return ({
    u'users': [{
```


`print`?


Test


These imports fit on one line:

```python
from robottelo.decorators import bz_bug_is_open, data, skip_if_bug_open
```


Moet je dis line aanpassen het klopt niet


Good job on fixing the javadoc comments.

<organization


Checkmarx scan results 17 Sep 2015 16:34 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
0
5
11
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

90003e56d4511b9e1f1b097c10578cb41f77ca30	@CxTest


Please add a header.   Examples are in:
TubeTK/CMake/KWStyle/Headers

Please create a header file in that directory for .qml files, and then please add that header to all qml files.


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 basic_test|__suite_setup__|0|1|0|0
basic_test|_basic/test_create_vm_|0|0|0|1
basic_test|_basic/test_stop_vm_|0|0|0|1
basic_test|_basic/test_start_vm_|0|0|0|1
basic_test|_basic/test_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_volume_|0|0|0|1
basic_test|_basic/test_add_vol_to_stopvm_|0|0|0|1
basic_test|_basic/test_add_volume_reboot_vm_|0|0|0|1
basic_test|_basic/test_add_multi_volumes_|0|0|0|1
basic_test|_basic/test_add_volume_negative_|0|0|0|1
basic_test|_basic/test_crt_temp_from_volume_|0|0|0|1
basic_test|_basic/test_crt_vm_with_volume_|0|0|0|1
basic_test|_basic/test_vm_securitygroup_|0|0|0|1
basic_test|_basic/test_create_iso_vm_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account_|0|0|0|1
basic_test|_account/test_crt_vm_by_normal_account2_|0|0|0|1
basic_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
basic_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
basic_test|_suite_teardown_|0|0|0|1
virtual_router_test|__suite_setup__|0|1|0|0
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
virtual_router_test|_novlan/test_create_vm_|0|0|0|1
virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_vlan/test_create_vm_|0|0|0|1
virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_snat_create_|0|0|0|1
virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
virtual_router_test|_vlan/test_start_vm_|0|0|0|1
virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
virtual_router_test|_volume/test_add_volume_|0|0|0|1
virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
virtual_router_test|_sg/test_ingress_|0|0|0|1
virtual_router_test|_sg/test_egress_|0|0|0|1
virtual_router_test|_sg/test_icmp_|0|0|0|1
virtual_router_test|_sg/test_udp_|0|0|0|1
virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
virtual_router_test|_sg/test_vm_ops_|0|0|0|1
virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
virtual_router_test|_sg/test_detach_l3_|0|0|0|1
virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
virtual_router_test|_pf/test_attach_pf_|0|0|0|1
virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_multi_rules_|0|0|0|1
virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
virtual_router_test|_pf/test_tcp_|0|0|0|1
virtual_router_test|_pf/test_udp_|0|0|0|1
virtual_router_test|_pf/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_2_eips_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
virtual_router_test|_eip/test_attach_eip_|0|0|0|1
virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
virtual_router_test|_eip/test_vm_ops_|0|0|0|1
virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
virtual_router_test|_lb/test_create_lb_|0|0|0|1
virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
virtual_router_test|_host/test_reconnect_host_|0|0|0|1
virtual_router_test|_suite_teardown_|0|0|0|1
local_ps_virtual_router_test|__suite_setup__|0|1|0|0
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_ps_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_ps_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_ps_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_ps_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_ps_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_ps_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_udp_|0|0|0|1
local_ps_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_ps_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_ps_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_ps_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_ps_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_ps_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_ps_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_ps_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_udp_|0|0|0|1
local_ps_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_ps_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_ps_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_ps_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_ps_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_ps_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_ps_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_ps_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_ps_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_ps_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_ps_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_ps_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_ps_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_ps_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_ps_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_ps_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_ps_virtual_router_test|_suite_teardown_|0|0|0|1
local_nfs_virtual_router_test|__suite_setup__|0|1|0|0
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_account/test_vm_ops_by_normal_account_|0|0|0|1
local_nfs_virtual_router_test|_account/test_multi_accounts_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_novlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_snat_create_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_dnat_reboot_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_ping_vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_create_3vms_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_3l3s_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_remove_dhcp_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_2l3_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.0_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vms_with_limited_ips2.1_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_start_vms_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_crt_vm_with_hostname_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_reboot_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_chg_static_ip_vm_|0|0|0|1
local_nfs_virtual_router_test|_vlan/test_attach_nic_with_static_ip_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_m_vols_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_add_volume_vr_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_volume_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_2_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_24_volumes_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_volume_ops_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_volume/test_crt_vm_from_dvt_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_crt_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_template_image/test_del_temp_image_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_icmp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_egress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_2vm_ingress_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_4vm_complx_|0|0|0|1
local_nfs_virtual_router_test|_sg/test_detach_l3_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_2l3s_pfs_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_attach_pf_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_delete_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_del_vip_before_add_pf_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_pfs_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_multi_rules_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_after_destroy_vm_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pf_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_udp_same_vip_ports_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_pfs_tcp_with_same_vip_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_tcp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_udp_|0|0|0|1
local_nfs_virtual_router_test|_pf/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eip_pf_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_2l3s_eips_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_attach_eip_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_create_eip_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_del_vip_before_add_eip_to_vm_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_eip_sg_tcp_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_pf_eip_attach_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops_|0|0|0|1
local_nfs_virtual_router_test|_eip/test_vm_ops2_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_alone_vr_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_source_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_create_lb_|0|0|0|1
local_nfs_virtual_router_test|_lb/test_lb_health_check_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_sp_after_delete_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_create_image_by_snapshot_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_for_both_root_..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm2_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm3_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_delete_snapshot_on_ru..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_in_live_vm_|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_crt_sp_from_vol_crt_f..._|0|0|0|1
local_nfs_virtual_router_test|_snapshot/test_change_max_sp_depth_l..._|0|0|0|1
local_nfs_virtual_router_test|_regression/delete_sg_with_2_attache..._|0|0|0|1
local_nfs_virtual_router_test|_regression/rm_nic_sg_on_stopped_vm_|0|0|0|1
local_nfs_virtual_router_test|_regression/test_data_volume_with_vm..._|0|0|0|1
local_nfs_virtual_router_test|_vr/test_ping_vms_reboot_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_reconnect_vr_|0|0|0|1
local_nfs_virtual_router_test|_vr/test_start_vrs_simultaneously_|0|0|0|1
local_nfs_virtual_router_test|_host/test_reconnect_host_|0|0|0|1
local_nfs_virtual_router_test|_suite_teardown_|0|0|0|1
Total|304|0|4|0|300


**Functionality - Awesome**
Did a good thing checking for duplicates.


Does it make sense to ignore non-zero return codes? When I call a function, I expect it to return only upon success.


If you're going to rebase you probably want to change the message ;-) Please don't rebase...


CharFields have a max length of 255.


Here I changed the meaning of some functions.

In layout.py everything that starts by *update_* means that none of robot parameters will be changed. It is just the display refresh.
And if after some changes (e.g. *sigma* becomes 2) robot parameters must be changed, *robot.reset_* is called.


Good catch. I too am stoked about Angular-Meteor.


Darf nicht übersetzt werden.


Looks good!


Please indent. This will trigger a parsing error by Sphinx.


Darf nicht übersetzt werden


```c++
                {
                    coil = me->FindNearestGameObject(GO_CONS_NOX_TESLA_FEUGEN, 1000.0f);
                    if (coil)
                         _myCoilGO = coil->GetGUID();
                }
```

can be replaced by
```c++
                    if (coil = me->FindNearestGameObject(GO_CONS_NOX_TESLA_FEUGEN, 1000.0f))
                         _myCoilGO = coil->GetGUID();
```


Hi @jjahns  - can you create a Pull Request for this code so that we can begin evaluating this?


OK


This reintroduces bug #10582 which was solved by NOT creating a temporary file. All my `git.latest` states suddenly stopped working again...


MiscReleaseFixes (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1873 of 1874 tests passed)

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


Does it make sense to ignore non-zero return codes? How can the user differentiate between "no transition data" and "an error was encountered when generating transition data"?


I don't see the problem here. What do you mean with "were using float to manage descriptors in the application" ? The `cameraLocalization` application ?


Nitpick: drop this line. The whitespace inside the docstring provides visual spacing.


Should ir return EditReleaseResponseModel or EdirReleaseResponse?


Nitpick: drop this line. It makes no difference to Sphinx, and IMO it's nice to have argument descriptions packed together.


Ideally `.gitignore` should only contain stuff that is shared by everyone, ie is directly related to the codebase. Environment specific things should be excluded in your local `.git` project (`.git/info/exclude`). With IDEA based IDEs you can manage this via the VCS [ignored files](https://www.jetbrains.com/idea/help/configuring-ignored-files.html) list.


I think you can from the size of the file and from a header read load the appropriate descriptor and convert it on the fly...


beautiful


Wow. Great idea!


Hi @tortuetorche,
sorry for the inconvenience, add this definition to fix the problem:

    #ifndef _WIN32_WINNT_WIN8
        #define _WIN32_WINNT_WIN8 0x0602
    #endif


Thanks


Das ist ein *anderer* Studiengang. Ist ein internationaler Master und nicht "Computerlinguistik", die jeweils eine eigene PO hat


Darf nicht übersetzt werden


This empty line was indeed to separate `r10` to `r11`. The reason is the end of `argument passing registers`.


@do3cc Yeah. Looks good. Thanks for working on this!


Darf nicht übersetzt werden, da "Wirtschaftsinformatik" feststehender Begriff ist


Don't know, if it's OK.


It probably would I didn't even think to use it....smh.


Is it necessary to return string like this one? Does anything bases its behaviour on this?


**Functionality - Critical**
This `Urban` is not the `Urban` in `Urban.php` — which is has a `static property $data`, that's the one you should use, and it doen't require the `new` key word to use it.


'confirm' for a removal?


this should be a `private static final int DEFAULT_INDICATOR_NUMBER`. Be sure to respect the Android naming convention which are used in this project.


If you create a pull request, I'll review it :)


No. It's legacy code which I had initially removed and then restored.


legacy :poop:


Good catch


@fernando-rodriguez  I shall acquire and attempt a testrun of these.
The bumped ebuild has a few minor format issues
Some lines are ridiculously long but some are unavoidable. e.g. DRIVERS_URI, XVBA_SDK_URI are 92 chars long. They could be avoided by making a var such as
VER="linux-installer-15.201.1151-x86.x86_64.zip"
then use ${VER} but it adds as much as it reduces.
lines such as line 157 can be reduced with
(/t)local CONFIG_CHECK="~MTRR ~!DRM ACPI PCI_MSI !LOCKDEP \
(/t/t)!PAX_KERNEXEC_PLUGIN_METHOD_OR"
but line 194 (MODULE_NAMES) again requires setting vars and re-writing it to reduce it.
The value is questionable.
However BUILD_PARAMS (lines 200, 202) can and should be broken up with use of '\'
ditto the test of ewarn in lines 597 - 607.

Remember this is merely formatting, no reflection on the content.


Do we care whether these organizations exist or not?


LGTM :+1:


Shouldn't it be a CDI bean?


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1340 of 1340 tests passed)

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


That won't work unless preprocessor is case insensitive. It should be ```TEXT_REPLY```


Remember the `a` tag needs a `href` to be valid, in this case a `#` will suffice.  It is possible that you experimented with it but found the page to jump to the top when the click fired.  The reason for this is that the default action of a hyperlink is to try to go to that link, a `#` is a bookmark in the current document and the `#` is the very beginning by omission of an id.

I'll address the fix as a comment in the JavaScript.


Possibly (probably) will conflict with our custom java cookbook, but we should rename ours, sooo.... LGTM :+1:


Shouldn't this condition be the opposite?


Yes, no error with the extra %s


LGTM :shipit:


@vpavlin, looks much better, isn't it?


that's what our old function was doing...


**Eslint**: 'return' outside of function


**no-undef**: "describe" is not defined.


**no-undef**: "TestUtils" is not defined.


**no-undef**: "expect" is not defined.


**react/jsx-sort-props**: Props should be sorted alphabetically


**no-undef**: "before" is not defined.


**no-undef**: "sinon" is not defined.


**no-unused-expressions**: Expected an assignment or function call and instead saw an expression.


**no-undef**: "ReactDOM" is not defined.


**valid-jsdoc**: Missing JSDoc @returns for function.


**no-undef**: "index" is not defined.


**no-unused-vars**: inputText is defined but never used


**no-undef**: "it" is not defined.


**new-cap**: A function with a name starting with an uppercase letter should only be used as a constructor.


I'll fix it. Thx


**Code Quality - Critical**
Use `autoloading` and `namespaces`. [This article](https://mattstauffer.co/blog/a-brief-introduction-to-php-namespacing) might be useful.

At the end of the day, your classes should be able to coexist with anyone's code, even if they have the same class names as yours.

As it is, your source-code cannot be packaged and distributed through a package manager like composer, etc.

Then, for your `require`s, use _absolute_ paths as opposed to _relative_ paths. See the use of `__DIR__`


LGTM :shipit:


If you would like to suppress the jshint error about console not being defined in Brackets (or alternative) you can include it in the special comment at the top i.e.

    /*global document, $, console */

It instructs the jshint program that know what you are doing and you expect that a console will be available.


I would recommend taking a look at the vmware modules recently created by @jcpowermac and @mtnbikenc -- In particular, it would be nice to remove the dependency on vconnector and to reuse the wait_for_task, connect_to_api, and get_all_objs functions inside of module_utils/vmware.py.


Good pick abstracting variables out and placing their definition at the beginning of the function.


Yea, but may make sense with others in the future if we add some, when we can't go back to the flag. I prefer to keep it like this.


Thanks for catching that!


Died*            You lost 1 Badge*


Done


Badges*


You're welcome, and thanks for update! :+1:


This looks like we should maybe continue using our forked modified version.


No. Loop should start from 0 but give incremented i by one as a page number


Uh, no, I can't tell 0 from 8.
```
src/irclog2html/irclogsearch     164      8    95%
```
:(


Extracting this line into a `Flow#start_node` method and using that from `SmartAnswerPresenter#start_node` might be a way to reduce the duplication slightly.


If no one else does, I do care :)
The `org_ids` refer to the ids of the organizations exported from Satellite 5. We are talking about importing the existing orgs from Sat5 to Sat6.
The data we use have some specific default values inside (like org_id 1-3). The `hammer import` script looks at the ID's at it has to map these (sat5) ids to sat6 ones.
Sat5 ids are needed as they are being used all across the dataset (e.g. to distinguish which user  belongs to which org, ...)
The truth is, we don't necessarily need to randomize ids, but IMHO it is a good touch as we can support testing of e.g. big numbers as org ids.


oops


**oops**


see above. The IDs are hardcoded in the default dataset we use.


**Code Quality - Nitpick**
The lines are too long, remember [the recommended max length is 80 characters](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md#23-lines)


Maybe it would be nice to change `list` variable name?


I mean it shouldn't do copytruncate... it should just do rotate.  We've fixed the HUP issues in 3.0.10.


legacy code :p but yeah, would be a good idea


it should work on Windows


**Code Quality - Nitpick**
Fix the alignment.


:+1: I spent A LOT of time figuring this out ...with no luck


Thanks Broda!


Can we have unit tests for that?


Good point! Thanx. Fixed with a check_boolean() function that can take case combination of yes OR 1.


Hey @jkeck , I added a similar `druid` method to find and extract druids from purl links. That works ok, but now `file_id` is returning an array of values. Seems a bit strange to me. Any quick thoughts on why that may be?

```ruby
def druid(link)
  link[:href].gsub(%r{^https?:\/\/purl.stanford.edu\/?}, '') if link[:href] =~ /purl.stanford.edu/
end
```


@soulfrost Good job ignoring my statement about compiler warning.


When an event fires such as a click event the information associated with it is packaged up and passed as an argument to the function.  As developers we can chose to use it or not as required.

The arguments comes in between the parenthesis and are comma separated.  It is common to use `e` for as short for event.

e.g.

    function readMore(e) {
        /* The following piece of code prevents the default action, in the case of a hyperlink with a # jumping to the top of the page.  In the case of a form it would stop the form being submitted */
        e.preventDefault();

        /* The rest of your code as before */
    }

The same principal applies to the readLess function.


Sorry, need to master Search&Replace with regex. Will remove.


np ;p

On 17 September 2015 at 19:26, Pranav Reddy <notifications@github.com>
wrote:

> Thanks Broda!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Radium-Devices/device_yu_lettuce/commit/094d1e5d31038630489b0949b153e1a1e5fcbe8d#commitcomment-13287881>
> .
>


The fix isnt complete, sos_reports directory still empty - working on a new patch.


trying to use ```font-weight: light```?


+DAEMON1_OPTS="-a user:password -p 11000 -b "


@tuexen  i will! :+1:

@micchie You've applied only the first change, what's about the second?

Cheers
Felix


This will break if we shorten `org_ids` or if we change this to `range(4)`. This will make the code more robust:

```python
            } for i in len(org_ids)]},
```


**Code Quality - Awesome**
I've observed that, except for a few mistakes I've noted in the other comment (line size), you've followed the PSR-2 code-style excellently! Keep it up!


I think you don't have to convert `email` to `set` explicitly. It is already done inside make_email.


Why not remove them later then? Until we remove `events.EventEmitter` (which may never even happen, it is the more used API afaik), we benefit from having the test coverage. There isn't anything wrong with it, it isn't even deprecated - we use plenty of deprecated things in tests.


It is, but with the name `to_list` I think it's a bit ugly to pass a plain string.


Overall very good job.  It may seem a bit harsh mark wise with the rubric but if you want to add in the fixes (markup and js) to prevent the default action and resubmit I'm happy to bump it up.


I predict that the messages would reach the same function endpoint at the other community, because of the name that is set in the message. Is this correct?

How can I make decide if a message received is from a own signature request or from the crawler?


Yes it's a bit inconvenient. Maybe we should look into configuring the automatic code formatting of our IDE's all the same way.


Burde vi heller flytte filen (og rename den f eks med timestamp i navnet)?


I don't think you need the fallback here i.e. I think you can just delegate to the presenter.

`StartNodePresenter#title` calls `super` (in  `NodePresenter#title`) if no title is available from ERB and then `NodePresenter#title` should fall back to `@node.name.to_s.humanize` which I think is the same as `@flow.name.to_s.humanize`, because the start node's name is set to `@flow.name.underscore.to_sym`. But you should definitely check whether I've got that right!


das hatte ich noch vergessen - hier ist `lock -u /tmp/VPN_LOCK` besser - sonst hängen diverse lock prozesse herum, weiß auch nicht genau wie das funktioniert.


Why do you need the `if start_node.has_meta_description?` bit?


This is an invasive change that came as a dependency with ... something.  It's a big change, but I'd prefer to use the latest and adapt old broken stuff than stay stuck in the past.


30 is a bit much but the app allows more than just 12, it also then recalulates when it gets the next value drawing a straight line between the two points and finding where the bg may have been at the time of calibration (so it should be... closeish)

Anyhow it was throwing out a few lenghty sensor sessions of mine because of it and I wanted to use them for testing :-D


Here is the source to the tool, I am going to compile it now...

http://pkgs.fedoraproject.org/cgit/glibc.git/tree/build-locale-archive.c


das verhält sich bei mir komisch, daher der logger - muss das nochmal testen, aber ich habe mit `_log do` keine ausgabe bekommen... hab grad keine zeit, schau mir das mal nächste woche an.


Is it possible to  have many Principals with the same `email`?


Ha!  And a later re-vendoring nullifies this commit, so whatever.


No. But I'm using principal objects as a dict key in the zodb import script so it needs to be hashable


There is a weird status at this point in time. Some b2g jobs are running on Buildbot and some on TaskCluster.
For the moment, we can only deal with the ones on the Buildbot side.


MiscReleaseFixes (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (806 of 806 tests passed)

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


Should be `getTitle()`, right?


I think most of this diff can be attributed to `chefignore`


Yeah, I agree. I will bring it back after the first release when it becomes stable.


Yup, see it now in libressl-portable/openbsd


> Just saw this. This is NOT ok for me and has to be immediately reverted

I am ok with everything not producing crash by default.

We know that the situation is not good, the python developers did not really
care about usage in libraries. I certainly do not like the memleaks.

The problem is that you have no way to repair the broken API. The "global"
mutex and counter are only local per dlopen'ed instance of the plugin. They
are useless in all scenarios where anybody uses python afterwards or in
between.

Luckily, lua seems to be much better in that regard.


I wonder if we added this ...


Ah, no it's been moved to the rmagick recipe (??)


Jak muze hlasit file not found? Ten skript se nespousti, ale sourcuje. Executable vubec nemusi byt, tak jako nejsou executable skripty v /etc/profile.d/ .


Okay, this is interesting...

The tool uses a locale-archive "template", namely `locale-archive.tmpl`, as assumed from the error message.

```
./build-locale-archive: cannot open locale archive template file "/usr/lib/locale/locale-archive.tmpl": No such file or directory
```

And the information on the template itself.

```
locale-archive.tmpl: locale archive 8117 strings
```

Which suggests an archive with **everything** in it.


This is just a basic idea of how it will work. It should load scripts fine but the way this is done may be changed in the near future.


what about g=0? 0^totient = 0. The simplified statement (with ==1) holds only for g coprime to the modulus.


> If no one else does, I do care :)

:wink:

> The truth is, we don't necessarily need to randomize ids, but IMHO it is a good touch as we can support testing of e.g. big numbers as org ids.

OK. Is it correct to say that the referenced organizations don't actually need to exist?


Gotcha.


cc: @nikolai-s @AlexanderBelonogov


FYI the utility is from the package (RedHat standard) `glibc-common`, since version 2.9.


That's all well and fine, but you need to actually look at and comprehend the change set.  Otherwise mystery code gets merged, devs get confused and users get an unstable app.


Should be just 'developed'


1: M111 unrecognized component name: kwalitee


hmmm, let me check on that.


test


cuida com esse `;` sobrando no meio do nada.


typo?


@akhilnarang My fellow YU users will start saying that U didnt build, then why did u post! :V


Is better if the messages are in the same order that the input forms(the password error the last)


Maybe is the moment to delete this message


If the tests pass, then it's possible as this doesn't add functionality.


Hi. This looks like it removes possibility to send a solicited NA. I'm wondering if you could add a function that sets or unsets the flag instead?


It seems that this is already included (in line 6356).


Additional discussion...might want to be more 'public' about the effects of this, since a lot of warnings are now getting thrown

https://groups.google.com/forum/#!topic/julia-dev/2Y4uglhJW08


whoops.


@thenewguy Hi, can you explain this change to me? It's breaking things for us - I'd like to understand it a bit more.


I'm just saying, WIN32 isn't defined on Linux, so that macro was never in play.  What features, "finally turned on," for you?  Maybe you coincidentally caught a driver update.  I've had fan speeds and temps on Linux since I rebased some old PR a couple weeks ago.  I haven't looked into what else NVML is used for.


@JulioCROliveira, sua implementação me preocupa por um motivo: você está fazendo coisas a mais do que está sendo solicitado. Não temos que fazer Armas por enquanto. Você acabou se preocupando demais com o que você inventou e acabou não testando de fato o que foi feito até então em aula.

Cuida para isso não te atrapalhar, sugiro re-escrever todos os testes e apagar a classe Arma.


Ok, agreed.  Will updated the PR.


We can expose `SwingHighs` publicly as an `IReadOnlyWindow<decimal>` so the user can access properties like `IsReady` and `Count`


We can do the same with `SwingLows`


formatting


@enricogior Thank you for this quick response.
I added your definition before line 196 and it works :+1:!
By the way, during compilation I've multiple warnings like these:
`Warning	23	warning C4244: 'function' : conversion from 'PORT_ULONGLONG' to 'size_t', possible loss of data	C:\Users\user\Documents\redis-2.8\src\redis-cli.c	1363	1	RedisCli` or `Warning	2	warning C4244: '=' : conversion from 'PORT_LONGLONG' to 'PORT_LONG', possible loss of data	C:\Users\user\Documents\redis-2.8\deps\hiredis\hiredis.c	408	1	hiredis`


oops sorry


I think we want to zero the other properties as well, `CurrentHigh` `CurrentLow` `CurrentSwingHigh` `CurrentSwingLow`


Actually I have to say srry! :p


Wollen wir nicht mal "hihihi" umsetzen?


prev_rtr_addr is kept for all iteration therefore the RTR will be counted one time only.


Unit test added.


Err… so this seems to currently set crossfade to 1 second, which is suboptimal. Maybe this should have been implemented with a slider.


`Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>`
`Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)`


gespeichert?


Hey, you probably should just pass `base_url` as a constructor parameter, like that:

    API('app_id', base_url='https://api.worldoftanks.eu/wot/')


@joewan  Yeah, add constness is good. Could you send a PR?


`new` would suffice


@joewan  If we don't add this line, the title position is wrong. So this is for fixing the title position.


Thank you for your feedback!

:+1:

@k4nar ?


I added back .env and public/uploads to the git ignore file. and committed to master.


klol :v


这个问题还是很简单的


Guter Commit


@tortuetorche
those warning are annoying but they are ok.
The original redis code (written on Linux) doesn't have explicit type casting and in Windows by default those type conversion are treated as warnings.


The button shouldn't to allow do the signUp if there are erros in the form.


bosen新加的内容，统一加前缀，而不是后缀


:V


Yep... they still don't like it.


Can you combine this? Check whether either response or status or both are nil.


I don't see any problems just looking at the changed function, but I will need to test the code to be sure (cannot do so until tomorrow at the earliest). I do see some potential optimizations I can make, though.


I updated `xdecor` : it can now cut nodes from any mod just in specifying these in [that table](https://github.com/kilbith/xdecor/blob/master/worktable.lua#L4). That's a simpler way for you.


Same version.  Iiinteresting.  Ought to be fun.


Luckily you had a family who seemed to have some inside information, *guessing* the exact date, weight and length, otherwise there would be a group needed I guess...


Should be private static final double multiplier  = 1.05; <layout


It would be great if you created an issue when you found this bug


MiscReleaseFixes (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1879 of 1880 tests passed)

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


filter_buildernames() only returns buildernames that are on allthethings.json, which means only buildbot jobs, so there is no risk of it returning a Taskcluster job, you don't need extra filters.


alle Punkte hier abgearbeitet und neu commited


OK


Yeah, you're right. It's just that I kept on changing the assertEquals expected value to try if the test failed when it should.


This was done mainly for https://github.com/c9/core/issues/147

We have pty.js and node installed in $C9_DIR, which are used for running vfs worker.
But for main cloud9 server people usually use global node by running `node server.js`  instead of `~/.c9/node/bin/node server.js`
From modules installed in sdk folder only ws have binary component, but it is optional so server.js can work both with v0.10 and v0.12 no matter which is used for npm install.
Unfortunately with node 4 binary component compiled for old version does not simply throw an error, but crashes whole process with segfault.

this should work with any node version unless global npm is broken


Hi Sven,

The function ```bp_avatar_create_item_avatar()``` is expecting a path and you're sending an url, that's why file_exists fails.
I advise you to use this [function](https://gist.github.com/imath/37ca3c2c650e82beb95b), so that you'll get the path to the thumbnail or the path to the original image if no thumbnail size is available on the blog.


I think this was the version that fixes #1


@mpj Sure.  It allows you to use "--host=0.0.0.0" to start the webpack-dev-server.  In this case it overrides the hostname that socketio uses to connect.  Since it uses a equality check for '0.0.0.0' it only overrides the hostname when the value is '0.0.0.0'.  Since socketio cannot use 0.0.0.0 to connect, we replace the provided hostname (of 0.0.0.0) with the hostname value from the browser (this is the hostname that was used to view the page).

This means that you can view http://localhost:2992, http://127.0.0.1:2992, or the machines network address... say http://192.168.1.22:2992.  Because of this you can test locally or using your cell phone/tablet over the network.

See https://github.com/thenewguy/react-starter/blob/3e65236d5963eda6c00282825f82df83db5308c4/package.json for example arguments.

Are you sure it was this specific commit that breaks your environment?  It only makes a change to the hostname when the configuration that would not work anyways (you can't connect to 0.0.0.0 because it a wildcard address used to configure the server to listen to attached addresses/network adapters.


Nice. There may be an existing issue about it that should be closed.


Inequality sign is weird..
> 6.8.9 is right.


This really should get a friendlier display name for the plugin manager.

https://groups.google.com/forum/#!msg/jenkinsci-dev/-p_ITQSa1YM/vfyp_VnbAwAJ


:+1: nice!


Ah, I think I see your point now.
First, let me give you more insight, so we're on the same page:

The dictionaries generated in these gen_* methods represent __changes__, not the actual dataset that we will pass to the `Import` command.
So they are first being passed as an argument to the `update_csv_values()` method.
This method will take and load the actual dataset file and update the dataset by the changes accordingly (by looking at `key` and `key_id` to know which record(s) to update). After the changes are done, it spits out a new CSV file.
This file will be used in the tests.

- so, in this particular case, the `org_id` (1..1000) __will__ exist as long as the `org_id` with id specified by `key_id` exists in the original sample file. We use the same set of file over and over. They contain org_ids 1..3.

The absolutely correct approach would be to parse the actual org_ids from the original sample file (as the ids might be different if we used another sample). However, the file is being loaded in the `SetUpClass` method so it can't be reached by the gen_* methods.
All the magic starts to happen in the test unit.
It would be possible to hack it somehow, but I believe it is out of scope.  Also it is of really little relevance to have randomized org ids.
So I think the effort wouldn't be worth it.


We can remove `org.wordpress.android.datasets.SuggestionTable` from the import section now that this has been dropped.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1343 of 1343 tests passed)

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


Ah, found out i actually removed that print in the other commit, so it is no longer there.